Add unloadability howto document

Summary
This document describes how to use unloadability in .NET Core 3.0 and
how to debug issues with unloading.
It also contains a link to a complete sample in dotnet/samples repo.

@AaronRobinsonMSFT I've updated the doc based on your feedback. The only place where I've left the type names without backticks is in the section titles, as it looks awkward in those.

Oh, I've forgotten about the TODO about explaining gc root. I am going to add that. I have not found it in any doc.

I've actually reworded the text that was mentioning the roots before and got rid of the "roots" name there as it doesn't seem to be necessary for the explanation.

@rpetrusha thank you so much for the detailed review and all the suggested corrections of the grammar. I can see I should get better especially in the comma usage.
I've applied all of your suggestions except for changing one of the titles and also changed "unloadable AssemblyLoadContext" at few places to "collectible AssemblyLoadContext".
I have also created a PR containing all the code snippets from this document here: https://github.com/dotnet/samples/pull/617. I will update this doc once that PR goes in to reference the snippets instead of having them inline.

LGTM.

There's a broken link in the build report and the article needs to be added to the TOC file as well (https://github.com/dotnet/docs/blob/master/docs/toc.md)

The broken link in the build report is to a .NET Core 3.0 API, @mairaw. I thought that it would be better to leave the xref than fence it and have to change it later.

Sounds good @rpetrusha about the link to the API!

@mairaw thank you so much for the review and the info on the docs authoring pack. I didn't know such a thing exists.

@rpetrusha, regarding the broken link - I wonder if the link is really correct, as it contains %2A in the middle. Or does it have a special meaning?
<xref:System.Runtime.Loader.AssemblyLoadContext.Unload%2A?displayProperty=nameWithType>

@janvorli, ordinarily, links have to be fully qualified with parameter types. %2A indicates that the link should be to the overload page rather than to a specific overload.

Closing and reopening to begin new build after dotnet/samples#617 was merged.

@rpetrusha, @mairaw  I've updated the doc so that the last two code blocks are not inline anymore and point to the samples repo. As for the rest of those, I am actually wondering whether to leave them all inline or whether to update the doc to replace the ones that are compilable with a reference to the samples and only keep the snippets that cannot be compiled on their own inline.
Do you have a preference w.r.t. that?

@rpetrusha, @mairaw can you please reply to my previous comment? I would like to merge the doc in soon.

I apologize for the long delay in responding, @janvorli. The best option is to place all compilable samples in the samples repo, with small snippets inline.  In addition, though, some of the inline snippets can be referenced from the samples repo. For example, the currently inline code
for (int i = 0; testAlcWeakRef.IsAlive && (i < 10); i++)
{
    GC.Collect();
    GC.WaitForPendingFinalizers();
}

is included in your main testing program. So we'd want to pull that snippet into the build, instead of having two versions existing independently. The way to do that is to add a sniippet tag with a label (the snippet tags are removed from the documentation when it's built):
// <Snippet1>
for (int i = 0; testAlcWeakRef.IsAlive && (i < 10); i++)
{
    GC.Collect();
    GC.WaitForPendingFinalizers();
}
// </Snippet1>

Then add the reference to the snippet to the topic by including the snippet label without the string "snippet":
[!code-csharp[Main testing program](~/samples/snippets/core/tutorials/unloading/unloadability_issues_example_main.cs#1)]
But it's OK to leave most the remaining one- or two-line snippets in the "Use a custom collectible AssemblyLoadContext" section inline, since they don't exactly correspond to code that would go in the repo.

Also closing and reopening this to see if the broken link would go away - it should since I loaded the 3.0 APIs.

Thanks, @mairaw; I'd meant to close and reopen, but I forgot. The broken link should definitely go away.
There are a couple of additional things to do, @janvorli:

Maira and I thought that the best location for this topic would be a how-to directory (dotnet/docs/core/how-to) rather than the tutorials directory. Could you create the directory and move the topic to it? (That also means that you should change the path in the samples-and-tutorials/index.md document in your PR.)
That also requires the creation of the an index.md page whose title is ".NET Core how-to articles"  for the directory, which at this point would just have a single entry (your topic).
It also requires that both the top-level how-to-page and your article be added to the TOC (found in dotnet/docs/docs/toc.md).
Finally, it would be good to add a link to your article to the managed reference documentation. Once your PR is merged, I can take care of doing that.

@rpetrusha, @mairaw  thank you for your feedback. Could you please give me an advise on the description  / title in the new dotnet/docs/core/how-to/index.md? I have hard time coming up with wording that would differentiate it from the tutorials and make it general enough for possible future articles in this area.

And I have additional questions:
Can snippets be embedded in other snippets? For example, can I do this?
class Test
{
    // <Snippet2>
    [MethodImpl(MethodImplOptions.NoInlining)]
    static int ExecuteAndUnload(string assemblyPath, out WeakReference alcWeakRef)
    {
    // <Snippet3>
        var alc = new TestAssemblyLoadContext();
        Assembly a = alc.LoadFromAssemblyPath(assemblyPath);
    // </Snippet3>

        alcWeakRef = new WeakReference(alc, trackResurrection: true);

    // <Snippet4>
        var args = new object[1] { new string[] {"Hello"}};
        int result = (int)a.EntryPoint.Invoke(null, args);
    // </Snippet4>

    // </Snippet5>
        alc.Unload();
    // </Snippet5>

        return result;
    }
    // </Snippet2>
}

And when you say the sample should be compilable, what does it really mean? Should it contain all the "using xxxx" and have methods placed inside of classes so that it can be pasted into a .cs file and compiled without any additional touches?

Yes, @janvorli,  snippets can be embedded in other snippets.
"Compilable" means that the source code in dotnet/samples should be able to be compiled from the command line (for .NET Framework source) or with the dotnet CLI. We've also usually been including a *.csproj or *.vbproj file for source code that targets .NET Core. You can choose whether to present a all of the source code file in the docs (in that case, it should be compilable, with using statements, etc., if it is copied and pasted) or just a code fragment.

I also suspect that I gave you some bad guidance, @janvorli. In thinking about it, rather than creating a separate .NET Core how to node in the TOC, it's better to locate this topic in the Assemblies in .NET section of the .NET Guide. The section is sparse (it only has a top-level overview topic and a child topic on the assembly file format) largely because a while ago assembly-related topics were treated as language topics and duplicated in the C# and Visual Basic documentation. We're planning to consolidate the documentation by moving it to the .NET Guide.
So the file can be moved to ~/docs/docs/standard/assembly/. Then the index.md file in that directory can be updated and the article can be added to the TOC at ~/docs/docs/toc.md.
I'm also wondering if we should rename it to "How to: Load, unload, and debug assembly loading in .NET Core".

@rpetrusha I've updated the snippets in the samples repo so that they use the  tags and  thus reduced their number. I've also moved them to a new location that corresponds to the new suggested location of the doc. Can you please take a look so that I can update my doc accordingly?

I've merged dotnet/samples#677, @janvorli.

I've somehow screwed the commits. Will fix it soon.

@karelz, @JRAlexander, @BillWagner, @cartermp, @Lxiamail - please ignore the request to review. I have not sent them, they somehow got added automatically based on a bad merge commit that I've fixed.

@rpetrusha I've tried to reference the snippets, but it works only for the snippet #1, for the rest it complains that the links are invalid. I have no idea what could be causing that. Do you have any idea what can be wrong?

Also, feel free to just push new commits, since we squash all of them when we merge. We like to have each PR as a single commit in our history.

Closing and reopeing so that it picks a typo fix in a snippet tag in the samples repo

This looks good, @janvorli. I'll merge your PR now.


Correct case of class names

Class names in the instructions differ by case (CosmosDB != CosmosDb) from the pre-written code that refers to them.

@gavincampbell : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Please support Windows

I don't have much in the way of experience writing windows batch files, especially for conda..
Hopefully this would get addressed at some point, however, I know there are some existing build scripts for the windows binaries around

I noticed that someone has built a bunch of the x-org libs using some MSYS2 conda-forge tools. Maybe the IPopt stack can copy them. For example: https://github.com/conda-forge/xorg-libxi-feedstock.

I'm concerned we may be dead in the water on this, so I wanted to post what I've learned so far in the hopes that someone else may see a path forward. MUMPS, the only permissively licensed linear solver for IPOPT, seems to require the Intel Fortran compiler on Windows. This is a problem since the Intel Fortran compiler is proprietary.
I think this means we have to go the semi-supported MinGW/MSYS route, but will this break binary compatibility with our Python, which we must assume was compiled with MSVC on Windows?

Other problems: MUMPS dependencies scotch (in-progress Windows support) and METIS (complete Windows support) both use MSVC for their builds.

@richardotis There is a nice summary of the MinGW/Python situation here:
https://mingwpy.github.io/motivation.html
https://mingwpy.github.io/issues.html
I have built and used MinGW-compiled Python extensions together with vanilla Python for Windows for quite some time now, and have not run in practical issues despite the warnings mentioned in the links above (MinGW is still the only practical choice on windows for C++ code relying on robust C++11 support, which MSVC still does not have).
The biggest potential issue is that when you mix MSVC-compiled code with MinGW-compiled code, you are bringing two separate C runtimes (or C standard libraries, as they are called on Unix) into the same program. This will create issues if, for instance, you invoke one stdlib's malloc() and later free the memory with the other stdlib's free(). If you are working with C++, there are additional issues related, e.g., to exception handling and ABI - but I don't think they apply to Ipopt, as we would not be mixing C++ code compiled with different compilers.
If I understood correctly, the plan for Ipopt on windows would be to use gfortran to compile mumps and then link to it from an MSVC-compiled Ipopt, assuming the binary interfaces are compatible. Linking to mumps would bring in the MinGW C runtime, and we would then need to test that the two runtimes are properly insulated from each other.
I am not sure what the status is regarding MinGW support in conda-forge, nor I have a clear idea what kind of hackery would be needed in the Ipopt build system though...

Relevant issues: https://github.com/Microsoft/vcpkg/issues/233 https://github.com/Microsoft/vcpkg/issues/485#issuecomment-270219347
Looks like it may be possible to use MSVC and gfortran together via CMake: https://cmake.org/cmake/help/v3.7/prop_tgt/GNUtoMS.html http://www.netlib.org/lapack/lawnspdf/lawn270.pdf
Other possibility, Fortran 90 to C conversion (no idea if it will work better than f2c): http://cci.lbl.gov/fable/

I'm also looking to find a way to build Ipopt on Windows using Visual Studio with MUMPS compiled with gfortran.

Not sure how the rest of the conda forge community would feel about this, but shipping pre-built ipopt binaries on Windows looks pretty tempting: https://anaconda.org/cachemeorg/ipopt_bin/files
What does everyone think?

I think it is fine if conda forge allows you to upload binaries that aren't built with conda forge.
Also, i noticed that SciPy 1.0 will be able to be built with openblas, MSVC, and gfortran in the versions release notes. Seems like whatever solution they have going might be something useful for getting the core dependencies going on on conda forge.

PR implementing this idea: #11

I've made significant progress getting ipopt to build on Windows using m2w64-toolchain and several other pieces of MSYS2. This requires MUMPS (see https://github.com/conda-forge/mumps-feedstock/pull/17#issuecomment-345535745 ) but not SCOTCH. No conda build yet, but the build is tricky, so I wanted to document what I've done so far here. Starting from a conda environment with m2w64-toolchain, m2w64-openblas and posix packages from msys2 channel installed, in the Ipopt-3.12.8 source directory:
conda install --use-local mumps # must build mumps according to above comment
rm /mingw-w64/bin/make.exe # should not do anything unless you are repeating these steps
bash -c "CXXDEFS=-DCOIN_USE_MUMPS_MPI_H ADD_CXXFLAGS='-I/include -L/lib' ./configure --build=x86_64-w64-mingw32 --disable-linear-solver-loader --with-blas-lib='-lopenblas' --with-mumps-lib='-ldmumps -lmumps_common -lpord -lgfortran' --enable-shared --enable-dependency-linking" # be prepared to wait 5 minutes or so
cp /mingw-w64/bin/mingw32-make.exe /mingw-w64/bin/make.exe
mingw32-make clean
mingw32-make test # be prepared to wait 30 minutes or more; verify tests pass
rm /mingw-w64/bin/make.exe

I'm still having trouble getting a statically-linked ipopt.dll to build.

Current trouble with DLL building:
*** Warning: linker path does not have real file for library -ldmumps.
*** I have the capability to make that library automatically link in when
*** you link to this library.  But I can only do this if you have a
*** shared version of the library, which you do not appear to have
*** because I did check the linker path looking for a file starting
*** with libdmumps and none of the candidates passed a file format test
*** using a file magic. Last file checked: /bin/libdmumps.dll.a

*** Warning: linker path does not have real file for library -lopenblas.
*** I have the capability to make that library automatically link in when
*** you link to this library.  But I can only do this if you have a
*** shared version of the library, which you do not appear to have
*** because I did check the linker path looking for a file starting
*** with libopenblas and none of the candidates passed a file format test
*** using a file magic. Last file checked: /mingw-w64/bin/libopenblas.dll
*** The inter-library dependencies that have been dropped here will be
*** automatically added whenever a program is linked with this library
*** or is declared to -dlopen it.

*** Since this library must not contain undefined symbols,
*** because either the platform does not support them or
*** it was explicitly requested with -no-undefined,
*** libtool will only create a static version of it.

It may be worth asking about these issues on the IPOPT mailing list.

Having some luck with this make rule added to Ipopt-3.12.8\src\Interfaces\Makefile ( https://list.coin-or.org/pipermail/ipopt/2013-August/003477.html ):
libipopt.dll: $(libipopt_la_OBJECTS) $(libipopt_la_DEPENDENCIES)
	$(CXXLINK) $(AM_CXXFLAGS) $(CXXFLAGS) $(AM_LDFLAGS) $(LDFLAGS) -static -shared -o $@ \
-Wl,-rpath,$(libdir) $(libipopt_la_LDFLAGS) $(libipopt_la_OBJECTS) -lipopt -ldmumps -lmumps_common -lopenblas -lgfortran

I can compile a DLL and link it to a test problem, but now I'm working through some runtime missing DLL errors.

A make clean seems to have solved it. I copied the libipopt.dll to the Ipopt-3.12.8\test folder, ran mingw32-make clean from the main folder to remove any static build artifacts, then ran gcc -o hs071_c.exe hs071_c.c -I../src/Interfaces/ -L. -lipopt and then hs071_c.exe. The resulting test problem runs and converges to the solution.
The next steps are to pull all of these steps into a reproducible build script.

I can't quite figure out how to get patching to work properly in conda build, and the paths are not entirely correct yet, but here are my WIP recipes for ipopt and mumps:
https://github.com/richardotis/ipopt-feedstock/tree/windows_support
https://github.com/richardotis/mumps-feedstock/tree/windows_support
In principle, all you need to do is add msys2 channel and run conda build . inside the feedstock directory. (You have to build mumps and then install it with --use-local, before ipopt.) The reality right now is the patching machinery doesn't seem to work quite right on my Windows machine (CRLF issue?), and so I'll need some help with (a) getting the CMakeLists.txt in the recipe directory into the mumps source root at build time, and (b) getting the ipopt dll_makefile.patch to apply cleanly.

https://github.com/richardotis/ipopt-feedstock/tree/ab2834339620425b9014c7fc9066af4a365d9055
https://github.com/richardotis/mumps-feedstock/tree/3c1a8e6bc78d0b33fae65cf4d2fe07ed5bb3af87
These are ready for a look. It should be as easy as adding msys2 to your list of channels and then conda build . in the mumps directory, followed by conda build .. I think you don't need to conda install mumps at all as long as the built package is in your build cache.

You really shouldn't disable the linear solver loader, and it can build fine with mingw. That's a pretty important feature given the licensing of the various linear solvers ipopt can use.

Hi everyone. Trying to link an application with ipopt on windows I noticed that the windows libraries are a bit old (VS2010).
There seem to be more recent ones here: https://github.com/coin-or/Ipopt/releases/tag/releases%2F3.13.2
These are built with VS2019 and they appear to work fine (they are dynamic libs however unlike the current static ones)
Would it be possible to update the windows package here with these somehow ?


Add alerting on go/process metrics

Once the metrics are tracked in grafana there will need to be a larger discussion around what to alert on and why, but the only blocker to that work is #8602

8632


MocoAverageSpeedGoal unrecognized when reading MocoStudy from xml

The MocoAverageSpeedGoal seems to be unrecognized when reading a saved MocoStudy xml file.
I am using MATLAB 2020b with OpenSim Moco 0.4.0 on windows 10. I used the example2DWalking with additional lines at the end:
study.print("study.xml");
MocoStudy("study.xml");
Error message:
Encountered unrecognized Object typename MocoAverageSpeedGoal while reading property goals. There is no registered Object of this type; ignoring.
Feels like this is a bug. Just to want to bring this to your attention.

Thanks @SolaWeng for reporting this issue.
Moco has now been merged in OpenSim, and this bug has been corrected, see here.
If this bug is blocking you for making further progress, you can remove the MocoAverageSpeedGoal  from the .xml file, create your study MocoStudy("study.xml");, and add the goal back programmatically.

Hi Antoine,
Thanks for the update! For now I can just redefine the MocoAverageSpeedGoal in MATLAB. It's not a big issue.
Cheers,
Sola


Add syntax highlighting for flow comment syntax

NB: I had some poking around at scoping in vs-code theme options, and it looks like the internals of comments don't have type information:

which implies that targeting parts of a comment isn't possible. If that's true, this should just be closed. Otherwise if there is a scope that can work to target these, happy to look into implementing this.
Describe the solution you'd like
In flow, you can use a comment syntax to add types:
/*::
type a = number;
*/

or:
const b/*: number */ = 5

It would be nice if the comments that contained type information had syntax highlighting instead of being in comment-grey as they include information. An example of this kind of highlighting:

You're correct- I can't target this directly. If you can either

point out a theme that targets this correctly or
file an issue with VS code so that I can target it
I will reopen this issue and work on it :)

Thanks for visiting, sorry about the lack of support here.

Thanks for the swift reply!
It looks like since the types for js are derived from ts, this means this syntax would need to be added into typescript itself, which has its own ongoing conversation around comment types.
Sorry for raising this non-useful request. Thanks for the awesome theme. :)


workload fails on node reboot

was on node 219, cockpit rc2 image with the switch to network manager preformed whitch is why suspect this happened. its still up but this appears to be unrecoverable. really just reporting for documentation sake, this is definitely a issue with my image not necessarily zos.

can you please let us know which env this is running on?

can you please let us know which env this is running on?

The deployment was on 22.1 kinetic kudu, with notable changes of an nginx install and migration from networkd-> network manager.

May I ask why do you need to switch to NetworkManager on this workload ? ZOS passes static network config to the node over cloud-init hence switching to NetworkManager is not supported from our side. There is nothing we can do for this. I suggest we close the issue.
If you have the same issue with a standard VM image you can reopen

May I ask why do you need to switch to NetworkManager on this workload ? ZOS passes static network config to the node over cloud-init hence switching to NetworkManager is not supported from our side. There is nothing we can do for this. I suggest we close the issue.
If you have the same issue with a standard VM image you can reopen

It's nessecary to make the cockpit image work, that means the entire cockpit project has been pointless from the start if there's no way for that to be compatible. Would of been nice to shut that too have been mentioned somewhere.

Reopening this, since I couldn't reproduce simply by switching a deployment to use Network Manager. To clarify, all VMs in question were booted successfully with networkd the first time, then the configuration was migrated to Network Manager. Such VMs survived reboots of the VM and the node in my tests, with fully functional networks.
The issue that remains here is why these VMs ended up in a state where the (testnet) playground finds no IP addresses for them and they can no longer be reached.


Jobs progress is reset and messed up. in MySQL job names look like "???????"

*Description of issue:
--- When I updated from Jobs-4.15.5 to Jobs-4.15.11 the jobs of all the players is messed up and very strange displayed. The progress of some jobs has been reset, and some jobs have been swapped or displayed twice. In MySQL I noticed that job names in the jobs_jobs table look
like "????????". Although, as in version 4.15.5, the encoding is utf8mb4_unicode_ci, so it seems to me that new version of the plugin writes data incorrectly.

Server Type (Spigot/Paperspigot/etc): PaperSpigot
Server Version (using /ver): git-Paper-1618 (MC: 1.12.2)

Please make a screenshot about your mysql jobs_jobs table where the ? occurs.

Please make a screenshot about your mysql jobs_jobs table where the ? occurs.

Try to re-create your database, and find the correct unicode code to accept your language. Re-creating I mean, make a backup to the current one, delete from Jobs folder and re-generate by restarting your server. If we use English characters in database, it will works, because there is no any special character in it.

Try to re-create your database, and find the correct unicode code to accept your language. Re-creating I mean, make a backup to the current one, delete from Jobs folder and re-generate by restarting your server. If we use English characters in database, it will works, because there is no any special character in it.

I do not quite understand you, why should I delete a file if the database is in MySQL and why should I recreate it if I can edit the table? I set the encoding "utf8_general_ci", which 100% supports the Russian language, but still the data is displayed as "?????", then the problem is in the plugin.

I'm sorry. I confused MySQL with SQLite... So, try saving the MySQL Jobs database, then deleting it, and then creating a new one by restarting the server. If you have this, check to see if it is still incorrect. There is nothing in the plugin that can change the job names in the database. These will only be saved if the database encoding can accept your language.

I'm sorry. I confused MySQL with SQLite... So, try saving the MySQL Jobs database, then deleting it, and then creating a new one by restarting the server. If you have this, check to see if it is still incorrect. There is nothing in the plugin that can change the job names in the database. These will only be saved if the database encoding can accept your language.

I just set the encoding to "utf8_general_ci", restarted the server, but the data is still displayed as "?????". After that I installed Jobs-4.15.5 (instead of Jobs-4.15.11) and with the same encoding everything works.
So the problem is not in the encoding, but in the plugin.


Support newer kernels in the CentOS builder

Motivation
Our[ CentOS builder] (https://github.com/falcosecurity/driverkit/blob/master/pkg/driverbuilder/builder/centos.go) does not seem to support kernels other than 2.6 and 3.10
Feature
We need to support the newer kernels.
Alternatives
Additional context
https://github.com/falcosecurity/falco/issues/1433

@fntlnz I want to pick this. If it is a good first issue

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

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


🚀 Feature: Allow to add additional titles for same entities to enhance searches

🔖 Feature description
You should add a form of aliases to entities like systems and components because especially systems shift over their life cycle and might be known under different names.
A user of backstage should be able to find the right system regardless of what name under the different known name it was used to search for it.
🎤 Context
In our company we have usually the setup with project name and system name. A system is usually known under both and depending on the usergroup one of them is more prevalent. Because we want to use backstage as single point of entry for development resources, to be able to lookup and find systems etc is a key capability.
✌ Possible Implementation
On a manifest level, one option would be to have a list of aliases in the metadata of all entities. In my point of view those aliases should not be an alternative for the name but for the title.
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, I don't have time to work on this right now

This probably fits best as a custom extension to the model.
You could add an annotation, for example
metadata:
  annotations:
    also-known-as: Older name, Even Older Name

and then make a collator for the search plugin that indexes that.

@freben I tried it like this, took a while until I understood how to extend an existing collator, and it worked like a charm.
But I still have one missing piece: In my opinion the aliases are part of the main system model metadata and should be part of the standard overview/about component.
How can I customize the entity about card?

I think you should either fork the card, or create another card with the information you want, or propose a change to inject generic JSX content in a specific place of AboutCard that should make sense for other use cases as well

Just searched through the discussion about the entity cards. Concluding from those I assume https://github.com/backstage/backstage/issues/16876 would implemented right implement this correctly? So either wait, contribute or customize completely.
Additionally I wanted to style the search result differently or more precisely just the aliases. Is this possible? (I'm a complete react newbie so I just tried to add html tag in the text which obviously didn't work.)


(fix) Remove attribute binding on value

Because of this attribute binding, assigning a date to the component's value attribute will cause Ember to assign the date to the underlying <input>'s value attribute, which is in the best case useless.
Flatpickr will then overwrite this value, except in some cases, where the visible string will become the Date instance cast to a string (i.e. default JavaScript date string formatting).

Does this fix the issue you filed @wmadden?

@rwwagner90 no, this is a different bug.  I'd love some assistance debugging the tests.

Actually, none of the tests seem to pass on master either so I don't think it's anything I changed in this PR. @rwwagner90 it'd be great if you could take a look.
I did some debugging in the tests and I believe the problem is caused by the flatpickrRef used here. This two-way binding means that when the first rendered ember-flatpickr component initializes, it sets the flatpickrRef attribute on the test view. That value contains the flatpickr instance which is torn down when willDestroyElement() is called. After that point, any calls to the flatpickr instance (e.g. setDate()) will raise an exception.
Unfortunately, because the flatpickrRef value is stored on the test instance, it is then assigned to each subsequently-rendered ember-flatpickr component - overriding the new flatpickr instances they create during initialization.
A couple of obvious solutions to this problem spring to mind. The simplest is to have one expectation per test - i.e. split the test function here so that there's only one render block per test.
However a more elegant solution would be to not use the flatpickrRef at all. Moreover, you're testing the functionality of the flatpickr library ("value accepts string, dateObject or array of string/dateObjects"). If I were you I'd delete the test entirely since it doesn't test what your library is actually testing and does test what flatpickr is doing.

@wmadden I don't think that the flatpickrRef binding has anything to do with it, as these tests historically worked. It's something about changes in the newer versions of flatpickr that caused issues. I'm going to work on debugging it.
I'm not sure I agree with changing from a TextField to a Component. What is the reasoning behind that?

The reasoning behind using Component instead of TextField is that TextField has an attributeBindings array of its own which already includes 'value'.

Thanks for all the work here! I ultimately had to tweak a few things, so I pulled in some of these changes and not others.


Can i use Number for mongoose ref?

When i have my mongoose _id's as Numbers (1, 2, 3, 4, 5) and trying to make {
type: Number,
ref: 'Model
}
Admin bro cant make these relations, it does not make select box. But when i change _id to ObjectId's it makes relationships and select box is vizible.
Is there any way to create relations in Admin Bro if my _id is Number ?

Using Number for _id is a valid method

Closing because the issue is old. If you still need help, please reopen (preferrably in adminjs-mongoose repository)


What is the pre-training objective to the model bart-base

❓ Questions and Help
Before asking:

search the issues.
search the docs.

What is your question?
What is the pre-training objective to the model: bart-base
I cannot find the pre-training objective either in the README file or the paper.
Code
N/A
What have you tried?
N/A
What's your environment?
N/A

From section 2.2 of the paper:

BART is trained by corrupting documents and then optimizing a reconstruction loss—the cross-entropy between the decoder’s output and the original document.
Unlike existing denoising autoencoders, which are tailored to specific noising schemes, BART allows us to
apply any type of document corruption. In the extreme
case, where all information about the source is lost,
BART is equivalent to a language model.

In fairseq, it uses the denoising task.

Thanks for your reply.
Let me clarify my question again.
In section 5.1,

Based on the results in Section 4, we use a combination of text infilling and sentence permutation.

Therefore, I can see that the pretrained bart-large used the objective above to train yet it does not state clearly in the README file.
I have checked the denosing task from fairseq/tasks/denoising.py. Is the bart-base trained with this file and using the default arguments?

If you download the checkpoint, you can load it and inspect the arguments that were used to train it.  For example:
import torch
chkpnt = torch.load('model.pt')
args = chkpnt['args']

Here is a gist containing a JSON dump of the args used to train bart.base

Thx, I will study the files you provided.


I cant integrate please help

I have exactly the same issue
https://github.com/SocalNick/ScnSocialAuth/issues/11
I have also checked the solution given by you multiple times in my project . Everything is exactly the same.
I have one doubt do i need to install the hybridauth seprately or its included in this mod.
I have seen the requirements where hybridauth is written, but i am confused as no proper help is provided for hybridauth. This mod is aewsome, but just cant get it working.
I am working with zf2 + Doctrine2.  Please help me out at this step.
I have installed
'Application',
'ScnSocialAuth',
'ZfcBase',
'ZfcUser',
'DoctrineModule',
'DoctrineORMModule',
'Album',
'ZfcUserDoctrineORM',
Thanks
Sandeep

How can I login with google . I mean to what is the url for social login. I have type this url /public/user/add-provider/google but it give 404 error. Please help me.


[infra-proxy-service] API to get cookbook affected nodes

:nut_and_bolt: Description: What code changed, and why?
In order to fetch nodes that are affected by the cookbook. Hit the chef server partial search API with required params.
params: name, chef_guid, url, check_in, uptime, platform, environment, policy_group

:chains: Related Resources
MSYS-1241
:+1: Definition of Done
Able to fetch the nodes with desired params.
:athletic_shoe: How to Build and Test the Change

rebuild components/infra-proxy-service
rebuild components/automate-gateway

chef-automate dev grpcurl infra-proxy-service chef.automate.domain.infra_proxy.service.InfraProxy.GetCookbookAffectedNodes -- -d '{"server_id": "SERVER_ID", "org_id": "ORG_ID", "name": "COOKBOOK_NAME", "version":"COOKBOOK_VERSIOB"}'

:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable

gRPC API:
chef-automate dev grpcurl infra-proxy-service chef.automate.domain.infra_proxy.service.InfraProxy.GetCookbookAffectedNodes -- -d '{"org_id": "8f1646d3-35ae-4562-84d7-ff013f3973f2", "server_id": "c7d36e02-4816-4bfa-a1b9-d239f56c9b8d", "name": "starter", "version": "1.0.13"}'

Response:
{
  "nodes": [
    {
      "name": "node1.test",
      "chefGuid": "d9af01f5-342b-4329-8986-2647b6383a63",
      "uptime": "10 minutes 55 seconds",
      "platform": "ubuntu",
      "environment": "_default"
    }
  ]
}

Is the intent to get all nodes within an ecosystem that could be affected by a cookbook? Theoretically there could be nodes not in chef-server or nodes not in automate that are affected by a cookbook. I'm not sure where this is intended to be displayed or what the context is that this API supports. If we want to show nodes in automate that ran with a cookbook version we can query automate directly without going to chef-server.

@kmacgugan here the intent is to get all nodes which stored on chef-server and affected by the cookbook. I am trying to wire these nodes with Automate's stored nodes possibly with chef_guid. Ref: http://bit.ly/2vpIz5N


add void return value for some callback signatures

Short description of what this resolves:
Build breaks if noImplicitAny is declared in tsconfig.json.
Changes proposed in this pull request:

add void return value to call signature of a few callbacks

Ionic Version: 2.0rc5

Thank you! Sorry about that. I released a nightly with this in it<EMAIL_ADDRESS>


fix(oauth): github and discord login error

fixed #322, fixed #302
Error in Microsoft (#322) still need to be located.

@stonith404 Could you please build a dev image and let them test it?

Great thank you! The dev image is pushed.

Great, I'll just wait until we get feedback from the users if it works. Then I'll merge it :)

@stonith404 Can you update the dev image to make changes in 4af97e4 take effect? It's important for locating the error in Microsoft authentication.

close #325

@zz5840  👍 Pushed

@stonith404 Maybe they are rest today,  I think we should merge this pr and release a new version to fix discord and github first. I'll open a new pr to fix error in Microsoft.

@zz5840 Makes sense. Thank you for the PR!


fix: add blobHash to challenge and piHash

Purpose or design rationale of this PR
Describe your change. Make sure to answer these three questions: What does this PR do? Why does it do it? How does it do it?
TBA
PR title
Your PR title must follow conventional commits (as we are doing squash merge for each PR), so it must start with one of the following types:

[X] fix: A bug fix

Deployment tag versioning
Has tag in common/version.go been updated or have you added bump-version label to this PR?

[ ] No, this PR doesn't involve a new deployment, git tag, docker image tag
[X] Yes

Breaking change label
Does this PR have the breaking-change label?

[ ] No, this PR is not a breaking change
[X] Yes

fyi, circuit side changes https://github.com/scroll-tech/scroll/pull/1267


custom BarTooltipItem as per BarChartRodStackItem

As per the current implementation of BarTooltipItem, the function to customize the tooltip is as below
BarTooltipItem touchTooltipData(BarChartGroupData group, int groupIndex, BarChartRodData rod, int rodIndex)
The callback is missing the index for stack position, because of this the tooltip cannot be customized as per stack position.
Versions

Flutter: 1.17.5
FlChart: 0.10.1

Added touchedStackItem and touchedStackItemIndex properties in the BarTouchedSpot ,
check it out in the 0.11.0 version.
Thanks for your suggestion!

But this still does not help with tooltop, I am specifically talking about callback defined to get tooltip text...
BarTooltipItem touchTooltipData(BarChartGroupData group, int groupIndex, BarChartRodData rod, int rodIndex),
Can you pass BarTouchedSpot to BarTouchData(touchTooltipData)? It would be a breaking change, but would be better than having 6 parameters to a function.


Add possibility to change the default "/api" directory

Describe the feature
There are scenarios when /api route handled by another server/proxy. To make it possible still send request to local Nuxt server without additional /server/routes/front-api directory, we can have defaultApiDir option in Nitro config.
Additional information

[ ] Would you be willing to help implement this feature?

Hi @enkot. PR is welcome to introduce apiDir (filesystem) and apiRoutePrefix (route prefix /api) options 👍🏼 BTW in meantime, you can simply make a symlink using ln -s server/api server/routes/front-api to alias.

Hi @pi0. Great, thanks for the almost instant feedback! Will be happy to add it 👍🏼


Fix invalid calls to logging method

The _warn method expects a String, and not a Format String + arguments

Hey! I am hosting a new master branch for this repo since the old creator has been MIA for 2 years. Hoping to get this repo back on it's feet and in good standing. I've been working on updating tests and adding all new functionality and have merged some of the PR's that I thought were sufficient for merging. If your PR didn't make it, resubmit it please.
Thanks!
https://github.com/Harrison97/spotipy


docs(readme): remove duplicated section

Please read and mark the following check list before creating a pull request:

[O] I read and followed the CONTRIBUTING.md guide.

Short description of what this resolves:

remove section [How can I support the developers?] was duplicated.

Hi! 👋 Sorry for the late reply. I'm going to merge this :)


Changelog wiki

The wiki hasn't been updated since 1.27.x (and even that was a placeholder).  How about updating the wiki regularly?
https://github.com/flutter/flutter/wiki/Changelog

Hi @hanskokx
Thanks for filing the issue, the page has been inactive and it might get deleted.
You can release notes here instead
For example, Flutter 2.2.0 release notes

Closing, thank you


Use dateutil for more flexible parsing of RFC-3339 formatted strings

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

retest this please

Run Portable PreCommit

Run Portable PreCommit

Run Python PreCommit

retest this please

retest this please


Fix case of ruleId

Why:
Closes: #35709

What's being changed (if available, include any code snippets, screenshots, or gifs):
Before

...

After

...

Check off the following:

[ ] A subject matter expert (SME) has reviewed the technical accuracy of the content in this PR. In most cases, the author can be the SME. Open source contributions may require a SME review from GitHub staff.
[ ] The changes in this PR meet the docs fundamentals that are required for all content.
[ ] All CI checks are passing.

Hi

@jsoref Thanks so much for opening a PR with this fix! I'll update the branch and get this merged once tests are passing 🍏


pythonPackages.tensorflow: Don't change the rpath to point to gcc4.9

When using cuda, the rpath was set to include GCC lib version 4.9.
I am not sure what this was attempting to do, but an effect was to
prevent certain python libraries to find the correct (newer) version
of the std lib.
Motivation for this change
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

It was added in https://github.com/NixOS/nixpkgs/pull/19701. No idea why.

@teh, do you recall?

Actually, I think I recall. Tensorflow requires the compiler during runtime.
Some time after @teh proposed adding extra paths to python.buildEnv. I think that was related.
What should be done, is hardcoding the paths to the compiler.
See also https://github.com/NixOS/nixpkgs/pull/18273#issuecomment-288489175

cuda's nvcc is tied to specific gcc versions (at least it was, maybe it's no longer?), that's why we needed to pass through a specific gcc library.

@teh: The CUDA compatibility table seems to allow for many different versions of GCC, but it does not mention version 4.9 (which is in place before the PR). It seems to me that CUDA is in fact compatible with most GCC versions, so I'd say to go for the default GCC of nixpkgs until tensorflow break. (With the added benefit to make tensorflow compatible with the rest of nixpkgs --- before this PR tensorflow is eg. incompatible with matplotlib)
Furthermore, if it is CUDA that requires a specifc GCC version, shouldn't the version-pinning happen in the CUDA package instead of tensorflow?
@FRidh: I do not know how to hardcode the path the compiler. Also, it does not seem that tensorflow requires the GCC at runtime in the current version. LLVM is used, according to https://www.tensorflow.org/performance/xla/

@jyp then I suggest you take into accounts @dezgeg 's comments and then it should be ready to go in.

@dezgeg I've applied the changes as you suggested. In fact I've removed all explicit mentions of gcc. Let me know if the code is now idiomatic or not. :)


isolating abstracts from full papers

The abstract is already included in the main paper, so returning papers Documents only won't lead to loss of abstract information
User could be willing to only look into abstracts, hence it is convenient to isolate abstracts from full papers

You got it @jerryjliu !


UserCacheConfig.java:100: Implement...

The puzzle 1-73e288ed from #1 has to be resolved:
https://github.com/shiruka/shiruka/blob/24d02de4e698c2d724c0a733186145a34d4b3621/src/main/java/net/shiruka/shiruka/config/UserCacheConfig.java#L100-L100
The puzzle was created by Hasan Demirtaş on 01-Feb-21.
Estimate: 10 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 1-73e288ed has disappeared from the source code, that's why I closed this issue.


FinOps hubs documented permissions prerequisites are insufficient for a successful deployment

⚠ Problem
The FinOps hubs permissions prerequisites documented here are not sufficient for a successful deployment. When validating a template deployment made by a user owning only the documented permissions/roles, the Azure portal throws errors such as the ones below:

The client 'user@tenant' with object id 'guid' does not have permission to perform action 'Microsoft.Resources/deploymentScripts/write' at scope 'ARM id'.

or

The client 'user@tenant' with object id 'guid' does not have permission to perform action 'Microsoft.Authorization/roleAssignments/write' at scope 'ARM id'.

🛠 Solution
Update the aforementioned documentation with the following additional roles

Custom role containing the Microsoft.Resources/deploymentScripts/write permission (there is no granular built-in role for this permission). Alternatively, suggesting using the Contributor role, which includes this permission and all resource-specific Contributor roles.
Role Based Access Control Administrator - required to grant permissions to FinOps hubs managed identities.

I can implement the changes myself. I just need to be sure about the docs structure to follow.

Thanks for finding and volunteering! 😊 Look for the /docs/_reporting/hubs/template.md file. The README.md file in that same folder may also have a reference. (I haven't checked.)


[CMake] Switch to building the overlays in Swift 5 mode

Now that that's stabilized, we don't have to keep them in Swift 4 mode any longer. (Arguably we don't need the CMake variable at all, but it may be useful again in the future.)
rdar://problem/49040980

@swift-ci Please test

@swift-ci Please test source compatibility

Build failed
Swift Test Linux Platform
Git Sha - d1bc9ae8a37699de210f66d4a6202e048798b02f

Build failed
Swift Test OS X Platform
Git Sha - d1bc9ae8a37699de210f66d4a6202e048798b02f

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - d1bc9ae8a37699de210f66d4a6202e048798b02f

Build failed
Swift Test Linux Platform
Git Sha - d1bc9ae8a37699de210f66d4a6202e048798b02f

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - d1bc9ae8a37699de210f66d4a6202e048798b02f

@swift-ci Please test Linux


flatpak applications improper dpi

Hyprland Version
 System/Version info
Hyprland, built from branch main at commit 3d89654254a3cc23a57e1bdcd39ad9f743bf98e2  (vector: New operator overloads and small fix in Vector2D. (3891)).
Tag: v0.32.3-37-g3d896542

flags: (if any)
debug

The dpi scaling does not properly work for flatpak applications when the monitor is scaled.
Bug or Regression?
Bug
Description
Flatpak font and ui elements display unsmooth, jagged and blurry.
How to reproduce
my configuration of secondary monitor which the problem occurs on:
monitor=DP-1,2560x1440@144,auto,1.3
Crash reports, logs, images, videos
No response

means they are using xwayland, which physically can't do fractional scaling. See the wiki https://wiki.hyprland.org/Configuring/XWayland/
on a side note, you're on a very outdated version.


Engine and/or entire computer crashes when resizing the game window and rendering/driver/threads/thread_model = Multi-Threaded

Godot version
v4.0.alpha.custom_build [61f045245]
System information
Arch Linux x86_64, Kernel 5.15.61-1-lts, AMD RX 6800 XT, Mesa 22.1.6, Plasma 5.25.4, KWin+X11
Issue description
Similar to https://github.com/godotengine/godot/issues/58606
================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.0.alpha.custom_build (61f045245fbdf260245555bdf5be46f720daee01)
Dumping the backtrace.  For me, resizing the window freezes the application on multi-threaded on X11. On Wayland it does crash, but with no stacktrace, it just prints double free or corruption (fasttop). Editor doesn't crash on X11 or Wayland.

I can confirm this on 4.0.alpha ff04cb827 (Fedora 36, KDE Plasma + KWin, NVIDIA 515.65.01).
Godot Engine v4.0.alpha.custom_build.ff04cb827 - https://godotengine.org
Vulkan API 1.2.0 - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 1080

================================================================
handle_crash: Program crashed with signal 4
Engine version: Godot Engine v4.0.alpha.custom_build (ff04cb8279080e5fae6856d5204abfd686727780)
Dumping the backtrace. Please include this when reporting the bug on: https://github.com/godotengine/godot/issues
[1] /lib64/libc.so.6(+0x3ea70) [0x7fbe35a3ea70] (??:0)
[2] PopupMenu::_get(StringName const&, Variant&) const (/home/hugo/Documents/Git/godotengine/godot/scene/gui/popup_menu.cpp:1833)
[3] [0xc7d5f70] (??:0)
-- END OF BACKTRACE --
================================================================
[1]    21475 IOT instruction (core dumped)  bin/godot.linuxbsd.tools.64.llvm --path ~/Downloads/empty/

Same/similar issue still in Godot v4.0.beta2.mono.official (f8745f2f71c79972df66f17a3da75f6e328bc55d)
Although only Godot freezes/crashes. Both on Wayland and X11 the compositor is fine.
Godot Engine v4.0.beta2.mono.official.f8745f2f7 - https://godotengine.org
[obs-vkcapture] Init Vulkan 1.2.0
Vulkan API 1.2.0 - Using Vulkan Device #0: AMD - AMD Radeon RX 6900 XT (RADV NAVI21)
[obs-vkcapture] Injecting VK_KHR_bind_memory2 extension
[obs-vkcapture] Injecting VK_KHR_get_memory_requirements2 extension
[obs-vkcapture] Injecting VK_KHR_external_memory_fd extension
[obs-vkcapture] Injecting VK_EXT_image_drm_format_modifier extension
libdbus-1.so: cannot open shared object file: No such file or directory
libdbus-1.so: cannot open shared object file: No such file or directory

double free or corruption (fasttop)

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.0.beta2.mono.official (f8745f2f71c79972df66f17a3da75f6e328bc55d)
Dumping the backtrace. Please include this when reporting the bug to the project developer.

I'm having the same issue in a project that isn't using the multi-threaded model; it's instead using Single-Safe. When I get the crash (both with my own project and the MRP from the OP), it completely crashes the desktop environment. Trying to kill the Godot process remotely over SSH results in it becoming a defunct process. The only way to bring the system back up is to do a hard reboot (even running reboot over SSH won't work).
I'm not sure if they are two different issues, as it seems if I switch to Single-Safe with OP's MRP, I can't make that crash, but Single-Safe in my own project results in the exact same crash when resizing the window (i.e. completely halting the DE).
System Info

Godot version: v4.0.3.stable.official [5222a99f5]
OS: Kubuntu 23.04, KDE Plasma 5.27.4, X11
Kernel: 6.2.0-24-generic (64-bit)
GPU: NVIDIA RTX 3070 Ti (using nvidia-driver-535)

@RobTheFiveNine I can also reproduce this crash with Single-Safe. The back trace generated by gdb points to _clean_up_swap_chain as it is in the back trace shared above. The only difference being that it is called by window_set_vsync_mode instead of a prepare_buffers call.
Although I can only reproduce the crash when using Wayland. X11 seems to work fine

So this is interesting, using GodotSteam "macos-g413-s158-gs4421" Godot 4.1.3, I get the resize crash when thread mode is set to Single Safe. If I set it to multi threaded - resizing works and the scenes run (from the editor) a lot nicer too.
MBP Apple M1, 32GB ram, macOS Sonoma 14.1 (23B74)

I think there's been some work on the multi-threaded renderer recently, and I can now only reproduce this crash (on Windows/NVIDIA) when vsync is disabled.
Here's an automatically-resizing MRP which crashes within seconds on a current build (5675c76461e197d3929a1142cfb84ab1a76ac9dd). It doesn't crash on macOS, haven't tested Linux.
mt_renderer_crash.zip


Landing Page complete

#80
Merge this pr with hacktoberfest labels
@0silverback0

Nicely done! 👏👏👏 Thank you for your contribution. maybe we can collaborate again in the future!

sure @0silverback0
Looking forward to collab


Release/azure communication common/1.1.0

Description
Release for azure-communication-common, version - 1.1.0
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - azure-communication-common - tests


AVRO-2473: Fix C# documentation warnings

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-2473
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

All existing tests remain intact. No new functionality was added.

Commits

[x] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

I cherry-picked these over to branch-1.9: 0492a6e9d3dcd2b3a2d58a236b11b2a7e4b83762 & f8c2f6cf593f57d4b5305387bc717267149cc49d


Open otherapp payload link in new tab

Otherapp payload link opened in same window, updated to open in new  tab as per the other links.

Pull requests should only modify en_US pages. The others will be updated through the translation system.


Remove service backup files in service networking folder

There might have been some problem during conflicts resolutions. so removing all remote and local backups

/lgtm

and
/approve

I checked https://deploy-preview-15008--kubernetes-io-master-staging.netlify.com/docs/concepts/services-networking/service/ and the updated made seem fine.


Pebble templates not processed

Expected Behavior
When from field of io.kestra.plugin.mqtt.Publish task contains a pebble template, it should be processed
Actual Behaviour
When from field of io.kestra.plugin.mqtt.Publish task contains a pebble template, it is NOT processed
E.g. from HomeAssistant I can see that this message was published
payload_json:
    value: Hello from {{ execution.id }}
    key: message

Steps To Reproduce
It can be reproduced by running example flow
Environment Information

Kestra Version: 0.7.1
Plugin version: 0.6.0
Operating System (OS / Docker / Kubernetes): Linux (NixOs)
Java Version (If not docker): JDK 17.0.5+8

Example flow
id: misc
namespace: messenger
tasks:
- id: "publish"
  type: "io.kestra.plugin.mqtt.Publish"
  server: tcp://localhost:1883
  clientId: kestra
  topic: misc/comms
  serdeType: JSON
  retain: true
  username: kestra
  password: 12345
  from:
    key: message
    value: Hello from {{ execution.id }}
- id: debug
  type: io.kestra.core.tasks.debugs.Echo
  format: Hello from {{ execution.id }}

Hello @sceptri
A PR has been opened, shouldn't take long before it's merged :
https://github.com/kestra-io/plugin-mqtt/pull/22/files
Sorry for the inconvenience!

Awesome! Thanks @Skraye, that was super quick!

we are always super fast :fast_forward: :smile:
The new image with the fix will be published at the end of this action

Nice!
Would it be possible to create a new tag here (on MQTT plugin repo) as well?

We are not ready to tag the plugin for now, we are preparing a 0.8.0 release right now and this plugins need to wait the release of the core.
We will release it next week for sure


Pack NuGet.Server from .nuspec instead of .csproj to control content subdirectory (net46)

.nupkg directory structure change:
content\Default.aspx                -->   content\net46\Default.aspx
content\favicon.ico                 -->   content\net46\favicon.ico
content\Packages                    -->   content\net46\Packages
content\Web.config                  -->   content\net46\Web.config
content\web.config.install.xdt      -->   content\net46\web.config.install.xdt
content\DataServices\Packages.svc   -->   content\net46\DataServices\Packages.svc
content\DataServices\Routes.cs.pp   -->   content\net46\DataServices\Routes.cs.pp
content\Packages\Readme.txt         -->   content\net46\Packages\Readme.txt
lib\net46\NuGet.Server.dll          -->   lib\net46\NuGet.Server.dll

Needed to pack a .nuspec for NuGet.Server instead of a .csproj so we can set the content files to go to content/net46.
Fixes https://github.com/NuGet/NuGetGallery/issues/3257
/cc @emgarten @xavierdecoster @maartenba

Hi @joelverhagen, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Note that this is merging to master instead of dev, so we can release this without worrying about all of the WCF to WebAPI migrations issues.

After this is merged, I plan on releasing NuGet.Server 2.11.3.

LGTM!


fix go network extension close without reason

Commit Message: fix go network extension close without reason
Additional Description: N/A
Risk Level: low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

same issue like #26856

cc @wbpcode


Add proto-MSC on the design of URL previews.

This is a document from the Synapse repo that has a lot more to do with the design of Matrix than with Synapse. I'd like to replace the Synapse version with more implementation information (as opposed to different ways to design an API). @ara4n suggested moving it to the attic (https://matrix.to/#/!XaqDhxuTIlvldquJaV:matrix.org/$6gTgMkQIVqQz-o79noMj21BaF0q2IRZCfqEmpM0kF1U?via=matrix.org&via=vector.modular.im&via=envs.net).
This was essentially an MSC before the MSC process existed.
Not sure if this needs a changelog?

Also this is Markdown, I don't know if it matters though since I don't think the attic gets rendered anywhere...I can do a quick rST conversation though if we need.


[VT-62] Backspace doesn't work in vit (xterm)

Ben Boeckel on 2012-11-28T22:27:13Z says:
In xterm, backspace just outputs '^?'. In tmux (screen-256color), the same thing happens.

Migrated metadata:
Created: 2012-11-28T22:27:13Z
Modified: 2014-02-09T02:23:41Z

steve rader on 2012-11-29T00:22:52Z says:
Brian - could you please try v0.4 and change line 885 from...

     if ( $ch eq "\ch" ) {

to...

     if ( $ch eq "\b" || $ch eq "\c?" ) {

and report back?  If you can't, no biggie.  I'm about 99% sure that's the workaround and this will be fixed in v0.5 due out on Friday, Nov 30 2012.

steve rader on 2012-11-29T00:23:37Z says:
steve rader wrote:

Brian - could you please try v0.4 and change line 885 from...
[...]
to...
[...]
and report back?  If you can't, no biggie.  I'm about 99% sure that's the workaround and this will be fixed in v0.5 due out on Friday, Nov 30 2012.

Umm, yeah, BEN!
Sorry for getting your name wrong.

Ben Boeckel on 2012-11-29T00:42:21Z says:
Nope. Other keys don't work as well (^W is one I've gotten used to by now). Any reason for not using Term::ReadLine-Gnu (or equivalent)?

Ben Boeckel on 2012-11-29T02:56:25Z says:
Oops, it does work (missed the h -> ? change). At least my ^W typos can be deleted now :) .

steve rader on 2012-11-30T17:57:37Z says:
So backspace and delete both work now.
And I added ^w, which was also on the todo list.
I assume you know about ^u.

steve rader on 2012-11-30T18:05:35Z says:
Btw, you might consider adding...

    stty erase +h

...to our environment, where "+h" is RAW.
In bash and/or vi, you can do that by doing...

     stty erase ^v^h

...because ^v allows you to "enter" raw characters.
That will/should "unify" backspace and delete.
Then you should find there's very VERY few contexts ^h WON'T work.
(Old telnet and RCS commands come to mind as the only examples.)
If it doesn't, then you can use +.


Move to TypeScript 5 and moduleResolution: bundler

This would improve our use of the ecosystem, as not every package can work with node16. Instead we want to use the new bundler option in typescript 5:
https://devblogs.microsoft.com/typescript/announcing-typescript-5-0/#moduleresolution-bundler
This would fix having to cast certain packages to 'any' to use them in typescript.

Wouldn't using bundler require us to actually go through a bundler before publishing the package?

On the other hand, if you’re writing a library that’s meant to be published on npm, using the bundler option can hide compatibility issues that may arise for your users who aren’t using a bundler. So in these cases, using the node16 or nodenext resolution options is likely to be a better path.

As for moving to ts5, +1 from me.

Maybe I'm wrong here. If it indeed doesn't work in node we probably just need to live with the casts.

Sorry, I just realized we were talking about different things. I was referring to consumers of our library, that may need to go through a bundler to consume it, based on the comment I linked. And I just noted that you were talking about the libraries we are consuming.
Can you point me to one of those any casts? This feels like a typing issue, that can be solved with a declaration file, as opposed to having to change module resolution altogether.


Bump account lock handler version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/983337021

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/983337021
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/983417409

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/983417409
Status: success


Timesheet - Allow Old project in timesheet

Currently timesheet allows only last 365 year project in timesheet
remove exception
Initiated by ashok bhurani
Verified by @mitulpanch

Now all projects are coming to list, without any date filter,
This might increase timesheet project get api timing

https://www.berrycast.com/conversations/5c6d6043-fdf3-51bc-a7d6-a265941b04f3


Unable to install insync on Ubuntu 22.04

Reporting a bug 🐛
Expected behavior
I have a fresh Ubuntu 22.04 machine, fresh deb-get install.
Insync should install successfully.
Other packages such as sublime-merge or zotero install successfully.
Actual behavior
user@host:$ deb-get install insync
gpg: keybox '/usr/share/keyrings/insync-archive-keyring.gpg' created
gpg: failed to create temporary file '/root/.gnupg/.#lk0x0000563ae03e5150.host.49322': No such file or directory
gpg: connecting dirmngr at '/root/.gnupg/S.dirmngr' failed: No such file or directory
gpg: keyserver receive failed: No dirmngr
  [+] Updating: /etc/apt/sources.list.d/insync.list
Get:1 http://apt.insync.io/ubuntu jammy InRelease [5,535 B]
Err:1 http://apt.insync.io/ubuntu jammy InRelease
  The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A684470CACCAF35C
Reading package lists... Done
W: GPG error: http://apt.insync.io/ubuntu jammy InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A684470CACCAF35C
E: The repository 'http://apt.insync.io/ubuntu jammy InRelease' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Steps to reproduce
Run deb-get install insync
System information
/etc/os-release
PRETTY_NAME="Ubuntu 22.04.3 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.3 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

/etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.3 LTS

deb-get version
0.3.9

deb-get update
  [+] Updating /etc/deb-get/01-main
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  110k    0  110k    0     0   141k      0 --:--:-- --:--:-- --:--:--  141k
  [+] Updating cache of supported apps in the background
Hit:1 http://de.archive.ubuntu.com/ubuntu jammy InRelease
Hit:2 http://de.archive.ubuntu.com/ubuntu jammy-updates InRelease
[...]
Reading package lists...
W: https://packages.microsoft.com/ubuntu/22.04/prod/dists/jammy/InRelease: Key is stored in legacy trusted.gpg keyring (/etc/apt/trusted.gpg), see the DEPRECATION section in apt-key(8) for details.

The root problem was that you didn't have the '/root/.gnupg' directory.
The fix would have been to sudo gpg -k > /dev/null
You should sudo chmod 700  /root/.gnupg


Fixed Issues some Issues with Personal Website

I'll look over these changes tomorrow and merge them.


[Bug Report] - Civic Profile Submission

Describe the bug:
Currently, been trying to get information added to the Civic Profile. However, I'm currently getting errors with regards to that functionality (see clip below).
Expected behavior:
Was expecting to let users update housing information from the submission forms and update the relevant TTL file in the back.
Screenshots:
https://github.com/codeforpdx/PASS/assets/14917816/7e3338fa-0a76-44e8-b90a-8229bdf724c7
To Reproduce:
Steps to reproduce the behavior:

Go to "Civic Profile"
Insert information for form
Click on "Submit" button
See error

Possible Fix:
Checking existing functionality for adding information onto Solid and updating the function if needed

Seems what's happening is that the storedDataset state is occasionally null when the addMutation is run. In useRdfCollection ln 74, mutate attempts to access the storedDataset and it doesn't exist throwing the error.
Easiest way I found to work around is to add a useEffect to HousingInfo, that forces a refetch if storedDataset is null.


Serialize Money with Entity Framework Core 2.0

Hi,
Since Noda's Money is a struct type, we cannot use it as field type for our entities in our .Net Core 2.0 project, using Entity Framework Core 2.0.
Since EF Core 2.0 now fully supports owned entities, it would be great to be able to do that!
Could you advise us a way to do that?
Thanks

Yes, I'm just thinking as well how to integrate it with EF Core, I will somehow need to persist all the currencies in the database to refer to them in the other entities. I started to create all those money entities myself but I don't think it's reasonable when a great library like NodaTime exists.

Owned entities have mostly the same traits as regular entities, so they are required to be reference types (so the change tracker can track them). A value convertor is not an option as well, because it can't convert an object to multiple objects (e.g. amount and currency) so they could be mapped to multiple DB columns. I had to wrap Money struct into my own reference type (configured as owned entity) to work around this.

@andriysavin Interesting approach, it seems like the EF team will provide support for value converters to map to multiple DB columns in the release of EF Core 6.
Meanwhile, are you interested in sharing your Money wrap workaround code?

@chrisbbe It's as simple as this:
public sealed class Price
       : IComparable<Price?>, IEquatable<Price?>
{
       private readonly Money price;

       public decimal Amount => price.Amount;
       public string Currency => price.Currency.Code;
...........
...........
}

..........
// In DbContext configuration  for enclosing entity (Product)

b.OwnsOne(prod => prod.Price, o =>
{
// Read-only properties are not mapped by convention.
// For decimal in SqlServer you should explicitly specify
// the column type to avoid warning.
o.Property(price => price!.Amount).HasColumnType("decimal(18,2)");
o.Property(price => price!.Currency);
})


Update SSL policy guidance to use latest pre-defined policy

Before selecting the "Create pull request" button:

Enter a meaningful title above^, using a prefix if necessary and keywords "New" or "Update" indicating the nature of changes.

Describe the summary, scope, and intent of this PR:
In this PR I am updating the SSL policy guidance to point to the latest pre-defined policy, as well as updating the link to point to the list of pre-defined policies

Insert links(s) to any related work item(s) or supporting detail:
https://learn.microsoft.com/en-us/azure/application-gateway/application-gateway-ssl-policy-overview#predefined-tls-policy

Pre-defined policy list used in the reference
AFTER YOUR PR HAS BEEN CREATED, expand this section for tips and additional instructions.
These are common guidelines for contributions across the repos managed by the Cloud Architecture Content Team (CACT). Some repositories may have additional specific requirements that are not listed here.
Guidance for all contributors

Topic
Guidance

Draft PR
If your PR will be a work-in-progress for more than a day or two, select the Convert to draft link in the upper right of the page (under Reviewers) to change it to a draft. For future reference, you can also do this using the Create pull request button drop-down during PR creation.

ms.date metadata
Don't update an article's "ms.date" metadata property unless you've done a full freshness review of the content. A full freshness review includes changes required to correct or improve the full technical accuracy of the article.Don't update "ms.date" if you're doing targeted changes to improve non-technical aspects of the article, such as editorial quality, art improvements, article template alignment, etc.If you've changed any "ms.date" properties for work that wasn't part of full review for freshness, please reset them to their previous value.

Placement and linking
If you're creating a new article or articles, include updates to the related TOC.yml file to propose where the article(s) should be placed. Also consider other places within the document set where it would be beneficial to cross-reference and link to your new article(s).

PR build
After you open your PR, and for each successive commit that you push to your branch, the publishing platform will run validation on the files in your pull request. A summary of the build results for each file will be inserted inline into your pull request, which includes any build suggestions/warnings/errors. PRs cannot be merged until all build errors and most warnings are resolved.

Publishing
Following a successful merge, most repos publish to the live site at least once per (business) day, usually around 10am Pacific.

Additional resources
Learn.Microsoft.Com contributor guide

Additional guidance for private repos and internal contributors

Topic
Guidance

PR size
If your PR is more than ~5 lines of changes, or you'd like for the changes to go through editorial or larger review, open a contribution request at https://aka.ms/Contribution and include a link to the PR in response #8. Once it's processed, you'll be notified of the next steps.

PR title prefix
Select the Edit button to the right of the PR title if you need to revise it. The following prefixes are reserved for specific contribution types:[Quality Check] - maintenance work related to content quality (edit passes, art improvements, template alignment)[LinkFix] - recurring/adhoc PRs to correct link URLs[Pipeline] - new/updated contributor success pipeline content[WIP] - a work-in-progress draft requiring several days/weeks

PR preview
Following successful build of your PR, publishable files will also include Preview URL links to staged previews of your new/updated articles. Be sure to review these for verification of your intended contributions, or to send to other internal contributors for review.

PR sign-off (public repo)
If an article you own is updated in a public repo PR, you are responsible for sign-off. You will be automatically notified via email. The PR will not be merged until you've had a chance to review and sign-off.

PR sign-off (private repo)
After you've completed your proposed changes, addressed build warnings, and completed all review work, you can begin the sign-off process for review and merge:If your PR is in draft mode, remove "[WIP]" from the title and select Ready for review button at the bottom of the PR.Enter "#sign-off" in a new comment. This comment indicates that you're confident the work meets or exceeds Microsoft's standards for publication, and will trigger the review process.Your PR may be selected for initial review by the CACT. Following CACT review, you may receive questions or requests for additional changes. You should have initial feedback from CACT review within a few business days. If you have an urgent request or need to contact the team, please mention @MicrosoftDocs/cloud-architecture-content-team-pr-reviewers in your PR and someone will get back to you. After CACT review is complete, a CACT #sign-off will be added.Final review/merge is done by the PR review team. The PR team may also respond with feedback, categorized as "Blocking" (requires action from you), or "Non-blocking" (to be addressed in a future PR).

Additional resources
Learn.Microsoft.Com internal contributor guideAuthoring templates: architecture-center-pr, well-architected-prTo contact the CACT use e-mail, or @mention our GitHub team in your PR comments using: @MicrosoftDocs/cloud-architecture-content-team-pr-reviewers

PnP #sign-off
#remove-label:"pnp-review-in-progress"
#remove-label:"do-not-merge"
#label:"ready-to-merge"

@ckittel - It looks like an outstanding change requested is blocking merging. Can you dismiss this request if the change no longer applies?

Thanks!

You bet!


error in vis.py

python demo/vis.py --video sample_video.mp4
Generating 2D pose...
100%|█████████████████████████████████████████| 291/291 [00:19<00:00, 14.91it/s]
Generating 2D pose successful!
['checkpoint/pretrained/351/model_4294.pth']
Generating 3D pose...
0%|                                                   | 0/291 [00:00<?, ?it/s]
Traceback (most recent call last):
File "demo/vis.py", line 279, in
get_pose3D(video_path, output_dir)
File "demo/vis.py", line 225, in get_pose3D
show3Dpose( post_out, ax)
File "demo/vis.py", line 72, in show3Dpose
ax.set_aspect('equal')
File "anaconda3/envs/lili/lib/python3.8/site-packages/mpl_toolkits/mplot3d/axes3d.py", line 323, in set_aspect
raise NotImplementedError(
NotImplementedError: Axes3D currently only supports the aspect argument 'auto'. You passed in 'equal'.

I download the code ,and change nothing, tks

i will try it tks

hello，I have encountered the same problem. How did you solve it.


Collect plan statistics for explain only by default

Plan stats collection creates considerable load on
Hive metastore (even when CBO is disabled). Therefore
is better not to collect stats for plan for
non-explain queries

@findepi AC


Frontend contributions in dashboard

The seven reverted commits have been squashed into one commit, as seen on no-cas. @reinvanimschoot can I count on you to bring this branch up to date?


nsm-operator first release - k8s - fixed CRD names

Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[x] Has you operator nested directory structure?
[x] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?

Updates to existing Operators

[ ] Is your new CSV pointing to the previous version with the replaces property?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

Hi @acmenezes, try to have a look on the following error:
error loading manifests from configmap: yaml: line 2: mapping values are not allowed in this context

Hello @J0zi , could you please point me towards where this message appeared? I'm trying to figure out how to reproduce it. I guess it may be related to the roles and role bindings that have also cluster role and role binding in the same yaml file and somehow the lines are broken. Not sure though.

This is likely a problem with the way we load catalogs from configMaps in the pipeline. The author has tested this with operator-registry and it worked.


text hidden in SQL Editor by a strange diagonal white rectangle

Description
See the image - half of the SQL Editor is hidden by a strange white diagonal rectangle.
This diagonal rectangle moves as I scroll the open script up or down.
This only happens after I open a connection to the database and the script contains at or after line 22 SELECT * FROM pg_trigg
This did not happen in previous versions.
ok:

if I add a few more newlines - the text is hidden by a strange white diagonal rectangle.

DBeaver Version
Community 23.3.1
Operating System
MacOs Sonoma 14.2.1
Database and driver
Postgres 15.4, default drivers
Steps to reproduce

Open a new SQL script
paste this script

SELECT 1;

SELECT * FROM pg_trigg

Connect to the DB the script defaults
DBeaver "hangs" for 2-3 seconds and then this rectangle appears
Disconnect the connection - the triangle is gone and the text reappears

Additional context
No response

I have the same issue after upgrade.
DBeaver Version
Community <IP_ADDRESS>312241705
Operating System
MacOs Sonoma 14.1.1
Database and driver
Oracle / default drivers ojdbc11
Steps to reproduce

I just opened an existing script; nothing special done to reproduce.

Please provide the log files. You can find instructions here https://dbeaver.com/docs/wiki/Log-files/.

Logs_Dbeaver.zip

Having the same issue with DBeaver Community Edition Version <IP_ADDRESS>312241705 Release Date: 25 December 2023
Running on macOS Sonoma Version 14.2.1

Have the same issue in SQL Editor
it appears after query run attempt
DBeaver Version
Community 23.3.1
Operating System
MacOs Sonoma 14.0 (23A344)
Database and driver
Postgres 12.16, default drivers
Logfiles:
DBeaverData.log.zip
Screenshot

Seeing the same. DBeaver CE Version <IP_ADDRESS>401072024 as installed by Homebrew on macOS 14.2.1.  Resizing the query pane fixes it temporarily but it comes back.

i have the same issue

I tried to reproduce this behaviour on the latest OS version 14.2.1 but all works as expected for me.

I would recommend to use latest OS version.

+1 for me, same behavior as others have reported, on latest versions of macos and dbeaver. after a dbeaver upgrade, this triangle started happening.

@bdewilde Could you provide more details (script, table ddl, theme settings, chip, etc) because it is not reproducible on on my side.

Hi @serjiokov , this only seems to be an issue in scripts; scrolling in / resizing of that window causes the triangle to move around, but it appears on its own shortly after opening a new script. Interestingly, I can't get it to appear using the "dark" theme -- which I see you are using -- only the "light" and "classic" themes.

I'm seeing the same thing. Here's an ultra simple script that is producing this blank space in the editor. I'm seeing this on 2 different MySQL databases, one local, one on AWS. 14-inch MacBook Pro, 2021, M1 Pro chip, 16 GB, Sonoma 14.2.1. DBeaver Community Version <IP_ADDRESS>401211839

create table test1 (
id int auto_increment primary key,
name varchar(100) not null,
created timestamp not null default current_timestamp
);
create table test2 (
id int auto_increment primary key,
name varchar(100) not null,
test1_id int null,
created timestamp not null default current_timestamp
);
insert into test1 (name)
select 'test1' as name
union
select 'test2' as name
union
select 'test3' as name
union
select 'test4' as name
union
select 'test5' as name
union
select 'test6' as name
union
select 'test7' as name
union
select 'test8' as name
union
select 'test9' as name ;
select n.id, p.wwwww
from xxx.YYY n

It appears to be something to do with the syntax validation. In my case the table name is being highlighted in red to suggest it is invalid (actually it isn't), as soon as I move to the next line the triangle appears. If Iput a space between my select &  from statements the syntax validation identifies the table name as valid (not highlighted in red) and moving the cursor does not cause the triangle to appear. It should be noted that depending on what row in the script I am on the triangle will either obscure half the screen or just me a small triangle on the right.

Thank you @TobyHewing, disabling Enable semantic analysis (experimental) under Preferences > Editors > SQL Editor > Code Editor seems to fix the issue over here.

Same issue https://github.com/dbeaver/dbeaver/issues/22813

Will this be addressed? Seems pointless to use an external client if this function does not work?
Just to add you can still enable semantic analysis but Read database table/columns for semantic analsysis must be off.
Still better to see objects color coded.

Thank you @TobyHewing, disabling Enable semantic analysis (experimental) under Preferences > Editors > SQL Editor > Code Editor seems to fix the issue over here.

Unfortunately, we can't reproduce it. It looks like some mac-specific rendering issue and we're unsure, what could cause it, to fix the bug blindly. We keep this issue in mind, but haven't figured out a solution yet.

I can confirm disabling "Enable semantic analysis (experimental)" in Code Editor preferences does stop the behaviour.

I was wondering, could that be an Apple Silicon issue or do all macs have the same issue?

I have an Intel Mac and I see the same issue.

Thanks for the hint @afreller-1wa . Just disabling "Read database table/columns for semantic analsysis" definitely helped to remove the white triangle.
I am using an M1 Mac on the latest MacOS + latest DBeaver version.

Also facing this issue on:
dbeaver 23.3.5
macos 14.3.1
Disabling "Read database table/columns for semantic analysis" fixies it.
In my case it seems that the "g" letter is triggering the behavior.
Here is one that is triggering the triangle shape.
 @E1izabeth can you reproduce with this snippet ?

@jean-francois-labbe thank you, we'll try

Just for reference, I updated to Version <IP_ADDRESS>404071654 and the issue is still there (MacOS).

@meglio the fix will be available in the next version 24.0.3


application config management

I'm not sure if this feature is there already (and I haven't found it), but it would be nice to support more natively application config management (set, get).
At the moment the best way I could find to config-set is:
await ops_test.juju("config", APP_NAME, "key=value")
However, this makes unsetting keys impossible:
await ops_test.juju("config", APP_NAME, 'key=""') --> 'key' is now '""' (quoted quotation marks), not None.
I solved this by doing
os.system(f'juju config {APP_NAME} key=""')
But this is an indication that a unit config wrapper would be very welcome :)

This is a feature available through libjuju.
async def test_my_config_changes(ops_test)
    app = ops_test.model.applications[APP_NAME]
    await app.set_config({"key": "value"})
    # test something happening with this key=value
    await app.reset_config(["key"])


Add big integer opcode

Problem
These opcodes will allow us to implement big group, which would in turn allow us to implement higher level primitives like ecdsa
Happy Case
Add BigIntAdd, BigIntNeg, BigIntMul, BigIntInv
Alternatives Considered
No response
Additional Context
We should be careful about BigIntInv and think about whether we want to just limit the bigint modulus to be prime
Would you like to submit a PR for this Issue?
No
Support Needs
No response

Should we rather have BigIntDiv instead of BigIntInv?
BigIntDiv works for non prime modulus as well.

Should we rather have BigIntDiv instead of BigIntInv?
BigIntDiv works for non prime modulus as well.

Yep lets go with Div!

Reopening as the opcode doesn't have an implementation in ACVM

Closed with https://github.com/AztecProtocol/aztec-packages/pull/4415.


centOS: command not found

I have used pip to install the scour properly, but when I input the command, it prompted that bash: scour: command not found

There are several options:

Try using python3 -m scour.scour instead of scour

Maybe you could try using sudo pip3 install scour or sudo yum install python3-pip python3-scour

Maybe try seaching the executable in the subfolders of /usr and your home-folder by using find /usr/ ~ -name scour

If you know where the executable is stored, than make a symbolic link in the folder ~/bin (or any folder in $PATH), if you prefer a grafical interface for symlinks you could use /tassaron/symlink-editor.

Restart the computer and type which scour, if it returns something the comand should work.


Skip finished tasks when checking for async leaked tasks

What:
Updated leaked tasks checking to skip finished coroutines, in order to avoid incorrect failures when tasks have been completed
Why:
With Python versions that doesnt include this fix (https://github.com/python/cpython/pull/7174), Testslide will raise a LeftOverActiveTasks exception even if all coroutines have been completed successfully.
Example:
 <Task finished coro=<TestAsyncRun.test_catches_leaked_tasks.<locals>.dummy_async_func() done, defined at *> result=None created at *>
Although the task is done, Testslide will fail.
This PR skips finished tasks from the async leak testing
How:

Added extra function get_running_tasks to return only non done tasks
Updated _async_ensure_no_leaked_tasks to use this new function

Checklist:

[x] Added tests, if you've added code that should be tested
[x] Updated the documentation, if you've changed APIs
[x] Ensured the test suite passes
[x] Made sure your code lints
[x] Completed the Contributor License Agreement ("CLA")

Thanks for the PR @kchatzii


Integartion

Hi. I want to integrate trustwallet with my website. Is there API or any other way of doing this?
Let me know please

Sorry I don't have such kind of api