Possibility to choose the toggle event

Hi !
I was wondering if you could maybe add the possibility to set toggle to true on mouseover ?
Thank you.

Hi @The-BlackWidow ,
I've just created a new branch adding this feature. Since I'm still not 100% sure that this a good practice to modify the event listeners after the component's DOM is compiled I haven't merged to master yet..
Anyway, you can try it out, it's working fine :)
Type in console:
npm install https://github.com/PygmySlowLoris/vue-fab.git#feature/dynamic-events --save

Then you just have to add a new available prop called main-event-toggle and set it with the whatever event you want. mouseover
<fab
   ...
   main-event-toggle="mouseover"
></fab>

Let me know if that worked for you.

I'm keeping this ticket opened to see if there are more people that would like this feature in a future release.. If so, we'll merge it :)
👍 Thumbs up?

I found a workaround but I will gladly test this branch to give you feedback =) !

Cool!
Awaiting for your feedback :)

@Edujugon

Module build failed: Error: Couldn't find preset "latest" relative to directory "C:\\Code\\reaweb\\node_modules\\vue-fab"
    at C:\Code\reaweb\node_modules\babel-core\lib\transformation\file\options\option-manager.js:293:19
    at Array.map (<anonymous>)
    at OptionManager.resolvePresets (C:\Code\reaweb\node_modules\babel-core\lib\transformation\file\options\option-manager.js:275:20)
    at OptionManager.mergePresets (C:\Code\reaweb\node_modules\babel-core\lib\transformation\file\options\option-manager.js:264:10)
    at OptionManager.mergeOptions (C:\Code\reaweb\node_modules\babel-core\lib\transformation\file\options\option-manager.js:249:14)
    at OptionManager.init (C:\Code\reaweb\node_modules\babel-core\lib\transformation\file\options\option-manager.js:368:12)
    at File.initOptions (C:\Code\reaweb\node_modules\babel-core\lib\transformation\file\index.js:212:65)
    at new File (C:\Code\reaweb\node_modules\babel-core\lib\transformation\file\index.js:135:24)
    at Pipeline.transform (C:\Code\reaweb\node_modules\babel-core\lib\transformation\pipeline.js:46:16)
    at transpile (C:\Code\reaweb\node_modules\babel-loader\lib\index.js:50:20)
    at C:\Code\reaweb\node_modules\babel-loader\lib\fs-cache.js:118:18
    at ReadFileContext.callback (C:\Code\reaweb\node_modules\babel-loader\lib\fs-cache.js:31:21)
    at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:420:13)

 @ ./node_modules/vue-fab/src/FAB.vue 9:21-395
 @ ./node_modules/vue-fab/src/index.js
 @ ./resources/assets/js/main.js
 @ multi ./resources/assets/js/main.js ./resources/assets/sass/app.scss

This is the error I get when compiling with Laravel mix. I am not able to find what is creating this error.

Hi @DrowningElysium ,
Are you trying the above mentioned branch? or the latest release?
Try to run this in the terminal:
npm install --save-dev babel-preset-latest
And then in your .babelrc (root directory) add this:
{
    "presets": [
        ["latest", { "modules": false }]
    ]
}

let me know if that worked..

I used the branch mentioned above, but as I am using Laravel mix which should do all these things for me. I am not sure how to manage this.

I will try to update the package to work with babel-preset-env instead of the "latest" one.. But since I can't provide a ETA, just try to do the following steps:
Run in you terminal:
npm install --save-dev babel-preset-latest
Create a .babelrc file in your project directory. and add this:
{
    "presets": [
        ["latest", { "modules": false }]
    ]
}

Then try again.

One better behaviour could be to open it on mouse over, but require a click to close it


x/tools/imports: imports.Process chooses wrong import paths when v2 module is used

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

go version go1.16 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/ilya/Library/Caches/go-build"
GOENV="/Users/ilya/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/ilya/go/pkg/mod"
GONOPROXY="gitlab.mstar.site"
GONOSUMDB="gitlab.mstar.site"
GOOS="darwin"
GOPATH="/Users/ilya/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.16/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.16/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/ilya/Development/importsbuglib/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/vs/hy8s9kz12dngnkz0gf49k0040000gp/T/go-build4119139561=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Run imports.Process("filename.go", filedata, nil) with file filename.go containing:
package libuserv2

import "github.com/IlyaFloppy/importsbuglib/v2"

func SomeFunc() importsbuglib.SomeType {
	typev2 := importsbuglib.SomeType{
		Value:        "v2",
		AnotherValue: "v2",
	}

	return typev2
}

and filedata containing:
package libuserv2

func SomeFunc() importsbuglib.SomeType {
	typev2 := importsbuglib.SomeType{
		Value:        "v2",
		AnotherValue: "v2",
	}

	return typev2
}

(same but no import)
What did you expect to see?
Original file with original import:
package libuserv2

import "github.com/IlyaFloppy/importsbuglib/v2"

func SomeFunc() importsbuglib.SomeType {
	typev2 := importsbuglib.SomeType{
		Value:        "v2",
		AnotherValue: "v2",
	}

	return typev2
}

What did you see instead?
Import without /v2 part:
package libuserv2

import "github.com/IlyaFloppy/importsbuglib"

func SomeFunc() importsbuglib.SomeType {
        typev2 := importsbuglib.SomeType{
                Value:        "v2",
                AnotherValue: "v2",
        }

        return typev2
}

Aliased imports like import alias "github.com/IlyaFloppy/importsbuglib" are not present after using imports.Process.
Full example is available at https://github.com/IlyaFloppy/importsbug which uses https://github.com/IlyaFloppy/importsbuglib

Sorry, maybe I'm being dense, but I don't understand. Once you remove the v2 import, how is goimports supposed to know to use v2 rather than v1? And once you remove the named import, how is it supposed to guess that importsbuglibalias refers to importsbuglib?

It is stated that filename's directory influences which imports can be chosen. Adding another file with v2 import makes Process substitute correct v2 import (added this case to the linked example). So I guess I was expecting Process to read imports from all files in the filename's directory including original filename.

No, sibling imports are only loaded from other files. That seems right to me; if I remove an import from a file, then process it in-memory before saving it, the fact that there's an old version of it on disk shouldn't affect results.


Correct placement flow to expose service only when deployment is ready

On the end of the loop we had
	// Only expose the service is the deployment succeeded 	if !instance.Status.Conditions.IsTrue(condition.DeploymentReadyCondition) { 		Log.Info("Waiting for the Deployment to become Ready before exposing the sevice in Keystone") 		return ctrl.Result{}, nil 	}
which doesn't make any sense and mess with correct status order of conditions. So ensureServiceExposed should be moved to correct place and check should be readded

Please be aware that as soon as you add the waiting logic back the unstable test PlacementAPI controller when The  service is fully running should move the finalizer to a new MariaDBAccount when create is complete will show up. The reason of that test failure is that the mariadb account rotation test case does not make the deployment ready after the rotation. But the rotation changes the config hash on the deployment and that will bump the generation of the Deployment. So the test needs an additional call to SimulateDeploymentReplicaReady after the deployment is updated by the controller. Right now it is not easy to add as the UpdateAccount callback runs before the new account is made ready by the mariadb-operator/api code and therefore runs before the deployment is updated.

https://github.com/openstack-k8s-operators/mariadb-operator/blob/9de2d1f1915e965d48534e183e3d55e5e789d610/api/test/helpers/harnesses.go#L286-L287
so we would need the SimulateDeploymentReplicaReady call after 			harness.mariaDBHelper.SimulateMariaDBAccountCompleted(newAccountName) (and after we ensured that the deployment is updated with the new hash) but we don't have a callback for it.


README info

It would be helpful if Sublime command (for creating key binding) appeared in README and on packagecontrol.io in description.

That's a good suggestion, I'll try to get it added tonight unless you want to take a stab at a PR to add it to the README file? packagecontrol.io just reads the README file so it should be the only place to update it.


Unknown device ESP10_SOCKET_01, skipping...

When trying to use my smart plugs (ESP10), I can't get it to work. I tried modifying the WizSocket/index.ts to include the model name but it still doesn't work. I'm down to help troubleshoot and get it working, whatever you need from my end. Just would love to have these plugs working in HomeKit.

If you look at the logs do they say anything about an unrecognized model number?


⚠ Jenkins Plugins Artifactory has degraded performance

In 68f20be, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1238 ms

Resolved: Jenkins Plugins Artifactory performance has improved in fa58476 after 12 minutes.


Format error with media query

@media only screen and (min-width: 0px) and (max-width: 679px){}

If stylus code have the above codes, it will be format error: Found a non-string argument of

Investigating... Please bear with me.

This has been fixed at version 0.3.4. Please update the extension.
I've made sure that Stylus Supremacy generates @media queries the same way as the original Stylus parser.
Thank you for reporting!

Very thanks.


changed twitter logo to x logo in all html files

Description
Replaced the all old twitter logos with the new X logos
Related Issues
#31
Type of PR
[X] (Bug)
Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have performed a self-review of my code
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

@Vd7905 please fill the pR template with issue number and preview video properly.

@Vd7905 please fill the pR template with issue number and preview video properly.

mam in my hactoberfest account PR/MR: Anjaliavv51/Retro#76 is showing waiting.Will it automatically gets accepted or you will do it?


IIS & UrlScan Issues

IIS and I believe Azure use UrlScan by default.
The default settings break this plugins catch all routes when using File Based routing due to the file name generated containing multiple dots eg. ...all-fc33d227.js
IIS Log

2024-02-22 12:08:00 ::1 GET /Rejected-By-UrlScan ~/assets/...all-fc33d227.js 8087 - ::1 Mozilla/5.0+(Windows+NT+10.0;+Win64;+x64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/<IP_ADDRESS>+Safari/537.36+Edg/<IP_ADDRESS> - 200 0 0 113

UrlScan
Below is from the UrlScan docs on the rule.

Rule
Description

AllowDotInPath
Allowed values are 0 or 1. The default value for AllowDotInPath is 1. If set to 1, UrlScan will allow requests that contain multiple instances of the dot (.) character in the URL. If set to 0, UrlScan will reject requests that contain multiple instances of the dot (.) character in the URL. Note: Because UrlScan operates at a level where IIS has not yet parsed the URL, it is not possible to determine in all cases whether a dot character denotes the extension or whether it is a part of the directory path or filename of the URL. For the purposes of extension analysis, UrlScan will always assume that an extension is the part of the URL beginning after the last dot in the string and ending at the first question mark or slash character after the dot or the end of the string. Setting AllowDotInPath to 0 defends against the case where an attacker uses path info to hide the true extension of the request (for example, something like "/path/TruePart.asp/FalsePart.htm"). Setting AllowDotInPath to 0 also causes UrlScan to deny any request that contains a dot in a directory or file name.

Note: Despite the above saying the default value here is 1, its actually 0, as confirmed by downloading the default template here

Its possible to create some workarounds for this in IIS by reinstalling another instance of the UrlScan dll and creating a new .ini file. But some of that is out of reach and complex for devs in userland.
I wonder it makes sense to create a workaround and document for this using extendRoutes() or similar ?

Its possible to create some workarounds for this in IIS by reinstalling another instance of the UrlScan dll and creating a new .ini file. But some of that is out of reach and complex for devs in userland.

I don't know what you are talking about but it doesn't sound related to this project
This sounds like something worth putting in documentation. Better yet: fixed in vite to replace the dot in filenames? It looks like so many other things could break this


Disable babel

Hi, I can compile to pure ES5 with typescript with source-maps and I was wondering how can I disable  the babel transpilation in storybook? In regular webpack you simply ignore loader, but here loader is already defined. Thanks.

Check this: https://getstorybook.io/docs/react-storybook/configurations/custom-webpack-config#full-control-mode

If you do this, you'll loose some other features based on react-docgen.


ENH: allow approx_fprime to work with vector-valued func

Closes #2993 and closes #6992.
This would allow approx_fprime to work with vector-valued functions.

All the PR does is remove the check on func returning a scalar. The differentiation is already done by _numdiff.approx_derivative.
check_grad is amended slightly to work for vector valued functions, but not along random projections.
The docstrings are amended.

What's the best way of adding a changelog to the docstring to let people know of the updated behaviour?

What's the best way of adding a changelog to the docstring to let people know of the updated behaviour?

Not sure about the best way; one way is to use the .. versionchanged:: directive maybe.

Actually there is a directive for that: .. versionchanged:: version

I used the versionchanged RST directive, I think it looks ok in the doc artefact. I fixed the failing test (needed a conversion to an array). I think the current test fails are unrelated.

@tupui, is this mergeable as is, it would close a couple of issues.


T08.03-Solution-FinishQuizExample is not running due to java.lang.NullPointerException:

The solution is not running and the app has stopped.
I think the cursor is null from some reason.

 Process: com.udacity.example.quizexample, PID: 14289
                                                   java.lang.NullPointerException: Attempt to invoke interface method 'int android.database.Cursor.getColumnIndex(java.lang.String)' on a null object reference
                                                       at com.udacity.example.quizexample.MainActivity$WordFetchTask.onPostExecute(MainActivity.java:158)
                                                       at com.udacity.example.quizexample.MainActivity$WordFetchTask.onPostExecute(MainActivity.java:136)
                                                       at android.os.AsyncTask.finish(AsyncTask.java:660)
                                                       at android.os.AsyncTask.-wrap1(AsyncTask.java)
                                                       at

Any one knows what is the problem and how to solve it?

Make sure u have downloaded [https://github.com/udacity/DroidTermsExample-APK/raw/master/droidtermsexample-release.apk] and installed it in the emulator(via drag drop or adb install filename from system tools dir in android install dir)
Do this before installing the quiz app. :)

For me the app worked on an actual device rather than the emulator. You need to have the  droidtermsexample-release.apk installed on the physical device before running your application.
And I still need to figure out why the app does not work for me on the emulator.

Closing this issue as it is not related to the repo's code.


Fix build badge

Changes

Fixes build badge - see here  for more information
Fixes build badge link - now takes you to the build.yml workflow results as intended
Convert HTML to markdown
Fix markdown formatting (linting errors from vscode extension)

Thanks!


Formalize slop as an interaction

It's currently an operator and requires use of runtime.connect. Ideally it would be an interaction with the relevant configurable properties, e.g. slop size.
Challenge here is building an interaction that supports n-dimensional slopping, e.g. vertical/horizontal slop or slop outside of a 2d region.

This is now available in https://github.com/material-motion/material-motion-swift/blob/develop/src/interactions/SlopRegion.swift


NNBD. No error on CFE when function with non-nullable return type may complete normally

According to the NNBD specification

It is an error if the body of a method, function, getter, or function expression with a potentially non-nullable return type may complete normally.

The following code shows a case where this statement doesn't work on CFE
// oped_out_lib.dart
// @dart=2.6
typedef void LegacyFoo();

// opted in
import "opted_out_lib.dart";

class C {
  static LegacyFoo sTest() {}
//                 ^^^^^
// [analyzer] unspecified
// [cfe] unspecified

  LegacyFoo mTest() {}
//          ^^^^^
// [analyzer] unspecified
// [cfe] unspecified

  LegacyFoo get gTest {}
//              ^^^^^
// [analyzer] unspecified
// [cfe] unspecified
}

LegacyFoo test() {}
//        ^^^^
// [analyzer] unspecified
// [cfe] unspecified

main() {
  C.sTest();
  C c = new C();
  c.mTest();
  c.gTest;
  test();
}

The code above produces all expected errors in analyzer but none of them in VM
Tesed on Dart VM version: 2.8.0-dev.20.0 (dev) (Fri Apr 3 10:19:55 2020 +0200) on "windows_x64"
Similar  issues #41496, #41498

I believe the expected errors will be emitted when #41505 is implemented.

A CL is in progress: https://dart-review.googlesource.com/c/sdk/+/143811


not able to create subnet using resource declaration

HI,
I am not able to create subnet in aws. following is my init.pp where i have declared the common variables and subnet creation steps.
init.pp:
define subnet_creation(
  $customer = 'devops',
  $company = 'test',
  $region = 'us-east-1',
  $availability_zone = 'us-east-1a',
  $index = '1',
  $priv_api_category = 'apiservers',
  $priv_db_category = 'dbservers',
  $pub_category = 'public',
  $priv_category = 'private',
  $redundant = 'yes',
  $redundant_availabilty_zone = 'us-east-1c',
  $ensure = 'present',
) {

    if $region == "us-east-1" {
        $region_name = 'nvirginia'
    }
    elsif $region == "us-west-1" {
        $region_name = 'ncalifornia'
    }
    elsif $region == "us-west-2" {
        $region_name = 'oregon'
    }
    elsif $region == "eu-west-1" {
        $region_name = 'ireland'
    }
    elsif $region == "eu-central-1" {
        $region_name = 'frankfurt'
    }
    elsif $region == "ap-southeast-1" {
        $region_name = 'singapore'
    }
    elsif $region == "ap-southeast-2" {
        $region_name = 'sydney'
    }
    elsif $region == "ap-northeast-1" {
        $region_name = 'tokyo'
    }
    elsif $region == "sa-east-1" {
        $region_name= 'saopaulo'
    }
    else {
          warning('Invalid region specified in definition.')
    }

ec2_vpc_subnet { "${customer}-${company}-${priv_category}-${priv_db_category}-${region_name}-${index}":
  ensure            => $ensure,
  region            => $region,
  vpc               => $customer-$company-$region_name,
  cidr_block        => '<IP_ADDRESS>/24',
  availability_zone => $availability_zone,
  tags              => {
                          name => "${customer}-${company}-${priv_category}-${priv_db_category}-${region_name}-${index}",
                        }
                }
  }

when i applied this manifest, its not printing any error/message.
[ec2-user@xxxxxxxxx manifests]$ puppet apply init.pp --test
2015-10-19 13:08:50.022361 WARN  puppetlabs.facter - locale environment variables were bad; continuing with LANG=C
Notice: Compiled catalog for xxxxxxxxxxxx in environment production in 0.58 seconds
Info: Applying configuration version '1445260131'
Notice: Applied catalog in 0.02 seconds
upon checking in AWS console, db subnet was not created. i already exported AWS keys information.
Could you assist on this.
Thanks,
Ram.

I'm presuming this is the same issue as #243. I'll close at least until we resolve that one.

Hi Gare,
Dont close this issue #242. still i am not able t create subnets using declaration. i pasted the init.pp, could you please assist on this.

If you could provide the output from the command run with the --debug flag.
Looking at the code, it looks like you have a defined type but you haven't realised it? That would explain why you're not seeing the subnet created. eg. you never actually call subnet_creation.

Hi Gare,
Could you share me a snippet of this requirement by testing in your setup.

@ram2valar you can see a working example of using a defined type in this example https://github.com/puppetlabs/puppetlabs-aws/tree/master/examples/create-your-own-abstractions
Note the init.pp manifest which uses the defined type.


Allow Rounds To Be Played

@devgianlu,
This ticket contains the "vital" stuff that needs done in order to have a proper game (for the most part).

[x] #44 takes priority first. My reasoning here is that the transition to WebSockets will require some changes, possibly ones that will be made as needed. In order to get the other stuff below fixed, we need to make sure that sessions work smoothly. Getting rid of long-poll and full-duplexing all comms will prevent the current fuckery that causes the current game to break (times out because, well, long-poll).
[ ] Make the white cards selectable

[ ] Make it so they can be submitted to the Czar, either via a button or modal (I'd say a button beside the arrow on the "Hand cards" bar, like a checkmark icon or something).

[ ] Implement interface for the judge to receive selected cards
[ ] Implement way to select cards for submission
[ ] Select a user to be Card Czar/judge
[ ] Update the Scoreboard in the menu to reflect which user is Czar each round
[x] #46
[x] #48

Cache issue


setting transaction isloation level for ms sql server

Hi
I am connecting to ms sql server database using "autocommit = false" option to run queries in transaction.
Is there a way to set transaction isolation level for ms sql server ?
thanks

Hi, @StudentForever, it seems to work fine if I just execute SET TRANSACTION ISOLATION LEVEL READ UNCOMMITTED on the connection object:
>>> import pyodbc
>>> conn = pyodbc.connect("driver={sql server native client 11.0};server=sql.server;database=database;uid=username;pwd=password")
>>> conn.execute("set transaction isolation level read uncommitted;")
<pyodbc.Cursor object at 0x000002CA8AB849F0>
>>> cur = conn.cursor()
>>> r = cur.execute("select transaction_isolation_level from sys.dm_exec_sessions where session_id = @@SPID;").fetchone()
>>> r
(1, )
>>>

Microsoft's article here says that 1 = ReadUncommitted.

thanks

thank you


Investigate "tasks have lost their waker" and fix if needed

Motivation
tokio-console, for which we added support in https://github.com/ZcashFoundation/zebra/pull/4519, is reporting a bunch os "tasks have lost their waker". We should investigate.
Most of them seem to be from the handshake code:
https://github.com/ZcashFoundation/zebra/blob/ca0520b2e8e7c40fa4c24da45896ff9430992a9d/zebra-network/src/peer_set/initialize.rs#L298-L299
https://github.com/ZcashFoundation/zebra/blob/766dd9357b780e49d95398c85259ac1ff4290fa5/zebra-network/src/peer_set/initialize.rs#L757-L758
https://github.com/ZcashFoundation/zebra/blob/766dd9357b780e49d95398c85259ac1ff4290fa5/zebra-network/src/peer/handshake.rs#L1016-L1020

Specifications

Designs

[ ] Investigate the cause of the issue
[ ] If it's an issue, fix it if possible. If it's too complex, create another issue for it.
[ ] If it's a false positive, consider reporting it to tokio-console

Related Work

I've had a look at all the lost wakers, and they seem pretty harmless.
If we're just dropping wakers for handshakes we're not using any more, that's probably a task cleanup bug. And we make new handshakes as needed anyway.

The warning might also be incorrect or confusing, I'm still seeing the poll number for those tasks going up, even after the lost waker warning.
But handshakes should only last for 4 seconds, so there's definitely a bug here.

This appears to be a known bug in tokio-console, related to tasks spawning other tasks:
https://github.com/tokio-rs/console/issues/345


stubgen can't generate stub file for libcarla

CARLA version: 0.9.12
Platform/OS: Ubuntu 20.04 LTS
Problem you have experienced:
I use vscode for carla python client development. VSCode use pylance as the language server. It requires a stub file for those python library which symbol coming from platform native libray.
For opencv, I can use
stubgen -m cv2

to generate the stub file. All types and symbols can be seen in generated file: cv2.pyi .
But for carla
command:
stubgen -m carla

generated carla/init.pyi .This file only contains one line: from .libcarla import *
No any valid symbol generated.
I can't find any way to solve this issue.

Hi @julee
I'm also looking forward to get stub files, because I would like to have autocomplete working in VSCode.
Could somebody comment on that issue how to get autocomplete working within VSCode (pylance) for Carla PYTHON package?
I think the stub file is required for that...


Bootstrap 5 set selected option value

Package version: 1.14.0
Django version: 4.0.2
Python version: 3.10.2
Template pack: (Optional) crispy-bootstrap5==0.6

Description:
I think by default, bootstrap selected option doesn't have value attribute. Is there anyway we can add it via this?
Or I have to create JavaScript function to add that?
Preferably also include:

[ x ] Example Django Crispy Forms code
[  ] Screenshots
[ x ] Actual HTML generated
[ x ] Expected HTML

Django code
class RandomChoices(models.TextChoices):
    DEFAULT = "", _("---")
    A = "A", _("A")
    B = "B", _("B")

class RandomForm(forms.Form):
    random = forms.ChoiceField(
        choices=RandomChoices.choices,
        required=False,
    )

Template

Random choice

    {{ random|add_class:"form-select form-select-sm mb-2" }}

Generated HTML
<select class="form-select form-select-sm mb-2" id="id_random" name="random">
    <option selected="">---</option>
    <option value="A">A</option>
    <option value="B">B</option>
</select>

Expected HTML
<select class="form-select form-select-sm mb-2" id="id_random" name="random">
    <option value="" selected>---</option>
    <option value="A">A</option>
    <option value="B">B</option>
</select>

Hi @shawnngtq 👋
Your template looks like django-widget-tweaks style rather than crispy-forms.
If you were using the crispy-filter (say) I'd have expected in the template to see {{ random|crispy }} rather than {{ random|add_class...}}. 🤔
Amending the existing test to have to have a default option like your form I get:
<form method="post">

    Select input
      <select class="form-select select" id="id_select_input" name="select_input">
        <option value="1">Option one</option>
        <option value="2">Option two</option>
        <option value="3">Option three</option>
        <option selected value="">Option 4</option>
      </select>

</form>

Hi @smithdc1
Yes, you are right, I am using both this and django-widget-tweaks, and I kinda mix them up. So this is considered off-topic, sorry about that.
I am going to try the crispy way to see if I can get expected output. Will update here again.

https://github.com/django-crispy-forms/crispy-bootstrap5/commit/eccd51491d6bfa9399349901247614ba44aed27e
Here's the example I mocked up. Tests's don't pass on everything as I broke unrelated tests with this change. Shows the scenario you ave though with the html being generated as per above.

Ahh. You are using TextChoices where as I used tuples.
What happens if you render your form with "just" Django?

I tried with list of tuples and tuple of tuples for the choices
random = forms.ChoiceField(
    # choices=RandomChoices.choices,
    choices=[("", _("---")), ("A", _("A")), ("B", _("B"))],
    required=False,
)

random = forms.ChoiceField(
    # choices=RandomChoices.choices,
    choices=(("", _("---")), ("A", _("A")), ("B", _("B"))),
    required=False,
)

The output is same as when I use TextChoices. I am quite sure it's not the reason.
<select class="select form-select" id="id_random" name="random">
  <option selected="">---</option>
  <option value="A">A</option>
  <option value="B">B</option>
</select>

I noticed that the empty string value is what matters. When I replace "" or None with something else, the selected attribute will be replaced by value.
class RandomChoices(models.TextChoices):
    DEFAULT = "Default", _("---")
    A = "A", _("A")
    B = "B", _("B")

random = forms.ChoiceField(
    choices=RandomChoices.choices,
    required=False,
)

<select class="select form-select" id="id_random" name="random">
  <option value="Default">---</option>
  <option value="A">A</option>
  <option value="B">B</option>
</select>

The thing is that django CharField prefer "" over None, that's why I kept it "" for DEFAULT

Here are some tests I did.
Vanilla django, it works fine: https://forum.djangoproject.com/t/django-choicefield-empty-string-value/12458/3
What is interesting here is that so long I add crispy_forms and crispy_bootstrap5 under INSTALLED_APPS, even if I don't use any crispy method (not even {% load crispy_forms_tags %}), it cause the selected=""

I found out the reason, it's not due to this library, but due to https://github.com/adamchainz/django-minify-html.


modify_bert

Which part of the bert model is modified

Make bertmodel output the original (unpooled) attention scores.

Why do some relations map to numbers?

Struggled to find a proper description so using [unused*] to represent.
You remind me. The "17" is not a causal number but the number of relations that "unable to find a proper description".


14: Rework readme

This updates the readme to accommodate the changes suggested by @celinedurniak.
Fixes #14

The new instructions to use sasdata with a virtual environment or conda are good to be merged.


Pollyfill is occuring without calling polyfill()

You say in the doc you need to do
require('es6-promise').polyfill();

in order to automatically polyfill.
However when I do
var a = require('es6-promise').Promise
console.log(a === Promise);

prints true. The browsers native Promise is getting replaced.

I think the reason is this line as verified on Chrome 48

Thats is why it fails that check and overrides the native Promise.

https://github.com/stefanpenner/es6-promise/issues/148 explain the failure in Chrome.

4.0 will include two builds, one that auto-polyfils if the host doesn't work, and one that must be explicitly requested to polyfil.


Provide a way for non-privileged users to !!/scan

Right now, !!/scan is an alias for !!/report, which is (rightfully) a privileged command.
However, this has the drawback that non-privileged users cannot !!/scan anymore, like they did before.
If you ask for my interpretation of the word 'scan', it's way closer to 'test' than to 'report'. So my preference would be removing the alias entirely, but I'm open to a solution where !!/scan only scans for non-privileged users but also reports (if applicable) for privileged users. Though that might be too confusing.
Context: here

I'll take this work. Hope it's not hard to do it.


Version 0.5.1 not working on node 4

Version 0.5.1 in dependencies has  "promish": ">=5.0.2". As promish was deployed to npm yesterday with 5.0.2 version now our build is no longer working. Following error while starting app
/Users/nitesh.malhotra/workspace/paytm-seats/node_modules/exceljs/node_modules/promish/lib/promish-class.js:12
class Promish extends Base {
^^^^^
SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:373:25)
at Object.Module._extensions..js (module.js:416:10)
at Module.load (module.js:343:32)
at Function.Module._load (module.js:300:12)
at Module.require (module.js:353:17)
at require (internal/module.js:12:17)
at Object. (/Users/nitesh.malhotra/workspace/paytm-seats/node_modules/exceljs/node_modules/promish/lib/promish.js:4:18)
at Module._compile (module.js:409:26)
at Object.Module._extensions..js (module.js:416:10)

I have the same issue.
@guyonroche could you fix it please?

Same issue here on Node 4.5.

Node 4.x LTS is EOL from a maintenance stand point at the end of the month. Probably best that you try to upgrade your environments.

Facing the same issue. Is there any way to fix this ?

I agree - the dist build is a bit of a mess. I will try to take some time to look into it...
In the mean time, the following workaround should at least allow the node v4 apps to run...
node --strict_mode app.js

Dropped non-native Promise implementations as of v3.0.0


Add xmax resolution ?

To gammaboard or as a specific function?
I don't think it's worth a specific function, it's a simple 1D resolution so there is already everything for it. Why not to gammaboard.

To gammaboard


Implement single draft "self" link generation

Follow what was done in invenio-records-resources

This task deals with implementing the "self" link (api). There is no landing page for a draft or rather, it's landing page is the edit endpoint, but action links are not covered by this task.

Closed via #29


cp437.py: Python 3 compatibility

Both Python 2 and Python 3 compatible code, tested with Python 2.7 and Python 3.4 in Fedora 22. It should work with Python 2.6 and 2.7 as well as with Python 3.2 and higher.

https://bugzilla.gnome.org/show_bug.cgi?id=765301

The script was obsolete and not included in the current vte package distribution, so it has been removed from the master branch.


Update nf-psapi-enumprocesses.md

Fixed typo

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


Udp protocol supported.

[x] I have searched the issues of this repository and believe that this is not a duplicate.
[x] I have checked the FAQ of this repository and believe that this is not a duplicate.

Anyone knows if Dubbo supports udp protocol.
If not supported, is it possible or easy to adapt other protocol than tcp .
Thanks.

it's not supported yet.


Pin clang-tools version to 17.0.6 in nix flake

Somehow nix clang-tools version control is off between linux and macOS, so we need to pin the clang-tools version for now.
According to the clang-tools source code, the version is determined by the parameter clang-unwrapped, which is why I do nix override clang-unwrapped instead of version.
By a quick check, using llvmPackages_18 would results in quite a large size for the nix derivation, I'm not sure the reason of this, but anyway, sticking to llvmPacakges_17 for now should be fine

@hanno-becker Mind trying out to see if this solves the clang-format issue on your M1 ?

Tested locally, LGTM


Re-route old URL to new child gallery

Re-route this link: https://www.mareinc.org/waiting-child-profiles#gallery so that if user attempts to visit that page they are routed to the new child gallery.

Re-routed via HubSpot control panel.


Card Reveal in Horizontal Card

It's possible in future, Horizontal Cards has Card Reveal effect?

Any update on this?
I'm currently using the horizontal card, and trying to implement the Card reveal.
Using the following, but can't seem to get the activator class to trigger, no matter where I place it.

{{ $post->title }}more_vert

        @foreach($post->tags as $tag)

            {{$tag->name}}

        @endforeach

{{ str_limit($post->body, $limit = 100, $end = '...') }}

        slug) }}">View Post

Post Infoclose

Created by:  {{ $post->user->name }}

Category:  {{ $post->category->name }}

Created on:  {{ $post->created_at }} ({{ $post->created_at->diffForHumans() }})

Last updated:  {{ $post->updated_at }} ({{ $post->updated_at->diffForHumans() }})

This seems to have fixed in [ c88e300]
via #3515


Cannot make it work, error in script

Hi I followed the tutorial  installing all the required components through bower and composer. And I get this error,
localhost/:8 GET http://localhost/ResumableProj/bower_components/resumablejs/resumable.js  (index):51 Uncaught ReferenceError: Resumable is not defined(anonymous function) @ (index):51.
Can you help ?

I tried changing , from this :  <script>  at line 50 of front end index.html  to
this : <script src="bower_components/resumablejs/resumable.js" type="application/javascript"> , the uncaught error goes away but the GET error remains.

Ohh I figured out the error.  In your front end file according to the tutorial , you include the scripts like this:
 <script src="bower_components/resumablejs/resumable.js" type="application/javascript"></script>
Instead do this as per the new folder change by resumable:
` <script src="bower_components/resumablejs/resumable.js" type="application/javascript"></script> `


Add support for multiple passage search

usage:
./bible "Phil 1:1-10; 2:3; Rom 3:28" --version NRSV --verse-numbers

Does this require using quotations around your arguments, or only if you use semicolon?


Relations not maintained (Prisma1 -> Prisma2)

What I Did

cat datamodel.prisma | prisma2 convert > schema.prisma
prisma2 lift save
Fixed a number of issues like Automatic related field generation would cause a naming conflict
prisma2 lift save

Then
⚠  There might be data loss when applying the migration:
  • You are about to drop the table `_ChatRoomFromTrip`, which is not empty (1 rows).
  • You are about to drop the table `_DateRangeToFlightOperatorSeason`, which is not empty (18 rows).
  • You are about to drop the table `_DateRangeToPropertySeason`, which is not empty (667 rows).
  • You are about to drop the table `_DateRangeToPropertySpecial`, which is not empty (2 rows).
  • You are about to drop the table `_FlightCharterRateToPlane`, which is not empty (2 rows).
  • You are about to drop the table `_FlightRouteDestinationAirportsOnFlightCharter`, which is not empty (2 rows).
  • You are about to drop the table `_FlightRouteToQuoteFlightRoute`, which is not empty (18 rows).
  • You are about to drop the table `_GPSCoordToPropertyAccess`, which is not empty (135 rows).
  • You are about to drop the table `_InsuranceCompanyToUserTravelInfo`, which is not empty (9 rows).
  • You are about to drop the table `_MeetAndGreetOperatorRateToQuoteArrivalDeparture`, which is not empty (4 rows).
  • You are about to drop the table `_PrimaryMeetAndGreetOperatorOnAirport`, which is not empty (1 rows).
  • You are about to drop the table `_PropertyInclusionBasicToPropertyRateGroup`, which is not empty (515 rows).
  • You are about to drop the table `_PropertyInclusionDrinkToPropertyRateGroup`, which is not empty (515 rows).
  • You are about to drop the table `_PropertyPackageRateGroupToPropertyRateGroup`, which is not empty (236 rows).
  • You are about to drop the table `_PropertyPackageRateGroupToRoomUnitCost`, which is not empty (236 rows).
  • You are about to drop the table `_PropertyPageOnProperty`, which is not empty (4 rows).
  • You are about to drop the table `_PropertyRateGroupToRoomRate`, which is not empty (1521 rows).
  • You are about to drop the table `_PropertySeasonToQuoteAccommodation`, which is not empty (55 rows).
  • You are about to drop the table `_PropertySeasonToRoomRate`, which is not empty (1550 rows).
  • You are about to drop the table `_PropertySeasonToRoomUnitCost`, which is not empty (224 rows).
  • You are about to drop the table `_PropertyToPropertyAccess`, which is not empty (129 rows).
  • You are about to drop the table `_QuoteAccommodationPaxToRoomRate`, which is not empty (60 rows).
  • You are about to drop the table `_QuoteAccommodationPaxToUser`, which is not empty (34 rows).
  • You are about to drop the table `_QuoteAccommodationToQuoteAccommodationPax`, which is not empty (30 rows).
  • You are about to drop the table `_QuoteAccommodationToRoom`, which is not empty (64 rows).
  • You are about to drop the table `_QuoteArrivalDepartureToUser`, which is not empty (15 rows).
  • You are about to drop the table `_QuoteFlightRouteToUser`, which is not empty (71 rows).
  • You are about to drop the table `_QuoteTransferToTransferRate`, which is not empty (14 rows).
  • You are about to drop the table `_QuoteTransferToUser`, which is not empty (51 rows).
  • You are about to drop the table `_TransferRateToTransferVehicle`, which is not empty (27 rows).
  • You are about to drop the table `_UserOnUserAuth`, which is not empty (10 rows).
  • You are about to drop the table `_UserOnUserBasicInfo`, which is not empty (52 rows).
  • You are about to drop the table `_UserOnUserTravelInfo`, which is not empty (11 rows).
  • You are about to alter the column `nodeId` on the `Property_exclusions` table, which still contains 1 non-null values. The data in that column will be lost.
  • You are about to alter the column `nodeId` on the `PropertyExtra_notes` table, which still contains 2 non-null values. The data in that column will be lost.
  • You are about to alter the column `nodeId` on the `Property_inclusions` table, which still contains 3 non-null values. The data in that column will be lost.
  • You are about to alter the column `nodeId` on the `PropertySpecial_notes` table, which still contains 1 non-null values. The data in that column will be lost.
Lift just created your migration<PHONE_NUMBER>5953-migration_from_v1_v2 in

• You are about to drop the table _PropertySeasonToRoomRate, which is not empty (1550 rows).
🧐

@williamluke4
I think you have removed many fields when you have fixed the naming conflicts(I checked the files that you shared with me privately). Those conflicts are unexpected. We are already considering reworking the migration flow from Prisma 1 to Prisma 2 so I think we should address this issue there.
There are also plans on making relations from implicit to explicit using the format command which may address the conflicts: https://github.com/prisma/specs/issues/267
So, in general, you will need to wait a bit before you can actually migrate.

@pantharshit00 Is there anything that I can do to help with the rework of the migration of V1 to V2.

Right now we need to define the exact migration process and we also need to make choices on whether we want to have dedicated convert tooling or Prisma 1 databases can be introspected just like any other database.
You just keeping testing out photon and report bugs to us. That helps us a ton and we want to make photon as stable as possible before the full release.

@pantharshit00 It appears that all the tables that would have been dropped were all using @relation(link: TABLE) in the V1 schema

Yes, and also looks like scalar lists tables are also dropped. That is why we won't recommend upgrading right now for larger schemas.

Ok thanks @pantharshit00, This should be added to the Prisma 1 to Prisma 2 Migration Guide

@pantharshit00 Above you mentioned that I should focus on testing Photon but I struggle to see how this is possible when I am unable to upgrade to V2. Am I missing something?

I would say try photon for a new project if you can't run the introspected project.

Thanks, we have removed the prisma2 convert command in the latest preview, please use the new introspection to work with existing databases. Here are the relevant release notes: https://github.com/prisma/prisma2/releases/tag/2.0.0-preview018


Allow interpreter-only builds

Problem
rbpf cannot run on wasm32 because of the libc dependency.
Solution
Make libc and the JIT compiler optional via a new jit crate feature.

Generally looking good, but this PR too has to wait for #348 to be merged.

@terorie Can you rebase this on the "main" branch?


change arb_triple to take 3 generators as parameters

We should change arb_triple to take 3 generators as parameters.
This would allow us to reuse it in ws_deque_test.

Fixed by #135


Change Install CLI to use Helm-style parameters

What would you like to be added:
For clarity, I'd like to see the kubectl kudo install experience with parameters to change to a Helm-style syntax.
Instead of:
kubectl kudo install kafka -p foo,bar
it would be:
kubectl kudo install kafka -p foo=bar
This would be more in-line with other tooling, and reduce confusion when supplying arrays as parameters, such as in the case of Zookeeper connection strings.

optionally what kubectl does for labels is that you can specify multiple separated by comma. We can support that as well https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/#api
kubectl get pods -l environment=production,tier=frontend

the challenge here though is that parameter can contain any characters, right? I don't feel like we can put any restrictions here. So what do you do if e.g. parameter needs to contain space? comma? 🤔 Labels in kubectl don't have that problems :-)

-p myparam="here's a param with all sorts of characters🥐" <-- is that reasonable? are strings enough, and if you need quotes they're escaped?
Maybe an alternative is a --params-file? but at that point why wouldn't you just write the instance YAML.
could have a --dry-run -o yaml command on install to generate out the YAML with defaults and then edit them.


Add support for other windows form packages

I wanted to ask, this project is great in itself but would it be possible to support MaterialSkin package?, the material design is very sleek and something I want to use for a personal project but this project doesn't seem to support this and though I'd directly ask here to see if this could be implemented.
Thanks!

We have not yet determined whether MaterialSkin will be supported, but please try the following method.

open SWD4CS in VSCode
nuget : add package(MaterialSkin2)
Add MaterialControls to SWD4CS (https://youtu.be/gF6VyEbLwcA)

Other corrections were needed.
SWD4CS-MaterialSkin.zip
There may be more.

Controls can be placed but not tested.
As the author of MaterialSkin says, it would be less hassle to move to WPF.
SWD4CS-MaterialSkin

VSCode(dotnet6.0)

Source Code
SWD4CS-MaterialSkin.zip

Controls can be placed but not tested. As the author of MaterialSkin says, it would be less hassle to move to WPF.
SWD4CS-MaterialSkin
VSCode(dotnet6.0)
Source Code SWD4CS-MaterialSkin.zip

I see that SWD4CS MaterialSkin seems more functional and complete than the source code you post in GitHub


How do I pass my own acquired CMSampleBufferRef objects?

I have my own capture pipeline and would like to pass CMSampleBufferRef from my pipeline into yours - probably into CardIOVideoFrame.  I have my own UI, so all I need is the credit card/expiry data available to me if found.  I've attempted to use the SDK approach but added CardIOVideoFrame.h to get direct access, but that doesn't work.  Great library thanks!

@benshepherd I'm not entirely clear on what you're trying to do. At a high level, could you explain which parts of our library you're trying to replace with your own code, and why?

Sure - I already have a "document capture" pipeline with several features including a UI.  My pipeline does both GPU and non-GPU-based processing.  I'd prefer to grab CMSampleBuffers from my pipeline, pass them to CardIO for credit card data extraction, and get the results back.  It looked to me as if the best way to do this would be to manually create CardIOVideoFrame objects and use your existing plumbing from there.

I guess that might work, if you can figure out where to feed in the images and get back the results...
You'll need to configure your session with the same resolution, pixel format, etc that we use. Off the top of my head, I think that all happens inside CardIOVideoStream.mm.
In general, I guess you'll be pretending to be a CardIOVideoStream...?

Yes - that was my thought.  However, when I pull in libCardIO.a and the CardIOVideoStream.h, the compiler doesn't like it.  Maybe there's some Xcode trick to make CardIOVideoStream's methods visible?  CardIOView works fine.

Ah, you're trying to use our official libCardIO.a. Yeah, most of its internal symbols are not public -- that's the result of a lot of work to help out apps that already have their own copy of the OpenCV library and were running into symbol conflicts with the OpenCV inside libCardIOa.
You'll probably need to directly pull in the code you need from this source repo, rather than getting away with the official library.
Or, hmm...
If you can clone this source repo and get the library to build, then you should be able to make the necessary symbols public by adding them to this file: https://github.com/card-io/card.io-iOS-source/blob/master/SupportFiles/CardIO_exported_symbols.txt

Great idea - thanks!  I'll give it a shot.

I added CardIOVideoFrame to the exported symbols and re-built from Xcode.  The new libCardIO.a has the symbols present (confirmed via the 'nm' command), but my Xcode project still doesn't recognize it.  I've included the headers in my project and #import "CardIOVideoFrame.h".  Strange.

@benshepherd be sure to include both _OBJC_CLASS_$_CardIOVideoFrame and _OBJC_METACLASS_$_CardIOVideoFrame as well as the specific method names.
This being a .mm file might also be complicating things, I suppose.
Since you're building from Xcode anyhow, and if you won't have any conflicts with OpenCV symbols, another approach might be simply to remove the linker settings that are hiding all non-exported symbols. If you can't figure out what and where those settings are in the project's build settings, let me know and I'll help you dig them out.

The main issue turned out to be the #if statements at the beginning of the CardIOVideoFrame.h !!  That was the only reason I couldn't see the symbols after adding them to the exports file.  My CMSampleBuffers are 1280x720, so I will need to downscale?  CardIOVideoStream defaults to 640x480, but it looks like you support other resolutions (1280x720, etc).  I have it running, but I'm getting no cardInfo back from the video frame.

The main issue turned out to be the #if statements at the beginning of the CardIOVideoFrame.h !! That was the only reason I couldn't see the symbols after adding them to the exports file.

We do have a bit of a collection of #defines, don't we? Sorry about that!

My CMSampleBuffers are 1280x720, so I will need to downscale?

Ah. Tricky...
Originally, card.io was designed and released for 640x480 resolution – because that was the only iPhone camera resolution available at the time.
A couple of years ago we did some experiments with using higher camera resolutions. As a result of those experiments, some of our pipeline became, to some extent anyhow, resolution-agnostic. However, we did not carry this resolution-agnosticity all the way through to the final stages of character detection and recognition. Doing so would require a lot more changes to our code, which makes quite a few resolution and aspect-ratio assumptions. Even more challenging, it would also require us to collect an entirely new set of higher-resolution training images with which to retrain our deep-learning character-recognition models. (We're talking thousands of training images.)
During those experiments, starting with a higher-resolution image and then downsampling it later in the pipeline proved disappointing. That result is consistent with my understanding that in-camera downsampling produces much better results than post-camera downsampling, because the former can take advantage of the complex geometries within the camera's sensor which were designed specifically for this purpose.
If you can achieve better success than we did in our higher-resolution experiments, that would be fantastic, and we'd love to incorporate any such improvements into the codebase. But it won't be a quick or easy task, I'm afraid.


chore(cairo_native): add git submodule update as a post-checkout hook

This change is

scripts/add_sumodule_hook.sh line 9 at r1 (raw file):
Previously, meship-starkware (Meshi Peled) wrote…
How does this script work? Do we have to run it, or is it part of the Build?

You run it once, and it adds the hook to your local .git folder.
Then you don't need to update the git submodule every time you checkout a commit.
(It will update them automatically with the hook.)

scripts/add_sumodule_hook.sh line 9 at r1 (raw file):
Previously, avi-starkware wrote…
You run it once, and it adds the hook to your local .git folder.
Then you don't need to update the git submodule every time you checkout a commit.
(It will update them automatically with the hook.)

I can add another script for safely removing the hook if you think that would be helpful

scripts/add_sumodule_hook.sh line 9 at r1 (raw file):
Previously, meship-starkware (Meshi Peled) wrote…
I think that would be good. Running once is also great.

Added a removal function and a help message

Actually, there might be a better solution:
git config --global submodule.recurse true

ensures that all submodules are automatically updated whenever you run commands like git checkout or git pull.


Change user window opens behind client window if 'window always on top' is enabled

fixed

I was running an old commit, so this can be closed.
I found another similar bug though (demonstration)
it can be fixed by adding 'Qt:WindowStaysOnTopHint' to the settings dialog flags,
not sure if it's the right solution though.


Can't get past "start" screen

I typed my ZIP code and pressed "Start," but it didn't work. I added the crawl and the greeting text, but it still doesn't work. I checked that my ZIP code is correct and tried again, but it's still not working. Sorry for the long description. :/

No I don't know but it might take a while because the creator hasn't contributed anything since January? :(


use TreeWalker for custom decorators

In our queryAllDeeplyAssignedElements, queryDeeplyAssignedElement, and queryParentElement decorators, we are manually walking each element in the DOM.
Instead of doing DOM walks using JavaScript + Live DOM, we should be using a TreeWalker so that we can utilize the browsers (faster) parser.

Measure or don't bother.
Realistically we don't walk DOM nodes that often, or even through that many relations.


Backport PHP to SilverStripe 3 compatible code

Partial resolution of #18

I expect tests to fail as silverstripe/mfa is not yet backported (and is a dependency).

Thanks. I've been playing around with this getting the MFA module backported. We still need the LoggerInterface calls converted to SS_Log.

Lets get the MFA PR across the line and then rerun the builds before merge.


RFC: Evolve host.id into a default, always present attribute (rethinking machine-id)

Similarly to #311, I'd like to first describe some issues with the existing host.id semantics around machine-id usage and propose some alternatives.
According to machine-id(5):

It should be considered "confidential", and must not be exposed in untrusted environments, in particular on the network. If a stable unique identifier that is tied to the machine is needed for some application, the machine ID or any part of it must not be used directly. Instead the machine ID should be hashed with a cryptographic, keyed hash function, using a fixed, application-specific key.

This goes against OpenTelemetry guidelines which dictate using the value of /etc/machine-id (or /var/lib/dbus/machine-id) verbatim. Besides following the recommendation in the manpage (keyed hash), another alternative is to use UUIDv5 (SHA1), similarly to #312.
A secondary issue is that OpenTelemetry guidelines do not specify a fallback for host.id in cases where the machine-id is missing. For example, this is quite common in containerized environments if the Docker volume mount does not exist. In such cases, UUIDv4 can be used to generate a value that may be cached by the client application and reused (for as long as it makes sense given a context that's specific to the application).

cc @svrnm @sumo-drosiek @mwear

I remember reading the machine-id man file, but missed that section about confidentiality. I don't see a reason given for that?
If I understand the UUIDv5 definition it is equivalent to a "keyed hash", so it appears to be a logical solution to do something similar to #312. The question is what the "namespace" is (assuming the name is the machine-id), I guess this is best a fixed value, since the goal here is to have a "unique identifier that is tied to the machine".

A secondary issue is that OpenTelemetry guidelines do not specify a fallback for host.id in cases where the machine-id is missing. For example, this is quite common in containerized environments if the Docker volume mount does not exist. In such cases, UUIDv4 can be used to generate a value that may be cached by the client application and reused (for as long as it makes sense given a context that's specific to the application).

This is a topic that drove me crazy. So, to get started: My perspective is that host.id is simply not defined within a container that does not provide it's own machine-id, probably it's just best to not have it at all: in the case of a container, the container.id is what you want to have. But there is no reliable way to obtain it from within the container, see https://github.com/containerd/containerd/issues/8185

We discussed this on the System Semantic Conventions WG, let me try to summarize what I said (plus some last-minute thoughts).
My main points were:

Having a unique identifier of a host (let's assume a well-defined concept of a host, even if that's its own can of worms :) ) is a very useful thing to have for correlation and infrastructure monitoring generally. I have worked on this extensively at Datadog, where we (as it roughly happens in OTel) we rely on multiple sources and have multiple implementations. At Datadog, we typically do not use machine-id but rather use other sources that can be more meaningful to end-users or can be more easily retrieved (e.g. operating system hostname, EC2 instance id, Azure VM id, Kubernetes node and cluster name...).
Despite their usefulness, there are some times in which there is no meaningful unique identifier. This is the case for containerized systems or certain virtualized environments: container IDs are hard, and it could be that /etc/machine-id is empty as it happens on many container base images, or you may not have access to the hostname or it be a random one (also the case on containers many times!). This has been an issue for the OTel project in the past (see open-telemetry/opentelemetry-collector-contrib/issues/24230).
Generating a random identifier when you cannot get one is fragile in that restarts of the monitoring application (e.g. the Collector) or of the container churns new IDs. This can be a huge issue because of cardinality explosion and is also a problem we have had on OTel (see https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/18618#issuecomment-1539413938 for a user report, see "Additional context" in previously mentioned issue for even more details on how that happened). It's also hard (impossible?) to reliably tell that you are in a container, so you cannot (to my knowledge!) just generate it and store to a file just when you know it's going to be robust across restarts.
If this is really a security concern, we shouldn't be adding it on the OTel SDKs or the OTel Collector. We should instead hash it or some other issue. How to do this is somewhat arbitrary, so we should look at what other monitoring solutions do here and hash.

Based on your summary, the subject of this issue (Evolve host.id into a default, always present attribute) seems not to be something that can be accomplished (at least from within the OpenTelemetry community alone)?
A few extra comments:

Having a unique identifier of a host [...] is a very useful thing to have for correlation and infrastructure monitoring generally.

💯 -- this is really a BIG concern, coming up again and again. In a large environment when something goes wrong but only one (or a subset) of instances within a service are affected, it's crucial to know which one(s). As you outlined above, this is already not trivial for a lasting instance (bare metal, VMs, ...) and get's even more complicated with ephemeral instances (containers, ...). That's also why I raised https://github.com/containerd/containerd/issues/8185, which has a history of similar issues attached to it (see https://github.com/opencontainers/runtime-spec/issues/1105). @mx-psi if there is any value to it I can give a rundown of this to the System Semantic Conventions WG eventually.

It's also hard (impossible?) to reliably tell that you are in a container [...]

To call this out, even if the issue above with containers gets solved somehow, it will very likely stay optional, as it's in the nature of a being in a container to not know about being in a container (by default).

Given that we have no guidelines for containerized environments, does it make sense to add a containerized section (or generify the non-containerized one) and specify that the machine-id should be used if it is mapped inside the container? This is quite common through the volume mount and even works on macOS even though /etc/machine-id doesn't exist on the host:
macos$ cat /etc/machine-id
cat: /etc/machine-id: No such file or directory
macos$ docker run -v /etc/machine-id:/etc/machine-id --rm -ti alpine /bin/sh
/ # cat /etc/machine-id
xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx

Does it make sense to further specify that in cases where machine-id is missing or empty, host.id may be populated with a 'stable' low-cardinality value and leave the implementation up to the SDK/user? An example of such a value could be a MAC address.

Given that we have no guidelines for containerized environments, does it make sense to add a containerized section (or generify the non-containerized one)

Under the assumption that a container is identified by container.id what is the value of having host.id as well? I am also wondering if host.id in that case is even well defined: a container is running within a container engine, that is running on a "host system", that may have a host.id itself. If I know want to say "container with ID x is running on host with ID y" what is the right host.id?
I think the implicit definition (so far), was that host.id is not set within a container.

Resurrecting this thread:

Under the assumption that a container is identified by container.id what is the value of having host.id as well?

These are different attributes with different lifetimes and different semantics. The value of host.id stems from it exhibiting less temporal changes than container.id', which allows for stable and meaningful correlation at the host` level.

I am also wondering if host.id in that case is even well defined: a container is running within a container engine, that is running on a "host system", that may have a host.id itself. If I know want to say "container with ID x is running on host with ID y" what is the right host.id?

Going by the proposed updates in #576, in priority order:

instance_id assigned by cloud provider (if cloud)
`machine-id' (if mapped inside the container)
Something else that we might or might not want to make explicit (e.g. low-cardinality computed value based on host attributes such as the MAC address)

I think the implicit definition (so far), was that host.id is not set within a container.

To give you an example, our universal profiling product comes with deployment instructions that map machine-id inside the container, and host.id is populated from it. This enables stable correlation across thousands of deployed agents that would otherwise not be possible. Enabling this volume mount is not uncommon and even Docker on macOS supports it.
To recap:

I think we are all in agreement regarding hashing the machine-id value and not using it verbatim (e.g. see #312)
Can we agree that there is value in populating host.id' inside containers, if a low-cardinality and stable value (e.g. machine-id`) is available? That is to both encourage clients that can (or already) do this, but also to ensure that this behavior is not breaking the spec.

Can we agree that there is value in populating host.id inside containers

Yes, it is valuable, if and only if host.id is the id of the container host.

(e.g. machine-id) is available

Very often machine-id is not available from within the container and mounting it is also not possible/allowed (e.g. in managed environments), and for the same reasons it may not be desirable to expose it verbatim into the container. Additionally the container may create a machine-id on initialization itself and you need to distinguish that (maybe you can check if that file is mounted or not from within the container?)

I can't think of a way to reliably retrieve this value on a container

I unfortunately hadn't have the time to follow up on this, but I still think that our best option is working with container projects to get an at least optional but standardized way of making a container + container host identifiable from within the container, see https://github.com/containerd/containerd/issues/8185


Missing check the transmit buffer size of the sending SPDM endpoint when transferring large SPDM messages

The current code just check the DataTransferSize of the receiving SPDM endpoint, but it also should check the transmit buffer size of the sending SPDM endpoint when transferring large SPDM messages.

Refer to paragraph 681 in DSP0274_1.2.1.

A large SPDM message is an SPDM message whose size is greater than the DataTransferSize of the receiving
SPDM endpoint or greater than the transmit buffer size of the sending SPDM endpoint.

I think:

if ((((context->connection_info.capability.data_transfer_size != 0) &&
      (my_response_size > context->connection_info.capability.data_transfer_size)) ||
     ((context->local_context.capability.data_transfer_size != 0) &&
      (my_response_size > context->local_context.capability.data_transfer_size))) &&
    libspdm_is_capabilities_flag_supported(
        context, false, 0,
        SPDM_GET_CAPABILITIES_RESPONSE_FLAGS_CHUNK_CAP)) {

Maybe the code at the following line 596~600 should be changed as the above.
https://github.com/DMTF/libspdm/blob/e2f08f1cac9aa66970be5bf1f6d4b125deacc054/library/spdm_responder_lib/libspdm_rsp_receive_send.c#L595-L600

The same issue at the following line 555~556.
https://github.com/DMTF/libspdm/blob/e2f08f1cac9aa66970be5bf1f6d4b125deacc054/library/spdm_requester_lib/libspdm_req_send_receive.c#L552-L557

now we have LIBSPDM_SENDER_DATA_TRANSFER_SIZE and LIBSPDM_RECEIVER_DATA_TRANSFER_SIZE


Can we turn off the PR-updating bot?

It doesn't look like it's doing any good right now, and forces frequent rebases (unless I'm doing it wrong). Template changes are just gonna be reflected in the next update, right?

It does force frequent rebases, which is an unfortunate artifact of the architecture. But it's important to get these PRs for last scrape notes on the page as well as timely updates as soon as they happen.


Default to Flex Consumption

Flex Consumption is the new default hosting plan for Azure Functions with new capabilities. In this Visual Studio Code extension we should:

Have Flex Consumption at the top of the choices and be the default in the Advanced create flow. The order of preference is as per this doc - Flex Consumption, Premium Plan, Dedicated Plan, Container Apps, Consumption
Have Flex Consumption replace Consumption as the hosting plan used for the simple create flow, using the defaults for Flex Consumption as per the Stacks API and what's already used as the defaults for Flex Consumption in this extension

@MicroFish91 - do we have ETA for this?

@pragnagopa We don't currently have a release date for Functions. What time do you need this to be released?

As soon as possible.


[add2app]  Widgets in tabs are not displaying when initial index != 0  in the module launched from Android

[✓] Flutter (Channel beta, 1.19.0-4.3.pre, on Mac OS X 10.15.5 19F101, locale ru-UA)
    • Flutter version 1.19.0-4.3.pre at /Users/vladimirkropivko/flutter
    • Framework revision 8fe7655ed2 (6 days ago), 2020-07-01 14:31:18 -0700
    • Engine revision 9a28c3bcf4
    • Dart version 2.9.0 (build 2.9.0-14.1.beta)

Attach video with bug
tabs_bug.webm.zip

Hi  @vkropivko
Can you please provide your flutter run --verbose and a complete reproducible minimal code sample
Thank you

https://pastecode.ru/a83a83/ run --verbose
https://pastecode.ru/3e50ca/ - use tabs
https://pastecode.ru/6f687c/ Tab widget

Hi @vkropivko
Code sample you've you isn't reproducible minimal code sample and has missing pieces
Please check How to create a Minimal, Reproducible Example
Thank you

Hello, this is example with bug
https://bitbucket.org/vkropivko/test-flutter/src/master/
in tab_bug_page.dart you can change defaultIndex

I can reproduce this issue using opening flutter module from Android but no issues when running module itself

flutter doctor -v
[✓] Flutter (Channel beta, 1.19.0-4.3.pre, on Mac OS X 10.15.5 19F101, locale
    en-GB)
    • Flutter version 1.19.0-4.3.pre at /Users/taha/Code/flutter_beta
    • Framework revision 8fe7655ed2 (8 days ago), 2020-07-01 14:31:18 -0700
    • Engine revision 9a28c3bcf4
    • Dart version 2.9.0 (build 2.9.0-14.1.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.2

[✓] Connected device (3 available)
    • SM M305F   • 32003c30dc19668f • android-arm64  • Android 10 (API 29)
    • Web Server • web-server       • web-javascript • Flutter Tools
    • Chrome     • chrome           • web-javascript • Google Chrome
      83.0.4103.116

• No issues found!

Hi, what progress with this bug?

Hi @vkropivko
I tried to reproduce on the latest stable and master channel, widgets are correctly displayed

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.6, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.22.6 at /Users/tahatesser/Code/flutter_stable
    • Framework revision 9b2d32b605 (11 days ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.4)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.19.0

[✓] Connected device (2 available)
    • RMX2001 (mobile)                   • EUYTFEUSQSRGDA6D • android-arm64 • Android 10 (API 29)
    • Android SDK built for x86 (mobile) • emulator-5554    • android-x86   • Android 10 (API 29) (emulator)

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 1.26.0-18.0.pre.128, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.26.0-18.0.pre.128 at /Users/tahatesser/Code/flutter_master
    • Framework revision 502ff3dbfe (59 minutes ago), 2021-02-02 05:41:03 -0500
    • Engine revision 7334085fc1
    • Dart version 2.12.0 (build 2.12.0-281.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.19.0

[✓] Connected device (4 available)
    • RMX2001 (mobile)                   • EUYTFEUSQSRGDA6D • android-arm64  • Android 10 (API 29)
    • Android SDK built for x86 (mobile) • emulator-5554    • android-x86    • Android 10 (API 29) (emulator)
    • macOS (desktop)                    • macos            • darwin-x64     • macOS 11.1 20C69 darwin-x64
    • Chrome (web)                       • chrome           • web-javascript • Google Chrome 88.0.4324.96

• No issues found!

If the problem persists, Can you please provide your flutter doctor -v with the upgrade


(ArgumentError) comparison of Float with nil failed

PDF file:
http://ceokarnataka.kar.nic.in/ElectionFinalroll2014/PCROLL_2014/English/WOIMG/AC169/AC1690002.pdf
Error
Tabula API version: 1.0.0
Filename: AC1690002.pdf
Internal Server Error (500)
    Request Method:
    POST

    Request URL:
  http://<IP_ADDRESS>:34555/pdf/a1f68fa19b2cc054399a2c7375df1a18239a1c2a/data

(ArgumentError) comparison of Float with nil failed

I am having the same issue with all my bank account statements.

Ok, I've got some more information on this issue:
It seems to happen only when I use the "Autodetect Tables" feature.
If I select the relevant areas manually, the extraction works.
Also, if I clean the file a bit in Adobe Acrobat, leaving only the tables I want to extract, autodetection works fine.

Yep it seems to work with manual selection.
On 11 Aug 2015 12:33, "victortoulouse"<EMAIL_ADDRESS>wrote:

Ok, I've got some more information on this issue:
It seems to happen only when I use the "Autodetect Tables" feature.
If I select the relevant areas manually, the extraction works.
Also, if I clean the file a bit in Adobe Acrobat, leaving only the tables
I want to extract, autodetection works fine.
—
Reply to this email directly or view it on GitHub
https://github.com/tabulapdf/tabula/issues/355#issuecomment-129846073.

I was able to replicate the error from the first document (I won't ask you to share your bank statements, of course :) ), here's the traceback:
ArgumentError: comparison of Float with nil failed
	org/jruby/RubyFloat.java:615:in `<'
	/home/merrillj/.rbenv/versions/jruby-1.7.15/lib/ruby/gems/shared/gems/tabula-extractor-0.7.6-java/lib/tabula/entities/page.rb:296:in `snap_points!'
	org/jruby/RubyArray.java:1613:in `each'
	org/jruby/RubyEnumerable.java:865:in `inject'
	/home/merrillj/.rbenv/versions/jruby-1.7.15/lib/ruby/gems/shared/gems/tabula-extractor-0.7.6-java/lib/tabula/entities/page.rb:293:in `snap_points!'
	org/jruby/RubyArray.java:1613:in `each'
	/home/merrillj/.rbenv/versions/jruby-1.7.15/lib/ruby/gems/shared/gems/tabula-extractor-0.7.6-java/lib/tabula/entities/page.rb:290:in `snap_points!'
	/home/merrillj/.rbenv/versions/jruby-1.7.15/lib/ruby/gems/shared/gems/tabula-extractor-0.7.6-java/lib/tabula/entities/page.rb:218:in `get_ruling_lines!'
	/home/merrillj/.rbenv/versions/jruby-1.7.15/lib/ruby/gems/shared/gems/tabula-extractor-0.7.6-java/lib/tabula/entities/page.rb:204:in `horizontal_ruling_lines'
	/home/merrillj/.rbenv/versions/jruby-1.7.15/lib/ruby/gems/shared/gems/tabula-extractor-0.7.6-java/lib/tabula/entities/page.rb:41:in `minimal_bounding_box_of_ruling_lines'
	/home/merrillj/.rbenv/versions/jruby-1.7.15/lib/ruby/gems/shared/gems/tabula-extractor-0.7.6-java/lib/tabula/entities/page.rb:25:in `initialize'
	/home/merrillj/.rbenv/versions/jruby-1.7.15/lib/ruby/gems/shared/gems/tabula-extractor-0.7.6-java/lib/tabula/entities/page.rb:85:in `get_area'
	/home/merrillj/code/tabula/webapp/tabula_web.rb:80:in `extract_tables'
	org/jruby/RubyArray.java:1613:in `each'
	/home/merrillj/code/tabula/webapp/tabula_web.rb:72:in `extract_tables'```

It looks like Autodetect Tables is surfacing some table selections that don't work. I'll take a look at why that is. (I'm suspecting it may be related to pagination on the selection page.)

The autodetect issue is, I believe, fixed via https://github.com/tabulapdf/tabula/commit/6d5c22203e6e683eef334aa343bd33259cfabc85, which will be part of the next bugfix release.
Thank you both for reporting this bug, it's appreciated.


Update the api gateway cache content

Description:
Need to update [1] based on the following comment.
In this doc we say we connect key manager to validate tokens. But since this is 4.0.0 i think we will have to make it clear.
[1] https://apim.docs.wso2.com/en/latest/install-and-setup/setup/advance-configurations/configuring-caching/#api-gateway-cache

@tharindu1st , please provide feedback with regard to Sanjeewa's comment and let us know how we should update the documentation.

@tharindu1st , we are still waiting for your feedback in this regard.


module found but does not contain package

Bug Report

import path of package in question, e.g. .../services/compute/mgmt/2018-06-01/compute
SDK version e.g. master, latest, 18.1.0

Specify the exact commit if possible; one way to get this is the REVISION
column output by dep status "github.com/Azure/azure-sdk-for-go.

output of go version
go version go1.20.6 darwin/amd64

What happened?
run these command:

go get -u
go mod tidy
and got this error message:

go: finding module for package github.com/Azure/azure-sdk-for-go/sdk/azcore/fake/server
go: finding module for package github.com/Azure/azure-sdk-for-go/sdk/azcore/fake
github.wdf.sap.corp/DBaaS/cloud-provider-exporter/pkg/cloud/azure/monitor imports
        github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor tested by
        github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor.test imports
        github.com/Azure/azure-sdk-for-go/sdk/azcore/fake: module github.com/Azure/azure-sdk-for-go/sdk/azcore@latest found (v1.7.0), but does not contain package github.com/Azure/azure-sdk-for-go/sdk/azcore/fake
github.wdf.sap.corp/DBaaS/cloud-provider-exporter/pkg/cloud/azure/monitor imports
        github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor tested by
        github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor.test imports
        github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor/fake imports
        github.com/Azure/azure-sdk-for-go/sdk/azcore/fake/server: module github.com/Azure/azure-sdk-for-go/sdk/azcore@latest found (v1.7.0), but does not contain package github.com/Azure/azure-sdk-for-go/sdk/azcore/fake/server

What did you expect or want to happen?
go mod command runs well

How can we reproduce it?

Anything we should know about your environment.

This is fixed in version v0.10.1.

I meet the same error again, and the error shows:
❯ go mod tidy
go: finding module for package github.com/Azure/azure-sdk-for-go/sdk/azcore/fake/server
go: finding module for package github.com/Azure/azure-sdk-for-go/sdk/azcore/fake
github.wdf.sap.corp/DBaaS/cloud-provider-exporter/pkg/cloud/azure/monitor imports
	github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor tested by
	github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor.test imports
	github.com/Azure/azure-sdk-for-go/sdk/azcore/fake: module github.com/Azure/azure-sdk-for-go/sdk/azcore@latest found (v1.8.0), but does not contain package github.com/Azure/azure-sdk-for-go/sdk/azcore/fake
github.wdf.sap.corp/DBaaS/cloud-provider-exporter/pkg/cloud/azure/monitor imports
	github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor tested by
	github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor.test imports
	github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/monitor/armmonitor/fake imports
	github.com/Azure/azure-sdk-for-go/sdk/azcore/fake/server: module github.com/Azure/azure-sdk-for-go/sdk/azcore@latest found (v1.8.0), but does not contain package github.com/Azure/azure-sdk-for-go/sdk/azcore/fake/server

The armmonitor SDK requires the latest azcore beta v1.9.0-beta.1.  What's the scenario where you're running into this?

When I run these commands in my project, I get this error:

go get -u
go mod tidy
Because I want to upgrade the package versions in my project.

The version v1.8.0-beta.3 is OK, but it will be changed to 1.8.0 after go get -u.
Now, I manually changed the version to v1.9.0-beta.1, it works fine.


Include latest generated files and add clean_all target to rebuild

repo includes el.pb.? files and compiles them (without generating them) by default when make is run. A make clean does not remove them, however, a new target clean_all does remove them and then a make does generate them again.

Doing an rm on files in the repo (which is what would happen with a make clean_all) is not something that's normally done. I suggest that instead of creating a clean_all target, we instead add a generate target, which forces the el.pb.h and el.pb.c to be built, and remove the protocol directory from the clean target. I.e., something like the following (the diffs are against the original Makefiles, not against yours):
diff --git a/Makefile b/Makefile
index ded4b38..bc6be73 100644
--- a/Makefile
+++ b/Makefile
@@ -18,8 +18,6 @@ SKY_PROTO_DIR = $(API_DIR)/protocol
 NANO_PB_DIR = .submodules/nanopb
 AES_DIR = .submodules/tiny-AES128-C

-GENERATED_SRCS = ${SKY_PROTO_DIR}/el.pb.h ${SKY_PROTO_DIR}/el.pb.c
-
 INCLUDES = -I${SKY_PROTO_DIR} -I${NANO_PB_DIR} -I${AES_DIR} -I${API_DIR}

 VPATH = ${SKY_PROTO_DIR}:${API_DIR}:${NANO_PB_DIR}:${AES_DIR}
@@ -39,14 +37,14 @@ unit_test: ${BUILD_DIR}/unit_test.o ${BIN_DIR}/libel.a
        $(CC) -lc -lm -o ${BIN_DIR}/unit_test \
        ${BUILD_DIR}/unit_test.o ${BIN_DIR}/libel.a

-${BIN_DIR}/libel.a: ${GENERATED_SRCS} ${LIBELG_OBJS}
+${BIN_DIR}/libel.a: ${LIBELG_OBJS}
        ar rcs $@ ${LIBELG_OBJS}

 ${BIN_DIR} ${BUILD_DIR}:
        mkdir -p $@

 # Generates the protobuf source files.
-${GENERATED_SRCS}:
+generate:
        make -C ${SKY_PROTO_DIR}

 # Need an explicit rule for this one since the source file is generated code.
@@ -57,5 +55,4 @@ ${BUILD_DIR}/%.o: %.c beacons.h  config.h  crc32.h  libel.h  utilities.h  worksp
        $(CC) -c $(CFLAGS) ${INCLUDES} -o $@ $<

 clean:
-       make -C ${SKY_PROTO_DIR} clean
        rm -rf ${BIN_DIR} ${BUILD_DIR}
diff --git a/libel/protocol/Makefile b/libel/protocol/Makefile
index d0ed336..028a4fe 100644
--- a/libel/protocol/Makefile
+++ b/libel/protocol/Makefile
@@ -3,6 +3,10 @@ include ../../.submodules/nanopb/extra/nanopb.mk

 TARGETS = el.pb.c el.pb.h

+${TARGETS}: always
+
+.PHONY: always
+
 default: ${TARGETS}

 clean:

I've seen recommendations for using touch to cause actions in child Makefiles. See updated Makefile which does not require forcing in the child, but includes your suggestions.


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.

let's bask in the glory of 0.2.7 for a bit longer

@yutongz  probably the bot shouldn't merged closed PRs ?
@sebastienvas is there a way for some of us to pause the bots for some time ?
(we can do updates but I was thinking we could stay with 0.2.7 for a few hours, could be useful in the future as well)

Yeah, I think that's bug.

That is weird. This is a race condition. Since there is no locking there is no way for us to check if a PR is closed before merging it. However, we are using the github API (as opposed to git push) to merge the PR, so if they allow that, that's a bug in their API. You can't file issue to github - that's funny.


ConfigServerHealthIndicator fails to mark as down when git repository becomes inaccessible

Describe the bug
Using spring cloud 3.1.3 we ran a scenario where we had a git repository when spring cloud started and then removed access (so as not to impact others, we essentially ran it behind tcpproxy and then stopped the proxy).  We expected a healtcheck to trigger.
We saw a stack trace occur:
2022-08-23 17:53:38.498 DEBUG 867680 --- [alth-1-thread-3] .s.c.c.s.s.GitCredentialsProviderFactory : Constructing UsernamePasswordCredentialsProvider for URI https://localhost:8433/SHY/shy-cloud-configuration.git
2022-08-23 17:53:38.499 DEBUG 867680 --- [alth-1-thread-3] .s.c.c.s.s.GitCredentialsProviderFactory : Constructing GitSkipSslValidationCredentialsProvider for URI https://localhost:8433/SHY/shy-cloud-configuration.git
2022-08-23 17:53:38.526  WARN 867680 --- [alth-1-thread-3] .c.s.e.MultipleJGitEnvironmentRepository : Could not fetch remote for local remote: https://localhost:8433/SHY/shy-cloud-configuration.git
2022-08-23 17:53:38.528 DEBUG 867680 --- [alth-1-thread-3] .c.s.e.MultipleJGitEnvironmentRepository : Stacktrace for: Could not fetch remote for local remote: https://localhost:8433/SHY/shy-cloud-configuration.git

org.eclipse.jgit.api.errors.TransportException: https://localhost:8433/SHY/shy-cloud-configuration.git: connection failed
	at org.eclipse.jgit.api.FetchCommand.call(FetchCommand.java:224)
	at org.springframework.cloud.config.server.environment.JGitEnvironmentRepository.fetch(JGitEnvironmentRepository.java:551)
	at org.springframework.cloud.config.server.environment.JGitEnvironmentRepository.refresh(JGitEnvironmentRepository.java:298)
	at org.springframework.cloud.config.server.environment.JGitEnvironmentRepository.getLocations(JGitEnvironmentRepository.java:262)
	at org.springframework.cloud.config.server.environment.MultipleJGitEnvironmentRepository.getLocations(MultipleJGitEnvironmentRepository.java:139)
	at org.springframework.cloud.config.server.environment.AbstractScmEnvironmentRepository.findOne(AbstractScmEnvironmentRepository.java:55)
	at org.springframework.cloud.config.server.environment.MultipleJGitEnvironmentRepository.findOneFromCandidate(MultipleJGitEnvironmentRepository.java:188)
	at org.springframework.cloud.config.server.environment.MultipleJGitEnvironmentRepository.findOne(MultipleJGitEnvironmentRepository.java:173)
	at org.springframework.cloud.config.server.environment.CompositeEnvironmentRepository.findOne(CompositeEnvironmentRepository.java:64)
	at org.springframework.cloud.config.server.config.ConfigServerHealthIndicator.doHealthCheck(ConfigServerHealthIndicator.java:72)
	at org.springframework.boot.actuate.health.AbstractHealthIndicator.health(AbstractHealthIndicator.java:82)
	at org.springframework.boot.actuate.health.HealthIndicator.getHealth(HealthIndicator.java:37)
	at org.springframework.boot.actuate.health.HealthEndpoint.getHealth(HealthEndpoint.java:94)
	at org.springframework.boot.actuate.health.HealthEndpoint.getHealth(HealthEndpoint.java:41)
	at org.springframework.boot.actuate.health.HealthEndpointSupport.getLoggedHealth(HealthEndpointSupport.java:172)
	at org.springframework.boot.actuate.health.HealthEndpointSupport.getContribution(HealthEndpointSupport.java:145)
	at org.springframework.boot.actuate.health.HealthEndpointSupport.getAggregateContribution(HealthEndpointSupport.java:156)
	at org.springframework.boot.actuate.health.HealthEndpointSupport.getContribution(HealthEndpointSupport.java:141)
	at org.springframework.boot.actuate.health.HealthEndpointSupport.getHealth(HealthEndpointSupport.java:110)
	at org.springframework.boot.actuate.health.HealthEndpointSupport.getHealth(HealthEndpointSupport.java:81)
	at org.springframework.boot.actuate.health.HealthEndpoint.health(HealthEndpoint.java:88)
	at org.springframework.boot.actuate.health.HealthEndpoint.health(HealthEndpoint.java:78)
	at com.cba.cas.shy.health.starter.ShyHealthAutoConfiguration$HealthRecorderConfiguration$healthRecorder$1.invoke(ShyHealthAutoConfiguration.kt:172)
	at com.cba.cas.shy.health.starter.ShyHealthAutoConfiguration$HealthRecorderConfiguration$healthRecorder$1.invoke(ShyHealthAutoConfiguration.kt:171)
	at com.cba.cas.shy.health.starter.HealthRecorder.determineOverallHealth(HealthRecorder.kt:68)
	at com.cba.cas.shy.health.starter.HealthRecorder.access$determineOverallHealth(HealthRecorder.kt:22)
	at com.cba.cas.shy.health.starter.HealthRecorder$start$1.invoke(HealthRecorder.kt:54)
	at com.cba.cas.shy.health.starter.HealthRecorder$start$1.invoke(HealthRecorder.kt:52)
	at com.cba.cas.shy.health.starter.ShyHealthAutoConfiguration$healthFixedDelayScheduler$1$1.invoke(ShyHealthAutoConfiguration.kt:147)
	at com.cba.cas.shy.health.starter.ShyHealthAutoConfiguration$healthFixedDelayScheduler$1$1.invoke(ShyHealthAutoConfiguration.kt:144)
	at com.cba.cas.shy.extensions.lang.LangExtensionsKt.scheduleWithFixedDelayFunc$lambda-11(LangExtensions.kt:503)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
	at java.base/java.util.concurrent.FutureTask.runAndReset$$$capture(FutureTask.java:305)
	at java.base/java.util.concurrent.FutureTask.runAndReset(FutureTask.java)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:305)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: org.eclipse.jgit.errors.TransportException: https://localhost:8433/SHY/shy-cloud-configuration.git: connection failed
	at org.eclipse.jgit.transport.TransportHttp.connect(TransportHttp.java:729)
	at org.eclipse.jgit.transport.TransportHttp.openFetch(TransportHttp.java:465)
	at org.eclipse.jgit.transport.FetchProcess.executeImp(FetchProcess.java:142)
	at org.eclipse.jgit.transport.FetchProcess.execute(FetchProcess.java:94)
	at org.eclipse.jgit.transport.Transport.fetch(Transport.java:1309)
	at org.eclipse.jgit.api.FetchCommand.call(FetchCommand.java:213)
	... 37 common frames omitted
Caused by: java.net.ConnectException: Connect to localhost:8433 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused (Connection refused) localhost
	at org.eclipse.jgit.util.HttpSupport.response(HttpSupport.java:213)
	at org.eclipse.jgit.transport.TransportHttp.connect(TransportHttp.java:654)
	... 42 common frames omitted

And tracing through, seems that whilst the source is expecting to catch and exception (ConfigServerHealthIndicator, line 91), that exception does not occur, because JGitEnvironmentRepository swallows it: you
protected void doHealthCheck(Health.Builder builder) throws Exception {
		builder.up();
		List<Map<String, Object>> details = new ArrayList<>();
		for (String name : this.repositories.keySet()) {
			Repository repository = this.repositories.get(name);
			String application = (repository.getName() == null) ? name : repository.getName();
			String profiles = repository.getProfiles();

			try {
				Environment environment = this.environmentRepository.findOne(application, profiles,
						repository.getLabel(), false);

				HashMap<String, Object> detail = new HashMap<>();
				detail.put("name", environment.getName());
				detail.put("label", environment.getLabel());
				if (environment.getProfiles() != null && environment.getProfiles().length > 0) {
					detail.put("profiles", Arrays.asList(environment.getProfiles()));
				}

				if (!CollectionUtils.isEmpty(environment.getPropertySources())) {
					List<String> sources = new ArrayList<>();
					for (PropertySource source : environment.getPropertySources()) {
						sources.add(source.getName());
					}
					detail.put("sources", sources);
				}
				details.add(detail);
			}
91:			catch (Exception e) {
				logger.debug("Could not read repository: " + application, e);
				HashMap<String, String> map = new HashMap<>();
				map.put("application", application);
				map.put("profiles", profiles);
				builder.withDetail("repository", map);
				builder.down(e);
				return;
			}
		}
		builder.withDetail("repositories", details);

	}

can see here the exception is logged and not thrown:
protected FetchResult fetch(Git git, String label) {
		FetchCommand fetch = git.fetch();
		fetch.setRemote("origin");
		fetch.setTagOpt(TagOpt.FETCH_TAGS);
		fetch.setRemoveDeletedRefs(this.deleteUntrackedBranches);
		if (this.refreshRate > 0) {
			this.setLastRefresh(System.currentTimeMillis());
		}

		configureCommand(fetch);
		try {
			FetchResult result = fetch.call();
			if (result.getTrackingRefUpdates() != null && result.getTrackingRefUpdates().size() > 0) {
				this.logger.info("Fetched for remote " + label + " and found " + result.getTrackingRefUpdates().size()
						+ " updates");
			}
			return result;
		}
		catch (Exception ex) {
			String message = "Could not fetch remote for " + label + " remote: "
					+ git.getRepository().getConfig().getString("remote", "origin", "url");
			warn(message, ex);
			return null;
		}
	}

Can someone please clarify if this is the correct intention, as I would have expected the health to indicate down.

This seems similar to https://github.com/spring-cloud/spring-cloud-config/issues/1934.  Let me know what you think.

This seems similar to #1934. Let me know what you think.

I dont think so.  My expectation was that if cloud config was running and then the git repository had been taken offline (we use our own on-prem solution, that is occasionally down for maintenance/update etc) that the health check would indicate something wrong, but in this case the exception that would have flowed back up seems to have been swallowed and therefore the health doesn't fire.
Having said that though perhaps in this case if git repo was down its better that the service is just marked as CRITICAL - ie still functions with the data it had before, whilst still running.
For now we have written our own healthcheck with status appropriate to our situation, but i wasn't sure we were intrepretting correctly how the cloud config healthcheck was expected to behave in this situation - ie started and then git repo unavailable.

When the git repository is taken offline, can the config server still serve configuration?

Yes,  I hit it with curl commands and got results.  We were basically trying to check that in our microservices if our internal gut repo was down that we could still function. In this case the healthcheck would just allow us to be aware of potential issues,  I.e. best not to restart config server, though we have now changed it to not clone on startup, which has had the behaviour such that if we had previously started before and the repo had at some point been cloned then the repo went off-line that we could still restart. It means our clients can come up, obviously at this point we can't change config, but we can operate off the most recent config whilst our repo comes back online.

OK so to me this seems very similar to the other issue.  It basically comes down to whether the health indicator should report some kind of degraded functionality because it can't reach the git repo.  However it can still do its job and serve configuration data in this state.


Fix autoscaling.Tags to use boolean instead of str

In the https://github.com/cloudtools/troposphere/pull/1409  all of the boolean parameters were converted from str to boolean when TROPO_REAL_BOOL=true,
The autoscaling.Tags was always converting the boolean parameter into str.

Thanks!