util: fix tracevol.py to use namespace argument

Fixed following issues in tracevol.py:

a bug where the existing namespace argument was not used.
added namespace argument for config map.

Signed-off-by: Mudit Agarwal<EMAIL_ADDRESS>Fixes: #1286

@humblec can you verify this works for you?

Please DNM, we found another bug in the script.
To get config map data we use this
cm_data = config_map['data']['config.json']
But when rook creates config map it has csi-cluster-config.json instead of config.json, need to fix it in a generic way.
Thanks a lot @humblec for the help in testing.

I just checked that if we use config map created by rook then their is no option to provide subvolumeGroup. Rook creates config map only telling about clusterID and monitor information.
Is my understanding correct?

I just checked that if we use config map created by rook then their is no option to provide subvolumeGroup. Rook creates config map only telling about clusterID and monitor information.
Is my understanding correct?

we have not exposed the subvolume group name configuration in rook yet.

Please DNM, we found another bug in the script.
To get config map data we use this
cm_data = config_map['data']['config.json']
But when rook creates config map it has csi-cluster-config.json instead of config.json, need to fix it in a generic way.
Thanks a lot @humblec for the help in testing.

yeah.. nw! Happy to test.. so in short the cm fetching fails with this current version , Isnt it @agarwal-mudit  ?

Please DNM, we found another bug in the script.
To get config map data we use this
cm_data = config_map['data']['config.json']
But when rook creates config map it has csi-cluster-config.json instead of config.json, need to fix it in a generic way.
Thanks a lot @humblec for the help in testing.

yeah.. nw! Happy to test.. so in short the cm fetching fails with this current version , Isnt it @agarwal-mudit ?

If cm is created by rook, it will pass if cm is created by ceph-csi. Working on it, will post another version in sometime

[hchiramm@control-plane ceph-csi]$ python3 /home/hchiramm/Downloads/tracevol.py -k demo/demo-kubeconfig -n test -rn openshift-storage -cm rook-ceph-csi-config -cmn openshift-storage -debug
+----------+---------+------------+-----------------+------------------+------------------+
| PVC Name | PV Name | Image Name | PV name in omap | Image ID in omap | Image in cluster |
+----------+---------+------------+-----------------+------------------+------------------+
+----------+---------+------------+-----------------+------------------+------------------+
+-----------------+------------------------------------------+----------------------------------------------+-----------------+----------------------+----------------------+
|     PVC Name    |                 PV Name                  |                Subvolume Name                | PV name in omap | Subvolume ID in omap | Subvolume in cluster |
+-----------------+------------------------------------------+----------------------------------------------+-----------------+----------------------+----------------------+
| cephfs-demo-pvc | pvc-5e06d05a-8fe2-4750-ba9c-5fa1c7ac68ea | csi-vol-7b4c9b39-d0b7-11ea-9049-0a580a800406 |       True      |         True         |         True         |
+-----------------+------------------------------------------+----------------------------------------------+-----------------+----------------------+----------------------+
[hchiramm@control-plane ceph-csi]$

Here is the final patched script out.. Thanks @agarwal-mudit ! :+1:

@Madhu-1 @humblec I have added more commits, PTAL
You can remove DNM label now.

@agarwal-mudit CI failure. PTAL

@agarwal-mudit CI failure. PTAL

commit message's header was longer than 72 characters, fixed it. Should pass now. Thanks.

ci/centos/containerized-tests

/retest ci/centos/containerized-tests

/retest ci/centos/containerized-tests

/retest ci/centos/containerized-tests


Create: 929-Unique-Email-Addresses.py

File(s) Modified: 929-Unique-Email-Addresses.py
Language(s) Used: python
Submission URL: _https://leetcode.com/problems/unique-email-addresses/submissions/832382968/_

Sorry, this is a duplicate


Trackpad doesn't work on Bigsur 11.16

Just don't know why

u can fix that with some kext, i cant boot into monterey but was a easy fix, just search a little bit :)

Wait a few seconds in opencore boot picker before pressing enter. Sometimes the trackpad won't work if I press enter just after opencore appears.


Registered npm version is older than the current on this repository

I can not update to the latest release because the registered version is 3.0.0 (see: npm-site).

Next Version will be published on NPM
Issue will be closed

Next version will be published


Automated dependency updates

Hello,
I discovered this project via the magic mirror project (https://magicmirror.builders). Snyk.io shows 2 vulnerabilities due to outdated rrule and after integrating dependabot on my fork, github security gave warning about a vulnerability in outdated diff as well. I'd like to recommend integrating dependabot or something similar to ease maintenance for everyone and automatically update dependencies.
Thank you!

Good idea, added, thanks!


Add final restart file at terminal heating points/ md runs

As discussed in #108 the addition is to dump additional xyz files at the end of heating steps, and md runs

There is a wip

hi @harveydevereux can you please open a pr, and @ElliottKasoar will give the first review

hi @harveydevereux can you please open a pr, and @ElliottKasoar will give the first review

will do once I have sorted the tests I broke as we just talked about

hi @harveydevereux can you please open a pr, and @ElliottKasoar will give the first review

will do once I have sorted the tests I broke as we just talked about

@harveydevereux you can always open it as a draft. I know you've linked it in this case so it doesn't make a huge difference, but generally the earlier we can start to take a look the better.


Make param.depends efficient

Its becoming more and more clear to me just how inefficient param.depends and param.bind are if you want to depend on them multiple times.
Thus I suggested at HoloViz meeting that for Panel that we start recommending pn.rx(some_func)(args,...) over pn.bind(some_func, args...) because the end usage is the same. But the pn.rx version is much more efficient. But the feedback was that pn.rx was not yet well enough understood.
As an alternative I would suggest making pn.depends  (and pn.bind) more efficient by caching results similar to what pn.rx does.
One more argument is that Panel tutorials promotes creating more advanced applications with the DataStore design pattern. And this also promotes depending multiple times on references. The tutorial uses pn.rx. I don't know if Philipp did it for efficiency reasons or just because he could? But using param.depends without watch=True would have led to an inefficient application https://panel.holoviz.org/tutorials/intermediate/structure_data_store.html.
Example
The below example shows how inefficient it is to depend on a param.depends annotated function multiple times.
https://github.com/holoviz/param/assets/42288570/c814b45f-d3e7-419f-8392-6c2e89930c4e
import param
import panel as pn

pn.extension()

slider = pn.widgets.IntSlider(value=0, start=1, end=2)

class Model(param.Parameterized):
    value = param.Integer(default=1, bounds=(1,10))

    count = param.Integer(default=0)

    @param.depends("value")
    def result(self):
        self.count+=1
        return self.value

    @param.depends("result")
    def other_result(self):
        return self.result()

class Model2(param.Parameterized):
    value = param.Integer(default=1, bounds=(1,10))
    result = param.Integer()
    other_result = param.Integer()

    count = param.Integer(default=0)

    @param.depends("value", watch=True)
    def _update_result(self):
        self.count+=1
        self.result = self.value

    @param.depends("result", watch=True)
    def _update_other_result(self):
        self.other_result= self.result

model = Model()
model2 = Model2()

pn.Row(
    pn.Column(model.param.value, model.param.count, model.result, model.other_result),
    pn.Column(model2.param.value, model2.param.count, pn.pane.Str(model2.param.result), pn.pane.Str(model2.param.other_result))
).servable()

As you can see, you can avoid the inefficiency by using watch=True and updating named parameters. But this makes your code much longer and much more complicated.

Enabling cache with param.depends should be done carefully as a method can have state information not put into the decorator but available in the class itself, making it a very hard problem.

Caching param.depends is effectively an absolute no-go for that reason. In the case of pn.bind you can at least have a reasonable expectation that the function is only dependent on the state of the arguments and memoize on those, in the case of pn.depends (especially in a class setting) you could memoize on the explicitly stated dependencies but that seems quite implicit and that assumption will break in certain scenarios.

@MarcSkovMadsen can you quantify how inefficient param.depends is?

I mean param.depends itself isn't inefficient, the problem is that it can lead users to write inefficient code. So I'd say it's at least partially a usage and documentation issue. It's perfectly possible to write apps using depends that cache intermediate results on parameters, in fact that's a pattern I use relatively frequently, i.e. instead of doing this:
class Adder(param.Parameterized):
    a = param.Number()
    b = param.Number()

    @param.depends('a', 'b')
    def result(self):
        return self.a+self.b

you write:
```python
class Adder(param.Parameterized):
    a = param.Number()
    b = param.Number()
    result = param.Number()

    @param.depends('a', 'b', watch=True)
    def _calculate(self):
        self.result = self.a+self.b

I mean param.depends itself isn't inefficient, the problem is that it can lead users to write inefficient code. So I'd say it's at least partially a usage and documentation issue. It's perfectly possible to write apps using depends that cache intermediate results on parameters, in fact that's a pattern I use relatively frequently, i.e. instead of doing this:
class Adder(param.Parameterized):
    a = param.Number()
    b = param.Number()

    @param.depends('a', 'b')
    def result(self):
        return self.a+self.b

you write:
class Adder(param.Parameterized):
    a = param.Number()
    b = param.Number()
    result = param.Number()

    @param.depends('a', 'b', watch=True)
    def _calculate(self):
        self.result = self.a+self.b

When using bind on the other hand there is no obvious way to cache the result.

Agree with Philipp. The main problem is that the bound functions are called the same number of times as you depend on them instead of one time.
The examples I provided are the same as Philipp provided. Philipp second example look simple, but if you start having multiple parameters to update it becomes less obvious what goes on. I think its friction that you have to write that extra boiler plate code in every Parameterized class you write instead of just providing a cache=True parameter to param.depends or similar.

I don't buy the argument of having state that is not-depended on. Right now without concrete examples I believe that is an antipattern.
Pro adding caching I would also argue that

There is a huge difference in how parameters and bound functions are treated by Panel. Parameters are displayed using widgets and reactive functions using panes. Thus if a bound function is inefficient and change to a Parameter+watch=True implementation its not only the Parameterized class that needs to change. Its all the places where its used.
Caching is not built into Param. Its built into Panel (pn.cache) thus you cannot actually build the caching your self in the Param world. Making it even harder to make your Parameterized classes efficient and at the same time working with Panel.


Build error with webpack 5

Describe the bug
With version 1.5.1 our builds show the following errors, and these also cause runtime errors.
ERROR in<EMAIL_ADDRESS>1:0-30
Module not found: Error: Can't resolve 'http' in '/Users/username/Downloads/beeplugin-react-starter-master/node_modules/@mailupinc/bee-plugin/dist'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "http": require.resolve("stream-http") }'
        - install 'stream-http'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "http": false }
 @ ./src/components/bee.js 15:17-49
 @ ./src/components/bee_page.js 13:11-27
 @ ./src/index.js 13:16-48

 ERROR in<EMAIL_ADDRESS>2:0-31
Module not found: Error: Can't resolve 'https' in '/Users/username/Downloads/beeplugin-react-starter-master/node_modules/@mailupinc/bee-plugin/dist'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "https": require.resolve("https-browserify") }'
        - install 'https-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "https": false }
 @ ./src/components/bee.js 15:17-49
 @ ./src/components/bee_page.js 13:11-27
 @ ./src/index.js 13:16-48

ERROR in<EMAIL_ADDRESS>4:0-32
Module not found: Error: Can't resolve 'stream' in '/Users/username/Downloads/beeplugin-react-starter-master/node_modules/@mailupinc/bee-plugin/dist'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "stream": require.resolve("stream-browserify") }'
        - install 'stream-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "stream": false }
 @ ./src/components/bee.js 15:17-49
 @ ./src/components/bee_page.js 13:11-27
 @ ./src/index.js 13:16-48

ERROR in<EMAIL_ADDRESS>5:0-32
Module not found: Error: Can't resolve 'assert' in '/Users/username/Downloads/beeplugin-react-starter-master/node_modules/@mailupinc/bee-plugin/dist'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "assert": require.resolve("assert/") }'
        - install 'assert'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "assert": false }
 @ ./src/components/bee.js 15:17-49
 @ ./src/components/bee_page.js 13:11-27
 @ ./src/index.js 13:16-48

ERROR in<EMAIL_ADDRESS>6:0-30
Module not found: Error: Can't resolve 'zlib' in '/Users/username/Downloads/beeplugin-react-starter-master/node_modules/@mailupinc/bee-plugin/dist'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "zlib": require.resolve("browserify-zlib") }'
        - install 'browserify-zlib'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "zlib": false }
 @ ./src/components/bee.js 15:17-49
 @ ./src/components/bee_page.js 13:11-27
 @ ./src/index.js 13:16-48

To Reproduce
Steps to reproduce the behavior:

Clone https://github.com/BEE-Plugin/beeplugin-react-starter
npm install (double-check if the installed version of @mailupinc/bee-plugin is greater than 1.3.8)
npm start
See the above error

Expected behavior
Successful build without errors.
Screenshots
From the BEE-Plugin/beeplugin-react-starter project:

Desktop (please complete the following information):

OS: MacOS Monterey 12.1
Browser: Chrome
Version 97

Additional context
Getting these same errors in both a company project and the BEE-Plugin/beeplugin-react-starter (both use webpack v5, for example, v5.65.0)
In both cases, I managed to fix the build and runtime errors by changing the rollup config to include browser: true, but didn't check deeply so far if this has any negative side-effects.
resolve(
  {
    preferBuiltins: true,
    browser: true, // new
  },
),

Your proposed change looks reasonable.
Until this gets fixed, installing/aliasing the missing modules in your project is another workaround:
npm install assert http@npm:stream-http https@npm:https-browserify stream@npm:stream-browserify url zlib@npm:browserify-zlib
(also works with create-react-app v5 based apps w/out ejecting)


Pytorch Sentiment Analysis with Pretrained Transformers

Topic Suggestion
Pytorch Sentiment Analysis with Pretrained Transformers
Proposed article introduction
Since the birth of Natural Language Processing, sentiment analysis has been a popular task (NLP). Subtask or application of text categorization, feelings or subjective information from distinct texts are extracted and identified in this case.
The Transformers library from Huggingface can be used to implement sentiment analysis rapidly and effectively. To save time, we'll be relying on pre-tuned transformers rather than building our own.
Key takeaways

Build Sentiment Analysis Pipeline
Input Text
Perform Semantic Analysis

Article quality
Using transformers library from Huggingface, we will implement sentiment analysis rapidly and effectively. To save time, we'll be relying on pre-tuned transformers rather than building our own. The code will be simple and easy to grasp.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Good afternoon and thank you for submitting your topic to the EngEd program.
After some careful consideration it struck us that this topic may be a bit over saturated throughout other blog sites and official documentations. We typically refrain from publishing content that is covered widely on the net or other blogs. We're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
But in order to approve the topic it has to serve value to the larger developer community at large. Please feel free to suggest an alternate topic to explore. 🚀
https://www.google.com/search?q=Pytorch+Sentiment+Analysis+with+Pretrained+Transformers&rlz=1C1CHBF_enUS891US891&sourceid=chrome&ie=UTF-8


[BugFix] thirdparty build bump POSIX to 200112L (#31667)

(cherry picked from commit b5de33ab6db242d11520fd0fc235ac7bb46ff000)
Fixes #issue
What type of PR is this:

[X] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[X] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

manual backport for #31667

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)


Docs page

You can create fully responsive docs page for sharing contents of each card under resource section systematically.
You can take reference from Astro Documentaion page here

please explain more on this issue

please explain more on this issue

Something like this

I would like to work on this !

@Astrodevil I want to work on this issue under gssoc'23

there is any particular design you have for the page !! like astro docs design  too much different for your site !!


About the checkpoint

May I ask if you can provide a trained model that I would like to use for reasoning

Please keep attention to our repository.
I'll clean up my workspace after eccv deadline and cvpr camera-ready preparation.
The checkpoints will be uploaded to the google drive either.


🐛 Bug Report: Plugin API Docs AsyncAPI examples not rendering

📜 Description
When viewing AsyncAPI definition (rendered), examples are empty. This disappears when refresh is triggered directly at the /definition path (but not if overview is refreshed). The examples are currently returned on page load and are visible in Raw format. When 'fixed' by single refresh, navigating away and back to definition, even just to overview and back, the problem manifests again.
👍 Expected behavior
Examples shouldn't need refresh of definition page
👎 Actual Behavior with Screenshots
Before refresh:

After refresh:

👟 Reproduction steps
This behavior doesn't happen on https://demo.backstage.io/, so it's likely a local issue, caused by an unknown element, which I marked as ?? :)

??
Navigate to API explorer
Choose an AsyncAPI entity
Click on definition
Try to view an example

edit: I've had it happen once on demo app so far, but I'm unable to reproduce it 🤔
edit2:  we found a reliable way to reproduce this
1.. Go to any OpenAPI definition and click on example (e.g. /pets)
2. Go to any AsyncAPI definition and click on example -> missing
3. Refresh site -> fixed
4. Repeat 1
📃 Provide the context for the Bug.
No response
🖥 Your Environment
Browser tested: Brave, Safari (and reports from multiple users, likely all running chromium based browsers)
OS:   Darwin 21.6.0 - darwin/arm64
node: v16.17.1
yarn: 1.22.19
cli:  0.22.2 (installed)
backstage:  1.11.0

Dependencies:
  @backstage/app-defaults                          1.2.0
  @backstage/backend-app-api                       0.4.0
  @backstage/backend-common                        0.18.2
  @backstage/backend-dev-utils                     0.1.0
  @backstage/backend-plugin-api                    0.4.0
  @backstage/backend-tasks                         0.4.3
  @backstage/catalog-client                        1.3.1
  @backstage/catalog-model                         1.2.0
  @backstage/cli-common                            0.1.11
  @backstage/cli                                   0.22.2
  @backstage/config-loader                         1.1.8
  @backstage/config                                1.0.6
  @backstage/core-app-api                          1.5.0
  @backstage/core-components                       0.12.4
  @backstage/core-plugin-api                       1.4.0
  @backstage/errors                                1.1.4
  @backstage/eslint-plugin                         0.1.0
  @backstage/integration-aws-node                  0.1.1
  @backstage/integration-react                     1.1.10
  @backstage/integration                           1.4.2
  @backstage/plugin-api-docs                       0.9.0
  @backstage/plugin-app-backend                    0.3.42
  @backstage/plugin-auth-backend                   0.18.0
  @backstage/plugin-auth-node                      0.2.11
  @backstage/plugin-catalog-backend-module-gitlab  0.1.13
  @backstage/plugin-catalog-backend-module-ldap    0.5.9
  @backstage/plugin-catalog-backend                1.7.2
  @backstage/plugin-catalog-common                 1.0.11
  @backstage/plugin-catalog-graph                  0.2.27
  @backstage/plugin-catalog-import                 0.9.5
  @backstage/plugin-catalog-node                   1.3.3
  @backstage/plugin-catalog-react                  1.3.0
  @backstage/plugin-catalog                        1.8.0
  @backstage/plugin-entity-feedback-backend        0.1.0
  @backstage/plugin-entity-feedback-common         0.1.0
  @backstage/plugin-entity-feedback                0.1.0
  @backstage/plugin-github-actions                 0.5.15
  @backstage/plugin-home                           0.4.31
  @backstage/plugin-kubernetes-backend             0.9.3
  @backstage/plugin-kubernetes-common              0.6.0
  @backstage/plugin-kubernetes                     0.7.8
  @backstage/plugin-org                            0.6.5
  @backstage/plugin-permission-common              0.7.3
  @backstage/plugin-permission-node                0.7.5
  @backstage/plugin-permission-react               0.4.10
  @backstage/plugin-proxy-backend                  0.2.36
  @backstage/plugin-scaffolder-backend             1.11.0
  @backstage/plugin-scaffolder-common              1.2.5
  @backstage/plugin-scaffolder-node                0.1.0
  @backstage/plugin-scaffolder-react               1.1.0
  @backstage/plugin-scaffolder                     1.11.0
  @backstage/plugin-search-backend-module-pg       0.5.3
  @backstage/plugin-search-backend-node            1.1.3
  @backstage/plugin-search-backend                 1.2.3
  @backstage/plugin-search-common                  1.2.1
  @backstage/plugin-search-react                   1.5.0
  @backstage/plugin-search                         1.1.0
  @backstage/plugin-tech-radar                     0.6.1
  @backstage/plugin-techdocs-backend               1.5.3
  @backstage/plugin-techdocs-module-addons-contrib 1.0.10
  @backstage/plugin-techdocs-node                  1.5.0
  @backstage/plugin-techdocs-react                 1.1.3
  @backstage/plugin-techdocs                       1.5.0
  @backstage/plugin-user-settings                  0.7.0
  @backstage/release-manifests                     0.0.8
  @backstage/test-utils                            1.2.5
  @backstage/theme                                 0.2.17
  @backstage/types                                 1.0.2
  @backstage/version-bridge                        1.0.3
✨  Done in 0.57s.

👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

Still quite present from what I can tell

I think that this is a duplicate of https://github.com/backstage/backstage/issues/18533. Going to close this as it's being worked on over there instead.


Ms 352 fix cli art pt

Description
Bug fix for ticket raised here https://github.com/aiverify-foundation/moonshot/issues/270
If we use prompt template with attack modules, there will be a pydantic validation error.
Motivation and Context
Bug fix
Type of Change
A bug fix
How to Test

Start Moonshot in CLI: python -m moonshot cli interactive
Start a new session with an endpoint you want to use with new_session
Set a prompt template in that session: use_prompt_template real-toxicity-prompt-template1
Run attack module: run_attack_module toxic_sentence_generator "this is my prompt". There shouldn't be a pydantic validation error when the attack module runs

Checklist
Please check all the boxes that apply to this pull request using "x":

[x]  I have tested the changes locally and verified that they work as expected.
[ ]  I have added or updated the necessary documentation (README, API docs, etc.).
[ ]  I have added appropriate unit tests or functional tests for the changes made.
[x]  I have followed the project's coding conventions and style guidelines.
[x]  I have rebased my branch onto the latest commit of the main branch.
[ ]  I have squashed or reorganized my commits into logical units.
[ ]  I have added any necessary dependencies or packages to the project's build configuration.
[x]  I have performed a self-review of my own code.
[x]  I have read, understood and agree to the Developer Certificate of Origin below, which this project utilises.

Screenshots (if applicable)

Additional Notes

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
   have the right to submit it under the open source license
   indicated in the file; or

(b) The contribution is based upon previous work that, to the best
   of my knowledge, is covered under an appropriate open source
   license and I have the right under that license to submit that
   work with modifications, whether created in whole or in part
   by me, under the same open source license (unless I am
   permitted to submit under a different license), as indicated
   in the file; or

(c) The contribution was provided directly to me by some other
   person who certified (a), (b) or (c) and I have not modified
   it.

(d) I understand and agree that this project and the contribution
   are public and that a record of the contribution (including all
   personal information I submit with it, including my sign-off) is
   maintained indefinitely and may be redistributed consistent with
   this project or the open source license(s) involved.

Tested Fix. Working as expected.
https://github.com/user-attachments/assets/5d39489a-4e9f-4091-b64d-cc6e2a7c0967

Link to Jira Ticket: https://imda-dsl.atlassian.net/browse/MS-352?atlOrigin=eyJpIjoiMTVjZjMxM2I4ZGI3NGM2ZGJlNGRjNGZmMTQ4N2Q4ZTAiLCJwIjoiaiJ9

Tested. Working as expected.
https://github.com/user-attachments/assets/9eda1c98-41f4-40a0-af09-bd86d52c9b73


Add filter capabilities to orienteer tables

Implemented in related commits with this merge: https://github.com/OrienteerBAP/Orienteer/commit/1fda0be3ea2014390c59427b670dc1f2ea2f7381


optimization: Agent 管理「业务拓扑」支持拖动改变宽度(closed #470)

Codecov Report

Merging #811 (a4d6a3c) into V2.2.16-rc1 (94c8007) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           V2.2.16-rc1     #811      +/-   ##
===============================================
- Coverage        71.05%   71.03%   -0.02%
===============================================
  Files              294      294
  Lines            21596    21596
===============================================
- Hits             15345    15341       -4
- Misses            6251     6255       +4

Impacted Files
Coverage Δ

apps/backend/subscription/steps/plugin.py
47.48% <0.00%> (-0.56%)
:arrow_down:

...nents/collections/agent_new/choose_access_point.py
95.12% <0.00%> (-0.49%)
:arrow_down:

Continue to review full report at Codecov.

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


If you create an MD5 on Main thread you're gonna have a bad time

When we do execute we do md5 for caching on background thread. When we do enqueue we also should do in background

After looking into the code this must be done within #280, so closing this one.


[FLINK-34441] Add guide to submit flink SQL scripts via the operator (using flink-sql-runner-example)

What is the purpose of the change
This pull request adds documentation about flink-sql-runner-example in the Kubernetes operator documentation page. Majority of the content is copied over from the flink-sql-runner-example's readme.
Brief change log

Added guide to submit flink SQL scripts via the kubernetes operator (using flink-sql-runner-example)

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
I don't know if we have tests covering the docs project. I ran the project locally and made sure that my changes are being rendered properly.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@mxm Here is a PR for the discussion we were having at PR #596 . Please let me know if any updates/improvements are needed. Thank you!


Fix bug on window notification propagation

Fix bug on window event propagation.
related: #47176 #51729

Thanks! And congrats for your first merged Godot contribution :tada:


Don't submit login info as url parameter

[x] Check if the redmine-net-api can submit the api key as header
[ ] Use username password if the api key can not be submitted as header

At the moment redmine-net-api can not submit the api key as a header. So i will change the redmine client to use username and password.


Don't give an error for 1.0/0 and 0.0/0

Closes https://github.com/jyn514/rcc/issues/442
I also found https://github.com/rust-lang/rust/issues/72411 while working on this, so that was fun.

I think a better solution would be to modify store_static to use a known bitpattern for NaN instead of blindly trusting to_bits.

Delaying the above ^ until someone complains, since it's not defined by any standard.


only use it on 2D size

I want to use the model GraphMVP using a 2D dataset to pretrain(classfication task ) , how can I do that?In the repo I can use your code such as GraphLoG to run my baseline

Hi @squarewhiteflag , do you mean only 2D topology is available?

Yes.My understanding is that graphMVP uses 3D topology for pre-training and then finetune downstream tasks without 3D topology. The dataset used is 3D. But I only use 2D datasets for pre-training, so I modify the loss of the code to make the code run, and fine-tune it in downstream task for testing.It seems reasonable for the data I got from downstream task. Is this the right thing to do? Sorry for the late reply

Hi @squarewhiteflag,
For GraphMVP, it supports:

pretraining on 2D & 3D
downstream on 2D or 3D

We didn't try 3D downstream because of the insufficient 3D pretraining dataset. Now we have added them in Geom3D

If you only have a 2D dataset for pretraining, the objective functions proposed in GraphMVP (i.e., EBM-NCE and VRR) still hold, and only the application setting changes.

I got it.Thank you very much


update psysh to v0.8 on v5.1

Laravel Version: 5.1.*
PHP Version:all

Description:
many package require nikic/php-parser v3 and on v5.1 we cant install pacakge that need it
v0.8 of psy/psysh support v3 (v5.3 allready support v0.8)

many package require nikic/php-parser v3

Packages should attempt to support multiple versions of php parser instead of just mindlessly upgrading.

v0.8 of psy/psysh support v3 (v5.3 allready support v0.8)

Yeh, though that's technically a new feature, and 5.1 is "LTS". We could technically support it I guess, but we don't want people complaining that stuff broke.

psysh is used for tinker so used in dev environment i think there is no problem in production environment

psysh is used for tinker so used in dev environment i think there is no problem in production environment

But niche developer might have other requirement to use psysh in their project, which still could cause issues.

I wouldn't recommend starting a project on Laravel 5.1 LTS. It's designed to give slow moving apps bug fixes and security assurance. It's not designed for people to be moving quickly with brand new packages. For these reasons, it's important that we do not brake the 5.1 branch with unneeded upgrades.

Disclaimer, these words are my own, and do not necessarily reflect the views of Taylor.


Add word "Triceps"

Add all the definitions of Triceps in a file named Triceps.json in folder T

Word triceps added for PR


backend worker dies

Hi, i have the same issue of an user here.
i opened this one in pytorch/serve
https://github.com/pytorch/serve/issues/991
thanks
Dario

@hellodario can you please help me out. i'm still not able to run the code


refactor route logic to reuse code

carved out from https://github.com/istio/istio/pull/34247 to simplify review. No functional changes.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[X] Does not have any user-facing changes.  Once this is merged, I will rebase https://github.com/istio/istio/pull/34247 so that we can review the merge logic.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
a2ce0b6f32784cb7598ff5e99666ceae94978b3a
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-multicluster-tests_istio

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
a2ce0b6f32784cb7598ff5e99666ceae94978b3a
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
a2ce0b6f32784cb7598ff5e99666ceae94978b3a
link
/test release-notes_istio

integ-pilot-multicluster-tests_istio
a2ce0b6f32784cb7598ff5e99666ceae94978b3a
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Define best practices, how to distribute the components

Motivation
Currently, there is no easy way, how to distribute the component as a nuget package.
Because as a part of the DLL is also the site cs files, that should not be there.
Proposed solution
Use the project profile condition to include just the files, that are required to be in the component package.

This issue relates to a solution that has been completely redesigned.
At the moment there is no issue with including unwanted C# files into the component library because the component files are kept separately from the sandbox sample site


Want to recieve only part of rss feeds

I want to receive only '[CDATA[National]] category'  feeds from http://www.thehindu.com/news/national/?service=rss . Is it possible by making my own book? I assume keep_only_tags in base file could be used for the same but have no idea of HTML. Can anybody pls guide for this please. Thanx in advance!

Yes, you can.
Create a subclass of BaseFeedBook, then copy and modify the function ParseFeedUrls(), add some statements for example:
if hasattr(e, 'category') and e.category != 'National':
     continue

Thanks @cdhigh for the wonderfull program and quick guidance. I added following in the basefile and it's working fine.
if url == 'http://www.thehindu.com/news/national/?service=rss':           if hasattr(e, 'category') and e.category != 'National':                             continue


Dynamic logger F# project fails to build

To reproduce:
$ dotnet new steeltoe-webapi --language 'F#' --logging-dynamic-logger
$ dotnet build
...
...Program.fs(27,14): error FS0039: The type 'IHostBuilder' does not define the field, constructor or member 'UseCloudHosting'. Maybe you want one of the following:   UseConsoleLifetime ...

Similar/same error for --management-endpoints

Looks like this bug was fixed


Website not reachable via IPv6

The 11ty.dev website is not reachable via IPv6, www.11ty.dev doesn't seem to have an AAAA record set. This makes it impossible to access the site from IPv6-only networks and on dual stack it will always connect using legacy IPv4

I'm not sure, but I think the 11ty website is now hosted on Vercel (at least PR previews are) and Vercel has not IPv6 support.
I didn't even know this before today, since I just assumed that noone would choose such a service nowadays with only legacy IP support...

Whoa—I wasn’t aware of this limitation.
We are currently diversifying our website hosting—at this very moment all of the versioned subdomains https://www.11ty.dev/docs/versions/ are on Vercel, one API service on AWS (via Vercel rewrite) https://www.zachleat.com/web/serverless-cost/ but www is still on Netlify with some plans to move to Vercel.


Doesn't alleviate performance

Installing the agent yielded no network performance improvements for me. It turns out that PHP is not issuing a fire-and-forget and instead ends up waiting for the response from the agent but it could be something else, I am not sure.
Hopefully I am missing something obvious, any guidance would be appreciated.
The instrumentation is correct and the agent does log the incoming requests and they do show up on the bugsnag web app as well.

After some poking around, it turns out this is because of curl behaviour.
For requests over 1024 bytes, curl automatically adds Expect: 100-continue which trips up the agent and the response is extremely slow.
On the client-side this can be overcome by forcing a curl request over HTTP/1.0 in the bugsnag-php lib which fixed things up. Or setting an empty header "Except:".

In our case, Bugsnag does not work either with a huge amount of data (example 729k of json in the body). It also gets lost and error never reach the Bugsnag server, or reaches it incorrectly cause it is never shown in the interface. Same test with a smaller json works just fine.
We are using bugsnag with nodejs.
Can we change the title of this issue to something like:
"Bugsnag gets lost when request is big"

We've fixed this issue with the PHP notifier regarding the expects header.

We are using bugsnag-node and it has this problem as well
github.com/bugsnag/bugsnag-node

@rafaelspecta maybe open a new issue?


Log4j error logs during prime cli commands

Describe the bug
Log4j spits out a series of error logs when running any prime cli command. This does not affect functionality of the prime cli but is worrisome to engineers running commands against production.
Impact on ReportStream
It is noisy and annoying when running any prime cli command.
Steps to reproduce
Steps to reproduce the behavior:

run the following

./prime fhirpath --help

Expected behavior
No error logs when a prime cli command is run.
Resources

Hey team! Please add your planning poker estimate with Zenhub @adegolier @arnejduranovic @brick-green @jack-h-wang @jalbinson @JFisk42 @mkalish @thetaurean

Here's where the format is defined: https://github.com/CDCgov/prime-reportstream/blob/6edf2ef3068e145d0b3798a7d27533c33ab89e42/prime-router/src/main/resources/jsonTemplate.json

prime-router-dependency-output.txt
I've run this to ground enough to where I'm pretty sure I know what's going on. However I'm confused by at least one thing.
The simple statement is that there appears to be an interaction between the shade plugin and multiple versions of log4j such that something gets shaded in a way that nerfs the xformer's ability to create the json logging output.
see
https://stackoverflow.com/questions/75838785/console-contains-an-invalid-element-or-attribute-jsontemplatelayout-even-after/76681723#76681723
https://stackoverflow.com/questions/71458136/log4j-jsontemplatelayout-doesnt-work-when-jar-file-built-by-shadowjar
https://stackoverflow.com/questions/44480197/log4j2-error-unrecognized-format-specifier-t

follow up as I'm the kind of guy who can't resist a good puzzle.

it hit me that if org.apache.log4j.* is included in the final uber jar but is not being picked up by the build system it's either got something to do with the shading or something we're pulling in has a local source copy that's namespaced as org.apache.log4j.* .

I did some digging and I'm almost certain it's a stripped down version of log4j 1.x. I quick-compared the vendor source of log4j1 against what's in the uber jar and the former appears to be a superset of what's in the uber jar.

I looked at the CVEs for log4j1 because I know this library has a history for critical security defects.  Here is another breakdown of the more recent ones. I'm not seeing anything in the part of log4j1 we have that matches with anything severe in the CVE list.

My hypothesis is that the presence of this second instance of log4j is somehow screwing with the shading plugin in the manner described in the SO links in the post above. We might be able to use the transformer plugin method (also in the SO links in my last post) and be done with it. That - if it works - will resolve the formatting error messages.
We can also try to exclude the org.apache.log4j.* package from being included but that might be tricky if the build system doesn't see it as a dependency. The shading plugin takes arguments. There might be something there.
It seems relatively harmless to have in there but I have to admit given the history of issues with that library and the fact that new issues continue to be discovered it would make me feel better if we could just get rid of it either now or at some future point.

turns out the Transformer suggested by the SO folks is already baked into the shade plugin.
https://imperceptiblethoughts.com/shadow/changes/#v4-0-0-2018-09-25
and it's not doing what we need it to do. My gut is telling me this is due to the fact that the second instance of log4j (the mini log4j1 one) is not registered with the build system as a dependency is making this complicated.
will try a few other things ...

fixed the issue and a few others to boot!
PR: https://github.com/CDCgov/prime-reportstream/pull/13994
tl'dr - using the baked in transformer ended up working - it just needed to be invoked in a particular manner.


Feature Name
Adding Policy Gradient Methods Visualizations
Feature Description
Develop policy gradient methods that directly optimize the policy by updating it toward higher expected rewards.
Motivation
Policy gradient methods are vital for solving reinforcement learning problems with continuous action spaces. Adding this feature will enhance the flexibility of the algorithms available.
Implementation Suggestions (Optional)
No response
Feature Type
New Algorithm
Does this feature require additional resources?

[X] Additional Documentation
[ ] Third-Party Libraries
[ ] UI/UX Design
[ ] Other

References (Optional)
No response

please assign this issue to me @ajay-dhangar


Add VGGFace2 format support

Summary

Added support for VGGFace2 format
How to test

Checklist

[x] I submit my changes into the develop branch
[x] I have added description of my changes into CHANGELOG
[x] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[ ] I have linked related issues)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

Yes, I think it is. But the official website of this dataset is not available and I don’t know how to get more information.


profileManager.get(true) randomly throws java.util.concurrent.TimeoutException

Context: play 2.6.3,  "org.pac4j" % "play-pac4j" % "4.0.0-SNAPSHOT" and pac4jVersion = "2.1.0".
I occasionally get java.util.concurrent.TimeoutException thrown by profileManager.get(readFromSession = true) (it's hanging/blocking for a few seconds where it shouldn't, and throws the exception afterwards).
this code is called from a controller action (basically based on example in https://github.com/pac4j/play-pac4j-scala-demo/blob/0e2a7fa80e5e7cd177d72b90a707174f04eb4fe0/app/controllers/ApplicationWithFilter.scala#L22-L27 )
  def getProfile(implicit request: RequestHeader) = {
    import scala.collection.JavaConverters._
    val webContext = new PlayWebContext(request, playSessionStore)
    val profileManager = new ProfileManager[CommonProfile](webContext)
    val profile = profileManager.get(true) // randomly throws java.util.concurrent.TimeoutException
    profile
  }

I use:
// in SecurityModule.configure:
bind(classOf[PlaySessionStore]).to(classOf[PlayCacheSessionStore])

// and in build.sbt
libraryDependencies += ehcache

https://gist.github.com/vidma/a7c2154e15f74285c77b48508a78f066

cc @leleuj

Do you have some specific configuration for EhCache? I mean: is it shared between nodes or something which could explain the hanging...

Also any idea what your load is? The get could be doing blocking i/o under the hood, if distributed you might be blocking on a network call. Under load this would escalate very quickly if you have a limited threadpool for play to work on. I'm not sure if play assumes non-blocking i/o or not though, so this is conjecture.

it's default play cache config (i.e. no ehcache config).
I get the error on my laptop with sbt run, with only a single request at once...

Have you already encountered this problem with the 4.0.0-SNAPSHOT or is it a very recent issue?

I tried 4.0.0-RC1 which never worked (was always throwing this error or a similar one), and 4.0.0-SNAPSHOT works 4/5 times (since yesterday or the day before).
just in case FYI, here's an excerpt from my build.sbt:
scalaVersion := "2.12.2"
val playVersion = "2.6.3"
val pac4jVersion = "2.1.0"
// see https://github.com/pac4j/play-pac4j-scala-demo/blob/20ccf821bc557347ca2e555fb1c85d4afea92366/build.sbt#L9-L29
val pac4jSecurityDeps = Seq(
  "org.pac4j" % "play-pac4j" % "4.0.0-SNAPSHOT" excludeAll ExclusionRule("org.pac4j", "pac4j-core") changing(),
  "org.pac4j" % "pac4j"  % pac4jVersion changing(),
  "com.typesafe.play" %% "play-cache" % playVersion,
  "org.pac4j" % "pac4j-core" % pac4jVersion changing(),
  //   "org.pac4j" % "pac4j-sql" % "2.1.0",
  "org.pac4j" % "pac4j-http" % pac4jVersion changing(),
  // needed for auth session storage
  ehcache
)
libraryDependencies += ehcache

even more weird with play 2.6.5 - it logs out after a few visits.
weird enough, downgrading to play 2.6.2 - seems to work so far. at least I didn't notice issues yet over ~20 visits.
does it have anything to do with how play-pac4j is built ?

same issue here - play 2.6.5, after successful login, one request isAuthenticated true, all after that all next requests got false, session killed, no profiles retrieved from session. Easy to replicate, just need to upgrade play framework with basic/default dependencies and configuration.

also can confirm:
play 2.6.2 - seems working
play 2.6.3 - timeoutexception
play 2.6.5 - session killed after first request after authentication

OK. Very strange that a minor upgrade of the Play framework breaks things so strongly. Will do some tests.

Even stranger, I cannot reproduce the issue with both demos (Java and Scala). I updated them to Play 2.6.5 with success: @vidma @aleksandrmelnyk can you confirm you have the issue with at least one of the two demos?

Demos: https://github.com/pac4j/play-pac4j-scala-demo and https://github.com/pac4j/play-pac4j-java-demo

weird, but with the scala demo could not replicate on 2.6.5. Will debug later more.

@aleksandrmelnyk Thanks. Weird indeed, but at least, it's a clue to find out the cause of the issue.

Does it depend what session storage is being used?

@millross I don't think so, as we all use EhCache (as defined in the dependencies)

@leleuj to replicate on 2.6.3 you need to visit the page which reads the profile from session (profileManager.get(true), expecting timeoutexception ).
tried 2.6.5 and 2.6.3 demo, however unable to replicate so far (but had seen the issue on our app). I have no time to dig this deeper now.

maybe it depends on how/where is the session accessed in the controller or is the anonymous client enabled (we had only FormClient, and used URL-based security filter rules).
P.S. I also used
@Singleton SomeController @Inject()(..., cc: ControllerComponents)
  extends AbstractController(cc) with I18nSupport {
  ...
}

instead of more limiting extends Security[CommonProfile].

@leleuj Actually I just noticed that you didn't upgrade the scala demo project. Please, in project/plugins.sbt, change
addSbtPlugin("com.typesafe.play" % "sbt-plugin" % "2.6.2")
to
addSbtPlugin("com.typesafe.play" % "sbt-plugin" % "2.6.5")

Good catch! I get the errors now. I will investigate further...

With Play 2.6.3, I encounter the TimeouException, it seems that the problem comes from the Play cache implementation. I read several comments about that and found: https://github.com/playframework/playframework/pull/7802
@gmethvin is this really fixed in Play 2.6.5?
With Play 2.6.5, the authentication works but the profile does not seem to be kept in the cache more than a few milliseconds.
Has something changed with the timeout definition?

OK. I debugged into the cache and indeed, the timeout has changed from seconds to milliseconds between 2.6.2 and 2.6.5. This explains the fact the profile is not saved long enough in the cache and is expired at the second call...
I will certainly change the default timeout in play-pac4j!


Cannot not get options from webpack config

webpack 2
For some reason, I canno't use postcss.config.js, so I put config in loader options
{
  loader: 'postcss-loader',
  options: {
    plugins: [
      require('autoprefixer')({
        browsers: [ 'iOS >= 5.1' ]
      }),
      require('postcss-pxtorem')({
        rootValue: 46.875,
        propWhiteList: []
      })
    ]
  }
}

But the following error occus:
 ERROR in ./~/css-loader?{"sourceMap":true}!./~/postcss-loader?{"plugins":[null,null]}!./~/sass-loader?{"sourceMap":true}!./app/components/Coupons/couponbag/_couponbag.scss
    Module build failed: TypeError: Cannot read property 'postcss' of null

webpack version please 😛 if you're webpack <= 2.2.1. That's a known bug with 'Complex Options' in loader options like using {Function}/require(). 2.2.1 fixed this and if you can't upgrade (highly recommended) then try this
webpack.config.js
{
  loader: 'postcss-loader',
  options: {
    ident: 'postcss' // <=
    plugins: [
      require('autoprefixer')({
        browsers: [ 'iOS >= 5.1' ]
      }),
      require('postcss-pxtorem')({
        rootValue: 46.875,
        propWhiteList: []
      })
    ]
  }
}

Also a fresh rm -rf node_modules && npm i can do wonders 😄

If you still have any regressions feel free to reopen 😛

Update webpack from 2.2.0  to 2.3 fixed this problem.
Thanks for your help.😄


reg-keygen-git-hash-plugin timeout

reg-keygen-git-hash-plugin hangs up(timeout on circleCI) after warning message. (e.g. warning: ignoring origin/xxxx; cannot handle more than 25 refs).

Closed via #85 .


feat(bmr): add bsc mock tests (#522)

Closes #522
Checklist:

[x] I have performed a self-review of my own code
[ ] I have documented my code in accordance with the documentation guidelines
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have run the unit tests
[x] I only have one commit (if not, squash them into one commit).
[x] I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Unit tests that made rpc calls have been removed.
This has reduced code coverage percentage.

@bbist @pragyanshrestha-ibriz
pending review.


Predefine searchFor

Hi, is it possible that i predefine the search filter for the first time i call the vuetable component? i try to use :search-for="searchvalue" but nothing happen. Meaning i would like the vuetable load with the filtering result.

@warrence Hmm, I haven't thought about that, but a very valid use case. Anyway, it could be done but let me check the code and get back to you on this later.

Thanks for your time.

@warrence Just notice that you're using search-for which does not exist for vuetable, so, there's no way that's going to work. Instead, you should use append-params prop and the predefined filter (search) is already working.
So, if your API backend already supports filter, you could do like this
  <vuetable
    fields="..."
    api-url="..."
    :append-params="['filter=xyz']"
  >

@ratiw i tried your way but is not working, however i get it working in my own way but I'm not sure is this a right way, but it works at least for now.
i grab my url parameter as below in js
`var urlParams = new URLSearchParams(window.location.search);
var filter = urlParams.get('filter');
if(filter == null){
filter = '';
}`
i initiate the searchFor as below:
data: {             searchFor: filter,
and then i add in the ready:
ready(){         if(filter != ''){             this.setFilter()         }     },
I would hope there is a better way for this

@warrence That would work too. (I assume you're using the code in the examples folder as a basis)
I think you should be able to move the first part of your code (grabbing the filter from the url) inside the ready() block as well.
Just curious, why do you need to grab the filter from the browser url? You do not know this in advance?

@ratiw i have a redirection from another page to this vuetable page, and i was redirect with a filter, i wan the vuetable page open with a filtered result initially. Lets take a look at a very simple example which is easier to explain than my actual scenario.

Product Item Listing Page
Use vuetable for listing all product ordered by recently added, with search. One of the searchable column is Product Category

Then i might have page that explain about a category of a products and i wan to have a link to bring user to Product Item Listing Page like /products-listing?filter=category_name, to show user what kind of product i have under this category.


PyTorch HuggingFace model not using GPU

This is the model code:

This is the PyTorch Dataset class code:

This is the Predictions code:

And this is the task manager:

My Gpu works on the tutorials of the PyTorch from youtube, this code has been in the same environment and it still does not work.
Any help would be appreciated please and thank you.

did you convert your variables and model device to cuda?did you ever used cuda in your codes?are you sure you config using pytorch from gpu corectly?

Make sure you are always using NUM_WORKERS=0. In Windows only that value is the one that works

Hi @Kashish-1426 ,
I've not had this issue before. I use computer vision models from Hugging Face quite often and they work excellent on the GPU.
I also use very similar code to you, e.g. model.to(device), just like in the PyTorch course.
Have you tried reaching out to anyone from Hugging Face?

@mrdbourke
I think He is not referring to the deployment in Huggingface. From what you can see in the images, he is referring to the code he wants to implement, but from what you can see he is executing it locally in Windows and in Windows NUM_WORKERS = 0 is the only value it supports. I'm almost sure that's the problem because I had the same problem.


It is time

Ready for liftoff.

it truly is time

its incredible yall deserve a hot chocolate fr


Test result aggregation for specific build variant

Currently, testAggregatedReport will run unit tests with all build variants, but I want to run only specific build variants. Can you provide more options in Gradle to support this requirement?
For example, as in your demo project, I want testAggregatedReport to execute the unit test for stageDebug or debug.

Hi there, thanks for reporting this.
The DSL mean for this use case was the one described at aggregateTestCoverage in the README.ME.
But I totally forgot about it when I extended this plugin to support Test Results as well, it's added at #68

Released at 2.1.0

Thank you so much


Create HetznerUbuntuSetup.md

Step-by-step guide for someone who is starting from scratch on this project with a bare server.
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.

[ ] Documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration:

Test Configuration:

Checklist:

[ x] My code follows the style guidelines of this project

Made changes. closing and reoopening with new updates.


Add EiskaltDC++.app v2.3.0 Built on Qt 5.5.0

based on v2.2.10
alternate version based on newest Qt framework
I didn't get how should appcast stanza work and how can I evaluate it's sha256 so I've skipped this stanza

Is there an appcast for this? I.e. a file online that it uses to check for updates? If, so, post the link and we’ll help you make the stanza.
Either way, this belongs in caskroom/versions.

What exactly are the differences between this and eiskaltdcpp? One uses qt and the other not? This seems to be a newer version.

I.e. a file online that it uses to check for updates?

there is a usual sourceforge rss for project. I think it can be used as appcast http://sourceforge.net/projects/eiskaltdcpp/rss?path=/Mac OS X

What exactly are the differences between this and eiskaltdcpp? One uses qt and the other not? This seems to be a newer version.

I't s based on the same tag version. But it contains small changes (because of moving to github infrastructure I think) and new major version of Qt, there is a version with Qt4 (this one EiskaltDC++-2.3.0-v2.2.10-62-ge42c04e-x86_64-qt4.dmg is tight candidate for eiskaltdcpp formuls)

After closing PR new commits don't be added. Should I create new one with additional new commit?

there is a usual sourceforge rss for project. I think it can be used as appcast http://sourceforge.net/projects/eiskaltdcpp/rss?path=/Mac OS X

In theory it could, yes. However, it unfortunately seems sourceforge casks won’t do.
This is how this one could be:
appcast 'http://sourceforge.net/projects/eiskaltdcpp/rss?path=/Mac%20OS%20X',
        :sha256 => 'c8f46da22da07f8ec68159b6f9a5a996e89669dcaf8dd0e5cdc3a2461506e4fe'

To get the :sha256 value, simply curl 'http://sourceforge.net/projects/eiskaltdcpp/rss?path=/Mac%20OS%20X' | shasum -a 256. However, sourceforge appcast have a <pubDate> value that changes depending on when the request was made, so we can’t actually track them.

After closing PR new commits don't be added. Should I create new one with additional new commit?

Looking further into it, it seems the version we now have is old compared to the one you submitted. Submit a new PR, yes, but instead of adding a new cask, simply submit that new url to the existing eiskaltdcpp cask.

Submit a new PR, yes, but instead of adding a new cask, simply submit that new url to the existing eiskaltdcpp cask.

You may miss my comment about two build versions, default - qt4 and my cask with qt5
File: EiskaltDC++-2.3.0-v2.2.10-62-ge42c04e-x86_64-qt5.dmg
Description: EiskaltDC++ from git master for Mac OS X 10.9 64-bit
Comment: Built on Mac OS X 10.9 with Qt 5.5.0

File: EiskaltDC++-2.3.0-v2.2.10-62-ge42c04e-x86_64-qt4.dmg
Description: EiskaltDC++ from git master for Mac OS X 10.9 64-bit
Comment: Built on Mac OS X 10.9 with Qt 4.8.7

Should I leave only one?

Ah, apologies, misinterpreted that. Yes, leave only the qt-5 one. Is there any reason a user would pick the other one?


Add a factory constructor for customizing snap behavior

If we want to add an additional snap position of the viewport offset, we would typically write code as follows, but it's a bit verbose.
const peekOffset = ViewportOffset.fractional(0.5);
controller = ExprollablePageController(
  initialViewportOffset: peekOffset,
  maxViewportOffset: peekOffset,
  snapViewportOffsets: [
    ViewportOffset.expanded,
    ViewportOffset.shrunk,
    peekOffset,
  ],
);

It would be nice to have a factory constructor of ExprollablePageController for simplicity like:
const peekOffset = ViewportOffset.fractional(0.5);
controller = ExprollablePageController.withAdditionalSnapOffsets(
  initialViewportOffset: peekOffset,
  additionalSnapViewportOffsets: [ peekOffset ],
);

Please close this issue as it has been implemented and will be released in the next release.


Meteor: icon fonts not working (Resource interpreted as Font but transferred with MIME type text/html)

After switching from nooitaf:semantic-ui to semantic:ui-css, icon fonts are not working.
The following errors now appear in the browser console:
Resource interpreted as Font but transferred with MIME type text/html: "http://localhost:3000/packages/semantic_ui-css/themes/default/assets/fonts/icons.woff".
Resource interpreted as Font but transferred with MIME type text/html: "http://localhost:3000/packages/semantic_ui-css/themes/default/assets/fonts/icons.ttf".```

This issue also seems to have come up in<EMAIL_ADDRESS>and was fixed in<EMAIL_ADDRESS> https://github.com/nooitaf/meteor-semantic-ui/issues/19

Oops. Upgrading to<EMAIL_ADDRESS>fixed this problem.

Also I seem to have filed this bug against the wrong project.... :P

Cool because I really confused!
On Fri, Feb 27, 2015, 9:37 PM ljmatkins<EMAIL_ADDRESS>wrote:

Also I seem to have filed this bug against the wrong project.... :P
—
Reply to this email directly or view it on GitHub
https://github.com/nodegit/nodegit/issues/443#issuecomment-76510125.


scanner/user/scanUser.c : memory leak

The number of memory allocations for message buffer is (requests per thread) * (number of threads)
but each thread frees it just once.

Does anyone solve this problem?


Feat(anvil): Implement GethDebugBuiltInTracerType::PreStateTracer in mined_geth_trace_transaction

Component
Anvil
Describe the feature you would like
I am trying to use PreStateTracer on a mined block of transactions from the anvil eth api backend's debug_trace_transaction, and I kept receiving an empty NoopFrame finally realizing it's unimplemented. It would be useful to have an implementation here for my use case.
I guess an additional question - Is there a simple way to get the pre/post state of a mined transaction from anvil currently? And if so how beyond re-executing the transactions through revm & passing the ResultAndState/db into geth_prestate_traces
Additional context
Ref: https://github.com/foundry-rs/foundry/blob/6de15b0136ccb7dcac412d6b97bb3269ac2361c3/crates/anvil/src/eth/backend/mem/storage.rs#L481
Ref: https://github.com/foundry-rs/foundry/blob/6de15b0136ccb7dcac412d6b97bb3269ac2361c3/crates/anvil/src/eth/backend/mem/mod.rs#L1957

going to track callTracer / prestateTracer with https://github.com/foundry-rs/foundry/issues/5855 and close this one as dupe


Not working

Not working =)
I have ComfyUI installed as extension in Auto1111(portable) and its not working for me, also i have ComfyUiPortable on another folder and its work fine, but integrated as extention doesnt work.

Path of SD(Portable) and extension ComfyUI is D:\stable-diffusion-portable-main\extensions\sd-webui-comfyui - its not working
Path of ComfyUI(standalone portable) D:\ComfyUI_windows_portable\ComfyUI - its working fine, but not extension
If i change standart "ComfyUI install location" to D:\ComfyUI_windows_portable\ or D:\ComfyUI_windows_portable\ComfyUI its still dont working with same issue.

Console writing

Versions

Change path to comfyui from D:\stable-diffusion-portable-main\extensions\sd-webui-comfyui to D:\ComfyUI_windows_portable\ComfyUI

First variant i tried already before and nothing changes.
With second variant ComfyUI tab proposed me to install ComfyUI

so its create new folder D:\stable-diffusion-portable-main\extensions\sd-webui-comfyui\ComfyUI and still doesnt work, no connection

btw if console wrote "ModuleNotFoundError: No module named 'Folder_paths'" now its write

I think this could be the same thing as #149

i think i solved it but dunno is in right way.
So i have SD(portable) --- D:\stable-diffusion-portable-main
ComfyUI(portable) --- D:\ComfyUI_windows_portable\ComfyUI (its work fine)
Installed ComfyUI extension in SD, indicate path in ComfyUI settings to portable version (D:\ComfyUI_windows_portable\ComfyUI).
Changed Address of the ComfyUI server on http://<IP_ADDRESS>:8188.
And its work fine in ComfyUI TAB in SD, but only when both launched (webui-user of SDPortable AND run_nvidia_gpu from ComfyUIPortable). Dunno should run_nvidia_gpu be running every time i want use ComfyUI as tab in SD

This won't work, the extension will not be able to communicate with comfyui like this. Can you try switching the IPC strategy in the settings for "File System"?

In the meantime I'll see if I can fix the code. I'm not sure why the shared memory is already created, probably an edge case.

with this settings still doesnt work when "webui-user.bat" of SDPortable only launched

Are there any messages in the console?

when i removing ComfyUI server adress its never works, when i type http://<IP_ADDRESS>:8188/ its work, but only when "run_nvidia_gpu.bat" launched

Yes, try it again with the "file system" IPC strategy and please share what the console outputs.

with this settings

ComfyUI tab doesnt work in any way until i add http://<IP_ADDRESS>:8188/ as server adress and doesnt matter which ICS (Deafault, Shared or File System)

with --port 8188 in Additional cli still wont work

with which settings you want to see console

if i launch with this

so

I'm really puzzled here, unable to reproduce. I will try to install your portable versions to debug. Could you share the repos you are using?

what is repos ))) sry im not strong in all of it

Sorry, I meant to ask: how do I install the portable versions of the webui and comfyui you are using? are there github repositories somewhere that I can clone or download?

sec

So I tested a complete reinstall against stable-diffusion-portable and after a few hurdles, it is working. Somehow, this version of stable diffusion does not install extension dependencies. The error I got is different from what you ran into, so I'm not sure what's going on. I'll work on a fix with the shared memory by closing the shm if it exists.

Well I'm confused because the shm is already getting closed before trying to open it again. I don't know much about using shared memory and I ran into a core bug of the python library when trying to implement this the first time. If anyone knows how to fix this, I'd be glad to have a hand.
As a side note, something you could try @NewmanMaxx is to install the webui yourself using the official v1.0.0-pre zip:
https://github.com/AUTOMATIC1111/stable-diffusion-webui/releases/tag/v1.0.0-pre
If sd-webui-comfyui works through this other installation, then I assume the problem comes somewhere from the webui you are using. In this case, we may or may not intend to support it, depending on how much help this issue gets and where the issue comes from.

In Stable Diffusion in webui-user.bat there is a line set TEMP=. In Portable Stable Diffusion there is set TEMP=tmp, but if you change set TEMP=temp, the extension will work.

it works

seems like solved atleast for me, would be nice to add that info in ComfyUI description maybe as interaction with SDPortable :3

The main note.
If we want to specify the already installed ComfyUI Settings>ComfyUI>Computer install location in the path as an example: D:\Portable-ComfyUI\ComfyUI, then Load ComfyUI iframes through a reverse proxy (requires reload UI. Needs --api. Default is on if webui is remote): Always. Don't forget to specify set COMMANDLINE_ARGS=--api in webui-user.bat

By the way @alexbofa do you understand what the problem was and why changing the TMP environment variable fixes it? I can't make much sense of it right now. I assume it is related to how shared memory is implemented with respect to the temporary folder?

With a fresh install, there should not be any "ComfyUI" directory anywhere in the repo. Where does it come from in the situation you are describing? We tested this quite a bit if I remember correctly, but let me try to do a fresh install to verify this again.


Use Gradle toolchain instead of if (JavaVersion.current() != JavaVersion.VERSION_11)

https://docs.gradle.org/current/userguide/toolchains.html

@PawelLipski Can I take this issue?

Sure! pls open a PR from your fork to develop branch in this project

Hey @salmansharifov, are you going to work on that?

TBD if it still makes sense now that the build supports both Java 11 and 17 though

Might be revisited when Java 11 support is dropped (#859)

Plugins for IntelliJ 2022.3 must be built on JDK 17 anyway (https://github.com/VirtusLab/git-machete-intellij-plugin/pull/1201)


{Compute} az vmss rolling-upgrade, vm extension image and vm host: Migrate commands to CodeGen V2

Related command

az vmss rolling-upgrade cancel/start/get-latest
az vm extension image list-versions/show/list-names
az vm host list/show/delete/restart
Description

Migrate all commands in az vmss rolling-upgrade group.
Migrate a part of commands of az vm extension image and az vm host group.
Some commands in these groups are not migrated yet:

az vm extension image list

az vm host get-instance-view/create/update

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute


Ensure that directory exists in create_test.py

Providing a path to a non-existent directory caused the script to fail.

Thanks!


Triple nested navigation

Describe your changes
Restructure our navigation system to be triple nested. So now it's Stack -> BottomTabs -> Stack. This is so that we can have screens that won't display a bottom tab.
Issue ticket number and link
#40
Checklist before requesting a review

[x] All code is linted and conforms to style guide
[x] All necessary context is described in the PR or Issue
[x] This branch is code and feature complete. If it is not complete, this PR is a draft.

Note: HeaderMenu branch must be merged before merging this branch


Removing DBSUpload and DBSBuffer

This PR fixes https://github.com/dmwm/WMCore/issues/4949

retest this please

At a minimum I am worried about these two.
Eric

On Feb 12, 2016, at 05:26, DMWM Jenkins<EMAIL_ADDRESS>wrote:
WMComponent_t.DBS3Buffer_t.DBSBufferFile_t.DBSBufferFileTest:testProperties changed from success to error
WMComponent_t.DBS3Buffer_t.DBSUpload_t.DBSUploadTest:testCloseSettingsPerWorkflow changed from success to failure

So the first of these:

WMComponent_t.DBS3Buffer_t.DBSBufferFile_t.DBSBufferFileTest:testProperties changed from success to error

comes from:
WMComponent_t.DBS3Buffer_t.DBSBufferFile_t.py
where it is clearly using dbs2 daoFactory2 defined:
https://github.com/dmwm/WMCore/blob/master/test/python/WMComponent_t/DBS3Buffer_t/DBSBufferFile_t.py#L39
So the problem here is that there is no dbs3 equivalent of  LoadInfoFromDAS.py in MySQL. Do we therefore want to keep this test if it was originally designed to test dbs2 functionality that we have now removed? or is there an equivalent dbs2 sql in a different filename?

test this please


amdkfd driver not loaded

Hi,
Using a  fresh install of Ubuntu 16.04 on a Dell Latitude e6540 with a HD Radeon 8790M. Installation goes fine, but running /opt/rocm/bin/rocminfo returns the famous

hsa api call failure at line 900, file: /home/jenkins/jenkins-root/workspace/compute-rocm-rel-1.8/rocminfo/rocminfo.cc. Call returned 4104

So I started troubleshooting. /opt/rocm/opencl/bin/x86_64/clinfo produces

Number of platforms:				 1
Platform Profile:				 FULL_PROFILE
Platform Version:				 OpenCL 1.2 LINUX
Platform Name:				 Intel(R) OpenCL
Platform Vendor:				 Intel(R) Corporation
Platform Extensions:				 cl_khr_icd cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_byte_addressable_store cl_khr_depth_images cl_khr_3d_image_writes cl_intel_exec_by_local_thread cl_khr_spir cl_khr_fp64

Platform Name:				 Intel(R) OpenCL
Number of devices:				 1
Device Type:					 CL_DEVICE_TYPE_CPU
Vendor ID:					 8086h
Max compute units:				 8
Max work items dimensions:			 3
Max work items[0]:				 8192
Max work items[1]:				 8192
Max work items[2]:				 8192
Max work group size:				 8192
Preferred vector width char:			 1
Preferred vector width short:			 1
Preferred vector width int:			 1
Preferred vector width long:			 1
Preferred vector width float:			 1
Preferred vector width double:		 1
Native vector width char:			 32
Native vector width short:			 16
Native vector width int:			 8
Native vector width long:			 4
Native vector width float:			 8
Native vector width double:			 4
Max clock frequency:				 2700Mhz
Address bits:					 64
Max memory allocation:<PHONE_NUMBER>
Image support:				 Yes
Max number of images read arguments:		 480
Max number of images write arguments:		 480
Max image 2D width:				 16384
Max image 2D height:				 16384
Max image 3D width:				 2048
Max image 3D height:				 2048
Max image 3D depth:				 2048
Max samplers within kernel:			 480
Max size of kernel argument:			 3840
Alignment (bits) of base address:		 1024
Minimum alignment (bytes) for any datatype:	 128
Single precision floating point capability
Denorms:					 Yes
Quiet NaNs:					 Yes
Round to nearest even:			 Yes
Round to zero:				 No
Round to +ve and infinity:			 No
IEEE754-2008 fused multiply-add:		 No
Cache type:					 Read/Write
Cache line size:				 64
Cache size:					 262144
Global memory size:<PHONE_NUMBER>2
Constant buffer size:				 131072
Max number of constant args:			 480
Local memory type:				 Global
Local memory size:				 32768
Kernel Preferred work group size multiple:	 128
Error correction support:			 0
Unified memory for Host and Device:		 1
Profiling timer resolution:			 1
Device endianess:				 Little
Available:					 Yes
Compiler available:				 Yes
Execution capabilities:
Execute OpenCL kernels:			 Yes
Execute native function:			 Yes
Queue on Host properties:
Out-of-Order:				 Yes
Profiling :					 Yes
Platform ID:					 0x166b950
Name:						 Intel(R) Core(TM) i7-4800MQ CPU @ 2.70GHz
Vendor:					 Intel(R) Corporation
Device OpenCL C version:			 OpenCL C 1.2
Driver version:				 <IP_ADDRESS>
Profile:					 FULL_PROFILE
Version:					 OpenCL 1.2 (Build 25)
Extensions:					 cl_khr_icd cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_byte_addressable_store cl_khr_depth_images cl_khr_3d_image_writes cl_intel_exec_by_local_thread cl_khr_spir cl_khr_fp64

dmesg | grep amdgpu produces:

[    0.922697] [drm] amdgpu kernel modesetting enabled.
[    0.926624] amdgpu 0000:01:00.0: enabling device (0000 -> 0003)
[    0.926736] amdgpu 0000:01:00.0: SI support provided by radeon.
[    0.926737] amdgpu 0000:01:00.0: Use radeon.si_support=0 amdgpu.si_support=1 to override.

lsmod | grep kfd produces:

amdkfd                188416  1
amd_iommu_v2           20480  1 amdkfd

dmesg | grep amdkfd however doesn't yield any results. So it seems the driver is not correctly loaded?  How can I fix this?

This is operator error since this GPU is not supported by ROCm it needs to be a GFX8. or GFX9 based GPU.


Channel into_iter works only ones but is it supposed to?

I was thinking that the loop will work generally indefinitely until it get some network issue but it seems like works only 1 iteration. I could imagine that is considered an expected behavior but I am not sure so I opened the issue.
#[tokio::main]
async fn main() {
    let conn = rabbit_conn().await.expect("cann't connect to rabbitmq");
    let ch = rabbit_channel(&conn).await.expect("rabbitmq: cannot create a consumer channel");

    // loop works only 1 iteration
    for delivery in ch {

    }
}

 2020-10-05T21:19:46.676Z WARN  pinky_swear > Failed resolving promise, promise has vanished. promise= error=sending on a closed channel

PS: I haven't found implementation of IntoIter :disappointed:

There is no IntoIter implementation on Channel, only on Consumer.

Btw, which version of lapin are you using? 1.4.1 might fix your issue

It seems works with 1.4.1 I used 1.3.1.
And yes I meant Consumer :(
I'll close the issue, Thank you


RTL?

I need to add RTL support to some of my notes, is there anyway to achieve this? even with css?

You can probably create a custom extension with your rtl styling
https://tiptap.dev/docs/editor/guide/custom-extensions


Cannot read property 'length' of undefined

Issue Id: 2880a85b-d496-07f0-d61e-024709cee1d6Versions - 1.14.1Stack TypeError: Cannot read property 'length' of undefined/vs/platform/actions/common/menu.ts#L51:18 (length)/vs/base/common/callbackList.ts#L63:26 (apply)/vs/base/common/event.ts#L123:26 (call)/vs/base/common/event.ts#L331:13 (fire)

I have confidence that a change in the debounce event fixed this


ISPN-12388 Add CloudEvents integration module

https://issues.redhat.com/browse/ISPN-12388

Send cache entry events to a Kafka topic
Use the CloudEvents JSON format
Add a source to cache entry events
The event source is either a transaction, a command invocation id,
or null (for expiration)

Looks good to me. However there is a new class Main in the rocksdb module which is spurious

I removed the class @tristantarrant, looks like I was working on too many things at the same time :)

Merged, thanks


frequency limit rule question

_get_token_eth_output_price trigger HttpError 429 too many request for url.  what is the frequency limit rule for the api.

Ah, are you working with this? https://github.com/uniswap-python/uniswap-python/blob/29b3e7226327dca5cd28c1f79c51f0ff5c458e9b/uniswap/uniswap.py#L221 -- this is an unofficial library, please bring your question to them


De-float version selection in docs

As mentioned in #1448 I'd like to de-float the version selector (it's in the bottom right).
However, I don't quite understand how the hosted readthedocs works. By default the version selector is indeed on the left, not on the right. See the official docs as an example: https://docs.readthedocs.io/en/latest/

However, on the sharp docs there are several CSS files that don't exist on the official website which override the CSS and among other things set left: auto and right: 20px.

sharp.pixelplumbing.com is CNAMEd to readthedocs, so there's no custom reverse proxy injecting those things. Could you check the settings on readthedocs? Maybe it can be reconfigured? I don't think the correct solution is to override it once again with more custom CSS.

Thanks for digging into this. Beyond the subnav improvement any extra CSS is defined somewhere inside the readthedocs theme processing. As you point out, their own site doesn't appear to have this so I'm pretty confused.
It might be time to reassess if readthedocs is the best place to host these given the lack of control over layout. I've briefly looked at a few other solutions and a combination of GitHub pages and docusaurus (for versioning) might be the way to go.
Any other suggestions are very welcome.

The replacement of readthedocs is being tracked at #1974.


PARQUET-627: Ensure that thrift headers are generated before source compilation

+1

Can you add PARQUET-627: to the PR title? Thanks

Done.


How to remove a series from layout view

This maybe a question, not an issue.
When there are multiple series in a study, they can be seen together in the tiled panels using "Layout" tool. Once a series is added to a layout panel, how can I remove it from the panel and add a different one? Sorry if it's an obvious question but I couldn't find a user interface to do it.

I just figured out that a series in the panel can be switched by using "Previous" and "Next" tools.


Brightest Sh-t ive seen xD

nothing big other than with shaders its the most brightest weapons Ive ever held, 10/10 gives me advantage at combat by blinding my enemies! xD love the weapons btw!

shining brighter than the sun for sure 💀
Don't worry, this issue has already been reported and a fix is coming next update!


Dynamic labeling of function pointers in a stack based ISA

Hello,
I've been reversing some VM lately which happens to be largely stack based. To show my point, here is an example I'll use thorought this entire question:
      pushap     r1,0xf0                                         00000a90
      push       s_event_00006728                                00000a94 = "event"
      push       0x3012c                                         00000a98
      push       0xc7c                                           00000a9e
      push       0x0                                             00000aa4
      push       0x0                                             00000aaa
      push       0x0                                             00000ab0
      push       0x0                                             00000ab6
      push       0x0                                             00000abc
      push       0x0                                             00000ac2
      push       0xc81                                           00000ac8
      push       0x0                                             00000ace
      syscall    0x1,0x6                                         00000ad4

In such an excerpt, the syscall actually eats up all the stack and uses it as an argument.
The issue I'm encountering is that some of those arguments are actually function pointers, which use the exact same opcode as a push  type. Those addresses, if present, are relocated to 0x10+(addr<<1).
It is also to note that the first three pushes, in that case, are NOT function pointers and as such should never be treated like it in ghidra, unless we want a broken analysis. The push that resolves to a string is because it uses a different opcode from the push  format. The push  opcode resolves inside the VM the relocation, while the other arguments of the syscall resolves inside the syscall, to treat the case where function pointers are NULL.
As the arguments are defined on a per syscall basis I was wondering what was the best way to detect those function pointers, as they are crucial to enabling further analysis of the binary, but cannot easily be defined in SLEIGH as far as I'm aware.

I ended up, writing an analyzer that finds said function pointers and create new functions.
With that said, I am at a loss when it comes to actually changing the assembly output: currently the only way that I found to note in the original assembly which function is pointed to is by adding a comment.
Using my first example, it looks like this:
      push.ap    r1,0xf0                                         00000a90
      push.l     s_event_00006728                                00000a94 = "event"
      push.v     0x3012c                                         00000a98
      push.v     0xc7c                                           00000a9e pointer to: 1908
      push.v     0x0                                             00000aa4
      push.v     0x0                                             00000aaa
      push.v     0x0                                             00000ab0
      push.v     0x0                                             00000ab6
      push.v     0x0                                             00000abc
      push.v     0x0                                             00000ac2
      push.v     0xc81                                           00000ac8 pointer to: 1912
      push.v     0x0                                             00000ace
      syscall    0x1,0x6                                         00000ad4

This unfortunately breaks X-Refs, so I wondered if there is a better way to do this.
Keep in mind that I'd also like to keep the assembler working, and as such I cannot ask the analyzer to blindly translate all push.v calls into push.l calls.
Thanks!


URL address is shown for internal Mattermost links when hovering with mouse.

I confirm (by marking "x" in the [ ] below):

[x] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary
URL address is shown for internal Mattermost links when hovering with mouse.
Steps to reproduce
Using macOS 10.12.1 and Desktop Mattermost 3.5.0.
If I hover over any of the channels, flag icon etc then the URL is shown at the bottom of the window. This appears to be a new behaviour in v3.5.0. These "internal" Mattermost URLs were not shown previously. I can understand showing the URL for external URLs for posted content but it's irritating and IMHO unnecessary to show the URL for internal Mattermost links.

@yuya-oc is this controlled by the Mattermost server, or are there optimizations we can do on our end?
The behaviour currently matches between the web app and the desktop app.

The hovering URL is controlled by our side. Currently the app doesn't check a URL, so it's merely appears even if it's internal address.

Ah okay -- it would make sense then not to show the URL for internal links by default.
Most of them aren't very useful, e.g.

Would it just ignore links that match the server URL?

Now what tha app can know is URL string only, so the only way is pattern matching.
For embedded link, I'm not sure whether there is a way to know the link is embedded or not for now.

Ah, I see. I don't know if we'd want that, since people can use embedded links and it's helpful to see the URL before clicking it.
@hamishgraham how much are the channel URLs bothering you?
For internal links that are shown when hovering over buttons like flags and channel names (see below) are handled on the server side. It might make sense to remove those, what do you think?

I'd get used to it sure but I'm currently using v3.4.1 because of this but I'm sure I'll upgrade at some point. It just seems like unnecessary behaviour given that it's meant to be closer to a native app rather than a web browser. Apps like Skype or Slack don't show internal links.

By embedded links, do you mean ones created with markdown like this: "somesite" ?
Then yes it's definitely useful to see the full URL when hovering over these.

^yeah, that's correct @hamishgraham. That's actually why this feature was added to v3.5, because people were having trouble knowing what the link is before clicking it.
Also, Slack/Skype apps don't show any URL when you hover over them (whether internal or external), at least on Windows 10.
I'll bring it up in our UX design meeting to consider removing internal links in Mattermost webapp for buttons like flags and recent mentions. That will at least help reduce how often you see the URL for internal links.

Adding a server side label for now -- we're looking at potentially hiding the URL for internal links, such as when hovering over icons.
This should help mitigate how often you see the URL for internal Mattermost links.

Hey all,
We have a ticket to make some changes on the server side where internal links will be hidden from the UI: https://mattermost.atlassian.net/browse/PLT-6828
The changes will automatically apply to the desktop app.
The improvement is tentatively scheduled for Mattermost server v4.1, released on August 16th.

Great news! Thanks!

The ticket PLT-6828 will change link-tags () to button-tags to hide the url? Ok, this is one solution. Another solution: Hide the browser native URL-Display (bottom) and implement a custom Display for URLs etc. For example a popups / popovers like https://semantic-ui.com/modules/popup.html by hovering over a link. Or implement a url protection. When a user clicked on a link, a popup-window (html) is shown with complete url and a forward button.
Or enable a OS native quickinfo tooltip when hovering the link.
The browser native url-display on the bottom of the page is really annoying. e.g. its overlapping the "User is typing..." info. etc.

@Dominik-Andreas-Geng Thanks for the suggestions! Sorry for the late reply,
I'd be curious to hear what advantages your solutions have over changing link-tags to button-tags?

@jasonblais Thanks for the reply. Hmm, advantages ... This wasn't my intention. Just listed a few alternative solutions. At the moment, I cannot guess what disadvantages buttons have. Ok, you should adjust some properties in CSS to remove unwanted behaviours like outline, etc. I think my solutions may looks prettier.  And are independent from browser URL view.
Perhaps we can add a placeholder line at the bottom for the URL view of the browser. (25px height)
To prevent overlapping some text like "User is typing ..." when hovering a link.
But anyway... I just wanted to throw some ideas into this thread. 😄

Thanks @Dominik-Andreas-Geng! Have forwarded this to our developer who was going to work on this ticket. Appreciate the input! :D

Hey @Dominik-Andreas-Geng, just so I understand, how are popovers, or tooltips going to aid here?
We just want to hide the url bar itself, like we don't see it adding a lot of value, so are you suggesting that we show the url in tooltips etc?

Hi @asaadmahmood, no please not! 😄 Hide the URL bar in any case! Especially for internal links.
No, I wanted to say: Disable the native url bar of the browser completely and add a own custom url viewer (like a popover or whatever) for links posted by users.
Why? It just fixed the issue of internal links in any case and looks prettier as the native url bar for user posted liks. Also it adds possibilities like Link-Preview for the feature. Because it's fully HTML.
This is just an idea I thought about.
For my understanding, the reason for this ticket is to hide ONLY internal links and not the links posted by users. Of course, the browser (chromium) does not show the targets (links) of buttons. So you can use this for a "quick" implementation. My intention only was to show some other solutions. Regardless from the button-solution. This solution (buttons) is also a great solution for me.
But I think, It looks more prettier in mattermost-style (html).
Again: Please do not misunderstand my suggestion. I want exactly the same aim as of this ticket.
Remove the display of internal links. But perhaps display the user posted links in a fancy way.

Sure, sounds cool. Thanks for the suggestion :).

@jasonblais Recently I have noticed though, by comparing address part of server URL and link URL, we might be able to hide the bar when they are in the same server.

@yuya-oc Would this be true for user-specified links as well (such as permalinks or Markdown formatted links in the form [some text](link containing server URL)

@jasonblais Yes. In the idea, the bar doesn't appear when a contained URL is an internal link. (Currently app can know only "URL")

That makes sense. It might not be desired though, as people might want to know the destination of an embedded link before clicking it.
Once https://github.com/mattermost/mattermost-webapp/pull/53 is merged, it should hide the URL for any links that weren't supplied by the user. I think that might be sufficient for the app too.