etsy oauth image upload

I am having a problem with this.
I have been experimenting for 3 days, but I cannot upload the image uploaded by postman with javascript.
I used such a code.
`
var authHeader = oa.authHeader(
"https://openapi.etsy.com/v2/listings/"+listingId+"/images",
req.session.oauth.access_token,
req.session.oauth.access_token_secret,
);
console.log(authHeader);

var options = {
    "method": "POST",
    "url": "https://openapi.etsy.com/v2/listings/"+listingId+"/images",
    "headers": {
        "Authorization": authHeader,
    },
    formData: {
        "image": {
            "value": fs.createReadStream(path.join(__dirname, "images/"+image_path+".jpg")),
            "options": {
                "filename": path.join(__dirname, "images/"+image_path+".jpg"),
                "contentType": null
            }
        }
    }
};
request(options, function (error, response) {
    if (error) throw new Error(error);
    console.log(response.body);
});

`
I get this error.
OAuth oauth_consumer_key="cpuao0wzd2q3l152x30nzebp",oauth_nonce="R88aVNfFRhz17yA3nQYeX4ZE1PfG1Zhb",oauth_signature_method="HMAC-SHA1",oauth_timestamp="1597230278",oauth_token="a02ee077a56d97527ade2ecd24423c",oauth_version="1.0A",oauth_ signature="iCBcTHEXb638wXabJPJ2RUQeeew%3D" oauth_problem=signature_invalid&debug_sbs=POST&https%3A%2F%2Fopenapi.etsy.com%2Fv2%2Flistings%2F839582160%2Fimages&oauth_consumer_key%3Dcpuao0wzd2q3l152x30nzebp%26oauth_nonce%3DR88aVNfFRhz17yA3nQYeX4ZE1PfG1Zhb%26oauth_signa ture_method%3DHMAC-SHA1%26oauth_timestamp%3D1597230278%26oauth_token%3Da02ee077a56d97527ade2ecd24423c%26oauth_version%3D1.0A

I have not been able to resolve this problem using node-oauth but I did manage to get the image upload working using request. It's a deprecated package but I wanted something that worked. Here is the code:
function uploadListingImage(listingId, image) {     const req = request.post({       url: https://openapi.etsy.com/v2/listings/${listingId}/images`,
oauth: {
consumer_key: etsyKeys.key,
consumer_secret: etsyKeys.secret,
token: etsyKeys.user_token,
token_secret: etsyKeys.user_secret
}
}, (err, response, body) => {
if (err) {
// error
}
if (response.statusCode === 201) {
// success
console.log(body.results[0]);
}
});
const form = req.form();
form.append('image', image, {
filename: 'cover-image.jpg',
contentType: 'image/jpeg'
});
}`
Where listingId is self-explanatory and image is a buffer of the image read using fs.readFile.

I think the problem is that node-oauth puts the authorization parameters in the header, whilst the etsy api expects them to be in the post body. I tried to hack it to make it work but in the end did more or less the same thing as you. In my case I used oauth-1.0a to generate authorization content.

I think the problem is that node-oauth puts the authorization parameters in the header, whilst the etsy api expects them to be in the post body. I tried to hack it to make it work but in the end did more or less the same thing as you. In my case I used oauth-1.0a to generate authorization content.


Vosk api: allow selecting different models and automatic model download

Hello,
I added/changed two parameters in the  recognize_vosk function, which i believe to be useful.
Firstly, I added a model parameter to allow to select a model based on a model directory. This was previously hard coded to one directory named 'model', making it impossible to easily switch models respectively languages.
Secondly, I noticed that Vosk-api is actually able to download models by itself, based on a given language code. So I implemented this as another parameter 'language'. Previously there was a default language parameter provided in the function, but it was never used.
I implemented it so that the model parameter has precedence over the language parameter, but it defaults to an empty string (False). So that by default the language model is downloaded automatically, because i believe this to be more convenient for the user. However I'm aware that this breaks the previous behavior and may break a user implementation if they want to use a very specific model that they already downloaded (for example one of the larger models). If you think this is a problem i could try to change that.
Also see the updated README in the commit.
Please let me know what you think
Thanks
ps.: if this gets merged i would also update the documentation and maybe write some tests for vosk, which i think are still missing.
pss: also it may be good to actually change to return value of the vosk function to make it more in line with the other functions, since i think this is currently returning a json string, instead of a simple string like the others. (Edit: mentioned here aswell: https://github.com/Uberi/speech_recognition/pull/592 )
Edit: also now i found this similar pull request https://github.com/Uberi/speech_recognition/pull/607 adding a model path

Hi everybody!
Just out of curiosity, there is any particular reason for not to merge this PR?
It looks like a nice-to-feature for which @rebootl proposed a valid implementation.
I would be happy to work on it if needed!


Customize Address & Batch

Address

Field : ดึง field Address Title  to  Link Name ในตาราง

Customize Lot No. to Batch

Batch : change label Batch Type, Batch
Job Card:  change label Batch
Quality Inspection:  change label Batch, Material Batch
Work order:  change label Batch
Stock Entry Detail: change label Batch

👍


Remove restart votes to be used by players.

People have been abusing this lately not for bugs or glitches or perhaps the server broke but instead use it because they don't like a certain map or other calls (dying and calling for restart)
It's become frequent and honestly sucks when someone selfishly calls for it when things don't go their way.
If something happens to the server or some glitch happens to make it unplayable, there are other ways to restart the server than relying on random players.

apply for miros 24/7 admin :trollface:

agree with ninruB, this is needed as a failsafe


proposal: update credential storage strategy

Argo CD has always used the label argocd.argoproj.io/secret-type with value repository or repo-creds on Secrets to indicate that they are either:

Credentials for one specific repository (repository)
Credentials multiple repositories, all starting with the same URL prefix (repo-creds)

The label name changed, but Kargo copied this strategy almost exactly. Even at the time I did that, I felt this was confusing and have since found that others are also confused by it.
Confusion stems from:

Both label values indicate a Secret represents credentials, but one has "creds" in its name and the other doesn't. 🤔
One label values indicates a Secret represents something akin to a "credential template," but nothing in the label value hints at such behavior.
Apart from these labels, a Secret's data.type field is used to with a value like git, helm, or image to denote what kind of repository the credentials are for. Overuse of "type" contributes to confusion.

I'm sure Argo CD has rational reasons for why it is this way, but seeing the confusion it can cause at times, and given that credential management features (#1523, #1444, #1531) are currently high priorities, I think we're at the right juncture to ask whether we were wrong to blindly copy the Argo CD approach and whether we want to make a small course correction.
Note that #1483 removed the option to "borrow" credentials from Argo CD. Doing so may have eliminated the only remaining incentive for emulating Argo CD's credential strategy, oddities and all.
What I propose is:

Eliminating the kargo.akuity.io/secret-type and its two values.
Begin using Secret's own type field with one of the following values to indicate the type of repository the credentials are for:

kargo.akuity.io/git-creds
kargo.akuity.io/image-creds
kargo.akuity.io/chart-creds

Support two fields for specifying repository / repositories the credentials are for:

repoURL -- a single specific repository
repoURLPattern -- a regular expression identifying repositories the credentials are for

Implementation-wise, none of this is very complicated.
@jessesuen wdyt?

First, I think it's fine to rework the Secret convention for Kargo. So let's do what we want here. Especially since we may have other type of credentials (S3? AWS?)

Begin using Secret's own type field

There's some downsides to using the type field. I know the type field is intended for that purpose, but for whatever reason, in Kubernetes the secret type is immutable so you have to delete the secret and recreate if you want to modify the type field.
Second, if there aren't any labels, then you also can't filter secrets by the label. I think there may be value in preserving that property, though admittedly, I'd be hard-pressed if you asked me the use case. I do know filtering by labels allows you to reduce the informer memory consumption and the contents of watch streams (not that we currently do that).

A third downside is that requiring the secret type field to be one of ours, say kargo.akuity.io/git-creds, means that users will not be able to "reuse" existing, well-known secret types such as kubernetes.io/basic-auth, and kubernetes.io/dockerconfigjson by applying our label to it.
I can imagine a use case where those secrets might already exist, and kargo can take advantage of them. In particular, I wonder why we couldn't ask users to label a kubernetes.io/dockerconfigjson Secret with:
metadata:
  labels:
    kargo.akuity.io/secret-type: image-creds

And then we would instantly have read privileges to their container image repository without having them deal with another Secret.

Ok. All good points. There are definitely some details to hammer out here, some requiring some research and experimentation, but I won't change any of this without your buy-in.

@jessesuen and I spoke offline about this.
We won't use the built-in type field because of concerns he cited above -- not the least of which is the potential for someday reading Secrets that are formatted differently than what we currently require -- image pull secrets, for instance. We're not tackling that now.
The strategy I settled on takes some small steps to relieve some of the confusion of the existing strategy, but it isn't a major departure from what we already do.


0.7.4 -> 0.7.8 Cannot read property 'getDatabaseVersion' of undefined

Issue Summary
During an upgrade (forgot a few), so was going from 0.7.4 to 0.7.8. I followed the source upgrade, like usual and after starting the service. I was met with this.
><EMAIL_ADDRESS>start /home/connor/ghost
> node index

ERROR: Cannot read property 'getDatabaseVersion' of undefined

 TypeError: Cannot read property 'getDatabaseVersion' of undefined
    at Object.init (/home/connor/ghost/core/server/data/migration/index.js:81:22)
    at /home/connor/ghost/core/server/index.js:72:27
    at tryCatcher (/home/connor/ghost/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/home/connor/ghost/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/home/connor/ghost/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/home/connor/ghost/node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (/home/connor/ghost/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/home/connor/ghost/node_modules/bluebird/js/main/async.js:133:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/home/connor/ghost/node_modules/bluebird/js/main/async.js:15:14)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

Steps to Reproduce

Have a Centos7 box using MySQL and Ghost 0.7.4
Attempt upgrade to 0.7.8
See failure.

Technical details:

Ghost Version: 0.7.4 (before upgrade), 0.7.8 (during failure)
Node Version: v4.2.6
NPM Version: 2.14.12
Browser/OS: n/a
Database: MySQL (MariaDB 10.1.11)

It looks like the current DB version is 005. My database is 004. So I need an upgrade I think. Either way I think this commit caused it - https://github.com/TryGhost/Ghost/commit/1ad7a91f4ddbe83bb95e26a3d3bf1d980f8a4bcf
EDIT: I downgraded back to 0.7.4 and everything is working again. I'll hold off on upgrading.

It looks like the current DB version is 005. My database is 004. So I need an upgrade I think. Either way I think this commit caused it - 1ad7a91

The current database version is still 004 - not sure where you got 005 from?

I followed the source upgrade, like usual

I'm not sure what this means, but the error you're reporting sounds to me like an upgrade that hasn't completed properly. Either step 3 was missed, or step 4 did not copy all files correctly: http://support.ghost.org/how-to-upgrade/#3.-delete-the-old-files
Note that there is no database or fixture migrations happening between 0.7.4 and 0.7.8.  Also, L81 of core/server/data/migration/index.js (the line throwing the error) runs to check if it needs to do something every single time Ghost boots.
Therefore, if this was a bug, rather than an upgrade issue, everyone starting Ghost in 0.7.8 would experience it.
To verify that the folder structure is correct after an upgrade, you can run ls core/server/data/* and verify that the file locations match those in GitHub

I got this exact error when upgrading from 0.74 -> 0.78, and the problem turned out to be just what @ErisDS speculated: I didn't delete the contents of core before copying the new core files over to the production install. After doing an rm -rf /myghostdirectory/core/* and then copying the 0.78 core files to a fresh empty directory, the problem behavior ceased and my blog loaded without any problems.

@pokrface solution works. Had the same issue and done exactly what proposed here.

Just to clarify: if you're not deleting core as part of the upgrade, then you're not upgrading properly 😉
These is the official upgrade guide: http://support.ghost.org/how-to-upgrade/

@ErisDS yep, thanks for info! I've used tutorial on digital ocean for updating ghost and it didn't work :smiley:

@ykushch you should probably let them know that!

There is a solution for that with updating core in comments section :)

@ykushch can you link me to the documentation and I will report it - a comment is not good enough IMO 😉

@ErisDS Sure thing. Here it is digital ocean ghost update.


Browser button navigation for remote app routes

Good Day!
We have two Angular apps where the remote app is used as a MFE web component wrapper in the shell app.
And it looks like multiple router states get created even after sharing the @angular/router in webpack.
This leads to the browser back and forward buttons not working! (The popstate tracking is a bit messed up)
Please can someone help me with a solution or a work around?

While listening to the router events this is what I have come across :
On the popstate (browser back button clicked to go to home/client-configuration) it looks like the remote app router instance runs an imperative route to (home/my-clients path) which should have never happened (and loads the content of home/my-clients) and meanwhile the shell app router instance tries to run popstate on the right url path.
Is there any way we can connect the two routers to work in sync?

I made a PR #217  for this, but I found that the only way to get the route correctly handled is to do it during ngOnInit.
Remove appType: 'microfrontend' from you bootstrap call.
export class MfeComponent implements OnInit {
  constructor(private router: Router, private location: Location) {}

  ngOnInit() {
    this.router.initialNavigation();
    void this.router.navigateByUrl(this.location.path(true));
  }
}

@r1m
So does this mean this change has to be done to each and every component (which has routes) in the remote app ?

Only on first level components : the ones that are federated. Not inner ones.

Actually, there are two bugs.

back/forward always brings you to the route that was used to display the component (because pop_state listener is not using the current url but a snapshot)
deep links for module already loaded are not working because route refresh is hapenning only during module bootstrap

My PR fixes the first one, but my suggestion to move connectRouter in component fixes the second one.

@manfredsteyer Is this actually solved? Are there any docs about this issue?

Not really. @brunoalod
Look at issue 451

Hey there @RajathVenkatesh
Thanks to your PR and some hours reading angular/router and mf-tools plugin, I came to a possible solution.
First, I removed appType: 'microfrontend' from bootstrap call as suggested by @r1m .
Also I added this code inside ngOnInit of my AppComponent (first level component)
customInitialNavigation() {
    const url = `${location.pathname.substring(1)}`;
    this.router.navigateByUrl(url);

    this.nonRouterCurrentEntryChangeSubscription ??= this.location.subscribe(event => {
      if (event['type'] === 'popstate') {
        const routeBelongsThisRouter = event.url!.startsWith('/search'); // here goes the url you use in startsWith() on shell *
        if (routeBelongsThisRouter) {
          this.router.navigateByUrl(event['url']!, { state: event.state });
        }
      }
    })
  }

The first two lines were exctracted from your PR and correspond to the initial navigation (but fixed) made by connectRouter() from mf-tools.
The nonRouterCurrentEntryChangeSubscription idea came from reading @angular/router's setUpLocationChangeListener() method, used by initialNavigation() from @angular/router too.
I tried using those two methods, but they kept the subscription alive all through the browser session because mfe's AppModule keeps loaded, and so does RouterModule, even if you navigate out of it to shell or other mfe's. But AppComponent gets destroyed when you navigate out of mfe, so I can unsubscribe from that subscription, knowing that if I come back to the mfe, it will be created again:
ngOnDestroy(): void {
    if (this.nonRouterCurrentEntryChangeSubscription) {
      this.nonRouterCurrentEntryChangeSubscription.unsubscribe();
      this.nonRouterCurrentEntryChangeSubscription = undefined;
    }
}

This is also part of @angular/router's dispose() method

I use that const routeBelongThisRouter because if I popped back from mfe to shell, it was doing one extra navigation to the mfe base URL, because mfe Router doesn't know shell's routes, being 'search' the URL used to load the application as MFE.

Hope this helps


Doesnt work on android

Hello,
I just tried the library on Mac it works, same on iOS, but on android the status is always initial. Even when you try with the demo link and you want to open the camera it doesnt work on android...
The only error i have is:
MediaRecorder build fails: Parameter 1 is not of type MediaStream

Hello,
I just tried the library on Mac it works, same on iOS, but on android the status is always initial. Even when you try with the demo link and you want to open the camera it doesnt work on android... The only error i have is:
MediaRecorder build fails: Parameter 1 is not of type MediaStream

Android has so many versions, can u give us more detailed information about the android device you used to do the test?


Button texts untranslatable

Some buttons are untranslated and are not made in a way that they can be translated.

I'll wrap these in the translation handlers in a bit, thanks for spotting 👍

Should be visible in crowdin now 👍

Thanks. However, besides the settings, there were some more missing translatable buttons. PR #494 should fix them all.


Comments on directives are lost when running Organize Imports

Given the code:
import 'dart:io';
// ignore: deprecated_member_use
import 'dart:async';

Stream<String> a;
File b;

If you sort it, the comment is removed:
import 'dart:async';
import 'dart:io';

Stream<String> a;
File b;

@bwilkerson I think this is because this code works by replacing the whole block of directives with a newly computed one, but it only captures end-of-line comments when it builds the new text:
https://github.com/dart-lang/sdk/blob/93313eb2449099e20ade80d4760f76a325a4e176/pkg/analysis_server/lib/src/services/correction/organize_directives.dart#L68-L76
I think it could be easily extended to also capture preceding comments, however it might go bad if there's a "file header" comment before it:
/// Copyright ...
import 'dart:io';
import 'dart:async';

Some thoughts on fixing it:

Only "attach" comments if it's not the first directive (this means a comment on top of the first directive will not move with the directive)
Only attach // ignore comments (this might not solve the whole problem - eg. if the user added an explanatory comment to an import - // we're using this until the new library is ready).

What do you think?

I suspect that one of the reasons this hasn't been implemented well is because of the difficulty of getting the right heuristics.
My opinion is that we should use the following heuristics:

Every comment that follows a directive d that is on the same line as the last token of d (always a semicolon in valid code) should stay with d.
Every comment that precedes a directive d and is not on the same line as last token of the previous directive, should stay with d unless

there is no library directive,
d is the first directive in the file, and
there is at least one blank line between the comment and d.

I can't think of any other special cases that might need to be addressed, but I'm sure we'll find or hear about some.
(Unfortunately for me, this code doesn't appear to be shared with the "sort members" feature. Perhaps if we fix this code someone can look at sharing the code between the two features.)

We really should allow library; without a name, so people have something to hang their library documentation off.

It would be good to allow library; for cases where there is no need for other directives, but given the number of libraries that already make use of Dartdoc's heuristic, I doubt that we'll remove the heuristic. Certainly for now we need to handle that case correctly.

🎉  Thanks @jonas-jonas!


Question about ZSRE experiment, how to reproduce results in your WISE paper?

Hi,
I am testing the ZSRE task with WISE, MEMIT, and GRACE. But my results were different from yours in the paper. After checking the code, I still couldn't understand how current EasyEdit can utilize loc_prompts. As a result, from all methods I got pretty low locality scores.
Below is my complete code:
def load_data(args):
    # test_data = json.load(open(os.path.join(args.data_dir, 'zsre_mend_eval_portability_gpt4.json'), 'r', encoding='utf-8'))
    test_data = json.load(open(os.path.join(args.data_dir, 'zsre_mend_edit.json'), 'r', encoding='utf-8'))
    loc_data  = json.load(open(os.path.join(args.data_dir, 'zsre_mend_train.json'), 'r', encoding='utf-8'))

    if args.ds_size is not None:
        test_data = random.sample(test_data, args.ds_size)
        loc_data = loc_data[:args.ds_size]

    prompts = [test_data_['src'] for test_data_ in test_data]
    rephrase_prompts = [edit_data_['rephrase'] for edit_data_ in test_data]
    target_new = [edit_data_['alt'] for edit_data_ in test_data]

    locality_prompts = [edit_data_['loc'] for edit_data_ in test_data]
    locality_ans = [edit_data_['loc_ans'] for edit_data_ in test_data]
    locality_inputs = {
        'neighborhood':{
            'prompt': locality_prompts,
            'ground_truth': locality_ans
        },
    }

    portability_prompts = [edit_data_['portability']['New Question'] for edit_data_ in test_data]
    portability_ans = [edit_data_['portability']['New Answer'] for edit_data_ in test_data]
    portability_inputs = {
        'one_hop':{
            'prompt': portability_prompts,
            'ground_truth': portability_ans
        },
    }

    portability_inputs = None
    subject = [edit_data_['subject'] for edit_data_ in test_data]
    loc_prompts = [edit_data_['loc'] + ' ' + edit_data_['loc_ans'] for edit_data_ in loc_data]

    return prompts, rephrase_prompts, target_new, subject, locality_inputs, portability_inputs, loc_prompts

def main(args):

    if args.editing_method == 'FT':
        editing_hparams = FTHyperParams
    elif args.editing_method == 'IKE':
        editing_hparams = IKEHyperParams
    elif args.editing_method == 'KN':
        editing_hparams = KNHyperParams
    elif args.editing_method == 'MEMIT':
        editing_hparams = MEMITHyperParams
    elif args.editing_method == 'ROME':
        editing_hparams = ROMEHyperParams
    elif args.editing_method == 'LoRA':
        editing_hparams = LoRAHyperParams
    elif args.editing_method == 'WISE':
        editing_hparams = WISEHyperParams
    elif args.editing_method == "GRACE":
        editing_hparams = GraceHyperParams
    elif args.editing_method == "ReFT":
        editing_hparams = ReFTHyperParams
    elif args.editing_method in ["MeFT", "SMeFT"]:
        # SMeFT shares the setting from MeFT
        editing_hparams = MeFTHyperParams
        args.hparams_dir = args.hparams_dir.replace("SMeFT", "MeFT")
    else:
        raise NotImplementedError

    hparams = editing_hparams.from_hparams(args.hparams_dir)
    hparams = editing_hparams.from_hparams(args.hparams_dir)
    if args.editing_method in ["MeFT", "SMeFT"]:
        hparams.alg_name = args.editing_method
    if args.editing_method == "SMeFT":
        hparams.smeft = args.smeft

    if "llama" in hparams.model_name.lower():
        hparams.model_name = "meta-llama/Llama-2-7b-hf"

    (
        prompts, rephrase_prompts, target_new, subject, locality_inputs, portability_inputs, loc_prompts
        ) = load_data(args)
    args.ds_size = len(prompts)

    output_folder = f"~/dev/edit/outputs/zsre_new"
    output_folder = os.path.expanduser(output_folder)
    os.makedirs(output_folder, exist_ok=True)

    edit_method_name = args.editing_method
    if args.editing_method == "SMeFT":
        edit_method_name += f"({args.smeft.replace('-', '')})"
    output_file = os.path.join(
        output_folder,
        f'{hparams.model_name.split("/")[-1]}_{edit_method_name}_N={args.ds_size}B={args.bs_size}S={args.sequential_edit}R={"T" if args.reset_seq else "F"}.json'
        )

    print("See results at: ", output_file)

    if os.path.exists(output_file) is False or args.retrain:
        # Prepare pre-edit stat
        args.pre_file = os.path.join(output_folder, f"./{hparams.model_name.split('/')[-1]}_PRE_N={args.ds_size}.json")
        if args.pre_file is not None and os.path.exists(args.pre_file):
            pre_edit = json.load(open(args.pre_file,'r'))
            assert len(pre_edit) == len(prompts)
        else:
            pre_edit = None

        if args.editing_method == 'IKE':
            train_data_path = os.path.join(args.data_dir, 'zsre_mend_train_10000.json')
            train_ds = ZsreDataset(train_data_path)
            sentence_model = SentenceTransformer(hparams.sentence_model_name).to(f'cuda:{hparams.device}')
            encode_ike_facts(sentence_model, train_ds, hparams)
        else:
            train_ds = None

        editor = BaseEditor.from_hparams(hparams)

        metrics, edited_model, _ = editor.edit(
            prompts=prompts,
            rephrase_prompts=rephrase_prompts,
            target_new=target_new,
            loc_prompts=loc_prompts,
            subject=subject,
            train_ds=train_ds,
            locality_inputs=locality_inputs,
            portability_inputs=portability_inputs,
            keep_original_weight=True,
            pre_file=args.pre_file,
            pre_edit=pre_edit,
            sequential_edit=(args.sequential_edit > 1),
            sequential_edit_size=args.sequential_edit,
            verbose=False,
            reset_seq=args.reset_seq,
        )

        with open(output_file, 'w') as f:
            json.dump(metrics, f, indent=4)

        # json.dump(metrics, open(os.path.join(args.metrics_save_dir, f'{args.editing_method}_results.json'), 'w'), indent=4)

    with open(output_file, "r") as f:
        metrics = json.load(f)

    if len(metrics) > 0:
        summary_metrics(metrics)

I got following results when using T=1 (no sequence edit).
WISE:
Metrics Summary:  {'pre': {'rewrite_acc': 0.011500000000000002, 'rephrase_acc': 0.004, 'portability': {'one_hop_acc': 0.012619047619047618}}, 'post': {'rewrite_acc': 0.9975, 'rephrase_acc': 0.976, 'locality': {'neighborhood_acc': 0.24815370534178582}}}
GRACE:
Metrics Summary:  {'pre': {'rewrite_acc': 0.011500000000000002, 'rephrase_acc': 0.004, 'portability': {'one_hop_acc': 0.012619047619047618}}, 'post': {'rewrite_acc': 0.9933333333333333, 'rephrase_acc': 0.0025, 'locality': {'neighborhood_acc': 0.4452706521204973}}}
MEMIT:
Metrics Summary:  {'pre': {'rewrite_acc': 0.011500000000000002, 'rephrase_acc': 0.004, 'portability': {'one_hop_acc': 0.012619047619047618}}, 'post': {'rewrite_acc': 0.9398736263736265, 'rephrase_acc': 0.914349816849817, 'locality': {'neighborhood_acc': 0.16812631660851476}}}

Hi,
It motivates me to put things together. Numerous past experiments have demonstrated that individual edits like ROME, MEMIT, and GRACE have good locality.
However, I couldn't understand what was happening in your code. But please don't worry, I have improved the README for the rerun experiments and provided the scripts(https://github.com/zjunlp/EasyEdit/blob/main/examples/run_wise_editing.py, https://github.com/zjunlp/EasyEdit/blob/main/examples/run_wise_editing.sh). You can follow my code to conduct the experiments. I have just tested it, and it can successfully run and reproduce the experimental results on the latest version of EasyEdit.
Best regards.

hi, have you solved your issue yet?

Thanks for providing the new instruction!
I noted that the main difference was from the eval_metric: the default value was exact match instead of token em, but I haven't got time to dive into the details here. Can you share some idea about this setting?
BTW, I am now able to achieve good locality from WISE and MEMIT, but for GRACE the locality was still lower than the results reported in the paper: I followed your setup and only tested with 3 samples. Both WISE and MEMIT achieved 100% locality. But GRACE result was only round 70%. Could you help confirm you could achieve 100% as well?
Thanks again for your help!

I will reply to you at weekday

Hi,
Q1: Both token em and exact match are fine, and this is not important because the default setting calculates token-level accuracy, which is exact match (em). For the hallucination task, previous literature uses perplexity (ppl), so we follow this setting as well. Therefore, you don't need to worry about the first question and can proceed with the script I provided.
Q2: I experiment with GRACE and achieve 100% locality.
{"post": {"rewrite_acc": 1.0, "rephrase_acc": 0.0, "locality": {"neighborhood_acc": 1.0}}}

The 70% locality you mentioned might be because you use the same pre_file to evaluate locality. However, GRACE should have an independent pre_file. For more details, you can refer to this issue comment. You can try testing GRACE's performance without providing a pre_file (or new pre_file).
I understand you might be confused about the operation for Q2, but different editing methods have different perspectives, leading to different utilization methods. Therefore, I have to handle this special case.
Hope you can be tolerant.
Best,

Got it. Thanks a lot for the information!


Dumb AP?

Would it be possible to add an option to just bridge WLAN0 to LAN0, without any DHCP or routing, so the device becomes a "dumb access point", please?

Hi @eduperez - I'm looking into this.

Yea this would be amazing
@mattlongman should be able to disable dhcpd, create a bridge device and add it to both interfaces? eth0 and wlan0 I am guessing for 99% of people
https://www.raspberrypi.com/documentation/computers/configuration.html#bridging
technically be possible with 2 wifi dongles as well if you want the pi to be an extender I guess (would save that for later)

I also looking for way to do a bridged dumb AP, since its done, AP could be a part of wifi coverage withing home (if same SSID and security) with normal roaming between other APs. Now it has to change IPs and data transmitting stops for a much more time..

A wifi/ethernet bridge would solve my problems too. Can you share progress? Happy to collaborate.
I can get a pi to bridge but still on steep learning curve with Docker...
Thanks @mattlongman for a great addon

@HLFCode agreed, the docker nightmare that HA imposes made me just rewrite my own home automation with uglier UI

My PR https://github.com/home-assistant/operating-system/pull/2304 got merged.
With HAOS 10 it will be possible to create a hotspot and bridge it with the ethernet interface directly within NetworkManager.

Fantastic @xonestonex .
Do I have to upgrade from HAOS 9.4 to 10 before I can use it?

Can someone explain how to configure it as "Dumb AP"? @xonestonex @eduperez maybe you have an example how to make the bridge?

Yes please

+1 - im looking to bridge wlan0 and end0, i cant find details of how to do this on HASSOS 13.2.
any suggestions ?


doesn't honour the "Connection: Close" HTTP client header

Version: 1.3.9
CUPS.org User: martin.pitt.canonical
From http://bugs.debian.org/504727:
Below is a wireshark capture of a client (nagios) doing an HTTP
request to cups. The client uses HTTP 1.1 and includes the
"Connection: close" header meaning that he wants that request to
be the last one. cups replies with a "Connection: keep-alive"
header and doesn't close its end of the connection, which is in
violation of the HTTP 1.1 RFC.
In the case of nagios, it has the effect of nagios timing out
after 10 seconds waiting for the connection to close, and thus
declares the cups service dead.
This is on the loopback interface, so no proxy comes into play.
Request from nagios:
[...]
Frame 4 (180 bytes on wire, 180 bytes captured)
Arrival Time: Nov  6, 2008 16:25:37.127285000
[...]
Hypertext Transfer Protocol
GET / HTTP/1.1\r\n
Request Method: GET
Request URI: /
Request Version: HTTP/1.1
Host: <IP_ADDRESS>:631\r\n
User-Agent: check_http/v1991 (nagios-plugins 1.4.12)\r\n
Connection: close\r\n
\r\n
[...]
answer from cups:
Frame 6 (323 bytes on wire, 323 bytes captured)
Arrival Time: Nov  6, 2008 16:25:37.127445000
[Time delta from previous captured frame: 0.000151000 seconds]
[Time delta from previous displayed frame: 0.000151000 seconds]
[Time since reference or first frame: 0.000405000 seconds]
[...]
Hypertext Transfer Protocol
HTTP/1.1 200 OK\r\n
Request Version: HTTP/1.1
Response Code: 200
Date: Thu, 06 Nov 2008 16:25:37 GMT\r\n
Server: CUPS/1.3\r\n
Connection: Keep-Alive\r\n
Keep-Alive: timeout=60\r\n
Content-Language: en_GB\r\n
Content-Type: text/html; charset=utf-8\r\n
Last-Modified: Sat, 11 Oct 2008 10:58:14 GMT\r\n
Content-Length: 5242
\r\n
[...]
After 10 seconds, nagios times out.
Frame 10 (68 bytes on wire, 68 bytes captured)
Arrival Time: Nov  6, 2008 16:25:47.127182000
[Time delta from previous captured frame: 9.999571000 seconds]
[Time delta from previous displayed frame: 9.999571000 seconds]
[Time since reference or first frame: 10.000142000 seconds]
[...]
Transmission Control Protocol, Src Port: 41750 (41750), Dst Port: ipp (631), Seq: 113, Ack: 5498, Len: 0
Source port: 41750 (41750)
Destination port: ipp (631)
Sequence number: 113    (relative sequence number)
Acknowledgement number: 5498    (relative ack number)
Header length: 32 bytes
Flags: 0x11 (FIN, ACK)
[...]
nagios log:
CURRENT SERVICE STATE: localhost;CUPS;CRITICAL;HARD;4;CRITICAL -
Socket timeout after 10 seconds
The "CUPS" nagios service does a:
/usr/lib/nagios/plugins/check_http -H localhost -p 631
Which does:
socket(PF_INET, SOCK_STREAM, IPPROTO_TCP) = 3
connect(3, {sa_family=AF_INET, sin_port=htons(631),
sin_addr=inet_addr("<IP_ADDRESS>")}, 16) = 0
send(3, "GET / HTTP/1.1\r\nHost: localhost:6"..., 112, 0) = 112
read(3, "HTTP/1.1 200 OK\r\nDate: Thu, 06 No"..., 8191) = 5497
read(3,
That second read is waiting for end of file that never comes
(well would come after cups keep-alive timeout: 1 minute) and
so times out.

@michaelrsweet
many thanks for making the embedded diff in https://github.com/apple/cups/issues/3010#issuecomment-197671298 readable!


Update Textures

textures.png has been updated with new textures and the README file has been modified to reflect the attribution required from the new textures.

Coverage remained the same at 5.601% when pulling b2db12df10207a62d45e502021f4f7776ff29003 on JLPrescott:master into 143c29b82519b33bda7bf3e09c80d35fea55aac1 on traverseda:master.

I like it. Gives us some visual distinction from minecraft.


⚠ AGEG website (prod) has degraded performance

In e3faf68, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 3118 ms

Resolved: AGEG website (prod) performance has improved in d60378a.


Serialize Measures and Roles as their own TMDL files

As a Tabular Editor user, I would like to have the option to serialize Measures and Roles as their own TMDL files (much like the good old json serialization), instead of having them embedded in the table files. This would make it easier for me to track changes, compare versions, and collaborate with other developers.
Given the comment on the last release of tabular editor 2 is this possible?

Hi @jobosol
It is currently not possible to do this using TMDL, but it is an option that would make sense.
As the release notes states it is something that is being actively listened on feedback for so it is great that you have open this to track interest


Add SBCONFIG_PORT to documentation

According to the docs, the following is the default method to run the storybook:
start-storybook -p 5700

However, I stumbled upon this undocumented SBCONFIG_PORT environment variable in the code while looking for a way to run the storybook remotely on a specific port defined as an ENV_VAR.
It turns out I can start the server this way, without specifying the port with -p:
export SBCONFIG_PORT=5700
start-storybook

Is it recommended to use SBCONFIG_PORT ? If so, it could be useful to find a mention of this in the docs.

As a side note, I edited my init script to use this environment variable only if it's set, otherwise, fallback to the default hardcoded. The reason is to be able to set a different PORT on a remote env without having to EXPORT one locally.
start-storybook -p ${SBCONFIG_PORT:-5700}

SBCONFIG_PORT is something we did for some internal tooling.
It's never meant for public use.
We may remove it in the future.


Symlink command to be brew-cask external command

While I was poking around the homebrew core, I discovered that brew-cask supports external commands in the form brewcask-{command}, as shown in the command-line interface source.
This change takes advantage of that, and makes it easy to call this command using brew cask upgrade, as opposed to the current brew cu. And, built-in commands take precedence, meaning that, if brew-cask ever adds an upgrade command, this will fall back to the expected behavior, but preserve calling with brew cu.

Well, I guess I didn't test this thoroughly enough, as brew-cask doesn't like when brew-cask-upgrade waits for user input, and fails saying the command wasn't found. I have a potential solution, but I can't test it until I have another cask that needs upgrading, so I'll close this and re-open another PR when I can test it more.
I also have no clue if a change like this is even something you want. Personally, I vastly prefer being able to use brew cask upgrade, as it fits in better with the existing hombrew style, but I can also just maintain this for my own use, too.

Only brew cask upgrade --yes works well with this PR as you mentioned. If you have a solution to  the problem, you can easily test it by just changing a directory name. For example,
cd /usr/local/Caskroom/atom
mv 1.15.0 1.14.0

Feel free to open a new PR if it works well. Thank you.

Awesome, that will help immensely in testing what I have in mind! I'll open a new PR once I can test out my changes.


[BUG][opensearch] CrashLoop when keystore secret has data but script can't create keystore

Describe the bug
When the key store secret is configured and has entries in it, the shell script imports it incorrectly.
To Reproduce
Steps to reproduce the behavior:

Create a Kubernetes Keystore Secret:

apiVersion: v1
kind: Secret
metadata:
  name: keystore-secrets
type: Opaque
data:
  s3.client.default.access_key: foo
  s3.client.default.secret_key: bar

Configure a keystore secret in value.yml

keystore:
  - secretName: keystore-secrets

Deploy the chart and observe CrashLoop from the keystore init container logs:

Created opensearch keystore in /usr/share/opensearch/config/opensearch.keystore
sh: line 6: file: unbound variable

Expected behavior
No CrashLoop
Chart Name
Opensearch
Screenshots
N/A
Host/Environment (please complete the following information):

Helm Version: [3.12.1]
Kubernetes Version: [1.22.15]
OpenSearch Helm Chart version [2.13.2]

Additional context
From my side any updates of value.yml files only pull of new version of helm chart
https://github.com/opensearch-project/helm-charts/commit/4e4f82672f79704b38ae9063ea6c67655dd7a101
Temporary reverted my cluster configuration to OpenSearch Helm Chart version [2.13.1]

Closing this as the fix has been pushed through the PR https://github.com/opensearch-project/helm-charts/pull/445, thanks to @hobti01 and @uncycler.


thx

Thanks for your contribution for OC,i have used STHTTPRequest for a long time,it's a so lightly framework that i can't help saying thx to you,thank you!!!

Thank you for writing.
I just pushed version 1.1.0 to CocoaPods.
This version is now based on NSURLSession instead of NSURLConnection.
Let me know if you meet any issue.


Utility script to convert Perl to JSON config

The old CLI version saved the configuration as a Perl dictionary. The current version can only handle a JSON configuration.
We need a utility script that can be used to convert any Perl configuration to JSON. This JSON config can then be imported in the current version of XKPasswd.

I have a question. What was the form of the Perl dictionary created by the old CLI version? I never got it to run and we don't have access anymore. Not sure how any of us could help without knowing more. Or maybe that's why you assigned it to @bbusschots.

Yes, @bbusschots will write a utility to convert the Perl format to JSON for further processing in the new version.


Technology Broker Icon

I needed to generate an icon for technology brokers to use in my app. This is the icon that shows up in the station services menus to indicate a tech broker.
It may not be 100% pixel accurate to the in game UI, but I think I got it very close. The image is 350px by 350px, but it that is too small I can scale it up.

Looks good, not seen the in game one but I'll trust you 🙂
You don't happen to have a vector as well do you?

I don't unfortunately, just whipped this PNG up in gimp. I actually use a raw SVG path generated from this image in my app, but the output I use doesn't display correctly in a browser so i decided not to upload it. If I get it in the proper format, i'll open another PR with it.

Figured out the SVG thing so added a vector version as well. Cleaned up the source image so re-exported the PNG too.
And for reference (in case you're curious) this reddit thread has a screenshot that has the icon in it, albeit a little blurry: https://www.reddit.com/r/EliteDangerous/comments/7rc8zk/tech_broker/
The icon is just to the right of the tech broker NPC portrait.

The SVG has a lot of whitespace around the vector, feel free to fix that up in a following PR. Merging this :)


[Style] Make the docstrings for Actor conform to project guidelines

This patch contains no changes in the behavior of the Actor class.
All it does is modify the doctstrings to conform with the project
guidelines.
Signed-off-by: Eric James Michael Ritz<EMAIL_ADDRESS>GitHub-Issue: #35
GitHub-Issue: #44
See-Also: https://github.com/lillian-lemmer/hypatia/wiki/Project-Guidelines

Merged to develop after I made some edits (herself>themselves), and added class links.


The link of data is dead

Hi,
Thanks for sharing the dataset!
I find that the link in file data/get_data.bash  is dead link.
Can you recheck the link or update a new link?
Thanks!

Hi, thank you for your interest and having noticed that problem. The server was down, but it should be fixed now.


Fix for #6633 (IDFGH-4835)

Moving #includes above #ifdef __cplusplus extern "C" { #endif
So that we can compile with CPP.
This is a fix for https://github.com/espressif/esp-idf/issues/6633
NB this PR fixes in the v4.3 branch. I presume there are other branches (like master) to which this should also be applied.

Thanks for your contribution.

Changes merged with https://github.com/espressif/esp-idf/commit/13bc5feb0a787a10a09a63184d7273292fd4d9c7


Include tests in git repository archives

The Debian/Ubuntu packages use the tests to validate the built packages. In order to push forward with the PHP 7.2 release in Ubuntu 18.04, we're going to remove the tests from the php-guzzlehttp-promises package. I'd like that change to be temporary, though.
If this PR is merged, the tests could be added back to the packaging process to produce better, more trustworthy packages.

Tests are ignored from archives on purpose, when using Composer dist packages they produce smaller builds without test and other unnecessary files (fixtures, etc). This is an accepted practice in the PHP world and I would prefer keeping things that way.
Is there any chance you can clone the git repository instead? This is a relatively small package, so I wouldn't expect too much overhead.

@sagikazarmark I tried to switch it to using a clone, but ran into a roadblock: Debian's packaging tools have the ability to create the upstream 'orig' tarballs from git clones but it uses git archive, which will use .gitattributes to generate the archive: https://anonscm.debian.org/cgit/collab-maint/devscripts.git/tree/scripts/uscan.pl#n4455

@sagikazarmark I should have closed this earlier. I'll try to work with Debian's devscripts team to update uscan to be able to handle this.

Great, good luck.


Bed file for Felis_Catus9.0

I have added the a notebook to extract bed file  for Felis_catus_9.0 (GCA_000181335.4).  The gzipped bed files can be found in data folder . The GTF file was downloaded from Ensembl.org.

Looks great, thanks for your contribution!


$DOCKERD_OPTIONS

This introduces the ability to pass arbitrary command line options to the in-container dockerd via the container's environment ($DOCKERD_OPTIONS).

Duplicate of #62 (also #12 and #20).


Add modify excel activity to catalogue

As an RPA Developer, I'd like to modify my excel spreadsheets using an RPA-activity

:thought_balloon: Hints & Thoughts
:question: Additional Questions to Answer / Consider

:white_check_mark: Conditions of satisfaction

[x] Excel can be selected from apps
[x] Write value can be selected from tasks
[ ] Hardcoded parameters are being stored in the background
[ ] XML updates with parameters

Hi @SanJSp,
what is missing here to move this issue to the DONE column?

Hi @SanJSp,
what is missing here to move this issue to the DONE column?

Was simply an accident. It can be moved into done.

Was simply an accident. It can be moved into done.


findmin and findmax do not work as expected 🐛

Contact Details
No response
What happened?
Here's a demo of the bug using SimpleSDMLayers v0.8.3 on Julia 1.8
bbox = (left=-83.0, bottom=46.4, right=-55.2, top=63.7);
temp = convert(Float32, SimpleSDMPredictor(WorldClim, BioClim, 7; bbox...))

tempmin, tempmax = extrema(temp)
> (1.0f0, 51.8575f0)

findmax(temp)
> ([-55.25, 52.083333333333336] => 28.3f0, [-55.25, 52.083333333333336])

findmin(temp)
> ([-82.91666666666667, 46.416666666666664] => 40.6895f0, [-82.91666666666667, 46.416666666666664])

So clearly something is off here, as the value returns by findmin is greater than that returned by findmax, and neither align with the extrema call
Stacktrace
No response

Yeah I tried with a simple layer and it Works on My Machine ™ , but I'll build an actual test and report back.

This has probably an easy fix, I'll work on this today.

See #146 which adds findmin and findmax + findall, findfirst, findlast


Adding Spanish translation

I just added a file with the Spanish version of the form messages.
Thanks for the module and regards!

Thanks @algm ;-)


creating option.json in mapping directory is not working

Issue guidelines
If you're asking a question, rather than reporting a bug or requesting a feature, please post your question on the mailing list, and do not open an issue.
Please do not log bugs regarding classpath issues (typically manifesting as 'NoClassDefFoundException' or 'ClassNotFoundException').
These are not WireMock bugs, and need to be diagnosed for your project using your build tool. Plenty has already been written in WireMock's issues and mailing list about how to resolve these issues
so please search these sources before asking for help.
Bug reports
We're a lot more likely to look at and fix bugs that are clearly described and reproduceable. Please including the following details when reporting a bug:

[ ] Which version of WireMock you're using
[ ] How you're starting and configuring WireMock, including configuration or CLI the command line
[ ] A failing test case that demonstrates the problem
[ ] Profiler data if the issue is performance related

Feature requests
Please include details of the use case and motivation for a feature when suggesting it.

As suggested, I have created option.json in mapping directory with below request & response, but still i am not seeing option call while mocking api calls.
{
"priority":10,
"request": {
"method": "OPTIONS",
"urlPattern": "."
},
"response": {
"status": 200,
"headers": {
"Content-Type": "application/json",
"Access-Control-Allow-Origin" : "",
"Access-Control-Allow-Methods" : "*",
"Access-Control-Allow-Headers": "Accept, Content-Type, Content-Encoding, Server, Transfer-Encoding, X-Requested-With, X-Authorization",
"X-Content-Type-Options" : "nosniff",
"x-frame-options" : "DENY",
"x-xss-protection" : "1; mode=block"
}
}
}

Per the guidelines, please post questions to the mailing list.


Nuxtjs: OrbitControls.js encountered an unknown camera type

Describe the bug
I added OrbitControls, and when scrolling I get the following error
WARNING: OrbitControls.js encountered an unknown camera type - dolly/zoom disabled.

Make it impossible to scroll to zoom in/out.

Reproduction
https://stackblitz.com/edit/nuxt-starter-3nnslu?file=package.json
Steps to reproduce
No response
System Info
See https://stackblitz.com/edit/nuxt-starter-3nnslu?file=package.json

Used Package Manager
npm
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hi there, please open the ticket in the correspondent repository https://github.com/Tresjs/nuxt


Remove Ubuntu 16.04 builds ~ April 30th 2021

Ubuntu 16.04 goes EOL April 30th 2021. We should remove this from our builds around that date so we can start building on Ubuntu 18.04 and reduce our overall pipeline costs.
https://github.com/chef/chef-server/blob/master/.expeditor/release.omnibus.yml#L15
Definition of done

[ ] Confirmed that no test scenarios rely on 16.04
[ ] Remove from pipeline config
[ ] Adhoc build passes
[ ] PR to update Docs with new requirements

https://github.com/chef/chef-server/commit/919818b22d251a94366acfa55f45a8324c9d50eb - 14.3.0 onwards.


Add RoomCard component

This PR adds the RoomCard component as part of #698.

Why isn't this deploying? @benawad


[3.4] When limit smoothing and smoothing are enabled simultaneously and the Camera2D is outside of the limits, there is a one frame delay before the smoothing starts.

Godot version
3.4.stable
System information
Ubuntu 20.04.3 LTS
Issue description
When Camera2D has limit smoothing enabled, smoothing enabled, and is outside of the limits, it starts sliding to the new limits one frame late.
This behavior did not exist in 3.3, since the camera would always immediately snap into the limits in that version, which was changed in #49409
It also does not occur for smoothing in general, even if Camera2D has the same properties. If position is changed and then reset_smoothing is called in the same function, the camera will immediately slide there. The behavior outside of limits is inconsistent in this regard.
Steps to reproduce

Create a Camera2D and set its limits to small numbers
Enable Limit Smoothing and Smoothing
Move the Camera2D outside of the limits.
Run the project

Since it's just a one frame delay, to make the bug visible would take a bit more, in my minimal reproduction project I did the following:

Create a script
Call reset_smoothing on the Camera2D
Pause the tree in the same function to see the delay.

Minimal reproduction project
Uploading gave an error for some reason. This is a link to the zip, hope it works: https://workupload.com/file/6Ewv34q59SB

This persists in 3.5.rc1, but isn't in 4.0.alpha8

Related to / possible duplicate of #56336.


swift: include missing library files

These aren't being compiled in right now.
Signed-off-by: Michael Rebello<EMAIL_ADDRESS>

I believe @goaway and @buildbreaker had a discussion about not globbing - personally I'm in favor of it 🤷♂


Update the Sign operation in ops.py

Fix the bug when Sign followed by True/False value.

Description

During the alpha generation exps, I found that when the Sign operation meets True/False value, it will raise a MaybeEncodingError. Here I transform the True/False value to np.float32.
Motivation and Context

How Has This Been Tested?

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[x ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[x ] Fix bugs
[ ] Add new feature
[ ] Update documentation

Hi,
Many thanks for sending the pull request.
But I can't understand why the data type conversion is required. Can you give us more details? Thanks

I mean when the sign function meets bool value, it will raise error. Here is the expression I test yesterday:
Sign(Gt($close, Sum(WMA($volume, 14),17)))
The $close value will be smaller than Sum(WMA($volume, 14),17)). So it is Sign(series([False, False, .......])) and it will raise error.
Though this expression is meaningless, but I think we should make it work?

@YifanDengWHU  Thanks. I see now.
Please add some comments to describe this case.
In addition, please add this comments  above the np.float32
# TODO:  The the precision should be configurable

Thanks for your contribution!


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Added Styles and a style sheet to the index.html


Extension Not available for Chrome

This extension is remove from chrome store .please check for necessary action

I know. You can try to upload it to Chrome yourself or submit a pull request? I stopped maintaining this software.


reactions.js: Show full name in reactions if only one reaction.

This shows the full name of the user who reacted if and only if
that reaction is the only reaction on the message and only one
person had reacted to that message.
Fixes #4513.

#4513

@zulipbot claim

@lonerz, This seems like a lot of new code. I'm wondering if you, @timabbott , and I aren't on the same page in terms of simplifying rendering here.  My thought is that we can simplify this code a lot by not doing any incremental rendering in add_reaction.  Instead, just always rebuild the line of emojis from scratch whenever you change the dataset.  Basically, you would update the data, and then call something like render_whole_row_of_emojis.
Either I misunderstood Tim's suggestion, thinking he was saying the same thing that I was thinking, or I am misunderstanding your code.

We should also be thinking about writing automated tests here.  Writing automated tests can help drive you toward a cleaner architecture, not to mention it can find bugs with edge cases.
Our architecture here should basically be a two-layer architecture.
The first layer should know how to create the data structures:
# first reaction
[(:smile:, 'alice', true)] # use name
[(:smile:, 2, false)], # add reaction, now have count
[(:smile:, 'alice', false)], # remove reaction
[(:smile:, 1, false), (:frown:, 1, false)] # add reaction
# etc.

If we can get the data piece super simple, then the second layer is just rendering.

Opened #5602. Closing this one.


Refactor LoadManager to Scheduler

What changes are proposed in this pull request?
Refactoring LoadManager to Scheduler.
Redefine the job progress metrics since total file count is hard to be accurate when we have retry and verification
Why are the changes needed?
for other jobs
Does this PR introduce any user facing changes?
updated API in FileSystem
  /**
   * Submit a job.
   *
   * @param jobRequest the job request
   * @return job id if job is submitted, empty if a load of the same path already exists
   */
  Optional<String> submitJob(JobRequest jobRequest);

  /**
   * Stop a job.
   *
   * @param jobDescription the job description
   * @return true if job is stopped, false if cannot find job
   */
  boolean stopJob(String jobDescription);

  /**
   * Get progress of a job.
   *
   * @param jobDescription   the job description
   * @param format  progress report format
   * @param verbose whether to return verbose report
   * @return the job progress
   */
  String getLoadProgress(String jobDescription,
      JobProgressReportFormat format, boolean verbose);

Tested locally with transferring leadership, we can still get the job information and run the unfinished jobs.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to dora please

alluxio-bot, cherry-pick this to dora please


What is the best way to implement a use case that doesn't have any params?

Do I need to create a separate base use case that doesn't expect params to be passed, or just pass Void as param? what would be your way to handle that ?

You could have use cases without the parent class(that doesn't have the param), like this:
class MyUseCase @Inject constructor(
    private val myRepo: MyRepo,
) {

    operator fun invoke(): Flowable<MyModel> =
        myRepo.getAllOfThem()

}

But in point of my view,

To have a concise code
Following the same pattern you follow for the other use cases that accept params

It is better to have a parent class for these kinds of use cases as well:
interface ParentUseCase<Type : Any> {

    operator fun invoke(params: Params): Single<Type>

}

and the usage:
class MyUseCase @Inject constructor(
    private val myRepo: MyRepo,
) : ParentUseCase<Flowable<MyModel> {

    override fun invoke(): Flowable<MyModel> =
        myRepo.getAllOfThem()

}

You could pass Nothing as well to avoid making the code complex and having too much abstraction in your code. If you have lots of abstraction classes, go this way.
A good code is

Not a few abstractions
Not too many abstractions
Reuse what you have as much as you can until you realized you're mixing different logic (if-else)


Alignment confidence indication

I am trying to align 2D LIDAR scan frames to frames in a database to get a position estimate. I am using image matching to suggest candidates for this, but there is a chance that some suggestions might be complete garbage or that the environment will have changed dramatically from the LIDAR's perspective to the point where alignment will not work anymore, so I want to find a way to filter out point cloud alignments that make no sense at all. Unfortunately it seems that the icp compute function does not return any sort of metrics that I can attempt to use for this purpose. As far as I can determine the function only returns the final transformation matrix estimate, but I would like to find out if there is some way that I can get access to for instance the final error value or the amount of inliers or any other useful metrics of the alignment quality?

What you are looking for is a covariance for the outputted transformation. Unfortunately, there is no functional way to compute it (you can search for covariance in the issues of this library to see the different discussion on that topic.
To get closer to what you want, you can have a look at residual errors after the minimization. From the documentation:

How to compute the residual error with ErrorMinimizer::getResidualError(...) See the example code provided here.

@pomerlef thanks I think the residual errors are indeed what I wanted. The example code is a bit strange though in that it seems that one has to force some calculations that I can only assume would already have been done as part of the optimisation process already so I assume this is not the most efficient approach possible but it does seem to work.

@pomerlef Something I have observed in the past is that the residual error value seems to be tied to the relative number of points in the reference and sensor point cloud. Is there a sound way to normalize this quantity?

By looking at what is implemented over here, it seems that the function returns the sum of the residuals. I don't recall the motivation behind that choice.
You would need to divide by the number of points to get the average residual error.


Regex with "+" sign not working correctly?

I write grammar with regex, sometimes getting trouble with "+" sign in regex.
WORKS:
from lark import Lark
grammar = """
start: /./+ "c"
"""
print(Lark(grammar).parse("abc"))

DOESN'T WORK:
from lark import Lark
grammar = """
start: /.+/ "c"
"""
print(Lark(grammar).parse("abc"))

with error
lark.exceptions.UnexpectedEOF: Unexpected end-of-input. Expected one of: * C

Sometimes /.+/ doesn't work while /./+ works. Is it violating any syntax or regulation?

The regex /./+ matches "abc" as one Token. Lark can not break that up. /./ doesn't, so Lark can deal with it.

I see. thanks!


Trouble with setting AD user as admin when group member

v4 build 6097
CreateAdminIfGroupMember does not make the user admin for an AD user assigned to a group with matching group name.  This appears to be related to _xcreds_oidc_groups vs _xcreds_groups in Directory Utility.
No problem when setting Azure user as admin when group member.

Log from example

There are two school districts that I am working with that are still using NoMad and NoMad Login. The AD group for macOS admin assignments is a big must. Still using onprem AD but "slowly" moving to AZURE.

tested and worked fine. see attached profile for sample.
xcreds-AD Only_admin_group.mobileconfig.zip


Fix deprecated all to moment

Very simple change, fixing a deprecated call to moment().subtract

I tested it.
The problem is not fixes with this change

What "problem"? @woehrer12

2017-12-18 22:45:48 (DEBUG):	Processing 2597 new trades. From 2017-12-18 21:07:08 UTC to 2017-12-18 21:45:45 UTC. (39 minutes)
Deprecation warning: moment().add(period, number) is deprecated. Please use moment().add(number, period). See http://momentjs.com/guides/#/warnings/add-inverted-param/ for more info.
events.js:160
throw er; // Unhandled 'error' event
^
Error: SQLITE_MISUSE: Database is closed
at Error (native)
at Database.

@woehrer12 Your warning is about a misused .add, this one fixes a misused .subtract. If you can find out where it is being incorrectly used please open a new issue and it will be fixed.


GUI: synchronisation screen - more info.

The current sync screen is this:

Now that Core can me managed by Liana, it requires more information.
Disclaimer (when sync is over 98%/about a month):
Bitcoin Core is synchronising the blockchain. This may take a few minutes, depending the last time it was done, your internet connection, and your computer performance.
Disclaimer (when sync is between 90%-98% / up to a year ago):
Bitcoin Core is synchronising the blockchain. This will take a while depending the last time it was done, your internet connection, and your computer performance.
Disclaimer (when sync is under 90%):
Bitcoin Core is synchronising the blockchain. A full synchronisation typically take a few days, and is resource intensive. Once the initial synchronisation is done, the next ones will be much faster.
We should also display the progress number, with enough digits to make is look like it's not stalled. Probably at least XX.XX%

what about display <block_validation_height> / <headers_height> ?

what about display <block_validation_height> / <headers_height> ?

It would have to be passed through from lianad and i think we shouldn't add anything Bitcoin Core specific in the JSONRPC interface.

where we get the sync % from?

The daemoN.
------- Original Message -------
On Tuesday, August 29th, 2023 at 7:11 PM, pythcoiner @.***> wrote:

where we get the sync % from?
—
Reply to this email directly, view it on GitHub, or unsubscribe.

The Liana software is in two parts: the daemon (lianad) that implements the core logic, and the GUI (liana-gui) that acts as a sophisticated wrapper around it.

OK, that was what I got, just was confused bc you mentioned it as lianad and then as  daemon.

i notice one thing today, when we get a new peer, a message about the peer overwrite the synch info, it ould be nice do display it separately or as a log.

To whomever gets to tackle this. I think this issue is now simply about adding some context messages (quoting OP):

Disclaimer (when sync is over 98%/about a month):
Bitcoin Core is synchronising the blockchain. This may take a few minutes, depending the last time it was done, your internet connection, and your computer performance.
Disclaimer (when sync is between 90%-98% / up to a year ago):
Bitcoin Core is synchronising the blockchain. This will take a while depending the last time it was done, your internet connection, and your computer performance.
Disclaimer (when sync is under 90%):
Bitcoin Core is synchronising the blockchain. A full synchronisation typically take a few days, and is resource intensive. Once the initial synchronisation is done, the next ones will be much faster.


Partially fix z -x command

The command z -x still does not work, despite the change in #125.
Previously, the command sed -i -e was used (incorrectly) and made a backup file named ~/.ze. Using sed -i '' prevents this backup file from being created (on OS X).
Previously, after running z -x, the datafile would be printed as if z failed to find results for a query. But, because z -x would never be used with arguments (i.e., a query), it is best to return like the z -h command.
Despite my attempt at a fix, z -x still does not work because of precmd, which adds the current directory to the datafile after every z run regardless of the arguments. z -x correctly modifies the datafile, but after, z adds the current directory right back (with a reset score).

Can this be closed? This PR says that the branch is up-to-date with the base branch.

@bean5 No. That message does not mean the PR has been merged; rather, it means there have not been any changes to the base branch that conflict with this PR.

I see. So no rebase or merge necessary. Good to know. Thanks.

@bean5 Not quite. This PR's changes have not been integrated and still need to be merged to take effect. The message means there will be no merge conflicts when the PR is merged. There will be changes, just not merge conflicts.

*no rebase or merge necessary into the request branch.

For me it was the Bash PROMPT_COMMAND that was breaking this.  Setting _Z_NO_PROMPT_COMMAND fixes the removal, but apparently this is the way z is collecting/adding entries, so z stops doing that automatically.  Perhaps -x could take an argument?

yeah this usage of sed isn't portable and I'll have to come up with something else. this proposal kinda works on lunix but throws an error
in terms of the directory getting added again, it's what i'd expect and I'm not inclined to go through hoops to prevent it. you are in that directory. It does lower the ranking.

This feature is tacked on to begin with, the point of z is to be useful without configuring it, using it is more of a conversation than an ordering it about (if it doesn't do what you expect you might do well to phrase the question differently).
maybe some more arguments about why it's important to have this feature that can't be solved some more cromulent way would help

@rupa My problem was that I have a directory which I z to very frequently, many times daily.  I don't think I cd to it at this point any longer.  Then one day, for one specific case, I cd to /a/very/annoyingly/long/path/somewhere/i/will/never/go/again/in/my/life/but/shares/a/single/word, and now my convenient z shortcut is broken and as far as I can tell there is nothing I could do to get it back.  I tried the obvious first thing of changing to the common directly several times to bump it in the ranking, but I could never get rid of the annoying dir :(  z -x didn't seem to lower its ranking enough that my common dir was ever used.  Solution?  Should simply cding to the common dir be bumping it's ranking?  If so then something is broken. I guess i'll test again.  I can try to reproduce and post the frecency scores/z db if you'd like.


NEXT-5762 | Category Tree is only build up to the second level.

The documention of NavigationLoader:load states that the full category tree
will be returned, but before this fix it only created a partical tree.

Removed conditions from categoryRepository search criteria.
Removed second level category searching.

1. Why is this change necessary?
See NEXT-5762
2. What does this change do, exactly?
See NEXT-5762
3. Describe each step to reproduce the issue or behaviour.
See NEXT-5762
4. Please link to the relevant issues (if any).
See NEXT-5762
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

HI @SakirSoft,
thanks for your contribution, but we've already addressed that issue internally and have an open PR to fix that.
Your solution would have a major performance implications on every shop with many categories, and we have to be especially carefull here as the navigationLoader is called on every page load.
So a fix to this problem will soon be published.


(Re)move editor specific styles

The editor.less file should only contain styles for the scroll-view. All remaining styles should be moved into the nord-atom-syntax theme package.

Resolved in the referenced commits. Merge in dc3237909cd8888caf68c38928c089076daf30f5.


how can we know the occupation number of corresponding active orbitals after CASSCF calculation?

Dear all,
I'm new to PySCF. I performed the CASSCF calculation, and I hope to know the occupation number of each active orbital, but I can't find the relevant information in the output file.  At the end of output file, I found the line "Density matrix diagonal elements [1.97623986 1.97036138 1.92039966 1.01726661 1.0227256  0.06913497  0.02387192]", are these numbers in square brackets the occupation numbers of active orbitals ? Also, there is no matching information about orbitals and its corresponding occupation number, so I don't know which orbital has the occupation number of 1.97623986. How to obtain this information? Many Thanks.
Below is the input file:
`import h5py
import os
from pyscf import gto, scf, mcscf
from pyscf import lib
from pyscf.tools import molden
mol = gto.Mole()
mol.atom = "/home/triplet-scan-2.4.xyz"
mol.basis= "def2svp"
mol.charge = 1
mol.spin = 2  # spin = alpha -beta, not 2S+1
mol.verbose = 4
mol.max_memory = 20480
mol.build()
mf = scf.RHF(mol).run()
mf.kernel()
mc = mcscf.CASSCF(mf, 7, (5,3))
mc.max_cycle = 500
mc.sorting_mo_energy = False
mc.frozen = 1   # frozen the inner-most 1s electrons
mc.dict.update(lib.chkfile.load('triplet-scan-2.4-rohf.hdf5', 'scf'))
mo = mc.sort_mo([25,26,29,34,36,38,87])
mc.chkfile="triplet-scan-2.4-cas87.hdf5"
mc.kernel(mo)
with open('triplet-scan-2.4-cas87.molden', 'w') as f1:
molden.header(mol, f1)
molden.orbital_coeff(mol, f1, mc.mo_coeff, ene=mc.mo_energy, occ=mc.mo_occ)
print(mo)`

The shortest path to the information you are looking for is simply to set mc.natorb=True prior to mc.kernel(mo). Under those circumstances, the Density matrix diagonal elements line is replaced with a Natural occ line that prints out natural-orbital occupation numbers in descending order, and the active-orbital columns of the MO coefficient array (i.e., mc.mo_coeff[:,mc.ncore:mc.ncore+mc.ncas]) are the corresponding natural orbitals in the same order.
In the unlikely event that mc.natorb=True has a deleterious effect on the convergence of CASSCF, you can instead do the natural-orbital transformation post facto by calling mc.cas_natorb_() immediately after mc.kernel (). There will then be both a Density matrix diagonal elements line and a Natural occ line in your output; you will want the latter.
PySCF canonicalizes the active orbitals by diagonalizing the Fock matrix by default (rather than the density matrix); the word "occupancy" is avoided in the printouts for such orbitals to avoid confusion with the natural orbital occupancy.

Also, maybe you know this, but the guess MOs provided to kernel are not changed in-place. The last line of your input prints the initial guess MO coefficient array, not the converged optimized MO coefficient array (the latter is accessed as mc.mo_coeff)

Dear Dr. Matthew Hermes, I really appreciate your reply, it's very helpful !


Json serialization issue

Save and load in a better format that handles numpy arrays

Update: cast everything to list instead of numpy array :) temporary fix until using funlib.persistence


Fix sol003-lifecycle-driver-vulnerability-issues

Fixes #33

Hi @dvaccarosenna
I have created PR for #issue/33 and #3373[TNC] , Kindly assign this PR to respective SME's

Done. For future requests please ask @hhellyer and @owen-lynch-ibm as they are the SME/maintainers of this project.


Launch ua_inspector.download.databases without mix

Hi,
I'm using distillery and I want to launch the databases download with my migration. How can I launch the download without mix (there is no mix in distillery) ?
Thanks

Do you know if the Mix modules (those from Elixir itself) are available in a distillery release?
If yes you could manually call the download logic using ["--force"] as an argument to prevent confirmation.
If not there is no direct way at the moment. Until the download logic is properly extracted (will try to shift my focus to that) there is a dependency on at least Mix.shell/0 to return Mix.Shell.IO. If the Mix modules are not available at all you would need to provide a dummy class providing the calls to Mix.shell.info/1 and Mix.shell.error/1.
That still would require you to manually trigger an internal reload by restarting the application as there is no public interface to do that (yet!).

In a distillery release, MIX is totaly unavailable. But I can launch action before the application start, like DB migrations or downloading ua_inspector YML files from piwik.
The difficulty is load the YML files.

I can launch migration but I don't know how to launch the ua_inspector download as mix is unavailable

Another possibility is to allow in config something like that: Path.join(["#{:code.priv_dir(:myapp)}", "ua_inspector")

The download is not yet possible without at least having some fake module for the Mix calls I mentioned. But you can move the database files in other ways, like including your priv/ folder or deploying them separately.
Using your example for configuration is actually allowed but cannot work because it is evaluated at compile time. Your compiled (and then deployed) configuration file would then include the local path returned when creating the release. So you need a different way to set the path, like a system environment
Or try Application.put_env(:ua_inspector, :database_path, Path.join([:code.priv_dir(:myapp), "ua_inspector"])) before starting the application and then restart the supervisor/application to force it to pick up the new values.

By now I have extracted the download into a completely separate module.
To use it before I have released a new version (only checking some things like uncleaned ets tables and stuff) you can update your dependency configuration:
{:ua_inspector, github: "elixytics/ua_inspector", ref: "f386ee490c6f099516214bd95d8f170106d7fdd6"}

This will allow you to use UAInspector.download/0 to download all files without mix. Be sure to call UAInspector.reload/0 after the download finishes (done synchronously, just wrap it in a Task if you like) if the application is not getting a full restart to ensure the database files are properly read.

Hi,
With those modifications, i can remove the yml files from my repository.
Thanks you very much
Is it possible to change the config to use by :code.priv_dir() as a default path ?

"Perhaps".
The problem is that the configuration is evaluated when building your release. If the path on the system creating the release and the path on the server is exactly the same you should be fine but I would not rely on that.
However you can simply set it manually in both your migration and application startup by calling Application.put_env/3 before calling UAInspector.download/0 and/or UAInspector.restart/0.

I think it will work.
When launching the app, I launch the migration.
I think I can create another task downloading then reloading ua_inspector.
In a task I think I can use priv_dir.
I will post here the result. Thanks
Le sam. 25 nov. 2017 à 12:24, Marc Neudert<EMAIL_ADDRESS>a
écrit :

"Perhaps".
The problem is that the configuration is evaluated when building your
release. If the path on the system creating the release and the path on the
server is exactly the same you should be fine but I would not rely on that.
However you can simply set it manually in both your migration and
application startup by calling Application.put_env/3 before calling
UAInspector.download/0 and/or UAInspector.restart/0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/elixytics/ua_inspector/issues/10#issuecomment-346934658,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABMbAcp6pkc_BfJ8k7fCguC5rceCaj4rks5s5_jegaJpZM4P_9xK
.

--
Claudio BERNARDES

I have released a new version with the aforementioned changes to downloading/reloading. If you still encounter any problems please add a comment here or open a new issue.


globalranking and levelrankings should be renamed

To follow our convention, globalranking and levelrankings should be globalRankings and levelRankings.

Hey I'd like to pick this up, discord - edinstance


Flaky tests due to connection reset when downloading dependencies

Test method
Happens before tests, in the build phase
Test logs
[2020-09-01T07:08:56.951Z] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M1:enforce (enforce-java) on project apm-agent-parent: Execution enforce-java of goal org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M1:enforce failed: Plugin org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M1 or one of its dependencies could not be resolved: The following artifacts could not be resolved: org.apache.maven:maven-settings:jar:3.0, org.apache.maven:maven-repository-metadata:jar:3.0, org.apache.maven:maven-model-builder:jar:3.0: Could not transfer artifact org.apache.maven:maven-settings:jar:3.0 from/to central (https://repo.maven.apache.org/maven2): Transfer failed for https://repo.maven.apache.org/maven2/org/apache/maven/maven-settings/3.0/maven-settings-3.0.jar: Connection reset -> [Help 1]

Additional context
Lately, we see a lot of test failures because of Connection reset errors.
@elastic/observablt-robots do you know if there are network issues?
Also, why does maven even need to resolve the dependencies? Shouldn't they be cached in the image via packer cache?

There is currently some work being done by the CI team on this, not sure about the current progress and priority though.

We may be able to add this to the image cache if it's not there already. I will investigate this.

I'm working on something else that seems similar to this issue, and this is my finding:

Caching is not happening since the .m2 folder is set to another location:

https://github.com/elastic/apm-agent-java/blob/master/Jenkinsfile#L17

That particular configuration was there since the very beginning:

https://github.com/elastic/apm-agent-java/commit/5fccf62177f015f9d7c40fdd98805013a7cd3dd3

https://github.com/elastic/apm-agent-java/pull/1376 might fix this issue, at least the one related to the cached artifacts

And guess what! I faced this particular issue in my above-mentioned fix:
For what I read the -Dhttps.protocols=TLSv1.2 flag seems to be the workaround:

build was manually triggered with the above flag as an input parameter

Even with a change to .m2 won't the fact that we do a clean install here force the download to happen regardless of what's in the cache? Should we remove the clean keyword to rely on the cache?

the clean goal should not affect with the cached dependencies, since it's in charge for the clean up of the generated files within the workspace which are normally hosted in the target folder. The only case I imagine it will be required is when the workspace location and the .m2 point to the some location

Yup, I just tested this and was coming back here to say that it doesn't appear to change the caching mechanism so this should be fine.


housekeeping: Added Central Package Versioning

btw those compile errors be useful to fix in tests at some point also.


Touch: Read of device family ID fails in release mode

This assertion fails because the read call returns an error:
https://github.com/embed-rs/stm32f7-discovery-rewrite/blob/a3ce451d9f9621efcff8a411cb1fcc4c7eccb7d2/src/touch.rs#L14-L15

Fixed in https://github.com/embed-rs/stm32f7-discovery-rewrite/commit/0b28f10e362e09d3bb5219cab164216d143493c2


Replaced usage of window.scrollX with window.pageXOffset

This is the fix discussed in issue #425

Thank you!


Update contributing_guide.md

Closes #146
Updated user guide to install requirements-docs.txt.

@cosmicBboy I have updated as per your recommendation...kindly review and let me know if any other changes are required!


Rustfmt on Windows changes line endings

Running cargo fmt on windows changes all line endings from LF to CRLF.
The rustfmt.toml contains newline_style = "Unix", could it be a rustfmt issue?

I just updated nightly to that version but it is still happening for me.
cargo fmt --version gives me: rustfmt 1.4.20-nightly (48f6c32e 2020-08-09)

I get the same version information.  I don't know if rustfmt looks at that, but it might
For the record, mine is input

It's true, it might be the issue

I think what is actually happening then is that git is checking out the files as CRLF, then rustfmt sees that and converts them to LF, then git sees that it is changed. But I think when you add the files to the index, (using git add), it will see that they are the same as on HEAD and so will remove those changes. So in theory it would be fine to leave it that way (as in the commits you make will have LF), but it just makes things painful for you.
Line endings are weird.
Hopefully that's fixed it.


Client-Side getTankInfo for EnderTank

Querying getTankInfo on in a client side method returns a null fluid (new FluidStack(0,0)) - somewhere on client side you have access to the fluid for rendering purpose so could you modify the method to return the correct (or at least a valid) fluid

The clients only receive their tank info when they're being looked at. I suppose I could say it has 0 water in it, but you shouldn't be asking for that info on the client anyway.

I'd really rather not be working from client side either but I don't really have a choice about that - the method calling it is client side only so unless I was to do some message handling and data caching which is not really optimal...
Is the frequency available clientside? I could just display that instead of the contents of EnderStorage stuff

It would appear that I do have the required info on the client side. Notice
that the clients interpolate the server, so do you want what's rendered, or
what the server says it has (eg change the frequency and it'll immediately
have the new liquid, but you'll see an animation on the client)

Preferably what the server says it has.
Thanks a million

Here you go dce6dbfd7644f8986624c0d579e81b2336522bb8 (1.7.10-<IP_ADDRESS>)

honestly can't tell what's the fix and what's refactoring, but super :)

Yeah sorry about that, just find the getTankInfo method
On Tue, Mar 31, 2015 at 6:49 AM, hilburn<EMAIL_ADDRESS>wrote:

honestly can't tell what's the fix and what's refactoring, but super :)
—
Reply to this email directly or view it on GitHub
https://github.com/Chicken-Bones/EnderStorage/issues/34#issuecomment-87827389
.

Was this commit rolled into a release?
Simply viewing the tank in NotEnoughItems or via creative mode tabs will cause a crash with <IP_ADDRESS>, the latest available download, complaining about a null fluid stack:
[21:51:44] [Client thread/WARN] [FML/]: * Null fluid supplied to fluidstack. Did you try and create a stack for an unregistered fluid?
[21:51:44] [Client thread/WARN] [FML/]: *  at net.minecraftforge.fluids.FluidStack.(FluidStack.java:28)
[21:51:44] [Client thread/WARN] [FML/]: *  at net.minecraftforge.fluids.FluidStack.(FluidStack.java:59)
[21:51:44] [Client thread/WARN] [FML/]: *  at codechicken.enderstorage.storage.liquid.TankSynchroniser$PlayerItemTankCache.getLiquid(TankSynchroniser.java:225)
[21:51:44] [Client thread/WARN] [FML/]: *  at codechicken.enderstorage.storage.liquid.TankSynchroniser.getClientLiquid(TankSynchroniser.java:264)
[21:51:44] [Client thread/WARN] [FML/]: *  at codechicken.enderstorage.common.ItemEnderStorageRenderer.renderItem(ItemEnderStorageRenderer.java:48)
[21:51:44] [Client thread/WARN] [FML/]: *  at net.minecraftforge.client.ForgeHooksClient.renderInventoryItem(ForgeHooksClient.java:183)...
[21:51:44] [Client thread/WARN] [FML/]: ****************************************
Thoughts?

Yeah, it was, that's a different bug which has since been fixed


Mobile Renderer with MSAA not working when using the viewport stretch mode (presents a black screen)

Godot version
Godot Engine v4.0.beta2.official.f8745f2f7 - https://godotengine.org
System information
Windows 10, Vulkan API 1.2.0 - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 960
Issue description

Steps to reproduce
Forward_Plus is OK, but something happened with mobile renderer

Project Settings->Renderer, switch Rendering method to "mobile"
Project Settings->Anti Aliasing, enable MSAA 2D "2x(Average)" and MSAA 3D "2x(Average)"
Present a black screen with massive error spam

Minimal reproduction project
demo.zip

Does this occur if you only change 2D MSAA or 3D MSAA, not both?

Does this occur if you only change 2D MSAA or 3D MSAA, not both?

yes, single one or both, all present black screen

Re-rested on beta4

Re-tested on Beta5

Re-tested on Beta7

It's the good old MSAA + viewport stretch mode + window size different from viewport size issue, but in 4.0:

Ran into this issue with Godot 4 beta 16, tried different combinations of MSAA, but I'm only getting a black screen when MSAA_2D is set to something other than disabled.

Recent events on Cybereality and my experience makes me feel that keep the hope for Godot, provide detailed technical analysis, keep track of some bugs that will never be fixed is meaningless, so close it to reduce the number of issues for Godot.

Reopening, as I can still reproduce this in 0f7625ab4.
Note that this does not occur if Scaling 3D > Scale is set to a value different than 1.0.

Reopening, as I can still reproduce this in 4.2.dev 0f7625a (Linux, GeForce RTX 4090 with NVIDIA 535.54.03).
Note that this does not occur if Scaling 3D > Scale is set to a value different than 1.0. This means that as a workaround, you can use a scale of 0.99 or 1.01.

Forward+ and Compatibility renderers works fine with my settings however when MSAA 2D is set to any value other than
Disabled, the scene does not render and gives a black screen in Mobile renderer.
It doesn't display any error information by the way.
Turning of MSAA for 2D makes it work fine again.
After the video, I downloaded latest stable version 4.1.3 (.Net) and the same issue persist there:
https://www.youtube.com/watch?v=XJAMBNlhN8U

On Android (Pixel 3), turning on MSAA 3D with any option in a blank project produces a black screen. Turning on MSAA 2D doesn't. No error via Godot Debugger. No error via adb logcat.


Doesn't work in full-screen apps w/ automatic menu bar hiding enabled

Works fine in full-screen apps without autohiding enabled.
Works fine on desktop with autohiding enabled/disabled.
However, in a full-screen app with autohiding enabled, the popup disappears as soon as you attempt to move the cursor down to interact with it, making Tuneful impossible to use.

Hi, can you please post a video of the behaviour? Tried to replicate it but it worked fine for me

https://github.com/martinfekete10/Tuneful/assets/52586835/e7058bfb-702c-4340-92bc-0198a9d6934c

Bump. Also I should add that I am using Bartender, so I wonder if other people who use it have the same issue as this would be demonstrative of some sort of conflict between the two applications.

I should add that any slight cursor movement in any direction after opening the Tuneful popup causes it to disappear, not just downward motion. Even moving to the right along the menu bar still causes the issue.

My apologies, I am actually experiencing this issue with a variety of third-party menu applications (but not all). So, obviously the issue is on my end and not yours. I will leave an update if I figure out the root cause of the problem.


[Bug]: Panic occurred at runtime - Message:  should have module graph module

System Info
System:
OS: macOS 14.5
CPU: (10) arm64 Apple M1 Max
Memory: 9.82 GB / 64.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm
pnpm: 8.15.4 - /opt/homebrew/bin/pnpm
Browsers:
Chrome: 125.0.6422.142
Edge: 125.0.2535.92
Safari: 17.5
Details
After upgrading Modern JS to version 2.52.0 I get the following error:
           Panic occurred at runtime. Please file an issue on GitHub with the backtrace below: https://github.com/web-infra-dev/rspack/issues

drop-in-ui:dev: Message:  should have module graph module
drop-in-ui:dev: Location: crates/rspack_core/src/module_graph/mod.rs:563
drop-in-ui:dev:
drop-in-ui:dev: Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.
drop-in-ui:dev: Run with RUST_BACKTRACE=full to include source snippets.
drop-in-ui:dev: ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
drop-in-ui:dev:  1: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev:  2: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev:  3: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev:  4: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev:  5: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev:  6: _wasmer_vm_imported_memory32_atomic_notify
drop-in-ui:dev:     at
drop-in-ui:dev:  7: _wasmer_vm_imported_memory32_atomic_notify
drop-in-ui:dev:     at
drop-in-ui:dev:  8: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev:  9: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 10: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 11: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 12: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 13: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 14: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 15: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 16: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 17: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 18: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 19: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 20: _napi_register_module_v1
drop-in-ui:dev:     at
drop-in-ui:dev: 21: __pthread_joiner_wake
drop-in-ui:dev:     at
error Command failed with signal "SIGABRT".
With a bit of digging it appears that there seems to be an issue with prependPlugins in rspack config for Modern JS when using the rspack.CopyRspackPlugin. If I remove that section it will fix the error.
I am unable to share the original problem repository that breaks on yarn dev because it is proprietary for our company and they will not give permission.
However I was able to create a new modern repository (see repro link). It will still run fine on modern dev but if you change something in modern.config.js and save it while modern dev is still running, it will have the same error, or at least the same stack that looks the same.
I was unsure if this is the right place or if it should be reported in Modern JS but the error said to post the issue here.
Reproduce link
https://github.com/bmoellers/modern-upgrade-error
Reproduce Steps

yarn dev
make a simple change to modern.config.ts
should see error in console where yarn dev was run

Original issue was seen just when running yarn dev but the above was the only way I could reproduce it.

@bmoellers i can not reproduce panic from your repo. Can you try 0.7.2 to check if it is fixed?

I will see if I can override it somehow but we are using the rspack version bundled with Modern JS and not a direct include. I can probably include and alias it so that even Modern JS requires that version or I can just wait for a new Modern JS version and see if that fixes it. Not in a super hurry to upgrade everything but want to keep things as close to the current version as possible.

same issue +1, it is caused when I use rspack with module federation.
if I remove the tools config, all things work fine

same issue +1, it is caused when I use rspack with module federation. if I remove the tools config, all things work fine

just tried 0.7.2, hmr got fixed :D


Windows are closing when open different database

Hi. I'm using Ubuntu 18.04 - DBeaver 5.1.1
After updating my OS to 18.04 , DBeaver closes my  pages like this: PROBLEM
Is this bug or there is any option to change ? Thanks.

I never saw such glitch.
Perhaps it is Dark theme rendering issue.
Could you try the same in classic theme?

Thanks for answering. I tried it on default theme but  it didn't work. Also I tried after updating 5.1.2 but result is same. Problem

Ah, now I see.
This is not actually a bug but a kind of limitation. This happens because DBeaver reconnects to another database once you change active database (it is PostgreSQL-specific behavior).
We have FR #3771 - it will resolve this issue.


Make the flatpak(-spawn) portal nicer to work with

The flatpak module currently doesn't seem very nice to work with; ideally it would more closely look like the flatpak-spawn command, and (imo) wouldn't look as "ugly".
Here's a quick draft of what the ideal struct for it would be like, this 100% needs tweaking in order to work with the Flatpak portal, and work "properly":
use std::os::fd::RawFd;
use std::fs::File;

#[derive(Debug)]
struct EnvVar {
    var: String,
    val: String,
}

/// A (more or less) 1:1 copy of the flatpak-spawn command, as a struct
#[derive(Default, Debug)]
struct FlatpakSpawn {
    /// Enable debug output
    verbose: bool,
    /// Forward file descriptor(s)
    forward_fd: Vec<RawFd>,
    /// Run with clean environment
    clear_env: bool,
    /// Make the spawned command exit if we do
    watch_bus: bool,
    /// Expose sandbox PIDs in calling sandbox
    expose_pids: bool,
    /// Use the same PID namespace as calling sandbox
    share_pids: bool,
    /// Set environment variable(s)
    env: Vec<EnvVar>,
    // Unset environment variable(s)
    // Remember to only pass through the variable name, not the contents!
    // TODO
    // unset-env: Vec<String>

    // TODO: env-fd

    /// Run with the latest vesion of the app and runtime
    latest_version: bool,
    /// Run sandboxed, without inheriting permissions.
    /// See the --sandbox argument in https://man7.org/linux/man-pages/man1/flatpak-run.1.html
    /// for more information.
    sandbox: bool,
    /// Whether to run with network access
    no_network: bool,
    /// Expose read-write access to the files/directories
    sandbox-expose-rw: Vec<File>,
    /// Expose read-only access to the files/directories
    sandbox-expose-ro: Vec<File>,

    // TODO: sandbox-flag

    // NOT PLANNED: host (use a separate struct, ideally closer to Command)

    /// Working directory in which to run the command
    directory: File,

    // Replace runtime's /app with DIR, or empty
    // TODO
    // app-path

    /// Replace runtime's /usr with DIR
    usr-path: File,
}

PRs are welcome.


Add @DoFn.yields_batches and @DoFn.yields_elements decorators to override defaults

Imported from Jira BEAM-14293. Original Jira may contain additional context.
Reported by: bhulette.
Subtask of issue #21650

Unable to assign user @theneuralbit. If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.


Android stretches horizontally when recording

The camera looks fine at first. Once recording begins, it gets stretched horizontally. Once you end recording though, the file itself is NOT stretched.
I do not have this issue with iOS.

Woops wrong library


Dev v1.16.2

Development work to bring the driver up-to-date with the current version of RethinkDB.

Coverage increased (+14.52%) to 93.48% when pulling 6b3fb910ade0f307f233b86765ca2b844f18a1cd on dev-v1.16.2 into 42076f21511e2a7063b59896e1bd7fa84a726bd5 on master.

Coverage increased (+14.52%) to 93.48% when pulling 6b3fb910ade0f307f233b86765ca2b844f18a1cd on dev-v1.16.2 into 42076f21511e2a7063b59896e1bd7fa84a726bd5 on master.

Coverage increased (+14.52%) to 93.48% when pulling 6b3fb910ade0f307f233b86765ca2b844f18a1cd on dev-v1.16.2 into 42076f21511e2a7063b59896e1bd7fa84a726bd5 on master.

Coverage increased (+14.53%) to 93.48% when pulling<PHONE_NUMBER>c3e8200e2416cd3139c04e88bfa5bd on dev-v1.16.2 into 42076f21511e2a7063b59896e1bd7fa84a726bd5 on master.

Coverage increased (+14.53%) to 93.48% when pulling<PHONE_NUMBER>c3e8200e2416cd3139c04e88bfa5bd on dev-v1.16.2 into 42076f21511e2a7063b59896e1bd7fa84a726bd5 on master.

I did it!