Pex migration issues with default group and chat prefixes

As of LuckPermsSponge-2.15.5.jar, after migrating from Pex, we're having several issues.
In Pex, a user with no group is defaulting to the "member" group. Also, prefix are prioritized using the "weight" option. Here's an extract from our Pex configuration file:
{
    "subjects": {
        "default": {
            "default": [
                {
                    "contexts": {
                        "localip": "<IP_ADDRESS>"
                    },
                    "permissions-default": 0
                },
...
            "user": [
                {
                    "parents": [
                        "group:member"
                    ],
                    "permissions-default": 0
                }
            ]
        },
        "group": {
            "default": [
                {
                    "options": {
                        "prefix": "&7Guest "
                    },
                    "permissions-default": -1
                }
            ],
            "member": [
                {
                    "options": {
                        "weight": "99999",
                        "rank-ladder": "staff",
                        "rank": "1000",
                        "prefix": "&7Member "
                    },
                    "permissions-default": 0,
...
                }
            ],
            "moderator": [
                {
                    "parents": [
                        "group:minimod"
                    ],
                    "options": {
                        "weight": "10400",
                        "rank-ladder": "staff",
                        "rank": "400",
                        "prefix": "&bMod "
                    }
                }
            ],
            "minimod": [
                {
                    "options": {
                        "weight": "10450",
                        "rank-ladder": "staff",
                        "rank": "450",
                        "prefix": "&bMiniMod "
                    },
                    "parents": [
                        "group:member"
                    ]
                }
            ],
...

After migrating to LP, I've noted the following issues:
1- users with no group defined, are attached to the "default" group.
To fix this, I've added "member" group has a parent of "default" group
and I've cleared the "default" group prefix.
2- staff users (a moderator) remain with the "Member " prefix.
From what I understand, all chat prefixes from all groups have the same priority of 100 in LP.
Looks like Pex weight needs to be converted into "chat priority" during migration.
3- I can't remove the migrated prefixes with commands:
>lp group moderator meta info
[19:59:47] [Server thread/INFO] [nucleus]: Server ran the command: /lp group moderator meta info
[19:59:47] [pool-3-thread-38/INFO]: [LP] moderator's Prefixes
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> 100 - "Member " (inherited from member)
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> 100 - "MiniMod " (inherited from minimod)
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> 100 - "Mod " (inherited from self)
[19:59:47] [pool-3-thread-38/INFO]: [LP] moderator has no suffixes.
[19:59:47] [pool-3-thread-38/INFO]: [LP] moderator's Meta
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> rank = "450" (inherited from minimod)
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> rank-ladder = "staff" (inherited from self)
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> weight = "10450" (inherited from minimod)
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> weight = "99999" (inherited from member)
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> rank = "1000" (inherited from member)
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> weight = "10400" (inherited from self)
[19:59:47] [pool-3-thread-38/INFO]: [LP] -> rank = "400" (inherited from self)
>lp group moderator meta removeprefix 100 "Mod "
[19:59:59] [Server thread/INFO] [nucleus]: Server ran the command: /lp group moderator meta removeprefix 100 "Mod "
[19:59:59] [pool-3-thread-42/INFO]: [LP] moderator doesn't have that prefix set.
>lp group moderator meta addprefix 10450 "Mod "
[20:00:07] [Server thread/INFO] [nucleus]: Server ran the command: /lp group moderator meta addprefix 10450 "Mod "
[20:00:07] [pool-3-thread-40/INFO]: [LP] moderator had prefix "Mod " set at a priority of 10450.
[20:00:07] [pool-3-thread-40/INFO]: [LP] (Group data was saved to storage)
>lp group moderator meta removeprefix 10450 "Mod "
[20:00:14] [Server thread/INFO] [nucleus]: Server ran the command: /lp group moderator meta removeprefix 10450 "Mod "
[20:00:14] [pool-3-thread-40/INFO]: [LP] moderator had prefix "Mod " at priority 10450 removed.
[20:00:14] [pool-3-thread-40/INFO]: [LP] (Group data was saved to storage)

Your fix to No. 1 is the best approach. It's documented in the wiki under the "default groups" section.
I'll look into trying to migrate weights. It's actually somewhat difficult to do though - that data doesn't get exposed directly to Sponge.
The issue with No. 3 is probably because of color codes? Not too sure. You could also use /lp group moderator meta removeprefix 100 null to remove all prefixes set at 100 priority.

For issue 1, a warning or something would help.
For issue 2, the weight is visible in LP, so I assumed LP could transfer it back to chat priority, or did I miss something?
For issue 3, the color code isn't escaped neither the color is rendered in console. Could we have either of those?.

Color rendering in Console just doesn't work for SpongeForge - it works for all other platforms.
As for the migration thing, I can't really add warnings for it. I simply cannot automate absolutely everything. The idea with migration is to be an aid, not just a tool to blindly move to LP. I expect users to read the wiki as well as using the migration commands. :)
Will work on the weight stuff soon.

You were going to get back to me about whether it's weight or rank. Any ideas?

I've tested SpongePex for prefix priority but couldn't make much sense of it so far.

One suggestion through, herited groups with the same priority shouldn't take priority on the child groups.
For example, minimod is the parent of the mod group, both having prefix with priority 100. Then mod prefix should prevail.

Then the mod prefix needs to have a higher priority

I mean it would be simpler if it worked by default as "my current group prefix takes priority over my parent prefix".
The same way it does with "player prefix takes priority over any group prefix".

That would go against the whole meta priority system :/

I'm talking only when priorities are the same. Obviously, if priorities are different, they should take precedence.

Sorry - again late reply...
Why did you mean by this: "my current group prefix takes priority over my parent prefix"
Not sure what you mean by "current" group - do you mean primary group? or something else.

I mean that inheritance should have less priority.
My supermod group has a parent group named moderator, which has a parent group named minimod.
When I set a prefix for each group with same priority, I expect the child group to overwrite its parent meta.

That should happen already? Are you sure it doesn't work?

Pretty sure it doesn't work

Still an issue as of LuckPerms-Sponge-2.16.24.jar

[14:14:43] [Server thread/INFO]: Admin LemADEC: t
[14:14:51] [Server thread/INFO] [nucleus]: LemADEC ran the command: /lp group member     meta addprefix 800 "&7Member "
[14:14:52] [pool-3-thread-31/INFO]: [LP] LOG > (LemADEC) [G] (member) --> meta addprefix 800 "Member "
[14:14:53] [Server thread/INFO]: Member LemADEC: t
[14:15:02] [Server thread/INFO] [nucleus]: LemADEC ran the command: /lp group member     meta removeprefix 800 "&7Member "
[14:15:02] [pool-3-thread-30/INFO]: [LP] LOG > (LemADEC) [G] (member) --> meta removeprefix 800 "Member "
[14:15:03] [Server thread/INFO]: Admin LemADEC: t

Are you a member of "admin" as well as "member"?

[15:35:51] [Server thread/INFO] [nucleus]: Server ran the command: /lp user LemADEC info
[15:35:51] [pool-3-thread-41/INFO]: [LP] > User Info: lemadec
[LP] - UUID: 92aae7f5-ebd8-46c7-b0b9-09c62643d64d
[LP] - Status: Offline
[LP] - Primary Group: default
[LP] - Counts:
[LP] -    Permissions: 3
[LP] -    Temporary Permissions: 0
[LP] -    Prefixes: 0
[LP] -    Suffixes: 0
[LP] -    Meta: 1
[15:35:51] [pool-3-thread-41/INFO]: [LP] - Parent Groups:
[15:35:51] [pool-3-thread-41/INFO]: [LP] -    > default
[15:35:51] [pool-3-thread-41/INFO]: [LP] -    > admin
[15:35:51] [pool-3-thread-41/INFO]: [LP] - Cached Data:
[LP] -    Is Loaded: false
[LP] -    Current Contexts: None
[LP] -    Current Prefix: None
[LP] -    Current Suffix: None

[15:38:18] [Server thread/INFO] [nucleus]: Server ran the command: /lp group default parent info
[15:38:18] [pool-3-thread-42/INFO]: [LP] default's Parent Groups:
> member
>
[15:38:18] [pool-3-thread-42/INFO]: [LP] default's Temporary Parent Groups:
None

member is a parent of default which is always parent of all users. Hence, ones need to force default prefix priority to very low value for other inheritances to work?

Correct.
The "default" tree is being applied before the "admin" one.
Download the latest CI build and run /lp group default permission set group.weight.-1 true. Should fix your problem.
Please feel free to re-open if you can make any sense of the PEX rank system regarding migration. Sorry, I really just don't have the time to experiment with it myself. :/


According to the doc output is System.IO.FileInfo but cmdlet returns nothing

According to the doc output is System.IO.FileInfo but cmdlet returns nothing
https://technet.microsoft.com/en-us/library/dn841358.aspx

The behavior is addressed via https://github.com/PowerShell/Microsoft.PowerShell.Archive/pull/18, however this is now a doc bug.  I don't think we want to change the default behavior as it would break existing scripts.

This was fixed in MicrosoftDocs/PowerShell-Docs#3696 on Feb 12. You need to run Update-Help -Force to get the new help files.


Number.isInteger() support

https://typescripttolua.github.io/play/#code/HIVwtgRgpgTgdASwM4EkB2AXKBzWAKAZgEog

it'd be nice to have 1:1 interop with javascript's Number.

Would love to have a look at this sometime


Small code improvement, using exist_ok=True

Using os.makedirs(filepath, exist_ok=True) instead 2 lines of if
block.

I would like to add this, but unfortunately that parameter was added in python 3.x and we need to support at least back to python 2.7. You can see this in the failing Travis CI build https://travis-ci.org/michael-lazar/rtv/jobs/205695887


is this possible?

is it possible to show a specific tag in the filename, only when the image has the tag.
Like I want all images that do have an incest tag, that they show it in the filename.
So when a pic has the tag it should look like this:
original - niliu chahui - original character+tokisaki mio+tokisaki asaba - incest - 21.01.2017 - 5824978 - 7a0d927af08f879858961d267e36b743
and when it doesn't have the tag, it should just ignore it:
original - niliu chahui - original character+tokisaki mio+tokisaki asaba - 21.01.2017 - 5824978 - 7a0d927af08f879858961d267e36b743

There's two options for this. You can add a "Conditional filename" in the settings, with the "incest" tag as a condition, and the filename you want as value. Or you can use tag conditionals directly in your main filename, with something that looks like: %artist% - %copyright<- "incest"> - %date%. Everything between the brackets will only get inserted if the "incest" tag is found.
You should read the Filename wiki page for more details.

I don't get it. What should I put in the condition field and what in the filename field?

thanks it works, I forgot to update the group


Use Emgu.Cv in QR Code Tests

Rewritten the QR Code tests. Self-explanatory.

Going with OpenCVSharp instead of Emgu.CV


Add note reg. free of charge PCR test to the FAQ entry rat_red_card_which_test

This PR adds a note about the free of charge PCR test, which is possible if CWA shows an encounter with increased risk, to the FAQ entry rat_red_card_which_test.

German
English

@Ein-Tim
Can you provide the legal reference for your statement? "An encounter with increased risk in the Corona-Warn-App allows you to take a free PCR test."
It might be better to open an issue first and check what legal statement can or should be made in the FAQs. That would probably need to be approved by RKI.

@MikeMcC399
This statement is already in the FAQ, here for example: https://www.coronawarn.app/en/faq/#vac_cert_red_warning
The legal basis is §2 Absatz 2 Ziffer 5 of the Coronavirus-Testverordnung – TestV which is available here https://www.bundesgesundheitsministerium.de/fileadmin/Dateien/3_Downloads/C/Coronavirus/Verordnungen/Corona-TestV_BAnz_AT_09.03.2021_V1.pdf

@Ein-Tim

This statement is already in the FAQ, here for example: https://www.coronawarn.app/en/faq/#vac_cert_red_warning

Thanks for pointing out that the FAQs already have that information.
It seems that the Coronavirus-Testverordnung – TestV version you quoted will be replaced again by:
"Verordnung zum Anspruch auf Testung in Bezug auf einen direkten Erregernachweis des Coronavirus SARS-CoV-2 vom 21.09.2021
Inkrafttreten: 11.10.2021"
(See https://www.bundesgesundheitsministerium.de/service/gesetze-und-verordnungen.html)
which I think says the same as before on this topic, but anyway I prefer to leave any legal interpretation to the experts! 🙂

@MikeMcC399

[...] I prefer to leave any legal interpretation to the experts!

Same here, I just opened this PR because the statement is already in another FAQ entry, so I'm quite sure it's not a problem to also mention it in rat_red_card_which_test.
I will wait for feedback from the Open-Source Team on this now.
Have a nice Sunday!

Thanks for evaluation of this topic.
I confirm, our first statement was made based on the former "Corona_Testverordnung", which now was updated in 2021. Still, the updated version describes the same supporting reasoning. Therefore, we are confident, the above-mentioned statement is still valid.

@GisoSchroederSAP
I will add this sentence when I have time, probably by today's evening.
Thanks for checking and approving!

@GisoSchroederSAP
Thanks for adding your comment about the doctor! It was my concern that the legal text says:
"Wenn  von  einem  behandelnden  Arzt  einer  mit  dem  Coronavirus  SARS-CoV-2  infizierten  Person  oder  vom öffentlichen  Gesundheitsdienst  asymptomatische  Kontaktpersonen  nach  Absatz  2  festgestellt  werden,  die  in  den letzten  14  Tagen  Kontakt  zu  einer  mit  dem  Coronavirus  SARS-CoV-2  infizierten  Person  hatten,  haben  diese  Anspruch auf  Testung."
so in my view the free test is only valid in conjunction with a doctor or the health authority. I didn't think that the app on its own had the legal authority to grant a free PCR test. So that is clarified by your additional text.
In your suggestion:

EN: Please confer with your doctor for the further procedure.

I think there is a typo. It should be "... Ihren Arzt ..." I think, or "... Ihren Arzt / Ihre Ärztin ...".

Actually, the Testverordnung clearly stated, that a person with a high risk warning is entitled . However, without a doctor's consultation it will be difficult to enforce the free testing - besides the fact, that the doctor can examine and evaluate other factors as well.
Therefore, I think, it is a valuable hint to guide the user here.
And yes, this was a late-night-typo ;-) It should be "Bitte konsultieren Sie ihre Ärztin/ Ihren Arzt für die weiteren Schritte." Thanks for pointing out.

@GisoSchroederSAP
I added your suggestion in the latest commit (ff9ed62 (#1767)).

@GisoSchroederSAP
Thank you! I created PR #1872 which removes the wrong information from the FAQ and replaces it with the recommendation to confer with the doctor.


test: Allow Cilium 1.4 to be run with K8s 1.14

This PR makes it possible to run upgrade tests on k8s 1.14 with Cilium 1.4.

This change is

test-me-please

test-me-please

Coverage increased (+0.004%) to 42.062% when pulling ef36d46cfbba1d0aa731fbf2b0c00b018c92f0e5 on pr/brb/bump-k8s-vsn-constraint into 4011e5af7e90fc20b3f163dee281cc1c6ca41ec2 on master.

CI failed due to #7615 and #7721.

test-me-please

Already applied to v1.4 by @eloycoto  removing need-backport tag.


Allow creating a debug callback at instance creation

Since this is now allowed by the specs.

Implemented by #1878.


haskell shellFor: Fix hoogle

See https://github.com/NixOS/nixpkgs/pull/76349#issuecomment-584703160
CC @pjones @teto

that fixed it for me. If you deem it worth, you could add a test to check the hoogle database exists in pkgs/test/haskell-shellFor/default.nix

I've also confirmed this works.
@Ericson2314 Thanks!


Tag compatibility with tactician 1.0

Hi!
Is there anything I can help to get this project to release a version compatible with tactician 1.0?
Right now I see tests passing with 1.0 and only thing that doesn't look covered is Traits and config.
Thanks!

I will tag a 1.0.0 release later tonight (it's not different from the current version except that it would depend on tactician 1.0.0)

Done https://github.com/madewithlove/tactician-laravel/releases/tag/1.0.0


fix temptifly wizard patternfly layout

Signed-off-by: James Talton<EMAIL_ADDRESS>Issue:  https://github.com/open-cluster-management/backlog/issues/12511

/hold

/unhold


Convert theano weights to tensorflow format

I would like to use this pretrained model: https://gist.github.com/baraldilorenzo/07d7802847aaad0a35d3
It is in theano layout, my code depends on tensorflow image dimension ordering.
There is a guide on converting weights between the formats: https://github.com/fchollet/keras/wiki/Converting-convolution-kernels-from-Theano-to-TensorFlow-and-vice-versa
But this seems broken. In the section to convert theano to tensorflow, the first instruction is to load the weights into the tensorflow model.

Keras backend should be TensorFlow in this case.
First, load the Theano-trained weights into your TensorFlow model:
model.load_weights('my_weights_theano.h5')

This raises an exception, the weight layouts would be incompatible.
And if the load_weights function would take theano weights for a tensorflow model, there wouldn't be any need to convert them.
I took a look at the convert_kernel function. To see, wether I could do the necessary steps myself: https://github.com/fchollet/keras/blob/master/keras/utils/np_utils.py
The code is rather simple - I don't understand why the guide makes use of a tensorflow session. That seems unnecessary.
I've copied the code from the pretrained model to create a model with tensorflow layer. This just meant changing the input shape and the backend.image_dim_ordering before adding any Convolutions.
Then I used this loop:
model is the original model, created from the code I linked at the beginning.
model_tensorflow is the exact same model, but with tensorflow layout.
for i in range(len(model.layers)):
    layer_theano=model.layers[i]
    layer_tensorflow=model_tensorflow.layers[i]

    if layer_theano.__class__.__name__ in ['Convolution1D', 'Convolution2D', 'Convolution3D', 'AtrousConvolution2D']:
        weights_theano=layer_theano.get_weights()

        kernel=weights_theano[0]
        bias=weights_theano[1]

        converted_kernel=convert_kernel(kernel, "th")
        converted_kernel=converted_kernel.transpose((3,2,1,0))

        weights_tensorflow=[converted_kernel, bias]

        layer_tensorflow.set_weights(weights_tensorflow)

    else:
        layer_tensorflow.set_weights(layer_theano.get_weights())

In the original code, there is a testcase: Prediction ran on the image of a cat.
I've downloaded a cat image and tried the testcase with the original model: 285.
The converted model predicts 585.
I don't know whether 285 is the correct label for a cat, but even if it isn't, the two models should be broken in the same way, I would expect the same prediction.
What is the correct way of converting weights between models ?

@lhk I was having the same trouble for quite some time. Finally, I was able to transfer the weights in quite a complicated way, but it works well.
I've made a script which auto translates theano backend (th dim ordering) trained model weights directly into the other 3 possible combinations of backend / dim ordering.
The script is available here
Since it is such a complicated process, it has a few instructions on how to use it, and you need to manually edit the script in some places (like importing your own script to build a model, creating a theano and tensorflow model and specifying the path to the weight files for theano backend th dim ordering model).
Also, it runs on the tensorflow backend only for now. But for all that, it returns 3 folders with the correctly translated weights.
Note: The weight shuffle algorithm is from Kent Sommers (@kentsommer). I haven't tested it yet, but I think it works correctly.
In a few days (I have exams coming up, so not much time), I hope to translate the script to work with models which were trained on Tensorflow with tf dim ordering, and get it to auto translate the weights to Theano.

That looks fantastic.
I can't try it right now, but it's the first thing I'll do in the morning.
Thank you very much.

@lhk No problem. This was a huge problem for me for several months, so it's nice to finally have a working answer.

@fchollet Would it be possible to edit the wiki to point to the above mentioned script? It simplifies the procedure quite a bit.

Nice job @titu1994!
Again @lhk, the row_shuffle algorithm has not been tested so it is very possible that it is incorrect, however, it matches to the best of my knowledge what was done for the VGG16 and VGG19 models in the Keras applications examples.

Unfortunately, it does seem to be incorrect.
I took the code for vgg16 in theano and changed the input shape to fit tensorflow. Then the code can be executed to create a model with tensorflow backend.
I provided the theano and tensorflow models, as well as the weights to your script.
Then I reran the vgg16 script, the if __name__=="__main__" part. The predictions from the original and the converted models are different.

Hmm it seems that the only time it converts weights in the wrong manner is when there are two or more dense layers after the final convolutional layers.
Does the model you are converting have more than one dense layer after the final convolutional layer ? If not, the weights are correctly converting.
I tested it out by checking each and every layer weight with already translated weight provided by fchollet for the ResNet model with np.allclose.
I will try and modify my checking script to see if I can make it print out the row translation scheme.
Sorry for the trouble.

Yes, it does have more than one dense layer.

Ok. I've modified my checking script to now print out the row shuffles.
It will take quite some time (it checks 25k rows by 25k rows) but once it finished I'll be able to see where the wrong translations are occuring.

Thank you very much

Sorry for the delay, I had classes.
Surprisingly, an 8-core i7 processor with 16 GB RAM with joblib parallelization took nearly 1 hour to process 25k rows by 25 cols. In any case, I have the results now and a simple diff shows me the changes I have to make.
Will push the changes and notify when done.

@lhk well Found the bug.
@kentsommer The algorithm you used is exactly matching the row shuffles done by fchollet, but the way you were updating the shuffled weights was a little faulty.
The corrected version of your code :
def shuffle_rows(original_w, nb_last_conv, nb_rows_dense):
    converted_w = np.zeros(original_w.shape)
    count = 0
    for index in range(original_w.shape[0]):
        if (index % nb_last_conv) == 0 and index != 0:
            count += 1
        new_index = ((index % nb_last_conv) * nb_rows_dense) + count
        print("index from " + str(index) + " -> " + str(new_index))
        converted_w[index] = original_w[new_index] # Here converted[i] needs to update to original[new_index]. Not the other way around.

    return converted_w

I've pushed the fixes to both of my scripts now. They finally work completely for all types of models, and match the weights provided by fchollet.

@titu1994
Good catch :+1:
Might want to confirm that this swap holds from TensorFlow -> Theano (I looked at this direction for my original code) as well as Theano -> TensorFlow (I'm assuming you tested this direction). Awesome work making this all into one script! Definitely useful :1st_place_medal:

There's another script for conversion from Tensorflow to Theano, since the folder names change and the dim ordering changes and also I'd have to interpret the weight shape myself to determine which transfer was occuring.
So I simply leave it to the user to determine the correct script. There's weight_transfer_theano.py for conversion from Theano kernels TH dim to all other and weight_transfer_tensorflow.py for converting from Tensorflow kernels TF dim ordering to all others.

@titu1994 ,  your script worked perfectly for me

@smarinka I've not tested this for weights generated by Keras 2.
Keras 2 may not even need weight weight shuffling anymore as the changelog suggests that weights will work regardless of which backend and dim ordering you use.

I have been trying to convert a h5+json model trained on Theano backend (on Keras 1), to Keras 2 on Tensorflow 1 backend and indeed just for this purpose weights work regardless.
However I want my Tensorflow weights in the format pb+txt as found here.
Any idea how to do that? h5+json here
Note to @titu1994: your script crashed on this model with
...snip...
Done tf-kernels-th-dim model.h5
Nb layers :  18
Saved layer 1 : convolution2d_10
Saved layer 2 : convolution2d_11
Saved layer 3 : convolution2d_12
Saved layer 4 : maxpooling2d_4
Saved layer 5 : convolution2d_13
Saved layer 6 : convolution2d_14
Saved layer 7 : convolution2d_15
Saved layer 8 : maxpooling2d_5
Saved layer 9 : convolution2d_16
Saved layer 10 : convolution2d_17
Saved layer 11 : convolution2d_18
Saved layer 12 : maxpooling2d_6
Saved layer 13 : flatten_2
Traceback (most recent call last):
  File "weight_conversion_theano.py", line 128, in <module>
    nb_rows_dense_layer = weights[0].shape[0] // nb_last_conv
ZeroDivisionError: integer division or modulo by zero

Code used to load model (not sure I got this right):
from keras.models import model_from_json
with open('model.json') as fd:
    jsn = model_from_json(fd.read()).to_json()  # converts from Keras 1

K.set_image_dim_ordering('th')
th_dim_model = model_from_json(jsn)
th_dim_model.load_weights('model.h5')

K.set_image_dim_ordering('tf')
tf_dim_model = model_from_json(jsn)

model_weights = ['model.h5']

Ideas?

@titu1994 thanks for the great script. I had been working on a deep learning algorithm for CRISPR guide-RNA design that was originally coded with Theano backend. I needed to convert to R, but the Theano backend was not working (may not be supported anymore in R/keras?).
FYI, my input was a 2D one-hot encoding matrix that represented a DNA sequence, and thus used Convolutions in the model. Therefore, the original script did not correspond (as I believe the script is meant for higher-dimensional image input). I modified the script for it to work with 2D one-hot encoding matrices if you'd be interested in implementing that feature.


[vite:build-import-analysis] Failed to parse source for import analysis because the content contains invalid JS syntax

Bug reported in Nuxt-Security : https://github.com/Baroshem/nuxt-security/issues/390
Original issue in Nuxt core : https://github.com/nuxt/nuxt/issues/26011
OP had the initial code with console.log statement:
onUnmounted(() => {
    removeCardServiceSubscription(cardServicesSubscription)
    console.log("🚀 ~ onUnmounted ~ removeCardServiceSubscription(cardServicesSubscription):", removeCardServiceSubscription(cardServicesSubscription))
});

The plugin transformed the initial code by cutting out the beginning of the console.log statement :
[vite:build-import-analysis] Failed to parse source for import analysis because the content contains invalid JS syntax. Install @vitejs/plugin-vue to handle .vue files.
file: D:/proj/components/details/Index.vue:205:64
203: onUnmounted(() => {
204:     removeCardServiceSubscription(cardServicesSubscription)
205:     :", removeCardServiceSubscription(cardServicesSubscription))
                                                                     ^
206: });

Line 205 is indeed invalid Javascript

fixed in V1.0.0


envoy: Bump envoy version to v1.25.9

This is for the below CVEs from the upstream.
CVEs:
https://github.com/envoyproxy/envoy/security/advisories/GHSA-pvgm-7jpg-pw5g https://github.com/envoyproxy/envoy/security/advisories/GHSA-69vr-g55c-v2v4 https://github.com/envoyproxy/envoy/security/advisories/GHSA-mc6h-6j9x-v3gq https://github.com/envoyproxy/envoy/security/advisories/GHSA-7mhv-gr67-hq55
Build: https://github.com/cilium/proxy/actions/runs/5666538565/job/15353476426
Release: https://github.com/envoyproxy/envoy/releases/tag/v1.25.9

/test-backport-1.14


feat: Trip Gallery Page is Been Updated

Title and Issue number

Title :
[Feature Request]: Trip Gallery Changes
Issue No. :
#925
Code Stack :
Close #<#925>

Description
Here in this PR i have do some of the major changes in Trip Gallery , i have improved the User Interface of the Page , by using cool animation , so due to this the experience of the users also get enhanced.
Video/Screenshots (mandatory)
BEFORE

AFTER

Video
https://github.com/apu52/Travel_Website/assets/114290748/e4dbeb5c-48c8-48f0-a25f-a12810c9a943
Type of PR

[X] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[X] I have performed a self-review of my code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have gone through the  contributing.md file before contributing

Additional context:
@apu52 please review this PR and merge with main branch under Gssoc24
##Are you contributing under any Open-source programme?
#GSSOC24

[X] I am contributing under GSSOC'24
[ ] I am contributing under VSOC'24

Hey @AbheetHacker4278, can you add an overlay layer on these card images so that the content on the images will be move visible on hovering on it

Hey @AbheetHacker4278, can you add an overlay layer on these card images so that the content on the images will be move visible on hovering on it

Okay i will do it

@itsekta I have done the necessary changes  , please review it and merge it with main branch

@itsekta please also add Gssoc labels with level 2

@itsekta if you like the changes please merge with main branch

@itsekta @apu52 please merge into main branch if there is no errors

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Okay i will do that in new PR , Because this take some time and i also have to re structure the code base from start to end , so better you merge this pr with main branch and tommorow i will open an new issue you just approve it, is it fine.

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Because code base are in very bad structure very messy , thats why i am asking you

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Please merge it , tommorow i will open new issue and accept it also 🙏

but here I can see there is a bug in your Deployed pR , please try to resolve this So that I can merge your PR

I havent see any kind of bugs from my side , please check it again please. and i think ,  this is not my PR sir.

but here I can see there is a bug in your Deployed pR , please try to resolve this So that I can merge your PR

but here I can see there is a bug in your Deployed pR , please try to resolve this So that I can merge your PR

I havent see any kind of bugs from my side , please check it again please. and i think , this is not my PR sir.

not this changes what u have made .the above image slider is facing some bugs

@apu52 all bugs is been removed , please review it and merge it with main branch

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Okay i will do that in new PR , Because this take some time and i also have to re structure the code base from start to end , so better you merge this pr with main branch and tommorow i will open an new issue you just approve it, is it fine.

I have merged the previous PR , have u raised a new issue what @itsekta ma'am has already suggested u ?? if not please raise it so that u will get assigned

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Okay i will do that in new PR , Because this take some time and i also have to re structure the code base from start to end , so better you merge this pr with main branch and tommorow i will open an new issue you just approve it, is it fine.

I have merged the previous PR , have u raised a new issue what @itsekta ma'am has already suggested u ?? if not please raise it so that u will get assigned

Okay , in 2 days i will do it , because my exams are starting , so after that i will do it


[Slider][material-ui] Pass narrowed value type to onChange and onChangeCommitted

closes: https://github.com/mui/material-ui/issues/37854

[x] I have followed (at least) the PR section of the contributing guide.

This looks ready to merge to me. @michaldudak, may I ask you for a final review? Specially regarding:

This comment
This comment
The overall Typescript implementation

These are somewhat complex type structures that I need more experience with to feel comfortable approving on my own.

I'm closing this as we have yet to find a proper implementation, and I'm not keen on releasing this with the last v5 release.
I added the original issue to the v7 draft milestone to revisit later.
Thanks for working on this anyway, @sai6855. I hope we can implement it soon.


Increase bddtest query timeout to 30; Fixes issue 1080

@gongsu832 and I were seeing lots of timeouts while running queries during bddtests. See #1080. Simply increasing the timeout from "3" to "30" fixes these errors (I tried "10" - didn't work). This change does not increase the time required to run the bddtest suite from inside Vagrant.
Speaking for myself, I was seeing these issues in a standalone (non-Vagrant) system. I'm running on a beefy X86 server, and the bddtest suits takes about 4x the time required within Vagrant for some reason. So there may be some other issue at play, but this change seems to do no harm inside Vagrant, and seems to be required outside.
Signed-off-by: Bishop Brock<EMAIL_ADDRESS>

Signed-off-by: Bishop Brock<EMAIL_ADDRESS>

LGTM


Intent catcher added

Description
Intent catcher moved from demo to this repo.
New directory "examples" added.
Checklist

[x] I have performed a self-review of the changes

To Consider

Add tests (if functionality is changed)
Update API reference / tutorials / guides
Update CONTRIBUTING.md (if devel workflow is changed)
Update .ignore files, scripts (such as lint), distribution manifest (if files are added/deleted)
Search for references to changed entities in the codebase

There has been a misunderstanding.
We needed to merge demo dff projects (as in ready to use projects written in dff that showcase some functionality) into this repository to make it easier to keep them up-to-date.
What this PR was supposed to do is to add all the files from here:
https://github.com/deeppavlov/dialog_flow_demo/tree/feat/intentcatcher
into an example directory in the https://github.com/deeppavlov/dialog_flow_framework repository.
I'll close this PR because everything needs to be changed:

The target branch (to include the examples/customer_service_bot and examples/frequently_asked_question_bot directories)
The name of the target branch (to something like merge/demo)
The name of the PR
Description of the PR

Apologies for not making the task clear enough. Let me know if there are any further questions.


📝 0.1.0 Release

CreateAPI has proven to be a very stable tool since its initial prerelease and to better reflect that, we're working to move to version 0.1. In doing so, we want to take this opportunity to make a few bigger than normal changes based on how the tool has been used out in the wild.
To summarise what is going into 0.1.0:
Documentation

[x] #70
[x] #58

Up until now, there hadn't been much of a focus on documentation but that needed to change, in 0.1.0 we're writing a lot of new documentation to help new users get started with CreateAPI.

[x] #82

Configuration

[x] #89
[x] #100
[x] #98
[x] #97
[x] #94
[x] #93
[x] #92
[x] #91
[x] #63
[x] #75
[x] #60
[x] #47
[x] #88
[x] #66
[x] #85
[x] #114

The Configuration Options are at the heart of how you tweak your generated code and its important to make sure that these options are clear and easy to understand. As a result, we'll be making some adjustments to how these options are styled which will result in some breaking changes. A complete set of notes for migration will be provided in the final release, but the aim here is to set good foundations for future changes.
In addition, we're making it easier to figure out when your configuration file contains typos, unsupported or deprecated options:

[x] #77
[x] #90
[x] #125

Installing CreateAPI
The easier it is to install CreateAPI, the more likely you are to adopt it (I hope)

[x] #69
[x] #48

Get
Get recently announced the 1.0.0 release, started from CreateAPI 0.1.0, generated packages will use Get 1.0.0

[x] #80

Performance Optimizations
While CreateAPI has previously focused on optimising code generation, we hadn't made much effort in terms of optimising the generated code for fast build times and slim binaries. In 0.1 we're making some changes to help with this:

[x] #74
[x] #75

It's now recommended (and the default) to split your generated entities and paths into individual source files. In addition, avoiding the generation of individual CodingKey enums is recommended to remove bloat from the compiled binary with larger schemas.

As always, feedback and suggestions are welcome 🌟

Release Notes (Draft)

Preview
Breaking Changes

Generated packages and paths now depend on Get 1.0.2 or later. If you don't use Get, your Request type must expose an initializer that matches the initializer defined in Get. For more information, see #83.

When generating a Swift Package, the Package.swift file and all other sources are written to the root of the --output directory instead of being nested inside a subdirectory.

Default output directory is now ./CreateAPI.

The rename.properties option now understands property names as defined in the original schema and not after applying CreateAPI transformations (such as case conversion or swifty style booleans). For more information, see #112.

The generator will now error if the path defined using --config did not contain a valid file (prior behaviour was to fallback to the default configuration). For more info, see #125.

Command Line Argument options that alter the generate output have now been moved into the configuration file and the behaviour may have also been adjusted. Please refer to the table below for more information:

Pre 0.1 Argument
Remark

--split (-s)
This is now the default behaviour. To generate entities and structs merged into a single file, use the mergeSources option in the configuration file.

--filename-template
Split into entities.filenameTemplate and paths.filenameTemplate in the configuration file.

--entityname-template
Replaced by the entities.nameTemplate config option.

--generate
Moved into the configuration file as generate. In addition, enums and package are also now valid options to consistently control the generation of all components.

--package
To generate as a package, ensure that generate is set to [entities, paths, enums, package]. To customise the module name, set the module option.

--module
To disable package generation, set generate to [entities, paths] (see above) and to customise the module name, set the module option.

--vendor
Replaced with the vendor config option.

Significant changes to options in the configuration file:

Pre 0.1 Option
Remark

isUsingIntegersWithPredefinedCapacity
Removed in favour of dataTypes customisation. Fixed width integers are now used by default so you must override the int32 and int64 formats if you wish to disable this.

isNaiveDateEnabled
Renamed to useNaiveDate

isPluralizationEnabled
Renamed to pluralizeProperties

isInliningTypealiases
Renamed to inlineTypealiases

isGeneratingSwiftyBooleanPropertyNames
Renamed to useSwiftyPropertyNames

isGeneratingEnums
Now part of the generate option (included by default).

isAddingDeprecations
Renamed to annotateDeprecations

isStrippingParentNameInNestedObjects
Renamed to stripParentNameInNestedObjects

isAddingDefaultValues
Renamed to includeDefaultValues

isSortingPropertiesAlphabetically
Renamed to sortPropertiesAlphabetically

isGeneratingEncodeWithEncoder
Renamed to alwaysIncludeEncodableImplementation

isGeneratingInitWithDecoder
Renamed to alwaysIncludeDecodableImplementation

isGeneratingInitializers
Renamed to includeInitializer

isSkippingRedundantProtocols
Renamed to skipRedundantProtocols

isGeneratingIdentifiableConformance
Renamed to includeIdentifiableConformance

isRemovingRedundantPaths
Renamed to removeRedundantPaths

isMakingOptionalPatchParametersDoubleOptional
Renamed to makeOptionalPatchParametersDoubleOptional

isInliningSimpleQueryParameters
Renamed to inlineSimpleQueryParameters

isInliningSimpleRequests
Renamed to inlineSimpleRequests

isGeneratingResponseHeaders
Renamed to includeResponseHeaders

overridenResponses
Renamed to overriddenResponses (double d)

overridenBodyTypes
Renamed to overriddenBodyTypes (double d)

isSwiftLintDisabled
Removed. Use fileHeaderComment to replicate similar behaviour

isReplacingCommonAcronyms
Replaced by acronyms

addedAcronyms
Replaced by acronyms

ignoredAcronyms
Replaced by acronyms

isAdditionalPropertiesOnByDefault
Now always enabled

isAddingOperationIds
Now always enabled

isGeneratingCustomCodingKeys
Replaced by optimizeCodingKeys with different default behaviour

isInliningPropertiesFromReferencedSchemas
Replaced by inlineReferencedSchemas with different default behaviour

isGeneratingMutableClassProperties
Replaced by mutableProperties

isGeneratingMutableStructProperties
Replaced by mutableProperties

isGeneratingStructs
Replaced by defaultType

isMakingClassesFinal
Replaced by defaultType

entitiesGeneratedAsClasses
Replaced by typeOverrides

entitiesGeneratedAsStructs
Replaced by typeOverrides

access
Accepted values limited to public or internal (public is the default)

Brilliant work with the release! 👏

Thanks! Big shoutout to @LePips too for a lot of great improvements 💪


TCD-2789:Slack integration

Description

Motivation and Context

Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

@peter-shao, what should be added in the slack.md file exactly? Because to me, the content I was trying to create is already on that page. Sorry for all the back and forth, but I'm new to the project. Thanks

@LoryApiFortress  Let me know when this PR is ready for review. It sounds like it is still a work in progress (based on previous comments ) :)


Add upgrade pip instructions to README and docs

With an out-of-date pip you run into errors. For example: pip install optimum-intel[openvino] did not install the onnx dependency with an outdated pip.
We recommend to install from source, and then it's nice to have the command for how to install the extras with it. It's not obvious that for the OpenVINO integration to work, you would need to install onnx manually for example.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[Question] Number of task with MPI applications

Hi, it is me again (back from the holidays) to ask some other questions about QCFractal. I am transitioning from psi4 to use orca 6.0.1 and I wrote a QCEngine implementation (that I will open source soon with a PR soon) to use it in the infrastructure.
The serial application work flawlessly but I am trying to use the parallel feature of ORCA and I encountered some difficulties in the implementation.
I made some modification to qcfractal and the underlying parsl to use the HighThroughputExecutor with the SlurmProvider to allow jobs with a higher number of tasks. As an example a config of the type:
workers_per_node: 2
max_nodes: 1
cores_per_worker:  4
memory_per_worker: 16

will spawn instead of the usual:
#SBATCH --nodes=1
#SBATCH --ntasks=1
#SBATCH --cpus-per-task=8

export PARSL_CORES=8

the modified:
#SBATCH --nodes=1
#SBATCH --ntasks=8
#SBATCH --cpus-per-task=1

export PARSL_CORES=4

with ORCA automatically configured to use 4 CPUs per calculation.
This is working correctly for single nodes but there are some conflicts with the ComputeManager and the number of active tasks and open slots.
It seems that because of this, parsl/ComputeManager when spawning another nodes because of the high number of available open slots, there is some overallocating extra tasks on the cpus of the nodes causing 4 calculations/tasks instead of 2 per node. I am diving deeper into parsl and the use of it in the manager but for now it has been tricky to solve this issue. Do you have any recommandation?
I also tried to fix the maximum number of open slots to avoid the overallocating but in this case, there is never a scaling to request new nodes and I am usually stuck to 1 node.

I'm not entirely sure, but the current manager is not particularly MPI friendly, as you are finding out :) . We do assume that the number of slots is num_nodes * ntasks_per_node, but it sounds like you want to do (non-hybrid?)-MPI.
I will have to think if there's an easy way to shove this into the manager. And I need to consult the Parsl docs as well.  I would certainly be interested if this could be done in a backwards-compatible way.
Let me know if you think of anything


[BUG] Audio playing first with video delayed from black screen

When I play the youtube video, the audio of the video plays fine but the video is first black and comes a few seconds later . But both the video and audio are in sync. But the video appear visually later in the screen.
Can you help me with this bug?
I get these logs while playing...
create MediaCodec video decoder, mime video/x-vnd.on2.vp9
I/OMXClient(10555): IOmx service obtained
I/MediaCodec(10555): MediaCodec will operate in async mode
D/SurfaceUtils(10555): connecting to surface 0x7766a22010, reason connectToSurface
I/MediaCodec(10555): [OMX.qcom.video.decoder.vp9] setting surface generation to 10808328
D/SurfaceUtils(10555): disconnecting from surface 0x7766a22010, reason connectToSurface(reconnect)
D/SurfaceUtils(10555): connecting to surface 0x7766a22010, reason connectToSurface(reconnect)
I/ExtendedACodec(10555): setupVideoDecoder()
I/ExtendedACodec(10555): Decoder will be in frame by frame mode
D/SurfaceUtils(10555): set up nativeWindow 0x7766a22010 for 480x854, color 0x7fa30c06, rotation 0, usage 0x20002900
W/MapperHal(10555): buffer descriptor with invalid usage bits 0x2000
I/chatty  (10555): uid=10314(np.com.sarbagyastha.youtube_player_iframe_example) CodecLooper identical 4 lines
W/MapperHal(10555): buffer descriptor with invalid usage bits 0x2000
D/SurfaceUtils(10555): set up nativeWindow 0x7766a22010 for 480x854, color 0x7fa30c06, rotation 0, usage 0x20002900
W/MapperHal(10555): buffer descriptor with invalid usage bits 0x2000
W/MapperHal(10555): buffer descriptor with invalid usage bits 0x2000
D/NdkImageReader(10555): acquireImageLocked: Overriding buffer format YUV_420_888 to 0x7fa30c06.
W/MapperHal(10555): buffer descriptor with invalid usage bits 0x2000
I/chatty  (10555): uid=10314(np.com.sarbagyastha.youtube_player_iframe_example) CodecLooper identical 1 line
W/MapperHal(10555): buffer descriptor with invalid usage bits 0x2000
W/AudioManager(10555): Use of stream types is deprecated for operations other than volume control
W/AudioManager(10555): See the documentation of requestAudioFocus() for what to use instead with android.media.AudioAttributes to
qualify your playback use case

Edit1: The audio with delayed video issue doesn't happen when I turned on the youtube controls. It only happens if I turned off youtube controls.

Could you share sample of your code? I think that you refresh Iframe so theres your delay with black screen.

@PcolBP Here's the code. I'm using a pageview and for each page I'm passing a link and creating a new youtube player widget. The audio with delayed video, issue does not happen if I enable youtube controls. It only happens if I disable youtube controls.
Here's the code
import 'dart:developer';

import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:youtube_player_iframe/youtube_player_iframe.dart';

import 'widgets/meta_data_section.dart';
import 'widgets/play_pause_button_bar.dart';
import 'widgets/player_state_section.dart';
import 'widgets/source_input_section.dart';
import 'widgets/volume_slider.dart';

Future<void> main() async {
  WidgetsFlutterBinding.ensureInitialized();
  runApp(YoutubeApp());
}

///
class YoutubeApp extends StatelessWidget {
  var link = [
    "xKT3deiaLEk",
    "gtm7m4tLUn0",
    "xKT3deiaLEk",
    "gtm7m4tLUn0",
  ];
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Youtube Player IFrame Demo',
      theme: ThemeData(
        brightness: Brightness.dark,
        primarySwatch: Colors.deepPurple,
        scaffoldBackgroundColor: Colors.black,
      ),
      debugShowCheckedModeBanner: false,
      home: PageView.builder(
        scrollDirection: Axis.horizontal,
        controller: PageController(),
        itemBuilder: (BuildContext context, int itemIndex) {
          return YoutubeAppDemo(link[0]);
        },
      ),
    );
  }
}

///
class YoutubeAppDemo extends StatefulWidget {
  final _link;
  YoutubeAppDemo(this._link);
  @override
  _YoutubeAppDemoState createState() => _YoutubeAppDemoState();
}

class _YoutubeAppDemoState extends State<YoutubeAppDemo> {
  late YoutubePlayerController _controller;

  @override
  void initState() {
    super.initState();
    _controller = YoutubePlayerController(
      initialVideoId: widget._link,
      params: const YoutubePlayerParams(
        playlist: [],
        startAt: const Duration(minutes: 0, seconds: 0),
        showControls: true,
        showFullscreenButton: true,
        desktopMode: false,
        privacyEnhanced: true,
        useHybridComposition: false,
      ),
    );
  }

  @override
  Widget build(BuildContext context) {
    const player = YoutubePlayerIFrame();
    return YoutubePlayerControllerProvider(
      // Passing controller to widgets below.
      controller: _controller,
      child: Scaffold(
        body: LayoutBuilder(
          builder: (context, constraints) {
            return ListView(
              children: [
                Stack(
                  children: [
                    player,
                    const Controls(),
                  ],
                ),
              ],
            );
          },
        ),
      ),
    );
  }

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

///
class Controls extends StatelessWidget {
  ///
  const Controls();

  @override
  Widget build(BuildContext context) {
    return Padding(
      padding: const EdgeInsets.all(16),
      child: Column(
        crossAxisAlignment: CrossAxisAlignment.start,
        children: [
          _space,
          // MetaDataSection(),
          _space,
          //  SourceInputSection(),
          _space,
          PlayPauseButtonBar(),
          _space,
          //VolumeSlider(),
          _space,
          // PlayerStateSection(),
        ],
      ),
    );
  }

  Widget get _space => const SizedBox(height: 10);
}

After copying your code and test, problem not occur. Player plays as it should, haven't notice black screen. Only blinking of widgets below. So it may be problem on your side.

@PcolBP It's doesn't happen often. But It occurs. If you keep testing repeatedly for 10 times , this issue happens one time on a real device.

@a-l-l-a-n it really rare cases when problem depends on phone. I think the problem is on platform_view and holding state. I also had problems with black screen on my side, solved it already but it was connected with holding controller state. Pageview has this to itself that its frees from resources a widget that you don't see and render new when when you will eneter it. Have you tried to add Key to in this case YoutubeAppDemo?

@PcolBP I don't know a lot about keys. Thanks for you reply. Can you suggest some keys to try in Pageview ? I'm just a real beginner

In that case I belieave that you dont need to get access to context of widget so use UniqueKey()
YoutubeAppDemo(
            link[0],
            key: UniqueKey(),
          );

 YoutubeAppDemo(
    this._link, {
    Key? key,
  }) : super(key: key);

@PcolBP  Really appreciate your time for answering. I've tried it and no luck. I'll figure some other way out. Thanks


Add support for publisher purposes consent string

The library's readme states that "This library is a Java reference implementation for dealing with consent strings in the IAB EU's GDPR Transparency and Consent Framework." There are two distinct consent string formats outlined in the specification (v.1.1):

global, vendor consent string (euconsent) as described in https://github.com/InteractiveAdvertisingBureau/GDPR-Transparency-and-Consent-Framework/blob/master/Consent string and vendor list formats v1.1 Final.md#vendor-consent-string-format-
publisher purposes consent string (pubconsent) as described in https://github.com/InteractiveAdvertisingBureau/GDPR-Transparency-and-Consent-Framework/blob/master/Consent string and vendor list formats v1.1 Final.md#publisher-purposes-consent-string-format-

However, currently the library appears to support only global, vendor consent string (correct me if I'm wrong). Looking from a publisher's perspective it would surely help if the publisher purposes consent string was supported as well. It makes sense since both consent strings have similar formats so there is potential for reusing significant part of codebase between the two implementations.
Roughly, these would be the things to do:

Refactor existing ConsentStringParser, extracting common logic out so that it can be reused when implementing publisher purposes consent string parser
Rename existing ConsentStringParser so that it's clear it supports global, vendor consent string
Implement publisher purposes consent string parser

What do you think?

I think that is a great idea, if you have the time can you write it and submit a pull request?

yep, We need it as well

Code was refactored in version 2.0.0 and it should be clear that current classes only deal with vendor consent.
At this point I don't have time to research publisher purposes consent as this is not something our company needs. But pull requests would be welcome :-)

Hi, I've built a support for publisher consent strings according to the specs.
I've intentionally split it from the vendor classes so that they can evolve independently. I've also reused the GDPR constants class.
Not sure if this can be of use? Any suggestions? Recommendations?


document new linter creation flow

Starting SL4 (#666) you can't start a new linter from the SL plugin, but there will be a template repository you can fork.
We may also drop some of the wizard-like UI and just let you manually edit the few files that need editing.
Additionally, we may drop the entire review proces and re-integrate the package_control repository to make linter plugin authors more autonomous and reduce unnecessary maintenance load on SL devs.

Additionally, we may drop the entire review proces and re-integrate the package_control repository to make linter plugin authors more autonomous and reduce unnecessary maintenance load on SL devs.

It would increase my load, however. Considering that SL authors and collaborators have a much better understanding of their provided API, it sounds like a much better idea to have plugins for this API to be reviewed by them.

Yes, that is a good point. Would it be possible and a good idea to organize that in the main package control repo?

Not sure what you want to organize. The status quo seems quite good to me.

Alright, let’s keep it that way.


Missing material modifications in docs

There is a table in section 2.2.4 of the documentation that describes the material modifications available across all the material classes in the framework:

This table is missing entries for the following material classes:

lithium
mox
uraniumOxide
uZr

The entries are drawn programmatically in blueprints.rst by looking at all subclasses of Material:

The problem appears to be that it does not recursively look into all subclasses of Material. For instance, the classes above are subclassed from:

lithium -- material.Fluid
mox -- UraniumOxide
uraniumOxide -- material.FuelMaterial
uZr -- material.FuelMaterial

This piece of the documentation should be updated to draw the entries recursively.

Nice find!
Thankfully, that's an easy fix.

I haven't forgotten about this ticket.  It appears that inside the if numArgs > 1: condition, we can do something like:
for subM in m.__subclasses__():

And do the same if numArgs > 1: checking again.  I just haven't decided the cleanest way to write the resultant two loops (or the final table) yet.
It'll happen this week.


AttributeError: 'ConfigEntries' object has no attribute 'async_setup_platforms'. Did you mean: 'async_unload_platforms'

I get an error after adding the integration:

Logger: homeassistant.config_entries
Source: config_entries.py:584
First occurred: 21:47:14 (1 occurrences)
Last logged: 21:47:14
Error setting up entry<EMAIL_ADDRESS>for tuya_v2
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 584, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/tuya_v2/init.py", line 118, in async_setup_entry
success = await _init_tuya_sdk(hass, entry)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/tuya_v2/init.py", line 176, in _init_tuya_sdk
hass.config_entries.async_setup_platforms(entry, PLATFORMS)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'ConfigEntries' object has no attribute 'async_setup_platforms'. Did you mean: 'async_unload_platforms'?

Algorithm for my actions:

Installed the add-on via HACS using these instructions
Restarted Home Assistant
Went to the integration section and found Tuya v2 there.
In the Cloud Project Type section I selected: ‘Smart Home PaaS’.
In credentials section I specified my credentials and country code: ‘ua’. In the field: Mobile app I specified ‘Tuya App’.
Integration is added, but I see: ‘Failed To Setup’.
I looked at the logs and saw the logs I specified at the beginning of the post.

Versions:

Home Assistant Core: 2024.5.5
Home Assistant Supervisor: 2024.05.1
Home Assistant Operating System: 12.3
Home Assistant Frontend: 20240501.1
HACS version: 1.34.0

Hardware

Raspberry Pi 4 (rpi4-64)

такая же проблема, есть решение?
Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/config_entries.py", line 586, in async_setup     result = await component.async_setup_entry(hass, self)              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/config/custom_components/tuya_v2/__init__.py", line 118, in async_setup_entry     success = await _init_tuya_sdk(hass, entry)               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/config/custom_components/tuya_v2/__init__.py", line 176, in _init_tuya_sdk     hass.config_entries.async_setup_platforms(entry, PLATFORMS)     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ AttributeError: 'ConfigEntries' object has no attribute 'async_setup_platforms'. Did you mean: 'async_unload_platforms'?

Having the same issue.
Logger: homeassistant.config_entries
Source: config_entries.py:640
First occurred: 3:24:41 PM (2 occurrences)
Last logged: 3:24:52 PM
Error setting up entry<EMAIL_ADDRESS>for tuya_v2
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 640, in __async_setup_with_context
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/tuya_v2/init.py", line 118, in async_setup_entry
success = await _init_tuya_sdk(hass, entry)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/tuya_v2/init.py", line 176, in _init_tuya_sdk
hass.config_entries.async_setup_platforms(entry, PLATFORMS)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'ConfigEntries' object has no attribute 'async_setup_platforms'. Did you mean: 'async_unload_platforms'?


Change the UI of cards on homepage

The cards on home page are not aligned properly.
Please assign this issue to me under GSSoC'23.

@Srishti-1602 go ahead, the issue has been assigned to you


Enable doc tests and run them against sandbox environment

Part of this issue also includes removing all inline docs to separate markdown files in a docs dir and reference them in code using #![doc = include_str!("<path_to_doc>.md")]


Add IN_PROGRESS to imageScanStatus

Description of the issue

It fails to run with the error below.

Error: Unknown status: IN_PROGRESS

We should only add statuses that we expect the scan to perform normally.
Since the IN_PROGRESS status is also expected to complete the scan normally, it seems that it should be exceptioned the same as PENDING in pollForScanCompletion function.

Description of changes

Added an additional check for the IN_PROGRESS scan status in the pollForScanCompletion function.

Can you try using v3?

Hi. @sshcokr. Thanks for raising. Sorry I had not tested actions from forked repos. It does not seem to run. I fixed this issue and tagged a v3.1 version.


Avoid race conditions when nodes terminate

Addresses #67. I attempted to put in reasonable lock commands for all schedulers, but I only have significant experience with SGE, so the other schedulers may need to be tweaked. Also, I have yet to rigorously test this code, so please test and confirm before merging. Thanks!

This has been pulled into the current develop branch.


Problem: can't use format with LSP detection

There is probably something I didn't understand here.
I want to show a "+" next to filetype icon when I have at least an LSP server attached, why isn't the following working?
local filetype_cmpnt = {
  "filetype",
  icon_only = true,
  icon = {align = "right"},
  fmt = function(s)
    if #vim.lsp.buf_get_clients() > 0 then
      return s .. "+"
    end

    return s
  end,
}

Nevermind: icon_only = false


Adding virtual node overview

Collapsing requirements and limitations in a single doc.
Updating TOC with the doc entry as well.
Addressing: https://github.com/Azure/AKS/issues/1522

@MikkelHegn : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mlearned let me know what you think of adding this overview article? I was about to just update requirements in both documents but found it might be more useful to have a simple overview.

@macolso - I pulled the region list from this: https://docs.microsoft.com/en-us/azure/container-instances/container-instances-region-availability#availability---virtual-network-deployment - can you please confirm that list is up-to-date?

Hi @MikkelHegn - This PR was created in the public repo. PMs should work in the private repo, and new articles need to be created in the private repo. We'll close this PR. Thanks.


Installation freezes at 65%

The installation freezes at 65% (THC_generated_THCTensorTypeUtils.cu.o) in both machines:
a) Ubuntu 14.04 CUDA 7.5 CuDNN 4
b) Ubuntu 16.04 CUDA 8.0 CuDNN 5.1
The CPU is working but there seems to be no progress.
Found CUDA on your machine. Installing CUDA packages
Warning: unmatched variable LUALIB
cmake -E make_directory build && cd build && cmake .. -DLUALIB= -DCMAKE_CXX_FLAGS=${CMAKE_CXX_FLAGS} -DCMAKE_BUILD_TYPE=Release -DCMAKE_PREFIX_PATH="/opt/torch/install/bin/.." -DCMAKE_INSTALL_PREFIX="/opt/torch/install/lib/luarocks/rocks/cutorch/scm-1" && make -j$(getconf _NPROCESSORS_ONLN) install

-- The C compiler identification is GNU 4.9.3
-- The CXX compiler identification is GNU 4.9.3
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found Torch7 in /opt/torch/install
-- Found CUDA: /usr/local/cuda-7.5 (found suitable version "7.5", minimum required is "6.5")
-- MAGMA not found. Compiling without MAGMA support
-- Automatic GPU detection failed. Building for common architectures.
 It's compiling for 5 different and detected archs in your case. That's why.  Thanks


BUG: Routes are not protected.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Description
The routes are note protected and can be navigates using url, that is, path traversing.
Steps to reproduce -

[x] Go to website.
[x] Change the url from https://foodies-web-app.vercel.app/, to https://foodies-web-app.vercel.app/home without logging in to the application.
[x] You'll be navigated to the Home Page of the website.

IMAGE : (Entered without LogIn)

Would love to contribute to Foodiesweb as GSSoC'24 Contributor. Looking forward for your response.
Record

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

Can you please explain me the proper solution to this issue , i will assign you the issue asap

Please assign me this issue under GSSoc'24.

This issue has alredy been taken !!


Failed to execute 'postMessage' on 'MessagePort': # could not be cloned.

Hello,
I am woking on a Blazor Webassembly app and I'd like to use your library.
I tried to modify your example, build it and copy  the files (ocr-app.bundle.js, tesseract-core.wasm, tesseract-core-fallback.wasm, tesseract-worker.js) to my project but I got the error:
Failed to execute 'postMessage' on 'MessagePort': # could not be cloned.
Error: Failed to execute 'postMessage' on 'MessagePort': # could not be cloned.
at ocr-app.bundle.js:256
at new Promise ()
at requestResponseMessage (tesseract.js:244)
at Object.apply (tesseract.js:180)
at OCRClient.loadImage (tesseract.js:346)
at async runOCR (ocr.js:34)
https://github.com/robertknight/tesseract-wasm/blob/main/examples/web/ocr-app.js
import { OCRClient } from "tesseract-wasm";
export function OcrClient() { return new OCRClient() };

Index.html (my project)
  <script type="module">
      import { OcrClient } from "./js/ocr-app.bundle.js";
      window.OcrClient = OcrClient;
  </script>

ocr.js (my project)
async function runOCR(base64String) {
    let start = new Date();

    // Initialize the OCR engine. This will start a Web Worker to do the
    // work in the background.
    const ocr = OcrClient();
    const imageResponse = await fetch(base64String);
    const imageBlob = await imageResponse.blob();
    const image = await createImageBitmap(imageBlob);

    try {
        // Load the appropriate OCR training data for the image(s) we want to
        // process.
        await ocr.loadModel('js/eng.traineddata.js');
        await ocr.loadImage(createImageBitmap(image));

        // Perform text recognition and return text in reading order.
        const text = await ocr.getText();

        console.log('OCR text: ', text);
    } finally {
        // Once all OCR-ing has been done, shut down the Web Worker and free up
        // resources.
        ocr.destroy();
    }

    let end = new Date();
    console.log((end - start) / 1000);
    return text;
}

Can you help me?
Is there a plain script tags (maybe with a CDN) to include it?
Thank you in advance.

The return value of createImageBitmap is a promise. The error is occurring on this line because you are passing a Promise<ImageBitmap> instead of an ImageBitmap to ocrClient.loadImage:
await ocr.loadImage(createImageBitmap(image));

Here image is already an ImageBitmap because it came from await createImageBitmap(...) above.
Regarding this line:
 await ocr.loadModel('js/eng.traineddata.js');

Is this file exactly the same as https://raw.githubusercontent.com/tesseract-ocr/tessdata_fast/main/eng.traineddata or did you modify it. That file should not have a .js extension because it isn't a JavaScript file. It is a binary blob of neural network weights.

Hello, Robert,
It was a mistake from me. It works!
Thank you.
Gaston


Create cibertec.txt

Link: http://www.cibertec.edu.pe/carreras-profesionales/computacion-e-informatica/

@gmondragon Could you please provide us with a bit more information about the university? The official website does not say much about the instituition itself: which courses it offers, how long the courses are, whether you have a campus, etc.

Hi @philipto. Thanks for answering.
In this link you can see the curriculum: http://www.cibertec.edu.pe/carreras-profesionales/computacion-e-informatica/plan-de-estudio/
The race lasts three years and has 8 campus: http://www.cibertec.edu.pe/acerca-de-cibertec/nuestras-sedes/
In the footer you can change the language to "English" so you can understand faster.

@gmondragon Thank you for the clarification! Pull request merged.


[fix][gws][workflow] delegatee selection

Coverage decreased (-0.02%) to 84.548% when pulling a768a34e1adfbd417c1533c0a900d23268cdd647 on fix-gws-delegatee-selection into bede95f9170e7f917454e2f5258b47187baf0f72 on master.


Extract Rack adapter into dependent gem

Since each adapter is it's own gem it would be helpful to standardize and re-use adapters like Rack, which is a common dependency between web based gems like Sinatra (and by extension Padrino which is based on Sinatra) and Rails. If there is desire to support Hanami in the future, this would be an easy first step to include the Rack gem as a dependency. Here is a rough list of tasks to complete this (I think):

[ ] Merge all of the separate Rack implementations back into the rack adapter gem
[ ] Make the rack adapter a dependency for the Sinatra gem
[ ] Stub out a very barebones Rails adapter by simply implementing the Rack gem as a dependency
[ ] Stub out a very barebones Hanami adapter by simply implementing the Rack gem as a dependency

Merge all of the separate Rack implementations back into the rack adapter gem
Can you elaborate on this part? I think I could help with it a little but want to make sure I understand the ask

Merge all of the separate Rack implementations back into the rack adapter gem

Can you elaborate on this part? I think I could help with it a little but want to make sure I understand the ask

I think this may be a reference to https://github.com/open-telemetry/opentelemetry-ruby/pull/225, in particular the registration of the Rack middleware in the Railtie and the definition of a Rails-specific middleware. I don't have more context than that, unfortunately.

I believe we can close this issue, right? Looks like there are now very few references to Rack in instrumentations other than the Rack instrumentation.


Standalone cue identifier in line 7

Hi, all validators are ok with the file.
Kindly please check. Thank you.
1999-0624-Wie-wir-uns-zu-einer-besseren-Personlichkeit-entwickeln.vtt.zip

Hi @amrutadotorg please provide the captions as plain text in a comment. But this error basically is raised when a line of text is found not being part of a block of lines with the expected format. In the recent latest release 0.5.0 the library now ignores the content that is not following the specs of WebVTT format instead of raising an exception. I am closing this issue but reopen if needed. Thanks.


[TASK] - Refactor reordering of mapped fields - pull from backend service.

The UI function handleIndexChange processes a mapped field index change.  The source field is represented by
the current index.  The target index is represented either by a number or by a mapped field (depending on whether the index was physically modified or d&d modified).

@igarashitm - having trouble coming up with the simplest way to transfer the target mapping field index to the backend service with the REST API as a request payload.  Do you have an example where a simple integer is passed? Do I need a full headers definition for it in JSON?  I'd like to keep it just as a parameter (@Parameter)

@Parameter(description = "Target Mapped Field Index") @PathParam("targetFieldIndex") Integer targetFieldIndex

The "RESTful" manner is to use request body but not the URL parameter for other than GET method. You should remove both of @Parameter and @PathParam. Here is an example of sending simple string body.
https://github.com/igarashitm/atlasmap/blob/3750/lib/service/src/main/java/io/atlasmap/service/DocumentService.java#L119-L128
https://github.com/igarashitm/atlasmap/blob/3750/ui/packages/atlasmap-core/src/services/document-management.service.ts#L990


Typo in help text

When running python manage.py makemigrations with a merge conflict.
SystemCheckError: System check identified some issues:

ERRORS:
?: (dlm.E002) dc's max_migration.txt contains multiple lines.
        HINT: This may be the result of a git merge. Fix the file to contain only the name of the latest migration, or maybe use the 'rebase-migartion' command.

Should be 'rebase-migration' command. not 'rebase-migartion' command..

Fixed in #51 just not released yet.


Resizes tooltip title text if it's too long for the area.

Summary

Shortened the title textbox size so it doesn't overlap the icons
Made the font size dynamic, with a max size of the current font size

Linked Issues
Closes #401
Screenshots (optional)

Replicated in #409 to avoid merge conflict issues


Add a preloader

A preloader could be displayed for the first few seconds of opening the website.

@J0SAL and I would like to work on this.


Add Universidad de Palermo file

I need to add this file please
The website of the university: https://www.palermo.edu/
Wikipedia: https://en.wikipedia.org/wiki/University_of_Palermo_(Buenos_Aires)

@adrianrosa I am sorry, the domain palermo.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Fix the feature state of PodSecurity (zh-cn)

Fix the feature state of PodSecurity in zh-cn doc.

Please resync the whole page.
Line 598, 612 should be changed.
/hold

Hey @houjun41544,
Although this page is out-of-sync, it's not hard to resync this page.
Please run this command in main branch:
./script/lsync.sh content/zh-cn/docs/reference/access-authn-authz/admission-controllers.md

You will see only 4 lines are changed. Then you can add them to this PR and force-push.

@Sea-n
Thanks a lot！

/remove-language en

@Sea-n  Done!

/lgtm


[Spell Check] Client context has incorrect base URI, so requests fail

Package Name: @azure/cognitiveservices-spellcheck
Package Version: 2.0.1
Operating system: Windows 10 Enterprise 1803
[x ] nodejs

version: 10.16.0

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
SpellCheckClientContext (https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/cognitiveservices/cognitiveservices-spellcheck/src/spellCheckClientContext.ts) does not set the base URI correctly (see line 42), so calls to the service fail.
This is the same issue as https://github.com/Azure/azure-sdk-for-js/issues/4160 and https://github.com/Azure/azure-sdk-for-js/issues/4414.
To Reproduce
Steps to reproduce the behavior:

Set environment variables SPELLCHECK_SUBSCRIPTION_KEY and SPELLCHECK_ENDPOINT to a valid Bing Spell Check subscription key and the corresponding endpoint, respectively.
Install @azure/ms-rest-js and @azure/cognitiveservices-spellcheck.
Run the following code with Node.

"use strict";

/* To run this sample, install the following modules.
 * npm install @azure/cognitiveservices-spellcheck
 * npm install @azure/ms-rest-js
 */
var SpellCheck = require("@azure/cognitiveservices-spellcheck");
var msRest = require("@azure/ms-rest-js");

const key_var = 'SPELLCHECK_SUBSCRIPTION_KEY';
if (!process.env[key_var]) {
    throw new Error('please set/export the following environment variable: ' + key_var);
}
const subscription_key = process.env[key_var];

const endpoint_var = 'SPELLCHECK_ENDPOINT';
if (!process.env[endpoint_var]) {
    throw new Error('please set/export the following environment variable: ' + endpoint_var);
}
const endpoint = process.env[endpoint_var];

const creds = new msRest.ApiKeyCredentials({ inHeader: { 'Ocp-Apim-Subscription-Key': subscription_key } });
const client = new SpellCheck.SpellCheckClient(creds, { endpoint: endpoint });

async function quickstart() {
    var result = await client.spellChecker("bill gtaes");
}

try {
    quickstart();
}
catch (error) {
    console.log(error);
}

Output:
(node:17028) UnhandledPromiseRejectionWarning: Error: Resource not found
    at new RestError (C:\Users\<user>\node_modules\@azure\cognitiveservices-spellcheck\node_modules\@azure\ms-rest-js\dist\msRest.node.js:1399:28)
    at C:\Users\<user>\node_modules\@azure\cognitiveservices-spellcheck\node_modules\@azure\ms-rest-js\dist\msRest.node.js:2494:37
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:17028) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:17028) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Expected behavior
Expected output: None (no errors).
Additional context
I am able to resolve the issue by making the following change to my local copy of the spellcheck package.
<EMAIL_ADDRESS>Change line 625 from:
_this.baseUri = "{Endpoint}";

to:
_this.baseUri = "{Endpoint}/bing/v7.0";

This library needs to be re-generated using the latest code generator which should fix the problem with baseuri

@v-jaswel With the release of the latest version of Spell Check -<EMAIL_ADDRESS>this issue has been fixed.


Ability to configure without using a file

I'm looking to incorporate this with something that I am building. One thing I've noticed is that there doesn't seem to be another way to configure the SDK without there being a config file present.
As my project already has it's own way of handling configuration, having an extra file to have to maintain just to use this code is rather cumbersome, and has lead me to try my hand at an SDK, but ultimately that is silly.
I have a few ideas I'd like to propose that I think would make this easier to integrate into other projects.
As a potential low-effort first step, what would you think about getting the config values from environment variables if a file doesn't exist?
Another option is to create a wrapper client that handles configuration, and acts as a entrypoint for the rest of the methods here. This gives the user absolute freedom on how the SDK is configured and this is what my iteration of the SDK does, which is  just a fork of dydx-v3-python.
e.g.
from gmx.v2 import GmxClient
from gmx.v2 import constants

ACCOUNT = "0x000"

gmx = GmxClient(account=ACCOUNT, rpc_url=constants.ARBITRUM_RPC_URL)

gmx.create_order(market, size, market_position)

What are your thoughts on this?

Hey sorry just seen you open this. There is a new development pending currently sitting on staging that will address this point of feedback
https://github.com/snipermonke01/gmx_python_sdk/blob/staging/gmx_python_sdk/scripts/v2/gmx_utils.py#L143


Position callback called twice on iOS

Hi,
I'm having issues since on iOS position callback is called twice while on android is working perfectly!
Is there any way to solve this?
Thanks in advance!

Hi @umbertogrimaldi
Thank you for opening an issue.
Please provide more information, codes, logs and anything you think that would help finding the issue;

Taken from your setup docs
THIS IS CALLED TWICE
static void callback(LocationDto locationDto) async {     final SendPort send = IsolateNameServer.lookupPortByName(_isolateName);     send?.send(locationDto);   }


Create a version of the API documentation Google Doc that includes sector

Overview
Create a version of the API documentation Google Doc that includes sector

Please also add answers to these questions (and to the extent that makes sense the reasoning behind it).

Can you explain to me the use and/or difference between aor_id and new_oar_id I get in the response json?
I tried calling the endpoint /api/facility-matches/{id}/ to test out, but was not able to get any results back. I'm not sure what id I should provide to get a response.

I created Open Supply Hub API Documentation by copying OAR API Documentation and adding sector related details


Link to public doc is broken

https://cloud.ibm.com/catalog/architecture/deploy-arch-ibm-core-security-svcs-0294f96e-7314-48d1-a710-c08a541b2119-global?kind=terraform&format=stack&version=c72d9380-c3d2-495b-adbe-05b60f8d1e4a-global - the docs link is going to a broken page.
It should be pointing to https://cloud.ibm.com/docs/security-hub?topic=security-hub-core-security-services-pattern

Fix coming in https://github.com/terraform-ibm-modules/stack-ibm-core-security-services/pull/157


Add variables file for AWS us-east-2

This PR adds a variables file for the "us-east-2" region on AWS. The variables file properly sets the aws_region variable to workaround issue #123.

Pinging @detiber for review and possible merge

@scottslowe does the README need to be updated for this change? The linked issue seems to indicate that it does.

@detiber This PR is separate/independent from the linked issue. This PR simply adds a variables file for a new US-based AWS region. It does include the "aws_region" variable to address the underlying problem in the linked issue, which itself can be solved a number of different ways (see the associated issue for my discussion). Updating the README is one way; adding the "aws_region" variable to the various region-specific variables files seems (to me) to be the most logical approach.


GC old and unused persistence-level metrics

These metrics are from the age of VDS and a dozen spinning disks per node 🦕

@baldersheim please review


Too many (3) achievements are given at once when making a tool part

I will probably include this in a bigger issue to rework the achievements. I won't bother touching them too much when I have planned to completely rework the in the bearish future

I'll incorporate a few into my other issue #433

Is fixed in current build because I removed them oops


fix concurrency problem with telemetry actors/messages

Had this little gem blow up on me today.  The root of the problem is that the map is passed into a message and sent to the connection message processors.  This causes a concurrent modification exception when new metrics are being sent created while the metrics list is being walked to send to a websocket client.

Using locking across actors seems like the wrong way to fix this long term. Happy to patch it, but what's the plan here? Can you file an issue for it?

Given how often this code is called, I'm just going to copy it into an immutable map.  Will have updated PR shortly.

Updated. I think this is more in line with what we want.


[install vsix] needs to install .vsix to .vscode-server (not .vscode)

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

VS Code Version: 1.58.2
OS Version: windows 10

Steps to Reproduce:

connect to Host using Remote-SSH
connect ssh (ssh username@host) on command tool
type code --install-extension ${VSIX}
extension is installed to .vscode folder (not .vscode-server)

when use code --install command,  is there any way to install vscode extension to .vscode-server?
(I know we can set the specific path using code --extensions-dir <dir>, but that way changes the root path)

You need to run code --install-extension ${VSIX} in the integrated terminal in the remote window. Only then, code can talk to the remote server. If called in a regular terminal, you are running an instance of (desktop) code installed on the machine.


Skipping already downloaded files

Ripme version: 1.7.92
Java version: 8 u151
Operating system: Win10 x64 1909
Please include any additional information about how to reproduce the problem:

Expected Behavior
RipMe should move to the next download if files are already downloaded after x-numer of files.
Actual Behavior
RipMe goes one by one on the list even though it's all downloaded.

YES!!! Please!!!

this feature already exists
https://github.com/RipMeApp/ripme/wiki/Config-options#historyend_rip_after_already_seen
note: it wont end immediately as it will finish checking all of the files its already added to it's internal queue - but much faster if checking 200+ files

this feature already exists
https://github.com/RipMeApp/ripme/wiki/Config-options#historyend_rip_after_already_seen
note: it wont end immediately as it will finish checking all of the files its already added to it's internal queue - but much faster if checking 200+ files

Can you give an instruction on how to activate it? What I need to write in rip.properties?

I add this:
history.end_rip_after_already_seen = 3
And it doesn't work for twitter


chore: refactor hardhat tests

Rearrange hardhat tests folder structure;
Split out the single FleekERC721 suit to multiple files.

LGTM. Let's merge this? @EmperorOrokuSaki are you still planning to review it?

Yes I had reviewed this while working on hardhat tests for the auto approval PR.


Fix: DOS-577 Address failure of get_current_value

Motivation and Context
Resolves an issue that caused variables to break when fetching their current value after the user navigates away and back to the application.
Implementation Description

Address the websocket session not being configured correctly in get_current_value after navigating away and back to a running kernel
The double if block ended up with a situation where the sessions could be empty even though a session existed which lead to the request failing even though there was a connection to use.

Any new dependencies Introduced
No
How Has This Been Tested?
Tested Locally
PR Checklist:

[x] I have implemented all requirements? (see JIRA, project documentation).
[x] I am not affecting someone else's work, If I am, they are included as a reviewer.
[ ] I have added relevant tests (unit, integration or regression).
[ ] I have added comments to all the bits that are hard to follow.
[ ] I have added/updated Documentation.
[x] I have updated the appropriate changelog with a line for my changes.

Screenshots (if appropriate):

Looks good, nice find. I believe we have some tests covering multi-user behaviour of this get_current_value/websockets piece (tests/python/test_websocket.py), could be worth adding a regression test case for this bug with same user reconnecting?

Yep, makes sense. Wasn't aware those tests were there.


org.scalatestplus.junit.JUnitSuite cannot be found

As org.scalatest.junit.JUnitSuite is deprecated in favor (in theory) of org.scalatestplus.junit.JUnitSuite but doesn't seem to be accessible.
I discovered it while I was trying to upgrade a project to ScalaTest 3.0.7, but it looks like the import import org.scalatestplus.junit.JUnitSuite doesn't work anymore (it worked in 3.0.6).
[error] /home/jlprat/playground/akka-http/akka-http-core/src/test/java/akka/http/javadsl/model/EntityDiscardingTest.java:15:1: cannot find symbol
[error]   symbol:   class JUnitSuite
[error]   location: package org.scalatestplus.junit
[error] import org.scalatestplus.junit.JUnitSuite;

@jlprat bummer, it seems like our type alias approach for classes in the new package does not work for java source, I'll discuss with @bvenners to see if we shall duplicate the code in the new package instead.
Thanks for reporting, and we apologize for the problem caused.

Sure thing!
It didn't cause any big disruption, no worries.

@jlprat fyi I released 3.0.8-RC1, do you mind to try it to see if it fixes this issue?
Thanks.

It works with RC1 :+1:
Thanks a lot!!

For reference: https://github.com/akka/akka-http/pull/2477

Not that there is a big hurry on it, but is three any E.T.A. for 3.0.8 release?

@jlprat just for your fyi 3.0.8 is now released.

Thank you very much @cheeseng
I already created a PR using the new types and it works on CI.
Feel free to close the ticket, or I can do it myself if you prefer.

Thank you @jlprat , closing this now.


Installation breaks after Make Install when calling tlmgr

When I try to add the buildpack I get
Unknown directive ...containerchecksum c59200574a316416a23695c258edf3a32531fbda43ccdc09360ee105c3f07f9fb77df17c4ba4c2ea4f3a5ea6667e064b51e3d8c2fe6c984ba3e71b4e32716955... , please fix it! at /tmp/build_5fd198f2598d92d2db67775fd7733513/Prescrypto-prescript-e3d5a91f350e910cbcb892fa303da9fb58de9185/.texlive/tlpkg/TeXLive/TLPOBJ.pm line 201, <$retfh> line 5579.
Google searches point me towards things like:
https://darrengoossens.wordpress.com/2016/06/20/a-little-trick-with-tlmgr-unknown-directive-containerchecksum-error/
http://tex.stackexchange.com/questions/313768/why-getting-this-error-tlmgr-unknown-directive
But I still can't figure it out.

FIxed it with the following line:
tlmgr option repository ftp://tug.org/historic/systems/texlive/2012/tlnet-final

This happens only when you install this buildpack after the original: https://github.com/gulopine/heroku-buildpack-tex

If you used this buildpack, you need to clear the cache with heroku-repo plugin and purge_cache command


Distinction between Assert::set_stdin() and with_stdin is confusing

No idea if it doesn't actually work, or if I'm doing something wrong and API is simply allowing for it, but either way it should be fixed.
assert_cmd = "0.6"
my test case:
#[test]
fn broek_dep() {
    Command::new("cat")
        .assert()
        .set_stdin(vec![0])
        .stdout("\0")
        .success();
}

Which results in:
thread 'broek_dep' panicked at 'Unexpected stdout
command=`"cat"`
stdin=``````
code=0
stdout=``````
stderr=``````
',

Also on a side note, output of assert_cmd when there's an error is rather confusing and not helpful when you're trying to deal with input/output that consists of raw, often unprintable, bytes. Debug output without additional ``` would work a lot better IMO.

Are you converting over from assert_cli?

No, but I've found out about assert_cli first from a blog post, and then I've looked at docs of both crates to see what's different.

Any ideas for better names or other means to help people use the right function and avoid the one more so meant for internal purposes (but is pub for anyone writing custom logic)

I think that having something like a bold Warning in the documentation of the function explaining that this might not be what they want and recommending using with_stdin instead would do the trick.

Forgot to reply to:

Also on a side note, output of assert_cmd when there's an error is rather confusing and not helpful when you're trying to deal with input/output that consists of raw, often unprintable, bytes. Debug output without additional ``` would work a lot better IMO.

Please open a distinct issue for this

I think that having something like a bold Warning in the documentation of the function explaining that this might not be what they want and recommending using with_stdin instead would do the trick.

Documentation just patches over problems and only helps so far as it is read (e.g. the current documentation already pretty clearly states how the parameter to set_stdin works).
I'm considering things like

Changing the name (adding "context")
Making this functions pub(crate) until there is a need and handle it then
Try to make clearer the idea that the problem has already run by the time Assert exists.


Add Logging Support

In order to be better able to diagnose rendering issues, or expose Vega logger output, we should expose log information to the visual via a custom logger.

Initial functionality introduced in 1.3's first alpha build. If there are any further issues, please create a new issue with the prefix [1.3 Alpha Testing] and I'll flag for follow-up.


Use BIP39/BIP44 for new account

For historical reasons, Parity products use a non standard logic and words dictionary for account recovery phrases. The Parity recovery phrases are not compatible with wallets such as Metamask, Jaxx or MyCrytpo.
We should:

Still support legacy parity seed phrase for account recovery
Support BIP39 recovery phrases
Implement and use BIP39 for new account generation.

In "Import account", what about asking the user what type of phrase he wants to recover from, before asking them to input the recovery phrase?
screen 1:
what method would you like to use to recover your account?

standard phrase
parity phrase (legacy)

screen 2:
type your recovery phrase
in the account creation we would use a bip39 phrase. is parity_generateSecretPhrase going to be updated with bip39? or should we generate the phrase on Fether's side?

I believe we are updating the relevant rust crypto library, for this issue https://github.com/paritytech/parity-signer/issues/154
Having an automated word detection (since word disctionnaries are fairly different) and retrieve the first account derivation for Bip39 is good enough for now. We should generate a 12 words recovery phrase for new accounts.

It looks like Parity has 7776 entries in its wordlist versus the 2018 entries of BIP39, and the words are fairly different indeed, so automatic detection should work! BIP39 includes a checksum; Parity phrases are just random words; so in case a recovery phrase exclusively uses words that appear in both wordlists, we can check if it's a valid BIP39 phrase, and if not we consider the phrase to be a Parity legacy phrase.
Need to follow up on what's being done on the Rust side re bip39 (blocking)
Account recovery should also accept phrases with more than 12 words as well I imagine: For BIP39 related stuff: both Trezor and Ledger default to 24 words (256 bits entropy), while popular web/mobile wallets (MetaMask/MyCrypto/Jaxx/BRD) default to 12 words (128 bits entropy). Out of the latter, only MyCrypto allows you to recover wallets for 12, 15, 18, 21 or 24 words, all others are hardcoded for 12 words, which means that if you have an actual hardware wallet, you can only recover it on another hardware wallet or MyCrypto (persumably also MEW and other stuff we haven’t tested

Implementation-wise
Create account

Rather than calling parity_generateSecretPhrase and parity_newAccountFromPhrase, we use some JS library (check work on Parity Signer / Metamask / MyCrypto) to generate a 12-word BIP39 phrase and derive the private key from it; we create the new account on the node via private key using parity_newAccountFromSecret

Recover account

If the phrase entered is a 12-word to 24-word valid BIP39 phrase, we derive the private key and create the account on the node via parity_newAccountFromSecret
Otherwise, if it's a 12-word phrase consisting solely of words from the Parity wordlist, we proceed as currently (parity_newAccountFromPhrase)
Else, reject. (In case of brain wallet the user can still add the account by calling RPCs directly on the node)

No changes to the UI/UX


Removed Strict ROS Melodic Dependency In 'compile_ros_workspace.sh'

I believe the --rosdistro melodic flag can safely be removed and allow this repo to still support ROS Kinetic (even if unofficially). With this removed everything appears to work with Kinetic.
P.S. Thanks for the great project! This was exactly what I needed.

I'm happy this helped you. However, how does the generator know which distro to pick if it's not specified in the command line?

Ah, I had assumed that it was detecting the operating system, but upon further inspection I've realized it was picking up the ROS_DISTRO environment variable. Would it be acceptable if I were to make the distro a flag for compile_ros_workspace.sh, and have it default to melodic?

Oh, in that case just remove the flag and add ROSDISTRO="${ROSDISTRO:-melodic}" to pass through the current setting but set a default. So you'll be able to tweak it if needed.

Sure, no problem. Changed!


grpc timeout setting

I want to set the timeout time of grpc for breakpoint debugging. I found that it can be passed through the ClientOption function. However, the ClientOptions defined in ClientOption is internal and cannot be compiled.
// MustNewClient returns a Client, exits on any error.
func MustNewClient(c RpcClientConf, options ...ClientOption) Client {
	cli, err := NewClient(c, options...)
	if err != nil {
		log.Fatal(err)
	}

	return cli
}

It's type aliased in zrpc package. You can use it.


Missing tried-and-tested syntax-theme?

According to the FAQ, there should be some tried-and-test syntax-themes "featured above" (that FAQ entry).
Where is it?

In the README, here.


Checks whether the rootfs has UDI or Subiquity snaps

Currently only checking for UDI is relevant. Soon Subiquity-only will be as well, which will break the possibility of rendering a GUI from inside Linux, since there will be no GUI there anymore.
Thus, this pull request modifies the function that checks whether the distro is GUI-enabled to check for the existence of the UDI snap.
Since I wanted to reuse your implementation of starts|ends_with algorithms, I created a new header to host them (plus some other bits with similar scope).

👍
In the future maybe we should refactor a bit and put all algorithms tests in their own test suite.
Also, some functions around the codebase might be better here as well. Things like wide_string_to_utf8, or safe_execute as quick examples.
That I'd say is outside the scope of this PR, so approving.

I like the idea.