Threads unexpectedly die when using labels in github

Jenkins version: 2.46.2
Plugin version: 1.36.2
We have a problem where threads die when we have created a pull request with labels in github. In github we have set the webhook to trigger when we have a Pull Request event.
Pull request
Pull request opened, closed, reopened, edited, assigned, unassigned, review requested, review request removed, labeled, unlabeled, or synchronized.
The output from our jenkins log looks like this:
INFO: Checking PR #5,761 for edgeware/monorepo
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbTrigger handlePR
INFO: Checking PR #5,761 for job Misc/pullrequest_check
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbTrigger handlePR
INFO: Checking PR #5,761 for job Misc/pullrequest_build
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRootAction handleAction
INFO: Checking PR #5,761 for edgeware/monorepo
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbTrigger handlePR
INFO: Checking PR #5,761 for job Misc/pullrequest_check
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRepository onPullRequestHook
WARNING: Unknown Pull Request hook action: assigned
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbTrigger handlePR
INFO: Checking PR #5,761 for job Misc/pullrequest_build
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRepository onPullRequestHook
WARNING: Unknown Pull Request hook action: assigned
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRootAction handleAction
INFO: Checking PR #5,761 for edgeware/monorepo
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbTrigger handlePR
INFO: Checking PR #5,761 for job Misc/pullrequest_check
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRepository onPullRequestHook
WARNING: Unknown Pull Request hook action: labeled
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbTrigger handlePR
INFO: Checking PR #5,761 for job Misc/pullrequest_build
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRepository onPullRequestHook
WARNING: Unknown Pull Request hook action: labeled
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRootAction handleAction
INFO: Checking PR #5,761 for edgeware/monorepo
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbTrigger handlePR
INFO: Checking PR #5,761 for job Misc/pullrequest_check
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbTrigger handlePR
INFO: Checking PR #5,761 for job Misc/pullrequest_build
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRepository onPullRequestHook
WARNING: Unknown Pull Request hook action: labeled
May 09, 2017 8:54:04 AM org.jenkinsci.plugins.ghprb.GhprbRepository onPullRequestHook
WARNING: Unknown Pull Request hook action: labeled
May 09, 2017 8:54:05 AM org.jenkinsci.plugins.ghprb.GhprbPullRequest <init>
INFO: Created Pull Request #5,761 on edgeware/monorepo by kohnech () updated at: 5/9/17 8:54 AM SHA: d16ffaada79638e25db32eaa649b84dadaa63009
May 09, 2017 8:54:05 AM org.jenkinsci.plugins.ghprb.GhprbPullRequest updatePR
INFO: Pull request #5,761 was updated on repo edgeware/monorepo but there aren't any new comments nor commits; that may mean that commit status was updated.
May 09, 2017 8:54:05 AM org.jenkinsci.plugins.ghprb.GhprbPullRequest <init>
INFO: Created Pull Request #5,761 on edgeware/monorepo by kohnech () updated at: 5/9/17 8:54 AM SHA: d16ffaada79638e25db32eaa649b84dadaa63009
May 09, 2017 8:54:05 AM org.jenkinsci.plugins.ghprb.GhprbPullRequest updatePR
INFO: Pull request #5,761 was updated on repo edgeware/monorepo but there aren't any new comments nor commits; that may mean that commit status was updated.
May 09, 2017 8:54:05 AM hudson.init.impl.InstallUncaughtExceptionHandler$DefaultUncaughtExceptionHandler uncaughtException
SEVERE: A thread (Thread-68829/131576) died unexpectedly due to an uncaught exception, this may leave your Jenkins in a bad way and is usually indicative of a bug in the code.
java.lang.Error: java.io.FileNotFoundException: {"message":"Not Found","documentation_url":"https://developer.github.com/v3"}
        at org.kohsuke.github.Requester$PagingIterator.fetch(Requester.java:505)
        at org.kohsuke.github.Requester$PagingIterator.hasNext(Requester.java:470)
        at org.kohsuke.github.PagedIterator.fetch(PagedIterator.java:44)
        at org.kohsuke.github.PagedIterator.hasNext(PagedIterator.java:32)
        at org.jenkinsci.plugins.ghprb.GhprbPullRequest.tryBuild(GhprbPullRequest.java:380)
        at org.jenkinsci.plugins.ghprb.GhprbPullRequest.check(GhprbPullRequest.java:163)
        at org.jenkinsci.plugins.ghprb.GhprbRepository.onPullRequestHook(GhprbRepository.java:382)
        at org.jenkinsci.plugins.ghprb.GhprbTrigger.handlePR(GhprbTrigger.java:627)
        at org.jenkinsci.plugins.ghprb.GhprbRootAction$2.run(GhprbRootAction.java:252)
Caused by: java.io.FileNotFoundException: {"message":"Not Found","documentation_url":"https://developer.github.com/v3"}
        at org.kohsuke.github.Requester.handleApiError(Requester.java:650)
        at org.kohsuke.github.Requester$PagingIterator.fetch(Requester.java:499)
        ... 8 more
Caused by: java.io.FileNotFoundException: https://api.github.com/repos/edgeware/monorepo/pulls/5761/commits
        at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
        at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
        at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
        at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
        at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1926)
        at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1921)
        at java.security.AccessController.doPrivileged(Native Method)
        at sun.net.www.protocol.http.HttpURLConnection.getChainedException(HttpURLConnection.java:1920)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1490)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1474)
        at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:254)
        at org.kohsuke.github.Requester.parse(Requester.java:596)
        at org.kohsuke.github.Requester.access$200(Requester.java:68)
        at org.kohsuke.github.Requester$PagingIterator.fetch(Requester.java:494)
        ... 8 more
Caused by: java.io.FileNotFoundException: https://api.github.com/repos/edgeware/monorepo/pulls/5761/commits
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1872)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1474)
        at java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:480)
        at sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(HttpsURLConnectionImpl.java:338)
        at org.kohsuke.github.Requester.parse(Requester.java:586)
        ... 10 more

@prediktorn That's interesting. I'd like to ask you some more questions if you don't mind...

What version of the github-api do you have installed?
Does this happen all the time, or only when receiving events for labels?

While the plugin supports deciding whether or not to build based on label, we haven't wired it specifically to handle label events. So that's my first or second suspicion here...

We are using version 1.85 of the github-api.
It does not happen all the time, and not even all the time when receiving events for labels. It seems to happen roughly once a week for us and has been happening for a couple of months now (so on different versions of the plugins).
Could it be some kind of race condition since jenkins will receive several events (one for opening the pull request, one for each label, one for the pull request being assigned, etc) basically at the same time?
Do you know if there is a way to setup the github webhhok to only notify when a pullrequest is opened or there a new commits?

This issue is caused by the github lib on these occasions will catch an IOException and wrap it in a java.lang.Error. This plugin has a catch-all in the thread for Exceptions, but that does not catch an Error.
I've changed that to catch Throwable, which will catch both Error and Exception. I've also added some catching of Error higher up where it catches IOException.
Pull request will come

https://github.com/jenkinsci/ghprb-plugin/pull/570

#570 is merged now, closing this for now.


feat: support ignoring packages only in specific groups

Resolves #1211 (half of)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 66.52%. Comparing base (0d6255f) to head (b5714c0).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1214      +/-   ##
==========================================
+ Coverage   65.88%   66.52%   +0.63%
==========================================
  Files         168      168
  Lines       14072    10974    -3098
==========================================
- Hits         9272     7300    -1972
+ Misses       4290     3154    -1136
- Partials      510      520      +10

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

(note that the docs should probably be updated, but I want to get confirmation that we're happy with this general change first)

Note that this should also be usable to resolve #1124 - I think technically you could do that right now by e.g. "consider all packages in the test group as having a license we're happy with" but it might be better to have a special "skip" type value that does actually mean the overridden package is completely omitted from license checking code paths...

@another-rex

I'm not sure about using the models.PackageVulns type here, but that's probably something to consider in the v2 rework of the types, (we have too many types that roughly represent the same thing currently), so happy to merge as is.

I think we're on exactly the same page - overall, my view here is naming wise this suffers because of already known issues which are a major epic to be addressed in v2 and that the more important thing is this change introduces a showcase of how the configuration could be improved that is very useful to have landed before v2 so it can be part of shaping v2 itself.
A good example of that imo is with files: it could be very reasonable to add the ability to ignore specific files but that's hard because models.PackageVulns doesn't have that information - so if the new interface(s) for v2 included that (which iirc I think it will?), we'd be able to very easily support that as a field here.


Fix 2-dot direct compare to use the right base commit

For 2-dot direct compare, we should use the base commit in the title and templates, as is used elsewhere, not the common ancestor which is used for 3-dot compare.  I believe that this change should have been included in #22949.

Codecov Report

Merging #24133 (84aa181) into main (f521e88) will decrease coverage by 0.02%.
The diff coverage is 41.48%.

:exclamation: Current head 84aa181 differs from pull request most recent head 766460c. Consider uploading reports for the commit 766460c to get more accurate results

@@            Coverage Diff             @@
##             main   #24133      +/-   ##
==========================================
- Coverage   47.14%   47.12%   -0.02%
==========================================
  Files        1149     1161      +12
  Lines      151446   153773    +2327
==========================================
+ Hits        71397    72472    +1075
- Misses      71611    72783    +1172
- Partials     8438     8518      +80

Impacted Files
Coverage Δ

cmd/dump.go
0.64% <0.00%> (-0.03%)
:arrow_down:

cmd/embedded.go
0.00% <0.00%> (ø)

cmd/mailer.go
0.00% <0.00%> (ø)

cmd/manager.go
0.00% <0.00%> (ø)

cmd/manager_logging.go
0.00% <0.00%> (ø)

cmd/migrate_storage.go
5.76% <0.00%> (-0.12%)
:arrow_down:

cmd/restore_repo.go
0.00% <0.00%> (ø)

cmd/web.go
0.00% <0.00%> (ø)

models/actions/run.go
1.63% <0.00%> (-0.10%)
:arrow_down:

models/actions/runner.go
1.44% <ø> (ø)

... and 59 more

... and 159 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


UPDATE agosabe.toml

Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

Merged PRs:
#826
#998
#1298
Closed PRs:
#1883

Whitespace found in filename. Found filename namada-public-testnet-15/UPDATE agosabe.toml

Whitespace found in filename. Found filename namada-public-testnet-15/UPDATE agosabe.toml

I created new PR #3196 without whitespaces


Add argument --additional_compile_args

This allows to pass more arguemnts to the IREE compiler Example:
python my-app.py --additional_compile_args="--mlir-pretty-debuginfo --mlir-timing"

Hm, this introduced some formatting changes. Isn't there a format check in the CI? I used yapf to format the code.

Fixed the formatting issue.

Lol, this has been sitting for a while.

@monorimet do you know if this PR has been superseded by something else?

I rebased it.


Backport to DuckDB v1.0.0

Thanks @carlopi for the guidance and assistance 🦆

Error is connected to conversation at https://github.com/lmangani/duckdb-extension-clickhouse-sql/issues, where this fix works only when building vs 1.0.0. I can figure it out the proper version of this and send a PR, or a workaround is pointing in .github/workflows/MainDistributionPipeline.yml to have duckdb_version as 1.0.0.

Maybe this one?

Thanks for keeping an eye on my rookie mistakes 😉

well... not going as smooth as I was hoping this morning :) any suggestion or perhaps a little PR to help us out of this spin?

I will send a PR later / tomorrow

The problem in this PR has been fixed in the extension-ci-tools repository, in this https://github.com/duckdb/extension-ci-tools/pull/44.
Possibly closing and reopening the PR is the easiest way to re-trigger CI while having it point to the new workflow (just restarting might not be enough).
I think that if you retrigger it should work now correctly.

I need to look at this then, back in a few minutes.


Add stats for time taken by script types in the update frame

This will be handy to see in profilers (mini stats or the editor profiler) to see if there are any scripts taking significant time in the update frame while the app is running/realtime.

This will be handy to see in profilers (mini stats or the editor profiler) to see if there are any scripts taking significant time in the update frame while the app is running/realtime.

It can go complex, where script is using other script, etc. Built in DevTools profiler in Chrome and similar, does great job to profile JS execution.

It can go complex, where script is using other script, etc. Built in DevTools profiler in Chrome and similar, does great job to profile JS execution.

But it doesn't do it in realtime which is the differentiator. Being able to see spikes during testing, especially in a FPS game can be very helpful in narrowing down the cause of frame drops.

From my experience, it's hard to compete with built in profiles in case of general profiling. What I've found useful in the past is to have an API to expose some profiling functionality, and let users mark up code they need profiling info for. Example:
Profiler.start("somename");
// ... lots of code
Profiler.stop("somename");

Maybe something that we could add to mini stats?

Sure, as long as we write that data into stats class in the engine, where engine internal stats go, mini stats class can access them already, and that can be configured by the user. I guess it'd be nice to automatically pick and display all user's stats .. but that should be reasonably easy to add.


Build error related with thread_local_const_init

Hello, when I tried to build wasmtime from source code, I got a following error.
error[E0658]: use of unstable library feature 'thread_local_const_init'
   --> crates/runtime/src/traphandlers.rs:390:9
    |
390 |         thread_local!(static PTR: Cell<(Ptr, bool)> = const { Cell::new((ptr::null(), false)) });
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: see issue #84223 <https://github.com/rust-lang/rust/issues/84223> for more information
    = note: this error originates in the macro `$crate::__thread_local_inner` (in Nightly builds, run with -Z macro-backtrace for more info)

For more information about this error, try `rustc --explain E0658`.

How can I fix it?

So I built wasmtime from current master branch 5d8dd64 so that I got the error.
But when I change the version to release-0.35.1 which is the latest version now, I successfully build wasmtime.

@kimdora It's likely that your Rust compiler is a bit out-of-date. We test Wasmtime with the latest stable Rust; you should be able to build our main branch if you update (if you installed via rustup) with:
$ rustup update

If that doesn't work, let us know what rustc --version says and we can try to help further!

@cfallin , thank you for your comment! After I updated rustup that you said, I can build successfully on master branch!
Thank you!


DEV: Yield capital change information

Yield packets for capital changes so downstream processes can consume then

Coverage decreased (-0.03%) to 84.994% when pulling b5b8a77c4933855ff095f85ba1cb4da764c66e21 on yield_capital_changes2 into 3b633011c6f9a82703e274680cafcf3c28cdcfd7 on master.

Coverage decreased (-0.05%) to 85.045% when pulling 94a8530caa70c188b94494d7779ffe5eb60a1310 on yield_capital_changes2 into a937d6e6b157c29017fc42855a35c2811f5b290d on master.

Coverage decreased (-0.06%) to 85.039% when pulling 2a6035863a5f806594be5e7c118d4b9dc5ee7345 on yield_capital_changes2 into a937d6e6b157c29017fc42855a35c2811f5b290d on master.

Coverage decreased (-0.1%) to 85.0% when pulling 717a5fa76552360b7c1ac81433ef68a073465445 on yield_capital_changes2 into a937d6e6b157c29017fc42855a35c2811f5b290d on master.

Coverage decreased (-0.05%) to 85.49% when pulling 5904ecb40fd0ec7848857b1a9d2053412796c1ec on yield_capital_changes2 into 18aba63da968ea1b913ed049ff14685eb3830f5b on master.


[RFR] Improve VM search efficiency for VMWare

A more efficient way of finding VM's using pyvmomi's SearchIndex functionality (instead of getting all VM objects and iterating over them and comparing their name)

Nice, looks good.

Lint report for commit 3b03f2b9a13f68da233ec7a913108d5cec23adfb:
wrapanapi/virtualcenter.py:

:heavy_exclamation_mark: Line 151:67: W291 trailing whitespace

Please, rectify these issues :smirk: .
No commit flaws detected.
CFME QE Bot


Hard to use your Tools...Honestly

Thanks for your work!
I find it really hard to use prisma-tools because enough example code isn't provided, just bunch of interfaces. I have been trying to generate SDL from my schema.prisma file for days now with no success using pal g command, even after creating pal.json in my root directory with schema.prisma in the prisma directory relative to root.

If you using pal.json you need to pass his name to pal g command like pal g -c pal.json  if you using pal.js so you can use pal g without any custom flags in the command
Also, you not provided any errors message that sees what is wrong with you to help you

Thanks for responding so quickly. It doesn't show any error, just displayed Generating your files in just a second. I am using pal.js with content:
module.exports = {
    schemaFolder: '/prisma',
    backend: {
        models: ['Profile', 'Senator'],
        generator: 'sdl',
        onDelete: true,
        excludeQueriesAndMutations: ['aggregate']
    }
};

I ran pal g in the root directory - it generated src with the wrong Models: Course, Group, Post, Product, User...with their corresponding typeDefs and resolvers.ts that are not in my schema.prisma file

Talk to me in Prisma slack

So the problem you need to run prisma generate command first before pal g command


User Guide: Add documentation for subquery syntax

Which issue does this PR close?

Part of https://github.com/apache/arrow-datafusion/issues/3091
Rationale for this change

We should tell users that we support subqueries and how to use them.
What changes are included in this PR?

New section on subquery support.
Are there any user-facing changes?

No

Codecov Report

Merging #3132 (86392b0) into master (b1765f7) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3132      +/-   ##
==========================================
- Coverage   85.95%   85.95%   -0.01%
==========================================
  Files         291      291
  Lines       52382    52382
==========================================
- Hits        45025    45023       -2
- Misses       7357     7359       +2

Impacted Files
Coverage Δ

datafusion/expr/src/window_frame.rs
92.43% <0.00%> (-0.85%)
:arrow_down:

datafusion/core/src/physical_plan/metrics/value.rs
86.93% <0.00%> (-0.51%)
:arrow_down:

datafusion/expr/src/logical_plan/plan.rs
77.77% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fix(react): remove useless variable assigning in useIsomorphicLayoutEffect

Overview

I remove isClient in useIsomorphicLayoutEffect
close #281
PR Checklist

[x] I did below actions if need

I read the Contributing Guide
I added documents and tests.

@manudeli Thank you! 🙏
@minsoo-web
There is also the same code in test-utils/ThrowError, which should be fixed 😄

@manudeli Yes! I add co-author with @ssi02014 thanks!
@ssi02014 I remove it too! fix(react): remove useless function define in ThrowError thanks!

Cool !


seed can't be None

Specifying seed to be None as the docs say should result in a date derived seed:
--seed (optional, int, default: None): Seed for random number generation. Default is time-based.

but instead results in an error as that does not seem implemented and the function expects an int:
    def generate_image(self, seed: int, prompt: str, config: Config = Config()) -> GeneratedImage:
        ...
        latents = mx.random.normal(
            ...
            key=mx.random.key(seed)
        )  # fmt: off

the None handling is only available when called from the CLI, implemented in src/mflux/generate.py
    # Generate an image
    image = flux.generate_image(
        seed=int(time.time()) if args.seed is None else args.seed,  # see here
        prompt=args.prompt,
        config=Config(
            num_inference_steps=args.steps,
            height=args.height,
            width=args.width,
            guidance=args.guidance,
        ),
    )

I think the quickest fix for you @Morriz is to do the same in your controller of the image generator, when you don't have a value, pass in your preferred seed.
As for changing the mflux implementation, I think there's a valid argument for generate_image(...) handler to expect a not-None seed, because that's part of the "formula", whereas giving a default value for seed=None is more of a "user interface" / controller opinion.

Tnx for the update. So your suggestion will end up in your code base then, yes?

@Morriz if you're on the shell calling mflux-generate, aren't you going through the CLI that automatically converts a None to a time based seed?
Otherwise, if you're calling the Python function directly, then your code is the controller and you have agency to pass a default non-None value to generate_image(...). I guess I'm confused about the original request - did you somehow find the problem by using the CLI, or you copied the example code into your workspace, and you now have .py file on your computer calling generate_image(...) - if that's the case, you just have to pass in any int for seed arg in the .py code that you control.


Added churn analytics documentation and updated text analytics

Improved sample for text analytics and added initial churn analytics documentation.

Hi @LuisCabrer, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (luisca). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Luis, I'm unable to automatically merge this pull request. Please submit it from a clean branch.


Refine GitHub Actions guide

This PR aims to refine the guide by adding more context into GITHUB_TOKEN and it's limitations, as well as rephrasing certain sentences to use more of a third-person voice.

@alissonperez If these changes are fine with you, that's you who needs to merge it - it's against your repo :)


Immutable queries?

Currently query.append mutates the original query, which makes cumbersome to reuse a base query in multiple places.
For example, this will not work - Every time the route handler gets called, a new WHERE segment will be appended to the original baseQuery:
const baseQuery = SQL`SELECT * FROM books`;

app.get("/", (req, res) => {
  const query = baseQuery.append(SQL` WHERE id = ${req.params.id}`);
  // Some response
});

Is there any immutable way of appending?

+1
I need it!

No, the idea was to always use queries inline for readability.

That seems like a very bad excuse. When building a data access layer it is very common to share queries, especially CTEs. Adding something like #72 also doesn't remove the ability to use queries inline.

I suggest .concat() as a method name.

A ready to use alternative
const baseQuery = SQL`SELECT * FROM books`;
...
const query = SQL`${baseQuery} WHERE id = ${req.params.id}`);


Image & PDF message Fails: Error 415

Message fails when attempting to send an Image or a PDF file. This issue started happening from version 0.0.6
ImageMessage image = new ImageMessageSimpleBuilder().media(attachmentBytes).caption(message).build();
api.sendMessage(contact, image);

Aug 31, 2024 2:32:11 A.M. it.auties.whatsapp.api.ErrorHandler lambda$defaultErrorHandler$3
SEVERE: [971558117219] Socket failure at MESSAGE
java.util.concurrent.CompletionException: java.lang.IllegalArgumentException: Unexpected response status code: 415
        at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:315)
        at java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:320)
        at java.base/java.util.concurrent.CompletableFuture$UniCompose.tryFire(CompletableFuture.java:1159)
        at java.base/java.util.concurrent.CompletableFuture$Completion.exec(CompletableFuture.java:483)
        at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:387)
        at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1312)
        at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1843)
        at java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1808)
        at java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:188)
Caused by: java.lang.IllegalArgumentException: Unexpected response status code: 415
        at it.auties.whatsapp.net.HttpClient.handlePartialResponse(HttpClient.java:168)
        at it.auties.whatsapp.net.HttpClient.lambda$readPartialResponse$7(HttpClient.java:152)
        at java.base/java.util.concurrent.CompletableFuture$UniCompose.tryFire(CompletableFuture.java:1150)
        ... 6 more
Aug 31, 2024 2:32:11 A.M. it.auties.whatsapp.api.ErrorHandler lambda$defaultErrorHandler$3
WARNING: [971558117219] Ignored failure

This is a bug in the http client, I’ll have it fixed in 0.0.7. I’ll probably release it by the end of the week

Fixed in 0.0.7


Running examples

Hi,
It seemed to have installed properly but when I try to run the examples or the test plots, it gives the following error:
AttributeError: 'CompositeGenericTransform' object has no attribute 'contains_branch'

I am attaching the complete error for your reference:
prateek@prateek-VirtualBox:~/git/mpld3$ python process_testplots.py
d3 url: https://mpld3.github.io/js/d3.v3.min.js
mpld3 url: https://mpld3.github.io/js/mpld3.v0.2git.js
plotting test_line_tooltips.py
Traceback (most recent call last):
  File "process_testplots.py", line 261, in <module>
    outfile, nolaunch = run_main()
  File "process_testplots.py", line 256, in run_main
    mpld3_url=args.mpld3_url)
  File "process_testplots.py", line 189, in combine_testplots
    fig_json.extend(result.iter_json())
  File "process_testplots.py", line 155, in iter_json
    Exporter(renderer, close_mpl=False).run(fig)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 49, in run
    self.crawl_fig(fig)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 116, in crawl_fig
    self.crawl_ax(ax)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 123, in crawl_ax
    self.draw_line(ax, line)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 179, in draw_line
    force_trans=force_trans)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 96, in process_transform
    if transform.contains_branch(trans):
AttributeError: 'CompositeGenericTransform' object has no attribute 'contains_branch'

the create_example on the other hand gives a different error:
prateek@prateek-VirtualBox:~/git/mpld3$ python create_example.py
/home/prateek/git/mpld3/mpld3/utils.py:24: DeprecationWarning: mpld3.show_d3 is deprecated and will be removed.  Use mpld3.show instead
  category=DeprecationWarning)
Traceback (most recent call last):
  File "create_example.py", line 73, in <module>
    show_d3()
  File "/home/prateek/git/mpld3/mpld3/utils.py", line 25, in new_func
    return func(*args, **kwargs)
  File "/home/prateek/git/mpld3/mpld3/_display.py", line 332, in show
    html = fig_to_html(fig, **kwargs)
  File "/home/prateek/git/mpld3/mpld3/_display.py", line 226, in fig_to_html
    Exporter(renderer, close_mpl=False, **kwargs).run(fig)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 49, in run
    self.crawl_fig(fig)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 116, in crawl_fig
    self.crawl_ax(ax)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 130, in crawl_ax
    text_type=ttp)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 203, in draw_text
    force_trans=force_trans)
  File "/home/prateek/git/mpld3/mpld3/mplexporter/exporter.py", line 87, in process_transform
    data = (transform - force_trans).transform(data)
TypeError: unsupported operand type(s) for -: 'CompositeAffine2D' and 'BboxTransformTo'

Anybody got anything on this?

Hi,
I'm having a similar error (below) when I try to use fig_to_html() on my VM (Ubuntu 12.04.5 LTS). Oddly, this doesn't happen when I run it on my Mac (OS X 10.10.5). The only difference between the VM version and the Mac version is that the VM version has matplotlib.use('Agg'), as there's no display, and this is how I have previously got around it. Matplotlib is up to date (1.4.3).
Any idea why I am getting this error?
Cheers!
Chris
Traceback (most recent call last):   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1836, in __call__     return self.wsgi_app(environ, start_response)   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1820, in wsgi_app     response = self.make_response(self.handle_exception(e))   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1403, in handle_exception     reraise(exc_type, exc_value, tb)   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1817, in wsgi_app     response = self.full_dispatch_request()   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1477, in full_dispatch_request     rv = self.handle_user_exception(e)   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1381, in handle_user_exception     reraise(exc_type, exc_value, tb)   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1475, in full_dispatch_request     rv = self.dispatch_request()   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1461, in dispatch_request     return self.view_functions[rule.endpoint](**req.view_args)   File "/home/christopher/dd/dd.py", line 501, in gsn_create     return gsn.gsn('gsn/csv/%s.csv' % output_filename)   File "/home/christopher/dd/gsn.py", line 254, in gsn     return makemap('', coordslist, output_filename, quality=quality)   File "/home/christopher/dd/gsn.py", line 190, in makemap     fig_html = mpld3.fig_to_html(fig, template_type='general')   File "/usr/local/lib/python2.7/dist-packages/mpld3/_display.py", line 225, in fig_to_html     Exporter(renderer, close_mpl=False, **kwargs).run(fig)   File "/usr/local/lib/python2.7/dist-packages/mpld3/mplexporter/exporter.py", line 49, in run     self.crawl_fig(fig)   File "/usr/local/lib/python2.7/dist-packages/mpld3/mplexporter/exporter.py", line 116, in crawl_fig     self.crawl_ax(ax)   File "/usr/local/lib/python2.7/dist-packages/mpld3/mplexporter/exporter.py", line 123, in crawl_ax     self.draw_line(ax, line)   File "/usr/local/lib/python2.7/dist-packages/mpld3/mplexporter/exporter.py", line 179, in draw_line     force_trans=force_trans)   File "/usr/local/lib/python2.7/dist-packages/mpld3/mplexporter/exporter.py", line 96, in process_transform     if transform.contains_branch(trans): AttributeError: 'CompositeGenericTransform' object has no attribute 'contains_branch'

mpld3 only currently works with Matplotlib version 1.3. If you use 1.4, you'll get errors. This is due to the fact that the entire mpld3 library is basically a hack, and uses private methods in matplotlib that change unannounced from release to release. If and when matplotlib's serialization framework lands in a release, we'll be able to do this in a more stable manner. But for now it's a known bug that is unlikely to be fixed (unless someone is motivated to dig in and do it). Sorry.

That's fantastic to know - thanks so much. Seems to work fine on my VM now
(with Basemap, too) - thanks again! :)
On 19 October 2015 at 18:01, Jake Vanderplas<EMAIL_ADDRESS>wrote:

mpld3 only currently works with Matplotlib version 1.3. If you use 1.4,
you'll get errors. This is due to the fact that the entire mpld3 library is
basically a hack, and uses private methods in matplotlib that change
unannounced from release to release. If and when matplotlib's serialization
framework lands in a release, we'll be able to do this in a more stable
manner. But for now it's a known bug that is unlikely to be fixed (unless
someone is motivated to dig in and do it). Sorry.
—
Reply to this email directly or view it on GitHub
https://github.com/jakevdp/mpld3/issues/176#issuecomment-149258882.


Use Keep Below for Tiled Windows Instead of Keep Above for Floating

Symptom
Occasionally, keeping floating windows above tiled windows using "keep above" causes problems. It makes it difficult to control which layer floating windows are on – and it is floating windows that need this ability more than tiled windows – and certain types of windows that are ignored by krohnkite can end up hidden by floating windows. The solution is to instead apply "keep below" to tiled windows, leaving the normal and "keep above" layers both for floating windows.
How to Reproduce

Float a window or go to Floating layout
For example, on the desktop, click create new [text file, URL, etc.]
Dialogue is hidden by any floating windows

Expected behavior
Dialogue appears on top layer since it was most recently opened, and any floating window clicked is brought to the top
Environment
Operating System: NixOS 25.05
KDE Plasma Version: 6.2.4
KDE Frameworks Version: 6.8.0
Qt Version: 6.8.0
Kernel Version: 6.11.4 (64-bit)
Graphics Platform: Wayland
KWin Scripts: Only Krohnkite

A screenshot or video would help me model the problem more accurately.

Related issue #97

Have you tried disabling keep above for floating windows in settings?

I do want floating windows kept above tiling ones. But achieving this by applying "keep below others" to tiled windows instead of applying "keep above others" to floating windows will make the behaviour more consistent. Ignored windows, which inherently are floating, will also be kept above tiled windows – on the same layer as floating windows that are controlled by krohnkite. Doing it this way will offer more control over the layering of floating windows (e.g. manually applying "keep above others" to certain floating windows to keep them above both other floating windows and tiled windows). I assume certain types of windows have the "keep above others" property applied automatically, and doing it this way would allow those to interact normally with other floating windows.

A screenshot or video would help me model the problem more accurately.

The video in #97 demonstrates the same behaviour.

Please test this, just delete zip ending
krohnkite-<IP_ADDRESS>.kwinscript.zip


How to get a relu-activated llama2 model with finetune? any supposed finetune scripts?

thx

Thank you very much for your interest in our project. Regarding the specific fine-tuning of relu-llama, the related code and work will still require some time to be ready. If you are very interested, a basic way to reproduce our fine-tuning process is to simply change the activation function from silu to relu and then start training the model on the corpus. You will observe that the loss rapidly decreases around 1 billion tokens, and as the number of training tokens increases, the model's capabilities will be restored more and more. An important thing to note is that our corpus needs to include as many. :)

Thank you very much for your interest in our project. Regarding the specific fine-tuning of relu-llama, the related code and work will still require some time to be ready. If you are very interested, a basic way to reproduce our fine-tuning process is to simply change the activation function from silu to relu and then start training the model on the corpus. You will observe that the loss rapidly decreases around 1 billion tokens, and as the number of training tokens increases, the model's capabilities will be restored more and more. An important thing to note is that our corpus needs to include as many. :)

OK.Your achievements are great, and I'm really looking forward to the future ones. 👍


AWS regression issue

Create AWS cluster is not working correctly with current code.
Need to root the cause and fix.

Problem is gone after rebasing with latest upstream master branch.


[stable/external-dns] Fix labels max length by truncating them

Signed-off-by: juan131<EMAIL_ADDRESS>What this PR does / why we need it:
The checksum label based on the sha256sum calculation of the secret can exceed the 63 max characters limitations. This PR fixes that and it also address some issues with CloudFlare configuration
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/cc @carrodher

/ok-to-test
/lgtm


Migrate primary branch from master to main

See https://github.com/Senzing/stream-loader/issues/283

Pull requests:

https://github.com/Senzing/g2s3/pull/11


merge with ckan-cloud-operator and deploy on datagov cluster

Viderum's cluster uses ckan-cloud-operator at https://github.com/ViderumGlobal/ckan-cloud-operator
Should merge the code base with cca-operator and deploy the ckan-cloud-operator on datagov cluster

Notable improvements and fixes in ckan-cloud-operator:

Docker architecture improvements based on OKFN docker stack - https://github.com/okfn/docker-ckan - a more production-ready and stable docker image, used successfully for most production deployments.
Centralized DB / SOLR / Storage - supports backup/restore, data migrations, custom initialization / configurationg
Jenkins - running on Kubernetes with worker nodes running in separate ckan-cloud-operator pods, allowing to continuously deploy the operator code, and ensure consistent version is used in all clients.
Routers - customizable routers management, supports SSL self-registration using Let's Encrypt


Change conflict zones and towns to IDP camps and vice versa

There are instances of conflict zones and towns turning to IDP camps and vice verse. Require a function introducing these changes.

@djgroen I think that could work, but how do we define in locations.csv file? Because one location can be a conflict zone, then turn to an idpcamp. Would it have two location types? or another column to indicate idpcamp date?

locations.csv would only contain the initial location type. Conflicts.csv would then supersede it.
Alternatively we can add in a new input file type, location_changes.csv.
Would you prefer that approach instead?

@djgroen if we need to change conflicts.csv manually every iteration, then I would prefer location_changes.csv.

@cspgdds Okay, this change is now incorporated and documented. Feel free to test it out!

@djgroen thank you. I will test it and let you know if there is anything.


ignore the window arc on macos

Objective

Since #12978, apps hang son macOS when closing the window
See #13208. This is not a fix but a workaround

Solution

Ignore the window arc. The original issue didn't happen on macOS anyway

So with this change, there's a RawHandleWrapper for a window that no longer exists?
That means the RawWindowHandle contains a dangling pointer, which is definitely an issue if its actually used anywhere after that.
Not necessarily bad for a temporarily workaround (given this issue already existed) but definitely warrants more investigation.

I can confirm that this fixes the issue, but I have not been following along with the relevant changes.
I am seeing this error spam when closing windows now, however:
2024-05-04T01:24:08.745268Z  INFO bevy_winit::system: Creating new window "App" (Entity { index: 0, generation: 1 })
2024-05-04T01:24:10.548038Z  INFO bevy_window::system: No windows are open, exiting
2024-05-04T01:24:10.549105Z  INFO bevy_winit::system: Closing window Entity { index: 0, generation: 1 }
2024-05-04T01:24:10.566197Z  INFO bevy_window::system: No windows are open, exiting
2024-05-04T01:24:10.567983Z ERROR bevy_winit: Failed to send a app exit notification! This is a bug. Reason: sending on a full channel
2024-05-04T01:24:10.568591Z ERROR bevy_winit: Failed to send a app exit notification! This is a bug. Reason: sending on a full channel
2024-05-04T01:24:10.568675Z ERROR bevy_winit: Failed to send a app exit notification! This is a bug. Reason: sending on a full channel

The errors go away if I revert 15687b5062d60cdea1025d955daa8a4ce5de8e4d.

#13203 should fix the warning spam: that's a cross platform problem that's not particularly related.

I've done a bit more investigation of the issue here, with some comments about it on https://github.com/bevyengine/bevy/issues/13208.
It looks like the issue that the winit::Window is dropped on a non-main thread. On macOS, this executes on the main thread and blocks on completion of that. But it seems something is running on the main thread and blocking it, so winit::EventLoop/CFRunLoop can't make progress and actually run that?
Not sure exactly have bevy is managing what the main thread is doing here, but this sounds non-trivial to fix. So this workaround may be needed for now.

"Resources that must be dropped on the main thread" is exactly what NonSend data is for.

This isn't a resource, it's a component... so it would mean we can't put the window in a component

Dropping the winit::Window from off the main thread isn't a problem in itself, but it is a problem if something is blocking the winit::EventLoop. Then it deadlocks. (This is also true if another thread tried to call other winit methods.)`

closing in favour of #13236


Fix duplicate name error

Fixes #46
Users can now add food items with the same name

Codecov Report

Merging #69 (bf962d1) into master (7f27bfe) will decrease coverage by 0.53%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #69      +/-   ##
============================================
- Coverage     73.84%   73.30%   -0.54%
+ Complexity      480      470      -10
============================================
  Files            80       79       -1
  Lines          1426     1416      -10
  Branches        136      132       -4
============================================
- Hits           1053     1038      -15
- Misses          327      332       +5
  Partials         46       46

Impacted Files
Coverage Δ

...va/seedu/nutrigoals/logic/commands/AddCommand.java
100.00% <ø> (ø)

...va/seedu/nutrigoals/model/meal/UniqueFoodList.java
77.77% <ø> (-15.08%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Move scholarships from code schools to "Scholarships" page.

On our scholarship page, which needs to be renamed to simply "Scholarships", there are no actual scholarships. They are on the code schools page. That doesn't follow. If I'm searching for scholarships and there is a scholarship page, my first instinct is to go there.
Also, the drop down menu in "What We Do" needs to be reordered by popularity. People come to us for code schools, scholarships, and then everything else. Thus, code schools and then scholarships should be ordered in the drop down menu 1 and 2 then so on. (See #489)
Please see the video for further visual clarification.
http://screencast.com/t/0ykBDdxuS

#489 created to reorder the menu.

Closing due to redesign


iphone x

on iphone X viewport is set to 354x724 instead of 375x812
css rules fail to determine it's iphone x
when AdMob.showBanner is called viewport switches to correct values

This problems only occurs when using cordova-plugin-wkwebview-engine, not with the default (UI)webview?

It's working now and I can't find the reason. I even works without viewport-fit=cover.
The only plausible explanation is that reinstalling the plugin did the trick.

Best bugs are those that solves themselves 🥇

I'm experiencing a similar issue where the height is incorrect on launch, but is corrected when I rotate the screen to landscape and back to portrait. Unfortunately, it's not resolving itself :/


(maint) Update for GCC 4.9.2

Fix signed/unsigned comparisons
Update Windows build dependencies

Also updates other build packages.

o.O with regards to those warnings; every one is a literal that can promoted to unsigned without any possibility of an incorrect comparison.  Also, I was building with GCC 4.9 on Arch for quite some time; is this warning coming only from Windows builds?

I was only testing on Windows, but I agree it seems overly pedantic.

I'll try a Linux build with 4.9.2 to compare.

I could also disable the warning.

Seems like a useful warning generally (especially when not comparing to positive signed literals), so I'm hesitant to disable it.  I can live with fixing up the warnings; curious as to why I wasn't seeing it on Linux though; perhaps it was changed in a minor release?

CLA signed by all contributors.

@stahnma should I relabel this RE-4468?

Should I merge or wait for response from @stahnma?

Please don't change the first token. The jira/git reconciliation script only knows about maint/docs/packaging and the project name (here FACT). So if there's some request to include an out-of-project jira reference, just put it after the first token.
Also AFAIK RE doesn't do any automated harvesting of RE ticket #s from git commits.

@haus isn't completely against it

That clarified nothing for me :)

I think wait for a response. I may rework this into two PRs, one for Boost/git/ruby, another for GCC and yaml-cpp.

Closed in favor of #954 until RelEng is ready to advance to GCC 4.9.2.


fix for json data opened in editor not the same as data in cell issue

This PR fixes #17657
we need to retrieve data from the data source that is being displayed.
before:

after:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 14 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 42.357%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/query/browser/gridPanel.ts
0
14
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
27570

Relevant Lines:
60842

💛  - Coveralls


Unhandled exception. System.EntryPointNotFoundException: Unable to find an entry point named 'objdetect_CascadeClassifier_newFromFile' in shared library 'OpenCvSharpExtern'.

Hey, @sdcb I know your library only includes some modules. My question is - how could I expand it?
I'm having trouble running OpenCVSharp4 on Debian 12, and your runtime is closest thing I could find.

Hey, @sdcb , I think this issue is a duplicate of #2 . Chances are that both projects are running .Net 8, and that is where the issue comes from.
You can see the description of the related warning here: https://learn.microsoft.com/en-us/dotnet/core/tools/sdk-errors/netsdk1206
You might want to move to portable RIDs.

OpenCVSharp4 on Debian 12 you can try the ubuntu 22.04 version


Fixing http proxy bug in direct mode

Closes #13524

UT to cover scenario?
@kirankumarkolli - we do have ConnectionConfigTests for this scenario, however it is missing the proxy API usages.

@simplynaveen20  - can you please add setProxy() API to ConnectionConfigTests - that should cover the testing for this scenario.

UT to cover scenario?

Added reflection method invocation in our ReflectionUtil to test this

UT to cover scenario?
@kirankumarkolli - we do have ConnectionConfigTests for this scenario, however it is missing the proxy API usages.

@simplynaveen20 - can you please add setProxy() API to ConnectionConfigTests - that should cover the testing for this scenario.
setProxy() with e2e test wont work as client initialization will fail looking for dummy proxy, added unit test instead


Is it possible to query local field resolvers as well as Prisma resolvers from within my server?

Here's a simplified scenario that demonstrates the problem I'm currently having. I have a Yoga server with a prisma-binding instance on context for communication with my Prisma service.
I created a field resolver which is just a calculated field for my User type that returns the user's full name based on the firstName and lastName fields in Prisma. From the playground or my web app, the following query works as expected.
query {
  me {
    id
    firstName
    fullName
  }
}

Since fullName isn't a field in Prisma, Yoga is able to use schema delegation to look at its own resolvers to add that value to the returned user object after the firstName and lastName are returned from Prisma.
The issue I'm having is that if I want to run a similar query from within one of my Yoga resolvers, we can't perform a similar schema delegation, because I'm not passing an info object to enable the delegation.
const user = await ctx.db.query.user({ where: { id } }, '{ id firstName fullName }');
This throws an error because fullName is not a resolver in the Prisma service and there's no way for Yoga to look for local resolvers after getting the firstName and lastName values back from Prisma without the info object to give it context. So my question is, how can I achieve the ability to query both Prisma resolvers and Yoga resolvers with a single query?

Hey @sreisner depends on how you are generating schema. If you can share a reproduction that would be great. We @the-guild-org are new maintainers of this project and actively working on v2. Would love to get your feedback on beta release @graphql-yoga/node@beta!


Handling database restart in app

Hello,
I am dealing with a disaster recovery scenario planning. When my app with this driver is running, and the database pod/server restarts, the connection starts failing on:
'{"error":true,"errorNum":11,"errorMessage":"not authorized to execute this request","code":401}'
What is the suggested recovery scenario in this case that would be in line with the code design.
The default implementation uses AddSingleton. I am aware this is solvable using AddScoped but that then defeats the purpose of having a reusable object.

what kind of replication are you using?
active fail over is no longer a thing since arango 3.12
and the cluster mode should handle this automatically.
I began rudimentary work to support multiple server endpoints (and switch on failure), however since for "real" production you ought to use cluster I never finished it.

ah maybe I misunderstood. are you even using replication?
if not you just have to fix your jwt secret in the settings

Oh its not about the db. I basically mean this usecase:

start arangodb
start dotnet app
turn off arangodb
call dotnet endpoint for db query - i get cant reach db - this is fine
turn on arangodb
call dotnet endpoint for db query - i get the 401 error posted above because jwt is expired and is not renewed because Arangodb is a singleton in dotnet and old token is invalid after arangodb restart

the jwt should not expire, just because arango restarts, see statment above.
If you cannot provide a static jwt secret, then yeah it needs a PR with either a invalidate-jwt method
or it calls it automatically when auth error occurs
as of now the jwt is only invalidated when the expiration time is near (5min offset if i remember correctly)

Alright, thats a good pointer for me. I am using the normal username + password login method which on restart is invalidated, and this is exactly that, on a 401 a new jwt isnt negotiated.
I will close this issue since my question was answered, I will look into handling the 401 and potentially PRing it, but no promises :D


Go quickstarter fails with latest MRO changes

Describe the bug
The initial build of the Go quickstarter fails. This is because there are no tests after provisioning, but the MRO stuff expects some tests.
Failure is:
Recording test results
None of the test reports contained any result

To Reproduce
Provision a new Go quickstarter and watch the Jenkins pipeline.
Expected behavior
The first build should be successful
Affected version (please complete the following information):

OpenShift: all
OpenDevStack: master

I suggest to add a dummy test into the quickstarter.

I think the problem is less on the MRO side, but because the call to junit inside the component may not accept an empty (undefined) argument.

@metmajer Ah, could be! Oh well ... the "fix" solves it anyway :)


Waterfall in Dev does not display node IDs. Shows them in prod.

Thanks Tony! What was the issue?
From: Tony Boyles<EMAIL_ADDRESS>Sent: Sunday, November 10, 2019 10:59 PM
To: CDCgov/MicrobeTrace<EMAIL_ADDRESS>Cc: Switzer, Bill (CDC/DDID/NCHHSTP/DHPSE<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [CDCgov/MicrobeTrace] Waterfall in Dev does not display node IDs. Shows them in prod. (#179)
Closed #179https://github.com/CDCgov/MicrobeTrace/issues/179 via 19bd16chttps://github.com/CDCgov/MicrobeTrace/commit/19bd16cc57a30ee24533391e26533e4cf8e77f47.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/CDCgov/MicrobeTrace/issues/179?email_source=notifications&email_token=AHMTDBLAQJONLBLFV5GXJOTQTDJ7RA5CNFSM4JJHBGJKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUYMG4AA#event-2786618880, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHMTDBMH4KC2N7PMMCUYY43QTDJ7RANCNFSM4JJHBGJA.

Wwhile back we had a user with an id column in her node list that wasn't the unique identifier. So we changed the internal node identifier field from id to _id. But I forgot to update the Waterfall view to be aware of that change.

Thx!
From: Tony Boyles<EMAIL_ADDRESS>Sent: Wednesday, November 13, 2019 8:44 AM
To: CDCgov/MicrobeTrace<EMAIL_ADDRESS>Cc: Switzer, Bill (CDC/DDID/NCHHSTP/DHPSE<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [CDCgov/MicrobeTrace] Waterfall in Dev does not display node IDs. Shows them in prod. (#179)
Wwhile back we had a user with an id column in her node list that wasn't the unique identifier. So we changed the internal node identifier field from id to _id. But I forgot to update the Waterfall view to be aware of that change.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/CDCgov/MicrobeTrace/issues/179?email_source=notifications&email_token=AHMTDBIY62CVDVSO6MWFJUDQTQADFA5CNFSM4JJHBGJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED6F4EI#issuecomment-553410065, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHMTDBN6Y2REK65AV7H55ALQTQADFANCNFSM4JJHBGJA.


Analyze data on e-government across countries

Add to the Introduction both text and a figure highlighting international comparisons on e-government development across regime types, possibly including some light regression analysis.  A possible source is the UN's E-Government Development Index.

I looked into this using the UN E-Gov Development  EDGI score and Freedom House Regime Typology of: Free, Partially Free and Not Free.  https://publicadministration.un.org/egovkb/Data-Center  I asked Bing to do some crude analysis here are the findings:

Free countries tend to have higher EGDI scores than partly free or not free countries in all regions, except for MENA where the difference is not significant.
Very high EGDI scores are mostly found in free countries, especially in Europe and Asia-Pacific. No not free country has a very high EGDI score.
Low EGDI scores are mostly found in not free countries, especially in Africa. No free country has a low EGDI score.
Partly free and not free countries have similar EGDI scores in some regions, such as Americas and MENA, but not in others, such as Africa and Asia-Pacific.

I also explored correcting for GDP Per Capita, which seems to correlate with e-government development, even when their economic development is taken into account, this doesn't seem to make a difference.
However, there are specific, striking example you can cite of authoritarian regimes scoring higher than democratic governments:
UAE, is 13th, Kazakhstan is 28th, and Saudia Arabia is 31, ahead of:
Canada 32, Italy 37, Brazil 49, and Mexico 62
This data can be cited as: United Nations Department of Economic and Social Affairs. E-Government Knowledge Database, 2022. https://publicadministration.un.org/egovkb/Data-Center.

I think a key question here is controlling for level of development.  Free countries are richer than unfree ones.  My guess is controlling for income, unfree ones are higher EDGI.
From: Jordanu12 @.>
Sent: Thursday, August 31, 2023 3:31 PM
To: pluralitybook/plurality @.>
Cc: pluralitybook @.>; Author @.>
Subject: Re: [pluralitybook/plurality] Analyze data on e-government across countries (Issue #15)
I looked into this using the UN E-Gov Development EDGI score and Freedom House Regime Typology of: Free, Partially Free and Not Free. https://publicadministration.un.org/egovkb/Data-Center I asked Bing to do some crude analysis here are the findings:

Free countries tend to have higher EGDI scores than partly free or not free countries in all regions, except for MENA where the difference is not significant.
Very high EGDI scores are mostly found in free countries, especially in Europe and Asia-Pacific. No not free country has a very high EGDI score.
Low EGDI scores are mostly found in not free countries, especially in Africa. No free country has a low EGDI score.
Partly free and not free countries have similar EGDI scores in some regions, such as Americas and MENA, but not in others, such as Africa and Asia-Pacific.

I also explored correcting for GDP Per Capita, which seems to correlate with e-government development, even when their economic development is taken into account, this doesn't seem to make a difference.
However, there are specific, striking example you can cite of authoritarian regimes scoring higher than democratic governments:
UAE, is 13th, Kazakhstan is 28th, and Saudia Arabia is 31, ahead of:
Canada 32, Italy 37, Brazil 49, and Mexico 62
This data can be cited as: United Nations Department of Economic and Social Affairs. E-Government Knowledge Database, 2022. https://publicadministration.un.org/egovkb/Data-Center.
—
Reply to this email directly, view it on GitHub https://github.com/pluralitybook/plurality/issues/15#issuecomment-1701668161 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A3BRRPTX6DUFKJG6VUNPKXTXYDQ6DANCNFSM6AAAAAAZJJIM3E .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/A3BRRPVCI52VSCRNNFW5SX3XYDQ6DA5CNFSM6AAAAAAZJJIM3GWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTFNVSUC.gif Message ID: @.*** @.***> >

Right, what I thought as well, but per above "I also explored correcting for GDP Per Capita, which correlates with e-government development; however, when economic development is taken into account and corrected for, this doesn't seem to make a difference. There is still a correlation between democracy and EGDI index.

OK, this was a cool dataset! I went with @Jordanu12 ideas and came up with two versions. I think the second one is more interesting.


Initial rendering for iphone and android

Initial rendering leaves content too wide for screen

For smaller screen sizes I changed the:

@media query to change width to 80% for smaller screen sizes
font size the intro header banner text


dev: performance regression tests

Todo:

[ ] Decide whether we really need a custom entry point

[ ] If so, make it automatically connect the correct one based on env vars

[ ] Comment and document
[ ] ... what else am I missing ?

whoops, not ready yet, nvm


feat: add default surname

add default surname for XML scheme before deposit.
a current existing plugin was an error when the family name is empty like this:
The content of element 'person_name' is not complete. One of '{"http://www.crossref.org/schema/4.3.6":surname}' is expected.
So the idea is use last section given name to provide surname when the family name is empty

Hi @forderation! The issue this is addressing is filed in Github with some discussion of a best approach here:
https://github.com/pkp/pkp-lib/issues/7528
Have you seen that issue and its discussion?

Y

Hi @forderation! The issue this is addressing is filed in Github with some discussion of a best approach here:
pkp/pkp-lib#7528
Have you seen that issue and its discussion?

yes the issue are same, but for alt-name i didn't see as well make the problem. AFAIK i already implemented in our modified ojs 3.2.1 and so far there is no problem with validation

Hi @forderation, thanks for putting this together! I've taken a look and left a few comments.
The discussion in pkp/pkp-lib#7528 recommends putting the OJS given name in the Crossref surname field if there is no family name provided in OJS. The pubmed plugin has a good example of how this has already been implemented in OJS:
https://github.com/pkp/ojs/blob/c8bba4d113a40ae3b9d997d90271b342aa8f9d29/plugins/importexport/pubmed/filter/ArticlePubMedXmlFilter.inc.php#L220-L235
Given the discussion mentioned in the issue and the pubmed example, I would recommend using a similar approach here as well. That way the same problems are solved in the same way across the code base. Thanks!


koordlet: fix BlkioReconcile file close

Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm

/lgtm


extra step line being displayed

1 extra line is being shown than the indicators.
I am using         stepIndicator.setCurrentStep();
method to set completed steps

They are considering 2 step completed and you are on the way for 3rd one.


Option to remove non-canonical chromosomes?

Some fastas have names like chr9_UnGL189 etc. that are unwanted in most cases and might cause problems downstream in the pipeline. Would be nice if you had an option that removed these.
It is trivial with pyfaidx: just read the fasta and remove all keys with "_" in them.

However, for some genomes these are the sequences you do want (scaffold_xxx). Might be not that consistent. Have to think on this.

For pro users: be able to give a regex on the cl for which sequences should be removed?

Yes, that could work. Either a negative or positive filter based on a regex.

--match or --not-match. Writing negative regexes can be hard :)
Yes, please think, measure ten times, cut once.
On Tue, May 2, 2017 at 10:54 AM, Simon van Heeringen <
<EMAIL_ADDRESS>wrote:

Yes, that could work. Either a negative or positive filter based on a
regex.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/simonvh/genomepy/issues/11#issuecomment-298563246,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ9I0tCLJtmiCWYDT7F14kQdzTwRcveZks5r1u9FgaJpZM4NLIn3
.


Trigger flows on different kubernetes clusters

Hi,
I'm currently exploring Metaflow, specifically its functionality related to Triggering Flows Based on External Events, and I'm curious about its applicability to the following use case:
Assume I have two flows F1 and F2 and kubernetes clusters A and B. Ideally I'd want to be able to execute F1 on cluster A and once it finishes, make it trigger the execution of F2 on cluster B.
Can it be done in some way?

@omriel1 we now have support for multi-cluster execution within a single flow - where different steps can run on different clusters. that would be one option to achieve your use case - where flow B runs all the steps in a different cluster but is still deployed in the original kubernetes cluster. yet another option would be to configure a shared event bus for both the clusters . there are pros and cons of each approach (as well as few other approaches). let us know which direction you would like to explore.

Thank you for your response!


Game Initiator - Arena - Game Catalog

Background
The game cards on the Arena page are styled to match the mock. However, it appears that the content within cards is positioned differently depending on the game artwork. The purpose of this ticket is to update the game card styling to be a consistent image size so that all game cards are the same size.
https://user-images.githubusercontent.com/58119095/221067751-eed45491-fca6-476d-89aa-b474a0bfb676.mov
Scenario: Update Cards
Given I am a game initiator or player

[ ] When I am in the arena
[ ] And I am viewing the game cards
[ ] Then the game cards have consistent image sizes so that all game cards are the same size

@amym114 this is prob written really funny, my brain is mush but please revise as you see fit and ping me if you have any questions.

@amandaphu This is written great imho. Makes sense to me.

noting that the content in the cards doesn't scale proportionately. discussed here and also received user feedback that they would expect the buttons to be aligned - moving to UAT


Add ability and script to test yank-examples

Fixes #25

Looks great, but should this be test_examples.py?

Oh, yes, I'll need to rename that. We have a larger issue where the minimize/iterate step is ultra slow on the CPU that is travis. Not sure how to speed it up

Are the structures good enough to just skip minimization altogether for tests?
Also, if it can make your life easier, you can just yaml.load(script), modify as a dictionary and then yaml.dump the dictionary instead of using regex. Or simply call YamlBuilder(dictionary).build_experiments(). For loading/dumping you may need to use yamlbuild.YamlLoader and yamlbuild.YamlDumper.

@andrrizzi Good idea, we can tweak once I figure out the speed issues.
I'm not sure skipping minimization is such a good idea since we want the examples to run with a little changes to the functionality as possible to show the whole thing is working. As a stop-gap, I put in code to set minimize_max_iterations: 1, but even that doesn't look like it works.
For the SMILES string tests (which are generally skipped), we may want to minimize since the structures are not going to be in any minimized position, although for small molecules its less important.

I'm not sure skipping minimization is such a good idea since we want the examples to run with a little changes to the functionality as possible

Yeah, good point! Maybe we should just run these tests on Jenkins/cluster (when that will be set up).

I think that might be the best idea. Even if we don't run the tests on travis, I want to leave the script in here and keep updating it as we add new tests

Yeah, good point! Maybe we should just run these tests on Jenkins/cluster (when that will be set up).

Still working on getting Jenkins back up!
Another possibility is to add a faster minimizer, like the Fire minimizer I'm working on inopenmoltools. It's not essential for 1.0, but it's something where we can punt on travis tests until then.

A faster minimizer is on the list as well. But I think the solution for now is to just disable running the tests (keep the file), and reenable them at a future date.

I think we're in agreement!


release: 0.5.4

Galileo new how-it-works section
document Galileo's new options
UPDATE.md -> UPGRADE.md
bump to Kong 0.5.4

If this is ready we can merge @thibaultCha


WIP: Add page for developer advocates

This adds a page at /developer-advocates which lists the current DAs, along with some other content.
Note: this adds an optional field to the team entries in the data directory; anyone with role = "developer-advocate" will show up on this page.
We are still working through the content, so this is a draft PR at the moment.
Signed-off-by: Matt Stratton<EMAIL_ADDRESS>

should this live under /about?
I don't mind that; the only thing I was thinking in my head was an easy to remember URL if we need it to rattle off to people?

should this be linked to from somewhere on the about page?
I defer to @aaronkao on this based on what his vision is for this page, but I don't mind that either :)

could we use the contact us form instead of the email link?
I already started to talk to @zchase about how we could do the form instead; I much prefer that myself. I just tossed the email link in so the WIP wasn't so empty, but ideally a form instead of having our email address out there for spam harvesting!

cool, @aaronkao when we are adding new pages id love for us to think about the navigation and how they fit into our current navigation structure. happy to help however i can.

LGTM, though defer to @aaronkao on the details.
Semi-relatedly, IIRC we've also talked about having a page at pulumi.com/community (which today still 404s -- 😢) that might be a good place to collect community resources & things; I can imaging something like this living on a page like that, too, or this page being linked from that page, or something -- just figured I'd mention that as something to think about as well.

Talking to Aaron, we think that having it under /community makes a lot of sense. I can update the PR for that.
What do we think about adding an alias in hugo so that /developer-advocates works as well as /community/developer-advocates? Just for ease of sharing the link.
Unrelated, I really would love if we had a pulumi branded shortlink thing, like pulu.mi/devadvocates
(note: .mi is not a valid TLD lol)

OK the latest commit moves the "real" URL of the page to /community/developer-advocates but also has /developer-advocates redirect to it as well:
http://pulumi-hugo-origin-pr-448-c340738f.s3-website.us-west-2.amazonaws.com/community/developer-advocates
http://pulumi-hugo-origin-pr-448-c340738f.s3-website.us-west-2.amazonaws.com/developer-advocates
We will want to put something inside /community at some point too of course, right now it just makes a relatively empty page.

What do we think about adding an alias in hugo so that /developer-advocates works as well as /community/developer-advocates? Just for ease of sharing the link.

Totally reasonable, alias away. 😄

Ugh! how did I mess this up. I rebased but I guess I should have done a merge commit; those files shouldn't show as changed. Ugh.

Since my attempts at rebasing just made things a huge mess, I'll try a new PR. Sorry!


[Feature-Request] Adding events to masonary

At the moment i'm wrapping complex components in the masonary and I could use events to the grid, eg. rendered(), rerendered(), etc...

If you're interested in changing some of the props you can you watchers.
watchers: {
  'cols': function () {
    this.$nextTick(()=> {
      this.recalculate();
    });
  },
}

This was typed on a phone so not 100% accurate but should get you there :+1:


Ran training on my custom data but generation fails with CUDA memory error.

I appear to have successfully run training on my custom tensor data, but generate.py is crashing with a CUDA out of memory error. I'm running on a 2x A100-40 machine. I can see that the model loads, but it crashes when entering the actual sampler_fn.
I also tried with device='cpu', but that almost seems to hang, consuming a huge amount of resident memory (in top), but never really progressing (from what I can tell). My input tensors are (1, 32, 2048) and are actually latents from another autoencoder, so I'm not using the VAE in Patch-Diffusion. Any thoughts?
EDIT: I'm running the EDM sampler, btw.

Hi, if it can be trained, it should be able to generate. Generation doesn't need backprop and compute any gradients so it is more GPU meomory efficient. The edm sampler should be fine with my experiments. The input tensors is in (1, 32, 2048) share, so it is not image shape? I am not sure what's happening here. But if it is able to be trained, it will be able to generate.

Yeah, it makes no sense to me...
The last line reached in your code is: eps = net(x, t, pos, class_labels).to(torch.float64) in sample_with_cfg.
I don't have class labels, so is that possibly an issue? I'm noticing that cfg and class_labels are both None when I hit that function.

In your case, if no class labels, then the cfg and class_labels are None, which are correct. Note in training eps = net(x, t, pos, class_labels).to(torch.float64) is called here https://github.com/Zhendong-Wang/Patch-Diffusion/blob/924f1ed4233e22a5917fc3fded8d65492cfc5c99/training/patch_loss.py#L76. Maybe you can check what is the difference in the inputs when you reach the two lines.

Okay, I see that during training the maximum yn/D_yn size is (32, 32), and it crashes before I get here during inference. These are presumably the patches, correct? So I think I misunderstood the generate.py code and it's trying to use (32, 2048) patches, which would explain the memory crash.
So, I commented out my changes to generate.py and it does run from my checkpoint, but it is clearly generated (32, 32) outputs. Not sure, but suspect I accidentally trained on (32, 32) crops of my input... bummer. Where should I check to modify the actual image size/shape during training (I modified the dataset to remove the "squareness" assert, but that's all.)

Actually, printing the images_ shape during training, from here:
https://github.com/Zhendong-Wang/Patch-Diffusion/blob/924f1ed4233e22a5917fc3fded8d65492cfc5c99/training/training_loop.py#L174
it is my expected shape of (32, 2048), so presumably it did train correctly. So if you could point me to what change needs to be made to generate (32, 2048), that would be awesome.

Yeah, the model could be trained on patches only. Here it shows how to allocate the probability of each patch size for efficiency requirement.  https://github.com/Zhendong-Wang/Patch-Diffusion/blob/924f1ed4233e22a5917fc3fded8d65492cfc5c99/training/training_loop.py#L153
I guess in your case you are always using the 32 patch size for training, while in our original version, we do need a small ratio of full size images to ensure good generation quality. Training only on patches would bring downgraded performance. I am not sure whether this will happen in your case, since your image is a long horizontal shape.
As for generation, in our original setting, we directly sample through the whole image, since each patch location on the image has been trained. In your case, the whole image generation would cause OOM, so you need to slice sampling. You need to move the patch window along with its locations to sample the whole image. You could also allow some overlapping in the denoising process to provide better quality.

Thanks for the reply!
I did notice a range of sizes when printing during training... I don't remember exactly where that was printing, unfortunately, but it was giving a range of different shapes (in multiples of 2). Also, I did have to hack batch_mul_dict to get it running, by adding 8: 128: i.e., batch_mul_dict = {512: 1, 256: 2, 128: 4, 64: 16, 32: 32, 16: 64, 8: 128}
I assumed that was to do with my unusual shape and the positioning of patches, but I wasn't certain. It did run and converge with that change though, so I'm guessing it was okay. Basically, I do think it trained in the way you originally intended, with varying patch sizes.
For generation all I really did was change the in the main batch loop:
        image_channel = 1  # single-channel images
        image_width = 2048  # Width of the image
        image_height = 32  # Height of the image, which is 'resolution' in this context

        # Assuming x_start and y_start are defined elsewhere in your code.
        x_pos = torch.arange(x_start, x_start + image_width).view(1, -1).repeat(image_height, 1)
        y_pos = torch.arange(y_start, y_start + image_height).view(-1, 1).repeat(1, image_width)
        x_pos = (x_pos / (image_width - 1) - 0.5) * 2.
        y_pos = (y_pos / (image_height - 1) - 0.5) * 2.

Just for some context, my inputs are latents from an audio encoder. I was interested in your model partly because of the need to improve training time and performance for small datasets, but also because it seems to me that patches could be good for music, where there tends to be a natural repetition structure that might benefit from the patch-based optimization.
Finally, yes, that makes sense what you're saying about the sampling; I didn't dig into the actual sampling code in detail, but I was surprised to hit the memory error, for sure... I get it now. If you have any tips on how to slice the sampling, that would be awesome, but otherwise thanks for the info.

I have a problem. RuntimeError: shape '[-1, 10]' is invalid for input of size 524288
When I try to run generate.py with your recomendations.


USHIFT-1297: Customize the iso in create-vm for image builder

/cc @ggiguash

Which issue(s) this PR addresses:

Closes #

@pacevedom , please also update this doc section.
https://github.com/openshift/microshift/blob/main/docs/devenv_cloud.md?plain=1#L120

/retest-required


FS Fit & Finish Testing: Issue 39

From @AnnaK111 on October 12, 2018 18:17
Submitter:  Jessica Hartono
Description:  After clicking “More organizations” CTA, landed on search result page that says: “Results for.”, which makes it looks like a mistake. This goes the same when clicking “More Articles”, “ More Videos”, etc. Can we program the title to say something related to the topic they click on? For example: when user click on “More Articles” on Housing topic page, the title on result page would say: “Results for Housing”.
Browser: Google Chrome Version 68.0.3440.106
Repro steps:  “More organizations”, “More Articles”, “ More Videos” CTA, go to search result page

Copied from original issue: Microsoft/Access2Justice#591

@grawdon Please confirm that this behavior is still desired.

@KellyeDC When I click on More Organizations now it isn't blank and shows these are for Hawaii. I think that is the desired behavior. I didn't see a More Articles option.


Popover

Component name: <pfe-popover>
Design specification updates
PatternFly v4 Popover

@eyevana this one might be good for a start, it would be similar to tooltip

Closed by #2320


Installing the source and building

Will document as I go along:

needed to install XCode and the XCode command line tools with xcode-select --install

needed to rebuild node-sass

there is a config file in /src/parsers that is set as example and seems to need a MERCURY_API_TOKEN  (instructions should say its required to signup and then edit this config example file)

Looking for where to put in my Apple Developer Credentials ...

I see that the MERCURY ID and config file instruction is there ... seems like it needs to be above the build instruction since that fails without first copying the config.js and getting your own MERCURY ID


Support for indeterminate prop with <Progress radial />

Describe the feature in detail (code, mocks, or screenshots encouraged)
Hi again :)
I wanted to have a spinner shown while loading the page, and though I could use the <Progress radial indeterminate /> but had to dive in the code to realize that it is only supported by Progress bars.
We could specify this limitation in the doc, but it would be just as quick to add a CSS animation for the radial one.
For now I added the following to my svelte component. The effect is not exactly "spinner-like" but good enough for now while I'm prototyping my app.
<Progress value={15} radial indeterminate type="info" />

<style>
:global(.stwui-progress-radial) {
	transform-origin: calc(0.5 * var(--spinnerSize)) calc(0.5 * var(--spinnerSize)) 0;
	animation: spinner 2s linear infinite;
}
@keyframes spinner {
	0% {
		transform: rotate(0deg);
		stroke-dashoffset: (0.66 * var(--spinnerSize));
	}
	50% {
		transform: rotate(720deg);
		stroke-dashoffset: (3.14 * var(--spinnerSize));
	}
	100% {
		transform: rotate(1080deg);
		stroke-dashoffset: (0.66 * var(--spinnerSize));
	}
}
</style>

Thanks again for STWUI !
What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
No response

Indeed I was looking for an easy way to add a loader and instead of looking elsewhere I thought there might be a  component in STWUI. Seen there wasn't I looked at the  component in the docs and when I saw indeterminate as an available props I thought "bingo let's do this." But it didn't exactly go as planned and I had to check the sources to realize why.
It seems rather logical that if the prop is available for progress bars it should also be available for progress circles. I don't see how use cases could be different, just the shape is.
If that's too much work to fit in, then at the very least the doc should mention that the prop won't work with radial.

Indeed I was looking for an easy way to add a loader and instead of looking elsewhere I thought there might be a component in STWUI. Seen there wasn't I looked at the component in the docs and when I saw indeterminate as an available props I thought "bingo let's do this." But it didn't exactly go as planned and I had to check the sources to realize why.
It seems rather logical that if the prop is available for progress bars it should also be available for progress circles. I don't see how use cases could be different, just the shape is.
If that's too much work to fit in, then at the very least the doc should mention that the prop won't work with radial.

Hi @jerefrer, unfortunately, the component make up of a linear and radial progress bar are incredibly different. The radial progress bar is a much more complex component than the linear alternative and I am unable to match the animation style of the linear to that of the radial without a complete overhaul of the component itself. I have added to the documentation to show indeterminate only works for linear and not radial.

@N00nDay Good enough for me, at least people would know immediately and won't be surprised if they try it with radial.
Thanks for taking the time to try to find a solution !


[BUG] Decimal font size exports with comma instead of point

Please prefix your issue with one of the following: [BUG] [PROPOSAL] [QUESTION].
Package version, Laravel version
2.1.7 / 5.3.2
Expected behaviour
Font size 13.5 in styling.xml
Actual behaviour
Font size 13,5 in styling.xml, which gives an error while opening in Excel. It will remove all styling before opening the stylesheet.
Steps to reproduce the behaviour
I just used <h3> tags, which apparently defaulted to 13.5px. When I replaced the 13,5 with 13.5 in the Excel zipfile, the problem was solved. Perhaps this has to do with internationalisation? In the Netherlands we use comma as decimal separator.

We don't have a styling.xml, nor do we create it. We just pass on everything to PHPExcel, and let them do the handling.
Not a lot we can do in this package about it.


Fix CHIRRTL bugs

Addresses #325, #332, and overrides #326.
It's a bit surprising that it has been working with wrong type inferences for CHIRRTL.

The types get over-written later on in the InferTypes pass, so that probably cleared out the incorrectly inferred types.


[Question] Reproduction problem about Compactness-based Densification

Dear authors, thanks for your great work, but I met problems while training with the Compactness-based Densification proposed in section 4.2 of your paper, like the figure below:

I found that in your conf/base.yaml:
renderer:
  ...
  densify:
    enabled: true
    type: official
    ...
    use_legacy: true

, which means that in function def densify() at gs/gaussian_splatting.py:
    def densify(self, step, verbose=True):
        # check if densify is enabled and triggered, if true, do densify
        if not self.densify_cfg.enabled:
            return
        if step < self.densify_cfg.warm_up or step > self.densify_cfg.end:
            return

        if step_check(step, self.densify_cfg.period, True):
            if self.densify_cfg.use_legacy:
                self.densify_legacy(step, verbose)
                if "compatness" in self.densify_cfg.type:
                    self.densify_by_compatness(K=self.densify_cfg.get("K", 3))

, the function self.densify_by_compatness() will not be executed and only self.densify_legacy() will be ran.
And then I set renderer.densify.type="compatness", to run it, but the proformance is not quite good with prompt "a zoomed out photo of a 3D model of an adorable cottage with a thatched roof" with initial prompt "a cottage", at iter7000 (because after including the self.densify_by_compatness(), the training stopped at iter7000)

with compatness

without compatness

There might be several reasons, the first that I came up with that  in defalult, the pruning operation is disabled in config. I will report the result after adding the pruning.
But the most important is authors' explaination, cause you are the most familiar with the code. Could you kindly provide some advice, since we are also doing Text-to-3D tasks, and the Compactness-based Densification really attracting us.
Thanks! 😄

Hi haodong,
Thanks for your interest. If you find the model contains an excessively large number of Gaussians, please set a smaller K by +renderer.densify.K=1 or a larger period on densification by renderer.densify.period=2000 which works fine in my case. Some of the configs have not been released yet since we want to do better (including guidance with DeepFloyd IF and VSD loss in ProlificDreamer).
The error that happened in the pruning is caused by the alpha_thresh: 1000 since alpha_thresh refers to the minimal opacity value thus all of the Gaussians are pruned. You can set the alpha_thresh=0.05 to enable a correct prune. Sorry for not making it clear in the readme. We are still working on GSGEN. If you have any more questions about the approach, feel free to email me.

Many thanks for your explaination! And sorry I did not set params like alpha_thresh and radii2d_thresh carefully :smile:
After adding the Compactness-based Densification, the details seems to be better!

with compactness

without compactness

I am also trying to add the pruning at the same time, thanks again!


Add support for global REPL

Description of the change
Adds support (when finished) for starting a REPL within a folder which has not been setup as a spago project (as described in #168).
Checklist:

[x] Added the change to the "Unreleased" section of the changelog
[ ] Added some example of the new feature to the README
[ ] Added a test for the contribution (if applicable)

@f-f I decided to take a look at this and I tried to implement it according to https://github.com/spacchetti/spago/issues/168#issuecomment-478526643. However, I’m a bit unsure of a couple of things (in addition to the comment above):

Any idea for how this could be tested? I couldn’t find a test for spago repl, is there a specific reason for that?
I saw that PackageSet.ensureFrozen was called in Config.ensureConfig, which is not used for this, but I'm not sure what freezing a package set means. Should it be used here?

@eahlberg looks good so far! I added some minor comments, and to answer your other points:

let's skip tests for this one as I don't have ideas about how to test spago repl
"freezing" a Dhall file means to add sha256 hashes to the remote imports (the package set URL in our case). Since in this case we're initing a new project it should be a no-op as the template already has hashes, so it should be fine to use ensureConfig.
We have a small section about freezing in the README, but it's very small as it's a detail that's supposed to be automated away from the user. However in there you can find a link to a longer description of Dhall's freezing mechanism

I just noticed that in addition to the install flags we'll also need to add the --package flag to add packages as specified in #168. It's going to look exactly as the packageNames argument:
https://github.com/spacchetti/spago/blob/92e9c64bdde0373a64514def96b7f78738601909/app/Spago.hs#L158
..except it will be an opt instead of an arg

@f-f thanks for the feedback! 🙇♂ I think I’ve addressed all of it and left a couple of questions above, so feel free to take a look when time permits.

@f-f cool, thanks!


Output of uname -a:
Linux hems-hub 3.19.0-49-generic #55~14.04.1-Ubuntu SMP Fri Jan 22 11:24:31 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Steps to reproduce the issue:

attempted to restart docker

Describe the results you received:
....
DEBU[0000] devmapper: Added deviceId=682 to DeviceIdMap
DEBU[0000] devmapper: Added deviceId=213 to DeviceIdMap
DEBU[0000] devmapper: Added deviceId=56 to DeviceIdMap
DEBU[0000] devmapper: Added deviceId=271 to DeviceIdMap
DEBU[0000] devmapper: Added deviceId=723 to DeviceIdMap
DEBU[0000] devmapper: constructDeviceIDMap() END
DEBU[0000] devmapper: activateDeviceIfNeeded()
DEBU[0000] devmapper: Error device setupBaseImage: devmapper: Base Device UUID and Filesystem verification failed.devicemapper: Error running deviceCreate (ActivateDevice) dm_task_run failed
DEBU[0000] Cleaning up old shm/mqueue mounts: start.
FATA[0000] Error starting daemon: error initializing graphdriver: devmapper: Base Device UUID and Filesystem verification failed.devicemapper: Error running deviceCreate (ActivateDevice) dm_task_run failed

Describe the results you expected:
Docker starts
Additional information you deem important (e.g. issue happens only occasionally):
DOCKER_OPTS=--storage-driver=devicemapper --storage-opt dm.datadev=/dev/datavg/docker-data --storage-opt dm.metadatadev=/dev/datavg/docker-meta
Output of blkid
...
/dev/sdc: UUID="MkggW3-1uxH-eDYF-L7w2-AvFf-aus0-P2UMpl" TYPE="LVM2_member"
/dev/mapper/datavg-spherestack: UUID="1cb2c312-33a5-4b21-a32f-240d5cd8006e" TYPE="ext4"
/dev/mapper/datavg-backups: UUID="10c711ff-e24c-4fbd-9747-5c59abefe4c7" TYPE="ext4"

Output of ls -ltr /dev/mapper:
ls -ltr /dev/mapper
total 0
crw------- 1 root root 10, 236 Mar 29 01:32 control
lrwxrwxrwx 1 root root       7 Mar 29 01:32 datavg-backups -> ../dm-3
lrwxrwxrwx 1 root root       7 Mar 29 01:32 datavg-spherestack -> ../dm-2
lrwxrwxrwx 1 root root       7 Mar 29 02:13 docker-8:1-1052708-pool -> ../dm-4
lrwxrwxrwx 1 root root       7 Mar 29 02:13 datavg-docker--meta -> ../dm-1
lrwxrwxrwx 1 root root       7 Mar 29 02:13 datavg-docker--data -> ../dm-0

Output of blkid /dev/dm-4:

(e.g. empty)
Output of dmesg:
[   14.074762] device-mapper: thin: Data device (dm-0) discard granularity larger than a block: Disabling discard passdown.
[   14.277748] device-mapper: table: 252:5: thin: Couldn't open thin internal device
[   14.277749] device-mapper: ioctl: error adding target to table

Output of cat /var/lib/docker/devicemapper/metadata/deviceset-metadata:
{"next_device_id":731,"BaseDeviceUUID":"ccdb3627-6f38-44cd-afaf-7d03c519abb4","BaseDeviceFilesystem":"ext4"}

Output of ls blk:
NAME                               MAJ:MIN RM   SIZE RO TYPE MOUNTPOINT
sda                                  8:0    0  29.3G  0 disk
└─sda1                               8:1    0  29.3G  0 part /
sdb                                  8:16   0    14G  0 disk
└─sdb1                               8:17   0    14G  0 part /mnt
sdc                                  8:32   0   512G  0 disk
├─datavg-docker--data (dm-0)       252:0    0   128G  0 lvm
│ └─docker-8:1-1052708-pool (dm-4) 252:4    0   128G  0 dm
├─datavg-docker--meta (dm-1)       252:1    0     4G  0 lvm
│ └─docker-8:1-1052708-pool (dm-4) 252:4    0   128G  0 dm
├─datavg-spherestack (dm-2)        252:2    0   128G  0 lvm  /var/lib/sphere-stack
└─datavg-backups (dm-3)            252:3    0   128G  0 lvm  /var/lib/backups
sdd                                  8:48   0   128G  0 disk
└─sdd1                               8:49   0   128G  0 part
sr0                                 11:0    1   1.1M  0 rom

Some output from ```dmesg | egrep "sdc|scsi":
[    8.221588] scsi host5: storvsc_host_t
[    8.224955] scsi 5:0:0:0: Direct-Access     Msft     Virtual Disk     1.0  PQ: 0 ANSI: 4
[    8.228258] scsi 5:0:0:1: Direct-Access     Msft     Virtual Disk     1.0  PQ: 0 ANSI: 4
[    8.299357] sd 5:0:0:0: Attached scsi generic sg3 type 0
[    8.301006] sd 5:0:0:0: [sdc]<PHONE_NUMBER> 512-byte logical blocks: (549 GB/512 GiB)
[    8.302887] sd 5:0:0:1: Attached scsi generic sg4 type 0
[    8.317444] sd 5:0:0:0: [sdc] 4096-byte physical blocks
[    8.320316] sd 5:0:0:0: [sdc] Write Protect is off
[    8.322285] sd 5:0:0:0: [sdc] Mode Sense: 0f 00 10 00
[    8.322427] sd 5:0:0:0: [sdc] Write cache: enabled, read cache: enabled, supports DPO and FUA
[    8.326340]  sdc: unknown partition table
[    8.329291] sd 5:0:0:0: [sdc] Attached SCSI disk

The issue occurred when I restarted the docker service while investigating an unrelated issue.
There have not been any attempts to resize lvs and AFAIK there devices all have plenty of room.
Output of dmsetup table:
docker-8:1-1052708-pool: 0 268435456 thin-pool 252:1 252:0 128 32768 1 skip_block_zeroing
datavg-spherestack: 0 268435456 linear 8:32 276826112
datavg-docker--data: 0 268435456 linear 8:32 2048
datavg-backups: 0 268435456 linear 8:32 545261568
datavg-docker--meta: 0 8388608 linear 8:32 268437504

CentOS Linux release 7.2.1511 (Core) has same question.
output of 'uname -a' command:
Linux example_fqdn_name 3.10.0-327.10.1.el7.x86_64 #1 SMP Tue Feb 16 17:03:50 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Output of docker daemon's log message:
FATA[0000] Error starting daemon: error initializing graphdriver: devmapper: Base Device UUID and Filesystem verification failed.devicemapper: Error running deviceCreate (ActivateDevice) dm_task_run failed
Boot opions:
ls /dev/sdb pvcreate /dev/sdb pvs vgcreate vg-docker /dev/sdb vgs lvcreate -L 4G -n lv-docker_metadata vg-docker lvcreate -l 100%FREE -n lv-docker_data vg-docker /usr/bin/docker daemon --log-level=info \     --storage-driver=devicemapper \     --storage-opt dm.datadev=/dev/vg-docker/lv-docker_data \     --storage-opt dm.metadatadev=/dev/vg-docker/lv-docker_metadata

I am going to close this issue since I suspect the thin pool device got damaged by my attempts to repair it with this_repair after the initial problem arose. I will open another report if the spontaneous failure occurs again.

I have an inkling about how the situation arose.
I used ansible (in particular, this role - https://github.com/angstwad/docker.ubuntu)  to setup the disks and the install docker. I think the issue arose that docker was initially started in the loop-lvm mode and was then reconfigured to use with direct-lvm mode without an intervening restart of docker. Docker was then extensively configured with the thin pool device bound to the loopback devices rather than the actual LVM logical volumes. Of course, when docker was eventually restarted with the intended storage options, the new thin pool device was effectively uninitialised.
I am going to see if I can reproduce the issue and will consider raising an issue against the ansible role.

For completeness, my issues were caused by the failure of the ansible role I was using to fully restart the docker daemon when new storage options were configured as part of the docker setup process. This resulted in the daemon adding new images to the loop back thin pool device, rather than the intended volume group backed thin pool device. When the daemon was eventually restarted with the configured storage options, the meta data no longer matched the (still uninitialised thin pool device), so that daemon refused to start.
The issue is documented here: https://github.com/angstwad/docker.ubuntu/issues/89 and a PR is documented here https://github.com/angstwad/docker.ubuntu/pull/90