py.test syntax error

I am getting the errors below when trying to execute py.test. Also, are you guys not supplying a test file for the OWASP ruleset? Are we expected to write our own?
Thanks in advance!
$ py.test test/test_default.py --ruledir test/yaml
usage: py.test [options] [file_or_dir] [file_or_dir] [...]
py.test: error: unrecognized arguments: --ruledir test/yaml
inifile: None
rootdir: /Users/sherman/Fastly/ftw

There is quick start documentation here:
https://github.com/fastly/ftw/blob/master/docs/basic_usage.md
If you want to run a test corpus specifically for the the OWASP CRS, they can be found here:
https://github.com/SpiderLabs/owasp-modsecurity-crs/tree/update-contributing-v3.2/util/regression-tests
The CRS tests assume that you have the Modsecurity log, so you will need to adjust the configuration
(settings.ini) to match your environment. More documentation can be found in the README.md for the repository path linked above.
Hope this helps. Thanks!

One last question: I don't want to test a local WAF - can I use this
framework for it by just pointing to an IP address?
Thank you!
On Fri, Mar 8, 2019 at 11:02 PM Christian S.J. Peron <
<EMAIL_ADDRESS>wrote:

There is quick start documentation here:
https://github.com/fastly/ftw/blob/master/docs/basic_usage.md
If you want to run a test corpus specifically for the the OWASP CRS, they
can be found here:
https://github.com/SpiderLabs/owasp-modsecurity-crs/tree/update-contributing-v3.2/util/regression-tests
The CRS tests assume that you have the Modsecurity log, so you will need
to adjust the configuration
(settings.ini) to match your environment. More documentation can be found
in the README.md for the repository path linked above.
Hope this helps. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fastly/ftw/issues/104#issuecomment-471075271, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKDFsoD5Rrnoyycmyp5wP1MxIpCwW5gHks5vUs_zgaJpZM4bkCKb
.

You can override the target using the --destaddr option on the command line. In some cases (like when going through a CDN) you will need to specify or override the Host: header too. If you use the docker image both options are supported:
See: https://github.com/fastly/ftw/blob/master/docs/Docker.md
However, otherwise you would need to add some code to whatever test harness you are using as done  here:
https://github.com/fastly/ftw/blob/master/tools/build_journal.py#L36

Having the same issue. Followed the readme and the installation via virtualenv and still getting a syntax error.

Same issue here as well.  Tried install and readme instructions from above link.  Running on AWS Linux.
Here's a quick dump of my environment:
`[ec2-user@ip-172-31-64-72 ~]$ cat test_rules.py
#!/usr/bin/python

from ftw import ruleset, testrunner, http, errors
import pytest
def test_default(ruleset, test, destaddr):
'''
Default test with no log [logger] obj.
'''
runner = testrunner.TestRunner()
try:
last_ua = http.HttpUA()
for stage in test.stages:
if destaddr is not None:
stage.input.dest_addr = destaddr
if stage.input.save_cookie:
runner.run_stage(stage, http_ua=last_ua)
else:
runner.run_stage(stage, logger_obj=None, http_ua=None)
except errors.TestError as e:
e.args[1]['meta'] = ruleset.meta
pytest.fail('Failure!


No Module Found markupsafe

I am receiving the following error:  Unable to import module 'app': No module named markupsafe
Below is what I have in requirements.txt:
botocore==1.4.41
click==6.2
docutils==0.12
Flask==0.11.1
itsdangerous==0.24
MarkupSafe==0.23
Jinja2==2.8
jmespath==0.9.0
marshmallow==2.9.1
PyMySQL==0.7.6
python-dateutil==2.5.3
python-memcached==1.58
simplejson==3.8.2
six==1.10.0
SQLAlchemy==1.0.14
typing==<IP_ADDRESS>
Werkzeug==0.11.10
I am trying to get a simple example to work from a Flask app.  I have modified my code and it compiles without a problem, however when trying to access the service, I receive this error.

Could you share steps to repro the issue?  I'm not able to repro.  Here's what I tried:

Create a new project:

$ chalice new-project test-reqs
$ cd test-reqs

I updated the requirements.txt file to match the exact contents you have above.

$ cat requirements.txt
botocore==1.4.41
click==6.2
docutils==0.12
Flask==0.11.1
itsdangerous==0.24
MarkupSafe==0.23
Jinja2==2.8
jmespath==0.9.0
marshmallow==2.9.1
PyMySQL==0.7.6
python-dateutil==2.5.3
python-memcached==1.58
simplejson==3.8.2
six==1.10.0
SQLAlchemy==1.0.14
typing==<IP_ADDRESS>
Werkzeug==0.11.10

Try to import markupsafe in my app:

$ cat app.py
from chalice import Chalice

import markupsafe

app = Chalice(app_name='test-reqs')

@app.route('/')
def index():
    return {'hello': 'world'}

Deploy and test out the app:

$ chalice deploy
...
$ http https://....execute-api.us-west-2.amazonaws.com/dev/
HTTP/1.1 200 OK
Connection: keep-alive
Content-Length: 18
Content-Type: application/json
Date: Mon, 01 Aug 2016 20:05:08 GMT
x-amzn-RequestId: 3f19bd99-5823-11e6-8f93-8f6c60407675

{
    "hello": "world"
}

Thanks for your prompt response.
I have done the following - shorter version of the actual application, but even the bottom below does not seem to work - deploying to us-east-1:
virtualenv ~/.virtualenvs/reviewsreqs
source ~/.virtualenvs/reviewsreqs/bin/activate
pip install chalice
chalice new-project reviews-reqs
cd reviews-reqs/
pip install flask
pip install marshmallow
pip install simplejson
pip install sqlalchemy
pip install python-memcached
vi requirements.txt
botocore==1.4.41
click==6.2
docutils==0.12
Flask==0.11.1
itsdangerous==0.24
MarkupSafe==0.23
Jinja2==2.8
jmespath==0.9.0
marshmallow==2.9.1
PyMySQL==0.7.6
python-dateutil==2.5.3
python-memcached==1.58
simplejson==3.8.2
six==1.10.0
SQLAlchemy==1.0.14
typing==<IP_ADDRESS>
Werkzeug==0.11.10

vi app.py
import markupsafe
from chalice import Chalice

app = Chalice(app_name='reviews-reqs')

@app.route('/')
def index():
    return {'hello': 'world'}

Below is what CloudWatch has:

START RequestId: 8d23b1fd-5830-11e6-9131-1f4f096f400a Version: $LATEST
Unable to import module 'app': No module named markupsafe
END RequestId: 8d23b1fd-5830-11e6-9131-1f4f096f400a
REPORT RequestId: 8d23b1fd-5830-11e6-9131-1f4f096f400a	Duration: 0.46 ms	Billed Duration: 100 ms 	Memory Size: 128 MB	Max Memory Used: 15 MB

Anything I could be doing incorrectly?  I have tried to create from my mac as well as an EC2 instance, both seem to result in the same error.

I followed the same steps you had above, and I'm still not able to get any errors.  I guess the next steps would be to see if we're actually creating the deployment package correctly.  Here's what I'd try next:
In your app directory, there should be a .chalice directory.  Within that directory there's a deployments directory that contains the zip file we created:
$ cd .chalice/deployments
$ ls
3d0d7f394c0cbf96ab351fc4d4b70491.zip

(Your filename hash might be different, that's ok.  There should still only be a single file in that directory).
I'd suggest trying to unzip the file, double check "markupsafe" is in that directory, and then try to import "app" from that directory:
$ mkdir tempdir
$ cp 3d0d7f394c0cbf96ab351fc4d4b70491.zip tempdir/
$ cd tempdir
$ unzip 3d0d7f394c0cbf96ab351fc4d4b70491.zip
# We should see markupsafe here.
$ ls | grep -i markupsafe
MarkupSafe-0.23.dist-info
markupsafe

And we should also be able to import app from this directory:
$ python
>>> import app
>>> app.app.routes
{'/': <chalice.RouteEntry object at 0x102e9bdd0>}

What do you get when you try the steps above?

Thanks.  I see that three modules were not in the zip file: markupsafe, simplejson, and sqlalchemy.  Not sure why they are not in the zip file while yours are?
Below is what I get when I run app.app.routes:

(clireviews)[ali@ip-172-31-70-156 tempdir]$ python
Python 2.7.10 (default, Dec  8 2015, 18:25:23)
[GCC 4.8.3 20140911 (Red Hat 4.8.3-9)] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import app
app.app.routes
{'/': <chalice.RouteEntry object at 0x7f8609903c50>}

Is there something that I can do to force chalice to include these modules?

In the above python repl (while you've cd'd into the zipfile directory), can you import markupsafe?
chalice is just running pip install -r requirements.txt (https://github.com/awslabs/chalice/blob/master/chalice/deployer.py#L699-L700), so I'm not sure why it would be skipping over those modules.  Perhaps pip thinks those modules are already installed somehow?  If the import markupsafe works in the python repl, perhaps it's being imported from somewhere else that's causing pip to already think it's installed.

I was able to get it to work by doing the following (really odd why it worked):

Created the project
Updated the requirements.txt file
Updated app.py to import markupsafe
Tried to deploy the project as is and was warned that markupsafe was missing - note that I did not install any other dependencies from the requirements.txt file.  Since I could not deploy, I installed markupsafe via pip.  I did not install anything else and then tried to redeploy the chalice project and it worked.

Really odd as to why markupsafe does not install by simply reading the requirements.txt.
Thanks for your help!


feat: added adminUser create new password if he forgets password and …

Overview

This PR is related to this issue
It contains the functionality for an admin user to create new password if they forget password by sending a password reset link to his email
this password reset link verifies a token that is embedded in the url which can also expire. Adding a new password upon successful verification updates the adminUser's DB details with the new password that is hashed.

How to test

To test this make sure you run npm install to install all dependencies
Run npm run lint:fix to fix all linting issues
Test the endpoints:
/api/auth/admin/forgot-password
/api/auth/admin/reset-password/{token}

Please rebase as i asked you on slack

Ok

@ruzor , please you can review now. Sorry for the delay


Add method to list projects a user has contributed to

This PR adds a method to the Projects client to allow calling the contributed_projects endpoint: https://docs.gitlab.com/ee/api/projects.html#list-projects-a-user-has-contributed-to
This enables users of this library to fetch all projects that a user has contributed to

@svanharmelen bumping you for review!


Migrate project.dart and all dependencies to null safety

Split bundle.dart into null safe bundle.dart and null unsafe bundle_builder.dart to isolate BundleBuilder class.  Update imports.
Migrate project.dart to null safety
Migrate all of project.dart's dependencies to null safety.

Part of #71511

This is a dump truck of a review, but these are so interdependent they need to be migrated together.


Integration has stopped working

running the latest version on HA 2024.4 and the integration fails to start without any logging.

My bad and fault on my part.


__decorate is not defined

All of my configuration files are similar to yours, but I made a mistake while I was at NPM run dev
Uncaught ReferenceError: __decorate is not defined

check your typescript version


test: openPropertyPane js helper update with new TS helper

Description
Task to remove one helper. It is to test app behaviour with new helper from TS.
Fixes #35263
Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!TIP]
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10156395702
Commit: 0b86f6393a9ea3dbd8a6edee5faea7335473f2f8
Cypress dashboard.
Tags: @tag.All
Spec:
Tue, 30 Jul 2024 06:48:56 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

Summary by CodeRabbit

New Features

Introduced a new method in the PropertyPane class to automate interactions with various widget property panes.

Refactor

Updated the method for accessing the property pane in multiple test cases to improve maintainability and organization.
Transitioned from cy.openPropertyPane to _.propPane.openPropertyPane across various widget tests, ensuring consistency in the test suite.

These changes enhance the clarity and structure of our test code while expanding interaction capabilities within the UI testing framework.

/ci-test-limited

/ci-test-limit

/ci-test-limit<PHONE_NUMBER>5 10

/ci-test-limit runId=10161624125 run_count=10


featch

i fetch date in beforeRouteEnter,when and 'next()' when date is back.like below
beforeRouteEnter(to, from, next) {
axios.get(https://kf.dianping.com/api/relations/listRecommendRelation,{
params: {
bu: '美食',
orderType: 'dealOrder',
orderId:<PHONE_NUMBER>,
orderIdStr: "4252500185",
userId: 787514285,
subSource: "MT_APP",
visitId: "480e6383-a8f7-40fb-b00c-bf85a1f16f6b"
}
})
.then((res) => {
next();
});
},
then vue warn like  'vue.runtime.esm.js:587 [Vue warn]: The client-side rendered virtual DOM tree is not matching server-rendered content. This is likely caused by incorrect HTML markup, for example nesting block-level elements inside , or missing . Bailing hydration and performing full client-side render.'
what shoule i do?

There is not enough detail here to be able to help you and also it is not really clear. The problem is not in the code you shared, but somewhere else.

when i use async code in beforeRouteEnter i can get the warn information like below
  beforeRouteEnter(to, from , next){       setTimeout(() => {         next()       }, 100)     }

Closing this as there's been no updates over 90 days.


Create kvis.txt

@ZittaLuthfiyyah First stage of verification passed, official domain is kvis.ac.th as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@ZittaLuthfiyyah Pull request merged. Thank you.


word highlighting occurrences

Issue Type: Bug
before last update, when i used to highlight a word by selecting it with mouse cursor then i used to get a little square in the side scrollbar for each occurrences of that word. Now, after last update, i'm no longer able to see the occurrences on the sidebar, i can see them if i just click a keyword (like funciton name, or variable) but if i select a part of a word or a word then i get all the occurrences highlited in the document but without little squars on the sidebar. please resolve.
VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:35:15.005Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (6.87GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

vsc-prolog
art
0.8.23

vscode-intelephense-client
bme
1.2.3

vscode-eslint
dba
1.9.1

tslint
eg2
1.0.44

nbpreviewer
jit
1.2.2

vscode-language-pack-it
MS-
1.38.4

python
ms-
2019.10.41019

vscode-typescript-tslint-plugin
ms-
1.2.2

prolog
reb
0.0.4

I have the same problem

@hobelas @charanvengatesh I cannot reproduce. Here I am selecting things in JSON and they get highlighted in the scrollbar. Do you encounter this in a specific file type?

I'm noticing this too. On my system, that feature works for most of the language modes I tried, but not TypeScript or JavaScript. Here's a screencast on the latest release build with extensions disabled:

Help > About:
Version: 1.39.0 (user setup)
Commit: 9df03c6d6ce97c6645c5846f6dfa2a6a7d276515
Date: 2019-10-09T06:58:04.439Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Windows_NT x64 6.1.7601

Maybe this is a Typescript issue then? Now that you've pointed that out, I'm seeing the behavior you describe but it only works for top level symbols. For instance, I can highlight on types/in-scope variable names, but I can't do that for properties of symbols (like this, globalThis, or window), even if I highlight the whole thing (this.doSomething() vs doSomething()).
In general, I prefer the default selection highlighter anyway, as that sees into things like method names, descriptive comments and similarly-named symbols (which IME are often related enough that I want to look at them; eg, myMemberVar and mutateMyMemberVar()). The language server can't really do that. For now I can fall back to Ctrl+F, but I'd much prefer a configuration option.


Documentation regarding tap gesture callbacks

cc @gspencergoog

will land on green

@fkorotkov Hard to say if it's our problem or yours, but the tests-windows shard here spun around doing nothing for like an hour. Then it did some more stuff and then eventually timed out. At a guess, there's some sort of resource starvation going on in the Windows cluster.

@Hixie me and @gspencergoog saw that but there were issues with reporting logs for such timed out tasks. I fixed it yesterday and at least now there are logs presented every time (they were missing sometimes for timed out tasks).
It seems it was hanging on running slivers_evil_test.dart.
If you are interested I can pretty easy add a way to run windows containers on GCP instead of Azure. The only problem will be that Windows Contianers on GCP will start in 4-5 minutes instead of 1:30-2 on Azure. But we can check if it will be more stable.

@fkorotkov I ran it again and this time it was 60 minutes on a different test (C:\Windows\Temp\flutter sdk\packages\flutter_localizations\test\date_picker_test.dart: DayPicker shows dates for ar_AR). I'll rerun it again.
Log of that failing run:<PHONE_NUMBER>596544-test_all.log


Error dropdown theme.required prop

Just update my dep to version 1.2.3 and got this error when testing some of my component which using Dropdown component
warning.js:36 Warning: Failed prop type: Invalid prop 'theme.required' of type 'string' supplied to 'Dropdown', expected 'boolean'.
in Dropdown (created by ThemedDropdown)
and my Markup almost same like in example page
<Dropdown auto onChange={this.handleChange} source={countries} value={this.state.value} />
how do i fix this issue?

I saw this issue to and I believe the change at this line
https://github.com/react-toolbox/react-toolbox/blob/dev/components/dropdown/Dropdown.js#L34
should not be a boolean but a string (I tested version 1.2.2 and 1.2.3) and the bug introduced in version 1.2.3
I will create a pullrequest

@javivelasco I met the same issue in my project. Whether this solution published in latest version? Thanks.

I'm releasing a patch just today or tomorrow, I need to check some other prs!

@javivelasco any word on this? Thanks!

Release 1.2.4 solved this issue for me. Thank you

Still having this issue on 1.2.4 (Chrome 54.0.2840.71 (64-bit), react 15.3.2)

Same error, 1.2.4 (React 15.3.2)

Just released a patch. Should work now!


Provide empty username in connection URIs

It's not used for Redis 5, and breaks libraries which support Redis 6, e.g.
newer versions of node-redis.

This seems to be the standard in different languages:
Ruby: https://github.com/redis/redis-rb/blame/fa76a2661739f56e3458fc5ed5a7b7c31861eff5/README.md#L35
Go: https://github.com/go-redis/redis/blob/master/options.go#L44-L47
Node: https://github.com/NodeRedis/node-redis/blame/4f85030e42da2eed6a178e54994330af5062761e/README.md#L175

golang examples: https://github.com/go-redis/redis/blob/master/options_test.go#L52 https://github.com/go-redis/redis/blob/master/example_test.go


Change Focus Camera

Hi,
I am using your library. I want to change the camera's focus in CustomCameraActivity. Can you help me??

Hi, please look at: https://github.com/memfis19/Annca/issues/17, https://github.com/memfis19/Annca/issues/14, https://github.com/memfis19/Annca/issues/10.


Do not special-case ArrayBufferInputStream in DataInputStream, then remove it.

Do not special-case ArrayBufferInputStream in DataInputStream.
The special-case fundamentally requires a dependency from the javalib to the Scala.js standard library, which we want to avoid.

Remove js.typedarray.ArrayBufferInputStream.
The only reasons this class existed in the core were that:

it could be special-cased in DataInputStream, and
it was used in the bootstrapped linker.

Now, neither is true anymore, so the existence of ArrayBufferInputStream is questionable.
Since it was part of the public API of the Scala.js library, its removal is a breaking change. Users of the class can still recover its functionality by copy-pasting the 0.6.x implementation into their own projects. I suspect this shouldn't be too much of an issue, because typed arrays are better suited to be treated as java.nio.Buffers than java.io.InputStreams anyway.

@gzm0 AFAICT, this is the only thing that really must go in 1.0.0 so that we have the possibility to make the javalib free of any dependency on the Scala library. Everything else is internal and can be changed later.


Fix typo.

This is just a minor fix to a man page.

@finnhakansson do you agree this is a trivial change?

(sorry, but we either need a CLA or we need you to definitely say yes it's trivial)

It is definitely trivial.

@richsalz: Feel free to merge this

This goes to master, 1.1.0 and 1.0.2.

Done, in all mentioned branches


virtualbox dhcpserver error on boot

get an error message when booting for the first time with virtualbox
will fill in more info with edits - just want an issue number at the moment.

@phinze closeable as of Vagrant 1.7.3., if I read https://github.com/mitchellh/vagrant/issues/3083 correctly?

Yep!


How to use tags and condition in the chart templates?

Hi,
This is a document question, but it is also important, and I think it is typical usage scenario for us.
Good: I can find the usage for tags and condition for the sub charts in conditions-and-tags
Question:

I just want to know can I use the tags or condition in the chart templates except for sub charts?
If yes, how can I use it? any limitation?

If I want to use global tags(subchart and parent chart can share the same tags and also can use it as condition in template if i have 3 or more level subcharts), how to define?

I think it is useful for us for using helm charts.
Hope you can add this section for it whether it can use or not in chart templates.
Thanks

I’m not totally sure I understand the issue. Would you mind providing a more concrete example of the issues you are running into wand why you think this would solve that issue?

Source: test/requirements.yaml
dependencies:
  - name: subchart1
    version: ~1.0.0
    repository: "https://test.com/chart-repo"
    tags:
      - t1
      - t2

Source: test/deployment.yaml
apiVersion: apps/v1
kind: Deployment
template:
  spec:
    containers:
    - name: test-container
      env:
        - name: ENV1
          value: value1
      {{- if index .Values "tags" "t1" }} # for question-1
        - name T1_ENV
          value: value2
      {{- end }}

Source: test/subchart1/deployment.yaml
apiVersion: apps/v1
kind: Deployment
template:
  spec:
    containers:
    - name: sub-container
      env:
      {{- if index .Values "tags" "t1" }} # for question-2
        - name T1_ENV
          value: value2
      {{- end }}

Is it right style to check tags "t1" enabled or not for the "T1_ENV" as it do not have example or usage in the document?
If I want to enabled "T1_ENV" in test chart and subchart1 chart at the same time,
from my understanding, i should set "tags.t1=true" and "subchart1.tags.t1=true" at the same time.
Can I have a simple way just set one tag to enabled all "t1" tags in any level charts like the "global" values?
e.g  just set  "global.tags.t1=true" is fine

I can understand why this is confusing. The docs on tags and conditions need to be expanded.
Tags and conditions are special cases. For example,
dependencies:
  - name: subchart1
    version: ~1.0.0
    repository: "https://test.com/chart-repo"
    condition: subchart1.enabled
    tags:
      - t1

In this case, the subchart1 won't be install into a cluster unless subchart1.enabled is set to true and the tag is listed and set to true in the values.yaml file like the following:
subchart1:
  enabled: true

tags:
  t1: true

These can also be passed through the command line.
Tags are not going to be passed to sub-charts. The structure of what is passed does not allow for that. I had not considered this situation until now.
Conditions can have their YAML structure passed to subcharts based on the location of the condition in the YAML.
Tags and conditions allow you to turn on and off dependencies as they are installed. They are usually used independently from each other but do not need to be. Conditions are processed before tags.

Hi @mattfarina,
You're right, for simply usage we just use tag or condition.
But in many business scenarios it is not enough, we should use advance composition style, and it may be widely used in each company.
We meet this issue,  so we just use the workaround,

use tags control the sub-charts,
and add a global value used in sub template
but we should keep both value aligned by manually.

But from coding craftmanship, we hope it can just use one value to control.
So I suggest you can consider to strengthen the tags function(support tags in sub charts), it would benefit the world.
And we can keep the existing tag function and scope, and then extend it with new style.
Of course, if you don't mind, I can provide below 2 options:
Option1 add a global section under tags(it may break the current code struct, should add tags.global pass to sub-chart along with global values):
# source: test/values.yaml
tags:
  global: #add a global section
     t1: true
# source: test/requirements.yaml
dependencies:
  - name: subchart1
    version: ~1.0.0
    repository: "https://test.com/chart-repo"
    condition: subchart1.enabled
    tags:
      - global.t1    #should append prefix "global."
# source:  test/subchart1/deployment.yaml
apiVersion: apps/v1
kind: Deployment
template:
  spec:
    containers:
    - name: sub-container
      env:
      {{- if index .Values "tags" "global" "t1" }} # support parent global tags used in sub-charts
        - name T1_ENV
          value: value2
      {{- end }}

Option2 add tags under global(it can reuse the global structure to pass value, just tags in requirement.yaml can handle and check tags under global section):
# source: test/values.yaml
global:
  tags: #add a tags section
     t1: true
# source: test/requirements.yaml
dependencies:
  - name: subchart1
    version: ~1.0.0
    repository: "https://test.com/chart-repo"
    condition: subchart1.enabled
    tags:
      - global.tags.t1   #should append prefix "global.tags."
# source:  test/subchart1/deployment.yaml
apiVersion: apps/v1
kind: Deployment
template:
  spec:
    containers:
    - name: sub-container
      env:
      {{- if index .Values "global" "tags" "t1" }}   # support parent global tags used in sub-charts
        - name T1_ENV
          value: value2
      {{- end }}

If you don't mind, and I also can try to create PR for it if you accept any option above.
Thanks.

I like Option 2.

Thanks @songdongsheng,
The tags is a list, actually each tag may have related k8s resources or env/config parameters, so we should use it to do the judge,
not only in currently level charts, also used in sub-charts.
If it just have combability to control sub-chart's sub-charts, it is a big limitation for the helm charts.
So I strongly propose owner to enhance the tags scope in sub-charts.
Wait for response.

Can we reopen this one? Would make it easier to make workflows based on the tags input in sub charts. At the moment the only way would be to duplicate the logic by using global enabled flag variables, which defeats the purpose of using tags in that case.

This would not work if the url of the subchart is for some reason not accessible. There should be a way to make subchart optional. For example, if you don't need it in certain environments.


leave() method

Hello! To follow up #1, I've also found that I need a way to leave() a particular topic when an archive is no longer in use.
Would a PR for that be welcome, too?

Hi @RangerMauve. Oh yeah, that sounds right.
Feel free to send your PR. I think #1 will be published pretty soon. Will let you know! Thanks.

Hi @RangerMauve. Absolutely, any help is always welcome!

https://github.com/geut/discovery-swarm-webrtc/pull/4


feature: add documentation on keyword check that headers location res…

…ponse has
Changes

First change

Related issue

Additional information and checks

[ ] Pull request linked in changelog

looks ok


img2img时报错TypeError: 'float' object cannot be interpreted as an integer

Traceback (most recent call last):
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/gradio/routes.py", line 488, in run_predict
output = await app.get_blocks().process_api(
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/gradio/blocks.py", line 1431, in process_api
result = await self.call_function(
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/gradio/blocks.py", line 1103, in call_function
prediction = await anyio.to_thread.run_sync(
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/anyio/to_thread.py", line 33, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 877, in run_sync_in_worker_thread
return await future
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 807, in run
result = context.run(func, *args)
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/gradio/utils.py", line 707, in wrapper
response = f(*args, **kwargs)
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/gradio/utils.py", line 707, in wrapper
response = f(*args, **kwargs)
File "/home/yaohs/Work/stable-diffusion-webui/extensions/sd-webui-lcm/scripts/main.py", line 172, in generate_i2i
result = pipe(
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/yaohs/Work/stable-diffusion-webui/extensions/sd-webui-lcm/lcm/lcm_i2i_pipeline.py", line 305, in call
self.scheduler.set_timesteps(strength, num_inference_steps, original_inference_steps)
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/diffusers/schedulers/scheduling_lcm.py", line 458, in set_timesteps
inference_indices = np.linspace(0, len(lcm_origin_timesteps), num=num_inference_steps, endpoint=False)
File "<array_function internals>", line 180, in linspace
File "/home/yaohs/miniconda3/envs/sd_lcm/lib/python3.10/site-packages/numpy/core/function_base.py", line 120, in linspace
num = operator.index(num)
TypeError: 'float' object cannot be interpreted as an integer

The same error

Having the same issue :(

The same error occurs when I run the img2img or vid2vid.

some issue :(

same error in img2img

same error

same error

This repo is non working garbage. Author still didnt nothing.


Can we add CarliniWagnerL0 and CarliniWagnerL1 attacks?

Describe the solution you'd like
I would like to have L0 and L1 version of CW attack in addition to the L2 version.
Describe alternatives you've considered
I have looked into the original code repo of the SP17 paper, which has L0 and L1 version, but I would like to have those in cleverhans as well.
Additional context
I assume it wouldn't be that hard to add the addition CW attacks. As one of the most powerful attack, I suggest that Cleverhans can support all of its variance. Thanks!

@carlini ?

I'm also looking for the solutions required in this post. Currently, I'm trying to see if I can mitigate the original li CW attack from https://github.com/carlini/nn_robust_attacks/blob/master/li_attack.py locally.

By L1 do you mean l_infinity? I haven't done a L1 attack. Or EAD is a L1 optimization-based attack? Do you mean that?
L0 sure. It still performs reasonably well. At one point I started moving it over to cleverhans but never finished. If it would be helpful I can do that. I personally no longer use the l_infinity attack as described in the paper. I didn't l_infinity regularize the gradients which in retrospect is clearly the right thing to do most of the time. Using l_infinity regularized PGD is faster and so far I've never seen it do worse, so I would prefer not to add it.

Yes, I mean L_infinity and L0.
I am working on a defense paper, and I would like to evaluate against all the attacks.
It will be great you can add those two attacks, thanks!

When I get a chance I'll add L_0. For now I do have code available here: https://github.com/carlini/nn_robust_attacks you can use
One word of unsolicited advice: please think carefully when applying these other attacks. I would much rather see an evaluation against only PGD than an evaluation that tried PGD as well as many other attacks, but just running the code for those.

Thanks for the advice!


Replace Travis-CI by GitHub Actions

Unfortunately, Travis-CI recently became unusable for open source projects. Anyone to create a GitHub Actions config for django-formtools?

Hi @claudep,
I can work on this.

Great, I know @jezdez also have a plan to some mass-conversion for Jazzband projects.

No need to work on this @hramezani, thank you for the offer though. I'm converting all projects to GHA this week. More news forthcoming..

This is now pending review in #188


Travel Funding for Coordination Meeting

Full name
Kelle Cruz
Tim Pickering
Matt Craig
Aarya Patel
Description
In-Person participation at the 2023 Astropy Project Coordination Meeting in Cambridge, MA.
Total request
TBD
Itemized budget
Each person should provide a budget estimate in this issue.
Advance needed
No response
Notes

Itemized meal expenses will be needed.
Reimbursement requests should be sent to the Moore project in Open Collective.

cc @tepickering

My budget:
Lodging: ~$1400 at Marriott - ideally paid by PEX card at checkout.
Travel: $183 Amtrak to/from NYC - to be reimbursed.
Meals: ~$200 - actual expenses to be reimbursed.
ESTIMATED TOTAL = $1,783

my budget:

Lodging: ~$1100 at sonder 907 main - actual expense to be reimbursed
Travel: ~$700 non-redeye airfare - actual expense to be reimbursed
Meals: ~$200 - actual expenses to be reimbursed
Transport: ~$50 - personal vehicle 40 miles r/t in tucson, T fares in boston - actual expenses to be reimbursed

ESTIMATED TOTAL = $2350
my lodging estimate is based on a may 1 check-in and may 5 check-out (4 nights) since that is what would be required to fly home on may 5. possible itineraries preclude attending meeting on may 4 and flying home on that day. i will also likely stay in boston longer, but will only submit may 1-5 for reimbursement.

@nstarman If you could post your request for financial assistance here (and a short sentence why it's coming now) that would be great.

My budget:

lodging: $0, staying with friends
Travel: CA$580 (~USD$427)
Transport: $30, (I have a bike rental some days)
Meals: USD$200 - actual expenses to be reimbursed

ESTIMATED TOTAL = $660
I always wanted to come to the Astropy conference, but didn't I could come due to a family event. My schedule freed up last minute and I would very much like to come in person, especially as a newer maintainer I have not met most Astropy people!

Remember, keep itemized meal receipts. When travel is complete, please submit reimbursement request to the Moore Project in Open Collective.


Using custom languages instead of pathfinder2e languages

User Experience
Hi! I am creating my own setting using the pathfinder 2e rules so I would like to customize my languages while still using the foundry module. I looked over the wiki and I'm not sure I understand how to add custom languages, I'm wondering if perhaps because I'm using the pathfinder 2e system that the option is locked off? Any help would be appreciated, thanks.
Have you read the Wiki?

[X] Yes

You have to add languages through PF2e's Homebrew Elements menu.


Post submit sha

Functionally equivalent to previous one except it also has OS X istioctl
And built from master by postsubmit
install/updateVersion.sh -pgcr.io/istio-testing,fb947d4c6d0902a841e0e82ffa3bee16786be0df
NONE

@kyessenov this is the same sha functionally - any objection to merging this ?
(separately from the mixer and CA updates which may or may not have worse issues)
if you accept in github I will merge. thanks.

thx

@ldemailly: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-rbac-auth

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.

@ldemailly: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-rbac-auth

prow/e2e-suite-rbac-no_auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-rbac-no_auth

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.

@ldemailly: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-rbac-auth

prow/e2e-suite-rbac-no_auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-rbac-no_auth

prow/e2e-suite-no_rbac-auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-no_rbac-auth

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.

@ldemailly: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-rbac-auth

prow/e2e-suite-rbac-no_auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-rbac-no_auth

prow/e2e-suite-no_rbac-auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-no_rbac-auth

prow/e2e-suite-no_rbac-no_auth.sh
07d300bfc9033e4eeea209a5eb6e8284aa416392
link
/test e2e-suite-no_rbac-no_auth

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.


Solve the mistery of empty object export for NPM packages

Issue has been seen there:
https://github.com/meteor/meteor/issues/10065
https://forums.meteor.com/t/importing-from-npm-returns-an-empty-object/44491
https://github.com/thereactivestack-legacy/meteor-webpack/issues/235
https://stackoverflow.com/questions/55382092/importing-gitlab-package-from-meteorjs-returns-empty-object
https://forums.meteor.com/t/solved-require-import-npm-require-all-return-empty-object/50153/6
https://ethereum.stackexchange.com/questions/60335/web3-empty-object-server-side-in-meteor
The issue seems to appear eratically at different point in time, and different Meteor versions.
Had this issue when coding the automated backoffice, for material-ui icons and no-ssr.
To be investigated.

Reproduction in Vulcan core repo:

Uncomment vulcan:ui-material ( and comment bootstrap)
Run Vulcan
Open /backoffice => SSR fails because MenuIcon is an empty object instead of being a component

Potential solution proposed by @captainn here:
Use a config similar to this, with babel-plugin-transform-imports
{

  "meteor": {
    "mainModule": {
      "client": "client/main.jsx",
      "server": "server/main.ts"
    },
    "testModule": "tests/main.ts",
    "nodeModules": {
      "recompile": {
        "simpl-schema": "legacy",
        "uniforms": true,
        "uniforms-material": true
      }
    }
  },
  "babel": {
    "plugins": [
      [
        "transform-imports",
        {
          "@material-ui/core": {
            "transform": "@material-ui/core/${member}",
            "preventFullImport": true
          },
          "@material-ui/icons": {
            "transform": "@material-ui/icons/${member}",
            "preventFullImport": true
          },
          "@material-ui/styles": {
            "transform": "@material-ui/styles/${member}",
            "preventFullImport": true
          },
          "uniforms": {
            "transform": "uniforms/src/${member}",
            "preventFullImport": true
          },
          "uniforms-material": {
            "transform": "uniforms-material/src/${member}",
            "preventFullImport": true
          }
        }
      ],
      "lodash",
      "npdev-react-loadable-babel"
    ]
  }
}

@benjamn Would you graciously take a look at this long-standing issue with Meteor module loader?

Just for the record we have since migrated toward Next.js (https://vulcan-next.vercel.app/) and we have recently managed to mimick Meteor package architecture with NPM packages and Tsup (https://github.com/VulcanJS/npm-the-right-way/tree/main/my-package-tsup) but maybe this ticket could help detecting/fixing issues in Meteor. Let me know if you need help to reproduce.

Opened a new Meteor issue with reproduction: https://github.com/meteor/meteor/issues/11900

Mystery solved : this bug affect specifically dev dependencies, and Material UI is a dev dependency in the Vulcan app 🎉 Now we simply need to wait for a fix in Meteor, see linked issue


memory leak in getLocalCachedMap, redisson version : 3.10.3

Expected behavior
no memory leak
Actual behavior
memory leak
Steps to reproduce or test case
run the attached code
Redis version
Redis server v=4.0.8 sha=00000000:0 malloc=jemalloc-4.0.3 bits=64 build=9d70bbb410c08012
Redisson version
3.10.3
Redisson configuration
    Config redissonConfig = new Config();
    redissonConfig.useSingleServer()
        .setTimeout(1000000)
        .setAddress("redis://<IP_ADDRESS>:6379")
        .setDatabase(10)
        .setPassword("123")
        .setConnectionPoolSize(10)
        .setConnectionMinimumIdleSize(2);
    redissonConfig.setCodec(new FstCodec());
    redissonConfig.setLockWatchdogTimeout(5000);

public static final LocalCachedMapOptions options = LocalCachedMapOptions.defaults()
       .evictionPolicy(EvictionPolicy.NONE)
       .cacheSize(0)
       .reconnectionStrategy(ReconnectionStrategy.NONE)
       .syncStrategy(SyncStrategy.UPDATE)
       .evictionPolicy(EvictionPolicy.WEAK)
       .timeToLive(0)
       .maxIdle(0);

RedissonMemoryLeakTest.zip

This issue happens because you create a new RLocalCachedMap instance each iteration. Please use it as a singleton object (per key).

you mean i should NOT get the LocalCachedMap more than 1 time even the key ("123") are identical.

RLocalCachedMap<String, DDD> cachedMap = client.getLocalCachedMap("123", options);

@gmailisgone
Yes. Also call RLocalCachedMap.destroy() method once object isn't used anymore.

Yes. Also call RLocalCachedMap.destroy() method once object isn't used anymore.
Got it.  I thought the operation is same as redis command.

But, what will RLocalCachedMap.destroy() do,

Destory the objects in local JVM? (if this one, it will disappeared if I restart the JAVA program)
Destory the objects in redis?

@gmailisgone
Yes, it will clear local cache, but not the data stored in redis

@mrniko
thanks for your reply.
There is one more thing i want to confirm,

For getLocalCachedMap, I can get only for 1 time,
but for other redisson data structures, I can get

Is there any document given information like these?


Base Goerli Chain Support

Add Base Goerli Chain Support

I'll do it.

https://github.com/TrueFiEng/useDApp/pull/1110

Done in #1110


Also trigger tint on focus gain/focus loss

Hi, nice plugin! I'm working with Neovim in tmux, and then it's also nice to tint the Neovim window when switching to a different tmux pane. This simple change should take care of that.
Alternatively, if you think this is too invasive, you could also make the set of events that trigger the autocommands configurable. As a third option, perhaps you could expose the callbacks somehow.

Caught it too late, in my opinion it would be better to expose it as an option rather than hard coding it.
I am using wezterm and it does tinting of inactive panes for me so it feels redundant now - I did not update to latest version of tint just yet so not sure how combined tinting will look like but I would probably very happily keep au events as they were before that or got merged if it was exposed as an option.

@gegoune yeah, you are probably right
@levouh perhaps you want to make the sets of events that trigger the autocommands configurable?

See #26 @gegoune. You will have to add events to your configuration now FYI @gstokkink.


Null-returns for Externals with object return values (e.g. NpcInstance) need to be allowed

Description:
ZenKitCS doesn't allow null as return value when a class is expected. Following example leads to an error:

Hlp_GetNpc() returns null and the whole execution stops with the error below
Be careful: Once we allow null-returns, we also need to make sure, that ZenKitCS recognizes these null-values as input parameters later (e.g. Hlp_GetInstanceID())

Error:
InvalidOperationException: Unsupported type:
ZenKit.DaedalusVm.Push[T] (T value) (at <44b35b083ac3477f98a25f9fe0b5f0ad>:0)
ZenKit.DaedalusVm+<>c__DisplayClass70_0`2[TR,TP0].<RegisterExternal>b__0 (ZenKit.DaedalusVm vm) (at <44b35b083ac3477f98a25f9fe0b5f0ad>:0)
ZenKit.DaedalusVm._nativeExternalCallbackHandler (System.IntPtr ctx, System.UIntPtr vm) (at <44b35b083ac3477f98a25f9fe0b5f0ad>:0)
(wrapper native-to-managed) ZenKit.DaedalusVm._nativeExternalCallbackHandler(intptr,uintptr)
ZenKit.DaedalusVm.Call (System.Int32 symId) (at <44b35b083ac3477f98a25f9fe0b5f0ad>:0)

Justification:

OpenGothic also leverages null as return values for (e.g.) Hlp_GetNpc().
In this B_AssessSc() Daedalus method, the author checks if the player is the rockefeller hero (test hero with many items in bag etc.). The instance is only set, if the player set it in Gothic.ini::playerInstanceName=PC_HERO|PC_ROCKEFELLER - Of course, we could "fake" an empty npcInstance, but this entry should only be available when ini tells us so.

Update:
I experienced the same behaviour for other external calls, which assume a return object, too.
e.g. Npc_GetEquippedMeleeWeapon()
FUNC C_Item Npc_GetEquippedMeleeWeapon 	(VAR C_NPC n0 ) { };

Hint: Unsupported type: [nothing] seems to mean the null return from call as in this example we had no Melee Weapon equipped.
InvalidOperationException: Unsupported type:
ZenKit.DaedalusVm.Push[T] (T value) (at <44b35b083ac3477f98a25f9fe0b5f0ad>:0)
ZenKit.DaedalusVm+<>c__DisplayClass70_0`2[TR,TP0].<RegisterExternal>b__0 (ZenKit.DaedalusVm vm) (at <44b35b083ac3477f98a25f9fe0b5f0ad>:0)
ZenKit.DaedalusVm._nativeExternalCallbackHandler (System.IntPtr ctx, System.UIntPtr vm) (at <44b35b083ac3477f98a25f9fe0b5f0ad>:0)
(wrapper native-to-managed) ZenKit.DaedalusVm._nativeExternalCallbackHandler(intptr,uintptr)
ZenKit.DaedalusVm.Call (System.Int32 symId) (at <44b35b083ac3477f98a25f9fe0b5f0ad>:0)

I tested and integrated your fixes now. It works. 👍


Batch processing videos failing after first loop.

(all resolved, delete this thread)

Why delete instead of sharing the knowledge with everyone else? :(


fix: inner pyTripleQuotes not working properly on multi-line f-strings

Treesitter captures have probably changed since the original pyTripleQuotes function was written. There can now be multiple string_content nodes in triple-quoted f-strings which causes the current code to not highlight multi-line triple-quoted f-strings properly using the "inner" version of the operator.
Using the iy operator while the cursor in on This in the following python code reproduces the issue.
a = 'a'

b = f"""
    This is {a} multiline triple-quoted f-string
"""

Changes tested on Neovim v0.9.4, v0.10.0 and v0.10.1
Checklist

[X] Used only camelCase variable names.
[X] If functionality is added or modified, also made respective changes to the
README.md (the .txt file is auto-generated and does not need to be modified).

Thanks!


Errors when using zero3

I am trying to finetune the models using lora and zero 3 on 4 GPUs. However, when I start the experiment on multiple GPUs, training freezes from the beginning. When I try with a single GPU, it raises an error after the first iteration, check below:
  0%|                                                                                | 0/32300 [00:00<?, ?it/s]/home/ashahin/.local/lib/python3.9/site-packages/torch/utils/checkpoint.py:429: UserWarning: torch.utils.checkpoint: please pass in use_reentrant=True or use_reentrant=False explicitly. The default value of use_reentrant will be updated to be False in the future. To maintain current behavior, pass use_reentrant=True. It is recommended that you use use_reentrant=False. Refer to docs for more details on the differences between the two variants.
  warnings.warn(
Invalidate trace cache @ step 414: expected module 2612, but got module 2248
{'loss': 3.623, 'learning_rate': 1.0319917440660473e-06, 'epoch': 0.0}
  0%|                                                                                | 1/32300 [00:17<153:24:26, 17.10s/it]Invalidate trace cache @ step 414: expected module 2612, but got module 2248
Invalidate trace cache @ step 414: expected module 2248, but got module 2612
Traceback (most recent call last):
  File "/home/ashahin/codes/bakllava/llava/train/train_mem.py", line 13, in <module>
    train()
  File "/home/ashahin/codes/bakllava/llava/train/train.py", line 1094, in train
    trainer.train()
  File "/home/ashahin/.local/lib/python3.9/site-packages/transformers/trainer.py", line 1591, in train
    return inner_training_loop(
  File "/home/ashahin/.local/lib/python3.9/site-packages/transformers/trainer.py", line 1892, in _inner_training_loop
    tr_loss_step = self.training_step(model, inputs)
  File "/home/ashahin/.local/lib/python3.9/site-packages/transformers/trainer.py", line 2787, in training_step
    self.accelerator.backward(loss)
  File "/home/ashahin/.local/lib/python3.9/site-packages/accelerate/accelerator.py", line 1847, in backward
    self.deepspeed_engine_wrapped.backward(loss, **kwargs)
  File "/home/ashahin/.local/lib/python3.9/site-packages/accelerate/utils/deepspeed.py", line 167, in backward
    self.engine.backward(loss, **kwargs)
  File "/home/ashahin/.local/lib/python3.9/site-packages/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
    ret_val = func(*args, **kwargs)
  File "/home/ashahin/.local/lib/python3.9/site-packages/deepspeed/runtime/engine.py", line 1955, in backward
    self.optimizer.backward(loss, retain_graph=retain_graph)
  File "/home/ashahin/.local/lib/python3.9/site-packages/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
    ret_val = func(*args, **kwargs)
  File "/home/ashahin/.local/lib/python3.9/site-packages/deepspeed/runtime/zero/stage3.py", line 2137, in backward
    self._get_param_coordinator(training=True).reset_step()
  File "/home/ashahin/.local/lib/python3.9/site-packages/deepspeed/runtime/zero/partitioned_param_coordinator.py", line 203, in reset_step
    raise RuntimeError(f"still have inflight params "
RuntimeError: still have inflight params [{'id': 743, 'status': 'AVAILABLE', 'numel': 2097152, 'ds_numel': 2097152, 'shape': (1024, 1, 8, 16, 16), 'ds_shape': (1024, 1, 8, 16, 16), 'requires_grad': False, 'grad_shape': None, 'persist': False, 'active_sub_modules': set(), 'ds_tensor.shape': torch.Size([2097152])}, {'id': 749, 'status': 'AVAILABLE', 'numel': 1048576, 'ds_numel': 1048576, 'shape': (1024, 1024), 'ds_shape': (1024, 1024), 'requires_grad': False, 'grad_shape': None, 'persist': False, 'active_sub_modules': set(), 'ds_tensor.shape': torch.Size([1048576])}, {'id': 753, 'status': 'AVAILABLE', 'numel': 1048576, 'ds_numel': 1048576, 'shape': (1024, 1024), 'ds_shape': (1024, 1024), 'requires_grad': False, 'grad_shape': None, 'persist': False, 'active_sub_modules': set(), 'ds_tensor.shape': torch.Size([1048576])}, {'id': 757, 'status': 'AVAILABLE', 'numel': 4194304, 'ds_numel': 4194304, 'shape': (4096, 1024), 'ds_shape': (4096, 1024), 'requires_grad': False, 'grad_shape': None, 'persist': False, 'active_sub_modules': set(), 'ds_tensor.shape': torch.Size([4194304])}, {'id': 747, 'status': 'AVAILABLE', 'numel': 1048576, 'ds_numel': 1048576, 'shape': (1024, 1024), 'ds_shape': (1024, 1024), 'requires_grad': False, 'grad_shape': None, 'persist': False, 'active_sub_modules': set(), 'ds_tensor.shape': torch.Size([1048576])}, {'id': 751, 'status': 'AVAILABLE', 'numel': 1048576, 'ds_numel': 1048576, 'shape': (1024, 1024), 'ds_shape': (1024, 1024), 'requires_grad': False, 'grad_shape': None, 'persist': False, 'active_sub_modules': set(), 'ds_tensor.shape': torch.Size([1048576])}]

Any idea how to solve this?

Closing the issue as the problem has been solved by updating other packages in my environment and it's not related to bakllava.

Same question How do you fix it?

Closing the issue as the problem has been solved by updating other packages in my environment and it's not related to bakllava.

Same question How do you fix it?

In my code, I had a for loop that I suspected was the issue when using zero 3. Number of iterations in the loop varied depending on each training sample, so some processes finishes while the others are waiting. Fixing the for loop to have the same number of iterations across all samples solved it. Not sure if this makes sense but it was enough to get my code to run.


seperate main.js and vendor.js files in webpack:prod

I'm submitting a ...
[ ] feature request

feature request *
seperate files for main.js and vendor.js in webpack:prod build

current behavior
main.js and vendor.js in webpack:prod build are compiled in same file

What is the expected behavior?
seperate files for main.js and vendor.js in webpack:prod build

What is the motivation / use case for changing the behavior?
User should not be forced to download new vendor.js file on each deployment as it hardly changes.

Please tell us about your environment:

Angular version: ~2.4.3
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

@shaikhspear16 that's not what happens in my build....

Also, it's not a feature request but a bug, if any...
Try to clone a fresh copy of the repo and see

I worked with a fresh copy.


Add more integration tests

This branch begins adding additional integration tests. In particular,
it adds the following:

Additional "hello world" integration tests with request bodies,
response bodies, and request and response bodies,
A tests/client.rs module to contain tests for "real" clients with
mocked-out servers, and,
A tests/support.rs module containing utility code factored out of
the server tests, which are used in both sets of tests.

I plan to add additional tests in subsequent PRs, but I thought it
would be best to get these changes merged now.
Test coverage (as reported by tarpaulin):
Before:
$ cargo tarpaulin -p tests --exclude-files tests/* --exclude-files tower-balance/*

Coverage Results
Tested/Total Lines:
src/body.rs: 4/28
src/buf.rs: 13/15
src/client/background.rs: 0/8
src/client/connect.rs: 0/34
src/client/connection.rs: 0/95
src/flush.rs: 35/55
src/recv_body.rs: 2/44
src/server/mod.rs: 57/158
src/service.rs: 0/14

24.61% coverage, 111/451 lines covered
Tarpaulin finished

After:
$ cargo tarpaulin -p tests --exclude-files tests/* --exclude-files tower-balance/*

Coverage Results
Tested/Total Lines:
src/body.rs: 4/28
src/buf.rs: 13/15
src/client/background.rs: 8/8
src/client/connect.rs: 16/34
src/client/connection.rs: 28/95
src/flush.rs: 35/55
src/recv_body.rs: 24/44
src/server/mod.rs: 57/158
src/service.rs: 0/14

41.02% coverage, 185/451 lines covered
Tarpaulin finished

Coverage diff: +16.41%
Signed-off-by: Eliza Weisman<EMAIL_ADDRESS>

Note that I mention some coverage percentages in the descriptions of commits in this branch that are higher than those in the PR description. Those percentages are incorrect; I had forgotten to configure tarpaulin to ignore coverage measurements from the actual test code (which is necessary since the tests live in their own crate).

@carllerche 8d0e6a9b37fbe7c62b240c9ac33db6518d951bf2 changes the tests to join the mock with the client/server future and block_on the joined future, is that sufficient? I'm assuming that  block_on panicking if the blocked task panics is correct?


Made Built-In-Contract components top sticky

hey @Chakravarthy7102 thanks for the PR, we actually just removed the "built in" contract deploy form as we're moving away from the distinction between "built in" and "published" contracts - we appreciate your PR regardless but since it now targets an no longer existing part of the flow I'll go ahead and close it

hey @Chakravarthy7102 thanks for the PR, we actually just removed the "built in" contract deploy form as we're moving away from the distinction between "built in" and "published" contracts - we appreciate your PR regardless but since it now targets an no longer existing part of the flow I'll go ahead and close it

Hi @jnsdls , since there are no much of  good first issues  to work on, but I would love to contribute on thirdweb projects would you please guide me with any other issues that I can contribute to?


fix(backend): Fix query to execute contract info

Resolves #878

I still don't see contract info on near account. Should I?

don't see

No, because we don't deploy backend on preview. You can only see it on local machine

@luixo Just for the reference, I checked it on localhost:


Patch 1

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The top level meta key patches is unexpected

For recipe:

Old-style Python selectors (py27, py34, py35, py36) are deprecated. Instead, consider using the int py. For example: # [py>=36]. See lines [17]
License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Old-style Python selectors (py27, py34, py35, py36) are deprecated. Instead, consider using the int py. For example: # [py>=36]. See lines [15]
License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.


LibGfx: Set opacity for BGRx8888 bitmaps in BMPLoader

In case the original bitmap has no alpha channel
(BitmapFormat::BGRx8888), we need to set the alpha value of very pixel to opaque (255).
Second solution:
Previously, when loading a 24 bit BMP image the alpha bytes were
uninitialized. This made the image look (mostly) transparent when it is
passed to Skia as BitmapFormat::BGRA8888.
Now, the alpha bytes are set to opaque (255) by BMPImageDecoderPlugin.
Fixes #1132

Superseded by #2132


Add CustomLogger to prevent IDE lint warnings when calling log.trace.

Closes bot#1830.
Implements the proposed CustomLogger class to fix the lint warnings.

here is the CustomLogger class, so anyone reviewing this doesn't have to scroll to find it.

Here is the CustomLogger class, so anyone reviewing this doesn't have to scroll to find it.

There's also the get_logger util function (entire file is new)
All other changes are removing the old monkeypatch trace or updating imports from
import logging

log = logging.getLogger(__name__)

to
from bot.utils.logging import get_logger

log = get_logger(__name__)

NB: Linting & Test currently failing due to maintenance on the coveralls API. Will rerun checks later :+1:

NB: The getLogger in  bot.utils.members will also need to be migrated once #1837 has been merged.


Create a new temporary/restricted database for users that fork template applications

When a user forks a template application, we want to also create a separate database just for this user, that they can use to play around with the app to learn and perhaps, even share.
The following must be true for this database:

The user should be able to export all data on this database, when they want to.
The user only has permissions to run SELECT, INSERT, UPDATE and DELETE queries, and nothing else.
The database tables will be limited in terms of row count, like 100 or 1000.
The database comes with an expiry date, on which the database will be deleted.

As a prudent step before this, perhaps we can take a dump of all data in this database and email it to that user.

Additionally, there should be clear communication to the user regarding the above limitations.

@sharat87 Currently the way this is implemented is that all users share mock DB, which resets after every n hours. For now I think it is good enough.
Other than this we are rethinking datasources in templates here
I think we can close this. Let me know if you disagree.

@rahulbarwal, feel free to close. This is now a very old issue, created when we had much less focus on templates. 🙂


Scheduler Warnings/Errors

Create a set of warnings and errors for the scheduler and system services similar to the ones for the nodes and add them to the status API.
Also add the scheduler state (and service state) the status API. Consider a degraded and/or a stopped/blocked state for the scheduler.
Change node state order to have scheduler stopped state come after node paused state.
Make a warning/error for when vault is closed and stop logging exception stacktraces when it is closed.

Dismissing the degraded/stopped/blocked state for the scheduler, all other issues are addressed in #1507 and other PRs.


[boringyuri-api]: Add support of a constant @EnumParam

Java Enum is a constant type supported by annotations. Add support of constant query params based on enum type similar to @StringParam or @LongParam.

It's not possible to define an annotation with an abstract enum type as a parameter. Enums are allowed in annotation parameters, but they must be concrete implementations of Enum. So there is no way to create an annotation like this:
public @interface EnumParam {
    /**
     * Query parameter name.
     */
    String name();

    /**
     * Query parameter value.
     */
    Enum<?> value(); // <--  this won't compile
}

So closing the issue.


verilator check doesn't print anything

Hello!
I have an issue, while running 'make' for verifing the IOMMU IP. Verilator 4.110 runs, but doesn't print anything. Is that okay?

The Makefile only makes lint checks. As far as I remember, it won't output something if everything is OK.
If you want to play with the IOMMU, it may be interesting to go through their demo where IOMMU is integrated in a CVA6: https://github.com/zero-day-labs/riscv-iommu-demo
(btw, I'm making experiments with this repo :wink:)

okay thanks!
Could I use the CVA6: https://github.com/zero-day-labs/riscv-iommu-demo with verilator?

There's an feat/iommu branch in their CVA6 fork (https://github.com/zero-day-labs/cva6/tree/feat/iommu). Not sure if this branch is fully functional.

Okay thank you very much


[Disk Manager] add tests for "incremental" table

@leftmain has the details

there's no tests for incremental table in DM (only general logic like here https://github.com/ydb-platform/nbs/blob/4775546bac548f5931bb42f9d2f356f19c1cc12b/cloud/disk_manager/internal/pkg/resources/snapshots_test.go)

сделано в рамках https://github.com/ydb-platform/nbs/issues/1636
в пре https://github.com/ydb-platform/nbs/pull/1980


Fix worker cannot be ssd only tier in docker issue

What changes were proposed in this pull request?
Fix #13601
Set alluxio ram folder only if tier 0 is not MEM and tier 0 dir is not set explicitly
Why are the changes needed?
To enable launching worker with SSD only storage through docker
Does this PR introduce any user-facing change?
No

@apc999 PTAL, validated and improved the shell code, thanks
@HelloHorizon FYI

alluxio-bot, merge this please

@HelloHorizon can you help cherry-pick to branch-2.6?

alluxio-bot, cherry-pick this to branch-2.6 please


UI changes leveraging metadata

This PR uses the metadata-addition branch to showcase some UI changes that will leverage the added metadata.
You can see the changes by visiting https://www.kouloumos.com/bitcointranscripts.github.io and using the sidebar to navigate to the transcripts.
Note: 36dd2bd81364a0dfcf5d238aa93ecf74c113cb59 is temporary, just to allow the build of the demo website on my own branch.
This demo-preview is a bit buggy when trying to display the taxonomy pages, but you can do that directly from:

https://www.kouloumos.com/bitcointranscripts.github.io/speakers/
https://www.kouloumos.com/bitcointranscripts.github.io/tags/
https://www.kouloumos.com/bitcointranscripts.github.io/categories/

I appreciate the speakers page but I don't think that adding the dates is much of an upgrade across the board. Most conference pages will look similar to this:

It makes more sense for a podcast or a meetup, but conferences happen within a day or two and mostly have subfolders, so the dates are of much use in an index view like this:

So maybe you display by category?

I appreciate the speakers page but I don't think that adding the dates is much of an upgrade across the board
So maybe you display by category?

I get it, although I don't believe that this adds much noise. I've tried to conditionally display date if tag != conference but it has some issues with transcripts in the Misc category that also have this tag. The other solution is to add metadata on the conference page and display by that. This is much cleaner. I'll open a PR adding this field to all conferences.
---
title: TABConf
conference: True
---

Talking about conferences metadata, that's also a place that we could add their dates and some more info (videos?) if we wanted to.
I've also changed the way taxonomies are displayed (verifiable using the links in the PR description):

Count of transcripts for each taxonomy

Inside a speakers page more info are display for each transcript (in Hugo terminology, a specific speaker is a "term" in the "speakers" taxonomy)

Ready to revisit this now that the content updates are in. Thanks for your patience.

Ready to revisit this now that the content updates are in. Thanks for your patience.

I was actually thinking to make a design brief and try to get some input from the Bitcoin Design community on the redesign.

I was actually thinking to make a design brief and try to get some input from the Bitcoin Design community on the redesign.

My thought around the design brief was mostly to showcase the "needs {transcript, proof reading}" functionality and get some input on how to display it. I never manage to do that, and as a redesign is not actually needed, I think that this can work as it is. Having the implementation there might make it easier to get design feedback in the future.

Ready to revisit this now that the content updates are in. Thanks for your patience.

Ready to revisit. I also updated the PR description with an overview of the changes.

With the new transcript queuing process nearing completion, I believe it's a good opportunity to revisit this PR. I've updated the PR description incorporating the latest changes.
Primarily, the 4th commit introduces changes aligned with the new queuing process. Additionally, I've utilized this opportunity to enhance the UI, aiming for improved discoverability and smoother navigation on the website.
Your input and review on these updates would be greatly appreciated. @adamjonas @aassoiants


Added noRequestBody function to check that HTTP GET and DELETE methods do not contain a requestBody

Requested from here: https://github.com/daveshanley/vacuum/issues/582

@daveshanley bump :)

Will get to this ASAP, apologies for my slowness.


build image from Dockerfile, not pulling from registry

Hi there!
Thank you guys for a great tool for testing purposes.
My question is gonna be about building images. As far as you know docker-compose includes feature that allows you to create image from Dockerfile, not only pulling a ready-to-use image from registry.
Right now I'm playing with containerlab and wondering is such feature presented within it? I can't find any reference to it...
Have a nice day,
Matthew

Hi @mdraevich,
the only dockerfile we expose in our repo is this one https://github.com/srl-labs/containerlab/blob/main/goreleaser.dockerfile
It is used in our build pipeline to build and upload the image to the registry. As you can see, it uses an apk package that is built by goreleaser.
If you need to build a binary and create an image yourself, then you will have to create your own dockerfile with build steps. That should be pretty straightforward I think.

Hi @mdraevich, the only dockerfile we expose in our repo is this one https://github.com/srl-labs/containerlab/blob/main/goreleaser.dockerfile
It is used in our build pipeline to build and upload the image to the registry. As you can see, it uses an apk package that is built by goreleaser. If you need to build a binary and create an image yourself, then you will have to create your own dockerfile with build steps. That should be pretty straightforward I think.

Dear @hellt
Sorry, I believe I messed up you with my question. Let me clarify a bit.
I'm talking about building images for nodes in topology.yml. As far as I see topology nodes can use already built images only.
As an example, docker-compose file allows you to create images from Dockerfile right inside docker-compose.yml file.
Example of how it may look:
topology:
  nodes:
    ready-to-use-image:
      kind: linux
      image: python:3.8

    use-dockerfile-to-build-new-image:
      kind: linux
      build:
        context: inside-folder/
        dockerfile: node.Dockerfile

Build definition may be implemented according to docker-compose.
Hope it's clear right now,
Matthew

ah, I see
yes, this is not supported in containerlab


Timezone Support - Mismatch from UTC and Local Timezone in frontend?

An example shown above.
This was a query that was just logged. Looks like the date/time in the database/backend is in UTC, but the front-end seems to be doing the 'difference' calculation using the browsers local time. (In this case, I'm in Australia, so a timezone of +10)
Wouldn't hurt to show the actual time, rather than just the 'hours ago', and in this case, I don't think any issue in just showing " UTC" after the time/date, without having to go to the trouble of implementing the whole "timezone support"
Very minor one, not a show stopper by any means :)

I've opened a Pull Request with this change timeSince() calculates in UTC, and added the timestamp (in UTC) as a title tag/hover for the offset display in the frontend.


Add usage example read-me document

I took a look at your code... I was trying to swap to your library from the Newtonsoft JSONPath implementation...
I can't work out how to do that.
How do I use your code? please can you update the readme with a worked example.. :)
Thanks,
Nigel

Thanks for that...
I'm still hitting problems running through your examples in your readme.
None of the expressions in [] seem to work (because the ScriptEvaluator is missing).
My goal is to get all the examples in your Readme working...
    class JsonPathExecutor
    {
        public JContainer Apply(string input, string path)
        {
            var source = JToken.Parse(input);
            return new JArray(ApplyObj(path, source));
        }

        private object[] ApplyObj(string path, JToken source)
        {
            var context = new JsonPathContext
            {
                ValueSystem = new JsonNetValueSystem(),
                ScriptEvaluator = EvaluateScript
            };

            return context.Select(source, path).ToArray();
        }

        private object EvaluateScript(string path, object obj, string arg3)
        {
            var p = Regex.Replace(path, "^@", "");
            var results = ApplyObj(p, obj as JToken);
            return results.Any() ? new JArray(results) : null;
        }

I'm currently doing this... but it's not working exactly...
https://bitbucket.org/nigel__thorne/jsonpathsandbox/branch/JSONPath-experiment

I can offer https://github.com/cburgmer/json-path-comparison/tree/master/implementations/dotNET_JsonPathLib


Fix AnimatableBody moving and colliding during the first frame

_update_kinematic_motion() shouldn't be called in the constructor, it's too early and causes a first call to set the body position to 0.
Then the second call to set the actual position triggers kinematic motion, colliding with objects on the path to the initial position.
Side effect of sync to physics now enabled by default.

Thanks!


Why does the PQL not run in Four Room?

Hi
Why does the PQL not run in the Four-Room environment since it has been designed to work in deterministic, episodic environments?

Hi @Saood1810,
PQL is a tabular algorithm. Therefore, I doubt it will scale to the number of state-action pairs that four rooms have -- the problem is just too big.

Thank you for the response!


Add simple pre-release tests

The release_test rake task pulls down a selection of real world Puppet
modules and runs them through the following process:

bundle install
bundle exec rake spec (to get a baseline result)
modify gemfile to point to HEAD of local rspec-puppet repo
bundle install
bundle exec rake spec (to get a result for rspec-puppet HEAD)
compare the summaries of the two rspec results and alert if different.

This process is rather time intensive and is not suitable for on-push CI, but
is instead intended to be used before pushing out a new release to increase
confidence in the new release.
The current module list includes some Puppet modules and some community
modules, targetting both Linux and Windows environments.

Coverage remained the same at 91.396% when pulling d520b7a4a7851cbfc0930e76a52beddd1961a79f on release_tests into 161ee236dae23fbcca18c849867646753ca12f46 on master.

Coverage remained the same at 91.396% when pulling d338cef975df86ce2d96a90835cf99adc411ec55 on release_tests into 161ee236dae23fbcca18c849867646753ca12f46 on master.

Coverage remained the same at 91.396% when pulling 5086f036012430fe730d43110fb6350a9149772a on release_tests into 161ee236dae23fbcca18c849867646753ca12f46 on master.

Great idea!


Zooming in/out diagram triggers "Unhandled Error" message in browser

https://user-images.githubusercontent.com/119422/163720527-15576eb2-9548-4b6e-825b-42b6bfdc5c28.mp4

Noticed that this message can also be sometimes triggered when clicking the "Info" button as well.
Just fyi this also happens in Synapse.Dashboard

I'm not able to reproduce the error. @tsurdilo could you please open up the dev console and get the complete error message form there please?
The one on the info button is probably different, if you can start a new issue with the details new time you get it, it would be great, thanks :)

@tsurdilo any update ?

@JBBianchi with latest update i get it on web ui load:

@JBBianchi with latest update i get it on web ui load. hope this helps:

I think this is related to #49

I'll close this issue because it's too generic and the error shown above should be fixed now.


Switch to use codecs in components/codec

Feature Request
Since https://github.com/tikv/tikv/pull/3629 is landed, we can gradually use the new codec in components/codec instead of src/util/codec, which is more efficient and also easier to use.

I would like to start working on this

@KeeganMyers Thanks a lot! You can change code module by module to keep each PR small and easy to merge :)

hi @KeeganMyers  had you already started working on this? I'd like to take this if you don't mind , since I'm working on the new row codec, and maybe this task can give me a better understanding on the codec component and the original codec logic.

@niedhui go for it. I have gotten a bit distracted trying to find a new job and have not been able to work on this much yet. I grep'd through the codebase and it seems like ~18 files will need to be changed in total for this issue.


probably an issue with current vendoring

I tried to make a release but it had problem when building the image:
$ make binary-image
Sending build context to Docker daemon  170.5MB
Step 1/7 : FROM golang:alpine
 ---> 00371bbb49d5
Step 2/7 : MAINTAINER "Cuong Manh Le<EMAIL_ADDRESS> ---> Running in 079fb66eb765
 ---> 473d417e13cb
Removing intermediate container 079fb66eb765
Step 3/7 : RUN apk update &&     apk add git build-base &&     rm -rf /var/cache/apk/* &&     mkdir -p "$GOPATH/src/github.com/bitnami-labs/kubewatch"
 ---> Running in 06f757787adc
fetch http://dl-cdn.alpinelinux.org/alpine/v3.4/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.4/community/x86_64/APKINDEX.tar.gz
v3.4.6-270-g88aa8f7 [http://dl-cdn.alpinelinux.org/alpine/v3.4/main]
v3.4.6-160-g14ad2a3 [http://dl-cdn.alpinelinux.org/alpine/v3.4/community]
OK: 5981 distinct packages available
(1/25) Upgrading musl (1.1.14-r14 -> 1.1.14-r16)
(2/25) Installing binutils-libs (2.26-r1)
(3/25) Installing binutils (2.26-r1)
(4/25) Installing gmp (6.1.0-r0)
(5/25) Installing isl (0.14.1-r0)
(6/25) Installing libgomp (5.3.0-r0)
(7/25) Installing libatomic (5.3.0-r0)
(8/25) Installing libgcc (5.3.0-r0)
(9/25) Installing pkgconf (0.9.12-r0)
(10/25) Installing pkgconfig (0.25-r1)
(11/25) Installing mpfr3 (3.1.2-r0)
(12/25) Installing mpc1 (1.0.3-r0)
(13/25) Installing libstdc++ (5.3.0-r0)
(14/25) Installing gcc (5.3.0-r0)
(15/25) Installing make (4.1-r1)
(16/25) Installing musl-dev (1.1.14-r16)
(17/25) Installing libc-dev (0.7-r0)
(18/25) Installing fortify-headers (0.8-r0)
(19/25) Installing g++ (5.3.0-r0)
(20/25) Installing build-base (0.4-r1)
(21/25) Installing libcurl (7.58.0-r0)
(22/25) Installing expat (2.2.0-r1)
(23/25) Installing pcre (8.38-r1)
(24/25) Installing git (2.8.6-r0)
(25/25) Upgrading musl-utils (1.1.14-r14 -> 1.1.14-r16)
Executing busybox-1.24.2-r12.trigger
OK: 173 MiB in 35 packages
 ---> 9d2552ea088b
Removing intermediate container 06f757787adc
Step 4/7 : ADD . "$GOPATH/src/github.com/bitnami-labs/kubewatch"
 ---> 2a79ff9dae97
Step 5/7 : RUN cd "$GOPATH/src/github.com/bitnami-labs/kubewatch" &&     CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a --installsuffix cgo --ldflags="-s" -o /kubewatch
 ---> Running in e105944a79c9
# github.com/bitnami-labs/kubewatch/vendor/k8s.io/apimachinery/pkg/util/net
vendor/k8s.io/apimachinery/pkg/util/net/http.go:279: req.URL.Hostname undefined (type *url.URL has no field or method Hostname)
The command '/bin/sh -c cd "$GOPATH/src/github.com/bitnami-labs/kubewatch" &&     CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a --installsuffix cgo --ldflags="-s" -o /kubewatch' returned a non-zero code: 2
make: *** [builder-image] Error 2

It probably happens because of vendoring. Will investigate more tomorrow morning and try to cut a release.

Hello there! Any luck fixing this issue? Our team is passionately looking forward to using kubewatch for Jobs but this problem turns out to be the blocker...

I'm having the same problem on "Debian GNU/Linux 9 (stretch)", I did
apt-get install golang-go
go get github.com/kubernetes-incubator/cri-tools/cmd/crictl

which resulted in
go/src/github.com/kubernetes-incubator/cri-tools/vendor/k8s.io/apimachinery/pkg/util/net/http.go:279: req.URL.Hostname undefined (type *url.URL has no field or method Hostname)

Running init with address specified as below
kubeadm init --apiserver-advertise-address=<server public ip address>

allowed the process to succeed in my case.

I've done this on a few different distros and everything appears to be fine. This may be more of an environmental issue, but if you still think this is an application with this specific code, feel free to reopen this issue.


Fixes #2464 Create Azure Cloud Service project error

Fixes #2464 Create Azure Cloud Service project error
Explicitly checks for an installed package rather than inferring it from a loadable type.
Removes dead code

👍


Feature request: Watch mode

So I have my docker compose file as such...
services:
  mock-api:
    container_name: openapi_mock
    image: muonsoft/openapi-mock
    volumes:
      - ./schema:/schema
    environment:
      OPENAPI_MOCK_SPECIFICATION_URL: '/schema/schema.json'
    ports:
      - '3333:8080'

Is it possible for the container to watch for file changes if the url is a local file?  This would greatly help as we use this container in the dev cycle.

Although a dev-mode would be beneficial, you could use something like this, too:
watch -n 10 -d docker-compose up -d --build --force-recreate

watch can be installed on macOS with Brew (brew install watch). I think it's installed by default on Ubuntu and other distros (although, I haven't checked).


Feature Request: Remove Concept of Inscription Numbers Entirely

As an Ordinal Maxi, the only number I care about is Sat Number. There's no need for "inscription numbers".
Bug solved!

ACK
It's a generally unhelpful abstract, and there are already several less abstract ways to go about age/rarity (sat number, block height, etc)

Bug solved!

I have been surprised (and disappointed) with the "winning" narrative of not wanting to solve the inscription number bugs, because it seems is just for the sake of collectors ($$$). IS STILL EARLY! Any early adopter should know the risks that come with being early.
ACK with the idea of just getting rid of the prioritization of inscription serial numbers. They could still be "used" in some way, like acting as a redirect to the TXIDiN of the actual (latest) indexed inscription position.

inscription number is more storage efficient for internal indexing then inscription id, and inscription number is in the order of inscription creation, so definitely very important for internal indexing. But if exposing in UI, that's a different issue.

It looks like those fuckers are here to stay 😂


Fix query params being empty

Fixes https://github.com/sveltejs/kit/issues/631
The query params are not parsed correctly, because URL does not return query - instead it returns the desired searchParams!
Tested here - https://sveltekit-vercel.vercel.app/queryparams?q=pizza

The previous job was successful, I could imagine that somekind of race condition happened because I force pushed to the branch 🤷♂

If this is broken, then I'm guessing adapter-node might be as well:
https://github.com/sveltejs/kit/blob/80241789644ca2add2fd03bc2a49e3c91689d472/packages/adapter-node/src/server.js#L40
Strange that this would only be reported now...

Ah, nevermind. adapter-node calls parse which returns a legacy URL which defines query. We should probably change adapter-node as well so that it doesn't use the legacy URL creation method

Should I include that in this PR as well or create a new one?

I've updated this PR


Mother of all Merges

I went ahead and cleaned up the repo and scoured all the forks I could find to merge them back into the mothership. Passing these changes upstream.
I don't like fragmenting repos like this, but I'll try to keep things together until the main repo looks alive again, if there are people want to send any PR's to me.

And CI failed because I removed it. I don't see a need for it right now.

Thank you, doing god's work.

Changes were merged.


Removed log in windows event laucher

What does this PR do?
Removed log in windows event laucher
Motivation
Start tailing windows event log log message appears all the time, let's remove it and just keep Starting windows event log tailing for channel %s query %s for each tailer.

@arbll could we merge this for rc.6 if we do a rc.6?


Remote Config does not work with custom SQLite3 libraries

Description
Currently SQLITE_OPEN_FILEPROTECTION_COMPLETEUNTILFIRSTUSERAUTHENTICATION is passed to sqlite3_open_v2(...) in - (void)createOrOpenDatabase in RCNConfigDBManager.m.
If an app eschews the standard Apple sqlite3 library shipped on iOS and uses their own custom sqlite3 build with source directly from the SQLite3 repo, the SQLITE_OPEN_FILEPROTECTION_COMPLETEUNTILFIRSTUSERAUTHENTICATION flag does not exist as Apple has only implemented this in their version of sqlite3 and not contributed it back to the main repo.
We cannot use the Apple's sqlite3 implementation as we need additional spatial querying capabilities built into the library for our application. It would be useful if Remote Config checked if the flag exists before using it.
e.g.:
#ifdef SQLITE_OPEN_FILEPROTECTION_COMPLETEUNTILFIRSTUSERAUTHENTICATION
    int flags = SQLITE_OPEN_CREATE | SQLITE_OPEN_READWRITE |
                SQLITE_OPEN_FILEPROTECTION_COMPLETEUNTILFIRSTUSERAUTHENTICATION |
                SQLITE_OPEN_FULLMUTEX;

#else
    int flags = SQLITE_OPEN_CREATE | SQLITE_OPEN_READWRITE |
                SQLITE_OPEN_FULLMUTEX;
#endif

Reproducing the issue
No response
Firebase SDK Version
10.6.0
Xcode Version
14.2.0
Installation Method
Swift Package Manager
Firebase Product(s)
Remote Config
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

Thanks for the report. I'm not sure that the suggested change would redirect the Remote Configuration to point to alternative headers for the sqlite3. Also, there may be other issues since we only test with the system sqlite3.

In the context of Package.swift builds, It does indeed look for other headers because there is nothing in the linkerSettings for  .linkedLibrary("sqlite3"), on that target. I verified that my headers were used by setting a flag in my local fork of Firebase and using a symbol only available from my sqlite3 package. I do realize that this does not apply to any of the other build systems you support.
I can continue with my fork, but I thought I would point this out as it is a blocker for Firebase adoption for anyone doing spatialite and geos stuff with iOS, and probably many other applications that require sqlite features not in Apples platform build.

Thanks. We may want to consider an alternative to sqlite3 for the Remote Config implementation.

Any update here? It's been a year.

We do not have plans to allocate dev time to make this change. Would you be willing to contribute a patch based on the code posted by Aaron above?

Closed via #12548. Thanks, @cprince-foreflight, for the fix!


Add JRE utils (#11393)

What is the purpose of the change
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #11396 (69d29f6) into 3.2 (b456d23) will decrease coverage by 4.78%.
The diff coverage is 48.83%.

@@             Coverage Diff              @@
##                3.2   #11396      +/-   ##
============================================
- Coverage     69.57%   64.79%   -4.78%
+ Complexity      119       14     -105
============================================
  Files          1525     1498      -27
  Lines         82580    62538   -20042
  Branches      14719     9149    -5570
============================================
- Hits          57453    40520   -16933
+ Misses        20273    17776    -2497
+ Partials       4854     4242     -612

Impacted Files
Coverage Δ

...che/dubbo/common/utils/ConcurrentHashMapUtils.java
14.28% <0.00%> (-52.39%)
:arrow_down:

...c/main/java/org/apache/dubbo/common/utils/JRE.java
50.00% <50.00%> (ø)

...luster/router/script/ScriptStateRouterFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/dubbo/test/common/impl/GreetingServiceImpl.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...zookeeper/curator/CuratorZookeeperTransporter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../serialize/hessian2/Hessian2SerializerFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...nfigure/DubboRelaxedBinding2AutoConfiguration.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/dubbo/test/spring/SpringXmlConfigTest.java
0.00% <0.00%> (-96.16%)
:arrow_down:

...egistrycenter/context/ZookeeperWindowsContext.java
0.00% <0.00%> (-81.82%)
:arrow_down:

...istrycenter/processor/FindPidWindowsProcessor.java
5.40% <0.00%> (-81.09%)
:arrow_down:

... and 479 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix code coverage upload

Description
This PR fixes uploads to codecov. While working on https://github.com/opensearch-project/performance-analyzer/pull/736, I noticed that uploads were hitting the rate limit and failing.
==> Uploading reports
    url: https://codecov.io
    query: branch=main&commit=c643111d76f63eadd5fe1196844f96a84df530bb&build=11[47](https://github.com/opensearch-project/performance-analyzer/actions/runs/11473403760/job/31927585948#step:9:48)3403760&build_url=https%3A%2F%2Fgithub.com%2Fopensearch-project%2Fperformance-analyzer%2Factions%2Fruns%2F11473403760&name=&tag=&slug=opensearch-project%2Fperformance-analyzer&service=github-actions&flags=&pr=&job=Java+CI&cmd_args=f
->  Pinging Codecov
https://codecov.io//upload/v4?package=bash-1.0.6&token=<hidden>&package=bash-1.0.6&token=&branch=main&commit=c643111d76f63eadd5fe1196844f96a84df530bb&build=11473403760&build_url=https%3A%2F%2Fgithub.com%2Fopensearch-project%2Fperformance-analyzer%2Factions%2Fruns%2F11473403760&name=&tag=&slug=opensearch-project%2Fperformance-analyzer&service=github-actions&flags=&pr=&job=Java+CI&cmd_args=f
{"message":"Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 2501s."}
{"message":"Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 2[49](https://github.com/opensearch-project/performance-analyzer/actions/runs/11473403760/job/31927585948#step:9:50)9s."}
{"message":"Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 2497s."}
{"message":"Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 2495s."}
{"message":"Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 2492s."}
{"message":"Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 2490s."}

This PR changes the upload step to use the GHA for uploading to codecov.
Workflow where it failed: https://github.com/opensearch-project/performance-analyzer/actions/runs/11473403760/job/31927585948
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] API changes companion pull request created.
[ ] Commits are signed per the DCO using --signoff.
[ ] Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Waiting for builds to complete.

Upload still failed. Looking into it.


Redhat/centos 6

Why Bracket does not support on RedHat/CentOS 6.5 or  lower version. Is there any workaround.
Thanks,
Avdhesh

If you are talking about rpm distribution, AFAIK it is not supported.
You could give a try to the AppImage of https://github.com/zaggino/brackets-electron/releases

Ok Thanks.

Not working . It gives same error as bracket.

I  tried on CentOS version 6.

@probonopd do you know if AppImage works on CentOS version 6 ?

@ficristo AppImage itself does for sure, but the software inside a particular AppImage must be built with CentOS 6 as a target in mind, too. The Subsurface, KDevelop, and Krita applications do this, for example.

I had download from above link. My issue is that how to install Bracket on centOS6 or lower version, Would be possible or not?


[WIP] Adds xblock-utils repository code into this repository

This PR's work is in progress
Adds xblock-utils repository code into this repository
Relevant ticket: https://github.com/openedx/xblock-utils/issues/197
Implementation details:
I have added xblock-utils code in the sibling of the xblock code directory and kept xblock-utils test cases within it.
The rationale of doing it is to keep the utils code in a separate layer.
Pending
Integration tests are failing.
Needs to update the ci.yml

@farhan rather than bringing the xbolckutils module directly, I think we want to move all the relevant code into a xblock.utils module. This keeps the code in the xblocks repository cleaner in the long run but will involve a bit more complexity to the move. Can you try to update this so that the code lives under the existing xblocks structure?

@feanil Yes we can
I kept it separate for following 2 pros:

All the repos who are using xblock-utils don't need to update the imports
xblock-utils will stay as a separate layer

But if we want to keep it intact with the xblocks repo, it should reside inside
Let me refactor it.
1 more thing I am not going to add the bok-choy integration tests as we have deprecated the package

Thanks for review and sharing thoughts @ormsbee

I did not review all the files in the utils and test/utils packages because I assume they're coming straight over from xblockutils (except for the bok choy stuff that was dropped).

Yes they are coming straight from repo, no need to review all

I'm also assuming that some manual testing is being done to make sure that things still work when we have a mix of XBlocks where some XBlocks are using xblock.utils and some are using xblockutils and we're installing both libraries at the same time–since this mode is likely to be very common for a while.
Otherwise, it looks good to me.

Yes, some manual testing is done on DoneXBlock and repo is working fine with different pip package cases:

Screen Shots

@feanil I have added the changelog and merged the docs PR into this PR as well.
Please review the change logs, are we good to merge it and release now?

@farhan I think you need to add a changelog entry to summarize that this is a move of code from the xblock-utils repository and then I think this is good to merge and release.

https://github.com/openedx/XBlock/pull/669/files#diff-2c623f3c6a917be56c59d43279244996836262cb1e12d9d0786c9c49eef6b43c

@feanil
Change in changelog.rst is done
I have organized the commits as well and will merge via Merge pull request
All good?


How to resize dialog window

Hi,
I'm using a custom layout with your library.. It works fine but the dialog is too much big. It's almost edge to edge. How to give it a normal size? Thanks
Device: Galaxy S6
Android: 6.0.1
Material Dialogs: <IP_ADDRESS> core

You can't. This library is based on AppCompat dialogs.