Tried to load /home/docker/ASRL/vtr3/models/sample_model.pt

We need pre-trained weight and its directory creation in the installation guide for new dockerfiles and code.
(https://github.com/utiasASRL/vtr3/wiki/Installation#building-vtr3)

Hi there,
Sorry for missing that, that sample code is not really designed to be run. It's there to show how to build the file.
I'm working on the update that will allow for easier selection of learned components or not.

This has been fixed now.


renable travis Build status on project pages shows last run was a few years ago.  [JIRA: RIAK-2404]

possible to turn travisci on again to make the test run status more transparent?

We can look into it. @Vagabond - if you have a moment - what was the history around turning off Travis in the first place?

It was unreliable, slow and we had switched to buildbot/bors.
Hopefully the Erlang builder isn't on the same host as the Java builder anymore, it should be fine.

I bet travisci may be a lot better now. MachineZone use travis for their mzbench testing that invokes a cluster setup for multi node testing. It may be nice to reenable it or at least make unit test for each run visible. A local run if tests already shows multiple failures.

:+1: for this. We've been using travis for esl/MongooseIM for long time already and recently I observed significant performance/stability improvement, especially when you use the container-based infrastructure. New repos added to travis have this by default but old can enable this infrastructure by specifying sudo: false in the .travis.yml file, from what I know.

No objection to re-enabling it.

I've turned it back on for PRs (other options include all pushes, which seemed not useful). We'll probably need some work to get it up and running again properly, so I wouldn't assume that the first couple will pass.

Just merged #340 to update the OTP releases. Going to close this now - please reopen if something seems amiss.

Ah, forgot to re-enable the TravisCI web hook on the repo. Is that something you can do @JeetKunDoug?

Should be on now. Let me know if it doesn't work. Closing this until I hear otherwise.


Fix sqlite3_opt_unlock_notify with USE_LIBSQLITE3

A valid sqlite header must always be included (like in the other files) but sqlite3-binding.h explicitly guards against the system library case.
This fixes forgejo compilation with libsqlite3 tag.

@mattn / @rittneje this commit simply fixes the sqlite3_opt_unlock_notify.{c,go} files so that they respect USE_LIBSQLITE3. Now that Fedora is moving to Forgejo it might be nice to help them out.

@rittneje thank you for merging this!!!


Feature: Add two new headers to proxy.conf

Fixes #1609. Adding both  X-Forwarded-Host  and X-Forwarded-Port, this is vital for some services behind a proxy (used to allow creation of absolute links in html). I've had to include at least the Host version in the past for Jenkins and Nexus.
Been running locally for 24 hours, does not appear to break any of my 15+ services currently running behind NPM would allow people to host those problem services without the need for advanced configuration.

This appears to be failing due to lack of python during the sqlite3 build in the backed.
[2021-11-29T13:53:52.222Z] make: Entering directory '/app/node_modules/sqlite3/build'
[2021-11-29T13:53:52.222Z]   ACTION deps_sqlite3_gyp_action_before_build_target_unpack_sqlite_dep Release/obj/gen/sqlite-autoconf-3310100/sqlite3.c
[2021-11-29T13:53:52.222Z] /bin/sh: 1: python: not found

This appears to be looking for python2 (/usr/bin/python) rather than python3 (/usr/bin/python3), anything changed in that regard in the CI system lately?

I think this is because node:latest is now based on Debian bullseye (SHA1 hashes for latest and current-bullseye match). Bullseye by default no longer has python 2 installed. So either we apt-get update ; apt-get install -y python2.7 as part of that build docker run or use node:current-buster as we did before node:latest moved.
If we do option 2 we are waiting until node-gyp and dependencies move to python 3 and hoping node support buster until that happens.

Wasn't the EOL for Python 2.7 in 2020 already? Would make absolutely no sense to use that going forwards.

If node-gyp does support python 3 , then we need to fix the bullseye alternatives to allow python 3 to be used for /usr/bin/python so the build scripts will find python in their path.
From a new minimal bullseye install.
# python
-bash: python: command not found
# update-alternatives --install /usr/bin/python python /usr/bin/python3 2
update-alternatives: using /usr/bin/python3 to provide /usr/bin/python (python) in auto mode
# python --version
Python 3.9.2

You might have to rebase on the develop branch as this python fix has been added

This is an automated message from CI:
Docker Image for build 3 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-1614
Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.

I have to revert this PR due to #1717. Perhaps there is another way around this.
Re-reading the PR comment above, I use jenkins behind NPM and haven't had a problem with it in the past.


Add 'none' to sort order

Sort order 'none'
currently we can sort using asc or desc, add a none that'll keep ordering defined by the user.
Reason
I just a funny problem where I had :
import 'zone.js/dist/zone';
import 'zone.js/dist/long-stack-trace-zone';

and it got organized by asc order that produced a  no zone error because the order matter. You need to first import zone and then the long stack. I've covered it by using desc order but what I really want here is no sorting at all.
Even if imports are not really supposed to have side effects, sometimes they does and in those cases order is important and is not asc nor desc it's specific.

Same problem here.
It would be great if we can have tslint style comments to ignore sorting for a set of lines (like zone.js imports in this example)


What you think about universal-webpack?

https://github.com/catamphetamine/universal-webpack
discussion on next.js
best boilerplate I've seen using it
sturdy codebase to extract common logic, plethora of examples, possibility to write server on whatever you want, why not?

 I‘d rather not have a dependency to that guy in my commercial stack.

@D1no I understand )


Improve performance of getting bookmarked tables

Summary of Changes
Previously, Metadata service was performing queries against each columns while getting bookmarked table. This change is to remove those query against columns.
Tests
Unit tested. Integration tested locally.
Documentation
N/A
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes make test
[x] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

Codecov Report

Merging #59 into master will decrease coverage by 0.1%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
- Coverage   75.15%   75.05%   -0.11%
==========================================
  Files          19       19
  Lines         946      946
  Branches       72       72
==========================================
- Hits          711      710       -1
- Misses        212      213       +1
  Partials       23       23

Impacted Files
Coverage Δ

metadata_service/proxy/neo4j_proxy.py
75.84% <50%> (-0.22%)
:arrow_down:

metadata_service/api/user.py
34.61% <66.66%> (+1.28%)
: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 7f8d8c1...0aad51e. Read the comment docs.

@jinhyukchang thanks for adding this. You need to add the new API call to base_proxy/atlas_proxy as well.

Done

looks good! Have you tried to push a pre-release pypi and tested in staging?

looks good! Have you tried pushing a pre-release pypi and tested in staging?

I've tested out locally by calling our staging Neo4j. Should be good enough but will test out in staging again once released.


Reminder support for SpringActorClassFinder

Hi @johnou could you check if https://github.com/electronicarts/orbit/commit/c6ac3b3c8a9cf528b8a044cc268f34725855bdd7 addresses this issue?


Test in node 0.4+ on Travis

An alternative to #41.  I had a change of heart.  If it still works on 0.4, why drop compatibility?

Man, I'm tempted not to test on node 0.4 just because the tests take forever on Travis

lol, node 0.4 doesn't even come with npm, I forgot that.  Let's try 0.6.

welp, looks like node-tap doesn't work with any versions of node older than 0.10, and I don't feel like using an ancient version of a testing library just to test for versions of node that nobody uses any more, so I'll close this pull request and merge #42 in the next few days if nobody objects.


⚠ ku-s has degraded performance

In bc46ae2, ku-s ($KU_API_STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 9428 ms

Resolved: ku-s performance has improved in 1e1fea2 after 9 minutes.


Please add more explanation text about Time conditions

Issue in ./help/using/building-journeys/condition-activity.md
On this page, under Time conditions, there are no descriptions of the settings except a screenshot. It will be helpful to add some explanatory text to each field input on the UI.
Thanks!

Resolved


Update to 1.20.2

Sorry for having this deleted before, original PR message below:

Been told https://github.com/QuiltMC/quilt-kotlin-libraries/pull/84 is urgent to merge otherwise built artifacts would be broken otherwise.
At the moment I've only updated versions to track with the latest FLK and get it to compile with 1.20.2. There are some other changes which should likely be addressed before this gets merged:

[x] The annotation @CodecSerializable.Polymorphic is not top level, @MetaSerializable used to allow this however it's now been deprecated and it only supports top level annotations.
[x] There has been networking refactors and I or someone else probably need to add more extensions for those.
[x] Parts of the code base is generated with Silver's math codegen project, those haven't been looked at but may need some updating.
[x] Fabric Language Kotlin's language adapter seems to have broken due to changes in Kotlin 1.9.20, should verify our language adapter still works: Changes for upstream since last bump
[x] Switch to using Yumi Licenser

I'll try and recap the other messages I sent, but I didn't save them like the original:

It seems like math types haven't changed between 1.20.1 and 1.20.2
Some of the methods have been renamed to match QSL, therefore API breakage, also fixed a deprecation with the annotation mentioned not being top level which is another breaking change.
Fixed licenser not working on java files in addition to switching to yumi-licenser.
Found out object function entry points didn't work when testing the language adapter so fixed that as well.

I was unaware of any Codec changes tbh, I assume any tests would have ran and passed, Codecs otherwise aren't in my area of expertise, if there's any notable mod I can test with real quick I'll give that a shot.

Sadly it appears there are Codec issues in the samples I didn't spot before. Will try and take a look at those now.

Managed to hack something together to run the Codec serialization tests and they all pass.
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.BasicSerializationTests:testBasicDecoding
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.BasicSerializationTests:testBasicEncoding
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.NullableSerializationTests:testAmbiguousInlineNullsUnwrapped
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.NullableSerializationTests:testAmbiguousInlineNullsWrapped
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.NullableSerializationTests:testBasicNullableValue
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: DataResult[Left[null]]
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.NullableSerializationTests:testExplicitNullFields
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.NullableSerializationTests:testImplicitNullFields
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.PolymorphicSerializationTests:testConflictingDiscriminator
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.PolymorphicSerializationTests:testFlattenedDecoding
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.PolymorphicSerializationTests:testFlattenedEncoding
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.PolymorphicSerializationTests:testStructuredDecoding
[17:54:24] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.PolymorphicSerializationTests:testStructuredEncoding
[17:54:25] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.MapSerializationTests:testAllowedMapDecoding
[17:54:25] [Render thread/INFO] (Minecraft) [STDOUT]: DataResult[Left[({AllowedCodecInline(field=foo:bar)=baz}, null)]]
[17:54:25] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.MapSerializationTests:testAllowedMapEncoding
[17:54:25] [Render thread/INFO] (Minecraft) [STDOUT]: Running samples.qkl.serialization.MapSerializationTests:testInvalidMapEncoding
[17:54:25] [Render thread/INFO] (Minecraft) [STDOUT]: All tests succeeded```


Hapi v17

Hapi v17 is a full rewrite and will cause some big changes.

https://github.com/hapijs/hapi/issues/3658

We need mongo-models and hapi-mongo-modles to get updated and then we should be in good shape to begin the migration. Help on these modules would be much appreciated 🙏.

[X] https://github.com/jedireza/mongo-models/issues/24
[x] https://github.com/jedireza/hapi-mongo-models/issues/74

We also need Frame to get updated for Hapi 17 first since we'll copy most of the code changes.

[x] https://github.com/jedireza/frame/issues/186

Hi there :)
Any estimation on the V17 rewrite release?
Thank you much in advance
R

Nope. I work on it when I can.

I've decided to archive this project. Thank you for your interest and contributions.
https://github.com/jedireza/aqua#no-longer-maintained


Option to customize dns config

Currently, using hickory-dns always reads from /etc/resolv.conf. It would be ideal to be able to customize the resolver config; one case would be when /etc/resolv.conf does not exist.

Some basic config that works for either backend could be proposed.
But also, there's an option to pass a completely custom resolver(), so I would prefer that over hickory-specific options.

Yes, it exists in blocking too: https://docs.rs/reqwest/latest/reqwest/blocking/struct.ClientBuilder.html#method.dns_resolver


rustup man

There's a longstanding issue with multirust that man pages can't be displayed. My current idea to solve this is a rustup man command.
The wrinkle though is that you can't just display man pages on windows, and I want to maintain feature parity. We may want to write a basic troff interpreter, and whatever else is needed to format man pages.

Dupe of #490


Use cache@v4 and save-always option

Cheers lovely people of rainforest!
TLDR
Use github's cache action and it's save-always option instead of a fork that still uses node 16.
Long story
So I noticed a Node 16 warning when running my RF suite

I checked out always-upload-cache action but it seems neglected. I started looking into Github's cache action issue linked in its README and noticed that the v4 Changelog mentions a save-always option.
While it's not documented a quick peek the code suggests it does exactly what always-upload-cache does.
I haven't tested that yet, I'm planning to test in next week on my suite.

Looks like that save-always parameter doesn't actually work in practice 😞  https://github.com/actions/cache/issues/1315

Shoot, nice catch @magni-
I'll take a stab at the suggested solution from that issue.


Optimize serialization and parsing performance on large SBOL files

Large SBOL files (including some in the round-trip test suite) currently bog down during serialization and parsing.  There are optimizations for this implemented in libSBOL which should be ported over to this new library.

A large genome to test against also: https://hub.sd2e.org/user/sd2e/design/W303/1

An exploration revealed that RDFlib takes more than half of our file loading time. We simply cannot go any faster than that, and that is 2x - 4x longer than pySBOL's loading which uses Raptor. We have implemented some of the optimizations used in libSBOL, and tried some others that did not yield any performance improvements.
I think we've done what we can reasonably do on this. It's possible we may need to revisit this topic in the future.

Is there any way to use Raptor with pySBOL3 without reopening the entire horrible cross-platform can of worms?


Improve dotnet publish in azure-release.yml

Codecov Report

Merging #43 (ff5aaec) into master (486e820) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #43   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            2         2
=========================================
  Hits             2         2

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 486e820...ff5aaec. Read the comment docs.


Not seeing anything reported here. Closing.

Not seeing anything reported here. Closing.
Originally posted by @iBotPeaches in https://github.com/iBotPeaches/Apktool/issues/2704#issuecomment-980547767

Originally posted by @gorwhh in https://github.com/gorwhh/sherlock/issues/1#issuecomment-982799428

What is this? Why did you spam me with like 8 emails of this? I don't even understand.


Apiserver monitoring indicator is empty

What's your question
Apiserver monitoring indicator is empty
Environment: OS & Hardware Information
VMware CentOS 7.7，k8s 1.17.3
prometheus error

I want to know which SA and clusterrole are used to get apiserver
Because the error notification uses anonymous users

The prometheus ServiceAccount in the monitoring namespace (assuming that's where you deployed it to). I'm fairly certain this is something else though, as we've never seen this in ~4 years.

I checked and found that RBAC permissions were correct

Pretty sure this is something misconfigured in your cluster indeed. Closing for now, let us know if you still think it's a problem with kube-prometheus itself.


Generic recovery mission by Denis Averin malfunc.

the generic recovery missions given by Denis Averin, Cydonia UC Exchange, no longer generates all needed items to recover on mission locations.
i played these missions along time ago in september, returning now. i updated SFSE and SCP so far, as well as the official game. back in september the mission was to recover 5 generic weapons of a random POI. in the POI's main chest were allways 5 mission required weapons present.
now its only 1 weapon. picking it up marks mission complete to return to Denis Averin. back at Denis Averin the turn in dialog returns a dialog from Denis, the starfare need to retrieve all neded 5 weapons.
i made the generic missions 5x in total, all yield only 1 mission weapon each. ( i liked this mission for its simplicity )

Details
Generic recovery mission
Denis Averin, UC Exchange, Cydonia
only 1 mission item is generated, instead of all needed 5. 5 were allways generated in sep/2023
Game Version
<IP_ADDRESS>
Language
German (DE)
New Game
Yes
Reproduction Steps
doing generic recovery mission given by Denis Averin

Duplicate of #921

Duplicate of #921
i was like browsing here before, on 921 my brain was like afk then

Closing as duplicate


[Rendering] phong shading

Now I see that this is utterly old branch, not going to add fixes there.
Will add Phong Feature changes in new fresh brand new branch and then create PR.


Add an optional .conplicity.overrides file

Coverage decreased (-0.2%) to 14.085% when pulling ab161281c96b94e9c56a0b0861636528b3ce868b on raphink:volume_overrides into 5688ad35ff157cd5b408a94b2a2210de39a3d50f on camptocamp:master.

Coverage decreased (-1.1%) to 13.166% when pulling 56bcc1ee75697755c6ac650a755547e9f854fbca on raphink:volume_overrides into 5688ad35ff157cd5b408a94b2a2210de39a3d50f on camptocamp:master.

Coverage increased (+0.4%) to 14.684% when pulling 4db738e16e239f734a5f10266f8d59786b513690 on raphink:volume_overrides into 5688ad35ff157cd5b408a94b2a2210de39a3d50f on camptocamp:master.

Coverage increased (+0.4%) to 14.684% when pulling 4db738e16e239f734a5f10266f8d59786b513690 on raphink:volume_overrides into 5688ad35ff157cd5b408a94b2a2210de39a3d50f on camptocamp:master.

Coverage increased (+0.4%) to 14.684% when pulling 8314ef4bdd82c2ff5a68b78954dfc642a69c98ad on raphink:volume_overrides into 5688ad35ff157cd5b408a94b2a2210de39a3d50f on camptocamp:master.

Coverage increased (+0.4%) to 14.684% when pulling 73588857f42ce5d3254502b84fecd65073d59329 on raphink:volume_overrides into 5688ad35ff157cd5b408a94b2a2210de39a3d50f on camptocamp:master.


[Doc Improvement][Updated personal tab image]

Updated image for personal tab.

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

Article
Score
Issues
Scorecard
Processed

msteams-platform/tabs/what-are-tabs.md
93
7
link
:white_check_mark:

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

Learn Build status updates of commit fc2ff78:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/tabs/personaltab.png
:white_check_mark:Succeeded
View

msteams-platform/tabs/what-are-tabs.md
:white_check_mark:Succeeded
View

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

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

Article
Score
Issues
Scorecard
Processed

msteams-platform/tabs/what-are-tabs.md
93
7
link
:white_check_mark:

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

Learn Build status updates of commit 0d4bb42:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/tabs/personaltab.png
:white_check_mark:Succeeded
View

msteams-platform/tabs/what-are-tabs.md
:white_check_mark:Succeeded
View

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

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

Article
Score
Issues
Scorecard
Processed

msteams-platform/tabs/what-are-tabs.md
93
7
link
:white_check_mark:

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


Section 1.1

Is tonnes/miles not in the table of units because they aren't SI units? (since they are used in the auxiliary constants, I figured they should maybe be added)
SRS

I thought that they were not the basic seven SI units so I didn't add them but it seems like there is no harm to do so. Fixed in ffdc718.


Bug] [ChatData] editor run sql error

Search before asking

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

Operating system information
Windows
Python version information
3.10
DB-GPT version
main
Related scenes

[X] Chat Data
[ ] Chat Excel
[ ] Chat DB
[ ] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
CPU
Models information
LLM: proxyllm
What happened
使用默认的样例数据，在ChatData输入问题  查询所有学生的姓名，专业和成绩，按成绩降序排序，在edito中将生成的sql中limit 50修改成limit 10,然后点击右上角的Run,报错了。

错误信息如下：
INFO  [pilot.openapi.api_v1.editor.api_editor_v1] editor_sql_run:{'db_name': 'case_1_student_manager'}
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\anyio\streams\memory.py", line 98, in receive
    return self.receive_nowait()
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\anyio\streams\memory.py", line 93, in receive_nowait
    raise WouldBlock
anyio.WouldBlock

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\base.py", line 78, in call_next
    message = await recv_stream.receive()
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\anyio\streams\memory.py", line 118, in receive
    raise EndOfStream
anyio.EndOfStream

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\uvicorn\protocols\http\httptools_impl.py", line 426, in run_asgi
    result = await app(  # type: ignore[func-returns-value]
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\uvicorn\middleware\proxy_headers.py", line 84, in __call__
    return await self.app(scope, receive, send)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\fastapi\applications.py", line 284, in __call__
    await super().__call__(scope, receive, send)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\applications.py", line 122, in __call__
    await self.middleware_stack(scope, receive, send)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\errors.py", line 184, in __call__
    raise exc
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\errors.py", line 162, in __call__
    await self.app(scope, receive, _send)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\base.py", line 108, in __call__
    response = await self.dispatch_func(request, call_next)
  File "D:\work\algo\大语言模型\DB-GPT\pilot\utils\tracer\tracer_middleware.py", line 44, in dispatch
    response = await call_next(request)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\base.py", line 84, in call_next
    raise app_exc
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\base.py", line 70, in coro
    await self.app(scope, receive_or_disconnect, send_no_error)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\cors.py", line 91, in __call__
    await self.simple_response(scope, receive, send, request_headers=headers)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\cors.py", line 146, in simple_response
    await self.app(scope, receive, send)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\exceptions.py", line 79, in __call__
    raise exc
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\middleware\exceptions.py", line 68, in __call__
    await self.app(scope, receive, sender)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\fastapi\middleware\asyncexitstack.py", line 20, in __call__
    raise e
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\fastapi\middleware\asyncexitstack.py", line 17, in __call__
    await self.app(scope, receive, send)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\routing.py", line 718, in __call__
    await route.handle(scope, receive, send)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\routing.py", line 276, in handle
    await self.app(scope, receive, send)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\starlette\routing.py", line 66, in app
    response = await func(request)
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\fastapi\routing.py", line 241, in app
    raw_response = await run_endpoint_function(
  File "D:\ProgramData\Anaconda3\envs\DB-GPT\lib\site-packages\fastapi\routing.py", line 167, in run_endpoint_function
    return await dependant.call(**values)
  File "D:\work\algo\大语言模型\DB-GPT\pilot\openapi\api_v1\editor\api_editor_v1.py", line 117, in editor_sql_run
    sql = run_param["sql"]
KeyError: 'sql'

What you expected to happen
正常运行SQL
How to reproduce
见上面的描述
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Yes, this will be fixed later, you can find from here

是的，这将在以后修复，您可以从这里找到

请问修复了吗？我使用最新版本，还是遇到了同样的问题

I have the same issue, will this be fixed in the next version?

请问修复了吗？我使用最新版本，还是遇到了同样的问题

我咋没有编辑模式 呢

我也遇到了


Consider adding binary fuse filters

We have updated the C, C++, Go and Java code with a new breed of filters (binary fuse filters). The implementation is nearly identical, but the construction time is at least 2x faster, and the resulting filter is significantly smaller.
Binary fuse 8 and fuse 16 in C:
https://github.com/FastFilter/xor_singleheader/blob/master/include/binaryfusefilter.h
Binary fuse 8 in Go:
https://github.com/FastFilter/xorfilter/blob/master/binaryfusefilter.go

@lemire can you elaborate on why the construction time for binary fuse is significantly faster? I understand that the structure of the fuse filters is such that the total space consumed is smaller, but if I recall correctly fuse filter construction may fail if there aren't enough keys.

@ayazhafiz
We worked on something new, which we call the binary fuse filter. It has the same lookup performance as the xor filter, but it has 2x the construction speed (roughly) and uses significantly less memory. If you have many keys (say a million), then it is much better.
We have an upcoming preprint, I can share it with you, just email me at<EMAIL_ADDRESS>and I will send you the PDF.

Released 0.5.0 with Fuse8 and Fuse16 implementation, now available at https://crates.io/crates/xorfilter-rs

Fantastic news!


"Discord seems not be installed on your system" macOS

Error above. Discord is installed.
System: macOS Sierra
heeelp

I recommend you join the BD support servers and talk to @qwerasd


Suggest a new output of nuget package

Example Nuspec file:
<?xml version="1.0"?>
<package >
  <metadata>
    <id>github-release</id>
    <version>0.5.3</version>
    <title>github-release</title>
    <authors>Nicholas Carpenter</authors>
    <owners>Nicolas Hillegeer</owners>
    <requireLicenseAcceptance>false</requireLicenseAcceptance>
    <description>CLI utility written in GO for Build Automation.  See https://github.com/aktau/github-release</description>
    <projectUrl>https://github.com/aktau/github-release</projectUrl>
  </metadata>

  <files>
  <file src="github-release.exe" target="tools" />
  </files>

</package>

Then call:
nuget pack github-release.nuspec

Attempting to build package from 'github-release.nuspec'.
Successfully created package 'C:\Users\ncarpenter\Desktop\github-release.0.5.3.nupkg'.

https://www.nuget.org/packages/github-release/ as an example

I'm not entirely sure what you mean. Do you mean that github-release should generate its own NuGet spec file?

No I was suggesting as part of your release process (pun-intended :)) - you currently ship a windows CLI utility in a zip.  I was looking to integrate this into C# based builds by using the Nuget gallery (C# based depdency management) and NAnt
So the link I posted above was the repackaged zip as a Nuget package that could be easily integrated into C# based solutions/build systems.  If you would like some more detail let me know.

The file I offered above was a sample file you could use to build a nuget package with your utility.

Well, I don't have anything against it. Yet I don't have a C# environment installed at home, do I need one? I tend to use only Linux and OSX since I don't feel like paying for a Windows license.
I guess my question is: what's your workflow for publishing such a package?

Usually this is Windows.
Basic Workflow.

Build the Windows based binaries (github-release.exe)
Call Nuget pack using the sample .nuspec to create a Nuget Package
Call Nuget push to publish it to nuget gallery

Looks like alot of people have figured out how to use mono to get Nuget to work on Linux/Mac:
https://github.com/dungpa/dsls-in-action-fsharp/blob/master/.nuget/nuget.sh
https://gist.github.com/andypiper/2636885
https://docs.nuget.org/consume/nuget-faq#getting-started
This isnt a problem if you dont have time for this.  Just thought it would be an easy way to get you into the c# based build systems.  I am using the utility from Nant, and I didnt want to deal with installing the utility globally on the build server.  Different versions, etc....
So I am pushing all build utilities into Nuget packages - NAnt, NUnit, and this is another one of those.  This makes the c# build server, real easy because all build utilities are downloaded from Nuget at build time.


Does it have to be a delta file

Does Assemblytics have to be in a delta format? I am having issues making delta files with MUMmer 3 and MUMmer 4 and will attache screenshots below. I obnly happened along Assemblytics yesterday and before hand I was making BAM files to import into Tablet to view deletions and also vcf files using sniffles. I really want to use Assemblytics but cannot seem to make .delta files.
I am very new to command line and a novice. My background is microbiology.
Cheers
Peter
MUMmer 3 screenshot

MUMmer 4 screenshot

Assemblytics does only work with delta files.
If you are aligning a genome assembly to a reference genome, then MUMmer + Assemblytics is the right choice for variant-calling. If you are instead aligning long reads directly (without first assembling them) then NGM-LR/Minimap2 + Sniffles is the right choice for variant-calling. Sniffles should not be used on assemblies, and Assemblytics should not be used on raw reads.
If you do have an assembly and want to go with MUMmer, then I think you're close, but you just have to try a few more things to run MUMmer correctly. Did you check what the manual says about supported file formats? If you need more help running MUMmer and reading the manual is not helping, then I recommend posting on Biostars: https://www.biostars.org/
You will also probably want to use the web version of Assemblytics at assemblytics.com, since it is very user-friendly.
Best of luck!

So I used porechop to trim my Oxford Nanopore basecalled reads then used NGMLR to align with a reference sequence and got a fasta output. Then I tried to run this through Assemblytics. Should I instead use BWA and then run it through Assemblytics? I’d rather use Assemblytics for the visual representation as uppose to sniffles which just gives me a vcf output.
Cheers

Assemblytics is for assemblies only, so it's not a good fit for your analysis, sorry!
NGMLR+Sniffles is the right thing to do, so I would recommend looking at visualization after the Sniffles step. Again, Biostars would be a good place to ask for recommendations about visualizing your VCF file.


Added SAPP + pathing.

Added "steam integration", but without the steam api. It finds the P2CE install directory and points to it, the config will be generated based on OS, though, obviously, the tooling for Linux doesn't work out of the box (because I cannot guarantee wine is installed.). The config now has a identifier ${INSTALLDIR} which is replaced at runtime with the full path to P2CE's install location. If you're concerned about the workshop uploader requiring the actual steam API, it'd be better if the workshop uploader didn't rely on the SDK Launcher to function. SAPP does not require steam to be running, thus allowing the SDK Launcher to be used offline. I plan to add more functionality soon related to improving the UX of the config file itself. But that's a separate item on my list. I hope this is good enough for a merge.
would resolve #3

All issues have been resolved, anything else that needs to be done?


Circular dependency in v4 causes problems with Webpack

Describe the bug
Reference requires wrap, then wrap immediately requires Reference.
This is a circular dependency, which is undefined behaviour.
This seems to work on vanilla Node, but in a Webpack context, Reference doesn't correctly get set to the same object everywhere, resulting in TypeError: Cannot read property '__meta' of undefined. This is because wrap doesn't see that the entity is an instanceof Reference, resulting in it falling through wrap without doing what's intended.
I added this to wrap:
    console.log('entity: ', entity);
    console.log('entity instanceof Reference_1.Reference', entity instanceof Reference_1.Reference);
    console.trace();
    console.log('--------------------');

Then called findAndCount once. Both the successful output and the error output happen from the same call to findAndCount.
Successful output:
entity:  Reference { entity: Location { id: '4' } }
entity instanceof Reference_1.Reference true
Trace:
    at Object.wrap (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/wrap.js?:17:13)
    at Function.autoWireOneToOne (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js?:65:30)
    at ObjectHydrator.hydrateManyToOne (webpack:///[project dir]/node_modules/@mikro-orm/core/hydration/ObjectHydrator.js?:79:37)
    at ObjectHydrator.hydrateProperty (webpack:///[project dir]/node_modules/@mikro-orm/core/hydration/ObjectHydrator.js?:10:18)
    at ObjectHydrator.hydrate (webpack:///[project dir]/node_modules/@mikro-orm/core/hydration/Hydrator.js?:20:18)
    at EntityFactory.create ([project dir]/node_modules/@mikro-orm/core/entity/EntityFactory.js:27:27)
    at SqlEntityManager.merge ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:258:80)
    at SqlEntityManager.find ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:78:33)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Promise.all (index 0)
    at SqlEntityManager.findAndCount ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:133:35)
(our code)
--------------------

Unsuccessful output:
entity:  Reference { entity: Organisation { id: '2' } }
entity instanceof Reference_1.Reference false
Trace:
    at Object.wrap (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/wrap.js?:17:13)
    at new Reference (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/Reference.js?:9:32)
    at Function.create (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/Reference.js?:29:16)
    at Function.wrapReference (webpack:///[project dir]/node_modules/@mikro-orm/core/utils/Utils.js?:409:39)
    at Shop.set [as organisation] (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/EntityHelper.js?:99:56)
    at Function.assignReference ([project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:89:31)
    at [project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:33:39
    at Array.forEach (<anonymous>)
    at Function.assign ([project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:20:27)
    at SqlEntityManager.merge ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:261:33)
    at SqlEntityManager.find ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:78:33)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Promise.all (index 0)
    at SqlEntityManager.findAndCount ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:133:35)
(our code)
--------------------

Stack trace
TypeError: Cannot read property '__meta' of undefined
    at new Reference (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/Reference.js?:10:17)
    at Function.create (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/Reference.js?:29:16)
    at Function.wrapReference (webpack:///[project dir]/node_modules/@mikro-orm/core/utils/Utils.js?:409:39)
    at Shop.set [as organisation] (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/EntityHelper.js?:99:56)
    at Function.assignReference ([project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:89:31)
    at [project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:33:39
    at Array.forEach (<anonymous>)
    at Function.assign ([project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:20:27)
    at SqlEntityManager.merge ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:261:33)
    at SqlEntityManager.find ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:78:33)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Promise.all (index 0)
    at SqlEntityManager.findAndCount ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:133:35)
(our code)

To Reproduce
I will provide a reproduction case, but I haven't had time to yet and wanted to get the issue some visibility so we can work towards eliminating circular dependencies in the project in general.
Expected behavior
Expected no circular dependency to exist as it's undefined behaviour.
Versions

Dependency
Version

node
12

typescript
3.9.5

mikro-orm
4.0.0-alpha.6

your-driver
Postgresql

I should note that this might not be caused by the circular reference itself, it might actually be because in one we're in the middle of a constructor, while in the other the Reference is already created. Either way it'd be a good idea to avoid circular dependencies.

We can remove the condition in wrap as unwrapReference is already taking care of that.

Can you verify it helps?
Btw I am quite sure the reason is that Reference_1.Reference is actually undefined when it fails.

Yeah, I suspect you're right, I'll give the latest a shot. There's still a cycle (Reference -> wrap -> Utils -> Reference) so it might not make much difference, but will verify before we worry about changing things.
(Note for later, if the change above doesn't fix it, could wrapReference and unwrapReference actually go on the Reference class? That should fix the cycles.)

Try alpha 7, that condition is now gone.

(Note for later, if the change above doesn't fix it, could wrapReference and unwrapReference actually go on the Reference class? That should fix the cycles.)

Yeah sure, makes sense.

FYI if we move the methods to Reference wrapper, we will still have Reference imported in wrap and vice versa.
What we could do is to define a __helper getter on Reference class so we do not have to do this at all (we can then remove the unwrapReference from wrap). This way we can even remove the usage of Reference from Utils, as we could use this fact to verify if given object is entity.
Overall the circular dependencies are haunting me from the very beginning :]
I guess I will do that right ahead, it makes sense.

Same result in alpha 7 unfortunately:
TypeError: Cannot read property '__meta' of undefined
    at new Reference (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/Reference.js?:10:17)
    at Function.create (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/Reference.js?:29:16)
    at Function.wrapReference (webpack:///[project dir]/node_modules/@mikro-orm/core/utils/Utils.js?:409:39)
    at Shop.set [as organisation] (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/EntityHelper.js?:99:56)
    at Function.assignReference ([project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:89:31)
    at [project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:33:39
    at Array.forEach (<anonymous>)
    at Function.assign ([project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:20:27)
    at SqlEntityManager.merge ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:261:33)
    at SqlEntityManager.find ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:78:33)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Promise.all (index 0)
    at SqlEntityManager.findAndCount ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:133:35)
(our code)

I made Utils.unwrapReference look like this:
    static unwrapReference(ref) {
        console.log('ref: ', ref);
        console.log('entity_1.Reference: ', entity_1.Reference);
        console.log('ref instanceof entity_1.Reference', ref instanceof entity_1.Reference);
        console.trace();
        console.log('--------------------');

        return Utils.isReference(ref) ? ref.unwrap() : ref;
    }

It showed the same behaviour in that if it comes from ObjectHydrator it works, but if it comes from new Reference it doesn't. entity_1.Reference is definitely initialised to something at that point:
ref:  Reference { entity: Organisation { id: '2' } }
entity_1.Reference:  [Function: Reference]
ref instanceof entity_1.Reference false
Trace:
    at Function.unwrapReference (webpack:///[project dir]/node_modules/@mikro-orm/core/utils/Utils.js?:328:17)
    at Object.wrap (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/wrap.js?:18:26)
    at new Reference (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/Reference.js?:9:32)
    at Function.create (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/Reference.js?:29:16)
    at Function.wrapReference (webpack:///[project dir]/node_modules/@mikro-orm/core/utils/Utils.js?:415:39)
    at Shop.set [as organisation] (webpack:///[project dir]/node_modules/@mikro-orm/core/entity/EntityHelper.js?:99:56)
    at Function.assignReference ([project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:89:31)
    at [project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:33:39
    at Array.forEach (<anonymous>)
    at Function.assign ([project dir]/node_modules/@mikro-orm/core/entity/EntityAssigner.js:20:27)
    at SqlEntityManager.merge ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:261:33)
    at SqlEntityManager.find ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:78:33)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Promise.all (index 0)
    at SqlEntityManager.findAndCount ([project dir]/node_modules/@mikro-orm/core/EntityManager.js:133:35)
(our code)

I then used yarn link to try latest dev and got:
MetadataError: Metadata for entity Reference not found
    at Function.missingMetadata (webpack:////Users/kevin/development/mikro-orm/packages/core/dist/utils/errors.js?:133:16)
    at MetadataStorage.get (webpack:////Users/kevin/development/mikro-orm/packages/core/dist/metadata/MetadataStorage.js?:37:41)
    at UnitOfWork.cascade (/Users/kevin/development/mikro-orm/packages/core/dist/unit-of-work/UnitOfWork.js:319:36)
    at UnitOfWork.cascadeReference (/Users/kevin/development/mikro-orm/packages/core/dist/unit-of-work/UnitOfWork.js:331:25)
    at UnitOfWork.cascade (/Users/kevin/development/mikro-orm/packages/core/dist/unit-of-work/UnitOfWork.js:321:18)
    at UnitOfWork.merge (/Users/kevin/development/mikro-orm/packages/core/dist/unit-of-work/UnitOfWork.js:42:14)
    at SqlEntityManager.merge (/Users/kevin/development/mikro-orm/packages/core/dist/EntityManager.js:262:30)
    at SqlEntityManager.find (/Users/kevin/development/mikro-orm/packages/core/dist/EntityManager.js:78:33)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Promise.all (index 0)
    at SqlEntityManager.findAndCount (/Users/kevin/development/mikro-orm/packages/core/dist/EntityManager.js:133:35)
(our code)

(We have no entity called Reference.)
I'll create a reproduction so you can run this locally.

Ok, here's the reproduction: https://github.com/thekevinbrown/mikro-676
It looks like it actually reproduces two things:

The __meta issue.
But also the long time for MikroORM.init() to run. With two entities that have 2 or 3 properties each, using ReflectMetadataProvider, init() is taking more than a second to run on my relatively modern MacBook Air, which feels like it's longer than it should be to me.

I added profiling to the reproduction. It looks like about half of the time in init is spent in highlight.js, the other half is spent in postgres getting a connection, and the rest is super tiny. You can see a flamegraph on the repo, or can generate one locally with yarn profile now.

Regarding the coloring, it is just about the highlighting, right? Not about the chalk usage I hope (now replaced with ansi-colors that should be much faster). It can be already disabled via configuration (from the very beginning of this addition), does that help? You can also disable coloring via process.env.FORCE_COLOR = 0.

So closing this as #679 is merged. Let's continue the coloring discussion in #658.


Functionality for compression

Hi, I'm using the MySeriesHelper example for inserting batch inserts in the db.. However, since its over the network, its slow and is blocking the main program.. Is there a way to compress the data before sending over the network..? Any recommendations?

Rereading the InfluxDB documentation, they recommend larger writes for better performance:

We recommend writing points in batches of 5,000 to 10,000 points. Smaller batches, and more HTTP requests, will result in sub-optimal performance.
It appears as though there was support at some point for gzip reading and writing (https://github.com/influxdata/influxdb/issues/1449) but I can't find any API references to those features now.

closing due to inactivity


Should an endpoint defined with an async function be requested using an async request library ?

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
No example code

Description
Suppose I define an endpoint with an async function:
...

@app.get("/endpoint")
async def f(...):
   ...

...

I noticed I can request it using the standard python requests library and it doesn't seem to cause problems.
Can it cause problem in some situation ? Should I use an async request library ? Am I missing some concept about async programming ?
Operating System
Linux
Operating System Details
No response
FastAPI Version
0.73.0
Python Version
3.9.5
Additional Context
No response

How u do the request does not depend on the server, only on how u want the application that is talking to the server to work.
If u have a simple script that needs the response of the server there is not much point in using an async requests library.
On the other hand, there can be advantages if u want to do multiple I/O related things at the same time.
If you have a script that needs to do a request and read a file, then u could benefit from using asyncio libraries so that u can do the request and start loading the file whilst you are waiting on the network response.
Either way this issue is not related to the implementation of a fastapi application.

@samv-yazzoom But do I gain some benefit (in terms of async programming) if the endpoints are defined as async functions and the requests are made through the standard python requests library ? I mean, can the fact that I am using a non-async request library kills the benefits of defining the endpoint functions as async ?

@samv-yazzoom Thanks, I believe I understand it now. Closing the question


Added parser and formatter props to input and input_number components

This PR tries to resolve issue https://github.com/thaw-ui/thaw/issues/129 by adding optional callbacks to Input and InputNumber components. The parser callback converts user input to the desired signal format, while the formatter callback presents the signal value to the user. You can check it working in the Docs.
Following the suggested implementation I've added the parser to the onchange event. There was the need to clone the parser in order to be consumed by two closures. Please, feel free to give advice on how to avoid the cloning.
Thank you!

Thank you for your contribution.
It looks good. Wouldn't it be nice to change InputNumber's formatter type to Callback<T, String> and parser type to Callback<String, T>?

Please, feel free to give advice on how to avoid the cloning.

Implementing Copy for OptionalProp solves this problem.
impl<T: Copy> Copy for OptionalProp<T> {}

I tried using callbacks with generic types, but InputNumber calls the Input component under the hood, which lacks a generic type. Next, I attempted to implement a generic type for the Input component but haven't found a solution yet. The easier approach would be to avoid using the Input component inside InputNumber, but I don't consider this an elegant alternative, so I'll continue exploring other options...

I don't think you need to implement generics for Input, just:
fn InputNumber(parser, formatter) {
   let parser = parser.map(|parser| {
        Callback::new(move |v| parser.call(v).to_string())
   });
   let formatter = formatter.map(|formatter| {
        Callback::new(move |v| formatter.call(v.parse::<T>()))
   });
   view! {
      <Input parser formatter/>
   }
}

Oh, is it okay to create a new callback? I thought it might introduce too much overhead. I was considering the possibility of using other Rust types with Input, such as std::net::IpAddr, in a similar manner to how InputNumber can hold an i32 or f64.
Anyway, I am happy with your suggestion; it is simpler, and we can move forward. I'll make the changes tonight.
Thank you for your kindness in helping me out.

The suggested changes have been made. Let me know if you have any other suggestions.

Thank you!


Update rstudio to 1.0.44

After making all changes to the cask:

[X] brew cask audit --download rstudio is error-free.
[X] brew cask style --fix rstudio reports no offenses.
[X] The commit message includes the cask’s name and version.

Could you please add the depends_on cask: 'r-app' stanza?

I'm not familiar with how cask dependencies are supposed to be like, but how would we deal with brewed R package in this case? For instance, I have R package installed by brew and rstudio works well with no dependency required for r-app.

Not sure about that - I was under the impression r-app Cask was the only version of R in Homebrew. Pinging @vitorgalvao.

brew install homebrew/science/r gives you a brewed version of R.

You are correct @tomyun - the website says that R is required, so I think it needs a depends_on stanza, but I am not sure which one to use.

If any r is going to be required, it will likely be the homebrew/science one. I don’t remember ever seeing a dependency on an app bundle.
But are we sure this does even need the CLI installed?

Their download page states that "RStudio requires R 2.11.1 (or higher)" and links to the download website for R.

@miccal Then it should have a depends_on formula: 'homebrew/science/r'.


3d package reference for IPC7351-Nominal.pretty/SOP65P640X120-20

This PR just links the ${KISYS3DMOD}/smd_dil/tssop-20.wrl model. This was achieved by modifying config/3dmap (first commit) and then running make ipc (second commit).

Done. The "make ipc" script is obsolete - I'm not sure if it gave you any trouble, I expect it probably modified a few files you didn't intend to touch as well. In the future feel free to just modify the footprints directly. A replacement is planned.

Thanks. Those 3 files were the only ones that got modified. I kind of liked the scripted approach -- kind of easier than editing the file directly.

The script is nice. The problem with ti is that there's no way to edit the libraries to add parts that aren't in the original ones the script uses as the source for conversion.


Make plot URLs more easily hackable

If I am given a link to https://probelab.io/websites/blog.ipfs.tech/#website-snapshot-performance-gauge-blogipfstech-kubo , in order to quickly "hack" the URL for another site, I need to change both the path AND the anchor.  I'd ideally only like to change the path.
-https://probelab.io/websites/blog.ipfs.tech/#website-snapshot-performance-gauge-blogipfstech-kubo
+https://probelab.io/websites/blog.ipfs.tech/#website-snapshot-performance-gauge-kubo

For example, if I want to go from blog.ipfs.tech link to docs.ipfs.tech I only have to change one thing that way:
-https://probelab.io/websites/blog.ipfs.tech/#website-snapshot-performance-gauge-kubo
+https://probelab.io/websites/docs.ipfs.tech/#website-snapshot-performance-gauge-kubo

@BigLep can you confirm you only see this as a problem in the website section, because the name of the website is duplicated?

@iand : yeah, I only noticed in in the "website" section.  Thanks for the fix!


Question: C++ an option?

Is it an option to change the codebase to c++11 over time?

Most of my projects are straight C. I wouldn't like to see this changed to C++.

It is basically c++ destructors, that I miss. Every now and then I find memory leaks in wakaama. And it really is quite challenging to handle object lifetime in C.
What exactly is your objection? Just keeping the C interface, having a c++ tool chain for a part of your project, overhead considerations or something else?

The C interface is a major thing, but there are other considerations also. First, the IAR compiler we use only supports ISO/IEC 14882:2003 C++, not C++11.
Just adding classes would be less objectionable than full C++. Some features can increase RAM/Flash requirements and slow processing. We're currently targeting an STM32L4 with 128k RAM and 512k flash running at 8MHz. Others may be targeting even smaller systems. Such resource usage is a significant concern.
Having destructors doesn't make C++ immune from memory leaks. If an object is created with new and never deleted it will still leak memory.
Speaking of new and delete, it can make memory management more challenging. Right now we have our own lwm2m_malloc and lwm2m_free that allocate from pools of fixed size buffers to avoid memory fragmentation. Ideally nothing in the system would use dynamic memory but I think we're beyond the level of complexity where that is possible and this is the compromise we've made.

New and delete can be custom:
#include <cstdio>
#include <cstdlib>
// replacement of a minimal set of functions:
void* operator new(std::size_t sz) {
    std::printf("global op new called, size = %zu\n",sz);
    return std::malloc(sz);
}
void operator delete(void* ptr) noexcept
{
    std::puts("global op delete called");
    std::free(ptr);
}
int main() {
     int* p1 = new int;
     delete p1;

     int* p2 = new int[10];
     delete[] p2;
}

C++ is not immune to leaks of course. But if an object is part of another object, on destruction the inner objects free their resources. That's where it requires a lot of care with C and some object/memory ownership and lifetime tracking.
The last memory leak I found was again a duplicated c-string that was not freed on the outer objects destruction.
I'm using a static memory block as well, and I encounter leaks if connection attempts are interrupted in between. At the moment I'm just zeroing the memory block, and reinitialise wakaama. But that's not how it should be.
I have to admit that without c++11's unique pointer and constexpr, it's less elegant to use c++.

I must admit I wasn't familiar with a custom new and delete. Would it be possible to do them custom just for Wakaama, or is it global?

The above form is global. Additionally c++ supports

per object new/delete operators,
custom allocators (within the standard namespace std::string and std::vector are examples for classes that take an allocator as template parameter),
placement new:

char* ptr = lwm2m_alloc[sizeof(T)]; // allocate memory
T* tptr = new(ptr) T;               // construct in allocated storage ("place")
tptr->~T();                         // destruct
lwm2m_free(ptr);                    // deallocate memory

@davidgraeff Thanks for referring this topic to me.
Actually we have been working with Wakaama quit a few years. We have our  devices designed with our own lwm2m stack and use the wakaama for server side software to talk with the devices.
To be frankly, we are very opposite to this idea. Sometimes it is  hard to maintain the memory allocation/free, but with appropriate design pattern and tools like valgrind, it can be done. And it has to be done to build a reliable product.
In the current wakaama design, memory freeing is largely triggered by external messages and timers. Some memory issues can't be simply solved by C++ destruction mechanism. Introducing the C++ mechanism will make it even much harder to troubleshoot those touch memory leak issues (like by the complexity of status management).
The memory free issue in the prv_getParameters() is actually easy to fix. it should should be fixed by diligence, rather than a "lazy" solution.

Should we close this discussion ?

Yes I think so :)


[Runtime] Support type descriptor map in LibPrespecialized.

The descriptor map is keyed by a simplified mangling that canonicalizes the differences that we accept in _contextDescriptorMatchesMangling, such as the ability to specify any kind of type with an OtherNominalType node.
This simplified mangling is not necessarily unique, but we use _contextDescriptorMatchesMangling for the final equality checking when looking up entries in the map, so occasional collisions are acceptable and get resolved when probing the table.
The table is meant to be comprehensive, so it includes all descriptors that can be looked up by name, and a negative result means the descriptor does not exist in the shared cache. We add a flag to the options that can mark it as non-definitive in case we ever need to degrade this, and fall back to a full search after a negative result.
The map encompasses the entire shared cache but we need to reject lookups for types in images that aren't loaded. The map includes an image index which allows us to cheaply query whether a given descriptor is in a loaded image or not, so we can ignore ones which are not.
TypeMetadataPrivateState now has a separate sections array for sections within the shared cache. _searchTypeMetadataRecords consults the map first. If no result is found in the map and the map is marked as comprehensive, then only the sections outside the shared cache need to be scanned.
Replace the SWIFT_DEBUG_ENABLE_LIB_PRESPECIALIZED environment variable with one specifically for metadata and one for descriptor lookup so they can be controlled independently. Also add SWIFT_DEBUG_VALIDATE_LIB_PRESPECIALIZED_DESCRIPTOR_LOOKUP which consults the map and does the full scan, and ensures they produce the same result, for debugging purposes.
Remove the disablePrespecializedMetadata global and instead modify the mapConfiguration to disable prespecialized metadata when an image is loaded that overrides one in the shared cache.
rdar://113059233

Pushed a fix for a spot where we were accessing data->getOptionFlags() where data could be NULL.

Updated with a smallish change to consult the map when searching for protocol descriptors.

@swift-ci please test

Linux failed due to issue addressed by https://github.com/apple/swift-foundation/pull/802.

@mikeash this PR causes a build failure for me on linux on main due to a missing #include <utility> for the use of std::pair.
Presumably due to recent libc++ granularized header work?


JsCreateExternalObject returns ArgumentNotObject

We have some code that calls JsCreateExternalObject (through the .NET wrapper) to get a reference to external C# objects. It works fine most of the time, but it looks like one time it returned ArgumentNotObject. Is there any valid scenario where this error would be returned from this method?
I looked at the implementation, and it seems this error can be returned from VALIDATE_INCOMING_OBJECT in JsCreateExternalObjectWithPrototype, but the prototype value should be JS_INVALID_REFERENCE, and this check shouldn't be executed. Could the error be related to the data parameter that we are passing in?
The version of ChakraCore is 1.8.4.

If you are using ChakraCore 1.8.4 then you're actually using an implementation closer to this: https://github.com/Microsoft/ChakraCore/blob/release/1.8/lib/Jsrt/Jsrt.cpp#L1291
I can't see any obvious reason that you'd get JsErrorArgumentNotObject from that function. Are you sure it was that and not an adjacent call which returned the error?

Thanks, @MSLaguana . I think it could also be coming from the following line that sets the prototype (I guess the line numbers in the call stack might be a little off due to optimizations etc.):
        public JavaScriptValue CreateJsObject(object obj)
        {
            var jsObject = JavaScriptValue.CreateExternalObject(obj.CreateIntPtr(), null);

            jsObject.Prototype = Prototype;

            return jsObject;
        }

And the prototype object is created as Prototype = JavaScriptValue.CreateObject(); (and then setting some properties on this Prototype) before the above code executes.

That also looks like it should work, unless the .net wrapper is doing strange things. JsCreateExternalObject should always result in a JsrtExternalObject, which is a valid object, and similarly JsCreateObject should always result in a valid object, so calling JsSetPrototype(jsObject, Prototype) should also always work. Is it possible that you have a race condition such that your Prototype isn't initialized properly?

I don't think there is a race condition since the Prototype is basically initialized at application startup and CreateJsObject calls are made later. I'm also checking if there could be some garbage collection happening leading to this state (unlikely)... I'll also add some logging to get Prototype.ValueType right before using it.

Actually garbage collection might be a likely candidate... After debugging this further I noticed that sometimes JsSetPrototype throws AccessViolationException, and from this thread any object created with JsCreateObject should be "pinned" by calling .AddRef(), unless it's used explicitly by the JS runtime (placed on some JS call stack etc.). So right after creating the Prototype I need to call Prototype.AddRef(). Is that correct?
And the final question is, could the JsErrorArgumentNotObject error have also been caused by the JS value being garbage collected?

Right; whenever you hold onto a reference to a javascript object in a way that chakracore can't detect (basically, if it isn't on the stack or referenced by another object known to chakracore) then you need to explicitly add a reference to it (and then later dereference once you're done with it, to avoid leaking memory). If you fail to do that, it may be considered unreferenced and then garbage collected out from underneath you, leading to unpredictable and bad behavior later on.
However, I'm not sure whether that would cause the JsErrorArgumentNotObject, since that check is based on looking at the pointer only. Some pointers are tagged and treated as integers or floats, and the check that can return that error is only checking for that tag. If the tag is not present, then we go ahead and treat it as though it is a pointer to one of our internal objects and potentially dereference it, and that could cause crashes or other bad things, but I don't expect that it would return that particular error...

Ah, there's one path that I hadn't considered and could result in that: the check doesn't just check for tagged values, it does go and try to fetch the internal type of the object, and so if that type happens to be a small number (< ~20) then it could cause that error. So it may be that the object was collected, something else allocated in its place, and after following a chain of bogus pointers the value it read was small or negative.

Great, thank you for confirming that it could happen. I'll try the .AddRef() fix, hope it solves the problem.

Feel free to comment back if you have any further questions and we will reopen the thread.


Fix recycled containers deleting items inside them

About the PR

Right now you can delete any item that fits in a survival box by placing it inside of one and then recycling the box in a Recycler/Material Reclaimer. This PR fixes this by dumping out any item inside containers.

For Reclaimers,  any recyclable items inside the container get placed on top of the Reclaimer.
For Recyclers, any recyclable items inside the container get recursively recycled as well.

Why / Balance

I doubt it's intended behavior to be able to effectively delete any item that fits in a box. Also ensures a recycled box provides the correct amount of material as its contents also get recycled.
Technical details

When trying to recycle, once all checks are passed it iterates through the object's containers if it has any and dumps out any items onto the machine doing the recycling.
Media

The result of recycling a survival box in a Recycler/Reclaimer.

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: Containers no longer delete the items inside when recycled.

 It just feels like a kinda jank and unintended way to get rid of evidence, plus it provides little to no interesting gameplay for either side.

How am I gonna destroy all my contraband after I complete my objectives as thief now

tell the chaplain you found 4 among pequeno and you must personally incinerate them

How am I gonna destroy all my contraband after I complete my objectives as thief now

But anyway I think this is actually a good change since det has more evidence to check now (unless they cremated it)

This is the first time I'm making something new for the event system so let me know if there's something iffy with the implementation. Behavior seems to be replicated and working fine though.


Asserting changes type of original variable inline (the change reverts in subsequent lines)

🔎 Search Terms
asserting type
as changing underlying type
🕗 Version & Regression Information

This changed between versions v4.2 and v4.3

⏯ Playground Link
https://www.typescriptlang.org/play?ts=5.4.3#code/MYewdgzgLgBAHjAvDA3gWAFAxgcgJ44BcuAFgKYA2FIOANJtjnETAIwAMmAvppgGYBXMMCgBLcDAggAtmQBiQkeLAAeACowycKGTAATCDADWZPCD4woeAA5lz8AHwAKawCcQ1wmoCUqBpIB3UShgEhgXd2tfdCxsGGAAQwgyXAJCfziYAHosmATYKBJRQ2sQUTBYNw8YYskxKhgAIjxGmAAfJrhWvQEUqBBLGzIIYFdRa1gwEFgAIzJygHM8mYo+gbAE13cAjLiczQgKcthXAVWmqYBaITJgYYhNvEurW0uk5NcxcFa3MgA3XRQQwbLYgIJgJYgASuZIUAEQXbYUCQE6sJAwKrWADciOyuUymQAegB+XHI6AwVwAJnRmLyhnwOBxsUy+wJcRJuJmrjICSMzOwPAw3EwQA
💻 Code
const x = {
  'y': 'hello',
  'x': 10
}

function someFunction<T extends keyof typeof x>(prop:T) {
  switch (prop) {
    case 'y':
      // at this point prop is still "y" | "x" due to typescript not being able to narrow
      // eslint rule "no-unecessary-type-assertion" prevents narrowing ourselves
      const r1 = prop;
      //         ^?
      const r2 = prop as 'y';
      //         ^?
      break;
  }

}

🙁 Actual behavior
If you inspect the types of prop with and without the assertion you can see that without the assertion TS reports the type of prop to be T  extends "y" | "x" but with the assertion TS reports the type of prop as "y".

This is a bug with TS itself - a behaviour changed in TS v4.3. In v4.2 TS correctly reports both locations as the generic - in v4.3+ it exhibits the above behaviour.
🙂 Expected behavior
The assertion should not be changing the type of the original variable.
Additional information about the issue
Discovered while debugging a eslint issue .

Note that the type of prop is not actually changed. If you look at the type in the next line it's still correctly T extends "y" | "x".
So it's at most an issue with the type shown in the tooltip.

Duplicate of #43873

We need to retain basically both the T and the "y" aspects of the type here, otherwise code like this fails:
function someFunction<T extends keyof typeof x>(prop:T): T {
  switch (prop) {
    case 'y':
      return prop;

Note that
const r1: 'y' = prop;

also doesn't error.
The "correct" type is both T and "y" depending on what you need to use it for. If the contextual target is a union we treat it as "y", otherwise it's T. So neither display is "wrong", but we can only show one at a time to the user obviously.

The issue is that from typescript-eslint's perspective for the rule no-unnecessary-type-assertion we inspect the left and right side of the assertion and compare the types to make a guess if they are the same type. If they're the same type then we error and flag the assertion as unnecessary.
The way we do this is in the most basic form - "are the type objects the same?" Not even doing any fancy type assignability checks - it all just boils down to if (left === right) report().
So in this particular case when we inspect the left we don't see the type T - we see the type "y". Obviously this is the exact same type as the right so we report.
There's no flags set on the left type either so we can't even guess that the left might be the generic type. The contextual type too is the same. Is there another API we can use to get the type of the left without the assertion here (eg see that it is the generic type)?

To clarify - it's not just what you show to the user - this is not a problem of intellisense has to display one so we pick the best one via some logic.
Instead it's what the type API itself reports for the expression. When there is no assertion it reports the type T. When there is an assertion it reports the type "y".

I suppose there'd be some issue if it became T & "y"?
function someFunction<T extends keyof typeof x>(prop: T) {
    if (prop === "y") {
        x[prop].toUpperCase(); // error
        const p: "y" = prop;
        x[p].toUpperCase(); // okay
        const q: T & "y" = prop; // error!
        x[q].toUpperCase(); // this would have been okay
    }
}

@jcalz my using of a switch in the example was probably a mistake. I don't think this is a duplicate of #43873 because the problem is not typescripts inability to narrow for extend but that it picks and choses in the same code block about whether it is narrowed (in a way that doesn't let us override without triggering reasonable eslint rules).
@RyanCavanaugh needing to keep the return type broad to enable returing T makes sense
IF statement example:
ES Lint Playground


What are the differences between the P4-DPDK Target and BMv2?

The biggest difference here lies in whether the former utilizes DPDK, or are there other distinctions as well?

If you do not mind, I would recommend asking this question in the public web forum https://forum.p4.org, because such questions are a searchable resource for such questions (and often answers, too).

I would recommend closing this issue (I do not have permission to do so on this repo), as it is discussed here: https://forum.p4.org/t/what-are-the-differences-between-the-p4-dpdk-target-and-bmv2/951


Model intersection syntax

Describe the feature
Currently, when you provide two args to the model selection syntax, dbt selects the union of the two parts of the graph:
e.g.
dbt run --models tag:hourly tag:marketing

would run all models that have either the hourly or marketing tag (or both)
I'd like a way to only run the intersection of these two parts of the graph:
dbt run --models [tag:hourly ∩ tag:marketing ]

That would only run models that have both the hourlyAND marketing tag. And of course ∩ is a terrible symbol to use here.
Describe alternatives you've considered
We've seen this come up a few times, normally when people want to run the models in a folder that have a certain tag. Our current advice is to add new tags.

Let's prioritize this for the Octavius Catto release. We should use a , to separate selectors, eg:
dbt run --models tag:abc,tag:def tag:ghi,jkl

This will select models that have:

tags abc AND def, OR
tags ghi AND jkl

do you mean:
dbt run --models tag:abc,tag:def tag:ghi,tag:jkl

If we are going to use the array syntax can I suggest we use the bash array syntax instead? This would make it easier to populate variables and use them later as tags:
TAGS=(tag:abc tag:def)
dbt run --models $TAGS[@] (tag:ghi tag:jkl)

Anyone who's in a rush solving this (myself included) can use a bit of shell and jq to solve this when invoking dbt run.
dbt run -m $(dbt ls --resource-type model --output json | jq '. | select(.tags | index("abc")) | select(.tags | index("def")) .name' -r | tr '\n' ' ')

@Raalsky I think you indicated that you were interested in picking this up -- is that still the case? If not, we're ready to prioritize this on our end :)

@drewbanin Sorry for being off. Yeah, I'm still interested. I'll post updates after the weekend, ok?


Chinese translations and other small fixes

Added the missing Chinese translations into the UI.
Would use the list of strings in the Chinese file as a starting point for the other languages as it has the most complete list of strings currently.

LGTM 🚀


How to add three condition in Rule for one Conclusion

I´d like to know how to add three conditions for one conclusion?
fuzzy.createRule(term1, a3, term2, b1, term3, c1, result, d1);

what type of fuzzy that used in this library..i mean like fuzzy mamdani, or sukamoto?,


Helper functions are positioned in between class and jsdoc comment

Describe the bug
If you have a doc code for a class, helper functions/import statements will be positioned in between doc code and class (see playground link).
To answer the question upfront why this matters: One potential use of swc for me would be to transpile the source code before running an old program to generate our doc codes from this.
Input code
/**
 * This is just a doc code for the class Foo.
 */
class Foo {
    async bar() {
        return 'foo';
    }
}

Config
{
  "jsc": {
    "parser": {
      "syntax": "ecmascript",
      "jsx": false
    },
    "target": "es5",
    "loose": false,
    "minify": {
      "compress": false,
      "mangle": false
    },
    "preserveAllComments": true
  },
  "module": {
    "type": "es6"
  },
  "minify": false,
  "isModule": true
}

Playground link
https://play.swc.rs/?version=1.3.23&code=H4sIAAAAAAAAAz3KQQqAIBSE4b3gHWZXucgDtO8EXcDUqAgf%2BHQR0d1Ton5m9TFaKSmgMK0bo2zPnGDgyMKS81goIq0e9jDMGIn6etdS%2FICrSMnwGSxmE9vuo1r0KceAZiFqhpdvKe4HI9b%2FKXgAAAA%3D&config=H4sIAAAAAAAAA0WNOwrDMAxA76LZaztkK517COEqwcU%2FJKXUhNy9VjB4E3pPTwd8xMNyQEUWYpukZcUfLEA%2BoXgOVcF1ra9WjEKnA0XeSE2RW2exFKFBHaSQw9qs5EuqTCITYd4izY5R4i89YnyWlChrd5V3Y6m8d3MP0Fbp%2BnWHc%2BZHMshriNfdHzhZ%2FNXQAAAA
Expected behavior
I would expect the helper functions to be generated above the doc code. I.e. in the order helper functions, followed by the doc code, followed by the class.
Actual behavior
No response
Version
1.3.23
Additional context
No response

This is generated code, so a lot of things can be ugly. Why you need it? Can you have real use case why it is bad for you?

As I've said before:
We need to transpile the source code before using the transpiled output to generate doc codes from.
Yeah for sure the transpiled output is ugly but in this case it matters where the helper functions are placed.

Closing as not planned


(Desktop mode) Amazon launcher can't enter login info

Currently only tested in desktop mode the Login screen on the Amazon Games Launcher clicking or tapping (on screen 😅) on either of the text fields brings up the steam keyboard but also deselects the text field. Reselecting field causes the keyboard to disappear. External keyboard doesn't change the the issue. Not sure what is causing it but can't enter login info because this is all that happens text field refuse to stay selected doesn't seem like it is more than a possible glitch or bug with wine/proton but im reporting it just in case on here.

@DarkenLX im going to close this because this has nothing to do with this app. This is a steam bug and is present in many cases. The keyboard sometimes cant open when proton is being ran with steam because of the overlay... thats something out of my control.


No such file or directory when building docs

Version: nbdev==2.2.10
When running nbdev_docs from the root of the nbdev project, everything seems to process smoothly, then there is an exception thrown from a rename operation. Most of the notebooks listed below have the skip_showdoc: true frontmatter as the first cell.
The only notebooks without skip_showdoc: true are 00_data.structures.misc and index.ipynb.
Notebooks are placed in <root>/nbs.
Output log (modified for brevity):
pandoc -o README.md
  to: gfm+footnotes+tex_math_dollars-yaml_metadata_block
  output-file: index.html
  standalone: true
  default-image-extension: png

metadata
  skip_showdoc: true
  title: nbdevtest

Output created: _docs/README.md

[ 1/4] 00_utils.logging.ipynb
[ 2/4] 00_data.structures.misc.ipynb
[ 3/4] 00_data.structures.core.ipynb
[ 4/4] export_test.ipynb
[ 5/5] index.ipynb

ERROR: NotFound: No such file or directory (os error 2), rename '/opt/github/test_nbdev_v2/nbs/data.structures.core.html' -> '/opt/github/test_nbdev_v2/nbs/_docs/data.structures.core.html'

nbdev config:
...

### nbdev ###
nbs_path = nbs
doc_path = _docs
recursive = False
tst_flags = notest

...

nbdev v2 looks like a fantastic upgrade to v1 so we are keen to get it working. Any help would be appreciated.
Cheers

Updating to nbdev==2.3.3 resolved this issue.

This is back with 2.3.12


Modal Not Being Populated Or Updated

Loading failed for the

https://github.com/Aureum01/RateMyProfessorsStatic/assets/71108402/ab1e63f7-7359-4225-8a39-c80fb8c70af2

Issues from ProfessorList.razor, ProfessorList.js, HomeController.cs (MVC). Check names.json for structure and professors.cs in models for insight.


Crashing all time

Prerequisites
Safe mode working
but i can not open my project file
I deleted my all packages and using only "remote ftp" but still crashing.
Description
I've been using the atom editor for a long time. Facebook started crashing while making php login system. It was after installing the Facebook SDK.
Steps to Reproduce

open
3 character writing
crash
repeat

Versions
Atom    : 1.32.2
Electron: 2.0.9
Chrome  : 61.0.3163.100
Node    : 8.9.3

Thanks for the report -

Safe mode working
but i can not open my project file
I deleted my all packages and using only "remote ftp" but still crashing.

If there's no problem in safe mode, then that means a community package or possibly something in your init script is causing the issue.  Since remote-ftp is the only community package you left installed, if you disable it does the crashing stop?
Going to go ahead and close the issue since the issue looks like it's related to a community package.


Add Monroe County, Michigan

Added Monroe County data found in https://cityofmonroegis.monroemi.gov/Apps/GISViewer/ over this ESRI server -> https://cityofmonroegis.monroemi.gov/arcgis/rest/services/Production/Parcels/Mapserver/0 to sources/us/mi/monroe.json
cc @ramyaragupathy @msirenko @ingalls @migurski

This looks like the City of Monroe, Michigan, not the county. Take a look at this example of a city source. We seem to have settled on these differences for city-based sources:

The filename for cities begin with city_of_ (whereas counties are just the name of the county)
We add a geometry key to the coverage object. In most cases its just a single point centered over the city, but in some cases we add a polygon
For US cities we use the US Census's "place" geoid. I look for that on Census Reporter: e.g. https://censusreporter.org/profiles/16000US2655020-monroe-mi/. The geoid (2655020) is the part in the URL after the US but before the dash.

Great! Thanks for the contribution.


Uncaught TypeError: Cannot read property 'didResolveConflict' of undefined

[Enter steps to reproduce below:]

...
...

Atom Version: 0.175.0
System: linux 3.16.0-29-generic
Thrown From: merge-conflicts package, v1.2.9
Stack Trace
Uncaught TypeError: Cannot read property 'didResolveConflict' of undefined
At /home/istinspring/.atom/packages/merge-conflicts/lib/merge-conflicts-view.coffee:118

TypeError: Cannot read property 'didResolveConflict' of undefined
  at /home/istinspring/.atom/packages/merge-conflicts/lib/merge-conflicts-view.coffee:118:13
  at proc.process.exit (/home/istinspring/.atom/packages/merge-conflicts/lib/git-bridge.coffee:145:11)
  at BufferedProcess.triggerExitCallback (/usr/share/atom/resources/app/src/buffered-process.js:63:47)
  at /usr/share/atom/resources/app/src/buffered-process.js:70:18
  at Socket.<anonymous> (/usr/share/atom/resources/app/src/buffered-process.js:150:18)
  at Socket.emit (events.js:129:20)
  at Pipe.close (net.js:469:12)

Commands
  4x -2:33.1 core:select-down (atom-text-editor.editor.conflicted.is-focused)
  3x -2:31.0 core:select-up (atom-text-editor.editor.conflicted.is-focused)
  2x -2:29.5 core:select-down (atom-text-editor.editor.conflicted.is-focused)
     -2:27.9 editor:consolidate-selections (atom-text-editor.editor.conflicted.is-focused)
     -2:27.9 core:cancel (atom-text-editor.editor.conflicted.is-focused)
     -1:29.1 core:paste (atom-text-editor.editor)
     -1:05.6 editor:consolidate-selections (atom-text-editor.editor.conflicted.is-focused)
     -1:05.6 core:cancel (atom-text-editor.editor.conflicted.is-focused)
     -1:04.6 editor:consolidate-selections (atom-text-editor.editor.conflicted.is-focused)
     -1:04.6 core:cancel (atom-text-editor.editor.conflicted.is-focused)
     -1:01.8 command-palette:toggle (atom-text-editor.editor.conflicted.is-focused)
 10x -0:57.6 core:backspace (atom-text-editor.editor.mini)
     -0:40.1 editor:consolidate-selections (atom-text-editor.editor.mini)
     -0:40.1 core:cancel (atom-text-editor.editor.mini)
  2x -0:38.3 settings-view:open (atom-text-editor.editor.conflicted.is-focused)
     -0:00.0 core:save (atom-text-editor.editor)

Config
{
  "core": {
    "ignoredNames": [
      ".git",
      "*.sublime-*",
      "__pycache__"
    ],
    "themes": [
      "atom-dark-ui",
      "monokai"
    ],
    "projectHome": "/home/istinspring/projects"
  }
}

Installed Packages
# User
angularjs, v0.1.0
editorconfig, v0.3.0
linter, v0.10.1
linter-jshint, v0.1.0
linter-pylint, v0.2.0
merge-conflicts, v1.2.9
monokai, v0.11.0
project-manager, v1.15.0
terminal-status, v1.3.5

# Dev
No dev packages

/cc @atom/core

I also see this error.

I experience this aswell.

I have seen this error as well.

Replicated and corrected. :dart: Silly mistake, this one will be fixed in the next release.
Thanks for the reports! :bow:


Compare to vim-stay

Useful plugin, and new to me. Interesting would be a comparison, or even symbiosis, with  https://github.com/kopischke/vim-stay which, on first glance, does something similar.

Please submit a patch. Thanks!

Hmm.. you know vim-lastplace better than the greenhorn over here. It seems to me that vim-stay restores folds and other information as well, whereas vim-lastplace focuses on restoring the cursor position and unfolding to see it, whenever necessary. It was more meant as a hint than a pull request of mine that blends them together.

Thanks, @Konfekt


Filter resources by status

It would be helpful to have the ability to filter resources by Running/Stopped/Suspended status.

Do you mean so you could terminate resources that are running but not those that are Stopped or Suspended? Did you see an error where leftovers was trying to shut down an already stopped resource? I can add functionality to skip resources that are already being shut down so that the user doesn't need to know that detail.

In the case that brought this up, we specifically want to wipe suspended VMs and unattached persistent disks.

There are a lot of resources across all the IaaSes that have no status, so should we skip those if a status flag is passed? Should the value of the status flag need to match precisely with the IaaS resource's specification for that value?

Going to close this for now since trying to filter by the various iaas-specific/resource-specific statuses will be rather complex and would require the user to know what terminology is used for each status. Instead, the user can go with being prompted and determine the status if need be!


Doesn't appear to work anymore

I went to the examples site and it failed to save any information. Errors appear in the chrome console log concerning the native API. I'm on Chrome 49 desktop.

Just tried with Chrome 49, and it works fine for me.
To further investigate, could you please share the exact error message that you get? Also, what was the exact URL you were visiting? Was it http://jensarps.github.io/IDBWrapper/example/ ?
Thanks!

It was on http://jensarps.github.io/IDBWrapper/example/basic/ but I just realized that I used a letter instead of a number for a key --  Uncaught DataError: Failed to execute 'put' on 'IDBObjectStore': Evaluating the object store's key path yielded a value that is not a valid key. was in the browser console. I guess I should have paid more attention to the label, but a little validation might help too ;)

Great that it worked for you in the end – and, yeah, I probably should add a validation to that example :)


Change preview icon in toolbar breaking functionality

icons hardcoded in css
.editor-toolbar.disabled-for-preview a:not(.fa-eye):not(.fa-arrows-alt):not(.fa-columns) {
	pointer-events: none;
	background: #fff;
}

This is by design. CSS is the easiest way to handle disabling pointer events when a particular class is present on a parent. If you've manually changed the icon for the preview, you'll also need to add a quick CSS rule to handle it properly when other icons are disabled.

I agree, but the documentation is not written about it. I spend time to solve this bug.

I look into ways of not hardcoding it to the class, which may change.

Fixed with commit a52a9a009803a75dab4b0760a89a6849669922b9


Fix nearby leaks, at least some ot them

Description
Fixes #1368 , current leaks. Currently I could only reproduce C, it took too much time to be able to reproduce it. A and B is same, and It seems like E and F is something we can ignore: https://medium.com/@amitshekhar/android-memory-leaks-inputmethodmanager-solved-a6f2fe1d1348
Since we have already solved B (also A) and D, we should only have C
Tests performed
Manual testing, API level 19

Codecov Report

Merging #1390 into 2.7.x-release will decrease coverage by <.01%.
The diff coverage is 0%.

@@               Coverage Diff                @@
##           2.7.x-release   #1390      +/-   ##
================================================
- Coverage           3.32%   3.31%   -0.01%
================================================
  Files                128     128
  Lines               6804    6808       +4
  Branches             667     668       +1
================================================
  Hits                 226     226
- Misses              6563    6567       +4
  Partials              15      15

Impacted Files
Coverage Δ

...rw/commons/upload/DetectUnwantedPicturesAsync.java
0% <0%> (ø)
:arrow_up:

...ava/fr/free/nrw/commons/nearby/NearbyActivity.java
0% <0%> (ø)
:arrow_up:

...java/fr/free/nrw/commons/upload/ShareActivity.java
0% <0%> (ø)
:arrow_up:

.../fr/free/nrw/commons/nearby/NearbyMapFragment.java
0% <0%> (ø)
: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 6fc294a...7b5c5ce. Read the comment docs.

I am merging my own PR, since release is urgent.

Thanks @neslihanturan !


Repeatable simulations

The class should provide means to initialize the random seed to a predefined constant. Via this it would be possible to have identical simulations.

Done


⚠ Bofrost.lu has degraded performance

In 06dfcb1, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1973 ms

Resolved: Bofrost.lu performance has improved in 9934c22 after 13 minutes.


Lib for making HTTP requests

I'm leaning towards axios as I've worked with it before and it seems to work fine. Another plus is that it's 100% promise-based, and that works well with the idea behind nyts — a promise-based wrapper for the YTS API.

Sindre's got isn't too bad either.

@arungalva what do you think? 🤔

using Axios sounds good to me.

Mature [X]
Used it before [X]
promise-based [X]
only 2 dependencies

Tracking with #24.


[9.x] Make Config facade macroable

Make the Config facade macroable so users can extend it in their own apps.
Reasoning: Since #43520 was closed without merging I was hoping to implement Config::collect() via a macro, however the Config facade is not currently macroable.

Renamed this PR title to what it actually does.


[Request] Add listary-beta

Website: https://www.listary.com/beta
Discussions: https://discussion.listary.com/t/listary-6-beta/4615
Download: https://www.listary.com/download/beta/listary6/ListaryInstaller.exe

Only non-portable


Added @hspv.nrw.de and @studium.hspv.nrw.de

Hey ho!
I hope I got the structure right.
The domains to validate are:

<EMAIL_ADDRESS>(for teachers)
<EMAIL_ADDRESS>(for students)

The university offers a Bachelor's degree course in "Administrative Informatics", which has IT relevance. The course of studies lasts the classic three years and should therefore meet the requirements.
Details for comparison:

Website of the university: https://www.hspv.nrw.de/ or https://www.hspv.nrw.de/en/
Course of studies "administrative informatics; Verwaltungsinformatik in German": https://www.hspv.nrw.de/studium/bachelorstudiengaenge/uebersicht/#c4052
Overview of e-mail addresses: https://www.hspv.nrw.de/services/it-an-der-hochschule/e-mail-roundcube/

Unfortunately the pages are only available in German.

@Summerfeeling Thank you very much for providing us with the detailed information!
I am sorry for a delay with processing your request.
Pull request merged. Thank you!


update package

First of all thanks for developing a handy lib. I've been using this on a teensy 4 + x8r successfully by installing the package through platformio.
I ran into a bug today and after checking with the github repo to find a fix, I noticed that I was running version 1.0.1 which is pretty old. However its the latest version that is available through the platformio package manager:
https://platformio.org/lib/show/5622/Bolder Flight Systems SBUS
Is there any chance of updating the package to the current release?

I would chat with the Platform IO folks and point them to this repo. I'm not sure how packaging is done in their system.

Thanks for the speedy reply. Turned out it was easy. I set this in my platformio.ini file and it all worked:
lib_deps =
	https://github.com/bolderflight/sbus-arduino/archive/refs/tags/v2.1.2.zip

After fiddling around with the PlatformIO console and getting information from their website, it seems they expect the maintainer of the library to register new versions. I was able to do it for myself with a current copy of the source code but now this library 'belongs' to my personal path, even all other information is kept intact: https://platformio.org/lib/show/12069/Bolder Flight Systems SBUS
Somewhen  @flybrianfly  must have done the same steps because formerly the library path was at bolder flight.


SignUp Page - Password Field Correction

In mobile view, Password field is not accessible by the user on SignUp Page.

Make the field accessible by the user.

i will take up this issue sir

I have assigned this issue to you.


First commit

This is the first commit for microservice functionality with 3 API endpoints.

why have you used force pushed?
and why have you closed the PR? @tushar517