Tileboard starts to ask for password when including a COVER

My Tileboard doesn't have a password ( password: null,) , but when I include a cover in config.js tileboard starts to ask for a password! Weird.  What is wrong?
{
position: [0, 3],
type: TYPES.COVER,
title: 'Garage door',
id: 'cover.garage1'
states: {
open: 'Open',
closed: 'Closed'
}
},

Usually that means there's a Javascript syntax error in your config.js.  Thanks for figure it out.  I got the example from
https://community.home-assistant.io/t/new-dashboard-for-ha/57173/266


feature request: opening behaviour customization

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
It would be cool to allow the user to define options how Oil should open.
For example: oil starts with the preview turned on
Or: oil starts in a split on the right side of the screen and the previously opened buffer stays in the split on the left
Provide background
No response
What is the significance of this feature?
nice to have
Additional details
No response

Or: oil starts with float window rather than full window width buffer

You can achieve these behaviours using the neovim API:
-- Open oil in split
vim.keymap.set("n", "<leader>os", "<cmd>vs | Oil<cr>", { desc = "Open oil in split" })

-- Open oil in float window
vim.keymap.set("n", "-", "<cmd>Oil --float<cr>", { desc = "Open parent directory" })

-- Open oil with preview
vim.keymap.set("n", "<leader>op", function()
	local oil = require("oil")
	local util = require("oil.util")

	oil.open()
	util.run_after_load(0, function()
		oil.select({ preview = true })
	end)
end, { desc = "Open oil with preview" })

-- Map q or <escape> to close buffer only for floating Oil buffers
vim.api.nvim_create_autocmd("User", {
	group = vim.api.nvim_create_augroup("OilFloatCustom", {}),
	pattern = "OilEnter",
	callback = function()
		local actions = require("oil.actions")
		local util = require("oil.util")
		if util.is_floating_win() then
			vim.keymap.set("n", "<Esc>", actions.close.callback, {
				buffer = true,
			})
			vim.keymap.set("n", "q", actions.close.callback, {
				buffer = true,
			})
		end
	end,
})

You can achieve these behaviours using the neovim API:

-- Open oil in split

vim.keymap.set("n", "<leader>os", "<cmd>vs | Oil<cr>", { desc = "Open oil in split" })

-- Open oil in float window

vim.keymap.set("n", "-", "<cmd>Oil --float<cr>", { desc = "Open parent directory" })

-- Open oil with preview

vim.keymap.set("n", "<leader>op", function()

	local oil = require("oil")

	local util = require("oil.util")

	oil.open()

	util.run_after_load(0, function()

		oil.select({ preview = true })

	end)

end, { desc = "Open oil with preview" })

-- Map q or <escape> to close buffer only for floating Oil buffers

vim.api.nvim_create_autocmd("User", {

	group = vim.api.nvim_create_augroup("OilFloatCustom", {}),

	pattern = "OilEnter",

	callback = function()

		local actions = require("oil.actions")

		local util = require("oil.util")

		if util.is_floating_win() then

			vim.keymap.set("n", "<Esc>", actions.close.callback, {

				buffer = true,

			})

			vim.keymap.set("n", "q", actions.close.callback, {

				buffer = true,

			})

		end

	end,

})

Thank you mayne!

UPDATE:
oil.select({ preview = true }) is deprecated and oil.open_preview() should be used instead.
This is the updated keymap to open the oil filetree with preview enabled:
	vim.keymap.set("n", "<leader>ee", function()
		local oil = require("oil")
		local util = require("oil.util")
		oil.open()
		util.run_after_load(0, function()
			oil.open_preview()
		end)
	end, { desc = "oil: open file explorer with preview" }),


Raw string literals (more than one #, grammar limitations prevent us from supporting escapes)

https://github.com/jtbandes/swift-tmlanguage/blob/4028d24f926bccca29204528ee97ecd455d5908e/Swift.tmLanguage.yaml#L2339 What limitation are you facing?
syntax examples?
https://github.com/apple/swift-evolution/blob/main/proposals/0168-multi-line-string-literals.md
doesn't seem to say anything about hashtags #
is it this section?
https://github.com/apple/swift-evolution/blob/main/proposals/0200-raw-string-escaping.md#customized-escape-delimiters

I see the problem now
could possibly add a single line raw string literal that supports all the delimiters
but then you probably couldn't support \#( 'meta.embedded.line.swift' ) 😔

I think I've managed to get something to work
and is probably most cursed textmate I've ever written (worse than ((?!(?<!\\\\)\\\\{2}*\").)*?)

just had "comment" as a place holder to visualize whats being tokenized
it works by chaining begin/end groups one after another
the end rule doesn't capture anything. allowing the begin rule to retarget the escape delimiter and "literals-string-raw-string-guts" to tokenize
annoying I had to make an exception for the first set of non-escape-delimiters
(?>\\\\(?!\\1(?!#))#+)*
otherwise the very first \# and \### would be passed into "literals-string-raw-string-guts" and be marked as invalid (for \## string delimiters)
{
	"name": "swift-raw-strings",
	"scopeName": "source.redcmd.syntax.swift-raw-strings",
	"patterns": [ { "include": "#swift-raw-strings" } ],
	"repository": {
		"swift-raw-strings": {
			"begin": "(##+)\"",
			"end": "\"\\1(#*)",
			"beginCaptures": { "0": { "name": "punctuation.definition.string.begin.raw.swift" } },
			"endCaptures": {
				"0": { "name": "punctuation.definition.string.end.raw.swift" },
				"1": { "name": "invalid.illegal.extra-closing-delimiter.swift" }
			},
			"patterns": [
				{
					"begin": "(?<=(?>[^#]|^)(#+)\")\\G(?>\\\\(?!\\1(?!#))#+)*",
					"end": "(?!\\G)(?=\\\\\\1(?!#))|(?=\"\\1)",
					"name": "comment",
					"patterns": [ { "include": "#literals-string-raw-string-guts" } ]
				},
				{
					"begin": "(?=\\\\(#+)(?!#))",
					"end": "(?!\\G)(?=\\\\\\1(?!#))|(?=\"\\1)",
					"name": "comment",
					"patterns": [ { "include": "#literals-string-raw-string-guts" } ]
				}
			]
		},
		"literals-string-raw-string-guts": {
			"comment": "the same as #string-guts but with # in escapes",
			"patterns": [
				{
					"name": "constant.character.escape.swift",
					"match": "\\G\\\\#+[0\\\\tnr\"']"
				},
				{
					"name": "constant.character.escape.unicode.swift",
					"match": "\\G\\\\#+u\\{[0-9a-fA-F]{1,8}\\}"
				},
				{
					"contentName": "source.swift",
					"name": "meta.embedded.line.swift",
					"begin": "\\G\\\\#+\\(",
					"end": "(\\))",
					"beginCaptures": { "0": { "name": "punctuation.section.embedded.begin.swift" } },
					"endCaptures": {
						"0": { "name": "punctuation.section.embedded.end.swift" },
						"1": { "name": "source.swift" }
					},
					"patterns": [
						{
							"comment": " --> $self <-- ",
							"include": "source.swift"
						},
						{
							"comment": "Nested parens",
							"begin": "\\(",
							"end": "\\)"
						}
					]
				},
				{
					"name": "invalid.illegal.escape-not-recognized",
					"match": "\\G\\\\#+."
				}
			]
		}
	}
}

a

b##"\#\###\##t\###\##z\#

c
\##t
d
\##z
e
\##(
func foo<T>(x: T) async throws -> Int where T: Equatable {}
)
f
\#
g
\##u{0123}
h
\###
i

j"##

k

Wow that's quite cursed, congrats 🫠  It's gonna take  me a while to wrap my head around this.
Sounds like you've answered your question yourself but I believe the limitations I was referring to was that I didn't think it was possible to reference groups from begin inside the scope content pattern itself (so that begin=####" would require escapes to be like \####(...) inside the scope)
Does your solution handle things like ##"  \##(this is interpolated) \#(but this is not interpolated) "## ?

Does your solution handle things like...

\#( & \###( will be treated as raw string
\##( well be recognized as the source.swift
I'm assuming that delimiters are not recognized inside \##( ... )##?
also
should be able to remove the exception from the first pattern
as long as that include can be removed
"begin": "(?<=(?>[^#]|^)(#+)\")\\G",
"end": "(?=\\\\\\1(?!#))|(?=\"\\1)",
"name": "comment"

better exception handling
"swift-raw-strings": {
	"begin": "(#+)\"",
	"end": "\"\\1(#*)",
	"beginCaptures": { "0": { "name": "punctuation.definition.string.begin.raw.swift" } },
	"endCaptures": {
		"0": { "name": "punctuation.definition.string.end.raw.swift" },
		"1": { "name": "invalid.illegal.extra-closing-delimiter.swift" }
	},
	"name": "string.quoted.double.single-line.raw.swift",
	"patterns": [
		{
			"begin": "(?<=(?>[^#]|^)(#+)\")\\G(?!\\\\\\1(?!#))",
			"end": "(?=\\\\\\1(?!#))|(?=\"\\1)",
			"patterns": [
				{
					"name": "invalid.illegal.returns-not-allowed.swift",
					"match": "\\r|\\n"
				}
			]
		},
		{
			"begin": "(?=\\\\(#+)(?!#))",
			"end": "(?!\\G)(?=\\\\\\1(?!#))|(?=\"\\1)",
			"patterns": [
				{
					"name": "invalid.illegal.returns-not-allowed.swift",
					"match": "\\r|\\n"
				},
				{ "include": "#literals-string-raw-string-guts" }
			]
		}
	]
}


v1.2.0 bump in TW plugin

Thanks for the new release! You probably forgot to bump version in tw-plugin/plugin.info. Update cannot be done:

Thanks, I forgot this spot due to a typo in my release instructions and then missed the problem because I only tested the install in a folder wiki (which worked fine). It should be working now, and I've updated the instructions and my smoke tests to avoid this issue in the future.

Thanks, works like a charm!

Adding commit cross-references since I forgot to include the issue number: 1f03bbd9f80fba40b982ed0dd58a2d8a975a1d53, 86c9a3949ab1d0bcf671a82947fae6407f164b53.


updated counter example to match latest API

I'll try to update other outdated examples as well, if any.

Just figured out that the version that I was using (the one from crates.io) it's outdated. So I'm closing this PR.


Use optional module "@microsoft/typescript-etw" for ETW logging

Use the @microsoft/typescript-etw native module for ETW logging.
@microsoft/typescript-etw (and ETW in general) are only supported on Windows, so it is not a good idea to add it to the "dependencies" in package.json. Even on Windows we still do not want this module installed by default. Consumers who are interested (such as Visual Studio) should have to opt in. Therefore ETW logging will only be supported if @microsoft/typescript-etw is installed manually. The code will work if the module is found, otherwise it will do nothing.
Types are added from @types/microsoft__typescript-etw so that we can have type checking even if the actual package is not installed.

@weswigham I added a PerfLogger and NullLogger class which addresses your first concern and makes things cleaner. Let me know if it should be organized differently (e.g. is that the correct file and namespace?)
I considered the second suggestion (moving conditional logic inside the logger class), but that is very unwieldy given the strong typing that is currently in place. We would need logInfoEventIf(), logStartCommandIf(), logStopCommandIf(), etc... It seems like overkill for this problem.

We'll also need to add "@microsoft/typescript-etw": false to the browser table in our package.json

@typescript-bot perf test this


Add Storage Class and Capacity to the Run CLI

User should be able to specify the storage class and capacity while running the Acorn like:
acorn run -v my-volume:data,class=fast,size=100 ...
This should also be on the dev command?

This has been addressed.
confirmed in acorn version v0.1.0+80d6e93d


Add Asset page for Classification , Tags

Feature
Similar to Glossary Term, Add Asset page. We should add an Assets Page and allow users to tag multiple assets from Classificaiton Page.

@harshach We have a separate page for glossary terms which we don't have for tags. Since we want the Add Asset page for tags, we will have to add a tags details page first to show the assets as well. Is that the idea here?

Hi there, is there any update on this issue? I see this feature still not available on the latest release v1.2


Fixes to mDNS resolver

If the host doesn't support IPv6 (or supports only IPv6), allow the client to resolve IPs
Better concurrency handling with picking an address from the list via round-robin

Lint

@berndverst fixed and ready for review


Added StrictMode in Android target

Current implementation crashes with IO operations in the main thread if StrictMode.detectDiskReads(). Looks like Firebase libraries reads preferences in main thread, and LaunchedEffect with 'Dispatchers.IO' solves the issue. Probably there are better ways fix this issue.
The second place is 'Image' component. Image read vector resources in the main thread in "1.6.10" version, but "1.7.0-alpha03" version have no issue with StrictMode. Probably it is better to wait "1.7.0" release version of compose multiplatform.

@siarhei-luskanau Thank you very much for bringing this up. Turns out StrictMode violation occurs because of initialization of Firebase. So basically if you just add  Firebase.auth code into your application start method, then there will be no issue :)

@siarhei-luskanau Closing this PR, as these changes would be irrelevant. But thank you for temporary solution!


Permissions: Template/Scaffolder:  READ missing User object in handle call, CREATE and DELETE have it

Expected Behavior
Template Access control:

permission.ts has hardcoded "admin" usernames that return ALLOW for all requests instantly when identity.userEntityRef matches
non-Template entities return ALLOW for READ but check for Ownership for DELETE (identity.ownershipEntityRefs)
Template entities check Ownership OR "unrestricted" label (catalogConditions.hasLabel)
Under the hood: Read permissions triggered when opening Create page/scaffolder, the handle function should receive existing BackstageIdentityResponse aka. _user for all READ attempts.

Actual Behavior
Production:

READ permission requests for Create/Scaffolder have "undefined" _user passed in handle function call, making admin or ownership checks impossible.
Delete restriction works correctly in catalog - _user is passed and non-owners are denied Deletion, while Admins are also recognised
Create permission requests also have the _user passed in

Local development - everything works as expected
Steps to Reproduce
Our packages/backend/src/plugins/permission.ts
const isAdmin = (user: BackstageIdentityResponse | undefined) =>
  ['filipswiatczak', '...', '...'].some(admin =>
    user?.identity.userEntityRef.includes(admin),
  );

class CustomPolicy implements PermissionPolicy {
  async handle(
    request: PolicyQuery,
    _user?: BackstageIdentityResponse,
  ): Promise<PolicyDecision> {
    console.log(
      `User: identity.type - ${_user?.identity.type} User: identity.userEntityRef - ${_user?.identity.userEntityRef} User: identity.ownershipEntityRefs.length - ${_user?.identity.ownershipEntityRefs.length} Request: type - ${request.permission.type}; name - ${request.permission.name}; action - ${request.permission.attributes.action}`,
    );
    // exempting admins from permission checks
    if (!_user?.identity.userEntityRef || isAdmin(_user)) {
      return { result: AuthorizeResult.ALLOW };
    }
    // Read permissions
    if (isPermission(request.permission, catalogEntityReadPermission)) {
      return createCatalogConditionalDecision(request.permission, {
        anyOf: [
          catalogConditions.isEntityKind([
            'Domain',
            'Component',
            'System',
            'API',
            'Group',
            'User',
            'Resource',
            'Location',
          ]),
          // Templates must be owned by the user, his team, or parent group to be visible
          {
            allOf: [
              catalogConditions.isEntityKind(['Template']),
              {
                anyOf: [
                  catalogConditions.isEntityOwner(
                    _user?.identity.ownershipEntityRefs ?? [],
                  ),
                  catalogConditions.hasLabel('unrestricted'),
                ],
              },
            ],
          },
        ],
      });
    }
    // deletion check for entities owned by user
    if (
      isDeletePermission(request.permission) &&
      // Narrow type of `request.permission` to `ResourcePermission<'catalog-entity'>
      isResourcePermission(request.permission, RESOURCE_TYPE_CATALOG_ENTITY)
    ) {
      return createCatalogConditionalDecision(
        request.permission,
        catalogConditions.isEntityOwner(
          _user?.identity.ownershipEntityRefs ?? [],
        ),
      );
    }
    return { result: AuthorizeResult.ALLOW };
  }
}

export default async function createPlugin(
  env: PluginEnvironment,
): Promise<Router> {
  return await createRouter({
    config: env.config,
    logger: env.logger,
    discovery: env.discovery,
    policy: new DeletionPolicy(),
    identity: IdentityClient.create({
      discovery: env.discovery,
      issuer: await env.discovery.getExternalBaseUrl('auth'),
    }),
  });
}

Context
We are attempting to control Template access to owner groups, with exception to admin core group

Authenticated user below, also admin, has entity claims present assigned by BB Cloud resolver:

Production Graylog printing request and _user details for permission attempts. This is for user authenticated with Bitbucket Cloud, with assigned entity claims. Note Create and Delete permissions have the _user object, the READ has "undefined"

Working in local:

Your Environment
Bitbucket Cloud resolves Identity by UUID:
providerFactories: {
      bitbucket: providers.bitbucket.create({
        signIn: {
          resolver:
            providers.bitbucket.resolvers.userIdMatchingUserEntityAnnotation(),
        },
      }),
    },

app-config.production.yaml backend section:

backend:
  baseUrl: ${BACKSTAGE_URL}:${BACKSTAGE_BACKEND_PORT}
  listen:
    port: ${BACKSTAGE_BACKEND_PORT}

app-config.yaml backend section, this works fine in local dockerised tests:

permission:
  enabled: true
backend:
  baseUrl: ${BACKSTAGE_URL}:${BACKSTAGE_BACKEND_PORT}
  listen:
    port: ${BACKSTAGE_BACKEND_PORT}
  auth:
    keys:
      - secret: ${AUTH_BITBUCKET_CLIENT_SECRET}

app-config.local.yaml backend section, this works fine:

backend:
  baseUrl: http://localhost:7007
  listen:
    port: 7007

Output of yarn backstage-cli info:

local - **working fine**:
OS:   Darwin 21.2.0 - darwin/x64
node: v14.17.4
yarn: 1.22.17
cli:  0.17.2 (installed)

prod - **failing** as described:
OS:   Linux 5.10.47-linuxkit - linux/x64
node: v16.15.1
yarn: 1.22.19
cli:  0.17.2 (installed)

Dependencies, same in all envs:
  @backstage/app-defaults                                  1.0.3
  @backstage/backend-common                                0.14.0
  @backstage/backend-tasks                                 0.3.2
  @backstage/backend-test-utils                            0.1.25
  @backstage/catalog-client                                1.0.3
  @backstage/catalog-model                                 1.0.3
  @backstage/cli-common                                    0.1.9
  @backstage/cli                                           0.17.2
  @backstage/config-loader                                 1.1.2
  @backstage/config                                        1.0.1
  @backstage/core-app-api                                  1.0.3
  @backstage/core-components                               0.9.5
  @backstage/core-plugin-api                               1.0.3
  @backstage/dev-utils                                     1.0.3
  @backstage/errors                                        1.0.0
  @backstage/integration-react                             1.1.1
  @backstage/integration                                   1.2.1
  @backstage/plugin-analytics-module-ga                    0.1.17
  @backstage/plugin-api-docs                               0.8.6
  @backstage/plugin-app-backend                            0.3.33
  @backstage/plugin-auth-backend                           0.14.1
  @backstage/plugin-auth-node                              0.2.2
  @backstage/plugin-badges-backend                         0.1.27
  @backstage/plugin-badges                                 0.2.30
  @backstage/plugin-bazaar-backend                         0.1.17
  @backstage/plugin-bazaar                                 0.1.21
  @backstage/plugin-bitbucket-cloud-common                 0.1.0
  @backstage/plugin-catalog-backend-module-bitbucket-cloud 0.1.0
  @backstage/plugin-catalog-backend                        1.2.0
  @backstage/plugin-catalog-common                         1.0.3
  @backstage/plugin-catalog-graph                          0.2.18
  @backstage/plugin-catalog-import                         0.8.9
  @backstage/plugin-catalog-react                          1.1.1
  @backstage/plugin-catalog                                1.3.0
  @backstage/plugin-explore-react                          0.0.18
  @backstage/plugin-explore                                0.3.37
  @backstage/plugin-github-actions                         0.5.6
  @backstage/plugin-home                                   0.4.22
  @backstage/plugin-jenkins-backend                        0.1.23
  @backstage/plugin-jenkins-common                         0.1.5
  @backstage/plugin-jenkins                                0.7.5
  @backstage/plugin-lighthouse                             0.3.6
  @backstage/plugin-org                                    0.5.6
  @backstage/plugin-permission-backend                     0.5.8
  @backstage/plugin-permission-common                      0.6.2
  @backstage/plugin-permission-node                        0.6.2
  @backstage/plugin-permission-react                       0.4.2
  @backstage/plugin-proxy-backend                          0.2.27
  @backstage/plugin-scaffolder-backend                     1.3.0
  @backstage/plugin-scaffolder-common                      1.1.1
  @backstage/plugin-scaffolder                             1.3.0
  @backstage/plugin-search-backend-module-elasticsearch    0.1.5
  @backstage/plugin-search-backend-module-pg               0.3.4
  @backstage/plugin-search-backend-node                    0.6.2
  @backstage/plugin-search-backend                         0.5.3
  @backstage/plugin-search-common                          0.3.5
  @backstage/plugin-search-react                           0.2.1
  @backstage/plugin-search                                 0.9.0
  @backstage/plugin-sentry                                 0.3.44
  @backstage/plugin-sonarqube                              0.3.6
  @backstage/plugin-stack-overflow                         0.1.2
  @backstage/plugin-tech-radar                             0.5.13
  @backstage/plugin-techdocs-backend                       1.1.2
  @backstage/plugin-techdocs-module-addons-contrib         1.0.1
  @backstage/plugin-techdocs-node                          1.1.2
  @backstage/plugin-techdocs-react                         1.0.1
  @backstage/plugin-techdocs                               1.2.0
  @backstage/plugin-user-settings                          0.4.5
  @backstage/release-manifests                             0.0.4
  @backstage/test-utils                                    1.1.1
  @backstage/theme                                         0.2.15
  @backstage/types                                         1.0.0
  @backstage/version-bridge                                1.0.1

@Joonpark13 @vinzscam This one might be for you! Could this be a case of a permission that should have been a resource permission but isn't?

Hi @FilipSwiatczak,
when are you experiencing the issue? Is it when you are actually viewing the /create page?

hello @vinzscam, thanks for looking into this. Yes it's when viewing the /create page. In local I can see the Read permission requests having the _user passed in while in production the _user is undefined.

@FilipSwiatczak wondering if you were ever able to get passed this issue?

This implementation interests me too. Are there any updates?

I don't think it was ever addressed right? Any other takers for implementing it?

could you please share also the types.ts?

We ran into this issue, and it turned out to be related to some customizations we made to apis.ts. We added a call to createApiFactory for the catalogApiRef api, and when we deleted that change the issue went away. I'm not entirely sure what was wrong with that bit of code, but it turns out what we had intended to do was no longer necessary, so we just reverted our changes. Hopefully this is helpful to someone encountering the same issue.


Long build times when using XCTUnimplemented on type with many closures

Summary
I am introducing XCTUnimplemented for a TCA based feature but was running into long build times once I started mocking an environment with 6 closures.
Attached is a test app with a simple environment, 1 function that uses is and a test which calls that function. With the .unimplemented code the build times go from <1s to > 90s
Changing f: XCTUnimplemented("\(Self.self).f") to f: { _ in fatalError() } and leaving the other unimplemented calls in place restores the original build time
This is using 0.4.0
Project
XCTUnimplementedIssue.zip
Environment + .unimplemented
struct Environment {
    let a: () -> Bool
    let b: () -> Bool
    let c: () -> Void
    let d: () -> Void
    let e: (Bool) -> Void
    let f: (Bool) -> Void
}

extension Environment {
    static let unimplemented: Environment = .init(
        a: XCTUnimplemented("\(Self.self).a", placeholder: Bool.random()),
        b: XCTUnimplemented("\(Self.self).b", placeholder: Bool.random()),
        c: XCTUnimplemented("\(Self.self).c"),
        d: XCTUnimplemented("\(Self.self).d"),
        e: XCTUnimplemented("\(Self.self).e"),
        f: XCTUnimplemented("\(Self.self).f")
    )
}

@wattson12 Hm, the project builds for me from clean in about 3 seconds. Can you provide more info? Xcode/Swift version, etc.?

Adding my comment from slack, if I swap
: Environment = .init(

for
: Environment = Self(

The build times are back under a second

Ah, sounds like the bug's on Apple's end, then! Wanna file it with them and link here?

@wattson12 Just curious if you ever filed a bug with Apple? Just had a related report in a totally different repo.

@wattson12 Just curious if you ever filed a bug with Apple? Just had a related report in a totally different repo.

It was on my list but thanks for the reminder 😄 raised here https://github.com/apple/swift/issues/61079. Feel free to add any other context from the other report

Although Long time has passed, I met same problem with Swift 5.9(Xcode 15).
For the anyone who finds workaround, I wanna leave my experience.
For reference, my build device specification

MacBook Pro 16, M1 Max, 64GB RAM, 1TB.

BaseCode
import Foundation
import XCTestDynamicOverlay

public struct EndPoints {
    public var endPoint1: @Sendable () -> Void
    public var endPoint2: @Sendable () -> String
    public var endPoint3: @Sendable () -> Int
    public var endPoint4: @Sendable () -> Double
    public var endPoint5: @Sendable () -> Data?

    public init(
        endPoint1: @escaping @Sendable () -> Void,
        endPoint2: @escaping @Sendable () -> String,
        endPoint3: @escaping @Sendable () -> Int,
        endPoint4: @escaping @Sendable () -> Double,
        endPoint5: @escaping @Sendable () -> Data?
    ) {
        self.endPoint1 = endPoint1
        self.endPoint2 = endPoint2
        self.endPoint3 = endPoint3
        self.endPoint4 = endPoint4
        self.endPoint5 = endPoint5
    }
}

Build Time: 0.2s
When Problematic code added
extension EndPoints {
    static var value: EndPoints {
        .init(
            endPoint1: unimplemented(),
            endPoint2: unimplemented(),
            endPoint3: unimplemented(),
            endPoint4: unimplemented(),
            endPoint5: unimplemented()
        )
    }
}

Build Time: 160.8s
When both description and placeholder parameter filled
extension EndPoints {
    static var value: EndPoints {
        .init(
            endPoint1: unimplemented("endPoint1 unimplemented"),
            endPoint2: unimplemented("endPoint2 unimplemented", placeholder: ""),
            endPoint3: unimplemented("endPoint3 unimplemented", placeholder: 0),
            endPoint4: unimplemented("endPoint4 unimplemented", placeholder: 0.0),
            endPoint5: unimplemented("endPoint5 unimplemented", placeholder: nil)
        )
    }
}

Build Time: 24.4s
Even though build time is still too long, It reduces build time dramatically. In our company's project case, it shows more sharp reduction of build time.


BUG-000113739  Mil-Std 2525B chg 2 / Mil-Std 2525D implemetation:
The „Echolon“-attribute (size indicator) [B] is missing in the line

Salesforce ID: BUG-000113739
Salesforce Submitter: Margaret Maher
Salesforce Submit Date: 5/04/2018 8:51 AM
Salesforce Bug Type: Data
Salesforce Severity: Medium
Repro Data: \\<>\sf_filestore\PRD\Attachments\Defects\BUG-000113739
Steps to Repro:

Creating a "planned" line symbol, the line is solid, not dashed.
Refer to attached "p395 line symbol.docx" that shows correct symbol on page 395 and page 68 of the standard.

Work Around: (n/a)
Product: ArcGIS Pro
Functional Category: ArcGIS for Defense
Client Platform: (n/a)
Version Found: 2.0
Planned Version Fixed: (n/a)
Comment: (n/a)

LDF Comment 5/17/2018
The reported issue is with the Boundary symbol, Hierarchy: 2.X.<IP_ADDRESS>; GGGLB---****X
B: echelon is not added to the symbol
In addition, T1: unique designation 2 is not labeled in the correct location

This issue was moved to ArcGIS/military-symbology-styles#53


Add more information regarding OTA

Currently, the v2 draft only defines the $ota topic. I think the OTA mechanism from the v2 homie-esp8266 "reference implementation" should be added to the spec. That is, $implementation/ota subtopics should be mentioned.

$implementation/ota/enabled
$implementation/ota/payload

I would even suggest to move these topics from the $implementation tree to the $ota tree because IMO Homie firmware updates should not be specific to a particular implementation. Generic tools like homie-ota should be usable out-of-the-box for any Homie device, whether it runs homie-esp8266 or some other Homie runtime.

I made this choice because, if you think about it, OTA is very much implementation related. Look at homie-python. This is not actually a firmware blob, the firmware is Python code, so you'll want to update it through git, for example. A Node-RED flow might also be updatable by pulling the latest flow file from an HTTP source. If you build something on a Raspberry Pi running Docker, your firmware could be updated by pulling the latest container layers. There are many, many OTA strategies. I'm even considering removing the top-level $ota property so everything OTA-related would be in $implementation/ota, it just makes more sense.
A GUI could be compatible with all these OTA strategies based on the value of $implementation. Therefore, it could adapt it's behavior.
Do you get the point?

I did get the point :-)
I actually had other embedded systems like Arduinos or mbeds in mind. But, yes, git or docker should definitely stay outside the scope of Homie.
I think moving the $ota function to something like $implementation/ota/version would improve consistency.

@marvinroger Coming back to potentially renaming $ota into $implementation/ota/version or something. Are you still considering this? I could work on a PR for homie-esp8266 and also work on a matching PR for homie-ota. Let me know.
Another question along these lines: having used OTA a lot during the past two weeks it was cumbersome having to change firmware versions all the time just for the version number check to pass. I ended up patching my homie-ota such that it deliberately sends a bad version number. Would it be acceptable if Homie added a convention that an OTA server can force the update by appending an exclamation mark to the version number? In production, the OTA server sends a version number 1.0.0 as of now. During development, the OTA server would send 1.0.0! in order to bypass the version number check. No code change to homie-esp8266 required.

I think it would be more consistent, because not every devices require OTA, so that's also implementation related. If you agree, then yes, I'm all for it.
Good idea, I also ended up doing that, so definitely!

Actually, if we remove $ota, OTA becomes implementation specific. So the ! at the end would not be a convention in Homie, but in Homie for ESP8266 instead. And as you say, there's no change required there, so it would only be a doc tip.

I agree, consistency is desirable, one way or the other. Since you asked, I have slightly different take on this. I personally think that Homie SHOULD have a generic convention for OTA. I would rather move stuff from $implementation/ota/... to $ota/... and extend the OTA definition to other embedded devices (not the "Docker league").

Given what we've done with homie-esp8266, I guess we can close this, right? 😄

Sure.
Only, appending "!" to force the update doesn't work anymore because "!" is not a valid md5 character and we're checking for exactly 32 chars. Anyway. It's easy to invalidate an md5 ;-).


Add umask to providers/plugin.rb

To resolve the ACL issue of file /etc/rabbitmq/enabled_plugins in some harden linux images

@autumnw Could you elaborate more?

Should be related to #182

Any updates on these suggestions/changes?

Any progress on this PR?

I pulled the umask part in with #219. The mirror commit will need a bit more work.

Closing in favor for #219


Compare ordered pkg_licenses for Package objects

When Tern compares two package objects to check if they are equal, it
iterates over the dictionary for each object and compares the values for
each key in both dictionaries. One of these keys is pkg_licenses whose
value is a list of package licenses. While most package managers only
return one package license per package, Debian-based container images
often will contain more than one as the package licenses are parsed from
their copyright text. If the pkg_licenses list is not sorted,
packages that are indeed equal will return false when the same list is
compared to itself in a different order. This issue was presenting
itself when Tern ran on a Debian-based dockerfile.
This commit resolves the issue with Debian-based dockerfiles by
sorting the pkg_licenses lists before comparing them in the
Package object is_equal method.
Resolves #899
Signed-off-by: Rose Judge<EMAIL_ADDRESS>

@nishakm ok to merge?


Working on a similar project

https://github.com/freeload101/Python/blob/master/GonePhishin/GonePhishin.py

ingest URLs clicked my mail gateway
click all the things fill all the forms
diff valid test credentials compared to fake ( if not different then phishing )
determine based on keywords like o365 company name etc if the site is phishing
inspect all documents 2-3 deep

unzip files
inspect documents with 'binaries' pdf,office js,zip etc
follow all links in documents (pdf word etc repeat from start )

very cool. I will check it.


Update styles for responsive

Describe this PR
Updated styles to improve the readability of some components
Changes

Fixed the footer for 768px and 992px breakpoints

Fixed the general links for  992px breakpoint

Fixed the breadcrumb for 425px breakpoint

Fixed typo

Thanks @JigolKa we'll get the webdev team to take a look!


Source tooltip should show the full Book name

When hovering over the Source you are presented with the same information you can already see, I think it would make sense to show the full book name here instead

Is this the intended format for the sourceBooks array? Abbreviation: Full source name?

Looks like 3.3 removed the tooltip


Build failure: Gnome Warp v0.6.2 Darwin

Steps to reproduce the behavior:
On an apple silicon:
$ nix-env -iA nixpkgs.warp
I thought this was supposed to add support for macos.
Build log
warp.log
Notify maintainers
@dotlambda
@foo-dogsquared

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-darwin"`
 - host os: `Darwin 23.3.0, macOS 14.3.1`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.21.0`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Why don't you ping @Atemu???

I don't own a mac currently and I don't use the package, can't help you unfortunately.


Rust!

I've ported exit to rust fairly literally.
Open Questions

How do you feel about the use of Hermit?
from_signal returns i32 because there aren't enumerated values for signals; should we add them?

How do you feel about the use of Hermit?

I'm for it! Two questions:

If this repo has been hermitized, I think using hermit would still be optional for contributors, is that correct?
i.e. I'd be reluctant to add a dependency to something this simple
We would use hermit in two [forthcoming] places, I'm assuming?

a GitHub Action that runs the tests
a section of the README about Contributing

from_signal returns i32 because there aren't enumerated values for signals; should we add them?

How often does the list of signals change? Is it fairly portable?
I read signal(7) and it seems like the answers are "Not often" and the list is basically the same of Intel and Arm processors.
Can realtime signals cause a program to terminate? If so, does it still terminate with 128 + N?

In the Go code, I made the type an alias to int instead of a "real" enum so that the exit codes could be passed directly to os.Exit. I don't think the Go code would be much improved by enumerating the signals, but if the Rust implementation would be — I'd take your word for it! 😃

How do you feel about the use of Hermit?

I'm for it! Two questions:

If this repo has been hermitized, I think using hermit would still be optional for contributors, is that correct?
(i.e. I'd be reluctant to add a dependency to something this simple)

Hermit will be optional!

We would use hermit in two [forthcoming] places?

a GitHub Action that runs the tests
a section of the README about Contributing

We could!
I saw neither a test action nor a part in the README that said how to run things so I didn't include anything. 😅

from_signal returns i32 because there aren't enumerated values for signals; should we add them?

How often does the list of signals change? Is it fairly portable?

Fantastic question! I have no idea!
Naively I'd assume that the older UNIX signals are all set in stone, where SYSV and BSD would have diverged, and Linux would have its own too…

I read signal(7) and it seems like the answers are "Not often" and the list is basically the same of Intel and Arm processors.

Whoa, I wouldn't have expected them to change based on processor architecture!

Can realtime signals cause a program to terminate? If so, does it still terminate with 128 + N?

Realtime signals are beyond my ken.

In the Go code, I made the type an alias to int instead of a "real" enum so that the exit codes could be passed directly to os.Exit. I don't think the Go code would be much improved by enumerating the signals, but if the Rust implementation would be — I'd take your word for it! 😃

Rust is a little more strict about its type aliases. 🔝 I might add an "examples" directory?

I might add an "examples" directory?

👍  Sure! I'm OK with that!


maybe a bug in documentation

https://github.com/jashkenas/underscore/issues/2607

Coverage remained the same at 96.863% when pulling 66427f52bceb9b6d2282893e7a3ad35f7c01777b on hanzichi:master into aa715a46173874495506a06027a2f4b7e4884871 on jashkenas:master.

Thanks for the pull request @hanzichi. I'll cherry pick this over to github-pages later today.

Done. This change is now live on http://underscorejs.org


🚀 Feature: Hydration script should call the same GitHub APIs as the setup script

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
This repo has two scripts that can run to set up a repository:

"Hydration" (added in #441): initializes an existing repository
"Setup": sets up a newly cloned repository from the template

Right now, the setup script calls to GitHub APIs to set up repository labels and assorted other GitHub repository settings. But the hydration script doesn't do that...
...yet! Let's have the hydration script call to those same APIs.
Additional Info
~I think we should find a way to share code between the two scripts. It'd be annoying to have the code exist in two locations. Perhaps moving the setup script to inside src/? Note that the setup script should be written in .js files so that folks don't have to run pnpm build before running it. We'd want to enable checkJs in tsconfig.json, I think.~ #468
We'll also want at least one --skip-* flag the same way that the setup script does. I'd want to be able to run this thing without modifying the main/upstream repository!

Ah, this was resolved by #489.


More documentation

Per the README.md, there's a number of low hanging fruit to document in the root directory (e.g., python_template/fastapi/) with boilerplate code.
Setting up a toy app that demonstrates basic functionality of a framework or concept (e.g., Ansible, Flask, FastAPI, WSL) would be lovely 🙏

For Hacktoberfest participants, PR can be done any time before October. I'll hold off on approval/merging until October 1st so you get credit.

Hi, I would like to contribute to this issue. Can I get more information regarding the specifics? I would like to be assigned to it if it is up for grabs.

Hi @sivani-kesara! Due to the broad ask on my part, I'd like to keep the issue open and unassigned.
With that said, if you look through the readme and pick one of the TODOs to cover, I'll be happy to review it by this weekend.
Even writing a single unit, integration, or end to end test would be a huge help!
Let me know if you need any more guidance.


FragmentShader API should use a Factory pattern

As per discussion in https://github.com/flutter/engine/pull/28353, the FragmentShader API (currently in beta) should use a factory object per SPIR-V shader that constructs unique instances of Shader per set of uniforms.

To make it easy to produce FragmentProgram objects asynchronously, should the factory return a Future<FragmentProgram>?

Sure, that sounds good to me - do we keep the synchronous constructor as well?

I can't say. For use in widgets, there are already a number of mechanisms to help them deal easily and naturally with Futures. It depends on what other use cases might be envisioned, but the Future version would mean you wouldn't really have to explain it.
Also, I guess it depends on if the Future version was implemented in such a way that the work was done on another thread. If it is just a cover for doing asynchronous work on the same thread then we'd still potentially see some disruption of frame build times maybe?
Perhaps @chinmaygarde or @zanderso might have some insights.

I'm in general agreement with designing an API that makes it impossible to do the wrong thing (in this case doing potentially expensive work synchronous with a build() method).

Since it is hard for code that needs asynchronicity to deal with synchronous code, and since synchronous code can always use await, I think that the asynchronous solution is fine by itself until someone demonstrates a case where it can be a pain.

I think the thing to do then is to make it so that FragmentProgram objects can only be constructed via a Future<FragmentProgram> compile({required spirv: Uint8List}) constructor. Shader construction shouldn't block the UI thread, but getting a Shader object from a FragmentProgram should be fast enough to use every frame. I can draft a change.

This is done as of flutter/engine#29513


Internal Server Error when first request after session expiration is an XHR

I'm using oidc-reverse-proxy with dex and kubernetes dashboard.
When I log in into k8s dashboard (fronted by oidc-reverse-proxy) and leave my browser untouched for longer than -session-inactivity-threshold, then click on something in the dashboard, the dashboard shows nicely styled "500 Internal Server Error".
In chrome dev tools, I see:

a request to https://dashboard.auth.hardway.testk8s.lan/api/v1/login/status returning 307 Temporary Redirect
a request to https://dex.auth.hardway.testk8s.lan/dex/auth?client_id=loginapp&redirect_uri=https%3A%2F%2Fdashboard.auth.hardway.testk8s.lan%2F&response_type=code&scope=openid+email+profile+groups&state=... returning 302
a request to https://dashboard.auth.hardway.testk8s.lan/api/v1/cluster?... returning 500 Internal Server Error

In logs of oidc-reverse-proxy I see:
I0322 09:22:48.338571       1 redirect.go:30] <IP_ADDRESS>:32848 /api/v1/login/status <<< 307 Temporary Redirect - https://dex.auth.hardway.testk8s.lan/dex/auth?client_id=loginapp&redirect_uri=https%3A%2F%2Fdashboard.auth.hardway.testk8s.lan%2F&response_type=code&scope=openid+email+profile+groups&state=amkcKmXmBRQfwe7nEvPA7ttj
E0322 09:22:48.423429       1 errors.go:22] [::1]:40332 /api/v1/cluster?filterBy=&itemsPerPage=10&name=&namespace=&page=1&sortBy=d,creationTimestamp <<< 500 Internal Server Error - paths don't match: expected /, got /api/v1/cluster

and after browser reload:
E0322 09:22:54.395604       1 errors.go:22] <IP_ADDRESS>:32854 / <<< 500 Internal Server Error - state didn't match: expected amkcKmXmBRQfwe7nEvPA7ttj, got

Interestingly enough, when I let even more time pass, another reload would heal it (redirect me to dex login page and it'll work afterwards)

By the way, keycloak-proxy seems to have a similar problem:
It uses identity lifetime from dex jwt token (which is 24h per default) for cookie expiry time.
I reduced it in dex and could see stale dashboard contents(the dashboard content wouldn't change when clicking links) after token expiry. Showing 500 Internal Server Error is better than that :)


[#118] Replace assertEquals with assertEqual in tests (4-3-stable)

Addresses #118
Test suite passed on Ubuntu 24.04 / mariadb 11.6.

#'d. Mergin.


HttpProcessor issue

Unfortunately I've found serious issue with the client. The fact that HttpProcessor is shared between all endpoints, and that each HttpProcessor.createProxy() replaces metadata and urlBuilder in it, it makes the whole client broken.
Consider the following example:
public class MultiServiceDemo extends AbstractDemo {

    public void demoCallMultiService() {
        // First do something with moderations...
        openAI.moderations();
        // Now play a little with models...
        openAI.models();
        // Now go back to moderation...
        var moderationRequest = ModerationRequest.builder()
                .input(Arrays.asList(
                        "I am not sure what to think about it."))
                .build();
        var futureModeration = openAI.moderations().create(moderationRequest);
        var moderationResponse = futureModeration.join();
        moderationResponse.getResults().stream()
                .forEach(System.out::println);
    }

    public static void main(String[] args) {
        var demo = new MultiServiceDemo();
        demo.addTitleAction("Little multiservice demo", demo::demoCallMultiService);
        demo.run();
    }
}

which generates the following exception:
Exception in thread "main" java.lang.NullPointerException: Cannot invoke "io.github.sashirestela.cleverclient.metadata.Metadata$Method.getUrl()" because "methodMetadata" is null
	at io.github.sashirestela.cleverclient.http.URLBuilder.build(URLBuilder.java:22)
	at io.github.sashirestela.cleverclient.http.HttpProcessor.resolve(HttpProcessor.java:61)
	at io.github.sashirestela.cleverclient.http.HttpInvocationHandler.invoke(HttpInvocationHandler.java:35)
	at jdk.proxy2/jdk.proxy2.$Proxy4.create(Unknown Source)
	at io.github.sashirestela.openai.demo.MultiServiceDemo.demoCallMultiService(MultiServiceDemo.java:19)

There is also an issue with SSE end of stream marker: [DONE]. It is passed in a constructor CleverClient(String urlBase, List<String> headers, HttpClient httpClient, String endOfStream), but ultimately ends in a static variable: CleverClientSSE.setEndOfStream(endOfStream), which means it's not client-local anymore.

Hey @didalgolab, thanks for detecting this issue. I'm going to work on that and hopefully fix it on weekend.

Maybe in class HttpProcessor, fields metadata and urlBuilder could be moved to HttpInvocationHandler class, thus making the HttpProcessor stateless? It's just a thought...

@didalgolab I have fixed the HttpProcessor issue.
Regarding the endOfStream marker, it is a kind of configuration variable, because I don't think you could have different markers for the same API. In the future we could refactor the library to have something like a Configurator class. So, no changes for this so far.


[camera] Add iOS and Android implementations for managing auto focus.

Description
Adds iOS and Android implementations for managing auto focus. It implements the interfaces defined in #3369.
It allows for:

Locking the current auto focus settings
Checking if focus points are supported
Setting the auto focus point

This PR is currently still in draft state, as it still depends on:

#3346 Implementations for auto exposure being merged.
#3345 Platform interface changes for auto exposure being merged.
#3369 Platform interface changes for auto focus being merged and published.

Related Issues

flutter/flutter#24575
flutter/flutter#31225

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@googlebot I consent

@googlebot I consent

LGTM except for the missing licenses. However, I just noticed that the majority of the Java source files are missing the license at the top. They can probably be added in a separate PR if that's easier.

@BeMacized added the missing license headers to all JAVA files, so I decided to resolve open comments and merge the PR.

LGTM except for the missing licenses. However, I just noticed that the majority of the Java source files are missing the license at the top. They can probably be added in a separate PR if that's easier.

@BeMacized added the missing license headers to all JAVA files, so I decided to resolve open comments and merge the PR.


feat: Add impls for reference types

This PR adds implementations of AsRawXcbConnection for &T, &mut T, Box<T>, Rc<T> and Arc<T> where T: AsRawXcbConnection + ?Sized. This allows APIs that parameterize over AsRawXcbConnection to use reference types as well.
This PR also adds no_std support.

Thanks! That seems like a good idea. But I guess this also means that I should update e.g. https://github.com/rust-x-bindings/xkbcommon-rs/pull/39/commits/fb6a4c409652128aa97b6e42a384b9c39ea12e14 to take the argument by value and not by ref... :thinking:


feat: add halloween easter egg

changelog

update gatsby theme dependency
temporarily shadow feedback button to have BeeBat icon
add toggle (ghost) button to footer
add halloween classname and context to Layout.js
add a bunch of halloween style overrides for sidenav, header, page tabs and link focus

SHOUT OUT TO @aledavila AND @joshblack for pair programming with me!

Girrrrrrrl!
Love the idea of halloween Easter eggs and don't want to be the heavy.
I don't mind the bat so much even though it's not an IDL icon. But I know the orange is going to be a NO NO

Also, the "Happy Halloween" ghost button took me a while to find and because when it's not hovered, it's not aligned with the other type below it.

Then the only payoff is the header, which 1) becomes an alert color and 2) doesn't seem like enough of a punch.

Let’s NOT change the header to orange.

@jeanservaas the navigation (active) colors and focus outlines also change to orange.

We'd be happy to get a design for a halloween theme, this is just put something together at our dev hackathon yesterday. I don't quite remember what the halloween theme that Tyler and Alison did in the past looks like, but we wanted to do something like that.

Also, as far as the button, we did want it to be an easter egg, and the whole idea of that is for it to be pretty hidden, which is why we made it a ghost button. Open for any other ideas that can also be hidden somewhere.

Bat options following the grid and style guidance.

@mjabbink do we have an svg for those?

@jnm2377 Making SVGs now. Standby.

What about the Bee bat larger and leave the smile face alone?

Get some garishness and ghoulishness going

@jeanservaas see above

Quite version

@conradennis made this which would be great for the feedback face.

^ LOVEEE that monster face! @mjabbink

@jeanservaas @jnm2377 What do you think of Bee bat in Feature card and Frankenstein in the Feedback button?
Also garish and ghoulish colors.
——————

monster.svg.zip

@jnm2377 sending SVGs soon. Conrad is tinkering on the monster

bee-bat.svg.zip
monster.svg.zip

@mjabbink those get started feature cards on the homepage are images, do you know who created them and can edit them to have the orange stripe/bat? Idk how to do that/don't have any photo editing software.

Not sure what the button in the footer does?

@jeanservaas or @laurenmrice Can you provide the svg feature card images to @jnm2377

————

@mjabbink the button in the footer changes the theme to halloween. I updated it so that it doesn't have an orange nav, rather just keeps everything dark and the navigation styles are orange. I was having a hard time changing the images in the homepage feature cards when the halloween theme is set, so I just changed the bg color of one of the callouts instead. However, if we wanted the halloween images in the feature card, we can still do that, it just won't be toggle-able like the rest of the theme. I also would need those updated images. The feature cards use art direction, so we'd need an image for mobile, tablet and desktop.
You should be able to see in the preview how the halloween theme changes:

Thoughts??? cc: @jeanservaas

I misunderstood the theme switching. I was suggesting just swapping out the two images for the week. Also the face icon for feedback. I’m less into the theme switching as that big hit of orange is not great especially with the blue link.

@jeanservaas @jnm2377 Could we just swap out the tab and left nav use of Blue for Orange for the week. As well as swap the two images on the home page. I think that would be enough. It’s more controlled but fun and expressive.

So because I’m confused about what we’re doing I’m outlining what I think we’re doing

[ ] — swap monster face with wink face in feedback button

[ ] — replace both images on homepage (Start Designing and Start Developing)

I think that is all we need to do for the week. I think the theme switcher might be too invisible to users and the above will hit anyone coming to the site or engaging the feedback dialogue.

Yup agree with @mjabbink, no switcher and no orange banners.
So we're half way there. We have Frankenstein guy in the feedback, all we still need to do then is just swap those two start designing and start developing images which mike attached above. Should be a fast swap and then we can leave it up all week.
Wondering if we can get this up by tomorrow so we can have it up for the week?

@jeanservaas I can update the PR as soon as I have all 6 images. Those images are using art direction, so we need one for desktop, tablet, and mobile, for each.

@jeanservaas @jnm2377 Could we just swap out the tab and left nav use of Blue for Orange for the week. As well as swap the two images on the home page. I think that would be enough. It’s more controlled but fun and expressive.

so to be clear, we don't want to have orange on the tabs/side navs when they're active instead of blue?

If the issue is that the theme isn't obvious, I just want to point out that that's the point of an easter egg. It's what we did before for our halloween theme. We had a hidden button that activated it, if I recall correctly.

@jnm2377 yes, the idea is not to make it an Easter egg this year, but just put it out there for a week.

The desktop images were supplied above so we need tablet and mobile only.

@jnm2377
desktop.zip
getting-started-developers-tablet.png.zip
getting-started-developers-mobile copy.png.zip

@jnm2377
getting-started-designers-tablet.png.zip
getting-started-designers-mobile.png.zip

Ok. I just wanted to verify the final decision before undoing things bc it wasn't easy to do in the first place. For context, though @jeanservaas, the easter egg was not just my idea. It was something a lot of people wanted, which is why we put dev effort into it via our hackathon:

@jnm2377 @jeanservaas Whatever is the most efficient and can have an effect on the most people. The latest proposal of monster face and halloween images seems to be fast to change and will be seen by every visitor versus minimal who catch the Easter egg. Let’s proceed with this please.

@jnm2377 Looks good to me. I think this will be great. For the record I was not opposed to changing the blue to orange for the tabs and left nav select stripes. But totally cool with this and seems the most efficient.

Welp.. I already undid the theme changes (which controlled the orange tabs) bc it seemed like a no from design. So let's just merge as is. Thanks for the work on the svgs.

Seeing this in the footer on the preview build, not sure if that's what you're seeing locally

Seeing this in the footer on the preview build, not sure if that's what you're seeing locally

That's so weird! It looked fine locally.

@jeanservaas @jnm2377 Monor thing and low priority but this scale is better for mobile version of bee bat.
getting-started-developers-mobile.png.zip

@jnm2377 if you're not seeing it locally, then I'm guessing it's just a weird Vercel thing 🤷

@tw15egan phewww! ok. Yeah, I doubled checked and everything's working fine locally.

@mjabbink updated the image


Add Google Scholar parser

A simple one.

Your test is not being excuted by CI. Specify your test in .travis.yml.
Good job!

Push the commits to master branch

Pull from upstream and fix the conflict.

It seems that Travis is forbidden from accessing Google... : /

Resolved #9


Change default location based on operating system

Instead of having it default to data which would get created based on where you run it from, I will move it to the a temporary location based on the operating system.
Example:
Win
%LOCALAPPDATA%/nexus-mod-scraper/data

Linux && Darwin
~/.nexus-mods-scraper/data

Completed via https://github.com/ondrovic/nexus-mods-scraper/releases/tag/v0.0.7


Tracking has been paused.

I've been seeing this a lot of late, @gcobb321.
What exactly does this mean?  A couple of times this prevented my home from seeing my departure and never locked it down.

It means the location data received from iCloud or the iOS app is old and can not be trusted. iCloud3 keeps asking for an updated location and is not getting one. I see this when I go to a doctors office and my phone loses the gps signal, when I’m on a plane in airplane mode or in a poor cell area for a long time. The Event Log will show how old the location data is after each request.

I'm seeing this at home. It's never done this before. I work from home, my office location hasn't changed. Has this always been included?

Closing this issue with the release of v3.0


<EMAIL_ADDRESS>

acb5491

https://lab.github.com/githubtraining/github-actions:-writing-javascript-actions


Create login-indeed.java

We need a file to handle login for indeed.com.
This will be called before proceeding with any scraping.
Please accomplish the following:

Create a new file: /src/main/login/indeed.java
Include the Selenium Java packages for web scraping.
File must navigate directly to https://secure.indeed.com/auth to handle auth
Program execution MUST be headless
Program MUST reference environment variables for credentials.  You may create your own .env file for testing.

Username variable: INDEED_USERNAME
Password variable: INDEED_PASSWORD
For testing account credentials, please reach out to a Project Lead.

Program MUST return page, context, browser to main program.

150 points

I am still familiarizing myself with all the nuts and bolts, but am interested in tackling this one. Adding this as a placeholder and will return tomorrow with some questions.

Yeah!  Go for it, when you have questions, I'll be here!

@PitaFLAME Few questions when its convenient for you:
I have been taking a stab at this for a couple days whilst trying to understand how Selenium works. As far as the actual Indeed login code goes, I am starting to make some progress.
What I am wondering, is should I be able to test what I am doing or does that require the other components of the project to be working? Forgive me as I am very new to this, but I couldn't tell how to ensure I am on the right track as I write the code, or if I need to somehow link it to other parts of the scraper.
That is the main thing I am stumped on now, I am sure other things will come up in due time though!
Cheers and thanks in advance for the help


[WIP] Feature Request: Buttons & Button Set

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Currently, the button styles and design are littered over the foundation-demo codebase. Let's refactor to a singular button design.
Issue Number: N/A
What is the new behavior?
A singlular daff-button component that can be applied to a button, a an element with a aria-role="button" that applies consistent styling, animations, and interactions.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@damienwebdev want to add an expanded variant.

@xelaint What's still outstanding on this PR? Just color support?

@damienwebdev @lderrickable ready for review

:(


Key Vault ARM Template Idempotency

Are the ARM templates for Key Vaults supposed to be idempotent? I'm getting a VaultAlreadyExists when I run a template more than once.

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

ID: 293335d7-76fd-5b4a-7f32-9272673e7df5
Version Independent ID: 7cb0b0ea-636e-b250-8604-d2d280e0d88f
Content: Azure Quickstart - Create an Azure key vault and a secret by using Azure Resource Manager template
Content Source: articles/key-vault/quick-create-template.md
Service: key-vault
GitHub Login: @mumian
Microsoft Alias: jgao

I think I figured out my problem :( my Resource Group I was deploying to was in a different region than where I was deploying my Key Vault. Once I made these the same my problem went away.

Yes, it should be idempotent, but Key Vault clears the access policies every time the template is used so it's not recommended to deploy the template more than once for the same parameters.
Glad you were able to get this resolved and thank you for sharing your solution.


Custom headers to HTTP request

Hi
I would like to use your library to download images from social networks. However for example Twitter requires additional Authorization header to http requests.
I looked at the possibility to extend BaseImageDownloader however it can be specified only for global singleton config. But I need to add different headers to different requests.
Please add such functionality. Thanks in advance.

Why is this issue closed? I tested this on v1.9.4 and it still persists. The CustomImageDownloader class never gets called.

I am facing same issue - createConnection method in CustomImageDownaloder class not call.

befor call ImageLoader.getInstance().init(config),you should call ImageLoader.getInstance().**destroy() ** first.


Build failed with an exception.

I am trying to connect my app with firebase but it shows and error :

Where:
Build file 'D:\AndroidStudio\login\android\app\build.gradle' line: 24

What went wrong:
A problem occurred evaluating project ':app'.

Here my app level build.gradle
`def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
localPropertiesFile.withReader('UTF-8') { reader ->
localProperties.load(reader)
}
}
def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}
def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
flutterVersionCode = '1'
}
def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
flutterVersionName = '1.0'
}
apply plugin: 'com.android.application'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"
android {
compileSdkVersion 28
lintOptions {
    disable 'InvalidPackage'
}

defaultConfig {
    // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
    applicationId "com.mdhv.login"
    minSdkVersion 16
    targetSdkVersion 28
    versionCode flutterVersionCode.toInteger()
    versionName flutterVersionName
    testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
}

buildTypes {
    release {
        // TODO: Add your own signing config for the release build.
        // Signing with the debug keys for now, so `flutter run --release` works.
        signingConfig signingConfigs.debug
    }
}

}
flutter {
source '../..'
}
dependencies {
testImplementation 'junit:junit:4.12'
androidTestImplementation 'com.android.support.test:runner:1.0.2'
androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.2'
}
apply plugin: 'com.google.gms.google-services'`
And projeceet level build.gradle
`buildscript {
repositories {
google()
jcenter()
}
dependencies {
    classpath 'com.android.tools.build:gradle:3.2.1'
    classpath 'com.google.gms:google-services:4.3.0'
}

}
allprojects {
repositories {
google()
jcenter()
}
}
rootProject.buildDir = '../build'
subprojects {
project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
project.evaluationDependsOn(':app')
}
task clean(type: Delete) {
delete rootProject.buildDir
}
`

Hi @mdhvkothari, does this resolve your issue? https://stackoverflow.com/questions/57238933/gradle-failure-a-problem-occurred-evaluating-project-app

this is my problem too!

https://flutter.dev/docs/development/packages-and-plugins/androidx-compatibility.
This works for me


How to show(cast) local mobile photo gallery image on TV in iOS

I Have seen that code is displaying live url images on TV correctly but i can not cast my local gallery image that is available in iphone only on TV.
The code i have seen for display photo https://github.com/ConnectSDK/Simple-Photo-Share-iOS
but this code is working for live URL images. How can i display my mobile photo gallery image on tv using this SDK in iPhone. Please give me some hint or reference so that i can proceed further.
Thank you.

i did this on my ios app.
you have to host a portable http server on your device and copy the images to its folder
http://<IP_ADDRESS>/yourimage.png
then cast the url

Is your app is approved by apple ?

Is your app is approved by apple ?

https://apps.apple.com/us/app/streambeam/id1179607280

i can make the source open if u guys want? there are some password and credentials that i need to remove first

Can you give some reference for using GCDWebServer

I also seen that the app using this SDK is not eligible in apple, is it true ?

what you mean by eligable? i havent updated my app on appstore for years.

i can make the source open if u guys want? there are some password and credentials that i need to remove first

Yes Please it would be really helpful for me

what you mean by eligable? i havent updated my app on appstore for years.

I have told this by reading this thread. So that i have a doubt about it.
https://github.com/ConnectSDK/Connect-SDK-iOS/issues/212#issue-493384057

i can make the source open if u guys want? there are some password and credentials that i need to remove first

Yes Please it would be really helpful for me

ok here it is. its 5 years old, so i dont u will be able to compile. but most important is how the local webserver + casting is done. the app does whole alot more, such as casting live video from your phone to chromecast so focus on this file LocalMediaController.swift
https://github.com/San-Jeevan/StreamBeamOldSource


DataLoader does not work with a general GraphQlTester

I updated my sample projects in these days, I found some tests broke because the dataloader is not available in a none-web environment, I know the DataLoader is registered as a request scoped by default.
If possible to provide a workaround to make the DataLoader worked in the general GraphQlTester thread scoped?

I found an older working version of my test, see here, it worked well with old SpringGraphQl.
This test failed in the latest Spring GraphQL.
After upgraded to the latest, I have to remove the author, comments which are loading by DataLoader, see here(the subscription tests are refactored to a standalone test), a general GraphQlTester can not recognize the batchloaders.
In the QueryTests, I used a WebGraphQlTester, it works. And till now subscription test does not work with a WebGraphQlTester. If used it, it will throw an exception which complains there is a 404 for /graphql text/event-streams, I also did not encountered this in the early tests.
The question is if I adding some return fields such as  author, comments which requires dataloaders, how to test it in a subscription?

Sorry, but your report is still unclear. You point to 3 different test classes, each containing multiple test cases, some are "older versions", and there is related application configuration in separate files.
Yes, some things in the programming model for the testing support did change, and your tests may need to be updated. We are in a milestone phase still. I suggest you review the most recent Boot Test section.
Please, provide a more focused description. Point to a current, concrete test that fails, before and after the upgrade and we'll try and help you.

Simply, assume there is query defined as the following.
 query posts{
     allPosts{
       id
       title
       content
       author{ id name email }
       comments{ id content }
     }

The author and comments are loaded by async data loader which requires a web environment.
If executing the query in a GraphQl execution environment with a GraphQlTester as following,  it will raise exceptions with such as dataloader is null similar messages.
@SpringBootTest
@AutoConfigureGraphQlTester
@Slf4j
public class DemoApplicationTests {

    @Autowired
    GraphQlTester graphQlTester;

}

@hantsy how is that data loader defined? Could you add a code snippet showing that?

To produce the issues and  understand the problem, see, the query string like https://github.com/hantsy/spring-graphql-sample/blob/master/spring-graphql/src/test/java/com/example/demo/QueryTests.java#L34-L35, if you change the @AutoConfigureWebGraphQlTester to @AutoConfigureGraphQlTester, and used GraphQlTester, it will not work, because author and comments data loaders defiend in DataLoaders, and configured in DataLoadersConfig.

I think I understand the issue now.
Your application is registering additional DataLoader in a WebInterceptor.
The main use case of WebInterceptor is to solve cross-cutting concerns by extracting/restoring metadata from the transport request. As its Javadoc points out, is really about

extracting information from the web input (HTTP request or WS session) and add it to the ExecutionInput
extracting information from the ExecutionResult and add it to the WebOutput

In your case, this is being used to add a new DataLoaderRegistry to the ExecutionInput. Since WebInterceptor is only meant for the web transports, it's not involved with GraphQlTester, which is bound directly with GraphQlService, no transport involved.
Spring GraphQL provides interesting ways to achieve the same thing at the Controller level. It looks like your sample doesn't interact with that programming model and merely uses it as a way to set up the infrastructure, and then interact directly with GraphQL Java.
I'm repurposing this issue to help with this use case: an application should be able to contribute DataLoaderRegistrar components so that they can add DataLoaderRegistry instances to the ExecutionInput.
Your configuration could be then written like:
@Configuration
public class DataLoadersConfig {
    @Bean
    public DataLoaderRegistrar customRegistrar() {
        return (registry, context) -> {
                registry.register("authorsLoader", loader.authorsLoader());
                registry.register("commentsLoader", loader.commentsLoader());
        };
    }
}

I think the sample was created before we added batch loading support, and using a WebInterceptor was the only way to register a DataLoader, and as Brian pointed out, if there is no web layer you don't get those registrations.
For the current programming model:

Start with the section on Batching and notice the BatchLoaderRegistry, which you can use to register your batch loading functions without having to declare a DataLoader.
Then have a look at the DataLoader argument section for an example that involves an @SchemaMapping method and the BatchLoaderRegistry.
Read the @BatchMapping section to see how you can completely bypass direct use of a DataLoader for loading associations.

@rstoyanchev Thanks for your explanation,  I have used these in my examples in https://github.com/hantsy/spring-graphql-sample.  The spring-graphql example use a WebInterceptor to register data loaders. The  spring-graphql-annotations used @Controller and DataLoaderRegistry argument to register data loaders. The spring-graphql-webflux used @BatchMapping to handle the dataloader case.
I want to know if possible to make the dataloaders work for web or no-web case through a internal DataLoaderRegistrar(maybe alternative beans) if not available for none-web case, make  my first example working seemlessly, register it through a WebInterceptor(accept the dataloaders and registered in other beans available in GrapphQlService), when using GraphQlTester to test the query included a dataloader ensure it working well.  Or there is simple guide to describe how to mock the missing dataloader in DataFetcthingEnvironment in a none-web environment.

@bclozel  If registering DataLoaderRegistrar instead of WebInterceptor, I got the following when fetching all posts(with author and comments data loaders.)
See the testing log in Github actions: https://github.com/hantsy/spring-graphql-sample/runs/4319708658?check_suite_focus=true#step:5:157
{"query":"query posts{\n   allPosts{\n     id\n     title\n     content\n     author{ id name email }\n     comments{ id content }\n   }\n }"}

    < 200 OK OK
    < Content-Type: [application/json]

    {"errors":[{"message":"Cannot invoke \"org.dataloader.DataLoader.load(Object)\" because \"dataLoader\" is null","locations":[{"line":6,"column":6}],"path":["allPosts",0,"author"],"

We've discussed this, and the using the BatchLoaderRegistry approach with the Controller programming model is really the way to go here. I'm reopening this issue to revert this change.

I've reverted this change.

@hantsy, I don't think you have looked very closely at the links I provided you with.

The spring-graphql example use a WebInterceptor to register data loaders.

This was the only way before we had dedicated support. Now, in a Spring GraphQL application I'd expect you to use the BatchLoaderRegistry and this independent of annotated methods. Please refer to the docs.

The spring-graphql-annotations used @Controller and DataLoaderRegistry argument to register data loaders.

This example here is not how I'd expect it to look. Again compare to the documentation.

The spring-graphql-webflux used @BatchMapping to handle the dataloader case.

Okay, although this is not at all tied to WebFlux and could be used just as well in spring-graphql-annotations.

I want to know if possible to make the dataloaders work for web or no-web case

Yes it is possible. DataLoader's have nothing to do with Web support.


use ampersand-events and update ampersand-state

Update state devDep
Swap out ampersand-events for backbone-events-standalone

Closes #16
I'm thinking this will be a minor version bump.

:+1:


test-branch/execute-job/wrong-commit-sha-on-pr/84e1dc07-708b-436a-80f2-e7f95911a83b

PR for testing dceea64b4ddccaef5092cfd4e4fde7056c07cfe5

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


remove run_error attribute in event tracking

We would prefer to no longer collect this information.

fixed by #817


Inconsistent State After Joining

I don't know how this can happen, but I have the same file opened twice, one dirty and the other not:

Notice the problems:

After a while they sync:

Are the screenshots you show from the session Guest?
If so, did you have the same project (or specifically the ghostText.ts file open) before you joined the session?
If you're able to reproduce, can you send the Live Share logs?

Closing as stale.


Incorrect get_num_losses return value when using custom loss function

The implementation of get_num_losses in domain.py is summing up the number of output names in constraints. Yet this is not the case when using custom loss function, e.g. when using variational constraint like the case in https://docs.nvidia.com/deeplearning/modulus/modulus-sym/user_guide/intermediate/variational_example.html

Thanks for the report, I agree this is a bug. The get_num_losses in domain uses the output names of each constraint to get a total number of losses.
The custom loss, particularly in the variational constraint does not give its output variables ahead of time and the variational constraint constructor will have 0 output keys (and thus say it has zero losses) since there are no output keys of a variational data set.
Unsure about a clean fix, without requiring additional user information its not possible presently to tell that a custom loss exists in this constraint without a forward pass. Example impacted.


where is setUserData method?

i found other SDK(iOS or Android) have a method to set user data, like:nickname,gender,email,organization.
how can do the same thing in Unity sdk?

Corresponding documentation is here:
http://resources.count.ly/v1.0/docs/countly-sdk-for-android
Basically you can send a custom key-value to the server so this special keys are populated in User Profiles.

Yeah, I konw the setUserData in Android SDK.I just not find the same method in Unity SDK.
‘’‘C#
CountlyManager.Init("def31073c215e19abe9e0ae5f0a9fa08c7a59ff0");
CountlyManager.Emit()
//can i used the CountlyManager.SetUserData()
’‘’

I see :)
Unity SDK is developed by Mario. This is a community driven SDK. You can ask him or open a ticket here:
https://github.com/imkira/unity-countly

So it seems that we need to extend the Unity SDK.


ONBS not accepting .MID file extension (uppercase)

Describe the bug
ONBS won't open midi files with capital .MID file extension but will open the same file with lowercase .mid
To Reproduce
Steps to reproduce the behavior:

Drag and drop midi file with .MID file extension on the program or open directly from program.
If former, nothing happens; if latter "Error: This file cannot be opened in this program" pops up.

Expected behavior
To open the file without having to manually rename every time.
Screenshots

Had a go at fixing this recently, however, it looks like the string specifying the target file extensions for the open file dialog is somehow case-insensitive (even specifying deliberately .MID didn't work). I'll need to have a look at it again sometime.


Add New Post Issue

If I want to add new post, but if I don't select any category the article is posted anyway (without any category, there is no warning "Category cannot be empty")

Thanks @vdanelia, added at the waiting list for bugs .

this issue was fixed and i will close it for now.


dell e7470 finger print

can you please provide something to make fingerprint work under linux?

Sorry - don't have skills nor expertise with libfprint.
There is a long standing bug related to it: https://bugs.launchpad.net/ubuntu/+source/libfprint/+bug/602071


[BEAM-560] Use ThreadLocal to cache Marshaller/Unmarshaller

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[X] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[X] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

same as PR title

R: @tgroh Thanks!

Coverage decreased (-0.01%) to 69.922% when pulling 94b2ee8e52f39e8ced7186869a6147e716338d8d on vectorijk:BEAM-560 into 603f4fb0493435f4eb451f52fc5c10aa12001486 on apache:master.

Coverage decreased (-0.09%) to 69.844% when pulling 6d3bac9901ba99440a5794d8d9dd3bfb223fea63 on vectorijk:BEAM-560 into 603f4fb0493435f4eb451f52fc5c10aa12001486 on apache:master.

Coverage decreased (-0.2%) to 69.768% when pulling ba8e48dc3156593e8454025ad720274cbafa7910 on vectorijk:BEAM-560 into 603f4fb0493435f4eb451f52fc5c10aa12001486 on apache:master.

I'm going to reorder and split your commits before merging into 2 parts - refactoring (only) of the EmptyOnDeserializationThreadLocal and the follow-up use in JAXBCoder, unless you'd prefer to do it (You'll be credited on all of the commits either way)

Thanks for your contribution!

Thank you so much for your code review!


Add KONA_EV FW

Description
Add  KONA_EV FPv2
Route
db8b25e291678484|2020-10-23--16-53-13

db8b25e291678484|2020-10-23--16-53-13
route for kona EV

@ivanivra the branch that route is from has too many changes. Can you get a route from the branch from this PR? it should ideally be engaged for at least a couple minutes.

Don’t know how to switch from
Master to this
Il giorno sab 24 ott 2020 alle 20:59 Adeeb Shihadeh <
<EMAIL_ADDRESS>ha scritto:

@ivanivra https://github.com/ivanivra the branch that route is from has
too many changes. Can you get a route from the branch from this PR? it
should ideally be engaged for at least a couple minutes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/pull/2404#issuecomment-716039678,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ3OHVNE6FFNT2NS64WDZH3SMMPXTANCNFSM4S5UESYQ
.

Don’t know how to switch from Master to this Il giorno sab 24 ott 2020 alle 20:59 Adeeb Shihadeh <<EMAIL_ADDRESS>ha scritto:
…
@ivanivra https://github.com/ivanivra the branch that route is from has too many changes. Can you get a route from the branch from this PR? it should ideally be engaged for at least a couple minutes. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2404 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3OHVNE6FFNT2NS64WDZH3SMMPXTANCNFSM4S5UESYQ .

work bench
URL: https://github.com/xps-genesis/openpilot.git
Branch: patch-4

can't provide a route from patch 4 because if iswitch op doesn't work (blank black screen) and neos update with flickering screen

db8b25e291678484|2020-10-25--14-57-59
try this?


[Storage] az storage container/blob list: Fix MemoryError when service returns less num than requested

Related command

az storage container/blob list
Description

Fix #25674
While listing blobs, CLI will get page by page and end up with no continuation token(line 78) or the total length equals the request num(line 83):
https://github.com/Azure/azure-cli/blob/e0cc1bc72fb444f54ded44eeefbeff4e73473d2d/src/azure-cli/azure/cli/command_modules/storage/track2_util.py#L70-L89
But storage service won't always return the exact number we request, as documented in List Blobs (REST API):

In certain cases, the service might return fewer results than specified by maxresults, and also return a continuation token.

This will cause problem in CLI. For example, we request 4000 results and the first page returns 3999 items. Since 3999 != 4000, CLI will continue to request for another page, and the total length grows to 7999. But 7999 != 4000, so CLI keeps requesting for new page until OOM happens.
This PR aims to handle this scenario to break the loop when total length >= request num

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage

@evelyn-ys which az cli release would this fix go into?

https://github.com/Azure/azure-cli/milestone/128


use standalone columns with external link icons

Current dependencies on/for this PR:

main

PR #269   👈

This comment was auto-generated by Graphite.


Compatibility with python 3.5.3

Hi,
There is a compatibility issue with python 3.5.3.
When I try to inject classes with no __init__ function I get the following error :
AttributeError: 'method-wrapper' object has no attribute '__func__'
I traced it back to calling
typing.get_type_hints(object.__init__)

It looks like the fix : https://github.com/python/typing/pull/368
was release with python 3.5.4, since then it is hardcoded to return an empty dict.
I understand if you would not want to add a workaround for an old python version, then the readme should be updated to specify the minimum version.
Injector works with CPython 3.5+ and PyPy 3 implementing Python 3.5+.

Hi, for full clarity: can you provide a stacktrace?

Yes of course, here is a code fragment that works on python:3.5.4 docker image but not on python:3.5.3
>>> from injector import Injector
>>> class Foo:
...     pass
...
>>> Injector().get(Foo)
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/site-packages/injector/__init__.py", line 938, in create_object
    self.call_with_injection(init, self_=instance, kwargs=additional_kwargs)
  File "/usr/local/lib/python3.5/site-packages/injector/__init__.py", line 958, in call_with_injection
    bindings = get_bindings(callable)
  File "/usr/local/lib/python3.5/site-packages/injector/__init__.py", line 1099, in get_bindings
    type_hints = get_type_hints(callable, include_extras=True)
  File "/usr/local/lib/python3.5/site-packages/injector/__init__.py", line 62, in get_type_hints
    return _get_type_hints(obj, globalns, localns)
  File "/usr/local/lib/python3.5/typing.py", line 1419, in get_type_hints
    'or function.'.format(obj))
TypeError: <slot wrapper '__init__' of 'object' objects> is not a module, class, method, or function.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.5/site-packages/injector/__init__.py", line 915, in get
    result = scope_instance.get(interface, binding.provider).get(self)
  File "/usr/local/lib/python3.5/site-packages/injector/__init__.py", line 276, in get
    return injector.create_object(self._cls)
  File "/usr/local/lib/python3.5/site-packages/injector/__init__.py", line 940, in create_object
    reraise(e, CallError(instance, instance.__init__.__func__, (), additional_kwargs, e, self._stack))
AttributeError: 'method-wrapper' object has no attribute '__func__'

Cheers. If you manually replace
            self.call_with_injection(init, self_=instance, kwargs=additional_kwargs)

with
            if init is not object.__init__:
                self.call_with_injection(init, self_=instance, kwargs=additional_kwargs)

In the Injector source code will things work ok?

Yes, it seems to work fine, both on the example and my real test :)

I just released version 0.18.3 with the fix.


Allowing model builders to be extended and to be provided by ServiceLoaders

This is a small prototype based on discussions originating from https://github.com/ai-for-java/openai4j/pull/13
The approach I took here is to allow for decorating the models/builders with additional functionality without having to extend model classes or builders. I did it for a single model in this prototype - the OpenAiChatModel, but this pattern could be applied to all of the other models across Langchain4J.
That doesn't mean you couldn't extend the model classes if you wanted to use inheritance. I just try to avoid it and use composition instead.
I also added a test which shows how it would be used. Downstream libraries (like Spring Boot or Quarkus) could use this mechanism to extend/enhance with their own capabilities which aren't necessarily part of the model.
Let me know what you think @geoand / @langchain4j !
Happy to continue conversation and see where we can bring this!

I've actually thought of a way I can clean this up a bit more. I'll do that on Monday.

@geoand / @langchain4j I've done some cleanup today and simplified the usage a bit. The generics in the new classes ModelBuilder and ModelCustomizer are a bit messy, but doing that allows a nice clean build dsl.
Take a look at the ModelBuilderCustomizerTest class I added which shows how you could use it to customize the same kind of model, whereas the new tests I added in OpenAiChatModelIT show how you could use it to return a completely different model that does something different.

I like the idea!

I like the idea!

As you can see from the changes I made in the  OpenAiChatModel class, they are pretty minimal changes which can be applied to all of the model classes across Langchain4J. It also keeps langchain agnostic to what the end user wants to do.
The other thing I thought about was to add a ServiceLoader lookup in the model's builder() method, like is done in the ai-for-java project. This would allow implementers to provide their own builders completely.
WDYT @geoand / @langchain4j ?

The other thing I thought about was to add a ServiceLoader lookup in the model's builder() method, like is done in the ai-for-java project. This would allow implementers to provide their own builders completely.

Right, that's another good option and IMHO probably more applicable to the use cases we have in mind, as I don't expect normal users would have to decorate the models.

+1 from me

+1 from me

I see there is a test suite, but that suite doesn't seem to get executed when building (main.yaml has -DskipITs). Since this would be a cross-cutting set of changes, is there a way to run the test suite to verify nothing broke?

@geoand / @langchain4j I've implemented this across the board, except for in the langchain4j-bedrock module. It uses this experimental Lombok SuperBuilder thing which doesn't allow for customization. It generates its builder classes as final and generates private constructors.
The final on the class you can get around by re-declaring the class as non-final, but you can not re-declare a constructor. That blows things up.
I didn't have much of an appetite to re-write everything in this module to not use lombok, so I've left things alone. I could run delombok on all these classes and use the result of that, but it isn't very pretty and I don't know anything about this module.

Gi @edeandrea thanks a lot! Sorry for the silence, will take a look at it today

I am probably missing something, but I am not sure I understand the need for curtomizeWith() API. If the user want's to decorate the model with his/her own behaviour (e.g. disabling requests), he can do this by using a regular decorator/proxy pattern instead of implementing custom interfaces with complex generics.

I am probably missing something, but I am not sure I understand the need for curtomizeWith() API. If the user want's to decorate the model with his/her own behaviour (e.g. disabling requests), he can do this by using a regular decorator/proxy pattern instead of implementing custom interfaces with complex generics.

If they wanted to re-use the existing model class (say OpenAiChatModel for example) the only current way would be for them to create their own builder implementation and instantiate it. They wouldn't be able to use OpenAiChatModel.builder(), unless they used the service loader mechanism, which they may not want to do in certain use cases.
Using the customizeWith allows users flexibility to return the same OpenAiChatModel instance without necessarily having to override the class. It will give the user full access to the built model class using the same builder dsl that they used to build the model in the first place.
Unless maybe I'm not understanding what you mean when you say

he can do this by using a regular decorator/proxy pattern

Are you saying something like this (as a trivial example):
public class MyModel implements ChatLanguageModel {
  private final OpenAiChatModel model;
  private final boolean someParam;

  public MyModel(OpenAiChatModel model, boolean someParam) {
    this.model = model;
    this.someParam = someParam;
  }

  // Other methods implemented from ChatLanguageModel
}

OpenAiChatModel chatModel = OpenAiChatModel.builder()
  .logRequests(true)
  .logResponses(true)
  // other things
  .build();

MyModel model = new MyModel(chatModel);

// Do stuff with the model

Are you saying something like this (as a trivial example):

public class MyModel implements ChatLanguageModel {
  private final OpenAiChatModel model;
  private final boolean someParam;

  public MyModel(OpenAiChatModel model, boolean someParam) {
    this.model = model;
    this.someParam = someParam;
  }

  // Other methods implemented from ChatLanguageModel
}

OpenAiChatModel chatModel = OpenAiChatModel.builder()
  .logRequests(true)
  .logResponses(true)
  // other things
  .build();

MyModel model = new MyModel(chatModel);

// Do stuff with the model

Yes this is what I mean.
This seems to be much more usual/common/intuitive approach than using a custom customizer in the builder, at least for me.

Using the customizeWith allows users flexibility to return the same OpenAiChatModel instance without necessarily having to override the class.

The example in ModelBuilderCustomizerTest allows customization only inside the buildInternal. This example is too synthetic, I am not sure I see how users will want to use this.
        ...

        @Override
        protected ChatModel buildInternal(ChatModel customizedModel) {
            return ChatModel.builder().text(this.altText).build();
        }

The example in OpenAiChatModelIT allows for more customization, but it does not return the same type actually, it returns ExtendedChatLanguageModel. In this case wrapping OpenAiChatModel into a ExtendedChatLanguageModel would be much simpler IMHO.

I think I see your point. I'll go ahead and remove the customizer stuff.
Do you think its worthwhile to have a base interface that all builders inherit from?

Do you think its worthwhile to have a base interface that all builders inherit from?

I am not sure TBH. What do you think?
If there is no common logic, does it make sense to introduce this dependency between all of them?

I've gone ahead and and removed the common interface and also re-named this PR to more accurately describe the change its proposing.
Let me know if you'd like me to re-sync from main as well.

@edeandrea now that this is in, do you plan to utilize it in the Quarkus extension (which has been updated to the latest LangChain4j version)?

I'm trying to get this to work for Quarkus Langchain4j OpenAI and I am failing hard.
@edeandrea do you have an example of how to use this SPI in Quarkus LangChain4j?

I don't know (or care) about the details, all I want to do is add some extra fields to the Quarkus version of the ChatLanguageModel :)


Separate flag and initialization logic into different files/classes

looks good, minor cleanup only. will let you know if I have any questions after I've been working with it (I probably will).
Thanks for the polishing...


Sometimes vscode make windows high cpu to 100%

VSCode Version:1.12.2
OS Version:windows 10

Steps to Reproduce:
Sometimes vscode make windows high cpu to 100%

Perhaps when I use vscode loads the project that use web.py framework, then it will make high cpu.

Can you pls try to provide reproducible steps? Pls see https://github.com/Microsoft/vscode/wiki/Submitting-Bugs-and-Suggestions#writing-good-bug-reports-and-feature-requests. Thanks!


⚠ Elibs has degraded performance

In e7b03af, Elibs (https://elibs.unmer.ac.id) experienced degraded performance:

HTTP code: 200
Response time: 9362 ms

Resolved: Elibs performance has improved in 54d1872.


Add AutoSelection

If possible when all textboxes are filled have the "Connect To Target via CyberArk" button autoselected so that you can press enter instead of having to use a mouse. Maybe shorten the button label to something such as "Connect via CyberArk"

Added Balloon Tool Tip on Mouse Hover & Mouse Click events for below Text Fields
Username  = "Username should be in format Domain\Username. Target Username can only contain AlphaNum _ (underscore) -(hyphen)"
Target Username - "Target Username cannot be in domain\user format. Target Username can only contain AlphaNum _ (underscore) -(hyphen)"
Target Address = "Address must be in either FQDN,hostname or IP format"