Wdio spec reporter issues 11

Please note: this PR is about fixing issue #11 please refer to that issue for more details.
The problem as described in the issue is that the title is being used to identify suite/specs & tests when rendering the report. This results in items with the same name being displayed only once.
To resolve this I added a UID to each library that can be used to identify the item as a unique item. This Resolves the issue and will make sure all items are always displayed when rendering the report.
This is a breaking change! Since we need now rely on the UID implementation of WDIO & the testing libraries.

@wvankuipers is this change compatible with earlier WebdriverIO versions? Just curious regarding versioning?

is this change compatible with earlier WebdriverIO versions? Just curious regarding versioning?

Yes it is, the only notabel change is the extra lookup for the title I added because we used the UID for reference. For an older version of WebdriverIO the UID will equal the title so it will still work.


Stream#gets should unblock when Stream is closed from another Task.

@orospakr, are you still facing this and have a repro?


Room Vortex

Would be nice to have some staircase going in spiral up or down and NFTS from collections could be hang on the wall while user is walking through, or it could be circle if so

I think I've got a design. Stairs will wrap in vortex, wall space to hang art, and there will be spaces for 3d objects all incorporated. Will hack on it more tomorrow 😈


Unable to find a specification

I am trying to install this with cocoapods
I first set (as per doc) in my podfile,
pod 'PromisesObjC', '~> 1.2.3'
it gave me following error
Unable to find a specification for PromisesObjC (~> 1.2.3)
Then I tried with pod 'PromisesObjC'
it gave me following error
Unable to find a specification for PromisesObjC
I am using xcode 9.4

thanks, let me share my pod file snippet in a while.
On Wed 12 Sep, 2018, 2:56 AM Anthony Shoumikhin<EMAIL_ADDRESS>wrote:

Just tried pod install for the following Podfiles:
target 'SomeTarget' do
pod 'PromisesObjC', '~> 1.2.3'end
Along with modifications like:
target 'SomeTarget' do
pod 'PromisesObjC'end
and:
target 'SomeTarget' do
use_frameworks!
pod 'PromisesObjC'end
Every time the pod was installed successful.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/promises/issues/69#issuecomment-420431143, or mute
the thread
https://github.com/notifications/unsubscribe-auth/APH9iQCWuN4iBWXIFAQtRYj79kKH9NnYks5uaCpzgaJpZM4WkPNu
.

`target 'xxxx' do
Uncomment the next line if you're using Swift or would like to use dynamic frameworks
use_frameworks!
Pods for XXX
pod  'Firebase/Core'
pod  'Firebase/Auth'
pod  'Firebase/Database'
pod  'Firebase/Storage'
pod  'Firebase/Analytics'
pod  'Firebase/Messaging'
pod  'Firebase/Crash'
pod  'Firebase/DynamicLinks'
pod  'SDWebImage', '~>3.8'
pod  'GoogleSignIn'
pod  'FBSDKLoginKit'
pod  'FBSDKShareKit'
pod  'ECSlidingViewController', '~> 2.0.3'
pod  'MBProgressHUD', '~> 0.9.2'
pod  'JTSImageViewController', '1.5.1'
pod  'IQKeyboardManager'
pod 'AFNetworking', '~> 3.0'
pod 'GooglePlaces'
pod 'PureLayout'
pod 'Branch'
pod  'AccountKit'
pod 'Fabric'
pod 'Crashlytics'
pod 'Appsee'
pod 'TwitterKit'
pod 'PromisesObjC'
pod 'RKTagsView'

end`

I have
cocoapods-1.5.3

I'm using the same version and just ran it on your Podfile:
$ pod install
Analyzing dependencies
Downloading dependencies
Installing AFNetworking (3.2.1)
Installing AccountKit (4.35.0)
Installing Appsee (2.4.3)
Installing Bolts (1.9.0)
Installing Branch (0.25.3)
Installing Crashlytics (3.10.7)
Installing ECSlidingViewController (2.0.3)
Installing FBSDKCoreKit (4.35.0)
Installing FBSDKLoginKit (4.35.0)
Installing FBSDKShareKit (4.35.0)
Installing Fabric (1.7.11)
Installing Firebase (5.6.0)
Installing FirebaseAnalytics (5.1.1)
Installing FirebaseAuth (5.0.3)
Installing FirebaseAuthInterop (1.0.0)
Installing FirebaseCore (5.1.1)
Installing FirebaseCrash (3.1.1)
Installing FirebaseDatabase (5.0.2)
Installing FirebaseDynamicLinks (3.0.2)
Installing FirebaseInstanceID (3.2.1)
Installing FirebaseMessaging (3.1.0)
Installing FirebaseStorage (3.0.0)
Installing GTMOAuth2 (1.1.6)
Installing GTMSessionFetcher (1.2.0)
Installing GoogleAppMeasurement (5.1.1)
Installing GoogleMaps (2.7.0)
Installing GooglePlaces (2.7.0)
Installing GoogleSignIn (4.2.0)
Installing GoogleToolboxForMac (2.1.4)
Installing GoogleUtilities (5.2.2)
Installing IQKeyboardManager (6.1.1)
Installing JTSImageViewController (1.5.1)
Installing MBProgressHUD (0.9.2)
Installing PromisesObjC (1.2.3)
Installing Protobuf (3.6.1)
Installing PureLayout (3.1)
Installing RKTagsView (1.4.0)
Installing SDWebImage (3.8.2)
Installing TwitterCore (3.2.0)
Installing TwitterKit (3.4.0)
Installing leveldb-library (1.20)
Installing nanopb (0.3.8)
Generating Pods project
Integrating client project
Sending stats
Pod installation complete! There are 27 dependencies from the Podfile and 42 total pods installed.

hmm which xcode version you are using? or this should not affected by that? I am using xcode 9.4

Same Xcode version, it shouldn't matter.
Try pod install --verbose, maybe you'd spot something interesting.

here is the output
pod install --verbose
Preparing
Analyzing dependencies
Inspecting targets to integrate
Using ARCHS setting to build architectures of target Pods-XXXX: (``)
Finding Podfile changes
A PromisesObjC

AFNetworking
AccountKit
Appsee
Branch
Crashlytics
ECSlidingViewController
FBSDKLoginKit
FBSDKShareKit
Fabric
Firebase
GooglePlaces
GoogleSignIn
IQKeyboardManager
JTSImageViewController
MBProgressHUD
PureLayout
RKTagsView
SDWebImage
TwitterKit

Resolving dependencies of Podfile
[!] Unable to find a specification for PromisesObjC (~> 1.2.3)
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/resolver.rb:402:in find_cached_set' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/resolver.rb:372:in specifications_for_dependency'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/resolver.rb:177:in search_for' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/resolver.rb:288:in block in sort_dependencies'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/resolver.rb:281:in each' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/resolver.rb:281:in sort_by'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/resolver.rb:281:in sort_dependencies' /Library/Ruby/Gems/2.3.0/gems/molinillo-0.6.6/lib/molinillo/delegates/specification_provider.rb:53:in block in sort_dependencies'
/Library/Ruby/Gems/2.3.0/gems/molinillo-0.6.6/lib/molinillo/delegates/specification_provider.rb:70:in with_no_such_dependency_error_handling' /Library/Ruby/Gems/2.3.0/gems/molinillo-0.6.6/lib/molinillo/delegates/specification_provider.rb:52:in sort_dependencies'
/Library/Ruby/Gems/2.3.0/gems/molinillo-0.6.6/lib/molinillo/resolution.rb:288:in initial_state' /Library/Ruby/Gems/2.3.0/gems/molinillo-0.6.6/lib/molinillo/resolution.rb:210:in start_resolution'
/Library/Ruby/Gems/2.3.0/gems/molinillo-0.6.6/lib/molinillo/resolution.rb:168:in resolve' /Library/Ruby/Gems/2.3.0/gems/molinillo-0.6.6/lib/molinillo/resolver.rb:43:in resolve'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/resolver.rb:123:in resolve' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/installer/analyzer.rb:781:in block in resolve_dependencies'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/user_interface.rb:64:in section' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/installer/analyzer.rb:779:in resolve_dependencies'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/installer/analyzer.rb:88:in analyze' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/installer.rb:243:in analyze'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/installer.rb:154:in block in resolve_dependencies' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/user_interface.rb:64:in section'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/installer.rb:153:in resolve_dependencies' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/installer.rb:116:in install!'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/command/install.rb:41:in run' /Library/Ruby/Gems/2.3.0/gems/claide-1.0.2/lib/claide/command.rb:334:in run'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/lib/cocoapods/command.rb:52:in run' /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.3/bin/pod:55:in <top (required)>'
/usr/local/bin/pod:22:in load' /usr/local/bin/pod:22:in '
[!] Automatically assigning platform ios with version 10.0 on target XXXX because no platform was specified. Please specify a platform for this target in your Podfile. See https://guides.cocoapods.org/syntax/podfile.html#platform

I believe something is broken with your CocoaPods setup.
Wonder if the following could help:
pod repo remove master
pod setup
pod install

okay, let me try this and get back to you
On Wed 12 Sep, 2018, 3:30 AM Anthony Shoumikhin<EMAIL_ADDRESS>wrote:

I believe something is broken with your CocoaPods setup.
Wonder if the following could help:
pod repo remove master
pod setup
pod install
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/promises/issues/69#issuecomment-420440390, or mute
the thread
https://github.com/notifications/unsubscribe-auth/APH9iZ9h9XvW7k1VxuaiQ1XzcS8lFJ8fks5uaDJjgaJpZM4WkPNu
.


TEXT_FILE_CONTENT not interpolated in attachment text

I have a task that looks like this:
  put: notify-slack
  params:
    text_file: metadata/github_link
    text: "*success*"
    attachments:
      - text: "Job <$ATC_EXTERNAL_URL/teams/$BUILD_TEAM_NAME/pipelines/$BUILD_PIPELINE_NAME/jobs/$BUILD_JOB_NAME|$BUILD_JOB_NAME> for $TEXT_FILE_CONTENT (<$ATC_EXTERNAL_URL/builds/$BUILD_ID|Details>)"
        color: "good"

A previous task generates the metadata/github_link file. I've verified that it is there and looks as expected in the notify-slack container.
In builds on August 1 the attachment text was getting interpolated as expected. In builds this week the ATC_* variables are being interpolated correctly, but TEXT_FILE_CONTENT ends up empty -
i.e., the resulting attachment text looks like:
Job build for (Details)

I haven't changed anything in my pipeline config in the meantime, so I believe this is a recently-introduced bug in the slack-notification-resource.

Is this addressed by #39 (now merged)?

Yes, just tested the latest :edge image and it appears to be working. Thanks!

@jhunt Would it be possible to tag a release with this fix? I'm a little wary of using the :edge tag, but afaik it's the only one with the fix. Thanks!

Will do

1.3.1 should be good to go @adamwg

Thanks! I'll update our pipelines to 1.3.1 when I have a moment.


Transition from Helm Repositories to OCI Artifacts for Storing Helm Charts - update to helm3 command

Resolves # . (This is applicable only if this pull request relates to a GitHub issue, delete the line otherwise)
Notes:
update helm to using helm 3 command . https://tools.hmcts.net/jira/browse/DTSPO-19019

Tested here https://github.com/hmcts/sds-toffee-frontend/pull/544


enable security on jobservice scan api

Coverage increased (+0.005%) to 54.64% when pulling 92258cd01235c46b7e038c10bd6ab34b81e306e9 on reasonerjt:query-clair-db into 0ade32eef5336dcf869f5de51a9b9e0deb9cdcdb on vmware:master.


App crashes if camera not working

My android camera doesn't work and that causes a crash to my app when trying to use this library.
Is there a way to validate if the camera is not broken before trying to access it or catch this exception and handle it?

+1

+1


dinar: update and change to netboot

otherwise lgtm

Realized yesterday that the README has to be updated, currently the dinar hosts are labeled as ISO images.

Pushed the missing README update straight to main: https://github.com/ponkila/homestaking-infra/commit/9d9b7e00a24b2e96702e1c43760e699389682bf8


Justfile for CosmWasm contracts

Adds a justfile for building our contracts, it also automatically deploys "ethereum-analyzer" as part of our localnet setup (requires Docker and jq)
closes #376

Summary by CodeRabbit

New Features

Added commands for managing and compiling CosmWasm contracts using Docker.
Introduced automatic deployment of contracts in the local development environment.

Improvements

Enhanced default settings to ensure uninterrupted Ethereum transactions.

Tried locally just localnet

My version of sed is not working with sed -i 's/cors_allowed_origins = \[\]/cors_allowed_origins = ["*"]/' ~/.warden/config/config.toml, but working with
sed -i '' -e 's/cors_allowed_origins = \[\]/cors_allowed_origins = ["*"]/' ~/.warden/config/config.toml
I have Mac Sonoma 14.4.1, text_cmds-165.100.8 (it's sed version in MacOs, from here https://opensource.apple.com/releases/)

After fixing have error unknown command "add-genesis-space" for "genesis"

Tried locally just localnet

My version of sed is not working with sed -i 's/cors_allowed_origins = \[\]/cors_allowed_origins = ["*"]/' ~/.warden/config/config.toml, but working with
sed -i '' -e 's/cors_allowed_origins = \[\]/cors_allowed_origins = ["*"]/' ~/.warden/config/config.toml
I have Mac Sonoma 14.4.1, text_cmds-165.100.8 (it's sed version in MacOs, from here https://opensource.apple.com/releases/)

I fixed that issue on main, I now rebased this PR so it contains the fix :)

After fixing have error unknown command "add-genesis-space" for "genesis"

I think you have a really old version of wardend. You can update it with just install.


Add service URL for CLI command: keptn get service

When executing keptn get service or keptn get service --project=xxx it would be great to also retrieve the service endpoint.
Right now this is the output:

Desired output:
Include one additional column with the service endpoint (URL + port) to make it easier to navigate to the service.

I would like to pick this up

Sorry ankit, this issue actually needs to be refined, before anyone can implement this.
Our deployment-finished cloud-events have the internal and external deployment URL in their payload, we can certainly use this information. Nevertheless, this is a topic that needs to be addressed at several levels:

api
cli
wherever we have a materialized view for our services

I'll schedule it for refinement.

API is handled in #2692 - this issue depends on #2692 and just needs to print the service URL; which can be fetched from the configuration-service API.

I'm not 100% sure that this is already handled in API.
I just tried to fetch service information from configuration-service (using Keptn 0.8.0-alpha), and I only get a list of cloudevents, e.g.:
GET /api/configuration-service/v1/project/sockshop/service?pageSize=20

{
  "nextPageKey": "0",
  "services": [
    {
      "creationDate": "1608735908346370799",
      "serviceName": "carts",
      "stageInfo": [
        {
          "lastEventTypes": {
            "sh.keptn.event.deployment.started": {
              "eventId": "4f7c40ef-20de-451d-a24a-f050b462d0c0",
              "keptnContext": "3141665c-dc68-4afb-8831-d70d762e7b0d",
              "time": "1609144600603155343"
            },
            ...
          },
          "stageName": "dev"
        },
        {
          "lastEventTypes": {
            "sh.keptn.event.deployment.started": {
              "eventId": "bbeb8734-624f-4319-8b6a-39b0039bdb9d",
              "keptnContext": "3141665c-dc68-4afb-8831-d70d762e7b0d",
              "time": "1609144683728966038"
            },
            ...
          },
          "stageName": "staging"
        },
        {
          "lastEventTypes": {
            "sh.keptn.event.action.finished": {
              "eventId": "eaca9735-5324-487b-b95f-d1ff3e75754f",
              "keptnContext": "38646432-6333-4161-b437-636632653738",
              "time": "1609172704186273261"
            },
            ...
          },
          "stageName": "production"
        }
      ]
    },
    {
      "creationDate": "1608735933618651267",
      "serviceName": "carts-db",
      ...
    }
  ],
  "totalCount": 2
}

I'm not 100% sure that this is already handled in API.
I just tried to fetch service information from configuration-service (using Keptn 0.8.0-alpha), and I only get a list of cloudevents, e.g.:
GET /api/configuration-service/v1/project/sockshop/service?pageSize=20

{
  "nextPageKey": "0",
  "services": [
    {
      "creationDate": "1608735908346370799",
      "serviceName": "carts",
      "stageInfo": [
        {
          "lastEventTypes": {
            "sh.keptn.event.deployment.started": {
              "eventId": "4f7c40ef-20de-451d-a24a-f050b462d0c0",
              "keptnContext": "3141665c-dc68-4afb-8831-d70d762e7b0d",
              "time": "1609144600603155343"
            },
            ...
          },
          "stageName": "dev"
        },
        {
          "lastEventTypes": {
            "sh.keptn.event.deployment.started": {
              "eventId": "bbeb8734-624f-4319-8b6a-39b0039bdb9d",
              "keptnContext": "3141665c-dc68-4afb-8831-d70d762e7b0d",
              "time": "1609144683728966038"
            },
            ...
          },
          "stageName": "staging"
        },
        {
          "lastEventTypes": {
            "sh.keptn.event.action.finished": {
              "eventId": "eaca9735-5324-487b-b95f-d1ff3e75754f",
              "keptnContext": "38646432-6333-4161-b437-636632653738",
              "time": "1609172704186273261"
            },
            ...
          },
          "stageName": "production"
        }
      ]
    },
    {
      "creationDate": "1608735933618651267",
      "serviceName": "carts-db",
      ...
    }
  ],
  "totalCount": 2
}


fix: add echo flag to my collection demand index and request price estimate

Description

Add missing echo feature flags to features.ts
Improve type to raise an error when we forget to add an echo feature flag when the feature is ready to be released

Note:
Suggestions to improve types are very welcome - There is a million ways to do things on ts and I am not sure if my way is the best 😄

PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

add echo flag to my collection demand index and request price estimate - mounir
improve types on features.ts - mounir

Need help with something? Have a look at our docs, or get in touch with us.

I will wait for the build to finish and I believe it might fail because some flags might be missing in echo that we were not aware of

I added the one flag that was missing, yesterday to echo. we should be good.

lol did you rebase or did we make the same PR https://github.com/artsy/eigen/pull/6846 🤣

ill change mine a bit, to make sense on top of yours.

lol did you rebase or did we make the same PR https://github.com/artsy/eigen/pull/6846 🤣

No we actually did the same thing 😄 (check the thread here) I rebased because I wanted to add Kizito's changes here since they got merged after I created the PR yesterday.


Vertical synchronization is reactivated when the window regains focus

Godot version:
3.3 rc9
OS/device including version:
Windows 10, Macbook pro retina 2013, Intel Iris Graphics 6100, Google Chrome
Windows 10, AMD Ryzen 5 2600, GTX 1650, Google Chrome
Issue description:
In HTML5 export, i disabled vsync and set FPS target to 30 to make my game more friendly for user with old devices.
Everything work when window's user keep focus.
When you decide to open another tab/window for 10 seconds and you get back on the game tab ; vsync is re-enabled and FPS target is set to 60
Steps to reproduce:

Make a project with target fps to 0 and vsync disabled, put a button to set FPS target to 30
Export to HTML5
Run on the browser, check your GPU consumption
Open/change tab for 10 seconds
Get back to the game tab, vsync is re-enabled, check your GPU consumption

Minimal reproduction project:
https://chapatizretro.com/fps/fps.zip
https://chapatizretro.com/fps/fps.html (web version)

In HTML5, you do not have control over V-Sync as the browser will likely force it (and the graphics driver may also be doing this, especially on macOS).

Can you reproduce this in 3.2.3?
Also, in HTML5, you do not have effective control over V-Sync as the browser will likely force it. The graphics driver may also force V-Sync by default, especially on macOS.

https://chapatizretro.com/fps/fps_323.html (vsync is disabled but seems to be still enabled, can't reach 30 fps)
So why you can set target fps to 30 if you can't control vsync in HTML5 ? I find that very useful because players with old devices can run my game without setting their old graphics card on fire

So why you can set target fps to 30 if you can't control vsync in HTML5 ? I find that very useful because players with old devices can run my game without setting their old graphics card on fire

The Target FPS setting is unrelated to V-Sync. The following combinations are all valid:

Zero (unlimited) target FPS and V-Sync enabled, the default.
Non-zero target FPS and V-Sync enabled.
Zero (unlimited) target FPS and V-Sync disabled. Uncapped framerate.
Non-zero target FPS and V-Sync disabled. Generally suboptimal as it can cause stuttering.

While you will always have control over the target FPS, you won't always have control over V-Sync, especially on mobile and web platforms.
As a workaround, you may be able to set Engine.target_fps in a script's _process() function to enforce the FPS limit continuously.

So why you can set target fps to 30 if you can't control vsync in HTML5 ? I find that very useful because players with old devices can run my game without setting their old graphics card on fire

The Target FPS setting is unrelated to V-Sync. The following combinations are all valid:

Zero (unlimited) target FPS and V-Sync enabled. This is the default. No tearing, but there will be more input lag and stuttering will happen if the device can't keep up.
Non-zero target FPS and V-Sync enabled. Generally suboptimal as it can cause stuttering.
Zero (unlimited) target FPS and V-Sync disabled. Uncapped framerate.
Non-zero target FPS and V-Sync disabled. Lower input lag compared to V-Sync, but tearing will be visible.

While you will always have control over the target FPS, you won't always have control over V-Sync, especially on mobile and web platforms.
As a workaround, you may be able to set Engine.target_fps in a script's _process() or _physics_process() function to enforce the FPS limit continuously.

Thanks Calinou, i will try your solution :D

So Calinou i tried your workaround with _process or _physics_process and the behavior is the same ...
Even with _process, there is a delay to set fps to 30 ...
I tried with the last version 3.3 stable

@Faless hello Fales, how i can fix this issue ?


Handle Calculator AoT error message text more gracefully

Describe the bug
Error messages in Always-on-Top mode may overflow, introducing arrows unintentionally.
Steps To Reproduce

Launch Calc
Enter AoT mode
Divide by zero

Notice that the error message text is big and does not fit all in the display area
Expected behavior
Error messages scale to fit the display without causing overflow with arrows.
Screenshots

Device and Application Information

OS Build: 10.0.18942.0
Architecture: X64
Application Version: 10.1906.53.0
Region: en-US
Dev Version Installed: True

Requested Assignment
I'm just reporting this problem.  I don't want to fix it.

This is your friendly Microsoft Issue Bot. I created this issue automatically as requested by a team member.

Does this still need help? Might as well finish what I started. :)

If @rudyhuyn hasn't claimed it yet as part of another issue/PR I can work on this.

Hi @greedyAI! #648 should fix it, don't prioritize this one if you have other tasks to work on.

Closing as a dupe of #648.


Install ca-certificates in Docker image

Describe the bug
Without the ca-certificates package an official Docker image failed to proxy images from docker.io
To Reproduce
docker run --rm --name trow --network host containersol/trow:0.3.1-PROXY --proxy-docker-hub --no-tls --port 8083
Starting Trow 0.3.1-PROXY on <IP_ADDRESS>:8083

**Validation callback configuration

  By default all remote images are denied, and all local images present in the repository are allowed

  These host names will be considered local (refer to this registry): ["<IP_ADDRESS>"]
  Images with these prefixes are explicitly allowed: ["k8s.gcr.io/", "docker.io/containersol/trow"]
  Images with these names are explicitly allowed: []
  Local images with these prefixes are explicitly denied: []
  Local images with these names are explicitly denied: []

Docker Hub repostories are being proxy-cached under f/docker/

Trow is up and running!

Pull:
docker pull <IP_ADDRESS>:8083/f/docker/mariadb:latest
Error response from daemon: error parsing HTTP 404 response body: json: cannot unmarshal string into Go value of type struct { Errors []errcode.Error }: "\"404 page not found\""

At that time, trow logs:
Trow is up and running!
[2021-02-24T16:51:36Z ERROR rocket::rocket] No matching routes for HEAD /v2/f/docker/mariadb/manifests/latest application/vnd.docker.distribution.manifest.v1+prettyjws.
[2021-02-24T16:51:36Z ERROR trow_server::server] Remote registry didn't repsond to auth request: error sending request for url (https://auth.docker.io/token?service=registry.docker.io&scope=repository:library/mariadb:pull): error trying to connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915: (unable to get local issuer certificate)
[2021-02-24T16:51:37Z ERROR trow_server::server] Remote registry didn't respond to HEAD request error sending request for url (https://registry-1.docker.io/v2/library/mariadb/manifests/latest): error trying to connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915: (unable to get local issuer certificate)
[2021-02-24T16:51:37Z ERROR trow_server::server] Failed to download proxied image error sending request for url (https://registry-1.docker.io/v2/library/mariadb/manifests/latest): error trying to connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915: (unable to get local issuer certificate)
[2021-02-24T16:51:37Z WARN  trow_server::server] Internal error with manifest Os { code: 2, kind: NotFound, message: "No such file or directory" }
[2021-02-24T16:51:37Z WARN  rocket::response::responder] Response was `None`.
[2021-02-24T16:51:37Z WARN  rocket::rocket] Responding with 404 Not Found catcher.
[2021-02-24T16:51:38Z ERROR trow_server::server] Remote registry didn't repsond to auth request: error sending request for url (https://auth.docker.io/token?service=registry.docker.io&scope=repository:library/mariadb:pull): error trying to connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915: (unable to get local issuer certificate)
[2021-02-24T16:51:38Z ERROR trow_server::server] Remote registry didn't respond to HEAD request error sending request for url (https://registry-1.docker.io/v2/library/mariadb/manifests/latest): error trying to connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915: (unable to get local issuer certificate)
[2021-02-24T16:51:39Z ERROR trow_server::server] Failed to download proxied image error sending request for url (https://registry-1.docker.io/v2/library/mariadb/manifests/latest): error trying to connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915: (unable to get local issuer certificate)
[2021-02-24T16:51:39Z WARN  trow_server::server] Internal error with manifest Os { code: 2, kind: NotFound, message: "No such file or directory" }
[2021-02-24T16:51:39Z WARN  rocket::response::responder] Response was `None`.
[2021-02-24T16:51:39Z WARN  rocket::rocket] Responding with 404 Not Found catcher.

Ooops. I suspect I removed that and it passed the tests so I thought it was ok. Facepalm.
I'll add it back with a test.
Thanks!

Rather than sit on this as it's pretty fundamental, I've updated the images in #235. Still need to do a release to publish the images so I'll leave this open until then.

@amouat, Are you planning to update Docker images? I found only one armv7 image was updated recently.
https://hub.docker.com/r/containersol/trow/tags?page=1&ordering=last_updated
TAG
2021-02-25-111-armv7
docker pull containersol/trow:2021-02-25-111-armv7
Last pushed 19 hours agobycsdockerhubpush

Yeah, they should all get published on release I think. I want to fix the bug I'm working on and do a patch release.

There are intermediate images pushed on merges to master - https://hub.docker.com/r/containersol/trow/tags?page=1&ordering=last_updated

Thanks. I will check soon (amd64 platform).

I checked containersol/trow:2021-02-27-112-amd64.
I found no problem. Good job!

Thanks!
I'll close this as it's addressed. Hopefully I'll get patch release out this week as well with the fix.


Linux AArch64 build fails with missing include <features.h>

What is the issue?
When compiling a simple program that includes the <algorithm> header the compiler is raising the following error:
external/toolchains_llvm~override~llvm~llvm_toolchain_llvm/bin/../include/c++/v1/__config:261:14: fatal error: 'features.h' file not found
#    include <features.h>

How did you compile?
Used this .bazelrc
build --cxxopt=-std=c++17 --enable_bzlmod --incompatible_enable_cc_toolchain_resolution

On which platform did you compile?
AArch64 Linux, Ubuntu 22.04
What version did you use?
Version 0.10.3 with LLVM 17.04
This looks like a sysroot issue?

features.h is a libc header. Short of a resolution on #155, you need to install glibc in your OS.
On Ubuntu 22.04, unless you are cross-compiling, the apt package libc6-dev is what you need (although it almost always comes preinstalled).
sudo apt-get install libc6-dev && (dpkg -L libc6-dev | grep features.h)

If this is not enough, please reopen the issue.


Performance of your ported weights on Kinetics400 validation set (data preprocessing)

Hi @piergiaj , thank you for this great repo. I have used your code to extract I3D features before, it works pretty well. Recently, I want to train it from scratch on Kinetics400/700 and try to reproduce the performance. The first step is to evaluate your model on Kinetics400 validation set to see what is the performance . However, the accuracy is very low.
Then I try to find the reason. The situation is, if I used your .pth model and the demo .npy file, I can get the prediction of CricketShot correctly.
Top classes and probabilities
[0.99999666] [25.856636] playing cricket
[1.3353539e-06] [12.330325] playing kickball
[4.5531763e-07] [11.254369] catching or throwing baseball
[3.1434402e-07] [10.883862] shooting goal (soccer)
[1.9243485e-07] [10.393131] catching or throwing softball

But if I use  (imgx/255.)*2 - 1  to preprocess the same video (v_CricketShot_g04_c01) to get the input data, I can't get the correct prediction. The label I get is actually robot dancing. I also tried several more videos, but none of them give me the correct prediction.
Top classes and probabilities
[0.57130724] [10.700808] robot dancing
[0.09980194] [8.956068] pumping fist
[0.04412121] [8.139821] dancing gangnam style
[0.03195825] [7.817311] dancing macarena
[0.01954201] [7.325447] using remote controller (not gaming)

IMO, I think your model is good. Then the reason should be on the data side. It is either my decoded frames are not the same as yours, or the image preprocessing is kind of tricky. Have you encountered this before? I mean did you test your model on the Kinetics400 validation set? Thank you very much and look forward to your reply.

I have tested this model on the kinetics-400 validation set and got very similar results to what was reported in the paper. I'm guessing your input preprocessing is slightly different than what this model was trained for. I.e., make sure you're sampling at 25 fps, resizing the height of each video to 256, saving as jpeg, then taking a center crop from that.
You can check your preprocessed video against the provided numpy version until you find the right settings.

Thank you for your suggestions, I will try it.


Is this library usable from Typescript?

I think it should be clear that this is a javascript library and from typescript we should not really expect anything to work see #2316.
Isn't it absurd not to have a Typescript first approach?

Like I can't even cast any of the objects I get because the interfaces are not exported?

I've provided an answer to this question at https://github.com/slackapi/bolt-js/issues/2316#issuecomment-2452765791 If you have follow-up questions, write in the the issue instead.

FYI in bolt v4 the types are re-exported: https://github.com/slackapi/bolt-js/blob/main/src/index.ts#L76


smaller images

Closing this one for the time being, need to find better ways of dealing with the image size


Added solutions from @mirpulatov, @Jlopezjlx, @Prrrince for calculator

PR request, changes done

All is Fixed

@Jlopezjlx  did you update the PR?


Fixed typo in SecondaryWindow.md

Summary of the pull request
Retrun -> Return
Detailed description of the pull request / Additional comments
Resolved a typing error, Retrun to Return.
PR checklist

[ ] Closes #xxx
[ ] Tests added/passed
[ ] Documentation updated

Please include in #1783


Make is_inrange_v6 more robust

If passed invalid IPv6 strings in the parameters, inet_pton() was
emitting warnings. Avoid this by first checking for valid IPv6
addresses.
This helps https://forum.pfsense.org/index.php?topic=119565.0 by stopping annoying "inet_pton(): Unrecognized address" warnings.

Related forum post: https://forum.pfsense.org/index.php?topic=119565.0

I've pushed a different version. Thanks!


TOML Configs pertaining to publishing statistics

Please provide the corresponding TOML configs:
identity.xml > Listener

org.wso2.carbon.identity.data.publisher.application.authentication.AuthnDataPublisherProxy
org.wso2.carbon.identity.data.publisher.authentication.analytics.login.AnalyticsLoginDataPublishHandler
org.wso2.carbon.identity.data.publisher.authentication.analytics.session.AnalyticsSessionDataPublishHandler

Login analytics: <IS_HOME>/repository/deployment/server/eventpublishers/IsAnalytics-Publisher-wso2event-AuthenticationData.xml
Session analytics: <IS_HOME>/repository/deployment/server/eventpublishers/IsAnalytics-Publisher-wso2event-SessionData.xml
REF:
https://docs.wso2.com/display/IS580/Prerequisites+to+Publish+Statistics
https://is.docs.wso2.com/en/5.9.0/learn/prerequisites-to-publish-statistics/

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Downgrade to NetworkManager-1.40.0-1.el9

related to https://github.com/metal3-io/metal3-dev-env/pull/1093

/lgtm


Settings: Remove transition animations

Before:

After:

Why? The animations look very nice.

@Jaiganeshkumaran I think we are don't need several animations(from bottom to top + from right to left), we are need one, which is already exists(from bottom to top)


Fix a typo in the DrawCube, DrawCubeWires and DrawCubeTexture definitions

I don't know if I should submit this PR against master or develop, so please tell me if I made a mistake.

Hi @nounoursheureux, thanks for the correction. I work on develop branch before moving changes to master, could you make the pull-request from your develop to my develop branch?
If you prefer, I can also implement changes myself on my side...


Merge pull request #1 from caskroom/master

Get up-to-date
If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked the cask was not already refused in closed issues.
[ ] Checked the cask is submitted to the correct repo.

oops, my bad. Wrong direction!


Add EditorResourcePicker control based on the Inspector editor for Resources

Part of https://github.com/godotengine/godot-proposals/issues/2276 and probably a bunch of others that I can't currently find.
This adds a control for creating and selecting arbitrary resources, similar to the one used by the Inspector dock.

Usage is straight-forward, and it can also be disabled.
var resource_picker = EditorResourcePicker.new()
resource_picker.base_type = "StyleBox"
resource_picker.edited_resource = StyleBoxFlat.new()
add_child(resource_picker)

This is not a complete replacement for the control used in the Inspector, so I didn't update the dock to use it. Namely, there is some functionality that relies on the edited scene being present and its script being edited. This didn't feel fitting for a generalized control, so I omitted it. However, I think it can later be enhanced and integrated back into the Inspector. Visual editors also use their own version of this, so it would make sense to unite the controls at some future point.
I should note, that this is just the button and the context menu with all of its options. This does not include the sub-Inspectors in any way, as this part is better left for users to implement the way they need it.
I will need this for my Theme Editor rework, but it didn't feel right to bundle it together when it can be released independently ahead of time.

Note: this is just a rebase, unrelated to the discussion above.

Unsupported resources should be rejected probably (with error message maybe?).

Unsupported how? Do you mean a mismatch with the base type, if one was set?

Unsupported how? Do you mean a mismatch with the base type, if one was set?

Yes. You shouldn't be able to assign e.g. StyleBox to Texture.

As for the feature itself... you mentioned that visual editors use their own version too, but I checked that and Visual Shader uses EditorPropertyResource class and Visual Script doesn't even have this - it just uses inspector. So I wonder if EditorPropertyResource wouldn't work in your case too.

Both Visual Shaders and Visual Script use CustomPropertyEditor, and they are the only things that use it as far as I can see. So I assume that CustomPropertyEditor was created for them specifically, or at least that's how it ended up. Whichever it was, it duplicates a lot of Inspector functionality and doesn't even rely on other common controls, like EditorSpinSlider for numeric values.
Examples of similar code are everywhere where it deals with Variant value of Object type, like here and here. This seems like it's not an easily reusable control, as it contains logic for all kind of Variant types, and likely is intimately attached to visual editors.
Even if I could make it work for the Theme Editor, this doesn't satisfy the need to expose it to the plugin creators.

FYI, CustomPropertyEditor predates the new EditorInspector. I don't think it was custom-written for Visual Shader/Script editors, it's just that those are the only two instances where it hasn't or couldn't be replaced yet.

Makes sense. This may be a bit of a dead code then. I've tried to see it in action, but due to checks for property hint this branch doesn't get triggered when trying to set, say, a theme property on a control. I'm not sure if it would work with script properties, because for the life of me I cannot figure out how to do anything in Visual Script.
I think my argumentation for extracting this into a dedicated component still stands though.

Just for the record, these use CustomPropertyEditor:

You can see that for int/enum a value editor opens up, but clicking on the theme resource doesn't do anything because it fails a property hint check.

Relevant, could be rebased/moved after merging #48438.

You can use EditorPropertyResource for this, which does basically the same and is not dependent on the inspector, there is no need for duplicate functionality.

See discussion at https://chat.godotengine.org/channel/editor?msg=DYKTiPEkv5caPEPNe for details. In short, we first discussed that this might not be needed as EditorPropertyResource should be usable standalone in editor code, but it turns out that it does expect a given object and set of properties to act upon as used in the Inspector, while this extracts the popup component from it.
A follow-up PR should address the code duplication by refactoring EditorPropertyResource to use EditorResourcePicker.

Thanks!


Assorted IO improvements

Since you added --print-prefix and --print-libdir (in 1dbf89833f4fef96b29f3541f0c45771df81b270), why not add --print-bindir as well for parity's sake?

Also, would this be a good time to make def IO.RealWorld : Type := Unit, constant IO.RealWorld : Type := Unit?

Also, would this be a good time to make def IO.RealWorld : Type := Unit, constant IO.RealWorld : Type := Unit?

I believe that could negatively influence the generated code.

@Kha
Ah, that explains it. I am surprised, though, that unsafe code can't see through a defined constant. Also, any feedback about the proposed --print-bindir addition?


New export options: Visible Objects, Renderable Objects, and Active Collection

Fixes #1139
The options (and the code) are self-explanatory. The behavior is exactly as you would expect:

Visible Objects: If an object is hidden (either with the Eye or the Hide in Viewport) then it won't be exported.
Renderable Objects: If an object is not renderable then it won't be exported.
Active Collection: If an object is not a child of the active collection then it won't be exported.

All options default to false, so this does not change existing behavior, it simply gives new optional features.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I think we should align options to other exporter, like FBX for example :

Limit to selected
Limit to active collection

(FBX also has limit export by object type.)

@julienduroure These features are extremely useful and are needed by many people, so I would advocate for them even if the other exporters didn't support it.
The linked topic goes into detail, but the situation is that every exporter does something different:

FBX has active collection and selection.
Collada has selection.
Alembic has selection, renderable, and visible.
USD has selection and visible.
PLY has selection.
STL has selection.
OBJ supports selection.
SVG supports selection.

So if you want to align to other exporters, then supporting renderable and visible does align with Alembic and USD.

+1 about Alembic, that is the most recently developed in Blender.
So OK to include all these options
Note that currently export by selection is quite buggy. This should be fixed with what is called two pass hierarchy export #1244
With current lockdown / context since 1 year, I don't have many free time to work on glTF-Blender-IO, but I hope to be back (soon ) on a more frequent way

@julienduroure I think that sort of big refactoring should be done in a separate PR. This PR is small, simple, and self-contained.

@Pauan Yes sure, this note was not here to ask working on two pass export here, but only to remember that current filter is currenlty buggy, and this PR will add some functionnalities based on this legacy system
I will perform some tests soon. My sugestion is to check that this PR is at same level of features (features and known issues) that other filters

Needed the "Visible Objects" feature for a project, I was able to cherry pick this code into the GLTF plugin and it worked - extremely helpful. Many thanks @Pauan for putting this together. Will be happy to see this make it into the main repo.

What is the status of this PR?

I will merge this PR tonight

Just tested this with blender 2.93 and it works! Thanks a lot! 👍🏼

Thanks for adding this setting! A real time and frustration saver!


Publish new version

Thank you for your plugin, but I cannot test it with nx&14
Bug Report
Current behavior
Using nx@14 doesn't works as version 1.0.8 as nx@13 as peerDependency
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: ecommerce-front-united@undefined
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@nrwl/workspace
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@mansagroup/nx-gcs-remote-cache
npm ERR!   dev @mansagroup/nx-gcs-remote-cache@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!

Expected behavior
Using nx@14 works
Possible Solution
peerDeps are already updated on package.json : cf https://github.com/MansaGroup/nx-gcs-remote-cache/pull/111
Last release was 7 months ago, and there is a lot of renovate works done since :)

Just released v1.1.0, do not hesitate to re-open this issue is something is wrong :)


Refine "venv in gitignore" check

Fixes #28 😊
But doesn't really cover all cases 😢
For example, people could just add a .gitignore containing * in the venv directory. This would still complain, while git would actually ignore the virtual environment (ok that's exotic, but not impossible!).
At the same time, I feel like we shouldn't go further in reimplementing git. What about using some git Python library?

Somewhat related: I see no CI configured. I configured some of my private GH projects to use Github Actions. I can POC some similar configuration in a PR if you wish 🙂
On the same note: what about black and pre-commit?

Thanks! I'm not a huge fan of black personally.
I have GitHub Actions on other projects, and I honestly thought I had it here already :P I'll definitely accept a PR!


Bad docs

It is never mentioned on how to deploy the appliance, you go to Azure and appear to deploy the "Azure AD Cloud App Discovery" appliance

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

ID: 2005aabb-11c8-3ab9-6e87-94b0ef5164be
Version Independent ID: 0878ecbe-fcd1-eb38-8ddb-4b4ea685d3c0
Content: Deploy Cloud Discovery with Microsoft Cloud App Security
Content Source: CloudAppSecurityDocs/set-up-cloud-discovery.md
Service: cloud-app-security
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

Hi Wesley, I'm sorry this was unclear. Cloud App Security is a cloud service, so there isn't a physical app that needs to be deployed. In order to use Cloud Discovery, you can upload logs manually (which gives you a snapshot report based on the uploaded log) or configure automatic log upload (which enables you to generate reports continuously, based on the data detected in the log collectors you connect to Cloud App Security). I can change the name of the article to Set up, rather than Deploy - I hope that helps. Thanks for your feedback!


Question on improvement for specific image

Hello I am working on a master project and I am interested in applying the solution into static pictures of a plant.
The pretrained model has already very good results but I am wondering what are some possible ways to improve it. I am wondering if there are some kind of techniques to improve it further. A second issue comes with the different lighting during the day that changes the plant colors.
Thank you in advance for your time.

@partizanos My suggestion would be to fine-tune/retrain the network with some training data from your own domain. The publicly available model was trained with COCO images and Pascal VOC images, which might look quite different to the images you're testing on.


Add pressure

Changes proposed in this Pull Request:

Added icons for Pressure (Pressure and Pressure Alternate)
Added icons for iOS Biometric Auth (Face ID and Touch ID)
Added icons for companies/platforms for 3rd Party Login (Facebook, Google, Amazon, Apple)
Added Amazon Alexa icon for 3rd party smart home ecosystem support

I don't get why we need the Apple touch id as part of the pxb icon page... I poked around the smart home project but didn't find where it is used.
If it is used more as an image instead of a 24x24 icon, then I'd suggest not including it here.

I don't get why we need the Apple touch id as part of the pxb icon page... I poked around the smart home project but didn't find where it is used.
If it is used more as an image instead of a 24x24 icon, then I'd suggest not including it here.

This is used at the moment in Control Xpert Ag for biometric authentication setup and on the login screen when enabled. They are used as 24x24 icons within buttons, within avatars on settings, and as 96x96 on full page setup and success screens.
I'd expect that this would be one of the additions to the login patterns/components in pxblue as well @huayunh

@huayunh @joebochill Any movement on this? Are there changes I need to make? The GP dev team is bottlenecked right now waiting for the biometric icons.

Taking a look at it today

Material already has a facebook icon, is there a reason for making another one? https://pxblue.github.io/style/iconography?icon=Facebook&isMaterial=true

I will take care of the technical side; meanwhile, Eric you may just send them the SVG icons, and it will be fairly easy to swap those to PX Blue packages once the new icons get published

They are used as 24x24 icons within buttons

That's not going to render well... I would not suggest doing that. It looks like this after you pixelate it:

In my commit above, both facebook logo and touch id are included, but I'd prefer to remove both, for the reasons mentioned above.

Where did we end up on the discussion about the duplicate facebook icon and the low resolution of the touch id?

For the Facebook icon we could get either get rid of it or rename to facebook_circle

@huayunh once the automatic publishing finishes here, you'll need to open up another PR in the icons-mui repo (with an updated minor version in package.json and a changelog entry - you can copy the wording from this one)


Handle messages with expired lock

We have amqp1 input and we're trying to achieve long batching. Currently we have an issue that when we're disconnected from the Azure we lose lock on the messages that we're processing and they are redelivered. This causes both, premature batch flush and duplicated claims. For example:

assume we have batch configured to max 4 elements
we receive messages: a,b,c
azure disconnects so we lose lock on messages a,b,c
we reconnect
azure redeliver messages a,b,c
first batch gets full (messages: a,b,c,a) so it's flushes
we process messages a,b,c,a
we receive 2 new messages d, e
the second batch is triggered (with messages b,c,d,e)

In the end messages a,b,c are processed twice. It would be nice to be able to remove messages a,b,c from the batch as soon as input loses control (and ability to lock them). In case of Azure service bus it's loosing lock on the message.
I hope it's clear for you, @Jeffail do you have any thoughts how it can be solved properly? I have some rough implementation of my idea here https://github.com/certifaction/benthos/pull/2 but not sure if this is the way you would like to proceed.

Hey @maxmati, this is an interesting one. It sounds like a general solution to this problem would be to have a mechanism of reducing the batch for each message that is added, or "deduplicating" newly added messages, in a way that also prevents the batching mechanisms from being triggered. We can't use a simple filter mapping here because the context as to whether a message is a duplicate of another in the batch regards the whole thing.
I'll think it over. We could definitely do something like add a push_mapping field where you could do some windowed map query like this:
batching:
  count: 10
  push_mapping: |
    let content_hash = content().hash("sha256")
    root = if content().from_all().slice(0, -1).any(b -> b.hash("sha256") == $content_hash) { deleted() }

That would calculate sha256 hashes of each message in the batch and if one already exists then the message is deleted before it's added to the batch. Obviously if the messages have a unique ID then that'd be preferred over calculating the hash.

I thought about more generic mechanism that will allow input mark downstream messages that it won't be able to ack given message. This will allow downstream decide if it should continue to process or just drop it. Your idea looks quite nice important think to keep in mind is that you have to remove already batched message not drop the incoming one (you are able to ack this new incoming one, but unable to ack the original one). Also this approach will work only with single instance of Benthos running because duplicated message can be delivered to another instance and duplication wouldn't be detected (so far not an issue for us).


Unable to run in Metanorma due to error

IndexError: undefined group name reference: year
  /Users/me/.rbenv/versions/2.4.3/lib/ruby/gems/2.4.0/gems/asciidoctor-iso-0.9.3/lib/asciidoctor/iso/ref.rb:70:in `[]'
  /Users/me/.rbenv/versions/2.4.3/lib/ruby/gems/2.4.0/gems/asciidoctor-iso-0.9.3/lib/asciidoctor/iso/ref.rb:70:in `isorefmatches3'

@opoudjis you can test this out here: https://github.com/riboseinc/gmt-0009-2012
Just clone and make clean all.

Fixed in asciidoctor-iso


update readme for deploystack

Hi @laurafrins, can you please remove the .DS_Store file please so I can merge the PR ?

LGTM


Split ILookupNormalizer into Name/Email normalizers

Fixes https://github.com/aspnet/AspNetCore/issues/6330

Yep announcement for break here https://github.com/aspnet/AspNetCore/issues/8572  This shouldn't affect many people


System error on simple query

I have issue with my simple query on ArangoDB v3.0.8 on MacOS and Ubuntu
I have very simple query
for user in users
    let isColleague = (user.company == @company)
    filter  isColleague
    return { user, isColleague }

Also i have hash index on field 'company' of collection 'users'
The execution in web interface show error:
Query: missing variable #1 (isColleague) for node CalculationNode while planning registers (exception location: /usr/src/packages/BUILD/arangod/Aql/ExecutionNode.cpp:1163). Please report this error to arangodb.com (while executing) (exception location: /usr/src/packages/BUILD/arangod/RestHandler/RestCursorHandler.cpp:129). Please report this error to arangodb.com

The explain show error too:
Explain: missing variable #1 (isColleague) for node CalculationNode while planning registers (exception location: /usr/src/packages/BUILD/arangod/Aql/ExecutionNode.cpp:1163). Please report this error to arangodb.com (while finalizing)

I have this error on both server: development on my MacOS and test on Ubuntu 14.04.
But i think that problem is not in OS.

Fixed in 3.0.10.
As a temporary workaround until 3.0.10 is available, you may repeat the isColleague predicate expression in the query:
FOR user IN users
  FILTER user.company == @company
  RETURN { user, isColleague : user.company == @company }

That will work in all versions and use the index. From 3.0.10 on, you can also use the following query, which will
FOR user IN users
  LET isColleague = user.company == @company
  FILTER isColleague
  RETURN { user, isColleague }

For reference: the error only occurred when an index was present, the FILTER condition was just a simple variable reference and the variable used in the FILTER condition was also used after the FILTER. The optimizer was overambitious here and optimized away the variable used by the FILTER condition, and did not check if it was used later (as in the RETURN part of the query that failed).

@jsteemann Thank you very much for this explain! :)
I thought that problem in optimizer and removing of variable before needed.  👍

3.0.10 is available for download now.

@dothebart Yeah! I have already checked that it's fixed!


cli: Make --help useful and add --help-all

Before this commit, running luigi --help is going to print 300 lines
of probably not useful flags, mostly being about all the combinations of
the registered various versions of the Range flags. Instead, I changed
--help to just print.
$ ./bin/luigi --help
usage: luigi [--local-scheduler] [--module CORE_MODULE] [--help] [--help-all]

optional arguments:
  --local-scheduler     Use a in-memory central scheduler. Useful for testing.
  --module CORE_MODULE  Used for dynamic loading of modules
  --help                Show most common flags and all task-specific flags
  --help-all            Show all command line flags
Exiting due to --help was passed

And when you're interested in an actual tasks parameter, they will be
displayed when specifying that task.
$ ./bin/luigi --module examples.foo examples.Bar --help
...
--num EXAMPLES.BAR_NUM, --examples.Bar-num EXAMPLES.BAR_NUM
...

Apart from that, LGTM.


docs: added jvs docs

fix #113

Let's put the disclaimer **JVS is not an official Google product.** on every markdown file.

done


Improve the text at the top of the home page

Received the latest home page text from the product team.

Wrong repo


Broken link to SSI wallet

Link to SSI wallet: https://github.com/cambriota/identity-cli-wallet is not working (404)

Yes, I'd also highly appreciate the cli code :-) THANK YOU.

Currently rebuilding the wallet to use @iota/identity-wasm:0.5.1 (account model, storage, new verification api). Will make it public once that's done.


animation rigs is correct only on fish

Hi, after I change the code, I tested the function of join_and_rig_parts.
(I add 'self.animation_mode="run"' on the line 209 in worldgen\assets\creatures\genomes\herbivore.py)
I found that the animation rigs is correct only on fish, while rigging on other animals can lead to distortion of the model.
I wonder the root cause, and I'd like to know if you will make optimizations in the future?

Thanks for raising this, I thought all were working but I will test and fix asap. Should be fixed by tommorow or day after, I will send you a special GitHub branch to use

I fixed an issue with the tiger IKs and verified that animation_mode='run' is at least funcitonal (if not the the most realistic, Id love to revisit / improve this later). Let me know what if any specific issues still persist


Migrating a field to PlaceholderField fails when using Postgres

$ pip install djangocms-installer psycopg2
$ djangocms -sqp . -d postgresql://test:test@localhost/test my_demo
$ python manage.py startapp testapp

Add testapp as penultimate entry in my_demo/settings.py:INSTALLED_APPS.
Edit testapp/models.py:
from django.db import models
class TestModel(models.Model):
    description = models.TextField("Description")

$ python manage.py makemigrations testapp
$ python manage.py migrate

Edit testapp/models.py:
from django.db import models
from cms.models.fields import PlaceholderField
class TestModel(models.Model):
    description = PlaceholderField("Description")

$ python manage.py makemigrations testapp
$ python manage.py migrate
...
Running migrations:
  Rendering model states... DONE
  Applying testapp.0002_auto_20151129_1448...Traceback (most recent call last):
  File "test/env/lib/python3.4/site-packages/django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
psycopg2.ProgrammingError: column "description_id" cannot be cast automatically to type integer
HINT:  You might need to specify "USING description_id::integer".

This only happens with Postgres (or rather, it doesn't happen with sqlite, i.e. if you remove the -d from the djangocms invocation above). It only happens if the column is being changed in one migration, if it's split into two migrations the first of which deletes the column and the second of which creates it, there is no issue.
I was originally actually getting ValueError: Related model 'cms.Placeholder' cannot be resolved, but I cannot reproduce that so easily since I already fixed it in my project - see mailing list thread https://groups.google.com/forum/#!topic/django-cms/tIT3g0gCpwc.

Postgres refuse to convert any text field to integer without a casting function: PlaceholderField is a ForeignKey so its type it's an integer and you can either write a `'RunSql`` function or split the migrations in two steps.

Shouldn't 'makemigrations' be detecting this situation rather than silently producing a broken migration?

I don't know the internals of migrations module to provide an answer

Possibly if you hadn't've closed the ticket and labelled it a non-issue, someone who does know whether it actually is an issue might answer? ;-)

Not being a django CMS issue I don't see any reason to keep this open.
The best option is to open an issue in Django and eventually link this here

OK I have created a django ticket: https://code.djangoproject.com/ticket/25832


Complete overhaul of acceptance criteria computation to fix benchmark issues

This is a complete overhaul of the acceptance criteria computation to fix issues raised by @juliebehr's benchmark. The changes are extensive, so may have ramifications for many tests I haven't had a chance to adapt yet.
Includes:

Overhaul of NCMC integrators
Correction of many inconsistencies between docstrings and actual return values
Correction of many inconsistencies between use of log probabilities and energies
Addition of missing logP contribution for two-stage causing the benchmark test to fail
Fixes to failing tests based on PointMutationEngine
Updates to which acceptance probability components are written to storage
Changes to log acceptance criteria contribution definitions, so that these are all additive
And much much more

@juliebehr's hybrid method was likely working just fine in the benchmark test---the issue seems to have been with the two-stage method.
Note that the NCMC delete and insert methods no longer return a "switch" logP. This contribution is now folded into the NCMC, requiring that the deletion and annihilation for the two-stage method always be paired. You can no longer expect to use one without the other.

The benchmark consistency check now works:
[LSKI1497:choderalab/perses/perses] choderaj% python perses/tests/benchmark.py

Now testing propane null transformations
Running 0 hybrid ExpandedEnsemble steps for 10 iterations
hybrid    logp_accept = -4.9415e+00 [ncmc = -4.2558e+00, geometry = -6.8571e-01]
two-stage logp_accept = -4.9415e+00 [ncmc = -4.2558e+00, geometry = -6.8571e-01]
...

Now testing naphthalene null transformations
Running 0 hybrid ExpandedEnsemble steps for 10 iterations
hybrid    logp_accept = -7.4741e+03 [ncmc = -7.4621e+03, geometry = -1.2011e+01]
two-stage logp_accept = -7.4741e+03 [ncmc = -7.4621e+03, geometry = -1.2011e+01]
...

Now testing butane null transformations
Running 0 hybrid ExpandedEnsemble steps for 10 iterations
hybrid    logp_accept = -1.9845e+01 [ncmc = -1.6875e+01, geometry = -2.9693e+00]
two-stage logp_accept = -1.9845e+01 [ncmc = -1.6875e+01, geometry = -2.9693e+00]
...

I did have to pare this test down to only work for 0 NCMC steps, but I don't understand how this test could have possibly worked for more than 0 steps. If someone could explain the concept of how that would work, I could restore it to work for > 0 steps.

I'm a bit surprised at this test failure since I had seen it pass locally. Let me look into this.
Also, somehow TESTSYSTEMS is now set to only test the valence test systems on travis. Not sure how that happened. Is that desired behavior?

Also, somehow TESTSYSTEMS is now set to only test the valence test systems on travis. Not sure how that happened.

I think you changed that a month ago, because the travis tests take so long. Maybe we could ditch py27 and py34 in return for more systems being tested?
https://github.com/choderalab/perses/commit/b032698141fa6f0d3c9d7388f77770307da4e5cd

Thanks! I'll revisit this.
I think I'd be fine with ditching the other Pythons, but it would be useful to figure out a way to test more systems. I simply recall that it is just really slow without GPUs. Still hoping there is some easy way to test on GPUs in the cloud since it seems like the Jenkins server is never coming back to life, given the fights I've had with the infosec people who are freaking out that we can't disable form autofill on Jenkins. Migrating to cloud hosted Jenkins and EC2 GPU instance workers might be one solution.

. Still hoping there is some easy way to test on GPUs in the cloud since it seems like the Jenkins server is never coming back to life

Yeah, I think we're going to have to figure out something with that. I was just search last weekend again to see if there was a ready-made GPU CI solution, but it doesn't seem like it (other than setting up Jenkins on EC2, as you suggest, which might be the best solution).

I did have to pare this test down to only work for 0 NCMC steps, but I don't understand how this test could have possibly worked for more than 0 steps. If someone could explain the concept of how that would work, I could restore it to work for > 0 steps.

It was never at any point running with > 0 steps; the test in question that is running is literally called benchmark_logp_0ncmc_schemes

Also I'm not sure why this is an explanation of why the values would have been changed and why they would be more accurate now.

The benchmark consistency check now works

Is the switch calculated in the same way but in a different place? Why would that change the two-stage ncmc values at all? If it does, how is it verified that the newly calculated values are now accurate, rather than agreeing with hybrid because the calculation from hybrid has been duplicated?
If the two schemes give logp values that are not consistent, this is a sign that at least one must have an error somewhere, but just being consistent is not verification that both are correct.

It was never at any point running with > 0 steps; the test in question that is running is literally called benchmark_logp_0ncmc_schemes

The function doing the printing is called _geometry_once_ncmc_twice and appears in a class that explicitly allows the number of NCMC steps to be set via options['nsteps']. The function has no documentation or asserts that enforce options['nsteps'] = 0, so I had no idea if it was designed with the intent of also being used with nsteps > 0 after testing with nsteps = 0 was complete.
To clarify: This function is only ever intended to be used with nsteps = 0, yes?

Also I'm not sure why this is an explanation of why the values would have been changed and why they would be more accurate now.

It's not an explanation, but it explains what I've done. I was not able to understand how the acceptance rate was currently being computed due to the large number of inconsistencies in documentation and variable naming. Instead, I overhauled the whole of acceptance probability calculation to be consistent: Each component now computes its own contribution toward the acceptance probability in a manner that is purely additive. The scheme matches what I drew on the whiteboard when we discussed this months ago, and recapitulated at the Rutgers visit with Zhiqiang Tan.
I also eliminated the special "0 steps" NCMC case, since even if we did debug the separate "0 steps" branch, we would still have to separately debug the ">0 steps" case, which would be much more difficult.

Is the switch calculated in the same way but in a different place?

Each stage now computes its own contribution toward the total log acceptance probability so they are purely additive.
For the two-stage delete, for example, we accumulate

The work in switching from a fully-interacting system to an alchemically-modified system
The work in switching lambda from 1 to 0 via NCMC
Half the work in switching from pre-geometry to post-geometry at lambda=0 (which previously was a contribution to logp_switch). This is more appropriately moved to geometry, but it is currently in the NCMC component so that geometry doesn't need to create alchemically-modified systems.

For the two-stage insert, we accumulate

The other half the work in switching from pre-geometry to post-geometry at lambda=0
The work in switching lambda from 0 to 1 via NCMC
The work in switching from alchemically-modified to fully-interacting

Why would that change the two-stage ncmc values at all?

I suspect it had to do with some inconsistencies in how we were extracting the lambda=0 and lambda=1 energies from the NCMC integrator.

If it does, how is it verified that the newly calculated values are now accurate, rather than agreeing with hybrid because the calculation from hybrid has been duplicated?

On my machine, the test_ncmc and test_elimination tests ran fine, but I'm trying to figure out why the failed on travis.
But we really need the valence test system analytical tests from @pgrinaway to verify that the acceptance probability is correct. There is no other test that ensures correctness of the overall acceptance probability.
If the two schemes give logp values that are not consistent, this is a sign that at least one must have an error somewhere, but just being consistent is not verification that both are correct.

Addition of missing logP contribution for two-stage causing the benchmark test to fail

What exactly is this? I can't find it in the code.

OK, I can replicate the hybrid test failures in test_elimination.py on my iMac. Investigating.

It looks like:

The wrong lambda functions were being used for check_hybrid_null_elimination. They all should be set to lambda, but were set to the same as the two-stage protocol.
This was missed because the tests were disabled in travis until my PR.

After correcting this, the test is still failing because the alchemical correction contribution I compute here is surprisingly large, even for simple systems like pentane. I am currently including the potential energy change to switch from single-real-topology to hybrid-alchemical-topology at the beginning, as well as the potential energy change to switch from hybrid-alchemical-topology to single-real-topology at the end, which I am now suspecting may be incorrect. This contribution cancels for nsteps = 0, but for even nsteps = 1, the thermal energy that partitions into degrees of freedom that are created or deleted seems to increase the variance of the logP through this effect. I was under the impression that this should cancel due to the geometry stage, however, so I'm not sure why it isn't...

Ah, that would be why! We don't include the geometry stages in this test, hence it won't cancel.
I think this suggests we either need to refactor NCMC and the geometry engine to move the logP_correction into the geometry engine, or we need to test the whole scheme (geometry-ncmc-geometry or ncmc-geometry-ncmc) in this test.

They all should be set to lambda

Hm, I don't think we want this, since we don't want some bonds going to lambda=0. Even in the hybrid case, we'll have bonds that are disappearing entirely.

This contribution cancels for nsteps = 0, but for even nsteps = 1, the thermal energy that partitions into degrees of freedom that are created or deleted seems to increase the variance of the logP through this effect.

Yeah, that makes sense. But this isn't strictly incorrect, right? It's just unfavorable.

I think this suggests we either need to refactor NCMC and the geometry engine to move the logP_correction into the geometry engine, or we need to test the whole scheme (geometry-ncmc-geometry or ncmc-geometry-ncmc) in this test.

I do think we should test the whole scheme, but shouldn't the NCMC null elimination still work by itself? I'd think that it should still preserve the equilibrium distribution of the configurations of whatever molecule is being "null" tested.. Perhaps it's not a very useful test in the long run, but I'd imagine that if the test is there and the NCMC bits are working, it should pass without the RJ stuff, unless I'm missing something.

Forgot to specify that this is specifically the hybrid test that is failing. There's we MUST have all lambdas start at 0 and end at 1, and the protocol must be symmetric.

There's we MUST have all lambdas start at 0 and end at 1, and the protocol must be symmetric.

Won't that cause an efficiency problem with some bonds being eliminated entirely?

No bonds are eliminated; all valence terms are retained at full strength throughout if softening = 1, as it must to avoid large geomtrey penalities.

I spoke with @pgrinaway today, who pointed out some major issues with my overhaul and areas in which I had messed things up. I've now rederived the hybrid scheme from scratch, and will put this derivation and the two-stage derivation in our overleaf document, and then make sure this implementation matches.

Will split this into two separate PRs.

Closing in favor of #307.


multitasking preemptive or cooperative ? (IDFGH-10571)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/system/freertos_idf.html
Hello,
after Reading this,
it is still unclear for me if the Task Scheduler is preemptive or cooperative ?
Thanks,
Franz

Priority-based preemptive with the dual-core mods.

@KaeLL is correct.
Described in more details here

TNX


write find(value) function

Write a find(value) function that returns the node with the given value.

once we write this, we need to leverage this potentially to ensure that when we try to .insert() a value, that value doesn't already exist


Request on behalf of user

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I really like Petio. I'm wanting to move over from another request app as i've been having issues. I want to be able to bring over all the requests from the other app by being able to request tv shows/films on behalf of other users so they still get notified when it's available.
Describe alternatives you've considered
No other alternatives that I know of

I came here to see if Petio supported this feature, so certainly a +1 from me


removing QML API import.

The callout will work with either C++ or QML APIs
@michael-tims please review and merge
cc @nandinirao

Makes sense. Thanks for catching this!

Looks good.


Yarn PnP support

What problem does this feature solve?
yarn 1.12 supports PnP: https://github.com/yarnpkg/rfcs/blob/master/accepted/0000-plug-an-play.md
https://github.com/yarnpkg/yarn/pull/6382
What does the proposed API look like?
I tried to get it to work, but the default project was missing a whole bunch of peer dependencies in the package.lock. Once that was resolved, the vue-cli webpack rules would have to be adjusted with:
https://github.com/arcanis/pnp-webpack-plugin
And for jest:
https://github.com/arcanis/jest-pnp-resolver

This would be awesome for many usecases: Using vue-cli in Docker, Windows WSL or Vagrant currently is a pain (because of >100.000 mounted and synced files inside node_modules). Without this everything would be gone.
Do you consider this integrating in vue-cli (when choosing "yarn" inside the installation wizard)?

yeah,i would glad see the supporting for pnp,i want know transpileDependencies how to defained

I was trying to setup pnp for a basic vue-cli application. See https://github.com/bodograumann/yarn-pnp-app.
pnp is enabled and php-webpack-plugin is installed. This already fixed a lot of errors about module loading.
Now I’m at a point where I have no idea to proceed. Running yarn build gives the following error:
 ERROR  Failed to compile with 1 errors                                                                                14.42.06

This relative module was not found:

* ./src/main.js in multi ./src/main.js
 ERROR  Build failed with errors.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Does anybody have an idea?

@bodograumann
We currently don't have the capacity to fix all the issues with yarn pnp compatibility. But here's a note from @lbogdan who have successfully made a basic vue cli project work with yarn v2:
https://gist.github.com/lbogdan/b8c652e6fd6f906492e0adf0ee044c82
Hope it helps.

@sodatea Would you be open to a PR that fixes (most of) the issues I encountered, while not breaking current functionality?

Yeah I would love to see that PR

I look forward to being able to support

@bodograumann Yes, I also encountered the same error

@archseer @bodograumann
I found an easy way to combine vue-cli and yarn pnp
https://github.com/hufeng/vue-love-pnp
key point，Add a ** webpack.config.js ** to the current project directory and Add ** missing ** dependencies。
// this file is for cases where we need to access the
// webpack config as a file when using CLI commands.
const Service = require('@vue/cli-service/lib/Service');
let service = process.VUE_CLI_SERVICE;

if (!service || process.env.VUE_CLI_API_MODE) {
  // const Service = require('./lib/Service');
  service = new Service(process.env.VUE_CLI_CONTEXT || process.cwd());
  service.init(process.env.VUE_CLI_MODE || process.env.NODE_ENV);
}

module.exports = service.resolveWebpackConfig();

After PR #4953 was merged, PnP works. But if global caching is enabled, an error occurs, it looks like there is a problem with eslint-plugin-vue.
ERROR STACK:

Module build failed (from ./.yarn/$$virtual/eslint-loader-virtual-9a87a5d115/7/.yarn/berry/cache/eslint-loader-npm-2.2.1-baa9138c1f-2.zip/node_modules/eslint-loader/index.js):
Error: Cannot read config file: /x/.yarn/$$virtual/eslint-plugin-vue-virtual-6249297965/7/.yarn/berry/cache/eslint-plugin-vue-npm-6.1.2-f3835ee917-2.zip/node_modules/eslint-plugin-vue/lib/configs/base.js
Error: Cannot find module '/x/.yarn/$$virtual/eslint-plugin-vue-virtual-6249297965/7/.yarn/berry/cache/eslint-plugin-vue-npm-6.1.2-f3835ee917-2.zip/node_modules/eslint-plugin-vue/lib/configs/base.js'

This is working mostly out of the box in recent versions thanks to what @fitzix mentions and @yarnpkg/plugin-compat. Even with typescript. All I had to do was modify the dependencies for some 3rd party loaders, and for typescript (for tslib). You mileage may vary depending on the state of @yarnpkg/plugin-compat - which is updated regularly.
I also had to get rid of references to node_modules in typeRoots in my tsconfig.json.
Recommend this issue be closed @LinusBorg
packageExtensions:
  "webpack-sharp-loader@*":
    dependencies:
      "file-loader": "*"
  "@vuetify/loader-shared@*":
    dependencies:
      "vue": "*"
  "vue-cli-plugin-vuetify@*":
    peerDependencies:
      "vue": "*"
  "vue-loader@*":
    dependencies:
      "vue": "*"
      "@vue/compiler-sfc": "*"
  "lipsurf.com@*":
    dependencies:
      "tslib": "*"

@vue/cli 5.0.4
@vue/compiler-sfc 3.2.32
yarn 3.1.1


doc add doc for NO_BACKSLASH_ESCAPES

Proposed changes
add doc for NO_BACKSLASH_ESCAPES

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Update all the checkpoint files to use a consistent json serialisation

The checkpoint files in sdk-lib/src/main/assets/saplingtree (currently) have either string or number serialisations for their height property.
This will only need to be done once and have the same representation as the output of #441

done by @ccjernigan in https://github.com/zcash/zcash-android-wallet-sdk/commit/e78139fa1a834aea052056345b8637533165b50c 🙌


Add Zenodo metadata

I am adding some metadata for Zenodo on all the MED-related packages.
Please let me know if you are happy to be included or if you think I missed someone.
Also, you can change the order.
We don't have clear directives on who we should add or not. My views are

Main code contributors should go first
People who have participated indirectly in testing, and discussion during the development phase (@headmetal)
Maintainers
Other small contributors (bug fixes)

I tend to be quite open as I think the work environment often influences how we do things, but that is my personal opinion.
@aidanheerdegen, @anton-seaice can you please add your ORCID?
Thanks!

We don't have clear directives on who we should add or not. My views are

Main code contributors should go first
People who have participated indirectly in testing, and discussion during the development phase (@headmetal)
Maintainers
Other small contributors (bug fixes)

I tend to be quite open as I think the work environment often influences how we do things, but that is my personal opinion.

Thanks @rbeucher. I agree. I've opened an issue in dev-docs because I think it is important to have guidelines documented. Please add your thoughts/opinions.

@aidanheerdegen, @anton-seaice can you please add your ORCID?

Ok done


Navigating to a link with anchor will cause client side language to be incorrect

Steps to Reproduce

Started at /es
Navigate to /es/blog using next-translate/link (<Link href="/blog" />)
Navigate back to /es#anchor using next-translate/link (<Link href="/#anchor" />)

Expectation
Navigating to a page using a link with an anchor will preserve the current language. The language should still be es instead of whatever the default language is.
Actual
Navigating to a page using a link with an anchor does not preserve the current language. Upon arriving at /es#anchor the lang value is incorrect (no longer es but now the default language), and the translations will be using the wrong file.
Setup and Versions
Using custom server setup.
"next-translate": "^0.17.3-canary.3",
"next": "^9.5.3",

I tried passing in Url objects to href with url.hash property but this library currently assumes href is always a string. But that is a separate issue.

I did some investigation and the cause is due to getLang and startsWithLang. Specifically, startsWithLang does not recognize if a url starts with a language if there is # in url path, and getLang also wouldn't take it into consideration.

This problem reappeared in version 0.20.2


Authentication timed out

Hello everybody!
I'm trying to setup this plugin againt Windows 2008R2 Active Directory server, my nginx.conf:
http {
...
auth_ldap_cache_enabled on;
auth_ldap_cache_expiration_time 3600000;
auth_ldap_cache_size 1000;
ldap_server platbox{
    url "ldap://<IP_ADDRESS>/dc=int,dc=domain,dc=com?sAMAccountName?sub?";
    binddn<EMAIL_ADDRESS>    binddn_passwd "pass";
    require valid_user;
    satisfy any;
    connections 50;

...
}
But I can't authenticate, errors:
2015/05/15 17:28:30 [error] 27034#0: *2 http_auth_ldap: Authentication timed out, client: x.x.x.x, server: xxx.domain.com, request: "GET / HTTP/1.1", host: "<IP_ADDRESS>"
2015/05/15 17:30:30 [error] 27068#0: *51 http_auth_ldap: Authentication timed out, client: x.x.x.x, server: xxx.domain.com, request: "GET / HTTP/1.1", host: "<IP_ADDRESS>"
I know that there is no firewall beetween hosts, and ldapsearch is working.
Please give me some advice. Thanks!

Can you check on the command-line with ldapsearch that your binddn is OK and LDAP URL are valid?  See this bug: https://github.com/kvspb/nginx-auth-ldap/issues/80

ldapsearch works fine!
ldapsearch -x -LLL -H ldap://<IP_ADDRESS> -D<EMAIL_ADDRESS>-W -b 'cn=users,dc=int,dc=domain,dc=com'
Enter LDAP Password:
dn: CN=Users,DC=int,DC=domain,DC=com
objectClass: top
objectClass: container
cn: Users
description: Default container for upgraded user accounts
distinguishedName: CN=Users,DC=int,DC=domain,DC=com
instanceType: 4
etc...

Can you try with connections 1?  And if possible activate debug with error_log /path/to/log/file debug and paste the output.  Thanks!

everything is the same and nothing new in logs with debug option :(
2015/05/20 12:16:13 [error] 5691#0: *4 http_auth_ldap: Authentication timed out, client: x.x.x.x, server: domain.com, request: "GET / HTTP/1.1", host: "domain.com"

oh maybe this can help?
2015/05/20 12:22:55 [info] 5810#0: *2 client closed connection while waiting for request, client: x.x.x.x, server: <IP_ADDRESS>:80

Can you check with nginx -V if the option --with-debug is present?  Nginx may be missing debugging support.
See http://nginx.org/en/docs/debugging_log.html
Anyway connection errors should be reported as errors and the debug output should not be necessary, but here the auth_ldap module behaves inconsistently.

yes. your right, nginx was without debug
here is debug: http://pastebin.com/ddQfvibT

I made it!
Everything works with 3268 or 3269 port!
So my last working config: http://pastebin.com/fhaUcRdZ

I submitted a patch for issue #80 if you wish to have proper error messages in the logs.  Someone may want to close this issue now that the problem has gone away.

follow this issue: #204
quote:

Port 389. This port is used for requesting information from the local domain controller. LDAP requests sent to port 389 can be used to search for objects only within the global catalog’s home domain. However, the requesting application can obtain all of the attributes for those objects. For example, a request to port 389 could be used to obtain a user’s department.
Port 3268. This port is used for queries specifically targeted for the global catalog. LDAP requests sent to port 3268 can be used to search for objects in the entire forest. However, only the attributes marked for replication to the global catalog can be returned. For example, a user’s department could not be returned using port 3268 since this attribute is not replicated to the global catalog.
Use 3268 port.

I ran into the same issue today with below mentioned nginx and nginx-auth-ldap version. But in my case I was not able to authenticate using LDAPS. I have to change my port from 636 to 3269 to make it work.

NGINX_VERSION: 1.19.6
NGINX_AUTH_LDAP_COMMIT: 83c059b73566c2ee9cbda920d91b66657cf120b7

ldap_server ldap_mycompany {
url "ldaps://abc.example.com:3269/dc=example,dc=com?samaccountname?sub?(objectClass=user)";
binddn<EMAIL_ADDRESS>binddn_passwd "XXXXXXXXX";
group_attribute member;
group_attribute_is_dn on;
require valid_user;
satisfy all;
}

follow this issue: #204 quote:

Port 389. This port is used for requesting information from the local domain controller. LDAP requests sent to port 389 can be used to search for objects only within the global catalog’s home domain. However, the requesting application can obtain all of the attributes for those objects. For example, a request to port 389 could be used to obtain a user’s department.
Port 3268. This port is used for queries specifically targeted for the global catalog. LDAP requests sent to port 3268 can be used to search for objects in the entire forest. However, only the attributes marked for replication to the global catalog can be returned. For example, a user’s department could not be returned using port 3268 since this attribute is not replicated to the global catalog.
Use 3268 port.

Hell, really works. Thanks so much.

I made it! Everything works with 3268 or 3269 port! So my last working config: http://pastebin.com/fhaUcRdZ

It works. Thanks.


connectivity: LRP with per packet LB

This PR introduces LRP connectivity tests with per packet LB enabled.
Ref: https://github.com/cilium/cilium/pull/33721
Add local redirect policy connectivity tests with per packet LB.

@aditighag @ysksuzuki

https://github.com/cilium/cilium/pull/34954 has added the LRP e2e test for per-packet LB


Pulse switch interval less than 1 second?

For first a big thank you for these great firmwares/softwares!
My first approach on these devices (CC2530) it was only some days ago and I'm studing on it very much.
When we select as output a "pulse switch" the inteval is only ever 1 second, it would be like to set also a scale of 100ms.
I need a relay that turn on with a push button (in Home assistant) and turn off itself after 300ms.
Is it possible?
Thank you very much

@Giro-S Sorry, but currently, you cannot use shorten intervals.

Thank you! I hope in the next future.
Best regards


Model: Task result

Replace the model in (5) with a call to the model from (6), when the final decision about the UI architecture is made.
The task should be relatively trivial, e.g. I assume we replace a call:
var taskResult = $.getJSON("./src/.../.../task-result-example.JSON");
...with another function to the model component.
Use Apiary as a target where to send the requests to.

Hi, as part of this (the history) might not be essential (if I understand it correctly) for the 1st iteration (the more so for the presentation on June 20th), please consider spltting this and postponing the history part of this issue, unless you have some valid reasons to not do so (for example the architecture you designed strongly and with good reason couples the history with the result provision).

Done.


fix: allow use limit as string

What kind of change does this PR introduce?

bugfix
Did you add tests for your changes?
not required
If relevant, did you update the README?
Summary
Ref: https://github.com/webpack-contrib/url-loader/issues/95
Does this PR introduce a breaking change?

no
Other information
not required

FWIW it looks like loader-utils does not parse numbers:
?xyz=1 -> { xyz: "1" } // numbers are NOT parsed

https://github.com/webpack/loader-utils#parsequery

@damonbauer we use https://github.com/webpack-contrib/url-loader/blob/master/index.js#L20 if it is string


Proposal: Online Chess game with bets

Description

The project is a 2-player online chess game.
Players connect to Kaia via Metamask wallet.
Before playing, each player bets 1 Klay.
2 players play chess with each other. Whoever loses will lose 1 Klay. Whoever wins will get 1 Klay.

Bounty track

Other

Is the chess engine onchain?
if not, I would prefer that you work with Chess.com API (https://www.chess.com/news/view/published-data-api) to build an Oracle service
Players would play chess on there and make bets. Whoever wins gets the pot. At the same time, other viewers can bet as well.

Amended the title

The game you sent is really good and professional and the algorithm is very complicated.
The game I created is just a 2-player version.
In which 2 people connect to the same room. The first person bets 1 Klay. Before the second person enters the room, if the first person decides to leave, they will receive 1 Klay.
When the second player enters the room, both of them have already bet 1 Klay.
Then the 2 people play together, the winner will receive 2 Klay (1 Klay bet and 1 Klay win).
The game has 3D graphics, not too complicated (because the team has only me, coding the game and coding the blockchain).
Thanks.

I agree. That's the reason why I recommend you to use Chess.com API. You would be primarily building the betting part rather than building the chess game.
Basically, you run a oracle service that watches the chess.com matches. Say 2 people entering chess.com, they can enter chess.com, to start a match and then make a bet on your DApp
This way, the best part is the betting will be completed and polished to great extent first (this is the part that makes you money). Then you can work on the chess game after.


Include dot files in output generation

This change will include dotfiles (such as .htaccess) in the generated output, resolving issue #121. It continues to ignore VCS-related files (such as .git).

I just ran into this... can this feature be released please?


Closes #473 assert_length: add length argument to assert_numeric_vector()

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[x] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md under the header # admiral (development version) if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

Do you think we should add the same argument to all assert_*_vector() functions for completeness? I know it's less useful there but still it would make our API more consistent.

I wouldn't add the length argument to other assertions until we need it. I think admiraldev should support the admiral packages but is not meant to be a general checking tool.

Do you think we should add the same argument to all assert_*_vector() functions for completeness? I know it's less useful there but still it would make our API more consistent.

I wouldn't add the length argument to other assertions until we need it. I think admiraldev should support the admiral packages but is not meant to be a general checking tool.

Ok, happy to go with your call


Add Feature for better UX

Hey! This is more of a feature for better UX than an issue.
I would love to see, if you could add icons for folders based on older name

Like Javascript gets JS logo,
Python gets its logo, stuff like that.

Unfortunately I am not a swift developer. I develop for web and have no idea how swift works, but I hope you could do something like that, like we see in VS Code Material Icon Theme
https://codespace.app/?ref=producthunt
The above link redirects to an app similar to yours which has this feature. But, the problem is, it's paid. And, I really don't want to spend money on this.
Love that you guys, are not doing that, that's why I would love this to happen

Hi,
Sure, I'll see what I can do 👌

Hi @kavin25,
I've integrated some atom file icons and I'm currently working on a folder icon selection but it will not be included in v1.3.0.

Nice. How do I get to use this?

Install the app update available on the AppStore (link in README)

Ok. Thanks for the update

Looks pretty good now. Thanks a lot


[WIP] Fix model loading collate_fn non-static

After fixing typing errors #1944  I believe there as a bug introduced which prevents trained checkpoints to be loaded.
Similar issue here:
https://github.com/pydantic/pydantic/issues/1596
Can anyone please confirm that loading of trained checkpoints is broken and this PR fixes it?

Hmm, Im able to load checkpoints.
Train for 2k iters
ns-train nerfacto --data data/nerfstudio/test/
Load checkpoint
ns-train nerfacto --data data/nerfstudio/test/ --load-dir outputs/test/nerfacto/2023-05-24_104341/nerfstudio_models/step-000002000.ckpt

I believe the error happens when you load pre 0.3.0 checkpoint in >=0.3.0 nerfstudio.
I will verify it.

Yes, you can reproduce the error if you remove the collate_fn field manually from config.json:

What I suggested here is also not good. I don't think collate_fn gets registered as a field now.

I guess the problem is in the yaml loader. Do you think it is worth fixing?