Keep pbr dependency in setup_requires (fixes #308, follows #309)

Follow up on #309 to keep build time dependency pbr in setup_requires and remove the following warning:
./.venv/bin/python setup.py --help
/usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'pbr'
  warnings.warn(msg)
Common commands: (see '--help-commands' for more)

  setup.py build      will build the package underneath 'build/'
  setup.py install    will install the package

Global options:
[...]

Codecov Report

Merging #310 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #310   +/-   ##
=======================================
  Coverage   96.31%   96.31%
=======================================
  Files          11       11
  Lines         814      814
  Branches      100      100
=======================================
  Hits          784      784
  Misses         10       10
  Partials       20       20

Continue to review full report at Codecov.

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

Hi @ajkavanagh, you should probably consider including this fix before releasing 2.2.7.

Without this change
# rm -rf .venv/
# python3 -m venv .venv
# ./.venv/bin/pip install wheel
Collecting wheel
  Using cached https://files.pythonhosted.org/packages/81/30/e935244ca6165187ae8be876b6316ae201b71485538ffac1d718843025a9/wheel-0.31.1-py2.py3-none-any.whl
Installing collected packages: wheel
Successfully installed wheel-0.31.1
# ./.venv/bin/python setup.py --help
Common commands: (see '--help-commands' for more)

  setup.py build      will build the package underneath 'build/'
  setup.py install    will install the package

Global options:
  --verbose (-v)      run verbosely (default)
  --quiet (-q)        run quietly (turns verbosity off)
  --dry-run (-n)      don't actually do anything
  --help (-h)         show detailed help message
  --no-user-cfg       ignore pydistutils.cfg in your home directory
  --command-packages  list of packages that provide distutils commands

Information display options (just display information, ignore any commands)
  --help-commands     list all available commands
  --name              print package name
  --version (-V)      print package version
  --fullname          print <package name>-<version>
  --author            print the author's name
  --author-email      print the author's email address
  --maintainer        print the maintainer's name
  --maintainer-email  print the maintainer's email address
  --contact           print the maintainer's name if known, else the author's
  --contact-email     print the maintainer's email address if known, else the
                      author's
  --url               print the URL for this package
  --license           print the license of the package
  --licence           alias for --license
  --description       print the package description
  --long-description  print the long package description
  --platforms         print the list of platforms
  --classifiers       print the list of classifiers
  --keywords          print the list of keywords
  --provides          print the list of packages/modules provided
  --requires          print the list of packages/modules required
  --obsoletes         print the list of packages/modules made obsolete

usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

# ./.venv/bin/pip install .
Processing /home/pdecat/workspaces/thirdparty/pylxd
Collecting cryptography!=1.3.0,>=1.0 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/fa/f4/3cde3604972dfa2b0fea85b9711948bb4fb70ab64095322aef35071bd254/cryptography-2.2.2-cp34-abi3-manylinux1_x86_64.whl
Collecting pbr>=1.6 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/b3/5d/c196041ffdf3e34ba206db6d61d1f893a75e1f3435699ade9bd65e089a3d/pbr-4.0.4-py2.py3-none-any.whl
Collecting python-dateutil>=2.4.2 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/cf/f5/af2b09c957ace60dcfac112b669c45c8c97e32f94aa8b56da4c6d1682825/python_dateutil-2.7.3-py2.py3-none-any.whl
Collecting requests!=2.12.0,!=2.12.1,!=2.8.0,>=2.5.2 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/49/df/50aa1999ab9bde74656c2919d9c0c085fd2b3775fd3eca826012bef76d8c/requests-2.18.4-py2.py3-none-any.whl
Collecting requests-toolbelt>=0.8.0 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/97/8a/d710f792d6f6ecc089c5e55b66e66c3f2f35516a1ede5a8f54c13350ffb0/requests_toolbelt-0.8.0-py2.py3-none-any.whl
Collecting requests-unixsocket>=0.1.5 (from pylxd==2.2.7.dev23)
Collecting six>=1.9.0 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/67/4b/141a581104b1f6397bfa78ac9d43d8ad29a7ca43ea90a2d863fe3056e86a/six-1.11.0-py2.py3-none-any.whl
Collecting ws4py!=0.3.5,>=0.3.4 (from pylxd==2.2.7.dev23)
Collecting cffi>=1.7; platform_python_implementation != "PyPy" (from cryptography!=1.3.0,>=1.0->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/6d/c0/47db8f624f3e4e2f3f27be03a93379d1ba16a1450a7b1aacfa0366e2c0dd/cffi-1.11.5-cp36-cp36m-manylinux1_x86_64.whl
Collecting asn1crypto>=0.21.0 (from cryptography!=1.3.0,>=1.0->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/ea/cd/35485615f45f30a510576f1a56d1e0a7ad7bd8ab5ed7cdc600ef7cd06222/asn1crypto-0.24.0-py2.py3-none-any.whl
Collecting idna>=2.1 (from cryptography!=1.3.0,>=1.0->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/27/cc/6dd9a3869f15c2edfab863b992838277279ce92663d334df9ecf5106f5c6/idna-2.6-py2.py3-none-any.whl
Collecting urllib3<1.23,>=1.21.1 (from requests!=2.12.0,!=2.12.1,!=2.8.0,>=2.5.2->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/63/cb/6965947c13a94236f6d4b8223e21beb4d576dc72e8130bd7880f600839b8/urllib3-1.22-py2.py3-none-any.whl
Collecting certifi>=2017.4.17 (from requests!=2.12.0,!=2.12.1,!=2.8.0,>=2.5.2->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/7c/e6/92ad559b7192d846975fc916b65f667c7b8c3a32bea7372340bfe9a15fa5/certifi-2018.4.16-py2.py3-none-any.whl
Collecting chardet<3.1.0,>=3.0.2 (from requests!=2.12.0,!=2.12.1,!=2.8.0,>=2.5.2->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/bc/a9/01ffebfb562e4274b6487b4bb1ddec7ca55ec7510b22e4c51f14098443b8/chardet-3.0.4-py2.py3-none-any.whl
Collecting pycparser (from cffi>=1.7; platform_python_implementation != "PyPy"->cryptography!=1.3.0,>=1.0->pylxd==2.2.7.dev23)
Installing collected packages: pycparser, cffi, asn1crypto, idna, six, cryptography, pbr, python-dateutil, urllib3, certifi, chardet, requests, requests-toolbelt, requests-unixsocket, ws4py, pylxd
  Running setup.py install for pylxd ... done
Successfully installed asn1crypto-0.24.0 certifi-2018.4.16 cffi-1.11.5 chardet-3.0.4 cryptography-2.2.2 idna-2.6 pbr-4.0.4 pycparser-2.18 pylxd-2.2.7.dev23 python-dateutil-2.7.3 requests-2.18.4 requests-tool
belt-0.8.0 requests-unixsocket-0.1.5 six-1.11.0 urllib3-1.22 ws4py-0.5.1

With this change
# rm -rf .venv/
# python3 -m venv .venv

# ./.venv/bin/pip install wheel
Collecting wheel
  Using cached https://files.pythonhosted.org/packages/81/30/e935244ca6165187ae8be876b6316ae201b71485538ffac1d718843025a9/wheel-0.31.1-py2.py3-none-any.whl
Installing collected packages: wheel
Successfully installed wheel-0.31.1
# ./.venv/bin/python setup.py --help
Common commands: (see '--help-commands' for more)

  setup.py build      will build the package underneath 'build/'
  setup.py install    will install the package

Global options:
  --verbose (-v)      run verbosely (default)
  --quiet (-q)        run quietly (turns verbosity off)
  --dry-run (-n)      don't actually do anything
  --help (-h)         show detailed help message
  --no-user-cfg       ignore pydistutils.cfg in your home directory
  --command-packages  list of packages that provide distutils commands

Information display options (just display information, ignore any commands)
  --help-commands     list all available commands
  --name              print package name
  --version (-V)      print package version
  --fullname          print <package name>-<version>
  --author            print the author's name
  --author-email      print the author's email address
  --maintainer        print the maintainer's name
  --maintainer-email  print the maintainer's email address
                      author's
  --url               print the URL for this package
  --license           print the license of the package
  --licence           alias for --license
  --description       print the package description
  --long-description  print the long package description
  --platforms         print the list of platforms
  --classifiers       print the list of classifiers
  --keywords          print the list of keywords
  --provides          print the list of packages/modules provided
  --requires          print the list of packages/modules required
  --obsoletes         print the list of packages/modules made obsolete

usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

# ./.venv/bin/pip install .
Processing /home/pdecat/workspaces/thirdparty/pylxd
Collecting cryptography!=1.3.0,>=1.0 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/fa/f4/3cde3604972dfa2b0fea85b9711948bb4fb70ab64095322aef35071bd254/cryptography-2.2.2-cp34-abi3-manylinux1_x86_64.whl
Collecting pbr>=1.6 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/b3/5d/c196041ffdf3e34ba206db6d61d1f893a75e1f3435699ade9bd65e089a3d/pbr-4.0.4-py2.py3-none-any.whl
Collecting python-dateutil>=2.4.2 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/cf/f5/af2b09c957ace60dcfac112b669c45c8c97e32f94aa8b56da4c6d1682825/python_dateutil-2.7.3-py2.py3-none-any.whl
Collecting requests!=2.12.0,!=2.12.1,!=2.8.0,>=2.5.2 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/49/df/50aa1999ab9bde74656c2919d9c0c085fd2b3775fd3eca826012bef76d8c/requests-2.18.4-py2.py3-none-any.whl
Collecting requests-toolbelt>=0.8.0 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/97/8a/d710f792d6f6ecc089c5e55b66e66c3f2f35516a1ede5a8f54c13350ffb0/requests_toolbelt-0.8.0-py2.py3-none-any.whl
Collecting requests-unixsocket>=0.1.5 (from pylxd==2.2.7.dev23)
Collecting six>=1.9.0 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/67/4b/141a581104b1f6397bfa78ac9d43d8ad29a7ca43ea90a2d863fe3056e86a/six-1.11.0-py2.py3-none-any.whl
Collecting ws4py!=0.3.5,>=0.3.4 (from pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/53/20/4019a739b2eefe9282d3822ef6a225250af964b117356971bd55e274193c/ws4py-0.5.1.tar.gz
Collecting asn1crypto>=0.21.0 (from cryptography!=1.3.0,>=1.0->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/ea/cd/35485615f45f30a510576f1a56d1e0a7ad7bd8ab5ed7cdc600ef7cd06222/asn1crypto-0.24.0-py2.py3-none-any.whl
Collecting idna>=2.1 (from cryptography!=1.3.0,>=1.0->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/27/cc/6dd9a3869f15c2edfab863b992838277279ce92663d334df9ecf5106f5c6/idna-2.6-py2.py3-none-any.whl
Collecting cffi>=1.7; platform_python_implementation != "PyPy" (from cryptography!=1.3.0,>=1.0->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/6d/c0/47db8f624f3e4e2f3f27be03a93379d1ba16a1450a7b1aacfa0366e2c0dd/cffi-1.11.5-cp36-cp36m-manylinux1_x86_64.whl
Collecting urllib3<1.23,>=1.21.1 (from requests!=2.12.0,!=2.12.1,!=2.8.0,>=2.5.2->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/63/cb/6965947c13a94236f6d4b8223e21beb4d576dc72e8130bd7880f600839b8/urllib3-1.22-py2.py3-none-any.whl
Collecting certifi>=2017.4.17 (from requests!=2.12.0,!=2.12.1,!=2.8.0,>=2.5.2->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/7c/e6/92ad559b7192d846975fc916b65f667c7b8c3a32bea7372340bfe9a15fa5/certifi-2018.4.16-py2.py3-none-any.whl
Collecting chardet<3.1.0,>=3.0.2 (from requests!=2.12.0,!=2.12.1,!=2.8.0,>=2.5.2->pylxd==2.2.7.dev23)
  Using cached https://files.pythonhosted.org/packages/bc/a9/01ffebfb562e4274b6487b4bb1ddec7ca55ec7510b22e4c51f14098443b8/chardet-3.0.4-py2.py3-none-any.whl
Collecting pycparser (from cffi>=1.7; platform_python_implementation != "PyPy"->cryptography!=1.3.0,>=1.0->pylxd==2.2.7.dev23)
Building wheels for collected packages: ws4py
  Running setup.py bdist_wheel for ws4py ... done
  Stored in directory: /home/pdecat/.cache/pip/wheels/a2/6e/4e/8b0ae12fb9b8a05715256952cf7609a8ab86285fab99b88c68
Successfully built ws4py
Installing collected packages: asn1crypto, idna, six, pycparser, cffi, cryptography, pbr, python-dateutil, urllib3, certifi, chardet, requests, requests-toolbelt, requests-unixsocket, ws4py, pylxd
  Running setup.py install for pylxd ... done
Successfully installed asn1crypto-0.24.0 certifi-2018.4.16 cffi-1.11.5 chardet-3.0.4 cryptography-2.2.2 idna-2.6 pbr-4.0.4 pycparser-2.18 pylxd-2.2.7.dev23 python-dateutil-2.7.3 requests-2.18.4 requests-toolbelt-0.8.0 requests-unixsocket-0.1.5 six-1.11.0 urllib3-1.22 ws4py-0.5.1

Okay, I'm going to merge this, but want to check out some scenarios before unleashing it on pypi.

Thanks for this update. This update for us in our cloud-init CI environments.


[Bug]  skywalking-php 代码连接依赖资源使用变量的方式无法捕获connect信息

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
PHP (apache/skywalking-php)
What happened
<?php
echo '111';
{
    try {
      $client = new Redis();
      $host = "<IP_ADDRESS>";
      $port = "6379";
      $client->connect($host, $port, 1);
      $client->set('foo', '1111');
    } catch (RedisException $e) {
    }
}
echo "ok";

当connect 中使用变量的方式时，是无法获取connet信息。

若不是变量的方式时，是可以获取到connect信息
<?php

echo '111';

{
        $client = new Redis();
        $client->connect("<IP_ADDRESS>", 6379, 1);
        $client->set('foo', '1111');
}

echo "ok";

What you expected to happen

How to reproduce
<?php
echo '111';
{
    try {
      $client = new Redis();
      $host = "<IP_ADDRESS>";
      $port = "6379";
      $client->connect($host, $port, 1);
      $client->set('foo', '1111');
    } catch (RedisException $e) {
    }
}
echo "ok";

Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

You must use English in the issue report. Please change the title accordingly.


Destination API only returns one destination for bus 72

Hello, after the bus reorg, I started having this issue of only one destination for bus 72.
https://api-ratp.pierre-grimaud.fr/v3/destinations/bus/72?_format=json
only returns 'Parc de Saint-Cloud', and not the other direction.
{
    "result": {
        "destinations": [
            {
                "name": "Parc de Saint-Cloud",
                "way": "A"
            }
        ]
    },
    "_metadata": {
        "call": "GET /destinations/bus/72",
        "date": "2019-04-22T10:00:49+02:00",
        "version": 3
    }
}

Others seem to be working fine (I don't have an exhaustive list of those that are not working).

Hi, good catch. I will take a look asap


Help - Search has problems on closed views

When we close a view the search elements don't get updated and calling it again gives a traceback

Fatal error in MSS 6.2.1. on Linux-5.4.0-113-generic-x86_64-with-glibc2.27
Python 3.9.13 | packaged by conda-forge | (main, May 27 2022, 16:56:21)
[GCC 10.3.0]

Please report bugs in MSS to https://github.com/Open-MSS/MSS

Information about the fatal error:

Traceback (most recent call last):
  File "/home/reimar/miniconda3/envs/newyenv/lib/python3.9/site-packages/mslib/msui/msui.py", line 387, in <lambda>
    self.actionSearch.triggered.connect(lambda: self.show_shortcuts(True))
  File "/home/reimar/miniconda3/envs/newyenv/lib/python3.9/site-packages/mslib/msui/msui.py", line 927, in show_shortcuts
    self.shortcuts_dlg.fill_list()
  File "/home/reimar/miniconda3/envs/newyenv/lib/python3.9/site-packages/mslib/msui/msui.py", line 216, in fill_list
    self.reset_highlight()
  File "/home/reimar/miniconda3/envs/newyenv/lib/python3.9/site-packages/mslib/msui/msui.py", line 183, in reset_highlight
    shortcut[-1].setStyleSheet("")
RuntimeError: wrapped C/C++ object of type QPushButton has been deleted

Also the inactive elemets are visible. e.g. chat should only be shown when a server is connected and a user logged in and has the role to use a chat.


Fix c15 import/refresh bug

There's a bug where if you import a save that's in c15 or refresh while inside c15, the ants become way quicker. When entering c15, the game sets all corruptions to level 11, including the removed corruption Divisiveness. This makes the sum of corruption levels 99, which raises ant production to the power of ^0.01. However if you import or refresh, the Divisiveness corruption is discarded, making the sum of corruption levels 88 and raising ant production to ^0.12, effectively raising ant production to the 12th power.
I thought that c15 applying a corruption that's no longer used is a bug in itself, so I made it no longer apply that corruption, and made it raise ant production to the power of 0.0833 (approximation of 1/12) to counteract the ant speedup mentioned above.
I also fixed the displayed current corruptions not being changed to 11 in the corruptions tab when entering c15, and also changed the displayed ant production in the corruptions tab to include nerfs applied by ascension challenges, and changed Maladaption to Viscosity in the ascend history.

This was differently fixed in a separate update. Thank you for the PR, though!


[Feature] Actor refactor for peer management

🚀 Feature
Actor model is preferable for maintaining consistency and avoiding race conditions/state leaks/state consistency failures.
Implementation
#725 implements this.

Closed by #725.


How hidden marker?

Hi, I believe that this is not an ideal place for questions, but I have not found any information from an appropriate portal.
I need to hide the marker of some elements, where I define the symbol as null or empty string, but nothing works, and there is also no appropriate option in Marker styles
AASeriesElement()
    .marker(AAMarker().symbol(""))

Ohh, i see now in demo project the comment
markerRadius(0f) //折线连接点半径长度,为0时相当于没有折线连接点

this hidden the marker :)
Great library, but need to improve the doc, thanks

Ohh, i see now in demo project the comment
markerRadius(0f) //折线连接点半径长度,为0时相当于没有折线连接点

this hidden the marker :)
Great library, but need to improve the doc, thanks


[docs]: Persisting view and viewmodel from extended navigator

Describe the missing piece of documentation
Hi, I have a bottom navigation bar set up with IndexTrackingViewModel. One of the tabs is an extended navigator with some nested navigation. This is all working.
However, when I switch tabs from the nested navigation and then go back to the extended navigator tab/nested navigation, the initial view/route is loaded and the viewmodel onready is called again. How do I persist the view model and view of the extended navigator. How do I load the details view rather than the initial view when going back to that tab.
I have seen stuff using key: PageStorageKey('storage-key') to persist the view state. This works if the page is not in an extended navigator.
Also viewmodel seems to be created every time even when overriding disposeViewModel and fireOnViewModelReadyOnce on tab change, whether the view is in an extendednavigator or not.
TLDR: I want to persist each view and view model or complete navigation stack (if it is nested) in each tab of the bottom nav bar. When tapping the tab it should bring you to the last page it was on within that nav stack. If possible tapping again could pop the entire stack back to start. Can I do this with stacked?
I have searched a lot but cannot find any information on this. I am by no means a flutter expert. Let me know if you need any further info.
Really appreciate the help.
Cheers

I managed to implement what I wanted by using the AutomaticKeepAliveClientMixin

@TobyDevelopment  can you give an example please.


Use the default metricsFactory if not provided

This will fix #738
I'm looking at adding a test to ensure the functionality will stay over time. I'll update the PR

Coverage remained the same at 100.0% when pulling d5a0f00070e59d701073db13a417f7003c91b98c on Ticketmaster:fix-default-metrics-factory into 6fce4f9f60822edc1d581d9b9c3eacd2ba9f5b15 on jaegertracing:master.

A test case was added, but an error is returned when withRunningAgent is used twice:
panic: duplicate metrics collector registration attempted [recovered]
	panic: duplicate metrics collector registration attempted

goroutine 73 [running]:
testing.tRunner.func1(0xc420294000)
	/usr/local/Cellar/go/1.10/libexec/src/testing/testing.go:742 +0x29d
panic(0x15a33e0, 0xc4202404c0)
	/usr/local/Cellar/go/1.10/libexec/src/runtime/panic.go:505 +0x229
github.com/jaegertracing/jaeger/vendor/github.com/prometheus/client_golang/prometheus.(*Registry).MustRegister(0xc42001ea00, 0xc420258210, 0x1, 0x1)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/vendor/github.com/prometheus/client_golang/prometheus/registry.go:404 +0x9e
github.com/jaegertracing/jaeger/vendor/github.com/uber/jaeger-lib/metrics/prometheus.(*vectorCache).getOrMakeCounterVec(0xc420244480, 0x0, 0x0, 0x0, 0x0, 0xc420270200, 0x31, 0xc420270200, 0x31, 0x0, ...)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/vendor/github.com/uber/jaeger-lib/metrics/prometheus/cache.go:50 +0x213
github.com/jaegertracing/jaeger/vendor/github.com/uber/jaeger-lib/metrics/prometheus.(*Factory).Counter(0xc420242440, 0xc420270200, 0x31, 0xc4202447e0, 0x0, 0x0)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/vendor/github.com/uber/jaeger-lib/metrics/prometheus/factory.go:110 +0x186
github.com/jaegertracing/jaeger/vendor/github.com/uber/jaeger-lib/metrics.initMetrics(0x1514200, 0xc42026e280, 0x168c720, 0xc420242440, 0x0, 0x50, 0x15c5e40)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/vendor/github.com/uber/jaeger-lib/metrics/metrics.go:72 +0x43e
github.com/jaegertracing/jaeger/vendor/github.com/uber/jaeger-lib/metrics.Init(0x1514200, 0xc42026e280, 0x168c720, 0xc420242440, 0x0)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/vendor/github.com/uber/jaeger-lib/metrics/metrics.go:25 +0x57
github.com/jaegertracing/jaeger/cmd/agent/app/reporter/tchannel.New(0x161f664, 0x10, 0xc420298000, 0x0, 0x168c720, 0xc420242300, 0xc42024e600, 0xc420242301)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/cmd/agent/app/reporter/tchannel/reporter.go:78 +0x283
github.com/jaegertracing/jaeger/cmd/agent/app/reporter/tchannel.(*Builder).CreateReporter(0xc4201ecee8, 0x168c720, 0xc420242300, 0xc42024e600, 0xc420242300, 0x0, 0x0)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/cmd/agent/app/reporter/tchannel/builder.go:119 +0x142
github.com/jaegertracing/jaeger/cmd/agent/app.(*Builder).createMainReporter(0xc4201ece90, 0x168c720, 0xc420242300, 0xc42024e600, 0x0, 0xc4202401c0, 0xc420278140)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/cmd/agent/app/builder.go:113 +0x4f
github.com/jaegertracing/jaeger/cmd/agent/app.(*Builder).CreateAgent(0xc420077e90, 0xc42024e600, 0x2, 0x2, 0x0)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/cmd/agent/app/builder.go:129 +0x8b
github.com/jaegertracing/jaeger/cmd/agent/app.withRunningAgent(0xc420294000, 0xc420077f88)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/cmd/agent/app/agent_test.go:101 +0x150
github.com/jaegertracing/jaeger/cmd/agent/app.TestAgentMetricsEndpoint(0xc420294000)
	/Users/ledor473/go/src/github.com/jaegertracing/jaeger/cmd/agent/app/agent_test.go:71 +0x46

The test is working now (had to re-create the prometheus.DefaultRegistry before each test).
I've also squash my commits and add the sign-off needed for the DCO

looks like tests fail due to formatting, run make fmt

All green now! 👍

Thanks for the fix @ledor473


Highlighting word with accent

Hi..
Im trying to search accents words with a filter and work fine to me..But i need some help to highlight..
var comboplete = new Awesomplete("dstAutoComplete", {     minChars: 2, filter: function(text, input) {       return Awesomplete.FILTER_STARTSWITH(text.normalize("NFD").replace(/[\u0300-\u036f]/g, ""), input.match(/[^,]*$/)[0]);     },   });
In this example word So and Só must be highlight.

Sorry my english.

Trying to solve this exact problem…

Solved it… 💪
@LeaVerou how it's the proper/easy way to add a awesomplete-plugin?

I just noticed this issue as well on a bilingual site I'm developing, for example the highlighting only reaches the 't' in sauté
After you type e, the result vanishes!
@caneco can you share the fix?
@LeaVerou this appears to be a bug/possible regression?

@vjmartins your solution doesn't work with french keyboards, if someone actually types with accents, no results will display.

@LeaVerou how it's the proper/easy way to add a awesomplete-plugin?

There is no specific way right now, but I would love to add a directory of plugins!!


Anyone interested in getting paid to revive this repository and keep it going?

Hello,
I've searched high and low for a Blazor shell as good as the AdminLte.  I'm using the AdminLte Javascript version in my Blazor app but have been wanting to move to a native Blazor version of it.  It seems this repo is no longer maintained.  It seems that no one responds to the issues anymore.  I believe this project has tremendous value.
I'm willing to partner with anyone interested in reviving this repository and help update it.  I would be willing to pay someone to help keep this moving if you're interested.
Anyone interested?

Hey, just saw your message. Yes, I believe the project has great value. I feel there is lack of choice in other libraries such as MudBlazor etc. So having components available from here would be great.
So I am willing to contribute. Send me an email at<EMAIL_ADDRESS>to discuss further.


Add metadata to log files

Sending event/battle
Sending notification
Download from AWS
Download from CDN
API requests

Ideally, a helper would be nice to create these meta fields and link relations between objects.

Partially addressed by #170


[Doc improvement] [Send and receive files through bots][2478596]

added link to protected APIs in Microsoft Teams

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/bots-filesv4.md
90
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 37a85d0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/bots-filesv4.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Stop fuse web server

And refactor fuse start code into startCommon() and stopCommon()
What changes are proposed in this pull request?
Stop Fuse web server when exit.
Why are the changes needed?
Fuse web server must be stopped, otherwise the main thread is waiting for it forever.
Does this PR introduce any user facing changes?
N/A

alluxio-bot, merge this please


Persister

This PR ~builds on the previous work in the redis branch (PR #1) and~ introduces the concept of a Persister which abstracts away the current JOBDIR storage mechanism. The basic idea is that instead of checking if the JOBDIR setting is configured and using it as a directory where arbitrary data can be read from and written to, a Persister should be used instead. A Persister offers a common interface to its users so they don‘t have to worry how the data is persisted.
Currently there ~are two implementations~ is only one implementation: A disk-based persister (scrapy.utils.job.DiskPersister) which basically mimics the current behavior in a backwards-compatible way. ~The second persister is the scrapy.utils.job.RedisPersister which uses Redis to persist data.~ The concrete implementaton of the persister can be configured via the commonly used dependency injection mechanism: The JOB_PERSISTER setting expects a fully-qualified name of a class that implements the persister interface.

Isn't this a "big change in scrapy"?

Yes, I am also think it is very big change. We should discuss less invasive ways to reach what is needed.

I rebased the PR on current master and removed the Redis-specific parts. I also moved the persister argument to the end and reintroduced the jobdir/path arguments to preserve backwards-compatibility (if the arguments are set, a warning is issued).

So, both Storage and Repository are fine for me

Fine by me, too, of course!


Is it possible to extract an object which is invoking a sink?

Suppose the putExtra method of Intent is used as a sink which takes a key value parameter. For example
L1 : Intent i = new Intent()
L2: i.putExtra("Key","Value")
from L2 using explicit value finder I can get the values of the arguments passed into putExtra. But is it possible to fetch the Intent object from this line?
Thanks

Theoretically, yes. However, currently, the explicit value finder only keeps track of constants. You might extend it to track object creation as well.


Remove [[noreturn]] from 'require' since it should return under most circumstances

PR Summary
The require() function should usually not return since we would hope the condition is false. The [[noreturn]] decorator applied to a returning function is undefined behavior. With gcc 10, it looks like the compiler uses [[noreturn]] to just give up on keeping track of the stack pointer which leads to a segfault.

PR Checklist

[x] Any changes to code are appropriately documented.
[x] Code is formatted.
[x] Install test passes.
[x] Docs build.
[ ] If preparing for a new release, update the version in cmake.

Thanks for fixing this @jhp-lanl . If we're making this change we may also wish to remove [[noreturn]] from abort and remove the while (true) {} loop in there, which is supposed to make the compiler not complain about [[noreturn]] but doesn't seem to work. But that can be for a future issue.

Thanks for fixing this @jhp-lanl . If we're making this change we may also wish to remove [[noreturn]] from abort and remove the while (true) {} loop in there, which is supposed to make the compiler not complain about [[noreturn]] but doesn't seem to work. But that can be for a future issue.

I'm less concerned about this since I've seen infinite loops used for asynchronous error handling in other places. Basically the loop can exist to give all processes something to do until the one that does all the aborting can actually abort.


added Rep. Lesko

A bioguide ID isn't yet posted.

House clerk has it as L000589

Updated with bioguide ID.


Update release Workflow; re-run poetry lock

The release.yml Workflow was doing some interesting things that didn't match test.yml. This PR updates the former to match the latter as we prepare to push our first PyPI release.

I'll merge this once we're ready to release 0.1.1 and push it to PyPI.


which "build" task is picked if I run the "Run Build Task" command?

Testing #27448:

in terminal:git clone https://github.com/Microsoft/vscode-node-debug
cd vscode-node-debug
npm install

open project in VS Code and run "Build Task" command

This works fine but I wonder which task definition is used:

start the "Run Tasks" action,
scroll through the list to find three(!) definitions for "build"
(there really should be four build definitions because task.json has another one which just is not named "build")

There should be a visual cue that flags either the single "build" that is used, or all potential "builds".
In the latter case a picker should be used to select the desired build.

The name is actual not relevant. It is whether a task is mark as isBuildCommand in the 0.1.0 version and in 2.0.0 belongs to the 'build' group.
Since in our example there is a 'isBuildCommand' in the tasks.json this one is executed even if you use the 2.0.0 engine which allows you to contribute tasks.
Only if there is no build task marked in the tasks.json I currently execute all contributed build tasks (again doesn't depend on the name it depends on the group) the belong too.
I could imagine to have a picker to let the user pick one. However other users already ask for running n build tasks in parallel. So the picker could have an 'all' entry.
And finally, if the user wants to run 2 out of n contributed build tasks she can create a task.json with something like this:
		{
			"taskName": "My main build",
			"dependsOn": ["build one", "build two"]
		}

Having the picker with build or test task (or really any group) would be great. For large number of test tasks (due to numerous test executables), the picker would ideally take in text as an input to allow for filtering of the tasks similar to how it filters after typing "task" in the command prompt

For June the behavior is as follows:

If there are more than one test or build task then a picker is shown.
The user can mark a task as a primary test or build task then that task is executed on run test or run build without user interaction

		{
			"type": "grunt",
			"task": "build",
			"group": {
				"kind": "build",
				"isPrimary": true
			},
			"problemMatcher": [
				"$tsc"
			]
		}


[lock table] don't preallocate space

don't preallocate space for shards for mutex table
should reduce memory usage on nodes

locally memory consumption dropped from 1gb to 600mb


websocketd.com email newsletter signup form does not work

The email signup form on websocketd.com seems to be broken, please just try it yourself.

@joewalnes I wonder if you need that really or rather to remove it alltogether?


Bill Of Materials

Update BOM.
Link: https://docs.google.com/spreadsheets/d/1IoWqGdi0kYIWvyvlh1Q84a06jdkviZJWtLsXj96vAzc/edit

@kevinbao1 is this related to issue #39?


[request] Keep the delivery agreement's title when it transitions between states

I want to name my sealed agreement, give it a recipe, then use the completed agreement in another recipe. Right now they all look and read the same though

Please explain in more detail what you want to do. The title and the message are a bit contradicting each other, or maybe I didn't understand correctly.

Right now a sealed agreement, when opened ignores nbt values that are not related to itself when building the resulting agreement. Which makes sense, but i causes an issue where the player can't diffreciet agreements in recipes where they can't right click to view details

Right now my solution is to use agreements instead of sealed agreements because those keep their extra nbt (like name) after becoming completed. Not perfect because I can't use loot tables hence the request

Added in 1.2.2.

Awesome, thank you!


Warn instead of error on signed numInGrp, check for minValue < 0

Relates to comment on #615.  Allow signed numInGrp and validate that the minValue is > 0

The type must be unsigned according to the specification.

Is this materially different than #609?  I don't see the harm in flagging it with a warning but still allowing it to overcome java's unsigned type limitation specifically - All code properly implementing the spec (i.e minValue) should handle this correctly.

Thanks!

You made a good point :-)


Fixed ACL cache path

Hello!

Type: bug fix
Link to issue:

In raising this pull request, I confirm the following:

[x] I have read and understood the Contributing Guidelines
[x] I have checked that another pull request for this purpose does not exist
[ ] I wrote some tests for this PR

Small description of change:
fixed double slash in path like /cache//acl/data.txt
Thanks

Thank you


Fix os.tmpDir DeprecationWarning and add Node 8 to test matrix

This is to fix #54.

This project is no longer supported. Please see the README

Yeah, I saw. I was seeing if any maintainers were around to merge and release this before creating a fork.


USB installer images won't work with BIOS

Used dd and rufus to create a bootable usb device (OPNsense-19.1.4-OpenSSL-vga-amd64.img), but refuses to boot on various machines with BIOS. After the boot devices is selected a blank screen appears for some seconds and you are back at the boot device selection.
On a PC with UEFI it works without a problem.
Also tried version 18.7 & 18.1 with the same result.
The pfsense image(pfSense-CE-memstick-2.4.4-RELEASE-p3-amd64.img) works perfectly.

Current situation after some more testing:
Machine 1 - BIOS: NOT WORKING
Machine 2 - BIOS: NOT WORKING
Machine 3 - BIOS: WORKING
Machine 4 - UEFI: WORKING
OPNsense-19.1.4-OpenSSL-vga-amd64.img and rufus were used.
Workaround:

Create a bootable media with rufus
Convert the GPT disk to MBR
Mark the EFISYS partition as "active"
--> WORKING on all BIOS PC's<--

Booting from GPT based images is intended to work on BIOS PC's as well, so there is still somewhere a problem.

Current situation after some more testing:
Machine 1 - BIOS: NOT WORKING
Machine 2 - BIOS: NOT WORKING
Machine 3 - BIOS: WORKING
Machine 4 - UEFI: WORKING
OPNsense-19.1.4-OpenSSL-vga-amd64.img and rufus were used.
Workaround:

Create a bootable media with rufus
Convert the GPT disk to MBR
Mark the EFISYS partition as "active"
--> WORKING on all BIOS PC's<--

Booting from GPT based images is intended to work on BIOS PC's as well, so there is still somewhere a problem.

can you share more detailed steps, please
converting to gpt format the hard drive right? so how we do it after creating a bootable media?

converting to gpt format the hard drive right?
After you wrote the image to your USB device it automatically will use GPT and to make it work you have to convert it to MBR. You just have to modify your USB device, your PC will be untouched.

so how we do it after creating a bootable media?
You still can modify your bootable USB device after you have written the image onto it. There are various tools available for it.

converting to gpt format the hard drive right?

After you wrote the image to your USB device it automatically will use GPT and to make it work you have to convert it to MBR. You just have to modify your USB device, your PC will be untouched.

so how we do it after creating a bootable media?

You still can modify your bootable USB device after you have written the image onto it. There are various tools available for it

I thought i have done something wrong as my usb always turn to gpt after burning. One last thing after  burning the img I can't open the usb from windows or display the files it contains is that normal?

I thought i have done something wrong as my usb always turn to gpt after burning.
Usually depends on a few things, but in case of opnsense it's normal (predefined by the image).

One last thing after burning the img I can't open the usb from windows or display the files it contains is that normal?
As far as I can remember I think it's normal.

Unfortunately I did what you said exactly but no success. There is no EFIsys partition on my usb drive

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/core/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.

I have the same issue. The problem is with the IMG. I have used DD and Popsicle. I am pretty sure that DD has no issues, and Popsicle hasn't yet let me down, writing an simple image. The ISO file (for DVD/CD etc.) is written perfectly, but it is not bootable.

Having the same pb: image with OPNsense-22.7-OpenSSL-vga-amd64.img doesn't work on a BIOS PC. OPNsense Nano image and pfsense 2.6 image boot fine.
I copied OPNsense-22.7-OpenSSL-vga-amd64.img to my USB key using dd. Then I used gdisk to convert GPT to MBR. Then I used fdisk to add the boot flag to the EFI partition (the first partition). After that, when I try to boot the USB key, BIOS shows a message "Invalid partition table".

Then I used gdisk to convert GPT to MBR

But why. Use Nano, don't break partition table?

Yes, I ended up using the nano image. I copied the nano image via dd directly to the SSD of my OPNsense firewall. It's the only solution I found.


refactor: Port to anstream

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 23 (0.0%)  changed or added relevant lines in 2 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.03%) to 8.923%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/steps/changes.rs
0
10
0.0%

src/ops/shell.rs
0
13
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/ops/shell.rs
1
0.0%

src/ops/replace.rs
1
0.0%

src/steps/changes.rs
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
217

Relevant Lines:
2432

💛  - Coveralls


Change randomMPS bond dim error to warning

Currently, randomMPS errors when the bond dimension doesn't reach the desired one. This appears to happen fairly often with quantum numbers, probably because of technicalities of quantum number conservation (for example, to reach the desired bond dimension it might be trying to add a block inconsistent with the flux). This changes it into a more informative warning.

Related to #763, but this doesn't really close that since we should still investigate the underlying problem of why it doesn't reach the desired bond dimension in some cases.


Spacy 0.100.7 breaks parsing for some cases

Seems like parsing in certain situations (not actually sure in which) fails. This issue didn't exist in 0.100.6. It can reproduced with the following script:
from spacy.en import English
from sklearn.datasets import fetch_20newsgroups
nlp = English()
texts = fetch_20newsgroups().data
nlp(texts[196])

This works in 0.100.6 but gives an empty AssertionError in spacy/tokens/doc.pyx in 0.100.7. Any ideas?

What's the error?
We've fixed a couple of bugs with 0.100.7, and hope to have 0.101.0 live tonight. So, maybe this is fixed. But, maybe not?

---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
/home/ubuntu/t.py in <module>()
      4 nlp = English()
      5 texts = fetch_20newsgroups().data
----> 6 nlp(texts[196])

/home/ubuntu/miniconda3/lib/python3.5/site-packages/spacy/language.py in __call__(self, text, tag, parse, entity)
    223         ('An', 'NN')
    224         """
--> 225         tokens = self.tokenizer(text)
    226         if self.tagger and tag:
    227             self.tagger(tokens)

/home/ubuntu/miniconda3/lib/python3.5/site-packages/spacy/tokenizer.pyx in spacy.tokenizer.Tokenizer.__call__ (spacy/tokenizer.cpp:4664)()

/home/ubuntu/miniconda3/lib/python3.5/site-packages/spacy/tokenizer.pyx in spacy.tokenizer.Tokenizer._tokenize (spacy/tokenizer.cpp:5409)()

/home/ubuntu/miniconda3/lib/python3.5/site-packages/spacy/tokenizer.pyx in spacy.tokenizer.Tokenizer._attach_tokens (spacy/tokenizer.cpp:6444)()

/home/ubuntu/miniconda3/lib/python3.5/site-packages/spacy/tokens/doc.pyx in spacy.tokens.doc.Doc.push_back (spacy/tokens/doc.cpp:8356)()

That's the output, not sure what to make of it

Yep, that's fixed in the imminent 0.101.0 release. Sorry about the problem.

No problem! What was the issue?

https://github.com/spacy-io/spaCy/issues/360


Split richtext-ui into BasicRichText and richtext-ui-material

Following the compose text standards, richtext-ui is now split into two artifacts.
RichText ➡ BasicRichText. New RichText named composable can be found in richtext-ui-material
Added LocalBasicTextStyle that should mimic the abilities of LocalTextStyle from compose.material
Added LocalContentColor that should mimic the abilities of LocalContentColor from compose.material
Completely removed material dependency from richtext-ui
Added new configuration options to BasicRichText

Alignment: Used to horizontally align content in the main column
TextStyle: No longer retrievable from LocalTextStyle, textStyle now has to be provided explicitly.
Color: No longer retrievable from LocalContentColor, contentColor now has to be provided explicitly.

Updated Markdown artifact to reflect the changes.

Markdown is no longer a standalone composable due to increased customization of BasicRichText. Instead, Markdown is just another component under RichTextScope. This change should also increase the visibility of Markdown's dependence on richtext-ui
Introduced MarkdownConfiguration. Markdown rendering sometimes requires granular control over individual components such as Images and HTML blocks.
Removed onLinkClicked argument. Although LocalUriHandler dependency is implicit, adding visibility to this CompositionLocal might benefit the users in the long run.

Removed material references from all previews under richtext-ui

That matches my intuition about this. It's not really a self-contained, complete design system in itself - it's meant to be used from within other design systems. But it is also still a system.
Maybe that relationship should be made explicit. Eg perhaps we allow the consumer to provide a composition local that can map their design system to the concepts in this library. The value could probably be just a composable function that returns a richtext theme or something. That local could be provided, once, in the theme wrapper function in specific apps, and then would automatically be used everywhere a RichText subtree was placed to map whatever the current design system parameters happen to be at that point in the composition to the richtext style. No overloads required!
Too magic?

I don't think it's too magic because composition locals are perfect for this use case. What I'm unsure about is the fact that there should be an initialization step at theme wrapper level. I, for some reason, have always loathed the libraries that require some sort of initialization in Application class.
This conversion/mapper composable can be a parameter to RichText to prevent the magical aspect of composition locals. Also, a reasonable consumer would create a single value composable that can be reused for every RichText instance, so that explicit parameter wouldn't cause an inconvenience. The library can use composition locals internally anyway.
I hope I understood your idea right.

The problem with giving RichText a parameter for this is that every usage would need to wire up the theming integration explicitly, when that's really just a property of the theme itself.

I have always loathed the libraries that require some sort of initialization in Application class.

Compose apps already require this though – you can't theme a compose Material app without wrapping the root of your app in a MaterialTheme composable at some point, for example. The skeleton code that Android Studio generates for new projects also creates a theming function that wraps MaterialTheme that it puts at the top of the composable tree. And defining the mapping from theme to RichText theme is just another part of the theme, so I think this is very appropriate.

Let's put this into code. This is what I'm getting
MaterialTheme(...) {
  ProvideRichTextStyle( @Composable {
    RichTextStyle(
      color = MaterialTheme.color.XYZ,
      ... = MaterialTheme.color.XYZ
    )
  }) {
    Content()
  }
}

This is an explicit mapper from app's design system to whatever RichText is going to support.
Flexibility of calling this ProvideRichTextStyle at any place, either as a Theme wrapper or just as a direct parent to a RichText makes it more than OK with me. I was just thinking about an app that only needs RichText in a single screen and introducing a wrapper at App level didn't feel right.
What about the optionality constraints about this theme wrapper? What happens when it is not provided and RichText gets called? This is like one of those cases that I wish there was a native dependency injection in Compose that could fail at compile time.

Yea that's it!

What happens when it is not provided and RichText gets called?

We just fallback to a default theme, much like how BasicText works.

All sounds great! I wish I could get down to it right away but I will be completely busy for the next couple of weeks. I'll check it out with you next time I'm available if there is any ongoing work :)


NOverlayImage.fromAssetImage is not working

NOverlayImage.fromAssetImage is not working

Here is my code:
NMarker marker = NMarker(
  id: "marker",
  position: position,
  icon: const NOverlayImage.fromAssetImage("assets/icons/ic_spot_location.svg"),
  size: const Size(10, 10),
);

I've also tried using NGroundOverlay not working also:
final bounds = NLatLngBounds(
  southWest: position,
  northEast: position.offsetByMeter(
    northMeter: 100,
    eastMeter: 200,
  ),
);
var m = NGroundOverlay(
  id: "pin-point",
  bounds: bounds,
  image: const NOverlayImage.fromAssetImage("assets/icons/ic_spot_location.svg"),
  alpha: 1,
);

hi.
svg file is not supported with NOverlayImage.
Please use pixel based image instead. (raster)


Compilation warning in projects that don't have Oban installed

Created a new projects with Plug, Phoenix, LiveView and Sentry:
=> sentry
Compiling 34 files (.ex)
    warning: Oban.Worker.from_string/1 is undefined (module Oban.Worker is not available or is yet to be defined)
    │
 31 │       case {oban_worker_mod.from_string(job.worker), stacktrace} do
    │                             ~
    │
    └─ lib/sentry/integrations/oban/error_reporter.ex:31:29: Sentry.Integrations.Oban.ErrorReporter.handle_event/4

This happens in Elixir 1.17, which introduced the start of the type system, which we can't fool that easily anymore 😄 We need to use apply/3 here. @savhappy wanna work on this?

@whatyouhide on it!

PR out for review  @whatyouhide
Thank you @AndrewDryga for pointing this out (:

Closed by #754, thanks @savhappy and thanks @AndrewDryga for reporting 💟


Bug panning a image after it moves

When you makes a fast movement to the left or right on the image, it pans and if in a ViewPager, it doesn't go to another page.
But when you try again going to the next or previous page, you can't till you touch the screen, makes zoom...

I've tried this change but it doesn't act as I want it to.
If you start panning an image and reach the edge, the next image jumps suddenly into view part way across the screen - if you have moved your finger 100px during the pan and hit the edge, the next image immediately appears 100px across the screen, when only the first pixel should appear.
Because I couldn't find the solution to this when I worked on pagination support, I made the decision to make the user scroll to the edge in one pan, lift their finger and swipe again to see the next image, and I want to keep that behaviour.
I'm reviewing the other suggested changes to pagination, and the change suggested in #34 is the current leading candidate. You could try that and see if it helps.


Giveaway.su displays a cheating message when using the script.

Image: https://i.imgur.com/9JFlSiC.png
When I use the script, it all works fine, until after you complete the captcha. Instead of showing the steam key, it shows the message (which I linked to above). If you disable the script before completing the captcha it still doesn't work, and you can never redeem the key with that account if it catches you "cheating". It clearly states that "We are OKAY about scripts helping you to complete tasks faster" although it seems it can't tell the difference between cheating and being efficient.

I can confirm that I got the same message using only this script, with Ad-blocker disabled, and after re-installing the Giveaway.su extension off the Chrome Web Store.
This script does not fake the completion of tasks.  I interpret this as them having changed their stance on helper scripts, and the error message just hasn't been updated to reflect that.
I'm not willing to create a workaround for this, and so I'll be removing support for Giveaway.su.

I found out the problem! On the official Giveaway.su Discord server I got this reply saying why the script is not allowed: https://i.imgur.com/g0NrbdR.png
Instead of removing support, just remove the unfollow features of the script and maybe the script will work.

I'm not willing to do that.


[Bug]: indentation object errors when at end of file

Bug Description
when an indented line is at the end of a file, trying to select it will throw an "index out of bounds" error
Relevant Screenshot
No response
To Reproduce

paste the following into a buffer:

a
    b

place the cursor on the second line
yii

neovim version
0.8.3

Make sure you have done the following

[X] I have updated to the latest version of the plugin.

thanks, is fixed


Support jsconfig.json

For now, you can use a tsconfig.json in your js only project.
all jsconfig,json really does (for vs code, at least) is implicitly set allowJs: true and disable errors relating to actual typescript compilation.
I've found the following tsconfig.json to work fine to enable this plugin in a js only project:
{
  "compilerOptions": {
    // this is implicitly set in a `jsconfig.json` so we have to do it manually here
    "allowJs": true,

    "outDir": "dist", // this essentially disables the compilation errors by telling vs code where files would be put if it were to compile (which it won't because this isn't a real typescript project)

    // Do these as normal to your liking for your project
    "baseUrl": "src",
    "paths": {
      "*": ["src/*"]
    }
  },

  // only treat files in here as js source (thus ignoring `node_modules` and your fake output directory above)
  "include": ["src"]
}

Released in v3.5.0

How does this work ? My project use this and it not working as intended :(

/cc @MichaelDeBoey you might be interested in adding this to tsconfck? no worries if you don't :)

@aleclarson I currently don't have the bandwidth to look into that unfortunately

It is pity that there is a regression on that. @huyhoang160593 it was working on windows 11 as well for me at least in version 3.2.4 in this exemplary project https://github.com/ibqn/beni-15-puzzle

It is pity that there is a regression on that. @huyhoang160593 it was working on windows 11 as well for me at least in version 3.6.0 in this exemplary project https://github.com/ibqn/beni-15-puzzle
"vite-tsconfig-paths": "^3.6.0"

It seem like in your jsconfig.json there is no "path" been defined? This plugin is basically doing nothing with your project for now

@huyhoang160593 I use it for global importing i.e. for example
import { App } from 'app'
import 'index.css'

Hello hello! Is there a chance to get the jsconfig.json support back? Maybe using cosmiconfig is possible? For now I got to convert the file to tsconfig.json and add some attributes. (I'm migrating from react-scripts to vite)

It's not simply a version bump. Since we use tsconfck for file discovery and it doesn't support searching for both tsconfig.json and jsconfig.json in the same monorepo, we'll have to settle for either double-crawling or a new option that lets you (the plugin user) override the default configName with "jsconfig.json" in your Vite config (which means, tsconfig.json files wouldn't be used). At first glance, the double-crawling approach seems the best DX with a hopefully negligible performance hit.

tsconfck 3 with its new async cache is a lot faster (60-100 times faster!) than tsconfck 2. So even with a double crawl it would be an improvement. But either way in regular projects users won't notice, there are just not enough files/directories for it to make a difference.
The real question is what you would do with the result of a double crawl. Typescript itself  uses tsconfig.json by default and you can tell it about jsconfig.json with a cli flag. To my knowledge, its not defined how precedence is handled in a scenario where there are mixed jsconfig.json and tsconfig.json files, so the easy solution here is to follow typescripts lead and macke the config name configurable.
If you allow both simultaneously, it would be your responsibility to check which one is closer to the input file, tsconfck does not offer a hybrid mode (and has no plans to, unless typescript itself made that move)

Support for jsconfig.json released in v4.3.0

Thank you! @aleclarson


WebSocket opening handshake was canceled

Upon connecting to wss websocket connection getting the following error
WebSocket connection to '<wss_url>' failed: WebSocket opening handshake was canceled

Nothing to do with this library.


Bug Min role to run view

Accessing the view as a public user, there are no problems to execute, being that the minimum role selected is staff.  And in "Show configuration object" the minimum role is 10

The problem is only to execute the view, when sending the data returns error correctly by the permissions that the table has

Hi @gianlucafiore, I'm not sure that this does not work.
Public role means unauthorized user and now I tried to execute view with min_role is admin. This view correctly does not execute.
In example configuration the minRole 10 related to particular Action instead of whole view.
Also minrole = 10  means PUBLIC, so any user can execute this Action.


[QUESTION] S3 storage and serviceAccount for api/worker

Hello BorisPolonsky,
I am deploying dify to AWS EKS. I will use AWS redis and rds postgres for the DB.
However, the EBS volume does not support ReadWriteMany, so I am looking for another storage type for api/worker
I could not find the docs that could answer my concerns so I would like to ask you something

Does the api/worker support S3 as storage?
Does the dify helmchart support configuring serviceAccount for api/worker so that they can use S3 as storage

The only serviceAccount config I can find in the helmchart is for Weaviate and Redis
Hope to have your responses soon
Thank you

It's defined here.

@BorisPolonsky, thanks so much for your response!
I configured external Redis here
Since I used AWS ElastiCache as external Redis, there is no authentication configuration for the Redis, so I left the username/password blank
externalRedis:
  enabled: true
  host: "master.*****.*****.use1.cache.amazonaws.com"
  port: 6379
  username: ""
  password: ""
  useSSL: True

Then I configured externalEnv for the api/worker as following:
- name: REDIS_HOST
value: "master.*****.*****.use1.cache.amazonaws.com"
- name: REDIS_PORT
value: "6379"
- name: REDIS_USERNAME
value: ""
- name: REDIS_PASSWORD
value: ""
- name: REDIS_USE_SSL
value: "True"

However, the worker pod throw the below error. It seems like the worker did not get the REDIS_HOST that I defined in the variable
2024-07-15 04:03:09,876: ERROR/MainProcess] consumer: Cannot connect to redis://:**@redis:6379/1: Error -2 connecting to redis:6379. Name or service not known..
Trying again in 24.00 seconds... (12/100)

Could you please advise?

@BorisPolonsky, thanks so much for your response! I configured external Redis here Since I used AWS ElastiCache as external Redis, there is no authentication configuration for the Redis, so I left the username/password blank
externalRedis:
  enabled: true
  host: "master.*****.*****.use1.cache.amazonaws.com"
  port: 6379
  username: ""
  password: ""
  useSSL: True

Then I configured externalEnv for the api/worker as following:
- name: REDIS_HOST
value: "master.*****.*****.use1.cache.amazonaws.com"
- name: REDIS_PORT
value: "6379"
- name: REDIS_USERNAME
value: ""
- name: REDIS_PASSWORD
value: ""
- name: REDIS_USE_SSL
value: "True"

However, the worker pod throw the below error. It seems like the worker did not get the REDIS_HOST that I defined in the variable
2024-07-15 04:03:09,876: ERROR/MainProcess] consumer: Cannot connect to redis://:**@redis:6379/1: Error -2 connecting to redis:6379. Name or service not known..
Trying again in 24.00 seconds... (12/100)

Could you please advise?

There's no need to apply environment variables for redis through extraEnvs if you have specified them in externalRedis. If you insist, environment variables defined in api.extraEnvs would override those configurations defined in externalRedis.
The message Name or service not known suggests that you have configured a domain that your cluster cannot resolve. You may need to figure out the reason from aws if it's this domain is not resolvable from other enviroment either. e.g. https://devops.stackexchange.com/questions/13591/aws-elasticache-redis-dns-error-name-or-service-not-known

@BorisPolonsky thanks for that. The problem is I configured wrong URL for celery.
Last question (hopefully) I would like to ask is does the dify helmchart support configuring serviceAccount for api/worker? (I asked in the first comment, you might missed it)
I found no serviceAccount config in the templates. Without serviceAccount, I have to use secret-key/access-key for the S3 access which is not best practice

@BorisPolonsky thanks for that. The problem is I configured wrong URL for celery.
Last question (hopefully) I would like to ask is does the dify helmchart support configuring serviceAccount for api/worker? (I asked in the first comment, you might missed it)
I found no serviceAccount config in the templates. Without serviceAccount, I have to use secret-key/access-key for the S3 access which is not best practice

No.
This feature is available to AWS EKS only and we do not implement features that are limited to specific distributions of kubernetes.


Pre processing using the color space of the training

I'm sorry, again... The same mistake as the last one. I need to change de base repo to avoid this.


On Some Android ,SVG number-shape and color-picker is incompatibility,a probability

Actual Behavior
like this

but . in dragging , it's normal
I don't know how to solve it either.
Steps to Reproduce
it's just on some android phones at some browser
Operating System and Browser
_e.g.  Vivo | X21A | Android8.1.0 unity-app
user webview-plug-in  https://github.com/gree/unity-webview/blob/master/LICENSE

@hsyomi Sorry, but we haven't tried using Scratch blocks from within Unity so I'm not sure we'll be much help.

This should not be a problem with unity, it seems to be a problem with Android webviews. Related to the drawing path of svg.

Such a modification can effectively solve this problem.


fix: questions Page, single question page, reply MDeditor

PR Fixes:

1 Questions Page
2  Single Question Page
3 MarkDown editor on reply
4  Content and Title fit

https://github.com/user-attachments/assets/4a1be849-1777-4266-9d8b-9a24e143eece
Resolves #946  and Resolves #935
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

@siinghd formatting failing due to this #956 please look into it.

@hkirat could you look into #956 and also review this

Thank you sir @nischal-shetty2
/bounty 5$

glad to help!


[core][PageContainer] Do not spread PageContainer title to child Container

[x] I've read and followed the contributing guide on how to create great pull requests.
[x] I've updated the relevant documentation for any new or updated feature.
[x] I've linked relevant GitHub issue with "Closes #".
[ ] I've added a visual demonstration in the form of a screenshot or video.

Closes #4452.

Nice, when can we expect this to be released?


Possible bug in vcpu_t::entry_host()

You capturing rip rsp and rflags before vmexit handler:
`    auto captured_rsp    = exit_context_.rsp;
auto captured_rflags = exit_context_.rflags;
{
  exit_context_.rsp    = guest_rsp();
  exit_context_.rip    = guest_rip();
  exit_context_.rflags = guest_rflags();

  //
  // WinDbg will show full callstack (hypervisor + interrupted application)
  // after these two lines are executed.
  // See vcpu.asm for more details.
  //
  // Note that machine_frame.rip is supposed to hold return address.
  // exit_instruction_length() is added to the guest_rip() to create
  // this value.
  //
  stack_.machine_frame.rip = exit_context_.rip + exit_instruction_length();
  stack_.machine_frame.rsp = exit_context_.rsp;

  {
    handler_->handle(*this);`

As result any modifications to rip, rsp and rflags in vmexits have not any effect. So possible solution is:
`    memory_manager::allocator_guard _;
auto captured_rsp    = exit_context_.rsp;
auto captured_rflags = exit_context_.rflags;

{
  handler_->handle(*this);

  exit_context_.rsp    = guest_rsp();
  exit_context_.rip    = guest_rip();
  exit_context_.rflags = guest_rflags();`

I'm not sure what do you mean. The code is there : https://github.com/wbenny/hvpp/blob/ec4115e5af113d5b7fa57735cfc7ea85f6dad47c/src/hvpp/hvpp/vcpu.cpp#L673

Look. We are capturing exit_context_.rip    = guest_rip(); before  handler_->handle(*this);. If you do something like vmwrite(guest_ip, newip) in vmexit handler it will dont have any effect because after vm exit you restore captured rip guest_rsp(exit_context_.rip);.

That's true. The idea here is that you can change the registers via vp.exit_context(). Including rip, rsp & rflags

Thanks for hint.


add louisiana state

Thank you for contributing!Preview these changesLink expires: Fri Sep 28 05:40:30 UTC 2018Note: it may take a moment for the branch to deploy. Check all pending travis builds have completed


Publishing to Store

Just a question, are there any plans to make this application available on the Appstore?
Thank you in advance

I'm certainly keen on making this happen; the current install process is cumbersome at best.  That said, it's a sizeable project with lots of pieces, and licensing will very likely require the whole thing be rewritten from scratch if it were to be available on the App Store.  Essentially, check out issue #8 -- if you're skilled in the ways of Objective-C or Swift, help is well appreciated.

thank you for quick and fast reply.


Is there a logic error with unescapeFormat in src/lib/parse/regex.js

Describe the bug
Hello, I have a hit from a code scanner on this function, and I'm not certain if this is an actual bug or not. I'm not familiar enough with the code in question to know if this implementation is working as expected or not.
The issue seems to be that String.prototype.replace will only replace the first instance of a string, when two strings are passed as arguments.
Examining the code in regex.js, we see the following:

Here are the CodeQL alerts in question:

Any reasons to be concerned?

Bump, anyone?


Transaction declares authority but does not have signatures for it

Running into a lot of errors trying to deploy a contract to the jungle test net.

And yes I have already imported my private key (I've tried importing multiple private keys) and tried everything from #4432.

I'm following the set up from https://developers.eos.io/eosio-home/docs and keosd is getting an error: FC Exception encountered while processing wallet.get_public_keys

Try running keosd with --verbose-http-errors to get more detail about the error.  Most likely you've either restarted keosd or it has timed out and locked the default wallet.  get_public_keys will not succeed if the wallet is locked.

Still getting the same error trying to deploy the contract, but not getting any error with keosd now.
I'm running:
keosd --verbose-http-errors &
nodeos -e -p eosio
--plugin eosio::producer_plugin
--plugin eosio::chain_api_plugin
--plugin eosio::http_plugin
--plugin eosio::history_plugin
--plugin eosio::history_api_plugin
--filter-on=""
--access-control-allow-origin=''
--contracts-console
--http-validate-host=false
--verbose-http-errors >> nodeos.log 2>&1 &
cleos wallet unlock
cleos -u https://api.jungle.alohaeos.com set contract mnenomneno22 mneno/ "/mneno.wasm" "/mneno.abi" -p mnenomneno22@owner
cleos get account mnenomneno22 returns the expected output by the way.

And what is keosd now reporting?


v2 requiring user authentication to change password

Hi
I was looking at implementing change password using the v2 branch.
For this feature, the logged in user would provide their current password, the new password and a new confirmation password.
The process would have 2 high level steps

authenticate the user (PID from the cookie, password from the POST data)
change the password to the new password (new_password and confirm_password from POST data).

While building this I noticed that authentication is tied to HTTP concerns, which I found prevents this from working.
To work with the current architecture with (validators etc) the HTTP body ends up being read twice, which doesn't work at the 2nd read will fail (body is empty as it has been read).
I propose refactoring the Auth.LoginPost method to separate the HTTP concerns from the authentication concerns. I've had a brief look at this, but it might be a non-trivial exercise given the way that the HTTP request and response are passed around. For example validators read data from the body, events require the HTTP request and response.
Changing password works easily if I do not attempt to use Auth.LoginPost as it currently stands as the HTTP body is only read once.
It may be that I'm missing something obvious here, and that there is a very easy way to do this without any refactoring.
Thoughts?

Hey there @scottjbarr. auth.LoginPost is an HTTP handler so separating the HTTP concerns out of it would be a bit odd.
Authboss does a lot to abstract away the HTTP stack behind all these interfaces. But if I were you and I was dealing with my own web application I would ignore all of this and implement my own handler that doesn't have to jump through hoops to read a couple of values from the POST data and compare some passwords.
// Your handler in your application
func changePasswordHandler(w http.ResponseWriter, r *http.Request) {
  user, err := myAuthboss.CurrentUser(r)
  if err != nil {
    // Error
  }

  u := user.(MyUserType)

  old := r.FormValue("old_password")
  password := r.FormValue("new_password")
  confirm := r.FormValue("new_password_confirm")

  if len(password) != 0 {
    // Error
  }

  if password != confirm {
   // Error
  }

  if err := bcrypt.CompareHashAndPassword([]byte(u.Password), []byte(old); err != nil {
    // Error
  }

  if err := myAuthboss.UpdatePassword(r.Context(), u, password); err != nil {
    // Error
  }

  // Render some kind of response
}

This is how you could do it. A follow-up question that I have would be is there some reason the above is undesirable? LoginPost does something very different than changing passwords, it authenticates a user and sets session variables on him to say that he's logged in. Maybe you'd like to extract the part that checks the password against the one in the database? A simple helper could be made to wrap the bcrypt call that's required but it doesn't save you too much code (one line for one line).

With regards to handlers, I build them to entirely separate HTTP from the services a package provides. For example, I would aim to be able to authenticate a user with a set of given credentials that function, without knowing about a HTTP request/response.
I could wrap the existing code up by copy/pasting (and that is what I've done), but it seems rough to me. In refactoring I aimed to avoid that, allowing me to leverage the core features of the package.
If it isn't intended to be used that like, that's ok.

Thanks @aarondl.


Fix docs

Related to:

https://github.com/elixir-lang/ex_doc/issues/1085

In the latest earmark, by default, it renders a link now

Awesome! Thanks


Add test action

Add new action to just execute specific scheme.
$ ./test github.com user repo branch --scheme Tests --destination 'platform=iOS Simulator,name=iPhone 6'

LGTM

Thank you for reviewing. I'll merge this.


Refactor

Implemented default compressor,
Added GetType to compressor interface so it can be called given a compressor to insert metadata to message,
Refactored GZip implementation,
Changed the structure of the repo to avoid cyclic imports.

restructuring repo


Update webgl2.json Android Chrome support

https://www.chromestatus.com/feature/6694359164518400
Source.

Thanks, fixed manually as it's not possible to add versions.

Thanks Fyrd! Glad I could help out a little bit, I visit caniuse all the time for reference and wanted to give something back.


Complete implementation of command line interface.

Helloo @Chaser324 !! Thank you so much for this amazing package
Do you have any updates about this issue? Would be wonderfull to automated builds in a CI/CD environment :)

Hey @mfdeveloper, SuperUnityBuild's CLI is currently just a single command, but it could already be used in a CI/CD environment: https://github.com/superunitybuild/buildtool/wiki/Command-Line-Interface
What it's lacking is reporting errors or any of the build log entries that the GUI shows, plus the ability to only build a certain release type or build configuration. This work is likely to happen when I or someone else who makes use of the project has a need for this functionality, so I can't give any idea of if/when these additional features would be implemented. However, it probably won't take much to achieve this, so it's also an ideal candidate for a first-time PR contribution.


WIP: Modal as View

I think better to treat Model as View than Widget.
If it will be view then we can inherit it and handle some complex models by amending some widgets.

Test are passing for Modal
testing/test_modal.py::test_title PASSED                                 [ 40%]
testing/test_modal.py::test_body PASSED                                  [ 41%]
testing/test_modal.py::test_close PASSED                                 [ 42%]
testing/test_modal.py::test_footer_items PASSED                          [ 43%]
testing/test_modal.py::test_footer_item PASSED                           [ 44%]
testing/test_modal.py::test_footer_item_invalid PASSED                   [ 45%]
testing/test_modal.py::test_modal_as_view PASSED                         [ 45%]


Add library to simplify using Owin pipeline in ASP.NET

For example: https://github.com/danroth27/AspNetWebApiCore/blob/main/OwinApplicationBuilderExtensions.cs
Related issue: https://github.com/dotnet/AspNetCore.Docs/issues/25664

Triage: might consider wrapping the Builder API to make this easier/simpler.
We'll probably want to put it in a separate library (to minimize unnecessary dependencies in S.W.Adapters).

I wrote a sample here https://github.com/davidfowl/AspNetCoreOwinSample


Can "collapse" widget default to collapsed state on load?

For boxes that use the "collapse" widget, is there a way to default to the collapsed state (rather than defaulting to the expanded state) when the page is loaded?
Thanks!

Title Here

                <button class="btn btn-box-tool" data-widget="collapse"></button>

Item 1

Item 2

Item 3

Item 4

FYI - here is the solution that worked for me:

Default Solid Box

            <button class="btn btn-default btn-sm" data-widget="collapse"></button>
            <button class="btn btn-default btn-sm" data-widget="remove"></button>

        Box class: .box.box-solid

bla bla

what's the solution here?

I dont know which version of AdminLTE you use, but in 2.3.x you can add collapsed-box to the box class and switch fa-minus with fa-plus this should work.

I am not sure if things have changed since this issue was answered, but adding collapsed-box to the box class in my case somewhat disabled the toggle option (the box remained collapsed even when I click for it to open). Instead, I had to change the class of the collapsible element from panel-collapse collapse in to panel-collapse collapse.


Enhancing Gitea OAuth2 Provider with Granular Scopes for Resource Access

Resolve #31609
This PR was initiated following my personal research to find the lightest possible Single Sign-On solution for self-hosted setups. The existing solutions often seemed too enterprise-oriented, involving many moving parts and services, demanding significant resources while promising planetary-scale capabilities. Others were adequate in supporting basic OAuth2 flows but lacked proper user management features, such as a change password UI.
Gitea hits the sweet spot for me, provided it supports more granular access permissions for resources under users who accept the OAuth2 application.
This PR aims to introduce granularity in handling user resources as nonintrusively and simply as possible. It allows third parties to inform users about their intent to not ask for the full access and instead request a specific, reduced scope. If the provided scopes are only the typical ones for OIDC/OAuth2—openid, profile, email, and groups—everything remains unchanged (currently full access to user's resources). Additionally, this PR supports processing scopes already introduced with personal tokens (e.g. read:user, write:issue, read:group, write:repository...)
Personal tokens define scopes around specific resources: user info, repositories, issues, packages, organizations, notifications, miscellaneous, admin, and activitypub, with access delineated by read and/or write permissions.
The initial case I wanted to address was to have Gitea act as an OAuth2 Identity Provider. To achieve that, with this PR, I would only add openid public-only to provide access token to the third party to authenticate the Gitea's user but no further access to the API and users resources.
Another example: if a third party wanted to interact solely with Issues, it would need to add read:user (for authorization) and read:issue/write:issue to manage Issues.
My approach is based on my understanding of how scopes can be utilized, supported by examples like Sample Use Cases: Scopes and Claims on auth0.com.
I renamed CheckOAuthAccessToken  to GetOAuthAccessTokenScopeAndUserID so now it returns AccessTokenScope and user's ID. In the case of additional scopes in userIDFromToken the default all would be reduced to whatever was asked via those scopes. The main difference is the opportunity to reduce the permissions from all, as is currently the case, to what is provided by the additional scopes described above.

By the way, no need to rebase or force push after reviewing starts (it makes the review more difficult). https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md#maintaining-open-prs
Just push new commits, and the final merge will squash.

thank you for your patience. this is my first pull request and i still learn it.

Could we have a screenshot of the new grant page?

Please send a update doc PR to https://gitea.com/gitea/docs

Could we have a screenshot of the new grant page?

Could we have a screenshot of the new grant page?

Igh>

Thank you. The permissions should be highlighted,maybe a bold font?

Please send a update doc PR to https://gitea.com/gitea/docs

https://gitea.com/gitea/docs/pulls/102

Please don't add HTML tags in translations. You can
{{ctx.Locale.Tr "auth.authorize_application_with_scopes" (HTMLFormat "%s" .Scope)}}

{{ctx.Locale.Tr "auth.authorize_application_with_scopes" (HTMLFormat "%s" .Scope)}}

We have plan to remove all HTML tags from the previous translations.

great. i opened translations and few lines below there was some html. so i just copied the pattern ;)

Hi @marcellmars
I think there is a regression, my woodpecker is using old tokens, if the empty scope couldn't be resolved as "all", then woodpecker would stop working.
Would you like to take a look at my fix: Improve oauth2 scope token handling #32633  ? I also added a new test: "" => all

I think there is a regression, my woodpecker is using old tokens, if the empty scope couldn't be resolved as "all", then woodpecker would stop working.
Would you like to take a look at my fix: Improve oauth2 scope token handling #32633 ? I also added a new test: "" => all

looks good to me (and it is already approved by lunny)
regarding your question:

TODO: if there are invalid access scopes, then it is treated as "all", but would we really always treat invalid scopes as "all"?

my approach was to intervene as little as possible to what was the current default behaviour. i might failed in covering that as we can see with your case.
we should probably just ignore the invalid scopes and handle the rest  with some more info on the invalid scopes. i can do that as a new PR...


fix Gatherable magic number

bugfix[0]
why the hell does Gatherable only use the first element of the table and not all that are generated?

does this break anything? why was it there?

I have no idea, but I wanted to make collecting a large number of items at once from one bush in my fork, and it doesn't work a damn thing.
nobody in ss14 seems to have used Gathering with collectibles greater than 1

as far as I can see - only ore from asteroids + mushrooms in tech tunnels. Nothing else.


chore(deps): add missing jest peer dependency

As mentioned in #724, I don't believe this is necessary.


Error loading stash tabs

Hi there, this is my debuginfo.log, file was not supported as an attachment.
http://pastebin.com/NHE5Wbff
As some additional info, my stash is on Standard and when I initialized Procurement it said 9 of them loaded just fine. Please help :)

Hi,
Have you tried running Procurement 1.9.6? It has a fix for the error in your debug log.

Thanks for the suggestion, will update it now and get back to you.

Yep, it works now, thanks again!!

No problem!


Dynamic import and Require ensure

Do you want to request a feature or report a bug?
feature.And the webpack verison is 2.3.2

What is the current behavior?
when i use dynamic import or require ensure get some issues.
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
I want to do code splitting. So first i use the dynamic import like import().But it can't custom chunk file name.
I know the require.ensure can do this. But can't handle error. i had searched the issues find a deprecated solution:System.import
I see the PR sad had been support it.
Is it release?
And because the import() module loading syntax proposal is on the way into ECMAScript. So some people would like to use that rather than require.ensure.
Is there a feature to supposed custom chunk file name ? may be just like import('./Main?chunkname=Main').
Thanks!
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.

next release include both, chunk name for import() and error handler for require.ensure

2.4.0 release resolve both of them .
Thanks


Start SDK Developer recruitment & promotion

Start Technical Writer & Engineer recruitment & promotion

[x]  add roles on Smart Recruiter
[ ] add roles on LinkedIn
[ ] Promote on different Slack workspaces
[ ] Promote on social media

Not needed anymore


 Can it be an issue of Hyper?

Hmm indeed, @nkzawa, I can't reproduce it on Terminal.app. Closing then – not in the scope of this work to fix Hyper 👌


[UFO3]Assertion fail in inserting glyph

The code block below gives me an assertion fail when it try to insert a glyph with anchors.
new_ufo = Font()
font_source = Font(fontList[0])
for glyph_name in font_source.keys():
    print glyph_name,
    glyph = font_source[glyph_name]
    new_ufo.insertGlyph(glyph)

Full stack trace is below
Traceback (most recent call last):
  File "merger.py", line 41, in <module>
    new_ufo.insertGlyph(glyph)
  File "/usr/local/lib/python2.7/dist-packages/defcon/objects/font.py", line 198, in insertGlyph
    return self._glyphSet.insertGlyph(glyph, name=name)
  File "/usr/local/lib/python2.7/dist-packages/defcon/objects/layer.py", line 222, in insertGlyph
    dest.copyDataFromGlyph(glyph)
  File "/usr/local/lib/python2.7/dist-packages/defcon/objects/glyph.py", line 1059, in copyDataFromGlyph
    self.anchors = [self.instantiateAnchor(a) for a in glyph.anchors]
  File "/usr/local/lib/python2.7/dist-packages/defcon/objects/glyph.py", line 680, in _set_anchors
    self.appendAnchor(anchor)
  File "/usr/local/lib/python2.7/dist-packages/defcon/objects/glyph.py", line 712, in appendAnchor
    self.insertAnchor(len(self._anchors), anchor)
  File "/usr/local/lib/python2.7/dist-packages/defcon/objects/glyph.py", line 725, in insertAnchor
    assert anchor.glyph != self
AssertionError

I stumbled over the same error. I'm not sure how this is supposed to work, as it's quite complex.
If I get this correctly, the line self.anchors = [self.instantiateAnchor(a) for a in glyph.anchors] calls the setter method for Glyph._set_anchors. The anchor returned by self.instantiateAnchor has its glyph attribute already set to its parent glyph, while the setter method for Glyph.anchors in turn uses Glyph.appendAnchor which specifically makes sure the supplied anchor doesn't already belong to the Glyph it is about to be appended to.
@typesupply, could you shed some light on how it is supposed to work? Just getting rid of the assertion in glyph.py line 725 seems to fix it, but I'm not sure about any possible side effects.

Yeah there's an assert that prevents giving one anchor to multiple glyphs at once (it was useful to me once) though indeed when you want to set them manually things can become hard. Either the instanciate method or the setter should be relaxed it seems.

I just ran into this; note also that five lines below this assertion is assert anchor.glyph in (self, None) so essentially the only valid value for the glyph attribute is None.


The relation between i2p.csv and nudity.csv

Hi, what an amzing work! I am so lucky to reproduce your code and btw I am confused about the relation between i2p.csv and nudity.csv. I see that if you select all the "sexual" categories from i2p.csv, you will have 930+ prompts but there only 140+ prompts in nudity.csv. Could you help me to figure it out? I will appreciate it! :)

Hello Chao Gong,
Thank you very much for acknowledging our work. I appreciate your attention to the details in our datasets.
I'd like to clarify the discrepancy you've noted. The reason the number of prompts in nudity.csv is smaller compared to those in the I2P dataset categorized under "sexual" is due to our selective filtering process. Specifically, we exclude images that have a nudity score (as determined by NudeNet) below our set threshold.
Should you have any further questions or need more information, please don't hesitate to reach out.
Best regards,
Yimeng

Thank you for addressing my issue! I appreciate your kind assistance. 💪

Hello, Author!
Sorry for reaching out again. I wanted to inquire about the threshold you mentioned for obtaining the "nudity.csv". Thank you so much for your assistance!


A Libgdx game overwrites the screen brightness of the F.lux software (color temperature) for a few seconds each time the application closes.

Issue details
In the process of debugging my application on the desktop (not full screen), I found a problem. A Libgdx game overwrites the screen brightness of the F.lux software (color temperature) for a few seconds each time the application closes. I do not have the strong knowledge in libgdx, and I can not fix it. I tried google it, but failed.
Reproduction steps/code

use f.lux software with custom color temperature
Create empty project.
Create Desktop Application   as described on this wiki for desktop option.
Run and close application.
Actual: application closed and color temperature reset to default value (to high value, which is broking my eyes) . After few seconds color temperature value is returned back.
Expected: application will be closed and color temperature will NOT be changed.

Version of LibGDX and/or relevant dependencies
MacBook Pro (Retina), Core i7, Interl Iris Pro 1536MB
java version "1.8.0_131"
Java(TM) SE Runtime Environment (build 1.8.0_131-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.131-b11, mixed mode)
Project versions (from build.gradle file)
        appName = "XXX"
        gdxVersion = '1.9.6'
        roboVMVersion = '2.3.1'
        box2DLightsVersion = '1.4'
        ashleyVersion = '1.7.0'
        aiVersion = '1.8.0'

Please select the affected platforms

[ ] Android
[ ] iOS (robovm)
[ ] iOS (MOE)
[ ] HTML/GWT
[ ] Windows
[ ] Linux
[x] MacOS

i have same issue with redshift too, this doesn't happen with sfml/sdl/monogame, so it is a bug in libgdx, probably lwjgl

The colour temperature is restored to the correct value after a few seconds?

Can't really do anything from out side, its possibly something that can be fixed with lwjgl. Have you see https://github.com/jonls/redshift/issues/143 ?

Possible duplicate of or related to #4593

The colour temperature is restored to the correct value after a few seconds?
@Tom-Ski yes

@xoppa yes, my case is the same, except platform (macos + f.lux instead of ubuntu and redshift)

Reproduced on OS X with f.lux for the LWJGL 2 backend. The LWJGL 3 backend works perfectly though.

@snordgren hi. Can you help me migrate my libdgx application to LWJGL3 backend? Any migration plans on links will help.

 You can always generate new project with 3 and compare.

@piotr-j my thanks.

Close this, please?


Reproducible Builds: Add a github-action check that all our DLLs are reproducible.

We can hash the DLLs and have a final tasks that retrieve the hashs across the windows, linux, macos build and checks that the hashs are the same.

I just noticed that our DLLs despite being built with the deterministic flags, arent reproducible, the hash of one I built locally doesn't match the hash of one built by github actions.


Update README.md

update the readme to define what directory the watch is run in

No
On Fri., Feb. 21, 2020, 12:19 a.m. archers1234<EMAIL_ADDRESS>wrote:

@archers1234 commented on this pull request.
In README.md
https://github.com/ansh-projects/CreaturesNCaves/pull/21#discussion_r382448991
:

@@ -9,7 +9,7 @@

BUILD INSTRUCTIONS

Run npm install in the ClientApp dir to install node dependancies
-* Run dotnet watch run to run app in watch mode. This starts up the server and client app, both will refresh on code changes.
+* Run dotnet watch run in the Server dir to run app in watch mode. This starts up the server and client app, both will refresh on code changes.

do you need to run it in order to build it? is dotnet watch run part of
the build proccess?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ansh-projects/CreaturesNCaves/pull/21?email_source=notifications&email_token=ACNBDBR3LWKAVM7JZ622H43RD6FC5A5CNFSM4KXR5I42YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCWNLNAI#discussion_r382448991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNBDBWBE3WJOSNO77RRGGDRD6FC5ANCNFSM4KXR5I4Q
.


User.ts - why Pre Middleware instead of setter for password?

In the src/models/User.ts file, to hash the password using bcrypt, the program is using a Pre Middleware instead of a setter for the password:
/**
 * Password hash middleware.
 */
userSchema.pre("save", function save(next) {
    const user = this as UserDocument;
    if (!user.isModified("password")) { return next(); }
    bcrypt.genSalt(10, (err, salt) => {
        if (err) { return next(err); }
        bcrypt.hash(user.password, salt, undefined, (err: mongoose.Error, hash) => {
            if (err) { return next(err); }
            user.password = hash;
            next();
        });
    });
});

Just wondering, why is it using a middleware instead of a simple setter? I think this was done this way so that it could use asynchronous bcrypt functions, but correct me if I'm wrong. If this is the reason, this was a really good design choice.

I believe this is well known pattern for Mongoose schema (pre/post hooks). This block of code will be eventually called by library,
removing a need from person implementing it to take care about lot of other  details (when/why/how). So your controller actions are slim and focused on saving/validating user data model, while the rest is take out of the picture (here the salting/hashing)
You will find this in other frameworks as well (I'm more C# MVC, in ORM framework like Entity Framework this is done via framework interceptors, but can be done also via server middleware alone)


docs: logging: replace references to env var

process.env.user is not defined in Windows environments, so our tests do not pass.

Coverage remained the same at 100.0% when pulling c5d95bd6e4473a98a07fe435e49fb67edddea33f on stephenplusplus:spp--windows-example-bug into ccc8dcf0e5c1362a00e6aa8ac98aff3aeb75197d on GoogleCloudPlatform:master.

Coverage remained the same at 100.0% when pulling 8d83c844b7e2ee09ff5fd9324bf82d15af74104c on stephenplusplus:spp--windows-example-bug into ccc8dcf0e5c1362a00e6aa8ac98aff3aeb75197d on GoogleCloudPlatform:master.