Link recently_discovered_pods widget to rpt

Background
Widgets link to reports by report name.
This is because the ids in our database are not consistent across installs.
There is one widget that references a report by id.
Action
I updated the recently_discovered_pods widget to point to the report with the same name.
MiqReport.find_by(id: 198)
# => nil
MiqReport.find_by(name: "Recently Discovered Pods")
# => <MiqReport .....>

why do you think this is the right report?

There is only one report with a name like "recently discovered pods" (searched on like '%discovered%')
Most widgets have the report name the same or very similar to the widget name.

links
No BZ associated. I found while scanning through widget meta-data
/cc @nimrodshn FYI

Thanks @kbrock makes perfect sense.
@nimrodshn did we introduce other of these? (We should fix them).
@kbrock I wonder if we should refuse to seed these in the database if they have an id (explicit ERROR), or if we should have a spec test over these yaml... these would catch future mistakes.

@kbrock Good catch!
@simon3z After re-checking the PR it seems like this was my only buggy reference.

ugh - bad push. trying to get green (think master is failing)
@nimrodshn to follow up on your comment. All the other reports you created were both good and consistent. So it took seconds to fix.

@kbrock Does this need to go to euwe?

This was introduced in 12/6/2016 by https://github.com/ManageIQ/manageiq/pull/13055
If that is in euwe then we want to get this fix into that version.
Note, there is no BZ for this PR (I stumbled upon this while parsing report files)


Add support for powershell on Windows + Powershell Plugin

Added support for PowerShell on Windows and its plugin via a PowerShell module in the user's profile.
This Pull Request tries to fix:

#570
#736
#896

I'm not that good in Rust so my code might not be the best, any review would be appreciated.
--- EDIT --- (To link those Issues in GitHub system)
Closes #570
Closes #736
Closes #896

@denisidoro there are lots of people who would love to use the full functionality on windows/powershell. Thanks @alexis-opolka for investing into this!!


Recognise Binary Operation

I noticed warnings on binary logical operations such as:
a && console.log('OK');
b || someFunc();

Expected an assignment or function call and instead saw an expression.

Note:
Following is a genuine error
const a = true;
let b;
a && b = 5;

JSHint

Bad assignment.

Firefox

Uncaught SyntaxError: invalid assignment left-hand side

Correct syntax:
const a = true;
let b;
a && (b = 5);

JSHint

Expected an assignment or function call and instead saw an expression.

The expected 'assignment' is just saving the result to somewhere.
If you use it in a function and return said expression, it shouldn't complain about it.
I'm using jshint on vscode with the fix I said:
const a = true;
let b;

function getB() {
    return a && (b = 5);
}

@fbarda has it (thanks, Arda!). By default, JSHint prefers that control flow be expressed with statements rather than expressions. So you could satisfy it by rewriting the code as follows:
-a && console.log('OK');
+if (a) {
+  console.log('OK');
+}
-b || someFunc();
+if (!b) {
+  someFunc();
+}

The motivation is that the verbosity is meaningful because it helps human readers better identify and understand the code paths.
But JSHint is configurable, so you can also resolve this by keeping your code just as it is and configuring JSHint to tolerate that shorthand. Here's how you'd do that with an in-line directive:
+// jshint expr: true
 a && console.log('OK');
 b || someFunc();

@fbarda has it (thanks, Arda!). By default, JSHint prefers that control flow be expressed with statements rather than expressions. So you could satisfy it by rewriting the code as follows:
-a && console.log('OK');
+if (a) {
+  console.log('OK');
+}
-b || someFunc();
+if (!b) {
+  someFunc();
+}

The motivation is that the verbosity is meaningful because it helps human readers better identify and understand the code paths.
But JSHint is configurable, so you can also resolve this by keeping your code just as it is and configuring JSHint to tolerate that shorthand. Here's how you'd do that with an in-line directive:
+// jshint expr: true
 a && console.log('OK');
 b || someFunc();

@jugglinmike The JS code that is being parsed is not my own code therefore, rewriting the code is not an option, neither is adding // jshint expr: true.
I use JSHint with Codemirror in a Firefox extension and the code that is being parsed is user's code (userscript) so I have no control over it.
The issue was that a && console.log('OK'); is valid JavaScript code but it is reported as an error by JSHint.

JSHint issues a warning for that code, but it should correctly recognize it as valid JavaScript. (JSHint reserves the term "error" for invalid syntax; error messages have identifiers which start with the letter "E", while identifiers for warning messages start with the letter "W". If it JSHint is reporting an error for syntactically valid code, then that is a bug.)
The expr option exists for exactly cases like yours, where the person running JSHint does not agree with the warning and wishes to tolerate the programming pattern it disallows.
I hear that you can't edit the code directly, but maybe you can change the linting options when you invoke JSHint. It sounds like you're using the JavaScript API. If so, you can write something like JSHint(theUserscriptCode, {expr: true}).

Thank you for the reply.

If it JSHint is reporting an error for syntactically valid code, then that is a bug.

I see... JSHint indeed reports as warning (not error).

I hear that you can't edit the code directly, but maybe you can change the linting options when you invoke JSHint. It sounds like you're using the JavaScript API. If so, you can write something like JSHint(theUserscriptCode, {expr: true}).

I do control the JSHint options and can set expr: true. Reading from JSHint Options, I was concerns that expr: true may miss other valid warnings. :thinking:


Option to disable phantoms

Is there an option to disable phantoms and only outline the errors?
I can add PR with such setting or maybe the plugin should honor global show_errors_inline setting.

Seems like a good idea.  I pushed a change to my work-in-progress project that adds some settings to control how messages render.


How to get a  fine-tuned image-caption huggingface-version OFA model ?

How could I get the fined-tuned image-caption OFA model of huggingface version, which had topped the MSCOCO Image Caption Leaderboard ?

Sorry for my late response. To reach a similar score on the leaderboard, you should use the finetuned checkpoint. We have already released the huge ckpt of captioning for the original code, and we'll soon release the one for HF transformers, I guess this week.

Is it possible to load the finetuned checkpoints using the huggingface model definition (e.g. from transformers import OFAModel)? The uploaded files only include the model checkpoint, not the model config. I tried copying over the config from OFA-base, but got some warnings about uninitialized layers and the generated captions are gibberish. Thanks!

No, the model definition is not the same as HF's, you basically start from a randomly initialised network.

Hi @JustinLin610, congrats on the awesome work.

and we'll soon release the one for HF transformers,

Do you have any any update in this?
@NohTow:

No, the model definition is not the same as HF's,

So does it mean that they are artifacts of uncompatible architectures, or is it possible to map parameter names from the fairseq version to HF Transformers version?
I'd like to help with it if it's possible.

I think it should be possible to translate the fairseq version to an HF one (since I guess that is what they did for the other regular versions) but I am not sure nor I know how to do this.

Ok I quickly had a look at it and for the base-sized checkpoint ~100 parameters out of 965 have different names. I'll try to make a conversion tomorrow --it's definitely not a job for Friday night 😬

I managed to convert weights from Fairseq version to Transformers-compatible one. Here's a PR on HF Hub for the base version --I'm also making another PR for the large size shortly.
@JustinLin610

There's no HF model repo for OFA-large-caption, so I couldn't make a PR. Instead, I uploaded the converted model to my storage for public download.

Very cool, thank you very much !
I’ll test it on monday and try to reproduce the paper results.
Converting models weights always seems a bit tricky but doable, do you mind sharing some ressources/insights on how you did this ? It has to do with named parameters, but if you have any useful inputs on that, it would be awesome !
But again, thank you already for doing it for this model !

I’ll test it on monday and try to reproduce the paper results.

Cool. I'd like to hear about the results of your tests as well. I'll also share the converted huge-sized captioning model --it's the model that topped the COCO Learderboard.

if you have any useful inputs on that, it would be awesome

Sure. I'll share my code with some explanatory notes on Monday.

And here comes the huge-sized image-captioning model. Download, zipped, 2.39 GB.

And here's the Colab notebook with explanations that I used for conversion.

And here's the Colab notebook with explanations that I used for conversion.

Thank you very much, I'll bookmark it so I know how to do this kind of translation in the future !

I managed to convert weights from Fairseq version to Transformers-compatible one. Here's a PR on HF Hub for the base version --I'm also making another PR for the large size shortly.

So I tried using these weights (each version, from base to huge) in my own loop based on HF examples and it seems like the model is correctly loaded:
[INFO|modeling_utils.py:1703] 2022-09-12 14:51:32,183 >> All model checkpoint weights were used when initializing OFAModel.

[INFO|modeling_utils.py:1711] 2022-09-12 14:51:32,183 >> All the weights of OFAModel were initialized from the model checkpoint at /nfs/nas4.irisa.fr/deepfakes/deepfakes/image_repurposing/multimodal-fake-news-detection/VisualNews/OFA/weights/OFA-base-caption.
If your task is similar to the task the model of the checkpoint was trained on, you can already use OFAModel for predictions without further training.

Also, the resulting model seems to be able to generate ok-ish captions, so I guess we can confirm that weights are indeed well imported (would result in total gibberish otherwise).
For example, it generates "<s> an airplane is parked on the runway at an airport}})\\)\\)\\)\\)\\)\\</s>" when one reference is "the airplane has landed behind a fence with barbed wire" so the model definitely works as intended. Yet, at you might have noticed, at the end of the caption, it often (nearly always ?) generate gibberish before the eos token.
I remember this issue https://github.com/OFA-Sys/OFA/issues/172 that seems very related. However, it should not happened on an "official version" of the model, so it's weird.

I observed this phenomenon in various other Seq2Seq models (i.e., jibberish artifact at the end of the sequence), and it's easy to get rid of them, so I didn't bother

Yeah sure we can filter it out, but it still means that the model tries to output this, which is not really a good behavior and I guess it is hurting the performance in the end.

The issue mentions about a fix for the captioning task, so it might be that the captioning checkpoint is exported before this fix. But this is only a guess --maybe we can a confirmation from the authors.

Yes, but again, it would be really strange that a model that output gibberish in the end will be SOTA imho. Maybe the filtering is enough to get good results and the learning is not hurt but I find it odd.

In their HF Spaces app, the author set that value to cfg.task.patch_image_size (here on line 53), and it's defined as 480 by default here on line 62.
Using a resolution of 480 makes the inference run longer, but a lower resolution lead to missing some details such as small or rare objects and colors. So I chose to stick to the value of 480 because it seems to yield more elaborate captions.

Very well, but maybe it would be nice to know the exact experimental setup if we want to reproduce the results.
I need to double check, but should not we use the resolution it has been trained on or it does not matter there ?
Overall, it would be really nice to have every information needed to reproduce the results with HF versions. I guess we can find them in evaluation and training scripts used with fairseq, but it needs some digging.

I guess it is hurting the performance in the end

Don't think so. As previously stated, most of the Seq2Seq models have this behavior. I also observed it in my TTS research. A high-quality TTS model I trained always produced one-second noise at the end of the synthesized speech, and I was simply trimming it out.
I also verified that the authors use postprocessing to remove them.

I observed something interesting with following image:

HD model produced the following with use_cache=True: "a man is standing next to a red punching bag".
However, it produced the following more detailed caption with use_cache=False: "a man wearing a helmet and boxing gloves standing next to a red punching bag".
Interestingly, the beam search decoder in Fairseq produced exactly the latter with use_cache=True, so it's faster and more elaborate. Again, this has nothing to do with the model itself, but it's related to how the beam search decoding is implemented. For some reason, we need to set use_cache=False in HF to match the quality of the Fairseq beam search decoding, thus ending up with 3-to-4x slower inference.

Don't think so. As previously stated, most of the Seq2Seq models have this behavior. I also observed it in my TTS research. A high-quality TTS model I trained always produced one-second noise at the end of the synthesized speech, and I was simply trimming it out.
I also verified that the authors use postprocessing to remove them.

Well, I evaluated the model with your regex to trim the bad ending and got these results:
- base-caption : 'eval_bleu': 0.3787352658847889
- large-caption : 'eval_bleu': 0.4061869408828353,
- huge-caption : eval_bleu': 0.4294927572391054

So results seems ok-ish so yeah it should do the trick, but they are still a bit different from reported ones. I'll try with the trimming used by the authors and double check that I have the exact same data as the paper (as well as my pipeline).

nterestingly, the beam search decoder in Fairseq produced exactly the latter with use_cache=True, so it's faster and more elaborate. Again, this has nothing to do with the model itself, but it's related to how the beam search decoding is implemented. For some reason, we need to set use_cache=False in HF to match the quality of the Fairseq beam search decoding, thus ending up with 3-to-4x slower inference.

Very interesting. I'll make sure to use use_cache=False for my attempt to reproduce results. Are you sure that this is due to how beam search is implemented or it might be coming from something wrong in the cached hidden states ? I mean, is it worth it for me to dig into this and debug or it is the expected behavior ?
Also, do you know which decoding method/parameters are used in the paper for image captioning ? It is mentioned for NLU but I can't find it for captioning.

Maybe you can try the repository,I succeed to use transformers to train OFA model and inference: https://github.com/yangjianxin1/OFA-Chinese


Add flag to specify filename other than CODENOTIFY

Context
For RFC 551 APPROVED: Documenting ownership with OWNERS files, we are adding the ability to specify OWNERS of certain files/directories in our code base. We want this to behave in basically the exact same way as CODENOTIFY. However, being listed in CODENOTIFY doesn't necessarily mean you are an owner, so we need to keep the two lists separate.
As discussed in OWNERS in Sourcegraph: Implementation details, there aren't any off-the-shelf solutions that would do exactly what we want. In fact, Codenotify is the closest thing — so I decided to modify it slightly so that it can read OWNERS files as well as CODENOTIFY files.
Changes in this PR

Add a -filename option to the codenotify tool, with a default value of "CODENOTIFY".
Add an input to the GitHub workflow, which allows users of the workflow to pass in a different value for the filename.
Modify the Dockerfile to plumb the workflow input through to the -filename parameter of codenotify.

@jhchabran Please take a look for early feedback. There's something not quite right about how I'm passing the parameter through from the GitHub action, but I'd like to get your feedback on the other aspects while I continue to debug that. Thanks!

Happy to see you all running with this! This codebase was a bit of a hack project for me so feel free to take it from here and change whatever you want. I'll stay out of the code reviews unless you explicitly tag me :)


Can it be used in comfyui?

This model may replace a bunch of controlnet models I have on my hard drive, but can it be used directly in comfyui?

I am working for it. probably I need some time, there is lot of works to do. I will release the promax version(8 control with tile & inpaiting) within 2 weeks, so be patient. I will try to fix the released model to comfyui in this weekend.

It currently works in ComfyUI using the standard ControlNet nodes. While it might not be optimal, it does appear to be able to pick up on the type from the input image well enough to function without explicitly being able to pass the type.

Yes, there exists some modification in the pipeline scripts to make it work in comfyui. I will try to update it in this weekend.

Yes, there exists some modification in the pipeline scripts to make it work in comfyui. I will try to update it in this weekend.

can it be used in SD A1111?

It currently works with ComfyUI and A1111, but it doesn't perform very well.
When using the example images from this project as control images, it performs well even without a control type id and seems to recognize the control type automatically.
But when using other images as control images, it starts to perform poorly. The most typical one is OpenPose, and most of the time the generated image will contain the OpenPose skeleton.

It currently works with ComfyUI and A1111, but it doesn't perform very well. When using the example images from this project as control images, it performs well even without a control type id and seems to recognize the control type automatically. But when using other images as control images, it starts to perform poorly. The most typical one is OpenPose, and most of the time the generated image will contain the OpenPose skeleton.

Dude how to use it at A1111?

https://github.com/Mikubill/sd-webui-controlnet/discussions/2989


Min for nullable types

Min doesn't treat null as a "value" for nullable types. I was expecting Min to produce null if a null is present in the input observable.
This behavior is notably different than System.Interactive, where Min does respect and return null if present.

Looking at the code, Min should return null for an empty or only null containing sequence. Are you expecting something else? What type are you using? There are some specialist versions for the common numeric values.

Oh, and are you talking about Rx or AsyncIx? My answer was for Rx.

Are you expecting something else?

Yes, though I also see the validity of the "special null" semantic.
What I expected: The default comparer for nullable numeric types like int? treats null as "less than" any other value, so given a null input value, I'd expect the following:
var values = new int?[] { int.MaxValue, null, int.MinValue }.ToObservable();
var result = await values.Min().LastAsync();
Assert.Null(result); // fails; `result` is `int.MinValue`

This behavior is different than Ix, which has the following behavior:
var values = new int?[] { int.MaxValue, null, int.MinValue };
var result = values.Min();
Assert.Null(result); // succeeds; `result` is `null`

The current semantics assume null is not a "real" value for sequences of nullable numeric types. I can see the validity of that argument; but to me it is more natural to treat null as a "real" value for IObservable<int?>, as it is for most other operators, and to use Where to filter out nulls if desired.

Well, it is the behaviour like in Linq:
using System;
using System.Linq;

public static class Program {
    public static void Main() {
        var result = new int?[] { int.MaxValue, null, int.MinValue }.Min();
        Console.WriteLine(result);                // result == -2147483648
    }
}

See https://sharplab.io/#v2:EYLgZgpghgLgrgJwgZwLQQHYAsoYMYQAmqcyAlhgOYA0MICcGAPgAIAMABCwIwDcAsACh2XbgDoAMhQCOA4QCY2QlgGZRANi7zRAdg4BvIR2Nc1PTSwAsHALJQKACgCUBoyfcA3KAg5JkcABsYDgBeDgwIAHcOChgAfgBtAF0DGIwYMTsADwA1KAC4CGpwwIDi2MyKPIKIDgBfSoxnOXd3HgBOBz9AmCcWkzqhOqA===
Afaik, the behaviour was changed from Rx1 to Rx2 to reflect the IEnumerable<T> semantics. Also the documentation of Enumerable.Min() gives the following example:
int?[] grades = { 78, 92, null, 99, 37, 81 };

int? min = grades.Min();

Console.WriteLine("The lowest grade is {0}.", min);

/*
 This code produces the following output:

 The lowest grade is 37.
*/

OK. In that case, Ix should probably be changed to match. My example code above is incorrect; the correct code should be:
var values = new int?[] { int.MaxValue, null, int.MinValue };
var result = values.Min(Comparer<int?>.Default);
Assert.Null(result); // succeeds; `result` is `null`

which uses the actual Ix Min and not the LINQ one.

@StephenCleary Not sure I follow completely, is there a bug in Ix we need to address here?

It's an inconsistency. All the following tests pass:
[Fact]
public async Task RxMin()
{
    var values = new int?[] { 0, null }.ToObservable();
    var result = await values.Min().LastAsync();
    Assert.Equal(0, result);
}

[Fact]
public async Task RxMin_WithComparer()
{
    var values = new int?[] { 0, null }.ToObservable();
    var result = await values.Min(Comparer<int?>.Default).LastAsync();
    Assert.Equal(0, result);
}

[Fact]
public void LinqMin()
{
    var values = new int?[] { 0, null };
    var result = values.Min();
    Assert.Equal(0, result);
}

[Fact]
public void IxMin()
{
    var values = new int?[] { 0, null };
    var result = values.Min(Comparer<int?>.Default);
    Assert.Null(result);
}

Not sure if it's worth fixing. I have no use case; I just ran into this edge case when testing my comparers library.

It's an inconsistency. All the following tests pass:
[Fact]
public async Task RxMin()
{
    var values = new int?[] { 0, null }.ToObservable();
    var result = await values.Min().LastAsync();
    Assert.Equal(0, result);
}

[Fact]
public async Task RxMin_WithComparer()
{
    var values = new int?[] { 0, null }.ToObservable();
    var result = await values.Min(Comparer<int?>.Default).LastAsync();
    Assert.Equal(0, result);
}

[Fact]
public void LinqMin()
{
    var values = new int?[] { 0, null };
    var result = values.Min();
    Assert.Equal(0, result);
}

[Fact]
public void IxMin()
{
    var values = new int?[] { 0, null };
    var result = values.Min(Comparer<int?>.Default);
    Assert.Null(result);
}

Not sure if it's worth fixing. I have no use case; I just ran into this edge case when testing my comparers library.

"Fixing" this would be a breaking change. In order to be consistent, the implementation of MinBy<T>() would also need to be reconsidered. But what should be the return value of new int?[] {}.MinBy(x => x)? An empty list or a list of a single null?
However, it would be possible to add a new method that match the Linq behavior:
public T Min(this IEnumerable<T> source, IComparer<T> comparer, bool treatNullSpecial)
{
    return (treatNullSpecail && default(T) == null)
        ? ...
        : source.Min(comparer);
}

Yes, this is a long-standing inconsistency that dates back all the way to Rx v1.0. It fell through the cracks in the final beta days where we were noodling around with the return type for aggregates and at some point null had a special meaning. By the time we shipped v1.0, the inconsistent nullable behavior stuck and it was reconsidered for "fixing" in v2.0, but considered too risky given the breaking change potential, with the advice to use a manual Aggregate (though verbose and clunky) if the LINQ to Objects null behavior is desired.
At this point, it's established (though inconsistent and very unfortunate) behavior, and the risk of breaking users outweighs the desire for consistency. Note for System.Linq.Async, the behavior is consistent with synchronous LINQ to Objects.


feat: optional arg [sign_weight] for Transaction.init

Make Transaction object locally without requests.

How about def sign(self, priv_key=..,  verify_sign_weight=True):?

How about def sign(self, priv_key=..,  verify_sign_weight=True):?

How about def sign(self, priv_key=.., verify_sign_weight=True):?

I think sign_weight is a must-have check, is not?

How about def sign(self, priv_key=.., verify_sign_weight=True):?

I think sign_weight is a must-have check, is not?

Making a Transaction object locally without requests is almost impossible for plain HTTP API.
Protobuf is needed.

Making a Transaction object locally without requests is almost impossible for plain HTTP API.
Protobuf is needed.

sorry, I don't get the Protobuf point.
Now we can ship the Transaction object use it's to_json() func between systems and services, but we cannot rebuild a Transaction object from "json" object(dict or str) without requests to tron network (maybe a private tron node).
For example, there are two microservices: business_srv and tron_srv,

business_srv call tron_srv.build_tx, and retain tx_json(tron_srv.build_tx using tx.to_json)
business_srv got tx_json, record it, build a Transaction object from it
business_srv sign Transaction object locally using tronpy
business_srv call tron_srv.submit_tx to submit this tx to tron network
if submit failed (maybe timeout or other exceptions) business_srv will call tron_srv.get_trancation_info to query this tx to make a confirm, even to re-send tx or other process.

With this feature, business_srv will not require a connection to tron network, just use tronpy locally, will simplify the private tron node management and microservices deployment.

sorry, I don't get the Protobuf point.
Now we can ship the Transaction object use it's to_json() func between systems and services, but we cannot rebuild a Transaction object from "json" object(dict or str) without requests to tron network (maybe a private tron node).
For example, there are two microservices: business_srv and tron_srv,

business_srv call tron_srv.build_tx, and retain tx_json(tron_srv.build_tx using tx.to_json)
business_srv got tx_json, record it, build a Transaction object from it
business_srv sign Transaction object locally using tronpy
business_srv call tron_srv.submit_tx to submit this tx to tron network
if submit failed (maybe timeout or other exceptions) business_srv will call tron_srv.get_trancation_info to query this tx to make a confirm, even to re-send tx or other process.

With this feature, business_srv will not require a connection to tron network, just use tronpy locally, will simplify the private tron node management and microservices deployment.

sign_weight is a trick used in tronpy. Actually, it is used to get the txID without calling the actual transaction APIs. (which is the SHA256 of protobuf-serialized data)
self.txid = sign_weight["transaction"]["transaction"]["txID"]

To build a transaction without a network, you also need to provide the 'ref_block' in TransactionBuilder.build().

sign_weight is a trick used in tronpy. Actually, it is used to get the txID without calling the actual transaction APIs. (which is the SHA256 of protobuf-serialized data)
self.txid = sign_weight["transaction"]["transaction"]["txID"]

To build a transaction without a network, you also need to provide the 'ref_block' in TransactionBuilder.build().

I got it, it's very clever.
I mean this PR support building a Transaction from an exists tx_json which generated by Transaction.to_json() without network, user scenario like:
def sign_tx_from_tx_json(tx_json: str):
    data = json.loads(tx_json)
    tx = Transaction(
        raw_data=data['raw_data'],
        client=Tron(),
        method=Some.contract.functions.Func,
        sign_weight=data['sign_weight']
    )	# without network
    # locally use
    tx.sign(....)
	....

I see the TransactionBuilder class is made to build a Transaction.
Maybe a classmethod TransactionBuilder.load_from_tx_json is better? like:
def sign_tx_from_tx_json(tx_json: str):
    data = json.loads(tx_json)	# {"txID":, "raw_data":, "signature":, "sign_weight"}
    tx_builder: TransactionBuilder = TransactionBuilder.load_from_tx_json(
        tx_json=data,
        client=Tron(),
        method=Some.contract.functions.Func,
    )	# pass tx_json to TransactionBuilder, include "raw_data", "signature" and "sign_weight"
    tx: Transaction = tx_builder.build(
        ref_block_id=False, 	# str(provide a ref_block_id to build raw_data)
        			# False(not get ref_block_id, leave tx_builder.raw_data as it is)
        			# None(default, get ref_block_id from network to build raw_data)
    )	# build tx using signature and sign_weight if exists
    # locally use
    tx.sign(....)

I got it, it's very clever.
I mean this PR support building a Transaction from an exists tx_json which generated by Transaction.to_json() without network, user scenario like:
def sign_tx_from_tx_json(tx_json: str):
    data = json.loads(tx_json)
    tx = Transaction(
        raw_data=data['raw_data'],
        client=Tron(),
        method=Some.contract.functions.Func,
        sign_weight=data['sign_weight']
    )	# without network
    # locally use
    tx.sign(....)
	....

I see the TransactionBuilder class is made to build a Transaction.
Maybe a classmethod TransactionBuilder.load_from_tx_json is better? like:
def sign_tx_from_tx_json(tx_json: str):
    data = json.loads(tx_json)	# {"txID":, "raw_data":, "signature":, "sign_weight"}
    tx_builder: TransactionBuilder = TransactionBuilder.load_from_tx_json(
        tx_json=data,
        client=Tron(),
        method=Some.contract.functions.Func,
    )	# pass tx_json to TransactionBuilder, include "raw_data", "signature" and "sign_weight"
    tx: Transaction = tx_builder.build(
        ref_block_id=False, 	# str(provide a ref_block_id to build raw_data)
        			# False(not get ref_block_id, leave tx_builder.raw_data as it is)
        			# None(default, get ref_block_id from network to build raw_data)
    )	# build tx using signature and sign_weight if exists
    # locally use
    tx.sign(....)

Got it. I have the following ideas.

Without a network, ref_block, txID will be empty

Also the transaction might be outdated if it is expired (maximum expiration is 1 day)

Add to_json, from_json to Transaction

Add update_ref_block to Transaction, this will change the txID, and remove all signatures.

Got it. I have the following ideas.

Without a network, ref_block, txID will be empty

Also the transaction might be outdated if it is expired (maximum expiration is 1 day)

Add to_json, from_json to Transaction

Add update_ref_block to Transaction, this will change the txID, and remove all signatures.

HOLD this. until a better idea comes.

HOLD this. until a better idea comes.

Knock Knock

Code updated ~.~

add from_json to Transaction
add update to Transaction
add property is_expired to Transaction, assert not expired when sign()
add tests: sign_offline, transaction_update

@andelf

Sorry for reviewing too late.
will publish the next version soon.


Record custom attributes in New Relic

FEATURE OVERVIEW
User Story
As a developer, I want to collect custom attributes in New Relic to help troubleshoot the cause of internal 500 http errors. We don't get any stack traces for these, so we don't know which users and/or campaigns received them.

Some parameters that seem useful to track:

[x] Mobile Commons Profile ID
[x] Mobile Commons Message ID
[x] Mobile Commons Keyword
[x] Mobile Commons Incoming Message
[x] Mobile Commons Incoming Image URL
[x] Mobile Commons Broadcast ID
[ ] Northstar User ID
[ ] Phoenix Campaign ID
[ ] Gambit Response Message Type (e.g. ask_photo, no_photo_sent)

Additional Information
Based on the timestamp of the error above ☝ (3/28 6:00pm PST), this error seems to correspond to the one triggered when a member submits a text field that's too long:
{"error":{"code":500,"message":"postReportback error:Signup.postDraftReportbackSubmission error:Unexpected end of JSON input"}}

Why This Matters
We've received more errors in New Relic (14) over the 24 hours than we have mData error emails (4). Mobile Commons does mention in their email:

Note: you may have more errors! Mobile Commons rate-limits notification emails to avoid swamping your inbox. Please check your server logs for more details.

Note -- we don't get stack traces because we're catching errors within Gambit and returning a response back.


Improve quorum code

This pull request takes the changes in #112 and cleans them up with the suggestions made by @cmurphy. Additionally, I've added a flag for the two_node option, and fixed/updated the tests. This is all caught up with master too.
Happy to receive feedback and make changes as needed.

Hmm - I'm not sure what I did to make Travis fail on older Puppet/Ruby versions.

Rebased against master.

Generally, the new parameters seem to lack validation. Is this intentional?

Good point regarding two-node arguments. And no, there's no reason to not validate the new parameters.
That said, since I submitted this, I note two other pull requests covering the same things: #126 and #134. It might be that one of them does a better job of this and makes my request obsolete.

They sure are overlapping. Since your's predates them, I'd be inclined to favor it, unless there are strong arguments in favor of either of the other ones.
Could you try and open discussions with @zdenekjanda and @bogdando, perhaps open a PR against either of their forks to find a solution that is acceptable for everyone?

I think it even predated some problems, if it was coming from my patch. For example $::operatingsystemrelease >= '14.04' will resolve true even on pre 14 ubuntu, thus making big mess on 12.04 systems, that i only realized before it broke some production systems after thinking my patch was perfect :(. Latest version of my #126 works reliably on both 12.04 and 14.04, including of service management of pacemaker which is also necessary, all that i can say. Clear problem here is too slow progress with patches, so  everybody creates something less or more ideal that works for him and we get lost into several solutions for single problem... Lets toss a dice ?

@zdenekjanda yes, I'd agree on the point about lack of progress with patches. That's discouraged me from working more on this pull request. If we can get some traction with the current PRs everyone will be in a better position.
Looking at the other two PRs, I think #126 covers most stuff that needs doing, and certainly appears to supersede this one (and sounds like it's been tested better!). However, #134 allows more rings to be specified in the node list. So maybe a combination of the two would be good.
The only thing going for my PR is that it adds tests which neither of the others do.

@tdb, thanks for the tests, because thats part which i always ommit due to #enotime :)
Looking at current implementation of #134, i have some notes. The implementation always assumes voteqorum enabled, which is not the case. If expected_votes or nodelist missing, votequorum will be disabled by default, and that should be reflected too (skip the section if this is the case). The implementation of variable corosync_nodes is misleading, i would call it a nodelist so we are in terminology of corosync. And last, there could be option to either send simple array as previous list, that will offer much wanted simplification, ex. from hiera, because in 90% times we are happy with that and its waste do write the whole structure over and over, and also enable to consume hash with full customization as suggested in patch.
When i look on my #126 in light of this, there is also a mistake by assuming nodelist only when expected_votes not present, in fact they are independent sections and this should be reflected - i use the most simple version with votequorum_expected_votes, which saves the time.
@bogdando, i would welcome your comments. We should really finalize this quickly to some final patch, so we have corosync module release which finally supports Ubuntu 14.04.

My vote goes for #126 then, instead of #124 or #134. Lets try and get some tests added to it though, and make sure there's nothing needed from #134. Also would like to hear @bogdando's view since it sounds like he had a different way of looking at things.

@tdb even #126 will need improvements, as we see, which i can try to make. As for tests, do you have experience with beaker ones ? It would be cool to produce a 2 and 3 VM clusters with different configs to really test this module. I would also like @bogdando comments on this.

@zdenekjanda Sorry, no experience with beaker.

This module was just recently moved to our care. We'll try hard to work through the backlog.
Perhaps @zdenekjanda could give @tdb push access to your fork? Maybe the tests from this PR can be adapted and committed to the #126 branch. I'd really like to see both of your credited for the work on this issue.
Acceptance tests would be great, but adding those from scratch is out of scope for these PRs. You could open an issue so that the notion isn't lost once these PRs close.

@ffrank re acceptance tests: do you think that spec tests would be a start?

@igalic definitely, but I do agree that for pacemaker, acceptance testing would be a huge boon because there is such a perceived multitude of things that can go more or less slightly wrong in operation.
So we've done a round of reviews on #126. @tdb, can this PR be closed in favor of the other one?

@ffrank Sure, I'll close it. I only left it open because of the above discussion happening here.


vaultwarden: 1.29.0 -> 1.29.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for vaultwarden is: Unofficial Bitwarden compatible server written in Rust
meta.homepage for vaultwarden is: https://github.com/dani-garcia/vaultwarden
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.29.1 with grep in /nix/store/dps217nmibx8j6bpn2v0kdikj71b6xyq-vaultwarden-1.29.1
found 1.29.1 in filename of file in /nix/store/dps217nmibx8j6bpn2v0kdikj71b6xyq-vaultwarden-1.29.1

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

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
nixos-install-tools
tests.nixos-functions.nixos-test
vaultwarden
vaultwarden-mysql
vaultwarden-postgresql
vaultwarden-sqlite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dps217nmibx8j6bpn2v0kdikj71b6xyq-vaultwarden-1.29.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

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

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

1 package blacklisted:

nixos-install-tools

3 packages built:

vaultwarden
vaultwarden-mysql
vaultwarden-postgresql

Maintainer pings
cc @msteen @ivan for testing.

Result of nixpkgs-review pr 246510 run on aarch64-darwin 1

3 packages built:

vaultwarden (vaultwarden-sqlite)
vaultwarden-mysql
vaultwarden-postgresql


Improve nested templates

A nice improvement made in v2 is to be able to insert a templated element back where it originally came from, without having to select its parent in any way. It does this by storing a reference to its parent as it gets extracted from the DOM into the Template object.
Then when you come to add it back to the page, you can do this:
$messageItem = $this->template->get("message-item");
$messageItem->insertTemplate();

The insert part was crucial - rather than append, it remembers the sibling element too in order to keep its original place.
But the question is, where does (or should) the template named "message-item" get inserted in the following example:

Title

            Message content

Once the first conversation element has been inserted, should inserting message-item elements go into the original, or the latest added? If the latter, how do we define this?
@j4m3s I wonder if you could share your ideas on this?

Thinking about it, it might make most sense to do something like the following:

Templated elements need to remember their ancestors with data-template attributes.
When being inserted, templated elements need to search for the latest-added data-template ancestor.
Some clever XPath here?

Tracked in https://github.com/PhpGt/DomTemplate/issues/1


Stops after some times...

first of all thanks for this wonderful project that works perfect. i have one issue that after 4-5 hours server on my ios doesn't respond. then when i restart my router it works perfectly again, is that any issue with my internet connection something i am missing from code side..!!
i have two device both work as client and server,  i send the message from one device and receive to another device same from second device. in that case after some time, this communication not working, if i restart my router then the communication will start again...

Chances are that is related to the sockets code I used. My intent is to use SwiftNetwork framework. I’m mostly done with the development in one of the branches. Try it out. And let me know if you come across any bugs.
Devin
On May 18, 2021, 7:09 AM -0700, joyharesh13 @.***>, wrote:

first of all thanks for this wonderful project that works perfect. i have one issue that after 4-5 hours server on my ios doesn't respond. then when i restart my router it works perfectly again, is that any issue with my internet connection something i am missing from code side..!!
i have two device both work as client and server, i send the message from one device and receive to another device same from second device. in that case after some time, this communication not working, if i restart my router then the communication will start again...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Alright, thanks for quick reply


Add Switchboard On-Demand

cargo add switchboard-on-demand
https://crates.io/crates/switchboard-on-demand

Would also love support for that crate.

would also love support for this crate

Yes, its would be awesome if you guys add this crate to playground


Bump PHP to 8.0

Questions
Answers

Description?
Bump PHP to 8.0

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

How to test?
N/A

Possible impacts?
N/A

Thanks @nicosomb


Easier subcommands / subcommand aliases

I feel like some subcommands should be changed / have an alias like:
/lp listtracks -> /lp tracks
/lp listgroups -> /lp groups
Also a universal /lp delete command would be easier to use, like /lp delete <group/track>
This is a fantastic plugin and is very easy to use, these are just some suggestions
It just feels a bit more intuitive to me

I'm going to be revising the aliases soon, although, it will most likely be to remove a lot of them.
Tried to find my previous comment on this - but haven't been able to. Basically, the issue with adding aliases is that it can be confusing/misleading when the aliases do weird things.
For example, there's an alias from /lp user <user> set <permission> to the actual command, /lp user <user> permission set <permission>.
However, users can mix this up, and run /lp user <user> set permission <permission> - which will actually set the permission permission.
And that's why I'm leaning towards scrapping aliases all together.

Aliases were removed in v5 - I still feel this is the right direction for the plugin. It will make things easier for everyone in the long run if there's only one alias for each command.


Align Gradle dependencies with Maven ones

Define the same set (and scopes) of dependencies for Gradle builds as we have now in Maven's pom.xml. Also, following the same order as in the POM.

Thanks @mpaluchowski


Cloud SQL Private IP through Cloud Proxy fails

Hi there,
I am trying to connect MySQL 5.7 instance that has a private IP only using my OSX terminal. The Cloud SQL Proxy starts fine using the command with PRIVATE flag. I have tried connecting 3 different clients but I am getting the following error:
Creating a session to 'root@<IP_ADDRESS>:3306' ArgumentError: X protocol error: MySQL server has gone away Classic protocol error: MySQL Error 2013 (HY000): Lost connection to MySQL server at 'waiting for initial communication packet', system error: 60

My understanding is that I should not require an Interconnect or VPN for this to work, if I do then there is no need for a Cloud SQL Proxy as the private IP Cloud SQL would then be local to my network.
I am relying on this diagram for my connection to remain private only:

As described here: https://cloud.google.com/sql/docs/mysql/sql-proxy
I am using the -ip_address_types=PRIVATE flag when starting the proxy.

What logs are printed from the Cloud SQL Proxy?

My understanding is that I should not require an Interconnect or VPN for this to work, if I do then there is no need for a Cloud SQL Proxy as the private IP Cloud SQL would then be local to my network.

The Cloud SQL proxy only provides authentication to allow you to create connections. It doesn't let you connect unless you have access to the IP. In this case, you can't connect to a private IP unless you have access to the VPC. See the Requirements for Private IP section in our docs.


[Performance] Setup a benchmark tool for Triton kernel and monitoring the performance changes of PR in CI.

In order to support the performance tasks, we need a Triton benchmark to test the kernel performances.
And to add the benchmark test to the CI to monitoring the performance changes of each PR.
The benchmark test can be an optional action not to blocking the PR merge.

We setup a repo as a relatively independent xetla version's implementation(easy to call via python module in intel-triton). softmax's kernel have been added in PR
Then we just need to add related xetla kernels into  tutorials test in  build-test in CI.
For enhancement, considered to enable history performance tracking, maybe we can lavage dashboard

Enable the infrastructure to do performance testing. Add softmax first, then adding more OPs.


[marketplace] support latest contracts changes

Update the submodule dependency from dagger-contracts to codex-contracts-eth.
The Storage.sol contract was removed from codex-contracts-eth. This PR supports that change by modifying nim-codex accordingly.

Think it's ok to merge without windows CI passing (and causing the rest to cancel)?

Think it's ok to merge without windows CI passing (and causing the rest to cancel)?

I wouldn't. It's too easy to introduce new errors without a working CI. I'll see whether I can find why the windows CI is failing.

Thanks to @Menduist's debugging, the Windows build error was relatively easy to fix: https://github.com/status-im/nimbus-build-system/pull/56. Once that is merged, we can update nimbus build system, and the build error should go away.


ERROR: InvalidStaticInvocation - RouteServiceProvider - Method Router::__construct is not static, but is called statically

I'm getting the following unexpected error on the RouteServiceProvider that comes with Laravel and hasn't been modified by me:
ERROR: InvalidStaticInvocation - app\Providers\RouteServiceProvider.php:12:36 - Method Illuminate\Routing\Router::__construct is not static, but is called statically
class RouteServiceProvider extends ServiceProvider

Unfortunately I cannot suppress it using @psalm-suppress InvalidStaticInvocation on the RouteServiceProvider class either.

RouteServiceProvider.php
<?php

namespace App\Providers;

use Illuminate\Support\Facades\Route;
use Illuminate\Foundation\Support\Providers\RouteServiceProvider as ServiceProvider;

class RouteServiceProvider extends ServiceProvider
{
    /**
     * This namespace is applied to your controller routes.
     *
     * In addition, it is set as the URL generator's root namespace.
     *
     * @var string
     */
    protected $namespace = 'App\Http\Controllers';

    /**
     * Define your route model bindings, pattern filters, etc.
     *
     * @return void
     */
    public function boot()
    {
        //

        parent::boot();
    }

    /**
     * Define the routes for the application.
     *
     * @return void
     */
    public function map()
    {
        $this->mapApiRoutes();

        $this->mapWebRoutes();

        //
    }

    /**
     * Define the "web" routes for the application.
     *
     * These routes all receive session state, CSRF protection, etc.
     *
     * @return void
     */
    protected function mapWebRoutes()
    {
        Route::middleware('web')
             ->namespace($this->namespace)
             ->group(base_path('routes/web.php'));
    }

    /**
     * Define the "api" routes for the application.
     *
     * These routes are typically stateless.
     *
     * @return void
     */
    protected function mapApiRoutes()
    {
        Route::prefix('api')
             ->middleware('api')
             ->namespace($this->namespace)
             ->group(base_path('routes/api.php'));
    }
}

@caugner
same here.
To workaround this, I did set baseline.
$ ./vendor/bin/psalm --set-baseline=psalm-baseline.xml

Thanks, looks like this is actually not a plugin problem, also occurs in a fresh Laravel site without this plugin.


Synology docker GUI : passing plexdrive arguments over

Hello,
I’m desperatly trying to pass the argument for root node id to plexdrive (as my root google drive contains really a lot of files which are not all videos) but I’m running it in docker from
https://hub.docker.com/r/1mmortal/docker-pms-plexdrive
Synology docker gui allow only to pass already defined environment variables and seems the arguments weren’t one in this docker (couldn’t find any that does actually)
Is there a file i can modify within the docker to add those arguments on start of the docker ? Or anyone knows a work around beside launching it through docker run in command line ?
Edit : i exported the config.json of the docker i just have no idea what i should add there for plexdrive arguments...
Thanks for any help
Regards

Ok finally found where you can modify the script command that runs on start if anyone encounters the same issue one day, modify the run file (dont forget to do a apt-get update and apt-get install nano as there is no text editor in the docker) in the directiry below to modify or add the arguments as you wish to
/var/run/s6/etc/services.d/plexdrive


Add Scott Williams to the list of supporters

Adding myself to this list. Speaking on my own behalf as someone who is an open source developer, advocate, and as someone who currently uses paid Hashicorp services at work.

Added in https://github.com/opentffoundation/manifesto/pull/203


migrates geolocator_web to package:web

This PR migrates from dart:html to package:web and js_interop.
A few notes/questions:

I've increased the dart/flutter sdk versions in geolocator_web in order to use js_interop. I'm unsure if we need to bump these versions in the other geolocator packages as well.
Mockito can no longer be used with the new js_interop types (at least I ran into errors with it being unable to generate mocks).
The package:web geolocation methods now require a timeout value, rather than allowing null. Infinity is the value for timeouts in the html geolocation api, but passing double.infinity.toInt as timeout throws an error that this is an invalid value when invoked in javascript. Instead of defaulted to a timeout of 1 day, but this needs more input.
the test now require running with flutter test --platform chrome, otherwise errors are thrown as js_interop can't be used in the dart vm.

Pre-launch Checklist

[x] I made sure the project builds.
[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is does not need version changes.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I rebased onto main.
[x] I added new tests to check the change I am making, or this PR does not need tests.
[x] I made sure all existing and new tests are passing.
[x] I ran dart format . and committed any changes.
[x] I ran flutter analyze and fixed any errors.

Hi @josh-burton,
Sorry for the delay and thank you for creating this PR. In general the changes look good, I have made a few small remarks which would be great if you can address those.

Thanks for the review. I believe I've addressed them now.

When merged solves #1513

@mvanbeusekom I see the workflows are failing. I had bumped the geolocator_web version to 4.0 as its a breaking change, but as 4.0 is not published the workflows now error.
What's the correct fix here?

The best would be to split the changes into 2 PRs. One updating the geolocator_web package and a second bumping the version for the geolocator package.

However in this case the change in the geolocator package is so small that I don't mind the failing build. As long as the steps for the geolocator_web are green.


Update format of runtime map arguments

This PR changes the format of map arguments in java.yaml. Before, arguments were passed to the java application in the form of JSON objects, but now they are passed in the form of lists of key/value mappings, with one mapping per argument.
See b/63102252 for more context.

@meltsufin PTAL when you have a chance - turns out it was possible to use Apache Commons CLI to parse lists of args like this after all. :smile:


See #31 only set seq. value if there is a sequence

Thank you for the contribution. Please submit to the next branch.


Strict Confinement of microk8s

This is a suggestion/feature request rather than a bug. Apologies if this is not the intended place to talk about it.
Is there anyway to get microk8s with strict confinement? I'm trying to use it on Ubuntu Core 18. Obviously this can't happen with classic confinement.
EDIT:
I keep coming across articles by people in Canonical (like this one) saying that microk8s is supposed to receive strict confinement, is there any timeline for this?

Hi @mewais we are actively working on this now. As you understand strict confinement may block some user operations and we are being careful not to hurt the UX.
You can build a strictly confined snap from this branch https://github.com/ubuntu/microk8s/tree/feature/jdb/strict (although at the time of this writing the certificate of https://www.netfilter.org seems to have expired and the build is failing. You can use a mirror in the snapcraft.yaml eg http://www.es.netfilter.org )

Thanks a lot, I will definitely give a try to that branch.

FYI the the build steps have also been updated https://github.com/ubuntu/microk8s/blob/feature/jdb/strict/docs/build.md

All the netfilter mirrors seem to have the same issue, I still managed to build using download links from ubuntu or fedora. Thank you for your help guys.


Failure of tutorial PremBunsen2D

Hi all,
It seems like the PremBunsen2D tutorial case fails to run with the given inputs. On the first time step, an massive streamwise velocity is estimated. Attached is part of the output. Any idea what is going on here? Thanks!
Mike
SUNDIALS initialized.
MPI initialized with 112 MPI processes
MPI initialized with thread support level 0
AMReX (22.05-27-g5d88558d2aef) initialized
Successfully read inputs file ...

 ================= Build infos =================
 PeleLMeX    git hash: v21.10-118-g23397a2-dirty
 AMReX       git hash: 22.05-27-g5d88558d2
 PelePhysics git hash: v0.1-1007-gb0d3f35b
 AMReX-Hydro git hash: 343e20e
 ===============================================

==============================================================================
 State components
==============================================================================
 Velocity X: 0, Velocity Y: 1
 Density: 2
 First species: 3
 Enthalpy: 24
 Temperature: 25
 thermo. pressure: 26
 => Total number of state variables: 27
==============================================================================

 Initializing data for 2 levels
 Initialization of Transport ...
 Initialization of chemical reactor ...
Creating ReactorBase instance: ReactorCvode
Initializing CVODE:
  Using atomic reductions
 Mixture fraction definition lacks fuelID: consider using peleLM.fuel_name keyword
25 variables found in PMF file
580 data lines found in PMF file
 Making new level 0 from scratch
 Making new level 1 from scratch
 Resetting fine-covered cells mask
==============================================================================
Typical values:
	Velocity: 0.1 3.923852814e+179
	Density: 0.6650455194
	Temp:    1260.890134
	H:       -218228.8977
	Y_H2: 0.0003458573634
	Y_H: 3.14226256e-05
	Y_O: 0.000241012551
	Y_O2: 0.2074301803
	Y_OH: 0.0006121461777
	Y_H2O: 0.01451381058
	Y_HO2: 5.371213803e-05
	Y_CH2: 6.546128626e-06
	Y_CH2(S): 4.950200534e-07
	Y_CH3: 0.0002058628013
	Y_CH4: 0.04688699785
	Y_CO: 0.006938410704
	Y_CO2: 0.01541563049
	Y_HCO: 9.539561314e-06
	Y_CH2O: 0.0002204527497
	Y_CH3O: 1.360707325e-05
	Y_C2H4: 0.0002558423542
	Y_C2H5: 9.233090566e-06
	Y_C2H6: 0.0003360756718
	Y_N2: 0.762207865
	Y_AR: -2.030418186e-127
==============================================================================
 Est. time step - Conv: 1.556568412e-184, divu: 1e+20
 Initial dt: 1.556568412e-187
 Initial velocity projection:   U: 0.1  V: 3.923852814e+179
 >> After initial velocity projection:   U: 0  V: 0
 Est. time step - Conv: 3e+199, divu: 1.831467954e-05
 Initial dt: 1.831467954e-08
 Initial velocity projection:   U: 0  V: 0
 >> After initial velocity projection:   U: 0  V: 0
 Doing initial pressure iteration(s)

 ================   INITIAL ITERATION [0]   ================
 Est. time step - Conv: 3e+199, divu: 7.105853017e-173
 STEP [-1] - Time: 0, dt 7.105853017e-176
   SDC iter [1]
   - oneSDC()::MACProjection()   --> Time: 0.009342319332
   - oneSDC()::ScalarAdvection() --> Time: 0.002244584262
   - oneSDC()::ScalarDiffusion() --> Time: 0.004823582247
From CVODE: At t = 0 and h = 6.01889e-195, the corrector convergence test failed repeatedly or with |h| = hmin.```
All other processors fail as well.

Something is definitely up. I'll have a look at it, it should be pretty easy to fix.

The fix in https://github.com/AMReX-Combustion/PeleLMeX/tree/fixBunsen2D should do the trick.
It should be merged in development soon, but you can also hop on that branch if you want to give it a try.

Got it! Thanks so much! I did hop on the branch, recompiled, and reran. Unfortunately I still received an error as follows
 ====================   NEW TIME STEP   ====================
 Est. time step - Conv: 1.500837686e-05, divu: 2.070807566e-05
 STEP [89] - Time: 3.371614555e-05, dt 3.372312753e-06
   SDC iter [1]
   - oneSDC()::MACProjection()   --> Time: 0.01127888262
   - oneSDC()::ScalarAdvection() --> Time: 0.001656992361
   - oneSDC()::ScalarDiffusion() --> Time: 0.03207364306
From CVODE: At t = 9.69709e-07 and h = 3.47613e-29, the corrector convergence test failed repeatedly or with |h| = hmin.
amrex::Abort::2::Aborting from CVODE !!!
SIGABRT
See Backtrace.2 file for details
Abort(6) on node 2 (rank 2 in comm 0): application called MPI_Abort(MPI_COMM_WORLD, 6) - process 2
TACC:  MPI job exited with code: 6

This is using the exact same inputs, compilation flags, etc. from the tutorial case. I did try both intel and gnu compilation, both erring at the same time step.

I'll admit I haven't run this case in a while and it's not part of regression tests we perform periodically. I remember putting together an initial solution that was a bit odd. I'll re-run to completion and ensure that it is possible to reach a stable premixed bunsen flame.
The error you get is most likely the result of the initial solution/increase step size and I think a couple of option we introduced recently should fix that.

Okay sounds good! Just let me know either way. It's not critical for me that the case runs to completion, I was able to drop the CFL condition lower to run the simulation further. This was more a test for me to ensure that I installed everything correctly, and since it broke, I wanted to bring it to you attention.

I've updated the case in the latest version of development. It does go through with ~2e-5 s step size, but this is definitely very coarse and there are significant overshoots of intermediate species.

Thank you @esclapez ! I ran the case to steady state successfully, both in it's current state and with an additional level of AMR. I really appreciate you looking into it so quickly!


Cant find Clerk FDW inside extension list

Bug report

[ ] I confirm this is a bug with Supabase, not with my own application.
[ ] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
Can't find Clerk FDW inside the extension list
To Reproduce
As per this blog post - https://supabase.com/blog/supabase-wrappers-v02 under "New Wrappers and Improvements", it is mentioned as Clerk as supported FDW.
Expected behavior
However, I cant find inside the Supabase instance, (updated to latest pg version as well)

Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: [e.g. macOS, Windows]
Browser (if applies) [e.g. chrome, safari]
Version of supabase-js: [e.g. 6.0.2]
Version of Node.js: [e.g. 10.10.0]

Additional context
Paul and Tembo team agreed that it will be supported as long as Tembo maintains this (on X)

From the article

We cannot support community Wrappers inside the Supabase Dashboard until the Wrappers API is stabilized. You can vote your favorite Wrapper if you'd like it to be added to Supabase in the future. If you have developed a Wrapper that you want inside the Supabase Dashboard, please contribute it as a PR in the wrappers repo. Once we release Wrappers 1.0, we will support community Wrappers within the Supabase Dashboard.

clerk_fdw is not currently available as mentioned in the blog post.


Initialize login dialog with prefilled settings

This is a draft to fix #223. We could add a way to store defaults in a special class method, what do you think?
However, I think the prefilled port value goes terrible with the field labels:

Pull Request Test Coverage Report for Build 757

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 74.667%

Totals

Change from base Build 750:
0.0%

Covered Lines:
1232

Relevant Lines:
1650

💛  - Coveralls

I agree with both. defaults should be stored somewhere, but this looks terrible. Did you test that with different color themes?
Maybe not our work to be done...

I guess the only acceptable way to deploy a prefilled value for the port field would be to alter the form to use external labels (on the left side of the field). However, I think the way the dialog is built now is the standard Squeak way, so we should settle on just having Use SSL automatically checked as .

As @Paulpanther suggest, a solution could be to make port 993 an implicit default and show that to the user with a field label like port (default = 993).

I like all the ideas of 933 being the default value, changing the field label and the prechecked ssl box.
This should make for a great user story

The base branch is heavily diverted recently, so as this a simple thing I will start this from scratch.


add images to the readme to show UI

Awesome idea, mind doing one with timeline tab?

Awesome idea, mind doing one with timeline tab?

Added a gif with a timeline


premove survey changes

Description
This PR makes the following changes based on this doc from design: https://docs.google.com/spreadsheets/d/1leyc06TJ5nGXKPIZXzZ4xLGIftRWFPr3MSKYfBVRQQ4/edit?usp=sharing

removes latest pickup date
removes earliest delivery date
changes latest delivery date to planned_delivery_date
changes pack_date and pickup_date to be prefixed with "planned"
adds a field for how the PM survey was conducted - pm_survey_method. i.e. In person, over the phone, or video.

Setup
-populate db with hhg awarded data: make db_populate_e2e
-run tsp app, select hhg move info from table - verify the values in the premove survey box match the data. Edit the values, ensure they persist.
Code Review Verification Steps

[ ] End to end tests pass (make e2e_test).
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in
Querying the Database Safely
have been satisfied.
[ ] Any migrations/schema changes also update the diagram in docs/schema/dp3.sqs
[ ] There are no aXe warnings for UI.
[ ] This works in IE.
Any new client dependencies (Google Analytics, hosted libraries, CDNs, etc) have been:

[ ] Communicated to @willowbl00
[ ] Added to the list of network dependencies

[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change
Reference dates doc: https://docs.google.com/spreadsheets/d/1leyc06TJ5nGXKPIZXzZ4xLGIftRWFPr3MSKYfBVRQQ4/edit?usp=sharing

Screenshots

This isn't directly related to changes in this PR but pkg/handlers/shipments_public.go is erroring on line 80 with "sql: no rows in result set" if there are no shipments. I don't think this should error. I'm also not getting any shipments in the tsp view when I run make db_populate_e2e, which seems to be a change (and contradicts the instructions on this PR).


connection pool support

Hello,
I am developing a web application in which I use pyexasol to query for data from Exasol.
Is it possible to use a connection pool in pyexasol?
Having a connection pool would allow the web server to maintain long-running connections in memory for efficient re-use.
Thanks

@nautilus28 , it might be a bit difficult to implement proper connection pooling with Exasol due to lack of function to reset session state. At this moment all state changes (e.g. autocommit, query timeout, script languages) will be carried over to the next session.
Also, all result handles and prepared statements will remain opened, unless we manually track the current state of each object.
On top of that, Exasol instance has only 100 slots for parallel query execution. But it's a good idea to keep this number lower than 20-30 and use "Queue" pattern instead to limit parallelism. Instead of having potentially large spikes of workload, just queue it and execute only 5-10 queries at the same time. All jobs will be finished faster that way.
If you have a busy web-interface querying analytical database, it is advised to add a simple caching layer (MySQL, PostgreSQL, Redis, Mongo, SQLite, Tarantool, etc.) and send web requests to this layer instead. You may update data in this layer using ETL process.

@wildraid Great answer, thank you so much.


Multiple clients in browser-client code

Hey there,
I'm using example_browserclient code by having seperate server and client pc (server.py code in server pc and index.html and client.js code in the client pc). I established the connection by connecting both the PCs in the same internet and hosting the server via server PC's IP address (instead of hosting it to localhost). It is working perfectly.
Can I handle multiple clients? Cos, I tried connecting another client and gave input to it, but got the same output in both the clients, eventhough the other client was set idle and given no input.

This is not implemented yet, so it can't handle multiple clients. RealtimeSTT can't do multiple transcriptions in parallel, so for this scenario some way of queueing the requests has to be implemented.

in what way? should the other clients wait for the previous client to finish?

Yes. Or you need to use multithreading or multiprocessing and probably use multiple gpus. This is due to how the dependent libraries are implemented, can't do much here.


Fix a typo (Request Component page)

Just a typo fix

While you are on the request component page, could you fix the typo in the Querystring section?
    return Response(f'Name: {request.args['name']}'.encode())

to
    return Response(f'Name: {request.args["name"]}'.encode())

So it limits all those docs PR.


Feedback from M4

Implement all minor
Major:
Minor:

[x] Invert cards to be green background and white text font
[x] Clearer about terms like "apparent temp" etc. Fix axis titles.
[x] Style for reload button
[x] Remove "Date" from x-axis
[x] Make font in parenthesis smaller
[x] Why is the date popup not close to the widget
[x] Select only most important temperatures
[x] Link to repo, names, at the bottom of the sidebar

Your suggestions are very helpful and we have made changes on our dashboard based on your all suggestion. Done! Thank you!


STATUS_ACCESS_VIOLATION on every page, even settings.

STATUS_ACCESS_VIOLATION on every page, even settings.
Supermium 117, W7 X64

Hello?

There were many reports of similar problems with that release of the browser so I did not respond to each one individually.
I have now made another release of the browser which should not have this issue. If it does persist, send dumps as stored in AppData/Local/Chromium/User Data/Crashpad,

Dumps:
reports.zip

These are DirectWrite-related crashes. I also see that MacType is loaded, so that may also be responsible. Does the browser work any different without MacType?
I briefly tested with MacType and I haven't been able to produce any crashes yet, but I will check to see if I can hit the same codepaths, even though this shouldn't be an issue as the offending DWrite factories are very recent or very old (Windows 7 with Platform Update or Windows 10 20xx).

Disabling FontSubstitutes in my MacType profile (the included ChicoThorn .ini) for Chrome.exe seems to fix this. Cent Browser 5.0.1002.354 (102.0.5005.167) and ungoogled-chromium (109.0.5414.120) work fine with FontSubstitutes

I made a fix for it, which seems to make it work just as it does on Chrome 109. I just have to compile for x64 (I've only compiled for x86 so far) and then I will upload it.

This may resolve your issue: https://github.com/win32ss/supermium/releases/tag/v117-hf

Fresh install, same issue.
new dumps:
reports.zip


Unprovided export error with Vite SSR

Describe the bug
When attempting to use solid-app-router with vite's SSR mode, vite complains that solid-js/web doesn't have an export named effect. Indeed, there are only two export statements from the solid-js/web es module and neither of them contain effect:
export { ErrorBoundary, For, Index, Match, Show, Suspense, SuspenseList, Switch, createComponent, mergeProps } from 'solid-js';
// ...
export { Assets, Dynamic, HydrationScript, NoHydration, Portal, escape, generateHydrationScript, getHydrationKey, isServer, pipeToNodeWritable, pipeToWritable, renderToStream, renderToString, renderToStringAsync, resolveSSRNode, spread, ssr, ssrAttribute, ssrClassList, ssrHydrationKey, ssrSpread, ssrStyle };

I wasn't sure if I should open this bug here so that solid-js/web could export effect or if I should open over at solid-app-router so it could grab the export from a different location.
Your Example Website or App
https://github.com/fastify/fastify-vite/tree/967be02cd66539b3cf70dc9df59b5c0af2c0f853/examples/solid-hydration
Steps to Reproduce the Bug or Issue

Attempt to run example provided
Get the following error:

> NODE_ENV=development ts-node server.ts

3:04:21 PM [vite] Error when evaluating SSR module /base.tsx:
file://.../node_modules/solid-app-router/dist/index.js:1
import { isServer, template, createComponent as createComponent$1, mergeProps, spread, insert, effect, setAttribute } from 'solid-js/web';
                                                                                ^^^^^^
SyntaxError: The requested module 'solid-js/web' does not provide an export named 'effect'
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:128:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:194:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:409:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:435:15)
    at async nodeImport .../node_modules/vite/dist/node/chunks/dep-59dc6e00.js:59403:21)
    at async eval (/base.tsx:7:31)
    at async instantiateModule (.../node_modules/vite/dist/node/chunks/dep-59dc6e00.js:59333:9)
3:04:21 PM [vite] Error when evaluating SSR module .../client/index.ts:
.../node_modules/solid-app-router/dist/index.js:1
import { isServer, template, createComponent as createComponent$1, mergeProps, spread, insert, effect, setAttribute } from 'solid-js/web';
                                                                                ^^^^^^
SyntaxError: The requested module 'solid-js/web' does not provide an export named 'effect'
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:128:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:194:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:409:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:435:15)
    at async nodeImport (.../node_modules/vite/dist/node/chunks/dep-59dc6e00.js:59403:21)
    at async eval (/base.tsx:7:31)
    at async instantiateModule (.../node_modules/vite/dist/node/chunks/dep-59dc6e00.js:59333:9)
.../node_modules/solid-app-router/dist/index.js:1
import { isServer, template, createComponent as createComponent$1, mergeProps, spread, insert, effect, setAttribute } from 'solid-js/web';
                                                                                ^^^^^^
SyntaxError: The requested module 'solid-js/web' does not provide an export named 'effect'
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:128:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:194:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:409:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:435:15)
    at async nodeImport (.../node_modules/vite/dist/node/chunks/dep-59dc6e00.js:59403:21)
    at async eval (/base.tsx:7:31)
    at async instantiateModule (.../node_modules/vite/dist/node/chunks/dep-59dc6e00.js:59333:9)
zsh: exit 1     npm run dev

Expected behavior
solid-app-router should only import things from solid-js/web that actually exist.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: n/a
Version: 1.4.3

Additional context
No response

This is mostly likely the client build getting pulled in on the server. We havent' found a great solve here other than basically adding all Solid libraries that use JSX  to ssr.noExternal in the Vite config. I know Svelte has been working on similar issues. At one point they hardcoded every known library, not sure if they found a better solve.

I'm unable to reproduce, the example boots and runs fine for me.

Indeed, I have noExternal set for solid-app-router in this example as well.

@ryansolid Maybe the Solid Vite plugin could automatically make this setting given solid-app-router is an official package?

That's what I meant by hardcoding. It seems like a poor solution since literally every library needs to do this. We need a smarter solution here. Just not sure how to do it.


[Bug Report] Nested any/all

odata2ts version: 0.31.1
API: Microsoft.AspNetCore.OData 8.2.4
Triying to do a query with nested any/all using query builder gives server response
"The range variable 'a' has already been declared."
since every any/all uses the same placeholder 'a'

Hi @JuncalLP,
using custom prefixes is actually supported: It's the second parameter of the any or all function.
From the tests:
  test("any with prefix", () => {
    const result = createToTest()
      .any((qTest) => qTest.id.gt(18), "testing")
      .toString();
    expect(result).toBe("test/any(testing:testing/id gt 18)");
  });

Added a todo for the documentation.
I close this issue assuming that this solves your problem.


Can emacs delete obsolete package and upgrade to latest one in elpa at startup?

two elpa questions:

how to automatically update package to the new version when it's available?
How to automatically remove the directory of old version of package automatically?

For example,
I installed rinari-2.2 through elpa 3 months ago, I need emacs upgrade rinari to 2.6 ASAP. But I hate to list-packages to check when new version is release from time to time.
My purpose is make my elpa/ clean and find a easy way upgrade the packages I care.

@GordonGustafson: I'm one of those folks that have a daemon running all the time :)
Did you ever found how to remove the obsolete packages? I'm quite interesting in the bit of code you pasted.

No, I never did get around to figuring out how to remove the packages. Also, that code broke in Emacs 24.4 due to package-desc-vers being replaced by package-desc-version. I thought the fix would be easy, but I remember struggling with it and never finding a solution.
I've since removed that code from my init file, and now I use M-x list-packages U x, which takes care of finding out of date packages and deleting old ones. I'm sure there exists a way to invoke this functionality on startup, but I'm pretty sure you'll have to actually open up a list-packages buffer to do it. If you're like me you'll find that you're updating packages on start because it's cool rather than because you need it.

"If you're like me you'll find that you're updating packages on start because it's cool rather than because you need it."
Anyways, good to read you and learn from your experience, thank you :)

I've got a script that runs daily 'tasks' - one of which is updating elpa packages.  It relies on the package-utils elpa package.
Here is the (bash) function that handles this:
function upgrade_emacs_packages {
  echo 'upgrading emacs packages'
  emacs -batch -u "$USER" -f package-utils-upgrade-all -kill
}

This updates any packages and removes the old ones.

Lovely hack @rprimus - works quite well indeed.

I use paradox installed from melpa. It has a command paradox-upgrade-packages which upgrades all packages and delete the old ones.

Thanks for the heads up Lin, didn't knew it :)

packages-utils-upgrade-all is no more exist (and some other there)
this post is outdated and then obsolete.
It should be deleted for clarity and not make users waste time.

@rprimus use what you want... i just use M-x package-install-packages then usual key bindings: U for update, d for select delete, etc... x for execute orders.
Nevermind, this thread is (again) obsolete and make waste time to users.

M-x package-autoremove is now built in.

When I do M-x list-packages , I see a lot of obsolete and incompat packages. Unfortunately, running M-x package-autoremove returns "Nothing to autoremove" :[

When I do M-x list-packages , I see a lot of obsolete and incompat packages. Unfortunately, running M-x package-autoremove returns "Nothing to autoremove" :[

I experienced the same. Some obsolete packages wont removed from the list but the new version is installed. It happens probably because I installing the new version without removing the old package. M-x Package-autoremove wont remove it, while other obsolete packages is disappear. But I am not sure.

When I do M-x list-packages , I see a lot of obsolete and incompat packages. Unfortunately, running M-x package-autoremove returns "Nothing to autoremove" :[

I experienced the same. Some obsolete packages wont removed from the list but the new version is installed. It happens probably because I installing the new version without removing the old package. M-x Package-autoremove wont remove it, while other obsolete packages is disappear. But I am not sure.

Any examples of what packages are getting listed as "obsolete" and "incompat"?

Any examples of what packages are getting listed as "obsolete" and "incompat"?

Here is mine, the inf-ruby package.

That's weird - upgraded packages should get removed automatically by package.el during upgrade.

Similar problem here with "realgud" :

I note there's a specific command called package-menu-mark-obsolete-for-deletion for targeted clean-up of such packages, so perhaps this is due to a known edge case. I found I had a few obsolete packages locally too, just older versions of other installed packages from MELPA. Perhaps when a package install fails, it sometimes leaves older versions in place, unsure.


updating how to apply removing reference to 2 sections

Trello card
https://trello.com/c/1fDma3EC
Context
In Apply, they're planning to move to 1 personal statement question and we need to reflect this on this page.
Changes proposed in this pull request

removed references to 2 questions
aligned advice on what to cover with apply
added info about deciding who to teach
added link to adviser support

Guidance to review

Apply changes will be in prod from 9am


Package com.javatodev.finance.rest is missing

In the internet-banking-user-service project you refer some classes (BankingCoreFeignClient and UserResponse) to the package
com.javatodev.finance.rest.*, but this pacakage doesn't exists

Now it's sorted and service is in working stage


Fix #7 Off by One Error

May want to add some additional commands regarding sub modules to the readme, for example;
git submodule update --remote --merge

travis seems to not like building branches where only submodules have changed.
https://travis-ci.org/OR13/truebit-os/builds/381889318

Huh, well thanks for this!


fix: macos laggy [#158]

I saw issue #158 and I decided to check it out because I'm using MacOS. I updated tween.js version and I have the impression that it works better now. I don't see MacOS lag in Safari and Chrome.

oh cool @cyntler I'll check it out and get back to you tomorrow

oh cool @cyntler I'll check it out and get back to you tomorrow

Ok! :)


Add Geneve tunneling protocol support

Introduce Geneve tunneling protocol
described in :
https://tools.ietf.org/html/draft-ietf-nvo3-geneve-03

I changed the copyright lines, thanks


🐞 - tui-select broken with SSR

Playground Link
No response
Description
I am trying to enable SSR for my website.
I have to apply ngSkipHydration to all s. If not i get the error:
NG0502: During serialization, Angular was unable to find an element in the DOM:
…
#node(projection)
  <-- AT THIS LOCATION
…
To fix this problem:

check corresponding component for hydration-related issues
check to see if your template has valid HTML structure
or skip hydration by adding the ngSkipHydration attribute to its host node in a template

Find more at https://angular.dev/errors/NG0502
at nodeNotFoundError
at calcPathForNode
at appendSerializedNodePath
at conditionallyAnnotateNodePath
at serializeLView
at annotateHostElementForHydration
at serializeLView
at annotateHostElementForHydration
at serializeLView
at annotateHostElementForHydration
Is this something that is known?
I may try to provide reproduction if it is needed.
Angular version
18.0.4
Taiga UI version
3.66.0
Which browsers have you used?

[ ] Chrome
[X] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS
[ ] Android

I've also tried with latest 3.x taiga, does not help.

@hheexx please add reproduction on github


Need to test on iPhone

So far this hasn't been run on iOS at all.  Especially interested in the location service and if it runs in the background properly.

Tested by several


terragrunt always re-runs terraform init with terraform 0.14.0-rc1

It seems terragrunt fails to detect when initialization is not necessary when using terraform 0.14.0-rc1. It always runs terraform init no matter what, and it takes a lot of time especially when dependencies are involved.
I also tried terragrunt apply --terragrunt-no-auto-init, and here's what happens:
$ terragrunt apply --terragrunt-no-auto-init
...
[terragrunt] 2020/11/13 01:29:18 Cannot continue because init is needed, but Auto-Init is disabled.  You must run 'terragrunt init' manually.
[terragrunt] 2020/11/13 01:29:18 Unable to determine underlying exit code, so Terragrunt will exit with error code 1

This is the problem: https://github.com/gruntwork-io/terragrunt/blob/7f0920ab4b961984a8d1cb60af48a2bb00961ad9/cli/cli_app.go#L723
It seems that terraform renamed .terraform/plugins into .terraform/providers. So the check needs to be:
        providersPath013 := util.JoinPath(terragruntOptions.DataDir(), "plugins")
	providersPath014 := util.JoinPath(terragruntOptions.DataDir(), "providers")
	return !(util.FileExists(providersPath013) || util.FileExists(providersPath013))

Thanks for reporting! Since 0.14.0 is not out yet, we have not yet tested or updated Terragrunt to work with it. We will do so once the final version is released.

Now that 0.14 is out I think this must be locked at

Ah, good point, thanks for the reminder!

Also just ran into this myself 😄

Facing the same issue

A temporary workaround for this issue is to create a empty folder plugins in
.terragrunt-cache/RANDOM_HASH/ANOTHER_RANDOM_HASH/.terraform/
like
mkdir .terragrunt-cache/X0em602TiSrcCnD8MSB9SfZ-Wiw/Dtif5ds3Xi5dZzANQWFL-b_aEkQ/.terraform/plugins
Terragrunt stops reinit after created this empty folder.

@johnlinvc thanks your workaround makes it a lot more usable was so slow before

can we release that!

can we release that!

Super thanks for the fix! Is it possible to get it released soon?

Super thanks for the fix! Is it possible to get it released soon?

We're working on it in https://github.com/gruntwork-io/terragrunt/pull/1459. We were out on vacation the last couple weeks for the holidays, but will pick it up again later this week.

We're working on it in https://github.com/gruntwork-io/terragrunt/pull/1459. We were out on vacation the last couple weeks for the holidays, but will pick it up again later this week.

Please give https://github.com/gruntwork-io/terragrunt/releases/tag/v0.27.0 a shot!

Please give https://github.com/gruntwork-io/terragrunt/releases/tag/v0.27.0 a shot!

Thanks @brikis98, 0.27.0 fixed this for me. 👍

Thanks @brikis98, 0.27.0 fixed this for me. 👍


Update docker-compose to use catalog

We have an awesome docker-compose.yml with tiled+prometheus+grafana, documented here https://blueskyproject.io/tiled/how-to/metrics.html.
This has had an annoying extra step: we need the user to provide some data to serve, or thread a toy config file in, which is not a very realistic scenario or useful in production. But now, with the writable catalog in Tiled, we can launch empty instance and just tell the user to write some data in. This is something that could be useful to just deploy as is.
Question: How to deal with the fact that the database needs to be initialized at some point? How do people normally handle this? We could make it a two-step process:

Run the tiled container with the COMMAND tiled catalog init ... to initialize the database.
mkdir storage
podman run -v /storage:./storage:rw tiled tiled catalog init /storage/catalog.db

podman-compose up

Is that the best we can do?
Secondary question: Should we ship the docker-compose.yml such that it builds the current repo or should we hard-code a tagged version of pre-built tiled? Or should we ship both and if so what should we call them?

Just to state an alternative: another possible answer to the first question is to add some kind of --init-if-empty flag. But this kind of conditional initialization makes me itchy. I can imagine it going very wrong in a scaled deployment where N containers are launching simultaneously. That seems like a pretty realistic scenario.

For the first question., I think the official mongo image handles this with external files that you can mount. I don't know if we want to go that far but see Initializing a fresh instance. Postgres has a similar mechanism. These both seem cool but a lot of work to maintain for us though.
For the second question, I think a tag to the prebuilt image is sufficient. Providing a docker-compose file is a favor for people who want kick the tires. If you're building from current source, you probably know how to volume mount the source into the repo.  Some projects provide a docker-compose-dev.yml or the like, but that seems excessive to me.

Oh, that Mongo link was interesting. I think we could apply that easily enough. Imagine docs like...

For the first use, run:
TILED_INIT_DB=./storage/catalog.db TILED_SINGLE_USER_SECRET_KEY=secret podman-compose up

The TILED_INIT_DB parameter creates and initializes a database that Tiled uses to store metadata.
If you restart the container later, omit TILED_INIT_DB; the database is already initialized.
Above we are using a simple file as the database (SQLite). Alternatively, a PostgreSQL URI may be used
TILED_CATALOG_URI="postgresql+asyncpg://username:password@localhost/database_name:5432"
TILED_INIT_DB=$CATALOG_URI
podman-compose up

In terms of maintanance, I think that just requires adding a small bash script to the image, using it as the default CMD. I wouldn't expect it evolve much from there.

I implemented something that is a bit simpler than one I outlined above. I like this.

Superseded by #507


Inconsistent result of overlap matrix of cc-pVDZ for H2 compared to libcint

The overlap matrix I got from the following code
from pyscf import gto
mol = gto.M(unit ='bohr', atom='H 0 0 0; H 1 0 0', basis='cc-pvdz')
s = mol.intor('int1e_ovlp')

has a few matrix elements quite different from what I would get by using libcint. Specifically, at indices:
[0, 0], [1, 0], [5, 0], [6, 0], [7, 0], [0, 1], [5, 1], [5, 2], [0, 5], [1, 5], [2, 5], [5, 5], [6, 5], [0, 6], [5, 6], [0, 7].

You may need to check the way you generate _env (compare with gto.mole.make_env), especially the structure convention of _env and the normalization step. If this does not help, you can post some demo codes describing how you use libcint and what results you have got.

You may need to check the way you generate _env (compare with gto.mole.make_env), especially the structure convention of _env and the normalization step. If this does not help, you can post some demo codes describing how you use libcint and what result you have got.

Hi! Thanks for the reply! This is the result I got from mol._env:
[ 0.          0.          0.          0.          0.          0.
  0.          0.          0.          0.          0.          0.
  0.          0.          0.          0.          0.          0.
  0.          0.          0.          0.          0.          0.
  1.          0.          0.          0.         13.01        1.962
  0.4446      0.57976406  0.98341958  1.11930215  0.122       0.52153673
  0.727       1.95840453]

But I couldn't make gto.mole.make_env(mol._atm, mol._basis) as it would prompt
AttributeError("'numpy.int32' object has no attribute 'isalpha'")

On the other hand, it's kind of hard for me to post a clean demo code using libcint because I was wrapping it using a programming language called Julia and designed a proprietary data type to represent the basis function. I can, however, give you the result I got from my code (the ordering of angular momentum part of GTO is (1,0,0), (0,1,0), (0,0,0) in cartesian representation):
  1.0        0.903687  0.0       0.0       0.0       0.858338  0.836484  -0.378876  0.0       0.0
  0.903687   1.0       0.0       0.0       0.0       0.836484  0.940823  -0.129713  0.0       0.0
  0.0        0.0       1.0       0.0       0.0       0.378876  0.129713   0.1898    0.0       0.0
  0.0        0.0       0.0       1.0       0.0       0.0       0.0        0.0       0.695239  0.0
  0.0        0.0       0.0       0.0       1.0       0.0       0.0        0.0       0.0       0.695239
  0.858338   0.836484  0.378876  0.0       0.0       1.0       0.903687   0.0       0.0       0.0
  0.836484   0.940823  0.129713  0.0       0.0       0.903687  1.0        0.0       0.0       0.0
 -0.378876  -0.129713  0.1898    0.0       0.0       0.0       0.0        1.0       0.0       0.0
  0.0        0.0       0.0       0.695239  0.0       0.0       0.0        0.0       1.0       0.0
  0.0        0.0       0.0       0.0       0.695239  0.0       0.0        0.0       0.0       1.0

I was able to reproduce this result in other packages like gbasis and pyqint, but not the one from PySCF. An interesting discovery was that when I used the PySCF interface function gbasis.wrappers.from_pyscf in gbasis to generate the basis set, I got the same result as what PySCF would generate. I suspect there's something wrong with the basis set building process in PySCF?

First, are you using exactly the same basis set as PySCF? You can compare PySCF's built-in basis set in pyscf/gto/basis with yours.
Second, whatever wrapper you use, you must pass a _env to libcint. Could you print it out and compare it with PySCF's mol._env?

Oh, I found that if I copy the cc-pvdz basis set from BSE, I can reproduce your result with PySCF.
PySCF's built-in cc-pvdz is different from BSE's default version of cc-pvdz but the same as BSE's "Optimize General Contractions" version of cc-pvdz.

Oh, I found that if I copy the cc-pvdz basis set from BSE, I can reproduce your result with PySCF. PySCF's built-in cc-pvdz is different from BSE's default version of cc-pvdz but the same as BSE's "Optimize General Contractions" version of cc-pvdz.

Oh, thanks for the reply! Later I noticed as well that the basis set data I used was different from the one in PySCF, but I didn't know which configuration the basis set in PySCF corresponds to. Thanks for the clarification!


charred should be tried before data.json

@lispyclouds charred should be tried before data.json since this library has a dependency on data.json for convenience, it will always win
https://github.com/babashka/json/blob/1d91f3ba62c50e1966e9dfd3c52519cc7d312fbb/src/babashka/json.clj#L13

ah, doing another PR.

Fixed in https://github.com/babashka/json/commit/99e984bf9dcf4f35b4d0911807e792b90213e9dc


HTML5 Video Player - failed to toggle FullScreen mode by clicking fullScreen control

Are you requesting a feature or reporting a bug?
bug
What is the current behavior?
Click on fullScreen control seems not working
What is the expected behavior?
Click on fullScreen control will switch video player to full screen mode
How would you reproduce the current behavior (if this is a bug)?
Following code should reproduces the issue:
  const client_isFullScreen = ClientFunction (() => {
        return document.webkitCurrentFullScreenElement &&
            document.webkitCurrentFullScreenElement.nodeName == "DIV";
  });

   isFullScreen: async function (){
        return client_isFullScreen()
    }

    playerFullScreenToggle: async function (t: TestController){
      await t
          .click(".player-button.player-control-fullscreen")
    }

fixture`Video Player Test Suite`
    .page("https://kabbalahmedia.info/en/lessons/cu/koUvHS4T?language=en");

test('fullScreenToggle', async t => {
    await player_utils.waitForPlayerToLoad();
    await t.expect(await player_utils.isFullScreen()).notOk();
    await player_utils.playerFullScreenToggle(t);
    await t.expect(await player_utils.isFullScreen()).ok();
    await player_utils.playerFullScreenToggle(t);
    await t.expect(await player_utils.isFullScreen()).notOk();
});

  AssertionError: expected null to be truthy

  Browser: Chrome 69.0.3497 / Mac OS X 10.13.6

      97 |test('fullScreenToggle', async t => {
      98 |    // await setHeight(selectors.player.controls.fullScreen);
      99 |    await player_utils.waitForPlayerToLoad();
     100 |    await t.expect(await player_utils.isFullScreen()).notOk();
     101 |    await player_utils.playerFullScreenToggle(t);
   > 102 |    await t.expect(await player_utils.isFullScreen()).ok();
     103 |    await player_utils.playerFullScreenToggle(t);
     104 |    await t.expect(await player_utils.isFullScreen()).notOk();
     105 |});

Provide the test code and the tested page URL (if applicable)
Tested page URL: https://kabbalahmedia.info/en/lessons/cu/koUvHS4T?language=en
Specify your

operating system: MacOS HighSierra
testcafe version: 0.21.1
node.js version: 9.8

I have a similar problem with jwplayer. No problem is detected with other player's actions (play, pause, mute, unmute, close ad, etc)