gRPC server doc

add grpc-server.md

fix bug: gRPC only support ip/hostname:port

Codecov Report
Patch coverage: 71.42% and project coverage change: +0.14 :tada:

Comparison is base (6ff9ac6) 81.67% compared to head (aff5187) 81.81%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1036      +/-   ##
==========================================
+ Coverage   81.67%   81.81%   +0.14%
==========================================
  Files         135      135
  Lines       15044    15053       +9
==========================================
+ Hits        12287    12316      +29
+ Misses       2199     2181      -18
+ Partials      558      556       -2

Impacted Files
Coverage Δ

pkg/filters/proxies/grpcproxy/loadbalance.go
95.65% <ø> (-0.65%)
:arrow_down:

pkg/filters/proxies/grpcproxy/pool.go
43.56% <71.42%> (+3.14%)
:arrow_up:

... and 5 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Hi, thanks a lot for your pr. Several days a ago, a user report an bug that pkg/protocols/grpcprot/request.go Host() method may return host with port, which cause match fail. Could you please to find a graceful way to solve this problem?

I got, I will try to fix


Registering components in V5 as jQuery plugins prevents treeshaking

It is nice that V5 removes dependency on jQuery, but every component still try to register as jQuery pluging if jQuery is available. But that makes all components as used in every treeshaking or dead code elimination bundler.
It would be good to have bootstrap build version without this code making applicactions much smaller if they will use only some components.

PRs welcome.

PRs welcome.


feat: make backspace command support not closing the input when value is empty

I'm not always decide to delete all content when I start to delete them. Usually when I have deleted half of the content, I realized the rest of them is wrong too, so I'll keep backspace to delete all of them.

I just got another report about this issue: https://github.com/sxyazi/yazi/issues/1681. It seems like enough people are bothered by this behavior, so I think we can just remove it since it's a breaking change anyway, and it's also hard to come up with a way (like warnings) to prompt users with the existing backspace --under config to replace it

What about change this option to --no-close to make it not break?

No, just remove it - if enough people are bothered by it, then its worth breaking.
I'll also keep this PR open for a while to see if there are any differing opinions


Exception thrown in Sneaker when capturing an exception (ErrorException: Trying to get property of non-object)

I have just one address in config file: app/sneaker.php but this doesn’t meter because I tried with two addresses also.
So, the solution for me was to add a foreach in capture($exception) function , class Sneaker.php:
private function capture($exception)
{
    $recipients = $this->config->get('sneaker.to');

    $subject = $this->handler->convertExceptionToString($exception);

    $body = $this->handler->convertExceptionToHtml($exception);

    foreach ($recipients as $recipient) {
        $this->mailer->to($recipient)->send(new ExceptionMailer($subject, $body));
    }
}

Can you correct this function?
Laravel version 5.3.6!
Thanks

local.ERROR: exception 'ErrorException' with message 'Trying to get property of non-object' in /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Mail/Mailable.php:376
Stack trace:
#0 /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Mail/Mailable.php(376): Illuminate\Foundation\Bootstrap\HandleExceptions->handleError(8, 'Trying to get p...', '/var/www/html/s...', 376, Array)
#1 /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Mail/Mailable.php(321): Illuminate\Mail\Mailable->setAddress(Array, NULL, 'to')
#2 /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Mail/MailableMailer.php(138): Illuminate\Mail\Mailable->to(Array)
#3 /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Mail/MailableMailer.php(121): Illuminate\Mail\MailableMailer->fill(Object(SquareBoat\Sneaker\ExceptionMailer))
#4 /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Mail/MailableMailer.php(96): Illuminate\Mail\MailableMailer->queue(Object(SquareBoat\Sneaker\ExceptionMailer))
#5 /var/www/html/simdev/vendor/squareboat/sneaker/src/Sneaker.php(111): Illuminate\Mail\MailableMailer->send(Object(SquareBoat\Sneaker\ExceptionMailer))
#6 /var/www/html/simdev/vendor/squareboat/sneaker/src/Sneaker.php(81): SquareBoat\Sneaker\Sneaker->capture(Object(ErrorException))
#7 /var/www/html/simdev/app/Exceptions/Handler.php(47): SquareBoat\Sneaker\Sneaker->captureException(Object(ErrorException))
#8 /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Routing/Pipeline.php(79): App\Exceptions\Handler->report(Object(ErrorException))
#9 /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Routing/Pipeline.php(55): Illuminate\Routing\Pipeline->handleException(Object(Illuminate\Http\Request), Object(ErrorException))
#10 /var/www/html/simdev/app/Http/Middleware/RedirectIfNoStudentCompany.php(32): Illuminate\Routing\Pipeline->Illuminate\Routing{closure}(Object(Illuminate\Http\Request))
#11 /var/www/html/simdev/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(137): App\Http\Middleware\RedirectIfNoStudentCompany->handle(Object(Illuminate\Http\Request), Object(Closure))

@trinca Can you specify which version of Laravel you are using?
Can you also check if from key is set in file config/mail.php?

Laravel version 5.3.6!
Yes the from key is set.

@trinca Closing this issue for now. If you face the issue again then feel free to open it again.


Finalizing language integration

Thai and Dutch now with the program.
(What is the use of the en_US.js file? It's not complete, but its presence doesn't seem to make a difference. Unless it is preparing for a refactoring.)

You are right about the US file..., unless we refactor we don't need it. So
maybe also delete it in your branch and update the pull request and I will
merge it in together :)
On 30 Oct 2015 22:51, "PePa"<EMAIL_ADDRESS>wrote:

Thai and Dutch now with the program.
(What is the use of the en_US.js file? It's not complete, but its presence
doesn't seem to make a difference. Unless it is preparing for a
refactoring.)
You can view, comment on, or merge this pull request online at:
https://github.com/ziahamza/webui-aria2/pull/172
Commit Summary

Added Language
Merge branch 'master' of https://github.com/ziahamza/webui-aria2
Amended language names in index.html

File Changes

M index.html
https://github.com/ziahamza/webui-aria2/pull/172/files#diff-0 (4)
M js/translate/nl_NL.js
https://github.com/ziahamza/webui-aria2/pull/172/files#diff-1 (1)
M js/translate/th_TH.js
https://github.com/ziahamza/webui-aria2/pull/172/files#diff-2 (1)

Patch Links:

https://github.com/ziahamza/webui-aria2/pull/172.patch
https://github.com/ziahamza/webui-aria2/pull/172.diff

—
Reply to this email directly or view it on GitHub
https://github.com/ziahamza/webui-aria2/pull/172.

Something changed and the en_US.js is now mandatory. I updated it to include all strings from the template.

Merged it in with the refactor :)

The problem with the current situation is, that the missing strings in Thai and Chinese no longer give the English, so it's just empty... Can we change this?? That way we can just gradually translate interfaces without having to demand full translations.
The other option is to 'manually' supply all English strings in every translation file, but that feels wrong...

Just pushed in a change, to automatically merge english translation in case anything is missing. The master should now have the change :)

Can you also automatically include all the language files in js/tranlations ?? Then future inclusions don't need to modify js/init.js and index.html


Should primary CO2 include distribution losses?

This is a bit of a philosophical question: "Why would you also take oil spilling into the sea, gas leaking from the pipes and coal dropping from the conveyor belt into account when calculating CO2 emissions (as we currently do)?"
Is this convention?

It is not convention (I believe), but just how our CO2 calculation works. IF we decide to change this, and account for CO2 emissions of actual energetic use, we would need to first map out how large the discrepancy is and if we can reproduce official numbers.
One issue might be that for compiling official usage statistics, losses are not always known precisely and it is simply assumed or reported that everything is used.

Thanks for your answer @AlexanderWirtz!

we would need to first map out how large the discrepancy is

I checked for the Dutch dataset and we currently assume lossless transportation and distribution for the major carriers (gas, coal, oil, biomass).
Also in the refinery sector, the losses of non-energetic oil are zero.
Closing this as I don't think this is worth pursuing anytime soon.


[Question] Line chart - Draw Y axis on the right side

Hello,
I was wondering if there is a way to draw Y axis on the right side of the chart.
I read there are some paddings to apply to the render functions.
https://github.com/indiespirit/react-native-chart-kit/blob/e39d8b67625c3dd1e4e0b8628b8240e3c8e64903/src/line-chart.js#L304-L319
Is there any example about that ?
Thanks.

I don't think I've seen anyone else doing that. You'd have to fork this project.


MD013 (line-length) and emphasized links

It is impossible to break these examples into multiple lines:
*[Open a pull request](https://help.github.com/articles/using-pull-requests/ "Using pull requests · GitHub Help")*

**[Open a pull request](https://help.github.com/articles/using-pull-requests/ "Using pull requests · GitHub Help")**

I think MD013 (line-length) should ignore these cases.

Whitespace/newlines are supported with links: http://spec.commonmark.org/0.27/#links
*[Open a pull request](
https://help.github.com/articles/using-pull-requests/
"Using pull requests - GitHub Help"
)*

Link references can also be used and also support whitespace/newlines: http://spec.commonmark.org/0.27/#link-reference-definitions
*[Open a pull request][opr]*

text

[opr]:
https://help.github.com/articles/using-pull-requests/
"Using pull requests - GitHub Help"

I think this covers your scenario without needing changes to MD013?
Link to playground to try

@DavidAnson If the goal of markdownlint is to support only the CommonMark specification, then yes, it does. And in that case 0c1f403 should be reverted.

I like using the CommonMark specification because it is clear, well written, and seems to be the thing most people are moving toward. I realize not all parsers completely support it, but it acts as a good reference point and basis for decision-making.
In the case of the previous issue, perhaps I should not have supported that scenario. However, it seems like the kind of thing an average user might try to do and it's nice to give them something that works without having to read a specification or change to a different parser.
This issue is only slightly more complicated, so I will probably handle it as well - but the thing I want to avoid is chasing more scenarios like this that are increasingly difficult and rare in practice.

@DavidAnson A linter absolutely should give a warning if a piece of code can be rewritten in a better way. But again, if the goal of markdownlint is to help write markdown that is going to be properly rendered in all the different markdown parsers, then markdownlint should stick to a subset of markdown and should not warn on the cases where the code can't be rewritten.


Allow local testing while using both moto & fleece X-Ray

moto is very handy for testing lambda functions offline. However, they have not yet implemented X-Ray support (spulec/moto#1006).
This creates a conflict when trying to test code that uses fleece's X-Ray support with moto.
I'm not sure if this is a widespread issue, but I'll attach a pull request for my workaround.

I can no longer reproduce (see comment on PR). I'll reopen if I can get a reproduction.


iOS Native linking failed

I am not able to get the plugin to run on iOS. It is working well in my Android Project. I get the following build error. Maybe I missed some configuration, but I followd the getting started guide.
Bug Information
Version Number of Plugin: 2.3.6
Simulator Tested On: iOS Simulator
Version of VS: VS Professional 2019
Version of Xamarin: <IP_ADDRESS>5677
Steps to reproduce the Behavior

Create Xamarin Forms Project
Install FirebasePushNotification package on all projects
Follow Getting Started Guide
Run Debug Config on iOS Simulator

Actual Behavior
Build Failed
Output
3>  App1.iOS -> C:\Users\MH\source\repos\App1\App1\App1.iOS\bin\iPhoneSimulator\Debug\App1.iOS.exe
3>  Detected signing identity:
3>    Bundle Id: com.companyname.App1
3>    App Id: com.companyname.App1
3>  /Library/Frameworks/Xamarin.iOS.framework/Versions/Current/bin/mtouch @/Users/martin/Library/Caches/Xamarin/mtbs/builds/App1.iOS/de81a7b852ce64b5acf94d1823606fec/obj/iPhoneSimulator/Debug/response-file.rsp --gcc_flags=-ObjC
3>C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Xamarin\iOS\Xamarin.iOS.Common.targets(797,3): error : linker command failed with exit code 1 (use -v to see invocation)
3>C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Xamarin\iOS\Xamarin.iOS.Common.targets(797,3): error MT5210: Native linking failed, undefined symbol: _FIRLogBasic. Please verify that all the necessary frameworks have been referenced and native libraries are properly linked in.
3>C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Xamarin\iOS\Xamarin.iOS.Common.targets(797,3): error MT5211: Native linking failed, undefined Objective-C class: FIROptions. The symbol 'OBJC_CLASS$_FIROptions' could not be found in any of the libraries or frameworks linked with your application.
3>C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Xamarin\iOS\Xamarin.iOS.Common.targets(797,3): error MT5211: Native linking failed, undefined Objective-C class: GULAppEnvironmentUtil. The symbol 'OBJC_CLASS$_GULAppEnvironmentUtil' could not be found in any of the libraries or frameworks linked with your application.
3>C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Xamarin\iOS\Xamarin.iOS.Common.targets(797,3): error MT5201: Native linking failed. Please review the build log and the user flags provided to gcc: -ObjC
3>C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Xamarin\iOS\Xamarin.iOS.Common.targets(797,3): error MT5202: Native linking failed. Please review the build log.

@mheydecke how did you solve it?

@mheydecke how did you solve it?


Cannot import modules in immediate window

In VS 2017 I was able to import python modules using the immediate window. When I try that in 2019 I get the error
SyntaxError('invalid syntax', ('', 1, 6, 'import pandas'))

I was able to "import pandas" on my local build.  Double check you have the correct Environment set in the Interactive window.

I think @jose-alex is referring to the debugger's immediate window. The difference here is that VS 2017 uses the legacy debugger (ptvsd 3.x) where VS 2019 uses ptvsd 4.x.
An alternative that works is to use the Python Debug Interactive window.

Like huguesv mentions, I am referring to the immediate window.
I can import modules in the Python Debug Interactive window, however as I reported in #5800 , one has to be extremely careful not to make any syntax mistakes in that window because any error stops the debugger. This is why I wanted to use the immediate window.

This scenario is not supported in the immediate window. #5800 is fixed and should be in the next release VS16.5


setup issues

Creating separate issue (deleting the previous due to a conflict)
I tried to rebuild in a new virtual environment in several different ways.
The development builds work.
The user build doesn't.
@parijatm Maybe we should change the README.md so that we don't suggest that as a way to do build it.
Fails:
pip install -v .
Works:
pip install -e .[test]
Works:
pip install setuptools wheel pybind11 scikit-build cmake ninja
pip install -v -e .[test] --no-build-isolation
Works (what I recommend if you are a python developer)
pip install setuptools wheel pybind11 scikit-build cmake ninja
pip install -v -e .[test] --no-build-isolation
pip install -v --no-build-isolation -e . --no-deps --install-option='--build-type=Release'
The pip install itself fails. Here is the end of the error log:
...
Configuring Project
Working directory:
/tmp/pip-req-build-xf5ew3uy/_skbuild/linux-x86_64-3.8/cmake-build
Command:
cmake /tmp/pip-req-build-xf5ew3uy -G Ninja -DCMAKE_INSTALL_PREFIX:PATH=/tmp/pip-req-build-xf5ew3uy/_skbuild/linux-x86_64-3.8/cmake-install -DPYTHON_EXECUTABLE:FILEPATH=/home/smburns/DARPA/ALIGN-public/.venv2/bin/python -DPYTHON_VERSION_STRING:STRING=3.8.0 -DPYTHON_INCLUDE_DIR:PATH=/usr/include/python3.8 -DPYTHON_LIBRARY:FILEPATH=/usr/lib/x86_64-linux-gnu/libpython3.8.so -DSKBUILD:BOOL=TRUE -DCMAKE_MODULE_PATH:PATH=/tmp/pip-build-env-xevgcl07/overlay/lib/python3.8/site-packages/skbuild/resources/cmake -DALIGN_VERSION:string=0.9.8 -DCMAKE_BUILD_TYPE:STRING=Release
Traceback (most recent call last):
An error occurred while configuring with CMake.
Command:
cmake /tmp/pip-req-build-xf5ew3uy -G Ninja -DCMAKE_INSTALL_PREFIX:PATH=/tmp/pip-req-build-xf5ew3uy/_skbuild/linux-x86_64-3.8/cmake-install -DPYTHON_EXECUTABLE:FILEPATH=/home/smburns/DARPA/ALIGN-public/.venv2/bin/python -DPYTHON_VERSION_STRING:STRING=3.8.0 -DPYTHON_INCLUDE_DIR:PATH=/usr/include/python3.8 -DPYTHON_LIBRARY:FILEPATH=/usr/lib/x86_64-linux-gnu/libpython3.8.so -DSKBUILD:BOOL=TRUE -DCMAKE_MODULE_PATH:PATH=/tmp/pip-build-env-xevgcl07/overlay/lib/python3.8/site-packages/skbuild/resources/cmake -DALIGN_VERSION:string=0.9.8 -DCMAKE_BUILD_TYPE:STRING=Release
Source directory:
/tmp/pip-req-build-xf5ew3uy
Working directory:
/tmp/pip-req-build-xf5ew3uy/_skbuild/linux-x86_64-3.8/cmake-build
Please see CMake's output for more information.
Building wheel for align (PEP 517) ... error
ERROR: Failed building wheel for align
Failed to build align
ERROR: Could not build wheels for align which use PEP 517 and cannot be installed directly
Exception information:
Traceback (most recent call last):
File "/home/smburns/DARPA/ALIGN-public/.venv2/lib/python3.8/site-packages/pip/_internal/cli/base_command.py", line 180, in _main
status = self.run(options, args)
File "/home/smburns/DARPA/ALIGN-public/.venv2/lib/python3.8/site-packages/pip/_internal/cli/req_command.py", line 204, in wrapper
return func(self, options, args)
File "/home/smburns/DARPA/ALIGN-public/.venv2/lib/python3.8/site-packages/pip/_internal/commands/install.py", line 360, in run
raise InstallationError(
pip._internal.exceptions.InstallationError: Could not build wheels for align which use PEP 517 and cannot be installed directly
Removed build tracker: '/tmp/pip-req-tracker-ff6or4v4'

Parijat comment:
I think its a cmake issue (you will need to scroll a bit further up to be sure). Removing the _skbuild directory should resolve it.
The error is encountered anytime you switch from one installation target to another. So going from pip install to pip install . will likely also result in the same error.
Side note: Looks like merging #738 closed this issue automatically. Reopening as there are a few opens to resolve.

Steve comment:
Yes, removing _skbuild makes the build work. (Maybe we need to add this in the README.md.)
Now, I can't get the tests to run because it can't find the module 'colorlog' although it is install and can be imported from a fresh python REPL.
There are weird things going on.
(.venv2) smburns@smburns-XPS-13-9370:~/DARPA/ALIGN-public$ pytest
ImportError while loading conftest '/home/smburns/DARPA/ALIGN-public/conftest.py'.
conftest.py:2: in
from align.utils import logmanager
align/init.py:3: in
from .utils import logmanager
align/utils/logmanager.py:4: in
import colorlog
E   ModuleNotFoundError: No module named 'colorlog'
(.venv2) smburns@smburns-XPS-13-9370:~/DARPA/ALIGN-public$ python
Python 3.8.0 (default, Feb 25 2021, 22:10:10)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

import colorlog
exit()

Steve:
Here is a smoking gun:
(.venv2) smburns@smburns-XPS-13-9370:~/DARPA/ALIGN-public$ pytest
ImportError while loading conftest '/home/smburns/DARPA/ALIGN-public/conftest.py'.
conftest.py:2: in
from align.utils import logmanager
align/init.py:3: in
from .utils import logmanager
align/utils/logmanager.py:4: in
import colorlog
E   ModuleNotFoundError: No module named 'colorlog'
(.venv2) smburns@smburns-XPS-13-9370:~/DARPA/ALIGN-public$ python -m pytest
====================================================== test session starts =======================================================
platform linux -- Python 3.8.0, pytest-6.2.4, py-1.10.0, pluggy-0.13.1
rootdir: /home/smburns/DARPA/ALIGN-public, configfile: pytest.ini, testpaths: tests
plugins: cov-2.12.1, forked-1.3.0, timeout-1.4.2, cpp-2.0.0, dash-1.20.0, xdist-2.2.1
collected 2360 items
tests/test_build_steps.py .                                                                                [  0%]
tests/test_cmdline.py ..^C

Parijat:
I am a bit flummoxed !! 😄
Looks like the pytest command is not pointing to the .venv2/bin/pytest. Can you try a which pytest to see if that's the case?
If that is the case, its possible pytest wasn't installed in .venv2 and its instead using the python2 pytest which doesn't have colorlog installed.


Document Egress Subnet Support in VPC

Antrea supports adding subnet other than Node's network, From Antrea v1.15, IPs can be allocated from a subnet different from the Node IPs.
In VPC scenario, need to document behavior as in VPC , Egress ip from subnet other than node's subnet have to be assigned as secondary ip on primary interface. In VPC, HA case, Usually k8 operator (as in general case without VPC)  can be implemented to configure the Egress IP as the secondary IP of the primary network interface of the Egress Node instance. However for subnet other than node's subnet an additional gateway must also exist to facilitate Egress traffic.
Document following scenarios
1.Configure Egress Subnet other node's subnet in VPC
2.Handle HA case in VPC with Egress subnet other than node's network.
3.Egress gateway configuration wrt VPC.
4.Configuring multiple subnets

I don't know which VPC it refers to and whether it is really supported. The working VPC I'm aware of is only AWS and it's already documented: https://github.com/antrea-io/antrea/blob/main/docs/egress.md#egress-on-cloud

Specially We would need to elaborate cases related to using externalippool crd in egress in vpc, currently secondary interface addition in aws is documented ,although it is explanatory for static ip case, however when using externalippool crd (different from nodes's subnet/same as node's subnet) ,  as secondary interface in aws is configured manually as given  here might not work!, also for HA case for externalippools , we might like to enhance the doc wrt mentioned points in subject.
Also it might be useful to create a task for automating HA case while using antrea with Kubernetes Operator which watches the Egress API.


Input Data Format

Hi, thank you for your excellent work! After reading the article and the code, I am not very clear about the form of the input data?
The article mentioned "We load the map1 into SUMO [40], by which the distances between gates and the depot are calculated automatically. On top of the map, we first generate AGH instances of 20, 50 and 100 flights which are referred as AGH20, AGH50 and AGH100, respectively.”
As I understand it in the code, the json file is loaded, and the json file contains two dataset files: 1. ID_MAP; 2. CVRPTW. The input data is also divided into SHARED_DATA, UNSHARED_DATA and FILE_MAP.
I really don't understand but I'm curious about the form of the input content. If I wanted to use this code to do a theoretical experiment on a problem such as vrp and use self-generated data as input, the shape of the data would be: (x, y, demand), where (x, y) is the 2D coordinate. Could I directly replace the original input in the original code with the data I generate? I did not find input content such as file_map in the folder.

Hi, sorry for my late reply. I was quite busy with conference rebuttals recently.
To generate the dataset, you need to run eval_generate_json.py. Then, you will get these files: SHARED_DATA, UNSHARED_DATA, and FILE_MAP. An example of the data format can be found here. You can also check the data format by "reading" these generated files.
It may not be easy to adapt to VRP directly, since the improvement-based code is not clean. I would suggest trying the construction-based one or other repo leveraging the LNS framework, such as NLNS.

Thank you for your time, I'll try to complete it.
I'm grateful for your reply, and hope the program chair will eventually accept your paper.
Best wishes.


Add search results highlighting like fzf

The hishtory is excellent! I would be happy to switch from fzf to it to synchronize the history on all my devices.
I see you actively developing the tool! It's great!
However, underlining is sorely lacking.
For example, how it looks like in fzf.

Thanks for the FR! To confirm, your suggestion is that for each command we somehow highlight the text in the command that is matching the query?

Just FYI, I've started to work on this and have a current prototype where matches are bolded:

I'll get this released relatively soon :)

If you run hishtory update to get the latest version of hishtory, and opt-in to beta features via hishtory config-set beta-mode true, you can try this out! I'd love any feedback you have on this feature.

This is now fully released and enabled by default! Please let me know if you have any feedback/thoughts on this.


Add support async Http library

Hi,
I used this library recently, it's fantastic very convenient but it doesn't support async pattern.
I and my collegue modified the library to support aiohttp, this is the link: https://github.com/Albo90/python-pyodata/tree/async_python
If you are interested, we can collaborate to integrate it in your project.
We are going to add test and more documentation in a couple of days.
Thanks

Impressive. Unfortunately, since you copied all files, I cannot see the needed changes. Can you somehow summarize what needs to be changed?

Impressive. Unfortunately, since you copied all files, I cannot see the needed changes. Can you somehow summarize what needs to be changed?

Hi filak-sap,
we had to create a bit of redundancy for our modifications otherwise it's really difficult for us to integragate async.  I try to describe them grouping for file.
async_client

Definition of new method accord to async strategy.
Definition of _fetch_metadata with get implementation accord to async.

AsyncService

Modification of execute method into OdataHttpRequest, we use "async with" to create request and build response in aim to avoid excessive ridefionition of project structure.
Modification of http_get method.

If you want, we can make a reasoning to build other ways of integration

Is there a way how to avoid huge code duplications?

Is there a way how to avoid huge code duplications?

Yes, in my option there is possibility to avoid creation of files (async_client, asyncService) rebuilding the existent methods.

@filak-sap we published a new version without duplication. The version is in this branch:   https://github.com/Albo90/python-pyodata/tree/no_duplication_v1. The library has a little syntax changement when it is used with async (pay attention to method getattr in example).
Sync version:
import requests
from pyodata.client import Client

client = Client('http://example.com', requests)
entities = client.entity_sets.resource.get_entities().execute()
for entity in entities:
      print(entity.attr_name)

print(entities)

Async version:
import asyncio
import aiohttp
from pyodata.client import Client

async def main():
    async with aiohttp.ClientSession() as client:
        service = await Client.build_async_client('http://example.com', client)
        entities = await service.entity_sets.resource.get_entities().select('key').async_execute()
        for entity in entities:
            attr_name = await entity.getattr('attr_name')
            print(attr_name)
        return entities

async_entities = asyncio.run(main())
print(async_entities)

Support of both requests lib and aiohttp lib is definitely a great idea. I had it on my personal backlog for "experiment someday", so nice to see it done and working :) Great job even just as a submitted proposal.
I had read through the issue comments and the commits:
https://github.com/Albo90/python-pyodata/commit/543d17535dc5c4a30d38411a14690550f580c4db
https://github.com/Albo90/python-pyodata/commit/00ad1dc4feb3b0ca779a496c28794481a5fa8f31
https://github.com/Albo90/python-pyodata/commit/e5b6ec1759917186a4995b9241c57b6fe2afdacb
Few high-level questions:

"we can make a reasoning to build other ways of integration" - do you have an alternative idea to compare with the code?
Proposal (open for discussion) - wouldn't be more maintainable in the future, instead of directly modify client.py and service.py to slap async variant inside, create an adapter kind of new modules e.g. async_client/async_service? Imagine in the future pyodata.v4 module and the all four sync/async + v2/v4 combinations.
There is no new test code in the three commits. Could you pls elaborate how would you like to approach unit testing of this new feature? From what I see, pytest-asyncio seems to be the choice, but I franky only started few weeks ago to look into the asyncio library itself, so I am not an educated one. Note: I also want to create a dockerized true odata v2/v4 test service as a target for integration testing, but that will take some time and is different use case anyway.

Feature work and Releases
I would at the moment not go into creating PR to master, but rather create new "async" feature branch and do all the needful there there, so pyodata is not stuck in situation where we must release a bugfix with half-baked async feature.. so far there was no need for maintenance branches and we do a flat versioning "from master".

Created new feature branch https://github.com/SAP/python-pyodata/tree/async

Hi @liyakun,
yes. But basically all I want is to move the initialization form tests to docs :) The feature itself is finished and released, so feel free to try it. Take https://github.com/SAP/python-pyodata/tree/master/tests/integration/networking_libraries as inspiration for initiliazation.

@phanak-sap thanks a lot, I will have a try.

support tasks finally done.


cilium: Make CLI more graceful on environments with IPv6 disabled

cilium: Make CLI more graceful on environments with IPv6 disabled
Description:
Issue: Some CLI's are complaining to stderr even on environments with IPV6 disabled
Root cause: As part of CLI implementation, it is always retrieving the GlobalMaps using true flag irrespective of IPV6 is enabled/disabled
Suggested fix: Now, GlobalMaps are based on correct IPV6 status from the cilium-agent else read from the system config (absence of agent)
Affected CLIs:
    $ cilium bpf ct ...
    $ cilium bpf nat ...
    $ cilium bpf recorder ...

Fixes: #13834
Signed-off-by: Mahadev Panchal<EMAIL_ADDRESS>

test-me-please


Change in number $filter doc

I think this is not correct because since 1.3.0-RC.0 sonic-boltification this change: https://github.com/angular/angular.js/commit/2ae10f67fcde3e172f695956301ef796b68a50c2 and null and undefined are passing through

I signed it!
2015-05-28 14:35 GMT+02:00 googlebot<EMAIL_ADDRESS>
Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project, in which case you'll need to
sign a Contributor License Agreement (CLA).
[image: :memo:] Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
Reply to this email directly or view it on GitHub
https://github.com/angular/angular.js/pull/11963#issuecomment-106296030.


Remove bbolt and lnd

This is an additional fix for this commit. https://github.com/lightningnetwork/lightning-onion/commit/69fa8b4dd5fb3cfaf6763bd021c9c1c356618673

@Roasbeef sorry. After glide update, I rebased it.


BUGFIX : Crash in the parser if resource descriptor not defined

When a ressource descriptor is not defined the JSONAPIResourceParser crash
See for more details : https://github.com/joshdholtz/jsonapi-ios/issues/19
Maybe add a logging message if an RessourceDescriptor is not found.

@BenjaminDigeon Thank you for the PR! Sorry for being a :snail: on it :wink:


Is there a difference between using alexa-app and alexa-app-server

Hello there,
I am sorry if this question is silly but I would love a quick answer on that subject.
There seems to be the two following ways of serving an Alexa Skill app over https (instead of a Lambda):
var app = express();
var alexaApp = new Alexa.app('myApp');
alexaApp.express({
  expressApp: app,
  ...
});

const httpServer = require('http').createServer(app);
httpServer.listen(8080, ...);

and
var AlexaAppServer = require('alexa-app-server');
AlexaAppServer.start({
  port: 8080,
  app_dir: "myAlexaAppDir",
  ...
});

Thanks.

Alexa-app-server can host multiple apps, supports HTTPs and has a development mode simulator. I think SSL is the only major differentiator for production skills with a single function.


Add Medtronic serial number field on device selection

FYI: I won't be merging this, since it lacks functionality from @gniezen's side. Purely to ease reviewing a smaller piece of the bigger puzzle. I'll close the PR once reviewed. Not much to it, but I used it as a chance to learn more about how devices are set up and uploaded! 🎉
@gniezen, a few of metric things from the Trello card I didn't see a good place to plug in to here, since it's based the master branch as we discussed:

Medtronic upload attempted (I went ahead and defined here)
Medtronic upload successful (I went ahead and defined here)
Medtronic upload failed (I went ahead and defined here)
and also the "remember me" checkbox, but that'll be implemented in a later card

If I can help adding those on another branch, or help any with integrating this into the overall branch, please let me know!

@hntrdglss I've now merged the Medtronic branch into this one, resolved conflicts, and passed the serial number down to the device driver so that I can use it there, and pushed it to your branch.
Do you want to close this PR and create a new one against gniezen/medtronic, so that we can see what both you and I changed? You can copy and paste relevant comments from this PR over to that one.

Closed. Merged code is reopened as #343.


MongoDBAASAPI::removeSubmodel does not update the db entry

Hi team,
I think that the implementation of MongoDBAASAPI::removeSubmodel does not update the database entry correctly.
After restarting the server, all the existing submodel references are added again and let the list grow with duplicates.

Many thanks for pointing this out. We will take a look at it and come back with further details.

This issue is fixed with the latest commit in Bugfix branch. The fix will be included in the next release of BaSyx (1.0.2).


Docs: location parameter goes in the formatter constructor, not the logger constructor

What were you searching in the docs?
Overriding log record format
Is this related to an existing documentation section?
https://awslabs.github.io/aws-lambda-powertools-python/latest/core/logger/#overriding-log-records
How can we improve?
The documentation at https://awslabs.github.io/aws-lambda-powertools-python/latest/core/logger/#overriding-log-records shows that the location formatter is part of the Logger constructor. Actually, it's part of the formatter constructor. The sample code in the code box under the line "Logger allows you to either change the format or suppress the following keys altogether at the initialization: location, timestamp, level, xray_trace_id." should be updated.

Got a suggestion in mind?
No response
Acknowledgment

[X] I understand the final update might be different from my proposed suggestion, or refused.

Hey @Nogginboink, thanks a lot for helping us improve the documentation! Logger propagates these to the formatter for convenience, hence why the docs show that.

Logger propagates a few formatting configurations to the built-in LambdaPowertoolsFormatter logging formatter.

Perhaps we can bring that earlier OR alternatively also show case the longer version by instantiating a Formatter and passing that via logger_formatter - what do you think?

No problem. I've tested and it worked. We're also in the process of moving all doc code snippets as runnable code so we can verify them continuously - appreciate the feedback either way.
Closing.


Fix: import Settings class, correct S3 path

Thank you for your PR @KonstantinCodes. As I've had countless PR's about S3 so far I'd like to provide a source to the documentation that suggests using this URL as the preferred way. Otherwise I cannot approve the change.

@luceos There are 3 things that come together in this PR.

Under Section "Accessing a Bucket" you find the official schema http://bucket.s3-aws-region.amazonaws.com. https://docs.aws.amazon.com/AmazonS3/latest/dev/UsingBucket.html
using "https://%s.s3-website-%s.amazonaws.com/" is actually insecure because AWS-S3 websites do not support HTTPS. Please have a look at the documentation. Also you can quickly verify yourself, that HTTPS is disabled and you can not enable it, if you use the "s3-website" domain. https://docs.aws.amazon.com/AmazonS3/latest/dev/WebsiteHosting.html
The missing Import statement is actually causing ClassNotFound Exceptions.

Thank you for making this extension. I love it!

Thanks, your changes are confirmed and approved. I'll tag a new version asap.


XCode 6.3

Has anyone got pact-consumer-swift  working with carthage using XCode 6.3?
I've tried a few different combinations of dependencies with a forked repo changing the Cartfile but without success yet.
The carthage log is complaining about the parameters of responseString on master where the Alamofire dependency is explicitly "master" (where Alamofire has merged its branch 'xcode-6.3').

Have a look in the swift1.2 branch, it was working with one of the betas of xcode-6.3 (although I guess the dependencies have moved on from then). I will update that branch when I get a second to make sure it works with the latest 6.3 release.

Thanks for the prompt update. Yes, I tried that and received the error, 'No object named "xcode-6.3" exists'" due to the Alamofire dependency no longer having the "xcode-6.3" branch they have merged into their Master branch.

OK, gotcha. I'll have a look at it tonight and see whats happened.

@richardPFisk I think I've fixed the swift1.2 branch. Let me know if you still have trouble. I'll look at merging these things into master, seeing as XCode 6.3 has been out for a while (I was waiting for travisci to support it, but it doesn't look like that will happen anytime soon).

That worked like a dream :+1:.  Thanks very much. The completionHandler:  addition is what I was missing.


:+1: Async server start stop

Add new denops-variable:

g:denops#server#close_timeout

Add new state for denops#server#status():

closing
closed

Change behavior denops-function:

denops#server#start() can now be called even when status is closing. In that case, the status will become stopped and then restart asynchronously.
denops#server#start() is changed to asynchronously.

It can now be called even when status is closing. In that case, the status will become stopped and then restart asynchronously.
If process startup fails, it does not retrie synchronously (does not use retry_threshold), it retries asynchronously (uses restart_threshold).

denops#server#stop() is changed to asynchronously.

It waits for the server to close gracefully, and force terminate if timeouted.

denops#server#restart() is changed to asynchronously.

Perform the stop and start steps above.

Summary by CodeRabbit

New Features

Enhanced error handling for job executions, especially on Windows environments.
Introduced new server management functions, including status checks and forced channel closures.
Added a close() method to the Service interface for better resource management.
Improved stream handling with additional parameters for stdout and stderr.

Bug Fixes

Addressed issues with job failures and callbacks by restructuring logic and adding error handlers.

Refactor

Replaced for loops with while loops for better connection retries.
Refactored function signatures to enhance parameter handling and improve code readability.

Documentation

Updated documentation to reflect changes in server behavior, timeout settings, and new functionalities.

Tests

Added new test cases for server management scenarios, including status transitions and stream handling.
Updated existing tests to align with new function signatures and enhanced destructuring of parameters.

I committed 👍 remove retry_interval, retry_threshold from proc#start() to make it easier to see the diff, and after review I want to amend it to 👍 change denops#server#start and ...#stop to asynchronous because it breaks tests.

@coderabbitai review

@Milly It seems the status is still draft. Let me know when the PR become ready (I already reviewed in case)

I rebased it. @lambdalisue


Add a benchmark using Nx.Batch.stack and Nx.serving with AxonOnnx

Reference is here:
https://qiita.com/RyoWakabayashi/items/2bc7b834001ba0d2906e

Branch: feature/67
remote:
remote: Create a pull request for 'feature/67' on GitHub by visiting:
remote:      https://github.com/zeam-vm/pelemay_backend/pull/new/feature/67
remote:

This requires implementing #75


Added Anuraag Tummanapally to Contributors List

Added My Name to Contributors.md

@TummanapallyAnuraag, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


az feature sub-commands non-functional

Describe the bug
There is no way to set the provider namespace for the az feature sub-commands. The namespace Microsoft.Features is always used.
To Reproduce
$ az feature list --help

Command
    az feature list: List preview features.

Arguments
    --namespace: The resource namespace, aka 'provider'.

$ az feature list --namespace "Microsoft.Network"
The resource namespace 'Microsoft.Features' is invalid.

$ az feature register --namespace "Microsoft.Network" -n "AllowMultipleIpConfigurationsPerNic"
Once the feature 'AllowMultipleIpConfigurationsPerNic' is registered, invoking 'az provider register -n Microsoft.Network' is required to get the change propagated
The resource namespace 'Microsoft.Features' is invalid.

Expected behavior
The correct provider namespace is used.
Environment summary
Running against an Azure Stack cloud.
Install Method
Homebrew on MacOS 10.13.4 in ZSH.
$ az --version
azure-cli (2.0.32)

acr (2.0.24)
acs (2.0.32)
advisor (0.5.1)
ams (0.1.0)
appservice (0.1.32)
backup (1.1.1)
batch (3.2.2)
batchai (0.2.2)
billing (0.1.8)
cdn (0.0.14)
cloud (2.0.13)
cognitiveservices (0.1.13)
command-modules-nspkg (2.0.1)
configure (2.0.15)
consumption (0.3.1)
container (0.1.23)
core (2.0.32)
cosmosdb (0.1.21)
dla (0.0.19)
dls (0.0.21)
dms (0.0.1)
eventgrid (0.1.12)
eventhubs (0.1.3)
extension (0.0.13)
feedback (2.1.1)
find (0.2.9)
interactive (0.3.20)
iot (0.1.20)
keyvault (2.0.21)
lab (0.0.22)
monitor (0.1.6)
network (2.1.0)
nspkg (3.0.2)
profile (2.0.24)
rdbms (0.2.2)
redis (0.2.13)
reservations (0.1.2)
resource (2.0.28)
role (2.0.23)
servicebus (0.1.2)
servicefabric (0.0.12)
sql (2.0.25)
storage (2.0.32)
vm (2.0.31)

Python location '/usr/local/opt/python/bin/python3.6'
Extensions directory '/Users/apeschel/.azure/cliextensions'

Python (Darwin) 3.6.5 (default, Apr 11 2018, 11:10:42)
[GCC 4.2.1 Compatible Apple LLVM 9.1.0 (clang-9<IP_ADDRESS>)]

Legal docs and information: aka.ms/AzureCliLegal

So it was actually a case of a misleading error message. The az feature subcommands appear to be functional.


Nullsafety

Nullsafety with avoidance of nullable variables. All tests are working.

Let me take a look at this, I will try to update it with current master


Fixes #2601 - Adds ability to filter TreeView branches/leaves

Fixes #2601 - Adds ability to filter TreeView branches/leaves (e.g. free text search).
I've created this targeting develop as it has quite a small footprint and its a really nice to have core feature I think.
It should be pretty easy to cherry pick to v2_develop
Because a tree is hierarchical its important to consider whether a filter matches the parent or child too.  I went for greatest matching so:

If a branch matches then all sub branches/leaves show even if they don't match the filter
If a leaf/branch matches all its parents are automatically shown (to expose the matching child)

As with all things in TableView it is only the expanded nodes that get processed.  If a node is not expanded its children are unknown (so not considered for matching).  This is because of how the table builds itself as the user expands rather than loading everything up front (which can be impossible for some data structures - e.g. file system).

Draft as now so I can write some tests
Pull Request checklist:

[x] I've named my PR in the form of "Fixes #issue. Terse description."
[x] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[x] My code follows the Terminal.Gui library design guidelines
[x] I ran dotnet test before commit
[x] I have made corresponding changes to the API documentation (using /// style comments)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any poor grammar or misspellings
[x] I conducted basic QA to assure all features are working

Ok after a bit of trial and git horror I have squashed. to 9c11f61
I will open another PR to cherry pick into v2_develop


Update cl_khr_extended_async_copies tests to the latest extension version

Update the 2D and 3D extended async copies tests. Previously they were based on
an older provisional version of the extension.
Also update the variable names to only use 'stride' to refer to the actual
stride values. Previously the tests used 'stride' to refer to the end of one
line or plane and the start of the next. This is not the commonly understood
meaning.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Awesome!
Is this PR sufficient to fix #1423?

I believe so, these tests cover all 4 of the new builtins from the extension

Looks like Mobileye and KALRAY were active on the extension proposal, so tagging @aharon-abramson and @Hopetech if they can spare the time for a review.

Thanks taking the time to review this @ayalz & @Hopetech. I've pushed a commit cleaning up the code with the suggestions. Hopefully it's not too noisy, as I reshuffled some variables and made them const.

Thanks for fixing! Adding some additional brackets for removal, while we're at it.

As we've started cleaning up more of the kernel code with these brackets removals, I decided to go a bit further and use C++ string literals for the kernel code which should be more readable.


CNN classification，cost does not converge

With CNN classification, I run your code, but the loss function does not converge, what is the reason？Your map can be put to look at it

There can be several reasons for that. Following are the suggestions:  training large model, tuning learning rate, use momentum. Keep in mind this is not an easy to solve problem. Please look at the discussion on blog post related to that for more pointers.

When you run this “Urban Sound Classification using RNN” code yourself, is the loss function converge? i I would like to know the network structure is ok or that the extraction of the data feature is ok.
I modified the learning rate, adjusted the network structure, but still not convergence, I would like to know the results of your operation is convergence? The problem is that there is a problem with my operation or the code I am modifying

Very slowly! It's not that simple to let these model learn and converge. This problem is particularly hard in itself.


[5.6] Add missing phpredis connection parameters to PhpRedisConnector.

At the moment, not all of the connection options are supported for phpredis. Namely, the persistent_id parameter for pconnect and the reserved and retry_interval parameters for connect are missing.
The persistent_id parameter is particularly useful, since multiple connections will use the previous state of the connection, including any options passed via setOption and select operations. Configuring multiple connections with different persistent_id values isolates these.
This PR adds support for the 3 missing parameters in their respective cases via config options.
This doesn't break any backwards compatibility. The order of the parameters has not changed and the additional parameters are added with the defaults from phpredis code. Any previous connections to Redis will remain the same, unless the new parameters are configured.
Note: I suspect Travis is running an old version of Redis, which doesn't support persistent connections. Should this be upgraded or the added it_persists_connection test be removed?
References:

phpredis pconnect, popen
phpredis connect, open

An update on the previously breaking test:
Travis has PHP_ZTS enabled, which PhpRedis doesn't support for persistent connections (reference). I've added a check for the test. Tests pass fine using the homestead box.

This PR is incomplete and introduces a non-existing option.
First, why does this not support the same additional retry_interval option for persistent connections?
According to https://github.com/phpredis/phpredis#pconnect-popen they are identical, and I can confirm from checking the C sources that they are.
Second, reserved is not an option. You're simply supposed to omit the fourth argument by passing NULL.
This is an implementational detail from how both connect() and pconnect() just pass through their arguments to the internal redis_connect(), together with a flag for persistence as the last argument: https://github.com/phpredis/phpredis/blob/4533920fde608e12fe763cf2c977ed711c77e120/redis.c#L872
That fourth argument is persistent_id, which is redundant with non-persistent connections.
@taylorotwell can you please revert this until the issues are addressed?

Related-ish: https://github.com/laravel/framework/issues/24748

@taylorotwell @SkepticalHippo ?


Get Dependency Graph and Task Config for each Task

I have two feature requests for the API
Allow to fetch a graph for how the Jobs and Evidences are connected.
At the moment i have no clue what kind of Jobs actually will get started if I process an evidence with a set of Jobs. It would be really cool to be able to know beforehand what Jobs will get triggered based on the Evidence-Type and the initial Jobs selected.
For now i do this with some sort of twisted reflection and create a graph to get an impression of what is going to happen:
from turbinia.jobs.interface import TurbiniaJob
from turbinia.workers import TurbiniaTask
result = {}
for subclass in TurbiniaJob.__subclasses__():
    evidence_in = []
    evidence_out = []
    tasks = {}
    for name, class_ in inspect.getmembers(inspect.getmodule(subclass), inspect.isclass):
        class_hierarchy = inspect.getmro(class_)
        if TurbiniaJob in class_hierarchy:
            evidence_in += [a.__name__ for a in class_.evidence_input]
            evidence_out += [a.__name__ for a in class_.evidence_output]
        elif TurbiniaTask in class_hierarchy:
            tasks[class_.__name__] = class_.TASK_CONFIG
    result[subclass.__name__] = {"evidence_in": evidence_in, "evidence_out": evidence_out, "tasks": tasks}
n = pp.Network(directed=True)
for jobname, data in result.items():
    n.add_node(jobname, type="job")
    for ev_out in data["evidence_out"]:
        n.add_node(ev_out, type="evidence")
        n.add_edge(jobname, ev_out)
    for ev_in in data["evidence_in"]:
        n.add_node(ev_in, type="evidence")
        n.add_edge(ev_in, jobname)
    for taskname, config in data["tasks"].items():
        n.add_node(taskname, type="task", config=config)
        n.add_edge(jobname, taskname)

A visiual representation looks something like this:

Allow to fetch the Task Config for each Task
As it is possible for each evidence type to fetch the needed and possible parameters it would be amazing to be able to fetch the possible task parameters for each Task.

Looks like an interesting script!    Just in case you hadn't seen it, we have something similar in https://github.com/google/turbinia/blob/master/tools/turbinia_job_graph.py
Is this something that would be helpful to be in the API server, or is having the job graph script enough, or are there things we could add to that rather than adding it into the API server?
Another somewhat related feature request we have listed somewhere is to get this same graph for a given request after it has completed which would require tracking the same flow to understand more easily how things were processed.
Regarding getting the task config for each task given the evidence type:  I think the part that is missing in order to do that is the Job -> Task mapping, which is currently done in each Jobs create_tasks method, so it doesn't have a static mapping for the Task types.  We could potentially add another attribute similar to evidence_input and evidence_output though, and potentially even refactor out most of the create_tasks methods altogether.   That being said, it should be easy to enumerate all tasks and their task config variables if that would be useful.

Yes the possibility to get a graph of the Jobs that are going to be run would indeed be very interesting to me. Maybe it'll help if i describe the intended use case:
My idea is to get the Jobs that could possibly be run based on the Evidence-Type. As Jobs trigger other Jobs based on their Output-Types it is not always clear from the beginning what can be done in the first place. E.g. if I want to search to run a Grep-Job, this can only be done if a PlasoFile-Output is generated. This type however is only created if i run the Plaso-Job in the first place...If I know beforehand what Jobs will be run, I can provide them with the appropriate parameters to do what I want.
To do this, getting a graph that shows me the dependencies between Evidence-Types, Jobs and Tasks is the first step. The second step would be to get the parameters for each Task.
So in a nutshell I would love to have accessible through the API:

A graph representation of the dependencies preferably as json
The possibility to get the Task-Config for each Task

How should we proceed regarding both ideas?

Would a static but regenerate-able representation of this data be OK instead of putting this into the API server?  Ie. if we were to update  https://github.com/google/turbinia/blob/master/tools/turbinia_job_graph.py to include .json output and the task configs, would that be good enough to meet your needs for this?

It would be more than I was hoping for 🎉☺

How is the status on this? Did anyone have the time to look into this yet?

@HolzmanoLagrene Sorry, I haven't gotten a chance to do that yet, but I'll try to carve out some time sometime soon.

Allow to fetch the Task Config for each Task
As it is possible for each evidence type to fetch the needed and possible parameters it would be amazing to be able to fetch the possible task parameters for each Task.

I think you can already accomplish this via GET /api/evidence/types or GET /api/evidence/types/{evidence_type}


offset result issue

clipper v2, 1.2.0
this gif shows the issue of clipper offset, these polygons I offset have been simplified by clipper.
the same polygons to offset with clipper v1 do not have this issue.
note that these polygons are like terrain contours, which have many holes.

I'm sure it would help a lot, if you could post a minimal reproducible example?

It's possibly related to the following block of code in OffsetPoint() @ line 319ff in clipper.offset.cpp.
Try commenting it out and see if this issue remains.
	if (AlmostZero(cos_a - 1, 0.01)) // almost straight
	{
		group.path.push_back(GetPerpendic(path[j], norms[k], group_delta_));
		group.path.push_back(GetPerpendic(path[j], norms[j], group_delta_)); // (#418)
	}

Otherwise, as suggested, please post a minimal reproducible data.

[[(123.86515808105469, -676.05029296875),<PHONE_NUMBER>298828, -493.0843200683594),<PHONE_NUMBER>498047, -442.18402099609375), (1.4292879104614258, -376.15118408203125), (39.94844055175781, -290.8587951660156),<PHONE_NUMBER>408203, -260.5937194824219),<PHONE_NUMBER>90625, -311.4940490722656),<PHONE_NUMBER>574219, -308.74267578125),<PHONE_NUMBER>847656, -162.920166015625),<PHONE_NUMBER>484375, -61.11954879760742),<PHONE_NUMBER>048828, -22.60039710998535),<PHONE_NUMBER>3007812, -50.11407470703125), (59.20801544189453, -118.89827728271484), (-15.078919410705566, -140.9092254638672), (-105.87406158447266, -87.2575454711914), (-85.23880004882812, 20.045804977416992), (0.0536041259765625, 80.5759048461914), (96.35147857666016,<PHONE_NUMBER>8632812), (-8.200499534606934,<PHONE_NUMBER>144531), (-50.846702575683594,<PHONE_NUMBER>964844), (-206.2989959716797,<PHONE_NUMBER>261719), (-365.8783264160156,<PHONE_NUMBER>5390625), (-565.3524780273438, 77.82453155517578), (-642.3908081054688, -52.86544418334961), (-605.247314453125, -84.50617218017578), (-515.827880859375, -66.62228393554688), (-383.76220703125, 61.31632614135742), (-338.3646545410156,<PHONE_NUMBER>8896484), (-236.5640411376953,<PHONE_NUMBER>0585938), (-136.1391143798828,<PHONE_NUMBER>5390625), (-140.2661590576172,<PHONE_NUMBER>2285156), (-141.641845703125, 81.95158386230469), (-166.40415954589844, 50.310855865478516), (-207.67466735839844, 13.167387008666992), (-240.6910858154297, -51.48976135253906), (-255.8236083984375, -74.87638854980469), (-268.20477294921875, -129.90374755859375), (-277.8345642089844, -176.67701721191406), (-290.2156982421875, -230.3286895751953), (-298.4698181152344, -271.5992126464844), (-301.22119140625, -303.23992919921875), (-303.9725341796875, -333.5049743652344), (-409.90020751953125, -245.46121215820312), (-324.6078186035156, -124.40100860595703), (-244.81814575195312, -15.721977233886719), (-169.155517578125, 90.2056884765625), (-185.66372680664062,<PHONE_NUMBER>5771484), (-598.368896484375, -348.63751220703125), (-295.71844482421875, -407.79193115234375), (-213.1774139404297, -517.8466186523438), (-317.7294006347656, -612.7688598632812), (-195.29351806640625, -698.0612182617188), (-38.465545654296875, -725.574951171875)], [(-275.08319091796875, -301.8642578125), (-242.0667724609375, -288.107421875), (-192.54214477539062, -279.8533020019531), (-152.64730834960938, -274.3505859375), (-218.68014526367188, -259.2180480957031), (-246.19381713867188, -211.06910705566406), (-167.7798309326172, -164.2958526611328), (-92.11721801757812, -164.2958526611328), (1.4292879104614258, -204.19068908691406), (30.31865119934082, -261.96942138671875), (6.932024002075195, -278.4776306152344), (-56.34943771362305, -303.23992919921875), (-127.88500213623047, -292.2344665527344), (-173.28257751464844, -292.2344665527344), (-228.3099365234375, -323.87518310546875), (-239.31539916992188, -350.01318359375)], [(-21.957338333129883, -571.498291015625), (-43.96828079223633, -517.8466186523438), (-68.7305908203125, -499.9627380371094), (-68.7305908203125, -546.7360229492188), (-59.100807189941406, -570.1226196289062), (-41.216915130615234, -608.6417846679688), (-103.12269592285156, -579.75244140625), (-155.398681640625, -491.7086486816406), (-188.41510009765625, -411.9189758300781), (-173.28257751464844, -373.3998107910156), (-70.10627746582031, -351.3888854980469), (-15.078919410705566, -405.0405578613281), (37.19707107543945, -466.9463195800781), (77.09190368652344, -516.470947265625), (74.34053802490234, -594.8849487304688), (57.83232879638672, -619.6472778320312), (41.32412338256836, -632.0283813476562), (15.186127662658691, -640.2825317382812)]]
Here is the data, I've try commenting these code and the issue still remain

Here is the data, I've try commenting these code and the issue still remain

Are you using InflatePaths(), or ClipperOffset directly?
What JoinType are you using?
And please state at least one offset delta at which you're seeing these errors.

join type: miter
end type: polygon
delta: 78.5f

Using your polygon data above and the following code ...
  PathsD subject, solution;
  subject.push_back(MakePathD({ ... }));
  subject.push_back(MakePathD({ ... }));
  subject.push_back(MakePathD({ ... }));

  solution = InflatePaths(subject, 78.5, JoinType::Miter, EndType::Polygon);
  SvgWriter svg;
  SvgAddSubject(svg, subject, FillRule::NonZero);
  SvgAddSolution(svg, solution, FillRule::NonZero, false);
  svg.SaveToFile("tmp.svg", 400, 400, 0);
  System("tmp.svg");

this is what I get ...

and using -78,5 as the delta ...

and everything looks fine to me.
So please provide your source code for a minimal reproducible example.

Here is a C# demo app that's an example of a "minimal reproducible example", except that it doesn't show the sorts of issues your suggesting.
And here is a very short video demonstrating this test app:

It seems that I use ClipperOffset::AddPath to cause this issue, if I use ClipperOffset::AddPaths to add paths, then the issue is gone.

ClipperOffset::AddPath only accepts integer coordinates so you must be scaling the data you've posted above. How you're doing that I don't know, and you've been asked multiple times to produce some code to demonstrate this issue. Since yous seem unwilling to do that I'm closing this thread.

So, you suggest scaling data when using ClipperOffset as it only accepts Path64/Paths64 data type?


🐛 Handle pinning by Docker URLs for GitHub actions workflows

What kind of change does this PR introduce?
Handle Docker/other container registry URLs for GitHub actions workflows. [jobs.<job_id>.steps[*].uses](https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsuses) in GitHub actions can take GitHub repos, docker containers, and local filepaths. This change supports repos and containers but not local filepaths.

[x] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
Only public action pinning was detected.
What is the new behavior (if this is a feature change)?**
Handle Docker image pinning. Stricter regex.

[x] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
Fixes #2580.
Does this PR introduce a user-facing change?
NONE

Thanks both, I'll revisit/simplify the regex and add some tests.


[UI suggestion] Attached inputs and input groups

I wish i could chain inputs as i can chain buttons, dropdowns and segments.

  <button class="ui button">One</button>
  <button class="ui button">Two</button>
  <button class="ui button">Three</button>

<input type="text" placeholder="1">

<input type="text" placeholder="2">

<input type="text" placeholder="3">

Is this possible or can it be made possible? Right now i am using "search dropdown button" in "ui buttons" and it can be somewhat hacked into behaving like it, but it gets quite verbose.

There are action and icon inputs for attaching to dropdowns, labels, and buttons. I dont believe vertically attached is something I will support.


feat: add theorem natDegree_modByMonic_lt

Adds the natDegree analogue of this theorem
Co-authored-by: Pratyush Mishra<EMAIL_ADDRESS>

Thanks!
bors merg

bors merge


Rigidbody3D jittering instead of settling on the ground, under certain conditions

Godot version
Godot_v4.0-beta1_win64
System information
windows 10, Vulkan API 1.2.0 - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 1050 Ti
Issue description
https://user-images.githubusercontent.com/6585372/191086126-a8dc46dd-d0c0-4e51-9c94-99e38b3d2aa6.mp4
A camera follows a rigidbody3D with a cube as collider.
The collider scale is (5,5,5)
On ready, launch the cube, using cube.apply_central_impulse(Vector3(1, 0, -1) * 10)
The cube then gets launched towards the floor (staticbody with a cube as collider).
As soon as it reachs the floor, the cube then proceeds to jitter without stopping.
It doesn't happen if:

the initial impulse if smaller (lower the scalar to 8 or less)
the scale of the rigidbody's collider is lowered
the inertia property is altered to something other than Vertor3.ZERO

Steps to reproduce

3d scene

camera (make this follow and look at cube, for your sake)
rigidbody3d (cube.apply_central_impulse(Vector3(1, 0, -1) * 10))

collider (scale to 5,5,5)

cube

staticbody3d (floor)

collider

cube

Minimal reproduction project
test.zip

Interesting note about the inertia. For starters we could look at what goes differently in servers/physics_3d/godot_body_3d.cpp when that property is set.

didn't look at the code, but found a comment in it mentioning collider scale and inertia:
link

duplicate of https://github.com/godotengine/godot/issues/61827 ?


OCPBUGS-38794: handle version skewed NodePools that do not have rhel9 binaries

/hold
for testing

/lgtm

/test e2e-aws

/close
in favor of https://github.com/openshift/hypershift/pull/4656


Update data for Y dwarfs with JWST spectra.

Lots of attention is being drawn to the Y dwarfs with JWST spectra. Let's make sure the database has as much data as possible on those sources.

This is captured in other specific issues.


[resolves #760] Generate camel subsystem modules

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/532/


Added new files

Fixes #1716

@geekyd @mariobehling the manual build is working -
Please review - https://agbilotia1998.github.io/open-event-webapp/

@agbilotia1998 Can you check why gh-pages script is not executing? for testing use personal github token & enable travis on your personal dev branch.


Show Total Votes for Delegate

Fix for #11 and #1

Can you take a screenshot how it will look? I can't reproduce here..

Here's how the Approval line now looks:

I just noticed that it always displays in Ark, should I make it display in the selected currency?

Thanks! Current token (ARK or DARK), take a look: https://github.com/ArkEcosystem/ark-explorer/blob/master/src/app/components/header/header.component.ts#L7

So it should use initCurrency.name instead of currencyName and currencyValue? I think it would make more sense to use the currency name and value that the user has selected because I think every other value changes with the selected currency, such as Forged, Fee, Block Reward, etc.

Hmm it makes sense, I think it will seem a bit weird to see the total votes in USD

but yeah, to follow the standard you can use the current currency

Sounds good :+1:

nice! +5 👍🏻


🐞 - Chip has non-interactive text label

Reproduction url
https://taiga-ui.dev/components/chip#interactive
Description

Open https://taiga-ui.dev/components/chip#interactive
Try to click on the text at the center of the chip

https://github.com/user-attachments/assets/7c828ba1-4fee-416b-bc9a-fa7b2914851f
Taiga UI version
4.11.0
Browser Used
Chrome
OS Used
macOS

I suppose the demo is broken, it shouldn't be tui-chip, it should be label tuiChip. Input is purposefully sank with z-index, in case it has an appearance with non-transparent background.


Code auto formatting

Please in VScode activate the options:

and

I will enable it... thanks for advice.


Rounded corner in the PDF template and adding margin for titles

Description

Added left margin for titles (e.g., in Notifications or My Assignments).
Added bottom-left rounding only for the last time block to ensure symmetrical rounding both at the top and bottom.

Checklist:

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

Hey @rhahao, thank you, I will do it

:tada: This PR is included in version 3.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[BUG] Remote Session Hanging when tab is closed, not returning execution to script.

System information

20.04.1 Ubuntu:
pip:
FiftyOne v0.16.2, Voxel51, Inc.:
Python 3.8.12:

Commands to reproduce
As thoroughly as possible, please provide the Python and/or shell commands used
to encounter the issue. Application steps can be described in the next section.
1. load a dataset
2. setup a remote session
3. wait on that session
4. on a remote machine view the session in a browser and then close that tab

Describe the problem
The issue I am seeing is when I use the session.wait() command. I am running the session on a remote server and viewing it on my local machine, which is macOS. When I close the tab of the session in my browser I expect the flow to return to the python script but nothing happens.
Code to reproduce issue
import fiftyone as fo
dataset = fo.Dataset.from_dir(
dataset_type=fo.types.COCODetectionDataset,
data_path=data_path,
labels_path=labels_path,
include_id=True,
tags="",
label_field=""
)
session = fo.launch_app(dataset, remote=True, port=5151)
session.wait()
Other info / logs
I can provide more on request.
What areas of FiftyOne does this bug affect?

[X] App: FiftyOne application issue
[ ] Core: Core fiftyone Python library issue
[X] Server: Fiftyone server issue

Willingness to contribute
The FiftyOne Community encourages bug fix contributions. Would you or another
member of your organization be willing to contribute a fix for this bug to the
FiftyOne codebase?

[ ] Yes. I can contribute a fix for this bug independently.
[ ] Yes. I would be willing to contribute a fix for this bug with guidance
from the FiftyOne community.
[X] No. I cannot contribute a bug fix at this time.

https://github.com/voxel51/fiftyone/issues/1897 being tracked here


Set up rubocop cops that duplicate codeclimate checks

To increase chances of detecting something it'll object to beforehand

...[sigh] Apparently this for After We Clean Up Those Metrics

Strange that it sometimes runs these but it's catching more when we enable them manually :thinking:

Afaict it's running it's own versions, and so when we turn these on it duplicates the warnings rather than doing something sane.


De-duplicate data source MIME types

Some toolkits like GTK offer twice the same MIME type.

Thanks!


Changes to make it build on dockerhub.

Thanks! Would have a appreciated a brief a description to learn what the real issue is (inferred it from your commit message). This shows once again that Docker Hub isn't (yet) the great solution they claim it be:

I wasn't notified that the last 3 builds failed, https://hub.docker.com/r/marcelstoer/nodemcu-build/builds/
I was never notified that the they changed the rules which made the builds fail in the first place
(the mediocre commenting feature is still not fixed)

Sorry for not being too verbose about it. I did the fix on my fork of your repo on GutHub, and then as an afterthought thought you might want to have it too.
As to why and what it fixes, I don't know, there doesn't seem to be much documentation on DockerHub. I just noticed that the version with sudo and -y was failing, googled around to see what others did, tried that, and it worked.
Thanks for your work anyway. The idea of providing a docker image as a cross platform build environment is great. I've borrowed it to do the same for another hobbyist project that I am involved in.


Reset passwords stay valid until the password is reset

Or until the validity period has expired. This reverts an issue caused by #1008 where users who follow the password reset link from an email multiple times, or those who have anti spam filters which check links, are unable to reset their passwords.

I am not sure about this, but it makes sense, can you add a test checking the reset password token is invalid after the password is changed?

Is this okay to merge in yet?

My bad, I tested with master and everything works as expected. Not sure where removing the password reset token happens :/


Backport 2.1: Disable SHA-1 in TLS by default

SHA-1 deprecation: remove it from the default allowed hash
algorithms for certificate verification and TLS 1.2 handshake
signatures. It can be turned back on at compile time with
MBEDTLS_TLS_DEFAULT_ALLOW_SHA1 or explicitly with ssl_conf functions.
Removed RIPEMD-160 from the default hash algorithms for
certificate verification.

Backport of #904

Will need to be superseded by a new PR based on #913 .


[HybridApp] - Reintroduce log out button on HybridApp but log out to OldApp log out screen

Problem
In https://github.com/Expensify/App/issues/43336 we completely removed the Sign out button from New Expensify while on HybridApp, which was a good first version, but it's a weird user experience that users have noticed.
Solution
Reintroduce the Sign out button, but instead of taking you to the NewDot sign in screen, we should take the user to the OldDot sign in screen.
cc @staszekscp @WoLewicki
This is a polish item, so let's save it until we are finished with all other implementation issues.

Since I'm currently waiting for some Crashlytics news, I'm going to take care of this issue :)

Nice, thanks @staszekscp. Assigning you!


Add support for diagrams-rasterific backend

As cairo is notoriously difficult to install on osx and rasterific is a native haskell library that supports animated gifs.

Am I missing something or brew install cairo does the trick?

I think he meant the haskell bindings for cairo, and not cairo itself.

Ah, I see. What about stack install cairo? :)

heh, I think there are some compilation issues that one can have with OSX - I haven't faced them, but I have with other libraries (eg. ghc-mod)
Plus it's a good idea to support as many backends as possible anyway - variety is the spice of life :)

Done! I've made the code backend-agnostic


Add symfony|php matrix testing to actions

need to run tests on

Symfony 5.4 with php 7.4
Symfony 6.0 with php 8
Symfony 6.1 with ph 8.1

done


Branch 1.5

Can one of the admins verify this patch?

Would you mind closing this PR?

Hey @xif10416s, do you mind closing this issue?

@xif10416s could you close this PR please?


Services

Linter check failed, with the following output:
The following problems have been found:
ERROR: /tmp/abs_b7jpn646uf/clone/recipe/meta.yaml:1: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_b7jpn646uf/clone/recipe/meta.yaml:15: avoid_noarch: noarch: python packages should be avoided
ERROR: /tmp/abs_b7jpn646uf/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_b7jpn646uf/clone/recipe/meta.yaml:35: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, closest match: BSD-3-Clause
WARNING: /tmp/abs_b7jpn646uf/clone/recipe/meta.yaml:38: missing_license_url: The recipe is missing a license_url
ERROR: /tmp/abs_b7jpn646uf/clone/recipe/meta.yaml:38: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_b7jpn646uf/clone/recipe/meta.yaml:38: missing_doc_source_url: The recipe is missing a doc_source_url
ERROR: /tmp/abs_b7jpn646uf/clone/recipe/meta.yaml:38: missing_doc_url: The recipe is missing a doc_url
Errors were found


Fix potential crash on failed move

My mistake from #81657. In case a move fails in _try_move_item() no key for it will exist in p_renames, but the UID change function would try to access it anyway, which would cause a crash if the file has a UID.

Thanks!


Single selection listbox can display buggy selection box

When holding down the left mouse button inside the bounds of a single selection listbox and then dragging the cursor past the right border of the widget, a selection box appears inside the widget that continues to be drawn once the mouse button is released. I'm using Nana 1.5 hotfix branch.
Edit: It's not necessary to press down the mouse button inside the listbox, you can do it outside the bounds of the widget, then drag the cursor inside, and then drag it past the right border. The selection box disappears if you drag the cursor past the left border.
A multiple selection listbox also has abnormal behavior when you press down the left mouse button outside its bounds, then drag the cursor inside, and then drag it past the left or right borders.
This is not a particularly important issue, but I thought it was worth mentioning anyway.

Thank you for reporting this bug! I'll check it.


[13pt] Identify existing data/tools for estimating global channel geometry parameters

We need to explore solutions for estimating the channel geometry parameters that can help us estimate the bathymetric component that is currently not represented in the DEM.

[ ] Establish priorities for implementing bathy solution (observed vs. approximation)
[ ] Take inventory of existing channel geometry datasets and estimation techniques
[ ] Take inventory of existing bathy observation datasets
[ ] Chart out solution pathways: merge obs bathy, estimation - burn DEM, estimation - modify rating curve
[ ] Develop prototype solution(s)
[ ] Evaluate performance metrics

Closing this card for sprint continuation purposes. Please reference #282 for continued development and testing


Create n-th element from Stern’s Diatomic Series.py

Hey @7thHorcrux edit the readme as well and add the respective links there


New version: NaiveNASflux v1.1.2

Registering package: NaiveNASflux
Repository: https://github.com/DrChainsaw/NaiveNASflux.jl
Created by: @DrChainsaw
Version: v1.1.2
Commit: c4fc824f6bf7cf14067fbf003c4408ddfc250787
Reviewed by: @DrChainsaw
Reference: https://github.com/DrChainsaw/NaiveNASflux.jl/commit/c4fc824f6bf7cf14067fbf003c4408ddfc250787#commitcomment-38373858

I think I need to take a closer look at this one...


Ajax Link Authentication

This is a (multiple allowed):

[ ] bug
[ ] enhancement
[x] feature-discussion (RFC)

Is it possible to make ajax link work with server side http basic authentication? I use ruby on rails on server and devise gem to authenticate user.

Yes, it is possible. Need to send username and password parameters in Ajax request

Previously it was doable with $$.ajaxSetup
How to do so in v2?


Implement livestream image classification

Hey Duy, would you mind updating the readme to explain what this app does, how to run the model download script (and that it needs to happen), and maybe a little information about how the code is set up/key classes.
Thanks!

Alright should just need to run dart format. I also ran into an issue where the camera froze and crashed the app when running from visual studio to Android, but it works fine in Android Studio, so I'm guessing it's more of a setup issue on my side.

Can you also update the UI to add a small section at the bottom to show the classifications (normalized as 0.0 -> 1.0) that is more easily visible (like what we have in the Android apps). We can keep it sorted and displaying 3 items consistently rather than changing those options at runtime. Thanks Duy!

Can you check for memory leak?
I think it is because if inference.run method on tflite.
I've run classification with your code, but I used my own mobileNetV2 with float16.
When I run 6 minute, memory goes up to about 630MB.

When I run 9 minute, memory goes up to about 700MB.

When I run 43 minute, memory goes up to about 2.09GB.

Memory goes up and never go down, even I change it to gallery mode.
I ran that with iPad pro 11 m1 model.

Memory issue on iOS being tracked in #110 and outside of the control of the sample app.

After your comment, I've found out it is outside of the control of the sample app. It happened only iOS.

LGTM. Thanks Duy!


Protect from operator errors by adding hash of the configuration into the initial prepare.

Here:
https://github.com/tigerbeetledb/tigerbeetle/blob/52ec5b823b9e41591b64739d28c9f102002161fe/src/vsr.zig#L705
it makes sense to set .parent to the hash of the initial configuration, to make sure that all members of the cluster use the same config. It's a bit unclear at the moment when we learn the initial config -- on format, or on the first start. So makes sense to maybe postpone this a bit.
#598 implemented this as a proof of concept.

Relatedly, we might also want to store hash(config) in the superblock so that the replica can guarantee that it is restarted with the same config. Though that is trickier since presumably we will want to add new values to the config over time/versions.


Nondeterministic Release Builds

Description
Using Xcode 16 and Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>) I'm seeing some variance in the compiled .o object files with optimized/release builds. It seems to happen consistently in the standard library index(_:offsetBy:limitedBy:) function. Using otool -tV on 2 object files I see this diff for the binary I'm building with swift build -c release for the project: https://github.com/jpsim/Yams
@@ -9419,7 +9419,7 @@ _$sSlsE5index_8offsetBy07limitedC05IndexQzSgAE_SiAEtF4Yams4NodeO7MappingV_Tgq5Tf
<PHONE_NUMBER>008ff8       sub     x8, x8, x9
<PHONE_NUMBER>008ffc       movi.2d v0, #0000000000000000
<PHONE_NUMBER>009000       movi.2d v1, #0000000000000000
-0000000000009004       mov.d   v1[0], x0
+0000000000009004       mov.d   v1[1], x0
<PHONE_NUMBER>009008       mov     w9, #0x1
<PHONE_NUMBER>00900c       dup.2d  v2, x9
<PHONE_NUMBER>009010       mov     x9, x8
@@ -9427,8 +9427,8 @@ _$sSlsE5index_8offsetBy07limitedC05IndexQzSgAE_SiAEtF4Yams4NodeO7MappingV_Tgq5Tf
<PHONE_NUMBER>009018       add.2d  v1, v1, v2
<PHONE_NUMBER>00901c       subs    x9, x9, #0x4
<PHONE_NUMBER>009020       b.ne    0x9014
-0000000000009024       zip1.2d v2, v1, v0
-0000000000009028       zip2.2d v0, v1, v0
+0000000000009024       zip2.2d v2, v1, v0
+0000000000009028       zip1.2d v0, v1, v0
<PHONE_NUMBER>00902c       add.2d  v0, v0, v2
<PHONE_NUMBER>009030       addp.2d d0, v0
<PHONE_NUMBER>009034       fmov    x3, d0

I see a similar diff when compiling SwiftSyntax for release on the same disassembled index(_:offsetBy:limitedBy:) function.
@@ -4315528,7 +4315528,7 @@ _$sSlsE5index_8offsetBy07limitedC05IndexQzSgAE_SiAEtF11SwiftSyntax6TriviaV_Tgq5T
<PHONE_NUMBER>5b3104       sub     x8, x8, x9
<PHONE_NUMBER>5b3108       movi.2d v0, #0000000000000000
<PHONE_NUMBER>5b310c       movi.2d v1, #0000000000000000
-00000001005b3110       mov.d   v1[1], x0
+00000001005b3110       mov.d   v1[0], x0
<PHONE_NUMBER>5b3114       mov     w9, #0x1
<PHONE_NUMBER>5b3118       dup.2d  v2, x9
<PHONE_NUMBER>5b311c       mov     x9, x8
@@ -4315536,8 +4315536,8 @@ _$sSlsE5index_8offsetBy07limitedC05IndexQzSgAE_SiAEtF11SwiftSyntax6TriviaV_Tgq5T
<PHONE_NUMBER>5b3124       add.2d  v1, v1, v2
<PHONE_NUMBER>5b3128       subs    x9, x9, #0x4
<PHONE_NUMBER>5b312c       b.ne    0x1005b3120
-00000001005b3130       zip2.2d v2, v1, v0
-00000001005b3134       zip1.2d v0, v1, v0
+00000001005b3130       zip1.2d v2, v1, v0
+00000001005b3134       zip2.2d v0, v1, v0
<PHONE_NUMBER>5b3138       add.2d  v0, v0, v2
<PHONE_NUMBER>5b313c       addp.2d d0, v0
<PHONE_NUMBER>5b3140       fmov    x3, d0

Reproduction
I'm still trying to come up with a smaller repro project, but here are the steps I ran locally to reproduce using the Yams project (https://github.com/jpsim/Yams).
As the issue is nondeterministic, it can take a couple rebuilds as commented below to repro.
swift package clean
swift build -c release
mkdir compare
cp $(swift build -c release --show-bin-path)/Yams.build/Node.Mapping.swift.o compare/

# Loop on the steps below until a diff happens
swift package clean
swift build -c release
diff  $(swift build -c release --show-bin-path)/Yams.build/Node.Mapping.swift.o compare/Node.Mapping.swift.o

Expected behavior
Built object files are deterministic and reproducible in repeated builds with the same inputs.
Environment
swift-driver version: 1.115 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx14.0
Additional information
No response

here is a reduction based on the YAMS project (i'm guessing it could be further reduced). if you repeatedly run something like
swiftc -O <file> -o <output> && md5 <output>

eventually the checksum changes b/c the vector instructions' ordering are flipped. empirically it seems like it takes b/w 10-20 tries before this happens. certainly seems like there is some nondeterminism with whatever produces the zip1 & zip2 vector instructions.
public enum Node: Hashable {
    case mapping(Mapping)
}

// MARK: - Mapping

extension Node {
    public struct Mapping {
        private var pairs: [(Node, Node)]
    }
}

extension Node.Mapping: Comparable {
    public static func < (lhs: Node.Mapping, rhs: Node.Mapping) -> Bool {
        fatalError()
    }
}

extension Node.Mapping: MutableCollection {
    public typealias Element = (key: Node, value: Node)

    // MARK: Sequence

    public func makeIterator() -> Array<Element>.Iterator {
        fatalError()
    }

    // MARK: Collection

    public typealias Index = Array<Element>.Index

    public var startIndex: Index {
        fatalError()
    }

    public var endIndex: Index {
        fatalError()
    }

    public func index(after index: Index) -> Index {
        return pairs.index(after: index)
    }

    public subscript(index: Index) -> Element {
        get { fatalError() }
        set { fatalError() }
    }
}

extension Node.Mapping: Equatable {
    public static func == (lhs: Node.Mapping, rhs: Node.Mapping) -> Bool {
        fatalError()
    }
}

extension Node.Mapping: Hashable {
    public func hash(into hasher: inout Hasher) {
        fatalError()
    }
}

Thanks for the reduced example file @jamieQ !
I was able to bisect the issue a bit and it seems to be caused by the SIL optimization step. Using -Xllvm -sil-opt-pass-count=<n> I found the issue repros at pass 23811 and using -Xllvm -sil-print-pass-name I can see the following run in the logs which aligns with the reported function from the original post.
Start function passes at stage: HighLevel,Function+EarlyLoopOpt
...
 Run #23811, stage HighLevel,Function+EarlyLoopOpt, pass 24: EarlyPerfInliner (early-inline), Function: $sSlsE5index_8offsetBy07limitedC05IndexQzSgAE_SiAEtF4main4NodeO7MappingV_Tgq5

Can take a deeper look at the stage later, but wanted to drop some findings.

i think this may not exactly be a swift issue, but rather an llvm one. i've yet to see the non-determinism manifest in the output of any swift compiler stage before -emit-assembly (not that i've done a particularly exhaustive search to eliminate this possibility, but it reproduces pretty reliably as soon as assembly is produced). using the llvm options to debug the various llvm passes, my current theory is that this behavior is somehow involved with the 'complex deinterleaving' pass. i ran the following repeatedly until both the 'correct' (where zip1 precedes zip2) and 'flipped' (zip2 precedes zip1) output were produced:
swiftc -O \
  -emit-assembly \
  -Xllvm -print-after-all \
  -Xllvm -filter-print-funcs='$sSlsE5index_8offsetBy07limitedC05IndexQzSgAE_SiAEtF4main4NodeO7MappingV_Tgq5Tf4nnnd_n' \
  <file> -o <output> 2&> llvm-print-all.log && md5 <output>

diffing the outputs of the llvm logs produced something like this (this is where the outputs first differ):
--- llvm-lowering.correct.log
+++ llvm-lowering.flipped.log
<snip>
 *** IR Dump After Complex Deinterleaving Pass (complex-deinterleaving) ***
 define linkonce_odr hidden swiftcc { i64, i8 } @"$sSlsE5index_8offsetBy07limitedC05IndexQzSgAE_SiAEtF4main4NodeO7MappingV_Tgq5Tf4nnnd_n"(i64 %0, i64 %1, i64 %2) local_unnamed_addr #2 {
   %4 = icmp slt i64 %1, 0
   br i1 %4, label %67, label %5, !prof !17

 5:                                                ; preds = %3
   %6 = icmp eq i64 %1, 0
   br i1 %6, label %61, label %7

 7:                                                ; preds = %5
   %8 = sub i64 %2, %0
   %9 = sub i64<PHONE_NUMBER>854775807, %0
   %10 = add i64 %1, -1
   %11 = tail call i64 @llvm.umin.i64(i64 %8, i64 %10)
   %12 = tail call i64 @llvm.umin.i64(i64 %11, i64 %9)
   %13 = add i64 %12, 1
   %14 = icmp ult i64 %13, 5
   br i1 %14, label %35, label %15

 15:                                               ; preds = %7
   %16 = and i64 %13, 3
   %17 = icmp eq i64 %16, 0
   %18 = select i1 %17, i64 4, i64 %16
   %19 = sub i64 %13, %18
   %20 = insertelement <2 x i64> <i64 poison, i64 0>, i64 %0, i64 0
-  %21 = call <4 x i64> @llvm.experimental.vector.interleave2.v4i64(<2 x i64> %20, <2 x i64> zeroinitializer)
+  %21 = call <4 x i64> @llvm.experimental.vector.interleave2.v4i64(<2 x i64> zeroinitializer, <2 x i64> %20)
   br label %22

 22:                                               ; preds = %22, %15
   %23 = phi i64 [ %27, %22 ], [ %19, %15 ]
   %24 = phi <4 x i64> [ %21, %15 ], [ %26, %22 ]
   %25 = call <4 x i64> @llvm.experimental.vector.interleave2.v4i64(<2 x i64> <i64 1, i64 1>, <2 x i64> <i64 1, i64 1>)
   %26 = add <4 x i64> %24, %25
   %27 = add i64 %23, -4
   %28 = icmp eq i64 %27, 0
   br i1 %28, label %29, label %22, !llvm.loop !19

 29:                                               ; preds = %22
   %30 = call { <2 x i64>, <2 x i64> } @llvm.experimental.vector.deinterleave2.v4i64(<4 x i64> %26)
   %31 = extractvalue { <2 x i64>, <2 x i64> } %30, 0
   %32 = extractvalue { <2 x i64>, <2 x i64> } %30, 1
-  %33 = add <2 x i64> %32, %31
+  %33 = add <2 x i64> %31, %32
   %34 = tail call i64 @llvm.vector.reduce.add.v2i64(<2 x i64> %33)
   br label %35
<snip>

which closely aligns with the original diff from the object file output as well as the assembly output. furthermore, if you compile the code and explicitly disable that pass (compile with -Xllvm --enable-complex-deinterleaving=0), the behavior seems to go away (no zip instructions are even produced).
so perhaps hunting for something non-deterministic within the logic for that pass is a good next place to investigate. additionally, disabling that pass explicitly for the affected code may serve as a workaround, though i'm unsure if there are any downsides associated with doing that.

had a chance to try and verify this theory on some of the development toolchains and it appears that in toolchains built after the LLVM rebranch which includes the suspected fix from LLVM main, the issue is resolved. repeatedly building & checking the md5 of the assembly output on the 10-27 snapshot (before the LLVM bump) consistently reproduces the issue, but using the 10-30 snapshot (after the LLVM bump) does not, even after several hundred compilations.


Authentication

Hi!
Great example you have :+1:
Any idea how to implement authentication?

I'm sure there are a great many ways to do this, but here's one take on it, using Firebase's email/password authentication:

Create some new container components for top level views like sign up and sign in, and add React Native's Navigator component to switch between the container components
Add new auth async actions for sign in and sign up, which would use Firebase's authWithPassword and createUser, and dispatch actions based on starting and then success/failure (e.g. AUTH, AUTH_SUCCESS and AUTH_FAILURE for sign in).
Add and auth reducer to set appropriate state for all the actions dispatched, for instance isSigningIn to show a progress indicator when rendering the login view.
Have the container components do checks in componentWillReceiveProps for transitions between being authenticated and not and triggering the changes through the Navigator (which could be passed down as a prop to the container components) to switch to the appropriate container.

That's pretty rough and leaves a lot of details to the imagination, but is it helpful?

Thank @bruz !
Unfortunately (for me) - I was looking for doing the oAuth authentication and since it's based on window.open method - it won't work on native. Need to use custom G+/FB signin and pass the token to firebase.
Thanks again, though!


xmltv.xml missing document header. <originally issue #109>

This issue went left by the wayside and went stale.  I guess it isn't common knowledge that you can use Cabernet with Kodi.  Actually, I'm a very recent user myself.  Anyway I guess that's why there wasn't much interest in this issue.
Header information is missing in the xmltv.xml that Cabernet is sending back as response.  Kodi/PVR IPTV Simple doesn't like that some of the schema/dtd info is missing at the top of the file to designate it as xml.  It considers the schema malformed without that header.
<?xml version="1.0" encoding="UTF-8"?><!DOCTYPE tv SYSTEM "xmltv.dtd">
If you paste this schema info at the top of the returned xmltv.xml file then EPG data is then loaded into Kodi.
https://github.com/cabernetwork/cabernet/blob/76501a79db6ef6d2c93ab97233a24bbf8f167649/lib/clients/epg2xml.py#L351
This is where the setup of the file begins and where you could put the data.  Of course there are multiple places you could append the data.  I kinda looked at the code, but haven't got a complete grasp of it.  I noticed some stripping out of the element tags <tv>/</tv> and possibly the xml header.  So I'm not completely sure of what or why you are trying to accomplish that in that section currently.  It's just another section of the code I looked at to possibly append the header information(before the data get's written out).
The simplest way looks to convert the current xml tree into a UTF-8 string and append a UTF-8 encoded header to the beginning of it before it is written out.  The reason I say this is because ElementTree doesn't appear to have built-in functions to manipulate that part.  So either you have to bring in another library to do it or just convert both parts to UTF-8 and combine them.

For anyone who might be having same problem, here is quick and dirty fix.  Yes, I'm sure someone is going to say "that isn't the proper way to fix it, you should use an XML library".  Not only is it easier to fix this way, but the code will execute faster because you're not flip-flopping back and forth between libraries just to ultimately get the same outcome.
Anyway here are instructions for anyone not afraid to mess with code a little:
Under /lib/clients/epg2xml.py starting at line 151 insert this.  Do NOT delete what is at line 151, just insert this as a new line of code.
self.webserver.wfile.write(('<?xml version="1.0" encoding="UTF-8"?><!DOCTYPE tv SYSTEM "xmltv.dtd">\n').encode())
In the same file at line 167 insert this.  Same rule applies, DO NOT delete what is there.  Just add this line of code.
self.webserver.wfile.write(('<?xml version="1.0" encoding="UTF-8"?><!DOCTYPE tv SYSTEM "xmltv.dtd">').encode())
The way the code actually works is it currently uses the second line.  But there is an option in the code to turn on a flag to "prettify" the xml and it uses the first way instead.  I have included both just in case you want to use that flag to make the xml more readable on output.

For anyone wanting a quick fix for the issue in Kodi, you can use this.  Yes, I'm sure there will be some that say, "that isn't the proper way to fix that, you should use an XML library".  Not only is this an easier fix, but the code will execute faster since you won't be flip-flopping back and forth between libraries just to get the same outcome.
In file /lib/clients/epg2xml.py "add" the following contents:
At line 151, insert this.  DO NOT delete what is already there.
self.webserver.wfile.write(('<?xml version="1.0" encoding="UTF-8"?><!DOCTYPE tv SYSTEM "xmltv.dtd">\n').encode())
At line 167, insert this code.  Same rule applies, DO NOT delete code already there insert this as a new line.
self.webserver.wfile.write(('<?xml version="1.0" encoding="UTF-8"?><!DOCTYPE tv SYSTEM "xmltv.dtd">').encode())
By default the server currently uses the second line of code.  There is an option to "beautify" the output of the xml though.  If that flag is set then the first line of code is used.


请提出你的问题
Hey. Is there any chance you have this dataset cached locally and can send it to me?
https://huggingface.co/datasets/ChristophSchuhmann/improved_aesthetics_6plus
I'm going to train miniSDXL (like lambdalabs/miniSD-diffusers, but for SDXL) and need the exact same dataset which was used here...

Sorry, we don't provide the dataset, you can download the dataset through huggingface or some other available downloads


Removed Unnecessary Navbar From Contributor Page

Description
Hey @thestarsahil
I have removed the unnecessary navbar from the contributor page.
Fixes #868
Screenshots

Checklist

[X] Tests have been added or updated to cover the changes
[X] Documentation has been updated to reflect the changes
[X] Code follows the established coding style guidelines
[X] All tests are passing

@RadhikaMalpani1702 In our live server navbar removed from contribution page

You can work on css here , Alignment not proper

@thestarsahil
I have fixed the alignments of the contribute page and I have also made the page completely responsive
https://github.com/Counselllor/Counsellor-Web/assets/163530398/d7a90ff6-1108-4d5b-b4f9-ce85f48c591b

Not Responsive , the context sequeezed
Work on that !