Is there anything custom about the vertical scrollbars in the code browser window of Ghidra?

There's something seriously broken with the vertical scrollbars found in Ghidra code browser tool window, but it only manifests if Ghidra is used with a screen reader or other assistive technology that queries the Java accessibility interface.
These scrollbars report that they have no parent object (AccessibleContext.indexInParent == -1), while also reporting themselves as their own parent (through AccessibleContext.getAccessibleParent). That should never happen.
When something makes these scrollbars move (e.g. some Pgdn keys are pressed in the listing panel), a value property change event is fired on them which the screen reader intercepts. It then begins to query the object for its own purposes. Among the first "questions" it asks from the Java accessibility interface is what is the top level object of the given scrollbar (should be code browser main frame). It needs this info to determine the native window handle the scrollbar belongs to. The Java-side follows the parent ancestry chain to fulfill this request, however it never terminates due to the incorrect reporting described above. A Java-side endless loop is formed wich freezes both Ghidra and the screen reader. The only way out from that is to restart the screen reader and then kill and restart the frozen Ghidra instance.
Oddly enough horizontal scrollbars in code browser window do not manifest this issue at all. They report everything correctly, and they report the scroll pane which they are in as parent (correct). I've not encountered anything like this in the main window of Ghidra, nor in any other Java app.
I've documented this issue in detail here:
https://github.com/nvaccess/nvda/issues/12045
and I also demonstrate a workaround code for NVDA screen reader that works great. However it would be best to get this fixed on Java-side. So I'd like to narrow down whether this is a Ghidra issue or a Java Swing bug (as these scrollbars are a subclass of javax.swing.JScrollBar) in order to determine where to report this issue. I know that Ghidra doesn't implement anything custom for these vertical scrollbars accessibility-wise, but I am curious what other custom info Ghidra feeds about them to Java Swing which makes that construct such mucked-up parent info about them.

I am almost sure that with very naive search approaches I have hit upon the problematic class. I am quite convinced that the vertical scrollbars that provoke havoc with screen readers are the instances of docking.widgets.SideKickVerticalScrollbar. This class has an (at least to me) quite confusing delegation pattern and hard-to-follow parent relationships. Furthermore it has the following code fragment:
	@Override
	public AccessibleContext getAccessibleContext() {
		return delegate.getAccessibleContext();
	}

The whole Ghidra code tree has only three mentions of AccessibleContext, and this fragment is eventually found in a class that implements a type of vertical scrollbar (the two other mentions are in totally unrelated places). I don't see the point of this code fragment as this scrollbar is a subclass of jScrollbar, so handling of accessibility is already transparent as with subclasses of other regular Swing controls (like combo boxes or push buttons) for which Ghidra doesn't do anything similar (and they do work all right). What's more this fragment is indeed a possible source of an endless recursion.
So I suggest focusing investigation efforts to this class, maybe together with the package docking.widgets.indexedscrollpane, the only place where this sidekick vertical scrollbar thing is incorporated.

Thanks for the tip.  I started by looking there and at our GTree implementation, which is a panel that contains an AutoScrollTree.  I'm assuming that there is something wrong with the auto scroll tree too.    Is the problem for vertical scroll bars for many of Ghidra's trees?

I don't know how many of the tree controls are affected. What I do know from the diagnosis period is that the culprit vertical scroll bar is both present near the listing panel and also the symbol tree and is a subclass of jScrollbar with some custom features and it also resides in a scroll pane. After having looked at its source code I see that this new "surprise finding" of this sidekick vertical scroll bar class fulfills all these criteria.

My instincts and reasoning has proven to be right when I've put it to the test. I hand-picked SideKickVerticalScrollbar.java out of docking-src.zip found in the Ghidra release. I removed just the method cited above as I suspected that it was the only cause of all these freezes. I recompiled and inserted the resulting three class files back into docking.jar found in the release overwriting the original three class files there. Hackish, isn't it?
Then I disengaged my workaround code from the screen reader, so it should have frozen up like before if I hadn't been on the right track. But it didn't. No matter how hard I'd been trying to reproduce the crashes, it was in vain, all of them were banished by this small fix and the code browser worked normally. I did all my diagnostics again (among them Jaccessinspector described above) and the problematic vertical scrollbars turned into absolutely normal scrollbars and spat out correct accessibility information. No more reporting of self as parent, parent and top level object is valid and correct, all is well.
So that was the whole story: one little method (most probably some left-over code) with
faulty delegation pattern that lead the accessibility interface astray. And this method is totally pointless as javax.swing.JScrollbar takes care of it on its own. So it can be safely removed from Ghidra and this tough issue is gone. I experience no "side-effects" from the removal and there shouldn't be any as the code is pointless.

As conclusion and for documentation I paste here the output of Jaccessinspector upon a value change on the sidekick vertical scrollbar attached to the listing panel.
First the output when querying the scroll bar produces endless loop and freeze (see comment):
Value changed event: old = 0; new = 850226

Version Information:
    Java virtual machine version: 11.0.7
    Access Bridge Java class version: 11.0.7
    Access Bridge Java DLL version: 11.0.7
    Access Bridge Windows DLL version: 11.0.7

AccessibleContext information:
    Name:
    Virtual Name:
    Description:
    Role:  scroll bar
    Role in en_US locale:  scroll bar
    States:  enabled,focusable,visible,showing,opaque,vertical
    States in en_US locale:  enabled,focusable,visible,showing,opaque,vertical
    Index in parent:  -1
    Children count:  2
    Bounding rectangle:  [530, 159, 547, 729]
ERROR: GetAccessibleContextInfo failed for top-level window // freeze takes place here, output is produced after killing Ghidra, further lines are meaningless
    No parent
ERROR: GetVisibleChildrenCount failed
GetAccessibleRelationSet failed.

Accessible Value information:

And now the correct info about the same scrollbar after removing the little faulty method by hand:
Value changed event: old = 447844; new = 446165

Version Information:

    Java virtual machine version: 11.0.7
    Access Bridge Java class version: 11.0.7
    Access Bridge Java DLL version: 11.0.7
    Access Bridge Windows DLL version: 11.0.7

AccessibleContext information:

    Name:
    Virtual Name:
    Description:
    Role:  scroll bar
    Role in en_US locale:  scroll bar
    States:  enabled,focusable,visible,showing,opaque,vertical
    States in en_US locale:  enabled,focusable,visible,showing,opaque,vertical
    Index in parent:  0
    Children count:  2
    Bounding rectangle:  [530, 159, 547, 729]
    Top-level window name:  CodeBrowser: procwin:/Procwin.EXE
    Top-level window role:  frame
    Parent name:
    Parent virtual name:
    Parent role:  scroll bar
    Visible descendents count:  2
    Descendent 0 name:
    Descendent 0 role:  push button
    Descendent 1 name:
    Descendent 1 role:  push button

Accessible Value information:
    Current Value:  446165
    Maximum Value:  1398810
    Minimum Value:  0

Nice work.

I note that not only the vertical scrollbar linked to the listing panel is affected, but e.g. the vertical scrollbar connected to the symbol tree exhibits the exact same behaviour

The symbol tree does not use the `SideKickVerticalScrollbar', AFAIK.    Were you able to verify if the tree is still broken in your tests?

Of course, the symbol tree is okay too and this little fix stops all freeze scenarios we know of (it was not just me who did the reproduction tests at the beginning of diagnosis).
Yes, as I opened my Jaccessinspector logs again, I see that the vertical scrollbar near the symbol tree that used to trigger freezes doesn't report exactly in the same manner as sidekick vertical scrollbars do. Jaccessinspector data has some strangeness for that, the most particular is that it reports [-1; -1; -2; -2] as bounding rectangle coordinates. But the screen reader doesn't panic on that, and figures out the correct values (it has more logic in it than Jaccessinspector and is usually extremely fault-tolerant). The important point is that it doesn't exhibit any self-parenting behaviour any more, my fix cured that also somehow. The screen reader only detects that vertical scrollbar if I expand e.g. the functions category in symbol tree, if it is closed, then only a horizontal scrollbar is presented.


Unpaginated calls to DescribeNetworkInterfaces getting blocked by EC2

Describe the Bug:
We've been in the process of rolling out security groups for pods across various EKS clusters in multiple AWS accounts that we own. Recently, we've attempted to do the same in one of our "larger" AWS accounts, and noticed that pods would hang indefinitely waiting for an ENI to be attached by vpc-resource-controller.
We recently raised the ENI limits on this account to a very large value, and was told that unpaginated calls to DescribeNetworkInterfaces would be blocked as a result, and it seems that this is impacting vpc-resource-controller as you can see here in this CloudTrail event:
{
    "eventVersion": "1.08",
    "userIdentity": {
        "type": "AssumedRole",
        "principalId": "<REDACTED>:amazon-vpc-resource-controller-k8s",
        "arn": "arn:aws:sts::<REDACTED>:assumed-role/<REDACTED>/amazon-vpc-resource-controller-k8s",
        "accountId": "<REDACTED>",
        "sessionContext": {
            "sessionIssuer": {
                "type": "Role",
                "principalId": "<REDACTED>",
                "arn": "arn:aws:iam::<REDACTED>:role/<REDACTED>-control-20210716225641340500000009",
                "accountId": "<REDACTED>",
                "userName": "<REDACTED>-control-20210716225641340500000009"
            },
            "webIdFederationData": {},
            "attributes": {
                "creationDate": "2023-03-14T21:23:53Z",
                "mfaAuthenticated": "false"
            }
        },
        "invokedBy": "eks.amazonaws.com"
    },
    "eventTime": "2023-03-14T21:35:11Z",
    "eventSource": "ec2.amazonaws.com",
    "eventName": "DescribeNetworkInterfaces",
    "awsRegion": "us-east-1",
    "sourceIPAddress": "eks.amazonaws.com",
    "userAgent": "eks.amazonaws.com",
    "errorCode": "Client.OperationNotPermitted",
    "errorMessage": "This operation is not permitted.",
    "requestParameters": {
        "networkInterfaceIdSet": {},
        "filterSet": {
            "items": [
                {
                    "name": "tag:vpcresources.k8s.aws/trunk-eni-id",
                    "valueSet": {
                        "items": [
                            {
                                "value": "eni-019dd4ee9f4aa06c8"
                            }
                        ]
                    }
                }
            ]
        }
    },
    "responseElements": null,
    "requestID": "8fd9723c-c4fa-4dd2-82be-47d8f328bf8e",
    "eventID": "fd8c44a0-b04c-430d-9e77-8c478b0f2de3",
    "readOnly": true,
    "eventType": "AwsApiCall",
    "managementEvent": true,
    "recipientAccountId": "<REDACTED>",
    "eventCategory": "Management"
}

For AWS engineers, our case # is<PHONE_NUMBER>1
Observed Behavior:
Calls to DescribeNetworkInterfaces are blocked by EC2 with a Client.OperationNotPermitted error code
Expected Behavior:
SGP should work even if unpaginated calls are blocked.
How to reproduce it (as minimally and precisely as possible):
Increase ENI limits in an AWS account high enough that EC2 blocks unpaginated calls to DescribeNetworkInterfaces
Additional Context:
Environment:

Kubernetes version (use kubectl version): Server Version: version.Info{Major:"1", Minor:"22+", GitVersion:"v1.22.16-eks-ffeb93d", GitCommit:"52e500d139bdef42fbc4540c357f0565c7867a81", GitTreeState:"clean", BuildDate:"2022-11-29T18:41:42Z", GoVersion:"go1.16.15", Compiler:"gc", Platform:"linux/amd64"}
CNI Version 1.11.4
OS (Linux/Windows): Amazon Linux (EKS AMI)

@ejholmes thanks for reporting the issue. As you mentioned, calls to EC2 shouldn't be blocked by EC2 just because they are unpaginated. I have found the request ID in our system. We need to check with EC2 team for the root cause. Thanks.

Just to update this issue for anybody else that manages to run into this, the issue was from being blocked by EC2 on ec2:DescribeNetworkInterfaces. We discovered an issue where Glue was leaking ENI's and leaving them orphaned, and managed to clean those up with some effort, and lowered our limit on ENI's (we were told we needed to be under 20k to be unblocked).
Once that was done, and EC2 unblocked us from ec2:DescribeNetworkInterfaces, security groups for pods was functional in the account again.


ns-export gaussian-splat  bug

ns-export gaussian-splat
I cannot create mesh or get ply because of that
Initialising Gaussian normals from SfM estimates
Loading latest checkpoint from load_dir
──────────────────────────────────────────────────────── Error ─────────────────────────────────────────────────────────
           No checkpoint directory found at outputs/unnamed/dn-splatter/2024-06-18_091416/nerfstudio_models,
             Please make sure the checkpoint exists, they should be generated periodically during training

hi @efyphil, it seems that you are trying out the dn-splatter model. Please note that dn-splatter is not a part of nerfstudio, so if you have issues with dn-splatter you should refer these issues into my repo here

All work, seem like some problem with save process due to my machine

@efyphil I am seeing the same thing; not sure why it's looking for the checkpoint file, it doesn't exist in the training output.  Did you end up having to re-run ns-train?  Else, how did you get past this? Thanks in advance.


[Feature] [Sort] Add JDBC connector on Flink 1.15

Description
Add JDBC connector on Flink 1.15
some extra notes: Oceanus has adopted Flink v1.16 to all its connectors, if the branches were to be in sync, I might want to upgrade all connectors to be compatible with flink 1.16
Will do for 1.15 for this one for now, hopefully the difference between 1.15 and 1.16 isnt too large
(but flink 1.16 made huge changes to jdbc...)
Use case
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

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

I wish to finish it~


Client rendering crashes when calling react_component twice in a rails view/partial

Calling react_component twice in a rails view/partial causes the client render to crash with the following error:
Could not find component registered with name XXX. Registered component names include [ YYY ]. Maybe you forgot to register the component?

VM4859 clientStartup.js:132 Uncaught Error: ReactOnRails encountered an error while rendering component: XXX. See above error message.
    at Object.get (ComponentRegistry.js:40:15)
    at Object.getComponent (ReactOnRails.js:211:44)
    at render (VM4859 clientStartup.js:103:53)
    at forEachReactOnRailsComponentRender (VM4859 clientStartup.js:138:9)
    at reactOnRailsPageLoaded (VM4859 clientStartup.js:164:5)
    at renderInit (VM4859 clientStartup.js:205:9)
    at onPageReady (VM4859 clientStartup.js:234:9)
    at HTMLDocument.onReadyStateChange (VM4859 clientStartup.js:238:13)

= react_component("XXX", props: @props)
= yield
= react_component("YYY", props: @props)

Environment

Ruby version: 3.1
Rails version: 7.0.6
Shakapacker version: 6.6.0
React on Rails version: 13.3.5

Expected behavior
Should render both components.
Actual behavior
Crashes on the client with the aforementioned error/stacktrace.

Seem related to either a configuration issue or maybe using the dynamic bundle loading and the bundles are loaded twice. Check out the source of the pages.

I had this issue once and the solution was to change the defer strategy to false
<%= javascript_pack_tag 'app', defer: false%>
With the defer equals to false the script is executed as soon as it's encountered in the document, which might block the HTML parsing until the script is fully loaded and executed

Thanks for the comments @justin808 and @devjoaov.
After digging into this issue with the help of @justin808 and @Judahmeek, we found out that the root cause was a tweak in the webpack's optimization configuration. Setting optimization.runtimeChunk to true or multiple causes webpack to load ReactOnRails one time for each runtime chunk which leads to ReactOnRails global variable being overridden and some registered components going missing - see https://webpack.js.org/configuration/optimization/#optimizationruntimechunk.
So, to overcome this issue, we could use shakapacker's default optimization configuration (pseudo-code):
const { webpackConfig: baseClientWebpackConfig } = require('shakapacker');

// ...

config.optimization = baseClientWebpackConfig.optimization;

As it set the optimization.runtimeChunk to single. See its source:
package/environments/base.js:115
  optimization: {
    splitChunks: { chunks: 'all' },

    runtimeChunk: 'single'
  },

https://github.com/shakacode/shakapacker/blob/cdf32835d3e0949952b8b4b53063807f714f9b24/package/environments/base.js#L115-L119
Or set optimization.runtimeChunk to single directly.


Ben CPT

[ ] Blog about outline
[ ] Do research on how to make data update automatically
[ ] Input stats and players into database
[ ] Make sure code works


Rebase friendship kernel implementation onto current master.

I have rebased the implementation of friendship kernel in Julia onto the current master branch. In addition, I've modified age back to be Int8 and now use 32-bit precision floats whenever a large number of them needs to be stored in memory.

Merged with https://github.com/MOCOS-COVID19/modelling-ncov2019/pull/39


[BUG] Odd Rendering Issue with Health Bar [Fabric 0.15.7/1.20.1]

Minecraft Version
1.20.x
Mod Loader Version
0.15.7
Mod Version
nameplate-1.1.4.jar
Describe the Issue
Weird Rendering Issues with the health bar.
https://github.com/Globox1997/Nameplate/assets/33227568/583610d4-def1-497b-9a13-7dbe23e2f4ec
latest.log
latest.log
Do you use any other mods except the required ones?
Happens with or without.

Lol never seen that before


bindeps feature breaks Lanzaboote build

I'm not familiar with crane, so apologies if this report is vague.
I have a system that uses Lanzaboote, which depends on crane. After a nix flake update, the Lanzaboote tool fails to build. I narrowed down the trigger to this crane PR: https://github.com/ipetkov/crane/pull/258
Build log:
error: builder for '/nix/store/lnnlznwwm3z9yyjjhh0pxxx3bp1kg3y7-lanzaboote_stub-deps-0.1.0.drv' failed with exit code 101;
       last 10 log lines:
       >    Compiling sha2 v0.10.6
       >    Compiling uefi-services v0.16.0
       >    Compiling lanzaboote_stub v0.1.0 (/build/source)
       > error: linking with `rust-lld` failed: exit status: 1
       >   |
       >   = note: "rust-lld" "-flavor" "link" "/NOLOGO" "/entry:efi_main" "/subsystem:efi_application" "/build/rustcD0qWoA/symbols.o" "/build/source/target/x86_64-unknown-uefi/release/deps/crane_dummy-40d86f973d4ced01.crane_dummy.2d9b5714-cgu.0.rcgu.o" "/LIBPATH:/build/source/target/x86_64-unknown-uefi/release/deps" "/LIBPATH:/build/source/target/release/deps" "/LIBPATH:/nix/store/f7nb3i8za08prjnzrqzrzwa5x3nafrzh-rust-default-1.67.0-nightly-2022-11-21/lib/rustlib/x86_64-unknown-uefi/lib" "/nix/store/509wjj0vhpakxk87b0syv5jxiczpa62b-rust-std-1.67.0-nightly-2022-11-21-x86_64-unknown-uefi/lib/rustlib/x86_64-unknown-uefi/lib/libcompiler_builtins-bb944ea2862a44ff.rlib" "/NXCOMPAT" "/LIBPATH:/nix/store/f7nb3i8za08prjnzrqzrzwa5x3nafrzh-rust-default-1.67.0-nightly-2022-11-21/lib/rustlib/x86_64-unknown-uefi/lib" "/OUT:/build/source/target/x86_64-unknown-uefi/release/deps/crane_dummy-40d86f973d4ced01.efi" "/OPT:REF,ICF" "/DEBUG" "/NODEFAULTLIB"
       >   = note: rust-lld: error: <root>: undefined symbol: efi_main
       >
       >
       > error: could not compile `lanzaboote_stub` due to previous error
       For full logs, run 'nix log /nix/store/lnnlznwwm3z9yyjjhh0pxxx3bp1kg3y7-lanzaboote_stub-deps-0.1.0.drv'.
error: 1 dependencies of derivation '/nix/store/hgxlcbc75b0x29py2b99cmrahqr4rp6g-lanzaboote_stub-0.1.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/42kivkzyx201sc4mf92j6zc36ihmjfix-lzbt.drv' failed to build
error: 1 dependencies of derivation '/nix/store/1dqhnlacyiii1vsi1rfzf16m7107lc6s-bootinstall.drv' failed to build

Other details:

x86_64-linux
nixpkgs @ a7cc81913bb3cd1ef05ed0ece048b773e1839e51 (with Bootspec patches applied on top)
lanzaboote @ 3efb505a2a29de97accade8efee7a0505386a86d
rust-overlay @ 802ff3314663ec7114f29a6e8b200dfc892023f8

Thanks for reporting this @Majiir, I apologize for the breakage!
Although #258 surfaced the issue, the real root cause is better noted by #268, specifically, autobin targets (e.g. having a src/main.rs or src/bin/whatever.rs) aren't well captured at this time. Before #258 buildDepsOnly was effectively only building the lib target, but after it buildDepsOnly is trying to build both the lib and main.rs targets. Since the main.rs target requires a efi_main symbol it fails to build since the default dummyrs file does not declare that symbol...

The solution to solving this is to provide your own custom dummyrs which introduces the necessary symbol! Currently it's a little awkward to write these dummy files since the same value is going to be used for build scripts (which are built for the build-host and not the provided target!), but I'm hoping once #268 is addressed the API will be a bit cleaner. But for now:
craneLib.buildPackage {
  # Other args go here

  dummyrs = pkgs.writeText "dummy.rs" ''
    #![allow(unused)]

    #![cfg_attr(
      any(target_os = "none", target_os = "uefi"),
      no_std,
      no_main,
    )]

    #[cfg_attr(any(target_os = "none", target_os = "uefi"), panic_handler)]
    fn panic(_info: &::core::panic::PanicInfo<'_>) -> ! {
        loop {}
    }

    #[cfg_attr(any(target_os = "none", target_os = "uefi"), export_name = "efi_main")]
    fn main() {}
  '';
}

@ipetkov Thanks for the workaround! I've added this to the lanzaboote repo for now.
Also thanks @Majiir for reporting this. :)

Latest lanzaboote and crane are working for me. Thanks, all.


lint: long chain expression checker v1

fixes #11

I'll take a closer at evening.
Thanks for doing this, by the way.
Can't wait to check it on a bigger code base.
Have you tried to run it with lintwalk on $GOOROOT/src?
(By the way, it can help to reveal bugs.)

If you have time, try running this checker with lower n and see if it finds issues in $GOROOT/src.
This could help us to find false positives and/or crashes.


The download link is not working and you need to tell us the version of the package you are using.

I have to admit that your paper is very well written and some of the ideas were very rewarding for me, but the download links to your data and models are not working and the version of gradle you used is no longer compatible with the latest version, you need to state the version you used to make it easier for us to reproduce it better.  I hope you can change these places, thanks!

Visit https://github.com/biringaChi/vRAG and follow the README instructions to download the missing data.

Thank you very much!
x =
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "VulDetProject/ReVeal"                                                                                @.>;
发送时间: 2023年10月7日(星期六) 凌晨2:26
@.>;
抄送: "x<EMAIL_ADDRESS>主题: Re: [VulDetProject/ReVeal] The download link is not working and you need to tell us the version of the package you are using. (Issue #23)
Visit https://github.com/biringaChi/vRAG and follow the README instructions to download the missing data.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Missing types in node export

Environment
node 21
Reproduction
<EMAIL_ADDRESS><EMAIL_ADDRESS>Describe the bug
node.d.ts is missing type exports present in index.d.ts
Additional context
< export { C as CreateFetchOptions, g as Fetch, b as FetchContext, F as FetchError, d as FetchOptions, h as FetchRequest, f as FetchResponse, G as GlobalOptions, I as IFetchError, M as MappedResponseType, R as ResponseMap, e as ResponseType, S as SearchParameters, c as createFetch, a as createFetchError } from './shared/ofetch.8459ad38.mjs';
---
> export { F as FetchError, c as createFetch, a as createFetchError } from './shared/ofetch.8459ad38.mjs';

Logs
No response

Can confirm that these types were previously exported in v1.3.3. Updated to v1.3.4 and now facing "Module 'ofetch' has no exported member FetchOptions" (Same as #395)


Button

Create Button Component

React button component created but need to add CSS and props, it will be done after vue button is created. @amitsiliguri please assign it to me after vue button is created and merge to Develop branch

@LearnAny please proceed with button styling


Extending styles of sticky and top states

I know you provided the className property but I'm curious if there's a way to specifically target the states (i.e. sticky, at the top of page, etc..) as I see that you do swap the classes.
When it's at the top I wanted to drop the opacity of the background so my hero image showed through and invert the colors of my header links.   I can do it fine targeting the randomly generated class names but my understanding is that they're apt to change eventually.  New to styled components so perhaps I'm missing something though.

Thanks for your question!
You're not missing anything; this is currently not supported (officially) by this package.
There are three modes: Pinned, Unpinned, and Static (default)
I am wondering what the best approach would be for you to consume it.
One option would be to add a callback prop à la onModeChanged which you can use to change your styling (or whatever else you want to do). This seems like the most flexible option, but you have to "duplicate" the mode state in your own component.
Another option would be to add a classname as a prop (one for each mode) that ReactStickyHeadroom would set as additional classname to the wrapper div depending on the mode.
Do you have a preference?


API authentification error when push several GBs

Hello! I'm having issues every time I try to push several GBs.
file_1.tar: googleapi: Error 401: Invalid Credentials, authError
The error doesn't interrupt the push action. However, the file in question is not uploaded.
My go version -->  go version go1.2.1 linux/amd64
Right now I started a push of 1.3TB (17299 files) which should end in one week. This could also be useful to test drive under heavy usage.

I reached 22.3% (pushing 1.3TB).
So far I got 12 error 401, which means 12 skipped files from uploading.

Hello again @Issam2204! Sorry for the late reply, I've been quite busy.
This issue is a duplicate of a couple of other issues, please take a look at the discussion here at https://github.com/odeke-em/drive/issues/116

Thank you very much @odeke-em!
And sirry for the duplicate :)

No worries, sorry for the issue btw. If it helps, you could run say three instances of drive on distinct directories -- this is a very duct tape solution until solving the auto OAuth timeout refresh is in. Thanks for the patience.

@odeke-em again thanks! I'll certainly do that :)
Have a nice one!

Cheers!

Hello again @Issam2204 btw this got fixed on master over the weekend.

Still happening with v 2.4.0
googleapi: Error 401: Invalid Credentials, authError
when doing a push for atleast an hour. I downloaded master today.

Hey folks please get the latest from master by
$ go get -u github.com/odeke-em/drive/drive-gen && drive-gen

Uploading should be faster and there is also a mechanism for exponential backoff on encountering retryable errors.


Support exposing services through multiple devices

NodePort services are currently exposed through the native device which has the default route on the host or a user specified device. In tunneling mode, they are additionally exposed through the tunnel interface (cilium_vxlan or cilium_geneve). This issue tracks exposing services through multiple native devices.

We have the same problem (i.e. each bare-metal machine has two upstream connections to two different switching leafs). Routing is done using BGP-to-the-host.


Fixed Azure Vector Store initialization where the embedding field was not set as Retrievable.

In Azure Vector Store, the embedding field need to set as retrievable.
com.azure.search.documents.indexes.models.SearchField class does not have a retrievable field, but the "hidden" field is equivalent to "retrievable". For details, refer to the "toJson" and "fromJson" methods.
fix https://github.com/spring-projects/spring-ai/issues/1628

Closing via https://github.com/spring-projects/spring-ai/commit/382fbaca48448caeaddf78f5fc7bed83c52ed90c.


Mappings in template do not maintain quotes during package step

Hello, using aws cloudformation package with the following simple snippet results in the loss of quotes which causes a ChangeSet to remain in CREATE_IN_PROGRESS indefinitely in the proper scenerio.
Mappings:
  '1234567890':
    kmskeys:
      'dev': 'arn:aws:kms:us-east-1:1234567890:key/1234567890-abcdefg-1234-1234'
      'test': 'arn:aws:kms:us-east-1:1234567890:key/1234567890-abcdefg-1234-12345'

after the aws cloudformation package step:
Mappings:
<PHONE_NUMBER>:
    kmskeys:
      dev: arn:aws:kms:us-east-1:1234567890:key/1234567890-abcdefg-1234-1234
      test: arn:aws:kms:us-east-1:1234567890:key/1234567890-abcdefg-1234-12345

This is a problem because when using the !Ref "AWS::AccountID" function, a string is returned so it will never match the mapping - if you are mapping per AccountID.
Per the docs, if a Mapping is not used, it is an invalid changeset
So, I'm curious if this is a bug. I feel like I should be able to create a Mapping with Account ID's and then reference the AccountID later in a later block and it is currently not possible with aws cloudformation package. Thoughts?
$ aws --version
aws-cli/1.11.47 Python/2.7.10 Darwin/16.0.0 botocore/1.5.10

Ah interesting. My first guess is that YAML parser/dumper does type conversion which removes types. Need to dig into this further

aws/aws-cli#2556 gives you a workaround. You can use --use-json to dump in JSON instead of YAML. We still have aws/aws-cli#2511 open which will end up quoting every single string in output just to be safe.
Does this solve your problem?

Sure. I've already made a workaround. If you feel it is sufficient to close this, go ahead. Thanks for the response!

Sounds good.

@jolexa  What was your workaround?

Using sed in a pre-deployment step. Ugly but quicker than waiting for awscli bugfixes :/


feat: add the main diff function

This is the PR that bootstraps all other functions. This will the main function which will be responsible for generating the diff.

@derberg @magicmatatjahu Need your reviews here :)

@aayushmau5 for duplications, please rename sonar config file from sonar-project.properties to .sonarcloud.properties and make sure that file that has duplications is listed there but only under sonar.cpd.exclusions

const breaking = 'breaking';
const nonBreaking = 'non-breaking';
const unclassified = 'unclassified';

are these declared in one or multiple places? we need to make sure it is just done once

ah, cool. Then I will make a contants.ts file and add them there.

Wow, that sonar cloud config worked. :rocket:


Limit violations data to 2015

Given this:
https://opendata.miamidade.gov/resource/tzia-umkx.json
Limit the response to tickets opened in 2015.

Hey @phiden - so this is the query for those same tickets opened after 1/1/2015:
https://opendata.miamidade.gov/resource/tzia-umkx.json?$where=ticket_created_date_time > '2015-01-01'
Is that what you're looking for?

I think so!
On Fri, Aug 14, 2015 at 1:40 PM, Ernie Hsiung<EMAIL_ADDRESS>wrote:

Hey @phiden - so this is the query for those same tickets opened after 1/1/2015:
https://opendata.miamidade.gov/resource/tzia-umkx.json?$where=ticket_created_date_time > '2015-01-01'
Is that what you're looking for?
Reply to this email directly or view it on GitHub:
https://github.com/codeforamerica/mdc-feedback/issues/173#issuecomment-131234131

Awesome. Going to close this ticket then :)


The explanation about autoscale is unclear.

"Occasionally, load balancing automatically occurs if the machine is unable to satisfy resource demand within a few minutes. The database remains online during load balancing except for a brief period at the end of the operation when connections are dropped."
For example, if 2vcore resource is unable to satisfy workload and more 2vcore can not be assigned within a few minuts, 4vcore on other machine is assigned and failover is happened after new instance  is ready.
Is it correct?

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

ID: 46602618-c925-75b6-bf3e-1039f168f141
Version Independent ID: aa73eb53-55b6-6aa2-2542-e26fd32764fe
Content: Azure SQL Database serverless (preview)
Content Source: articles/sql-database/sql-database-serverless.md
Service: sql-database
Sub-service: service
GitHub Login: @oslake
Microsoft Alias: moslake

Hi @yuharucan That is correct but let me research this to ensure that is the case. I will get back to you with confirming or additional information. Thank you!

@oslake PR has been completed but needs approval. @yuharucan A doc update will be published shortly.
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


How to add a new backend, like openfaas

how to add a new backend , like openfaas

@tanwww The easiest way is to copy-paste a backend from https://github.com/lithops-cloud/lithops/tree/master/lithops/serverless/backends. For example, you can copy the ibm_cf backend, rename it to openfaas, and then change the implementation of all the internal public methods. (you can delete all the private methods and create new ones if necessary)
As with the other backends, A new backend must have (at least) 3 files:

the entry_point.py: This file will go in the runtime to handle the incoming requests and run the functions.
the config.py: Used to verify the user-provided config variables, and fill those that can be defaulted.
and (in your case) the openfaas.py: Contains the logic to invoke the functions, and to create/build/delete the runtime

@tanwww The easiest way is to copy-paste a backend from https://github.com/lithops-cloud/lithops/tree/master/lithops/serverless/backends. For example, you can copy the ibm_cf backend, rename it to openfaas, and then change the implementation of all the internal public methods. (you can delete all the private methods and create new ones if necessary)
As with the other backends, A new backend must have (at least) 3 files:

the entry_point.py: This file will go in the runtime to handle the incoming requests and run the functions.
the config.py: Used to verify the user-provided config variables, and fill those that can be defaulted.
and (in your case) the openfaas.py: Contains the logic to invoke the functions, and to create/build/delete the runtime

@JosepSampe  thanks for your reply, I'll try.

@JosepSampe  thanks for your reply, I'll try.


Restrict sensitive values from Inspect

With no restrictions put on the inspect values of access_token and client, it's currently possible to leak sensitive values into log files or other logger backends.  This PR removes the sensitive values that I am aware of.
I think I have everything covered by tests, but let me know if I missed something and I'll get it fixed for you.

I didn't realize @derive was a 1.8 thing. I'll get something else worked out and reopen this.


Added A par_group_by method

Hello, I found this method useful in my code and thought it could be used here. However it relies on #![feature(slice_group_by)] unstable feature of the rust std library and you would probably want to wait for that method to be stabilized anyways so as to stay consistent with whatever naming they choose. This pull request is here for when that happens. Also if any one has any comments on the code I'd really appreciate hearing them.

Thanks, it's an interesting feature! Do you think you'll try a mutable version as well?
And yes, we'll want to wait for the API to stabilize in the standard library so we can match it.

That should be the mutable version.

Just for reference, that slice_group_by feature is tracked in rust-lang/rust#80552.

I'm interested in this, but I found a bug in it, the following call:
let data = vec![1, 1, 2, 2, 2, 2, 2, 3, 3, 3, 4, 4];
let groups: Vec<_> = data.par_group_by(|&a, &b| a == b).collect::<Vec<_>>();

println!("{:?}", groups);

will result in 5 groups instead of the expected 4, these groups are:
[[1, 1], [2, 2, 2, 2, 2], [3], [3, 3], [4, 4]]

I think that fixed it

I think that fixed it

It did, nice job!

FYI, that stabilization just landed with a rename to chunk_by / chunk_by_mut:
https://github.com/rust-lang/rust/pull/117678

I wonder if this can be refactored to use the internal SplitProducer infrastructure,

I didn't find a good way to shoehorn this into SplitProducer, per se, but I've now made the implementation quite similar to that, and a test asserts that we get exactly len() - 1 comparisons.

Thanks @jakeKonrad! Hope you don't mind that I wrapped this up myself.


Implement full wrappers for pointers and aliases

Currently implemented as strings.

Reassigned 0.2 as it will break API.

Moving back into 0.1.x as it can be done in a backwards compatible fashion using __toString overloading.


Authentication logic is broken

You can get stuck "phantom logged in" if the backend invalidates your session but the frontend doesn't. The frontend won't update the state to unauthenticated if it gets back an unauthenticated response from the server.
One way to reproduce:

Navigate to https://app.couchers.org and log in
Navigate to https://develop--frontend.preview.coucher.org and log in
Log out of https://app.couchers.org (this will remove the cookies)
Now https://develop--frontend.preview.coucher.org is phantom logged in

Possibly easier:

Go to https://develop--frontend.preview.coucher.org and log in
Remove cookies for domain api.couchers.org
You're now phantom logged in

You can get out of it by deleting local storage on the site the frontend is running on. (Delete local storage and then leave the site so it doesn't re-store the state).

Related to #345

Does both being jailed and not logged in return "unauthenticated" error? And these are the only situations?

Yes

On 17 Dec 2020, at 11:36, lucaslcode<EMAIL_ADDRESS>wrote:

Does both being jailed and not logged in return "unauthenticated" error? And these are the only situations?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Can't install latest version (1.1.0) via pods

Probably not yet uploaded to CocoaPods

Absolutely, sorry about that. It’s now available on CocoaPods. CocoaPods/Specs@9854fd5c26743861fc46a8ac343b5be2a3b69c17


Let's a find a nice logo

Is your feature request related to a problem? Please describe.
There is currently no logo for Lotus application, although it is not very important feature it would be nice for documentation and sandbox site.
Describe the solution you'd like
We need a clean and accurate logo for Lotus. We needs a vector source to be able to use it everywhere.
I found those ones:

https://www.svgrepo.com/svg/237667/lotus for a full logo;
https://www.svgrepo.com/svg/237911/lotus-flower usable as clear alternative;

Describe alternatives you've considered
There may be some other nice ones from https://www.svgrepo.com/vectors/lotus/
Or a graphic designer hero to propose something well designed.
Additional context
Lotus is a free and open source project, so there is no fund, we need every sources and license usage rights.

Finally choosed a better one https://www.svgrepo.com/svg/261083/lotus

Done in 0.5.3-pre.9


Poll for image processing when returning a 423

When using transformations that require processing thus returning a 423, poll for that image on the client so it loads upon completion.

This can potentially use the onError function available in next/future/image: https://nextjs.org/docs/api-reference/next/future/image#onerror

Hi @colbyfayock, I would like to take on this issue.

Hey @colbyfayock I too have tried my hands on this issue...so can i submit a PR for it?

@michizhou was first to ask so they're first to give a try. @Smartmind12 if they dont end up working on it you can give it a shot

@michizhou was first to ask so they're first to give a try. @Smartmind12 if they dont end up working on it you can give it a shot

Okay thanks!
Requesting you to kindly open up some new issues to work on in the meantime😊

@Smartmind12 thanks for your interest, I'll try to add some more, in the meantime feel free to open any you think could be valuable, especially in terms of bringing new Cloudinary features into the library

@colbyfayock Could you point out which files I should modify for this issue? I have a rough idea on how to resolve this issue but I'm not certain where a 423 error code would be returned when using transformations that need processing.

I have two thought processes around this
my original intent was to use a ref on the Image inside of here:
https://github.com/colbyfayock/next-cloudinary/blob/main/next-cloudinary/src/components/CldImage/CldImage.js#L60
where we could set an onerror handler: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/img#image_loading_errors
such as what I'm doing here:
https://github.com/colbyfayock/cloudycam/blob/4293c0551ebc64b97f221a13cccf6c4024e15442/src/components/Camera/Camera.js#L38
but instead of onload like in that example, use onerror
howver, Next.js is releasing a new onError prop https://nextjs.org/docs/api-reference/next/future/image#onerror
So it might make the most sense to develop this for the new Image component which will be launched I believe at Next.js Conf at the end of this month
Here's a PR i made that updates the component to use Future Image: https://github.com/colbyfayock/next-cloudinary/pull/3
as far as polling goes I haven't tested to see if they return an error object with that information, so what we could potentially do is when an error occurs, make a simple fetch to the image URL, check THAT response, and if that's a 423, continue polling until it's available
what do you think?

I would think the second approach probably makes more sense in terms of being up to date with the upcoming release. Just to confirm, the new onError prop and new Image component are available in Future Image right? If not, we can go with your original intent.

yup they're available in Future Image
that PR has all the changes I needed to make
if you're familiar with how to do it, you can create a new branch from the Future branch and target that so that we can merge your PR into the Future PR, then once Future is released, I'll update the import and merge it all in
i would think it would still count for hacktoberfest for you given it's a merged PR, if not we can figure something out to get you credit
would that work?

Yeah that sounds good! I'll create a PR on top of yours and we can go from there.

@colbyfayock I have pushed a PR with initial changes on top of your Future PR. It's in an early stage, and I'm not sure if it's using a proper approach, so I'd appreciate it if you could review it and let me know what additional changes are needed. In the meantime, could you reopen this issue? I think another PR you recently merged accidentally closed this.

oops, reopened! i'll take a look :)

@colbyfayock I just noticed that you merged in the feature branch I based my PR on, which closed it. I'll rebase my feature branch to be on top of the main branch and open a new PR.

ohhhh sorry about that - please let me know if there are any issues or conflicts, there were a lot of big  changes, i can help address the conflicts if needed

No worries! I'll let you know if I run into issues.

@michizhou did you ever get this running locally? I just wrote some docs to improve helping people get set up, including writing a script that will upload images to your Cloudinary account for running the Docs site
if you haven't would love to see you go through that to make sure i didnt miss anything and to validate its usefulness
https://github.com/colbyfayock/next-cloudinary#-contributing
thanks

@colbyfayock I haven't yet, but I would love to go through the docs and validate them as I set up!

@colbyfayock It seems you accidentally closed this issue again with another PR you recently merged. Please reopen this issue whenever you can. In the meantime, perhaps the PR template should be modified until this is actually resolved?

what's actually happening that this keeps getting closed? if you know the issue, mind submitting a PR to fix it?

@colbyfayock I have made changes to my latest PR that resolves this issue and also submitted a new PR which fixes the bug that causes this issue to keep getting closed. Please feel free to review them both whenever you are available.
I also went through the docs which you introduced here to help people get set up with running the Docs site and uploading example images. Your changes look good and were very useful, especially when setting up my local environment!


TestRunner App not being created

Question Checklist

[X] Updated fastlane to the latest version
[X] I read the Contribution Guidelines
[X] I read docs.fastlane.tools
[X] I searched for existing GitHub issues

Question Subject
What tool/action do you have a question about?
Fastlane Gym
Is this a question about documentation?
No
Is this a question about a third party plugin? (If so, please go to the plugin repository first)
No
Question Description

So my company has been using Fastlane for awhile and I recently started to work on this. We currently use Microsofts AppCenter to test our apps, but I'm having issues uploading the app to the App Center after making a build with Gym. So normally when I make a build it produces an app in Derived Data called -Runner. When you go to edit schemes in Xcode you'll find this:

I've tried using Gym to supplied this to both xcargs and export_xcargs
"build-for-testing"
But no luck still. I found this post and tried using xcodebuild, but that didn't work either. Is there a way for me to use Gym and produce that Runner app?
https://github.com/fastlane/fastlane/issues/12184

What kind of Runner app do you mean?
Did you maybe also change the Xcode version you are using?
Please also see the comment of @fastlane-bot

@janpio Closing this ticket due to user error.

User error is best error 🚀


⚠ Bofrost.de has degraded performance

In 5a1432e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 995 ms

Resolved: Bofrost.de performance has improved in 9ef9bd0 after 7 minutes.


let people know where the wallet is, on collision

Codecov Report
Base: 58.85% // Head: 58.88% // Increases project coverage by +0.03% tada

Coverage data is based on head (7bd31d0) compared to base (d23559d).
Patch coverage: 100.00% of modified lines in pull request are covered.

mega This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
umbrella View full report at Codecov. loudspeaker Do you have feedback about the report comment? Let us know in this issue.

this is a good idea

see https://github.com/zingolabs/zingolib/pull/260#issuecomment-1447281195

This is useful! It makes the situation of loading a new wallet with zingo-cli easier, because now there's no struggle about finding the existing wallet.
One day, managing that existing wallet file could be a function of zingo-cli. This is a good glance in that direction.

Hi @fluidvanadium this PR staled, so I went through the process of rebasing it, fixing merge conflicts and re-pushing it.   Can you take a look?


Comelit icona

Someone have do the porting of comelit icona inside this addon?
https://github.com/madchicken/homebridge-comelit-icona

Hello,
No one has worked on this feature. Do you think you could help?

Hello, No one has worked on this feature. Do you think you could help?

Yes it could be.


Druid SQL multi-lingual decoded problem

Hi! I'm using 0.18.1.
When I executed sql with Korean characters, I got the empty result.
So, I debugged the broker code and there was the problem as follows.
https://issues.apache.org/jira/browse/CALCITE-2704
We need to bump to avatica-0.17.0.
I'd like to ask you to review.

Thanks for the report and suggested fix @leedohyun. I've opened #10074 and added a test which appears to show the updated Avatica dependency resolve the issue.


different number of queries in Academic, IMDB, etc.  (compared to the original distribution)

Thank you for releasing this, great work!
I have noticed that the "classical datasets" in your release always have less examples than in the original release. For example:
Academic: 185 vs 189
Yelp: 122 vs 128
IMDB: 97 vs 131
Can you comment on what specific examples were left out? And maybe add a few words to README.md?

Which original release are you referring to?
We took the data from https://github.com/jkkummerfeld/text2sql-data . Note that this repo groups all the SQLs that share the same template, so there might be fewer SQLs than the original release.
(we then created separate data points for each SQL, but retrospectively it might be a better decision to create separate data points for each natural language question, rather than SQL queries.)

Thanks for your response!
The original release I'm referring to is indeed https://github.com/jkkummerfeld/text2sql-data. Having read your response, my understanding now is that you only took 1 query per template when you created the distilled test-suite. Does this imply that I can't use the test-suite databases to validate the correctness of the other queries? This would significantly reduce the dataset size. For example, there are 131 examples in the original IMDB dataset for 89 templates. Losing 42 out of 131 examples is not great. Do you plan to create a test-suite for the complete classical test sets? It would be a very valuable resource for the community.
I have also found other anomalies in your dataset release:

In some IMDB queries the variables are not substituted. For example, I have found a query SELECT BIRTH_CITY FROM DIRECTOR WHERE NAME = "director_name0". Is that by design, or should such queries be discarded?

When I looked at the texts field of the dataset entries, I see that some questions are associated with semantically different queries. For example, the question "How many reviews does " business_name0 " have ?" is in texts for both

SELECT REVIEW_COUNT FROM BUSINESS WHERE NAME = "Acacia Cafe" ;
and
SELECT COUNT( DISTINCT ( T1.TEXT ) ) FROM REVIEW AS T1 JOIN BUSINESS AS T2 ON T1.BUSINESS_ID = T2.BUSINESS_ID WHERE T2.NAME = "Acacia Cafe";

Two IMDB queries are not covered:

SELECT MOVIEalias0.TITLE FROM CLASSIFICATION AS CLASSIFICATIONalias0 , GENRE AS GENREalias0 , MOVIE AS MOVIEalias0 WHERE GENREalias0.GID = CLASSIFICATIONalias0.GID AND MOVIEalias0.MID = CLASSIFICATIONalias0.MSID GROUP BY MOVIEalias0.TITLE ORDER BY COUNT( DISTINCT ( GENREalias0.GENRE ) ) DESC LIMIT 1 ;
... that corresponds to "Find the movie which is classified in the most number of genres", and
SELECT PRODUCERalias0.NAME FROM DIRECTED_BY AS DIRECTED_BYalias0 , DIRECTOR AS DIRECTORalias0 , MADE_BY AS MADE_BYalias0 , MOVIE AS MOVIEalias0 , PRODUCER AS PRODUCERalias0 WHERE DIRECTORalias0.DID = DIRECTED_BYalias0.DID AND MOVIEalias0.MID = DIRECTED_BYalias0.MSID AND MOVIEalias0.MID = MADE_BYalias0.MSID AND PRODUCERalias0.PID = MADE_BYalias0.PID GROUP BY PRODUCERalias0.NAME ORDER BY COUNT( DISTINCT ( DIRECTORalias0.NAME ) ) DESC LIMIT 1
... that corresponds to "Which producer has worked with the most number of directors ?"

Thanks for carefully examining the dataset! I think these recommendations are very helpful. Here are my responses to each of your point:

Each template is associated with potentially multiple texts and SQLs. I mistakenly created one data point for each SQL template, which is retrospectively a bad decision. We will fix this. That said, all the texts are still kept in the .pkl file, so this should be a relatively easy fix for us.

another teammate rewrites the IMDB queries into SPIDER style; I will check with him.
The unsubstituted variable name in the SQL query is not intended. Our code made the assumption that variable examples for all SQLs were already provided and hence produced this artifact. Below is the original data dictionary:
`
{'difficulty': 1, 'query-split': '3',
'sentences': [{'question-split': '8', 'text': 'When was " writer_name0 " born ?', 'variables': {'writer_name0': 'Kevin Spacey'}}, {'question-split': '9', 'text': 'In what year was " writer_name0 " born ?', 'variables': {'writer_name0': 'Kevin Spacey'}}],
'sql': ['SELECT WRITERalias0.BIRTH_YEAR FROM WRITER AS WRITERalias0 WHERE WRITERalias0.NAME = "writer_name0" ;', 'SELECT ACTORalias0.BIRTH_YEAR FROM ACTOR AS ACTORalias0 WHERE ACTORalias0.NAME = "actor_name0" ;', 'SELECT DIRECTORalias0.BIRTH_YEAR FROM DIRECTOR AS DIRECTORalias0 WHERE DIRECTORalias0.NAME = "director_name0" ;', 'SELECT PRODUCERalias0.BIRTH_YEAR FROM PRODUCER AS PRODUCERalias0 WHERE PRODUCERalias0.NAME = "producer_name0" ;'],
'variables': [{'example': 'Kevin Spacey', 'location': 'both', 'name': 'writer_name0', 'type': 'writer_name'}]}
`
Here, the second SQL template has an "actor_name0" variable, but no example is provided.

it seems that another teammate made a mistake when rewriting these two queries, and I threw it away thinking that the datapoint contains an error.

Again, thanks for all the comments. I will add a note in the README indicating that the current version has a couple of issues and is useful only for preliminary exploration. Additionally, in our next update, we will release the complete provenance of the data transformation process such that it becomes transparent and maintainable.

I really appreciate that you responded so quickly, @ruiqi-zhong. I am also happy hear that you want to release a better version of the dataset. My team and I are really looking forward to it!
Meanwhile, please find more comments below:
Regarding the missing questions.

That said, all the texts are still kept in the .pkl file, so this should be a relatively easy fix for us.

Hmm, I'm afraid the pkl file does not contain all the information that you'd need to fix everything. Consider the query template from the IMDB dataset. Your pkl file contains the following:
{'db_id': 'imdb',
 'query': 'SELECT RELEASE_YEAR FROM MOVIE WHERE TITLE = "The Imitation Game" ;',
 'texts': ['What year is the movie " movie_title0 " from ?',
  'What year was the movie " movie_title0 " produced'],
 'variables': [{'example': 'The Imitation Game',
   'location': 'both',
   'name': 'movie_title0',
   'type': 'movie_title',
   'ancestor_of_occuring_column': {('movie', 'TITLE')}}],
 'testsuite': {'database/imdb/imdbv9211stround0group128.sqlite',
  'database/imdb/imdbv9229thround0.sqlite',
  'database/imdb/imdbv9229thround1.sqlite',
  'database/imdb/imdbv9229thround2.sqlite',
  'database/imdb/imdbv9229thround3.sqlite',
  'database/imdb/imdbv9229thround4.sqlite'}}

Meanwhile, the original entry in the data release by Universtity of Michigan researchers is
{'difficulty': 1,
 'query-split': '5',
 'sentences': [{'question-split': '9',
   'text': 'What year is the movie " movie_title0 " from ?',
   'variables': {'movie_title0': 'Dead Poets Society'}},
  {'question-split': '6',
   'text': 'What year was the movie " movie_title0 " produced',
   'variables': {'movie_title0': 'The Imitation Game'}}],
 'sql': ['SELECT MOVIEalias0.RELEASE_YEAR FROM MOVIE AS MOVIEalias0 WHERE MOVIEalias0.TITLE = "movie_title0" ;'],
 'variables': [{'example': 'The Imitation Game',
   'location': 'both',
   'name': 'movie_title0',
   'type': 'movie_title'}]}

It appears that your pkl file does not include the variable assignments that would be required to reconstruct the second query ("What year was the movie The Imitation Game produced"). I also imagine that test-suite databases that you constructed probably do not contain the content that would be required to properly evaluate the corresponding query against model predictions. Is my understanding correct? It would imply that the databases would need to be regenerated to fix the issue once for all.
Regarding the queries that were not deanonymized
Thank you for your clarification, I think I understand now what happened. Note that the documentation for text2sql-data says that they "only use the first query, but retain the variants for completeness (e.g. using joins vs. conditions)." Queries other than the first one often do not even have the variable assignments provided, as you correctly noted in your analysis.
I think it would make sense for you to also use the first query only when you regenerate the data.
This also explains why you have different queries associated with different question texts (i.e. the review count example in my previous post).
Regarding the missing queries
Thanks for making clear what happened here.

I think I have found another issue. While I can see that a lot of effort was put into rewriting the original SQL queries from text2sql-data into Spider SQL, there are still some issues:
In [10]: from process_sql import Schema, get_sql, get_schema

In [11]: get_sql(Schema(get_schema('/home/dzmitry/dist/duorat/data/database/imdb_test/imdb_test.sqlite')),
    ...:         'SELECT COUNT( DISTINCT ( T3.TITLE ) ) FROM DIRECTOR AS T1 JOIN DIRECTED_BY AS T2 ON T1.DID = T2.DID JOIN MOVIE AS T3 ON
    ...:  T3.MID = T2.MSID WHERE T1.NAME = "Quentin Tarantino" AND T3.RELEASE_YEAR < 2010')

...

~/dist/test-suite-sql-eval/process_sql.py in parse_col(toks, start_idx, tables_with_alias, schema, default_tables)
    186             return start_idx+1, schema.idMap[key]
    187
--> 188     assert False, "Error col: {}".format(tok)
    189
    190

AssertionError: Error col: (

It appears that it should be SELECT COUNT(DISTINCT T3.TITLE)) instead of SELECT COUNT(DISTINCT(T3.TITLE)) for the Spider-style parser to function correctly.

thanks for all the feedbacks! I just pushed an update.

missing queries from the original Jonathan's repo.

I created one data point for each natural language query. The data conversion process is now in classical_provenance.ipynb . Now for each datapoint in classical_test.pkl, we can trace back to the original datapoint in Jonathan's repo using the orig_id field

incorrect value replacement

I fixed it and the details can be seen in classical_provenance.ipynb. Specifically, I used the variable values for the Questions-SQLs pairs to replace the values such that 1) every placeholder gets replaced by a value 2) both the natural language and SQLs are replaced with the same value for each place holder. I also fixed some queries that result in execution errors.

semantically inequivalent query

In this version, I decided to back-off and directly use the SQLs from Jonathan's repo.
I changed some details on how to create neighbor queries. 1) Instead of dropping every sub span to create neighbor queries, I now only drop every sub span that corresponds to a derivation of an AST intermediate symbol (i.e. subtree). This is because SQLs from Jonathan's repo are usually very long, and dropping every possible subspan will create a quadratic number of neighbor queries, which is too many. 2) I forbid dropping any predicate of join statement (e.g. WHERE child.id == parent.id). This is because dropping those subspans will drastically increase computational time since it will then take a cartesian product of two tables.
I also added two features: evaluating on a subset of the classical datasets and caching results for SQLs that have been evaluated. This is because queries on ATIS sometimes can take a while to run.
Again, I really appreciate your time examining the queries closely. I am closing this issue. Please feel free to open a new issue if you have any comments on the updated version.

Hi Ruiqi, thanks a lot for your continued effort to make this test-suite correct for classical datasets. Unfortunately, I think I have found another issue, this time probably the last one.
In Jonathan's repo examples are grouped by SQL queries. Each query can have an arbitrary number of associated sentences. Every sentences has its own set of values that should be substituted for variables. E.g. in geography.json in the first query the city is Arizona, in the second it is Texas, etc.
Your code in classical_provenance.ipynb takes values for variables from another source:
instance_name2examples = {d['name']: d['example'] for d in instance_variables}
As a result, many queries in classical.pkl have wrong literals. For example, in classical.pkl you have a question
'What year is the movie The Imitation Game from ?', which instead should be "What year is the movie Dead Poets Society from ?".
The consequence of this mismatch is that the generated test suite can not be used to evaluate correctness of the original queries from the classical dataset. I noticed that when the evaluation returned some really surprising false positives.
I would be extremely grateful if you fix this issue and regenerate the test suite again. This kind of resource makes a big difference in making meaningful evaluation reproducible.

(I'm re-opening the issue)
Thanks for your feedback! Would you mind clarifying which one of the following cases is happening? Suppose the dictionary associated with each text-sql pair is d, then

There is a misalignment between the d['text'] and d['query']. If so would you mind tell me the index (in the .pkl file) of where this misalignment happens?
The test suite cannot distinguish the gold SQL query (as is provided in the .pkl) and another query. If so, can you let me know which gold query has the problem, and what does the false-positive look like?
The classical.pkl is missing certain combinations of natural language + literals from Jonathans' repo. This was in fact intentional: we only keep one way to insert literal for each SQL (template) and change the corresponding natural language text, such that we only need to create one test suite for each SQL template. However, I can also see the advantage of keeping the original variety of literals (e.g. especially for evaluating models where literals in the input are not anonymized/models that need to predict values, etc). if you think that we should add back those combinations, I will also be happy to do that soon in the future.

I ran the following code
import pickle as pkl
test = pkl.load(open('classical_test.pkl', 'rb'))
for idx, d in enumerate(test):
    t = d['text']
    if 'The Imitation Game' in t:
        print(t)
        print(d['query'])
        print(d)

and did not find "dead poet" in the output. Looks like it is problem 3 rather than 1?
This will help me pin down the problem with the current evaluation, and I will definitely fix it if there is an issue. Thanks!

It is the case 3 that happens, namely:

The classical.pkl is missing certain combinations of natural language + literals from Jonathans' repo.

I would argue this is a significant change compared to the original data, because linking literals to the right column in the schema is in general a non-trivial issue. For example, in "How big is New Mexico?" New Mexico is clearly a state, because there is no city called New Mexico. Whereas in "How big is New York?" New York can refer to either a city or a state.
You can also check out the preprocessing code for this data that was released by Google: https://github.com/google-research/language/blob/89e5b53a0e7f9f3e2da25a5da71ce5bd466acabd/language/xsp/data_preprocessing/michigan_preprocessing.py#L30 . It is very well documented, and does variable substitution in what I think is the right way.

This was in fact intentional: we only keep one way to insert literal for each SQL (template) and change the corresponding natural language text, such that we only need to create one test suite for each SQL template.

I understand the computational effort considerations, but technically this simplification changes the original question texts in the datasets.

thanks for the clarification. I will probably make this update by the end of this month. Thanks!

Great, thank you so much! I am looking forward to the new release.


Darkest dark theme

How to change color of menu bar and scrollings to dark color in darkest dark theme?
and light borders around the fields?
it looks strange in dark theme

Thank you for your question. It doesn't look very pretty.

I'm afraid we can'f fix this. All customizations come from DarkestDark and/or Eclipse Dark theme .


[pytorch] Adds image classification resnet18 base model to model zoo

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report
Base: 72.08% // Head: 70.19% // Decreases project coverage by -1.89% :warning:

Coverage data is based on head (2bed909) compared to base (bb5073f).
Patch coverage: 71.74% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2079      +/-   ##
============================================
- Coverage     72.08%   70.19%   -1.90%
- Complexity     5126     6076     +950
============================================
  Files           473      605     +132
  Lines         21970    26924    +4954
  Branches       2351     2917     +566
============================================
+ Hits          15838    18898    +3060
- Misses         4925     6613    +1688
- Partials       1207     1413     +206

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...modality/cv/translator/wrapper/FileTranslator.java
44.44% <ø> (ø)

...y/cv/translator/wrapper/InputStreamTranslator.java
44.44% <ø> (ø)

... and 526 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Create terraform.yml

Please give more information why you made this pull request? How should it improve Tesseract?


Example AsyncThunks

Hi first of all thanks for the library! It make it easy the mock of redux!
I'm looking for examples to implement an async actions  but without success.
It's posible with the current implementation, make tests for async thunk actions?
I've tryed with the Examples but without success.
I'm trying with the current code (importing original async action)
test("Should dispatch if all fields exists and were validated", async () => {
    const { dispatch } = mockReactRedux();

    const mockedSubmit = jest.fn().mockImplementation(() => {
      dispatch(newContact(values));
    });

    render(<ContactForm onSubmit={mockedSubmit} />);

    userEvent.type(screen.getByPlaceholderText("Nombre"), "John");
    userEvent.type(screen.getByPlaceholderText("Email"), "john@gmail.com");
    userEvent.type(screen.getByPlaceholderText("Teléfono"), "23456789");
    userEvent.type(screen.getByPlaceholderText("Escribe tu mensaje"), "Lorem ipsum dolor sit");

    userEvent.click(screen.getByTestId("contactButton"));

    await waitFor(() => {
      expect(dispatch).toHaveBeenCalled();
      expect(dispatch).toHaveBeenCalledWith(newContact(values));
    });
  });

But the result is:

I think maybe is for the thunk middleware.
PD: Sorry if my english is not good, I'm still learning!
Best Regards!

Hi @AntonyFagundez, thanks for the issue! Totally clear & understandable English; what you're asking for makes sense.
This is an annoying quirk we found around working with dispatched thunks, yeah. I'm guessing your newContact function looks something like:
export const newContact = (values) => (dispatch, getState) => {
  /* ... */
};

expect(dispatch).toHaveBeenCalledWith(newContact(values)); // New function ❌

Structure like that means newContact returns a new [Function anonymous] every time it's called. I'll add an example to the docs.
In the meantime...
Strategy: Same Function
One solution would be to make a function that stays the same between them:
export const newContentThunk = (dispatch, getState) => {
  /* ... */
};

export const newContent = () => newContentThunk;

expect(dispatch).toHaveBeenCalledWith(newContentThunk()); // Same function ✔

...though that direct line of code doesn't support an action prop the way your newContact takes in a values.
Strategy: Memoized Function
If you want to support parameters, you could use something like Lodash's memoize to cache the function returned by newContent:
import { memoize } from "lodash";

export const newContent = memoize((values) => {
  return (dispatch, getState) => {
    /* ... */
  };
});

const values = {
  /* ... */
};

newContent(values);

expect(dispatch).toHaveBeenCalledWith(newContent(values)); // Same function ✔

Note that Lodash by default does a strict equality check, so while the above snippet works, newContent({}) === newContent({}) would result in false because the {} is different between those two calls.

Hi @JoshuaKGoldberg thanks for the detailed answer>
I was indeed making a mistake comparing two references in memory.
With your help I finally get a solution (with memoize from lodash)

  test("Should dispatch if all fields exists and were validated", async () => {
    const { dispatch, state } = mockReactRedux();

    const sameFunction = memoize(() => newContact(values)(dispatch, state));

    const mockedSubmit = jest.fn().mockImplementation(() => {
      dispatch(sameFunction());
    });

    render(<ContactForm onSubmit={mockedSubmit} />);

    userEvent.type(screen.getByPlaceholderText("Nombre"), "John");
    userEvent.type(screen.getByPlaceholderText("Email"), "john@gmail.com");
    userEvent.type(screen.getByPlaceholderText("Teléfono"), "23456789");
    userEvent.type(screen.getByPlaceholderText("Escribe tu mensaje"), "Lorem ipsum dolor sit");

    userEvent.click(screen.getByTestId("contactButton"));

    await waitFor(() => {
      expect(dispatch).toHaveBeenCalled();
      expect(dispatch).toHaveBeenCalledWith(sameFunction());
    });
  });

Thanks again!!


Remove bogus "@TestOn" annotation in mocks.dart

This file doesn't define any tests, so there's no need for the
annotation.  Fixes a static analysis error, since mocks.dart doesn't
import the necessary package to even resolve the symbol.
TBR @bwilkerson

LGTM


Configure Otel Agent to monitor external MySQL and Redis metrics via Terraform in Google

To enhance our monitoring capabilities by configuring the OpenTelemetry (Otel) agent running on our clusters to collect key performance metrics (CPU, memory usage) from external MySQL, Redis and any other key infrastructure aspects. The setup should be executed using Terraform to ensure all permissions and configurations are correctly provisioned. The end goal is to integrate this data into Datadog for centralized monitoring in dedicated cloud.
Requirements
The Otel agent must be able to access metrics from external MySQL and Redis databases.
Use Terraform to provision necessary Kubernetes resources and permissions.
Confirm the ability of Otel to collect and relay accurate metrics.
Make sure the otel metrics are tag correctly with attriubes from the k8s cluster
Acceptance Criteria
The Otel agent must successfully collect and display MySQL and Redis CPU and memory usage metrics from external sources.
All configurations must be managed via Terraform with proper versioning and module usage.

There is 2 Prs linked with this issue :
https://github.com/wandb/helm-charts/pull/141 >> stackdriver subchart support
https://github.com/wandb/terraform-google-wandb/pull/126 >> otel with stackdriver support


UI for building the environments

Context
Thanks for putting this repo together!
Just stumbled on the related blog post today, which seems to be announcing the project a bit more officially: https://2i2c.org/blog/2024/jupyterhub-binderhub-gesis/
The blog post mentions the following sentence:

A pre-selected list of environments (container images), provided by the administrators who set up this JupyterHub

Which leads to the following question: is there also a UI that administrators can use to pre-build these environments? Or are there plans to make one?
A while ago I opened this topic on Discourse to ask this question, which led to this binderhub-service repo on GitHub: https://discourse.jupyter.org/t/ui-for-building-ztjh-ready-images-with-repo2docker-binderhub/22610.
The reason I'm asking is because we have recently been looking updating the existing tljh-repo2docker plugin to make it more modular, and also work with ZTJH deployments. And initiated some work to revamp the UI for building environments (the UI is only available to JupyterHub administrators).
In the end, it looks like both binderhub-service and tljh-repo2docker may have quite a bit of overlap.
So this issue is mostly to get an idea whether the maintainers of this binderhub-service repo would be interested in having a UI to pre-build images (even as opt-in), and potentially join efforts.
Proposal
At QuantStack we currently have some funding to:

Work on revamping the tljh-repo2docker UI with a modern stack, and so the plugin can be installed as a JupyterHub service instead of the current use of c.JupyterHub.extra_handlers:

See https://github.com/plasmabio/tljh-repo2docker/pull/69 for some demo screencasts
https://github.com/plasmabio/tljh-repo2docker/issues/63

Make tljh-repo2docker work with both TLJH and ZTJH

Which is why switching to JupyterHub service could make sense, so the functionality could be available in more JupyterHub scenarios and not just TLJH
It also sounds like tljh-repo2docker may in fact be able to reuse the binderhub package directly for building the images. Building on the recent API mode addition: https://github.com/jupyterhub/binderhub/pull/1647

Updates and actions
Maybe a first step could be to check if tljh-repo2docker could be converted to use binderhub for its backend (or the same stack as used here in binderhub-service). And make it possible to install the UI separately easily, so it could still be used in TLJH but also now in ZTJH with binderhub-service.
Maybe this could be structured in a way it can be useful to multiple deployment scenarios (local hub, TLJH, ZTJH).
Let us know what you think, and if there might be some challenges for providing such UI. Thanks!

cc @trungleduc @atrawog @SylvainCorlay who may be interested in this discussion

Pinging @yuvipanda, @consideRatio, and @GeorgianaElena who may be interested in this discussion, @jtpio 😉

Sorry this got lost in github.
This is super exciting, @jtpio! As you know I'm a huge fan of tljh-repo2docker. We explored basing off that to begin with, but instead choose to go the binderhub route. I think expanding the UI that tljh-repo2docker has to work on z2jh has well would be excellent and extremely valuble to a lot of communities! binderhub can work without kubernetes (https://github.com/jupyterhub/binderhub/tree/main/testing/local-binder-local-hub for example), so it can work well with tljh too. There's also now a published binderhub API client.
I'd love to find ways to collaborate with Quanstack and you on this.
What do you think would be a useful next step? A sync meeting to explore options?

Just noting that this is something the Binder community has discussed a few times and generally there's been a lot of enthusiasm for this. The last time I can remember, we discussed it in this issue:

https://github.com/jupyterhub/repo2docker/issues/702

I think I even had a little ipywidgets demo of this back in the day
In particular I think that CodeOcean (linked in the issue) has roughly the workflow that I had in mind. They also use JupyterLab components (though I believe their system is all proprietary), so may be good to look towards for inspiration.

I think this is actually quite different from the linked to repo2docker / binderhub issues, as that's about managing the source of the environment itself, while this is more about managing built environments. I think the original announcement of tljh-repo2docker (https://discourse.jupyter.org/t/a-tljh-plugin-to-build-user-environments-with-repo2docker/4258) has screenshots illustrating this quite nicely.

Oops - disregard my comment in that case, I must have misunderstood it. Was just trying to provide some helpful breadcrumbs 🙂

@jtpio I am off on the 19th. Can we do 22nd? I can do 8-9am pacific then.

@jtpio yes, we'll need to port the UI for building the profile list, and it needs to be stored somewhere. I suspect a lot of what you currently have can be reused.
The list can be dynamically populated - profile_list can be set to a coroutine, which can then read that list from somewhere else easily.

Can we do 22nd? I can do 8-9am (although ofc 9am-10am is much better, but may be super difficult for you) pacific then.

I think 22nd 9am-10am should work :+1:


Following "Codecov outside docker" documentation results in 400 response

While following the documentation for Testing with docker(Codecov outside docker) I retrieve 400 response(s) when uploading report.
It could be interesting to know that I'm trying this in Docker cloud(autotesting)
(Note that I've replaced some uninteresting config specific values with <SOMETHING>)
As stated in the documentation(linked above) I run bash <(curl -s https://codecov.io/bash) as a final step.
Running with the yaml config file:
.codecov.yml
codecov:
  branch: develop

comment:
  layout: "reach, diff, flags, files"
  behavior: default
  require_changes: false
  require_base: no
  require_head: yes
  branches: null

I get this result:
_____ _
/ ____| | |
| | ___ __| | ___ ___ _____ __
| | / _ \ / _` |/ _ \/ __/ _ \ \ / /
| |___| (_) | (_| | __/ (_| (_) \ V /
\_____\___/ \__,_|\___|\___\___/ \_/
Bash-e3f8e98
==> Docker detected.
project root: .
--> token set from env
Yaml found at: .codecov.yml
==> Running gcov in . (disable via -X gcov)
==> Python coveragepy not found
==> Searching for coverage reports in:
+ .
-> Found 1 reports
==> Detecting git/mercurial file structure
==> Appending build variables
+ IMAGE_NAME
==> Reading reports
+ ./coverage-vol/coverage.xml bytes=41817
==> Appending adjustments
http://docs.codecov.io/docs/fixing-reports
-> No adjustments found
==> Gzipping contents
==> Uploading reports
url: https://codecov.io
query: branch=develop&commit=<HASH>&build=&build_url=&name=&tag=latest&slug=index.docker.io%2F<ORG NAME>%2F<REPO NAME>&service=docker&flags=&pr=&job=
-> Pinging Codecov
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again...
-> Pinging Codecov
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again...
-> Pinging Codecov
/dev/fd/63: line 1441: echo: write error: Broken pipe
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again...
-> Pinging Codecov
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again...
-> Uploading to Codecov
HTTP 400
slug must match pattern ^[\w\-\.]{1,255}\/[\w\-\.]{1,255}$

I've tried another run by changing the slug value by setting it through the .codecov.yml file. This run results in the following:
.codecov.yml
codecov:
  slug: <ORG NAME>/<REPO NAME>
  branch: develop

comment:
  layout: "reach, diff, flags, files"
  behavior: default
  require_changes: false
  require_base: no
  require_head: yes
  branches: null

I get this result:
_____ _
/ ____| | |
| | ___ __| | ___ ___ _____ __
| | / _ \ / _` |/ _ \/ __/ _ \ \ / /
| |___| (_) | (_| | __/ (_| (_) \ V /
\_____\___/ \__,_|\___|\___\___/ \_/
Bash-e3f8e98
==> Docker detected.
project root: .
--> token set from env
Yaml found at: .codecov.yml
--> slug set from yaml
==> Running gcov in . (disable via -X gcov)
==> Python coveragepy not found
==> Searching for coverage reports in:
+ .
-> Found 1 reports
==> Detecting git/mercurial file structure
==> Appending build variables
+ IMAGE_NAME
==> Reading reports
+ ./coverage-vol/coverage.xml bytes=41817
==> Appending adjustments
http://docs.codecov.io/docs/fixing-reports
-> No adjustments found
==> Gzipping contents
==> Uploading reports
url: https://codecov.io
query: branch=develop&commit=<HASH>&build=&build_url=&name=&tag=latest&slug=<ORG NAME>%2F<REPO NAME>&service=docker&flags=&pr=&job=
-> Pinging Codecov
/dev/fd/63: line 1441: echo: write error: Broken pipe
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again...
-> Pinging Codecov
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again...
-> Pinging Codecov
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again...
-> Pinging Codecov
/dev/fd/63: line 1441: echo: write error: Broken pipe
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again...
-> Uploading to Codecov
HTTP 400
service must be ci-provider

I see. So there is an issue with service=docker. I'll patch that server side now.


folder name interfaze should change to interface?

as title said

Hi, @linking123
Good question!  interface is reserved word of Java, so the package name can not be interface and if you are going to modify interfaze to interface, it won't be success.  The pronunciation of interfaze is much similar to interface's, aha? 😉

Hi, @linking123
If there is no other problems, I will close this issue.


Bumping crossplane provider with TF v0.22.1

Description of your changes

Fixes #
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@TylerGillson I tried remove .work directory and run make generate still check-diff is failing. not able to reproduce it in local

@SivaanandM I fixed your DCO, deleted .work locally, ran make reviewable, and uploaded the diff.
I was getting lint failures locally, but I just ignored them and the failure isn't happening in CI 🤷🏼♂


Add BSDmakefile to prevent errors when make is called under FreeBSD

The syntax of the gitea Makefile is not platform-agnostic and is
specific to the GNU version of make. BSD platforms such as FreeBSD
ship with bmake (BSD make) as their default make program; attempting
to compile gitea by simply executing make causes a wall of errors to
show as a result of syntax incompatible with BSD make.
If a file named BSDmakefile is present, bmake will give it
preference over a generic Makefile. This BSDmakefile is taken from
the BSD-licensed gmake-proxy project, which transparently proxies
all make commands to gmake (GNU make) on systems where bmake is
the default, and if gmake is not installed an error message is
displayed.

Codecov Report

Merging #4446 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4446   +/-   ##
=======================================
  Coverage   20.06%   20.06%
=======================================
  Files         153      153
  Lines       30769    30769
=======================================
  Hits         6174     6174
  Misses      23652    23652
  Partials      943      943

Continue to review full report at Codecov.

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

LGTM


Unsupported argument at load-balancer-target-group

Getting unsupported argument error on "terraform plan"
Error trace:
Error: Unsupported argument
on .terraform/modules/couchbase_target_group/modules/load-balancer-target-group/main.tf line 59, in resource "aws_alb_listener_rule" "http_path":
59:       field  = condition.value["field"]
An argument named "field" is not expected here.
Error: Unsupported argument
on .terraform/modules/couchbase_target_group/modules/load-balancer-target-group/main.tf line 60, in resource "aws_alb_listener_rule" "http_path":
60:       values = condition.value["values"]
An argument named "values" is not expected here.
Error: Unsupported argument
on .terraform/modules/sync_gateway_target_group/modules/load-balancer-target-group/main.tf line 59, in resource "aws_alb_listener_rule" "http_path":
59:       field  = condition.value["field"]
An argument named "field" is not expected here.
Error: Unsupported argument
on .terraform/modules/sync_gateway_target_group/modules/load-balancer-target-group/main.tf line 60, in resource "aws_alb_listener_rule" "http_path":
60:       values = condition.value["values"]
An argument named "values" is not expected here.
Terraform v0.12.29

provider.aws v3.0.0
provider.template v2.1.2

Ubuntu 20.04

downgrading version of provider.aws to 2.70.0 resolved the issue.
provider "aws" {
...
version = "~> 2.0"
}

That's correct! AWS Provider 3.0.0 just came out with lots of backwards incompatible changes. We haven't had a chance to update this repo yet. PRs to fix those incompatibilities are very welcome!


[Enhancement][Proposal][DevTools] Custom Property Editor  for the value of control properties

I would like to propose to replace this with DataGridTemplateColumn and select the Template based on the property type, if no match is found the current behavior will be used.

@grokys


Add inverter to cyan LED

Right now the cyan LED is being used as link status, which is high when there is no link, and low when there is a link. We should invert it using a single gate inverter.

Actually the link LED might just be a case of needing to reverse it and connect to VCC instead of GND.  Need to re-read the datasheet to make sure that's the case. Not sure if it's current sinking it when low.

Yes, correct. Reversing polarity and hooking it up to VCC instead of GND will work.


1 GitHub action

requires https://github.com/KaotoIO/vscode-kaoto/pull/14

create PR based on branch of this organization so that jobs are exectued ont he PR, see https://github.com/KaotoIO/vscode-kaoto/pull/26


add_participant doesn't set the mailbox_type, which may be related to a larger issue

I just discovered the add_participant method, which is just a perfect fit for what I'm trying to do now.
But the thing is that when I use it it leaves the Receipt with a nil mailbox_type, so it's not found when querying for user.mailbox.inbox for example.
Digging in the code it seems that when the feature was first introduced it did actually set the mailbox_type to 'inbox' : https://github.com/mailboxer/mailboxer/pull/158/files#diff-a16448f1ee3d8bd75e4d733276662d8bR117
receipt = Receipt.new
receipt.notification = message
receipt.is_read = false
receipt.receiver = participant
receipt.mailbox_type = 'inbox' # <------------------- HERE
receipt.updated_at = message.updated_at
receipt.created_at = message.created_at
receipt.save

Then some refactoring was made to use the ReceiptBuilder, and now it's built like this:
Mailboxer::ReceiptBuilder.new({
  :notification => message,
  :receiver     => participant,
  :updated_at   => message.updated_at,
  :created_at   => message.created_at
}).build.save

You can notice that mailbox_type is not provided as an argument to ReceiptBuilder.
When I go to ReceiptBuilder I can see that it looks like the mailbox_type would be set automatically to 'inbox' because there is a mailbox_type method defaulting to 'inbox'.
class Mailboxer::ReceiptBuilder < Mailboxer::BaseBuilder
  [...]
  def mailbox_type
    params.fetch(:mailbox_type, 'inbox')
  end
end

But looking at the BaseBuilder I don't think this method gets ever called:
class Mailboxer::BaseBuilder
  [...]
  def build
    klass.new.tap do |object|
      params.keys.each do |field|
        object.send("#{field}=", get(field)) unless get(field).nil?
      end
    end
  end

So I'm a bit concerned about the pattern used with Builders.
Is this the only place where some defaults are not being picked up correctly ?
To just fix the add_participant issue I think one could just explicitly add :mailbox_type => 'inbox' to the parameters passed to the ReceiptBuilder.
What do you think ?

@jerefrer Sounds reasonable.  I don't really have my head into this code ATM, so if you write a PR with a test to fix the problem, It will likely be accepted

@jcoyne Done !

:+1:  for this feature.  I noticed this also, not sure if it's intended behavior.  Having to wrap this call to set the type after adding a participant.


Please help me

I use this.$nextTick in mounted, I want to get this.$refs.chart,but it is undefined.later use hot-loader it is ok .  my English is very bad. Sorry

You'll want to ask this in the Vue Forum. The issues are only for bugs or suggestions.
Scott

If you want to ask such question, please use our forum or gitter. The issue list is reserved exclusively for bug reports and feature requests.
Thanks.

I think is a bug


coref_clusters

While executing project i am facing the following issue please help ? I have attached my screen shot for reference
python3 corefrence.py
My sister has a dog. She loves him
Traceback (most recent call last):
File "corefrence.py", line 6, in
b = doc._.coref_clusters
File "/Users/shahsamad/Desktop/Angular4/pythonProj/env/lib/python3.7/site-packages/spacy/tokens/underscore.py", line 28, in getattr
raise AttributeError(Errors.E046.format(name=name))
AttributeError: [E046] Can't retrieve unregistered extension attribute 'coref_clusters'. Did you forget to call the set_extension method?
(env) Shahs-Mac-mini:pythonProj shahsamad$

I have encountered the same problem, but I don't know how to modify it. Have you solved it? Can you teach me?

I have solved this issue. Just reinstall the 'MODEL_URL' , try :
pip install https://github.com/huggingface/neuralcoref-models/releases/download/en_coref_sm-3.0.0/en_coref_sm-3.0.0.tar.gz

This should be fixed in the latest release (4.0)

Same, I'm having this issue exclusively in Jupyter Notebook but note in Pycharm

I haven't run into this specific issue myself, and the fact that it works in PyCharm does suggest a problem with your virtual environment in your notebook. Can you make sure it's running from the correct venv? Have you tried restarting your kernel etc? I sometimes have issues with Jupyter with other libraries, and restarting 3 times sometimes helps :|

Yes I fixed I apologize...simply copied code incorrectly...

No worries, thanks for letting us know!


How to set the theme in the website examples?

I am trying to apply this theme: http://www.highcharts.com/demo/line-time-series/grid-light
I checked this issue #5  but still could not figure out how to do it.
Would appreciate any help.

I used the exact config from the examples just replace the part referencing Highcharts with
stops: [
            [0, ReactHighcharts.Highcharts.getOptions().colors[0]],
           [1, ReactHighcharts.Highcharts.Color(ReactHighcharts.Highcharts.getOptions().colors[0]).setOpacity(0).get('rgba')]

                    ]

Just change Highcharts to ReactHighcharts. It works for me.
Hope this piece of code is useful.
import ReactHighcharts from 'react-highcharts';
const config = {
series: [{
name: 'Target',
type: 'polygon',
data: [[153, 42], [149, 46], [149, 55], [152, 60], [159, 70], [170, 77], [180, 70],
[180, 60], [173, 52], [166, 45]],
color: ReactHighcharts.Highcharts.Color(ReactHighcharts.Highcharts.getOptions().colors[0]).setOpacity(0.5).get(),
enableMouseTracking: false
        }]

}


[BUG] Get-PnPClientSidePage does not return web parts for team site

Reporting an Issue or Missing Feature
Issue
Expected behavior
I expect to see home page elements (webPart.Title)
Actual behavior
After runing the script on a newly created team site, the script does not return any elements. If I add or remove any element to the default home page then the script returns the all elements. Env sharepoint online.
Steps to reproduce behavior

Create a team site
Run script ($sourceUrl = "https://mysharepointonline.sharepoint.com/sites/myTeamSite"
Connect-PnPOnline -Url $sourceUrl -Credentials $global:credentials
page = Get-PnPClientSidePage -Identity "Home.aspx"
$webParts = $page.Controls
foreach ($webpart in $webparts) {
$webpart.Title
}

What is the version of the Cmdlet module you are running?
1.5.0
Which operating system/environment are you running PnP PowerShell on?

[x] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

@Jelon3k - According to the documentation, this seems an expected behavior from what you're describing?

This command allows the retrieval of a modern sitepage along with its properties and contents on it. Note that for a newly created modern site, the Columns and Sections of the Home.aspx page will not be filled according to the actual site page contents. This is because the underlying CanvasContent1 will not be populated until the homepage has been edited and published. The reason for this behavior is to allow for the default homepage to be able to be updated by Microsoft as long as it hasn't been modified. For any other site page or after editing and publishing the homepage, this command will return the correct columns and sections as they are positioned on the site page.

@veronicageek Thank you, you're right. The method works according to the documentation. Can you suggest me any other method to access these components?

What do you mean?


Input focus not working

When using the awesome dragula plugin I came across a bug, the input element value can't be modified, I have test it on various browser so it doesn't seem like a browser specific bug. I have made a quick demonstration about this issue, https://jsfiddle.net/qoaqba62/

Looks like a bug. Yes. https://jsfiddle.net/qoaqba62/1/

Temporary workaround :  https://jsfiddle.net/qoaqba62/2/

It's weird because we just prevent the default event on mousedown and mousemove, but not click. I presume focus happens on mousedown, maybe. We need the mousedown prevention, so a potential fix might be to just .focus() on e.target during mousedown.

Yeah. mousedown it is

Yes the fix works.

Can you pr?

Fixed in 3.0.5

thank you, works like a charm.


Transpiler:  Implement field wildcards

SELECT *           Returns all fields and tags.

The way to implement this will involve pivot() most likely. While we will need join at some point in the future, any wildcards (including regex wildcards) are going to need pivot while named joins, where you have a query like select value1, value2, will involve a join.


https://yeumoney.com/

Link
https://yeumoney.com/
Version
Manifest-v2
What browser(s) are you using?
Chrome browsers (includes brave, edge, opera, etc)
What platform are you using?
Computer (Windows, Linux, MacOS)
(Optional) Anything else?
No response

bump?


[refactor] use multierr

https://pkg.go.dev/go.uber.org/multierr#section-readme

Once we bump to golang 1.20 we should use https://go.dev/doc/go1.20#errors