Swiper element with Svelte causes glitching when scrolling.

Check that this is really a bug

[X] I confirm

Reproduction link
https://stackblitz.com/edit/sveltejs-kit-template-default-1tbqpx?file=src%2Froutes%2F%2Bpage.svelte&terminal=dev
Bug description
Adding more slides to swiper container causes glitching when scrolling vertically. Also, I'm not able to figure out how to exactly use virtual scrolling where each slide is complex  with events and nested components and hence cannot be "appended" as HTML to swiper container's virtual.appendSlide method
Expected Behavior
In v8, this used to work fine without flickering.
Actual Behavior
No response
Swiper version
v9.0.4
Platform/Target and Browser Versions
macOS Chrome 109
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Still isn't solved with 9.2.0

It is because your on:slidechange={handleChange} where you during transition changes the slides and scrolling apparently does something wrong. You should add slides probably within a reachend event


integration  test failed?

I use ubuntu 14.04 64-bit  on VMware，
host pc： windows 10， Intel Core i7-6560U， 64-bit operating system x64-based processor 。
I used   Kitt-AI snowboy  resources：
https://github.com/Kitt-AI/snowboy/blob/master/lib/ubuntu64/libsnowboy-detect.a
https://github.com/Kitt-AI/snowboy/blob/master/resources/alexa.umdl
https://github.com/Kitt-AI/snowboy/blob/master/resources/common.res
The following error is my choice of resources wrong?
[----------] 18 tests from AudioInputProcessorTest (370870 ms total)
[----------] Global test environment tear-down
[==========] 18 tests from 1 test case ran. (370870 ms total)
[  PASSED  ] 13 tests.
[  FAILED  ] 5 tests, listed below:
[  FAILED  ] AudioInputProcessorTest.wakeWordMultiturn
[  FAILED  ] AudioInputProcessorTest.wakeWordMultiturnWithoutUserResponse
[  FAILED  ] AudioInputProcessorTest.tapToTalkMultiturn
[  FAILED  ] AudioInputProcessorTest.tapToTalkMultiturnWithoutUserResponse
[  FAILED  ] AudioInputProcessorTest.holdToTalkMultiTurnWithSilence
99% tests passed, 8 tests failed out of 579
Total Test time (real) = 151.54 sec
The following tests FAILED:
417 - KittAiKeyWordTest.invalidStream (Failed)
418 - KittAiKeyWordTest.incompatibleEndianness (Failed)
419 - KittAiKeyWordTest.getExpectedNumberOfDetectionsInFourAlexasAudioFileForOneObserver (Failed)
420 - KittAiKeyWordTest.getExpectedNumberOfDetectionsInFourAlexasAudioFileForTwoObservers (Failed)
421 - KittAiKeyWordTest.getExpectedNumberOfDetectionsInAlexaStopAlexaJokeAudioFileForOneObserver (Failed)
422 - KittAiKeyWordTest.getExpectedNumberOfDetectionsInAlexaStopAlexaJokeAudioFileForTwoObservers (Failed)
423 - KittAiKeyWordTest.getActiveState (Failed)
424 - KittAiKeyWordTest.getStreamClosedState (Failed)
thanks！

Hi zuowanbushiwo ,
Take a look here, it tells you where to download the files from for integration test.
https://github.com/alexa/avs-device-sdk/wiki/Linux-Quick-Start-Guide#run-integration-tests

@jjamazon
thanks！


Stopped working after CodePush

:beetle: Description
Issue happens on release build.
It's working perfect while the app receives updates through CodePush
Event if update is very simple, like style changes (not related with the react-native-sound).
Other things in app works as before, there are no other issues or crashes.
:beetle: What is the observed behavior?
No any sound after receiving updates from Appcenter.
:beetle: What is the expected behavior?
It should continue working.
:beetle: Please post your code:
const windSound = new Sound(require('../../assets/sounds/wind.mp3'));
windSound.play();

Is your issue with...

[x] iOS
[x] Android

Are you using...

[x] React Native CLI (e.g. react-native run-android)

Which versions are you using?
"react-native-code-push": "^6.0.0",
"react-native-sound": "^0.11.0"

Does the problem occur on...

[x] Simulator
[x] Device

If your problem is happening on a device, which device?

Device: Tested on diffrent devices

Have you tried to play the sound only in the callback instead of calling .play right after instantiating?
ex:
const windSound = new Sound(require('../../assets/sounds/wind.mp3'), err => {
     if (!err) {
         windSound.play();
    }
});

Also, it might be returning an error, you could try to log this error err somewhere

@murilobast Thanks for response.
Just checked but not worked. and here is the error:
{      "message":  "resource not found",     "code": -1 }

I found workaround for android.
I'm putting file into android/app/src/main/res/raw/wind.mp3
and call it like:
const windSound = new Sound('wind.mp3');
windSound.play();

But not sure about ios, where should I put files?

I found workaround for android.
I'm putting file into android/app/src/main/res/raw/wind.mp3
and call it like:
const windSound = new Sound('wind.mp3');
windSound.play();

But not sure about ios, where should I put files?
I tried to drag and drop files into xcode's project like in this tutorial but it didn't worked

I ended up having the same issue. Codepush is messing with the audio url.
Your workaround is what I did as well.
For iOS, you should open xcode and drag your audio files to your project, there's a guide in the readme.
The only problem is that I can't change the audio files without having to bundle and uploading to google play and apple store

Now the sound stops playing after a while =/
Ex code:
import { useState, useEffect } from 'react'
import { AppState } from 'react-native'
import Sound from 'react-native-sound'

// Helpers
import randomBetween from '../helpers/randomBetween'

const BGM_KEYS = {
	main: 'bgm_1.mp3',
	battle: 'bgm_5.mp3'
}

const SFX_KEYS = {
	swing2: 'axe1.wav',
	swing3: 'axe4.wav',
	swing4: 'sword4.wav',
	hit1: 'monster2.wav',
	hit2: 'monster12.wav',
	hit3: 'bow1.wav',
	confirm: 'confirm.wav',
	cancel: 'cancel.wav',
	enter: 'enter.wav',
}

const enemySounds = ['swing2', 'swing2', 'swing3']
const playerSounds = ['hit1', 'hit2', 'hit3']

const arrayToObject = array => array.reduce((current, { key, value }) => {
	return {
		...current,
		[key]: value
	}
}, {})

export default function useAudioPlayback(muteBGM, muteSFX, advertiseActive) {
	const [inBackground, setInBackground] = useState(false)
	const [BGM, setBGM] = useState({})
	const [SFX, setSFX] = useState({})
	const [current, setCurrent] = useState('main')

	const playBGM = name => {
		const audio = BGM[name]
		if (!audio) return
		if (BGM[current]) BGM[current].stop()

		setCurrent(name)

		if (!muteBGM) {
			audio.setNumberOfLoops(-1)
			audio.play()
		}
	}

	const playSFX = type => {
		if (muteSFX) return
		const sounds = type === 'enemy' ? enemySounds : playerSounds
		const name = sounds[randomBetween(0, sounds.length - 1)]

		const audio = SFX[name]

		if (!audio) {
			console.warn('SFX not found', name)
			return
		}

		audio.setVolume(0.5)
		audio.stop()
		audio.currentTime = 0
		audio.play()
	}

	const playUI = name => {
		if (muteSFX) return

		const audio = SFX[name]
		if (!audio) {
			console.warn('SFX not found', name)
			return
		}

		audio.setVolume(1)
		audio.stop()
		audio.currentTime = 0
		audio.play()
	}

	const preload = () => {
		const BGMArray = Object.keys(BGM_KEYS).map(key => {
			const name = BGM_KEYS[key]
			const audio = new Sound(name, Sound.MAIN_BUNDLE, err => {
				if (!err) {
					audio.setNumberOfLoops(-1)
					audio.setVolume(0.7)
					if (key === 'main' && !muteBGM) audio.play()
				}
			})

			return {
				key,
				value: audio
			}
		})

		const SFXArray = Object.keys(SFX_KEYS).map(key => {
			const name = SFX_KEYS[key]
			const audio = new Sound(name, Sound.MAIN_BUNDLE, err => {
				console.log('SFX', name, err)
			})

			return {
				key,
				value: audio
			}
		})

		setBGM(arrayToObject(BGMArray))
		setSFX(arrayToObject(SFXArray))
	}

	const checkAudio = shouldMute => {
		if (shouldMute && BGM[current]) {
			BGM[current].pause()
		} else if (!shouldMute && BGM[current]) {
			BGM[current].setNumberOfLoops(-1)
			BGM[current].play()
		}
	}

	const handleAppStateChange = state => {
		setInBackground(state !== 'active')
	}

	useEffect(() => {
		preload()
		AppState.addEventListener('change', handleAppStateChange)
	}, [])

	useEffect(() => {
		checkAudio(muteBGM || inBackground || advertiseActive)
	}, [muteBGM, inBackground, advertiseActive])

	return {
		playBGM,
		playSFX,
		playUI
	}
}

@murilobast have you been able to solve the issue? After doing some research on sound playback on Android I noticed that many people struggle with it. I think that Android isn't able to manage
a) the same sound played multiple times
b) a large amount of preloaded sounds
due to memory issues. For example, I need to preload approx. 200 sounds (each just some kb in size). Depending on the Android device it is either fails during preloading at a random sound or after some playbacks. Loading (and unloading) sounds as they are used is not an option for me since I need immediate sound feedback after pressing a button.
I think you have/had a similar use case here. How did you handle this? Thanks!

@murilobast have you been able to solve the issue? After doing some research on sound playback on Android I noticed that many people struggle with it. I think that Android isn't able to manage
a) the same sound played multiple times
b) a large amount of preloaded sounds
due to memory issues. For example, I need to preload approx. 200 sounds (each just some kb in size). Depending on the Android device it is either fails during preloading at a random sound or after some playbacks. Loading (and unloading) sounds as they are used is not an option for me since I need immediate sound feedback after pressing a button.
I think you have/had a similar use case here. How did you handle this? Thanks!

@murilobast have you been able to solve the issue? After doing some research on sound playback on Android I noticed that many people struggle with it. I think that Android isn't able to manage
a) the same sound played multiple times b) a large amount of preloaded sounds
due to memory issues. For example, I need to preload approx. 200 sounds (each just some kb in size). Depending on the Android device it is either fails during preloading at a random sound or after some playbacks. Loading (and unloading) sounds as they are used is not an option for me since I need immediate sound feedback after pressing a button.
I think you have/had a similar use case here. How did you handle this? Thanks!

I am facing the same issue. Did you by any chance found a solution for this issue?


Strange behaviour when editing earlier records

Could you be more specific?

Closed because unclear. @PipBrewer Re-open if necessary...


Machine view is stuck on loading indicator when ELK throws

Description
In case of any runtime errors in ELK and how it renders the machine configuration, the viz is stuck on the loading screen.
Expected Result
An error screen is rendered showing the error message or even the error stack. Potentially, a way to report the bug to the Stately team is a bonus.
Actual Result
Loading screen is shown forever.

Reproduction
Try this machine: https://stately.ai/viz/161452bd-efb9-4ee5-bd93-879401dedfd5

@davidkpiano This is fixed by #387.


dofile on first line false detection

Tic report:

even if the dofile is on first line:

Here the cartridge renamed in txt
do.txt

With double quotes work )

That's right. Thanks.

Wait we can add external libraries this way!?

Don't think so. You can include just one file. It's a limit and is an exception to the single IDE rule. Thought it like a trick to allow to use an external editor.
At the end you have to move all the code back to the Code Editor to compile correctly.

will fix dofile with single quotes

fixed on .21


Add cinemachine dependency

This is now required for VRCSDK3 to compile.

This has been added in 1.0 as part of #19


Implement register and flag::register for Windows.

This is another attempt at Windows support in #18.
Main differences from #18:

pipe/iterator supports are postponed.
Not using SetConsoleCtrlHandler directly. Instead, use signal/raise directly.
We no longer have the __emulate_kill workaround. raise works.

As for #18, confirmed to catch Ctrl-C in some cases, but not all cases.

Getting there :-). There are these little details I'm not sure about I'd like you to have a look at, but if you think I'm wrong, it'd be fine to merge it as it is.
I'm thinking about one other thing. This sub-crate is already 1.0. Adding new functions is fine, but that means promising not to ever remove or change their signatures. I'll probably ask around a bit, if there's a convention how to get an unstable part of API in otherwise stable crate ‒ so maybe I'll hold releasing it a bit after it gets merged, to be on the safe side. I hope it's not a problem. If you have a suggestion how to solve this, I'd be interested in hearing it.

Thank you. I'll mark the new functions/the ones that have different signature in some way to clearly state they may still change a bit and will release the registry crate over the weekend.

Looking through the newly introduced API, there seems to be nearly nothing (only the register_signal_unchecked function), so I think it's fine to promise API stability on this.
Do you want to add yourself to the crate authors, before I release?

Do you want to add yourself to the crate authors, before I release?

Yes, I'll prepare a PR in minutes...


path64: remove

Motivation for this change
The project seem to be inactive since 2010 (http://www.path64.org/). The
repository is also unavailable now.
Things done

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Fits CONTRIBUTING.md.

Found at https://github.com/NixOS/nixpkgs/issues/15558.

/cc @viric who seems to have used it years ago.

Here's an explanation why the company abandoned it: https://www.phoronix.com/forums/forum/software/general-linux-open-source/880291-what-happened-to-pathscale-s-open-source-path64-compiler-suite?p=880485#post880485


Deploy fee distribution updates to prod

Would you do a full resync or resync from a certain block?
https://github.com/perpetual-protocol/perp-curie-subgraph/blob/develop/scripts/updateConfigs.ts#L38-L46

Closing this PR in replacement for https://github.com/perpetual-protocol/perp-curie-subgraph/pull/95/files


Some REST responses lack content-length

Request handlers need to set a content-length response header or else the web framework will use chunked encoding to send the response content.

Assigning to @ljpearlman to sanity-check the patch.

I'm going to declare this self-reviewed...


Address flakiness by waiting for modal to appear before proceeding

Resolves partly -  #2711
Description
This PR adds an additional step inside the system specs that appear to keep failing under the donations system spec. It has become clear that the modal part of that interaction is causing some flakiness with our specs. This PR adds an addition find step which has been said to address async issues in Capybara from this thoughtbot article.
🤞 that this addresses this flakiness
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
It is fairly hard to reproduce the flakiness, so I'll be re-running this CI on this 10 times and see if the failures arise.
Screenshots
N/A

I haven't seen the specific flaky tests fail after the many runs. I do still see that we have another flaky test to address with https://github.com/rubyforgood/human-essentials/issues/3127
For now I'll merge this in and 🤞 that specs fail less now.


AckTimeout should be exposed as a parameter to ConsumerConfig

Describe the bug
When I tried to wait for a timeout to trigger the redeliver mechanism, I found that no ackTimeout parameter could be set, and I thought I could expose a parameter.

This is implemented now https://github.com/sksamuel/pulsar4s/blob/19e88e1838cb29aaf51263da6af3cf99e7b2b212/pulsar4s-core/src/main/scala/com/sksamuel/pulsar4s/ProducerConfig.scala#L41


Supercedes #1900

I have signed the OCA.  This PR is identical to #1900  but I will submit it for OCA purposes.
modified:   utility.sh
Fix text parsing error on Ubuntu 18.04.5 LTS
modified:   validate.sh
Fix yaml parsing error on MacOS.
Wait for resource ready on MacOS and remove code for helm less than 3.0
Add text for domain status troubleshooting
Use tag that includes AKS docs.
Per Reza, inline OCR authentication
Update _index.md
Update create-domain-on-aks-inputs.yaml
Update _index.md
Update _index.md
Update _index.md
Update _index.md
Update _index.md
Update _index.md
Remove UNIX
Update _index.md
Use AKS addon name
Use alias  as Rosemary suggested in our last PR.

@edburns, I'll make the edits requested by @rosemarymarano and then merge the PR.

Considering approved as I've made the requested minor updates. I'll validate the link once we commit.


[ELASTIC-108] Specify YAML block to configure cluster-wide settings

WIP to see how bootstrap (and specifically, github.com/nickbp/mustache) handles rendering newlines and spaces in a shell env var

That would be good to cover extra scenarios. Also is this per machine type or applied to all types (Master/Data/Coordinator/Ingest) ?

it would be applied to all types.

On Oct 3, 2017, at 10:25 AM, Rafael Grillo<EMAIL_ADDRESS>wrote:
That would be good to cover extra scenarios. Also is this per machine type or applied to all types (Master/Data/Coordinator/Ingest) ?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/mesosphere/dcos-commons/pull/1694#issuecomment-333917900, or mute the thread https://github.com/notifications/unsubscribe-auth/AACnWGz0V7Bz760OlJg63_UCZbbuZfGbks5som3_gaJpZM4PnuQz.

@loren: Is it possible to write a couple test cases that validate that indents are maintained from input to mustache injection?

Hey guys,
Quick question as I'm not used to the release cycles. When is this due to go to universal registry?


Add FP8 Support

Nvidia would like to see it...
FP8 is tricky b/c you need to do weird scaling stuff. Conceptually you're just replacing Linear layers with a special kind of stateful linear layer (whose state updates during backward)
References:

https://docs.nvidia.com/deeplearning/transformer-engine/user-guide/examples/fp8_primer.html
https://github.com/google/flax/blob/main/flax/linen/fp8_ops.py

The approach in Flax is probably the simplest and we should follow it. The trick is that we need to emulate the "override with gradient behavior". Otherwise I think we just patch in FP8Linear layers using a mechanism similar to what we do for LORA.
(I've been told the version in Flax is as good as the version in TE)

mostly fixed in #526 . need to make a config and do some tests

done


Action bar dimensions defined differently than in ICS

Dimensions for the action bar and buttons are defined in the following folders and they shouldn't be:
values-large-land
values-large-land-hdpi
values-large-mdpi
values-xlarge
In ICS they are defined only in these folders
values
values-land
values-sw600dp
This is causing the action bar to be sized incorrectly on 7" tablets like the original Galaxy Tab.

The 'sw' qualifier is only available on 3.2+ which means we have to do our
best to fake it using the old crappy filtering.
I'm open to additional filters to try and emulate the correct behavior. I'd
like to get "correct" behavior on pre-3.0 devices that are popular. The
Kindle Fire, Nook Touch, and original Galaxy Tab probably will have the
largest market share.
What is the screen size (normal, large, xlarge), density (mdpi, hdpi,
xhdpi), and exact resolution of that Tab?
On Feb 18, 2012 4:05 PM, "Matt Cosentino" <
<EMAIL_ADDRESS>wrote:

Dimensions for the action bar and buttons are defined in the following
folders and they shouldn't be:
values-large-land
values-large-land-hdpi
values-large-mdpi
values-xlarge
In ICS they are defined only in these folders
values
values-land
values-sw600dp
This is causing the action bar to be sized incorrectly on 7" tablets like
the original Galaxy Tab.

Reply to this email directly or view it on GitHub:
https://github.com/JakeWharton/ActionBarSherlock/issues/292

Also what do you mean by 'incorrectly sized'? Is it 48dp and you think it
should be 56dp or is it 56dp and you think it should be 48dp? Or are you
referring to it in landscape?
On Feb 18, 2012 4:22 PM, "Jake Wharton"<EMAIL_ADDRESS>wrote:

The 'sw' qualifier is only available on 3.2+ which means we have to do our
best to fake it using the old crappy filtering.
I'm open to additional filters to try and emulate the correct behavior.
I'd like to get "correct" behavior on pre-3.0 devices that are popular. The
Kindle Fire, Nook Touch, and original Galaxy Tab probably will have the
largest market share.
What is the screen size (normal, large, xlarge), density (mdpi, hdpi,
xhdpi), and exact resolution of that Tab?
On Feb 18, 2012 4:05 PM, "Matt Cosentino" <
<EMAIL_ADDRESS>wrote:

Dimensions for the action bar and buttons are defined in the following
folders and they shouldn't be:
values-large-land
values-large-land-hdpi
values-large-mdpi
values-xlarge
In ICS they are defined only in these folders
values
values-land
values-sw600dp
This is causing the action bar to be sized incorrectly on 7" tablets like
the original Galaxy Tab.

Reply to this email directly or view it on GitHub:
https://github.com/JakeWharton/ActionBarSherlock/issues/292

Check this logic for portrait/landscape configurations please:
Default: values/ @ 48dp and values-land/ @ 40dp
Large: values/ @ 48dp and values-large-land/ @ 48dp
Large MDPI: values-large-mdpi @ 56dp and values-large-land @ 48dp (e.g, Nook Color and Kindle Fire)
Large HDPI: values-large-hdpi @ 56dp and values-large-land @ 48dp (e.g., original Galaxy Tab)
Xlarge: values-xlarge @ 56dp
3.2+/sw600dp @ 56dp

That's still not right.  The Galaxy Tab should be using the default values and the Nook and Fire should be using the sw600dp values.  56dp on the Tab is huge.  values-large-land and values-large-hdpi should be removed.  Honestly, I think 48dp for the Nook and Fire in landscape would be better, but that's not what ICS uses.

If I remove values-large-hdpi you'll get 48dp in both portrait and landscape.

I guess the problem is that it translates 56dp into 84px, instead of the correct 56px if it had reported itself as mdpi.

I guess it comes down to your definition of what looks right.  I'll just change it in my code to match ICS.

Why didn't Samsung just update the OS to fix their screw-up in the first place?

http://realmike.org/blog/2010/12/21/multiple-screen-sizes-with-processing-for-android/

Apparently, that’s also what Samsung found when they made the Galaxy Tab. The Galaxy Tab has a 7″, 1024×600 screen with 170 dpi. Yet, the Tab does not report its density as “mdpi” but as “hdpi”, so the layout looks exactly as in the second screenshot. If they used “mdpi”, the icons would be .28″ wide, with “hdpi”, they are .42″ wide—not a big deal, and I must admit, the layout does look prettier this way.

http://android-developers.blogspot.com/2010/09/screen-geometry-fun.html

Similarly, the Galaxy Tab’s screen is at the large end of “large”, so declaring it as high-density applies a useful compensation.

Meh. Crappy pre-3.0 tablets cause nothing but problems...

if you use just values-large, would it be work with Nook Color, Kindle Fire and Galaxy Tab?

Yes but the Nook and the Kindle look good with a 56dp action bar because they're MDPI. We put them at 48dp in landscape because they just don't have the room for 56dp in that configuration.

The key here is getting things to look right on these devices because there will be a lot of them. I'm sure there are plenty of other pre-3.0 tablets where the action bar may be slightly too large or slightly too small but their case is insignificant since they have no market share (and the difference isn't really that bad since the whole OS looks like crap on them anyways).

But, I see values-large-mdpi and values-large-hdpi have 56 dpi or I miss something here.

I didn't think it was a screw-up but a design decision.  What's interesting though is that both the Galaxy Tab 7.0 and Galaxy Tab 2 use MDPI, so for whatever reason they changed their mind.
If you think the Nook and Kindle don't have enough room for 56dp in landscape, why would you think the Tab would have enough room for 48dp?  56dp on the Nook/Kindle is 56px while 48dp on the Tab is 72px.  This is why ICS's use of 40dp looks better on the Tab.

Yeah this needs a second look. Thinking back I was doing the pixel math wrong in my head because of the different DPI.


[analytics] Add ClientAnalytics tests

ℹ What is the context for these changes?
Adds tests for the ClientAnalytics util function added in https://github.com/Shopify/blockchain-components/pull/39
🕹 Demonstration

✅ Checklist

[ ] ~Tested on mobile~ N/A
[ ] ~Tested on multiple browsers~ N/A
[ ] ~Tested for accessibility~ N/A
[x] Includes unit tests
[ ] ~Updated relevant documentation for the changes (if necessary)~ N/A

@jamiely good suggestions for the tests! I added them in this commit: https://github.com/Shopify/blockchain-components/pull/43/commits/5fa1e8a373788ccd5c098d01aea8eafed1bb3ebb


Conflicts with existing .bashrc

sdkman does not work if you have existing commands in your .bashrc,

Definitely not true.

It is true, the presence of aliases for commands such as ls and others have somehow led to sdkman to believe that I was in Proxy mode and not online. Commenting out my .bashrc made it work again


Kerberos and Keycloak auth

Hi,
I'm using AD (Active Directory) with Kerberos protocol configured as my authentication server for my gateway (spring-cloud-gateway 3.0.4). I'm integrated with it by spring-security-kerberos 1.0.1. All routings (for all microservices/apps behind gateway) defined in my gateway require that authentication. Unfortunately some of those apps require also authentication in Keycloak (access_token, refresh_token scheme) for different credentials (not the same like in Kerberos, technical users). I've made workaround (Retry filter + 2 custom filters + some config in .yml file) but It would be nice to get some support in getting tokens from KC, store them and refreshing. It may be achieved by adding possibility to provide a Keycloak configuration to TokenRelay filter. Here is my custom configuration:
keycloaks:
  instances:
    - id: keycloak-system1-dev # it's used by other filters to pin a right KC instance
      clientId: system1-dev-client
      realmUrl: <keycloak_url>/auth/realms/system1-dev-realm
      token-url: <keycloak_url>/auth/realms/system1-dev-realm/protocol/openid-connect/token
      signedJwt:
        tmpTokenExpirationTime: 1000
        keystoreFilePath: system1.jks
        keyPassword: 123456
        keystorePassword: 123456
        keyAlias: system1-dev-client
    - id: keycloak-system2-dev
		...

Maybe this case will be covered by spring-security and spring-security-kerberos in the future but kerberos library hasn't been maintained since 2015. It isn't even prepared to work with gateway's reactive stack so It will take some time.
btw. Documentation for TokenRelay filter has been available here for last few weeks but I don't see it now. What has happened? Here is an older doc.
Best wishes

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


How to make a menu?

I would like to make a menu with glfw but I don't know how to. I don't really prefer using cocoa but is there any way to use glfw for the windows etc and glut for the menu bars and popup menus etc?

Drawing GUIs etc is really out of scope for glfw, so you'll need to use an extra library. I don't think glfw plays well with glut, but https://github.com/ocornut/imgui is great for making functional GUIs (although they won't look the same as native or cross-platform ones).

Yes I am using ImGui.


Can Hidden Header Title Youtube at start video?

pls help

I'm waiting for the answer too.
Thank you in advance.

https://developers.google.com/youtube/player_parameters?hl=en
YouTube Documention:
The showinfo parameter, which indicates whether the player should display information like the video title and uploader before the video starts playing, is also being deprecated. Following the change, the channel avatar and video title will always display before playback begins, when playback is paused, and when playback ends. The avatar being displayed is new behavior that will be consistent across all embedded players.
These changes will become effective on or after September 25, 2018. After that time, the showinfo parameter will be ignored. The behavior following the changes is consistent with the current default behavior for embedded players with the exception of the channel avatar changes mentioned above.
https://github.com/up-inside/react-native-youtube-sdk/blob/8fa70d0b7d8c34e590ce8f100cbe57b30958ddf0/ios/YouTubeView.swift#L24
You can try to change these parameters but many of them not working anymore. If you find a way to do that, you can send pull request.


Include a demo GIF in the README

This would help users discover what Lokomotive is about.

Should the demo gif show off lokoctl component apply?

Woops, meant cluster apply

Implemeneted in #636


_Session row keeps increasing, Same user login.

Hi,
After I setup my self hosted parse-server,
With some simple function testing, I try to login a user again & again (without logout),
I found that the _Session table in mongodb keep increase, +1 per login.
this was not happen in Parse-hosted version. is it a bug?
or I need to make some special configuration to correct this issue?
Thanks.

One user can definitely have many sessions on hosted parse, but your client may send an installation id which allows it to be merged.  The logic for de-duplicating sessions may not be entirely correct, but I would need more information.  What client SDK are you using and what are the differences between rows?

Hi, Fosco,
thanks for your reply.
I am developing a Unity project (with Parse-hosted), and my experience is that everytime the same user 'Login' at the same device (without logout logic), the _Session table row keeps unchange (I suppose the old session record of this user is merged/replaced with the new one...) I didn't send any installation id at all.
But after I hosted parse-server myself, for some reason, I use .Net sdk (v1.7.0) with Visual Studio, I try the same logic code, same user login at same device multiple times, I found that the server keep adding new session to _Session table, and I can see many rows of the same user. ( with different session token , createAt, expiresAt...)
one of the row looks like
{
    "_id": "QAP7fyLXTW",
    "_session_token": "r:e35029903c1b391ae37d39fa74f1a9fe",
    "_p_user": "_User$7O0DqnQwA0",
    "createdWith": {
        "action": "login",
        "authProvider": "password"
    },
    "restricted": false,
    "expiresAt": {
        "$date": "2017-02-04T08:18:02.991Z"
    },
    "installationId": "bafcb7ba-f5b2-4c9b-a478-b3cb5a5b1622",
    "_updated_at": {
        "$date": "2016-02-04T08:18:02.991Z"
    },
    "_created_at": {
        "$date": "2016-02-04T08:18:02.991Z"
    }
}

This is likely to be related to #672, and I'll close this one.. Installation de-duplication may not be built quite right, and I think stephentuso might track it down.


Help

Hi um this is not exactly an issue and I am not a programmer but what is "OTD" where do I find "settings" and where are "plugins" located. Also i dont know the domain that I have to put in obs URL.

OTD is Open Tablet Driver https://opentabletdriver.net/
It is downloaded under plugins -> plugin manager -> nzbasic's Hand Speed
Then it is enabled in filters -> Hand Speed
The URL is opened in the browser when you enable the plugin

om thank you so much


TypedDict type inference failing since v2022.8.10

Environment data

Language Server version: v2022.8.10
OS and version: Ubuntu 21.10
Python version: 3.9.5

Code Snippet

from typing import Iterable, Union
from typing_extensions import NotRequired, TypeAlias, TypedDict

class QueryDict(TypedDict):
    and_: NotRequired[Iterable["Query"]]
    field: NotRequired[str]

Query: TypeAlias = Union[QueryDict, str]

x: Query = {"and_": ["", {"field": "foo"}]}
y: Query = {"and_": [{"field": "bar"}, {"field": "foo"}]}

Expected behavior
This should not be an error. Each entry in the list is a dictionary with a valid key.
Actual behavior
For both x and y, pylance marks this as:
Expression of type "dict[str, list[str | dict[str, str]]]" cannot be assigned to declared type "Query"
  Type "dict[str, list[str | dict[str, str]]]" cannot be assigned to type "Query"
    "dict[str, list[str | dict[str, str]]]" is incompatible with "QueryDict"
    "dict[str, list[str | dict[str, str]]]" is incompatible with "str"Pylance[reportGeneralTypeIssues](https://github.com/microsoft/pylance-release/blob/main/DIAGNOSTIC_SEVERITY_RULES.md#diagnostic-severity-rules)

Notably I believe this is a regression since the previous release, where this wasn't an issue. Also this may relate to https://github.com/microsoft/pyright/issues/3796, where we had issues with a similar scenario from the type checking perspective.

The changes in pylance 1.1.266 are not yet included in the latest production release of pylance (2022.8.10). If you switch to the prerelease (insiders) build of pylance, you should see that it's fixed there.

Hmm, then I'm confused because it says they are: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#2022812-8-august-2022

Oh, I see, 2022.8.12 is a prerelease version. I'll check if this is present in that version.

That is the 2022.8.12 release, which is an insiders build.

Apologies, you are right, this works correctly in 2022.8.20 and I was just confused about the release numbers.


FLEX-3209 Moves ColorConverter helper to Shared

All is well.

Reopened the pull request since the ColorConverter is not used by any OSGP software.

All is well.


Fix poor styling on the website

Describe the bug
A clear and concise description of what the bug is.
Whenever the website is opened on a phone there are two main issues that I came across

The elements in the footer lose their responsiveness.
A few category names appear on the slideshow with barely visible font color.

To Reproduce
Steps to reproduce the behavior:

Go to the website on your phone
Scroll down to to view each of the things.

Expected behavior
A clear and concise description of what you expected to happen.
the category names should be placed according to the design and the footer elements should be made responsive preferably by reducing the font size
Screenshots
If applicable, add screenshots to help explain your problem.

Smartphone (please complete the following information):

Device: iphone 8 plus
OS: ios
Browser:chrome
Version 93.0.4577.63

Additional context
Add any other context about the problem here.

Okay, thanks for informing @roshaan005

Hi! Can I work on this issue? Thanks :)

Sure @Anna719

hey @Anna719 are you still working?

Hi, i saw that you closed the issue and everything was already fixed :)
On Sun, Nov 28, 2021, 15:59 Avneesh Agarwal @.***>
wrote:

hey @Anna719 https://github.com/Anna719 are you still working?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avneesh0612/audible-clone/issues/26#issuecomment-981090358,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCD3Z3VSBPNFBVJDP2PFE3UOIYVJANCNFSM5E7SK4SQ
.
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.

Um the issue is open and I haven't pushed something for that?

Yup, the issue is there-

If you want and the issue is not fixed yet, I can fix it


Breadcrumbs and links have "role="img" ?

Is this expected?

Linked to #176

Closing this as the issue above #176 should resolve this


Image rotated after render high resolution image

Hi,
I have an issue when using croppie. This error only with high resolution image (or Large image for example 5MB).
Step1: Select an high resolution image with height > width:

And it render with rotated image:

This issue not occur on some computer.
I don't know why and how to fix this.
Do you have any idea for this case?
Thanks!
Original Image:

Hi, you should enableExif:

Enable exif orientation reading. Tells Croppie to read exif orientation from the image data and orient the image correctly before rendering to the page.
Requires exif.js

@thinhpc you didn't have any issue with bigs images like me?
https://github.com/Foliotek/Croppie/issues/328


User can't find where they saved their keys

"Omg, i still cant find it. My phone is Samsung A11. Tried searching all of my files but there was none."
"Yep yep. Dont see any.
The app should have a promt message that the file had been saved and where it is located when clicng the "save to phone" button so the users wont get confused.
Like me :)"

see #18


unable to resolve module react/lib/ReactPropTypes

am trying to connect the react component with my native android app(Time being am experimenting with hello world project ) but its failing with following error. Any help would be appreciated.
Pkg.json config
            "react": "^16.0.0-alpha.6",
	"react-dom": "^15.6.1",
	"react-native": "^0.44.0",
	"react-native-web": "0.0.103",
	"webpack": "^2.6.1"

StackTrace
bundling: UnableToResolveError: Unable to resolve module react/lib/ReactPropTypes from /Users/myMac/Documents/Androidapps/TestReactNative/node_modules/react-native/Libraries/Image/Image.android.js: Module does not exist in the module map or in these directories:

I'm getting the same error.
Why the issue was closed? Is there any solution for this issue that I'm missing?
Thanks,
Joao
"dependencies": {
    "native-base": "^2.1.4",
    "react": "^16.0.0-alpha.12",
    "react-native": "^0.44.0",
    "react-native-calendars": "^1.4.0",
    "react-native-camera": "git+https://github.com/lwansbrough/react-native-camera.git",
    "react-native-checkbox": "^1.1.0",
    "react-native-fcm": "^6.2.0",
    "react-native-fetch-blob": "^0.10.5",
    "react-native-image-picker": "^0.26.3",
    "react-native-localization": "^0.1.29",
    "react-native-modal-dropdown": "^0.4.2",
    "react-native-modal-picker": "0.0.16",
    "react-native-pdf-view": "git+https://github.com/cnjon/react-native-pdf-view.git",
    "react-native-popup-dialog": "^0.7.22",
    "react-native-side-menu": "^0.20.1",
    "react-native-tabs": "^1.0.9",
    "react-native-vector-icons": "^4.0.0",
    "react-navigation": "git+https://github.com/react-community/react-navigation.git",
    "uuid": "^3.0.1"
  },

You need to require PropTypes from the prop-types package, not react now.

Im not using PropTypes anywhere in my code and I am having this issue too. Do we need to update react or react-native?
My package.json
"dependencies": {
    "axios": "^0.16.1",
    "firebase": "^3.6.6",
    "native-base": "^2.1.4",
    "prop-types": "^15.5.10",
    "react": "^16.0.0-alpha.6",
    "react-native": "^0.44.0",
    "react-native-fbsdk": "0.6.0",
    "react-native-i18n": "^1.0.0",
    "react-native-router-flux": "^3.37.0",
    "react-native-vector-icons": "^4.0.0",
    "react-redux": "^5.0.1",
    "redux": "^3.6.0",
    "redux-persist": "^4.4.0",
    "redux-promise": "^0.5.3",
    "redux-thunk": "^2.1.0"
  }

If any of your dependencies are using React.PropTypes they will need to be updated too so I would recommend auditing your dependencies (and their dependencies, etc.).

Hello,
So, apparently, in the React 16 we still have it, but it's deprecated. (will be removed in the next version of the react?
)
What worked for me was to do (I'm also using a similar configuration as Cotel is using):
    "native-base": "^2.1.4",
    "react": "^16.0.0-alpha.6",
    "react-native": "^0.44.0",
    "react-native-calendars": "^1.4.0", // This is the component that was causing the error

was calling this command:
npm install<EMAIL_ADDRESS>--save
Not sure why, but after running this command, in my case worked. I think that the --save option makes it global and this made the trick to get away from the error.
I know that the correct would be to fix the React.PropTypes, but once that the component was installed via npm, I would prefer not touch the node_modules.
Cheers,
Joao

The issue comes from the little caret in your package.json, the ^
Since it matches all minor versions, if you let the caret in 16.0.0-alpha.6 and do npm i, it will install 16.0.0-alpha.12 or the latest.
If you remove it in your package.json like this :
"react": "16.0.0-alpha.6",
"react-native": "0.44.0",

It will install EXACTLY the same version as written, and since alpha 6 and 12 import prop-types differently, the problem surely comes from here

@VinceBT are we need to remove the ^ for react and react-native or for all dependencies?

Just for react and react-native, but is this issue still relevant today ?

Yes I am getting this issue. I am using follwoing version
"react": "16.0.0-alpha.12",
"react-native": "0.44.3",


exit unknown on exception or bad args

when a plugin throws an unexpected exception we should exit(3) unknown.
As it means the plugin was unable to signal warning or critical itself
meaning the state of the thing being checked is unknown. This commit
changes the behavior from exit(2) critical which generally results in
someone getting paged with a false alarm.
Similarly, when someone deploys a check and has given it the wrong
arguments we should also exit(3) unknown. We don't know the state of the
thing we are attempting to check.  We only know the check was
misconfigured because the options are wrong.

Hi @fessyfoo, thanks for the pull request and the explanation of your reasoning.
I'd initially tagged this for consideration in the v1.4.0 release, but I'm starting to think that if we accept this change it will constitute a breaking change to our public API. I'm removing this PR from the 1.4.0 milestone, but I think it would be good to hear from other folks as to whether this is something we want for 2.0.0.
cc @sensu-plugins/commit-bit

@fessyfoo I concur with the idea of an unknown if we have an exception, but not on a config error as that is a know issue.
In our stuff we have these exit codes for a reason. It allows us to be more descriptive with exits and then tailor monitoring to filter on specific codes, so anything getting a runtime issue goes to developer foo, anything with a critical error goes to on-call bar, etc.
// MonitoringErrorCodes provides a standard set of error codes to use.
// Please use the below codes instead of random non-zero so that monitoring can
// utilize existing maps for alerting and help avoid unnecessary noise.
var MonitoringErrorCodes = map[string]int{
	"GENERALGOLANGERROR": 129, // internal script error
	"CONFIGERROR":        127, // unix config error, not enough parms, etc
	"PERMISSIONERROR":    126, // not executable, etc
	"RUNTIMEERROR":       42,  // self explantory
	"DEBUG":              37,  // You had the Alliance on you, criminals and savages… half the people on this
	// ship have been shot or wounded, including yourself, and you’re harboring known fugitives.
	"UNKNOWN":  3, // Would it save you a lot of time if I just gave up and went mad now?
	"CRITICAL": 2, // “The ships hung in the sky in much the same way that bricks don't.”
	"WARNING":  1, // this kinda sucks but don't get out of bed to deal with it
	"OK":       0, // “We’re still flying”

}

The point is the 12N range are std *nix convention and while not everything lives in *nix land this seems to be a good path to follow. We don't need to get this complicated by no means but it does help to work towards using something more than 0-3 which IMHO has gone the way of WindowsNT ;)

@mattyjones  I believe that in addition to agreeing with me that an unhandled  exception should exit(3) unknown,  you're also in agreement with me that invalid arguments should not exit(1) warning
I think you further suggest that incorrectly configured arguments should get it's own special exit code.   Since the sensu plugin docs  suggest that any other exit code is also treated as unknown I'm not against that.   however I don't think there's any standard to adhere too unless the sensu team wants to define one.   until them I'd leave it exit(3) unknown
furthermore the nagios plugin api  only specifies exit codes 0, 1, 2, 3  and I believe other exit codes result in nagios return code of NNN is out of bounds  we likely don't care about backwards compatibility with nagios though?

@mattyjones on a side note.   your code above uses 127  as config error?  this is the exit code that most posix shells will return for command not found. I wouldn't intentionally exit(127).

Ending a script with exit 127 would certainly cause confusion when troubleshooting (is the error code a "command not found" or a user-defined one?).

http://tldp.org/LDP/abs/html/exitcodes.html

@cwjohnston thanks for the response.  I agree this  subtly alters behavior.  I'd be interested to know if anyone had any issue with it.


[FR] Improve UI to patches capabilities

Proposal Summary
I would want to_patches() to be as though I am seeing a bunch of crops of the image and it's metadata. So I would expect to retain any image level metadata and any annotation that overlaps with the crop.
Specifically within the UI there is no ability to select added fields to display in patch form. so I wold suggest defaulting to all fields rather than no fields.
Motivation

What is the use case for this feature?
allowing non-technical team members navigate through patches from within the UI.

Why is this use case valuable to support for FiftyOne users in general?
many practitioners break up images into patches and work on the patches since

images are too large to process whole (especially with ViTs, full res image synth, segmentation, super res...etc)
objects are spread across images (in real world images).
it allows for better control over distributions as compared to per image work.
On the other hand , data enters the fiftyone ecosystem unprocessed preferably because:
quicker/cleaner (less code / assets / mappings...etc)
we eventually do want to get image level stats
we want the image level context available during evaluations / patch selection ...etc

Why is this use case valuable to support for your project(s) or
organization?

many of our team members work primarily with patches.
our non technical team members (domain experts) are also interested in the patch level at times.

Why is it currently difficult to achieve this use case? (please be as
specific as possible about why related FiftyOne features and components are
insufficient)

can't directly see overlapping annotations of the same field in patch view
UI doesn't allow us to choose which fields to see in patch view.

What areas of FiftyOne does this feature affect?

[x] App: FiftyOne application
[?] Core: Core fiftyone Python library
[?] Server: FiftyOne server

Willingness to contribute
The FiftyOne Community encourages new feature contributions. Would you or
another member of your organization be willing to contribute an implementation
of this feature?

[ ] Yes. I can contribute this feature independently.
[ ] Yes. I would be willing to contribute this feature with guidance from
the FiftyOne community.
[x] No. I cannot contribute this feature at this time.

Thanks for the feature request!
For the benefit of others that may come across this, it is possible to include other fields via the to_patches() API method (and the App if you create a ToPatches stage in the view bar), like so:
import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset("quickstart")
patches = dataset.to_patches("ground_truth", other_fields=["predictions"])
session = fo.launch_app(patches)

The reason other_fields = ["all", "other", "fields"] isn't the default right now is that would substantially increase the amount of data in the view. If there are 100 objects per image, a patches view that includes "copies" of all other fields would 100x the size of the raw data in the view.
WRT the idea of a patches view that also includes all overlapping patches from the same field, you can achieve the outcome you want just by clicking on an object in a regular (non-patches) view and then using the z shortcut to zoom to the selected content. That will show exactly what you're after.
Patches views still load the entire image, so if the source image is gigantic then the patches view will still be limited by that. If the sample has so many labels that the expanded modal is slow to render, then using select_fields() to only select the fields of interest is a good option.

@brimoor thanks for the detailed answer.
i was thinking that the data itself would be by reference and not duplicated. if it is duplicated i would think that filtering only the tags that overlap would reduce the duplicates significantly in most cases (but it would take some time to compute).
anyways -
would it be possible to add that filter to the list of filters available in the ui so it becomes no code. (I'm talking about the views bar - like "sort by"...etc)

if it's simple I don't mind try to contribute it.

i was thinking that the data itself would be by reference and not duplicated. if it is duplicated i would think that filtering only the annotations that overlap would reduce the duplicates significantly in most cases (but it would take some time to compute).

I put "copied" in quotes because it's not that the data is duplicated on disk, say, but rather that it is effectively duplicated in the sense that any statistics computed on the data would involve multiple copies of the data.
As an implementation detail, patches views actually do currently create a temporary duplication of the data on disk (so I reallllly don't think other_fields = ["all", "other", "fields"] is a reasonable default right now), but I'd like to improve the implementation to avoid that when I (a) have some bandwidth, and (b) can figure out how to actually make it happen 😅
The thing about DatasetViews is that they must be express-able as a query that can be dynamically run against the dataset (or more painstakingly implemented---like to_patches()---to make it seems like that's what's happening).
Here's the best way I can think of to achieve a view that contains what you want.
To get there, I had to do this:

Clone the field that contains the objects of interest
Create a patches view against the original field that also includes the cloned field
Add a filter_labels() stage that dynamically filters the cloned field to only include overlapping objects for the "reference" object of that patch

So it's almost a dynamic DatasetView, but I did have to clone the field to start with.
import fiftyone as fo
import fiftyone.zoo as foz
from fiftyone import ViewField as F

def is_overlapping(field, iou_thresh=None):
    """Returns an expression that computes whether a ``Detection`` overlaps
    with the ``Detection`` in the specified field.

    Args:
        field: the field
        iou_thresh (None): an IoU threshold to use to determine overlap

    Returns:
        a :class:`fiftyone.core.expressions.ViewExpression`
    """
    bbox = "$" + field + ".bounding_box"
    x0 = F(bbox)[0]
    y0 = F(bbox)[1]
    w0 = F(bbox)[2]
    h0 = F(bbox)[3]
    x = F("bounding_box")[0]
    y = F("bounding_box")[1]
    w = F("bounding_box")[2]
    h = F("bounding_box")[3]
    inter_width = (x + w).min(x0 + w0) - x.max(x0)
    inter_height = (y + h).min(y0 + h0) - y.max(y0)

    if iou_thresh is None:
        return (inter_width > 0) & (inter_height > 0)

    area = w * h
    area0 = w0 * h0
    inter = inter_width * inter_height
    union = area + area0 - inter
    return inter / union

dataset = foz.load_zoo_dataset("quickstart")

dataset.clone_sample_field("ground_truth", "gt")

patches = (
    dataset
    .to_patches("ground_truth", other_fields=["gt"])
    .filter_labels("gt", is_overlapping("ground_truth"))
)

session = fo.launch_app(patches)

looking at your FR (1299) I understood that I'm actually asking for 2 things.

exactly what you describe - a way to see objects as a view that can be toggled back and forth easily to get more context. In this usecase the full image information is not really needed in the patches (other than maybe for filtering purposes). BTW it would be nice to have a smoother flow with the embedding vectors for these objects.
My main work here is to find subclasses, misslabels, out of distribution ...etc on an object level.

a way to define a patches dataset that I can then work with during training. these are often constant sized patches with a controlled distribution of objects in our use-cases. In this situation I would likely want to be able to get the overlapping annotations and metadata of each patch for training.
My main work here is defining the dataset I want to train on (and likely updating this dataset with more examples from the original full image dataset as I progress).


get datums using pallas observer

Content

Includes:

get current datums using the new pallas observer;
brings all compatibilities with the previous cardano-cli format;
uses the current cardano-cli as a default fallback for the remaining methods;

Pre-submit checklist

Branch

[x] Tests are provided (if possible)
[ ] Crates versions are updated (if relevant)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages

PR

[x] No clippy warnings in the CI
[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

Comments

The automatic fallback mode has been implemented to make sure we progressively rollout the new integration.

@scarmuega the canonical trait worked like a charm!
all the requests was addressed and is ready for a new review.
thanks


jest unit test is broken after upgrade @preact/signals from 1.3.1 to 2.0.0.

[x] Check if updating to the latest version resolves the issue

Environment

[ ] I am using @preact/signals-core
[x] I am using @preact/signals
[ ] I am using @preact/signals-react

[ ] I understand usage changed in v2, and I've followed the React Integration instructions

Describe the bug
jest unit test is broken after upgrading @preact/signals from 1.3.1 to 2.0.0.
Follow the instructions for release updates: When you perform a signal update, you'll need to wrap it in act.
I did it, but it still failed. This test can pass after rolling back to 1.3.1
To Reproduce
https://stackblitz.com/~/github.com/rock-57blocks/NeverReact
Steps to reproduce the behavior:

Go to stackblitz
open terminal
run 'npm test'

Expected behavior
This test is passed on @preact/signals 1.3.1, but failed on 2.0.0.

I found that if a child component uses useEffect, and its dependencies have prop signal value from the parent component, it will make the parent component's useSignalEffect callback invalid.
e.g

Hmm yes, looks like the signal-effect does not rerun when the dependency changes

Hmm yes, looks like the signal-effect does not rerun when the dependency changes - does it work at runtime?

@JoviDeCroock Thanks for the quick replay.
I tested it and it works fine at runtime. This issue only occurs during unit testing.
BTW,  what confuses me is that if the child component's useEffect does not depend on the signal parameter in the signal props but only on something else, then this test will work.
See the follow code:
const MuteButton: FunctionalComponent<MuteButtonProps> = ({
  isMuted,
  isTestMuted,
}) => {
  useEffect(() => {
    console.log('isMuted', isTestMuted);
  }, [isTestMuted]);

  return null;
};

export const muteVideo = signal(false);
const TestPlayer: FunctionalComponent = () => {
  const videoRef = useRef<HTMLVideoElement>(null);
  useSignalEffect(() => {
    videoRef.current!.muted = muteVideo.value;
  });

  return (
    <Fragment>
      <div>
        <video
          data-testid='video-player'
          ref={videoRef}>
        </video>
        <MuteButton
          isMuted={muteVideo.value}
          isTestMuted={true}
          />
      </div>
    </Fragment>
  );

};

Yes, the act override all explains that. We queue twice because the timing of rerender --> effect queued up, signal update --> effect queued up are slightly different, see my last edits to the above which includes the fix

I see..
Will it be include in a new version 2.0.1 very soon or later？
Thanks a lot. @JoviDeCroock

As I said, I have no time now... it's Christmas today 😅 I gave you the solution so you can PR or patch it if it's urgent, this is from my phone

oh sorry. I'm not rushing you just to ask about a possible release date, don't worry it's not a very urgent need, please enjoy your holidays and Merry Christmas！

This will be fixed in 10.25.4, thanks to @rock-57blocks! You can track https://github.com/preactjs/preact/pull/4620 to see when the release goes out

Released in 10.25.4


Heading Renderer makes it difficult to support custom classes

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.111.2-4164f8fef9d71f50ef3962897e319ab6219a1dad+extended linux/amd64 BuildDate=2023-03-05T12:32:20Z VendorInfo=gohugoio

Does this issue reproduce with the latest release?
Yes.
The Issue
The custom renderer for headings (render-heading.html) does not clearly support custom classes specified in Markdown (though Goldmark allows this, and the default heading renderer works correctly).  The only available method is not documented anywhere.
Proposed Solution
The proposed solution is to promote class to a top level variable on the context passed to the Heading Renderer (eg. .Class).  Adjusting the documentation accordingly would not go amiss.
Workaround
The current workaround is to access an undocumented field (.AttributesHolder.Attributes.class).

I don't understand.
https://gohugo.io/templates/render-hooks/#context-passed-to-render-heading

And now I'm just plain wondering what I was ~smoking~ reading.  I read what must have been a very similar page, and I could swear that .Attributes was not listed.  That led to a spelunking of the source code, which led to the workaround (which is also needlessly complicated, because I forgot about golang's automatic member promotion).
Yeah.  Apparently I'm either hallucinating, or I somehow got lost in an older version of the documentation.  Or maybe both.
I guess this one can be closed, and I do apologize for the waste of time.  :-)


The generator will hang

generatemethod will hang if don't stop spring.
before using generate, we should add run "spring stop"
etc:
  run "spring stop"
  generate "clearance:install"

@wenweih Good remind.
I will add this command to after_bundle block at the first position.


Email with CSS styling

I tried sending email body as an Html but doesnt quite seem to work.
I have tried the following
message.setBody(new MessageBody(BodyType.HTML, _body));
But still in outlook I can see the format is Plain text.
Even in gmail styles seem to be removed.
Almost all the html tags and inline css, tables etc. seems to be removed.
Can you please share some info on how to get this working.

I managed to solve this issue by doing the below.
It may be of help to someone.
http://support.risualblogs.com/blog/2011/02/24/html-mails-sent-via-owa-and-outlook-2011-are-received-as-plain-text-mails-externally/
Summary:
The final thing we checked which revealed what the issue was to run the following exchange powershell “get-remotedomain | fl” . We then spotted that the content type was set to MimeText which “converts all messages to MIME messages that use text formatting” (i.e plain text). We then changed the content type to mimehtmltext (Converts messages to MIME messages that use HTML formatting, unless the original message is a text message. If the original message is a text message, the outbound message is a MIME message that uses text formatting) by running the following command “get-remotedomain | set-remotedomain -ContentType MimeHtmlText”.


Roman to Integer

Can you assigned this to me for C++.

Please assign this to me in C++

@khushij12 @anushkarjain Sorry :) The issue is already assigned to @Piyush228 here :: https://github.com/piyushsharma220699/Hacktoberfest-2021/issues/13
Please create an issue for another problem or check the existing issues to get them assigned to you 👍🏻 For now, I'm closing this issue.


unable to pull nest doorbell that was migrated to google home

Which version are you using?
latest
Which operating system are you using?

[ ] Linux amd64 standard
[x ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
Google recently prompted me to migrate my nest doorbell to google home from the nest app. Since then, my public URL no longer functions which brought me to this project through some research.
I have nestmtx functioning and oauth configured and working but, as i should have expected, it does not pull the camera when forcing a sync.
Describe how to replicate the issue
in a fresh and otherwise working container, attempt to access a nest doorbell cam that was migrated to the google home app from the nest app
Did you attach the server logs?
no
Did you attach a network dump?
no
probably not a bug in the strictest sense but curious if it is something that can be addressed as I believe the writing may be on the wall for the nest app and brand.

I did find this large blue star mentioning the migration in the docs. it makes it seem as though this should still work but perhaps with some tweaks.

Have you enabled access to the camera via the device access console?

I didnt know i had to but i also dont see that as an option anywhere or documented in the how-to
I absolutely could have just missed it. I do have a project set up in device access.

I can also enable debug logging and provide a pastebin if that will help, because of the migration i figured the problem may be known and way more fundamental. even so, it could rule things out.

here we go. im going to migrate back to the nest app and report if that fixes it.
https://support.google.com/googlenest/answer/13038234?hl=en#zippy=%2Cnest-app-only-features%2Cgo-back-to-the-nest-app


birdnet appliance

Running BirdNET on Migration Call Data
Defining Baseline Performance

[ ] Run BirdNET on our bird migration call data to establish a baseline performance for comparison with future models.

Expected Results:

[ ] Identifying Misclassifications: common errors and misclassifications that occur due to BirdNET’s focus on daytime calls and general bird sounds.
[ ] Performance metrics (accuracy, false positives, etc.)
[ ] Report on misclassified bird species and time-of-day mismatches
[ ] Insights for future model improvements

birdnet classification with highest confidence is chosen for each file
if birdnet detected species that is not in the true labels, birdnet detection = otherspecies
if birdnet did not classify any bird = nopred


Support automatic abortion  s3 multipartUpload after a timeout.

If a MultipartUpload request fails and the MultipartUpload request is not aborted,  directory  xxx_s3_multipart_tmp will be preserved, and future MultipartUpload requests will always error.
Describe the solution you'd like

Create a callable instance when a MultipartUpload request received which used to abort this MultipartUpload. Callable instance contains:

upload file name (xxx)
uploadId
last_update_time (xxx_s3_multipart_tmp modify time or xxx modify time)

Create a ScheduledExecutorService  instance used to schedule  auto abort MultipartUpload callable instance.

last_update_time + timeout = delay

When execute auto abort func:

check xxx_s3_multipart_tmp if exist
check xxx_s3_multipart_tmp dir id if equals uploadId
check if xxx_s3_multipart_tmp  modify time changed
check xxx if exist and if xxx modify time changed

May caused problem

Upload object part or finish MultipartUpload request may be distributed to different proxies, how to coordinate different proxies? Leader to do this?

Urgency
Normal

Regarding

Upload object part or finish MultipartUpload request may be distributed to different proxies, how to coordinate different proxies? Leader to do this?

I don't see a clear way to get around this if someone is using multiple proxies. Alluxio just does not allow multiple clients to upload+complete the same file. I think we will just have to document a restriction that the same proxy must be used during a multipart upload to properly complete the request. If a load balancer in front of the proxies is desired it should deterministically redirect the request to the correct proxy

Hey @jffree , is this answer you question?

@ZacBlanco are there any code changes needed for this issue? feel free to open this issue if that is the case.

yes, we do need to make code changes -- in fact there is a TODO in the code for it left by the original author of the S3 handler

@ZacBlanco I can do the preliminary implementation first. Then you can help to review the code.

How does S3 handle this? I think that there should still be some timeout where the upload becomes invalid.
I did a quick search, apparently the policy is configurable (as of 2016, when this was answered)
According to the recent AWS docs - this is still the case (AbortIncompleteMultipartUpload):

AbortIncompleteMultipartUpload action element – Use this element to set a maximum time (in days) that you want to allow multipart uploads to remain in progress. If the applicable multipart uploads (determined by the key name prefix specified in the Lifecycle rule) are not successfully completed within the predefined time period, Amazon S3 stops the incomplete multipart uploads. For more information, see Aborting a multipart upload.

So, I think we should still implement this and make it configurable if S3 also has some kind of timeout. It will also prevent memory waste over time if users forget to abort their uploads.

Thanks for the improvement @jffree ! Closing this issue as your PR has been merged.


Ship debugger data viewer integration

For https://github.com/microsoft/vscode-python/issues/15529
Do not merge until https://github.com/microsoft/vscode-jupyter/pull/5007 has shipped in stable to avoid breaking users on the insiders channel for the Python extension (as their version of the Jupyter extension will still require that context key to be set but the code in the Python extension to set the context key will be gone).

https://github.com/microsoft/vscode-jupyter/pull/5007 went into the stable release on Wednesday: https://github.com/microsoft/vscode-jupyter/commits/release-2021.03 so this should be safe to merge.


Store data as text.

Are you requesting a feature, reporting a bug or ask a question?
I'm asking a question.
What happened to the a file that the user uploads if I don't select "storeDataAsText" and I'm not using the dxService.

I'm using react.

@nrhoffmann hello! In that case you need to handle file uploading by yourself. Please find onUploadFile in the doc: http://surveyjs.org/docs/api_surveymodel.html. And for example something like that: https://plnkr.co/edit/W2gKg1EEVzgDDO2POniV?p=preview

If I modify model.data, will the change be reflected in sender.data?

To clarify: will that change be reflected in a subsequent event's sender / model?

@nrhoffmann model.data and sender.data in the events are the same instance.
It is better to use sender, because you may have more then one surveys instance on the page and event handler function may be used by several surveys.
However you may use any variant, since you have only one survey instance:
var survey = new Survey.Model({...});

survey.onComplete.add(function(result) {
      var data = survey.data;
});
survey.onUploadFile.add(function(model, options) {
    var data = survey.data;
});

==
survey.onComplete.add(function(result) {
      var data = result.data;
});
survey.onUploadFile.add(function(model, options) {
    var data = model.data;
});

Thank you,
Andrew

No reply for a long time. I am closing the issue.


Update/Remove Old Devskim from Packages.microsoft.com

See #121.  Devskim appears to be have been distributed in some way that apt can get it.  We are no longer publishing this way so should we take down the old version to avoid confusion?

I can't find this link anymore.


[UpdateDownloadAll] Race Condition

Describe the bug
After clicking on UpdateDownloadAll, in case of novel-entry in the middle of the library, if currently selected novel has some updates, then the code-execution splits into 2 -

First goes into the next novel-entry and starts loading that
Second keeps downloading the chapters of the currently selected novel-entry

This bug happens mostly when the computer isn't switched off for a long time, for example my system hasn't been turned off for 3 weeks. Bug still appears very randomly. In some PCs, it can't be reproduced at all.
To Reproduce
Do as mentioned above
Expected behavior
Point 2 should be the one happening and not point 1.

From my issue earlier with updatedownloadall:
Added a check to update novel to make sure before it runs it had all known chapters and currently its working, if it keeps working for a few days I will give a update on it, also could not reproduce the one you posted but my client at least has been restarted a lot of times (Computer only restarts on major win updates or my ram gets screwed over by mem leaks)

The following half works (Will break it partially, but semi works out if you keep running it but working on fixing that)
updateNovel(t_novel) {
   if (
      t_novel.ChapterCount != t_novel.DownloadedCount
    ) {
      this.downloadNovel(t_novel);
    }

In short it checks for a chapter imbalance before hand, but if you have multiple novel imbalances it will cause issues currently.

Just giving a heads up updatedownloadall has not broken on me for a long while and only change I am really seeing between new version and mine for the update function is:
if (this.iframe_working) {
return;
}
if (
t_novel.ChapterCount != t_novel.DownloadedCount
) {
this.downloadNovel(t_novel);
}


feat: skillset distribution plot

Summary by Sourcery
Add a skillset distribution plot to the appendix to visualize and describe the distribution of required skills in the ChemBench corpus, enhancing the documentation with a new figure and accompanying text.
New Features:

Introduce a skillset distribution plot to visualize the distribution of required skills in the ChemBench corpus.

Documentation:

Add a figure and description to the appendix to illustrate the composition of required skills in the ChemBench corpus.

looks fancy! are you our new figure wizard, @n0w0f?

looks fancy! are you our new figure wizard, @n0w0f?

if only i could script to do this.

looks fancy! are you our new figure wizard, @n0w0f?

if only i could script to do this.

apple script or the new computer mode from anthropic ;)


how to train?

I am very interested in your research.
I want to perform custom learning on my data, but I am curious about the example training code and how to organize the data.
If this is not an excuse, I would like you to provide the code to create the dataset.
Thank you.

For the training on Something-V1 dataset.  I use the script below:
python main.py --dist-url 'tcp://localhost:6666' --dist-backend 'nccl' --multiprocessing-distributed --available_gpus 0,1,2,3 --world-size 1 --rank 0 --dataset something --gd 20 --shift --shift_div=8 --shift_place=blockres --npb --lr 0.02 --wd 1e-3 --dropout 0.8 --num_segments 8 --batch_size 16 --batch_multiplier 4 --use_warmup --warmup_epochs 1 --lr_type cos --epochs 50 --non_local  --suffix 1
you should get the same result on the paper.  The trained models will release recently.
About how to organize data, the code of my data is acturally based on https://github.com/yjxiong/tsn-pytorch, which can provide you more details.

Thank you so much!


[HELP] How to export llm.pt to onnx?

And how to fix the onnx input length while the text_encoder not output a fix length token？

we export llm.pt to libtorch in our production system, but this export code is not opensourced yet, we will consider it later

we export llm.pt to libtorch in our production system, but this export code is not opensourced yet, we will consider it later

is the export model has fix input length?

A simple Time Cost Test Result:
llm encoder cost: 5.938025942072272s
llm decoder cost: 0.1739195343106985s
llm forward_chunk cost: 6.113153585232794s
llm cost: 6.119191139005125s
flow cost: 0.6851033219136298s
hift cost: 0.06346304202452302s
total cost: 6.9849236300215125s

So the bottleneck is the llm encoder.

A simple Time Cost Test Result:
llm encoder cost: 5.93s
llm decoder cost: 0.17s
llm forward_chunk cost: 6.11s
llm cost: 6.11s
flow cost: 0.68s
hift cost: 0.06s
total cost: 6.98s

Any way to reduce the llm encoder time cost ？(about 82% of total time)

llm encoder generates token one by one, we are also trying to reduce its computation time, maybe export to onnx or libtorch will reduce computation

we export llm.pt to libtorch in our production system, but this export code is not opensourced yet, we will consider it later

Hi, when would you update codes converting .pt to .onnx or libtorch? Thanks


included markdown documents are parsed as rst

I have tried the recommonmark plugin locally and on RTFD.org, and it seems to fail fairly basic Commonmark markup:
http://monkeysign.readthedocs.io/en/latest/
Notice how the following tags are not parsed properly:

### Requirements header
code blocks (apt-get install monkeysign is rendered in a <blockquote>)
links ([OpenKeychain][] and others)

All this works fine on the standard Commonmark tester here:
http://spec.commonmark.org/dingus/
direct link
is this a bug in commonmark.py or am i doing something wrong?

obviously, i was doing something wrong: it's an include from a .rst file... but the question still stands - is this a desirable feature?

Just ran into the same problem/question. Looks like the best solution is to use symlinks to the files you want. You can see an example on the recommonmark documentation files

I run into this issue today and found this alternative to recommonmark which fixes the issue by providing an alternative .. mdinclude directive to include Markdown documents.

@arximboldi I have the same problem. Which package does that (you just say "this")?

@bilderbuchi sorry I didn't realize I didn't paste a link. I think the project was this one: https://github.com/miyakogi/m2r
Admittedly, in the end I just submitted to getting used to RST everywhere. It is still a bit annoying that Github does not render admonitions and so on but I guess I can live with it :-)

Thank you, I'll see where I can go with this. My docs being on a windows network share (i.e. no proper symlinks), an mdinclude directive is the only way I can see to include out-of-tree markdown files into my Sphinx documents. Would be great if recommonmark added that feature. :-/

I guess recommonmark can add an mdinclude directive like m2r has for this. It would be really useful for projects wanting to include a markdown file from outside the Sphinx source tree.

I found a better way to pull this off. Just use copyfile from python at the begining of the conf.py and copy the files that are markdown that you want to convert into the appropriate places and then they will be parsed by recommonmark.

I am still failing to use mdinclude in my sample project using file/document intro.rst
Readme File
===========

.. mdinclude:: ../../README.md

resulting in
jirka@jirka-UX430UN:~/py_sample-project/docs$ make html
Running Sphinx v1.8.5
loading pickled environment... done
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 1 source files that are out of date
updating environment: [] 0 added, 1 changed, 0 removed
reading sources... [100%] intro
~/py_sample-project/docs/source/intro.rst:4: WARNING: Unknown directive type "mdinclude".
.. mdinclude:: ../../README.md
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
writing output... [100%] intro
generating indices... genindex
writing additional pages... search
copying static files... done
copying extra files... done
dumping search index in English (code: en) ... done
dumping object inventory... done
build succeeded, 1 warning.

@Borda perhaps you misunderstood: the mdinclude directive is provided by the m2r extension, and not yet by recommonmark.
In the current version of recommonmark, many users can get by by symlinking their readme to their docs source directory. The main limitation of this workaround is that it will not work correctly when the docs are built on Windows.

Perhaps related: https://github.com/sphinx-doc/sphinx/issues/2840

@FeralBytes Is there a way to delete the copied files after the documentation has been built? I.e. how can I execute Python code in conf.py that is only executed once the documentation generation has been completed?

@CsatiZoltan I am not sure this is possible to ensure it happens after the documentation generation is executed. But my only thought is to try a few things like, what happens if you execute the remove at the end of conf.py? Does it execute too early? If so is it possible to try and run a thread or use sleep, these are probably harder to pull off.
Sorry I can not be more help.

@FeralBytes Thanks for the tips. I already tried executing remove at the end of conf.py. It executes before the documentation has been built.
With some help, I managed to solve it. Its improvement for ReadTheDocs is here.


Python-docx does not recognise/extract all table in docx

Hi there, I am using python-docx 0.8.11 and Python 3.8.  I am using this code:
dfs = []
for table in document.tables:
    df = [['' for i in range(len(table.columns))] for j in range(len(table.rows))]
    for i, row in enumerate(table.rows):
        for j, cell in enumerate(row.cells):
            if cell.text:
                df[i][j] = cell.text.replace('\n', '')
    dfs.append(pd.DataFrame(df))

However it seems that a lot of tables are not extracted.
Do you have any idea why that might be?
Thank you very much!

Thank you. I will try it

Hi Guys,
Regarding to this issue

Thank you. I will try it

Have you solved the problem? I am having the same issue. I have a docx word file and it contains several tables and library doesnt detect any table.
My function is
`def format_tables_and_paragraphs(self, docfinal):
try:
        # Abre el documento generado
        doc = Document(self.docpath)

        # Recorre todas las tablas del documento
        for table in doc.tables:
            for row in table.rows:
                for cell in row.cells:
                    # Recorre cada párrafo dentro de la celda
                    for paragraph in cell.paragraphs:
                        for run in paragraph.runs:
                            apply_formatting(run)  # Aplica formato basado en prefijos

        doc.save(docfinal)

    except Exception as e:
        tb = traceback.format_exc(1)
        logging.error('ERROR Exception in edit_word the error is: '+repr(e) + " - args - "+repr(tb))
    `

Thanks


Lwaftr refactorings

Cc @kbara for reiew on this one.  See the individual commits and commit notes for best explanations.
So, I don't know which one of these patches pushes us over the limit, but as we saw in #196, I get terrible perf sometimes with this one.  The reason is lots of unroll errors in the trace, culminating in a blacklist:
[TRACE 126 app.lua:236 -> 93]
[TRACE --- lwaftr.lua:415 -- loop unroll limit reached at lib.lua:361]
[TRACE --- lwutil.lua:41 -- leaving loop in root trace at lwaftr.lua:433]
[TRACE --- lwaftr.lua:102 -- blacklisted at lwutil.lua:41]

The real reason is that tail calls in LuaJIT count towards the loop unrolling limit.  Do more than 15 tail calls in a loop and you run afoul of this hazard!  It turns out to be quite easy to do.  If I comment out this bit in lj_record.c:
  /* Note: the new TREF_FRAME is now at J->base[-1] (even for slot #0). */
  /* Tailcalls can form a loop, so count towards the loop unroll limit. */
  if (++J->tailcalled > J->loopunroll)
    lj_trace_err(J, LJ_TRERR_LUNROLL);

then the perf is fine.  However I didn't see these errors on olive -- I suspect that something changed in Snabb between the olive and rambutan releases that in total is pushing us over the limit.  Quite silly.  Cc @lukego because I think he'll be amused (no action needed).  I think for our upcoming milestone, knowing that we don't loop via tail calls, it would be OK to comment out these lines in our LuaJIT, to remove this hazard.

I'm fairly uneasy about not knowing which patch causes this or why, and about modifying a LuaJIT heuristic that shouldn't be directly triggered by any of the patches to work around it. I also have significant disagreements with some of the individual commits, though I like the overall direction of this patch set.
Would you be ok with making this several different PRs, and figuring out which one is triggering the problem? I know that comes with tedious overhead. :/

@kbara If I understand you correctly, you would like for this to be multiple PRs because of a concern about performance.  Is that correct?  I would be OK with splitting the PR for some other reason, but I don't think perf is a good reason.
The argument is, that ever since we rebased to Rambutan I started seeing trace aborts.  There is a simple symptom shown in all these cases: we were hitting the loop unroll limit, but not in a loop.  Indeed we were hitting it at tail calls.  I think between Olive and Rambutan, snabb switch changed more functions to return function calls (i.e. return f()) instead of returning values.  LuaJIT counts tail calls towards the loop unrolling limit, under the pessimistic assumption that the user is looping via tail calls instead of loops.  This is an inappropriate check and we should remove it in the Snabb copy of LuaJIT.  The snabb idiom is not to loop via tail calls, and 15 tail calls should not cause a trace to abort.  Even if there is a loop, the maxrecord limit of traced instructions should be sufficient to catch an out-of-control loop.
So I was seeing aborts when just porting #196 to rambutan.  So it's not a problem in our code or in this PR.  However this PR does add more small functions, with more return foo() idioms, so of course it will run afoul of these artificial, misguided limits.  The solution is to fix those limits.  I'll do so in another PR.

See https://github.com/SnabbCo/snabbswitch/pull/707 for the LuaJIT PR.  I made it a PR onto Snabb because hey, I don't know if this is a general Lua thing or not -- not sure how much I want to argue with upstream :)  Probably should make an upstream PR though.

I verify that the LuaJIT change fixes the performance.  I get 3.50+3.50 MPPS, which is a slight improvement over lwaftr_rambutan.

Nice. I'm still uneasy about this one, but as long as we clearly document we've done it somewhere (in the documentation within the repo so it's hard to forget and easy to look up, LGTM.

Wheeeeeeeeeeeeeeeeeee


Add support for old version document of pulsar-admin

Is your feature request related to a problem? Please describe.
The current pulsar-admin cli document can be generated automatically. For the latest version, you can view it here http://pulsar.apache.org/tools/pulsar-admin/2.6.0-SNAPSHOT. We need to add support for the old version.

Thanks @tuteng for working on this issue

This is merged #6240


Node not found PopupMenu

Godot Engine v3.4.5.stable.official (c) 2007-2022 Juan Linietsky, Ariel Manzur & Godot Contributors.
--- GDScript language server started ---
res://addons/text_editor/TE_TextEditor.gd:99 - Invalid call. Nonexistent function 'info' in base 'RichTextLabel (TE_Console.gd)'.
(Node not found: "Extensions" (relative to "/root/EditorNode/@@592/@@593/@@601/@@603/@@607/@@611/@@612/@@613/@@629/@@630/@@639/@@640/text_editor/c/c/c/file_button/PopupMenu").)
(Node not found: "Extensions" (relative to "/root/EditorNode/@@592/@@593/@@601/@@603/@@607/@@611/@@612/@@613/@@629/@@630/@@639/@@640/text_editor/c/c/c/file_button/PopupMenu").)
I find your plugin very useful，but it doesn't work properly!Can you help me?

I cant create a new Node named PopupMenu, it rename PopupMenu2


Password doesn't autofill when using lastpass and the autofill icon disappears - Reported by @rushatgabhane

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

Action Performed:

Add NewDot credentials to Lastpass.
In login page, click the lastpass icon and autofill.
Click continue.

Expected Result:
User should see password autofilled
Actual Result:
Password doesn't autofill when using lastpass and the autofill icon disappears until you refresh
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.26-0
Reproducible in staging?:  Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Potentially related issues
https://github.com/Expensify/App/issues/7058
https://expensify.slack.com/archives/C01GTK53T8Q/p1641544949112600
https://user-images.githubusercontent.com/43995119/148618276-cd332a22-8f69-454a-93ab-bb7dc9f90c70.mp4
Expensify/Expensify Issue URL:
Issue reported by: @rushatgabhane
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1641438717010000
View all open jobs on GitHub

Triggered auto assignment to @nickmurray47 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Has this always been the case? I'm not quite sure this is newly broken or just something that was never implemented. Would you happen to know @mvtglobally? Will post in Slack channel to ask otherwise.

@nickmurray47 we do not specifically test for 3rd party password management tools during QA. The regular Chrome autofill or IOS app autofill works fine for me.

ok, good to know! I am putting it as a weekly issue then. This can almost be external but I'm interested in taking a look later first.

This was working when @parasharrajat fixed it in a PR about a month ago.
https://github.com/Expensify/App/pull/5275#issuecomment-962638461

Interesting ah, so it did break. Thanks @rushatgabhane.

This issue is now fixed in production and can be closed.
https://user-images.githubusercontent.com/29673073/150040064-4c9c19b4-25d0-4c51-93fc-332166cfd7f4.mp4

nice, closing!

@nickmurray47 I think you forgot to give issue reporting bonus, if it's applicable here😅


Unable to set trivy.command via Helm chart

What steps did you take and what happened:
I want to try the filesystem mode of trivy, and I know from looking at the code in trivy-operator that it should be supported by the operator. But I cannot find a way to configure it using the Helm chart.
What did you expect to happen:
I can configure trivy.command via the Helm chart.
Anything else you would like to add:
This should be an easy fix, but I am unsure if we should use the term command in the Helm chart values. I consider this term more like CLI-wording, but at the same time it will be a bit confusing to alias names. WDYT @chen-keinan?
Environment:

Trivy-Operator version (use trivy-operator version): 0.1.5

@erikgb you should be able to configure fs mode by setting it in trivy-operator-trivy-config with trivy.command: filesystem

@erikgb you should be able to configure fs mode by setting it in trivy-operator-trivy-config with trivy.command: filesystem

@chen-keinan I know, but not available through the Helm chart, ref. the bug title. Also asked on Slack: https://aquasecurity.slack.com/archives/C03L2H45L3H/p1658393524973549?thread_ts=1657901726.745519&cid=C03L2H45L3H

I see but this is a trivy param, not sure it should be part of helm , we are working now for introducing trivy.config.yaml to give all trivy options included in one yaml structure which will be passed to trivy job via configmap. so we could have all options in once place

@chen-keinan Think this one makes sense. We currently allow users for example to change the trivy mode on the helm installation, so it would make sense to allow to change the command too, even if later we move to the configmap.
https://github.com/aquasecurity/trivy-operator/blob/main/deploy/helm/values.yaml#L126

@chen-keinan Think this one makes sense. We currently allow users for example to change the trivy mode on the helm installation, so it would make sense to allow to change the command too, even if later we move to the configmap.
https://github.com/aquasecurity/trivy-operator/blob/main/deploy/helm/values.yaml#L126

Note sure about is as Mode is not a trivy param while fs is , therefore I think its place is with the params

Note sure about it as Mode is not a trivy param while fs is , therefore I think its place is with the params , we can externalised many of trivy params to helm not sure its right

Gotcha. Interesting semantics. I see them as the same because, mode control if you use a trivy flag --server or not, on the operator mode decides if you get getPodSpecForStandaloneMode or getPodSpecForClientServerMode. While command control if you get the two options mentioned (depending on mode) orgetPodSpecForStandaloneFSMode.
At least for me they feel very similar to each other, one controlling a command, and another a flag.
And thinking about it mode was controlling a command not to long ago, because the way trivy would access a server before was trivy client and not trivy image

@chen-keinan Thinking about it, and maybe we should not have command at all, because we don't support all trivy commands, we support different ways of running trivy. Maybe mode should support 3 options Standalone, ClientServer and Filesystem, each configure trivy to run on a different way. WDYT?

Not sure,  it sound wired , Standalone or server mean where the Vuln DB is located (locally or remote) , Filesystem is scanning param , so there could be many combination as Standalone Image , Standalone Fs , Server Image and Server fs .IMO how trivy  operate,  FS or Image its Trivy config and should not be as part of helm chart .

@josedonizetti Thinking  about it again  , I agree with you that this FS param can be and should exposed via helm chart.


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 21a7ef7:
:clock10: Full build: the average full build time is 7 min(s) 39 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 21a7ef7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Issue on pricecatcher dataset

I have a question on the PriceCatcher dataset. I realize that some items such as "Roti Canai" are not included in the list of items for all PriceCatcher in 2023. I can only find it in the PriceCatcher dataset for Jan-2022. May I know why some data are not provided?
Thank you.

Automate Regular Updates for PriceCatcher Dataset
Steps to Implement Automated Update Process for PriceCatcher Dataset:

Assessment of Current Workflow:

Review the current process in place for updating the PriceCatcher dataset.
Identify manually intensive tasks and areas prone to delays or errors.

Source and API Evaluation:

Validate the sources from which the PriceCatcher data is gathered.
Check if these sources provide APIs or automated data feeds that can be accessed programmatically.
Ensure the APIs or data feeds offer reliable, real-time, or periodically updated data.

Automation Strategy Development:

Design a workflow for data extraction, transformation, and loading (ETL) that can run on a schedule.
Choose appropriate tools for automation (e.g., Python scripts, cron jobs, ETL tools like Apache NiFi, or cloud solutions like AWS Lambda).

Script Development:

Write scripts to fetch data from the identified APIs or data feeds.
Ensure the scripts include error handling and logging mechanisms.
Transform raw data into the required format and ensure consistency.

Data Validation and Integrity Checks:

Implement validation checks to compare incoming data with historical data to identify anomalies.
Cleanse the data to remove duplicates and correct inconsistencies.

Scheduling and Execution:

Set up a scheduling mechanism (e.g., cron jobs if on a UNIX-based system, or Task Scheduler for Windows) to run the update scripts at regular intervals.
Check the scheduling frequency required to keep the dataset up-to-date without overloading the system or hitting API rate limits.

Testing and Monitoring:

Conduct thorough testing in a staging environment to ensure the automation works as expected without disrupting existing operations.
Monitor the automated process regularly to ensure it continues to function correctly, and set up alerts for any failures or issues.

User Notification and Documentation:

Notify stakeholders of the changes and how they enhance dataset accuracy and reliability.
Update the repository documentation to reflect the automated update process, providing details on how the system works and where stakeholders can find the updated data.

Continuous Improvement:

Collect feedback from users to identify any issues or areas for further improvement.
Periodically review and refine the automation process to maintain optimal performance and data accuracy.

Example Implementation Using Python and Cron Jobs:

Data Fetching Script (update_pricecatcher.py):
import requests
import json
from datetime import datetime

API_URL = "https://api.example.com/pricecatcher"
DATA_FILE_PATH = "path/to/pricecatcher_data.json"

def fetch_data():
    response = requests.get(API_URL)
    if response.status_code == 200:
        return response.json()
    else:
        raise Exception("Failed to fetch data. Status Code: {}".format(response.status_code))

def validate_data(data):
    # Implement data validation logic
    return True

def save_data(data):
    with open(DATA_FILE_PATH, 'w') as f:
        json.dump(data, f, ensure_ascii=False, indent=4)

def main():
    try:
        data = fetch_data()
        if validate_data(data):
            save_data(data)
            print("Data updated successfully at:", datetime.now())
        else:
            print("Data validation failed.")
    except Exception as e:
        print("Error during update:", e)

if __name__ == "__main__":
    main()

Setting Up Cron Job:

Open the crontab editor by running crontab -e in the terminal.
Add the following line to schedule the script to run daily at midnight:0 0 * * * /usr/bin/python3 /path/to/update_pricecatcher.py

Save and exit the editor. This will automate the update process to run at the specified frequency.

By following these steps, the PriceCatcher dataset will be updated regularly and automatically, ensuring that users always have access to the most current market prices, thereby improving the dataset’s reliability and accuracy.


init_new_gem: Update comment on manifest.yaml

It's better to describe the manifest.yaml should have a list of "stdlib" gems.

I think manifest.yaml has to allow a gem as a dependency. https://github.com/ruby/rbs/issues/1538

I'll close it because of a lack of activity.


perf: Ont downsampling normalization

Description
downsampling and clipping of ONT reads handled by notramp
Related Issue

Checklist

[x] I've formatted the PR title in accordance with the structure of the conventional commit messages.
[x] I've updated the code style using pre-commit if needed.
[x] I've updated or supplemented the documentation as needed.
[x] I've read the CODE_OF_CONDUCT.md document.
[x] I've read the CONTRIBUTING.md guide.

Seems like NanoFilt doesn't work: https://github.com/IKIM-Essen/uncovar/runs/5407712905?check_suite_focus=true#step:11:839

There is a new snakemake release: https://github.com/snakemake/snakemake/releases/tag/v7.2.1
Lets hope that fixes the report problem.