[Improvement] Use alpine as base image

Instead of debian and ubuntu, which contain outdated mpd version, try using alpine as the base image

falled back to debian builds for now


Added method where(spec)

This pull request closes #2

Added new method where(spec) to Array.prototype
Added the unit test
Added the documentation
The example files can run in the terminal with node example/each.js for example and receive the same output as described in the comments

Thanks @fhernandezn for the tip!

I use the Standard, it seems more readable for me.

It's OK, if you want to use it for your own projects (like this), but be prepared to follow the conventions your current team decides to use.

LGMT 👍


Fixes #568 Fixes singularity install failures

Fixes unpredictable singularity install failures

@ivotron The build ran successfully two times consecutively and it passed now ! Please have a look! Thanks.

@ivotron  I have updated the PR ! Please have a look. Thanks.


Blind Data (ABI=9)

Adding a transitive offset to the RootNode, InternalNode and LeafNodes. This is an ABI=9 feature and doesn't change the size of the nodes in memory, just adds a 32-bit integer to each.
I hesitated on the naming and decided to call this a "transitive offset" - we use offset quite frequently in various contexts and making the transitive nature of this explicit seemed like a good idea.
The other aspect I hesitated on is the size of the integer type. In this implementation, the underlying size is a 32-bit integer (so as not the need to expand the node), but input and output is a 64-bit integer. As mentioned previously, we can steal a few unused bits from the origin to stretch the size of the offset we can store - by making the interface 64-bit, this saves needing to make an ABI change at a later date.

@danrbailey  - thanks for addressing my review comments. I've approved. I have non-blocking request thought, What do you say to changing the name from "TransientData" to "BlindData"? It's much shorter and (IMO) more standard (Houdini uses the same terminology). Also, this is what the your PR was originally named :)

@danrbailey - thanks for addressing my review comments. I've approved. I have a non-blocking request thought, What do you say to changing the name from "TransientData" to "BlindData"? It's much shorter and (IMO) more standard (Houdini uses the same terminology). Also, this is what the your PR was originally named :)

Thanks @kmuseth! We actually discussed the name in one of our earlier TSC meetings and settled on "transientData()" because it was deemed clearer that the data is indeed "transient" and thus not serialized with the grid whereas "blindData()" is a bit more ambiguous in that sense. Meeting notes here:
https://github.com/AcademySoftwareFoundation/openvdb/blame/master/tsc/meetings/2021-05-04.md#L75

okay, I bow to the majority decision


Add conductor icing model from weather

Carryover from GRIP project.

@aivanova5, how much of this was address by Allison's work this summer?


How can I deserialize to a private field?

I have this xml:
    <Terminate>
        <All/> <!-- may or may not be included, I want to check if it has been though -->
    </Terminate>

and would like to deserialize it to a class like this:
public class Terminate {
    [XmlElement(ElementName = "All")]
    private bool? _all { get; set; } = null;
    public bool All {
        get { return _all != null; }
    }
}

Is this possible?

Hey there @push-22 thank you for writing in with this.  I had to do about take here on your _all field as it's actually a property.  Oddly enough this does not work with our default pipeline but could probably be modified to work via extension.
If you convert that property into a field it works as expected:
	public sealed class Issue630Tests
	{
		[Fact]
		public void Verify()
		{
			var container = new ConfigurationContainer().Create().ForTesting();
			var instance = new Terminate(true);
			container.Cycle(instance).All.Should().BeTrue();

		}

		public class Terminate {
			public Terminate() : this(null) {}

			public Terminate(bool? all) => _all = all;

			[XmlElement(ElementName = "All")]
			private bool? _all = null;
			public bool All {
				get { return _all != null; }
			}
		}

	}

Are you able to use a field?  That would be the easiest solution here.  Otherwise, you may have to look into writing an extension for this and that would be some work.

Great - that works - Thanks.


Fixed docs badge link

It now appropriately goes to the actual documentation which most people would expect.

Merged! Thank you for contributing :)


[FLINK-25220][test] Write an architectural rule for all IT cases w.r.t. the MiniCluster

What is the purpose of the change
Architectural rules are required to verify that all IT cases should have:

for JUnit4

a public, static, final member of type MiniClusterWithClientResource annotated with ClassRule.
or
a public, non-static, final member of type MiniClusterWithClientResource annotated with Rule.

for JUnit5

a public, static, final member of type MiniClusterExtension
and
a public, static, final member of type AllCallbackWrapper annotated with RegisterExtension
The inheritance hierarchy need to be checked, because the member of MiniCluster could be defined in the super class.
Brief change log

update dependencies to include test code
develop fine-grained JavaField predicates
extend common predicates for class internal architecture check
exclude test classes for the ArchitectureTest and only include test classes for TestCodeArchitectureTest
develop the archunit rules and test for ITCase based on the purposed mentioned above
add log4j test log properties for debugging support
update archunit.properties to add hint for creating new violation store when new ArchRules are defined
update the violation stores

Verifying this change
This change is to build architecture test.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)

@flinkbot run azure

@Airblader Would you like to review this PR?

Apart from non-recorded violations being found, some recorded violations are also resolved when I execute the tests (even through Maven).

I don't think it's a good idea to break IntelliJ support for this test. Not being able to run the tests in the IDE is very frustrating, and will almost certainly cause people to repeatedly ask why that is. We need to figure out how to make it work in both cases (as it does currently).
A first step to debugging this would be to just print all the locations which are being checked, once for IntelliJ and once for Maven, and seeing how they differ; I'm pretty sure it's the import options that just need to be adjusted.

I don't think it's a good idea to break IntelliJ support for this test. Not being able to run the tests in the IDE is very frustrating, and will almost certainly cause people to repeatedly ask why that is. We need to figure out how to make it work in both cases (as it does currently).
A first step to debugging this would be to just print all the locations which are being checked, once for IntelliJ and once for Maven, and seeing how they differ; I'm pretty sure it's the import options that just need to be adjusted.

Me too, It would be great if it could simply work. But I don't think we have any promise to support IntelliJ with any price. It is clear that the import options need to be adjusted. The question is how and how hard. The debug info of ArchUnit is not clean and hard to check and there is almost no javadoc in the ArchUnit codebase. Current ArchUnit rules do not touch the most test classes, therefore is relative simple to handle. BTW, there is also some implicit issue with the current status, since some test classes are considered as production code.
What if all the violations found by IntelliJ have been resolved? We can only find them by running the TestCodeArchitectureTest via IntelliJ. There is no impact on the ArchitectureTest. If anyone later found any new violations for the ITCase, that means they are working on it on purpose, I would recommend to fix them asap.

I don't think we have any promise to support IntelliJ with any price.

Maybe not, but de facto this is what we recommend and what people use. If we make tests difficult to execute or, worse, produce incorrect results when executed, it will lead to a lead of friction and questions.

What if all the violations found by IntelliJ have been resolved?

That doesn't help, because if the CI doesn't catch such cases, any PR might introduce a new "IntelliJ-only" violation and the next time someone needs to execute the architectural tests locally, they see differences to the maven execution.

I also pinged @codecholeric to see if he happens to have any insights on the differences between IntelliJ and Maven.

I don't think we have any promise to support IntelliJ with any price.

Maybe not, but de facto this is what we recommend and what people use. If we make tests difficult to execute or, worse, produce incorrect results when executed, it will lead to a lead of friction and questions.

What if all the violations found by IntelliJ have been resolved?

That doesn't help, because if the CI doesn't catch such cases, any PR might introduce a new "IntelliJ-only" violation and the next time someone needs to execute the architectural tests locally, they see differences to the maven execution.

Fire enough. I just have big concern that, if perfection is required, we might need a big refactoring on the codebase, since the whole test modules in Flink are not in a good shape, as I mentioned previously.

I also pinged @codecholeric to see if he happens to have any insights on the differences between IntelliJ and Maven.

Thanks, hopefully there would be a simple solution.

if perfection is required

I think we're already not going for perfection. :-) My biggest pain point is the fact that the test executes just fine, but produces a different ("incorrect") result. If the test failed to execute altogether from IntelliJ, that'd be much better (but still quite sad). People wouldn't realize that and enter some fight of fixing locally versus fixing the CI, and get very frustrated in the process.

hopefully there would be a simple solution.

He let me know that he basically thinks the same as I did – making it work by adjusting the import options to cover both scenarios.
I'll try to look into this myself, too, to see if I can figure out how to adjust them.

I think we're already not going for perfection. :-) My biggest pain point is the fact that the test executes just fine, but produces a different ("incorrect") result. If the test failed to execute altogether from IntelliJ, that'd be much better (but still quite sad). People wouldn't realize that and enter some fight of fixing locally versus fixing the CI, and get very frustrated in the process.

I have to respectively disagree :-). It is NOT incorrect, it just works in a limited scope and within this scope it works correctly. And one thing should be pointed out is that even the current ArchUnit tests are only cover part of the whole Flink project, i.e. it has been running in a limited scope too. I knew the difference is about the scope i.e. ImportOption and it is a good question why ArchUnit works differently on maven and on IntelliJ with the same ImportOption?

One option could be that I shrink the scope to make IntelliJ works too, just like it currently does. But just because it currently works both on maven and on IntelliJ, does not mean there is no issue. It works now just because the complex scope might not be touched.

Many thanks for the review and your effort, appreciate it!

It is NOT incorrect

I think we have a misunderstanding here :-) What I mean is this: the results between IntelliJ and Maven are different = inconsistent. Since the CI is generally the source of truth, that means what a Maven run produces is "correct", making what IntelliJ produces "incorrect".
Of course technically it seems like IntelliJ actually finds more correct violations, so the IntelliJ result is "more correct", but IntelliJ is not our source of truth.

it is a good question why ArchUnit works differently on maven and on IntelliJ with the same ImportOption?

The problem here are the test-jars and how they're handled; we currently don't have an issue because we do not use JARs for the dependencies in the module.
When IntelliJ executes the tests, it handles the dependencies differently than Maven, and that is causing the inconsistency. We can fix this by improving the import options to cover both scenarios equally.

Of course technically it seems like IntelliJ actually finds more correct violations, so the IntelliJ result is "more correct", but IntelliJ is not our source of truth.

It means that maven and IntelliJ are running in different scopes, i.e. "more violations", IMHO, it has nothing to do with correctness.

The problem here are the test-jars and how they're handled; we currently don't have an issue because we do not use JARs for the dependencies in the module.

that is what I mentioned that complex scope has not been touched.

When IntelliJ executes the tests, it handles the dependencies differently than Maven, and that is causing the inconsistency. We can fix this by improving the import options to cover both scenarios equally.

Tt sounds more like a compatible IntelliJ setup with maven project w.r.t. dependencies or ArchUnit abstracts and hides the hybrid dependencies with maven, IntelliJ, gradle, Bazel, etc.  than working on it within this PR?

It means that maven and IntelliJ are running in different scopes, i.e. "more violations" will be found by IntelliJ because it use a bigger scope than maven

Running your branch in IntelliJ adds new violations, but also removes some violations.

[…] than working on it within this PR?

I'm also OK if you want to resolve this outside of this PR, but then the PR would have to wait until then. If we want to merge this PR, we need to address it here. As long as the test executes in both IntelliJ and Maven, but produces different results in each, I will not be +1, because I think it is too confusing and frustrating for developers.

One additional information I found to share: some Flink modules customized test-jar build logic, which turns out not all test code could be found in test jars. This might also be a reason that test running with maven only cover a smaller scope than running in IntelliJ

some Flink modules customized test-jar build logic, which turns out not all test code could be found in test jars

Yeah, that sounds like it might well be the cause, or at least be partially the reason. Do you know which or how many modules? If removing one or two such modules fixes the issue, we could use that as a temporary solution. I'd rather cover less code with the tests than have inconsistent results.

sure, this is the option I mentioned previously to shrink the scope.

Hey, I quickly looked over it (it's quite late for me already :wink:), but here are my 2 cents why things probably differ (as you already guessed in the last comments):
For the case of KafkaShuffleITCase or FlinkKafkaInternalProducerITCase the test-jar goal is defined like this:
<plugin>
	<groupId>org.apache.maven.plugins</groupId>
	<artifactId>maven-jar-plugin</artifactId>
	<executions>
		<execution>
			<goals>
				<goal>test-jar</goal>
			</goals>
			<configuration>
				<includes>
					<include>**/KafkaTestEnvironment*</include>
					<include>**/testutils/*</include>
					<include>META-INF/LICENSE</include>
					<include>META-INF/NOTICE</include>
				</includes>
			</configuration>
		</execution>
	</executions>
</plugin>

AFAIK IntelliJ doesn't support this fine-grained configuration. It emulates Maven and if there is <goal>test-jar</goal> in one project and <type>test-jar</type> as dependency in another, then it will simply dump all classes compiled from src/test/java (i.e. target/test-classes) onto the classpath. I think that's why KafkaShuffleITCase doesn't appear when you run it with Maven (I also checked the contents of the test-jar archive and KafkaShuffleITCase is indeed not contained).
I see 2 solutions:

do you really need this optimization to only include a little bit of those classes in the test jars? Most other modules seem to simply dump everything in the test jars, which makes it simpler and the difference here should vanish (after all test jars are not released, right? Thus, maybe it's not really relevant to optimize for a couple KB more or less?)
exclude those classes from the test, even though they are not tested with Maven (but then it also doesn't hurt)

Theoretically there would be 3), don't go via classpath, but compile everything and then just put in the Flink root folder to look for .class files on the file system, but I wouldn't go this way if you can get around it.
If there is any way, I would simply follow 1), because that's also the only way to really test all relevant classes in a downstream module.

@codecholeric many thanks for your reply. I would(and also had) suggest the first solution to use test jar in maven without any customized filter logic. @zentol Do you have any concern about this?

@codecholeric I found that ArchUnit will not scan any jars when running it in IntelliJ by trying this ImportOption:
    /** Only include classes when the location is jar. */
    public static final class OnlyIncludeJars implements ImportOption {
        @Override
        public boolean includes(Location location) {
            return location.isJar();
        }
    }

It will only scan paths defined by the PatternPredicate in ImportOption and figure out classes to evaluate, which makes the running behaviours between using maven(test-jar) and IntelliJ(Classes) different. Could you help confirming whether it is true? Thanks.

@Airblader Many thanks for your effort!

Create https://issues.apache.org/jira/browse/FLINK-25858 for removing ArchUnit rules for JUnit 4 in ITCaseRules the JUnit 4->5 migration is closed.

Thanks @Airblader. I have pushed the changes and squashed the commits. Please help merging it. Thanks.


doc update

Updated docs for lift and subscribe
No other changes.
closes #343
Please review.
Thank you,
Grigoriy

@kirkshoop
btw, I noticed two lift_if methods, but haven't found any usage or tests. Are they required?

@gchudnov, I would not be surprised if they are not used. They were one attempt to delay the evaluation of a template method until it was used. They never worked well.
If they are not in use they should be deleted.

I cancelled the OSX build - apparently there is an outage on travisci


Cleanup and tweaks

General cleanups
dumper: allow - to read from STDIN
cleanup: use --force instead of --dry-run
cleanup: also uninstall casks.
Add check command
Readme: add example of using system.
gitignore: ignore bin directory.
100% test coverage.

CC @xu-cheng @josh @technicalpickles
Closes #91 and #100.

@xu-cheng Whoops, yep, I did (although for consistency I might make it write to STDOUT). Any other objections or thoughts?

although for consistency I might make it write to STDOUT

no objection. If you do this, you need bypass file existing check as /dev/stdout will always exist.

Another the thing come to my mind, would -y or --yes be better than --force for cleanup?

@xu-cheng I think that implies there's a prompt without. --force feels more appropriate given the destructiveness too.

:cool: it's just --force feels like something you would try after normal invoking failed.

@xu-cheng Addressed feedback. Let me know what you think.


Drop pin_compatible('numpy')

Description

Drop pin_compatible('numpy') (this is handled via numpy's run_exports)

Please see: https://github.com/conda-forge/conda-forge.github.io/issues/2156

Tweak PIN_PKG_COMPILER code to lookup key only once

can you rebase and resolve the conflicts with main branch please?

The conflicts appear to be introduced by PR: https://github.com/conda/grayskull/pull/552
However there are some changes there that don't quite make sense to me. Let's discuss a bit more in that PR to see what next steps would be


Saved Events: Crash when removing events on Saved page

Description of issue
The app is crashing when rapidly removing events in a somewhat random order. Although there seems to be something about the events around the 4th July which are consistently triggering the crash.
A precursor to the crash is usually that the card(s) is removed from the list but the list doesn't collapse to fill the space and the date header remains:

Expected behaviour
The app should never crash ;) Events are removed correctly and the list collapses appropriately.
Steps to reproduce

Open the Events page
Save every event from today until 6th July
Switch to the Saved page
Start removing some events in a random order, moving down the list
Scroll to the bottom and quickly remove the events around the 4th July in a random order

Should first see the event tile being removed without the list collapsing
Removing some more events will trigger crash

OS / Browser / Device model
Galaxy S8
Haven't been able to reproduce this yet on iOS.
The bugsnag report for this should be #414

I've tried to reproduce this bug on multiple devices but am unable to. Maybe it was only those specific events causing the crash?


Plug in not working as expected

So i installed this plugin plus all the following:
    ● alpha-nvim 0.59ms
    ● bufferline.nvim 0.26ms
    ● cmp-nvim-lsp 0.21ms
    ● cmp_luasnip 0.16ms
    ● completion-nvim 0.38ms
    ● impatient.nvim 0.32ms
    ● indent-blankline.nvim 0.25ms
    ● kanagawa.nvim 0.16ms
    ● lazy.nvim 2.35ms  init.lua
    ● lspkind-nvim 0.04ms
    ● lspsaga.nvim 0.39ms
    ● lualine.nvim 0.35ms
    ● LuaSnip 2.83ms  luasnip.loaders.from_lua  luasnip-latex-snippets.nvim
    ● luasnip-latex-snippets.nvim 4.34ms
    ● markdown-preview.nvim 0.47ms
    ● nerdcommenter 1.25ms
    ● nvim-autopairs 0.22ms
    ● nvim-cmp 1.17ms
    ● nvim-colorizer.lua 0.09ms
    ● nvim-lspconfig 4.08ms
    ● nvim-tree.lua 0.28ms
    ● nvim-treesitter 3.48ms
    ● nvim-web-devicons 0.17ms  bufferline.nvim
    ● plenary.nvim 0.22ms  telescope.nvim
    ● sniprun 1.08ms
    ● symbols-outline.nvim 0.25ms
    ● telescope.nvim 0.77ms
    ● tex-conceal.vim 0.2ms
    ● vimtex 0.29ms
    ● yuck.vim 0.22ms

and my issue is that every time I open a .tex file, the snippets dont appear. I also have lsp+texlab configured, so i dont know how the  issue appears.

I'm having the same issue. Only commands from commands.lua and environments.lua seem to work, and I can't figure out why.


Feature/planetexpress/psa 25834 improve netboot stack

This PR represents our new cachingserver architecture, that will be deployed with ansible. We've removed unused folders / scripts and are providing the docker images now within a public accessible registry.
changed services

netboot-tftp (now inlcudes initial bootloaders)
netboot-cleaner (migrated cleaner to golang)
netboot-monitoring ( adjusted to new setup )
netboot-sync (sync squashFS files from azure blobstorage)
netboot-ipxeMenuGenerator (migrated to to golang by @ccadruvi )
netboot-http (as is, available on public repo)

Note: Before merging this PR, we need to make sure that we can replace all locations with either the central-cachingserver in azure or replace the current ones with the new ansible setup

Thanks @SwissGipfel & @ccadruvi for the review. I've applied a lot of changes which we mostly discussed bilateral. In my view, we should be good to merge. Only waiting for https://github.com/DigitecGalaxus/netboot/pull/10#issuecomment-1621688365 .


Unhandled exception - ChangeFeed Sample

Hi,
An unhandled exception of type 'System.AggregateException' occurred in mscorlib.dll is thrown when executing the ChangeFeed sample.
System.AggregateException was unhandled
HResult=-2146233088
Message=One or more errors occurred.
Source=mscorlib
StackTrace:
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at System.Threading.Tasks.Task.Wait()
at DocumentDB.Samples.Queries.Program.Main(String[] args) in c:\Users\igizotov\Documents__Code\azure-documentdb-dotnet\samples\code-samples\ChangeFeed\Program.cs:line 38
InnerException:
ActivityId=1502f56b-abf0-4951-825f-98c91890264d
HResult=-2146233088
Message=ResourceType Collection is unexpected.
ActivityId: 1502f56b-abf0-4951-825f-98c91890264d
RequestCharge=0
Source=Microsoft.Azure.Documents.Client
StackTrace:
at Microsoft.Azure.Documents.PathsHelper.GeneratePathForNameBased(ResourceType resourceType, String resourceFullName, Boolean isFeed, Boolean notRequireValidation)
at Microsoft.Azure.Documents.PathsHelper.GeneratePath(ResourceType resourceType, DocumentServiceRequest request, Boolean isFeed, Boolean notRequireValidation)
at Microsoft.Azure.Documents.GatewayStoreModel.GetFeedUri(DocumentServiceRequest request)
at Microsoft.Azure.Documents.GatewayStoreModel.d__23.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Documents.GatewayStoreModel.<>c__DisplayClass10.<b__f>d__12.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Documents.BackoffRetryUtility1.<>c__DisplayClass2.<<ExecuteAsync>b__0>d__4.MoveNext()          --- End of stack trace from previous location where exception was thrown ---             at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)             at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)             at Microsoft.Azure.Documents.BackoffRetryUtility1.d__1b.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.Azure.Documents.BackoffRetryUtility1.<ExecuteRetry>d__1b.MoveNext()          --- End of stack trace from previous location where exception was thrown ---             at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)             at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)             at Microsoft.Azure.Documents.BackoffRetryUtility1.d__a.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Documents.GatewayStoreModel.d__1f.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Documents.GatewayStoreModel.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Documents.Client.DocumentClient.d__2ea.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Documents.Client.DocumentClient.d__75.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Documents.BackoffRetryUtility1.<>c__DisplayClass2.<<ExecuteAsync>b__0>d__4.MoveNext()          --- End of stack trace from previous location where exception was thrown ---             at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)             at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)             at Microsoft.Azure.Documents.BackoffRetryUtility1.d__1b.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.Azure.Documents.BackoffRetryUtility1.<ExecuteRetry>d__1b.MoveNext()          --- End of stack trace from previous location where exception was thrown ---             at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)             at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)             at Microsoft.Azure.Documents.BackoffRetryUtility1.d__a.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Documents.Client.DocumentClient.d__351.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at DocumentDB.Samples.Queries.Program.d__6.MoveNext() in c:\Users\igizotov\Documents__Code\azure-documentdb-dotnet\samples\code-samples\ChangeFeed\Program.cs:line 63
InnerException:

@iizotov This is fixed now.


Callback function + strict mode

Added callback to particlesJS, implement strict mode, absolute equivalence checks (===) for typeof conditionals, formatting

Appreciate the PR, but deprecated since the 2.0.0 version ;)
https://github.com/VincentGarreau/particles.js/releases/tag/2.0.0


[improve][tieredstorage] Add API to scan objects on Tiered Storage

Motivation
Pulsar is missing some tool to verify the contents of the external TieredStorage.
In particular there is no easy way to perform this tasks:

inspect the contents of the Buckets using Pulsar tools (you have to connect to GCP, AWS, Azure console or use their specific tools)
find out if there are objects that should have been deleted by Pulsar but their are not

Modifications

add a new API http://localhost:8080/admin/v2/namespaces/public/default/scanOffloadedLedgers to scan all the objects for a given Namespace (please note that a bucket maybe shared by multiple namespaces, in that case listing one namespace will analyse all the objects in the bucket, as there is no way to map a object to a namespace)
the new API returns this structure:

number of objects
number of "errors" (things that shouldn't be there)
number of "unknown" objects (objects not from Pulsar, not from the same Cluster, or from older versions that did not track pulsarClusterName and managedLedgerName metadata)
any useful information about each object

For each Object we set a 'status':

"OK": all good
"UNKNOWN": the object is not generated by this cluster
all the other statuses are "errors"

This API is supposed to be used for two goals:
Alerting
check if "errors" is non zero -> the admin should do something
Fixing problems
if there is some error then you have to delete or fix the objects, and you can use the detailed data returned by this API
How it works
The new API uses "listObjects" API to scan all the objects in the Bucket (not recursively).
We are only performing a single request to S3 (with paging), we are not downloading the objects.
The API can be called on any broker.
For each object we compare the pulsarClusterName against the local clustername, and if there is a match then we get the name of the ManagedLedger from the Object metadata (se #14907).
Then we can get the ManagedLedgerInfo for the ManagedLedger, this way we know the list and the expected status of each Ledger.
For each ledger there is a UUID (offload context uuid) that allows us to verify that the object is exactly the object that is supposed to be there.
This way we can deal well with:

multiple clusters sharing the same bucket
multiple "lives" of the same "cluster" on the same bucket (in this case if there are objects left from the previous life of the same cluster they will be considered as 'errors')

Costs of executing this check:

we are calling one the ListObject API on S3/GCP/Azure once
we are executing a read request to ZooKeeper per each Object (in order to the the ManagedLedgerInfo)

It is suggested to run this scan only daily in order to not add too much burden on the ZooKeeper
Verifying this change
This change added tests, but there is some manual testing to do. I have tested it manually against S3, GCP and Azure
Documentation
I will follow up with a PR that explains the feature and how to setup alerts

@momo-jun a soft reminder: here is a PR w/ doc-required label

I will push the changes to switch to "streaming style"

@momo-jun this PR is merged. Can you work with @eolivelli to update docs? Thanks

Hi @eolivelli , any updates or plans for the follow-up PR that explains the feature and how to set up alerts?

Hi @eolivelli , can you pls let us know your updated plan to add the docs for this feature? Thanks.

@eolivelli this pr seems only support jclouds-type offloader. How to support filesystem-offloader?


Added automated greetings bot

Fixed #87
Description:
This pull request implements a new GitHub Actions workflow named "Greetings," which automatically greets users who open new issues or pull requests in this repository.
Key Features:
Automatic Greeting: Users will receive an immediate acknowledgment when they create an issue or pull request. The messages are:
Implementation Details:

Created a workflow file named greetings.yml in the .github/workflows directory.
Configured the workflow to trigger on both new issues and pull requests.
Utilized the actions/first-interaction action to manage the greeting messages.
This enhancement aims to foster a welcoming environment for contributors and improve community engagement.

Hey @saurabhbakolia can you please add labels for it,
or else it will not be counted as contributed.


[resolves #92] make navbar sticky

bandaid fix for sticky navbar. note that this makes a sticky navbar the default. can also add a flag to enable/disable based on user preference, although sticky navbar is generally good practice regardless
demo:
https://user-images.githubusercontent.com/15808601/217377698-9dd2b2dd-8047-451d-863b-ad4bde79ee00.mov

It looks good.

Should this be optional with a flag in the config.yml?

Should this be optional with a flag in the config.yml?

Yes, @benarmstead

And @pantera-rosa please, check other pages behaviour over index page.
https://deploy-preview-100--hugo-profile.netlify.app/

@pantera-rosa @gurusabarish
Created a PR: https://github.com/pantera-rosa/hugo-profile/pull/1/files, into this branch to make it optional.

thanks @benarmstead!

Sticky navbar support added in https://github.com/gurusabarish/hugo-profile/commit/70ea254099da0239185824c04adf77af33aafa7a for all pages

@gurusabarish
Looking back through this, there seems to be an issue in the class, the commit you made: https://github.com/gurusabarish/hugo-profile/blame/master/layouts/partials/sections/header.html#L33:
profileHeaderElem.classList.add("showHeaderOnTop");\
Adds a custom class that does not work properly with a sticky navbar.
Where the original PR made the correct class:
https://github.com/gurusabarish/hugo-profile/pull/100/files#diff-71f0045cd057a8a32a4f7ffead11e64f44558e9d0e3d01afe65d9c7c0048e819R27
When changing this back to fixed-top based on: https://mdbootstrap.com/learn/mdb-foundations/mdb-ui-kit/sticky-navbar/
This resolves the issue.
This behavior that I commented on in #92 , can also be seen on your netlify demo app site.


Suggest a better design

Hmmm..

What? happened

Propositions for a better design are welcome.

I am opening a pull request..
Read it and believe me, it will be worth it if it had no mistakes in typo...

OK. I didn't realize there was a pull request until after I sent the message.
PR is merged.

#3

Can you add me as maintainer or co-author so that I can try my best to resolve the issues..?

I don't think that's necessary for the time being, but I'll contact you if that's the case.

Ok thanks.


[FLINK-7669] Always load Flink classes via parent ClassLoader

Good fix, with minor comments.
I think we want to have even more default parent-first classes, because they leak through the public API.

org.slf4j
javax.annotation

Thanks for the reviews, @zentol and @StephanEwen?
@StephanEwen Regarding your last two comments, it would be nice if the ConfigOption itself did the parsing, i.e. if I could have a ConfigOption<String, String[]> or some such that parses a string to a list of strings and deals with all exceptions. Same would be true for the resolve order, where I would like to have ConfigOption<String, ResolveOrder> and it internally does the enum parsing. As it is now I have to do the parsing and exception handling in all places where the config is used. What do you think?
Regarding the list of parent-first patterns, javax.annotation I can see, but where do loggers bleed through the API?

@aljoscha I've suggested non-primitive/string ConfigOptions in https://issues.apache.org/jira/browse/FLINK-6577, along with moving the validation into the ConfigOptions in https://issues.apache.org/jira/browse/FLINK-6576.

@zentol That would be excellent to have!

Concerning loggers: I think for logging frameworks, they should rarely bleed through, but if you instantiate multiple ones, you get problems. It is more serious for the logging backend (where having multiple ones may mean logging does not happen at all or get at least crazy warnings), but preventing duplication in the logging frontend cannot hurt I believe.
Regarding non-primitive config options: Nice idea, but I would suggest to do it without for now, because it does not seem crucial enough for me to slow down a release for that.
How about configuring the default parent first options as:
java.;org.apache.flink.;javax.annotation;org.slf4j;org.apache.log4j;org.apache.logging.log4j;ch.qos.logback

Yes, I wasn't expecting to do that for the release. 😅
The defaults make sense, I'll push a fixup commit.

Looks good to me, +1
When we start introducing even more config options for class loading, it makes sense to introduce a class to bundle those up (like resolve order, parent first patterns, and future strings and arrays).

Merged


PowerToys Run does nothing?

Environment
Windows build number: 10.0.18362.900
PowerToys version: 0.18.2
PowerToy module: PowerToys Run

Steps to reproduce
I just installed PowerToys and I tried to use Alt + Space, and it didn't work. I also tried changing the shortcut and it still didn't work. I am using a keyboard plugged into my laptop but I also tried using the keyboard built into the laptop and it still didn't work.

Expected behavior
I am on my desktop and I am expecting the search bar to come up when I use the shortcut keys.

Actual behavior
When I press the shortcut keys nothing comes up.

I reinstalled using Powershell withWinGet install powertoys and that seemed to fix it for me.

I reinstalled using Powershell withWinGet install powertoys and that seemed to fix it for me.

I tried doing this and it had the same results as before for me.

in task manager, do you see PowerLauncher?

Does anything show up in your event logs?  We're a day from shipping 0.19 with a ton of fixes.

in task manager, do you see PowerLauncher?

Does anything show up in your event logs? We're a day from shipping 0.19 with a ton of fixes.

Yes PowerLauncher does show up. How do I check the event logs from PowerToys? Hopefully this is fixed in 0.19.

@Spencer1O1 so it is running but alt-space (assuming that is what your settings has it set to) doesn't display it?  Hmmm, lets revisit tomorrow on 0.19 when we release.

Can also confirm myself that PowerLauncher is running, but Alt+Space only opens it once. Once i esc out of it, alt+space just opens the context menu for a program as if you were just pressing Alt. I have to disable and re-enable for it to work once more, but then it stops working again.
I've only installed PowerTools today and Run isn't working

PowerLauncher was working in 0.18, but no longer works after installing 0.19. PowerLauncher is NOT running in task manager. I do have PowerToys Runner.

The update didn't fix it for me. I thought my autohotkey scripts might be interfering so I turned them off, but it didn't help. Also the shortcut guide isn't showing up when I hold down the windows key.

For some reason it is working now, but as soon as I type even just one letter the launcher stops responding and crashes.

I also cannot launch Powertoys Run using any shortcuts UNLESS I turn off "Enable Keyboard Manager" (the hotkey power toy), then it works. FWIW I'm using a Satechi Mac keyboard.

I also cannot launch Powertoys Run using any shortcuts UNLESS I turn off "Enable Keyboard Manager" (the hotkey power toy), then it works. FWIW I'm using a Satechi Mac keyboard.

I have tried with Enable Keyboard Manager both enabled and disabled, and the hotkey does not work in either case.

I also found that if the Powertoys Settings window was focused, the launch hotkey doesn't work

I also cannot launch Powertoys Run using any shortcuts UNLESS I turn off "Enable Keyboard Manager" (the hotkey power toy), then it works. FWIW I'm using a Satechi Mac keyboard.

I have tried with Enable Keyboard Manager both enabled and disabled, and the hotkey does not work in either case.

we've fixed a lot of bugs in PT Run since this was first filed.  Are you still seeing the issue?  if so, can we please get your logs?
%localappdata%\Microsoft\PowerToys\PowerToys Run\Logs

Just tried it again after updating - pressing Win+Space now seems to open my keyboard language changer?

we've fixed a lot of bugs in PT Run since this was first filed. Are you still seeing the issue? if so, can we please get your logs?
%localappdata%\Microsoft\PowerToys\PowerToys Run\Logs

2020-08-24.txt
2020-08-28.txt

For me "Alt"+"Space" shortcut to get PowerToys Run is not working while "Event Viewer" window is active.


No ECS API calls

My issue is that there are no ECS API calls anywhere in your code.

Thank you for your feedback. We are waiting for SanDisk USB thumbdrive support within ECS before coding the integration.

Attach them to docker hosts and your wish will be fulfilled. Serious, let's do it just for fun.

It was quite impressive how quickly you were able to propose a workable solution. Yet another example of how valuable EMC technical resources are.

CoprHD handles all of my storage responses near instantaneously.


Aliases or'ed with other functions causes a panic Undefined validation function

Package version eg. v9, v10:
v10
Issue, Question or Enhancement:
Custom validation aliases with "or" (|) panic with Undefined validation function
Code sample, to showcase or reproduce:
I use a custom alias for custom identifiers, that's an alias including a custom function
const rawBase64URLRegexString = "^[A-Za-z0-9-_]*$"
var rawBase64URLRegex = regexp.MustCompile(rawBase64URLRegexString)
v.validate.RegisterValidation("rawbase64url", func(fl validator.FieldLevel) bool {
	return rawBase64URLRegex.MatchString(fl.Field().String())
})
v.validate.RegisterAlias("bid2", "rawbase64url,len=11")

var req struct {
	Order string `binding:"numeric|bid2"`
}

This results in a panic with Undefined validation function. However, this does not result in a panic and works great:
var req struct {
	Order string `binding:"bid2"`
}

Using a custom function (not an alias) also seems to work without issues
var req struct {
	Order string `binding:"rawbase64url|numeric"`
}

Upon further testing, it seems that even built in aliases fail when or'ed. The following also panics with Undefined validation function 'iscolor' on field 'Order'
var req struct {
	Order string `binding:"numeric|iscolor"`
}

I was able to work around this issue with
v.validate.RegisterAlias("numeric_or_bid2", "rawbase64url,len=11|numeric")


PR merge options presented even if merging is blocked due to a required review

You can configure GitHub to block pull requests from being merged until a specific number of reviews have been completed (see GitHub documentation). However this VS Code extension doesn't seem to respect that and presents the merge buttons in the PR description view in VS Code even if those options are unavailable on the GitHub PR webpage.
I have not checked if the same problem exists for other merge blocking rules (e.g. if CI tests pass, etc.)

Extension version: 0.22.0
VSCode Version: 1.52.1
OS: Windows 10 (WSL2 Remote)

Still true:

Extension version: 0.32.0
VSCode Version: 1.62.2
OS: Fedora 34


CouchDB _users session user doc is not deleted on logout

In my configuration with CouchDB 2.0, I noticed that the user "session" in the CouchDB _users db is not deleted on logout, although the user in the security document of the per user db removed.  I looked at the code and I found that in the couchdb.js this.removeKeys function the call to couchAuthDB.allDocs returns an error not_found on each session key passed to it.
I then tried to replicate the POST with {"keys": "org.couchdb.user:key-i-am-looking-for"}, using Postman or CURL and noticed that it also returns the not_found error.  I check the docs and it seems to be formatted properly but for some reason (maybe a special case for _users db) it does not find the keys.  SO I tried a single key using GET with key="org.couchdb.user:key-i-am-looking-for" and it works as expected.  Also tried, GET with keys=["org.couchdb.user:key-i-am-looking-for"] and that fails just like the POST does with each key returning a "not_found" error.
This explains all the mess I was documenting in #179 , and it would be nice to hear some confirmation of this issue of _users session docs not being deleted.  Does anyone else have this problem?  Since it appears to be an issue with my CouchDB 2.0 configuration, maybe if you don't see this problem you could confirm here what version of CouchDB you are using?
EDIT: Just tested on one of the user dbs by doing _all_docs POST with keys and it works as expected.  This points to some difference in the _users db that is causing _all_docs keys (with POST or GET) to fail where a single key with GET succeeds.

I posted an issue over on the couchdb github repo after doing more testing to validate this issue.  I also verified that the actual delete via bulkDocs does work as is in the couchdb.js source, so that part is good.  A workaround would be to make multiple call to all_docs using GET for each key to validate, but that seems silly.  The other option is to not validate the keys at all and simply call bulkDocs to delete, in this case not sure what will happen if the _users key is not there, but that would be quite unlikely anyway, so for now I may just do that.

While I am waiting for a response from the couchdb github repo issue I posted, I did implement the workaround using GET with a single key, and it works exactly as expected for a superlogin logout of a single session, but would need do some more work when used with a multiple session logout call such as, logoutAll etc...  Now that I have a solution for one of the problems I identified in #179, I think I will try to add the per user db 'user:<user_id> as the member in order to eliminate the case of public access on the per user db when no user sessions exist.

Just heard back from the CouchDB github repo and they say the problem is not reproducible in CouchDB 2.1, so it could be it was fixed for the first time or was a bug in CouchDB 2.0.  I will give 2.1 a try and report back here with the results for future reference.

Confirmed this is not an issue with superlogin, it is due to a bug in CouchDB 2.0 so I would advise not using CouchDB 2.0 with superlogin.  I have upgraded and tested with CouchDB 2.1 and it resolves the issue.  There is still the security issue noted in #179 , but I will also test that, maybe it too has been resolved by CouchDB 2.1.  Here is my closed issue from the couchdb github repo: https://github.com/apache/couchdb/issues/878


lib.d.ts Updates for TypeScript 4.2

https://github.com/microsoft/TSJS-lib-generator/

This is done now.


Prep for v1.1.0 release.

Codecov Report

Merging #939 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #939      +/-   ##
==========================================
+ Coverage   90.38%   90.38%   +<.01%
==========================================
  Files          46       46
  Lines        7623     7627       +4
==========================================
+ Hits         6890     6894       +4
  Misses        733      733

Impacted Files
Coverage Δ

bluesky/tests/test_suspenders.py
100% <ø> (ø)
:arrow_up:

bluesky/suspenders.py
89.02% <100%> (+0.27%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a72de3...54fa836. Read the comment docs.


Campaign Tracking Settings Missing

Hi,
In the documentation, Campaigns can be queried for Tracking Settings.
I see that in the stubs, there's a line referencing them, but it looks like the Campaign object I get back in the GoogleAdsRow (Google.Ads.GoogleAds.V0.Resources.Campaign) does not have Tracking Settings in it.
Is there a different place to get those values in the response row?
Best,
Kevin

Hi Kevin,
Let me try regenerating the stubs just to make sure I didn't miss anything.

Tracking setting is present in the latest stubs: https://github.com/googleads/google-ads-dotnet/blob/master/src/V1/Stubs/Campaign.cs#L684


Native automation: Cannot read properties of undefined (reading 'reqOpts')

What is your Scenario?
Switch the NA on.
What is the Current behavior?
Tests randomly fail with
  1) Unhandled promise rejection:

      TypeError: Cannot read properties of undefined (reading 'reqOpts')
      at NativeAutomationRequestPipeline._handleOtherRequests
      (C:\dev\gmc\modules\frontend\node_modules\testcafe\src\native-automation\request-pipeline\index.ts:313:128)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)
      at Chrome.<anonymous> (C:\dev\gmc\modules\frontend\node_modules\testcafe\src\native-automation\request-pipeline\index.ts:404:17)

      Browser: Chrome 113.0.5672.127 / Windows 11

What is the Expected behavior?
The tests do not fail randomly.
What is your public website URL? (or attach your complete example)

What is your TestCafe test code?

Your complete configuration file
No response
Your complete test report
No response
Screenshots
No response
Steps to Reproduce
I was not able to find the minimal steps to reproduce. Some of the tests in the suite use request hooks. Sometimes, only 10 out of 44 tests fail, sometimes all. After reducing concurrency to 4, the issue disappears.
TestCafe version
2.6.1-rc.1
Node.js version
18.15.0
Command-line arguments
testcafe chrome:headless .\e2e\fixtures-hosted -c 10
Browser name(s) and version(s)
Chrome 113
Platform(s) and version(s)
Windows 11
Other
No response

I researched the code and found possible mistakes in it. I created a build with a fix. Please try installing TestCafe from the following link and share your results with us: https://github.com/AlexKamaev/issues/raw/master/testcafe-2.6.2.tgz

I confirm that the linked version fixes the error. Thank you 👍

fixed in https://github.com/DevExpress/testcafe/pull/7738/files


shipyard: 0.3.48 -> 0.4.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/shipyard-run/shipyard/releases.
meta.description for shipyard is: "Shipyard is a tool for building modern cloud native development environments"
meta.homepage for shipyard is: "https://shipyard.run"
meta.changelog for shipyard is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.5 with grep in /nix/store/vi7rg5b1khyvcygnaa4y5imlicf0yxpa-shipyard-0.4.5
found 0.4.5 in filename of file in /nix/store/vi7rg5b1khyvcygnaa4y5imlicf0yxpa-shipyard-0.4.5

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
shipyard

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vi7rg5b1khyvcygnaa4y5imlicf0yxpa-shipyard-0.4.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A shipyard https://github.com/r-ryantm/nixpkgs/archive/a00678b22dc74db41b6d0533126dbacef1f0b665.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vi7rg5b1khyvcygnaa4y5imlicf0yxpa-shipyard-0.4.5
ls -la /nix/store/vi7rg5b1khyvcygnaa4y5imlicf0yxpa-shipyard-0.4.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

shipyard

Maintainer pings
cc @cpcloud for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/shipyard-run/shipyard/releases.
meta.description for shipyard is: "Shipyard is a tool for building modern cloud native development environments"
meta.homepage for shipyard is: "https://shipyard.run"
meta.changelog for shipyard is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.9 with grep in /nix/store/4rnjwwhykz2fxxs1mx0lzys0w0jcj6af-shipyard-0.4.9
found 0.4.9 in filename of file in /nix/store/4rnjwwhykz2fxxs1mx0lzys0w0jcj6af-shipyard-0.4.9

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
shipyard

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4rnjwwhykz2fxxs1mx0lzys0w0jcj6af-shipyard-0.4.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A shipyard https://github.com/r-ryantm/nixpkgs/archive/b945591fd7308aaa70e6c7868b0343e6752c8966.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4rnjwwhykz2fxxs1mx0lzys0w0jcj6af-shipyard-0.4.9
ls -la /nix/store/4rnjwwhykz2fxxs1mx0lzys0w0jcj6af-shipyard-0.4.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

shipyard

Maintainer pings
cc @cpcloud for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/shipyard-run/shipyard/releases.
meta.description for shipyard is: "Shipyard is a tool for building modern cloud native development environments"
meta.homepage for shipyard is: "https://shipyard.run"
meta.changelog for shipyard is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.10 with grep in /nix/store/8rcdf2mfn6hp6lak1zi2b3qix04as163-shipyard-0.4.10
found 0.4.10 in filename of file in /nix/store/8rcdf2mfn6hp6lak1zi2b3qix04as163-shipyard-0.4.10

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
shipyard

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8rcdf2mfn6hp6lak1zi2b3qix04as163-shipyard-0.4.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A shipyard https://github.com/r-ryantm/nixpkgs/archive/9a7b08c637c6d5ae279a1fd39610d1e215d1db1c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8rcdf2mfn6hp6lak1zi2b3qix04as163-shipyard-0.4.10
ls -la /nix/store/8rcdf2mfn6hp6lak1zi2b3qix04as163-shipyard-0.4.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

shipyard

Maintainer pings
cc @cpcloud for testing.

Result of nixpkgs-review pr 183846 run on x86_64-linux 1

1 package built:

shipyard


bpf: Avoid implicit shorten-64-to-32 in clang 19

As per the below PR, clang 19.1.x is having -Wshorten-64-to32 under the -Wimplicit-int-conversion, which is already forbidden in Makefile.bpf. This commit is to avoid the conversion error as per below sample
./lib/nodeport.h:1825:18: error: implicit conversion loses integer precision: '__u64' (aka 'unsigned long long') to '__s32' (aka 'int') [-Werror,-Wshorten-64-to-32]
 1825 |         __s32 len_old = ctx_full_len(ctx);

Relates: https://github.com/llvm/llvm-project/pull/80814

/test

Thanks to your awesome work, the main branch is not with llvm 18 now (https://github.com/cilium/cilium/pull/36197).
I think we can slowly proceed with fixes for future LLVM versions now.

/test

/test


qemu_ppc64le: EXpose fs support explicitly

Since fs sharing is not assumed as supported by default, expose
explicitly that the qemu_ppc64le supports it.
Fixes: #2584
Signed-off-by: Nitesh Konkar<EMAIL_ADDRESS>

/test


Minor improvement on the Kinesis package info (to be aligned with the  other IOs in the generated Javadoc)

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

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

R: @dhalperi
CC: @davorbonaci

LGTM.


Update README.md

Before first release, we need to make README.md more friendly for users.

[x] Implementation status compared to the spec
[x] Good-looking Badges
[x] Wording
[x] ...

Work in progress in PR #30.

Added in 72d104afd65a1823f31ebcbf4a75b20e94eb782a / PR#67. Closing.


Crashes on Windows

What happened?

The command to execute as normal
How can we reproduce the bug? (as minimally and precisely as possible)
Run "devspace" in a terminal on Windows.
Local Environment:

DevSpace Version: 5.0.2
Operating System: windows
Deployment method: helm

Kubernetes Cluster:

Cloud Provider: aws and Docker Desktop
Kubernetes Version:
Client Version: version.Info{Major:"1", Minor:"16+", GitVersion:"v1.16.6-beta.0", GitCommit:"e7f962ba86f4ce7033828210ca3556393c377bcc", GitTreeState:"clean", BuildDate:"2020-01-15T08:26:26Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"16+", GitVersion:"v1.16.6-beta.0", GitCommit:"e7f962ba86f4ce7033828210ca3556393c377bcc", GitTreeState:"clean", BuildDate:"2020-01-15T08:18:29Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}

Anything else we need to know?
It was working just fine for a month and all of a sudden it started throwing the exceptions. I tried to replace the current .exe file with a freshly downloaded from here https://github.com/devspace-cloud/devspace/releases/download/v5.0.2/devspace-windows-amd64.exe but the result is the same. I tried to run it from CMD and Powershell.
kubectl commands work as expected.
Docker Desktop details

/kind bug

It still works inside WSL on the same machine but not on Windows itself.

@iSeiryu thanks for opening this issue! We tested this on our windows machines, but it works without any issues. From the error message it seems there is a problem loading your UserProfileDirectory. Do you have any special configuration made there?

@FabianKramm Not that I'm aware of, the folder structures and the environment variables are the same.
I can run echo %USERPROFILE% or cd ~/. and it shows the correct info.
Are there any special system/environment variables devspace x64 looks for?
devspace-windows-386.exe version works just fine.

@iSeiryu strange, do you have any other issues with 64bit programs?

@FabianKramm Not that I'm aware of.

It just happened for another dev who installed it for the first time.

@iSeiryu can you send me the output of systeminfo for your computer and for the one of your other dev?

Mine. I'll get the other one later.
❯ systeminfo

Host Name:                 hidden
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.19041 N/A Build 19041
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Standalone Workstation
OS Build Type:             Multiprocessor Free
Registered Owner:          hidden
Registered Organization:   N/A
Product ID:                hidden
Original Install Date:     5/27/2020, 7:18:47 PM
System Boot Time:          9/24/2020, 10:00:00 AM
System Manufacturer:       Dell Inc.
System Model:              XPS 15 7590
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
                           [01]: Intel64 Family 6 Model 158 Stepping 13 GenuineIntel ~2400 Mhz
BIOS Version:              Dell Inc. 1.8.1, 7/3/2020
Windows Directory:         C:\WINDOWS
System Directory:          C:\WINDOWS\system32
Boot Device:               \Device\HarddiskVolume1
System Locale:             en-us;English (United States)
Input Locale:              en-us;English (United States)
Time Zone:                 (UTC-05:00) Eastern Time (US & Canada)
Total Physical Memory:     32,495 MB
Available Physical Memory: 13,209 MB
Virtual Memory: Max Size:  37,359 MB
Virtual Memory: Available: 4,815 MB
Virtual Memory: In Use:    32,544 MB
Page File Location(s):     C:\pagefile.sys
Domain:                    WORKGROUP
Logon Server:              \\hidden
Hotfix(s):                 10 Hotfix(s) Installed.
                           [01]: KB4576478
                           [02]: KB4537759
                           [03]: KB4557968
                           [04]: KB4559309
                           [05]: KB4560366
                           [06]: KB4561600
                           [07]: KB4566785
                           [08]: KB4570334
                           [09]: KB4577266
                           [10]: KB4571756
Network Card(s):           4 NIC(s) Installed.
                           [01]: hidden
                           [02]: hidden
                           [03]: hidden
                           [04]: hidden
Hyper-V Requirements:      A hypervisor has been detected. Features required for Hyper-V will not be displayed.

Another dev's machine (pretty much same as mine).
❯ systeminfo

Host Name:                 hidden
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.19041 N/A Build 19041
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Standalone Workstation
OS Build Type:             Multiprocessor Free
Registered Owner:          hidden
Registered Organization:   N/A
Product ID:                hidden
Original Install Date:     6/30/2020, 12:59:16 AM
System Boot Time:          9/13/2020, 10:36:15 PM
System Manufacturer:       Dell Inc.
System Model:              XPS 15 7590
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
                           [01]: Intel64 Family 6 Model 158 Stepping 13 GenuineIntel ~2400 Mhz
BIOS Version:              Dell Inc. 1.8.1, 7/3/2020
Windows Directory:         C:\WINDOWS
System Directory:          C:\WINDOWS\system32
Boot Device:               \Device\HarddiskVolume1
System Locale:             en-us;English (United States)
Input Locale:              en-us;English (United States)
Time Zone:                 (UTC-05:00) Eastern Time (US & Canada)
Total Physical Memory:     32,495 MB
Available Physical Memory: 12,723 MB
Virtual Memory: Max Size:  40,519 MB
Virtual Memory: Available: 4,494 MB
Virtual Memory: In Use:    36,025 MB
Page File Location(s):     C:\pagefile.sys
Domain:                    WORKGROUP
Logon Server:              \\hidden
Hotfix(s):                 8 Hotfix(s) Installed.
                           [01]: KB4576478
                           [02]: KB4559309
                           [03]: KB4560366
                           [04]: KB4561600
                           [05]: KB4566785
                           [06]: KB4570334
                           [07]: KB4577266
                           [08]: KB4571756
Network Card(s):           6 NIC(s) Installed.
                           [01]: hidden
                           [02]: hidden
                           [03]: hidden
                           [04]: hidden
                           [05]: hidden
                           [06]: hidden
Hyper-V Requirements:      A hypervisor has been detected. Features required for Hyper-V will not be displayed.

@iSeiryu Thanks! We'll be taking another close look at this. One info upfront: Please, feel free to just use the 32bit version since it is exactly the same as the 64bit version with the only difference that the 64bit version can use more RAM but DevSpace does not need a whole lot of memory, so you should be fine using the 32bit version.
If we need more information on your environment, I'll let you know. Thanks for providing so many details so quickly.

After following this issue https://github.com/pulumi/pulumi/issues/6093#issuecomment-773579956, the solution is to remove Webroot antivirus. For our user that was the exact antivirus causing problems, but there may be others causing the same issue. Confirmed to help on 3 different PCs.
Hope this helps anyone 🍻

We do have Webroot too. Good to know. Apparently that antivirus causes problems with a range of different software and one of them is Git.

 Feel free to reopen if this can be tied back to any issue/bug here in devspace!


(Bug) Unable to Sort NPC Category Tabs

I am suddenly unable to sort NPC Category Tabs.  I can usually grab one tab and drag to the desired spot.  However, now it just snaps back to its current position.   Been this way at least since last update a few days ago.
(Ideally, a desired feature would be to have the option to set the Categories to auto-sort alphabetically upon creation - they do not right now, which is why I try to move them into such an order.)

I just released v1.1.24. It includes a bugfix for this, and also includes a dropdown select where you can choose to let NPC Categories be automatically sorted alphabetically 🤖 ✌

Thank you!!!
On Thu, Oct 31, 2024 at 2:12 PM WBHarry @.***> wrote:

I just released v1.1.24. It includes a bugfix for this, and also includes
a dropdown select where you can choose to let NPC Categories be
automatically sorted alphabetically 🤖 ✌
image.png (view on web)
https://github.com/user-attachments/assets/98b6749d-eb43-4144-996c-954e9c3d9ac4
—
Reply to this email directly, view it on GitHub
https://github.com/WBHarry/pf2e-bestiary-tracking/issues/146#issuecomment-2450531743,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMOPOCKBAVDKQ2WI3255F3DZ6JXJPAVCNFSM6AAAAABQVOR55KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJQGUZTCNZUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


cp() should not create dst dir

Although it's nice to do not bother about dst dir, but as fs-tools tries to mimic UNIX commands in general, cp should not attempt create dst dir. There's no problem to call mkdir_p manually before for the user:
fstools.mkdir_p(dirname(dst), function (err) {
  if (err) {
    // handle error
    return
  }

  fstools.cp(src, dst, function (err) {
    // ....
  });
});

More than that I believe there should be three different functions: cp, cp_r and cp_a.

WTF? We don't rewrite *nix ih js. We do convenient tools for our needs. Don't multiply functions count.

Although I totally agree that we are not implementing coreutils, but even on other platforms copy does not bother with creation of directories - that's out of scope of the copy operation. I might admit that there's no need for cp_r probably, but I truly believe that cp should do exctly what it supposed to be - copy files and nothing more.

Well, then rename back cp -> copy, to avoid confusion, and keep API simple.


Attempt to enable linux_aarch64

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@willend this PR is for you :-)

Everything looks good @tkittel - including an Linux aarch64 package via Travis


Venue request form refactor

(Updated)

Ported venue request form logic to openreview-py package.
Added submission revision stage as Submission Revision Stage.

Fixes https://github.com/openreview/openreview-py/issues/696
Fixes https://github.com/openreview/openreview-py/issues/705

the changes look good to me but I'm not fully convinced that the support group should be listed everywhere, what if we run the different stages using the PC group?

the changes look good to me but I'm not fully convinced that the support group should be listed everywhere, what if we run the different stages using the PC group?

As discussed, this would require updating the request forum to make it accessible to the PC group.


Required when using as a form input

When using angular-trix as an input for a from, I want to be able to put a required attribute so I can use the build-in form validation from HTML5 browsers.
Any idea how to implement this? I am happy to do it, but I cannot think of a neat and simple solution right now (checking if the ng-model is set could be a solution before submitting, but this seams not very solid.)

perhaps a temporary workaround for you would be to add a hidden input item that is also bound to the sam model as your editor and make that required?


Temsten

Temsten

Why:
Closes [issue link]

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Y

Temsten

Delete all
On Sat, 26 Feb 2022, 15:02 welcome[bot], @.***> wrote:

Thanks for opening this pull request! A GitHub docs team member should be
by to give feedback soon. In the meantime, please check out the contributing
guidelines https://github.com/github/docs/blob/main/CONTRIBUTING.md.
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/15873#issuecomment-1051938911, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWKMZ54V2OLRBS6XARONFNTU5CQJXANCNFSM5PMOZD5Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Delete this all
On Sat, 26 Feb 2022, 15:03 TemSten, @.***> wrote:

Temsten
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/15873#issuecomment-1051940683, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWKMZ57UVU3BB6ZO7EDW2ITU5CQPHANCNFSM5PMOZD5Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


WIP Feat/wired datepicker

Alternate implementation of the calendar, based on CSS grid instead of a table.

Tried to keep a similar interface (same properties, attributes, events...).
Reduce internal state (only 4 private elements) to minimize unwanted side effects
light input validation for the dates with Date.parse
more functionnal oriented programming with html template (UI = f(state))
extends the based class instead of LitElement
The calendar actually extends WiredCard, to benefit from all the implementation of elevation and rerender for free!
Add tests with Karma \o/ first time and was real fun to do. WIP but already good coverage

It's getting real!

I'm quite happy with the result now, because it handles keyboard events really well, hence accessibility is really improved.
Tests are also quite complete, were really fun to make and not that hard. The open-wc testing tools are DOPE ❤
Still a bit of CSS to make for focus, and testing on different browsers and with different widths/heights.

Again .too much to look at .
Once you resolve the conflicts, I will merge this in. But wont push a release yet.
Thanks !


Gazetteer entities

Description:
This PR introduces the use of gazetteer entities through new 3 builtin entities with initial support in French:

snips/musicAlbum
snips/musicArtist
snips/musicTrack

A new CustomEntityParser object is introduced. It relies on the gazetteer entity parser library, and replaces all the previous logics regarding the matching and the resolution of custom entities.
The serialization of the SnipsNLUEngine has also evolved, as there are now two new folders:

builtin_entity_parser
custom_entity_parser

Hence, the model version is bumped to 0.17.0.

Codecov Report

Merging #668 into develop will decrease coverage by 0.21%.
The diff coverage is 83.19%.

@@             Coverage Diff             @@
##           develop     #668      +/-   ##
===========================================
- Coverage    89.17%   88.95%   -0.22%
===========================================
  Files           59       64       +5
  Lines         3306     3648     +342
  Branches       627      672      +45
===========================================
+ Hits          2948     3245     +297
- Misses         258      307      +49
+ Partials       100       96       -4

🎉 🎉 🎉 🎉


First small round of changes to Gradle/Groovy.

Contains
This is a small chunk of work split out of my local Gradle overhaul to start at least getting some functional pieces online and reviewable. Rather than wait for the complete removal of build.gradle for modules, a switch to IDE Gradle imports instead of gradlew idea, and so on. Some of those bits are working fine but I'm not happy with it all yet.
This helps speed up gradlew in general because it upgrades from Gradle 2.10 to 3.3. I have an additional speed-up that avoids some early (and possibly repetitive) dependency resolution, but it causes quirky failures with gradlew idea I didn't see earlier as I removed that whole thing in my main overhaul. It especially matters for Gradle commands with a nested target (like gradlew engine-tests:test) which at present are nearly unusable with a large amount of modules present (seems like Gradle almost reruns for every module)
It also introduces my shiny new Groovy Wrapper which provides a groovyw script that does exactly what gradlew does but for Groovy scripts, using the Groovy bundled with Gradle (it is there already anyway so use it!). Run Groovy scripts without ever having to worry about downloading or setting up Groovy! There is an initial module.groovy with two sub functions implemented so far:

get - retrieves one or more modules in source form (separate with spaces)
recurse - retrieves the given module(s) and their dependencies in source form

The get function effectively replaces gradlew fetchModuleX and is run with something like groovyw module get GooeysQuests instead. I haven't removed fetchModule in Gradle land yet, but will consider it deprecated for removal later. I named it "get" instead of "fetch" to reduce confusion vs Git commands (I hope) while in code it mostly uses "retrieve"
The recurse variant is more fun. groovyw module recurse GooeysQuests will retrieve that module and all its dependencies as source modules - finally! It'll skip any dependencies already found locally as source. Both functions support multiple module names and then process all of them (such as groovyw module recurse GooeysQuests JoshariasSurvival NeoTTA LightAndShadow which will grab a lot of stuff - yet is super fast!)
Groovy itself isn't particularly speedy, but using this approach we skip all the extras Gradle wraps around simple things we don't really need Gradle for. At present you would still run gradlew idea after changing your local module lineup - later my goal is to make it so you could simply click a refresh button in the IDE (or even enable auto-refresh) and the structure will update, no gradlew idea needed. I have that working locally. It is neat.
Expanding the module script base should be super easy. Breaking it down:

groovyw - invokes the Groovy Wrapper I prepared. It is just a minimally hacked Gradle Wrapper that executes Groovy instead - but it uses all the Gradle distribution magic to set up anything needed automagically. Whichever one you run first will download Gradle then use that dist to execute either Gradle or Groovy scripts
module - targets the script module.groovy - so later we could have other scripts for targeting other things, like libs or meta module repos.
recurse - instructs the script to run the recurse sub function, passing further variables in there. You can actually stop here - the script will prompt the user for module names! Thanks @mbrotz (#2634) and @smsunarto (#2657) :-)
GooeysQuests ... - accepts a space separated list of modules (casing matters). Plain get just retrieves those modules while recurse as the name indicates recursively resolves all dependencies in the tree and retrieves all of them

How to test

Make sure gradlew runs without error - you should get Gradle 3.3 dist files downloaded, could take a few minutes depending on bandwith
Make sure gradlew idea runs without error (or gradlew eclipse etc) and that the project still opens normally in an IDE
Make sure the game runs either from IDE or via gradlew game
Use the new groovyw module get or groovyw module recurse commands to retrieve some modules! Visually verify that the newly downloaded module source dirs look right and/or use gradlew idea or whatever to test that they work
Try being creative with the groovyw module like feeding it the same module multiple times or attempting to trigger a circular dependency. I put some decent effort into hardening it but maybe there are still edge cases
Check out groovyw module usage or gibberish sub function names to see what happens

Outstanding before merging

[ ] Need to tweak at some docs to both link to and explain the Groovy Wrapper + new commands
[ ] Need to note in docs that gradlew fetchModuleX is now deprecated, although we'll keep it around a while
[ ] Should probably submit some bite-sized issues for further enhancements to the module script and other stuff too. Should be pretty easy with the Groovy Wrapper and initial script as an example

Uh oh, something went wrong with the build. Need to check on that

Looks like the Gradle update affected the code analytics, including where unit test results get stored (now at test-results/test/ instead of the current pattern**/test-results/*.xml) - need to fix that in Jenkins, maybe during the merge.
FindBugs also got waaaaay more verbose. I think. Lots of log spam.

Went ahead and merged this after realizing trying to update all the doc things right now would be a beast of a task. So much is out of date and @oniatus' new doc site https://github.com/MovingBlocks/TerasologyDocs already has converted and updated a bunch.
Not actually deleting the gradlew fetchModuleX approach yet, aiming to do that in engine v2. So probably we should soon put a disclaimer on the top of all engine wiki pages to see the new doc site and to be advised that wiki pages may be out of date or reworked entirely plus link to a forum thread going over the expected v2 changes. Which I guess I should be writing already ...
Did make a follow-up issue with some bite-sized tasks in #3099


feat: update regular comment syntax, add block comment

Hello, @luizribeiro. Thank you for your work on this plugin for cooklang.
This change updates the regular comment expression to support the current cooklang spec. It also adds support for the new block comments.
Summary:

Description
Example

Removed previous comment syntax
// This is a comment

Added comment syntax
-- This is a comment

Added block comment syntax
[- this is a comment -]

Resolves: #1

Hello, @luizribeiro - would you mind reviewing this PR? I welcome any feedback you could share. Thanks!

Hello, @luizribeiro - would you mind reviewing this PR? I welcome any feedback you could share. Thanks!

Hey! So sorry for having this slip through the cracks. This looks great, thank you so much!


Bump version to 2.0.0 app version

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Bump version to 2.0.0 app version
Issues Resolved
Part of opensearch-project/opensearch-build#2086
Check List

[x] Commits are signed per the DCO using --signoff

For any changes to files within Helm chart directories:

[x] Helm chart version bumped
[x] Helm chart CHANGELOG.md updated to reflect change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

IMO this should be in draft stage 🙂


[Event Request] codeunit 5988 "Serv-Documents Mgt." - Event for procedure CheckAndBlankQtys

Can we get the following event for the "Serv-Documents Mgt." Codeunit: OnCheckAndBlankQtysOnRepeatStart
    procedure CheckAndBlankQtys(ServDocType: Integer)
    begin
        ServLine.Reset();
        OnCheckAndBlankQtysOnAfterServLineSetFilters(ServLine);
        if ServLine.Find('-') then
            repeat
                with ServLine do begin

                    OnCheckAndBlankQtysOnRepeatStart(ServLine);      //NEW

                    // Service Charge line should not be tested.
                    if (Type <> Type::" ") and not "System-Created Entry" then begin
                        if ServDocType = DATABASE::"Service Contract Header" then
                            TestField("Contract No.");
                        if ServDocType = DATABASE::"Service Header" then
                            TestField("Shipment No.");
                    end;
    [...]
    end;

[IntegrationEvent(false, false)]
local procedure OnCheckAndBlankQtysOnRepeatStart(var ServLine: Record "Service Line" temporary)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


doc: TGT REST API returns 401 on incorrect credentials

Documentation of REST protocol erroneously says that "400 Bad Request" is returned if incorrect credentials are sent.
Instead, "401 Unauthorized" is returned in this case, which is much more logical.
See org.apereo.cas.support.rest.resources.TicketGrantingTicketResource
See org.apereo.cas.support.rest.resources.RestResourceUtils.createResponseEntityForAuthnFailure(...)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Multiple interfaces problem

Hey there,
I habe two bluetooth interfaces on my raspberry pi.
One should be used by noble and the other for continuous scanning.
So the problem is that it seems to be impossible to prevent noble touching the second interface.
It seems like noble is resetting all interfaces when loaded and than also performing commands on the second interface even if the first one is selected.
Greetings
Samuel

Set NOBLE_HCI_DEVICE_ID to control which adapter noble uses
https://github.com/sandeepmistry/noble#multiple-adapters

Could it be that there is a problem when using this in coffee script?
I did notice now that when I am using the command in a javascript file it does work but not in coffe script.
I am using this command:
process.env['NOBLE_HCI_DEVICE_ID'] = 1
Is there anything I have to change from javascript to coffeescript?

Is there anything I have to change from javascript to coffeescript?

Probably, especially since it works in JavaScript. I don't use coffeescript so I can't help, sorry.
Another option is to set the env from your shell before calling your script
sudo NOBLE_HCI_DEVICE_ID=1 node <your file>.js


[Select] Can't attach events to select's options when there are more than 2 selects

I have two selects in one page and I want to attach an event handler to do something when they are selected, but there's no way to target each one of these divs, because as far as I can see there's no id or name I can know in advance.

Hi @jpelay, check out our Select API documentation and Events section. There are available events in the Select component. If you want to do something when the option is selected, just get a select element by id and add an event listener on it - if there is no id, just add it to the select tag.


test(bts): withdraw ICX to NEAR account

Feature
If applicable, include a link to the Git issue of the feature. Otherwise, describe what the feature being tested is.
Scenario
some amount of ICX is transferred from ICON account to NEAR account. Check if withdraw is successful

Given

Some amount of ICX already transferred from ICON acount

When

NEAR user withdraws the ICX

Then

ICX should be deposited to NEAR user account

Test Passed.
Able to withdraw ICX and withdrawn amount of ICX is deposited to user account.
tx hash : 66qxcxuG6K1uTNWLgshHA628QiwjpGgizbeTBprnxXpD


[Snyk] Upgrade @lion/ui from 0.4.1 to 0.4.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @lion/ui from 0.4.1 to 0.4.2.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
:sparkles: Snyk has automatically assigned this pull request, set who gets assigned.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a month ago, on 2023-10-18.

Release notes

Package name: @lion/ui

0.4.2 - 2023-10-18

0.4.1 - 2023-09-14

    from @lion/ui GitHub release notes

Commit messages

Package name: @lion/ui

83eefe4 Version Packages (#2083)
be36bf3 fix(input-amount): Link currency to label after setting currency from… (#2091)
63a8e72 fix(lion-calendar): Make calendar buttons selectable on firefox through datepicker (#2080)
b1320a9 fix(overlay): avoid calling push in iterable object to avoid error loading overlays (#2088)
dcf3a4b fix(combobox): Fix search issues when modifying the middle of the input word (#2085)
5e8e417 chore: add code-owner
2ce41a5 Merge pull request #1947 from ing-bank/chore/fix-sharp-dep-version
7ee9191 chore: a higher sharp version that can be built on Mac
8bf54e6 docs: fix code indentation (#2081)
0952647 fix: make fetchjson returntype generic (#2082)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
👩💻 Set who automatically gets assigned
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

https://github.com/tokens-studio/style-dictionary-configurator/pull/63/


Android will report an error after performing multiple recognition.  libOpenSLES              E  Too many objects       Leaving Engine::CreateAudioRecorder (SL_RESULT_MEMORY_FAILURE)

When I use microphone recognition, Android will not be able to continue to recognize after continuous recognition exceeds a certain number of times, and the error content is as follows:
libOpenSLES              E  Too many objects
Leaving Engine::CreateAudioRecorder (SL_RESULT_MEMORY_FAILURE)
Even if I use the close function, it still doesn't work.
SDK failed to release OpenSL ES-related resources, this is a bad experience, please fix the bug immediately
com.microsoft.cognitiveservices.speech:client-sdk:1.30.0

It happened to me, too, and I still can't work it out

@luojt @amelons Thanks for reporting the issue. Do you get the same error with earlier Speech SDK releases or is 1.30.0 the first one you've tried with?

@luojt @amelons We've tested continuous recognition of up to 100 phrases without errors, using https://github.com/Azure-Samples/cognitive-services-speech-sdk/tree/master/samples/java/android/sdkdemo (scenario "Recognize continuously") on Google Pixel 7 with Android 13.
Can you reproduce the error with the linked sample? If yes, how many phrases does it take before the error occurs? What is the device you are using and the Android version? Also please provide a log file of a run where the error occurs, see https://learn.microsoft.com/azure/cognitive-services/speech-service/how-to-use-logging#android for instructions.

感谢您报告问题。在早期的语音 SDK 版本中是否遇到相同的错误，或者 1.30.0 是你尝试过的第一个版本？

It can be repeated in any version of Android, and the number of times the recognition varies according to the device, and versions before 1.30.0 can also appear.

我们已经在装有 Android 100 的 Google Pixel 7 上使用 https://github.com/Azure-Samples/cognitive-services-speech-sdk/tree/master/samples/java/android/sdkdemo（场景“连续识别”）测试了多达 13 个短语的连续识别功能，没有错误。
您可以使用链接的示例重现错误吗？如果是，在发生错误之前需要多少个短语？您使用的设备和安卓版本是什么？另请提供发生错误的运行的日志文件，有关说明，请参阅 https://learn.microsoft.com/azure/cognitive-services/speech-service/how-to-use-logging#android。

Not continuous identification, but need to disconnect the identification, open the identification, loop several times can appear this error: ```
libOpenSLES E Too many objects
Leaving Engine::CreateAudioRecorder (SL_RESULT_MEMORY_FAILURE)

@luojt @amelons We've tested continuous recognition of up to 100 phrases without errors, using https://github.com/Azure-Samples/cognitive-services-speech-sdk/tree/master/samples/java/android/sdkdemo (scenario "Recognize continuously") on Google Pixel 7 with Android 13.
Can you reproduce the error with the linked sample? If yes, how many phrases does it take before the error occurs? What is the device you are using and the Android version? Also please provide a log file of a run where the error occurs, see https://learn.microsoft.com/azure/cognitive-services/speech-service/how-to-use-logging#android for instructions.

Version 1.29.0 also has this error, which is caused by the following lines of code, which occurs when the text of speech recognition is frequently called multiple times:
speechConfig = SpeechConfig.fromSubscription(speechSubscriptionKey,serviceRegion)         speechConfig?.speechRecognitionLanguage = "zh-CN"         voiceSpeechRecognizer = SpeechRecognizer(speechConfig)

@luojt @amelons We've tested continuous recognition of up to 100 phrases without errors, using https://github.com/Azure-Samples/cognitive-services-speech-sdk/tree/master/samples/java/android/sdkdemo (scenario "Recognize continuously") on Google Pixel 7 with Android 13.
Can you reproduce the error with the linked sample? If yes, how many phrases does it take before the error occurs? What is the device you are using and the Android version? Also please provide a log file of a run where the error occurs, see https://learn.microsoft.com/azure/cognitive-services/speech-service/how-to-use-logging#android for instructions.

Version 1.29.0 also has this error, which is caused by the following lines of code, which occurs when the text of speech recognition is frequently called multiple times:
speechConfig = SpeechConfig.fromSubscription(speechSubscriptionKey,serviceRegion) speechConfig?.speechRecognitionLanguage = "zh-CN" voiceSpeechRecognizer = SpeechRecognizer(speechConfig)

You can customize the recording.

@luojt @amelons The first post https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/2007#issue-1798219566 said

not be able to continue to recognize after continuous recognition exceeds a certain number of times

(Speech SDK continuous recognition = startContinuousRecognitionAsync)
but then it was added in https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/2007#issuecomment-1631718124

Not continuous identification, but need to disconnect the identification, open the identification, loop several times

Presuming "identification" = "recognition", this is ambiguous. (Otherwise, even more so.)

Please describe your code (see above for examples). Try to write as clearly as possible.
Also please provide a log file (that was already requested in https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/2007#issuecomment-1631541853) where the error occurs, see https://learn.microsoft.com/azure/cognitive-services/speech-service/how-to-use-logging#android for instructions.
Otherwise it will be difficult to proceed - there hasn't been any other report of this kind.

@luojt @amelons The first post #2007 (comment) said

not be able to continue to recognize after continuous recognition exceeds a certain number of times

(Speech SDK continuous recognition = startContinuousRecognitionAsync)
but then it was added in #2007 (comment)

Not continuous identification, but need to disconnect the identification, open the identification, loop several times

Presuming "identification" = "recognition", this is ambiguous. (Otherwise, even more so.)
 Try to write as clearly as possible. Also please provide a log file (that was already requested in #2007 (comment)) where the error occurs, see https://learn.microsoft.com/azure/cognitive-services/speech-service/how-to-use-logging#android for instructions. Otherwise it will be difficult to proceed - there hasn't been any other report of this kind.

create recognizer
recognizer.startContinuousRecognitionAsync
get result1 =“”
get result2=“”
get result3=“”
get result4=“”
get result5=“”
...
get resultN=“hello,world”
recognizer.stopContinuousRecognitionAsync

@luojt This case is what we've tested without errors as noted in https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/2007#issuecomment-1631541853. So it's possible the issue depends on vendor specific Android HW and/or OS configuration. To proceed, we would need more information (incl. logs that have been requested several times by now) from you.

Closed since not reproduced and no logs. If you get a log file, please create a new Bug report (be sure to fill in the template) and attach the log.


support includeTokenTypes in TextEmbeddingBatchTranslator

Description
This allows embedding models to obtain token_type_ids from encodings in TextEmbeddingBatchTranslator (TextEmbeddingTranslator already supports it).

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 72.28%. Comparing base (b6b8729) to head (1e80f64).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3032      +/-   ##
============================================
- Coverage     72.29%   72.28%   -0.02%
+ Complexity     7293     7290       -3
============================================
  Files           722      722
  Lines         32512    32513       +1
  Branches       3396     3396
============================================
- Hits          23506    23501       -5
- Misses         7386     7391       +5
- Partials       1620     1621       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.