Version: Add parser for additional hyphenated filenames

[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.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

The version for the luajit-openresty formula is currently parsed from the stable URL (https://github.com/openresty/luajit2/archive/refs/tags/v2.1-20210510.tar.gz) as 20210510 instead of 2.1-20210510 (see Homebrew/homebrew-core#78791). This should have used version to specify the full version that wasn't being properly parsed when the formula was added but that wasn't addressed at the time. [Other formulae with a similar version format run into the same parsing issue and they use version because of it.]
We could simply add version "v2.1-20210510" to luajit-openresty (doing what formulae with a similar version format have done) but I figured it would be better to support this version format internally (i.e., fixing this issue for both existing and future formulae without needing to use version).

This PR addresses this by adding a StemParser to Version::VERSION_PARSERS that matches versions that start with a numeric version like 1.2/v1.2, 1.2.3/v1.2.3, etc. followed by a hyphen and a trailing number (e.g., -12), numeric version (e.g., -0.2), date (e.g., 20210605), etc. This allows these versions to be correctly parsed without needing to use version in the formula/cask.
This not only fixes the version parsing for luajit-openresty but also for six formulae that are currently using  version due to this shortcoming in the current version parsing logic. This would allow us to remove version from the following formulae: bandcamp-dl, inxi, jnethack, luv, redir, youtube-dlc.

The only other formula or cask that this affects is the ammonite-repl formula, which uses a filename like 2.13-2.3.8, where the numeric version before the hyphen is a Scala version and the numeric version after the hyphen is the ammonite-repl version. The aforementioned version parsing bug was unintentionally producing the correct version for this formula because the leading 2.13- was treated like a software name prefix (e.g., brew-) and omitted from the capture group.
Accounting for filenames like v0.0.8-12.tar.gz means that we have to use version in ammonite-repl to manually isolate the formula version. This would need to be done before this PR could be merged.
It's worth mentioning that there are other formulae that have a leading Scala version in the numeric portion of the filename (akka, giter8, scalastyle, kafka) but they're unaffected by the change in this PR because they all have a software name prefix before the numeric version (e.g., akka_2.11-2.4.20.zip) and the added parser in this PR only matches filenames without one.

Review period will end on 2021-06-08 at 00:00:00 UTC.

I'm personally not convinced this is needed and forces us to carry around version_scheme in luajit-openresty when there's zero need for it.

To me, we either correctly parse the version or we don't. At the moment, we're not correctly parsing the version for luajit-openresty and this is only due to a shortcoming in Version's parsing logic (i.e., this isn't an intentional decision).
The trailing date may be sufficient for the foreseeable future (due to upstream's current version constraints) but it's not the full version. Upstream may currently hold the position that they won't release a new major/minor but I don't see any reason to explicitly rely on this, as there's nothing to prevent it from changing in the future. It may be unlikely but it's not impossible.
The only benefit I see in leaving this alone is that we wouldn't have to bump the version_scheme for luajit-openresty. Maybe I'm being naive but having to bump the version_scheme in one formula to fix this doesn't seem like an undue burden.

It also currently applies to only one formula.

Looking at it from a different perspective, this is one outlier where brew isn't reporting the correct, full version and this change would bring it in line with the majority.

I've updated this to use a more general regex (not specifically tied to a date-based suffix) and this now fixes version parsing for six other formulae as well: bandcamp-dl, inxi, jnethack, luv, redir, youtube-dlc. These currently use version in the formula because the previous Version parsing behavior didn't produce the correct, full version.
This means that we could remove version from these formulae and CI is predictably failing in "tap_syntax (Linux)" with errors like:
Error: 6 problems in 6 formulae detected
Error: Stable: version 0.0.8-12 is redundant with version scanned from URL
Error: Stable: version 3.3.04-1 is redundant with version scanned from URL
Error: Stable: version 3.6.6-0.2 is redundant with version scanned from URL
Error: Stable: version 1.41.0-0 is redundant with version scanned from URL
Error: Stable: version 2.2.1-9 is redundant with version scanned from URL
Error: Stable: version 2020.11.11-3 is redundant with version scanned from URL
Error: Process completed with exit code 1.

Past that, ammonite-repl is parsed differently due to the change in this PR and we would need to use version in the formula to ensure the correct version continues to be used. A more lengthy explanation of this situation can be found in the updated PR description but this is expected, as ammonite-repl uses an unusual filename format.

Review period ended.

How do we want to handle the Stable: version X is redundant with version scanned from URL errors here and in the ammonite-repl PR (Homebrew/homebrew-core#78878)?
To maintain the correct version, the ammonite-repl PR would need to be merged before this brew PR but it will fail CI until this is merged. Similarly, this brew PR fails CI because it fixes Version parsing for six formulae, making their existing version statements redundant. Do we simply accept that CI will fail or is there another way around this?
The luajit-openresty PR that bumps the version_scheme (Homebrew/homebrew-core#78955) will need to be merged after this brew PR. I'll also create a follow-up homebrew/core PR to remove the redundant version statements afterward. Am I correct in assuming we'll need to wait to merge these particular PRs until this change is in a tagged brew release?

To maintain the correct version, the ammonite-repl PR would need to be merged before this brew PR but it will fail CI until then. Similarly, this brew PR fails CI because it fixes Version parsing for six formulae, making their existing version statements redundant. Do we simply accept that CI will fail or is there a way to get these PRs to pass?

Yes, there's a bit of a chicken-and-egg here that I'm not sure how best to work around. My suggestion would be to amend brew audit so that the is redundant with version scanned from URL is not output when running brew audit on an entire tap but only on a single formula. This would allow this PR to be merged. Thoughts?

My suggestion would be to amend brew audit so that the is redundant with version scanned from URL is not output when running brew audit on an entire tap but only on a single formula. This would allow this PR to be merged.

I tinkered with this and the way I approached it was to modify ResourceAuditor to allow for only and except options, borrowing related code from FormulaAuditor. This allows the brew audit --only and --except options to apply to ResourceAuditor, not just FormulaAuditor.
[One thing to note is that an audit with the same name in FormulaAuditor and ResourceAuditor would both be affected by --only or --except. There aren't any overlapping names right now but, depending on the audit, it could be either desirable or undesirable behavior.]
diff --git a/Library/Homebrew/formula_auditor.rb b/Library/Homebrew/formula_auditor.rb
index 06b54db3c0a1ac8dd5b34f6ba39271f2e42e7da6..19be6c03b58503e61f56b061e074e691b9a26c08 100644
--- a/Library/Homebrew/formula_auditor.rb
+++ b/Library/Homebrew/formula_auditor.rb
@@ -500,7 +500,7 @@ module Homebrew
         spec_name = name.downcase.to_sym
         next unless (spec = formula.send(spec_name))

-        ra = ResourceAuditor.new(spec, spec_name, online: @online, strict: @strict).audit
+        ra = ResourceAuditor.new(spec, spec_name, online: @online, strict: @strict, only: @only, except: @except).audit
         ra.problems.each do |message|
           problem "#{name}: #{message}"
         end
@@ -508,7 +508,7 @@ module Homebrew
         spec.resources.each_value do |resource|
           problem "Resource name should be different from the formula name" if resource.name == formula.name

-          ra = ResourceAuditor.new(resource, spec_name, online: @online, strict: @strict).audit
+          ra = ResourceAuditor.new(resource, spec_name, online: @online, strict: @strict, only: @only, except: @except).audit
           ra.problems.each do |message|
             problem "#{name} resource #{resource.name.inspect}: #{message}"
           end
diff --git a/Library/Homebrew/resource_auditor.rb b/Library/Homebrew/resource_auditor.rb
index f18652715fcfc4ddda5958271e49990a23742f60..bfc71517ca06d6cd0f3edde5a63d610d3192455e 100644
--- a/Library/Homebrew/resource_auditor.rb
+++ b/Library/Homebrew/resource_auditor.rb
@@ -20,14 +20,25 @@ module Homebrew
       @spec_name = spec_name
       @online    = options[:online]
       @strict    = options[:strict]
+      @only      = options[:only]
+      @except    = options[:except]
       @problems  = []
     end

     def audit
-      audit_version
-      audit_download_strategy
-      audit_checksum
-      audit_urls
+      only_audits = @only
+      except_audits = @except
+
+      methods.map(&:to_s).grep(/^audit_/).each do |audit_method_name|
+        name = audit_method_name.delete_prefix("audit_")
+        if only_audits
+          next unless only_audits.include?(name)
+        elsif except_audits
+          next if except_audits.include?(name)
+        end
+        send(audit_method_name)
+      end
+
       self
     end

With these changes in place, we could handle this issue in one of two ways:
A) Modify dev-cmd/audit.rb to automatically exclude ResourceAuditor#audit_version when brew audit --tap is used:
diff --git a/Library/Homebrew/dev-cmd/audit.rb b/Library/Homebrew/dev-cmd/audit.rb
index b7e1bfb433ba2a80e597dbe6cb33dfdf83ea8c90..b97aaf0fe80b0e7df099ba365027ef8d2fee1250 100644
--- a/Library/Homebrew/dev-cmd/audit.rb
+++ b/Library/Homebrew/dev-cmd/audit.rb
@@ -182,7 +182,16 @@ module Homebrew
         display_cop_names:    args.display_cop_names?,
       }.compact

-      fa = FormulaAuditor.new(f, **options)
+      fa = if args.tap
+        # Skip ResourceAuditor#audit_version when auditing a tap
+        FormulaAuditor.new(
+          f,
+          except: args.except&.union(["version"]) || ["version"],
+          **options
+        )
+      else
+        FormulaAuditor.new(f, **options)
+      end
       fa.audit

       if fa.problems.any? || fa.new_formula_problems.any?

The downside is that this applies wherever brew audit --tap is used, not just on CI. I feel like maintainers/contributors should be able to surface redundant versions when running brew audit --tap locally and the exclusion should only happen on CI/test-bot. That is to say, we shouldn't have to run brew audit on specific formulae to get the version audit.
B) Modify related GitHub Actions workflows and test-bot to use the --except=version option for brew audit steps (e.g., the "Run brew audit --skip-style on all taps" step of the Homebrew/brew tests.yml workflow). This only impacts CI/test-bot and doesn't affect local brew audit --tap= behavior, so this may be a better solution if we want to keep this around long term.
Homebrew/brew:
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 12c6f8444b8f37914a263f04b9d77b4a356f8547..e77235d71046829640cdb2532d1a4d7322581a23 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -78,7 +78,7 @@ jobs:
         run: brew style --display-cop-names homebrew/core

       - name: Run brew audit --skip-style on all taps
-        run: brew audit --skip-style --display-failures-only
+        run: brew audit --display-failures-only --skip-style --except=version

       - name: Set up all Homebrew taps
         run: |

test-bot:
diff --git a/lib/tests/tap_syntax.rb b/lib/tests/tap_syntax.rb
index 661638e6243df2a5b6f34fbd7ddfa180c7058a7e..482076a7e427a6cf62dec4904769f78717528b86 100644
--- a/lib/tests/tap_syntax.rb
+++ b/lib/tests/tap_syntax.rb
@@ -12,7 +12,7 @@ module Homebrew
                                 MacOS.active_developer_dir == "/Applications/Xcode.app/Contents/Developer"
         test "brew", "style", tap.name unless broken_xcode_rubygems

-        test "brew", "audit", "--tap=#{tap.name}"
+        test "brew", "audit", "--tap=#{tap.name}", "--except=version"
       end
     end
   end

What do you think about these approaches?
Depending on how we handle this, these audit changes don't necessarily have to stick around long-term. However, if we revert the changes after merging related PRs, we would have to do this dance again the next time we modify Version parsing.
That said, I'll create a PR once we decide on how to approach this and we can go from there.

B) Modify related GitHub Actions workflows and test-bot to use the --except=version option for brew audit steps (e.g., the "Run brew audit --skip-style on all taps" step of the Homebrew/brew tests.yml workflow). This only impacts CI/test-bot and doesn't affect local brew audit --tap= behavior, so this may be a better solution if we want to keep this around long term.

This feels like the best approach to me 👍🏻

That said, I'll create a PR once we decide on how to approach this and we can go from there.

Perfect, thanks @samford. I love how well you spelled out both of these options. You're making my life easy here 😆🏝

You're making my life easy here 😆🏝

I feel like I make enough work for you in PR review, so I try to make it easy when I can, ahah.

brew and test-bot PRs are now up: #11542, Homebrew/homebrew-test-bot#617.

I feel like I make enough work for you in PR review, so I try to make it easy when I can, ahah.

Haha, I feel more guilty about my PR reviews. Thanks again @samford ❤

With #11542 merged, this now passes CI but we can't merge it until the ammonite-repl PR (Homebrew/homebrew-core#78878) is merged. We can't merge that until the CI-syntax-only label is working again in homebrew/core (it broke with the changes made to .github/workflows/test.yml earlier today), as a full CI run will fail without the additional test-bot change I omitted in response to feedback.
So close!

The CI-syntax-only fix was merged and the homebrew/core PR to preemptively add a version to ammonite-repl passed CI after being rebased, so that's now merged. I'm going to go ahead and merge this PR now, so it will be included in the next tagged brew version.
Once it's in a tagged version, I'll merge the luajit-openresty PR (Homebrew/homebrew-core#78955) and create a PR to remove the newly-redundant version calls from the aforementioned formulae.


refactor: theme types to use csstype

BREAKING CHANGE: changes to exported types

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Suppress instantiate output in demo.ipynb

The docs are now built automatically, but unfortunately instantiate https://deltares.github.io/Ribasim.jl/demo.html produces a lot of noise.
Any idea how to suppress this @visr or @evetion?

Pkg.instantiate(; io=devnull) should work
https://pkgdocs.julialang.org/v1/api/#Pkg.instantiate
I see that GLMakie gives an error in the docs. This package needs a GPU. We could simply take it out of the run environment, or perhaps it is better to add a docs environment that has only the packages needed for the docs. Because GLMakie is not used in the demo notebook, but CairoMakie is.

Pkg.instantiate(; io=devnull) should work

👍🏻

perhaps it is better to add a docs environment that has only the packages needed for the docs

I was a bit hesitant to add a docs environment, because that means yet another environment to maintain, but maybe it's indeed the best option.


Why the overlap for AOs differ from direct calculation?

I run the below code and get the overlap matrix for AOs
import pyscf.gto molecule = pyscf.gto.M(     atom = 'H 0 0 0; H 0 0 2',     basis = "sto-3g", ) print(molecule.intor('int1e_ovlp'))
the result
[[1.         0.11921653]
[0.11921653 1.        ]]
Then I directly calculate the overlap between two 1s basis for two H with 2A distance (using the mathematica software), code is below, the STO-3G exponent and coefficient is from basis-set-exchange website and identical to the PySCF's (using pyscf.gto.basis.parse_nwchem.convert_basis_to_nwchem('H', pyscf.gto.load('sto3g', 'H')) verified)
Clear["Global*"]
a1 = 3.425250914;
c1 = 0.1543289673;
a2 = 0.6239137298;
c2 = 0.5353281423;
a3 = 0.1688554040;
c3 = 0.4446345422;
[Phi][x_, y_, z_] :=
c1 ((2 a1)/[Pi])^(3/4) E^(-a1 (x^2 + y^2 + z^2)) +
c2 ((2 a2)/[Pi])^(3/4) E^(-a2 (x^2 + y^2 + z^2)) +
c3 ((2 a3)/[Pi])^(3/4) E^(-a3 (x^2 + y^2 + z^2))
Integrate[[Phi][x, y, z] [Phi][x - 2, y,
z], {x, -[Infinity], [Infinity]}, {y, -[Infinity],
[Infinity]}, {z, -[Infinity], [Infinity]}]result is0.462778 - 1.43876*10^-16 I`
I don't know why.

sorry, here is the mathematica code

Because the default unit for atom coordinates is angstrom. If you set unit='bohr' in gto.M(), the mma result can be reproduced.

I also want to ask, how PySCF deal with the infinite Coulomb and exchange series in PBC-HF calculation, are there some reference to learn this?

Thank you very much!


Fix setting resource prefix

Resource prefix was not set correctly.
I also updated readme of the build crate to make it similar to the main readme.
With this things work as expected, should be ready for release.

0.10.0-beta.0 is now available on crates.io

Can you publish relm4-icons-build too?

Both published, now also under the correct version 0.10.0-beta.1


Unable to provision a virtual machine as tenant administrator

Occurs in: Euwe RC1 (tested with a tenant account on the Global Region, provider VMware vCenter)
Steps to reproduce the issue:

Login as user having the role EvmRole-tenant_administrator
Go to Compute -> Infrastructure -> Virtual Machines;
Select Lifecycle -> Provision VMs
Enter the VM properties
Click the Submit button

The activity indicator is showing for a second but after that nothing happens.
The following lines are written to production.log
[----] I, [2016-11-03T14:26:00.186931 #3267:13adef4]  INFO -- : Started POST "/vm_infra/pre_prov_continue?button=submit" for <IP_ADDRESS> at 2016-11-03 14:26:00 +0100
[----] I, [2016-11-03T14:26:00.189639 #3267:13adef4]  INFO -- : Processing by VmInfraController#pre_prov_continue as JS
[----] I, [2016-11-03T14:26:00.189751 #3267:13adef4]  INFO -- :   Parameters: {"button"=>"submit"}
[----] F, [2016-11-03T14:26:00.317476 #3267:13adef4] FATAL -- : Error caught: [RuntimeError] Invalid System Authentication Token specified

@carbonin More central admin stuff.  I'm pretty sure Invalid System Authentication Token specified comes from that effort.

Yeah, it sounds like you didn't properly set up the central administration authentication.
This can be done on the same page as the replication subscriptions on the global region appliances.
The authentication needs to be configured once for each remote region.

Our guess is that it could be another issue in its own right (i.e. not having to deal with setting up C-A authentication), since we are able to do certain Central Admin things, from the Global Region in other regions.

The issue here is that the user needs to be created on the remote region as well as the global region. We use the currently logged in user for API authentication so if the remote region doesn't have a record of the currently logged in user this is the error you will get.


Why is Dwarf::borrow deprecated?

See title.
Introduced by e6e90c9a26021bda4bffe22f0df69a412942763f, which states:

Also replace Dwarf::borrow with DwarfSections::borrow for consistency.

I looked for it, but couldn't find said consistency. Now we have borrow on DwarfSections and...nothing for Dwarf. What if Dwarf has a sup set and I want to borrow that? Seems now I am forced to DwarfSections::borrow_with_sup again, which allocates a new Arc with a sup which seems consistently worse than what was possible beforehand, right? Can we please consider reverting the deprecation?

I looked for it, but couldn't find said consistency

For consistency with how DwarfPackage needs to borrow from DwarfPackageSections, which is the primary addition in that commit. It also means users no longer need to create a Dwarf from fields, so adding more fields to Dwarf won't break them.

Seems now I am forced to DwarfSections::borrow_with_sup again, which allocates a new Arc with a sup which seems consistently worse than what was possible beforehand, right?

That was the intent. Dwarf::borrow also allocate a new Arc. How is that consistently worse?

Can we please consider reverting the deprecation?

Sure, but I'll need to understand the reason why it is needed.

Seems now I am forced to DwarfSections::borrow_with_sup again, which allocates a new Arc with a sup which seems consistently worse than what was possible beforehand, right?

That was the intent. Dwarf::borrow also allocate a new Arc. How is that consistently worse?

Oh lol. I skimmed the code and read it as doing a cheap Arc::clone.
In any case, it's worse because users are forced to do the same borrow_with_sup dance twice.

Can we please consider reverting the deprecation?

Sure, but I'll need to understand the reason why it is needed.

It's necessary for when a second Dwarf instance is needed because the first one has already been consumed, for example.

I still don't understand. Can you demonstrate with some code?
For example, here's what I think the two alternatives are, using code from the documentation examples:
let dwarf = owned_dwarf.borrow(|section| {
    gimli::EndianSlice::new(&section, gimli::LittleEndian)
});

let dwarf = dwarf_sections.borrow_with_sup(&dwarf_sup_sections, |section| {
    gimli::EndianSlice::new(&section, gimli::LittleEndian)
});

The only difference I can see is an extra argument, which seems like a trivial difference to me.

For an argument in favor of DwarfSections::borrow_with_sup, it lets you store the two DwarfSections in the same place, so it actually avoids an extra Arc.

Can you demonstrate with some code?

Here is an example:
    let sections = gimli::DwarfSections::load(/* .. */).unwrap();
    let dwarf = if let Some(...) = /* maybe-some-sup-object */ {
        let sup_secs = gimli::DwarfSections::load(/* load sections from sup object */).unwrap();
        sections.borrow_with_sup(&sup_secs, |b| *b)
    } else {
        sections.borrow(|b| *b)
    };

    let ctx = addr2line::Context::from_dwarf(dwarf).unwrap();

    // Down here we may need a `Dwarf` again. So now we need to keep the `sup_secs` around,
    // recreate everything again with the conditional `sections.borrow_with_sup` dance, instead
    // of just being able to pass `dwarf.borrow(|b| *b)` to `addr2line::Context::from_dwarf`

Ok. That seems like a design mistake with borrow_with_sup. If we changed borrow_with_sup to take an Option instead, then you could do:
    let sections = gimli::DwarfSections::load(/* .. */).unwrap();
    let sup_secs = if let Some(...) = /* maybe-some-sup-object */ {
        Some(gimli::DwarfSections::load(/* load sections from sup object */).unwrap())
    } else {
        None
    };

    let dwarf = sections.borrow_with_sup(sup_secs.as_ref(), |b| *b)
    let ctx = addr2line::Context::from_dwarf(dwarf).unwrap();

    // Down here we may need a `Dwarf` again.
    let dwarf = sections.borrow_with_sup(sup_secs.as_ref(), |b| *b);

A better solution is https://github.com/gimli-rs/addr2line/pull/327, which allows you to use the same Dwarf in both places.

Yep, either should work for the use case at hand. Thanks!


add coverage support

Fix #35
Report at: https://codecov.io/gh/gengjiawen/uvwasi/tree/f38d38beebfc4b4ce824305b307d6c26bbeecfbe.

This looks very cool, thank you! I probably won't be able to test this out until tonight due to travel, but I'm excited to have this functionality.

If we want to show code coverage in PR, we need to setup a codecov account, and set an secret env.

Wait, I just realized that codecov is a paid service. Isn't there a free way to do this? The coverage report doesn't have to be linkable. I'd be fine with it just getting dumped to the CI logs.

The website says "Free for open source". It limits the number of users to 5, but maybe we will only need one for automatic coverage reports?

Free for open source. Plans starting at $10/user per month.

Gosh. It was right next to the part that I did read. 🤦♂

So I went to register for codecov, and it made me a bit uneasy. It seems to want access to a lot of things in my GitHub account just to sign up.

I have never used it, maybe we should consider alternatives? Can we use the same tools that we use for node core?

Can we use the same tools that we use for node core?

We could probably collect the coverage data, but we don't have anywhere like coverage.nodejs.org to put the reports for review (from what I can tell, that's the biggest thing that codecov.io offers). Ideally, I'd prefer something like the coverage information at the bottom of https://travis-ci.org/cjihrig/grpc-server-js/jobs/596687685... but that's a JavaScript project, and I don't know what C has that's similar.

We can report upload the result page to GH pages.

the biggest thing that codecov.io offers

It also can show coverage change in PR.
Maybe I remove coverage.io support in this PR, and just leave the html report as a n artifact.

Maybe I remove coverage.io support in this PR, and just leave the html report as a n artifact.

That sounds like a good idea until we can figure out a way to better integrate coverage into the CI.

Can you squash these commits please.

I always use the Github Squash and merge ? Is it okay for you too ?

 No worries, I can squash it for you.

 No worries, I can squash it for you.

Github can edit message when you squash. I heavily use this feature because I worry lose my code when I do rebase :) (being to cautious maybe)

Github can edit message when you squash.

OK, I'm fine with using it then.

Thanks for seeing this through!


Add support for window titles

#7 adds support for window titles.
There might be problems with Windows, since the title is currently marshalled as a utf-8 string (required by SDL), but who knows what a windows user will come up with.
(If I recall correctly, UTF16-LE is the internal encoding used in dotnet on all platforms).

Fixed by #7


fix(windows): prevent ghost window from showing up on taskbar

Co-authored-by: Osspial<EMAIL_ADDRESS>ref: https://github.com/rust-windowing/winit/commit/f92803d80ecd9a0d798ee0e3e3e32a87eca36693

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information

I've tested it and it works as expected. Thank you.


Mapping in anchor

⚠ Don't use issues for support questions ("How does x feature work in Anchor? How to do x in Anchor?"). Use the discord instead https://discord.gg/GVRw4kHB (but don't ping maintainers). ⚠
can i using hashmap or btreemap in anchor?

Please read the issue template.


⚠ Bofrost.it has degraded performance

In 56e765d, Bofrost.it (https://www.bofrost.it) experienced degraded performance:

HTTP code: 200
Response time: 1558 ms

Resolved: Bofrost.it performance has improved in 6ac6f3e after 6 minutes.


Clang tidy

Adds support for enabling clang-tidy and adds a separate CI run for it.

@KingKili can I get a review from you on this one?


proposal: time: add format constants for date and date+time

I propose the declaration of the following format constants to the time package:
const (
    DateTime = "2006-01-02 15:04:05"
    DateOnly = "2006-01-02"
)

Analyzing all Go code on the module proxy (as of 2022-04-01), here's a table of all common time formats sorted in popularity order:

Rank
Frequency
Format

1
75616
time.RFC3339

2
23954
time.RFC3339Nano

3
13312
"2006-01-02 15:04:05"

4
12332
"2006-01-02"

5
11940
time.RFC1123

6
2998
time.RFC1123Z

7
2871
time.UnixDate

8
2843
time.ANSIC

9
2820
"20060102"

10
2600
time.RFC822

11
2522
time.Kitchen

12
2499
"20060102150405"

13
2225
"15:04:05"

14
1503
time.RFC850

15
1410
time.Stamp

16
1374
"2006-01-02T15:04:05Z"

17
1302
"2006-01-02T15:04:05"

18
1152
"2006"

19
1149
"2006/01/02 15:04:05"

20
1133
time.StampMilli

21
1122
time.RFC822Z

22
1059
time.RubyDate

23
962
"15:04"

24
934
"2006-01-02 15:04:05.000"

25
879
"2006010215"

26
685
"15:04:05.000"

27
668
"2006-01-02 15:04"

28
646
"2006/01/02 - 15:04:05"

29
637
"2006/01/02"

30
552
time.StampNano

31
533
time.StampMicro

As can be seen, "2006-01-02 15:04:05" and "2006-01-02" are the 3rd and 4th most popular format constants. They are more popular than most of the other time format constants that did get a named declaration.
Given their popularity, I propose declaring:
const (
    DateTime = "2006-01-02 15:04:05"
    DateOnly = "2006-01-02"
)

Other constant name suggestions welcome.

Perhaps TimeOnly for "15:04:05" at 13th place would also make sense.

ISODate: "2006-01-02" Is the ISO 8601 extended date format (basic is 20060102).
ISOTime: "15:04:05" is the ISO 8601 extended time format (basic is 150405).
RFC3339 provides the extended ISO date and time format - using T as the separator (mandatory) and including the timezone.
Perhaps something like ISODateAndTime could be used? Technically, ISODateTime probably should be RFC3339 without the timezone since ISO requires using the T.

It sounds like maybe
const (
    ISO8601 = "2006-01-02 15:04:05"
    ISO8601Nano = "2006-01-02 15:04:05.999999999"
    ISO8601Date = "2006-01-02"
)

This would align with RFC3339 and RFC3339Nano and add the date-only form.
Thoughts?
/cc @robpike

Also, thanks for the great data @dsnet

The ISO names are fine, but I see advantages to also having the nicer names like TimeOnly and DateOnly as aliases. I remain a fan of time.Kitchen, which is underused.

I'm used to reading specs, but I'm overwhelmed by ISO 8601 🤯. On Wikipedia, it says:

Separating date and time parts with other characters such as space is not allowed in ISO 8601, but allowed in its profile RFC 3339.

If true, it seems that ISO8601 = "2006-01-02 15:04:05" is not be valid ISO 8601. What's really odd is that the source of that statement comes from RFC 3339 with no reference to where in ISO 8601 it says that.
(Side comment: Thank you Go language for having a simple and concise language specification).

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

The proposed formats are defined in the Internet Date/Time Format https://datatracker.ietf.org/doc/html/rfc3339#section-5.6 so maybe names could be either RFC3339DateTime, RFC3339Date and RFC3339Time  or InternetDateTime InternetDate and InternetTime.

names could be either RFC3339DateTime, RFC3339Date and RFC3339Time

My preference is for a human-friendly name without yet another number sequence.

My preference is for a human-friendly name without yet another number sequence.

What about InternetDateTime, InternetDate and InternetTime then to get away without lame Only suffix?

My preference is for a human-friendly name without yet another number sequence.

What about InternetDateTime, InternetDate and InternetTime then to get away without lame Only suffix?

Only has an obvious meaning. The Internet prefix has nothing to do with these time formats.

Analyzing all Go code on the module proxy (as of 2022-04-01), here's a table of all common time formats sorted in popularity order:

Would it be possible to tell how users name these constants?

names could be either RFC3339DateTime, RFC3339Date and RFC3339Time

My preference is for a human-friendly name without yet another number sequence.

Looking at the top two popular formats I think users mean to use "reduced" RFC3339 format. It is not something else because only RFC3339 format puts fields in descending order making string representations sortable. The field separator is also the same -.

The Internet prefix has nothing to do with these time formats.

As per https://github.com/golang/go/issues/52746#issuecomment-1140041186 these formats are somewhat defined in the RFC section named "Internet Date/Time Format". It also has a note about space separator used in these formats:

NOTE: ISO 8601 defines date and time separated by "T".
Applications using this syntax may choose, for the sake of
readability, to specify a full-date and full-time separated by
(say) a space character.

Usage in situ:
time.Parse(time.DateTime, s)
time.Parse(time.DateOnly, s)
time.Parse(time.TimeOnly, s)

vs
time.Parse(time.RFC3339DateTime, s)
time.Parse(time.RFC3339Date, s)
time.Parse(time.RFC3339Time, s)

vs
time.Parse(time.InternetDateTime, s)
time.Parse(time.InternetDate, s)
time.Parse(time.InternetTime, s)

Would it be possible to tell how users name these constants?

I think these were taken directly from the call sites. They name them like "2006-01-02".

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

@gouguoyin this proposal is accepted and implemented, and you don't seem to be proposing anything for the time package, so that seems off-topic to me.


Search+Tones block doesn't hide

Hi!
I wanted to hide block that is goes between filters and recent but after setting options, this block is still visible and have no content. Here is my code.
$("#add-post-area").emojioneArea({
        pickerPosition: "right",
        search: false,
        tones: false
});

Result:

I checked that even deleting SEARCH and TONES blocks from DOM tree does not move content, because it's stick to bottom and have it's height. Seems like there is no ready solution in current version...

@gulakov-designer you can use css for avoid this behavior

That's exactly what I did 😆
It's just perfectionist inside me, that is whispering: "It must work automatically..."
But aside jokes, maybe it's a good idea to not displaying this blocks as they are not in need and change the height accordingly? Seems to be not a big deal.

I agree with you, this should be fixed


fix issue 753

SUMMARY

fix issue #753

Automated Checks

[X] I have run ./gradlew test and made sure that my PR does not break any unit test.

Running tests.

Re-running tests.


UI glitches make Ferdi unusable for me

Describe the bug
Service button not working. Probably the integrated server issues.

Severe update check trolling while I tried to update in order to fix above issue.

To Reproduce
see the gifs
Expected behavior
Both buttons doing things
Screenshots
See bug description
Desktop (please complete the following information):

OS: Windows Server 2016 14393

The first issue looks to be related to the internal server bug I mentioned and should be fixed in beta 5.
The second issue is probably an issue with how Ferdi communicates with the Electron updater. Can you try activating the option, restarting Ferdi (using File > Quit Ferdi instead of just closing the window) and trying to update?

I tried that before reporting. The setting is off after the restart. :\
On Tue, 14 Jan 2020 17:34:31 +0000 (UTC)
Bennett<EMAIL_ADDRESS>wrote:

The first issue looks to be related to the internal server bug I mentioned and should be fixed in beta 5.
The second issue is probably an issue with how Ferdi communicates with the Electron updater. Can you try activating the option, restarting Ferdi (using File > Quit Ferdi instead of just closing the window) and trying to update?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/getferdi/ferdi/issues/299#issuecomment-574287999

Is there setup for the beta ?

This command fixed everything for me: attrib -r "%APPDATA%\Ferdi\server.sqlite".
Please consider adding equivalent functionality to the setup and an explicit check during app startup.

Thank you for the report. We are already discussing this issue and are trying all kinds of fixes over at https://github.com/getferdi/internal-server/issues/2 so this isn't anything new. The problem is that Ferdi doesn't seem to be able to remove the write protection - at least with the techniques we tried.

I did some experimenting with the nodejs repl and fs.chmodSync(dbPath, fs.statSync(dbPath).mode | 146); worked for me after re-adding the read-only attribute with attrib +r "%APPDATA%\Ferdi\server.sqlite".

Thank you for the suggestion, we'll try if that works.
What is the | 146 for?

It is supposed to just remove the 'read-only' attribute without affecting anything else.
C:\Users\Administrator>node --version
v10.13.0

C:\Users\Administrator>attrib +r %dbPath% & attrib %dbPath%
A    R  I    C:\Users\Administrator\AppData\Roaming\Ferdi\server.sqlite

C:\Users\Administrator>node
> let dbPath = 'C:\\Users\\Administrator\\AppData\\Roaming\\Ferdi\\server.sqlite';
undefined
> fs.chmodSync(dbPath, fs.statSync(dbPath).mode | 146);
undefined
> .exit

C:\Users\Administrator>attrib %dbPath%
A       I    C:\Users\Administrator\AppData\Roaming\Ferdi\server.sqlite

See also : https://github.com/gulpjs/gulp/issues/1057#issuecomment-99211021


项目运行 react-native run-android 运行报错

报错内容如下：
* What went wrong:
Execution failed for task ':app:processDebugResources'.
> Error: more than one library with package name 'com.projectseptember.RNGL'
  You can temporarily disable this error with android.enforceUniquePackageName=false
  However, this is temporary and will be enforced in 1.0

相当的沮丧，看 android/settings.gradle 和 android/app/build.gradle 之后，发现 @fangwei716 重复引入了gl-react-native
android/settings.gradle 多引入了 :gl-react-native， 删掉如下行
include ':gl-react-native'
project(':gl-react-native').projectDir = new File(rootProject.projectDir, '../node_modules/gl-react-native/android')

android/app/build.gradle  删掉如下行：
    compile project(':gl-react-native')

然后，就可以 happy 的运行起来了。

@xiajian
Thank you very much, it worked.

@flyher +1

useful

a issue for:Execution failed for task ':gl-react-native:prepareComFacebookFrescoDrawee0110Library'.
what can i do?

I had done it, and it told me  'Execution failed for task :app:prepareThirtyDaysOfReactNativeRNGLUnspecifiedLibrary', so what can I do?

这样之后，若提示找不到':gl-react-native'，可删除android目录下的build文件夹，然后重新运行：react-native run-android


QuestionMatrix throwing exception when question options are integers presented as strings

Example: https://www.expectedparrot.com/content/7c8d610b-066a-4400-b9c5-5ab84c1a27b7
Code to reproduce:
q = QuestionMatrix(
  question_name = "matrix",
  question_text = "Rate the following items on a scale of 1 to 5.",
  question_items = ["balloons", "bubble gum", "shoe polish"],
  question_options = ["1","2","3","4","5"], # this causes validation error; it is expecting [1,2,3,4,5]
  # option_labels = {1: "Terrible", 5: "Excellent"} # optional
)

I think it should modify the type to validate the responses.

I think this might be a "use_code" thing...


fitsSystemWindows does not work for ArFragment

Hi. I want to make ArFragment match parent for full screen support, but fitsSystemWindows does not work.
<?xml version="1.0" encoding="utf-8"?>
<android.support.design.widget.CoordinatorLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:fitsSystemWindows="true">

    <fragment
        android:id="@+id/sceneform_fragment"
        android:name="com.owlylabs.arshowme.view.fragment.WritingArFragment"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:fitsSystemWindows="true"/>

    <RelativeLayout android:id="@+id/fragment_container"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:layout_gravity="bottom"
        android:background="@drawable/rounded_corners_bottom_sheet"
        android:foreground="@drawable/foreground_line"
        android:orientation="vertical"
        android:paddingTop="80dp"
        app:behavior_hideable="true"
        app:layout_behavior="@string/view_pager_bottom_sheet_behavior"/>
</android.support.design.widget.CoordinatorLayout >

For this xml i have fragmentContainer that fitsSystemWindows, but ArFragment - not

@claywilkinson

The ArFragment installs a listener to change  the app to full screen see installing the listener and handling the listener
Does the hellosceneform sample work correctly?
You also may want to look at this Stackoverflow question and look more closely at how fitsSystemWindows works with fragments.  I don't think it is specific to Sceneform.

@claywilkinson Thank you for reply. You can see on the screenshot that another element fits system windows. I have mentioned a layout hierarchy earlier. Pls check this issue in any sceneform sampler.


Find bug during RecordController integration test

When testing with action save, a null pointer exception was thrown.

The reason is that there is no field named 'userId' in the request. So I changed the way controller getting the userId.


Make users able to limit the lifespan of their DAB text

A feature that allows users to set a text that's just displayed a certain amount of time (e.g. 10 minute) before the text is set back to default (i.e. the name of the show).

Working on this on the branch feature-#25


Parse go package names

Follow up of #662
These are problematic because there is no clear documented rule for where a core repo stops and when a path inside a repo starts. The only way will be to use the Go code.

The way to get this is to use the same code used by Go:

use this code and port it to Python https://go.googlesource.com/pkgsite/+/master/internal/source (for instance with a first pass using https://github.com/go-python/gotopy )

the source is source.go https://go.googlesource.com/pkgsite/+/master/internal/source/source.go
the tests in https://go.googlesource.com/pkgsite/+/master/internal/source/source_test.go show the rules to get the root of a Go package to find its corresponding repo

for a minimal vendored version see also https://github.com/google/go-licenses/pull/95 and https://github.com/google/go-licenses/tree/master/internal/third_party/pkgsite


Add Creative Cloud

Here's Adobe CC, verified working in jamf on Catalina. Let me know if there's a better way to get the download link!
Here's the jamf log:
[STEP 1 of 4]

Executing Policy Creative Cloud
[STEP 2 of 4]
Running script Installomator...
Script exit code: 0
Script result: 2021-04-28 14:48:51 shifting arguments for Jamf2021-04-28 14:48:51 ################## Start Installomator 2021-04-28 14:48:51 ################## adobecc 2021-04-28 14:48:51 no blocking processes defined, using Creative Cloud as default 2021-04-28 14:48:51 Changing directory to /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/tmp.zLq7goVz 2021-04-28 14:48:51 Spotlight not returning any app, trying manually in /Applications. 2021-04-28 14:48:51 could not find Creative Cloud.app 2021-04-28 14:48:51 no more blocking processes, continue with update 2021-04-28 14:48:51 Downloading https://prod-rel-ffc-ccm.oobesaas.adobe.com/adobe-ffc-external/core/v1/wam/download?sapCode=KCCC&productName=Creative%20Cloud&os=mac&guid=3cda2676-f4fc-4260-8b61-f9d820105e29&contextParams=%7B%22component%22%3A%22cc-home%22%2C%22visitor_guid%22%3A%2219436244748138622232565995318656911370%22%2C%22browser%22%3A%22Safari%22%2C%22context_guid%22%3A%226a5696ea-4858-48cd-bc83-694698f4d2a0%22%2C%22planCodeList%22%3A%22%22%2C%22installerWaitTime%22%3A30%2C%22Product_ID_Promoted%22%3A%22KCCC%22%2C%22contextComName%22%3A%22ACom%3ACCH%22%2C%22contextSvcName%22%3A%22NO-CCD%22%2C%22contextOrigin%22%3A%22ACom%3ACCH%22%2C%22aam_uuid%22%3A%2223074960036023656461797120837856430647%22%2C%22gpv%22%3A%22helpx.adobe.com%3Adownload-install%3Akb%3Acreative-cloud-desktop-app-download%22%2C%22creative-cloud-referrer%22%3A%22https%3A%2F%2Fhelpx.adobe.com%2Fdownload-install%2Fkb%2Fcreative-cloud-desktop-app-download.html%22%2C%22AMCV_D6FAAFAD54CA9F560A4C98A5%2540AdobeOrg%22%3A%22102365995%257CMCMID%257C19436244748138622232565995318656911370%257CMCAAMLH-1620249276%257C9%257CMCAAMB-1620249276%257C6G1ynYcLPuiQxYZrsz_pkqfLG9yMXBpb2zX5dvJdYQJzPXImdj0y%257CMCOPTOUT-1619651676s%257CNONE%257CMCSYNCSOP%257C411-18753%257CvVersion%257C2.2.0%22%2C%22mid%22%3A%2223275764998481040001781260498026036921%22%2C%22aid%22%3A%22%22%2C%22AppMeasurementVersion%22%3A%222.20.0%22%2C%22kaizenTrialDuration%22%3A7%7D&wamFeature=nuj-live&environment=prod&api_key=CCHomeWeb1 to Creative Cloud.dmg 2021-04-28 14:48:52 Mounting /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/tmp.zLq7goVz/Creative Cloud.dmg 2021-04-28 14:48:53 Mounted: /Volumes/Creative Cloud Installer 2021-04-28 14:48:53 looking for app: /Volumes/Creative Cloud Installer/Creative Cloud.app 2021-04-28 14:48:53 found app: /Volumes/Creative Cloud Installer/Creative Cloud Installer.app 2021-04-28 14:48:53 Verifying: /Volumes/Creative Cloud Installer/Creative Cloud Installer.app 2021-04-28 14:48:53 Team ID: JQ525L2MZD (expected: JQ525L2MZD ) 2021-04-28 14:48:53 Copy /Volumes/Creative Cloud Installer/Creative Cloud Installer.app to /Applications 2021-04-28 14:48:53 Changing owner to rezhko 2021-04-28 14:49:04 found app at /Applications/Creative Cloud.app, version <IP_ADDRESS> 2021-04-28 14:49:04 Installed Creative Cloud, version <IP_ADDRESS> 2021-04-28 14:49:04 notifying 2021-04-28 14:49:04 Deleting /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/tmp.zLq7goVz 2021-04-28 14:49:04 Unmounting /Volumes/Creative Cloud Installer "disk2" ejected. 2021-04-28 14:49:04 ################## End Installomator
[STEP 3 of 4]
[STEP 4 of 4]

Isn't the Adobe CC download URL different for each version?

I think this PR can be closed if the Adobe installer being downloaded can't be silently installed. Any comments?


Release 0.62.0

@cavalle this release will now include your MX schemas 👍

@samlown thanks a lot! It seems I forgot to git-add the schemas in e194d1904b64b6dc894777057fc58f41358d69d2 🤦


Real-time suitable?

Thanks for sharing your work and code!
Can you tell us a bit about the runtime for inference?
Is it generally suited for real-time applications and did you have some experience running it on the iPhone?

Hi, as mentioned in our paper, our 2D-3D matching module takes around 88ms which is tested on a single V100 GPU. The RansacPnP will also take a little time which is dependent on your CPU condition. Based on this, our method currently cannot perform real-time inference. We observe that reducing the number of 3D points for matching can help improve inference speed with little compensation for accuracy, as mentioned in the supplementary. Therefore, I think our method can be further elaborated for faster inference :).

Thanks for the info! 👍


When I remove a node, the memory is not released

I have some quite heavy models, each model adds around 300MB of memory when added to the scene view. When more than 10 models are added, the application starts reaching the onTrimMemory state. My task is to remove some of the initial models to free up memory. However, after calling arSceneView.removeChildNode(node), the memory usage remains unchanged. Memory is only fully released when I call recreate() to reload the activity, but this isn’t a good solution as all previous models will be lost.
How can I free up the entire 300MB of memory for each ModelNode after I remove it?

Here is a sample file: https://thinhnd-nal.github.io/sample_files/17_RKQ-A5325%2CRKS-098A.glb


Send key_up/key_down when lone modifiers change (macOS)

This introduces the convention that we send key_up and key_down when
modifier keys are pressed on their own. This implements this behaviour on
macOS, but not on windows, because I'm not sure what's going on there.
if we don't end up keeping muggle (which seems very unlikely) we'll have to backport this to master.

I believe Windows sends these as-is (Windows tends to be lower level) but at some point we'll want to go through these carefully and verify that we're getting pretty much the same events on both platforms.


style is not applied to all the components

Bug, feature request, or proposal:
Hi I'm using angular cli (scss) I added angular material but for some reason the css is not applied to all the component. For example the button don't take the primary color

As you can see the select and the input take the style but not the button
In my style.scss I did:
@import<EMAIL_ADDRESS>And in my mainmodule:
MaterialModule.forRoot(),

What is the expected behavior?
The css is applied to all the material component
Providing a Plunker (or similar) is the best way to get the team to see your issue.
https://github.com/daniele-zurico/table-order
Which versions of Angular, Material, OS, browsers are affected?
^2.0.0-alpha.11-3

Your main module uses forRoot which will give your app all the service provider dependencies for material, but that will not provide the components to every module in your app.
Your PersonInputModule includes HTML that uses two components: md-input and md-button. To use these in this module, you must import them. You correctly imported MdInputModule but did not import MdButtonModule. That is why the button seems to lack the styling.
To fix your issue, simply import  MdButtonModule in  PersonInputModule.


GRID-235: Externalize hyper-analytics (first step)

dataModel/JSON.js:

Now functions without aggregator or filter data sources.
setData now re-uses previous data source if dataSource (1st param) is omitted.
setData now passes 2nd param to constructor of first data source in the pipeline.

behaviors/Behavior.js, behaviors/JSON.js:
Changed calling signature for constructor (initialize method) from (grid, schema, dataRows) to (grid, dataRows, options).
Added options.fields, passed to dataMode/JSON.setData as 2nd param.

@joneit Will consume after I get aggregations to work again


Educating documenters, Q3

In our Q3 budget we identified the need for some training on using the Meeting Summary tool, aimed at newer people in the ecosystem.
We originally allocated $400 to this, and planned to do several sessions.
But, due to budget-fitting, this needs to be reduced - also, given that the tool will be redesigned this quarter, it would be better to look at further training once the redesign is done.
So the budget for this quarter has been reduced to $200.

Vanessa and Onyeka have done several 1-2-1 sessions with individuals; and also planned and delivered 2 group training sessions, on 19th and 21st June.
These are costed inder this issue, in Q3, but were delivered early so that they could also help those working on the "Backlog" task https://github.com/SingularityNET-Archive/SingularityNET-Archive/issues/82
See https://drive.google.com/drive/folders/10JDEHfcE-1H6WSfAIWKkbwI63iKoNGQt?usp=drive_link for videos of the sessions
This task can now be moved to "done" and sent for payment
To be paid:
$100 each to Lord_Kizzy and CallyFromAuron


Mastermind plugin not working

Hello,
I have been trying to get Mastermind annotations through the REST API, but in my tests the parameter seems to be ineffectual and no Mastermind annotation is returned. The Mastermind plugin works in the web interface as expected, so I suspect an error on my part, or an issue with the REST API. Python request to replicate:
from requests import get

server = "https://grch37.rest.ensembl.org/vep/human/hgvs/"
hgvs_var = "6:g.51947999G>A"  # PKHD1 p.T36M, well-known pathogenic variant, annotated with the correct Mastermind link on the VEP web interface as expected - Mastermind links 94 publications for this variant
headers = {"Content-Type": "application/json", "Accept": "application/json"}
params = {"Mastermind": 1}

res_json = get(server + hgvs_var, headers=headers, params=params).json()

The returned json contains no Mastermind data. From the documentation it is not clear to me what the problem could be.
Appreciate any help!
Best,
Florian

Hello Florian,
Thanks for your feedback,
I will check this endpoint and come back to you soon
Kind regards,
Mira

On 28 Apr 2021, at 09:44, FlorianErger @.***> wrote:
Hello,
I have been trying to get Mastermind annotations through the REST API, but in my tests the parameter seems to be ineffectual and no Mastermind annotation is returned. The Mastermind plugin works in the web interface as expected, so I suspect an error on my part, or an issue with the REST API. Python request to replicate:
from requests import get
server = "https://grch37.rest.ensembl.org/vep/human/hgvs/"
hgvs_var = "6:g.51947999G>A"  # PKHD1 p.T36M, well-known pathogenic variant, annotated with the correct Mastermind link on the VEP web interface as expected - Mastermind links 94 publications for this variant
headers = {"Content-Type": "application/json", "Accept": "application/json"}
params = {"Mastermind": 1}
res_json = get(server + hgvs_var, headers=headers, params=params).json()
The returned json contains no Mastermind data. From the documentation it is not clear to me what the problem could be.
Appreciate any help!
Best,
Florian
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/Ensembl/ensembl-rest/issues/483, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACSSFVZBNAO6OFP5W4NP65LTK7DH7ANCNFSM43WTRDDQ.

Sorry to reopen this, but do you happen to know how the web team fixed the issue? I have set up a local ensembl-rest server and while other VEP-plugins work well (CADD, MaxEntScan, SpliceAI), I am not getting Mastermind annotation.
My ensembl_rest.conf is pointed to my Plugins folder and the plugin_config.txt
The Mastermind section in my plugin_config.txt I have edited as follows:
{
  "key" => "Mastermind",
  "label" => "Mastermind",
  "helptip" => "Variants with clinical evidence cited in the medical literature reported by Mastermind Genomic Search Engine (https://www.genomenon.com/mastermind)",
  "available" => 1,
  "enabled" => 0,
  "requires_data" => 1,
  "species" => [
    "homo_sapiens"
  ],
  "section" => "Phenotype data and citations",
  "plugin_url" => "https://raw.githubusercontent.com/Ensembl/VEP_plugins/release/104/Mastermind.pm",
  "params" => [
    "/home/iroot/Programs/ensembl-vep/databases/mastermind_cited_variants_reference-2021.04.02-grch37.vcf.gz",0,1
  ]
}

I know that the plugin is called, because when I set an invalid path to my mastermind file, I get an error ERROR: Mastermind input file not specified or found!. With the correct path, however, I get no error but also no output on a variant for which Mastermind output should definitely exist. I have tried the params without the ,0,1 at the end, I have tried it with ,0,1,0, with ,"0","1" and all permutations I could think of.
I am accessing http://<IP_ADDRESS>:3000/vep/human/hgvs/6:g.51947999G>A?content-type=application/json&refseq=1&CADD=1&vcf_string=1&hgvs=1&Mastermind=1&SpliceAI=1&MaxEntScan=1
Thanks for any suggestions you might have!
Best,
Florian

Closing for now. @FlorianErger pls feel free to reopen, should you have further concerns or issues.
Thanks

Hi @FlorianErger, I'm sorry for the late reply. We noticed there was a missing file in ensembl_rest.conf You should make sure your local server includes this update: https://github.com/Ensembl/ensembl-rest_private/pull/99/files Let me know if you have more questions.
Best wishes, Diana

Dear @dglemos ,
thanks for the description of the problem. I unfortunately cannot access your links (error 404, private repo?) and am still facing the same issue (also on a fresh v109 installation).
Could you update the links or describe the fix?
Many thanks and best wishes,
Florian

Hi @FlorianErger,
I'm sorry, the link I sent is from a private repo.
The missing config has been fixed which means you should not have the same issue.
I will have a look at it and will let you know.
Best wishes,
Diana

Mastermind is working on grch37: https://grch37.rest.ensembl.org/vep/human/id/rs699?content-type=application/json&Mastermind=1
Are you pointing to a local installation of REST?

Yes, I was referring to the solution you kindly described in your last post. I am unable to implement it as I cannot access the links you provided.

Just for completeness, my plugins_config.txt section on Mastermind is:

Mastermind

{
  "key" => "Mastermind",
  "label" => "Mastermind",
  "helptip" => "Variants with clinical evidence cited in the medical literature reported by Mastermind Genomic Search Engine (https://www.genomenon.com/mastermind)",
  "available" => 1,
  "enabled" => 0,
  "species" => [
    "homo_sapiens"
  ],
  "params" => [
    "/opt/Ensembl/ensembl-vep/cache/Plugins/Mastermind/mastermind_cited_variants_reference-2023.01.02-grch37.vcf.gz"
  ],
  "section" => "Phenotype data and citations",
  "plugin_url" => "https://raw.githubusercontent.com/Ensembl/VEP_plugins/release/109/Mastermind.pm"
},

My test request to my local server is:

curl 'http://localhost:3000/vep/human/hgvs?Mastermind=1' -H 'Content-type:application/json' -H 'Accept:application/json' -X POST -d '{ "hgvs_notations" : ["6:g51947999G>A" ] }'

If I annotate directly with VEP from the command line, I get the Mastermind annotation, so the plugin itself works.

Mastermind requires a file of chromosome synonyms (vep option). The file should be tab-delimited with the primary identifier in column 1 and the synonym in column 2. You can find this file in the VEP cache, the file name is chr_synonyms.txt.
For Mastermind to be able to run you have to add the option synonyms to your VEP config. Something like this:
<Controller::VEP>
  fasta = {{ homo_sapiens_fasta }}
  dir    = {{ vep_cache_dir }}/
  cache_version = {{ ensembl_release }}
  cache               = 1
  failed               = 0
  sift                   = b
  polyphen         = b
  symbol            = 1
  regulatory       = 1
  check_existing = 1
  rest                   = 1
  fork                   = 0
  assembly	   = GRCh37
  biotype             = 1
  max_post_size = 300
  warning_file      = STDERR # controls VEP logging, not Catalyst
  plugin_config   = {{ vep_plugin_config }}      # as found in this dir
  dir_plugins       = {{ vep_plugin_dir }}       # checkout of https://github.com/ensembl-variation/VEP_plugins
  human_assembly    = {{ human_assembly }}
  synonyms         = path/chr_synonyms.txt

Dear Diana,
perfect! That was the missing piece, the plugin now works on my local REST setup.
Thank you very much,
Best,
Florian

I'm glad it's working now!
Best wishes,
Diana


SALs: process (local and remote executors based on fabric)

TBD:
http://www.fabfile.org/

I can not find it, should it be like j.sals.fabric or something else?

j.sals.process.execute docs not showing it is remote
JS-NG> j.sals.process.execute(

execute(cmd, showout=False, cwd=None, shell='/bin/bash', timeout=600, asynchronous=False, env=None, replace_env=False, die=False)

execute a command.

accepts command as a list too, with auto-escaping.

Arguments:
    cmd (str or list): command to be executed, e.g. `"ls -la"` or `["ls", "-la"]

Keyword Arguments:
    showout (bool): show stdout of the command (default: False)
    cwd (str): specify a working directory for the command (default: None)

https://threefoldtech.github.io/js-ng/wiki/#/./executors

Thanks , Please add a more details docs for remote executor

Thanks , Please add a more details docs for remote executor

can you please open a feature request for that?


Put angular-gettext on Nuget

Hi
I want to ask you, what do you think about putting this plugin on Nuget.
P.S. Great job with this plugin, thanks a lot.

+1. for those of us who don't want to convert all our projects to use bower.

Sure, but I have no clue how that works. Anyone want to volunteer for this?

@rubenv I can handle that.

@banarualexandru @thepavel @rubenv Published nuget package.
https://www.nuget.org/packages/angular-gettext/
:tada:

@crissdev Thanks a lot! Gread job.
Regards

Updated Nuget package to 2.2.0 release.
@rubenv Would you consider adding installation notes as other projects do? Here's an example

They are here: https://angular-gettext.rocketeer.be/dev-guide/install/
Feel free to add a section about NuGet, source is here: https://github.com/rubenv/angular-gettext-website

thanks, @crissdev!


Secondary Email Verification

In gitlab by @CalumTowers on Aug 3, 2016, 18:15
Secondary email verification emails are sent to the account's main email and they should be sent to the newly added secondary email.

In gitlab by @A-Lawrence on Aug 24, 2016, 00:10
Milestone removed

In gitlab by @atoff on Sep 17, 2016, 18:40
Mentioned in merge request !30

In gitlab by @atoff on Sep 25, 2016, 21:54
Status changed to closed by commit 65cfc88b31f51b733b9b538b42ba3216bb9160c7

In gitlab by @A-Lawrence on Oct 3, 2016, 16:57
Removed ~558948 label

In gitlab by @atoff on Oct 3, 2016, 19:12
Mentioned in commit 65cfc88b31f51b733b9b538b42ba3216bb9160c7

In gitlab by @atoff on Nov 12, 2016, 14:28
Mentioned in issue #571


Fix bug and enhance Genre section UI

Description

Changed Genres carousel to multi-image form, for better UX
Enhanced UI
https://github.com/Praful932/Kitabe/issues/62#issue-828858522

Affected Dependencies
How has this been tested?
Using flake8, python manage.py test.
No test fails.
GIF Before

For large screens:

For small screens:

GIF After
-For large screens:

For small screens:

Checklist

[X] I have followed the Contribution Guidelines.
[X] My changes do not edit/add any unrequired files.
[X] My changes are covered by tests.

Also for small screen the controllers at center would be much better like we have for popular among users section

Okay, will work on it.

@Greeshma2903 Nice Work 🚀

Thank you so much😊


Shelley integration

One thought: perhaps we can implement something like snapshot: EpochInfo IO -> EpochInfo Identity which would produce an EpochInfo "for the epochs we currently know of" and throw an error for any epochs later than that. T

This will need rebasing after #1657 is merged. Everything from this PR should move to ouroboros-consensus-shelley, see ouroboros-consensus-byron for a template.

Superseded by #1695. I will bring over the tests to that PR too.


Update test_mms.cpp to work on macOS

Description
Changes the number of electrons in the TwoStreamGrowthRate test, so that the test can pass in a direct build on MacOS.
Fixes # (issue)
Fixed issue with TwoStreamGrowthRate test not passing in MacOS.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change)

Testing
Please describe the tests that you ran to verify your changes and provide instructions for reproducibility. Please also list any relevant details for your test configuration.

Ran TwoStreamGrowthRate test and passed

Test Configuration:

OS:MacOS
SYCL implementation:
MPI details:
Hardware:M1 Pro MacBook Pro

Checklist:

[] New and existing unit tests pass locally with my changes

######### DELETE THIS AND BELOW #########
How-to:

Format C++ source code with clang-format

find ./src ./include ./test -iname \*.hpp -o -iname \*.cpp | xargs clang-format -i

Format cmake files with cmake-format

cmake-format -i CMakeLists.txt src/nektar/CMakeLists.txt test/CMakeLists.txt

Format python with black

find ./python -iname \*.py | xargs black

Integration and unit tests all pass for me on Ubuntu 22.04 with oneAPI
ParticleInitialisationLine.Points fails when using gcc, but that's due to an unrelated issue which @will-saunders-ukaea has fixed on feature/3d-particle-base.


Broken link to ETS ontology

This link in the README does not load https://terminologies.gfbio.org/terminology/?ontology=ETS. Is there a new link or alternative way to reach this page?

Hey David,
since yesterday we have an issue on the backend of the GFBio Terminology Service. We are aware of this issue, but fixing it turns out to be a bit tricky, hopefully it should be back online soon. So this is not a permanent issue.
An alternative would be to go to our dev-server: https://dev-gfbio.bgbm.org/terminology/?ontology=ETS . The classical documentation pages however still work: https://terminologies.gfbio.org/terms/ets/pages/ .
I will keep you updated on our progress, so I am leaving this issue open until then.
Cheers, David

Hey David, the issue has been fixed now. Thanks for pointing this out. All the best, David

Thank you!


Set compiler flags (e.g. -std= and -Werror) as add_compile_options.

This is to avoid polluting the global variables

Coverage remained the same at 93.329% when pulling ec2e0afc7bd47394a06a3d45f3aba943ff93b5b2 on zanebeckwith:compile-flags into 700248a84a99b2a073459fee4a1025c4c8918e27 on xaptum:master.


Testnet hotfix: Warm active set cache for hare

Motivation
Testnet hotfix workaround for #2478
Unclear yet whether we want to apply this workaround to develop, or address the underlying issue in another way
Changes

When hare gets active set, it checks if it's near a safe epoch boundary, and if so, it warms the cache for the next layer/safe epoch ahead of time

Test Plan

Adds regression test

TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[ ] Update documentation as needed

DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

@dshulyak thanks for taking a look. I agree this solution is not bulletproof and is not completely production-grade. We'll want to do something more robust to solve this issue for develop branch. This is a rough-and-ready hotfix for the testnet, and it does appear to be working.
Another reason this isn't a bulletproof solution: it'll only work for nodes that are already synced and running hare continuously.


The example code: Aspect_Term_Extraction.ipynb in the subfolder of aspect_term_extraction in examples-v2 is not working

Version
See the console output for PyABSA, Torch, Transformers Version
PyABSA: 2.4.0
Torch: 2.0.1
transformers: 4.31.0
Describe the bug
A clear and concise description of what the bug is.
In the below sample code under the folder of examples-v2/aspect_term_extraction
trainer = ATEPC.ATEPCTrainer(
config=config,
dataset=dataset,
from_checkpoint="english",  # if you want to resume training from our pretrained checkpoints, you can pass the checkpoint name here
auto_device=DeviceTypeOption.AUTO,  # use cuda if available
checkpoint_save_mode=ModelSaveOption.SAVE_MODEL_STATE_DICT,  # save state dict only instead of the whole model
load_aug=False,  # there are some augmentation dataset for integrated datasets, you use them by setting load_aug=True to improve performance
)
When running the above code, it will have the below error:
File /databricks/conda/lib/python3.9/site-packages/torch/nn/modules/module.py:2041, in Module.load_state_dict(self, state_dict, strict)
2036         error_msgs.insert(
2037             0, 'Missing key(s) in state_dict: {}. '.format(
2038                 ', '.join('"{}"'.format(k) for k in missing_keys)))
2040 if len(error_msgs) > 0:
-> 2041     raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
2042                        self.class.name, "\n\t".join(error_msgs)))
2043 return _IncompatibleKeys(missing_keys, unexpected_keys)
RuntimeError: Error(s) in loading state_dict for FAST_LCF_ATEPC:
Unexpected key(s) in state_dict: "bert4global.embeddings.position_ids".
However, if I commented the line of  "from_checkpoint="english", ", then it works well.
Code To Reproduce
rainer = ATEPC.ATEPCTrainer(
config=config,
dataset=dataset,
from_checkpoint="english",  # if you want to resume training from our pretrained checkpoints, you can pass the checkpoint name here
auto_device=DeviceTypeOption.AUTO,  # use cuda if available
checkpoint_save_mode=ModelSaveOption.SAVE_MODEL_STATE_DICT,  # save state dict only instead of the whole model
load_aug=False,  # there are some augmentation dataset for integrated datasets, you use them by setting load_aug=True to improve performance
)
Expected behavior
Could you please troubleshoot this bug? I think resume training from your pretrained checkpoints such as "english" is very important. Otherwise, if training from scratch will generate an underperformed model.
Thank you very much in advance!
The above example code is available at:
https://github.com/yangheng95/PyABSA/blob/v2/examples-v2/aspect_term_extraction/Aspect_Term_Extraction.ipynb
Screenshots

Disable checkpoint resuming will work

Thanks Heng for your reply.
Yep, I know disable the checkpoint resuming can work, but wondering how can we resume the checkpoint (as it is a very important feature in some situtations) ? Thanks in advance if you could help with the bug?

Clone and revise line 451 in instructor_template.py, i.e., torch.load(..., strict=False)

Setting "strict=False",  it works well now. Thank you so much!


New version: sylikc.JPEGView version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @KaranKad,

This PR's version number <IP_ADDRESS> has 4 parameters (sets of numbers between dots - major, minor, etc), which is greater than the current manifest's version 1.2.45, which has 3 parameters.

Is this intentional?
(Automated response - build 684)

Hi @KaranKad,
This manifest removes Apps and Features entries that are present in previous PR versions. Should these entries also be added to this version?
(Automated response - build 684.)

@stephengillie
I used 4 parameters as that's what added in ARP. Hence, we do not need to add Apps and Features entries due to already accurate versioning.

I just checked that for older versions ARP in inaccurate I will fix it in upcoming PR.


Incomplete Gitea repository list fixed

Woodpecker may display incomplete list of Gitea repositores user has access to because Gitea forces paginated repo list reading.
How to reproduce: create 35 test repos in Gitea and sync it to Woodpecker - only about 30 will be displayed.
This mod fixes it.
Related: https://gitea.com/gitea/go-sdk/issues/507
Author-Change-Id: IB#1107754

This should be marked as workaround so

This mod was designed to workaround Giteas SDK problem but also to be compatible with paging requirement after Gitea is fixed so I don't see point in marking this as workaroud.
BTW: repo list paging in Woodpecker would be nice feature but its separate topic.

ok https://gitea.com/gitea/go-sdk/pulls/510 got merged ... se we can & should update the sdk ...

with #184 you dont need the PageSize: 50 :)

with #184 you dont need the PageSize: 50 :)

Seems that code
https://gitea.com/gitea/go-sdk/src/branch/master/gitea/list_options.go#L35
does not touch PageSize if it's not set explicitly (=0) and Gitea called with
https://gitea-url/api/v1/user/repos?page=1&limit=0
returns hardcoded page limit 30 so I think its better to leave explicit paging request on Woodpecker side to avoid doubts and allow Woodpecker to fetch repos with max page size allowed by Gitea SDK which above code limits to 50.
Not sure but maybe there should be

if o.PageSize < 0 || o.PageSize > maxPageSize {

if o.PageSize < 1 || o.PageSize > maxPageSize {

in
https://gitea.com/gitea/go-sdk/src/branch/master/gitea/list_options.go#L35
if 0 will not be defined in Gitea API for fetching all repos in one shot (now 0 does not return all repos). If Gitea will allow to fetch all repos in one shot using PageSize=0, o.PageSize < 0 should be lef untouched.


Update using-maven-with-mule.adoc

@robertpliu Thank you for your contribution. Please read the contributor’s agreement: https://github.com/mulesoft/docs-connectors/blob/latest/README.adoc, and follow the steps in this link: https://api-notebook.anypoint.mulesoft.com/notebooks/#d28e8f95e739ee1e1e30d5ec577f4a27” to register as a contributor so that we can accept your pull request.

Closing as fixed in https://github.com/mulesoft/docs-mule-runtime/pull/1298


metric_prefix does not work in mysql check

In the conf.yaml.example of mysql check, metric_prefix is specified, but no prefix is actually applied.
https://github.com/DataDog/integrations-core/blob/d1f76d4a13305315493a7ea14b7e89dbe69fecab/mysql/datadog_checks/mysql/data/conf.yaml.example#L11-L15
Steps to reproduce the issue:

create mysql.yaml, which include global_custom_queries and metric_prefix
run agent check mysql

Describe the results you received:
metric name does not has a prefix, just columns: name only.
Describe the results you expected:
The metric name is prefixed with the value specified by metric_prefix.

Hi @komeda-shinji , can you submit a support ticket to https://www.datadoghq.com/support/ so we can analyze why this is occurring?

"why this is occurring?" ??
Could you compare it with the implementation in postgresql integration?
The metric_prefix handling does not seem to be implemented in the mysql integration.

Hi 👋 , we looked into the mysql code, and you are right it's not being used in mysql. The metric_prefix option is deprecated and was removed in https://github.com/DataDog/integrations-core/pull/7583 as a standard option in database integrations. However, some integrations (like the postgres integration) still support metric_prefix due to its older custom query logic. I just submitted a PR to remove the option for integrations that don't support metric_prefix.


chore: Update branding options

Summary | Résumé

Adds FINTRAC & Canadian Nuclear Safety Commission

Pull Request Checklist
Please complete the following items in the checklist before you request a review:

[ ] Have you completely tested the functionality of change introduced in this PR? Is the PR solving the problem it's meant to solve within the scope of the related issue?
[ ] The PR does not introduce any new issues such as failed tests, console warnings or new bugs.
[ ] If this PR adds a package have you ensured its licensed correctly and does not add additional security issues?
[ ] Is the code clean, readable and maintainable? Is it easy to understand and comprehend.
[ ] Does your code have adequate comprehensible comments? Do new functions have docstrings?
[ ] Have you modified the change log and updated any relevant documentation?
[ ] Is there adequate test coverage? Both unit tests and end-to-end tests where applicable?
[ ] If your PR is touching any UI is it accessible? Have you tested it with a screen reader? Have you tested it with automated testing tools such as axe?

Waiting on CNSC svgs

We have been provided with the following SVG file for CNSC.

Updated -- back on hold for this one.

@timarney Here is the French version for CNSC!


Chrome warning

Hi, it is just a small report about the latest 6.0.9. I got 'soft error' in Chrome with it with the following message:

Manifest version 2 is deprecated, and support will be removed in 2023. See https://developer.chrome.com/blog/mv2-transition/ for more details.

Hope you can solve it easily...

Mozilla has not yet added support for Manifest version 3:
https://blog.mozilla.org/addons/2022/05/18/manifest-v3-in-firefox-recap-next-steps/
I am waiting for them to add support, which according to that blog post should happen before the end of 2022 (so before Chrome removes the support for Manifest v2) and then I will change it for both versions.


typeahead-editable="false"  should select first on enter and validate

My  configuration:
 <input name="type" type="text" class="form-control"
          ng-model="mytype"
          typeahead-template-url="typeAutocompleteTemplate.html"
          typeahead-editable="false"
          typeahead="b for b in autocompleteService.mytypeQuery($viewValue)"
          typeahead-input-formatter="autocompleteService.mytypeFormatLabel($model)"
       required>

When typing value, first value is selected. Hitting enter puts it in input (after formatting with  typeahead-input-formatter). But input does not validate.
When selecting with mouse or keydown, then everything works.
My values are:
Type 1
Type 2, .....
When typing "Type" and hitting enter, it works
When typing "Type 1" then it does not work
One way to fix it is to change parsers:
       if (isEditable) {
          return inputValue;
        } else {
          if (!inputValue) {
            // Reset in case user had typed something previously.
            modelCtrl.$setValidity('editable', true);
          } else {
            modelCtrl.$setValidity('editable', false);
          }
          return inputValue;
        }

Return value always, because value really is never invalid.

@MayGo Could you set up a simple plunkr?
It would be great to send a PR with tests associated to it. rgds

+++

guys please fix this bug!

@ivschukin Please send us a pull request!

What is the status of this? I'm having this issue with a typeahead-editable="false" typeahead. If you type in exactly (case-sensitive) the matching item it selects the value correctly, however it leaves a $error.parse = true property.

If I may add.  The documentation is wrong for this attribute.  The default should be false not true.

From reading the description, I think I have the same issue. When I type the full value of one of the options and then select the option, the field is invalid with ng-invalid-parse. When I type a partial value and select the option, the field is valid.
I don't have a test case to demo, but upgrading from 0.13.0 to 0.13.1 fixes the issue for me.

Thank you Stijn! I just upgraded and that resolved my issues, everything is working correctly now.

Not able to reproduce this issue here - this appears to be fixed.

@wesleycho You can reproduce it as:

Enter something in the control and check the class of the control as:
"form-control ng-dirty ng-touched ng-invalid ng-invalid-editable ng-invalid-parse"
Till you select some value from dropdown the control remains invalid. It should ideally not be invalid if one enters potential matching values.
Can we have some resolution for the same please.

Hey guys!
Why is this issue closed? I didn't find any solution to the case described by @MayGo . I am using version 1.3.3 of angular-bootstrap.
Thanks!


Graphite does not render graphs and throws Http error 500 on render call

I deployed graphite application in Centos 7 and I see the following error as  Trackback -
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/django/core/handlers/base.py", line 113, in get_response
    response = callback(request, *callback_args, **callback_kwargs)
  File "/opt/graphite/webapp/graphite/render/views.py", line 56, in renderView
    (graphOptions, requestOptions) = parseOptions(request)
  File "/opt/graphite/webapp/graphite/render/views.py", line 241, in parseOptions
    graphType = queryParams.get('graphType','line')
AttributeError: 'WSGIRequest' object has no attribute 'get'

I am using the following versions -
Django (1.5.12)
whisper (0.9.15)
django-tagging (0.3.6)
Pythong(2.7)
Graphite-Web(0.9.15)
Carbon(0.9.15)
Any ideas?

I also made sure to install all the required dependencies -
[root@server graphite]# ./check_dependency.py
[OPTIONAL] Unable to import the 'python-rrdtool' module, this is required for reading RRD.
[OPTIONAL] Unable to import the 'whitenoise' module. This is useful for serving static files.
2 optional dependencies not met. Please consider the optional items before proceeding.
All necessary dependencies are met.

Unable to import the 'python-rrdtool' module, this is required for reading RRD.
This is required for the rendering of rrd files. pip install python-rrd. You need a slew of deps related to cario. It's quite the trek to get them all.

I don't believe rrdtool has anything to do with the original exception. @skilledmonster are you still running into this issue? Apologies for the slow response.

I ran into a similar issue installing graphite-web into a virtualenv on centos6.  Downgrading to django==1.8.15 fixes it for me.
`[root@localhost vagrant]# /opt/graphite/bin/pip freeze
cairocffi==0.7.2
carbon==0.9.15
cffi==1.8.3
Django==1.9.10
django-tagging==0.4.3
graphite-web==0.9.15
pycparser==2.14
pytz==2016.7
Twisted==16.4.1
txAMQP==0.6.2
uWSGI==2.0.14
whisper==0.9.15
zope.interface==4.3.2
[root@localhost vagrant]# curl "http://<IP_ADDRESS>:80/render/"

Graphite encountered an unexpected error while handling your request.
Please contact your site administrator if the problem persists.

Traceback (most recent call last):
  File "/opt/graphite/lib/python2.7/site-packages/django/core/handlers/base.py", line 149, in get_response
    response = self.process_exception_by_middleware(e, request)
  File "/opt/graphite/lib/python2.7/site-packages/django/core/handlers/base.py", line 147, in get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/opt/graphite/webapp/graphite/render/views.py", line 56, in renderView
    (graphOptions, requestOptions) = parseOptions(request)
  File "/opt/graphite/webapp/graphite/render/views.py", line 236, in parseOptions
    queryParams = request.REQUEST
AttributeError: 'WSGIRequest' object has no attribute 'REQUEST'

`

I'm getting this problem as well.

Graphite-web 0.9.15 is intended to be run with Django 1.4. Please try that version and report back.

Works fine with 1.4. Chef cookbook I was using was setting the Django version at 1.9.

Cool. I'm going to close this issue since it appears to be a dependency versioning issue.


Use dmr client for validation. Additional validation workflows

Consolidates workflow validation to dmr client; removing various js files.
Adds workflow failures for DTDL edits, renames, deletes.
Adds workflow failures for non-DTDL items.

Conflict aren't cleanly resolving. Will issue new PR.


Foralls not always displaying

I ran into an issue where there was a forall that wasn't being displayed in a branch of a conjunction resulting from an induction. As a result, it looks like there is an equivalence, but eqRhs doesn't work (unless you use forall-body first).
Likely due to some last minute refactoring before my defense... we used to pretty print empty foralls, but that was annoying, so I stopped printing them and tried to make sure we normalize them away when we construct lemmas. This normalization must not be happening somewhere.
I would like to change the Clause type so Forall only binds a single quantifier, akin to Lam in Core. Multiple quantifiers would just be nested foralls. This should simplify the implementation of many of the primitive operations on Clause. Maybe that is worth doing instead of fixing this directly.
I should be back online by next week, in case no one does it before then.

I support that change.  It'll probably require us to update a bunch of scripts in the two case studies though.  I suggest also adding a navigation command "foralls-body" analagous to "lams-body".  (HERMIT/Dictionary/Navigation.hs)

Finally got this done (mostly) in e78bc8935b1811d60ea035ff4f06b4298d6ce4e5. It took longer than I thought because it involved some pretty printer changes. Right now the non-clean pretty printers won't work for lemmas. I'll fix that later.


Fix: ems.ems_kubernetes.miq_namespace setting was partially ignored.

Followup to #7168.
This only affects users that configure ems.ems_kubernetes.miq_namespace to something other than the default management-infra.
AFAICT, prior functionality wasn't impacted, only the authentication with the service account that #7168 added could fail for them.
@miq-bot add_label darga/yes

@simon3z please review

@moolitayer @enoodle can you please review this? It's not the right time to break this part.

@miq-bot add_label bug, providers/containers

LGTM
I feel obligated to say I've been awake for too long :bowtie:

LGTM


Read timed out

Hi,
I'm having issues setting up the plugin.
When I test the connection it is working but when I open views I'm getting an error message.
IO Error during method execution 'https://jenkins.tracfone.com/job/Kubernetes/job/Micro-Services//api/json?tree=name,url,displayName,fullDisplayName,jobs[name,fullName,displayName,fullDisplayName,jobs,url,color,buildable,inQueue,healthReport[description,iconUrl],lastBuild[id,url,building,result,number,displayName,fullDisplayName,timestamp,duration],property[parameterDefinitions[name,type,defaultParameterValue[value],description,choices]]]': Read timed out

What is causing that issue and how can I fix that ?
Regards !

Hi,
it seems that the server not response within the timeut for the Job 'cop' .
But there are jobs loaded visible as I can see.
Is message message shown on each refresh?
What happen if you open URL in Browser?
If it occurs I should add an configurable timeout setting

Hi,
yes other jobs are able to load only this specific job is getting Timeout.
And when i open the same URL in browser I'm able to see the response.

So do u think adding an configurable timeout setting could solve this ??

How long it loads in browser?

i clocked 50 sec

general for all?
maybe configurable I think. I will try to add this in the update this week

50 sec for the Job which is getting this error.
But it would be great to set 1 min Timeout

yes. I will create an setting in config dialog so you can configure it for all projects

Thanks :) 👍

thanks for feedback for the plugin

released with 0.13.3


Issues with Zabbix 3.0

While running the code with Zabbix 3.0 I get this error
2016-06-29 22:38:50,546 ERROR 4282 Uncaught exception: local variable 'timestamp' referenced before assignment
I am running it as
/usr/lib/zabbix/externalscripts/emc_vnx_stats_debug.py --serial  --ecom_ip  --ecom_user admin --ecom_pass #1Password --poolperf ""
This was working fine with Zabbix 2.0

Is there any way you can run it from the command line and give me the full stacktrace of the failure that would be great.  It's strange that it's throwing this exception.
I noticed that the script is named _debug.py, have you made any modifications to it that may have also caused this error to be raised?


Versioned "share" links are broken

Whenever I go to a record page on one of our dev servers and click on the "Versioned Link" from the "Share and Cite" button, I get this error:
Invalid Input
Given catalog object is not the same catalog used in the url. You may need to login to see the model or data. Please login to continue.

I'm pretty sure this will happen on any record page, but an example of one I've tried is https://dev.facebase.org/id/1-4ZW6. I'm also pretty sure this was working two weeks ago.
Note: for this issue, it's probably better to test somewhere other than rbk-dev, since rbk-dev is currently running on a non-master ermrest branch.

This is because of a recent change that I made in ermrestjs. This should be fixed with the recent commit in ermrestjs. The following versioned link is working now:
https://dev.facebase.org/id/1-4ZW6@2SG-NYW0-BNPW
@ljpearlman Can you please verify it?

Looks good, thanks!


feat: support subscription mode (year/month)

https://github.com/casdoor/casdoor/assets/54203997/961e0cf2-6ed5-44a8-a225-8203adad2708

https://github.com/casdoor/casdoor/assets/54203997/1be05f53-b387-4791-999b-c39149a1a015

@Chinoholo0807 fix:

Uploading 2023-08-30_15-06-02.mp4…


audio-device: update system effects when output changes

Description
When the output device changes, the "Play sound effects through" in Sound Preferences also changes
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

resolves #4706

Should this be a user preference?

Should this be a user preference?

in my use case the system sounds should always be listened from the current output device. Which use cases do you think would need that user preference?

I switch the audio to the headphones during calls but I don't want the system sounds there. Same when I listen to music on my HomePod.

Makes sense, I will add the preference.
On the other hand, the current audio-device binary asset used to change output and input device is only compatible with intel chips and I have a M2 chip so every time I want to make tests or used the extension I need to compile as universal binary, I want to change the package.json to use another library which compiles with an universal binary here. Do you think I should create another PR o can I change it here.

updated with the user preference, do you think is there any better naming for title and description?


Executable is unsigned

This may be a problem when distributing a nwjs app using node-webkit-updater (since it uses node-tar.gz).

I will close this issue because of the code rewrite, fell free to reopen if the problem is still there.


Regarding-your-joss-submission

Link to the joss review issue :
https://github.com/openjournals/joss-reviews/issues/5702
This issue is regarding your submission of the software paper BayesFlow to JOSS. I was trying to run the tests you mentioned in the tests folder using pytest. I am encountering the error because of the pyproject.toml file(this line in particular --cov BayesFlow). Can you please look into it once and respond.

Thank you for opening this issue.
Could you please try to install pytest-cov for the coverage metrics and see whether that resolves the issue?
pip install pytest-cov

Thanks,
Marvin

Hi @sandeshkatakam , the prerequisites for development should also be described here: https://bayesflow.org/contributing.html#environment
Specifically, running pip install -r requirements_dev.txt should install all necessary dependencies. @marvinschmitt it seems https://github.com/stefanradev93/BayesFlow/blob/Development/requirements_dev.txt could be updated, esp. to only require >= minimum versions?

I have updated requirements_dev.txt in Development to use at least the recent versions of the listed packages (>=) .

Thank you. The tests are running fine now.


updated microsoft office to 2016

As Office 2016 is now final, updates casks accordingly
Ref: https://github.com/caskroom/homebrew-cask/issues/14945

Merged as 980a41121cfcf0aa605d653b2d32a3cb2923cb18.


ensure controllers have a chance to run at least once

Currently, it is possible for the CLI to exit before controllers have a chance to act on changes. For example:

Run idp builder with custom package ./idpbuilder create --package-dir pkg/controllers/custompackage/test/resources/customPackages/testDir. Then wait for it to finish.
Make modifications to manifests under testDir.
Run the same command again. Expectation is for everything to sync. Instead, you may get error messages like:1.7054368071870215e+09	ERROR	controller.gitrepository	failed updating repo status	{"reconciler group": "idpbuilder.cnoe.io", "reconciler kind": "GitRepository", "name": "my-app2-app2", "namespace": "idpbuilder-localdev", "error": "client rate limiter Wait returned an error: context canceled"}
github.com/cnoe-io/idpbuilder/pkg/controllers/gitrepository.(*RepositoryReconciler).Reconcile
    /home/ubuntu/idpbuilder/pkg/controllers/gitrepository/controller.go:114
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
    /home/ubuntu/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:114
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
    /home/ubuntu/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:311
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
    /home/ubuntu/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
    /home/ubuntu/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:227

This is because we do not check if custom package and git repo controllers have worked on objects before exiting CLI.
We should use the commonly used observedGeneration field in Status, then check if the value matches with the manifest generation value before exiting.
However, it comes with an issue. For manifest generation to update, spec or status field must be updated. See: https://github.com/kubernetes/apiextensions-apiserver/blob/80788265e19419eafebe95c670234eb2659bc072/pkg/registry/customresource/strategy.go#L123-L151
I see a few ways to go about this.

Introduce a new optional spec field for each type that gets populated with a random value every CLI run to force generation update. Not great because run ID doesn't really have anything to do with CR spec like GitRepository or CustomPackages.
Update the status field of CustomPackages from localbuild controller to force generation update. This isn't good because status field is not meant to be used like this.
Don't use observedGeneration.  Annotate the CustomPackages objects with run ID. Then, CustomPackagess controllers can annotate them with a known value once they are done working on them. Could use a custom status field here as well.

I went with the first option for this PR to be inline with more common practices.

I don't really care which way we go. It's just that observedGeneration was perfect for this kind of uses so I wanted to stay closer to general practices, instead of inventing our own annotations.  I'll re-work it.

As an end user of the tool it is more confusing if I check the Spec for the resource and I am not sure what the value for the field in the spec is or what purpose it has. Also I think what we put in the spec becomes part of the API, where an input is expected. Which is why I am more hesitant here.

This is exactly what I would think. Why not make it an annotation and maybe instead of making it a random value make it the exact execution time of the cli run with enough precision to ensure uniqueness? Maybe we can call it cli-creation-time or something?

I agree that it's a better user experience, and uniqueness guarantee is probably not needed. Re-worked it to use annotations with timestamp (RFC3339Nano) instead.
Introduces two annotations.

cnoe.io/cli-start-time Contains timestamp that gets updated by CLI on startup.
cnoe.io/last-observed-cli-start-time Gitrepo and custom pkg controllers updates this value to match cnoe.io/cli-start-timevalue.


Extruded SVG image in AR.js throws getUniforms of undefined error.

What is the current behavior?
I want to import a SVG file into my AR.js scene to be displayed on a marker.
For that I am using the official Three.js SVGLloader[1]. While using this with pure Three.js I am able to load a SVG into my scene and display it. But when I use the exact same function combined with AR.js I get this error:
TypeError: Cannot read property 'getUniforms' of undefined
setProgram
https://localhost:3000/libs/three/example/vendor/three.js/build/three.js:24307:26
WebGLRenderer.renderBufferDirect
https://localhost:3000/libs/three/example/vendor/three.js/build/three.js:23313:18
renderObject
https://localhost:3000/libs/three/example/vendor/three.js/build/three.js:24041:11
renderObjects
https://localhost:3000/libs/three/example/vendor/three.js/build/three.js:24011:6
WebGLRenderer.render
https://localhost:3000/libs/three/example/vendor/three.js/build/three.js:23798:33

Please mention other relevant information such as the browser version, Operating System and Device Name
Windows - Chrome
What is the expected behavior?
To  see my extruded SVG logo in AR.js like I see it on Three.js
[1] https://threejs.org/docs/#examples/en/loaders/SVGLoader

Hi @ZHAWluci have you solved this? Maybe you are using a SVGLoader with a different version of Three.js.
I also moving this issue to discussions because not strictly related to AR.js.


Cleanup NetBeans license header for module html

Review and cleanup the license header in html
Oracle donated NetBeans to the Apache Software Foundation.
Your task now is to replace the old license header that has Oracle (and/or Sun Microsystems
and/or NetBeans) in it with the standard Apache Software Foundation license header.
For an empty file that supports headers you must add the Apache license header.
Note that you have to use the proper license header for Java, .properties or HTML files.
Look at the nearby files for templates and copy-paste the license header.
When done commit the changes with a commit message that starts with first line
"[NETBEANS-54] Module Review " and a message body describing your review.
See a longer explanation about what we are doing here: https://cwiki.apache.org/confluence/display/NETBEANS/List+of+Modules+to+Review
Please announce in a comment when you have started work.
Files that need to be looked at in html:
html/src/org/netbeans/modules/html/templates/html.html
html/src/org/netbeans/modules/html/templates/xhtml.xhtml
html/test/unit/src/org/netbeans/modules/html/data/UTF8.html
html/test/unit/src/org/netbeans/modules/html/data/apostrof.html
html/test/unit/src/org/netbeans/modules/html/data/apostrofwithoutquote.html
html/test/unit/src/org/netbeans/modules/html/data/empty.html
html/test/unit/src/org/netbeans/modules/html/data/sample.html
html/test/unit/src/org/netbeans/modules/html/data/wrongencoding.html

Working on this

Note to self: reserved in https://cwiki.apache.org/confluence/display/NETBEANS/List+of+Modules+to+Review

Hacktoberfest is over. Good job everybody!