[Build] default option for extern libs

Set default option off for external libraries (AITT and Paho-MQTT).
Signed-off-by: Jaeyun<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #98. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

IMO default build option of aitt and mqtt should be off.


Strange issue with react-slingshot

Hi guys,
Thank you for this project, it is really helpful ! But for a reason I can't figure out, I can't make the following jsfiddle working with react-slingshot:
https://jsfiddle.net/q3jroknc/
here is the repository that show the issue:
https://github.com/namlook/react-slingshot-semantic-ui-bug
The files to look at are:

./src/index.html (I added the CDN resources. Gross, I know, but I want to be as close as possible to the fiddle)
./src/index.js (where I added the component)

npm start and go to http://localhost:3000 to see the issue.
Any ideas of what is going wrong here ?
Thanks !

You need to add the 'type' attribute to your Semantic css and js tags
<link rel="stylesheet" type="text/css" href="https://rawgit.com/Semantic-Org/Semantic-UI/next/dist/semantic.css" />
<script type="text/javascript" src="https://rawgit.com/Semantic-Org/Semantic-UI/next/dist/semantic.js"></script>

Damn ! How could I missed this ? It works perfectly now. Sorry for the noise 😳


Several of the mods simply don't work on Mac .dmg version

Some do-- mostly themes but simpler databases, truncated table titles, collapsible headers, and probably a handful more don't work. I'm on Big Sur 11.5.1

hm, yeah. I just gave simple databases a test and it didn't work for me either. @CloudHill is there a quick fix or not really?
they haven't been updated in a little while so if notion has changed enough they're probably just broken.

Collapsible headers aren't working for me either, and I'm on Windows

So do I，but I'm on Windows. In addtion, outliner doesn't work as expected that it is blank most of the time. And word count doesnt work either.

Same for me regarding the collapsible headers. I'm also on Windows 10, using a fresh install of Notion Enhanced 2.0.16 rev 5.


KAFKA-3658: Validate retention period be longer than window size

ping @HenryCaiHaiying @enothereska @ijuma for reviews.

Looks good to me.

The PR does what the JIRA says, but I'm not particularly familiar with the Streams windowing code. So probably good to get a +1 from @enothereska as well.

Merged to trunk and 0.10.0.


Set-DbaMaxDop fails due to missing instance name

Type of Change

[X] Bug fix (non-breaking change, fixes #2657)

Purpose
Fix this command to run on servers with named instances, currently it is using the wrong names as they are given from Test-DBAMaxDOP
Approach
Using names assigned to Test-DBAMaxDOP

plz check my changes @wsmelton

Unfortunately, my changes blew. We will keep this one for the mass rewrite which will be coming soon!


How to use onTap function in GestureDetector in this version json_dynamic_widget: ^5.1.7+2

json example :
  final String json = '''{
  "type":"padding",
  "args":{
  "padding":{
  "right":"50",
  "left":"50",
  "bottom":"100"
  }
  },
  "child":{
  "type": "row",
  "args":{
  "mainAxisAlignment":"center"
  },
  "children": [
    {
      "type": "gesture_detector",
   "onTap": "showMessage",
      "child": {
        "type": "text",
         "args": {
            "text": "Flutter"
          }
      }

    }
  ]
}
  }'''

I don't support old versions on the package. That said, there is an example of using a gesture detector in the example folder. I hope that can answer your question. If not, this is a better post to stack overflow as it is not an issue.
I wish you luck.


[FEATURE] Implement T5 Model; Improve flexibility in SentencepieceTokenizer instantiation

Description
This PR contributes the T5 model to GluonNLP codebase, with a parameter conversion feature that allows for loading weights from a Huggingface T5 model. Besides, this PR enables instantiating a sentencepiece tokenizer with user-specified tokens (other than control tokens) using **kwargs, and hence user could directly control the number of <extra_id>s in T5 Tokenizer.
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] All changes have test coverage

Changes

[x] T5 Model (forward tested)
[x] Parameter conversion from Huggingface T5Model
[x] Improved sentencepiece tokenizer instantiation
[x] Bug and typo fixes

cc @dmlc/gluon-nlp-team

@ZheyuYe Would you help review it also if you have time? Also ping @szhengac to help review if you have time.

@ZheyuYe Would you help review it also if you have time? Also ping @szhengac to help review if you have time.

I told @yongyi-wu to implement incremental_decode in a follow-up PR and we will focus on the correctness of the forward pass first in this PR.

I told @yongyi-wu to implement incremental_decode in a follow-up PR and we will focus on the correctness of the forward pass first in this PR.

I am working on convert_t5.py and uploading model weights to S3, as @sxjscience noted previously. I will update here when I'm done.

I am working on convert_t5.py and uploading model weights to S3, as @sxjscience noted previously. I will update here when I'm done.

Hi all, thank you for your patience. I've updated the code based on your feedback. At this point, google_t5_small is up on S3 already, whereas other files should be uploaded shortly. By the way, there are some non-trivial changes in t5.py as compared to the previous commits, so it would help a lot if you could review it again :D

Hi all, thank you for your patience. I've updated the code based on your feedback. At this point, google_t5_small is up on S3 already, whereas other files should be uploaded shortly. By the way, there are some non-trivial changes in t5.py as compared to the previous commits, so it would help a lot if you could review it again :D

Some other follow-up items:

Convert the T5ConditionalGenerator so that we can directly use it to fill in missing values in the input sentence
Add incremental_decode to support BeamSearch and fast inference.

Some other follow-up items:

Convert the T5ConditionalGenerator so that we can directly use it to fill in missing values in the input sentence
Add incremental_decode to support BeamSearch and fast inference.

Looks good to me. A very nice addition! Let's try to convert 3B + 11B when we are fixing the CI. Basically, T5-3B + T5-11B are quite large and they can help test the large-tensor support in MXNet. If we met any issues, they might be related to bugs in MXNet.

T5-3B and T5-11B are good to go. Previous failure was probably due to OOM, while MXNet itself worked well.

Looks good to me. A very nice addition! Let's try to convert 3B + 11B when we are fixing the CI. Basically, T5-3B + T5-11B are quite large and they can help test the large-tensor support in MXNet. If we met any issues, they might be related to bugs in MXNet.

T5-3B and T5-11B are good to go. Previous failure was probably due to OOM, while MXNet itself worked well.

Let's also add the test here: https://github.com/dmlc/gluon-nlp/blob/4b061305285f3c4e491b77b046a92c32e306cbc6/tests/test_models.py#L26-L39 We may skip the t5-large, t5-3B, t5-11B and only test for small and base.

Let's also add the test here: https://github.com/dmlc/gluon-nlp/blob/4b061305285f3c4e491b77b046a92c32e306cbc6/tests/test_models.py#L26-L39 We may skip the t5-large, t5-3B, t5-11B and only test for small and base.

@yongyi-wu After a second thought, I think we can skip t5 in test_models.py and just test it in the test_models_t5.py. We may try to load the t5-small and t5-base and do a basic forward. This helps us check if we can correctly load the model.

@yongyi-wu After a second thought, I think we can skip t5 in test_models.py and just test it in the test_models_t5.py. We may try to load the t5-small and t5-base and do a basic forward. This helps us check if we can correctly load the model.

@yongyi-wu After a second thought, I think we can skip t5 in test_models.py and just test it in the test_models_t5.py. We may try to load the t5-small and t5-base and do a basic forward. This helps us check if we can correctly load the model.

After discussing with @sxjscience, we decided to still test T5 backbones in test_models.py, but skip T5-3B and T5-11B.

@yongyi-wu After a second thought, I think we can skip t5 in test_models.py and just test it in the test_models_t5.py. We may try to load the t5-small and t5-base and do a basic forward. This helps us check if we can correctly load the model.

After discussing with @sxjscience, we decided to still test T5 backbones in test_models.py, but skip T5-3B and T5-11B.


Add as_url method

It would be great to be able to generate URLs that can be used.
S3Path("s3://bucket/file.txt").as_url()
# > https://bucket.amazons3.com/file.txt

S3Path("s3://bucket/file.txt").as_url(presign_expiry_min=100)
# > https://bucket.amazons3.com/file.txt?presign=accesstokenwithexpiry

More complete discussion in #235. Consolidating into that issue.


Update introduction.md

We probably also want to update all the other versions, especially 1.0, 1.1, and next.


Display Custom Error Message for the APB via termination-log

Describe what this PR does and why we need it:
If the APB provision fails, this PR will check for a non-empty termination message, and will display it in the WebUI, instead of the generic error message.
Which issue this PR fixes (This will close that issue when PR gets merged)
fixes #675

 (8) AWS Broker RFE: Enhance error messages, so when a provision request fails with a CloudFormation error, the error is preserved and displayed to end user in web console.

Could you provide an APB to be used to see this in action?

@djzager https://github.com/johnkim76/lastop-cem-demo-apb
docker.io/jkig/lastop-cem-demo-apb


How to use jest-sonar-reporter

How to use jest-sonar-reporter.
In confg.json we were using     "testResultsProcessor": "/config/jest/resultsProcessor",
resultsProcessor.js
module.exports = (results) => {
process.env.TEST_REPORT_PATH = root('reports/unit')
process.env.JEST_SUITE_NAME = 'React Unit Tests'
process.env.JEST_JUNIT_OUTPUT = root('reports/unit/junit.xml')
require(root('node_modules/jest-sonar-reporter/index')).apply(this, [results])
require(root('node_modules/jest-junit/index')).apply(this, [results])
// add any other processor you need
return results
}
this was working fine when we were using "jest": "^21.2.1",
but after updating jest to "jest": "^26.1.0",
as per documentation...
"reporters": [ "default",[
"jest-junit", {
"outputDirectory":"reports/unit",
"suiteName": "ANZ-React-BoilerPlate Unit Tests",
"outputName": "junit.xml"
}]],
jest-junit is working fine but how to support jest-sonar-reporter.
please help.

Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.


Fix #20271: Bring for comprehension spec up to date.

Since Scala 3.4, a withFilter is generated if and only if the generator has the case modifier.
If it does not, the pattern must be irrefutable.

Rendered version of the changed section:

@sjrd Read "produced" instead of "produces" in sentence "[...] tests whether elements produces by ´e´ match [...]".


feat: add dropdown to modus-navbar

Description
This PR adds a component that is in the design and docs for the modus-navbar but was never implemented. It must be dynamic enough to support any text and value (string is the simplest approach with good performance).
The new modus-navbar-dropdown component only renders when a dropdownOptions object is passed to the navbar. It then uses the items provided in those options to render a modus-dropdown. A default value can be provided to set the initially selected item. On item selection an event is emitted from the navbar dropdownItemSelect that passes the item to the consumer.
Apparently the linter and prettier haven't been run in a while. I ran them locally and thus my PR includes many code styling changes.
The key files to look at:

modus-navbar.tsx
modus-navbar.scss
modus-navbar.models.ts
modus-navbar-dropdown.tsx

I am not fully sure how we want to document this feature but we definitely should. I'll look to you for guidance.
References #
Design

Screenshots
Validating rendering with no default value and empty item

Validating default item selection and event output

Rendered RTL (right-to-left aka reverse)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How Has This Been Tested?
Manually
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Abandoning due to the many formatting changes from running the linter and prettier. Creating a new PR.


Allow for registered content providers to determine the key used to extract values from context

sablon allows for custom content providers to be registered via
class FormsUrn < Sablon::Content::String
  def self.id
    :urn
  end
end

Sablon::Content.register(FormsUrn)

To use a mailMege tag for this Content type, you specify  «=urn:example», but in the context, you're only supposed to pass in a non-type-qualified name e.g.
{
  "example" => "my value"
}

This PR allows you to change how the key transformation works for a registered Sablon::Content type, by specifying the #key(regex) method
e.g.
  class FormsUrn < Sablon::Content::String
    def key(keyRegex)
      keyRegex.to_s # use the original key, do not only match to the second part of the regex
    end

    def self.id
      :urn
    end
  end

would then allow you to pass through the fully-qualified name
{
  "urn:example" => "my value"
}

Hi @Mongey, thank you for your contribution. I have only skimmed the patch quickly but it looks good. Please give me a few days to get to reviewing it properly.

@Mongey could you provide some background on why the default naming within context does not work for you?

Thanks for taking a look @senny
We have a bunch of documents that are already tagged using a mailmerge format of «= urn:example:input » , «= urn:example:foobar » etc.
Without this change we would need to pass in the following context.
{
  "example:input" => "hello",
  "example:foobar" => "world"
}

However, we want to keep the mapping 1-1, for these types of tags, to avoid remapping the context keys to remove the urn: prefix
{
  "urn:example:input" => "hello",
  "urn:example:foobar" => "world"
}

It's probably a bit of a strange use case 😅

@Mongey thanks for the additional context. If I understand correctly, you are not really looking to add a custom content provider but mainly want to work around the fact that a <type>: prefix is something that Sablon treats differently in the context hash?
I understand your use-case and I do believe we can get this working (maybe by adding a runtime patch on your end). I'm not sure we want to extend the API to support this case though. It seems very specific to your current constraints.

@Mongey thanks for the additional context. If I understand correctly, you are not really looking to add a custom content provider but mainly want to work around the fact that a : prefix is something that Sablon treats differently in the context hash?

Yeah exactly.

I'm not sure we want to extend the API to support this case though. It seems very specific to your current constraints

no problem, I guess I can monkey patch or just use the fork

@Mongey I'll take another look at the issue today. If anything, I think we can rearrange the internals a bit to make your patch a lot easier.

@Mongey I pushed a proposed change in 46bfeeb9f220f14e560dabf30b3e8ccfa08398f0 that allows you to configure Sablon like so:
Sablon::Context.content_regex = /\A((?!urn:)[^:]+):(.+)\z/

You also no longer need the custom FormsUrn type as keys not matching content_regex will just be treated as normal strings.
Let me know if that works for your use-case. If so, I can cut a new release.

@senny That works perfectly, thank you

@Mongey awesome. Thanks for the confirmation. I just released 0.4.1 that contains the patch.


feat: support DML with Returning clause in Connection API

This PR adds support for running DML statements with Returning clause in the Connection API, and incorporates the following:

API documentation changes to document the behaviour of execute, executeAsync, executeQuery, executeQueryAsync, executeUpdate, executeUpdateAsync, executeBatchUpdate, and executeBatchUpdateAsync APIs when executing DML statements with Returning clause.
Parses the SQL string to check if the DML statement has a Returning clause. Parsing logic has been added for both GoogleStandardSql and PostgreSql dialects.
Changes to MockSpannerServiceImpl to put results for DML statements with Returning clause, and to add support in executeBatchDml.
Integration tests for running DML statements with Returning clause using each of the above APIs, for both GoogleStandardSql and PostgreSql dialects.
Unit tests for abort scenarios while running DML statements with Returning clause.
Parser tests for both GoogleStandardSql and PostgreSql dialects.

Note: Integration tests are not expected to pass until DML with Returning clause support is launched.

@olavloite: Thank you for a very detailed review of this PR. :)

@olavloite: Can you take a quick look at the changes in https://github.com/googleapis/java-spanner/pull/1978/commits/e2a463e4f91c602cd4a237634f557bcf4793de21? Thanks!

@olavloite: Can you take a quick look at the changes in e2a463e? Thanks!

Can we also add a test for insert into t1 select 10.returning* (this is equivalent with insert into t1 select 10.0 returning *)

@olavloite: Can you take a quick look at the changes in e2a463e? Thanks!

Can we also add a test for insert into t1 select 10.returning* (this is equivalent with insert into t1 select 10.0 returning *)

Added in https://github.com/googleapis/java-spanner/pull/1978/commits/60fcb4cdfb47fdc64924bff914e35d5cd454aa3d.

60fcb4c

Thanks. LGTM (once CI errors have been cleared)


Enable fgac support for machine learning plugin.

Signed-off-by: Alex<EMAIL_ADDRESS>opensearch-security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
New feature

Github Issue # or road-map entry, if available:
https://github.com/opensearch-project/machine-learning/issues/55

Description of changes:
Enable the security support to machine learning plugin.

The models will be saved in system index.

Permission control (transport actions) by pre-defined security roles.

Backend roles with users.

Why these changes are required?
The new machine learning plugin needs to support security control.

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)
Machine learning plugin is a new plugin to support machine learning for open search.

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)
Unit test and manual testing done.

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)
N/A.

Is it backport from main branch? (If yes, please add backport PR # and commits #)
N/A.

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1308 (f457edf) into main (2b69d9c) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1308      +/-   ##
============================================
- Coverage     64.73%   64.68%   -0.05%
- Complexity     3193     3194       +1
============================================
  Files           247      247
  Lines         17230    17230
  Branches       3045     3045
============================================
- Hits          11153    11145       -8
- Misses         4527     4536       +9
+ Partials       1550     1549       -1

Impacted Files
Coverage Δ

...org/opensearch/security/rest/TenantInfoAction.java
77.94% <0.00%> (-10.30%)
:arrow_down:

...security/configuration/DlsFlsFilterLeafReader.java
59.76% <0.00%> (-0.71%)
:arrow_down:

...a/org/opensearch/security/tools/SecurityAdmin.java
47.52% <0.00%> (+0.26%)
:arrow_up:

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 2b69d9c...f457edf. Read the comment docs.

@spbjss Please sign commits

@spbjss it looks like this is almost ready to go, after resolving merge conflicts.

We opened a new PR https://github.com/opensearch-project/security/pull/1654, close this one.


scRGB HDR seems generally broken on the OLED Steam Deck

When using a R16G16B16A16_FLOAT swapchain buffer for HDR, in Windows, the compositor expects scRGB color space (DXGI_COLOR_SPACE_RGB_FULL_G10_NONE_P709).
I.e. (1,1,1) should be about 80 nits, and 1000 nits is something like (12,12,12).
When presenting the same frame on the OLED Steam Deck, it is completely blown out (and the gamma is also way off).
Note: I'm not sure if this is actually a DXVK bug or a bug elsewhere in the display chain -- that's hard for me to tell. If I should report it elsewhere please let me know.
Software information
Unreleased game. But I really don't think this is a game-specific problem.
System information
Steam Deck OLED
I haven't included an apitrace or log, since I don't think it makes much sense or helps for this type of problem.

Do we actually get an R16G16B16A16_SFLOAT swap chain on the Vulkan side or is it R10G10B10A2? The D3D11 log should include the swapchain format.
Either way this is probably something for @Joshua-Ashton to look at. scRGB is more or less untested and known to be broken with Special K, but I don't have an HDR-capable setup to test things with.

Thanks for the fast replies!

Are you calling SetColorSpace1, or relying on some implicit behaviour we do not implement?

We are not (at least not right now, I'll implement that to check to see if that fixes it).
We're currently relying on the (documented) behaviour that R16G16B16A16_FLOAT swapchains are automatically DXGI_COLOR_SPACE_RGB_FULL_G10_NONE_P709.
If setting the color space manually does not solve the issue I'll send you an email.

Yeah we are not doing that rn. We should, I am going to sleep now so I'll do that tomorrow.

I was beaten to it =)

That also was the issue: manually setting DXGI_COLOR_SPACE_RGB_FULL_G10_NONE_P709 solved the problem.

Should be fixed in current master, let me know if there are issues with your use case.

I expect this change to fix it, but we will ship with explicitly setting the color space.
I'll try the old version of the game with new DXVK once I can easily get that on the Deck and let you know.
Side not that is completely unrelated to DXVK, but related to scRGB and might get a chuckle out of you:
I found a pretty severe bug in the Windows 11 compositor -- if any (sub)pixel of the FP16 frame you hand to the compositor has a negative value, it completely and statefully breaks the compositor's internal surface at that location. To the extent that displaying all other, unrelated applications is then also broken at those screen positions. Seems pretty wild for a user-space program to be able to do.


ability to specify registryIds for IAM instance profiles

We're using IAM instance profiles so specifying AWS credentials is not required, however that does not work if the the ECR registry is in a different account. There should be a way to specify what registryId should be passed to the GetAuthorizationToken call.

Added in https://github.com/objectpartners/rancher-ecr-credentials/commit/7ac823f01d7226a2884a73dbfffbeeb570713404.
Currently available on latest and will be part of 1.2.0 release


[Enhancement] Placeholder for skill GUI

Missing Placeholders are:

Skillname
current Skilllevel
Manacost of the skill with current skilllevel (should change when updating the Skilllevel).
Cooldowntime of the skill
damage value of the skill
duration of the skill effect
blockradius of the skill
required level of the skill
required skillpoints to upgrade
clickcombo

It might would be nice to just have to add something like <skillapi.skillname>. And the Lore and Name would be generated. This would save alot of time when adding skills to a custom Menu.
Also when upgrading a skill with force by console it dosn't show that it has been upgraded. It will need to reload the UI. I might be able to just reload the UI with a command.

I'm a little confused where or how you're planning to implement these placeholders? Are you wanting to use these for a custom GUI, or within the /class skill menu? The class skill menu uses the icons you define for the skill, so having placeholders here doesn't make a lot of sense when you'd just write the values in when you build the skill.


[Scatter] When changing the amount of data in series the chart breaks if there are more series than before

Codepen
https://codepen.io/anon/pen/xQqjzG
Explanation

What is the behavior you expect?
When updating a series with a new one with a greater length, the chart should update correctly.

What is happening instead?
Chart breaks and throws an error

What error message are you getting?
TypeError: Cannot read property '0' of undefined

Thanks for reporting, @JCQuintas
Fixed it


Flat Boat Frames + some slight cleanup in how we register stuff for each TFC wood to better support addons

This pr is pretty light. It has flat boat frames, the assets and the relating data gen in addition to a new helper method to make doing things for each TFC wood easier

Rebased so that the history makes sense for updating the compartment tags

This PR is good to go


[Feature request] Automatically reload certificates when expired

(I mistakenly wrote this feature request in the Helm repo first, hence the structured layout.)
Is your feature request related to a problem ?
When running the elasticsearch-exporter against an ES cluster setup by the elastic operator, it uses CA certificates with a validity of (by default) a year. If the cert expires, it automatically gets replaced by the elastic-operator. However, the exporter keeps using the old certificate, even when it is expired. Restarting the Pod solves the issue.
Describe the solution you'd like.
Simplest would be if the elasticsearch-exporter would fail its health check if it detects it's using a certificate that is expired.
Better would be to check if the mounted certificate has been changed and use that from that point onwards, as it no longer requires a restart.
Additional context.
Although we get warnings regarding scrape failures, this issue still means we lost some metrics. It's not super crucial, of course, but it would be nice to have it fixed.

Just noticed #414!


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.8.11

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.8.9 -> v1.8.11

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper
v1.8.11
Compare Source
Bug Fixes

deps: update module github.com/gruntwork-io/terratest to v0.41.23 (#543) (a6dc684)

v1.8.10
Compare Source
Bug Fixes

deps: update gomod (#534) (e9bace0)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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

GitHub release
v3.8.4

Your semantic-release bot :package::rocket:


Add missing space between { and | for Minecraft Server

Small fix missed in https://github.com/caskroom/homebrew-cask/pull/13893.

Merged as da3d42156f164c4ef2ef1be884a4ad102a6469eb.


Update confirmation page

Pivotal

Also includes

Changing 'Question x of 14' to 'Step x of 16' and adding this to Summary and Confirmation
adding JS for print page on confirmation

Printing the page looks awful right now. There is a separate ticket to create a sensible print stylesheet to fix this.

:+1:

Great, merge at will. :+1:


AiB Accepts the Deployment of a GEN 2 VM and the Distribution Process Fails.

Hello team!
On the public documentation, I see that the AiB does not support the GEN 2 VM.
https://docs.microsoft.com/en-us/azure/virtual-machines/image-builder-overview#hyper-v-generation
But it allows me to trigger the deployment and the process fails on the Image Distribution.
Source Image as a Gen 2 VM.

AiB template deployed with success.

AiB generates the Managed Image as generation 1

SIG image with Unknow state

AiB error on Azure Portal:
    "provisioningState": "Succeeded",
    "lastRunStatus": {
        "startTime": "2021-11-12T15:04:52.400830569Z",
        "endTime": "2021-11-12T15:28:44.353389414Z",
        "runState": "Failed",
        "message": "Failed in distributing 1 images out of total 1: Some error happened, please check the error details."
    },

Thanks, very interested in this bugfix/feature

Hey Guys!
The public document was updated!

I also did a test building a Generation 2 Image and it worked =D

Did you deploy the template with e.g. some powershell files you used to customize the image further?

Hello @schiffne,
My deployment runs a PowerShell script that enabled IIS and other settings on the Windows Server.
If you want, I can share the template with you.
Best regards,
Estevão

I will run my deployment now as well, installing MECM and AntiVirus, it does work with Gen1 images...
will let you know

This would be great if you can share it here: For us it fails here, where it shows yellow, the script currently contains only EXIT 0

This would be great if you can share it here: For us it fails here, where it shows yellow, the script currently contains only EXIT 0

What's the output on your customization.log file?

it looks similiar to this. The VM WIN11! does not proceed it just stops and waits for the timeout in the template and than does the cleanup and deletes itself, here is a screenshot, where the VM is just waiting to execute that one single script:

Seeing the same issue. Same set of customization work fine with Gen1 (Win10), but any powershell customisation steps fail on a Gen2 (Win11) run.
Attempts to run the PS, but sits and waits until the timeout and then moves to cleanup.

@StevybSC did you ever find a solution to this?  Currently seeing the same behaviour too

So after further experimentation today, adding "runAsSystem": true to the PowerShell customiser appears to resolve this issue.  By default, if it isn't specified the value is false.  Changing this allowed my builds to continue without hanging


Translate concepts/services-networking/ingress-controllers into Japanese

This is a Feature Request
What would you like to be added
Translate concepts/services-networking/ingress-controllers into Japanese.
Why is this needed
This page has not been translated.
Comments
/language ja
/assign

Closing because this page has been translated already.


update links to Discord

What does this PR do?
switching to discord

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

@Borda Discord invite links expire after 7 days. Is this a permanent one? @lantiga created a permanent invite link last week and it looks like this: https://discord.gg/VptPCZkGNa
It is different from the one you have, could you please double check it? See internal slack message.

Discord invite links expire after 7 days. Is this a permanent one? @lantiga created a permanent invite link last week and it looks like this: discord.gg/VptPCZkGNa

thank you, fixed :)


Search PATHEXT instead of 3 hardcoded values

Redo of #134.
Since it was only a few lines, it was easy to rewrite against master.

@isiahmeadows could you generate-docs?

LGTM
FYI: Windows also has a built-in where command.

Do note that I use Linux, my Windows installation is rather broken ATM, and my computer is too old/underpowered to run a VM, so I don't know how well this runs, and I kinda feel guilty not being able to test this. :frowning:
But feel free to test this yourself. I did add a test. It's also a trivial enough patch that it should work.
(@nfischer If you all got Appveyor to run for PRs as well, that'd be awesome, probably even for you, considering this supports Windows. :wink:)
I submitted the original PR, #134, about 1 1/2 years ago, when my Windows installation actually worked (mine needs way more troubleshooting than I have time for ATM - my standard user directory requires Admin privileges to even read).

I'll test this out on my Windows machine today and see how it runs. Thanks again!

The nits are all totally optional. They're just for stylistic consistency (which we aren't very good at anyway, as you can probably see by other lines in this test that differ from the style I'm suggesting).
The big issue is the comment about case-insensitive comparison (and also see the typo I pointed out). Once that gets updated, I'll test again and give the thumbs up.

@isiahmeadows Could you rebase this off master? We now have Windows CI set up with appveyor, and it'd be great to verify this under that (although I'm fairly certain it'll pass after the rebase).

I like that GitHub has this "Update Branch" button where I could do that here. :smile:

What? They added such a button‽‽‽
Thanks! Ari
On Thu, Jan 28, 2016 at 3:38 PM, shelljs/shelljs
<EMAIL_ADDRESS>wrote:
I like that GitHub has this "Update Branch" button where I could do that here.:smile: [https://assets-cdn.github.com/images/icons/emoji/unicode/1f604.png]
—
Reply to this email directly or view it on GitHub
[https://github.com/shelljs/shelljs/pull/290#issuecomment-176482283] .[https://github.com/notifications/beacon/ABu7pJGYEXZGHWuh2J1oFc_U2wgsCAwAks5pep3-gaJpZM4HFNsu.gif]

That's what I just used. And it doesn't appear to be passing... I'll fix it when I get home. I've been a bit busy lately.

This latest push won't fix the build, but it'll give me a better idea what happened.

@nfischer It's surprising that the AppVeyor build isn't required...I'm guessing that was an oversight? (I just noticed this.)

I've rebased against master to remove all the superflouous commits.

@ariporad I think this is a recent addition, where if you don't have merge conflicts, you can just update your remote branch through GitHub itself instead of locally having to do it. It's just like with merging branches.

@isiahmeadows is checkpath() case insensitive? Could you add a

LGTM (in case lgtm.co missed it)

Thanks @isiahmeadows!
And yeah that was an oversight, At the time we added it, our windows build was broken.

@ariporad Okay.


Develop

Pull Request Test Coverage Report for Build 9

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+6.7%) to 96.667%

Totals

Change from base Build 8:
6.7%

Covered Lines:
17

Relevant Lines:
17

💛  - Coveralls


Fix istio

Which issue is resolved by this Pull Request:
Resolves #
Description of your changes:
-Replace Istio-1-14 base with awsconfig to patch manifiest
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
- deny_all_authorizationpolicy.yaml
- gateway.yaml
- gateway_authorizationpolicy.yaml

bases:
- ../../../upstream/common/istio-1-14/istio-install/base

patchesStrategicMerge:
- patches/service.yaml

Testing:

[ ] Unit tests pass
[x] e2e tests pass
Details about new tests (If this PR adds a new feature)
Details about any manual tests performed

pytest tests/test_sanity.py -s -q --region us-west-2 --root-domain-name <> --root-domain-hosted-zone-id <> --keepsuccess

=================================================================
                          Reading Config
=================================================================
=================================================================
                  Creating Subdomain in Route 53
=================================================================
INFO:botocore.credentials:Found credentials in environment variables.
INFO:e2e.utils.aws.route53:created hosted zone platformem4b4tpsp3.jsitu.people.aws.dev with id Z019966235MWFSLL12QIC
INFO:e2e.utils.aws.route53:waiting for for change id: C046451624R8D84CSPG8S to be IN_SYNC in hosted zone: Z01796955ZRKDEFXBKXA, domain: jsitu.people.aws.dev
=================================================================
                   Creating Certificate in ACM
=================================================================
INFO:e2e.utils.aws.acm:requested DNS validation for domain *.jsitu.people.aws.dev.                 certificate ARN is arn:aws:acm:us-west-2:351537711515:certificate/ab083d0a-8901-457c-bb26-8f9c939bc8df.
INFO:e2e.utils.aws.acm:arn:aws:acm:us-west-2:351537711515:certificate/ab083d0a-8901-457c-bb26-8f9c939bc8df: waiting for domain validation record ...
INFO:e2e.utils.aws.route53:waiting for for change id: C10162323LWPLUUJB21C8 to be IN_SYNC in hosted zone: Z01796955ZRKDEFXBKXA, domain: jsitu.people.aws.dev
INFO:e2e.utils.aws.acm:arn:aws:acm:us-west-2:351537711515:certificate/ab083d0a-8901-457c-bb26-8f9c939bc8df: waiting for validation ...
INFO:e2e.utils.aws.acm:requested DNS validation for domain *.platformem4b4tpsp3.jsitu.people.aws.dev.                 certificate ARN is arn:aws:acm:us-west-2:351537711515:certificate/29d62907-8632-49bb-9e40-b46f8d7a3e6d.
INFO:e2e.utils.aws.acm:arn:aws:acm:us-west-2:351537711515:certificate/29d62907-8632-49bb-9e40-b46f8d7a3e6d: waiting for domain validation record ...
INFO:e2e.utils.aws.route53:waiting for for change id: C03146601U8HXCVOEQPVT to be IN_SYNC in hosted zone: Z019966235MWFSLL12QIC, domain: platformem4b4tpsp3.jsitu.people.aws.dev
INFO:e2e.utils.aws.acm:arn:aws:acm:us-west-2:351537711515:certificate/29d62907-8632-49bb-9e40-b46f8d7a3e6d: waiting for validation ...
=================================================================
    Configuring Ingress and load balancer controller manifests
=================================================================
2022-09-08 18:35:15 [ℹ]  IAM Open ID Connect provider is already associated with cluster "helm-bugbash-2" in "us-west-2"
INFO:e2e.utils.aws.iam:created iam policy arn:aws:iam::351537711515:policy/alb_ingress_controller_helm-bugbash-29qlbkxlocb.
2022-09-08 18:35:21 [ℹ]  1 existing iamserviceaccount(s) (kube-system/aws-node) will be excluded
2022-09-08 18:35:21 [ℹ]  1 iamserviceaccount (kube-system/aws-load-balancer-controller) was included (based on the include/exclude rules)
2022-09-08 18:35:21 [!]  metadata of serviceaccounts that exist in Kubernetes will be updated, as --override-existing-serviceaccounts was set
2022-09-08 18:35:21 [ℹ]  1 task: {
    2 sequential sub-tasks: {
        create IAM role for serviceaccount "kube-system/aws-load-balancer-controller",
        create serviceaccount "kube-system/aws-load-balancer-controller",
    } }2022-09-08 18:35:21 [ℹ]  building iamserviceaccount stack "eksctl-helm-bugbash-2-addon-iamserviceaccount-kube-system-aws-load-balancer-controller"
2022-09-08 18:35:21 [ℹ]  deploying stack "eksctl-helm-bugbash-2-addon-iamserviceaccount-kube-system-aws-load-balancer-controller"
2022-09-08 18:35:21 [ℹ]  waiting for CloudFormation stack "eksctl-helm-bugbash-2-addon-iamserviceaccount-kube-system-aws-load-balancer-controller"
2022-09-08 18:35:53 [ℹ]  waiting for CloudFormation stack "eksctl-helm-bugbash-2-addon-iamserviceaccount-kube-system-aws-load-balancer-controller"
2022-09-08 18:35:54 [ℹ]  created serviceaccount "kube-system/aws-load-balancer-controller"
=================================================================
Creating Ingress, load balancer and updating the domain's DNS record
=================================================================

-------------------------------------------------------- live log call --------------------------------------------------------
INFO     root:_client.py:453 Creating experiment experiment-fsjfe21zsb.
PASSED
tests/test_sanity.py::TestSanity::test_run_pipeline
-------------------------------------------------------- live log call --------------------------------------------------------
INFO     root:_client.py:457 Creating experiment experiment-q5htcqwakf.
PASSED
tests/test_sanity.py::TestSanity::test_katib_experiment PASSED

========================================================= 3 passed in 3586.76s (0:59:46) =========================================================

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Run into RBAC access denied in vanilla deployment when we include

deny_all_authorizationpolicy.yaml
gateway.yaml
gateway_authorizationpolicy.yaml

Change istio awsconfig to include service patch only
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: istio-system
bases:
- ../../../upstream/common/istio-1-14/istio-install/base

patchesStrategicMerge:
- patches/service.yaml

Integration test:
Sanity:
pytest tests/test_sanity.py -s -q --region eu-west-2 --root-domain-name <> --root-domain-hosted-zone-id <> --keepsuccess

tests/test_sanity.py::TestSanity::test_kfp_experiment
=================================================================
                          Reading Config
=================================================================
=================================================================
                  Creating Subdomain in Route 53
=================================================================
INFO:botocore.credentials:Found credentials in environment variables.
INFO:e2e.utils.aws.route53:created hosted zone platform0zzrbh0azg.jsitu.people.aws.dev with id Z02323092ZPH8S38CS9LR
INFO:e2e.utils.aws.route53:waiting for for change id: C024667114NPAOABOFM8K to be IN_SYNC in hosted zone: Z01796955ZRKDEFXBKXA, domain: jsitu.people.aws.dev
=================================================================
                   Creating Certificate in ACM
=================================================================
INFO:e2e.utils.aws.acm:requested DNS validation for domain *.jsitu.people.aws.dev.                 certificate ARN is arn:aws:acm:eu-west-2:351537711515:certificate/f54ded33-f144-44a4-bf34-b4890ca21c43.
INFO:e2e.utils.aws.acm:arn:aws:acm:eu-west-2:351537711515:certificate/f54ded33-f144-44a4-bf34-b4890ca21c43: waiting for domain validation record ...
INFO:e2e.utils.aws.route53:waiting for for change id: C004158310NWDX9F1XFKP to be IN_SYNC in hosted zone: Z01796955ZRKDEFXBKXA, domain: jsitu.people.aws.dev
INFO:e2e.utils.aws.acm:arn:aws:acm:eu-west-2:351537711515:certificate/f54ded33-f144-44a4-bf34-b4890ca21c43: waiting for validation ...
INFO:e2e.utils.aws.acm:requested DNS validation for domain *.platform0zzrbh0azg.jsitu.people.aws.dev.                 certificate ARN is arn:aws:acm:eu-west-2:351537711515:certificate/1aed85ce-d5e8-4b8d-b3b4-e57c1adcb8b9.
INFO:e2e.utils.aws.acm:arn:aws:acm:eu-west-2:351537711515:certificate/1aed85ce-d5e8-4b8d-b3b4-e57c1adcb8b9: waiting for domain validation record ...
INFO:e2e.utils.aws.route53:waiting for for change id: C03637993IPLOCL8KQM6 to be IN_SYNC in hosted zone: Z02323092ZPH8S38CS9LR, domain: platform0zzrbh0azg.jsitu.people.aws.dev
INFO:e2e.utils.aws.acm:arn:aws:acm:eu-west-2:351537711515:certificate/1aed85ce-d5e8-4b8d-b3b4-e57c1adcb8b9: waiting for validation ...
=================================================================
    Configuring Ingress and load balancer controller manifests
=================================================================
2022-09-09 02:38:41 [ℹ]  will create IAM Open ID Connect provider for cluster "e2e-test-cluster-t6r7e0beue" in "eu-west-2"
2022-09-09 02:38:43 [✔]  created IAM Open ID Connect provider for cluster "e2e-test-cluster-t6r7e0beue" in "eu-west-2"
INFO:e2e.utils.aws.iam:created iam policy arn:aws:iam::351537711515:policy/alb_ingress_controller_e2e-test-cluster-t6r7e0beueo56772mfxa.
2022-09-09 02:38:51 [ℹ]  1 iamserviceaccount (kube-system/aws-load-balancer-controller) was included (based on the include/exclude rules)
2022-09-09 02:38:51 [!]  metadata of serviceaccounts that exist in Kubernetes will be updated, as --override-existing-serviceaccounts was set
2022-09-09 02:38:51 [ℹ]  1 task: {
    2 sequential sub-tasks: {
        create IAM role for serviceaccount "kube-system/aws-load-balancer-controller",
        create serviceaccount "kube-system/aws-load-balancer-controller",
    } }2022-09-09 02:38:51 [ℹ]  building iamserviceaccount stack "eksctl-e2e-test-cluster-t6r7e0beue-addon-iamserviceaccount-kube-system-aws-load-balancer-controller"
2022-09-09 02:38:51 [ℹ]  deploying stack "eksctl-e2e-test-cluster-t6r7e0beue-addon-iamserviceaccount-kube-system-aws-load-balancer-controller"
2022-09-09 02:38:51 [ℹ]  waiting for CloudFormation stack "eksctl-e2e-test-cluster-t6r7e0beue-addon-iamserviceaccount-kube-system-aws-load-balancer-controller"
2022-09-09 02:39:23 [ℹ]  waiting for CloudFormation stack "eksctl-e2e-test-cluster-t6r7e0beue-addon-iamserviceaccount-kube-system-aws-load-balancer-controller"
2022-09-09 02:39:25 [ℹ]  created serviceaccount "kube-system/aws-load-balancer-controller"
=================================================================
Creating Ingress, load balancer and updating the domain's DNS record
=================================================================
2022/09/09 02:39:27 well-defined vars that were never replaced: clusterName
customresourcedefinition.apiextensions.k8s.io/ingressclassparams.elbv2.k8s.aws created
customresourcedefinition.apiextensions.k8s.io/targetgroupbindings.elbv2.k8s.aws created
mutatingwebhookconfiguration.admissionregistration.k8s.io/aws-load-balancer-webhook created
Warning: resource serviceaccounts/aws-load-balancer-controller is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
serviceaccount/aws-load-balancer-controller configured
role.rbac.authorization.k8s.io/aws-load-balancer-controller-leader-election-role created
clusterrole.rbac.authorization.k8s.io/aws-load-balancer-controller-role created
rolebinding.rbac.authorization.k8s.io/aws-load-balancer-controller-leader-election-rolebinding created
clusterrolebinding.rbac.authorization.k8s.io/aws-load-balancer-controller-rolebinding created
configmap/istio-ingress-parameters created
configmap/aws-load-balancer-controller-config created
service/aws-load-balancer-webhook-service created
deployment.apps/aws-load-balancer-controller created
certificate.cert-manager.io/aws-load-balancer-serving-cert created
issuer.cert-manager.io/aws-load-balancer-selfsigned-issuer created
ingress.networking.k8s.io/istio-ingress created
ingressclass.networking.k8s.io/alb created
validatingwebhookconfiguration.admissionregistration.k8s.io/aws-load-balancer-webhook created
error: resource mapping not found for name: "alb" namespace: "kube-system" from "/var/folders/0f/szh5g9_j2812fqn26nn5nybh0000gr/T/tmphu3kxlky": no matches for kind "IngressClassParams" in version "elbv2.k8s.aws/v1beta1"
ensure CRDs are installed first
2022/09/09 02:40:36 well-defined vars that were never replaced: clusterName
customresourcedefinition.apiextensions.k8s.io/ingressclassparams.elbv2.k8s.aws configured
customresourcedefinition.apiextensions.k8s.io/targetgroupbindings.elbv2.k8s.aws configured
mutatingwebhookconfiguration.admissionregistration.k8s.io/aws-load-balancer-webhook configured
serviceaccount/aws-load-balancer-controller unchanged
role.rbac.authorization.k8s.io/aws-load-balancer-controller-leader-election-role unchanged
clusterrole.rbac.authorization.k8s.io/aws-load-balancer-controller-role configured
rolebinding.rbac.authorization.k8s.io/aws-load-balancer-controller-leader-election-rolebinding unchanged
clusterrolebinding.rbac.authorization.k8s.io/aws-load-balancer-controller-rolebinding unchanged
configmap/istio-ingress-parameters unchanged
configmap/aws-load-balancer-controller-config unchanged
service/aws-load-balancer-webhook-service unchanged
deployment.apps/aws-load-balancer-controller unchanged
certificate.cert-manager.io/aws-load-balancer-serving-cert unchanged
issuer.cert-manager.io/aws-load-balancer-selfsigned-issuer unchanged
ingressclassparams.elbv2.k8s.aws/alb created
ingress.networking.k8s.io/istio-ingress unchanged
ingressclass.networking.k8s.io/alb unchanged
validatingwebhookconfiguration.admissionregistration.k8s.io/aws-load-balancer-webhook configured
INFO:__main__:waiting for ALB creation ...
INFO:__main__: k8s-istiosys-istioing-3cd59be6cf-1910625459.eu-west-2.elb.amazonaws.com waiting for ALB status = active ...
INFO:e2e.utils.aws.route53:waiting for for change id: C03989082HC2EPXHXSWR8 to be IN_SYNC in hosted zone: Z02323092ZPH8S38CS9LR, domain: platform0zzrbh0azg.jsitu.people.aws.dev

------------------------------------------------------------------------ live log call -------------------------------------------------------------------------
INFO     root:_client.py:457 Creating experiment experiment-tlf0svll2c.
PASSED
tests/test_sanity.py::TestSanity::test_run_pipeline
------------------------------------------------------------------------ live log call -------------------------------------------------------------------------
INFO     root:_client.py:457 Creating experiment experiment-ooxgw6y95a.
PASSED
tests/test_sanity.py::TestSanity::test_katib_experiment PASSED

================================================================ 3 passed in 4194.58s (1:09:54) ================================================================

@surajkota


Conflict between external reactive class prop and internal classList prop of A, Link and NavLink components

Describe the bug
Issue
When using a reactive class prop with the NavLink component, the active or inactive class get removed when the class updates even if the NavLink is still active. Based on this documentation, I suspect a conflict between how the class prop and classList works.

It's also possible, but dangerous, to mix class and classList. The main safe situation is when class is set to a static string (or nothing), and classList is reactive. (class could also be set to a static computed value as in class={baseClass()}, but then it should appear before any classList pseudo-attributes.) If both class and classList are reactive, you can get unexpected behavior: when the class value changes, Solid sets the entire class attribute, so will overwrite any toggles made by classList.

Proposed solution
If we look at the code of the A component (which is re-exported as Link and NavLink), we can see that we are using the classList prop. One solution to avoid this conflict, could be to stop using the classList prop and manually compose the class, because their will always be people using a reactive class prop.
Current code
https://github.com/solidjs/solid-router/blob/b725fc5bedf910bfcd129d4a18491992d2720dcb/src/components.tsx#L215-L228
Suggestion
  const className = createMemo(() => {
    const classes = [props.class];

    if(isActive()) {
      classes.push(props.inactiveClass)
    } else {
      classes.push(props.activeClass)
    }

    return classes.filter(Boolean).join(" ");
  });

  return (
    <a
      link
      {...rest}
      class={className}
      href={href() || props.href}
      state={JSON.stringify(props.state)}
      aria-current={isActive() ? "page" : undefined}
    />
  );

Your Example Website or App
CodeSandbox example
Steps to Reproduce the Bug or Issue

Make the class prop of the NavLine reactive
Update the class and the active and inactive classes of the NavLink get removed

Expected behavior
The active and inactive classes should not be removed when a reactive class prop is updated.
Screenshots or Videos
https://user-images.githubusercontent.com/4420524/194949660-a7e988a0-2220-430c-89c1-49d1c2618d0b.mp4
Platform
The issue is probably on all plaforms.

OS: Linux
Browser: Firefox
Version: 105

Additional context
I noticed this issue when passing a NavLink component to the styled function of the solid-styled-components and using a theme that can be toggled (from dark to light mode). All the styles on the active NavLink was disappearing, because the styled function passes a reactive class prop to the NavLink.

I think this is a real problem, and admittedly something I introduced by allowing classList to exist. The proposed solution has the problem if the developer introduces a classList themselves I believe we hit this all over again. I think we should fix this but I think it probably has to work more granularly like classList so that we are applying styles independently instead of overwriting.

Hi @ryansolid,
I've opened this pull request to make sure that we are only using the classList to set the class to the A component. As you've suggested on discord we can achieve this by merging the external class prop in the classList. That way we'll be able to use the class or the classList.


do not support AFNetworkingSession

hope you can update support AFNetwoking 2.0+

Hey, friends, I removed AFNetWorking support just now.
But, the great idea is you could use qiniu_upload without any depency.
Not cool ? Or you don't know how to use it without  AFNetwoking ?
Whatever, thanks for your advise.

Do you have any question again? 😁

if you have other question, you could open another issue.
good luck~~

👍  Got it. You did a great jod. I realized you finish this all by yourself, not based on qiniu offical objc-sdk. Cool and fantastic job to solve multi-file upload task.

Thanks, friends. 😁
I want to make a universal uploader supporting all backends on iOS and  Android.
But the first step is Qiniu, and so much things need do do.
if you interested in it, you could open a pull request for this repo.

😄 that's too difficult for me~

Don't overthink it, just have fun and play.


Allow overridden and new methods on hubs

https://github.com/aspnet/SignalR/issues/333
@anurse @moozzyk @mikaelm12

@BrennanConroy,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

:up: :date: No more new methods

@rynowak Could you take a look?


[SPARK-19916][SQL] simplify bad file handling

What changes were proposed in this pull request?
We should only have one centre place to try catch the exception for corrupted files.
How was this patch tested?
existing test

cc @sameeragarwal @viirya

Test build #74367 has started for PR 17253 at commit 05febbd.

LGTM with very minor comment.

Test build #74367 has finished for PR 17253 at commit 05febbd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74367/
Test PASSed.

LGTM

Test build #74420 has started for PR 17253 at commit ad64848.

Test build #74420 has finished for PR 17253 at commit ad64848.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74420/
Test PASSed.

Thanks! Merging to master.


BUG: Gaps in neuralynx not handled properly

The low-level read_neuralynx_ncs function detects the presence of gaps in the .ncs file and issues a warning.

It sounds like mne.io.read_raw_neuralynx() should minimally check for temporal gaps between neo segments and issue a warning? For example, it should check that each neo.Segment[i] object starts when the neo.Segment[i-1] ended and raise a warning if this is not the case (i.e. there's temporal gap, assuming the information in neo is accurate)? And potentially also reconstruct/fill/mark missing samples such that the time axis (i.e. raw.times) is continuous and valid.
If this is on track, happy to open a separate issue and work on this.
Originally posted by @KristijanArmeni in https://github.com/mne-tools/mne-python/issues/11969#issuecomment-1832082578

Starting with a warning is good. Correct behavior would be (instead of warning) to fill these gaps with zeros and annotate with BAD_ACQ_SKIP, then MNE-Python functions will handle them properly and as you say the time axis will be correct. So if you want to jump straight to that and skip the warning business feel free!
Then the separate sample rate problem will be solved by #11408, which I came up with a tentative workable plan for.

Starting with a warning is good. Correct behavior would be (instead of warning) to fill these gaps with zeros and annotate with BAD_ACQ_SKIP, then MNE-Python functions will handle them properly and as you say the time axis will be correct. So if you want to jump straight to that and skip the warning business feel free!

Eyelink systems sometimes also have tracker dropout between trials, resulting in gaps in those recordings. We filled these gaps with NaN and annotated with BAD_ACQ_SKIP (we used NaN becasue 0 technically is a real value for eyegaze data), but the presence of those NaN's does cause an issue for a lot of preprocessing functions that operate on the time axis and don't respect the BAD_ACQ_SKIP annotations, i.e. resampling.. I'll definitely be interested in seeing how you handle the gaps in this case so that maybe we can try to make read_raw_eyelink consistent with your design choice.

@scott-huberty I hatched what I think is a workable plan for resampling in https://github.com/mne-tools/mne-python/pull/11408#issuecomment-1831037882 -- feel free to look there and let me know what you think


Processing Events with Salesforce Connector - Topic Revamp

IMPORTANT: This is the original topic that was done for v10.10. Since the connector is now on v10.11, the updated file needs to be copied to this PR: https://github.com/mulesoft/docs-connectors/pull/1486

Closed this PR to avoid merge conflicts. The topic in this PR was moved to https://github.com/mulesoft/docs-connectors/pull/1484.


added app_id param to get requests

@dmitrig01 @aaustin

I can't actually speak to the functionality, but:
@aaustin - we need to include the app id on all requests going forward, so we know which shard to queries to. for now, we will send legacy requests to all three shards and see which one gives us back data.

ok cool - that sounds really ugly

:+1:

yep see https://github.com/BranchMetrics/branch-backend/pull/688/files#diff-54ee6cd60658735b0afa26e8af585600R733


juicefs: fix build

Description of changes
JuiceFS 1.1.2 fails to compile with Go 1.22. - https://github.com/juicedata/juicefs/issues/4339
So use Go 1.21

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@dit7ya

Add a :+1: reaction to pull requests you find important.

Please squash your commits.


Add a map and show database on map

Value proposition
As a user,
I want to have a map
so that i can see where the different activities are located.
Description

Acceptance criteria

[ ] map should display the activities from database with marker on the map
[ ] marker should display onclick tooltip with name and location of the activity
[ ] Map will be accessible through navigation

Complexity
medium
Tasks

[ ] create a branch
[ ] add story for map
[ ] add spec for map
[ ] import database to map
[ ] import location and name to the map
[ ] add navigation to map
[ ] add map to app

Updated scribble and AC.


Left nav not loading

When you navigate to some books, the left nav doesn't load at all. For example:
https://main--prisma-cloud-docs-website--hlxsites.hlx.live/prisma/prisma-cloud/en/compute/30-xx/admin-guide/welcome/welcome

@iansk this was caused by the all-books metadata entry not containing the "main book", ie. an entry that has the same path as the book column's path.
I updated the metadata workbook, all-books sheet now has an optional version column. If included, it will only be included in that version's all-books entries. If undefined it will apply to all versions. This is just a shorthand to avoid duplicating the non-versioned books across all books in a product, the real issue is missing rows in the all-books sheet.
if you could help out by completing those rows that would be helpful

@maxakuru I'll fill in these rows today

@maxakuru I updated the all-books tab in the spreadsheet. Can you please take a look and see if I did it correctly?
Quick question: After making changes to the metadata sheet, I can go to the .page domain to see the impact of my changes, right?

@iansk yes, preview in sidekick to view on .page, publish -> .live
I commented on the changes in #64

looks like this is fixed, closing


Red box on appears under case notes after getting tagged

Describe the bug
In case details page, red box appears under case notes section after being tagged

Steps to reproduce
List any steps to reproduce behavior

On home page, click on any case
On case details page, have another user tag you in case notes (@yourname)
Refresh page

Expected behavior
Case notes should display as expected with no red box

To deprioritize as this is not functionality used by Hawaii for now (Low priority)


Hero minor refactor- blurb

Changes
Added a blurb, goal is to provide a clear concise summary of what Snowpack is and how it benefits devs
Would love feedback on copy and design
Testing
There are no tests in docs

👍 👍 👍  huge improvement!

Thanks for all the great feedback! Let me know what you think of this revision

That's for all the helpful feedback! I tried to distill all of it into new copy. Let me know what you think.

I'm wondering if there is a way to convey that it's not even always rebuilding every time you change either without it getting too wordy because it feels like that's an important benefit

I'm wondering if there is a way to convey that it's not even always rebuilding every time you change either

Yeah that’s a good thought… I personally have felt like that’s covered in the “Snowpack is fast” story (it’s more of a “how it’s fast” detail) but that may just be me?

I'm wondering if there is a way to convey that it's not even always rebuilding every time you change either

Yeah that’s a good thought… I personally have felt like that’s covered in the “Snowpack is fast” story (it’s more of a “how it’s fast” detail) but that may just be me?

Cool, for now changing back to the structure of the last edit + Fred's suggested changes

Great idea!


Social Login - none working

@danslo , is there a known issue with the Social Login feature? I've configured all Facebook, Twitter and Google and got keys and secrets but none of them is working as expected through back different issues. Should I use this issue to detail everything is this already a known issue?

Seems to work here. If you have steps to reproduce, feel free to re-open.


Small docs improvements

/format

/format

looks pretty good to me, with some small things to change. I'll split this up a bit and merge in smaller parts.

@stelmo this was soaked in already in the other 2 MRs, right? If possible, can you pls check and close this?

Thanks for fixing it! closing :)


Example Compile Issues

OS: Ubuntu 18.10
Websocket async server example will not compile.  These are the errors Visual Studio Code is displaying:

I just downloaded and installed Boost v1.69.0.  Am I missing something?

Are you mixing the 1.70 beta example with the 1.69.0 boost? You can't do that. If you want to use Boost 1.69 then you have to use the Beast 1.69 examples:
https://github.com/boostorg/beast/tree/320037a6bc6e70adb5ec715dc2bdfce9ed0ba41e/example

@vinniefalco Bingo!  Thank you sir again.


Allow ordering by Comparable compareTo

We already have listByPriority and listByAnnotation but now we are able to sort it when our interface implements Comparable without using any reflection or Runtime annotations

Sorry didn't think of toList() being a 16+ method, fixing it using collect()...

Thanks for taking the time to contribute.

now we are able to sort it when our interface implements Comparable without using any reflection or Runtime annotations

Since whatever interface you're looking for extends Comparable, wouldn't something like this be sufficient instead of adding another method?
var list = scope.list(SomethingThatExtendsComparable.class);
list.sort(null);

I added it because of https://github.com/avaje/avaje-inject/issues/723 but yes you are right 😄 sorry for overcomplicating things

I added it because of #723

But 723  only says ... "It would be cool if ..." - it does not actually give a use case of why it is desired or necessary.  As in, we shouldn't really add a feature because it's cool but more because we understand the use case that needs it (and that use case isn't best satisfied some other way).
Also note that this PR doesn't yet have a test that exercises the method and confirms it's behaviour.


Send message and receive other mesage couple times

Question

When I try to send unformation about my Age it asking my Gen and then my height. And if I'm writing my height it's asking my gen 2 times and 1 time my age.
How make it works like this:
It asking something i answer something, if answere do doesn't fit, it asking this question again, if answer fits it send new questions wich work with same algorithm.?
Code:
let weight, height, age, dailyNorm, A, proteine, fat, glucide, gen;
let low=1.2, small=1.4, middle=1.6, big=1.7;
const TelegramBot = require('node-telegram-bot-api');
const token = '734206894:... ';
const bot = new TelegramBot(token, {polling: true,
onlyFirstMatch:true, });
bot.onText(//start/, (msg) => {
bot.sendMessage(msg.chat.id,"Бот активирован🤖" , { "reply_markup": {
"keyboard":[["Calculate🍎" ], ["Report🐞"]]
}});
console.log (msg.text);
});
function dataGen(a){
bot.sendMessage(a.chat.id,"Your gen(👦-m,👧-f)?"); console.log (1)
bot.on("message", (msg) => {
if (msg.text=="m"){ gen="m"; dataAge(msg);}
else if (msg.text=="f"){ gen="f"; dataAge(msg);}
else { dataGen(msg);}
console.log (2)
});
}
function dataAge(b){
bot.sendMessage(b.chat.id,"Your age👶?");
bot.on("message" , (msg) => {
if (msg.text>1 && msg.text<100){age=msg.text; dataHeight(msg);}
else{ dataAge(msg)}
});
}
function dataHeight(c){
bot.sendMessage(c.chat.id,"You height📏?");
bot.on("message" , (msg) => {
if (msg.text>100 && msg.text<300){ height=msg.text; }
else{ dataHeight(msg);}
});
}
bot.onText(/Calculate🍎/, (msg) => {
dataGen(msg);
})

I think I have the same issue:
https://hastebin.com/xupatetiti.js
It haves something to do with the chatId. Would be nice to see it fixed.

For the original issue, the problem is that .on() does not get removed but is a static listener and it is not linked to one specific message. So you should set only one or multiple (with completely different purposes) .on("message").
For you, @RFlintstone, you are registering a new .onText() every time a user sends a message. Please note that a message listener captures any message.
If you want to capture a specific message, you should set a global .onText(/yourregex/, () => {...}) for each message.

.onText(/yourregex/, () => {...})

I know that but each time the 'chatId' is activated it will execute the reaction one more in the next mention
Ex:
1)
User: /echo Hi
Bot: Hi

2)
User: /echo Hi
Bot: Hi
Bot: Hi

3)
User: just some text

4)
User: /echo Hi
Bot: Hi
Bot: Hi
Bot: Hi
Bot: Hi


openshift v4.6.0 generation fails

There are a couple of things that are causing the DSL generation to fail:
There is an all upper case named Kind: "DNS", causes an index out of bounds exception in DslKindMeta#methodName.
com.github.operator-framework.api.pkg.lib.version.OperatorVersion is definition of type "string" and format "semver" which is not supported.
There are 3 kinds that don't have the "kind" and "apiVersion" attributes:

io.cncf.cni.whereabouts.v1alpha1.OverlappingRangeIPReservation
io.metal3.v1alpha1.Provisioning
io.openshift.cloudcredential.v1.CredentialsRequest

Additionally the current way we deal with packages with '-' is to replace them with a '.'. That also is problematic as with this version of openshift it results in a package level named "package" which is a keyword in Kotlin.

The DslKindMeta#methodName did not expect all uppercase kind names. In case we don't find any lowercase letters it now returns the all lowercase version of the kind name.

The fallback default for format handling has been changed so it generates a "string" attribute. The warning is still logged. Not sure if we should keep the warning message as it sounds as something is wrong.

Added compensation for when we a BasicResource is detected that does not have the kind and apiVersion attributes. We now add them manually if needed.

Regarding the package naming, '-' are now replaced by '_' this fixes the 'package' keyword issue.
This is however a really breaking change.


Add Rev, OneCompiler, update ByDoor

Added Rev (closes #1059)
Added OneCompiler

I couldn't link directly to the account settings page since the URL must contain the logged in user's account ID number.

Updated ByDoor notes

Changed duplicate notes key to notes_pt_br

Thanks for your contribution!


Images disappear on collectionview scroll Xamarin Forms

Description

I implemeneted experimental collectionview with Xamarin Forms 4.0, images are loading when app launched but after we scroll images are getting disappeared.
I'm not sure this is the issue of CollectionView or FFImageLoading , so posting in both github Repo
To Track : https://github.com/xamarin/Xamarin.Forms/issues/6341

Steps to Reproduce

On scroll of collectionview(Xamarin forms 4.0) images getting disappeared in iOS, Android works fine.

Expected Behavior

Images should not get disappeared.

Actual Behavior

On scroll of collectionview(Xamarin forms 4.0) images getting disappeared in iOS, Android works fine.

Basic Information

Version with issue: Current Latest version
Last known good version: never tried with collectionview before
Platform: iOS

Reproduction Link / Code
Here is the code
       <CollectionView
                ItemsLayout="{x:Static ListItemsLayout.HorizontalList}"
                ItemSizingStrategy="MeasureFirstItem"
                ItemsSource="{Binding FavouriteRecipes}"
                HeightRequest="220">
         <CollectionView.ItemTemplate>
                    <DataTemplate>
                        <Grid
                            Padding="5,2"
                            WidthRequest="140">
                            <Frame
                                HasShadow="true"
                                Padding="0"
                                Margin="0"
                                IsClippedToBounds="true"
                                CornerRadius="2"
                                BackgroundColor="White">
                                <Grid
                                    Margin="0">
                                    <Grid.RowDefinitions>
                                        <RowDefinition
                                            Height="150" />
                                        <RowDefinition
                                            Height="*" />
                                    </Grid.RowDefinitions>

                                        <ffimageloading:CachedImage
                                            Grid.Row="0"
                                            HeightRequest="150"
                                            LoadingPlaceholder="placeholder_square.png"
                                            ErrorPlaceholder="placeholder_square.png"
                                            Aspect="AspectFill"
                                            Source="{Binding Image}" />
                                    <StackLayout
                                        Grid.Row="1"
                                        Orientation="Vertical"
                                        Padding="5,0">
                                        <Label
                                            Font="14"
                                            Text="{Binding Title}"/>
                                        <StackLayout
                                            Orientation="Horizontal"
                                            HeightRequest="30">
                                            <Image
                                                Source="ic_clock_black.png"
                                                Aspect="AspectFill"
                                                WidthRequest="20"
                                                HeightRequest="20"
                                                VerticalOptions="Center"
                                                HorizontalOptions="Start" />
                                            <Label
                                                Font="13"
                                                Text="{Binding Time}"
                                                VerticalTextAlignment="Center" />
                                        </StackLayout>
                                    </StackLayout>
                                      </Grid>
                            </Frame>
                        </Grid>
                    </DataTemplate>
                </CollectionView.ItemTemplate>
            </CollectionView>

Here is the link of video
https://www.dropbox.com/s/wpgxkl6prtmn0iu/collectionview ffimageloading issue.mov?dl=0

I can confirm we are seeing the same issue with cached images in listview and flowlistview, pretty sure issue related to this Lib

Some times they show the first time some times they don't, other times scrolling away and then back then images disappear  as well

@SagarPanwala Could you make a simple repo so I could add it to FFImageLoading project and debug it?

@SagarPanwala we are actually facing issue on android. We havnt tested on iOS

@daniel-luberda https://github.com/xamarin/Xamarin.Forms/issues/6341#issuecomment-503448744

@daniel-luberda Is this an issue with FFImageLoading or Forms?

I am still facing this images dissapering after scroll issue. Any solution?

I faced the same issue. It doesn't seem to have been fixed yet.


Feature Request: Add Timeline / elapsed time indicator

To provide an indicator how much time elapsed for different stages or steps it would be awesome to reflect another row showing any amount of hours, days, weeks, months.
Nothing too fancy just another dotted line with "bullets" as indicators in timeline.

Have added an optional row (called 'timeline') to add a short line of text below each item. See the updated example csv


Updates demo with ASCII map and gets rid of writing file

The demo will no longer write the synthesized time data to file.
The demo will print an ASCII if matplotlib is not available.
Docstring error is corrected.

It turns out that  'three_sources.h5' are needed for the getting started documentation. Maybe this could be changed in the future. For now I leave it.


Large hashlist support

It seems that Phatcrack refuses to save large hashlists (> 10000 hashes), giving an error 500, "Failed to create hashlist".
Would it be possible to add file upload support or otherwise support larger hashlists please?

Can you check with your administrator what the corresponding error is in the backend logs?
When you receive a 500, you should hopefully also get a UUID that can be grepped in the backend. The UUIDs aren't displayed everywhere yet though, unfortunately, so you might have to open your browser's network tab to find it.
Finally, just in case its something specific, which hash mode as you using?
Regardless though, I'll definitely get to work on supporting uploading hashlists as a text file instead, which will be part of wider work to support other formats (such as zip, keepass files, etc).
Thanks!

Howdy,
We are running v0.1.3.
Im not sure if this is the exact error that we're running in to as I've also opened another issue detailing another error I found in the logs today.
However it is the only one complaining about size:
2023/11/15 08:56:37 /app/api/internal/db/project.go:112 extended protocol limited to 65535 parameters
[322.567ms] [rows:1] INSERT INTO "hashlists" ("created_at","updated_at","deleted_at","project_id","name","version","hash_type") VALUES ('2023-11-15 08:56:37.663','2023-11-15 08:56:37.663',NULL,'127844f8-a6e9-483b-bcdf-259b35a7c099','large',1,1000) RETURNING "id"

Corresponding web response:
{"URI":"/api/v1/hashlist/create","content_length":"1165379","error":"extended protocol limited to 65535 parameters","error_id":"0166cedb-75b3-4b56-933c-021ee6bb1c7f","latency_ms":947,"level":"error","method":"POST","msg":"request error 0166cedb-75b3-4b56-933c-021ee6bb1c7f"}

Ah, looks like Postgres has limit of 65,535 parameters in a parameterized query. Each inserted hash has 6 parameters, placing the limit of hashes inserted in a single query at just over 10k hashes.
I'll fix this by splitting the hashes into chunks of 1,000 and use several queries, wrapped in a transaction.
For sanity's sake though, I'll probably add a hard limit of 100k hashes (configurable by admins) to prevent resource exhaustion.

Dang postgres


Break apart Maven Plugin in the DFD.

we can talk about this on Monday

is it not due tonight?

already done!


Updated the Powershell Script

Updated the Powershell Script to be compatible with the latest Azure Powershell (1.3.3)

Hi @johnkattenhorn, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). 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, AZPRBOT;

@johnkattenhorn - this file has been deleted from master, so I need to close.

@tysonn I've had a look through the latest documentation and Operational Insight seems to have been removed. I currently have an open ticket with support has OMS doesn't seem to be reading our SF logs either. Do you know if this is the reason that the information was removed ? Can OMS still be used to read Service Fabric generated logs or do I know have to use ELK ?


Don't force --noUpadate flag

Given the current issue in the upstream from here: https://github.com/jeremylong/DependencyCheck/issues/6029, the action forced the --noUpdate flag here: https://github.com/dependency-check/Dependency-Check_Action/blob/3102a65fd5f36d0000297576acc56a475b0de98d/action.yml#L33. This flag appears to not sync when you try and pass in --purge.

Hi
The update is not done as part of the scan. Every night the image is recreated with the newer database

@javixeneize When is that image created which is then used by the action? We are trying to get the upstream RetireJs fix and the action won't pull it in/force an update/purge using the CLI, best we have come up with is due to the embedded --noUpdate which is in the CLI args as it is burned in from your code.

It is built here - https://github.com/dependency-check/DependencyCheck_Builder/actions


WIP: refactor to new api

API has changed a bit and parameters are now a Record<string, Parameter>.
This PR should refactor the UI code to work with the new API.

@dagda1 Please remove WIP: in the title and/or ping me when the PR is ready for review. Thanks.


add distance functions

perhaps we should also consider incrementing version number

Please check your formating (tabs vs spaces) since this is Python code this can be even critical. While at it why don't you squash your two commits into one? Also a stylistic recommendation would be to capitalize your commit message.

I converted tabs into spaces like rest in the file. Thank you for noticing it.

Sorry for the delay on this, I've been traveling for the holidays.  I'm skeptical about the delineation between ISO and SAE being exposed.  I think we should prefer to make our software as unitless as possible, so that the user doesn't have to specify whether their inputs are in inches or meters, and they can rely on a Clothoid to spit out results defined in the same units that they put in.  It would also be more consistent to only expose one distance method on Clothoid, and we could make the offset an optional argument with default value of zero.  I'm thinking of something like:
def Distance(self, qx, qy, offset=0):
return self._ClothoidCurve.DistanceISO(qx, qy, offset)
If there's a significant performance difference in the C++ code between DistanceISO and Distance, we could explicitly check if the offset is zero and call Distance if so.  Not sure if DistanceISO uses an AABB tree or what the performance implications are if it does, but that's something we should keep an eye out for.

I was trying to keep python code as close to the original c++ code as possible, otherwise we will also need to supply documentation on how that offset is calculated. If you think it is ok to diverge from the library then I will rewrite that PR

I would also try to keep close to the original C++ library. If there's additional logic on the Python side it will become much more effort to maintain it.

I was trying to keep python code as close to the original c++ code as possible, otherwise we will also need to supply documentation on how that offset is calculated. If you think it is ok to diverge from the library then I will rewrite that PR

Sorry, I accidentally closed that thread


NetBSD compatibility

webview compiles successfully on NetBSD when netbsd is added into the 2nd and 3rd line in webview.go, changing them to
#cgo linux openbsd freebsd netbsd CXXFLAGS: -DWEBVIEW_GTK -std=c++11 and
#cgo linux openbsd freebsd netbsd pkg-config: gtk+-3.0 webkit2gtk-4.0
could this be added? It would allow NetBSD users to use go get directly without editing the source code.

Related to #431


Fix for WebGL Context Lost on Windows Devices with Some Integrated Intel GPUs

Seems like there is a bug in Chrome for Windows users with some models of the integrated Intel GPUs. As a result, OHIF and/or Cornerstone3D demos may experience rendering issues, with the GPU process consuming an excessive amount of memory for volume viewports. As a result, the webGL context may be lost and the browser crashes.
The bug has been narrowed down to be inside the Chrome and not from us, more especifically the Angle backend component of Chromium. While we are working on reporting this bug, we have a solution that has proven to be effective in fixing this issue which has worked on 4 separate machines having the same issue.

Ensure that your GPU is not blacklisted by your browser. To enable ignore-blacklist, go to chrome://flags/#ignore-gpu-blocklist and enable it. We have one machine who had NVIDIA GeForce RTX 2060 and it was blacklisted.
Next, make sure that your browser is configured to use one of the following:

Angle with the latest available WebGL backend (options are: d3d11on12, vulkan)
Angle with OpenGL

For Chrome users, you can change your angle value by going to chrome://flags/#use-angle and changing the value as described.

(Note that your selected Angle must be supported by your graphics card and driver. While Chrome and Firefox will always display available options, it's important to check that your selected Angle is compatible with your machine.)
🎸 Special thanks to Rodolfo (@ladeirarodolfo )for finding this solution and sharing it with the community! 🎸
PS:

d3d11 is different than d3d11on12, use d3d11on12 if you have it
For edge browser, you can set the same flag using edge://flags/#use-angle
For Mac users, we recommend using Default Angle settings.
Firefox users, please navigate to https://wiki.mozilla.org/Blocklisting/Blocked_Graphics_Drivers and refer to the section "How to force enable...." to enable ignore-blacklist.

no luck here....same error no matter which ANGLE I use

Hi there I have a device with this problem,
In chome and edge turning the angle on D11OnD12 solved the problem,
But in firefox no luck I tried to pass true
webgl.force-enable
webgl.msaa-force
layers.acceleration.force-enabled
also try
webgl.angle.force-d3ond11
Non of these settings solves this problem on latest firefox, any idea of any settings should i play with ? Or should we just say to users to avoid firefox ?

no luck here....same error no matter which ANGLE I use

same with u, have u ever solved it

No i have not


can not crop

all config follow on README.md
it works well ,can upload image ,can delete image
when I corp a image ，it alert “Error: item is not  cropped”

Can you show me your ajax request response? When the crop apply the code send a ajax request with the position and sizes. With that data php cropped the image. But its possible that php hasn't got enough memory and than the code crashed.
If it crashe php return error in the ajax request. Can you show me the response?

5968e9a84db70: POST http://<IP_ADDRESS>/index.php?r=imagemanager%2Fmanager/crop at 2017-07-14 03:56:24 pm by <IP_ADDRESS>
CropData
[
'x' => '75.5086956521739'
'y' => '33.93260869565217'
'width' => '456'
'height' => '378.3999999999999'
'rotate' => '0'
'scaleX' => '1'
'scaleY' => '1'
]

Thanks for the post data, but can you give me also the response data?

596a19468b51d: POST http://<IP_ADDRESS>/index.php?r=imagemanager%2Fmanager/crop at 2017-07-15 01:31:50 pm by <IP_ADDRESS>
Response Headers

Name
Value

X-Powered-By
'PHP/7.0.12'

Content-Type
'application/json; charset=UTF-8'

X-Debug-Tag
'596a19468b51d'

X-Debug-Duration
'136'

X-Debug-Link
'/index.php?r=debug%2Fdefault%2Fview&tag=596a19468b51d'

596a19406d72e: POST http://<IP_ADDRESS>/index.php?r=imagemanager%2Fmanager/get-original-image at 2017-07-15 01:31:44 pm by <IP_ADDRESS>
Response Headers

Name
Value

X-Powered-By
'PHP/7.0.12'

Content-Type
'application/json; charset=UTF-8'

X-Debug-Tag
'596a19406d72e'

X-Debug-Duration
'159'

X-Debug-Link
'/index.php?r=debug%2Fdefault%2Fview&tag=596a19406d72e'

596a193d56365: POST http://<IP_ADDRESS>/index.php?r=imagemanager%2Fmanager/view at 2017-07-15 01:31:41 pm by <IP_ADDRESS>

Name
Value

Content Type
'application/x-www-form-urlencoded; charset=UTF-8'

Raw
'ImageManager_id=10&_csrf=70K70ZUQonWVuptthuLHECj6vc3KqYDMiyACrEWsPF90BEk4NP1FvwZc5DalBnj0CKp3cuqVHj_L2EfH58CZ_g%3D%3D'

Decoded to Params
[
    'ImageManager_id' => '10'
    '_csrf' => '70K70ZUQonWVuptthuLHECj6vc3KqYDMiyACrEWsPF90BEk4NP1FvwZc5DalBnj0CKp3cuqVHj_L2EfH58CZ_g=='
]

Sorry if im not clear, but I need the response of the crop request. See below:

{"name":"Exception","message":"Class 'Imagine\Image\Palette\RGB' not found","code":0,"type":"Error"
,"file":"D:\qampp_app\htdocs\vendor\noam148\yii2-image-manager\controllers\ManagerController.php"
,"line":296,"stack-trace":["#0 [internal function]: noam148\imagemanager\controllers\ManagerController-

actionCrop()","#1 D:\qampp_app\htdocs\vendor\yiisoft\yii2\base\InlineAction.php(57): call_user_func_array
(Array, Array)","#2 D:\qampp_app\htdocs\vendor\yiisoft\yii2\base\Controller.php(156): yii\base
\InlineAction->runWithParams(Array)","#3 D:\qampp_app\htdocs\vendor\yiisoft\yii2\base\Module
.php(523): yii\base\Controller->runAction('crop', Array)","#4 D:\qampp_app\htdocs\vendor\yiisoft
\yii2\web\Application.php(102): yii\base\Module->runAction('imagemanager/ma...', Array)","#5 D:
\qampp_app\htdocs\vendor\yiisoft\yii2\base\Application.php(380): yii\web\Application->handleRequest
(Object(yii\web\Request))","#6 D:\qampp_app\htdocs\web\index.php(12): yii\base\Application->run
()","#7 {main}"]}

POST http://<IP_ADDRESS>/index.php?r=imagemanager%2Fmanager/crop | 500 Internal Server Error |   | 189ms

Thats what i need! Do you use a specific version of Imagine in your composer file? I looks like Imagine\Image\Palette\RGB not exists in the version that you used.

Thank you for your reply。let me check it，and  version of Imagine is 0.5.0

Try to update it to 0.6.3
That the version I used.

I'm getting the same crash.  With the same error.    I think a dependency in the image manager composer file should be added.


ContentSwitcher with Icons

It would be nice to be able to use icons as the contents for content switchers.

Agreed. Would you be interested in submitting a PR?

It will depend on whether we decide to go this route in our own project. If so, then I will submit a PR on the feature, if you don't get to it first.

This feature has been added in v0.8.0.
A slot wraps the text prop in the Content Switcher Switch component.
In this reduced example, the first two switches use the text prop while the third switch overrides it with an icon.
<ContentSwitcher>
  <Switch text="First section" />
  <Switch text="Second section" />
  <Switch>
    <span style="display: flex; align-items: center;">
      <Add16 style="margin-right: .25rem;" />
      Third section
    </span>
  </Switch>
</ContentSwitcher>

Very cool!


Create Your Blog Section

By Previewing your Blog section I realised you haven't made any database for your blogs to maintain. So i can do this for you where you can add your blogs from Mongodb.
Please Assign me this task under GSSOC'23.
Here the Attachment

@Hemant2335 Will you be working on the frontend part or the backend part?

I can do both but I will prefer more the Frontend part .

oh ok, so if you agree then I can work on the backend part of creating the blogs.

Yaa absolutely you can .

I would like to contribute to this issue under GSSOC'23. Please assign me this issue.


Revert "Adding K2T Documentation"

Reverts MicrosoftDocs/kaizala-docs#236

Docs Build status updates of commit 8bc3cc3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Articles/PartnerDocs/PartnerTrainings.md
:warning:Warning

Details

Articles/PartnerDocs/EndUserTrainings.md
:warning:Warning

Details

Articles/connectors/actions_get.md
:warning:Warning

Details

Articles/connectors/actions.md
:warning:Warning

Details

Articles/images/K2T_Acion_Logo.png
:white_check_mark:Succeeded

Articles/images/K2T_Add_button.png
:white_check_mark:Succeeded

Articles/images/K2T_Page_App_overview_Icon.png
:white_check_mark:Succeeded

K2T Documentation.md
:white_check_mark:Succeeded

Articles/PartnerDocs/PartnerTrainings.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Warning-duplicate-titles] Title 'Get Started with Kaizala' is duplicated in: 'PartnerDocs/EndUserTrainings.md, PartnerDocs/PartnerTrainings.md'. Titles must be unique within a docset.
Line 1, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

Articles/PartnerDocs/EndUserTrainings.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Warning-duplicate-titles] Title 'Get Started with Kaizala' is duplicated in: 'PartnerDocs/EndUserTrainings.md, PartnerDocs/PartnerTrainings.md'. Titles must be unique within a docset.
Line 1, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 4, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 11, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 22, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

Articles/connectors/actions_get.md

Line 2, Column 8: [Warning-duplicate-titles] Title '/actions' is duplicated in: 'connectors/actions.md(2,8), connectors/actions_get.md(2,8)'. Titles must be unique within a docset.
Line 10, Column 1: [Warning-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
Line 14, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

Articles/connectors/actions.md

Line 2, Column 8: [Warning-duplicate-titles] Title '/actions' is duplicated in: 'connectors/actions.md(2,8), connectors/actions_get.md(2,8)'. Titles must be unique within a docset.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel