Segfault when using connection specifier

I'm getting a segfault when I attempt to use a source and a connection through the CLI. The connection errors whenever I use either a valid or invalid connection. I get no errors when only specifying the source. Link to line segfaulting here
hookdeck listen 8080 test testconn
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x8 pc=0x10040af84]

goroutine 1 [running]:
github.com/hookdeck/hookdeck-cli/pkg/listen.getConnections(0x140001dedc0, 0x14000244960, {0x16fdbe85b, 0x8})
	/Users/runner/work/hookdeck-cli/hookdeck-cli/pkg/listen/connection.go:41 +0x2a4
github.com/hookdeck/hookdeck-cli/pkg/listen.Listen(0x140001483f0, {0x16fdbe855, 0x5}, {0x16fdbe85b, 0x8}, {0x64?}, 0x1008a7180)
	/Users/runner/work/hookdeck-cli/hookdeck-cli/pkg/listen/listen.go:54 +0xa8
github.com/hookdeck/hookdeck-cli/pkg/cmd.(*listenCmd).runListenCmd(0x1400015e820, 0x1001d2710?, {0x140001d0f90, 0x140001b3bb8?, 0x0?})
	/Users/runner/work/hookdeck-cli/hookdeck-cli/pkg/cmd/listen.go:115 +0x194
github.com/spf13/cobra.(*Command).execute(0x140001e0000, {0x140001d0f60, 0x3, 0x3})
	/Users/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:842 +0x4c4
github.com/spf13/cobra.(*Command).ExecuteC(0x10089dec0)
	/Users/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:950 +0x34c
github.com/spf13/cobra.(*Command).Execute(...)
	/Users/runner/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:887
github.com/hookdeck/hookdeck-cli/pkg/cmd.Execute()
	/Users/runner/work/hookdeck-cli/hookdeck-cli/pkg/cmd/root.go:46 +0x34
main.main()
	/Users/runner/work/hookdeck-cli/hookdeck-cli/main.go:21 +0x20```

@alexluong could you take a look at this, please?

This might be an error on my part (let me know if it is), but a better error message would be great if it is! Thanks

@baileywickham could you please confirm the version of the CLI you are running?
https://github.com/hookdeck/hookdeck-cli?tab=readme-ov-file#version

Thanks for the quick response.
❯ hookdeck version
hookdeck version 0.10.0
A newer version of the Hookdeck CLI is available, please update to: v0.9.3

Installed via brew on macos

I think this should be fixed by #83. This issue is because the source has a nameless connection. I assume this is the case, @baileywickham?

I thought my connection had a name, am I missing something?
I also thought it might be related to #76, but I'm on v0.10.0.
If you want I can pull and test #83

I see what you mean, one of the other connections was nameless, not bwtesting. Naming that fixed the error. Feel free to close once #83 is merged. Thanks for the quick responses.

Thanks for the quick response.
❯ hookdeck version
hookdeck version 0.10.0
A newer version of the Hookdeck CLI is available, please update to: v0.9.3

Installed via brew on macos

Thanks for this. This flagged that we published a version on GitHub out of order.
I've just triggered the release of v0.10.1 which contains #83.

Thanks, I can confirm that v0.10.1 fixed my issue.

Thanks, @baileywickham 🙌


Unable to install Angular CLI

Command
other
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Currently unable to install the Angular CLI globally, through the documented command: npm install -g @angular/cli
Issue started occuring May 1st, 22:25 UTC, or that's when my team first noticed the error in our pipelines.
Since then we've tested on multiple node versions and OS's.
Minimal Reproduction
After installing Node + NPM, run: npm install -g @angular/cli
Exception or Error
This output is from my local test environment.
 WARN deprecated<EMAIL_ADDRESS>This functionality has been moved to @npmcli/fs
npm ERR! code 127
npm ERR! git dep preparation failed
npm ERR! command /Users/username/.nvm/versions/node/v18.15.0/bin/node /Users/username/.nvm/versions/node/v18.15.0/lib/node_modules/npm/bin/npm-cli.js install --force --cache=/Users/username/.npm --prefer-offline=false --prefer-online=false --offline=false --no-progress --no-save --no-audit --include=dev --include=peer --include=optional --no-package-lock-only --no-dry-run
npm ERR! npm WARN using --force Recommended protections disabled.
npm ERR! npm ERR! code 127
npm ERR! npm ERR! path /Users/username/.npm/_cacache/tmp/git-clonefXNQwG
npm ERR! npm ERR! command failed
npm ERR! npm ERR! command sh -c npm run compile
npm ERR! npm ERR! ><EMAIL_ADDRESS>precompile
npm ERR! npm ERR! > rimraf build
npm ERR! npm ERR! npm WARN using --force Recommended protections disabled.
npm ERR! npm ERR! sh: rimraf: command not found
npm ERR!
npm ERR! npm ERR!      /Users/username/.npm/_logs/2023-05-02T00_59_30_442Z-debug-0.log

npm ERR!      /Users/username/.npm/_logs/2023-05-02T00_59_25_854Z-debug-0.log

Your Environment
This output is from my local test environment.
Node: v18.15.0
npm: v9.6.2

Anything else relevant?
No response

Looks like this issue got submitted just a minute before mine: https://github.com/angular/angular-cli/issues/25110


[feature]: Enable sending of multiple different assets with one send transaction

Is your feature request related to a problem? Please describe.
When sending assets, all Taproot Asset addresses must be of the same asset ID (see e.g. code comment in rpcserver.go). Therefore, sending many assets would necessitate making many BTC on-chain transactions with infeasibly high transaction costs.
Describe the solution you'd like
Ideally, I would like to send many different unique assets (e.g. 100k collectibles) with one send transaction and, respectively, one BTC on-chain transaction. When I asked about this feature on Slack some time ago, @guggero suggested that it involves anchoring multiple vPSBTs into a single transaction for which everything should be in place.

tapd currently supports anchoring one vpsbt only, see: https://github.com/lightninglabs/taproot-assets/blob/76bb5ae01fc0aa147233f332c468943f8c47b053/rpcserver.go#L1750-L1750
I'll make an issue to enable multi vpsbt anchoring. I don't think an issue currently exists for that.


Structured kubeconfig errors

This makes the errors introspectable, which makes things like https://github.com/yaahc/color-eyre/ much nicer. It also allows you to inspect the error contents (though this API probably shouldn't be considered very stable..).

Although fails when building with rustls on ci ^

Released in 0.34.0.


Weak security with AES/CBC/PKCS7Padding

Hi Mr.juliansteenbakker
I found a problem when running the security testing tool

The code that causes the problem is here
protected Cipher getCipher() throws Exception {
	return Cipher.getInstance("AES/CBC/PKCS7Padding");
}

The problem with CBC mode is the padding. When there is a padding error, the server must respond to a message back to you so that you can send the message back again or encrypt the message from the beginning.
The padding oracle attack is solely based on this idea. The attacker changes the byte and looks at the response of the server to execute the attack.
CBC mode has no integrity and authentication like all basic.
I recommend you replace “AES/CBC/PKCS7Padding” with “AES/GCM/NoPadding” to improve the security of your package.
Please follow here to get information about “AES/CBC/PKCS7Padding”
I hope you will update this in the next version.
Many thank.

@volinhkhanh do you receive an answer for this? @mogol can you help understand it?

This will help https://www.youtube.com/watch?v=JogvnnaJ8TI

@mnnlthmpsn Did you end up using a different plugin for storing secrets? (I also encounter this issue in our security scan tool)

I also found this link: https://docs.flutter.dev/reference/security-false-positives with this information:
CBC with PKCS5/PKCS7 padding vulnerability
We have received vague reports that there is a “CBC with PKCS5/PKCS7 padding vulnerability” in some Flutter packages.
As far as we can tell, this is triggered by the HLS implementation in ExoPlayer (the com.google.android.exoplayer2.source.hls.Aes128DataSource class). HLS is Apple’s streaming format, which defines the type of encryption that must be used for DRM; this isn’t a vulnerability, as DRM doesn’t protect the user’s machine or data but instead merely provides obfuscation to limit the user’s ability to fully use their software and hardware.

static AndroidOptions _getAndroidOptions() => const AndroidOptions(
encryptedSharedPreferences: true,
keyCipherAlgorithm: KeyCipherAlgorithm.RSA_ECB_OAEPwithSHA_256andMGF1Padding,
storageCipherAlgorithm: StorageCipherAlgorithm.AES_GCM_NoPadding,
);
static final storage = FlutterSecureStorage(aOptions: _getAndroidOptions());
static writeSecureData(String key, String value) async {
await storage.write(key: key, value: value);
}
static Future readSecureData(String key) async {
return await storage.read(key: key) ?? 'No data found';
}
I am using this code but still get the same issue.


Wasmplugin cache expired cause LDS update failed

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
I created a CLIENT mode wasmplugin, and  waited until the wasm module cache expired. Then I changed the wasmplugin mode to CLIENT_AND_SERVER. This cause LDS update failed.
A more common case is changing the Telemetry config after wasm cache expired, which cause listener rebuild. Then LDS update failed.
Version
istiod 1.22.2

Additional Information
I've seen the discussion in #42154 and have some doubts about current implementation.
Why we cache wasm modules in filesystem and then fill filename in ecds?
Maybe use inline_bytes can avoid this error.

cc @kyessenov @ingwonsong

You can avoid caching in the agent by using remote data source and/or local file system mounts for provide the module.


Also read stderr

Nevermind, this is actually not really possible with unix pipes without changing the pipe.
But we can update the README to use |& (and cmd 2> >(filter >&2))


Request for joining the organisation

Hello, my name is Rituraj. I am from India.
I would like to join this organization.
I love to contribute to open source and will love to contribute to this organization.
Please add me to your organization.

@amanjainn please add them to Jobream.

@riturajjain2000 Check invitation


Correct event gen, take two

Replaces #1759
Fixes #1526, tweak generation for sponge event factory
Signed-off-by: liach<EMAIL_ADDRESS>

Note: This PR was automatically closed because the 1.13 branch was deleted. It will need to be reopened to target api-8 instead.  See https://github.com/SpongePowered/SpongeAPI/pull/1428#issuecomment-539689855.


Support for TabLayout from Android Design Support library.

Thank you for this amazing library.
I am having an issue with the TabLayout widget introduced in the new Design Support Library launched by the Android team. The textviews in the tabs are not responding to the font change.
I have followed the steps from http://chrisjenx.com/calligraphy-2-0-0/. I am using v2.1.1 of your library. Am I missing something?

See #176. I have not Inflation hook. No typeface is applied to the Layout. You need to whine at Goog. @cb

Thanks @chrisjenx. I have solved the issue with a little bit of code. Do I post the solution here?

Sure, would be interested how you worked around it.
On Mon, 8 Jun 2015 08:06 Awanish Raj<EMAIL_ADDRESS>wrote:

Thanks @chrisjenx https://github.com/chrisjenx. I have solved the issue
with a little bit of code. Do I post the solution here?
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/180#issuecomment-109885751
.

This solution picks up the titles from the ViewPager's adapter automatically. The text highlighting is handled by the support library itself.
Create following 3 files
res/layout/item_tab.xml
<TextView
	xmlns:android="http://schemas.android.com/apk/res/android"
	android:layout_width="match_parent"
	android:layout_height="match_parent"
	android:textColor="@drawable/sel_tab_text"
	/>

res/drawable/sel_tab_text.xml
<selector xmlns:android="http://schemas.android.com/apk/res/android">
	<item android:state_selected="true" android:color="@color/tab_text_selected" />
	<item android:state_focused="true" android:color="@color/tab_text_selected" />
	<item android:state_pressed="true" android:color="@color/tab_text_selected" />
	<item android:color="@color/tab_text_unselected" />
</selector>

The colors tab_text_selected and tab_text_unselected shall be appropriately defined in colors.xml
Utils.java
public static void applyFontedTab(Activity activity, ViewPager viewPager, TabLayout tabLayout) {
    for (int i = 0; i < viewPager.getAdapter().getCount(); i++) {
        TextView tv = (TextView) activity.getLayoutInflater().inflate(R.layout.item_tab, null);
        if (i == viewPager.getCurrentItem()) tv.setSelected(true);
        tv.setText(viewPager.getAdapter().getPageTitle(i));
        tabLayout.getTabAt(i).setCustomView(tv);
    }
}

Usage
In TabActivity.java
//After calling tabLayout.setupWithViewPager(viewPager);
Utils.applyFontedTab(TabActivity.this, viewPager, tabLayout)

Yeah, inflating into the custom view would work. Would be nice if you could
define a custom view when creating the Tab.
On Mon, 8 Jun 2015 at 08:43 Awanish Raj<EMAIL_ADDRESS>wrote:

This solution picks up the titles from the ViewPager's adapter
automatically. The text highlighting is handled by the support library
itself.
Create following 3 files
res/layout/item_tab.xml

res/drawable/sel_tab_text.xml

The colors tab_text_selected and tab_text_unselected shall be
appropriately defined in colors.xml
Utils.java
public static void applyFontedTab(Activity activity, ViewPager viewPager, TabLayout tabLayout) {
for (int i = 0; i < viewPager.getAdapter().getCount(); i++) {
TextView tv = (TextView) activity.getLayoutInflater().inflate(R.layout.item_tab, null);
if (i == viewPager.getCurrentItem()) tv.setSelected(true);
tv.setText(viewPager.getAdapter().getPageTitle(i));
tabLayout.getTabAt(i).setCustomView(tv);
}
}
Usage
In TabActivity.java
//After calling tabLayout.setupWithViewPager(viewPager);
Utils.applyFontedTab(TabActivity.this, viewPager, tabLayout)
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/180#issuecomment-109893974
.

Actually worth noting, setCustomView <https://developer.android.com/reference/android/support/design/widget/TabLayout.Tab.html#setCustomView(int)>(int layoutResId) should work.
On Mon, 8 Jun 2015 at 09:11 Christopher Jenkins <
<EMAIL_ADDRESS>wrote:

Yeah, inflating into the custom view would work. Would be nice if you
could define a custom view when creating the Tab.
On Mon, 8 Jun 2015 at 08:43 Awanish Raj<EMAIL_ADDRESS>wrote:

This solution picks up the titles from the ViewPager's adapter
automatically. The text highlighting is handled by the support library
itself.
Create following 3 files
res/layout/item_tab.xml

res/drawable/sel_tab_text.xml

The colors tab_text_selected and tab_text_unselected shall be
appropriately defined in colors.xml
Utils.java
public static void applyFontedTab(Activity activity, ViewPager viewPager, TabLayout tabLayout) {
for (int i = 0; i < viewPager.getAdapter().getCount(); i++) {
TextView tv = (TextView) activity.getLayoutInflater().inflate(R.layout.item_tab, null);
if (i == viewPager.getCurrentItem()) tv.setSelected(true);
tv.setText(viewPager.getAdapter().getPageTitle(i));
tabLayout.getTabAt(i).setCustomView(tv);
}
}
Usage
In TabActivity.java
//After calling tabLayout.setupWithViewPager(viewPager);
Utils.applyFontedTab(TabActivity.this, viewPager, tabLayout)
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/180#issuecomment-109893974
.

The tabs are created internally with tabLayout.setupWithViewPager(viewPager);
After this, using the setCustomView with resId does not retain the text in the tabs. This is why I had to inflate a TextView and setText before setting the custom view.
Thanks for being so prompt. I appreciate your work. All the best.

I'm going to add a few things to enable this in the next release.
First, TabLayout will inflate it's TextView and ImageView from layout resources.
Second, you can provide your own layouts.

@chrisbanes cheers.

That would allow the default font to be applied correctly (win).
Then we can define the fontPath. (double win!)

alt strategy in your PagerAdapter:
@Override public CharSequence getPageTitle(int position) {
  CharSequence text = ...;
  CalligraphyTypefaceSpan titleType = ...;
  SpannableStringBuilder s = new SpannableStringBuilder(text);
  s.setSpan(titleType, 0, text.length(), 0);
  return s;
}

Actually that's a better way if using the view pager.
On Fri, 10 Jul 2015 01:52 Daniel McCartney<EMAIL_ADDRESS>wrote:

alt strategy in your PagerAdapter:
@Override public CharSequence getPageTitle(int position) {
CharSequence text = ...;
CalligraphyTypefaceSpan titleType = ...;
SpannableStringBuilder s = new SpannableStringBuilder(text);
s.setSpan(titleType, 0, text.length(), 0);
return s;
}
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/180#issuecomment-120187888
.

Both workarounds not working since I've upated design support library and appcompat-v7 to 22.2.1 version (from 22.2.0).
@awanishraj workarounh with customView results in empty Tabs in TabLayout.
@dmccartney workaround results in not applying Calligraphy font to pageTitle. TabLayout displays it with default system font. I've checked and Typeface for TypefaceSpan is created correctly.

I'll look at the source code diff and see what's changed; @chrisbanes said
they were going to improve this.
So hopefully there is a more elegant solution. :)
On Wed, 29 Jul 2015 09:36 Paweł Urban<EMAIL_ADDRESS>wrote:

Both workarounds not working since I've upated design support library and
appcompat-v7 to 22.2.1 version (from 22.2.0).
@awanishraj https://github.com/awanishraj workarounh with customView
results in empty Tabs in TabLayout.
@dmccartney https://github.com/dmccartney workaround results in not
applying Calligraphy font to pageTitle. TabLayout displays it with default
system font. I've checked and Typeface for TypefaceSpan is created
correctly.
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/180#issuecomment-125881122
.

OK, so I'm looking forward for some elegant solution :)
Greetings.

Cool, my solution is giving empty tabs quite a few times now. I will also try to workaround this. Thanks!

What I said earlier is true.

The default TextView is now inflated via a layout.
For any custom view you provide for the Tab, TabLayout will try to pick out a TextView via findViewById(android.R.id.text1)

@awanishraj empty tabs is a design lib bug.

@chrisbanes In what version of the Design Support Library can this new functionality be found? Is it still unreleased?

@onyxmueller this is in 22.2.1, Just checked.
https://code.google.com/p/android/issues/detail?id=180462 might be related to the empty tabs.
/cc @awanishraj @papcio28
I had to do the above work around for the TabLayout apart from that it's working in a few app's for me.

how to change font color of customlayout on tabselected ...... using design layout tabs @chrisbanes @onyxmueller

I'm not giving any support to this stuff here.

This is not the place for that question. Go to stack overflow or Android
developers gplus.
On Thu, 13 Aug 2015 11:11 erumhannan<EMAIL_ADDRESS>wrote:

how to change font color of customlayout on tabselected ...... using
design layout tabs @chrisbanes https://github.com/chrisbanes
@onyxmueller https://github.com/onyxmueller
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/180#issuecomment-130587000
.

Sorry to open this thread again, I think this might be helpful
@Override
public CharSequence getPageTitle(int position) {
String title = mFragmentTitleList.get(position);
CalligraphyTypefaceSpan typefaceSpan = new CalligraphyTypefaceSpan(TypefaceUtils.load(activity.getAssets(), "fonts/OpenSans-Semibold.ttf"));
SpannableStringBuilder s = new SpannableStringBuilder();
s.append(title);
s.setSpan(typefaceSpan, 0, title.length(), Spanned.SPAN_EXCLUSIVE_EXCLUSIVE);
return SpannableString.valueOf(s);
}
I am using tablayout with view pager and it worked for me.

@srinivaskrify I don't think that is working. I did the same but thing but the font isn't being changed

It worked for me @sreejithraman
see screenshot attached below.
I am using font opensans-regular

@scottyab
Thanks alot, you just saved me so much time.

@scottyab your code work fine and the custom font  was applied, but the problem is the title won't change anymore, it's always the same color whether it's selected or not despite It does apply different color for the selected item when I remove your code.

Hmm apply alpha to text while tab getting changed
On 15 Aug 2016 3:03 p.m., "Haroun SMIDA"<EMAIL_ADDRESS>wrote:

@scottyab https://github.com/scottyab your code work fine and the
custom font was applied, but the problem is the title won't change anymore,
it's always the same color whether it's selected or not despite It does
apply different color for the selected item when I remove your code.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/180#issuecomment-239763641,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHWACIIs5iSo1kJn64yuEcPC8S3G3Gt-ks5qgDJZgaJpZM4E6UdP
.

+1 to the comment from @smidaharoun. I found the workaround of using custom views independently, which makes the custom font work - but at the cost of losing the behavior of changing colors based on selection.
Has anyone figured out the magic combination of themes, defaults, and hacks to make this "just work" yet? :-)

Ah, I can answer my own question. The "simplest" way I've found to fix this is to extend TabLayout and override the "addTab" method. At this point, you can reach in and set the typeface on textviews.
You can see my implementation of this here: https://gist.github.com/tmtrademarked/09926077a406959be15fc8a824a52751

Here my solution bases on @tmtrademarked work: https://gist.github.com/LuigiPapino/4612cddbb8fef25cb419c0a9a3a86d84
It allow to specify the font in the fontPath of the TabLayout and use calligraphy to apply it in the TextView

Great workarounds. Thanks @tmtrademarked!
I whipped up a slightly adjusted version that just uses the default Calligraphy font if anyone is in the same position:
https://gist.github.com/timkrins/030ff266d90788408bce0b6e7bbca8fd


Scope to packages inside a conda environment

@raimon49 thanks for this awesome project!
I'd like to request a feature...could you make it possible to exclude globally-visible packages if you are inside a conda environment?
For my use case, I'd like to use pip-licenses in CI to catch dependencies with problematic licenses. However, right now I have to manually exclude a lot of stuff because things installed in my local site-packages are making it through.
Basically, I'm asking if pip-licenses could replicate whatever pip freeze --local does.

@jameslamb Thank you for the feature request.
I currently don't use the conda environment. However, if I can reproduce your conda environment, I may be able to implement this feature request.
Please comment on the information to reproduce your conda environment.

Python version
Anaconda version
And more


add main to package.json for compatibility

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:


Handle versioning in client-common

:tada: This issue has been resolved in version 1.0.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[SMALLFIX] Small fixes in several classes

Fixed documentation in OpenFileEntry
Removed not needed cast in ResourcePoolTest
Collapsed multiple catch statements into one in TieredBlockStore
Used foreach instead of for in TieredBlockStoreTestUtils
Fixed documentation in TtlIntervalRule

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11223/
Test PASSed.

left one minor comment

after fixing the minor comment from Jiri, LGTM
thanks for keeping improving the code health!

@jsimsa Good catch and thanks for the comment. I updated the PR accordingly.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11236/
Test PASSed.

LGTM


Exception of type 'Open.Nat.NatDeviceNotFoundException' was thrown.

Here's my code:
using System;
using System.Net;
using System.Threading;
using Open;
using Open.Nat;

namespace CPortOpener
{
	class MainClass
	{
		public static void Main (string[] args)
		{
			openPort ();
			Console.ReadLine ();
		}

		public static async void openPort ()
		{
			try
			{
				var discoverer = new NatDiscoverer();
				var cts = new CancellationTokenSource(15000);
				var device = await discoverer.DiscoverDeviceAsync(PortMapper.Upnp, cts);

				await device.CreatePortMapAsync(new Mapping(Protocol.Tcp, 1600, 1700, "The mapping name"));
			}
			catch (Exception ex)
			{
				Console.WriteLine (ex.Message);
			}
		}
	}
}

I'm getting this Exception:
Exception of type 'Open.Nat.NatDeviceNotFoundException' was thrown.

@Mmokhtar that means it wasn't able to find any UPnp NAT device in your network. If you are sure it should have found one, please share the logs with me and i will review it asap.

Hello again Mr Ontivero,
I'm getting same error when I start to app with VS. But most of time when I start to exe on bin/debug/ConsoleTest.exe, app found device with there is no throwing errors.
I don't know why cannot found on VS but maybe you need check something Async Functions. Maybe some breakpoints could be causing this.
Have a good day, good works...


handling none ijar case

If ijar is none that means dep target does not contain ijar.
This happens to us when working on intellij integration (which we'll contribute back in a few weeks).
Essentially the intellij plugin needs the scala_import rule to set ijar as None so that it will take the class_jar. This is important for the whole macros handling.
This is also why there isn't a test for this right now.
This will be covered when we contribute back the scala_import
@natansil I'd appreciate your review

👍  Looks good.
I think breaking it into two lines will be even better.


Exposing NewCursor to allow plugin to add multiple cursor in custom Loc

It is very difficult to add new cursor from a plugin standpoint since NewCursor() is not exposed.
While this can be worked around by manipulating the cursor after calling SpawnMultiCursorDown(), it is pretty much impossible to retain selections from previous cursors.
This PR allows the plugin to add new cursor without any limitations

There is another pull request where NewCursor is exposed but it is not merged. I do not usually use multiple cursors so I do not know what resolution would be good, but there is a comment where other things that may be done are written: https://github.com/zyedidia/micro/pull/2941#issuecomment-2028088952

@niten94
Yeah, I just noticed it, thanks.
@dmaluka

Still, what about my question #2920 (comment)?
I.e. isn't NewCursor() too low-level a detail to be exposed to plugins? Perhaps the needed functionality can be already achieved from Lua with existing BufPane methods like SpawnMultiCursor()?

SpawnMultiCursor() hardcodes it multi cursor to be the selected word and there's no way to customize it. I don't see how it can be used by plugin to insert cursors in custom places. And other multi-cursor functions deselect existing selections.

Or if those methods are not quite suitable since they do "too much", perhaps we should just add a new simple method to BufPane or to Buffer, which would, roughly speaking, just do NewCursor() + AddCursor() + MergeCursors() and nothing else?

I personally think it doesn't hurt to expose a function that creates a Cursor struct, which can be used in many other places, other scenarios that we might not foresee. And the plugin developer can already look at how NewCursor() is used in the source code which is pretty clear on how it is "meant" to used I think.
That being said, I don't mind having a dedicated function just to allow plugins to add a cursor to the buffer if you want it.

I personally think it doesn't hurt to expose a function that creates a Cursor struct, which can be used in many other places, other scenarios that we might not foresee. And the plugin developer can already look at how NewCursor() is used in the source code which is pretty clear on how it is "meant" to used I think.

So if later for some reason we want to change the way how it is done in the source code, we cannot just do that, since there are already plugins that already do the same, so we would need to update those plugins as well.
Things are already quite messy due to the fact that micro's plugin system doesn't clearly define which APIs are supposed to be used by plugins (and thus are expected to preserve backward compatibility, etc) and which are only supposed to be used internally by micro. So let's not make it even messier.

@dmaluka
Yeah fair enough, I get what you mean.
I added a function that adds a cursor to a buffer and reverted the previous changes.


[DC-1347] Create questionnaire_response_id mapping rule

Created questionnaire_response_id_map.py module which:

Creates the _deid_questionnaire_response_map lookup table if it does not already exists which populates with the questionnaire_response_id column from the observation table and generates a random value for the research_response_id column.
Performs the QRID to RID mapping by replacing the QRID in the observation table with the RID found in the _deid_questionnaire_response_map lookup table.

Unit test created to test the query generation in questionnaire_response_id_map.py module

I was able to modify the query to start the research_response_id values at 1mil as well as fixed the branch conflicts!


Ignore symbols from code.

Given
// misspelledd
void misspelledd();

Current behavior
"misspelledd" is flagged as spelling error.
Desired behavior
"misspelledd" is not flagged because it is a symbol in the code.

It has the symbols but there is no API that returns true if a string is a defined symbol. Therefore we would need to implement that by indexing all the symbols treesitter has parsed. If we don't index then we'd need to loop through all the symbols until we get a match for every word we want to spell check.


Frontend with Framworks - 2.3 Functional Components

Description 📜
add proper documentation on Functional Components in React.

Domain of Contribution 📊

[ ] Android Dev (Flutter, Java, Kotlin)
[ ] Backend Dev (Java, .NET, Python)
[ ] C/CPP
[ ] Competitive Programming
[ ] Cyber Security
[ ] DSA
[ ] Database
[ ] Datascience with Python
[ ] Datascience with R
[ ] Frontend Dev HTML/CSS/JS
[×] Frontend Dev React/Angular/Vue
[ ] Golang
[ ] Interview Prep
[ ] Java_Domain
[ ] JavaScript
[ ] MERN
[ ] Machine Learning
[ ] Open Source
[ ] Python_Domain
[ ] Rust
[ ] Statistics
[ ] UI/UX

/assign


How can multiple people take part in a lottery?

Closing this as it’s a discussion more than an issue. Basically as is, many people can enter the pottery when it’s started.

When I try to enter the lottery from my other account on rinkeby then the players array does not get updated which means the new player has not been added or am Imissing something.


vector vs raster rendering

Fallback to raster rendering should happen when the number of vertices we need to render it's too hight for the client making it unresponsive.
To start with a basic if (number_of_vertices > 1e5) could do the trick.
ref https://github.com/CartoDB/d3.cartodb/issues/165
cc @saleiva @fdansv

@rochoa shouldn't this be in cartodb.js?

I don't know: https://github.com/CartoDB/d3.cartodb/issues/165#issuecomment-216915600
Feel free to move it if you know more than me 😉 .

yes please!
2016-05-04 18:13 GMT+02:00 Raul Ochoa<EMAIL_ADDRESS>
I don't know: CartoDB/d3.cartodb#165 (comment)
https://github.com/CartoDB/d3.cartodb/issues/165#issuecomment-216915600
Feel free to move it if you know more than me 😉 .
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/deep-insights.js/issues/339#issuecomment-216916149

--
Sergio Alvarez Leiva


Wrap block/position and from/nonce details

Closes #2380.
Allows the Block/Position and From/Nonce transaction detail rows to wrap.
Enough room:

Compressed:

also, as seen in the original issue screenshots, I think we should (independently of wrapping of not) make the individual components more resilient by:

the "<<", ">>" navigation buttons should be made "not wrappable" regardless of their size
the same for the "N confirmations" and "N / N" boxes

of course that would not solve this issue, it would actually make the entire row overflow or wrap by itself in this case, but I think it is worth it to apply those fixes as well.

I have implemented everything listed except hiding the vertical divider when it is part of the first element in a new flex row. I spent a while trying to find an alternative that doesn't resort to using media queries for the viewport width, but there is no easy way of doing it. If it's any consolation, other top block explorers also have the vertical divider separating items between rows.
New vertical spacing example:

I have implemented everything listed except hiding the vertical divider when it is part of the first element in a new flex row. I spent a while trying to find an alternative that doesn't resort to using media queries for the viewport width, but there is no easy way of doing it. If it's any consolation, other top block explorers also have the vertical divider separating items between rows.
New vertical spacing example:

they are all ugly IMHO :)
but let's merge it, we can change it later in case of better ideas


Replace response header is case sensitive

Hi,
Method replaceHeader should be case insensitive but is currently case sensitive.
Example :

e.getHttpResponse().replaceHeader("content-length", "1...") doesn't replace the header
e.getHttpResponse().replaceHeader("Content-Length", "1...") replace the header

Regards,

Already fixed in master branch. Waiting for a release.


fix(EventStack): fix compatibility with IE11

Fixes #3043.

What was done?
Move to a separate repo
EventStack was moved to separate package and was published as @semantic-ui-react/event-stack.
Motivation: this package requires a specific test config for browsers. However, I'm not happy with this decision, I prefer monorepos :smiley_cat: Possible, Stardust will adopt it.
Browser compability
EventStack is now fully compatible with IE11. To avoid regresions I added browser tests:

Chrome & Firefox: https://circleci.com/gh/layershifter/event-stack
IE11: https://ci.appveyor.com/project/layershifter/event-stack

Performance :rocket:
I and my сolleagues carefully profiled this part of the lib, performance was drasticaly improved. The most slow was EventSet:

constructor() - almost 10x
addHandlers() - almost 10x
dispatchEvent() - 10x for default pool, 1.5x for named pool
removeHandlers() - almost 5x

Feel free to edit benchmarks and propose more performant solution.

There I also couple serious bugs with EventStack event handling, I will concentrate on them after merge. I'm also planning to move #2990 to this package.

Possible, Stardust will adopt it.

Yes please!

We have a fork of this in Stardust as well.  Yes, I would love to have a single utility here.  It is also duplicated in Fabric (Microsoft).  One of the goals of Stardust is to normalize these kinds of utilities between frameworks.  Let's merge this for now, then we can port it to Stardust and publish it as a monorepo.
There might still be a bug here, not sure.  Here's what I think I've found.  I noticed the bug in #3062 goes away if we fix the sub method.  Currently, the entire pool is removed then re-added on sub.  However, the pool shares handlers with other components under the same pool name.  When a second Dropdown opens it calls sub for close on document click, which removes the entire pool (including the first Dropdown's close on document click), then adds only the second Dropdown's close on document click.
When I use document.addEventListener instead, all works as expected.  When I remove the line in EventStack that removes the entire pool on sub, all works fine as well.  So, I think we need to skip sub if that handler already exists and not remove/add the entire pool.

Released in<EMAIL_ADDRESS>


WIP: Integrating the new k8s-metacollector + k8smeta plugin with falco 0.36.2

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

/area event-generator-chart

/area k8s-metacollector

What this PR does / why we need it:
This PR updates the Falco charts to leverage the new k8s-metacollector and k8smeta plugin.
How to deploy:
# Clone the right branch.
git clone --branch kcl/falco-integrate-k8s-metacollector --single-branch https://github.com/alacuku/charts.git

# Change directory to Falco chart.
cd charts/charts/falco

# Build dependencies.
helm dependency build

# Install falco + k8smeta plugin and the k8s-metacollector in namespace falco.
helm install falco ./ -n falco --create-namespace --set tty=true

# Install falco + k8smeta plugin in namespace falco and the k8s-metacollector in namespace collector.
kubectl create namespace collector
helm install falco ./
    -n falco \
    --create-namespace \
    --set k8s-metacollector.namespaceOverride=collector \
    --set tty=true

Check that the pods are in place:
kubectl get pods -n falco

You should have something like (the number of Falco pods will depend on the number of your nodes):
NAME                                       READY   STATUS    RESTARTS   AGE
falco-dkntd                                2/2     Running   0          27m
falco-k8s-metacollector-674dc59c98-sblwv   1/1     Running   0          27m

NOTE: The new fields exported by the plugins start with prefix k8smeta. The container runtime engines still use the old prefix k8s. The majority of those fields will return <NA>. Please update your old fields to match the ones of the plugin.
Some implementation details of this PR:
The old kubernetes collector has been replaced by the k8smeta plugin. The k8s-metacollector chart has been added as a dependency to the Falco chart and is enabled by default. The Falco and falcoctl configurations are updated at deployment time by a custom helper. It does the following:

Adds the k8smeta plugin to the load_plugins list;
Adds the k8smeta plugin configuration to the plugins section;
Adds the plugin artifact type to the  allowedTypes in the falcoctl configuration;
Adds the pluigin OCI references to the falcoctl install references.

Furthermore, all the resources needed by Falco to connect to the k8s api-server has been removed:

service account;
cluster role;
cluster role binding;
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Checklist

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku
Once this PR has been reviewed and has the lgtm label, please assign issif for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold
The PR is only for testing the new k8s-metacollector and k8smeta plugin.

/close
Not more relevant. See #601

@alacuku: Closed this PR.

In response to this:

/close
Not more relevant. See #601

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Consider replacing included BusyBox 'wget' with Alpine's own

Currently when using a rootless Docker setup to run this action, the image fails to build with the following:
#7 [3/4] RUN wget -O - -q https://raw.githubusercontent.com/reviewdog/reviewdog/master/install.sh | sh -s -- -b /usr/local/bin/ v0.18.0 &&     mkdir -p /opt/lib &&     wget -q -O /opt/lib/checkstyle.jar https://github.com/checkstyle/checkstyle/releases/download/checkstyle-10.17.0/checkstyle-10.17.0-all.jar
#7 0.297 reviewdog/reviewdog info checking GitHub for tag 'v0.18.0'
#7 0.311 wget: bad address 'github.com'
#7 0.312 reviewdog/reviewdog crit unable to find 'v0.18.0' - use 'latest' or see https://github.com/reviewdog/reviewdog/releases for details
#7 ERROR: process "/bin/ash -eo pipefail -c wget -O - -q https://raw.githubusercontent.com/reviewdog/reviewdog/master/install.sh | sh -s -- -b /usr/local/bin/ ${REVIEWDOG_VERSION} &&     mkdir -p /opt/lib &&     wget -q -O /opt/lib/checkstyle.jar [https://github.com/checkstyle/checkstyle/releases/download/checkstyle-${CHECKSTYLE_VERSION}/checkstyle-${CHECKSTYLE_VERSION}-all.jar](https://github.com/checkstyle/checkstyle/releases/download/checkstyle-$%7BCHECKSTYLE_VERSION%7D/checkstyle-$%7BCHECKSTYLE_VERSION%7D-all.jar)" did not complete successfully: exit code: 1

This was not an issue when using a non-rootless setup. However, this can easily be remedied within the Dockerfile by adding the wget package to be installed so that the BusyBox version won't be used. The BusyBox version could be used if it would support the -4 flag to force name resolution to just IPv4 addresses.
I've tried disabling IPv6 on the underlying host, but that didn't help, so I'm turning to you in the hopes you might make something akin to the following change:
@@ -6,12 +6,12 @@
 SHELL ["/bin/ash", "-eo", "pipefail", "-c"]

 # hadolint ignore=DL3018
-RUN apk --no-cache add git
+RUN apk --no-cache add git wget

 # pre-install reviewdog and checkstyle
-RUN wget -O - -q https://raw.githubusercontent.com/reviewdog/reviewdog/master/install.sh | sh -s -- -b /usr/local/bin/ ${REVIEWDOG_VERSION} && \
+RUN wget -4 -O - -q https://raw.githubusercontent.com/reviewdog/reviewdog/master/install.sh | sh -s -- -b /usr/local/bin/ ${REVIEWDOG_VERSION} && \
     mkdir -p /opt/lib && \
-    wget -q -O /opt/lib/checkstyle.jar https://github.com/checkstyle/checkstyle/releases/download/checkstyle-${CHECKSTYLE_VERSION}/checkstyle-${CHECKSTYLE_VERSION}-all.jar
+    wget -4 -q -O /opt/lib/checkstyle.jar https://github.com/checkstyle/checkstyle/releases/download/checkstyle-${CHECKSTYLE_VERSION}/checkstyle-${CHECKSTYLE_VERSION}-all.jar

 COPY entrypoint.sh /entrypoint.sh

I'd be more than willing to submit a PR myself. Here is more information regarding this: https://github.com/alpinelinux/docker-alpine/issues/155.

Scratch the above. Since the downloaded install.sh script still uses wget without the -4 flag, then things still do not work properly. I'll try to find some other way to get this working.

The following change set would result in a functioning image as the downloaded install.sh first checks for the existence of curl and uses that instead of wget:
@@ -6,12 +6,12 @@
 SHELL ["/bin/ash", "-eo", "pipefail", "-c"]

 # hadolint ignore=DL3018
-RUN apk --no-cache add git
+RUN apk --no-cache add curl git wget

 # pre-install reviewdog and checkstyle
-RUN wget -O - -q https://raw.githubusercontent.com/reviewdog/reviewdog/master/install.sh | sh -s -- -b /usr/local/bin/ ${REVIEWDOG_VERSION} && \
+RUN wget -4 -O - -q https://raw.githubusercontent.com/reviewdog/reviewdog/master/install.sh | sh -s -- -b /usr/local/bin/ ${REVIEWDOG_VERSION} && \
     mkdir -p /opt/lib && \
-    wget -q -O /opt/lib/checkstyle.jar https://github.com/checkstyle/checkstyle/releases/download/checkstyle-${CHECKSTYLE_VERSION}/checkstyle-${CHECKSTYLE_VERSION}-all.jar
+    wget -4 -q -O /opt/lib/checkstyle.jar https://github.com/checkstyle/checkstyle/releases/download/checkstyle-${CHECKSTYLE_VERSION}/checkstyle-${CHECKSTYLE_VERSION}-all.jar

 COPY entrypoint.sh /entrypoint.sh

@usrme can you submit an PR on this?

Would you also be okay with me updating the ENV declarations as well?
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 3)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 4)

https://github.com/dbelyaev/action-checkstyle/pull/231.

@usrme
Your fix is available in the latest release: https://github.com/dbelyaev/action-checkstyle/releases/tag/v1.16.2.


[FEATURE] Edit button on manage links page looks out of place

Description
On the links page, we have the edit button over the button itself, and it's covering the icon such that it looks a little bit weird. Is there no way to shift the edit circle? Maybe put it at the end of the link button, or within it like the milestone page?
Screenshots

Additional information
No response

Good suggestion, it should be kept the same as the Milestone page as the edit button was recently updated for similar reasons.

I can work on this issue.
Please assign this to me.

@Njong392 Thanks for raising this issue but this issue is the duplicate issue of https://github.com/EddieHubCommunity/BioDrop/issues/8774. So, you can close this as duplicate issue.

I can work on this issue.
Please assign this to me.

@mishra-nikhil0110 I have assigned this to you

Can I Work if @mishra-nikhil0110  is not working on it.

Can I Work if @mishra-nikhil0110 is not working on it.

@umar-webdev please familiarise yourself with our Contributing Guide. This issue was only assigned to @mishra-nikhil0110 last week.  Please allow for no activity for two weeks we mention in the Guide, before asking for this issue to be assigned to you.

Hey I have fixed this please check
Thank you!

Hey I have fixed this issue please check
Thank you!

Hey, I have already fixed this issue but as a guideline, I can't make a pull request so Please check if @mishra-nikhil0110 is working on this issue or not If not please assign this issue to me.
Thank you!

@mishra-nikhil0110  Can you give an update on why is it taking so long?

As there has been no activity for over two weeks I am unassigning this issue from @mishra-nikhil0110

Hey @SaraJaoude I wanna work on this issue please assign this to me

Hey @SaraJaoude I wanna work on this issue please assign this to me

@harshu0026 I have assigned this to you.

Hey @SaraJaoude I have Open a PR please Chcek
Thank you!

Hey SaraJaoude I have Open a PR please Chcek
Thank you!

We got the notification - please do not chase in the issue as well.  Please be respectful that maintainers are working through the PRs and we will get to yours in due course.

Since there has been no activity, I'd like to request that this issue be assigned to me

Given the lack of recent activity, I'd like to request that this issue be assigned to me. Additionally, I have a few questions I'd like to address.

@TusharKumawat1 We are glad that you want to work on the issue but @harshu0026 has already raised  a PR and it is ready for review so you can look for another issue

There has been on activity for 2 weeks, so I have closed the PR. @TusharKumawat1 would you still like the issue?

Hello I would like to contribute to this issue. If there is no response from the others, please assign this to me. Thank you!

Assigning the issue to you @gaurav1607

Hello, I would like to contribute to this issue. If there is no response from the others, please assign this to me. Thank you!

@sandhyabaiju86 We are glad that you want to contribute but there is already a open PR ready to be merged.
Please find another unassigned and status:ready for dev issue and familarize with our contributing guidelines

I am unassigning due to the reasons mentioned here: https://github.com/EddieHubCommunity/BioDrop/pull/9956#issuecomment-1925676309

Hi, I have made a PR which fixes this issue and also tried keeping the code changes clean and minimal, I have also addressed a common query raised here #9886 in my PR #10227

Hi, I have made a PR which fixes this issue and also tried keeping the code changes clean and minimal, I have also addressed a common query raised in ( #9886 and #9956 ) in my PR #10227

Hi Kartik,
If this issue is assigned to you then you can start your work and raise PR. check more in CONTRIBUTING.md

Hi, I have made a PR which fixes this issue and also tried keeping the code changes clean and minimal, I have also addressed a common query raised in ( #9886 and #9956 ) in my PR #10227

@hi-Kartik2004 as mentioned by another contributor please read through this project's Contributing Guide.  We ask our contributors to only open PRs once the Issue has been assigned to them.  On this occasion only I will assign this Issue to you, but in future please follow the project's requirements.

@hi-Kartik2004 as mentioned by another contributor please read through this project's Contributing Guide. We ask our contributors to only open PRs once the Issue has been assigned to them. On this occasion only I will assign this Issue to you, but in future please follow the project's requirements.

Thank you for assigning the issue to me. Regrettably, I overlooked its initial assignment status and raised the PR hastily. I will ensure to address this aspect in the future, as I have in the past.

Merged pull request.


Leaving full-screen on Linux deactivates second monitor

When leaving full-screen (either by closing the window or calling create() with a non-full-screen mode), the resolution of the primary monitor is correctly restored but the second one is left deactivated, even after logging out and in again. I have to manually reactivate it in System Settings -> Display and Monitors. (Relative) screen positions are also lost.
This minimal program is enough to (always) reproduce the bug on my Kubuntu 13.04 system with two monitors (notebook monitor and external one set as primary) and SFML commit 6c1899d0f70e4dc845d7f482b22c9d79f7cd4f2f (aka 2.1):
#include <SFML/Window/Window.hpp>

int main()
{
    sf::Window w(sf::VideoMode(800, 600), "My window", sf::Style::Fullscreen);
}

The issue also appears when leaving the full-screen mode with Window::create().
(As a side note, the window is also created in the top-left corner of the primary screen instead of the center.)
Additional system information
ATI Mobility Radeon 5650 with open source drivers.
Output of X -version:
X.Org X Server 1.13.3
Release Date: 2013-03-07
X Protocol Version 11, Revision 0
Build Operating System: Linux 3.2.0-37-generic x86_64 Ubuntu
Current Operating System: Linux cn-0-u 3.8.0-27-generic #40-Ubuntu SMP Tue Jul 9 00:17:05 UTC 2013 x86_64
Kernel command line: BOOT_IMAGE=/boot/vmlinuz-3.8.0-27-generic root=UUID=(left out) ro quiet splash vt.handoff=7
Build Date: 17 April 2013  10:43:13PM
xorg-server 2:1.13.3-0ubuntu6 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.28.2
        Before reporting problems, check http://wiki.x.org
        to make sure that you have the latest version.

I'm not an experienced Linux user but if you give me pointers, I would be happy to provide you with more relevant information.

I can confirm this bug with the latest version of SFML. I'm running Linux Mint Cinnamon 17.2 and basically if my fullscreen window crashes, the second monitor gets turned off completely and the resolution on my first one is reduced to a minimum. It's a very localized bug but you should still re-open the issue

if my fullscreen window crashes

So it only happens when it crashes?

In my case yes, only when the window crashes. In my case the crash was the icon if that helps but I'm sure that's unrelated.

I'm still a bit confused.
What happens if your fullscreen application closes normally?
What happens if your fullscreen application crashes?
What Window Manager do you use?
Is your SFML version 2.3.1?

What happens if your fullscreen application closes normally?

If the game has no crashes, then it simply starts and behaves normally, covering one whole monitor in fullscreen state. If I close it after that, nothing happens to my monitors

What happens if your fullscreen application crashes?

One display gets disabled, the other one has its resolution lowered to around 800x600

What Window Manager do you use?

MATE 1.10

Is your SFML version 2.3.1?

Yes, installed from the official repository

So... SFML is supposed to do something after it crashes?


WIP Persistent node store

Implements a NATS KV backed store for Node Information to persist node information across requester node restarts.  Currently we only have an in-memory store, and as a result we lose all knowledge of any compute nodes when we restart the requester.
The new persistent store is the first step in providing more effective node information and being able to track compute nodes as they become inaccessible to the cluster for short periods of time.  The KV Store depends on NATS JetStream and so there have been changes to the set up of the NAT Server, and a first pass at using a test NATS server in Bacalhau tests (cf https://github.com/bacalhau-project/bacalhau/blob/1c1662c595dfacbcbdb124bd4bd4cef3c6bdd6de/pkg/routing/kvstore/kvstore_test.go#L33-L40)
TIme metrics and spans are available for both stores (inmemory for libp2p and kvstore for NATS) although it is expected that the inmemory store eventually be removed.
These changes did involve modifications to how the network transport performed pubsub for adding newly discovered node, only because the transport depends on the store, but this particular store depends on the transport and so construction is now a two-step process.

@coderabbitai review


Highlight 'Resolve' option on dropdown selection #272

Highlight 'Resolve' option on dropdown selection #272
Description
I have highlighted Resolve on click and defaulted it to 'Welcome' page

The click in this case automatically sets the home page every time the user attempts to view the other dropdown options (in this case 'elections' is only navigable via dropdown) which isn't ideal. The current behavior is fine as-is.


Moving the caret to the right is impossible on an input having a two characters currency sign on the right hand side

Using those options : { aSep: '.', aDec: ',', altDec: '.', aSign: '\u00a0€',   pSign: 's', mRound: 'U', lZero: 'deny', vMin: '0.00',  vMax: '999999999999.99' }, if you select all the input content then delete it, you obtain with the caret position (note: I symbolize the space by an underscore for better seeing what's going on) : |_€.
Now, if you try to hit right arrow to move the cursor to the right, then you can see the cursor go over the space to _|€, then jump back immediately to the initial position |_€.
Either the cursor should not move at all, or move to the far right hand side of the input to _€|.
(Use case; I'm using speadsheet-like divs containing each one input, and when the cursor in on the far right of the text in an input, and the right arrow is used, then I switch the focus to the cell on the right. This current behavior prevents that.)
Note : I use \u00a0 for non-breaking space close to the Euro sign.
cf. live example of the bug.

Also, the behavior is inconsistent since when you have 123| € in the input, you can hit the right arrow key and obtains 123 |€ then 123 €|.

Fixed in next.


Implement git pull on every bock start

TODOs

[x] add gams update command --> run git pull for gams docker folder
[ ] run a git fetch / each time gams is started?
[ ] update bock from inside bock? - launch command for it? is this possible? run code that checks if newer version of bock is available

implemented


Umbrella Helm research

As PO ,
I want to support the umbrella initiative ,
so that we can show what is possible with TX .
Link

blocks #226

Hints / Details

Check all solutions
MXD
IRS Localhost deployment
Umbrella Helm Chart from 23.12.
Trace-X Data Provider
Check https://github.com/eclipse-tractusx/sig-release/ repos for more information

Outcome / Acceptance Criteria

Outcome

...

Acceptance Criteria

[ ] Get in contact @evegufy to align on their initial research
[ ] to have a good overview of the current solutions - check the solutions of other teams (please refer to (#hint) section)
[ ] irs helm seeding data of policies with only helm functionalities
[ ] have a strategic dev meeting how to handle the e2e initiative (Ask @jzbmw for participants to invite, invite @mkanal as well)
[ ] Outcome of the research, is documented in a sufficient way in the concept folder structure of the repository https://github.com/eclipse-tractusx/item-relationship-service/tree/main/docs/concept

Out of Scope

...

further usefull source of info the matrix chat: https://chat.eclipse.org/#/room/#tractusx-umbrella-chart:matrix.eclipse.org

Planning 2:

for seeding of data, take a look at Helm Chart Hooks. This might be suitable

Please get in contact with @jzbmw

Helm Chart for a Data Provider : EDC, SubmodelServer, Registry

Created a Data Provider Helm Chart to be used in e2e-testing Umbrella Chart: https://github.com/catenax-ng/tx-item-relationship-service/pull/788
After review, the Data Provider as well as the Submodelserver Chart will be contributed to https://github.com/eclipse-tractusx/e2e-testing

Created contribution in E2E-testing repository https://github.com/eclipse-tractusx/e2e-testing/pull/53

[x] Get in contact evegufy to align on their initial research
[x] to have a good overview of the current solutions - check the solutions of other teams (please refer to (#hint) section)
[x] irs helm seeding data of policies with only helm functionalities
[x] have a strategic dev meeting how to handle the e2e initiative (Ask @jzbmw for participants to invite, invite @mkanal as well)
[ ] ~Outcome of the research, is documented in a sufficient way in the concept folder structure of the repository https://github.com/eclipse-tractusx/item-relationship-service/tree/main/docs/concept~ did not create a concept, since I already created the testdata seeding helm hook
[x] Helm Chart for a Data Provider : EDC, SubmodelServer, Registry


Events, add More Details button and extend Summary text

In the events, please add a View Details button, onClick it must open the Event to see the whole details in a bigger option
This should apply to all roles
Also in the Gallery view, extend the text in the Summary.

If mouse over on summary area, the whole summary text is displayed on hover popup.
And then on edit event modal, you can also see whole summary. Is it necessary to add more detail button?

@SolMon9099 the View Event button requested is not shown

@SolMon9099 yes it is necessary to add View Event button.


Enhances the tile view

Adds subscriptions to the subscriptions-array from parent to clear them, after destroying the view
Removes filter and sort service, if the selected view is the tile-view
Adds a second variable to hold the current selected view by user
Adds a property for color
Increases the size of the badge

@emanuelschuetze The color property is temporarly to see the difference between the two options. This could be removed, before it will be merged into the repository.

@GabrielInTheWorld Do you want to review my changes? Otherwise I'll rebase and merge it tomorrow.

Rebased


garm-cli pool create leads to a SIGSEGV: segmentation violation

Hello,
I'm running a garm instance (version 0.1.4) and it has worked fine so far.
However, recently I started getting the following error when trying to create a new pool.
Do you have any suggestion on what might be broken?
I've tried restarting garm but it did not help.
ubuntu@github-actions-runner-manager:~$ sudo pkgx garm-cli pool add --org b3e09917-64b3-4cf2-a3eb-89ee1d2ef415 --provider-name openstack_external --flavor vd.2 --image ubuntu-24.04-server-cloudimg-amd64.img --tags 'virtualdata, virtualdata-vd.2.extradisk'  --max-runners 1 --min-idle-runners 0
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x61f312f0314c]

goroutine 1 [running]:
github.com/cloudbase/garm/cmd/garm-cli/cmd.init.func23(0x61f313750e60, {0x61f312f13772?, 0x4?, 0x61f312f13776?})
	/__w/pantry/pantry/builds/github.com__cloudbase__garm-0.1.4/cmd/garm-cli/cmd/pool.go:261 +0x4cc
github.com/spf13/cobra.(*Command).execute(0x61f313750e60, {0xc00072e380, 0xe, 0xe})
	/__w/pantry/pantry/builds/github.com__cloudbase__garm-0.1.4/vendor/github.com/spf13/cobra/command.go:983 +0xaca
github.com/spf13/cobra.(*Command).ExecuteC(0x61f3137536a0)
	/__w/pantry/pantry/builds/github.com__cloudbase__garm-0.1.4/vendor/github.com/spf13/cobra/command.go:1115 +0x3ff
github.com/spf13/cobra.(*Command).Execute(...)
	/__w/pantry/pantry/builds/github.com__cloudbase__garm-0.1.4/vendor/github.com/spf13/cobra/command.go:1039
github.com/cloudbase/garm/cmd/garm-cli/cmd.Execute()
	/__w/pantry/pantry/builds/github.com__cloudbase__garm-0.1.4/cmd/garm-cli/cmd/root.go:57 +0x13b
main.main()
	/__w/pantry/pantry/builds/github.com__cloudbase__garm-0.1.4/cmd/garm-cli/main.go:20 +0xf

Would you mind trying out the CLI from the latest stable version. Just the CLI. You don't need to update the server. There was a nil pointer dereference that was fixed a while back in the cli.

Thank you @gabriel-samfira , indeed it solved the problem.
The error I get is now
Error: [POST /organizations/{orgID}/pools][409] CreateOrgPool default {"error":"Conflict","details":"pool with the same image and flavor already exists on this provider"}

This happens because I'm trying to create a pool with the same image/flavor combination as one that already exists, but change some --extra-specs to make it boot from the volume and with much more disk space.
Is there some way to work around this issue?

Yup. That is a limitation that has been removed in 0.1.5. The reasoning back then was that if you need to define another pool with the same image and flavor, you might as well scale up the pool you already have. But if we take extra specs and runner pools into account, it became clear that the over opinionated limitation was more of a pain than something desirable.
So in 0.1.5 this was removed. You will have to upgrade GARM itself. If you're willing to do that (should be safe), please make a backup of the database first. And make sure you read the release notes. A number of settings have been moved to the database, including github credentials. Also, credentials have been split into "github endpoints" (github.com, your own GHES, etc) and the actual credentials (App, PAT) that are tied to endpoints.
The migration of settings is done automatically when GARM starts for the first time using v0.1.5. But this also means that some configs have been deprecated. They still exist to facilitate the migration to the DB, but will no longer be used once the migration is done.
Feel free to join us on Slack if you would like to upgrade and if you run into any issues.

Closing this for now. Feel free to reopen if needed.


Landscape :Fluctuation on loading screen or home screen when app is being loaded in landscape mode as default.

Steps to reproduce :
1.Load the app in landscape mode.
2.On loading time , observe the screen activity.
Actual: Screen fluctuates on loading in landscape mode.
Expected : App should load without any fluctuation as it loads in the portrait mode.

Hello, is this issue still open? Am interested to work on this if that's okay.


Remove contextPath from base url

Is there a way to remove contextPath from the base url using 2.5 version ?
I can't use the 'try out ' button since all rest url's are prefixed with the tomcat contextPath.
There's an apache server in front of apache which rewrite all url's without the contextPath...
I've tried to extend an AbstractPathProvider, and overriding
@Override 		public String getApplicationBasePath() { 			return "/"; 		}
seems to have no effect...

That will not work if you're request mappings are pointing to urls with the context path. You'd have to make your url re-writing work differently


[BUG] Documented Parameter for Delete-Index API Call is Unsupported

What is the bug?
Within the Index Lifecycle guide, the "Delete an Index" example, demonstrates that an ignore: 404 parameter should be passed in order to prevent an error from being returned if one or more indices do not exist.
When using this parameter, however, an error is thrown with the message: request [/{index-name-here}] contains unrecognized parameter: [ignore] (where {index-name-here} is the name of the index that was identified for deletion. A similar error occurs if you pass an array (as in the example) of index names.
In looking at the code for the delete-index method (line 444 begins the method definition), there indeed seems to be no such parameter accepted. There is, however, a Boolean ignore_unavailable parameter which seems to do the job.
How can one reproduce the bug?
Call the delete-index API method, passing the name of an index that does not exist and passing ignore: 404 as in the Index Lifecycle example.
What is the expected behavior?
According to the Index Lifecycle guide, no error should be thrown.
What is your host/environment?
Host environment is an AWS Lambda function, so some version of Linux running our Node.js lambda (which exercises the @opensearch-project/opensearch API.
Proposed Resolution
I don't quite care which approach is taken, but one of the following should be taken with preference given to whichever best aligns with the mission and/or future direction of this project.

Update the documentation to reflect the correct parameter to pass that will cause the index-delete API call to not throw an error when an index does not exist.
Update the code (the IndicesApi.prototype.delete  function) to operate as the documentation prescribes.

Needs some digging on which parameter is correct, probably a mistake in the guide. Want to help fix this @jasel-lewis? I would add a working sample to https://github.com/opensearch-project/opensearch-js/tree/main/samples extracted from that guide, and fix the guide.

@dblock When I can find the time, I'd love to.

@nhtruong can I take a look at this issue?

@tejeshreddy of course. Feel free to pick up any issues you want. Just left a comment so others know. And thank you :)

@jasel-lewis The JS client doesn't have this feature. The guide was copy-pasted from the Ruby Client which does support this.
@tejeshreddy You can update the guide accordingly to remove this. We can raise this as a feature request.

@jasel-lewis - there's another parameter that is responsible for ignoring errors while deleting indices that do not exist. Please refer below -
await client.indices.delete({
    index: ['movies', 'paintings', 'burner'],
    ignore_unavailable: true,
  });

Also, sorry for the misinformation in the guide. The upcoming PR should fix it.
@nhtruong - The guide currently mentions about a parameter called ignore. Should we have to honor this parameter? If so, it would require code changes else updating the guide would do.

Doc fixed by https://github.com/opensearch-project/opensearch-js/pull/665 with the correct param


TabBarController

TabBarController is not showing with Parchment, why?

Hi @Michele404! It doesn't sound like this is an issue with Parchment. Can you explain a bit more what is not working and how to reproduce it?


TypeError: expected dynamic type string', but had type int64'

Error while updating property 'strokeWidth' in shadow node of type: RNSVGLine
TypeError: expected dynamic type string', but had type int64'
updateShadowNodeProp
ViewManagersPropertyCache.java:116
setProperty
ViewManagerPropertyUpdater.java:154
updateProps
ViewManagerPropertyUpdater.java:58
updateProperties
ReactShadowNode.java:272
createView
UIImplementation.java:197
createView
UIManagerModule.java:249
invoke
Method.java
invoke
JavaMethodWrapper.java:363
invoke
JavaModuleWrapper.java:166
run
NativeRunnable.java
handleCallback
Handler.java:751
dispatchMessage
Handler.java:95
dispatchMessage
MessageQueueThreadHandler.java:31
loop
Looper.java:154
run
MessageQueueThreadImpl.java:194
run
Thread.java:761

<VictoryCandlestick
data={[
{x: new Date(2016, 6, 1), open: 5, close: 10, high: 15, low: 0},
{x: new Date(2016, 6, 2), open: 10, close: 15, high: 20, low: 5},
{x: new Date(2016, 6, 3), open: 15, close: 20, high: 22, low: 10},
{x: new Date(2016, 6, 4), open: 20, close: 10, high: 25, low: 7},
{x: new Date(2016, 6, 5), open: 10, close: 8, high: 15, low: 5}
]}
/>

@hxx1197411446 I'm sorry, but I'm going to close this issue, as there isn't enough information for me to understand what might be going wrong. Please refer to our issue template and open a new issue with more information if your problem persists.


reference before assignment bug - python

Describe the bug
In the file "recognizers_date_time/date_time/base_dateperiod.py" In class BaseDatePeriodParser and method parse, I get an UnboundLocalError: local variable 'result_value' referenced before assignment. in the line  result.value = result_value (602). Fixed it by adding a None assignment at the start of the function.
Additional context
I'm not elaborating because this is a simple bug.

I caught more bugs of this sort. If you find it useful I will keep posting them as I really enjoy this package but don't want to bother with many bug issues that are easy to solve.

Hi @yuvalkirstain,
yes, these are of interest and potentially very useful. You can collect all similar issues in this same bug.
BTW, would you be able to submit a PR with fixes? If not, we'll triage this and try to address it as soon as possible.
Could you also provide one (or a few) sentences that hit each separate issue/case?
Thanks!

Sure :)

Let's not close it. Let keep this issue open as a central point to track such issues, ok?


Update GitHub actions: [upload|download]-artifact v1|v2|v3

An email was sent to<EMAIL_ADDRESS>titled:
[eclipse.org-committers] Deprecation Warning for GitHub actions [upload|download]-artifact v1|v2|v3
With call to update the following GitHub actions to v4 or later:

actions/upload-artifact
actions/download-artifact

The following versions are deprecated and will stop working in the not so near future:

v1 and v2: from 30/06/2024
v3: from 30/11/2024

See the following searches for instances of these actions in the Adoptium organization.
Code search for upload-artifact:
https://github.com/search?q=org%3Aadoptium actions%2Fupload-artifact&type=code
Code search for download-artifact:
https://github.com/search?q=org%3Aadoptium+actions%2Fdownload-artifact&type=code

The majority of the upload-artifact query hits are false positives.

many are in the mirrored OpenJDK repositories (i.e., adoptium/jdk8u, adoptium/jdk11u, etc.) and these are not used, and are configured not to be run, by the Adoptium project.
others are in documentation .md and .adoc files and are not run.

Those that are in .yml files that are run by the project are already at v4 level so are not part of the notice. The two .yml files that are run and are currently at v3 have been updated via the PRs linked above.

All the download-artifacts query hits are false positives.

most are in the mirrored OpenJDK repositories (i.e., adoptium/jdk8u, adoptium/jdk11u, etc.) and these are not used, and are configured not to be run, by the Adoptium project.
one .yml file is an example not run by the project.


Custom ContextCalculator does not work with permissions

luckperms info:
Running LuckPerms v5.0.83 by Luck.
-  Platform: Sponge
-  Server Brand: SpongeForge
-  Server Version:
-  SpongeAPI: 7.1.0-ea50f0c2 - SpongeForge: 1.12.2-2838-7.1.9
-  Storage:
-     Type: H2
-     File Size: 0.03MB
-  Messaging: None
-  Instance:
-     Static contexts: None
-     Online Players: 1 (1 unique)
-     Uptime: 4m 22s
-     Local Data: 1 users, 1 groups, 0 tracks

How to reproduce
I create a fresh installation of the server with the details above. Plugins: LuckPerms and my Test Plugin.
This test plugin was made only to register a custom ContextCalculator defined like this:
public class CustomCalculator implements ContextCalculator<Subject> {
  @Override
  public void calculate(Subject subject, ContextConsumer contextConsumer) {
    if (subject instanceof Identifiable) {
      contextConsumer.accept("test", "0");
      System.out.println("calculated context test=0");
    }
  }

  @Override
  public ContextSet estimatePotentialContexts() {
    ImmutableContextSet.Builder builder = ImmutableContextSet.builder();
    builder.add("test", "0");
    builder.add("test", "1");
    return builder.build();
  }
}

This ContextCalculator is registered in the main plugin class like this:
@Listener
public void onStarted(GameStartedServerEvent event) {
  LuckPermsProvider.get().getContextManager().registerCalculator(new CustomCalculator());
}

/lp user KaiNoMood info:
> User Info: kainomood
- UUID: a54a7884-9ea4-4d54-ac35-1623d70ffbe1 (type: mojang)
- Status: Online
- Primary Group: default
- Parent Groups:
-    > default
- Contextual Data:
-    Has contextual data: true
-    Applicable contexts: (test=0) (world=world) (dimension=minecraft.overworld) (remoteip=**redacted**) (localip=**redacted**) (localport=25566) (localhost=**redacted**)
-    Prefix: None
-    Suffix: None
-    Meta: None

When I do /lp user KaiNoMood info, the console shows calculated context test=0, so my custom ContextCalculator is "working" there. My context shows in the list of "Applicable contexts".
Added a permission that I did not have with /lp user KaiNoMood perm set minecraft.command.effect test=0
/lp user KaiNoMood perm info:
kainomood's Permissions:  (page 1 of 1 - 2 entries)
> minecraft.command.effect (true) (test=0)
> luckperms (true)

The error:
I enabled verbose with /lp verbose on minecraft.command.effect, then used the /effect command:

Console does not show calculated context test=0 as specified on my context calculator.
What I expected:
The /effect command should work because the user has the "test" context set to "0".
More details:
/lp user KaiNoMood perm checkinherits minecraft.command.effect:
kainomood has permission minecraft.command.effect set to undefined in context global. (inherited from self)
/lp user KaiNoMood perm checkinherits minecraft.command.effect test=0:
kainomood has permission minecraft.command.effect set to true in context test=0. (inherited from self)
Console does not show calculated context test=0 when I run the 2 commands above.
I am following the instructions provided in the wiki here. I can't make it work no matter what. Changing the name of the context, the value, registering it during GameInitializationEvent, or using a "custom" permission like "test.permission" and using it with something like player.hasPermission("test.permission") still does not produce any positive result.
Any help is appreciated! Thank you for reading!

Could you run /lp verbose paste after attempting to run the effect command and send the link here.

That contains IP addresses. Can I DM it to you on Discord or somewhere else privately? Sorry.

Yes, you can send the link privately if you like.

Go to https://keybase.io/encrypt
Set the recipient to luck
Copy "the secret messge" plaintext and post it here.

-----BEGIN PGP MESSAGE-----
Version: Keybase OpenPGP v2.1.13
Comment: https://keybase.io/crypto
wcFMAxavkAf6LzVtARAAjcqvGYbbB6RynPvrSR612NPLuOHuJNY4RhMHcgE8ajIi
sxFeHTqvA66c8ZZTaCc30+kaHI1YG8dUJ3lgerNhDjiJUJmF5udlWiNAxi/Umzqf
hHHx1P/sLT+jvFzZj7gtobok51t2pzT7rNobZOPl3gRH1j5iy3f+3/DiIYXfDSxR
cUKtDSxIO1/skw6d2Fdme/QvawQNspRinFRnyvT6b1PZUPb/hD/HHiA627lHnH4O
/MWlc/xVb/6S91VooVvxEK3iSgv5LVM/940D8DWWBZHcdIsPEIDQr7H+Pi1G2W7M
37IV2ME5Ub+Zhizv9KamRRccXtuH9mFR5TdEAt9LQnA01nHlv/jE0wdUO3fUcELK
Mrm2Sbru3ptqKaU2d/629lpFJjHo06dQEHyNW0fyh8JXNxjTLPqaHDjYmDvKQ/th
qnLRHN7S0MHkbkCbyRFVX8Mo4RgPxf0DnS1ABmTO6oWVljb8s/rFzQweVhB4wH2O
QagOTkgBUfbWEOfY9Ep5n7efwIpakUAuuBO3P2ZMInd/HC6dgxHxm+Aej0w4ern/
T4i2GPlugjFCvrUuGe5er4//MkHoj1Mu32nwECTBDXb66j3c7Epauc14r8h6LUjY
+YonaUDYSUWU9GzzYNTtZHjVdGND6fyGp+p0w4C22a0n2qu7+4SWALsADyc30JHS
ZQF3FNkmYN4ub1jbYbSviLLQxdXOgTbZBQyN0PxwSSQgkdqMzGB4C1YXwFXtOHRi
D/lMMIspj5J708hrCb0KUi/RpFU36bj2SR//XG1TalrigGcM6Gj0EUJ7dJv48TCn
0CFQ/f/O
=NpZS
-----END PGP MESSAGE-----

Hmm ok interesting.
I suspect the issue is with this line:
if (subject instanceof Identifiable)

Could you try removing that from your test to see if it works?

Yeah, that works.
The reason I added if (subject instanceof Identifiable) is because in my actual code, also used in Spigot, I actually need the UUID object.
I didn't want to use UUID.fromString(subject.getIdentifier()) for performance optimization.
The Player interface in Sponge extends both Subject and Identifiable, and I just needed the UUID object, so I used Identifiable.
Is there any reason the Subject is not actually the Player object (or at least User)?
Do you have any suggestion other than deserializing the subject identifier? Thank you for your quick help!

Is there any reason the Subject is not actually the Player object (or at least User)?

Yes - there are (usually) two implementations of Subject for a given unique player present at runtime.

One is the actual Player Minecraft object - the one Sponge mixes into to implement it's API - this object will implement Identifiable and other interfaces.
The other is an object provided by the Permission Service (LuckPerms in this case) which actually implements the permission methods.

Sponge proxies the permission related methods from the actual Player object to the LuckPerms one automagically at runtime with Mixin.

Do you have any suggestion other than deserializing the subject identifier? Thank you for your quick help!

If you want to guarantee that you'll end up with a "player like" object every time, you need to use Subject#getCommandSource.
Something like this should work:
@Override
public void calculate(@NonNull Subject subject, @NonNull ContextConsumer consumer) {
    Identifiable identifiable;
    if (subject instanceof Identifiable) {
        identifiable = (Identifiable) subject;
    } else {
        Optional<CommandSource> commandSource = subject.getCommandSource();
        if (commandSource.isPresent() && commandSource.get() instanceof Identifiable) {
            identifiable = (Identifiable) commandSource.get();
        } else {
            return;
        }
    }

    // TODO: do something with identifiable to calculate contexts

}

Or even just this will work:
@Override
public void calculate(@NonNull Subject subject, @NonNull ContextConsumer consumer) {
    CommandSource commandSource = subject.getCommandSource().orElse(null);
    if (!(commandSource instanceof Identifiable)) {
        return; // not a command source or identifiable
    }

    Identifiable identifiable = (Identifiable) commandSource;

    // TODO: do something with identifiable to calculate contexts
}

Ah I get it, I just assumed the Subject object was the actual Player object, as that's the "behaviour" in Bukkit as you get the actual Player object.
Thank you very much for your help!


Fixes #18

I made a PR to "fix" #18
I'm not experienced in developing for PostCSS and I do not use Javascript at all :-D
Please do not laugh and please do not hesitate to give me advice for improvement.
I'm hungry to learn and help.
I left comments in the PR to the things that look for me like code-smell.
(Okay, Javascript always looks like codesmell for me, but I marked the place where I introduced codesmell)

Uups, closed by accident

Just for completion:
Plays very well with the following PR
AtRule @important https://github.com/tailwindcss/tailwindcss/pull/99

Hey @psren feel like changing the at-rule to @tailwind screens;? Decided that's the name we're gonna roll with, then I can merge this after conflicts are figured out (sorry 😬 )

@adamwathan renamed the @screen-utilities to @screens.
Additionally I merged with master, there where some conflicts regarding code-style and the other PR #89


unclutter-xfixes: 1.2 -> 1.3

Semi-automatic update. These checks were performed:

built on NixOS
ran /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3/bin/unclutter -v and found version 1.3
ran /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3/bin/unclutter --version and found version 1.3
found 1.3 with grep in /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3
found 1.3 in filename of file in /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3

Success on x86_64-darwin (full log)
Partial log (click to expand)
cc obj/util.o obj/event.o obj/cursor.o obj/unclutter.o obj/extensions.o -L/nix/store/0fz2hj3k32ylhsa171kkclnwky4bhklk-libX11-1.6.5/lib -L/nix/store/7grfd6fgf3pig1ha7bcr1aprkafrz2p7-libXi-1.7.9/lib -L/nix/store/4gw99cpmy96jicr75igkgjsswdj7z4pb-libXfixes-5.0.2/lib -lX11 -lXi -lXfixes -lev -o "unclutter"
install -Dm 0755 "unclutter" "/nix/store/52gj1mqw5p6380rw911m61mziqnjk6jd-unclutter-xfixes-1.3/usr/bin/unclutter"
install -Dm 0644 man/unclutter-xfixes.1 "/nix/store/52gj1mqw5p6380rw911m61mziqnjk6jd-unclutter-xfixes-1.3/usr/share/man/man1/unclutter.1"
post-installation fixup
moving /nix/store/52gj1mqw5p6380rw911m61mziqnjk6jd-unclutter-xfixes-1.3/man to /nix/store/52gj1mqw5p6380rw911m61mziqnjk6jd-unclutter-xfixes-1.3/share/man
gzipping man pages under /nix/store/52gj1mqw5p6380rw911m61mziqnjk6jd-unclutter-xfixes-1.3/share/man/
strip is /nix/store/4sdh09gmvl15cy0zb6i7mbvxh5syz206-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/52gj1mqw5p6380rw911m61mziqnjk6jd-unclutter-xfixes-1.3/bin
patching script interpreter paths in /nix/store/52gj1mqw5p6380rw911m61mziqnjk6jd-unclutter-xfixes-1.3
/nix/store/52gj1mqw5p6380rw911m61mziqnjk6jd-unclutter-xfixes-1.3

Success on x86_64-linux (full log)
Partial log (click to expand)
post-installation fixup
moving /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3/man to /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3/share/man
shrinking RPATHs of ELF executables and libraries in /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3
shrinking /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3/bin/unclutter
gzipping man pages under /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3/share/man/
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3/bin
patching script interpreter paths in /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3
checking for references to /tmp/nix-build-unclutter-xfixes-1.3.drv-0 in /nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3...
/nix/store/ypik1dbi5xwyf65q8qgs6l4xg1siiinm-unclutter-xfixes-1.3

Success on aarch64-linux (full log)
Partial log (click to expand)
post-installation fixup
moving /nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3/man to /nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3/share/man
shrinking RPATHs of ELF executables and libraries in /nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3
shrinking /nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3/bin/unclutter
gzipping man pages under /nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3/share/man/
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3/bin
patching script interpreter paths in /nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3
checking for references to /build in /nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3...
/nix/store/8a6zc1my49yqlr28ldlp6n4k7iwlr1f0-unclutter-xfixes-1.3


Update run_hipporag_ircot_main_exps.sh

The current semicolon syntax is correct, changing it to a colon would not work.


Run Glow with Lua command

What are the arguments for running with the lua command?
I tried this but it does not seem to work:
require("glow").execute()
require("glow").execute('%')

Hey I've been trying to do the same thing! It's looking for an array called fargs containing the path to the file. You can call it like this:
require("glow").execute({
  fargs = {
    vim.fn.expand("%"),
  },
})

That works! Thanks!


[QUESTION] failed to list snapshot: cannot get client for volume pvc-*: more than one engine exists

Question

Environment

Longhorn version: longhorn 1.3.2
Kubernetes version: 1.24.6
Node config

OS type and version  kylin v10
Kernel version  4.19.90-24.4.v2101.ky10.x86_64
CPU per node: 28c
Memory per node: 512Gb
Disk type
Network bandwidth and latency between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):

Additional context
purpose: backup virtual machine <IP_ADDRESS>
error:  when i start the backup ,no response to snapshot and no data to the nfs storge
question: Is it related to  two engine？and if i cut down one engine,how to make sure the  virtual machine sucess
Specific details：
virt-launcher-budsc-vm-1639-w24fl  on two of master (hg-clueser-2-master02 and hg-clueser-2-master03)

This behavior is improved in later versions of Harvester/Longhorn. Closing since the discussion has ended.


access control doesn't work correctly

What steps will reproduce the problem?
when session is expire access control must redirect user to loginUrl but it doesn't happen and can check the page
session config:
'session' => [
            'class' => 'yii\web\Session',
            'timeout' => 60, // for test
        ],

user config:
        'user' => [
            'identityClass' => 'app\models\mysql\User',
            'loginUrl' => ['/auth/login'],
            'enableAutoLogin' => false,
        ],

public function behaviors()
    {
        return [
            'access' => [
                'class' => AccessControl::className(),
                'rules' => [
                    [
                        'allow' => true,
                        'roles' => ['@'],
                    ],
                ],
            ],
        ];
    }

Additional info

Q
A

Yii version
last version

PHP version
7.3

Operating system
windows 10

please close this issue , this is my mistake in return before action


Nuget Package Creation - v1.9.4

MimeKitLite dependency update from ('4.0.0' -> '4.1.0').
Nuget package creation - v1.9.4.

Codecov Report
Patch and project coverage have no change.

Comparison is base (db5b9bd) 99.48% compared to head (3bffe47) 99.48%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage   99.48%   99.48%
=======================================
  Files           1        1
  Lines         781      781
  Branches      183      183
=======================================
  Hits          777      777
  Misses          2        2
  Partials        2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add support for a tenant field on interfaces

Environment

Python version:  3.5.6
NetBox version:  2.5.10

Proposed Functionality
Adding support for a tenant field on interfaces for the purpose of tenant administration.
Use Case
A tenant can use an interface on a device owned by another tenant. For purposes such as maintenance announcements, monitoring and (traffic) billing on the tenant level, it is useful to connect ownership information to the interface.
Example:
A datacenter provider is the tenant for an access switch. A customer with rackspace in the datacenter is assigned a (virtual) interface on that switch for internet connectivity. For purposes mentioned earlier, the tenancy of this interface needs to be known.
When reviewing this request, consider that gathering this information via other means is inconsistent and may not always be possible. It would require an indirect relation to something owned by the customer tenant, such as a cable, address within a prefix or a VLAN. Those may not exist and may not be owned by that specific tenant either. Therefore, my conclusion was to request this additional field.
Database Changes
This field already exists for other objects and I can't think of any conflicts with other functionality in Netbox. I guess it would simply mean adding a single column to the interface table?
External Dependencies
N/A

NetBox only supports tenancy assignment at the device level. Tenancy would be inferred from the device connected to the interface.


[Bug]: Datepicker month selector is broken in some of the demos

Reproduction
N/A, read the description
Describe the bug
Step 1) Navigate to the demo website: "https://www.shadcn-vue.com/docs/components/date-picker.html"
Part 1 - Date range picker
2) Go to the "Date Range Picker" section
3) Observe that the next month button works, but the previous month button does not.
Part 2 - Independent month date picker
2) Go to the "Date Range Picker with Independent Months" section
3) Observe that the date picker on the right side, both of the "month buttons" navigate to the next month (The icon reflects this as well)
Side note: I don't have it installed in any of my projects yet, I simply noticed this from the demo website, which I speculated was running the latest code from the repo. I tested this 5 minutes ago from the time of writing this bug report, so the repro steps/observations mentioned above are not stale.
--- Edit ---
I played around with the code a bit the issue with the "Date Range Picker with Independent Months" variant seems to be a relatively simple fix of just altering the code snippet from the demo (at line 222 for the event handler & line 224 for the icon).
However, the issue with the "Date Range Picker" variant appears to be deeper. So far I haven't been able to find the handler which manages that logic which leads me to suspect that, that issue lies somewhere within the Radix-Vue dependency.
--- Edit 2 ---
Just found the handler in radix-vue, changed the part in my node_modules that I suspected was causing the issue, did a temp-patch with "npx patch-package", and tested it again, and it worked. Can't really describe where it is since the modification I made was in transpiled code, but I can at least confirm that the issue lies with the radix-vue dependency.
--- Edit 3 ---
Found the broken code in the radix-vue repo:
https://github.com/radix-vue/radix-vue/blob/a605a6fe1552341479e02233dcba28095bbf2a6b/packages/radix-vue/src/Calendar/useCalendar.ts#L138
The nextPage function has grid.value = newGrid (Line 149) while the prevPage function does not. The patch I did in "Edit 2" would have been the equivalent of adding grid.value = newGrid to prevPage function in the same relative position within that function.
System Info
Browser:
  - Chrome v123.0.6312.106
  - Chrome v124.0.6367.61
OS:
  - Windows 10

Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

I am willing to submit a PR to fix this issue

Just make sure to fix the "Independent month date picker" as well, since that issue was independent of the radix-vue bug.

@JustinBordage the issue with the second month having 2 next buttons was fixed in this PR: https://github.com/radix-vue/shadcn-vue/pull/501

@epr3 Oh I didn't see that it had been split into a different issue, my bad.

@JustinBordage NP


Documentation updates

Update a few things in the documentation to make it easier to follow.

I signed it!


Turn on Work Queue coprocess mode

This should probably be the preferred way of using the work queue executor, as it can speed things up incredibly on shared filesystem supercomputers, and even on my laptop can reduce WQ pytest run time from around 550s to 350s.
Type of change

New feature (non-breaking change that adds functionality)
?

this hung in CI trying to run WQ tests... further investigation needed so converting to draft

in CI runinfo logs:
003$ cat ./submit_scripts/parsl.WorkQueueExecutor.block-0.1688148747.9381468.sh.err
sh: 1: parsl/executors/workqueue//parsl_coprocess.py: not found
so this looks like a packaging/install issue of some kind

subsequent fiddling has shown there are some subtleties with the PATH mechanism set up by the Makefile... i'll make a separate PR for this first

see PR #2804 for build PATH simplifications which should make coprocesses work


How does an issue work?

Hey Lucas, just checking how you create an issue for class today.

cat("you did a great job!\n")


ensure that MPI is initialized properly if Pthread

per the MPI standard, one cannot use MPI_INIT or
MPI_INIT_THREAD(..,MPI_THREAD_SINGLE,..) if threads are used anywhere
in the program.
in practice, this does not matter, thanks to glibc being always
thread-safe today, but it is prudent to hedge against incorrect usage of
MPI in case there are systems that break under the aforementioned
circumstances.
i assume that GASNet sets up MPI's use of threads properly when it uses
MPI+Threads.

Many thanks for the PR, Jeff! Originally, I left it up to the user application to be a "good citizen" w.r.t. thread support level when using MPI-dependent operations. In retrospect, though, cm should treat insufficient thread support as erroneous. I'll merge later today and update the docs to be more clear about this behavior.

Happy to help.  Feel free to bother me with any MPI-related issues you have.  PGAS + MPI-3 is of some interest to me :-)


Update Graphics to follow never hardware configurations

As Ti-Nspire saw their hardware change once more, the screen is now differently rendered on the newer devices and thus the main drawing routines should be updated to detect and follow the changes.

I'll take a look at how they did it in Ndless.  From what I've heard it's just a different framebuffer layout though, and the controller is the same.

Just linking this here so I don't forget where it is, the apis necessary to detect screen type were added in https://github.com/ndless-nspire/Ndless/commit/e9bd69685678383ef0c2a2043c07e054f3317222


Adding align support and nodejs-align.js example

@halton This change implemented APIs for align, and included the nodejs-align.js sample. PTAL, thanks.

@halton Updated, thanks.

@tingshao thanks for your quick fix, merging.


PDF extension doesn't exist on mobile

I'll replace the PDF viewer with PDF.js viewer
let me fix

Imported ViewerJS yeahhhh


unable to run html5 player in dev build for youtube LIVE

html5 player works fine in regular youtube video. but youtube live, it forces embed flash player no matter what settings are

confirmed problem, need to disable youtube center for html5 youtube live.

Use gaming.youtube.com. Uses html5 instead of flash.


Is this repo active?

Hey, I was looking for a way to interact with openai whisper in java and saw your github. Sadly, there is no code yet. I wanted to ask if you still plan on commiting something.
Thank you, have a nice day

Oh what? I mussed have messed something up. Gonn

The code has now been published.