Set login permission for authenticator role

This was confusing when I got started with the docs. Above in the docs you're using foo as the authenticator role. Later, you're giving the example of using the authenticator role for this purpose. When using CREATE ROLE, the role will not have the LOGIN permission and starting the server with postgrest postgres://authenticator@localhost:5432/mydb --anonymous anon will result in FATAL: role "authenticator" is not permitted to log in. The docs should either use CREATE USER (like you door set LOGIN when using CREATE ROLE, as I understand it.

Thanks for the fix. I must have created the authenticator role on my machine prior to testing out the example.


docs(techreport): Update link

Point to our official tech report PDF

@EbbRT-Homu r+

:pushpin: Commit f634554 has been approved by dschatzberg

:hourglass: Testing commit f634554 with merge 439d1bd...

:sunny: Test successful - runtests
Approved by=dschatzberg
Pushing 439d1bd926ae497fcd4743e57a7f81080b2cc5f3 to master...


Improve TensorFlow quantization demos

https://petewarden.com/2016/05/03/how-to-quantize-neural-networks-with-tensorflow/
https://github.com/fluxcapacitor/pipeline/tree/master/myapps/tensorflow

migrating to new Tensorflow Graph Transform Tool in Tensorflow 1.0


Enable Discussions

Enable Discussions so that we can ask questions

Done! https://github.com/patriksvensson/cupboard/discussions


chore(bolt): update lockfile

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1030
#1029  👈
#1028
#1014
#1013
#1006
#1005
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


App crashes on iOS

Hello, after flutter update to 3.0.5 app started to crash on iOS using this plugin version 2.0.0
[DATADOG SDK] 🐶 → 13:43:55.060 [ERROR] 'dd.trace_id' is a reserved attribute name. This attribute will be ignored.
[DATADOG SDK] 🐶 → 13:43:55.060 [ERROR] 'dd.span_id' is a reserved attribute name. This attribute will be ignored.
flutter: AuthenticationStatus.authenticated
2
datadog_flutter/SwiftDatadogFlutterPlugin.swift:267: Fatal error: Unexpectedly found nil while unwrapping an Optional value
* thread #1, queue = 'com.apple.main-thread', stop reason = Fatal error: Unexpectedly found nil while unwrapping an Optional value
    frame #0: 0x00000001e4a4a458 libswiftCore.dylib`_swift_runtime_on_report
libswiftCore.dylib`_swift_runtime_on_report:
->  0x1e4a4a458 <+0>: ret
libswiftCore.dylib`_swift_reportToDebugger:
    0x1e4a4a45c <+0>: b      0x1e4a4a458               ; _swift_runtime_on_report
libswiftCore.dylib`_swift_shouldReportFatalErrorsToDebugger:
    0x1e4a4a460 <+0>: adrp   x8, 335240
    0x1e4a4a464 <+4>: ldrb   w0, [x8, #0x45c]
Target 0: (Runner) stopped.
Lost connection to device.

Also for some reasons i see another error in here dd.trace_id' is a reserved attribute name

Yeah, we are also having it when upgrading from 3.0 to 3.3. Also, the official Datadog flutter simply does not work.

Sorry - this plugin is deprecated. Please open any issues with the official plugin on their repo at https://github.com/DataDog/dd-sdk-flutter


Chrome click issue

Hello! Is this issue still not fixed? I have this bug constantly.
https://github.com/nolimits4web/Swiper/issues/1283
https://github.com/nolimits4web/Swiper/issues/1256

Nope, it is Chrome + Windows issue, not the Swiper's. Don't need to open duplicate issues


Not all ACS locations actually available for k8s

From @mboersma on March 8, 2017 15:55
In the Azure Quick Start docs, we suggest running this command to discover what locations exist for container services:
$ az account list-locations --query [].name --output tsv
eastasia
southeastasia
centralus
eastus
eastus2
westus
northcentralus
southcentralus
northeurope
westeurope
westus2
....

However, many of those locations will later fail when trying to actually create the service:
$  az acs create --resource-group="${AZURE_RG_NAME}" --location="${AZURE_DC_LOCATION}" \
>   --orchestrator-type=kubernetes --master-count=1 --agent-count=1 \
>   --agent-vm-size="Standard_D2_v2" \
>   --admin-username="matt" \
>   --name="${AZURE_SERVICE_NAME}" --dns-prefix="mboersma" \
>   --ssh-key-value @/Users/matt/.ssh/id_rsa.pub
creating service principal.........done
waiting for AAD role to propagate.done
The provided location 'westus2' is not available for resource type 'Microsoft.ContainerService/containerServices'. List of available regions for the resource type is 'japaneast,centralus,eastus2,japanwest,eastasia,southcentralus,australiaeast,australiasoutheast,brazilsouth,southeastasia,westus,northcentralus,westeurope,northeurope,eastus'.

The first command should be replaced with one that lists only the appropriate regions.
Copied from original issue: deis/workflow#761

From @bacongobbler on March 8, 2017 16:12
I actually opened an issue on the azure cli for this as well: https://github.com/Azure/azure-cli/issues/2396


BrowserModule has already been loaded error when importing an angular library that imports BrowserModule

I am trying to use a component from an angular library  and get the console error:
BrowserModule has already been loaded. If you need access to common directives such as NgIf and NgFor from a lazy loaded module, import CommonModule instead.
Both the SharedModule from the library and the AppModule in the app have the imports
import { BrowserModule } from '@angular/platform-browser'; import { BrowserAnimationsModule } from '@angular/platform-browser/animations';
The library is imported into the AppModule using
import { SharedModule as pdlSharedModule } from 'pdl';
If I remove the Browser imports from pdlSharedModule and try to use  instead get the console error message
NullInjectorError: R3InjectorError(t)[y -> y -> y -> y]: when I try to access
If I remove the Browser imports from AppModule that gets the same NullInjectorError but on the first page.
I am just not sure of the correct way to import an angular library with angular-ngrx-material-starter.  I am on angular cli version 12.2.6

Angular support has indicated this looks like an angular bug. https://github.com/angular/angular/issues/44015


On Hover Validation

Make validation step a visual process where users can see what they can/cannot add. If they proceed to drop the item anyway, they will be shown a notification for why the change was not processed.

Potentially related to #786 .


Content changes to same sex marriages in Germany

Supersedes #2571
Trello card

Content changes to same sex marriages in Germany

Factcheck Same sex in Germany

user lives in Germany partner is German
Preview link
GOV.UK

user lives elsewhere and partner is German
Preview link
GOV.UK

user lives in the UK and partner is German
Preview link
GOV.UK

Expected changes

Update on Content where:

user lives in Germany or elsewhere and partner is German
user lives in the UK and partner is German

Before

After

Expected changes

Update on Content where:

user lives in the UK and partner is German

Before

After

Affected pages
https://smart-answers-pr-2572.herokuapp.com/marriage-abroad/germany/ceremony_country/partner_local/same_sex
https://smart-answers-pr-2572.herokuapp.com/marriage-abroad/germany/third_country/partner_local/same_sex
https://smart-answers-pr-2572.herokuapp.com/marriage-abroad/germany/uk/partner_local/same_sex

LGTM

LGTM 👍


fix(FDS-397): [Table] - fix broken styles

Resolves FDS-397.
Fix a bug that was caused by a bad renaming of css classes.
PR Author Checklist

[x] I have finished reviewing the contents of this PR for accuracy
[x] If needed, I have included a Changeset and it follows Semantic Versioning principles
[x] I have completed all of the above and have enabled auto-merge for this PR

Fails

:no_entry_sign:
Please add a '### Snapshots' section to explain the reason we are changing snapshots.

  Generated by :no_entry_sign: dangerJS against 693338cf46290a0a1f351e9a32e12481d7c613a7

@brianespinosa I think it makes more sense to just rename Table and keep all children as they are, do you agree? Also, does this PR requires a changeset?


Simplify ARM content injection script

Is your feature request related to a problem? Please describe.
Simplify ARM content injection script so that we can provide better error messages in case something goes wrong

Moved to next milestone, to allow us release version more quickly, so we can already use the newly created scripts.


Initialize model callbacks on boot

See commits.
Manually mixing the behaviour in to ActiveRecord classes does not achieve what we want to achieve: it missed out HABTM relations. So hook up everything automatically — analytics.yml will determine what gets sent. As part of this piece of work we now refuse to boot if that file isn't correct, which should surface configuration issues as early as possible.
Depends on #36.

Might be worth baseing this PR onto the with_model PR, just to keep it's diff set clean?

OK: all updated


adapt yurt-controller-manager and yurthub for kubernetes 1.16.9

fix #62
and yurthub and yurt-controller-manager components are adapted, @charleszheng44 please take care of yurtctl

and image version updated to v0.1.1


test(datasets): skip tensorflow tests on Windows

Description

Tests are flaky; they pass if you run them enough times.
Development notes

https://kedro-org.slack.com/archives/C054U0LJLAC/p1696272611234859?thread_ts=1693846107.068859&cid=C054U0LJLAC
Checklist

[x] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added a description of this change in the relevant RELEASE.md file
[ ] Added tests to cover my changes

Tests are flaky; they pass if you run them enough times.

Thinking about this, wonder if it makes sense to use something like https://github.com/box/flaky? It's used fairly widely, but it's not that actively-maintained (not that need new functionality). I'm just not sure if we even want the tests slowed down rerunning the test X times.

Tests are flaky; they pass if you run them enough times.

Thinking about this, wonder if it makes sense to use something like https://github.com/box/flaky? It's used fairly widely, but it's not that actively-maintained (not that need new functionality). I'm just not sure if we even want the tests slowed down rerunning the test X times.

We should create a follow up ticket to explore that.

I still feel like this is pretty low priority. Like, who knows; maybe the issue will just go away once the deprecation stuff is removed in 2.0.0. :D


Improve failures matching

Instead of throwing a RuntimeException with the log message not found, would be helpful to display the logs captured.

Done f54c2fdb588919afa97872960a342f1083c5af3a


Unnamed Layers in Sample ResNet-50 Cache File

@rmccorm4 Hi Ryan,
You have put a sample resnet50.cache file you in the below link:
https://github.com/rmccorm4/tensorrt-utils/blob/master/int8/calibration/caches/resnet50.cache
There are some unnamed layers at the end of the sample. Why are these layers unnmaed?
gpu_0/pool5_1: 3ec805a6
OC2_DUMMY_0: 3d9942c2
(Unnamed Layer* 174) [Constant]_output: 3996cbd8
(Unnamed Layer* 175) [Constant]_output: 3b64f8bc
(Unnamed Layer* 176) [Matrix Multiply]_output: 3e5b5330
gpu_0/pred_1: 3e526e4b
gpu_0/softmax_1: 3afb76dc
Thank you.

Hi Hassan, the reasoning for unnamed layers is likely the same across all models/frameworks: https://github.com/rmccorm4/tensorrt-utils/issues/18#issuecomment-650826000
I don't know too many details on why this happens, but likely just no 1:1 mapping between ops somewhere in the conversion code.

@rmccorm4 Thank you for your response. Do you know who has the information in NVidia, so I can ask from him/her?

Hi @Hassan313 , I would try asking on the NVIDIA Developer forums for TensorRT: https://forums.developer.nvidia.com/c/ai-deep-learning/libraries-sdk/tensorrt


windows support

Hey man, thanks for merging in the PR, but could you please publish a new package to the npm?

Done. Sorry about that. My workflow wasn't working.

Done. Sorry about that. My workflow wasn't working.


Resource Manager permissions bug/issue in DNN 9.8.1 and above (HTML Editor Browse Server showing all folders)

I believe the Resource Manager has a folder permissions bug/issue. The HTML Editor "Browse Server" is showing ALL folders to Roles after upgrading from DNN 9.7.2 to 9.8.1 no matter how the permissions are set in the Resource Manager.
When using the HTML Editor, selecting the image or file icon, selecting the "Browse Server" button. The Folder Tree shows all folders no matter what the folder permissions are set to for Roles in the Resource Manager. In DNN 9.7.2 I had folder permissions set so that I could hide some of the folders from certain Roles, but after upgrading to 9.8.1 it shows all the folders.
I have tried different folder permission settings in the PB > Site Assets, but nothing works. I have even tried replacing with the new Resource Manger but that did not work either.
I have tested this on a test production site and a test dnndev.me site and both have the same issue.
To replicate the issue. Go to the Site Assets and assign folder permissions on a folder to hide it from a "Role". Then login as a user assigned to that Role and use the HTML module "Browse Server" and you will see the folder you are trying to hide.
I have attached a couple of screen shots.

Thanks,
Craig

Also, if you have even one folder, with the "Open File in Folders" and the "Write to Folder" set to deny, then the Folder Tree shows NO folders at all.

Using deny permission should only be used, if necessary and if global roles "all Users" are granted access.
Deny for "all users" role should be disabled, because it prevents anyone from accessing files or subfolders.

Even if I only have the "Browse" checked and the "Open Files in Folder" and "Write to Folder" not set. I can still see the Test folder when logged in as the Role. Previously when our site was on DNN 9.3.2 this is how I had it set. Then after upgrading to DNN 9.7.2 I had to set the folders I did not want the Role to see to deny "Open Files in Folder".
Can you test your DNN site and see if you can restrict view to a folder to a role?
Thanks.

We are a local City Government and I have around 20 Roles setup. I need to have the Site Assets folder permissions set so All Users cannot see all the folders when using the HTML Editor to "Browse Server". Then I assign permissions in the Site Assets for only certain Roles to see their folders. This use to work prior to DNN 9.8.1
Note: the permissions should also be so that anyone, even the public, could view a file or image in the folder when using their browser.

you don't need to deny permission for "all users" - this is default.
Access will only be possible for users in roles with dedicated grants. "Deny" option is meant to be used for situations, where you grant (explicitely) for a large role like "registered users" and want to exclude just members of a smaller role (like "editors"),
I will test folder permissions later

I understand. I only used deny after I upgraded to DNN 9.7.2 because the permissions were not working properly. Now in DNN 9.8.1 no matter how I have the permissions set in Asset Manager you cannot hide folders per Roles. Yes, please let me know how your tests go. Thanks

Has anyone tested to verify?

Can anyone test their DNN site 9.8.1 and above, and see if you can restrict viewing a folder to a "Role" when using the HTML Editor "Browse Server"?
To replicate the issue I am describing above, go to the Site Assets and assign folder permissions on a folder to hide it from a "Role". Then login as a user assigned to that Role and use the HTML module "Browse Server" and you will see the folder you are trying to hide.

I tested this out, first with a fresh install of DNN 9.9.1 before and after removing the Digital Asset Manager and the permissions do not work correctly as described.  I then installed a fresh copy of DNN 9.3.2 and confirm they do work accordingly.  I then upgraded to the DNN 9.7.2 and the users that should not be able to see a specific folder now can when browsing for files in the CKE file browser.  Using the "deny" does stop users that should not see it from seeing it, but as @sleupold said, by default the users should not be able to see the folder or its contents without being given permissions to do so.
I don't think the problem is in the Resource Manager or the Digital Asset Manager, but rather with the CKE editor's file browser.

Thank you for testing and verifying! Everything you said about 9.3.2, then upgrading to 9.7.2, and then to 9.8.1 or above is true and things I noticed when I upgrading. The only thing I would ask is, did anything change with the DNN CKE editor from versions 9.3.2 to 9.7.2 and then 9.8.1. If not, then you would think it would have to be something with the Core DNN software and maybe the Resource Manager. My thoughts were that it was with the Resource Manager because that is where the most changes have been taking place lately. At this point I CANNOT upgrade to anything above 9.7.2 because of the permissions issues described in this issue. I hope someone looks into this issue soon. Thanks, Craig

Actually I'm not sure 9.7.2 is the culprit. I tested upgrading the working copy 9.3.2 and the issue is partially introduced in version 9.4.0.  I found that once I move to 9.4.0 the test user that was given rights to browse folder and files can see the test directory they should but can't see the files, and they can also see a second test folder and files in it that no users but the admin was supposed to see.  I'm going to keep testing and checking the changes from version to version in the 9.4.x path.

Thanks for looking into this further, I really appreciate it. I was looking back through the version notes of DNN above 9.3.2 and below is what I found. Could it be related to any of these updates/fixes?
CKE editor related fixes:
Noteworthy Changes in v9.8.1
Fixed an issue where Denying the ADD permission for a role on an asset would make it invisible to users on CKE Editor. #4364 Thanks @mikebigun
Noteworthy Changes in v9.8.0
Merged CKEditor Provider in our main repository #4096 Thanks @bdukes
Release Notes for v9.4.1
Fixed an issue where the CkEditor provider would have the dll twice in the install package. Thanks @valadas #126
Permissions, Resource Manager or the Digital Asset Manager related fixes:
Noteworthy Changes in v9.8.1
Fixed an issue where Denying the ADD permission for a role on an asset would make it invisible to users on CKE Editor. #4364 Thanks @mikebigun
Noteworthy Changes in v9.8.0
(there are a number of changes to the Asset Manager)
Noteworthy Changes in v9.4.3
Fixed an issue where Select All was not working in site assets. Thanks @mikebigun #3251
Noteworthy Changes in v9.4.2
Updated default portal template so it provides default permissions on portal folders. Thanks @SCullman #3024
Noteworthy Changes in v9.4.1
Fixed an issue where the assets manager activity wheel would constantly spin. Thanks @berkarslan-xo #2970

You directed me right to the issue.  #4364 alters the GetFoldersByPermissions stored procedure. That change correctly fixes the check for deny permission per the UserID or User's Roles.  It also needed the same check but for when AllowAccess is true in the second part of the query.  The change can be found in my fork (yog-it).  If you want to give it a test run in your development/test environment, feel free.  Just copy and paste the code from the 09.09.02.SqlDataProvider file into your SQL Console in DNN and run.

Will this issue be resolved when issue #4649 is resolved?
I cannot upgrade our website from DNN 9.7.2 because of this issue.

@sleupold you have any input on the correct fix for this issue ?

@yog-it  says that his #4649 fixes this issue. I was wanting to know if it would be resolved in DNN 9.10.0?

Well #4649 was refused due to the solution not being correct following up a discussion with @sleupold so that's why I pinged him up as I don't exactly remember what was wrong...

@cgentryls the code I originally wrote was not correct and @sleupold had already been working on a solution that he provided the code for.  We altered the PR with his code, but he may still be working on it, so the PR was closed.  I think he'll post a full solution with his code in a PR to solve this issue.

I am still looking for a proper solution - my stored procedure does not handle all permission correctly.
besides, I noticed another issue with API code using a different way of checking permission, which is not efficient.

Do any of you know when this issue will be resolved? I cannot upgrade our website from DNN 9.7.2 because of this issue.

This being an open source project, we never know when any specific issue will get fixed until someone submits a fix. @sleupold any progress on this?

I propose I re-submit my code again from the original PR to fix the issue addressed. I understand that the current SP is in need of more work. I can see that by the wonderful code that was provided. I can also help out @sleupold with the efficiency and use of permissions issues you mentioned if you would need a hand.  Version 10 could have a better working version of this SP.

@mitchelsellers crystal clear 👍


[Diagnostics] In DefineMemberBasedOnUse::diagnoseForAmbiguity, use the base type from each solution

Previously, the code only used the base type from the first solution. This resulted in potentially calling solution.simplifyType on a type that involved type variables that didn't appear in that solution.
Resolves: rdar://problem/61005878

@swift-ci please smoke test

@swift-ci please smoke test

Oh, yeah that's easy, thanks!

@swift-ci please smoke test

@swift-ci please smoke test OS X platform


[gql_exec] Print operation

errors are hard to parse without having the actual operation printed.
@klavs On the other hand, maybe that's ok/desired so that the output is legible?

No strict opinions here. Looks good.


Bubblewrap doesn't work on Konsole Wayland if launched with a KDE shortcut

Describe the bug
EDIT: This might affect other programs too.
Attempting to use the command bwrap in Konsole Wayland launched with a KDE shortcut (or children of it, whether it's steam-run or another terminal launched from it), will return the error:
bwrap: Unexpected capabilities but not setuid, old file caps config?

No matter the arguments (even --help).
Yes, Konsole needs to be launched with a KDE shortcut. And it must be on Wayland. You also must not have root access (not using sudo). It happens no way else. It's THAT specific.
Steps to Reproduce

Place the following file in $HOME/.config (remove the .txt, it's there because GitHub forces it): kglobalshortcutsrc.txt;
Start a Plasma Wayland session (the file must be already placed);
Launch Konsole with CTRL+ALT+T.
Enter bwrap.

Additional context
It seems that something is causing capget to return a different value.
The error message comes from the function https://github.com/containers/bubblewrap/blob/main/bubblewrap.c#L853.
Look at the strace logs (line 174):

With Konsole Wayland launched with KDE shortcut: bwrap-strace-fail.txt
Otherwise: bwrap-strace-success.txt

Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.18.16-zen1, NixOS, 22.11 (Raccoon), 22.11.20220806.998bc5d`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.10.3`
 - channels(root): `"nixos"`
 - channels(martino): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I finally found the exact reproducing conditions (edited the post above).
I wonder why and how they are so specific.

I have the same problem, reported it to kde before thought to find the similar issue here :)
https://bugs.kde.org/show_bug.cgi?id=457844

Can be fixed with this patch: https://github.com/kurnevsky/nixfiles/commit/4a60a78ff1519b14df6c2b1b7fe5183692e38899

Here is the fix: https://github.com/NixOS/nixpkgs/issues/185582


How to analyze offline pcap file suricata plugin?

I am using suricata as a plug-in.
Real-time traffic is well analyzed by arkime and suricata.
However, the offline pcap file is not analyzed by suricata.
I wonder how to analyze offline pacp files with suricat.

You have to change suricataExpireMinutes to include the date for when the pcap is from. See https://arkime.com/settings for more information.

You have to change suricataExpireMinutes to include the date for when the pcap is from. See https://arkime.com/settings for more information.

Reuteras,
Thanks for the speedy and helpful reply.


fix sol transaction parse bug

What does this implement/fix? Explain your changes.

replace getConfirmedSignaturesForAddress2 with getSignaturesForAddress
fix transaction parse bug

Does this close any currently open issues?
If it fixes a bug or resolves a feature request, be sure to link to that issue.
OK-2603
Pull request type

Put an x in the boxes that apply

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Where has this been tested?
local
Any other comments?
nil

@liyanhrxy 需要 rebase

@liyanhrxy 需要 rebase

👌


Binding value to momentjs object does not work

I am currently trying to edit a form and since my object is a moment.js object instead of a Date object, it's not binding to the control.
My setup:
I am currently in EDT (UTC-4).
I have a project model, with startDate and endDate – I am saving this as just a date.
With a naive implementation (Ember Data's date transform), when a date is selected, the date is then saved to the backend API without any issues. But when the date is retrieved from the API, since the time is missing from the date, it is converted to my local time, which is a day before.
To fix this, I implemented the ember data UTC transform as outlined by Tom Dale in http://blog.skylight.io/bringing-sanity-to-javascript-utc-dates-with-moment-js-and-ember-data/. This converts my date object to a momentjs object and correctly handles UTC.
While this fixed my issue with displaying dates as UTC, and I can save new bootstrap-datepicker values correctly, when I try to edit, since my date is a momentjs object, bootstrap-datepicker doesn't bind to it properly. If I convert it back to a date, datepicker binds correctly but with the wrong date (since it's UTC-4)
What's the shared community solution for this? Surely everyone deals with this?

Oh, didn't noticed yet that moment.js objects aren't supported. I would recommend a computed property: https://ember-twiddle.com/7ff74a9f6ee7028dbf712b2b02ba06b5?openFiles=controllers.application.js%2C
I guess this should be documented.


Use global timer to measure performance

This makes syntax a little bit easier. Also, measure more accurately performance
of modal solver under investigation.

Coverage increased (+0.04%) to 91.792% when pulling 8b9bf1f43c38743d9131d8665be7b45e5a520169 on fix/global_timer into b72db9398139d3cd62968fdddcd77d0f612fc064 on master.


Added Testimonial Section

Resolved issue #439
I have added a testimonial section and made it responsive.

https://screenrec.com/share/WQFNS28Gdo
@varshith257 can you please check this?

@Hemashree21 Seems you are an of RCB :)

@Hemashree21 Seems you are fan of RCB :)

Yeah I am, couldn't think of any names so I put it😁


RestartPolicyNever leaves infra pods running forever

Pods with RestartPolicyNever are leaving behind infra pods.  Those infra pods should be nuked as soon as the last successful run is completed.
For people with lots of successful / failed never pods this consumes IP addresses which is bad when IP addresses are in use.
I suspect this is because we don't have another marker that the pod has completed, but we probably need to get that fixed before 1.0.

I can work on it, if no one is already.

I will work on in later, as I don't want to duplicate the work @wojtek-t is doing while parallelizing kubelet.
I thought about it, and I can't see any 'cheap' way of checking if infra container should be killed. Namely I believe that we have to check that nothing's running and nothing will be restarted, i.e.
for each container defined in pod.Spec there exist a "recent container" in Docker which succeeded (for OnFailure RestartPolicy) or died in any way (for Always RestartPolicy).
I don't think there's any weaker condition that is sufficient and necessary, with most common corner case being an event in which Node goes crazy, and decided to kill everything running on it - such case has to be distinguished from starting a pod on a Node first time.
It's more or less what syncPods is doing right now, with the exception of applying the changes (starting/killing containers). Wojtek is already splitting this method into 'analysis' and 'application' parts, which will make change needed to fix this bug much simpler.

I think the right thing to do here is to kill the pause container, if nothing else is running and the pause pod is older than N minutes (where N is something long, like 10 minutes)
In the worst case (e.g. a really slow image pull(, we'll just re-create a pause container, when the pull completes and we try to start the other container.

I think killing is easy, figuring out if it needs to be started is harder. The way I understand the flow, it starts in syncLoop, which calls GetBoundPods, getting all the pods bound to the given Kubelet, no matter the status. Afterwards SyncPods is called on them. In SyncPods we go through each pod, and add all of them to workers which in turn run syncPod in parallel.
The whole logic of figuring out what to start is in syncPod method, but first thing it does is checking if infra container is running and starting it if it's not. After this it goes through containers and checks if anything needs to be started. Which is exactly what I meant: we need to separate analysis and starting/stopping pods.
Am I missing something, like some earlier mechanism of checking if a pod requires attention at all? Or do you mean that we should keep a timestamp of last container event (start/failure/whatever) and kill the infra container iff nothing's running and last event occurred more than X minutes ago, as a proxy for 'everything's dead for good'?

@gmarek +1 to separating analysis from action in syncLoop(). You should also sync with @dchen1107 who is taking a look at #4787. I suspect changes for both of these issues are very related.

One more related case - I was starting to look at graceful delete and the kubelet would need to know whether to restart based on the metadata persisted with the pod (a pod in graceful termination should not restart on exit).  So having separate analysis will be important there.

On Mar 3, 2015, at 11:46 AM, Victor Marmol<EMAIL_ADDRESS>wrote:
@gmarek +1 to separating analysis from action in syncLoop(). You should also sync with @dchen1107 who is taking a look at #4787. I suspect changes for both of these issues are very related.
—
Reply to this email directly or view it on GitHub.

I started working on a PR today, but I didn't manage to finish it. I got stuck under multiple different 'Container' structs and translating ones into others.
@dchen1107 are you working on #4787 now? It'll be probably much easier with my change.

I believe this was fixed by the PR #5094.

Closing as per @gmarek's PR. Thanks!


A dataset with extension types cannot be queried by DuckDB

Here's a simple reproducer.
import duckdb
import pyarrow as pa
import pyarrow.dataset
import uuid
import lance

class UuidType(pa.ExtensionType):

    def __init__(self):
        super().__init__(pa.binary(16), "my_package.uuid")

    def __arrow_ext_serialize__(self):
        # Since we don't have a parameterized type, we don't need extra
        # metadata to be deserialized
        return b''

    @classmethod
    def __arrow_ext_deserialize__(cls, storage_type, serialized):
        # Sanity checks, not required but illustrate the method signature.
        assert storage_type == pa.binary(16)
        assert serialized == b''
        # Return an instance of this subclass given the serialized
        # metadata.
        return UuidType()

storage_array = pa.array([uuid.uuid4().bytes for _ in range(4)], pa.binary(16))
arr = pa.ExtensionArray.from_storage(UuidType(), storage_array)
tab = pa.table({"uuids": arr, "normal_type": [1, 2, 3, 4]})

ds = lance.write_dataset(tab, "/tmp/foo.lance", mode="overwrite", data_storage_version="2.0")

print(duckdb.sql("SELECT normal_type from ds").fetchall())

I get the error: duckdb.duckdb.NotImplementedException: Not implemented Error: Arrow Type with extension name: my_package.uuid and format: w:16, is not currently supported in DuckDB

I've raised this with duckdb here: https://github.com/duckdb/duckdb/discussions/13931
I can workaround this using a scanner:
scanner = ds.scanner(columns=["normal_type"])
print(duckdb.sql("SELECT normal_type from scanner").fetchall())

However, that has the downside of preventing filter pushdown.
I can also work around this in a rather hacky way:
class WrappedDataset(pyarrow.dataset.Dataset):

    def __init__(self, ds, schema):
        self._ds = ds
        self.pruned_schema = schema

    @property
    def schema(self):
        return self._schema

    def __getattribute__(self, attr):
        if attr == "schema":
            return object.__getattribute__(self, "pruned_schema")
        else:
            ds = super(WrappedDataset, self).__getattribute__("_ds")
            return object.__getattribute__(ds, attr)

pruned_schema = ds.schema
pruned_schema = pruned_schema.remove(0)
wrapped = WrappedDataset(ds, pruned_schema)

print(duckdb.sql("SELECT normal_type from wrapped").fetchall())

In the meantime I suppose we could implement some kind of "dataset view" which automatically applies a projection and has a revised schema.

Perhaps if we get NotImplementedException we could do a fallback projection that strips extension metadata, and just deals with storage types.
I think that's sort of what we might want for vectors. If they don't support it, drop the FixedShapeTensorArray metadata and just provide them the FixedSizeList.

Unfortunately we do get the exception.  DuckDb grabs the schema from us and then determines it cannot handle it and raises it to the user.


YAML build pipelines example for old .NET version

.NET 2.1 is removed from all images , So C# example in this page using  windows-latest will fail with Functions 2.x project.
It is better to add example which use .NET Core task for install specific version .NET SDK
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/tool/dotnet-core-tool-installer?view=azure-devops

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0ab62930-d35d-d6d3-8bb7-2ed6b34636bb
Version Independent ID: 86725233-d4a5-a796-cd6e-c05c3675952b
Content: Continuously update function app code using Azure Pipelines
Content Source: articles/azure-functions/functions-how-to-azure-devops.md
Service: azure-functions
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Thanks @georgeOsdDev! I'll make that update.

@georgeOsdDev After investigating, I think the correct solution is to update the code sample to use Functions 3 or 4 so that we are not relying on an EOL version of .NET. I've tried to install .NET 2.1 with the .NET Core task and it doesn't appear to be supported.

The code sample is updated now so I'm closing this issue.

#please-close


feat: show hint if AENS name is in revoked state 2

Description
resolves #454
Demo
https://github.com/aeternity/aescan/assets/15363559/0259fed4-7a4b-4ce3-ab1f-32260405cdd5
Checklist:

[x] I have read and followed the Contributing Guide
[x] Add conditional hint for revoked state

if I click on Next button to see other expires names the page becomes uploading status forever. The same issue is on product.
2023-12-<IP_ADDRESS>.mov

Hmm, interesting. I tried the same scenario without any probs. Maye there was some server downtime at that moment? Can I ask you to try again today?

I found additional issue on product, when I click on next button to see other In-auction names the previous button becomes unavailable. But maybe this is middleware issue? if not, I will register it separately.
2023-12-<IP_ADDRESS>.mov
@janmichek please check it.

Yes, I confirm this issue. WIll solve it separately. Good finding, thanks

@janmichek I have a question: I can see some names in both tabs - Expired and In Auction, for example 77.chain and public.chain.  Is this expected?

https://github.com/aeternity/aescan/assets/69896204/16abfaa4-00a7-41b1-a062-47dbc7c28578

Thanks, should not be like that. I will check and possibly fix

2023-12-<IP_ADDRESS>.mov

Followup https://github.com/aeternity/aescan/issues/621


[BW Clock/BW Clock Lite/Clock Info Module] Battery gfx shows a solid filled square instead of battery icon

Affected hardware version
Bangle 2
Your firmware version
2v17.106
The bug
It now looks like this:

Before https://github.com/espruino/BangleApps/commit/cde38188150acedc2950da8d4ade74aa5283a826 there was a static battery icon where the filled square is.
Expected behavior would be for the new dynamic icon introduced with https://github.com/espruino/BangleApps/commit/cde38188150acedc2950da8d4ade74aa5283a826 to be shown, as it is on e.g. Pebble++ clock face.
Installed apps
notify (0.12)
podadrem (0.07)
spotrem (0.07)
android (0.23)
lightswitch (0.06)
widanclk (0.02)
backswipe (0.02)
torch (0.11)
calculator (0.07)
agenda (0.13)
smpltmr (0.07)
kbmulti (0.05)
widram (0.03)
widbthide (0.01)
messageicons (0.05)
widalarm (0.01)
boot (0.57)
runplus (0.18)
dtlaunch (0.23)
agpsdata (0.06)
messagesmusic (0.05)
alarm (0.41)
hrmmar (0.02)
taglaunch (0.03)
swscroll (0.03)
messagegui (0.65)
longpressbuzz (0.01)
bwclklite (0.33)
bwclk (0.32)
widmessages (0.05)
clkinfostopw (0.03)
recorder (0.23)
health (0.23)
quicklaunch (0.15)
polymath (0.01)
fastload (0.04)
sched (0.22)
fastreset (0.01)
setting (0.60)
clock_info (0.03)
messages (0.59)
drained (0.03)
widminbat (0.03)
powermanager (0.10)
pebblepp (0.01)

Ahh - thanks! That's an annoying one - so the main issue is that BW Clock/Lite set BG and FG to the same thing (which I just fixed), but then ideally asImage would be able to create an image string that was transparent.
I could hack up something for that, but I think instead I will fix it properly in the Espruino firmware as hopefully now those clocks are fixed there won't be any more immediate problems


Feature request: Drag and group rows into folders

Something like this

Would also appreciate any pointers on how to implement it myself using this library.

Is this function implemented?

@gargVader have you completed this part of the function?

This library was designed to handle reordering in a LazyList and not as a generic purpose Drag&Drop handler.

@aclassen Is there any other alternative library that can be used


[newfeature]: implement concept lib loading on GUI

When using the web interface, huggingface concept lib concepts such as <old-brno> or <poolrooms> do not get downloaded/replaced.
To debug/reproduce: put a breakpoint in triggers_to_concepts() and concepts_to_triggers(), note that neither are called when using the web UI to invoke a dog in <old-brno>

fixed by #1608


Merging search results from different collections

Again not an issue, just another query.
Does anyone know how to merge search results from different collections. I have created multiple collections to have better control over data, but I need to merge the search results from different collections.
Thanks

look at the search api. you can do federated search across multiple collections:
http://www.gigablast.com/api.html?c=main#/search
just use something like &c=mycoll1+mycoll2+...

Thanks, that was helpful.
Appreciate that.


Nixify treefmt.toml

We want to nixify treefmt.toml, and have it directly use the nix packages, in addition to having treefmt in nix shell run with the nixified configuration.
We can use https://github.com/numtide/treefmt/pull/169 but the main blocker is the lack of PRJ_ROOT (see comments).

This is now possible using "github:srid/treefmt/withConfig-stable"
          treefmt.buildInputs = [
            (inputs'.treefmt.legacyPackages.default.withConfig {
              projectRootFile = "flake.nix";
              settings = {
                formatter = {
                  nix = {
                    command = lib.getExe pkgs.nixpkgs-fmt;
                    includes = [ "*.nix" ];
                  };
                  purescript = {
                    command = lib.getExe pkgs.nodePackages.purs-tidy;
                    options = [ "format-in-place" ];
                    # Note that given how purs-tidy is designed (takes glob, rather than explicit
                    # list of files), these patters inevitably slow down purs-tidy (thus treefmt)
                    # during the initial run.
                    includes = [
                      "src/**/*.purs"
                      "core/**/*.purs"
                      "lib/**/*.purs"
                      "src/**/*.purs"
                      "domain/**/*.purs"
                      "infrastructure/**/*.purs"
                      "ui-guide/**/*.purs"
                      "test/**/*.purs"
                    ];
                  };
                };
              };
            })
          ];

May as well retire this repo right away by moving it to upstream repo after https://github.com/numtide/treefmt/pull/169 gets merged to master.


add binary suffixes

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Does this need to be done in both services?


Update readme.md

Learning Pull request , commits and etc. how all works.

This isn't a useful or meaningful change, and I'm not really sure why you picked this project to learn on?
If you want to learn how to use git, please practice on your own repositories. Doing this is considered to be spammy and a waste of maintainer's time.
Thank you.


Fetch from DNS when a known DSP is added

When someone uploads a DSP that is already known, with current behavior no new DNS lookup is performed. Refreshing will happen in the recurring batch job.
Perform re-fetching if the DSP has not been fetched the last 1 hour.

Olof, [2024-02-25 08:41]
should we also do re-fetching from DNS when someone uploads an already known domain/selector-pair?
(note that then, popular domains such as twitter, will be re-fetched all the time, as almost everyone has them in their inbox)
Yush G, [2024-02-25 08:48]
yeah maybe we can have a 1hr cooldown for the email uploads and a 10 sec cooldown for the refresh button?

fixed by 6da717d6e458467f69fef26b0c081b2c7c51208b


OpenVR HMD not tracking

Hi,
I'm testing the latest v2.6.0b1 beta release with Unity 2019.3.12f1 using a StarVR headset (OpenVR-based). HMD tracking is not working with either the SteamVR [CameraRig] or an XR Tracked Pose Driver component. The headset does render the scene correctly but does not respond to any physical movement.
The steps I've taken are as follows:

Create a new 2019.3.12f1 project using the 3D template
Install both XR Plugin Management and OpenVR XR Plugin through the Package Manager
Enable the OpenVR Loader under Project Settings
Test by opening the SteamVR/Simple Sample scene and entering Play mode
Test by creating a new empty scene and adding a Tracked Pose Driver to the Camera game object and then entering Play mode

I've uploaded the SteamVR System report and editor log here.
Is it possible to have other HMDs using OpenVR such as StarVR supported?

It looks like steamvr recognizes the headset but unity doesn't for some reason. Your unity editor log is oddly empty, can you try to enter play mode with it again (with a tracked pose driver component) and reupload a log?

Thanks for the feedback. Here are the logs for running with a tracked pose driver using both StarVR and Vive Pro Eye headsets for comparison (tracking works fine using Vive).

I forgot to mention: when exiting play mode using the StarVR HMD the following error shows up in the console:
[XRInputSubsystem] A device disconnection with the id 0 has been reported but no device with that id was connected.

@zite would it be possible to get a quick update on this? If the problem seems to be on Unity's side then I'll reach out to their support and raise the issue there. Thanks

@zite @1runeberg any chance to get an update on this?

this is ridiculous... A full month to just have a look at a log file... Great support

Any update with this issue? I'm currently having the same problem.

Any update on this issue? I'm currently having the same problem.

With Star VR, or another headset?


fix: fix event duration on mode month

please merge hot fix for mode month lib @acro5piano

@Pakile Thank you for your contribution! It was a huge bug so far.
Also thank you @cannn5 to ping me.
It was released on v3.2.3
https://github.com/acro5piano/react-native-big-calendar/releases/tag/v3.2.3


[Bug] Hard disk temperature read fail.

Describe the issue

Some disk drive temperature information is missing or 0.
How to reproduce the issue

Steps to reproduce the behavior:
I removed some drive from the pool and set it online again (not the temperature read fail one), but same pool. The Ada0 is not upgrading temperature.
Expected behavior

Screenshots

I run my shell awk script to get de hdd temperature, and works well.
_#! /bin/sh
for i in $(sysctl -n kern.disks)
do
DevTemp=smartctl -a /dev/$i | awk '/Temperature_Celsius/{print $0}' | awk '{print $10 "C"}'
DevSerNum=smartctl -a /dev/$i | awk '/Serial Number:/{print $0}' | awk '{print $3}'
DevName=smartctl -a /dev/$i | awk '/Device Model:/{print $0}' | awk '{print $3}'
echo $i $DevTemp $DevSerNum $DevName
done_
With this result (right one):
root@Temido[~]# ./disktemps.sh
ada5 48C K42249R001653 Lexar
ada4 25C Z52B666K ST2000VN004-2E4164
ada3 33C ZFL37NA3 ST2000DM008-2FR102
ada2 27C WD-WCC4M3LGH3 WDC
ada1 27C Z52BBEF6 ST2000VN004-2E4164
ada0 25C 5YD1DRGT ST2000DL003-9VT166

A Truenas Integration Screenshoot appears as 0 (also in developer tools).
Software versions

Home Assistant version:
TrueNAS integration version:
TrueNAS version:

Diagnostics data

Traceback/Error logs

Additional context

is the temperature for ada0 visible in TrueNAS directly? If it is, I will need debugs.

I cleared smart options of the HDD and it became to 27º. Will keep and eye on it!

ok, remember to check in truenas gui too, if it can see data.

Got it. In the GUI did not appear
El dv., 15 d’abr. 2022, 3:14, Tomaae @.***> va escriure:

ok, remember to check in truenas gui too, if it can see data.
—
Reply to this email directly, view it on GitHub
https://github.com/tomaae/homeassistant-truenas/issues/14#issuecomment-1099766174,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMI2DA7CJWCKAF3B2P53KXDVFC7IFANCNFSM5TME2SKQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

yea, if it did not appear in truenas gui too, that will have to be taken care by truenas team.

closing as this is not related to integration


spi: Adds mock implementation for WriteIter

Happy for a few runs at this as I'm relatively new to rust.

@silvergasp are you interested in doing the suggested change, or should someone else take over your branch?

Oh yeah sorry I missed this one, I'll take another pass at it

@silvergasp friendly reminder 🙂 If you don't have time, just let me know.

Hmm, I just pushed a version that calls self.write, but I realized that while it re-uses code, it makes error reporting worse (since error messages on missing expectation will mention spi::write, and not spi::write_iter). I'll undo that commit.


support generating codes for service?

support generating codes for service?

Why would they be used?
Generic service definitions are not useful for grpc. grpc or grpcio crates have own generators of services.

These are useful for RPC systems that aren't gRPC, like Twirp.  I'm sure you're well aware but the usual support for this is quite limited - in Java, protoc will just generate an interface, for example.  I think having rust-protobuf just generating a simple trait would be quite helpful.


Thanks for this project

Dear Julien,
Thanks for making the cover as well as flat panel available.
Converted the cad files to Fusion360 to make them fit my scope. Combined both flat panel and cover functionality.
Honesty my mistake of not having used an SAMD21 / Seeeduino XIAO before.
Instead the usual servo lib I'm using one called slowServo allowing good control of the speed and it is working asynchronously and not blocking when servo moves. It therewith better follows the ASCOM specs like reporting status when moving.
The fact of fast closure move on power-up when cover is not closed I could overcome by writing either panel open, closed or moving to the Flash memory of the XIAO. Therewith current cover status remains on power up, unless the XIAO is disconnected while servo is moving. In this case the cover opens on the re-connection, as feels more save to me before moving fast into the dew shield.
3d parts are printed, cover is moving but wiring is still on a breadboard. Will assemble it next days and happily share some pictures.
Kind regards from Germany,
Alex

Thanks for the kind words, Alex! These are really good suggestions, and I will incorporate some of them into the version 2 of this project, next year. Among other things, I want the cover to know which state it is in when powered up, and this can be accomplished using a potentiometer and a bit of logic in the software. Tschüss!

By the way regarding reading the cover position. I havent been aware they exist but just ordered two of these:
Feetech FB5118M Digital Servo 7.4V 17kg.cm with Position Feedback
There is a 4. wire linked to the internal potentiometer of the servo and therewith having the build-in capability of providing postion.

I did not know that some servos had position feedback. I just ordered one on Amazon, I’ll be experimenting with it. If it works, this would be a brilliant solution! Thanks a lot :)


group balancer: support KIP-792

This improves problems in the face of re-joining consumer zombies.

Depends on #304


Deprecate public methods and variables that contain 'master' terminology in class 'NoMasterBlockService' and 'MasterService'

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4006 (fae7a08) into main (efde8c5) will decrease coverage by 0.12%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##               main    #4006      +/-   ##
============================================
- Coverage     70.51%   70.39%   -0.13%
+ Complexity    56800    56712      -88
============================================
  Files          4573     4573
  Lines        273476   273484       +8
  Branches      40101    40101
============================================
- Hits         192844   192521     -323
- Misses        64420    64687     +267
- Partials      16212    16276      +64

Impacted Files
Coverage Δ

.../opensearch/common/util/concurrent/BaseFuture.java
62.71% <0.00%> (ø)

.../org/opensearch/cluster/service/MasterService.java
81.79% <20.00%> (-2.09%)
:arrow_down:

...g/opensearch/cluster/coordination/Coordinator.java
78.00% <50.00%> (+3.03%)
:arrow_up:

...nsearch/cluster/coordination/JoinTaskExecutor.java
74.43% <100.00%> (-1.44%)
:arrow_down:

...ter/coordination/NoClusterManagerBlockService.java
95.65% <100.00%> (-4.35%)
:arrow_down:

...ster/coordination/AbstractCoordinatorTestCase.java
93.83% <100.00%> (+1.79%)
:arrow_up:

...c/main/java/org/opensearch/test/RandomObjects.java
94.80% <100.00%> (ø)

...java/org/opensearch/client/indices/DataStream.java
0.00% <0.00%> (-76.09%)
:arrow_down:

.../opensearch/client/indices/CloseIndexResponse.java
17.50% <0.00%> (-73.75%)
:arrow_down:

...ch/transport/ReceiveTimeoutTransportException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

... and 474 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


[Xcode-15] Multiple commands produce fail

Multiple commands produce '/Users/i.romanenko/Library/Developer/Xcode/DerivedData/BSB-Mobile-gopohlcxmrgqtndtowscxqyljieu/Build/Intermediates.noindex/BSB-Mobile.build/Debug-iphonesimulator/BSB-Mobile.build/Objects-normal/arm64/R.generated.stringsdata'
I'm try build project on Xcode 15. In 14.3.1 still worked.
This situation reproduced in new clean project too
Please, anyone, help.

I can't help you. Sorry man. I have same problem.

This happened when I upgrade from the Podfile to the SPM. Try updating your podfile and execute pod install to clean things up.

Still a problem. Reverting to R.swift 6. Which does not support Strict Concurrency.

Just added R.swift 7.5.0 to a fresh Xcode 16 project. Except that Folder references aren't supported (you must convert it to groups) everything does work out of the box.
If something is broken please do describe the steps from Create new project until you get the error so we can investigate it!


[SPARK-21848][SQL] Add trait UserDefinedExpression to identify user-defined functions

What changes were proposed in this pull request?
Add trait UserDefinedExpression to identify user-defined functions.
UDF can be expensive. In optimizer we may need to avoid executing UDF multiple times.
E.g.
table.select(UDF as 'a).select('a, ('a + 1) as 'b)

If UDF is expensive in this case, optimizer should not collapse the project to
table.select(UDF as 'a, (UDF+1) as 'b)

Currently UDF classes like PythonUDF, HiveGenericUDF are not defined in catalyst.
This PR is to add a new trait to make it easier to identify user-defined functions.
How was this patch tested?
Unit test

Yes. This is a potential optimization we should do, but it should be cost based. Let us improve the resolution logics using this common base trait in this PR first.

Test build #81174 has started for PR 19064 at commit 5f508ab.

Test build #81177 has started for PR 19064 at commit de642d9.

Test build #81174 has finished for PR 19064 at commit 5f508ab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #81177 has finished for PR 19064 at commit de642d9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81174/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81177/
Test PASSed.

Merged build finished. Test PASSed.

I'm +1 on this, currently we are pattern matching ScalaUDF in optimize rules which is suboptional.

I'm +1 on this, it should be useful to pattern match the trait instead of handling ScalaUDF/ScalaUDAF/PythonUDF... in future optimize rules.

Test build #81192 has started for PR 19064 at commit 7b67cd3.

Test build #81192 has finished for PR 19064 at commit 7b67cd3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81192/
Test PASSed.

Test build #81202 has started for PR 19064 at commit be85df3.

It is better to review PR title and description to reflect the change from UDFType to UserDefinedExpression.

Test build #81202 has finished for PR 19064 at commit be85df3.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81202/
Test FAILed.

retest this please.

Test build #81204 has started for PR 19064 at commit be85df3.

Test build #81204 has finished for PR 19064 at commit be85df3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81204/
Test PASSed.

LGTM

Thanks! Merged to master.


PR #801 to postgres image breaks docker-entrypoint initialization logic in CI environments in k8s

Pull request https://github.com/docker-library/postgres/pull/801 breaks the initialization logic of docker-entrypoint script if the postgres runs as a sidecar container to the main one in a CI environment and has PGHOST variable defined.
psql command tries to connect to postgres daemon and respects the well-known PGxxx variables during the try.
if PGHOST equals to <IP_ADDRESS> (or localhost), the client fails with connection refused error breaking the entrypoint script:
2021-01-18 16:44:23.932 UTC [489] LOG:  database system was shut down at 2021-01-18 16:44:21 UTC
2021-01-18 16:44:23.936 UTC [488] LOG:  database system is ready to accept connections
 done
server started

psql: could not connect to server: Connection refused
	Is the server running on host "<IP_ADDRESS>" and accepting
	TCP/IP connections on port 5432?```

I'm kind of confused here -- why are you setting the client parameter PGHOST on your server container?

I'm kind of confused here -- why are you setting the client parameter PGHOST on your server container?

what about our specific case - we're using gitlab for our CI and it provides 'service' entity which is a kind of abstraction for some auxiliary services for the main build container. good examples of service are different DBs, caches and similar stuff which are needed for run the integration tests or so (instead of, e.g., using test-containers library). the implementation details of the service abstraction depends on the platform gitlab-runner (jenkins-slave or bamboo-agent equivalent in gitlab) deployed to. in k8s they are implemented as sidecar containers (2...N containers in k8s next to the main one sharing the same linux network namespace) in the same build Pod. by default, service containers see the same set of environment variables as the main build container exposed to. that is we provide the build containers with the PGHOST and sibling environment variables and the service see the same them too.

what about our specific case - we're using gitlab for our CI and it provides 'service' entity which is a kind of abstraction for some auxiliary services for the main build container. good examples of service are different DBs, caches and similar stuff which are needed for run the integration tests or so (instead of, e.g., using test-containers library). the implementation details of the service abstraction depends on the platform gitlab-runner (jenkins-slave or bamboo-agent equivalent in gitlab) deployed to. in k8s they are implemented as sidecar containers (2...N containers in k8s next to the main one sharing the same linux network namespace) in the same build Pod. by default, service containers see the same set of environment variables as the main build container exposed to. that is we provide the build containers with the PGHOST and sibling environment variables and the service see the same them too.

Doh, that's unfortunate, and also kind of a miracle it ever worked honestly (because any of our more advanced initialization behavior would've caused the exact same failure).
I guess we need to start filtering/overriding more known-bad client variables in our psql invocation. :confused:

Doh, that's unfortunate, and also kind of a miracle it ever worked honestly (because any of our more advanced initialization behavior would've caused the exact same failure).
I guess we need to start filtering/overriding more known-bad client variables in our psql invocation. :confused:

It would be helpful to get some testing/validation on https://github.com/docker-library/postgres/pull/808 :+1:

It would be helpful to get some testing/validation on https://github.com/docker-library/postgres/pull/808 :+1:


feat(avm-simulator): external calls + integration

This PR finishes the external call opcodes, adds the necessary aztec.nr machinery and tests. It's based on David's work.
Relates to #4313, #4127.

#5051  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fcarreiro and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

note_history_successful_decrypting_time_in_ms (5): 1,243 (-47%)
note_history_successful_decrypting_time_in_ms (10): 2,400 (-21%)
note_trial_decrypting_time_in_ms (8): 107 (+1075%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 48bd22ea and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,700
18,884
36,452

l1_rollup_calldata_gas
66,360
238,940
470,120

l1_rollup_execution_gas
194,308
500,246
909,378

l2_block_processing_time_in_ms
1,185 (-1%)
4,475
9,068 (+1%)

note_successful_decrypting_time_in_ms
199 (-1%)
536 (-1%)
1,060 (+6%)

note_trial_decrypting_time_in_ms
:warning: 107 (+1075%)
57.8 (-9%)
61.1 (-53%)

l2_block_building_time_in_ms
11,878 (+6%)
47,038 (+6%)
93,980 (+5%)

l2_block_rollup_simulation_time_in_ms
7,460 (+2%)
29,749 (+2%)
59,392

l2_block_public_tx_process_time_in_ms
4,388 (+12%)
17,226 (+13%)
34,471 (+13%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,351 (-11%)
27,290 (-5%)

note_history_successful_decrypting_time_in_ms
:warning: 1,243 (-47%)
:warning: 2,400 (-21%)

note_history_trial_decrypting_time_in_ms
91.3 (-8%)
130 (-30%)

node_database_size_in_bytes
18,767,952
35,434,576

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
287 (+9%)
44,736
28,001

private-kernel-ordering
209 (+3%)
52,625
14,627

base-rollup
671 (-5%)
177,932
997

root-rollup
68.4 (-3%)
4,320
825

private-kernel-inner
368 (+12%)
73,715
28,001

public-kernel-app-logic
200 (+10%)
32,254
25,379

merge-rollup
8.58 (-2%)
2,840
997

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts

tx_size_in_bytes
19,179

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
2,934 (+9%)
1,478 (+4%)

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.0317 (-11%)
538 (+7%)

Merge activity

Mar 11, 6:51 AM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


3.0.0-5, indented sass: spaces at EOF throw error

combing (without any config file):
body
  background-color: red
  color: white
  float: left
  width: 100%
  padding: 4em

(on the last line are only two spaces.)
throws:
stack:  Error: Parsing error at test/_test-partial1.sass: Cannot read property 'type' of undefined
    at cssToAST (/usr/local/lib/node_modules/csscomb/lib/csscomb.js:464:19)
    at processString (/usr/local/lib/node_modules/csscomb/lib/csscomb.js:351:20)
    at /usr/local/lib/node_modules/csscomb/lib/csscomb.js:323:39
    at Array.0 (/usr/local/lib/node_modules/csscomb/node_modules/vow/lib/vow.js:555:56)
    at Object.callFns [as _onImmediate] (/usr/local/lib/node_modules/csscomb/node_modules/vow/lib/vow.js:1148:35)
    at processImmediate [as _immediateCallback] (timers.js:336:15)

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

This error is no longer reproducible on the latest release 👍


Alias $.fn.imageLightbox to $.fn.imagelightbox

This is a just a small quality of life tweak for end users. It's not uncommon to overlook capitol letters or mix-up which letters are capital even when shown in the readme (my experience).
This tweaks allow the end-user to invoke $.fn.imageLightbox as $.fn.imagelightbox in addition to its canonical reference.
Thoughts?

Hmm. I'm trying to weigh the added convenience for first-time users against the added byte-weight of the library forever for everyone. Not sure right now it's worth it... (= invitation to try to convince me otherwise)

@marekdedic  Feel free to reject it, I did this as a "break" from working on typescript branch :D


Web Zip file importing

fixes #1033
Initial version of minimum viable product with basic functionality for uploading a zip containing flat list of media files and then running a post process on them to create a working annotation dataset.
Completed:

Added in Zip upload Button to the upload menu.  This included changing the import-button component to have a small tag so the buttons better fit in the upload dialog.  A Secondary fact was making it so when uploading a zip file you can only specify the folder name and the annotation FPS.
Post Process will now first look for a zip file which doesn't have the ZipFileExtractedMarker marker on it and will kick off a task to extract the zip file.  On subsequent calls it will ignore the zip files which have this marker preventing zip files from being extracted multiple times.
Checks are in place for ZipBombs, based on the compression level of the zip file as well as nested Zip files.  This should hopefully prevent nefarious uploads of zip files from being used.
Zip extraction for a single flat zip file works by checking for bad zip files (Nested or high compression ratios) and then validating the media inside of the zip file using the dive_dataset/validate_files endpoint.
After extracting in the worker the data is pushed to the containing folder.  The zip file has the ZipFileExtractedMarker added to it and the dive_rpc/postprocess endpoint is called once again to perform any transcoding.
Second dive_rpc/postprocess will also move any zip files that have been processed into a source folder to keep the root of the dataset directory clean.
integration testing was implemented for zip files independently of the other upload/postprocess tests.  New zip package was added to the testing folder for all of these files.  These include tests of malformed zip files and nested zip files, as well as files which contain a high compression ratio (potential zipBombs)

@subdavis, @marySalvi, @waxlamp  - Soliciting opinions on two things related to uploading Zip files.

Should uploaded zip files allow for multiple projects to be extracted.  I.E. if you upload a zip file which contains multiple folder and each one of them is a valid dataset (verified using the validation endpoint) should it create and upload each one?

I see the argument for having this ability because it may simplify bulk importing.  I also see the counter argument for the complication that it may cause.

Should a backup of the source zip file be placed in a "SourceZip" folder or should it just stay in the root directory?

@subdavis, @marySalvi, @waxlamp - Soliciting opinions on two things related to uploading Zip files.

Should uploaded zip files allow for multiple projects to be extracted.  I.E. if you upload a zip file which contains multiple folder and each one of them is a valid dataset (verified using the validation endpoint) should it create and upload each one?

I see the argument for having this ability because it may simplify bulk importing.  I also see the counter argument for the complication that it may cause.

Should a backup of the source zip file be placed in a "SourceZip" folder or should it just stay in the root directory?

This was talked about and resulted in the following:

File and issue for multi-project import using zip, for now stay with a single flat project.
Keep the zip file in a folder named 'source' that can handle having the data

Addressed comments.

Left the zip marker in there.  I like the idea of knowing explicitly that this file has been imported, also useful if we add other items into the source folder (like the fact that we don't backup source transcoded images, but do it for source videos)
added in Job status constants instead of the integer values
the girder worker task will now do the movement of the zip file to the 'source' folder
limited the zip file processing to only the first file found in the folder

Importing Zip File Exports:
background:
An exported dataset in a single zip should also be imported properly into the system.
Contains auxiliary folder which can be ignored except for the most recent result.json file may be inside of it.
Can contain multiple video files (one transcoded and one source)
Logic:

First check subfolders (exports have a named subfolder which contains the exported data)
check for a root file in the subfolder "meta.json" - this indicates that this is a valid export zip.
base on type upload the video or imageData to the girder folder
If the type is 'video', upload the video["filename"] file and set the proper metadata - codec: h264, originalFps: {fps}, orignalFPsString: {fpsString}, source_video: false, transcoder:ffmpeg
if the type is 'video' also upload the source video and apply the proper metadata - codec:{codec}, originalFps: {fps}, orignalFPsString: {fpsString}, source_video: true
If the type is 'image-sequence' upload all identified images in the meta.json file
set the folder metadata - annotate: true, fps: {meta.fps}, type: {image-sequence | video}
upload the meta.json and the csv
Call postprocess (if everything is tagged correctly it shouldn't do anything to videos for transcoding),  This should take the csv and convert it to JSON as well as import the meta.json file to set styling and confidence filters.

Erroring:

If anything in the above logic fails it should drop back to flat file importing.
So if the meta.json is missing from subfolder (without multiple subfolders it will fall back to standard import)
Text output in the job need to be updated to indicate that there is no meta.json located in the subfolder so it is treating it as a flat import
Any misalignment between the meta.json and the contents of the zipFolder should result in an error

I'm not totally sure we want to duplicate possibly fragile logic here in the worker task.

We now have to keep track of another place where metadata gets set if logic changes (it sometimes does), which means more information that only exists in our brains.
We have new code to worry about breaking if the content of the export zip changes, which it has several times.

I think it'd be better to try and make this function as dumb as possible, leaving the decision making to postprocess where all the hard decisions about what to do with a folder full of unmarked files is already handled.
Once you know you're dealing with an export (meta.json is present) you can just push everything to the server and make the server deal with it.  Postprocess could even pick the dataset type based on what it finds so the worker doesn't need to open the meta file at all.

The thing I'm mostly guarding against here is having one logical path mimic the private behavior of another, because that puts a lot of burden on us to remember that we did that.

This isn't complete yet I still need to double check a bunch of stuff as well as create some integration tests.
Core of what this latest update does:

dataset export can be used for import now with properly setting metadata based on the meta.json
multi flat list image/video import
You can import multiple flat files in a zip file and have them extract to their own folder and become their own datasets.

TODO:

[ ] Remove auxiliary root folder on multi-imports
[ ] Clean up flat vs dataset determination when importing datasets
[ ] Create integration tests for importing multi-flat, datasets, multi-datasets, mixed flat/dataset
[ ] Test FPS markers as well as more meta.json markers properly.

I think #1099 is needed for this to work as expected.


Update annotation to support displaying real choice in searchspace

Before if you use annotation, searchspace will only display choice indices in type choice or function_choice.
Now it will display either the real value of the choice or the name of the function_choice
Fix exception printing cuz exception.arg might not be a string

Note that the example 'smartparamter' will be temporarily disabled
Further modification should be done


Setting own Model

Describe the bug
In the docs it says you can set your own Admin model
https://bitfumes.github.io/laravel-multiauth/settings.html#models
I have set the following
 'admin'          => App\Admin::class,
But this has no affect. I have set in - vendors/config/auth.php and it sets.
What am I doing wrong?
Laravel Version
7.24
To Reproduce
Steps to reproduce the behavior:

Go to 'config/multiauth.php'
Set model to local model - app/Admin.php

     dd(Auth::guard('admin')->user());

See error - Not the App/Admin auth

Expected behavior
Expected to see admin from my app folder

This is now fixed with new version
try 5.2.0

This is still not fixed..
'models' => [
'admin'          => App\Admin::class,
'role'           => Bitfumes\Multiauth\Model\Role::class,
'permission'     => Bitfumes\Multiauth\Model\Permission::class,
],
dd(Auth::guard('admin')->user());
is still showing -
Bitfumes\Multiauth\Model\Admin


Better errors page significantly slower with Puma 3.x

Hi,
I noticed the better_errors page became very slow to show the right panel. The page loads quickly, but the right panel takes several seconds to load.
I believe I have traced the problem to Puma 3.x.
Puma 2.16.0 works fast
Puma 3.0.0 and the latest 3.1.1 load the right pane slowly.

Definitely significant change in load time for the right panel from Puma 2.16 to Puma 3.1.

Same issue here when using rails s, I did notice when I run puma directly it is no longer slow.  I'm not sure the difference, but running bundle exec puma works as expected with better_errors.

Also as another note this appears to be related to the binding_of_caller gem, not better_errors...

@mikereinmiller - good observation. maybe one of us should open a ticket at binding_of_caller then?

@DannyBen Probably a good idea, I'll let you get it started since you started this thread as well. I suppose this issue on better_errors could be closed as well.

I would keep this ticket open for now, even if just for the benefit of users observing the issue and coming to report.

When running rails s, the __better_errors/xxxxx/variables request has 2MB response body instead of ~100 KB when running puma

@umhan35 is right, when running the app with bundle exec puma or bin/puma the better_errors page is as fast as expected. It's only when running the server with bin/rails s that it's super slow. Sure enough the __better_errors/xxxxx/variables page is HUGE when bin/rails s is running the server... I was seeing a 1.5MB response.
Here's the output of both (I replaced \n with actual line breaks to make it slightly more readable): https://gist.github.com/cannikin/1e62e4a9b4f6f691de358bf84232c4da
It appears the @request dumps in the rails version are muuuuuuuuuch longer. Like thousands of lines longer.

I've got a hack if you want to keep using binding_of_caller! If you just want to use the REPL and don't care about the dumped list of rack, local and instance variables beneath the console output, see our fork here: https://github.com/workingnotworking/better_errors
local_variables, instance_variables and rack_session methods (that dump those variables) all just return an empty hash. This brings the payload back down to a couple of KB.
In the long run I don't think this is actually a problem with binding_of_caller or better_errors. I think something about Rails object structure changed between 4.2 and 5.0 which just causes MUCH more data to be dumped when inspecting those variable groups.

Worth mentioning that the chrome thread was using 50% CPU for my machine with the better_errors page open in the background (!!!)
Would really love to see @cannikin's solution applied as a configuration option to disable these features since this also prevents large ActiveRecord queries from being performed (since #inspect will realise the collection, regardless of if the collection is yet paginated/scoped properly). The stack inspecting REPL is the bread and butter and printing all the instance variables is unfortunately having a huge impact on this.

Just want to point out that the fork of @cannikin seems to break REPL for me (which is the only reason I have binding_of_caller in my Gemfile anyways). So it seems worthless to me.
By the way: the web-console gem (which is included in Rails 5 by default) offers a very fast REPL from within standard error pages (or by placing a <%= console %> statement somewhere in your code. Maybe using this could speed things up?

as a temp solution to reduce response size from __better_errors/xxxxx/variables you can add next code to application_controller.rb:
before_action :better_errors_hack, if: -> { Rails.env.development? }

def better_errors_hack
  env['puma.config'].options.user_options.delete :app
end

in this case response size using rails s will be comparable with puma command

@vavgustov that wasn't working for me, this does:

@inopinatus you probably use rails 5.1.
I'm using 5.0.2 and env worked ok but some time ago I noticed this message in server logs:

DEPRECATION  WARNING: env is deprecated and will be removed from Rails 5.1.

So yes better to use request.env instead of env.

confirmed, this came up during 5.1rc1 upgrade preparation.

Hmm, I'm on 5.0.2 and @vavgustov / @inopinatus hack doesn't seem to have any effect for me. better_errors takes forever to render again and the Rack session output is MASSIVE. Full dump here: https://gist.github.com/cannikin/d71e442b6ca46fba130448207ce8f2c9

I was making a PR that might fix this issue in https://github.com/charliesome/better_errors/pull/360, can anyone confirm?
The root cause of the problem is just because puma has a "big variable" and better_errors tries to serialize everything. The PR allows users to configure a maximum size, above which they do not attempt to serve the heftier payloads to the browser.
I haven't tested this on 5.x yet at all, but progress on getting the PR reviewed/merged seems to have stalled a little (no hate, we're all busy people 😎 )
Busy this weekend but I can take a look on Monday if there is any feedback on the PR, keen to help get this in people's hands as it's such a relatively small fix for huge Quality of Life gains!

Any progress on this ? It feels kind of silly that Puma and Better Errors being both stock packages on Rails do not work together (unless complete request freeze for 20 seconds is considered working).
Is there anything we can help with in order to speed up the process ?

This is NOT better_errors' fault! That being said, here's what I'm doing to help this:
In my application controller:
before_action :better_errors_hack, if: -> { Rails.env.development? }

def better_errors_hack
  request.env['puma.config'].options.user_options.delete :app
end

Although this may not be a better_errors fault, it is observed only when using better errors, and renders the better_errors gem a little less useful (or at least less... better...). Why not enable this hack (or similar) by default in the better_errors gem, so that all its users can return to using it happily, and being eternally grateful?
Judging by the length of this thread, it is obvious that it bothers a few people, and many think (correctly or not) that it is a better_errors problem.

@pachonk I would certainly class it as a behaviour that better_errors needs to fix  - handling of large objects leading to an unusable error page is a fairly significant limitation right now whether it's the puma object or just any large instance variable being resolved and #inspected.
If anyone can find problems with the PR (#360) I will happily make the changes, but I've used it in development for months (from my fork) without problem. As far as I am aware the PR is classed as "failing" due to a problem with the Continuous Integration for the project rather than any failing/missing tests, but I'm happy to be corrected and put the work in if we can get this merged and released :)

I've just revised my workaround to be
  def better_errors_hack
    request.env['puma.config'].options.user_options.delete(:app) if request.env.has_key?('puma.config')
  end

to support things like app.get '/' at the console.

Seems this issue will be fixed in version 2.5. Meanwhile I added fix from above comments
(comment-1, comment-2, comment-3) to experimental debug_extras gem. Originally
this gem has been developed for other purposes which you can see in 'Features' and 'Screenshots' section. However I found that it make sense to implement this 'hack' as middleware and added it to gem to avoid junk in application_controller.rb.

This should be fixed by #402, which was included in v2.4.0.

@RobinDaugherty thanks, it seems fixed to me!

Can confirm! I updated better_errors from 2.1.1 to 2.4 and the console loads way faster now (2.1.1: 30-60 seconds / 2.4: 1 second).
This is life changing! Thank you sooo much!


Compatibility

Not exactly an issue:
This plugin is not working with Webstorm 7. I didn't see is that's mentioned anywhere on the Jetbrains plugin page or the git Readme.
Works great with Webstorm 8

Not sure if its the same thing, but this also doesn't work with Intellij Idea 1.1.5


Cannot (easily) expand last object in debug tip

Mac, Insiders
when hovering over a value in break mode, it is very hard to click on the expando triangle to expand the last item listed because it is mostly covered by the horizontal scrollbar.

Dup of https://github.com/Microsoft/vscode/issues/59452
Assigning that to november to try to fix


nbgitpuller 404 error

Using nbgitpuller links result in a 404 error. I've added jupyter serverextension enable --py nbgitpuller --sys-prefix to the postBuild file as recommended, but this didn't resolve the issue. The links seem to work with sp22's image. The difference between that image and the current is versions of packages in requirements.txt.

Changing from Python 3.10.* to 3.9.* did not resolve the issue.

Updating JupyterLab from 2.3.0 to 2.3.1 did not resolve this.

Changing the default environment from Classic to JupterLab seems to have resolved the issue.


In build script activate env before installing.

In making the changes for VANILLA_INSTALL this logic bug crept in:
Essentially, the optional packages are erroneously being installed
in the root env and not in the newly created env as it wasn't
activated at install time. This fixes it!

Codecov Report

Merging #3167 into master will increase coverage by 2.43%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3167      +/-   ##
==========================================
+ Coverage    78.7%   81.13%   +2.43%
==========================================
  Files         384      384
  Lines       75091    75091
  Branches     8436     8436
==========================================
+ Hits        59098    60923    +1825
+ Misses      14740    12879    -1861
- Partials     1253     1289      +36

The VANILLA_INSTALL build has these packages: https://travis-ci.org/numba/numba/jobs/408398170#L3322-L3355
Picking one of the non-VANILLA_INSTALL builds, these are the packages: https://travis-ci.org/numba/numba/jobs/408398164#L3392-L3445
The VANILLA_INSTALL has no scipy, cffi, ipython, pygments etc, the non-VANILLA_INSTALL does, as intended!


useInfiniteQuery Modify Custom Props Not Work Correctly

Describe the bug

Not sure it's a bug or I missing knowledge but when using useInfiniteQuery at second parameters

here is fetchMovies when I assign new properties from custom prop like this

the fetch won't fetch correctly, it keep fetching the initialize result. to fix this have to clone the object

Note: the querySearch is the prop of component, I pass this prop into useInfiniteQuery

sorry, I have no idea what you are trying to achieve. converting this to a discussion so that we can discuss it.
pageParam is "calculated" in the getNextPageParam function - have you implemented that? The result of that function is passed as pageParam to the queryFn ....


Fixes Airflow syncing bug with API response limit

Describe your changes and why you are making these changes
This PR fixes a bug where if there were more runs than the default Airflow API limit, they would never be synced with Aqueduct. This PR ensures that all DAG runs get synced from Airflow into Aqueduct.
It doesn't do anything to improve the performance of syncing since that would be more involved and can be implemented for a later release.
Related issue number (if any)
ENG 1531
Checklist before requesting a review

[ ] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[ ] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[ ] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

@likawind Yup, I tested it against some actual Airflow DAGs.


Pipeline

**Hello Everyone,
Could you please explain the following section of the code in relation to the pipeline.  My question is why did you use the conv2d followed by conv1d. What are the benefits and why you consider this.**
def conv2d(in_channel, out_channel, kernel_size):
layers = [
nn.Conv2d(
in_channel, out_channel, kernel_size, padding=kernel_size // 2, bias=False
),
nn.BatchNorm2d(out_channel),
nn.ReLU(),
]
return nn.Sequential(*layers)

def conv1d(in_channel, out_channel):
layers = [
nn.Conv1d(in_channel, out_channel, 1, bias=False),
nn.BatchNorm1d(out_channel),
nn.ReLU(),
]
return nn.Sequential(*layers)

class OCR(nn.Module):
def init(self, n_class, backbone, feat_channels=[768, 1024]):
super().init()
I didn't see in the model how you refer resnet 101 or HRNet.
self.backbone = backbone
What does this feat channels and ch16, ch32 means.
ch16, ch32 = feat_channels
what does this mean?
self.L = nn.Conv2d(ch16, n_class, 1)
self.X = conv2d(ch32, 512, 3)
I found in the article these phi, psi are transformation functions and were used in self-attention. Could you please explain the benefit of using these function?
    self.phi = conv1d(512, 256)
    self.psi = conv1d(512, 256)
    self.delta = conv1d(512, 256)
    self.rho = conv1d(256, 512)
    self.g = conv2d(512 + 512, 512, 1)

    self.out = nn.Conv2d(512, n_class, 1)

    self.criterion = nn.CrossEntropyLoss(ignore_index=0)

def forward(self, input, target=None):
    input_size = input.shape[2:]
    stg16, stg32 = self.backbone(input)[-2:]

    X = self.X(stg32)

Thanks in advance

I didn't see in the model how you refer resnet 101 or HRNet.
self.backbone = backbone

I made it that you can pass specific backbones into the model. See: https://github.com/rosinality/ocr-pytorch/blob/master/train.py#L179
What does this feat channels and ch16, ch32 means.
ch16, ch32 = feat_channels

It specifies number of channels of stride 16, stride 32 feature maps.
what does this mean?
self.L = nn.Conv2d(ch16, n_class, 1)
self.X = conv2d(ch32, 512, 3)

X corresponds to pixel representations, L corresponds to soft object regions. You can find this in the paper version 1.
phi & psi is for computing logits similar to commonly used for self attentions. One difference is activation is used for that. Maybe authors found that adding more nonlinearities is better for this.

Hello Rosinality,
Thank you for your prompt response.
Could you please answer these 2 questions.

why did you use the conv2d followed by conv1d. What are the benefits of using conv2d and conv1d and why you consider both.

I have kind of futuristic question also. If I want to change the model of OCR and include attention in the model. Is that possible? Also what modification would you suggest to implement OCR.

Thank you

I have used Conv1d as feature maps will be flattened to compute self attentions. Actually you can implement it solely using Conv2d.
Do you want to use OCR attentions to your models? One core idea of OCR is create soft attention region maps (corresponds to each semantic class) and using it for attention. I think you need to check that it is appropriate to your tasks and models.

In my case, there will be a grayscale image showing temperature values of an object. Where I have to use semantic segmentation to delineate between defective and non-defective area in one image. Could you please suggest some ideas that you have. Thank you once again for your assistance.

You can try to use defective and non-defective as 2 classes and use OCR on it. I don't know OCR is very appropriate for your tasks, but it may be worth try if you have some baseline segmentation models.

Yeah sure. Thanks I will try. Also, based on your broad knowledge about several models. What other models would you suggest that would be appropriate to try. Or any modification in OCR models that would be poissibly helpful.  Thanks for your time.

I think UNet or FPN and DeepLab v3+ will be simple but powerful baselines. If I use OCR then I will try to use more powerful backbones or use decoder like approaches (that is, for example, concatenate stride 4 features) such as DeepLab v3+

Hello Rosinality,
In table 4 in the article, you mentioned comparison with other methods.
Question 1) Did you actually implement all those methods yourselves. Or you just compare with them.
I am interested in CC-Attention, Self-Attention and Double Attention.
Question 2) If you implement these three methods your selves. Could you please share your code.
Thank you
Kind regards
Rahmat

Sorry, I'm not the author of the paper.

Thanks
On Tue, Feb 18, 2020 at 6:13 PM Kim Seonghyeon<EMAIL_ADDRESS>wrote:

Sorry, I'm not the author of the paper.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rosinality/ocr-pytorch/issues/3?email_source=notifications&email_token=AMWOBYWQZUHQ42OZUSFGE6LRDR2Q3A5CNFSM4KV7HRS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMF3CMQ#issuecomment-587968818,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMWOBYX3HPIPXL4AGAEJZ6TRDR2Q3ANCNFSM4KV7HRSQ
.

Thanks Rosinality

Hi rosianlity,
Could you please share your email.

<EMAIL_ADDRESS>

Hi Rosinality,
Could you please explain how to apply Global Weight average pooling in pytorch. For your refernce the article which discussed is mentioned as below. Thank you
@inproceedings{kolesnikov2016seed,
title={Seed, Expand and Constrain: Three Principles for Weakly-Supervised Image Segmentation},
author={Kolesnikov, Alexander and Lampert, Christoph H.},
booktitle={European Conference on Computer Vision ({ECCV})},
year={2016},
organization={Springer}

pooled = (weight * input).sum([2, 3], keepdim=True)


Added the Pod Blog and Post Template | Issue no 7: [Add pod blog #7]

Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Okay, Added the pod Blog, It add the blog on a separate page on /blog and you can add the posts inside the _post folder in markdown. it adds only a few lines of CSS in the All.scss file. it may have some problems. I think my css formatter for VS code reformatted the whole File so the tabs and spaces is a bit messed up. it should be a problem. Let me know if any issue arises. Thanks!

Cool! can you talk to the navbar team and ask them if they could create a button for the /blog if they haven't?


Feat/check available bumps pelorus operator

Describe the behavior changes introduced in this PR
Create scheduled work to check if pelorus operator can be bumped.
Linked Issues?
No linked issues.
Testing Instructions
To see the new command help, run make help
To run new command, run make check-bump-pelorus-operator

discuss after https://github.com/konveyor/pelorus/issues/841 is resolved if this PR still makes sense

Alright, I think this PR is not necessary
I believe what made me want to create it was that each time you run the create-operator script, you can get a different operator sdk version from here. So, we only need to check if there is a newer version of it, instead of creating all this logic.
But I think that our logic is wrong in our scripts today. For example, the latest releases of operator sdk today are 1.26.1 and 1.27.0, and our scripts install 1.26.1 because its newer. We should check which is bigger, right?
My idea is:

create a CI job to check if the operator sdk is the newest (and bigger) possible when there are changes in pelorus-operator folder; if not, break and warn the about the newer version available.
update our installation scripts to install the newest (and bigger) possible versions of each software.

Do you agree? @mpryc @KevinMGranger @weshayutin
If so, I will close this PR and open issues to address each point.

@mpryc  but we are not forced to just release bigger versions, are we?
For example, current Python is 3.11, but this does not mean they will not release patches to 3.10, right? I believe this is not a mistake from the software versioning.

@mpryc but we are not forced to just release bigger versions, are we?
For example, current Python is 3.11, but this does not mean they will not release patches to 3.10, right? I believe this is not a mistake from the software versioning.

That is correct, however latest tag should in this case point to the 3.11 and not the 3.10-update which may have happened after 3.11 release.

I am closing this PR.
In my opinion it is not required. There was an issue with operator-sdk release where the following order was made:
$ curl --silent "https://api.github.com/repos/operator-framework/operator-sdk/releases" | jq -r '.[].tag_name'
v1.28.0
v1.26.1
v1.27.0
v1.25.4
v1.26.0
v1.25.3
v1.25.2
v1.25.1
v1.25.0
v1.24.1
v1.24.0
v1.23.0
v1.22.2
v1.22.1
v1.22.0
v1.21.0
v1.20.1
v1.20.0
v1.19.1
v1.19.0
v1.18.1
v1.18.0
v1.17.0
v1.16.0
v1.15.0
v1.14.0
v1.13.1
v1.13.0
v1.12.0
v1.11.0

As you can see  there were two times where the latest tag was not the latest number:
v1.26.1       <- After  v1.27.0
v1.25.4       <- After  v1.26.0

But it's not our responsibility to decide what is the latest tag for a particular repository, maybe there was some decision behind making.
Logged but in the operator-sdk project: https://github.com/operator-framework/operator-sdk/issues/6403
Closing this one, we should monitor the answers of the above issue and see if this one will need to be re-opened.
With the latest tag v1.28.0 we are safe, but we need to get answers to not hit this again in the future.


Update API naming and code conventions.

This PR implements the OpenThread API naming and code conventions proposed in #1100.

API Naming Convention

Commands

ot[Component][Command] (e.g. otLinkEnergyScan(...))

Properties

ot[Component]Get[Property] (e.g. otLinkGetChannel(...))
ot[Component]Set[Property] (e.g. otLinkSetChannel(...))

File Naming Convention

Client API header files

include/openthread/[component].h (e.g. include/openthread/link.h)

Platform API header files

include/openthread/platform/[component].h (e.g. include/openthread/platform/radio.h)

Client API implementation files

src/core/api/[component]_api.cpp: (e.g. src/core/api/link_api.cpp)

Current coverage is 70.87% (diff: 46.42%)

Merging #1173 into master will increase coverage by 0.35%

@@             master      #1173   diff @@
==========================================
  Files           127        138    +11
  Lines         18800      18321   -479
  Methods        2635       2559    -76
  Messages          0          0
  Branches       2273       2214    -59
==========================================
- Hits          13258      12985   -273
+ Misses         4678       4504   -174
+ Partials        864        832    -32

Powered by Codecov. Last update ab13e71...8e30fe0

@jwhui Do you see any chance to merge this PR in this week?

@LuDuda, that's my goal.  I'm working on resolving the merge conflicts.

Great news! Thank you :LGTM:

All merge conflicts have been resolved.  I'll be merging this within the next couple days.  If anyone has any last-minute review comments, please get them in now :)

I wouldn't block the PR for this, but I'd like to keep these up to date with the header files. The goal here is to packet the headers and libs necessary to build an application that uses the OpenThread APIs on Windows.

@nibanks, I updated .appveyor/make_release.cmd.  Please review when you get a chance.

@jwhui, LGTM. Thanks!