removeMaskOnSubmit

I don't think this is an actual issue with the plugin but rather me not understanding how to successfully set the removeMaskOnSubmit option.  Someone else installed the plugin and I'm trying to figure out how to correct it so that when a search is submitted, the mask is removed.  If I remove the attributes all together and just have the raw input sent to MySQL db, the query works fine, but I can't seem to get it to work with the input mask attributes.  In the JS file we have:
masked_inputs: function () {
Inputmask().mask(document.querySelectorAll("input"));
and in our php file we have the following attributes:
data-inputmask="'alias': 'numeric', 'groupSeparator': ',', 'autoGroup': true, 'digits': 0, 'digitsOptional': false, 'prefix': '$ ', 'placeholder': '0'"/>
When I add 'removeMaskOnSubmit': true as an additional attribute in the php file; it still doesn't work.  On the front-end, the attributes work, meaning the masks are displaying how they should; however, the parameter does not work when a search is submitted to our MySQL db. Thank you in advance for any assistance that can be provided in order to get this to work.

@universe416 ,
Or you add the removeMaskOnSubmit to the data-inputmask attribute or add data-inputmask-removemaskonsubmit="true".  This options is triggered by a form submit,  Verify the formdata in the briowser dev tools.

Thank for the reply Robin.  I'm not sure if part of your answer accidentally got deleted, but I don't understand what you wrote.  Like I said in my question, I added removeMaskOnSubmit  to the form attribute, and it does not work.  Perhaps you can provide a working example?

@universe416 ,
Can you create a jsfiddle so I can see what you exactly did.

I can't, unfortunately.  Our html is as follows:

Sale Price Range

Min Sale Price

Our JS file related to this plugin is as follows:
masked_inputs: function () {
Inputmask().mask(document.querySelectorAll("input"));

@universe416 ,
https://jsfiddle.net/qddLqqgx/3/
Open the fiddle, open dev tools browser (f12) see in the network tab
Type something in the input press submit => verify formdata of the request.
Btw you can set code snippets by typing ```   at the beginning and ending

The way your example is setup is exactly how I have added the attribute to the form; however, when I submit the search, its as though the form input parameter that I added the attribute to is skipped.  Its not just due to the removeMaskOnSubmit option; it no longer works when I add any data-inputmask options.  Could it be a conflict with something else?  The form works fine when I remove the data-inputmask attributes all together.

@universe416 ,
When you submit your form what is the formdata.  Can put a copy here.  Or send it to my gmail if you prefer.

What would be ideal is if we could do a screen share so I can show you the entire thing.  I'm happy to pay you for an 30mins-hour of consulting if you would be willing.  I'm not a front-end dev expert (I deal mostly with DB mgmt).  Our front-end dev is currently away on vacation.

@universe416 ,
Ok, but can this wait till monday?

The sooner the better.  I'm going to keep torubelshooting it on my own.  Our front-end dev is coming back sometime next week (not sure what day).  Please let me know whenever you have the time to allocate and I'll let you know if I solved the problem by then.  Not sure how to get your email but I prefer not to post mine on this open post.

I've been struggling for almost one day with a similar problem. The value submitted still had the mask applied even when I have set the removeMaskOnSubmit option.
I'm using Ruby on Rails and imported this plugin JS files using the gem 'jquery-inputmask-rails' that packages your plugin for Rails, but I just realized that it's completely outdated.
It was solved using a Git fork of that gem that is keep up to date (now packages version 3.3.4)

The jquery.inputmask.bundle.js file we have is our app indicates the follow:
/*!

jquery.inputmask.bundle.js
https://github.com/RobinHerbots/jquery.inputmask
Copyright (c) 2010 - 2016 Robin Herbots
Licensed under the MIT license (http://www.opensource.org/licenses/mit-license.php)
Version: 3.3.1
*/

Robin, is this the not the most up-to-date version? If not, could this be the cause of our problem?

I downloaded the most up-to-date file from Github and replaced it; same problem exists..

Which scrips are loaded?
Op do 22 sep. 2016 18:11 schreef universe416<EMAIL_ADDRESS>
I downloaded the most up-to-date file from Github and replaced it; same
problem exists..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/RobinHerbots/Inputmask/issues/1394#issuecomment-248950296,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATb70_AUZlTHEo-xTfbs9DwjKSqJMFlks5qsqingaJpZM4KDWlf
.

@universe416 ,
Just a thought, is there somewhere in the code a binding to the submit event?

Not sure...the form data is passed to a MySQL query.

Hi Robin.  Can we communicate through email?

<EMAIL_ADDRESS>Op vr 23 sep. 2016 18:13 schreef universe416<EMAIL_ADDRESS>
Hi Robin. Can we communicate through email?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/RobinHerbots/Inputmask/issues/1394#issuecomment-249235477,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATb7wbhRQZk-5Z3cdMQB-EOek9rE4vDks5qs_qlgaJpZM4KDWlf
.

Hi Robin...do you realize that the jfiddle you created does not work?

@universe416 ,
In a developers point of view it works.  Look at the submitted formdata.

sir i also have the same problem as this....i tried the same as what you write up there in jsfidle but not working... and also just downloaded the newest version...

@t0n1zz ,
How are you submitting the data?

Listen to Robin.

i am not sure how to answer that question, but i use laravel framework for handling backend... maybe that's will help?

Are you using a jquery date picker of some sort?  I'm using the datepicker from uikit.

no, for date input i only using inputmask but beside using inputmask i also using this https://github.com/1000hz/bootstrap-validator for validating form input.

what's the purpose of the date inputs in your application?  Are they date ranges?
(e.g. min/begin date max/end date)

If so.. your PHP page that include the HTML input might look something like this (please note, the md/material design and uk/uikit is relevant to the file I took this out of to provide an example for you, so Replace Input_Date with whatever you'd like)
<input id="Input_Date" name="Input_Date[mindate]" type="text" class="md-input" data-uk-datepicker="{format:'MM-DD-YYYY'}" value="<?php echo (isset($_SESSION['form_data']['Input_Date']['mindate']) ? $_SESSION['form_data']['Input_Date']['mindate'] : ''); ?>" />
and your php sql query might look something like this...
} else if(isset($value['mindate']) && isset($value['maxdate']) && $value['mindate'] != '' && $value['maxdate'] != '') {                 $valMIN = date_format(date_create_from_format('m-d-Y', $value['mindate']), 'Ymd');                 $valMAX = date_format(date_create_from_format('m-d-Y', $value['maxdate']), 'Ymd');                 $sql[] = " $key BETWEEN $valMIN AND $valMAX";

no, just make sure they type dd/mm/yyyy,
but my problem is not in date input, i already have a way to handle that, my problem is in thousand separator... so when i type 1000000 it will appear 1.000.000, it just for better user experience and the problem is when i post the form..
what i expect is 15000000 and what i get is 15.000.000 and the result is what saved into my mysql database is 15 , just the number before thousand separator.... because my field in mysql is float not varchar

No PHP included....This works for me...
<input id="your_input" name="your_input" type="text" class="md-input" maxlength="14" data-inputmask="'alias': 'numeric', 'groupSeparator': ',', 'autoGroup': true, 'digits': 0, 'digitsOptional': false, 'autoUnmask': true, 'removeMaskOnSubmit': true"/>

BTW..if the field does not have a decimal, maybe consider changing the field type in MySQL to BIGINT

I'm a little confused.  What does the input represent?  A date? number? decimal? geometry? sign language?

represent value for financial report and it will be too short if i using bigint so i use float since it have the biggest one.....

well i think we are talking a little out of scope... my question is just how to make removeMaskOnSubmit working.... since it is not working in here

@t0n1zz ,
The removeMaskOnSubmit only works on a simple form submit.  When using an ajax call, the removeMaskOnSubmit is not used.  In the case you subscribe, I guess using autoUnmask and unmaskAsNumber will do.
If this doesn't work, create a new issue with an example in a jsfiddle.
I am going to close this issue.


Method name already has 'Async' suffix

In API.md under promisifyall, it is explained:

If a method name already has an "Async"-suffix, it will be duplicated. E.g. getAsync's promisified name is getAsyncAsync.

However I have also received this error before

This error indicates you have tried to call Promise.promisifyAll on an object that already has a property with the Async suffix:

The above error is thrown from line 40 in src/promisify.js
    // Verify that in the list of methods to promisify there is no
    // method that has a name ending in "Async"-suffix while
    // also having a method with the same name but no Async suffix

I assume the documentation just needs to be updated, but wanted to create an issue before blindly creating a pr.

?
On Wed, Feb 11, 2015 at 8:30 AM, olsonpm<EMAIL_ADDRESS>wrote:

Closed #491 https://github.com/petkaantonov/bluebird/issues/491.
—
Reply to this email directly or view it on GitHub
https://github.com/petkaantonov/bluebird/issues/491#event-233778781.

Ah just got confused with the wording.  The above is perfectly valid.


github actions: master build and use -latest runners

Now that we 'only' release once a day we should run tests on each
merge to master, just to be safe. There's a few paths how things get
merged to master that are red, e.g. two PRs that are merged at the
same time that bring in incompatible changes. Or simply someone using
admin rights to merge.
Also use the -latest runners to avoid being in the 'no runner
available because they've been sunsetted' situation again.

no idea where this comes from, those are the ones that are being dropped in this very PR...


fix: type exports

类型导出好像有问题的样子
There are types at '——/node_modules/vue-flex-waterfall/dist/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'vue-flex-waterfall' library may need to update its package.json or typings.

感谢，已发布 2.3.1


Documentation for new release wasn't generated

there is new kubevirt release v0.3.0-alpha.0, documentation wasn't generated for this one.

Maybe because it was not an annotated tag?

@fabiand after v0.3.0 release is out (annotated tag), it is obvious that it didn't get generated.
I will make sure that it does.

Good catch, @lukas-bednar!

https://travis-ci.org/kubevirt/kubevirt/builds/350727371
Skipping a deployment with the script provider because this branch is not permitted

Please wait, it's still building here:
https://travis-ci.org/kubevirt/kubevirt/builds/350738276

Same result
Skipping a deployment with the script provider because this branch is not permitted

I see it now. You will need  to add another deployment with tag: true instead of branch: master. It can have identical content if you don't need $TRAVIS_TAG to specify the version.

Also see the duplication for the container deployment in .travis.yaml:
- provider: script
  script: docker login -u="$DOCKER_USER" -p="$DOCKER_PASS" && make publish
  skip_cleanup: true
  on:
    branch: master
- provider: script
  script: docker login -u="$DOCKER_USER" -p="$DOCKER_PASS" && make publish DOCKER_TAG=$TRAVIS_TAG
  skip_cleanup: true
  file:
  on:
    tags: true

As an alternative to not duplicate it, you can use a custom condition in the on section.

@rmohr thank you for suggestions, I will go over it.

@rmohr at the end I had to duplicate it anyway, because that condition is an addition condition to branch & tags options.
So if I will leave branch: master and add condition: -n $TRAVIS_TAG then when running for annotated tag deployment will be skipped anyway because Skipping a deployment with the script provider because this branch is not permitted.


Update bug report, and feature request template

To have pre-fixed fields which people can fill in.

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] Review requested

Examples
Bug report: https://github.com/wdg/CodeEdit/issues/new?assignees=&labels=bug&template=bug_report.yml&title=🐞+-+YOUR_DESCRIPTION
Feature request: https://github.com/wdg/CodeEdit/issues/new?assignees=&labels=enhancement&template=feature_request.yml&title=✨+-+YOUR_DESCRIPTION

@wdg : #267 ?

@wdg : In what sense you have not seen, if you have commented as well: https://github.com/CodeEditApp/CodeEdit/pull/267#issuecomment-1079731633

@wdg @Angelk90 can we merge these two PRs? You two figure this out please. Nice work, both of you!

@wdg : In what sense you have not seen, if you have commented as well: https://github.com/CodeEditApp/CodeEdit/pull/267#issuecomment-1079731633

Then i didn't recall correctly, it's 4 days ago. Still, i don't really see an issue.

@austincondiff : I made a pull request with some suggestions let's see if it accepts.

@Angelk90 Do you have discord/twitter?, i'd like to discuss some things.
e.g.
In your pull you use
  - type: textarea
    attributes:
      label: "Desktop (please complete the following information):"
      value: |
        CodeEdit: `[e.g. 1.0]`
        MacOS: `[e.g. 12.3.0]`
        Xcode: `[e.g. 13.3]`

maybe we should use:
  - type: textarea
    attributes:
      label: "Version information (click on the version number on the welcome screen):"
      value: |
        CodeEdit: [e.g. 1.0]
        MacOS: [e.g. 12.3.0]
        Xcode: [e.g. 13.3]

@wdg : The initial purpose for which I developed that feature was for this.

@wdg : Have you seen the pr?

@Angelk90 Do you agree with the changes?
https://github.com/wdg/CodeEdit/issues/new?assignees=&labels=bug&template=bug_report.yml&title=🐞+-+YOUR_DESCRIPTION

@wdg : Have you seen the pr?

@Angelk90 Did you submit a new PR? Should i merge this, so you can make changes?

@wdg : Looks good.

87a5b3f4c18238c0d06cbfce595e1ca9b11cfd8e

87a5b3f4c18238c0d06cbfce595e1ca9b11cfd8e

#337 11765e7f0ac73c0e14d57a27ee31b7f940355277


Check for presence of 'bundler' gem

Fixes #25

@daidekman Can you confirm that this branch is working for you?
This should be the correct URL for the buildpack: https://github.com/febeling/webpack-rails-buildpack.git#bundler-test-25

@febeling I have confirmed that the Heroku deployment worked with that branch URL for the buildpack. Thanks so much for the quick and excellent attention on this!

@daidekman Thanks for reporting and testing!


  Here is an example.
split_lines
def split_lines(str):
return str.split('\n')
This is bad coding - so please rename str to something else.

Feel free to open a PR with a better naming convention. Some of these snippets have been written by people that come from a non-Pythonian background and we might have made some mistakes in the process. I would love to see some things like this, that might fly under the radar for us, fixed by the community.

Thanks for the feedback. If you don't feel like opening a PR just let me know the proposed change here and I'll do it for you 👍

I would like to tweak the functions. Would you do that function by function or in a single pull request?
Going through the examples I found many stumble stones. I am not perfect myself, but I could give it a first wash:

Naming is one thing: don't use str (built-in), filter (built-in), string (std module)
Creating a function called zip() is "sub-optimal" to say the least but also shadows the real built-in "zip" function which does exactly what the function tries to do: https://docs.python.org/3.5/library/functions.html#zip

I have for example adapted chunk.md - based on recipe found on stack-overflow... do you mention those sources: # https://stackoverflow.com/questions/9671224/split-a-python-list-into-other-sublists-i-e-smaller-lists

title: chunk
tags: list,intermediate
Chunks a list into smaller lists of a specified size.
By using  the step parameter of range() we create a list of indices
where to start each sub-list. By employing these indices we can create
the sub-lists by splicing lst.
def chunk(lst, chunk_size):
    return [lst[x:x + chunk_size] for x in range(0, len(lst), chunk_size)]

chunk([1,2,3,4,5], 2)  # [[1, 2], [3, 4], [5]]

Or then count_by.md:

title: count_by
tags: list,intermediate
Groups the elements of a list based on the given function and returns the count of elements in each group.
Use map() to map the values of the given list using the given function.
Use collections.defaultdict to avoid the check whether a key is present in the count_dict.
Iterate over the list and increase the counter for each mapped element.
Freeze count_dict after filling it.
from collections import defaultdict

def count_by(lst, fn=None):
    if fn is None:
        fn = lambda x: x
    count_dict = defaultdict(int)
    for el in map(fn, lst):
        count_dict[el] += 1
    count_dict.default_factory = None
    return count_dict

from math import floor
count_by([6.1, 4.2, 6.3], floor) # {4: 1, 6: 2}
count_by(['one', 'two', 'three'], len) # {3: 2, 5: 1}

If you allow me to push changes, I will gladly bring them in. Should I do a pull-request for each md or can I pull them in all together?

@Dekker1967 I would rather see changes bundled up in PRs thematically (e.g. replacing all uses of str variable), so we can more easily check them.
Sources are not something we are doing across repos for now, but you can mention in the PR for further reference.
zip itself is something I didn't know existed, leave it as is and we will figure it out independently (I'll get back to you later about this).

Meanwhile I went through much more examples. The variables named 'str' and 'filter' are the least problem. Forget about the inconsistent naming (lst vs arr). But they show the underlying issue: I realized that the person who wrote all this examples is not very savvy in the Python language. Most of the "issues" the 30-seconds-function try to solve, are already solved (mostly solutions are "hidden" in the modules: itertools, collections, random). Often the created function do not feel very pythonic (eg. the all_equal.md feels c-like where we can compare by pointer, but the solution proposed will create two new lists in Python, which is a big expense for large lists; the Python solution would be more something like: return set(lst) <= 1. Much easier to grasp! And works also for empty lists.). Finally the desire to cast all the results to list doesn't feel right and very efficient. Is the result really needed as list? To begin with: Maybe the developer should have used a set from the beginning because a set can be expanded: set.add(). Data structures are here for a purpose.
I like the idea of 30-second-functions. But as they are now, they are not very helpful. This is not a question of renaming 'str' to something else... all the content is affected. I analyzed the last few functions and added some notes to each function. I retreat my offer to help but I wish you all the best for your project!
Here are a few comments on the last few function (alphabetically):
zip.md - not needed is a built-in python function
values_only.md - no need to create a function for that .values() returns a list of values - no need to surround it with list()
unique_elements.md - no need to create a function for that
union_by.md - this is a very specific use case
union.md - why always render result into a list - a set has its advantages and a "cast" to list should happen for a reason
tail.md - a short and concise one-liner... is it worth to create a function for that?
symmetric_difference_by.md - analoguous to symmetric_differenc.md: list(set(map(fn, a)) ^ set(map(fn, b)))
symmetric_differenc.md - the following code does the same: list(set(a) ^ set(b)) --> although I would not return a list
sum_by.md - a very specific use case and in the end almost no code - is it worth a function?
spread.md - a very specific use case - who mixes lists_of_ints and ints into the same list - the "expected" case would be: [[1],[2],[3],[4,5,6],[7],[8],[9]]
split_lines.md - this is not worth a function... split('\n') is enough - no need for indirection; 'str' is a bad variable name (shadows str-function)
some.md - exists already in the form of any()
l1 = [0, 1, 2, 0]
l2 = [1, 1, 1, 1]
l3 = [0, 0, 1, 0]
l4 = [0, 0, 0, 0]
print any(map(lambda x: x >= 2, l1))
print any(map(lambda x: x >= 2, l2))
print any(l3)
print any(l4)
snake.md - https://stackoverflow.com/questions/1175208/elegant-python-function-to-convert-camelcase-to-snake-case
similarity.md - achieved via: set(a) & set(b)
shuffle.md - module random --> random.shuffle(lst) or if you want a copy: random.shuffle(list[:])
sample.md - module random --> random.sample() is your friend

@Dekker1967 PR anything you see being "not very pythonic" and we will get right on it. Like I said, some of the people who contributed the content might have been either beginners or people not very well-versed in Python.


middleware locale redirector should NOT redirect on POST and AJAX requests

Tests added and code peer reviewed and check.py reviewed

We decided to actually patch urlresolvers after all.  We're just waiting on filing a Django bug so we can link to it in the comments. https://github.com/mozilla/funfactory/pull/8

This seems stale... I am closing.

Maybe during pycon I'll go on a Django bug-filing spree for all our monkeypatches


seems a process is blocking some frontend actions

It seems that in the block explorer there's a process blocking some actions... like: sometimes when I click on a block, it takes a while until it appears something...
I think the indexing process is blocking some frontend actions, somehow. It's not perceptible on fast machines (or when you're not utilizing a lot of CPU).... but if you try on an overloaded system, you can feel it, and it's bad. 😄
I tried to show it in the following gif, but the exact moment when I click on the block is (unfortunately) not visible. But maybe you can realize that I clicked on block 6, then moved the mouse a bit down, waited some secs, then when I was going to click again the block 6 box suddenly appeared.

I suspect this may manifest more on machines that are IO constrained. There are a few places where I need to read small files from disk within the main extension process and have lazily used sync reads.
The extension and the panels both have their own single-threaded JS process and they communicate by posting messages between each other.  I believe that the UI is posting messages to the extension at a time when it is blocked synchronously reading a file, when it is done it will post a message back (causing the panel to update) but it could be multiple seconds after the event triggering the update when IO constrained.
Will make all file IO async...

I suspect this may manifest more on machines that are IO constrained. There are a few places where I need to read small files from disk within the main extension process and have lazily used sync reads.
The extension and the panels both have their own single-threaded JS process and they communicate by posting messages between each other.  I believe that the UI is posting messages to the extension at a time when it is blocked synchronously reading a file, when it is done it will post a message back (causing the panel to update) but it could be multiple seconds after the event triggering the update when IO constrained.
Will make all file IO async...

I suspect this may manifest more on machines that are IO constrained. There are a few places where I need to read small files from disk within the main extension process and have lazily used sync reads.

yeah, that sounds like a really good guess on where to start... 😄

Will make all file IO async...

alright... I can try to test it once a new version is out. 👍

I suspect this may manifest more on machines that are IO constrained. There are a few places where I need to read small files from disk within the main extension process and have lazily used sync reads.

yeah, that sounds like a really good guess on where to start... 😄

Will make all file IO async...

alright... I can try to test it once a new version is out. 👍

alright... I can try to test it once a new version is out. 👍

@gsmachado can you give this one a whirl? https://github.com/ngdenterprise/neo3-visual-tracker/releases/tag/v0.0-master
It has most of the synchronous file IO removed (there are a few remaining that are architecturally harder to remove, but also not in any background code paths).
If you are still seeing issues, a Performance trace from the Chrome dev tools within VS Code would be useful (let me know if you need further instructions for generating this).

alright... I can try to test it once a new version is out. 👍

@gsmachado can you give this one a whirl? https://github.com/ngdenterprise/neo3-visual-tracker/releases/tag/v0.0-master
It has most of the synchronous file IO removed (there are a few remaining that are architecturally harder to remove, but also not in any background code paths).
If you are still seeing issues, a Performance trace from the Chrome dev tools within VS Code would be useful (let me know if you need further instructions for generating this).

Ok, some feedback after some testing: I tried to test in docker containers (with constrained IO, which I can set some policies), and yes, it's better. However, I can still feel a small delay in some actions (that the UI becomes irresponsive for a fraction of a second), mainly in the block explorer part.
However, to be honest, it's good for now. 🎉  💪
If I were you, I would keep the task to remove the few remaining synchronous file IO calls in the backlog, just in case. 😄
I will close the issue now. Looking forward to a release with that fix. 💯

Ok, some feedback after some testing: I tried to test in docker containers (with constrained IO, which I can set some policies), and yes, it's better. However, I can still feel a small delay in some actions (that the UI becomes irresponsive for a fraction of a second), mainly in the block explorer part.
However, to be honest, it's good for now. 🎉  💪
If I were you, I would keep the task to remove the few remaining synchronous file IO calls in the backlog, just in case. 😄
I will close the issue now. Looking forward to a release with that fix. 💯


mutiple hosts with different username and passwords

I have use case where i will be creating new machines and setting up different users for access. So is there any way i can provide different username and passwords using testinfra.

Testinfra cannot handle password yet #34 , so you must setup a password less key or use ssh-agent.
But you can connect with different users:
testinfra --hosts=user1@host1,user2@host2
You can also provide a ssh-config with --ssh-config
See also the documentation for connection backends: https://testinfra.readthedocs.org/en/latest/backends.html


Support for react native 0.72.x

Hi, react native 0.72 was released a while ago and with 0.73 RC released, is there any plans in the near future to update SDK to allow it on 0.72 without hacking peer dependencies?
  "peerDependencies": {
    "react": "^17.0.2 || ^18.0.0",
    "react-native": "<=0.71.8"
  },

Hi @jangsoori , will be updated in new version of Exponea RN SDK

When is the new update coming out? RN 0.71 has a bunch of issues, that are solved with newer releases and I would love to update already

Hi @karel-suchomel-ed , we are currently working on new version of React Native SDK -> should be available during next week.


Fix#110

fixes #110

Looks like this may have inadvertently caused another issue. One of my developers is seeing an issue with the private key password.
Looking at the source code of cert-utils-operator which we use to generate keystore from tls secrets – I noticed that it is changed recently to fix frequent restarts of the pods every time the operator is restarted.. Below is the issue and the commit that fixes it.

https://github.com/redhat-cop/cert-utils-operator/issues/110
https://github.com/redhat-cop/cert-utils-operator/commit/c55ca62a3d9b67529476744c07eaa9097a648486#diff-266e243251ed42b2e1bcf54f2a041cdd69176644a9f4b4e2ec140edd32ea56e3L179

But the problem here is when the private key is being created and added to the keystore – it is added without any password – this line: https://github.com/redhat-cop/cert-utils-operator/blob/c55ca62a3d9b67529476744c07eaa9097a648486/controllers/secrettokeystore/secret_to_keystore_controller.go#L265

It needs to be changed to []byte(getPassword(secret)) instead of []byte{} – which means it is setting up empty password for the private key. And to test my hypothesis – I tried accessing the private key inside the keystore using an empty password and I was able to access it just fine.

So to sum it up – Keystore password is set to “jbosscert” and key password is set to empty string when the JKS files are generated.


Test suite does not compile with C++20 and Clang 17

Description
With Clang 16, the test suite could be compiled with C++20, see https://github.com/nlohmann/json/actions/runs/4364410245/jobs/7631709068
When switching to Clang 17, there is an error: https://github.com/nlohmann/json/actions/runs/4409390311/jobs/7725631191#step:7:68
Reproduction steps
Compile the test suite with Clang 17 and C++20 standard enabled.
Expected vs. actual results
Expected: Test suite compiles.
Actual: Error message:
json/tests/src/unit-constructor1.cpp:282:34: error: invalid operands to binary expression ('Expression_lhs<const basic_string<char, char_traits<char>, allocator<char>> &>' and 'const value_type' (aka 'const nlohmann::basic_json<>'))
            CHECK(std::get<2>(t) == j[2]);
            ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~

Minimal code example
No response
Error messages
json/tests/src/unit-constructor1.cpp:282:34: error: invalid operands to binary expression ('Expression_lhs<const basic_string<char, char_traits<char>, allocator<char>> &>' and 'const value_type' (aka 'const nlohmann::basic_json<>'))
            CHECK(std::get<2>(t) == j[2]);
            ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~

Compiler and operating system
Clang 17, Ubuntu
Library version
develop
Validation

[X] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

The CI now contains multiple jobs to compile with C++20 - both with Clang and GCC.


LPFG-948: added frequency to audience configuration

This doesn't appear to be working

replaced by: https://github.com/Civil-Service-Human-Resources/lpg-management/pull/197


Podman slurm

Support for Slurm Runner with Podman

@ivotron Can you take a quick look at this?

thanks Eddie! 🙏


No flow is given in the code

Please share the flowchart of the given code so I can modify the same.
Thank you.
Kartikey Kushwah

Hey @cosme12
Are you still considering adding a flowchart for the code ?
I believe as the code gets more complicated , maintaining the flowchart and adding more details to it would be a burden
However , a high-level flowchart without so much detail would be good. for example , instead of explaining every step in the signature validation process , it would only be represented with a black box , titled "Validate Signature"
If this is what you're looking for , I'm interested in designing one for the project

Hey @cosme12 Are you still considering adding a flowchart for the code ? I believe as the code gets more complicated , maintaining the flowchart and adding more details to it would be a burden However , a high-level flowchart without so much detail would be good. for example , instead of explaining every step in the signature validation process , it would only be represented with a black box , titled "Validate Signature" If this is what you're looking for , I'm interested in designing one for the project

Sure, go for it. The idea is to make this project as simple as possible for new "users".

Hey @cosme12 Are you still considering adding a flowchart for the code ? I believe as the code gets more complicated , maintaining the flowchart and adding more details to it would be a burden However , a high-level flowchart without so much detail would be good. for example , instead of explaining every step in the signature validation process , it would only be represented with a black box , titled "Validate Signature" If this is what you're looking for , I'm interested in designing one for the project

Sure, go for it. The idea is to make this project as simple as possible for new "users".

Okay , Thanks!
Also it would be great if you assigned me to this issue
Thanks again!

Hey @cosme12
Please have a look at my PR (#91) and let me know if any changes are necessary
Thanks!


Text min/max length validation didn't work

@jeroenooms - On text prompt I entered min length of 5 then max length 0, app submitted the campaign.

I know, dynamic is a bit difficult to implement because the validity of one field can change depending on the other. Hopefully most users will understand that max has to be greater than min once they see this error.

@jeroenooms I added this validation to the server.

Added a visual hint on the client as well: https://github.com/mobilizingcs/author/commit/62137b90105d439ee411964258940c6825e8de88

fixed


Run buildifier

FYI, I ran buildifier on this PR so we might be able to abandon this

Abandoning since I just formatted the whole codebase in #155


feat: add "model" and "asciibib" to top level nav

We need to add Table of Contents to the AsciiBib, Citation guides and Model pages.


Disable the Learn plugin

The Explore plugin conflicts with the Learn plugin at the moment.
https://phabricator.endlessm.com/T33407

Ha! I was doing almost the same change but local docker was taking a while. Grabbing the APK from Jenkins now to test.


docs: Add Sphinx based docs

Requires PR #248 to go in first.
Do not merge this as is, as there is some debug info in here that needs to get cleaned up.

Hi @matthewfeickert - since we now do have a sphinx-based docs setup, is this PR still relevant?

since we now do have a sphinx-based docs setup, is this PR still relevant?

@ponyisi  If there are docs then no, this PR should get closed out. But the docs are not linked anywhere on the README on either master or 3.0_develop branch, the repository side bars, or the PyPI metadata
https://github.com/ssl-hep/ServiceX_frontend/blob/5bc9037e19016afc8adeaea211e4df01bb87071c/pyproject.toml#L58
. This means users don't know they exist and won't find them! :grimacing:
While this PR can get closed, my suggestion is to have a PR that:

Uses the Closes #249 syntax
Adds docs information to the README
Updates the project URL metadata

https://github.com/ssl-hep/ServiceX_frontend/blob/5bc9037e19016afc8adeaea211e4df01bb87071c/pyproject.toml#L57-L63

Thanks @matthewfeickert - the new version generates sphinx docs


Enhanced grade 'A' not shown correctly in Privacy Dashboard

I think this only happens with 'A'.
App version: 5.52.5
Device/OS: OP6, OOS10

Thanks for reporting this. I was able to reproduce it. 👍
Created an internal task https://app.asana.com/0/414730916066338/1173797528771833/f

@cmonfortep it seems to me that there is no need to reproduce it, because this is a feature of this image resource

This is fixed now. It will be included in our next release. Thanks.


How do we know which version of BrAPI is implemented? (and I mean sub-version)

In the URL we got "/v1/" but how how do we know if it is v1**.1** or v1**.2** or ...?
I propose to use the root "/" call (not defined yet) to display information about current implementation:
{     "version": "1.1",     "provider": "Musa Germplasm Information System",     "description": "Explore Banana Diversity",     "contact": {         "email"<EMAIL_ADDRESS>        "www": "https://www.crop-diversity.org/mgis/"     }, }

Indeed - maybe the root call could point to the swagger file for the server?

Another proposition has been made to add versioning to the "call" calls so it could be managed at a call level. It might be useful when you implemented all the calls at v1.2 and only updated a couple of other calls to v1.3 for instance...
But then, what to do with calls that changed name like "allelematrix" into "allelematrices"?

Closing: solved in v1.2
A versions array was added to each call response in the /calls call.
If the path of a call changes (ie "allelematrix" into "allelematrices") the old call will be marked Deprecated, but will still be present in the spec, and the new call will be added to the spec. so to support multiple versions, you will need to implement both calls, the new and the deprecated. If you only care about supporting the latest version, you can ignore the deprecated call.
Similarly, if a response object changes, any existing fields which are changed or removed will be marked as Deprecated and new fields will be added as needed. To support multiple versions, simply return all valid and deprecated fields. To only support the latest version, return only the valid fields.


Aggregate does not support options

The current aggregate method exposed by Mongoose does not support passing an options object such as {explain: true} to mongodb as the second parameter.
See: http://docs.mongodb.org/manual/reference/method/db.collection.aggregate/

That's by design - MyModel.aggregate() is currently designed to take a spread of pipeline stages. aggregate() returns an Aggregate object that you can manipulate options on, but currently there's no particularly elegant way to set explain: true on that object. That's something I should add. As a workaround:
var aggregate = MyModel.aggregate();
aggregate.options.explain = true;
aggregate.append(stage1, stage2 /* ... */).exec(callback);

If you're looking for suggestions, it should probably behave like the other mongoose functions, where options is passed as an additional parameter. ie:
MyModel.aggregate([stage1, stage2], options).exec(callback);

It would be icing on the cake if it supported .explain() like the others as well.
Thanks for the workaround.

Yeah I would think that would be the more ideal solution going forward, see #2716. For now we'll have to add an explain() helper to the Aggregate object in 4.1.


Add OSM map (Leaflet) to Tabi

I'd like to add Openstreetmap maps to my blog post, preferably in Leaflet library. I have done it in "normal" HTML but I don't know how to do it in Zola and Tabi. Maybe it is built-in Zola's feature but I don't know how to call it.

You can add HTML directly on your posts, and it'll be rendered normally. Have you tried that?
One issue you might encounter is related to CSP: you might need to allow the OpenStreetMap connection or disable CSP.

Thank you. I added IFRAME code generated by OSM.org and the frame was visible but no content. I added OSM to CSP rules and it works now. Thank you!

My problem is solved so I close the issue.

No problem! Glad to hear you solved it :)


Is it possible to retain only last N successful builds?

Hi everybody,
the docs on this page are not mentioning this explicitly, but does the "number of recent runs to keep for each pipeline" include only successful builds, or also failed ones?
For example: I want to make sure, that the latest successful build on "main" is always available. Now, if I configure the "number of recent runs to keep for each pipeline" to 10, but the next 10 runs are all failures, I would no longer have access to a successful build from "main" anymore.
So does the "number of recent runs to keep for each pipeline" count only the successful builds, or also the failed ones?
And, in case it accounts for both successful and failed builds, is there any way to ensure that the last successful build from a protected branch is always retained?
Cheers,
Torben

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

ID: 2931f833-b378-efa9-2c43-0d2afea22bd7
Version Independent ID: ba3138ef-dc42-16c9-fcb3-1af441be7eb8
Content: Retention policies for builds, releases, and test - Azure Pipelines
Content Source: docs/pipelines/policies/retention.md
Product: devops
Technology: devops-cicd-policies
GitHub Login: @juliakm
Microsoft Alias: rabououn

@tknerr - You probably want to reach out to the dev team for product questions. Also, you can make feature requests there.
Also, also, consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

If there are docs changes following on this, we can reopen this issue.

@TimShererWithAquent thanks, yes makes sense, I will check on developercommunity as well, so to request that as a feature in case it isn't one yet.
Still I believe the docs should be updated to be more specific on the current behaviour of "number of recent runs to keep for each pipeline" (once clarified).

FYI - I found a related thread on developercommunity and re-posted the same question there:
https://developercommunity.visualstudio.com/t/Retention-setting-to-keep-a-minimum-numb/606884?space=21&q=number+of+recent+runs+to+keep+only+successful
It's already closed, but hope to still get a response over there.

/cc @moswald who was active in the related developercommunity thread as well, maybe he knows the answer off the top of his head

Thanks for the quick response on the developercommunity thread @moswald ! 👍


New 1.8.1 release?

Hi,
Please make a new release of the changes since 1.8. I am especially interested in the SO_READDR stuff being available in a released version.
Thanks!

Hi Ian,
Let me just finish the two pending pull requests and we can do that.
Regards.

Ok great, looks like you are getting through them.

Any update?

Hi Ian,
I am ok with a new release, I am just too overwhelmed with work to stop and do it now.
Any volunteer?
Regards,
Marcelo.

Perhaps @ffontaine could take a look? I could do it if you tell me steps! 👍 Though I wont have access for SF etc.
Perhaps switch the release to be a simple github "release" it may be easier to releases in future, 1.8 is already done this way.

It can be done either place. I suppose you prefer to do it in github, I am fine with it, I can echo it to SF and create the tar.bz2 file, no problem.
Generating a new release is done by creating two commits. The first one updates the library numbers in the file configure.ac. This is a little tricky because of libtool numbering scheeme, but the instructions are in the same file. This is the "time consuming" part, because you should look at every commit since the last release to see if there is a change in code or interface. This commit is usually called "Adjust the library numbers for release" and it receives the tag with the library number like "release-1.8.2".
The second commit is usually called "Homekeeping for the next release" and adjusts some library numbers spread all over the project.
What I usually do is to check for the last release tag in gitk to see the diff, so that I can do something similar every time.
If you decide to do it, it will be great!
Best regards,
Marcelo.

Hello, I made PR #39. Code has changed in ixml (b87c294edced7966fc0959dc6e329a1189f04bda) and as threadutil has been merged in upnp, we can consider that a new interface has been added to upnp.

Release 1.8.1 is out. @v00d00, if it's ok for you, you can close this issue.

@ffontaine, Great, thanks!
(Can you take a look at #37 also :+1: )

Yes, I will look at it.


feat: Added styles.yml File in ISSUE TEMPLATES List

Related Issue

Information about the related issue No : #592

Closes: #592
Describe the changes you've made

Created styles.yml File in .github/ISSUE_TEMPLATES Section for Raising Enhancement and Code Style Related Issues

Type of change

Added Suitable YAML Code for styles.yml File

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update.
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)
Code of Conduct

[x] I agree to follow this project's Code of Conduct

Hi @amanjaiman1 . Please Review on this PR as in your Free Time and add Valid Level Label under GSSOC'23.


Two potential issues with MFA.py

(I'm a novice at Python so apologies if these are not correct observations)

In MFA.py, in line 55, 'all_num' is static since it is set in a prior loop in line 50. Should you either add 'all_num = all_nums[name]' or merge the two loops (line 45 and line 54)?

On line 88, you have "X[cols] / self.partial_factor_analysis_[name].s_[0]", but it does not seem to work with categorical pandas dataframes as X[cols] is not numeric and cannot be divided by a scalar

Hey!
You're right on both points. I just fixed them. Thank you so much for opening the issue.
Regards.


Public Date/Time Composer Error

When creating a new page where you have set the attribute for Public Date Time to required, it doesn't validate the submitted data and tries to obtain this information from the page itself.
During testing I found that this did not behave in the same way concrete/src/Page/Type/Composer/Control/CorePageProperty/NameCorePageProperty.php where it would actually get the information from the request if it was present.
Adding the above snippet in this PR seems to resolve the issue.

Looks like the above tracis test is failing due to docker cannot find base tables...

Looks like the above travis test is failing due to docker cannot find base tables...

Yep, that's fixed by https://github.com/concrete5/concrete5/pull/8433

Looks like the above travis test is failing due to docker cannot find base tables...

Yep, that's fixed by #8433

Great I'll wait, if I need todo anything else please let me know :)


ci: fix the gitlab trigger

There were two issues:

${{ }} converts the expression to a string, and strings are apparently always truthy
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idif
There was a typo in pr_data

Therefore, the check didn't work properly.

Merging, the triggers still work on a PR.


[Plugin] Notion

Summary

Notion is wiki/blog software we use as a home for technical documentation.  It can be organized using their concept of a "database", mainly a container for documents.  Backstage should have links to Notion databases, either ad-hoc, or database having same key fields as Backstage catalog YAML
Project website (if applicable)

https://developers.notion.com/
Context

We organize by micro-service; teams own microservices. Backstage holds the promise of centralizing and organizing all information related to a given service, but the current integration with Notion, where our rich doc lives, is just a link.  It should be able to at least render Notion databases (and maybe pages as well), link to databases by team, service, tag, discover and index new content for Backstage search.

Having a omni search that searches both Backstage and documents in Notion
I did a quick spike of this. The main issue is that by default you won't get any results when using the API. Each API token is linked to an 'integration'. To allow an integration to find, read and modify your pages you need to give explicit permission, for every page individually.
As far as I can tell there's no option to automatically make every new page available to certain integrations.

Hmm, we use glean which is like an omni search over a bunch of tools. It can index pages in Notion, but I'm not sure exactly how. I suspect there is some other automation or API that adds the integration to pages automatically so that they can be indexed more broadly. But that's just a guess. Someone from Notion could probably provide some guidance here.

I came here as I was almost looking to implement a bespoke solution for this until I realised we have a backstage instance spun up and starting searching.
I have done something using Streamlit but the issue I hit was the limitation of the Python Notion markdown converter in rendering different page items. Having seen some of the JS based alternatives like [notion-to-md[(https://github.com/souvikinator/notion-to-md) or react-notion It seems that backstage can be a good home for this idea!
To answer some of the concerns above from my knowledge:

The main issue is that by default you won't get any results when using the API. Each API token is linked to an 'integration'.
If you have all your docs under a parent page and having the integration enabled at the parent level then it will get inherited to all child pages. So you can have a parent "wiki" page and that's it. For us; we have all our docs in a database actually; this allow us to annotate docs with properties and then enabling the integration at the database level is sufficient

You then need two main components; one component to fetch the documents from that database (we have created a new mirror checkbox property that will only show documents that have been marked as such in the mirrored solution (Streamlit for now)) and a renderer that will take the page and fetch the markdown
This is how it looks now in Streamlit! but will have a look on how to do this in Backstage


tsdb: update vm to v1.76.1; expose more params

What problem does this PR solve?

Issue Number: close #212
What is changed and how it works?

Expose -search.maxUniqueTimeseries  to users. This is done by adding a new item to the configuration file.
Also add -retentionPeriod to the configuration file. (Note: cmdline arg take precedence over configuration file.)
Upgrade vm to v1.76.1 for -search.maxUniqueTimeseries to take effect.

Example:
# config.toml
# ...

[tsdb]
# Data with timestamps outside the retentionPeriod is automatically deleted
# The following optional suffixes are supported: h (hour), d (day), w (week), y (year).
# If suffix isn't set, then the duration is counted in months.
retention-period = "1"
# `search-max-unique-timeseries` limits the number of unique time series a single query can find and process.
# VictoriaMetrics(tsdb) keeps in memory some metainformation about the time series located by each query
# and spends some CPU time for processing the found time series. This means that the maximum memory usage
# and CPU usage a single query can use is proportional to `search-max-unique-timeseries`.
search-max-unique-timeseries = 300000

/merge

/cherry-pick 6.5

/cherry-pick release-7.1

/cherry-pick release-7.5


Rate Limiting Feature Discussion

As pointed out in the feature suggestion issue, seamless rate limitation is important for the usability of the bot. I think it's a good idea to dedicate an issue to this feature in order to discuss an approach for implemetnation.
The rate limitation threshold depends on the Bot's user status in the channel:

20 per 30 seconds, if not mod / operator
100 per 30 seconds if mod / operator
50 per 30 seconds for known bots
7500 per 30 seconds for known bots

Source
This requires to introduce a new data structure that fulfills the following tasks:

keep track of messages sent in a particular channel
if rate limit is reached, buffer messages in a FIFO queue and send them whenever a message slot becomes available again
keep track of the Bot's user level
keep track of a reference to the say-function so it can decide on its own when to send buffered messages

A useful data structure for implementation is a circular buffer.
In the future this data structure can also be used to save channel-specific information like current users.

Standalone working sample implementation:
https://gist.github.com/anXieTyPB/66622eae5c0da2a37869bfbed055a5d2


how can i integrate squoosh to a blazor webassembly api

hello friends I hope not to disturb, I would like to know if there is any way to integrate the Squoosh API in my Blazor WebAssembly API.
I appreciate any help you could give me.
thank you very much

Unfortunately, due to a few people leaving the team, and staffing issues resulting from the current economic climate (ugh), the CLI and libsquoosh packages are no longer actively maintained. I know that sucks, but there simply isn't the time & people to work on this. If anyone from the community wants to fork it, you have my blessing. The squoosh.app web app will continue to be supported and improved.


Implement noex::InstanceAdmin:AsyncGetIamPolicy.

Model this as a typical async unary RPC call: add a CompletionQueue and a Functor to the input parameters, return void, and the Functor takes the return parameter and a grpc::Status as inputs.
The first step is to propose the API in the bug, once approved, you must:

[ ] Implement this new function.
[ ] Implement an integration test for this new function.

Here I am thinking, API should be like
template
void AsyncGetIamPolicy(
std::string const& instance_id,
CompletionQueue& cq,
Functor&& callback)
typename std::enable_if<
google:cloud:internal::is_invocable<Functor, CompletionQueue&, google::cloud::IamPolicy&,
grpc::Status&>::value,int>::type valid_callback_type = 0>

That sounds good, please proceed.


full functionality (still reloading the banner)

@awennersteen have a look at the code.
I used the following package: https://hugocodex.org/add-ons/cookie-consent/

@awennersteen weird, I still see the widget when I navigate to another page


State that it's the MIT

Just thought that you should state the license here, rather than just the text. Easiest would just be to point folks to the License file, but doesn't make sense to just include the text of the license, under the name.

Thank you for pointing out this omission. The license statements were added to this file and all Node.js code files per instructions from the new copyright holder upon transfer of the copyright. I shall ask the copyright holder whether the omission of “MIT License” everywhere except in the license file is required. If not, then I agree with you that it is helpful to include it, and not only in the README file. So I’m leaving this PR open until I get a response.

I have obtained approval to insert “MIT License” as you propose. There are 175 files with the license text in them, and I have added “MIT License” to them all, but I prefer to merge your PR in order to get the “## License” into the README.md file. Are you willing to complete the contribution license agreement, so I can do this? It is at:
https://forms.office.com/pages/responsepage.aspx?id=uGG7-v46dU65NKR_eCuM1xbiih2MIwxBuRvO0D_wqVFUQ1k0OE5SVVJWWkY4MTVJMkY3Sk9GM1FHRC4u


MS Edge Promise performance extremely slow

Although I've read this somewhere before, it's surprising actually how slow it is. :smile:
I've tried both of our bridges and I guess we should use the bluebird library in the catalog app to compensate for that.

Much better!


MudDataGrid: Fix Filter Panel Positioning with FixedHeader = true

Description

Filter Panel when DataGridFilterMode.Simple is placed in  which scrolls normally even under Sticky Header conditions. I was not able to relocate the components to any other part of the table without ill side effects and none that rendered the panel where it needed to be. Added js exception handling in mudPopover.js to relocate any filters-panel to the first headercell in a table when opened.
Resolves #10282
How Has This Been Tested?

Visual Tests, new component to test with as well as going through entire doc page for DataGrid testing.

Type of Changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

https://github.com/MudBlazor/MudBlazor/blob/fdf2f459d8d3b78c6bb43d3fd716c4ea46d49b4e/src/MudBlazor/TScripts/mudPopover.js#L346
The only way to fix this would be to add an exception case in mudPopover.js. If there are no other good alternatives, I probably wouldn’t fix it at all. I don’t think it's ideal to apply patches to individual components in popover. I made an exception for MudList (https://github.com/MudBlazor/MudBlazor/pull/10216) because the popover menu with list is a logical part of the popover, but the filter panel is not.
@henon @danielchalmers, thoughts?

https://github.com/MudBlazor/MudBlazor/blob/fdf2f459d8d3b78c6bb43d3fd716c4ea46d49b4e/src/MudBlazor/TScripts/mudPopover.js#L346
The only way to fix this would be to add an exception case in mudPopover.js? If there are no other good alternatives, I probably wouldn’t fix it at all. I don’t think it's ideal to apply patches to individual components in popover. I made an exception for MudList (#10216) because the popover menu with list is a logical part of the popover, but the filter panel is not.
@henon @danielchalmers, thoughts?

I may have found a razor solution. Working on it now.

I may have found a razor solution. Working on it now.

Now it looks good to me

I may have found a razor solution. Working on it now.

Now it looks good to me

It does push down the first row when you open the filter menu now. It's not bad but it is there. I can't seem to remove it. And in it's new position it does it for all.

Shouldn't shift rows

I will look again, not sure I can keep it from shifting rows. The existing rows are solid, but also dynamic so I couldn't add it there. It needs a parent to relate to so I added the row. When scrolling it takes the row to the "top" of the grid which moves the popover to the top of the grid. I added a top: 58px to push it down (which is pushing the row). I'll work on it some more this weekend.

Shouldn't shift rows

it was as simple as setting visibility: collapse instead of hidden. I'll take it. Checked all datagrids (with filter simple) in datagrid examples as well as the test subject.

Thanks @versile2

Thank you @versile2 !

Hi @versile2, thanks for this work! I was looking out for a solution to this bug for a while.
When I tried it out, the behavior is a little different than expected. When scrolled down, I expect the filter pop-up to show below the header like it does when scrolled up and like your screenshots show. Instead, it covers the header. Is this something that can be fixed?

I don't believe this PR is in Version 7, in V8 it works perfectly. You can try it out and see if your issue is fixed at
https://try.arctechonline.tech
Note the try I host does not save snippets so you will have to copy/paste code.

I should have mentioned I am using V8. The code snippet I'm testing with is from the test component in your PR:
src/MudBlazor.UnitTests.Viewer/TestComponents/DataGrid/DataGridFixedHeaderFilterTest.razor
Interesting, I see it does work correctly in the playground, but the exact snippet in my project works differently. I'm on .NET 8 though. I'll see if I can find if it's another library or something interfering with the css.

I should have mentioned I am using V8. The code snippet I'm testing with is from the test component in your PR: src/MudBlazor.UnitTests.Viewer/TestComponents/DataGrid/DataGridFixedHeaderFilterTest.razor
Interesting, I see it does work correctly in the playground, but the exact snippet in my project works differently. I'm on .NET 8 though. I'll see if I can find if it's another library or something interfering with the css.

Make sure to refresh css files and/or try in "private or incognito" mode to force reload of css. I've had that issue myself with this version and so many style changes.

Also that's preview 6 or 7 I'm hosting (I don't remember which block I copied that from but the datetime stamp is correct)

Oops, turns out it was some custom css I added a while back and forgot about. After getting rid of that it works perfectly. Thanks again for the fix!

Oops, turns out it was some custom css I added a while back and forgot about. After getting rid of that it works perfectly. Thanks again for the fix!

I suspect there might be some of that with V8, so many of the popover issues will be fixed. Thanks for the feedback, hopefully it's all good but don't hesitate if it's not.

One small thing I noticed is that it doesn't take into account Dense="true". The shorter row height causes a gap between the header and filter popover.


sof: framework is redefnining MAX, MIN to version with limited capabilities

Describe the bug
SOF is redefining MAX, MIN macros to macros with single evaluation of argument. However, such macros cannot be used in certain scenarios, e.g. in BUILD_ASSERT. Include with redefinition is used by the wrapper modules/audio/sof/zephyr/wrapper.c which is using zephyr code base. If anywhere in this file MAX is used in a way not supported by redefined macro compilation fails.
To Reproduce

Compile samples/subsys/audio/sof (e.g. intel_adsp_cavs25 board)
Add BUILD_ASSERT(MAX(1, 0), "Test build assert"); somewhere in modules/audio/sof/zephyr/wrapper.c
Compile again and observe that compilation fails.
Add following code above and observe that compilation is successful again

#undef MAX
#define MAX(a, b) (((a) > (b)) ? (a) : (b))

Expected behavior
Code should compile, any macro redefinition within zephyr code space should provide equivalent functionality.
Impact
Sample fail to compile.
Logs and console output
/home/krch/work/modules/audio/sof/zephyr/../src/include/sof/math/numbers.h:31:19: error: braced-group within expression allowed only inside a function
   31 | #define MAX(a, b) ({  \
      |                   ^
/home/krch/work/modules/audio/sof/zephyr/wrapper.c:34:14: note: in expansion of macro 'MAX'
   34 | BUILD_ASSERT(MAX(1, 0), "Test build assert");
      |              ^~~
/home/krch/work/modules/audio/sof/zephyr/../src/include/sof/math/numbers.h:31:19: error: expression in static assertion is not an integer
   31 | #define MAX(a, b) ({  \
      |                   ^
/home/krch/work/modules/audio/sof/zephyr/wrapper.c:34:14: note: in expansion of macro 'MAX'
   34 | BUILD_ASSERT(MAX(1, 0), "Test build assert");

Environment (please complete the following information):

Commit SHA 911ba334a0

Additional context
It is a blocker for #31535 since CI fails on that sample.

@lgirdwood  FYI

@marc-hb can you take a look at this, thanks.

Any news there? I currently used following workaround: https://github.com/nordic-krch/sof/commit/2f8ef204454d278f0c189e8c7cc70e84f745561a

@marc-hb any update ? Fwiw @lyakh and @andyross spotted a few others and we need to align SOF to use the Zephyr native version of all common and all RTOS macros. We can have a temporary sof-rtos.h header that can home the SOF versions of these macros until xtos is removed (this new header should also fail to build is ZEPHYR is defined).

Join the party in https://github.com/thesofproject/sof/pull/4002 if you think you can help.

Join the party in thesofproject/sof#4002 if you think you can help.

That Z_MAX attempt did not fly.
Sorry for the delay, there was some debate on the SOF changes required and then I got caught into something else.
I see you pointed zephyr/west.yml at your workaround in the mean time: https://github.com/nordic-krch/sof/commit/2f8ef204454d278f0c18
The final SOF decision is to drop the fancy, run-time gcc statement-expressions and align SOF with the basic, compile-time, unsafe and usual MIN and MAX macros used by Zephyr, NOT with the fancier Z_MAX macros. I just submitted that at https://github.com/thesofproject/sof/pull/4119


Seq.range is causing out of memory

Description
This is a valid code ['1'..'9'] to generate a char list. However, it the Fable-Core's Seq.range function does not handle it correctly. Seq.range only handle the case the first and last are numbers.
Expected behavior
Expect the code to generate a char list from '1' to '9'.
Actual behavior
The build is passing, the webpack works fine. Finally, IE gives me out-of-memory error, Chrome hangs for seconds and finally be killed - because the steps continuously going up but never touch the last element - it is a char (a string actually after convert to JavaScript).
Known workarounds
For 1 to 9 digit char, it is fine to write
[1..9] |> List.map char // CAUTE: this ONLY works in compiled JavaScript, not in F#

However, for a to z character, and for code works both in F# and JavaScript, I am writing a full list:
let charsA2Z = ['a'; 'b'; 'c'; 'd'; 'e'; 'f'; 'g'; 'h'; 'i'; 'j'; 'k'; 'l'; 'm'; 'n'; 'o'; 'p'; 'q'; 'r'; 's'; 't'; 'u'; 'v'; 'w'; 'x'; 'y'; 'z']

Related: https://github.com/fsprojects/Fable/issues/106

@lijunle Please check if #107 solves it for you.
Note that it requires a change into fable-core.js.

@fdcastel thanks for handling this. I find that not only integer and char can be seq. All type class implementing IEnumable<T> can be pass into seq.
So, it could be better to throw an exception for the unsupported type and maybe support then later.

Yes. Any IEnumerable can be used as a sequence in F#. But I thought you were having problems only with the range constructors. The fix was essentially to solve that.
Could you share what are you trying to accomplish? Maybe we can find a better way :)

Yes, I'm only using char here. But I spent a lot of time to figure out which piece of code of causing the problem. In case someone use another enumerable type in seq, he will spend another couples of hours to find out which piece of code is causing problem. That is why the error message for unsupported type is important and useful.

I completely agree that the error messages should be as informative as possible and we're trying to do that. Unfortunately there're still some blind spots which are difficult to preview in advance. @fdcastel's commit now allows to create ranges from chars (thanks for that!) and it's published in versions fable-compiler 0.2.3 & fable-core 0.0.12 (remember to update both). Please give it a try and feel free to close the issue if it's solved.
By other enumerables I guess you mean it's possible to generate ranges from custom types other than numeric and char values. In theory it should be possible but I couldn't find the exact requirements for that and I'm not sure there'll be much practical value for that. If this is not exactly what you meant, could you please provide a code example and the error message you would expect?
Thanks a lot for your help!

This is fixed. Thanks very much! I am closing the bug.

In regards to enumerable seq, I don't have real requirement based on that. I understand that, sometime it is hard to change something, sometime it is not worth to implement a feature.
My point is, it is very hard and very wasting time to debug a bug causing page hanging and infinite loop. If it cannot be detected from build time, a runtime exception is fine. It won't cause full page hanging and long-time debugging.
Next guy using enumerable seq gets the exception and google will leads him here. Otherwise, he will spent a full night to debug and find out the cause is the Seq.range.


Fix owlbot check

Identify if the owlbot postprocessor check is needed, if it is ensure owlbot is run automatically without manual label addition needed.

owlbot check has been removed.


lutron integration can lose connection

the connection to the lutron bridge can be lost and there is no error message and it does not recover.

Moved to lutron repo: https://github.com/parrot-ha/lutron-integration/issues/2


Could not determine the dependencies of task ':app:processDebugResources'.

I just installed flutter in my new laptop and ran an app which got stuck at running Running Gradle task 'assembleDebug' i then did gradlew clean build which after installing "Install Android SDK Platform 29 (revision: 5)" finished. it threw an error saying

FAILURE: Build failed with an exception.
What went wrong:
Could not determine the dependencies of task ':app:processDebugResources'.
Could not resolve all task dependencies for configuration ':app:debugRuntimeClasspath'.
Could not resolve androidx.core:core:1.1.0.
Required by:
project :app > io.flutter:flutter_embedding_debug:1.0.0-ae90085a8437c0ae94d6b5ad2741739ebc742cb4 > androidx.fragment:fragment:1.1.0
Could not resolve androidx.core:core:1.1.0.
Could not get resource 'https://dl.google.com/dl/android/maven2/androidx/core/core/1.1.0/core-1.1.0.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/androidx/core/core/1.1.0/core-1.1.0.pom'.
dl.google.com
Could not resolve androidx.collection:collection:1.1.0.
Required by:
project :app > io.flutter:flutter_embedding_debug:1.0.0-ae90085a8437c0ae94d6b5ad2741739ebc742cb4 > androidx.fragment:fragment:1.1.0
Could not resolve androidx.collection:collection:1.1.0.
Could not get resource 'https://dl.google.com/dl/android/maven2/androidx/collection/collection/1.1.0/collection-1.1.0.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/androidx/collection/collection/1.1.0/collection-1.1.0.pom'.
dl.google.com
Could not resolve androidx.viewpager:viewpager:1.0.0.
Required by:
project :app > io.flutter:flutter_embedding_debug:1.0.0-ae90085a8437c0ae94d6b5ad2741739ebc742cb4 > androidx.fragment:fragment:1.1.0
Could not resolve androidx.viewpager:viewpager:1.0.0.
Could not get resource 'https://dl.google.com/dl/android/maven2/androidx/viewpager/viewpager/1.0.0/viewpager-1.0.0.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/androidx/viewpager/viewpager/1.0.0/viewpager-1.0.0.pom'.
dl.google.com
Could not resolve androidx.loader:loader:1.0.0.
Required by:
project :app > io.flutter:flutter_embedding_debug:1.0.0-ae90085a8437c0ae94d6b5ad2741739ebc742cb4 > androidx.fragment:fragment:1.1.0
Could not resolve androidx.loader:loader:1.0.0.
Could not get resource 'https://dl.google.com/dl/android/maven2/androidx/loader/loader/1.0.0/loader-1.0.0.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/androidx/loader/loader/1.0.0/loader-1.0.0.pom'.
dl.google.com
Could not resolve androidx.activity:activity:1.0.0.
Required by:
project :app > io.flutter:flutter_embedding_debug:1.0.0-ae90085a8437c0ae94d6b5ad2741739ebc742cb4 > androidx.fragment:fragment:1.1.0
Could not resolve androidx.activity:activity:1.0.0.
Could not get resource 'https://dl.google.com/dl/android/maven2/androidx/activity/activity/1.0.0/activity-1.0.0.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/androidx/activity/activity/1.0.0/activity-1.0.0.pom'.
dl.google.com
Could not resolve androidx.lifecycle:lifecycle-viewmodel:2.0.0.
Required by:
project :app > io.flutter:flutter_embedding_debug:1.0.0-ae90085a8437c0ae94d6b5ad2741739ebc742cb4 > androidx.fragment:fragment:1.1.0
Could not resolve androidx.lifecycle:lifecycle-viewmodel:2.0.0.
Could not get resource 'https://dl.google.com/dl/android/maven2/androidx/lifecycle/lifecycle-viewmodel/2.0.0/lifecycle-viewmodel-2.0.0.pom'.
Could not GET 'https://dl.google.com/dl/android/maven2/androidx/lifecycle/lifecycle-viewmodel/2.0.0/lifecycle-viewmodel-2.0.0.pom'.
dl.google.com
Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
Get more help at https://help.gradle.org
BUILD FAILED in 13m 54s
Exception: Gradle task assembleDebug failed with exit code 1

flutter doctor -v

D:\flutter\Project\ohms>flutter doctor -v
[√] Flutter (Channel stable, 1.22.5, on Microsoft Windows [Version 6.3.9600], locale en-GB)
• Flutter version 1.22.5 at D:\flutter\Sdk\flutter
• Framework revision<PHONE_NUMBER> (2 weeks ago), 2020-12-10 11:54:40 -0800
• Engine revision ae90085a84
• Dart version 2.10.4
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at C:\Users\bright\AppData\Local\Android\sdk
• Platform android-30, build-tools 30.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
• All Android licenses accepted.
[√] Android Studio (version 3.5)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 44.0.1
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
[√] VS Code (version 1.52.1)
• VS Code at C:\Users\bright\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.18.0
[√] Connected device (1 available)
• Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)
• No issues found!

please how do i go about this. Is there something am supposed to do

Hi @BrightCode1
Looks like the tool was not able to get the required dependencies to build the app. Please make sure you have a stable internet connection and try to run the same command again.
If the issue persists, please provide your flutter run --verbose and your pubspec.yaml.
Thank you


Latest main fails with previosly working keymap

Using the main version of the tool locally and on github to try out the solution for #116, I get the following error on the parse step for my keymap that was working before the treesitter refactoring:
2024-11-19T16:34:53.3708818Z INFO: using config args: -c draw-config/config.yaml
2024-11-19T16:34:53.3723324Z INFO: drawing for glove80
2024-11-19T16:34:53.3729772Z INFO:   got extra parse args:
2024-11-19T16:34:53.3733302Z INFO:   got extra draw args:
2024-11-19T16:34:53.7109801Z Traceback (most recent call last):
2024-11-19T16:34:53.7111152Z   File "/opt/pipx_bin/keymap", line 8, in <module>
2024-11-19T16:34:53.7112077Z     sys.exit(main())
2024-11-19T16:34:53.7113987Z   File "/opt/pipx/venvs/keymap-drawer/lib/python3.10/site-packages/keymap_drawer/__main__.py", line 236, in main
2024-11-19T16:34:53.7115452Z     parse(args, config)
2024-11-19T16:34:53.7116954Z   File "/opt/pipx/venvs/keymap-drawer/lib/python3.10/site-packages/keymap_drawer/__main__.py", line 83, in parse
2024-11-19T16:34:53.7118089Z     ).parse(args.zmk_keymap)
2024-11-19T16:34:53.7118898Z   File "/opt/pipx/venvs/keymap-drawer/lib/python3.10/site-packages/keymap_drawer/parse/parse.py", line 167, in parse
2024-11-19T16:34:53.7119878Z     layout, keymap_data = self._parse(in_buf.read(), in_buf.name)
2024-11-19T16:34:53.7120842Z   File "/opt/pipx/venvs/keymap-drawer/lib/python3.10/site-packages/keymap_drawer/parse/zmk.py", line 268, in _parse
2024-11-19T16:34:53.7121596Z     layers = self._get_layers(dts)
2024-11-19T16:34:53.7122546Z   File "/opt/pipx/venvs/keymap-drawer/lib/python3.10/site-packages/keymap_drawer/parse/zmk.py", line 166, in _get_layers
2024-11-19T16:34:53.7123877Z     raise ParseError('Could not find any keymap nodes with "zmk,keymap" compatible property')
2024-11-19T16:34:53.7124842Z keymap_drawer.parse.parse.ParseError: Could not find any keymap nodes with "zmk,keymap" compatible property

(extract from https://github.com/magicDGS/glove80-keymap-config/actions/runs/11917519204).

Hintd: using the previous commit before introducing the tree-sitter implementation works: https://github.com/magicDGS/glove80-keymap-config/actions/runs/11917649047 - this means that indeed the main problem is with tree-sitter parsing.

Thanks, I'll add your keymap to your test cases and figure out why it's not working.

I believe a31b4487c510a9ee347bc31bc3e574fde54b10c6 should fix it, can you test also? The undefined preprocessor macros like ZMK_MACRO that come from ZMK threw off tree-sitter, so some of the root nodes weren't detected.

Looks like it is still failing: https://github.com/magicDGS/glove80-keymap-config/actions/runs/11921610231/job/33226104809
Now it doesn't find the zmk_defines.h newly added definition.

Off-topic as it came to my mind when looking at the header file: would this work with modules such ZMK_HELPERS, as I am planning to eventually include it on my keyboard config?

Yeah, I messed up and forgot to add the new file to the manifest. Just pushed a fix, please try again.

Off-topic as it came to my mind when looking at the header file: would this work with modules such ZMK_HELPERS, as I am planning to eventually include it on my keyboard config?

Yes, though you have to configure a bit to get it to read the extra headers: https://github.com/urob/zmk-helpers/blob/main/docs/keymap_drawer.md

Parsing now works (looks like), but the timeout is happening. So I will close this issue as fixed.

For the off-topic question, I was referring if it will work with the overriden ZMK_MACRO from the helpers (and also, as we are here, with the ZMK_LAYAOUT that I have tried with the released version and it was not detecting any layer....)


add new table for launch data

here is an example of the new table for launches:
{
"id": 1,
"sort_date": "12343234"
"name": "starlink 1-2",
"launch_site": "Cape Canaveral",
"provider": "SpaceX",
"vehicle": "falcon 9",
"date_str": "Mar 9, 2020",
"quicktext": "H-3 - ALOS-3 - Mar 07 (estimated) - https://rocketlaunch.live/launch/alos-3 for info/stream"
}

Also combine any site that is in Florida.  so launchIndex will have a site called "Cape Canaveral / Kennedy Space Center" that holds all the Florida launches.

for the above object, also add the "result" key. keep the number.


Add unittests for SplitString

This adds some unit test cases.

Thank you.
On Wed, Jan 25, 2017 at 6:27 AM, Ivan Naranjo<EMAIL_ADDRESS>wrote:

@ivannaranjo approved this pull request.
Thank you for adding this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-visualstudio/pull/376#pullrequestreview-18408672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVxZpOubLMSoIuu4LVzpeY4Te8ztrcKrks5rV1vggaJpZM4LtLEM
.


vulkan build failed

gen_vulkan_shaders failed at
    Command::new(vulkan_shaders_gen_bin)
        .args([
            "--glslc".as_ref(), "glslc".as_ref(),
            "--input-dir".as_ref(), vulkan_shaders_src.as_os_str(),
            "--output-dir".as_ref(), out_path.as_ref().join("vulkan-shaders.spv").as_os_str(),
            "--target-hpp".as_ref(), header.as_os_str(),
            "--target-cpp".as_ref(), source.as_os_str(),
            "--no-clean".as_ref()
        ])
        .output().expect("Could not run Vulkan shader generator");

error happend at
cxx.to_command()
        .args([
            vulkan_shaders_src.join("vulkan-shaders-gen.cpp").as_os_str(),
            "-o".as_ref(), vulkan_shaders_gen_bin.as_os_str()
        ])
        .output().expect("Could not compile Vulkan shader generator");

this output error:
cc1plus.exe: fatal error: ./llama.cpp\ggml\src\vulkan-shaders\vulkan-shaders-gen.cpp: No such file or directory

dir ./llama.cpp\ggml\src\vulkan-shaders not exists
dir ./llama.cpp\ggml\src\ggml-cuda exists

[llama-cpp-rs/llama-cpp-sys-2
/Cargo.tom](https://github.com/utilityai/llama-cpp-rs/blob/8c1430d82e1df0e1cff25f1fd822f3d723592685/llama-cpp-sys-2/Cargo.toml#L10)
missing

a PR would be welcome! Vulkan is supported almost entirely by not me. If you wan to ensure it stays not broken as llama-cpp updates, feel free to add a github workflow to test it.

Why the project uses cxx to compile llama.cpp? this way you need to manually handle a lot of things and maintain them. for instance
https://github.com/utilityai/llama-cpp-rs/blob/main/llama-cpp-sys-2/build.rs#L654
This is handled by cmake fluently. is there any reason this project not uses cmake?
When compiling llama.cpp with vulkan there's no need to compile manually the shaders. all is needed is just to enable the flag

there was an attempt which was decided against in https://github.com/utilityai/llama-cpp-rs/pull/221

there was an attempt which was decided against in #221. If I recall, I could not get static linking working which made building docker images (which is how we deployed this) tricky.
If someone is willing to bring that home, I'm fine accepting a PR. We (Dial AI) do not currently use this in our inference solution, so it's no longer a hard requirement.

I'll try to create new PR with cmake.
Do you have speical requirement in the building? eg. some non standard compile with llama.cpp / hack?
Can we remove the build with docker images and simplify the tests with matrix on (macos arm, macos intel, windows x86, linux x86) inference of usage example?

Cuda on Linux x86 is the only hard requirement (I currently test this manually as it's impossibly slow on the CI runners + #398)
There's a best effort to maintain CPU on Linux, everything else I'm unable to test outside of CI, so if you can get something clean working CI-wise, that's great!


Cannot find module './api.js'

Hello, I updated the project but meet the error  "Cannot find module './api.js'". Then I try to reinstall the tape-promise package but it doesn't work. How could I fix this problem? Tks~
Btw, I think that create a new branch for developing is better.
Here is the details of the error:
# #######Caliper Test######
not ok 1 Error: Cannot find module './api.js'
  ---
    operator: error
    expected: |-
      undefined
    actual: |-
      { [Error: Cannot find module './api.js'] code: 'MODULE_NOT_FOUND' }
    at: Test.onetime [as end] (/home/hatuw/Documents/deployBC/hyperledger/caliper/node_modules/tape-promise/node_modules/onetime/index.js:22:12)
    stack: |-
      Error: Cannot find module './api.js'
          at Function.Module._resolveFilename (internal/modules/cjs/loader.js:548:15)
          at Function.Module._load (internal/modules/cjs/loader.js:475:25)
          at Module.require (internal/modules/cjs/loader.js:598:17)
          at require (internal/modules/cjs/helpers.js:11:18)
          at Object.<anonymous> (/home/hatuw/Documents/deployBC/hyperledger/caliper/node_modules/fabric-ca-client/lib/FabricCAClientImpl.js:19:11)
          at Module._compile (internal/modules/cjs/loader.js:654:30)
          at Object.Module._extensions..js (internal/modules/cjs/loader.js:665:10)
          at Module.load (internal/modules/cjs/loader.js:566:32)
          at tryModuleLoad (internal/modules/cjs/loader.js:506:12)
          at Function.Module._load (internal/modules/cjs/loader.js:498:3)
  ...

1..1
# tests 1
# pass  0
# fail  1

or else you can Reinstall node report module inside the composer-cli

Change directory into the composer-cli:

cd /usr/lib/node_modules/composer-cli

Reinstall node report  locally inside the composer-cli:

sudo npm install node-report --unsafe-perm

Note : Don't put -g param in sudo npm install. Make the installation local to the composer-cli itself.
The node-report will run make with g++ to compile something (which for some reason was not run during composer-cli installation) and it just works...

or else you can Reinstall node report module inside the tape-promise
1.Change directory into the tape-promise:
cd /usr/lib/node_modules/tape-promise

2.Reinstall node report locally inside the tape-promise:
sudo npm install node-report --unsafe-perm

Note : Don't put -g param in sudo npm install. Make the installation local to the tape-promise itself.
The node-report will run make with g++ to compile something (which for some reason was not run during tape-promise installation) and it just works...


Allow init container to set kernel core pattern

Fixes https://github.com/istio/manager/issues/364

This change is

This is the only way that (I know) it works. I am open to suggestions. While this is still in development, it's a compromise we can live with. I will remove capabilities.

Consider adding a start script to debug container that sets the core pattern instead.

Nothing will work unless is privileged.

This is done through injection script.


some transaction amounts are truncated

https://live.arkscan.io/addresses/AZm9LxRGH2RugWbCPk4t38zsAu5gzhmmns

the transaction amount of the first two listed transactions on this wallet is 0.00000001 each, yet the first amount is truncated and displayed as 0.00, with the amounts in the respective tooltips differing in precision as well.

the reason that one shows as 0.00 is because it's sent to the same address ("sent to self"). Currently the UI does not distinguish that from a regular outgoing transaction, which is confusing. We'll improve that in a future release


Load Balancer Inbound NAT Rule -Decompiling ARM template JSON to Bicep

Load Balancer Inbound NAT Rule -Decompiling ARM template JSON to Bicep.
The decompiler added module "location" which is not valid for load balancer inbound NAT rule.

Can you provide the code sample? The screenshot is covering some important info.
This means that the location property was in the source ARM template that you decompiled with. The decompiler does not strip out properties it does not think are valid because the type definitions could be wrong.

Here it is,
@.***
From: Alex Frankel @.>
Sent: Wednesday, September 8, 2021 12:04
To: Azure/bicep @.>
Cc: Anil Aluru (Intuitive-Cloud) @.>; Author @.>
Subject: Re: [Azure/bicep] Load Balancer Inbound NAT Rule -Decompiling ARM template JSON to Bicep (#4347)
[External Email: Please DO NOT click on any links or open attachments, unless it is from a trusted sender]
Can you provide the code sample? The screenshot is covering some important info.
This means that the location property was in the source ARM template that you decompiled with. The decompiler does not strip out properties it does not think are valid because the type definitions could be wrong.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/bicep/issues/4347#issuecomment-915493925, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVBAZ2CNGAYJRQF4KZFDLLTUA6XQRANCNFSM5DVLYATQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
This message and any attachments are solely for the intended recipient and may contain confidential or privileged information. If you are not the intended recipient, any disclosure, copying, use, or distribution of the information included in this message and any attachments is prohibited. If you have received this communication in error, please notify us by reply e-mail and immediately and permanently delete this message and any attachments. Thank you.

Can you provide the code sample? The screenshot is covering some important info.
This means that the location property was in the source ARM template that you decompiled with. The decompiler does not strip out properties it does not think are valid because the type definitions could be wrong.

Here it is,

Can you provide the code sample? The screenshot is covering some important info.
This means that the location property was in the source ARM template that you decompiled with. The decompiler does not strip out properties it does not think are valid because the type definitions could be wrong.

Understood, I raised the issue to know more and possibily may be imporove it.
Thanks for the explanantion.

Hi @anilaluruintuitivecloud It looks like your template is happy now?
Was your original query a Network Interface or a Load balancer, it is hard to tell.
Sometimes updating the API version to the newer version also helps, since exporting (from the portal or samples) may not have the latest API version.
--
I just see 1 warning and 1 error in your template at the moment.

The warning it appears that you can just hover over the yellow line and it should recommend to swap to string interpolation if you want to fix that one.

With the error, is that loadbalancer inboundNatRules defined in the same template? if it is not, you can just remove it from the dependsOn list.

If you have things working now, let us know and we can resolve the issue for you.

Hey @brwilkinson,
I think it is happy now.
I’ll fix those and see I’m able to deploy it.
Thanks for the help and support.


Extract eslint configuration

Checklist

[X] I've searched through the plugins to make sure this feature isn't already available, or think it shouldn't require a plugin.
[X] I've searched through the current issues to make sure this feature hasn't been requested already.
[X] I agree to follow the Code of Conduct that this project adheres to.

Motivation
Hey there!
I like to spend my evening casually scrolling any repos that are seemed interesting to me.
And throughout my career I always thought that everything has its own goals and reasons.
What am I talking about? I am talking about eslintConfig inside of package.json.
As for me, package.json is a package configuration file, not a linter but a package and there's a clear difference which shouldn't be explained.
Suggested solution
I propose to extract ESLint configuration from package.json to a separate config without changing any of the rules. I want to be assigned to this issue, so that I can create a PR as soon as possible.
By the way, I want to ask, would you like to add any new rules?
Just in case, 'cause I do not know if you have any agreements regarding the code style.
Thanks for the attention!
Alternatives considered
No response

No.

Okay

I do want to clean up the ESLint config over time, but that will come naturally with refactoring as we go.

It's fine :)


Protocol tests are failing while waiting for sessions to be up

Turns out, it's an issue with protocol-engine build for scenarios where 2 cores were present.

Fixed by https://github.com/open-traffic-generator/conformance/commit/4c717d5ad5cd8f62df3fb72ad3ed8d66bd509236


Cannot find a declaration file for module

Hi,
I am using react-d3-tree for my React + TS project. I tried to import the library but it has some weird error and I am not sure if anyone encountered it before.
Are you reporting a bug, or opening a feature request?
A potential bug
What is the actual behavior/output?
The error on import
Could not find a declaration file for module 'react-d3-tree'. '___/node_modules/react-d3-tree/lib/esm/index.js' implicitly has an 'any' type.   There are types at ___/node_modules/react-d3-tree/lib/types/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'react-d3-tree' library may need to update its package.json or typings.ts(7016)
But I can still use the library fine but all the typing is gone
What is the behavior/output you expect?
Should be able to import just fine
What version of react-d3-tree are you using?
3.6.1

Same issue here. For a quick fix, create a declaration file, e.g. react-d3-tree.d.ts, and paste the following:
declare module 'react-d3-tree' {
  export * from 'node_modules/react-d3-tree/lib/types';
}

It's still not solved. I am getting same error.

Fixed via https://github.com/bkrem/react-d3-tree/releases/tag/v3.6.2
Thanks for sharing the workaround here in the meantime @letelete 🙏


Mojave; Chrome 69, Yandex.music

Touch bar media keys not works in Mojave - Chrome 69.0.3497 - Yandex.music

@axejko not sure if it's the same case but make sure that Chrome has all required permission to control your computer via System Preferences -> Security & Privacy -> Accessibility. It fixed same issue with physical media keys on MBP'15

Worked for me too! Thanks @pavel-slepenkov 👍


HTML code in variable

Hi people,
I have a problem when I want replace a value containing html code in a variable of SparkPost.
Example code in C#:
string variable = "Hello<>World";
...
transmission.SubstitutionData["myVariable"] = variable;
Example code in HTML (email):
{{ myVariable }}
Result:
Hello<>World
But I want this result:
Hello
World
Anyone know a solution?
Thanks!!!!

Try
{{{ myVariable }}}

I believe that's the way to show the raw value.

That's right, use triple-curlies if you do not want the value to be HTML-escaped. Double-curlies do automatic HTML escaping. There is some more detail about this in https://developers.sparkpost.com/api/#/introduction/substitutions-reference

Perfect!!!
{{{ myvariable }}} is the solution.
Thanks for the responses

No problem.


[WIP] WEBAPP-516: CI/CD

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword WEBAPP.

somehow the commit history seems messed up. What happend @klinzo ?

@Taggotty I am using @maxammann 's subtree branch as base. Otherwise I would have to include several commands because of git submodules. I'll change the base branch. However, as this is still WIP, no need to review already.

Moved to https://github.com/Integreat/integreat-webapp/pull/332


Error in Google Colaboratory

Hey!
First, thank you for the wonderful project. I was trying out your notebook on segregation in Colab.
The example notebook https://agentpy.readthedocs.io/en/latest/agentpy_segregation.html gives for line
animation = ap.animate(model, fig, ax, animation_plot)

The error
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
/var/folders/hy/zyr6jvld6rscmbnqml0fvq1c0000gn/T/ipykernel_45243/1434696550.py in <module>
      7 fig, ax = plt.subplots()
      8 model = SegregationModel(parameters)
----> 9 animation = ap.animate(model, fig, ax, animation_plot)
     10 IPython.display.HTML(animation.to_jshtml())

~/opt/anaconda3/envs/ABM/lib/python3.10/site-packages/agentpy/visualization.py in animate(model, fig, axs, plot, steps, seed, skip, fargs, **kwargs)
     83     save_count = 10000 if model._steps is np.nan else model._steps
     84
---> 85     ani = matplotlib.animation.FuncAnimation(
     86         fig, update,
     87         frames=frames,

~/opt/anaconda3/envs/ABM/lib/python3.10/site-packages/matplotlib/_api/__init__.py in __getattr__(name)
    220         if name in props:
    221             return props[name].__get__(instance)
--> 222         raise AttributeError(
    223             f"module {cls.__module__!r} has no attribute {name!r}")
    224

AttributeError: module 'matplotlib' has no attribute 'animation'

The package was installed with
!pip install agentpy

Thanks for the report! I need to change the import statement

Fixed with d3179979f07a01df569586664f13883633ad5dca


feat: add usePluginHook to demo app

Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
[ ] Did you follow the contribution guidelines?

PR-Codex overview
This PR focuses on updating the code to use the IMSCA interface instead of MSCA in the useAlchemyProvider and usePlugin hooks.
Detailed summary

Replaced MSCA with IMSCA in useAlchemyProvider hook.
Added import statements for IMSCA, Plugin, AlchemyProvider, and UserOperationOverrides in the usePlugin hook.
Added a check for IMSCA connection in the usePlugin hook.
Updated the installPlugin and uninstallPlugin functions in the usePlugin hook to include the pluginAddress parameter.
Added the isPluginInstalled function in the usePlugin hook to check if the plugin is installed.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

this might need to change based on the plugingen changes in https://github.com/alchemyplatform/aa-sdk/pull/361


test: :white_check_mark: add test to check the hackerrank module

Description
Add test file to check hackerrank module
Resolves: #808
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[x] Have you adhered to the repository's defined coding convention rules?
[x] Have you updated the 'documentation.md' file with the method/function documentation?
[x] Have you sent a message along with the result or response?
[x] Have you used the try-catch technique?
[x] Has the method/class been added to the documentation (md file)?

Screenshot

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Great work @inclinedadarsh !!
Do not forget to

Share your contribution on socials and tag Clueless 🫂
Give this repository a star ✨
Follow Clueless Community on GitHub ✅