Moving a column also sort it.

I'm using ag-grid in a Angular 4 project (version 9.0.3). The problem is when I move a column (with the mouse - drag & drop) then the grid sort the column too.

Hi
I am unable to reproduce this - do you have a plunker I can take a look at?
thanks

At this moment no, but to reproduce the bug when you drop de column the mouse pointer must stay into de column header.
I can show you my template, I don't know if this can help you:
<ag-grid-angular
    [style.width.%]="100"
    [style.height.px]="_gridHeight"

    class="ag-fresh ag-without-pagination-panel"

    [gridOptions]="_gridOptions"
    rowModelType="pagination"
    [paginationPageSize]="_pageSizeSelected"
    [datasource]="_datasource"
    [getRowNodeId]="getRowNodeId"

    animateRows
    enableServerSideSorting
    enableServerSideFilter

    enableColResize
    groupHeaders
    suppressDragLeaveHidesColumns

    toolPanelSuppressGroups
    toolPanelSuppressValues

    [headerHeight]="_gridUtils.HEADER_HEIGHT"
    [rowHeight]="_gridUtils.ROW_HEIGHT"
    rowSelection="single"

    (gridReady)="onReady($event)"
    (paginationPageLoaded)="onRowDataChanged($event)"
    (gridSizeChanged)="onGridSizeChanged($event)"
    (selectionChanged)="onRowSelected($event)"
>

    <ag-grid-column colId="meassure" headerName="Meassure" field="meassure" [width]="150" filter="text"   [filterParams]="getDefaultFilterParams()"></ag-grid-column>
    <ag-grid-column colId="item"     headerName="Item"     field="item"     [width]="160" filter="text"   [filterParams]="getDefaultFilterParams()"></ag-grid-column>
    <ag-grid-column colId="os"       headerName="OS"       field="os"       [width]="120"
                    [cellRenderer]="platformCellRender"
                    [filterFramework]="getPlatformFilter()"
                    [filterParams]="getPlatformFilterParams()"
    >
    </ag-grid-column>
    <ag-grid-column colId="country"  headerName="Country"  field="country"  [width]="120"
                    [cellRenderer]="countryCellRender"
                    [filterFramework]="getCountriesFilter()"
                    [filterParams]="getCountryFilterParams()"
                    [cellStyle]="{'text-align': 'center'}"
    >
    </ag-grid-column>
    <ag-grid-column colId="value"    headerName="Value"    field="value"    [width]="160" filter="text"   [filterParams]="getDefaultFilterParams()"></ag-grid-column>
    <ag-grid-column colId="year"     headerName="Year"     field="year"     [width]="120" filter="number" [filterParams]="getDefaultFilterParams()"></ag-grid-column>
    <ag-grid-column colId="month"    headerName="Month"    field="month"    [width]="120" filter="number" [filterParams]="getDefaultFilterParams()"></ag-grid-column>

</ag-grid-angular>

HI, I can confirm this issue. Having it with ag-grid 8.0.0 then upgraded to 9.0.0 and 9.1.0 but still the same.

Any news about this?

Hi,
No, I'm afraid no updates - we are still unable to reproduce this

Do you know why there is a difference at dragging the column (with / without the dragging rectangle)?
Is it a special mode? Maybe this will be a hint and I could investigate more.

Looking at the head component code we can see the following:
this.eLabel.addEventListener('click', (event:MouseEvent) => {
this.params.progressSort(event.shiftKey);
});
So the grid is looking for click events for sorting - I guess good place to start would be to find out why a click is being sent when the column is being released

Hm, I've nothing like that in my sample. Just a simple grid.

Sorry - thats in the core grid code. Was meant to offer a point of where you could look until we are able to reproduce it ourselves

I've added some debug code and the click handler above is called as soon as the mouse button is released over the header, after dragging the column. But only in the not-working variant.
The working variant displays this rectangle containing the column title. When releasing the mouse button it doesn't affect the column header. I guess that the clicked event goes to the moving rectangle instead of the column header, therefore nothing happens.
The question is why do I not get this rectangle while dragging?
Do you have any hint where to debug?

I also had the same issue in my project. In my case, the problem was with z-index. I increased z-index for ag-dnd-ghost element and problem was gone.

@raivis-v Could you post a code sample how to do this?

Just put this code somewhere in your own css file
.ag-dnd-ghost{
z-index: 5000;
}
Value depends on your actual project and what you have done in it. Could be much higher (or lower) in your case. Assuming you have the same issue I had.

Hm, doesn't work for me. I did put it into the style sheet which belongs to the component where I use the ag-grid. I tried even bigger values, like 99999999 for z-index.

Ok then, it's not your case. Try to do element inspection while dragging column and look for ghost element if it's even there. It should appear as the last element in the DOM while dragging column.

If it is present, I think it should be something wrong with the styles. Try !important for z-index. I can't imagine anything else at the moment.
p.s.
Where did you get the second screenshot (not working one) in your previous posts? Is it from your own project or did you make sample from the scratch?

It seems that there is no ghost element. When I drag a column all that changes is this:

PS: My second screenshot is from our project. The same ag-grid in our production code, not demo.

If the ghost object is the panel which appears normally during column dragging, which lies under the mouse cursor, then it is understandable why the mouse button release event is directed to the column header. Most probably it would have been directed to the ghost object's panel, if it was there. Then everything would be ok.
Question is why the "dragging panel" does not appear at all.

Yes, ghost element is the panel which appears while dragging column. Scroll down your DOM tree to the bottom so that "close body" tag is visible. Ghost should appear right above the close body tag (like in my screenshot). At least it does when using pure js version of the grid.

Wow, it's actually placed at the end of the body!
So I have to place the CSS with the fix into index.html, not into the component. Now it works!
Thanks!
@seanlandsman Maybe it would be a good idea to increase the z-order of ag-dnd-ghost to a bigger value, as for example the 5000. The reason seems to be that we are using a UI framework, which provides windows having a greater z-order than ag-grid's drag n drop ghost.

In my case, I have problem with ag-dnd-ghost too. This element isn't correctly positioned.
My layout is composed by a fixed header and a body with position absolute and its own scroll.

Hi,
If you provide us with a plunkr with a clear scenario where this issue is highlighted we will create a bug ticket for this
Thanks
===============
If you are an ag-Grid Enterprise Customer, please raise this in our Zendesk support system - contact<EMAIL_ADDRESS>for access. This is our primary channel for Support.
Bugs Raised Here:
We will do a periodic review of GitHub issues for any bugs raised and these can be tracked on our pipeline page: https://www.ag-grid.com/ag-grid-pipeline. Note that bugs raised in github will only be processed if they include a plunkr with a reproducible scenario.
Feature Requests/Support Raised Here:
We are reviewing how we process Feature Requests that come from GitHub. We currently have a very full pipeline of work which will be our priority for the coming months.
As a result, we don’t have capacity to take feature requests from GitHub.
We will review this as we move through our current backlog.
Thanks for your understanding.
ag-Grid Team.

I case someone else stumbles across this, we were seeing the same issue until we added
.ag-dnd-ghost{   position: absolute; }


dhcpcd: 9.4.1 -> 10.0.3

Description of changes

https://github.com/NetworkConfiguration/dhcpcd/releases/tag/v10.0.2
Remove upsteamed patch.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 253129 run on aarch64-darwin 1

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2766

@ofborg eval

Does this have any chance of getting merged before the freeze for 23.11 happens? @jiegec

Worth updating to 10.0.5?  https://github.com/NetworkConfiguration/dhcpcd/releases/tag/v10.0.5

https://github.com/NetworkConfiguration/dhcpcd/releases/tag/v10.0.5

10.0.5 has tough bugs that require many patches to fix: https://github.com/NetworkConfiguration/dhcpcd/issues/262

@jiegec Maybe the time is ripe for merging this?

@jiegec Maybe the time is ripe for merging this?

I have no permission to merge.


Switch to use the OSS git-sync with githubapp support

We have been using an internal fork for the githubapp feature. Now this feature is officially included in OSS git-sync v4.3.0. This commit switches to the OSS project for continuous update.
b/376738669

Needs a new LGTM after rebasing @sdowell

/lgtm


Treat *.bat files as valid executables when building on Windows OS

Add new withExtension method version in the FileUtils accepting
an additional vararg array of alternative acceptable extensions to
verify the provided file path against before resorting to replacing
original provided file's extension with the provided replacement
extension
Update OperatingSystem.Windows.getExecutableName to pass .bat
to the FileUtils.withExtension method call as an alternative
acceptable extension in addition to .exe

Fixes gradle/gradle-native#1075 Gradle does not recognize bat files as executables on Windows preventing using drop-in replacements for gcc/clang
Context

Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test this please

OK, I've already triggered ReadyForMerge build for you.

@andreiled Please rebase these changes on master

@Mergifyio rebase

@bot-gradle test this please

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

@JLLeitschuh, looks like the build failed with this exception is the primary cause:
installation("build/install/main", OperatingSystem.forName(installOS)).assertInstalled()
|                                  |               |       |           |
|                                  |               |       windows     java.lang.AssertionError: /mnt/tcagent1/work/e67123fb5b9af0ac/subprojects/platform-native/build/tmp/test files/InstallExec.Test/can_create_._one/5z2pc/build/install/main/lib/main.bat is not a file
|                                  |               |                   	at org.gradle.test.fixtures.file.TestFile.assertIsFile(TestFile.java:429)
|                                  |               |                   	at org.gradle.nativeplatform.fixtures.NativeInstallationFixture.assertInstalled(NativeInstallationFixture.groovy:52)
|                                  |               |                   	at org.gradle.nativeplatform.platform.InstallExecutableIntegrationTest.can create installation for a different os than the current one(InstallExecutableIntegrationTest.groovy:92)
|                                  |               Linux 4.4.0-1052-aws amd64
|                                  class org.gradle.internal.os.OperatingSystem
<org.gradle.nativeplatform.fixtures.NativeInstallationFixture@5a50700c installDir=/mnt/tcagent1/work/e67123fb5b9af0ac/subprojects/platform-native/build/tmp/test files/InstallExec.Test/can_create_._one/5z2pc/build/install/main os=Linux 4.4.0-1052-aws amd64>

For example:

InstallExecutableIntegrationTest.can create installation for a different os than the current one
CppLibraryPublishingIntegrationTest.can publish a library and its dependencies to a Maven repository when multiple target architectures are specified

Assuming this is caused by my change (unfortunately, I do not know the Gradle code good enough neither to guess how nor to argue against it), would you be able to point me to an instruction how I can run failed Integration Tests locally in order to investigate the issue?

@blindpirate  thoughts?

@andreiled Looking at the failing test and your change, it's very likely to be caused by your change. To rerun the failing test locally, you can check out the failing task in the build log, in your case,
Task :platform-native:embeddedIntegTest FAILED

So it's ./gradlew :platform-native:embeddedIntegTest --tests InstallExecutableIntegrationTest, note this is failing on Linux, so you might get a successful build if you run it on Windows.

Thanks for the info, @blindpirate.
If I don't manage to reproduce the issue on Windows, I will try running the test in a WSL container.


Using on CI

In CI, we manually run webpack to compile the assets to public and set config.webpack.dev_server.enabled to false. - How do you set that config on your CI/which CI do you use?

In our test.rb we have the following line:
  config.webpack.dev_server.enabled = !ENV['CI']

And we have export CI=yes at the start of our CI script.
We use the (absolutely amazing) buildkite.com

We also have the following in our CI script to build the assets:
echo "--- compile webpack assets"
mkdir -p public/webpack
TARGET=production ./node_modules/.bin/webpack --bail --config config/webpack.config.js

(technically we could use the rake task to do this, but we don't want to wait for Ruby to start)

Oh, cool. Incidentally I arrived at a near identical solution since posting. Maybe this should be in the readme?

I think you are right! I will leave this issue open as a reminder.


Source code

Implemented source code functionality. ProjectInfoFinder has a new method called retrieveCodeFiles which, for every classPath retrieves the code from that class and stores it as a CodeFile object in the field  codeFiles. These are then written as JSON format to a javascript file called CodeFiles.js (src/main/resources), where it is stored as a variable.
All code is documented and tested :dancer:

🐓 Good job Michiel! I took a quick look and saw that everything is nicely documented. Lets merge this branch before the deadline 🐫

Looks good to me. Nice to see that the source code can now be exported. This is needed for CodeMirror.
Though can you fix the warnings? It went from 9 CheckStyle to 21 CheckStyle warnings?

Decreased to 8 checkstyle warnings now :)

Nice fix for the warnings! I will merge this now.


After using, sponsored language changes to Taiwanese?

Hey, I think after using this library the sponsored ads become Taiwanese or something.
If anyone has the same effect on his/her account, please let me know and I will try to understand if it is only me or it is about the API keys that extracted.
Thanks.

I didn't have this issue.
Let's wait to hear from someone else.

Yeah, I didn't think so, however it's a doubt that I did a rageshake, reported wrong ad language, it turned English, then I used it again and it swapped to Taiwanese again, then I changed whole device info with mgp25's and seems it gets more stable. However still I am not sure about this is about API or friends that I have, so I wanted to ask it to other users. Thank you for labeling as question and have a good day.


add weekly-ci action

Signed-off-by: Evgeny Fedin<EMAIL_ADDRESS>

This action will trigger the CI system to test the role every weekend. It will ensure the role didn't break because of dependencies and other factors while being inactively developed. See https://url.corp.redhat.com/1ab3ab1 for more details

Isn't there any simpler way? One that doesn't involve pushing empty commits and creating empty PRs?

@t184256, I'm afraid, there is no better solution with the existing CI infrastructure.
We use BaseOS CI infrastructure, which runs gluetool as a pipeline system. The gluetool requires a testing artifact which is provided as PR data by the github module (https://gitlab.com/testing-farm/gluetool-modules/-/blob/main/gluetool_modules_framework/infrastructure/github.py#L497). So the first limitation is the PR is required to test the role. Even if it is possible to implement the ability of gluetool test just a main branch, here comes the second limitation. We need to report the results of testing somewhere that it is visible for both Red Hat and external contributors. The most convenient, and the only implemented way, for now, is through PR statuses. Other test systems such as testing farm, packit, and zuul can't provide needed functionality, specifically test roles on all variations of supported composes and supported ansible versions.

the first limitation is the PR is required to test the role

Can it reuse the last closed PR, maybe, if it's pointing at current HEAD? Just brainstorming.

We need to report the results of testing somewhere that it is visible for both Red Hat and external contributors. The most convenient, and the only implemented way, for now, is through PR statuses.

Actions statuses themselves seems like the perfect candidate...
It was just last week when I've set up scheduled pipelines on a couple of my GitLab errors and it has been so straightforward that this proposal to open 0-commit PRs weekly sounds awkward on so many levels in comparison...

Can it reuse the last closed PR, maybe, if it's pointing at current HEAD? Just brainstorming.

No, the GitHub API doesn't allow to set statuses to a closed PR

Actions statuses themselves seems like the perfect candidate...

I failed to find any information about this. I only found docs about commit statuses which Ci currently uses https://docs.github.com/en/rest/commits/statuses#about-the-commit-statuses-api

Can it reuse the last closed PR, maybe, if it's pointing at current HEAD? Just brainstorming.

No, the GitHub API doesn't allow to set statuses to a closed PR

Actions statuses themselves seems like the perfect candidate...

I failed to find any information about this. I only found docs about commit statuses which Ci currently uses https://docs.github.com/en/rest/commits/statuses#about-the-commit-statuses-api

That will involve changes to the gluetool code, which will take some time.  In the meantime, this current approach gives us a lot of useful information now.

Not a fan, but you, gentlemen, seem insistent, so what can I do.

The PR is updated to the latest version @richm


Add --onlyChanges option for tests

Problem
Sometimes I only want to test what I've recently changed and not wait for all the tests to run.
Solution
Add new command line switch, --onlyChanges.  This uses git to check what's changed locally and on your branch.  Any files matching *Test.java (or main classes having a corresponding Test) will be run.  By default, it will attempt gh pr diff to find changes.  If a PR isn't created yet, it will fall back to basically git diff --no-merges origin..HEAD
An additional argument, --commitRange=<commit> can be supplied to tell git diff what to look at.
E.g., sb test --onlyChanges --commitRange=HEAD~3 will look at what changed locally and in the last 3 commits.
Another new option, --keepXML, can be supplied to keep the testNG XML file.  This file can be used in IntelliJ to run the specific test suite that was just run.
Reviewer Notes
This PR change can be run locally be pulling it, running sb int, and then updating your project's build.savant to have:
javaTestNG = loadPlugin(id: "org.savantbuild.plugin:java-testng:2.1.0-{integration}")

If at all possible we should add something to JavaTestNGPluginTest to reflect these new features.

I don't disagree, but I'm not sure how to test this code that depends on the output of git (and gh).


Remove hardcoded --bootnodes in docker-compose file and replace with a script to join the cluster together

--bootnodes='enode://288b97262895b1c7ec61cf314c2e2004407d0a5dc77566877aad1f2a36659c8b698f4b56fd06c4a0c0bf007b4cfb3e7122d907da3b005fa90e724441902eb19e@<IP_ADDRESS>:30303'

is a bit brittle with hard-coded IP address will only work in some situations on boot2docker and fail on docker for mac (for example)

You can assign static ip addresses using docker network capabilities:
   networks:       lotho:           ipv4_address: <IP_ADDRESS> networks:   lotho:     driver: bridge     ipam:       config:         - subnet: <IP_ADDRESS>/24           gateway: <IP_ADDRESS>
I got it running, should I log a pull request ?

Yes please !

Yes please !

This was done in https://github.com/Capgemini-AIE/ethereum-docker/commit/2fc935fc4a1c0d7a034fb8cd06f4991fecc00f44 — this issue can be closed.


[Bug]: some twitter links are inaccessible

Describe the bug
No response
To Reproduce
No response
Expected Behavior
No response
Screenshot/ Video
No response
Desktop (please complete the following information)
No response
Additional context
No response

resolved


Feature Request: Random Photos

I would love the option to randomly shuffle through my entire library, as well as a feature to randomize the order when viewing an album slideshow.

@krissheppard - The Random shuffle used to be a feature but was taken out after some GooglePhotos API changes which made it very difficult to impliment. My hope is to impliment this again at some point.
Regarding "Randomize the Order" during slideshow, this is doable now.
1.) Main Menu -> Settings -> Slideshow Order (this saves on exit)
2.) At runtime -> * Button -> Slideshow Order . (this doesn't save on exit)


fix Received RecordBatch prior to Schema for query

Fix error "Received RecordBatch prior to Schema" at client side when a query is executed.

Close it. Will reopen it when code is ready for review.


cannot load the driver: Invalid parameters while run ./insert.sh

sudo /sbin/insmod gdrdrv/gdrdrv.ko dbg_enabled=0 info_enabled=0
insmod: ERROR: could not insert module gdrdrv/gdrdrv.ko: Invalid parameters
so I tried:
insmod gdrdrv.ko
insmod: ERROR: could not insert module gdrdrv.ko: Invalid parameters
Could you take a look and do a quick fix on it, now it is not working.

@hanyunfan that is strange.
Was that a clean build, i.e. from a pristine directory ?
Which linux kernel ?
Which NVIDIA driver ?
Can you copy here the relevant lines of the system log ? e.g. on RH/Fedora, 'dmesg |tail -20' should be enough.

@drossetti Thank you for your reply.
It was a clean build on RHCE 7.2 64bit with cuda driver 370.25 and 4x P100 GPUs.
It's going to be hard to find the dmesg info after a month, but I will try to reproduct it asap.

The issue was gone.

Hi,
I am hitting the same error. My dmesg error is as follows
[55580.715174] gdrdrv: disagrees about version of symbol nvidia_p2p_get_pages
[55580.715176] gdrdrv: Unknown symbol nvidia_p2p_get_pages (err -22)
[55580.715179] gdrdrv: disagrees about version of symbol nvidia_p2p_put_pages
[55580.715180] gdrdrv: Unknown symbol nvidia_p2p_put_pages (err -22)
[55580.715182] gdrdrv: disagrees about version of symbol nvidia_p2p_free_page_table
[55580.715183] gdrdrv: Unknown symbol nvidia_p2p_free_page_table (err -22)
[66343.379831] gdrdrv: disagrees about version of symbol nvidia_p2p_get_pages
[66343.379836] gdrdrv: Unknown symbol nvidia_p2p_get_pages (err -22)
[66343.379841] gdrdrv: disagrees about version of symbol nvidia_p2p_put_pages
[66343.379842] gdrdrv: Unknown symbol nvidia_p2p_put_pages (err -22)
[66343.379846] gdrdrv: disagrees about version of symbol nvidia_p2p_free_page_table
[66343.379847] gdrdrv: Unknown symbol nvidia_p2p_free_page_table (err -22)
[66394.676111] gdrdrv: disagrees about version of symbol nvidia_p2p_get_pages
[66394.676113] gdrdrv: Unknown symbol nvidia_p2p_get_pages (err -22)
[66394.676117] gdrdrv: disagrees about version of symbol nvidia_p2p_put_pages
[66394.676118] gdrdrv: Unknown symbol nvidia_p2p_put_pages (err -22)
[66394.676120] gdrdrv: disagrees about version of symbol nvidia_p2p_free_page_table
[66394.676121] gdrdrv: Unknown symbol nvidia_p2p_free_page_table (err -22)
[66825.826332] gdrdrv: disagrees about version of symbol nvidia_p2p_get_pages
[66825.826339] gdrdrv: Unknown symbol nvidia_p2p_get_pages (err -22)
[66825.826352] gdrdrv: disagrees about version of symbol nvidia_p2p_put_pages
[66825.826356] gdrdrv: Unknown symbol nvidia_p2p_put_pages (err -22)
[66825.826365] gdrdrv: disagrees about version of symbol nvidia_p2p_free_page_table
[66825.826369] gdrdrv: Unknown symbol nvidia_p2p_free_page_table (err -22)
I am using cuda7.5 on driver version 384 and GeForceGT640 on Ubuntu14.04

@sharan013 what if you make the following change?
diff --git a/gdrdrv/Makefile b/gdrdrv/Makefile
index 32e2a9f..a729acf 100644
--- a/gdrdrv/Makefile
+++ b/gdrdrv/Makefile
@@ -45,6 +45,7 @@ REL_MINOR  := $(word 3,$(REL))
all: module
module:

  @ echo "Picking NVIDIA driver sources from NVIDIA_SRC_DIR=$(NVIDIA_SRC_DIR). If that does not meet your expectation, you might have a stale driver still around and that might cause problems."
  @ $(MAKE) -C $(KDIR) $(MAKE_PARAMS) M=$(PWD) modules

then try to build and install it:
$ cd gdrdrv
$ make clean all
rm -rf .o .ko .mod. ..cmd Module.symvers modules.order .tmp_versions/ *~ core .depend TAGS
Picking NVIDIA driver sources from NVIDIA_SRC_DIR=/usr/src/nvidia-387.00/nvidia. If that does not meet your expectation, you might have a stale driver still around and that might cause problems.
make[1]: Entering directory `/usr/src/linux-headers-4.4.0-45-generic'
CC [M]  /.../gdrcopy/gdrdrv/nv-p2p-dummy.o
CC [M]  /.../gdrcopy/gdrdrv/gdrdrv.o
Building modules, stage 2.
...
$ cd ..
$ ./insmod.sh
INFO: driver major is 238
INFO: creating /dev/gdrdrv inode
$ lsmod|grep gdr
gdrdrv                 20480  0
nvidia              14626816  9 gdrdrv,nvidia_modeset,nvidia_uvm

@drossetti, that patch pointed out that I was picking up old driver sources. Thanks!


Conditional dependecies

Hello,
I'm making an rpm package for an app that depends on ffmpeg,
but since ffmpeg is removed from fedora's default repos, I want to check if the user already has ffmpeg or has access to it from a different repo, and download it,
if NOT then install ffmpeg-free since it does what I need
something similar to Requires: (ffmpeg or ffmpeg-free) in the spec file
I want to do that because if I just download ffmpeg-free regardless, it will conflict with ffmpeg as I tested on my machine,
how do I do conditional dependencies in this situation?

As you know, currently rpm-rs does not support rich/boolean dependency. So it is difficult to support it.
In your case, how about just specify file dependency : /usr/bin/ffmpeg ?

ah, I see,
you mean adding ffmpeg as an asset?
if so, don't I need to add all of it's dependencies too?

welp, after slamming my head for a bit, I opened the rpm file in a hex editor and found out that they just append (ffmpeg or ffmpeg-free) as plain text, so I just needed to do something like this
[package.metadata.generate-rpm.requires]
"(ffmpeg or ffmpeg-free)" = "*"

I intended to do the following:
[package.metadata.generate-rpm.requires]
"/usr/bin/ffmpeg" = "*"

Anyway, your investigation clearly that conditional dependency also supported in current implement. Thank you so much!

no problem! I was even trying to read the source code for rpmbuild to figure out how they store that information, but my very first simple guess was right, but I didn't trust my guts XD
love your crate btw, thank you for making it!


feat: have forge verify-contract take standard JSON input

Component
Forge
Describe the feature you would like
Verification of contracts after some time passes is tricky because the metadata content changes with Foundry updates (e.g. https://github.com/foundry-rs/foundry/issues/8611, but I also noticed it myself). This seems to be a difficult problem to solve globally, so it's better to work it around by keep the standard JSON input generated during the contract deployment. Such JSON can be reliably submitted later, probably using different Foundry version, different tooling and a new scanner. Unfortunately forge verify-contract doesn't accept standard JSON input, it always wants to compile the current sources using its latest logic. This forces the user to use the scanners' web APIs, which accept the JSONs, but are clunky and slow. Using foundry's CLI would be so much move convenient.
Additional context
No response

can you assign this problem to me ?


Fix some comments

Thanks for your contribution @eveneast !
We also have a Discord server, which you’re more than welcome to join. It's a great place to connect with fellow contributors and stay updated with the latest developments!


Fix error in CUBIC_MINIMIZER2 documentation

Thanks for the patch. I've rebased this due to conflicts, so the history looks a bit funny but the patch has landed in master.


Add DNS over TCP tests

Add a new dns-tcp test to the suite that tests DNS proxy forwarding over TCP to external address used in the test suite. No payload protocol is used or tested, so this is a bare DNS test.
Plenty of other test cases cover DNS proxying for UDP, so this new test is specific to TCP.
Flow validation is extended to work on tests without payload protocol and for DNS validation on a UDP or TCP only, or both.

@jrajahalme there is a legitimate failure in the Go static checks: https://github.com/cilium/cilium-cli/actions/runs/7038093191?pr=2135

@jrajahalme there is a legitimate failure in the Go static checks: https://github.com/cilium/cilium-cli/actions/runs/7038093191?pr=2135

Thanks for noting, fixed.

@jrajahalme the newly introduced test seems to be failing in the external workloads CI workflow:
2023-12-01T01:37:10.182657701Z ❌ 1/48 tests failed (2/314 actions), 11 tests skipped, 0 scenarios skipped:
2023-12-01T01:37:10.182662275Z Test [dns-tcp]:
2023-12-01T01:37:10.182676662Z   ❌ dns-tcp/pod-to-world/dns-tcp-google.com: cilium-test/client-75bff5f5b9-mrhr5 (<IP_ADDRESS>) -> google.com-dns-tcp (google.com:0)
2023-12-01T01:37:10.182681730Z   ❌ dns-tcp/pod-to-world/dns-tcp-google.com: cilium-test/client2-88575dbb7-flmx9 (<IP_ADDRESS>) -> google.com-dns-tcp (google.com:0)

@jrajahalme any progress on this? Should we move this PR to draft while you're investigating or just exclude the DNS over TCP tests on external workloads?

@jrajahalme any progress on this? Should we move this PR to draft while you're investigating or just exclude the DNS over TCP tests on external workloads?

I‘ve moved this to draft for now to avoid this PR from showing up in reviewer‘s queues. Please feel free to move it out of draft once the external workloads failure is resolved.


Add probability param for random flip functions.

Based on the conversation in #37 . Will work on adding one or two functions in the upcoming weeks.
Apologies for the extra unnecessary commits, vs studio butcher the notebook and had to revert that one, we can squash it. Happy to update the notebook in a follow up as well.
A few thoughts:

I have not added checks (asserts) on the prob value to allow jitting without needing static args. Let me know if this is okay, Im not super familiar with chex so maybe Im missing something there.
I had to change the way the extra parameters are evaluated in the test to kwargs always. I think this should be fine as long as we dont make some of those parameter positional only. Let me know if you rather take a different approach on this one.
I love how clever the test setup is! 🚀🚀🚀

Feel free to add any comments on the wording of the docstring as well.

Thanks a lot for the review! Let me know if I missed any of the comments

Let me know if there is anything left to iron out :)

Once internal review is passed, the PR will be automatically merged!
It might take a bit of time, but rest assure it will go through 😄
Thanks again for contributing, and feel free to open any other PR! 🚀

@pabloduque0, we have a release freeze in effect for NeurIPS 2022 until end of 2022/05/19.
After that date, the PR will be automatically merged!
Thanks for you patience 😄

@claudiofantacci no worries at all! No rush from my side, after all I have the changes locally to use them :)
Yes, I will look into adding a couple functions in the near future, might follow up on #37 to check with you if they are a good fit or not.
Thanks a lot again for the review and the merge!


HBASE-25685 asyncprofiler2.0 no longer supports svg; wants html

Suggestion for 2.3 and 2.4.... just make it so can pass 'output=HTML'.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 43s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
3m 40s
branch-2.3 passed

+1 :green_heart:
compile
0m 21s
branch-2.3 passed

+1 :green_heart:
shadedjars
5m  4s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 18s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 19s
the patch passed

+1 :green_heart:
compile
0m 21s
the patch passed

+1 :green_heart:
javac
0m 21s
the patch passed

+1 :green_heart:
shadedjars
5m  2s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 48s
hbase-http in the patch passed.

20m 56s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3078/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3078

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 4c0bbef3b737 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / b6d4c9e114

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3078/1/testReport/

Max. process+thread count
539 (vs. ulimit of 12500)

modules
C: hbase-http U: hbase-http

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3078/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
3m  6s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
5m 36s
branch-2.3 passed

+1 :green_heart:
compile
0m 26s
branch-2.3 passed

+1 :green_heart:
shadedjars
8m  6s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 25s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m  0s
the patch passed

+1 :green_heart:
compile
0m 25s
the patch passed

+1 :green_heart:
javac
0m 25s
the patch passed

+1 :green_heart:
shadedjars
7m 52s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 22s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 53s
hbase-http in the patch passed.

33m 27s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3078/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3078

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux cd6dc93326f9 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / b6d4c9e114

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3078/1/testReport/

Max. process+thread count
322 (vs. ulimit of 12500)

modules
C: hbase-http U: hbase-http

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3078/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m  2s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
4m 21s
branch-2.3 passed

+1 :green_heart:
compile
0m 36s
branch-2.3 passed

+1 :green_heart:
checkstyle
0m 17s
branch-2.3 passed

+1 :green_heart:
spotbugs
0m 42s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 44s
the patch passed

+1 :green_heart:
compile
0m 34s
the patch passed

+1 :green_heart:
javac
0m 34s
the patch passed

+1 :green_heart:
checkstyle
0m 13s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
19m 38s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
0m 49s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

41m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3078/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3078

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux b81640b4aea9 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / b6d4c9e114

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
96 (vs. ulimit of 12500)

modules
C: hbase-http U: hbase-http

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3078/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


LFS support with GitHub Actions?

The problem
For some reason I can not get the python-semantic-release github action to work on my main branch. My workflow looks like this:
name: Semantic Release

on:
  push:
    branches:
      - main

jobs:
  release:
    name: Publish the package
    runs-on: ubuntu-latest
    if: github.repository == 'H0R5E/SNL-Delft3D-CEC-Verify'
    steps:
    - uses: actions/checkout@v2
      with:
        fetch-depth: 0
        lfs: 'true'
        ref : 'main'
        token: ${{ secrets.ADMIN_TOKEN }}
    - name: Expose git commit data
      uses: rlespinasse/git-commit-data-action@v1
    - name: Wait for tests to succeed
      uses<EMAIL_ADDRESS>      with:
        ref: ${{ github.ref }}
        running-workflow-name: 'Publish the package'
        repo-token: ${{ secrets.GITHUB_TOKEN }}
        wait-interval: 10
      if: env.GIT_COMMIT_COMMITTER_NAME != 'github-actions'
    - name: Set up Git-LFS
      run: |
        sudo apt-get update && sudo apt-get install git-lfs
      if: env.GIT_COMMIT_COMMITTER_NAME != 'github-actions'
    - name: Python Semantic Release
      uses: relekang/python-semantic-release@master
      with:
        github_token: ${{ secrets.ADMIN_TOKEN }}
      if: env.GIT_COMMIT_COMMITTER_NAME != 'github-actions'

And my configuration in pyproject.toml looks like this:
[tool.semantic_release]
branch = "main"
version_pattern = [
    'setup.cfg:version = (\d+\.\d+\.\d+)',
    '.conda/recipe/meta.yaml:{{% set version = "(\d+\.\d+\.\d+)" %}}'
]
commit_parser = "semantic_release.history.emoji_parser"
upload_to_repository = "false"

But, when I run the job it tries to get the main branch again and errors:
...
debug: Running publish on branch main
debug: checkout(main)
error: Cmd('git') failed due to: exit code(128)
error:   cmdline: git checkout main
error:   stderr: 'git-lfs filter-process: 1: git-lfs: not found
error: fatal: the remote end hung up unexpectedly'

Expected behavior
I don't understand why python-semantic-release is trying to check out the main branch again. I wondered if it was an issue with LFS, but installing LFS in the runner doesn't seem to make it available to python-semantic-release.
Additional context
If I take away the branch variable in the configuration then it tries to check out master:
debug: Running publish on branch master
debug: checkout('master')
error: Cmd('git') failed due to: exit code(1)
error:   cmdline: git checkout master
error:   stderr: 'error: pathspec 'master' did not match any file(s) known to git'

Are you sure your container has git-lfs installed? Judging from the error message it has not.
Perhaps this might fix your builds: https://stackoverflow.com/a/66453723/2186184

@jmartens, I'm not using any containers. The problem is with the python-semantic-release container, no?


Adds default split for libri-train-clean-100

[ ] Tests added / passed
[ ] Associated doc strings if new function
[ ] Passes flake8

Actually, @mlomnitz please add pandas and the version you're using to the requirements file so that your changes can be used in the developing api.

Good catch! Got it


Fix pwl check

corrects the pwl check to verify convexity.

@sourabhdalvi this doesn't resolve yet the implementation of the limits for the units under PWL costs.

I’m not convinced we should do what we discussed on the phone. If we do, we’ll either have to multiply by commitment status or we would be adding the P=0 cost when the unit is off. This way only fails when the 0th traunch is steeper than the others, and otherwise gets the right objective function. It will work in the other case, it just adds sos.

I’m not convinced we should do what we discussed on the phone. If we do, we’ll either have to multiply by commitment status or we would be adding the P=0 cost when the unit is off.

It will imply that a Gen formulated as no-min is always on but has no minimum stable limit.

This way only fails when the 0th traunch is steeper than the others, and otherwise gets the right objective function. It will work in the other case, it just adds sos.

I don't think is accurate to say that if a unit formulated as no-min then the cost = 0 when output = 0. This case will then imply that "almost" by default no min + pwl cost will imply an SOS cost function thus defeating the purpose of having simple linear model to get fast and dirty solutions.


How to call BYO Function API using Postman when AAD Auth is enabled

Hi,
I am using a BYO Powershell function and have turned on AAD auth for my SWA. When using the browser and accessing my SWA I get the AAD login page and can login and authenticate fine, once logged in my SWA can call my function and all works great. I'd like to be able to call my function using postman however Postman keeps responding with a Sign In page. I am able to get a beaker token successfully via postman however it doesn't seem to help. Any help would be appreciated! Thanks

same problem here, if I try to autheticate with Bearer authentiaction I always get permission denied.

It looks like that (I cuted of my bearer because it was still vaid)


Access to FAST5

Hello!
Would there be any way to access the raw data, in FAST5 format?
Thanks a lot for providing such an amazing dataset!

we are in talks with AWS representatives to have the fast5's hosted there. I will let you know as soon as I have more information.
S

Great! That's really good news to hear!  Have a great day!

I suggest to keep this open until the data is up.

Hello @tobiasmarschall, just so we can maybe close that thread and maybe if you would want to add a supplementary link in your github repo, I saw that there is a new link on your preprint. Raw files seems available on AWS :
https://s3.amazonaws.com/1000g-ont/index.html?prefix=pod5_data/
Can you confirm this is it?
Thanks a lot!


webpackConfig() plugins are merged before the default config

Laravel Mix Version: 5.0.3
Node Version (node -v): 12
NPM Version (npm -v): n/a
OS: n/a

Description:
vuetify-loader expects to be initialised after vue-loader, but mix.webpackConfig() applies the user plugins first so VueLoaderPlugin fails due to the use of oneOf (https://github.com/vuetifyjs/vuetify-loader/issues/144)
Steps To Reproduce:

laravel new test
yarn add --dev vuetify vuetify-loader
Set webpack.mix.js to:

const mix = require('laravel-mix');
const VuetifyLoaderPlugin = require('vuetify-loader/lib/plugin');

mix.js('resources/js/app.js', 'public/js')
    .sass('resources/sass/app.scss', 'public/css')
    .webpackConfig({
        plugins: [
            new VuetifyLoaderPlugin({
                options: {}
            }),
        ]
    });

run yarn dev; it will fail

Try upgrading to the current version of mix (5.0.4 for me), I'm basically using the same setup and it works fine. If that's not it, try putting .webpackConfig first and see if that changes anything.

Are you using vuetify-loader? It needs to run after VueLoaderPlugin so we can use oneOf

Yes, but despite updating it seems I'm still on 1.5 which apparently doesn't have the issue.
But my final plugin list doesn't even include vue-loader, so Mix doesn't seem to inject it at all for me. Maybe check if it even is in your plugin list (I use Mix.listen('configReady', config => {}), perhaps you can just load it manually before vuetify-loader?

This mix extension should work: https://github.com/Nothing-Works/vuetifyjs-mix-extension
Or you can create your own:
mix.extend('vuetify', new class {
    webpackConfig (config) {
        config.plugins.push(new VuetifyLoaderPlugin())
    }
})

The problem is that the vue extension uses webpackConfig: https://github.com/JeffreyWay/laravel-mix/blob/master/src/components/Vue.js
Given the following config we can see that extension.webpackConfig is called after mix.webpackConfig and configReady, in fact mix.webpackConfig is applied before any extensions.
mix.extend('extension', new class {
    webpackConfig (config) {
        console.log('extension webpackConfig')
    }
    webpackRules () {
        console.log('webpackRules')
    }
    webpackPlugins () {
        console.log('webpackPlugins')
    }
})

mix.js('resources/js/app.js', 'public/js')
    .sass('resources/sass/app.scss', 'public/css')
    .extension()
    .webpackConfig(() => console.log('webpackConfig'))

Mix.listen('configReady', config => {
    console.log('configReady')
})

if mix.webpackConfig ran after all extensions and had access to the config object then this wouldn't be a problem.

Ok, that's bad. One could argue about .webpackConfig, but configReady definitely should run last (or there should be another trigger that does)

Just bumping this as I'm having the exact same issue.
Had to use this as a work around: https://github.com/vuetifyjs/vuetify-loader/issues/144#issuecomment-659308887
Thanks @KaelWD


BaseArrayHelper removeValue enhancement

What steps will reproduce the problem?
Create two arrays with string values. The second one should be a subset of the first one, i.e.:
$arrayOne = ['one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine', 'ten'];
$arrayTwo = ['three', 'five', 'seven'];

Call ArrayHelper::removeValue($arrayOne, $arrayTwo);
What is the expected result?
$arrayOne should look like ['one', 'two', 'four', 'six', 'eight', 'nine', 'ten'];
What do you get instead?
$arrayOne is untouched.
The suggestion is to enhance the removeValue method to accept an array of values, or to create a new method removeValues which will do the same. My opinion that semantically it is better to create a new method, but that is only my opinion.
For now I have modified an existing method removeValue, and it does the trick
    public static function removeValue(&$array, $value)
    {
        $result = [];
        if (is_array($array)) {
            foreach ($array as $key => $val) {
                if ($val === $value || (is_array($value) && in_array($val, $value))) {
                    $result[$key] = $val;
                    unset($array[$key]);
                }
            }
        }

        return $result;
    }

Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.2.7

Operating system
CentOS 7 x64

How this would differ from array_diff()?

@rob006 I thought that too but then realised ArrayHelper::removeValue() modifies the original array and returns a new one containing the values that were removed.

and returns a new one containing the values that were removed

array_intersect()
I'm not sure if we need a helper which will be wrapper for 2 native PHP functions. It would make sense as readability improvement (I guess combination of array_intersect() and array_diff() is not obvious), but for me ArrayHelper::remove* methods were never intuitive, so this will actually decrease readability IMO.

So finally should I create a pull-request for this or just close the Issue?


Admin Login Panel Customization

I looked around but didn't see anything in the docs regarding the admin login panel - is it possible to customize (or override) the login UI, including the logo? Apologies if I missed it.

Yes. There are two main ways depending on how you wish to customize it. The admin panel itself is an Enduro app so you can head to its github and check it out. There are instructions on how to npm link it so that while running it separately your current project uses it.
The other way is to use the admin extensions functionality. Inside the example app you can see an example of a js file inside assets/admin_extensions that gets called on the admin. This would be useful if you just need to override something or add something small.


how to get a Cartesian product

Please make sure that the boxes below are checked before you submit your issue.
If your issue is an implementation question, please ask your question on StackOverflow or on the Keras Slack channel instead of opening a GitHub issue.
Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[x] Check that your version of TensorFlow is up-to-date. The installation instructions can be found here.

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

I need a keras layer to do such calculation:
def f( [a, b], [c, d] ):
return [ac, ad, bc, bd]
what is the easiest way?

use https://www.tensorflow.org/api_docs/python/tf/math/multiply

https://www.tensorflow.org/api_docs/python/tf/math/multiply

that would rather return [ac, bd].
matrix multiplication and extending rows there after should help.
[a, b].T  X [c, d] that returns a matrix of shape(2,2)
X stands for matrix multiplication

https://www.tensorflow.org/api_docs/python/tf/math/multiply

that would rather return [ac, bd].
matrix multiplication and extending rows there after should help.
[a, b].T X [c, d] that returns a matrix of shape(2,2)
X stands for matrix multiplication

thanks. I use K.batch_dot( [[a], [b]], [c,d] ) to do this. is it the easiest way?


add new gene

I tried to do another commit on this PR after I added these two files to import the new gene, and the new commit does not show up here. I reverted it in GitHub desktop but I suspect there may be errors to merge this branch now, like in https://github.com/monarch-initiative/mondo/pull/7397
It seems there is some issue with doing commits after running the import new term pipeline: https://mondo.readthedocs.io/en/latest/editors-guide/import-terms-for-logical-axioms/#preferred-instructions
which I do not understand at all
@sabrinatoro @matentzn @twhetzel

I want to add this term and use the gene in the subclassOf axiom
[Term]
id: MONDO:0100551
name: SPATA5L1-related complex neurodevelopmental disorder with motor features and hearing loss
def: "A neurodevelopmental disorder related to biallelic variants in SPATA5L1 and characterized by a spectrum of intellectual disability, hearing loss, and motor features including spasticity, dystonia, and/or hypotonia. Other phenotypic features commonly reported with the neurodevelopmental presentation include spasticity, focal or generalized epilepsy, and microcephaly." [https://clinicalgenome.org/affiliation/40006/]
synonym: "AFG2B-related complex neurodevelopmental disorder with motor features and hearing loss" EXACT [https://orcid.org/0000-0001-5208-3432]
is_a: MONDO:0100038 {source="https://clinicalgenome.org/affiliation/40006/"} ! complex neurodevelopmental disorder
property_value: http://purl.org/dc/terms/creator https://orcid.org/0000-0001-5208-3432
property_value: IAO:0000233 "https://github.com/monarch-initiative/mondo/issues/7317" xsd:anyURI

closing duplicate of https://github.com/monarch-initiative/mondo/pull/7399


EHPR-108 | Fix DB connection issue

[x] Uncouple nest application config code from bootstrap code

I'd be surprised if changing the file would work. I was thinking in the line of our TF config. Maybe VPC?

If the file main->config file change isn't needed lets revert it

@arranfw  There are multiple failure on the code de-coupling solve one of them.

If the file main->config file change isn't needed lets revert it

@arranfw There are multiple failure on the code de-coupling solve one of them.

can you please list them and what each one solves?

@hitankar-freshworks @vyasworks @arranfw @dbayly-freshworks Please check my comments on code.


Templating & content for all types

Discussed in https://github.com/house-of-abbey/GarminHomeAssistant/discussions/148
Would it be possible to add templating possibilities for all types. Let's say I have a group, there are multiple lights in it. On the top view (when navigating items) I would like to have the "content" so I can check my custom state.

We likw this idea and see how it could be done. Just need to do it now!

Sorry if this seems like an obvious question, but how would you use a toggle with a template text?

I can think of some (but will be specific to my smart home/plans):

you would want to see brightness level of a light
see number of people in your house when arming alarm system
see humidity level in your garage when turning on exhaust fan


Added icon for Laravel artisan file

Added icon & file association for Laravel artisan file.
Request: #29
I used new icon instead of blade icon to avoid confusion.

Hey @mallowigi, any feedback?

@all-contributors please add @amstiel , @aparajita , @ihorvorotnov  for code and ideas


Infer lambda return types

Alternative to #7741

Motivation

This doesn't rely on a large, hairy, tricky-to-get-right addition to
guessOverload, making it less brittle and faster at the expense of
maybe being less principled.
Commit summary
The original implementation in this PR involved making Kernel#lambda-specific changes to cfg::BlockReturn in processBinding which worked but felt hacky.
I've since realized that it's possible to write that code directly into the Kernel#lambda intrinsic that we already have.
Test plan

[x] TODO @jez write tests

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_QNWlKXTMlNJUR9
→ https://go/builds/bui_QNWlZUYCQu4jdd
→ https://go/builds/bui_QNWl0a9cwdPSQ3

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_QNY0LFICG5cgGL
→ https://go/builds/bui_QNY0Pn1tmrqY7r
→ https://go/builds/bui_QNY0PZiDmMkoad

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_QOhuOGx134VZ3g
→ https://go/builds/bui_QOhujeZ79LuCs4
→ https://go/builds/bui_QOhuD9Ders3hkJ


Publish typescript types

Would you be willing to publish typescript typings? Looks like someone already set some up for factory-girl (https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/factory-girl) so it might not be too much work to bring them into this package.

Added in v0.6.1


How to improve e4e inversion

Hi, first of all thanks your your amazing work :)
Is there a way to improve e4e image inversion (e.g. increasing some kind of iteration parameters?). I've tried with some faces and sometimes the inverted result has little resemblance with original one...
many thanks in advance

Hi @smithee77 ,
I am happy to hear that you liked our work 😊.
Image inversion is a quite active research area and in the past year, there are quite a lot of works on this subject.
I recommend you to take a look on ReStyle, which is a newer work with more accurate results.
Or

thank you very much! definitely I'll take a look and try to figure out how to use ReStyle with your CLIP notebook :)


Allow model to set table_name after acts_as_paranoid macro

I originally opened #117  without tests, and since then have deleted my fork, so reforked and opened this PR with a spec @mvz
Thought about also asserting that ParanoidWithExplicitTableNameAfterMacro.only_deleted is working since that's how I found the error, but I think this test is clearer what exactly we're testing. Lemme know what ya think

paranoid_column_reference is currently a class attribute which is set
when a class is loaded and the acts_as_paranoid method is invoked. A
rails model can set its table_name after it declares itself as
acts_as_paranoid. This means that whenever paranoid_column_reference is
referenced in the gem it does not match the table_name for the model
which can cause errors.

e.g. ActsAsParanoid::Core#only_deleted references
paranoid_column_reference so in the below examples when we call this
method on the paranoid class it tries to look up a table_name matching
the model name. If the table doesn't exist a sql error is raised because
the table does not exist. If the table does exist (maybe you're moving
data to a new table and deprecating old ones) you'll end up returning
the old records.

This change makes it a method so that it returns whatever the current
value of table_name is

  # current incorrect behavior
    class User
      acts_as_paranoid
      self.table_name = "Account"
    end

    User.paranoid_column_reference
    => "user.deleted_at"

   # new correct behavior
    class User
      acts_as_paranoid
      self.table_name = "Account"
    end

    User.paranoid_column_reference
    => "account.deleted_at"

Thanks, @AlexWheeler!


creating JSON dump no longer working

JSON dump broken after f35b757c826027ab5263d431bbe363c6403bd66d, f35b757c826027ab5263d431bbe363c6403bd66d is the last working commit, after this commit no working JSON dump can be created.
ytdl = YoutubeDL({'nocheckcertificate':True, 'restrictfilenames':True, 'no_warnings':True, 'quiet' : True, 'print_json': True, 'extract_flat': True})
results = ytdl.extract_info(sys.argv[1], ie_key="Youtube", download=False, process=True)
print json.dumps(results)
This worked before but now no longer is working

Thanks for the report, but it's missing a verbose log showing the error (see #30839, Bugs FAQ).
Please ensure you can reproduce on the latest master code, as several issues related to JSON output were squashed after 88f28f62.

Ok thats odd you are right calling youtube-dl -v -j '15XDprAqgDs' does indeed dump proper JSON data, but calling the python function does not see https://gist.github.com/dhwz/221cd0f639fc8e4f9b181bb8e1135de1
FYI thats on Python 2.7.x (if it's relevant).

Maybe you have updated the code in the Python3 directory but not in the Python2 directory? Both are good for me, but I'm using the code directory, same for both 2 and 3:
$ python -m youtube_dl -v -j 'TlkLD0flJpw' | jq '.id'
[debug] System config: [u'--prefer-ffmpeg']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-j', u'TlkLD0flJpw']
[debug] Encodings: locale UTF-8, fs UTF-8, out None, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: 9024d45f1
[debug] Python 2.7.18 (CPython 32bit) - Linux-4.4.0-210-generic-i686-with-Ubuntu-16.04-xenial - OpenSSL 1.1.1t  7 Feb 2023 - glibc 2.15
[debug] exe versions: avconv 4.3, avprobe 4.3, ffmpeg 4.3, ffprobe 4.3
[debug] Proxy map: {}
[debug] Default format spec: bestvideo+bestaudio/best
"TlkLD0flJpw"
$

No there is no Python3 on the system I'm having that issue on.
As said I do agree it's working if I call the youtube_dl "binary" directly, but I'm importing youtube_dl into my own application and calling the youtube resolver, thats where I get the problem now.
Maybe you can verifiy with my small example:
# -*- coding: utf-8 -*-
import sys, json
from youtube_dl import YoutubeDL
from youtube_dl.utils import DownloadError
try:
	if 'playlist' in sys.argv[1]:
		ytdl = YoutubeDL({'nocheckcertificate':True, 'restrictfilenames':True, 'no_warnings':True, 'quiet' : True, 'simulate' : True, 'print_json': True, 'extract_flat': True, 'list_thumbnails': True})
		results = ytdl.extract_info(sys.argv[1], ie_key="YoutubePlaylist", download=False, process=True)
		print json.dumps(results)
	else:
		ytdl = YoutubeDL({'nocheckcertificate':True, 'restrictfilenames':True, 'no_warnings':True, 'quiet' : True, 'print_json': True, 'extract_flat': True})
		results = ytdl.extract_info(sys.argv[1], ie_key="Youtube", download=False, process=True)
		print json.dumps(results)
except DownloadError as e:
	print str(e)

Your symptoms are consistent with your configuration having a newer single file yt-dl "binary" together with an older yt-dl installed as a Python module.

Yes but I don't have a single file binary.
My site-package is up-to-date matching the latest commit on the repository.
You're still insisting I'm doing something wrong but I don't :-)

OK, I think I can see what's happening here.
With the new code you can't just print(results). If you want JSON sent to standard output, use the forcejson (or some similar) API option, but not print_json: it's not a valid option (see YoutubeDL.py ll.150 on). To get JSON and no download, don't set the download argument of extract_info(): use the simulate option.
from youtube_dl import YoutubeDL
ytdl = YoutubeDL({'simulate': True, 'quiet' : True, 'forcejson': True})
# JSON sent to stdout here
results = ytdl.extract_info('TlkLD0flJpw')
# to `print()` the return value of `extract_info()` (duplicate JSON)
print(json.dumps(ytdl.sanitize_info(results)))

Ok, great that did the job.


Create access cable component and its tests

Coverage increased (+4.4%) to 22.664% when pulling ce44f5d5204f9a4d36a9713a3d77e79010af88f2 on US11-Manter_Cabos_Acesso into 084c89424d2ec15ac918806a5a53848d60ac5ee3 on master.


Add new bot responses

Add bot responses to bot_responses/bot_responses.json

@Jac08H Working on it!

@Jac08H Hi.  I would like to go ahead with this :)

@Jac08H  This is open right ? Or this is no more an issue ?


[Question] Embedded async relationship with Symmetric Multiprocessing (SMP)?

It's a open question.
I found this Symmetric Multiprocessing (SMP) with FreeRTOS
Just wondering deeper insight form you.

SMP is totally supported—one of embassy's supported chips, the RP2040, operates in SMP with dual cores. Here's an example of activating the second core with embassy: https://github.com/embassy-rs/embassy/blob/main/examples/rp/src/bin/multicore.rs

I don't believe Embassy has any sort of multicore scheduler though—you spawn tasks on a specific core and they will never switch, so tasks on the same core will never run in parallel. Tasks running on different cores will run in parallel, though.

Small clarification on the terms used. I think embassy does not support SMP (S = Symmetric), however it supports AMP (A = Asymmetric).
The difference being that in SMP any processing unit takes work from a shared queue (as is the case in FreeRTOS SMP), while in AMP each processing unit has a predefined list of tasks that it may execute.
I guess that you could implement something like having a few executors that are shared between cores, that you then .pool() whichever executor has work to do from whichever core is free, which would achieve SMP, but it's not build-in, I don't think.


Errors related to "unexpected key: hl_name"

Problem
When editing Lua or JavaScript files, I'm constantly seeing errors related to indent-blankline.nvim/lua/ibl/inlay_hints.lua:17:

I haven't seen this error while editing C# files.
Steps to reproduce
return {
	"lukas-reineke/indent-blankline.nvim",
	main = "ibl",
	opts = {},
	config = function(_, opts)
		require("ibl").setup(opts)
	end,
}

Expected behavior
The error is not present.
Neovim version (nvim -v)
0.8.3 release

Your Neovim version is too old. Please update to at least the latest stable version, which is v0.9.4 currently.


CM-414: Created Containerfile for istio-csr with redhat base image.

Manual cherry-pick of https://github.com/openshift/cert-manager-operator-release/pull/94
/assign @anirudhAgniRedhat

Added submodule for istio-csr
Added containerfile for istio-csr

Changes are not exact cherry-pick of the master commit.

/lgtm

/approve


Ir xor generate examples

bsiyer created the following on May 04:
Script and instructions to generate additional XORTyDI training examples through BM25 retrieval
Original issue: https://github.ibm.com/ai-foundation/PrimeQA/pull/172

closing this as the original issue was marked complete


Add PromptString

Part of #3642
We're introducing a new concept of PromptString into the Cody code base. The goal of this is to create a safe way for us to concatenate prompts while still knowing what context was referenced along the way. This will give us peace of mind when adding features like Cody Ignore since we can guarantee that the prompts we send to the backend LLMs do not accidentally contain blocked files.
What changes for me?
When working on prompts in Cody, you can no longer concatenate strings and call it a day. Instead, you will have to use our PromptString utilities. We tried to make this experience as smooth as possible and added a lot of helper methods.
For example:
// Old and unsafe:
const prompt = `Please rewrite this code: ${document.getText(range)}`

// New and will retain a reference to the document.uri that was interpolated:
const prompt = ps`Please rewrite this code: ${PromptString.fromDocumentText(document, range)}`

You can create PromptStrings from other PromptStrings, literals, and use string helper functions just like you did before. Here’s an example from the newly added unit tests:
const s = ps`  Foo${ps`bar`}baz  `
expect(s.toString()).toBe('  Foobarbaz  ')
expect(s.length).toBe(13)
expect(s.slice(1, 3).toString()).toBe(' F')
expect(s.trim().toString()).toBe('Foobarbaz')
expect(s.trimEnd().toString()).toBe('  Foobarbaz')
expect(s.indexOf('Foo')).toBe(2)
expect(s.indexOf(ps`Foo`)).toBe(2)
expect(s.toLocaleLowerCase().toString()).toBe('  foobarbaz  ')
expect(s.includes('Foo')).toBe(true)

A few of the helper functions are prefixed with and considered  unsafe_ since we can't fully track the references in all cases (e.g. when a user copy-pasts something into the text box). @dominiccooney and me are tracking usages of these APIs and will work on reducing the unsafe surface (e.g. we want to make it safe to serialize/deserialize prompt string).
Let us know if you have any issues with this when working in your code.
Follow-up

[ ] Revisit the type for storing references. Should we use a tree of frozen arrays?
[ ] Handle serialization of chat transcriptions with a resolved reference.

Test plan

[x] Test Chat/Edit/Autocomplete
[x] CI passes
[x] New unit tests for prompt string
[x] TypeScript passes

testing @sourcegraph/cody-security

@dominiccooney @willdollman it might get a bit spammy for you guys now, I promise I will keep it as minimal as possible 🥺

@pkukielka  There are some renames in the kotlin typegen. Can you check if this is an issue of the JetBrains client? The actual types are kept the same (so the API will still expose prompts as strings when talking to the agent).

@dominiccooney This is almost done. I need to make the CI pass again and then document the follow-up tasks in your GitHub issue and "arm" the CI rule again (I temporarily disabled it). I also want to remove some of the TODOs from the prompt-string class as well (i.e moving the types). So I'll likely land this Monday my morning if that's alright.

Let's goooo

@philipp-spiess Noticed a weird regression from this commit:
https://github.com/sourcegraph/cody/assets/9516420/9ff34037-55b2-4923-bcf5-aa4293465b8f
Seems like the "New Chat" command is automatically executing a chat

@umpox thanks will fix asap, I’m sure it's a string coercion bug :(

@umpox https://github.com/sourcegraph/cody/pull/3811


TypeError: Cannot read properties of undefined (reading 'isSupported') on Flutter Web

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
I am trying to initialize the VideoPlayerController in Flutter Web but everytime I load the screen it ends up with this error
 TypeError: Cannot read properties of undefined (reading 'isSupported')
Note: I tried different browsers and I got the same error in all of them.
Packages I use in Pubspec.yaml file:
video_player: ^2.7.1
video_player_web_hls: ^1.1.0
Here is how I initialize the controller:
//Code
_VideoPlayerController = VideoPlayerController.networkUrl(Uri.parse(streamUrl))..initialize().
		  then((_) {
            print("initialized");
            // Ensure the first frame is shown after the video is initialized, even before the play button has been pressed.

            _controller!.play();
            setState(() {});
          }).onError((error, stackTrace) {
			print("====================================================");
			print(error);

            setState(() {});
          });
//End of code

Expected results
It should load the video stream and plays it on web.
 Please report issues with third-party packages in their respective repositories.
The flutter/flutter repository is only for packages under flutter/packages.
However, the details of the stack trace led me to a JS interop definition in https://github.com/balvinderz/video_player_web_hls/blob/master/lib/hls.dart#L8
In their setup, it is documented that you should add ```

``` in your index.html which would explain why the JS interop API fails to bind to Hls.isSupported.

As noted above, this should be reported video_player_hls repo.
Closing this as the error is from a third-party package.


defmt-macros depends on proc-macro-error which is considered unmaintained

I'm using defmt and thus defmt-macros. I've recently got a security warning because I depend (through defmt) on proc-macro-error. See https://rustsec.org/advisories/RUSTSEC-2024-0370.html for more details. Would it be something you would consider to either migrate to proc-macro-error2 or avoid depending on proc-macro-error through another mean?

Thank you for notifying us @ia0.
Either using proc-macro-error2 or just using syn::Result seem like viable options to me.
Would you be interested in making either change?

Thanks for the answer.
This is not a big issue on my side so far (unmaintained crates don't really have an urgency aspect as other security issues may have), so I won't be able to allocate time for it, sorry.


[Documentation] - We need thorough documentation on README or Wiki

As title states :-) to build support for this community we need this

@ZkHaider which parts of the Fluent Postgres Driver are you wanting to document? For this repo, all the docs should be at https://docs.vapor.codes

+1 is not enough
Need more examples of how it can be used without vapor


Android Basics: change-app-theme - Night mode not show right color

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-change-app-theme?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-change-app-theme#4
In which task and step of the codelab can this issue be found?
task 5
Describe the problem
I choose night mode in editor

it show like this, not the green light color

Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

it seem show the default color


Question: How to programmaticaly add correct href paths?

Hi, how would I add the correct SVG path when using native JavaScript?
This won't replace the correct path:
const svg = document.createElement('svg');
const svgUse = document.createElement('use');
svgUse.setAttribute('href', '/__spritemap#sprite-comment');

svg.appendChild(svgUse);

Thank you!

Seems I need to use createElementNS with the correct SVG namespace if there is no other way provided by the plugin.

Hi, the plugin doesn't provide any kind of API to manage SVG with javascript. I didn't test it with this kind of usage in mind. Maybe in a future feature?


Make indexers slaves of web's redis

Filter data out on indexer by hitting local redis

Done (infrastructure change)


sisco.lv2: init at 0.7.0

Motivation for this change
I needed sisco.lv2 myself for analyzing waveforms.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Also added maintainer e-user

That change didn't go as smooth at it ought to be, but there you go.

The CI error looks completely bogus to me.

Almost there!

What's left to do, now?

All good now :)


StandardCamera to Suggestions to ObsEdit back to Sugggestions traps you

Describe the bug
When you take a new photo with the StandardCamera, choose a taxon on Suggestions, tap the back button on ObsEdit to get back to Suggestions, and then tap the back button from Suggestions, you end up on ObsEdit again.
To Reproduce
Steps to reproduce the behavior:

Take a new photo with the StandardCamera
Choose a taxon on Suggestions
On ObsEdit, tap the back button in the upper left of the header
On Suggestions, tap the back button in the upper left of the header
You end up back on ObsEdit

Expected behavior
When you tap back from Suggestions, you should return to the StandardCamera, and any photos you took should remain in the preview area.
Context (please complete the following information):

Device: iPhone SE (3rd generation
OS: iOS 17.5.1
App version: 0.34.0 (97)

Additional context
The intended flow is diagrammed at https://www.figma.com/board/gzrhrvjMvaL5aFvgqnVtZ9/iNat-Mobile-Flows?node-id=29-4740&t=oqfZ7iLdS2Zanudf-0

I can't replicate this, even on 0.34.0 (97), even though I filed this 2 days ago and could replicate it then.


feat(resolver): add resolver for layui-vue

support for layui-vue

Please fix the linter

Please fix the linter

sorry, i fixed it.


[Bug] Command being shared all field variable across session

Describe the bug
Command being shared all field variable across session made Pudel no respond
To Reproduce
p!play aaaaaa
1
p!play sdsadsad (This one cause bug no respond)
Expected behavior
perform searching another song
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Windows

Resolve by this commit of CommandManager to perform new instance every time when session was created
https://github.com/Zazalng/Pudel/commit/d8ac063e8eb964d6f87173c6628c5d68a20bafc0


fix(sector): skip drawing corner radius if no arc.

When the corner radius is specified but the angle of sector is smaller than 1e-4, we should not draw the corners.
Otherwise, we will get an unexpected result like the following screenshots.

@pissang Please help review this.

LGTM


[Bug] Controller Not Recognized After 2.1.1 update (MacOS Sonoma 14.0)

Describe the bug
I have been using previous versions without issue (after enabling input monitoring for the Greenlight app), up until today when I installed the latest update.  Now my controller (Xbox Elite Series 2) is not being recognized by the Greenlight app.
It's successfully paired to the computer (pressing the xbox button opens the "Launchpad" on Mac) however it does not appear in the inputs in Greenlight.  I also started a stream just in case, and confirmed the controller did not work.
I was able to get it to connect by using the workaround posted here from a previous issue, launching Greenlight through the Terminal allowed the controller to be recognized and play normally.  However even after doing this, opening the app normally still does not result in controller connection.
While it technically does work through launching from Terminal every time, obviously it would ideally work seamlessly through normal launches as it did before.  I did a full restart of my computer and controller to troubleshoot and neither of those resolved the issue.
To Reproduce

Open Greenlight
Go to Input Settings
Controller not appearing even when pressing buttons

Expected behavior
Expected behavior is for controller to connect and appear in input settings and work within xbox stream.
Environment
- OS: MacOS Sonoma 14.0
- Device: Macbook Pro 2019
- Greenlight version: 2.1.1

Anything else?
No response
New issue

[X] I confirm that this is a new issue and not a duplicate

can confirm this issue (and solution so far) persists on v2.20

I have experiencing the same issue.

Same here! @eqlion Also connecting the gamepad with USB doesn't work either

I'm experiencing this from Windows as well. If I connect my controller wirelessly to my console it works just fine however if I plug it in via usb to my computer my console throws up a notification stating that its no longer detected.

Problem persists on the latest version 2.3.1 in macOS Sonoma.

I have the same issue with 2.3.1 and Sonoma. Mac Pro with A1 Max chip. Didn't have this issue on prior version. So disappointed


Transaction information format is irregular

The number field is missing in the execute state, and the executedAt is missing in the finalize state.
Reference

https://github.com/ainblockchain/ain-blockchain/blob/develop/tx-pool/index.js#L34
https://github.com/ainblockchain/ain-blockchain/blob/develop/tx-pool/index.js#L488

It works well.


Make a sample of how to use with ObjectMapper

I want to use this library but I'm struggling with using this lib with ObjectMapper
https://github.com/tristanhimmelman/ObjectMapper
So can you write an example of how to use it?

Okay, I figure it out here the sample. So closed this issue.
let task = HTTP.request(GET: "https://myapi/feeds")!.parseAsJSON(using: { response, json in
            return try json.getObject()
        }).performRequest(withCompletionQueue: .main, completion: { (task, result) in
            switch result {
            case let .success(_, feed):
                if let feedResponse = Mapper<FeedResponse>().map(JSONObject: feed.ns) {
                    print(feedResponse)
                }
            case let .error(_, error):
                print(error.localizedDescription)
            case .canceled: break
                // The task was canceled. Ignore or handle as appropriate.
            }
        })
        task.resume()

I'm not familiar with ObjectMapper but assuming neither it nor FeedResponse care about the thread they're on, you can do something like
HTTP.request(GET: "https://myapi/feeds").parseAsJSON(using: { (response, json) in
    return Mapper<FeedResponse>().map(JSONObject: try response.getObject().ns)
}).performRequest(withCompletionQueue: .main, completion: { (task, result) in
    // result's success value is a `FeedResponse?`
})

BTW, the task.resume() in your code is unnecessary, performRequest(withCompletionQueue:completion:) already resumes the task. There's a separate method you use if you want to create the task without resuming it.

Thank you very much


about radiallndicator

hello, s-yadav,I want to ask, Providing a color range how can I make it?

You can pass a range of color in objects, check barColor option in http://www.ignitersworld.com/lab/radialIndicator.html#documentation
You can also enable interpolate option if you want to interpolate color in the range.


Fixed issue - not recognised dependency fields

The use is missing and therefore the ignore of the Dependencies that are not satisfied is not working or Nova actions

good catch on the missing import there Thanks!


[Suggestion] Game date under suspects

I use the suspect feature relatively frequently and I would like the suspect tab to show the date of the game.
So you could possibly remove older entries, or look again to possibly decide otherwise.
Currently you have to click on "show suspicious demos", which again resets the status of the demo and the analysis.

A new major version has been released, closing the issue 🧹


efficientnet-b0 weights url is broken

http://storage.googleapis.com/public-models/efficientnet/efficientnet-b0-355c32eb.pth
this url doesn't work anymore... HTTP Error 403: Forbidden
from where can I download valid weights?

All the weights here...
https://www.kaggle.com/raghaw/efficientnet-pytorch-b0-b7/version/1
after, copy them with the same name to ~/.cache/torch/checkpoints
that works for me.


Publish a new version

I appreciate your contribution to this SDK, i also really love it and feel sad that CartoDB doesn't have any more updates for a long time.
Are you planning to publish a separate version from this fork? If there is, I hope to see it soon,
Thank you very much.

@mrpham I should be clearer. We had some discussions with the old Dev about creating a new framework. I truly believe this framework is crazy strong, much stronger than most (except for the  styling part) and that it would deserve to be maintained. We had many ideas for new features like 3d map, splitting the framework so that you only get what you need, layers to mix vector/raster... But all that requires a lot of work and the old Dev has a full time job.
So for this to work we would need funding to support the old Dev (not me!)
If some parties (companies maybe) were interested (which is why I drop it here) it could become a thing.
PS: I have a dream this could become the official OSM framework for mobile/desktop(web in the future?)

@mrpham if you want to test i just published a rc build of my fork https://github.com/Akylas/mobile-sdk/releases/tag/v5.0.0-rc.1. Not released on maven/cocoapod yet. I need to find a new framework name first...

@mrpham if you want to test i just published a rc build of my fork https://github.com/Akylas/mobile-sdk/releases/tag/v5.0.0-rc.1. Not released on maven/cocoapod yet. I need to find a new framework name first... WARNING: all carto online stuff + licensing have been removed.

@farfromrefug , many thanks.

@farfromrefug , I tested your v5.0.0-rc1 build. It works fine, I haven't encountered any issues. Thanks again for your contributions
Also, I would like to ask if this v5.0.0-rc1 build supports "16 KB page sizes" as stated by Google? (https://developer.android.com/guide/practices/page-sizes?hl=en)
Thank you.

@mrpham from a quick reading I would need to use latest ndk which might require a bit of work. But you could still test using the described instructions on a 16kb emulator. I will look at it
On a side note if you want to help me and possibly @mtehver, please consider sponsoring me. For now it is my personal sponsor page, but if more people want that project to grow we will setup an organization.

@mrpham 16kb page size is now supported in latest published versions!


Architecture Review Discussion @ DevConnect

Devconnect's coming up soon and will be the first time many of us meet in person. It's an ideal opportunity to review the current architecture of Trin, and establish which design patterns we want to embrace moving forward. This is just an evolving list to record various topics that we'd like to discuss. Everybody is welcome to edit this and add whatever topics they'd like.
Topics

[ ] Evaluate logging strategy
[ ] Evaluate Trin general architecture
[ ] Extract JSONRPC service from trin-core?

[ ] How to delegate more logic to overlay crates (i.e. content key derivation)
[ ] Bridge node functionality
[ ] Docs location?

[ ] Evaluate database strategy

[ ] Move to just one of rocksdb / sqlite?
[ ] How to handle a preferred cache of data (which isn't within node's radius)?
[ ] trieDB location

[x] Improving trin-cli ux

Remove trin-cli jsonrpc functionality, in favor of using web3.py

[ ] Backlog grooming session
[ ] Style guide review
[ ] Switch to nightly Rust with fixed date?
Probably a good idea, we get imports granularity formatter. Cons are there may be some compiler instability but if we bind to a fixed nightly version, we don't care.
[x] Remove chain id from history content keys
[ ] add client version to enr?

We can do something similar to the Lighthouse Book for docs.


KAFKA-10601; Add support for append linger to Raft implementation

The main purpose of this patch is to add quorum.linger.ms behavior to the raft implementation. This gives users a powerful knob to tune the impact of fsync.  When an append is accepted from the state machine, it is held in an accumulator (similar to the producer) until the configured linger time is exceeded. This allows the implementation to amortize fsync overhead at the expense of some write latency.
The patch also improves our methodology for testing performance. Up to now, we have relied on the producer performance test, but it is difficult to simulate expected controller loads because producer performance is limited by other factors such as the number of producer clients and head-of-line blocking. Instead, this patch adds a workload generator which runs on the leader after election.
Finally, this patch brings us nearer to the write semantics expected by the KIP-500 controller. It makes the following changes:

Introduce RecordSerde<T> interface which abstracts the underlying log implementation from RaftClient. The generic type is carried over to RaftClient<T> and is exposed through the read/write APIs.
RaftClient.append is changed to RaftClient.scheduleAppend and returns the last offset of the expected log append.
RaftClient.scheduleAppend accepts a list of records and ensures that the full set are included in a single batch.
Introduce RaftClient.Listener with a single handleCommit API which will eventually replace RaftClient.read in order to surface committed data to the controller state machine. Currently handleCommit is only used for records appended by the leader.

(Note that this patch piggybacks on top of #9352. I will rebase as soon as this PR has been merged.)

LGTM.


Install instructions only mention plugin directory

The "How to install" section mentions this in point 3:

Locate the plugin's configuration file (usually found within /plugins/LuckPerms/) and adjust any settings to your liking.

This is obviously favouring the plugin version for Spigot, Bungee and Velocity. Afaik (Haven't used LP for mods) does the folder for the mod version differ?
This should probably be improved a bit.

Seems to be fixed already in translations, but not yet on the website itself?

Translations from crowdin refresh every 6 hours.

Yeah. Point 3 is still telling you to configure stuff in plugins/LuckPerms/ which afaik isn't true for the Fabric and forge version since those are mods?
Not sure how Nukkit and Sponge work here tho. But either way, given that point 2 mentions plugins and mods folder but point 3 doesn't mention the possible locations for the platforms is this a bit confusing...


[$1000] LHN header avatar disappears for a brief moment when refreshing page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Login to any account
Reload website
Notice Avatar doesn't instantly show

Expected Result:
Avatar will always remain/show
Actual Result:
Avatar doesn't show until page is fully loaded - there is a ~1 second delay.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.38-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/9caaa8af-3497-4aa8-85e9-3c7e6b103c6e
https://github.com/Expensify/App/assets/43996225/0caf7461-034d-4c06-bbf2-a8dd8462d8b3
Expensify/Expensify Issue URL:
Issue reported by: @situchan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688843824514419
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a250dc528d0d38dd
Upwork Job ID:<PHONE_NUMBER>823577088
Last Price Increase: 2023-07-10

Also just tested in v1.3.38-3 and I don't see this behaviour anymore:
https://github.com/Expensify/App/assets/30815269/e30c478e-07c7-40fe-94db-c83dbee199ae

Actually, I take that back - I notice that in my last video, the avatar initially was a default one, before switching to mine.
So I tested on another random convo and I can now reproduce:
https://github.com/Expensify/App/assets/30815269/e4bf1340-ccab-4e46-808f-28369e4a9dfe

Proposal
Please re-state the problem that we are trying to solve in this issue.
LHN header avatar disappears for a brief moment when refreshing page
What is the root cause of that problem?
AvatarWithIndicator is wrapped with withOnyx HOC.
There's Onyx data loading time before render.
What changes do you think we should make in order to solve the problem?
As only indicator uses onyx data, we can render Avatar always and lazy-load only indicator.
Implementation detail:
separate AvatarWithIndicator component into 2:

Indicator - render only indicator view conditionally based on Onyx data
AvatarWithIndicator - remove Onyx part and replace this line with <Indicator />
https://github.com/Expensify/App/blob/60f48175c9e3342ef192d878218820e5ed5a76cd/src/components/AvatarWithIndicator.js#L106

Demo

Before fix:

https://github.com/Expensify/App/assets/108292595/a2ca6962-f108-4432-a37b-f0ad4f7ca82e

After fix:

https://github.com/Expensify/App/assets/108292595/faf55940-4733-492c-bd71-acc2f7812cce

Proposal
Please re-state the problem that we are trying to solve in this issue.
LHN header avatar disappears for a brief moment when refreshing page.
What is the root cause of that problem?
I think the main reason, for it is whenever a component mounts, it triggers lifecycle methods, and in this case, we are fetching data from Onyx (a state management library used in your application). If the Avatar's source is fetched asynchronously, this would lead to a delay in rendering the avatar image, causing it to disappear during the page refresh.
What changes do you think we should make in order to solve the problem?

To solve this issue, what I think we can do is add a placeholder image and then when we reload the actual avatar get displayed.
import PlaceholderImage from './path-to-placeholder-image'; // Import placeholder image

function AvatarWithIndicator(props) {

    // Use placeholder image if the source URL is not yet available
    const avatarSource = UserUtils.getSmallSizeAvatar(props.source) || PlaceholderImage;

    return (
        <Tooltip text={props.tooltipText}>
            <View style={[styles.sidebarAvatar]}>
                <Avatar source={avatarSource} />
                {(shouldShowErrorIndicator || shouldShowInfoIndicator) && <View style={StyleSheet.flatten(indicatorStyles)} />}
            </View>
        </Tooltip>
    );
}

What alternative solutions did you explore? (Optional)

If images are needed I will add it as well

Proposal

Please re-state the problem that we are trying to solve in this issue.
The profile picture in the LHN header briefly vanishes when the page is refreshed.
What is the root cause of that problem?
There is a slight delay when loading the asset image.
What changes do you think we should make in order to solve the problem?
I discovered that this issue is exclusive to https://new.expensify.com/ and does not occur locally.
So I've implemented caching for the avatar image.
React Native inherently supports this functionality.

Hi @rushatgabhane , we've got a couple of proposals coming in - could you review?

@situchan sounds good! what happens when even the Avatar isn't available? eg: after a fresh logout and sign in

It won't be affected by my change, same as production. This issue happens only when app refresh.
https://github.com/Expensify/App/assets/108292595/0a9f4bab-8e50-4a34-9487-4ab37e041485

thanks for confirming @situchan https://github.com/Expensify/App/issues/22514#issuecomment-1628233400
c+ reviewed 🎀 👀 🎀

@AnshuAgarwal24 sorry,
@situchan posted a good proposal before you

Sounds good to me as well, let's go with @situchan's proposal.

@rushatgabhane PR is ready for review

@situchan just invited you: $250 + $1500
@rushatgabhane you'll be paid via New.Dot, please request with Anu!

Bump @rushatgabhane to complete the checklist.
Also, this qualifies for a bonus.

Bump @rushatgabhane to complete the list before payment is released:

@situchan : $250 (Reporting) + $1000 (job) + $500 (bonus)
@rushatgabhane : $1000 (C+) + $500 (bonus) - to request in New.Dot

Is this correct?

Bump @rushatgabhane to complete the list before payment

https://staging.new.expensify.com/r/5799653570845860

Nice, thanks you @rushatgabhane - will sort out payment today.

Paid and job closed - cheers!

Reviewed details for @rushatgabhane. These details are accurate based on summary from Business Reviewer and are now approved for payment in NewDot.


Previous dates filter still visible in background when choose language box view in front

Previous dates filter still visible in background when choose language

/claim

In mobile version it is working fine but issue is still present in desktop version, find attached the screenshot.

@bhanprakash23 my changes are not released yet, the pull request is still open

#2231

Can be fixed by #2231