启动的时候出现问题

11-27 13:17:25 [SUCCESS] nonebot | Succeeded to import "nonebot_plugin_gspanel"
11-27 13:17:25 [WARNING] uvicorn | The --reload flag should not be used in production on Windows.
WARNING:uvicorn.error:The --reload flag should not be used in production on Windows.
11-27 13:17:25 [INFO] uvicorn | Started server process [14244]
INFO:uvicorn.error:Started server process [14244]
11-27 13:17:25 [INFO] uvicorn | Waiting for application startup.
INFO:uvicorn.error:Waiting for application startup.
ERROR:asyncio:Task exception was never retrieved
future: <Task finished name='Task-3' coro=<Connection.run() done, defined at C:\Program Files\Python38\lib\site-packages\playwright_impl_connection.py:240> exception=NotImplementedError()>
Traceback (most recent call last):
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_connection.py", line 247, in run
await self._transport.connect()
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_transport.py", line 132, in connect
raise exc
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_transport.py", line 120, in connect
self._proc = await asyncio.create_subprocess_exec(
File "C:\Program Files\Python38\lib\asyncio\subprocess.py", line 236, in create_subprocess_exec
transport, protocol = await loop.subprocess_exec(
File "C:\Program Files\Python38\lib\asyncio\base_events.py", line 1630, in subprocess_exec
transport = await self._make_subprocess_transport(
File "C:\Program Files\Python38\lib\asyncio\base_events.py", line 491, in _make_subprocess_transport
raise NotImplementedError
NotImplementedError
11-27 13:17:25 [ERROR] uvicorn | Traceback (most recent call last):
File "C:\Program Files\Python38\lib\site-packages\starlette\routing.py", line 635, in lifespan
async with self.lifespan_context(app):
File "C:\Program Files\Python38\lib\site-packages\starlette\routing.py", line 530, in aenter
await self.router.startup()
File "C:\Program Files\Python38\lib\site-packages\starlette\routing.py", line 612, in startup
await handler()
File "C:\Program Files\Python38\lib\site-packages\nonebot_plugin_htmlrender_init.py", line 25, in init
browser = await get_browser(**kwargs)
File "C:\Program Files\Python38\lib\site-packages\nonebot_plugin_htmlrender\browser.py", line 42, in get_browser
return _browser or await init(**kwargs)
File "C:\Program Files\Python38\lib\site-packages\nonebot_plugin_htmlrender\browser.py", line 27, in init
_playwright = await async_playwright().start()
File "C:\Program Files\Python38\lib\site-packages\playwright\async_api_context_manager.py", line 51, in start
return await self.aenter()
File "C:\Program Files\Python38\lib\site-packages\playwright\async_api_context_manager.py", line 46, in aenter
playwright = AsyncPlaywright(next(iter(done)).result())
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_connection.py", line 247, in run
await self._transport.connect()
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_transport.py", line 132, in connect
raise exc
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_transport.py", line 120, in connect
self._proc = await asyncio.create_subprocess_exec(
File "C:\Program Files\Python38\lib\asyncio\subprocess.py", line 236, in create_subprocess_exec
transport, protocol = await loop.subprocess_exec(
File "C:\Program Files\Python38\lib\asyncio\base_events.py", line 1630, in subprocess_exec
transport = await self._make_subprocess_transport(
File "C:\Program Files\Python38\lib\asyncio\base_events.py", line 491, in _make_subprocess_transport
raise NotImplementedError
NotImplementedError
ERROR:uvicorn.error:Traceback (most recent call last):
File "C:\Program Files\Python38\lib\site-packages\starlette\routing.py", line 635, in lifespan
async with self.lifespan_context(app):
File "C:\Program Files\Python38\lib\site-packages\starlette\routing.py", line 530, in aenter
await self.router.startup()
File "C:\Program Files\Python38\lib\site-packages\starlette\routing.py", line 612, in startup
await handler()
File "C:\Program Files\Python38\lib\site-packages\nonebot_plugin_htmlrender_init.py", line 25, in init
browser = await get_browser(**kwargs)
File "C:\Program Files\Python38\lib\site-packages\nonebot_plugin_htmlrender\browser.py", line 42, in get_browser
return _browser or await init(**kwargs)
File "C:\Program Files\Python38\lib\site-packages\nonebot_plugin_htmlrender\browser.py", line 27, in init
_playwright = await async_playwright().start()
File "C:\Program Files\Python38\lib\site-packages\playwright\async_api_context_manager.py", line 51, in start
return await self.aenter()
File "C:\Program Files\Python38\lib\site-packages\playwright\async_api_context_manager.py", line 46, in aenter
playwright = AsyncPlaywright(next(iter(done)).result())
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_connection.py", line 247, in run
await self._transport.connect()
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_transport.py", line 132, in connect
raise exc
File "C:\Program Files\Python38\lib\site-packages\playwright_impl_transport.py", line 120, in connect
self._proc = await asyncio.create_subprocess_exec(
File "C:\Program Files\Python38\lib\asyncio\subprocess.py", line 236, in create_subprocess_exec
transport, protocol = await loop.subprocess_exec(
File "C:\Program Files\Python38\lib\asyncio\base_events.py", line 1630, in subprocess_exec
transport = await self._make_subprocess_transport(
File "C:\Program Files\Python38\lib\asyncio\base_events.py", line 491, in _make_subprocess_transport
raise NotImplementedError
NotImplementedError
11-27 13:17:25 [ERROR] uvicorn | Application startup failed. Exiting.
ERROR:uvicorn.error:Application startup failed. Exiting.

nonebot2 当前正在使用的 .env 文件中添加：
FASTAPI_RELOAD=false


Streaming API

Which version are you running? The lastest version is on Github. Pip is for major releases.
import pandas_ta as ta
print(ta.version)

0.4.14b0
Do you have TA Lib also installed in your environment?
$ pip list

yes
Upgrade.
$ pip install -U git+https://github.com/twopirllc/pandas-ta

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I find it frustrating to recalculate a techincal indicator for the entire data set for every new data point when operating in a real-time setting.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Currently if you are using this library for streaming data, real time updates require you to append the data to your dataframe and calculate the technical indicator for the entire history. It would be nice to have a streaming API that allows you to give just the latest piece of data and get the updated value for the technical indicator.
For example, let's say you have 100,000 rows of data and you are calculating the 200 EMA. For backtesting, you can just calculate the 200 EMA for every row and move forward with your back test. With the streaming API feature, if you had the 100,001st data point, you would need to append it to your historical data and then calculate the 200 EMA for every data point, which is slow and redundant.
Similar to this: https://github.com/mrjbq7/ta-lib#streaming-api
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
I've considered only keeping X number of historic data points so recalculating the newest technical indicator value is fast but this is a pain to implement.
Additional context
Add any other context or screenshots about the feature request here.
I'm a software engineer and would be happy to help build this with others.
Thanks for using Pandas TA!

Thanks for the reply @twopirllc, I had missed that discussion in #87 thanks for pointing that out. Additionally I'd never heard of talipp, which sounds like an excellent solutions. Thanks for the information!

@michaelto20
No worries! Hope it helps! 🍀
KJ


Add support for TI's LP_MSPM0G3507

This contribution adds support for TI's MSPM0 devices starting with support for board LP_MSPM0G3507.
This PR currently includes support for:

GPIO
UART
~I2C~
~ADC~
~CAN~

Hi, thank you for this contribution! However, please leave out I2C, ADC, and CAN support from the initial PR in order to reduce the number of code lines to review.
Once the basic support has been reviewed and accepted, you can follow-up with individual PRs for adding more drivers.
Keep in mind that these areas may need to be reviewed by different maintainers. Mixing it all together in one big PR makes it much harder to get it reviewed and approved.
Please see https://docs.zephyrproject.org/latest/contribute/contributor_expectations.html for further tips.

Sounds good. I resubmitted the PR after removing I2C, ADC and CAN, and fixed some copyright issues

@msp-ti for the missing hal files that are causing the CI failure, you can follow the process https://docs.zephyrproject.org/latest/develop/modules.html#process-for-submitting-changes-to-existing-modules to include the in-flight PR in this one, basically you have to change the ref in west.yml for hal_ti to pull/51/head (the PR number of https://github.com/zephyrproject-rtos/hal_ti/pull/51). You can do that as its own commit, once this one pulls and passes we can then merge the HAL PR and you can update this one with the final sha.

Just found out about this new line of Micros from digikey, very exciting ! Looking forward to testing them !

Fixed the problems with style-checks. Still other work to be done

@msp-ti are you planning to come back to this PR?

@msp-ti are you planning to come back to this PR?

Yes the plan is to pick this back up soon. We were working to add openOCD and had some other items come up that required work, but the hope is to get this PR back merged. I apologise for the delay

@msp-ti Any updates?

Sorry for the delay, I am working to port this PR to the newer updates with Zephyr and add some additional information

Reopening


ledgerutil.md

Location: docs/locale/en_US/source/

review:  https://github.com/lfapac-open-source-evangelist/fabric-docs-i18n/pull/43/commits/54a8fb78810d6d712261fbc7584f14c8f27c457a


Press ESC in Titlescreen will Close Game

IDK is it useful or not But It works :)

I forgot something wait pls

also pretty useless as you can press alt+f4 anyways

also pretty useless as you can press alt+f4 anyways

agreed

sorry

also pretty useless as you can press alt+f4 anyways
i mean just an alternative way thank you

oh I just want it as an alternative way btw thank you

itg noway


CommandSpec injected into Mixins doesn't have a CommandLine

When a CommandSpec is injected into a @Mixin class, the CommandSpec.commandLine() method returns null.
This prevents custom setter methods from throwing ParameterExceptions, since the exception constructor requires a not-null CommandLine object.
    static class ExampleMixin {
        @Spec CommandSpec spec;

        @Option(names = "--trex")
        void setTRexFences(final String value) {
            throw new CommandLine.ParameterException(spec.commandLine(), "Didn't say the magic word.");
        }
    }

    static class ExampleCommand implements Callable<Void> {
        @Spec CommandSpec spec;

        @Option(names = "--raptor")
        void setRaptorFences(final String value) {
            throw new CommandLine.ParameterException(spec.commandLine(), "Didn't say the magic word.");
        }

        @Mixin
        ExampleMixin mixin;

        @Override
        public Void call() throws Exception {
            return null;
        }
    }

    public static void main(String[] args) {
        CommandLine.call(new ExampleCommand(), args);
    }

When run with --raptor off (an option in the root Callable):
Didn't say the magic word.

When run with --trex off (an option in the mixin):
Could not invoke void com.zegelin.SpecRepro$ExampleMixin.setTRexFences(java.lang.String) with off

The InvocationTargetException cause is a NPE, from the assert @ CommandLine.java:8852, since the spec commandline is null.
Version: 3.5.0

Fixed in master.
I will do a release for this now.

The fix is included in piococli-3.5.1. Enjoy!


Cart System Implemented

Related Issue
Closes #248
Proposed Changes

User can add items into the cart by clicking on add to cart button on product detail page
User can increase/decrease the quantity of items in the cart
User can delete an item from the cart
shimmer effect added while data is loading

Known Issues

Price in payment window hardcoded

Checklist

[ ] Tested
[x] No Conflicts
[x] Change In Code
[ ] Change In Documentation
[ ] Follows linting rules

Screenshots

Hi @Atharv-Joshi, can you tell me how dis you choose get over other state management tools?

Hi @Atharv-Joshi, can you tell me how dis you choose get over other state management tools?

actually i was just trying out getx here and forgot to delete that code in pr
I chose getx as its lightweight
I've added a commit which removes all getx related code and follows type annotations where mentioned

I'll also add the price in payment window in this PR itself.

Sure @Atharv-Joshi, just tag me when done, because GitHub notifications got skipped from my inbox earlier.

sure!


Labels

This pull request is based on https://github.com/twosigma/Cook/pull/127.
This one includes a fix (correcting an attribute name in Datomic) and also introduces a bit of unit test coverage to make sure the labels are really populated.

Can you also validate  https://github.com/twosigma/Cook/pull/126
This will allow us to set labels in the CoaseGrainedScheduler.

OK - I have actually never used JobClient yet so depending on what's involved it may take me some time to get that set up.  Will get to it this afternoon.

This all looks good. Did you have a chance to test it against a mesos cluster to ensure that it is set correctly on mesos's side?

@wyegelwel Ah, I didn't even realize that this was intended to make labels attached to the actual Mesos tasks.  I just tried it and no, the tasks that come from the jobs don't have any labels.
Just to be clear:   We want the labels attached to the Cook jobs to also be attached to all Mesos tasks that are instances of the job?

@mforsyth Attaching the labels to all of the generated Mesos tasks from a Cook job is the intention.

OK, the problem was with clj-mesos.  This PR is now waiting on
https://github.com/twosigma/clj-mesos/pull/1
... which adds Labels support.  Once that PR is merged, in this branch I'll need to change Cook scheduler's project.clj to use Two Sigma's fork of clj-mesos in order to get access to the fix.

@mforsyth I merged twosigma/clj-mesos#1, you can go ahead and update project.clj and run any last tests you want to.  Let me know when you're ready to have this merged.

OK, I added the new dep for clj-mesos  that supports Labels.  I just tested it locally end-to-end one more time, and it works (labels populated on Mesos tasks).


Create + use actinia processes dict with only one returns

While GRASS GIS modules can have multiple outputs, the openEO API allows only one. This PR duplicates the ACTINIA_PROCESS_DESCRIPTION_DICT and manipulates the copy to split up modules with multiple outputs. Instead of an array of outputs, the module then has only one output object.
Example:
r.slope.aspect will not appear as a single module but as many for each of its outputs:
{
	"id": "r.slope.aspect_slope",
	"parameters": [...],
	"returns": {
		"description": "Name for output slope raster map. ",
		"name": "slope",
		"optional": true,
		"schema": {
			"subtype": "cell",
			"type": "string"
		}
	}
},
{
	"id": "r.slope.aspect_aspect",
	"parameters": [...],
	"returns": {
		"description": "Name for output aspect raster map. ",
		"name": "aspect",
		"optional": true,
		"schema": {
			"subtype": "cell",
			"type": "string"
		}
	}
},
...

Hint:
To see this change in action, the actinia module filters need to be deployed in the used actinia instance. The current configured instance https://actinia-dev.mundialis.de is not yet up-to-date. These changes won't break things though.
For each touched file the lint errors were fixed.

So openEO API still allows only a single output for a process?

As per API specification returns is an object with description and schema and not an array of objects - yes, only a single output for predefined processes is allowed.


Shortcuts

Crtl + H  This is a common shortcut for displaying hidden files.
Ctrl + Q Close Window.
Backspace For going back to the previous folder in the history.

Implemented. :)

Ctrl + R to reload.
Escape to close dialogs ?

Ctrl + R : Its like you said, the library which is organizing files and etc. is monitor based. This means there is no reason for reloading :)
Rest: agreed :)

IN!! :)


API version configuration

Description
Could we have a configuration option / environmental variable to set the API version? Right now the master is at 2.2 while I have some docker running with API 2.1 and in order to make them work together I must recompile the master changing the CONSTANT string in https://github.com/keylime/keylime/blob/bafa8801da1d1b7db0c5ce63b6bb93df557149a6/keylime/api_version.py#L9
Expected behavior vs. actual behavior
Expect to have a configuration parameters and/or an environmental variable to change the version of the API (and thus the URL invoked between components). It might be necessary to check that the version set in the config is actually supported by the SW version

Which agent version/commit and what version/commit of the agent and server?
The server should still accept older API versions (currently accepted this is 2.0, 2.1 and 2.2). If this is not the case this is a bug.

Which agent version/commit and what version/commit of the agent and server?
The server should still accept older API versions (currently accepted this is 2.0, 2.1 and 2.2). If this is not the case this is a bug.

The new server components support older agents, not the other way around. So, the new agent is incompatible with old servers. The agent only supports the latest version.
I think in the medium to long term, the agent should support multiple API versions and have a way to negotiate the version to use.


node 20

just opening here as a reminder of deprecation of node 16
https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/
thank you for the great github action!

fixed by #177

Great one!
can you please make a new release also?
Thank you


Adds example project using jib-maven-plugin.

This project builds a simple HelloWorld application that uses one resource file and one dependency.

@https://github.com/mattmoor This may be handy as an example to play around with.


Editable packages

Is there a flow where things included with CPM are editable?
I have 2 projects that I maintain: ctoot and moduru. ctoot depends on moduru. moduru's CMakeLists.txt has a static lib target that ctoot consumes, as well as a moduru-tests executable target. When I create a Visual Studio solution for ctoot, that solution should include all the targets from moduru, so I can work on the moduru library itself and run moduru's unit tests while doing so.
When I link moduru to ctoot via target_link_libraries, the moduru target appears in my Visual Studio solution's project explorer. So far so good.
However, moduru-tests does not appear, which I suppose makes sense, considering a CMake dependency graph can easily contain many transient executable targets for unit tests, examples, etc. Is there a way to expose all or user-selected targets of a package that is included via CPM?
See https://github.com/izzyreal/ctoot/blob/no-conan/CMakeLists.txt and https://github.com/izzyreal/moduru/blob/no-conan/CMakeLists.txt

I don't use Visual Studio, but I think the issue is that the target is not added to the default CMake build target all. This can be resolved by explicitly setting the EXCLUDE_FROM_ALL flag to NO in the explicit API. In the single-argument API, the argument defaults to YES.
You can try changing CPMAddPackage("gh:izzyreal/moduru#no-conan") to the following to have all targets added to the all target:
CPMAddPackage(
  NAME moduru
  GITHUB_REPOSITORY izzyreal/moduru
  GIT_TAG no-conan
  EXCLUDE_FROM_ALL NO # actually optional here, as it defaults to false on the explicit syntax
)

Thanks, I may try that, though currently I'm experimenting with FetchContent.


Website invalid

yo bro the website auction destroyer or whatever is no longer valid pls fix

Already working it.

ok thanks


[security] Slightly improve CSP header

A starting point for a CSP rule with permissive rules that we can narrow down overtime:  https://stackoverflow.com/questions/35978863/allow-all-content-security-policy
Context: https://groups.google.com/a/mui.com/g/team-store/c/6tkWMOQ2ho0.

The problem seems to be that the iframe lives on about:blank which likely has some elevated policies being a chromium internal page. I didn't investigate deeper, just changed the test to go to https://example.com first and now it seems to pass.

Oh, I didn't think about this. You nailed it 👍, thanks


auto-fix the usage of fixture.api with native fetch

Fixes #74
Fixes #76
progress tracking and high level guide here

a few notes:

ts-init folder is needed because typescript requires physical file presented, services folder is copied here for easier testing of service typing stuff without creating circular dependency with openapi tools.
@checkdigit/agent config is created to separate agent/fetch conversion; fixes won't be applied unless services specifically add this eslint config; no extra cli is required.
no-full-response and require-resolve-full-response rules are added into all config as prerequisite so that bad typings are not in the way of auto conversion


Feature/api v2 and siri extensions support

Add support for v2 API plus misc other changes
Query TXC files directly using v2 API
Add support for Siri version attribute
Add support for Extensions element in MonitoredVehicleJourney

v2 is currently in private mode at the minute and won't be included here.


suggestion to add toast message

we can add a toast message on click of favorite button on book to notify user the book is added to his favorites

@Arshad-ashuu Sure. Do star 🌟 the repo to start.

Fixed


Fix fuzzy edges on the flat template

Before:

After:

This also increases the file size by five bytes.  I don't see an easy way to
reduce that though.  (Yet.  Still experimenting with SVGs.)

Great work!
Five bytes isn't an issue, especially after SVGO compression and
gzipping. The PNG images that were widely deployed a year ago were an
order of magnitude bigger.

I feel like we can have a more descriptive name. rounded perhaps?

Can you remove the space before the slash? I know, nitpick.

Ah, oops.  Missed that :)  The five bytes is after SVGO :P

Mac

Before
After

There's everything to love about this contribution, as it removes all of the clunky things we did. Stellar work, thanks a lot!

  Hopefully I'll have a couple more PRs for you over the next few weeks :)


Creates new asset type data template

Need an asset type for data template. Data files will then need to satisfy that data template. The data template can also be downloaded and populated to provide actual data.
This probably involves use of Rightfield and also liaison with protocols and revised assay types

#6


Add sysstat package (e.g. for iostat commands)

What this PR does / why we need it:
Adds the sysstat package to the list of apt packages.
This contains useful utilities like iostat.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

`sysstat` package is added to the ops-toolbelt containing useful utilities like `iostat`

Ah, thanks for checking the size, I didn't check. So I understand that we generally don't want to "explode" the size, makes sense :)


{{Foo}} doesn't link to the Foo namespace

It looks like {{Foo}} won't link to namespace dfns. They're in the same namespace as interfaces, so this should work.

.euiv/dxvcfgr6666

Jane gie aimlessly s:$/22z’i”/8oodjh


0292_Nim_Game

IMPORTANT: Please do not create a Pull Request without an issue.
All Submissions:

[x] I have read the CONTRIBUTING document.
[x] My code is written in Python3 and is ending with .py
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have checked that my submission does pass the test on LeetCode.com
[x] Does your filename follow the naming Conventions?
[x] Have you linked your PR to an Issue?

Closing issues
Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

@sarthak55k No issue linked

#132

close #132

closes #132


Ignoring the RHS value type in HLIL

Version and Platform (required):

Binary Ninja Version:4.2.6204-dev Personal (2277d8cd)

Bug Description:
HLIL just ignores the type of expression in variable declaration. In attached database you can see the function at 0x1800B8390, there is some boiler-plate repetitive code patterns, and at the beginning of the function it works fine (there is gNetworkMessagesVersion001_2 has been propagated with correct type from gNetworkMessagesVersion001):

But if scroll down a bit, HLIL just starting to ignore the type of gNetworkMessagesVersion001 variable, setting local var to void* instead of proper type as earlier in function:

Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

Open attached database
Go to 0x1800B8390 func
0x1800b847a - example of correct definition
0x1800b8cca - from there starts the wrong ones

Expected Behavior:
Not sure, mb it the expected already and to fix this i should tweak some of the limits settings, but function not looks to big for me, to reach the default analysis limits. So i think the expected behaviour should be to propagate correct type in the whole function.
Binary:
The archive is to big for uploading it on github, i've sent it to binaryninja at vector35.com

Thanks for the report. Binary available internally asgiggle sapphire whisper.


Slider elements diagram in readme :: change TOOLTIP to ::tooltip

The TOOLTIP was changed from an "accepted child" (data-slot role) The current image reflects the old (data-slot) technique and needs to be updated to show ::tooltip

Have a VPN issue, can you please push this image for me.


Update standard library list for go 1.23

It'd be great to get a release after the standard library list is updated as well (so golangci-lint and such can better support Go 1.23).

@daixiang0 Please merge this PR 👍

@silas thanks for your contribution!

Please sign all commits then we can make it land.

@daixiang0 Commits should be signed now.

Also interested in an updated release 👍

The version variable (also gci --version) still points to the old version: https://github.com/daixiang0/gci/blob/dca258a399bf32ab20c73ab4edb726cfff7224ef/main.go#L9
I'm not sure what the best way to fix it would be. Releasing a new version?


support for parsing OSM XML files

this looks like a good library: https://crates.io/crates/xml-rs


dont replace err unless err is nil

b67bccbb introduced this regression. This fixes the return for err as to not be overwritten if MANIFEST_URL is not defined. sorry.
~/go/src/github.com/canaryio/canary/cmd/canaryd$ git branch | grep \*
* master
~/go/src/github.com/canaryio/canary/cmd/canaryd$ go build && ./canaryd
panic: runtime error: slice bounds out of range

vs
~/go/src/github.com/canaryio/canary/cmd/canaryd$ git checkout manifest_error_handling
Switched to branch 'manifest_error_handling'
~/go/src/github.com/canaryio/canary/cmd/canaryd$ go build && ./canaryd
2015/03/19 20:31:06 MANIFEST_URL not defined in ENV

Closing in favor of #34


Webhook for application readiness

HTTP request should be posted, whenever the server has been started and is ready for receiving requests. Event, that triggers webhook is starting of HTTP server, which is only possible after all preliminary commands were executed (e.g. optional management commands).
Webhook should not block the main event loop of the application, so it should be implemented as an async task on the event of HTTP server start. Post request can be in a form of JSON.

I'd say we need another webhook trigger in case of management command failures. It should include the reason and the trace of the exception plus some meta data.
I would suggest a list of new command options in order to specify webhook target URLs. The payload could be of a static message format at the moment which includes relevant data.

Thank you for your implementation @vvvityaaa.
As I looked into the current master I found following misconception: the LivenessWebhook is only fired once upon startup. I think the webhook should be moved to the probe handler and trigger upon state changes: none -> healthy and healthy -> unhealthy and eventually from unhealthy -> healthy.
In addition to that, please always add the hostname and a timestamp to each message.

Thank you for your comments @Schille. I think, the idea of state variables for liveness and readiness-webhooks is very good.


OOB invitation: only works with auto_respond_presentation_request configured?

I am using aca-py 0.7.4-rc1 and try to get the OOB invitations with a presentation request attached running. I have noticed that only auto-responding to presentation requests leads to some useful result. However, I don't want that. I want to present the request to the user and let the user decide whether to answer it or not. It actually works, as aca-py does post data to ".../topic/present_proof_v2_0/" where I can initiate the presentation request/response cycle. However, before that, there is an exception thrown which leads to the answer of the out-of-band/receive-invitation to be http code 500:

2022-05-07 00:14:04,291 aries_cloudagent.core.dispatcher ERROR Handler error: invitation_receive
Traceback (most recent call last):
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/routes.py", line 221, in invitation_receive
    result = await oob_mgr.receive_invitation(
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/manager.py", line 565, in receive_invitation
    await self._process_pres_request_v2(
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/manager.py", line 832, in _process_pres_request_v2
    raise OutOfBandManagerError(
aries_cloudagent.protocols.out_of_band.v1_0.manager.OutOfBandManagerError: Configuration set auto_present false: cannot respond automatically to presentation requests
2022-05-07 00:14:04,314 aries_cloudagent.admin.server ERROR Handler error with exception: Configuration set auto_present false: cannot respond automatically to presentation requests
Traceback (most recent call last):
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/admin/server.py", line 159, in ready_middleware
    return await handler(request)
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/admin/server.py", line 196, in debug_middleware
    return await handler(request)
  File "/home/user/.local/lib/python3.10/site-packages/aiohttp_apispec/middlewares.py", line 45, in validation_middleware
    return await handler(request)
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/admin/server.py", line 340, in check_multitenant_authorization
    return await handler(request)
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/admin/server.py", line 384, in setup_context
    return await task
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/routes.py", line 221, in invitation_receive
    result = await oob_mgr.receive_invitation(
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/manager.py", line 565, in receive_invitation
    await self._process_pres_request_v2(
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/manager.py", line 832, in _process_pres_request_v2
    raise OutOfBandManagerError(
aries_cloudagent.protocols.out_of_band.v1_0.manager.OutOfBandManagerError: Configuration set auto_present false: cannot respond automatically to presentation requests
2022-05-07 00:14:04,355 aiohttp.server ERROR Error handling request
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 435, in _handle_request
    resp = await request_handler(request)
  File "/home/user/.local/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/home/user/.local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/admin/server.py", line 159, in ready_middleware
    return await handler(request)
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/admin/server.py", line 196, in debug_middleware
    return await handler(request)
  File "/home/user/.local/lib/python3.10/site-packages/aiohttp_apispec/middlewares.py", line 45, in validation_middleware
    return await handler(request)
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/admin/server.py", line 340, in check_multitenant_authorization
    return await handler(request)
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/admin/server.py", line 384, in setup_context
    return await task
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/routes.py", line 221, in invitation_receive
    result = await oob_mgr.receive_invitation(
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/manager.py", line 565, in receive_invitation
    await self._process_pres_request_v2(
  File "/home/user/projects/hyperledger/aries-cloudagent-python/aries_cloudagent/protocols/out_of_band/v1_0/manager.py", line 832, in _process_pres_request_v2
    raise OutOfBandManagerError(
aries_cloudagent.protocols.out_of_band.v1_0.manager.OutOfBandManagerError: Configuration set auto_present false: cannot respond automatically to presentation requests

And, sure enough, that's what the code says in manager.py:

        pres_ex_record = V20PresExRecord(
            connection_id=conn_rec.connection_id,
            thread_id=pres_request_msg["@id"],
            initiator=V20PresExRecord.INITIATOR_EXTERNAL,
            role=V20PresExRecord.ROLE_PROVER,
            pres_request=pres_request_msg,
            auto_present=self.profile.context.settings.get(
                "debug.auto_respond_presentation_request"                                    <------------
            ),
            trace=trace,
        )

        pres_ex_record = await pres_mgr.receive_pres_request(pres_ex_record)
        if pres_ex_record.auto_present:                                                      <------------
            (pres_ex_record, pres_msg) = await pres_mgr.create_pres(
                pres_ex_record=pres_ex_record,
                comment=(
                    f"auto-presented for proof requests"
                    f", pres_ex_record: {pres_ex_record.pres_ex_id}"
                ),
            )
            responder = self.profile.inject_or(BaseResponder)
            if responder:
                await responder.send(
                    message=pres_msg,
                    target_list=await self.fetch_connection_targets(
                        connection=conn_rec
                    ),
                )
        else:
            raise OutOfBandManagerError(
                (
                    "Configuration set auto_present false: cannot "
                    "respond automatically to presentation requests"
                )
            )

From what I understand in this code snippet, if the flag "--auto-respond-presentation-request" is not set, the OOB invitation with attached presentation requests will not work as intended. Is that right?
This exception leads to the server answer:

500 Internal Server Error
500 Internal Server Error
Server got itself in trouble

...instead of a connection record.
Any ideas? I don't want any auto-response, not in this case, not for any other presentation requests. I want to let the user decide what to do with such requests (as in "self-sovereign" ;-) ).
Thanks for your kind help! Any idea how to do this is appreciated. Should I just ignore this error message?

This has been addressed I think in https://github.com/hyperledger/aries-cloudagent-python/pull/1710

Closing as addressed in #1710 . Please let me know if this should be re-opened. It would be great to get a formal confirmation that issue is addressed.

I have upgraded to 0.7.4-rc2 and can confirm that this error does not come up any longer. Thank you very much, everyone!


Multiple quality improvements

This pull request is focused on resolving occurrences of Sonar rules squid:S1068 - Unused private fields should be removed
squid:S2293 - The diamond operator ("<>") should be used
squid:ClassVariableVisibilityCheck - Class variable fields should not have public accessibility
You can find more information about the issues here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1068
https://dev.eclipse.org/sonar/coding_rules#q=squid:S2293
https://dev.eclipse.org/sonar/coding_rules#q=squid:ClassVariableVisibilityCheck
Please let me know if you have any questions.
M-Ezzat

I am not sure whether you do not agree with these changes or not. Please close this PR if not.

No its just as you can see by looking at my history, i havent been active on github due to some life reasons


on_progres doesn't work fully

My progress bar is displayed with a delay, and when it appears, it is either with 100% or another large value.

from pytubefix import YouTube
from pytubefix.cli import on_progress
url = "https://www.youtube.com/watch?v=25N1pdzvp4c"
yt = YouTube(url, on_progress_callback = on_progress)
ys = yt.streams.get_highest_resolution()
ys.download()

When you start the program, a progress bar immediately appears with 0%, gradually filling up to 100%.

Desktop (please complete the following information):

OS: [Windows 10 Pro]
Python Version [3.9.6]
Pytubefix Version [15.0.0]

Is it because I am using a proxy server when downloading?

Insufficient information to debug.
I verified that the on_progress_callback works correctly on Windows 10 x64 today with PyTubeFix version 6.16.3.
I have been developing constantly for several months. I have never noticed a problem on Ubuntu or Windows. I suspect you have an issue with your code, but you did not include code to review that implements on_progress().


feat: like a post

going to close this one for now as we will need a websocket to do this properly


VTOL Front transition controls ramp up

Ramp up roll and ramp down throttle when front transition completes.
This avoids aggressive behavior after front transition.

Do you have a log showing what you're trying to address?
Throttle
Depending on the underlying issue you're seeing I'd like to consider initializing the FW position controller throttle (TECS) properly. This would also be a nice improvement when changing between stabilize/manual/acro -> altitude/position/auto.
Roll
Is the problem the attitude setpoint in the FW position controller? If that's the problem I'd be worried about other phases of flight where jumping from one extreme (eg commanding full left bank) to the other (full right bank) is perfectly legal. Involving L1 during transition might also help depending on the specifics.

Hi Daniel,
For throttle it is mostly the massive instant decrease that results in braking action at a moment that generally needs both bank and pitch change.
The roll is similar to the takeoff bank limitation, under normal flight conditions you can limit the rate but that limitation is not enough during a takeoff or front transition condition. You need to smooth in to it.

This change has the intended effect. It managed to keep the vehicle stable through the front transition on a highly (over) loaded vehicle: https://review.px4.io/plot_app?log=c7a94509-6825-4cd4-98b5-16eaea79d311
Transition faded smoothly to normal flight.

@sanderux From the results of your tests I think it's clear what we need to do:

add a slew rate to the output of the fw l1 controller, similar to how TECS slew rate limits the pitch angle. Not only would it solve your problem but also would it avoid the sudden step in roll angle which can be observed when flying a normal mission at the time instant when a new waypoint gets active

after a front transition we need to initialise TECS such that the throttle demand is continuous and does not drop suddenly like it does now.

Doing it this way has the advantage that all platforms can benefit from these changes and not just quadplane. All vtol platforms as well as planes suffer from the same problem.

@sanderux From the results of your tests I think it's clear what we need to do:

add a slew rate to the output of the fw l1 controller, similar to how TECS slew rate limits the pitch angle. Not only would it solve your problem but also would it avoid the sudden step in roll angle which can be observed when flying a normal mission at the time instant when a new waypoint gets active

after a front transition we need to initialise TECS such that the throttle demand is continuous and does not drop suddenly like it does now.

Doing it this way has the advantage that all platforms can benefit from these changes and not just quadplane. All vtol platforms as well as planes suffer from the same problem.


Testing mellanox ci version2

Thank you for your PR,
You can test hw-offload configuration by commenting one of the following:

/test-hw-offload.
/test-all.

These commands can be run by anyone.
Best regards.


Cannot deploy / debug after VSCode Update

{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "al",
            "request": "launch",
            "name": "Microsoft cloud sandbox",
            "startupObjectId": 22,
            "startupObjectType": "Page",
            "server": "https://myserver-xyz.westeurope.cloudapp.azure.com",
            "serverInstance": "NAV",
            "tenant": "",
            "authentication": "UserPassword",
            "schemaUpdateMode": "Recreate",
            "internalConsoleOptions": "openOnSessionStart"
        }
    ]
}

AL Language:

VS Code:
Version 1.24.0
Commit 6a6e02cef0f2122ee1469765b704faf5d0e0d859
Date 2018-06-06T17:35:40.560Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

Where can I search for more Information? Is there a detailed logfile, what's going wrong?

@megel Maybe VS Live Share is enabled? That results in the same error

@tfenster No VS Live Share wasn't it - I found the Problem :)
In my current VSCode Workshop - I have used a Folder Name with whitespaces for my example:
d:\workshop\01 start here\...
The full Story:
…Today, one of my colleagues got the problem in his VS Code Workshop VM. I tried everything but could not fix it - so he got a new VM. After lunch, the VS Code on my machine sugessted me to install an update. After the installation I got the same problem. So I removed Extension, each line of code and tried several docker versions - nothing worked. Last try with a new AL GO Project could be deployed to the NST. Then I realized the difference :) the WHITESPACES in the Project Folder Name was the fault.

@megel I had the same issue, and your post helped me solve it. Thanks!

@bwdeleeuw seem to me this is a VSCode bug because it was present direct after install the VSCode Update.

Thanks for reporting, we will investigate it

Hi @megel. Thanks once again for reporting this issue. With the latest AL extension from the marketplace I'm unable to reproduce this issue. A scaffolded project using AL:Go! with a project folder with spaces in it deploys as expected to the cloud sandbox. To be crystal clear - I have a project located at: C:\Users\<user>\Documents\AL\AL whitespace test, and in that folder the app.json resides.
Can you verify that the latest extension works properly for you as well? If not, please provide some additional repro steps and I will continue investigating.


Debug minifier (Lines) converts "lorem"/"ipsum"/ to expression with syntax error: "lorem"/"+""ipsum"/

Hello!
 It assumes, that first " is opening quote, but it can also be closing.
Code before (caught in some open source library for JavaScript):
var foo = "lorem"/*"ipsum"*/ ;

Code after:
/* 41 */ ...
/* 42 */ var foo = "lorem"/"+"*"ipsum"*/ ;
/* 43 */

As the result we have string "lorem" and unexpected regexp literal /"+"*"ipsum"*/ , which cause a syntax error .
Why that line ever exists? Why it does care about "/*" only and not about '/*'?

#538


HACS integration

Is there any expected date for it to be installable via HACS?

Just add this project’s gitlab link to custom repository and it will work like any other item you see in HACS.


directus dataprovider

would be cool to have directus as a data/authprovider.
https://github.com/directus/examples
https://github.com/directus/examples/tree/main/nextjs

great initiative @avping! thanks for this one.
I'll take a look at directus and the documentation you shared.

+1 for this, I want to use both supabase and directus together, they work well together would be great if this can be put on top of them.


Website updates training

Update the Training page(s)

Just pushed fixes to your comments! @clemiller

LGTM!


Mix is always fake and doesnt include more than one song

I used the mix from #31 https://www.youtube.com/watch?v=zhl-Cs1-sG4&list=RDzhl-Cs1-sG4 to test the new mix feature as well as multiple from my own YouTube Music but i always get a fake playlist from the fallback as a result. This is the data for the link:

@Shiny1708 this can happen for most of the mixes because youtube mixes are generated per user and it is not available to youtube-sr. It might work if you supply your youtube cookie:
getPlaylist(url, {
  requestOptions: { headers: { cookie: '...' } }
})

when youtube-sr cant find mix information, it returns a fake playlist with one track in it.

I did some digging and from my understanding visting that a private mix with a link just generates a mix based on the first video. Would it be possible to use the new generated mix? From what i can tell thats what lavaplayer does to get mixes

Lavaplayer seems to be using singleColumn for the result while youtube-sr is using twoColumn

I will see what I can do

I think detecting mixes only with the regex /(RDC|O)LAK5uy_[a-zA-Z0-9-_]{33}/; doesnt detect all mixes. Mixes from yt music dont necessarily have the RD part at the start. Lavaplayer uses this for mix recognition: "https://music.youtube.com/watch?v=f9awN8BCUe8&list=PLXftWo8BlN2lx5iHhkp-3Yg5aRRbe6Qme". So mixes are catergorized by the fact that they have the watch part followed by the list part and that they use music,youtube instead of youtube.com.

The youtube music mixes are not consistent across clients so the bot would have a different queue than the user but i think the ability to access the youtube music autoqueue in itself is extremly useful since its very good in finding related songs

The you link gave has playlist page too https://music.youtube.com/playlist?list=PLXftWo8BlN2lx5iHhkp-3Yg5aRRbe6Qme

Mixes usually dont have playlist page

The youtube music mixes are not consistent across clients

yeah I think so. I will try to wrap around innertube next time using this library https://github.com/LuanRT/YouTube.js

I just compared the mix and playlist result. From what i can tell the links to the videos are extracted from the thumbnail image links when loading playlists correct? The problem I'm facing when entering a mix url is that the  tag where the queue would be located if the procedure is the same for mixes is just empty.

Why is this part not loaded for the yt music link?
The approach to get the link via thumbnail should work with mixes as well if we get that ytmusic-app tag populated

(I'm still new to webdev so I hope this makes some sense and im not completly missing the mark)

The you link gave has playlist page too https://music.youtube.com/playlist?list=PLXftWo8BlN2lx5iHhkp-3Yg5aRRbe6Qme

Mixes usually dont have playlist page

The youtube music mixes are not consistent across clients

yeah I think so. I will try to wrap around innertube next time using this library https://github.com/LuanRT/YouTube.js

You're right I didn't realize I started the mix from a playlist. Sorry

Lavaplayer seems to be using singleColumn for the result while youtube-sr is using twoColumn

Where is this code snippet from? I just looked at the code for mixes again and its also using twoColumn


add namespace field

Signed-off-by: Miguel Angel Medina Mondragon<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md

/assign @rasheedamir

Hi @kanolato, can you please explain the need of namespace. If you want to deploy Reloader in specific namespace, you can achieve that by updating the flag watchGlobally to false.

Hi @kanolato, can you please explain the need of namespace. If you want to deploy Reloader in specific namespace, you can achieve that by updating the flag watchGlobally to false.

Hi @faizanahmad055, im using helm template as follows:
helm template --name example --namespace ex\
  --output-dir ./build \
    reloader

The output is the following:
---
# Source: reloader/templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    chart: "reloader-1.0.1"
    heritage: "Tiller"
    app: reloader
    release: "example"
    group: com.stakater.platform
    provider: stakater

  name: reloader
spec:
  replicas: 1
  revisionHistoryLimit: 2
  selector:
    matchLabels:
      app: reloader
      release: "example"
  template:
    metadata:
      labels:
        app: reloader
        release: "example"
        chart: "reloader-1.0.1"
        heritage: "Tiller"
        group: com.stakater.platform
        provider: stakater

    spec:
      containers:
      - env:
        - name: KUBERNETES_NAMESPACE
          valueFrom:
            fieldRef:
              fieldPath: metadata.namespace
        image: "stakater/reloader:v0.0.25"
        imagePullPolicy: IfNotPresent
        name: reloader
      serviceAccountName: reloader

Thats why i put the namespace: {{ .Release.Namespace }}. Most charts have that for the ones not using tiller.
With namespace: {{ .Release.Namespace }} we get:
---
# Source: reloader/templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    chart: "reloader-1.0.1"
    heritage: "Tiller"
    app: reloader
    release: "example"
    group: com.stakater.platform
    provider: stakater

  name: reloader
  namespace: example
spec:
  replicas: 1
  revisionHistoryLimit: 2
  selector:
    matchLabels:
      app: reloader
      release: "example"
  template:
    metadata:
      labels:
        app: reloader
        release: "example"
        chart: "reloader-1.0.1"
        heritage: "Tiller"
        group: com.stakater.platform
        provider: stakater

    spec:
      containers:
      - env:
        - name: KUBERNETES_NAMESPACE
          valueFrom:
            fieldRef:
              fieldPath: metadata.namespace
        image: "stakater/reloader:v0.0.25"
        imagePullPolicy: IfNotPresent
        name: reloader
      serviceAccountName: reloader

/ok-to-test

/approve

/lgtm


[Algo] Insersion Sort [C]

Please assign this to me.
https://en.wikipedia.org/wiki/Insertion_sort

Hey, can I work on this?

@zeddmaxx  Sorry, I am already working on it and its near completion.


No able to install firestore via pods

all other pods are working fine but after adding firestore to pod list it give me this error
`Analyzing dependencies
 [!] Unable to satisfy the following requirements:

- `Firebase/Firestore` required by `Podfile`

Specs satisfying the `Firebase/Firestore` dependency were found, but they required a higher minimum deployment target.`

My app pod file
`platform :ios, '8.0'
 use_frameworks!

 target 'MyApp' do

 pod 'MBProgressHUD', '~> 0.9.2'
 pod 'Firebase/Core'
 pod 'Firebase/Firestore'`
end

target 'MyAppTests' do

end

 target 'MyAppUITests' do

end

Other version details
Swift - 3x
Xcode - 9
ios deployment target - iOS 9
cocoapod version - 1.3.1

if you run pod update does the issue go away? Your podfile looks fine at first glance.

@morganchen12  I have updated the pods which I was using previously and I guess some of the other pod is the issue which not allows firestore to install. because if I comment all the pod and just install
 pod 'Firebase/Firestore' then the installation works.

What's the minimum deployment target of your app?

every version details are mention above
ios deployment target - iOS 9

I'm unable to reproduce this issue with your Podfile. Here's the output when I run pod install.
morganchen-macbookpro:TestProj morganchen$ pod install
Analyzing dependencies
Downloading dependencies
Using Analytics (3.6.7)
Using BoringSSL (9.0)
Using Crashlytics (3.9.0)
Using Fabric (1.7.0)
Using Firebase (4.3.0)
Using FirebaseAnalytics (4.0.4)
Using FirebaseAuth (4.2.1)
Using FirebaseCore (4.0.8)
Using FirebaseDatabase (4.1.0)
Using FirebaseFirestore (0.8.0)
Using FirebaseInstanceID (2.0.4)
Using GTMSessionFetcher (1.1.12)
Using Google (2.0.4)
Using GoogleAnalytics (3.17.0)
Using GoogleAppUtilities (1.1.2)
Using GoogleAuthUtilities (2.0.2)
Using GoogleIDFASupport (3.14.0)
Using GoogleInterchangeUtilities (1.2.2)
Using GoogleNetworkingUtilities (1.2.2)
Using GoogleSignIn (3.0.0)
Using GoogleSymbolUtilities (1.1.2)
Using GoogleToolboxForMac (2.1.3)
Using GoogleUtilities (1.3.2)
Using IBAnimatable (5.0.0)
Using IQKeyboardManager (5.0.3)
Using Kingfisher (3.13.1)
Using MBProgressHUD (0.9.2)
Using Mixpanel-swift (2.2.1)
Using OneSignal (2.5.6)
Using Protobuf (3.4.0)
Using ReverseExtension (0.4.8)
Using SwiftyJSON (3.1.4)
Using Zip (0.8.0)
Using gRPC (1.6.5)
Using gRPC-Core (1.6.5)
Using gRPC-ProtoRPC (1.6.5)
Using gRPC-RxLibrary (1.6.5)
Using leveldb-library (1.18.3)
Using nanopb (0.3.8)

Try clearing out your CocoaPods cache (rm -rf ~/Library/Caches/CocoaPods) and see if running pod install again resolves the issue.

@morganchen12 thanks know after clearing the cache its works


generate_parameter_library: 0.3.3-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository generate_parameter_library to 0.3.3-1:

upstream repository: https://github.com/PickNikRobotics/generate_parameter_library.git
release repository: https://github.com/ros2-gbp/generate_parameter_library-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.3.1-1

generate_parameter_library

No changes

generate_parameter_library_example

No changes

generate_parameter_library_py
* Fix Parameter Descriptor Incorrectly Populating Range Constraints for size_lt and size_gt (#105 <https://github.com/PickNikRobotics/generate_parameter_library/issues/105>)
* Contributors: Chance Cardona

parameter_traits

No changes

Sorry for the noise on this one, this is a bug fix for a feature I pushed out in 0.3.2


fix: XSS vulnerability by changing typeahead gem

Fixes #2797
Describe the changes you have made in this PR -

Replaced bootstrap-typeahead-rails by twitter-typeahead-rails
Configure bloodhound as per new gem

Screenshots of the changes (If any) -

UI changes as per new gem

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 81.785% when pulling b6c953b493256eaafc298cb3a00212a030b788a7 on tintin9999:edu-typeahead into b5d49ca6f25ff4cb7493fe59e4ad06dbaa473bf3 on CircuitVerse:master.

@tachyons @gr455 Review whenever possible!

Them doesn't seems maintained, can we use the one from npm instead ?

Are you talking about this one? If not, which one are you talking about exactly? I can make a PR with that one instead of the current one :+1: @tachyons

Sure thing! I'll close this PR as this one will have unrequired commits.


RKE2 Windows node tries to pull linux image rancher/system-agent:v0.0.1-alpha37-suc, causes imagepullbackoff

Mirror of https://github.com/rancher/rancher/issues/34282

Should be ready to test with the latest v1.21.3-rc7+rke2r2

Will be validated and closed when testing this https://github.com/rancher/rancher/issues/34282


README.md issue

sklearn is not the proper name of the library, needs to be updated to scikit-learn.

Updated the README.md file to address the changes.


Paper Sources

Hi @MaybeShewill-CV
I used your network and train my own data on it. It worked pretty well. Now I want some more information about LaneNet. I found a paper from Ze Wang, Weiqiang Ren, and Qiang Qiu -> "LaneNet: Real-Time Lane Detection Networks for Autonomous Driving" is it the same LaneNet, or is it related to the LaneNet described in "Towards End-to-End Lane Detection: an Instance Segmentation Approach"?
Can you help me to understand it because I want to know how it works?  They are a lot of LaneNet papers but I cannot distinguish between the LaneNet used in your Implementation from the other LaneNet described in other papers.
I want to cite your work and I thank you in advance for any answer.

@LukasProgram The repo implements the paper "Towards End-to-End Lane Detection: an Instance Segmentation Approach". As for method details you may refer to the paper for details:). Btw could you please share some test result on your own dataset here if possible? :)

Hi @MaybeShewill-CV
I can share the results from tensorboard if you want. I also did the validation step.
I am trying now to run the model on a smartphone, so I need to run the python3 export_inference_graph.py
--input_type
--pipeline_config_path
--trained_checkpoint_prefix
--output_directory
I don't know what argument should I use for pipeline_config_path. Can you help me?

@LukasProgram Sorry I do not know what is it:)

@LukasProgram Sorry I do not know what it is:)

Ok, and what output nodes have the model or graph?
I try to run the script in
https://stackoverflow.com/questions/45864363/tensorflow-how-to-convert-meta-data-and-index-model-files-into-one-graph-pb

@LukasProgram You may find the node you need in the LaneNet model script:)

@MaybeShewill-CV i think the name of the input node is "input_tensor" and the output node is "lanenet_model" am I right ?

@LukasProgram No you may print all the variable in the compute graph and select what you need by yourself:)

@LukasProgram Hi ,
You mentioned your trying to run the code on a smartphone but the reported run-time on NVIDIA 1080 TI is 52 FPS which sounds not very "mobile" friendly. can you elaborate how you plan to use this network on a mobile platform ?

@mbrv
I am trying also to run the network on a mobile phone.
After I get the values from the output nodes I need to make the clustering in android. I don't know how the performance will be but let's find out.
Do you have experience with the integration of a network in Android? Do you think it will perform badly?


[SPARK-38791][SQL][3.3] Output parameter values of error classes in the SQL style

What changes were proposed in this pull request?
In the PR, I propose new trait QueryErrorsBase which is supposed to be used by Query.*Errors, and new method toSQLValue(). The method converts a parameter value of error classes to its SQL representation.
Why are the changes needed?
To improve user experience with Spark SQL. Users should see values in error messages in unified SQL style.
Does this PR introduce any user-facing change?
Yes.
How was this patch tested?
By running the modified test suites:
$ build/sbt "test:testOnly *QueryExecutionErrorsSuite"

Authored-by: Max Gekk<EMAIL_ADDRESS>Signed-off-by: Max Gekk<EMAIL_ADDRESS>(cherry picked from commit bc8c264851457d8ef59f5b332c79296651ec5d1e)
Signed-off-by: Max Gekk<EMAIL_ADDRESS>

All GAs passed https://github.com/MaxGekk/spark/actions/runs/2148687685.
Merging to 3.3. Thank you, @cloud-fan for review.

Hi, @MaxGekk . For the PRs on non-master branches, you should close them manually after merging. :)


Recurrent PersistenceExceptions thrown in factories due to Primary Key collisions

Hello, this is the issue I've been talking about in the CakePHP #support Slack recently. I'm going to do my best to type it up here and explain all my debugging methods I've attempted thus far, but I'm pretty much stumped.
Please let me know if any additional information is required.
The Problem
I get errors like these from 17 different tests:
CakephpFixtureFactories\Error\PersistenceException : Error in Factory App\Test\Factory\AccessTokenFactory.
 Message: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'yfwOHj3X2SGe4914XD721lrG90dGr0eBzWJRGPMH' for key 'PRIMARY'

The Setup
AccessTokenFactory looks like this:
    protected function setDefaultTemplate(): void
    {
        $this->setDefaultData(function (Generator $faker) {
            /* for clarity, let's call this the "default data array" */
            return [
                'oauth_token' => $faker->regexify('[a-zA-Z0-9]{40}'),        // <-- this is the primary key
                'client_id' => $faker->regexify('[a-zA-Z0-9]{32}'),
//                'user_id' => $faker->uuid,
                'expires' => $faker->unixTime,
            ];
        });
    }

Notes:

The FK column is commented out in the sample above. I've tried it both with and without, but have noticed that WITH sometimes the FKs get messed up for associated entities but WITHOUT sometimes I get user_id does not have a default value. I feel like that might be a separate issue, though.
At this exact moment, the only field that is colliding is the oauth_token field listed above. BUT, it has been other fields in the past, such as IDs and FKs. Once I switched to using combination fixtures+factories, the ID collisions went away (more on this in a minute)

An example test:
    public function testAuthenticateReusesExistingAccessTokenIfPresent()
    {
//        $client = ClientFactory::make()->with('Users.Roles')->persist();
        $client = $this->Clients->find()->contain(['Users'])->first();
        $creds = base64_encode(sprintf('%s:%s', $client->client_id, $client->client_secret));

        $time = FrozenTime::now()->addHour(3);
        $testToken = AccessTokenFactory::make([
            'oauth_token' => '4308be68821e7a952691de06395ea401ce368424',
            'client_id' => $client->client_id,
            'expires' => $time->format('U'),
        ])->withUsers($client->user)->persist();

       // ...

        $this->post('/users/authenticate');

       // ...
    }

Notes:

I have tried making entities for tests two ways: directly in the test with AccessTokenFactory::make() and also in combination with CakePHP fixtures (demonstrated below).
When only using direct factory calls, I was using the TruncateDirtyTables trait. At this exact moment I am using the TransactionStrategy instead.

Example fixture:
class AccessTokensFixture extends TestFixture
{
    public $connection = 'my_test_connection';            // Important: all my tables are located on a non-standard connection

    /**
     * Init method
     *
     * @return void
     */
    public function init(): void
    {
        AccessTokenFactory::make(50)->with('Users.Roles')->persist();
        AccessTokenFactory::make(['expires' => FrozenTime::now()->addDay()->format('U')], 50)->with('Users.Roles')->persist();

        parent::init();
    }
}

Notes:

I know this is not the way either Cake fixtures or the Fixture Factories are technically supposed to be used, but this is a last-ditch effort I was trying yesterday to see if it helped.
Once I switched all my tests over to the fixtures I switched to the TransactionsStrategy and commented out the TruncateDirtyTables trait because I can't have two fixture strategies in play at once.

Other Notes

When I run these tests individually, they pass. When I run the entire test suite, 12 fail with the error mentioned above, and the remaining 5 fail with errors that I believe are downstream of this error (related entity save failures)
The key collisions are not happening in my CI/CD pipeline, which is using GitLab Pipelines with an attached MySQL 5.7 service container. My local is using a MariaDB-10.6 container.

Expected Behavior
My entities are generated with unique primary keys every time, and all entities are dropped in between test runs (*caveat: I know some strategies clean up tables before or after, please take my meaning to be the correct one regardless of what I said)

TODO:

[ ] use the test suite light
[ ] use only factories created in a per tests: Test should be Arrange, Assert, Act.
[ ] Create a debug testsuite with all your tests
[ ] Narrow down the number of tests while still having the tests fail

What has changed
I have dwindled down the debug test suite to the ~three~ two tests that, when run together, cause EITHER an integrity constraint violation in the primary key column OR an entity save failure:
Both of these test classes have been modified to use the TruncateDirtyTables trait as well as generate/persist their entities with the fixture factories (no more transactions, no more static fixtures). I've also updated to the latest minor version of the fixture factories plugin.
Here is the default data for the factory that is having the problem:
protected function setDefaultTemplate(): void
{
    $this->setDefaultData(function (Generator $faker) {
        $faker->seed(md5(microtime()));    // note: now seeding faker with microtime
        $token = $faker->regexify('[a-zA-Z0-9]{40}');
        echo sprintf("    - Generated token: \"%s\"\n", $token);

        return [
            'oauth_token' => $token,
            'client_id' => $faker->regexify('[a-zA-Z0-9]{32}'),
              'user_id' => $faker->uuid,
            'expires' => $faker->unixTime,
        ];
    });
}

Current error(s)
Two different exceptions get thrown by the broken test suite, in different quantities per test run.
I used this command to run the broken test suite three times:
vendor/bin/phpunit --testdox --testsuite debug --repeat 10

Then applied some grep magic to the output to sort and count exceptions thrown, output was:
## InternalErrorException = my app, entity save failure; PersistenceException = this plugin, constraint violation
# 1st run
  34    │ Cake\Http\Exception\InternalErrorException
 106    │ CakephpFixtureFactories\Error\PersistenceException

# 2nd run
  30    │ Cake\Http\Exception\InternalErrorException
 110    │ CakephpFixtureFactories\Error\PersistenceException

# 3rd run
  26    │ Cake\Http\Exception\InternalErrorException
 114    │ CakephpFixtureFactories\Error\PersistenceException

# 4th run
  34    │ Cake\Http\Exception\InternalErrorException
 104    │ CakephpFixtureFactories\Error\PersistenceException
# (note: this run doesn't even add up to 140 total errors, like all the others)

Question/Comments
So, where I'm confused at the moment is why the exceptions would be happening in different quantities each for each run of the test suite when nothing at all is changing about the test suite in between the runs. It would make sense to me for the numbers to never change and be the same between runs.

Another lens into the problem (or at least, a related one) which shows that "when" the user saves can cause errors to occur in different tests over different runs:
Scenario: 4 different tests ~

1 - make token with user, role, and client - persist
2 - make token with user and role - persist
3 - make client with user and role - persist
4 - make client with user (no role) - persist

Over 10 repeats of the test suite saw errors thrown in tests 1 and 3 sometimes, 2 and 3 sometimes, 3 only sometimes, and sometimes not at all. With zero change to the tests code in between.
The exact error is an InternalErrorException generated when the application is unable to create a particular entity (NOT a factory entity, a real one) because it has a dependency on a User entity (factory-made) that is supposed to be persisted at that time but is not:
// these are validation errors for an application-generated entity, not from a factory
// in this case it depends on a factory-generated User entity, which is supposed to be persisted by now
{
    "user_id": {
        "_existsIn": "The specified user ID does not exist in the Users table"
    }
}

So it seems like the tests can sometimes get ahead of ->persist(), or at least that is my takeaway.

It would also appear that seeding the generator in this manner:
$faker->seed(md5(microtime()));
// or

is insufficient

On a recommendation from someone I created a new homebrew random string generator function and my factory now calls this method instead of $faker->regexify(...).
This appears to resolve the integrity constraint violations at first glance, but I need to finish repairing my test suite fully (from all the changes trying to diagnose this issue) to say for certain.
I will update this comment with a final conclusion when it becomes available, but it appears as if the problem may actually lie with the FakerPHP library not being able to generate enough randomness for regexify(), even when seeded.

Closing as it appears there is a strong possibility the defect is in the Faker generator (though what it could be I have been unable to tell, as the results of test runs are so random and irreproducible). The fix, in this case, was to homebrew a random string generator and use that instead of $faker->regexify(). Thanks, all.


Remove-AzureRmApiManagementUser does not work after updating AzureRM.ApiManagement from 4.3.1

We use Azure Automation Runbook to clean up Azure resources on certain actions from our product. The cleanup script recently stopped working with the message 'uid' does not match expected pattern '(^[\w]+$)|(^[\w][\w\-]+[\w]$)'. This happens when API management user is being deleted.
The command we use is
Remove-AzureRmApiManagementUser -Context $ApiMgmtContext -UserId $apiUser.UserId -DeleteSubscriptions
Upon investigation, I found that the module versions were recently updated in Automation Account in order to include AzureRM.KeyVault module and as part of that, the version of AzureRM.ApiManagement module was upgraded from 4.3.1 to 6.1.0. Looking at the diff of the source code, I found that new validations have been added that prevents us from passing Emails as UserId values.
I see this as a breaking change and haven't seen it documented. In our product, we create users in API management using the REST API documented here and it still says uid's length can be up to 256 chars and does allow EmailId to be used as uid.

@KK911 please open the issue in the Azure PS repo, this is a repo for tracking Azure .NET SDK issues and not Azure Powershell.
https://github.com/Azure/azure-powershell/issues/new


Aquaplanet with gray radiation

Requirements

45 km domain top

200km horizontal resolution

63 vertical levels

0M Microphysics Scheme

Equilibrium moisture

Prescribed Exchange Coefficients

Vertical Diffusion tendency

Gray Radiation

[ ] Merge completed simulation configuration into long-run buildkite pipeline.

200-590 day average:


Save grid user preferences also in customers, orders

Description
Please describe your pull request:

Why is it necessary?
User can customize columns of orders, customers
What does it improve?
User grid preferences in orders and customers will be also saved like article list
Does it have side effects?
Nope

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?

How to test?

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-17846.
Please use this issue to track the state of your pull request.

Hey @shyim, thanks for your contribution. We decided to decline your PR due to the fact we experienced a lot of trouble and unwanted behavior with the ExtJS StateManager. We had planned to enable it in every administration module we're having but sadly found out it has major faults in the ExtJS version we're using.


Change default of interpreter path setting from "python" to "default"

https://github.com/microsoft/vscode-python/pull/20457#discussion_r1061767208
This enables users to select "python" as an interpreter similar to how they can select "python3" or the like.

[ ] Change default of python.defaultInterpreterPath setting.
[ ] Change all the places in the extension that assumes default is "python". For eg. check these files

[ ] https://github.com/microsoft/vscode-python/blob/ad0286c9c44884c6d43a7648b14ca69bec9ae989/src/client/common/interpreterPathService.ts#L81-L97
[ ] https://github.com/microsoft/vscode-python/blob/29a72b4fca446eeb8629213f7128dd6cdb33e812/src/client/common/configSettings.ts#L584
[ ] https://github.com/microsoft/vscode-python/blob/bddcb20298629f30935312a26c85eb95af41df34/src/client/common/process/pythonExecutionFactory.ts#L54-L58
[ ] https://github.com/microsoft/vscode-python/blob/847055539d6578e4e99af4a2308febc66693da4c/src/client/common/constants.ts#L94
[ ] ... and more

[ ] Make sure any external API consumers relying on this fact are notified.

VSCode uses "auto": https://github.com/microsoft/vscode-python/pull/20457#discussion_r1061860745


tvOS platform followup

followup to tvOS platform support added in PR #91, with some TODO items:

[x] add a quick pointer to the documentation - done by @brodybits
[ ] test and document how to create a module with a working example on tvOS
(@brodybits could use some pointers on this)
[ ] test and document tvOS support on React Native 0.59 vs 0.60 (I would like to keep both since React Native 0.59 is still needed for Expo and the partially supported Windows platform)
[ ] test and document how to create a view module (using the --view option) with a working example on tvOS
[ ] gain and document a better understanding of motivation for supporting the tvOS platform
[ ] improve the documentation
[ ] consider a more general approach for out-of-tree platforms

/cc @dlowder-salesforce

Awesome! Please assign this to me :)

Thanks, assigned to you as a bug now

For the generated example, there is already a --example-react-native-version option that should be able to specify fork and version to use React Native from. In case this is not enough to support tvOS, I think we should try to fix the existing option if possible.

Creating a bare bones module for tvOS:
# use latest version
create-react-native-module tv-test --generate-example --example-name TvTest --example-react-native-version react-native@npm:react-native-tvos
# use a specific version number
create-react-native-module tv-test --generate-example --example-name TvTest --example-react-native-version react-native<EMAIL_ADDRESS># install example dependencies
cd react-native-tv-test/TvTest
yarn
# Cocoapods
cd ios
pod install
# Open the workspace
open TvTest.xcworkspace

At the top bar of the project window, select the TvTest-tvOS target and one of the available Apple TV simulators:

Then build and run, and you should see the example screen on the Apple TV simulator:

Creating a view module:
Same steps as above, but pass in the --view parameter.
create-react-native-module tv-view-test --view --generate-example --example-name TvViewTest --example-react-native-version react-native@npm:react-native-tvos

Resulting screen shot:

Support for tvOS in 0.59 for this tool will require some work:

There is no tvOS target in the generated Xcode project file for a module created with this tool
There seem to be compilation issues for iOS in react-native init - generated projects with<EMAIL_ADDRESS>(although tvOS compiles fine)

Since tvOS is not supported by expo, I think it's probably fine to only support 0.60.x and higher.

Thanks @dlowder-salesforce, I hope to try it out soon (no promises right now). One more question is if you can explain the motivation for supporting tvOS.
The one thing about supporting tvOS is that it becomes one more thing for us to test and avoid breaking anytime we make a significant change that affects iOS.
In comparison, I do hope to include support for macOS someday (tracked in #94), while I would love to drop Windows support as discussed in #43.


Asapy feature analysis

add feature analysis.
working:

box/violin
correlation
clustering

Codecov Report

Merging #39 into development will decrease coverage by 5.75%.
The diff coverage is 65.05%.

@@               Coverage Diff               @@
##           development      #39      +/-   ##
===============================================
- Coverage        91.88%   86.12%   -5.76%
===============================================
  Files               12       14       +2
  Lines              936     1204     +268
===============================================
+ Hits               860     1037     +177
- Misses              76      167      +91

Impacted Files
Coverage Δ

spysmac/spysmac_cli.py
95.23% <100%> (+0.64%)
:arrow_up:

spysmac/plot/confs_viz/viz_sampled_confs.py
94.27% <100%> (+0.02%)
:arrow_up:

spysmac/asapy/utils/util_funcs.py
20% <20%> (ø)

spysmac/analyzer.py
96.21% <50%> (-1.45%)
:arrow_down:

spysmac/asapy/feature_analysis.py
63.51% <63.51%> (ø)

spysmac/spyfacade.py
96.1% <88.46%> (-1.56%)
:arrow_down:

spysmac/plot/scatter.py
76.66% <0%> (+0.55%)
:arrow_up:

spysmac/plot/plotter.py
100% <0%> (+2.08%)
:arrow_up:

Continue to review full report at Codecov.

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


irmin-trace: drop usage of Stdlib.Stream

This package is deprecated in OCaml 4.14 and due to be removed in OCaml 5.0, so we should avoid using it here. This is currently causing our 4.14 CI to fail:
File "bench/irmin-pack/tree.ml", line 82, characters 14-25:
82 |         match Stream.next json with
                   ^^^^^^^^^^^
Error (alert deprecated): module Stdlib.Stream
Use the camlp-streams library instead.
File "bench/irmin-pack/tree.ml", line 83, characters 20-34:
83 |         | exception Stream.Failure ->
                         ^^^^^^^^^^^^^^
Error (alert deprecated): module Stdlib.Stream
Use the camlp-streams library instead.
        test alias test/irmin-containers/runtest

I don't see any use of Stream in that file 🤔
https://github.com/mirage/irmin/blob/main/bench/irmin-pack/tree.ml#L82

I was probably looking at a PR into 2.10 without realising it: https://github.com/mirage/irmin/blob/2.10/bench/irmin-pack/tree.ml#L82.
Closing; sorry for the noise.


Add tasks to upgrade percona to latest minor

This is needed to upgrade our percona to the latest minor release
offered. Installing the package will cause the DB to restart, so we want
to take care that the cluster is in a healthy state before we move on.
Also, becuase this is risky, we want to add an additional run-time flag
of 'dbupgrade' to determine whether or not these tasks run.
Change-Id: I0dbdf21a828325a2dbac08c2c812f97b7f779ec2

retest

retest


Docker link in description is broken

https://hub.docker.com/repository/docker/gameroasters/flatbuffers-unity leads to a 404.

yeah the docker hub is no longer maintained. the code to build the container in this repo should still workd


Frontend: "List" component

for:

Searching samples in the DB
All the mixes the user has made with the application
User's latest chat
Friends list
Random users list

Description (user story): a sidebar containing sub-windows (for searching samples in the db, view room messages, etc. as mentioned above) which the user can scroll through and interact with.


[ENH] Switch to sphinx-theme-builder

This PR switches to using sphinx-theme-builder, following this issue https://github.com/executablebooks/sphinx-book-theme/issues/417

[x] Use sphinx-theme-builder as the build backend
[x] Use webpack to compile and build assets. Run stb compile.
[x] Build docs.
[x] Run tests.
[x] Do a thorough cleanout
[ ] Update Docs

@pradyunsg where do you suggest something like a https://github.com/executablebooks/sphinx-book-theme/tree/master/sphinx_book_theme/translations folder should go in terms of the layout system according to sphinx-theme-builder https://sphinx-theme-builder.readthedocs.io/en/latest/reference/filesystem-layout/ .

I think we could just put it under theme/sphinx_book_theme/translations

I think we could just put it under theme/sphinx_book_theme/translations

@choldgraf since technically we are compiling jsons to *.mo files and and sphinx-theme-builder technically encourages any assets that need to be compiled to be put in src/sphinx_book_theme/assets, so I am a bit confused. Also, I think we should add the creation of *.mo files in webpack instead of committing them?

we are compiling jsons to *.mo files, and sphinx-theme-builder technically encourages any assets that need to be compiled to be put in src/sphinx_book_theme/assets, so I am a bit confused sphinx-theme-builder.readthedocs.io/en/latest/reference/filesystem-layout. Also, I think we should add the creation of *.mo files in webpack instead of committing them?

Hurrah! I wanted to make it easier / explicit for folks to keep compiled assets and uncompiled assets separate -- and I am happy to see that this caught non-CSS/JS stuff here! :)

where do you suggest something like a master/sphinx_book_theme/translations folder

I'd suggest leaving them under theme/sphinx_book_theme/ for now, and then coming back around to cleaning that up in a follow up PR.

My heuristic for thinking about such things is: Would it be less work for reviewers if this was in a separate PR? In this case, I think the answer is a yes.
It's technically where they are on main at the moment as well (so... in theory, someone could do {% include "sphinx_book_theme/translations/.gitignore" %} in their templates[^1]). We can certainly clean this up in well-scoped follow up PR. It makes reviewing easier, and, gives you a clearer list of "what all did I end up doing". :)
[^1]: Yes, that's why we should move those files out! :)

I think @pradyunsg has a good intuition here - we should get this PR into a mergeable state ASAP, and we can always use follow-up PRs to clean things up if need be - those won't be nearly as disruptive from a merge conflict standpoint.

Yes, totally agree with doing it in follow-up PRs. I guess mostly wanted to bring this issue to light here. Will create a separate issue once this is merged.
I have moved the translations folder to src/sphinx_book_theme instead of leaving them under src/sphinx_book_theme/theme/sphinx_book_theme. Because to make the translations module importable I had to create empty __init__.py in both theme folder and in theme/sphinx_book_theme folder (make them modules as well). To Make the dotted import possible.
As we are importing _convert python file of translations for testing:
from sphinx_book_theme.translations._convert import convert_json
Let me know if I should revert it and this isn't following proper convention.

[ ]  Removed theme_dev_mode, config variable, as I think it's not needed anymore, given sphinx-theme-builder. https://github.com/executablebooks/sphinx-book-theme/pull/469/files#diff-827e5b7309121354fc6d5411c429a4a07800bc6abee0db2f47b60c400ec4be99L26

Have no idea, how to fix the pre-commit.ci failure😢

Hmm maybe this is related to the "additional dependencies" config? Could we specify the extra dependency in the error message there?
https://github.com/mgedmin/check-manifest#version-control-integration
I wonder if @pradyunsg could weigh in on whether this is somehow related to the sphinx theme builder configuration setup 🎉

You don't need check-manifest anymore. That said, that looks like something that's a bit of a Ubuntu mess related to breaking ensurepip?

Is that because we're not checking in programmatically generated assets into the repo anymore? If that's the case then I am +1 on removing it.

Is that because we're not checking in programmatically generated assets into the repo anymore?

and by extension, what does this mean for MANIFEST.in?

That can be removed as well! sphinx-theme-builder determines what files to include by asking git!
https://github.com/pradyunsg/sphinx-theme-builder/blob/10450cfeb65bc3610a6708ee731069c707978fef/src/sphinx_theme_builder/_internal/distributions.py#L33

I guess that's similar to flit 👍

Basically, the sdist ends up being the same as a git archive or whatever is the equivalent of downloading from the GitHub repository after generating the assets.

Yup, basically the same as flit. I'm pretty sure I copy-pasted some code for that too.

(side comment - I think all of these more recent developments in Python packaging are pretty cool, thanks @pradyunsg for all of the work you're doing in that space :-) )

Basically, the sdist ends up being the same as a git archive or whatever is the equivalent of downloading from the GitHub repository after generating the assets.

This is pretty cool, I will remove the MANIFEST file.

The docs look good, apart from the margin directive: https://sphinx-book-theme--469.org.readthedocs.build/en/469/reference/special-theme-elements.html .
Because the margin element is aside instead of div. Will add styling for aside.

@AakashGfude the aside thing is broken because this PR increases the docutils version, I think

@choldgraf have done a quick fix here, but will open an issue, to check style updates needed for docutils update.

[ ] open issue to address style changes for docutils update.

@AakashGfude I think there's already an issue for it, so double check that one first. Though hopefully there's not a ton to change because we've been removing the "div" assumption here and there and just using class names

Looks okayish to review.
I still need to update the docs though.

Provide some basic information about how to build the theme locally with tox
Provide some basic information about the sphinx-theme-builder
Provide some basic information about the webpack and JS-specific files that you've added so others know what's going on

@choldgraf I updated the contributing guide https://github.com/executablebooks/sphinx-book-theme/pull/469/commits/fb5ed0adfcbd84cb5e658ccc25951334c2e1dfe1 , which briefly touches or just mentions all these points. I reckon we should add more, but we can do it in a follow-up PR?
I will check the translations README, and update it as well.

Those contributing docs look good to me 👍 - anything else you want to tackle in this PR? Otherwise I'd say we leave it open until tomorrow in case others want to make comments, and merge it in unless there are any objections. This PR touches a lot of stuff so it'd be good to get it in so that we can start iterating.

[ ] Add @choldgraf's email address

Sounds good @choldgraf. I think I am done with any edits/additions in this PR.

OK, I took another pass and added in one quick comment, there are likely still some things that we need to clean up. The styles look good to me in general, but there seem to be a few tweaks to make (maybe related to docutils). I will tackle those in the refactor PR (#471) and we can hold off on closing #392 until that PR is merged as well.
I'll merge this PR so that we can start iterating a bit from here.


Integrate with PostgreSQL

Hi @truongtphat, kindly help support on this feature request. Please check with @nhathoang989 in case if you have any questions.

Hi @truongtphat, kindly help support on this feature request. Please check with @nhathoang989 in case if you have any questions.

I like SQLite

Hi @biapar SQLite already added in the latest Beta.9 release. Kindly try and have fun :)


Gradle not available in Android Images

I have a poc Cordova project I'm trying to get running with the v2 script, but Gradle doesn't seem to be available as a global on the command line. It looks like the OpenJava image should be installing it.
I'm new to Circle's v2 and Cordova, there seems to be an issue, but maybe I'm just missing something.
I've tried with both the node and non-node Android images (same result).
Is there something special I have to do to call Gradle?
Any thoughts?
Error:
#!/bin/bash -eo pipefail
gradle -v
/bin/bash: gradle: command not found
Exited with code 127

Script:
version: 2
jobs:
  #build-and-test:
  build:
    # macos:
    #   xcode: "9.0"
    working_directory: ~/code
    docker:
      - image: circleci/android:api-25-node8-alpha
    environment:
      JVM_OPTS: -Xmx3200m
    steps:
      - checkout
      - run:
          name: Install Node Dependencies
          command: gradle -v

Gradle is not built into the image. I think the sample projects uses greadlew which is a wrapper that is checked into your repo.
Also, I am a bit confused here:

name: Install Node Dependencies

Why are you using gradle to install node dependencies?


[W4.5c][T15-A3] Kaiser Tan

Completed [LO-Exceptions]

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


Social media icons+links  for website

Feature Title
Social links
Description
We need to create new NextJS component which displays social media icons with clickable hyperlinks, Please follow below reference for design in tailwindcss.
Code reference - https://codepen.io/Vigneshkanna-js/pen/PoerVNX
The component should be placed in homepage under banner and should be mobile friendly

Motivation
Website without social media links ? Uhfff 🥲
Issue Type
Frontend
Screenshot of the feature if done ?

Are you taking this fetaure implementaion
no

@jsvigneshkanna  - I would like to work on this. Can you please assign to me?

Sure @javamultiplex Sir,
Please make sure the new component is mobile responsive

@jsvigneshkanna  - Can you please share the steps to start the application? I can't find on README.md file.

@javamultiplex ,

npm install (install all the dependencies)
npm run dev
it connects to localhost:3000
make changes and commit your code

Social media icons location is correct ? or it should be somewhere else?

@javamultiplex ,
Please it as a seperate component named socialButtons and keep it components folder
We basically need it above footer and under contact form which are both under construction

@jsvigneshkanna  - Correct me If I am wrong, as part of this issue, I just need to create the socialButtons component? There is no need to place it below Home Page Banner?

@javamultiplex
For now, we can display new component under banner which need padding and background color as component page,


Flow type checking within objects

Hi Flow Developer Team,
we love flow and we would like to use it in one of our web projects. Meanwhile our project is very big, so that we intend to introduce flow file by file. Adapting a small test file, we realized that flow seems to misunderstand the class definitions of the UI framework we use.
Most of our JavaScript files look like the following example:
/** one of our ui elements **/
framework.ui.controller("some_unique_id", {
    foo: function() {
        this.bar("some string");
    },
      bar: function (n: number): number {
          return n * n;
    }
});

There is no library definition for the UI framework we use, thus I tried to define a small definition on my own:
/* @flow */

/** library definition */
declare class framework{
    static ui : UI;
};

declare class UI{
  controller(bla : string, obj: {}): {};
};

/** one of our ui elements **/
framework.ui.controller("some_unique_id", {
    foo: function() {
        this.bar("some string");
    },
      bar: function (n: number): number {
          return n * n;
    }
});

The expected behavior we would like to see is that flow shows an error on line this.bar("some string"); because bar is expecting a number. Unfortunately, flow shows no errors. Giving flow a plain function definition it points to the expected error:
/* @flow */

/** using a plain function it shows an error message **/
function square(n: number): number {
  return n * n;
}

square("some string");

You can find a flow try link here.
Is there a way to get flow to do the type checking within the object?
Thank you for your Help :)
Robin

/* @flow */

class FooBar {
  foo(): void {
    this.bar("some string")
  }
  bar(n: number): number {
    return n * n
  }
}

/** library definition */
declare class framework{
    static ui : UI
};

declare class UI{
  controller(bla : string, obj: FooBar): {}
};

/** one of our ui elements **/
framework.ui.controller("some_unique_id", new FooBar())

Oh thank you a lot! I will test it in the productive code immediately  :D


[SP-6274]-Backport of PDI-19623 - PDI "Delete files" Step - An error …

…does not occur when you specify a file that is stored in a directory that does not have permissions. (9.3 Suite)

:white_check_mark: Build finished in 20m 21s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:exclamation: No tests found!
:information_source:  This is an automatic message


Improvement - add other type in RType which Redisson used.

I am not so sure :

Whether these changes are necessary ?
If necessary, ('none' type use "NONE" instand of 'null') is it the right change ?

String parsing is made here org.redisson.client.protocol.convertor.TypeConvertor. I don't mind to move it to RType. NONE type should be avoided and null used instead. Like in current implementation.

Is there a follow up to this question, the org.redisson.RedissonKeys#getType method doesn't work properly if no other types are added to the RType

@zyh-stu
Can you create an issue regarding this task?

@zyh-stu
Can you create an issue regarding this task?

Created #6102


Only suspend Camera tracks in background

Related:
https://github.com/livekit/client-sdk-swift/issues/219
https://github.com/livekit/client-sdk-swift/issues/182

I think this is safe to merge I will go ahead and merge this one.


fix off-by-one bug in cross-attention-control

there is currently an off-by-one issue that causes .swap() to operate on tokens at one index earlier than it is supposed to.
prompt token sequences begin with a "beginning-of-sequence" marker <bos> and end with a repeated "end-of-sequence" marker <eos> - to make a default prompt length of <bos> + 75 prompt tokens + <eos>. the .swap() code was failing to take the column for <bos> at index 0 into account. the changes here do that, and also add extra handling for a single <eos> (which may be redundant but which is included for completeness).
based on my understanding and some assumptions about how this all works, the reason .swap() nevertheless seemed to do the right thing, to some extent, is because over multiple steps the conditioning process in Stable Diffusion operates as a feedback loop. a change to token n-1 has flow-on effects to how the [1x4x64x64] latent tensor is modified by all the tokens after it, - and as the next step is processed, all the tokens before it as well. intuitively, a token's conditioning effects "echo" throughout the whole length of the prompt. so even though the token at n-1 was being edited when what the user actually wanted was to edit the token at n, it nevertheless still had some non-negligible effect, in roughly the right direction, often enough that it seemed like it was working properly.
a cat eating a hotdog.swap(pizza) k_lms 10 steps without this PR:

with this PR, note how much more coherently the hotdog has been swapped for a pizza:

(blue face attack from low step count)

I've been playing with these changes and this is what I expected cross-attention control to look like.
"cat playing with ball, oil painting, extreme details, by rembrandt" -s 50 -S<PHONE_NUMBER> -W 512 -H 512 -C 7.5 -A ddim

"octopus playing with ball, oil painting, extreme details, by rembrandt" -s 50 -S<PHONE_NUMBER> -W 512 -H 512 -C 7.5 -A ddim

"cat.swap(octopus, s_start=0.3, s_end=0.5, t_start=0.3, t_end=1) playing with ball, oil painting, extreme details, by rembrandt" -s 50 -S<PHONE_NUMBER> -W 512 -H 512 -C 7.5 -A ddim

I've had even more success making s_end equal to s_start, but that may depend on the image and prompt.

Working great.
man eating banana

man.swap(tiger) eating banana

@keturn wrote:

Would it make more sense to include those beginning-of-stream/end-of-stream markers in the thing this is iterating over?

good points. i think i found some other bugs so i'll open a new PR and be sure to include them. thanks!

The nature of the bug makes me ask: Is this the sort of thing we could have a unit test for?

it is. i will see if i can make one with the next PR.

A