Child process exited with code: null

Hello,
as asked by you in issue #89 I opened a new issue for this.
I am not able to use the vscode extension and I am getting the following Error messages:
Error: Child process exited with code: null
Error: Child process already exited.

The Electron in your computer seems to be having a problem.
Try this:

Uninstall the extension via VS Code.
Remove this directory:
/home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5
Execute these commands in terminal app (not "TERMINAL" panel in VS Code):
npm uninstall -g electron
npm uninstall -g electron-prebuilt
npm cache clean
Execute this command in terminal app to confirm the cache data were cleared (i.e. no file is shown):
npm cache ls
Remove this directory:
~/.electron.
Press Ctrl + @ keys (Shift key also might be required) in VS Code. Then a "TERMINAL" panel is opened.
In the "TERMINAL" panel above (not terminal app), execute this command:
node -e "console.log(require('electron-prebuilt'))"
Then an error Error: Cannot find module 'electron-prebuilt' should occur (this is correct result).
Re-install the extension via VS Code.

Thank you but unfortunately after performing all of the steps you mentioned the same error still persists...

Try this in terminal app (not "TERMINAL" panel in VS Code):
cd /home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5
npm uninstall electron-prebuilt
npm i<EMAIL_ADDRESS>--no-audit

I'm still having the same issue... no changes

Could you show me the result of the commands above?
Also, show me each result of these:

In the "TERMINAL" panel:
node -e "console.log(require('electron-prebuilt'))"
In terminal app:

cd /home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5
node -e "console.log(require('electron-prebuilt'))"

That's the command and their outputs from above:
$ pwd
/home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5
$ npm uninstall electron-prebuilt
removed 69 packages and audited 834 packages in 2.258s
found 32 vulnerabilities (8 low, 7 moderate, 16 high, 1 critical)
  run `npm audit fix` to fix them, or `npm audit` for details

$ npm i<EMAIL_ADDRESS>--no-audit
><EMAIL_ADDRESS>postinstall /home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5/node_modules/electron-prebuilt
> node install.js

+<EMAIL_ADDRESS>added 69 packages from 47 contributors in 8.449s

That's the new commands output after executing the above and without reinstalling the extension within vscode:
From VSCode:
internal/modules/cjs/loader.js:775
    throw err;
    ^

Error: Cannot find module 'electron-prebuilt'
Require stack:
- /home/anonymized_user/Desktop/some_folder/[eval]
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:772:15)
    at Function.Module._load (internal/modules/cjs/loader.js:677:27)
    at Module.require (internal/modules/cjs/loader.js:830:19)
    at require (internal/modules/cjs/helpers.js:68:18)
    at [eval]:1:13
    at Script.runInThisContext (vm.js:126:20)
    at Object.runInThisContext (vm.js:316:38)
    at Object.<anonymous> ([eval]-wrapper:9:26)
    at Module._compile (internal/modules/cjs/loader.js:936:30)
    at evalScript (internal/process/execution.js:80:25) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/home/anonymized_user/Desktop/some_folder/[eval]' ]
}

From terminal:
/home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5/node_modules/electron-prebuilt/dist/electron

Save this test script to /home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5/test.js file.
process.stdin.on('error', error => { throw error; });
process.stdin.on('close', () => { process.exit(); });
require('process-bridge').sendRequest({command: 'pick'},
    [require('path').join(__dirname, 'lib/app.asar')], error => {
  if (error) {
    if (error.isRetried) { return; }
    throw error;
  }
  process.exit();
});

And, execute these in terminal app:
cd /home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5
node test

This is the output of that script:
Start child process...
Check version of: electron-prebuilt
Check version of: comma-separated-values
Check version of: js-yaml
Check version of: process-bridge
Try to send IPC message: 1
Child process disconnected
Child process exited with code: null
/home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5/test.js:7
            throw error;
            ^

Error: Child process exited with code: null
    at /home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5/node_modules/process-bridge/process-bridge.js:478:19
    at ChildProcess.<anonymous> (/home/anonymized_user/.vscode-oss/extensions/anseki.vscode-color-0.4.5/node_modules/process-bridge/process-bridge.js:394:19)
    at ChildProcess.emit (events.js:209:13)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:272:12) {
  code: null,
  signal: 'SIGSEGV'
}

Ah... I see.
It is SIGSEGV...
The Arch (Manjaro) Linux seems to be having a problem about the Electron and "glibc".
https://github.com/electron/electron/issues/13972
The problem might be solved by downgrading the glibc, but I don't know whether this is best solution or not.
Sorry, I can't help you about this...
If you report about the downgrading the glibc, that should help other users.
Or, you can close this issue and choose another extension.

Hmm, ok, very strange. All other electron apps I have installed work without a problem and don't cause a segmentation fault. I will look into this issue some more and report if I find a solution.

I close this issue.

Hmm, ok, very strange. All other electron apps I have installed work without a problem and don't cause a segmentation fault.

Nothing strange here really. This otherwise nice and useful vscode extension I used to use daily unfortunately relies on 4 years old prebuilt version of Electron.
This isn't Arch Linux problem. Eventually all Linux distribution will not be able to run old binary.

Hi @AloisJanicek, thank you for the information. :smile:


[v2] include existing group_vars files

Issue Type

Feature request

Molecule and Ansible details
molecule, version 1.14.1.dev207

Molecule installation method: pip

Desired Behaviour
Hi there,
I'd like to know which is the proper way to include an existing group_vars directory instead of declaring all variables within molecule.yml
Checking #171 it seems this was/is possible, but couldn't find anything similar in V2 with the new list group/host vars list management.
Some other issues suggest symlinking folder into .molecule generated structure, but this would mean I would pre-allocate the structure before molecule running. Since such folder is intended to be manage by molecule this doesn't seems a good practice.
Isolated inclusion in molecule is sometime useful, but they should be able to shift/merge over existing group_vars files.
Any hint on the best way to proceed?
thanks in advance!

@aroundthecode It is not possible.  We do not support this functionality.  Yes, #171 had it, but it is rather hackish, therefore we do not support it in v2.
It may be re-evaluated once we have a version of ansible supporting the new plugin inventory.
https://github.com/ansible/ansible/pull/23001.
You're free to do the symlinking yourself, outside of molecule.

@retr0h, I created a shell script which pre-allocate the .molecule/group_vars folder and include my files before triggering molecule, but it seems that molecule is re-creating such folder at each run (which is more or less what I was expecting).

just to let you know, my current workaround was to create a role which just include Group_vars needed files.
Such role is set to be a dependency of all other roles needing such configuration.
Working this way any role will first invoke the new dependent one before executing, including all needed vars.

👍

I have to change my opinion on this workaround since this drastically changes the variable priority during ansible execution.
Variables in group_vars can be overwritten by inventory parameters, while variables as default in a dependency role cannot.
Probably the best solution is to add a task in the create playbook (which runs AFTER the .molecule folder has been reset) to copy my group_vars files into the proper path, but now I'm hitting the issue described into #855

Reimplemented #171 into v2.

thanks @retr0h , I'll give it a try on Monday and let you know!

@retr0h I'm checking the new behaviour.
It's ok that enumeration in molecule.yaml and linking are mutually exclusive, can it be done separately for group var and host_vars sections?
e.g. Keeping host_vars enumerated in molecule.yaml while group_vars are linked ?
The actual behaviour now is that if the link section is present it wins over all,

The actual behaviour now is that if the link section is present it wins over all, while if the check could be split for the two sections it would be great!

Right, linking wins overall.  I really don't think uses are going to be mixing and matching.  All of the feedback with molecule has been.  Hey I don't like having to add inventory to molecule.yml, can I supply my own from existing inventory.  Previously, I said no.  Now, I can answer that question with yes.
Mixing and matching could be done, but I would prefer to keep it simple for the time being and see what ppl think.


apugli:modify_enchantment_level still doesn't work if Item Stack is missing with some enchantments

Specified Power fails to apply the enchantment modifier to the power holder if the power holder has no item stack at all in any of their Equipment Slots.
Example:
Lets say I have this Power on a Player:
{
    "type": "apugli:modify_enchantment_level",
    "enchantment": "minecraft:respiration",
    "modifier": {
        "operation": "add_base_early",
        "value": 3
    }
}

This Power only grants III levels of Respiration to the player IF they hold at least an Item Stack in their Main Hand, Off Hand or Armor Slot. If none of them are filled the power fails to add the needed enchantment level modifications.

I think it's not working because of another reason entirely.
I made a change last update to the logic that didn't stick.

I've managed to fix this, it'll be fixed in the next update.


[Feature] Add Sharepoint datasoure Issue #19

Adding nesis api implementation for sharepoint datasource this is the backend that will support adding sharepoint document library sources.

Fixed by #61


Add auth

Auth was recently added tomBackstage APIs (it's possible to secure endpoints now), default plugins support auth headers (if any) out of the box. This PR adds same behaviour.

Hello @K-Phoen.
Please take a look.
I've tested my fork with my Backstage setup (with secured endpoints) and it seems to work fine.
Thanks.


Bimodal behaviour on complex_layout_scroll_perf_ios__timeline_summary worst_frame_rasterizer_time_millis

The complex_layout_scroll_perf_ios__timeline_summary worst_frame_rasterizer_time_millis benchmark is bimodal, with some results being around 20ms and some being around 60ms. We should figure out why and either fix the benchmark or the engine to be more reliable.
cc @cbracken @liyuqian

Regressions started to flake since c96806ec887bc8120e236ae37ecb966e483e5451 on 7/16

Interesting, it seems that the old bimodal behaviour is gone on 7/16. Instead, we get some red bars, which is much slower than before, but also less frequent than the old modal. It's hard to tell which commit changes the behavior due to the noise. Maybe an automated bisection is needed to understand this.

check if the bimodal behavior corresponds to running on different hosts, could just be two different hardware configurations

worst frame times are frequently unstable, as host select or a random GC can make them much worse. Closing as unactionable.


Resolve #11727

Summary of the changes (Less than 80 chars)

Add a BuildServiceProviderValidator for checking startup method 'ConfigureServices(IServiceCollection,..)'
Detail 2

Addresses #11727

It seems no RPs for this issue.So,I do it~ :smile:

Why are Startup analyzers in MVC?

There's a work item on my plate to get the more general Microsoft.AspNetCore.Analzyers assembly shipped in the SDK. It's a purely point in time issue.

There are no tests or asserts for DiagnosticDescriptor properties such as Id,`Title.
Should I add tests or asserts for them?

Should I add tests or asserts for them?

I don't think it's necessary, those tests would just verify that the text is what you typed in. Those kinds of tests are valuable if the diagnostic output contains format parameters.

Thanks!


promscale: 0.10.0 -> 0.17.0

Description of changes

promscale was updated from 0.10.0 to 0.17.0
which required adding separate promscale_extension for postgresql built using  buildPgxExtension function developed here
Added myself to maintainers

P.S. This PR was prepared before the promscale team announced deprecation of the project, but since it was already made I'm submitting it anyways.

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

NixOS test(s) for postgresql 12, 13, 14

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

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

[X] Fits CONTRIBUTING.md.

@ofborg eval

Thanks.
Could you format your commit message according to CONTRIBUTIONS.md.
There are also 10 commits changing 7 files, that can be condensed into (e.g.):

maintainers: add anpin
promscale: 0.10.0 -> 0.17.0
postgresqlPackages.promscale_extension: init at 0.8.0
nixosTests.promscale: add tests for promscale_extension

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2046

LGTM, but remember to remove unsupported software from nixpkgs.
A warning should also be thrown, might be implemented with lib.warn:

This needs to be reverted because it does not go well with ofborg

LGTM, but remember to remove unsupported software from nixpkgs.
A warning should also be thrown, might be implemented with lib.warn:

This needs to be reverted because it does not go well with ofborg

How can we add this warning back? It's important to communicate this to users of this package.

We can't unless it shouldn't be cached on cache.nixos.org. Maybe we can rewrite a release note entry but then we should also remove it probably.

It's already not cached because it's unfree license.


[UXSite] Improve color contrast between form inputs and placeholders

Q
A

Bug fix?
yes/no

New feature?
no

License
MIT

The contrast between input values and/or placeholder texts was a bit light
(exemple: https://ux.symfony.com/live-component/demos/form-collection-type)
Before :

After :

Thanks Simon!


Chaos readiness check

There are no conditions in the chaos CR. Hence, the check is only for existence.

Replaced by #1345


[Windows]: Resize event not fired when window snapping

When either using winkey+arrow, the new Windows 11 snapping options, or the "simulated snapping" with a double click on the title bar, the resize event isn't being triggered

Can you give a minimum reproducible code?

seeing this too. The reproducible code is trivial. add a window manager with an onWindowResize override and then use the windows 11 snap layouts feature either with win+z or hovering over the expand icon. When you choose a window size no resize event fires. Is there support for this windows 11 feature?


Fix target height not being computed in site nav logo

targetHeight value is not being computed because of missing template literals, therefore wrong CSS is being produced as seen in the following screenshot. This PR fixes it.

Thanks for catching this bug!

all-contributors please add @flaviolivolsi for code

@all-contributors please add @flaviolivolsi for code


Blendshape DNA hookup

Added BlendShapeDnaConverterBehaviour and BlendShapeDnaAsset
Simple DnaName<->BlendShapeName pair and value.

There's a lot of extra commits from the work branch in there.  Let me know if you want me to refork from the work branch and add my two new files and then commit so there's not all the extra unnecessary stuff.


run/build_sdk_container: support Podman

When the docker wrapper script for Podman is used, we need to
explicitly create a root user container with "sudo podman".
Podman also has its own bridge for root user containers which we need
to detect.
Add a "$docker" variable that uses sudo as needed, and also check which
bridge interface to use.
How to use
Port to all active branches (except flatcar-lts-2605)
Testing done
Used it for both regular container starts and the building of the SDK container (still running, though)

Pushed one more change to use the full container image reference for caddy


Revamp the command system of terminal

Add more linux based commands to xterm or find an alternative to xterm for linux based cli.

I'll take this @AbhigyaKrishna

/assign

Assigned

please take a look at this, as of now it cannot execute bash script
Screencast from 2024-12-22 22-54-21.webm

Looks good. Change the ls command to list entries like linux dir format

Hey @AbhigyaKrishna -
does this look good?
Screencast from 2024-12-24 15-07-58.webm


Fix query string in rack handler

Hi,
This PR fixes QUERY_STRING filling in rack handler (currently it's always empty).

Oops. Thanks for spotting that.


Token and Derivation get compared somehow

Trying to parse
'$u_delta < 1.5 and $u_i < $u_elements'
, without single quotes, using this Lark invocation:
self.expr_parser = Lark(open("gplus_expr.lark.txt").read(), start='expr')
and the attached grammar gplus_expr.lark.txt
yields the attached stack trace. trace.txt
Wrapping a try/catch around the implementation of compare in lark/lark/utils.py seems to work, as in
    def compare(a, b):
        try:
            if a == b:
                return 0
            elif a > b:
                return 1
            else:
                return -1
        except TypeError:
            return 0

and results in no failing tests, but this seems very unprincipled.

Fixed in master.
Btw, you should probably make dsvar_name and uservar into tokens (USERVAR etc.) to get their strings in the tree instead of lists of characters.

Oh, that's what was doing that? Thanks!


istio-remote helm chart should optionally render services' info

Describe the bug
Not every Istio installation will have all possible Istio project services instantiated.  The istio-remote helm chart should be able to be rendered to exclude unused services.
Expected behavior
For example, if zipkin is not part of the Istio control-plane installation, the whole selectorless services section and endpoints section for zipkin should not be included in the resulting manifest.
Steps to reproduce the bug
The procedure in the multicluster documentation:  https://preliminary.istio.io/docs/setup/kubernetes/multicluster-install/
helm template install/kubernetes/helm/istio-remote --namespace istio-system --name istio-remote --set global.pilotEndpoint=${PILOT_POD_IP} --set global.policyEndpoint=${POLICY_POD_IP} --set global.statsdEndpoint=${STATSD_POD_IP} > $HOME/istio-remote.yaml

Version
Istio: master as of 6/12/2018
kubernetes: any
kubectl: any
Is Istio Auth enabled or not?
Any
Environment
Kubernetes multicluster (any cloud)

With the merge of #6366 the configs are conditionally applied.  Closing this.


fix: dont pass fetchParallel log concurrency value to fetchAll entry concurrency

Was still linearly fetching entries. Concurrency values passed to fetchParallel was undefined, then set to 1 or hash.length. It looks like this value was intended to be the fetchParrallel (log) concurrency. But is was also being passed to fetchAll in the fetchOne function. Often this value was 1 or much less than 32 so often fetched linearly or not much faster.
Otherwise the entry refs are awesome! saw huge speed up when syncing from remote ipfs node once parallelized, compared to before.
Let me know if this was not what was intended or if there is any code changes needed elsewhere with this.

LGTM! @shamb0t , @haadcode ?

Thanks @zachferland you're right, good catch!


add serviceAnnotation to KubernetesResourcesSpec.

resolve: https://github.com/istio/istio/issues/19959

@morvencao sorry about this but you may need to redo the PR due to https://github.com/istio/operator/pull/713 as I wasn't able to merge this cleanly. Please wait a couple of days for operator to merge to istio/istio and check if your code looks ok?


Feat: Add file tree, add include and exclude prefix for auto ingestion

This PR add file tree to show knowledge files. Demo is here https://www.loom.com/share/0e707405c1194ea1a9a1ffd595347a22
It also adds a backend piece of functionality to include/exclude folder prefix so that user can add a folder in UI to automatically ingest files inside the folder.

Noticing that the add button is getting cutoff when the file name is too long. I think we need to truncate the text and add a tooltip to it

@ryanhopperlowe good catch, should be fixed 🙏

Beautiful 🙌


test-ec2 ci job fails with PATH issues

I'm unsure why this started failing, but it appears to be a pathing issue:
[test-ec2] $ /bin/bash /tmp/hudson1646044018615115181.sh
WARNING: login credentials saved in /home/jenkins/.docker/config.json
Login Succeeded
/tmp/hudson1646044018615115181.sh: line 14: cd: src/github.com/deis/deis: No such file or directory
Changes:
------------

The "check out to a sub-directory" build step apparently was lost. Not sure how that happened, but it is fixed now.


cache layer around get requets

Wondering what it would be best as an approach here... Implementation idea:

Adding a $app->pipe(CachingMiddleware::class); call after $app->pipeRoutingMiddleware(); in pipeline.php, so to "intercept" all GET requests and then act upon requested URL (either fetching data and creating a cached result; or returning the latter);

Store contents somewhere on the Filesystem (where shall I putt cached stuff?)

I came up with two ways to perform cache invalidation:
a] at each call, we check for the last update time of cached result and invalidate it once a certain time interval has passed (pro: everything is self contained; cons: it'd never be a good approach for production)
b] we have an external CLI script, to be invoked by cron (or by hand, in our context), which takes care of cleaning up old resources (cons: additional software, DRY violation (multiple places - framework and OS - to handle same thing); pros: additional way to interact with the framework to be shown at the workshop - IE the CLI; similar to something I'd use in production).
Which one would be best in your opinion? Or do you think there could be other, better ways to handle this? Thanks for your support! :)

random thoughts:

I would rather act at a route level, instead of a pipeline one;
caching must be done before (i.e. outside) dispatch
I would choose the expiration time approach; way simpler for our actual purpose. We will mention that it's not good for production

Why so? If we aim at catching all GET requests, it might make sense not to spread to many $app->pipe(CachingMiddleware::class); statements. Having an high Fan-In at the beginning easily allows us to become more specific when and if needed. I might be missing something in the big picture, though.

I was thinking about putting it before the $app->pipeDispatchMiddleware(); call. Just after  $app->pipeRoutingMiddleware(); (so to know enought about the reuqest), but definitely before domain logic kicked in. At least that was the idea... Again I might have missed something and picked the wrong location, thinking to do the right thing

Agreed. Will follow this hint. And add a comment on code. Would feel umconfortable otherwise :)

The reason is to keep the middleware very simple and give the user the possibility to decide which routes to cache easily (you maybe want to cache the lists, but not the details)
I didn't say you were wrong, I just said before dispatch, but certainly after routing

Made a first implementation. I still need to fix a JSON formatting issue on cached responses. I'd also like to take configuration parameters (IE cached files path, expire time) out of factory class and into configuration files. Feel free to take a look at response-cache and let me know if you notice anything which not ought to be there. Tnx!


Build(deps-dev): Bump mochawesome-report-generator from 6.1.1 to 6.2.0 (#3201)

Pull request opened by the merge tool on behalf of #3201

Merged in a7d7bad152235f4e3406f67c6a415d8ebc47bea9


Create User Authentication

I need to make user Authentication for the site

The authentication of the site is done, I believe that users can sign in using there google or GitHub accounts


Automatic build stopped on Docker Hub

The automatic image build seems to have stopped because the token has been revoked.
It will return to normal if you tie it again. Please do the procedure.

Which token would that be?

Docker Hub hack exposed data of 190,000 users
https://zd.net/2V1PxJK

A while ago, an accident occurred inside the Docker Hub.
As a result, all github tokens associated with Docker Hub were revoked.
You need to access Docker Hub and re-associate your github account.

Should be re-linked now. Please take a look and let me know if that's the case for you.
Gerardo

Closing this, feel free to reopen it if needed. Thanks!


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 110 ms

Resolved: ClamAV performance has improved in a2e2a21 after 5 minutes.


Make gained command work properly with inferred username

Not sure if this behaviour was meant to be like this, but if you have set a username and use the command !gained without any arguments, the default "week" argument didn't trigger. This fixes that.

I guess it assumed there'd be atleast one other argument (the name). That's why it only worked if you specified the time period, interesting..
Thanks!


[RF] Block store refactor

Its suppose to make the store closer tied to consensus, then consensus can read missing blocks when it needs to recover from a reorg.
One more thing this will allow us to have a pruned node in a proper way (where we keep the last X blocks in disk and delete anything before that).


Translation PR with only classic files

What

Translation PR with only classic files

Codecov Report

Merging #4414 (131a6dd) into develop (3f2b7a7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #4414   +/-   ##
========================================
  Coverage    10.14%   10.14%
========================================
  Files          295      295
  Lines        15088    15088
========================================
  Hits          1531     1531
  Misses       13557    13557

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


Training the model using with a custom triplet loss function causes the weights to explode and start predicting nan values

Created a triplet-generator while viewing this facenet implementation of triplet loss function. Then combined the model from the python notebook mentioned in this repo. I am training using the vgg2face dataset.
But training fails and stop after just 1 epoch step. The weights seem to be exploding,unable to figure out why. Attached the python notebook which shows the code I used for training
project-cleaned_2.0.zip

So I tried with alpha as follows:
 def triplet_loss_v2(y_true,y_preds):     print()     print("Yo,this is the loss fn,thanks for calling")     print()     reshape_triplets_embeddings=tf.reshape(y_preds,[-1,3,128])     an,pn,nn=tf.split(reshape_triplets_embeddings,3,1)     a=tf.reshape(an,[-1,128])     p=tf.reshape(pn,[-1,128])     n=tf.reshape(nn,[-1,128])     p_dist = K.sum(K.square(a-p), axis=-1)     n_dist = K.sum(K.square(a-n), axis=-1)     return K.sum(K.maximum(p_dist - n_dist+0.2, 0), axis=0)
The result remains the same,network predicts nan values at the same eoch step. changing the dataset incase I had any bad input also gives the same result.
In the face-recognition python notebook attached in this repo, you have added a Triplet loss Layer to a new model(#2) and then trained that model(#2) what I didnt understand was is how will the weights for the model(#1) which generate the embeddings be modified if one is training a model(#2) which only takes the input from embedding generating model(#1)

Assuming you mean by model 1 nn4_small2  and by model 2 nn4_small2_train. Model 2 in this case uses model 1 as shared layer. Take a look at triplet_loss_layer which takes as input the outputs of model 1 (i.e. emb_a, emb_p, emb_n) in this case. So when training model 2 the weights of model 1 will be updated.

What do you mean with with "your custom triplet_generator"? The dummy triplet generator as in https://github.com/krasserm/face-recognition/blob/master/data.py.
What do you mean with "the notebook"? The one you provided or the one in this repo?

Or maybe a more general question: what can I try running that you can't try yourself?
When running the notebook in this repo as is with the dummy triplet generator the loss oscillates around 0.8 over all epochs i.e. no nans are produced.

could you please explain the shared layer thing or put a link that could easily explain this part of the project and is the nn4.small2.v1 trained on the triplet loss function of the datasets? Thank you so much.

@shadymohsen

Are the nn4.small2.v1 weights output of the trained model on the triplet loss function of the datasets or the trained model to get f(x)i

The weights are the same for both models as nn4_small is shared within nn4_small_train and nn4_small_train doesn't add any additional learnable weights.

how can i load the weights of model #2 in model #1?

When you train nn4_small_train with the triplet loss you can immediately use nn4_small for prediction afterwards (i.e. in the same way as nn4_small2_pretrained).

i'm very sorry but i still didn't understand well. so know we have a model nn4_small=createmodel() this is a model which takes the images and get output which is the f(x) right?
another thing is that we have a pre trained model which is trained on the faces of people which is the nn4.small2.v1 this is trained to get the output which is the triplet loss of three images input which used the nn4_small=createmodel(). right?
now how can we upload the weights of the pre trained model which is already trained on three images as an input and the triplet loss as an output to the model which its output is f(x) and has 1 input image.
thank you so much in advance :)

i'm very sorry but i still didn't understand well. so know we have a model nn4_small=createmodel() this is a model which takes the images and get output which is the f(x) right?

Yes. The same is true for nn4_small2_pretrained which the model instance that is used in the remainder of the notebook. nn4_small2 is only there to demonstrate how it can be trained in context of nn4_small2_train.

another thing is that we have a pre trained model which is trained on the faces of people which is the nn4.small2.v1 this is trained to get the output which is the triplet loss of three images input which used the nn4_small=createmodel(). right?

nn4.small2.v1.h5 are only the weights that can be loaded into a model returned by create_model(). The notebook loads it into nn4_small2_pretrained. The training model nn4_small2_train is setup such that it shares the weights with nn4_small. nn4_small2_train itself doesn't add any further weights. Hence, the learned weights can be saved and loaded into a model instance returned by create_model().

now how can we upload the weights of the pre trained model which is already trained on three images as an input and the triplet loss as an output to the model which its output is f(x) and has 1 input image.

See previous answer. Key is that nn4_small2_train and nn4_small2 share weights.

also, when i input my images of my library to the model how does it know the anchor from the positive and the negative?

Only the training model nn4_small_train needs to discriminate them. For that purpose it has three inputs in_a, in_p, in_n.
Hope that helps.

hello I have read your blog . which is really grate
but in the end i cant understand . You pass the image as
with some number 29 and it gives the output as some person .. but whats that number 29 can you plzzz tell

i'm very sorry but i still didn't understand well. so know we have a model nn4_small=createmodel() this is a model which takes the images and get output which is the f(x) right?

Yes. The same is true for nn4_small2_pretrained which the model instance that is used in the remainder of the notebook. nn4_small2 is only there to demonstrate how it can be trained in context of nn4_small2_train.

another thing is that we have a pre trained model which is trained on the faces of people which is the nn4.small2.v1 this is trained to get the output which is the triplet loss of three images input which used the nn4_small=createmodel(). right?

nn4.small2.v1.h5 are only the weights that can be loaded into a model returned by create_model(). The notebook loads it into nn4_small2_pretrained. The training model nn4_small2_train is setup such that it shares the weights with nn4_small. nn4_small2_train itself doesn't add any further weights. Hence, the learned weights can be saved and loaded into a model instance returned by create_model().

now how can we upload the weights of the pre trained model which is already trained on three images as an input and the triplet loss as an output to the model which its output is f(x) and has 1 input image.

See previous answer. Key is that nn4_small2_train and nn4_small2 share weights.

also, when i input my images of my library to the model how does it know the anchor from the positive and the negative?

Only the training model nn4_small_train needs to discriminate them. For that purpose, it has three inputs in_a, in_p, in_n.
Hope that helps.

Gives some kinda error saying weights are of model #2(nn4_small_train)  and cannot be loaded into nn4_small as it is a layer of nn4_small_train. Don't know how to fix it. If you can help
This is the code link:
Code:-  https://github.com/Dhruv2012/Deep-Learning/blob/Align(by-Tess)/facenet/CustomModel/main.py
Else portion refers Training the model and saving the weights.
If portion refers running the model and running it.


Use pubspec_overrides for Dev/CI

For anyone using Dart 2.17+ and for Github actions, this will resolve locally instead of failing builds due to unpublished versions being specified.

It seems that this changes are being added to the source instead of only being added when running GitHub actions. This is not desired. Could you change it so it's only overriden when running the actions?
For local development, it's easy do simply override manually.

Well, I think that its a problem that someone checks out the project and it doesn't work unless they figure out which overrides they have to specify.  With the overrides the project always "just works" and it it is ignored for pub publish, this is exactly the use case it was added to Dart for.

I don't If this is the case, why they don't use dependencies_override/pubspec_overrides on their own packages?

Well the Dart SDK is a bad example, it uses a lot of automation to release its huge amount of packages.
And it is a pretty new Dart 2.17 feature.
But a lot well maintained packages started using it lately, for example freezed: https://github.com/rrousselGit/freezed/blob/master/packages/freezed/pubspec_overrides.yaml

Yeah, I don't think this repository is a good example, but thinking again I don't see a strong argument to not accept this PR.


WARNING: Could not find ssh-agent

Hi
I first had the same error as here : Start-SSHAgent missing in v<IP_ADDRESS>5 #224
I applied the work around sugested by t0yv0 and added the missing exported commands to the posh-git.psd1.
When I re-open the posh prompt I get the following message:
WARNING: Could not find ssh-agent
It seems like this is what also blocks me to push changes to my repository
Environment infos:
Windows 10
$psversion: 5.0.10240.16384
Path: contains
C:\Program Files\git\cmd
C:\Program Files\Git\cmd
downloaded version from PSGallery

It seems like the issue came from a missing path in the $env:path variable.
Executing the following line in my ISE and re opening a fresh powershell prompt solved this issue for me.
$MyPath = $env:Path + ";" + "${env:ProgramFiles}\git\usr\bin"
[Environment]::SetEnvironmentVariable("Path", $($MyPath ), "Machine")

It seems that one workaround was suggested already. I found another solution which seems to be a little bit cleaner for me.
I added git executables into a path
$Script:GitPath = "$env:ProgramFiles\Git"

Add $GitPath\bin and $GitPath\cmd to a PATH environment variable.
Add aliasies for ssh executables
Set-Alias ssh-agent "$GitPath\usr\bin\ssh-agent.exe"
Set-Alias ssh-add "$GitPath\usr\bin\ssh-add.exe"


x/tools/gopls: Extract function not handle anonymous functions correctly

gopls version
v14.2
go env
unimportant

What did you do?
In the following example, select all main function body, then toggle "Extract function"
func main() {
	var s []string
	slices.SortFunc(s, func(a, b string) int {
		return cmp.Compare(a, b)
	})
	println(s)
}

What did you expect to see?
func main() {
	newFunction()
}

func newFunction() {
	var s []string
	slices.SortFunc(s, func(a, b string) int {
		return cmp.Compare(a, b)
	})
	println(s)
}

What did you see instead?
func main() {
	shouldReturn := newFunction()
	if shouldReturn {
		return
	}
}

func newFunction() bool {
	var s []string
	slices.SortFunc(s, func(a, b string) int {
		return true, cmp.Compare(a, b)
	})
	println(s)
	return false
}

Editor and settings
No response
Logs
No response

The bad news is that "extract function" has a lot of serious bugs that make it a use-at-your-own peril sort of feature: notably, it discards all comments in the code that migrates. The good news is that it is a priority for our team in 2024 to improve all of our refactoring tools.

Thanks @golopot for this fix.


Updated readme links for 2.6 docs

Enyo-DCO-1.1-Signed-off-by: Derek Tor<EMAIL_ADDRESS>

Updated domains to reflect enyojs.com domains. I believe these will be the correct ones.

Updated urls to reflect @webOS101 last comments.


Update readme. Step 4

Hi!!
Thanks for sharing the code!!
just a small comment on readme Step 4 I think it need to be update:
yarn test test/polygon/flashloan.test.ts
to
yarn test test/polygon/dodoflash.test.ts
Looks the test/polygon/flashloan.test.ts file don't exist any more.
Thanks

Thank you for pointing it out!


[BUG] VolumeAttachment stuck in detachError when deleting a guest cluster node.

Describe the bug
When deleting a guest cluster node with attached PVCs for any reason (like deleting the node in harvester UI, rancher autoscaler downscaling, or rancher node unhealthy autoreplace), the PVC VolumeAttachments on the guest cluster become stuck in a Removing state. The harvester csi finalizer in the guest cluster is unable to confirm the detachment from the upstream harvester longhorn.  This is specifically a gap in the longhorn manager functionality as it has no mechanism to report the reason for the failed detachment.
The volume is successfully detached in the upstream harvester cluster from the deleted node.  However, the PVC is stuck forever on the guest cluster in this detaching state with the guest cluster harvester csi driver unable to finalize.  This prevents the PVC from being re-attached in the guest cluster until the VolumeAttachment harvester csi finalizer is removed.
This is really quite a show stopper as it locks any stateful deployments from coming up again after their underlying node is removed.
To Reproduce
Steps to reproduce the behavior:

Create a harvester cluster
Create a guest cluster using Rancher with multiple worker nodes
Deploy harvester-csi-driver on the guest cluster
Create a deployment with a PVC using the guest cluster
Delete the guest cluster node in the harvester UI to which the pod in step 4 has a PVC bound to.
Ensure the pod is re-created in the guest cluster on another node
Observe the VolumeAttachment for the PVC from step 4 enter a Removing state with a status.detachError.

Expected behavior
The VolumeAttachment in the guest cluster is successfully removed when a node is deleted in the upstream harvester cluster.
Support bundle
I can generate one if you'd like but I don't believe this is required here.
Environment

Harvester ISO version: 1.2.1
Underlying Infrastructure Described here

Additional context
I've implemented an automated workaround using a prometheusRule to alert on this failure case and robusta to implement an automatic custom action to remediate if anyone else encounters this issue.

Hi @ionfury
Thanks for opening this issue and providing a detailed analysis.
We will also check this issue for both v1.2.1 and v1.3.0 to reproduce and check the latest status.
Related to https://github.com/longhorn/longhorn/issues/1875


Reference returned ID in destroy.

This module is a great workaround, thanks for putting in the work!
I am trying to implement a simple workflow with the Azure CLI to create/destroy resources that have not been implemented in the official Azure terraform provider yet.
Basically I want to perform a post to an API, which returns a JSON.
I.e. to create:
ORGA="my-orga"
NAME="PAT"
az rest
--method post
--uri "https://vssps.dev.azure.com/${ORGA}/_apis/Tokens/Pats?pi-version=7.1-preview.1"
--resource "https://management.core.windows.net/"
--body '{ "displayName": "'$NAME'" , "scope": "Read" }'
--headers Content-Type=application/json
The returned output looks like:
OUTPUT='{
  "patToken": {
    "authorizationId": "71bdc7b3-dbfe-4649-bb85-c03710057e67",
    "displayName": "PAT",
    "scope": "Read",
    "targetAccounts": [
      "a3fbd8cc-4b0e-4a80-93b4-54ee2e30ee94"
    ],
    "token": "g5ffffffffffffffffffffffggggggggggggggggggeeeeeeeeeeeee",
    "validFrom": "2022-04-20T12:27:48.8Z",
    "validTo": "2023-04-20T12:27:48.8Z"
  },
  "

Which I can parse out like:
TOKEN=$(echo $OUTPUT | jq -r .patToken.token)
AUTH_ID=$(echo $OUTPUT | jq -r .patToken.authorizationId)

I need to be able to get back two values here:

The TOKEN, which I need for downstream purposes and can simply return as an output
The AUTH_ID (which I need in order to destroy the token).

The AUTH_ID is the part I need help with. I need access to this ID when calling destroy, which looks something like:
az rest \
--method delete  \
--uri "https://vssps.dev.azure.com/${ORGA}/_apis/tokens/pats?authorizationId=${AUTH_ID}&api-version=7.1-preview.1"  \
--resource "https://management.core.windows.net/"

So I need to persist the AUTH_ID somehow and then use it when calling destroy.

Hi, I'm not quite clear on what you're asking. Are you asking how you would retrieve authorizationId, or how you would use it in the destruction?

How would I persist authorizationId during create and then use it when running destroy?

Unfortunately, that's not possible. Terraform requires destroy provisioners to be defined at the time that a resource is created (see the destroy-time provisioner documentation. This means that you can't modify the destroy command after the resource is created.
To achieve your goal, perhaps you can find some sort of list command or similar that can find the authorizationId at destroy-time, based on tags or something similar?

That's what I was afraid of, thanks for confirming.
I have only an inexact workaround, which is to delete all tokens that have the declared displayName, but since that is not unique it will also delete others. I can live with it though.
It is quite a pity that destroy-time provisioners doesn't support this, since I think this is quite a typical approach when using CLIs (which seem to be a nice candidate for wrapping with a shell resource). I.e., use CLI to send a POST request and create a resource, get the unique id of the resource back in the response. Parse out the ID and use that for all further interactions with the resource.

Since there is no way to deal with this that is supported in Terraform, I'll close this issue.


Calculating $(TargetPlatformDisplayName) takes up 0.4% of evaluation of a solution-wide design-time build

See: https://github.com/dotnet/sdk/issues/1586
This isn't even used in any of the of the project types in the solution that was built.

Pass
File
Line #
Expression
Inc (ms)
Inc (%)
Exc (ms)
Exc (%)
#
Bug

Total Evaluation

6159
100%
54
0.9%

Properties (Pass 1)

3312
53.8%
9
0.1%

ItemDefinitionGroup (Pass 2)

14
0.2%
2
0%

Items (Pass 3)

566
9.2%
24
0.4%

Lazy Items (Pass 3.1)

1826
29.6%
158
2.6%

UsingTasks (Pass 4)

59
1%
59
1%

Targets (Pass 5)

327
5.3%
208
3.4%

Properties (Pass 1)
Microsoft.Common.CurrentVersion.targets
99
<TargetPlatformDisplayName Condition="'$(TargetPlatformDisplayName)' == ''" >$([Microsoft.Build.Util...
25
0.4%
24
0.4%
223

Duping this against https://github.com/Microsoft/msbuild/issues/2519.


Simple animation uses much higher CPU than SwiftUI in iOS

Steps to reproduce
Flutter

run the code with simple continuous scaling animation in profile mode on iPhone
open Xcode and hit "Show the Debug navigator" icon
the tool shows approximate 30% CPU usage

SwiftUI

run the code with simple continuous scaling animation
open Xcode and hit "Show the Debug navigator" icon
the tool shows approximate 1% CPU usage

Code sample
Flutter
Code sample
import 'dart:async';

import 'package:flutter/material.dart';

void main() => runApp(const MaterialApp(home: TapCancelDemo()));

class TapCancelDemo extends StatefulWidget {
  const TapCancelDemo({super.key});

  @override
  State<TapCancelDemo> createState() => _TapCancelDemoState();
}

class _TapCancelDemoState extends State<TapCancelDemo>
    with TickerProviderStateMixin {
  bool _isLarge = true;

  @override
  void initState() {
    super.initState();
    WidgetsBinding.instance.addPostFrameCallback((_) {
      setState(() => _isLarge = false);
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: AnimatedContainer(
          duration: const Duration(seconds: 1),
          onEnd: () {
            setState(() => _isLarge = !_isLarge);
          },
          width: _isLarge ? 200 : 100,
          height: _isLarge ? 200 : 100,
          color: Colors.blue,
        ),
      ),
    );
  }
}

SwiftUI
Code sample
import SwiftUI

struct ContentView: View {
    @State private var isAnimating = false

    var body: some View {
        Rectangle()
            .fill(.blue)
            .frame(width: 100, height: 100)
            .scaleEffect(isAnimating ? 2 : 1.0)
            .onAppear {
                withAnimation(
                    .easeInOut(duration: 1.0)
                    .repeatForever(autoreverses: true)
                ) {
                    isAnimating = true
                }
            }
    }
}

Performance profiling on master channel

[X] The issue still persists on the master channel

Timeline Traces
Timeline Traces JSON
dart_devtools_2024-12-19_10_38_29.942.json.zip

Video demonstration

Video demonstration
[Upload media here]

What target platforms are you seeing this bug on?
iOS
OS/Browser name and version | Device information
iPhone 12 Pro max (OS 18.11)
Does the problem occur on emulator/simulator as well as on physical devices?
Yes
Is the problem only reproducible with Impeller?
No
Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
$ fvm flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.27.0, on macOS 14.3.1 23D60 darwin-arm64, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.95.3)
[✓] VS Code (version 1.61.2)
[✓] Connected device (4 available)
[✓] Network resources

• No issues found!

@chooyan-eng Thanks for the report.
Please check if your case resembles https://github.com/flutter/flutter/issues/142218 or not.

Two notes If you're using Xcode's built in CPU profiler:

You need to run the Flutter applicaiton in release mode. XCode doesn't know what parts of the process are the app and which part is our own profiler/debugger, so you're essentially measuring debugger/profiler performance.

iOS defers some of the animation work to a secondary render server process, and the XCode CPU profiler does not count this workload. The XCode cpu profiler therefore does not give you an apples to apples comparison between a Flutter and SwiftUI/UIKit application as the latter is only partially profiled.


Closing a modal from a cancel button causes unexpected behavior, except Escape key

According to the official doc ... article, I can close a modal from a button click with the following:
<button wire:click="$emit('closeModal')">Close Modal</button>
After clicking on the button, the real-time validation seems to trigger anyway and yes the modal closes. After being closed, my livewire datatable stops working.
However, (I reload the page to open again the modal) when I hit the "Escape" key to close the modal, everything is working fine after the modal is closed.
Maybe there's a bug out there?
Otherwise, is there a simple or neat way to close the modal right from clicking a "cancel" button?
Meanwhile, I have just left a help text "press escape to cancel".

Add type="button" to your button element, otherwise, it will trigger the form submit.

Thanks. I have added that type="button". However, the issue still persists.
I have implemented a very simple livewire datatable. Again, after having opened the modal, when I press the "Esc" key, the livewire datatable still works as normal.
However, if I click on the "Cancel" button either these ways:
1st way:
<button type="button" wire:click="$emit('closeModal')" class="...">Cancel</button>
2nd way:
<button type="button" wire:click="closeTheModal" class="...">Cancel</button>

    public function closeTheModal()
    {
        dd('Indeed, I do want to close the modal by cancelling ....')
        $this->closeModal();
    }

My datatable stops working.
Debugging errors.
Actually, when I open the modal, there's this message in the console log:

Uncaught TypeError: e.$refs[t] is undefined

Then, when I hit on the "Cancel" button, other error shows up (that causes the livewire datatable stop working):

Uncaught (in promise)
Uncaught (in promise) TypeError: l is not a function

What am I missing? How do I fix it?

I'm unable to replicate this issue. If you can provide a repository with a fresh install of Laravel and the dependencies above and replicating the issue I can take a further look.

I'm unable to replicate this issue. If you can provide a repository with a fresh install of Laravel and the dependencies above and replicating the issue I can take a further look.

There's something strange, even with the simplest table and Modal without using packages I get the same error. There must be something wrong now with the combination of latest npm livewire and alpineJS ... The error actually is reported by alpineJS.
I'll try again as soon as Laravel 9 is released...

I am facing this issue too. Everything was fine until i upgraded composer packages. I am starting to think this could be a livewire issue.

Try downgrading alpinejs to a version from 3.2.0 or below. Worked for me.


Payment Fields in query results need documentation

The following field is in the output of the payment data query results. However, they are not documented. Can you please include the definition for these fields in the documentation?
CurrencyCode
Docs: https://docs.microsoft.com/en-us/windows/uwp/publish/payout-summary#payments
https://apidocs.microsoft.com/services/partnerpayouts

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 65a40199-aaff-a750-b2df-cc8533606f0a
Version Independent ID: 76160f31-f715-009a-e73e-202f8ed27006
Content: Payout reports - UWP applications
Content Source: windows-apps-src/publish/payout-summary.md
Product: uwp
Technology: publish
GitHub Login: @hickeys
Microsoft Alias: hickeys

This item was added in an earlier update as part of a transition to a new report format. Accordingly, I'm closing this issue. Thank you for reporting the discrepancy.


Translate README.md to different languages

Like in project https://github.com/good-first-issues/good-first-issues.github.io

@Jetas18 Hello!
You already have experience translating REDME.md files into several languages (https://github.com/good-first-issues/good-first-issues.github.io/issues/33).
If you have the desire and time, I can assign this issue to you ;)

Well I can only translate it to hindi

@KrshnK Hello!
Some time ago you translated the README.md file into 70+ languages as part of PR https://github.com/good-first-issues/good-first-issues.github.io/pull/77
Do you have the opportunity to make similar translations for this project?

@KrshnK Hello!
Some time ago you translated the README.md file into 70+ languages as part of PR good-first-issues/good-first-issues.github.io#77
Do you have the opportunity to make similar translations for this project?

@gomzyakov  Sure, I'd be happy to help with translations for your project. Could you provide more details and specify the languages you need? Thanks!


ingress 404 not found

@amorey I switched to the new helm repo addr and upgrade to the latest chart and version. Unfortunately, when I navigate to my ingress host I am getting 404 page not found.
Below is my full values.yaml:
kubetail:
  podTemplate:
    resources:
      requests:
        cpu: 250m
        memory: 64Mi
      limits:
        cpu: 1
  ingress:
    enabled: true
    annotations:
      traefik.ingress.kubernetes.io/router.entrypoints: websecure
      traefik.ingress.kubernetes.io/router.tls: "true"
      traefik.ingress.kubernetes.io/router.tls.certresolver: le
    rules:
    - host: kubetail.mydomain.com
      paths:
      - path: /
        pathType: Prefix
    tls:
    - hosts:
      - kubetail.mydomain.com

Sorry about the 404 issue. Here are the old/new Ingress manifests:
Old:
# Source: kubetail/templates/ingress.yaml
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: kubetail
  annotations:
    traefik.ingress.kubernetes.io/router.entrypoints: websecure
    traefik.ingress.kubernetes.io/router.tls: "true"
    traefik.ingress.kubernetes.io/router.tls.certresolver: le
spec:
  rules:
  - host: kubetail.mydomain.com
    http:
      paths:
      - path: /
        pathType: Prefix
        backend:
          service:
            name: kubetail
            port:
              number: 4000
  tls:
  - hosts:
    - kubetail.mydomain.com
    secretName: kubetail

New:
# Source: kubetail/templates/ingress.yaml
kind: Ingress
apiVersion: networking.k8s.io/v1
metadata:
  name: kubetail
  namespace: kubetail
  labels:
    helm.sh/chart: kubetail-0.4.4
    app.kubernetes.io/name: kubetail
    app.kubernetes.io/instance: kubetail
    app.kubernetes.io/version: "0.4.6"
    app.kubernetes.io/managed-by: Helm
  annotations:
    traefik.ingress.kubernetes.io/router.entrypoints: websecure
    traefik.ingress.kubernetes.io/router.tls: "true"
    traefik.ingress.kubernetes.io/router.tls.certresolver: le
spec:
  rules:
  - host: kubetail.mydomain.com
    paths:
    - path: /
      pathType: Prefix
  tls:
  - hosts:
    - kubetail.mydomain.com

The main differences are:

the new resource is namespaced
the new resource is missing the secretName attribute under tls settings

The new chart creates a Secret resource named kubetail for config secrets so it's possible that the new resource might have overwritten your old resource that you were using for the Ingress's tls secret. Try creating a tls secret and adding a reference to it in your values.yaml file (e.g. kubetail-cert):
kubetail:
  podTemplate:
    resources:
      requests:
        cpu: 250m
        memory: 64Mi
      limits:
        cpu: 1
  ingress:
    enabled: true
    annotations:
      traefik.ingress.kubernetes.io/router.entrypoints: websecure
      traefik.ingress.kubernetes.io/router.tls: "true"
      traefik.ingress.kubernetes.io/router.tls.certresolver: le
    rules:
    - host: kubetail.mydomain.com
      paths:
      - path: /
        pathType: Prefix
    tls:
    - hosts:
      - kubetail.mydomain.com
      secretName: kubetail-cert

Let me know if that helps.

Adding secretName did not help. The SSL certificate is valid and being created by Let's Encrypt, but it seems to be a path issue. Still getting 404 not found when hitting the host https://kubetail.mydomain.com.
$ kubectl describe ingress kubetail
Name:             kubetail
Labels:           app.kubernetes.io/instance=kubetail
                  app.kubernetes.io/managed-by=Helm
                  app.kubernetes.io/name=kubetail
                  app.kubernetes.io/version=0.4.6
                  helm.sh/chart=kubetail-0.4.4
Namespace:        default
Address:
Ingress Class:    traefik
Default backend:  <default>
TLS:
  kubetail-cert terminates kubetail.mydomain.com
Rules:
  Host        Path  Backends
  ----        ----  --------
  *           *     <default>
Annotations:  meta.helm.sh/release-name: kubetail
              meta.helm.sh/release-namespace: default
              traefik.ingress.kubernetes.io/router.entrypoints: websecure
              traefik.ingress.kubernetes.io/router.tls: true
              traefik.ingress.kubernetes.io/router.tls.certresolver: le
Events:       <none>

Sorry, I'm just seeing it now but the other thing that's missing from the new Ingress resource is the backend declaration under rules. Try this for your values.yaml file (note that I replaced the port number with a name and kept secretName kubernetes-cert):
kubetail:
  podTemplate:
    resources:
      requests:
        cpu: 250m
        memory: 64Mi
      limits:
        cpu: 1
  ingress:
    enabled: true
    annotations:
      traefik.ingress.kubernetes.io/router.entrypoints: websecure
      traefik.ingress.kubernetes.io/router.tls: "true"
      traefik.ingress.kubernetes.io/router.tls.certresolver: le
    rules:
    - host: kubetail.mydomain.com
      paths:
      - path: /
        pathType: Prefix
        backend:
          service:
            name: kubetail
            port:
              name: kubetail
    tls:
    - hosts:
      - kubetail.mydomain.com
      secretName: kubetail-cert

still the same 404 page not found. Here are describes on both the service and ingress. This used to work, so something changed.
$ kubectl describe service kubetail
Name:              kubetail
Namespace:         default
Labels:            app.kubernetes.io/instance=kubetail
                   app.kubernetes.io/managed-by=Helm
                   app.kubernetes.io/name=kubetail
                   app.kubernetes.io/version=0.4.6
                   helm.sh/chart=kubetail-0.4.4
Annotations:       meta.helm.sh/release-name: kubetail
                   meta.helm.sh/release-namespace: default
Selector:          app.kubernetes.io/instance=kubetail,app.kubernetes.io/name=kubetail
Type:              ClusterIP
IP Family Policy:  SingleStack
IP Families:       IPv4
IP:                <IP_ADDRESS>
IPs:               <IP_ADDRESS>
Port:              kubetail  80/TCP
TargetPort:        80/TCP
Endpoints:         <IP_ADDRESS>:80
Session Affinity:  None
Events:            <none>

$ kubectl describe ingress kubetail
Name:             kubetail
Labels:           app.kubernetes.io/instance=kubetail
                  app.kubernetes.io/managed-by=Helm
                  app.kubernetes.io/name=kubetail
                  app.kubernetes.io/version=0.4.6
                  helm.sh/chart=kubetail-0.4.4
Namespace:        default
Address:
Ingress Class:    traefik
Default backend:  <default>
TLS:
  kubetail-cert terminates kubetail.mecasa.dev
Rules:
  Host        Path  Backends
  ----        ----  --------
  *           *     <default>
Annotations:  meta.helm.sh/release-name: kubetail
              meta.helm.sh/release-namespace: default
              traefik.ingress.kubernetes.io/router.entrypoints: websecure
              traefik.ingress.kubernetes.io/router.tls: true
              traefik.ingress.kubernetes.io/router.tls.certresolver: le
Events:       <none>

Ok, I'll try something else. In the mean time, can you tear down the chart and install it again? The ingress description lists <default> as the backend but I would have expected it to show the kubetail service as the backend.

Also, I just noticed that paths should be nested under http. Try this:
kubetail:
  podTemplate:
    resources:
      requests:
        cpu: 250m
        memory: 64Mi
      limits:
        cpu: 1
  ingress:
    enabled: true
    annotations:
      traefik.ingress.kubernetes.io/router.entrypoints: websecure
      traefik.ingress.kubernetes.io/router.tls: "true"
      traefik.ingress.kubernetes.io/router.tls.certresolver: le
    rules:
    - host: kubetail.mydomain.com
      http:
        paths:
        - path: /
          pathType: Prefix
          backend:
            service:
              name: kubetail
              port:
                name: kubetail
    tls:
    - hosts:
      - kubetail.mydomain.com
      secretName: kubetail-cert

@amorey still getting 404 page not found. Let me know what else you need to help debug this.

Sorry about that and thanks for your patience. 🙏
I think I figured it out... the last change fixed the issue with the Ingress but there was a bug in the Service that was causing the Bad Gateway error (the targetPort was missing from the Service template). I fixed that bug in the latest chart and also re-implemented the Ingress backend definition logic from your original template so you no longer have to define it in your values file.
Try upgrading to the latest chart (0.4.5) and using these values:
kubetail:
  podTemplate:
    resources:
      requests:
        cpu: 250m
        memory: 64Mi
      limits:
        cpu: 1
  ingress:
    enabled: true
    annotations:
      traefik.ingress.kubernetes.io/router.entrypoints: websecure
      traefik.ingress.kubernetes.io/router.tls: "true"
      traefik.ingress.kubernetes.io/router.tls.certresolver: le
    rules:
    - host: kubetail.mydomain.com
      http:
        paths:
        - path: /
          pathType: Prefix
    tls:
    - hosts:
      - kubetail.mydomain.com
      secretName: kubetail-cert

I think that should work. 🤞 Let me know!

@amorey great success that worked. Thanks for all the help. 🙏🏻

Awesome! Thanks for your help debugging it!

@nodesocket Just a quick update - I released a new version yesterday (0.7.0) that gives you a birds eye view of log sizes and event frequencies across the entire cluster. Here's a live demo:
https://demo.kubetail.com
I had to make some changes to the values.yaml schema so you'll need to update your values.yaml file to upgrade. Let me know if you need help or if you run into trouble.

Hi @amorey. Awesome to see progress of kubetail progressing.
What are the breaking changes? Here is my current values.yaml:
kubetail:
  podTemplate:
    resources:
      requests:
        cpu: 250m
        memory: 64Mi
      limits:
        cpu: 1
  ingress:
    enabled: true
    annotations:
      traefik.ingress.kubernetes.io/router.entrypoints: websecure
      traefik.ingress.kubernetes.io/router.tls: "true"
      traefik.ingress.kubernetes.io/router.tls.certresolver: le
    rules:
    - host: kubetail.mydomain.com
      http:
        paths:
        - path: /
          pathType: Prefix
    tls:
    - hosts:
      - kubetail.mydomain.com
      secretName: kubetail-cert

Actually... I just noticed a small bug in the Ingress template. I will have a fix out in 5 min.

Ok, it's fixed in 0.7.1. Let me know if you run into any problems.

@nodesocket Have you had a chance to upgrade Kubetail? Let me know if you run into any problems.

@amorey I have not, hope to get around to it later this week. Thanks for the follow up

No problem, let me know if anything comes up.

@amorey decided to be productive and just upgrade. Everything looks good. 👍🏻


Change in Ionic2 RC3 for Gesture Controller

As per #14 Ionic2 RC3 has changed the way the Gesture Controller works.

Thanks for the PR ! :+1:


Incorrect API key provided: undefined.

Just keep getting 'Incorrect API key provided: undefined. You can find your API key at https://platform.openai.com/account/api-keys.'
Have just created an account with openAI and 2 API keys, but both have resulted in the above error.
in the .env file I have tried
REACT_APP_OPENAI_KEY = 'sk-8dL......' & REACT_APP_OPENAI_KEY = sk-8dL......
Any help to get this running is much appreciated.

Was being a numpty .env file needed to be inside the client folder not root - d'oh


feature request: provide an optional {{reference}} modifier

Currently, the this._referenceElement always refers to the trigger element, but in practice, some cases do not fit in this assumption.
For example, consider this UI pattern:

The Trigger is the little downward triangle (with a light gray hovering background)
The entire button surrounded with dark gray border is the Reference
The Popover is the menu

Users can only show the popover menu by clicking the downward triangle precisely, but not the entire button because the whole button has a different interactive meaning here (that is, clicking to select/unselect)
However, we still need the entire button to be the reference element because we need to align the popover menu with the left side of the button as the whole, not the downward angle.
If developers can specify the reference element optionally, they can archive this pattern easily and quickly.
In our application, we use ember-ref-bucket to hold a reference element and pass it to the popperjs modifier (we made our popperjs modifier before, but we would like to use this addon instead).
Maybe a better way is to provide an optional {{reference}} modifier, so developers can use it to assign a custom element instead of always as the same as the trigger element.

for your example, I think that would look like this with ember-popperjs:
<PopperJS @placement="bottom-start" as |reference popover|>

    Some Text

    <Menu >
      <:trigger>
        {{! this button is the trigger element }}
        <Button />
      </:trigger>

      <:content as |state|>
        <List {{popover}}>
          {{! here is the menu popover }}
        </List>
      </:content>
    </Menu>

</PopperJS>

Sort of, only if ember-popperjs provides the {{reference}} modifier, we don't need the <Menu /> component anymore, since it also wraps the popper.js.

that's what's already provided. The thing that this addon won't do though is providing a way to manage hide/show of the popover -- too many ways to do that (or animate it! <3)
thanks for the discussion! let me know if anything is still unclear.


test: up test cover perrcent

Codecov Report

Merging #19 into master will decrease coverage by 0.39%.
The diff coverage is 94.73%.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   67.34%   66.95%   -0.40%
==========================================
  Files          33       32       -1
  Lines        2055     1979      -76
  Branches      499      482      -17
==========================================
- Hits         1384     1325      -59
+ Misses        617      600      -17
  Partials       54       54

Impacted Files
Coverage Δ

packages/command-core/src/index.ts
100.00% <ø> (ø)

packages/command-core/src/npm.ts
16.66% <ø> (-0.73%)
:arrow_down:

packages/command-core/src/plugin.ts
19.51% <0.00%> (-5.49%)
:arrow_down:

packages/command-core/src/cli.ts
100.00% <100.00%> (+90.32%)
:arrow_up:

packages/command-core/src/core.ts
64.82% <100.00%> (-17.33%)
:arrow_down:

packages/serverless-spec-builder/src/index.ts
66.66% <100.00%> (ø)

packages/command-core/src/lock.ts
20.00% <0.00%> (-77.15%)
:arrow_down:

packages/command-core/src/errorMap.ts
43.75% <0.00%> (-56.25%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6dfe442...18be6d9. Read the comment docs.


Ge price history trimming

Could significant outliers be removed as it really messes with the scaling.

In the next update I'm trying a different (weekly average) price history which should fix this.


[Paper] GenCast Diffusion model for weather forecasting

Arxiv/Blog/Paper Link
https://arxiv.org/abs/2312.15796
Detailed Description
A diffusion-based approach to weather forecasting, that is quite stable autoregressively. Probably fits in more with diffusion_weather but here to be with the other weather papers. They also say the model is based on GraphCast, but with a different graph connectivity, and a sparse transformer instead of the Processor GNN in GraphCast.
Context
Cool way of doing ensemble predictions, scalable, only at a 1 degree resolution though, 12 hour timesteps as well, so less than Graphcast and the like for some reason?

The encoder and decoder are the same as GraphCast, but the latent grid is a 5-refined mesh, not a multi-mesh, with 10242 nodes and 61440 edges.

I think this adds more support to modularizing graph_weather, what is being done in #76, so that it is easier to experiment with/replicate this kind of result

They train on the 12 hour timestep to be in different data assimilation windows, as ERA5 only has 2 a day.

Overall, really impressive results I think. Interesting combination of graph and diffusion model. A lot slower to run and train because of how diffusion models work, and still requires NWP analysis field for initialization. Compared to a few seconds for a 0.25 degree forecast with GraphCast, 1min per forecast with GenCast at 1 degree is a lot slower. But the better results, and it still be a lot faster than traditional methods make it quite interesting.

Would be really keen to implement this here.

Seems like an interesting project. Is this open as a GSOC project? And if so, what would the scope/length be? (I don't see GraphCast in the repo, so I imagine that would also need to be ported over from DM ?)

Yes, this could work as a GSoC project. It would be a large project (350h). GraphCast wouldn't need to be ported over, we already have the encoder/decoder graph networks implemented, although they definitely can be improved! It would be more the diffusion model that would need to be added. And some changes to make the code more modular, so that we can easily swap out Encoders/Processor/Decoders easily.

They have released an updated paper now, increasing the resolution to 0.25 degrees, and more comparison against ENS, which they claim to beat in 97 percent of the time.

This is very cool! Also, it seems that you don't need to restart training from scratch to go from 1 degree to 0.25: just a few small modifications and fine-tuning are enough, similar to graph weather!

May I have the weather in Saint Pryve Saint Mesmin


Added Basic Testcases

Description
Added Testcases for Views and Models
Related issues
Fixes #21
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist

[ ] I have read and understood the rules about how to Contribute to this project
[ ] The pull request is for the branch dev
[ ] The tests gave 0 errors.
[ ] Linters (Black, Flake, Isort) gave 0 errors. If you have correctly installed pre-commit, it does these checks and adjustments on your behalf.
[ ] The commits were squashed into a single one (optional, they will be squashed anyway by the maintainer)

Important Rules

If your changes decrease the overall tests coverage (you will know after the Codecov CI job is done), you should add the required tests to fix the problem
Everytime you make changes to the PR and you think the work is done, you should explicitly ask for a review

First batch of tests makes sense,
we also need to add those tests in the CI (pull_request_automation.yml). See IntelOwl for reference

From what i can figure out is that intelOwl has tests running through a shell script inside a docker container. Is this what you want? I think #23 will fail the docker build.

yes, as already mentioned in the issue #21. This could also be a thing for another PR if you want to split the work. The important thing here is to run in the CI at least the new tests that you have created

Cool then you can merge this PR and i'll create a new for automating it with CI

I meant to have the CI in this PR. Otherwise how can we test if the tests work?

@mlodic
i’m kind of stucked. I successfully got the containers running but the workflow gets stucked in greedybear_uwsgi | [uWSGI] getting INI configuration from /etc/uwsgi/sites/greedybear.ini
can you please review it and point out what wrong i’m doing.
For the time being i’ve commented out the ngnix and trying to get the django container up without it. will include ngnix once django’s container works fine

use -d so it goes on the background :

use -d so it goes on the background :P

Ohh crap how did i forgot to run it in detach mode ;(*
It worked fine after that.
But now while adding up ngnix i'm facing another roadblock.
ERROR: for greedybear_uwsgi  Cannot start service uwsgi: OCI runtime create failed: container_linux.go:380: starting container process caused: exec: "./docker/entrypoint_uwsgi.sh": stat ./docker/entrypoint_uwsgi.sh: no such file or directory: unknown

ERROR: for uwsgi  Cannot start service uwsgi: OCI runtime create failed: container_linux.go:380: starting container process caused: exec: "./docker/entrypoint_uwsgi.sh": stat ./docker/entrypoint_uwsgi.sh: no such file or directory: unknown
Encountered errors while bringing up the project.

It says that docker/entrypoint_uwsgi.sh does not exist but it does. i'm unable to understand this. can you please point out the mistake. Sorry in advance if its a silly mistake like the previous one

use -d so it goes on the background :P

Ohh crap how did i forgot to run it in detach mode ;(* It worked fine after that.
But now while adding up ngnix i'm facing another roadblock.
ERROR: for greedybear_uwsgi  Cannot start service uwsgi: OCI runtime create failed: container_linux.go:380: starting container process caused: exec: "./docker/entrypoint_uwsgi.sh": stat ./docker/entrypoint_uwsgi.sh: no such file or directory: unknown

ERROR: for uwsgi  Cannot start service uwsgi: OCI runtime create failed: container_linux.go:380: starting container process caused: exec: "./docker/entrypoint_uwsgi.sh": stat ./docker/entrypoint_uwsgi.sh: no such file or directory: unknown
Encountered errors while bringing up the project.

It says that docker/entrypoint_uwsgi.sh does not exist but it does. i'm unable to understand this. can you please point out the mistake. Sorry in advance if its a silly mistake like the previous one

try to make some tests locally with the same commands to get which can be the error

Hey @mlodic
I've been trying to resolve the above mentioned issue since yesterday. Ran it locally and got the same thing tried a lot of things for my end as well as from internet as well but all in vain. Can you please review it.
Thanks!

Seems like it is good to go

great! we are done!


Add @dotnet-bot to prefix of "retest [job] please"

While this actually is not needed, it reduces confusion
for community members, whom in the past have throught
the comment was directed at them.

Thanks. I'll take this change and then later update this to be configurable.


Make api deserialisation O(log(n)) instead of O(n^?)

Fixed in 030db2e.
Previous osmApiElement to osmElement conversion:
Run 1: 1,860ms
Run 2: 1,890ms
Avg: 1875ms
New version:
Run 1: 44ms
Run 2: 40ms
Avg: 42ms
So yeah, 44.6 times faster. Pretty good I'd say.


Is it possible to configure bearer_token/bearer_token_file in encrypted form?

Need to configure the K8s cluster metric scraping in Prometheus running outside the cluster... For this am using 'kubernetes_sd_config' configuration...
From the Prometheus guide looks like the bearer_token/bearer_token_file supports only bearer tokens in plain text format...
https://prometheus.io/docs/prometheus/latest/configuration/configuration/#remote_read
As K8s serviceaccount tokens are valid forever, if token is compromised will be a major security concern...
Apart from the restricted file permissions for the token file, is there any way to avoid the plain text access_token in bearer_token/bearer_token_file fields...

There's not, and this is how Kubernetes provides tokens in any case.
It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

Ok... Thanks...


1806-android-chrome-color

Coverage remained the same at 74.197% when pulling 4e4086ef0df3d4a56de863ec3289ef07990aeb46 on 1806-android-chrome-color into dbf02310b71ad9f904923b6ca0c042d04f175af6 on master.


Validate failed: oneOf

What version of Ajv are you using? Does the issue happen if you use the latest version?
8.11.0
Ajv options object

{
    useDefaults: true,
    coerceTypes: 'array',
    allErrors: true,
    removeAdditional: true,
}

JSON Schema

{
  "oneOf": [
    { "type": "null" },
    { "type": "integer" }
  ]
}

Sample data

null

Your code

            const ajv = new Ajv({
                useDefaults: true,
                coerceTypes: 'array',
                allErrors: true,
                removeAdditional: true,
            })
            const validate = ajv.compile({
                oneOf: [{ type: 'null' }, { type: 'integer' }],
            })
            validate(null) // false

Validation result, data AFTER validation, error messages
//valid: false
[
  {
    instancePath: '',
    schemaPath: '#/oneOf',
    keyword: 'oneOf',
    params: { passingSchemas: [Array] },
    message: 'must match exactly one schema in oneOf'
  }
]

What results did you expect?
The result should pass, if you remove the coerceTypes flag, then everything works fine
Are you going to resolve the issue?
Not Sure

This is not a bug, this is how coerceTypes option works - as you wrote, removing it fixed the problem.
As a result of coercion your data matches both schemas, that’s why it fails validation.
In this case you should use type: [“null”, “integer”] instead of oneOf, while they are equivalent without type coercion, with coercion they are not (and in any case the former is much more efficient).

@epoberezkin
Thank you for your reply and your time

you're welcome!

What happens when a value has a complex type as well?
value: {
  type: ["string", "boolean", "number"],
  nullable: true
}

value can have string[] as well.
How to resolve it in a valid schema?


500 Internal Server Error when deleting apps in dashborad

bug, latest UI on rancher:v2.5-cb8a1755ccd1ed7186c24e582e6fc91fa06f87ec-head
500 Internal Server Error is returned when trying to delete an app

The response is EOF

Closing this issue in favor of this one - https://github.com/rancher/dashboard/issues/1234


I am struggling a lot to launch share link in flutter_inappwebview plugin

Environment
Flutter version: 1.17.1
Plugin version: flutter_inappwebview 3.2.0
Android Version: 10
Description
I am trying to convert a website into webview it has two share links

Directly share in Whatsapp
Second it shows multiple share options

It is working perfectly in mobile version chrome browser but using flutter_inappwebview and url_launcher that functionality is not working. Can you please suggest. I am attaching code gist
https://gist.github.com/Rupak-66/73778886f95561132ed5d7943854dbd0

@pichillilorenzo can you please suggest something

You are not returning the right value to the shouldOverrideUrlLoading event. As it is written in the API docs, you should return a ShouldOverrideUrlLoadingAction.
For example, using your example, you can simply return ShouldOverrideUrlLoadingAction.ALLOW; (I don't know, it depends on your use case):
import 'dart:async';
import 'package:flutter/material.dart';
import 'package:flutter_inappwebview/flutter_inappwebview.dart';
import 'package:url_launcher/url_launcher.dart';

Future main() async {
  runApp(new MyApp());
}

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => new _MyAppState();
}

class _MyAppState extends State<MyApp> {
  InAppWebViewController webView;
  String url = "";
  double progress = 0;

  @override
  void initState() {
    super.initState();
  }

  @override
  void dispose() {
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: SafeArea(
          child: Container(
            child: Column(children: <Widget>[
              Expanded(
                child: Container(
                  child: InAppWebView(
                    initialUrl: "https://www.onestare.com/",
                    shouldOverrideUrlLoading:
                        (InAppWebViewController controller,
                            ShouldOverrideUrlLoadingRequest
                                shouldOverrideUrlLoadingRequest) {
                      print("****"+shouldOverrideUrlLoadingRequest.url);

                      _launchURL(shouldOverrideUrlLoadingRequest.url);
		     return ShouldOverrideUrlLoadingAction.ALLOW;
                    },
                    initialOptions: InAppWebViewGroupOptions(
                      crossPlatform: InAppWebViewOptions(
                        useShouldOverrideUrlLoading: true,
                        debuggingEnabled: true,
                      ),
                    ),
                    onWebViewCreated: (InAppWebViewController controller) {
                      webView = controller;
                    },
                    onLoadStart:
                        (InAppWebViewController controller, String url) {},
                    onLoadStop:
                        (InAppWebViewController controller, String url) {},
                  ),
                ),
              ),
            ]),
          ),
        ),
      ),
    );
  }
}

_launchURL(String url) async {
  if (await canLaunch(url)) {
    await launch(url);
  } else {
    throw 'Could not launch $url';
  }
}


Trailing white space fixes

Describe problem solved by the proposed pull request
This PR just fixes up a number of trailing whitespace instances that git complains about.
Let me know if you have any questions on this PR.
-Mark

Looks good to me. I'm not sure I'd bother with the doxygen config though, it's generated by the wizard.


CFP: Support for TFTP protocol with NodePort service

Cilium Feature Proposal
Thanks for taking time to make a feature request for Cilium! If you have usage questions, please try the slack channel and see the FAQ first.
Is your feature request related to a problem?

Yes
If so, please describe the problem
NodePort service exposed for a pod running TFTP server in Kubernetes cluster (CNI: Cilium) does not always work. It works only if external traffic lands on the node running the pod. If external traffic lands on a node not running the pod, clients do not get a response.

Describe the feature you'd like

Protocols like TFTP ( FTP, H.323, SIP etc) use different ports for control and data traffic. In my case, TFTP server listens on UDP port 69 for control traffic, but uses a random higher number port to send data.
For exposing these services to external client (i.e. not within k8s cluster), simplest way is to go for NodePort.
The iptables based kube-proxy implementation handles these protocols by conntrack handlers (e.g. in my case: nf_conntrack_tftp and nf_nat_tftp).
When using cilium to strictly replace kube-proxy and exposing TFTP server via NodePort, there are two possibilities:

External traffic lands on a node which runs TFTP server pod. In this case, pod sees real IP address of TFTP client and client gets TFTP data traffic.
External traffic lands on a node which does not run TFTP server pod. The traffic then jumps to a remote node where the pod runs. Pod does not see real IP address of external TFTP client (as it is now sourced from cilium_host of node that initially received traffic). The problem is TFTP data traffic does not reach client.

Include any specific requirements you need
(Optional) Describe your proposed solution

Other CNI plugins which use kube-proxy (e.g. Calico) are working perfectly fine when conntrack helpers are used. Maybe Cilium's conntrack implementation should also be enhanced for TFTP (and similar protocols).

Please complete this section if you have ideas / suggestions on how to implement the feature. We strongly recommend discussing your approach with Cilium committers before spending lots of time implementing a change.
For longer proposals, you are welcome to link to an external doc (e.g. a Google doc). We have a Cilium Feature Proposal template to help you structure your proposal - if you would like to use it, please make a copy and ensure it's publicly visible, and then add the link here.

External Client IP: <IP_ADDRESS>
Pod IP: <IP_ADDRESS>
Node-1 IP:  <IP_ADDRESS>, cilium_host: <IP_ADDRESS>, tftp pod is running here
Node-2 IP:  <IP_ADDRESS>, cilium_host: <IP_ADDRESS>, tftp pod is not running here
NodePort: 69
ContainerPort:69
Case 1: When client connects to NodeIP:NodePort (<IP_ADDRESS>:69) where the TFTP pod is running
Conntrack entries on cilium-agent from the Node-1 (where the TFTP pod is running):
Ethernet	{Contents=[..14..] Payload=[..54..] SrcMAC=36:dd:54:6f:62:07 DstMAC=6a:ae:e1:ba:71:6e EthernetType=IPv4 Length=0}
IPv4	{Contents=[..20..] Payload=[..28..] Version=4 IHL=5 TOS=0 Length=48 Id=51911 Flags=DF FragOffset=0 TTL=62 Protocol=UDP Checksum=14061 SrcIP=<IP_ADDRESS> DstIP=<IP_ADDRESS> Options=[] Padding=[]}
UDP	{Contents=[..8..] Payload=[..20..] SrcPort=52539 DstPort=69(tftp) Length=28 Checksum=46317}
Failed to decode layer: No decoder for layer type Payload
CPU 01: MARK 0x328f0a14 FROM 2929 to-endpoint: 62 bytes (62 captured), state established, interface lxc7a021907a17f, , identity world->4347, orig-ip <IP_ADDRESS>, to endpoint 2929
Ethernet	{Contents=[..14..] Payload=[..118..] SrcMAC=6a:ae:e1:ba:71:6e DstMAC=36:dd:54:6f:62:07 EthernetType=IPv4 Length=0}
IPv4	{Contents=[..20..] Payload=[..98..] Version=4 IHL=5 TOS=0 Length=544 Id=58181 Flags=DF FragOffset=0 TTL=63 Protocol=UDP Checksum=7039 SrcIP=<IP_ADDRESS> DstIP=<IP_ADDRESS> Options=[] Padding=[]}
UDP	{Contents=[..8..] Payload=[..90..] SrcPort=60424 DstPort=52539 Length=524 Checksum=15654}
Packet has been truncated
Failed to decode layer: No decoder for layer type Payload
CPU 01: MARK 0xfb536bf0 FROM 2929 to-stack: 558 bytes (128 captured), state new, , identity 4347->world, orig-ip <IP_ADDRESS>
Ethernet	{Contents=[..14..] Payload=[..38..] SrcMAC=36:dd:54:6f:62:07 DstMAC=6a:ae:e1:ba:71:6e EthernetType=IPv4 Length=0}
IPv4	{Contents=[..20..] Payload=[..12..] Version=4 IHL=5 TOS=0 Length=32 Id=51916 Flags=DF FragOffset=0 TTL=62 Protocol=UDP Checksum=14072 SrcIP=<IP_ADDRESS> DstIP=<IP_ADDRESS> Options=[] Padding=[]}
UDP	{Contents=[..8..] Payload=[0, 4, 0, 1] SrcPort=52539 DstPort=60424 Length=12 Checksum=2948}
Failed to decode layer: No decoder for layer type Payload
CPU 01: MARK 0x328f0a14 FROM 2929 to-endpoint: 46 bytes (46 captured), state reply, interface lxc7a021907a17f, , identity world->4347, orig-ip <IP_ADDRESS>, to endpoint 2929
Case 2: When client connects to NodeIP:NodePort (<IP_ADDRESS>:69) where the TFTP pod is not running:
Conntrack entries on cilium-agent from the Node-1 (where the TFTP pod is running):
Ethernet	{Contents=[..14..] Payload=[..54..] SrcMAC=36:dd:54:6f:62:07 DstMAC=6a:ae:e1:ba:71:6e EthernetType=IPv4 Length=0}
IPv4	{Contents=[..20..] Payload=[..28..] Version=4 IHL=5 TOS=0 Length=48 Id=36242 Flags=DF FragOffset=0 TTL=63 Protocol=UDP Checksum=10860 SrcIP=<IP_ADDRESS> DstIP=<IP_ADDRESS> Options=[] Padding=[]}
UDP	{Contents=[..8..] Payload=[..20..] SrcPort=58585 DstPort=69(tftp) Length=28 Checksum=21657}
Failed to decode layer: No decoder for layer type Payload
CPU 01: MARK 0x0 FROM 2929 to-endpoint: 62 bytes (62 captured), state established, interface lxc7a021907a17f, , identity remote-node->4347, orig-ip <IP_ADDRESS>, to endpoint 2929
Ethernet	{Contents=[..14..] Payload=[..118..] SrcMAC=6a:ae:e1:ba:71:6e DstMAC=36:dd:54:6f:62:07 EthernetType=IPv4 Length=0}
IPv4	{Contents=[..20..] Payload=[..98..] Version=4 IHL=5 TOS=0 Length=544 Id=47014 Flags=DF FragOffset=0 TTL=64 Protocol=UDP Checksum=64871 SrcIP=<IP_ADDRESS> DstIP=<IP_ADDRESS> Options=[] Padding=[]}
UDP	{Contents=[..8..] Payload=[..90..] SrcPort=42476 DstPort=58585 Length=524 Checksum=34268}
Packet has been truncated
Failed to decode layer: No decoder for layer type Payload
CPU 01: MARK 0x569d890c FROM 2929 to-overlay: 558 bytes (128 captured), state new, interface cilium_vxlan, , identity 4347->unknown, orig-ip <IP_ADDRESS>
Ethernet	{Contents=[..14..] Payload=[..118..] SrcMAC=36:dd:54:6f:62:07 DstMAC=6a:ae:e1:ba:71:6e EthernetType=IPv4 Length=0}
IPv4	{Contents=[..20..] Payload=[..98..] Version=4 IHL=5 TOS=192 Length=572 Id=48941 Flags= FragOffset=0 TTL=63 Protocol=ICMPv4 Checksum=13845 SrcIP=<IP_ADDRESS> DstIP=<IP_ADDRESS> Options=[] Padding=[]}
ICMPv4	{Contents=[..8..] Payload=[..90..] TypeCode=DestinationUnreachable(Port) Checksum=33497 Id=0 Seq=0}
Packet has been truncated
Failed to decode layer: No decoder for layer type Payload
CPU 01: MARK 0x0 FROM 2929 to-endpoint: 586 bytes (128 captured), state related, interface lxc7a021907a17f, , identity remote-node->4347, orig-ip <IP_ADDRESS>, to endpoint 2929


OW18B Question on sign bit

Hi,
I found your OW18B BT project and obviously you've done a lot of research on the data coming from the meter. I've used your information to create an Arduino project to read and display meter information. One problem I'm seeing is that the sign bit coming from my meter always seems to be Zero, even on a negative display. Are you actually getting a sign bit coming from your meter on negative readings? Everything else seems to be working as you have documented. Thanks.

Hi,
nice to see that somebody actually benefits from this stuff :)
I just tested the example code with my own multimeter and a negative voltage, it works fine for me. Did you try that sample program with your device?
The strange thing about the sign bit is the fact that negative numbers are encoded as sign-magnitude instead of one's or two's complement.

Hi,
Yes, your work was very useful.  Thanks for getting back to me.
I’m not on Linux but took sections of your code for my Arduino project, and haven’t tried the example code.
But before that, I started with a small sketch that just read and displayed the 6-byte data coming over Bluetooth in my attempt to decode it (before I found your Github project). I was able to determine that the last two bytes were the value being read by the meter, but even at that time I did notice that the last 2 bytes didn’t change when I reversed test leads on the battery I was using as a voltage source. A 1.5V battery showed 2E06 (062E in real life is 1.582V, which is what the meter read) regardless of whether the meter read positive or negative. I know I’m reading and displaying the byte data coming from the meter correctly, because bytes 1 and 2 are coming in correctly – it’s not like I’m missing the first bit anywhere else and only getting 00-7F.
I also checked in case one of the other bytes (like byte 4, which always seems to be 00) might happen to a sign indicator – but no.
Also, I have tried their iPhone App, and it fails to connect to the OW18B with an Authentication Error. Even though they say to use that App, I’m wondering if it was only intended for the OW35 – which uses a completely different data stream.
Strange, very strange.
Ira Schonfeld
From: JayTee<EMAIL_ADDRESS>Sent: Friday, December 27, 2019 3:50 PM
To: JayTee42/ow18b<EMAIL_ADDRESS>Cc: IraSch<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [JayTee42/ow18b] OW18B Question on sign bit (#1)
Hi,
nice to see that somebody actually benefits from this stuff :)
I just tested the example code https://github.com/JayTee42/ow18b/blob/master/src/example.c  with my own multimeter and a negative voltage, it works fine for me. Did you try that sample program with your device?
The strange thing about the sign bit is the fact that negative numbers are encoded as sign-magnitude instead of one's or two's complement.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/JayTee42/ow18b/issues/1?email_source=notifications&email_token=AEIPWTB6R6NKHUWSRNRSHDLQ2ZS6ZA5CNFSM4J7NIQD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHXX3KQ#issuecomment-569343402 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AEIPWTECWXDZ2EJWXYI6Z3TQ2ZS6ZANCNFSM4J7NIQDQ .  https://github.com/notifications/beacon/AEIPWTHN5RARG5QWKNWZ6RTQ2ZS6ZA5CNFSM4J7NIQD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHXX3KQ.gif


PR: Update conda-based installer

Accommodating upstream updates from napari/label/bundle_tools_2

@mrclary, could you expand the OP to describe what improvements are available in napari/label/bundle_tools_2?

I don't understand what happened here, but the PR was correctly merged to 5.x and master. There seems to be a bug on Github that doesn't reflect that on its interface.


Remove RunParallel test cases that are taking too much time (and misr…

…eporting benchmarks)
When that 'individual' function that is run inside of RunParallel() starts taking too much time, the benchmark results will show incorrect values and take a lot of time to complete.
This in turn causes the overall benchmark testing to exceed 600 seconds at which point 'go test' aborts (fixes issue #49).

It put it in #61 since this wasn't go fmt'ed. Thanks for your contribution!

Ok, no problem, thanks.


Ability to show/hide layers based on zoom level

This is something that has come up several times, most recently in this post on the form.
@kring any reason we can't simply add a somthing like a  minimumDisplayLevel/maximumDisplayLevel to the ImageryProvider interface?  My naive understanding of our imagery code tells me this should be simple.

I agree, should be straightforward.  Switching between layers at different scales is a really common thing to do, but I've mostly seen people do it on the server side.

This is actually already implemented via the ImageryLayer.minimumTerrainLevel/maximumTerrainLevel options

Any way to "fade" between imageryLayers depending on zoom?

@riccardolardi there's not built-in way but there's a feature request here: https://github.com/CesiumGS/cesium/issues/8140


Different maps with different gravity scales and wind directions

Before we do different maps, we should establish one test room where things like gravity and the weight of the object is editable.

update() {
let friction = this.vel.copy();
friction.mult(-1);
friction.normalize();
friction.mult(this.friction);
this.applyForce(friction);
this.vel.mult(this.airResistance);
this.pos.add(this.vel);

}


⚠ Bofrost.at has degraded performance

In c0379ff, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1302 ms

Resolved: Bofrost.at performance has improved in a49c78f.


Improved Finnish translations

Modified the translation originally made by @bleadof to be more similar to the other translations.

Thanks.  Would you please also update the unit tests to match?

@mj1856 sure, I'll do it as soon as I have time.

Sure.  Just push to this same branch when you're ready.  (The PR will update automatically.)

Yeah I know, thanks. :)
On Tue 23 Jun 2015 at 02:33 Matt Johnson<EMAIL_ADDRESS>wrote:

Sure. Just push to this same branch when you're ready. (The PR will update
automatically.)
—
Reply to this email directly or view it on GitHub
https://github.com/moment/moment/pull/2429#issuecomment-114302232.

@bleadof what do you think?
@crisu83 we can only merge after tests are added, just a heads up :)

I will update the tests as soon as I have time to set up the test suite (hopefully today).
Sorry about the delay but I've been extremely busy the last two weeks.

This needs to be rebased on top of the latest code (es6) and it needs tests.

Closing for now. Feel free to reopen if you address the issues above.

Fair enough, I don't have any spare time for this right now.

Sorry! I've totally missed this. I'm currently ill, so I'll check it later this week.


Minimum viable support for always blocks.

Only basic syntactic and contextual analysis support; does not deal with update priorities etc. yet.

No objections 👍


[stable/fluent-bit] Cater for full set of Elastic Output Plugin parameters (Auth & TLS)

Caters for the full set of parameters for the Fluent-Bit Elastic Output Plugin. Useful when logging to an external Elastic platform.
Parameters include:

X-Pack Authentication
TLS

Mentioning maintainers
@kfox1111
@edsiper

/ok-to-test
/lgtm

/lgtm

/assign @unguiculus
Danke schön Reinhard

@timstoop @unguiculus update RBAC and ServiceAccount to standards. Merged with master again and bumped the chart version

Please bump the minor version. Also, the cluster role binding needs to be updated for the service account.

@timstoop @unguiculus bumped minor version and set ClusterRoleBinding to use serviceAccountName

/lgtm