standardrb: init at 1.35.1

Description of changes
standard gem is used for linting/formatting ruby code
there is a pr (215167) for this gem, but it has merge conflicts and it looks like the author is not going to fix it
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@3JlOy-PYCCKUi, any updates on this?

Any news on this?

there is a pr (215167) for this gem, but it has merge conflicts and it looks like the author is not going to fix it

@3JlOy-PYCCKUi I fixed the merge conflict in #215167 now.
Note that one doesn't get notified when the "merge conflict" label is added to one of their PRs.
Also, the PR author(s) don't get notified if you omit the # when mentioning a PR.

if we want to merge this directly to master, we must remove the other updates
We mostly want to stay on 1.X.X for now

@SuperSandro2000 these remarks have been addressed in the original PR #215167 (which was created more than one year prior to this very similar PR).


Update App.js / optimize filter setVariables

We have quite large styleguide and seems that setVariables filter have some performance problems.
Our variables arrays has almost 300 items and  using _.memoize we can reduce cpu 100% usage from ~65s -> ~2.5s.

fixing test, then ill make new one


Add Terms and Definitions and add entry for Reference Plane

There are two definitions that express positions that happen to be on the optical axis. One is the position of the lens entrance pupil. One is the position of the intersection of the optical axis and the object on which the camera has been focused. In both cases, an exact definition of the place on the optical axis from which those two positions are measured is needed.
This definition (from the Terms and Definitions of SMPTE RDD 55, "Material Exchange Format — Carriage of ARRI Camera System Metadata", p. 5) uses the term 'Reference Plane'. We could keep 'Reference Plane' or we could just as well change it to 'Image Plane'; a possible reason to keep it would be to leave 'image plane' as the plane on which a pinhole camera would have formed its image.

If the group thought it would be less confusing, one could add to that 'such as' phrase the typical IR filter. I am not sure whether the original author of this definition meant 'sensor' to include or exclude any microlens array or color filter array material but will ask.

Speaking with our optics group, they suggest a different definition that avoids mention of any hardware between the lens and the "photosensitive material", whether that be electronic or photochemical.
There is no conflict per se with the RDD 55 definition above.
Here is what I was given:

"Nominal Image Plane
The nominal image plane is an imaginary plane, which represents the expected location of the camera's image plane, given the location of the mount flange plane and the specified nominal mount flange focal distance. For LPL mount, the nominal image plane is offset by 44mm from the mount flange plane (see flange focal distance).
Since the nominal image plane is often used as the origin when specifying points in space (e.g. object distance, entrance pupil position) its location is indicated on a camera body."

Given that the Cooke Optics' "Camera and lens definitions for VFX" document is the only one of its kind, as far as I know, I suggest we stay closer to its terminology than the terminology used in RDD 55 (to read which you must part with a cool $240). Now, the Cooke document sometimes says "image plane" (2X on p. 5, 2X on p. 6, 2X on p. 7, 3X on p. 9, 3X on p. 11, 1X on p.12), and sometimes says "nominal image plane" (4X on p. 11, 1X on p. 12). That last one is important, though: that is the definitions section:

I suggest we add a Terms and Definitions section to camera-lens-tables.md, and that as a start, it only have one definition: Nominal Image Plane. I suggest it read thusly:

The nominal image plane is an imaginary plane, which represents the expected location of the camera's image plane, given the location of the mount flange plane and the specified nominal mount flange focal distance.

Though one could jump straight into adding definitions for Mount Flange Plane and Nominal Mount Flange Focal Distance, I suggest we do not, until pressed for them.
Once the definition for Nominal Image Plane exists, we can (e.g.) define the location of the object in focus and of the entrance pupil as offsets from the intersection of the optical axis and the nominal image plane; but altering the existing definitions (and the name of the defined thing, if need be) is something for a separate Issue.


Wip attach

@Newman-Hui I do not feel this PR will be accepted by the maintainers of AstroMatt/atlassian-python-api.

It is possible, but it would not be maintainable. We would not be able to take any new features the AstroMatt is developing etc. If we cannot write this code as something useful for the library then IMHO the best solution is to put it in the other repo, and use it from there. In the other repository you can still access the _session from the confluence object. That is not a good solution either but for me is better than having our own atlassian-python-api repository to maintain for those two functions.

He sent me this PR when i mentioned what i did
https://github.com/AstroMatt/atlassian-python-api/pull/65/files

@svyotov correct, they (AstroMatt) made a commit addressing the bug i was running into, this is no longer needed


Hidden GitHub App

For a first implementation of #430 we want to implement a hidden App auth feature. When available  cla-assistant should use the GitHub App Installation token to interact with GitHub, if not available it should fall back to the "admin user" token like currently.
For this following steps should be implemented:

[x] Identify all places where GitHub App installation token should be used (incl. GraphQL calls)
[x] Write a wrapper for github.call
[x] Write a wrapper for github.callGraphql
[x] Replace all calls from task 1 with wrapper-calls.
[x] Inside the wrapper add logic to identify the associated installation based on org/owner/username
[x] Inside the wrapper replace call replace the passed argument call.token with the installation token. If no installation exists keep it unmodified.

Images from our whiteboard session about that:

done via #781


[SR-16010] Swift 5.6 iOS backdeploy concurrency library has incorrect minimum deployment target

Previous ID
SR-16010

Radar
None

Original Reporter
@keith

Type
Bug

Additional Detail from JIRA

Votes
0

Component/s

Labels
Bug

Assignee
None

Priority
Medium

md5: a791795626bd53465e60639873a8d8ef

Issue Description:
Previously:
% vtool -show /Applications/Xcode-13.2.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/iphoneos/libswift_Concurrency.dylib
/Applications/Xcode-13.2.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/iphoneos/libswift_Concurrency.dylib (architecture arm64):
Load command 10
      cmd LC_BUILD_VERSION
  cmdsize 32
 platform IOS
    minos 13.0
      sdk 15.2
   ntools 1
     tool LD
  version 711.0
Load command 11
      cmd LC_SOURCE_VERSION
  cmdsize 16
  version 13<IP_ADDRESS>

With Swift 5.6:
% vtool -show /Applications/Xcode-13.3.0-RC1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/iphoneos/libswift_Concurrency.dylib
/Applications/Xcode-13.3.0-RC1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift-5.5/iphoneos/libswift_Concurrency.dylib:
Load command 9
      cmd LC_VERSION_MIN_IPHONEOS
  cmdsize 16
  version 7.0
      sdk 15.2
Load command 10
      cmd LC_SOURCE_VERSION
  cmdsize 16
  version 13<IP_ADDRESS>

This has the side effect of swift-stdlib-tool over-including swift libraries in apps, because the 7.0 targeting version references<EMAIL_ADDRESS>instead of `/usr/lib/libswiftCore.dylib`. I imagine this could also have other downsides like people accidentally shipping this with apps targeting lower than 13, but maybe something else would catch that case.
It's unclear to me exactly how I should build Swift to directly reproduce this issue, but form the release/5.6 branch building the `source_compat_suite_macos_base` preset (chosen just because it builds these targets) I get libraries with the correct 13 minimum deployment target, so it must be that a different build configuration is used for the releases. The most recent snapshot from the the 5.6 branch from 03/02 doesn't have this issue either. Nor does the most recent main snapshot from 03/13.
This might be better suited as an apple bug, but I'm not sure where the relevant configuration for this lives.

Still the case with Xcode 13.3.1

Still the case in the Xcode 14 betas. I blindly submitted https://github.com/apple/swift/pull/60118 hoping that might help

Ok so I learned that the 7.0 deployment target is correct. The purpose is to allow people to include the back deployed library for any version of the OS that supports Swift at all, to allow you to use concurrency still only wrapped in available conditions for iOS 13+. If the deployment target was 13+ it would crash on launch(?) instead.
So really the problem here comes back to swift-stdlib-tool which only uses the load commands to decide what to do, instead of having any knowledge of the specific deployment target limitations. To fix it we should probably encode that info there.


v1.11: HTTP Binding: Document mtlsRenegotiation metadata option

Please document mtlsRenegotiation metadata option (and allowed values) for the HTTP Binding. This must be done in the v1.11 branch of the docs.
PR Author: @mdarii
PR: https://github.com/dapr/components-contrib/pull/2822
Should be done here:
https://github.com/dapr/docs/blob/v1.11/daprdocs/content/en/reference/components-reference/supported-bindings/http.md

/assign


Paginate Pipe is incompatible with property getters/setters

Angular 2 version:
2.4.3
ng2-pagination version:
2.0.0
Description of issue:
The paginate pipe fails on PaginationInstance implementations that use property getters/setters. paginate.pipe.ts: 97 checks config.hasOwnProperty(prop), which will return false for accessor properties, since they're defined on the object's prototype. Using the 'in' operator (prop in config) instead would resolve the issue.
Steps to reproduce:
// test.component.ts
class PaginationImpl implements PaginationInstance {
  get currentPage { ... }
  set currentPage { ... }
  ( ... )
}

<!-- test.component.html -->
<div *ngFor="let item of collection | paginate: paginationImpl"></div>

Expected result:
Paginated collection
Actual result:
PaginatePipe: Argument is missing the following required properties: 'currentPage'
Demo:
http://plnkr.co/edit/QJILAnQiR3AqS3fK2Mq9?p=preview
Any relevant code:
// paginate-pipe.ts: 97
const missing = required.filter(prop => !config.hasOwnProperty(prop));
// Possible replacement
const missing = required.filter(prop => !prop in config);

Add pull request: #129

Added pull request: #130


ed25519 is not supported by this version of OpenSSL

ed25519 is not supported by this version of OpenSSL in OpenSSL 1.1.1f
My docker container is
lsb_release -a
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.1 LTS
Release:	20.04
Codename:	focal
uname -a
Linux raspberrypi 5.4.83-v7+ armv7l armv7l armv7l GNU/Linux
Debug in Python, Python is in virtualenv
Python 3.9.0 (default, Oct 28 2020, 23:50:03)
[GCC 7.4.0] on linux
from cryptography.hazmat.backends import default_backend
default_backend().openssl_version_text()
'OpenSSL 1.1.1f 31 Mar 2020'
So, I have version 1.1.1f
when I try
from cryptography.hazmat.primitives.asymmetric.ed25519 import Ed25519PrivateKey
Ed25519PrivateKey.generate()
Traceback (most recent call last):
File "", line 1, in
File "/home/nick252/homeassistant/lib/python3.9/site-packages/cryptography/hazmat/primitives/asymmetric/ed25519.py", line 52, in generate
raise UnsupportedAlgorithm(
cryptography.exceptions.UnsupportedAlgorithm: ed25519 is not supported by this version of OpenSSL.
===============================================================================================
More info in https://github.com/home-assistant/core/issues/45363

I have same issue like nick2525
Use python 3.87 and OpenSSL 1.1.1i in virtualenv
OS Linux bananapim2plus 4.14.78-sunxi armv7l (Armbian Stretch)

I have same issue like nick2525
Use python 3.87 and OpenSSL 1.1.1i in virtualenv
OS Linux bananapim2plus 4.14.78-sunxi armv7l (Armbian Stretch)

Looks like this question would be better asked of the python guys:
https://github.com/pyca/cryptography

Looks like this question would be better asked of the python guys:
https://github.com/pyca/cryptography

@mattcaswell ok, I will also post bug in cryptography, but very strange, that Python confirms, that used version is OpenSSL 1.1.1f
from cryptography.hazmat.backends import default_backend
default_backend().openssl_version_text()
'OpenSSL 1.1.1f 31 Mar 2020'

@mattcaswell ok, I will also post bug in cryptography, but very strange, that Python confirms, that used version is OpenSSL 1.1.1f
from cryptography.hazmat.backends import default_backend
default_backend().openssl_version_text()
'OpenSSL 1.1.1f 31 Mar 2020'

May close, remove python cache ./.cache/pip/wheels/ everywhere works

May close, remove python cache ./.cache/pip/wheels/ everywhere works

Thanks for the confirmation. Closing.

Thanks for the confirmation. Closing.

cryptography has bug .
Write error OpenSSL 1.1.1b version number，The problem was with Ed25519 support.
ref https://github.com/pyca/cryptography/pull/5452

@reaperhulk ... FYI

Thanks for the ping. cryptography allows ed25519 in 1.1.1b+; anyone who is seeing a problem should file an issue on the pyca/cryptography tracker and not OpenSSL 😄 Please be sure to provide instructions on how to reproduce the behavior so we can debug the environment.

Don't be nervous。the bug has fixed。I write this comment  only to notice anyone who has same problem。
https://github.com/pyca/cryptography/pull/5452


Original appearance of an uploaded non square album art isn't cropped

@ofsound Is this still an issue?

@Manfred Yes, right after uploading, when the new image first appears on the page, before the playlist is saved, when it's in this state:

Ah, that image element is created by the JavaScript based on the file loaded from disk. The easiest way to solve this is probably to change the JavaScript to create a style attribute with a background-image data URL and then style it properly.


update test case to ensure shouldUndo sees a string with a /

@client9 this makes it so this path:
        // perhaps a URL
        if strings.Contains(s, "/") {
                return true
        }

is actually executed in the tests, since before the string was > 20 chars and was returning true here:
        // this is some blob of text that has no spaces for 20 characters!
        // Smells like programming or some base64 mess
        if len(s) > 20 {
                return true
        }

(coverage increases by a percentage point with this)

grazie


WaitForSelector and waitForXpath timesout inside iframe

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.13 puppeteer-core
Platform / OS version: Windows10
URLs (if applicable):
Node.js version: 10.14

What steps will reproduce the problem?
The iframe do not have any name so I find it using
page.mainFrame().childFrames().find(frame => frame.name() === '')
When  I try to type inside the element in iframe, I do
frame.type('#selector',string) and this works
When I try to wait for the xpath/selector inside the iframe for the same selector it timeout
await frame.waitForSelector('#selector');
I tried putting in options {visible:true,timeout:10000} but no luck.

@mayankchincholkar can you give us a script so that we can run it locally and see what's going on?

Hey @aslushnikov
I tried this using below script and it seems to work.
const puppeteer = require('puppeteer');
(async () => {
const browser = await puppeteer.launch({ headless: false });
const page = await browser.newPage();
await page.setContent(
    '');
const frame = await page.mainFrame().childFrames().find(frame => frame.name() === '');
await frame.waitForXPath('//*[@id="mapDiv"]/div/div/div[9]/div/div/div/div[1]/div[1]');
await browser.close();

})();
However, the exact same thing doesn't seem to work on the application I am working on in my organisation.
The only difference is the iframe opens as a pop up when clicked on a button instead of the same page. Also, when I do
const frame=page.mainFrame().childFrames().find(frame => frame.name() === '');
frame.url(); // gives null and _url is not set
Any help will be appreciated.
Is there any other way which I can use to wait for an element to be ready on screen.
Like waitForFunction and use Page.eval to get document object to find the element.
Thanks
Mayank

Same thing is happening for me too.
const frame = page.frames();
await frame[0].$( selector ); //THIS WORKS
await frame[0].waitForSelector(selector); //THIS TIMES OUT

Me too.
await frame.waitForSelector('[ng-controller="CopySetupCtrl"]', {visible: true});  // THIS TIMES OUT
await frame.$('[ng-controller="CopySetupCtrl"]', {visible: true});  // THIS WORKS
My frame is a bit unusual because the name is a big JSON string.  form.name() does not work either, not sure if that's related.  form.name() returns ''.  I am finding my form w/ form.url(), which works.
node v10.17.0
puppeteer v2.0.0

I am also experiencing this issue. Has there been any updates on this?

also experiencing this
Used this instead
await page.$( iFrameSelector )

The same. Looks like nothing changed

I found this recently: https://stackoverflow.com/a/59999976/4564097
Not tested, I'm still searching for a cleaner option.


[WIP] print status message when linking binary

This adds a status print when linking binaries.
Linking can take quite some time when using lto so this adds some additional information.
I also had situations where the base/root crate was actually not build last in the dependecy chain which looked a bit strange because in most cases buiding the base crate also builds its binaries right after.
I started updating some of the tests but wanted to have some comments on the idea before I update the rest.

r? @matklad
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks for the PR! Cargo has historically been pretty bad about informing you what's actually happening at any one point in time, and it's always something I've wished we could do better at!
I'm somewhat hesitant though to add a per-target or per-binary message in the sense that it can on some projects cause a deluge of status messages. I'd love, though, if we could explore various ways of updating a status prompt with what Cargo is doing so it tells you at least one thing it's doing rather than attempting to print out everything perhaps?
I'm also slightly hesitant about the name "Linking" here in the sense that some binaries actually spend a significant amount of time during compilation (aka not in the linker), so it could be slightly misldeaing and cause somet to think that the linker is what's actually being super slow.

I'd love, though, if we could explore various ways of updating a status prompt with what Cargo is doing so it tells you at least one thing it's doing rather than attempting to print out everything perhaps?

Make just prints the currently running command, and cmake also tells the user when linking is happening, I took some inspiration from this.
There is also ninja which when not having its output piped into something else, prints only a single line of the most recent job launched and updates that line. Do you mean something like this?

I'm also slightly hesitant about the name "Linking" here in the sense that some binaries actually spend a significant amount of time during compilation (aka not in the linker), so it could be slightly misldeaing and cause somet to think that the linker is what's actually being super slow.

Are you referring to linktime optimizations here? If not I don't think I understand what you mean.

Yeah in general we don't want to be make/cmake as they're quite a bit more verbose than Cargo right now (plus there's always cargo build -v for that), but something like Ninja I think would be fine where the last line is basically a preview of one of the currently running jobs. That I think is "relatively easy" to implement and pretty portable across terminals at least.
And yeah "Linking" to me implies "running the system linker" but often times the final binary is stuck in optimizations, be them normal or LTO or whatnot.

(plus there's always cargo build -v for that)

Maybe I can hook up the "Linking ..." message in the verbose output instead in the meantime...

And yeah "Linking" to me implies "running the system linker" but often times the final binary is stuck in optimizations, be them normal or LTO or whatnot.

Perhaps the term "Link time optimizations" is a bit trickier than I previously though... :)

:umbrella: The latest upstream changes (presumably #5063) made this pull request unmergeable. Please resolve the merge conflicts.

I think in general this may just be a case where Cargo wants a far richer set of information coming out of rustc. In theory Cargo wants to be informed of all status updates that rustc is doing to always know what phase it's in, and that way Cargo could always print out at least one status for one crate so you can see what's going on.
That's a bit of a braoder issue though :(
For this PR though the message isn't quite right as Cargo doesn't know when rustc is invoking the linker or performing optimizations.


Fix avtUnstructuredDomainBoundaries::ExchangeScalars and ExchangeVectors

Description
Ensure all procs are calling the same Exchange function based on dataType that they get from an MPI_Allreduce.
This fixes problems in parallel (segv/hang) when avtExtraGhostNodes/Zones are exchanged, because the proc with data was calling Exchange_uchar but the proc without data was calling Exchange_float.
Modified avtGenericDatabase to ensure all procs participate in the exchange of avtExtraGhostNodes/Zones as originally intended.
Resolves #19988
Also performed whitespace cleanup in the files touched.
Type of change

[X] Bug fix
[ ] New feature
[ ] Documentation update
[ ] Other

How Has This Been Tested?
I ran the simple example in the bug ticket without a hang, and I ran the mili.py regression test in parallel mode with success.
Checklist:

[X] I have commented my code where applicable.~~
[ ] I have updated the release notes.
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[X] I have confirmed new and existing unit tests pass locally with my changes.~~
[ ] I have added new baselines for any new tests to the repo.
[X] I have NOT made any changes to protocol or public interfaces in an RC branch.~~

I'm glad you sorted this out, thanks!!!

Your welcome.  It was our group discussion today that gave me ideas.

Just saw segv when running other tests in parallel, so I will do more digging before I commit this.

If no processor has extraGhost information, the call to dbi->ExchangeScalar  receives and returns an empty vector of extraGhost arrays.
The logic after the call to ExchangeScalar assumes that the output from the exhange has data.
I am  adding logic to query all procs (with an MPI_ALL_Reduce) as to whether they have extraGhost info and skip the Exchange altogether if no proc has this information.

@markcmiller86 is there a mechanism whereby the Mili plugin can publish something like 'Hey I populated extraGhosts so please handle this for me' which would be a simpler check that could be done by avtGenericDatabase instead of assuming the information should exist and then doing parallel-communication to figure out whether the assumption was true?

@markcmiller86 is there a mechanism whereby the Mili plugin can publish something like 'Hey I populated extraGhosts so please handle this for me' which would be a simpler check that could be done by avtGenericDatabase instead of assuming the information should exist and then doing parallel-communication to figure out whether the assumption was true?

Possibly. I am looking. That said, I am not sure how that resolves this issue. Because in this case the problem is that some MPI ranks have extra ghost and some (apparently) do not.

Possibly. I am looking. That said, I am not sure how that resolves this issue. Because in this case the problem is that some MPI ranks have extra ghost and some (apparently) do not.

Actually, the latest problem is when NO process has the extra ghost info (any database plugin that isn't Mili for instance).
The extraGhost logic assumes that the output from ExchangeScalar should contain valid data, but it is an empty vector. I could check for that, I suppose, but I thought it might be better to determine whether or not the Exchange function should even be called in the first place.

Something like this:
https://github.com/visit-dav/visit/blob/e883c552d71e7a396344ba84bc3c557bfacf806c/src/avt/Database/Database/avtGenericDatabase.C#L8583

I could check for that, I suppose, but I thought it might be better to determine whether or not the Exchange function should even be called in the first place.

I agree that avoiding any MPI comm. when you know it is not necessary is a good thing. So, is the "extra" ghost currently only showing up during GetMesh()? If so, that is likely the cause of parallel comm issue. I think for all preceding stuff of this ilk, all processors would agree on something they put into avtMeshMetaData during ...::PopulateDatabaseMetaData() (which is collective by definition meaning all processors will agree on that) and then that would be used to drive participation in collective parallel comms.

Something like this:
https://github.com/visit-dav/visit/blob/e883c552d71e7a396344ba84bc3c557bfacf806c/src/avt/Database/Database/avtGenericDatabase.C#L8583

the items in avtDataRequest (spec) are set by filters/operators/plots  to indicate what kinds of extra information they may need from the databases. I don't think it is accessible for the database to push information to it.

@markcmiller86 after looking at avtMiliFileFormat, yes the extraGhost information is added in the GetMesh call.
@JustinPrivitera  is there any way to know during PopulateDatabaseMetaData whether or not extraGhost information will be added?

@JustinPrivitera is there any way to know during PopulateDatabaseMetaData whether or not extraGhost information will be added?

@biagas yes: miliMetaData[meshId]->ContainsSand() will tell you if have sand which means you will have extra ghosts

thanks @JustinPrivitera and @markcmiller86 . It seems we might be able to add a new entry in avtMeshMetaData that Mili plugin can set to true, something like 'hasExtraGhostInformation'.  Then avtGenericDatabase could query that value and know whether or not to attempt processing of the extraGhost data.  Would have to wait for 3.5, though, due to addition of new attribute.

I added some parallel logic to test whether the Exchange of extraGhost info should be done.
This should get 3.4RC in shape, though its ugly.

It seems we might be able to add a new entry in avtMeshMetaData that Mili plugin can set to true, something like 'hasExtraGhostInformation'. Then avtGenericDatabase could query that value and know whether or not to attempt processing of the extraGhost data. Would have to wait for 3.5, though, due to addition of new attribute.

Should I open a ticket for this?

We could, I already have something in place on develop so it will get fixed right away.

Great, thanks!

@JustinPrivitera I cannot find your requested changes. I might have made them disappear when I rebased this branch off of 3.4RC.


OrbitClientGgp: project structure and basic capture flow

Create OrbitClientGgp to manage a capture request from the ggp instance
Main files and the structure of the project has been created. Basic flow is working: initial set up, starts capture, captures for a period of time and stops the capture.
Capture data is not collected or handled yet; this pull request is intended to include the project in Orbit so there is not a huge list of commits later. Also it avoids being 'silently' broken in updates related to Capture, etc.
The options related to the capture cycle must be provided as an argument to OrbitClientGgp. Some values are given a default value but others are required to be provided by the user:

pid. (Mandatory) Id of the process to capture.
grpc_port. (Optional) Required for gRPC, by default is 44765.
sampling_rate. (Optional) Required for CaptureClient to work, by default is 1000.
frame_pointer_unwinding. (Optional) Required for CaptureClient to work.
capture_length. (Optional) Sets the length of the capture in seconds, by default is 10

The new project contains the following files:

main.cpp Contains the linear flow and handles the different classes instantiation and requests
ClientGgp.h/cc Contains methods that handle the gRPC client and capture requests. Extends and implements CaptureListener (although it does not handle the calls yet).
ClientGgpOptions.h  Struct containing the options needed for the client initialisation.

Also a Run_client_ssh.ps1 script has been created to facilitate the deployment and test of OrbitClientGgp.

Hi Cris,
I hope you don't mind that I added some minor comments. It looks really good! :-)
But someone else who is more familiar with what you're trying to do should also have a look. Supposedly Daniel. Thanks.

Hi,
All feedback is welcome so thanks a lot for taking the time to take a look at the code :)
I have gone through the comments and suggestions and updated the code accordingly. Thanks!

Looks good! There is really only some nit picking.

I have updated the code with the suggestions, thanks!

Besides the points below: maybe have a look into minimize use of vertical whitespace. The style guide asks for as little empty lines as possible. This is not a fixed rule thought, use your best judgement.

Do you mean I should include less empty lines between code? I can take a look and update it. I like including them because I think it helps readability and the separation between different concepts but I can update it to follow the guidelines

@danielfenner I have removed some of the vertical whitespace as suggested to follow the guidelines. I think the code is still clear like this. Thanks!

The code will need to be updated as a result of PR #980 which changes Capture and includes the thread creation inside OrbitClient, so I will wait for that to be merged and update the code accordingly. It will need new review when done.

Do you mean I should include less empty lines between code? I can take a look and update it. I like including them because I think it helps readability and the separation between different concepts but I can update it to follow the guidelines

Yes exactly, the java style guide encourages exactly this - use empty lines as paragraphs. The c++ style guide asks for the opposite. As I wrote above, just using your own judgement is totaly fine.

I have updated the code to integrate the changes related to Capture and CaptureClient

Including OrbitClientGgp into master


Expose SQL data explorer

Currently, the SQL data explorer is not discoverable.
https://github.com/tripleee/metasmoke/tree/sql-menu has my no doubt incompetent attempt to figure out how to add it. The first commit might actually be useful. The second mainly just illustrates how lost I am.

The nav_link function is defined here. From my exploration of the Blazer source, this should work:
<%= nav_link Blazer::QueriesController, action: :home, label: 'SQL Data Explorer' %>


No way to calculate any indicator from the box

I am trying to calculate provision for the city, but I dont have any of required data. Unfortunately there are no option to do so from the box. Also I cant understand what kind of data is uploaded in the example file :(

@GeorgeKontsevik , you have to prepare all the required data yourself, but it should match the specification (data-like classes described in methods' parameters)


When updating related entities, Eloquent events such as saved and updated are NOT fired.

The problem
When using the update() method without retrieving the entity first, the Laravel events updating, updated, saving and saved are not fired. I noticed this when I was working on an Event for my application and I was wondering why the saved event was only fired when a new related entity is added, but it is not fired when an existing entity is edited.
https://github.com/alfa6661/laravel-hasmany-sync/blob/16aac13f31050d47a8adff761fd5f6ae314476bb/src/ServiceProvider.php#L75-L76
The reason why saved event is fired when a new related entity is added is because createMany method is used.
https://github.com/alfa6661/laravel-hasmany-sync/blob/16aac13f31050d47a8adff761fd5f6ae314476bb/src/ServiceProvider.php#L68
Possible solution?
I would solve the issue by replacing the use of update method with
$related = $this->getRelated()
                    ->where($relatedKeyName, $castKey(array_get($row, $relatedKeyName)))
                    ->first()
                    ->fill($row)
                    ->save();

Literature
https://laravel.com/docs/8.x/eloquent#updates
https://stackoverflow.com/questions/41295032/laravel-eloquent-model-update-event-is-not-fired

@Excelautomation I don't have rights to modify your fork directly, that's why I needed to create a PR. :)


Portrait Aspect ratio windows are cut with RMC 3

See the screenshots.
The same code with 2.2.2 works fine.
This is the window I am sharing:

This is what is displayed in the video control:

In your HTML file, you can add following snippet:
connection.__getScreenConstraints = connection.getScreenConstraints;
connection.getScreenConstraints = function(callback) {
    connection.__getScreenConstraints(function(error, screen_constraints) {
        if (connection.DetectRTC.browser.name === 'Chrome') {
            delete screen_constraints.mandatory.minAspectRatio;
            delete screen_constraints.mandatory.googLeakyBucket;
            delete screen_constraints.mandatory.googTemporalLayeredScreencast;
            delete screen_constraints.mandatory.maxWidth;
            delete screen_constraints.mandatory.maxHeight;
            delete screen_constraints.mandatory.minFrameRate;
            delete screen_constraints.mandatory.maxFrameRate;
        }
        callback(error, screen_constraints);
    });
};

Or to more simplify it:
connection.__getScreenConstraints = connection.getScreenConstraints;
connection.getScreenConstraints = function(callback) {
    connection.__getScreenConstraints(function(error, screen_constraints) {
        if (connection.DetectRTC.browser.name === 'Chrome') {
            screen_constraints.mandatory = {
                chromeMediaSource: screen_constraints.mandatory.chromeMediaSource,
                chromeMediaSourceId: screen_constraints.mandatory.chromeMediaSourceId
            };
        }
        callback(error, screen_constraints);
    });
};

Now minAspectRatio and all other redundant args will be skipped. And your screen with work in latest Canary as well.

But why is it working in 2.2.2 ?

v2.2.2 is NOT using minAspectRatio. Try this demo and you'll see following logs in the chrome console:
invoked getUserMedia with constraints: {
	"audio": false,
	"video": {
		"mandatory": {
			"chromeMediaSource": "desktop",
			"maxWidth": 1920,
			"maxHeight": 1080,
			"chromeMediaSourceId": "fhC/skmCDOEaZ9CyAQkmWg=="
		},
		"optional": []
	}
}

connection.__getScreenConstraints = connection.getScreenConstraints;
connection.getScreenConstraints = function(callback) {
    connection.__getScreenConstraints(function(error, screen_constraints) {
	if (connection.DetectRTC.browser.name === 'Chrome') {
	    delete screen_constraints.mandatory.minAspectRatio;
	}
	callback(error, screen_constraints);
    });
};

is enough.
What about Firefox?

You can delete width and height for Firefox:
connection.__getScreenConstraints = connection.getScreenConstraints;
connection.getScreenConstraints = function(callback) {
    connection.__getScreenConstraints(function(error, screen_constraints) {
        if (connection.DetectRTC.browser.name === 'Chrome') {
            delete screen_constraints.mandatory.minAspectRatio;
        }
        if (connection.DetectRTC.browser.name === 'Firefox') {
            delete screen_constraints.width;
            delete screen_constraints.height;
        }
        callback(error, screen_constraints);
    });
};


Add a way of searching SpigotMC resources

Hi there,
Great work with the API, it's looking really good.
My suggestion is essentially a way to directly search through resources on Spigot with a query and the possibility to search only specific categories.
Many thanks

You can use listResources to search only specific categories, but you can't search by resource name at the moment. I will try to make a patch that lets you do that at some point in the future.

Thanks for the update! Appreciate the word :)


Add cross browser animations classes

Hi , I want to suggest a feature to you , add some coll transitions and animations like fadein, fadeout, bounce, rotate and ......

Is there something you are unable to achieve using standard CSS? We have some utility classes, but we try to keep them at a minimum and this may be beyond our scope.

There are libraries that provide generic helpers for that already, no plans for us to add our own. We'd map them to components if anything.


Cut release v0.26.3

Changed

File tree acts more like VS Code's file tree (#4392)
Creating new projects asks for project name before creating (#4109)

Added

Snap-to-origin and axis behavior for sketch profile starts and segments (#4344)
Implement a simple startSketchOn / offsetPlane lint rule (#4384)
KCL: More ways to reference paths (#4387)
Error if assertEqual's epsilon value is invalid (#4329)
Show top-level project directory name in file tree pane header (#4165)
Add ability to open, create, rename, and delete projects from inside one via command bar (#4109)

Fixed

Fix KCL source ranges to know which source file they point to (#4418)
Fixed directory/file selection logic to deselect folders properly and always hightlight files.  (#4408)

Contributors

Thanks again to @guptaarnav for more thoughtful contributions!

MacOS smoke testing:

[ ] Mash keyboard in code editor
[ ] https://github.com/KittyCAD/modeling-app/issues/3373
[ ] coredump will open github issue with stuff populated (from home and project page) there are different ways to trigger it as well (! button at top, bug at bottom, mod + shift + .)
[ ] running app.{app,exe} path/to/file opens the kcl file (all oses)
[ ] running app.{app,exe} path/to/dir opens the directory and any kcl file in it or creates a main.kcl if empty (all oses)
[ ] If someone at the office, test you can hit make from the app and it works
[ ] Test updating versions works
[ ] until we have a performance test, sanity check that time to open file directly from command line is not shit
[ ] https://github.com/KittyCAD/modeling-app/issues/3455
[ ] https://github.com/KittyCAD/modeling-app/issues/3469
[ ] https://github.com/KittyCAD/modeling-app/issues/3465
[ ] double click to open files from finder opens new window each time (mac/windows)
[ ] export from the app works and opens a file picker ux (needs to be manual since we override for playwright)
[ ] Ensure that a loading toast appears when testing the auto-updater, saying the download has begun
[ ] https://github.com/KittyCAD/modeling-app/issues/4038

Windows smoke testing:

[x] Mash keyboard in code editor
[x] https://github.com/KittyCAD/modeling-app/issues/3373
[x] coredump will open github issue with stuff populated (from home and project page) there are different ways to trigger it as well (! button at top, bug at bottom, mod + shift + .)
[x] running app.{app,exe} path/to/file opens the kcl file (all oses)
[x] running app.{app,exe} path/to/dir opens the directory and any kcl file in it or creates a main.kcl if empty (all oses)
[ ] If someone at the office, test you can hit make from the app and it works
[x] Test updating versions works
[x] until we have a performance test, sanity check that time to open file directly from command line is not shit
[ ] https://github.com/KittyCAD/modeling-app/issues/3455
[x] https://github.com/KittyCAD/modeling-app/issues/3469
[x] https://github.com/KittyCAD/modeling-app/issues/3465
[x] double click to open files from finder opens new window each time (mac/windows)
[x] export from the app works and opens a file picker ux (needs to be manual since we override for playwright)
[x] Ensure that a loading toast appears when testing the auto-updater, saying the download has begun
[x] https://github.com/KittyCAD/modeling-app/issues/4038

Both MacOS and Windows tests were solid, and updater tests went on without a hitch


[FIX] changed "copy" to "mutableCopy" for NSMutableArray types

Albite PBAppendableArray uses the same backing as PBArray it is okay to just call "copy" for those. For NSMutableArray a call to "copy" creates an immutable NSArray which is assigned to the mutable typed variable. This will create unrecognised selector crashes (see test).

Thanks!

np. ;)


bug: Unable to create a reception address

versions :
windows 11 prox64
btc cli 25.0
ord 0.16.0-rc1
rpc.conf

##
## bitcoin.conf configuration file. Lines beginning with # are comments.
##
daemon=0
regtest=1

listen=1

# JSON-RPC options (for controlling a running Bitcoin/bitcoind process)

#rpcport=8332

# server=1 tells Bitcoin-Qt and bitcoind to accept JSON-RPC commands
server=1
#prune=5500
txindex=1
datadir=H:\btc\data

fallbackfee=0.000002
minrelaytxfee=0.000002

bitcoind -conf="H:\btc\rpc.conf"
bitcoin-cli -conf="H:\btc\rpc.conf" createwallet dev
bitcoin-cli -conf="H:\btc\rpc.conf" generatetoaddress 101 $(bitcoin-cli -conf="H:\test\rpc.conf" -rpcwallet=dev getnewaddress)
ord -r --cookie-file "H:\test\data\regtest.cookie" wallet --name=ord_dev create
ord -r --cookie-file "H:\test\data\regtest.cookie" wallet --name=ord_dev receive
thread 'main' panicked at src\wallet.rs:112:54:
called `Result::unwrap()` on an `Err` value: ParseIntError { kind: InvalidDigit }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Did you start the ord server?
The new wallet requires running it in parallel

Did you start the ord server?
The new wallet requires running it in parallel

And possible to show a warning asking users to run the server in this case? I'm afraid most users won't read the changelogs.

I read the changelogs but missed this information. Indeed, adding a warning would be great.

Added a warning in #3272


Tile events are not being cleaned up

Checklist

[X] I've looked at the documentation to make sure the behavior isn't documented and expected.
[X] I'm sure this is an issue with Leaflet, not with my app or other dependencies (Angular, Cordova, React, etc.).
[X] I've searched through the current issues to make sure this hasn't been reported yet.
[X] I agree to follow the Code of Conduct that this project adheres to.

Steps to reproduce
I noticed when zoomin and out on the map the event listeners are growing quite quickly. It looks like the createTile function is creating events inside the TileLayer.js. I don't think these are cleaned up properly.
To reproduce just look at the event listerners inside Chromes performance monitor while zooming in and out. The events keep climbing.
Expected behavior
Events should be cleaned up when calling map.off() or map.remove().
Current behavior
"load" and "error" Events are leaking
Minimal example reproducing the issue
No response
Environment

Leaflet version:  latest
Browser (with version): Chrome Version 123.0.6312.122
OS/Platform (with version): Windows 11

Noticed the issue as well. While the delay is no problems on android or desktop, it seems to induce some crash/map reload on iOS to due maybe to more strict memory management


Remove Newtonsoft JSON.NET from Blazor WASM template

Addresses #13422 and #14039.
Closing original PR #14028 which was targeting the master branch in favour of this PR which correctly targets the release/3.1 branch.

Thanks for your PR, @adrianwright109.


How can I provide a minimum and maximum month which can be scrolled to?

Hi,
I would like to stop people from scrolling to months before and after certain dates, perhaps the first of said months.
I've seen that I can use canSelectDate to stop people from selecting dates, but I'd  months / dates beyond and before a maximum / minimum date not to be visible?
e.g. Don't disable after January 2017 or before December 2013.
Can you talk me through how I can achieve this, if it's possible?
I did see someone has posted this..
https://gist.github.com/HaloZero/66fec90151503d09f9c9
... but I don't have a panning view, don't want to have my own version of the pod, as I don't feel confident making changes to it.
Thanks so much for your great control :)
Jules.

Currently it's not possible, but you're not the first to ask for it.

Hmmm, how about being able to disable the scrolling on the month bar?
Then I could just add some next / previous buttons?

You can do it, I just don't have the time now.

Ah right, wasn't sure that would exposed.
Can you briefly summarize what I would need to do?
Thanks
:)

Ahhhh after reloaddata in viewdidload...
[self.calendarMenuView setScrollEnabled:NO];

I can this in my next a previous buttons now I've turned off scrolling, I have an example of this here... https://github.com/Jules2010/JTCalendar


Reload shard map on restart

Fixes https://github.com/influxdb/influxdb/issues/2012
@pauldix @benbjohnson

The fact that this issue wasn't causing any real problems until I started using s.shards for stats, tells me s.shards is probably useless now, and we could just get the shards direct from the retention policies. Perhaps it's still a useful shortcut.

This makes sense to me.

:+1:

Thanks @toddboom @benbjohnson


helm chart

This is a stub to track "create a helm chart" in case someone has already done the necessary work for this. Looking at deploying this for a fairly large log cluster. Requirements are:

Need to be able to add data nodes as required.
Need to be able to run on k8s 1.5+ (or whatever the most recent k8s is at the time of development).

I won't get to this for a while. But creating issue anyway!

Created. First draft is at https://github.com/clockworksoul/helm-elasticsearch

Feel free to add instructions to the README.md file on how to run your Helm chart and close this issue with your PR.

Instructions added with PR https://github.com/pires/kubernetes-elasticsearch-cluster/pull/95

Fixed by #95.

@pires Sincere apologies if I've insulted you (which I think I may have). I have no other way to reach you, and I typed quickly and without thinking.


关卡理智数据不正确

Payload
{"owned":{"双极纳米片":4,"白马醇":3,"扭转醇":5,"三水锰矿":4,"五水研磨石":4,"RMA70-12":6},"required":{"D32钢":4,"双极纳米片":4,"白马醇":6,"三水锰矿":5,"轻锰矿":5,"固源岩组":8,"固源岩":5,"装置":6,"聚酸酯组":4,"聚酸酯":13,"糖组":11,"糖":11,"代糖":4,"异铁组":5,"异铁":16,"异铁碎片":5,"酮凝集":9},"exp_demand":false,"extra_outc":false,"gold_demand":true,"exclude":["5-1","5-2","5-3","5-4","5-5","5-6","5-7","5-8","5-9","5-10","5-11","6-1","6-2","6-3","6-4","6-5","6-6","6-7","6-8","6-9","6-10","6-11","6-12","6-13","6-14","6-15","6-16","6-17","6-18","S5-1","S5-3","S5-4","S5-5","S5-6","S5-7","S5-8","S6-1","S6-2","S6-3"]}

材料规划路线（试用版）
预计体力消耗: 1640
刷图获得：
[1-7] x53 - 固源岩 x66, 源岩 x6, 破损装置 x2, 酯原料 x3, 双酮 x2, 异铁碎片 x2, 代糖 x3
[S3-2] x2 - 固源岩 x0.5, 源岩 x0.4, 装置 x0.2, 破损装置 x0.2, 酯原料 x0.3, 聚酸酯 x3
[S4-6] x98 - 轻锰矿 x4, 糖组 x4, 异铁 x15, 异铁碎片 x9, 炽合金 x2, 糖 x19, 代糖 x12, 异铁组 x4
[4-4] x7 - 固源岩组 x0.2, 固源岩 x2, 源岩 x2, 装置 x0.9, 扭转醇 x3, 破损装置 x0.8, 白马醇 x0.4
[4-9] x2 - 酮凝集 x0.1, 研磨石 x0.0, RMA70-12 x0.6, 酯原料 x0.8, 酮凝集组 x0.0, 聚酸酯组 x0.1, 聚酸酯 x0.2, 双酮 x0.6, RMA70-24 x0.1
[3-1] x11 - 酮凝集 x1, 酮凝集组 x4, 双酮 x1, 异铁 x1, 异铁碎片 x1, 糖 x1, 代糖 x2
[3-2] x29 - 固源岩 x6, 源岩 x6, 装置 x2, 破损装置 x3, 酯原料 x4, 聚酸酯 x4, 轻锰矿 x11
[3-7] x4 - 酮凝集 x5, 双酮 x0.5, 异铁 x0.4, 异铁碎片 x0.5, 糖 x0.5, 代糖 x0.6
[2-5] x14 - 双酮 x3, 糖组 x5, 异铁碎片 x3, 糖 x0.4, 代糖 x4
[2-6] x25 - 固源岩 x0.7, 源岩 x10, 破损装置 x4, 酯原料 x7, 聚酸酯组 x9, 聚酸酯 x0.6
合成获得：
[D32钢] x4 - 三水锰矿 x4, 五水研磨石 x4, RMA70-24 x4
[RMA70-24] x4 - RMA70-12 x4, 固源岩组 x8, 酮凝集组 x4
[三水锰矿] x5 - 轻锰矿 x10, 聚酸酯组 x5, 扭转醇 x5
[白马醇] x3 - 扭转醇 x3, 糖组 x3, RMA70-12 x3
[装置] x3 - 破损装置 x9
[酮凝集] x3 - 双酮 x9
[异铁] x4 - 异铁碎片 x12
[异铁组] x2 - 异铁 x8
[聚酸酯] x5 - 酯原料 x15
[糖] x6 - 代糖 x18
[糖组] x4 - 糖 x16
[固源岩] x9 - 源岩 x27
[固源岩组] x16 - 固源岩 x80
来源：明日方舟工具箱: https://aktools.graueneko.xyz & ArkPlanner: https://github.com/ycremar/ArkPlanner , 结果仅供参考


fix: allow for first level indentation of lists (#242)

This is a proposed fix for #242. It should maintain backwards-compatibility, while allowing users like me to achieve our goal as well. If there's a different way you'd like it implemented, I'm happy to rework it with your direction.

Thank you! I started to review the PR, but the fact that the new parameter is a Boolean got me thinking: could this change be made without a parameter at all? As you note in the issue, MD006 already works for people who want to force a left-most start, so since your implementation assumes an indented start will be the same number of spaces as the indent, maybe we can just allow that as well? To work this way, I think the rule would need to track whether the list started out indented or not, but that doesn’t seem too different than what you’re doing already and doesn’t seem like it would complicate the logic much further. What do you think?

Hmmm ... when I first read it, I liked your idea. But then I started thinking about how I'd implement it, and I realized that there's something I'd like to enforce that I don't think would be enforced by your proposal: I always want the lists to be indented one level. With your suggestion, if I understood it correctly, someone could start their list not indented (at char zero of each line), and that would pass because we'd have to allow either 0 or one level.
Is there a way to make MD006 enforce that lists do start indented? Or should that be enforced here in MD007? Or a separate rule altogether?

Let’s do it here. I’ll review the PR now...

@DavidAnson I believe I address all your comments. Let me know if I missed anything or you have further requests. Thanks!

You got everything, thank you! There’s just one small request remaining and then I should be able to accept the PR this evening or over the weekend. Thank you for the contribution!

I renamed both variables to make them consistent with what you mentioned wanting.


Fix issue with assertion caused by pluggable schema validator

Fix issue with type adapters present when plugins are installed.

One other super annoying thing that we should fix soon is that you can't run the plugins tests on their own. I'm not sure where we are affecting state that causes them to pass when you run pytest tests but to fail when you run pytest tests/test_plugins.py but ... I think that should get fixed.
I'll note that if you change the _plugins: ... = None to _plugins: ... = {} this goes away and all tests pass. BUT I think we have special logic when _plugins is None, so maybe we want to add a test that actually ensures this behavior is respected?


Rizz Comics

Source name
Rizz Comics
Source link
https://rizzcomic.com
Source language
English
Other details
No response
Acknowledgements

[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

Thank you so much!
On Wed, 10 Jan, 2024, 2:02 pm beerpsi, @.***> wrote:

Closed #77 https://github.com/keiyoushi/extensions-source/issues/77 as
completed via #82 https://github.com/keiyoushi/extensions-source/pull/82
.
—
Reply to this email directly, view it on GitHub
https://github.com/keiyoushi/extensions-source/issues/77#event-11440044309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXR33OFVZSJRRDZILD65OR3YNZGZXAVCNFSM6AAAAABBUIQR7KVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGQ2DAMBUGQZTAOI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Web app's UI doesn't fit on mobile.

It shouldn't be strictly desktop

GitLight was never intended to be used on mobile as I don't think it would be particularly useful


chore: bump ginkgo package

What this PR does / why we need it:
chore: bump ginkgo package
add rule for dependabot to ignore ginkgo package, because it needs to be bumped manually.
Bump ginkgo to newest version
Release note:
Bump ginkgo package

/retest

@0xFelix, @akrejcir can you please review?


Editor GUI not updating

Operating system or device - Godot version:
Windows 10, 2.1.3 recent stable version
Issue description:
Godot's gui won't update unless I alt+tab. My drivers are up to date but I'm running on a laptop with integrated intel graphics (Intel HD 520), but from what I read intel bugs were fixed in 2.0
thanks for any help you can give me c:

Is this still reproducible in the current master branch / 3.1-alpha?

Closing due to lack of update. Please comment if you can still reproduce this issue in the latest stable version of Godot.


Update hax and rustc

This is the companion PR to https://github.com/hacspec/hax/pull/690

@msprotz This is a PR that updates the rustc version. Rustc changed some of its code generation as you can see from the test differences. This apparently breaks kyber, do you understand why? The error output is:
[ INFO charon_lib::export:111] [GCrateData<_, _>::serialize_to_file]: Generated the file: /run/github-runner/charon-ci/charon/charon/kyber-crate/libcrux_kyber.llbc
warning: `libcrux_kyber` (lib) generated 2 warnings (run `cargo fix --lib -p libcrux_kyber` to apply 2 suggestions)
    Finished dev [unoptimized + debuginfo] target(s) in 26.62s
Running eurydice ...
Cannot re-check libcrux_kyber.sampling.sample_from_uniform_distribution_next as valid Low* and will not extract it.  If libcrux_kyber.sampling.sample_from_uniform_distribution_next is not meant to be extracted, consider marking it as Ghost, noextract, or using a bundle. If it is meant to be extracted, use -dast for further debugging.

1⃣ LLBC ➡  AST
2⃣ Cleanup
Warning 4: in the then branch, after the definition of uu____981, in the sequence statement at index 0, after the definition of iter, in top-level declaration libcrux_kyber.sampling.sample_from_uniform_distribution_next, in file libcrux_kyber_common: Malformed input:
subtype mismatch, () (a.k.a. ()) vs bool (a.k.a. bool)
LowStar.Ignore.ignore doesn't go anywhere
File order after topological sort: LowStar_Ignore, core, libcrux_digest, libcrux_kyber_common, libcrux_kyber768
Error: Process completed with exit code 1.

Could you try passing --log '*' to Eurydice and seeing where it differs?

kyber main is broken on charon+eurydice main right now so we'll have to debug that first.


Airflow: Add log_url to TaskInstanceInfo

Problem
It is quite useful to have an URL of Airflow task log as a part of airflow facet. Currently I have to reimplement the logic present in Airflow, and keep it in sync with a particular Airflow version, e.g.:
https://github.com/apache/airflow/blob/2.9.0/airflow/models/taskinstance.py#L1719-L1731
https://github.com/apache/airflow/blob/2.1.0/airflow/models/taskinstance.py#L524-L528
Solution
One-line summary:
Add log_url field to Airflow TaskInstance facet.
Checklist

[X] You've signed-off your work
[X] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[X] Your change contains a small diff and is self-contained
[X] You've updated any relevant documentation (if relevant)
[X] Your comment includes a one-liner for the changelog about the specific purpose of the change (not required for changes to tests, docs, or CI config)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2024 contributors to the OpenLineage project

I like the idea. I was thinking about adding some new config facet that would include the content of config [webserver] BASE URL, but this may be cleaner solution if having that in task level event is enough

BASE_URL may also be useful in some cases, e.g. to have a direct URL to a page of specific dagRun.
But for me, OL namespace for Airflow integration is just the same as BASE_URL, so I see no difference here.

Would you mind doing the same for Airflow provider so that we also have this feature for Airflow 2.7+ ?

https://github.com/apache/airflow/pull/40797

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.75%. Comparing base (cc6eb12) to head (ce8e74e).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2852      +/-   ##
==========================================
- Coverage   85.52%   84.75%   -0.78%
==========================================
  Files          58       53       -5
  Lines        3413     3240     -173
==========================================
- Hits         2919     2746     -173
  Misses        494      494

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

Thanks!


Tree-Widget: Fix invert button in categories tree

closes #405

@grigasp

Can you please also post the test coverage report to see how it looks after all the test improvements?


Chat API

Resurface https://github.com/micro/services/tree/master/test/chat

Done https://m3o.com/chat


question ordering clarification

where does the question order come here
https://norppa.helsinki.fi/targets/72438142
(why koulutusohjelma question after course specific questions)

Both organization and university level open questions are placed last in question ordering
So this is working as specified at https://github.com/UniversityOfHelsinkiCS/palaute/blob/6c16cc478e8a056932d3f7097d020baa73dfc59c/config/default.js#L203C7-L203C7


fix: kctl incorrect image repository

What type of PR is this?

/kind bug
What does this PR do?

kctl incorrect image repository
Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

/approve
/lgtm


how to use scrollUntilVisible on flutter java

hello, how to use command scrollUntilVisible on flutter java.  thanks

As of now, I'm not implemented will try figure out.


haskellPackages.optparse-applicative: Fix for compgen error

Motivation for this change
Fix optparse-applicative's call to bash when running in a nix-shell. Although nix-shell invokes bashInteractive, it has a non-interactive bash on PATH most of the time, because that's what stdenv uses. optparse-applicative can get a better bash via $SHELL, fixing the error:
[nix-shell]$ mycommand --file <TAB>bash: compgen: command not found
bash: compgen: command not found
bash: compgen: command not found

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@roberth Thanks for fixing this.  I think I've run into this in the past and not realized what was happening.
Would you be able to rebase this on the haskell-updates branch?  Also, do you have a specific command in mind that I'd be able to test this with?  I imagine I'd just have to do something like nix-shell -p haskellPackages.some-command and then try the bash completion?

I discovered this while working on the new Hercules CI CLI which is isn't packaged yet.
It turns out that not so many commands use bashCompleter, which is the offending function. For example, stack implements its own path completion.
I did find atsfmt which is packaged but doesn't have
PATH=$(nix-build --expr 'let pkgs = import ./. {}; in pkgs.haskell.lib.generateOptparseApplicativeCompletion "atsfmt" pkgs.haskellPackages.ats-format')/bin:$PATH
PATH=$(nix-build '<nixpkgs>' -A bash)/bin:$PATH
source <(atsfmt --bash-completion-script $(which atsfmt))

I now realize that I didn't have to call generateOptparseApplicativeCompletion because that only affects nix-env -i and environment.systemPackages etc. Oh well.

Rebased in #100563

@roberth I guess you fixed this in #100563, so I am going to go ahead and close this PR.  Please feel free to reopen it if this is a mistake.


Unsupported network error when initializing custom chain in Prime SDK [BUG]

Describe the bug
I am encountering an "Unsupported network" error when attempting to initialize a custom chain using the Prime SDK. This issue seems to occur during the initialization process with the provided configuration.
Expected behavior
I expected the Prime SDK to successfully initialize the custom chain with the provided parameters, connecting to the specified network without encountering an "Unsupported network" error.
Steps to reproduce

Initialize the Prime SDK with the following configuration:

   { privateKey: process.env.WALLET_PRIVATE_KEY },
   {
      chainId: 49088,
      projectKey: 'public-prime-testnet-key',
      entryPointAddress: '0x793949275B35D3E394f5C218DC88ae6C7D04a252',
      bundlerRpcUrl: 'https://bifrost-testnet-bundler.kanalabs.io/',
      factoryWallet: Factory.ETHERSPOT,
      walletFactoryAddress: '0x91037ED7B25d72F61B80D4376C0E66680997Ce5E',
   },
);

Observe the "Unsupported network" error during initialization.

Additional context
code

error

Operating system
Windows
Prime SDK version or commit hash
v1.3.12

Thanks for reporting @0xAnto . We should have a fix ready for this shortly.


Revert AVP compat

themaster402/AstronomersVisualPack#23 updated AVP's remote version file to reflect the correct compatibility
https://github.com/themaster402/AstronomersVisualPack/commit/05059fa817597b150bc479f6e3fa72aa2a3ebac5 changed the VERSION property before the referenced version was released, so those changes were lost in e0718307b1a12373d9e951e42e8f82dac6d2948c
After the next version is released, merging this PR will fix it

4bdb32d9c38400024baf21daaf0e2f154bd80140


Added carousel project

Issue reference:
Proposed changes:
Added new project
Type of change:
Carousel project added
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Additional info (if any):

Nice work @anshrusia200 🙌


es2015 => env

Explanation About What Code Achieves:

Explanation

Screenshots/GIFs:

Screenshot

Steps To Test:

Steps

TODOs:

None

ah thanks!


Cant download api&

klk@Microserver:/data/inp/Download$ youtube-dl -user<EMAIL_ADDRESS>-password +++++  -f 22 https://youtu.be/Ezc4HdLGxg4 --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-user', 'xxxxx', '-password', 'xxxxx', '-f', '22', 'https://youtu.be/Ezc4HdLGxg4', '--verbose']
[debug] Encodings: locale 'UTF-8', fs 'UTF-8', out 'UTF-8', pref: 'UTF-8'
[debug] youtube-dl version 2014.02.17
[debug] Python version 2.7.6 - Linux-3.19.0-28-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] Proxy map: {}
[youtube:search] query<EMAIL_ADDRESS>Downloading page 1
ERROR: Unable to download API page: HTTP Error 410: Gone; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 191, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1157, in urlopen
return self._opener.open(req)
File "/usr/lib/python2.7/urllib2.py", line 410, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 523, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 448, in error
return self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 531, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

The options --username and --password must be preceded by double, not single dashes (also mind the name in --username). youtube-dl will then also then not show the password in the output. You just posted your password to the internet, please change it.

You are also using an outdated version of youtube-dl. Read our FAQ if you have troubles updating.

Updating helps me. Thanks


Installing Silently in User Context via Intune

Hello together,
i have one question i stumbled upon. We want to install Spotify via Winget Install and distribute it via Intune. Normally we would install it as System Context, which would be automatically quiet (or atleast no powershell window opens). However, Spotify is not allowing it to install it as Systems Context or with Administrator Rights.
If we install it with User context, it works fine, but the Users see the PowerShell popping up when installing Spotify. Im talking about the PowerShell which is displaying what the Winget-Install.ps1 Script is doing. Is there any way we can disable that this powershell Window is open? We tried -NonInteractive as a PowerShell Launch Parameter, but that didn't work either, it keeps popping up.
Thank you very much!

hello. Try powershell.exe -WindowStyle hidden

hello. Try powershell.exe -WindowStyle hidden

Thanks, that did work!

no worries :)


RVSDG logic

Learn and propagate "situational" predicates over operands.
So far, the only interesting situation we consider is the last iteration of a Theta, in which we know the predicate must be false. This lets us to rewrite print res to print 8 in the following program, addressing #189:
i = 1
do while i < 10
  if i < 10
    res = 7
  else
    res = 8
  i++
print res

There are a lot of straightforward ways to extend this proof-of-concept - the standard interval analysis rules, situational constant folding (so far, we only fold situationally-constant Gamma predicates, as is necessary for the above), and adding more situations (e.g. to encode loop invariants; branches of a Gamma).

This PR is very cool, however I'm a little concerned about adding all these situation / predicate combos. We need to think carefully about how to bound the number of these. For example, interval analysis is nice because you get exactly one interval per class. We should restrict it more if possible.

@oflatt Thanks for the review! I agree that these could easily blow up if we are not careful about how they interact. I commented out the parts you suggested removing
I think the main appeal of an analysis like this is that -- similar to eqsat -- we can write a small number of orthogonal rules that interact in useful and surprising ways. Blowup for now, as there are few situations and we don't create terms (while many other optimizations do). I am inclined to explore this more and find what rules are useful before worrying too much about blowup. Now, it's linear, so it's in fast-analyses, but if it gets slower we can just move to another ruleset and bound the number of iterations. Note that the example test case only required a single iteration.


Invalid value returned by AbstractTagImport::getLocationAttribute()

Issue
When getting the value of the attribute schemaLocation or location, if the value starts with http://, we only get the value after :// => //.
This is due to the fact that by default AbstractAttributeHandler::getValue() returns the attribute value without the namespace which is suffixed by :.
How to fix
When calling AbstractAttributeHandler::getValue(), true must be passed as first argument to avoid namespace handling.

only close when merged in master


drivers: dp: swdp_bitbang: power optimization

This patch changes GPIO initialization
to be in PORT_OFF state by default.
Also, if no transceiver is attached to the signals, the GPIOs are configured to be disconnected to preserve power.
I tested this on a prototype board where we are going to have a debugger in a low-power context.

@fabiobaltieri @henrikbrixandersen @mniestroj @tmon-nordic @jfischer-no Could you have a look?

You folks probably wants to set a maintainers area for this.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 86 ms

Resolved: ClamAV performance has improved in a1de2f6.


Adding tensorboard logging

@ananis25 - Is this a right interface for custom anomaly?  Idea is that user might want to define their custom measure which they want to observe

custom initialize and custom check functions are mutating attributes on a foreign object, which is not great. Maybe use a generator as an interface, since you are performing a reduction over a stream of input data (logs). Generators might feel uncomfortable for many users though.

def custom_metric(inputs, outputs, gts=None, extra_args={}):
    self.initial_acc = None
    self.acc_arr = []
    self.count = 0
    self.thres = 0.02
    self.window_size = 200
    self.is_drift_detected = False

    while True:
        self.count += 1
        self.acc_arr.append(outputs[0]==gts[0])
        if self.count == self.window_size:
            self.initial_acc = sum(self.acc_arr)/self.window_size
        if (self.initial_acc is not None):
            recent_acc = sum(self.acc_arr[-self.window_size:])/self.window_size
            self.plot_scalars('custom_metrics', {
                'initial_acc': self.initial_acc,
                'recent_acc': recent_acc,
            }, self.count)
            if (self.initial_acc - recent_acc > self.thres) and (not self.is_drift_detected):
                    print("Concept drift detected with custom metric at time: ", self.count)
                    self.is_drift_detected = True
        yield

# call it as
for _ in custom_metric():
    ...

cfg should also have a defined schema, where you specify what properties are legal and can be used. Could use the pydantic library to define those.


In SPDX output, have PackageLicenseDeclared give the license directly instead of creating an unnecessary LicenseRef.

Describe the Feature
In SPDX output, have PackageLicenseDeclared give the license directly instead of creating an unnecessary LicenseRef.
Use Cases
It will be easier for tools to process the SPDX files.
Implementation Changes
One would need to be able to identify a license already in SPDX format.
Tern is run with the following command:
tern report -f spdxtagvalue -i danmcni/danm-cni-plugins:latest -o danm-cni-plugins.spdx

danm-cni-plugins.spdx.txt
In the SPDX output file, you have the following:
PackageLicenseDeclared: LicenseRef-c66410f

and
LicenseID: LicenseRef-c66410f
ExtractedText: <text>Original license: GPL-2.0-only</text>

The license here is correctly detected and the name is the one found in the SPDX license list.
So it would be much better to output:
PackageLicenseDeclared: GPL-2.0-only

Thanks for the issue @vargenau. Unfortunately, we can't always use the direct license value as we have no way of checking currently if the license is an approved license on the SPDX license list. Additionally, sometimes licenses that are on the approved license list are not reported using the correct identifier and we would need a way to interpret these causes and augment these licenses to their correct license identifier. Therefore, we have to default to using a LicenseRef currently.
Your suggestion would certainly be a good enhancement and we have an issue open for this: https://github.com/tern-tools/tern/issues/281. There was also talk in the SPDX community a few years ago about creating a license mapping repo but not much work has been done on this yet. I think the repo was created but no commits have been added.

Hi Rose,
The following Python code should be able to do the job: https://pypi.org/project/spdx-lookup/.
Are you aware of it?

I can work on this.

Hi Rose,
The following Python code should be able to do the job: https://pypi.org/project/spdx-lookup/.
Are you aware of it?

I was not aware, thank you! We will try to use this.

Hi,
I have found a better SPDX license expression parser:
https://github.com/nexB/license-expression
To install it:
pip install license-expression

Then you have to apply the following patch to file formats/spdx/spdxtagvalue/package_helpers.py
12a13
> from license_expression import get_spdx_licensing
13a15,19
> def is_spdx_license_expression(license):
>     '''Return True if the license is a valid SPDX license expression, else
>     return False'''
>     licensing = get_spdx_licensing()
>     return licensing.validate(license).errors == []
56,58c62,68
<     if mapping['PackageLicenseDeclared']:
<         block += 'PackageLicenseDeclared: {}\n'.format(
<             spdx_common.get_license_ref(mapping['PackageLicenseDeclared']))
---
>     declared = mapping['PackageLicenseDeclared']
>     if declared:
>         if is_spdx_license_expression(declared):
>             block += 'PackageLicenseDeclared: ' + declared + '\n'
>         else:
>             block += 'PackageLicenseDeclared: {}\n'.format(
>                 spdx_common.get_license_ref(declared))

This will produce a PackageLicenseDeclared as proposed above.
What remains to do is to remove the now useless LicenseID and ExtractedText that are still generated, I did not find an easy way to do it.
Also, the is for the SPDX tag;value format only, I did tot check what needs to be done for the SPDX JSON format.
Tell me if this is useful and can be integrated.

Thank you @vargenau , it was very useful. It's addressed for both tag/value and json in #1159.


[#8] Add an option to fold subtrees of already compared derivations

Problem: User may want to skip derivations, that has only already compared input below
Solution: Add new option to the DerivationDiff, add transformation, that will fold such subtrees, and add new cli option to enable it.

Also, let me know whenever you'd like to me to cut a release


Add Rafael Roman to Contributors list

Hey thanks this repo!

Hi  @raftael, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


 It would be nice to have an indication of the last commit as a date or a human indication (such as "two years ago"). This is quite essential information to know how alive a project is.

Good suggestion, thanks!

Maybe if the last commit is more than eg. two years ago, it could decrease their ranking. Take for example draw2d. The number one entry is the deprecated repository for more than three years. The project moved to github (as stated on https://code.google.com/p/draw2d), but the github repo one is ranked second.
The risk is that now more people might now use the deprecated repo as it ranks higher.

Yeah, good example. A fast thing that can be done is to low rank code.google.com since it is going to be closed.

I've just decreased ranks of all packages served in Google code because any active projects should be migrated to Github or some other service sooner or later. Results for "draw2d" now have github.com/llgcode/draw2d as the first result.
Systematically utilizing last commit date is  not a trivial thing, I'll consider it in next update.
Thanks for the suggestion again!

Just displaying the last commit date would be already a great help. That
could be step one. Later you can decide how it affects rankings.
Op 18 jul. 2015 06:46 schreef "David Deng"<EMAIL_ADDRESS>
I've just decreased ranks of all packages served in Google code because
any active projects should be migrated to Github or some other service
sooner or later. Results for "draw2d" now have github.com/llgcode/draw2d
as the first result.
Systematically utilizing last commit date is not a trivial thing, I'll
consider it in next update.
Thanks for the suggestion again!
—
Reply to this email directly or view it on GitHub
https://github.com/daviddengcn/gcse/issues/23#issuecomment-122483521.


Improve documentation

Changes to get started (installation) and introduction docs pages. Monaco 2.0.

@didiladi @CruzanCaramele @UnseenWizzard @patrickpichler any chance we can review this?


Error Invalid Time

Hey guys.
I have a problem when I finish running my tests, it uploads it to x-ray, but it is giving an error as shown in the screenshot below

Hey @leonardolnovaes,
thanks for creating the issue. Would you mind sharing the following information?

Your cypress version
Your cypress-xray-plugin version

I have never encountered this problem. What would help the investigation immensely are your actual run results. You can obtain them like this:
// cypress.config.js
const { defineConfig } = require("cypress");
const fs = require("fs");

module.exports = defineConfig({
    e2e: {
        setupNodeEvents(on, config) {
            // other stuff
            // ...
            on("after:run", (results) =>
                // Write run results to JSON file, but skip the config containing sensitive data.
                fs.writeFileSync(
                    "resultsRun.json",
                    JSON.stringify(results, (k, v) => (k === "config" ? undefined : v), 2)
                )
            );
        },
    },
});

Feel free to obfuscate any test titles and file names/paths. The most important values would obviously be the time related ones.

My Cypress version is 13.3.0 and my cypress-xray-plugin version is 3.3.3, this resultsRun.json is not being created when running giving the error, I can't get any other type of return to send here. I'll keep trying to debug this

Can you share your Cypress configuration file? Feel free to replace sensitive data (Jira URL, any environment variables, ...) with garbage.

Thank you, it solved my problem!!
I needed to update the plugin, after that, I had to change some things but everything worked fine!
Thank you friend

Glad to hear that!


Update the filename that we're extracting before it's done extracting rather than after.

This makes the filename more accurate and may help in case of errors.

Forgot that this was already like that.


AboveTustin screenshot is not correct with dump1090-fa

Screenshots with dump1090-fa are not correct - only show table of aircraft. No map.
https://twitter.com/AboveWaterford/status/1003662513899278336

This issue pertains to a third party application and should be posted to the AboveTustin issue tracker.
https://github.com/kevinabrandon/AboveTustin/issues


FB.Canvas.setSize

Hi,
any plans to include a ready to include (link) .js-file wo easily get this to work:
https://developers.facebook.com/docs/reference/javascript/FB.Canvas.setSize/
You need to load the JS SDK and then it is simple.
Would be nice to have one maintained file to just script-link in an get a auto sized canvas/tab.
Here like I use it with asynchronous load of the JS SDK:
window.fbAsyncInit = function() {
FB.Canvas.setSize();
};
(function() {
var e = document.createElement('script');
e.src = document.location.protocol + '//connect.facebook.net/en_US/all.js';
e.async = true;
document.getElementById('fb-root').appendChild(e);
}());

yep, it's planned to do that!
https://github.com/ckrack/fbootstrapp/blob/master/TODO.md

there should be a default example set of functions to get this going - is there a dev branch to see changes? i wouldnt mind pushing some examples ;)

Yeah i agree.
The dev branch is here:
https://github.com/ckrack/fbootstrapp/tree/development
It's currently on the same status as the master branch.
If you want to push that'd be great.
The examples are the examples/fb-* files.
These are supposed to be the starting point.
I'd prefer to have the javascript code seperate in js/application.js


Please provide argument or configuration to support multiple CUDA devices

Current set up defaults to GPU 0, and will run out of memory on a single GPU <= 8GB VRAM.  Ideally there is some configuration to support specifying the GPU to use, or a number of GPUs to use.

Thanks for your interest. We will take your suggestions into consideration and may update the multi-GPU version of the inference code in the future.

Oh, is it just this?  CUDA_VISIBLE_DEVICES=0
Change 0 to 0,1,...

Currently only the first GPU is used by default