HTTP tasks stuck scheduled, discrepancy between local dynomite/in memory

I've got conductor up and running with a local dynomite/redis instance running in localhost but HTTP tasks seem to be getting stuck in a scheduled state because the poll worker does not pick it up, even though I can see 1 pending HTTP task through swagger:
http://localhost:8080/api/tasks/queue/sizes?taskType=HTTP
{
  "HTTP": 1
}

Polling worker continually prints this while the task is in queue:
18570 [pool-7-thread-1] DEBUG com.netflix.conductor.core.execution.tasks.SystemTaskWorkerCoordinator  - Polling for HTTP, got 0 tasks
The thing is, that the same workflow gets picked up and started using the db=MEMORY configuration
16530 [pool-7-thread-1] DEBUG com.netflix.conductor.core.execution.tasks.SystemTaskWorkerCoordinator  - Polling for HTTP, got 1 tasks
16530 [pool-7-thread-1] DEBUG com.netflix.conductor.core.execution.tasks.SystemTaskWorkerCoordinator  - Task: 9f446691-e872-4c5f-97d0-9a2e32b80890 of type: HTTP being sent to the workflow executor

workflow definition:
{
  "name": "kitchensink",
  "description": "test workflow",
  "version": 2,
  "tasks": [
    {
      "name": "task_2",
      "taskReferenceName": "t2",
      "inputParameters": {
        "http_request": {
          "uri": "http://www.google.com",
          "method": "GET"
        }
      },
      "type": "HTTP",
      "taskDefinition": {
        "ownerApp": null,
        "createTime": null,
        "updateTime": null,
        "createdBy": null,
        "updatedBy": null,
        "name": "task_2",
        "description": "task_2",
        "retryCount": 1,
        "timeoutSeconds": 0,
        "inputKeys": [],
        "outputKeys": [],
        "timeoutPolicy": "TIME_OUT_WF",
        "retryLogic": "FIXED",
        "retryDelaySeconds": 60,
        "responseTimeoutSeconds": 3600,
        "concurrentExecLimit": null,
        "inputTemplate": {}
      }
    }
  ],
  "schemaVersion": 3
}

Redis config:
./redis-server --port 22122
Dynomite config:
dyn_o_mite:
  listen: <IP_ADDRESS>:8102
  dyn_listen: <IP_ADDRESS>:8101
  tokens: '101134286'
  servers:
  - <IP_ADDRESS>:22122:1
  data_store: 0
  mbuf_size: 16384
  max_msgs: 300000

server.properties:
#Dynomite Cluster details.
#format is host:port:rack separated by semicolon

workflow.dynomite.cluster.hosts=<IP_ADDRESS>:8102:us-east-1a

db=dynomite

# Dynomite cluster name
workflow.dynomite.cluster.name=dyno_cluster_name

# Maximum connections to redis/dynomite
workflow.dynomite.connection.maxConnsPerHost=31

# Namespace for the keys stored in Dynomite/Redis
workflow.namespace.prefix=conductor

# Namespace prefix for the dyno queues
workflow.namespace.queue.prefix=conductor_queues

# No. of threads allocated to dyno-queues (optional)
queues.dynomite.threads=10

# Non-quorum port used to connect to local redis.  Used by dyno-queues.
# When using redis directly, set this to the same port as redis server
# For Dynomite, this is 22122 by default or the local redis-server port used by Dynomite.
queues.dynomite.nonQuorum.port=22122

# Transport address to elasticsearch
workflow.elasticsearch.url=localhost:9300

# Name of the elasticsearch cluster
workflow.elasticsearch.index.name=conductor

@ndidi-did Please make sure your dynomite cluster name is correct. I'm not sure if you local dynomite cluster name is dyno_cluster_name (which is a placeholder in readme), but it is supposed to be replaced by your running cluster name. Please refer to docker implementation for example.
I'm closing this issue as it's been a while, please feel free to reopen if you still have questions.


There is an error after upadting Microsoft.Azure.DocumentDB.Core furthter than 1.9.1

While trying to register new user using this:
var result = await _userManager.CreateAsync(user, password);
I get this error:
Method not found: 'System.Threading.Tasks.Task1<Microsoft.Azure.Documents.Client.ResourceResponse1<Microsoft.Azure.Documents.Document>> Microsoft.Azure.Documents.Client.DocumentClient.CreateDocumentAsync(System.String, System.Object, Microsoft.Azure.Documents.Client.RequestOptions, Boolean)'.
It happens when Microsoft.Azure.DocumentDB.Core library is updated into 2.0.0 +

I've noticed this issue right when DocumentDB.Core 2.0 was released but I haven't found a solution yet.
I wasn't sure if it was perhaps a bug in the SDK itself but it's been out for a while now and I guess I should look into this again.

Unfortunately, this is showstopper at the moment. I am voting +1 for the fix!

The issue is that DocumentDB SDK 2.0 added new CancellationToken arguments to various methods which isn't backwards compatible.
The fastest fix would be to remove compatibility with < 2.0 versions.
Since our 2.0 beta release also has breaking changes I think there's no reason to make sure older DocumentDB SDK versions are supported.
I'll publish a new beta release today with DocumentDB SDK 2.0 compatibility but bear in mind that this beta is not compatible with v1.0 datasets of this library.
There is also some missing functionality with the new partitioning support.
For more information of the changes in the beta you can read the change logs here:
2.0.0-beta
2.0.0-beta2

https://www.nuget.org/packages/AspNetCore.Identity.DocumentDB/2.0.0-beta3


Howard County Public School System

Institution/School Name
Howard County Public School System
Instiution/School Website
HCPSS
Email Users
Please place an x between the square brackets if yes

[X] Students
[ ] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[X] High School or equivalent
[ ] Elementary School or equivalent

@24680FPIC I am sorry, I have to close the request without merging, because you made a mistake in a request and requested to merge too many files, most of them do not relate to the organization which you would like to add to this repository. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for the understanding!


Environment Page Redesign

Demo: https://www.loom.com/share/562c2806a7e1494bb6b0476543512854
Demo of new changes after addressing the comments:
https://www.loom.com/share/89eec13bb03e4544a12b4dff38c70ec5

@kouthar Ready for design review when you have a chance :)

First photo: card near the bottom - bottom right aligned
Second photo: demonstrating alignment explicitly, but obvi don't show the kebab button when the menu is open lol

Third and Fourth: card near the top/middle - top right aligned

Upd

Nice work as always Vivian! Couple of things:

Can we add some top and bottom padding to the card options when you click on the three dot menu? I recommend you check out the CSS on Figma
Can we remove the 'Admin Dashboard' text at the top?

Updated, you can check out the second loom video I added to the PR to see the changes

First photo: card near the bottom - bottom right aligned
Second photo: demonstrating alignment explicitly, but obvi don't show the kebab button when the menu is open lol

Third and Fourth: card near the top/middle - top right aligned


Update trace4cats-tail-sampling, ... to 0.12.0-RC1+188-05746b3c

Updates

io.janstenpickle:trace4cats-tail-sampling
io.janstenpickle:trace4cats-testkit

from 0.12.0-RC1+156-7ea07b63 to 0.12.0-RC1+188-05746b3c.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.janstenpickle" } ]

labels: library-update, semver-build-metadata

Superseded by #8.


Minor improvements

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (6570c77) to head (cb255e6).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #361   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           92        92
  Lines         2323      2323
  Branches       190       190
=========================================
  Hits          2323      2323

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Send updates to the user when their wishlisted product is back in stock

Issue Report
Title
Adding a feature of sending notifications to the user when their wishlisted product is back in stock
Description
Users often face issues that their wanted product is out of stock so we can create a system where we can send notifications to users when the products they wanted are back in stock.
Steps to Reproduce the Issue

Action 1: I added products to my wishlist.
Action 2: Waited for it to go out of stock.
Action 3: There was no notification as to when it was restored in stock.

Expected Behavior
The user should receive notifications when their wishlisted products are back in stock.
Actual Behavior
The item was simply back in stock without any notifications.

Thank you for taking the time to report this issue! We appreciate your help in improving our project.

hi please assign me this issue


time: unexpected error in Parse when parsing January 31st (day 031) using "002" day of year format

What version of Go are you using (go version)?

$ go version

go version go1.13.8 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"

What did you do?
I tried to parse a date with Year + YearDay + Hour format, using time.Parse().
My actual layout is 0600215
I need to parse dates in the especified format to get the corresponding time.Time structure.
Code in playground: https://play.golang.org/p/nhbg1xnT93L
The input parameter to time.Parse() function that shows the issue is: "1803114"
What did you expect to see?
The expected result is a time.Time structure with the date "2018-01-31 14:00:00", and no error.
What did you see instead?
Instead of a time.Time structure, I got a parsing error: parsing time "1803114": day out of range
This happens only for the 031 day of yaer, wich corresponds to January 31st, and it happens both for leap-years and non leap-years.
The rest of the days of year (001 to 030, and 032 to 365) works fine.

Ouch, thanks for reporting that.  I will send a patch.

Hi @ianlancetaylor,
Now that 1.14.1 is released I've noticed that this fix was not included. Is this considered a breaking change? should we not expect to see this released until 1.15?
Thanks!

This isn't a regression and it doesn't seem likely to affect very many people, it seems unlikely to lead to silen failures, and it's not to hard to work around.  Unless we hear of a good reason to backport it, I think it will wait until 1.15.

Thanks for answering.
As you've said, we have this issue but we've worked around it for the time being, I was just asking to check about it.


Improve Key Events

[ ] Improve styling of events (links related to event, see attached)
[ ] Return the trial status to accused level.

cc @kiphale


Add tax info on "New expense" email

Related to https://github.com/opencollective/opencollective/issues/6588
We're not surfacing the tax info on the new expense email.

Will look at this as part of https://github.com/opencollective/opencollective-api/pull/9529, since we need to edit the email to show the new item currency / FX rate infos.


[FR] read_feather without going through pandas

Currently we have
read_feather(path, nthreads=1):
    warnings.warn("Defaulting to Pandas implementation", UserWarning)
    port_frame = pandas.read_feather(path)
    ...

Would be nice to have more native way of loading feather into modin. Filling this issue to be notified when that happens.

Thanks @jangorecki for posting this feature request!
I'm going to flag as good first issue because it should be relatively straightforward to do.

Resolved via #292
Performance numbers posted here

Great, it was fast! thanks


LE Audio: Support for a minimum CCP client

The CCP client implementation right now is a fully-featured implementation, with support for everything, including multiple instances of TBS.
The minimum CCP client should only subscribe to e.g. Call State list (or perhaps the Current Calls list) and the control point, and then either only use GTBS, or in the case of a single TBS instance, just one TBS instance.
The Simple client should be enabled by a KConfig option, which may or may not set some other Kconfig options.

Zephyr-bt F2F: Already done in related PR - closing


No Rating plugin in Android expert after intallation

Godot version
4.1
Plugin version
v1.0.1
Phone information
Xiaomi 9T
Issue description
First thank you for the plugin but I didn't have a chance to test it.
I copied files from zip to your_project/android/plugins but no plugin appeared. Although iOS plugin showed up.
Should I copy something else?
Steps to reproduce

follow instruction
install android plugin
no plugin in android preset

Additional context
No response

That's odd, i just using it today and it just work. Can you download this example and try if it showing in android export:
https://github.com/kyoz/godot-native-rating/tree/main/example/godot_4

I tried it but no luck. Even iOS doesn't show the plugin in the preset. But I've noticed there is no android folder in the example.

Yup, you have to install Android Build Template your self, then there will be android/plugins folder for you to copy this plugin to.

Also don't forget to check this in export window:

When ever you change godot version, you have to delete the android folder and Install Android Build Template over again.

The same with this example project :( It works for iOS but Android.
Here is the video:
https://github.com/kyoz/godot-native-rating/assets/6623349/3b33519c-1eb2-42a3-9b87-65366d5d0458

I just try it. It work as it should. Are you sure you downloaded the correct version for the plugin. You should download android-template-4.1.zip
If it still not work, you should try delete android folder, run Project -> Install Android Build Template.... Then copy the plugin again.

It just isn't working. I did what you said - no luck. I deleted Android preset and created new one - same. I tried to rename folder as "Rating", "rating", leave it as is - doesn't work. Changed renderer from Compatibility to Mobile - same.
I don't know what else it can be!
I have Godot version 4.1.1

You no need to create a folder for the plugin or change compatibility mode, just paste extracted content of the plugin directly in android/plugins folder.
Here is a exactly build for 4.1.1.
android-template-4.1.1.zip

Yeah I know. I mean it can be anything so I tried all the stuff but nothing helped. I tried build for 4.1.1 but nothing changes.
Can you please show me your build.gradle may be that's the problem.

Just an example: I added another plugin files to android/plugins and the plugin immediately appeared in export preset

I though it was Godot version is error so i tried to install Godot 4.1.1 and the plugin still work normally (Tried with the example i send you before).

I don't think it's by build.gradle but here is it.
// Gradle build config for Godot Engine's Android port.
buildscript {
    apply from: 'config.gradle'

    repositories {
        google()
        mavenCentral()
    }
    dependencies {
        classpath libraries.androidGradlePlugin
        classpath libraries.kotlinGradlePlugin
    }
}

plugins {
    id 'com.android.application'
    id 'org.jetbrains.kotlin.android'
}

apply from: 'config.gradle'

allprojects {
    repositories {
        google()
        mavenCentral()

        // Godot user plugins custom maven repos
        String[] mavenRepos = getGodotPluginsMavenRepos()
        if (mavenRepos != null && mavenRepos.size() > 0) {
            for (String repoUrl : mavenRepos) {
                maven {
                    url repoUrl
                }
            }
        }
    }
}

configurations {
    // Initializes a placeholder for the devImplementation dependency configuration.
    devImplementation {}
}

dependencies {
    implementation libraries.kotlinStdLib
    implementation libraries.androidxFragment

    if (rootProject.findProject(":lib")) {
        implementation project(":lib")
    } else if (rootProject.findProject(":godot:lib")) {
        implementation project(":godot:lib")
    } else {
        // Godot gradle build mode. In this scenario this project is the only one around and the Godot
        // library is available through the pre-generated godot-lib.*.aar android archive files.
        debugImplementation fileTree(dir: 'libs/debug', include: ['*.jar', '*.aar'])
        devImplementation fileTree(dir: 'libs/dev', include: ['*.jar', '*.aar'])
        releaseImplementation fileTree(dir: 'libs/release', include: ['*.jar', '*.aar'])
    }

    // Godot user plugins remote dependencies
    String[] remoteDeps = getGodotPluginsRemoteBinaries()
    if (remoteDeps != null && remoteDeps.size() > 0) {
        for (String dep : remoteDeps) {
            implementation dep
        }
    }

    // Godot user plugins local dependencies
    String[] pluginsBinaries = getGodotPluginsLocalBinaries()
    if (pluginsBinaries != null && pluginsBinaries.size() > 0) {
        implementation files(pluginsBinaries)
    }
}

android {
    compileSdkVersion versions.compileSdk
    buildToolsVersion versions.buildTools
    ndkVersion versions.ndkVersion

    compileOptions {
        sourceCompatibility versions.javaVersion
        targetCompatibility versions.javaVersion
    }

    kotlinOptions {
        jvmTarget = versions.javaVersion
    }

    assetPacks = [":assetPacks:installTime"]

    defaultConfig {
        // The default ignore pattern for the 'assets' directory includes hidden files and directories which are used by Godot projects.
        aaptOptions {
            ignoreAssetsPattern "!.svn:!.git:!.gitignore:!.ds_store:!*.scc:<dir>_*:!CVS:!thumbs.db:!picasa.ini:!*~"
        }

        ndk {
            String[] export_abi_list = getExportEnabledABIs()
            abiFilters export_abi_list
        }

        manifestPlaceholders = [godotEditorVersion: getGodotEditorVersion()]

        // Feel free to modify the application id to your own.
        applicationId getExportPackageName()
        versionCode getExportVersionCode()
        versionName getExportVersionName()
        minSdkVersion getExportMinSdkVersion()
        targetSdkVersion getExportTargetSdkVersion()

        missingDimensionStrategy 'products', 'template'
    }

    lintOptions {
        abortOnError false
        disable 'MissingTranslation', 'UnusedResources'
    }

    ndkVersion versions.ndkVersion

    packagingOptions {
        exclude 'META-INF/LICENSE'
        exclude 'META-INF/NOTICE'

        // 'doNotStrip' is enabled for development within Android Studio
        if (shouldNotStrip()) {
            doNotStrip '**/*.so'
        }
    }

    signingConfigs {
        debug {
            if (hasCustomDebugKeystore()) {
                storeFile new File(getDebugKeystoreFile())
                storePassword getDebugKeystorePassword()
                keyAlias getDebugKeyAlias()
                keyPassword getDebugKeystorePassword()
            }
        }

        release {
            File keystoreFile = new File(getReleaseKeystoreFile())
            if (keystoreFile.isFile()) {
                storeFile keystoreFile
                storePassword getReleaseKeystorePassword()
                keyAlias getReleaseKeyAlias()
                keyPassword getReleaseKeystorePassword()
            }
        }
    }

    buildTypes {

        debug {
            // Signing and zip-aligning are skipped for prebuilt builds, but
            // performed for Godot gradle builds.
            zipAlignEnabled shouldZipAlign()
            if (shouldSign()) {
                signingConfig signingConfigs.debug
            } else {
                signingConfig null
            }
        }

        dev {
            initWith debug
            // Signing and zip-aligning are skipped for prebuilt builds, but
            // performed for Godot gradle builds.
            zipAlignEnabled shouldZipAlign()
            if (shouldSign()) {
                signingConfig signingConfigs.debug
            } else {
                signingConfig null
            }
        }

        release {
            // Signing and zip-aligning are skipped for prebuilt builds, but
            // performed for Godot gradle builds.
            zipAlignEnabled shouldZipAlign()
            if (shouldSign()) {
                signingConfig signingConfigs.release
            } else {
                signingConfig null
            }
        }
    }

    sourceSets {
        main {
            manifest.srcFile 'AndroidManifest.xml'
            java.srcDirs = ['src']
            res.srcDirs = ['res']
            aidl.srcDirs = ['aidl']
            assets.srcDirs = ['assets']
        }
        debug.jniLibs.srcDirs = ['libs/debug', 'libs/debug/vulkan_validation_layers']
        dev.jniLibs.srcDirs = ['libs/dev']
        release.jniLibs.srcDirs = ['libs/release']
    }

    applicationVariants.all { variant ->
        variant.outputs.all { output ->
            output.outputFileName = "android_${variant.name}.apk"
        }
    }
}

task copyAndRenameDebugApk(type: Copy) {
    from "$buildDir/outputs/apk/debug/android_debug.apk"
    into getExportPath()
    rename "android_debug.apk", getExportFilename()
}

task copyAndRenameDevApk(type: Copy) {
    from "$buildDir/outputs/apk/dev/android_dev.apk"
    into getExportPath()
    rename "android_dev.apk", getExportFilename()
}

task copyAndRenameReleaseApk(type: Copy) {
    from "$buildDir/outputs/apk/release/android_release.apk"
    into getExportPath()
    rename "android_release.apk", getExportFilename()
}

task copyAndRenameDebugAab(type: Copy) {
    from "$buildDir/outputs/bundle/debug/build-debug.aab"
    into getExportPath()
    rename "build-debug.aab", getExportFilename()
}

task copyAndRenameDevAab(type: Copy) {
    from "$buildDir/outputs/bundle/dev/build-dev.aab"
    into getExportPath()
    rename "build-dev.aab", getExportFilename()
}

task copyAndRenameReleaseAab(type: Copy) {
    from "$buildDir/outputs/bundle/release/build-release.aab"
    into getExportPath()
    rename "build-release.aab", getExportFilename()
}

Can you send me the full directory path where your project folder is? Sometime that maybe cause the problem.

my project is here:
/Users/user/Documents/Godot projects/Game Folder
and your example project here:
/Users/user/Downloads/Example

Unfortunately, i still can't reprocedure your problem.Tried on all of my pc, laptop with various of Godot version but still doesn't face that issue. Are you tried other godot versions?

I will try, thanks! It's really weird because other plugins work just like that.
Can you please show me what the whole path to the plugin looks like in your case? Do you rename the extracted folder or something?

That's why i told it really weird. Here is my full path:

/Users/kyoz/Documents/Github/godot-plugins/godot-native-rating/example/godot_4/android/plugins

First i think maybe space in path maybe the cause the problem, but as you showed, your example are in:
/Users/user/Downloads/Example

So it may not because of space.  I'v used like every godot plugins in the github but i've never faced something like that @@.
Is there ary error in the Output when you restart Godot?

 I finally solved it. When you extract the archive it gives you a folder, so I put the folder inside plugins folder, even though I should have put only files from extracted folder inside plugins folder. Just like you showed it.
Thank you once again! You have useful plugins, I'll def test them all!
P.S what do you think about a plugin for some analytics like Firebase but for Godot 4?

No problem, bro. For analytics, because of something happen google policy in the past which is very confused (for both dev and user), so i no longer using that. I just using Admob metric.
And of course, the main reason is that the SDK have some gap. Mean, when you using several plugins (like admob, firebase...etc.) It can cause conflict and will unable to build.
I'm still watching for it. I'll rewrite and maintain as much as Godot plugin as i can.

Awesome!


Improper Error Handling in pkg\model\model_easyjson.go

Scanning our code with CxSAST we have a Low Security issue: Improper Error Handling in file pkg\model\model_easyjson.go, in the following method:

The outcome of Unmareshalling provided JSON is not being verified. It is actually being ignored.
If it is to ignore, then we can get rid of the _ on the unmarshalling assignment, and the issue should be fixed.

Not an issue, since the code is generated.
Closing the bug

Not an issue, since the code is generated.
Closing the bug


BUG IN CLIENT OF LIBDISPATCH: dispatch_sync called on queue already owned by current thread

Hello,
First of all thanks for the library :)
I am seeing crashes with this message in Firabase crashlytics:
"BUG IN CLIENT OF LIBDISPATCH: dispatch_sync called on queue already owned by current thread"
Any suggestions ? Trace shows SRHubConnection.registerCallback as the problem.
Best

Zsign technical support:
Apple signature tool supporting ios15 and ios16
Support the removal of dylib dynamic library
Completely free, permanently free!
https://urlzs.com/44zCx


ui-sref not working with ba-close

I tried to upgrade to 2.0, but I have links inside a ba-panel that are not working anymore.
Clicking the link should open a new page and close the panel, so it had ba-close together with ui-sref.
This worked fine before but does not work now.
Removing the ba-close allows to change page but the panel doesn't close.
Leaving the  ba-close and ui-sref allows to close the panel, but the page does not change.
This is written in the doc but I think it's not working at all:

Links (any element with an href or ui-sref attribute) inside panels automatically close the panel when clicked.

Also if you change this behaviour, it should be written in the migration doc. And then what is the solution if we want to change page and to keep the panel open?

@Seb- Please provide an example of your use case.
The following use case is working in the docs:

Resize browser to match small device
Click Show Components at the top
Click a link inside the panel

The panel will close and navigate to the desired link.

Here is a snippet:
 TITLE_BAR.MENU
<ba-panel id="menuPanel" position="left" ba-swipe-close="left">
			 MENU.HOME
			 MENU.ABOUT
</ba-panel>

The panel doesn't close when clicking the links.

Not sure if you just had a typo, but did you try zf-close or ba-close?  Please provide the markup that isn't working and I'll see if I can figure this out.  If you can create a jsfiddle or other online example that would help as well.
Your current example is working as expected.  Adding ba-close="" to the  tags should cause the panel to close when clicking the links.  Adding ba-close="" to the <ba-panel> would cause it to close clicking anywhere inside the panel.  Please also try removing ba-swipe-close="left" when testing as that directive may be interfering with the others.

You can test it with this: https://github.com/Seb-/test-for-angular-base-apps

Note that the example has been stripped too much from my original project, and adding ba-close to the links there will indeed close the panel. I will need to re-add my features progressively to notice at which points the links start to fail with ba-close.
The example still shows that the panel does not auto-close when a link is clicked, which is the opposite of what the doc says.

Following more testing and all things aside similar, this works:
<ba-panel id="menuPanel">
			x

Home

About

		</ba-panel>

and this doesn't (as soon as there is another tag inside the link):
<ba-panel id="menuPanel">
			x

Home

About

		</ba-panel>

This was working fine before version 2.

@Seb- you're missing the closing  tag on your home link

Sorry I guess I deleted too much while cleaning up, but it doesn't change the issue though, adding it won't work better, and the other link doesn't work either.

dang, it's never that easy... I'll look into it.

@Seb- I found the issue.  The original fix only fixes the case when the clicked element has the href attribute.  In your example, the parent of the clicked element is the one with the href attribute.

Thank you, ba-close works as intended now.
This note in the doc is still wrong though:

Links (any element with an href or ui-sref attribute) inside panels automatically close the panel when clicked.


[Update] - update format of papers in section of Prompt Engineering

Detailed Description

add github and project page link for these papers
format modification refering other sections

Additional Information
Deadline (if any): Before Monday

As mentioned in #166 , "All papers in this section have undergone full-text indexing, and only papers with a GitHub link are open-source."
I will close this issue. If necessary, I or other contributors would open it again.  @Cominclip


Replace deprecated call to std::{i32, usize, i16, u8} and of max/min_…

…value

Coverage decreased (-0.01%) to 83.515% when pulling a8cf4cfe0988d50502de3e45154dc7b0dc115e57 on remilauzier:master into e09f555d279e7d4dbd0063f072d57695dcc4da7e on xiph:master.


Removing empty values from role template rules

Role template rules can have emtpy slices in them, unlike the rules used by normal rbac objects (roles, cluster roles). We remove these empty slices in order to ensure a fair comparison with the rbac objects later on.

Related to, but does not necessarily resolve, #36702.
Background
When we create a Role Template, we create Roles/Cluster roles which actually grant the permissions defined in the Role Template. We also have handlers which check to ensure that the downstream roles/cluster roles have the same rules as defined in the Role Template. This way, if you change the Role Template, these permissions are applied to active users (who have an assignment to these Roles).
Core Issue
K8s RBAC types specify omitempty tags on various fields for RBAC rules (APIGroups, Resources, ResourceNames, NonResourceUrls). This causes the value to be nil - even if an empty string slice (of length 0) is specified on input. However, we don't appear to define this in the CRDS that we specify in k8s (we do specify inside of rancher, but not inside of k8s). Because of this, when Role Templates are created with empty slices (rather than not-specifyed nil values), our controller sees the downstream roles as always having different rules than the upstream role and is unable to update them to have the same rules. This causes a massive re-sync effort every 10 hours when lasso forces a refresh.
Solution
The 'minimal code' solution to this is simply to remove the offending fields from the Rules defined in the RoleTemplates. This doesn't change the RoleTemplate object in the K8s api, but just allows us to us reflect.DeepEqual to compare the role template rules and the rules that are already present in the downstream RBAC objects.
Notes

I would have preferred to have the objects well structured or properly created in the first place. This would have likely required some form of fix script or more complex solution, so I defaulted to what was simple


process.env.ENV is not defined, assuming 'prod' env

Open teamdaily in Chrome and open dev tools.

Nevermind, caused by Grammarly extension.


Abount Bulk_RNA_matrix.csv file

Excuse the interruption, but could you confirm if the Bulk_RNA_matrix.csv file contains count data? Thank you for your response.

Excuse the interruption, but could you confirm if the Bulk_RNA_matrix.csv file contains count data? Thank you for your response.

Hi DingfanGuo,
Bulk_RNA_matrix.csv is a normalised count matrix.
Thanks,
Ateeq


require doesn't assume index.js

In Node, require('./somedirectory') will attempt to load somedirectory/index.js and then somedirectory/index.json. Should parcel be doing something similar?
require('./somedirectory')
// attempt "somedirectory"
// attempt "somedirectory.js"
// attempt "somedirectory.json"
// attempt "somedirectory/index.js"
// attempt "somedirectory/index.json"
// throw error

Yes, it already does.

🤔 At some point I thought it didn't - must've been some kind of mistake on my part. Whoops, sorry!

Oh, I see why. parcel requires a trailing slash, which isn't Node behaviour iirc...

parcel requires a trailing slash

That's not correct. Here is a working example:
// index.js
console.log(require('./sub'))

// sub/index.js
module.exports = 'foobar'

$ node -e "$(prcl index.js)"
foobar

Could you provide an example of something that doesn't match node's behavior?

Are you on Windows, by any chance? I don't use it very frequently, so the path behavior might not be correct.

Nope, not on Windows. Fedora user here.
Ugh, suddenly I can't reproduce it - sorry 😜 If I encounter it again I'll mention it.

No problem. Let me know if you have other issues :)


HW1: Update ss5945.md with README links and project documents

Adding link to ss5945's GitHub profile README.

lgtm


Deepspeed integration for XPU/ccl

Deepspeed has exposed  support to different backends .So for using deepspeed through accelerate on xpus, the ccl backend is required. Also mpich support was added in deepspeed before (by me) so that can be added too for multinode.
@sgugger @muellerzr

cc @pacman100

@pacman100  could you take a look ? Thanks

@sgugger  any updates/reviews on this PR ? Thanks

Still waiting for @pacman100 to take a look.

@pacman100 You will need to merge the PR if you are happy with it ;-)


no output in jupyter when the kernel is julia1.0

After I run a cell, no output appears.

On Windows?

Windows 10

Duplicate of #693.

NO, I scroll back to Julia 0.6.4.

Please file a new issue, then, specify Julia 0.6, give your system information and the output of Pkg.build("IJulia"), and try the https://github.com/JuliaLang/IJulia.jl#debugging-ijulia-problems suggestion.


start error

/opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/bin/hue syncdb --noinput
Could not find platform dependent libraries <exec_prefix>
Consider setting $PYTHONHOME to [:<exec_prefix>]
Traceback (most recent call last):
File "/opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/bin/hue", line 6, in
from pkg_resources import load_entry_point
File "/opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/lib/python2.7/site-packages/pkg_resources/init.py", line 18, in
from future import absolute_import
ImportError: No module named future
'[' -z '' ']'
MERGE_FLAG=--merge
'[' 5 -ge 5 ']'
/opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/bin/hue migrate --merge
Could not find platform dependent libraries <exec_prefix>
Consider setting $PYTHONHOME to [:<exec_prefix>]
Traceback (most recent call last):
File "/opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/bin/hue", line 6, in
from pkg_resources import load_entry_point
File "/opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/lib/python2.7/site-packages/pkg_resources/init.py", line 18, in
from future import absolute_import
ImportError: No module named future
'[' dumpdata = runcpserver ']'
'[' syncdb = runcpserver ']'
exec /opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/bin/hue runcpserver
Could not find platform dependent libraries <exec_prefix>
Consider setting $PYTHONHOME to [:<exec_prefix>]
Traceback (most recent call last):
File "/opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/bin/hue", line 6, in
from pkg_resources import load_entry_point
File "/opt/cloudera/parcels/CDH-5.9.0-1.cdh5.9.0.p0.23/lib/hue/build/env/lib/python2.7/site-packages/pkg_resources/init.py", line 18, in
from future import absolute_import
ImportError: No module named future

find

CDH-5.9.0, this is very old and Cloudera. Any luck with using latest upstream release/Docker instead?
PS: maybe your Python version is not >= 2.7


RxDialog oneliners

For your Q&D needs, it is possible to spin an observable dialog in a single line. This is helpful to quickly create a picker for yes/no, date, number, decimal or single element in a list. i.e. button click flatmap into confirmation dialog flatmap into execute action.
Is that within scope of this library, or as a helper would it live outside?

My two cents: out of scope for this library.
To me, it seems like a fairly large task to make a proper Rx dialog library. There's a lot of depth to dialogs, if you're dealing with them in a general sense, since you can do just about anything with them - they're just floating windows.

Okay, assume my pitch is out and I put my weekends on another library. What does a proper stock Rx dialog library need? Listeners for each of the mentioned types, styling is still done at theme level, anything else? Those two I've already covered before.

I do so many crazy things with dialogs that I don't think a library could ever cover all the use cases. E.g., some dialogs have custom layouts inside of them which have lots of logic/control outside the bounds of any normal dialog listener.
If I were to make a library I'd make it tightly focused on some very common use cases, such as displaying a title/message/buttons. Once you start expanding beyond that (into things like adapters and custom views) it gets a lot more complex.

Yes, if you want the custom ones. I was thinking something more stock for those cases, with stock UI components like a simple picker. Without styling and only allowing changes to title, message and elements/boundaries, most of them are trivial to wire.
http://developer.android.com/images/ui/dialogs.png
http://developer.android.com/images/ui/dialogs_regions.png
http://developer.android.com/images/ui/dialog_checkboxes.png
http://developer.android.com/images/ui/dialog_custom.png
http://developer.android.com/images/ui/pickers.png
http://i.stack.imgur.com/k5xNW.png


Does it check for issues made by other people?

Also setup Github or TravisCI to automate the deployment when a new issue is added/removed

Fixed it with this
5d9c582


[light-client] Guard against circular dependencies

In the case in which light client A is configured to sync to light client B while light client B is configured sync against Light Client A, the circular dependency might cause some weird behaviour depending on the additional peer lists of the respective light clients. This is a very rare case and would require pathological manual configuration of both. This issue is just meant to document this rare case and potentially serve as a gateway to new contributors 🙏

Not sure I understand this - what does it mean for one client to sync against another?

@brapse can you clarify? Or should we close?

Yes it's an edge case and can be closed.


Revert "Enable Dataflow managed service for Python tests"

Reverts apache/beam#33134
I accidentally merged this, but it was not ready - I meant to merge https://github.com/apache/beam/pull/33150

R: @ahmedabu98

LGTM.


bump PyTorch version to 1.11

This bumps PyTorch to 1.11 and fixes 3 test failures. The bump is required to enable the libtorch_ops fallback due to DLPack version incompatibilities.
QAT training has its own fuse_modules version (fuse_modules_qat) in PyTorch, so I changed the test.
Two amendments to the front end:

searchsorted gets more (optional) parameters to its signature,
There is a sub variant with alpha (a - alpha * b). PyTorch rewrites rsub with alpha to this, but we ignored it. Now we handle sub with alpha.

Thank you,  @masahi for getting me started with the bump and pointing out the test failures. Any errors are my own.

Caffe2 has been dropped from PyTorch, which is what we are getting here: https://github.com/pytorch/pytorch/issues/67151

I think I could need a hint how to proceed, given that this is likely a major issue for all caffe2 use in TVM.

Unless there is a standalone way to install caffe2, then I think removing caffe2 support entirely is the only way forward. Our caffe2 frontend hasn't been updated for years, so I don't think people would object...

We can propose dropping caffe2 support to the community next week. In the meantime, we can remove caffe2 bits from CI:

Remove https://github.com/apache/tvm/blob/main/docker/Dockerfile.ci_gpu#L80-L81
Remove https://github.com/apache/tvm/blob/92a80e93446331780cfa81452b164862845e7c65/tests/scripts/task_python_frontend.sh#L60
Probably need to remove https://github.com/apache/tvm/blob/44549e623433dd10d9e97e442ef529fb44c46c14/gallery/how_to/compile_models/from_caffe2.py

Also ci-qemu has failed to install PT 1.11 for some version conflict reason https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-10794/1/pipeline/67. But I don't know why it needs to install pytorch... probably it only needs onnx. So I suggest decoupling pytorch install from ubuntu_install_onnx.sh and create ubuntu_install_pytorch.sh.

Also ci-qemu has failed to install PT 1.11 for some version conflict reason

Ohoh. I think this is because Python 3.6 is EOL upstream and PyTorch doesn't support it anymore...
This is a larger can of worms than I had hoped for. :slightly_smiling_face:

I thought we are now running Python 3.7, see

https://github.com/apache/tvm/pull/10247
https://github.com/apache/tvm/pull/10654

but maybe the qemu image hasn't been updated yet.

I thought we are now running Python 3.7, see

[docker] Update CI to Python 3.7 and Ubuntu 18 #10247
[CI] Update docker images to bring Python 3.7 and TensorFlow 2.6 #10654

but maybe the qemu image hasn't been updated yet.

On this, it looks like ci_qemu (https://github.com/apache/tvm/blob/main/docker/Dockerfile.ci_qemu) is not installing Python using the commons script: https://github.com/apache/tvm/blob/main/docker/install/ubuntu1804_install_python.sh.
This probably needs to be fixed in a separate PR. Do you want to send that fix? (Asking just because I won’t be able to take this for the next ~two weeks).

Note that probably we need to update the ci image version in https://github.com/apache/tvm/blob/main/Jenkinsfile#L54 as well. For that, we need to wait until a new nightly image containing the update from your previous PR is pushed to https://hub.docker.com/r/tlcpackstaging/ci_qemu/tags.
So please wait another day before resuming PT 1.11 update work, or remove pytorch install from ubuntu_install_onnx.sh.

@masahi Thank you for merging the qemu Python bump and the advice. I think waiting for it to show up might be the cleanest option, especially given that I still need to learn so much about how the CI works. :)

@masahi OK, so now we're part of the nightly with the qemu update. How would I get a version tag that is useful for bumping the version in the Jenkinsfile?

Yeah, so the way it works is as follows

We update the ci-docker-staging branch https://github.com/apache/tvm/tree/ci-docker-staging to point Jenkinsfile to the new image in tlcpackstaging.
After the CI passes, we retag the nightly image as v0.83 etc and push the image to tlcpack dockerhub.
Update Jenkinsfile in main to point to the new image on tlcpack, and send a PR to merge this change.

Note that the above process needs to happen for every CI image update. Right now we are in the middle of ci-qemu update, but after we merge this PR to update PT, we need to go through the same exercises to update ci-gpu.
I've been trying to run a CI job on ci-docker-staging, I keep getting various errors but hopefully I can pass it in a few hours.

Thank you @masahi . So if I understand this right, the next step is something you need to do? I'd appreciate a shout if I can proceed here or help other bits along.

Yes, pushing changes to ci-docker-staging, or pushing a new image to tlcpack dockerhub need to be done by a committer. No worry, I've done this many times. But since we need to wait for at least two CI runs (one for ci-docker-staging to test the new image, another for main to actually update the image), things won't be done by today.

Yes, pushing changes to ci-docker-staging, or pushing a new image to tlcpack dockerhub need to be done by a committer. No worry, I've done this many times. But since we need to wait for at least two CI runs (one for ci-docker-staging to test the new image, another for main to actually update the image), things won't be done by today.

Just a heads up that you're likely to see the issue with updated containers - https://github.com/apache/tvm/issues/10696.

Yeah I hit that error once a couple of hours before, fortunately the ongoing run https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/238/pipeline/ didn't hit it.

@t-vi the PR https://github.com/apache/tvm/pull/10814 should unblock this work.

https://github.com/apache/tvm/pull/10815

https://github.com/apache/tvm/pull/10815 was merged so we can resume the work on this PR. I restarted a CI job by closing / reopening (there should be a better way than this...)

[2022-03-29T23:00:39.045Z] unknown parent image ID sha256:f75815a47f249990da41ca0e349ede16f8710bd2d573de74a9afbd1a9b528055
in the docker build. I would not even know what to look at here... :confused:

Hopefully it is just a flaky issue, since the error came from the unrelated image ci-hexagon

I'm not sure whether it is flakiness or merging main, but it seems to be past that bit now. Hopefully, if there are more failures, it'll be something I can look into to fix. Thank you for all your help @masahi !

you got an error in one of tests because the CI environment is still using PT 1.10. We get to use 1.11 at the last step of https://github.com/apache/tvm/pull/10794#issuecomment-1081579377
So can you revert that change or use different code paths depending on versions?

I think I have the error from a premature 1.12 compat change, I'm fixing this right now but want to test locally.

Another unknown parent image ID error https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-10794/6/pipeline
cc @driazati (the other one was https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-10794/4/pipeline/)

All green. :slightly_smiling_face:

Thank you, @masahi for merging and helping me. So next we would need to update the docker gpu image used before I can return to enabling the test that needs PyTorch 1.11?

Yes, the next step is to wait until the nightly image appears in https://hub.docker.com/r/tlcpackstaging/ci_gpu/tags. It will happen about 12h later.

I started a manual build so we don't have to wait: https://ci.tlcpack.ai/job/docker-images-ci/job/daily-docker-image-rebuild/272/
Usually takes about an hour

To confirm here, as I did the update this time... the latest version of the images now contains PyTorch 1.11.
$ docker run -it --rm tlcpack/ci-gpu:v0.84 bash
root@6ba934f13b82:/# python3
Python 3.7.5 (default, Dec  9 2021, 17:04:37)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch
>>> torch.__version__
'1.11.0+cu102'
>>>

The update to PT 1.11 was already done in https://github.com/apache/tvm/pull/10849


Upgrade error

Please describe what you were doing when this error happened.
Specifications

OS: darwin
IPFS Desktop Version: 0.12.2
Electron Version: 8.2.5
Chrome Version: 80.0.3987.165

Error
Error: Command failed: npm install -g ipfs-npm
npm WARN deprecated<EMAIL_ADDRESS>request-promise has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>This module has moved and is now available at @hapi/boom. Please update your dependencies as this version is no longer maintained an may contain bugs and security issues.
npm WARN deprecated<EMAIL_ADDRESS>stable api reached
npm WARN deprecated<EMAIL_ADDRESS>This module has moved and is now available at @hapi/hoek. Please update your dependencies as this version is no longer maintained an may contain bugs and security issues.
npm ERR! Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t<EMAIL_ADDRESS>npm ERR!
npm ERR! xcrun: error: invalid active developer path (/Library/Developer/CommandLineTools), missing xcrun at: /Library/Developer/CommandLineTools/usr/bin/xcrun
npm ERR!
npm ERR! exited with error code: 1

npm ERR!      /Users/gmr/.npm/_logs/2020-08-20T21_14_58_652Z-debug.log

    at ChildProcess.exithandler (child_process.js:308:12)
    at ChildProcess.emit (events.js:210:5)
    at maybeClose (internal/child_process.js:1021:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5)

Thanks, @bloxster! We'll bring this up in our next weekly triage meeting.

Hi @bloxster -- can you please try disabling the 'npm on IPFS' experiment under the cube menu? This should take care of the issue in the immediacy. We're aware of the underlying issue, so I'll close this, but please let us know if you continue to have issues.


Update repositories.txt

@ArduinoBot

Hello @Khuuxuanngoc. The automated system here only operates on open pull requests. Since your submission was already accepted and the pull request merged, the bot won't pay any more attention to this pull request.
But this human is happy to provide assistance. Was there something you needed from the bot?

Hello @per1234 , I am very pleased to receive your feedback. I just upgraded the version of library.property to 1.0.3, because version 1.0.2 does not install additional libraries in the “depends” field. However, with version 1.0.1, it does.

With 1.0.1: depends=kxnTask,OneButton (=2.1.0)
With 1.0.2: depends=Makerlabvn_I2C_Motor_Driver,kxnTask (=1.0.0),OneButton (=2.1.0),LiquidCrystal_I2C (=1.1.4),DFRobotDFPlayerMini(=1.0.6),Dabble(=1.5.2)
With 1.0.3: depends=kxnTask,OneButton (=2.1.0),LiquidCrystal_I2C (=1.1.4),DFRobotDFPlayerMini (=1.0.6),Dabble (=1.5.2),Makerlabvn_I2C_Motor_Driver

Because I was eager to try the new version, I called “ArduinoBot” to check for me. I apologize for not using “ArduinoBot” correctly.
Best regards


allow buying multi fractured items

allow buying items like the base for my next crafting project
buy exact "Metadata/Items/Weapons/OneHandWeapons/Wands/Wand14"
  with fractured "SpellAddedFireDamage9"
  with fractured "GlobalFireSpellGemsLevel1_"
  with fractured "LocalCriticalMultiplier6"

Good idea. The limitation no longer makes sense now with recombinators.
I had a look and the only place where ~fracture is set to true is when buying. So we can just remove the check actually. I did that (crediting you for the suggestion in the commit message ;) ).


Hosting Webpart from O365 CDN unsuccessfull

Hi there!
I followed the instructions to publish my webpart to the O365 CDN.
Everything worked fine, activated the CDN, bundled and packed the solution and uploaded it to the appcatalog, but when I am trying to add the webpart to a site i get the following errors:
In Google Chrome Browser:
[SPLoaderError.loadComponentError]:
***Failed to load component "86113633-af30-48de-86b4-672a9d39f8ac" (xxxxWebPart).
Original error: ***Failed to load URL 'https://publiccdn.sharepointonline.com/xxxxxx/sites/apps/ClientSideAssets/656be2fa-23ca-4725-8b0c-d12d12430922/xxxx-web-part_47d8dbfc137ae36ba64e68c30e3466dc.js' for resource 'xxxxxx-web-part' in component '86113633-af30-48de-86b4-672a9d39f8ac' (xxxxxWebPart). There was a problem in the server.
In Edge Browser:
[SPLoaderError.loadComponentError]:
***Failed to load component "86113633-af30-48de-86b4-672a9d39f8ac" (xxxxWebPart).
Original error: ***Failed to load URL 'https://publiccdn.sharepointonline.com/xxxxx/sites/apps/ClientSideAssets/656be2fa-23ca-4725-8b0c-d12d12430922/xxxxxx_47d8dbfc137ae36ba64e68c30e3466dc.js' for resource 'xxxxx-web-part' in component '86113633-af30-48de-86b4-672a9d39f8ac' (xxxxWebPart). There was a network problem.
This may be a problem with a HTTPS certificate. Make sure you have the right certificate.
SPFx Version 1.5.0
What am I doing wrong?
Thanks for the help.

Dokumentdetails
⚠ Bearbeiten Sie diesen Abschnitt nicht. Er ist für die Verknüpfung von docs.microsoft.com zum GitHub-Artikel erforderlich.

ID: e911c86d-76ea-3a23-f98e-7ad7b804a408
Version Independent ID: 9c7748a8-3e63-19ed-4e69-15136603f302
Content: Hosten clientseitiger Webparts in Office 365 CDN (Hello Word, Teil 4)
Content Source: docs/spfx/web-parts/get-started/hosting-webpart-from-office-365-cdn.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

This is an unfortunate known issue which should be now getting resolved within a matter of days. If that's not the case, please let us know within a week.


feat: Add profile fragment viewModel

Fixes #22

LGTM 👍

Looks great 👍

@iamareebjamal Please review

PR updated


We should probably beef up our GUI test and do more development releases in the future...


HDBSCAN testing

We have a synthetically generated training and (anomaly) test data (attached herewith) for testing Tribuo HDBSCAN . We first trained a model with the python HDBSCAN and we predicted the anomalies from the synthetic test/anomaly data .
Then we used the same data to create a Tribuo HDBSCAN model following its standard workflow. (The crux of the code flow is attached herewith for ref.) and used the test data for predictions. However, Tribuo model did not predict any anomaly against the test data including when I played aound changing the  hdbscan parameters . All the predicted score was pretty low like 0.10809591 .. using the getScore() method from prediction.  My understanding from the Tribuo tutorial is that the low value predicted scores like 0.1, 0.2 etc are not anomalies and scores >=0.8 are anomalies . Is that correct?
Also, In general w.r.t Tribuo, I was able to test the Tribuo HDBSCAN model training and prediction working fine with the real world large size card holder labeled  dataset mentioned as a benchmark with Python HDBSCAN in the Tribuo HDBSCAN paper, and I was able to test its labeled anomalies working. Further , the smaller toy  dataset in the Tribuo repo is another thing that works fine.
Based on this,my question is what  might be the possible reasons that I am unable to get anomaly outputs from Tribuo from my test data Vis-a-vis Python HDBSCAN? Does Tribuo's HDBSCAN require more training data than its python counterpart? or Is it dependent on any data characteristics?
Attachments:-
train and test data
tribuo-syn-test-data.zip
crux of the hdbscan code flow.
tribuo_code_crux.txt

Is this with Tribuo v4.2? The way noise points are tagged at inference time has been changed in main as it wasn't consistently marking things as noise when they are far away from the original training points. If you've not already done this could you test it out on the current head of main?

@Craigacp  The one I have tested was in the current head of main only and not in v4.2 There is also another observation. According to the tutorial, the predictions with higher scores like 0.8 and above are mostly outliers . Also, per Tribuo, the labeling for the outlying points are to be 0. But , on the contrary , for very low prediction scores like 0.01, 0.02 etc which means it is not an outlier, I am getting labels as 0 in the above test which should not be the case. Pls. see the attached file of predictions on the test dataset with the last 2 columns where I have appended the prediction scores and the predicted label.
synthetic_data_runtime_stressed_resptime_5thJuly.csv
?

The scores are a function of which cluster exemplar the new point was assigned to or a pre-computed outlier score if it is assigned as noise, they are not a function of distance between that exemplar and the test point.
The inference implementation in Tribuo is approximate so it doesn't need to keep the whole dataset around at inference time, it computes distances to the cluster exemplars and assigns the label based on the closest exemplar. Which Python implementation are you comparing against?
@geoffreydstewart could you take a look at this prediction issue?

@Craigacp noted. I am testing  some more datasets. Will report any observed discrepancies if any In this regard. Thanks.

Hello, I can try the attached datasets to see if I observe any unexpected results. I'll be back from vacation next week and will look into this then.

@nsankar I have performed some tests using this dataset of 2500 records and 1013 features. Before considering predictions, first we should look at the model that is trained with this data. To do this, I prepared both a Python hdbscan and Tribuo Hdbscan notebook in this github repo here.
The Python model identifies only 2 clusters containing 5 points and 6 points respectively. The rest of the points are all outliers. The outlier scores for many of the outlier points have very low values which are not typical outlier scores for noise points.
The Tribuo model does not identify any clusters. All the points are determined to be outliers. Again, the outlier scores are mostly very low values which are not typical outlier scores for noise points. A model with characteristics like this will not be very useful when making predictions.
In general, I am not sure that the HDBSCAN algorithm is the best choice for this particular dataset. Perhaps you could try using KMeans and experiment with different values for k to see what clusters are established. Otherwise for outlier/anomaly detection, you may want to consider using a LibSVM anomaly detection as described here.

@geoffreydstewart  Thanks for the inputs. On one such similar data, I tuned the minclusterSize parameter using an optimizer such that the HDBSCAN optimizes (maximizes) distinct number of clusters. (like from coarse grained to fine grained distinct clusters with good boundaries). Before I did this, all the labels for my test were '0' labels indicating ots an outlier, but the scores were pretty low on a scale of 0 to 1 like you found above. However, after optimizing the minclustersize, all the scores became 0.999

@geoffreydstewart For performance,  what is the  max. safe  no. of parallel thread setting I can keep in the HDBSCAN trainer? based on your experiences, pls. suggest. thanks in advance.

@nsankar I have not experimented with extremely large values for numThreads when training HDBSCAN models. Generally, the most notable performance gains are achieved by increasing the number of threads up to the number of cores available on the test system. Beyond that, I haven't seen much evidence of significant gains.
Keep in mind that the only step of the algorithm being parallelized are the neighbour queries, while the rest of the algorithm is executed in a single thread. In Tribuo 4.3 we're adding a K-D Tree structure which should always out perform a brute force technique for neighbour queries. However, in practice we have observed that training HDBSCAN models on bigger machines with lots of cores, using a NeighboursBruteForce implementation actually performs better. This is due to the fact that data vectors are retained in the CPU cache which causes the next step of the algorithm to execute much faster. We are continuing to work on optimizations of this algorithm.
Hopefully these details assist with your optimization activities.

@geoffreydstewart @Craigacp
I was able to serialize the HDBSCAN transformed model with the below code. I didn't know how to deserialize it properly using the read object as below..
Could you pls. help me with the snippet for deserialization ? . Thanks in advance.
String model_save_path = "C:\\Users\\Lenovo\\Documents\\tribuo\\ss\\model_serialized";         String outputModelPath = model_save_path.replace("\\", "/");         final Path outputModelPath = Paths.get(outputModelPath,"SerModel.bin");        ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(outputModelPath.toFile()));         oos.writeObject(transformedModel);
Deserializarion ?
ObjectInputStream mois = new ObjectInputStream(new BufferedInputStream(new FileInputStream(modelPath.toFile()))) ?? = <?? > mois.readObject();
ObjectInputStream mois = new ObjectInputStream(new BufferedInputStream(new FileInputStream(modelPath.toFile())))
model = () mois.readObject();

Model serialization and deserialization is covered in the intro tutorial here - https://tribuo.org/learn/4.2/tutorials/irises-tribuo-v4.html#Loading-and-saving-models. You should load it in as a Model<?> then perform the appropriate Model.validate() check for your prediction type (in this case ClusterID).

Thanks @Craigacp


Fix syntax error in JavaScript code sample

There was an extra brace } in the JS code sample. I noticed it when I tried to implement speech systhesis by following instructions in this article.

@huaigulin : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


ISO Broadcaster fails silently on unsupported RTN/SDU_Interval combination

When a user selects a specific RTN/SDU_Interval combination in the ISO_BROADCAST sample, broacaster fails without an error message. The user doesn't know what has happened.
As @Thalley suggested, a short error message would be nice to have.
Note: tested on two nRF52840DK boards with the code of the following draft PR: https://github.com/zephyrproject-rtos/zephyr/pull/46187

The error seen is that no ISO data is seemingly received on the sync receiver, right @GatCode ?

Exactly, only the BigInfo is printed on the receiver side.
Additionally the output on the sender side is exactly the screenshot above but it should print Sending value ... every second.

Exactly, only the BigInfo is printed on the receiver side. Additionally the output on the sender side is exactly the screenshot above but it should print Sending value ... every second.

So at least the periodic advertisement seems to be fine. From your log, it seems that just the BIG sync never completes.


Authentication with zuliprc file

I prepared a constructor for it with some bullet points what I think will be required.

@zulipbot unlabel "in progress"

@zulipbot close.


Get feature attachment

ArcRest or ArcRestHelper
ArcREST
Version or date of download
5/20
Bug or Enhancement
Enhancement
Repo Steps or Enhancement details
The FeatureLayer Object has functions from adding, updating, and delete attachments from a single feature by referencing the objectid. A function to get an attachment would be great for transferring attachments from one feature to another.

@DShokes - check out the function addAttachments, deleteAttachments, updateAttachements, and listAttachments on a FeatureLayer object.

@achapkowski does listAttachments return the actual file? It looks like it's only the dictionary description.


Make mtk classes more specific to avoid CSS conflicts

Currently the editor classes are in the format .mtk${number} format, and are not very specific.
With the use with other libraries, this might conflict with generic css (i.e. .parent span {color: inherit}). Making it more specific, will avoid that and mtk classes should always be inside .monaco-editor as per my understanding.

Thank you!

Hey @cipacda, we had to revert this since it caused issues in a few cases: #106312
Did you run into a specific problem with the generated class names conflicting the class names from a library? If so, can you share more info about it. I'm just trying to understand if we to try re-applying the change this iteration

Hey @mjbvz ,
The issue that I encountered was that if there are other libraries in the application that for example will have a css of .container > span { color: red }, they will overwrite the CSS settings applied from the auto generated .mtk{number} classes as the other library will take precedence, having a more specific definition.
Thanks for looking into this,
Ciprian

Ok. That seems like a bad css rule to me. Even with this fix, I think any more specific rule would trigger same issue, such as .a .b > span { color: red; }

I agree it's a bad css rule, but if that happens in a 3rd party library there's no easy way to fix this (I got this issues when using vscode editor inside a rich text editor). If the vscode CSS classes are more specific it gives possibility to overcome this issue. I managed to find a workaround locally, but I think this could help others as well.

@cipacda But how did this rule .container > span { color: red } match editor tokens, since the HTML looks like this:

const myCar = new Car();

The spans with .mtk do not have a parent dom node with the class name .container in there.

My example was a generic one, sorry. And the actual issue I had was with something like .container span { color: red }, with container being a class from the parent rich text editor.
I mistyped the above css rule in hurry, my bad.


[5.7] Add whenEmpty + variants to Collection

I often find myself in need of executing actions with a collection if it is / is not empty. At the moment this is possible in several ways. For example:

Assign the collection to a variable, then reuse the variable in when()
$collection = collect()->filter(...);

$collection->when($collection->isEmpty(), function($c) {
    return $c->push('item');
});

Pipe the collection and use if or when() in there:
collect()->filter(...)->pipe(function($collection) {
    return $collection->when($collection->isEmpty(), function($c) {
        return $c->push('item');
    });
});

The first example seems to defeat the purpose of a collection (creating a chain of calls on a collection instance), the second example just seems to be one hell of nested closures to me.
I always add the whenEmpty() method (and variants) to my applications as macros. This allows me to fluently do the following and keep chaining:
collect()->filter(...)->whenEmpty(function($c) {
    return $c->push('item');
});

I thought it could prove useful to suggest these shortcuts for implementation. If not I'lll happily continue defining these as macros.

I too often find myself doing macros to execute a callback under any condition. I think it's a good start to add a series of features of this type to the collection.

Isn't unlessNotEmpty() just the same as whenEmpty()? And unlessEmpty() the same as whenNotEmpty()?

Isn't unlessNotEmpty() just the same as whenEmpty()? And unlessEmpty() the same as whenNotEmpty()?

It surely is. It seemed "incomplete" to me only suggesting the whenX() variants while the Collection class offers both when() and unless().


add ASGI support for embedded apps

Description
This commit replaces our WSGI implementation with a new ASGI one,
which then uses asgiref's compatibility mode to still support WSGI applications.
The ASGI implementation is a bit bare-bone, but good enough for our purposes.
The major changes are:

We now support ASGI apps.
Instead of taking connections out of mitmproxy's normal processing,
we now just set flow.response and let things continue as usual.
This allows users to see responses in mitmproxy, use the response hook
to modify app responses, etc. Also important for us,
this makes the new implementation work for shenanigans like sans-io.

@mplattner: this should fix the onboarding app to show proper flows. Can you test that everything works for you? 😃
Checklist

[x] I have updated tests where applicable.
[x] I have added an entry to the CHANGELOG.

Nice, thanks. Will check and let you know! 😄

Only got to test it now. Replay works 👍  Intercepting a requests causes this error:
error: Addon error: Traceback (most recent call last):
  File "/home/martin/mitmproxy/mitmproxy/addons/intercept.py", line 48, in request
    self.process_flow(f)
  File "/home/martin/mitmproxy/mitmproxy/addons/intercept.py", line 43, in process_flow
    f.intercept()
  File "/home/martin/mitmproxy/mitmproxy/flow.py", line 172, in intercept
    self.reply.take()
  File "/home/martin/mitmproxy/mitmproxy/controller.py", line 92, in take
    "Reply is {}, but expected it to be start.".format(self.state)
mitmproxy.exceptions.ControlException: Reply is taken, but expected it to be start.

Other question:
To test this with my tutorial app, I temporarily cherry picked your two commits.
I wonder what's the cleanest/preferred approach to get your commits into my tutorial branch and continue working on the branch?
I see a couple of options:

Rebase such that my tutorial commits branch off HEAD of the current master branch. Con: pushing the tutorial branch "invalidates" all current commits in the PR (comments etc.)
Cherry pick your two commits only. Con: we have the same commits twice once we merge. (Unless we squash/rebase first and remove the duplicate commits)
Merge the current master branch in my tutorial branch.
Temporarily cherry-pick for the test, immediate reset after testing. Pro: clean history, con: cumbersome

Maybe I'm overthinking this, but I like to have a clean history and don't want to cause a mess. 🤷♂ What do you suggest?

Uh oh. Interception is actually a really annoying issue and nontrivial to fix: We essentially have two parties competing over control of the flow.reply object in the request hook. Probably needs some coordination, I'll take a look.

Maybe I'm overthinking this, but I like to have a clean history and don't want to cause a mess. 🤷♂ What do you suggest?

One of these two:

Rebase such that my tutorial commits branch off HEAD of the current master branch. Con: pushing the tutorial branch "invalidates" all current commits in the PR (comments etc.)

Merge the current master branch in my tutorial branch.

I would not recommend cherry-picking, I don't see any advantages over a merge commit while it comes with some ugly side-effects. Other than that I don't mind, I take a rather pragmatic stance when it comes to git history. :)


The accuracy of tensorRT is very low.

Thanks for your fantastic work.
I have deployed the pretrained model following the instruction.
However, when I tried to evaluate the outputs of the trt model, I found the results are such lower from that of the onnx model.
Could you please give some instructions?
OS：ubuntu 20.04 with Nvidia 3070
Results of the onnx:

Results of the trt：

Solved. The model should be tested with the reduced kitti data.


TEM2015 Test 02:  Mission Report domain issues

The Mission Report geoform (item cf0c106d2bf6489b97dc9f97165f4072) has one dropdown that shows selectable items.  This is coming from a coded value domain, I think.  The order of the items is strange (not alphabetized,  appears second in list).
Items in the list do not have consistent spacing (Troops in Contact vs PatternOfLife)
Also, several other fields look like they could benefit from coded value domains, but don't have them (e.g. Classification, Platform Type, Sensor).

This is something we can fix in the next release.

@conklinbd @ScottCecilio  "Report Classification" and "PlatformType" should have a drop-down as well.  The coded value domains are missing for these fields.  Is it possible that something happened in the copy that it only kept the "required" field coded value domains? This is part of the Land Ops demo.


Where can I get the test configuration of the subspace_of_polices example?

Hi,
Thanks for sharing your interesting project!
I recently have been interested in the project related to the paper, Learning a Subspace of Policies for Online Adaptation in RL.
In the paper, they commented the code is released in this repository, and I tried to run the code, subspace_of_policies.
I succeeded in running the training code, train.py, after slightly modifying it.
Also, after relatively much modifying the evaluation code, I checked the code was working.
However, I think to confirm the contents represented in the paper the evaluation configuration should be given, which has the components such as the torso, thig, shin, foot, gravity, and friction.
I think the test_cfgs is needed.
from salina_examples.rl.subspace_of_policies.envs import test_cfgs
Could you share the code?
Best regards,

Hi @aithlab
Glad you liked the project and thanks for finding the bug. The test environments are already available in halfcheetah.py but there was a typo in evaluation.py. Should be working now !
Can you tell what you modified for the training ? Could be interesting to share if you found a bug.
Best

Thanks for the update and your reply!
The training code uploaded in this repository was working well, and I meant that I slightly modified it for personal research.
Best regards,


remove utf8 dependency.

First time contributor. I wanted to help so I plucked an issue with the "good first issue" tag.
This commit addresses #4289
Summary:
The author of one of the wc dependencies opened an issue stating that the crate and repo were being deprecated. They suggested incorporating what was needed into coreutils directly and removing the dependency. I incorporated and built uu_wc which should fix this issue.
Of note: there are a few 'unused' warnings in the incorporated code - should I try to remove that as well?

could you please also fix the clippy warnings?
thanks
(in a different commit - ie don't force push)

@sylvestre what's your opinion on the unused code? I think we should delete it if we're not using it.

as we are forking the lib, yes

FWIW this is what I meant by "relevant code" in https://github.com/uutils/coreutils/issues/4289: copy the parts you actually use.

Indeed! @publicmatt to answer this question then:

Of note: there are a few 'unused' warnings in the incorporated code - should I try to remove that as well?

Let's only use the relevant code like @SimonSapin suggests :)

That makes sense - thanks for the guidance.

@publicmatt do you have an update?

@publicmatt do you have an update?

I just pushed an update where I removed all dead code from the utf8 crate that was incorporated. I'm not sure what the merge process looks like - should I squash all those commits or does that happen with the pull request?

should I squash all those commits or does that happen with the pull request?

We usually keep the commits. So I'd be nice if you could clean those up. In this case I think 1 commit would suffice.

should I squash all those commits or does that happen with the pull request?

We usually keep the commits, but we squash sometimes if a contributor made a mess of the history. So I'd be nice if you could clean those up. In this case I think 1 commit would suffice.

Makes sense! I did the following to combine my mess of commits:
git reset --soft [last commit from main before I started working on this]
git commit -m "replace utf8 dependency"
git push --force

I think that's right??

Getting acquainted with all the checks - I'll fix them up and push again.

Needs rustfmt & clippy fixes please :)

I ran and fixed suggestions from:  cargo fmt and cargo clippy and RUSTDOCFLAGS="-Dwarnings" cargo doc  --features "feat_os_unix" --no-deps --workspace --document-private-items. These were all failing the checks last time.


Automated cherry pick of #12217

Cherry pick of #12217 on cloud.
/cc  @ConorMacpherson
NONE

/update-branch


Reducing height

Hi, while i'm at it.
Been looking at reducing the height of the box, it is quite large on smaller screens, but i have been unsuccessful.
Is there any way to reduce padding, height etc?

Unfortunately nope, it's fixed as described in the Guideline.
However you could get every element in it with getters and then customize their dimensions if you want.


live legend updating

It seems to work!

it works well. I have also added the change of props for legend chart axes labels


major refactoring of local.py

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

No.

[x] (If applicable) Documentation has been added / updated (for bug fixes / features).
[x] (If applicable) Tests have been added.
[x] HISTORY.rst has been updated (with summary of main changes).

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added.

What kind of change does this PR introduce?

Class Data mostly replaced by get_yearly_op, both in indicators.py
Everything that Data did should be supported, but using xclim's stats indicator, with indexers & xclim.core.missing.

This will also allow for getting min instead of only max/volume.
This also means that you can use date_bounds (i.e. [03-01, 06-30]) as well as dayofyear.

Added a rolling window to get_yearly_op (e.g. Q14max).
The streamflow to volume transformation was moved to xhydro.indicators.compute_volume() and uses xclim.core.units.rate2amount().
Mostly re-written fit, parametric_quantiles & criteria functions in local.py (and split them in 3 for more control on what is computed).

The functions now stay fully lazy (no more .to_dataframe() or .load())
get_criterions (now _get_criteria_1d()) should be better optimized and should never crash for AICC.

Added xs.health_checks to xhydro.utils.

Does this PR introduce a breaking change?

Multiple functions were removed and not replaced, since alternatives already exist. Let me know if some of them need to be reincluded.

xhydro.utils.get_julian_day --> ds.time.dt.dayofyear should pretty much cover this, no? Also, date_bounds should cover what was needed from this function.
xhydro.utils.get_timestep --> xarray.infer_freq(ds.time) is a more robust version of this.
Data.select_catchments --> ds.where(ds.id.str.contains(), drop=True) should cover this. .contains() works with regex.

No more DataFrames anywhere, since they break the lazy computing and are horrible for memory management. If you want something more than a simple .to_dataframe() for your notebooks, we could code that in .utils.

Other information:

For now, the season dimension has been fully removed and get_yearly_op instead outputs multiple variables. One reason for this is to have much better metadata. We can discuss this.
Local has for now been replaced with 3 functions. If we still want a class, we'll need to see its usefulness.

@sebastienlanglois @TC-FF I'm not quite done, but I wanted to put the files here before our meeting so you can see what I'm working on.

Thanks @RondeauG, your work is fantastic! The code is super easy to understand and well-organized :) I know there's more to be done, but I couldn't resist giving your branch a try with the notebook example to see how it's coming along. The fit function worked like a charm! I will modify the notebook directly in you branch as the code is being developed so we can have a fully working example!

I'm having a small issue with parametric_quantiles. It seems like thenp.nan parameters are passed to each distribution and they except less. Maybe I'm not using it correctly
.
Also, I would like to understand a little better how to use the code you provided below. My dataset, ds, contains a streamflow variable, but I'm a bit uncertain about how to properly pass it in the example you've given. Currently, I'm encountering the following issue:

# Get the maximum streamflow between DOY 1 and 120, tolerance of 15% missing values
qmax_winter = xc.core.indicator.Indicator.from_dict(
                                 data={"base": "stats",
                                            "input": {"da": "streamflow"},
                                            "parameters":
                                                            {"op": "max",
                                                             "indexer": {"doy_bounds": [1, 120]}},
                                            "missing": "pct",
                                            "missing_options": {"tolerance": 0.15}
                                            },
                                  identifier="qmax_winter",
                                  module="fa",
                              )

Thanks again for all the work and looking forward to the next meeting!

@sebastienlanglois @TC-FF Ça devrait être prêt pour révision ! Quelques points spécifiques pour vous:

Comme c'est un refactoring quasi-complet de #20, je crois que le plus simple pour cette PR est de merger directement dans le main. Quand on est contents avec cette PR, je crois qu'on pourra seulement fermer l'autre PR.
@TC-FF en particulier. Il y a certaines fonctions de l'ancien xhydro comme get_julian_day ou select_catchments que je n'ai pas transférées parce que des alternatives existent directement dans xarray, mais on pourrait toujours les remettre dans xhydro.utils si tu les veux. Juste me le faire savoir.
J'ai mis à jour le notebook avec les nouvelles fonctions.
La classe Local est actuellement dans un entre deux un peu étrange où elle n'est plus trop utile (à mes yeux, en tout cas). Pour le moment, je l'ai commentée et j'ai aussi gardé en commentaires certaines options de visualisation to_dataframe qui existaient dans l'ancien xhydro. Est-ce que vous voulez encore ça ?
@sebastienlanglois J'ai ajouté tes infos à la liste d'auteurs, mais il me manque notamment l'adresse courriel que tu voudrais utiliser. À toi aussi de voir si tu veux une affiliation ou pas dans le Zenodo (idem @TC-FF ).


Add information on how to enable cookies to the cookie page

This adds links to information on how users can enable cookies for their browsers.
Before

## After

closing this and taking a different approach


Updated benchmark reference

Updated a100 benchmark numbers after the FSDP-ignore -modules fix

@achew010 this CSV would correspond to numbers in #31 correct?

@achew010 this CSV would correspond to numbers in #31 correct?

Yes the tables in the description of #31 are referenced from this csv


Need to be upgraded to go-ipfs  v0.11.0 Infura

Now "try client?.pin.add(addHash)" could not work.

@Lisfee has been updated and tested. Infura also now pins data without having to access the pin sub command. As long as it is added the b85 conversion of the first MultiAddress will be your gateway hash to access the content from any gateway once it propagates to each.


Resolve #596 : add support for enabling sendgrid click tracking

Close #596

Our organization is also hoping this gets merged soon! We opened https://github.com/thoughtbot/bamboo/issues/596.

@danadaldos unfortunately, I haven't had much time to maintain Bamboo since leaving thoughtbot.
But I just gave this a look, and it looks good to me! Thanks for contributing @sreecodeslayer!

@germsvel Thank you!


[CLOSED] Gridcoin openbazaar integration

 Issue by grctest
Sunday Apr 23, 2017 at 00:41 GMT
Originally opened as https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/116

https://github.com/OpenBazaar/openbazaar-go/blob/master/docs/altcoins.md
https://openbazaar.org/

 Comment by Erkan-Yilmaz
Sunday Apr 23, 2017 at 04:36 GMT

see #30


Accounts-Stripe Support

Hi, are there any plans to support any accounts-stripe repo?
Thanks

Yep. If we have another package for stripe I will included it.


Update scrollview.md

StickyHeaderIndices updated.

@Simek This is my first time contributing. Sorry if I made a mistake

@Simek This is my first time contributing. Sorry if I made a mistake

Not a problem, take all the time you need. 🙂 You can find the guide about the site setup, structure and how to test changes in the Readme file - https://github.com/facebook/react-native-website#-getting-started


bugfix:fix can not access a member of DruidDataSourceWrapper

Ⅰ. Describe what this PR did
fix can not access a member of DruidDataSourceWrapper with modifiers "public"
Ⅱ. Does this pull request fix one issue?

fixes #1690
Ⅲ. Why don't you add test cases (unit test/integration test)?
I've done integration test.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1691 into develop will decrease coverage by 0.02%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop    #1691      +/-   ##
=============================================
- Coverage      47.15%   47.13%   -0.03%
  Complexity      1784     1784
=============================================
  Files            364      364
  Lines          13251    13252       +1
  Branches        1642     1642
=============================================
- Hits            6249     6246       -3
- Misses          6323     6326       +3
- Partials         679      680       +1

Impacted Files
Coverage Δ
Complexity Δ

...ta/spring/annotation/GlobalTransactionScanner.java
50.92% <0%> (-0.48%)
17 <0> (ø)

...server/store/file/FileTransactionStoreManager.java
45.99% <0%> (-1.05%)
19% <0%> (ø)

Continue to review full report at Codecov.

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


make patch/minor/major - pass --no-commit to allow review and commit with signoff

Do you want to just have it sign when running the command?
commit = True
tag = False
commit_args = -s

Also this should be done via config not via the command ideally


[Feature request] ctxt menu

Please consider this addition:
https://github.com/cytoscape/cytoscape.js-cxtmenu
Enables a lot of interactivity (yet I got a feeling that it is challenging to implement)
Anyway, are there any plans about cytoscape addons in near future?Lots of precious functions there.
Thanks!

Any chance this will be integrated?

If integration is desired I can update the PR for the current version of dash-cytoscape, just let me know.


Fixing os.GetWd() usage for getting component name

Signed-off-by: Mohammed Zeeshan Ahmed<EMAIL_ADDRESS>What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind cleanup
/kind deprecation
/kind design
/kind failing-test
/kind feature
/kind flake
/kind code-refactoring
Documentation changes: Please include [skip ci] in your commit message as well
/kind documentation
[skip ci]

What does does this PR do / why we need it:
See title
Which issue(s) this PR fixes:
Fixes #2959
PR acceptance criteria:

[ ] Existing tests should not break

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:

Unit test failed

Yep, copy paste golangci issue now fixed @girishramnani

Timeout error: https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_odo/3844/pull-ci-openshift-odo-master-v4.5-integration-e2e/1298851539779063808
/retest

Timeout error https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_odo/3844/pull-ci-openshift-odo-master-v4.5-integration-e2e/1298934238824894464
/retest


mode:list would be a good feature

Because maps are not ordered and it would be good to retain the order of files returned by glob.
At the moment I either have to convert the hash to array (with no guarantee of having the same order) or to provide a custom function.
Instead of hash it would return the array of objects with properties, for example, "name" and "module".

Ok, I'll add this in the next release.

great, thanks

Should be there, can you test it and let me know if it works for you?
How important is the order of the matches? (because right now matches are sorted to ensure consistency)

Works beautifully, I replaced :) Thank you!
https://github.com/epoberezkin/ajv/commit/73b21e0383cc4efd20567b28e27992685e5b2748


feat(frontend): 709 6 toggle in top menu

PR 6: TopMenu Component
Merged:
ListWithNavigationDrawer: https://github.com/dreammall-earth/dreammall.earth/pull/1187
Name: TopMenu Component
frontend/src/components/menu/TopMenu.vue
frontend/src/components/menu/TopMenu.stories.ts

letzter stand

I have the ListWithNavigationDrawer component in the top and bottom menus. at this point it only needs to be filled.

https://github.com/dreammall-earth/dreammall.earth/assets/1324583/dffa0521-ec41-45f9-a01f-ca21a0bd5c2d

i have added jsdom which fixes our error in the frontend/src/components/buttons/CreateButtonMobile.test.ts. the tests should now throw fewer warnings.


Notification about UI state change for long running tasks

I play with simple NLP application inside browser.
I want to add loading indicator. Right now I use following approach
this.loading = true;
this.StateHasChanged();
try {
    this.result = await httpClient.GetJsonAsync<SearchResult[]>($"api/search/sentence?content={content}");
} finally {
     this.loading = false;
}

Main trick is using StateHasChanged. My question is: Is this correct way to do that? How Blazor in general think such cases should be handled. I'm fine with calling StateHasChanged() but I prefer that it would be called automatically before await.
Example 1
this.loading = true;
try {
    // this.StateHasChanged(); called here by runtime
    this.result = await httpClient.GetJsonAsync<SearchResult[]>($"api/search/sentence?content={content}");
} finally {
     this.loading = false;
}

Example 2
or better, if I could control that using for example .ConfigureAwait(false)
this.loading = true;
try {
    // this.StateHasChanged(); called here by runtime
    this.result = await httpClient.GetJsonAsync<SearchResult[]>($"api/search/sentence?content={content}")
             .ConfigureAwait(false);
} finally {
     this.loading = false;
}

This example works as expected:
<button onclick="@(async() => await LongRunningTaskAsync())">Start long running task</button>
@if (loading)
{

Loading, please wait

}
else
{

Loaded

}

@functions {
    bool loading = false;

    async Task LongRunningTaskAsync()
    {
        loading = true;
        await Task.Delay(2000);
        loading = false;
    }
}

You have to call your method using async lambda expression and await. Your method have to be async and return Task.

You are right, I somehow miss that obvious fact. Searching for the documentation and did not find that example. Apologize for noise.

I think the issue is somewhere present, or I miss something obvious. For simple cases the UI is really updated as stated in the example posted above. But in some cases I don't see any changes until I manually trigger UI refresh by clicking on the some checkbox (this is trigger UI refresh for me, by changing state)
I essentially do following

Made call to API method 1
Update UI state with results
Made call to API method 2
Update UI state with results
Made bunch of calls to method 3 in the loop
Update UI with results

For me UI does not refreshed only after step 6 completes. But if I press checkbox on the step 2, or 4 I see the actual results.
Below short code which exhibit error.
    async Task FindReplacementsAsync(string content)
    {
        var currentSearchCandidates = new string[] { content };
        if (useAbbreviationReplacement)
        {
            currentSearchCandidates = (await api.ReplaceAbbreviationsAsync(content)).ToArray();
        }

        this.searchCandidates = currentSearchCandidates;

        var currentSearchQueries = this.searchCandidates;
        if (this.useSynonymReplacement)
        {
            var replacements = await Task.WhenAll(this.searchCandidates.Select(_ => api.FindSynonymQueryAsync(_)));
            currentSearchQueries = replacements.ToArray();
        }

        this.searchQueries = currentSearchQueries;
    }

    async Task ProcessDocument()
    {
        if (this.loading)
        {
            return;
        }
        this.loading = true;
        this.searchQueries = new string[0];
        this.searchCandidates = new string[0];
        try
        {
            await this.FindReplacementsAsync(this.content);

            var result = new List<SearchResultModel>();
            foreach (var query in this.searchQueries)
            {
                var tempResult = await api.SearchAllAsync(query);
                result.Add(tempResult );            }
            }
            this.result = result;
        }
        finally
        {
            this.loading = false;
        }
    }


TDI-44819 - Disable DTDs when checking excel files

We don't disable DTDs when checking excel files.

Pull request test result: success :star:


fix: handle invalid version validation for installation script

fix: https://github.com/dojoengine/dojo/issues/1831

dojoup script will fail with proper error handling when invalid version is mentioned by the user while installation.
currently downloading latest dojo was being displayed in spite of the user downloading a mentioned version. Fixed that by displaying the message only when a version is not provided by the user in the command

Thank you! Saw these rewordings. Will make sure to follow them from next time!


setValueToVerifyAgainst should have concrete types where possible

setValueToVerifyAgainst should not take Object as a parameter if the verifier's actual type is known

Delivered in Version 1.1


Add convenience method populateWithAvailableLanguages(List)

Closes issue #36.

Add a new method to the LRS interface, and its RemoteLRS implementation. The new method is called retrieveActivity. It sends a GET to the "activities" endpoint and returns the canonical description of the activity found here, if any.

Thanks for this @dcdave!
There's also a related issue on TinCanPHP with some code: https://github.com/RusticiSoftware/TinCanPHP/issues/24
We should agree on the method name and parameters.

Let's wrap up the discussion in https://github.com/RusticiSoftware/TinCanPHP/issues/24 and then see where this needs to land.

This languished long enough that I felt bad about sending it back with review comments, so I just made what would be the required changes and have set it up pending as #51. Effectively .retrievePerson and .retrieveActivity needed new LRS response types so that they could store proper content bodies based on what is returned from an LRS. That included adding a new Person object and adjusting the RemoteLRS implementation to not call the getDocument method. The existing implementation would have worked I guess because the used response types are so generic. These changes bring the library more inline with the original PHP approach and I think are similar to the approach used elsewhere in this lib. The changes once merged will go out in the next minor version release which will quickly be followed by a major version release for JDK 8.


Add a missing comma

Motivation and context

Without the comma, the string literals are joined together, so only one test case is run instead of two.
How has this been tested?
N/A

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@IRDonch , could you please remove unused import as well in line 10 (django.http)?


error: unknown switch `r'

I'm using the most up to date yadr (f2fdd4506d7d4356d5b5a9e2e4c133e8b0740ed7). When I do :Gsearch I see an error "error: unknown switch `r'" and usage message from git grep.  It looks like greplace is expecting regular grep to be used but git grep is being used instead. For now i have replaced line 224 of greplace.vim with     let grep_opt  = '' and it's working again. Wondering if i'm doing something wrong here. Also not sure if I should report against yadr or greplace, but I'm leaning towards yadr. Thanks in advance.

I see this too on some installations. @joshuacronemeyer are you still doing this to fix it?

@joshuacronemeyer and @ramontayag is this still happening? I've been using Gsearch since this issue was open and nobody else complained in all these years so I'm assuming it's (was) something local to your installation.
Hence I'm closing this issue. Feel free to reopen it. Cheers

It's working fine for me on my current setup! No longer using a modified greplace.vim

Sorry I no longer remember. An update or a different installation may have fixed it. I no longer see it. Thanks!


Some text elements are inconsistent between editor, preview, and publish in RTL mode

Bug Description

During investigation of RTL support for Page Layouts/Templates in #5909, it was found that some text elements render differently in both previews and the editor when in RTL mode than they do when published/previewed.
Pages created in RTL or LTR mode both look like the LTR when published, but differ in the editor/page previews.
Expected Behaviour

LTR and RTL should not change how elements appear in the editor. Pages created from templates/page layouts in RTL mode should render the same when previewed/published than when they are edited.
Steps to Reproduce

Switch to RTL mode
Add a page layout that has an issue (Ex: Cooking List "Top 8" page)
Text elements render right aligned but should be left aligned
Publish/Preview. The elements are left aligned as they should appear in the editor.

Screenshots

Editor content of the same exact same story page (from template/page layout):

Editor RTL
Editor LTR

Additional Context

Plugin Version:
WordPress Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

cc @miina @barklund

cc @miina @barklund


site.url missing from generated index

In version 2.3.3 the index contains the correct site.url string. In the latest version its missing. Im not sure if this has to do with antora or the site generator.
Works when you install using the 0.4.0 tag
Working in version 2.3.3
window.antoraLunr.init({"index":{"version":"2.3.3","fields":["title","name","text","component"],"fieldVectors":[["title/https://opensource.suse.com/doc-susemanager/suse-manager/common_gfdl1.2_i.html"

Broken in 2.3.8:
window.antoraLunr.init({"index":{"version":"2.3.8","fields":["title","name","text","component"],"fieldVectors":[["title//suse-manager/common_gfdl1.2_i.html"

We've removed the site.url from the index to make it portable. The latest version includes version 0.7.1 of Antora Lunr: https://github.com/Mogztter/antora-lunr/releases/tag/v0.7.0

Don't forget to update your UI to at least define window.antora.basePath. This variable will be used to construct the complete URL of search results client-side:

<script>
  window.antora = window.antora || {}
  window.antora.basePath = '{{or siteRootPath (or site.url siteRootUrl)}}'
</script>

In other words, you need to update your UI to configure window.antora.basePath. And don't forget to synchronize the https://github.com/Mogztter/antora-lunr/tree/v0.7.1/supplemental_ui directory.
Related issues: https://github.com/Mogztter/antora-lunr/issues/61

@jcayouette Can you please confirm that updating the UI fixed this issue? Can we close?

Closing, feel free to reopen if it still does not work.