Allow the rep to configure kernel parameters

Is your feature request related to a problem? Please describe.
Configuration on the kubelets could cause the following message when trying to run apps:
11:45:49.704: [PROXY.0] [2020-07-30 18:45:49.703][70][critical][assert] [source/common/filesystem/inotify/watcher_impl.cc:28] assert failure: inotify_fd_ >= 0. Details: Consider increasing value of user.max_inotify_watches via sysctl

Describe the solution you'd like
Don't do this:
https://github.com/cloudfoundry-incubator/kubecf/blob/master/deploy/helm/kubecf/assets/operations/instance_groups/diego-cell.yaml#L45-L47
This does not exist anymore:
https://github.com/cloudfoundry-incubator/kubecf/blob/11797e25f3f1427f3b061d574eb255b795f23bf3/deploy/helm/kubecf/assets/operations/instance_groups/api.yaml#L164
We should look at these as well:
https://github.com/cloudfoundry-incubator/kubecf/blob/1e3b51ca95dd6469815089c809d1f2747a659a69/deploy/helm/kubecf/assets/operations/instance_groups/router.yaml#L26
https://github.com/cloudfoundry/diego-release/search?q=set_kernel_parameters&unscoped_q=set_kernel_parameters

@mook-as noted this on AWS:
==> /proc/sys/user/max_inotify_instances <==
128
==> /proc/sys/user/max_inotify_watches <==
8388608

Places to look:
https://github.com/cloudfoundry/diego-release/blob/e22fda5d7367e606b20392b732431bf6ce82a164/jobs/rep/templates/set-rep-kernel-params.erb#L24-L26
https://github.com/cloudfoundry-incubator/kubecf/blob/6a25968a58af5e2f5b9809f2cc5373996d92c75c/deploy/helm/kubecf/assets/operations/instance_groups/diego-cell.yaml#L45-L47

Mark also filed https://github.com/envoyproxy/envoy/issues/12383 upstream to ask for a more correct error.

Mainly @viovanov , plus @mook-as
Questions:

What was the k8s foundation (minikube, kind, AWS, CaaSP, ...) ?
What kubecf release/commit was used, what cf-deployment version ?
What log file contains the assert failure message ?

Is that generally something we should see with minikube (also) ?

When you (Vlad) said This does not exist anymore, before the reference to
https://github.com/cloudfoundry-incubator/kubecf/blob/11797e25f3f1427f3b061d574eb255b795f23bf3/deploy/helm/kubecf/assets/operations/instance_groups/api.yaml#L164
What does not exist exactly ?
Looking into kubecf commit 25ecfdf34d (current master), file src/cf-deployment/cf-deployment.yml I find:

IG api exists
Job file_server exists.
Release diego is version 2.47.0.
Job in that release has the property set_kernel_parameters (end of spec file).
https://github.com/cloudfoundry/diego-release/blob/2c9202c470bc6e28c448f6f9ce18356a2461cc7c/jobs/file_server/spec#L83-L85

Ditto all around for job rep in IG diego-cell.

Is the requested solution the removal of all the patch instructions setting all set_kernel_parameters properties to false ?

Trialed removal of the patches for router, file_server, and rep.
This breaks the containing IGs (router, api, diego-cell), they are not completing their initialization.

The last thing I confirmed Friday was that the writes in the garden init container were visible to the garden job container.
According to @viovanov setting kernel parameters in rep is ok because they affect the kernel of the worker node running the entire diego-cell group of jobs, and are thus visible to garden, like in a BOSH setup.

Confirmed


fix spelling mistake

was looking at how to use combo and realized that shift was spelled wrong

Thanks!


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 4704 ms

Resolved: Bofrost.at performance has improved in d338440 after 1 hour, 26 minutes.


Add negative example warning

This closes #1986.
I suggest a simple HTML comment to warn readers about the negative example. I saw a similar usage in the Slots page.
However, the whole section could be rearranged differently, by providing a better context before showing the example. We could even use a tip.
Any thoughts?
Thanks!

Interesting, I wouldn't have thought this would be an issue, but I appreciate the reference to the issue and also an existing example. If it helps people grok things faster, I don't see the harm. Thanks!


Experiments of 3D tea pod

Hi, I am planning to reproduce your experiments on 3D teapot experiment, but I am unable to find in this repo the corresponding code. Would please give me a pointer / add the code to this repository?

Hi, thanks for your interest. The 3D teapot experiment is in another repo from my coauthor @ondrejba. You can try emailing him. I'll also let him know.

Thank you. I'll send a message to him.


What about add these 2 features?

If there are amount of complex data in datatable, I think the following two features will be very helpful.

An option to show header both on top and bottom.
Make header on top fixed when user scrolls down. (like this one: http://legacy.datatables.net/extras/fixedheader/)

Both of the features can avoid showing double vertical scrollbars and make users know the title of each column. (browser scrollbar and datatable scrollbar)

@thundermiracle thanks your suggestion, I'll consider to implement it. 👍

I don't think the no1 is difficult to implement.
And about no2, I got a CSS trick to make it possible.

Create a class and add following CSS:

position: sticky;
top: 0;

Pass the class to headerContainerClass

I think you can make a new prop to avoid achieving this feature by CSS  and let user set top freely as well.


SetMaxMessageSize not working in GRPC 1.0.0

Code is like this:
ServerBuilder builder;
const int DefaultMaxMessageSize = 64 * 1024 * 1024;
// set max request size
builder.SetMaxMessageSize(DefaultMaxMessageSize);
builder.AddListeningPort(serverAddr_, grpc::InsecureServerCredentials());
builder.RegisterService(&service_);
server_ = builder.BuildAndStart();
Finally, when I want to receive a very big package. GRPC returns 13 with "error msg: Max message size exceeded"

How can I set max response size in C++ of GRPC 1.0.0?


[gassist] FYI this also works for samsung's bixby but not yet chatgpt

hi,
originally i wanna modify this for chatgpt,
however i found that this app simply calls android.intent.voice_command.
i.e. it's non specifc, it calls any default voice command input.
if it is google assistant, then this app calls it.
if it is samsung bixby (as in my samsung a33), then this app calls bixby.
as in the espruino forum's project page, i wanna make it for chatgpt but currently need extra help from OPENAI as the chatgpt apk's use of intent is not well documented.
thanks

Please can you avoid posting 'project' posts/threads like this? If you have questions please ask them in the forum - but it looks like you might already be asking for help there on exactly this?
https://forum.espruino.com/conversations/391050/#comment17161344

Well that's fine as you closed it, but posting here won't notify the author.
You'll have to look at the commit history for the app and then reference the author directly with @ and their name.


Error: Input required and not supplied: token

I keep getting an error at Actions check-out. Why am I getting an error when I input GH_PAT?
Run<EMAIL_ADDRESS>  with:
    repository: r2fresh/chelsea
    ssh-strict: true
    persist-credentials: true
    clean: true
    fetch-depth: 1
    lfs: false
    submodules: false
Error: Input required and not supplied: token

Seems like the GH_PAT was not saved then, it all looks good now! https://github.com/r2fresh/chelsea/actions


OBR hl7v2 to fhir mapping updates

This PR updates the Service Request and Diagnostic Report resources from the HL7 v2 OBR segment using the latest HL7 v2 to FHIR mapping documents. It also added some mapping between OBR and the Specimen resource.
Updated mapping documents for OBR, copy of file attached:
HL7 to FHIR - OBR Segment.xlsx
Test Steps:

Build this branch and run the unit and integration tests. Verify they all pass. Alternatively you can verify the Github Actions build passes. Note the HL7 to FHIR to HL7 integration test data was not changed, so the result of our HL7 to FHIR to HL7 conversion remains the same = no data changes.

Changes

Updated HL7 v2 to FHIR mappings
Updated FHIR files in integrations tests

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?

Linked Issues

Fixes #7566

Integration tests passed locally. Reviewed the changes on the FHIR samples and nothing major changed so looks good. I tried reviewing which fields changed on the spreadsheet, but I couldn't find them, did you mark them as Updated?

@victor-chaparro I suppose I didn't use the Updated label, I just made updates as needed. I can go back and trying to change the labels, but especially since the HL7 working group's documentation doesn't have a change log or anything, we'd probably have to go through the whole set next time we go through this process anyway, so the distinction between "Added" and "Updated 1/20/23" isn't too useful anyway.

There was a merge conflict and I'm having trouble resolving that, going to convert this to a draft PR while I try to work through this.


Add TEST_CASEs for .serialise() and .deserialise(...) methods (surrogate models)

Subtask of #56

[ ] surrogateModel.cpp
[ ] surrogateModel/polynomialFunctionModel.cpp
[ ] surrogateModel/radialBasisFunctionModel.cpp
[ ] surrogateModel/krigingModel.cpp
[ ] surrogateModel/multivariateAdaptiveRegressionSplinesModel.cpp
[ ] surrogateModel/artificialNeuralNetwork

Done by 3751b33a9572c94112341b3f21f60a88f035de98


rkts data import: override previous version

I changed rdf:eventWho to bdo:eventWho in https://github.com/buda-base/rKTs-migration/commit/992d4d1b533c60fbb52f52556b33724956d15426 but now http://purl.bdrc.io/resource/W23703_3082.ttl contains both, it should just contain the new one

I think the issue is w/ gittodbs. The W23703.trig in the repo written by xmltold looks fine - just the :eventWho. I'll look at gittodbs. I think it just reads in the trig file and writes it to fuseki w/o dropping the graph from fuseki so it's just adding content. No deletions


Made with CNL

I think it'd be useful to have a list of projects (if we know of any) that are using the library to handle their numeric calculations. Likewise it may be worth adding the cppcon youtube videos of CNL onto the README as an intro.

Oh you mean embedded? I hadn't thought of that. Generally I need to do a major sweep of the docs. Thanks

I'll add to this issue as I try integrating this library. I think other stuff missing from the docs is the multiply api has change, there's safe_multiply now as well, etc.
I do like the saturated_overflow and the multiply(tag idea. It allows me to develop with the exception tag and then switch to deterministic saturated tag in production.

ok so I kept getting confused by the docs / presentations / current code.
It sounds like the explicit arithmetic operators ("multiply", "divide") got dropped from the proposal.
And so it seems at the moment, if I want to implement fixed point math, is I should do something like
template<
        int IntegerDigits,
        int FractionalDigits,
        class Narrowest>
using saturated_elastic_fixed_point = cnl::fixed_point<
        cnl::overflow_integer<
                cnl::elastic_integer<
                        IntegerDigits+FractionalDigits,
                        Narrowest>,
                cnl::saturated_overflow_tag>,
        -FractionalDigits>;
using real_t3 = saturated_elastic_fixed_point<7, 8, std::int16_t>;

and then just treat it as if were a real float type. I can change the tag from saturated to throwing as I develop, and it seems this will just play well with std::sqrt too

I don't see a divide function available in the api besides divide, but divide only seems to work on ints. It seems to work on fractions like 0.71 or 0.6, but with 0.7, the final value is reported to be zero?
    template<
            int IntegerDigits,
            int FractionalDigits,
            class Narrowest>
    using safe_elastic_fixed_point = cnl::fixed_point<
            cnl::overflow_integer<
                    cnl::elastic_integer<
                            IntegerDigits+FractionalDigits,
                            Narrowest>,
                    cnl::throwing_overflow_tag>,
            -FractionalDigits>;

    using real_pos_t = safe_elastic_fixed_point<32, 8, std::int16_t>;
    real_pos_t not_zero = real_pos_t(7) / real_pos_t(127);
    ASSERT_NEAR(float(not_zero), 90.f/127.f, 0.01);

tl;dr: The API has changed a little since the last time I updated the docs. (Sorry about that.) See option 1 below for the new API.
The result of the / operation has no fractional digits as both numerator and denominator have the same exponent. Here's the same test without the conversion back to real_pos_t. It demonstrates that the result has no fractional digits.
This is vexing because the user almost always expects a result with the same precision as the inputs. However, consider 1/3. How do you chose the right number of digits? The answer is that it's arbitrary and the user has to make a choice. The default which CNL provides is the one with the least run-time cost but there are some alternatives:

The free function, divide, has been replaced with a type, fraction. (It was briefly called fractional.) You can now chose exactly how much precision you get using the fraction API. Here's your example using fraction.
You can manually increase the precision of the numerator ahead of performing the division. (Example)
Finally, for lossless arithmetic, you can deal with quotient and remainder just like with raw integer division. Remember that fixed-point is essentially a superset of integer and division is backed by integer division. Any attempts to make fixed-point division work like floating-point division will be limited.

(Note in that last example, I removed elastic_integer from the type. For some reason, it's producing an error. I've filed an issue. It's probably not your first choice anyway but let me know if I'm wrong.)
I've also filed an issue to make it clear how to generate the docs in the README. The reason for the delay updating the docs is that a lot of source has moved around as I break down large header files and Doxygen isn't the best for diffing changes to make sure that nothing has been lost. Again, apologies for not doing this sooner.

this all worked for me thanks.

CDSP is mentioned in the readme. And multiply is removed from the docs. However, there is still work to do with the examples on the front page.


Terms of Service URL mismatch

[ 2.5.0] What version of the library are you using? Is it the latest version? The latest released version is

What kind of issue is this?

[ X] Bug report.
If a specify a termsOfServiceURL in my ApiInfo, it is returned in the api-docs json structure as termsOfService:

"info": { "version": "0.5", "title": "Service", "termsOfService": "https://www.blah.com", "contact": { "name": "Someone", "url": "https://www.blah.com", "email"<EMAIL_ADDRESS>} }
But the swagger-ui code is expecting this value to be termsOfServiceURL:
https://github.com/swagger-api/swagger-ui/blob/master/src/main/template/main.handlebars#L9
ApiInfo class used to generate APIDocs:
https://gist.github.com/jonhines/71686e651bc08737ad9f4e2472626eaa

We haven't updated swagger ui to master (2.1.5). This is a duplicate of #1415

I still have this issue.
swagger-annotations: 1.5.10
springfox-swagger-ui: 2.6.0
springfox-swagger2: 2.6.0

What you see when you look at the api-docs /v2/api-docs? If that looks correct, then its an issue with swagger-ui using a cached version.

It returns termsOfService

Then try clearing your browser cache or force reloading resources so that you have the latest swagger-ui


CASMTRIAGE-6575-1.5: One additional pod getting listed for keycloak - unexpectedly

Description
Fixes CASMTRIAGE-6375 for CSM 1.5
Tested on starlord
Checklist

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.6


Fix/consul poller option

Fixes / consul poller option#
Proposed Changes
pollingInterval property of serviceDiscoveryProvider setting is now effective

Hello,

Do we need to update documentation to explain how this feature/fix works?
We need unit test the new code, please can you do it?
Integration tests if it's the case too.

Hello.
this fix don't need to update documentations.
i created an unit test for "ConsulFileConfigurationPollerOptionTest"


Unable to compile with 4.1.3 (probably anything since 4.1)

Attempting to compile for 4.1.3 (or probably anything since 4.1 wont work due to changes in the way gdextension_init() works in register_types
i'll fix it up and make a PR

Mostly fixed in https://github.com/nathanfranke/gdextension/pull/8
I omitted your suggestion to have absolute paths in example.gdextension. It works fine for me with relative paths.
Working for me, Godot 4.2.1 on Linux.

Should be working as of #8


Automatize docker image update

After last merge docker image on docker hub was not regenerated.

I checked the settings on docker hub. It should work... Let's see if this persist or was a one-time event only.

@philipphenkel
Can you please update the image on dockerhub. I see it is last pushed 2 years ago.

@philipphenkel  I forked and made an autobuild docker image on dockerhub (misi/bittorrent-tracker)until you can have the time to update your original reposiotry.


Fix typo

Hi @krysros, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Thanks!


Format misplaces namespace with commented attribute

Input:
//@armNamespace
namespace Microsoft.Foo {
}

Run this command:
cadl format **\*.cadl

Result:
namespace Microsoft.//@armNamespace
Foo {

Run it again:
namespace Microsoft.Foo { //@armNamespace
}

Did get into this issue as well. Should have been fixed in https://github.com/microsoft/cadl/pull/374

https://aka.ms/trycadl playground is always using the current version of cadl (main branch) so the fix should be there. Please reopen if it is still happening for you there


Add /api/v1/accounts &  /api/v1/accounts/balance

I finally opted to add the support of

accounts:account_id\balance

method instead of

account\balance?account_id=

Note that I didn't upload the documentation, I'm not sure about what tool you use to generate it,
Let me know if you want me to make any change !
Cheers,

Hey @etiennetatur. I'm running the tests on this and getting the following fails.
Can you revisit and make sure your tests are still passing?
 1) coinbase.accounts.balance should return the user's account balance:

  CoinbaseError: Invalid account
      at Request._callback (/Users/matt/development/coinbase/lib/index.js:64:27)
      at Request.self.callback (/Users/matt/development/coinbase/node_modules/request/request.js:344:22)
      at Request.emit (events.js:98:17)
      at Request.<anonymous> (/Users/matt/development/coinbase/node_modules/request/request.js:1239:14)
      at Request.emit (events.js:117:20)
      at IncomingMessage.<anonymous> (/Users/matt/development/coinbase/node_modules/request/request.js:1187:12)
      at IncomingMessage.emit (events.js:117:20)
      at _stream_readable.js:944:16
      at process._tickCallback (node.js:442:13)

  2) coinbase.button should generate a new button:

  CoinbaseError: Merchant tools are disabled for this account. Please contact support.
      at Request._callback (/Users/matt/development/coinbase/lib/index.js:96:25)
      at Request.self.callback (/Users/matt/development/coinbase/node_modules/request/request.js:344:22)
      at Request.emit (events.js:98:17)
      at Request.<anonymous> (/Users/matt/development/coinbase/node_modules/request/request.js:1239:14)
      at Request.emit (events.js:117:20)
      at IncomingMessage.<anonymous> (/Users/matt/development/coinbase/node_modules/request/request.js:1187:12)
      at IncomingMessage.emit (events.js:117:20)
      at _stream_readable.js:944:16
      at process._tickCallback (node.js:442:13)

Hi Mateo,
Account balance : In the test you can specify an account id for the balance. I guess it could have been set up as a environment variable but it's not.
Coinbase button : Did you enable merchant tools on your account ?

Sounds like it should be made into an env variable. Can you refactor it into one and throw if it's not provided, similar to here: https://github.com/etiennetatur/coinbase/blob/master/test/index.js#L10 ?

Updated


hhea.advanceWidthMax might be a bad value to rely on for monospace_max_advancewidth check

Observed behaviour
In Recursive Mono, I get notified that This seems to be a monospaced font, so advanceWidth value should be the same across all glyphs, but 99.6% of them have a different value.
 >> com.google.fonts/check/monospace_max_advancewidth
    Monospace font has hhea.advanceWidthMax equal to each glyph's advanceWidth?
    with fonts_1.036/Static_OTF/RecursiveMonoLnr-ExtraBold.otf

    * WARN: This seems to be a monospaced font, so advanceWidth value should be the same across all glyphs, but 99.6% of them have a different value: space, uni00A0, uni2007, uni2008, uni2
009, uni200A, A, Agrave, Aacute, Acircumflex and 1232 more. [code: should-be-monospaced]
    * WARN: Double-width and/or zero-width glyphs were detected. These glyphs should be set to the same width as all others and then add GPOS single pos lookups that zeros/doubles the widt
hs as needed: uni02BC, uni0300, uni0301, uni0302, uni0303, uni0304, uni0306, uni0307, uni0308, uni0309 and 60 more. [code: variable-monospaced]

    Result: WARN

However, this assumes that the font's maximum width value is its primary width value, which is not the case here. Rather, the normal value is 600 units, and the maximum of 2400 comes from only a few code ligatures.
1099 of 1247 glyphs have a width of 600, meaning that only 11.87% of them have a different value. Moreover, a check shows that all glyph widths are divisible by the normal width of 600 with no remainder, so the font should theoretically have no issues of non-monospaced glyphs.
Of course, some terminals (e.g. the VS Code terminal) will not work with glyphs over the 600-unit width, but those also don't support dlig, so the only problems I've run into in the VS Code terminal are:

The f.italic gets cut off because it goes beyond its sidebearings
I occasionally get a warning about it being a non-monospaced font (but it still works, anyway)

Expected behaviour
Instead of basing the "correct" monospace value on hhea.advanceWidthMax, I think it might make sense to base it on something like the space or a glyph.
If there is reason to base it on the hhea.advanceWidthMax, we should at least update the check rationale to say why we are flagging it as a WARN, and perhaps calculate the "different value" metric.
Resources and exact process needed to replicate
Here's a recent version of Recursive which can be used for this check:
https://github.com/arrowtype/recursive/blob/1a1680256d89d6856f9e4eef06ceae247a804537/fonts_1.036/Static_TTF/RecursiveMonoLnr-ExtraBold.ttf

I am seeing that the com.google.fonts/check/monospace does provide more nuance:
 >> com.google.fonts/check/monospace
    Checking correctness of monospaced metadata.
    with fonts_1.036/Static_TTF/RecursiveMonoLnr-Regular.ttf

      Rationale:
      There are various metadata in the OpenType spec to specify if a font is
      monospaced or not. If the font is not trully monospaced, then no
      monospaced metadata should be set (as sometimes they mistakenly are...)

      Monospace fonts must:

      * post.isFixedWidth "Set to 0 if the font is proportionally spaced,
      non-zero if the font is not proportionally spaced (monospaced)"
        www.microsoft.com/typography/otspec/post.htm

      * hhea.advanceWidthMax must be correct, meaning no glyph's width value is
      greater.
        www.microsoft.com/typography/otspec/hhea.htm

      * OS/2.panose.bProportion must be set to 9 (monospace). Spec says: "The
      PANOSE definition contains ten digits each of which currently describes
      up to sixteen variations. Windows uses bFamilyType, bSerifStyle and
      bProportion in the font mapper to determine family type. It also uses
      bProportion to determine if the font is monospaced."
        www.microsoft.com/typography/otspec/os2.htm#pan
        monotypecom-test.monotype.de/services/pan2

      * OS/2.xAverageWidth must be set accurately.
        "OS/2.xAverageWidth IS used when rendering monospaced fonts, at least
      by Windows GDI"
        http://typedrawers.com/discussion/comment/15397/#Comment_15397

      Also we should report an error for glyphs not of average width

    * WARN: Font is monospaced but 148 glyphs (11.858974358974358%) have a different width. You should check the widths of: ['uni02BC', 'uni0300', 'uni0301', 'uni0302', 'uni0303', 'uni0304', 'uni0306', 'uni0307', 'uni0308', 'uni0309', 'uni030A', 'uni030B', 'uni030C', 'uni030F', 'uni0311', 'uni0312', 'uni0315', 'uni031B', 'uni0323', 'uni0324', 'uni0326', 'uni0327', 'uni0328', 'uni032E', 'uni0331', 'uni0335', 'uni200B', 'caronslovakcomb', 'dotsidecomb', 'horncombo', 'ogonekcombo', 'ringacute', 'uni0337', 'f_f', 'uniFB01', 'uniFB02', 'f_f_i', 'f_f_l', 'uni0302_uni0301', 'uni0302_uni0300', 'uni0302_hookabovecomb', 'uni0302_uni0303', 'uni0306_uni0301', 'uni0306_uni0300', 'uni0306_hookabovecomb', 'uni0306_uni0303', 'uni030C.alt', 'uni0300.case', 'uni0301.case', 'uni0302.case', 'uni0303.case', 'uni0304.case', 'uni0306.case', 'uni0307.case', 'uni0308.case', 'uni0309.case', 'uni030A.case', 'uni030B.case', 'uni030C.case', 'uni030F.case', 'uni0311.case', 'uni0312.case', 'uni0315.case', 'uni031B.case', 'uni0323.case', 'uni0326.case', 'uni0327.case', 'uni0328.case', 'acutecombviet.case', 'gravecombviet.case', 'horncombo.case', 'ogonekcombo.case', 'ringacute.case', 'uni0337.case', 'tildecombviet.case', 'uni01C6.italic', 'uni01C9.italic', 'uni01CC.italic', 'uni01C8.italic', 'uni01CB.italic', 'f_f.italic', 'f_f_i.italic', 'f_f_l.italic', 'uniFB01.italic', 'uniFB02.italic', 'f_f.mono', 'f_f_i.mono', 'f_f_l.mono', 'uniFB01.mono', 'uniFB02.mono', 'quotesingle_quotesingle_quotesingle.code', 'quotedbl_quotedbl_quotedbl.code', 'asterisk_asterisk.code', 'asterisk_asterisk_asterisk.code', 'asterisk_equal.code', 'asterisk_slash.code', 'plus_equal.code', 'plus_plus.code', 'plus_plus_plus.code', 'minus_equal.code', 'equal_equal.code', 'equal_equal_equal.code', 'equal_greater.code', 'equal_slash_equal.code', 'less_equal.code', 'less_exclam_hyphen_hyphen.code', 'less_hyphen.code', 'less_less.code', 'less_less_less.code', 'greater_equal.code', 'greater_greater.code', 'greater_greater_greater.code', 'greater_hyphen.code', 'f_quotesingle.code', 'underscore_underscore.code', 'hyphen_greater.code', 'hyphen_hyphen.code', 'hyphen_hyphen_greater.code', 'hyphen_hyphen_hyphen.code', 'hyphen_less.code', 'hyphen_space_bracketleft_space_bracketright.code', 'hyphen_space_bracketleft_x_bracketright.code', 'numbersign_numbersign.code', 'numbersign_numbersign_numbersign.code', 'numbersign_numbersign_numbersign_numbersign.code', 'ampersand_ampersand.code', 'ampersand_ampersand_ampersand.code', 'slash_asterisk.code', 'slash_equal.code', 'slash_slash.code', 'slash_slash_slash.code', 'backslash_b.code', 'backslash_n.code', 'backslash_r.code', 'backslash_t.code', 'backslash_v.code', 'bar_bar.code', 'bar_bar_bar.code', 'colon_colon.code', 'colon_slash_slash.code', 'exclam_equal.code', 'exclam_equal_equal.code', 'exclam_exclam.code', 'question_colon.code', 'question_period.code', 'question_question.code', 'dollar_braceleft.code', 'grave_grave_grave.code'] [code: mono-outliers]

According to the specs, hhea.advanceWidthMax is a calculated value:

I think that the problem with com.google.fonts/check/monospace_max_advancewidth is the false assumption that all glyphs in a monospaced font would have the same advanceWidth as the maximum value.
If the purpose of the check was to ensure that hhea.advanceWidthMax is properly calculated, then the wording of the log messages convey the wrong idea by suggesting the problem is with the advanceWidths of a bunch of glyphs.
Also, we already have a check for ensuring hhea.advanceWidthMax is properly calculated: That's com.google.fonts/check/maxadvancewidth and it looks fine.
And we also have a check for the advanceWidths of all glyphs that carefully determines which is the largest group of glyphs with the same value and then ensures such group corresponds to more than 80% of the glyphset, which is pretty reasonable to accommodate the eventual outliers. That's com.google.fonts/check/monospace,  as @thundernixon mentioned above.

So, my conclusion is that com.google.fonts/check/monospace_max_advancewidth was a mistake and has no actual usefulness, so it will be deprecated.
If anyone knows of an actual value of keeping it (even if including changes to it's implementation) please let me know and I'll be glad to revert the deprecation, as long as we document the rationale for keeping it.

So, there are instances in which the max advance width can cause issues if it is too wide, especially in some older code/terminal environments. For example:
https://github.com/googlefonts/Inconsolata/issues/42
I know that Fira Code and Hasklig use a surprising approach to code ligatures, to ensure that everything stays within the same single-letter width.
So, I don’t think it’s necessarily a terrible check, but the rationale definitely could use an upgrade.

thanks, @thundernixon!
If we can craft a short rationale text describing the actual purpose of the check, we can open a new issue to either bring back the check or to implement a new, reasonable one.

Good plan! I’ll try to help think of something.
Found some detail from the Fira Code author on why the libraries have single width, plus blank spaces: it makes it easier for programs to insert the cursor at the proper place.
I know that even the terminal in VS Code gets messed up from proportional glyphs (somewhat surprising, because the code editor part has no trouble), though I think it probably uses average width rather than max width.
https://www.patreon.com/posts/fira-code-inside-28074541
Input Mono describes some issues with editors and terminals that don’t support non-fixed-width fonts. This is probably slightly out of date, but likely some/many of the issues still exist.
https://input.fontbureau.com/workarounds/
Fonts that aren’t marked as isFixedPitch will not show I up in many Windows software font menus (https://github.com/google/fonts/pull/1824). So, perhaps there is another check for that, but if not, this check could maybe suggest that people set that to 1 if they intend their font to be used for code.

Maybe the rationale could be something like:

INFO: The hhea.advanceMaxWidth is X, but the average width is Y. This can cause layout and cursor-placement issues in some terminals and code editors. If this font is intended for code, make sure that isFixedPitch is set to 1, and consider whether glyphs can be given a uniform width. For a way to make code ligatures with fixed widths, see https://www.patreon.com/posts/fira-code-inside-28074541.

(Though, this ignores that I think code ligatures can be better as dlig than as calt. I need to try the Fira code width hack with dlig, to see whether that can work. If so, I could write a document on GitHub about the workflow, and we could link to that.)

I've check in st, and xOff value returned in XGlyphInfo is 3 times as high as it should be. As mentioned before it seems to be related to added ligatures (biggest ligature glyph, eg <=> is thrice as big as normal letter). In st apply this patch:
diff --git a/x.c b/x.c
index 36dd8bc..68781cf 100644
--- a/x.c
+++ b/x.c
@@ -953,7 +953,7 @@ xloadfont(Font *f, FcPattern *pattern)
 	f->rbearing = f->match->max_advance_width;

 	f->height = f->ascent + f->descent;
-	f->width = DIVCEIL(extents.xOff, strlen(ascii_printable));
+	f->width = DIVCEIL(extents.xOff > 18 ? extents.xOff / 3 : extents.xOff, strlen(ascii_printable));

 	return 0;
 }

Increase the font with Ctrl+Shift+PageUp. The font looks normal.


Make sure backup/restore tests are non-parallel

Get frequent conflicts when running live tests; though, I'm not sure how
much this will help because the RecordingTestBase itself is marked as
[NonParallelizable], which is an inheritted attribute. Won't hurt,
though. Could just be a service issue because, sometimes, an LRO may not
be completely done when it says it is and we're stressing the service
pretty hard (and rather impractically).

API change check
API changes are not detected in this pull request.


[Bug]: Component with form - runtime replacement

Package name
components
Package version
10.0.0-next
Reproduction steps
on the latest version 10.0, in the showcase app, on the component replacement chapter, there is a hebrew flavor of date-picker.
Add a button which will switch the hebrew flavor with the default (gregorian) one via the config change.
ref: component replacement doc
Current result
The components switch is ok, but there is an error for the propagation of the from control changes to the parent. It seems that the register function for 'onChanges' and 'onTouched' are not called by angular at a second initialialization (or switch) of a component via the c11n directive. (there is the error message in the console as ref)
Expected result
The components to be switched and the changes from the form control propagated to the parent.
Keep the switch button into the showcase comp replacement page.
Additional comments
No response

Fixed, it was linked to pnp and double angular version


fix: Ensure drilldown has the right height inside of responsive top b…

…ar when toggled #11759
Fixed the _back function so that $currentMenu is updated to the root menu if the current menu doesn't have a parent submenu.

Description

...

Closes

Types of changes

[ ] Documentation
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (anything that would change an existing functionality)
[ ] Maintenance (refactor, code cleaning, development tools...)

Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] The pull request title and template are correctly filled.
[x] The pull request targets the right branch (develop or develop-v...).
[x] My commits are correctly titled and contain all relevant information.
[ ] I have updated the documentation accordingly to my changes (if relevant).
[ ] I have added tests to cover my changes (if relevant).

Hi @Buddierdl,
Thanks for finding this bug and especially for providing the PR.

Glad to help. It is my first PR and bug report so I hope I did everything right.

How do I get this merge approved?

Hi @Buddierdl,
Can you rebase so that the latest changes are included and we get a fresh CI build?

@DanielRuf Done. Can it be merged now?

@DanielRuf Done. Can it be merged now?

Awesome, I'll try to check it in the next days and give a review.


Private/Public

Hi,
Were these 2 definitions intentionally switched  => private points to public & public points to private ?
https://github.com/futurice/terraform-examples/blob/818250f9d65464c095e7d47f0fe03711a2a7d7ae/aws/wordpress_fargate/vpc.tf#L6

No good point. That is a bug. Will fix it soon.


React-dnd 3.0.0 Error in IE 11:  "Unhandled promise rejection SyntaxError"

I try the version 2.6.0 is ok, but the 3.0.0 will get this error
in react-dnd/lib/DragDropContext.js

This is probably because it's transpile target is ES2015. The next version of 3.x will target ES5. Please use 2.x for now


fix(ui): codemirror editor + module graph

This PR fixes:

codemirror mode map: source code editor loads without highligthing
graph tab: the graph style should fix its size and so the svg will fit to it, added style with css var + resize observer to fix the size

/cc @DerYeger

@DerYeger when selecting the graph of any test first time resizing the file details tab makes the scg to be centered, but once we select another module stops working (the resize seems to be no effect).

@DerYeger when selecting the graph of any test first time resizing the file details tab makes the scg to be centered, but once we select another module stops working (the resize seems to be no effect).
EDIT: it seems complex graphs also fails, only with simplest ones seems to work.

I'm not sure what your changes are attempting to do. The graph is already resizing to fit the available space exactly.

@DerYeger  https://imgur.com/glnIBCN

@DerYeger https://imgur.com/glnIBCN

See my review comment. Maybe changing the value to 0.01 is the best solution. AFAIK there's no way to detect if a resize was caused by the container actually being resized or v-show being toggled. That's the core issue here.

@DerYeger with 0.1 seems to be ok, 0.01 is very small.

Merging now so these issues are solved, we should check how state is going to be handled more generally in the app later. @antfu maybe we should use pinia to manage the state of the App instead of injecting individual pieces?

Merging now so these issues are solved, we should check how state is going to be handled more generally in the app later. @antfu maybe we should use pinia to manage the state of the App instead of injecting individual pieces?

I don't mind either options, feel free to do so if you find it's neccessary


Would there be an option to swap out the BME280 for BME680 which monitors CO?

Would there be an option to swap out the BME280 for BME680 which monitors CO among other things?
CO2meter lists other sensors but are way more expensive to monitor carbon monoxide
https://www.co2meter.com/collections/co-sensors
An article that talks about CO sensors
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC6541011/
Specs of BME680 but Adafruit also has one
https://www.waveshare.com/wiki/BME680_Environmental_Sensor

Yes I thought about adding a CO sensor but ultimately decided not to:

I already had commercial CO monitors
I did not want to be tempted to just use my own and have even the slightest possibility to die from a leak because of some silly bug I introduced

The BME680, if I'm not mistaken, has no separate readout for just CO.


Fix typo in private-cloud storage policy

Fixes typo in private-cloud storage IRSA policy.
Available values for "Effect" field are "Allow" and "Deny".

Thanks for the code fix Anton!


CSHARP-819 Add option to keep contact points unresolved and always resolve contact points on total connectivity loss

To accomplish this I have created the concept of IContactPoint which is separate from IConnectionEndPoint and IEndPointResolver. Both IContactPoint and IEndPointResolver can be resolved or not depending on the implementation.
IEndPointResolver is used to obtain IConnectionEndPoint instances from Host instances (for either control connections or pool connections). Multiple different enpoint instances resolved from the same Host can be used to connect to that same host.
IContactPoint is used to resolve IConnectionEndPoint instances from the provided contact points (for control connections only). Endpoints resolved from a single contact point can belong to different hosts.

I ran a test with two different remote clusters (n=3) and a local dns server. I coded a simple app that continously ran requests while I stopped one cluster and changed the dns server to start resolving to the other cluster which was still up and the app was able to reconnect with the new setting disabled and enabled.
I also tested taking down individual nodes instead of the entire cluster until the control connection attempted to reconnect to a host that was resolved with the contact point (host.ContactPoint != null) and in this case the app started using the other cluster when the new setting was enabled even when there wasn't a total connectivity loss which is expected.


psycopg2.errors.DuplicateTable: relation "health_check_db_testmodel" already exists

Environment

Python version: 3.9
Nautobot version: 1.1.3-beta.1 (9f52e6ece1f9f0e3b8a68fd66b63dc1a2c6b05d2)

Steps to Reproduce

Run an earlier version of Nautobot 1.1.x
Upgrade to 9f52e6ece1f9f0e3b8a68fd66b63dc1a2c6b05d2
Run nautobot-server migrate or nautobot-server post_upgrade

Expected Behavior
Server continues functioning

Observed Behavior
Performing database migrations...
Operations to perform:
  Apply all migrations: admin, auth, circuits, contenttypes, dcim, dummy_plugin, extras, ipam, sessions, social_django, taggit, tenancy, users, virtualization
Running migrations:
  Applying extras.0012_healthchecktestmodel...Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 82, in _execute
    return self.cursor.execute(sql)
psycopg2.errors.DuplicateTable: relation "health_check_db_testmodel" already exists

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/bin/nautobot-server", line 5, in <module>
    main()
  File "/source/nautobot/core/cli.py", line 55, in main
    run_app(
  File "/source/nautobot/core/runner/runner.py", line 266, in run_app
    management.execute_from_command_line([runner_name, command] + command_args)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 330, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 371, in execute
    output = self.handle(*args, **options)
  File "/source/nautobot/core/management/commands/post_upgrade.py", line 78, in handle
    call_command(
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 168, in call_command
    return command.execute(*args, **defaults)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 371, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 85, in wrapped
    res = handle_func(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/commands/migrate.py", line 243, in handle
    post_migrate_state = executor.migrate(
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 117, in migrate
    state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
    state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 227, in apply_migration
    state = migration.apply(state, schema_editor)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/migration.py", line 124, in apply
    operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/operations/models.py", line 92, in database_forwards
    schema_editor.create_model(model)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/base/schema.py", line 324, in create_model
    self.execute(sql, params or None)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/base/schema.py", line 142, in execute
    cursor.execute(sql, params)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 98, in execute
    return super().execute(sql, params)
  File "/usr/local/lib/python3.9/site-packages/cacheops/transaction.py", line 93, in execute
    result = self._no_monkey.execute(self, sql, params)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 66, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 75, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/usr/local/lib/python3.9/site-packages/django/db/utils.py", line 90, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 82, in _execute
    return self.cursor.execute(sql)
django.db.utils.ProgrammingError: relation "health_check_db_testmodel" already exists

This appears likely to be a result of #819.

I have the same issue.


Update Astropad to 1.3.3

Thank you for the contribution. It was merged directly as 7439388cbed75dfa64f2e7647c77716dc988adfa to keep commit history cleaner. Your contribution is still credited to you.


Created gradle structure and build file

Also disabled the failing test case.

Still using JUnit 3 style (no @RunWith and @Test, and still extending TestCase)
Wasn't sure if you want to move to JUnit 4. Cleaner to be a separate change anyway.
gradle javadoc builds the documentation
gradle build and gradle install create the kxml2-2.3.1-sources.jar
The naming convention is different from the old one, but more "standard"
(kxml2-2.3.1-sources.jar instead of kxml2-src-2.3.1.zip)
Not building the min version (yet?)

Figured out how to update a pull request :-)
Updated version to 2.3.2
Updated Java version to 1.2
This seems crazy old, please change as you wish, or let me know what version to use.
I went with 1.2 based on the version of the .class files in the last public release in maven (2.3.0)
javap -v -cp kxml2-2.3.0.jar org.kxml2.io.KXmlParser | grep version
The results is
  minor version: 0
  major version: 46

and that matches Java 1.2

Updated library version to 2.5.0 and JKD version to 1.3


npm WARN<EMAIL_ADDRESS>No repository field.

i have runned this command:
"npm instal express" but it gives me back this error:
npm WARN<EMAIL_ADDRESS>No repository field.

This is a standard npm warning, not an error.  Nothing to be concerned about.
https://stackoverflow.com/questions/16827858/npm-warn-package-json-no-repository-field


fix(manage-issue-due-dates): don't add assignee ping comment if already exists

:pencil: Description

Will prevent this helper from adding a new assignee ping comment (such as @assignee, this issue assigned to you is now overdue) if the comment already exists
Still adds a new comment if the assignee or status (due soon vs. overdue) changes

:tada: This PR is included in version 1.45.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Getting index.js:29 error

I updated npm, updated my linux system, magicmirror is runnign fine, but after I installed MMM-Googlephotos, I cant seem to run it, I deleted MMM-GooglePhotos folder and reinstalled, but same issue. Can you please help? I did paste credentials.json after I installed MMM-GooglePhotos
node auth_and_test.js
/home/pi/MagicMirror/modules/MMM-GooglePhotos/node_modules/open/index.js:29
...options
^^^
SyntaxError: Unexpected token ...
at createScript (vm.js:53:10)
at Object.runInThisContext (vm.js:95:10)
at Module._compile (module.js:543:28)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:488:32)
at tryModuleLoad (module.js:447:12)
at Function.Module._load (module.js:439:3)
at Module.require (module.js:498:17)
at require (internal/module.js:20:19)
at Object. (/home/pi/MagicMirror/modules/MMM-GooglePhotos/auth.js:6:13)

I think you just did git pull. npm install is also needed to update this module.

I did npm install within MMM-GooglePhotos folder, it goes through fine with one warn message but installs successfully

But still getting that error after npm install, just want to make it clear.

Hmmm. it seems node module open problem. Try npm install --save open in MMM-GooglePhotos directory then try again.

Just tried command, it ran successfully with a message of<EMAIL_ADDRESS>but when I run npm auth_and_test.js, still getting same error. I also tried the electron rebuild, but same error when I run js.

What is your node version? open/index.js is using ES6 style script. So it might probablly become an issue on old system.

How do I check node version and how do I update it if needed?

In terminal,
node -v

... is called as rest or spread operator. In this case, it is spread operator and supported from node version 6.4.0

7.10.1

Hmmm... weird. All I can say is just that your node cannot understand spread operator, and that is not the issue of module itself.

Ok I downgraded my node to 6.4.0 and I am getting a different error this time:
I am using magicmirror on raspberry pi, so not using it on windows.
node auth_and_test.js
/home/pi/test1/MMM-GooglePhotos/node_modules/open/index.js:16
const wslToWindowsPath = async path => {
^^^^
SyntaxError: Unexpected identifier
at Object.exports.runInThisContext (vm.js:76:16)
at Module._compile (module.js:528:28)
at Object.Module._extensions..js (module.js:565:10)
at Module.load (module.js:473:32)
at tryModuleLoad (module.js:432:12)
at Function.Module._load (module.js:424:3)
at Module.require (module.js:483:17)
at require (internal/module.js:20:19)
at Object. (/home/pi/test1/MMM-GooglePhotos/auth.js:6:13)
at Module._compile (module.js:556:32)

Hmmm.. I've tested to re-install on fresh RPI (RPI4, Raspbian Buster 2019-07-10, node v10.16.0, npm v6.9.0, gcc v7.3.0, MM v2.8.0) And there is no problem to install.

Have you any progress? I'll close this issue because there is no more feedback anymore. Feel free to reopen the issue when you have still the problem.


Creating Wumpus World

Contribution to Chapter 7 and 13
This is a visualisation of the Wumpus World needed for the Logic agent in Chapter 7 and the Probability Inferences in Chapter 13.
Current State
The Present version of this is extremely rudimentary and implemented in Chapter 7 on the Website

Plans
This is Just for the Logic Agent Segment in Chapter 7

Have Each Block show 2 things

The Measurement: Stinch or Breeze (telling us if there is Wumpus or a Pit nearby)
The Inference: What can be in the block - Wumpus, Pit, Both, or is it safe?

Remove the Explicit Dots

The Player should not be able to see where exactly Wumpus / Pits / Gold are.

Have a table of Inferences below

For all 16 squares, the current state and possible inferences that can be drawn from it.
These possibilities keep getting narrower as more information about measurements comes in

I was thinking of the following:

rim color indicates the measurement
text inside the square indicates inferences. i.e. Wumpus (W), Pit (P), Both (WP), or Nothing ().

Also, as you said, when I implement the grid I should have to expose some methods for you to use. Can you elaborate what would those methods be?
Afaict, there are two buttons in the UCB dialog you've shown above, and also some information top-right. What do those do and are those the only API methods you would need, or are more required?

No no, We need very different API. Each Grid must be an object. And That grid object must have as one of its instance variables an array of small block objects. Each of the small block objects must provide setMeasurement(measurement) and setInference(inference). Also, we must have a marker showing where we are, like the green dot in the current game and a way to move it, but that maybe later. Finally we need a Click Event Listener, onclick(function callback(x : int, y : int))
We will need to scale this as we go along. On the probability chapter, we will need to change our inferences with are W or no W, to 0.86 W form, color the background to give a feel of what the gradient is like, and maybe even make our grid bigger. So, 4 x 4 may not suffice. Let's try to be as flexible as can be, cause this model of the world spans many chapters and gives us great power, even if we don't implement those chapters.

This is what my SVG outline looks like. See if you can use the SVG library to improve it. The DOM thing works, and my implementation is similar, but SVG-TypeScript combo seem to give more flexibility.

Having Labels inside is the goal now. Then we can start thinking of a propositional logic agent.
For Documentation on the SVG Manipulation, check: SVGjs

I need to know what exactly were the flexibility issues in the previous files to be sure I won't reintroduce the same issue again.

I didn't use the class syntax because of lack of support of private variables.
I don't understand why we need SVG instead of DOM.
I also don't get the need to start from scratch.

And that's probably because I haven't played the game yet. Please give me a link to whichever implementation on the internet is closest to your thoughts and I'll play it first.

I have implemented the full version, please take a look at the demo here. Closing out this issue.


Change Filter.text to search using icontains instead of iexact by default in the basic search mode

Codecov Report

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

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   98.15%   98.15%
=======================================
  Files          60       60
  Lines        7516     7516
=======================================
  Hits         7377     7377
  Misses        139      139

Impacted Files
Coverage Δ

iommi/query.py
98.89% <ø> (ø)

Continue to review full report at Codecov.

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


feat: align Vote type label in modal to center

Closes https://github.com/snapshot-labs/sx-ui/issues/539
Screenshots
Before

After

Test plan

Open votes modal on http://localhost:8080/#/gor:0x928d54f69d15b7dfdac297ca2e1785325e29d653/proposal/1

I think we should not change the align here. Does this come from SX testing review?

@bonustrack From John's review on previous PR, some results look off currently, like:


Uncaught TypeError: Cannot read property 'cleanup' of null

Chrome 46 using demo viewer
Open sidebar without a document loaded will throw following exception:
You can "unload" the file in demo viewer by open a non-pdf file.
Uncaught TypeError: Cannot read property 'cleanup' of null
pdfViewCleanup @ viewer.js:6863
setTimeout (async)PDFRenderingQueue_renderHighestPriority @ viewer.js:3407
pdfViewForceRendering @ viewer.js:6869
(anonymous function) @ viewer.js:7202

This might be relevant here: https://github.com/mozilla/pdf.js/pull/5633#issuecomment-69432198.

Closing, since this should fixed by PR #6546.


Rename cartpole with cart

Related with this PR in gazebo_ros2_control https://github.com/ros-controls/gazebo_ros2_control/pull/252

Btw the "Check Docs" CI job fails because of another issue with control.ros.org

@christophfroehlich  not really sure about how to solve this issue
 /home/runner/work/gz_ros2_control/gz_ros2_control/doc/acknowledgements/acknowledgements.rst:70: CRITICAL: Problems with "raw" directive path:
InputError: [Errno 2] No such file or directory: 'doc/acknowledgements/reviewers_stats.html'.

@christophfroehlich not really sure about how to solve this issue
 /home/runner/work/gz_ros2_control/gz_ros2_control/doc/acknowledgements/acknowledgements.rst:70: CRITICAL: Problems with "raw" directive path:
InputError: [Errno 2] No such file or directory: 'doc/acknowledgements/reviewers_stats.html'.

As I said, a bug on control.ros.org -> I fixed it in the meantime.

thank you @christophfroehlich for pushing gazebo_ros2_control and gz_ros2_control

https://github.com/Mergifyio backport humble iron


Package: Localized application description

Package: Localized application description
Motivation & Context

Localize English more

PR Checklist

[ ] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

Screenshots
None

This description is shown in the Settings app on Windows 10 and 11.

Where?

I forgot, but I remember it is probably shown when you refer app info in the Settings app.
As you can see, this appears to users and can be localized https://learn.microsoft.com/en-us/uwp/schemas/appxpackage/uapmanifestschema/element-description

If you think it is not needed, I'm fine with that and then close.

I wonder if we can remove it all together, is it a required value in the manifest?

Yes. Required field.
https://learn.microsoft.com/en-us/uwp/schemas/appxpackage/uapmanifestschema/element-uap-visualelements


在VSCODE 下有什么自动补全的方法吗？

试了几个vscode里的qml的插件，基本没有什么效果。
把 FluentUI复制到resource的qml下也是索引不到

确实没啥办法，还是QtCreator写QML，vscode写python吧。。用QtCreator可以补全QML

在 Qt for Python 中提供了 QML 的补全，但是使用各种方法都无法导入 FluentUI，不是找不到就是报错，这可能是什么原因？

在 Qt for Python 中提供了 QML 的补全，但是使用各种方法都无法导入 FluentUI，不是找不到就是报错，这可能是什么原因？

Try  add --build-dir C:\Qt\6.5.3\msvc2019_64\qml  options  for qt For Python.qmlls.options,this will fix this problem

@suterberg 试了一下，我的 Qt 版本是 6.6.0，插件出现了崩溃：
[INFO  - 14:02:53] {
  "stdout": "",
  "command": "d:\\workspace\\repo\\automate\\qt-client\\venv\\Scripts\\python.exe c:\\Users\\10069\\.vscode\\extensions\\seanwu.vscode-qt-for-python-7.6.0\\python\\scripts\\rcc.py -o d:\\workspace\\repo\\automate\\qt-client\\gl_ui\\resources\\resources_rc.py d:\\workspace\\repo\\automate\\qt-client\\gl_ui\\resources\\resources.qrc"
}
[INFO  - 14:02:53] qmlls is running with the command: 'd:\workspace\repo\automate\qt-client\venv\Scripts\python.exe c:\Users\10069\.vscode\extensions\seanwu.vscode-qt-for-python-7.6.0\python\scripts\qmlls.py --build-dir D:/Program/Qt6/6.6.0/msvc2019_64/qml'
[Warn  - 14:02:53] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:53] Connection to server got closed. Server will restart.
[Warn  - 14:02:53] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:53] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:54] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:54] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Error - 14:02:55] The QML Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

目前不知道什么原因。而且插件的 rcc 有的时候疯狂编译不停下来，通常是插件用了十几分钟后才出现这种情况。

@suterberg 试了一下，我的 Qt 版本是 6.6.0，插件出现了崩溃：
[INFO  - 14:02:53] {
  "stdout": "",
  "command": "d:\\workspace\\repo\\automate\\qt-client\\venv\\Scripts\\python.exe c:\\Users\\10069\\.vscode\\extensions\\seanwu.vscode-qt-for-python-7.6.0\\python\\scripts\\rcc.py -o d:\\workspace\\repo\\automate\\qt-client\\gl_ui\\resources\\resources_rc.py d:\\workspace\\repo\\automate\\qt-client\\gl_ui\\resources\\resources.qrc"
}
[INFO  - 14:02:53] qmlls is running with the command: 'd:\workspace\repo\automate\qt-client\venv\Scripts\python.exe c:\Users\10069\.vscode\extensions\seanwu.vscode-qt-for-python-7.6.0\python\scripts\qmlls.py --build-dir D:/Program/Qt6/6.6.0/msvc2019_64/qml'
[Warn  - 14:02:53] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:53] Connection to server got closed. Server will restart.
[Warn  - 14:02:53] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:53] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:54] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:54] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Error - 14:02:55] The QML Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

目前不知道什么原因。而且插件的 rcc 有的时候疯狂编译不停下来，通常是插件用了十几分钟后才出现这种情况。

just Qt6.6 has this problem? try Qt6.5,and don't use python script ,just use  D:\Program Files\Python310\Lib\site-packages\PySide6\qmlls.exe  you maybe hasn't set the true position

or maybe you edit all with vscode ,In my mind you can edit qml with vscode and build python with pycharm like what i did

close all qml file then run rcc

---- 回复的原邮件 ----

发件人
Alex<EMAIL_ADDRESS>

日期
2023年12月20日 14:07

收件人
<EMAIL_ADDRESS>

抄送至
***<EMAIL_ADDRESS>

主题
Re: [zhuzichu520/PySide6-FluentUI-QML] 在VSCODE 下有什么自动补全的方法吗？ (Issue #11)

 试了一下，我的 Qt 版本是 6.6.0，插件出现了崩溃：
[INFO  - 14:02:53] {
  "stdout": "",
  "command": "d:\\workspace\\repo\\automate\\qt-client\\venv\\Scripts\\python.exe c:\\Users\\10069\\.vscode\\extensions\\seanwu.vscode-qt-for-python-7.6.0\\python\\scripts\\rcc.py -o d:\\workspace\\repo\\automate\\qt-client\\gl_ui\\resources\\resources_rc.py d:\\workspace\\repo\\automate\\qt-client\\gl_ui\\resources\\resources.qrc"
}
[INFO  - 14:02:53] qmlls is running with the command: 'd:\workspace\repo\automate\qt-client\venv\Scripts\python.exe c:\Users\10069\.vscode\extensions\seanwu.vscode-qt-for-python-7.6.0\python\scripts\qmlls.py --build-dir D:/Program/Qt6/6.6.0/msvc2019_64/qml'
[Warn  - 14:02:53] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:53] Connection to server got closed. Server will restart.
[Warn  - 14:02:53] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:53] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:54] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 14:02:54] Connection to server got closed. Server will restart.
[Warn  - 14:02:54] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Error - 14:02:55] The QML Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

目前不知道什么原因。而且插件的 rcc 有的时候疯狂编译不停下来，通常是插件用了十几分钟后才出现这种情况。
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

经过各种方法，在我的 Qt 6.6.0 上还是出现错误。有时是插件运行了一段时间才崩溃，有时候是打开项目时立即崩溃。等抽空我试试 Qt 6.5，只是 PySide6-FluentUI-QML 目前需要 Qt 版本大于 6.6.0 才能运行。
[INFO  - 10:06:40] qmlls is running with the command: 'd:\workspace\repo\automate\qt-client\venv\Scripts\python.exe c:\Users\10069\.vscode\extensions\seanwu.vscode-qt-for-python-7.6.0\python\scripts\qmlls.py'
[Warn  - 10:06:40] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 10:08:36] Connection to server got closed. Server will restart.
[Warn  - 10:08:36] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 10:08:36] Connection to server got closed. Server will restart.
[Warn  - 10:08:36] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 10:08:37] Connection to server got closed. Server will restart.
[Warn  - 10:08:37] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 10:08:37] Connection to server got closed. Server will restart.
[Warn  - 10:08:37] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Error - 10:08:37] The QML Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

仍然会出现弹框，插件是最新的。

建议我们先移步到 vscode-qt-for-python/issues，开一个新的 Issue 来讨论这个问题。

经过各种方法，在我的 Qt 6.6.0 上还是出现错误。有时是插件运行了一段时间才崩溃，有时候是打开项目时立即崩溃。等抽空我试试 Qt 6.5，只是 PySide6-FluentUI-QML 目前需要 Qt 版本大于 6.6.0 才能运行。
[INFO  - 10:06:40] qmlls is running with the command: 'd:\workspace\repo\automate\qt-client\venv\Scripts\python.exe c:\Users\10069\.vscode\extensions\seanwu.vscode-qt-for-python-7.6.0\python\scripts\qmlls.py'
[Warn  - 10:06:40] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 10:08:36] Connection to server got closed. Server will restart.
[Warn  - 10:08:36] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 10:08:36] Connection to server got closed. Server will restart.
[Warn  - 10:08:36] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 10:08:37] Connection to server got closed. Server will restart.
[Warn  - 10:08:37] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Info  - 10:08:37] Connection to server got closed. Server will restart.
[Warn  - 10:08:37] registration of file udates failed, will miss file changes done outside the editor due to error -32603 Request client/registerCapability failed with message: No feature implementation for workspace.didChangeWatchedFiles found. Registration failed.
[Error - 10:08:37] The QML Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

仍然会出现弹框，插件是最新的。

pop window is normal and you can release FluentUI with QT6.5 once and replace file then you can use it buy pyside6.5

建议我们先移步到 vscode-qt-for-python/issues，开一个新的 Issue 来讨论这个问题。

set all path in your pyside6 files like D:\Program Files\Python310\Lib\site-packages\PySide6\uic.exe,
you need set
qtForPython.designer.path": "D:\\Program Files\\Python310\\Lib\\site-packages\\PySide6\\designer.exe",
"qtForPython.linguist.path": "D:\\Program Files\\Python310\\Lib\\site-packages\\PySide6\\linguist.exe",
"qtForPython.lrelease.path": "D:\\Program Files\\Python310\\Lib\\site-packages\\PySide6\\lrelease.exe",
"qtForPython.lupdate.path": "D:\\Program Files\\Python310\\Lib\\site-packages\\PySide6\\lupdate.exe",
"qtForPython.qml.path": "D:\\Program Files\\Python310\\Lib\\site-packages\\PySide6\\qml.exe",
"qtForPython.qmlformat.path": "D:\\Program Files\\Python310\\Lib\\site-packages\\PySide6\\qmlformat.exe",
"qtForPython.qmlls.path": "D:\\Program Files\\Python310\\Lib\\site-packages\\PySide6\\qmlls.exe",
"qtForPython.rcc.path": "D:\\Program Files\\Python310\\Lib\\site-packages\\PySide6\\rcc.exe",
"qtForPython.uic.path"


[FEATURE]: Access to AccLab alumni directly (long-term)

What areas do you propose needs enhancement or improvement?
Adding it now so I don't forget, but not urgent at all.
I think it would make a lot of sense to provide access to our alumni network, either as a simple directory feature with names, contact details and areas of expertise, or a more "fancy" feature based on the outcome of the service catalogue to request services. TBD
Where is the current state/behavior?
No response
How would it improve the project?
No response
Add why this is beneficial.
No response
Add any other relevant details
No response

Hi @ErikaAntoine. Just want to make sure we understand this request properly. I see two things:

making sure that alumni still have access to content. Technically, this is the case. They can log in with their old credentials (not UNDP) and see all content.  Adding new content is another question, but this will be for later.
it seems you are suggesting we have a directory of accelerator labs members, kind of like a roster or a linked in page. This IMHO seems beyond the point of the SDG Commons as it is very AccLab focused.


[Question] How to enable ldap properly?

I've tried using following command to build an image
sudo docker build --build-arg "CONFIGUREARGS=--enable-extras=m_ldap.cpp" --build-arg "VERSION=v3.4.0" --build-arg "BUILD_DEPENDENCIES=openldap" inspircd-docker

But I got this
	BUILD:		modules/m_ldap.cpp
/inspircd-src/src/modules/m_ldap.cpp:36:10: fatal error: ldap.h: No such file or directory
 #include <ldap.h>
          ^~~~~~~~
compilation terminated.
make[1]: *** [real.mk:594: modules/m_ldap.so] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [GNUmakefile:162: target] Error 2
The command '/bin/sh -c make -j`getconf _NPROCESSORS_ONLN` install' returned a non-zero code: 2

You forgot to add the build dependency:
--build-arg BUILD_DEPENDENCIES=openldap-dev

Is there any way we could add some kind of profile to automate this?

Sure, but we would need to keep track of the dependencies on alpine side in this case and (hopefully) test this script entirely, which is something, I don't really want to maintain.


Prevent overwriting of packages in blob storage

Added an overwrite parameter to IFileStorageService.SaveFileAsync. If overwrite is false, the file storage service will not overwrite existing files.
Added logic to the ApiController and PackagesController upload flows to surface a UI message.
Added unit testing to verify that overwriting is handled correctly.

@skofman1 @joelverhagen @ryuyu @shishirx34 @cristinamanum @chenriksson @jainaashish

:shipit:


Wrong SMTP headers for initial sync of new account

Expected behavior
New smartphone. Copied K9 settings from old phone. Entered server passwords and performed initial sync.
All msgs from IMAP account synched. But for exactly four emails incomplete SMTP headers are parsed. Leading to receiving date to current time of sync. Not values from IMAP.
Reproducible, after a Reset Data in K9 always same issue.
Old phone with K9 has CORRECT date/time etc. Headers. Also Roundcube Webmail shows all correct. Also GMail App shows all correct, so IMHO for sure a K9 bug
Actual behavior
always the same (4) emails have wrong headers, are set to current date/time
Steps to reproduce
Described above.
Environment
K-9 Mail version: 5.208
Android version: 6.0
Account type (IMAP, POP3, WebDAV/Exchange):
IMAP, Dovecot, own VPS

See #1356


List command

Lists metadata for all projects in manifest.
This PR implements #20.

I'm not sure why the static code analysis check has failed.

I'm not sure why the static code analysis check has failed.

@KrisEDNA  it is probably due to codacy coverage decrease or complexity increase.
See here: https://app.codacy.com/gh/dfetch-org/dfetch/pullRequests
Note: sometimes it is acceptable :relaxed:

@KrisEDNA  it is probably due to codacy coverage decrease or complexity increase.
See here: https://app.codacy.com/gh/dfetch-org/dfetch/pullRequests
Note: sometimes it is acceptable :relaxed:

@spoorcc probably if you use (and optionally extend) the suggested DLogger, you can remove the colorama import, which is causing codacy to become angry.

@spoorcc probably if you use (and optionally extend) the suggested DLogger, you can remove the colorama import, which is causing codacy to become angry.

Also a basic unit test would improve the coverage (yes, I've been sloppy before with the proof-of-concept 😉, but I'm trying to improve). You could copy test_check.py probably and reuse a lot of that code

Also a basic unit test would improve the coverage (yes, I've been sloppy before with the proof-of-concept 😉, but I'm trying to improve). You could copy test_check.py probably and reuse a lot of that code

After agreement of @KrisEDNA , I will try to get the PR merged. I have rebased the branch on the current master. I have also added feature tests (git and svn). I'm not sure whether the unit test have an added value over the feature tests.
Maybe I should also add a feature test with a fetch on invalid path.

Thanks! Could you add an changelog entry, and I'll merge it

Rebased and updated changelog.
Thanks to @KrisEDNA for the initial effort.


Create holyredeemerhs

For access to create an educational account for WebStrom

@StevennHaa lib/domains/org/holyredeemerhs: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@StevennHaa Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Lumira Designer 2 SP02

Hi Mike and Co,
Just having a look at the Kpi Tile 2.0 in Lumira Designer 2 SP02 and comparing it to 1.6 . In the default tile, it seems to be missing the last x and y values, missing colours and chart lines seem also displaced. Please see:

Any info on this is greatly appreciated,
Thanks.

Hi Neil,
Is it possible you can share me a sample app?

I figured it out....  You must run Blue Crystal theme.  In Lumira 2.0, it appears that the default newer 'Belize' theme doesn't work.

Hi Mike,
Changing the Theme to Blue Crystal worked.
Thanks heaps for looking into this one and resolving.
Cheers,
-Neil


[bug]   continued issue with react dependencies

Still have issues with the unneeded react dependencies. they are still pulling in 15.4 despite forcing the version to 15 with shrinkwrap.  have isolated this issue to the material-ui dep.  I hate to remove this wonderful library but it is currently impacting our deployments.

above are all listed deps in material-ui that force 15.4.0 download

Downgrade the react-addons-test-utils and  that should be fine.

peerDependencies: { 'react-dom': '^15.4.0' },

I don't think we can do much at the Material-UI level. I'm closing this issue.
If you find any solution, feel free to submit a PR :).

react-addons-test-utils is in the material-ui repo not my package.json

Oh right. Still, it's a devDependencies, that's not something that matters.

im at a loss then.  its installing its own react version (15.4.0) in node_modules/material-ui/node_modules.  if it delete this file then everything works.  what on earth could be causing it to do that?  even shrinkwrap isnt powerful enough to stop it lol


Update Policy CRD

Refs:

https://github.com/stolostron/backlog/issues/26187

Signed-off-by: Justin Kulikauskas<EMAIL_ADDRESS>

/cc @JakobGray
/cc @willkutler
/cc @cameronmwall


Add version: TopazLabs.TopazPhotoAI version 0.1.0

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335226 2024-01-25 14:08:40.113 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(267)\WindowsPackageManager.dll!00007FFB23F8A384: (caller: 00007FFB23F8AD01) Exception(1) tid(b64) 80D02002  2024-01-25 14:09:10.138 [CLI ] ShellExecute installer failed: 1603 CustomAction DownloadModelsAction returned actual error code 1157 (note this may not be 100% accurate if translation happened inside sandbox) MSI (s) (A4:CC) [14:09:09:730]: Product: Topaz Photo AI -- Error 1723. There is a problem with this Windows Installer package. A DLL required for this install to complete could not be run. Contact your support personnel or package vendor. Action DownloadModelsAction, entry: DownloadModels, library: C:\Windows\Installer\MSI83C8.tmp  Error 1723. There is a problem with this Windows Installer package. A DLL required for this install to complete could not be run. Contact your support personnel or package vendor. Action DownloadModelsAction, entry: DownloadModels, library: C:\Windows\Installer\MSI83C8.tmp  Property(S): ErrorDialog = ErrorDlg MSI (s) (A4:CC) [14:09:09:839]: Windows Installer installed the product. Product Name: Topaz Photo AI. Product Version: 0.1.0. Product Language: 1033. Manufacturer: Topaz Labs LLC. Installation success or error status: 1603.

(Automated response - build 655.)


fix: fix crash extracting struct with type parameter

FTL panics when compiling code like this:
//ftl:data
type DataWithType[T any] struct {
	Value T
}

VisitStruct was iterating through type parameters and type parameter values, but there are no values in this case.

Actually, putting this on hold, I think https://github.com/TBD54566975/ftl/pull/1522 might have unblocked what I needed.

It wasn't blocking me in the end, but submitting again because I do think its a legit issue.


java.lang.NoSuchFieldError: Companion. When my project reference okhttp3.

My project refeenced okhttp3, version 4.7.2. Keep reporting errors when using clickhouse-jdbc, "java.lang.NoSuchFieldError: Companion" 。During the inspection, the io.rpc.jar package of clickhouse was found and the okio code was packaged, which led to version compatibility problems.
Solution method，exclude grpc and io.grpc package.
    <dependency>
            <groupId>com.clickhouse</groupId>
            <artifactId>clickhouse-jdbc</artifactId>
            <version>0.4.0</version>
            <classifier>http</classifier>
            <exclusions>
                <exclusion>
                    <groupId>com.clickhouse</groupId>
                    <artifactId>io.grpc</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>com.clickhouse</groupId>
                    <artifactId>clickhouse-grpc-client</artifactId>
                </exclusion>
            </exclusions>
        </dependency>

Sorry for the inconvenience @xywendz. I'll make all dependencies in clickhouse-jdbc optional, so that nobody has to do this ever again.


Parsing Exception on newly downloaded schema

Steps to Reproduce:

Download  Schema
Focus on .graphql file with query, as it attempts to parse popups show parse exception

Parsing Errors prevent plugin from functioning normally
Expected:
Can successfully parse and start introspecting and highlighting queries
Latest Version of Plugin, using in Android App on .graphql files
Have been able to manually edit and find a lot of value in the plugin, but in order to increase ease of adoption I would like to find a solution for downloading the schema and parsing smoothly.
Link to Repo with Reproduction or Steps to Reproduce:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Version and Environment Details
Operation system: [version]
IDE name and version: [version]
Plugin version: [version]
Additional context
Add any other context about the problem here.

Hi @SriwanKm ! Do you still experience this issue? Looks like you had this error due to a default values mismatch. You could skip them  in introspection by choosing this option:

Actually, I think that we could skip validating default values (at least do not fail schema building completely on conflicting definitions) and remove that option because IMHO it's confusing for the user (as for me, I also always forget why do we have it here 😄). I'm not sure if a default value is a part of the parameter signature, so I'm waiting for the response from the GraphQL team, but I don't think I'll get it ever 😞.

Closing for no response. Also, I suppose it should be fixed in the most recent release. If it's not the case, please, feel free to reopen.


Black/grey color but no model shown

Hello,
i installed the card via hacs.
i did everything like you showed on youtube but for me the card only shows a grey color sometimes black.

i made the model in sweet home 3d and exported it with the plugin.

i tred it on several browsers and the mobile app
i reduced the yaml code to this but still the same:
type: custom:floor3d-card
path: /local/floorplan/
objfile: home.obj
mtlfile: home.mtl`

Hello based on the info you provided, the thing that worries me most is the size of the model. I've seen up to 200 Mb but I'm scared that 500 Mb is too hard to handle for the WebGL library behind the card. If you do not mind to send me your sh3d model file I will try to see if I stumble on the same issue

i did some test, i removed some things and my model went down to 204mb, but the card now stops at 97% and it than takes like 2 minutes than the model loads. also at 84mb it stops at 97%
so it seems really to be the size. is there a way to shrink objects (.obj, .3ds, .dae) down? since the model is already at more than 60mb with only some walls

The size of the model can be compressed using the glb format (it requires some tweaking, I will do a tutorial about that) on the other hand the glb will make the card faster to load but if it is too heavy, the card will be hard to render and animate. Some furniture are so detailed that have thousands of vertices; these objects make the rendering very intense for cpu and gpu. You need to strike a balance between esthetic and performance. A model of 100mb can be handled reasonably well by average cpus, but if you have detailed furniture even a simple light (and the shadows) may require thousand of computations to be rendered.

ok i got my model down to 105mb.
but the card still gets "stuck" on 96%, now even google chrom hangs and i need to click "wait" and after 2 minutes the model loads

Please send a message to my mail, it is public and attach a link to a file drop with your model (sh3d), I've never heard about this behaviour. Also what is your Hardware both pc and mobile ?

Hi,
i have sent you an Mail.
It seems i need to redo some furniture since they are exported or loaded in the card incorrectly (wrong color or even not all elements are loaded)
my hardware is:

A i7 Nas system where i run HA in docker, shouldnt be the problem there
a 5 year old i7 HP notebook, 16gb ram, ssd etc, its old but should also be ok
my smartphone is a Samsung Galaxy S20

KrX, can you please state the status of this issue ?

ah hi sorry forgot about that.
like i said above it helped a lot reducing the furniture models down in size and after shrinking the floor model to below 100mb helped a lot, but it was still taking about 40-60 seconds to load.
with your help the model was converted to the glb format which now loads in 2-3 seconds on my notebook and maybe 8 seconds on my mobile phone.
so as you said via mail you will make a tutorial about how to convert the obj model to glb in the future, so im happy right now ;)
thanks again for all your help and this nice project


Fix for beefy

sno-516

Have you been able to test the non-handover case? ... but there are also a lot of Discarded invalid commitment messages and no non-handover messages (New commitment added to channel)...

@doubledup Thanks for pointing it out! Also notice that and fix in https://github.com/Snowfork/snowbridge/pull/863/commits/1c67cb4f37e1e2fcf50db014429af78a1e44d203


matrix-synapse: <IP_ADDRESS> -> 1.0.0

Motivation for this change
https://matrix.org/blog/2019/06/11/synapse-1-0-0-released
https://matrix.org/blog/2019/06/11/introducing-matrix-1-0-and-the-matrix-org-foundation
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This might come with need of changes to the NixOS module, there is at least a new federation_verify_certificates parameter. Following the spirit of https://github.com/NixOS/rfcs/pull/42 we might rather deprecate all the options, but discussion over there isn't over yet, so maybe not deprecate now if the final decision is in the other direction in the end.
What do you think about this question?

Oh and:
@GrahamcOfBorg build nixosTests.matrix-synapse

What do you think about this question?

To be honest, I'm not really fond of https://github.com/NixOS/rfcs/pull/42.
But you're right about the new options. I'll check for new options and add those.

You may want to self-nominate on NixOS/rfcs#42, this would help you discuss it properly :)

I don't believe it's necessary to add the new options; I think there are already many options we don't expose, and one more shouldn't block updating.

@Ralith Exposing that many options, suggesting we are exposing them all, yet not actually exposing them all is in itself an issue.
Totally agree that doesn't necessarily need to be addressed in this PR, just wanted to start the discussion someplace :)

Wouldn't it be easier to expose a minimal set of options and merge the rest using extraConfig that is of type types.attrs;?
I did something similar in the mxisd module: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/networking/mxisd.nix
It works quite well. Whenever mxisd got an update I only changed my extraConfig instead of changing the module and my config.

I'm on board with reducing the complexity of the module, but that's a long-term matter.

@mguentner Sounds like something compatible with the current state of RFC42, which you may want to comment over there so that it gets an additional data point :)


Add global service config

Add global auto-used fixtures to overwrite service config and don't use the local one.

@jpopelka PTAL


[Android] No notification after device restarts OR after leaving device idle for a very long time

RN version:- 0.50
REACT-NATIVE-FCM Version: 11.0.1
Android device:- 7.0
When I start the application, I receive Push Notification in the app if it's running, or in the android notification bar if the app is running in background.
But if I restart the phone or leave it idle for hours, I cannot receive any notification anymore. I have to open the app for at least once to start receiving notifications again.
I have also tried to send notifications using Firebase console just to test if this is a case of silent payload issue and I receive the notifications just fine.
My payload looks like this:-
{
 "to":"FCM_TOKEN",
 "priority" : "high",
 "content_available" : true,
 "data": {
   "custom_notification": {
     "body": "test body",
     "title": "test title",
     "color":"#00ACD4",
     "priority":"high",
     "icon":"ic_notif",
     "group": "GROUP",
     "id": "id",
     "show_in_foreground": true
   },
   content: {
   }
 }
}

@Kabangi How did you solve your second issue?
I'm currently facing the same problem for a critical App. A broadcast I'm having can't start a service and all I'm getting is Auto-start Limit.


answerShippingQuery calls Error

I'm trying to make an answerShippingQuery with code example below, but something went wrong.
I've the following error:

Unhandled rejection Error: ETELEGRAM: 400 Bad Request: shipping options are not specified

bot.on("shipping_query", shipping => {
  const ok = true;
  const opts = {
    shipping_options: [
      {
        id: "shipping",
        title: "Shipping by seller",
        prices: [
          { label: "Shipping by seller 1", amount: 500 },
          { label: "Shipping by seller 2", amount: 500 }
        ]
      },
      {
        id: "takeaway",
        title: "Take away",
        prices: [{ label: "-Take away", amount: 0 }]
      }
    ],
    error_message: ""
  };

  bot.answerShippingQuery(shipping.id, ok, opts);
});

I have the same problem

Fixed in v0.30.0.


[Feature request] Add option to disable script update notice

I opened YouTube today and got an annoying banner telling me to update the dev version to 501, even though it's supposed to update itself. I am using Greasemonkey and the addon has always updated from time to time by itself.

As a preemptive comment, just in case this suggestion is rejected, the following entry in AdBlock will cut it out anyways for anyone else interested:
youtube.com##div[class="alerts-wrapper"]

@Kanegasi Be aware that YTC makes use of YouTube's warning box, so by blocking it you will also not receive any warning given by YouTube.
Also It would be best if you'd change the title to something like [Feature request] Add option to disable script update notice

@Yonezpt That's nice to know, even though I haven't recalled a YouTube site notification I was interested in. What specifically annoys me about that update box is that it is nested in the player's div, pushing the player down. I use the fill option so that the player fills my browser window automatically. It's a workaround to get a fullscreen html5 player by using f11. When that update notification popped up, the entire player was pushed down past the bottom of the displayed part of the browser.
I also changed the title as you suggested.

Does disabling Enable update checker setting under Update not disable the update banner?

It does for me.

Also you can set the update interval to higher times so it doesn't bother you every video. Once you close it it wont appear again after that amount of time have passed if you are interested in knowing when there is an update but don't want to be bothered.

@Kanegasi Could you send me your source script please ? There is problem with newer version of the developer build and I want to rollback to the previous one but cannot find it.

Sorry for the length of time since I last replied.
@Zarggg @Inyokus @rayman89 I honestly thought that option was what did the updating. I guess updating is entirely through Greasemonkey itself. I unchecked the box. I assume I won't get any update banners now. If this option is exactly the feature I am requesting, I guess this ticket can be closed.
@mirogg Sorry to be blunt, but do you not understand what Github's purpose is? Every version of a piece of code is archived and all changes tracked. That's the point of a repository. I'll give you step-by-step instructions to access any version of YouTube Center as well as the direct raw link to the last version, 500.

On the main page to the YTC repository (click the bold YouTubeCenter link at the top of the page), click "dist" in the list of folders.

Click "YouTubeCenter.user.js".

Click "History" on the right, next to "Raw".

Click the carrots "< >" next to "2c47938".

Click "View Raw".

[Here's that "View Raw" link] (https://raw.githubusercontent.com/YePpHa/YouTubeCenter/2c479384813bbe0a3189b0bbc6dcf6e36cdac636/dist/YouTubeCenter.user.js)
Since the version numbers are internal to the script, they won't show anywhere in Github's UI. In that list where you clicked the carrots, the top entry is the current version, so just count down the rows.

Noticed that the script updated to 503 with the check update option turned off. Confirmed to be the feature I'm requesting since I didn't see any update notification. Closing the issue.


bot crashes when Slack thread pin limit is exceeded

There seems to be a Slack limit set which prevents pinning more than 100 threads in a channel (not sure if this is server specific). The bot is pinning every Jira ticket it is set to watch and when it hits this limit it will crash because the exception is not handled now:
Traceback (most recent call last):
  File "/usr/local/bin/triagebot.py", line 523, in wrapper
    return f(config, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/triagebot.py", line 808, in _check_jira
    [issue.post](http://issue.post/)()
  File "/usr/local/bin/triagebot.py", line 369, in post
    self._client.pins_add(channel=self.channel, timestamp=self.ts)
  File "/usr/local/lib/python3.11/site-packages/slack_sdk/web/client.py", line 4026, in pins_add
    return self.api_call("pins.add", params=kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/slack_sdk/web/base_client.py", line 156, in api_call
    return self._sync_send(api_url=api_url, req_args=req_args)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/slack_sdk/web/base_client.py", line 187, in _sync_send
    return self._urllib_api_call(
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/slack_sdk/web/base_client.py", line 317, in _urllib_api_call
    ).validate()
      ^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/slack_sdk/web/slack_response.py", line 199, in validate
    raise e.SlackApiError(message=msg, response=self)
slack_sdk.errors.SlackApiError: The request to the Slack API failed. (url: [https://www.slack.com/api/pins.add](https://www.slack.com/api/pins.add))
The server responded with: {'ok': False, 'error': 'too_many_pins'}

The bot could log a warning instead and skip pinning until the count is reduced in a channel.
A second part of the issue is recovering from a large amount of pinned messages. Currently any issue tracked by the bot can be untracked with "Resolve" button for each issue/thread posted in the channel. Similarly it can be unpinned one by one manually. It would be helpful to have a new command, similar to refresh-all that would allow mass resolving (and unpinning) issues after they've been assigned (resolve-all?).

We've had this one recently and I wonder if we should "just" not pin issues at all. They are always quickly available in the chat anyway so we don't strictly need it.

I guess we could live without it, the only downside is that for each issue/thread we would have to "click through" into history and hit "Resolve" there - which is slightly less convenient. Also if there's a huge amount of bug reports (with CVEs usually) the need for mass resolve still stands, although this is more of an RFE and not related to this issue.

I've merged https://github.com/coreos/triagebot/pull/16. This does not "fixes" this but works around it.
We'll happily take a PR with the "full" fix 🙂.