BUG-ID: CLOUDSTACK-8484 - Hosts without tag are not listed while list…

…ing the hosts for migration for instance with tag
While preparing the suitable hosts we are accidentally removing the incompatible (host does not have host tag) hosts from other host list(incorrect use of List.retainAll).

Can you add some unit/marvin tests?
Also, fix the commit msg header to 72 :) http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html

LGTM, I guess unit tests would be cool.

Code looks good, although I didn't test it yet. Before I do, I'd like to understand why do you want to push this to 4.5 and not to master?

@sudhansu7 there is a question here from @remibergsma , can you answer it so it can get merged.

@remibergsma  @runseb
This issue exists in 4.5 , hence fixed it in 4.5. this can me merged to master as well.

@sudhansu7 Let's first merge this to master, then it can go to 4.5. I just tried to merge your PR on master and that seems to work just fine so that is nice.
There were two other questions: about unit tests and one about the commit message. If you could respond to that please? In the mean time, I'll go ahead and give it a test run.

@remibergsma @karuturi
I have added marvin test for this issue.
Test result from local setup:
Sudhansus-MAC:cloudstack sudhansu$ nosetests --with-marvin --marvin-config=setup/dev/advanced.cfg ./test/integration/smoke/find_hosts_for_migration.py
==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Jul_03_2015_17_38_01_JVMT7H. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
===final results are now copied to: /tmp//MarvinLogs/find_hosts_for_migration_FKMWAR===
Sudhansus-MAC:cloudstack sudhansu$ cat /tmp//MarvinLogs/find_hosts_for_migration_FKMWAR/results.txt
Test find suitable and not-suitable list of hosts for migration ... === TestName: test_01_find_hosts_for_migration | Status : SUCCESS ===
ok

Ran 1 test in 27.003s
OK
Sudhansus-MAC:cloudstack sudhansu$ cat /tmp//MarvinLogs/find_hosts_for_migration_FKMWAR/failed_plus_exceptions.txt
Sudhansus-MAC:cloudstack sudhansu$ cat /tmp//MarvinLogs/find_hosts_for_migration_FKMWAR/runinfo.txt | grep -i 'Suitable Hosts'
2015-07-03 17:38:18,171 - DEBUG - Suitable Hosts: set([])
2015-07-03 17:38:18,171 - DEBUG - Not suitable Hosts: set([{cpuwithoverprovisioning : u'32000.0', version : u'4.5.2-SNAPSHOT', memorytotal :<PHONE_NUMBER>, jobstatus : 0, zoneid : u'7a83176c-4ae7-49cc-88ff-a794716e74f9', cpunumber : 4, managementserverid :<PHONE_NUMBER>, cpuallocated : u'0%', memoryused : 0, id : u'b1b25f17-18f7-4b4d-a8e1-a353ebf33660', cpuused : u'0%', suitableformigration : False, clusterid : u'c138587b-c149-4819-9d30-d4fa9d6d5da4', capabilities : u'hvm', state : u'Up', memoryallocated :<PHONE_NUMBER>, networkkbswrite : 16384, cpuspeed : 8000, type : u'Routing', events : u'Ping; StartAgentRebalance; ShutdownRequested; ManagementServerDown; HostDown; AgentDisconnected; PingTimeout; Remove; AgentConnected', zonename : u'Sandbox-simulator', podid : u'b0b72cfb-db0a-4e9c-a9da-84854d8c4a1f', clustertype : u'CloudManaged', hahost : False, lastpinged : u'1970-01-17T11:01:01+0530', ipaddress : u'<IP_ADDRESS>', disconnected : u'2015-07-03T14:32:16+0530', name : u'SimulatedAgent.3481871b-7626-450a-b2c1-ab4e6a2e5965', networkkbsread : 32768, created : u'2015-07-03T13:39:27+0530', clustername : u'C0', hypervisor : u'Simulator', islocalstorageactive : False, resourcestate : u'Enabled', requiresStorageMotion : False, podname : u'POD0'}])

Awesome @sudhansu7. Congratulations on the first marvin test :+1: :)
I will push this to master in sometime.


Added FAQ in GTN for dataset downloading incompletely

Thanks @andreasSune. Unfortunately, one of these FAQ has already been moved by @Melkeb in the PR #3373, we had that indicated in the issue https://github.com/galaxyproject/galaxy-hub/issues/1262. So now you can:

Delete the file for the FAQ that has been addressed already
Rename this PR for the other FAQ
Follow the same structure as the other PRs that are moving FAQ (esp. the metadata in the header). This tutorial can help you with that. Please also try to make it as short and concise as possible.

Hope that helps!

Hi, @beatrizserrano . I made some changes. Could you please review this new pull request? https://github.com/galaxyproject/training-material/pull/3382


Fixups for current release

Fixed typos
downgraded terminal-table dependency
Bumped version
Updated changelog

+1

:+1:


BlobClient set_metadata() Example?

Hello
Thanks for the great SDK :-) I did search before posting this but, alas, could find no examples of setting metadata for a blob. I'm am still learning Rust so apologies for missing anything that should be obvious.
I am able to set tags on the blob with the easy to follow API but set_metadata diverges from the pattern and discovery of how to use it is eluding me.
Example:
let destination = DestinationStorageAccount::new();

let destination_service_client = BlobServiceClient::new(destination.name, destination.creds);

let destination_blob = destination_service_client
	.container_client(message.container.clone())
	.blob_client(message.blob.clone());

let meta_data_response = destination_blob.set_metadata().await?;

set_metadata() has no arguments to process the metadata one wishes to set. Where do we actually set the values?
The dotnet SDK has a way to set metadata and tags in the same request via BlobCopyFromUriOptions.

Figured it out. You can close this.
Updated Example:
let destination = DestinationStorageAccount::new();

let destination_service_client = BlobServiceClient::new(destination.name, destination.creds);

let destination_blob = destination_service_client
	.container_client(message.container.clone())
	.blob_client(message.blob.clone());

let mut meta_data = Metadata::new();
meta_data.insert("test", "123");

let meta_data_builder: SetMetadataBuilder = destination_blob.set_metadata();
let meta_data_response = meta_data_builder.metadata(meta_data).await?;


LayaIDE English

Hello I wanted to use Laya but I can't change it to english, is it still possible to change it to english?
I followed the steps here
https://github.com/layabox/layaair/issues/19
But it doesn't change the other menus to english.
Is it not on full english?
Thank you

Hmm it seems that it is not changing everything to english, only the stuff on the code editor/vs code

Next build will have full english be support

Thank you for replying and for the great news!
Closing

The issue will be solve next version.

How long does it take for the next release?

Sad, still no release


Can't build native UWP app. ILC.exe, always a 32-bit image, fails with: Internal compiler error: Exception of type 'System.OutOfMemoryException' was thrown.

I'm trying to build a UWP unit test app in Release mode using the Compile with .NET Native tool chain option enabled. My UWP app's minimum and target versions are both set to Windows 10 build 16299. I'm using Visual Studio 15.7.3, and the build always fails because ILC.exe runs out of memory:

Internal compiler error: Exception of type 'System.OutOfMemoryException' was thrown.

I could see that the ILC.exe executable is a 32-bit image and isn't able to take advantage of most of the 32GB memory available on my box. ILC.exe failed after allocating about 3.4GB memory.
I tried to get Visual Studio to invoke a 64-bit ILC.exe using in each of the following ways, but ILC.exe is, according to Process Explorer, always running as a 32-bit process.

I tried adding <UseNativeEnvironment>true</UseNativeEnvironment> to my UWP project's default <PropertyGroup> element. I found a mention of this in relation to Visual C++, so I might have been barking up the wrong tree for a C# app on UWP.

I tried adding <Use64BitCompiler>true</Use64BitCompiler> to my UWP project's default <PropertyGroup> element. I had found this mentioned at https://github.com/dotnet/corert/issues/5604.

I tried explicitly referencing the package runtime.win10-x64.Microsoft.Net.Native.Compiler v2.1.8 in my UWP project. The package description says:

.Net Native Compilers package. Referencing this package will cause the project to be built using the specific version of the .NET Native compiler contained in the package, as opposed to any system installed version.

While it does appear that the correct package folder is referenced and a tools\x64\ilc\ilc.exe image is being executed by Visual Studio, the ILC.exe process remains a 32-bit process even despite the x64 claimed by parent folder naming. Here's what Process Explorer shows:

Am I doing something wrong?
Or is the ILC.exe that is shipped as part of runtime.win10-x64.Microsoft.Net.Native.Compiler actually a 32-bit program image and not a 64-bit program image? And if it is actually a 32-bit program image, is that by design, or an oversight?
How can I get my UWP app building with the native tool chain? Thank you.

We don't have a 64-bit version of ILC.exe, unfortunately.
Could you try placing <ShortcutGenericAnalysis>true</ShortcutGenericAnalysis> in one of your property groups?
This will try to shortcut analysis of generic code (if that's the problem) and might fix the issue for you. We recommend re-testing your app since it might make the analysis miss reflection patterns or serialization uses (it's possible to manually fix those later with RD.XML if that happens) and shouldn't be a big problem.

I'll try that shortly, thanks. But, re:

We don't have a 64-bit version of ILC.exe, unfortunately.

Perhaps note that in the package description, somehow? The naming runtime.win10-x64.Microsoft.Net.Native.Compiler can lead one to believe the whole set of tools would be x64, as opposed to just some of the bits.

The runtime.win10-x64 part refers to the compilation target (the architecture that the compiler compiles for). There's also runtime.win10-x86.Microsoft.Net.Native.Compiler and runtime.win10-arm.Microsoft.Net.Native.Compiler. It's not the architecture of the compiler EXE. I know it's confusing, but compilers are confusing.

@MichalStrehovsky I just got around to trying this with the option you mentioned above, <ShortcutGenericAnalysis>true</ShortcutGenericAnalysis>. The build of my sample app succeeded for the first time with the .NET Native tool chain enabled. Thank you.
I looked for information about this ShortcutGenericAnalysis option, but I can't find anything about it — Google has no relevant hits other than this same issue. Is ShortcutGenericAnalysis an official setting, or a private setting for troubleshooting issues such as this one?
FWIW, while building I was closely watching memory usage for both the ilc.exe and nutc_driver.exe processes in Process Explorer. This time, ilc.exe did not exceed ~1.9GB private bytes. Interestingly, nutc_driver.exe got as high as 6.7GB private bytes. Why is nutc_driver.exe a 64-bit image, while ilc.exe is not?

Has any progress been made on a 64-bit version of ilc?
I'm on VS 16.2.0 with UWP 6.2.8 and I am receiving the same error as the OP. However, all the ShortcutGenericAnalysis workaround does it give me another error:
System.InvalidOperationException: Unable to generate a temporary class (result=1).
error CS0400: The type or namespace name 'Ewn' could not be found in the global namespace (are you missing an assembly reference?)
error CS0400: The type or namespace name 'Ewn' could not be found in the global namespace (are you missing an assembly reference?)
    at System.Xml.Serialization.Compiler.Compile(XmlSerializerCompilerParameters xmlParameters, Evidence evidence, String
    at System.Xml.Serialization.TempAssembly.GenerateAssembly(XmlMapping[] xmlMappings, Type[] types, String defaultNamespace, Evidence evidence, XmlSerializerCompilerParameters parameters, Hashtable assemblies, String outputDir, String assemblyNameBase, IEnumerable`1 referenceDirectories, String intermediateDir, Boolean loadAssembly)
    at System.Xml.Serialization.XmlSerializer.GenerateSerializer(Type[] types, XmlMapping[] mappings, CompilerParameters
    at System.Xml.Serialization.XmlSerializer.GenerateSerializer(Type[] types, String outputDir, String assemblyNameBase,
    at SerializationAssemblyGenerator.Program.Main(String[] args)
Generating System.Reflection.DispatchProxy proxy code.
error : ILT0032: Failed to compile serialization code. See the build log for error details.

I assume these are related to missing entries in rd.xml, but the error message is worthless since we have many projects/packages with the namespace Ewn. Detailed build log doesn't show any more details as to what specific types or child namespace is the issue. I've tried adding the following (and many others), but recieve the exact same error message:
<Namespace Name="Ewn" Serialize="All" />

@gtbuchanan Compilation failures are never a manifestation of missing RD.XML entries. This looks like a bug. Can you reach out to dotnetnative at microsoft com? We're better equipped to help there.

I am hitting a similar problem as well when building in Azure Pipelines. I have all these flags set in my .csproj:
true
true
true
The error message is not mentioning "OutOfMemoryException" however:
Launching 'C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\runtime.win10-x86.microsoft.net.native.compiler\2.2.7-rel-27913-00\tools\x86\ilc\Tools64\nutc_driver.exe @"d:\a\1\s\myapp\myapp.UWP\obj\x86\Release\ilc\intermediate\MDIL\myapp.rsp"'
##[error]C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.7-rel-27913-00\tools\Microsoft.NetNative.targets(801,5): Error : Internal Compiler Error
C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.7-rel-27913-00\tools\Microsoft.NetNative.targets(801,5): error : Internal Compiler Error [d:\a\1\s\myapp\myapp.UWP\myapp.UWP.csproj]
##[error]C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.7-rel-27913-00\tools\Microsoft.NetNative.targets(801,5): Error : ILT0005: 'C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\runtime.win10-x86.microsoft.net.native.compiler\2.2.7-rel-27913-00\tools\x86\ilc\Tools64\nutc_driver.exe @"d:\a\1\s\myapp\myapp.UWP\obj\x86\Release\ilc\intermediate\MDIL\myapp.rsp"' returned exit code 1
C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.7-rel-27913-00\tools\Microsoft.NetNative.targets(801,5): error : ILT0005: 'C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\runtime.win10-x86.microsoft.net.native.compiler\2.2.7-rel-27913-00\tools\x86\ilc\Tools64\nutc_driver.exe @"d:\a\1\s\myapp\myapp.UWP\obj\x86\Release\ilc\intermediate\MDIL\myapp.rsp"' returned exit code 1 [d:\a\1\s\myapp\myapp.UWP\myapp.UWP.csproj]
Done Building Project "d:\a\1\s\myapp\myapp.UWP\myapp.UWP.csproj" (default targets) -- FAILED.

Anything else that I could try?


Pull in failed images support.

Avg response time
coverage on master

![Issue Stats][PR response img]
![Codecov branch][coverage]

Summary of changes
Added failed images support.
Fixed #309.
Fixed #354.
Rationale for changes
It wasn't present yet?!
Additional info and certifications
This pull request (PR) is a:

[X] Bug fix
[X] Feature addition
[ ] Other, Please describe:

I certify that:

[X] I have reviewed the [contributing guidelines]
[X] If this PR is a work in progress I have added WIP: to the
beginning of the PR title
[ ] If this PR is problematic for any reason, I have added
DO NOT MERGE: to the beginning of the title
[ ] The branch name and title of this PR contains the text
issue-<#> where <#> is replaced by the issue that this PR
is addressing
[ ] I have deleted trailing white space on any lines that this PR
touches
[X] I have used spaces for indentation on any lines that this PR
touches
[X] I have included some comments to explain non-obvious code
changes
[X] I have run the tests localy (ctest) and all tests pass
[ ] Each commit is a logically atomic, self-consistent, cohesive
set of changes

Come on! Get a grip on all these options. My time is limited!
ATTENTION !!! Build-system work is needed !!!
For this patch to be most effective these requirements need to be added to the build system:
I propose to add a general flag to the build system, to allow the user to switch on failed_images support on demand. Additionally to this flag the following lines are needed in the build system:
if (compiler is 'gcc' and gcc-version >= 7 and mpi is 'mpich' and mpich-version >= 3.2) then
add_definition ('-DUSE_FAILED_IMAGES') to compile-statement of caf_mpi.c
// There is currently no sufficient support for openmpi or other mpi-implementations for fault tolerant  mpi. When they add support other may be added.
In the top-most CMakeList, the #ifdef WITH_FAIL_IMAGES needs to be replaced by something reasonable to switch on these tests only when failed images support is present.
I have also added another preprocessor flag 'STOP_ON_UNSUPPORTED' independent of the above to allow stopping execution when an unsupported or unimplemented feature is hit. (see caf_mpi.c at the end).
Tested with gcc-trunk (aka 8.0), gcc-6.3.1 (the failed images test are not compilable, because 6.3 does know nothing about the new functions) and mpich v3.2 on Fedora 25 x86_64.

@zbeekman, one option would be to replace the issue template with a link to the CONTRIBUTING.md file and then put the template there -- presumably at the bottom.  The link can even go directly to the template section of the CONTRIBUTING.md file.   Even though this means many people will skip it, it gives us something to reference.  When someone submits an issue but doesn't give sufficient information, we can point them to the template, close the issue, and instruct the submitter to review the template before resubmitting.

I propose to add a style guide document to opencoarrays, and replace all of

Fingers too big for smartphone screen, sorry.
What I wanted to point out is, that all of these check boxes are tedious and time consuming. I would prefer having a code style section in the contribution.md and just having to check, that I have read it and made my changes accordingly. You may even add this to the cla agreement and have it done just once per dev. There are also automatic tools out there that check the style and prevent checking code into git when it violates the style. But beware using this to strictly may stop contributions.
What i do think critical is that the style of opencoarrays and gcc differs significantly. Adding opencoarray code back to gcc is hardened by this.

What I wanted to point out is, that all of these check boxes are tedious and time consuming. I would prefer having a code style section in the contribution.md and just having to check, that I have read it and made my changes accordingly. You may even add this to the cla agreement and have it done just once per dev. There are also automatic tools out there that check the style and prevent checking code into git when it violates the style. But beware using this to strictly may stop contributions.

I completely agree with all of this.... it's on my todo list

What i do think critical is that the style of opencoarrays and gcc differs significantly. Adding opencoarray code back to gcc is hardened by this.

Is there a place where we can see the style guide and assess the differences? Is it a difference in our official style, or a difference in the style as it is in the codebase? It would be nice to adopt GCC style/practices where it makes sense to do so, however there may be some cases where it still makes sense to diverge from GCC.

@vehre: Failed images are NOT supported in GFortran < 7, correct? If so, then it looks like I need to disable some tests when GFortran < 7.
This may take a little while for me to review... the changes are rather numerous.

For the gcc project in general coding conventions are here:
https://gcc.gnu.org/codingconventions.html
Unfortunately gfortran handles some of them quite relaxed.
Correct no failed images in gcc < 7, that's what I tried to tell in the 3rd and 5th paragraph from the end of the pull request description.
Some of the changes are merely style. So try to switch off taking whitespace changes into account when diffing. I tried to document anything that is unusual.

This looks good to me, all that's left is to merge in the latest changes and provide some CMake logic to ensure tests are only run with GCC 7 and MPICH 3.2, as well as minor cleanups etc.
Very nice work @afanfa and @vehre 👏

@zbeekman Any chance this can be merged this week?  I start teaching the short course at the University of Cyprus on Wednesday and would like to give the attendees an outline of what I'll cover.  I hope to introduce failed images next week.

Yes I'm close. Adding required build system stuff. Hope it will be done by
the call.
On Mon, May 22, 2017 at 11:20 PM Damian Rouson<EMAIL_ADDRESS>wrote:

@zbeekman https://github.com/zbeekman Any chance this can be merged
this week? I start teaching the short course at the University of Cyprus on
Wednesday and would like to give the attendees an outline of what I'll
cover. I hope to introduce failed images next week.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sourceryinstitute/OpenCoarrays/pull/370#issuecomment-303279489,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAREPAahlaNzb-d21QYawiV2vb1BFKJJks5r8lBugaJpZM4NMi7C
.

@verhe It looks like there may be a race condition or another parallelism issue: image_fail_and_sync_test_2 seems to reliably hang if you run the test multiple times in succession: https://travis-ci.org/sourceryinstitute/OpenCoarrays/jobs/235412457#L1160
If you run with, e.g.,  ctest --output-on-failure --repeat-until-fail 10 you might be able to trigger this on your own machine... another idea may be to try running it more oversubscribed.

@vehre: Adding the extra synchronization in 3e11eac4b82b8befaf578d81ff17643570bcf1c6 causes image_fail_and_sync_test_2 to pass every time now: https://travis-ci.org/sourceryinstitute/OpenCoarrays/jobs/235695125#L4287
This leads me to suspect that there is a race condition in sync images(*, STAT=...) when an image is failing... I think for now I'll add this to the "developer only" tests and open a new issue for it.

@verhe: Additional update: It seems that some other the other failed images tests can fail sometimes and that my attempted work around exhibits this... I'm not really sure what's going on.
Is it possible that MPI could have a bug/issue where the -disable-auto-cleanup flag causes some issues if a new job is launched too quickly afterwards? I'm going to push some other small commits and see if I see this behavior again or not.. if it always works the first time it is launched, perhaps MPI has a bug that is dependent on machine state. If this is the case I guess I'll put any test that fails regularly in the "developer tests" so that we can move on.

@zbeekman I don't have a single issue with running the testcases even multiple times. My mpi-version is mpich-3.2-6 Fedora system supplied.
As for the failing travis-run for gcc7, I suppose that it is simply timing out after 10 minutes total runtime.

I only see segfaults on massive oversubscription like 64 processes on a 4-core machine. I have added a commit that addresses these.

Ok, the time out is because the one test hangs for 10+ minutes, not because
the entire run takes over ten minutes. But I suspect this might be a
virtualization/cloud/MPI issue so until we get reports of others
encountering it or find a way to reliably reproduce I'm comfortable closing
the issue & merging
On Thu, May 25, 2017 at 7:54 AM Dr. Andre Vehreschild <
<EMAIL_ADDRESS>wrote:

I only see segfaults on massive oversubscription like 64 processes on a
4-core machine. I have added a commit that addresses these.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sourceryinstitute/OpenCoarrays/pull/370#issuecomment-303992276,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAREPMu3_Vf5S5cl1pBtciMaZix4sd9Oks5r9WwEgaJpZM4NMi7C
.

LGTM


Include symbols for Tasks and Tools projects in Razor.Design

Fixes https://github.com/aspnet/Razor/issues/2215

Spoke to @Eilon - he'd rather not us ship the pdbs in the main nupkg. He's also 🆗  with us not having symbols for preview2 as long as we do the right thing for rc. Closing this, but I'll leave the branch alone. @ajaybhargavb re-assigning the issue to you so you could take a stab at doing the right thing ™

Should we just 'pack' the tasks package as non-shipping? That sounds a like a great solution to this.

Not sure if that works for how they do symbol publishing today (which is look for a symbol package per regular nupkg).


Align Fragment allocation.

Not doing so results in immediate crashes on Linux. Exclude MSVC since it apparently works anyway (larger default alignment?).
There are a myriad ways to do this. Perhaps it'd be cleaner to change the new[]/delete[] to one of the existing aligned allocation macros, or add ones for arrays specifically.
Just doing struct ALIGNED( 32 ) Fragment works too, but I imagine this may waste more memory(?)

This can (and should) simply be done with the existing MALLOC_ALIGNED, no need for new macros here.

I am in the middle of some changes, will publish a fix with the next commit.


Created Pull Request Template

Change Description

Created a template for GitHub pull requests:
/.github/PULL_REQUEST_TEMPLATE.md
Template follows this format. This template will allow AWS lambda functions to pull data from pull requests automagically. Impacts GitHub web interface, no impact on product.
API Changes

None.
Documentation Additions

None.

Additional feedback has resulted in changes which I will introduce in a new pull request.


scroll to prompt editor after sending context items to chat

When adding a file to the prompt via the menu it was unclear if anything happened if the prompt editor was out of view. We now ensure that after we give focus to the editor we scroll to it.
Test plan
With a long message ensuring the follow-up prompt was out of view, I added a file to the context via right click.
https://github.com/sourcegraph/cody/assets/187831/62bbadfc-ff43-4166-8009-99407a2847a0

@sqs not happy with how this is implement and how the scrolling works, but keen to get some feedback on how to do this. I see you put effort in to avoid scrolling, so maybe this doesn't make sense to do? This was feedback from @toolmantim when he was testing out the add file to chat menu item.


sublist: Possible missing edge case

Found a clever implementation (link) in Python for the sublist exercise that shouldn't pass all tests. It basically joins all the digits ([1, 2, 3] -> "123") and uses the in operator that checks if one string is a substring of the other.
If we would have two list [1, 0, 1] and [10, 1] that particular solution would create two strings "101" and "101" and determine incorrectly that those lists are equal. Is this way to solve this too unlikely or should we add this as a test case?

I think that is a good test case to add. :+1:
Part of a good test suite is detecting things that pass that should not.


Improves watcher tests

Purpose
Adds more coverage to watcher testing.  Also adds generalizable directory output validation.
Implementation
Parametrizes standard cases over intensity extraction.  Uses functools.partial to help create per-case validators within the watcher case
Remaining issues
per_run testing, but that's left to whoever first implements a per_run callback

Right. I think the main reason to test FOVEventHandler directly would be for our own benefit, not to catch bugs for users. In general, the shorter the stacktrace, the easier to debug when you're making changes.
In particular, I imagine in the future we'll be expanding FOVEventHandler to do some preprocessing of the data for inputs that are shared across multiple callbacks

Could you add argument types and descriptions for these parameters in the docstring?
https://github.com/angelolab/toffy/blob/525335af6ff17eeb57570c58481d86ff9c9a02db/toffy/fov_watcher_test.py#L47
https://github.com/angelolab/toffy/blob/525335af6ff17eeb57570c58481d86ff9c9a02db/toffy/test_utils.py#L124-L127

The timeout error exception is not caught appropriately. We should add a test for this and fix it

Could you add argument types and descriptions for these parameters in the docstring?
https://github.com/angelolab/toffy/blob/525335af6ff17eeb57570c58481d86ff9c9a02db/toffy/fov_watcher_test.py#L47
https://github.com/angelolab/toffy/blob/525335af6ff17eeb57570c58481d86ff9c9a02db/toffy/test_utils.py#L124-L127

for check_extraction_dir_structure, for sure.
The other function is a test function, which we usually don't add a docstring to.  In this case, the only arguments it will be provided are filled by the WatcherCases class, in test_utils.py.
If the way the code-body is working seems unclear, I could make some variable names more descriptive, add some comments to the code body, or break some of it into a different function (particularly reusable portions).

@ngreenwald adding the changes you suggested in PR #53 , then requesting review


2.0 Update - Make splashscreen loading bar not progress until finished loading things

Make the loading bar on the splashscreen not progress unless certain things are loaded (like settings)

Decided to not do this


R Documentation hyperlinks at http://mxnet.io/api/r/index.html all lead to something other than R

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System:
Compiler:
Package used (Python/R/Scala/Julia):
MXNet version:
Or if installed from source:
MXNet commit hash (git rev-parse HEAD):
If you are using python package, please provide
Python version and distribution:
If you are using R package, please provide
R sessionInfo():
Error Message:
Please paste the full error message, including stack trace.
Minimum reproducible example
if you are using your own code, please provide a short script that reproduces the error.
Steps to reproduce
or if you are running standard examples, please provide the commands you have run that lead to the error.

What have you tried to solve it?

@paulsendavidjay Really sorry for this. We are rewriting all the documents and will fix this soon.

Now they lead to Julia.  Progress!

Thumbs up!

You can the R tutorials from https://github.com/apache/incubator-mxnet/tree/master/R-package/vignettes

Can you make a priority to update the installation documents? Many still reference https://github.com/dmlc/mxnet and give incorrect directions (i.e. the make targets don't exist, the amazon CRAN mirror doesn't work, etc.).

@topepo Can you give more details about the error? Especially the make error.


K8s API Fairness configuration

Currently on Vintage we have a dynamic logic for setting the api fairness based on master size/usage. We do not have any feature for this in CAPI in general. Let's see if that is even possible or we can come up with a sane default.
Vintage feature: https://github.com/giantswarm/roadmap/issues/1360

I've started looking into this, dumping my mental notes here so tomorrow we can discuss more easily.
Goal
We need to set the --max-requests-inflight and --max-mutating-requests-inflight flags in the apiserver manifest, based on the host resources.
--enable-priority-and-fairness already defaults to true for kubernetes 1.22+, but we might want to make the flag configurable (probably not).
We might also want to set the resources requests and limits for the API server pods, based on the host resources.
k8s doc: https://kubernetes.io/docs/concepts/cluster-administration/flow-control/
Vintage
In k8scloudconfig we have:

an apiserver manifest template (https://github.com/giantswarm/k8scloudconfig/blob/master/files/manifests/k8s-api-server.yaml.tmpl)
a script that gathers the number of CPUs and the RAM of the machine and computes the proper values for the two flags (https://github.com/giantswarm/k8scloudconfig/blob/master/files/conf/setup-apiserver-environment)
a systemd service that runs at startup and renders the apiserver manifest using the computed values (https://github.com/giantswarm/k8scloudconfig/blob/a00e8039f7ae6a6a9c4d0ab6ca2216545d035537/pkg/template/master_template.go#L116).

introduced with PR in k8scloudconfig: https://github.com/giantswarm/k8scloudconfig/pull/1075
CAPI
These settings are provider independent, and in my opinion belong to the internal settings we set for our k8s foundation and that are not really meant to be changed (unless something is very wrong).
In the new cluster chart for CAPI we render the yaml manifest for kubeadm using various templates/helpers (apiserver: https://github.com/giantswarm/cluster/blob/main/helm/cluster/templates/clusterapi/controlplane/_helpers_clusterconfiguration_apiserver.tpl)
So we would need to add the max-requests-inflight and max-mutating-requests-inflight flags in the extraArgs section, and find a way to have their values computed based on the host resources.
Do we have a way of altering the clusterconfig manifest from the controlplane machine?
I guess the pre-kubeadm commands would be the right place to do that (https://github.com/giantswarm/cluster/blob/main/helm/cluster/templates/clusterapi/controlplane/_helpers_prekubeadmcommands.tpl), but currently I'm not sure how.
I'm also not sure what would be the proper way to set the apiserver pod resources.
One possible way to address both would be using kubeadm patches, e.g.:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/control-plane-flags/#patches
https://serverfault.com/questions/1089688/setting-resource-limits-on-kube-apiserver
But we need to keep in mind to re-use the same patches if we do a kubeadm upgrade or something similar.
I will check with Nikola tomorrow morning and see what is the proper way to address this.

I think we can do it in a similar way to how vintage is doing, we are already doing something similar currently for hostnames in initConfiguration, for example:

here in the initConfiguration we reference an environment variable https://github.com/giantswarm/cluster/blob/e9d4bdbbe325aa5379d3aa7a3b31b57a3aae8c6e/helm/cluster/templates/clusterapi/controlplane/_helpers_initconfiguration.tpl#L15 coming from /run/metadata/coreos that is filled by the coreos-metadata service which is a dependency of the kubeadm service https://github.com/giantswarm/cluster/blob/e9d4bdbbe325aa5379d3aa7a3b31b57a3aae8c6e/helm/cluster/ci/ci-values.yaml#L227

So, in a similar way, we can add a OneShot systemd service that executes the same script as vintage (https://github.com/giantswarm/k8scloudconfig/blob/master/files/conf/setup-apiserver-environment) writing the env variables to a file, then add that service as another dependecy to the kubeadm service and adding another EnvironmentFile directive below the current one (https://github.com/giantswarm/cluster/blob/e9d4bdbbe325aa5379d3aa7a3b31b57a3aae8c6e/helm/cluster/ci/ci-values.yaml#L234).
That should allow us to reference env variables in the apiserver section of the clusterconfig that the kubeadm service will render when executing on the machine.

To summarize, we need to add in the cluster chart, for controlplane nodes:

a script that has to write the environment file (same as the vintage one) to be put as executable (755, owner root) in /opt/bin/setup-apiserver-environment:

#!/bin/bash
env_file="/etc/apiserver-environment"

cpus="$(grep -c ^processor /proc/cpuinfo)"
memory="$(awk '/MemTotal/ { printf "%d \n", $2/1024/1024 }' /proc/meminfo)"

declare -i cpus
declare -i memory

# convert to millicores
cpus=$((cpus*1000))

# subtract dedicated millicores (see 'systemReserved' and 'kubeReserved' fields in the kubelet config file)
cpus=$((cpus-600))

# we dedicate 3/4 of all millicores to the api server
dedicated=$((cpus * 3 / 4))

# for every core we have dedicated to api server we allow 200 requests for fairness.
# 1/3 of such number is for mutating requests, 2/3 are for all requests.
echo "MAX_REQUESTS_INFLIGHT=$((dedicated / 1000 * 200 * 2 / 3))" >$env_file
echo "MAX_MUTATING_REQUESTS_INFLIGHT=$((dedicated / 1000 * 200 * 1 / 3 ))" >>$env_file
echo "CPU_LIMIT=${dedicated}m" >>$env_file
echo "MEMORY_LIMIT=$((memory * 3 / 4))Gi" >>$env_file

# We request 1/3 of the available CPUs and 1/2 of the system memory just for api server.
echo "CPU_REQUEST=$((cpus / 3))m" >>$env_file
echo "MEMORY_REQUEST=$((memory / 2))Gi" >>$env_file

a systemd unit file setup-apiserver-environment that executes the script, and on which the kubeadm service is dependent:

[Unit]
Description="Write environment file for kubeadm apiserver settings"
Before=kubeadm.service
[Service]
Type=oneshot
TimeoutStartSec=0
ExecStart=/opt/bin/setup-apiserver-environment
[Install]
WantedBy=multi-user.target

in the cluster apiserver template extraArgs (https://github.com/giantswarm/cluster/blob/0947d1b4393329aea6df0a50cf85827dee39e5e3/helm/cluster/templates/clusterapi/controlplane/_helpers_clusterconfiguration_apiserver.tpl#L16), add:

max-requests-inflight: ${MAX_REQUESTS_INFLIGHT}
max-mutating-requests-inflight: ${MAX_MUTATING_REQUESTS_INFLIGHT}
enable-priority-and-fairness: true

in the values (e.g., https://github.com/giantswarm/cluster/blob/e9d4bdbbe325aa5379d3aa7a3b31b57a3aae8c6e/helm/cluster/ci/ci-values.yaml#L234C35-L234C35), make sure the kubeadm.service runs after  the setup-apiserver-environment service, and add a new EnvironmentFile directive:

EnvironmentFile=/etc/apiserver-environment

In case you see the comment before you drop off to vacation:

max-requests-inflight: ${MAX_REQUESTS_INFLIGHT}
max-mutating-requests-inflight: ${MAX_MUTATING_REQUESTS_INFLIGHT}

Not sure if something like this (setting CLI args to env vars) works in KubeadmControPlane, at least haven't seen it before. We will try and see :)

I share the same doubt, but seeing how an env variable was used in the initConfiguration section I figured it might work also for the apiserver, since they are both part of the KubeadmControlPlane template... If not, we'll find another way :)

Opened PR https://github.com/giantswarm/cluster/pull/47

E2E tests are failing, investigating https://github.com/giantswarm/cluster-aws/pull/500

Fixed, e2e tests are passing, https://github.com/giantswarm/cluster/pull/47 merged.

Reworked fairness settings using kubeadm patches, now they work properly through upgrades.
Tested in cluster-aws and with e2e tests again, merged in cluster with https://github.com/giantswarm/cluster/pull/90


Fix version number for openttd

When the installation files of openttd were updated the version number itself was forgotten.

Thanks!


fix acceptance tests for mono

re #151
superset of #211

LGTM. TravisCI seems to be failing b/c it can't find scriptcs?

Yeah, that's the problem with TravisCI. It can't find scriptcs. In TeamCity, we set the SCRIPTCS_PATH env var and that is what the acceptance tests use to locate it. I tried to set the SCRIPTCS_PATH env var in the latest .travis.yml change but the relative path is wrong. Perhaps this just needs to be expanded to the absolute path somehow.

OK, the acceptance tests are now running on Travis, but there seem to be some genuine failures.

There seems to be three flavours of faliures

DisplayingTasks.* - these are all because of Windows vs *nix line endings in the assertions. These should be easy enough to fix.
Plugins.Exec.* - for some reason mono is returning an exit code of 2. I suspect this may be due to incorrect paths, but I'm not sure.
ScriptExecution.CompilationFails - I suspect this is because of https://github.com/scriptcs/scriptcs/issues/1054

I've fixed the DisplayingTasks.* tests.

I've skipped ScriptExecution.CompilationFails for now. This is indeed falling fowl of scriptcs/scriptcs#1054.
That only leaves the Plugins.Exec.* tests. I think the best thing to do is to simplify these by removing the Bau.Test.Acceptance.CreateFile project and switching between executing a cmd file and a bash script, depending on OS. The OS can be determined by checking the environment variable OS. In Windows this is set to Windows_NT. In *nix it is either unset or is something else.

Sorry, I should have fixed all of these when I rebased.
Agreed that removing the CreateFile project would simplify the acceptance tests. Anything I can do to help?

Branch pushed to https://github.com/bau-build/bau/tree/151


[KYLIN-819] Fix necessary ColumnMetaData order for Calcite (Optic)

See https://issues.apache.org/jira/browse/KYLIN-819

I've patched this into 0.7-staging, commit 716bd9dcc2ab43e642b106db838458a684932495

So we can close this Pull Request.


Add patch field to .Capabilities.KubeVersion

There is a use-case where cadvisor endpoint has changed between pre 1.7.2 and post 1.7.3. I want to put prometheus endpoints accordingly. We have major and minor fields, it will be useful to also get patch field.

Hey! This is a great idea, but it would have to be contributed upstream as kubernetes's core libraries only provide the major and minor version in the API. Would you mind opening a ticket upstream first? Once that's been implemented, we can then take advantage of that new feature. :)

I have cut an issue upstream #kubernetes/kubernetes/53582


[Feature]: Add testimonial for user trust

By adding stronger testimonials, we can build user trust, improve credibility, and significantly enhance the overall experience on our platform

Something like this i can redesign it according to website theme and make it more beautiful
on clicking on the arrow cards slide with an animation with create great user impact

You can add a Home page and this testimonial there.

@BamaCharanChhandogi
yup i can please assign me

do not need


Replace team leaf images with corresponding ones

Description
We are using the same placeholder for most of the team icons.
Plan

[ ] Get images from @labcollective
[ ] Put them in /src/assests/images/[teamname.png]

@labcollective if you get a chance, please add the svg files here or in trello


don't remove impure reachable parts of logical expressions

An attempt at dealing with #159.
Impure statements like function calls were removed from the right-hand side of && expressions if they were inferred to return a falsy value (e.g. when using the void operator).
This patch checks if the thing to be removed is pure. There's a lot more manual checking involved but it also simplifies more stuff.

I think I fixed my logic mistakes. I'm not entirely sure if all the branches I put in are necessary or if flip-expressions obsoletes some of them.


feat: added some basic badges reflecting the code/build status

What this PR does / why it's submitted / why we need it:
Part of the issue discussed and placed here: https://github.com/thecogworks/Cogworks.AzureSearch/issues/24. Requires some additional config on Codecov + discussion if we'd need/want to have a Github Package status. But initial state it might be like that.
To Be Reviewed.

Checklist:

[x] My PR has a descriptive title (not a vague title like Needed this)
[x] Title and all the commits follow commitlint guidelines
[x] My PR targets the develop branch or appropriate release branch
[ ] My PR provides test coverage for my change (where applicable)
[ ] I've placed the link to the corresponding Trello card or issue which this PR fixes/address below ⤵

Fixes: N/A

@mzajkowski add also budget info for nuget packages.

@aochmann There is a badge for NuGet added 👍

@aochmann There is a badge for NuGet added 👍

@mzajkowski
Should we add for now also for all extension methods ?


AppEngine NotFound and silent error when initialising project location

In the GCP Pilot API base class here, the location is set, in the following priority:

explicitly provided value
default from a setting
project default determined using AppEngine here

I think historically, GCP projects had a default AppEngine resource always available. Nowadays they don't (I think this only happened in the last year or so).
This code handles a NotFound exception silently, catching it to return None. This however, causes many problems later on, a variety of NotFound or PermissionDenied errors when trying to create or access resources at project/<project-id>/location/None/rest-of-it
I think that it's an antipattern allow the location value to be None at the end of the base init, we should know at the point of failure there's been an error rather than much later. How about somehting like:

class GoogleCloudPilotAPI(abc.ABC):
    def __init__(
        #stuff
    ):
        #stuff
        self.location = self._set_location(location=location)
        if location is None:
            raise NoLocationError("Couldn't set location from either explicit argument, GCP_LOCATION environment variable or from AppEngine")

?

You're totally correct regarding the location requirement.
Since there's APIs that require and APIs that don't, we should only fail when the location is used (or maybe earlier if not set but required?).
If you have an approach in mind, please create a PR. :)


symtrack - dead code (?) and decision directed equaliser

In the symtrack object, the member variables agc_bandwidth, eq_bandwidth and pll_bandwidth don't seem to be used anywhere in liquid. I did a quick test of removing them and building from source, everything works fine. I can see in the function SYMTRACK(_set_bandwidth) that identically named local variables are used. So maybe in a past life, the member variables were used.
In SYMTRACK(_execute), there is a comment to use decision directed equalisation.
Is it as simple as doing the following:
TO d_prime;
MODEM(_get_demodulator_sample)(_q->demod, &d_prime);

and replacing
EQLMS(_step)(_q->eq, d_hat/cabsf(d_hat), d_hat);

with
EQLMS(_step)(_q->eq, d_prime, d_hat);

?
There is another comment about checking the lock conditions of the previous object. Is it the case of tracking the error vector magnitude (evm) using
float evm = MODEM(_get_demodulator_evm)(_q->demod);

and activating the equaliser when it gets below a certain value?

Dupplicate of https://github.com/jgaeddert/liquid-dsp/issues/210


Accessibility and Instructions for Azure Data Factory Data Flow Sinks

Hey folks -
Was looking for more information about data flow sinks settings since I had questions for my specific use case. What is provided is a non-captioned video and no option of written instructions.
For a few reasons, this is not a great choice. To be accessible, this video player needs to be able to provide captions OR it needs to be put into the video itself. Furthermore, this should have listed out instructions with the options, or links to other parts of the Microsoft Docs where this information is covered in detail. To do neither feels incomplete.
Are there any plans to include this documentation written out or changing the video to have captions? Both would be preferred/recommended. (As well as any other videos on this section of the ADF documentation.)

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

ID: b52f7d74-8aac-9e9a-e13e-112eaf2e434b
Version Independent ID: 2e16d8b4-2081-f6d4-9879-5774e9fb1dd1
Content: Sink transformation in mapping data flow - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/data-flow-sink.md
Service: data-factory
Sub-service: data-flows
GitHub Login: @kromerm
Microsoft Alias: makromer

Thanks for the feedback! We have assigned the issue to the content author to further review this and update the document as appropriate.
Hi @kromerm, @jonburchel  - Could you please review the above feedback and help improve the content. Thank you

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Dockerize the application

Heyho,
I dockerized git-badges^^

replied

could you update the readme too?

Sure. Give me a couple of minutes.


Clarify the Location fields with additional documentation

[x] finalize description in the official documentation
[x] decide on recommendations for crosswalking solr_geom
[ ] create separate help page that goes into more depth about how the Spatial fields relate to other schemas and how function within GeoBlacklight

Re: crosswalking solr_geom...
I think all the old values of solr_geom should be in the ENVELOPE format, so the value of solr_geom can safely be copied to both the new locn_geometry and dcat_bbox fields.

I like @rmseifried's suggestion to add a "geometries" section to the "fields" page currently in progress at https://github.com/OpenGeoMetadata/opengeometadata.github.io/blob/28-migrate-content-from-wiki/docs/about-our-schema/fields.md
This section would explain the differences between the 3 fields, and also illustrate with some map images.

I agree that this sounds like a good place for the geometry field documentation.

We agreed that we want a new page (under Resources for now) that discusses the differences between these fields:

Spatial Coverage (S)
Geometry (S)
Bounding Box (S)
Centroid


Update tutorial contracts with Options/Results types

This resolves #11 -- the public functions in the tutorial contracts now return Response types, and the fetcher/getters now handle the Option results correctly.
This also required a change in the blockstack-core provider, since the "Transaction successful" output changed slightly (it now also prints out the response object).

Awesome. I'm working on updating the clarity binaries / git tags as well.


WebView crashes on Android while opening the file chooser with RN v0.74 in bridgeless mode

Bug description:
The WebView crashes on Android while opening the file chooser with the following message:
Handling uncaught Java exception
java.lang.IllegalStateException: Trying to call native module before CatalystInstance has been set!
	at com.facebook.react.bridge.ReactContext.raiseCatalystInstanceMissingException(ReactContext.java:154)
	at com.facebook.react.bridge.ReactContext.getNativeModule(ReactContext.java:207)
	at com.reactnativecommunity.webview.RNCWebChromeClient.onShowFileChooser(RNCWebChromeClient.java:336)
	at WV.RZ.h(chromium-TrichromeWebViewGoogle6432.aab-stable-642216533:35)
	at WV.W8.runFileChooser(chromium-TrichromeWebViewGoogle6432.aab-stable-642216533:34)
	at android.os.MessageQueue.nativePollOnce(Native Method)
	at android.os.MessageQueue.next(MessageQueue.java:335)
	at android.os.Looper.loopOnce(Looper.java:162)
	at android.os.Looper.loop(Looper.java:294)
	at android.app.ActivityThread.main(ActivityThread.java:8177)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:971)

This only happens with RN v0.74 in bridgeless mode.
When bridgeless mode is disabled, as described here, everything works as expected.
This might be related to #3352.
To Reproduce:

Load a website with a file input
Click on the file input
App / WebView crashes

Expected behavior:

Load a website with a file input
Click on the file input
File chooser opens

Environment:

OS: Android
OS version: 14
react-native version: 0.74.2
react-native-webview version: 13.10.4

I got same issue and found out the cause.
                                                                                Caused by: java.lang.IllegalStateException: Trying to call native module before CatalystInstance has been set!                                                                                at com.facebook.react.bridge.ReactContext.raiseCatalystInstanceMissingException(ReactContext.java:154)                                                                                at com.facebook.react.bridge.ReactContext.getNativeModule(ReactContext.java:207)                                                                                at com.reactnativecommunity.webview.RNCWebChromeClient.onShowFileChooser(RNCWebChromeClient.java:336)
So I inspected RNCWebChromeClient.java and found that it sill called getNativeModule() on architecture context.
You can use my patch to make your input=file work again.
patch-webview.patch

@moonoiza The patch only applied if I kept the changes in RNCWebChromeClient.java.
Would it be okay for you to submit a PR for this change?

Any updates on this, How do i apply the patch ?

looks like this is fixed in RN version >=0.75.0 can not repro on neither 0.75.4 (on new architecture on bridgeless) and on 0.76.1 (on default i.e on new architecture) as well


Entity Framwork Designer crashes VS 2019 Preview 4

From VS Feedback:
When I pull up an existing EDMX for editing, VS 2019 crashed as soon as I click into the designer. Everything works fine in VS 2017.

EF Team Triage: This issue is lacking enough information for us to be able to effectively triage it. In particular, it is missing the following information requested in the new issue template. Can you please provide this information?

Steps to reproduce

Ideally include a complete code listing that we can run to reproduce the issue.
Alternatively, you can provide a project/solution that we can run.
BTW we're not just doing this to be mean :smile:... we get a lot traffic on this project and it takes time to attempt to reproduce an issue based on fragments of information. In addition, our attempt is often unsuccessful as the exact conditions required to hit the issue are often not explicitly included in the code provided. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we ask that folks give us a self-contained way to reproduce an issue.
For a guide on submitting good bug reports, read Painless Bug Tracking.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


support manual_define prev_output_tokens

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[x] Did you read the contributor guideline?
[x] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Add support for manually defined prev_output_tokens in LanguagePairDataset.
This is important for recent models such as MASS(not start with PAD but the last token), ProphetNet, .etc.

Merged in ffecb4e3496379edf5ecae1483df5b7e0886c264.


Validate values being stored for preferred boost version and switch to cookies

This pull request validates the versions being stored (per #1168) for the preferred Boost version selection. It also switches this functionality to using cookies.

Suggested commit messages:
"sessions reject invalid versions from URL"
"session versioning uses cookies"
As for the "ruff" well just squash that :)

@sdarwin tagging you so you can take a look at the new cookies being used for CDN usage.
The cookies are named: library_view and boost_version


[Impeller] Provide example on how to build and use Impeller without relying on Flutter and its build system

Use case
I'm having trouble using/building the Impeller library on its own in conjunctions with GLFW to make use of the AIKS/Entity API.
Proposal
I think it would be useful if Impeller had an examples folder that showed how one could build/use the library and create a simple program with an OpenGLES backed context in GLFW to use the AIKS/Entity API. Something similar to the Playground but simpler and in a single file. Impeller could be a viable option to users making use of Skia or IMGUI in their application.

The Impeller API is a bit of a work in progress. We're definitely focusing on the integration with Flutter right now, and I don't anticipate that we'll do a lot work in the near term to make this eaasier.


Add endpoint for fetching channels a user is a member of

GET /users/:user_id/channels
If called unauthorized, only publicly accessible channels will be returned.

Codecov Report

Merging #112 (69e9467) into main (4078e47) will increase coverage by 0.02%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
+ Coverage   79.82%   79.85%   +0.02%
==========================================
  Files          98       98
  Lines        4674     4690      +16
  Branches      641      644       +3
==========================================
+ Hits         3731     3745      +14
- Misses        761      762       +1
- Partials      182      183       +1

Impacted Files
Coverage Δ

app/routers/users.py
98.46% <100.00%> (+0.12%)
:arrow_up:

app/services/channels.py
78.21% <100.00%> (+0.82%)
:arrow_up:

... and 1 file with indirect coverage changes

Resolved by 7fe2209482d0354688c229acff3cb4781e69da9c


Issue with load_km() - Python crashes arbitrary

Hello, everyone,
I have a problem with the load_km() routine.
When I try to get the mass and stiffness matrix from a .full file with the load_km() routine, Python sometimes crashes and sometimes not. Is there a problem with the load_km() routine?
Here is the code I use for testing:
    # load .full file
    result = pyansys.read_binary('file.full')

    # get stiffness matrix k and mass matrix m
    dof_ref, k, m = result.load_km(sort=True)

I am using Python 3.6 in the Anaconda IDE as well as ANSYS 2020 R1. I installed pyansys via pip install pyansys.
Thanks for your help.

Here is the .full file that I use for testing.
fullFileTest.zip

Thanks!  Sounds like an array allocation error.  I'll take a look at this.

Thank you very much!
Von: Alex Kaszynski<EMAIL_ADDRESS>Gesendet: Donnerstag, 4. Juni 2020 19:38
An: akaszynski/pyansys<EMAIL_ADDRESS>Cc: Holzinger, Stefan<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [akaszynski/pyansys] Issue with load_km() - Python crashes arbitrary (#229)
Thanks! Sounds like an array allocation error. I'll take a look at this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/akaszynski/pyansys/issues/229#issuecomment-639001286, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF6YV2UVMF2YPTTZZKU5Q6DRU7LWBANCNFSM4NS2NNGA.

Fixed in #231.  I'll have this out in the next patch fix.

Hello,
I'm pinging you on this closed issue because GitHub doesn't have an email address for you.
ANSYS is busy improving and extending pyansys.  As an existing user, we are interested in learning about your experience and opinions using this package.  Most notably:

What is your application?
How does pyansys facilitate your work?
What new pyansys features or improvements would you find valuable?
Your feedback will help us prioritize our investments and make pyansys even better.

On behalf of the pyansys development team, I look forward to hearing any feedback from you as it will be used to improve and advance pyansys.
Feel free to respond here to this issue, or email me at<EMAIL_ADDRESS>Alex Kaszynski


try-it-out for Issue classification using wikipedia data in VS2017/VS2019 doesnt work

Validated Wikipedia sentiment data set works in Issue Classification.


Improve README.md

Motivation and Context

README.md had several visual problems, which I decided to fix.
Description

The first problem was a dotnet-ci status badge, which is no longer available. I replaced it with two status badges: dotnet-ci-docker and dotnet-ci-windows.
The second one was Using Semantic Kernel in C# and Python. The icons were positioned out of place. I have made a small table with the icons and hyperlinks present earlier.
And finally, small corrections such as h3 lines for C# and Python approaches above getting started with the basics instructions.

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

@MixeroTN please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

Thank you @MixeroTN!


Add plugin/swAutoSubmit/onChangeSelection-event

Description
Please describe your pull request:

Why is it necessary? Missing event in jquery-plugin
What does it improve? Developer-Experience. It's always about developer-experience for me
Does it have side effects? No

Questions
Answers

BC breaks?
no

Tests pass?
yes

How to test?
$.subscribe('plugin/swAutoSubmit/onChangeSelection', e => console.log('autosubmit', e)

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-18236.
Please use this issue to track the state of your pull request.


Fix tests

There are four problems with the tests right now, all also present in upstream.
1- (DONE) Labextension-ui tests fail due to the JupyterLab announcements pop-up. The simple solution is to disable the extension causing it. Ideally this should be in playwright config and doesn't interfere with users own settings, but I wasn't sure how to do that. So I just do it in the python script that calls the tests.

This means that if you run the tests locally, the "state" of your computer is changed because the announcement extension is disabled. This is pretty minor though so I think it is okay.
One weird thing is that running the playwright tests in debug mode works fine. So I suspect something like increasing the size of the window enough to remove the popup from important ui elements might also be a band-aid fix.

2- (DONE) The SDist/check-release errors are (most likely, I am pretty sure but I didn't look too carefully yet) caused by something along the JupyterLab chain requesting node-canvas pinned to a version that doesn't have a build for the node version used in testing. Then the package manager tries to build the package itself and fails due to missing pangocairo.

I don't know what is the best fix yet, but something as simple as fixing the node version should work.
A better idea is to update the node-canvas version. JupyterLab repo itself had this issue at one point and this is probably what they did too. I'll take a look when I can.

3- (DONE) A link in docs needs to be corrected.
4- (DONE) Some tests take too long for the current timeout setting.

This is now merged upstream, except the fixes are slightly different for c777ce3 and 391dc40.


revert client.py

I misunderstand the method and found that we only need to remove "is_favorited" in the construction of the class Experiment.
So, make the parse_obj back to training.
And also remove the "is_favorited" in the test case.

LGTM


Gems incompatibility is blocking the installation of alphavantage!! ❌

Steps to reproduce 🦮

Add gem 'alphavantage' to Gemfile
bundle install

The result 🪲
Bundler could not find compatible versions for gem "faraday":
  In snapshot (Gemfile.lock):
    faraday (= 1.1.0)

  In Gemfile:
    alphavantage was resolved to 1.2.0, which depends on
      faraday (~> 1.4)

    pronto-fasterer was resolved to 0.10.0, which depends on
      pronto (~> 0.10.0) was resolved to 0.10.0, which depends on
        octokit (>= 4.7.0, ~> 4.7) was resolved to 4.19.0, which depends on
          faraday (>= 0.9)

    pronto-fasterer was resolved to 0.10.0, which depends on
      pronto (~> 0.10.0) was resolved to 0.10.0, which depends on
        octokit (>= 4.7.0, ~> 4.7) was resolved to 4.19.0, which depends on
          sawyer (>= 0.5.3, ~> 0.8.0) was resolved to 0.8.2, which depends on
            faraday (> 0.8, < 2.0)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

@Victorcorcos I think you need to resolve your gem compatibility issue with the rest of your other dependencies. From what I can see, there doesn't seem to be any issue on the gem. Both your dependencies require faraday (>= 0.9) and faraday (> 0.8, < 2.0), which meets those criteria


added functions for cleaning RiverMapper invalid geoms

The problem and solution are explained here
The following functions are added:
validate_poly
find_polyneighbors
validate_RMmesh
triangulate_rivermapper_poly
the previous triangulate_rivermapper_poly function was renamed to triangulate_poly

@SorooshMani-NOAA, this PR took a few attempts because as you might remember the GitHub Actions workflows using v3 were deprecated earlier this month. So, I had to update all the workflows to use v4.
The Pylint check also seem to have been updated. Our Pylint score went down (failed), apparently because now our too-many-arguments functions are being flagged. It will require a lot of work to refactor all the OCSMesh functions flagged. For now I just updated the pylint run to be:  pylint --disable=too-many-arguments ocsmesh


chore,refaactor(egen): minor changes to sdk feature store pandas notebook

This PR includes the following updates:

Adds Colab Enterprise link and other structural changes from the new template.
Changes REGION variable name to LOCATION.
Removed os.getenv("IS_TESTING") while cleanup bucket.
Adds comments for cells needed.

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

/gcbrun


Checkstyle formatter does not include error free files.

Bug Report

TSLint version: 5.9.1
TypeScript version: 2.8.1
Running TSLint via: CLI

TypeScript code being linted
// error free code

with tslint.json configuration:
{
    "extends": [
        "tslint:recommended"
    ]
}

tslint --project . --format checkstyle index.ts
Actual behavior (newlines added for readability)
<?xml version="1.0" encoding="utf-8"?>
<checkstyle version="4.3">
</checkstyle>

Expected behavior
<?xml version="1.0" encoding="utf-8"?>
<checkstyle version="4.3">
<file name="/path/to/src/index.ts"></file>
</checkstyle>

As noted in https://github.com/palantir/tslint/issues/250, this file is the closest thing to an official spec on the format.
The relevant bit is that writeFileMessages writes a tag for each file linted. Errorless files are just empty tags.
As I'm finding out, some parsers need those tags. Otherwise they return an error message stating no lint checks were preformed and/or they allows show 100% of files have issues.
I opted to open an issue on this instead of submitting a PR as I'm unsure of the preferred way to handle this. The Linter class doesn't store all the files it lints and therefore can't pass them to the formatter in getResults() as is. One option would be to just add each file in turn in lint(), then add a third arg to IFormatter.format.

Seems related to https://github.com/palantir/tslint/issues/3744

This was resolved with https://github.com/palantir/tslint/pull/3838


LL-1589: See related transactions in ETH fee operations

:information_source: Related PR: https://github.com/LedgerHQ/ledger-live-desktop/pull/2201

This PR adds related subOperations to the operation detail view. It helps users understand why some transactions they may not have made are there (e.g. in the case of ETH, when sending tokens, an ETH fee will apply which shows up as a separate transaction).
:camera_flash: Screenshots

:ok_hand: Type
Feature / UX
:mag: Context
LL-1589
:clipboard: Parts of the app affected / Test plan
Operation details

Possibly to-do: being able to go back to the previous operation details when opening sub-operations?


rebase on current dune-fem or grid views

We get deprecation warnings in the old grid part code copied from dune-fem (to be removed after 2.3). We should rather depend on dune-fem and base stuff on their grid parts.

Except that we do not necessarily want to depend on dune-fem, of course. If possible, we could provide our own implementations if dune-fem is not available, else use the ones from dune-fem.


Disable multiple tabs being open side-by-side

Is your feature request related to a problem? Please describe.
Multiple tabs open side by side cause issues with navigation. I have an id set in the path parameters and I want this id to reflect the open panel. It makes it hard to track which tab is open when multiple tabs are open side-by-side.
Describe the solution you'd like
A prop on dockview component that allows me to disable the side-by-side and drag and drop into multi window opened layout.
Describe alternatives you've considered
I could not create a solution with existing props available.

Could you explain in more detail the problem pleasesince I'm not entirely sure what the problem you are describing is?

Will re-open if issue has a response


Supabase api url vs hosting locally

Is there an easy way to host the database directly on Supabase using an api url vs locally hosting it through a docker container?

Read up on the supabase docs more and found a solution my bad

Any chance you'd willing to remove this issue I'm going to make a pr later with steps with deploying to supabase.com rather then just a localhost docker setup.

If you want to use a Supabase project instead of running a local Supabase instance, you can copy the supabase api url and the anon public key and then paste them into the .env file.
I will setup a guide for this.

added a guide on how to link to a supabase project in #23


nuxt 3 case sensitive url

Describe the feature
I found out in default nuxt 3 is case insensitive for URLs
as you know duplicate contents are not ok for SEO, please change this default to case sensitive mode
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

You can do this with an app/router.options.ts file:
import type { RouterOptions } from 'nuxt/schema'

export default <RouterOptions> {
  sensitive: true
}

See https://nuxt.com/docs/guide/going-further/custom-routing#using-approuteroptions

Yes, you are right
but I say the default value with out customizing.
Why the default value is false? why all projects build with nuxt should have case insensitive URLs in default!
we know whats the bad thing duplicate content in SEO. and if we have case insensitive URLs we have multi URLs show same content!
So I mean that its better to have case sensitive URL in default. if some one is special case need case insensitive URL he/she will change it!

This is a vue-router default: https://router.vuejs.org/guide/essentials/route-matching-syntax.html#Sensitive-and-strict-route-options.

As a site note: In most cases (there are exceptions!), uppercase URLs can be confusing for the user and I'd suggest to keep them all lowercase. You would add a middleware or redirects then to ensure /AbC will be transformed to /abc.


staticpod: fix cert sync controller types to be unrevisioned

Follow-up of https://github.com/openshift/library-go/pull/1089.

/lgtm


[BUG]Kibana alert can not send to customer webhook with '/Path' through 'Http'

 Issue by lijingfz
Wednesday May 13, 2020 at 06:16 GMT
Originally opened as https://github.com/opendistro-for-elasticsearch/alerting/issues/202

Describe the bug
Find a strange behavior for Kibana alert destination for 'customer webhook'.
Http post can not add customer define '/Path' , so can not trigger customer webhook.
Other plugins installed
Please mention if you are using this plugin along side any other plugin. Security for example.
To Reproduce
Kibana alert destination config as below :
http
host ： ec2-52-83-117-220.cn-northwest-1.compute.amazonaws.com.cn
port ： 1337
path : /webhook
Key : webhook value: jingamz-test-20200412
Meanwhile, build a simple webhook server to simulate customer webhook server. detail procedure as below :
mkdir messenger-webhook
cd messenger-webhook
touch index.js
npm init
npm install express body-parser --save
[ec2-user@ec2 messenger-webhook]$ vi index.js
'use strict';
// Imports dependencies and set up http server
const
express = require('express'),
bodyParser = require('body-parser'),
app = express().use(bodyParser.json()); // creates express http server
// Sets server port and logs message on success
app.listen(process.env.PORT || 1337, () => console.log('webhook is listening'));
//Creates the endpoint for our webhook
app.post('/', (req, res) => {
let body = req.body;
console.log(body);
console.log("Hit path /");
}
);
app.post('/webhook', (req, res) => {
let body = req.body;
console.log(body);
console.log("Hit path /webhook");
}
);
[ec2-user@ec2 messenger-webhook]$ node index.js
webhook is listening
{
text: 'Monitor jingamz0412 just entered alert status. Please investigate the issue. - Trigger: 0412 - Severity: 1 - Period start: 2020-04-16T02:57:47.977Z - Period end: 2020-04-16T02:58:47.977Z'
}
Hit path / <---- Notice here , hit '/' and NOT include path '/webhook'
^C

And also use 'nc -l 1337' to capture alert post information as below :
[ec2-user@ec2 messenger-webhook]$ nc -l 1337
POST /?webhook=jingamz-test-20200412 HTTP/1.1
Content-Type: application/json
Content-Length: 201
Host: ec2-52-83-117-220.cn-northwest-1.compute.amazonaws.com.cn:1337
Connection: Keep-Alive
User-Agent: Apache-HttpClient/4.5.7 (Java/1.8.0_242)
Accept-Encoding: gzip,deflate
{ "text": "Monitor jingamz0412 just entered alert status. Please investigate the issue. - Trigger: 0412 - Severity: 1 - Period start: 2020-04-16T01:38:49.284Z - Period end: 2020-04-16T01:39:49.284Z" }
Below we add path /webhook, but Kibana alert do not add path in header.
[ec2-user@ec2 messenger-webhook]$ nc -l 1337
POST /?webhook=jingamz-test-20200412 HTTP/1.1
Content-Type: application/json
Content-Length: 201
Host: ec2-52-83-117-220.cn-northwest-1.compute.amazonaws.com.cn:1337
Connection: Keep-Alive
User-Agent: Apache-HttpClient/4.5.7 (Java/1.8.0_242)
Accept-Encoding: gzip,deflate
{ "text": "Monitor jingamz0412 just entered alert status. Please investigate the issue. - Trigger: 0412 - Severity: 1 - Period start: 2020-04-16T01:47:47.665Z - Period end: 2020-04-16T01:48:47.665Z" }

 Comment by skkosuri-amzn
Monday Apr 05, 2021 at 02:00 GMT

hi @lijingfz, whats the error message or exception you are getting? Could you please paste it.


Add support for Apple Silicon Macs

Adds code to detect when mac users are on ARM, and adds ARM binaries for Pycairo, Pygame, and PIL

Looks good to me! Why did the old binaries (e.g. cairo_mac/cair/cpython-36-darwin_dylibs) change?

Also, can we enable python 3.10 tests on Windows? I'm not sure why they are disabled, but if it is something like Appveyor didn't have python 3.10 installed, maybe that has changed.

Good catch with the old binaries. The change might have just been due to notarization, but I might've made a mistake too. So I reverted them for now and then I'll re-sign if necessary.
As for why we're skipping 3.10 tests on Windows, here's what we said in the code:
# The pip version fails because of an issue with the Windows binaries for pycairo,
# so skip it here. We also have to skip py310-zip because its version of pycairo
# has the same bug.

I just tried enabling them anyway. We'll see if the bug is fixed.


[Bot] Support helping bot with assigning

Example:
/assign Tyriar

Does this work for me?
/assign roblourens

Does it need /unassign? 🤔

Does it need /unassign? 🤔

Probably helpful, especially as bot will soon be auto-assigning people.

#96600


⚠ Bofrost.at has degraded performance

In 171124b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1622 ms

Resolved: Bofrost.at performance has improved in 4ff28ec.


Append search term to file name

Would it be possible to append the search term onto each of the downloaded samples (or have an option to do so).
For example, running python3 shabda_cli.py cats --num 10  1517  python3 shabda_cli.py cats --num 10 would create the folder cats and each file in the folder would be 1_cats.wav, 2_cats.wav, 3_cats.wav etc.
The use case for this is that I am currently using Shabda to put random samples on an MPC Live. It stores all of the used samples in the same directory, so if two files have the same name they get overwritten.

That's a totally valid use case :)
I'm gonna tackle this one after #1 .
By the way it's possible that I slightly change the cli arguments.

Your use case should now be covered in the latest version :)
Please tell me if it fits your needs!


setPosition does not trigger requestOccurencesHighlight

The setPosition function moves the position of the cursor but does not highlight Occurences
Playground example:
https://hediet.github.io/monaco-editor/playground.html?source=v0.35.0#XQAAAAI_AQAAAAAAAABBqQkHQ5NjdMjwa-jY7SIQ9S7DNlzs5W-mwj0fe1ZCDRFc9ws9XQE0SJE1jc2VKxhaLFIw9vEWSxW3yscrWEItUcqLhiEseIhl2yvSTKsikuAnq3yOVCHCMPueylLD-VoHOlF3w7elBXLBzOq_Ka4_q7xdHjgZ-qzK6kFkzMBm0ZwMw3Gbr1YZ6zmAYxn0cCVF1phVXwEveHgU5jpptNHId3fhAFIUeiyUDKF-NGP89sgNqakT1kv442b9c8lHmcC3iNglRH0u_3lSWFJhSCSbG8qK-4okKyQyw2PYMvEsAMO6eh2mJfje_t9AfBcGlBJ1hJbxT345NtazqmOIt_sk0KxsM__QK84A

This is intentional. The occurrence highlighter only runs for explicit cursor gestures -- https://github.com/microsoft/vscode/blob/d912e6c92514b3dd524fe6ff1332a06c5f1d5534/src/vs/editor/contrib/wordHighlighter/browser/wordHighlighter.ts#L332-L336
@siddhant1 You can try pretending to be the mouse and use editor.trigger('mouse', '_moveTo', { position: { column: 13, lineNumber: 1 } }). See playground example.


lag?

why is it that when i have this theme on and i type fast it seems to lag lol?

i removed the theme and it seemed to fix it 🤔

https://i.imgur.com/JJj2yI8.png
reading names in VC's is also pretty horrible.

reading names in VC's is also pretty horrible.

i'm aware, there are a lot of small places that need to be patched because discord sucks

why is it that when i have this theme on and i type fast it seems to lag lol?

how strong is your computer? themes will inevitably cause lag, especially attribute selector based themes like this one. i just need to know if there's something making it really bad.
can you try my midnight theme and see if you still get lag? because if you get lag on that as well then its just your computer not being good enough.

GTX 1060 6gb
16gb ram
i5 8600

can you try my midnight theme and see if you still get lag? because if you get lag on that as well then its just your computer not being good enough.

please try this.

midnight theme feels smooth when typing fast

alright i'll see what i can do, but to get same performance as midnight theme would require a complete rewrite of the theme.

The lag isn't anything crazy, just more of an OCD thing, and only happens when I'm typing extremely fasting.

hey, can you check if performance has improved?

hey, can you check if performance has improved?

Hey, I was having the same problem, when typing fast, get slow for some reason.
With this update seems to be working like a charm now.
Btw, your theme is really beautiful!

If this helps in someway, it used to happen in a channel with a lot of text.

Like this one:

great. if anyone still has lag problems, please comment, otherwise i will close this issue.

At least for me it's fixed

I dont even notice it anymore like i used to, so i would say its fixed

I noticed that lags occur on my main computer:
Xeon E5 2650 V2
GTX 1660 Super
16 GB RAM
And on my laptop everything works smoothly:
HP Pavilion Plus Laptop 14-ey00
It has an AMD processor and integrated graphics. On the main computer it lags when typing text, but not on the laptop. Strange.


sample "Proactive message bots app" missing AAD environment variabes when provision into azure env

I use VSC teams toolkit to create sample "Proactive message bot" in sample gallery.
Sample: https://github.com/OfficeDev/Microsoft-Teams-Samples/tree/main/samples/bot-proactive-messaging-teamsfx
When I provision in to azure, I have following issue.

@ayachensiyuan - Thanks for reporting your issue. We will check and update you soon.

@Nivedipa-MSFT thank for your reply. I still face this issue, the version of TTK is the same. It works in local debug. but when I run provision in to AZURE, it failed with that error.

@ayachensiyuan - Could you please verify that all necessary environment variables are correctly set in your Azure portal?
Verify that your Azure resources are properly configured and that you have the required permissions?

@Nivedipa-MSFT We have daily auto test for TTK, So this sample in TTK gallery failed since half month ago. Usually we test it according to readme file, It is certain that using the current version of TTK to create this sample and following the instructions in the readme to provision and deploy will encounter the problems mentioned above.  All of this was automatically deployed by TTK, and I did not have to manually operate on Azure. Here's the test: https://github.com/OfficeDev/teams-toolkit/actions/runs/11419663804/job/31778539159

@ayachensiyuan - We have tested this on our end but were unable to reproduce the issue. Could you please clone the latest repository and test again?

Double confirm with you the repo steps:

clone the latest sample repository main branch
switch the folder root to bot-proactive-messaging-teamsfx

open sidebar teams toolkit extension
login azure and m365 account
click provision button and fill in prompt information

wait for error log

Closing this issue for now. Please free to reopen if required.


Isolate Served Notice V2

Closes #PROD-2112
Description Of Changes
Plus feature https://github.com/ethyca/fidesplus/pull/1429 stops writing to lastservednoticev2 altogether.
Not removing this table yet, but in the meantime, isolating the table so no further changes are made to it.
Code Changes

[ ] Separate LastServedNoticeV2 so it no longer inherits from shared mixins with other privacy preference related models
[ ] Mark as deprecated in code comments

Steps to Confirm

[ ] list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

@adamsachs when you have a minute, this is a small non-blocking accompanying change to the Privacy Preferences Low Hanging Fruit


Fleet management

From discussion: we should add a feature that supports fleet management via e.g. a hierarchical objective. We can provide some sort of class for this.

This has to happen for SINTEF. I'll write a bit more here later today.

Some thoughts on this:

maybe to determine how long we should try to find a feasible solution, we could run all SINTEF cases with the BKS num routes as num vehicles and measure how long it takes to find a feasible solution over a few seeds? Then we could set a value that is a nice trade-off in time vs prob of finding minimum number of vehicles.
not sure how often we attain a lower bound (e.g. based on demand vs vehicle capacity) but it may be worth stopping based on this

The more I think about this issue the less I want to do it. The hierarchical objective is nice, but not something we really care about outside of the SINTEF thing. It'd need quite a bit more code to support, and I don't really see any value in that in the long run.
What if we decide this is a non-goal? The argument "we do routing, not fleet minimisation" is pretty strong. We can then probably just 'cheat' and use the current best number of vehicles if we're explicit about the why in the paper.

I think it's nice if it's not too difficult but I don't see a real need either in the homogeneous case. I could use it for heterogeneous fleet, but then it is also (much) more complex. I'm ok with the decision not to implement it now.


Error on VS Code Version 1.34.0

When I try to format any php file always return an error:
Cannot format 'file-name.php'

I think this file hadn't been saved yet thats why it was showing up as /Untitled-1


[BUG]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[ ] My answer given in the checkbox above is a lie.

What problem did you encounter?
Shaw (Extremely popular internet provider in Canada) is blacklisting bloxstrap if the user has "Advanced Security" set on

Blacklists any URL with bloxstrap.*
Discord thread id "1263952551540166656"

bloxstrap.pizzaboxer.xyz is the real website these are just false-positives. However, bloxstrap.org is a fake website that... well I don't know actually, people say it literally is just a website that gives a real Bloxstrap installer.

I'd like to contact them but this happens for whatever reason:

So I can't really do anything about it. If you're willing to put forward a tip then that would be nice.

give me the message to send it to them
On Sun, Jul 28, 2024, 2:03 p.m. pizzaboxer @.***> wrote:

I'd like to contact them but this happens for whatever reason:
image.png (view on web)
https://github.com/user-attachments/assets/2092e268-7e5e-4f3b-a571-8d35a419a400
So I can't really do anything about it. If you're willing to put forward a
tip then that would be nice.
—
Reply to this email directly, view it on GitHub
https://github.com/pizzaboxer/bloxstrap/issues/2430#issuecomment-2254628946,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLKLQL7K34RHRVXP3K5KY3ZOVE75AVCNFSM6AAAAABLFGQLCOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGYZDQOJUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Just tell them something along the lines of "https://bloxstrap.pizzaboxer.xyz is falsely being blocked for being malicious, when it is a legitimate website as shown on https://github.com/pizzaboxer/bloxstrap"

done
On Mon, Jul 29, 2024, 9:59 a.m. pizzaboxer @.***> wrote:

Just tell them something along the lines of "
https://bloxstrap.pizzaboxer.xyz is falsely being blocked for being
malicious, when it is a legitimate website as shown on
https://github.com/pizzaboxer/bloxstrap"
—
Reply to this email directly, view it on GitHub
https://github.com/pizzaboxer/bloxstrap/issues/2430#issuecomment-2256308102,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLKLQP5PBZNUPRWKZPHU7LZOZRGNAVCNFSM6AAAAABLFGQLCOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJWGMYDQMJQGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add "awesome" topic

[x] I followed the contributing guidelines
[x] I'm curating a new Topic page
[x] I've formatted my changes as a new folder directory ...
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs

Awesome lists are awesome.
There are already over 800 of them.
They are an awe... a great contribution to the open source community.

If you have any suggestions on how I could improve the wording of the description (using "awesome" all the time is weird), please let me know.

The tests are failing because the links to the repositories I mentioned as examples of awesome lists github in lowercase. This should be fixed. @cheshire137, @MikeMcQuaid could you look into this?

@shroudedcode Sorry about that, have opened https://github.com/github/explore/pull/54 to address this.

@shroudedcode I looove that you curated this topic! 🙌
As mentioned in https://github.com/github/explore/pull/27#issuecomment-336217124, we don't yet support Markdown in full descriptions. Fix is in progress, waiting for that to land before merging this one. Please stand by. 😎

Thanks for your first contribution to github/explore, @shroudedcode!  You rock!


Since 0.6.14, overlay is extremely slow while PoE has focus

I'm talking control+d for a lookup, the UI itself fades in slowly over the course of about 15 seconds.
Additionally, the UI is VERY sluggish and responds very slowly to mouse events (hover, click, etc) and also takes forever to go away when dismissed with escape.
On 6.14 I was able to alt tab to put the overlay in focus which would basically make it appear instantly which was fine enough, but now with 6.15 that doesn't seem to work anymore.

Hi, thanks for creating an issue. Can you confirm this is still an issue with the latest version? Thanks!

Hi, I have a similar problem. The overlay opens up pretty quickly, but the connection to pathofexile.trade takes about 10 seconds to load, even if it's unique items. With TradeMacro it took 1 second at most for unique items. I am in PoE focus as well.


Entry points to methods from Mockito API via interfaces

It could be useful (especially for Eclipse users) to be able to implement an interface and have direct access to methods from Mockito API (without a need to add static imports).
The idea taken from David Gageot - http://blog.javabien.net/2014/04/23/what-if-assertj-used-java-8/

Available in 2.0.0-beta and 1.0.0-beta.


I got 404 when use www

What kind of issue it could be?
I specified both www.domain.com and domain.com on .env file.
But still getting 404 for www
Registrator name config is correct.

If you get a 404 the problem lies with nginx-proxy/docker-gen. The letsencrypt-companion only functions are to create / renew certificates, put them and the private key on the correct place with the correct name, and reload the nginx proxy on cert creation or renewal.


create task catalogue and task to run

task in catalogue is basically read only, task to run overrides the task in the catalogue and links to it

superseeded


Move @types/jasmine into devDependencies

Types should always be specified as devDependencies. In this particular case, the inclusion of @types/jasmine in the compiled package leads to conflicts in systems that make use of Jest

@sjaakiejj Please rebase on main to get the CI tests to run. Thanks!

@sjaakiejj Please rebase on main to get the CI tests to run. Thanks!

The workflow file still didn't trigger test runs since the changes only applied to the main branch. I've updated the github workflow file now to also trigger on PRs.


Update dependencies to the defined ones.

Backport #320 to 1.3
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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.

Codecov Report

Merging #343 (b47d1c6) into 1.3 (d9a45fc) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head b47d1c6 differs from pull request most recent head 1eb21f7. Consider uploading reports for the commit 1eb21f7 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##                1.3     #343   +/-   ##
=========================================
  Coverage     71.61%   71.61%
- Complexity     2941     2950    +9
=========================================
  Files           376      376
  Lines         18839    18839
  Branches       1466     1466
=========================================
  Hits          13491    13491
- Misses         4764     4765    +1
+ Partials        584      583    -1

see 3 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Upgrade to http4s version 0.15.9a

Upgrade jawn to version 0.10.4
Decouple quasar from sbt-slamdata's version of http4s
Remove code that worked around a jawn bug

I just got so fed up with staring at that bugFreeArgonautFacade that I decided to see if the fixed version of jawn was released which it was but it required an upgrade to http4s as well. I then remembered #1852 which is likely not fixed but perhaps an open PR will catch our attention to get around to fixing that bug so that we don't get too far behind on http4s (and our dependencies in general). A quick glance at the http4s changelog tells me that substantial amount of bugs were fixed since 0.14.1 so if it doesn't fix #1852 then at least it fixes a bunch of others.
@drostron Since you were able to reproduce the issue for #1852, can you see if you can reproduce it on this branch and perhaps post the reproduction steps if you can.
Also, I didn't test any connectors, so I wouldn't be all that surprised if there is some kind of binary incompatibility there.

Revision label assumes we are still versioning according to the web api

@drostron Oh, awesome! That's good news.


"Parent Logs" section for notes that track conversations with parents

Classroom teacher suggestion: Add a "parent logs" section for notes that track conversations with parents. This would be parallel to the "Notes" and "Services" section, but specifically focused on parent interactions.

This feature is already there, right?  Good learning from that user that they didn't learn or discover it!

@kevinrobinson Omgggggggggg you're right!


Error

Hi, can you help me with this error please? Thank you
~/apt2# ./apt2.py -v -v -b -s 1 -f demo.xml
Traceback (most recent call last):
File "./apt2.py", line 14, in
framework = Framework()
File "/root/apt2/core/framework.py", line 40, in init
self.config['lhost'] = Utils.getIP()
File "/root/apt2/core/utils.py", line 148, in getIP
ip = socket.gethostbyname(socket.gethostname())
socket.gaierror: [Errno -5] No address associated with hostname

What can you tell us about your environment?  Linux? Which distribution? How many ips are defined? Does the system have a defined hostname?

Just pushed out a new code change that should properly handle your situation.  I would suggest that you make use of the "--ip" command line flag to specify your desired IP address.
Please let us know if the latest code updates do not properly handle your issue.


Create Task4.txt

Issue: #5

Hey @kalim-Asim
Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.
Thanks for contributing in OpenCode'24 ✨✨!

Hey @kalim-Asim
Your PR has been merged 🥳🥳 and you have earned 10 points.
Thanks for contributing in OpenCode'24 ✨✨


Added link to babel-macros article

What: Added link to babel-macros article

Why: -

How: Added using markdown links

Thanks so much for the pull request! I got two others like it and merged one of the others. You'll still get credit for Hacktoberfest though! If you'd like to get something merged, the glamorous README could probably have a link to the v4 announcement blog post. You could probably put it at the top where it says "Read the intro blogpost" and change that to "Read the intro blogpost and the v4 announcement blog post"
Thanks again!


Remove hotwire turbo

Turbo does not support IE 11/ES5 and has no plans to add support.
This leaves 3 options:

Transpile Turbo with Babel and add required polyfills such that it works in IE11 and modern browsers (same approach as with Stimulus). So far unsuccessful.
Conditionally load Turbo based on browser feature detection with modernizr (https://github.com/communitiesuk/submit-social-housing-lettings-and-sales-data/pull/430)
Remove Turbo altogether

Without Turbo each form submission or link click requires a full page load. This does have a perceptible impact on UX/speed but provides the most compatibility with older browsers.
Given the time spent on this so far it seems worth merging this for now and reverting later if we get it working at some point in the future.

Thanks for writing this up! I was actually looking for examples of "gov rails projects using turbo" to see how viable it is, and whether it has any impact on browser support or accessibility (thinking SPA style page navigation could interfere with some screen readers).


Update NfcManager.java

Added broadcast event to send android.nfc.Tag object to native libraries

@jineshfrancs sure, please use v3.11.0