date value changed between editor and grid

I've added a simple date editor.
When I enter the date, I can see correct date emitted in getValue()
but grids newValueHandler gets called with another update where the new date is decremented by 1 day.
I am writing out events to the console:

DateEditorComponent getValue() called 2001-01-01
newValueHandler  Object {node: RowNode, data: Object, oldValue: "2001-01-01T00:00:00", newValue: "2001-01-01", colDef: Object…}
generalvalueChangedHandler  undefined Object {node: RowNode, data: Object, oldValue: "2001-01-01T00:00:00", newValue: Sun Dec 31 2000 19:00:00 GMT-0500 (Eastern Standard Time), colDef: Object…}

notice that newValueHandler gets called with correct value. somewhere after that, it gets changed to a day earlier.
I have no idea what's changing the value from 1/1/2001 to 12/31/2000.
@Component({
    selector: 'date-cell',
    template: `
        <input #input type="date" (keydown)="onKeyDown($event)" [(ngModel)]="value" />`
})
export class DateEditorComponent implements AgEditorComponent, AfterViewInit {
    private params:any;
    private value:Date;
    private cancelBeforeStart:boolean = false;

    @ViewChild('input', {read: ViewContainerRef}) private input;

    agInit(params:any):void {
        this.params = params;
        this.value = this.params.value;
        this.cancelBeforeStart = params.charPress && ('1234567890/'.indexOf(params.charPress) < 0);
    }
    getValue():any {
        console.debug('DateEditorComponent getValue() called', this.value);
        return this.value;
    }
    isCancelBeforeStart():boolean {
        return this.cancelBeforeStart;
    }
    isCancelAfterEnd(): boolean {
        return this.value > new Date(9999, 1, 1);
    };
    onKeyDown(event):void {
        if (!this.isKeyPressedNumeric(event)) {
            if (event.preventDefault) event.preventDefault();
        }
    }
    ngAfterViewInit() {
        this.input.element.nativeElement.focus();
    }
    private getCharCodeFromEvent(event):any {
        event = event || window.event;
        return (typeof event.which == "undefined") ? event.keyCode : event.which;
    }
    private isCharNumeric(charStr):boolean {
        return !!/\d/.test(charStr);
    }
    private isKeyPressedNumeric(event):boolean {
        var charCode = this.getCharCodeFromEvent(event);
        var charStr = String.fromCharCode(charCode);
        return this.isCharNumeric(charStr);
    }
}

closing as not an issue with ag-Grid - Github issues are for tracking items we need to to fix in ag-Grid :)
with the above, i see nothing wrong, however my next step would be to debug through it, start at getValue() and follow what the grid is doing. my guess is grid is doing nothing, but it's seomthing to do with your browser and timezones.

this is a local call right now so there is no timezone offset. getValue() returns correct value, grids newValueHandler gets the wrong value. not sure where to troubleshoot what happens in between these 2 events

ah you say newValueHandler is getting called with correct result (i misread first time).
as to why your data is magically changing, i have no idea. if you have newValueHandler set up, the grid should be never changing your data.


Updation of name field in FixedAddress record

Presently 'name' field is left out so updating that field with
vmID value

Just curious, is this a requested feature?
The VM ID is not very descriptive, so, IMHO, it's not particular useful for presentation purposes. It is already available as EA for search purposes.


Fix vault version

apt-get install -y --download-only vault=1.14.2
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Package vault is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
E: Version '1.14.2' for 'vault' was not found

Hi,
the vault role has been switched to a binary release in order to support:

other operating system than only Debian
other vault version, like enterprise

I added clarification on that in the variable description in the role documentation
The version with the -1 suffix is specific to the debian packages versionning and will not come back.
As far as I checked, the command you state should just not occur in the latest releases ( > v0.9.10 ).
I'm closing this for now and ecourage you to open an issue with more context description if it were to happen again.
Thanks.


Add support for Redis backend requiring TLS

It looks like the RedisCacheBackend doesn't support specifying the ssl passthrough option to aioredis.create_redis_pool() in order to be able to connect to a Redis endpoint that requires TLS, e.g. ElastiCache in AWS.

Nevermind, using address URL with rediss:// instead  of redis:// does the trick.


feat: Einsum

Feature Request
Describe the Feature Request
Tensor contraction over specified indices and outer product.
Here is a full description of Einsum in ONNX:
https://github.com/onnx/onnx/blob/main/docs/Operators.md#einsum
Additional Context
You can take inspiration to EZKL or Julia-lang implementation of Einsum:

https://github.com/zkonduit/ezkl/blob/bc5abf50a23bc00b95e133d380c5b0926a1528fd/src/tensor/ops.rs#L367
https://github.com/ahwillia/Einsum.jl

lol


Unable to remove license key

Describe the bug
I'm unable to completely remove an expired license key – it reappears every time I reopen the plugin along with the red error message about the key being suspended.
To Reproduce
Steps to reproduce the behavior:

Go to 'Settings'
Click on 'Remove key'
Close plugin window
Open it again

Expected behavior
Key is not there anymore.
Actual behavior

Key is still there
Error message about the key being suspended

relates to: #1590


Fix deprecation warnings by consolidating coding calls

Fixes #5086
Hopefully this catches all the deprecation warnings.
cc: @paulb777 Together, we need to write some test automation that sets the deployment version of all targets to the highest OS version for any particular platform.

Hold off, there's a couple of issues. I'm fixing test usage of NSKeyed(Un)Archivers, and the result variable is unused on older iOS versions (which is hard to test because the simulator runtimes don't go down that far).


How to run a gsettings command?

I am trying to toggle touchpad from keyboard using kmonand under Arch/Gnome/Wayland but the fact that kmonad runs as root makes impossible to update use gsettings.
I have tried this:
runuser -u fairuser -- command /home/fairuser/bin/disable-touchpad

but it does not work either...
Is there any workaround?

You can make it run on user level, see the FAQ.

I have read the FAQ but did  not find how to do it

If you follow the steps explained in Q: How do I get Uinput permissions?, you will be able to run KMonad on user level.

I have followed all the steps in the section. I have updated the daemon service file to make kmonad run by the user ulys as you can see here:
❯ ps -aux | grep kmonad
ulys         393  0.0  0.0<PHONE_NUMBER> 12576 ?    S<sl Aug09   0:10 /usr/bin/kmonad /etc/kmonad/config.kbd

Everything works fine and I am able to run some scripts with a one-key cmd-button. But I cannot find a workaround to update gnome gsettings. This is the error I get:
(process:226062): dconf-WARNING **: 12:10:59.228: failed to commit changes to dconf: Cannot autolaunch D-Bus without X11 $DISPLAY

WIth some scripts I run with cmd-button I have checked that $USER equals ulys as expected. But $DISPLAY is empty.

Most probably an issue with your daemon? When I ran kmonad in my terminal, and pressed below button [1], correct value was printed. Have you tested your cmd-button using kmonad instance ran from a terminal?
(cmd-button "echo DISPLAY=$DISPLAY")

@pietrodito Also, if you are using a systemd service for KMonad, be sure to start it after graphical-session.target if you want DISPLAY to be available to your KMonad instance by adding this to your service file:
[Install]
WantedBy=graphical-session.target


iOS plugin with debug and release variant of a .framework

I have a plugin (let's say it's named plugin) that needs to have a separate debug build (say debug/foo.framework) and a release build (release/foo.framework) and they should be used in the respective app builds. Is there a way to do this? Ideally, on the plugin's side but if necessary even some changes to the app config would be acceptable.
The closest thing I've found is this: pod 'Lookback', :configurations => ['Debug'] (https://blog.cocoapods.org/CocoaPods-0.34/) but not sure if/how that's usable.
Here's the current .podspec but I need to manually replace the foo.framework with the variant I want used with the current flutter run/build.
Pod::Spec.new do |s|
  s.name             = 'plugin'
  s.version          = '0.0.1'
  s.license          = { :file => '../LICENSE' }
  s.source           = { :path => '.' }
  s.source_files = 'Classes/**/*'
  s.dependency 'Flutter'
  s.platform = :ios, '8.0'
  s.xcconfig = {
        'HEADER_SEARCH_PATHS' => [
            '"${PODS_TARGET_SRCROOT}"'
        ],
  }
  s.preserve_paths = 'foo.framework'
  s.xcconfig = { 'OTHER_LDFLAGS' => '-framework foo' }
  s.vendored_frameworks = 'foo.framework'
  # Flutter.framework does not contain a i386 slice.
  s.pod_target_xcconfig = { 'DEFINES_MODULE' => 'YES', 'EXCLUDED_ARCHS[sdk=iphonesimulator*]' => 'i386' }
  s.swift_version = '5.0'
end

I've also had a look at this but didn't figure out the change needed in the flutter build process... https://github.com/CocoaPods/CocoaPods/issues/2847

I'm trying to get an answer on SO as well: https://stackoverflow.com/questions/69437773/podspec-debug-release-vendored-framework-variants


v2.12.0 (issue210-address-as-parameter) - Adding ADDRESS as a potential parameter

This PR is to address issue210-address-as-parameter. I've made the changes to now allow developers to use an address instead of latitude and longitude coordinates. This is what I had commented on earlier and you, Thomas Schoffelen, had commented and said, It's not a use case I personally have, because I want to send users to a specific location rather than doing a search, but I can understand why that would be useful. I'm still happy to accept a PR for that functionality if you'd like to add it!
So here is my pull request. Please review it, give me any feedback, if you see anything I need to update or change please let me know.

I just noticed you have another PR that is going to convert this library into TypeScript (v3.0.0). If you want to do that first then we can go in and make similar changes we can do that too.

Hi @trevorrd - apologies for the delay! The Typescript migration has now been completed, so if you're willing to update this, I'm happy to get this merged. Apologies for the duplicated work!

Hi @trevorrd - apologies for the delay! The Typescript migration has now been completed, so if you're willing to update this, I'm happy to get this merged. Apologies for the duplicated work!

Hey Thomas,
I've made the changes and pushed them up. This PR should be ready for a review. I've also been pretty busy in the last few weeks but I still feel these changes would be very beneficial.
Thanks!

It's taken me way too long (apologies again), but it's merged! Thanks again @trevorrd for the amazing work on this, quite a big new feature you've shipped here!

Hi,
Is anybody able to confirm this works with google maps on ios?
Every time we use the address field it, then launch maps it gets "google maps cant open this link"

Update:
After some debugging it looks like the url structure is wrong.
Referring to the google docs here: https://developers.google.com/maps/documentation/urls/get-started it says all urls should have api=1 in, which we can see is used later on:

However for the address line it instead uses ?q=$address, whereas the docs about says it should be ?api=1&query=$address upon changing that, it worked, so i believe there is a bug there

Update:
After some debugging it looks like the url structure is wrong.
Referring to the google docs here: https://developers.google.com/maps/documentation/urls/get-started it says all urls should have api=1 in, which we can see is used later on:

However for the address line it instead uses `?q=$address`

whereas the docs about says it should be `?api=1&query=$address` upon changing that, it worked, so i believe there is a bug there

PR to fix is here: #294
Would be great to get this merged asap if possible as we caught this issue during testing for a big app update which meant to be going out today :D

Nice catch! Yes, open a bug ticket. I'm not sure how I missed that small query string parameter. I know I used it in my project for work and must have missed it as I was adding it to this library. If we could get the ticket opened and change that, it would be great. Thanks!

There's already a PR up for it here: https://github.com/includable/react-native-map-link/pull/294


runners: openocd: Default to an actual executable when not configured

Whenever build/zephyr/runners.yaml does not specify the location of the
openocd executable we seemd to be leaving the executable variable as
None. Instead, use its executable name.
Signed-off-by: Carles Cufi<EMAIL_ADDRESS>

I was wondering if we set default at https://github.com/zephyrproject-rtos/zephyr/blob/458ac71fe8b20b2b2a6e4a8cfc2a44820e446cf3/scripts/west_commands/runners/core.py#L270 and check here for an Empty string ? This could indicate that system is overriding to a null string and something is not correct in system setup.

I was wondering if we set default at
https://github.com/zephyrproject-rtos/zephyr/blob/458ac71fe8b20b2b2a6e4a8cfc2a44820e446cf3/scripts/west_commands/runners/core.py#L270
and check here for an Empty string ? This could indicate that system is overriding to a null string and something is not correct in system setup.

I don't think so. That string is an optional path to an openocd instance. If it's None, then it means no path has been specified, and at that point the knowledge about the executable name belongs in openocd.py, no the common code.


Move to Trash menu item changes to Delete from Trash

[x] My code follows the:

[x] Google Java style guide (for .java files)

[x] ant tests passes on my machine

For all other changes:

[x] I branched from master
[x] My pull request has master as the base

What does this PR accomplish?
Move To Trash menu item will change to Delete From Trash Button when View Trash Button is Clicked
Fixes #2107

Can one of the admins verify this patch?

@AppInventorWorkerBee ok to test

@SusanRatiLane following changes implemented in recent commit

[x] Rather than changing the Text and action of a menu item, created both menu items and then make them enabled/visible.
[x]  Added allowDelete Check
[x]  Removed all toggle flags

@SusanRatiLane Please do review whenever you have the time.

@beinganonymous Sorry for the delay. I was looking at it yesterday, and I'm testing today.

@beinganonymous I note two things in testing:

After I delete a project from trash and return to the My Projects view, the menu item does not always switch back to Move to Trash.

More concerning, I can use the menu item to delete a project from the My Projects view.

It's probably worth adding a check to be sure that the project is actually in the trash before deleting it.
A third thing -- When a project is open, the Move to Trash menu option is present but disabled. We should be able to send the active project to trash and then return to the My Projects view. If this is straightforward to do, we should add it to this PR. Otherwise, I can open a separate issue to add this feature.

@SusanRatiLane No need to open a separate issue to send the active project to trash, I'll cover this feature in this PR.

@SusanRatiLane Implemented following changes

[x] Menu Item will switch back to Move To Trash after returning to My Project View.
[x] Move To Trash menu item will be enabled when the project is active.

Delete From Trash menu item will not be used from My Project view, hence no need to check that the project is actually in the trash before deleting it.

@SusanRatiLane please do review whenever you have the time.

One last detail: The Move To Trash option grays out when there are no projects selected, but the Delete From Trash option stays enabled and shows a message to the user if it is clicked with no projects selected. Delete From Trash should behave like Move To Trash.
Otherwise, this checks out.

@SusanRatiLane Updated Delete From Trash button, it will be gray out if the project is not selected

@SusanRatiLane please do review the changes whenever you have time.

@beinganonymous I've been chatting with the team about the DeleteForeverProjectAction. We already have one on ProjectToolbar, but it's in a completely different scope from this one. We don't want to duplicate the code and are probably going to make this action its own class so that it can be accessed both places. I'm going to take care of that, but it will delay getting the PR merged.
Other than that, the final thing I see is that all strings shown to the users should be internationalized. I think all that we need to do here is change "Move to Trash" and "Delete From Trash" to MESSAGES.trashProjectMenuItem() and MESSAGES.deleteFromTrashButton(). These aren't buttons, but the strings should match.

@SusanRatiLane  Sounds great. I'll make strings internationalized

@jisqyv This should be ready to merge, but I'd appreciate your eyes on it, since I had to resolve a conflict with the new Gallery code.


Century: Age of Ashes

Game Title
Century: Age of Ashes
Game Website
https://www.century-age-of-ashes.com/
Anti-Cheat Software
Easy Anti-Cheat
Status of the Anti-Cheat
❔ Unconfirmed
Proof of Mention
N/A

Steam Link: https://store.steampowered.com/app/918570/Century_Age_of_Ashes/

Game is supported, see #280

Closing duplicate issue.


Show previously opened files :boom:

Changes
This PR will show the previously opened files in the Go To file list, with the last opened file first in the list and a button to clear the list of opened files:

This makes it very easy to reopen a previously edited/browsed file. :boom:
Checklist

[x] have tested my change
[x] eslint is not complaining
[x] for feature PRs: PR only includes one feature enhancement.

Before I approve, I have a question: shouldn't we limit the number of previously opened entries saved? So far it's unlimited and it's up to the user to clear this list now and then. Would it make sense to set a hard limit or add a new global setting to define this limit?

Good question - I haven't thought of that... What is an appropriate limit? 10? 20? 100? Maybe a global settings, as you suggest, so the user can decide - with a limit of 20... Agree?
I also thought of having the timestamp for the file open somewhere (description, tooltip, whatever) - could be helpful for the user...
But I decided that it could be in another PR...

I see that VSCode keeps track of the 10 last opened files, so let's say that a global setting that defaults to 10 may be good!
Do as you see fit for the timestamp, it sounds great anyway.

There you go @sebjulliand:

Previously opened files has been changed to Recently opened files
A configuration setting has been added to control the number of recently opened files in the list


Added Cloud9 EBS resizing script and updated Linuxbrew install URL

Added resize.sh to assets directory for quick access to EBS volume resizing if needed. Developers reusing the same Cloud9 environment for multiple workshops have encountered ''no space left on device'' errors.
Updated the Linuxbrew install url -- fixed #35

Hi @ejschaefer , thanks for the contribution, it is really appreciated.
For future reference, pull requests should be made against the dev branch as per our README instructions. The reason for this is because we have a verification/deployment workflow that we run on the dev branch before releasing an update to master.


opensuse/system-libdrm.patch suitable for portable build ?

@Eloston
In my custom build i add this patch to the portable build (patch_order.list)
opensuse/system-libdrm.patch
Is it suitable to add that to portable release mainstream ?

Why would you need that patch in your builds of Portable Linux?

What the patch does :
--- a/ui/gfx/native_pixmap_handle.cc
+++ b/ui/gfx/native_pixmap_handle.cc
@@ -12,6 +12,9 @@
 namespace gfx {

 #if defined(OS_LINUX)
+#ifndef DRM_FORMAT_MOD_INVALID
+#define DRM_FORMAT_MOD_INVALID ((1ULL<<56) - 1)
+#endif
 static_assert(NativePixmapPlane::kNoModifier == DRM_FORMAT_MOD_INVALID,
               "gfx::NativePixmapPlane::kNoModifier should be an alias for"
"DRM_FORMAT_MOD_INVALID");

Why it does what it does :
DRM_FORMAT_MOD_INVALID is part of the linux kernel and was introduced in 4.14 https://github.com/torvalds/linux/commit/e6fc3b68558e4c6d8d160b5daf2511b99afa8814#diff-94ab8a47ff02fa7f35cb0451290e1d85 this patch permit compatibility backward post 4.14
Why
The patch is not really tied to opensuse... its why i suggested adding it to the portable

I see. I'll include it then.

Fixed with 94b7493c72a3a5767e8b1bca6c7f160f5449fa02


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 521

Questions ?
Please refer to the Backport tool documentation


feat: Add Chongyoucar Control in node-hub

Added a control node for chongyoucar to receive text commands, 'forward' 'left' 'right' 'backward' and 'stop' to control the car's forward turning and backward stopping etc.
Author: Leon<EMAIL_ADDRESS>

To make this release easy to install. We can push a prebuilt version on pip.
I can help with it, or you can try on your own.
The idea is that you need a:

pyproject.toml like the following: https://github.com/dora-rs/dora/blob/f447dec976a8a8abaa13450ec86275879c6d6431/node-hub/dora-rerun/pyproject.toml
Add at the end of your Cargo.toml

pyo3 = { workspace = true, features = [
    "extension-module",
    "abi3",
], optional = true }

[lib]
name = "dora_chongyoucar"
path = "src/lib.rs"
crate-type = ["lib", "cdylib"]

Split your crate into a main.rs and a lib.rs
Make sure your main.rs look like this: https://github.com/dora-rs/dora/blob/f447dec976a8a8abaa13450ec86275879c6d6431/node-hub/dora-rerun/src/main.rs

fn main() -> Result<(), eyre::Error> {
    dora_rerun::lib_main()
}

Make sure your lib.rs contains this: https://github.com/dora-rs/dora/blob/f447dec976a8a8abaa13450ec86275879c6d6431/node-hub/dora-rerun/src/lib.rs

#[cfg(feature = "python")]
use pyo3::{
    pyfunction, pymodule,
    types::{PyModule, PyModuleMethods},
    wrap_pyfunction, Bound, PyResult, Python,
};

#[cfg(feature = "python")]
#[pyfunction]
fn py_main(_py: Python) -> PyResult<()> {
    pyo3::prepare_freethreaded_python();
    lib_main().map_err(|e| pyo3::exceptions::PyException::new_err(e.to_string()))
}

/// A Python module implemented in Rust.
#[cfg(feature = "python")]
#[pymodule]
fn dora_rerun(_py: Python, m: Bound<'_, PyModule>) -> PyResult<()> {
    m.add_function(wrap_pyfunction!(py_main, &m)?)?;
    Ok(())
}

And that's it.
Publishing on pip making it super easy for people to just:
pip install your project on multiple platform from macos to linux.

I would like to try to define a receive data json format
Something like
{
    “command”: “forward”,
    “speed”:1.0
}

containing a control command and the current speed, so that I can satisfy both the control of the cart and customize the speed, "command" receives two commands, one for movement and one for turning. "speed" greater than 0 is forward and right, less than 0 is backward and left.
what do you think?

I don't think we should use json. It's going to make the whole process slower and more difficult to interpolate with other components.
Can it just be something in the likes of:
let float64array = [x, y, rz, speed]

That's great

Now My last remaining question is how to convert an ArrowData data type into an f64 array.

Sorry forgot to finish the review.
Could we double check for the name of this package. Is chongyou-car really the name we want to use?

This is mainly for the dora-kit car, or you could even call it the dora-kit-car. Got any better name ideas?

That might be better yes


Make functions return the status object

Heya! I'm using your Package to automate downloading SFTP Files and then sending them via E-Mail and deleting them off the external SFTP Server. When I'm done I'd like to upload it to github for others to use, with parameters, etc.
I see that in some of your functions, you create a custom Object "Status" and then print that object into the console, which is great!
Would it be possible that you return this object instead of just outputting it to the console? I'd like to run an if check on "Status = True/False" after running the func. If true, move the file to an array of downloaded files and only if it's contained in that array, delete it on the SFTP server.
And I don't want to just copy the function into my script or change your module, I'd like people to be able to use the module as-is when they use my script :) Or are you okay with me copying the function, adding a return statement and giving you credit for it in the script?

I am not sure what you mean. The Status is not pushed into console. If you catch it to variable it will be an object... unless you use $Supress it always returns an object.

Sorry, talking specifically about the Receive-SFTPFile function. That one doesn't return the status :)

It's missing not suppress but that also means it returns status and has no ability to suppress it. But the object is being returned

So should i be able to call it using $Status after running the function?

This doesn't return the Variable, does it? From what I gather, it just outputs it to the console.

$YourVariable = Receive-SFTPFile
$Status when used like I am using it - is scoped to the function. You can also do
$Status = Receive-SFTPFile - but this will be a "different $Status variable

My god I'm so sorry, you're totally right. My train of thought was way too complicated, didn't think about just assigning the function to a variable. I'm still learning :) Thank you!


STS Request was denied due to user flow control

Describe the bug
The query may have errors like this
ErrorCode: Throttling.User. Request was denied due to user flow control. q=Throttling.User&product=Sts

Steampipe version (steampipe -v)
Steampipe v1.0.0
Plugin version (steampipe plugin list)
hub.steampipe.io/plugins/turbot/alicloud@latest   | 1.1.0
To reproduce

Execute query like select * from alicloud_sandbox.alicloud_ecs_instance

Expected behavior
The error should not happen and should show the resource info
Additional context

It seems that the plugin will call GetCallerIdentity frequently. Will the cache not work in some cases?

I set the limit, but it seems not to work.

    plugin "alicloud" {
      limiter "exec_max_concurrency_limiter" {
        max_concurrency = 5
        bucket_size     = 5
        fill_rate       = 5
      }
    }

Hello, @QiXingchuan,
I’m sorry to hear that you’re encountering this issue. Let me ask a few follow-up questions to better understand and assist you:

Which authentication mode are you using to query the tables?
Could you share the content of your ~/.steampipe/config/oci.spc file? (Please ensure any sensitive information is removed before sharing.)
If the issue occurs with a specific profile authentication, would it be possible to try using a different profile authentication mode to see if the problem persists?
Have you tested the query with and without the rate-limiter configuration? Does the issue occur in both cases?

Looking forward to your response so we can troubleshoot this further. 😊

Hi @ParthaI,

Which authentication mode are you using to query the tables?

I am using RamRoleArn mode

Could you share the content of your ~/.steampipe/config/oci.spc file? (Please ensure any sensitive information is removed before sharing.)

~/.steampipe/config/alicloud.spc

plugin "alicloud" {
   limiter "exec_max_concurrency_limiter" {
      max_concurrency = 5
      bucket_size     = 5
      fill_rate       = 5
   }
}

connection "alicloud_all" {
  type        = "aggregator"
  plugin      = "alicloud"
  connections = ["ali*"]
}

connection "ali_sandbox_cloud1" {
  plugin  = "alicloud"
  regions = ["ap-southeast-1"]
  profile = "sandbox_cloud1"
}

connection "ali_sandbox_cloud2" {
  plugin  = "alicloud"
  regions = ["ap-southeast-1"]
  profile = "sandbox_cloud2"
}

...
connection "ali_sandbox_cloud100" {
  plugin  = "alicloud"
  regions = ["ap-southeast-1"]
  profile = "sandbox_cloud100"
}

If the issue occurs with a specific profile authentication, would it be possible to try using a different profile authentication mode to see if the problem persists?

If I only use AKSK mode, it will only query one account, and the STS issues will not happen (I think it is due to one account)
But if I use AKSK mode to query some resources needing to get attributes by get func like getEcsDiskAutoSnapshotPolicy, it will also have the user flow control issue, and the throttled API will be from ECS. Example: select * from ali_sandbox.alicloud_ecs_disk(The Rate Limiting doesn't work)

Have you tested the query with and without the rate-limiter configuration? Does the issue occur in both cases?

Yes, the issue happens both
I checked the audit log, It has far more queries than the limit value I set


Improve Player Colors

Player colors are pretty truly hacked in at the moment. They get assigned directly based on their order in the lobby. Besides not giving player's a choice to express their preference, this also becomes extremely problematic if someone disconnects who isn't the last player in the lobby. When a new player connects they will have the same color as the previous last player in the lobby. My proposed design is as follows:

Players will be able to choose their color while in the lobby. This choice will be persisted locally and used as the default for future lobbies if that color is available.
The available colors will be determined by us, and the player's choice will be represented internally by a new enum in clash-lib, stored on the PlayerOptions struct.

To reduce risk of collisions we will want to have more than 6 colors available. Likely around 12 or so.

When a player connects to a lobby they will tell the server the color they want. If it is available they will be given it but if it's already taken they will be given the next available color.

This will not overwrite the player's persisted color preference.

Blocked by #73


Ubuntu 16.04 ccache 3.2.4 Fallthrough Bug

Describe the bug
When I try to build current master for v4, the build fails with the following error message.
../../src/lib/led/led.cpp: In member function 'void LedController::get_control_data(LedControlData&)':
../../src/lib/led/led.cpp:229:5: error: this statement may fall through [-Werror=implicit-fallthrough=]
if (cur_data.blink_times_left % 10 < 6) { // 2 blinks, then turn off for the rest of the cycle
^~
compilation terminated due to -Wfatal-errors.
To Reproduce
Steps to reproduce the behavior:

checkout master@2f10c315b56f0bd9cb200956b112ad1f0930b90a
make px4_fmu-v4_default

Additional context
I use the gcc toolchain version:
arm-none-eabi-gcc (GNU Tools for Arm Embedded Processors 7-2018-q3-update) 7.3.1 20180622 (release) [ARM/embedded-7-branch revision 261907]

The question is why you have a newer arm-none-eabi version installed than recommended in:
https://github.com/PX4/Devguide/blob/8456e743470793414b6eda119844b8a320c138c9/build_scripts/ubuntu_sim_nuttx.sh#L29-L44
or:
https://github.com/PX4/Firmware/blob/2f10c315b56f0bd9cb200956b112ad1f0930b90a/Tools/setup/ubuntu.sh#L115-L140

@baumanta can you do which arm-none-eabi-gcc please?

@julianoes that returns: /usr/lib/ccache/arm-none-eabi-gcc

Ok, and that's a symlink. Where does it go?
ls -l /usr/lib/ccache/

I get same error. Installed from ubuntu.sh.
ubuntu@ubuntu-VirtualBox:~/github/Firmware$ which arm-none-eabi-gcc
/opt/gcc-arm-none-eabi-7-2017-q4-major/bin/arm-none-eabi-gcc

@hamishwillee so you're saying the build is broken in general, not just with @baumanta's newer compiler?

I'm saying that if you install with ubuntu.sh from that PR we've been discussing, then the problem happens irrespective of the build target - ie FMUv2, 3, 4 etc.

Ok, this is really weird with the script from ubuntu.sh it fails to compile with the fallthrough error. With the scripts from the devguide it's fine.'
I tried to find differences and found that the arm-none-eabi-gcc version as well as the cmake versions are identical. The only difference is that the Firmware script also installs ccache.
I tried to remove the ccache cache using ccache -C but it did not resolve the issue! Only sudo apt-get remove ccache helped.

Presumably the comments are stripped by the preprocessor before it goes into ccache. The workaround export CCACHE_CPP2=yes seems to work for me.

@julianoes about the symlink, running your command, I get:
lrwxrwxrwx 1 root root 16 Nov 26  2018 arm-none-eabi-g++ -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 arm-none-eabi-gcc -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Mär 12 08:57 arm-none-eabi-gcc-7.3.1 -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 c++ -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 c89-gcc -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 c99-gcc -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 cc -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 g++ -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 g++-5 -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 gcc -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 gcc-5 -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 x86_64-linux-gnu-g++ -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 x86_64-linux-gnu-g++-5 -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 x86_64-linux-gnu-gcc -> ../../bin/ccache
lrwxrwxrwx 1 root root 16 Nov 26  2018 x86_64-linux-gnu-gcc-5 -> ../../bin/ccache

I just mentioned the build target, because I noticed that SITL builds are fine. I did not try any other targets, e.g. v2 or such

I updated the title since we long found out what the problem and possible fix is:
https://github.com/PX4/Firmware/pull/12820#issuecomment-527415694
https://github.com/PX4/Firmware/pull/12827#issuecomment-527958040
To still work without manual ccache upgrade the compile warnings were temporarily disabled:
https://github.com/PX4/Firmware/pull/12825/files#diff-903c2a19ede406370227cbda7ee03051R40
https://github.com/PX4/Firmware/pull/12901/files#diff-dfaa0b66e8c5239fe60fe79fd5b29749R36
https://github.com/PX4/Firmware/commit/1f63d85869b3495f3f66a3300b365c57469f1020#diff-f56b373864d883619e4d0961090a9e9aR52 (not sure if NuttX implements the fallthrough guards)
And the desired solution is mentioned multiple times
https://github.com/PX4/Firmware/pull/13104#issuecomment-538816225
https://github.com/PX4/Firmware/pull/13084#issuecomment-537914922
Switching to C++17 and using the [[fallthrough]]; statement.
Let's keep this issue to remember/track that. I'm not aware what's missing to be fully C++17 compatible. @dagar mentioned the compiler version for one board once the snapdragon toolchain is updated.

So is this still something that needs to be alive? ie is this just alive as a reminder to people there is a problem, or are we planning on doing something?

For Ubuntu 16.04 users this can be resolved by installing a newer ccache version as it is done in:
https://github.com/PX4/Firmware/blob/0d36e5094b4655c960846d481f62361d40cf1d9f/Tools/setup/ubuntu.sh#L103-L107


Can't open _seg.npy generated by the GUI [BUG]

Hello,
I'm running cellpose 3D using Google colab and a custom model. I loaded my _seg.npy files from colab into the GUI to proofread and correct any errors in segmentation. I saved my corrected segmentation as a new _seg.npy file, but now the GUI can not open these corrected _seg.npy files. Instead, I get the following error:
ERROR: no image file found and no image in npy
The GUI still opens the original, uncorrected _seg.npy files that were generated in colab.
Thanks

do you no longer have the images in the same folder? I think that's the issue?


Create test for useWallet

Closes #176
Description
Add @testing-library/react-hooks, make use of Provider for context and test that useWallet can instantiate and trigger the request to connect to the correct network.
📝 Additional Information

Anything else you'd want reviewers to know.

Pretty basic tests, but extensible if we want to either deeply mock window.ethereum, make use of another external library (though none jump out as best-choice), or solve this in another way.

 If you want to, then you could re open the PR from a branch in the repo. that said, since it's only a test, i feel like it would be safe without chromatic running

@allcontributors add @gjsyme for code


Add testing of external components on more platforms

Testing of external components has only really been done for Linux variants. This issue is to suggest making them pass and adding them to CI for more platforms, e.g., moving a discussion from here to this issue / a future PR.

This would be certainly nice to have. On the other hand the testing of external components with Linux provides most of the benefits and although testing them on additional platforms is surely useful, I do not expect that many problems to be found by this. So not a critical priority but we would certainly take patches that make the external testing on additional platforms work (+ corresponding on-push CI runs).

the testing of external components with Linux provides most of the benefits

Sorry, I don't think I understand it. Unless spoken from a Linux vendor point of view (which I didn't think you were)?

and although testing them on additional platforms is surely useful, I do not expect that many problems to be found by this

Respectfully disagree - unless you intend to have Linux as the only "fully-supported" platform, and consider other platforms as "supported if/when it works" - like with apps written for Linux that may or may not compile and run elsewhere.


[New Feature]: Design Products page

🎮 feature Request
Only product page needs styling. I would like to style it
Also, I need to discuss with you like what content do you want me to put inside the cards?
I will take care of the styling but i need to know the content
Point down the features

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

For now, I don't have the details on what to add to the product page's card. I believe @Durgesh4993  will be the right person to guide you regarding that.
Let me know if you need any further adjustments!


Prevent duplicate tailwind css reference

On a newly generated 8.0.0.beta1 app it seems that the tailwind.css is referenced twice with the head of the generated layout
<link rel="stylesheet" href="/assets/tailwind-aaa98186.css" data-turbo-track="reload" />
<link rel="stylesheet" href="/assets/inter-font-1d90a608.css" data-turbo-track="reload" />
<link rel="stylesheet" href="/assets/application-8b441ae0.css" data-turbo-track="reload" />
<link rel="stylesheet" href="/assets/application.tailwind-bb2039bf.css" data-turbo-track="reload" />
<link rel="stylesheet" href="/assets/tailwind-aaa98186.css" data-turbo-track="reload" />

it seems that the injected stylesheet_link_tag is including the tailwind stylesheet
<%= stylesheet_link_tag "tailwind", "inter-font", "data-turbo-track": "reload" %>

it is automatically included from the 'app' stylesheet call from the app/build directory
<%= stylesheet_link_tag :app, "data-turbo-track": "reload" %>
removing the tailwind reference
<%= stylesheet_link_tag "inter-font", "data-turbo-track": "reload" %>

prevents the duplicate in the generated layout
<link rel="stylesheet" href="/assets/inter-font-1d90a608.css" data-turbo-track="reload" />
<link rel="stylesheet" href="/assets/application-8b441ae0.css" data-turbo-track="reload" />
<link rel="stylesheet" href="/assets/application.tailwind-bb2039bf.css" data-turbo-track="reload" />
<link rel="stylesheet" href="/assets/tailwind-aaa98186.css" data-turbo-track="reload" />

@rip747 This change fails the tests in CI for the reasons @brunoprietog mentions. I'm going to close this, but please feel free to ask questions and if you feel like it propose a change upstream in Rails.


Dynamic silence and ingress type

Two small changes.

Using the new ingress type for the api server. We used the new type already for workloads.
Backfilling the silence fix for the "app create" PR

Fix for https://github.com/epinio/epinio/pull/553

Looks ok. I guess the ingress type changes are what came out of the discussion while I was out.

Yes, we already use ingress v1 in other parts of the code base, so I fixed the TODO. However I typoed when I was unable to cherry pick https://github.com/epinio/epinio/commit/7721d610fdf27a79be980af522783671d3ffc198

I replace the remaining occurrences and fixed the epinio ingress commit.


Representer Error

Description
Connecting to postgres:
START /entrypoint.sh
Trying to generate openapi.yml
Traceback (most recent call last):
   File "/usr/lib/python3/dist-packages/yaml/representer.py", line 118, in represent_mapping
    node_value = self.represent_data(item_value)
  File "/usr/lib/python3/dist-packages/yaml/representer.py", line 48, in represent_data
    node = self.yaml_representers[data_types[0]](self, data)
  File "/usr/lib/python3/dist-packages/yaml/representer.py", line 199, in represent_list
    return self.represent_sequence('tag:yaml.org,2002:seq', data)
  File "/usr/lib/python3/dist-packages/yaml/representer.py", line 92, in represent_sequence
    node_item = self.represent_data(item)
.......
  File "/usr/lib/python3/dist-packages/yaml/representer.py", line 58, in represent_data
    node = self.yaml_representers[None](self, data)
  File "/usr/lib/python3/dist-packages/yaml/representer.py", line 231, in represent_undefined
    raise RepresenterError("cannot represent an object", data)
yaml.representer.RepresenterError: ('cannot represent an object', 'landslide_number')
ERROR: openapi.yml could not be generated ERROR

If I # comment out the landslide layer, error is still found on another layer (ql_acd or ukwm):
yaml.representer.RepresenterError: ('cannot represent an object', 'item_index_id')
ERROR: openapi.yml could not be generated ERROR

Again if I comment these layers out, still get the error on another layer...

Failiures are always on column 1 attribute, although sometimes this isn't actually the unique ID

The issue here seems to be that the reflection we use to get the column/field names gives us field names of the type sqlalchemy.sql.elements.quoted_name. This is a type that represents the text of the name and a boolean flag representing whether it needs quoting. see sqlalchemy docs. This works for subsequent queries but these types can't be handled by the initial code that tries to create the openapi.yml file and so the deployment fails.
This can be worked around, as in the commit here https://github.com/BritishGeologicalSurvey/pygeoapi/commit/db8fdbca4b845736437b558dac357f4b0010933b which switches the yaml.safe_dump function to yaml.dump. This branch will generate an openapi.yml file, though that not useful:
        schema:
          items:
            enum:
            - !!python/object/apply:sqlalchemy.sql.elements.quoted_name
              - ql_map_title_id
              - null
            - !!python/object/apply:sqlalchemy.sql.elements.quoted_name
              - map_title_id
              - null
...
            - !!python/object/apply:sqlalchemy.sql.elements.quoted_name
              - scan_url
              - null
            type: string
          type: array

This branch will probably allow the app to be deployed so would be worth testing.
However, what we probably need to look at how we deal with this type in the reflection stage and what is passed through to the openapi.yml generation. In addition we also need to look at the whole of that generation, does it represent our endpoints fully, query names and parameters?

@ximenesuk thanks for looking into that and the detailed response with solutions! I'll have to talk it through when we're all back at work next week for a better understanding. In the interim I'll have a go with solution 2.

If you manage to deploy the server, can you look at the Swagger page (usually /docs) and see how the /items is documented for a collection? Can you successfully “try it out” or does the API not support what is documented? Or, does it also not document what the API supports? This should help in outlining an issue to fix the openapi generation from out config files.
On 19 Oct 2022, 21:49 +0100, KoalaGeo @.***>, wrote:
@ximenesukhttps://github.com/ximenesuk thanks for looking into that and the detailed response with solutions! I'll have to talk it through when we're all back at work next week for a better understanding. In the interim I'll have a go with solution 2.
—
Reply to this email directly, view it on GitHubhttps://github.com/BritishGeologicalSurvey/pygeoapi/issues/14#issuecomment-1284557763, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIEBOJ7T3HBNHUJDZ2BBUTWEBNG7ANCNFSM6AAAAAARIMNSFE.
You are receiving this because you were mentioned.Message ID: @.***>
This email and any attachments are intended solely for the use of the named recipients. If you are not the intended recipient you must not use, disclose, copy or distribute this email or any of its attachments and should notify the sender immediately and delete this email from your system. UK Research and Innovation (UKRI) has taken every reasonable precaution to minimise risk of this email or any attachments containing viruses or malware but the recipient should carry out its own virus and malware checks before opening the attachments. UKRI does not accept any liability for any losses or damages which the recipient may sustain due to presence of any viruses.

@ximenesuk I've got that deployed an working with @volcan01010 today in dev. Please can you open a PR from unquote to 'pygeofilter-sqlalchemy` to make https://github.com/geopython/pygeoapi/pull/964 ready to merge.
We can do another PR later for the openapi config...


docs: Clarify need for credentials when running make codegen

docs: Clarify need for credentials when running make codegen
How was this change tested?
N/A
Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can we make this change in all versions of the docs?

Updated the PR @engedaam

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.246%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5332

Relevant Lines:
6483

💛  - Coveralls


Update dependencies

This PR updates the dependencies to their latest versions.
A new release is required too.

Merging this will solve https://github.com/drand/drand/pull/1088#discussion_r1045865026


[5.1] Time limit for queue listener

We already have a timeout option.

--timeout limits each job
I proposed time limit for queue:listen process.
$php artisan queue:listen --runtimelimit=300
stops listening after 5 minutes just after finishing current job.
Till now I had to use $timeout 300 php artisan queue:listen
But sometimes it kills process while job is running.
Can you reconsider?

@kuczmaja Why would you want to set a timeout on a queue:listen?
If you want to process only one job you can do php artisan queue:work

I know. I just need to run listener for particular time, not for only one job.
I use timeout command, but this way I cannot avoid killing listener while processing jobs.


Do not update version in package-lock.json

Based on recommendations for pipeline caching at https://docs.microsoft.com/en-us/azure/devops/pipelines/caching/index?view=azure-devops#nodejsnpm, NBGV shouyld not stamp the version in package-lock.json. I talked with our JavaScript partners for Azure SDK and they can't think of a valid scenario why you'd need to update package-lock.json anyway.
I'm aware of the workaround instructions to copy all files, but - especially for large projects with many files - that seems like overkill. And some tools like vsce (the VSCode extension utility) don't appear to let you re-base a package (maybe it's just a matter of setting the working dir, but that kinda proves my point: there's lots of "fixes" everyone would have to do with this workaround).
Seems it'd be best for everyone if package-lock.json just wasn't touched at all. Let npm manage that. Same would go for yarn if you update its lock file as well.
I'd be happy to make a fix if you agree.

I'm not sure what you're talking about, since nbgv doesn't ever write to package-lock.json. Not directly, anyway. All we do is:
https://github.com/AArnott/Nerdbank.GitVersioning/blob/6b0a602909dcba13279e73e3c74f6e5f8ec57b6d/src/nerdbank-gitversioning.npm/ts/index.ts#L73
And I'd love to avoid touching files at all, but I don't know how to convince NPM to build a package with any version other than what is hard-coded in the package.json file, so my intent was to stamp the package.json file's version, and I use the npm tool to do that.

Good to know. To be honest, I hadn't dug into it yet. So it seems npm is doing it and I can explore more there. But you'd be fine with avoiding modifying the lock file then?

All I care about is that npm pack will use the applied version. If that doesn't require changing package-lock.json, and avoiding that solves a problem for you, yes I'll take a PR.

Looking into this more, I think it's probably just best to document an example similar to what I used. There's a lot of ways people might do this, what lockfiles they use along with what package managers (e.g. npm vs. yarn), so probably just documenting it would be best,  yeah?


Allow for Form Tag Helper Methods in Model-Backed Forms

Hello,
As I was using this excellent gem,  I found myself wanting to be able to mix form tag helpers into my model-backed forms.  In order to accomplish this, I check for the presence of a tag option, and if it's present, within #form_group_options, I temporarily set @object to nil, @object_name to an empty string, and @acts_like_form_tag to true.  The original values are passed into #form_group and set back to their original values before the next form helper method is called.
Let me know what you think!
Thanks,
Vince

Hi, thanks for the PR!
Is your goal to allow things like email_field_tag that are Bootstrap styled?
I think that would be useful to have, but I disagree with the tag: true approach to accomplishing that. It still requires you use bootstrap_form_for, right? That contradicts how the standard Rails *_tag helpers work.
Is there a more natural way to approach this? For example, how to libraries like simple_form and formtastic handle it?


Validate Wallet for User (Demo purpose)

I'm working on a login page which validate User's wallet.
We will have 2 options

Connect Wallet:
If authorize => redirect to ClassRoom page, else should be show the warning message "Invalid Wallet!"
Without Wallet:
Go directly to ClassRoom

This issue only served for demo to client only. Should remove when we proceed to connect a wallet feature.

Add some feedbacks from @jason065:

After successfully validating wallet, the button should change into "Enter Metaverse"
Should remove button "Without Wallet"
Adding loop in response from calling api to match the exact user's Token Address


Script defects

Good afternoon, I contacted you earlier, you said that you are very busy, but still I want to report some bugs that I noticed, if you are free and can fix them, because I do not know so much denizen to do something myself(
The first bug is that when I enter the coordinates in the chat myself instead of clicking on the block, the output is this:

Next, it is necessary to hide somehow the rest of the Bossbar, because when the player already has one, the upper part of the screen black texture becomes larger than the bottom and there is some unevenness:
With another bossbar:

Without another bossbar:

Next, the play argument has a broken tab compliter and shows partials as I understand it:

Further, on 1.20 alas textures of objects are broken, everything else works fine:

Next, I entered the name with a dot and now there's this cat scene that won't delete:

Well and the very bug that interrupts the actionbar and the sides are removed for a while, I've already talked about it but in case you forgot, it would be better if the player will wear a special pumpkin with these sides.
Understand me correctly, I do not think that your scripts are terrible, they are divine, beautifully designed, very extensive functionality and want to use this script, because it can create a very smooth cut scenes, which can not be done in paid plugins.

Alright, I'll take a look at this next week when I'm free

I'll be waiting, I really hope your cutscenes get even better.


Add whenEndpointReady() to Central Dogma client

Motivation:
Central Dogma client waits for the initial endpoints of a
DynamicEndpointGroup by default when creating a new instance.
https://github.com/line/centraldogma/blob/b167d594af5abc06af30d7d6d7d8b68b320861d8/client/java-armeria/src/main/java/com/linecorp/centraldogma/client/armeria/AbstractArmeriaCentralDogmaBuilder.java#L169-L179
Some users might expect this blocking operation and want
to use Central Dogma client later when needed.
Modifications:

Add whenEndpointReady() to CentralDogma client so that users can wait
for the initial endpoints accoding to their requirement.

Result:

Breaking) Central Dogma client no longer waits for the initial
endpoints by default. You should use CentralDogma.whenEndpointReady()
to wait for the initial endponts.

Reset the base branch to master.
I will cherry-pick the commit from master branch to 0.53.0 branch and release this feature separately with Armeria 1.14.0.
Other new features are scheduled to release after @minwoox comes back from parental leave.

I will cherry-pick the commit from master branch to 0.53.0 branch

I will create a new branch 0.54.0 instead of reusing the old branch.


Autocomplete results markup

Hi,
Is it anyhow possible to change markup in results from autocomplete?
I need some wrappers there instead string with <br>.

Hej @konsalex
After last update and changes from #3 the formatting isn't working, it's always default one.


Update get-started-partner-integration.md

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@mjcaparas Please merge this PR. Thanks


"Multi-query input not implemented"

What is the fastest way to handle multiple sequences? (e.g. without loading the db into memory each time?).
I know there are some sample scripts, but I'm having a hard time seeing how they relate to the simple, step-by-step example (that is only good for a single fasta file).

@DaRinker Thank you for your message. Currently, multi-query input is not implemented, but this is a feature that we will make available very soon.

@DaRinker Thank you for your message. Currently, multi-query input is not implemented, but this is a feature that we will make available very soon.

In the meantime, can you help me with the question I asked? I'm finding the single query process to run very slowly (much more slowly than the MPI web portal) and I think it's because the database is having to load into memory each time.
What is the (current) best practice for running 1000s of sequences locally?
Thank you

Can you contact me at<EMAIL_ADDRESS>and we will figure out what the problem is?

Thanks for your help. Using the examples.sh script helped cut the processing time by about 60%

Hi, I am having 2 million sequences (query) and I want to do homology search of these against 250 sequence database. Currently these is no multi query option. For 2 million sequences, running each query individually will take such a long time. Is there any way to fast pLMBLAST?

@Citugulia40 Hi! The multi-query option is already implemented, but needs some testing before merging with the main branch. We expect to release it very soon along with other updates.

Thank you so much.
Sorry to ask you, Are you expecting the multi-query to be merged in the main branch this month or later than this?

Definitely this month, maybe even next week.

Hi, I'm curious to know if you have an estimate of how long pLMBLAST would take to execute when running 2 million query sequences against a database of 250 sequences (after implementing the multi-query option)?

We will provide a speed benchmark along with the multi-query support. As a rough estimate, the ECOD benchmark (all-versus-all comparison of 1500 sequences) took about 30 minutes on a 20-core CPU. Running times will depend heavily on the cosine similarity cutoff and the length of the sequences. You may also want to consider clustering your 2M sequences to 40-50% identity at a high coverage cutoff (e.g. with MMSeqs2). Given the sensitivity of pLM-based methods, searching with 1-2 examples per cluster should be sufficient.

Thank you so much for your kind support, Eagerly waiting for the multi-query option.

Definitely this month, maybe even next week.

Hi, I just want to ask that is there any update regarding the multi-query option?
Thanks in advance

Hi, all changes are in: https://github.com/labstructbioinf/pLM-BLAST/tree/multi_query_feature i will merge them on Thursday. There is still some work to do

Ok, Thank you very much

Changes are now live, looking forward for your feedback :)


fix: syntax highlighting of type groups and function signatures

fixes: #233
This commit uses branching to distinguish type groups and function signatures.
A function signature is of the form (arg: type, ...) -> return-type

Thanks!


Read bbox subset of mesh

Users of a code have been spotted reading in huge ProE mesh files with abandon, then complaining when they run out of memory.    O the embarrassment.")  My first answer is "well, don't do that."
Beyond that, though, there may be a legitimate need to read in only part of a mesh, either ProE, STL, or some other.  I could imagine modifying https://github.com/LLNL/axom/blob/develop/src/axom/quest/readers/ProEReader.cpp by only adding tets or tris that intersect a user-supplied bounding box.
At a different level, we could write a "prune to bounding box" operator for Mint meshes (which is what the ProE and STL readers output).  Is this something anyone has asked for?

After some discussion, the Axom developers made the following points:

We should do selection or pruning by passing in a lambda to give the answer in/out.  Specifying one bounding box may not be flexible enough.
This idea changes Axom's model for parallel readers.  Usually we read everything in on one rank and broadcast to all ranks.
For real memory savings, we want to process the file out-of-core.

If the criterion is "all points within a bounding box" (which leaves out tets that are cut by the bbox), "out of core" is really easy.
If the criterion is "all tets that intersect the bounding box" (includes tets that are cut by the bbox), we have to read in all points but can discard tets and then afterward filter out unused points.

We should canvas user applications to see what workflows need.

Discussion with the user shed further light on the situation.  He thinks has ProE files in the GB (!) range.  His workflow has grown to include a step to partition the ProE files so the user code can deal with them.


FileFetcher::Base.repo_contents_path

This extends the repo_contents_path attribute that was added to FileParser+FileUpdater to FileFetcher.
By making the FileFetcher aware of the target directory for a checkout at construction time:

we can leverage this checkout in #dependency_files, to avoid double fetching (as demonstrated by _jurre in https://github.com/dependabot/dependabot-core/pull/2551/commits/77857ec217451287b017fc3b112be56cc119299b#diff-d76f068189ee62d3127d9dbc19dcf4d0R51-R65)
avoid passing target_directory at runtime, as it's baked in

Related

Implements https://github.com/dependabot/dependabot-core/pull/2476#discussion_r476390866
Slice of https://github.com/dependabot/dependabot-core/pull/2551
Continues #2461

One thing I was thinking about, is if we could (in the future) have a FileFetchers::Git class that doesn't inherit from FileFetchers::Base?

💯 ; aligns with this comment - it'd be noice if instead of registering a FileFetcher, then registering that FileFetcher as always_clone... you could just register a FileFetcher that does the needful.

:x: this changes the behaviour of fetch_file_if_present().
Previously it would list the repository, then return nil if not found.
As of this rev it raises DependencyFileNotFound  in the repo_contents_path case.
Caught when integrating with https://github.com/dependabot/dependabot-core/pull/2551 ; fetching go.sums that aren't present.


Update .release-please-manifest.json

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

com.google.cloud.pubsub.v1.Subscriber

P


Fix gitbook sync

We are having issues syncing gitbook latest to docs/main branch on git.
Let's try to do a snapshot of the changes, clean docs branch, and resync gitbook with a new docs. Afterward, we should periodically merge docs into main.

@harshach as there's quite a lot of movement in gitbook these days, we could try to jump into a call and tackle this next week. If we're having issues we could also rediscuss to sync git -> gitbook instead of the other way around. Thanks!


Closes #27410 - support multi-select on open file/changes commands

See https://github.com/Microsoft/vscode/issues/27410
Also stops the opening of an editor from stealing focus -- so that it is much easier to keyboard around (it also aligns with the behavior of the main file explorer)
That there are comments in the code noting things that need to be changed once PR https://github.com/Microsoft/vscode/pull/27357 is merged.

I think that failure is just a timing issue unrelated to my changes

@joaomoreno any thoughts on this? I'd love to get this in ;)

@eamodio Was out on vacation, will look at this in the next few days.

@joaomoreno I resolved the conflicts on this -- any chance this can make it into the August milestone? ;)

Thanks!

@joaomoreno I didn't see this PR in the release notes for August -- did this not make it in? Or just an oversight?

Definitely an oversight.
@egamma The PR tool seems to have failed to capture this PR for the Thank You section. I've added it manually: https://github.com/Microsoft/vscode-docs/pull/1159

@joaomoreno the PR tool checks the milestone whether a PR should be considerd and this PR is assigned to backlog and not the August milestone.

That explains it!

@joaomoreno @egamma thank you!


Bonding on Android

This enables bonding on Android, which is now required to workaround what appears to be an Android bug causing the pairing dialog to appear twice.  It's possible there should be some sort of IsBondingSupported property as well, as bonding isn't supported on iOS or (I believe) UWP.

You may be aware that we have another open PR on this topic (#691). Could you comment on how this PR relates to the other one and how they differ?

I was not aware, sorry. You can close this, I just needed bonding support.

I was not aware, sorry.

Nevermind ;)

You can close this, I just needed bonding support.

You're not the only one with this demand, apparently :)
Could you maybe check if #691 works well for your purpose, or if you see any problems?

I'll have to try it out later but glancing through it it looks like it should work. I'll switch over once the PR is merged. Thanks.

I'll have to try it out later but glancing through it it looks like it should work. I'll switch over once the PR is merged. Thanks.

Well, the other PR is not fully done yet, there are still some problems. If you have an interest in having this feature available, it would be good if you could try it now. Otherwise it might not get merged for 3.0 at all (or only partially).
Maybe your PR even works better than the other one? That should be checked, and I only have limited time for this right now.

I've merged the changes from this branch into #691 and created a new PR #730
This PR may be closed?


Revisit single day mark styling.

White or black fill that reverses on highlight.

Colored stroke with opaque white fill normally, black stroke with colored fill when highlighted.

Duplicate of #87


Payload is not polled to completion

We have a Payload-wrapper implementation that wants to take some action (e.g. record stats) once the payload is complete.  I expect poll_data to be called until None is returned or until is_end_stream returns true. However, in practice I see that poll_data is called just once, returns data (not None), and is just dropped by hyper.
Now, of course, I can just implement Drop; but it would be helpful to clarify the contract for Payloads. Should they be polled to completion? Should they typically be dropped before is_end_stream returns true?

In the usual case, the payload should be polled to completion. Here's the cases I can think of where it wouldn't be:

There was an error on the connection.
After a poll, is_end_stream claimed it was done.
The body was not allowed, such as in response to a HEAD request, or as a 204 or 304 status code.
The headers (or the content_length method of the payload) declared a length, and the state machine detects the length has been reached. Any extra chunks would be wasted.

In the specific case I'm encountering, Conduit is proxying an HTTP/1 request with a Hyper server and client.

The client receives a response (and Body) and returns it to the server.
We wrap the client's body type with an implementation that does some extra accounting.
When we poll_data the underlying body, we get data that matches content-length.

At this point I expect that body.is_end_stream() should return true, but it does not.

After a poll, is_end_stream claimed it was done.
The headers (or the content_length method of the payload) declared a length, and the state machine detects the length has been reached. Any extra chunks would be wasted.

Is it a bug that body.is_end_stream() returns false even though the complete body data has been returned from poll_data?

is_end_stream is a hint:
https://github.com/hyperium/hyper/blob/c3c35e866c06d8d0f7347cebff86b0442b33e2b3/src/body.rs#L57-L70

@sfackler thanks for the clarification!


Multi-root workspaces require all folders to a valid python interpreter

This adds additional context to issue #370 by another user which was closed due to lack of response. Since it appears not possible to re-open that ticket I'm adding a new one here.
In a multi-root workspace, it appears that the black server will not start up unless all folders have a valid Python interpreter. The suggested solution, to configure the same python in all folders, is not feasible for us. I wanted to add some context about our use case to explain why.
We use an older version of Linux as our primary development desktop (Amazon Linux 2). This version provides Python 2.7 and 3.7 by default. The minimum Python version for Black is 3.8. This is not a problem because in our setup, build tools are normally provided by our build system instead of the host OS.  Packages that use Python configure the interpreter path to point to a wrapper in the local folder, which at runtime asks the build system for the actual Python location, and then executes that with the provided arguments. This has been working well for Pylance.
Why can't we configure all packages to work this way? Our build system contains a very large number of packages, and many of them are not using Python. These packages come together multi-root workspaces managed by our build tool, which is the primary location where developers develop. It is common to end up with workspaces containing packages that use Python and packages that don't. In such a case, the black extension falls back to the system Python for the latter packages, which does not meet the minimum version requirements, and is therefore preventing the black server from starting up.
EDIT: the ticket also suggested to use an "entire workspace" python version to set the Python version for all folders. While this is possible, the workspace-level configuration is not checked into source control. Therefore this adds friction for our developers that would need to edit this file manually for every workspace that they create.

@geertj We are working on a potential fix that should make this easier. Meanwhile, you can pin the version of the extension to use older version which supported 3.7.


Test gather tree in eager mode only because it's a custom op

We only need to run this test in eager mode, it makes it more readable too.

Thanks a lot!


Model weights and inference code

Thanks for the great work.
I would like to know when you will release the model's weights and the inference code。
Best.

Same. Thanks,

Same. Thanks,

I'm very sorry, everyone. Thank you for your patience. We have a project deadline coming up soon, so I don't have time to organize the weights at the moment.

any plan?

Great work and innovative idea!
same question from my side, any plans to release the model's weights and the inference code?

Great work.
I have the same question.

干得好。
我有同样的问题。

我也有同样的问题


TypeScript "as" keyword is not recognized in Intellisense

Search Terms:
as, intellisense
Code
var x = this as

Expected behavior:
If you type as like above, Intellisense should recognize it as an acceptable keyword
var x = this as HTMLInputElement;

Actual behavior:
Intellisense suggests async so you end up with
var x = this async HTMLInputElement;

Related Issues:
Visual Studio feedback ticket

This seems to repro with if as well. It's a poor experience when VS autocompletes partial matching identifiers.
User feedback:  https://developercommunity.visualstudio.com/content/problem/828762/javascript-intellisense-replaces-exactly-typed-key.html


Handle Node failure/restart

A single node of a cluster fails as a result of OOM and restarts.  The cluster status remained at NORMAL, during the startup of the failed node.  Once the node was completely up, queries work as expected.
Expected behavior
cluster status would change to something not normal until the node has returned to the cluster
Actual behavior
cluster status stays at NORMAL during the whole window of node failure

If replication > 1, then I would expect cluster status to stay NORMAL. I agree though that if replication == 1, then the cluster should not be serving queries.

I think there are some other issues related to this - I've seen situations where one node in the cluster is stuck in state "STARTING" while the others are in "NORMAL". I think if a node dissapears from the cluster, we need to put everything into state "STARTING" (or similar) until it rejoins.

Digging into this found another related issue which is that memberlist won't allow a node to leave the cluster and then rejoin again with a different address but the same ID/Name. I've filed a ticket with them here


Document versions for each release

Resolved in https://github.com/spring-cloud/spring-cloud-app-broker/commit/14aa96050c48cfeab01d4fa15dd25801d7034c83


Improve docs and import structure

General

pyfar version: 0.5.4

Description

[ ] Unify documentation of classes, especially of magic functions like __init__
[ ] Document functions contained in the class files, e.g., add or deg2rad
[ ] Add information about handling complex time data to concepts/Arithmetic operations
[ ] stop importing class moduls, e.g., pyfar.classes.coordinates

Maybe add the sorting of import blocks to the list?

Do you mean the import ... statements in the Python files?

Do you mean the import ... statements in the Python files?

Yes. Maybe that is unrelated, though...


[FEATURE REQUEST] save some clips

It would be great to have the ability to save some clips for later usage through some shortcut 🙄
I.E. Alt+S from the rofi menu to save some clip for example

What does "save" mean in this context? Save them where?

maybe somewhere in $XDG_DATA_HOME? something like ${XDG_DATA_HOME:="$HOME/.local/share"}/clipmenu/

Hmm, why does that need to be built in to clipmenu? You can already do something like:
CM_OUTPUT_CLIP=1 clipmenu > ${XDG_DATA_HOME:="$HOME/.local/share"}/clipmenu/whatever

Multi select is just waiting for a PR, #153 didn't get resubmitted after feedback.


🐛 [firebase_analytics] I Can't remove the user id or the user properties for android.

Bug report
When I try to remove the user id/property using the plugin, it is not possible for android devices.
Expected behavior
import 'package:firebase_analytics/firebase_analytics.dart';

class Analytics {
  FirebaseAnalytics analytics = FirebaseAnalytics();

  void setUserId(String? userId) {
    analytics.setUserId(userId);
  }

  void setUserProperty(String name, String? value) {
    analytics.setUserProperty(name: name, value: value);
  }
}

Analytics analytics = Analytics();

analytics.setUserId('12345'); // Add user ID
analytics.setUserProperty('user_name', 'Jhon Doe') //Add user property "user_name"

//The error occurs when I try to remove that user ID or an user property for android Plugin i cpou
analytics.setUserId(null); // Should remove the user ID, but not works on android plugin.
analytics.setUserProperty('user_name', null); // Should remove the user property "user_name", but not works on android plugin.

Additional context
Versions:
firebase_analytics: 8.1.2
Flutter plugin documentation.
[Setting a null [id] removes the user id(https://github.com/FirebaseExtended/flutterfire/blob/master/packages/firebase_analytics/firebase_analytics/lib/src/firebase_analytics.dart#L133)
Setting a null [value] removes the user property.

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.1, on macOS 12.0.1 21A559 darwin-x64, locale en)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.62.3)
[✓] Connected device (3 available)

• No issues found!

@julianjab
Thanks for the report. I see that you are on a bit older version of stable and the plugin itself. Can you try upgrading both (stable to 2.5.3) and plugin to ^8.3.4 and see if it helps ?

Same problem on Flutter stable 2.5.3 and firebase_analytics 9.0.2.
FirebaseAnalytics.instance.setUserId();

This throws an exception:
PlatformException(firebase_analytics, null, null, null)

StandardMethodCodec.decodeEnvelope (/Users/james/Development/flutter/packages/flutter/lib/src/services/message_codecs.dart:607)
MethodChannel._invokeMethod (/Users/james/Development/flutter/packages/flutter/lib/src/services/platform_channel.dart:156)
<asynchronous gap> (Unknown Source:0)
FirebaseAnalytics.setUserId (/Users/james/Development/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_analytics-9.0.2/lib/src/firebase_analytics.dart:142)
<asynchronous gap> (Unknown Source:0)

This was fixed in the 9.0.3 release - please try it out


Empty PR to test toolchain generation

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5754
@swift-ci Please build toolchain

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5754
@swift-ci Please test

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5754
@swift-ci Please build toolchain Windows

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
@swift-ci Please build toolchain

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
@swift-ci Please test

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
https://github.com/apple/sourcekit-lsp/pull/622
@swift-ci Please build toolchain

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
https://github.com/apple/sourcekit-lsp/pull/622
@swift-ci Please test

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
https://github.com/apple/sourcekit-lsp/pull/622
@swift-ci Please build toolchain

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
https://github.com/apple/sourcekit-lsp/pull/622
@swift-ci Please test

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
https://github.com/apple/swift-driver/pull/1169
@swift-ci Please build toolchain

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
https://github.com/apple/swift-driver/pull/1169
@swift-ci Please build test

Please test with following pull request:
https://github.com/apple/swift-package-manager/pull/5755
https://github.com/apple/swift-driver/pull/1169
@swift-ci Please build Apple Silicon

Please test with following pull request:
https://github.com/apple/swift-driver/pull/1168
@swift-ci Please build toolchain

Please test with following pull request:
https://github.com/apple/swift-driver/pull/1168
@swift-ci Please test


Richie home redo

Redo of index page that I presented in class last week.
Took some time this week to fix formatting/zoom problems before submitting PR.
Changes only effect index/home page.

Currently breaks the dashboard. No items are showing

@KyleRessLiere is correct (good catch), I did not go past the homepage trying to get these in quickly for the Lint update, I am pulling my approval, but am not sure we should close the PR?
If there is a quick fix that can be implemented to add this to main today re-submit for approval. Please make sure it is not breaking other parts of the UI

Yeah no problem richie said he gonna look into it. I just stopped because I wasnt the reviewer and wanted to make sure it didnt fall onto main


There have problem with upgrading cluster from v1.23.6 to v1.23.9

Describe the Bug
There have problem with upgrading cluster from v1.23.6 to v1.23.9
Versions Used
Kubernetes: v1.23.6
Kubeclipper: master
Environment
CentOS 7.7.1908
masters: 4cpu/8g
How To Reproduce
Steps to reproduce the behavior:

Go to 'Cluster Manager - Cluster'
Click on 'Cluster Update'
Choose 'Offline-v1.23.9' and click 'OK'
Jump to 'Cluster Manager - Cluster' and click 'View Log'

Expected Screenshot

/assign @lixd


get array from const enum

I hope there is a way to get an array from const enum.
for example:
const enum gender{
male = 1,
female = 2,
}
I need to there is a way to get an array from the const enum gender, and the result like this:
[1,2]
Thanks a lot!

const enum gender{
male = 1,
female = 2,
}
const genderValue = gender.toArray() // maybe just like this

Duplicate of #21391.

I am pretty sure he meant to allow some syntax that like const enum which is replaced with literal strings at runtime will be replaced with the literal array of the const enum options.
So if we have say:
const enum Test {
  A = 'a',
  B = 'b'
}
const values = valueof Test;

It will get compiled to const values = ['a', 'b']

Yes. Please implement it.
Thanks!

It will get compiled to const values = ['a', 'b']

This would be type-directed emit, which is unlikely to be implemented.  See point 5 at https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals#non-goals:

[Do not] Add or rely on run-time type information in programs, or emit different code based on the results of the type system. Instead, encourage programming patterns that do not require run-time metadata.

@fatcerberus Which means const enum goes against it too... I believe its just an enhancement to the already existing functionality

To expedite the triage process, we need everyone to follow the issue template and instructions.
When you clicked "Create New Issue", the issue form was pre-populated with a template and some instructions. We need you to read those instructions completely, follow them, and then fill in all the fields in that template.
We are not able to assist with issues that don't follow the template instructions as they represent a significantly larger amount of work compared to issues which are correctly specified. Thank you for understanding.

enum X { A='A', B='B'};
const enum Y { A='A', B='B'};

every enum is defining a group of related values
we can use them like:
const x:X = X.A;
const y:Y = Y.A;

compiles to
const x = X.A;
const y = 'A';

then it is logical to have also a way to access the whole group of values.
like:
 const Xs = Object.values(X);
and (((or any other syntax which doesn't matter at all)))
const Ys = valuesof Y;
then they compile to:
 const Xs = Object.values(X);
and
 const Ys = ['A','B'];


ODS-Component - Card -  horizontal card

Card with Image side by side with the Text
The content of this task is described in #17
Note the the divider can be taked out, by default there is one.

@mccart77  when no text is shown should the available text be centered vertically?

@mccart77 when no text is shown should the available text be centered vertically?

as discussed earlier, yes, vertically centrally aligned.


Update aws-anywhere build tooling repo

Update https://github.com/aws/eks-anywhere-build-tooling/ repo to start using all upstream version that have VLAN trunk functionality.
Patches for the following repos will be needed.
tinkerbell/cluster-api-provider-tinkerbell
tinkerbell/hub

Completed via https://github.com/aws/eks-anywhere-build-tooling/pull/1529


7TV Emote Provider Implementation

Pull request checklist:

[ ] CHANGELOG.md was updated, if applicable
[ ] Add an option to hide unlisted emotes
[ ] Implement the 7TV EventAPI
[ ] Adjust scaling of emotes
[ ] Append instructions to building docs regarding WebP
[ ] Add the 4x CDN link in the right click context
[ ] Use data-driven CDN URLs
[ ] Use the REST API instead of GQL for fetching emotes
[x] Support zero-width emotes

Description

Implementing 7TV (https://7tv.app) as a new Emote Provider in Chatterino2. The changes include adding a new provider to fetch global emotes, channel emotes and user badges from the 7TV API.
This pull request mirrors the SevenTV/chatterino7 fork maintained by myself and @zneix, while backtracking a few changes (such as versioning and workflows) to upstream.

Hi, I was wondering what is preventing this minimal support from being merged into master?

Hi, I was wondering what is preventing this minimal support from being merged into master?

Most of the list that you see in the pr description. As far as I'm aware, anatole still needs to add REST to 7tv (which I believe is coming in v3).

Hi, I was wondering what is preventing this minimal support from being merged into master?

I believe since minimal support already exists, the currently intended approach is to only merge full support into master.

@Felanbird Correct me if I am wrong, but currently there is no support for seeing or emote completing 7tv emotes in the master branch.
Besides the above improvement todos I don't see why the current code would not be great to have already merged so people can see these emotes. If you could expand what needs to be performed in the todo list that would be great. I know the websocket / event api allows for live changes instead of having to reload emotes correct? And most other things could be done in a follow up PR as people have time.

Correct me if I am wrong, but currently there is no support for seeing or emote completing 7tv emotes in the master branch.

Just use the 7TV fork if you want the emotes in chatterino, there is no reason to implement rudimentary support for a soon deprecated API in the main version until 7TV itself has released their v3 API, which if I understood everything correctly is basically a full overhaul and supposed to be a definitive and long term version where implementing proper support for it is actually worth the effort.

Thanks for the context. I am always for upstreaming so that things are easier down the line. In terms of the V3 api you mean this GQL work in progress one and REST work in progress one? The chatterino7 also uses the V2 GQL so the same update to V3 would be require / applicable to both.
My specific questions about the todo list:

Add an option to hide unlisted emotes - This refers to checking is the visibility_simple has UNLISTED in it, if so then don't show it. Would need to be implemented correct?
Implement the 7TV EventAPI - Isn't necessary to have support but is instead a more ideal implementation? Not sure about benefit over GQL or REST.
Adjust scaling of emotes - When I built I didn't see any issue, is this for non-regular sizes?
Append instructions to building docs regarding WebP - What details need to be added?
Add the 4x CDN link in the right click context - https://github.com/SevenTV/chatterino7/pull/62
Use data-driven CDN URLs - The current url https://cdn.7tv.app/emote/%1/%2 matches what the chrome client does. Not sure what this refers to.
Use the REST API instead of GQL for fetching emote - Isn't applicable in this PR any more since the new api is GQL based, and the chatterino7 uses the GQLv2.

I'm just too lazy to work on this, that's all.

Thanks for the context. I am always for upstreaming so that things are easier down the line. In terms of the V3 api you mean this GQL work in progress one and REST work in progress one? The chatterino7 also uses the V2 GQL so the same update to V3 would be require / applicable to both.
My specific questions about the todo list:

Add an option to hide unlisted emotes - This refers to checking is the visibility_simple has UNLISTED in it, if so then don't show it. Would need to be implemented correct?
Implement the 7TV EventAPI - Isn't necessary to have support but is instead a more ideal implementation? Not sure about benefit over GQL or REST.
Adjust scaling of emotes - When I built I didn't see any issue, is this for non-regular sizes?
Append instructions to building docs regarding WebP - What details need to be added?
Add the 4x CDN link in the right click context - Added 4x links to open/copy emote link menu. SevenTV/chatterino7#62
Use data-driven CDN URLs - The current url https://cdn.7tv.app/emote/%1/%2 matches what the chrome client does. Not sure what this refers to.
Use the REST API instead of GQL for fetching emote - Isn't applicable in this PR any more since the new api is GQL based, and the chatterino7 uses the GQLv2.

The current status for upstreaming is pending at least some of the tasks on this PR draft.

Unlisted emotes are those that don't quite pass standards for safety, but aren't bad enough to be completely deleted. (i.e sexually suggestive, but not porn); these can still be added to channels, so there could be situations where a streamer open a chat on screen and show potentially tos content. Ideally the option to hide these can be auto-enabled with streamer mode as well.
I marked the EventAPI implementation as optional, since it is more QoL than required. It means chatterino users wouldn't need to F5 when an emote is added/removed in channels
I believe this was mostly about missing the 4x size (though there is also the issue that 2x and 3x sizes are not correct on the 7TV cdn, which is something that will be fixed alongside API V3)
Data-driven CDN URLs means using the urls provided by the API response, rather than hardcoding the cdn's base url. This is mainly for future proofing for some planned features.
The usage of GQL in chat client applications isn't recommended, as it lacks caching and is typically slower in v2. The plan has been to switch SevenTV/chatterino7 to the REST API, though no one has taken on that task as of yet.

Also to emphasize this, the lack of progress on upstreaming isn't the fault of the c2 maintainers. This is a draft, because it's uncomplete, and a draft doesn't get reviewed.

Great, thanks so much for all the clarifications. To be upfront, I wanted to spend time to help, thus I simply wanted clarification on what I can do to complete the PR, and your explanations really give exactly what needs to be done, so thanks so much for that. I have started working on the changes you have mentioned on this branch:
https://github.com/goldbattle/chatterino2/tree/to-upstream
What is included so far:

Handle zero width emotes https://github.com/SevenTV/chatterino7/pull/36
Emote searching from https://github.com/SevenTV/chatterino7/pull/64
Initial 4x code from https://github.com/SevenTV/chatterino7/pull/62
Fixed scaling of emotes and properly handle 4x emotes  https://github.com/SevenTV/chatterino7/commit/403b947b6351f2c6bc6cf3f756e536cdfc00ab96
Handle unapproved / hidden emotes from displaying https://github.com/SevenTV/chatterino2/commit/125efefb6cea6a745206cdf469aef707cd19ab75 and https://github.com/SevenTV/chatterino2/commit/42c9ff02d464479dfed438a03c8a303ec65aad9e

From your description I plan to work on the following:

Add to my current unlisted emote logic to allow for seeing of unlisted emotes (default is not seeing them)
Switch to the REST api endpoint instead of the GQL one
Use urls from the API endpoint (once REST is used, this seems straight forward)

Questions:

Should global emotes always be shown (my current logic)? or should their visibility also be checked?
Should I create another PR to this repo once these three points are completed, or try to merge to the fork repo? (my initial PR addressing the above was closed on the c7 repo)

You should PR those changes to the chatterino7 branch on the fork, the to-upstream branch is heavily cherrypicked and require some additional care to merge. (i.e ensuring that the versioning & other changes are not tracked)

Should global emotes always be shown (my current logic)? or should their visibility also be checked?
Global emotes would never also be unlisted, so you can leave whatever behavior is produced by your implementation - it won't matter.

@goldbattle

You should PR those changes to the chatterino7 branch on the fork, the to-upstream branch is heavily cherrypicked and require some additional care to merge. (i.e ensuring that the versioning & other changes are not tracked)

I pulled and implemented everything required for minimal support for upstreaming 7tv integration into my up-stream branch since upstreaming support is my goal. I can make some PRs to the chatterino7 so they can get those changes also, but this doesn't address the former. Do you have a recommendation for this?
@AnatoleAM


False positive : db2jcc-1.4.2.jar reported as cpe:/a:ibm:db2:1.4.2

False positive :

reported by dependency-check-maven 3.0.1
on library db2jcc-1.4.2.jar
reported as cpe:/a:ibm:db2:1.4.2

db2jcc-1.4.2.jar (cpe:/a:ibm:db2:1.4.2, cn.guoyukun.jdbc:db2jcc:1.4.2) : CVE...
<dependency>
    <groupId>cn.guoyukun.jdbc</groupId>
    <artifactId>db2jcc</artifactId>
    <version>1.4.2</version>
</dependency>

https://mvnrepository.com/artifact/cn.guoyukun.jdbc/db2jcc/1.4.2

I tried to work on this FP and it appears the JAR is no longer available? mvnrepository indicates this is hosted on Central: https://mvnrepository.com/artifact/cn.guoyukun.jdbc/db2jcc_license_cu/1.4.2
However, the link to download the JAR returns a 404.

Indeed, how would you like to proceed ?

use a copy available on another site ?
for example : http://repo.spring.io/list/plugins-release/cn/guoyukun/jdbc/db2jcc/1.4.2/
use a copy of the jar, that I can provide to you ?
ignore the feedback due to this 404 ?

The SHA1 sum of the jar in my environement match the one on spring.io :
92382d35b3f0d04d466a1b03a414e92f0a4f1aa5 *db2jcc-1.4.2.jar

Thanks for pointing me to the correct repo.


Fix typos in readme.md

Please be aware that this pull request was automatically created using gtf - a typo fixing bot
You should be able to merge this with no other problems.
In case the proposed changes do not make sense, I would be glad to hear about it.

Addressed in #11.


fix(macos/scan): parse networks when shifted

A line was shifted in airport output (cf scan-shifted.log) and it was breaking parser for this line. It is now fixed

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Warn users when using static configuration methods

Fixes # .
Did you remember to?

[ ] Add test case(s)
[ ] Update CHANGES.txt

We encourage pull requests that:

Add new features to TestNG (or)
Fix bugs in TestNG

If your pull request involves fixing SonarQube issues then we would suggest that you please discuss this with the
TestNG-dev before you spend time working on it.

@juherr - this PR is to add the warnings as discussed in https://github.com/cbeust/testng/issues/2294#issuecomment-688160294


validate fails to identify missing disk property

Related to testing in #145, after correcting the mount source_path issue, validate failed to identify that I had added a mount, but had not added a disk property to the config, which resulted in
  E: Error parsing configuration files:
      - applications./.platform.app.yaml: The application "platformify_cookies" is diskless but defines local mounts and such mounts require disks. This means the configuration has to have a "disk" key with an integer value greater or equal to 128.

when attempting to push the change.

This should also take into account things like missing service names, etc.
For example (submitted by @devicezero):
Add a service:
    mounts:
      "/foobar":
        source: "service"
        source_path: "foobar"

Pushing the code will fail validation on the git side with:
 E: Error parsing configuration files:
    - applications.config.yaml:express.mounts./foobar: When source is 'service', a service name is required. Set `service: <service name>` to specify the service that will provide the networked mount.


google-http-client : For controlling whether to pass authorization header in redirected request or not.

Apache http client and many other clients, by default allow authorization headers to be passed to redirected request.

Here I have added one parameter , "passAuthHeaderInRedirects" is added for enabling and disabling of passing authorization header in redirections.

I signed it


add possibility for customizing dnsconfig

I faced with the issue of extra latency withing communication with redis-sentinel cluster.
I was related to poor internal dns performance.
Such option allows to customize resolve.conf inside pod to use node-local-dns (https://kubernetes.io/docs/tasks/administer-cluster/nodelocaldns/#motivation)

Thanks @yurymuski !

@FZambia  thanks for the instant merge :)


Installing ChannelWorm vs channelworm

Both the main ChannelWorm directory and the channelworm subdirectory each have setup.py files.  Is a user supposed to install both of them?  It would be more conventional for the content of the channelworm setup.py file to be merged with the content of the ChannelWorm setup.py file and then just let ChannelWorm setup.py handle all installation.

The ChannelWorm setup.py already calls setup.py file from channelworm. For
those who just need to work with the Django app, setup.py from channelworm
would be enough.
It's possible to merge them, especially, for the testing procedure case.
On Fri, Dec 16, 2016 at 2:49 AM, Richard C Gerkin<EMAIL_ADDRESS>wrote:

Both the main ChannelWorm directory and the channelworm subdirectory each
have setup.py files. Is a user supposed to install both of them? It would
be more conventional for the content of the channelworm setup.py file to be
merged with the content of the ChannelWorm setup.py file and then just let
ChannelWorm setup.py handle all installation.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openworm/ChannelWorm/issues/188, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGWMgOD1k8CgyB6WaQE7ehZStaDN-ZbQks5rIcsYgaJpZM4LOrsR
.


Completion is not triggered on the follow-up lines still used to provide addresses for a To:/Cc:/... header

Hey @codybuell, following https://github.com/codybuell/cmp-lbdb/issues/7, I'm using the mail_header_only option that I like a lot.
But I realised that, maybe/likely related to my nvim setup, I have a non-optimal usage experience when I've a somewhat long list of addresses that are causing line returns, as the completion is only triggered when I'm on the line of the header.
In the example below, the completion doesn't seem to be triggered when I'm on the second line for the To:, with<EMAIL_ADDRESS>Thus, if I want to add a new address, I need to do it on the To: line, which is a bit suboptimal/counterintuitive.
Enabling completing after the headers till the next one, or to the body, if possible would be great.
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Cc:
Subject: Test

This is the body

Thanks!

Thanks for the suggestion @codybuell, but the issue with the mail line light, is that for some emails or mailing lists/recipients it's not appropriate to send email with long lines.
I've been also using for some timeformat=flowed emails, and while I think it helps a lot on things like smartphones or tiny/unusual screens, I feel it's also a bit the same: it has some drawbacks and some people/channel still prefer or need to get hard-wrapped emails, and unfortunately I've not been able to find yet a solution that seems to play nicely with everything and everyone, so I moved a bit away from this.
If it could be possible to come up with a clean/working way to address this it would be great, otherwise I can leave with it, it's clearly not a big deal.