Cline removing the code comments in every line

Which API Provider are you using?
Anthropic
Which Model are you using?
Claude 3.5 Sonnet
What happened?
I have asked my code to be edited by providing some context but the cline deleted all the comments.
Steps to reproduce

Just try out any command which causes the file to be edited

Relevant API REQUEST output
No response
Additional context
No response

yup, Claude removes comments but (sometimes), it does have a benefit of less files, plus if the code is well written its self documenting, IMHO, I turned off XML commenting (used for documentation in c# etc) until the end of the development cycle. Then, we get less tokens and faster api requests (smaller) and hopefully more expressive code and less noise for Claude to ignore.

I have seen this happen, I always ask it not to remove comments, specially during developmentime its useful for itself, but it wanders off to remove comments when I did not ask it to, like it gets ADHD and gets distracted

Regarding lazy coding //rest of code here, there's been an increase in reports about this so I've made the following changes in the latest v2.0.15 that should help mitigate the issue:

Adjusted system prompt to keep Cline from doing this as much as possible, but for large enough files it's impossible to force him to output beyond the model's max output token limit. I found that trying to keep him from lazy coding too aggressively would lead to him outputting partial file contents.
When Cline does truncate code you are now alerted by the extension of 'potential code truncation' along with a link to a troubleshooting guide to work around the issue.
The guide essentially walks you through how to use the diff editor to replace the placeholder comments with the original content. I've also improved the prompts around this so that the model is made aware that the user fixed its response and to proceed accordingly with the new file contents as the baseline.

If it's any consolation, these are temporary measures since Anthropic will soon be releasing a 'Fast Edit Mode' feature to Claude 3.5 Sonnet that will enable faster + more reliable file edits, and hopefully fix this issue for good.

I have seen this happen, I always ask it not to remove comments, specially during developmentime its useful for itself, but it wanders off to remove comments when I did not ask it to, like it gets ADHD and gets distracted

Agreed but I'm finding as you automate building the code using Cline you will ask, who is going to use these comments?
I used to think Claude will and it certainly does! Claude puts in place holder and reminders and doesn't take them out, then comments I thought were very informative are later stripped out but I'm less bothered the LLM knows best I find :)
E.g. I put in "Ignore the warning generated by the following field, its a bug in the preview release and not our code so stop fixing it for the next 6 hours, LOL"
Soon it takes it out and trys to fix that warning, but its early days and Saoud had done an amazing job, but needs Claude to evolved to and the above comment will be great.


[Flutter 3] There is no hot-reload in Hover v0.47.0

Console:
hover run
hover: Using engine from cache
hover: Cleaning the build directory
hover: ⚠ The go-flutter project tries to stay compatible with the beta channel of Flutter.
hover: ⚠     It's advised to use the beta channel: `flutter channel beta`
hover: Bundling flutter app

💪 Building with sound null safety 💪

hover: Compiling 'go-flutter' and plugins
hover: Successfully compiled executable binary for linux
hover: Build finished, starting app...
hover: Running dashboard_plus in debug_unopt mode
flutter: The Dart VM service is listening on http://<IP_ADDRESS>:50300/
go-flutter: no method handler registered for method 'selectSingleEntryHistory' on channel 'flutter/navigation'
go-flutter: no method handler registered for method 'routeInformationUpdated' on channel 'flutter/navigation'
flutter: DataTable2 built: 4ms

And that's it. When I click r or something like that nothing happens.
hover version
Hover v0.47.0
hover doctor
hover: Hover version v0.47.0 running on linux
hover: Sharing packaging tools
hover: darwin-bundle is supported
hover: darwin-dmg is supported
hover: To package darwin-pkg these tools are required: mkbom,xar
hover: Install bomutils from your package manager or from https://github.com/hogliux/bomutils
hover: Install xar from your package manager or from https://github.com/mackyle/xar
hover: To still package darwin-pkg without the required tools installed you need to run hover with the `--docker` flag.
hover: To package linux-appimage these tools are required: appimagetool
hover: Install appimagetool from your package manager or from https://github.com/AppImage/AppImageKit#appimagetool-usage
hover: To still package linux-appimage without the required tools installed you need to run hover with the `--docker` flag.
hover: linux-deb is supported
hover: To package linux-pkg these tools are required: makepkg
hover: You need to be on Arch Linux or another distro that uses pacman as package manager to use this. Installing makepkg on other distros is hard and dangerous.
hover: To still package linux-pkg without the required tools installed you need to run hover with the `--docker` flag.
hover: To package linux-rpm these tools are required: rpmbuild
hover: You need to be on Red Hat Linux or another distro that uses rpm as package manager to use this. Installing rpmbuild on other distros is hard and dangerous.
hover: To still package linux-rpm without the required tools installed you need to run hover with the `--docker` flag.
hover: To package linux-snap these tools are required: snapcraft
hover: Install snapd from your package manager or from https://snapcraft.io/docs/installing-snapd
hover: To still package linux-snap without the required tools installed you need to run hover with the `--docker` flag.
hover: To package windows-msi these tools are required: wixl
hover: Install msitools from your package manager or from https://wiki.gnome.org/msitools/
hover: To still package windows-msi without the required tools installed you need to run hover with the `--docker` flag.
hover:
hover: Sharing flutter version
Flutter 3.0.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision f1875d570e (5 days ago) • 2022-07-13 11:24:16 -0700
Engine • revision e85ea0e79c
Tools • Dart 2.17.6 • DevTools 2.12.2
hover: Flutter engine commit: https://github.com/flutter/engine/commit/e85ea0e79c6d894c120cda4ee8ee10fe6745e187
hover: ⚠ The go-flutter project tries to stay compatible with the beta channel of Flutter.
hover: ⚠     It's advised to use the beta channel: `flutter channel beta`
hover: Finding out the C compiler version
gcc (Debian 11.3.0-4) 11.3.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

hover: Sharing the content of go.mod
module: dashboard/go

go 1.18

require (
	github.com/go-flutter-desktop/go-flutter v0.52.1
	github.com/go-flutter-desktop/plugins/path_provider v0.4.0
	github.com/go-flutter-desktop/plugins/url_launcher v0.1.3
	github.com/pkg/errors v0.9.1
)

require (
	github.com/Xuanwo/go-locale v1.1.0 // indirect
	github.com/adrg/xdg v0.2.1 // indirect
	github.com/go-gl/gl v0.0.0-20211210172815-726fda9656d6 // indirect
	github.com/go-gl/glfw/v3.3/glfw v0.0.0-20220712193148-63cf1f4ef61f // indirect
	golang.org/x/sys v0.0.0-20211023085530-d6a326fbbf70 // indirect
	golang.org/x/text v0.3.7 // indirect
)
hover: Sharing the content of hover.yaml
application-name: ""
executable-name: ""
package-name: ""
organization-name: com.dashboardplus
license: ""
target: lib/main_desktop.dart
branch: ""
cache-path: ""
opengl: ""
engine-version: ""
hover: Sharing the content of go/cmd
go/cmd/main.go	go/cmd/options.go

This is similar to  https://github.com/go-flutter-desktop/go-flutter/issues/410

On my older flutter version I see:

(Hover v0.47.0)

So the problem is from Flutter 3.0.5?

On my older flutter version I see ...

What is your flutter version?

Flutter 2.8.0-3.3.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision 262b70ece1 (8 months ago) • 2021-12-01 13:00:48 -0800
Engine • revision 06a7363b0c
Tools • Dart 2.15.0 (build 2.15.0-268.18.beta)


Internal Server Error : code 500

Hi,
I installed openocr on Ubuntu on my laptop and tested the API with curl and it works perfect.
But I'm trying to create an android app which uploads file to openocr and gets back the text.
But i'm getting this response from the server:
SERVER RESPONSE: Response{protocol=http/1.1, code=500, message=Internal Server Error, url=http://<IP_ADDRESS>:8080/ocr-file-upload}
Please can you help with this, I'm trying this from 3-4 day and I'm still not able to get it work.
I'm using okHttp library to make the request and following is my code:
    public void doRequest() throws Exception {

        String BOUNDRY = "---BOUNDARY";
        StringBuffer requestBodyString = new StringBuffer();
        requestBodyString.append(BOUNDRY);
        requestBodyString.append("Content-Type: application/json");
        requestBodyString.append('\n');
        requestBodyString.append("{\"engine\":\"tesseract\"}");
        requestBodyString.append(BOUNDRY);
        requestBodyString.append('\n');
        requestBodyString.append(BOUNDRY);
        requestBodyString.append("Content-Disposition: attachment;");
        requestBodyString.append("Content-Type: image/png");
        requestBodyString.append("filename=\"attachment.txt\".");
        requestBodyString.append('\n');
        requestBodyString.append(new String(getBytesFromFile(uploadFile1)));
        requestBodyString.append(BOUNDRY);

        String reqBody = requestBodyString.toString();
        RequestBody requestBody = RequestBody.create(null, reqBody);

        final Request request = new Request.Builder().addHeader("Content-Type", "multipart/related").addHeader(" boundary", "---BOUNDARY")
                .url(serverUrl)
                .post(requestBody)
                .build();

        client.newCall(request).enqueue(new Callback() {

            @Override
            public void onFailure(Call call, IOException e) {
                Log.v("MainActivity", "FAILED: " + e.getMessage());
            }

            @Override
            public void onResponse(Call call, Response response) throws IOException {
                Log.v("MainActivity", "SERVER RESPONSE: " + response.toString());
            }

        });

        //Response response = client.newCall(request).execute();
        // if (!response.isSuccessful()) throw new IOException("Unexpected code " + response);

    }//end do request

Please can you help in finding an error in this code, or provide a better code for android.
Thanks!

I think the best way to debug this would be to:

Run https://github.com/tleyden/open-ocr-client and upload the same file you are trying to upload with the android app
Use HTTPScoop (or Wireshark, or similar tool) to look at the traffic on the wire to see what's different

If you can collect the server logs, it might reveal something.
If you started it via docker-compose, you can just run:
$ docker logs dockercompose_openocr_1

otherwise you'll need to find the container id with docker ps
$ docker ps
$ docker logs <container-id>

Another approach might be to ask on the OkHttp stack overflow
This line looks suspect to me: requestBodyString.append(new String(getBytesFromFile(uploadFile1)));

Closing due to inactivity.  Doesn't look like an openocr issue


[WIP] Fix Autobahn test failures

There was a failure in the Autobahn tests because they were searching for a file that no longer exists.

Doesn't this need to go into 2.1 first and then get merged forward?

Doesn't this need to go into 2.1 first and then get merged forward?

Yes, you're right

Replacing in favor of https://github.com/aspnet/AspNetCore/pull/4302


Recognition of Ellipinema (Polypodiaceae) and rejection of synonymization of 6 well-established genera with Lepisorus

Author(s) of proposal
Li-Bing Zhang
Name of taxon
Ellipinema, Lemmaphyllum, Lepidomicrosorium, Lepisorus, Neocheiropteris, Neolepisorus, Paragramma, Tricholepidium
Rank of taxon
Genus
Approximate number of species affected
40
Description of change
A new genus, Ellipinema, was described by Zhang et al. (2020) to accommodate one newly sampled species with a surprising phylogenetic position so that the 6 Old World well-established genera and the species names under them can continue to be used.
In contrast, based on the similar results Zhao et al. (2019) lumped the all other 6 genera in Lepisorus and made dozens of new combinations.
These relevant 6 genera are Lemmaphyllum (ca. 10 spp.), Lepidomicrosorium (ca. 10 spp.), Neocheiropteris (2 spp.), Neolepisorus (6 spp.), Paragramma (2 spp.), Tricholepidium (ca. 8 spp.). Lepisorus itself contains ca. 80 spp.
Reason for change
Wang et al. (2010) resurrected Paragramma from a synonym of Lepisorus and re-circumscribed the tribe Lepisoreae (Polypodiaceae) to contain 7 genera, which were all accepted by PPG I (2016). Later, both Zhao et al. (2019) and Zhang et al. (2020) found a newly sampled species, Lepisorus jakonensis, from western China (also found in N India), to be the second diverging lineage in the tribe Lepisoreae. Zhao et al. (2019) thus lumped all other 6 genera in Lepisorus, made dozens of new combinations, and treated these genera as (new) sections in Lepisorus. In contrast, Zhang et al. (2020) took a different approach by describing a new genus, Ellipinema, to accommodate the single species Lepisorus jakonensis, so that the other morphologically diagnosable and well-established genera can be retained, minimizing the name instability.
The 6 non-Lepisorus genera, synonymized by Zhao et al. (2019), have long been widely recognized (e.g., Ching, 1978a, b; Ching and Shing, 1983; Lin, 2000; Wang et al., 2010a, b; Zhang et al., 2013; Fraser-Jenkins, 2015; PPG I, 2016; Ebihara, 2017; Chen et al., 2019).
Notably, Zhao et al. (2019) had very small taxon sampling of non-Lepisorus genera, e.g., 1 or 2 accessions only each from Lepidomicrosorium, Neocheiropteris, Paragramma, and Tricholepidium, and 4 accessions only from Neolepisorus. Additionally, the relationships among the non-Lepisorus genera, especially those of Neolepisorus were not well supported in Zhao et al. (2019). Compared to the previous studies, Zhang et al. (2020) achieved complete or near-complete sampling of the tribe, as estimated by PPG I. As a result, the most of the relationships presented in Zhang et al. (2020) were well-supported, particularly when compared to the other recent studies (Wang et al., 2010; Chen et al., 2019; Zhao et al., 2019). Based on the work of Zhang et al., 2020, it was clear that collapsing the 6 well-established genera to accommodate the surprising phylogenetic position of one species did little more than obscure the genetic diversity of the lepisoroids as done by Zhao et al. (2019).
One additional aspect to consider is that species of nearly all these six genera, particularly those of Neocheiropteris and Neolepisorus, have great value in horticulture. Changing their generic names would cause much confusion and inconvenience for horticulture industry.
Importantly, two generic names Zhao et al.'s (2019) lumped in Lepisorus, Lemmaphyllum (1851) and Neocheiropteris (1905), are much older than Lepisorus (1933), threatening the application of Lepisorus, the largest genus in the tribe. Wei & Zhao (2019) thus proposed to conserve Lepisorus against Lemmaphyllum and Neocheiropteris, complicating the matter further. The fate of their proposal depends on the Botanical congress in 2024, causing a dilemma for PPG II now. What if their proposal is rejected?
Indeed, Ellipinema hybridized with one species of Lepisorus, but this issue has been solved by the description of a hybrid genus: xEllipisorus (Zhang & Zhang, 2020). Hybrid genera are rare but still acceptable in ferns. Again, only one name is affected.
Wei et al. (2021) presented two nuclear trees of the tribe with relatively poor and somehow conflicting resolution based on a small sampling in a short communication in MPE (? not peer-reviewed). They proposed some ancient hybridization and incomplete lineage sorting (ILS) for the poor and conflicting resolution, which were hardly convincing at all. If the incongruence is largely due to ILS, the plastid trees in Zhao et al. (2019) and Zhang et al. (2020) are more accurate, since plastid coalesces more rapidly.
In summary, it seems to be the best and simplest practice to recognize Ellipinema due to deep phylogenetic divergence, morphological diagnosability, and nomenclatural stability.
Reference(s) for publication of the name
Important references:
Wang, L., Wu, Z.-Q., Xiang, Q.-P., Heinrichs, J., Schneider, H., Zhang, X.-C., 2010. Molecular phylogeny and a revised classification of tribe Lepisoreae (Polypodiaceae) based on an analysis of four plastid DNA regions. Bot. J. Linn. Soc. 162, 28–38. https://doi.org/10.1111/j.1095-8339.2009.01018.x
Wang Li et al. 2010_Lepisoreae.pdf
Chen, C.-C., Hyvönen, J., Schneider, H., 2019. Exploring phylogeny of the microsoroid ferns (Polypodiaceae) based on six plastid DNA markers. Mol. Phylogenet. Evol., 106665.
Chen et al. 2019 microsoroid ferns.pdf
Zhao, C.-F., Wei, R., Zhang, X.-C., Xiang, Q.-P., 2019. Backbone phylogeny of Lepisorus (Polypodiaceae) and a novel infrageneric classification based on the total evidence from plastid and morphological data. Cladistics 1–24. https://doi.org/10.1111/cla.12403.
Zhao et al. 2019 Lepisorus.pdf
Wei, R., Zhao, C-F. 2019. Proposal to conserve Lepisorus nom. cons. against the additional names Lemmaphyllum and Neocheiropteris (Pteridophyta, Polypodiaceae). Taxon 68 (6): 1363–1371.
Wei & Zhao 2019_ Proposal to conserve Lepisorus.pdf
Zhang, L, Zhou, X-M, Liang, Z-L, Fan, X-P, Lu, NT, Song, M-S, Knapp, R, Gao, X-F, Sun, H, Zhang, L-B. 2020. Phylogeny and classification of the tribe Lepisoreae (Polypodiaceae; pteridophyta) with the description of a new genus, Ellipinema gen. nov., segregated from Lepisorus. Molecular Phylogenetics and Evolution 148 (July 2020): 106803. doi: https://doi.org/10.1016/j.ympev.2020.106803.
Zhang et al. 2020_Lepisoreae_Ellipinema.pdf
Wei, R., Zhao, C-F, Xiang, Q-P, Zhang, X-C. 2021. Ellipinema and ×Ellipisorus? Just Lepisorus (Polypodiaceae)! Molecular Phylogenetics and Evolution. 161: https://doi.org/10.1016/j.ympev.2021.107176.
List the numbers of any related issues
#47
Code of Conduct

[X] I agree to follow the PPG Code of Conduct

The fate of their proposal depends on the Botanical congress in 2024, causing a dilemma for PPG II now. What if their proposal is rejected?

I would just like to clarify the policy about cases where a PPG proposal depends on a nomenclatural change that needs to be approved by IBC. Our current policy is that such proposals may be voted on by PPG as long as a formal nomenclatural proposal to IBC has been submitted. For more details, please see the Discussion, "How to proceed when PPG proposals are related to proposals before the IBC nomenclature committee".

It is not task of PPGII to handle proposals that are part of the decision making process during the taxonomy meetings held in association with IBC meetings. Thus, I would prefer that this proposal is eliminated from the discussions at this stage.

@SchneiderHarald did you see my post above? I believe our current policy is to allow such PPG proposals as long as a nomenclatural proposal to IBC has been submitted (not necessarily yet passed at the IBC meeting).

This proposal was voted on during PPG Ballot 5 (voting period November 2023). A total of 64 votes were cast. There were 18 'Yes' votes (28.1%) and 46 'No' votes (71.9%). The proposal does not pass.


Fixed mutable default argument problem

Description
This fixes the problem pointed out by Alex where I had used a mutable default argument, which is apparently a common python gotcha. I fixed it using the approach laid out here. I've fixed this bug in the place Alex pointed out, and checked to see if I had made the same mistake elsewhere (I luckily hadn't).
Fixes #85
Type of change

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

I'm puzzled here that mypy doesn't insist on Optional[list] here?

@davidorme That's a good spot! I think you're right, strict_optional = true is a more sensible setting -- otherwise any value can be implicitly None, which really isn't good from a type-checking perspective. It's handy to have the types annotated Optional as appropriate and helps mypy give better linting.


[Backend] Integrate Delete Faq with Backend

Deliverables-

[ ] Have an icon/button for each Faq present on the Manage FAQs Page and clicking the same should use the DELETE API to remove the data from backend.
[ ] There should be a toast showing Faq deleted successfully after successful deletion and should show proper message if the deletion fails

NOTE-

This issue depends on Issue-677

I want to work on this issue.

I am a GSSOC'22 participant, please assign me to this issue.

I want to work on this issue, Can you please assign it to me under GSSOC22.

I would like to work on this issue under GSSOC '22

Hello @hazemessam please go ahead with this issue.

@Kajol-Kumari
Sorry I was thinking that it is a backend task.
Please, unassign me and assign it to a frontend dev

@Kushal-2002 are you willing to working this issue?

@Kajol-Kumari yes definitely please assign me.

@Kushal-2002 can I please get an update on this issue?

Hi @Kajol-Kumari I am unassigning myself as I have not been able to find a solution to this problem in the issue.

@Bhawna147 please go ahead with this issue.
Happy Learning :)

Hey @Kajol-Kumari , I found that this issue is not yet been fixed so I would like to take up this one.

Hello @Kajol-Kumari, I would like to work on this issue under JWOC 2023.

Can I work on this issue?

@Anand930singh  please go ahead with this issue. Just make sure to provide an update on the progress within 1 week.
Happy Learning :)

I will work on this issue :)

@WYGIN  Sure, please go ahead, Happy Learning :)

@Kajol-Kumari the DELETE API no more exists!

Hey @WYGIN  you can find it here - https://github.com/HITK-TECH-Community/Community-Website/blob/main/backend/app/routes/faq/deleteFaq.js

@WYGIN can you update the status of this issue?

I am currently not available to work, please assign it to someone else if anyone is interested.

please assign me this issue as GGSoc '24


Fix R installation in WSL

There is some problem with staged installation ( https://developer.r-project.org/Blog/public/2019/02/14/staged-install/) in WSL
This pull request deactivates this method of installation
Tasks

[x] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[x] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[x] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[ ] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[ ] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[x] Please use yamllint to check that your YAML files do not contain mistakes
[x] (Only if really necessary) Add any additional dependencies needed for the diagnostic script to setup.py, esmvaltool/install/R/r_requirements.txt or esmvaltool/install/Julia/Project.toml (depending on the language of your script) and also to package/meta.yaml for conda dependencies (includes Python and others, but not R/Julia)
[x] If new dependencies are introduced, check that the license is compatible with Apache2.0

Closes ESMValGroup/ESMValTool_Tutorial#115

@stefsmeets Could you please check if this works for you, since you're also into WSL?

Hi @bouweandela , I actually just updated to WSL 2 this morning and on a fresh Ubuntu distro the installation is working as expected.

This seems to work :-)


Minor typo in New-MailboxImportRequest documentation

In the documentation for the New-MailboxImportRequest cmdlet, you have the following snippet:

New-MailboxImportRequest User2 -FilePath \\server\share\User1.pst -IsArchive -TargetRootFolder /
This example imports a .pst file into Kweku's archive folder[...]

The code block says "User2", but in the description, the user's name is given as Kweku.  While the example is still understandable, hese two names should be the same for clarity.

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

ID: 1521bf62-ae37-50f3-3cf7-3ec385399f96
Version Independent ID: 1ad16056-d5c5-a815-c6d6-8e38b96fdbd8
Content: New-MailboxImportRequest (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/New-MailboxImportRequest.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@rhymeswithmogul, fixed. In the future, you too can make changes like this without the need to open an Issue first. For instructions, see https://github.com/MicrosoftDocs/office-docs-powershell/blob/master/README.md.
Thanks for the feedback.
Closing


:recycle: working on #136

Breaking pages up into components, making stuff dynamic and cleaning up redundant schemas in Hygraph, fixing a few styling issues along the way.
Still a lot of work todo:
DRY css strategy
Refactor organisms/molecule structure
Refactor static stuff
Consistent page structure (KnowledgeClips, DrawingSpace)

@iBadr49 & @Daniquedejong  I'm merging this PR due to breaking changes in Hygraph.
Please review for bugs and inconsistencies anyway 😁


the installer will only work on python 3.5 32 bit unless the following is fixed

in core.py, line 21, you have the following:
rt = ctypes.PyDLL('libmgrs.cp35-win32.pyd')
this failed on my python 3.6 32 bit. I ended up changing it to this to get it to work:
rt = ctypes.PyDLL('libmgrs.cp36-win32.pyd')
The file itself is located in:
\Lib\site-packages\libmgrs.cp36-win32.pyd
I figure there should be some code that generates the libmgrs pyd file name based on the version of python installed and if it's 32 bit vs 64 bit. I don't know how to do this, but I figured I'd post this here incase someone knows how to and is willing to do so.

Commenting because I'm responsible for hard-coding the .pyd name.
The setup/distutils package is responsible for compiling the DLL so I imagine that it's also naming it. This method stands out to me as a way to get a portable solution.


Emphasize to Conda that this needs to be noarch

Despite having noarch: python in the top level build block, the last recipe was built per-platform.
I will mark those packages as broken once we get this fixed.

Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@twiecki this looks better. Could you please merge? (I don't have permissions here.)


[Bug]: Error casting int column database

What happened?
I am working with a TextInput where I want to display the price in EUR, and the value coming from the database is of integer type. When I apply ->currencyMask(thousandSeparator: '.', decimalSeparator: ',', precision: 2), it gives me an error in the console:

I'm using this casting:
https://filamentphp.com/docs/3.x/panels/getting-started#casting-the-price-to-an-integer
How to reproduce the bug
TextInput::make('price')                     ->required()                     ->numeric()                     ->suffix('€')                     ->currencyMask(thousandSeparator: '.',decimalSeparator: ',',precision: 2)
Package Version
1.1.1
PHP Version
8.1
Laravel Version
10.10
Which operating systems does with happen with?
macOS
Notes
No response

If I delete the casting that I created the TextColumn works fine for me:
TextColumn::make('price')->label('Precio')->currency('EUR') but TextInput have the same error in console. I'm testing in Chrome

@danielreales7, can you create a public repo that reproduces this bug so I can investigate the problem?

@ariaieboy https://github.com/danielreales7/test-filament-currency I create a new project for this with a seeder.
Thank you so much for your help!

@ariaieboy I think you have to use https://alpinejs.dev/magics/nextTick when import dynamic mask on x-init

@danielreales7 test version 1.1.2, the problem should be fixed

Hi @ariaieboy
I'm getting same error:

replaceAll for number input it's not possible.

Hi @ariaieboy
I'm getting same error:

replaceAll for number input it's not possible.

can you test it in your repo and reproduce this error on that?
https://github.com/danielreales7/test-filament-currency

fixed in 1.1.3


Securing the ELMAH axd file

Hi!
Would it be possible to secure the axd file from the middleware? In the asp.net version, the web.config was able to determine who was allowed or not (e.g., deny users="*"). Right now, anyone can access the file, logged in or not.
I would appreciate any insight on this matter.
Thank you!

merge this one to master pls


Add Documentation to class definition of RandGen

RandGen has no class level doxygen comment. Please add one.

@lavinrp you


[Docs]: Add jest-environment-jsdom information to guides for dom manipulation

🚀 Feature Proposal
Nothing too huge. While trying to integrate JSDom with Jest (for use with React Testing Lib), I could not for the life of me figure out what was missing until I dug into the change logs (https://jestjs.io/blog/2022/04/25/jest-28#breaking-changes). Just a quick addition to this documentation  clarifying the need to both manually add jest-environment-jsdom and setting the environment to jsdom would help in setting up jest quickly for dom testing.
Motivation
Makes your docs easier to use and understand so that developers would not need to dig into change logs to find answers.
Example
No response
Pitch
This is a quick and easy way to improve documentation so that users can get started on testing a lot quicker.

Good idea. Just send a PR. By the way, pointing to test environment docs (like here) could be useful too.


B2C article typo fixes

Internal Review

@CamSoper,  just a heads up I didn't Squash and Merge.  If I do it for you, but you decided you still wanted to make change in the branch in the last minute, the branch would be gone and the content merged, so best I think if internal MS authors squash/merge/delete the branch themselves. If you would rather I always do it after approving, that's fine.

I'll go ahead and squash and merge.

@wadepickett Thanks, Wade -- I completely forgot I sent this PR.


Test names formatting for BDD

Hi!
First of all, Karma is awesome! You're doing great job! ;)
But I have a specific problem.
I use Jasmine JS test framework. For example, one of my tests:
describe("QueryDict utility", () => {
  it("should parse '?a=1&b=2' into {a: [1], b: [2]}", () => {
    expect(new QueryDict('a=1&b=2').toObject()).toEqual({a: ['1'], b: ['2']});
  });
});

It will provide the following information for the reporter:

result.suite == ["QueryDict utility"]
result.description == "should parse '?a=1&b=2' into {a: [1], b: [2]}"

If I use karma-teamcity-reporter, it generates the following:
##teamcity[testSuiteStarted name='QueryDict utility.Firefox 52.0.0 (Linux 0.0.0)']
##teamcity[testStarted name='should parse |'?a=1&b=2|' into {a: |[1|], b: |[2|]}']
##teamcity[testFinished name='should parse |'?a=1&b=2|' into {a: |[1|], b: |[2|]}' duration='11']
##teamcity[testSuiteFinished name='QueryDict utility.Firefox 52.0.0 (Linux 0.0.0)']

This has several problems:

Suite/class/test names
I don't think that suiteName + '.' + browserName is the best decision for the suite name in TC (e.g. QueryDict utility.Firefox 52.0.0 (Linux 0.0.0)).
In my example, QueryDict utility is more like a class name.
One JS test file can have more than one describe() and each describe() can contain multiple it(). It looks similar to:

JS test file:

describe() ↔ test class

it() ↔ test class method

And suite name is something above the test classes. Suite groups test classes. Maybe it's good to have the JS test filename as the suite name but I'm not sure that it's available in Karma reporter. So it may be e.g. the browser name.
What I suggest

TC suite name: browser.name (e.g. Firefox 52.0.0 (Linux 0.0.0))
TC class name: require('snake-case')(result.suite.join(' ')) (e.g. query_dict_utility)
TC test name: result.suite.join(' ') + ' ' + result.description (e.g. QueryDict utility should parse '?a=1&b=2' into {a: [1], b: [2]})

So it'll look like this in TC:

TeamCity wrongly parses :\u0020 in test names
As you can see on my first screenshot, my test name is awfully parsed and displayed in TC tests report:

TC test name: [2]}
TC class name: n/a
TC suite name: QueryDict utility.Firefox 52.0.0 (Linux 0.0.0): should parse '?a=1&b=2' into {a: [1], b

That's not your fault:

TC finds the last :\u0020 entry in the test name
everything before it considered as the suite name
... despite that the suite name is already set and shouldn't be parsed at all.

There's a bug in TC YouTrack. However, there's a workaround that may be implemented in karma-teamcity-reporter: you can replace all :\u0020 with :\u00a0 - then TeamCity will skip it.
What I suggest
Replace all :\u0020 with :\u00a0 in the test name.
Custom names formatting
Compared to karma-junit-reporter, you don't provide any way to customize suite/class/test names.
What I suggest
Please, provide a way to customize suite/class/test names e.g. similar to how it's done in karma-junit-reporter:
config.set({
  // ...
  teamcityReporter: {
    suiteNameFormatter(browser, result) {
      // ...
    },
    testNameFormatter(browser, result) {
      // ...
    }
  }
});

Suite/class/test names +1, i`d love to group all unit tests in one suite and all describes as classes)


ADS exports dates like 1993-00-00

Example: https://ui.adsabs.harvard.edu/#abs/1993AIPC..280..291W/abstract

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>33 ("fixes #1066")
Install in Zotero by downloading test build <IP_ADDRESS>33, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".


Add server API

#135  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @emilpriver and the rest of your teammates on  Graphite


Bump package - possible memory leaks

Reverts annuaire-entreprises-data-gouv-fr/site#1270

relevevant packages have been bumped with #1290


Add VAS Face Recognition pipeline

Added Warp Affine to preprocessors
Added custom VAS face recognition metric

please resolve conflict

@hojae-yoon  please provide info about new components to appropriate readme files (preprocessor/README.md, metrics/README.md)

jenkins ok to test

Thank you @eaidova for the review. I will make the required changes and upload a new patch.


Allow DelayedLoadWrapper to late-construct its content

Not only does this tidy up the DelayedLoadUnloadWrapper flow, but it allows us to (in both cases) avoid the overhead of construction of the content drawable until a load actually occurs.
Tested against my beatmap carousel changes, this reduces finalizer disposal to zero (and greatly reduces allocs in fast scrolling) as expected.

Make sure you are okay with the Relative/AutoSize propgation. I haven't changed anything for the old-style ctor for compatibility reasons, but now the transfer is done at load time, overwriting anything set locally. Maybe this needs to be mentioned somewhere in the xmldoc?
Should we obsolete the old ctor?

The change itself seems fine to me.
As for the obsoletion of the old ctor - we actually have game-side usages that would arguably become uglier if we forced late construction. See LeaderboardScore for one - the constructed-but-not-loaded drawable is then used to register an OnLoadComplete callback; forcing late construction would probably make this way worse to read (would have to subscribe to DelayedLoadComplete and then nested-subscribe to OnLoadComplete).
Also, just so there's no blockage - this technically conflicts with #3936 but the resolution is relatively easy. Here's the combined diff:

combined diff
diff --cc osu.Framework/Graphics/Containers/DelayedLoadUnloadWrapper.cs
index 49d012b81a,f7c72c2e94..d383017ee8
--- a/osu.Framework/Graphics/Containers/DelayedLoadUnloadWrapper.cs
+++ b/osu.Framework/Graphics/Containers/DelayedLoadUnloadWrapper.cs
@@@ -56,8 -58,10 +56,8 @@@ namespace osu.Framework.Graphics.Contai
              }
          }

-         private bool contentLoaded;
 -        public override Drawable Content => base.Content ?? (Content = createContentFunction());
 -
          private ScheduledDelegate scheduledLifetimeUpdate;
+         private ScheduledDelegate scheduledUnloadCheckRegistration;

          protected override void EndDelayedLoad(Drawable content)
          {
diff --cc osu.Framework/Graphics/Containers/DelayedLoadWrapper.cs
index 7895f5e13c,904f4ec90c..d3922c3e20
--- a/osu.Framework/Graphics/Containers/DelayedLoadWrapper.cs
+++ b/osu.Framework/Graphics/Containers/DelayedLoadWrapper.cs
@@@ -121,14 -83,16 +121,18 @@@ namespace osu.Framework.Graphics.Contai

          protected void BeginDelayedLoad()
          {
-             if (loadTask != null) throw new InvalidOperationException("Load is already started!");
+             if (DelayedLoadTriggered || DelayedLoadCompleted)
+                 throw new InvalidOperationException("Load has already started!");

 +            Content ??= createFunc();
 +
+             DelayedLoadTriggered = true;
              DelayedLoadStarted?.Invoke(Content);

+             cancellationTokenSource = new CancellationTokenSource();
+
              // The callback is run on the game's scheduler since DLUW needs to unload when no updates are being received.
-             loadTask = LoadComponentAsync(Content, EndDelayedLoad, scheduler: Game.Scheduler);
+             LoadComponentAsync(Content, EndDelayedLoad, scheduler: Game.Scheduler, cancellation: cancellationTokenSource.Token);
          }

          protected virtual void EndDelayedLoad(Drawable content)

(in short, take both additions in DLW and both removals in DLUW).


Update README.md

Migration of CONTRIBUTING.md documentation to README.md

Hi @deneha, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;


Bugfix: FE 334 edit user crash

App was crashing because the UserForm didn't wrap onSubmit in an async await, so the function resolved immediately and the error was never caught by the try catch in the DialogForm (instead caught by CRA)
Hide the email field on UserForm during edit user.  Showing but disabling is more involved because on the EditUser component the email field is not recognized by typescript because it's reading the input type as UpdateUserInput.

What about pulling email out of input on update?
onSubmit={async ({ email, ...input }) => {}}

I tried that but get Property 'email' does not exist on type 'UpdateUserInput' because it's reading input as UpdateUserInputfrom<UserForm`

Alternatively you could check the initialValues prop in UserForm we only have it on update not create so that could be used instead of adding another prop.

I can't get TS to give me the initialValues?.person.email value Property 'user' does not exist on type 'NonNullable<T>'.ts(2339)

I tried that but get Property 'email' does not exist on type 'UpdateUserInput' because it's reading input as UpdateUserInput from <UserForm<UpdateUserInput>

You could define the form values with that field
<UserForm<CreatePersonInput & { person: { email: string } }>

That's probably easiest.
Also I'm just realizing that email is optional on creating a user so we actually do need it to be updatable to some extent.

<UserForm<CreatePersonInput & { person: { email: string } }>

Good call, applied as suggested.  I also updated the fragments and queries so the UI on the detail and formscenes  updates after the mutation.


usermod: UID '0' already exists

root@ai:/home/ai/Autoware/docker/generic# bash run.sh
Using options:
ROS distro: kinetic
Image name: autoware/autoware
Tag prefix: latest
Cuda support: on
Base only: false
Pre-release version: off
Launching autoware/autoware:latest-kinetic-cuda
Changing autoware user ID to match your host's user ID (0).
This operation can take a while...
usermod: UID '0' already exists

I see you're running docker as a root user, please switch to a non-root user. Don't forget to add it to the docker group.

Thanks for using Autoware and for your question. However the issue tracker is for posting confirmed bugs and feature requests.
We ask that you please ask questions on the Autoware Discourse category following our support guidelines: https://github.com/CPFL/Autoware/wiki/Support-guidelines. Please pay particular attention to the information we ask you to provide.


Fix issue #63 add alertDialog before signout

#63

Just incorporate one comment I added, and squash the commits into a single one.
Rest looks good 👍

You might need to create a new PR. The changed files count is 65.

Yeah. Later I will look at it. It seems I messed up something


Fix #123 - Union types without sub-classes should be sealed

This implements the F# Language User Voice feature request to change the F# compiled representations of union types by making union types sealed when their compiled representations have no sub-types.
This is also covered by suggestion #123.
Since this changes compiled forms in FSharp.Core.dll it is a candidate to consider for F# 4.0.

:+1:

Can you please update the codegen tests which are now failing? There are a bunch that use DUs which are now out of date.

@latkin - Ok, will do
I found it a bit tricky to update the one micro codegen test I added, because the *.il generated files get deleted by run RunTests.cmd process somewhere.  Is there an option to keep these around?  That would make it much simpler.

@dsyme yes this is a pain point. Let me see if there is something simple we could add to make that easier...

baselines now updated

These are still failing for me:

CodeGen\EmittedIL\SerializableAttribute (ToplevelNamespace.fs - Desktop)
CodeGen\EmittedIL\SerializableAttribute (ToplevelNamespace.fs - Portable)
Optimizations\GenericComparison (Compare05.fs - NetFx40)
Optimizations\GenericComparison (Compare07.fs - NetFx40)
Optimizations\GenericComparison (Compare10.fs - NetFx40)
Optimizations\GenericComparison (Equals04.fs - NetFx40)
Optimizations\GenericComparison (Equals06.fs - NetFx40)
Optimizations\GenericComparison (Equals09.fs - NetFx40)
Optimizations\GenericComparison (Hash05.fs - NetFx40)
Optimizations\GenericComparison (Hash06.fs - NetFx40)
Optimizations\GenericComparison (Hash09.fs - NetFx40)
Optimizations\GenericComparison (Hash12.fs - NetFx40)

Detective work indicates the TopLevelNamespace guys were patched up by hand based on similar fix to TopLevelModule. However namespace guys turn out to need fix in more locations.
And I assume it was not realized that there are codegen tests under 'Optimizations'.

Updated, thanks.  BTW if you get a chance could you look at the Perl problem in AppVeyor? https://ci.appveyor.com/project/KevinRansom/visualfsharp-radou/build/<IP_ADDRESS>

Those "Optimization" codegen tests are still failing, will look at it in the morning :)

Don't forget the portable stuff in the core\array tests, too ;-)

Now updated


Model Selection & Hyper-parameter selection

CNNs share the same fate with other Neural Nets. There are too many parameters to tune; optimal input patch size, number of hidden layers, number of feature maps per layer, pooling and stride sizes, normalization windows, learning rate and others. Thus, the problem of model selection is relatively harder compared to other ML techniques.
LibSVM comes with a nice script grid.py that performs grid search to optimize hyper-parameter for model selection. Is it possible to equip this toolbox with such tool as well? Or, it's better to utilize another model selection library to optimize hyper-parameter?
Thanks.

Some people have had good experience with https://www.whetlab.com/. Even if it doesn't optimize all parameters automatically, at least it gives you insight which hyperparameters actually matter and which don't. There is nice API for Matlab.


Support for Partner Channel Name

👋 @ellismg I'm cleaning up old issues, mind moving this to the appropriate milestone when you get a chance?

@maorleger - Done!

Updated milestone based on current understanding of when the service will go GA.  We added support for this feature in preview in #21411.
Note as part of #21411 we had to update the event grid performance test to depend on the pre-release version of the package, since the general constraint of ^4.0.0 would no longer pick up the preview package.  As part of going GA, we should undo that change so that the performance test depends on the GA'd version again.


Neue hot module updates

This PR makes some updates to neue that were introduced with the hot module work happening in DoSomething app.

Added a -center option to the figure -left pattern so that we can have text vertically centered next to the media.

Added a new .avatar pattern to show a rounded image.

@DoSomething/front-end

Looking good! Two tiny Qs but otherwise 100% :ship:-able.


cquery

https://github.com/jacobdufault/cquery/

https://github.com/Microsoft/vscode-languageserver-node


add contact form inside this file contact.html

I will add a contact form to this file please @khushi-joshi-05 can you assign me to work on this?

i will add conatct form  please assign this task to me @khushi-joshi-05

I can help with this as well. Willing to collaborate and test if it works on different devices.


Trivial change to force rerun of CI

Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


MySQL Tasks Elizaveta Panasiuk

https://travis-ci.com/github/ElizavetaPanasiuk/db-assignments

Looks good. See comments.
I'll mark the PR as "Draft", please click "ready for review" when it will be finished. Thank you!

accepted


[AspireToAzure azd] Deploy a .NET Aspire app to Azure Container Apps - need some updates and unable to create AZD environment

Issue description
Using the latest Aspire bits (VS 17.9 P1 build), we tried running through the Deploy a .NET Aspire app to Azure Container Apps document. Chose 'Azure Developer CLI' option.
Some updates are needed.
Prerequisites

[x] We should copy this section from the caching quickstart, which gives both the VS and CLI install options.

Install the Azure Developer CLI

[x] This section occurs twice in the document. Below 'Prerequisites', it simply says that you'll learn about Azure Developer CLI. And then below 'Create the template solution' section, there is a link to the actual instructions to get azd. I think just the second Azure Developer CLI section is sufficient, or we should combine the two sections into one.

Create the template solution

[ ] For the text Enter a Project Name of AspireSample., the project name should be AspireToAzure instead of AspireSample because that's the name we show in the azd screenshots.

Create a new AZD Environment

[ ] When doing azd init, I don't see the options described in this document. I only got a prompt for the environment name and a bicep file got generated which shows that a new resource group will get created. And that's it. Am I missing something?

Generated main.bicep file

Hi @balachir - can we close this, I cannot reproduce any of these issues?

Verified that the Aspire + AZD deployment workflow for ACA is working fine with the latest versions, going to close this for now.


[Figma] Content group cards

Add the Content group cards component to Figma kit. Remember we are only building variants for xlg, md, and sm.
Design spec
Website
Must haves:

[ ] Content should be flexible - the designer should be able to condense or stretch the component and the content should respond. Removing text for example should maintain the right padding automatically through auto layout.
[ ] This component uses a fluid type token for the heading. If this is not created already, create a base to help swap the header type based on breakpoint size. See Content group in the library.
[ ] Use the existing Figma component for Card group.
[ ] Variant toggle for Body copy. If body copy is removed or edited, vertical padding should adjust through auto layout.

Resources

C4IBM.com Component checklist is available in the Read Me 👀 page of the Figma library
Bases demo from the Figma Fridays (Episode 23)
Reference the best practices from Figma guild
Info on auto layout

Duplicate of #8559


If description or summary are left empty, workspace request fails

@afoerster @rk-fsdev @satish-sanjeev

@shafi1379 , could you check why some build checkes were not successful?

@shafi1379 , and in workspace request page, we need to set description and summary inputs fields as required.

@shafi1379 , and in workspace request page, we need to set description and summary inputs fields as required.

Now, it won't allow user without entering at least one character. do we need to highlight with red border?

Yeah we should, otherwise they might be confused why they can't move
forward.
On Mon, Feb 3, 2020 at 10:54 AM Shafi Parammal<EMAIL_ADDRESS>wrote:

@shafi1379 https://github.com/shafi1379 , and in workspace request
page, we need to set description and summary inputs fields as required.
Now, it won't allow user without entering at least one character. do we
need to highlight with red border?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phdata/archway/pull/60?email_source=notifications&email_token=AAD63QBS44DZVWWVW6AAEYLRBBD55A5CNFSM4KPCFMAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKUSPBA#issuecomment-581511044,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD63QCLM3YVWY5F6LIQDZTRBBD55ANCNFSM4KPCFMAA
.

Highlight will get enabled either user empty the field or touching the field.

@shafi1379 , I am sure your solution works properly. Could you dig into it by using getFieldDecorator of antd form?
        <Form.Item validateStatus={passwordError ? 'error' : ''} help={passwordError || ''}>
          {getFieldDecorator('password', {
            rules: [{ required: true, message: 'Please input your Password!' }],
          })(
            <Input
              prefix={<Icon type="lock" style={{ color: 'rgba(0,0,0,.25)' }} />}
              type="password"
              placeholder="Password"
            />,
          )}
        </Form.Item>

I think if you use getFieldDecorator of antd, we don't need isDescriptionTouched states.


Add support for missing operators

See: #167, #150, #135,
List of missing ops:
Postgres: (#126)

||
:=
=>

Unspecified/Multiple:

=>
!=

#175
Spark: ->

Fixed in #36


Only generating one .json file

Everything in the .json is coming out correct.. but its only showing the last .json in the file. Any thoughts?

Nevermind! Fully restarted again and figured out where I messed up.

closing issue


[QA-1984] UI Test results Slack notification from CircleCI

QA-1984
CircleCI workflow example: terra-ui/11131/workflows/2cad15b5-9051-4064-8ab0-0c93928649fc
Google doc.
An example of failed test notification to `cromwell_jenkins_ci_errors" channel.
Watch notify-test-results in action.
https://user-images.githubusercontent.com/35533885/182917802-e813c36b-2c42-499a-a13f-b12cc0a58088.mov

As I understand it, this starts a job (notify-test-results) that runs in parallel with integration-tests-staging/alpha. notify-test-results occasionally checks the CircleCI API to see if integration-tests-staging/alpha has finished.
Since build minutes are how CircleCI bills, I'm not keen on doubling our usage for integration-tests-staging/alpha jobs. I think there are a few alternatives that would not involve a job that polls for another job to finish.

When each parallel job of integration-tests-staging/alpha finishes, it could check if the overall job has finished and, if so, send the Slack notification.
Make the integration-tests-staging/alpha job always pass and use requires to run another job afterward. That job checks tests results, sends the Slack notification, and fails if any tests failed.

1 and 2 are good suggestions. Either requires some good amount of time for code refactoring and testing. I'll start looking into this. In the mean time, I've addressed the majority of feedback.

@nawatts Hi, I have a question. In order to make the change that you suggested (avoid running monitoring job in parallel), I would need more time for changes and testing (more commits). I don't know how long it would take and I don't know if I can make it work. So just to check, is that change absolutely required? If yes, I'd like to put PR back to draft. If not, maybe I can do the change in a follow-up PR after merge this PR first.


Candidate for v1.8.0

This is a candidate for v1.8.0. These commits constitute mostly updates to testing and the installation of test data.

Updates to setup.py
Use UGALIDIR to configure test file paths

Closes #5 #24 #67 .


feat: move filters from superset-ui to incubator

SUMMARY
Move Select and Range ant filters from superset-ui to incubator + support in it for queriesData param
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12154 (51153f2) into master (ba952ba) will decrease coverage by 3.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12154      +/-   ##
==========================================
- Coverage   66.31%   63.29%   -3.03%
==========================================
  Files         994      484     -510
  Lines       49033    29744   -19289
  Branches     4977        0    -4977
==========================================
- Hits        32515    18825   -13690
+ Misses      16373    10919    -5454
+ Partials      145        0     -145

Flag
Coverage Δ

cypress
?

javascript
?

python
63.29% <ø> (-0.93%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/hive.py
54.61% <0.00%> (-29.62%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/presto.py
70.56% <0.00%> (-12.78%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

superset/views/database/validators.py
78.94% <0.00%> (-5.27%)
:arrow_down:

... and 529 more

Continue to review full report at Codecov.

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

Hi @simchaNielsen Thanks for the PR. I think the advantage to have native filter component in superset core js codebase is obvious: quicker development cycle, easier shared function or components with other dashboard components.
But since filter component was a superset-ui plugin, may i know what is original architectural design consideration?
After moving it to superset core js, can we use/build filter component as regular dashboard component, or still need to treat native filter like other plugin? cc @villebro

Related: https://github.com/apache-superset/superset-ui/pull/840

Related: https://github.com/apache-superset/superset-ui/pull/840


fix(ElementalPageExtension): Make ElementArea gridfield insert before 'Metadata' if it exists.

fix(ElementalPageExtension): Make ElementArea gridfield insert before 'Metadata' if it exists.

Any particular reason for this not being pulled back in? Running a few 3.2 sites, and the ElementArea GridField looks ugly when its placed underneath the 'Metadata' ToggleCompositeFIeld.
I assume that field must either be gone for you DNADesign guys or it was removed in a later SS version.

No reason apart from I didn't see it..


WSLg windows not showing on RDP Connection

Environment
Windows build number: 10.0.22000
Your Distribution version: 20.04
Your WSLg version: 1.0.26

Steps to reproduce

Connect to remote host with WSLg enabled via RDP.
Run X app (in my case Chrome)

WSL logs:

Attach WSLg logs from  /mnt/wslg

weston.log
stderr.log
WSLg ( x86_64 ): 1.0.26+Branch.main.Sha.26ce2c09b86442f3c7f4f6462f770ed2afa76a25
Mariner: VERSION="1.0.20210224"
FreeRDP: b05321cd4e6a862aef76163a69db4e1910245736
weston: 46756d0e77e5c01b5995fbbee6f3ab0db9b30612
pulseaudio: 2f0f0b8c3872780f15e275fc12899f4564f01bd5
mesa:

Expected behavior
Chrome window should be visible
Actual behavior
Chrome window is not showing. There is only button on taskbar.

This happens to me after using it for a while too - first the application I am using freezes (for example Dbeaver-CE) then I am not able to start new GUI apps, they will show on taskbar but not render anything

How can I disable WSLg instalation ? I removed it but it get installed after few days by Windows Update.


Issues while getting interaction data

Hi,
While fetching data from the azure blob storage i'm getting an error that the connection is being forcibly closed

@baibhav-82 Please provide a documentation link so I can assign the issue to the appropriate writer. What article/topic were you following when you ran into this issue?

@phecke https://docs.microsoft.com/en-us/dynamics365/marketing/custom-analytics#set-up-azure-blob-storage-and-connect-it-to-marketing
This is what i'm following, and the issue i'm getting is in step 9:The Marketing analytics configuration page opens and on the reports

Thanks @alfergus

Hi @baibhav-82 -- after some digging, this is likely a timeout in blob storage. To get this resolved as quickly as possible, I suggest raising a customer support request. The customer support team should have more familiarity with the blob issue and will be able to help get it squared away.

Hi @alfergus , i had raised a customer support request but it was not provided because this feature is still in preview

@baibhav-82 I'm sorry to hear you're having issues with that. They are correct -- customer support does not handle preview features. I'm going to dig into this a bit more for you though and see if I can find anything.

Thanks @alfergus

Hey @baibhav-82 - sounds like we'll need to get a specialist on this. You will be able to get help through the Power Platform admin center. To create a support request through the admin center, go to the Get Help + Support - Power Platform Admin center article.


Introduce measurement count tables

Analysis scripts and the private API often use count() on measurements.
Investigate introducing tables that contains msm counts or extend the ones named ooexpl* to improve speed.

We have migrated everything to it's own issue. Closing this as done.


CoreML support?

How can we add CoreML support? Thx!

Whisper.cpp now has CoreML support:
https://github.com/ggerganov/whisper.cpp/pull/566
Using just whisper.cpp, should be as simple as compiling with the appropriate flags:
cd build
cmake -DWHISPER_COREML=1 ..

Check by running:
./main -m models/ggml-base.en.bin -f samples/gb0.wav

...

whisper_init_state: loading Core ML model from 'models/ggml-base.en-encoder.mlmodelc'
whisper_init_state: first run on a device may take a while ...
whisper_init_state: Core ML model loaded

system_info: n_threads = 4 / 10 | AVX = 0 | AVX2 = 0 | AVX512 = 0 | FMA = 0 | NEON = 1 | ARM_FMA = 1 | F16C = 0 | FP16_VA = 1 | WASM_SIMD = 0 | BLAS = 1 | SSE3 = 0 | VSX = 0 | COREML = 1 |

...

note: COREML = 1.
For whispercpp.py, we can add the flag for CoreML here inside setup.py:
if sys.platform == 'darwin':
    os.environ['CFLAGS']   = '-DWHISPER_COREML=1 -DGGML_USE_ACCELERATE -O3 -std=gnu11'
    os.environ['CXXFLAGS'] = '-DWHISPER_COREML=1 -DGGML_USE_ACCELERATE -O3 -std=c++11'
    os.environ['LDFLAGS']  = '-framework Accelerate'

First update the submodule inside whispercpp.py for whisper.cpp. Check that it still runs, it might need some changes if the API has changed. Given it still works, add the flag inside setup.py.
I can't test this at the moment, but feel free to make the pull request, and we can get this feature added.

Thx!
I decided to use OpenAI's Whisper API for the current moment, in my tests it beats the local Whisper with CoreML by 3-4 times (comparing to Macbook M1 32GB)

Whisper.cpp now has CoreML support:
ggerganov/whisper.cpp#566
Using just with whisper.cpp, should be as simple as compiling with the appropriate flags:
cd build
cmake -DWHISPER_COREML=1 ..

Check by running:
./main -m models/ggml-base.en.bin -f samples/gb0.wav

...

whisper_init_state: loading Core ML model from 'models/ggml-base.en-encoder.mlmodelc'
whisper_init_state: first run on a device may take a while ...
whisper_init_state: Core ML model loaded

system_info: n_threads = 4 / 10 | AVX = 0 | AVX2 = 0 | AVX512 = 0 | FMA = 0 | NEON = 1 | ARM_FMA = 1 | F16C = 0 | FP16_VA = 1 | WASM_SIMD = 0 | BLAS = 1 | SSE3 = 0 | VSX = 0 | COREML = 1 |

...

note: COREML = 1.
For whispercpp.py, we can add the flag for CoreML here inside setup.py:
if sys.platform == 'darwin':
    os.environ['CFLAGS']   = '-DWHISPER_COREML=1 -DGGML_USE_ACCELERATE -O3 -std=gnu11'
    os.environ['CXXFLAGS'] = '-DWHISPER_COREML=1 -DGGML_USE_ACCELERATE -O3 -std=c++11'
    os.environ['LDFLAGS']  = '-framework Accelerate'

First update the submodule inside whispercpp.py for whisper.cpp. Check that it still runs, it might need some changes if the API has changed. Given it still works, add the flag inside setup.py.
I can't test this at the moment, but feel free to make the pull request, and we can get this feature added.
@stlukey

I have verified that my computer is an M2. I found that CoreML does not seem to be enabled through this command.
I also added this compile flag, which also does not seem to work:
if sys.platform == 'darwin':
    print("run here.....")
    os.environ['CFLAGS'] = '-DWHISPER_COREML=1 -DGGML_USE_ACCELERATE -O3 -std=gnu11'
    os.environ['CXXFLAGS'] = '-DWHISPER_COREML=1 -DGGML_USE_ACCELERATE -O3 -std=c++11'
    os.environ['LDFLAGS'] = '-framework Accelerate'

That is, I added -DWHISPER_COREML=1 and used the latest whisper.cpp code. When I run the generated whisper.xxxx.so file to transcribe the same audio, it takes 12 minutes. But if I compile whisper.cpp with the same commit using cmake -DWHISPER_COREML=1 and run ./main on the same audio, it only takes 7 minutes. Also, I can see the loading process has:
whisper_init_state: loading Core ML model from 'models/ggml-large-encoder.mlmodelc'
whisper_init_state: first run on a device may take a while ...
whisper_init_state: Core ML model loaded
system_info: n_threads = 4 / 12 | AVX = 0 | AVX2 = 0 | AVX512 = 0 | FMA = 0 | NEON = 1 | ARM_FMA = 1 | F16C = 0 | FP16_VA = 1 | WASM_SIMD = 0 | BLAS = 1 | SSE3 = 0 | VSX = 0 | COREML = 1 | OPENVINO = 0 |

It loading the Core ML model. I expected the .so call can also do this. How can I modify it?


Improve wget update command

Improves the changes done in #3715 by adding space after wget. It also removed unused build.sh which was a leftover post #3613
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[x] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #3723 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3723   +/-   ##
=======================================
  Coverage   75.05%   75.05%
=======================================
  Files         127      127
  Lines        6041     6041
  Branches      391      391
=======================================
  Hits         4534     4534
  Misses       1507     1507

Continue to review full report at Codecov.

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


Add Span support

System.Random has it since .NET Standard 2.1.

Thanks @kasthack, I will release a new version today.
Bye,
Alessio


Some WARN after upgrading alpha 30, to make sure it's normal

Hello,
After upgrading alpha 30 I have a lot of warn that look impacting:
Aug 03 10:47:53.124 INFO uplink @3374689388 us, 868.30 MHz, Ok(DataRate(SF9, BW125)), snr: -4.3, rssi: -114, len: 23 from AC:1F:09:FF:FE:05:22:28, module: gateway
Aug 03 10:47:53.615 WARN ignoring failed uplink Service(Rpc(Status { code: Unknown, message: "no response", metadata: MetadataMap { headers: {"user-agent": "grpc-erlang/0.1.0", "content-type": "application/grpc+proto", "grpc-encoding": "identity"} }, source: None })), oui: 9, uri: http://<IP_ADDRESS>:8080/, pubkey: 11w77YQLhgUt8HUJrMtntGGr97RyXmot1ofs5Ct2ELTmbFoYsQa, module: router

This is printed on every frame, may be related to one of the router 9 not responding. But i don't see why this frame should be sent to router 9 (not looking like a JOIN
I also have this, only concerning certain frames, can come from non LoRaWan frame. Just for your info if it helps to manage some specific situations :
Aug 03 10:47:20.011 INFO uplink @3341566044 us, 867.70 MHz, Ok(DataRate(SF11, BW125)), snr: -11.5, rssi: -122, len: 20 from AC:1F:09:FF:FE:05:22:28, module: gateway
Aug 03 10:47:20.983 WARN ignoring failed uplink Service(Rpc(Status { code: Unknown, message: "no response", metadata: MetadataMap { headers: {"user-agent": "grpc-erlang/0.1.0", "content-type": "application/grpc+proto", "grpc-encoding": "identity"} }, source: None })), oui: 9, uri: http://<IP_ADDRESS>:8080/, pubkey: 11w77YQLhgUt8HUJrMtntGGr97RyXmot1ofs5Ct2ELTmbFoYsQa, module: router
Aug 03 10:47:21.029 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 9))), module: gateway
Aug 03 10:47:21.746 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 9))), module: gateway
Aug 03 10:47:24.498 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 8))), module: gateway
Aug 03 10:47:24.985 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 8))), module: gateway
Aug 03 10:47:26.796 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 9))), module: gateway

Thanks for the report!

Hello, After upgrading alpha 30 I have a lot of warn that look impacting:
Aug 03 10:47:53.124 INFO uplink @3374689388 us, 868.30 MHz, Ok(DataRate(SF9, BW125)), snr: -4.3, rssi: -114, len: 23 from AC:1F:09:FF:FE:05:22:28, module: gateway
Aug 03 10:47:53.615 WARN ignoring failed uplink Service(Rpc(Status { code: Unknown, message: "no response", metadata: MetadataMap { headers: {"user-agent": "grpc-erlang/0.1.0", "content-type": "application/grpc+proto", "grpc-encoding": "identity"} }, source: None })), oui: 9, uri: http://<IP_ADDRESS>:8080/, pubkey: 11w77YQLhgUt8HUJrMtntGGr97RyXmot1ofs5Ct2ELTmbFoYsQa, module: router

This is printed on every frame, may be related to one of the router 9 not responding. But i don't see why this frame should be sent to router 9 (not looking like a JOIN

gateway-rs doesn't know about join or not join.. It just knows to send packets to a known on chain router, or if no such router exists to to go the (default) roaming routers today. Those routers may decline that packet if the device is not found or if they're not interested in the packet. Which I believe  is the no response error you're seeing.

I also have this, only concerning certain frames, can come from non LoRaWan frame. Just for your info if it helps to manage some specific situations :
Aug 03 10:47:20.011 INFO uplink @3341566044 us, 867.70 MHz, Ok(DataRate(SF11, BW125)), snr: -11.5, rssi: -122, len: 20 from AC:1F:09:FF:FE:05:22:28, module: gateway
Aug 03 10:47:20.983 WARN ignoring failed uplink Service(Rpc(Status { code: Unknown, message: "no response", metadata: MetadataMap { headers: {"user-agent": "grpc-erlang/0.1.0", "content-type": "application/grpc+proto", "grpc-encoding": "identity"} }, source: None })), oui: 9, uri: http://<IP_ADDRESS>:8080/, pubkey: 11w77YQLhgUt8HUJrMtntGGr97RyXmot1ofs5Ct2ELTmbFoYsQa, module: router
Aug 03 10:47:21.029 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 9))), module: gateway
Aug 03 10:47:21.746 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 9))), module: gateway
Aug 03 10:47:24.498 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 8))), module: gateway
Aug 03 10:47:24.985 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 8))), module: gateway
Aug 03 10:47:26.796 WARN ignoring push_data: Decode(LoraWan(InvalidPacketSize(JoinRequest, 9))), module: gateway

Those warning for ignored push_data are indeed the gateway dropping invalid frames on the floor. Are you saying the "ignoring" message is not clear enough?


Unsupported type : Mediaplayer (Squeezebox)

Hardest part will be to design the interface. Probably won't implement a full remote as I want to keep the widgets tiny. Maybe just play pause?
Example output for Squeezebox:
{"Battery":255,"RSSI":12,"description":"","dtype":"Lighting 2","id":"0000001","idx":80,"name":"Squeezebox Radio","nvalue":1,"stype":"AC","svalue1":"0","switchType":"Media Player","unit":1}

No documentation on domoticz wiki, found these by watching requests from the original dashboard.
Turn on/off:
json.htm?type=command&param=switchlight&idx=80&switchcmd=On&level=0&passcode=
json.htm?type=command&param=switchlight&idx=80&switchcmd=Off&level=0&passcode=

From the web remote:
json.htm?type=command&param=lmsmediacommand&idx=80&action=Play

Actions : Play, Stop, Pause, Forward, Rewind, VolumeUp, VolumeDown, Mute, Shuffle, Repeat
Other action not doing anything on my Squeezebox Radio: Favorites, Browse, NowPlaying, Up, Down, Left, Right

Pull #41 fixing #38 also added On/Off support for Squeezebox here.
I still want to add play/pause eventually but that's good enough for now.

This issue was moved to domoticz/Reacticz#1


Item Width Variation Issue

AS you can see in this picture, 2-digit number and 3-digit number have the same width.
Is there any way that I can use it with different width so that 2-digit number and 3-digit number have the same spacing between.

Using different items' widths is not allowed at the moment as this would be really tricky to handle, but this is something I've considered adding in the future.
For now, you might be able to hack your way around by using letterSpacing.

@erickreutz Unfortunately, this would prove incredibly complex with the plugin's current logic.
I'm working on replacing the FlatList component altogether; this will maybe help with this request.


Re-do backend in Sails.js or actually learn Express

Current backend is either a template or just following steps from a tutorial. Either switch to Sails and learn as you go or actually learn Express. Either way do something useful.

Just learned Sails is built on top of Express. Screw magic, learn Express.

Turns out Express is pretty straightforward.


fix: crash when there are no bdevs and lvm export

fix: crash when there are no bdevs

When there are no bdevs the iterator will return nothing.
In this case we should not crash.

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

fix(lvm): don't export if not enabled

There's no point trying and erroring with message..

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

bors merge


drivers/exoscale: fix configuration of exoscale endpoint

The environment variable was EXOSCALE_ENDPOINT but the flag was
--exoscale-url. When configuring from flags, the wrong name was
used. Therefore, a user was unable to use an alternate endpoint.
Signed-off-by: Vincent Bernat<EMAIL_ADDRESS>

LGTM


up simple chinese translation

Sorry to bother you,
I don’t know what your willingness is for localized software. I have uploaded a translation into Simplified Chinese. If you are interested, I can also add a translation into Traditional Chinese.
if author is not interested in localized software. Can you also tell me how to build winlator software? Because git cloned it locally and then opened android studio, I found that it kept prompting the error as shown below. I also tried the prompts and tried it. I felt like it was some files missing.

Add this line on line 5 of build.gradle
namespace 'com.winlator'

Don't remove English texts add Chinese translation in values-zh or values-zh-rTW folder in strings.xml

res/
  ├── values/
  ├── values-zh/          # For Simplified Chinese
  └── values-zh-rTW/      # For Traditional Chinese

The correct way to add localization is to create a resource folder for the corresponding language. https://developer.android.google.cn/guide/topics/resources/localization?hl=zh-cn

Thanks, I'll submit the correct localization files right away

I've placed the translation in the correct location as instructed.
I apologize, but I'm not very proficient in Android programming or Android Studio. I believe I encountered an error: 'Constant expression required' when compiling the main branch using Winlator. Here are the detailed error messages:
error.log

I directly cloned the main branch of winlator, but Android Studio consistently failed to recognize it as an Android project. As a workaround, I created a new project using the native C++ template, deleted the app directory to replace it with the original winlator directory, and finally, the project was recognized. I also added the namespace com.winlator on line 5. After this commit, I don't believe there was an error in the original main branch.

Add these files in the project with its contents.
and also do git submodule init and git submodule update in project's app folder and then compile the project.
also only add xml files in values-zh/ which has English texts you want to translate colors.xml and attrs.xml values will be taken from normal values folder.

Thank you, I'm able to compile it normally now, and the translation works out of the box. However, there are too many meaningless commits in this branch, and I'm still working on adding CJK support to this software for WINE+Linux environments. I've found that simply installing fonts doesn't solve the problem. When opening Chinese, Japanese, or Korean software, the windows are filled with '???' or garbled characters, and there's a high chance of garbled text within the software or games.
I'll close this PR and resubmit a final PR when I'm finished.
I'm sorry to trouble you and I hope you have a great day every day.


DeleteState context regex matching

Proposal
It's possible to create context names based on input parameters and use them this way.
When deleting contexts, you need to pass each context name for deletion. It'll be nice to have a feature of context name matching using regex. For example:
"serveEventListeners": [
		{
			"name": "deleteState",
			"parameters": {
				"context": "resource-xyz-.*"
			}
		}
	]

References
No response

should this delete all contexts matching the regex?

should this delete all contexts matching the regex?

Yes.
e.g. two contexts
resourceXyz-uuid1
resourceXyz-uuid2
deleteState for context "resourceXyz-.*" will delete both contexts.
What do you think about this feature?

sounds good. Was wondering whether it should be more explicit:
"serveEventListeners": [
		{
			"name": "deleteState",
			"parameters": {
				"contextsMatching": "resource-xyz-.*"
			}
		}
	]

sounds good. Was wondering whether it should be more explicit:
"serveEventListeners": [
		{
			"name": "deleteState",
			"parameters": {
				"contextsMatching": "resource-xyz-.*"
			}
		}
	]

Good, explicit approach! I like it.

@dragutint can you check #98 (for the feature and also the code if you like)?

PR is merged. In case you miss something of find a bug: feedback is very welcome.


fix typo in README.md

What kind of change does this PR introduce?
No bugs, just a corrected typo in the readme.md
What is the current behavior?

What is the new behavior?

What might this PR break?
Nothng
Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

Thankyou!


Groovy syntax on GitHub

You can get GitHub to play nicely with the NextFlow scripts file extensions with these two tricks:
Add the following to the top of every .nf script: (does syntax highlighting)
vim: syntax=groovy
-*- mode: groovy;-*-

Create a file called .gitattributes with the following: (changes the coloured bar at the top of the repo to say 100% Groovy instead of 100% Shell.
*.nf linguist-language=Groovy

Hope this helps! It was annoying me on our NextFlow repos 😉

Nice! I'm bit confused with the first tip. It does not break the nextflow script if you put that at the top of the file?

Sorry, just near the top of the file. eg: https://github.com/SciLifeLab/NGI-RNAseq/blob/master/main.nf#L3-L4

I take it back - they can go anywhere in the file. See https://github.com/github/linguist#using-emacs-or-vim-modelines

Cool! So in a comment like this:
/*
vim: syntax=groovy
-*- mode: groovy;-*-
*/

Exactly, yup 👍


Should I resubmit a Chinese translation?

Hello,
In 2019, I translated the BOT chat from English to Chinese. issu
When I re-downloaded yapb recently, I found that there was only chs_chat.cfg, and I did not see the relevant Chinese translation in yapb/cfg/addons/yapb/conf/lang/ of the master branch, I guess there must be a reason for this.
I re-translated it, hope it can help you.
chs_chat_02.cfg.txt

Hello! Yes, sure. It was totally outdated, so was removed, as there was no translators  who can do the work.

Hello! Yes, sure. It was totally outdated, so was removed, as there was no translators who can do the work.

oh! I'm glad I helped you!
If you have chs (simplified Chinese) and cht (traditional Chinese) needs, you can @HaoJun0823 me next time and I will help you!


Update request: kmonad 0.4.1 → 0.4.2

Package name: haskellPackages.kmonad
Latest released version: 0.4.2

Current version on the unstable channel: 0.4.1
Current version on the stable/release channel: 0.4.1

[x] Checked the nixpkgs pull requests

Recent update might be able to fix the broken status
https://github.com/NixOS/nixpkgs/issues/224190
Notify maintainers

Note for maintainers: Please tag this issue in your PR.

@jfvillablanca Done in https://github.com/NixOS/nixpkgs/pull/267450 :)


Add initial support for OpenHarmony

This adds support all the surfman APIs servo seems to be requiring on OpenHarmony.
Generic surfaces are not supported at the moment.
This patch is heavily based on the existing android implementation.

Could it just reuse the same Android implementation and add #[cfg(ohos)] when it's necessary?

Could it just reuse the same Android implementation and add #[cfg(ohos)] when it's necessary?

Hmm, perhaps the common parts could be extracted to a shared egl implementation.

@mrobinson Do you have any opinions regarding this PR and https://github.com/servo/surfman/pull/290? Would you prefer if the android and ohos implementations would be merged into one directory?

@mrobinson Do you have any opinions regarding this PR and #290? Would you prefer if the android and ohos implementations would be merged into one directory?

Sorry for taking so long to look at this. I think that if both platforms use EGL and they have a lot of shared code it probably makes sense to make a shard EGL backend, as you say.

Superceded by #293


SPIFFE/SPIRE to incubating

Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?
[ ] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Signed-off-by: Amye Scavarda Perrin<EMAIL_ADDRESS>

Deploy preview for landscape ready!
Built with commit 2c529554f2ce7d96738a5ca417709078cb174a2f
https://deploy-preview-1726--landscape.netlify.app


Kafka Connect HDFS/Hive

Hi, great Docker image btw!
I successfully made it work with HDFS/Hive, and I wanted to share the experience. Most of the issues I think are actually related to Confluent, more than Landoop, but I like this packaging better and I hope hdfs will be supported out-of-the-box soon :)
First, my config:

HDFS/Hive in HA
format.class=io.confluent.connect.hdfs.parquet.ParquetFormat

Here are the issues I had to solve:

class path order -- stream-reactor has java lib versions which are incompatible with confluent connect 3.0.1. Depending who is loaded first, you can have connect hdfs working or not (and, in my case, if I turn on hdfs then I break stream-reactor, e.g. cassandra is no longer working).
Temp solution:
ln -s /opt/confluent-3.0.1/share/java/kafka-connect-hdfs /opt/confluent-3.0.1/share/java/kafka-connect-0hdfs
(so kafka-connect-hdfs is included first in the cp)
Parquet by default compresses with Snappy. Snappy lib requires libstdc++ (to be installed via apk)
for HA to work, /etc/hadoop/conf needs to be available to the container. Moreover, Connect v3.0.1 has an issue with Parquet and HA:
https://github.com/confluentinc/kafka-connect-hdfs/commit/176e7d1ba114e496c6bc52c7fa72d198c9aa103f
finally, there are several improvement to connect post v3.0.1 that I think are pretty fundamental, for instance this one:
https://github.com/confluentinc/kafka-connect-hdfs/commit/38aaded6b00801f8c4df4de91d3ccea1598d2e1f

I'm maintaining a branch with the minimal changes to v3.0.1 that makes kafka-connect-hdfs working in my settings here:
https://github.com/shopkick/kafka-connect-hdfs/tree/sk_v3.0.1

Thank you for sharing this. It is very interesting and also encouraging!
You already solved everything, but here are some thoughts about your issues:

Earlier today we pushed a new version with the latest stream-reactor (build from
master) which adds support for CP 3.0.1 and many other fixes as well; it may help.
Nonetheless, since you maintain a custom hdfs connector version, something you could
try, is to mount your version under /connectors/. I think that way it may work without the ln trick:

  docker run -v /path/to/custom/hdfs-connect/jars:/connectors/ landoop/fast-data-dev

Thanks! We will add libstdc++ in the next commit. We haven't tested fast-data-dev with hdfs.
We do write some connector tests
now using docker-compose but I avoided HDFS because I am not eager to set it up in
a container and there are some complex use cases to test (i.e hdfs/hive).
Yes, you are correct. Same goes for most hadoop software, for hbase for example you
would have to add /etc/hbase/conf/hbase-site.xml and possibly other files in this
directory.
I admit I didn't know about the issue you mention. We do some simple HDFS tests on our CDH (without fast-data-dev) but do not use it otherwise.
This looks interesting. If I wanted to give it a spin with CP 3.0.1, is there a
branch  I can follow or just the specific commit you linked to?

I'll definitely check the new version.
On point 4., the branch is here:
https://github.com/shopkick/kafka-connect-hdfs/tree/sk_v3.0.1
It's from confluent v3.0.1 tag, with the 2 commits and some other related/necessary commits cherry picked. It's the minimal set I was able to make work.

I did a quick test docker pulling your new image and rebuilding mine (same link trick).
When I run cassandra connector, I still get:
[2016-10-20 17:17:41,835] ERROR Task cassandra-sink-0 threw an uncaught and unrecoverable exception (org.apache.kafka.connect.runtime.WorkerTask:142)
java.lang.ExceptionInInitializerError
        at com.datamountaineer.streamreactor.connect.cassandra.CassandraConnection$.getCluster(CassandraConnection.scala:43)
        at com.datamountaineer.streamreactor.connect.cassandra.CassandraConnection$.apply(CassandraConnection.scala:33)
        at com.datamountaineer.streamreactor.connect.cassandra.sink.CassandraWriter$$anonfun$1.apply(CassandraWriter.scala:33)
        at com.datamountaineer.streamreactor.connect.cassandra.sink.CassandraWriter$$anonfun$1.apply(CassandraWriter.scala:33)
        at scala.util.Try$.apply(Try.scala:192)
        at com.datamountaineer.streamreactor.connect.cassandra.sink.CassandraWriter$.apply(CassandraWriter.scala:33)
        at com.datamountaineer.streamreactor.connect.cassandra.sink.CassandraSinkTask.start(CassandraSinkTask.scala:78)
        at org.apache.kafka.connect.runtime.WorkerSinkTask.initializeAndStart(WorkerSinkTask.java:207)
        at org.apache.kafka.connect.runtime.WorkerSinkTask.execute(WorkerSinkTask.java:139)
        at org.apache.kafka.connect.runtime.WorkerTask.doRun(WorkerTask.java:140)
        at org.apache.kafka.connect.runtime.WorkerTask.run(WorkerTask.java:175)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
        at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.IllegalStateException: Detected Guava issue #1635 which indicates that a version of Guava less than 16.01 is in use.  This introduces codec resolution iss
ues and potentially other incompatibility issues in the driver.  Please upgrade to Guava 16.01 or later.
        at com.datastax.driver.core.SanityChecks.checkGuava(SanityChecks.java:62)
        at com.datastax.driver.core.SanityChecks.check(SanityChecks.java:36)
        at com.datastax.driver.core.Cluster.<clinit>(Cluster.java:67)
        ... 16 more

(Today I'm not sure I'll have much time to work on this, will retry tomorrow without the symlinks)

I will close the issue as it is too old and we have created our kafka-connectors-tests  repository where we perform some basic integration tests (inside fast-data-dev) for most connectors.
You can use it as an example on how to setup various connectors. For example if you still have issues with the Cassandra connector (hope not, after so many months :p), you could take a look at: https://github.com/Landoop/kafka-connectors-tests/tree/master/kafka-connect-cassandra
Thanks again for participating!


dont's uninstall mco if failed to checkmcoaddon in install step

https://github.com/open-cluster-management/backlog/issues/13875
Signed-off-by: Marco<EMAIL_ADDRESS>

/lgtm


I ran the mini_imagenet_generator.py and only the floder tiered_imagenet generated

Thank you for sharing such great work. I'm new to it and it helps me a lot. I ran the tiered_imagenet_generator.py and only the floder tiered_imagenet generated. This folder contains trian, val, and test. Further, the processed_images directory is not created. Can I sample images directly from each of the three folders in  tiered_imagenet directory to generate the NwayKshot task for train,val,test ? In my code, the MiniImagenet dataset also takes this form.

Hi,
Thanks for your interest in our work!
This project will not directly generate N-way K-shot tasks. You may use a dataloader like this one to load images from train, val, and test folders to generate tasks during meta-training and meta-testing.

Thank you very much for your timely reply！Sorry to bother you again, I ran the tiered_imagenet_generator.py and only the floder tiered_imagenet generated(about 4G), Can I use dataloader you said directly on this folder generate N-way K-shot tasks in my project? But the processed tieredimagenet datasets you provide is about 18G.

Hi,
When running the project, the tasks will only be generated when they are used, and they won't be stored on the disk.
The whole tieredImageNet is about 19.4G. Maybe the dataset you generated is not completed. You may directly download tireredImageNet with this link.

I got it. Thank you for your generous help!

@yaoyao-liu Sorry to bother you again, I am using miniImagenet and Tieredimagenet data sets to perform pre-training in few shot learning algorithm(for example,MAML). I saw your other excellent article about the miniImagenet dataset pre-training process. For instance, there are totally 64 classes in the training split of miniimagenet and each class contains 600 samples used to pre-train a 64-class classifier,then the feature extractor  is learned. In the next meta-train process, fixed feature Extractor only updates the weight of the last full connection layer. There are some problems with both pre-training and meta-train using the same 'train' in miniImagenet?

I don't fully understand your question. Could you please make it more specific?
If you wonder why we use the same samples for pre-training and meta-training phase, the answer is as follows,
It is because of the benchmark we use. If we use miniImageNet, we are not allowed to use other samples to train the model except for the samples in the meta-train set of miniImageNet. In real-world applications, you can pre-train the model on a larger dataset, e.g., ImageNet and than meta-train the model on your own meta-train set.

Thank you for your quick reply！I don't know if I'm right. In my opinion,if use the same samples for pre-training and meta-training phase, think of pre-training as a special case of NwayKshot,(N=64, K=600), feature extractor obtained in this way is definitely better(because more way and more shot). It's like adding more samples artificially. Is it against the original purpose(few shot)?And in meta-train,the sampled images have been used in pre-training. Is that not a very rigorous way?

All the meta-learning-based methods for few-shot classification are based on transfer learning. That means we can use all the samples and their labels for the source domain (meta-train set). ''Few-shot'' means we only have a few labeled data for the target domain (meta-test set). So it is not against the original purpose for few-shot classification. The original purpose of meta-learning-based methods is using many-shot labeled data (e.g. ImageNet) to improve the performance for novel class data with a few labeled samples (e.g. medical images).
We use the same samples for pre-training and meta-training because we're only allowed to access limited data and we need to make most of them. We won't do the same thing in read-world applications.
I agree that the current benchmark for few-shot classification is not optimal. But if you hope to compare your results with all previous papers, you need to follow that. Of course, you can propose your own benchmark protocol, but you need to re-implement the previous methods in the same protocol for fair comparison.

Thank you for your patient reply! That was a wonderful explanation! I got it. Then, During each iteration of pre-training, shall we input all images in all categories into the network at once or sampling a certain number of images from all images produces a batch?

For the details of the pre-training phase, you may follow this repo.


⚠ Samsung has degraded performance

In ddaf9e2, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2931 ms

Resolved: Samsung performance has improved in bae283f.


Upgrade Steamworks.NET to version 15.0.1

Steamworks.NET version 15.0.0 introduced IL2CPP support, updated the Steamworks SDK to 1.51 and includes several other fixes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Liam Cary seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Added the email address used in the commit to my GitHub account and signed the CLA. Should be ok now?

Yes, CLA looks all good now. Hope I'll get some time to review this soon.


get_sqlite_version() reports old version

Hello @vladkens ,
I have to comment out raising the SystemError on check_version() constantly because even though I have a compatible newer sqlite installation, this still triggers every time. Any ideea? On manually checking what version gets imported it's always the new one and also everything works as expected without any issues.
Is by posibility the function loading the system-wide install of sqlite3 instead of the one from the venv in which it is running?
Also, unrelated. Is a port to mysql (or option) on the to do list?

Hi. Python comes with some sqlite version in the package and its very difficult to update its version.
If you are doing manual check from python program and version is different than twscrape shows then send example code please.
If you are checking the sqlite version in the system, it may be different. Python will not use it.

Fixed since 0.7.0


Inshorts Clone

** Its a news paper Application . **
** We can read newspaper with updated news using news api**

please assign me this issue under GSSoC'23.


update kuadrant core controller manifests

integration test fixed in https://github.com/Kuadrant/kuadrant-operator/pull/45


STN correct clock

Hello, I only used full+++_st.pth to correct the clock, the effect is not very good. Is something wrong or is full+++_st.pth not the optimal weight?  if its the optimal weight, and could u tell me how many epochs u choose? thank u very much!!!

Hi, sorry for late response.
STN is far from perfect (as it's only trained on synthetic data). It only makes clock a bit easier to read, but doesn't align entirely.


Is a generated standalone parser really standalone?

When I open a generated stand-alone parser, VSCode marks quite a few identifiers as unknown, e.g., .lexer, InteractiveParser, _serialize, Enumerator, etc.

Are they somehow dynamically resolved during execution?

Thanks for the explanation. Indeed, I have not encountered any case in which this dead code has caused problems.
One more nitpick though: I initially thought that the issue was solved (since you closed it as "completed", instead of "not planned"), but the file generated by Lark v1.1.7 still presents the same issue.
PS: I don't know about InteractiveParser, but I can confirm that PyLance doesn't know how to resolve it.


Push to npm

Hi!
I know it's possible to install the package from Github, but obviously npm is more common =)

maybe someone should fork it and push?

Already available on npm : https://www.npmjs.com/package/@meetfranz/electron-notification-state

@moocat in what way is it outdated? There is only one commit here.