Update latex/markdown renderer to drop nteract (use something better)

We're constantly fixing bugs here,
I think we should try to address this by using something else.
Also, add support for offline usage (currently I think we need network connection).

This seems to handle everything https://marketplace.visualstudio.com/items?itemName=goessner.mdmath
Or use markdown it latex extension (create our own renderer, the above extension is already doing that)
Or submit fixes upstream
Or use another

I'm seeing a similar issue, however, for me it appears that the rendered markdown shows the underlying LaTex.

When I waas redirected here I thought there might be a workaround or something. So do we need to be online to be able to render latex in jupyter notebook? I was directed here from this question in stackoverflow here. Hope this issue is solved soon.

When I waas redirected here I thought there might be a workaround or something. So do we need to be online to be able to render latex in jupyter notebook? I was directed here from this question in stackoverflow here. Hope this issue is solved soon.

Not a long term solution, but as a workaround for people working on offline VMs then presumably if the required web address is found then you could put an exception in the firewall in the same way you do for pip.

Not a long term solution, but as a workaround for people working on offline VMs then presumably if the required web address is found then you could put an exception in the firewall in the same way you do for pip.

Notebooks now use builtin KaTeX for math rendering, provided by VS Code. Please file an issue on https://github.com/microsoft/vscode if you encounter bugs in markdown rendering.


Fix the slack-notifier job

What does this PR do?
Fix the slack-notifier job
Motivation
This job is broken ever since this PR got merged. We use a special image in this job which is still with python 3.10 and pip 22, so this option does not exist. Example here.
❯ docker run -it registry.ddbuild.io/slack-notifier:v27936653-9a2a7db-sdm-gbi-jammy@sha256:c9d1145319d1904fa72ea97904a15200d3cb684324723f9e1700bc02cc85065c /bin/bash
root@56105703d3b4:/src# pip install ddqa --break-system-packages

Usage:
  pip install [options] <requirement specifier> [package-index-options] ...
  pip install [options] -r <requirements file> [package-index-options] ...
  pip install [options] [-e] <vcs project url> ...
  pip install [options] [-e] <local project path> ...
  pip install [options] <archive url/path> ...

no such option: --break-system-packages
root@56105703d3b4:/src# pip --version
pip 22.0.2 from /usr/lib/python3/dist-packages/pip (python 3.10)

Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge


no search icon after click in the drop-down box

Prerequisites

[ x] I have attempted to find the simplest possible steps to reproduce the issue

Steps to reproduce the issue

browsing bettween the versions 3.5.3 and 4 I realized there is no search icon after click in the dropdown box, that was removed proposital or is a bug?

css class:
.select2-search input {
    width: 100%;
    height: auto !important;
    min-height: 26px;
    padding: 4px 20px 4px 5px;
    margin: 0;
    outline: 0;
    font-family: sans-serif;
    font-size: 1em;
    border: 1px solid #aaa;
    border-radius: 0;
    -webkit-box-shadow: none;
    box-shadow: none;
    background: #fff url(select2.png) no-repeat 100% -22px;
    background: url(select2.png) no-repeat 100% -22px, -webkit-gradient(linear, left bottom, left top, color-stop(0.85, #fff), color-stop(0.99, #eee));
    background: url(select2.png) no-repeat 100% -22px, -webkit-linear-gradient(center bottom, #fff 85%, #eee 99%);
    background: url(select2.png) no-repeat 100% -22px, -moz-linear-gradient(center bottom, #fff 85%, #eee 99%);
    background: url(select2.png) no-repeat 100% -22px, linear-gradient(to bottom, #fff 85%, #eee 99%) 0 0;
}

Isolating the problem

[ x] The bug happens consistently across all tested browsers

Select2 3.5.3
http://select2.github.io/select2/index.html
image removed:
http://select2.github.io/select2/select2-3.5.3/select2.png
version 4
https://select2.github.io/examples.html

I miss this feature too. The magnifying glass image and the CSS to display it seem to be missing from the v4 themes. My workaround was to lift select2.png and select2x2.png from the v3 files and insert the following CSS into my application:
`
.select2-container--default .select2-search--dropdown .select2-search__field {
background: url(../images/icons/select2.png) right top no-repeat;
background-position: right -22px;
}
@media only screen and (-webkit-min-device-pixel-ratio: 1.5), only screen and (min-resolution: 2dppx)  {
.select2-container--default .select2-search--dropdown .select2-search__field {
background-image: url(../images/select2x2.png);
background-repeat: no-repeat;
background-size: 60px 40px;
background-position: 100% -21px;
}
}
`
I notice that some people have made their own themes that put the icon back. For example, this Yii2 integration:
http://demos.krajee.com/widget-details/select2

OK, I gather from your response then that this feature wasn't deliberately removed in V4, but that it isn't sufficiently important for the core developers to spend any time on this, other than code review.
Shall I open a PR that adds the missing code to _dropdown.scss?
Assuming that no JS needs changing, I assume unit tests wouldn't be applicable.

@rohanc Thanks for feedback! Yeah, please, open a PR with this missing code in CSS (and link this issue there). Then I can merge it into develop branch :+1:


[NOSTR]  Add a notification badge overlaying the chat Tab icon at the bottom of the screen - Bounty #10

This task is to add a notification badge to the Chat Tab icon.
When a message is received from any other user (excluding the support public chat) the user will be alerted by seeing the number of messages in a badge near the top right corner of the chat tab icon. If the same contact messages multiple times, that should still only count as 1 increment on the message badge. So if 3 different contacts have messaged the user, the user should see a badge with the number 3 on it.
BOUNTY available ⚡⚡⚡
for testing the frontend you can connect the staging environment to http://api-development.flashapp.me:4002/graphql

Bounty has been posted!

Hey if you don't mind could you please tell me the skill requirement for it. I want to know this iff I'll be able to do this or not

Hey if you don't mind could you please tell me the skill requirement for it. I want to know this iff I'll be able to do this or not

react-native
typescript
NOSTR experience
chat app development experience

Hi @ecurrencyhodler  @islandbitcoin  is this still up for grab?

closing this issue and rolling the task up to https://github.com/lnflash/flash-mobile/pull/102


Bundler

Add a bundler built into the website and allows user to pick and choose the plugins they need

This is looking great! Some questions...

Can you merge main into the PR? I noticed a few missing plugins.
Does the standard bundle include all plugins?
Should we perhaps build for a version of ECMAScript prior to ES2023 for better compatibility?

As per your Discord comment, ES2023 is supported everywhere 🚀

https://caniuse.com/?feats=mdn-javascript_builtins_array_findlast,mdn-javascript_builtins_array_findlastindex,mdn-javascript_grammar_hashbang_comments,mdn-javascript_builtins_weakmap_symbol_as_keys,mdn-javascript_builtins_array_toreversed,mdn-javascript_builtins_array_tosorted,mdn-javascript_builtins_array_tospliced,mdn-javascript_builtins_array_with


Error in command: reg add "" /v "FriendlyTypeName" /d "" /f

Hi,
I get an error when trying to run the batch file in Windows 7 64-bit:

I'm not sure what this could be. Can you confirm you're running the script as administrator (or with UAC disabled) and that you downloaded the script from https://github.com/rossy/mpv-install/archive/master.zip (if you download from other parts of the GitHub interface, the file may have the wrong line endings.)

I just tested on a vanilla Windows 7 install and I get the same error if I use the script as downloaded from here: https://raw.githubusercontent.com/rossy/mpv-install/master/mpv-install.bat
Instead, it should be downloaded from here: https://github.com/mpv-player/mpv/archive/master.zip
I think this is what happened, so I'm tentatively closing this issue. If this wasn't the problem, please reopen.

@rossy just stumbled to this problem myself and can confirm that downloading the zip solved the issue

@moopie Yeah, I tend to instinctively use GitHub's raw links too. I should document this.


There is a vulnerability in Apache Hadoop 2.10.1,upgrade recommended

https://github.com/apache/inlong/blob/99189bedacca354e5244be251235a32ad8ccf1e2/pom.xml#L147
CVE-2021-37404 CVE-2022-25168 CVE-2022-26612  CVE-2021-33036
Recommended upgrade version：2.10.2

fixed in https://github.com/apache/inlong/pull/5790, close it.


feat(dashboard): add a new panel: views

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Resolves #12951

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Don't know why it's 'Data Source' instead of simply 'Source', so I did a little renaming 🥰 rename 'Data Source' -> 'Source'


After saving the model, when restoring it from the file, I don't get the proper results

Dear developers,
Last week or so I asked you on another thread how to save my model. Today I tried what you instructed:

But then when restoring the model I don't get the same results:

Do you know why that is?

Hellos Cristian,  Can you maybe share this trained model? I saw this work very good on cities, I need some similar to detect urban trees. Thank you.

Can you make the colab fully reproducible, so I can see the model original image, the way you saved and then the reloaded and the final image, all together in one script. It's not possible to guess. There is a test written that confirms save and reload passes
def test_save_and_reload_checkpoint(m, tmpdir):
    img_path = get_data(path="2019_YELL_2_528000_4978000_image_crop2.png")
    m.config["train"]["fast_dev_run"] = True
    m.create_trainer()
    #save the prediction dataframe after training and compare with prediction after reload checkpoint
    m.trainer.fit(m)
    pred_after_train = m.predict_image(path = img_path)
    m.save_model("{}/checkpoint.pl".format(tmpdir))

    #reload the checkpoint to model object
    after = main.deepforest.load_from_checkpoint("{}/checkpoint.pl".format(tmpdir))
    pred_after_reload = after.predict_image(path = img_path)

    assert not pred_after_train.empty
    assert not pred_after_reload.empty
    pd.testing.assert_frame_equal(pred_after_train,pred_after_reload)

def test_save_and_reload_weights(m, tmpdir):
    img_path = get_data(path="2019_YELL_2_528000_4978000_image_crop2.png")
    m.config["train"]["fast_dev_run"] = True
    m.create_trainer()
    #save the prediction dataframe after training and compare with prediction after reload checkpoint
    m.trainer.fit(m)
    pred_after_train = m.predict_image(path = img_path)
    torch.save(m.model.state_dict(),"{}/checkpoint.pt".format(tmpdir))

    #reload the checkpoint to model object
    after = main.deepforest()
    after.model.load_state_dict(torch.load("{}/checkpoint.pt".format(tmpdir)))
    pred_after_reload = after.predict_image(path = img_path)

    assert not pred_after_train.empty
    assert not pred_after_reload.empty
    pd.testing.assert_frame_equal(pred_after_train,pred_after_reload)

What do you need in order to make it fully reproducible? The training data?

For some strange reason now it works pretty well. I don't really know why last time it didn't but probably the mistake was mine. Thank you very much for the help.
Also now that I have kind of finished my project and I think my results are pretty good do you want me to send you the trained model so that you see how it works and if you are also happy with the results we can probably share it for other people in the community that will want to predict trees in urban environments.
Also one more question:
The organization I am working for wants me to write a paper about the projects and the results, so do you mind of I do that. I will of course mention the library, the algorithm and will provide links and references to the documentation and this GitHub repo. Thanks so much in advance!


Attribute Error Leading to Internal Server Error?

So the error logs I am getting in the mail from the SMTP server keeps saying this over and over and I believe it has to do with the internal server error I have been getting whenever I view the site.

AttributeError: 'module' object has no attribute 'plugin'

Does anyone know how to fix this?

Hey there, we'd be glad to help you figure things out on the [#production help](https://chat.zulip.org/#narrow/stream/production help) stream on the development community :)


[spectoh5] Improve handling of links

Improve handling of soft links when converting spec file to hdf5 file. This is to prepare for a unique spech5 + fabioh5 to hdf5 converter (#609).

The latest commit addresses #614 by adding a follow_links (default False) parameter to visit and visititems.

LGTM


Change multi-file approach to use action for each word

e.g. Create foo, update bar and delete bazz

Currently it does

Various changes to baz.txt, bar.js and README.md

This is much easier to do now that #38 is mostly done.
This issue is also very similar to #52, which will handle many files e.g. update foo.txt and delete 6 files
The intention of this issue is to handle just a few files. e.g. update foo.txt and delete bar.js

This currently isn't nice for only 2 files with different actions
build(deps): create 1 file and update 1 file

This should be
build(deps): update package.json and create package-lock.json


Bug Report: Crashing CMD window

Environment
Windows build number: 18362.175
Windows Terminal version (if applicable): Latest Master(as of 8/15/2019)

Any other software?
Just the original Command Prompt
Steps to reproduce
1)Build new terminal,
2) run it without debugging
3) ...
4) Profit
5) Try to run standard command shell
Expected behavior
Should be able to run cmd without issues like normal.
Actual behavior
I noticed a peculiar behavior when trying to run a gulp task in the new terminal.  The drawn portion of the terminal window shrinks to a few lines wide.  When I try to open the regular command window it opens, shrinks in size then closes.

This captures just the brief flash of the command prompt:

Just the original Command Prompt

Run a gulp task ...

There might be a little more to this issue than you're describing ☹

To be honest that's all I did.  I only was trying to run the task.  I don't think it was responsible for the issue, just made it visible.  Basically I try to open the plain command prompt and it crashes.  If I try to open a tab in the new terminal of the command prompt it closes immediately.  When I try to run a command in powershell that executes via the command prompt (i.e. a BAT file) the powershell view shrinks to an extremely skinny window as I've shown above.  I've gotten around my issue for now by running gulp in the WSL bash prompt.  But it still closes.  Hopefully some updates will fix it but for now that's it.
Actually now that I think about it, the one thing I did, but ignored was run the project's command console directly from VS2019, before running the correct project, and it did open and close in the same way as the command prompt is doing lately, just I paid it no mind...

What exactly are you running that's causing mode 20,5 to be called?

This doesn't look like an issue on us. Something in your CMD startup script (autorun, perhaps?) is calling mode 20,5 to set the console window to, well, 20 x 5. Perhaps that's why the console window is becoming 20 x 5.

I'm not sure, there was no startup script before, I'd just like to know how to revert this issue.  Tried uninstalling the new terminal, reinstalling, etc. No dice.

Does it repro for all conhost launches?

[ ] cmd.exe from the start menu
[ ] cmd.exe from the run box
[ ] powershell from the start menu
[ ] powershell from the run box
[ ] wsl/your favorite distro from the start menu
[ ] wsl from the run box

Neither powershell nor wsl crash.  But if I run any command on powershell that triggers a bat file (gulp is my example) it shrinks the output apparently via mode but never executes the commands for that file.   I even downloaded an alternate command console and it also shrinks and closes without much time to react.  Is there any registry settings I could look at to modify default properties or something?
If I run any developer prompts the same thing happens so it seems to be a problem with the root command (cmd.exe)

What registry values do you have in Computer\HKEY_CURRENT_USER\SOFTWARE\Microsoft\Command Processor?
How about Computer\HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Command Processor?

The first registry key had an entry that was trying to start soundmixer.exe and if it didn't find it it would close... looks like that may have been it.  Not sure what that is doing in the autorun entry of the command processor but since it worked before somehow something was removed.  Yup and that fixed it
Thank you so much.


Determine if we need to migrate the April Fools cat blocks

Personally, I think this is an important part of the culture of Scratch which we should preserve.

Everybody loves the cat blocks! If there's a way to preserve them, that'd be great. I kind of want to make them automatically appear as an option for April 1 every year...


encryption-at-rest RFC

Signed-off-by: Yi Wu<EMAIL_ADDRESS>

Original proposal can be find at: https://docs.google.com/document/d/14u0YKHXxjE0a9wAd3tsu0Hqjmknjr0bYU_EMzZSbzkA/edit?usp=sharing
Updates include:

Reorganize the flow of the proposal, to discuss higher level usage first, then the key management details, then specific details about rocksdb and snapshot and BR.
Updated content for KMS usage
Update content for snapshot and BR handling.

Do we support to enable this feature for a cluster already have data in it? Or users should never change this configuration once they have their clusters deployed?
If users want to enable this feature for their existing data, they have to deploy a new cluster with this feature enabled, then migrate data from their old cluster?

It can be enabled for an existing cluster: https://docs.pingcap.com/tidb/dev/encryption-at-rest

Should this be merged?

Should this be merged?

That would make sense since it is implemented.

No bandwidth to get this merged. Closing for now.

Isn't it implemented already in tikv repository?

Yes, this is implemented now.

There should be something wrong with the development cycle that implementation is merged before RFC.

Most development does not have an RFC. I think that we should celebrate that there is one here and try to bring out more rather than feel bad about something.


[tw] Update config.toml

Describe your changes
Solve the problem of updating config.toml in the main branch.
For example:

weight changes from 11 to 12
change of [languages.zh-tw.params]

Related issue number or link (ex: resolves #issue-number)
resolves #2252
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Hi @ydFu :)
Why don't we resolve this issue by rebasing (updating) dev-tw ?
(There will be some conflicts that need to be manually fixed though)

Hi @ydFu :) Why don't we resolve this issue by rebasing (updating) dev-tw ? (There will be some conflicts that need to be manually fixed though)

Thanks for your reminder,Explain the PR change thinking
In this PR, there are parts that include updating main and adding new tw content. I think that small changes to a single file can reduce the code review workload of cncf owners consist of Maintainers.
Update content:

other language config

New tw content:

tw weight changes from 11 to 12
change of [languages.zh-tw.params]

Use direct dev-tw merge/rebase to main, it will be a relatively large project such as this issue(https://github.com/cncf/glossary/issues/2355).
If you have any other suggestions, I can also adjust accordingly.

Hi @ydFu
Thank you for sharing your idea. :)
Yes, there is pros and cons within the given branch configurations in the Glossary repo.
I still think rebasing dev-tw is more beneficial method this time on the perspective of minimize merge conflicts (changing contents for the same file in different branches may cause merge conflicts. and sometimes we can face conflicts that we are not sure)"
Regarding the auto-assigned reviewers when we open a PR to merge branches, please nevermind, I think reviewers understand the circumstances. :)

thank for @seokho-son


Incompatible type for ES6 Proxy set handler using ES6 shorthand syntax

I recently tried to use the ES6 Proxy class to create immutable objects from given 'target' objects, using the following code:
function makeImmutable(target) {
  return new Proxy(target, {
    set(target, property, value, receiver) {
      throw '...';
    }
  });
}

Compiling this with:
tsc --lib es6 test.ts
fails with the following error message:
test.ts(2,28): error TS2345: Argument of type '{ set(target: any, property: PropertyKey, value: any, receiver: any): void; }' is not assignable to parameter of type 'ProxyHandler<any>'.
  Types of property 'set' are incompatible.
    Type '(target: any, property: PropertyKey, value: any, receiver: any) => void' is not assignable to type '(target: any, p: PropertyKey, value: any, receiver: any) => boolean'.
      Type 'void' is not assignable to type 'boolean'.

However, when not using the ES6 "function property" shorthand syntax, and instead doing the following:
function makeImmutable(target) {
  return new Proxy(target, {
    set: function(target, property, value, receiver) {
      throw '...';
    }
  });
}

Everything compiles fine.
Typescript version: 2.1.0-dev.20161007

I guess it is because JS interprets the set as a property setter instead of a shorthand property name and it expects set propName (key, value) instead of set funcname (args). This is a just a tip though.

Got the same

This is a duplicate of https://github.com/Microsoft/TypeScript/issues/16608, you can find more information about why never is not automatically inferred in this case in https://github.com/Microsoft/TypeScript/pull/8767.
Adding an explicit type annotation, be it never, should address the issue.


Questions about loss and model architecture

Dear Authors,
Thank you for your inspiring work. I have some questions regarding of the loss function and model architecture.

MSE loss might produce blurry result. Would it help if I replace it with L1 Loss?

It is believed that vanilla 2D convolution does not suffice to catch spatial-temporal correlation while it seems your model handles this well. I couldn't find detailed explanation in your paper. Could you please explain it?

With kind regrads.

In my practice, replacing MSE with L1 loss is not a good solution because it may be hard to optimize. I have seen some papers using MSE+L1 loss, which seems to work well.
I would like to suggest you refer to our manuscript: SimVP: Towards Simple yet Powerful Spatiotemporal Predictive Learning. The most important component is the Translator. Using vanilla conv as the translator usually fails. IncepU translator focuses on multi-scale feature learning and thus has great ability in long-range modeling. Our new gSTA translator is inspired by large kernel convs, and models long-range interactions in a more efficient way.

I will close this issue if there is no more question. Please feel free to open a new issue when you have new questions.


changelog has html unescaped after 1.2.14 feisty-cryokinesis (2014-03-01)

If you look in changelog after 1.2.14 feisty-cryokinesis (2014-03-01), you will see a bunch of html displayed.

Interesting...

I think we've got another issue for this already... From what I understand it is GitHub processing of md that fails...

Yes, there's another issue. But the question is, what makes the parsing fail?

This should do it, what do you think? https://github.com/angular/angular.js/pull/11778


BXMSPROD-1579 Send build summary as PR comment

https://issues.redhat.com/browse/BXMSPROD-1579
Related PRs:

https://github.com/kiegroup/jenkins-pipeline-shared-libraries/pull/179
https://github.com/kiegroup/kogito-pipelines/pull/340
https://github.com/kiegroup/kogito-images/pull/1043
https://github.com/kiegroup/kogito-operator/pull/1120

Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[ ] You have read the contributors' guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains a link to the JIRA issue
[ ] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've ran make before-pr and everything is working accordingly
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

How to retest this PR or trigger a specific build:

Run operator BDD testing
Please add comment: /jenkins test

Run RHPAM operator BDD testing
Please add comment: /jenkins RHPAM test

Depends on https://github.com/kiegroup/jenkins-pipeline-shared-libraries/pull/179

/jenkins test

/jenkins test

/jenkins RHPAM test

/jenkins test

/jenkins RHPAM test

/jenkins test

/jenkins RHPAM test

PR job #23 was: FAILURE
Possible explanation: Pipeline failure or project build failure
You can find test error logs here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/23//artifact/test/logs/error/
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/23/display/redirect or see console log:
2022-01-24 10:31:51,719 resource.py:155        WARNING No value found for 'name' in '{"dest": "/usr/local/bin", "image": "operator-builder", "path": "/workspace/rhpam-kogito-operator-manager"}' artifact; using auto-generated value of 'rhpam-kogito-operator-manager'
2022-01-24 10:31:51,720 cli.py:346        ERROR '/home/jenkins/agent/workspace/kogito/rhpam-kogito-operator-PR-check-new/content_sets.yaml' file not found!
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/cekit/cli.py", line 339, in run
    command.execute()
  File "/usr/local/lib/python3.6/site-packages/cekit/builder.py", line 53, in execute
    self.before_generate()
  File "/usr/local/lib/python3.6/site-packages/cekit/builder.py", line 92, in before_generate
    self.generator.init()
  File "/usr/local/lib/python3.6/site-packages/cekit/generator/base.py", line 118, in init
    self.image = Image(descriptor, os.path.dirname(os.path.abspath(self._descriptor_path)))
  File "/usr/local/lib/python3.6/site-packages/cekit/descriptor/image.py", line 49, in __init__
    self._prepare()
  File "/usr/local/lib/python3.6/site-packages/cekit/descriptor/image.py", line 60, in _prepare
    self._descriptor['packages'] = Packages(self._descriptor.get('packages', {}), self.path)
  File "/usr/local/lib/python3.6/site-packages/cekit/descriptor/packages.py", line 69, in __init__
    raise CekitError("'%s' file not found!" % content_sets_file)
cekit.errors.CekitError: '/home/jenkins/agent/workspace/kogito/rhpam-kogito-operator-PR-check-new/content_sets.yaml' file not found!
make: *** [Makefile.rhpam:37: container-build] Error 1
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Push Operator Image(s) to Openshift Registry)
Stage "Push Operator Image(s) to Openshift Registry" skipped due to earlier failure(s)
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Run BDD tests)
Stage "Run BDD tests" skipped due to earlier failure(s)
[Pipeline] stage
[Pipeline] { (Running smoke tests)
Stage "Run BDD tests" skipped due to earlier failure(s)
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Declarative: Post Actions)
[Pipeline] script
[Pipeline] {
[Pipeline] }
[Pipeline] // script
[Pipeline] script
[Pipeline] {
[Pipeline] sh
+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/23/api/json
[Pipeline] readJSON
[Pipeline] sh

/jenkins RHPAM test

PR job #24 was: FAILURE
Possible explanation: Pipeline failure or project build failure
You can find test error logs here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/24//artifact/test/logs/error/
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/24/display/redirect or see console log:
2022-01-24 11:09:44,086 resource.py:155        WARNING No value found for 'name' in '{"dest": "/tmp/artifacts/", "path": "modules"}' artifact; using auto-generated value of 'modules'
2022-01-24 11:09:44,097 cli.py:346        ERROR '/home/jenkins/agent/workspace/kogito/rhpam-kogito-operator-PR-check-new/content_sets.yaml' file not found!
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/cekit/cli.py", line 339, in run
    command.execute()
  File "/usr/local/lib/python3.6/site-packages/cekit/builder.py", line 53, in execute
    self.before_generate()
  File "/usr/local/lib/python3.6/site-packages/cekit/builder.py", line 92, in before_generate
    self.generator.init()
  File "/usr/local/lib/python3.6/site-packages/cekit/generator/base.py", line 114, in init
    Image(image_descriptor, os.path.dirname(os.path.abspath(self._descriptor_path))))
  File "/usr/local/lib/python3.6/site-packages/cekit/descriptor/image.py", line 49, in __init__
    self._prepare()
  File "/usr/local/lib/python3.6/site-packages/cekit/descriptor/image.py", line 60, in _prepare
    self._descriptor['packages'] = Packages(self._descriptor.get('packages', {}), self.path)
  File "/usr/local/lib/python3.6/site-packages/cekit/descriptor/packages.py", line 69, in __init__
    raise CekitError("'%s' file not found!" % content_sets_file)
cekit.errors.CekitError: '/home/jenkins/agent/workspace/kogito/rhpam-kogito-operator-PR-check-new/content_sets.yaml' file not found!
make: *** [Makefile.rhpam:37: container-build] Error 1
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Push Operator Image(s) to Openshift Registry)
Stage "Push Operator Image(s) to Openshift Registry" skipped due to earlier failure(s)
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Run BDD tests)
Stage "Run BDD tests" skipped due to earlier failure(s)
[Pipeline] stage
[Pipeline] { (Running smoke tests)
Stage "Run BDD tests" skipped due to earlier failure(s)
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Declarative: Post Actions)
[Pipeline] script
[Pipeline] {
[Pipeline] }
[Pipeline] // script
[Pipeline] script
[Pipeline] {
[Pipeline] sh
+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/24/api/json
[Pipeline] readJSON
[Pipeline] sh

/jenkins RHPAM test

PR job #25 was: FAILURE
Possible explanation: Pipeline failure or project build failure
You can find test error logs here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/25//artifact/test/logs/error/
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/25/display/redirect or see console log:
  File "/usr/local/lib/python3.6/site-packages/cekit/generator/base.py", line 379, in prepare_repositories
    url = self._prepare_content_sets(self.image.get('packages').get('content_sets'))
  File "/usr/local/lib/python3.6/site-packages/cekit/generator/base.py", line 432, in _prepare_content_sets
    raise CekitError("Could not create ODCS compose", ex)
cekit.errors.CekitError: ('Could not create ODCS compose', HTTPError('401 Client Error: Unauthorized for url: https://odcs.fedoraproject.org/api/1/composes/',))
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>401 Unauthorized</title>

Unauthorized

This server could not verify that you
are authorized to access the document
requested.  Either you supplied the wrong
credentials (e.g., bad password), or your
browser doesn't understand how to supply
the credentials required.

make: *** [Makefile.rhpam:37: container-build] Error 1
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Push Operator Image(s) to Openshift Registry)
Stage "Push Operator Image(s) to Openshift Registry" skipped due to earlier failure(s)
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Run BDD tests)
Stage "Run BDD tests" skipped due to earlier failure(s)
[Pipeline] stage
[Pipeline] { (Running smoke tests)
Stage "Run BDD tests" skipped due to earlier failure(s)
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Declarative: Post Actions)
[Pipeline] script
[Pipeline] {
[Pipeline] }
[Pipeline] // script
[Pipeline] script
[Pipeline] {
[Pipeline] sh
+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/25/api/json
[Pipeline] readJSON
[Pipeline] sh

/jenkins RHPAM test

RHPAM PR job #26 was: FAILURE
Possible explanation: Pipeline failure or project build failure
You can find test error logs here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/26//artifact/test/logs/error/
Please look here: https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/26/display/redirect or see console log:
  File "/usr/local/lib/python3.6/site-packages/cekit/generator/base.py", line 379, in prepare_repositories
    url = self._prepare_content_sets(self.image.get('packages').get('content_sets'))
  File "/usr/local/lib/python3.6/site-packages/cekit/generator/base.py", line 432, in _prepare_content_sets
    raise CekitError("Could not create ODCS compose", ex)
cekit.errors.CekitError: ('Could not create ODCS compose', HTTPError('401 Client Error: Unauthorized for url: https://odcs.fedoraproject.org/api/1/composes/',))
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>401 Unauthorized</title>

Unauthorized

This server could not verify that you
are authorized to access the document
requested.  Either you supplied the wrong
credentials (e.g., bad password), or your
browser doesn't understand how to supply
the credentials required.

make: *** [Makefile.rhpam:37: container-build] Error 1
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Push Operator Image(s) to Openshift Registry)
Stage "Push Operator Image(s) to Openshift Registry" skipped due to earlier failure(s)
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Run BDD tests)
Stage "Run BDD tests" skipped due to earlier failure(s)
[Pipeline] stage
[Pipeline] { (Running smoke tests)
Stage "Run BDD tests" skipped due to earlier failure(s)
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Declarative: Post Actions)
[Pipeline] script
[Pipeline] {
[Pipeline] }
[Pipeline] // script
[Pipeline] script
[Pipeline] {
[Pipeline] sh
+ wget --no-check-certificate -qO - https://jenkins-kogito-tools.kogito-cluster-0ad6762cc85bcef5745bb684498c2436-0000.us-south.containers.appdomain.cloud/job/kogito/job/rhpam-kogito-operator-PR-check-new/26/api/json
[Pipeline] readJSON
[Pipeline] sh

failure on RHPAM BDD tests is unrelated to the change. The message is however working fine :)


Recent Activity stretches the screen too much

Describe the bug
The Recent Activity pages stretch out the screen too much. It needs scrolling and a load more button.

Resolved


Procedure to enable communication with ClusterIP using kubefwd

We are using the kubefwd utility to access k8s cluster IP services outside the k8s cluster from a ubuntu VM. The tool is really helpful in providing the access cluster IP services, thankful to you for making such a useful utility.
our objective in using kubefwd utility to enable communication between ubuntu VM ( outside the K8S cluster) and servicers on K8S services , and to run some shell scripts on the K8S services.
The shell scripts firsts configures DB instances (i.e crunchy db postgres operator DB pod instances) on K8S,  then apply the network policy on K8S cluster after the name resolution (the scripts runs dig command  for DB service name resolution ), the network polices required to enable communication between application services with postgres DB services
since the kubefwd tool updating the etc/hosts  with IPs 127.1.27.X , the name resolution of the DB instance resolving to 127.1.27.X , therefore  network policies are configured 127.1.27.X IP range instead of DB service ClusterIP, resulting the application services failing to make proper communication with DB services, to unblock ourselves from this situation as workaround we are updating network policy manually replacing the 127.1.27.x ( which got configured by the scripts after executing the dig command)  with <IP_ADDRESS>
want to check whether kubefwd utility can populate the /etc/hosts with actual cluster IP and enable kubefwd communication with ClusterIP, this helps to populate the network polices exactly with required DB service ClusterIP
Also, we have configured kube-dns as load-balancer service, update dnsmasq.conf file with kube-dns loadbalancer as DNS forwarder to resolve the k8S Services, with this the name resolution with help of dnsmasq happening with ClusterIP ( until we restart the dnsmasq service), but the sql communication to PGO DB services is failing from Ubututu VM .
Once after restarting dnsmasq service, name resolution is happing with 127.1.27.X IP,  and the sql communication is happing dns, but this needs the adjustment of network policies by replacing 127.1.27.x with <IP_ADDRESS>.
Previous to implementing of kubefwd configuration, we have made use the DB instances as LoadBalacer service with Bind9 for name resolution. the kubefwd greatly reduced efforts, but needs to adjust the network policy.
looking for your input on whether kubefwd helps us in using ClusterIP instead of 127.1.27.X IP  to enable the communication with required actual  ClusterIP   rather adjusting manually with <IP_ADDRESS> range
ubuntu@prodagdrgen3adminvm:~$ kubectl get svc -n pgo
NAME                              TYPE          CLUSTER-IP        EXTERNAL-IP     PORT(S)                                           AGE
pig                                     ClusterIP   <IP_ADDRESS>                 2022/TCP,5432/TCP                          78m
pi-backrest-shared-repo   ClusterIP   <IP_ADDRESS>             2022/TCP                                          78m
postgres-operator             ClusterIP   <IP_ADDRESS>             8443/TCP,4171/TCP,4150/TCP          81m
cat /etc/dnsmasq.conf
interface=lo
bind-interfaces
listen-address=<IP_ADDRESS>
server=/cluster.local/<IP_ADDRESS>  --------> kube-dns SVC as loadbalancer
server=<IP_ADDRESS>
nslookup pig.pgo.svc.cluster.local   --------> Host name lookup of the desired DB services
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53
Name:   pig.pgo.svc.cluster.local
Address: <IP_ADDRESS>  --------> Resolving to ClusterIP as expected
ubuntu@prodagdrgen3adminvm:~$ jobs  -------> run the kubefwd in the back-end
[1]+  Running                 sudo kubefwd svc -n pgo &
cat /etc/hosts     -------> /etc/hosts entries after executing  kubefwd
<IP_ADDRESS>       pig-backrest-shared-repo pig-backrest-shared-repo.pgo pig-backrest-shared-repo.pgo.svc pig-backrest-shared-repo.pgo.svc.cluster.local pig-backrest-shared-repo.pgo.default pig-backrest-shared-repo.pgo.svc.default pig-backrest-shared-repo.pgo.svc.cluster.default
<IP_ADDRESS>       pig pig.pgo pig.pgo.svc pig.pgo.svc.cluster.local pig.pgo.default pig.pgo.svc.default pig.pgo.svc.cluster.default
<IP_ADDRESS>       postgres-operator postgres-operator.pgo postgres-operator.pgo.svc postgres-operator.pgo.svc.cluster.local postgres-operator.pgo.default postgres-operator.pgo.svc.default postgres-operator.pgo.svc.cluster.default
$ psql --host=pig.pgo.svc.cluster.local --port=5432  --username="xxxxx" --password --dbname=pig -------> try establish sql connection from Ubuntu VM to DB service with hostname resolved CluserIP
Password for user xxxxxx:
^C            -------> No connection, cancel the command
ubuntu@prodagdrgen3adminvm:~$ ping pig.pgo.svc.cluster.local -------> Ping from Ubuntu VM to DB service with hostname resolved CluserIP
PING pig.pgo.svc.cluster.local (<IP_ADDRESS>) 56(84) bytes of data.
From XXXXXXXX (103.XXX.XXX.1) icmp_seq=1 Destination Host Unreachable
From XXXXXXXX (103.XXX.XXX.1) icmp_seq=1 Destination Host Unreachable
From XXXXXXXX (103.XXX.XXX.1) icmp_seq=1 Destination Host Unreachable
^C
--- pig.pgo.svc.cluster.local ping statistics ---
6 packets transmitted, 0 received, +3 errors, 100% packet loss, time 5093ms
pipe 3
$ sudo systemctl restart dnsmasq   ---------> restart the dnsmasq service
$ nslookup pig.pgo.svc.cluster.local --------> Host name lookup of the desired DB services
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53
Name:   pig.pgo.svc.cluster.local
Address: <IP_ADDRESS> --------> resolving to <IP_ADDRESS> to /etc/host entry
$ ping pig.pgo.svc.cluster.local --------> ng from Ubuntu VM to DB service with hostname resolved to <IP_ADDRESS>
PING pig.pgo.svc.cluster.local (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from fence (<IP_ADDRESS>): icmp_seq=1 ttl=64 time=0.040 ms
ubuntu@prodagdrgen3adminvm:~$ psql --host=pig.pgo.svc.cluster.local --port=5432  --username="xxxxx" --password --dbname=pig
Password for user xxxx:
psql (10.18 (Ubuntu 10.18-0ubuntu<IP_ADDRESS>), server 13.3)
WARNING: psql major version 10, server major version 13.
Some psql features might not work.
Type "help" for help.
pig=#

@nagannv There is a lot here so forgive me if I misunderstand. Kuberfwd can only port-froward to a Pod, as far as I am aware there is no way to port-forward to a ClusterIP.  Kubefwd uses 127. addresses to bind its port-forward to since they are unlikely to conflict with local network services as they are reserved for loopback testing. Kubefwd does not create any port-forward on <IP_ADDRESS> as this represents all loopbacks, therefore there could not be more than one of each port number, So main-db:5432on <IP_ADDRESS> and replica-db:5432 on <IP_ADDRESS> would conflict.

@cjimti
Thank you for your kind response. now I understood the significance of 127.X. since there is no way to use the actual clusterIP for foring and have the clicuterIP in the /etc/hosts, I will continue to use the work-around.

@nagannv This may not be a problem that kubefwd is trying to solve. You want to use host ip to access the service, maybe you should consider iptables SNAT.
ex: iptables -t nat -A POSTOUTING -s <IP_ADDRESS>/24 -j SNAT --to-source (192.168.0.xx you host net ip) -o eth0 (you host net interface)
In this way, it can use the host's IP communication, but this will introduce additional complexity. iptables is a network tool of the linux kernel, with it you can control the network.


CDPS-783: Integrate with Prisoner Search to check prisoner exists

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.58%. Comparing base (7276f65) to head (0d8fd15).

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #14      +/-   ##
============================================
+ Coverage     57.70%   60.58%   +2.88%
- Complexity       82       97      +15
============================================
  Files            22       24       +2
  Lines           383      411      +28
  Branches         38       38
============================================
+ Hits            221      249      +28
  Misses          154      154
  Partials          8        8

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Windows 10 2004 x86 only 5 Connections

RDP is working, but only 5 Connections possible.
With the 6th Connection there is just this message:

Windows 10 Pro 32Bit Version 2004.
Status :

ini from autoupdate.bat
[Main]
Updated=2020-08-14

Did a new Fresh install with Windows 10 Pro 64Bit Version 2004, did initial Setup with autoupdate.bat
64Bit works without Problem, tested with 10 Connections
32Bit only 5 Connections

32bit 1809 windows 10 running fine , no Limit
32bit 1903 and 1909 online 5 connections
I did the Quick check with Stock termsrv.dll
Any ideas what do do ? Anyone facing same issue?

I'm also having the same problem, windows 10 x86 build 2004 and only 5 connections. Were you able to solve this problem?

no solution so far, but another information.
the 6.th Connections creates an Event in Windows protocolls - application
eventid 1001 : Windows Error Reporting

actually i dont know how to read These error reports, i'm working on it...

The lack of free memory - ?

Applicable to https://github.com/stascorp/rdpwrap/issues/1004


Recycle the s3 client.

The S3 client hosts a connection pool.
Right now this connection pool is instantiated at the creation of the Storage which itself happens once per leaf_search.
Establishing connection takes several ms. We should build only one s3 client per configuration.

reopening...
tracing shows 2 to 3ms spent in the when resolving the object storage uri. This happens twice in the same request (search + fetch_docs). On an experimental s3 storage that simply recycles a http client, this takes a few microsecs.

Supposedly this is already done. The http connector is initiated once in the aws_config.
We need to double check if we still have the problem, and whether this is really a client recycling problem.

tested with Wireshark. When doing multiple queries in sequence, we don't get a new tcp/tls handshake for each query. Doing 1 query per second over 300s, I get 9 handshakes. If querying another index sharing the same storage config, no new handshake is made. Connection pooling works as intended.


update dev guide and peaky test notebook

@ocallesp : When this is ready, any chance you can dogfood the new guidelines. Jon was mentioning this could use another pair of eyes for validation last week.


Correct Pi result

Off by .0001

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thanks again @IvenBach!


Alacritty is consistently slower than urxvt

Alacritty’s biggest claim is that it’s the fastest terminal emulator available. If there’s a case where it’s not, then it’s either a bug in Alacritty or a misconfigured system

After a fresh install of Alacritty on OS X, I generated a 38MB text file consisting of 10 million lines of "foo" with the command printf "%0.sfoo\n" {1..10000000} > 10mil.txt and then attempted to compare the speed of Alacritty to the speed of urxvt running in Xquartz by running time cat 10mil.txt a few times in each terminal at a normal size after priming caches.
Times taken in urxvt
2.833s
2.966s
2.555s
Times taken in alacritty
5.611s
5.665s
5.553s

Alacritty’s performance scales well with screen size. Running at larger resolutions will tip the scale further in Alacritty’s favor.

Times taken in urxvt (full screen)
3.348s
3.614s
3.639s
Times taken in alacritty (full screen)
12.819s
12.641s
13.320s
I normally wouldn't consider this a bug, but I figure you probably want all the comparisons you can get if you're trying to make alacritty the fastest terminal emulator. Also thought it was better to open a new issue rather than commenting on an existing performance issue both for organization and because the root causes might be totally different.
Let me know if there's any additional information you'd like me to provide.

Are you using Mesa?

On OS X? I think Xquartz at least packages the library. No idea if it's being used.
Alacritty is just the normal OS X install though. From my understanding of Mesa it shouldn't be involved at all.

Oh, sorry. I saw urxvt and made an assumption without reading carefully. Yeah, this doesn't sound good. I have a few things in mind that may have a big impact here. I'll follow up soon.

Similar results when I run the same tests in full screen (5376x3024) on my Dell XPS 13 running Archlinux with the modesetting driver:
urxvt

cat 10mil.txt  0.00s user 3.06s system 98% cpu 3.093 total

alacritty

cat 10mil.txt  0.00s user 22.32s system 22% cpu 1:37.24 total

That's quite a difference in urxvt's favour.

Another comparison to gnome-terminal with similar findings.

@b52 are you using the Mesa driver?

Another comparison with Terminator where Alacritty is slower as well.

@jwilm I assume so that it is used, yes.

I tested on Ubuntu 16.10 with time hexdump file.txt in full screen. The size of the file was 65MB.  The numbers below are from the second run to warm up the caches:
Alacritty:
real    0m51.136s
user    0m14.060s
sys     0m8.764s

Urxvt:
real	0m17.579s
user	0m10.396s
sys	0m6.604s

Here goes my time cat 10mil.txt results, tested in ArchLinux on MacBookPro 12,1, Dell U2412M(1920*1200), i3wm, xft:mononoki:size=8, all fullscreen:
urxvt:
real	0m3.063s
user	0m0.000s
sys	0m2.963s

alacritty:
real    1m38.291s
user    0m0.003s
sys     0m4.930s

BTW, how can I check which graphic driver I'm using? I've installed xf86-video-intel, wanna check if this is activated or not.

@nykma glxinfo | grep OpenGL

I'm just going to chime in here and say on Ubuntu gnome I get marked increases in speed when using Nvidia drivers from edgers. Gnome terminal is a no contest and I get about a second difference compared to uxvrt

@b52 try out the proprietary drivers

I am hoping to fix the Mesa issue so this part of the perf problem will go away. Although perf can currently be improved by installing the proprietary driver, I don't think it's the project's place to be encouraging that.

@jwilm Thanks.
OpenGL vendor string: Intel Open Source Technology Center
OpenGL renderer string: Mesa DRI Intel(R) Iris 6100 (Broadwell GT3)
OpenGL core profile version string: 4.5 (Core Profile) Mesa 13.0.3
OpenGL core profile shading language version string: 4.50
OpenGL core profile context flags: (none)
OpenGL core profile profile mask: core profile
OpenGL core profile extensions:
OpenGL version string: 3.0 Mesa 13.0.3
OpenGL shading language version string: 1.30
OpenGL context flags: (none)
OpenGL extensions:
OpenGL ES profile version string: OpenGL ES 3.1 Mesa 13.0.3
OpenGL ES profile shading language version string: OpenGL ES GLSL ES 3.10

I was wondering what the progress on this one is.
I think the claim made in the README is kinda bold, given that there are serious performance issues compared to other terminals on some standard platforms.

Hey @b52,
This issue has actually been resolved. There were several duplicates open at one point, and I must have missed this one after the patch.
Did you experience an issue yourself?

@jwilm,
This hasn't been fixed and shouldn't be closed. I just rebuilt alacritty from head and tried and tried the benchmark as outlined in the original issue, and alacritty is the same speed as before.
I know there have been some people who commented on this issue who were actually having mesa issues, but the original issue has nothing to do with that.

@spaghetti2514 oops, my mistake! I looked up at @b52's last comment and it showed the Mesa driver. Going to update the title so I don't make this mistake again.

@jwilm I also rebuilt the latest HEAD and still have the same performance issues as outlined in this issue.

@b52 the Mesa driver issue should be resolved. If that's not the case for you, could you please open a separate issue with current perf details?

@jwilm what ref was that fixed in? I'm just curious is all to see sht the issue was

@Senjai d40d61a73629aae797ab9ff8b36938ea1237acc2

I'd be curious how this benchmark performs on the scrollback PR. It should be faster than urxvt/st now. At least it has been on my machine.

@chrisduerr Performance of the scrollback branch is significantly worse for me.
For the same time cat 10mil.txt with primed caches
urxvt
2.824s
2.826s
2.984s
alacritty
10.714s
10.354s
10.483s
alacritty (fullscreen)
20.166s
20.745s
20.518s
Using master, the times taken are more in line with the numbers from my original comment, so this is caused by something in the scrollback branch specifically.

@spaghetti2514 Are those numbers inside of tmux? That makes a big difference.

@chrisduerr No. Should they be?

Yes. Updated to the latest stable rustc, git pull'd, checked out scrollback, cargo build --release

Hmm, for now I'd wait until #1316 gets fixed, even though that shouldn't be the issue, but it might still help. After that it might make sense to look at this issue again.

Alright. Ping me again when it's fixed and I'll recompile and try again.

There has been some in-depth testing for the Scrollback release an my Mesa system and it outperformed URxvt in every benchmark.
The #1316 issue has significantly improved some aspects of it, so re-testing the current master probably makes sense.
@spaghetti2514 if you're still around and want to benchmark it again, please let me know what your results are.
Otherwise this issue can probably be closed since performance when scrolling should now be pretty much identical to URxvt on the latest master.

@chrisduerr It's gotten somewhat faster, especially at larger window sizes, but it's still not as fast as urxvt.
Compiled alacritty from master using the latest rustc nightly with cargo build --release
urxvt: 2.8s
alacritty: 4.4s
urxvt (full screen): 3.1s
alacritty (full screen): 5.9s
The above are all averages of a few runs


Use the trait for Resolver in ActionHandler

Why would this be necessary?

@shadowhand It's just respect the DRY principle (aka "Don't Repeat Yourself").
Some Responders have the same case, and there it was useful. Where else to use this trait as not here?

Ah my mistake, I was going through PRs too fast. This looks reasonable.

👍


Do the artists who made the "20K hours of licensed music" are aware this project was fed with their work ?

Everything is in the title.
I'm astonished there's only mention of the "20K hours of licensed music" without giving any sources for this (or it could be hidden somewhere I didn't see it). Which artists ? Which original material ?
Without those, you would have to train your program yourself, or the community. At least it would be a real open source philosophy.
But without mention from the original material - used with or without permission - this could be theft.
Thanks in advance for your answers.

https://github.com/facebookresearch/audiocraft/blob/main/MODEL_CARD.md#training-datasets

@rsxdalv Thanks for the links. It is said "See the paper for more details about the training set and corresponding preprocessing."
Do we know what this "paper" is ? There's no easy link to it, anyway.
But even if these are the sources for training this program, it doesn't answer my question.
For instance, does this artist know he has sold his music in order to feed some AI that could possibly one day replace him ?
https://www.pond5.com/fr/artist/marscott
"Selling my music on Pond5 allows me to focus on what I love: making music. I'm able to experiment with different genres and new ways of creating tracks because Pond5 gives me access to a pool of very diverse customers. The interface is easy to use, and the team is friendly and responsive."
Diego Martínez a/k/a Marscott, Music Artist, Chile.
Let's read what says Pond5 to the artists selling their music on this platform (their FAQ) :
"How will my content be used?"
"- Pond5 has a global client base and sales team working for you—your content could be used for a wide variety of projects around the world, ranging from student films to television series and feature films."
No mention of Meta/Facebook/AI developers here, of course...

https://arxiv.org/abs/2306.05284

Alright for the "paper". Still I really don't see anything about the artists sampled and used in the data sets or any consultation, just to check they're aware of all this. In music industry, you can borrow or sample some other artist to produce another title, you have to pay royalties (Pond5 is actually doing this), but you have to also probably to ask for permission. We all know some stories about an artist who didn't let his music be used for some publicity or marketing advert, yet agreed to let his music be covered by others.
That's why I opened this issue and asked about it. Not really a technical one, but still could the biggest of all on this kind of "sorcerer's apprentice" AI developments...

the dataset is 'internal' (according to the paper) so until it is released to the public I don't think we will be seeing any details about the creators or who is involved.
I wouldn't be quite so harsh to the developers of this model. Due to the lack of information on this subject, most likely a nondisclosure agreement, we cant expect them to respond with potentially confidential information other then what is readily available and already published.
Although I presume that they made sure to take the proper precautions before creating a dataset like this to not infringe on the authors rights or interests, which if not the case could lead to issues later on.

Actually, according to the paper, the dataset is half 'internal" ('10K high-quality music tracks', licensed), and the other half coming from Shutterstock and Pond5 (so 10K too).
There's an interesting discussion about all those matters here : https://youtu.be/Wij2wH_Xi1c
One of the conclusions is about transparency, and that's why I opened this issue, since we don't know who are the artists who made this technology usable - beside developers, of course. And we don't know if those are aware this is happening, and if they are involved beyond what they think they signed for.
Am I harsh, really ? When you can see that open source Chat GPT and DALL-E can already replace some basic code developer, Midjourney can achieve some pictures for many marketing requirements without really the need to hire some visual artist... Isn't this harsh ? Is it too much to ask about some kind of equity and transparency ?
Well, I'm not convinced myself 'they made sure to take the proper precautions'. All the GAFAM are known to push and break the ethical or legal boundaries in order to gain some monopoly, then when public opinion is becoming aware of some issues, they have a bunch of lawyers to resolve the problem or try to lower the fines... But in the meantime, it can too late and harm is done. Meta / Facebook is clearly not the last, and has been sentenced numerous times.

Actually, according to the paper, the dataset is half 'internal" ('10K high-quality music tracks', licensed), and the other half coming from Shutterstock and Pond5 (so 10K too). There's an interesting discussion about all those matters here : https://youtu.be/Wij2wH_Xi1c One of the conclusions is about transparency, and that's why I opened this issue, since we don't know who are the artists who made this technology usable - beside developers, of course. And we don't know if those are aware this is happening, and if they are involved beyond what they think they signed for. Am I harsh, really ? When you can see that open source Chat GPT can already replace some basic code developer, Midjourney and DALL-E can achieve some pictures for many marketing requirements without really the need to hire some visual artist... Isn't this harsh ? Is it too much to ask about some kind of equity and transparency ? Well, I'm not convinced myself 'they made sure to take the proper precautions'. All the GAFAM are known to push and break the ethical or legal boundaries in order to gain some monopoly, then when public opinion is becoming aware of some issues, they have a bunch of lawyers to resolve the problem or try to lower the fines... But in the meantime, it can be too late and harm is done. Meta / Facebook is clearly not the last, and has been sentenced numerous times.

I purposefully left out the ethical implications of work like this from that perspective not to start an argument about it, which was both ways inevitable. The main reason I said to not treat the researchers so "harshly" is the fact that the researchers are not intitled to the work that they do working under a major company like Meta Research, so pointing a finger at the researchers who have poured hours into their own preferred fields of computer science would be unreasonable. I am not trying to defend Meta, or any other possible monopoly, I want to defend the researchers. I am in favor of the current push to open source the hardware and software that platforms like ChatGPT and CharacterAI depend on, but I am also aware that this must be regulated on some level (whatever that might look like).
So please, lets not start this debate. As I think this is not the place to talk about such a topic (mainly being for error reports to improve this software), If you want to debate about this I am sure there are multiple Reddit forums or other places to talk about it.
Back to the topic at hand, I guess you wouldn't be able to get any answers on how this dataset was put together and what restrictions and freedoms where placed on the composers, writers, and band members, unless you managed to find somebody that contributed to it (which seems like a slim chance now looking at it).

Just a reminder that there's other completely opaque companies that offer
description based music generation. Not open source, no information about
the data set, nothing.
In addition, this model is under a very strict license - CC BY NC 4.0. I am
sure you are familiar with it. Although it's not a guarantee for the future
of all possible scenarios, we can't say "legally" that they have unleashed
music generation.
On Mon, Jul 10, 2023, 1:23 PM Kelvin Clien @.***> wrote:

Actually, according to the paper, the dataset is half 'internal" ('10K
high-quality music tracks', licensed), and the other half coming from
Shutterstock and Pond5 (so 10K too). There's an interesting discussion
about all those matters here : https://youtu.be/Wij2wH_Xi1c http://url
One of the conclusions is about transparency, and that's why I opened this
issue, since we don't know who are the artists who made this technology
usable - beside developers, of course. And we don't know if those are aware
this is happening, and if they are involved beyond what they think they
signed for. Am I harsh, really ? When you can see that open source Chat GPT
can already replace some basic code developer, Midjourney and DALL-E can
achieve some pictures for many marketing requirements without really the
need to hire some visual artist... Isn't this harsh ? Is it too much to ask
about some kind of equity and transparency ? Well, I'm not convinced myself
'they made sure to take the proper precautions'. All the GAFAM are known to
push and break the ethical or legal boundaries in order to gain some
monopoly, then when public opinion is becoming aware of some issues, they
have a bunch of lawyers to resolve the problem or try to lower the fines...
But in the meantime, it can be too late and harm is done. Meta / Facebook
is clearly not the last, and has been sentenced numerous times.
I purposefully left out the ethical implications of work like this from
that perspective not to start an argument about it, which was both ways
inevitable. The main reason I said to not treat the researchers so
"harshly" is the fact that the researchers are not intitled to the work
that they do working under a major company like Meta Research, so pointing
a finger at the researchers who have poured hours into their own preferred
fields of computer science would be unreasonable. I am not trying to defend
Meta, or any other possible monopoly, I want to defend the researchers. I
am in favor of the current push to open source the hardware and software
that platforms like ChatGPT and CharacterAI depend on, but I am also aware
that this must be regulated on some level (whatever that might look like).
So please, lets not start this debate. As I think this is not the place to
talk about such a topic (mainly being for error reports to improve this
software), If you want to debate about this I am sure there are multiple
Reddit forums or other places to talk about it.
Back to the topic at hand, I guess you wouldn't be able to get any answers
on how this dataset was put together and what restrictions and freedoms
where placed on the composers, writers, and band members, unless you
managed to find somebody that contributed to it (which seems like a slim
chance now looking at it).
—
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/audiocraft/issues/137#issuecomment-1628660734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTRXI72W6DV2X2ISRDQFUDXPPJZHANCNFSM6AAAAAAZZWNWKI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@rsxdalv Hey thanks for your input! Two questions:

Where did you find the model license, here? https://github.com/facebookresearch/audiocraft/blob/main/LICENSE_weights

What other "opaque companies that offer description-based music generation" are you specifically talking about? Just wondering, I'm trying to learn the space. Thanks!

Also the "CC BY NC 4.0" license does not seem so strict. Accoridng to this docs page on Creative Commons it allows sharing for commercial purposes (which I'm guessing is the purpose those 10k/20k artists would care about: https://creativecommons.org/licenses/by-nd/4.0/).

Also the "CC BY NC 4.0" license does not seem so strict. According to this docs page on Creative Commons it allows sharing for commercial purposes (which I'm guessing is the purpose those 10k/20k artists would care about: https://creativecommons.org/licenses/by-nd/4.0/).

Please don't mix BY ND with BY NC, they are very different.
Here's NC: https://creativecommons.org/licenses/by-nc/4.0/

@rsxdalv Hey thanks for your input! Two questions:

Where did you find the model license, here? https://github.com/facebookresearch/audiocraft/blob/main/LICENSE_weights
What other "opaque companies that offer description-based music generation" are you specifically talking about? Just wondering, I'm trying to learn the space. Thanks!

Yes. Model is MIT, so if you had a perfectly legally ideal dataset and a huge amount of resources you could make your own musicgen without worrying about the Model weights which are not MIT.

To round this discussion off, here is a wiki page on CC's definition of NonCommercial (for informational, not legal, use only):
https://wiki.creativecommons.org/wiki/NonCommercial_interpretation

@rsxdalv Hey thanks for your input! Two questions:

Where did you find the model license, here? https://github.com/facebookresearch/audiocraft/blob/main/LICENSE_weights
What other "opaque companies that offer description-based music generation" are you specifically talking about? Just wondering, I'm trying to learn the space. Thanks!

Yes. Model is MIT, so if you had a perfectly legally ideal dataset and a huge amount of resources you could make your own musicgen without worrying about the Model weights which are not MIT.

What's the difference between a model and its weights? I thought weights were the model...

@rsxdalv Hey thanks for your input! Two questions:

Where did you find the model license, here? https://github.com/facebookresearch/audiocraft/blob/main/LICENSE_weights
What other "opaque companies that offer description-based music generation" are you specifically talking about? Just wondering, I'm trying to learn the space. Thanks!

Yes. Model is MIT, so if you had a perfectly legally ideal dataset and a huge amount of resources you could make your own musicgen without worrying about the Model weights which are not MIT.

What's the difference between a model and its weights? I thought weights were the model...

A model is the digital representation of a neural network, so the structure, encoder etcetera (most exclusively as programmed by the developers). the weights of a model tell that model how much it should change the input and how it should change the input, this is mostly done through a process called training and fine-tuning. Training is done at a network scale, where an expected input and an expected output is given, and using an algorithm it changes the weights and biases of the neurons in the network to achieve the expected output with the given input.
so in simple terms, the model is the code and the weights and biases allow the code to give the expected output for any given input.

Please do note that that is a high level overview, and should not be generalized between all neural networks, as they can differ quite a lot depending on their intended use case

@rsxdalv Hey thanks for your input! Two questions:

Where did you find the model license, here? https://github.com/facebookresearch/audiocraft/blob/main/LICENSE_weights
What other "opaque companies that offer description-based music generation" are you specifically talking about? Just wondering, I'm trying to learn the space. Thanks!

Yes. Model is MIT, so if you had a perfectly legally ideal dataset and a huge amount of resources you could make your own musicgen without worrying about the Model weights which are not MIT.

What's the difference between a model and its weights? I thought weights were the model...

The model is the structure of the "child's" brain, the training dataset is the curriculum, and the final weights after training is the "educated child". But sometimes this is smudged to call the actual functional "app" the model.
For a well known reference, ChatGPT takes months to train and has cost millions to get the correct weight values.

I am confused: can we use the music generated by audiocraft musicgen for commercial use or not?

@rsxdalv I am confused: can we use the music generated by musicgen for commercial use or not?

No you can't, see the link he posted:
https://creativecommons.org/licenses/by-nc/4.0/

@rsxdalv I am confused: can we use the music generated by musicgen for commercial use or not?

No you can't, see the link he posted:
https://creativecommons.org/licenses/by-nc/4.0/

ok, thanks... no wonder nobody is talking about it anymore ;)

@rsxdalv Hey thanks for your input! Two questions:

Where did you find the model license, here? https://github.com/facebookresearch/audiocraft/blob/main/LICENSE_weights
What other "opaque companies that offer description-based music generation" are you specifically talking about? Just wondering, I'm trying to learn the space. Thanks!

Yes. Model is MIT, so if you had a perfectly legally ideal dataset and a huge amount of resources you could make your own musicgen without worrying about the Model weights which are not MIT.

@rsxdalv I run a generative AI startup that's raising funds right now, would you want to try and get those resources to make our own MusicGen weights?

All this is interesting, but I'm still struggling to find the answer about this issue  ^^

who cares

who cares

Hello grumpy smurf. Well, maybe a few million of people, I guess...


Remove dnxcore50 from program path template

@gregg-miskelly

@caslan

don't merge, Charlie's saying it's not right

It needs to be  and not just  as they will need to enter names of sub folders.

I meant path-to-program and not my-project-name

FYI using brackets in markdown makes your words disappear so that comment is bogus.

how does the diff look now?

LGTM

:+1:


Sync reading files

I've just started looking code.
I spotted that a lot of synchronous code for reading/checking files fs.readFileSync, fs.existsSync.
Is it necessary? Can you give short explanation?

These operations are blocking in nature. And building process does not have concurrency in most case. Node's single thread model precludes multi-core building, indeed.
So sync is clearly better.

@HerringtonDarkholme Not that I have a stake here, and am only aware of this module since it is influenced by my progeny. But I strongly disagree with the perspective you just expressed. There's a significant perf difference using sync vs async methods with the file system. And there is obvious i/o concurrency in this use case when crawling through a dependency graph.
You'd be much better off actually using progeny under this than the reimplementation you ended up with.

@es128 I respect your authority, and I don't want to argue with a brilliant developer solely based on my own opinion.
Feel free to leave this reimplementation I ended up with. But if you are very interested. Here is a little test.js I jotted down.
var fs = require('fs')
var startSync = Date.now()
for (var i = 0; i < 10000; i++) {
  fs.readFileSync(__dirname + '/test.js')
}
console.log('sync read: ' + (Date.now() - startSync))

var startAsync = Date.now()
var i = 0
function readFile() {
  fs.readFile(__dirname + '/test.js', function() {
    if (++i >= 10000) {
      console.log('async read: ' + (Date.now() - startAsync))
    }
  })
}
for (var j = 0; j < 10000; j++) {
  readFile()
}

Not trying to be argumentative, just discussing what appears to be a misconception on your part. The points you made in response to the question are wrong, and this module would almost certainly benefit from refactoring to async.
You're completely misleading yourself with this test. Using async doesn't make the file get read faster, it unblocks the node process.

Benchmark, please.
I don't think refactoring to async will be a big help. Bottleneck of building is compiling, not reading file. It's compiling blocks readFileSync, not vice-versa.
And async incurs context-switching overhead, closure overhead, and I'm not sure concurrent file reading can compensate these overhead. Even it does, I don't think it worth several callback recursion in code.

Bottleneck of building is compiling, not reading file

That's pretty funny coming from someone working within the gulp ecosystem.

Please avoid ad hominem. I cannot understand your point.

It wasn't ad hominem. The point is that you're effectively arguing that grunt should be just as performant as gulp.

I don't see the rhetorical technique to be valid, though.
Again, please provide factual evidence to support gulp is more performant than grunt. And tell how it relates to this issue.
I have stated my opinion towards synchronous reading, and provides factual benchmark(regardless of its validity).

It's ok, I won't be spending any more time on this. I am genuinely sorry if any of my comments came across as hurtful, that wasn't my intention - I was just reacting to your initial response which came across to me as very far off base. I believe your perception of fs sync methods is incorrect, but it's up to you to be receptive to that feedback and to look further into it. It's your module, you're free to run it however you like.

Agree.
Using *-sync is a bad practice. It is not gulpy way.
Consider to replace filter(fs.exists) by filter.asyc or similar, using map from event-stream and so on.

Pull request is welcom


Config changes to reflect the default branch from master to main. Please merge to complete the default branch renaming job.

Auto branch renaming content.

Docs Build status updates of commit 9843f3f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch 130c6cec-1a30-4896-a13f-9b254123c1b6 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix the controller updateServiceBrokerClient  method.

This PR is a

[ ] Feature Implementation
[x] Bug Fix
[ ] Documentation

What this PR does / why we need it:
This PR fixes storing OSB clients for namespaced service brokers. The line:
https://github.com/kubernetes-incubator/service-catalog/blob/master/pkg/controller/controller_servicebroker.go#L133
treats the broker like cluster wide, but namespaced broker is being processed. Additionally, a unit test was added, which was failing before the fix.
Please leave this checklist in the PR comment so that maintainers can ensure a good PR.
Merge Checklist:

[ ] New feature

[ ] Tests
[ ] Documentation

[ ] SVCat CLI flag
[ ] Server Flag for config

[ ] Chart changes
[ ] removing a flag by marking deprecated and hiding to avoid
breaking the chart release and existing clients who provide a
flag that will get an error when they try to update

/lgtm


Create javascript SDK for verification App

What is the feature request for?
Registry Core
What problem/inconvenience (if any) will this feature solve?
It is inconvenient for me to do .... If we add this feature, it would make it easier to do ....
Describe the feature clearly.
https://digit-discuss.atlassian.net/browse/DPG-2159

@shiva-beehyv Please add description

Verification sdk:
Purpose of this sdk:
The aim is to create a common utility package so that new users who are trying to create their own Verifier app can just import this as a npm package and have the vc validation code functionality.
Cururently, the library exports the following method:
validateVC(vc, options)

vc - the verfiable credetntials
options - valid issuers, ignore expiry date, etc. and more options as needed (maybe looking for a particular attributes in the vc, for example)

Verification process:

get public key using did
validate the vc - signature
validate metadata - check for expiry time, issuer, etc. fields
validate options if passed
check if the vc is revoked

Let's also look at downloading revocation list, which will be mentioned in the URL. @shiva-beehyv can you also list function signatures and libraries.

Duplicate of https://github.com/Sunbird-RC/community/issues/917


flutter text field issues on ios

latest flutter v.7 has text field issues on ios. can type myanmar unicode font correctly on text field but if back delete the the text skip some character and not deleting. but in flutter 1.5 it has no issues so please review it

ok i will reply a video later thank you for your reply
On Mon, Aug 26, 2019 at 11:42 PM Hans Muller<EMAIL_ADDRESS>wrote:

CC @GaryQian https://github.com/GaryQian
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/39169?email_source=notifications&email_token=AJXVZ27GRIEPR4TIBEXTQIDQGQFJVA5CNFSM4IPFEOCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5FAJGI#issuecomment-524944537,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJXVZ25FXJ765NBX56EGIYLQGQFJVANCNFSM4IPFEOCA
.

in flutter version 1.9 unicode is fix bro but i have to test zaw gyi font
On Fri, Sep 13, 2019 at 3:20 AM htetwaiyan<EMAIL_ADDRESS>wrote:

hello. is it already fix???? bro
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/39169?email_source=notifications&email_token=AJXVZ24KRSCFWILSZIHYZX3QJKTSTA5CNFSM4IPFEOCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6TGXZY#issuecomment-531000295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJXVZ23N3SOHB2IRYQZ25KTQJKTSTANCNFSM4IPFEOCA
.

I will consider this fixed then. If this is not fixed, feel free to comment here and we can reopen this!


query should be able to specify redirect uri

For installed clients like ios apps, the redirect comes back to the phone, and the call to the meteor server to exchange the code for a token is made explicitly. Slack requires the token exchange call to specify a redirect uri that has to match the original redirect uri used for obtaining the code. Thus, the installed clients need to be able to pass a redirect uri into the accounts-slack getAccessToken() method that can override the default.

Merged


Ensure we rely only on dnsmasq

This patch corrects and finalize the move from CRC hard-coded hosts
entries to full dnsmasq DHCP and DNS.
In a previous PR (#1908) we switched CRC dns entries to dnsmasq, but the
base CRC config weren't cleaned, leading to a broken state.
It was mostly working because controller-0 still had some hardcoded
entries in /etc/hosts.
This patch fixes the initial issue with dnsmasq configuration overlaps
(ours vs CRC), and ensures we don't rely on /etc/hosts entries on
controller-0.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

/approve


fix: saved searches clear all button logic

ONYX-455
Description
This PR fixes an issue where the artist id is getting removed after pressing on the clear all button.
https://github.com/artsy/eigen/assets/11945712/a5bc319b-420b-4b5c-b3e5-3d435d121980

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

fix saved searches clear all button logic - mounir

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (fix saved searches clear all button logic - mounir)

  Generated by :no_entry_sign: dangerJS against f623fa732a11698caf68efcf394e777b107e3316


Fix race condition when using the Uchiwa filters

Fix https://github.com/sensu/uchiwa/issues/543

Coverage decreased (-0.5%) to 27.94% when pulling 0a8221374645e3a2a0512b59f6d3155fe9f9bb25 on palourde:go16-concurrent-map-rw into caafa493c12cf34d4724f0b8c2e12d787c850b65 on sensu:master.


Renaming a test doesnt delete verified file

Describe the bug
When renaming a test the old verified file does not get deleted. I'm using NUnit 3
Minimal Repro
Create a test, run it, save the verified file.
Rename test, run it, save the new verified file.
The first verified file still exists. I would expect it to be deleted

unfortunately i know of no way to detect a rename. happy to hear suggestions

If we had an opt-in option like .CleanUpUnmatchedVerificationFiles() any files that don't match a list of patterns generated from a list of tests in the project (provided by the specific test library) could be deleted or moved to another directory. This would assume a few things, but these might be acceptable trade-offs to have unused files removed:

They aren't rewriting the method name with .UseMethodName() though it could throw an exception in this case.
The full list of tests is discoverable, this might slow down single test runs but possibly could be cached or run only when a few tests are run.

I can see how implementing this might be more complicated than its worth, but may also help to make adopting Verify more accessible.

@rebelzach  Happy to consider a PullRequest

This might be easier to implement: after checking that the received and verified files are the same, Verify could modify the last modified date of the verified file to now. That way if I look on the folder of snapshots in some time and see files that have not updated I know they are probably obsolete (they could belong to a test that is not executed by default so I would not rush to auto-delete anything)


fix support level for HTTPRequestHeaderFilter

What type of PR is this?

/kind bug
What this PR does / why we need it:
As noted in #655, a core filter with extended support fields is
confusing and also seems to be incorrect.
This patch drops the extended field from the filter and is a partial fix
for the referenced issue.
HTTPRequestMirroFilter has not been patch as GEP-718 could potentially
simplify that.
Which issue(s) this PR fixes:

Partial fix for #655
Does this PR introduce a user-facing change?:

bugfix: To avoid confusion, fields within HTTPRequestHeaderFilter are no more marked as 'extended'. All fields must be supported by implementations for this core filter.

/cc @banks @robscott

Thanks!
/lgtm


Update README with Data safety section

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes b/200620733
Description
A data safety section was added to the readme as external documentation informing users that this SDK does not collect or share any personal or sensitive user data.
Type
Documentation
Checklist

[X] I have read and acknowledged the Code of conduct
[X] I have read How to Contribute
[X] I have read the Developer's guide
[X] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[X] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[ ] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[ ] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

@fredhersch fyi


Add sidebar_mini option

I've added an option to enable "sidebar-mini" class in the body tag :) Enjoy

Nice and clean, thank you for your contribution!


Remove mode parameters from RSA functions

The following functions in the RSA module take a mode parameter, which allows using a private key where a public key would normally be used or vice versa:

mbedtls_rsa_pkcs1_encrypt
mbedtls_rsa_rsaes_pkcs1_v15_encrypt
mbedtls_rsa_rsaes_oaep_encrypt
mbedtls_rsa_pkcs1_decrypt
mbedtls_rsa_rsaes_pkcs1_v15_decrypt
mbedtls_rsa_rsaes_oaep_decrypt
mbedtls_rsa_pkcs1_sign
mbedtls_rsa_rsassa_pkcs1_v15_sign
mbedtls_rsa_rsassa_pss_sign
mbedtls_rsa_pkcs1_verify
mbedtls_rsa_rsassa_pkcs1_v15_verify
mbedtls_rsa_rsassa_pss_verify
mbedtls_rsa_rsassa_pss_verify_ext

Using the wrong mode is deprecated and should not be supported anymore.
Scope:

If there are any unit tests with the wrong mode, remove them.
Global search-and-replace all calls to these functions to remove the mode parameter.
Remove the mention and checks on the mode parameter in rsa.c. Keep only the code path for the correct mode: MBEDTLS_RSA_PRIVATE for decrypt and sign, MBEDTLS_RSA_PUBLIC for encrypt and verify.
Remove all mentions of the mode parameter in rsa.h, and the constants MBEDLTS_RSA_PUBLIC and MBEDTLS_RSA_PRIVATE.

Migration path: Don't pass a mode parameter to these functions. If you were passing the correct mode, just remove the parameter. If you were passing the wrong mode, don't do it.

Not completed yet.


GEODE-10135: Use anonymous classes in function service tests

PROBLEM
The function service tests use lambdas to test the function service. The
default implementation of Function.getId() returns the canonical name
of the function's class. In JDK17, the class of a lambda has no
canonical name, and so each function returns null as its ID. This
causes nearly every function service test to fail.
SOLUTION
Change the tests to use an abstract class that returns its class's name
as its ID.

This PR fixes the 160 function service tests on JDK 17. Compare

This PR's distributed test failures: http://files.apachegeode-ci.info/builds/apache-develop-pr/geode-pr-7492/test-results/distributedTest/1648253657/
Failures on another PR that does not have this fix: http://files.apachegeode-ci.info/builds/apache-develop-pr/geode-pr-7475/test-results/distributedTest/1648253596/

The other test failures on JDK17 are expected, and will be fixed on other PRs.

Will fix product instead of tests.


CLI example repository

Create a repository with CLI examples. Link to relevant PRs below.

is this still a relevant issue, or could it be closed?


Replaces the name gemm by matrix_multiply

gemm is the interface from BLAS. Since the interface in T-LAPACK is different, it makes sense to rename it.
Also, by renaming it we could make different design choices.

matrix_multiply has two versions: one with and one without beta.

The first version computes C := alpha*A*B + beta*C;
The second computes C := alpha*A*B.
They are semantically different: in the first, C is [in,out]; in the second, C is [out] only.
Legacy gemm covers both functionalities depending on the value of beta (if beta is zero, use second expression).
The problem with the strategy adopted in gemm is: if beta evaluates to zero in some part of an algorithm and there are Infs or NaNs in C, those will not propagate.

The proposal is: if one wants to obtain the same behavior of gemm and

alpha=0 and beta=0: just use laset to set the matrix C to 0.
alpha=0 and beta is not zero: just use lascl to scale the matrix C.
alpha is not zero and beta=0: use the version of matrix_multiply that considers C as output only.
otherwise, use matrix_multiply that considers C as input and output.

This PR also introduces the StrongZero internal data type. This is to avoid code duplication.
Also, it solves failing tests in https://github.com/tlapack/testBLAS/runs/7583337777.

After talking to @langou, I decided to address only the following part on this PR:
Split the 2 behaviors of gemm (and other matrix multiplication routines) as follows:

The first version computes C := alpha*A*B + beta*C;
The second computes C := alpha*A*B.
They are semantically different: in the first, C is [in,out]; in the second, C is [out] only.
Legacy gemm covers both functionalities depending on the value of beta (if beta is zero, use second expression).
The problem with the strategy adopted in legacy gemm is: if beta evaluates to zero in some part of an algorithm and there are Infs or NaNs in C, those will not propagate. Those 2 versions aims to improve it.

If one wants to obtain the same behavior of legacy gemm and

alpha=0 and beta=0: just use laset to set the matrix C to 0.
alpha=0 and beta is not zero: just use lascl to scale the matrix C.
alpha is not zero and beta=0: use the version of gemm that considers C as output only.
otherwise, use gemm that considers C as input and output.

This commit also introduces the StrongZero internal data type. This is to avoid code duplication without losing performance. StrongZero evaluates to 0 at compile time.
TO-DO list:

[ ] Apply similar modifications to all lv-3 BLAS routines.
[ ] Change the documentation.
[ ] Add tests to the test suite.

About the internal class StrongZero:
This is class, for now, defines multiplication and sum with another class T. Other functionality may be added in future if needed.
Suppose x is of type T. Then:

T(StrongZero()) is equivalent to T(0).
x *= StrongZero() is equivalent to x = T(0).
x += StrongZero() does not modify x.

This class satisfies:
 x * StrongZero() == StrongZero()
 StrongZero() * x == StrongZero()
 x + StrongZero() == x
 StrongZero() + x == x

https://github.com/tlapack/tlapack/blob/f84114d665f6a0fce336cc0b0f903b35decdcf67/include/tlapack/base/types.hpp#L60-L110
I use this class inside the wrapper for C := alpha*A*B as return gemm( transA, transB, alpha, A, B, internal::StrongZero(), C ). See:
https://github.com/tlapack/tlapack/blob/f84114d665f6a0fce336cc0b0f903b35decdcf67/include/tlapack/blas/gemm.hpp#L200-L254


[Feature]: Provide progreesbar for table data load.

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Progress bar is required for data loading in table. OnPageChange also It has to show the loading indicator otherwise it is difficult to understand which page data it is showing in the meanwhile.
Why should this be worked on?
Without progress bar, it is very cumbersome to work with table

@lavanyaG4 the page number is shown in the table is that not enough? Can you share a screenshot of what you expect this to look like?

Now It shows 100 Records Page 1 of 2.
I want like this 100 Records(1-50) Page1 of 2
100 Records(50-100) Page 2 of 2.
On Fri, Aug 26, 2022 at 6:39 PM Nikhil Nandagopal @.***>
wrote:

@lavanyaG4 https://github.com/lavanyaG4 the page number is shown in the
table is that not enough? Can you share a screenshot of what you expect
this to look like?
—
Reply to this email directly, view it on GitHub
https://github.com/appsmithorg/appsmith/issues/16319#issuecomment-1228468377,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZHTPVEOVNTK766XYXIHZ3TV3C62JANCNFSM57VRF7UQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

hey @lavanyaG4 the title says you are looking for a progress bar, but your last comment says you want to show number of records. Can you clarify this? Are you looking to show a progress bar when data is still loading on the table? Or simply show no.of records on the page?

Yes, I want both.1. Progressbar while data loading on table and
Onpagechange data loading. 2.Record count
On Mon, Aug 29, 2022 at 2:45 PM dilippitchika @.***>
wrote:

hey @lavanyaG4 https://github.com/lavanyaG4 the title says you are
looking for a progress bar, but your last comment says you want to show
number of records. Can you clarify this? Are you looking to show a progress
bar when data is still loading on the table? Or simply show no.of records
on the page?
—
Reply to this email directly, view it on GitHub
https://github.com/appsmithorg/appsmith/issues/16319#issuecomment-1229999039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZHTPVCW3WCPE25Z6FBJUJTV3R5S3ANCNFSM57VRF7UQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

What @lavanyaG4 said. I would like to know when a table is loading; "isLoading" for tables. Not when the data has been retrieved but when the table is being populated, et all. "isLoading" should return false when the data is usable in the table. Maybe even better... Put a loading icon that spins right in the middle of the table until it's all ready to display.

@tlawrence-TAG is this because of #12308 if the table skeleton did not disappear, would that solve all problems?


docs(explore): fix squid assets path

Pull request checklist
Use the following template to make sure your PR fits the ConsenSys documentation standard.
Before creating the PR
Make sure that:

[x] You've read the contribution guidelines.
[x] You've previewed your changes locally.

Describe the change

Fixes assets path
Issue fixed

No current issue exists
Impacted parts

For content changes:

[x] Documentation content
[ ] Documentation page organization

For tool changes:

[ ] Build and QA tools
[ ] Node dependencies and JavaScript
[ ] GitHub integration

After creating your PR and tests have finished
Make sure that:

[ ] You've fixed any issues raised by the tests.
[ ] You've previewed your changes on Vercel below.

@odcey can you rebase on the main repo so that the lint errors will be resolved
You can run npm run lint to check

@emilyJLin95
Should be fixed now !