CFE error when failing over EIP's between AZ's: Cannot read property 'NetworkInterfaceId' of undefined

Do you already have an issue opened with F5 support?
Customer has been working closely with myself and his SE for over 1 month on this.
Description
I have tried everything I can think of but cannot get around this error. Customer has the following requirements

EIP re-association at time of failover between AZ's.
Route table updates at time of failover

Route table updates are working if they are the only thing that is configured in CFE config. But when we add the section for Address failover, we get the error:
Wed, 08 Feb 2023 20:33:29 GMT - fine: [f5-cloud-failover] Generated Address Operations {"disassociate":[],"associate":[]}
Wed, 08 Feb 2023 20:33:29 GMT - finest: [f5-cloud-failover] Next hop address: <IP_ADDRESS>
Wed, 08 Feb 2023 20:33:29 GMT - fine: [f5-cloud-failover] Using IPv4 filtering for to get nics <IP_ADDRESS>
Wed, 08 Feb 2023 20:33:29 GMT - finest: [f5-cloud-failover] Next hop address: <IP_ADDRESS>
Wed, 08 Feb 2023 20:33:29 GMT - fine: [f5-cloud-failover] Using IPv4 filtering for to get nics <IP_ADDRESS>
Wed, 08 Feb 2023 20:33:29 GMT - severe: [f5-cloud-failover] error in _getFailoverDiscovery: TypeError: Cannot read property 'NetworkInterfaceId' of undefined
Wed, 08 Feb 2023 20:33:29 GMT - severe: [f5-cloud-failover] Cannot read property 'NetworkInterfaceId' of undefined TypeError: Cannot read property 'NetworkInterfaceId' of undefined
    at _listNics.then (/var/config/rest/iapps/f5-cloud-failover/nodejs/providers/aws/cloud.js:718:53)
    at tryCatcher (/usr/share/rest/node/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/usr/share/rest/node/node_modules/bluebird/js/release/promise.js:512:31)
    at Promise._settlePromise (/usr/share/rest/node/node_modules/bluebird/js/release/promise.js:569:18)
    at Promise._settlePromise0 (/usr/share/rest/node/node_modules/bluebird/js/release/promise.js:614:10)
    at Promise._settlePromises (/usr/share/rest/node/node_modules/bluebird/js/release/promise.js:693:18)
    at Async._drainQueue (/usr/share/rest/node/node_modules/bluebird/js/release/async.js:133:16)
    at Async._drainQueues (/usr/share/rest/node/node_modules/bluebird/js/release/async.js:143:10)
    at Immediate.Async.drainQueues (/usr/share/rest/node/node_modules/bluebird/js/release/async.js:17:14)
    at runCallback (timers.js:794:20)
    at tryOnImmediate (timers.js:752:5)
    at processImmediate [as _immediateCallback] (timers.js:729:5)
Wed, 08 Feb 2023 20:33:29 GMT - finest: [f5-cloud-failover] Uploading data to: f5cloudfailover/f5cloudfailoverstate.json

We have double-checked all tags on ENI's, EIP's, RouteTables and S3 buckets.
We have tried referencing addresses and route tables by AWS ARN and using tag-based discovery.
We have been unable to replicate the problem in another environment.

Environment information
CFE version 1.13
AWS environment (East-US-2 , 2xAZs)
BIG-IP 16.x
Severity Level
2 (High). I have been working with this customer for over a month and they are now up against a deadline to have failover address working. I will need some support with this.

Hi Michael, looking into this. We will keep you posted.

@G-gonzalezjimenez , thank you. Shyawn also reached out via Teams. He gave me an idea to troubleshoot with the customer. I will do that before I come to you again. I only submitted this issue as a last resort and don't want to waste your time, so let me get back to you on this.

@G-gonzalezjimenez , update below and a couple ideas for PM from the customer before we close.
FYI - the customer had added a 4th ENI to both devices after CFT deployment, and had configured an AWS Route to failover between these 2 ENI's. They had not correctly tagged these ENI's. Once tagged correctly, issue was resolved. But this raised a couple interesting ideas from customer.
Explanation: this was missed because if failing over Routes only between 2x ENI's, the CFE failover worked fine even when the ENI's were not tagged. Also, if failover over EIP's only between 2 ENI's, the CFE failover still worked fine, because the external ENI's that were in scope were correctly configured.
It was only when the declaration included both Routes and ENI's that the customer saw unsuccessful failovers. We now understand this is because the tags on the 4th, additional, ENI's are required for Address failover, even if the Address failover is done on External ENI's, because the IP addresses of these 4th ENI's are referenced in the CFE config Route failover section.
Ideas for PM from customer

Please note that the logging section of docs mention 6 levels of verbosity but only describes 3 of them. It's unclear which is most verbose.
Customer mentioned it could be helpful to have instructions if adding a 4th ENI after deploying the CFT.
Customer mentioned it could be useful to know that the default IAM role created by the CFT includes a condition for Route failover, where a tag must exist. For new routes or other resources added, a customer will likely receive a permissions error if they do not update this IAM role. Example condition here

Many thanks to PM guys btw, I reached out as a last resort but should have found this myself. Mike.

Closing.


Add eslint

I'm 100% in favor of this. @paulirish @denar90 do either of you have any specific eslint rule files you want to use or shall we use lighthouse's one ?

Id be happy to use lighthouses rules
On Mar 8, 2017 2:46 AM, "Pedro Silva"<EMAIL_ADDRESS>wrote:

I'm 100% in favor of this. @paulirish https://github.com/paulirish
@denar90 https://github.com/denar90 do either of you have any specific
eslint rule files you want to use or shall we use lighthouse's one
https://github.com/GoogleChrome/lighthouse/blob/master/.eslintrc.js ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/paulirish/pwmetrics/issues/43#issuecomment-285008764,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACZF6T1IQRKr-GCIWPNOAhtkApP34LCks5rjobrgaJpZM4Lw7z0
.

Since we added and merged eslint rules in https://github.com/paulirish/pwmetrics/pull/73 im closing this, feel free to reopen.


VS Code intellisense does not suggest members of js object.

VSCode Version:1.3.0 - insider
OS Version: windows 7 enterprise

Steps to Reproduce:

Create a simple object like var person = { name = "foo" , age = 24 };
write person. (type dot). Nothing happens. Press ctrl + space, it shows no suggestion.

I just want to know is this the desired behavior ? I have tried the same thing in brackets and it works perfectly fine.

Duplicate of #3465


aiocache compatible with aioredis 1.0.0b2

I created pull request #344 which implements support for aioredis 0.3.3 and 1.0.0b2.

Closed with https://github.com/argaen/aiocache/pull/344

Reopening, tests don't pass with aioredis==1.0.0b2.
If you run make ut you'll see tests failing (25 with current master status).


Do normal calculations directly when intersecting

just wanted to know what you think about it @RubenNijhuis

Looks good, saw versions of this in other repo's aswell. LGTM!


Chore: add React 17 to peerDependencies

React 17 was released on 2020-10-20 (announcement); add it to peerDependencies to avoid NPM warnings.

Thanks. Will put out a patch release tomorrow. If I forget, just ping me.

super friendly ping as we approach 20 hours from previous comment 😅

Published. Sorry for the wait.

No worries, thank you!!

@StringEpsilon thanks for the release. However, I think there was an oversight:

0.4.0 was published 8 months ago (although the commit for that release does not seem to exist in the repo). Could you please publish 0.4.1 that includes the widened peer dep range?

Shoot. I'm on it.

Thanks @StringEpsilon, much appreciated!


Fix ability of user to give endless amount of upvotes to upvoty cards

Acceptance criteria:
1- User limited to a reasonable amount of upvotes on one issue (preferably 1). No longer susceptible to individual users maliciously attacking upvoty page through app.
Time Estimate - 1.5 hours

Tackled this issue for 30-40 minutes.

The issue occurs because in incognito mode, the set-cookie part of vote response header changes every refresh. So instead of toggling your vote, their endpoint creates a new vote for that new cookie. I made a bug report on it, so perhaps they'll fix it instead of us needing to do something.


Obtain USB Blaster Programmer for FPGA

Hours for ECE Stores (E2-3350) are normally 11:00-11:30am and 1:00-1:30pm Monday to Friday.

No longer needed with the new FPGA.


feat: support named pipes

This PR contains a:

[ ] bugfix
[x] new feature
[ ] code refactor
[ ] test update
[ ] typo fix
[ ] metadata update

Motivation / Use-Case
Currently the getPort method is always called if port is set to anything but 0. This causes an issue because getPort calls const port = server.address().port which doesn't return the correct value when using named pipes.

Breaking Changes
Potentially breaks if the port number is passed as a string i.e. an environment variable. Any ideas on how we could improve this?

Additional Info
This is important for use cases involving iisnode and similar that interact with the node process via a named pipe as the port instead.

@Craga89 Please fill out the PR template correctly, and then we'll review.

@shellscape Done. Sorry about that.

Thanks. So this is in conflict with the docs https://github.com/webpack-contrib/webpack-serve/tree/master#port. port is only supposed to be a Number. Named pipes are not presently supported.

Yeah I realise that. To be honest now that I've had a think about it this
really isn't an ideal solution to the problem for us either... so let's
close this 👍 sorry for the time waste!
On Thu, 12 Jul 2018, 16:49 Andrew Powell<EMAIL_ADDRESS>wrote:

Thanks. So this is in conflict with the docs
https://github.com/webpack-contrib/webpack-serve/tree/master#port. port
is only supposed to be a Number. Named pipes are not presently supported.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webpack-contrib/webpack-serve/pull/230#issuecomment-404559469,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALeO9xO9JuEGunGKWAIyRGyBpZdK3_9ks5uF3AfgaJpZM4VMcoR
.

--

*Craig Michael Thompson *
a Front-end Web Developer / Consultant
JavaScript development, together with HTML5/CSS3 and emerging WHATWG
standards
Email: http://www.craigsworks.com
Phone: +(44) 7878 062 458 <+447878062458>
Twitter*: @craigsworks https://twitter.com/craigsworks*

No worries and no time wasted. Always willing to discuss these kinds ideas. Please feel free to ping if you feel this should be reopened at a later date.


Is it possible to send customData?

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
^3.3.2
Please ask your question
I'm migrating from VideoJS to ShalaPlayer and VideoJS has a plugin to add the cast button silvermine/videojs-chromecast, that plugin can send customData to the receiver as part of the requestData.media (ChromecastTech.js#L172), I already tried with controls.getCastProxy().setAppData({}) but it is different and I do not control the receiver web app. I see the receiver can handle it (cast_receiver.js#L786)
My questions is: Is there a way to include customData as part of the request?

The shaka.cast APIs are meant to be used on the sender side with shaka.cast used also on the receiver side.  For a receiver you don't control, you should probably use the Cast SDK directly.  In a future release, Shaka Player's shaka.cast API will be replaced with a cast button in our UI that talks directly to generic receivers.
See also #4214
Are you using the Shaka Player UI or a custom UI?  If your UI is custom, you can just choose not to use shaka.cast and talk directly to the Cast SDK.
If you are using our UI, it will use shaka.cast.CastProxy no matter what.  You could choose not to configure it with an app ID, but you would still need to hook into or replace the various UI elements to make them communicate with your receiver.
Sorry for the inconvenience!


(PUP-7176) Handle merge of scalar or mixed types as unique

This commit ensures that a 'deep' merge that encounters a mixed set of
types or non mergable types defaults to a 'first found' strategy. This
is done by simply trusting the logic in the DeepMerge gem.

good to merge when tests go green


Urgent SEO issues - Fix developer docs broken links, markup and errors from Flow.com

Context:
Flow.com is linking to various developer docs that were hardcoded, the 404 errors and markup issues they are producing are causing big issues for our SEO and need to be fixed ASAP with redirects. The broken links can be found here, upon fixing the redirect update the rows that are now returning valid pages. For more information on the issues and recommended fixes, read this.
### Tasks
- [ ] Add Redirect handler for https://developers.flow.com/architecture/node-ops/nodes/access-api-rate-limits
- [ ] https://github.com/onflow/docs/issues/638
- [ ] https://github.com/onflow/docs/issues/639
- [ ] Fix link issues in flow.com_http_4xx_client_errors_20240216

Where should we be redirecting the rate limits page to?  This page has been removed & I can't find anything about rate limits other than forum posts when they change but nothing up to date.  This seems like a gap in the docs, or do we just want to redirect to the general access node page...?
cc @gregsantos @Aliserag


General Release

Description
Fixes # remove @vercel/ncc and update README badges.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore(main): release 1.0.2

:robot: I have created a release beep boop
1.0.2 (2024-10-30)
Features

Cleanup and restructure (064ea7b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.2
:sunflower:


Xiaomi temperature sensor shows weird values

I put a temperature sensor in the freezer and it shows up as 640-645 degrees celsius instead of a negative value (temp in freezer is about -18).
I put another in the fridge, and it shows as 6 degrees (should be about 4 degrees). This I'm not certain is an error, and I'll make some tests to verify.

You're probably running an older version of deCONZ, see https://github.com/dresden-elektronik/deconz-rest-plugin/issues/292.

Great help, as always. Seems like my version was a bit old.
After updating the freezer reports as 0 degrees and the fridge still as 6 degrees. I did a api-request and got the response below:
"7": {
		"config": {
			"on": true,
			"reachable": true
		},
		"ep": 1,
		"etag": "3f20e0d8a793deac483b4119f2abd1ae",
		"manufacturername": "LUMI",
		"modelid": "lumi.sensor_ht",
		"name": "Temperature 7",
		"state": {
			"lastupdated": "2017-12-21T17:06:29",
			"temperature": 0
		},
		"swversion": "3000-0001",
		"type": "ZHATemperature",
		"uniqueid": "00:15:8d:00:01:b9:16:e6-01-0402"
	}

Do you know if deconz even reports values below zero? The sensor should have a range to about -20.

I don't know about the Mi Temperature/Humidity sensor, but the Aqara weather sensor (Temperature, Humidity, Air Pressure) did register my freezer as -18°C.
You could check the cluster info panel in the deCONZ GUI, to see whether the Measured Value (0x0000) in the Temperature measurement has been updated.


chore: prepare API bump

Summary
Subgraphs are already deployed and indexed, removing reset so we can change main API.
Will bring reset back once migrated so secondary instance will reset entirely as well.

@bonustrack if possible, can you review and merge this PR and once merged swap api.snapshot.box?


Login details

Login details did not get saved after the app is closed.

This only happens in Android in Release config.


Update gems, puppet and base container

switching to puppet 8
using ruby 3.1.4 as base container
updating all ruby gems

onceover-codequality needs a new release: https://github.com/declarativesystems/onceover-codequality/pull/28


ItemsRepeater's UniformGridLayout properties MinColumnSpacing and MinRowSpacing are flipped

Describe the bug
The effects of MinColumnSpacing and MinRowSpacing seems to be flipped. MinColumnSpacing is effecting the row spacing and MinRowSpacing the column spacing
Steps to reproduce the bug
<controls:ItemsRepeater>
    <controls:ItemsRepeater.ItemTemplate>
        <DataTemplate x:DataType="viewModels:ContentViewModel">
            <local:ContentView ViewModel="{x:Bind}" />
        </DataTemplate>
    </controls:ItemsRepeater.ItemTemplate>
    <controls:ItemsRepeater.Layout>
        <controls:UniformGridLayout Orientation="Horizontal" MinColumnSpacing="20" MinRowSpacing="10"/>
    </controls:ItemsRepeater.Layout>
</controls:ItemsRepeater>

Version Info
NuGet package version: Microsoft.UI.Xaml 2.1.190405004.1

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

October 2018 Update (17763)

April 2018 Update (17134)
Yes

Fall Creators Update (16299)

Creators Update (15063)

Anniversary Update (14393)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

@ranjeshj known issue? can you take a look?

@ranjeshj I just tried this as well and see it repro.  Looks like we're not accounting for the orientation when applying the column/row spacing.  Fixing this will be a "breaking" change, but one we should make as the current behavior isn't expected.  :)


Shuffle Page Fails on Refresh

Refreshing causes the app to lose context and consequently is unable to display followed playlists.
Cause: Shuffle page uses the user id (stored in context) to filter followed playlists. The user id is lost on refresh, and the filter throws an error.

Addressed in https://github.com/wong-alan/splotchify/commit/6681f021a6fe1a633a7b9af8a6df7df22dc2911f


More css3 mixins

I added some additional css 3 mixins:

background-clip
background-origin
background-size
font-face
opacity
text-shadow

As well as these additions to border-radius:

border-top-radius
border-top-left-radius
border-top-right-radius
border-bottom-radius
border-bottom-left-radius
border-bottom-right-radius
border-left-radius
border-right-radius

I merged in the border-radius mixins. Thanks again @travishaynes.


*akefile

https://github.com/sakuro/dotfiles/blob/2f99ac3be599b7454f3a7d671aea4b17cc34f3ff/Makefile
https://qiita.com/mizcii/items/cfbd2aa17f6b7517c37f
https://ota42y.com/blog/2020/08/02/cago-make/
https://qiita.com/yoskeoka/items/317a3afab370155b3ae8
https://github.com/crystal-lang/crystal/blob/master/Makefile
I'm not positive Rakefile for this purpose 🤔

Might be considered in #116


docs: document features

Summary
Documents majority of the API; rearranges some keyword-only params to suit disnake docs; fixes a bug with a missing default for .group deco
Closes #2

Thanks for the help ^^


v10.1 [svelte] ... pnpm story:dev error

Describe the bug
for svelte example i had 0.10.0 working but with 0.10.1 i see this error when running pnpm story:dev

7:35:06 PM [vite] Internal server error: Failed to resolve import<EMAIL_ADDRESS>from "..\..\..\..\$histoire-config-resolved". Does the file exist?
  Plugin: vite:import-analysis
  File: /$histoire-config-resolved
  1  |  export const config = {"plugins":[{"name":"builtin:tailwind-tokens"},{"name":"builtin:vanilla-support","supportPlugin":{"id":"vanilla","moduleName":"C:/Users/oneezy/Desktop/test/node_modules/.pnpm/histoire@0.10.1_vite@3.0.7/node_modules/histoire/dist/node/builtin-plugins/vanilla-support","setupFn":"setupVanilla"}},{"name":"@histoire/plugin-svelte","supportPlugin":{"id":"svelte3","moduleName":"@histoire/plugin-svelte","setupFn":"setupSvelte3"}}],"outDir":"C:/Users/oneezy/Desktop/test/.histoire/dist","storyMatch":["**/*.story.vue","**/*.story.svelte"],"storyIgnored":["**/node_modules/**","**/dist/**"],"supportMatch":[{"id":"vanilla","patterns":["**/*.js"],"pluginIds":["vanilla"]},{"id":"svelte","patterns":["**/*.svelte"],"pluginIds":["svelte3"]}],"tree":{"file":"title","order":"asc","groups":[{"id":"top","title":""}]},"theme":{"title":"Histoire","colors":{"primary":{"50":"#fff7ed","100":"#ffedd5","200":"#fed7aa","300":"#fdba74","400":"#fb923c","500":"#f97316","600":"#ea580c","700":"#c2410c","800":"#9a3412","900":"#7c2d12"},"gray":{"50":"#fafafa","100":"#f4f4f5","200":"#e4e4e7","300":"#d4d4d8","400":"#a1a1aa","500":"#71717a","600":"#52525b","700":"#3f3f46","750":"#323238","800":"#27272a","850":"#1f1f21","900":"#18181b","950":"#101012"}},"logo":{"square":"@histoire/plugin-svelte/assets/histoire-svelte.svg","light":"@histoire/plugin-svelte/assets/histoire-svelte-text.svg","dark":"@histoire/plugin-svelte/assets/histoire-svelte-text.svg"}},"responsivePresets":[{"label":"Mobile (Small)","width":320,"height":560},{"label":"Mobile (Medium)","width":360,"height":640},{"label":"Mobile (Large)","width":414,"height":896},{"label":"Tablet","width":768,"height":1024},{"label":"Laptop (Small)","width":1024,"height":null},{"label":"Laptop (Large)","width":1366,"height":null},{"label":"Desktop","width":1920,"height":null},{"label":"4K","width":3840,"height":null}],"backgroundPresets":[{"label":"Transparent","color":"transparent"},{"label":"White","color":"#fff"},{"label":"Light gray","color":"#aaa"},{"label":"Dark gray","color":"#333"},{"label":"Black","color":"#000"}],"sandboxDarkClass":"dark","routerMode":"history"}  2  |  import Logo_square from<EMAIL_ADDRESS>   |                           ^  3  |  import Logo_light from<EMAIL_ADDRESS>4  |  import Logo_dark from<EMAIL_ADDRESS>    at formatError (file:///C:/Users/oneezy/Desktop/test/node_modules/.pnpm/vite@3.0.7/node_modules/vite/dist/node/chunks/dep-0f13c890.js:35044:46)      at TransformContext.error (file:///C:/Users/oneezy/Desktop/test/node_modules/.pnpm/vite@3.0.7/node_modules/vite/dist/node/chunks/dep-0f13c890.js:35040:19)      at normalizeUrl (file:///C:/Users/oneezy/Desktop/test/node_modules/.pnpm/vite@3.0.7/node_modules/vite/dist/node/chunks/dep-0f13c890.js:40248:33)      at processTicksAndRejections (node:internal/process/task_queues:96:5)      at async TransformContext.transform (file:///C:/Users/oneezy/Desktop/test/node_modules/.pnpm/vite@3.0.7/node_modules/vite/dist/node/chunks/dep-0f13c890.js:40382:47)      at async Object.transform (file:///C:/Users/oneezy/Desktop/test/node_modules/.pnpm/vite@3.0.7/node_modules/vite/dist/node/chunks/dep-0f13c890.js:35293:30)      at async loadAndTransform (file:///C:/Users/oneezy/Desktop/test/node_modules/.pnpm/vite@3.0.7/node_modules/vite/dist/node/chunks/dep-0f13c890.js:39879:29)

Reproduction
Repro: https://github.com/oneezy/histoire-svelte
(steps inside README.md)
System Info
- Windows 11
- node v16.13.2
- pnpm 7.9.0

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

closing. appears to be fixed from this end

closed the issue to soon. I recorded a quick video to show the problem in action.
histoire-0.10.1-svelte-bug.webm


Centering tiles at MAP location makes procedure not Bayesian

As discussed during the last meeting, we should think about centering at samples from the variational distribution of location at each tile predicted from the encoder.
Currently not such a big deal since the current master branch galaxy autoencoder makes the procedure not Bayesian anyway. Maybe rethink after #339 is merged.

Technically it's still a Bayesian procedure if you're using a MAP estimate for some latent variables---we're just not able to report the uncertainty. I agree though, we should switch to using samples from the variational distribution.

Duplicate of #363


FR: Can you add Save purchased items

Is your feature request related to a problem? Please describe.
Currently when i want to sell to the shops if A im an admin it will always add the items to the inventory, i donnto want this.
and B if im a player it will not store the items. i donnot want this.
Describe the solution you'd like
Can you please add 2 Toggles in the settings
1 - That allows it to prevent admin perms to create new items on the sheet
2 - That allows the Vender / Merchant to Store and create items that have been sold to it, adding old and new items of the same name together in the quantity valile
Additional context
I run a westmarch with 10+ Gms and 160+ Players, i want to be able to make a thrift store where players can buy and sell there goods.
in addition it gets really messy curently if nothing is done

I'm not sure what you want.
So you want a toggle that a sell to merchant does not update the quantity or create the item at the merchant?
I'm not sure I understand the 2 toggles?

When a player sells an item to the merchant it would add that item to the
merchants list, and add 1 of that quantity but once that quantity hits 0
item is then removed
On Sun, 23 Jan 2022, 21:00 Lars N., @.***> wrote:

I'm not sure what you want.
So you want a toggle that a sell to merchant does not update the quantity
or create the item at the merchant?
I'm not sure I understand the 2 toggles?
—
Reply to this email directly, view it on GitHub
https://github.com/whelan/fvtt-merchant-sheet-npc/issues/94#issuecomment-1019566364,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASG2HQRI5OD4WIDL6IDE353UXRUAJANCNFSM5LXXDKDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

You mean this setting under settings for the merchant?

If your referring to a setting that already exists, if that's what it's for
it dose not seem to work. This may be easier to sit down and talk over
disord to understand, if I'm asking for somthing else but now I'm. Not
sure.
On Mon, 24 Jan 2022, 13:16 Lars N., @.***> wrote:

You mean this setting under settings for the merchant?
[image: Screen Shot 2022-01-24 at 14 15 25]
https://user-images.githubusercontent.com/75122/150789495-8ac3fc68-d0ca-4199-9b87-52fa9bfdb8f1.png
—
Reply to this email directly, view it on GitHub
https://github.com/whelan/fvtt-merchant-sheet-npc/issues/94#issuecomment-1020090271,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASG2HQRBR4O7CIUBT6WQSFLUXVGMLANCNFSM5LXXDKDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes I think that would be a great idea. What timezone are you in. I have normally time around this time. Where I can talk. But it can also in the next 6 hours, but I'm working so we have to schedule it. :)
So from 8-16 CET


[6.1] Fix the Android build

Explanation: This import was missed and the Android cross-compile complains.
Scope: A single import for C functions
Issue: None
Original PRs: #492
Risk: Practically non-existent
Testing: Passed all CI on trunk
Reviewer: @owenv
This pull from @jakepetroules was merged right after the 6.1 branch: I don't want to have to apply it for the 6.1 builds on my Android CI for the next year.

@bnbarham, need a CI run here.

@swift-ci test

need a CI run here

Sorry, I've been off and only catching up now. But looks like you can now run now 🥳!

@swift-ci test windows

@shahmishal, does something need to be setup to get the 6.1 branch of this repo tested on Windows, because my above attempts so far don't appear to do anything?

Please try again, I just fixed the issue with Windows trigger.

@swift-ci test windows

Yep, kicked off again. 👍 Thanks, Mishal, will let you know if I see any other problems.


KAFKA-294

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This pull request doesn't merge cleanly anymore, can you please close it?
Sorry for not replying to this earlier, but pull requests are not currently monitored as the Kafka project uses JIRA and Review Board for contributions. There is a plan to change this and we would like to close stale PRs before we start. Unfortunately we can't do it ourselves (a JIRA needs to be filed with Apache Infra) so your help is appreciated.
If this change is still relevant, please see http://kafka.apache.org/contributing.html. Alternatively wait until the new approach based on GitHub pull requests is in place (hopefully soon).

I believe you can close it, this is a really old PR.

@fsaintjacques, for Apache projects, no-one but Apache Infra has write permission to the repository (including pull requests and issues). So, either you (the author) must close it, or we must file a ticket asking Infra to do it.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.

This issue can be caused by a non-existing path but also a misunderstanding from the config file. A short example will help the user.


rename index.js to index.cjs for common js package

Hello @claudio-OST, do you want to take this issue?


[bug] Appkit does no longer allow to switch to an unsupported chain in the wallet with allowUnsupportedChain set to true

Link to minimal reproducible example
https://stackblitz.com/edit/stackblitz-starters-36aam4?file=context%2Findex.tsx
Summary
With web3modal v5.1.9 it was possible to use allowUnsupportedChain: true in the createWeb3Modal. Then switch to another chain in the wallet, while still using the application.
With appkit, has that option been removed? When I switch network in rabby, a modal pops up blocking the whole application. This makes our app impossible to use since we have an integrated swap widget that needs to switch to other networks during the swap. While the application surrounding it only supports one network. I can't migrate to appkit for this reason.
createAppKit({
  adapters: [wagmiAdapter],
  networks: [mainnet],
  metadata,
  projectId,
  allowUnsupportedChain: true,
});

I supplied a project on stackblitz but it can't be tested there since the application won't detect wallet switches. Need to be downloaded and run locally.
yarn install
yarn dev
See this video:
https://github.com/user-attachments/assets/e452f814-6cca-484a-bd4e-29833c537852
Looking at the old code I see it being referenced here (as in it won't display the modal if that param is set): https://github.com/reown-com/appkit/blob/V5/packages/core/src/controllers/NetworkController.ts#L121
But in the new code the whole NetworkController.ts has been removed and I can't find any reference to allowUnsupportedChain being used (when searching the whole repo). It seems the parameter is straight ignored here: https://github.com/reown-com/appkit/blob/main/packages/core/src/controllers/ChainController.ts#L222

Another issue is after switching to a supported network, which Rabby handles automatically, the modal does not close. Minor UX issue. Seen at the end of the video.
List of related npm package versions
"@reown/appkit": "^1.5.2",
"@reown/appkit-adapter-wagmi": "^1.5.2",
"@tanstack/react-query": "^5.61.4",
"next": "13.5.1",
"react": "18.2.0",
"viem": "^2.21.51",
"wagmi": "^2.13.0"

Same issue

@vanyasav @paintoshi apologies for this - @Sam-Newman can you please triage?

Just double-checking - @vanyasav @paintoshi why is disclosing more networks not an option to support the swap?
Because you'd need to disclose all networks the user might have?
Can we try your existing app somehow to confirm?

hi @paintoshi, changed this issue to a feature-request. I will be discussing with the team to try to find a good solution.

@paintoshi @vanyasav we just fixed this in  https://github.com/reown-com/appkit/pull/3447
It will be in the next release ~today or Monday


无法安装 求助

Platform
群晖ds923
Panel Version
最新
Daemon Version (Optional)
No response
Recurrence Probability
0
Bug Description
sudo su -c "wget -qO- https://script.mcsmanager.com/setup_cn.sh | bash"
回车后
提示
+----------------------------------------------------------------------
| MCSManager 安装脚本
+----------------------------------------------------------------------
[-] Architecture: x64
[+] Installing dependent software (git, tar, wget)... [!] Cannot find your package manager! You may need to install git, tar and wget manually!
[x] Failed to find git, tar and wget, please install them manually!
这怎么解决呢
有没有docker可以安装的办法

readme 里面有手动安装教程：
https://github.com/MCSManager/MCSManager?tab=readme-ov-file#linux


domains dataset, used in multiple notebooks

This PR models a solution to #509 by putting the known email domain categorization metadata in a datasets section of the bigbang module. This is now potentially documented in the autodocs that go to the RTD website. The same dataset submodule can then be used across multiple notebooks.
For this PR, some remaining work:

When I get the full IETF mailing list history loaded, will update the Full Archive Study and Classifying Email Domains notebooks
Add the autodocs for this dataset and further documentation in the appropriate part of the RTD section.

The idea is that this models what can be done with other data we package with BigBang, such as the organizational metadata collected by @nllz

See also #437

The new file datasets/domains/all_email_provider_domains.txt contains all email domains found in the IETF datatracker correct? I guess I should scan over them and add any that appear in 3GPP but are currently missing?

The new file datasets/domains/all_email_provider_domains.txt contains all email domains found in the IETF datatracker correct? I guess I should scan over them and add any that appear in 3GPP but are currently missing?

No, it does not contain all those domains.
It contains only domains that have been identified as generic email address domains.
These get combined with another list of domains of known categories (which is much shorter).
It's a good point that I should document this more carefully. I'll do that before merging.


add function for calculating the real model size

refer to #1947

I'd like to provide another implementation which is more general to me.
import logging

import torch
import torch.nn as nn

__all__ = ["flops_counter"]

def count_convNd(m, _, y):
    cin = m.in_channels
    kernel_ops = m.weight.size()[2] * m.weight.size()[3]
    ops_per_element = kernel_ops
    output_elements = y.nelement()
    total_ops = cin * output_elements * ops_per_element // m.groups  # cout x oW x oH
    m.total_ops = torch.Tensor([int(total_ops)])
    m.module_used = torch.tensor([1])

def count_linear(m, _, __):
    total_ops = m.in_features * m.out_features
    m.total_ops = torch.Tensor([int(total_ops)])
    m.module_used = torch.tensor([1])

def count_naive(m, _, __):
    m.module_used = torch.tensor([1])

register_hooks = {
    nn.Conv1d: count_convNd,
    nn.Conv2d: count_convNd,
    nn.Conv3d: count_convNd,
    nn.Linear: count_linear,
}

def flops_counter(model, input_size):
    handler_collection = []
    logger = logging.getLogger(__name__)

    def add_hooks(m_):
        if len(list(m_.children())) > 0:
            return

        m_.register_buffer('total_ops', torch.zeros(1))
        m_.register_buffer('total_params', torch.zeros(1))
        m_.register_buffer('module_used', torch.zeros(1))

        for p in m_.parameters():
            m_.total_params += torch.Tensor([p.numel()])

        m_type = type(m_)
        fn = register_hooks.get(m_type, count_naive)

        if fn is not None:
            _handler = m_.register_forward_hook(fn)
            handler_collection.append(_handler)

    def remove_buffer(m_):
        if len(list(m_.children())) > 0:
            return

        del m_.total_ops, m_.total_params, m_.module_used

    original_device = next(model.parameters()).device
    training = model.training

    model.eval()
    model.apply(add_hooks)

    assert isinstance(input_size, tuple)
    if torch.is_tensor(input_size[0]):
        x = (t.to(original_device) for t in input_size)
    else:
        x = (torch.zeros(input_size).to(original_device), )
    with torch.no_grad():
        model(*x)

    total_ops = 0
    total_params = 0
    for name, m in model.named_modules():
        if len(list(m.children())) > 0:  # skip for non-leaf module
            continue
        if not m.module_used:
            continue
        total_ops += m.total_ops
        total_params += m.total_params
        logger.debug("%s: %.2f %.2f", name, m.total_ops.item(), m.total_params.item())

    total_ops = total_ops.item()
    total_params = total_params.item()

    model.train(training).to(original_device)
    for handler in handler_collection:
        handler.remove()
    model.apply(remove_buffer)

    return total_ops, total_params

Also I'm interested in the purpose of this get_real_model_size? Is it for calculating and showing the model size after apply_fixed_architecture, or do you want to use the model size to guide your search strategy during search pharse? Because if it's the former case, it's actually a planned item for release v1.6.

@marsggbo thanks for your contribution. please check whether the pr #2420 solves your problem. close this pr for now.


Ironically localhost:3000/404 still gives me the old error page; also, cant redirect 500 errors to new 404 page =[

localhost:3000/404 gives the standard error page. I've been struggling to change this in routes.rb, maybe someone can tell me what I'm doing wrong.

Every other page gives me my 404 as expected (i'm very pleased overall, besides the exception ive mentioned above):

But also, I don't get my new error page with 500 errors. I get the follow:

How can i make it so that 500 goes to the same page as 400?

Can you share your repository?
In terms of the 404 error page, I think it's because you may have your /public/404.html page still intact. If you remove that, it should work.
Rich

Hey @richpeck
I can't share the repo bc it's for work, but I tried removing /public/404.html and it worked!
However, I still get the 500 page i showed above. I tried deleting the /public/500.html. What else do you think needs to be done?

That error you're seeing is basically the "failsafe" for Rails, meaning that it hit a 500 error, tried to show the "exception" page (provided by ExceptionHandler) and it failed again.
The issue is typically because you're trying to use exactly the same layout as your main app / 404 errors page (hence it will be using something like the same variables or similar). If this is the case, you need to limit any db calls when using the 500 error layout.

Hi,
I have also something similar, maybe someone can help me.
ruby "2.3.3"
gem 'rails', '~> 5.0.6'
gem 'exception_handler', '~><IP_ADDRESS>'
I add also config.exception_handler at development.rb:
config.exception_handler = {
  dev: true,
  layouts: {
    '404' => 'exception',
    '500' => 'exception'
  },
}

When I force a 500 error (raise ....), than I see the new error page, but why when I call just /404 I don't see also the new error page? I get just this text:
500 Internal Server Error
If you are the administrator of this website, then please read this web application's log file and/or the web server's log file to find out what went wrong.

I see on the logs that is trying to use the application.html.erb, this layout is doing some db calls and that is producing a 500 error, should not use the same layout as the 500? And why is not at least showing the 500 new error page?
I hope someone understand the problem.
Thanks!

Hi, thanks for the message.
You are correct in your assumption, that the "default" 500 error page is raised by the "application" layout's DB calls etc. In terms of why that layout is showing, you will be better doing the following:
config.exception_handler = {
  dev: true,
  layouts: {
    404 => 'exception',
    500 => 'exception'
  },
}

The "layouts" hash uses integers not strings.

I see on the logs that is trying to use the application.html.erb

If you could show me the logs for this call, I'll be in a much better position to diagnose the error.
--
Looking forward to getting this sorted for you. And thanks again for trying the gem!

Hi,
Thanks for your fast answer. The problem was using strings and not inegers 😤
Is there a way to make that /500 the 500 new error page render? Now all those static error pages paths are now 404. And how can I test the error pages without triggering an error such the one I want to test?
Thanks for this great gem!

Hmmmm I don't think so.
The behavior you're reporting is actually how it should work (/404 doesn't exist so it goes to a 404 page).
The way I generally test it is to make an obvious error in a method (.each do), which will then trigger the 500 error page. This will give me the ability to change it to my tastes as required.
Other than that, I guess I could put some functionality into the system but might be hacking it them.

I agree, that the behavior is the way it should work, but we are used to have all those urls (/404, /500 ...). What do you think if for testing we add those routes and we just raise there the http error code?
Is any easy why to do something like this?

I add the routes and build a small controller to raise the HTTP Errors, but is not working 😔
The controller try do something like this to show the error pages:
raise ActionController::RoutingError.new('Internal Server Error')

But this is not working. Is there a way I can do that?
Thanks!

Do you have a repo I could look at? Thanks

Sorry, the repo is private. But what I did is really simple.
routes.rb:
get '/500', to: 'p/http_errors#internal_server_error', as: :internal_server_error

http_errors_controller.rb:
class P::HttpErrorsController < P::BaseController
  def internal_server_error
    raise ActionController::RoutingError.new('Internal Server Error')
  end
end

Should work like this? you think it have something to do with my repo?
Thanks!

What happens if you do this...
# config/routes.rb
get '500', controller: 'p/http_errors', action: :internal_server_error, as: :internal_server_error

# config/application.rb
config.exception_handler = { dev: true }

# app/p/controllers/http_errors_controller.rb
class P::HttpErrorsController < P::BaseController
  def internal_server_error
    raise "Internal Server Error"
  end
end

The errors that Rails raises are all mapped to different "HTTP" errors. Any that are not mapped will default to the 500 server error:

Ok, now is almost working:
routes.rb
# HTTP Error Pages
#
#
get '401', controller: 'p/http_errors', action: :unauthorized, as: :unauthorized
get '404', controller: 'p/http_errors', action: :not_found, as: :not_found
get '406', controller: 'p/http_errors', action: :not_acceptable, as: :not_acceptable
get '422', controller: 'p/http_errors', action: :unprocessable_entity, as: :unprocessable_entity
get '500', controller: 'p/http_errors', action: :internal_server_error, as: :internal_server_error

http_errors_controller.rb
class P::HttpErrorsController < P::BaseController

  def unauthorized
    raise ActionController::Unauthorized
  end

  def not_found
    raise ActiveRecord::RecordNotFound
  end

  def not_acceptable
    raise ActionController::UnknownFormat
  end

  def unprocessable_entity
    raise ActionController::InvalidAuthenticityToken
  end

  def internal_server_error
    raise ActionController::InternalServerError
  end
end

401 is not working, I need to add it but I am not sure how. And maybe find a better way to raise 500.
Can you help me with that?
I think that maybe adding a setting to show error pages even when dev config is false would be a good idea.
For example for development environment, you want to see the standard rails error pages, but is nice to see the pretty ones when you use the static urls (404, 500...), so you can see them without needing to change the global config. What do you think?
BTW: Thanks for all your help and this great gem!

# config/routes.rb
{
  401: :unauthorized,
  404: :not_found,
  406: :not_acceptable,
  422: :unprocessable_entity,
  500: :internal_server_error
}.each do |k,v|
  get k.to_s, controller: 'p/http_errors', action: :show, as: v, code: v
end

# config/application.rb
config.action_dispatch.rescue_responses.merge!(
   'ApplicationController::NotAuthorized'   => :unauthorized,
   'ActionController::UnknownFormat'        => :not_acceptable,
   'ActionController::InternalServerError'  => :internal_server_error
)

# app/controllers/p/http_errors_controller.rb
class P::HttpErrorsController < P::BaseController

  def show
    raise Rails.application.config.action_dispatch.rescue_responses.key( params[:code].to_s )
  end

end

As explained in this StackOverflow answer, the class to handle the error looks to be ApplicationController::NotAuthorized .
I've not tested this, so I will have to have a closer look to see if it is the right one.
The above probably needs some sort of failsafe if the hash value is not present.
Rich

Does the log have any further details about what might be causing the 500 error?

@richpeck Did you have time to test it?
Thanks!

Not yet - had to sort something out. Will test it tonight.

Works fine for me...

Can you show me how did you implemented?
Thanks!

I just copied & pasted the code you had and the code I had.

Ok, I test it again and now works. The only one still not working is 401:
raise ActionController::Unauthorized
This still giving 500 Error page, because  ActionController::Unauthorized do not exist -> errors_rb.html

There is a way around this - using custom exceptions:

Perhaps if you defined something like the following:
#config/application.rb
config.action_dispatch.rescue_responses = {
  'ActiveRecord::DangerousAttributeError' => :unauthorized
}

This way, you're able to invoke that particular class and it will be treated as a 401 error.

Still not working... did you try and works for you?
Would be nice to have those routes working on development always, like this you can check http pages design without setting dev mode.

No I didn't test

Would be possible to have routes for each error page, but with http status 200?

yes. Just a route /500 where you get the HTTP Error page but with 200 code.

This is coming to the next version -

Thanks!


Opensource : 6.7 Explain everything about DefinitelyTyped

Description 📜
Opensource : 6.7 Explain everything about DefinitelyTyped
NOTE 👇
Please put your doc 6.Popular_Projects with the name 6.7_DefinitelyTyped.md

Domain
Open Source
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


[HOLD for payment 2024-02-26] [$500] Chat - "Hidden" is not shown on the header of avatar for "Hidden" contact

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.34-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open 1:1 DM with Hidden user<EMAIL_ADDRESS>Click on the chat header.
Click on the avatar.

Expected Result:
The header shows "Hidden"
Actual Result:
The header is blank and missing "Hidden"
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/448ff4c3-a7d5-42b1-be4a-830e9863ffc0
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010dd17add55a87ae6
Upwork Job ID:<PHONE_NUMBER>354193920
Last Price Increase: 2024-01-31
Automatic offers:

mollfpr | Reviewer | 0
ZhenjaHorbach | Contributor | 0

We think that this bug might be related to #vip-vsb
CC @quinthar

Proposal
Please re-state the problem that we are trying to solve in this issue.
Chat - "Hidden" is not shown on the header of avatar for "Hidden" contact
What is the root cause of that problem?
In the ReportAvatar component, we do not use hidden but just an empty string.
What changes do you think we should make in order to solve the problem?

headerTitle={personalDetail?.displayName ?? translate('common.hidden')}

@RohanSasne, I noticed you changed your Prospal after I posted mine.

Thank you guys for the proposals! The proposal from @ZhenjaHorbach looks the best for the fix in this issue. I agree to use getDisplayNameOrDefault to show the modal title.
🎀 👀 🎀 C+ reviewed!

Friendly bump @tylerkaraszewski

[@mollfpr] The PR that introduced the bug has been identified. Link to the PR:

https://github.com/Expensify/App/pull/28999

[@mollfpr] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

https://github.com/Expensify/App/pull/28999/files#r1503614748

[@mollfpr] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

The regression step should be enough.

[@mollfpr] Determine if we should create a regression test for this bug.
[@mollfpr] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Open 1:1 DM with Hidden user<EMAIL_ADDRESS>Click on the chat header.
Click on the avatar.
Verify that Hiden is displayed on the avatar screen
👍 or 👎

@miljakljajic I'll do the manual request in NewDot, could you give the payment summary? Thank you!

@mollfpr is owed 500 USD for their role as reviewer

@ZhenjaHorbach are you paid in upwork? I can't access the offer via that link, it throws up an error. I can send another?

@ZhenjaHorbach what's your name in upwork? I tried to search for you but it isn't coming up

Sent - let me know when you accept

Paid!

$500 approved for @mollfpr based on summary above.


[MM-25763] show email invite errors

Summary
Show if an email invite failed to be sent
Ticket Link
MM-25763
Related Pull Requests

Has server changes: https://github.com/mattermost/mattermost-server/pull/14933

Screenshots

Hey thank you for handling this. Is it possible that this error is even more explicit that the user will know that the server isn't configured?

I don't think I can tell it isn't configured, not misconfigured. The server attempts to send to the user and it fails. There are a couple of technical errors in the server, but nothing more (see the other PR). And for those errors it is a bit too technical for the average user, so I felt it is ok to group them.
If they go complain to the admin, then he can check the logs and see the reason

I checked and I see your point. I am just thinking even for the average user it will be frustrating to see an unclear error. Not saying we should handle this now as you did exactly what the ticket says. But I am thinking to have a separate ticket, both serverside and our side to show that the user needs to have the server configured 🤔 coz then they will know what to do. Just a suggestion :)

@Willyfrog What is the level of risk with this bug fix? Since we're past code complete and RC testing has started, I'd just want to make sure this won't have any risk of introducing a new regression if this gets merged into v5.25

since it needs server changes, there is definitely some risk to having some sort of regression.

/update-branch

no need to change code on webapp for guest accounts, only the server was needed to be changed

/update-branch

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

Hey @Willyfrog - I'm helping to review stale PRs. Are you still working on this?  If not, please help close this PR?  Thanks!

Heads up that as part of our efforts to move to a monorepo, we're closing out a number of older pull requests like this one to help streamline the effort.
If you'd like to preserve these changes -- even if you're not the original author! -- feel free to resubmit the pull request against the monorepo once it's ready. You can subscribe to https://github.com/mattermost/mattermost-server/issues/22420 for status updates on this effort.


Update to MAPL 2.6.5

This updates GEOSgcm to use MAPL 2.6.5. This is zero-diff except for 2-moment microphysics which isn't. For some reason cleaning up some code in MAPL to do with profiling changes the 2-moment answers. Why? Our only thought at this point is some memory state issues in 2-moment?
Still, this is an important release. It has fixes for GEOSadas-to-MAPL2, updates to move to the new Baselibs, and I think GOCART2G might need it (@gmao-esherman could let us know). @tclune might know of other reasons we want MAPL 2.6.5

For @gmao-esherman, this is essentially MAPL develop which was released today. So, if your GOCART2G work required develop, hopefully you can move it to use MAPL 2.6.5.

For @gmao-esherman, this is essentially MAPL develop which was released today. So, if your GOCART2G work required develop, hopefully you can move it to use MAPL 2.6.5.

Awesome, thank you Matt! I have been on develop because it has updates to HISTORY that GOCART2G needs. This is very useful because I can now use this tag in my components.yaml instead of just referencing develop.

This has a few things that NOAA had wanted.  None are crucial.

There is a bug in MAPL 2.6.5. Blocking this until a hotfix can be applied and 2.6.6 released

Okay I've updated this to use MAPL 2.6.6 which has the hotfix for the Debug issue. @gmao-esherman please move to use MAPL v2.6.6


Reusing previously generated index file

Description
I intend to use Fuse with Google firebase
in the doc:
const myIndex = Fuse.createIndex(options.keys, books)
const myFuse = new Fuse(books, options, myIndex)
we need to recollect all books and regenerate the index whenever a new book is added.
Describe the solution you'd like
is it possible to:

loads the old index file,
add one new book to it,
and then regenerate the new index file ?

and in this line const myFuse = new Fuse(books, options, myIndex)
why do we need all books as an argument? why not use some
information like an ID that is already accessible from myIndex ? as how they do it in Lunr.js
Thank you !

Feature already exists:
Whenever you add/remove an item, it is of course added to / removed from the index, and you can always get the newly generate index file via fuse.getIndex().

Feature already exists:
Whenever you add/remove an item via the functions add or removeAt, it will automatically also update the index. You can always get the newly generated index file via fuse.getIndex().

interesting ! you do realize there is no add or removeAt in the documentation right ? 😂

and also flexsearch.js gave this table, any rebut from fuse.js ?

Would need to dig in on how the comparison is made. They’re also using an older version of fuse, which has seen gone through several performance improvements.

@tesla-cat, I took a look at the performance comparison they're running.
I'm not seeing a fuzzy-search (with actual typos) performance check in there (does the library support it? see https://github.com/nextapps-de/flexsearch/issues/118). This performance test is always against an exact match against the following queries:
var text_queries = "gulliver;great;country;time;people;little;master;took;feet;houyhnhnms".split(";");

Notably, it seems like flexsearch.js is pre-generating a dictionary of all the words in the list, with the key being the word, and value the location where it appears. For exact-string search, this will always be an O(1) operation (i.e, map[<exact_word>]), and thus always faster than what Fuse.js does, which is fuzzy-matching.
In the test, as soon as I introduce a typo, for example:
var text_queries = "guliver" // one L

flexsearch.js returns 0 results, and still shows 500k+ op/s, while Fuse.js returns actual results.

@krisk
hey thank you for teaching me this kind of fun stuff, that table looked absurd to me at the very beginning, 1000 times better than everyone else? that must be either turing's work or an ignorant joke.
great work from you !

@krisk
and also by the way, you have the coolest github profile photo i have seen so far !

@krisk @tesla-cat
Can't I search with just the index I've previously created? Is it necessary to get all the books every-time I'm trying to use an old index? It seems odd and wasteful.


Error when querying other posts in single product (WC)

Hi,
when I query another post in a WC single product context using Timber class methods, WooCommerce (or TImber) treats it as the current product, causing a fatal error.
For example the following code is hooked into the description tab and I use it to query another post:
 $content_block_args = array(
    'name'   => 'disco-vetture',
    'post_type'   => 'fe_block',
    'no_found_rows' => true,
    'suppress_filters' => true
  );
  $content_block = Timber::get_post($content_block_args);

The Timber::get_post causes a fatal error, while if I use wordpress default WP function like get_posts() with the same args, everything works fine :-)
What version of WordPress, PHP and Timber are you using?
WordPress 5.6.1, PHP 7.3, Timber 1.18.2
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)
WP.org

Closing this issue as stale


latex_envs notebook extension?

I have a series of jupyter notebooks that I'm trying to put into a book using jupyter-book. My notebooks make heavy use of latex_envs in order to display lemmas, theorems, and proofs. However, I don't know how to make  jupyter-book build use this notebook extension.
Documentation of latex_envs instructs me to use the following command: jupyter nbconvert --to latex_envs.LenvsHTMLExporter FILE.ipynb. Is there a way to have jupyter-book send command line arguments to nbconvert?
Thanks in advance.

Hmmm - right now there's nothing in Jupyter Book that makes this straightforward, though I could imagine adding the ability to define custom nbconvert preprocessors that would be applied before the final HTML conversion.
That said, I suspect that in the long term, we need to find a better solution than an ever-growing set of nbconvert preprocessors for different kinds of books (maybe this is something that an eventual switch to pandoc would improve upon)

Unfortunately, without this feature I won't be able to use jupyter-book effectively.
What is the best course of action here? Wait until the eventual switch to pandoc, or look into implementing a custom preprocessor?

I think we could try and build in something like a "bring your own HTML Exporter" functionality, with the caveat that:

It would be "advanced users only", meaning probably no official support to help debug etc (otherwise maintaining will become a mess)
It would probably be deprecated whenever a more long-term build system is agreed upon (like Pandoc, or a different markdown -> HTML converter)

What do you think about that?

Is there any timeline on the move to pandoc? If it's not too soon, then this sounds good to me.

sounds good - no timeline right now, this is a purely-volunteer project so it's just a matter of hours in the day. We may have some resources to work on this soon but my guess is that any major changes will be several months in the future one way or another.

Sounds good - I'd like to take a stab at this (implementing "bring your own HTML Exporter"), if such contributions from outsiders are welcome. Any pointers of where to start appreciated!

Closing as this should be superceded by beta.jupyterbook.org - the new Beta build will support latex-style math via Sphinx!


The VAD.c is loss

After I download the  the C code  ,  I can only find the vad.h. The vad.c is dismiss.
What wrong with it?

nothing is wrong with it. There is no VAD.cc. The implementation is included in the VAD.h


Improved checking for homepage and usage of .Site.BaseURL

There's no need to create a variable containing .Site.BaseURL.
Also, baseURL should not contain a trailing slash, so adding one after every mention of .Site.BaseURL causes double slashes (e.g. http://example.org//myfile.html)
There's no guarantee that .Title is different from .Site.Title on a post page. Hugo v0.15 will contain .IsHome, but meanwhile eq (relURL .URL) ("/" | safeHTML) is recommended.

Hi, thank you for your improvement.
I think you're right, but your change doesn't work in my environment (hugo v0.14 - darwin x64, Mac OSX).
Here are the error logs.
If you know what the problem is, please let me know.
Thank you.
$ hugo server -w
ERROR: 2015/07/19 template: theme/partials/header.html:38:41: executing "theme/partials/header.html" at <.Site.BaseURL>: can't evaluate field Site in type interface {} in theme/partials/header.html
ERROR: 2015/07/19 template: theme/partials/header.html:38:41: executing "theme/partials/header.html" at <.Site.BaseURL>: can't evaluate field Site in type interface {} in theme/partials/header.html
ERROR: 2015/07/19 template: theme/partials/sidebar-list.html:6:41: executing "theme/partials/sidebar-list.html" at <.Site.BaseURL>: can't evaluate field Site in
 type hugolib.WeightedPages in theme/partials/sidebar-list.html
ERROR: 2015/07/19 template: theme/partials/sidebar-list.html:6:41: executing "theme/partials/sidebar-list.html" at <.Site.BaseURL>: can't evaluate field Site in
 type hugolib.WeightedPages in theme/partials/sidebar-list.html
ERROR: 2015/07/19 template: theme/partials/header.html:38:41: executing "theme/partials/header.html" at <.Site.BaseURL>: can't evaluate field Site in type inter
face {} in theme/partials/header.html
ERROR: 2015/07/19 template: theme/partials/sidebar-list.html:6:41: executing "theme/partials/sidebar-list.html" at <.Site.BaseURL>: can't evaluate field Site in
 type hugolib.WeightedPages in theme/partials/sidebar-list.html
ERROR: 2015/07/19 template: theme/partials/header.html:38:41: executing "theme/partials/header.html" at <.Site.BaseURL>: can't evaluate field Site in type inter
face {} in theme/partials/header.html
ERROR: 2015/07/19 template: theme/partials/header.html:38:41: executing "theme/partials/header.html" at <.Site.BaseURL>: can't evaluate field Site in type inter
face {} in theme/partials/header.html
ERROR: 2015/07/19 template: theme/partials/header.html:38:41: executing "theme/partials/header.html" at <.Site.BaseURL>: can't evaluate field Site in type inter
face {} in theme/partials/header.html
ERROR: 2015/07/19 template: theme/partials/sidebar-single.html:15:37: executing "theme/partials/sidebar-single.html" at <.Site.BaseURL>: can't evaluate field Si
te in type hugolib.WeightedPages in theme/partials/sidebar-single.html
ERROR: 2015/07/19 template: theme/partials/sidebar-single.html:15:37: executing "theme/partials/sidebar-single.html" at <.Site.BaseURL>: can't evaluate field Si
te in type hugolib.WeightedPages in theme/partials/sidebar-single.html
ERROR: 2015/07/19 template: theme/partials/sidebar-single.html:15:37: executing "theme/partials/sidebar-single.html" at <.Site.BaseURL>: can't evaluate field Si
te in type hugolib.WeightedPages in theme/partials/sidebar-single.html
ERROR: 2015/07/19 template: theme/partials/header.html:38:41: executing "theme/partials/header.html" at <.Site.BaseURL>: can't evaluate field Site in type interface {} in theme/partials/header.html

Ok, I'll check it out tomorrow.

I can't reproduce the errors you got. Could you please post the output of hugo -v and hugo check -v. Do you have a link to the code of the full website you use to test the theme?

I'll create a new branch with cherry picked commits and send a new pull request from that branch since you don't need the Wercker integration I added to this branch.

What code I work with is here: https://github.com/key-amb/bootie-docs-demo/ .
And I saved the outputs of hugo -v and hugo check -v at https://gist.github.com/key-amb/334f105e7001db1ada95 .
Thank you.

I think I misspelled the isHomePage variable which I fixed in #3
Could you check again now? I ran the same commands on https://github.com/SamuelDebruyn/bootie-docs-demo/tree/testing and didn't notice any errors.


Error message is unclear when compiling

I had a schema in Postgres, with a column of type DATE.
I created a struct to match this in Rust, and tried to map that DATE column to a chrono::NaiveDateTime type, like this:
#[derive(Clone, Debug, Deserialize, Eq, Ord, PartialEq, PartialOrd, Queryable)]
#[insertable_into(periods)]
#[changeset_for(periods)]
pub struct Period {
	pub id: i32,
	pub name: String,
	pub start_date: NaiveDateTime,
	pub end_date: NaiveDateTime,
}

However, this failed to compile (only when the insertable_into or changeset_for attributes were there), but the error message was very unhelpful, and didn't point to the root cause at all:
error: the trait bound `chrono::NaiveDateTime: diesel::Expression` is not satisfied

(and the error span was pointing at the #[insertable_into(...)] attribute.

Unsure how much control we have here, but I'd like the error message to either be:
`NaiveDateTime: AsExpression<types::Date>` is not satisfied

or
`NaiveDateTime: ToSql<types::Date, Pg>` is not satisfied

Those are only marginally more clear, but at least they mention the fact that the column is of type date. Ideally we could give an even better message, explicitly calling out the mistake, but I don't believe we have the ability to do that at the moment.

This is another instance of https://github.com/rust-lang/rust/issues/28894 biting us. It's picking up on the impl<T: Expression> AsExpression<T::SqlType> for T blanket impl, and complaining about Expression missing even though the underlying problem is the missing AsExpression impl. I'm trying to see if there's a way for me to move this around to circumvent the problem, but this might just need to be fixed upstream.

This should be better in rustc 1.10.


Fix typo in CONTRIBUTING.md

s/example/examples/ to match the actual directory name.
This makes the last command in that file actually run.

Great catch, thanks for the PR!


fedora 22 rpm contains the /etc/sysconfig/docker and it should not

docker version
Client:
 Version:      1.8.2
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   0a8c2e3
 Built:        Thu Sep 10 19:20:17 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.8.2
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   0a8c2e3
 Built:        Thu Sep 10 19:20:17 UTC 2015
 OS/Arch:      linux/amd64

docker info
Containers: 5
Images: 165
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.1.6-201.fc22.x86_64
Operating System: Fedora 22 (Twenty Two)
CPUs: 4
Total Memory: 15.62 GiB
Name: fedolooz.kalaari.net
ID: TMPI:T5I4:XRWA:UXE2:IQJK:RUWS:HTIZ:4MIR:TNUX:ZULS:G4R5:MZ6J
Username: looztra
Registry: https://index.docker.io/v1/

uname -a
Linux fedolooz.kalaari.net 4.1.6-201.fc22.x86_64 #1 SMP Fri Sep 4 17:49:24 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

description
Others have already reported it in the blog post new-apt-and-yum-repos. See comment-246075 and comment-257043.
According to the spec file, the init.d related files (and /etc/sysconfig/docker is identified as one of them) should not be present in 'systemd' distro related rpm.
But all the init.d files are present at least on Fedora 22 and Centos 7 (I did not check other systemd distro like fc21/rhel7)
start of the output of rpm -ql docker-engine
/etc/rc.d/init.d/docker
/etc/sysconfig/docker
/etc/udev/rules.d/80-docker.rules
/usr/bin/docker
/usr/lib/systemd/system/docker.service
/usr/lib/systemd/system/docker.socket
...

The fact that the init.d files are present should not be a problem, except that the /etc/sysconfig/docker file is not tagged as %config(noreplace), hence the file is overwritten after each package update...and that is a problem for people that did customize their docker systemd config using a drop-in that specifies /etc/sysconfig/docker as an Environment file.

it doesnt anymore, this will be fixed in the next release, you can view the rpm sepc file for proof

https://github.com/docker/docker/blob/master/hack/make/.build-rpm/docker-engine.spec#L168-L174

Thanks for reporting. Actually, I think this was resolved recently in this PR https://github.com/docker/docker/pull/16192

Oh, hmf, sorry, see @jfrazelle just commented :relaxed:

awesome! I like issue solved even before they are reported even if it make you feel like someone coming after the storm :D
Now I understand why the specs in the git repo looked good but not the actual rpm :) (could have dug harder)
Thanks!


Enhancement: Add fromKeptn label

Enhancement request: Add fromKeptn label to each ticket from this service.
Rationale: Provides the ability to show all keptn issues in a single filter.
Requested By: @robertjahn

I don't think this is necessary since #10 and #12 now provide comprehensive label support. So it's now easy to filter by any keptn_* label (or any other label you choose to send in via JSON).
The service also has the option to create any type of issue (ie. not just Tasks) by adjusting the value of the secret. Thus you could create an issue type dedicated to Keptn output. That's what I'd recommend in this case.

I agree, if keptn-project, keptn-service, keptn-stage labels are being added then we don't also need "fromKeptn"


requestOauthAccessToken - not working due to mismatch with public api

In Casdoor.kt the request to get the access token (in requestOauthAccessToken) returns an error because the hardcoded path is wrong
The problematic row:
var httpUrl = "${config.endpoint}/login/oauth/access_token".toHttpUrlOrNull()
should be
var httpUrl = "${config.endpoint}api/login/oauth/access_token".toHttpUrlOrNull()
The word "api" is missing from several urls in the file

@yuvalSure can you make a PR to fix it?

Hi, I will be glad to
From: Eric Luo @.>
Date: Thursday, 11 July 2024 at 18:49
To: casdoor/casdoor-android-sdk @.>
Cc: Yuval Snir @.>, Mention @.>
Subject: Re: [casdoor/casdoor-android-sdk] requestOauthAccessToken - not working due to mismatch with public api (Issue #20)
@yuvalSurehttps://github.com/yuvalSure can you make a PR to fix it?
—
Reply to this email directly, view it on GitHubhttps://github.com/casdoor/casdoor-android-sdk/issues/20#issuecomment-2223295043, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AP77MEITM3H3TURRN2R2MELZL2SRFAVCNFSM6AAAAABKW4EAQKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRTGI4TKMBUGM.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks a lot. When it’ll be available through the maven repository?
From: Eric Luo @.>
Date: Monday, 15 July 2024 at 18:32
To: casdoor/casdoor-android-sdk @.>
Cc: Yuval Snir @.>, Mention @.>
Subject: Re: [casdoor/casdoor-android-sdk] requestOauthAccessToken - not working due to mismatch with public api (Issue #20)
Closed #20https://github.com/casdoor/casdoor-android-sdk/issues/20 as completed via #21https://github.com/casdoor/casdoor-android-sdk/pull/21.
—
Reply to this email directly, view it on GitHubhttps://github.com/casdoor/casdoor-android-sdk/issues/20#event-13507456232, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AP77MEKX6M74OK3QLB3GVULZMPTPLAVCNFSM6AAAAABKW4EAQKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGUYDONBVGYZDGMQ.
You are receiving this because you were mentioned.Message ID: @.***>


Allow templating of webhook requests

The problem faced currently?
Currently the webhook security rule has a fixed payload and authorisation header. This can cause problems when we are trying to chain this request with another remoteservice.
How can we solve it?
Accept a template field along with a claims field in the security rule object. I think the rest is pretty self explainatory.

If you want this feature to be implemented, give it a thumbs up reaction, so that we can determine which features are important to you.
👍

Done in Gateway

Done in mission control as well!


Partitioner skeleton

This all looks pretty good to me. The directory structure is familiar, and the argument checking seems fine.
While it's not a thing right now, keep in mind that in your __init__.py files you'll need to explicitly export any dunderscore/"private" functions that you want to create unit tests for.


Remove target_args from Array.find_index()

Because closure can't access this_arg.
Referencing issue #475 .

Thanks!


It's not clear from xrdp/xrdp.ini how to set up audio on a BSD or linux system

xrdp version
<IP_ADDRESS>
Detailed xrdp version, build options
n/a
Operating system & version
FreeBSD 13.2
Installation method
Doesn't matter
Which backend do you use?
No response
What desktop environment do you use?
No response
Environment xrdp running on
No response
What's your client?
No response
Area(s) with issue?
No response
Steps to reproduce
xrdp/xrdp.ini allows to set up the remote display+keyboard+mouse, but it isn't clear how to redirect the audio.
The words "audio" or "sound" don't appear in this file.
✔ Expected Behavior
Being able to use sound over RDP.
❌ Actual Behavior
Unclear.
Anything else?
No response

Hello yuri,
you don't need to configure xrdp.ini to enable audio redirection but PulseAudio is required.
audio/pulseaudio-module-xrdp will be installed together if you installed xrdp from pkg/ports. You will see 2 virtual sound cards xrdp-sink and xrdp-source. xrdp-source is for microphone, sound redirection from client to server.
Play some sound specifying the output device to xrdp-sink. Then audio redirection will work.
Pipewire is in progress. I just added audio/pipewire-module-xrdp recently but I am still not familiar with pipewire.
$  pacmd list-cards
2 card(s) available.
    index: 2
        name: <xrdp.sink>
        driver: <module-xrdp-sink.c>
        properties:
                device.icon_name = "audio-card"
        profiles:
                output:xrdp: xrdp audio output (priority 10, available: unknown)
        active profile: <output:xrdp>
        sinks:
                xrdp-sink/#3: remote audio output
        sources:
                xrdp-sink.monitor/#4: Monitor of remote audio output
        ports:
                xrdp-output: xrdp output (priority 0, latency offset 0 usec, available: yes)
                        properties:

    index: 3
        name: <xrdp.source>
        driver: <module-xrdp-source.c>
        properties:
                device.icon_name = "audio-card"
        profiles:
                input:xrdp: xrdp audio input (priority 10, available: unknown)
        active profile: <input:xrdp>
        sources:
                xrdp-source/#5: remote audio input
        ports:
                xrdp-input: xrdp input (priority 0, latency offset 0 usec, available: yes)
                        properties:


split raw data into own sub-module

nah it's fine. Need to improve all import in #44
if need to split into own sub-module, then will do there

frames put into its own module in #69
raw data is a type of frame so it's all good
could abstract further if benefit


⚠ Bofrost.fr has degraded performance

In f823b6c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1711 ms

Resolved: Bofrost.fr performance has improved in 7aeaa2a after 8 minutes.


Support page for the partner center is broken (please redirect)

I want to publish an addin via the partner center, but I am having a specific issue there for which I would like to contact the support of the partner center. The problem is that the support page of the partner center is broken, there is just an endlessly spinning spinner in both Firefox and Chrome, see the screenshot below. The console logs this, I am not sure whether it is of relevance:
action-center.ts:67 cux-ac Error acParentErrors :
action-center.ts:67 cux-ac Error acParentErrors : retrieving_notifications

There is no other way to contact support or anyone else from the partner center, that is why I am opening the issue here, it seems like the only chance to establish contact. I would like to ask you to forward this to your colleagues from the partner center so that they can fix the problem with the support page. (If possible, it would be great if you could further ask them to contact me at<EMAIL_ADDRESS>about my actual support request.)

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

ID: f3054088-7ada-7241-39fe-b6569eef399e
Version Independent ID: ff3ebc2e-6984-c246-c8ac-da19317aa94b
Content: Microsoft 365 Store step-by-step submission guide
Content Source: articles/marketplace/add-in-submission-guide.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @mingshen-ms
Microsoft Alias: mingshen

@davidpomerenke
Thank you for reaching out to us. We will investigate this further and will get back to you.

@davidpomerenke Partner can create a support case from Partner Center portal: https://docs.microsoft.com/en-us/partner-center/report-problems-with-partner-center and https://docs.microsoft.com/en-us/partner-center/support-from-microsoft

@davidpomerenke Checking in to see if the above answer helped. Let us know if there are still any additional issues we can help with.

On your first link there was a link to recommended content on verifying my company profile, which accidentally helped me to solve my original issue 😃 And with the company profile, the support page also seems to work. So I am satisfied now.
However, the problem that the support page is broken when there is no verified company profile is probably not resolved and may still affect other new customers. Note that most of them will completely fail to contact Microsoft in any way, just as I did, until I had the idea of (ab)using Github issues for it, so I would recommend to you to investigate it and fix it, even if there are no complaints by other customers.


Parsing of kubeconfig is more strict than client-go

Current and expected behavior
I noticed that Kubeconfig::read() is a bit more strict than client-go's when parsing the kubeconfig. For example:
In client-go, it's possible to have a context without a cluster or user. It won't actually work if you try to use that context, but it doesn't fail to parse it, meaning that users can use the other contexts that have all the required fields.
These are the differences I found:

the contexts.context is optional in client-go, mandatory in kube-rs.
the contexts.context.name is optional in client-go, mandatory in kube-rs.
the contexts.context.user is optional in client-go, mandatory in kube-rs.
the users.user.auth-provider.config is optional in client-go, mandatory in kube-rs.
the users.user.auth-provider.config can be set to empty, ~ or null in client-go, but not in kube-rs
there might be more though

Possible solution
Relax the parsing of kubeconfig to match client-go as much as possible. To avoid further breaking changes, ideally if those fields are missing on kubeconfig, kube-rs would default it to empty strings / empty hash maps.
Additional context
No response
Environment
Client Version: v1.25.3
Kustomize Version: v4.5.7
Server Version: v1.22.8
WARNING: version difference between client (1.25) and server (1.22) exceeds the supported minor version skew of +/-1
Configuration and features
kube = { version = "0.76.0", features = ["client", "runtime", "ws", "gzip"]  }
k8s-openapi = { version = "0.16.0", features = ["v1_24"] }

Affected crates
kube-client
Would you like to work on fixing this bug?
yes

I think that makes sense. We did add Kubeconfig::default to give out an empty config (which kubectl happily parses).
Not quite sure where we make all of that stuff there mandatory, but I see at least some missing #[serde(default)] annotations on maps. Feel free to do a PR on this!
NB: There are possibly some finer points here about what "should consider parseable" and what we "should consider valid". If we are going to use it internally, we probably need it to be usable, but if we are just going to ::merge it, then we should definitely be as flexible as client-go.


Does this project completed to read all types of STEP entities ?

I am specifically want to read dimensions , annotations information from STEP file. Have you done that ?
Regards,
Onkar

No, this project isn't really in any sort of a usable state, it's mainly for my own entertainment.  Pinned issue #15 addresses this.


Angular2 PreRendering not working correctly when base href set to subdirectory

I'm trying to make PreRendering work with an Angular2 app based in a subdirectory.
Steps to recreate:

Created a new project from the Microsoft SPA Templates dotnet new angular -n DashboardTest (dotnet new --install Microsoft.AspNetCore.SpaTemplates::* to install the templates package)
Removed all routing from Startup.cs and replaced with app.UseMvc();. This means that only routes I've explicitly defined will be mapped.
Added [Route("/dashboard/{*angularRoute}")] attribute to the Index() action on the HomeController to catch all requests to my chosen subdirectory of /dashboard and pre-render the angular app (the Home/Index view already contains <app asp-prerender-module="ClientApp/dist/main-server">Loading...</app> for pre-rendering.)
Updated the base href tag in _Layout.cshtml to <base href="/dashboard" /> to correctly configure the angular app to treat /dashboard as it's root.

On the surface everything appears to work beautifully, however behind the scenes the pre-rendering has broken. To prove this, load view-source:http://localhost:5000/dashboard/counter (in Chrome) - you'll see the rendered output is actually the /home route, not the /counter route as expected.
Suggested fix:
Add public string BaseHref { get; set; } and unencodedPathAndQuery = !string.IsNullOrEmpty(BaseHref) && unencodedPathAndQuery.StartsWith(BaseHref) ? unencodedPathAndQuery.Substring(BaseHref.Length) : unencodedPathAndQuery; to PrerenderTagHelper

This is the fix that is already in the Dev branch that should address the issue:
https://github.com/aspnet/JavaScriptServices/issues/961#issuecomment-303067929

#961 appears to solve a problem with the Angular app root directory - this isn't an issue for me. It's the pre-rendering that's not working correctly.

There are indeed some known issues with workarounds. Please see #488 and #571.
Closing this one as duplicate.


Cecret update

Minor release, BUT! there are changes to the summary file. There are also changes to the config file template, as this is no longer called in the workflow.
Of note, the Github Actions (GA) to look for new container versions is working. This dramatically increases the amount of time needed to keep current as well as dramatically increases the notifications that this workflow receives.
This means I should finally focus on this workflow and simplify it's results channels.
Most notable changes are internal, so here's hoping nothing breaks for the end users.
In this version:

All container designations have moved into each process in the module files
UPHL ICA options have been added to each process in the modules files. These are commented out for the general user
There are no more 'latest' container tags due to GA actions ability to notify of most relevant changes every Tuesday
The results channels have been simplified to for_multiqc, for_summary, and for_version
The summary file is created in a different way
Fastas and multifastas can be read in via the sample sheet
There is now ASCII art. You're welcome
Updates to the wiki

Should take care of https://github.com/UPHL-BioNGS/Cecret/issues/142

Should fix https://github.com/UPHL-BioNGS/Cecret/issues/145

As un update: this looks like it's ready to go. It's going through the list of tests locally, and if those pass this will get turned into a real PR from a draft.


Feature/#21689

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Adding zoom button

This PR fixes #18380
Before:

Now:

Tooltip:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14816

Relevant Lines:
29137

💛  - Coveralls


[TF 2.0 API Docs] tf.keras.metrics.AUC

Highlight codes in the description

I signed it!

Closing this PR as this not against master, please open a new PR against master
CC @mihaimaruseac


Explorer listing shouldn't need to be approved before a project can manually be enabled on discover page via "Display app on discover page"

Developers should be able to display any app on the Discover page, without needing the explorer listing to be approved.

SekT


Installer Automatically Selects Drive Not There

It always go to the E:\ drive and I don't know how to reset the installer
Does this issue occur when all extensions are disabled?: No, as it takes place in the installer

VS Code Version: 1.89.1
OS Version: Windows 11

Is there a solution to this or what?


Creating second session obbject with app Root sends WinAppDriver in tailspin

Hi,
I have WinAppDriver 1.1 installed. I'm using C#.
I create a session in my test to open and manipulate Alarms app. It works fine.
Until I go to clock screen and right click a tile to delete it.
I created a second session for this purpose. But the WinAppDriver console starts showing 404 not found error many times.
Exact text is given below:
GET /wd/hub/status HTTP/1.1
Host: localhost:4723
HTTP/1.1 404 Not Found
Code:

DesiredCapabilities capDesktop = new DesiredCapabilities();
            capDesktop.SetCapability("app", "Root");

            WindowsDriver<WindowsElement> sessionDesktop = new WindowsDriver<WindowsElement>(capDesktop);

Update:
I have two monitors. I tried to create the desktop session standalone, it also gives same result.

Update:
I started the WinAppDriver with a port number other than default and tried again, this time it worked. But I'm still curious to know what was wrong with the first time?

Note: I also updated Appium before trying again. Maybe that's making a difference.
npm up -g appium


[PTRun][Docs] Updated Third party plugins to include UnicodeInput

Summary of the Pull Request
Adds a new plugin, UnicodeInput, (developed by myself) to the third-party plugins list for PowerToys Run.
Detailed Description of the Pull Request / Additional comments
This plugin allows for the rapid input of some unicode characters and symbols

@microsoft-github-policy-service agree

words literally out of my mouth ... ooo that is neat.
One idea neat pivot here could be leverage quick accent as a pt run extension too.

that's a really good idea! i was already planning to expand it to also include HTML entity shorthands, so a quick accent option probably wouldn't be too hard either (although it might need to be implemented as a set of parallel plugins, so they can have all different trigger keywords?)


The notary config file section had a comma missing

Docker SA from South East

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit db7d8d4b3f9df06cf6387a8f6e148dcf89dc8052
https://deploy-preview-9018--docsdocker.netlify.com

LGTM. @adrian-plata PTAL


Unit tests for lexer

Requires #12

Started: 147afbd2dcbc818b6f2e3a81bb7817b9d86e5ed5

Additional tests for identifiers added in e3702ffd184c4798429dc3130477d5e79d92b147

Tests for numbers added in 9c1bad62d3c179982906ed589f7d3790b439cd88

6d5774eaf537230547b57fec60b1ce1cd35067e8 disables two of the tests for float lexing because floating-point equality is a bag full of spiders. It would be nice to fix these eventually; but the correct tokens are being generated.

f4bd1f883e1a7742f18b3a0f27146be886c3926c and 51b5aa5e6aa8a0b0e87c6b385f9818495acef6dc add tests for single character constants and R6RS-style named character constants.


Super duper cool! Will be trying this out

Thank you for creating this! I will for sure be trying this out!!  Just posting to say thanks and ask if there is any updates needed or will it work out of the box?

I think it still works out of the box (still used in production for [https://getyeti.co](Yeti Smart Home)).
Thanks for your comment, as well. The repo is supposed to be only a proof of concept so I did not develop further content.
Other links that might be interesting:

https://github.com/surialabs/react-native-widget-example
https://www.npmjs.com/package/react-native-widget-manager

Thanks very much @jsdario !


ci: update to lycheeverse/lychee-action@v1

Use lychee instead of peter-evans/link-checker, which is deprecated.
Signed-off-by: Tim Whisonant<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
13125

Relevant Lines:
19877

💛  - Coveralls


capture C-c also when user is in a long-running execution

We can't easily cancel the current computation on the jvm, so just notify the user instead
For reference, ammonite behaves the same way, only that it prints an entire
stack trace. Example:
^C Exception in thread "SIGINT handler"
java.lang.UnsupportedOperationException
      at java.base/java.lang.Thread.stop(Thread.java:1705)
      at ammonite.repl.Repl.$anonfun$action$3(Repl.scala:184)
      at ammonite.repl.Signaller$$anon$1.handle(Signaller.scala:21)
      at
jdk.unsupported/sun.misc.Signal$InternalMiscHandler.handle(Signal.java:198)
      at java.base/jdk.internal.misc.Signal$1.run(Signal.java:218)
      at java.base/java.lang.Thread.run(Thread.java:1623)

downside of this is that users cannot cancel anything with C-c any longer, e.g. if they got themselves into an endless loop, or Thread.sleep(9999999). Will modify for that use case, e.g. 'if you really want to cancel, press Ctrl-c again (within 3 seconds)`


Load Balancing not verifying tokens from other instance

Hi,
I have two identical site that are API end points. Both these are not the UI for the identity server but merely an api back-end (CRUD basically) for the users in the system.
everything is working fine for the following when on the same server:

Generate an email confirmation token. [ userManager.GenerateEmailConfirmationTokenAsync(user) ]
verify email token. [ userManager.VerifyUserTokenAsync(user, tTokenProvider, tPurpose, request.Token) ]

However when in the load balancing situation is machine A is generating the email confirmation token and machine B goes to verify the token, then the token is always invalid.
Can you tell me what I am doing wrong here, Is this a bug?
         --- The following is the configuration for the AspIdentity stuff ----
        // NOTE: This is the database connection class to be used
        services.AddDbContext<ApplicationDbContext>(options =>
            options.UseSqlServer(connectionString));

        // NOTE: This needs to stay in line with the Startup of the Identity Server projects
        services.AddIdentity<ApplicationUser, ApplicationRole>()
            .AddEntityFrameworkStores<ApplicationDbContext>()
            .AddDefaultTokenProviders();

        // NOTE: This is the Identity User classes to be used
        services.AddTransient<ApplicationUserStore>();
        services.AddTransient<ApplicationUserManager>();
        services.AddTransient<ApplicationRoleManager>();

        // NOTE: this is to set the password requirements for the user.
        services.Configure<IdentityOptions>(io =>
        {
            // Password Properties
            io.Password.RequireDigit = false;
            io.Password.RequiredLength = 10;
            io.Password.RequireNonAlphanumeric = false;
            io.Password.RequireUppercase = false;
            io.Password.RequireLowercase = false;
            // Lockout Properties
            io.Lockout.DefaultLockoutTimeSpan = TimeSpan.FromMinutes(5);
            io.Lockout.MaxFailedAccessAttempts = 5;
            io.Lockout.AllowedForNewUsers = true;
            // Configuration
            io.SignIn.RequireConfirmedEmail = true;
        });

        // NOTE:

        // NOTE: this is the messaging classes to be used
        services.AddTransient<IEmailSender, AuthMessageSender>();
        services.AddTransient<ISmsSender, AuthMessageSender>();

GenerateEmailConfirmationTokenAsync

This issue tracker is for IdentityServer issues. That's a feature of ASP.NET Identity, not IdentityServer. Check with Microsoft support.


TaskCanceledException

When calling EventHubClient.GetRuntimeInformationAsync() we are getting a TaskCanceledException ... but not every time, only a few times per day (we call this every 5 minutes).
Tried to check InnerException etc, but nothing useful ... any idea what could be the reason for it?
The EventHubClient instance we are instantiating once upon app init (lazy), and reusing afterwards ...

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

ID: d5b68363-44de-93fe-d564-336e32768e7c
Version Independent ID: 40f681e7-df94-2620-87c1-201d402203e5
Content: Troubleshooting guide - Azure Event Hubs
Content Source: articles/event-hubs/event-hubs-messaging-exceptions.md
Service: event-hubs
GitHub Login: @ShubhaVijayasarathy
Microsoft Alias: shvija

@deyanp : I see that somebody did respond to you on the SR on Jan 31st .
At this time we are going ahead and closing the issues . You can always reopen this is you think that's appropriate .

@himanshu-mishra , can you help me still on this one? The ticket brought questionable results - last recommendation is to call EventHubClient.GetRuntimeInformationAsync() every 15-30 minutes instead of every 5 minutes, which sounds really funny.
In addition to the TaskCancelledException which happens about 40 times a day (out of 288 total requests, every 5 minutes 1) I see that I am getting once or twice a day a number of 60 second long timeouts from EventHubClient.GetRuntimeInformationAsync(), e.g. Exception.Message:
One or more errors occurred. (The operation did not complete within the allocated time 00:01:00 for object request6.) The operation did not complete within the allocated time 00:01:00 for object request6.
Any idea why EventHubClient.GetRuntimeInformationAsync() is such a problematic method for us, even though we call it only 1-2 every 5 minutes??


refactor(cmd-socket-server, cartrade): use cmd-socket-server as a module

Add module exports to cmd-socket-server, change it's usage in cartrade sample app.
Cleanup redundant imports in cartrade.
Add new method of starting cmd-socket-server with BLP (like cartrade) - startCactusSocketIOServer call.
Old way of copying BLP_config is still supported, so other samples should work without a change.

Closes: #1817
Signed-off-by: Michal Bajer<EMAIL_ADDRESS>More details in the issue description.
Depends on #1813

actually, it depends on one of the previous commits, but I don't want to introduce complex dependency graph and I "chain" them one after another :)

@petermetz @izuru0 @takeutak Please review - only the last commit, others are handled by separate PRs
refactor(cmd-socket-server, cartrade): use cmd-socket-server as a module


[windows] Add Windows compatibility for the replacement ESBuild plugin

Previously, the separator for the filter regex was hardcoded to / (Unix separator).
Now, we're also taking account for the \ (Windows separator) in the filter regex.

Sounds good to me.


move issues from quarkus core repo and remove the extension there

Is it over? couldn't find it

I couldn't find this code in quarkus/core

only onther xxx-messaging-xxx

This is probably obsolete then.
@fjtirado @ricardozanini do you guys know anything about this? Otherwise, we can close it.

Yes, this can be closed.


Fix http method for translation api methods

According to docs of translation api v2 methods translate and detect should be requested by POST http method.
This patch will fix it in translate ServiceDefinition json file

(for reference, #287, #250)

Thanks for the contribution, @Erliz! I'm working to confirm this with our team. I should have more information by the end of the day.

Hey @Erliz, you are correct that we are now able to use the POST methods. I opened a PR at #522 which includes that change, as well as some other improvements with documentation and system tests. Since that change covers the goals of this one, I'm going to close this out.
Thanks again for bringing this back to our attention!.

I assumed that you are exporting this configuration from somewhere. Thanks for your improvement and support!