Pavan/awscontrib

kubespray Init script and ssh-agent forwarding is added

We now have dedicated support for AWS in our Kubespray wrapper, so this PR is full of conflicts. I imported the most important parts of it manually: https://github.com/elastisys/compliantkubernetes/commit/c1b1a558acbb43d07b68bbf96f2c0c8ac703d6a0.


Revert "Fix LLM Server Failing CIs"

Reverts nod-ai/SHARK-Platform#362

need to revert this before reverting the other one

You can revert multiple commits in one PR. Keep this as simple as possible - get the builds green ASAP.


First draft of unit vnv plan

See commit 104d1506a1f58a21e9e9edd975f5bc07cd9ea97c
:+1:

Thank you.  I look forward to reviewing it.

@PeaWagon I have added a few comments to your unit vnv plan at 7846fb2.
I apologize for the delay in the review.  You don't need to worry about making any changes to your final documentation.  Your vnv plan looks good.


VS CODE SSH connection reconnect

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.63.2
OS Version: Mac OS Big Sur 11.6

Steps to Reproduce:

Prior to VS code, I would connect to my company VPN, set up port forwarding, connect via ssh to a GPU cluster, use tmux to keep the terminal alive and request for a jupyter notebook on a given port, open the kernel link on a browser and use the kernel to get a notebook running, disconnect VPN, log back in after some time, connect again and the notebook would still be running.

Now I have set up VS code to connect via ssh to host and can run my .ipynb files on the server directly in VS code. However, when I close of my laptop and log back in, the kernel dies. I tried to use tmux to keep the terminal alive behind the scenes but not sure how to prevent the kernel from dying when I disconnect the VPN connection?

Is there a way to get large jobs to run on VS code when connected to host via SSH, disconnect and come back later?

See https://github.com/microsoft/vscode-jupyter/issues/3998 and https://github.com/microsoft/vscode-remote-release/issues/3096


Fastify swagger implementation

https://github.com/Ethan-Arrowood/matterhorn/issues/22
Fixed with linting Rules.

Looks like npm run test is failing too. I pulled your changes locally and it looks like the schema you're passing is invalid. Try fixing that so npm run test passes

This has something to do with the TypeScript types. I believe fastify-swagger is adding additional properties to fastify's RouteSchema interface as defined in fastify.d.ts. I believe you'll need to open a PR to fastify-swagger that extends the RouteSchema interface to include the properties used by swagger.

Just corrected the schema and wrote the unit test case and now npm test is passing.

I don't think removing those properties is the correct change. You'll need to extend fastify's RouteSchema interface in fastify-swagger index.d.ts file. You'll want to add
interface RouteSchema {
    description: String,
    tags: String[],
    summary: String,
    security: any[]
  }

to this module block https://github.com/fastify/fastify-swagger/blob/55cba82f6ec9823a9d6f50accd5ce64ca9b85d1b/index.d.ts#L45

I am little confused via This. Your suggestion seems correct. However RouteSchema is already defined in Fastify.d.ts. I think we need to add it there, rather then in fastify-swagger. In case I extends it in fastify-swagger.ts with
 interface RouteSchema extends fastify.RouteSchema{     description: String,     tags: String[],     summary: String,     security: any[]   }
how will that make change to Fastify Schema. i.e. when we are defining the methods, it will link itself to RouteSchema in fastify.d.ts. Please help.

In TypeScript world when you want to expand upon an already existing type (as in modify) but you cannot directly change the type source then you simply redeclare the module using the same name and TypeScript compiler will merge the two together. fastify-swagger is already doing this:
declare module 'fastify' {
  interface FastifyInstance<
    HttpServer = http.Server,
    HttpRequest = http.IncomingMessage,
    HttpResponse = http.ServerResponse
  > {
    swagger: (
      opts?: {
        yaml?: boolean;
      }
    ) => SwaggerSchema.Spec;
  }
}

(copied and pasted from index.d.ts)
In this snipped, fastify-swagger is expanding the FastifyInstance interface to include the swagger property.
You'll do the same thing when expanding RouteSchema

Just raise the PULL Request, once it is merged, I will put the GIT URL in package.json and will update the code. https://github.com/fastify/fastify-swagger/pull/157

@panbhatt fastify-swagger was update so next you'll need to update it in your package.json. Then add back in the correct parameters.

Hi @panbhatt i'm back from my trip so I'll spend some time on your PR today. Expect some feedback soon!

Hey, All suggested changes are done. however CI test is failing, please can you have a look at let me know what needs to be done there by me (if anything).

Awesome work. The issue is not your fault, something on Azure's side. I'll figure it out and get back to you.

@panbhatt can you give me push access to your branch? I'd like to try and update the package-lock which I believe may be apart of the issue

Just did.

Thank you, I'll update you when things change

Let me know, if you want me to work on some other feature. I will try to make it as solid as i can.

I'm having issues pushing to your branch still

Check branch permissions maybe

Disregard previous comments. I was pushing wrong (forgot my local branch had different name than your remote one). Checking ci now

oh Great. i thought i did some mistake. The page says awaiting your response.
https://github.com/panbhatt/matterhorn/invitations

Yeah you're all good, I'm still fighting the CI. Consider this complete though. Once I figure out why the CI is failing I will push and merge your PR. Great work 👍

Hey @panbhatt so I've spent a while trying to figure this out. Check out my issue: https://github.com/fastify/fastify-swagger/issues/162 if you want some more details.
For now I'm going to mark this PR as blocked. When I get some help on resolving it i'll merge it into master. Thank you again for your contribution. If you'd like to try and solve this yourself, I encourage you to attempt it! Maybe you'll see something I don't.

🎉 figure it out! merging now 😄 if you're looking for more ways to contribute I think adding more fastify plugins like this will be a great way to get involved.

That's Great. Let me know wherever u think i can add like writing CLI  etc, i will jump in to make it a better product and improve myself.


Can't sign in with App Password or regular password for Google workspace account with custom domain

Describe the bug
I have a Google Workspace account with my own personal domain (does not end in @gmail.com). I have created an App Password for the integration which gave me the 16 digit password (tried with an without spaces after every 4 characters. I get the error message "Username/Password is incorrect. If your google account has 2FA enabled please generate app password (https://myaccount.google.com/apppasswords). If authentication still fails refer to https://github.com/leikoilja/ha-google-home#troubleshooting"
I have not been able to get a Master Token as I an unable to understand the guidance.
Version of the integration
If you are unsure about the version check the github release here.

v1.11.0

Logs

2024-04-06 10:37:20.944 ERROR (SyncWorker_40) [glocaltokens.client] [!] Could not get master token.

hey there i have absolut the same problem. i probably was able to create the token but i do not if it ist correct. can someone explain how the token looks like ? is it a sha256code ? i got the question if my token has "aas_et/" do not know what his means? help

Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht
Logger: glocaltokens.client
Quelle: custom_components/google_home/api.py:60
Integration: google_home (Dokumentation, Probleme)
Erstmals aufgetreten: 02:50:05 (13 Vorkommnisse)
Zuletzt protokolliert: 17:46:09
master_token doesn't follow the AAS_ET format

Master Token ungültig. Stelle sicher, dass das vollständige Token angegeben wurde (inklusive 'aas_et/')

help?

Can someone clarify where I get the Google account master token? That's the only thing holding me up.
Many Thanks!

Can someone clarify where I get the Google account master token? That's the only thing holding me up. Many Thanks!

@eebrault , please read the README: https://github.com/leikoilja/ha-google-home#master-token

Hello, same problem unable to connect

Can someone clarify where I get the Google account master token? That's the only thing holding me up. Many Thanks!

@eebrault , please read the README: https://github.com/leikoilja/ha-google-home#master-token

It doesn't really matter if it's a Google workspace account or just a regular Google account.
root@docker1[12:52:53]~# docker run --rm -it breph/ha-google-home_get-token
Unable to find image 'breph/ha-google-home_get-token:latest' locally
latest: Pulling from breph/ha-google-home_get-token
2f088d622efd: Pull complete
0188599e4719: Pull complete
0c29a314dea0: Pull complete
2f106c5bb655: Pull complete
50a99c299447: Pull complete
987516cbd67e: Pull complete
Digest: sha256:03433b75c70daedb445ef1d51996ecafe070313c9e42c1cdf10e8a835b01ea1b
Status: Downloaded newer image for breph/ha-google-home_get-token:latest
what is your google mail address? (include @gmail.com): «redacted»
what is the password? (either use regular password or app password): «redacted»

[*] Getting master token...
[!] Could not get master token.
[*] Master token: None

[*] Getting access token...
[!] Could not get access token.
[*] Access token: None

[*] Done.
root@docker1[12:54:44]~#

Thank for all, I the previous comment [CoryAlbrecht] and I used as password the password App. I recover the master token and the access token. Il works fine!

@leikoilja I Still cannot get a master token.
Did Google change their login processes at all?

@CoryAlbrecht, not that i am aware of. Have you tried logging using master token alone?

No. I cannot get a master token at all.


test: Improve tests of decrease_position_swap_utils library.

Description
Implement and improve tests of decrease_position_swap_utils library.
File: test/position/decrease_position_swap_utils.cairo.
Acceptance criteria

Unit tests (nominal and failure) are implemented and passing.

Yo, Can I take this one ?

Assigned 🔥

Any updates ? @axelizsak

Hi! Can I take it?


Linux 4.15 already ships the drm_file*

...which means I'll have to apply this patch to make the bulid work. Please do some research on version numbers before adding these preprocessor checks.
diff --git a/module/evdi_connector.c b/module/evdi_connector.c
index 811b85a..69a98b9 100644
--- a/module/evdi_connector.c
+++ b/module/evdi_connector.c
@@ -85,7 +85,7 @@ static struct drm_encoder *evdi_best_single_encoder(struct drm_connector
        int enc_id = connector->encoder_ids[0];

        return drm_encoder_find(connector->dev,
-#if KERNEL_VERSION(4, 16, 0) <= LINUX_VERSION_CODE
+#if KERNEL_VERSION(4, 15, 0) <= LINUX_VERSION_CODE
                                 NULL,
 #endif
                                 enc_id);

Yep, this is my typo, should be '4, 15' from the start.

How long would this take to be released in an updated displaylink driver package?

You can apply the patch yourself if you use any DKMS package, because these are built to the current kernel at hand. It's simple enough, really.

It is easy. Just a pain to manage when you have over 200 Dell D1000 and a CTO shouting at you about "Spectre and Meltdown security concerns".
It'd be easier for my use case to have them bundled up.

Sorry a bit of a c++ noob here. How the #@$%& do I get this to work with 4.15 kernel. No idea how to patch and I'd rather not build from source, but will if I need to.

I'm with RyanNerd -- lost my secondary display with 4.15. Currently not pleased.

Hi @RyanNerd and @cacarr-pdxweb -- I'm not sure which OS you're running, but I got it working on Arch by cloning the evdi-git repository from the AUR, adding the patch for PR #118 to the PKGBUILD, and then installing that. If either of you need more help doing that, I'd be happy to assist.

Thanks @zachzundel I'm running Linux Mint 18.2. I got it working by thunking my kernel down to 4.14.19 and running the DisplayLink script while running on that kernel. The script executed without issues. On a whim I installed the new 4.15.4 kernel and for some reason DisplayLink worked without the need to run the script against the new kernel. I hope this helps you @cacarr-pdxweb
The main challenge I had was getting the nouveau drivers working (DisplayLink will not work with NVida's proprietary drivers) -- since every SO or other web page when searching for "nouveau" tell you how to disable it and not how to get the ^#$% nouveau driver to actually work -- I still have no idea how I got nouveau to start working...

I don't think this is an issue with the displaylink package, but with the EVDI package, which DisplayLink depends on. There is already a PR that solves this issue, all that needs to happen is a maintainer merging that into master and then installing evdi-git from the AUR should fix it.

Is this even related?


the generative art component in Flask/heroku

Flask app that generates a basic image with stars and can be accessed like an api
the app is hosted on heroku on https://mintsolana.herokuapp.com/
the index page contains a form where you can add image url and number of stars
the APIish part examples:
https://mintsolana.herokuapp.com/mint?stars=1000
https://mintsolana.herokuapp.com/mint?stars=1000&r=0&g=250&b=250

we need to figure out a better solution for go-live. If it's freemium I don't think it's available 24/7, plus ideally we can just use something like DO droplets to manage all of this.

What you're contributing with this PR is the generative art component. We'll likely add the mint functionality to the same flask_app, but unless you beat me to the punch it's not in there yet.
source: https://github.com/cnp0/repomint/pull/34

I think the submodule may have not been added fully. I might just merge and help you out on this. A few other nits:

commit messages are helpful for debugging. try to lean on better commit messages.
the github url thing is cool but this is probably something were going to have faith/myself add to the react app. so we can probably pull this component out.
the endpoint for art gen should be something other than /mint since we're going to use that for the minting functionality (#29)

Again I wouldnt sweat these comments too much. I can probably handle this after we just merge your contributions as-is, but let me get back to this when im finished with my full-time job work.
Good stuff tho!

I looked at the heroku link and thats quite cool! We can build and improve on top of this


⚠ Bofrost.de has degraded performance

In 7c5cdb2, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2547 ms

Resolved: Bofrost.de performance has improved in 2f110be after 20 minutes.


Support for Switch Case

Description
switch (time) {
    case "morning":
        drink = "Tea";
        break;
    case "evening":
        drink = "Shake";
        break;
    default:
        drink="Water";
 }

The following Switch case can be written as:
bhai dekh (time) {
    agar "morning":
        drink = "Tea";
        bas kar bhai;
    agar "evening":
        drink = "Shake";
        bas kar bhai;
    nhi to:
        drink="Water";
 }

Domain
Bhai Lang
Type of Contribution
Enhancement
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of bhai-lang.

+1

nhi to:

nahin to:

+1

I would like to give it a try.

switch (time) {
case "morning":
drink = "Tea";
break;
case "evening":
drink = "Shake";
break;
default:
drink="Water";
}
The following Switch case can be written as:
as we have already used the word "agar bhai" for "if" it would be confusing for people to only write agar for case
as the hindi word for switch is badalana
and for case I suggest using mamla/halaat

badalana bhai (time){
mamla bhai "morning":
   drink = "Tea";
   bas kar bhai;
mamla bhai "evening":
   drink = "shake"
   bas kar bhai;
ye sab nahi to :
   drink ="water"';
}

INSTEAD OF "mamla" WE CAN ALSO USE "halaat"


Update dataset attributes and sync to disk store

Summary: I would like to be able to update dataset/dataarray attributes and sync the updates to disk.
Regarding the Zarr backend in Xarray, at the moment once datasets are read from disk all modifications are in memory only. This means that for the particular case of metadata stored as attributes, in order to update an attribute on disk there are two choices:

Assign the new attribute to the dataset and write the whole dataset again to disk.
Use Zarr directly to access the dataset attributes and perform an 'attribute sync'.

The former is prohibitive , in particular for large datasets, but in general it is not a good practice that in order to modify one attribute I need to rewrite the whole pixel data.
The second is possible, and certainly I could imagine the need to write a custom function to do this. Is that something that should be available in the xarray library though? Something like ds.sync_attributes() that updates the attributes on disk?

I guess this issue is related https://github.com/pydata/xarray/issues/8116
From what I've seen when working on the PR, the attributes are now being synced for Datasets and I've created a PR to do so with DataArrays


Deal with both old and new locations of Var of docplex

Summary
Follow-up of #1520
Backport of https://github.com/Qiskit/qiskit-optimization/pull/23
Details and comments
I replaced mp.linear.Var with mp.dvar.Var to be compatible with the latest docplex 2.20.204.
mp.linear.Var was used until 2.15.194. So, I modified the code to work with both styles.

I see 2.16.196 was released in early Nov last year - so its not that old. If the two classes are compatible in terms of internals could we do code that imported the Var from the new location and if that failed try the location where we imported it from before? This would give a broader compatibility if its reasonably doable.

I would like to include it as part of Aqua 0.8.2 because some users may use docplex 2.15 or older. We need to discuss it as well as #1530

For now perhaps we should add the equivalent setup.py version pin of docplex as was done in the stable branch for the 0.8.2. release. This can be revisited if some better solution is found but at least then it will compatible to with what has just been released.

Yes, it might be good to pin docplex==2.15 until a bug fix of docplex is released so that docplex works without cplex.

Lint fails due to codes not related to this PR.
************* Module aqua.algorithms.minimum_eigen_solvers.cplex.classical_cplex
qiskit/aqua/algorithms/minimum_eigen_solvers/cplex/classical_cplex.py:101:4: E1101: Instance of 'RootParameterGroup' has no 'timelimit' member (no-member)
qiskit/aqua/algorithms/minimum_eigen_solvers/cplex/classical_cplex.py:102:4: E1101: Instance of 'RootParameterGroup' has no 'threads' member (no-member)
qiskit/aqua/algorithms/minimum_eigen_solvers/cplex/classical_cplex.py:103:4: E1101: Instance of 'RootParameterGroup' has no 'mip' member (no-member)
qiskit/aqua/algorithms/minimum_eigen_solvers/cplex/classical_cplex.py:104:4: E1101: Instance of 'RootParameterGroup' has no 'mip' member (no-member)
qiskit/aqua/algorithms/minimum_eigen_solvers/cplex/classical_cplex.py:105:4: E1101: Instance of 'RootParameterGroup' has no 'mip' member (no-member)
************* Module chemistry.qmolecule
qiskit/chemistry/qmolecule.py:203:43: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:207:42: E1101: Instance of 'Group' has no 'tobytes' member (no-member)
qiskit/chemistry/qmolecule.py:211:49: E1101: Instance of 'Group' has no 'tobytes' member (no-member)
qiskit/chemistry/qmolecule.py:213:44: E1101: Instance of 'Group' has no 'tobytes' member (no-member)
qiskit/chemistry/qmolecule.py:217:48: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:219:63: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:223:49: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:225:46: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:227:45: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:236:53: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:238:49: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:240:46: E1101: Instance of 'Group' has no 'dtype' member (no-member)
qiskit/chemistry/qmolecule.py:586:53: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:589:53: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:597:51: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:600:56: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:603:53: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:606:53: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:609:49: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:613:59: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:616:59: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:620:64: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:623:64: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:626:64: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:630:54: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:633:54: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:636:54: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:640:59: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:643:59: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:646:59: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:649:59: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:652:59: E1101: Instance of 'Group' has no 'shape' member (no-member)
qiskit/chemistry/qmolecule.py:655:59: E1101: Instance of 'Group' has no 'shape' member (no-member)
************* Module test.aqua.operators.test_state_construction
test/aqua/operators/test_state_construction.py:74:27: E1123: Unexpected keyword argument 'meas' in method call (unexpected-keyword-arg)

-------------------------------------------------------------------
Your code has been rated at 9.95/10 (previous run: 10.00/10, -0.05)

This requires #1533 to avoid issues caused by astroid 2.5.

I see pylint 2.6.2 fixes the lint issues.


Use user language for default translated names

If a user chooses to translate an existing string, default language should just be the user language, and the language list should be ordered according to the OSM user preferences for easy selection.

This will be very helpful for those working with non latin scripts as searching the list is not very convenient.

:+1:

@bhousel , any head start on this one?

Sure.. localized.js contains the code for this field.  It's a normal input field with a '+' button that lets users add a localized value.
The click handler for the '+' button calls addBlank.
You can rename addBlank to addNew, and change the code in there to:

get user's locale using iD.detect().locale.toLowerCase()
probably need to split on '-' and take only the first part
use that as the default value for lang instead of setting it to ''
^ but don't do that if

the user's lang starts with en or
an entry for that language already exists in data

if either of those conditions are true leave default new entry as lang: ''


point to development first steps

@rokroskar we should focus on making development branch merged to master since the maint-0.1 was created by @erbou.

Absolutely, but it should be done when the other repos are ready. Until then we should keep links consistent to avoid confusion.


Is there a way to pass a sourcemap to ngAnnotate?

I am trying to concat, then annotate, then minify and I need to create source-map along the way but not sure I can pass a source-map to ng-annotate. Is this possible?

Currently grunt-ng-annotate combines source maps but AFAIK only if the input source map is inline in the generated file. This is mostly handled by ng-annotate, though, I'm mostly just constructing proper options so that it works. Maybe open an issue there? I can add support but only if ng-annotate handles it correctly.
Note, though, that grunt-ng-annotate doesn't support creating a source map for multiple source files (see #26). This shouldn't be an issue if you're concatenating using another task, I guess.

@mzgol do I understand you correctly that the following scenario should work?

concat several files and generate an inline source map
annotate the concatenated file and create a source map
the resulting source map should be a combination of both source maps

@stefan-- Interesting; I have a test for combining source maps that ensures combining with Babel inline source maps works:
https://github.com/mzgol/grunt-ng-annotate/blob/259f56efd9f51a32ac1e54ea50f640bcca0efc29/test/spec/api.js#L116-L138.
Both the fact that sources point to the original file and the mappings are checked.
Could you post a simple test case showing the bug?

@mzgol thanks for your quick response. I just had a look at your test case in order to understand whether it maps to my case and it does with the only difference that the sourcemap from the transpilation has only one source and the sourcemap from my concatenation task has several sources. In addition, babel is capable of adding the sourcesRoot property whereas grunt-contrib-concat does not support it yet. Any further suggestions?

Hmm, OK, it seems a basic example with grunt-contrib-concat should reproduce the problem then. I'll see what I can do (once I find time, it's been busy for me recently).
One thing you could do is to try to run ng-annotate directly on the output of grunt-contrib-concat with proper parameters and see what needs to be passed there to make it work. This is to determine whether the bug lies in the Grunt plugin or ng-annotate itself.

@mzgol it works with ng-annotate

Could you write how you call it (with what parameters)? I'll check later what I do incorrectly here.
For reference, this is where ng-annotate is called:
https://github.com/mzgol/grunt-ng-annotate/blob/259f56efd9f51a32ac1e54ea50f640bcca0efc29/tasks/ng-annotate.js#L168

@mzgol just 'solved' the problem. I attempted to save the source map not inline but as a separate file with `sourceMap: '/path/to/file'. This is where it fails. Maybe the documentation should take this into account or the lib should support this case. Thanks for your help!

Ah, so it breaks not with inline source maps but with external ones?
That's possible, I have a test for combining inline source maps but no test for combining external ones... Does that work with pure ng-annotate? If so, could you post how a snippet showing how you call ng-annotate?

@mzgol exactly. It works with pure ng-annotate with the following command ng-annotate -a -o app.annotated.js --sourcemap app.js, however, it does only support an inline source map, as far as I know.

Any news on this issue?

No news. I'll happily merge a PR if someone submits one.


Suggested change to use the bitrate provided from the metadata

Way to cover use of overestimated bitrate in case of bogus metdata .
Also add a min_audio_bitrate if no audio_bitrate in info data use the lowest possible.
Searched my video files 115 out of 1214 files had no audio bitrate when encoded from matroska, so it's quite common at least for me.

Cleaned up the estimateVideoBitrate method to remove some redundant checks, shrink the overall code, and use more consistent variables, let me know if there's any issues
#1482
    # Estimate the video bitrate
    def estimateVideoBitrate(self, info, baserate=64000, tolerance=0.95):
        # attempt to return the detected video bitrate, if applicable
        min_video_bitrate = (info.video.bitrate / 1000) if info.video and info.video.bitrate else None

        try:
            total_bitrate = info.format.bitrate
            audio_bitrate = 0
            min_audio_bitrate = 0
            for a in info.audio:
                audio_bitrate += a.bitrate if a.birate else (baserate * (a.audio_channels or 2))

            self.log.debug("Total bitrate is %s." % info.format.bitrate)
            self.log.debug("Total audio bitrate is %s." % audio_bitrate)
            audio_bitrate += min_audio_bitrate
            calculated_bitrate = (total_bitrate - audio_bitrate) / 1000
            self.log.debug("Estimated video bitrate is %s." % (calculated_bitrate * 1000))
            return min_video_bitrate if min_video_bitrate and min_video_bitrate < (calculated_bitrate * tolerance) else (calculated_bitrate * tolerance)
        except:
            if info.format.bitrate:
                return min_video_bitrate if min_video_bitrate and min_video_bitrate < (info.format.bitrate / 1000) else (info.format.bitrate / 1000)
        return min_video_bitrate

ebd26fbc84bb0bd03163b7ec8404666a20e75f7e
@threadproc


Switch to Bt. 709?

ffmpeg's rgb --> yuv conversion uses bt.601 as the default colorspace instead of bt.709, which results in shitty looking videos by default

colors look washed out and dull

bt.601 is a smaller colorspace and can't properly represent the colors bt.709 has

Default ffmpeg video CLI should use bt709 mode:
-vf scale=out_color_matrix=bt709 -color_primaries bt709 -color_trc bt709 -colorspace bt709

youtube likely expects bt.709 (same colorspace as srgb), but it'll handle bt.601 completely fine too

https://medium.com/invideo-io/talking-about-colorspaces-and-ffmpeg-f6d0b037cc2f idk

https://blog.maxofs2d.net/post/148346073513/bt601-vs-bt709

-color_primaries bt709 -color_trc bt709 -colorspace bt709 results in #ff0000 turning into #ff1800.

do i use full-range color? obs forum says youtube is not designed to use it.

closed by #384

Use -vf scale=out_color_matrix=bt709:flags=accurate_rnd  -colorspace bt709  -color_primaries bt709 -color_trc bt709. oogh.

is it necessary to set color_range metadata, or not really

Yes, it is. It is set usually for limited. swscale is defaulting to limited so both -vf scale=out_color_matrix=bt601:out_range=pc,format=yuv420p and -color_range pc are needed.


Clip exponential in ffunc to avoid overflow

This should avoid (hopefully) some possible FloatingPointError overflow errors.
The sigmoid function ffunc is for any x<-20 and x>20 already 0 resp. 1 up to 10^-9
and cutting will therefore not change the function substantially.
This idea is from @tmbdev in https://github.com/tmbdev/ocropy/issues/5#issuecomment-65329170
Implemented first in https://github.com/tmbdev/ocropy/issues/49#issuecomment-139919235
Additional infos from https://github.com/tmbdev/ocropy/issues/79#issue-126999594

Note that some lines above we have:
def sigmoid(x):
    """Compute the sigmoid function.
    We don't bother with clipping the input value because IEEE floating
    point behaves reasonably with this function even for infinities."""
return 1.0/(1.0+exp(-x))

Do we need both identical functions ffunc and sigmoid?
What are "IEEE floating points"?
Should we reconsider the comment here?

-20/20 seems to be overly conservative but should work. Ultimately the gradients are important and they are negligible even at -10/10. On another note scipy.special.expit is significantly faster than .0/(1.0+exp(-x)), so you might consider using that.
You won't need both ffunc and sigmoid as pickling only retains references to classes and function. Adjusting the forward/backward methods of the appropriate classes should be enough.
IEEE floating points are standard IEEE 754 floating point numbers. The comment suggests they are stable even for extreme values. The linked bug reports seem to contradict that notion.

I'd like to merge it, and/or try @mittagessen's proposed scipy fix. @zuphilip do you have data to test this?

@kba I think it is good to merge this. Maybe reformulate the contradicting comment before... I don't have data for reproduce the error, but there is at least one confirmation https://github.com/tmbdev/ocropy/issues/246#issuecomment-331435380 that after this fix it worked fine.

Yes, this will fix a breaking bug, so let's merge it. Just would be interesting to test scipy.special.expit. But working is better than fast.

@mittagessen and @kba Thank you for helping here!


scmrepo v0.1.8

sync deps with scmrepo's setup.cfg
update to 0.1.8

@skshetry I matched the order in scmrepo for the sake of consistency. If you prefer alphabetical order, I can open a PR in scmrepo too.

No need to change in scmrepo, here it was already alphabetically ordered, that's why I asked. No strong opinions though.

Sorted alphabetically in the latest push


Scribble Handwriting Recognition support (iPadOS14)

iPadOS14 added Handwriting Recognition support so you can just hand write in a TextField and it will translate it to a string.

Here is the UIScribbleInteractionDelegate
documentation link: https://developer.apple.com/documentation/uikit/uiscribbleinteractiondelegate
Related issue: #38135

Duplicate of https://github.com/flutter/flutter/issues/61278


Unable to download or upload or open or do any activities on Storage Explorer

Unable to download or upload or open or do any activities on Storage Explorer. Getting the below error
"Invalid IV Length"

Could you tell us more info?

which version of Storage Explorer
Which OS
Did you sign into Azure or attach to Storage resources? or use local emulator? or use Azure Stack?
Did you have AzCopy enabled?
Did you try to upload/download blobs or files?
Thanks.

Dont bother. They never gonna fix it. Try alternatives

@RobertK73 We love to hear from our customers. Before we can do anything here, we need to first understand the issues.  There was not much information in the original post. Therefore the follow up questions.  If it turns out to be a bug in the product, we likely would fix it.  If you already knew the cause of the error message shown in the original post, please help and share so we can further investigate and fix the issue(s).  Thanks.

Let's just catch the IV error and if it happens make a new IV.

@shweta777 can you get me some more information?

Open Storage Explorer
Open Devtools (F12)

Press ctrl+p
Type TextEncrypter.js and then hit enter

Set a breakpoint on line 53 by clicking on the number 53

Try downloading/opening your file
When the breakpoint is hit, go to the console, type in ecryptionKey, press enter, and copy the resulting value to your clipboard

Close the dev tools
Let us know if the download/open works now, and also let us know the value you copied to your clipboard.

Closing due to lack of reponse.


Regarding Auto-Tiering  concepts.

Hi ,
I have deployed the azure local blob storage on the edge device. my aim is to check weather the auto tiering function of azure is working fine and to check weather its uploaded to the could and delete the file for the localblob.
I have found how to delete from the local blob but, i couldn't see any notification or some return statement saying that the file is completed uploaded to the clould blob with the auto-tearing kindly let me know if such thing exist.

@Alberto-Vega-MSFT  @himanshusinha-msft @AshokPeddakotla-MSFT @SumanthMarigowda-MSFT @DiegoRamirez-MSFT @sergaz-msft
hi all
Please let me sort this issue.
Thanks,
Avinash

@avinashchadalavada Thanks for the feedback! We are currently investigating and will update you shortly.
Please help with the link of document on which above document feedback request is created.

Link: -
https://docs.microsoft.com/en-us/azure/iot-edge/how-to-store-data-blob
i have created the setup of the link already. i need a to know is there a way to check the file auto-tieried is uplaoded to the cloud sents back any response saying its uploaded.
boz on that responce i have to delete the file from te blob

The problem we’re trying to solve is auto-expiration of the files from the gateway after they’ve been uploaded to the cloud.  The auto-exparing feature seems to only be time-based, so when the TTL has expired the file is deleted, even if the file has not be auto-tiered to the cloud.  We’re looking for a way to ensure that files are only deleted from the gateway AFTER they’ve been successfully uploaded to the cloud.

@AshokPeddakotla-MSFT  let me know if you need any other info on this.

@kgremban Could you please check this and provide your insights?

@avinashchadalavada -
We have the feature enhancement request in our backlog, it will be available sometime in Future.
For your scenario, you can either set the TTL values high enough to give the tiering module ample time to upload the data to Azure. Or turn the TTL off and call delete blob when the blob exists in destination container.
Are you interested in getting a notification when the data is tiered to Azure like FileUpload:- https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-devguide-file-upload#notify-iot-hub-of-a-completed-file-upload ?

Thanks @arduppal and @AshokPeddakotla-MSFT
thanks for the link @arduppal
i have to build like  a custom module which deletes the file when the auto-tearing is completed.
The videos file which is auto-tiered sends the data directly to the blob (which the connection string is provided) or it goes through the iotHub and then then to the blob ?, if the module sends the file to the blob how does the local blob come to know the whole file is uploaded or not.
In Detail: -When  auto-tiering happens what if the network is failed in the middle of uploading the file, so some chunk of data is already uploaded to the blob, how will the edge know that the particular chunk is uploaded and other is remaining not uploaded .  or else it starts uploading the whole date from the beginning.

The tiered data directly goes to Azure.
We maintain the information about how much data is tiered in our local database
For your other question am pasting the text from this doc:

@avinashchadalavada Please check the above response and let us know if that answers your queries.

yes thanks @AshokPeddakotla-MSFT  and @arduppal .
if i have anything else i ll connect back thanks.

@avinashchadalavada Thanks for the confirmation.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


MacOS crashes on Controllers.getControllers()

It works fine on Linux, Android, etc.
But for some reason, when it gets to this specific line in MacOS, the program hangs.
Controllers.getControllers()
I've proven this with debug statements.
It doesn't matter if I have a controller connected via USB, paired with bluetooth (and working in another game), or no controller at all. Same result.
I'm on the "latest" 2.2.3 gdx-controllers. Yes, I put it in quotes, because it's 1.3 years old, and I noticed at least 2 commits since the 2.2.3 release which could affect this.
Using the latest LibGDX as well.
1.12.1

The lib uses Jamepad internally, maybe try to use a newer version.

So you mean fork this repo and make that upgrade to the Jamepad library, publish my forked version to Maven or something, and then include that in my Android LibGDX project instead of gdx-controllers?

No; see "Installation" on the main readme page

Ok, I have the following versions
gdxControllersVersion=2.2.4-SNAPSHOT
jamepadVersion=<IP_ADDRESS>
gdxVersion=1.12.1
And I updated my Desktop build gradle:
dependencies {
implementation project(':core')
implementation "com.badlogicgames.gdx:gdx-backend-lwjgl3:$gdxVersion"
implementation "com.badlogicgames.gdx:gdx-platform:$gdxVersion:natives-desktop"
implementation "com.badlogicgames.gdx-controllers:gdx-controllers-desktop:$gdxControllersVersion"
implementation "com.badlogicgames.gdx:gdx-box2d-platform:$gdxVersion:natives-desktop"
implementation("com.badlogicgames.jamepad:jamepad:$jamepadVersion") {
exclude group: 'com.badlogicgames.gdx', module: 'gdx-jnigen-loader'
}
}
Still having the same lock-up issue. The program freezes at Controllers.getControllers(), but only on MacOS desktop.

Could you try to get a stack trace while the program is hanging? VisualVM or a similar program should let you do that.

WHILE the program is hanging? That sounds like Logcat. If this were Android, I'd check the Logcat output with a crash like this. But is there such an equivalent on MacOS?
I don't have VisualVM. Is that free? (This is a hobby project, not a commercial company with a budget to purchase software)
What do VisualVM and similar programs do? Give you details about a crash when a non-debug executable is running?

VisualVM is a Java profiler. It's a free download, and at one point was even packaged with the JDK. It runs as its own application, displays a list of java applications that are running. You can click on one to start monitoring it, and it will show you various information about what the program is doing. In the Threads tab, there is a button labeled "Thread Dump" that will give you a stack trace of your threads.
Download page: https://visualvm.github.io/
Documentation: https://docs.oracle.com/javase/8/docs/technotes/guides/visualvm/
Page that explains Thread Dump: https://docs.oracle.com/javase/8/docs/technotes/guides/visualvm/threads.html


Reporting Queries

@nrshapiro
I few reporting queries:

How can I tell the language of a document?
Do you store co-authors separately?  If so, how do I get to it?  I need to report on them independently.

Thanks

@jwoosnam
You can get all document metadata using the API, including language and authors.  You can use /v2/Documents/Abstract (or you can even use search by putting the article ID in the smartSearch parameter.

@jwoosnam
While it would be less "protected" against changes in infrastructure, you could also connect to the AWS-RDS database "opascentral" directly, and get article data from table api_articles.

@nrshapiro
My oversight, I can see that the language against the journal, which is fine.
In terms of authors I was hoping you had an authors table with an intersecting "document author" table.  Since this morning I have had a brief discussion with David about this and he says authors have always been troublesome. If the format of the authors field is consistent then I can work with that.
Thanks


Adapt monitoring to restricted PSP clusters

Address issue: https://github.com/rancher/rancher/issues/20229
Problem:
Fail to enable monitoring on restricted PSP clusters.
Solution:

Update grafana to run as grafana user.
Update nginx sidecars to run as nginx user, move files to the user
home directory(/var/cache/nginx) and listen on non-privileged 8080 port.
With 1 & 2, project monitoring works in restricted clusters out of the
box.
Add necessary PSP/role/rb resource to node-exporter, which is used by
cluster monitoring.
The added PSP is bound to node-exporter SA. Its rules include: restricted rules + hostPath + hostNetwork + hostPorts + hostPID

LGTM

@cloudnautique or @drpebcak can you review

I need to talk through this with @cloudnautique tomorrow. This is the opposite of what I suggested. Instead of leveraging the PSP features of rancher, we just put the raw psp resources into this helm chart. That means they are completely outside the purview of Rancher PSP management, which effectively defeats the purpose of the PSP feature in rancher.

Discussed the implementation with @cloudnautique and we are good with it. I have opened this enhancement so that Rancher will do a better job of surfacing PSP bindings in general https://github.com/rancher/rancher/issues/22525


utils: If shell.run_parallel is not called before run.shell(), then the lock is not initialized nor in scope

Fix for a problem with shell.run_parallel where this function must be called before shell.run().

@swift-ci Please smoke test


ensure hive puts a domain into the apiserver object

The domain field must not include the protocol or port information from the clusterDeployment.status.APIURL field.
https://issues.redhat.com/browse/CO-799

@staebler should this controller bother to reconcile before the clusterDeployment has reached the "install completed" state?

@staebler i went ahead and started using the remoteclient builder to fetch the URL as suggested
i also put in the check early in the controller to not bother reconciling a ClusterDeployment until it is marked 'installed' WDYT?

/test e2e

@staebler PTAL the most recent commit addressing your feedback. then i can squash.

@staebler PTAL the most recent commit addressing your feedback. then i can squash.

@joelddiaz I am not seeing any recent commits addressing my feedback.

@joelddiaz I am not seeing any recent commits addressing my feedback.
It certainly helps if I actually push the commit :)

/test e2e

/test e2e


Bump version to v0.8.3 and add dockerfile

ref: https://github.com/fermyon/installer/pull/100#issuecomment-1309297406

Hi @vdice, I would like to add the docker release pipeline for fermyon/bindle. May you help me review it? Thank you.

Oh interesting approach :) I forgot we had a fork for the v0.8.0 backport(s).
As of now, the workflows for this fork are disabled:

Before enabling, we'd need to attend to a few other items in the release.yml workflow: there still are steps to publish to azure blob storage and crates.io, neither of which we're prepared to do from the fork (don't have sufficient creds, wouldn't want to overwrite).  I guess we didn't envision actually creating a release from this fork :)
I suppose one way forward would be to remove the blob storage/crates.io steps and then introduce the image push to this ghcr.io/fermyon/bindle repo.
Alternatively, if we think this may be a one-off or seldom used (i.e. eventually we'd use the images that are now being pushed in deislabs/bindle when we/downstream consumers are ready to bump to v0.9.0+), maybe instead of enabling the workflow, we manually push a ghcr.io/fermyon/bindle/0.8.2 image based on the fermyon/0.8 branch as it stands today, for use in eg https://github.com/fermyon/installer/pull/100#issuecomment-1310430877
Let's check in with the team here - thoughts @endocrimes @dicej ?

Yeah, I wouldn't spend a lot of time on CI/CD for this fork, since we're planning to switch back to upstream eventually, and I doubt we'll be making many (if any) more changes here.  Manually pushing an image seems reasonable to me.

Yeah, if we want to switch backup to upstream eventually, manually pushing an image makes sense to me.
@vdice, may you help me create a fermyon/bindle:v0.8.2 image for temporary use? The Dockerfile in this PR is different from deislabs/bindle because we don't need keyring.toml in v0.8.x. After the image is pushed, feel free to close this PR. Thank you!

@FrankYang0529 okay, ghcr.io/fermyon/bindle:v0.8.2 has been pushed and should be accessible.  It was built directly from the most recent commit on the fermyon/0.8 branch.
I wouldn't oppose adding the Dockerfile and make target to this branch, but we can wait to see if we'll need to build/push from this branch in the future.
Thanks again for supplying the Dockerfile and helping with a solution here!

Hi @vdice, I tried to run https://github.com/fermyon/installer/pull/100 with ghcr.io/fermyon/bindle:v0.8.2. It looks like the architecture of the image is arm64. I got error exec /usr/local/bin/bindle-server: exec format error on nomad.

@FrankYang0529 Oh, sorry for the trouble here!  I've rebuilt the image for linux/amd64 on my Apple Silicon machine via buildx and pushed that to GHCR.  Let me know if you see any further issues.


Readme.txt is missing a -p in the plcncli install command

Line 12 in the https://github.com/PLCnext/PLCnext_CLI/blob/master/src/PlcNext/README.txt is missing a "-p" for the path to the sdk archive.

Fix is planned for the next version.


Remove double semicolo

This should fix #23 and fix #24

It looks lgtm
Can I ask you to update this line as well?
https://github.com/mirus-ua/hugo-theme-re-terminal/blob/fab58b1697f22639672dc1756b42707a8134c844/demoSite/hugo.toml#L4

It looks lgtm Can I ask you to update this line as well?

Oh, I thought I did.
Yep, I'll do that during the day


Test renewal of annual membership

Create an annual membership, renew it before expiration, make sure the date is pushed back the proper amount regardless of when the renewal occurs.
Then try renewing a lapsed annual membership.  Make sure fees have reasonable semantics in the gap between expiry and renewal.

Related bugs:  #455 #539

This issue was moved to bitshares/bitshares-2#42


Fast switching of selected gamma mode

It would be really useful if there was a command that would tell the JVC to switch directly to a particular gamma 1 - 3.  For instance "sg 1" as in "switch to gamma 1". This would then do the same as us navigating in the JVC menus to manually change it to custom gamma 1. This way we can do "sg 3" to switch directly to custom gamma 3 which will speed up the process.  BTW some of my requests are not "issues" but rather "feature requests". I don't see an option in Github for "feature requests" so I am listing them under issues. Sorry about that.

I plan to add support to set, save and restore picture mode settings, which should also cover this use case.


feature: format detection outputs

#205

Coverage increased (+0.1%) to 91.085% when pulling 5d9e09979a9fe69038890413cdb3fa11cd8a6885 on terryyz:master into e5c950555944d930bdbd71297a71a56d48a419cc on ssut:master.


Added "Enroll Household Member" button to the monitoree details section

Description
As a part of SARAALERT-1221, the PO team noticed that the previously indirect way of enrolling a household member (through the edit prompts) was even more indirect with the change in 1221.  Added a button to the household information section of the monitoree details for HoH and single households that allows them to enroll a new household monitoree straight from that screen.  This button should not be present if the current user does not have enroller permissions.
(Feature) Demo/Screenshots
https://user-images.githubusercontent.com/35042815/109983529-e6135300-7cd0-11eb-8581-e8dbb1f7fc2b.mov
Important Changes
Please list important files (meaning substantial or integral to the PR) along with a list of the general changes that should be highlighted for reviewers.
EnrollHouseholdMember.js

New component for a button to enroll a new household member
When clicked, the button loads a new enrollment page for group members

Patient.js

Added new EnrollHouseholdMember component to the household information section for HoH and single household monitorees

PatientPage.js, show.html.erb

Updated props to include if current user can create a patient

EnrollHouseholdMember.test.js, Patient.js

Added/updated necessary react component test files

PublicHealthHeader.js, index.html.erb

Changed enroll monitoree icon to match the new button icon

patients_controller.rb

Reordered address fields into an order that makes sense

PauseNotifications.test.js

Updated spacing to match the rest (2 space tabs vs 4)

Testing
This fix was tested on the following browsers (submitter must check all those that apply):

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] IE11

Please describe the tests needed to verify your changes. Provide instructions for reproducing these tests. List any relevant details for your test configuration.

@sgober It looks like enroll system tests are failing - not sure if a problem or intermittent. Might be worth investigating and re-running in parallel.

@sgober It looks like enroll system tests are failing - not sure if a problem or intermittent. Might be worth investigating and re-running in parallel.

Rerunning right now.  They were passing before this minor change https://github.com/SaraAlert/SaraAlert/pull/737/commits/8e671f47720b3de9006b03fbc0ba2d96d4a9668f so not sure why that would have changed anything.  @timwongj thoughts?

@kylietmo @timwongj @DPC15038 @mdube7 I just pushed some changes that now include a confirmation modal when clicking the "Enroll Household Member" button (see video below).  Please re-review!
https://user-images.githubusercontent.com/35042815/110025019-d9a4ef80-7cfc-11eb-8546-970ae020acc5.mov


opt_expr: Add more $alu optimizations.

Detect the places in the $alu where the carry bit is constant (due to
const A[i] == B[i] ^ BI) and split it into smaller $alu at these points.
Also, make the existing const-carry detection for low bits more generic
(now handles cases where both BI and CI are constant, but not equal to
one another).
Fixes #1912.

(draft because tests pending)

(fixed up review comments)


Account generation issue when running multiple plugins

Dear ConsenSys,
When using the security plugin alongside this plugin, account generation via geth command
returns an error message of :

Fatal: unable to create plugin-backed account: unsupported plugins configured: [security]

My current solution is to either disable the security plugin on startup and create the account or duplicate the quorum.json file and revert by removing the security plugin portion.
Just to be clearer, the error only appears when running the account generation geth command.
On startup, it works perfectly and the account unlock works as well.
The geth command i ran:
geth account plugin new \       --plugins file:///path/to/quorum.json \       --plugins.skipverify \       --plugins.account.config '{"secretName": "demoacct"}'
my current quorum.json file:
{     "providers": {         "account": {             "name": "quorum-account-plugin-hashicorp-vault",             "version": "0.1.0",             "config": "file:///etc/plugins/plugin.json"         },         "security": {             "name": "quorum-security-plugin-enterprise",             "version": "0.1.2",             "config": "file:///etc/plugins/security-plugin.json"         }     } }
reverted quorum.json file:
{     "providers": {         "account": {             "name": "quorum-account-plugin-hashicorp-vault",             "version": "0.1.0",             "config": "file:///etc/plugins/plugin.json"         }     } }
Error log:

Hi Chris, Thank you for the clarification. I have understood the rational behind it.


The authentication is undocumented. Need to disable it for now.

Until it is documented and usable, we should default to disabling it.
Signed-off-by: Mark-Sturdevant<EMAIL_ADDRESS>

Seems like a good plan to me. will help get people moving forward with no action on their part. Thanks for doing this, @markstur !


Fix missing comma in package.json

Ouch, thank you! Sorry for breaking it. :(

@tycho01 No worries, thanks for the quick turnaround. I discovered this when trying to npm link the package; the npm registry package seems quite out of date.

Yeah... interested to find out how to update that, but I'm kinda new to @types


Proper types for sampled images in HLSL4/5

Fixes passing to functions,...

Using a struct to represent combined image samplers seems like the cleanest approach, but I'm a bit worried that this will not work in all cases.
Can you use : register(t#) and  : register(s#) for the texture and sampler in those structs?
How will combining the image and sampler work? Can you build a structure safely from a standalone Texture2D and SamplerState and use that to sample an image?

No luck with register declarations according to https://msdn.microsoft.com/en-us/library/windows/desktop/dn899207(v=vs.85).aspx
Have to flatten it. Should I wait for your other pull request?

I will continue on that PR soon-ish. It is also needed to sort out some corner cases with shadow samplers as well, so I suggest we go with that approach instead.

Okidoki!

Unfortunate about the duplicated work, but oh well.


[bug] ethers dependency

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
0.6.0
wagmi Version
0.6.6
Current Behavior
When trying to install dependencies having a newer version of ethers, I get this error, unless I use --force.
I'm currently using ethers 5.7.1. But seems to be incompatible with rainbowkit-siwe-next-auth.
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/siwe
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@rainbow-me/rainbowkit-siwe-next-auth

Expected Behavior
No response
Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

@soccerarenadev This is related to the siwe version. In v2.0.3-beta they updated the ethers version; before that, you'll get the error described above. The current latest siwe version is 2.0.5-beta and upgrading to that should solve the issue.

@soccerarenadev This is related to the siwe version. In v2.0.3-beta they updated the ethers version; before that, you'll get the error described above. The current latest siwe version is 2.0.5-beta and upgrading to that should solve the issue.

I've tried updating the siwe version. And still the same error. I guess we should wait rainbowkit-siwe-next-auth to update it as well.

Please update siwe package, every time I want to use it it is impossible and I have to think for a few hours how to solve it so that the program does not crash

Please update siwe package, every time I want to use it it is impossible and I have to think for a few hours how to solve it so that the program does not crash

Until siwe is updated, you can just copy over the rainbowkit-siwe-next-auth source files into your ts project and use any siwe version you like.

Hi all! We're working on getting the updates to siwe into RainbowKit to resolve this peer dependency conflict. We're working with Spruce on this, and expect their prod API to settle soon. https://github.com/spruceid/siwe/pull/138


Add ingress support for grafana and prometheus

Sorry. the previous one (https://github.com/istio/istio/pull/9450) is closed by accident.
Let us use this new one to add ingress for grafana and prometheus to be consistent with other. This decision was made in this pr: #9217
Signed-off-by: clyang82<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: clyang82
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: mandarjog
If they are not already assigned, you can assign the PR to them by writing /assign @mandarjog in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @sdake
/assign @gyliu513
/assign @morvencao

/retest

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: clyang82, gyliu513
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [gyliu513]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Replace Android system TTS engine

You can find two demo videos below:

YouTube: https://www.youtube.com/watch?v=33QYuVzDORA
Bilibili: https://www.bilibili.com/video/BV1Wc411b7fJ/

The following two repositories are used as a reference during implementation:

https://github.com/Miserlou/Android-SDK-Samples/blob/master/TtsEngine/src/com/example/android/ttsengine/CheckVoiceData.java
https://github.com/benjaminwan/ChineseTtsTflite

Note that you can use all supported models with this PR, e.g., models from piper and coqui-ai/TTS, etc.
Fixes #488
See also
https://github.com/rhasspy/piper/issues/257#issuecomment-1872650072
@Pantyhose-X
@christiankaye
@ekorn
@LuekWasHere
@beqabeqa473
@paolo-caroni

I can't seem to figure how to build this pull request.
It seems (from my unqualified perspective), that there is an error to do with a file that is being declared though doesnt exist.
It gives this error specifically:
Expecting a top level declaration

Is this an issue with me or perhaps an issue with the Package directive?

I can't seem to figure how to build this pull request.
It seems (from my unqualified perspective), that there is an error to do with a file that is being declared though doesnt exist.
It gives this error specifically:
Expecting a top level declaration

Is this an issue with me or perhaps an issue with the Package directive?

Looks like you are using windows.
That file is a symlink. Please delete that file and use the real file to replace it.
Windows does not support symlinks.

Looks like you are using windows.
That file is a symlink. Please delete that file and use the real file to replace it.
Windows does not support symlinks.

By the way, we don't hard-code a model in the code since we try to support as many models as possible.
You need to change the code to select a model. The following is an example:
diff --git a/android/SherpaOnnxTtsEngine/app/src/main/java/com/k2fsa/sherpa/onnx/tts/engine/TtsEngine.kt b/android/SherpaOnnxTtsEngine/app/src/main/java/com/k2fsa/sherpa/onnx/tts/engine/TtsEngine.kt
index 4fc935b..52626e1 100644
--- a/android/SherpaOnnxTtsEngine/app/src/main/java/com/k2fsa/sherpa/onnx/tts/engine/TtsEngine.kt
+++ b/android/SherpaOnnxTtsEngine/app/src/main/java/com/k2fsa/sherpa/onnx/tts/engine/TtsEngine.kt
@@ -60,12 +60,12 @@ object TtsEngine {
         // The purpose of such a design is to make the CI test easier
         // Please see
         // https://github.com/k2-fsa/sherpa-onnx/blob/master/scripts/apk/generate-tts-apk-script.py
-        modelDir = null
-        modelName = null
+        modelDir = "vits-piper-en_GB-southern_english_female-low"
+        modelName = "en_GB-southern_english_female-low.onnx"
         ruleFsts = null
         lexicon = null
-        dataDir = null
-        lang = null
+        dataDir = "vits-piper-en_GB-southern_english_female-low/espeak-ng-data"
+        lang = "eng"

We give several detailed examples in the code. Please do read them before you build an APK.

By the way, we don't hard-code a model in the code since we try to support as many models as possible.
You need to change the code to select a model. The following is an example:
diff --git a/android/SherpaOnnxTtsEngine/app/src/main/java/com/k2fsa/sherpa/onnx/tts/engine/TtsEngine.kt b/android/SherpaOnnxTtsEngine/app/src/main/java/com/k2fsa/sherpa/onnx/tts/engine/TtsEngine.kt
index 4fc935b..52626e1 100644
--- a/android/SherpaOnnxTtsEngine/app/src/main/java/com/k2fsa/sherpa/onnx/tts/engine/TtsEngine.kt
+++ b/android/SherpaOnnxTtsEngine/app/src/main/java/com/k2fsa/sherpa/onnx/tts/engine/TtsEngine.kt
@@ -60,12 +60,12 @@ object TtsEngine {
         // The purpose of such a design is to make the CI test easier
         // Please see
         // https://github.com/k2-fsa/sherpa-onnx/blob/master/scripts/apk/generate-tts-apk-script.py
-        modelDir = null
-        modelName = null
+        modelDir = "vits-piper-en_GB-southern_english_female-low"
+        modelName = "en_GB-southern_english_female-low.onnx"
         ruleFsts = null
         lexicon = null
-        dataDir = null
-        lang = null
+        dataDir = "vits-piper-en_GB-southern_english_female-low/espeak-ng-data"
+        lang = "eng"

We give several detailed examples in the code. Please do read them before you build an APK.

Oh I see I did see those, though it seems I assumed that it would automatically swap out the details when built. I'll change that now, thank you.

Oh I see I did see those, though it seems I assumed that it would automatically swap out the details when built. I'll change that now, thank you.

It seems so so close, to functioning. I am getting this error which if i'm interpretting correctly means that it doesnt function with my two phones chipset?
logcat:
---------------------------- PROCESS STARTED (29519) for package com.k2fsa.sherpa.onnx.tts.engine ---------------------------- 2024-01-01 14:03:27.346 29519-29519 Compatibil...geReporter com.k2fsa.sherpa.onnx.tts.engine     D  Compat change id reported: 171979766; UID 10246; state: ENABLED 2024-01-01 14:03:27.351 29519-29519 ziparchive              com.k2fsa.sherpa.onnx.tts.engine     W  Unable to open '/data/app/~~7NVuguQ1TPkSsdQHxd54QQ==/com.k2fsa.sherpa.onnx.tts.engine-gl5i0272h3A4d7WKloPwuw==/base.dm': No such file or directory 2024-01-01 14:03:27.351 29519-29519 ziparchive              com.k2fsa.sherpa.onnx.tts.engine     W  Unable to open '/data/app/~~7NVuguQ1TPkSsdQHxd54QQ==/com.k2fsa.sherpa.onnx.tts.engine-gl5i0272h3A4d7WKloPwuw==/base.dm': No such file or directory 2024-01-01 14:03:27.523 29519-29519 nativeloader            com.k2fsa.sherpa.onnx.tts.engine     D  Configuring clns-4 for other apk /data/app/~~7NVuguQ1TPkSsdQHxd54QQ==/com.k2fsa.sherpa.onnx.tts.engine-gl5i0272h3A4d7WKloPwuw==/base.apk. target_sdk_version=34, uses_libraries=, library_path=/data/app/~~7NVuguQ1TPkSsdQHxd54QQ==/com.k2fsa.sherpa.onnx.tts.engine-gl5i0272h3A4d7WKloPwuw==/lib/arm64:/data/app/~~7NVuguQ1TPkSsdQHxd54QQ==/com.k2fsa.sherpa.onnx.tts.engine-gl5i0272h3A4d7WKloPwuw==/base.apk!/lib/arm64-v8a, permitted_path=/data:/mnt/expand:/data/user/0/com.k2fsa.sherpa.onnx.tts.engine 2024-01-01 14:03:27.530 29519-29519 Perf                    com.k2fsa.sherpa.onnx.tts.engine     I  Connecting to perf service. 2024-01-01 14:03:27.533 29519-29519 GraphicsEnvironment     com.k2fsa.sherpa.onnx.tts.engine     V  ANGLE Developer option for 'com.k2fsa.sherpa.onnx.tts.engine' set to: 'default' 2024-01-01 14:03:27.533 29519-29519 GraphicsEnvironment     com.k2fsa.sherpa.onnx.tts.engine     V  ANGLE GameManagerService for com.k2fsa.sherpa.onnx.tts.engine: false 2024-01-01 14:03:27.533 29519-29519 GraphicsEnvironment     com.k2fsa.sherpa.onnx.tts.engine     V  Neither updatable production driver nor prerelease driver is supported. 2024-01-01 14:03:27.534 29519-29519 NetworkSecurityConfig   com.k2fsa.sherpa.onnx.tts.engine     D  No Network Security Config specified, using platform default 2024-01-01 14:03:27.535 29519-29519 NetworkSecurityConfig   com.k2fsa.sherpa.onnx.tts.engine     D  No Network Security Config specified, using platform default 2024-01-01 14:03:27.545 29519-29519 Monotype                com.k2fsa.sherpa.onnx.tts.engine     V  SetAppTypeFace- try to flip, app = com.k2fsa.sherpa.onnx.tts.engine 2024-01-01 14:03:27.547 29519-29519 Monotype                com.k2fsa.sherpa.onnx.tts.engine     V      Typeface getFontPathFlipFont - systemFont = default#default 2024-01-01 14:03:27.757 29519-29519 Monotype                com.k2fsa.sherpa.onnx.tts.engine     V  SetAppTypeFace- try to flip, app = com.k2fsa.sherpa.onnx.tts.engine 2024-01-01 14:03:27.757 29519-29519 Monotype                com.k2fsa.sherpa.onnx.tts.engine     V      Typeface getFontPathFlipFont - systemFont = default#default 2024-01-01 14:03:27.761 29519-29519 sherpa-onnx-tts-engine  com.k2fsa.sherpa.onnx.tts.engine     I  Init Next-gen Kaldi TTS 2024-01-01 14:03:27.761 29519-29519 System.out              com.k2fsa.sherpa.onnx.tts.engine     I  data dir is vits-piper-en_US-glados 2024-01-01 14:03:31.321 29519-29519 System.out              com.k2fsa.sherpa.onnx.tts.engine     I  newDataDir: /storage/emulated/0/Android/data/com.k2fsa.sherpa.onnx.tts.engine/files 2024-01-01 14:03:31.341 29519-29519 sherpa-onnx             com.k2fsa.sherpa.onnx.tts.engine     W  config:                                                                                OfflineTtsConfig(model=OfflineTtsModelConfig(vits=OfflineTtsVitsModelConfig(model="/storage/emulated/0/Android/data/com.k2fsa.sherpa.onnx.tts.engine/files/vits-piper-en_US-glados/en_US-glados.onnx", lexicon="/storage/emulated/0/Android/data/com.k2fsa.sherpa.onnx.tts.engine/files/vits-piper-en_US-glados/", tokens="/storage/emulated/0/Android/data/com.k2fsa.sherpa.onnx.tts.engine/files/vits-piper-en_US-glados/tokens.txt", data_dir="/storage/emulated/0/Android/data/com.k2fsa.sherpa.onnx.tts.engine/files/vits-piper-en_US-glados/espeak-ng-data", noise_scale=0.667, noise_scale_w=0.8, length_scale=1), num_threads=2, debug=True, provider="cpu"), rule_fsts="", max_num_sentences=1) 2024-01-01 14:03:32.633 29519-29519 libc                    com.k2fsa.sherpa.onnx.tts.engine     W  Access denied finding property "ro.hardware.chipname" 2024-01-01 14:03:38.592 29519-29519 sherpa-onnx             com.k2fsa.sherpa.onnx.tts.engine     W  ---vits model---                                                                                sample_rate=22050                                                                                has_espeak=1                                                                                n_speakers=1                                                                                voice=en-us                                                                                model_type=vits                                                                                language=English                                                                                comment=piper                                                                                ----------input names----------                                                                                0 input                                                                                1 input_lengths                                                                                2 scales                                                                                ----------output names----------                                                                                0 output ---------------------------- PROCESS ENDED (29519) for package com.k2fsa.sherpa.onnx.tts.engine ----------------------------
It does this in a loop.  Im going to try rebuilt with a different app build.

https://k2-fsa.github.io/sherpa/onnx/tts/apk-engine.html
The above page contains TTS engine APKs for different languages.

https://k2-fsa.github.io/sherpa/onnx/tts/apk-engine.html
The above page contains TTS engine APKs for different languages.

You have done incredible mate. This is simply exactly what I have been wanting. I have lots of difficulty reading and the other tts options seem to be challenging to read along side. This though is awesome. I cannot express how greatful I am. I hope my persistent communication hasnt been difficult. You're a legend.

https://k2-fsa.github.io/sherpa/onnx/tts/apk-engine.html
The above page contains TTS engine APKs for different languages.

I LOVE it, would you please to consider to mantain a repo for F-Droid?
You work at Xiaomi, you are being payed to do it?


OCPP Params: Store a new Custom param in the configuration collection in DB

Custom Param in DB are now tagged with the boolean property 'custom' which identify them as such.
To test :

with a live Charging Station, enter a known custom parameter that would be accepted by the station.
verify in db that the property is set to true

This is the back-end component

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Feedback sent by email!!!!


Spark lines

Spark lines are the first item featured in the Readme, but there isn't a shortcut for building them. Just wanted to point this out since they do seem handy enough that they might merit a quick example. Thanks for creating this library!

Hey @carbocation, you probably found the answer already. But I had the same question and came across this issue.
In case if somebody else is having the same question — the answer is pretty simple because the only code example in readme IS the sparkline. Sparkline is the line chart.
import (
    ...
    "bytes"
    ...
    "github.com/wcharczuk/go-chart" //exposes "chart"
)

graph := chart.Chart{
    Series: []chart.Series{
        chart.ContinuousSeries{
            XValues: []float64{1.0, 2.0, 3.0, 4.0},
            YValues: []float64{1.0, 2.0, 3.0, 4.0},
        },
    },
}

buffer := bytes.NewBuffer([]byte{})
err := graph.Render(chart.PNG, buffer)

Will generate this:

That doesn't look like sparkline, but here's what exactly the same code will do with with real data:


[Feat]: Convert jpg, jpeg, png Files into webp Format🌟🌟 in gh-pages Branch Directory

Is your feature request related to a problem? Please describe.
Purpose of Raising this Issue

WebP is a modern image format that provides superior lossless and lossy compression for images on the web. Using WebP, webmasters and web developers can create smaller, richer images that make the web faster. WebP lossless images are 26% smaller in size compared to PNGs.

Describe the solution you'd like.

I have taken One PNG File from Your Repository. Observe in Screenshot, It's almost 89% of File Size Reduced While Converting PNG file into WEBP Format.
I want to work on those changes in Entire gh-pages Branch related Codebase Directory and It's Files.

Describe alternatives you've considered.
No Alternatives
Show us the magic with screenshots
Screenshot

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Hi @avinashkranjan . Please Review on this Issue.

Hi @avinashkranjan . Please Review on this Issue.


docker-export.sh produces a file with only container names

I appreciate this program, very handy! I'm also using WatchYourLan. These have both made a couple of my markdown files irrelevant, and that's a good thing!
I just want to share an issue I had with the docker-export.sh file. As it currently is, the only thing it produced for me is a file, /tmp/wyp-docker.txt, containing a list of all my docker container names. The information echoed to the command line was correct, but it didn't get saved to the file. I copied the list of container names to the hosts.yaml file, but it didn't seem to do anything for me.
It's also very possible I've misunderstood or done something wrong with the provided docker-export.sh file.
I'm sharing my modified version of the script that got me what I believe is the intended file.
This modified version outputs to a yaml file and cleans up the txt file. It has a done message, but doesn't output anything while executing. Even for my ~50 containers, run time is maybe a second or two, so it didn't seem necessary. My docker containers are on a Debian 12 host in case that's important. After I ran my modified version and did a scan, there were only two docker container ports it didn't catch.
Apologies if I messed up, misunderstood, or misread, something.
#!/usr/bin/env bash

# This script generates a WatchYourPorts config from Docker containers.

# HOW TO USE
#    1. Run this script on a server, where Docker is installed:
#   ./docker-export.sh $ADDR
#    $ADDR is IP or domain name of the server, without http(s):// prefix
#    It will be used to ping ports
#    2. Paste the output to hosts.yaml file in WYP config dir
#    3. You can add as many servers to hosts.yaml, as you want

docker ps -a --format "{{.Names}}">/tmp/wyp-docker.txt

{
  echo $1':'
  echo '    name:'
  echo '    addr: '$1
  echo '    portmap:'

  while read NAME; do
    PORT=$(docker inspect $NAME | grep HostPort | sed '1!d;s/"HostPort": //;s/,//;s/"//g')

    if [ ${#PORT} -ge 1 ]; then
      echo '       '$PORT':'
      echo '            name: '$NAME
      echo '            port: '$PORT
      echo '            state: false'
      echo '            watch: true'
    fi
  done </tmp/wyp-docker.txt
} > /tmp/wyp-docker-output.yaml

echo "Output has been saved to /tmp/wyp-docker-output.yaml"
rm /tmp/wyp-docker.txt

And thank you again for sharing this program!

I decided to go another route, because I like to control prints.
#!/bin/bash

# #
#   This script generates a WatchYourPorts config from Docker containers.
#
#   HOW TO USE
#   1. Run this script on a server, where Docker is installed:
#           ./docker-export.sh $ADDR
#       $ADDR is IP or domain name of the server, without http(s):// prefix
#       It will be used to ping ports
#   2. Paste the output to hosts.yaml file in WYP config dir
#   3. You can add as many servers to hosts.yaml, as you want
#
#   If you get the error:
#       bash: ./ports_docker_import.sh: /bin/bash^M: bad interpreter: No such file or directory
#   Run the command:
#       sudo sed -i -e 's/\r$//' ports_docker_import.sh
# #

# #
#   Output File
# #

file_output="$(pwd)/data/wyp/hosts.yaml"

# #
#   Permissions
# #

sudo touch $file_output
sudo chmod 0644 $file_output

# #
#   Docker Container List
# #

CONTAINERS=$(docker ps -a --format "{{.Names}}")

# #
#   Build Host Config
# #

echo "$1:" | tee $file_output
echo "    name: $2" | tee -a $file_output
echo "    addr: $1" | tee -a $file_output
echo "    portmap:" | tee -a $file_output

# #
#   Iterate docker container names
# #

while IFS= read -r id; do
    PORT=`docker inspect $id | grep HostPort | sed '1!d;s/"HostPort": //;s/,//;s/"//g' | tr -d '[:space:]'`

    if [ ${#PORT} -ge 1 ]; then
        echo "        $PORT:" | tee -a $file_output
        echo "            name: $id" | tee -a $file_output
        echo "            port: $PORT" | tee -a $file_output
        echo "            state: false" | tee -a $file_output
        echo "            watch: true" | tee -a $file_output
    fi
done <<< "$CONTAINERS"

Also supports the name of the host
./ports_docker_import.sh xx.xx.xx.xx domain.com


Add nested delegate call warnings

What it solves
Resolves #3115
How this PR fixes it
Nested delegate calls now display that they are a) a delegate call and b) (un-)expected. They also, like #3091, base their legitimacy on whether the address is known or not.
An unexpected, nested delegate call can be found here.
How to test it
@mikheevm, can you give an overview of how to create nested delegate calls please.
Screenshots
(The following "parent" delegate calls are "unexpected" because our MultiSend contract is not a known address on Rinkeby.)

The known address name was set manually here in order to trigger the warning:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 32.463%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/safe/components/Transactions/TxList/MultiSendDetails.tsx
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/routes/safe/components/Transactions/TxList/MultiSendDetails.tsx
1
4.26%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
3086

Relevant Lines:
8470

💛  - Coveralls

Can we remove the extra gap and add a gap in the action section?

I don't have a way to reproduce a tx like the one in the example.
The tx given as example works fine in this PR and shows the warning message so I'm ok with passing this ticket.
@katspaugh Do you think we should try to find a contract and reproduce a similar tx?

Merging as confirmed to work by @mikheevm.


connect to local docker services

Right now I have openvpn running on my host machine.  My host machine has other docker container running on the local network.  I would like to connect to the docker container services once I am connceted to openvpn.  For example I would like to access the jellyfin service when I connect to the vpn, but the browser does not connect to the service.
These are the steps I ran to setup openvpn:
docker-compose run --rm openvpn ovpn_genconfig -u udp://xxx.xxx.xx.xxx:1194 -N -d -c -p "route <IP_ADDRESS> <IP_ADDRESS>" docker-compose run --rm openvpn ovpn_initpki export CLIENTNAME="ipro3" docker-compose run --rm openvpn easyrsa build-client-full $CLIENTNAME nopass docker-compose run --rm openvpn ovpn_getclient $CLIENTNAME > $CLIENTNAME.ovpn docker-compose up -d openvpn

I am also experiencing this.

If you're trying to connect via hostname you'll need to setup a dns proxy also.

I created a concept using dnsmasq as DNS proxy. See here: https://github.com/kylemanna/docker-openvpn/issues/622
Works for one network.


Script / Cron Help

I need to write a bash script that will search for a folder check all files with .mkv .avi or .mp4 extension then check if it has a file with the same name but the lang.srt extension next to it and if not run subliminal for that specific video file. And a cron job that will run that script once an hour.
Could anyone help me with that?
I know that for searching the video and executing the command would be something like that
#!/bin/sh find /mnt/data/tv/ \( -iname "*.mkv" -or -iname "*.avi" -or -iname "*.mp4" \) -exec subliminal -l pt-BR -- "{}" \;
And the cron would be something like
0 * * * * sh /mnt/data/tvsubs.sh
I just need to filter out the specific files I need so subliminal only runs for the necessary files.
Also would be nice to exclude certain folders that I know that doesnt have subtitles available for them.

Subliminal already does all that by default. Please refer to the documentation: http://subliminal.readthedocs.io/en/latest/user/usage.html#cli.
Also every hour is overkill and will likely get you banned from subtitles providers. I suggest you run that once or twice a day and use the --age parameter.


Use the proper Big types in the IBM prior

Codecov Report

Merging #35 (13379a2) into main (159409e) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
+ Coverage   93.12%   93.16%   +0.04%
==========================================
  Files          19       19
  Lines         640      644       +4
==========================================
+ Hits          596      600       +4
  Misses         44       44

Impacted Files
Coverage Δ

src/priors.jl
100.00% <100.00%> (ø)

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 159409e...13379a2. Read the comment docs.


A few test env improvments

As well as docs
@roymarantz @maddalab @byxorna @defect

:+1:  got for it.


Css Styles are not applied to the html editor

I have used the initialText parameter to load my HTML string to get the formatted text. But, the style I applied to the html is not applied in the editor.

Dharani

I also tried using the separate style.css file and mapping it to the HTML and that too does not works.
Is any way to achieve this.
Thanks in advance.

try to change from  .paragraph{color: #008000;} to  p {color: #008000;}


ISPN-5647 ClassCastException on transactional cache

https://issues.jboss.org/browse/ISPN-5647

PR Updated

Pulling...

Integrated into master, thanks @gustavonalle !

Testing 7.2.x real quick then pushing there too.


Normalize YAML in Travis CI config

The YAML was not quite valid. Travis can read it
and wasn't complaining, but this ensures that the YAML
is clean.

Looking at this a second time, the YAML was valid, but I ran it through a script which normalized it and I'm trying to make all the travis scripts consistent across all the language tracks.


ITK_MINIMUM_COMPLIANCE_LEVEL != Module not found

Description
A module blocked by Compliance Level

doesn't generate an error during ITK compilation
generates an seemingly unrelated error when that module is attempted to be used by package being compiled against that ITK
if I am using a build directory from a version of ITK from a few weeks ago and just do a git update master and recompile, the above error does NOT happen, and all appears to work...waiting for me to remove my build directory when the above errors will then appear

Steps to Reproduce
When configuring ITK with the command:
HEAD is now at 1554b66a55 Merge pull request #1262 from SimonRit/shadow_warning_eigen
+ cd ..
+ mkdir ITK-Debug
+ cd ITK-Debug
+ cmake -GNinja -DBUILD_EXAMPLES=OFF -DBUILD_TESTING=OFF -DModule_MinimalPathExtraction=ON -DCMAKE_BUILD_TYPE=Debug ../ITK
No error / warning is ever reported, BUT since MinimalPathExtraction is at Compliance Level 2 and the default Minimum Compliance Level is 4, the MinimalPathExtraction module is not actually built!
However, when I go to compile ITKTubeTK against that ITK and have MinimalPathExtraction as a required component (via find_package), I get the subtle error
CMake Error at /home/vsts/work/1/ITK/CMake/ITKModuleAPI.cmake:78 (message):
  No such module: "MinimalPathExtraction"
Call Stack (most recent call first):
  /home/vsts/work/1/ITK/CMake/ITKModuleAPI.cmake:31 (itk_module_load)
  /home/vsts/work/1/ITK/CMake/ITKModuleAPI.cmake:129 (_itk_module_config_recurse)
  /home/vsts/work/1/ITK-Debug/ITKConfig.cmake:83 (itk_module_config)
  CMakeLists.txt:135 (find_package)

"No such module" is really hard to track down.   The error is reported in CMake/ITKModuleAPI.cmake, but the real problem to fix the error is the compliance level test in a different file: CMake/ITKRemoteModuleAPI.cmake.
One hour later, after dashboards were failing but local build was happily working, I figured this out and realized that had to specify -DITK_MINIMUM_COMPLIANCE_LEVEL=1 when compiling ITK.
Recommendations

Throw an error if a module is requested that doesn't meet minimum compliance
Throw a different error that is descriptive if a module is requested and it is not possible to add it because it doesn't meet the minimum_compliance_level.

This has actually gotten a bit worse.
Now it isn't possible, via cmake-gui, to enable any of the modules below the default compliance level (I think that is what is happening).
Even after using cmake-gui to lower the ITK_MINIMUM_COMPLIANCE_LEVEL to 1 and then re-configuring, none of the additional modules appear in the list.   See the screenshot below.    As a result, none of my code that depends on ITK and its remote modules (e.g., MinimalPath...) can compile at this time!!!

@aylward I did not see this 6 days ago... sorry.  I'm dealing with ISBI crisis early this week, but will look into fortifying the process later in the week.
I still feel strongly that it is important to find a way to hide/designate the intended quality of the remote modules.  Knowing the expected quality and what "should" compile with high confidence is key to allowing the remote modules to act as a testbed for core changes.
I think there is a quick fix that will simply turn everything back on.  I'll try to deal with that late tonight.

Hi,
I appreciate what you are trying to do.  It is a good idea to pursue.
Perhaps the level limit and the level of each module is currently too
subtle?   For example, I believe that compliance level and each Module_XXXX
variables should be a standard (not advanced) cmake variables?    Perhaps
all should be listed, but if enabled, an error message is thrown indicating
it is below the compliance level?
s
On Mon, Mar 9, 2020 at 8:21 AM Hans Johnson<EMAIL_ADDRESS>wrote:

@aylward https://github.com/aylward I did not see this 6 days ago...
sorry. I'm dealing with ISBI crisis early this week, but will look into
fortifying the process later in the week.
I still feel strongly that it is important to find a way to hide/designate
the intended quality of the remote modules. Knowing the expected quality
and what "should" compile with high confidence is key to allowing the
remote modules to act as a testbed for core changes.
I think there is a quick fix that will simply turn everything back on.
I'll try to deal with that late tonight.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/InsightSoftwareConsortium/ITK/issues/1675?email_source=notifications&email_token=AACEJL4NYG6KCXV26BZ5B6LRGTUNPA5CNFSM4LAVISYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOG4MNY#issuecomment-596493879,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEJL3CFV5CGP55ZQCLAW3RGTUNPANCNFSM4LAVISYA
.

--
Stephen R. Aylward, Ph.D.
Senior Director of Strategic Initiatives
Kitware: Advancing the frontiers of understanding
by developing innovative open-source software platforms
and integrating them into research, processes, and products.

We have encountered the same problems with the RTK module and I haven't been able to comment on this change before (sorry @hjmjohnson!). Here is probably a good place. The minimum compliance level is quite a drastic change and after feeling very welcomed with an easy integration of RTK as a remote module in ITK, the feeling is a bit opposite with this new policy. Yet, I fully understand and respect it, it's good to push everyone to improve ITK code, including the remote one. We'll try to meet the new standard to have the Module_RTK option directly available without having to change the ITK_MINIMUM_COMPLIANCE_LEVEL (which I prefer). Personally, I would suggest

to start with a lower level than 4 (e.g., 3) in the next release and let the community know that the level will be raised to 4 in the following one. This gives us more time to implement the requested changes for those who want to keep their module visible with the default options.
to only display a warning if a module below the minimum compliance level is activated from the command line instead of the current behavior. I also find it a bit strange that the passed option is silently ignored.

In any case, we will adjust and we are still happier with the current situation than with the one before Module_RTK.cmake was added!

A reasonable expectation when passing -DModule_<RemoteModule> in a command line (non-interactive) configuration is that it will enable <RemoteModule> without any other steps or variables.

That is a great idea!   If someone knows the name, they probably know that
they want / need it.
s
On Mon, Mar 9, 2020 at 3:25 PM Matt McCormick<EMAIL_ADDRESS>wrote:

A reasonable expectation when passing -DModule_ in a
command line (non-interactive) configuration is that it will enable
 without any other steps or variables.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/InsightSoftwareConsortium/ITK/issues/1675?email_source=notifications&email_token=AACEJL4Q7JN4E3RZ7BEG2KDRGVGDZA5CNFSM4LAVISYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOIXICQ#issuecomment-596734986,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEJL622IR3QB4QAJRICKTRGVGDZANCNFSM4LAVISYA
.

--
Stephen R. Aylward, Ph.D.
Senior Director of Strategic Initiatives
Kitware: Advancing the frontiers of understanding
by developing innovative open-source software platforms
and integrating them into research, processes, and products.

I think is has been fixed.  Please re-open if the issue persists.

Thanks @hjmjohnson !


Tar files + Globus -> TACC, Part 3

Follow-on from https://github.com/AgPipeline/data/issues/14

[x] Try activating endpoint according to Sean's instructions below Never mind. We're just going to use a different location for the tar & md5 files. Update: Got it working by using my NCSA identity.
[x] Test transfer from ncsa#terra to TACC Mig2 Ranch with XSede Authentication
[ ] Start batching and transferring to TACC
[ ] Compile notes from Slack into Markdown and submit to Sean for review
[ ] Get grep for season from Sean
[ ] Sean: Talk to Rob about deletion policy/migration plan

Regarding endpoint Sean is using, he says:

Sean Stevens - 2020-06-08 16:08 (UTC): ok, you probably need to go to "endpoints" on the left bar.  Then, select ncsa#terra and "deactivate" it.  Then you'll want to reactivate, but when you reactivate, use your NCSA logon.  You might have to deactivate several times. Globus can be a little weird

Julian  4:15 PM
Did you see the endpoint that David & Rob sent me in the @ncsa_internal group?
4:15
Is that different?
Sean Stevens  4:15 PM
once you have an endpoint somewhere authenticated using DUO, it tries to use those credentials everywhere
Julian  4:15 PM
Huh. OK.
Sean Stevens  4:15 PM
I'll take a look
4:18
the one Rob sent da262cbf-6d04-11e5-ba46-22000b92c6ec is the ncsa#terra endpoint.  The one David sent is the shared endpoint, Terraref,  shared from ncsa#terra.  It's top level directory is /terraref/sites
Julian  4:22 PM
So, I should be able to use the one David sent me for using as the origin of the tar files?(I can probably answer this myself by logging in to the server and checking.)
4:25
Ah. No. Doesn't look like it.
4:26
/terraref/users/scratch/archive_work/
Sean Stevens  4:26 PM
tar files are under /terraref/users/scratch... so it won't be visible in the one David sent.  If we changed where the tar files got generate, that one could be used.
4:26
yeah
Julian  4:26 PM
Snap.
4:26
That's what I was wondering. OK.
New
Sean Stevens  4:27 PM
if that'd be easier, we could probably add a tars directory somewhere under the /terraref/sites/ and then just use that as the output space for the .tar and .md5sum files or something like that

The endpoint has stopped working:
$ globus endpoint activate 403204c4-6004-11e6-8316-22000b97daec
Endpoint is already activated. Activation expires at 2020-06-10 16:39:11+00:00

$ globus ls 403204c4-6004-11e6-8316-22000b97daec:/
Globus CLI Error: A Transfer API Error Occurred.
HTTP status:      502
request_id:       PCGlTAHXH
code:             ExternalError.DirListingFailed.LoginFailed
message:
                  Command Failed: Error (login)
                  Endpoint: Terraref (403204c4-6004-11e6-8316-22000b97daec)
                  Server: cg-hm02.ncsa.illinois.edu:2811
                  Message: Login Failed
                  ---
                  Details: globus_ftp_control: gss_init_sec_context failed\nGSS Major Status: Unexpected Gatekeeper or Service Name\nglobus_gsi_gssapi: Authorization denied: The name of the remote entity (/DC=org/DC=incommon/C=US/ST=Illinois/L=Urbana/O=University of Illinois/OU=NCSA/CN=cg-hm02.ncsa.illinois.edu), and the expected name for the remote entity (/DC=org/DC=incommon/C=US/ST=IL/L=Urbana/O=University of Illinois/OU=NCSA/CN=cg-hm02.ncsa.illinois.edu) do not match\n

A: /DC=org/DC=incommon/C=US/ST=Illinois
B: /DC=org/DC=incommon/C=US/ST=IL

The endpoint started working again. Sean said:

There was a new certificate that went into place, but it looks like the Subject DN was generated with Illinois instead of IL.   On the Globus side it will capture that when the server is added to the endpoint.  Had to reset that on the Globus side.  Should be happy now.

Achievement unlocked: Successfully transferred test files from NCSA to TACC.

Update from Sean about deletion policy:

I did talk to Rob. Sounds like he's still in discussions regarding deletion.

Instructions condensed from conversation with Sean Stevens:
https://hackmd.io/@julianp-diag/SkKtF09TI
/terraref/users/scratch/archive_work/file_lists
Break down by the larger sensors:

scanner3DTop
stereoTop
SWIR
VNIR

Then group the smaller sensors together:

co2Sensor
cropCircle
EnvironmentLogger
flirIrCamera
lightning
ndviSensor
priSensor
weather
ps2Top

(To see the grouping of smaller sensors see: /terraref/users/scratch/archive_work/file_lists/raw_S3/sensor_group.txt)
Do a find a 2 levels down to get the best breakdown to split on for the tar size. Look at stereoTop.du as an example.
Example:
find /terraref/sites/ua-mac/raw_data/stereoTop -maxdepth 2 -mindepth 2 -type d

Use grep to deal with grabbing the right date range for teh season and dump that to the .du file.
Run du -ms in a loop against that file to get the output:
for i in $(cat sensor.du); do du -ms ${i}; done

See /terraref/users/scratch/archive_work/file_lists/raw_S3/stereoTop/stereoStop.du.out for example output.
Run split_size.pl against the *.du.out with the chosen tar file size. It will generate the *_split.P* (part) files.
Feed the *_split.P* files to the gen_tar.sh script. For example:
/terraref/users/scratch/archive_work/scripts/gen_tar.sh arch_ua-mac_raw_data_VNIR_S2.p22 /terraref/users/scratch/archive_work/file_lists/raw_S2/VNIR/VNIR_split.P22

Run this manually, one by one, per split, to generate a few tar files (as many as possible without filling the disk).
Note: The gen_tar.sh takes a fair bit of time because it goes back and generates an md5sum for every file in the tar file once it's completed.
Kick off the Globus transfers for the generated tar files and the corresponding .md5 files.
Once successfully transferred using Globus, delete the local tar files.
Repeat till the whole season is tarred and transferred.
Repeat for all the seasons.

Did not get to actual batching and transferring. Follow-up issue: https://github.com/AgPipeline/data/issues/16


Null Characters erroring in Postgres

Durign a routine security check we found an interesting bug where Postgres errors due to Null bytes E.g. %00 and as a result we get back an error detailing the query (we are in process now of swapping to a custom error message).
Given this is a Postgres limitation it seems to make sense for the ORM to check for this type of character and either strip it or return an error before it even gets to the database.
<EMAIL_ADDRESS>A scenario is selecting a record based on an email address e.g<EMAIL_ADDRESS>will return the following error message from Postgres due to the %00 being considerd a null byte.
"select * from "tableName" where lower("email") = $1 - invalid byte sequence for encoding "UTF8": 0x00"

This definitely shouldn't be done in knex and that also sounds like a perfectly good error message from postgres side, that data which was sent is not good.
It is application code's responsibility to cleanup user input if that is wanted behaviour, I would suggest using some validator library for that.  Many times 0x00 is perfectly fine value (when not used in utf8  string).