add automation to submit PRs to update upon cosign releases

Currently, when we have a new version of cosign, we need to manually update for our users:

sigstore/cosign-installer
sigstore/homebrew-tap
bazel-contrib/rules_oci (see https://github.com/bazel-contrib/rules_oci/pull/82/files)
actions/starter-workflows

It would be nice to have creation of a tag on the cosign repo trigger some automation that would automatically submit the correct PRs to the above list of repos.
Thoughts @cpanato?

for sigstore/homebrew-tap we can add in the goreleaser
the other will require some automation but I think nothing is hard
i will work on that


Soft delete Admin

Laravel Version: 5.5.*
Voyager Version: the last one
PHP Version:7.0.22-0ubuntu<IP_ADDRESS> (cli) ( NTS )
Database Driver & Version: mysql

Description:
Hi thank you it was really simple to get the soft delete working
the problem is that i cannot load the deleted items in the admin panel!!
i need to restore some items or edit them!!

as u can see the table is empty but in the DB i already have some deleted products!

Voyager doesn't add withTrashed() to it's queries.  We need to add a UI element to allow toggling that.  I'm open for Pull Requests if anyone else wants to take a stab at it.

I can jump on this.
I'm thinking of a "View Deleted" bootstrap toggle near the existing Add New / Bulk Delete / Order buttons at the top of the browse view. This would appear only for models that use SoftDeletes. Clicking the toggle will redirect the user to ?withTrashed or similar and reload the page. When the page loads, if the toggle is on, the controller query will use ->withTrashed(), if it is off it will not.
Should I branch off of 1.1?

Fixed in https://github.com/the-control-group/voyager/pull/3758, merged into 1.x


Allow creation of custom EC profiles during deployment

Is your feature request related to a problem? Please describe.
I want to have custom EC profiles setup on a new Ceph cluster
Describe the solution you'd like
I would like to be able to setup customized EC profiles when running deployment with ceph-ansible.
Describe alternatives you've considered
I could do in a post deployment playbook but since we can already define "simple rules" in ceph-ansible it would make sense to define these here as well.

Hello,
will this feature be available in the 4.0.0 release ? Thanks.
Best regards,
Yoann


AjaxFileUploadHandler.axd not found in subsite

The AJAX Control Toolkit \<IP_ADDRESS>>
The installation method

[X] Installer
[X] NuGet package
[ ] A custom build from the source code

Minimal steps to reproduce the bug
Create subsite of Default Web Site in IIS
Add ajaxfileupload control to a usercontrol on a page in the subsite
Actual result
The AjaxFileUploadHandler.axd will be search in the root site, and it won't be found there, because it lives in the subsite.
Logging from IIS:
... POST /AjaxFileUploadHandler.axd contextKey=.... 404 0 2 8
Expected result
The AjaxFileUploadHandler.axd should be found at the level where is has been added.
Browser(s) used
Internet Explorer 11
A site deployment method

[ ] VS development web-server,
[X] IIS

Hi @EdwinRoestenburg ,
AjaxFileUpload does not detect the site structure, so you need to specify the upload handler path explicitly using the UploadHandlerPath property:
<ajaxToolkit:AjaxFileUpload
    runat="server"
    UploadHandlerPath="MyApplication/AjaxFileUploadHandler.axd" />

I will give that a go, but that would mean that I would have to configure these settings for different environments. Depending on where I publish to, the application may or may not live in a subsite.
For now I reverted to the last 1.15 release, where this is not an issue.
Edwin.
From: MikhailTymchukDX<EMAIL_ADDRESS>Sent: 28 September 2016 10:31
To: DevExpress/AjaxControlToolkit<EMAIL_ADDRESS>Cc: EdwinRoestenburg<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [DevExpress/AjaxControlToolkit] AjaxFileUploadHandler.axd not found in subsite (#231)
Hi @EdwinRoestenburg https://github.com/EdwinRoestenburg  ,
AjaxFileUpload does not detect the site structure, so you need to specify the upload handler path explicitly using the UploadHandlerPath property:
<ajaxToolkit:AjaxFileUpload
runat="server"
UploadHandlerPath="MyApplication/AjaxFileUploadHandler.axd" />
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/DevExpress/AjaxControlToolkit/issues/231#issuecomment-250104781 , or mute the thread https://github.com/notifications/unsubscribe-auth/ARdWP6FfOkCi8Xd2CBjwtd_ptpD2E61Oks5quiW1gaJpZM4KHr0e .  https://github.com/notifications/beacon/ARdWP6u_f3EFDHbPCYQQXg1Imnp_jSW6ks5quiW1gaJpZM4KHr0e.gif

Would you please clarify what is the 1.15 release, so we can check it and see the difference with the current release?

Sorry :( , I was too quick here. I meant release 15.1.4
E.
From: MikhailTymchukDX<EMAIL_ADDRESS>Sent: 28 September 2016 13:38
To: DevExpress/AjaxControlToolkit<EMAIL_ADDRESS>Cc: EdwinRoestenburg<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [DevExpress/AjaxControlToolkit] AjaxFileUploadHandler.axd not found in subsite (#231)
Would you please clarify what is the 1.15 release, so we can check it and see the difference with the current release?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/DevExpress/AjaxControlToolkit/issues/231#issuecomment-250142815 , or mute the thread https://github.com/notifications/unsubscribe-auth/ARdWP9fhUQKUwOPhpx4rursHj2HS7LOcks5qulGQgaJpZM4KHr0e .  https://github.com/notifications/beacon/ARdWP9azf65Wi_ObS-N6iVauKn-4mIzOks5qulGQgaJpZM4KHr0e.gif


[BUG] failed to run sealer  alpha mount

What happen?
sealer alpha mount my-app:v1

Relevant log output?
No response
What you expected to happen?
No response
How to reproduce it (as minimally and precisely as possible)?
No response
Anything else we need to know?
No response
What is the version of Sealer you using?
sealer 0.9.1
What is your OS environment?
No response
What is the Kernel version?
No response
Other environment you want to tell us?

Cloud provider or hardware configuration:
Install tools:
Others:

BTW: How do we handle multi-architecture images, my suggestion is to choose the corresponding image to mount according to the machine architecture

BTW: How do we handle multi-architecture images, my suggestion is to choose the corresponding image to mount according to the machine architecture

If we can already choose the architecture intelligently, we can also provide a flag like --platform to allow users to decide for themselves.


Update README.md

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

Comparison is base (efba80f) 100.00% compared to head (bec5a56) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##            master      #107   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           16        16
  Lines          137       137
  Branches        18        18
=========================================
  Hits           137       137

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


[FEATURE] - Avoid displaying the webview while running on gitpod

Similar to issue #23 we need the same for gitpod. Every time I open you a new pod I always get the WhatsNew message.
Please also fix disable it for gitpod.
thx

Started playing with Gitpod and this is always popping up. :(


Dev tool source map.

First of all, loving glamorous, great job guys.  My question is does Glamorous support source mapping?

Hi @gt3240! Glad you're enjoying glamorous. Because glamorous is built in glamor, the place to look for information on source maps is in glamor docs/issues. I saved you a search and you'll find some discussion for it here. TL;DR: Can't do sourcemaps with generated code. This is unfortunate. However I think that it's not a huge deal because you can generally figure out what component is responsible for what you're looking at pretty easily (especially with the babel plugin and this config) and finding the styles for that is easy because it's right there with the component. So I'm not too bummed that sourcemaps aren't supported.
Would you care to add a FAQ section to the README explaining this briefly?

Thanks @kentcdodds for getting back to me.  I kind of knew this not being possible due to the css is generated in javascript, but just wanted to confirm it.  I will be happy to add the FAQ section =).


Updated restore.sh to move Palworld backup folder during restore

Context

Implements #521
Choices
Only moving default palworld backups if the world exists in our backup/restore
Test instructions

Create a world which has game generated backups
Create a backup using our backup tool
Restore the backup
Verify the palworld backup folder exists in the restore

Checklist before requesting a review

[x] I have performed a self-review/test of my code
[x] I've added documentation about this change to the docs.
[x] I've not introduced breaking changes.
[x] My changes do not violate linting rules.

Hi @Dashboy1998 just so you know, I am planning on merging these as soon as the arm builds are fixed, once again thanks for the contributions :)


Update sbt-scalafix, scalafix-rules, ... to 0.10.1

Updates

ch.epfl.scala:sbt-scalafix
ch.epfl.scala:scalafix-rules
ch.epfl.scala:scalafix-testkit

from 0.9.34 to 0.10.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala" } ]

labels: library-update, early-semver-major, semver-spec-minor, commit-count:1

https://github.com/scala/scala-rewrites/pull/153#issuecomment-1109363364


Support configuring the GitHub server URL

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
We started the upgrade from v5 to v6 today - specifically ghcr.io/super-linter/super-linter:slim-v6.4.1. However in our GHES environment the super-linter run fails with the error below.
Expected Behavior
The built-in github context variables should be supported by the action<EMAIL_ADDRESS>The documentation says:
github.server_url	string	The URL of the GitHub server. For example: https://github.com.

It does not make sense as a big corporate customer to reinvent the wheel. In the URL building logic of the Action a condition needs to be added to strip an extra schema from the input so that the github.server_url variable can be handled correctly.
Super-Linter version
v6.4.1

Relevant log output
Super-Linter initialization
2024-04-25 15:43:00 [INFO]   Command output when running linters:
------
BASH
GITHUB_ACTIONS
GITLEAKS
JSON
MARKDOWN
YAML
------
2024-04-25 15:43:00 [INFO]   ----------------------------------------------
2024-04-25 15:43:00 [INFO]   ----------------------------------------------
Notice: 25 15:43:00 [NOTICE]   Successfully linted BASH
2024-04-25 15:43:00 [INFO]   Failed to call GitHub Status API: curl: (6) Could not resolve host: api.https
Notice: 25 15:43:00 [NOTICE]   Successfully linted GITHUB_ACTIONS
2024-04-25 15:43:00 [INFO]   Failed to call GitHub Status API: curl: (6) Could not resolve host: api.https
Error: -25 15:43:00 [ERROR]   Errors found in GITLEAKS
2024-04-25 15:43:00 [INFO]   Failed to call GitHub Status API: curl: (6) Could not resolve host: api.https
Notice: 25 15:43:00 [NOTICE]   Successfully linted JSON
2024-04-25 15:43:00 [INFO]   Failed to call GitHub Status API: curl: (6) Could not resolve host: api.https
Notice: 25 15:43:00 [NOTICE]   Successfully linted MARKDOWN
2024-04-25 15:43:00 [INFO]   Failed to call GitHub Status API: curl: (6) Could not resolve host: api.https
Notice: 25 15:43:01 [NOTICE]   Successfully linted YAML
2024-04-25 15:43:01 [INFO]   Failed to call GitHub Status API: curl: (6) Could not resolve host: api.https
Error: -25 15:43:01 [ERROR]   Super-linter detected linting errors

I ran the workflow in debug mode to see what exactly is going on and it seems that you are incorrectly building the URL paths:
  [...]
  2024-04-25 15:55:59 [INFO]   Successfully found GITHUB_SHA: 36f8c32e4a1fffa7047eeb7965e8ab77d5d42493
  2024-04-25 15:55:59 [DEBUG]   Successfully found the root commit: 27261c4ca041a0bccdedf1987d212cfe107dcb35
  2024-04-25 15:55:59 [DEBUG]   This is a GitHub pull request. Updating the current GITHUB_SHA   (36f8c32e4a1fffa7047eeb7965e8ab77d5d42493) to the pull request HEAD SHA
  2024-04-25 15:55:59 [DEBUG]   Updated GITHUB_SHA: 83815cb76f3bf28e4ce229d1e116b5299352e335
  2024-04-25 15:55:59 [INFO]   Successfully found GITHUB_ORG: my-cool-org
  2024-04-25 15:55:59 [INFO]   Successfully found GITHUB_REPO: star-chaser-config
  2024-04-25 15:55:59 [DEBUG]   Successfully detected the default branch for this repository: master-caas-v3
  2024-04-25 15:55:59 [INFO]   The default branch for this repository is set to: master-caas-v3
  2024-04-25 15:55:59 [INFO]   Successfully found GITHUB_TOKEN.
  2024-04-25 15:55:59 [INFO]   Successfully found GITHUB_REPOSITORY: my-cool-org/star-chaser-config
  2024-04-25 15:55:59 [INFO]   Successfully found GITHUB_RUN_ID 6446359
- 2024-04-25 15:55:59 [DEBUG]   GitHub Status URL: https://api.https://git.contoso.com/repos/my-cool-org/star-chaser-config/statuses/83815cb76f3bf28e4ce229d1e116b5299352e335
- 2024-04-25 15:55:59 [DEBUG]   GitHub Status target URL: https://https://git.contoso.com/my-cool-org/star-chaser-config/actions/runs/6446359
  2024-04-25 15:55:59 [DEBUG]   Configuring Git safe directories
  2024-04-25 15:55:59 [DEBUG]   Set /github/workspace as a Git safe directory
  2024-04-25 15:55:59 [DEBUG]   Set /tmp/lint as a Git safe directory
  2024-04-25 15:55:59 [DEBUG]   Set /tmp/lint as a Git safe directory
  2024-04-25 15:55:59 [DEBUG]   TYPESCRIPT_STANDARD_TSCONFIG_FILE: /github/workspace/tsconfig.json
  2024-04-25 15:55:59 [DEBUG]   R_RULES_FILE_PATH_IN_ROOT: /github/workspace/.lintr
  2024-04-25 15:55:59 [INFO]   --------------------------------------------
  2024-04-25 15:55:59 [INFO]   Validating the configuration
  2024-04-25 15:55:59 [DEBUG]   Set VALIDATE_LANGUAGE while validating the configuration: VALIDATE_ANSIBLE
  2024-04-25 15:55:59 [DEBUG]   Configuration didn't provide a custom value for VALIDATE_ANSIBLE
  2024-04-25 15:55:59 [DEBUG]   Set VALIDATE_LANGUAGE while validating the configuration: VALIDATE_BASH
  [...]

Steps To Reproduce
The Action itself is called as follows and has been working fine with v5 - switching to v6.4.1 fails with the linked error.
# [...]

jobs:
  lint:
    runs-on: [contoso-x86-2core]
    steps:
      - name: Checkout Code
        uses: actions/checkout@v4
        with:
          fetch-depth: 0

      - name: Super-Linter
        uses: github/super-linter/slim@v6
        env:
          GITHUB_DOMAIN: ${{ github.server_url }}
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          VALIDATE_MARKDOWN: true
          VALIDATE_YAML: true
          VALIDATE_BASH: true
          VALIDATE_GITLEAKS: true
          VALIDATE_GITHUB_ACTIONS: true
          VALIDATE_DOCKERFILE_HADOLINT: true
          LINTER_RULES_PATH: /
          YAML_CONFIG_FILE: .yamllint.yaml

# [...]

Anything else?
No response

@ferrarimarco  introduced this regression with this change:
https://github.com/super-linter/super-linter/commit/6fd6830fb4a277ab1eefd57b10e2d05fd978d929#diff-0b89e6e14353371aacf69d4b9067d4481dd99c6994409b32e3b906c69d478c21L147-L149 (working code left, regression right).

@megamorf You are setting GITHUB_DOMAIN, which is for the domain of your installation, not for the server URL, as you can see from the README.
The commit you reference fixed a few inconsistencies about variables not behaving as described in the documentation, and also it disallowed some confusing corner cases that allowed you to set GITHUB_DOMAIN and GITHUB_CUSTOM_API_URL in a completely independent way, which is likely error prone.
The current implementation works if you set GITHUB_DOMAIN to your domain and not to github.server_url.
This is an enhancement request, rather than a bug because support for github.server_url was never implemented in the first place.
Happy to review a PR for this.

At least you can set GITHUB_API_URL: ${{ github.api_url }} which should make the API call work, but the resulting status check would have the wrong target URL because GITHUB_DOMAIN points to the wrong link, this provided that the GitHub API doesn't reject invalid target links for status checks. Given that we can do this in the current implementation, it probably makes sense to have something comparable for the GitHub server URL.
I think we likely need to:

Expose the GITHUB_SERVER_URL variable because we cannot use GITHUB_DOMAIN for this. GITHUB_DOMAIN is a convenience variable to automatically build the URL of the GitHub instance and the URL of its API server. Maybe we should make this clearer in the README.
Validate all these variables to handle corner cases, such as when the user sets GITHUB_DOMAIN and any of GITHUB_SERVER_URL and GITHUB_API_URL, or both.

As it is now, it would require a GHES instance with +20.000 users to needlessly introduce a workaround across all workflows (stripping the schema from github.server_url) just to adhere to what appears to be arbitrarily chosen input requirements.
server_url is part of the github context for both github.com and GHES. So I'd very much be in favour of introducing support for those built-in variables.
Also, for transparency, in our case the following two context variables are not pointing to different domains:
github.server_url: https://git.foo.contoso.com
github.api_url     https://git.foo.contoso.com/api/v3

I think we likely need to:

Expose the GITHUB_SERVER_URL variable because we cannot use GITHUB_DOMAIN for this. GITHUB_DOMAIN is a convenience variable to automatically build the URL of the GitHub instance and the URL of its API server. Maybe we should make this clearer in the README.
Validate all these variables to handle corner cases, such as when the user sets GITHUB_DOMAIN and any of GITHUB_SERVER_URL and GITHUB_API_URL, or both.

Yeah, that sounds like the most reasonable approach. I guess we first need to compile the different allowed/forbidden states and then refactor the logic accordingly. Let me know how I can help here to remove the blocker.

#5792 should take care of this.


await is ignored when used with pause()

What are you trying to achieve?
Debugging in asynchronous methods
What do you get instead?
Could not debug since the statements are not executed after await statement and the execution ends. This happens only when pause() is used

Provide console output if related. Use --verbose mode for more details.

# paste output here

Provide test source code if related

// paste test

Details

CodeceptJS version:
NodeJS Version:
Operating System:
Protractor || WebDriverIO || Nightmare version (if related)
Configuration file:

# paste suite config here

Sorry I didn't get it.
Could you show me the code? And the actual output of execution

I think its my problem... Sorry for bothering


The replies to messages are missing from CSV file, only standalone messages are exported

Operating system: Windows 10
Build versio:  v0.1.7-alpha.2

Detailed description
I was able to decode the backed up messages in a CSV file (XML doesn't work) but missing from the export are messages which are replies to other messages. That can be observed by looking for specific text that simply doesn't exist in the Excel file. It doesn't matter if it's a reply to media file or text file, it just doesn't show up as text.

Just as a note, this is because the replies are saved as media file types and so they're in a different table altogether. This is just to note that they don't appear, you can find them if you export signal messages to an sql database and browse the table. If all tables are concatenated in CSV it would be too many empty messages.


Confusion around SslStream::accept and SslStream::connect

Hi, I've been updating one of my pet projects to the latest version of tokio and tokio-openssl. Now I have one problem left that I'm really confused about. The relevant part of the code is the following:
impl Shared<Socket> {
    ...

    fn spawn_accept_connections_task(
        &self,
        mut incomming_connections: tokio::sync::mpsc::Receiver<crate::network::io::Connection>,
    ) -> Result<(), openssl::error::ErrorStack> {
        let acceptor = acceptor_for_id(&self.read().id)?;
        let ssl = openssl::ssl::Ssl::new(acceptor.context())?;
        let socket = self.clone();
        tokio::spawn(async move {
            loop {
                let Some(stream) = incomming_connections.recv().await else {
                    return;
                };
                let mut ssl_stream = match tokio_openssl::SslStream::new(ssl, stream) {
                    Ok(s) => s,
                    Err(e) => {
                        warn!("Error creating new SSL stream: {}", e);
                        continue;
                    }
                };
                let ssl_stream = Pin::new(&mut ssl_stream);
                if let Err(err) = ssl_stream.accept().await {
                    warn!("Error performing TLS handshake: {}", err);
                    continue;
                };
                let id = match id_from_connection(&ssl_stream) {
                    Ok(id) => id,
                    Err(err) => {
                        warn!("Error authenticating peer: {}", err);
                        continue;
                    }
                };
                let conn = Connection::from_tls_stream(
                    ssl_stream,
                    id,
                    socket.read().deliver_frame.clone(),
                );
                socket.write().connections.insert(id, conn);
            }
        });
        Ok(())
    }

    pub async fn open_connection(
        &self,
        identifier: Identifier,
        address: SocketAddrV6,
    ) -> Result<Connection, io::Error> {
        let connector = match connector_for_id(&self.read().id).configure() {
            Ok(c) => c,
            Err(e) => {
                return Err(io::Error::new(
                    io::ErrorKind::Other,
                    format!("Unable to build TLS client configuration: {}", e),
                ))
            }
        };
        let ssl = connector.into_ssl(&identifier.to_string())?;
        let conn = self.read().io_socket.connect(address.into())?;
        let mut ssl_stream = tokio_openssl::SslStream::new(ssl, conn)?;
        let ssl_stream = Pin::new(&mut ssl_stream);
        if let Err(err) = ssl_stream.connect().await {
            return Err(io::Error::new(
                io::ErrorKind::ConnectionAborted,
                format!("TLS handshake error: {}", err),
            ));
        };
        let conn =
            Connection::from_tls_stream(ssl_stream, identifier, self.read().deliver_frame.clone());
        Ok(conn)
    }

   ...
}

You can find the full code here https://github.com/maufl/uip
My problem is that the compiler complains that I can't do let conn = Connection::from_tls_stream(ssl_stream, identifier, self.read().deliver_frame.clone()); since I moved ssl_stream when I called ssl_stream.connect(). Looking at the signature, this makes sense, it takes a mut self. However, there seems to be no such issue in the spawn_accept_connections_task method after I called ssl_stream.accept, even though the signature of both functions is basically the same. I tried to find a difference but it seems to be an identical case to me.
Generally I wonder, why does accept and connect take a mut self and doesn't return Self anymore? How are you supposed to use this and continue using the SslStream? (Are there any examples anywhere?) Or did I miss-understand something about moves of Pined types?

Your pin is shadowing the stream itself. Rename the variable here to something else: let ssl_stream = Pin::new(&mut ssl_stream);

Hm, I don't think this was the issue, but I was able to solve it anyway. I realized than then Pin doesn't take ownershipt of the stream, so I made it live short enough to reuse the ssl stream. After fixing the connect method, the compiler complained about the  accept method, so I guess this was just a matter of lifetime issues rolling in slowly.
Thanks!


Copy btn to clipboard is not working properly (its copying wrong btn)

the problem is probably with the Id passed down to the copy to clipboard function

is the issue with all buttons or only with certain buttons..can you specify

the function works fine, but the id passed to the function is the problem a wrong text is copied to the clipboard

Okay
Please can I be assigned this issue

@Michelle031 have a look at that page, and try copying some of the buttons (then see if its copying the correct button)

Hey there! I think I can help with this issue!!
Can you please assign it to me?

@Jaivrat12 i assigned it to you, you can have a check

Hey @khazifire! I fixed the bug and I think that the btnAreaRef in pages/docs/buttons.js file isn't needed anymore. Can I remove it from the code and then make a PR? Or should I ignore that as you may need it in the future?
BTW the btnAreaRef can only be used on one element at a time that's why it won't work for multiple elements, you need to have an array of btnAreaRefs to do that. So, I think removing it is better.

@Jaivrat12 You can simply comment it out 😌🙂


Faster Index Build- Phase 1

This builds on #209 and tweaks the parser + the vocabulary building pipeline s.t. the first stage of index building gets faster.

@joka921 ok I've got this running with the newly built case insensitive Wikidata Full and completion now works flawlessly, well done! It's not even significantly slower so that's pretty nice too.

@niklas88 Thanks for your review. I have added some comment clarifications, please check whether you agree. All the stuff that has no comments will be fixed exactly in the way you suggested since I agree with you in those places.


Migration tool could understand manually lazily initialized fields

It would be great if the tool recognized a manually lazily initialized field:
class C {
  int _i;
  int get i {
    _i ??= ...;
    return _i;
  }
}

In this case , the tool makes _i nullable, and often makes get i nullable as well (depending on how the body is structured).
The big problems come when get i is considered nullable, when it should be non-nullable. I can generally force non-nullability by hinting /*late final*/ int _i;, but I think this would make illegal code (or just warnings) w.r.t. _i ??= ...;. The fix that I would do as a human are, perhaps strangely, to flip the field _i and the getter i, to use lazy initialization:
class C {
  late final int i = _i;
  int get _i {
    var i_ = ...;
    return i_;
  }
}

But I'm not sure that this is what we want the tool to do... sometimes the field and the getter are not adjacent class elements.
This all holds for top-level variables too.

Personally I think the first step in addressing this would be to have flow analysis's edge builder treat fields as promotable for purposes of understanding user intent (see https://github.com/dart-lang/sdk/issues/40566).  If we did this, then it would not try to mark the getter as nullable (because it would see that _i is intended to be non-null after the ??=), so it would produce this:
class C {
  int? _i;
  int get i {
    _i ??= ...;
    return _i!;
  }
}

(Note that the fix builder would still use proper flow analysis, which doesn't promote fields, so the tool would see that a ! is needed on the return statement).
I realize this is not the migration you're asking for, but it's still a lot better than what we do today.  In particular, since it doesn't mark the getter as nullable, it won't propagate nullability virally into other parts of the user's code.
Once we've done that I we could circle back and consider more sophisticated logic that would detect this pattern and replace it with a late variable.

As of 1c7fe7196dd7a542f7f91cd6ea2ac81e1a13146c, the null safety migration tool has been removed from active development and retired. No further work on the tool is planned.
If you still need help, or you believe this issue has been closed in error, please feel free to reopen.


Binding buttons(actions)

Is it possible to create an adaptive card with dynamic buttons? the number of buttons and the content of each button may vary. I found samples where you can bind the number and type of input controls but I can't reproduce it for buttons.

@pedro-ramirez-suarez
Yes, you can. If you go to the Designer. Once you add an ActionSet, you can create a payload with actions by clicking "Add an Action".

Or if you'are asking if you can create a card that can display a different set of actions depending on the data you have. You can use Templating SDK.


Nginx port and environment options

As discussed in #9, this provides a --port option for cyber-dojo, as well as support for an nginx.env file similar to the existing grafana.env.
The target use-case is to expose cyber-dojo running on a shared machine via https://github.com/jwilder/nginx-proxy. This proxy will publish any exposed port using name-based virtual hosting, according to a VIRTUAL_HOST environment variable set in the container.

I wasn't sure where to document the nginx.env file. The grafana.env file isn't documented, but as noted in cyber-dojo/grafana#1, it's still WIP.


Ensemble Task Implementation

@HobbitLong Thank you very much for making the effort to clean and post your code for these benchmarks! I'm sure that you don't have time to post code for the ensemble distillation task, but I am going to try reproducing that benchmark so perhaps if there are any tricks or different hyperparameters settings that you can remember for that particular task off the top of your head then we can document them in this issue.

For Figure 4 in the paper, I'm wondering exactly how a single point is generated in those plots. For example, for the point that is ResNet distillation from four teachers, is that an average over multiple trials? And if so, for each trial are four new teachers trained from scratch for that trial? Or was there a pool of e.g. 8 teachers and each 4-teacher trial randomly selects four from among those 8, each 6-teacher trial randomly selects 6 from among those 8 etc?

Hi, were you able to reproduce the ensemble distillation task?


overlapping staff groups not currently permitted

From<EMAIL_ADDRESS>on October 03, 2013 11:55:52
Overlapping staff groups can be handled by adding @startid, @endid, and @level on <grpSym>. @startid and @endid point to the first and last staffDef included in the group, while @-level indicates how this symbol should be drawn relative to any other group symbols. Symbols with level='1' should be drawn closest to the start of the staves, level='2' to the left, etc. (Note that this is the opposite of the treatment of "normal", nested <staffGrp> elements.)
In order to distinguish <grpSym> elements that mark overlapping groups from those that simply provide details about the symbol associated with a non-overlapping group, <grpSym> should be added to the content model of <scoreDef> following <staffGrp>.  Grouping symbols that follow <staffGrp> are expected to carry @startid, @endid, and @-level attributes, but <grpSym> elements within a <staffGrp> must not allow these attributes. These requirements must be enforced via Schematron.
An example, where staves 1 - 4 are grouped using a bracket and staves 3 - 5 are grouped by a brace:
<scoreDef>
<staffGrp>
<staffDef xml:id="P1"/>
<staffDef xml:id="P2"/>
<staffDef xml:id="P3"/>
<staffDef xml:id="P4"/>
<staffDef xml:id="P5"/>
</staffGrp>
<grpSym symbol="bracket" startid="#P1" endid="#P4" level="1"/>
<grpSym symbol="brace" startid="#P3" endid="#P5" level="1"/>
</scoreDef>
Original issue: http://code.google.com/p/music-encoding/issues/detail?id=178

From<EMAIL_ADDRESS>on October 04, 2013 07:52:52
The description of @level above is backwards -- symbols with level='1' should be drawn farthest from the start of the staves, level='2' closer, etc.  @-level must work this way in order to allow <grpSym> elements following <staffGrp> to be used as a substitute for nested <staffGrp> elements, where the grouping symbols go from left to right, getting closer to the start of the staves.


upgrade pip to 8.1.0

Right now the buildpack has 8.0.2

I've opened PR #284 to update pip/setuptools :-)


Eats up all my RAM during building then dies.

So I'm realizing that what I actually need is a parser/lexer. However, I don't know if that relates to chewing up all the memory or not so I figured I'd submit this anyway just in case it helps solve a problem.
This is the lexer I'm working with
#[derive(Logos, Debug, PartialEq)]
enum Token {
    #[end]
    End,
    #[error]
    Error,

    // #[regex = r"(?s)/\*.*\*/"]
    // BlockCommentStart,
    //
    // #[regex = r"//.*\n?"]
    // LineComment,

    #[regex = r"/\s*\{"]
    RootNode,

    #[regex = r"\s*([[:digit:][:alpha:]_]*:)?\s*[[:digit:][:alpha:],._+@-]*\s*\{"]
    Node,

    #[regex = r"\s*};"]
    NodeEnd,

    #[regex = r#"(?m)\s*[#[:digit:][:alpha:],._+-]*\s*=\s*(<|"|\[)[[:alpha:][:digit:],]*(>|"|\]);"#]
    Property,

    #[regex = r"\s/[[:alpha:]-]*/\s[[:alpha:][:digit:]_&@-]*;\n?"]
    Directive,

    #[regex = r#"\s*#include (<|")[[:alpha:][:digit:]_-]*\.(h|(dtsi))(>|")"#]
    Include,

    // #[regex = r"\s*[[:alpha:][:digit:],#]*\n"]
    // NonStdProperty,
}

The problem is that, in addition to taking forever to finish, it will use up all 16GB of RAM on my machine and then die when it tries to grab another ~41MB at some point.
If this is doing more than what Logos is supposed to, go ahead and close this out.

Going to aggregate this under #70.


feat: add bundle command

Description
Integrating bundler into CLI by introducing the command asyncapi bundle.
Usage:
asyncapi bundle ./file1.yml ./file2.yml --output asyncapi.yaml --referenceIntoComponents

Related issue(s)

Resolves #219

@Souvikns remember to click arrows when ready for another review

Not sure why but now  it is unable to install dependencies https://github.com/asyncapi/cli/actions/runs/4163522302/jobs/7204027231#step:7:69

I think it is random npm issue, saw it few times already in other projects -> https://github.com/asyncapi/cli/actions/runs/4163522302/jobs/7204027231#step:7:41
I rerun, let's see

@derberg could this be a ubuntu related issues, on mac at least npm i works

@Souvikns yo, with all the recent changes here and there, try fixing conflicts and maybe 🤞🏼 this time will be good

@Souvikns you have them there again. Remember to ping me once you solve them. Rerequest review using GitHub buttons. I do not get notifications that you pushed code


Create icai.txt

@MayankSood Is this institute a purely online university? Does it have offline classes, where students study in a classroom? If yes, could you please give us a link to the page on the official website where an IT-related long-term (>1 year) course is offered by the university.

@philipto
ICAI is The Institute of Chartered Accountants of India.
Students here are studying Charted Accountancy. They might use some programming languages for their personal needs or to automate some accounts related scenarios.
This is one and the only institute in India for Charted Accountancy (Run by Government of India)
Below is the website.
https://www.icai.org/
They provide IT Training (both 100 Hours ITT and Advance ITT) for students (mandatory), and Activities related to ITT Centres set up by ICAI etc., where students can learn and motivate themselves to try IT as well.
below is the Link:
https://www.icai.org/new_post.html?post_id=8577&c_id=355

@philipto
Do we have any update on this?

@MayankSood Yes. As far as I found out, Institute of Chartered Accountants of India is not just an educational institution, it is a much broader community of Chartered Accountants. You can see https://en.wikipedia.org/wiki/Institute_of_Chartered_Accountants_of_India for the details.
Regrettably, the link to the page, which you have provided us with, does not lead to a course description. As far as I understand you, the students can (but not obliged to) educate themselves in IT. However, JetBrains does not grant free educational licenses for self-education.
Therefore, it is unlikely that Institute of Chartered Accountants of India email domain can be added to this repository, because formally the Institute is not a university, nor a college, and the students there do not study IT as a part of their curriculum.
Also, it remains unclear to me if the Institute has classrooms where students study, or the students study online.
Please leave a comment if you wish to provide some clarification. Thank you.

No problem @philipto
I guess I was unclear about the terms and conditions of JetBrains students license.
Thanks for the help and time spent.
I am closing the pull request.
Regards
Mayank S


Set Min Date

Hi!
I need define a year before 1970, but I can see that in the array of years it's defined since 1970.
Thanks! :)

Hi, I have changed the year range, now it starts from 1900.
A better implementation of the selector will come as soon as the next version of Angular Material will be released.

Ok, thanks for your time :)


Autoreload other python files if they've been imported

User scenario:
User has two files, foo.py and bar.py.
They are working in foo.py and they run this cell
from bar import my_func
my_func()

Then they go to bar.py and modify my_func().
They run a cell like so:
#%%
my_func()

It would be nice if my_func automatically picked up the changes from bar without having to redo the import.
The autoreload magic does this already.
https://ipython.readthedocs.io/en/stable/config/extensions/autoreload.html

Isn't it possible to simply leverage the existing ipython magic commands? I imagine the following would do the trick:
#%%
%load_ext autoreload
%autoreload 2

This feature is about doing that part automatically, so people wouldn't have to know about autoreload.

I agree that it would be nice if this could be done automatically e.g. via a provided (and user-extensible) startup script that is run whenever a new Python Interactive Window is launched.

Actually, I believe this feature request is different from
#%%
%load_ext autoreload
%autoreload 2

As far as I understand, the above only auto-reloads the module as a whole when the source changes, not individual imports from that module. If you import a function as
from bar import my_func
my_func()

then it will not reflect changes. For that you would need to do
import bar
bar.my_func()

Just to be clear, @rchiodo, you're advocating the latter, correct? I'd also really appreciate this feature.

@janosh 	I didn't realize that autoreload didn't work in the 'from' case. Our intent is to track all open files and functions from each file and reload them if they change.

According to the iPython docs, autoreload is actually supposed to support the from case:
In [1]: %load_ext autoreload

In [2]: %autoreload 2

In [3]: from foo import some_function

In [4]: some_function()
Out[4]: 42

In [5]: # open foo.py in an editor and change some_function to return 43

In [6]: some_function()
Out[6]: 43

However, that has never worked for me.

Just ran into a problem with auto-reloading: it doesn't work with decorators, i.e. modifying a decorator does not update previously imported decorated functions.
Took me 15 minutes to figure this out. Here's the corresponding iPython issue. Not that this extension can necessarily do something about it, but I thought I'd mention it anyway in case someone else runs into this issue.

There is an option in the preferences to run startup commands for Interactive Python. Mine is set to
"python.dataScience.runStartupCommands": "%load_ext autoreload\n%autoreload 2"

@krystophny Same.

☝  Syntax has been updated, it's "python.dataScience.runStartupCommands": ["%load_ext autoreload","%autoreload 2"]

@timvink Both still work (see #12827) but the string version now triggers a warning.

Hi @rdrey! We're really sorry for the inconvenience but are looking forward to reviewing your solution to #3564! We are in the end stages of a restructuring to enable a more manageable relationship with the Python core extension's codebase. We're getting very close to having this completed and once we do we will make a more public statement about it. Please stay tuned.

@greazer No problem. I was basically just going to wrap @krystophny & @timvink's solution in a new setting to append the magic commands to runStartupCommands when it's turned on. I did a search for runStartupCommands in the code base, looked at how #12827 was implemented and was very confused that the code was gone till I read about the new Notebook API.


Install module doc snippet incorrect

Docs here https://developer.lightbend.com/docs/akka-serverless/javascript/index.html#_add_sdk_libraries says npm install<EMAIL_ADDRESS>--save-exact but that's not right.
npm install @lightbend/akkaserverless-javascript-sdk --save-exact works but I don't know how to get the specific sdk version in there, adding another @0.33.1 gives me an error

Scrap that last part, including the version works.

Fixed by #197


Add webhook server

Description
Add webhook server that handles both inbound-message and message-status.

Thanks Alex!


[release/1.6] Revert "Downgrade MinGW to version 10.2.0"

This reverts commit fa2016d58ada2438d0af51a522b528dc228cb4ba
Previously we were downgrading mingw to work around an issue in the race detector in Go on Windows when used with a newer version of GCC. The issue was first reported here:
golang/go#46099
Shortly after the release of 1.19 someone had commented this issue was solved for them, and after trying it out in some test runs on actions machines, it seems to be the case. Disabling ASLR got things in order, and PIE was disabled for -race builds in 1.19, so this is likely the reason things work now:
https://github.com/golang/go/commit/0c7fcf6bd1fd8df2bfae3a482f1261886f6313c1.
The downgrade was mostly harmless except for two shortcomings:

It took quite a while for the package to get downloaded+installed.

Chocolatey would frequently fail to download with The remote file either doesn't exist, is unauthorized, or is forbidden for url ... Exception calling "GetResponse" with "0" argument(s): "The request was aborted: Could not create SSL/TLS secure channel." Restarting the failed run would often resolve this, but a 50-50 shot of things working is not a great situation.

(cherry picked from commit 912d43b2bdfac622cc46f510c21379cf672be918)
clean cherry pick of #7308

/cc @estesp @mikebrow
As per https://github.com/containerd/containerd/pull/8660#issuecomment-1584874129

A nice side effect is saving 8-9 min on average per Windows test run! Just saw that the downgrade step averages 8 or 9 minutes per run.


[T3A3][W09-A2] Shaun Chee

Ready for Review.

Good effort @ShaunChee  Some comments added. Please close the PR after reading comments.


How can I download the transcript of a video?

I see in the documentation that a transcript is created but couldn't find instructions for downloading it.
Ellen Finkelstein

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

ID: 7f08a5b8-5d07-c53d-8ff2-04cf7a146e5a
Version Independent ID: 2f277d08-e5ab-7ab9-865d-d70f887a3bc4
Content: What is Azure Media Services Video Indexer? - Azure Media Services
Content Source: articles/media-services/video-indexer/video-indexer-overview.md
Service: media-services
Sub-service: video-indexer
GitHub Login: @Juliako
Microsoft Alias: juliako

Thank you for your detailed feedback.  We are looking into the issue, and we will respond when we have more information.

@EllenFinkelstein Thanks again for the feedback! You may check below document link might be helpful:
https://github.com/Azure-Samples/media-services-video-indexer/blob/master/IntroToVideoIndexer.md

@EllenFinkelstein We will now proceed to close this thread. If there are further questions regarding this matter, please let us know.


Lazy Layout: don't assume bg starts out cleared

Thanks!

Does this mean there's no need to call g.clear() at all in an app now?

Yes, as long as the outermost layoutObject fills the screen and has bgCol set.


TV versión

Describe your suggested feature
Make a version that is compatible with TVs, be it Google TV, Android TV, etc.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is a request regarding an extension, I should be opening an issue in the extension's repository.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I will fill out all of the requested information in this form.

Duplicate of #162


Possible typo in onDragEnter

I'm getting errors on this line in the onDragEnter function:
var dataTransferItems = Array.prototype.slice.call(e.dataTransfer ? e.dataTransfer.items : e.target.files);

because e.dataTransfer.items is undefined. I believe this is supposed to read e.dataTransfer.files to match up with the rest of the usages.
For reference here's the mozilla documentation on the DataTransfer object: https://developer.mozilla.org/en-US/docs/Web/API/DataTransfer

I believe this bug was introduced in commit d4d0867

Yes, you are right. @okonet could you please help out?

Well, funny thing is that in Chrome it is items. I'll check for either one is defined.

Ah, yep, I found this on Safari and Firefox.


gpg: no valid OpenPGP data found

Issue Description
wget -q -O - "https://repos.ripple.com/repos/api/gpg/key/public" |
sudo apt-key add -
gpg: no valid OpenPGP data found.
Steps to Reproduce
stuck at wget and apt-key add step of installing rippled
Expected Result
GPG Key should be added
Actual Result
gpg error
Environment
Digital Ocean Droplet
16 GB Memory / 8 Intel vCPUs / 320 GB Disk / SFO3 - Ubuntu 20.04 (LTS) x64
Supporting Files

I just checked and the data I received seems reasonable. Please make sure you're typing everything on a single line:
wget -q -O - "https://repos.ripple.com/repos/api/gpg/key/public" | sudo apt-key add -

Is this still an issue @debragail, or can we close this issue?

@nbougalis I paused the project but if it occurs again i will reference this issue


style: add shfmt exit status to brew style

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?
[X] Have you written new tests for your changes? Here's an example.

I refactor some functions in utils/shfmt.sh in #12044 to reuse code fragments and improve code readability.
And adding exit code of run_shfmt to brew style is needed in CI tests.

Thanks again @XuehaiPan!


[logging] Extra flag problem

Now we use flag package to support command line flags, and glog also uses it by default. Then you can see our binary have more flags than we thing although we use logrus instead of glog:
➜  tf-operator git:(416) ✗ ./tf-operator -h
Usage of ./tf-operator:
  -alsologtostderr
    	log to standard error as well as files
  -chaos-level int
    	DO NOT USE IN PRODUCTION - level of chaos injected into the TFJob created by the operator. (default -1)
  -controller-config-file string
    	Path to file containing the controller config.
  -gc-interval duration
    	GC interval (default 10m0s)
  -json-log-format
    	Set true to use json style log format. Set false to use plaintext style log format (default true)
  -log_backtrace_at value
    	when logging hits line file:N, emit a stack trace
  -log_dir string
    	If non-empty, write log files in this directory
  -logtostderr
    	log to standard error instead of files
  -stderrthreshold value
    	logs at or above this threshold go to stderr
  -v value
    	log level for V logs
  -version
    	Show version and quit
  -vmodule value
    	comma-separated list of pattern=N settings for file-filtered logging

There are some pros and cons:

We can support vendor and client's glog, since we have glog's flags
But the users may be confused since the tf-operator outputs logs regardless of the flag -logtostderr

We can refer to etcd/etcd-operator, which also uses logrus.

How can i set this flags to log to a dir and the log file same as the stderr .Now I tried to set -log_dir but the log file have less log than stderr

https://github.com/sirupsen/logrus/issues/156


Adding extra line from vim

I'm not sure how to determine if this is caused by vim or prettier, so sorry in advance if it is caused by vim.
After installing prettier globally and adding autocmd FileType javascript set formatprg=prettier\ --stdin to my .vimrc, I am seeing the following.
Initial state:
function test() {
  return 'hello';
}

Action:
Type gggqG, which selects the whole file and runs prettier on it. Unfortunately, the formatting part works, but I am left with an extra newline at the end of the file. The same thing happens if you go in visual mode, select the whole file and type gq.
Let me know if there is any other info that would be helpful or if there is any way I can help. Thank you so much for your work, I have been loving prettier so far!

Me too. I don't know that much about gq to think of why this may be happening, but my temporary workaround is this:
autocmd FileType javascript nnoremap gq gqdd
autocmd FileType javascript vnoremap gq gqdd

Prettier adds a newline to the end of each file, so this is working as unexpected, unless you mean that there are two newlines at the end of the file?

It's not necessarily at the end of a file, but I think that's because prettier assumes what it receives in stdin is a "file". Anyway, here are some screenshots:
Before prettier

After prettier in vim

After prettier in command line (I undid the vim changes first)

This happens because console.log() adds yet a newline at the end of file. This is not specific to vim. PR coming soon.

Thanks @lydell!


Add automatic browser compatibility prefixes using Grunt

To be less likely to break in browsers.

https://github.com/Eyra-is/Eyra/pull/92


Add a new type of stones

What does this pull request do?
Adds a new type of stones, with challenge editor controls and adaptative rendering.
How was it tested?
In editor and in challenge to test display, code execution and fulfillement of objective.
Notes
Didn't take into account the pickedStones value as it never was an issue.

 Add more categories of stones


Fix/armored header multiple uids

The current implementation of ArmorUtils only serialize one identity in the header comment. This has not to be the primary identity, just the one that happens to be the first in the iterator.
This change fixes that behaviour, resulting in all identities being serialized in the comments.
Furthermore, one example in the README is not working as the Passphrase wrapper class is not used.

Hey!
I am not sure what the use of having all user-ids in the armor really is. Comments inside the ASCII armor are not authenticated, so there is no point in relying on that information.
Currently PGPainless adds the user-id purely for convenience reasons. It is intended as a quick and dirty hint on the content of the certificate file. I think adding ALL user-ids would inflate the armor too much and possibly do more damage than good.
However, it is probably a good idea to make sure that the user-id that makes it to the armor is actually the primary user-id rather than merely the first one.

You're right about the convenience. We intended to have the primary key in it but as the user-ids are Strings only we didn't change that but have all identities in it. This way it gets nasty when having a lot of user id in a key as it will blow up the armored representation.
As an idea it might be a solution having only the primary ID as comment and a message like 'Contains 6 user id' like
-----BEGIN PGP PUBLIC KEY BLOCK-----
Version: PGPainless
Comment: 9F53 AC3E 18ED CDBF 47EE  570D 02F2 C38C 3E65 9A52
Comment: Foo Bar<EMAIL_ADDRESS>(Primary User-ID)
Comment: KeyRing contains 6 User IDs
...
-----END PGP PUBLIC KEY BLOCK-----

IMO I'd like to know at least the number of identities the keyring holds.

That might be a good compromise.
I'd use "Certificate" instead of "KeyRing" and also only add that line if there is more than one user-id.

That sounds good, I'll update my PR to reflect the suggested changes.

Not sure about the build error, I'm not familiar with "animalsniffer"?

Animalsniffer a gradle plugin is ensuring Java/Android API level compliance. PGPainless is aiming to be compatible to Android API level 10 (~Java 7). That means that java.util.Optional is not available unfortunately.

Thank you for the suggestions, i'll think about / try out the different approaches and then update this PR. Could take a while though, I'm pretty booked this week.

@vanitasvitae I've removed the Optional-stuff, wasn't that much to change and I think it's still readable

Hey!
I merged the code with some smaller changes.
Specifically I omitted the "(Primary)" behind the primary user-id.
Further, if there is no user-id marked as primary (some legacy applications forget the marking), then we print the first user-id instead.
Lastly, I changed the wording of the remaining user-ids to "1 further identity" or respectively "X further identities".
I hope this suites your needs ;)


Distribution map within a county

In most highly infected counties (Harju), more detailed distribution map (municipalities) could be produced to pinpoint on infection hot-spots for locals to be aware.

Right now this data is not available from terviseamet, if it becomes so in the future we will add it. I will close the issue for now.

Could you make a request also from the developers side to Terviseamet, and
I try to push them from local public side. Probably they just do not have trust in abilities to present that.
But Finish case of closing on small region shows its value.
J

Saatja: keeganmcbride<EMAIL_ADDRESS>Saadetud: teisipäev, 24. märts 2020 22:39
Adressaat: okestonia/koroonakaart
Koopia: Jaan Liira; Author
Teema: Re: [okestonia/koroonakaart] Distribution map within a county (#43)
Right now this data is not available from terviseamet, if it becomes so in the future we will add it. I will close the issue for now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/okestonia/koroonakaart/issues/43#issuecomment-603492542, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO5RUYI5RWK5AYDAZ4YQ2PDRJEK73ANCNFSM4LSP32KQ.


[Track Tax] Tax options do not come back after disabling and reenabling them

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Found when validating PR : https://github.com/Expensify/App/pull/36784
Version Number: v1.4.44-7
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation:
Action Performed:
Precondition:

User is an employee of Collect workspace.
The Collect workspace has some tax options.

[Employee] Go to workspace chat.
[Employee] Open request money page > Manual > Confirmation page.
[Employee] Click Show more > Tax rate.
[Admin] Disable all the tax options.
[Admin] Enable all the tax options.

Expected Result:
The tax options will be showing for Employee when re-enabled.
Actual Result:
The tax options are not showing for Employee when re-enabled. The options only show up after relogin.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/e5b4b206-67d0-4511-9b5b-73c92fb7f369

View all open jobs on GitHub

@jliexpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think that this bug might be related to #wave6-collect-submitters
CC @greg-schroeder

Feature is WIP, so we can put this HOLD until we finish the implementation as it is being rolled out in phases https://github.com/Expensify/Expensify/issues/305057

@MonilBhavsar do you want to be assigned this issue?

@MonilBhavsar just moving to weekly so we're not pinged, feel free to change the priority.

Oh hold

@MonilBhavsar moved this to the Collect project as that deals with Taxes, right?

Still on hold

This is related to a pusher update. I'll work on the fix now

This is not a bug anymore. @izarutskaya you we please retest and confirm. Thank you!
Uploading Screen Recording 2024-04-12 at 5.18.40 PM.mov…

I'm closing this. Feel free to reopen if this happens and I'll take a look


Added blog post about omnivore import

Not very inspired, feel free to improve it.

Looks great. There's a wee typo in "joining" in the link to Omnivore's blog, but otherwise it's fine.


Disfunction typeahead on url input type

For input type="url" is not possible use typeahead.
If you click on item, this is ignored.
Plunkr: http://plnkr.co/edit/dgN6reY0N02vgx2FTyAu?p=preview

I'm not sure what the problem is here exactly, I tried the plunker and it seems to be working... Please retest against the latest version of angularJS and this project. If it is still a problem reopen this issue.


play_from_queue always makes call to self.avTransport.SetAVTransportURI

The play_from_queue function in soco.core always makes a call to self.avTransport.SetAVTransportURI. As the comment in the python code explains, this is to set the queue as the source URI on the device.
In some circumstances this may be a necessary step (e.g. if the device has been set to play from line-in), but they seldom apply.
The downside of this approach is that every time a user wishes to jump to another track in the queue, the queue is first reinstated on the device. This is an expensive operation which triggers additional events and causes delay. In an application which is monitoring events to keep the user interface updated, these additional events are undesirable.
For example, there will be an unnecessary transitional event that the queue position has jumped to the first track in the queue. A user interface monitoring these events will jump to the first track before jumping to the track being set by the call to play_from_queue.
As a result, I'm no longer using this soco function, but rather making a direct call to self.avTransport.Seek. This avoids the unnecessary overhead of reinstating the queue on the device.
This solves my problem, but I wonder if soco.core might be updated to address the concern. Could a check be made to see if the queue is already on the device before reloading it, so that reloading occurs only if the queue is not loaded? Or could there be another soco.core function (jump_to_position) which only calls self.avTransport.Seek to set the position? This would be consistent with the previous and next functions, which don't reinstate the queue.

Hi @relevitt,
if you are subscribed to the AV events then you know what is currently playing, but without subscription you would have to issue a request to find out if it was playing from the queue, which is probably a similar in impact to just setting it to the queue.
We could split of the seek part and make it callable - say, jump_to_track(5, play=True)  and also use it inplay_from_queue to avoid duplication. It would then be necessary to handle errors if it wasn't already playing from the queue.
Easy to do, adds an additional function to SoCO - what do others think?
Cheers David

Expect my PR.


Mesh as a FEniCS Mesh() object

Users should be able to give the measurements dx and ds based on mesh.

dx and ds measurements should be based on user-defined subdomains (FEniCS)

Done in #133


Error: Timeout - Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

$ ng --version

    _                      _                 ____ _     ___
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/

Angular CLI: 1.5.0
Node: 8.9.1
OS: win32 x64
Angular: 4.4.6
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router, tsc-wrapped

@angular/cli: 1.5.0
@angular-devkit/build-optimizer: 0.0.32
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.35
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0
@schematics/angular: 0.1.2
typescript: 2.3.4
webpack: 3.8.1

Repro steps.

run ng e2e.
wait for it to fail the test
run ng e2e.
when browser window comes up, resize the window.  Test will pass.

The log given by the failure.

$ ng e2e
** NG Live Development Server is listening on localhost:49152, open your browser on http://localhost:49152/ **
Date: 2017-11-08T22:26:15.552Z
Hash: d2d7aac667932ee3b422
Time: 22028ms
chunk {inline} inline.bundle.js, inline.bundle.js.map (inline) 5.83 kB [entry] [rendered]
chunk {main} main.bundle.js, main.bundle.js.map (main) 395 kB [initial] [rendered]
chunk {polyfills} polyfills.bundle.js, polyfills.bundle.js.map (polyfills) 217 kB [initial] [rendered]
chunk {styles} styles.bundle.js, styles.bundle.js.map (styles) 57.5 kB [initial] [rendered]
chunk {vendor} vendor.bundle.js, vendor.bundle.js.map (vendor) 14.4 MB [initial] [rendered]
(node:7916) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.

webpack: Compiled successfully.
[15:26:15] I/update - chromedriver: file exists C:\Users\jakeh\Documents\work\tdportal\node_modules\protractor\node_modules\webdriver-manager\selenium\chromedriver_2.33.zip
[15:26:15] I/update - chromedriver: unzipping chromedriver_2.33.zip
[15:26:16] I/update - chromedriver: chromedriver_2.33.exe up to date
[15:26:16] I/launcher - Running 1 instances of WebDriver
[15:26:16] I/direct - Using ChromeDriver directly...
Jasmine started
A Jasmine spec timed out. Resetting the WebDriver Control Flow.

  tdportal App
    × should display message saying Team Digital Portal
      - Error: Timeout - Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.
          at ontimeout (timers.js:475:11)
          at tryOnTimeout (timers.js:310:5)
      - Failed: Timed out waiting for asynchronous Angular tasks to finish after 30 seconds. This may be because the current page is not an Angular application. Please see the FAQ for mor
e details: https://github.com/angular/protractor/blob/master/docs/timeouts.md#waiting-for-angular
      While waiting for element with locator - Locator: By(css selector, header.header > div.header-content > div.header-logo > a.logo > span.header-title)
        (Session info: chrome=61.0.3163.100)
        (Driver info: chromedriver=2.33.506120 (e3e53437346286c0bc2d2dc9aa4915ba81d9023f),platform=Windows NT 10.0.15063 x86_64)
          at Object.checkLegacyResponse (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\error.js:505:15)
          at parseHttpResponse (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\http.js:509:13)
          at doSend.then.response (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\http.js:440:13)
          at <anonymous>
          at process._tickCallback (internal/process/next_tick.js:188:7)
      From: Task: Protractor.waitForAngular() - Locator: By(css selector, header.header > div.header-content > div.header-logo > a.logo > span.header-title)
          at Driver.schedule (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\webdriver.js:816:17)
          at ProtractorBrowser.executeAsyncScript_ (C:\Users\jakeh\Documents\work\tdportal\node_modules\protractor\lib\browser.ts:609:24)
          at angularAppRoot.then (C:\Users\jakeh\Documents\work\tdportal\node_modules\protractor\lib\browser.ts:643:23)
          at ManagedPromise.invokeCallback_ (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:1366:14)
          at TaskQueue.execute_ (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2970:14)
          at TaskQueue.executeNext_ (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2953:27)
          at asyncRun (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2813:27)
          at C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:676:7
          at <anonymous>
          at process._tickCallback (internal/process/next_tick.js:188:7)Error
          at ElementArrayFinder.applyAction_ (C:\Users\jakeh\Documents\work\tdportal\node_modules\protractor\lib\element.ts:482:23)
          at ElementArrayFinder.(anonymous function).args [as getText] (C:\Users\jakeh\Documents\work\tdportal\node_modules\protractor\lib\element.ts:96:21)
          at ElementFinder.(anonymous function).args [as getText] (C:\Users\jakeh\Documents\work\tdportal\node_modules\protractor\lib\element.ts:873:14)
          at TDPortalPage.getParagraphText (C:\Users\jakeh\Documents\work\tdportal\e2e\app.po.ts:9:113)
          at UserContext.<anonymous> (C:\Users\jakeh\Documents\work\tdportal\e2e\app.e2e-spec.ts:12:22)
          at new ManagedPromise (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:1067:7)
          at ControlFlow.promise (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2396:12)
          at TaskQueue.execute_ (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2970:14)
          at TaskQueue.executeNext_ (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2953:27)
          at asyncRun (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2860:25)
          at C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:676:7
          at <anonymous>
          at process._tickCallback (internal/process/next_tick.js:188:7)
      From: Task: Run it("should display message saying Team Digital Portal") in control flow
          at ControlFlow.emit (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\events.js:62:21)
          at ControlFlow.shutdown_ (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2565:10)
          at shutdownTask_.MicroTask (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2490:53)
          at MicroTask.asyncRun (C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:2619:9)
          at C:\Users\jakeh\Documents\work\tdportal\node_modules\selenium-webdriver\lib\promise.js:676:7
          at <anonymous>
          at process._tickCallback (internal/process/next_tick.js:188:7)
      From asynchronous test:
      Error
          at Suite.<anonymous> (C:\Users\jakeh\Documents\work\tdportal\e2e\app.e2e-spec.ts:10:3)
          at Object.<anonymous> (C:\Users\jakeh\Documents\work\tdportal\e2e\app.e2e-spec.ts:3:1)
          at Module._compile (module.js:635:30)
          at Module.m._compile (C:\Users\jakeh\Documents\work\tdportal\node_modules\ts-node\src\index.ts:392:23)
          at Module._extensions..js (module.js:646:10)
          at Object.require.extensions.(anonymous function) [as .ts] (C:\Users\jakeh\Documents\work\tdportal\node_modules\ts-node\src\index.ts:395:12)
          at Module.load (C:\Users\jakeh\Documents\work\tdportal\node_modules\coffee-script\lib\coffee-script\register.js:45:36)
          at tryModuleLoad (module.js:497:12)
          at Function.Module._load (module.js:489:3)
          at Module.require (module.js:579:17)
          at require (internal/module.js:11:18)
          at Array.forEach (<anonymous>)
          at Function.promise (C:\Users\jakeh\Documents\work\tdportal\node_modules\q\q.js:682:9)
          at _fulfilled (C:\Users\jakeh\Documents\work\tdportal\node_modules\q\q.js:834:54)
          at self.promiseDispatch.done (C:\Users\jakeh\Documents\work\tdportal\node_modules\q\q.js:863:30)
          at Promise.promise.promiseDispatch (C:\Users\jakeh\Documents\work\tdportal\node_modules\q\q.js:796:13)
          at C:\Users\jakeh\Documents\work\tdportal\node_modules\q\q.js:556:49
          at runSingle (C:\Users\jakeh\Documents\work\tdportal\node_modules\q\q.js:137:13)
          at flush (C:\Users\jakeh\Documents\work\tdportal\node_modules\q\q.js:125:13)
          at _combinedTickCallback (internal/process/next_tick.js:131:7)
          at process._tickCallback (internal/process/next_tick.js:180:9)

**************************************************
*                    Failures                    *
**************************************************

1) tdportal App should display message saying Team Digital Portal
  - Error: Timeout - Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.
  - Failed: Timed out waiting for asynchronous Angular tasks to finish after 30 seconds. This may be because the current page is not an Angular application. Please see the FAQ for more de
tails: https://github.com/angular/protractor/blob/master/docs/timeouts.md#waiting-for-angular
  While waiting for element with locator - Locator: By(css selector, header.header > div.header-content > div.header-logo > a.logo > span.header-title)

Executed 1 of 1 spec (1 FAILED) in 34 secs.
[15:26:52] I/launcher - 0 instance(s) of WebDriver still running
[15:26:52] I/launcher - chrome #01 failed 1 test(s)
[15:26:52] I/launcher - overall: 1 failed spec(s)
[15:26:52] E/launcher - Process exited with error code 1

Desired functionality.

My end-to-end test stopped passing and I found out that if I interact with the browser window that comes up, the test passes.  It's like the test doesn't know the window is there and ready until I manually interact with it.
Mention any other details that might be useful.

+1

I may assume it's related to interceptors. It doesn't work for me too.

I get this error:
Error: Timeout - Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.

I'm having the same issue with my application, but I can't reproduce it on a brand new application.  I'm not sure what causes this, but it doesn't seem to be a multiple monitor setup that causes it.  Just tested that.  In order for the e2e test to run, I have to click and drag the edge of the window to resize it.  Interacting with the window in any other way does not help the test run.
Any other ideas?

I still haven't been able to reproduce this in a different, more shareable application, even though the e2e tests have virtually not changed from what CLI gives me.  But here is an update to my corresponding StackOverflow question: https://stackoverflow.com/questions/48194109/ng-e2e-test-timesout

is this related too https://github.com/angular/angular-cli/issues/9466 ?

I was facing the same issue with chrome 65. In my case issue was resolved by updating chrome driver file to 2.37
You need to put chrome driver file here
Your_project_path/node_modules\protractor\node_modules\webdriver-manager\selenium

Closing it looks like the solution was posted above
If the problem persists after upgrading, please open a new issue, provide a simple repository reproducing the problem, and describe the difference between the expected and current behavior.


Updates to README

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@srirajan

Please sign the CLA here.

@martinb3 can we get this merged once @srirajan signs? The updates are around the new features.

Sweet, documentation!

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

LGTM.

I have signed the CLA now. Thanks


Add x-ms-enum.modelAsString=false to single value enums in Azure.ResourceManager.Storage

seal-single-value-enum-by-default in autorest.md is a temporary flag. When it stops working, enums with single value will no longer be treated as constants. To avoid this, add x-ms-enum.modelAsString=false to all enums that should remain constants, e.g.: e.g.:
https://github.com/Azure/autorest.csharp/blob/25d9b624ae1e6c5faf65fbb8176e718f85c6b4c8/test/TestProjects/BodyAndPath-LowLevel/BodyAndPath-LowLevel.json#L106 (in this example, however, parameter is still created cause EnumName2 has 2 values).

We have already done that in the latest api-version of storage (2021-08-01). Could you please confirm that when you are available @AlexanderSher ?
But unfortunately we did not change it in all the previous api-versions.


Twig Renderer creation & initial Template rendering Jest test

Jira
BDS-585
Summary
Adds @bolt/twig-renderer which exports a render() and renderString() function that can be passed a template and data then will return an HTML string. Introduces ability to create individual template rendering snapshot testing.
Details
See bolt-button/__tests__/button.js for how tests are created. In a nutshell, it's this:
const { render } = require('@bolt/twig-renderer');
const { readYamlFileSync } = require('@bolt/build-tools/utils/yaml');
const { join } = require('path');
const schema = readYamlFileSync(join(__dirname, '../button.schema.yml'));
const { tag } = schema.properties;

describe('button', async () => {
  test('basic button', async () => {
    const results = await<EMAIL_ADDRESS>{
      text: 'Hello World',
    });
    // console.log({ results });
    expect(results.ok).toBe(true);
    expect(results.html).toMatchSnapshot();
  });

  tag.enum.forEach(async tagChoice => {
    test(`button tag: ${tagChoice}`, async () => {
      const results = await<EMAIL_ADDRESS>{
        text: 'Hello World',
        tag: tagChoice,
      });
      // console.log({ results });
      expect(results.ok).toBe(true);
      expect(results.html).toMatchSnapshot();
    });
  });
});

How to test

A bolt build in apps/bolt-site must be ran first as this creates all the files in the data directory that the Bolt Core Twig Extension expects to find to fill bolt.data. You can see that the .boltrc.js at the top of the repo points to that same directory for reading files.
Run yarn test:js (which just runs jest) to fire off tests. A couple commands that may be of interest:

jest path/to/dir/ - will just run tests found in that dir; so we could make a test:components script with jest packages/components if we'd like. Or one could run yarn test:js packages/components/bolt-button to just run that single test.
jest -u will update any out of date snapshots with latest examples.

Lies! 🍰

@EvanLovely head’s up: our Nightwatch.js safety net that tests to verify Pattern Pab and the Docs sites compile and deploy successfully randomly stopped working very recently... likely due to an API change in Now.sh.

CC @margoromo and @remydenton since I mentioned this Nightwatch check breaking...

A bolt build in apps/bolt-site must be ran first as this creates all the files in the data directory that the Bolt Core Twig Extension expects to find to fill bolt.data. You can see that the .boltrc.js at the top of the repo points to that same directory for reading files.

Out of curiosity, what specifically do our Jest tests actually need in order to run? The Twig namespace JSON data (plus maybe some might need the global JSON data coming from our export data Sass function)?
Similar to how I’ve been refactoring much of the build to still work when tasks or functions are run in a different sequence than we originally planned for  (async / await), I’ve been working on reducing the amount of stuff we need to do before we can compile our Twig templates.
For example, rather than erroring out if a file requested by our Twig getData function, I recently added a less heavy handed “fileExists” check to grab a file if it’s there — and not worry about it if it’s not —> allows for more parallel Webpack and Twig compiles

^ a relatively easy solution could be to just have Webpack compile a watered down version of the .boltrc config — say with just the Bolt core package included to generate Sass exported data

Out of curiosity, what specifically do our Jest tests actually need in order to run? The Twig namespace JSON data (plus maybe some might need the global JSON data coming from our export data Sass function)?

Our Jest tests need to render a Twig template, and in order to do that the Bolt Core Twig Extension needs to be able to initialize. So here's the minimum:

A config file needs to be read so it can find out the list of components, and the path to the data directory can be found
The manifest needs to be created (grabbed via function but is written to full-manifest.bolt.json)
The twig namespaces config needs to be created (grabbed via function but is written to twig-namespaces.bolt.json)
Twig Env gets bootstrapped by declaring namespaces, then registering all Twig Extensions
When the BoltCore Twig Extension is initiated here, it looks in the data directory and reads any *.bolt.json and places them in the global bolt.data - this is how the components can get access to their own schema via the full manifest

All the scss created files (like breakpoints.bolt.json) could be absent, but if there were any Twig files that tried to access bolt.data.breakpoints, then it wouldn't find it. This should be good as those tend to only be used on the demo patterns for breakpoints, colors, etc. However, I'd suggest keeping the Twig Renderer using the environment that is close as possible to what gets rendered in that environment (static, pl, drupal, etc). So basically we could skip the webpack compile phase for a quicker start.

@sghoweri ok, I believe I've addressed all your questions; take a peak!

@sghoweri ok, I believe I've addressed all your questions; take a peak!

@EvanLovely I think you’re right. Thanks!
Any ideas on why the docs site can’t compile (on Travis and locally on this branch) — the problem I mentioned above? I think that’s the only thing currently holding this back!

Strange, if this line is commented out, then it works:
https://github.com/bolt-design-system/bolt/blob/97ddf1b9095d6f7cc30909c3487621ab41bdb347/packages/build-tools/tasks/task-collections.js#L153


evaluate_segmentation_results missing

Thanks for your great work, but when i am testing the model,  the evaluate_segmentation_results function is missing. Could you upload it at you convenience? Thank you very much.

Oh, another problem, is the function in utils.py (save_img and save_per_img) are same? save_per_img is also missing.

Hi,
Thank you for your interest in our work.
But now, I don't have time to maintain the code.
For the evaluate_segmentation_results function, please refer to https://github.com/ignaciorlando/refuge-evaluation/blob/b8fc1a8bd1bdd3eddd63bad8775ccfa4258488bb/evaluation_metrics/evaluation_metrics_for_segmentation.py#L213
Our function is inherited from their evaluation codes.
And for the 'save_img and save_per_img', please feel free to update it by yourself.
Hope this could help you.

Please refer #5 for more details. #


Longest Substring without repeating characters

this question should be in this list , please assign me this under Hacktoberfest. Thank you

Please assign me this issue

please assign this to me.Thank you

please assign me this I want to work on it.

please assign me this work , it will be my first contribution

I would like to request you to assign this project to me!

Hello, Can you please assign this issue to me?
Thanks in advance :)

Please assign me this issue.


Design for seed backup and  move new wallet seed verification to after setup

closes #461

This issue is moved to version 2 UI  #527


[BUG] Error 239 on interaction with feed item

Describe the bug
Get an 239 error on trying to interact with a tweet in my feed.
To Reproduce
Tweet in question: https://twitter.com/ironmouse/status/1418622317513646087
Steps to reproduce the behavior:

Go to Feed.
Click on a tweet's author or comments.
See 239 error.

Screenshots
Author:

Comments:

Device
Pixel 3a
Android 11
v2.13.0+20210716

I haven't run into this bug in newer versions, so I assume we're all good.


Suggestion : Adding safe versions of Operator functions.

Right now, Operator::Mult(const Vector &x, Vector &y) and related operations don't have guard rails to make sure that x and y are the right size. Some derived classes of Operator do their own verification, and some don't. This means that it's up to the users to make sure x and y have the right size by default unless they have extra knowledge of the derived class they're working with.
I think it would reduce cognitive load, make debugging easier and also reduce code to add safe versions of some these operations. For example, something like:
void Operator::SafeMult(const Vector &x, Vector &y) const
{
   MFEM_VERIFY(x.Size() == width, "input vector is not the correct size.");
   y.SetSize(height);
   Mult(x,y);
}

Then the user could reduce
x.SetSize(A.Width());
y.SetSize(A.Height());
A.Mult(x,y);

to
A.SafeMult(x,y);

I think it could help with other mfem functions too, since so many of them use Operator. For example, the function:
void ParGridFunction::ParallelProject(Vector &tv) const
{
   pfes->GetRestrictionMatrix()->Mult(*this, tv);  // could use SafeMult;
}

currently does not check if tv is the right size, or do the resizing, which may cause a segfault. For the user to do resizing themselves, they have to figure out how to get the size of the fespace in true dofs and set tv to that beforehand, whereas the SafeMult version does it automatically. I'm not suggesting this should be done everywhere as I'm sure there is code that relies on the current behavior of Mult, and also may not be good for performance reasons with something like DenseMatrix which is called a lot in hot loops.
Does this seem like a reasonable idea? If so, I could draft up a PR. I also wasn't sure if calling SetSize on y would have any implications for syncing memory between host and device, and if SafeMult would need to be more complicated to account for this.

It is a good suggestion :wink:, but I am not sure we should implement it. It is a basic user's responsibility to ensure that and motivates him/her to consider basic optimizations related to that like preallocating the vectors or so. Secondly, the interface of Operator is larger, so it is not really systematic to add another version of only this method. Another thing is that Safe is not that much self-explaining really and can be interpreted in different ways, so it would rather confuse users, I think 🤔 . Maybe with a better name, but still I am not sure how many people would use that really. Internal usage is then questionable again and entails some extra overhead. However, I have experienced many segfaults or aborts coming exactly from this, so it might be a good idea to do something about it 💣 😅 . I would rather suggest to systematically go through the code and identify the places that are unguarded (but mind that sometimes that guards are done at a lower level) and add there MFEM_ASSERT, which does not entail any performance hit for the production codes :wink:.


Adding instagram

I really need instagram to be part of the share buttons on my website. Is there a fix for this?

It's currently impossible to share on Instagram outside of the native applications.


Selecting minute hand non-intuitive

I was playing around with demo page for a while to include it in one of my projects.
The minute selection part of timepicker was not very intuitive atleast for me. I  tried in vain to double click, drag on minutes hand to select the minutes. (Finally a colleague of mine figured it out)
Can you make it similar to default android behaviour,

where only minute hand is invisible while selecting the hour hand and vice-versa
And also instead of clicking ok to switch between hour and minute hand, auto switch to minutes hand once hour is selected.

I'm sitting here trying to get it work with no luck. How exactly do you use the minute hand?

Click on OK button below

Of course! I was assuming I was expected to interact with the minute hand before clicking OK.

@pos1tron You can set switchOnClick to true if you want view to change without clicking "ok" button

Thanks! And thanks for your hard work making this repo a reality, it's a
great tool.
On Thu, Feb 11, 2016 at 5:40 PM, T00rk<EMAIL_ADDRESS>wrote:

@pos1tron https://github.com/pos1tron You can set switchOnClick to
true if you want view to change without clicking "ok" button
—
Reply to this email directly or view it on GitHub
https://github.com/T00rk/bootstrap-material-datetimepicker/issues/47#issuecomment-183088417
.

--

Thank you for the config option


Raise keyboard events using special keys

Pull request description
Implements #101.
Added KeyDown and KeyUp helper methods to raise keyboard events with special keys. Available special keys are represented by Key enum. Special keys can be combined with control keys: Key.Enter | Key.Alt. KeyPress helper method is not added, because this event does not emit special keys.
Key enum is subset of values from Selenium Keys class. Implementation of KeyEventFactory is based on real values in Chrome browser. I created small component that displays KeyboardEventArgs values. Then simple Selenium application collected the values for all special keys.
Documentation has not been updated. Documentation of API reference should be enough.
PR meta checklist

[x] Pull request is targeting at DEV branch.
[x] Pull request is linked to all related issues, if any.
[x] I have read the CONTRIBUTING.md document.

Content checklist

[x] My code follows the code style of this project and AspNetCore coding guidelines.
[x] My change requires a change to the documentation.

[x] I have updated the documentation accordingly.

[x] I have updated the appropriate sub section in the CHANGELOG.md.
[x] I have added, updated or removed tests to according to my changes.

[x] All tests passed.

Thanks for this @duracellko. I hope I get some time to day to review and provide feedback. Might first be tomorrow though.

Open question: Current implementation does not return Task to wait for event dispatcher. Should this be changed to Async method?

Open question: Current implementation does not return Task to wait for event dispatcher. Should this be changed to Async method?

Short answer, no.
Longer answer: I am actually thinking about removing the "async" versions of the event dispatcher helper methods, as they are not useful at all as far as I can see, and just a leftover from when I imported some of Steves testing prototype code, and didn't understand what was going on.
If the event handler is async and is not done processing when the dispatcher returns, it is most likely waiting for data from the user, e.g. the test code, so awaiting it there does nothing. Removing the async methods would simply makes things less confusing to folks.

I do have some concerns with the current implementation, that I think we need to address:

I do not see a way I can pass in a regular key, e.g. "A", nither by itself or in a combination with a control key, such as CTRL+A.

With regular keys, it's hard to tell, where to stop. Letters and numbers? What special characters? Currency symbols? Non-english letters? Maybe it can be cleared up after point 3.

We need to include Mac keyboard keys as well (isnt it called the option key?).

I don't have Mac, I cannot verify, how Option key behaves. Isn't it emulated as Alt key?

I am not certain of the intuitiveness of using the | to indicate multiple pressed keys. To me, the | operator indicates "or" operation. A more correct operator would be the & "and" operator. But I actually think it is more intuitive if we simple allow users to pass param Key[] keys in. That would look like this: KeyDown(Key.Enter, Key.Control).
It would be even nicer if it was possible pass CTRL+A like this: KeyDown(Key.Control, "a"). I think that should be possible if Key was a struct/class instead of an enum. Each of the control characters would be a static property on it, and then it would also need an implicit conversion from string, making it possible to both pass Key and a string to the same param Key[] keys list, as in the example.

What do you think about that?

I think separating of input to key and key-modifier might be good idea. It removes that weird key-bit arithmetic that works. It is a bit weird, because it is possible to do Key.Enter | Key.Escape, but it's not valid option.
Separating key and key modifiers also enables more cases to simulate. For example, pressing Control, while holding Shift, is different than pressing Shift, while holding Control. I realized it's not possible to express both cases with current implementation.
I see following option:
Option 1
Use enums for key and key modifiers.
// Represents key being pressed or released
public enum Key
{
  Enter,
  Escape,
  ...
  Control,
  Alt
}

// Represents key being hold down, when event occured.
[Flags]
public enum KeyModifiers
{
  Control,
  Shift,
  Alt
}

public void KeyDown(this IElement element, Key key, KeyModifiers keyModifiers)

// Example:
element.KeyDown(Key.Enter, KeyModifiers.Control | KeyModifiers.Shift);

Option 2
Use struct for KeyModifiers.
public struct KeyModifiers
{
  public KeyModifiers(bool control, bool alt, bool shift)
  {
    Control = control;
    Alt = alt;
    Shift = shift;
  }
  public bool Control { get; }
  public bool Alt { get; }
  public bool Shift { get; }

  // Implement Equals and GetHashCode
}

// Same Key type as in Option 1
public void KeyDown(this IElement element, Key key, KeyModifiers keyModifiers)

// Example:
element.KeyDown(Key.Enter, new KeyModifiers(true, false, true));

Option 3
Use parameters. It's not much different from the generic KeyDown method.
// Same Key type as in Option 1
public void KeyDown(this IElement element, Key key, bool controlKey = false, bool altKey = false, bool shiftKey = false)

// Example:
element.KeyDown(Key.Enter, controlKey:true, shiftKey: true);

Remarks
I wouldn't go with array as input. Consumer might do element.KeyDown(Key.Enter, Key.Escape), but that does not make sense. Therefore explicit splitting key and key modifiers is probably better way.

I like your thought about trying to make something work that the compiler will complain about. But I want to consider just doing runtime validation if it means providing a simpler API:
Something like this could work, with the runtime of the valid combinations:
using Microsoft.AspNetCore.Components.Web;

namespace KeyNameSpace
{
    public class KeyCombination
    {
        // This class contains validation logic in ctor that checks
        // if it (the key combination) is legal.

        public static KeyCombination operator +(KeyCombination a, Key b) => new KeyCombination();
        public static implicit operator KeyboardEventArgs(KeyCombination key) => new KeyboardEventArgs();
    }

    public class Key
    {
        public string Value { get; }
        public string Code { get; }

        private Key(string key, string code)
        {
            Value = key;
            Code = code;
        }

        public static Key Enter { get; } = new Key("Enter", "Enter");
        public static Key Control { get; } = new Key("Control", "ControlLeft");
        // ... add the rest of the special keys here

        public static implicit operator Key(char key) => new Key(key.ToString(), key.ToString());
        public static KeyCombination operator +(Key a, Key b) => new KeyCombination();
        public static implicit operator KeyboardEventArgs(Key key) => new KeyboardEventArgs();
    }
}

namespace UserNameSpace
{
    using KeyNameSpace;
    using static KeyNameSpace.Key;

    public class Dummy
    {
        public static void KeyUp(KeyboardEventArgs args) {}

        public static void Blah()
        {
            KeyUp(Control + Enter);
            KeyUp(Control + Enter + 'x');
            KeyUp(Control + 'x');
            KeyUp(Enter);
            KeyUp(Enter);
        }
    }
}

I like this because:

The plus operator makes it obvious that it is a combination that is being provided.
The using static trick, which makes the code very expressive I think.
This also has the added benefit that we do not need to add new methods to the keyboard event  event dispatcher helpers, as the implicit conversion to KeyboardEventArgs  does the work for us.
Use of chars to ensure folks dont send in more than one char (or is that incompatible with certain languages?)

Obviously more properties could needs to be added to the Key and KeyCombination class. Its just stubs.
What do you think?

Good idea. I will try to update PR later.
I assume that Control + Enter + 'x' is supposed to throw exception.

Good idea. I will try to update PR later.
I assume that Control + Enter + 'x' is supposed to throw exception.

Only if it is an invalid key combination. It was just an example of a combination, did not think about its validity or not :)

I created new Key class. Let me know, what you think.
Tests are still in progress.

I finished the implementation and testing.

I finished the implementation and testing.

Sorry for the radio silence. Been super busy last few days. Will take a look asap this weekend and provide feedback.

Comments are addressed. Also conflicts resolved. Hope I didn't miss anything.

Almost there, good work 👍
You need to build the bUnit locally in release mode and fix the build issues. In release mode all warnings are treated as errors, and we cant merge with build errors.
Other than that, I only have two more suggestions, adding an extra implicit conversion from string to key, and consider making the Get methods private.
Ps. my build script is bit messed up right now, but these three checks should pass:

I commented on the problem with build above:
x is not null is not recognized as checking for null and it causes code analysis warning. Should I revert the syntax back to !(x is null) or add exceptions to warnings?
Just checked, the warning should be fixed in next version of FxCop. https://github.com/dotnet/roslyn-analyzers/pull/4060

I commented on the problem with build above:

My bad. Didnt see that.

x is not null is not recognized as checking for null and it causes code analysis warning. Should I revert the syntax back to !(x is null) or add exceptions to warnings?
Just checked, the warning should be fixed in next version of FxCop. dotnet/roslyn-analyzers#4060

Leave the warnings in there for now, I will investigate and apply a fix locally.

Update:

Added repeat and type parameters to KeyXXX methods.
Added conversion from string to Key.
Added suppress warning to + operator.

My decision is to keep Get method public


Android Gradle experimental: migrate to stable v2.2.0+

See doc:
http://tools.android.com/tech-docs/new-build-system/gradle-experimental/migrate-to-stable
https://developer.android.com/studio/projects/add-native-code.html#link-gradle
See existing Gradle configs:
https://github.com/readium/SDKLauncher-Android/blob/develop/SDKLauncher-Android/app/build.gradle#L1
+
https://github.com/readium/SDKLauncher-Android/blob/develop/SDKLauncher-Android/app/build_experimental.gradle
https://github.com/readium/readium-sdk/blob/develop/Platform/Android/epub3/build.gradle#L3
+
https://github.com/readium/readium-sdk/blob/develop/Platform/Android/epub3/build_experimental.gradle

PS: there are ongoing Gradle / JNI experiments in the feature/lcp and feature/lsd branches, where native build configs are imported from the readium-sdk and readium-lcp-client codebase (whereas the master and develop branch only import the readium-sdk ePub3 static *.so lib).

Closing this as the Gradle configs are correctly maintained across develop and feature/lcp branches.


[IOTDB-2844] Implementation of SeriesAggregateScanOperator and AggregateOperator - Part I

Implementation of SeriesAggregateScanOperator:

init SeriesAggregateScanOperator from SeriesAggregateScanNode
implement the function of SeriesAggregateScanOperator
Init TimeRangeIterator from groupByParameter.
Unit test of SeriesAggregateScanOperator

Leave for next PR:

Implement AggregateOperator.

Coverage increased (+0.0008%) to 55.188% when pulling f0982c560629d1df2c74ba8cfaba3a8ad04167da on seriesAggregateOperator into 2691ade495513b54f00810c6cdd97c08326293e5 on master.


alwaysOn capability doesn't seem to work for devices discovered via enableDiscovery callback

I was trying to use the alwaysOn capability, but it seems that it's not being propagated to devices that are discovered.

I couldn't reproduce with 0.49.2.
Which version of the SDK are you using?
Can you also show some code where the issue occur?

Can you be a bit more specific about the issue, and also show some code to reproduce it?

close because no feedback, feel free to reopen @bpruessm if this is still an issue.


Add or update CI tests to use Python 3.11

Python 3.12 is due out in a month. It would be great for dscim to follow numpy and try to support the two most recent python releases as best as it can. CI for other maintained CIL packages use the most recent Python 3 release. Currently, CI for dscim only tests on Python 3.10. Let's either update our tests in CI to use Python 3.11 or have it start testing with both Python 3.10 and 3.11. Testing against a single Python version is likely to be the simplest solution.
It would be a matter of changing this line in our CI config.

Could test against Python 3.12 now, too.