$VOC_TREE_PATH $MOST_SIMILAR_IMAGES_NUM

你好，小白问一下这两个参数分别指的是什么？第一个时词典路径还是指什么？需要是什么格式呢？后一个参数是随便设置吗？

$VOC_TREE_PATH denotes the absolute path of the vocabulary tree provided by COLMAP. The second parameter denotes the upper bound of the images you want to match (you need to adjust it according to the scale of your own dataset).

这里是不是对应的应该是points2才对...

you're right. Which file is it? I'll fix it soon.

you're right. Which file is it? I'll fix it soon.

global_mapper.cpp里面的函数RefineRelativeTranslationWithKnownRotation(...)

you're right. Which file is it? I'll fix it soon.

顺便请问一下，这个错误是什么原因的？看着像是求解过程为之原因崩溃了，看样子求解器好像是您自己写的，所以想问一下您有没有头绪，莫非是我自己的数据集重复纹理过多或其他噪音太多导致数据较差而导致求解器无法求解？

It may be due to the view graph being too sparse and it contains multiple connected components. It can try to identify if there exists more than one connected component in the view graph. If so, you can try to increase the number of candidate matching pairs for each view to obtain a denser graph.


Design: Adapt the play button

https://tinyurl.com/2gox7lms has an "empty" play button which is also a different triangle: Cf. https://tinyurl.com/2gazthq2 and attachment.

Icon changed in commit c544c12d6fba728116ba51fa72b2176d2cbd7938!
Demo: https://polimediaupv.github.io/paella-ethz/?id=belmar-multiresolution-remote

Confirmed, thanks.


Document how to use partial admission

What would you like to be cleaned:
How to enable the feature gate and how to setup the annotation.
Also add notes to integration that don't support the feature like JobSet.
/kind documentation

/remove-kind cleanup

/assign @trasc

/close
done in $954

#954


ParseInclude middleware error

Steps to reproduce

Installed Lumen 5.4
composer required  "barryvdh/laravel-cors": "^0.9.2
composer require "ellipsesynergie/api-response": "^0.14.0"
Added middleware and service providers to the bootstrap/app.php file:

$app->middleware([
    'cors' => \Barryvdh\Cors\HandleCors::class,
    \EllipseSynergie\ApiResponse\Laravel\Middleware\ParseInclude::class,
]);

$app->register(Barryvdh\Cors\ServiceProvider::class);
$app->register(EllipseSynergie\ApiResponse\Laravel\LumenServiceProvider::class);

Added the following to the lumen controller:

use EllipseSynergie\ApiResponse\Contracts\Response;
$posts = Post::all();
return $this->response->withCollection($posts, new PostTransformer);

Expected behaviour
Must return the post model data.
Actual behaviour
The api-response package class \EllipseSynergie\ApiResponse\Laravel\Middleware\ParseInclude::class is causing the error.
Specifically the handle methods returned response on line 41.  return $next($next);
Change the return to  return $next($request); then the query responds correctly.
Browser log
`Type error: Argument 1 passed to Barryvdh\Cors\CorsService::isCorsRequest() must be an instance of Symfony\Component\HttpFoundation\Request, instance of Closure given, called in /Users/waynegibson/Projects/spacecomx-ecosystem/firefly/vendor/barryvdh/laravel-cors/src/HandleCors.php on line 35`

@spacecomx Fixed with https://github.com/ellipsesynergie/api-response/commit/78544c2fab1473fdafb2f9be344d9aacdd08e005. Tag 0.14.2 created. Thank you.


Toml support

Add support for pyproject.toml
see https://github.com/ome/cookiecutter-omero-webapp/pull/8

Tested with setup.py https://github.com/jburel/omero-test-build/actions/runs/9275144245
Tested with pyproject.toml (no setup.py) https://github.com/jburel/omero-test-build/actions/runs/9275516600

cc @will-moore


[BUG] new RKE2 releases do not show CSI Options in the UI for vsphere cluster

Rancher Server Setup

Rancher version: tested on 2.8.2 & 2.7.10

Installation option (Docker install/Helm Chart): Helm tested on RKE2 1.27.11 & 1.26.14

tested for Downstreamclusters 1.27.11 & 1.26.14 vsphere based clusters
-->

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)
Admin

Describe the bug
During setup of new vsphere clusters under Add-on Config there is no field "vSphere CSI Configuration" anymore as well the following error in the browser development console occurs:
Failed to fetch or process chart info for rancher-vsphere-csi

also after the clusterdeployment there are some vsphere-csi-controller pods in CrashLoopBackOff state.
To Reproduce

Create new vsphere based downstream cluster
At Cluster Configuration select Basics
select Kubernetes Version 1.27.11 or 1.26.14
select vsphere as Cloud Provider
switch to Add-on Config
expect vSphere CPI Configuration and vSphere CSI Configuration but CSI Configuration form is missing:

Result
Expected Result
can be tested by following the same procedure but selecting 1.27.10 or below or 1.26.14 or below.
expected is the configuration dialog of  vSphere CPI Configuration and vSphere CSI Configuration as well as not getting errors in the browser console.

Additional context
I know there is some documentation about the new out-of-tree implementation here: https://ranchermanager.docs.rancher.com/how-to-guides/new-user-guides/kubernetes-clusters-in-rancher-setup/migrate-to-an-out-of-tree-cloud-provider/migrate-to-out-of-tree-vsphere
but it's quite confusing to me so i am not sure if the new behavior is as designed as well as the documentation states that there should be the CPI as well as CSI configured during cluster creation AND the CPI and CSI plugin should be installed afterwards as App in the Application menu.

why both? what exactly is the difference between the rancher-vsphere-csi /cpi which is configured during installation VS the vsphere-csi /cpi which can be installed afterwards in the application section?
is one of the procedures using the in-tree implementation? for my understanding both of them are the out-of-tree implementation from vmware.

EDIT: tried installing the CSI App after the deployment but it fails:
Error: INSTALLATION FAILED: rendered manifests contain a resource that already exists. Unable to continue with install: ServiceAccount "vsphere-csi-controller" in namespace "kube-system" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-name" must equal "vsphere-csi": current value is "rancher-vsphere-csi"

Any news about this issue ? thanks

I am facing this issue too and the CSI shows errors

for everybody facing the same issue: The only workaround i found is to use the older versions which do not have the problem but are marked as deprecated
using 1.26.14 is also affected so even tough it's not deprecated, using it would not solve the issue.

As UI understands, this ties into getting https://github.com/rancher/rke2/issues/5217 in and there may be overlapped testing in https://github.com/rancher/rancher/issues/43980 so Dashboard doesn't need to update anything. Do keep me posted if this changes and we should get a ticket in rancher/dashboard

How to solve this issue ? I upgrade rke2 to 1.27 and configs are messed up. How can they make a mistake about this i don't understand.  This is a major issue

This has been validated by the k3s/rke2 team as fixed with the latest KDM changes. Therefore, this will be fixed in live environments for March k8s patches of rke2.

Closing per the above comment as March RKE/K3s versions are included in upcoming 2.8.3 release.


[AutoPR] cognitiveservices/data-plane/CustomVision/Training

Created to accumulate context: cognitiveservices/data-plane/CustomVision/Training

(message created by the CI based on PR content)

in review


Doesn't work with vue 3

Even the simplest empty invocation of VueGlobalEvents (<GlobalEvents />) in a vue 3 app will render a blank page with an uncaught error in the console log.

Interesting! Thank you for reporting. Will look it up during the weekend most likely. Or if you want, you can try to debug it and provide a PR with a fix :)

It does: https://github.com/shentao/vue-global-events/tree/next. I will add a link on the readme
FYI: as a maintainer, we cannot help if you don't at the very very least provide a stack trace

Uh, I assumed he meant the next version...

The next version works fine. Thanks.


This fix forbids this, protecting orders' metadata
I want to merge this change because I think,
order's metadata should be protected form third parties trying to modify it

Closes #8884
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

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

Hey, thanks for opening the PR. Updating public metadata of orders is part of the public API which is accessible without any permissions. In this case, we should require passing UUID tokens instead of IDs, as they are hard to guess, while order IDs are just database IDs decoded with Base64. We have a open pull request to disable using IDs for orders and checkouts: https://github.com/saleor/saleor/pull/8906.

Closing this PR in favor of https://github.com/saleor/saleor/pull/8906.

Hi @maarcingebala, I understand. Thanks for replying.
Will PR #8906 be merged to version 3.0?
Thanks

Yes, these changes will be applied to 3.0 and 3.1 (main).

Thank you very much. When can I expect this to be merged to 3.0?
Thanks!

@sfehlandt We will try to port these changes to 3.0 today or tomorrow.

Hi @maarcingebala sorry to insist with this but I was wondering if there is any chance this gets merged to 3.0 this week?
We are close to a deadline and this change is very important to us.
Kind regards

@sfehlandt I'm sorry to hear that, unfortunately, we are currently blocked with this, as we are waiting for our Saleor Cloud users to migrate their solutions to use the new approach. We're going to push it anyway and I hope that we will merge it this week, but cannot promise.


ERROR: Failed to resolve: com.google.android.gms:play-services:4.3.23

Thanks for stopping by to let us know something could be better!

PLEASE READ
If you have a support contract with Google, please create an issue in the support console. This will ensure a timely response.
Discover additional support services for the Google Maps Platform, including developer communities, technical guidance, and expert support at the Google Maps Platform support resources page.
If your bug or feature request is not related to this particular library, please visit the Google Maps Platform issue trackers.
Check for answers on StackOverflow with the google-maps tag.

Please be sure to include as much information as possible:
Environment details

Specify the API at the beginning of the title (for example, "Places: ...")
OS type and version
Library version and other environment information

Steps to reproduce

?

Code example
# example

Stack trace
# example

Following these steps will guarantee the quickest resolution possible.
Thanks!

Looks like the version you are using is quite outdated. Please update to 1.0.0 if you are trying to use this utility library, otherwise, the latest maps is 17.0.0.


Environment Variable Substitution doesn't work with array values

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2481
Originally created by @cooley1 on 2020-12-16 00:40:22

What is the current bug behavior?
Environment variables can't be used in the meltano.yml file for arrays.
What is the expected correct behavior?
The yaml parser should traverse arrays to substitute environment variables. It would also be nice to add custom yaml variable support to the parser. (e.g. custom_var: var -> database: '{{ custom_var }}')
Steps to reproduce

In a meltano.yml file try to use an environement variable in an array.

Relevant logs and/or screenshots
plugins:
  extractors:
  - name: tap-mysql
    variant: transferwise
    pip_url: pipelinewise-tap-mysql
    config:
      host: host
      user: root
      database: ${DB_NAME}
    select:
    - ${DB_NAME}-users.id
    - ${DB_NAME}-users.email
    metadata:
      ${DB_NAME}-users:
        replication-method: FULL_TABLE

database: ${DB_NAME} works because it's an object. But it doesn't work in the select because it's an array.
Possible fixes
Not sure, sorry.
Further regression test
Ensure we automatically catch similar issues in the future

[ ] Write additional adequate test cases and submit test results
[ ] Test results should be reviewed by a person from the team

View 6 previous comments from the original issue on GitLab

I'm going to close this as a duplicate of https://github.com/meltano/meltano/issues/3171 and keep the work on the newer issue.

Duplicate of https://github.com/meltano/meltano/issues/3171


[fix]rename compound_symbol.py

Description

Fixes #
Where has this been tested?

Platform(s): ... xx
Python version(s): ... xx
Unreal Engine version(s): ... xx
CARLA version: ...master

Possible Drawbacks

When I compile the project, I see this error: srunner/osc2/symbol_manager/compound _symbol.py Is there an extra space here?

This change is

@PabloVD please review

@starrye thanks for noticing it and creating the PR!


Integrated terminal does not measure character width correctly

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:1.68.1
OS Version: Windows_NT x64

It seems that the integrated terminal does not measure character width according to current font.
I noticed this while using tqdm in python, which by default draws progress bar with block characters U+2588 to U+258F. The module treats these elements as half-width characters and so does my default font, but the terminal seems to count them as full width characters and start a new line before reaching the right edge of the terminal as a result.
Here is a simple example:

what version of windows are you on? we cannot reproduce, indicating it's likely an issue w conpty and will be fixed when your windows version is updated

Thanks for reply.
The laptop encountering this problem is on version 1809. I've just tested on my own PC (21H1) and failed to reproduce it, so it seems likely.
Sadly, this laptop belongs to my company, and I cannot update the computer by myself. Are there any suggestions about any workarounds?

unfortunately this is not something we can control


Enterprise-level CodeQL pull request alerts report

Summary
Elevate your enterprise's security with this new enterprise-level report, designed to proactively monitor and track preventative remediation activities and autofix metrics, ensuring vulnerabilities are addressed swiftly and efficiently. Additionally, this report is also designed to highlight the substantial value and return on investment that GitHub Advanced Security prevention tactics brings to your security operations.
Intended Outcome
Many security managers struggle to quantify the preventative impact of GitHub Advanced Security, understand the value that autofix delivers and communicate its benefits effectively. Wit this report, you'll have the data-driven metrics at your fingertips to effectively drive prevention strategies, demonstrate impact, and, ultimately, significantly mitigate enterprise risks.
How will it work?
Introducing a new, comprehensive report that aggregates CodeQL pull request alerts and breaks them down by repository, providing deep insights into your security landscape. This report will feature key metrics, including the most prevalent rules triggering pull request alerts, and detailed breakdowns of alert statuses—unresolved, merged, dismissed, and specifically, those fixed with or without autofix. Additionally, it will track and display trends in pull request alert volumes, offering a historical view that can be filtered extensively to suit your analytical needs.
Prototype report

This shipped with GHES 3.15: https://github.blog/changelog/2024-11-12-the-github-enterprise-server-3-15-release-candidate-is-available

Edit to clarify: This feature will be available with the GHES 3.15 general availability release on December 3


This is more like a question

Can i do in the view something like
$user->present()->fullname

I guess you could, but that would be a bad design pattern.

If you wanted this, you could write yourself a short trait to drop into your models.

Ok, thank you


"Syslog" module not found error

One or more required packages (Syslog) could not be found.

Thank you, syslog module issue is fixed, but now not able to access UI on webpage, it says
{"m2m:dbg": "Release Version Indicator is missing in request, falling back to RVI='1'. But Release Version '1' is not supported. Use RVI with one of ['2a', '3', '4']."}

Thank you, i was trying to access the UI using <IP_ADDRESS>:8080/webpage instead of <IP_ADDRESS>:8080.
I can access the UI on <IP_ADDRESS>:8080.
Thank you very much, it is working perfectly fine.

Great! Good to hear.
BTW, you can change the end point for the web UI setting or adding the following setting to the acme.ini configuration file:
[cse.webui]
; Root path of the web UI. Default: /webui
root=/webui

Yes, thank you very much once again.


Release Google.Cloud.SecurityCenterManagement.V1 version 1.0.0-beta03

Changes in this release:
Bug fixes

Annotate EffectiveSecurityHealthAnalyticsCustomModule.name as IDENTIFIER (commit 78eb921)
Annotate SecurityHealthAnalyticsCustomModule.name as IDENTIFIER (commit 78eb921)
Annotate EffectiveEventThreatDetectionCustomModule.name as IDENTIFIER (commit 78eb921)
Annotate EventThreatDetectionCustomModule.name as IDENTIFIER (commit 78eb921)

New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Documentation improvements

Updated comments (commit 78eb921)

Created release for Google.Cloud.SecurityCenterManagement.V1-1.0.0-beta03


Feature/allow same name certs

hoi @bakito
we had cases where certs had the same name and entries get overwritten. this pr solves this problem by appending the index.
LG :)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.545%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/configmap/jks.go
1
3
33.33%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
93

Relevant Lines:
110

💛  - Coveralls

@m-terra Thank you for the contribution, will create a new release later this day.


Fix NullPointerException in failUnfinishedResponses

Motivation:
Fix https://github.com/line/armeria/issues/3036
Modification:
Due to for each loop calling HttpResponseWrapper#close and trigger this#removeResponse during looping. We think it may be the reason of NPE at HttpResponseWrapper#close. Try to remove through iterator first then calling close method.

Would it be possible to make a test for this? I'm fine without the test though. 😄

Yeah I want, just curious any idea to test this change? I'm not sure how to test this case now. Because  looks like it won't crash easily when we didn't change the code.(And how to reproduce? Server stop when client is still waiting result will trigger the method call.

just curious any idea to test this change?

I have no idea. 😅 So I guess it's fine not to have the test.


Labware V2: Labware Definitions Bugs

Expected Behavior

Labware definitions should match manufacturer's technical drawing, found here:

https://drive.google.com/drive/u/0/folders/1EJGJAV8y1d6G_v9vTakb09CecYWmMULM

Steps to Recreate

Used labware library to compare dimensional specs to technical drawings.

http://sandbox.labware.opentrons.com/edge

Observed Behavior
See mismatches, below

[ ] opentrons_24_tuberack_eppendorf_2ml_safelock_snapcap

Depth should be 39.1, diameter should be 8.8

[ ] opentrons_15_tuberack_falcon_15ml_conical

Diameter should be 14.9, inner depth not in tech drawing

[ ] opentrons_6_tuberack_falcon_50ml_conical

Diameter should be 27.81, inner depth not in tech drawing

[ ] corning_48_wellplate_1.6ml_flat

Length should be 127.89
width should be 85.6
Y offset should be 10.08

[ ] corning_384_wellplate_112ul_flat

Max vol mismatch between drawing name and def name
X offset should be 8.99

[ ] opentrons_10_tuberack_falcon_4x50ml_6x15ml_conical

Diameter of 50ml should be 27.81
Diameter of 15ml should be 14.9

[ ] corning_24_wellplate_3.4ml_flat

Y offset should be 13.77

Note Opentrons Tiprack has known issues, but we are waiting on final technical diagram from Danny
Issues found with current diagram:

opentrons_96_tiprack_300ul

Length should be 127.75
Width should be 85.5
Total Length should be 59.17
Diameter not in drawing
Y offset should be 11.2
X&Y spacing should be 9.01


Disable test_read_hive_fixed_length_char on Spark 3.4+.

Fixes #8321.
This commit disables test_read_hive_fixed_length_char for Spark 3.4 until #8324 is resolved (i.e. the change in behaviour of CHAR columns is addressed).

Build

Build

so to be clear, by disabling the test, we fallback to the CPU here?  want to ensure since https://github.com/NVIDIA/spark-rapids/issues/8324 is marked low priority.  If that is the case should we have test to verify it falls back?

It's not a fallback, exactly.
On 3.4, Spark adds a code-gen step to modify the results (i.e. pad the result column out to the required width). That step causes the Project exec to fall off the GPU:
+- *(1) Project [staticinvoke(class org.apache.spark.sql.catalyst.util.CharVarcharCodegenUtils, StringType, readSidePadding, foo#20, 3, true, false, true) AS foo#24]
   +- GpuColumnarToRow false
      +- GpuFileGpuScan orc spark_catalog.default.foobar[foo#20] Batched: true, DataFilters: [], Format: ORC, Location: InMemoryFileIndex[file:/tmp/foobar], PartitionFilters: [], PushedFilters: [], ReadSchema: struct<foo:string>

I don't know if it's worth adding a test verifying that Project isn't on GPU on 3.4.

ok main thing is it falls back and doesn't fail, I'm fine with skipping test since we have the issue to track.

Actually, you've convinced me. I've added an equivalent test on 3.4, allowing for ProjectExec fallback.
You're right. It would be best to codify that that the GPU results match the CPU results, even when ProjectExec falls back.

Build

Build


fixed cxunsavedfile Length field being incorrect size.

This fixes the bug I emailed you about in November, if the array of Unsaved files passed to clang_reparseTranslationUnit contains more than 1 unsaved file, then the call crashes.
It was caused by the Length field being a long instead of uint, this bug showed up on Windows 64bit.

whoa, I apologize that I didn't notice those PRs. Thanks!


Make @TechnologyRoot optional in article-only documentation

Bug/issue #, if applicable:
Summary
This makes @TechnologyRoot optional in article-only documentation.
When building documentation for a catalog of only articles with no explicit root article, DocC will follow the following logic to find a root:

If there's only a single article in the catalog, that becomes the root.
Otherwise, if there's an article with the same file name as the catalog name, that's becomes the root.
Otherwise, synthesize a minimal root with the same name as the catalog.

If this looks like a decent approach then we should document this behavior somewhere before merging.
Dependencies
None.
Testing
One article

Create a new documentation catalog:
mkdir MyCatalog.docc
Add a minimal article without a technology root:
echo "# One article\n\nArticle abstract" > MyCatalog.docc/MyArticle.md
Preview documentation for the new catalog
swift run docc preview MyCatalog.docc

DocC should preview the article at http://localhost:8080/documentation/myarticle
Question: should this path be "documentation/mycatalog" instead?

Stop the preview server.

Article with same name as catalog

Add another article without a technology root to the catalog, with the same name as the catalog this time:
echo "# Another article\n\nArticle abstract" > MyCatalog.docc/MyCatalog.md
Preview documentation for the catalog:
swift run docc preview MyCatalog.docc

DocC should preview the article at http://localhost:8080/documentation/mycatalog
MyArticle should be automatically curated.

Stop the preview server.

Synthesized root page

Rename the second article so that it's no longer named the same as the catalog:
mv MyCatalog.docc/MyCatalog.md MyCatalog.docc/OtherArticle.md
Preview documentation for the catalog:
swift run docc preview MyCatalog.docc

DocC should preview the article at http://localhost:8080/documentation/mycatalog
A new empty root page should be synthesized.
Both MyArticle and OtherArticle should be automatically curated.

Stop the preview server.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[ ] Updated documentation if necessary

@d-ronnqvist Thanks for working on this. I think this is a good idea, since it makes the entry barrier to this type of documentation way lower.
A couple of comments:

Did you consider adding index.md as another replacement of the technology root?. This could be a fallback in case there's no markdown file with the same name as the doc catalog, but there's an article with the file name index.

If we are moving ahead towards a direction where @TechnologyRoot is optional it might be a good idea to add some kind of deprecation note to that directive in the docs, where the use of the same name of the catalog, or index, is preferred over the use of the directive.

I left a minor code comment, but overall this looks good to me. To merge it, tho, it's needed to post info about this in the swift forums first.

Regarding your question:

should this path be "documentation/mycatalog" instead?

I think that keeping it as currently works is best, since it is consistent with the behaviour of docc for other types of catalogs, where the URL last path component is the same as the top level file name.

Did you consider adding index.md as another replacement of the technology root?. This could be a fallback in case there's no markdown file with the same name as the doc catalog, but there's an article with the file name index.

I didn't but that's always something we can add later.

If we are moving ahead towards a direction where @TechnologyRoot is optional it might be a good idea to add some kind of deprecation note to that directive in the docs, where the use of the same name of the catalog, or index, is preferred over the use of the directive.

We should find some way to document that @TechnologyRoot isn't always needed but I don't think we should deprecate. A developer may prefer to add @TechnologyRoot over renaming one of their articles.

I left a minor code comment, but overall this looks good to me. To merge it, tho, it's needed to post info about this in the swift forums first.

I can post something small in the forums about this to make people aware of the change but I don't think that it needs to block the PR. I view this more as a small enhancement than as a new feature.

@swift-ci please test

@swift-ci please test


Database view emulator. ActiveRecord for query statement.

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
yes

Fixed issues
#12025

@klimov-paul what do you think about it?

@klimov-paul what do you think about it?

This should not belong to the core.
As I already mentioned at th issue, there is no necessity of such thing as it can be easily achieved via default scopes feature:
https://github.com/yiisoft/yii2/issues/12025#issuecomment-235575887
Of course, there could be a situation, when such trick will not work, however I consider this to be a rare case.
I suggest this solution should be composed into an extension and maintained out of 'yiisoft' scope.

I agree with @klimov-paul. @mdmunir feel free to publish it as an extension.

As i said, this is View Emulator. I mean, everithing that can be do using view, can be done using this class. Some people prefer use view another people prefer using sql directly. If i have ability to create view in aplication, i chose create it in aplication.
@klimov-paul code not work for situation like this
Invoice::find()
    ->alias('i')
    ->joinWith(['paid p'])
    ->where('[[i.value]] > [[p.total]]') // must use having instead of where

^ that code not a rare case (for me). That the reason why i need this class.
Simple way to create the same class without ViewRecord is directly inherit from BaseActiveRecord. But, if that so, i must override methods getDb(), insert(), primaryKey() many many times because that methods are abstract.

feel free to publish it as an extension.

I guess my own extension. Ya, i use this in my project (not as extension, but as a single class in my project). I submit it to core yii just to share my code to other yii user.


Do not warn on font-style

Echoing discussion on this issue, the linter shouldn't warn on direct usages of font-style. There's no way to configure this via a token and the only available options are normal and italic anyway. I think it's reasonable to modify the linter rule to not warn on usages of font-style.

Added option to accept font style acceptCarbonFontStyleFunction and an issue to review defaults before publishing v3.
https://github.com/carbon-design-system/stylelint-plugin-carbon-tokens/issues/119

Fixed in 2.4.0

Is there any case where a user would want to set that acceptCarbonFontStyleFunction option to false?

@wkeese I will be reviewing the defaults and options for V3, for V2 I added this option to allow users to continue to get the same results.

I think you're overthinking it.  All bug fixes change the results.
And even if there were a backward-compatibility issue with V2, I don't see any reason for V3 to have an acceptCarbonFontStyleFunction option.
The goal here (as explained in #15131) is to not flag this CSS:
font-style: italic

Notice that it has nothing to do with a "font style function".
But even if you renamed the option to acceptDirectFontStyle or something like that, what is the value of letting the user toggle that option?
The only justification I can think of is to allow/disallow directly setting font-style, font-weight, line-height etc., rather than using CSS like @include type.type-style("productive-heading-01").
But in that case, the flag should be called something different (and do something different).

Reopening and considering options for v3

@wkeese should no longer warn from 2.8.0 onwards


Cannot specify the values for the user profile when userProfileEnabled = true

Hi dear reader
I like to have my custom user profile attributes and I can do it thanks to this lovely provider with this terraform scripts:
variable "backend-roles" {
  type    = list(string)
  default = ["admin", "user"]
}

variable "required-for-anyone" {
  type    = list(string)
  default = ["admin", "user"]
}

variable "permissions-for-anyone" {
  type    = list(string)
  default = ["admin", "user"]
}

provider "keycloak" {
  client_id = "admin-cli"
  url       = "http://localhost:8080/"
  username  = "admin"
  password  = "admin"
}

terraform {
  required_version = ">= 1.4, <= 1.4.2"

  required_providers {
    keycloak = {
      source  = "mrparkers/keycloak"
      version = "4.2.0"
    }
  }
}

resource "keycloak_realm" "you-say-realm" {
  realm             = "you-say-realm"
  display_name      = "You-Say app"
  display_name_html = "you-say"
  enabled           = true

  registration_allowed           = true
  registration_email_as_username = true
  password_policy                = "length(20) and forceExpiredPasswordChange(365) and notUsername"
  attributes = {
    userProfileEnabled = true
  }
}

resource "keycloak_realm_user_profile" "you-say-user-profile" {
  realm_id = keycloak_realm.you-say-realm.id

  attribute {
    name               = "email"
    display_name       = "$${email}"
    required_for_roles = var.required-for-anyone
    permissions {
      view = var.permissions-for-anyone
      edit = var.permissions-for-anyone
    }
    validator {
      name   = "email"
      config = {}
    }
    validator {
      name = "length"
      config = {
        "min" : 4
        "max" : 320
        "trim-disabled" : false
      }
    }
  }

  attribute {
    name               = "password"
    display_name       = "$${password}"
    required_for_roles = var.required-for-anyone
    permissions {
      view = var.permissions-for-anyone
      edit = var.permissions-for-anyone
    }
    validator {
      name = "length"
      config = {
        "min" : 20
        "trim-disabled" : false
      }
    }
  }

  attribute {
    name               = "firstName"
    display_name       = "$${firstName}"
    required_for_roles = var.required-for-anyone
    permissions {
      view = var.permissions-for-anyone
      edit = var.permissions-for-anyone
    }
    validator {
      name   = "person-name-prohibited-characters"
      config = {}
    }
    validator {
      name = "length"
      config = {
        "min" : 2,
        "max" : 255
        "trim-disabled" : false
      }
    }
  }

  attribute {
    name               = "lastName"
    display_name       = "$${lastName}"
    required_for_roles = var.required-for-anyone
    permissions {
      view = var.permissions-for-anyone
      edit = var.permissions-for-anyone
    }
    validator {
      name   = "person-name-prohibited-characters"
      config = {}
    }
    validator {
      name = "length"
      config = {
        "min" : 2,
        "max" : 255
        "trim-disabled" : false
      }
    }
  }

  attribute {
    name               = "occupation"
    display_name       = "$${occupation}"
    required_for_roles = var.required-for-anyone
    permissions {
      view = var.permissions-for-anyone
      edit = var.permissions-for-anyone
    }
    validator {
      name = "pattern"
      config = {
        pattern       = "^[a-zA-Z0-9_][a-zA-Z0-9_ ]*[a-zA-Z0-9_]$"
        error-message = "Please use only characters and space between words, And do not start or end your occupation with white space"
      }
    }
  }

  attribute {
    name               = "location"
    display_name       = "$${location}"
    required_for_roles = var.required-for-anyone
    permissions {
      view = var.permissions-for-anyone
      edit = var.permissions-for-anyone
    }
    validator {
      name = "pattern"
      config = {
        pattern       = "^[a-zA-Z\u0080-\u024F]+(?:. |-| |')*([1-9a-zA-Z\u0080-\u024F]+(?:. |-| |'))*[a-zA-Z\u0080-\u024F]*$"
        error-message = "Please enter a valid city name followed by the country name. e.x. Tokyo, Japan"
      }
    }
  }
}

resource "keycloak_user" "kasir-user" {
  username   =<EMAIL_ADDRESS>  enabled    = true
  realm_id   = keycloak_realm.you-say-realm.id
  email      =<EMAIL_ADDRESS>  first_name = "Kasir"
  last_name  = "Barati"
  initial_password {
    value = "Kasir123"
  }
  # How can I specify the value for occupation and location?
}

I thought that attributes is what should do the trick but it was not the case.
Any help appreciated.

Same issue here

Update: this is due to the order of operations. By default the user profile is created after the user, wiping the user attributes. You need to specify the attributes in the user, and enforce the dependence between the user creation and the user profile creation. See below
User profile creation:
resource "keycloak_realm_user_profile" "userprofile" {
  realm_id = keycloak_realm.realm.id

  attribute {
    name         = "username"
    display_name = "Username"
  }

  attribute {
    name         = "firstName"
    display_name = "First Name"

    permissions {
      view = ["user", "admin"]
      edit = ["user", "admin"]
    }
  }

  attribute {
    name         = "lastName"
    display_name = "Last Name"

    permissions {
      view = ["user", "admin"]
      edit = ["user", "admin"]
    }
  }

  attribute {
    name = "email"

    permissions {
      view = ["admin"]
      edit = ["admin"]
    }
  }

  attribute {
    name         = "organization"
    display_name = "Organization"

    permissions {
      view = ["admin"]
      edit = ["admin"]
    }
  }

  attribute {
    name = "department"

    permissions {
      view = ["admin"]
      edit = ["admin"]
    }
  }
}

User creation, including the depends_on parameter
resource "keycloak_user" "alice" {
  realm_id   = keycloak_realm.realm.id
  depends_on = [keycloak_realm_user_profile.userprofile]
  username   = "alice"
  email      =<EMAIL_ADDRESS>  first_name = "Alice"
  last_name  = "A"

  attributes = {
    "organization" = jsonencode(["NDtech"])
    "department" = jsonencode(["acquisitions"])
  }

  initial_password {
    value     = "alice"
    temporary = false
  }
}


Failed to connect to local system.

Before reporting a bug, please see our FAQ in FAQ.md!
Description
Failed to connect to local system.
Failed Step

More Details

Okay
Pada Sel, 26 Mar 2024, 11.57, Matthew Tanner @.***>
menulis:

Before reporting a bug, please see our FAQ in FAQ.md
https://github.com/whatsapp/proxy/blob/main/FAQ.md!
Description
Failed to connect to local system.
Failed Step More Details
—
Reply to this email directly, view it on GitHub
https://github.com/WhatsApp/proxy/issues/246, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BB4A44OS6BSW4NPIENXXFE3Y2DW35AVCNFSM6AAAAABFIFEKHGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYDOMJZGA2DGNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @imatthewtanner!
Please share more details on the issue you are experiencing in the form.


Enhancement: add the ability to parse messages from Peertube, Kbin, and Lemmy

Currently, if an user is following an account on a service that is not a microblog (such as Peertube, Kbin, or Lemmy) the backfill requests fails, typically with the error message "Error parsing toot URL". Checking on the source code, it seems like the application is hardcoded to support only URLs from Mastodon, Pleroma, and PixelFed. What is required to parse URLs from the other services listed above?

Thanks for your interest in this. I’d love to see FediFetcher extended for other servers/protocols, so this is something Od very gladly see any PRs for.
The following is needed, in order to make this possible:
Firstly you need to figure out if peertube, kbin, and lemmy actually support the Mastodon context api? Any server that doesn’t, is impossible to support without major rewrite. (I did look into doing this via the ActivityPub API, but support for that is amazingly even sketchier than for the mastodon api). I’d be open to this, but it would introduce extra complexity.
For any server that supports the mastodon api, it’s really easy to implement:

write a case for extracting status ID from status url in the parse_url function:

https://github.com/nanos/FediFetcher/blob/535bf1f40435a58d4581977da9c9056113646d47/find_posts.py#L446

write a case for extracting profile ID from profile URL in the parse_user_url function here:

https://github.com/nanos/FediFetcher/blob/535bf1f40435a58d4581977da9c9056113646d47/find_posts.py#L429
that would be it.

I'm really really please to see @Teqed is currently working on a PR to implement Lemmy support for both backfilling profiles and fetching context in #56
Also serves as a good template on how to import posts from software that doesn't support the Mastodon API!

With Lemmy implemented, I’ve split this out into two separate issues: #73 and #74


[flash_ctrl] RMA VCS SIMULATION TIME DEBUG

THIS SHOULD NEVER BE MERGED TO MASTER
The intention of this Draft PR is to illustrate the difference in Simulation
time between the Xcelium and VCS Simulators
Xcelium: ~13mins (seed=1)
VCS    : ~5Hours (seed=1, profiling on)
Signed-off-by: TIM EWINS<EMAIL_ADDRESS>

Here are the VCS profiling reports in text.  I have the report in html, but it seems I cannot upload this here
profileReport.txt

Here are the VCS profiling reports in text. I have the report in html, but it seems I cannot upload this here
profileReport.txt

Looks like this always block takes 80%+ time, but the code looks fine to me.

prim_generic_ram_1p.sv:62-75

Looks like the command to reproduce should be
./util/dvsim/dvsim.py hw/ip/flash_ctrl/dv/flash_ctrl_sim_cfg.hjson -i flash_ctrl_hw_rma

Hi ..  sorry,missed these pieces of information ...
Command to reproduce ...
./util/dvsim/dvsim.py hw/ip/flash_ctrl/dv/flash_ctrl_sim_cfg.hjson -i flash_ctrl_hw_rma --tool vcs -r 1 -v h --verbosity m --purge --max-parallel 1 --profile time --fixed-seed 1
Tool versions used
VCS - /tools/synopsys/installs/R-2020.12-SP2-2_vcs/bin/vcs
Xcelium - /tools/cadence/installs/XCELIUM2103/tools/bin/xrun

Closing this PR since we don't want to merge it


docs: clarity the difference of path in public API and raw API

Currently, it is not explicit in code that, where path is expected to be in normalized form, and where it is allowed for arbitrary input.
It will be helpful if we have a typr struct NormalPath<'p>(Cow<'p, str>), which encodes only normaized path. And we can use type invariant to represent guarantees we require/have in different places.

All public APIs accept user input in its original form, while all internal or raw APIs use normalized form for processing. We can ensure clarity on this distinction in our documentation.

Thank you, it seems like a nice idea, but I'm not interested in this change.

Was just a suggestion.


Build fails

Following the instructions exactly I get this error when attempting to build:
In file included from /usr/include/X11/Xlib.h:44,
                 from ../wine-mirror-git/dlls/winex11.drv/systray.c:31:
../wine-mirror-git/include/processthreadsapi.h:43:11: error: expected identifier or ‘(’ before numeric constant
   43 |     ULONG ControlMask;
      |           ^~~~~~~~~~~

This happens in multiple places. I've tried a clean build. I've updated my packages. I've built successfully in the past. Distro is Arch Linux (not an "i use arch btw" I mention for context about build dependency versions).

Same here kernel 5.18.0-arch1-1 (fresh build)

This seems to fail the build on normal wine-tkg-git without any patches.

I have made an issue on the repo now.
https://github.com/doitsujin/dxvk/issues/2657

This seems like a tough one...I'll try to build it myself and see if I could reproduce the issue.
Updates will be edited onto this comment.

This seems like a tough one...I'll try to build it myself and see if I could reproduce the issue.
Update 1: Found upstream issue: Frogging-Family/wine-tkg-git#779 wine-tkg-git builds appear to not work currently ...Wait, rolexlinux found this issue already, ¯_(ツ)_/¯
Update 2: Reproduced the error on default config, rebuilding without staging... Update 3: Reproduced the error on Mainline master, trying wine-7.8 branch...

7.7 was even failing for me, not sure 7.8 will be any better

You happen to be online? Nice! Let's meet on #wine Grapejuice

xdMatthewbx#1337

The faulty commit is https://github.com/wine-mirror/wine/commit/1e7c6d73c8f558668c6140d4a97fae0ad51d71eb

Further update & workarounds can be found in upstream https://github.com/Frogging-Family/wine-tkg-git/issues/779 , this issue will stay open until upstream fixes the issue.

Fixed in upstream: https://github.com/Frogging-Family/wine-tkg-git/commit/134bc9bda15fce13c378af3d5e6e35d629664a56
The fix was practically identical to my workaround, just implemented differently ¯_(ツ)_/¯
Closing issue.


Added missing page_canvas and removed web_fonts_disabled.

This removes the console warning #page_canvas is undefined..

Closing since we cannot merge this as the changes will be overwritten on the next sync. Instead, please send this change to one of the Arabic translation maintainers; you can find their contact information here: https://wiki.mozilla.org/L10n:Teams:ar. Thank you.


How to pack godot game(s) into a native android/ios app and start it?

I need to add a game to my app(used for some math learning) to make it funnier for kids so I start to learn godot.
But firstly I must figure out how to export a godot game as not a single app but a part of my app like a plugin and can be started by pressing a button in my native android/ios app.
Since the exported-pck file contents all data of a godot game , and I see the export template of IOS and Html seems to be some file to call the pck file(?) , is it possible to do this with our own java/objc code?
Sorry for my poor English and hope for your responses!

No response after a week.T_T
Was my issue not clear enough or being wrong somewhere?

AFAIK nobody ever tried this. If no one answered is likely that no one knows. Godot wasn't made to be embedded in other apps, so while it may be possible, you won't get much help here unless it's a more specific issue.

I'm trying to use "Custom Packge" to set my own packaged apk as export template.
So far, it  is successful to export a apk but fail to start the Godot activity and throw an exception of invalid reference in call to  “GetLongField”（called  from GodotLib.initiaize()?）. Maybe  somthing wrong in manifest or build.gradle configuration .

emmm...
This way(Custom Packge) works.
I can boot godot game from my app ....................... only once.
I change the code in Godot.java and GodotView.java about  back button so that when I press back button in godot game , it will just finish the activity not quit the whole app.
But it seems not enough to quit the godot part safely just simply adding some "this.finish()".
When i next boot godo game from my app , it throws these err:
E/godot   (16992): ERROR: Condition ' configured ' is true.
E/godot   (16992):    At: core\string_db.cpp:55:setup() - Condition ' configured ' is true.
E/godot   (16992): ERROR: Condition ' singleton ' is true. returned: __null
E/godot   (16992):    At: core\io\ip.cpp:270:create() - Condition ' singleton ' is true. returned: __null
E/godot   (16992): ERROR: Condition ' singleton ' is true.
E/godot   (16992):    At: core\input_map.cpp:289:InputMap() - Condition ' singleton ' is true.
E/godot   (16992): ERROR: Condition ' singleton != __null ' is true.
E/godot   (16992):    At: core\message_queue.cpp:339:MessageQueue() - Condition ' singleton != __null ' is true.
I/godot   (16992): *****SETUP OK
I/godot   (16992): ANDROID MODULES: Nothing to load, aborting
Maybe some properties should be initialized or some method should be call and I haven't figured it out so far.Hoping for help==.

I'm also looking for the same thing.

@First-AI Android subview embedding is supported since 3.2.3, see https://github.com/godotengine/godot-proposals/issues/1064.
iOS subview embedding isn't implemented yet, see https://github.com/godotengine/godot-proposals/issues/1473.


MPS CIRA connection not working

MPS CIRA connection not working on Node v8.11.4 & npm v5.6.0
All the other functionality of project like REST API's, Web UI etc working fine
AMT device doesn't initiate CIRA on above mentioned Node version for some reason

Hi @acveigas
There was an issue with express-ws library (ws.pause definition to be specific) which had to be updated. The latest master should not have any connection issues.


UpdateLogpushJobParams: Make it possible to remove logpull_options on update

Description
For reference: #1709
Make it possible to remove the logpull_options on update. I think this is still an issue with the Cloudflare API: I would expect a PUT to overwrite the existing resource, not only change the specified values.
Before:
PUT request (TF_LOG=trace output):
PUT /client/v4/zones/[redacted]/logpush/jobs/[redacted] HTTP/1.1
Host: api.cloudflare.com
User-Agent: terraform-provider-cloudflare/dev terraform-plugin-sdk/2.32.0 terraform/1.7.5
Content-Type: application/json
Body: {"dataset":"http_requests","enabled":false,"name":"example","output_options":{"field_names":[],"output_type":"ndjson","record_prefix":"{","record_suffix":"}","field_delimiter":",","timestamp_format":"unixnano","sample_rate":1,"CVE-2021-44228":false},"destination_conf":"[redacted]","frequency":"high"}

With this change:
PUT request (TF_LOG=trace output):
PUT /client/v4/zones/[redacted]/logpush/jobs/[redacted] HTTP/1.1
Host: api.cloudflare.com
User-Agent: terraform-provider-cloudflare/dev terraform-plugin-sdk/2.32.0 terraform/1.7.5
Body: {"dataset":"http_requests","enabled":false,"name":"example","logpull_options":"","output_options":{"field_names":[],"output_type":"ndjson","record_prefix":"{","record_suffix":"}","field_delimiter":",","timestamp_format":"unixnano","sample_rate":1,"CVE-2021-44228":false},"destination_conf":"[redacted]","frequency":"high"}

Output of the second terraform plan
No changes. Your infrastructure matches the configuration.

Types of changes
What sort of change does your code introduce/modify?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] This change is using publicly documented in cloudflare/api-schemas
and relies on stable APIs.

we shouldn't be sending empty strings to the API in order to remove values when using PUT given it should represent the final state of the resource. the omitempty here is the correct way of handling it so we i can't accept the PR as is.
let me check with the internal team as to which behaviour we are expecting here.

Thanks Jacob! Yeah, I think it would be better to solve this right at the API level. Just let me know in case further details are needed.

@jacobbednarz any update on this getting resolved upstream in the API ?

no further updates at this point. the team have it in their backlog to investigate and look at a fix to maintain compatibility.
the output_options takes precedence over logput_options though so you should be fine defining the newer version and it taking effect.


Explicit Model Binding Cannot Pass Column Name

Laravel Version: 7.14
PHP Version: 7.4.6
Database Driver & Version: Irrelevant

Description:
Using explicit model binding, I cannot pass in an optional parameter
Steps To Reproduce:
RouteServiceProvider.php
Route::model('page', \App\Page:class)
web.php
Route::post('pages/{page:slug}/publish', 'PageController@publish')->name('admin.pages.publish');
I expect to route to use the slug column to fetch the page, but instead it does not pass that value through to the \Illuminate\Routing\RouteBinding.php@forModel(...) function. This causes the resolveRouteBinding to fallback to the key name for the model, which I do not want to change for a simple route binding.
    public function resolveRouteBinding($value, $field = null)
    {
        return $this->where($field ?? $this->getRouteKeyName(), $value)->first();
    }

Specifically to the line
70    if ($model = $instance->resolveRouteBinding($value)) {
71         return $model;
72    }

I'd expect it to be something like
70    if ($model = $instance->resolveRouteBinding($value, $column)) {
71         return $model;
72    }

with $column = 'slug'
As a result, I can either use explicit model binding, and change getRouteKeyName to return slug or I can use implicit model binding (which I don't want to because I need explicit route binding for the update and delete functions)

I'm not sure we ever document this as being supported?


cellFilter applied for cellTooltip

For the cellTooltip, if the function returns true than the tooltip value would be the cell value.
I think that instead of returning the actual cell value we should return the cell value with the cellFilter applied to it.
Or if this is not the desired behaviour for the default tooltip, please indicate how we could simply achieve this without compromising performance.

Oversight on my part.  I only use tooltips for long description fields, so I hadn't considered the need for filters.

That was fast! Thanks guys!


python312Packages.plugwise: 1.4.1 -> 1.4.2

Diff: https://github.com/plugwise/python-plugwise/compare/refs/tags/v1.4.1...v1.4.2
Changelog: https://github.com/plugwise/python-plugwise/releases/tag/v1.4.2

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 349392 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.plugwise
python311Packages.plugwise
python311Packages.plugwise.dist
python312Packages.plugwise
python312Packages.plugwise.dist


First time boot takes incredibly long due to chown -R "$PUID:$PGID" /opt/certbot

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug

When I start the container for the first time it's taking an incredibly long due to it setting ownership:
nginx-proxy-manager  | ❯ Configuring npm user ...
nginx-proxy-manager  | ❯ Configuring npm group ...
nginx-proxy-manager  | ❯ Checking paths ...
nginx-proxy-manager  | ❯ Setting ownership ...

I did a bit of investigation and found out it's due to this command:
chown -R "$PUID:$PGID" /opt/certbot

This code was added in this commit:
https://github.com/NginxProxyManager/nginx-proxy-manager/commit/05307aa253c073cf94237fc96d816ec2919f4d7f
I ran this command myself inside the container and it took about 5 minutes to complete:

To me it feels a bit strange to have to execute this command on the /opt folder which isn't even mapped to my host system. So why not do this during the creation of the container rather then during boot?
Nginx Proxy Manager Version

2.10.4
To Reproduce
See above
Expected behavior

Boot quickly
Screenshots

N/A
Operating System

Docker inside a container on a Synology NAS
Additional context

I can confirm this. Sometimes it pretty annyoing

Any workaround at least? 3-4 minutes downtime just to restart npm is a lot, and very annoying when changing configs and have to restart to see the results.

I guess you could map a volume to the root fs that persists the /opt folder

I guess you could map a volume to the root fs that persists the /opt folder

Thanks, it worked!

I've ran into the exact same issue at startup of the docker container where the chown command (Setting Ownership) took around 25 minutes to complete and this happens every time I turn the container on.

I guess you could map a volume to the root fs that persists the /opt folder

Thanks, it worked!

How? can you please explain? I am not getting pass to the settings ownership and it simply timesout in my case. I am mounting the data and letsencrypt folder on an nfs share.

I too am hitting this issue on my TrueNAS system.  How I resolve this for now.
Step 1: Create a PVC mount called /opt/certbot2 (ie. an external mount is managed by TrueNAS - this is not the same as a named mount managed by docker, from docker's perspective it's still a host mount, but it's as good as you get with truenas).
Step 2: Start the container, then log into it via. shell (use heavyscript or truenas to get into the shell)
Step 3: Copy everything in /opt/certbot to /opt/certbot2 (cp -a /opt/certbot/* /opt/certbot2)
Step 4: Stop the container and change the PVC mount from /opt/certbot2 to /opt/certbot
Step 5: Start the container again.
It should now be very quick to chown, because the storage is external, not copy on write.
Of course, this is a terrible solution, because we are now overriding the docker image for what is in /opt/certbot, so as the container updates, our /opt/certbot will not unless you remove the PVC and do the above steps again.

I guess you could map a volume to the root fs that persists the /opt folder

Thanks, it worked!

@panos-stavrianos
How did you do that? I'm having the same problem running on my server, when I run it on my machine it doesn't take long.

I would guess,
run the container and wait for it to start,
use docker copy and copy the /opt from the container to a folder on your
harddisk
once this is done, stop the container
add a volume that maps the copy on your local disk to the /opt inside the
container
start the container again
Am Mo., 19. Feb. 2024 um 18:45 Uhr schrieb Lapo @.***>:

I guess you could map a volume to the root fs that persists the /opt folder
Thanks, it worked!
@panos-stavrianos https://github.com/panos-stavrianos
How did you do that? I'm having the same problem running on my server,
when I run it on my machine it doesn't take long.
—
Reply to this email directly, view it on GitHub
https://github.com/NginxProxyManager/nginx-proxy-manager/issues/3154#issuecomment-1952944043,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAY6MDHQYN3XKWID27OJX3LYUOFSHAVCNFSM6AAAAAA34FODYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJSHE2DIMBUGM
.
You are receiving this because you commented.Message ID:
@.***>

I would guess, run the container and wait for it to start, use docker copy and copy the /opt from the container to a folder on your harddisk once this is done, stop the container add a volume that maps the copy on your local disk to the /opt inside the container start the container again Am Mo., 19. Feb. 2024 um 18:45 Uhr schrieb Lapo @.>:
…
I guess you could map a volume to the root fs that persists the /opt folder Thanks, it worked! @panos-stavrianos https://github.com/panos-stavrianos How did you do that? I'm having the same problem running on my server, when I run it on my machine it doesn't take long. — Reply to this email directly, view it on GitHub <#3154 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAY6MDHQYN3XKWID27OJX3LYUOFSHAVCNFSM6AAAAAA34FODYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJSHE2DIMBUGM . You are receiving this because you commented.Message ID: @.>

@Nightreaver
Thank you!! it worked :)

I'm having the same problem with 2.11.1 via the TrueNAS catalog chart version 1.0.27
I never seem to get past the chown before TrueNAS decides that the container is unhealthy and attempts to restart it, causing an endless loop
2024-02-25 09:32:46.464991-05:00❯ Configuring npm user ...
2024-02-25 09:36:39.192117-05:00❯ Configuring npm group ...
2024-02-25 09:36:51.272227-05:00❯ Checking paths ...
2024-02-25 09:37:09.147531-05:00❯ Setting ownership ...

2024-02-25 09:58:04.848091-05:00❯ Configuring npm user ...

I haven't tried the proposed workaround

Affected by this issue as well on 23.10.2.

I too am hitting this issue on my TrueNAS system.  How I resolve this for now.
Step 1: Create a PVC mount called /opt/certbot2 (ie. an external mount is managed by TrueNAS - this is not the same as a named mount managed by docker, from docker's perspective it's still a host mount, but it's as good as you get with truenas).
Step 2: Start the container, then log into it via. shell (use heavyscript or truenas to get into the shell)
Step 3: Copy everything in /opt/certbot to /opt/certbot2 (cp -a /opt/certbot/* /opt/certbot2)
Step 4: Stop the container and change the PVC mount from /opt/certbot2 to /opt/certbot
Step 5: Start the container again.
It should now be very quick to chown, because the storage is external, not copy on write.
Of course, this is a terrible solution, because we are now overriding the docker image for what is in /opt/certbot, so as the container updates, our /opt/certbot will not unless you remove the PVC and do the above steps again.

So I copied the files to my HDD pool in my /data mount and created a folder there called certbot and mounted a host path of /opt/certbot to /data/certbot it has the same effect. Boots in second now.

So I copied the files to my HDD pool in my /data mount that mounts to npm and created a folder there called certbot and mounted a host path of /opt/certbot to /data/certbot it has the same effect. Boots in second now.

This actually really helped me out.
Working in Portainer, I just edited my stack compose.
First added the following
volumes:
  - ./data/opt:/opt/certbot-cp # command will generate the /certbot folder in /data/opt
command: 'cp -avr /opt/certbot /opt/certbot-cp'

waited for the app to output the command response (a ton of lines starting with '/opt/certbot
then changed to
volumes:
  - ./data/opt/certbot:/opt/certbot # added /certbot to host path, removed -cp from container path
# removed command setting

and restarted. npm loaded pretty much immediately, and I was able to access the admin panel.
restarted again just to make sure, and npm loaded in less than 10 seconds.
I'm still learning docker so I may not be doing it the best way, but it worked for me.

So I copied the files to my HDD pool in my /data mount that mounts to npm and created a folder there called certbot and mounted a host path of /opt/certbot to /data/certbot it has the same effect. Boots in second now.

This actually really helped me out.
Working in Portainer, I just edited my stack compose.
First added the following
volumes:
  - ./data/opt:/opt/certbot-cp # command will generate the /certbot folder in /data/opt
command: 'cp -avr /opt/certbot /opt/certbot-cp'

waited for the app to output the command response (a ton of lines starting with '/opt/certbot
then changed to
volumes:
  - ./data/opt/certbot:/opt/certbot # added /certbot to host path, removed -cp from container path
# removed command setting

and restarted. npm loaded pretty much immediately, and I was able to access the admin panel.
restarted again just to make sure, and npm loaded in less than 10 seconds.
I'm still learning docker so I may not be doing it the best way, but it worked for me.

Yep, same thing I did, just using docker compose.

I too am hitting this issue on my TrueNAS system.  How I resolve this for now.
Step 1: Create a PVC mount called /opt/certbot2 (ie. an external mount is managed by TrueNAS - this is not the same as a named mount managed by docker, from docker's perspective it's still a host mount, but it's as good as you get with truenas).
Step 2: Start the container, then log into it via. shell (use heavyscript or truenas to get into the shell)
Step 3: Copy everything in /opt/certbot to /opt/certbot2 (cp -a /opt/certbot/* /opt/certbot2)
Step 4: Stop the container and change the PVC mount from /opt/certbot2 to /opt/certbot
Step 5: Start the container again.
It should now be very quick to chown, because the storage is external, not copy on write.
Of course, this is a terrible solution, because we are now overriding the docker image for what is in /opt/certbot, so as the container updates, our /opt/certbot will not unless you remove the PVC and do the above steps again.

So I copied the files to my HDD pool in my /data mount that mounts to npm and created a folder there called certbot and mounted a host path of /opt/certbot to /data/certbot it has the same effect. Boots in seconds now.

This cant be an solution, only an workaround. Its VERY anoying...at every update of the container or helm forces an downtime for over an hour...

I too am hitting this issue on my TrueNAS system.  How I resolve this for now.
Step 1: Create a PVC mount called /opt/certbot2 (ie. an external mount is managed by TrueNAS - this is not the same as a named mount managed by docker, from docker's perspective it's still a host mount, but it's as good as you get with truenas).
Step 2: Start the container, then log into it via. shell (use heavyscript or truenas to get into the shell)
Step 3: Copy everything in /opt/certbot to /opt/certbot2 (cp -a /opt/certbot/* /opt/certbot2)
Step 4: Stop the container and change the PVC mount from /opt/certbot2 to /opt/certbot
Step 5: Start the container again.
It should now be very quick to chown, because the storage is external, not copy on write.
Of course, this is a terrible solution, because we are now overriding the docker image for what is in /opt/certbot, so as the container updates, our /opt/certbot will not unless you remove the PVC and do the above steps again.

So I copied the files to my HDD pool in my /data mount that mounts to npm and created a folder there called certbot and mounted a host path of /opt/certbot to /data/certbot it has the same effect. Boots in seconds now.

This cant be an solution, only an workaround. Its VERY anoying...at every update of the container or helm forces an downtime for over an hour...

Not a solution, just a workaround for now.

So I copied the files to my HDD pool in my /data mount that mounts to npm and created a folder there called certbot and mounted a host path of /opt/certbot to /data/certbot it has the same effect. Boots in seconds now.

This cant be an solution, only an workaround. Its VERY anoying...at every update of the container or helm forces an downtime for over an hour...

Nobody claimed it was a solution. Hence why the subject is still open.

I too am hitting this issue on my TrueNAS system.  How I resolve this for now.
Step 1: Create a PVC mount called /opt/certbot2 (ie. an external mount is managed by TrueNAS - this is not the same as a named mount managed by docker, from docker's perspective it's still a host mount, but it's as good as you get with truenas).
Step 2: Start the container, then log into it via. shell (use heavyscript or truenas to get into the shell)
Step 3: Copy everything in /opt/certbot to /opt/certbot2 (cp -a /opt/certbot/* /opt/certbot2)
Step 4: Stop the container and change the PVC mount from /opt/certbot2 to /opt/certbot
Step 5: Start the container again.
It should now be very quick to chown, because the storage is external, not copy on write.
Of course, this is a terrible solution, because we are now overriding the docker image for what is in /opt/certbot, so as the container updates, our /opt/certbot will not unless you remove the PVC and do the above steps again.

So I copied the files to my HDD pool in my /data mount that mounts to npm and created a folder there called certbot and mounted a host path of /opt/certbot to /data/certbot it has the same effect. Boots in seconds now.

This cant be an solution, only an workaround. Its VERY anoying...at every update of the container or helm forces an downtime for over an hour...

Not a solution, just a workaround for now.

Ok, sorry if my comment was a bit salty...lets be productive:
Is there an elegant solution for this? What about executing chrown only if PUID/PGID is set or check if the first file under "/data/certbot" has the right permissions and owner and then skip it?

Did anybody ever figure out an answer to the original question of why this is required? Could we simply...remove the offending line?....

Same issue here unfortunately

Same here, Truenas 24.10-Beta.1.
10 to 15 minutes downtime, on an SSD array. I can imagine how frustrating it must be on a pure HDD array.

I think instead of using a hacky workaround, perhaps the underlying issue with whatever causes certbot to complain about can be resolved? Like, if it's to make it not complain about not running as root... isn't there something with the python stuff that lets user packages to  be used?

Workaround add an Additional Environment Variable:
S6_STAGE2_HOOK=sed -i $d /etc/s6-overlay/s6-rc.d/prepare/30-ownership.sh

Workaround add an Additional Environment Variable on Truenas 24.10-Beta.1: S6_STAGE2_HOOK=sed -i $d /etc/s6-overlay/s6-rc.d/prepare/30-ownership.sh

That's indeed the current (sometime buggy) workaround, but that won't resolve the issue itself.
The issue was caused by going from a python user package to a system package (see commit linked in the issue), which, if I understand it well, can only be run as root unless you chown it back, which take ages.
The question is, should we really chown the whole /opt/certbot, or is it possible to do it another way, like chowning only a subset (like only the plugin directory, and only if there is a plugin installed) to reduce the time required, or, ideally, finding a better solution that the commit introducing this was trying to address.

It's only 20 minutes startup time with HDD in a truenas array. WIth the fix, the startup time is 5 seconds.

This happens on straight docker as well on zfs.

Wow, this is annoying. I just installed it on my TrueNAS 24.10 and while I was searching for a solution and writing this text it is still setting ownershop.
I fixed it by setting UID and GID to 0 because everything in /opt/certbot belongs to root:root anyway.

Wow, this is annoying. I just installed it on my TrueNAS 24.10 and while I was searching for a solution and writing this text it is still setting ownershop. I fixed it by setting UID and GID to 0 because everything in /opt/certbot belongs to root:root anyway.

I experienced the same issue, thank you for your solution!

Wow, this is annoying. I just installed it on my TrueNAS 24.10 and while I was searching for a solution and writing this text it is still setting ownershop. I fixed it by setting UID and GID to 0 because everything in /opt/certbot belongs to root:root anyway.

Thank you! Finally, it starts up.


Improvements for WeightedLasso class

[ ] support sparse design matrices and centering
[ ] pass scikitlearn's check_estimator
[ ] in path, keep XtR (to be called gradient later on, for generic datafits) up to date (it's computed at first iteration for WS creation, and before exit to check KKT)
[ ] add support for generic triplet penalty(w, alpha), prox(w[j], alpha, stepsize), kkt_violation(XtR, alpha) to handle any penalty  (maybe we should pass a Penalty jit class instead ?)
[ ] handle generic datafit (needs an objective, a gradient for KKT, and grad_j())
[ ] bench against celer and scikit-learn
[ ] add ***CV estimator
[ ] find new name for class as it will handle enet

@QB3 I have edited the todolist for monday's sprint, feel free to add stuff. I think having a functional enet at the end of the day would be nice and useful for the community !


Add -o/--out-file flag to CLI

Summary
Adds new -o/--out-file option to the fqm-execution CLI.
New behavior
Now, when a user specifies -o/--out-file with a file path, fqm-execution will write the contents of the calculation results to that file instead of console.logging it to stdout. When this happens, a message will be logged to the user with the name of the file path.
If the user specifies the -o/--out-file flag but does not specify a file path, the CLI will default to using output.json as the file path.
If the user does not use this flag, the results will instead be console.logged to stdout.
Code changes
Changes were made to the CLI to add the -o/--out-file flag and to add a function writeToFile() that takes in the file path and a results string and writes the results to the file. The writeToFile() function is called after the results have been calculated. If the flag was not supplied, then the results will be logged to stdout like before.
Changes were made to the README to reflect these changes.
Testing guidance
Run npm run build, then try the following:

Run a command using -o/--out-file with a file path —> check that the file was created and that the appropriate results are in the file
Run a command using -o/--out-file without a file path —> the results should be written to output.json
Run a command without this flag —> the output should be logged to the console
Run a command using -o/--out-file and also add a few console.log statements that will be hit during calculation (can add a few in calculate() in src/Calculator.ts —> the calculation results should appear in the newly created file, and the console.log statements should be logged to the console and they should not be present in the new file

Good catch @natjoe4 , I did that originally and then changed it to have a default value and I forgot to test whether we could still do the console.logs


the test result is 0

Dear author, I configured my training model on MMDetection, and the bbox_mAP_50 of the validation set was preferably 0.883, but the test result was 0.My detailed steps are as follows:
Add and Replace the codes
1.Add folders a_my_configs/ to the configs/ in mmdetectin's codes.
2.Add my_faster_rcnn_r50_fpn_2x_coco.py and my_faster_rcnn_r101_fpn_2x_coco.py to the configs/a_my_configs/.
3.In my_faster_rcnn_r50_fpn_2x_coco.py,   base as same as configs/faster_rcnn/faster_rcnn_r50_fpn_2x_coco.py, add
data=dict(
samples_per_gpu=1,  # batch size
workers_per_gpu=1,  # num_workers
train=dict(path1),val=dict(path2),test=dict(path3))
4.Modify the
student_cfg = 'configs/a_my_configs/my_faster_rcnn_r50_fpn_2x_coco.py'
teacher_cfg = 'configs/a_my_configs/my_faster_rcnn_r101_fpn_2x_coco.py'
in configs/distillers/fgd/fgd_faster_rcnn_r101_fpn_2x_distill_faster_rcnn_r50_fpn_2x_coco.py
Train
python tools/train.py configs/distillers/fgd/fgd_faster_rcnn_r101_fpn_2x_distill_faster_rcnn_r50_fpn_2x_coco.py
The training process went smoothly and the best bbox_mAP_50=0.883
Test
python tools/test.py configs/a_my_configs/my_faster_rcnn_r50_fpn_2x_coco.py work_dirs/fgd_faster_rcnn_r101_fpn_2x_distill_faster_rcnn_r50_fpn_2x_coco/latest.pth --eval bbox
But the test results(bbox_mAP,bbox_mAP_50,bbox_mAP_l……) were 0, I guess the problem appears on the configuration file, but do not know how to solve, looking forward to your reply, I can provide my complete code

You have to transfer the FGD model into MMDet model because they have different keys. You can use pth_transfer.py to transfer it. The details can be seen in README Transfer.

I am so happy to receive your reply. Depending on the method you provided, the testing issue has been solved.
But I still have another question, can I combine two algorithms of different architectures in the mmdetection you provided?
For example, use swin transformer as the teacher model and rnn50 as the student model

I am so happy to receive your reply. Depending on the method you provided, the testing issue has been solved. But I still have another question, can I combine two algorithms of different architectures in the mmdetection you provided?For example, use swin transformer as the teacher model and rnn50 as the student model
-----原始邮件-----
发件人:Mesopotamia @.>
发送时间:2022-03-16 14:34:04 (星期三)
收件人: yzd-v/FGD @.>
抄送: AnLingla @.>, Author @.>
主题: Re: [yzd-v/FGD] the test result is 0 (Issue #9)
Closed #9.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

If the teacher and student have the same H,W of the feature, FGD can be applied successfully. However, transformer and CNN are very different. You can try it by using swin transformer to distill Res-50.

OK，thanks for your advice


image dataset on HDFS

Hi everybody,
i am curious about how TensorFlowOnSpark process the image dataset on HDFS?
I have asked the same question on the CaffeOnSpark repo and it was told that in case of COS one must load the image dataset manually and convert it whether in spark dataframes or sequenceFiles, which could be then used as input for the model. I am curious how is the case here in TFOS?
I hope that someone here would have give me his time and answer my questions, and for that i would be very thankfull :)

After reading this.
I understood that for storing images in HDFS for use with TFOS, they must be in CSV Files or TFRecords, right?
And that loading images into HDFS must be done manually like in CaffeOnSpark.
Now i have another questions how one could do the conversion, is there some kind of template script?
Is it possible to store the images in DataFrames or SequenceFiles. It is just a question of interoperability in the Hadoop/Spark ecosystem. and please how?
I would be very grateful if i could get some more insights in here.


Add 2D ruler tool

Closes half of #27443
Just adds another tool to CanvasItemEditor, for all measuring needs. I didn't want to introduce another color setting, so uses editor's accent color.

It gets much more useful with snapping enabled:

You can use this to e.g. measure distances between points in your level or check whether a ledge is not too high to jump on (I always had to count tiles manually, which is terrible with small cells).
Optional stuff that could be nice (and could be done in another PR):

object snapping (it's not as straightforward as with grid)
angle measuring

I'll also need some icon, because I can't make one >_> Although LineEdit looks almost legit, if not the green color.

Cool feature but I am not sure about how useful it is. It might better suit a plugin I think.

@groud: Are you joking? This is an EXCELLENT feature for practically any 2D project, be it tilemap-based or just sprite based. Counting tiles, for tilemaps, does get dull, and for sprites you either have to manually calculate based on each thing's transform displayed in the inspector, or just eyeball. This would allow me to use actual distances and sizes, instead of just eyeballing them.

This could also be useful for UI design, as you could measure the "real" UI component sizes as they will be displayed (including margins or paddings).

@groud: Are you joking? This is an EXCELLENT feature for practically any 2D project, be it tilemap-based or just sprite based. Counting tiles, for tilemaps, does get dull, and for sprites you either have to manually calculate based on each thing's transform displayed in the inspector, or just eyeball. This would allow me to use actual distances and sizes, instead of just eyeballing them.

Indeed it can help here. :) I am not sure I would use it myself, but if it is useful for a lot of people it should indeed be merged.

Marked as ready for review. The functionality is finished, it works well with zoom, grid snaps and weird cursor positions (like, you start measure then go a kilometer away using pan). The code is complex, but I tried to keep it clear and high quality. It turned out pretty well 😏
What could be improved is adding support for smart snapping (snap_point doesn't seem to do it automatically) and adding angle measuring (would be neat too, but the ruler gets crowded with numbers and it needs special handling). Unless that's necessary to merge this, I'd live it for later, as it's less important IMO (grid alone works pretty well).
Also... icon ;_; It needs a new icon. Yeah.

Thanks!


[2.1] Dictionary copy-on-write can cause invalid memory accesses due to undefined subexpression ordering

Operating system or device - Godot version:
Windows 10 - Godot 2.1 - VS2017
Issue description:
I ran into a problem with an optimized 64-bit build built with VS2017: Capsules were always radius 0.
The problem turned out to be this line in CapsuleShapeSW::set_data:
_setup(d["height"], d["radius"]);
Because C++ doesn't create a sequence point between parameters, the compiler can evaluate the parameters and their subexpressions (including the float conversions) in any order it feels like as long is the dependencies are in order.  The new VS C++ code generator apparently likes to schedule calls of the same function together, so now it emits that line as the equivalent of:
Variant& temp1 = d["radius"];
Variant& temp2 = d["height"];
float temp3 = temp1.operator float();
float temp4 = temp2.operator float();
_setup(temp4, temp3);
"d" is mutable, and indexing a mutable dictionaries always copies and deletes the original if it's unreferenced elsewhere, so by the time temp1 is converted, the lookup of "height" has already invalidated the memory location looked up from "radius", and the case I was seeing, its type was stomped with 0.
So, this might not be a problem in 3.0 since that functionality is gone in master, but it can affect 2.1.  It's hard to really tell how widespread it is, but searching for [" yields a decent number of cases.
The simplest solution I think is to just make dictionaries not copy if their reference count is 1, which would prevent it from relocating its contents after the first access as long as none of the other accesses insert a key.  It should probably do anyway since right now it's doing a copy of the entire dictionary on every mutable access.

Copy-on-write was removed on master, I think we can close this?

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about its current status here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

This doesn't apply to 3.0.

The 2.1 branch is no longer actively worked on by engine developers, unless critical fixes are needed for current games in production. As such we are now closing non-critical bug reports affecting only the 2.1 branch.
Please comment if this was incorrectly triaged and is still relevant in 3.x versions.


NPC Indicator respawn timer would be off if the game froze

I would sometimes tag-all mobs I'm slaying just to see their respawn locations and timers. If my game froze for whatever reason after the respawn timers were set, the timer would usually be too early in comparison to their real respawn. Un-tagging and then retagging them would fix the problem.
Its quite hard to reproduce the problem I imagine, since you need your game to freeze, but its quite simple to understand how to otherwise replicate the problem lol
I'm using windows 10, RL version 1.7.0, launcher version 2.1.5

I don't think this is a bug we can reasonably solve. If your game is freezing, that is the issue which needs to be fixed, not timers being off after said freezing.


Fix demo page

ember-cli-github-pages seems to throw an error when trying to generate the demo app in Ember 2.16, not sure why.

> ember github-pages:commit --message "Deploy gh-pages from commit $(git rev-parse HEAD)"; git push; git checkout -

stdout maxBuffer exceeded
Error: stdout maxBuffer exceeded
    at Socket.onChildStdout (child_process.js:325:14)
    at emitOne (events.js:115:13)
    at Socket.emit (events.js:210:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:246:13)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:595:20)

For now I'll roll back the demo to the previous version.


Feature Request: provide "vscode.executeDocumentSymbolProvider" for range

Disclaimer: In case the feature below already exists, it's difficult to find. In that case, I would ask for improving the documentation (perhaps here: https://code.visualstudio.com/api/references/commands )
Current Situation:
VS Code provides the command vscode.executeDocumentSymbolProvider. The command returns "1st-level symbol information". Example (Python):
From the following code:
class MyClass:
  def foo(self):
     def foo2():
         pass
     return foo2()

only MyClass would be returned (a symbol of the "1st level").
Problems:

A custom extension has no means of finding foo or foo2 (symbols of n-th level)
executeDocumentSymbolProvider runs on the whole document - this might be overkill in certain situations

Suggestion
Provide a command, similar to executeDocumentSymbolProvider, but allow it to specify a range (e.g. executeSymbolProvider). This command would return the following:

Array of:

Symbol information of all 1st-level symbols within the given range.
For each symbol information: a Range specifying the range of its second level

This information is necessary in order to allow an extension to recall the function within the subrange, in order to get sub-level outlining information.

Example (Python)
1  class MyClass:
2    def foo(self):
3       def foo2():
4           pass
5       return foo2()

executeSymbolProvider" with Range  line 1 to line 5 returns:

MyClass, sublevel range: line 2 to 5

executeSymbolProvider" with Range  line 2 to line 5 returns:

foo, sublevel range: 3 to 5

Use Case?
"Why would that be necessary?"
I currently see (only) the use case of any code-outliner like extension.
Notes
edit: VS Code seems to be able to show any arbitrary language outline, if there is a language server registered. So the suggested functionality itself already exists. That means, this feature request is just about creating an API, so extensions can also use that functionality.

I close this FR in favor of the new, more generalized FR here: https://github.com/microsoft/vscode/issues/118094


Pytorch crashes on ARM image

We have currently two errors when scanning images on the arm image:
backend_1   | Current thread 0x0000007fad883150 (most recent call first):
backend_1   |   File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/conv.py", line 439 in _conv_forward
backend_1   |   File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/conv.py", line 443 in forward
backend_1   |   File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1051 in _call_impl
backend_1   |   File "/code/wideresnet.py", line 143 in forward
backend_1   |   File "/code/api/places365/places365.py", line 140 in inference_places365
backend_1   |   File "/code/api/models/photo.py", line 204 in _generate_captions
backend_1   |   File "/code/api/directory_watcher.py", line 120 in handle_new_image
backend_1   |   File "/code/api/directory_watcher.py", line 219 in photo_scanner
backend_1   |   File "/usr/lib/python3.9/multiprocessing/pool.py", line 51 in starmapstar
backend_1   |   File "/usr/lib/python3.9/multiprocessing/pool.py", line 125 in worker
backend_1   |   File "/usr/lib/python3.9/multiprocessing/process.py", line 108 in run
backend_1   |   File "/usr/lib/python3.9/multiprocessing/process.py", line 315 in _bootstrap
backend_1   |   File "/usr/lib/python3.9/multiprocessing/popen_fork.py", line 71 in _launch
backend_1   |   File "/usr/lib/python3.9/multiprocessing/popen_fork.py", line 19 in __init__
backend_1   |   File "/usr/lib/python3.9/multiprocessing/context.py", line 277 in _Popen
backend_1   |   File "/usr/lib/python3.9/multiprocessing/process.py", line 121 in start
backend_1   |   File "/usr/lib/python3.9/multiprocessing/pool.py", line 326 in _repopulate_pool_static
backend_1   |   File "/usr/lib/python3.9/multiprocessing/pool.py", line 337 in _maintain_pool
backend_1   |   File "/usr/lib/python3.9/multiprocessing/pool.py", line 513 in _handle_workers
backend_1   |   File "/usr/lib/python3.9/threading.py", line 892 in run
backend_1   |   File "/usr/lib/python3.9/threading.py", line 954 in _bootstrap_inner
backend_1   |   File "/usr/lib/python3.9/threading.py", line 912 in _bootstrap
backend_1   | INFO:ownphotos:job 219f17da-92f0-4c28-abdd-1450ff56a69c: handling image /data/368037759.JPG
backend_1   | Fatal Python error: Illegal instruction

Current thread 0x0000007fad883150 (most recent call first):
  File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/conv.py", line 439 in _conv_forward
  File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/conv.py", line 443 in forward
  File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1051 in _call_impl
  File "/code/wideresnet.py", line 143 in forward
  File "/code/api/places365/places365.py", line 140 in inference_places365
  File "/code/api/models/photo.py", line 204 in _generate_captions
  File "/code/api/directory_watcher.py", line 120 in handle_new_image
  File "/code/api/directory_watcher.py", line 219 in photo_scanner
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 51 in starmapstar
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 125 in worker
  File "/usr/lib/python3.9/multiprocessing/process.py", line 108 in run
  File "/usr/lib/python3.9/multiprocessing/process.py", line 315 in _bootstrap
  File "/usr/lib/python3.9/multiprocessing/popen_fork.py", line 71 in _launch
  File "/usr/lib/python3.9/multiprocessing/popen_fork.py", line 19 in __init__
  File "/usr/lib/python3.9/multiprocessing/context.py", line 277 in _Popen
  File "/usr/lib/python3.9/multiprocessing/process.py", line 121 in start
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 326 in _repopulate_pool_static
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 337 in _maintain_pool
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 513 in _handle_workers
  File "/usr/lib/python3.9/threading.py", line 892 in run
  File "/usr/lib/python3.9/threading.py", line 954 in _bootstrap_inner
  File "/usr/lib/python3.9/threading.py", line 912 in _bootstrap
INFO:ownphotos:job 219f17da-92f0-4c28-abdd-1450ff56a69c: handling image /data/368037759.JPG
Fatal Python error: Illegal instruction

This seems to be an error regarding pytorch on arm64. Maybe this will be fixed in the future, or we have to try older versions of pytorch.
A fast way to fix this issue would be to implement deactivating the torch related features as outlined here #255

There seems to be a difference between the arm specs.