enzyme is not a binary

According to README.md, this should be built like this:
$ cargo install enzyme
$ enzyme-install

Executing cargo install enzyme fails however, since enzyme is a library, not a binary.
    Updating crates.io index
error: there is nothing to install in `enzyme v0.1.0`, because it has no binaries
`cargo install` is only for installing programs, and can't be used with libraries.
To use a library crate, add it as a dependency in a Cargo project instead.

I am using cargo 1.58.0-nightly (40dc28175 2021-12-06)
Running cargo install --path <path to Cargo.toml> works fine. Is the version on crates.io the correct one?

Indeed, crates.io only had the old Readme which someone used to reserve the name.
I've prepared everything for the final release, but then decided to wait for the weekend and my new SSD,
in order to test the build on a fresh Ubuntu, to see if I missed some dependencies (spoiler: I did).
I've published it right now.


spread operator ignores order when used as an arguments list

here is an example
const f = function(x: string, y: number) {
    return;
}
const a = ['5', 6];

//this works
f.apply(null, a);

/* but this gives 2 errors:
* "string. This type is incompatible with number "
* and
* "number. This type is incompatible with string"
*/
f(...a);

It looks like Flow works with a list of arguments as an unordered list with
live example

This works in the latest version of Flow 👍
https://flow.org/try/#0MYewdgzgLgBAZjAvPArmYUCW4AUAPALhmgCdMwBzAGhgE8iwUBbAIwFMSBKGAbwCgYgmCTZQUJMAG4+AXz6hIsAIZIYAbQDkAVg00AbAF1pfAPQmoAC0wQYAdxAkA1hD5wAdEoAOngDa0cjD4+NEqcxiYAVDAsKLCW1jAUmABubDYATDAcJA4QBHxRAESk5BRuMAAqVjZQtJ5sMAnkoEyeSlgsPg22mJYwjKwcMIUFMEpgACajhQPsJOVVCbX1jTbNIK3tmJ3dvRbEUGSUIxEmrjhul6HSQA


Show currently-computed path on saveinput if valid

re-use space used by the validation error label
Show fully-resolved path, but try to show it relative to ~ if that works.
e.g.:

user types ./foo, show ~/projects/data_project/foo if cwd is ~/projects/data_project
user types ./foo, show /tmp/abc123/foo if cwd is /tmp/abc123

see https://github.com/tconbeer/harlequin/discussions/511

see this event handler:
https://github.com/tconbeer/textual-textarea/blob/a4c3291c77c48192f2f2c1aa2d8fbdc988d15bf3/src/textual_textarea/text_editor.py#L1132-L1139


V2 Stackdriver Telemetry: STS call credential support

V2 Stackdriver telemetry is required to support STS for fetching call credentials to support workload Identity.

[ ] 1. Update Opencensus CPP to support STS for SD tracing and metrics
[ ] 2. Update upstream Envoy to use STS SD Tracing
[ ] 3. Update istio-proxy SD metrics extension to use STS SD metrics.
[ ] 4. Update Istio-proxy SD logging and context graph extension to pass thru Envoy STS call credentials.
[ ] 5. Update istio integration tests using reference PR

@JimmyCYJ

Thanks @mandarjog
cc @liminw

merged: https://github.com/istio/proxy/pull/2638  STS SD changes in master.


godep save -r doesn't rewrite dependencies in Godeps/ until godep update

So, I did a godep save -r and my dependency tree got updated. I have deps on A, B, and C in my code.
Except one of my deps, let's call it A, depends on another dep I had in my tree B. In my code, my A and B import were rewritten correctly.
But then I went to godep update C, and noticed that A's import were rewritten to use my Godeps/workspace/_src/B path.
I'm quite happy with A's imports being rewritten, I just wish it happened a) at godep save time and b) not as a side-effect of updating C.

Actually, let me try this again in another ticket. I misread which paths were getting rewritten.


--format trims one char from {{.Names}} each time it is used

Output of docker version:
$ docker version
Client:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.6
 Git commit:   20f81dd
 Built:        Mon, 04 Apr 2016 16:27:28 +1200
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.6
 Git commit:   20f81dd
 Built:        Mon, 04 Apr 2016 16:27:28 +1200
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 7
 Running: 1
 Paused: 0
 Stopped: 6
Images: 91
Server Version: 1.10.3
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 88
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: null host bridge
Kernel Version: 4.4.0-16-generic
Operating System: Ubuntu 16.04
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 15.65 GiB
Name: plata
ID: OERF:H3PM:YRZA:W34S:LM4N:QN2S:AYF4:QYYG:DFZV:JPKW:WROU:6LBO
WARNING: No swap limit support

Additional environment details (AWS, VirtualBox, physical, etc.):
Physical machine, but this issue has happened on several other machines I have tested on running on different OSes and different Docker versions.
Steps to reproduce the issue:

Run this script:

docker ps --format="
1  Names:  {{.Names}}
2  Names:  {{.Names}}
3  Names:  {{.Names}}
4  Names:  {{.Names}}
5  Names:  {{.Names}}
6  Names:  {{.Names}}
7  Names:  {{.Names}}
8  Names:  {{.Names}}
"

Describe the results you received:
Output of the above script on a machine with one container:
1  Names:  bitlbee
2  Names:  itlbee
3  Names:  tlbee
4  Names:  lbee
5  Names:  bee
6  Names:  ee
7  Names:  e
8  Names:

This behavior does not occur with the other available format vars.

that's pretty funny  :-)

Still able to reproduce this on 1.11.0-rc3

/cc @calavera

Could possibly be this; https://github.com/docker/docker/blob/3d13fddd2bc4d679f0eaa68b0be877e5a816ad53/api/client/formatter/custom.go#L231-L237

@thaJeztah yes that's it

let me have a quick look if I can fix it

@thaJeztah I just opened a possible fix in #21783

Closed by #21783


Gravity Changer crashes Minecraft

Mod Name
Gravity Changer (qouteall fork)
Mod Homepage
https://github.com/qouteall/GravityChanger
Describe the bug
Gravity Changer crashes Minecraft
Steps to reproduce
No response
Logs
https://mclo.gs/vG5itLR
Additional context
No response

Maybe it still can be fixed? A lot players wanna see this mod on forge

Would kill to use this on forge, almost not exaggerating.


Coverage QC: thresholds for completeness plot and other updates

Region coverage QC (completeness plot) currently checks a questionable number of depth cutoffs (1x, 5x, 10x, 20x, 40x, ... 100x, - step of 10, but skipping 30x for some reason). I find the following set of thresholds more reasonable: 1, 5, 10, 20, 50, 100, 250, 500, 1000, 5000, 10000, 50000 (high thresholds are only exposed if there is coverage at least at one sample), that what's we are used to use in TargQC, but let me know if you have another opinion.
Additionally into the text coverage files I added an extra cutoff of 80% of average coverage, which is not going to go into MultiQC, but I'm going to use it to parse in my oncology reports, hope you don't mind.
Other minor changes in this PR include:

fix of Duplicate pct metric for RNAseq,
expose target/genome info in MultiQC only for variant calling pipelines,
some refactoring of average coverage calculation.

Vlad -- thanks again for cleaning this up to be more useful for high depth targeted data as well. I'll work on testing this on WGS runs prior to the next release, and also will look into incorporating goleft indexcov (https://github.com/brentp/goleft/tree/master/indexcov) for areas where need estimates only and are scanning the BAM file.


Discord rate-limiting

Discord's APIs are rate-limited. As a result, we'll have to figure out how we can make sure that every request can still be processed as expected by the user, even if we have to delay it for some time to work around the rate limits. - Discord rate-limits
Countermeasures
Add a global rate limit handler to reschedule calls that couldn't be completed due to rate limiting.
Provide a global rate limiter for scheduling calls that will likely get rate limited:
A rate limiter implementation could look something like this and would of curse also be customizable by the user.
/**
* Custom handlers of a rate limiter need to
* implement the minimal features of this interface
*/
interface IRateLimitHandler {
  /**
  * Call the `cb` and check if it was rate limited.
  * If so retry the method call after `x` delay.
  *
  * The framework will also use this method
  * by default if a command or event route
  * call is being rate limited.
  *
  * @returns A `Promise` that resolves once the
  * `cb` was successfully called.
  */
  retryOnLimit(cb: () => void): Promise<void>;
  /**
  * When having to make a lot of API calls to
  * delete channel messages or update a bunch
  * of users you can use this method as a way
  * to schedule them. You provide an array of
  * inputs which are then being used to execute
  * the callback function. Items are processed
  * every `x` interval or if we know for sure
  * we are not getting rate limited.
  *
  * @returns A `Promise` that resolves once all
  * arguments were successfully processed.
  */
  scheduleBulk<T>(args: T[], (arg: T) => void): Promise<void>;
}

The Rate Limit handler should be available via DI and overwrote by using a custom provider.

I assume Discord.js is wrapped all in one place as you go down lower and lower in levels of abstraction -- when you get to the point where you actually dispatch the commands to their API, why not do it with a queue system? You could just try it and then if it fails with a 429 you can just not remove it from the queue. If Discord gives a Retry-After header you can even use that.

That sounds like a plan. To do this though we'd first need an internal client as a replacement for Discordjs and basically re-implement the features it has which is going to take a lot of time. For the start, I guess we'll just go with what's already there from Discordjs.


Card should handle value values correctly in initial rendering

When the form is partially pre-filled with address data, it should display this on the card.
I can't figure out how to get it take the pre-fill data and use it, tabbing around the form causes the image to update, but triggering clicks, focus, change, etc does not. Also, it only updates the image with pre-fill data from a field you tab into focus, not tabbing out (blur).
What's the easiest way to update the image with data place in the form from the DB?

the answer isn't that clear. I have same problem. Help me please


perf(utils): improve getProp(), setProp(), delProp(), setGetter() performance

I found that on a huge site, the execution time of String.split() / extractPropKey() is very long.
function extractPropKey(key) {
  return key.split('.');
}

(https://github.com/hexojs/warehouse/blob/master/lib/util.js#L22-L24)
A benchmark using a hexo dummy website (with 300 posts): (clone https://github.com/SukkaLab/hexo-many-posts.git)

extractPropKey() has been called for 7,201,086 times.

For input values, they are all simple keys without "."

3325949  tag_id
2474477  post_id
 894848  _id
 178803  name
 131535  category_id
  17070  path
  16950  permalink
  16936  slug
  14418  length
  12964  date
  12490  posts
   9722  source
   7588  modified
   7468  hash
   6580  parent
   5084  published
   4460  updated
   4460  title
   4460  raw
   4460  layout
   4460  full_source
   4460  comments
   4460  _content
   4446  tags
   4446  photos
   4446  link
   4446  categories
   4446  asset_dir
   3820  more
   3820  excerpt
   3820  content
   3174  id
    120  renderable

A benchmark using a hexo dummy website (with 1500 posts): (clone https://github.com/SukkaLab/hexo-many-posts.git 5 times)

Execution Time of extractPropKey(): 19.78s

Solution
This PR skips the call to split() and for-loops, handle simple keys directly.

@SukkaW
Here is data from test/benchmark.sh ran on Travis.  (master, this pr)
A benchmark using a hexo dummy website (with 300 posts):
Node.js 8

Total time (Cold)
Save DB (Cold)
Memory (Cold)
Total time (Hot)
Save DB (Hot)
Memory (Hot)

Current Master Branch
24.228s
0.611s
584.676MB
15.823s
0.667s
767.297MB

This PR
21.134s
0.562s
578.789MB
13.698s
0.557s
753.648MB

Node.js 10

Total time (Cold)
Save DB (Cold)
Memory (Cold)
Total time (Hot)
Save DB (Hot)
Memory (Hot)

Current Master Branch
25.258s
0.780s
589.504MB
15.836s
0.777s
789.211MB

This PR
18.150s
0.576s
609.074MB
11.686s
0.591s
746.203MB

Node.js 12

Total time (Cold)
Save DB (Cold)
Memory (Cold)
Total time (Hot)
Save DB (Hot)
Memory (Hot)

Current Master Branch
24.923s
0.784s
573.012MB
14.703s
0.783s
808.902MB

This PR
19.895s
0.596s
547.336MB
11.947s
0.712s
807.457MB

Node.js 13

Total time (Cold)
Save DB (Cold)
Memory (Cold)
Total time (Hot)
Save DB (Hot)
Memory (Hot)

Current Master Branch
25.070s
0.707s
589.926MB
15.299s
0.716s
807.445MB

This PR
22.133s
0.620s
584.203MB
13.568s
0.717s
824.426MB


Specify ncpu/workers/processes for each parallelisation step

Currently the pipeline uses the one less than the number of CPUs available for all parallel operations.
This is silly because

the user should be able to specify how many are used in general (e.g. when running on a shared machine)
it is not necessarily efficient to base the number of workers on the number of cores available.

For example, the forced fitting parallelisation is limited foremost by IO (on nimbus using >~ 10 workers slows things down), but also by memory (each worker loads an image, background map and rms map or ~2.1GB of data in total, into RAM)
We should therefore add arguments in the config file for the user to specify how many workers should be used for each parallel computation.

Closed by #777


[BUG] - Withdrawing rewards documentation has wrong amount in tx-out

Internal
Summary
The tx-out amount in https://github.com/input-output-hk/cardano-node/blob/2c74060d02ec03c8b5a92a53447ca58c582d988c/doc/stake-pool-operations/withdraw-rewards.md#build-the-raw-transaction doesn't include the reward amount.
Should be 743882981 (194054070 + 550000000 - 171089) instead of 193882981.

hi @CarlosLopezDeLara please can you have a look at this one

Thanks, fixed in:
https://github.com/input-output-hk/cardano-node/pull/1893


center should be half size

ANCHOR_CENTER is exactly same as ANCHOR_SE in the current version. Should it be + 128 rather than + 256?
https://github.com/muety/pyquadkey2/blob/12e44a46f2ee6a8ce9c68b51e9c4e10408116943/src/pyquadkey2/quadkey/tilesystem/tilesystem.pyx#L73

Yeah, this is a heck of a bug. It's a shame that this repository was forked from the original specifically because it was unmaintained, and now this one has also gone unmaintained. C'est la vie.

It's a shame that this repository was forked from the original specifically because it was unmaintained, and now this one has also gone unmaintained. C'est la vie.

Contributions are always welcome...


Updating index.html

Modifying "WireMock Admin" to "WireMock Admin"

Thanks. Unfortunately this is dead code that I'm guilty of not removing :-(
This has reminded me I need to clean it up.


K8SPG-407 allow perconapgclusters/finalizers in role

CHANGE DESCRIPTION
Problem:
You can find description in this PR https://github.com/percona/percona-helm-charts/pull/229
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are the manifests (crd/bundle) regenerated if needed?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PG version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

demand-backup
passed

init-deploy
passed

monitoring
passed

operator-self-healing
passed

scaling
passed

scheduled-backup
passed

self-healing
passed

start-from-backup
passed

telemetry-transfer
passed

users
passed

We run 10 out of 10

commit: https://github.com/percona/percona-postgresql-operator/pull/492/commits/6275e68295295cae4c14b1b35ab462193ee5d7fa
image: perconalab/percona-postgresql-operator:PR-492-6275e6829

will be fixed under https://github.com/percona/percona-postgresql-operator/pull/556


BSAseq parameters, source?

Hey! First off: thank you for an excellent paper! Really nice seeing this written up, and definitely something that will be useful to me and others.
In the paper, supplementary materials and online documentation I haven't managed to find the exact parameters/scripts used to run the steps of the pipeline, beyond the GUI provided by SciApps. Are these available somewhere? Will the source of the Shiny app be made available?
Once again, thank you!

Thanks for asking. Here is a detailed tutorial for running BSAseq: https://learning.cyverse.org/projects/sciapps_guide/en/latest/bsa.html
And you can load the example data from this workflow: https://www.sciapps.org/wf_id/8
The second link has all parameters preset. Generally, you only need to use the default parameters. The Shiny app is currently embedded with the workflow and it will not work without the workflow. E.g. we pre-computed everything before launching the Shiny app.


Missing API depending on core-js@2

deprecate<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.

This dependancy has caused me to abandon element-ui altogether. I can't user<EMAIL_ADDRESS>:(


Jav-508

support publish metrics via web rest —— provider-metrics, how to use please see bmi example

Coverage decreased (-0.2%) to 86.943% when pulling 63aa19f5f8936e5fb86a632f930e4b90e8eba6a1 on zhengyangyong:JAV-508 into 8eee2a2a4557bd82422205c663d12ca8f118f8eb on ServiceComb:master.

Coverage decreased (-0.2%) to 86.924% when pulling c93f9a96f7e4024d53bdf9036b983f095586eb73 on zhengyangyong:JAV-508 into 8eee2a2a4557bd82422205c663d12ca8f118f8eb on ServiceComb:master.

don't merge this PR before JAV-526 , I will rebase an improve it later


Viewing Results on SIBR VIEWER

Hi when I run the command to view results it neither gives error nor show any output in windows.
In Ubuntu it only shows static scene

Hi, are you talking about the dynamic 4DGS? Would you mind sharing your command both in Ubuntu and Windows?

Hi @lulinliu
./viewers/bin/SIBR_remoteGaussian_app.exe --port 6017
I'm using this command


Cannot change locale after setup is called

After Session.setupSharedSessionWithConfiguration is called, any change to the Configuration object will not have any effect, since Configuration is a struct, and struct is copied over.
This makes sense is most cases, except when the app allows users to change language inside app. Many travel apps allow that (Eg, Booking.com, Kaligo,...)

Thank you, I didn't think about such case. I will take a look what I can do as soon as I have time.


BaseTexture error event is not getting fired.

Expected Behavior
When a BaseTexture fails to load I expect BaseTexture to throw an error event and not stop the JS execution.
Based on: https://pixijs.download/dev/docs/PIXI.BaseTexture.html#event:error ( weirdly the documentation offers 2 different error and loaded events )
Current Behavior
BaseTexture is not firing the error event and stops my JS execution.
Steps to Reproduce
See example.
Environment

pixi.js version:  5.0.4
Browser & Version:  Chrome 75( probably not limited to )
OS & Version:  Windows 10
Running Example: https://jsfiddle.net/c7psehxj/

Excuse me. Updated the fiddle to use the error event string instead of our predefined variable.

Sorry to bump this.
But isn't there anyone that can help with this?
Or is there anything wrong with my issue?

Nothing wrong with your issue; I took a quick look and there doesn't appear to be anything linking up resources failing to load and the base texture firing an error event. Not that I can see in v5, anyway :/

Thank you for your reply.
I didn't get quit how you mean that.
If you look at the fiddle and the documentation page linked, the least I'd expect is that I get an error event fired because 'https://google.com/foo.png' is of course not defined and cross-origin which I could catch then.
But this way there is no real way to tell the basetexture souce was not able to load without a timeout.
The documentation defines an error event though.

I think in the change of how base textures work in relation to resource from pixi v4 to pixi v5, we accidentally lost the functionality of sending an error in your use case.

Ok, I've been thinking maybe I am just being dumb. 👍
So this is something that needs to be fixed then.

We definitely need test coverage for this. PR welcome to fix this and add tests in v5.


google_sign_in crash when the user cancel the signing flow

I think I found the cause. I'm using a headless runner in the app and the plugins are initialised twice once for the main isolate and again for the background isolate. This way the delegate is set for the last that initialises the plugin witch is the background isolate. So the response comes back on background where the _accountRequest is indeed nil.

I've made _accountRequest static an it worked perfectly.

Why?

oh, I thought the issue is solved. Can you confirm it's the same while using current stable and google_sign_in 4.0.7?

I'm confused by the above conversation.  It sounds like this has been resolved?  Please comment if that's not the case.  Thanks!


Actually use the pose bounding box confidence threshold. For now use …

…the default of 0.25.

Also, I provisioned the latest/"greatest" R18 models here.
Now, instead of a strong tendency for the tracker to jump way ahead, this tracker much more frequently fails to progress, especially on steps 2 & 3 for my local testing.

Style check is not happy.


Fix hostname max length (#233)

As there are a lot of path changes in 2.1.0, this PR will need to be rebased manually.


Issue generating ovpn file

The .ovpn generated file is empty. I also see nothing on the output when removing the redirection to the file :
export CLIENT_NAME=client_name
docker-compose run --rm openvpn easyrsa build-client-full $CLIENT_NAME nopass
docker-compose run --rm openvpn ovpn_getclient $CLIENT_NAME

wow it's super weird sometime the output has something and sometime nothing....
$ docker info
Containers: 2
 Running: 2
 Paused: 0
 Stopped: 0
Images: 5
Server Version: 1.13.0
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 27
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 03e5862ec0d8d3b3f750e19fca3ee367e13c090e
runc version: 2f7393a47307a16f8cee44a37b262e8b81021e3e
init version: 949e6fa
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.4.0-21-lowlatency
Operating System: Ubuntu 16.04.1 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 462.4 MiB
Name: arwen
ID: AK7Z:YSOR:KLHB:T2UZ:O7NC:QLTR:TR45:UBRB:BHD3:2U65:DOH2:SKYH
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

$ dc version
docker-compose version 1.10.0, build 4bd6f1a
docker-py version: 2.0.2
CPython version: 3.5.2
OpenSSL version: OpenSSL 1.0.2g  1 Mar 2016

Seems to work out of the box using docker-compose exec openvpn instead of docker-compose run --rm openvpn.
By the way docker-compose documentation should be on the README.md (it tooks me time to find it).
Beside that, great work !!!

See #212 about the doc.


Added component MatLink

MatLink is a link component styled exactly like a MatButton.
The MatButton component already has a Link parameter but it is in fact a button and doesn't have the features of a link like being able to open in a new tab, copying the link address e.t.c....
The styles are based on mdc-button styles, but extra styles are added for a class "mat-link"
It doesn't have ForceReload, Name, Value, Label and Type params which are present in MatButton, and Link param was renamed to Href. Every other parameter from MatButton is present.

@enkodellc  I deleted my fork and started over. This time, I set the PR branch to develop, and I edited the matButton.scss instead of the main css file, according to the contributing tips.
What's left is the docs, but I'm not sure if I should create a seperate section for the component, or add examples for the component under the MatButton section. What do you advise?

Refer to others like Material-UI / react or Material.io. If there Link is under Button then I think we should do the same. I think it should have its own Menu Item under Navigation / https://material-ui.com/components/links/   Examples are needed for the new menu Item. Thanks again for making the changes!

I renamed the component to MatButtonLink because that's exactly what it is. It makes more sense that way. I also added ForceLoad. Check out the Demo examples.

Ok, thank you for this contribution.


[Bug]Cant change bg opacity

The bug
I can change the background tint opacity
To Reproduce
Steps to reproduce the behavior.
https://user-images.githubusercontent.com/77114710/155479835-773e3092-abd2-4d71-8508-2a48ef4976c6.mp4
Expected behavior
The bg opacity should change to make the notepads app bg blur and provide a good loo
Desktop:

OS:  Windows 10
21H2

Additional context
Notepads version : 1.4.6
Downloaded from :- Microsoft Store

Encountered the same thing, have you found any solution?

There is a system level transparency setting you need to turn on before you can change the opcacity on a per-app basis.

Thanks a lot. It would be nice if you can put that in the README.md, anyways you've made a great app and it's  free, so thank you for that again.

Thanks for answering


RBT-243 Run tests in docker before uploading

Merged in https://github.com/carsales/pyheif/pull/62


update/re-pull docker image

Hiho,
is there a way to update/change/pull docker image e.g. within a running service or on host directly over rancher ui? For me it seems a necessity to be able to do that.
One way to that would be a web console to host as mentioned in #1858. Or just a button "re-pull image".
Also (or) would it be nice to be able to clean up the host from not needed images. (docker rmi)

We just created an issue to track this concept through rancher-compose, but I will leave this one open for UI updating. https://github.com/rancher/rancher/issues/1874

+1

bumping this. A UI feature to allow for updating existing images would be great. I have not looked at the API, but I assume that option does exist.
On a positive note: Rancher is awesome! Keep it up guys!

An easy way to always pull the latest image would be to add a label to your service. This would cause it to always pull a new image onto the host before deploying the service.
io.rancher.container.pull_image: always
http://docs.rancher.com/rancher/labels/

Setting a 'io.rancher.container.pull_image: always'-label doesn't seem to work for me, at least not for private repo's. Is there something else that needs to be set for this to work?

@rayvbr What version are you running? V0.39.0+ should have pull working on private registries.

I'm running 0.41.0.
Just to be clear: it should re-pull after every restart (of both the specific container, as well as the application), right?

A restart would not cause it as you would be using the SAME container. You would need to stop the container and delete. When a new container is started, it would re-pull an image on the host before deploying a NEW container.

io.rancher.container.pull_image: always works, however I noticed that when I clone a service, that label isn't kept. Is that intentional?

@etlweather that is a bug, https://github.com/rancher/rancher/issues/2342.

I would like to propose three pull ways from the UI.

On service's options dropdown, where you could pull image on all hosts.
On stack's options dropdown, where you could pull all services images on all hosts.
On service upgrade where you could mark Pull before start (same as rancher-compose up --pull)

I'd like to close this as the label and option in the UI (checked by default) allows you to re-pull docker images.
@cusspvz
If you do an in-service upgrade and select the "always pull image" checkbox, then you would be pulling the image for any time the service is deployed on a host.

I just realized this is closed. Can we reopen it?
The label and option in the UI allows you to decide between "always pull" and "never pull". But as @cusspvz say above, how would I do the following in the UI, "For this service that does not have always pull set, pull now."?

@deitch You would do an upgrade and select the checkbox. Going forward, it would always pull before launching the container.
Can you describe a use case of when you wouldn't want always or never because I'm assuming that if you want to pull, then you'd probably want to always pull as you are re-using the same tag in Dockerhub.

Running an upgrade without changing the image would force a pull? That would work.
In the end, we decided to go for "always pull", so I added it to the tag in the compose yml, but I can see use cases where you might not want always pull, but want to do it on-demand.

@deitch Yes, you'd need to make sure you check the "pull always" checkbox under the image name. That would mean any time the container is launched, it would pull a new image to the host.

Or
  labels:
    io.rancher.container.pull_image=always

@deitch If you want to leave it at that, that option needs to be better documented. Took me a while to find out it existed.

@DamonGant me too. Actually, I found it by creating a new service (but not starting it) with the option "always pull" checked, then looking at the config. Later I found it in the docs.

All labels that are supported in rancher-compose are listed in the docs at this location:
http://docs.rancher.com/rancher/labels/

Hi. This option is sorely needed in our case. Always pulling the image isn't the same as manually triggering a pull. This is how we do a deploy:

Build a new image
Upgrade a migration service (which is simply a run-and-die service that runs the database migrations) through rancher-compose -p
Upgrade the rest of the stack through rancher-compose -p

The problem is after 1 completes but before 2 completes, we cannot have any container in the stack other than migration service running the new code base. However occasionally the migration can take up to multiple hours, and meanwhile some containers would be killed, rebalanced and recreated. If the always pull label is set, then it will start with the new code base.
Our solution right now is to stick to rancher-compose, but so far this is a CLI feature with no GUI equivalent.

The updated link for rancher labels is: http://rancher.com/docs/rancher/latest/en/cattle/labels/
@deniseschannon I didn't understand if this is possible also when launching a normal container (not a service) through the rancher API. Or do I have to pull or remove the image before that? How, I can't find a way for that too... thanks

Made a test today on latest rancher 1.6; by adding the label describe above to the API creation of a single container the image is effectively updated/pulled before starting.


Testing framework enhancements

Added windows tests
Removed main.py
Added main_test.yml which runs specified tests depending on the platforms.
Addresses #62 #60

The command mentioned in README.md should be:
ansible-playbook --inventory 'inventory.ini' main_test.yml --extra-vars ibmMqLicense=accept
as this is required for ubuntu installation.
Also, with main.py before, we ran "cleanup_test.yml" regardless of how far the test goes - I think we should keep this behaviour, so if we have a test failure on a machine, the Queue Managers being involved will be removed, allowing for a quick re-run.


Scam Thread Submission

Duplicate Submissions

[X] Yes, I checked that the name is not currently in the list
[X] I read the wiki explaining each different categories and examples before submitting

(Required) Spam Comment Direct Link
https://www.youtube.com/watch?v=--W4KF8Di3M&lc=UgxIHDn-9HFsIxeEWOt4AaABAg
(Optional) Screenshot of comments referencing the scammer names
No response
(Required) List of the scammer names
rachel from accounts

This might catch some false positives but I'll add it for now


Off Topic

Why would you open an issue here for such a topic? And sorry no, I cannot handle anybody else for I am not even able to devote enough time to learning myself either. Its great you can code but you can look up on the internet for whatever you want to learn. Please do not open any irrelevant issues here, it causes a mess in the section. I am sorry again.

I need your help please


feat: add summed fee to header

Part of off-site hacking around.
The increment of the sequencer balance based on users paying fees don't need to happen on L2, even if all the payments happen on L2. Namely, instead of making individual increments at each tx, which requires ~64 bytes of DA, we can burn and aggregate it throughout the full rollup and just spit out a single 32 bytes value for the amount that the sequencers balances must be incremented in the form of an IOU. The IOU is passed to the portal handling the fees.
From the portals POV, the sequencer have a claim on L1 for some funds. Solvency is ensures since the supply on L2 have been reduced through the burn of fee payments.
It is practically an exit that is just following a non-standard flow.
We will likely need to extend it to have summed_fee, summed_gas and base_gas. The summed_fee to be used as described above. The summed_gas to be used as part of computing the next base_fee where the current base_fee value is also used.
Also, the portal setup should practically have 2 portals for the canonical asset, but just have one of them active. If canonical having a portal that is moving active, if no longer canonical, a backup that ensure that fees can still be bridged in. We can replace the fee_recipient value from the variables with the fee_portal that is populated at the l1 contract following a simple fee_portal = is_canonical ? portal_a : portal_b;

Current dependencies on/for this PR:

master

PR #4675   👈

This stack of pull requests is managed by Graphite.

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 2dd53fd6 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
46,052
181,700
724,292

l1_rollup_calldata_gas
227,048
888,404
3,536,096

l1_rollup_execution_gas
321,635
1,008,255
3,757,906

l2_block_processing_time_in_ms
1,203
4,473 (-8%)
17,719 (-1%)

note_successful_decrypting_time_in_ms
218 (+2%)
601 (-6%)
2,147

note_trial_decrypting_time_in_ms
33.3 (+29%)
137 (+66%)
179 (+6%)

l2_block_building_time_in_ms
18,974
75,078
301,387

l2_block_rollup_simulation_time_in_ms
13,857 (+1%)
54,790
220,419

l2_block_public_tx_process_time_in_ms
5,089
20,212 (-1%)
80,731

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,679 (+1%)
30,138

note_history_successful_decrypting_time_in_ms
1,406 (-1%)
2,715 (-1%)

note_history_trial_decrypting_time_in_ms
178 (+8%)
212 (-5%)

node_database_size_in_bytes
17,293,392
33,181,776

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
330
44,696
25,441

private-kernel-ordering
211
43,761
14,659

base-rollup
1,502 (+1%)
128,144
997 (+7%)

root-rollup
83.9
4,320 (+3%)
857 (+4%)

private-kernel-inner
432
71,115
25,441

public-kernel-app-logic
261
32,190
25,283

merge-rollup
8.02 (+2%)
2,840 (+5%)
997 (+7%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,420
36,230 (+1%)

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
566 (-1%)
1,488

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.0288 (-15%)
629

Closing this for now. @just-mitch it might be useful for you later in your fee adventures.

@LHerskind tracked in https://github.com/AztecProtocol/aztec-packages/issues/5008 FYI


Popover(react-aria-components) Allow passing portal target to Popover

Closes https://github.com/adobe/react-spectrum/issues/4404
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[x] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[x] Filled out test instructions.
[x] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

Create special DOM node in your HTML document
Select the DOM node
Pass the DOM node into Popover using portalContainer prop
The content of the popover should be rendered inside the DOM node instead of document.body

🧢 Your Project:

@aspirisen Thank you for the PR! The team is concerned that other part of our code (like our positioning code) may have made assumptions and thus relies that the popover is portaled to document.body.

@LFDanLu in my particular case it is working as expected

So we actually made use of the portalContainer prop recently for a new feature in Menu where a Popover would be rendered to display contextual information for a unavailable menu item and ran into an issue where the Popover was mispositioned. Still investigating this, but for now this PR will be on hold until we can track down all the cases where positioning issues may occur when altering the portalContainer

Hi, I'm facing the same need as @aspirisen since I use the 'focus-trap-react' lib to "trap" the focus in a modal.
In this modal, I use the Select provided by react-aria-components, but since the items displayed (within Popover) are appended to document.body, they are considered outside of the "focus trap", and I can't set the focus on one of these items (using the keyboard arrows for example).
Being able to pass portalContainer with a prop to Popover would allow me to specify to append the items displayed to the modal container, and allow the focus on them.
Here is a CodeSandbox I made to illustrate my problematic : https://codesandbox.io/s/react-aria-components-select-focus-trap-react-jrq5z5?file=/src/App.tsx

Closing, addressed in https://github.com/adobe/react-spectrum/issues/4404
Thanks for your help and understanding!


VS Code puts non-breaking whitespace in copied syntax highlighted code

Issue Type: Bug
When copying text from VS Code, normal space characters are converted to non-breaking whitespace ( ) in clipboard. Due to this, if I copy paste code from VS Code -> (some other app which supports html) -> back to VS Code, the code breaks because of different space characters.
Usecase is that I regularly copy/paste code from VS Code to OneNote and then back to OneNote. I want to retain syntax highlighting but don't want the space to be converted.
VS Code version: Code - Insiders 1.48.0-insider (db40434f562994116e5b21c24015a2e40b2504e6, 2020-08-11T23:21:50.282Z)
OS version: Windows_NT x64 10.0.19041

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9700K CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.83GB (21.66GB free)

Process Argv

Screen Reader
no

VM
0%

/duplicate

Duplicate of which issue?

I think this is considered a duplicate of issue #88178


Add newrelic metrics

This PR merely adds NR APM to the project. We can build out component metrics at another time.

Looks good to me @sonnysideup


Import from µTorrent

Hi, is there a way to import my current torrents in µTorrent into PicoTorrent?

We had that way back in 0.13. Unfortunately it got removed, however you might be able to import your torrents by running the 0.13 portable version, importing your torrents from µTorrent, and then running the portable version of 0.18 from the same directory. In theory, this would work.
Let me know if you decide to try it!

The steps you gave me work! Any reason why it was removed?

Great! I initially pulled it since the plugin API went deprecated when we switched to wxWidgets. Now we have a JS API in the works and I'm thinking this should fit well in that when it's ready :)


Installation page of website should mention the required Python versions

Issue
The install page suggests:
python -m pip install -U aider-chat
If the user has only installed Python 3.13 on his Windows machine this will not work. So the user will try the detailled installation instructions where he finally learns that simply the install command needs to be different:
python -m pip install -U --upgrade-strategy only-if-needed aider-chat
This however still fails.
A hint that a different version is required would be useful. (As well as a version range specification.)
Version and model info
No response

Thanks for trying aider and filing this issue.
I'm adding a note to the install page.


bcrypt requires a no-longer-supported version of MSBUILD on windows

Trying to install a project that uses bcrypt on Windows 10 with the olderst possible build tools still supported by Microsoft, the 2013 version, installed, yields the following node-gyp error once the compile tries to run:
c:\Users\Pomax\collabproject\node_modules\bcrypt>if not defined npm_config_node_gyp
(node "C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )

Building the projects in this solution one at a time. To enable parallel build, please
add the "/m" switch.

C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.Cpp.Platform.targets(54,5):
error MSB8020: The build tools for Visual Studio 2012 (Platform Toolset = 'v110') cannot be found.
To build using the v110 build tools, please install Visual Studio 2012 build tools. Alternatively,
you may upgrade to the current Visual Studio tools by selecting the Project menu or right-click
the solution, and then selecting "Retarget solution".

[c:\Users\Pomax\collabproject\node_modules\bcrypt\build\bcrypt_lib.vcxproj]

Can bcrypt's settings be updated so that the VC++ project file that node-gyp builds uses the 2013 build tools instead?

turns out that a npm install --msvs_version=2015, with a manually updated node-gyp (as per http://stackoverflow.com/questions/38149603/npm-install-fails-with-error-c2373-with-vs2015-update-3) fixes this issue.


springbootspring-cloud-openfeign-core:3.1.4 Response compression Failure after HttpClient dependency is introduced

springboot、spirngcloud、springcloudalibaba version-dependent：
<spring-boot-version>2.6.11</spring-boot-version>
<spirng-cloud-version>2021.0.4</spirng-cloud-version>
<spring-cloud-alibaba-version>2<IP_ADDRESS></spring-cloud-alibaba-version>
<feign-httpclient-version>11.8</feign-httpclient-version>

When openfeign uses HttpClient to initiate remote calls, response compression is invalid. The response compression takes effect after the HttpClient dependency is removed
May I ask if there is a solution,Thanks

feign-gzip-demo.zipThis is my test code. After lowering the version in the global pom.xml to the version I annotated, and after annotating the bootstrap dependency out of the submodule, GZIP compression will take effect even when the httpclient dependency response is introduced

One problem with this version is that because ContentLength is always -1, the threshold for the server.min-Response-size configuration does not take effect until I add a ResonseConfig configuration class to the demo. GZIP compression is enabled regardless of the number of responders

Hello @wangliuliu1115, these versions of Spring Cloud and Spring Boot are  out of open-source support. Please upgrade to Spring Cloud 2022.0.4 or 2023.0.0 (and Java 17) and the corresponding Boot version, and if the issue persists, please provide an upgraded sample.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.


Including Azure Storage to the supported list in the sentence.

Proposed changes:

In user-guide docs, Added Azure Storage to the list of supported cloud storage in the sentence as currently only S3 and GCS are included.

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog
[ ] reformat files using black (please check Readme for instructions)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Sinha, Utkal seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for submitting this PR, we'll give it a review as soon as possible

Thanks for the edit @utkalsinha!


How to use this bot with multiple accounts

Hello everybody
Is there a way to use this bot with multiple accounts at the same time?
If yes, is there a way to scheduling this function?

Just run multiple instances of the script you want to use.


Unable to run TransformComponents

Method not found: 'System.Object Windows.UI.Xaml.Hosting.ElementCompositionPreview.GetContainerVisual(Windows.UI.Xaml.UIElement)'.
System.MissingMethodException: Method not found: 'System.Object Windows.UI.Xaml.Hosting.ElementCompositionPreview.GetContainerVisual(Windows.UI.Xaml.UIElement)'.
at TransformComponents.MainPage.MainPage_Loaded(Object sender, RoutedEventArgs e)

It sounds like you might be on an updated (Nov Update) build of Windows running the old 10240 sample. Could you try running the TransformComponents sample in the 10586 folder?

Haven't seen any traction on this, please reopen if you are still running into the above issue. Thanks!


[enhancement](jdbc catalog) Add mysql jdbc url param rewriteBatchedStatements=true

Proposed changes
Issue Number: close #xxx
When rewriteBatchedStatements=false, the JDBC driver will not merge multiple insert statements into one larger insert statement. Therefore, during the batch insertion process, each insert statement needs to be sent to the MySQL server individually, leading to a higher number of network roundtrips. Network latency could potentially be a significant factor contributing to the performance degradation. For this reason, we propose to set this parameter to true by default, to enhance the performance of prepared statement batch inserts.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 53.16 seconds
stream load tsv:          541 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size:<PHONE_NUMBER>2 Bytes


fix: stub.serve() shutdown

stub.serve() is currently unable to await app.disconnect() and stop an app, because async_utils.on_shutdown(AioClient.stop_env_client()) runs before the disconnect call and disconnects the client. (it's a race that the disconnect() never wins)
I tried a couple dozen things, but can't get around the fact that once Ctrl + C has initiated the async loop's shutdown, you can't control the ordering of the app disconnection and the client stop.
The proper fix is probably to rewrite all client usage to be done by context manager, but that's much more involved than this stop-gap I've done here.

Ref: Shutdown hook is from https://github.com/modal-labs/modal/pull/2957.

This doesn't seem 100% obvious to me – what if someone wants to call stub.serve from within their code? They might not expect the client to get torn down when stub.serve finishes. Maybe it's a weird case.

I think that's a weird case. The .serve() blocks and doesn't return, so it only really finishes when a termination is sent, such as SIGTERM or SIGINT or SIGKILL.

Would prefer to solve it in a nicer way – fine to merge this but let's at least add some comments explaining what's the purpose of this code. Would also prefer to add a test.

If it's hard to add the test, it's not the end of the world to skip it. Would be great to add some comments though!

Attempt at removing this here: https://github.com/modal-labs/modal-client/pull/108


[pigeon] Enhanced enums fields are coalesced into the next class

What package does this bug report belong to?
pigeon
What target platforms are you seeing this bug on?
Windows
Have you already upgraded your packages?
Yes
Dependency versions
pubspec.lock
  pigeon:
    dependency: "direct dev"
    description:
      name: pigeon
      sha256: "6c318e3ef13f52db30ab85ef45b4746ba33b837aeecc549eb8578f78a3dde19a"
      url: "https://pub.dev"
    source: hosted
    version: "22.7.0"

Steps to reproduce

Run dart pub add --dev pigeon
Ensure that you get Pigeon version 22.7.0
Save the attached Code Sample file to pigeon/temp.dart
Run dart run pigeon --input pigeon/temp.dart
Observe analysis errors

Expected results
Since the Pigeon file has an enhanced enum with fields, the Dart output should as well. In other words:
enum MyEnum {
  a(1),
  b(2);

  const MyEnum(this.myEnumValue)
  final int myEnumValue;
}

class MyMessage1 {
  MyMessage1({
    this.field1,
  });

  int? field1;
  // ...
}

class MyMessage2 {
  MyMessage2({
    this.field2,
  });

  int? field2;
  // ...
}

Actual results
Pigeon merges the enhanced enum's field with the preceding class. This is doubly bad if there are two enums with the same named field, as that will cause duplicate field errors in the class.
enum MyEnum {
  a,
  b,

  // missing myEnumValue
}

class MyMessage1 {
  MyMessage1({
    this.field1,
    required this.myEnumValue,
  });

  int? field1;

  int myEnumValue;  // <-- This shouldn't be here!

  // ...
}

class MyMessage2 {
  MyMessage2({
    this.field2,
  });

  int? field2;

  // ...
}

Code sample
Code sample
// pigeon/temp.dart
@ConfigurePigeon(PigeonOptions(
  dartOut: 'pigeon/temp.g.dart',
  dartOptions: DartOptions(),
))
library;

import 'package:pigeon/pigeon.dart';

class MyMessage1 {
  int? field1;
}

enum MyEnum {
  a(1),
  b(2);

  final int myEnumValue;
  const MyEnum(this.myEnumValue);
}

class MyMessage2 {
  int? field2;
}

Screenshots or Videos (None)

Screenshots / Video demonstration
[Upload media here]

Logs (None)
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.27.0, on Microsoft Windows [Version 10.0.26100.2605], locale en-US)
    • Flutter version 3.27.0 on channel stable at C:\Users\Levi\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 8495dee1fd (2 weeks ago), 2024-12-10 14:23:39 -0800
    • Engine revision 83bacfc525
    • Dart version 3.6.0
    • DevTools version 2.40.2

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at C:\Users\Levi\AppData\Local\Android\sdk
    • Platform android-35, build-tools 35.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0--11572160)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop Windows apps (Visual Studio Build Tools 2022 17.7.1)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools
    • Visual Studio Build Tools 2022 version 17.7.34009.444
    • Windows 10 SDK version 10.0.22621.0

[✓] Android Studio (version 2024.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0--11572160)

[✓] VS Code, 64-bit edition (version 1.95.3)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.102.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.26100.2605]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 131.0.6778.140
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 131.0.2903.112

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Reproducible using the code snippet provided above.

Generated code
// Autogenerated from Pigeon (v22.7.1), do not edit directly.
// See also: https://pub.dev/packages/pigeon
// ignore_for_file: public_member_api_docs, non_constant_identifier_names, avoid_as, unused_import, unnecessary_parenthesis, prefer_null_aware_operators, omit_local_variable_types, unused_shown_name, unnecessary_import, no_leading_underscores_for_local_identifiers

import 'dart:async';
import 'dart:typed_data' show Float64List, Int32List, Int64List, Uint8List;

import 'package:flutter/foundation.dart' show ReadBuffer, WriteBuffer;
import 'package:flutter/services.dart';

enum MyEnum {
  a,
  b,
}

class MyMessage1 {
  MyMessage1({
    this.field1,
    required this.myEnumValue,
  });

  int? field1;

  int myEnumValue;

  Object encode() {
    return <Object?>[
      field1,
      myEnumValue,
    ];
  }

  static MyMessage1 decode(Object result) {
    result as List<Object?>;
    return MyMessage1(
      field1: result[0] as int?,
      myEnumValue: result[1]! as int,
    );
  }
}

class MyMessage2 {
  MyMessage2({
    this.field2,
  });

  int? field2;

  Object encode() {
    return <Object?>[
      field2,
    ];
  }

  static MyMessage2 decode(Object result) {
    result as List<Object?>;
    return MyMessage2(
      field2: result[0] as int?,
    );
  }
}

class _PigeonCodec extends StandardMessageCodec {
  const _PigeonCodec();
  @override
  void writeValue(WriteBuffer buffer, Object? value) {
    if (value is int) {
      buffer.putUint8(4);
      buffer.putInt64(value);
    }    else if (value is MyEnum) {
      buffer.putUint8(129);
      writeValue(buffer, value.index);
    }    else if (value is MyMessage1) {
      buffer.putUint8(130);
      writeValue(buffer, value.encode());
    }    else if (value is MyMessage2) {
      buffer.putUint8(131);
      writeValue(buffer, value.encode());
    } else {
      super.writeValue(buffer, value);
    }
  }

  @override
  Object? readValueOfType(int type, ReadBuffer buffer) {
    switch (type) {
      case 129:
        final int? value = readValue(buffer) as int?;
        return value == null ? null : MyEnum.values[value];
      case 130:
        return MyMessage1.decode(readValue(buffer)!);
      case 131:
        return MyMessage2.decode(readValue(buffer)!);
      default:
        return super.readValueOfType(type, buffer);
    }
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 3.27.1, on macOS 15.1.1 24B91 darwin-arm64, locale en-US)
    • Flutter version 3.27.1 on channel stable at /Users/deanli/dev/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 17025dd882 (2 weeks ago), 2024-12-17 03:23:09 +0900
    • Engine revision cb4b5fff73
    • Dart version 3.6.0
    • DevTools version 2.40.2

[!] Flutter (Channel master, 3.28.0-2.0.pre.38590, on macOS 15.1.1 24B91 darwin-arm64, locale en-US)
    • Flutter version 3.28.0-2.0.pre.38590 on channel master at /Users/deanli/dev/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9fd5bddc65 (28 hours ago), 2024-12-30 10:26:27 +0800
    • Engine revision 9fd5bddc65
    • Dart version 3.7.0 (build 3.7.0-267.0.dev)
    • DevTools version 2.41.0


Test failing on GCC and Clang

Hi Bengt,
Jon has added a branch which adds GitHub actions to build this project across Visual Studio, GCC and Clang for Windows, Linux and MacOS.  We've noticed the test fails under GCC and Clang currently (please see here for details: https://github.com/jbcoe/PolymorphicValue/actions/runs/5169070294).
I have reproduced the failure locally, and will shortly raise a PR addressing one issue, but there appear to be a number of other failures which I am currently investigating.
I'll add more details to the PR and let me know if you have any questions.
Best wishes,
Antony

Thanks for the effort. I'm aware that I only compiled this on VS, and may not have added /permissive- which actually detects most of the things that otherwise causes gcc/clang compilation to fail even if MSVC compiles it. I will check out your PRs now.


Release nodejs-iot v0.1.1

This pull request was generated using releasetool.

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #10   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           4      4
=====================================
  Hits            4      4

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 6e7e443...6289f96. Read the comment docs.

@kinwa91 eugh, sorry, please re-gen the lock files


ape: Add version 3.1.0

ApE (A plasmid Editor) is a tool for analyzing DNA plasmid sequences. It supports DNA Strider, Fasta, Genbank and EMBL data formats.
NOTES:

The app is built in 32-bit.
config location: $Env:AppData\Ape

SCREENSHOTS:

What a cool program!
Here's a more up to date screenshot:


Overlay graph renderer minor tweaks.

The content of this PR has been "manually" migrated to the unify-views branch by Tobias.

[x]  ScreenVertexMath do not use AWT Ellipse class, but return ellipse parameters as a double[] array. We could even make it more general and remove the explicit dependency on OverlayVertex and just accept something that has a covariance and is RealLocalizable. I am sure this code is useful to a very wide audience.

[x] Fix the getVertexAt method per @tpietzsch comments on how it should be in synch with what is painted.

[x] As @tpietzsch suggestion: moved the ConvexPolytope methods to utility class.

[x] Also made a PR in Imglib2 for symmetric matrix inversion system, so that we can remove this code from here.

[x] Javadocs for methods and fields in OverlayGraphRenderer.

[x] Fix and use the selection actions.  …

shift SPACE to select all edges/vertices in a track.
shift PAGE_UP for track upward selection.
shift PAGE_DOWN for track downward selection.

Hi @tpietzsch
So all the content of this PR has already been integrated in the main branch unify-views.
This PR and the corresponding branch can be safely deleted.

👍 thanks!


Fix warning in RN 0.62.0+

As of React Native 0.62.0, nativeDriver must always be set in Animated.
Running this package in a RN 0.62.0+ app will result in the following error being show:
Animated: 'useNativeDriver' was not specified. This is a required option and must be explicitly set to 'true' or 'false'.
The question at this point, is whether it should be set to false, but I will leave that for you to decide.

@BaseCase @mcavaliere Could you help review and merge?


How to debug transpyle

Hi
I want to learn your lib with some demo, but I use vscode can't debug step into transpyle.transpile function, it direct going to compiled swig function, can you show some techniques for how to debug it?
https://user-images.githubusercontent.com/26156999/132944225-904b82ad-81ee-4fea-9ca6-b4ac5238f6e6.mov

I find download code and pip install -e ., then can debug into transpyle.transpile. Maybe have some options in make python package need to set.


Make from/to date range overridable in admin

e.g. France has a "5021" typo in its data, and another source (or maybe France again) has a minimum year of "21": https://data.open-contracting.org/en/publication/23
Also, some publications don't set the date correctly (they set it to the current/retrieval date), so we will need to override those as well (I think Argentina Vialidad).
I think this field is currently a property of the job (which makes sense). For the override, we would put it on the publication itself (and we'll need to remember to change the end date periodically).

We can also add an admin filter to check for future "to" dates (and/or early "from" dates).

I fixed the impossible dates by limiting Kingfisher Process's metadata endpoint to > Unix epoch and <= today.

some publications don't set the date correctly (they set it to the current/retrieval date), so we will need to override those as well (I think Argentina Vialidad).

This issue could remain open for this reason, but ...

we'll need to remember to change the end date periodically

That's too much overhead. So, we can just make a note in the quality issues.
FWIW, Vialidad seems to not have the issue anymore: https://data.open-contracting.org/en/publication/18

FWIW, Vialidad seems to not have the issue anymore: https://data.open-contracting.org/en/publication/18

Hmm well, they are not setting it to the current date but the dates are still incorrect.

Does anyone else have known incorrect dates (between 1970 and today)?

Yes, Indonesia Corruption Watch, for example https://data.open-contracting.org/es/publication/101

With the updated query in Kingfisher Process their dates will be in a reasonable range (see above commit). Will they have incorrect dates even within that range?

I think your fix will work (the files will still be segmented by all the available years though, right?)

Yeah, the exporter worker reads directly from Kingfisher Process without applying similar conditions/filters.

It also happens in Paraguay Hacienda https://data.open-contracting.org/es/publication/62 but they are not updating their data anymore anyway so I think we can leave the issue closed


Spanner: IndexError: pop from empty list when merge_array

return merger(lhs, rhs, type_)
  File ".../google/cloud/spanner_v1/streamed.py", line 259, in _merge_array
    first = rhs.pop(0)
IndexError: pop from empty list

@yixizhang Thanks for the report!
@vkedia How is it happening that the back-end is returning an empty partial response chunk?

It can happen if we decide to chunk at the boundary of an array (either top level or nested). In that case we would not mistakenly want to merge with the next Value (in case that too is an array), so we send an empty array as the first value of the next partial result set. For eg say a Row has two columns both of which are of type ARRAY<INTEGER>. The values in the first row for these 2 columns are
[1], [2]. If we decide to chunk after [1], we need to send an empty list as the first Value in next chunk otherwise you would end merging [1] and [2] to get a single value [1, 2] which is incorrect.

@vkedia OK, seems like we should have an example like that in the JSON-based acceptance tests.

Yeah I agree. I will add this.

Can this be closed?

@vkedia PR #4170 will close it when merged.


feat: add Dockerfiles for development purposes

Signed-off-by: Harikrishnan Balagopal<EMAIL_ADDRESS>

Do we also want to add GitHub actions to auto push the images? We can do it along with the versioning of the website too.

Not yet, the versioning requires a more substantial over haul of the website and possibly the theme itself.
See https://github.com/just-the-docs/just-the-docs/issues/728


Systematic Connection to SIGMA

Is your feature request related to a problem? Please describe.
Kestrel allow to use SIGMA engine as an analytics. But we may want to have a dedicated method of calling SIGMA since it is an important source of data/knolwedge for hunting.

I would like to see ability to use a SIGMA "rule" YAML (the detection section of the YAML) as the query instead of STIX. This really is an RFE for STIX Shifter as well and will log it there https://github.com/opencybersecurityalliance/stix-shifter/issues/37

A Sigma rule is logically equivalent to a STIX pattern (roughly speaking), so I assume a user would somehow name a Sigma rule in a WHERE clause:
x = GET process
    FROM ds
    WHERE rules/windows/image_load/image_load_mimikatz_inmemory_detection.yml

In some config file, the user would need to set the root of the sigma repo.


Composer Install Fails

Hello,
I'm trying to get a project up and running and I'm running into trouble with the way/generators installation.
Here's my composer.json:
{
  "require": {
    "laravel/framework": "4.2.9",
    "jenssegers/mongodb": "dev-master",
    "davejamesmiller/laravel-breadcrumbs": "~2.1.0",
    "way/generators": "dev-master"
  },
  "minimum-stability": "dev"
}

I started with laravel/framework and have been adding each requirement one at a time then running composer update each time. Everything installs fine until I add the last line:  "way/generators": "dev-master"
Then this happens:
$ php composer.phar update
Loading composer repositories with package information                                                                            Updating dependencies (including require-dev)         Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for way/generators dev-master -> satisfiable by way/generators[dev-master].
    - Conclusion: remove laravel/framework v4.2.9
    - way/generators dev-master requires illuminate/support ~5.0 -> satisfiable by laravel/framework[5.0.x-dev], illuminate/support[5.0.x-dev].
    - Can only install one of: laravel/framework[v4.2.9, 5.0.x-dev].
    - don't install illuminate/support 5.0.x-dev|don't install laravel/framework v4.2.9
    - Installation request for laravel/framework 4.2.9 -> satisfiable by laravel/framework[v4.2.9].

If I swap out "laravel/framework": "4.2.9" for "laravel/framework": "5.0.*@dev", this happens:
$ php composer.phar update
Loading composer repositories with package information                                                                            Updating dependencies (including require-dev)         Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - jenssegers/mongodb dev-master requires illuminate/events 4.2.* -> satisfiable by laravel/framework[v4.2.9, 4.2.x-dev], illuminate/events[4.2.x-dev, v4.2.0-BETA1, v4.2.1, v4.2.2, v4.2.3, v4.2.4, v4.2.5, v4.2.6, v4.2.7, v4.2.8, v4.2.9].
    - jenssegers/mongodb dev-master requires illuminate/events 4.2.* -> satisfiable by laravel/framework[v4.2.9, 4.2.x-dev], illuminate/events[4.2.x-dev, v4.2.0-BETA1, v4.2.1, v4.2.2, v4.2.3, v4.2.4, v4.2.5, v4.2.6, v4.2.7, v4.2.8, v4.2.9].
    - Can only install one of: laravel/framework[5.0.x-dev, v4.2.9].
    - Can only install one of: laravel/framework[5.0.x-dev, 4.2.x-dev].
    - don't install illuminate/events 4.2.x-dev|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.0-BETA1|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.1|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.2|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.3|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.4|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.5|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.6|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.7|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.8|don't install laravel/framework 5.0.x-dev
    - don't install illuminate/events v4.2.9|don't install laravel/framework 5.0.x-dev
    - Installation request for laravel/framework 5.0.*@dev -> satisfiable by laravel/framework[5.0.x-dev].
    - Installation request for jenssegers/mongodb dev-master -> satisfiable by jenssegers/mongodb[dev-master].

Seems like jenssegers is specifying laravel 4 - and I'm stuck.
Any thoughts on how to resolve this situation?

excelente me sirvió


TC-CNET-1.3 Matter 1.3 TE#2 incorrect verdict

Please follow the below template to file issues/concerns/feature enhancement against Test Scripts:
Summary Title:
Test script is giving an incorrect verdict, not assessing PICS declaration vs implementation correctly.
Description:
Attributes A0009 and A000A marked as FALSE in PICS declaration and PICS uploaded into tool. Attributes are true in the attribute list but the test harness has not flagged this resulting in an incorrect PASS verdict.
Associated feature is not marked as True, potentially root for the attributes not being checked correctly.
Steps to reproduce:
Logs:
Not applicable, trace was uploaded in TE#2.
Additional Info:

this test assumes the pics values are correct as provided. The pics checker test will catch such errors.

IMO, this issue should now be closed - IDM 10.4 is for PICS validation, this test assumes the pics are correct, as do ALL of the other tests in the test suite.


feat: add support for ElasticBatch job

Ⅰ. Describe what this PR does
This PR add a support for ElasticBatch,  an offline batch elastic inference job type.
The design architecture is shown in the following figure.

LGTM, thanks for your contribution!


Fix darwin arm64(m1 Mac) broken binary

Because of bug of upx on m1 Mac, we should skip upx post hook in goreleaser when build darwin arm64 binary. This can fix process killed problem in issue.

The test release in available in v1.6.1, test passed on my m1 Mac.


Rename fulfillLaunched to handlePopupLaunch

The name fulfillLaunched was exposing too much information about the
implimentation a single platform (Chrome) had for what to do when a
popup was launched.  This renames that to handlePopupLaunch and defines
that it should be called once the popup is done registering callback.
The time where it is called is moved from whenever the popup receives
a full state update to calling it as soon as all callbacks have been
registered.

@salomegeo are you fine looking at this? (if not, I can reassign it to someone else)

:+1: this is great, thanks for doing this

Reviewed 6 of 6 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from the review on Reviewable.io


The default compute rate for containers should not be editable

https://bugzilla.redhat.com/show_bug.cgi?id=1410012
Next, we should have the new category: Compute for Containers
/cc @zeari
@miq-bot add_label chargeback, bug
@miq-bot assign @gtanzillo

Note that we have two kinda-default compute rates. That breaks some UX assumptions in Chargeback/Assignment editor. But it is not big deal, bright users will be able to come up with workarounds.

@isimluk We cant have this change right now. The demand was specifically for a 'fallback' rate for container images that we can edit. We should find a better solution but we cant just disable the feature...

:+1: for 'Compute for containers'

Next, we should have the new category: Compute for Containers
Maybe also with restriction of rates in rate assignment when you select 'assign to' xxxxContainers.(if it makes sense)

@lpichler yes, absolutely. itll save a lot of confusion for users.

right now we need to disable editing on the name of this rate. since we find this specific rate by name :\


run_latest_build.sh does not deploy console

This form is for bug reports and feature requests. Major features will go through a spec process.
Feature:
OpenShift >3.7.1 deployment using ./run_latest_build.sh
Bug:
For :latest openshift, the ./run_latest_build.sh skips deployment of the Web Console.
Most probably related to: https://github.com/openshift/origin/issues/18207
For lower version (v3.7.1), the script works fine:
ORIGIN_VERSION=v3.7.1 ./run_latest_build.sh

What happened:
WebConsole can not be accessed, url https://[MY_CLUSTER]:8443/console leads to:
missing service (service "webconsole" not found)
missing route (service "webconsole" not found)

What you expected to happen:
Web console is deployed and can be accessed
How to reproduce it:

./run_latest_build.sh
access https://[MY_CLUSTER]:8443/console

@marekjelen do you mind telling me what version of oc you are using? oc version. It might be using a different version of the binary than 3.7.1.

I had success using 3.9 binaries as shown below:
[dymurray@dymurray scripts]$ oc version
oc v3.9.0-alpha.4+0d9f796-159
kubernetes v1.9.1+a0ce1bc657
features: Basic-Auth GSSAPI Kerberos SPNEGO

Server https://<IP_ADDRESS>:8443
openshift v3.9.0-alpha.4+76c1143-274
kubernetes v1.9.1+a0ce1bc657

[dymurray@dymurray scripts]$ ./run_latest_build.sh
Starting OpenShift using docker.io/openshift/origin:latest ...

The server is accessible via web console at:
    https://<IP_ADDRESS>.nip.io:8443

You are logged in as:
    User:     developer
    Password: <any value>

To login as administrator:
    oc login -u system:admin

Logged into "https://<IP_ADDRESS>:8443" as "system:admin" using existing credentials.

You have access to the following projects and can switch between them with 'oc project <projectname>':

    default
    kube-public
    kube-service-catalog
    kube-system
  * myproject
    openshift
    openshift-infra
    openshift-node
    openshift-template-service-broker
    openshift-web-console

Using project "myproject".
Now using project "ansible-service-broker" on server "https://<IP_ADDRESS>:8443".

You can add applications to this project with the 'new-app' command. For example, try:

    oc new-app centos/ruby-22-centos7~https://github.com/openshift/ruby-ex.git

to build a new example application in Ruby.
Generating a 4096 bit RSA private key
..........................................................................++
.......++
writing new private key to '/tmp/etcd-cert/key.pem'
-----
Generating RSA private key, 2048 bit long modulus
.........................................................................................+++
............................+++
e is 65537 (0x10001)
Signature ok
subject=/CN=client
Getting CA Private Key
service "asb" created
service "asb-etcd" created
serviceaccount "asb" created
clusterrolebinding "asb" created
clusterrole "asb-auth" created
clusterrolebinding "asb-auth-bind" created
clusterrole "access-asb-role" created
persistentvolumeclaim "etcd" created
deploymentconfig "asb" created
deploymentconfig "asb-etcd" created
secret "asb-auth-secret" created
secret "registry-auth-secret" created
secret "etcd-auth-secret" created
secret "broker-etcd-auth-secret" created
configmap "broker-config" created
serviceaccount "ansibleservicebroker-client" created
clusterrolebinding "ansibleservicebroker-client" created
secret "ansibleservicebroker-client" created
route "asb-1338" created
clusterservicebroker "ansible-service-broker" created

The root issue here is the oc binary in the path and has nothing to do with our script. I don't think this is a bug that should be aligned against the broker project.

@mareklibra Can you try with a recent oc binary?

Indeed, the root issue is in "old" version of oc which is in fact the latest officially released. Taking the one from 3.9 alpha 3 helps.
I think the script should either:

pull images conforming the oc version on path
or download corresponding oc for its run
or at least check&warn for incompatibility so ORIGIN_VERSION shall be explicitly set

Since the script downloads :latest images, requirement to have "latest" oc version is not obviously valid. The proper version here is the one, which was not yet released.
IMO, this issue will repeat for ongoing versions as well.


If I change the playlist or stop a song, the connection from strawberry to jack get lost

Describe the Bug

System Information

Operating system:
Strawberry version (if applicable):

Additional Context

Hi @PeterWenger, I think this was meant for https://github.com/strawberrymusicplayer/strawberry :D

System: Xubuntu 22.04
Every change of the playlist or stop the connection to the eq (calfplugin, Carla with Lv2 plugins) get lost. I can solve it when I bridge pulseaudio to jack. But  there is no solution with pipwire. Candence dos not bridge pulsaudio to Jack.

Sorry the wrong list


Make it easier for one process to report spans as multiple services

I have a process which listens on a queue with spans in our internal format and reports them to jaeger. Right now I create N tracers and it's acceptable because N is rather low (~50) - one per service, and I ignore other "process" tags like IP. And that was the simplest way.
I do that because UDPSender caches process tags and additionally process tags are taken from Tracer instance which adds them automatically.
Here's what could work:

disable sampling for a given span so that it is not reported when finished - that should work currently
somehow bypass caching of Process inside UDPSender that would allow to have just one UDPSender instance (or one per thread, one per consumer, etc.)
add new method to UDPSender which appends a span but takes additionally a Process and processBytesSize, something like:

  // old append delegates to the new method
  @Override
  public int append(Span span) throws SenderException {
    if (process == null) {
      process = new Process(span.getTracer().getServiceName());
      process.setTags(JaegerThriftSpanConverter.buildTags(span.getTracer().tags()));
      processBytesSize = getSizeOfSerializedThrift(process);
      byteBufferSize += processBytesSize;
    }
    append(span, process, processBytesSize);
}

public int append(Span span, Process process, int processBytesSize) throws SenderException {
    (...)
}

What I don't like here is the risk that passed processBytesSize won't match the real process size. But maybe it would be possible to create a separate class which contains Process and caches its size.
What do you think?

There was a discussion in OT spec repo about introducing a service tag for a very similar reason of creating spans for multiple services in one process (e.g. a routing mesh). I started thinking then how we could support it in Jaeger clients, given that we made a point of extracting process tags out of spans to avoid sending the same ones over the wire. However, once a batch reaches the collectors, each span is processed independently of the others, and process tags are currently stored in the db every time for each span.
So I think the simplest way to support multiple services per Tracer is to add a preprocessing step to the collectors that will look for a service tag on the span and if found will replace span.Process with a new Process representing that service name, e.g.
span.process = span.process.copy(service=serviceTag)

No changes will be required in the clients in this case.

Right, but that handles only the case of the "service" tag and there could be other process tags too. UI presents them differently so there would need to be some way to differentiate them from regular ones.

Sorry, I misunderstood your use case. If the spans are produced by another instrumentation, is it worth going through Jaeger client at all? Jaeger collectors have tchannel and http ports that accept spans in jaeger.thrift format, perhaps it would be a better option to just convert your spans to that format and submit directly to collectors? You would still need to do batching by different services in your queue processor, since both endpoints expect a Batch representing spans from the same service.

Well, now that you suggested it - it sounds like a better approach.
I used Jaeger client because it was convenient. Just new Span(..), report it, done (plus map of tracers per service name).

@mabn it's now possible to use your own senders via the Sender API. Would it solve your original problem? You could have one custom Sender that sends each reported Span to multiple locations.

I tried to upgrade to most recent version and use Senders - but it's problematic.
A) I can use Sender.send(Process, List<io.jaegertracing.thriftjava.Span>) but then I need to implement my own batching/flushing - so basically something like RemoteReporter.
B) On the other hand it would be nice to use Sender.append(Span) but it has more issues:

need to create Span instance with a particular context (ideally via: https://github.com/opentracing/specification/issues/81). The only way as far as I know is to use protected Span constructor - which leads to ugly workaround with "io.jaegertracing" package
each Span has a reference to Tracer instance from which Sender extracts process tags and service name. So if I want to specify process tags then I need a pool of Tracers, one for each process tags combination.

So the A) option seems fine, but basically avoids whole opentracing API and requires some extra coding (custom pooling+batching+flushing)

@mabn sorry, I should have been clearer :-/ I meant a change that has not been released yet, #449. With that, you should be able to just implement your SenderResolver/Sender and you'll have the chance of changing the Span before passing it down to an actual Sender (UdpSender/HttpSender from jaeger-thrift). Or completely bypass the jaeger-thrift Senders.

I looked at #449 but I don't think it helps here because the problems mentioned in B) remain - Span has a package-protected constructor and ThriftSender retrieves tags from a Tracer obtained from a Span.
But using Sender.send(Process, List<io.jaegertracing.thriftjava.Span>) without any opentracing types is good enough

We are currently thinking about having a better public API (#396). If there's anything you'd like to see changed as part of that task, do let me know.

Is the original request reported by this issue still relevant? Should this issue be closed?

I think it's the same use case as why the "service" tag is being proposed. But I think the simplest way for us to support service tag is by adding a rewrite rule to the collector which will split the batch and assign different Process objects to spans.


Replacing user primary key breaks 2017_11_26_015000_create_user_roles_table

Version information

Laravel: v5.8.31
Voyager: v1.2.6
PHP: 7.2
Database: sqlite3

Description
If I have a renamed primary key 2017_11_26_015000_create_user_roles_table migration breaks
Steps To Reproduce
Steps to reproduce the behavior:

Change User migration to
public function up()  {      Schema::create('users', function (Blueprint $table) {          $table->uuid('uuid')->primary();          $table->string('name');          $table->string('email')->unique();          $table->timestamp('email_verified_at')->nullable();          $table->string('password');          $table->rememberToken();          $table->timestampsTz();      });  }

update User model
 protected $primaryKey = 'uuid';  public $incrementing = false;

run php artisan migrate:fresh

See error
`
Migrating: 2017_11_26_015000_create_user_roles_table
Doctrine\DBAL\Schema\SchemaException  : There is no column with name 'id' on table 'users'.

at /var/www/test.tabletopstock.com/vendor/doctrine/dbal/lib/Doctrine/DBAL/Schema/SchemaException.php:82
78|      * @return \Doctrine\DBAL\Schema\SchemaException
79|      */
80|     public static function columnDoesNotExist($columnName, $table)
81|     {

82|         return new self(
83|             sprintf("There is no column with name '%s' on table '%s'.", $columnName, $table),
84|             self::COLUMN_DOESNT_EXIST
85|         );
86|     }

Exception trace:
1   Doctrine\DBAL\Schema\SchemaException::columnDoesNotExist("id", "users")
/var/www/test.tabletopstock.com/vendor/doctrine/dbal/lib/Doctrine/DBAL/Schema/Table.php:658
2   Doctrine\DBAL\Schema\Table::getColumn("id")
/var/www/test.tabletopstock.com/vendor/laravel/framework/src/Illuminate/Database/Connection.php:877
Please use the argument -v to see more details.
`
Expected behavior
Having changed everything necessary to use a custom Id column as per Laravel's guidelines Voyager still works.

Hello,
This error is ocurring because Voyager Migrations use the column ID as reference in the table users_roles as seen here in the migration:
// File: 2017_11_26_015000_create_user_roles_table.php Line24: $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
You probably need to change this core migration from voyager for that to work.

@orien2k is correct.  Overriding migrations is simple.  Just create one in the default migrations directory with the exact same file name, and include the changes you need.


Feature request: jpegtran support

@saschazar21 I have been testing and see many useful use cases to modify the image like cropping. Since this comes as a separate utility to do this (jpegtran), it would complement well what you have already. Probably porting this follows the same pattern as the others you have on this repo.
For more info, the usage document talks about how to use jpegtran:
https://github.com/mozilla/mozjpeg/blob/master/usage.txt
Thanks for the great work so far!

Hi @pbreah - interesting topic indeed. I'm not sure yet how to include those features though.
Maybe a separate .wasm binary or a totally separate package within this monorepo might do the trick. However, I need to think how and to what extent I may include those features. I'll leave this issue open for now and come back to it once I've come up with a solution.

I believe a separate .wasm file is the way to go. All these utilities are standalone, so it is something that would work well having a separate package on your monorepo. I have been testing several utilities from other wasm repos and all of these are just many "attempts" to deliver usable tools, but they lack the flexibility to be implemented on a real world project for the web. I already tried 5 repos, and none worked well for the project I am working on. Some of these repos have even huge .wasm files - which is not good.
You have a great structure already with these other tools, and the flexibility needed to continue to grow this repo. Adding extra tools to manipulate the image would make this monorepo a one stop place to get all you need to break free from using canvas on the browser. Plus nobody has ported jpegtran to wasm, you'll be the first one. I wouldn't hesitate to help with this, but don't know how much I could help without c++ experience to write glue code.
I also found this repo that may help see the tool in a standalone way and not from the same mozjpeg repo:
https://github.com/cloudflare/jpegtran.


Fix ResponseRecorder Flush

This PR adds Flush implementation to ResponseRecorder middleware.
Fixes #560
Signed-off-by: Emile Vauge<EMAIL_ADDRESS>

@emilevauge #560 is closed, is this still required ?

@vdemeester yep it is still needed. Maybe we could merge that in the master instead, not sure I want to release 1.0.3 for that...

@emilevauge yep, let's put this into master :angel:

(ah there is some conflicts with master :sweat_smile:

Hum... We have to merge #610 before ^^

Closing in favor of #776


find vault approle config secret in runtime

What does this PR do?
find secret with vault credentials on startup time instead of use mounted files.
TODO:

AWS
somehow support both filemount approach and this dynamic one at the same time

Screenshot/screencast of this PR

What issues does this PR fix or reference?

How to test this PR?

quay.io/redhat-appstudio/pull-request-builds:spi-operator-#
quay.io/redhat-appstudio/pull-request-builds:spi-oauth-#

closing in favor of https://github.com/redhat-appstudio/service-provider-integration-operator/pull/502. New issue created for this https://issues.redhat.com/browse/SVPI-388


Fix: error converting yaml: line 14: mapping values are not allowed

Fixing error:
  [debug] error converting YAML to JSON: yaml: line 14: mapping values are not allowed in this context
How to reproduce
Enable Elastic Search in helm chart values.yaml
global:
  conf:
    es:
      enabled: true
      hosts: "papermerge-es-elasticsearch-coordinating-only"
      port: 9200

The Output before:
  PAPERMERGE_DATABASE_HOST: "papermerge-db-primary"
  PAPERMERGE_REDIS_HOST: "papermerge-redis-master"
  PAPERMERGE_MEDIA_DIR: /data/mediaPAPERMERGE_ELASTICSEARCH_HOSTS: "papermerge-es-elasticsearch-coordinating-only"
  PAPERMERGE_ELASTICSEARCH_PORT:

The Output after that change:
  PAPERMERGE_DATABASE_HOST: "papermerge-db-primary"
  PAPERMERGE_REDIS_HOST: "papermerge-redis-master"
  PAPERMERGE_MEDIA_DIR: /data/media
  PAPERMERGE_ELASTICSEARCH_HOSTS: "papermerge-es-elasticsearch-coordinating-only"
  PAPERMERGE_ELASTICSEARCH_PORT:

@timschneider
Thank you very much for your pull request !


Define curriculum pages

Description

Create internal pages
Update breadcrumbs and tab links on curriculum pages
Update style for current tab

How to test

Go to https://deploy-preview-1824--oak-web-application.netlify.thenational.academy/beta/teachers/curriculum/secondary-geography/overview
Click on tab Unit sequence
You should see the tab content update and the new tab highlighted

Checklist

[x] Added or updated tests where appropriate
[x] Manually tested across browsers / devices
[x] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner

Looks good!
Edit: @lioneljingles just to note that the link in the PR testing instructions isn't valid - I think it's beta/teachers/curriculum

Fixed!


Add upperTersity

Closes #29

Thanks for the PR. I have fixed in with a simpler approach in commit https://github.com/desktop-dart/duration/commit/f6040f3b5b5d587d62ebde37e93f6139b9b57c0e.


Pragmatic question: working with JS Object

I'm in the process of evaluating ElixirScript for use along with the Elixir/Phoenix back-ends and JavaScript/TypeScript fronts we're developing. It is unlikely for "ElixirScript-only" front-ends to be viable for us (as I would guess is the case with many organizations) but I think we could realize great advantage writing specific components and libraries.
The major hang-up I'm hitting is JS Object/Map conversion - it looks like ElixirScript used to represent Elixir Map and Struct with standard JS Objects and Classes, but moved to Maps a few years back. Maps obviously correspond much better to Elixir, but, as I'm sure you're aware, they're essentially non-existent in the JS ecosystem, both in terms of use and support. React, Redux, Lodash... I can't recall seeing JS Maps used or supported anywhere.
Besides the annoyance and potential for error of coveting every time we cross the compilation line with an associative structure, it looks like map_to_object and object_to_map create new instances on every call, which loses us identity comparison (together with "immutable by practice") as a way of telling if something has changed in React and Redux, which is important from a performance standpoint.
If ElixirScript is preferring Elixir compatibility to JS ecosystem interop that's a totally reasonable choice - I'm sure there were tons of drawback to using Object and Class with their String/Symbol-only keys - but it does seem to lead in more of an "all or nothing" direction, which is not really somewhere we're willing to go with any JS tech.
Are there ways to work with Object in ElixirScript instead of Map to avoid the conversion and resulting identity change? Any other features or approaches for heavy interop and the Object/Map issue besides converting all the time?
Thanks, Neil.

Hi @nrser, the decision to use JS Maps over Objects was to make sure that there were no unexpected pitfalls to using maps as they are defined in Elixir and Erlang. They aren't exactly one to one.

As I said, I totally see the rationale - it matches Elixir's semantics much better, and then Elixir's structs are maps, so there's that.
I honestly wish that that native JS Map was used and supported in the JS ecosystem due to the exact funkiness that I'm sure steered you guys away from using Object, but it's not used, it's not supported, and I doubt it will ever be.
My idea for potentially using ElixirScript would be to fork and replace the Map representation with Immutable.js Map/Record. That way we get immutability (which would seem to match Elixir even better), don't have to write our own deep update functions, and get better ecosystem support (crazy, right? But native JS Map has basically nothing... I brought it up to the Lodash people one and they were like "No... Why would anyone ever use a Map?").
Was that option considered as a backing representation? Any idea how much of a project it would be to swap in?

And since I closed the issue, for anyone that comes across this with the same question, I'm drawing the answer to be:
No, no direct Object support. You convert back and forth every time and all the time.


Apply fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Coverage remained the same at 0.0% when pulling 509520efbf4902482cde25369495589f56551020 on analysis-qB3QM7 into 0fad909b47c73bb9539310b60d3ce5b0672c2664 on master.


feat: add findById method on DataPlaneSelectorService

What this PR changes/adds
Add findById method on DataPlaneSelectorService
Why it does that
Avoid unnecessary calls to getAll
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes #4210
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

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

Project coverage is 75.36%. Comparing base (7f20ba5) to head (f9730db).
Report is 283 commits behind head on main.

Files
Patch %
Lines

...plane/selector/RemoteDataPlaneSelectorService.java
84.61%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4225      +/-   ##
==========================================
+ Coverage   71.74%   75.36%   +3.62%
==========================================
  Files         919     1039     +120
  Lines       18457    20640    +2183
  Branches     1037     1152     +115
==========================================
+ Hits        13242    15556    +2314
+ Misses       4756     4573     -183
- Partials      459      511      +52

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


LUCENE-8183: Added the abbility to get noSubMatches and noOverlapping Matches

Description
This is the solution for LUCENE-8183 / #9231. The change is taken form the patch of Rupert Westenthaler in the workitem.
Solution
New Parameters:
noSubMatches: true/false
noOverlappingMatches: true/false
together with the existing onlyLongestMatch those can be used to define what subwords should be added as tokens. Functionality is as described above.
Typically users will only want to include one of the three attributes as enabling noOverlappingMatches is the most restrictive and noSubMatches is more restrictive as onlyLongestMatch. When enabling a more restrictive option the state of the less restrictive does not have any effect.
Because of that it would be an option to refactor this to an single attribute with different setting, but this would require to think about backward compatibility for configurations that do use onlyLongestMatch=true at the moment.
Algorithm
If processing of subWords is deactivated (any of onlyLongestMatch, noSubMatches, noOverlappingMatches is active) the algorithm first checks if the token is part of the dictionary. If so it returns immediately. This is to avoid adding tokens for subwords if the token itself is in the dictionary (see #testNoSubAndTokenInDictionary for more info).
I changed the iteration direction of the inner for loop to start with the longest possible subword as this simplified the code.
NOTE: that this also changes the order of the Tokens in the token stream but as all tokens are at the same position that should not make any difference. I had however to modify some existing tests as those where sensitive to the ordering
Tests
I added two test methods in TestCompoundWordTokenFilter

#testNoSubAndNoOverlap() tests the expected behaviour of the noSubMatches and noOverlappingMatches options
#testNoSubAndTokenInDictionary() tests that no tokens for subwords are added in the case that the token in part of the dictionary

In addition TestHyphenationCompoundWordTokenFilterFactory#testLucene8183() asserts that the new configuration options are parsed.

There are some complains by the ecj linter, please check your code (unused variables and/or formatting).

This is my first PR for lucene. Thank you for your patience and help. If I can improve something please let me know.