Internal compiler error. Aborting!

System overview:
Ubuntu 18.04
TF-GPU 1.15 installed from binary
Problem:
I am trying to compile a quantized TFlite model which was converted from a frozen graph enabling pose-estimation(openpose). I was able to generate the fully quantized tflite model, but unable to compile it using the edgetpucompiler. As the TF model was generated using TF1.x I made use of the tfliteconverter from TF1.15. Below is the code that I used for the conversion :
`import tensorflow as tf
import numpy as np
def representative_dataset_gen():
for _ in range(100):
fake_image = np.random.random((1,432,368,3)).astype(np.float32)
yield [fake_image]
graph_pb = 'graph_freeze.pb'
inp = ['image']
out = ['Openpose/concat_stage7']
converter=tf.lite.TFLiteConverter.from_frozen_graph(
graph_pb, inp, out,input_shapes={"image":[1,432,368,3]})
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.representative_dataset = representative_dataset_gen
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
converter.inference_input_type = tf.uint8
converter.inference_output_type = tf.uint8
tflite_model = converter.convert()
f = open("tflite_model/mobilenet_thin_openpose_opt_fullint_tf1.tflite", "wb")
f.write(tflite_model)
f.close()
print("conversion complete")`
I then tried to compile this and I'm just getting the following error :

edgetpu_compiler mobilenet_thin_openpose_opt_fullint_tf1.tflite
Edge TPU Compiler version 2.1.302470888
Internal compiler error. Aborting!

I tried going through similar issues but could not find a proper solution. It would be very nice if I could get through with the issue.

@Eashwar93
Apologies, most compiler issues are reported to our internal teams for evaluation and then fixes. Since the compilers are not open source at this time, the fixes are usually delayed until each release, this is why you may have found some issues that are not fix.
Do you mind trying this solution at the moment?

@Namburger Thanks. I actually feel that it is a long way around , though I use the same tf-lite converter except for the fact that he converts it from a keras model , but I convert it directly from a frozen graph. Theoretically there should not be any problem as at the end I have a fully quantized tflite model to be passed on to the edgeTPU compiler.

@Eashwar93 could you share the cpu model, I can take a look at it. In general, I don't think there should be difference between graph vs .h5 but we do have limited compatibilities at the moment, something we are working to expand

@Namburger Sure

@Eashwar93 sorry, I mean the cpu's tflite model, not the cpu info

@Namburger Oops sorry my bad.
https://drive.google.com/file/d/1BZ0gVX00Vnqhthx_wxmMXQAbkisSAftN/view?usp=sharing

@Eashwar93 taking a look at this now, sorry that our compiler currently doesn't give much error message, I know it's frustrating. From my knowledge, you are using tf1.15 for ptq, correct?

yes tf1.15 for ptq

@Namburger it is completely fine :) Yes I use tf-1.15 for ptq

@Eashwar93 Are you familiar with the visualized tools or netron to visualize your model?
There is a layer name Openpose/MConv_Stage3_concat with 2 preceeded layers with mismatching quantization parameters, this causes the compiler to rejects the entire model. This is most likely a bug with the tflite converter tool, could you open a bug here also?
On another note, maybe you could try using tf2.* for ptq (maybe the above bug is fixed). PTQ with tf2.* will cause your models i/o tensors to be of type float. We are still in experimental stage of supporting this but the compiler should allows float i/o now, here is an example usage in the form of test! If it still fails, please reattach the model so I can check again. Apologies for this issue and please also checkout project-posenet.

@Namburger Ah ok thanks a ton. I will look into it. I will open a bug in the tensorflow git as well. I tried using netron now and could not find any quantization parameters there. I am trying to use visualizer.py now. Once I can pin point the issue in TFLite converter I will start a Git thread at Tensorflow.
Also I tried to convert with Tf2.1, there was an issue which I possibly think is due to the fact that this Network which I'm converting uses tf.contrib which is no longer supported in tf2.x I guess. I'm not sure if that's the issue but it's the most likely I can think off.
I also checked out your project posenet. It's great, I wanted to compare it with Openpose which is why I'm embarked on this task.
Thanks a lot for your help. I will keep you posted. :)

@Namburger I went through the model and I'm unable to spot what is wrong actually. Apologies that if my question is too trivial but I'm new to this entire framework. So you were referring to the tensor Openpose/MConv_Stage3_concat. I took a look at it.

So then I went to the ops section to see the inputs for the output tensor [217] and found them to be
387 Openpose/MConv_Stage2_L1_5_pointwise/BatchNorm/FusedBatchNorm_requantized INT8 [1, 54, 46, 38]
388 Openpose/MConv_Stage2_L2_5_pointwise/BatchNorm/FusedBatchNorm_requantized INT8 [1, 54, 46, 19]
389 feat_concat_requantized INT8 [1, 54, 46, 864]:

When I checked the quantization params of these input tensors, there were no mismatch as shown below.

Apologies again, but I really want to solve this issue.
Thanks

@Eashwar93 no worries, I found the visualize output is not super intuitive to read :/
These are the offenders:
387 | Openpose/MConv_Stage2_L1_5_pointwise/BatchNorm/FusedBatchNorm_requantized | INT8 | [1, 54, 46, 38] | None | 0 | {'quantized_dimension': 0, 'scale': [0.12664233], 'min': None, 'max': None, 'details_type': 0, 'zero_point': [-128], 'details': None}

390 | Openpose/MConv_Stage1_L1_5_pointwise/BatchNorm/FusedBatchNorm_requantized | INT8 | [1, 54, 46, 38] | None | 0 | {'quantized_dimension': 0, 'scale': [0.12680078], 'min': None, 'max': None, 'details_type': 0, 'zero_point': [-127], 'details': None}

The problem is that AFAIK there isn't a way to fix this on the user side, did you check the model's accuracy after the tflite conversion?

@Namburger Thank you. I guess with this info I can create a bug with the TF team. No I haven't tested the model after conversion. Will do that as well probably in python just to test. As of now I'm trying to explore posenet in C++

@Namburger I am running into similar issues with my model (trained and quantized with TF-2.4) and I am trying to understand how to locate such mismatches. I've inspected the model @Eashwar93 attached to this ticket using Netron but was not able to find the mismatch you found.
Inspection of the tensor Openpose/MConv_Stage3_concat
inputs:
name: Openpose/MConv_Stage2_L1_5_pointwise/BatchNorm/FusedBatchNorm_requantized
type: int8[1,54,46,38]
quantization: 0.12664233148097992 * (q - -128)
location: 387
name: Openpose/MConv_Stage2_L2_5_pointwise/BatchNorm/FusedBatchNorm_requantized
type: int8[1,54,46,19]
quantization: 0.12664233148097992 * (q - -128)
location: 388
name: feat_concat_requantized
type: int8[1,54,46,864]
quantization: 0.12664233148097992 * (q - -128)
location: 389
shows that the quantization is consistent and the locations are 387, 388 and 389.
The tensor in location 392 (that you've mention is no compatible with 387) is used in Openpose/MConv_Stage2_concat together with 390 and 391.
name: Openpose/MConv_Stage1_L1_5_pointwise/BatchNorm/FusedBatchNorm_requantized
type: int8[1,54,46,38]
quantization: 0.1268007755279541 * (q - -127)
location: 390
name: Openpose/MConv_Stage1_L2_5_pointwise/BatchNorm/FusedBatchNorm_requantized
type: int8[1,54,46,19]
quantization: 0.1268007755279541 * (q - -127)
location: 391
name: feat_concat_requantized
type: int8[1,54,46,864]
quantization: 0.1268007755279541 * (q - -127)
location: 392
I was not able to find where the two tensors you've mentioned are being concatenated.
Can you please elaborate how you found the issue?


Entrust Middleware: Argument 1 passed to Zizaco\Entrust\Middleware\EntrustPermission::__construct() must be an instance of Illuminate\Contracts\Auth\Guard, none given,

Edit: Fixed
Happens when I'm trying to seed my database.
Error:
[ErrorException]
  Argument 1 passed to Zizaco\Entrust\Middleware\EntrustPermission::__construct() must be an instance of Illuminate\Contracts\Auth\Guard, none given, called in /database/seeds/PermissionsSeeder.php on line 32 and defined

PermissionSeeder.php:
    public function run()
    {
        $admin = new \App\Models\Role();
        $admin->name = 'admin';
        $admin->display_name = 'Admin';
        $admin->description = 'The admin of the site';
        $admin->save();

        $siteAdmin = \App\Models\User::find(1);
        $siteAdmin->attachRole($admin);

        $isAdmin = new \App\Models\Permission();
        $isAdmin->name = 'is-admin';
        $isAdmin->display_name = 'Is a admin';
        $isAdmin->display_name = 'Is an admin of this site';
        $isAdmin->save();

        $admin->attachPermission($isAdmin);
    }

Kernel.php:
     */
    protected $routeMiddleware = [
        'auth' => \App\Http\Middleware\Authenticate::class,
        'auth.basic' => \Illuminate\Auth\Middleware\AuthenticateWithBasicAuth::class,
        'guest' => \App\Http\Middleware\RedirectIfAuthenticated::class,
        'role' => \Zizaco\Entrust\Middleware\EntrustRole::class,
        'permission' => \Zizaco\Entrust\Middleware\EntrustPermission::class,
        'ability' => \Zizaco\Entrust\Middleware\EntrustAbility::class,
    ];

Version 1.4.1 is broken, and it seems that 1.5.0 is also broken. Or am I doing something wrong?

Seems like namespaced my Permission model wrong. Instead of using Zizaco\Entrust\EntrustPermission I used the middleware file. (Yay for phpstorm auto importing)
If someone can close this issue I'm happy. :)

I just had the same problem. Thx mate :+1:


update gjson version

@u2takey thank you for all of your work on this project! this pr updates the gjson dependency to avoid a DDoS vulnerability for versions < 1.6.6 https://snyk.io/vuln/SNYK-GOLANG-GITHUBCOMTIDWALLGJSON-1056415

@u2takey thank you for all of your work on this project! this pr updates the gjson dependency to avoid a DDoS vulnerability for versions < 1.6.6 https://snyk.io/vuln/SNYK-GOLANG-GITHUBCOMTIDWALLGJSON-1056415

tks


🐛 Bug: eslint does not report problems on tasks.json in VS Code

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Expected
I created a new repo yesterday using create-typescript-app using the "everything" setting (woo!). I set up a few tasks in VS Code's tasks.json and pushed a PR, at which point I noticed that I was getting some eslint errors because of unsorted keys in my tasks.json.
These were reported via GitHub Actions CI and when I run pnpm lint, but they were not reported in VS Code like lint errors in .ts files are.
Actual
I would expect eslint issues in tasks.json to be reported in VS Code (so that I can quick fix them!).
Additional Info
I noticed that "json" is in the eslint.probe list in .code/settings.json, but maybe it also needs to be told about JSON files some other place?
Here's the PR and repo: https://github.com/danvk/gravlax/pull/3

@all-contributors please add @danvk for bug.

🤖 Beep boop! This comment was added automatically by all-contributors-auto-action.
Not all contributions can be detected from Git & GitHub alone. Please comment any missing contribution types this bot missed.
...and of course, thank you for contributing! 💙

Hmm, looks like using eslint.probe for jsonc files isn't yet supported in vscode-eslint: https://github.com/microsoft/vscode-eslint/issues/1786. Marking as blocked pending discussion there.
Worst case scenario, we can go back to eslint.validate. But that's described as old and legacy so I'd rather not for a bit.


include definitions and not just descriptions

@UbuntuScott @DavidCrosswellGSQ Just realised that the document-licence vocabulary (ie creative commons) didn't include a skos:definition for 2 of the 3 licence types.
Basic schoolboy error. Hoping this is a super quick review.

@DavidCrosswellGSQ just fixed that. Thanks for pointing that out. And our preference is to use a definition and not a description.

This vocab is now refreshed in VocPrez


feat: update metadata fields from the challenger content

Description

Related Issue
Fix #23

Motivation and Context

How Has This Been Tested?
Tested with a customer on their live website where they were relying on the template meta property for how things rendered.

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@shsteimer Thanks for the contribution! We've started having use cases around the Commerce Boilerplate for this, so merged it on our end in a dedicated branch they can use.


Conflicts with existing KVM and DNS

I had a first try and found out that it conflict with my pre-existing  virbr1.
As well, I already had named.service and the playbook is trying to change it.
Finally, I am running with Fedora 32, again another change to the playbook.
__
Ease of use seems to be there only when starting with a fresh install of the host.
I put this aside until I have more time.
Not a simple 30 minutes in my case.

Hi, I already changed the default name of the bridge used for the guests @ https://github.com/ccamacho/kubeinit/blob/master/kubeinit/playbooks/okd.yml#L127
It should not conflict with an existing bridge, the named service is configured in the services VM so it should not conflict.


[2.1.1] Websocket connection closes with error code 1007

Consider taking https://github.com/aspnet/SignalR/issues/2323 for 2.1.1

This is a tracking issue, we just need to track that https://github.com/dotnet/corefx/pull/29840 goes in to 2.1.1

https://github.com/dotnet/corefx/pull/29853

The corefx issue was approved by Shiproom. Leaving this open until we confirm it's in and merged. @BrennanConroy do you have any test code you could run to verify that the fix is in 2.1.1 when we get a build?

Yes

Verified that the fix is in


WIP

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.41-alpha.ea30b6.0@codecademy/gamut-labs@10.0.7-alpha.ea30b6.0@codecademy/gamut@23.0.1-alpha.ea30b6.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602c22b0d423b513f85d1872--gamut-preview.netlify.appDeploy Logs


GetDnsDomains() not defined in interface SoftLayer_Account_Service

I noticed that method GetDnsDomains() is not defined in interface SoftLayer_Account_Service (softlayer_account_service.go), but implemented in softlayer_account.go. In this case, GetDnsDomains() can't be invoked from a SoftLayer_Account_Service interface object, right?
If you think it's worth adding the method, I can contribute a PR and unit tests.

Hi, @Huang-Wei, yes this could be useful. However, please add it as GetDomains() (change current to that name) since the official name is getDomains from the SL API docs
Also, please make sure to add some unit tests and possibly an integration test. The last one (integration test) can be tricky but are important since that's the only way to make sure code works OR to manually test it.
Let me know if you need help. Thanks in advance for contribution.

Thanks @maximilien, I've prepared a PR, please review.
BTW: I found some other issues in current integration tests:
domain test should expect an error when domain has been deleted
Look at https://github.com/maximilien/softlayer-go/blob/master/test_helpers/test_helpers.go#L701, it expected no errors. And I've changed ToNot to To in the PR.
test.domain.records.name already exists
I guess it was caused by somebody running domain tests. And I can confirm it's not under my sl account:
⇒  curl<EMAIL_ADDRESS>[]%                                                                                hweicdl@wei-mbp-ibm:~/gospace/src/github.com/maximilien/softlayer-go|master⚡
⇒  curl<EMAIL_ADDRESS>[]%

We can change test.domain.records.name to delme.domain.records.name to fix this if we cannot find who in deed holds this domain entry.
(BTW: I didn't include this change in PR)

@Huang-Wei sounds reasonable. Please update PR with these. I will review and comment and/or merge. Best.

got it, commented in PR: https://github.com/maximilien/softlayer-go/pull/126#issuecomment-239336263


Does this really require the exact versions for rouge 3.26+, zeitwerk 2.5.0+, yard 0.9.24+, htmlentities 4.3.4+?

Our production ecosystem is based on debian and not on rubygems. Our development systems are basically a VM of the production environment (think docker, but then just a stable debian install). This means that we cannot install gems from RubyGems on production, and shouldn't install gems on development, with the upside being that any gem we install via debian gets automatic security updates, and that our development environments exactly match the production environment.

For example
$ apt-get install ruby-css-parser

[...]

The following NEW packages will be installed:
  ruby-css-parser
0 upgraded, 1 newly installed, 0 to remove and 40 not upgraded.

[...]

Setting up ruby-css-parser (1.6.0-1) ...

This installs the package in a system location and can now be found when using gem 'css-parser'
$ dpkg -L ruby-css-parser

/.
/usr
/usr/lib
/usr/lib/ruby
/usr/lib/ruby/vendor_ruby
/usr/lib/ruby/vendor_ruby/css_parser
/usr/lib/ruby/vendor_ruby/css_parser/parser.rb
/usr/lib/ruby/vendor_ruby/css_parser/regexps.rb
/usr/lib/ruby/vendor_ruby/css_parser/rule_set.rb
/usr/lib/ruby/vendor_ruby/css_parser/version.rb
/usr/lib/ruby/vendor_ruby/css_parser.rb
/usr/share
/usr/share/doc
/usr/share/doc/ruby-css-parser
/usr/share/doc/ruby-css-parser/README.md
/usr/share/doc/ruby-css-parser/changelog.Debian.gz
/usr/share/doc/ruby-css-parser/changelog.gz
/usr/share/doc/ruby-css-parser/copyright
/usr/share/rubygems-integration
/usr/share/rubygems-integration/all
/usr/share/rubygems-integration/all/specifications
/usr/share/rubygems-integration/all/specifications/css_parser-1.6.0.gemspec

There are many gems maintained by the debian ruby team, but of course a gazillion more that are not in debian. For example, view_component (2.55.0) doesn't exist on stable, but will be added in the next version (~1.8 years from now). For these gems, if the dependencies allow for it, they can be added in the vendor folder.
gem 'view_component', path: File.join(__dir__, 'vendor', 'view_component-2.55.0')

This works because the two runtime dependencies view_component has exist on debian!

activesupport (>= 5.0.0, < 8.0) as ruby-activesupport (<IP_ADDRESS>)
method_source (~> 1.0) as ruby-method-source (1.0.0)

This means that installing those packages first, followed by vendoring the gem can make a working Gemfile.lock using bundle install --local. Swell!

lookbook lists the following:

required_ruby_version >= 2.7.0 matching ruby (2.7)
css_parser matching ruby-css-parser (6.0)
actioncable matching ruby-actioncable (<IP_ADDRESS>)
activemodel matching ruby-activemodel (<IP_ADDRESS>)
zeitwerk ~> 2.5.0 NOT matching ruby-zeitwerk (2.4.2)
railties ~> 5 matching ruby-railties (<IP_ADDRESS>)
view_component ~> 2.0 missing
redcarpet ~> 3.5 matching ruby-redcarpet (3.5.1)
rouge >= 3.26, < 5.0 NOT matching ruby-rouge (3.21.0)
listen ~> 3.0 matching ruby-listen (3.2.1)
yard ~> 0.9.25 NOT matching yard (0.9.24)
htmlbeautifier ~> 1.3 missing
htmlentities ~> 4.3.4 NOT matching ruby-htmlentities (4.3.3)

Most of these versions were added in this commit: https://github.com/ViewComponent/lookbook/commit/77d14397cc9fe63e191a0930629c3c344aafd7a9, which seems to "just add" versions as they appeared in the lock, and not be the actual dependent versions.
Describe the solution you'd like
I'd like to know if the version added were added with a reason other than having versions in there or not. If there is an actual feature that was added in rouge after 3.21 that is missing in 3.21, or zeitwerk` 2.5.0, that is missing in 2.4.2, we won't be able to use lookbook, but in every other case:
yard has had a security update, so the security patch from 0.9.25 made it to the 0.9.24 version in debian. The same is true for htmlentities. The missing gems htmlbeautifier and view_component can be vendored because either they don't have dependencies, or all dependencies are matching in debian or can be vendored themselves.
Describe alternatives you've considered
Going down the rabbit hole myself before asking.
Additional context
If you confirm that these versions were just added because it's the versions you know of, I will go through the motion of testing lookbook with the lower versions (with the debian packages). You don't need to lower the gemspec versions or anything, although be aware that the gemspec versions should be the minimum version required, or at least that's how I understood gem development to be, and how I deal with it in my own gems.

Hey @SleeplessByte thanks for opening this issue - I'm always amazed at how different everyone's setups are but I'd love to help get this working for you.
In answer to your question, there is no particular reason (that I can remember!) that the gem dependencies need to be tied to those specific versions. Unfortunately my commit message on that commit is spectacularly unhelpful in helping remember but I think at the time I was just trying to ensure I had a know, workable set of dependency versions to build on. Lookbook was somewhat in it's infancy then :-)
I'm pretty sure they could be lowered to meet your requirements without any side effects. Would you be interested in opening a PR to lower the versions to better match your requirements to unlock this for you? I'd be happy to then double check everything to make sure it's not causing any issues.
Then when I have a bit of time I can see if I can drop those version still lower to help others that might be in similar situations.
Let me know what you think :-)

Oh sweet.
Yeah, I'll setup a fork with the lowered versions so they would be good to go with our setup. FWIW I won't need you to lower the versions in the repo as well, as this isn't packaged yet so I'd just do that before packaging or vendoring -- but of course you could!

Okay great, nice one - there would be no harm in lowering the versions once they are confirmed to work, so if you get your branch running ok then I might as well match the spec in here I think :-)
Let me know if I can do anything else to help!

@SleeplessByte I'm going to close down this issue now, if you end up putting together a PR we can always continue any discussion in there - or feel free to reopen if needed!

Yep, thanks!
Currently running a few live events, but will pick this up after. See you in the future!

Fast forward to 2024. I got extremely ill and hospitalised for almost a year, so I wasn't able to pick this up but "luckily" I no longer need to.
Here to report that this made it into current stable @ 2.74, and the current version @ 3.12 is in the next version of debian (testing).
No further work needs to be done from your end.

Fast forward to 2024. I got extremely ill and hospitalised for almost a year, so I wasn't able to pick this up but "luckily" some things have happened.
Here to report that this made it into current stable @ 2.74, and the current version @ 3.12 is in the next version of debian (testing).
No further work needs to be done to get view components into Debian.
As for lookbook, the dependencies have changed over time. I'll see if this now runs, and if not I'll let you know which things to (potentially) change.

I got lookbook 2.3.2 working on debian stable (bookworm) by making the following changes:
- spec.add_dependency "htmlentities", "~> 4.3.4"
+ spec.add_dependency "htmlentities", "~> 4.3.3"

In order to use system packages only, I needed to then:
group :development do
  gem 'htmlbeautifier', path: File.join(__dir__, 'vendor', 'htmlbeautifier-1.4.3')
  gem 'htmlentities', '4.3.3'
  gem 'lookbook', path: File.join(__dir__, 'vendor', 'lookbook-2.3.2')
end

Finally, vendor that version of htmlbeautifier in the vendor folder and install htmlentities:
apt-get install ruby-htmlentities

All the other dependencies are already installed by other gems such as rails. Now bundle install --local succeeds.
Leaving this here for a future reader. As for this project, you may want to consider to downgrade the gemspec for htmlentities to ~> 4.3.3, as the next version of debian currently still includes that non-latest version.


How to use show_result() function in detectors/rpn.py?

I want to only use RPN and show RPN proposals on a image. Could somebody please teach me how to use this api? I want some codes just like example about testing image in readme.md.
thx!

You may refer to https://github.com/open-mmlab/mmdetection/blob/master/mmdet/models/detectors/rpn.py#L79

You may refer to https://github.com/open-mmlab/mmdetection/blob/master/mmdet/models/detectors/rpn.py#L79

@hellock Thanks for your reply! Yes, I saw this function. But I don't know how to use it: How to construct a pure RPN model and load checkpoint? And how to prepare data and result this show_result() function need?
So I am wondering if you could provide some example code for using this function? Just like example about testing faster r-cnn detector on an image in readme.md.
Thank you very much!

I guess this api can be used in tools/test.py. Command input 'python3 tools/test.py configs/any_rpn_config_path.py any_rpn_checkpoint.pth' and program will call show_result() function of rpn automatically.


Improve prefixes description visibility

CIDR notation is not that human readable so when you have a lot of prefixes and children prefixes, descriptions are really helpful to understand your network infrastructure.
Current displays kind of hide descriptions instead of highlighting it. This patch tries to fix that.
Signed-off-by: Sylvain Rabot<EMAIL_ADDRESS>
Fixes:

Descriptions are already listed in a column on the table. There is no reason to concatenate them with the prefix.
In the future, please open a feature request before submitting a pull request, as indicated by the pull request template.


Bump Github Actions to use Node20

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

@gmllt can you please have a look into this PR? Thanks


model.create - primaryKey

I have below model with primary key id:
attributes: {
id: {
type: 'integer',
autoIncrement: true,
primaryKey: true,
unique: true
},
name: {
type: 'string',
unique: true,
required: true
},
}
I am creating model as below:
var model = {
id: undefined,
name: 'name',
};
waterlinemodel.create(model).exec(function(error, result) {});
But it throws below error:
Error (E_UNKNOWN) Encountered an unexpected error] Details:  error: null value in column "id" violates not-null constraint
As, 'id' is a primary key, waterline should not look at what is the value of 'id' property.
How to resolve this error? I do not want to remove 'id' because I have created value object for the model and it contains all the attributes of model.I am setting value object property as I need. I do not need to set id property for creation.

Hi @nitulkukadia, instead of setting id: undefined have you tried not setting id at all? Which adapter are you using?

According to java script. If I do not define id or i assign value as undefine both are same. Still it gives me an error.

That's not true, if you do something like:
var obj = {
  id: undefined,
  someProperty: 1
};
for(var key in obj) {
  console.log(key, obj[key];
}

You'll see that setting id to undefined it's different than not setting id.
Anyway you haven't told us which adapter you're using.

I am not using any adapter. I am just using a waterline model method to create model.
But blow code prints undefined for id and undefinedProperty.
var obj = {
id: undefined,
someProperty: 1
};
console.log(obj.id);
console.log(obj.undefinedProperty);

@nitulkukadia Just because console.log prints the same value does not mean and undefined value and a missing value are the same in Javascript. They are very different.

As, 'id' is a primary key, waterline should not look at what is the value of 'id' property.

Are you filing this issue because you honestly want to set an actual database id to undefined? This is impossible, and is not something any ORM is going to support.

@tjwebb I think that @nitulkukadia wants waterline to strip undefined properties.

A primary key cannot be undefined, or missing
Either way though, magically stripping undefined values would cover up a
lot of legit validation errors. If an undefined value makes some validation
fail, then I think it should fail.
On Apr 10, 2015 7:53 AM, "devin ivy"<EMAIL_ADDRESS>wrote:

@tjwebb https://github.com/tjwebb I think that @nitulkukadia
https://github.com/nitulkukadia wants waterline to strip undefined
properties.
—
Reply to this email directly or view it on GitHub
https://github.com/balderdashy/waterline/issues/943#issuecomment-91529216
.

Yes, agreed!

@tjwebb   If an undefined value makes some validation
fail, then I think it should fail.
Yes it should fail the validation, you are right.
But I think:
Update and delete operation can be performed by deletion based on id.
But, while creating model record in database, primary key never exist. Primary key created only after inserting record in database.
So, while creating, why waterline is looking for primary key.

But, while creating model record in database, primary key never exist. Primary key created only after inserting record in database.

You're fundamentally mistaken about how databases work. There exists no database where you can define a PRIMARY KEY field and be able to insert a null/undefined value.

I think @nitulkukadia is saying that it's some sort of auto-increment / uuid column that will be assigned at the time of record creation by the database itself.  That said, I still think waterline needs to report a failure if the id is specified but not actually set.  After all, even in an auto-incrementing column you can often set the primary key manually.

After all, even in an auto-incrementing column you can often set the primary key manually.

Depends on the database. You cannot do this in SQL Server, for example. But even where you are allowed to manually set it, you still cannot ever set it to null.


of total addresses with positive STX balances

Overview
Currently, we have 21 Telegram Communities and 52 Meetup Groups in more than 30 countries. Overall, the Stacks ecosystem has more than 50,000 dedicated community members. We're planning to give the community an OKR for 2021 to accelerate the growth of STX.
Is your feature request related to a problem? Please describe.
It was always hard to measure a community's impact. A clear definitive goal on what success looks like is needed and we think the # total Stacks Addresses w/ positive STX balances is one way to measure their attribution.
Describe the solution you'd like

Display the total number of addresses with positive STX balances
Display the # of TVL with Stacking
Display the total number of transactions processed lifetime

Additional context
You can see how other projects use this as a sign of adoption within the crypto space. Adding some social proofs below:

https://defipulse.com/ - For TVL, adding it in the explorer's API will make it easier to integrate as well.
https://twitter.com/XVGhulk/status/1360970529377812487

This could be considered as an addition to the high-level information described here https://github.com/hirosystems/explorer/issues/345

Closing it in favor of #345


Django: Re-order tarot keys according to number class object in Generators

The carousel tarot cards appear in random order. Instead they should ascend in sequential order (0 through 21) accomplished by using the number class object specified in generators/models.py

Issue #31 resolved  in PR #49


Update Word.Documents.CheckOut.md

It falsely says expression represents a Document object, but it actually represents a Documents object.

Docs Build status updates of commit cf25312:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Word.Documents.CheckOut.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@PhiGegner Thanks for catching this.

@PhiGegner In the future, please set your pull requests to merge to MASTER, not LIVE. Thanks.


v0.1.0 - Base package for AutoMLPipeline.jl and TSML.jl

@JuliaRegistrator register()

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/22956
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" c738a18051047e5e34a7c83b9b3aa2da953f7144
git push origin v0.1.0

@JuliaRegistrator register()

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/22956
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" e38b0b0eae1a53bb6e55521e67c661033a1704e6
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/22956
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" a6d21e90a33653d29bdf7098aa47ef1067f8341e
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/23385
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" e779c747738cc533b3ca811ce3870f234ab7b935
git push origin v0.1.0

[noblock]
ok, renamed it.

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/23385
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 76662117f34e4fc7796f1fe8fa00376e62ef9c10
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/23385
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" ff6af48bc56d2f9968d4d659c0bdff03d5804832
git push origin v0.1.0


Support AZURE_CONFIG_DIR

Plugin name and version
azure-webapp-maven-plugin 1.6.0
Plugin configuration in your pom.xml
            <plugin>
                <groupId>com.microsoft.azure</groupId>
                <artifactId>azure-webapp-maven-plugin</artifactId>
                <version>1.6.0</version>
                <configuration>
                    <allowTelemetry>false</allowTelemetry>
                    <appName>${deployment.appName}</appName>
                    <resourceGroup>${deployment.resourceGroup}</resourceGroup>
                    <pricingTier>F1</pricingTier>
                    <javaVersion>1.8</javaVersion>
                    <stopAppDuringDeployment>true</stopAppDuringDeployment>
                </configuration>
            </plugin>

Expected behavior
The Maven plugin uses the right configuration directory, set via AZURE_CONFIG_DIR as described in the documentation for Azure CLI.
Actual behavior
The plugin always uses the configuration in $HOME/.azure if present.
If that is missing, an error is thrown:
[INFO] Authenticate with Azure CLI 2.0
Cannot read files .../.azure/azureProfile.json and .../.azure/accessTokens.json. Are you logged in Azure CLI?

Steps to reproduce the problem

Ensure $HOME/.azure is missing (e.g., move it away, or delete it)
Set AZURE_CONFIG_DIR to something else than $HOME, e.g., export AZURE_CONFIG_DIR="$(mktemp)" (under Linux in a Bash)
Run mvn azure-webapp:deploy

As far as I can tell, the Maven plugin uses System.getProperty("user.home") to find the Azure CLI configuration files in two places:

src/main/java/com/microsoft/azure/maven/AbstractAzureMojo.java
src/main/java/com/microsoft/azure/maven/auth/AzureAuthHelper.java

It might be feasible to replace that with a helper method that looks up System.getenv(AZURE_CONFIG_DIR) first with a fallback to System.getProperty("user.home").

FWIW, I made a quick hack an created PR #674 for this issue. I hope that is okay.

@alexandrospanagiotidis Thanks for your report and your contribution! Maven Plugin use  $HOME/.azure as the config folder to save some configurations, and we use AzureCliCredentials.create() to get the Authenticated object from Azure CLI, which uses $HOME/.azure as the path for cli config folder. I've made an issue here.
Codes which refers $HOME/.azure in AzureAuthHelper is the workaround to solve AzureCliCredentials can't get the correct subscription in azure cloud shell or cli authed with service principal, we will fix them once the issue is fixed.

Hi @Flanker32, thank you for your reply. Should I delete my PR, since this is something that needs to be fixed via the other issue?

@alexandrospanagiotidis Yes, thanks for your support, I'll track this issue and fixed it once the related one is done.

Hi @alexandrospanagiotidis, we have already solve this issue by adopting azure-identity, could you please take a try again.

Dear @andxu,
Thank you for your reply!
As a side-note: we moved away from deploying via azure-maven-plugin (in favor of az-cli) as we needed to set a higher timeout for deployments to Azure China, which was not possible with the plugin at that time.
Thank you for your efforts!
All the best
Alex


[Improvement] Add kubernetes Integration test CI

Code of Conduct

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

Search before asking

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

What would you like to be improved?
We already have k8s e2e tests. But we should  add the e2e tests to our CI pipeline.
https://github.com/apache/incubator-uniffle/blob/master/deploy/kubernetes/integration-test/e2e/start-e2e.sh
How should we improve?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

/assign

@jerqi do we want to fail the pipeline if the e2e tests fail ?

Yes. we want.


python support for openvdb

I want to add the python wrappers for openvdb. I changed the corresponding variable in portfile.cmake from off to on
-DOPENVDB_BUILD_PYTHON_MODULE=ON

and added the necessary dependencies to the control file. Here's the added section in CONTROL
Feature: python
Description: Python wrapper for OpenVDB
Build-Depends: python3, boost-python

When I install openvdb with the command
 vcpkg install openvdb[python]:x64-windows

I get the following error message:
    -- Found the following Boost libraries:
    --   iostreams
    --   system
    --   thread
    --   date_time
    --   regex
    --   chrono
    --   atomic
    -- Found Blosc: C:/work/openvdb/vcpkg/vcpkg/installed/x64-windows
    -- Found TBB: C:/work/openvdb/vcpkg/vcpkg/installed/x64-windows
    -- Found ZLIB: C:\work\openvdb\vcpkg\vcpkg\installed\x64-windows\lib\zlib.lib (found version "1.2.11")
    -- Found ILMBase: C:/work/openvdb/vcpkg/vcpkg/installed/x64-windows
    -- Found OpenEXR: C:/work/openvdb/vcpkg/vcpkg/installed/x64-windows
    -- Found OpenEXR v2.2 at C:/work/openvdb/vcpkg/vcpkg/installed/x64-windows
    CMake Warning (dev) at openvdb/CMakeLists.txt:129 (LINK_DIRECTORIES):
      This command specifies the relative path

        SYSTEM

      as a link directory.

      Policy CMP0015 is not set: link_directories() treats paths relative to the
      source dir.  Run "cmake --help-policy CMP0015" for policy details.  Use the
      cmake_policy command to set the policy and suppress this warning.
    This warning is for project developers.  Use -Wno-dev to suppress it.

    -- Found PythonInterp: C:/Python/Python36/python.exe (found version "3.6.6")
    -- Found PythonLibs: optimized;C:/work/vcpkg/installed/x64-windows/lib/python36.lib;debug;C:/work/vcpkg/installed/x64-windows/debug/lib/python36_d.lib (found version "3.6.4")
    CMake Error at C:/Program Files/CMake/share/cmake-3.12/Modules/FindBoost.cmake:2048 (message):
      Unable to find the requested Boost libraries.

      Boost version: 1.68.0

      Boost include path:
      C:/work/openvdb/vcpkg/vcpkg/installed/x64-windows/include

      Could not find the following Boost libraries:

              boost_python

      No Boost libraries were found.  You may need to set BOOST_LIBRARYDIR to the
      directory containing Boost libraries or BOOST_ROOT to the location of
      Boost.

However, boost-python was correctly built and installed, along with the other boost dependencies. What do I need to change such that boost-python is recognized the same way as e.g boost-iostreams?

This issue is too old to be fixed for the versions reported.


ci: chang e2e test running rate

run at 2:34 am and 5:34 am at UTC+8
use github filter instead of market filter

Description
Modules

[ ] UI
[ ] Controller
[ ] Agent
[ ] Client
[ ] Python-SDK
[x] Others

Checklist

[ ] run code format and lint check
[ ] add unit test
[ ] add necessary doc

Does getsentry/paths-filter@v2 have some problems ?  @anda-ren

Use another explicated title?


how to get Location Permission Always in this Package?

I want to know how to get the permission of gps location .always in Android 10 or above (Api 29 orabove)?
I am using this package to get current location for periodic of time, no matter what the screen on or off.So, this package working just fine in Android 9 or bellow, but this doesn't working in android 10 or above.To achieve it, I need to get .always permission.
I just want to know how you guys achieve this in your example app?
I edited androidManifest.xml as your example, but nothing Happend!

All I was getting this  "kind" of popup for permission, but I want this as your example -

my scenario in Manifes -
    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
I also tried this permissions too -
<uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
<uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
<uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
<uses-permission android:name="android.permission.ACCESS_BACKGROUND_LOCATION"/>

All I need this -

But I am getting this  -

my Project Scenario -
class MyHomePage extends StatefulWidget {
  MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  List _loc = [];
  bool _isSwitchon = false;
  int _sec = 0;
  String _permissonStatus = "";

  Future _incrementCounter() async {
    Timer.periodic(Duration(seconds: 1), (timer) async {
      setState(() {
        _sec++;
      });
      if (_isSwitchon == false) {
        timer.cancel();
        setState(() {});
      } else if (_isSwitchon == true) {
        Position currentPosition = await Geolocator.getCurrentPosition();
        setState(() {
          _loc.add(currentPosition.toString());
        });
      }
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text("Time Elapsed"),
            Text(_sec.toString() + "  sec"),
            Text(
              'current Location',
            ),
            Container(
              height: 300,
              child: ListView.builder(
                itemCount: _loc.length,
                itemBuilder: (BuildContext context, int i) {
                  return Text(
                    _loc[i],
                  );
                },
              ),
            ),
            Row(
              children: [
                Expanded(
                  child: Container(),
                ),
                Column(
                  children: [
                    FloatingActionButton(
                      onPressed: () {
                        setState(() {
                          _isSwitchon = true;
                        });
                        _incrementCounter();
                      },
                      child: Icon(Icons.play_arrow),
                    ),
                    Text("Start")
                  ],
                ),
                SizedBox(
                  width: 50,
                ),
                Column(
                  children: [
                    FloatingActionButton(
                      onPressed: () {
                        setState(() {
                          _isSwitchon = false;
                        });
                      },
                      child: Icon(Icons.cancel),
                    ),
                    Text("Stop")
                  ],
                ),
                Expanded(
                  child: Container(),
                )
              ],
            ),
            Text(_permissonStatus),
            Row(
              mainAxisAlignment: MainAxisAlignment.spaceBetween,
              children: [
                Column(
                  children: [
                    FloatingActionButton(
                      onPressed: () async {
                        await Geolocator.checkPermission().then((value) =>
                            setState(
                                () => _permissonStatus = value.toString()));
                      },
                      child: Icon(Icons
                          .signal_wifi_statusbar_connected_no_internet_4_sharp),
                    ),
                    Text("status")
                  ],
                ),
                Column(
                  children: [
                    FloatingActionButton(
                      onPressed: () {
                        setState(() {
                          _loc = [];
                        });
                      },
                      tooltip: 'Increment',
                      child: Icon(Icons.add),
                    ),
                    Text("clear Location")
                  ],
                ),
                Column(
                  children: [
                    FloatingActionButton(
                      onPressed: () async {
                        await Geolocator.requestPermission();
                      },
                      tooltip: 'Increment',
                      child: Icon(Icons.gps_fixed),
                    ),
                    Text("Permission Seeking")
                  ],
                ),
              ],
            )
          ],
        ),
      ),
    );
  }
}

Thanks is Advance!

Hi, did you manage to find a solution? I have the same problem...

I created a blanc Flutter application and tried to reproduce your issue. I added the following permissions to the AndroidManifest.xml:

I created a blanc Flutter application and tried to reproduce your issue. I added the following permissions to the AndroidManifest.xml:

Hi, did you manage to find a solution? I have the same problem...

Not yet!

I created a blanc Flutter application and tried to reproduce your issue. I added the following permissions to the AndroidManifest.xml:
    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
    <uses-permission android:name="android.permission.ACCESS_BACKGROUND_LOCATION" />

Also I've set the targetSdkVersion to 30 in my build.gradle like below:
     minSdkVersion 16
     targetSdkVersion 30

Then I made a button to request background permission in my application:
import 'package:flutter/material.dart';
import 'package:geolocator/geolocator.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Center(
        child: TextButton(child: Text('Press me to request background permission'), onPressed: () async {
          await Geolocator.requestPermission();
        },)
      )
    );
  }
}

It's totally working fine for me. Keep in mind that in Android 10 and above u need to request background permission two times.
On the first try, you should give the Allow while using the app permission. And the second time you will be forwarded to the location settings screen with the Allow all the time option available.


Update flask_mail.py

Fix exception: "ValueError: invalid literal for int() with base 10: 'True'".

Don't use a string to set a boolean value?  Now this will fail if you write "False", because non-empty strings are True.

ups. Sorry for the issue. It was an error in my app :(


Py file of Functions + Nb for rating a given tt

I also fixed the fact that I never saved the model. The Model is also diff now T_T


RP2350 default clock is still 125MHz

PIO max. clock (and SYSCLK) is still 125MHz (as for RP2040).
Please, change SYSCLK to 150MHz (as specified) for a RP2350 MicroPython FW.
minor issue: it can be overridden by explicit SYSCLK setting.

Huh? You have thoroughly lost me here!


enhancementmake error message more readable when bind logicalRepeat node

pick from master https://github.com/apache/doris/pull/26744
commit id: a78e0f8309afe911d684b072ee6fea6d4ad76c2c
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Update publications

Update the references to publications in README and/or documentation that use G-SMOTE.

PR #29 fixes this issue


June 25/19 - I think they changed their code again

All 4 scripts not working as of June 25/19. Please check.

I may be wrong, I think the files were just not there.


Proposed update for README intro

I think the README intro paragraph should be re-written https://github.com/typesafegithub/github-workflows-kt#readme
It starts off by talking about YAML, which I don’t think is a strong start. I’d expect it to start with a strong statement about what workflows-kt is and does before talking about what problems it solves. For example:

workflows-kt is a tool for creating GitHub Workflows in a typesafe script, helping you to build robust workflows for your GitHub projects without mistakes. You won’t go back to YAML!

(just an example to show my point, I think it should be expanded a little more)

It's a good idea to have a link back to the repo to help when the repo gets rehosted
The catchphrase "You won't get back to YAML!" is good, but a little awkward in English (but maybe that's the point, and it's hard to judge if it's intended to be a little off :)) - "You won't go back to YAML!" flows better.

@aSemy thanks! Please review: https://github.com/typesafegithub/github-workflows-kt/pull/859


German and Chinese localisations

I've been really impressed by the UI of this app. I'd now like to work on adding values for German and Chinese (my languages). I'll try to work on them in my spare time.
Regards,
Dongchen

Thank you, it would be really helpful.

Regarding the name of this app, it's not really easy to be uniformly applied across all languages. Perhaps the displayed name could be shortened to "QBScan" or similar for a mulilingual (untranslatable) name. Your thoughts?

The name of the app can be translated, no? For example, in English it is "QR & Barcode Scanner", in Russian it is "Сканер QR и штрихкодов". I can't come up with any untranslatable name right now. If you have any ideas, feel free to share.

I personally think "QBScan" is quite good. If you think that's not suitable, I'll just translate it literally.

Yes, I think literal translation will be fine for now.

Hello. If you are still interested in this issue, I created a project on Transifex. You can add the Chinese translation there or create a PR. Thanks again for your help.

Thanks, I propose to deal with the french translation. Love your app :+1:

Sorry for the delay. I plan to finish the Chinese ones this weekend.

Thanks, I propose to deal with the french translation. Love your app

When I try to join the team in transiflex, French isn't an available language :C
Could you add it ? Thanks !

Thanks, I propose to deal with the french translation. Love your app

When I try to join the team in transiflex, French isn't an available language :C
Could you add it ? Thanks !

That's weird. https://www.transifex.com/a-302/qr-barcode-scanner/language/fr_FR/

Thanks, I propose to deal with the french translation. Love your app

When I try to join the team in transiflex, French isn't an available language :C
Could you add it ? Thanks !

That's weird. https://www.transifex.com/a-302/qr-barcode-scanner/language/fr_FR/

Yes, really weird. I got a 404 error with your link. A sent a message to transiflex to inform them of this issue.

@dmitriy-ilchenko ,
I don't seem to have been approved on Transifex one week after sending a request. Could you please check?
Thanks.

Same for me, and I think there is a bug with French langage. Could you delete it (like this, I would be able to "ask a langage"). thanks

@yue-dongchen Sorry for the delay, I guess I approved you.
@tifriis208 I guess that the problem was that I didn't approved you to join the team. Sorry for the inconvenience, I am not very familiar with Transifex. Could you please try now?

@dmitriy-ilchenko yes it worked.  Scrat15 made an huge and fast work. I finished the french translation and corrected some mistakes. It's all goof for french langage :)

I had a look at the simplified Chinese (zh_CN) translations and they seem fine. I'll close this for now.


v1.5 backports 2019-06-21

#8328 -- fqdn: correctly populate Source IP and Port in notifyOnDNSMsg (@ianvernon)

Had to make minor fixups because the function was refactored out

#8350 -- CI: Report last seen error in CiliumPreFlightCheck (@raybejjani)
#8352 -- CI: Cleanup VMs and reclaim disk on job completion (@raybejjani)
#8361 -- pkg/metrics: re-register newStatusCollector function (@aanm)
#8366 -- docs: Clarify about legacy services enabled by default (@brb)

[ Round 2, added CI fixes]

#7911

This was partially backported before in f7a1bd149a40f237850c91baa9b6987de6c67061, so I went back and backported the other half.

#8148
#8341
#8355

Skipped:

Changes that require vendor updates

@aanm how do you update vendor dependencies during backport ? I get errors[1]
https://github.com/cilium/cilium/pull/8280 -- Add support for k8s 1.15 (@aanm)
https://github.com/cilium/cilium/pull/8351 -- Workaround for systemd 242 assigning MAC addr for virtual devices (@brb)

[1]
$ dep ensure
Solving failure: No versions of github.com/go-openapi/spec met constraints:
        v0.18.0: Could not introduce<EMAIL_ADDRESS>as it requires github.com/go-openapi/jsonpointer at revision v0.18.0, but that revision does not exist
        v0.19.2: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        v0.19.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        v0.18.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        v0.17.2: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        v0.17.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        v0.17.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        0.16.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        0.15.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        master: Could not introduce github.com/go-openapi/spec@master, as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        fix-definitions-omitempty: Could not introduce github.com/go-openapi/spec@fix-definitions-omitempty, as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        fixSpelling: Could not introduce github.com/go-openapi/spec@fixSpelling, as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        revert-79-fix-circular: Could not introduce github.com/go-openapi/spec@revert-79-fix-circular, as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.
        transitiveResolverLocalRefCheck: Could not introduce github.com/go-openapi/spec@transitiveResolverLocalRefCheck, as it is not allowed by constraint v0.18.0 from project github.com/cilium/cilium.

Once this PR is merged, you can update the PR labels via:
$ for pr in 7911 8148 8328 8341 8350 8352 8355 8361 8366; do contrib/backporting/set-labels.py $pr done 1.5; done

This change is

never-tell-me-the-odds

test-me-please

test-me-please

never-tell-me-the-odds

@joestringer I've pushed pr/v1.5-backport-2019-06-21-2 that contains PRs #8280 and #8351 backported.
I had to backported only the file Gopkg.toml for the kubernetes changes and then ran dep ensure -v

I've pushed pr/v1.5-backport-2019-06-21-2 that contains PRs #8280 and #8351 backported.

Are we OK to close this PR and open for pr/v1.5-backport-2019-06-21-2?

test-me-please

Superseded by #8381.

Reopening to get the CI run results.


feat: verify project exists before sending destroy task

This is a quick fix to the issue of cargo project rm always returning destroying. Draft while I look into improving cli feedback.

This seems good to me. Is there a test we can update for this too?

Probably! I'll do that now. I'm looking into updating the cli feedback/ws status stream, but we can merge this first anyway.


Cocoapods

Please, include a podspec file.
thanks, very simple and cute HUD.

I will do! Thank you! 😊

Many thanks, but I think it is also necessary to specify s.ios.deployment_target = '7.1', because on an iOS 8 project the pod update displays the message

Specs satisfying the AMTumblrHud dependency were found, but they required a higher minimum target deployment.

I just updated spec, seems works normal now

many thanks


Odd behavior for jax.tree_util.Partial and interactions with eqx.Module

Here's something pretty odd about jax.tree_util.Partial: it mimics the equality behavior of functools.partial, which returns that two partial calls on the same function on the same arguments are not equal, unless they are the same object.
This is a problem, because equinox wraps its method calls to return a jax.tree_util.Partial, giving you the following very unexpected behavior:
import equinox as eqx

class M(eqx.Module):
  def f(self, x):
    return x

m = M()
M.f == M.f # True
m.f == m.f # False

The same instance method isn't the same!  This is because they come from separate calls to the wrapper that turns the
class's method into jax.tree_util.Partial.
One possible fix is to switch the wrapper to use equinox's own Partial, which behaves sensibly with equality.  Except, you can't use functools.wraps on an equinox Partial because it's a frozen dataclass and wraps tries to change attributes on the method.  The only simple fix is to drop the wraps call altogether but that seems like a bad choice too.  Maybe the Partial could be unfrozen while it is wrapped?
Probably the better solution is for jax to change how __eq__ works on jax.tree_util.Partial, because it leads to more unusual behavior.  The choice for equality may be a fine decision for functools.partial, but jax.tree_util.Partial is PyTree compatible and so can be flattened.  I would expect that if two flattened PyTrees are equal that the unflattened PyTrees would be equal.  That doesn't hold for jax.tree_util.Partial, which flattens to the function and its arguments, which are equal for two equivalent jax.tree_util.Partial objects.
Equinox's Partial doesn't have any of these problems.  See below, that only jax.tree_util.Partial is inconsistent between equality of its flattened and unflattened representations.
import functools as ft
import jax.tree_util as jtu
import equinox as eqx

def f(x):
  return x

print('functools: ',
      ft.partial(f, 3) == ft.partial(f, 3),
      jtu.tree_flatten(ft.partial(f, 3)) == jtu.tree_flatten(ft.partial(f, 3)))
print('jax.tree_utils: ',
      jtu.Partial(f, 3) == jtu.Partial(f, 3),
      jtu.tree_flatten(jtu.Partial(f, 3)) == jtu.tree_flatten(jtu.Partial(f, 3)))
print('equinox: ',
      eqx.Partial(f, 3) == eqx.Partial(f, 3),
      jtu.tree_flatten(eqx.Partial(f, 3)) == jtu.tree_flatten(eqx.Partial(f, 3)))

gives the output:
functools:  False False
jax.tree_utils:  False True
equinox:  True True

Thanks for the report! This is quite the edge-case. Indeed, it looks like jax.tree_util.Partial has some odd behaviour on this front.
I've just written #485, which should fix this.

Thanks!

Closing as fixed in #485. This will be included the next release (v0.11.0) of Equinox.


Rename enums *ToLocation to *Target.

Fixes #739

Thanks Jon and Mike! All fixed

The failure seems to be an issue with mono?
"Got a SIGABRT while executing native code. This usually indicates
a fatal error in the mono runtime or one of the native libraries
used by your application."
I ran this again on a ubuntu machine and didn't get any failures.  I'm going to merge and I will revert if this causes issues


no Docker image for RTX 30 series GPU  and CUDA >=11.1

I don't know if I am doing something obviously wrong but I have had massive struggles with this repository! Basically due to my GPU being a 3080, the common way of using cuda 10.0 or 10.1 or 10.2 does not work for me. Since my graphics card only supports CUDA>=11.1  When directly installing OpenPCDet the recommended way, I always run into some for me unsolvable undefined symbols errors (#755) The provided Docker image also does not work for me due to it being incompatible with the compute capabilities 8.6! I would really appreciate if somebody with an RTX30 series graphics card could describe either the used python/pytroch/gcc/g++/nvcc/cuda/spconv version or maybe share a docker file!
Help is appreciated!

I guess you can try to simply modify some lines in the docker file such as https://github.com/open-mmlab/OpenPCDet/blob/830fba91605f236bc2183f83ae1e986df35200d7/docker/Dockerfile#L1
and https://github.com/open-mmlab/OpenPCDet/blob/6a1495ca610186d3b9d5142a4a6506b8dc78bce1/docker/Dockerfile#L55 to build on your needed cuda version.

Thank you very much for your quick reply!!!
I did the suggested changes and replaced the lines you suggest as well as the PyTorch line with:
FROM nvidia/cuda:11.3.0-cudnn8-devel-ubuntu18.04

RUN pip install --user torch==1.6 torchvision==0.7.0 -f https://download.pytorch.org/whl/cu113/torch_stable.html

RUN pip install spconv-cu113

How ever I still encounter an error this time it explicitly says RTX3080 not supported for my pytorch version:
building 'pcdet.ops.iou3d_nms.iou3d_nms_cuda' extension
/root/.local/lib/python3.7/site-packages/torch/cuda/__init__.py:125: UserWarning:
NVIDIA GeForce RTX 3080 Laptop GPU with CUDA capability sm_86 is not compatible with the current PyTorch installation.
The current PyTorch install supports CUDA capabilities sm_37 sm_50 sm_60 sm_70 sm_75.
If you want to use the NVIDIA GeForce RTX 3080 Laptop GPU GPU with PyTorch, please check the instructions at https://pytorch.org/get-started/locally/

  warnings.warn(incompatible_device_warn.format(device_name, capability, " ".join(arch_list), device_name))
Traceback (most recent call last):
  File "setup.py", line 116, in <module>
    'src/sampling_gpu.cu',
  File "/usr/local/lib/python3.7/dist-packages/setuptools/__init__.py", line 155, in setup
    return distutils.core.setup(**attrs)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/core.py", line 148, in setup
    return run_commands(dist)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/core.py", line 163, in run_commands
    dist.run_commands()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 967, in run_commands
    self.run_command(cmd)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 986, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/command/develop.py", line 34, in run
    self.install_for_development()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/command/develop.py", line 114, in install_for_development
    self.run_command('build_ext')
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 986, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/command/build_ext.py", line 79, in run
    _build_ext.run(self)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 339, in run
    self.build_extensions()
  File "/root/.local/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 649, in build_extensions
    build_ext.build_extensions(self)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 448, in build_extensions
    self._build_extensions_serial()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 473, in _build_extensions_serial
    self.build_extension(ext)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/command/build_ext.py", line 202, in build_extension
    _build_ext.build_extension(self, ext)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 534, in build_extension
    depends=ext.depends)
  File "/root/.local/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 464, in unix_wrap_ninja_compile
    cuda_post_cflags = unix_cuda_flags(cuda_post_cflags)
  File "/root/.local/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 372, in unix_cuda_flags
    cflags + _get_cuda_arch_flags(cflags))
  File "/root/.local/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 1414, in _get_cuda_arch_flags
    raise ValueError("Unknown CUDA arch ({}) or GPU not supported".format(arch))

Solved! Thank you very much @jihanyang for your help!
I am new to docker and understanding it took me quite a bit, so in case it helps someone, here is exactly what I did:
I had to change the pytorch line again to :
RUN pip3 install torch==1.10.2+cu113 torchvision==0.11.3+cu113 torchaudio==0.10.2+cu113 -f https://download.pytorch.org/whl/cu113/torch_stable.html

because I encountered some version conflicts with numpy after adding opencv and open3d to my docker image I also added the following line:
RUN pip3 install numpy==1.20 #to solve numpy dependency issues with opencv!

So here is my working docker file:
FROM nvidia/cuda:11.3.0-cudnn8-devel-ubuntu18.04

RUN echo 'debconf debconf/frontend select Noninteractive' | debconf-set-selections

# Install basics
RUN apt-get update -y \
    && apt-get install build-essential \
    && apt-get install -y apt-utils git curl ca-certificates bzip2 tree htop wget \
    && apt-get install -y libglib2.0-0 libsm6 libxext6 libxrender-dev bmon iotop g++ python3.7 python3.7-dev python3.7-distutils

# Install cmake v3.13.2
RUN apt-get purge -y cmake && \
    mkdir /root/temp && \
    cd /root/temp && \
    wget https://github.com/Kitware/CMake/releases/download/v3.13.2/cmake-3.13.2.tar.gz && \
    tar -xzvf cmake-3.13.2.tar.gz && \
    cd cmake-3.13.2 && \
    bash ./bootstrap && \
    make && \
    make install && \
    cmake --version && \
    rm -rf /root/temp

# Install python
RUN ln -sv /usr/bin/python3.7 /usr/bin/python
RUN wget https://bootstrap.pypa.io/get-pip.py && \
	python get-pip.py && \
	rm get-pip.py

# Install python packages
RUN PIP_INSTALL="python -m pip --no-cache-dir install" && \
    $PIP_INSTALL numpy==1.19.3 llvmlite numba

# Install torch and torchvision
# See https://pytorch.org/ for other options if you use a different version of CUDA
RUN pip3 install torch==1.10.2+cu113 torchvision==0.11.3+cu113 torchaudio==0.10.2+cu113 -f https://download.pytorch.org/whl/cu113/torch_stable.html

RUN pip3 install numpy==1.20 #to solve numpy dependency issues with opencv!

# Install python packages
RUN PIP_INSTALL="python -m pip --no-cache-dir install" && \
    $PIP_INSTALL tensorboardX easydict pyyaml scikit-image tqdm SharedArray six

WORKDIR /root

# Install Boost geometry
RUN wget https://jaist.dl.sourceforge.net/project/boost/boost/1.68.0/boost_1_68_0.tar.gz && \
    tar xzvf boost_1_68_0.tar.gz && \
    cp -r ./boost_1_68_0/boost /usr/include && \
    rm -rf ./boost_1_68_0 && \
    rm -rf ./boost_1_68_0.tar.gz

# A weired problem that hasn't been solved yet
RUN pip uninstall -y SharedArray && \
    pip install SharedArray

RUN pip install spconv-cu113

And if you want to run the demo add this to the end of your Dockerfile:

#just for running the Demo(without visualization) we need to add:
RUN apt-get update && apt-get install -y python3-opencv
RUN pip install opencv-python
RUN pip install open3d

I replaced the original docker file with my file and ran:
docker build ./ -t openpcdet-docker

Then I ran the Docker image and mounted a volume to it, which I opened on my host computer and put the OpenPCDet git folder in it (located in /var/lib/docker/volumes/data/_data):
docker run  -i -t -v data:/root/data --gpus all openpcdet-docker

Then I built OpenPCDet inside the docker image and inside the mounted volume, so it stays built!
I now have to export the build location to PYTHONPATH every time I run the docker image to use it:
export PYTHONPATH=$HOME/data/OpenPCDet/:$PYTHONPATH

Then I made sure that the data folder structure was correct and that it had all the reqired data in it and that the model was in the right place and then I could run the Demo successfully but unfortunately I don't know if docker supports a visualization?
python demo.py --cfg_file cfgs/kitti_models/pv_rcnn.yaml     --ckpt pv_rcnn_8369.pth     --data_path ~/data/OpenPCDet/data

@makeradventures Great!

Thanks for sharing a detailed explanation.
I faced issue with pcdet while running demo.py code. Installation pcdet was required. Used below command in OpenPCDet root folder to install pcdet
python setup.py develop
Coming to my question.
How to run docker without interactive shell?  I don't need visualization so currently using the docker file you shared earlier.
I have added below code at the end of without visualization Dockerfile to install pcdet
`
#to permanently export the location of OpenPCdet (installed in a persistent volume) to PYTHONPATH
ENV PYTHONPATH="/root/data/OpenPCDet/:$PYTHONPATH"
copy the contents from local to docker
WORKDIR /
ADD . /app
To install pcdet
WORKDIR /app
RUN python setup.py develop
To execute demo code
WORKDIR /app/tools
CMD ["python", "demo_analysis_waymo.py", "--cfg_file","cfgs/waymo_models/pv_rcnn.yaml", "--ckpt","../H_pvrcnn_checkpoint_epoch_30.pth", "--data_path", "../DataAnalysis"]
However its failing at RUN python setup.py develop`
(openpc) vikasnr@lambda-dual:~/calibration/code/new/OpenPCDet$ docker build ./ -t openpcdet-docker Sending build context to Docker daemon  425.1MB Step 1/24 : FROM nvidia/cuda:11.3.0-cudnn8-devel-ubuntu18.04  ---> 0aa6703ebcf1 Step 2/24 : RUN echo 'debconf debconf/frontend select Noninteractive' | debconf-set-selections  ---> Using cache  ---> 6425b0112b31 Step 3/24 : RUN apt-get update -y     && apt-get install build-essential     && apt-get install -y apt-utils git curl ca-certificates bzip2 tree htop wget     && apt-get install -y libglib2.0-0 libsm6 libxext6 libxrender-dev bmon iotop g++ python3.7 python3.7-dev python3.7-distutils  ---> Using cache  ---> 54b58526c4d2 Step 4/24 : RUN apt-get purge -y cmake &&     mkdir /root/temp &&     cd /root/temp &&     wget https://github.com/Kitware/CMake/releases/download/v3.13.2/cmake-3.13.2.tar.gz &&     tar -xzvf cmake-3.13.2.tar.gz &&     cd cmake-3.13.2 &&     bash ./bootstrap &&     make &&     make install &&     cmake --version &&     rm -rf /root/temp  ---> Using cache  ---> 0f6a08f59f74 Step 5/24 : RUN ln -sv /usr/bin/python3.7 /usr/bin/python  ---> Using cache  ---> 919132b1ca87 Step 6/24 : RUN wget https://bootstrap.pypa.io/get-pip.py &&    python get-pip.py &&    rm get-pip.py  ---> Using cache  ---> 7981e8faed37 Step 7/24 : RUN PIP_INSTALL="python -m pip --no-cache-dir install" &&     $PIP_INSTALL numpy==1.19.3 llvmlite numba  ---> Using cache  ---> 6a465c7e3c92 Step 8/24 : RUN pip3 install torch==1.10.2+cu113 torchvision==0.11.3+cu113 torchaudio==0.10.2+cu113 -f https://download.pytorch.org/whl/cu113/torch_stable.html  ---> Using cache  ---> a73a38bf24f1 Step 9/24 : RUN pip3 install numpy==1.20 #to solve numpy dependency issues with opencv!  ---> Using cache  ---> 25da8f105c3d Step 10/24 : RUN PIP_INSTALL="python -m pip --no-cache-dir install" &&     $PIP_INSTALL tensorboardX easydict pyyaml scikit-image tqdm SharedArray six  ---> Using cache  ---> ce96f9b67669 Step 11/24 : WORKDIR /root  ---> Using cache  ---> 316667c251f3 Step 12/24 : RUN wget https://jaist.dl.sourceforge.net/project/boost/boost/1.68.0/boost_1_68_0.tar.gz &&     tar xzvf boost_1_68_0.tar.gz &&     cp -r ./boost_1_68_0/boost /usr/include &&     rm -rf ./boost_1_68_0 &&     rm -rf ./boost_1_68_0.tar.gz  ---> Using cache  ---> 37fdaa7caed9 Step 13/24 : RUN pip uninstall -y SharedArray &&     pip install SharedArray  ---> Using cache  ---> a67c529f257e Step 14/24 : RUN pip install spconv-cu113  ---> Using cache  ---> 31a207876a0b Step 15/24 : RUN apt-get update && apt-get install -y python3-opencv  ---> Using cache  ---> 6dcb2b31a3e1 Step 16/24 : RUN pip install opencv-python  ---> Using cache  ---> cf021c126b6e Step 17/24 : RUN pip install open3d  ---> Using cache  ---> 58ca37bcca0b Step 18/24 : ENV PYTHONPATH="/root/data/OpenPCDet/:$PYTHONPATH"  ---> Using cache  ---> 8475849fc173 Step 19/24 : WORKDIR /  ---> Running in 1ffc86cc2ea1 Removing intermediate container 1ffc86cc2ea1  ---> b343d72c3944 Step 20/24 : ADD . /app  ---> d01242732076 Step 21/24 : WORKDIR /app  ---> Running in 37ed99ba7a82 Removing intermediate container 37ed99ba7a82  ---> 4da706a8965e Step 22/24 : RUN python setup.py develop  ---> Running in 7e188ea1c734 No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda' running develop running egg_info writing pcdet.egg-info/PKG-INFO writing dependency_links to pcdet.egg-info/dependency_links.txt writing requirements to pcdet.egg-info/requires.txt writing top-level names to pcdet.egg-info/top_level.txt reading manifest file 'pcdet.egg-info/SOURCES.txt' adding license file 'LICENSE' writing manifest file 'pcdet.egg-info/SOURCES.txt' running build_ext building 'pcdet.ops.iou3d_nms.iou3d_nms_cuda' extension creating /app/build/temp.linux-x86_64-cpython-37 creating /app/build/temp.linux-x86_64-cpython-37/pcdet creating /app/build/temp.linux-x86_64-cpython-37/pcdet/ops creating /app/build/temp.linux-x86_64-cpython-37/pcdet/ops/iou3d_nms creating /app/build/temp.linux-x86_64-cpython-37/pcdet/ops/iou3d_nms/src /usr/local/lib/python3.7/dist-packages/setuptools/command/easy_install.py:147: EasyInstallDeprecationWarning: easy_install command is deprecated. Use build and pip and other standards-based tools.   EasyInstallDeprecationWarning, /usr/local/lib/python3.7/dist-packages/setuptools/command/install.py:37: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.   setuptools.SetuptoolsDeprecationWarning, Traceback (most recent call last):   File "setup.py", line 116, in <module>     'src/sampling_gpu.cu',   File "/usr/local/lib/python3.7/dist-packages/setuptools/__init__.py", line 87, in setup     return distutils.core.setup(**attrs)   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/core.py", line 185, in setup     return run_commands(dist)   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/core.py", line 201, in run_commands     dist.run_commands()   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 969, in run_commands     self.run_command(cmd)   File "/usr/local/lib/python3.7/dist-packages/setuptools/dist.py", line 1208, in run_command     super().run_command(command)   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 988, in run_command     cmd_obj.run()   File "/usr/local/lib/python3.7/dist-packages/setuptools/command/develop.py", line 34, in run     self.install_for_development()   File "/usr/local/lib/python3.7/dist-packages/setuptools/command/develop.py", line 114, in install_for_development     self.run_command('build_ext')   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/cmd.py", line 318, in run_command     self.distribution.run_command(command)   File "/usr/local/lib/python3.7/dist-packages/setuptools/dist.py", line 1208, in run_command     super().run_command(command)   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 988, in run_command     cmd_obj.run()   File "/usr/local/lib/python3.7/dist-packages/setuptools/command/build_ext.py", line 84, in run     _build_ext.run(self)   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 346, in run     self.build_extensions()   File "/usr/local/lib/python3.7/dist-packages/torch/utils/cpp_extension.py", line 735, in build_extensions     build_ext.build_extensions(self)   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 468, in build_extensions     self._build_extensions_serial()   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 494, in _build_extensions_serial     self.build_extension(ext)   File "/usr/local/lib/python3.7/dist-packages/setuptools/command/build_ext.py", line 246, in build_extension     _build_ext.build_extension(self, ext)   File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 556, in build_extension     depends=ext.depends,   File "/usr/local/lib/python3.7/dist-packages/torch/utils/cpp_extension.py", line 551, in unix_wrap_ninja_compile     cuda_post_cflags = unix_cuda_flags(cuda_post_cflags)   File "/usr/local/lib/python3.7/dist-packages/torch/utils/cpp_extension.py", line 450, in unix_cuda_flags     cflags + _get_cuda_arch_flags(cflags))   File "/usr/local/lib/python3.7/dist-packages/torch/utils/cpp_extension.py", line 1606, in _get_cuda_arch_flags     arch_list[-1] += '+PTX' IndexError: list index out of range

Thanks for sharing a detailed explanation.
I faced issue with pcdet while running demo.py code. Installation pcdet was required. Used below command in OpenPCDet root folder to install pcdet
python setup.py develop
Coming to my question.
How to run docker without interactive shell?  I don't need visualization so currently using the docker file you shared earlier.
I have added below code at the end of without visualization Dockerfile to install pcdet
......
 # just for running the Demo we need to add:
RUN apt-get update && apt-get install -y python3-opencv
RUN pip install opencv-python
RUN pip install open3d

# To permanently export the location of OpenPCdet (installed in a persistent volume) to PYTHONPATH
ENV PYTHONPATH="/root/data/OpenPCDet/:$PYTHONPATH"

 # copy OpenPCDet code from local to docker
WORKDIR /
ADD . /app

# To install pcdet
WORKDIR /app
RUN python setup.py develop

# To execute demo code
WORKDIR /app/tools
CMD ["python", "demo_analysis_waymo.py", "--cfg_file","cfgs/waymo_models/pv_rcnn.yaml", "--ckpt","../H_pvrcnn_checkpoint_epoch_30.pth", "--data_path", "../DataAnalysis"]`

However its failing at  RUN python setup.py develop
(openpc) vikasnr@lambda-dual:~/calibration/code/new/OpenPCDet$ docker build ./ -t openpcdet-docker
Sending build context to Docker daemon  425.1MB
Step 1/24 : FROM nvidia/cuda:11.3.0-cudnn8-devel-ubuntu18.04
 ---> 0aa6703ebcf1
Step 2/24 : RUN echo 'debconf debconf/frontend select Noninteractive' | debconf-set-selections
 ---> Using cache
 ---> 6425b0112b31
Step 3/24 : RUN apt-get update -y     && apt-get install build-essential     && apt-get install -y apt-utils git curl ca-certificates bzip2 tree htop wget     && apt-get install -y libglib2.0-0 libsm6 libxext6 libxrender-dev bmon iotop g++ python3.7 python3.7-dev python3.7-distutils
 ---> Using cache
 ---> 54b58526c4d2
Step 4/24 : RUN apt-get purge -y cmake &&     mkdir /root/temp &&     cd /root/temp &&     wget https://github.com/Kitware/CMake/releases/download/v3.13.2/cmake-3.13.2.tar.gz &&     tar -xzvf cmake-3.13.2.tar.gz &&     cd cmake-3.13.2 &&     bash ./bootstrap &&     make &&     make install &&     cmake --version &&     rm -rf /root/temp
 ---> Using cache
 ---> 0f6a08f59f74
Step 5/24 : RUN ln -sv /usr/bin/python3.7 /usr/bin/python
 ---> Using cache
 ---> 919132b1ca87
Step 6/24 : RUN wget https://bootstrap.pypa.io/get-pip.py &&    python get-pip.py &&    rm get-pip.py
 ---> Using cache
 ---> 7981e8faed37
Step 7/24 : RUN PIP_INSTALL="python -m pip --no-cache-dir install" &&     $PIP_INSTALL numpy==1.19.3 llvmlite numba
 ---> Using cache
 ---> 6a465c7e3c92
Step 8/24 : RUN pip3 install torch==1.10.2+cu113 torchvision==0.11.3+cu113 torchaudio==0.10.2+cu113 -f https://download.pytorch.org/whl/cu113/torch_stable.html
 ---> Using cache
 ---> a73a38bf24f1
Step 9/24 : RUN pip3 install numpy==1.20 #to solve numpy dependency issues with opencv!
 ---> Using cache
 ---> 25da8f105c3d
Step 10/24 : RUN PIP_INSTALL="python -m pip --no-cache-dir install" &&     $PIP_INSTALL tensorboardX easydict pyyaml scikit-image tqdm SharedArray six
 ---> Using cache
 ---> ce96f9b67669
Step 11/24 : WORKDIR /root
 ---> Using cache
 ---> 316667c251f3
Step 12/24 : RUN wget https://jaist.dl.sourceforge.net/project/boost/boost/1.68.0/boost_1_68_0.tar.gz &&     tar xzvf boost_1_68_0.tar.gz &&     cp -r ./boost_1_68_0/boost /usr/include &&     rm -rf ./boost_1_68_0 &&     rm -rf ./boost_1_68_0.tar.gz
 ---> Using cache
 ---> 37fdaa7caed9
Step 13/24 : RUN pip uninstall -y SharedArray &&     pip install SharedArray
 ---> Using cache
 ---> a67c529f257e
Step 14/24 : RUN pip install spconv-cu113
 ---> Using cache
 ---> 31a207876a0b
Step 15/24 : RUN apt-get update && apt-get install -y python3-opencv
 ---> Using cache
 ---> 6dcb2b31a3e1
Step 16/24 : RUN pip install opencv-python
 ---> Using cache
 ---> cf021c126b6e
Step 17/24 : RUN pip install open3d
 ---> Using cache
 ---> 58ca37bcca0b
Step 18/24 : ENV PYTHONPATH="/root/data/OpenPCDet/:$PYTHONPATH"
 ---> Using cache
 ---> 8475849fc173
Step 19/24 : WORKDIR /
 ---> Running in 1ffc86cc2ea1
Removing intermediate container 1ffc86cc2ea1
 ---> b343d72c3944
Step 20/24 : ADD . /app
 ---> d01242732076
Step 21/24 : WORKDIR /app
 ---> Running in 37ed99ba7a82
Removing intermediate container 37ed99ba7a82
 ---> 4da706a8965e
Step 22/24 : RUN python setup.py develop
 ---> Running in 7e188ea1c734
No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda'
running develop
running egg_info
writing pcdet.egg-info/PKG-INFO
writing dependency_links to pcdet.egg-info/dependency_links.txt
writing requirements to pcdet.egg-info/requires.txt
writing top-level names to pcdet.egg-info/top_level.txt
reading manifest file 'pcdet.egg-info/SOURCES.txt'
adding license file 'LICENSE'
writing manifest file 'pcdet.egg-info/SOURCES.txt'
running build_ext
building 'pcdet.ops.iou3d_nms.iou3d_nms_cuda' extension
creating /app/build/temp.linux-x86_64-cpython-37
creating /app/build/temp.linux-x86_64-cpython-37/pcdet
creating /app/build/temp.linux-x86_64-cpython-37/pcdet/ops
creating /app/build/temp.linux-x86_64-cpython-37/pcdet/ops/iou3d_nms
creating /app/build/temp.linux-x86_64-cpython-37/pcdet/ops/iou3d_nms/src
/usr/local/lib/python3.7/dist-packages/setuptools/command/easy_install.py:147: EasyInstallDeprecationWarning: easy_install command is deprecated. Use build and pip and other standards-based tools.
  EasyInstallDeprecationWarning,
/usr/local/lib/python3.7/dist-packages/setuptools/command/install.py:37: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
  setuptools.SetuptoolsDeprecationWarning,
Traceback (most recent call last):
  File "setup.py", line 116, in <module>
    'src/sampling_gpu.cu',
  File "/usr/local/lib/python3.7/dist-packages/setuptools/__init__.py", line 87, in setup
    return distutils.core.setup(**attrs)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/core.py", line 185, in setup
    return run_commands(dist)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/core.py", line 201, in run_commands
    dist.run_commands()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 969, in run_commands
    self.run_command(cmd)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/dist.py", line 1208, in run_command
    super().run_command(command)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 988, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/command/develop.py", line 34, in run
    self.install_for_development()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/command/develop.py", line 114, in install_for_development
    self.run_command('build_ext')
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/cmd.py", line 318, in run_command
    self.distribution.run_command(command)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/dist.py", line 1208, in run_command
    super().run_command(command)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/dist.py", line 988, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/command/build_ext.py", line 84, in run
    _build_ext.run(self)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 346, in run
    self.build_extensions()
  File "/usr/local/lib/python3.7/dist-packages/torch/utils/cpp_extension.py", line 735, in build_extensions
    build_ext.build_extensions(self)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 468, in build_extensions
    self._build_extensions_serial()
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 494, in _build_extensions_serial
    self.build_extension(ext)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/command/build_ext.py", line 246, in build_extension
    _build_ext.build_extension(self, ext)
  File "/usr/local/lib/python3.7/dist-packages/setuptools/_distutils/command/build_ext.py", line 556, in build_extension
    depends=ext.depends,
  File "/usr/local/lib/python3.7/dist-packages/torch/utils/cpp_extension.py", line 551, in unix_wrap_ninja_compile
    cuda_post_cflags = unix_cuda_flags(cuda_post_cflags)
  File "/usr/local/lib/python3.7/dist-packages/torch/utils/cpp_extension.py", line 450, in unix_cuda_flags
    cflags + _get_cuda_arch_flags(cflags))
  File "/usr/local/lib/python3.7/dist-packages/torch/utils/cpp_extension.py", line 1606, in _get_cuda_arch_flags
    arch_list[-1] += '+PTX'
IndexError: list index out of range

Note : I'm running this on a GPU machine.


DuckDB crashes when doing query on schema named "main" in Postgres.

What happens?
When using DuckDB v1.1.3 with the Postgres extension (https://duckdb.org/docs/extensions/postgres.html), DuckDB crashes when querying a schema named "main."
I can successfully attach several databases and use the SHOW ALL TABLES; command to view all tables, including those under the schema named "main." However, attempting to execute a query such as SELECT * FROM alias.main.streets; causes DuckDB to crash.
Seems like a schema name conflict?
To Reproduce
INSTALL postgres;
LOAD postgres;

ATTACH 'dbname=db_info user=beaver password=123 host=beavertown.postgresql-dev.com' as postgres_db_info (TYPE POSTGRES);
ATTACH 'dbname=db_main user=beaver password=123 host=beavertown.postgresql-dev.com' as postgres_db_main (TYPE POSTGRES);

SHOW ALL TABLES;
-- this shows all schemas and tables for both databases.

SELECT * FROM postgres_db_info.info.test;
-- this query works

SELECT * FROM postgres_db_main.main.test;
--duckdb crashes.

OS:
Windows
pg_duckdb Version (if built from source use commit hash):
0.1.0
Postgres Version (if built from source use commit hash):
16.3
Hardware:
No response
Full Name:
William Vagle
Affiliation:
Fraktal
What is the latest build you tested with? If possible, we recommend testing with the latest nightly build.
I have not tested with any build
Did you include all relevant data sets for reproducing the issue?
No - I cannot share the data sets because they are confidential
Did you include all code required to reproduce the issue?

[X] Yes, I have

Did you include all relevant configuration (e.g., CPU architecture, Linux distribution) to reproduce the issue?

[X] Yes, I have

Hi! You're looking for the postgres_scanner DuckDB extension, not pg_duckdb, which is a Postgres extension. Please re-file your issue at https://github.com/duckdb/postgres_scanner.


Remove float from currency pair

@sagikazarmark While testing issue #189 I found that CurrencyPair is using (float). I think it would be better if we get rid of it, and use (string). This also has consequences for the json representation, because that will be using quotation marks now. What do you think?

Not sure. We use string for Money as a compromise. It is very unlikely that we run into issues with the float representation.
What's the advantage?

@sagikazarmark It is not that I want to use string. I think we should we avoid float whenever we can. Maybe that a simple conversion to float via (float) is harmless, but why do we want a floating point number in our class?

The reason behind using integer is to avoid rounding issues. The reason behind using string for representing numbers is to allow using Calculators handling big integers. Other words: the reason why we don't use float is rounding.
However in this case we usually want to have a decimal value. Storing it as string, just to not store it as float does not come with any advantage that I can see. On the other hand, it is inconvenient, just like in case of Money, but there we have bigger justification for doing so.

@sagikazarmark You have a point there. I cannot see the advantage too. We should be aware though.

Do we really want to have float in our codebase? Please read the RFC on higher float precision. Especially the current behaviour of some values. We should avoid float, at almost any cost.

What's the alternative to represent non-integer values (apart from string)? Keep in mind, that casting is almost as bad as rounding issues that might occur. As an alternative, we could do this on the JSON level to preserve higher precision: cast to string when serialization happens, do the reverse when deserialization. This is a one time operation and does not mess up mathematical ones with string representation. See my reasoning above why I think string is actually worse than float.
I see the precision problem, but I highly doubt that it might cause any problems (it didn't in the past). However, I don't think we should leave it, but I don't think string is a solution. Until we find one, we should have a separate issue and mention it in the docs.

@sagikazarmark The problem is that we are casting to float. We are not securing what is going in, only that the value is numeric, not the type. Still agree though that it is highly unlikely that it causes issues.

But it is an internal, one-time operation. Also, we do that to make sure the input value is float. Ideally nothing happens, as the input should already be float. Internally afterwards we use float as the representation.

@frederikbosch can we consider this discussed and close? For now I am happy with the current solution. If we ever experience any issues regarding this, we can always reconsider this feature, although either as a bugfix or rather with a major version dump.

:+1:

@sagikazarmark I still think we should remove any reference to float in this library. Take what I had today.
php -r "echo (new NumberFormatter('nl_NL', NumberFormatter::DECIMAL))->parse('64,46') * 100;" // result: double(6446)
php -r "echo intval((new NumberFormatter('nl_NL', NumberFormatter::DECIMAL))->parse('64,46') * 100);" // result: int(6445)

If I was not convinced enough, now I am really convinced: working with float/double is really harmful. The PR I put forward was not even strict enough. We should only accepts string and throw an exception on float/double. Let the user do the math (conversion to string) outside this library. Float inside Money must be avoided at any cost.


Unable to load module.

Bug Report

Q
A

PHP version
8.0.17

SAP NW RFC SDK version
7.50.9

Extension version/git ref
2.0.0-beta2

Summary
PHP write this error when loading the module:
The procedure entry point RfcGetInt8 could not be located in the dynamic link library C:\Program Files\PHP\v8.0.17\ext\php_sapnwrfc.dll
I have tried your dll and I have tried a new build with php8.0.17, but both throw the same error.

It worked on another computer. I think I don’t find an old build garbage on the original machine.... I'm sorry to bother you

I encountered the same problem.
I've tried PHP version 8.1.0 and 8.1.4.


Widgets menu not loading

Dev Home version
0.1401.505.0
Windows build number
10.0.22631.3593
Other software
No response
Steps to reproduce the bug
i tried adding a new Widget (open the widget menu)
Expected result
I expected the widget menu to open and load all widgets
Actual result

Empty widget menu
Included System Information
No response
Included Extensions Information
No response

/logs /needsinfo


Improve error messages for MissingHost and MissingPort

These can be deceptive in their current state - I had a URL with underscores, which org.http4s.Uri was happy with, but when I used it with ember client, ip4s.Uri was NOT happy with it, so I was getting a misleading "missing host" error, even though there was a host.

Looks like the binary compatibility check failed - should I PR this into main instead?

@george-wilson-rea sorry, I missed your comment. These are private implementation details so we can just add binary-compatibility filters.

Hoe do we do that?


neonavigation: 0.8.7-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository neonavigation to 0.8.7-1:

upstream repository: https://github.com/at-wat/neonavigation.git
release repository: https://github.com/at-wat/neonavigation-release.git
distro file: noetic/distribution.yaml
bloom version: 0.9.7
previous version for package: 0.8.6-1

costmap_cspace

No changes

joystick_interrupt

No changes

map_organizer

No changes

neonavigation

No changes

neonavigation_common

No changes

neonavigation_launch

No changes

obj_to_pointcloud

No changes

planner_cspace

No changes

safety_limiter
* [Noetic] Add missing includes and libraries (#497 <https://github.com/at-wat/neonavigation/issues/497>)
* Contributors: Shane Loretz

track_odometry

No changes

trajectory_tracker

No changes

Will merge with #24998