Question about generating Contents from n values on pb

Question for @martindholmes (when you are back):
In the hamburger menu for semi-diplomatic (OS) texts, the items in the Contents list are generated from the @n value on the <pb> element.
Why are these not being generated for some files, including the Douai files (such as emdDouai_JC)? Is it because of the catDescs? The encoding of <pb/> is exactly the same in emdFV_Q1 and in emdDouai_JC. But the former has lbfQuarto and the latter has lbfManuscript.
Example:
<pb n="A1v" /> in the XML of emdFV_Q1 yields the following HTML.

--
  |
A1r

  |
A1v

It looks like the TOC is only generated from the pbs when the text qualifies as diplomatic:
<xsl:function name="hcmc:isDiplomatic" as="xs:boolean">
    <xsl:param name="root" as="element(tei:TEI)"/>
    <xsl:choose>

      <xsl:when test="hcmc:isPrimary($root) and not($root/descendant::tei:catRef[matches(@target,'Modern')])">
        <xsl:value-of select="true()"/>
      </xsl:when>
      <xsl:otherwise>
        <xsl:value-of select="false()"/>
      </xsl:otherwise>
    </xsl:choose>
  </xsl:function>

To qualify, it needs to be NOT Modern, and to be primary:
<xsl:function name="hcmc:isPrimary" as="xs:boolean">
    <xsl:param name="root" as="element(tei:TEI)"/>
    <xsl:choose>
      <xsl:when test="$root/descendant::tei:catRef[matches(@target,'ldtPrimary')]">
        <xsl:value-of select="true()"/>
      </xsl:when>
      <xsl:otherwise>
        <xsl:value-of select="false()"/>
      </xsl:otherwise>
    </xsl:choose>
  </xsl:function>

So as far as I can see, emdDouai_JC should qualify, so this is a bit puzzling. I'll look into it further.

Ah -- pbs only qualify if their @n attribute matches the regex for a signature, because the TOC is supposed to be constructed based on signatures:
<xsl:variable name="sigRegex" select="'\d*[A-Z]+\d+(r|v)'" as="xs:string"/>
i.e. zero-or-more digits, one or more capital letters, one or more digits, and r or v.
But emdDouai_JC does not have signatures, it has page numbers /^\d+[rv]$/. So there's your problem. Do we want to modify the definition of a signature to include anything that occurs in a page-number? We could do that by making the upper-case letter(s) optional. If we do that, will there now be other texts which suddenly get TOCs listing every page when they didn't before?

Incidentally 167r is missing its r.

Got it ... and found two more. Merci!

Yes, modifying the regex for a signature would be a nice solution and fix a whole bunch of other files at the same time. Please go ahead.

As of rev 12937, sigRegex is now sigFolRegex, and foliation numbers will be matched and treated in the same way as signatures. @JanelleJenstad bear in mind that the way the code is written to generate a TOC from pb/@n values, if it finds a single pb/@n that does not match the regex, it will not build the TOC, so these values have to be right. If the build works as hoped, this can be closed.

This ended up proving very messy and not working at all, so I've now rolled back to the original rev of the XSLT before I did anything. We'll need a comprehensive list of all the different forms that can appear in pb/@n and how each of them should be handled when it comes to constructing, or deciding not to construct, a table of contents.

Final decision on how to proceed:

Leading instances of /\d+;\s*/ in pb/@n have been removed per issue #145.
The $sigRegex is far to restrictive, since all sorts of things occur in genuine sigs, as well as folio numeration, so the only restriction should be that there are no spaces.
pb/@n should always be used to construct ids, and therefore Schematron should mandate that all pb/@n values are unique within the document.
When constructing a TOC, check to see if there are divs with heads; if so, use those. Failing that, if there are pb/@ns, then use those.

There are six instances of pb/@n which are not unique, found with this XPath: //pb[@n = following::pb/@n]. These will need to be fixed before we proceed.

Thanks for the XPath. We have now found and fixed all six instances. Please proceed at your convenience.

There's one more thing I think we should consider before we proceed. A TOC which consists of every pb in the document is going to be read out by a screen reader at painful length, and will be a horrible thing for someone trying to navigate the document using a keyboard. Are we absolutely sure it's a good idea to create an index to every page?

In the case of emdDouai_JC, there are labels throughout the text. Wouldn't it be better, in the absence of div/heads, to use any label elements to create a more meaningful TOC? @JanelleJenstad @LEMDO-PM don't you think this would be more reader-friendy than a list of all the pages?

Whatever is decided here should wait on issue #146.

In rev 13154, I've committed changes build the in-page TOC as follows:

If there are div/head elements in the text, use those.
If there are no div/heads, but there are more than four label elements, use those.
Otherwise, if there are signatures, use those.

This seems a decent compromise all round, and works nicely for e.g. emdDouai_JC. Waiting to see if it builds successfully.

This is working, but there is an extra feature we might want to allow: for cases where the actual text of the label is perhaps not ideal as a TOC entry, we could provide an option to put @n on a label, to provide an edited or canonical caption to use in the TOC. Then I would first check to see if there are more than four labels with @n, and if so, use only labels with @n; if there are fewer than four, but still more than four labels, then I would use all labels but substitute @n where it appears; and if there are fewer than four labels, I would use pb/@n. Waiting to see if the Douai folks would like to use this feature before adding it to the implementation.

No response from the Douai folks on this, so I'm closing it.

I only just asked the Douai folks in a Zoom consultation. Yes, the Douai folks would like to be able to add an @n value to  so that they can (in a few cases) override the text node of the  element. I am re-opening this ticket and asking @martindholmes to work on it after his vacation.

@JanelleJenstad Let's nail down the specification, then. The options are:

If there are more than four labels, we use labels, and whenever one has @n we use that, but for all others we use their content. This means all labels would be used whenever there are four or more in the document.
If there are more than four labels with @n, then we use only labels with @n, ignoring those which don't have @n. This enables the encoders to distinguish between labels which they want to use in a TOC and those they don't. If there are fewer than four with @n, then we proceed as in no. 1 above.
If there are any labels at all, we use labels for the TOC and ignore pbs; we use @n where it exists, and the label content where it doesn't.
Something else?

Four is just an arbitrary number I thought might make sense for a decent-sized text, but that number can be anything you think is appropriate.

Changing the milestone on this one because Douai wants it.
This specification is great. Four seems reasonable to me. We can tweak the number if necessary.
Please go for it.

The label "Question" threw me on this -- it should be implemented, if in fact it hasn't already been done. I'll go back and look at commits.

Final implementation added in rev 16361. I'll now add some documentation.

Documentation added in rev 16362. If builds are all OK, this can be closed.

Note before closing: At JJ's request, the threshold for using labels has been changed from four labels to twenty labels. This does not affect current Douai texts, which all have more than twenty, but if/when the Douai team start tagging new texts, the labels will not be used until there are twenty of them.


Updating comments

Fixed Issue
Edited the comments of
i)GetAllQuestions API of questions controller
ii)GetAllQuestionsAsync method of QuestionRepository
iii)getAllQuestions method of questions-dashboard.component
iv) getAllQuestions method of questions-service.ts
Files Changed
QuestionController.cs
QuestionRepository.cs
question-dashboard.component.ts
questions-services.ts
Checks

[x] Naming Conventions
[x] Server side code comments (proper English, grammar and no spelling mistakes)
[x] Client side code comments (proper English, grammar and no spelling mistakes)
[x] Unused variables, methods, blank spaces and name spaces.
[x] Optimal Code and code formatting
[x] Commit History is proper, no merge is done.
[x] Commit/pull request messages should be proper to justify your feature
[ ] All test cases are executed provided by tester.

Known Issues
N/A
Comments
Only the comments of few methods are updated in this pull request.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 QuestionRepository.cs#L39: Remove this commented out code.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 QuestionRepository.cs#L38: Remove this commented out code.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 QuestionRepository.cs#L38: Remove this commented out code.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 QuestionRepository.cs#L38: Remove this commented out code.


Enable to hide image (instead of display fallbackImageSrc ) when no image

when no image preview, displaying placeholder doesnt help to mucjh and takes extra space. if it is possible to not display the Image in case no preview?

Should be available in v1.13 !


Importing specific modules from CDN

What problem does this feature solve?
I'm not sure if this due to lack of docs around this, but it does not seem possible to only import specific modules as described here and let the browser do the tree-shaking when using the CDN (jsdelivr), only one large monolithic bundle is available.
What does the proposed API look like?
It would be good to have built versions on jsdelivr of all modules one can import via the npm syntax, so that people can do things like this while using a CDN as well.

I remember we ever built ES modules into the esm directory in several previous versions since v5.0.0. But it seems only a minimized version remained since v5.0.2. It would be great if we provide such a feature like the uuid package, which provides ESM support for both browser and Node.

Hi, I'm afraid we can only import the whole bundle in the browser now.
Importing ES module in the browser needs to have .js extension explictly. But we don't use .js extension in our source code. If we importing modules from 'echarts/core', it will try to load other modules without giving .js` extension and failed.
import-map seems to be a potential solution, but it don't have wildcard to match all the paths.
I'm not sure if importing specific modules with hundreds of dependency files has the scenario rather than importing a single large bundled file.  If there is a scenario, I think we can plan to add '.js' extension to fix this issue in our import statements.

Instead of changing the source code, how about handling the .js extension name in a build script?

Instead of changing the source code, how about handling the .js extension name in a build script?

Yeah, that's what I was suggesting. Keep your source code as-is, just mirror the structure of the lib directory in dist, with output files that can be used in a browser.


fix: field check fields model into _DcmModel

Fix issue in ha_toyota "there are no devices not entities loaded" #290

Thanks for your contribution. But that's propably not needed in future. Please have a look at https://github.com/DurgNomis-drol/mytoyota/pull/365


Re-fetch the Specific Application when APP_ALLOC event is received.

closes #380

Closed in favor of https://github.com/G-Research/yunikorn-core/pull/12

Closed in favor of https://github.com/G-Research/yunikorn-core/pull/12


[bp/1.27] datadog: fix span error property

This is a backport of https://github.com/envoyproxy/envoy/pull/31669 onto the v1.27 release branch.
These changes fix a bug in the Datadog tracing extension. In releases before v1.27, when the "error" tag was set on a Datadog span, a corresponding private property on the span would also be set to indicate the presence of an error. Release v1.27 included a rewrite of the Datadog tracing extension that removed this behavior, and the removal is a bug. These changes restore the old behavior.

failures are unrelated flakey upstream - there is a pr to fix this somewhere - i think it got closed as unneeded - ill check now ...

@dgoffredo should be fixed - could you rebase these 2 prs please, and ill land

GitHub is not having a nice time.

yeah - its a known/acknowledged problem - https://www.githubstatus.com/

/retest


why the cat emojis have different colors?

Because why not? They are less boring that way

It was just a stylistic choice, nothing particular.
I think these could be improved in terms of visibility of the expressions, however I would like to keep them different as it can help to distinguish them at a glance.


Mesos backend

Are there any plans for this in near future?
Also, possible to merge mesos and marathon backends in one.
 What do you think about that?

This could be nice to merge Marathon and Mesos backend.
But I'm not sure that Mesos API will be as precise nor provide as many features as Marathon (health checks).
I need to look deeper in Mesos API. Be we could create a mesos backend without merging with Marathon.

@emilevauge yes, we could, but we may face duplications because is we use both mesos and marathon backends at the same time, because all the apps running in marathon are actually tasks in mesos. You can see in the output above i have, for example, kafka-mesos framework running:
the scheduler is launched via marathon: kafka-scheduler marathon mesos.traefik :  [31200-31200]
and brokers are simple mesos tasks, not running on marathon: broker-1 kafka mesos.traefik :  [9092-9092]

Yep you're right, we have to deal with dups.
Did you already create a mesos backend?

@emilevauge nope, i just played a little with megos client with my cluster. I'm not a developer myself, so it can take time for me to actually implement this to traefik :)
But i surely can try :)

Closing since we support a Mesos backend by now: https://docs.traefik.io/toml/#mesos-generic-backend
Please file more specific issues if there's anything missing.


adds find my partner program documentation

Learn Build status updates of commit c868f91:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

data-explorer/findmypartner.md
:bulb:Suggestion

Details

data-explorer/toc.yml
:white_check_mark:Succeeded

data-explorer/findmypartner.md

Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/AM2.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/beanTech.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/DataMastery.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/Data3.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/DI_RGB_black_Medium.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/elastacloud.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image<EMAIL_ADDRESS>references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/iLinkDigitalSquareRGB400px.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/eis.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/LINKIT.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/MAQ-logo.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/MeshSystemsLARGE.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/NealAnalytics.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/noLogo.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/option40.svg' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/OrionInnovationLogo-black.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/terawe.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/tc_logo_horiz_black.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/uptake-logo-black-PNG.png' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.
Line 0, Column 0: [Suggestion: external-image] Image 'https://kustopartnersa.z6.web.core.windows.net/Zure-logo.jpg' references an external site. This poses a security risk and external images are unavailable in some environments. Reference an image file within the repo instead.

For more details, please refer to the build report.
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 or suggestions 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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

@cosh,
Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
We'll close this PR. Thank you.

Ill create a new PR


Crash in the case if header columns count less then general columns count

I have CSV where 2nd row has more values than header columns. Library doesn't work in this case.
Expected behaviour: no crash, skipping of redundant values at the end of row.

Perhaps you might be interested in adding a strict option to the constructor?  When set to strict=true, behave as currently implemented.  With strict=false, ignore extra columns and default to empty string when a value is missing.

Yes, thank you for the hint.
Obviously, it's better if lib doesn't crash. Described issue can be resolved with small fix in loop:
  // combine data and header into a row object
  for (size_t i=0, s=std::min(data.size(), header.size()); i<s; ++i) {
    row[i] = make_pair(header[i], data[i]);
  }

CooL!  Care to make a unit test and a PR?

@awdeorio sure

Closed by #20


Non-ldap models getting access to LDAP model info in mixed authentication with multiple guards

Laravel Version: 8.15.0
Adldap2-Laravel Version: 6.1.4
PHP Version: 7.4.11
LDAP Type:  OpenLDAP

Description:
I have three separate guards: admin, employee and trainer.  Admin and Trainer login using eloquent driver and uses 'admin' and 'trainer' guard respectively. Employees login using ldap driver and uses 'employee' guard.
I'm using jwt-auth library for token based authentication.
When I login as admin, I get a token. I can use that token to access admin info. And if I use the same token on Trainer routes, I don't get access to them (as expected).
However, if I use the admin token on an Employee route, I can access the employee's information with the same id as the admin (e.g admin with id 2 can access employee with id 2's info).
Similarly, Trainer can access Employee info when the id's match. Which is a security flaw.
This only occurs with LDAP. I've noticed that the employee guard, which was only supposed to work on employee routes, gets assigned to Admin and Trainer also. Thus Admin has both 'admin' and 'employee' guards available to him, therefore he's able to access employee info.
Steps To Reproduce:

This repo's new_role_test_ldap branch contains all my code related to this issue.

https://github.com/rummanrc/laravelguardissue8/tree/new_role_test_ldap

This is the env config for the publicly available LDAP server I'm using to test:

LDAP_CONNECTION=default
LDAP_BASE_DN='dc=example,dc=com'
LDAP_LOGGING=true
LDAP_HOSTS=ldap.forumsys.com
LDAP_PORT=389
LDAP_TIMEOUT=5
LDAP_SSL=false
LDAP_TLS=false
LDAP_USERNAME=cn=read-only-admin,dc=example,dc=com
LDAP_PASSWORD=password

Migrate and seed, then login as employee with any username and password from here https://www.forumsys.com/tutorials/integration-how-to/ldap/online-ldap-test-server/
Then login as admin or trainer.

Use the admin token as Bearer Token and make a post request on http://localhost:8000/api/employee/me
and you'll see that the employee info is visible

It seems like for every Auth::attempt or Auth::guard()->attempt() the AdldapAuthServiceProvider class is being used, regardless of the guard. I don't know why it's passing through though, without credentials.

Does the Adldap's DatabaseUserProvider override the laravel Auth's DatabaseUserProvider? I think that might be causing this conflict?  If yes, can I bypass this so that the DatabaseUserProvider will be used based on my current guard (or request parameter, or anything)

So I had to override the Adldap2-laravel's DatabaseUserProvider class and put a condition on the retrieveById function.
I added custom claims in the tokens of my models, and checked for the custom claim (which contains the role) to decide whether to return the model from that function or not.
This branch contains the code for this solution: https://github.com/rummanrc/laravelguardissue8/tree/bypass_ldap_auto_login
Let me know if there's a better solution for this... and provide the feature for eloquent and ldap based separate auth methods to work side by side, if possible...


ENHANCEMENT Cropper.js v2.3.4

@willmorgan would there need to be a new branch if a PR for this would be submitted? I've been working through the latest tag of the plugin and many of the plugin options have changed.
I'm working to see how feasible it is to get the default options to function the same as the current version, but some things may have been dropped so in that respect there would be breaking changes.

@muskie9 Thanks for doing this. Yes, a new semver branch would need to be created but I could sort that for you 😄

Perfect, thanks! I have most of the plugin specific things working in the cms, just working through some save issues. Hope to open a PR soon!

Cool, thanks. Just open a PR and I'll get to it as soon as I can. I will keep this issue open in the meantime.


Crowdin integration to translate the docs

Fixes #3890.
This wires up Crowdin to make the docs translatable.
As with most PRs which change deployment / CI, I expect we'll need some follow-up tweaks to this PR.
For now I have enabled Japanese and French as first locales, so that @isaomatsunami and I can try out the integration before trying to recruit translators in other languages.
At the moment this will only translate the current development version, not the latest stable release, so the translated pages will be deployed at URLs such as https://docs.openrefine.org/fr/next/ and not just https://docs.openrefine.org/fr/. When we cut out the next stable version (3.5) this would release the translations at that point.

@wetneb Should I delete my #4018 draft, I guess? probably right?

It seems to be doing things that I didn't do, but I cannot tell exactly what! If you are happy with the current result, then I guess another PR is not needed…

@wetneb yeah, let's build on what you have merged now. If we need some of the custom variable declarations (that I borrowed from Docusaurus's own CI/CD), we can always add them in as necessary.  I'll delete my PR.


Ready to publish on crates.io?

I've updated my WIP demo to use the new github version, but I think it would be nice if it was available for users on Crates too. Thanks

it is, I will publish an updated package https://crates.io/crates/rust-rocket

it is, I will publish an updated package https://crates.io/crates/rust-rocket

Version 0.7.1 is published https://crates.io/crates/rust-rocket

Version 0.7.1 is published https://crates.io/crates/rust-rocket


Use a pooch data store for docs to allow retry_if_failed

The docs use pooch.retreive, which doesn't expose retry_if_failed.

@annehaley CircleCI sometimes gets a 500 error when downloading from dkc.  The retry on the main datastore gets over this; we should do the same in the docs so that there are fewer spurious CI failures.

While we are at it, the urls for dkc data should use the api/v1/file/hashsum/{algo}/{hashvalue}/download route rather than the item id route.


Logging normalizer

Normalizer use is not currently logged in model parameters. Consider either:

Converting normalizer to a hyperparameter
Logging the normalizer with hyperparameters.json

Completed.


fix error variable name

Issue #, if available:
Non
Description of changes:
Variable name e is not defined. It should be error.

[x] By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you @rcmdnk for submitting this PR and bringing the issue to our attention. We will include the fix in the upcoming release.

Thank you @rcmdnk for submitting this PR and bringing the issue to our attention. We will include the fix in the upcoming release.

Hey @rcmdnk, thank you for your contribution. We have incorporated them as part of v1.4.0 release today.


failed to post notification on channel null android

I've integrated the library. On Checking crash with emulator v api 27, the notification is not appearing. Just showing a toast
Developer warning for package "mypackage.appname" failed to post notification on channel null
Please update .

Thanks will update over this weekend and will let you know. Thanks

I think it's due to the new oreo notification system, where a channel is required for notifications I guess ...


Composer Electron app - start bot fails (bot has LU content)

Describe the bug

Created a new bot
Add some OnIntent triggers with LU content
Provide LUIS authoring key and click on start bot

Version

Composer app.
Application SHA
test
SDK runtime packages
4.7.0-preview-191208-1

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Launch composer electron app
Create a new bot
Add one or more triggers with LU content
Click startbot
Provide LUIS authoring key

Expected behavior
Bot can start
Screenshots

Additional context
Sorry, something went wrong with connecting bot runtime

EPERM: operation not permitted, mkdir 'C:\Program Files\Bot Framework Composer\resources\app.asar.unpacked\build\plugins\localPublish\hostedBots'

@srinaath is looking at this

addressed via https://github.com/microsoft/BotFramework-Composer/pull/2894


An option to enforce emitting order

I know I can control the order of files via the ordering provided in my config (either in files or include). But since things like types can change ordering so that even if we have files:['a.ts', 'b.ts'], then the resulting output will have b.ts's compiled output first (see here)
Would it be possible to add a compiler flag so that we can enforce that a.ts doesn't pull in b.ts?  To enforce the ordering provided in files or include? I tried looking into doing this myself but this project's quite big. Something like a compiler error when trying to pull in a file that is "after" the current file in the include order.
Ideally, being able to make it so that if I have:
  include: [       "library/*",       "core/*",       "ui/*" ]
then files within library/* could depend on each other but not on things in core/* , and core/* could not depend on anything in ui/* (but could depend on things in core/* or lib/*). This would already help to solve many of my ordering-related headaches which mean I cannot use things like the import statement safely.

The only thing you need to do to get this behavior is not have /// <reference directives in your files. If you have a tsconfig listing your files, there's no reason to have those directives, so that should be a straightforward policy you could enforce with TSLint (or a simple commandline script).
I'm also a little confused on import - if you're using import (as in modules), there's no "order" per se.

Please note that we have a bug where the order of files listed in include is not honored at the moment. see https://github.com/Microsoft/TypeScript/issues/11461. this should be fixed in upcoming releases.

This might be a case of the include issue, though my issue went away once I moved around some type definitions, so I feel like "something" is happening.
In my case I had a.d.ts, b.ts , and c.ts.  I wanted the output to be in the sam order.
b.ts used types from a.d.ts which had a type from c.ts, so c.ts came out first, despite the fact that the include order was something like include: ['a.d.ts', 'b.ts', 'c.ts'].  I have no reference paths in my code.
Once I moved c.ts's interface definitions from c.ts to a.d.ts, the code started coming out in the right order.
Unfortunately I am unable to get this to happen for me in a minimal example, making me think something else is also going on. But the reality is that the output order was not the same as the import order in include. This could be a duplicate of #11461

use tsc --listFiles to see what order the compiler sees the input. you can see --traceResolution to know what files "imported" your file.

I do see the behaviour of #11461 with that command. I don't seem to be able to replicate my "fix" for the output, though. Might have been a hallucination on my part.
If the include order is fixed, that will probably solve the reasoning behind this issue as well.


feat(ui-shell): replace unset with property defaults for IE11

Closes carbon-design-system/carbon-website#958.
This needs done elsewhere, documented:

https://github.com/IBM/carbon-components/issues/1761
https://github.com/IBM/carbon-elements/issues/336

One nit, we might want the merge commit to be fix(ui-shell) versus feat

@joshblack good catch with typo in commit message, thanks.


To give a brief explainer, checkpoint loading was only working correctly if you'd saved + loaded a checkpoint previously.
Spent a good few days trying to figure out why, and it turns out in 'load_checkpoint' (not save_checkpoint, of course) megatron set a global variable called 'checkpoint version'.
Checkpoint version controls how some layers are transposed in the self attention block. But since it's only set on load, we never set this global variable until we load a checkpoint and therefore after the very first loading from a checkpoint, the way the last dimension is transposed in the transformer layer is totally different, and it fucks everything up.
 Megatron code big dumb.

Grey is the first checkpoint load before we fix the bug, blue is the first checkpoint load after


sort.go> all functions of Sorters

Which version of Elastic are you using?
[  ] elastic.v3 (for Elasticsearch 2.x)
Please describe the expected behavior
ss := NewScoreSort()
if true {
  ss.Asc()
}
e,  _  := ss.Source()
fmt.Println(e) //map[_score:map[order:asc]]

Please describe the actual behavior
fmt.Println(e) //map[_score:map[order:desc]]

Hi @olivere I resolve this by
ss := NewScoreSort()
if true {
  ss = ss.Asc()
}
e,  _  := ss.Source()
fmt.Println(e) //map[_score:map[order:asc]]

But I think change all functions in sort.go to pointer function is very well.
Example:
// ScoreSort sorts by relevancy score.
type ScoreSort struct {
	Sorter
	ascending bool
}

// NewScoreSort creates a new ScoreSort.
func NewScoreSort() *ScoreSort {
	return &ScoreSort{ascending: false} // Descending by default!
}

// Order defines whether sorting ascending (default) or descending.
func (s *ScoreSort) Order(ascending bool) *ScoreSort {
	s.ascending = ascending
	return s
}

// Asc sets ascending sort order.
func (s *ScoreSort) Asc() *ScoreSort {
	s.ascending = true
	return s
}

// Desc sets descending sort order.
func (s *ScoreSort) Desc() *ScoreSort {
	s.ascending = false
	return s
}

// Source returns the JSON-serializable data.
func (s *ScoreSort) Source() (interface{}, error) {
	source := make(map[string]interface{})
	x := make(map[string]interface{})
	source["_score"] = x
	if s.ascending {
		x["reverse"] = true
	}
	return source, nil
}

Would you mind have a check?  Look forward to your reply. Thanks very much.

You're right, we may want to do this. However, doing so would in elastic.v3 would break the existing API. I might change that in elastic.v5. Thanks for reporting!

Changed sorters in Elastic v5 (release-branch.v5) to pointer receivers.


File.Exists returns true, but File.Encrypt throws IOException: Element not found

File.Exists returns true, but File.Encrypt throws IOException: Element not found
Due to increased security criteria in my application I need to encrypt multiple files on startup. However, It seems to be impossible, and exception message does not indicate actual reason - on the contrary, it seems to be plain wrong.
General
In Program.cs during application startup following method is called:
private static void EncryptAppSettings()
{
    var path = Path.Combine(AppContext.BaseDirectory, "appsettings.json");
    if (File.Exists(path))
    {
        File.Encrypt(path); // Unhandled exception. System.IO.IOException: Element not found. : 'P:\Balsam\appsettings.json'
    }
}

This method works fine on dev machine (Windows 10 Enterprise Microsoft Visual Studio Professional 2019 Version 16.6.3). However, after deploying on Windows Server 2016 Datacenter application crashes on startup. After retrieving crash info from Event Viewer, following exception was found:
Application: Balsam.Front.exe
CoreCLR Version: 4.700.20.41105
.NET Core Version: 3.1.8
Description: The process was terminated due to an unhandled exception.
Exception Info: System.IO.IOException: Element not found. : 'P:\Balsam\appsettings.json'
   at System.IO.FileSystem.ThrowExceptionEncryptDecryptFail(String fullPath)
   at System.IO.FileSystem.Encrypt(String path)
   at System.IO.File.Encrypt(String path)
   at Balsam.Front.Program.EncryptAppSettings() in P:\JENKINS\x\Balsam.Front\Program.cs:line 40
   at Balsam.Front.Program.Main(String[] args) in P:\JENKINS\x\Balsam.Front\Program.cs:line 16

The file exists on specified path, and user account is in IIS_IUSRS group. IIS_IUSRS has every permission, Full Control included, on whole application folder.
I think that either this should work, or exception message should include more data about what is exact reason.

The error message is the Win32 error message from the Window API EncryptFileW function. It basically does the same as setting the encrypt attribute on the file. What happens if you check that manually in File Explorer?

The error message is the Win32 error message from the Window API EncryptFileW function. It basically does the same as setting the encrypt attribute on the file. What happens if you check that manually in File Explorer?

It fails with exactly same error message. Well, it seems it is not problem with .NET Core itself, but with OS Configuration. I will check it with Sysadmins. Thank you for help


Additional retryable error classifications

When diagnosing NRAO failures, observed a few failures that could be re-classified as retryable:
failed upload to ap1-origin.facility.path-cc.io:1095: dial tcp <IP_ADDRESS>:1095: connect: connection refused

The above could be a sign of a server that is simply offline; however, there's really nothing a user can do besides retry.
All of the below are various symptoms of a network connectivity issue:
failed upload to ap1-origin.facility.path-cc.io:1095: read tcp <IP_ADDRESS>:38264-><IP_ADDRESS>:1095: read: connection reset by peer
failed upload to ap1-origin.facility.path-cc.io:1095: write tcp <IP_ADDRESS>:35088-><IP_ADDRESS>:1095: write: connection reset by peer
failed upload to ap1-origin.facility.path-cc.io:1095: write tcp <IP_ADDRESS>:43902-><IP_ADDRESS>:1095: write: broken pipe

In this case, it was a server that was restarted.  These are almost always going to be a transient state -- if the server crashed and stays offline, then it should give a "connection refused" on the retry.

We decided not to make "connection refused" a returnable error after discussion with Brian.


ACME Client can't validate with override DNS ip

Do you want to request a feature or report a bug?
Bug
What did you do?
acme dns challenge.
Traefik's acme client can't validate my domain when I override the ip on a machine on my local network
More precisely, I have an A machine. On this machine there's Traefik with services.
When I try to connect to this machine from the internet. the DNS A of my domain will redirect to my wan IP address. necessarily to be able to access it from the internet. However, when I am on my local network, I asked my router to router for my domain's IP address to my local network's IP address. To avoid using the internet when I want to consult the domain. That way everything can pass only over LAN. Unfortunately traefik's acme client doesn't seem to like it too much when I override the domain ip. So he loses the area, I think.
I want to clarify that when I don't override the ip, everything works very well (thanks traefik :) )
What did you expect to see?
Get the certificate like when i didn't override my DNS
What did you see instead?
I expressly replaced my domain by SECRETDOMAIN
traefik    | time="2018-07-16T13:09:05Z" level=error msg="Unable to obtain ACME certificate for domains \"*.SECRETDOMAIN.net\" : cannot obtain certificates: acme: Error -> One or more domains had a problem:\n[SECRETDOMAIN.net] Error presenting token: Zone net. not found in CloudFlare for domain _acme-challenge.SECRETDOMAIN.net.\n"

Output of traefik version: (What version of Traefik are you using?)
Version:      v1.6.5
Codename:     tetedemoine
Go version:   go1.10.3
Built:        2018-07-10_03:54:03PM
OS/Arch:      linux/amd64

What is your environment & configuration (arguments, toml, provider, platform, ...)?
defaultEntryPoints = ["https","http"]

[entryPoints]
  [entryPoints.http]
  address = ":80"
    [entryPoints.http.redirect]
    entryPoint = "https"
  [entryPoints.https]
  address = ":443"
  [entryPoints.https.tls]

[retry]

[docker]
endpoint = "unix:///var/run/docker.sock"
domain = "SECRETDOMAIN.net"
watch = true
exposedByDefault = true

[acme]
email =<EMAIL_ADDRESS>storage = "acme.json"
caServer = "https://acme-staging-v02.api.letsencrypt.org/directory"
entryPoint = "https"
  [acme.dnsChallenge]
  provider = "cloudflare"
  delayBeforeCheck = 0

[[acme.domains]]
  main = "*.SECRETDOMAIN.net"

Hello @trustfather The ACME client uses DNS resolution to generate the challenge request. If the local DNS resolution does not match what LE resolves, the challenge fails.
However, the feature request in #3585 should resolve the issue. I will mark this as a duplicate, since the same feature will work here.

Yeah, I think it should work. However I do not understand why traefik would need the DNS to do the challenge. It's on the Let's Encrypt side, right? Thank you @dtomcej

Related to https://github.com/xenolf/lego/issues/481

Alright so i prefer to use another DNS servers


Install via choco fails

In a Windows 10 64 bit machine with pristine conditions WRT to manim.
I mean a) no Python version at all. b) Pending LaTeX installation as per Readthedocs directions.
Issuing
choco install manim

fails.
The error message in the Powershell shows that the script attempted to call PIP but the installation of Python as requirement doesn't install pip.
I don't believe the massive log needs to be attached, but if this becomes non reproduceble by the dev team. I'll add the required reports, etc.

I definitely need those error logs that to in debug mode, -debug -verbose with parameters. Make sure you remove personally identifiable details from that. Possibly a gist would help instead of pasting here.

The error message in the Powershell shows that the script attempted to call PIP but the installation of Python as requirement doesn't install pip.

You are wrong here. It's windows and not Linux. The default python installer includes pip by default and chocolatey uses that.  It must be the case I think that you had a python installation previously that didn't have pip. If that's the case you would have to manually install pip or reinstall python and then run chocolatey.
Ref:

https://chocolatey.org/packages/python3#notes
https://docs.python.org/3/using/windows.html#installation-steps

You're absolutely right about « It's windows and not Linux. » ‼
I don't have an account in GitLab to post a gist, is it possible to attach here zipped?
Also, another friend attempted to install, and his results are as dismaying as mine:
This the capture of the cmd shell (notice the need to elevate rights):
16:31 30/09/2020

C:\WINDOWS\system32>cd \Users\manimuser\

C:\Users\manimuser>ver

Microsoft Windows [versão 10.0.19041.508]

C:\Users\manimuser>choco install manim
Chocolatey v0.10.15
Installing the following packages:
manim
By installing you accept licenses for the packages.
Progress: Downloading manim 1.0... 100%

sox.portable v14.4.1
sox.portable package files install completed. Performing other installation steps.
The package sox.portable wants to run 'chocolateyInstall.ps1'.
Note: If you don't run this script, the installation will fail.
Note: To confirm automatically next time, use '-y' or consider:
choco feature enable -n allowGlobalConfirmation
Do you want to run the script?([Y]es/[A]ll - yes to all/[N]o/[P]rint): A

WARNING: Write-ChocolateySuccess is deprecated and will be removed in v2. If you are the maintainer, please remove it from your package file.
 ShimGen has successfully created a shim for play.exe
 ShimGen has successfully created a shim for rec.exe
 ShimGen has successfully created a shim for sox.exe
 ShimGen has successfully created a shim for soxi.exe
 The install of sox.portable was successful.
  Software install location not explicitly set, could be in package or
  default install location if installer.

ffmpeg v<IP_ADDRESS>00831 [Approved]
ffmpeg package files install completed. Performing other installation steps.
Extracting 64-bit C:\ProgramData\chocolatey\lib\ffmpeg\tools\ffmpeg-4.3.1-win64-static_x64.zip to C:\ProgramData\chocolatey\lib\ffmpeg\tools...
C:\ProgramData\chocolatey\lib\ffmpeg\tools
ERROR: The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: Access to the path 'C:\ProgramData\chocolatey\lib\ffmpeg\tools\ffmpeg-4.3.1-win64-static' is denied.
The install of ffmpeg was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\ffmpeg\tools\chocolateyInstall.ps1'.
 See log for details.

An interesting thing to observe is the user of this machine has ImageMagick installed (which has a ffmpeg executable) but chocolatey ignores a system wide existing program. . .
He sent the extract of the log, but not with the switches as you request, so I think I'm not able to report in the detail you require.
I think the discussion you put about the issues in installing Python via chocolatey and the insistence of this tools on creating a subsystem of binaries is enough for me.
I'm abandoning chocolatey and will wait a release for PIP and go from there.
Thanks for your attention, and please close this bug as it seems the environments are too much unstable to be able to create a machine for reproducing thing short of a fresh install of SO, which is not feasible for any useful use of manimce.
Another subtle issue occurs in Windows machines that don't have Python installed Who put Python in the Windows 10 May 2019 Update?, and since the only way I could go back to this state in the machine would be restoring a "restore point", which is also unfeasible, at least for me right now, this issue would have to find another "guinea pig" machine to test this possible interaction

I noticed I can myself close this issue.
I leave it as non reproducible or hard to reproduce...

From the logs I could infer two things

You haven't elevated the shell I think. This could be the possible issue.
The install fail for ffmpeg which is not under my control. So, you should be contacting ffmpeg's maintainers here

Also, that python update includes pip as mentioned in the article. So there is no reason to change the current installer.

What does "elevate the shell" mean, and how would they go about doing it?

"elevate the shell" is similar to giving sudo permissions to linux terminal commands. Here, users would be doing something like the below post
https://www.howtogeek.com/194041/how-to-open-the-command-prompt-as-administrator-in-windows-8.1/
Chocolatey pretty much all the run in elevated shell or it gives warning.


Importing createMiddleware() doesn't work as the tutorial suggests in type:module packages

Description
In a type:module package, the suggested way to import createMiddleware() doesn't work.
Workaround1: Remove the type:module line of your package.json
Workaround2: Change the suggested middleware.js file to something like
import middleware from "next-intl/middleware";

const createMiddleware = middleware.default;

export default createMiddleware({...});

Verifications

[X] I've verified that the problem I'm experiencing isn't covered in the docs.
[X] I've searched for similar, existing issues on GitHub and Stack Overflow.
[X] I've compared my app to a working example to look for differences.

Mandatory reproduction URL
https://github.com/DamienCassou/next-intl-issue-createMiddleware
Reproduction description
Steps to reproduce:

clone the reproduction project
type yarn
type yarn run dev
open http://localhost:3000

Actual behavior:
TypeError: next_intl_middleware__WEBPACK_IMPORTED_MODULE_0__ is not a function

src/middleware.js (3:1) @ <unknown>

  1 | import createMiddleware from 'next-intl/middleware';
  2 |
> 3 | export default createMiddleware({
    | ^
  4 |   // A list of all locales that are supported
  5 |   locales: ['en', 'de'],
  6 |

Expected behaviour
I see the content of Page.jsx in English.

Thank you for the link. Feel free to close this issue if it doesn't bring any more value.

Definitely fine to keep it for now as it's a bug. Also, thank you for posting your workaround, maybe other users will run into this too!

This has been resolved in the next-intl 4.0 beta!


(v3.x) Errors when lazy-loading

I followed this explanation on how to lazy-load lsp-zero (v3.x) with lazy.nvim. When I'm using the latest stable version (0.9.2 when I'm writing this), I get the flashing error on startup (see the video) and everything works fine after that. When I use nightly:
NVIM v0.10.0-dev-1031+gc431d820e
Build type: RelWithDebInfo
LuaJIT 2.1.1693350652

Neovim either displays an error (see screenshot) or more commonly crashes instantly (Error: Process exited with error code 139).

I apologize if this was not the right place to open an issue.

I've updated the dev-v3 branch to solve a few issues with lazy-loading. I've also updated the guide in your link. Try updating lsp-zero and update your lazy.nvim config with the new settings.

lsp-zero doesn't have an event property like nvim-lspconfig or nvim-cmp. By adding lazy = true to lsp-zero it will be loaded the first time require('lsp-zero') is executed. This way you don't have to declare it as a dependency in nvim-lspconfig and nvim-cmp.
Mason.nvim has lazy = false to make sure is loaded on startup. With mason.nvim the order of execution matters, it needs to be configured before you use any of its tools. So its just a way to prevent errors if its loaded too late.
One last thing, the stable branch for version 3 was created. You can now change the dev-v3 branch with v3.x.

Nice, thanks!


Added support STM32CubeMX v4.20.0

Using  file .cproject.
Added auto find file  system. (Support source, include and startup) .
Added auto symbol definitions.

Hi,
I, probably, won't merge this directly, because I'm in progress with the completely new code, but you provided me with the good idea of using .cproject file, so you'll be mentioned.
Anyways, thank you!

After reviewing your code and it's results I have a thing to mention: your code produces CMakeLists.txt with hardcoded lists of files. It doesn't make any use of GLOB_RECURSE. I will fix that and integrate the code in my working branch. Also, I'd like to recommend you using logging library.


ISSUE: Manual Install help?

Not sure if this is the best way to get support. If there are other channels please let me know. I dont have hacs and was wondering for manual install if i just custom_components/keymaster dir to ha_conf_dir/custom_compoents?
I think that is how i should set it up, but am not 100% sure. Dont run a lot of custom integrations.

for manual install if i just custom_components/keymaster dir to ha_conf_dir/custom_compoents?

Yes

If there are other channels please let me know.

There's an extensive thread on the HA forums.

Thanks, i should have paid more attention to the info. I jumped straight to install.
Would it be ok to submit a PR to add appropriate info to the install section of the wiki?

I cant quite figure out how to send a PR to wiki repo. This was the change I was thinking
diff --git a/Install-the-integration.md b/Install-the-integration.md
index 2a361a5..712fc81 100644
--- a/Install-the-integration.md
+++ b/Install-the-integration.md
@@ -2,6 +2,8 @@

 This integration can be added to your installation manually, but the *supported* method requires you to use The [Home Assistant Community Store](https
://community.home-assistant.io/t/custom-component-hacs/121727).  If you dont't already have HACS, it is [simple to install](https://hacs.xyz/docs/insta
llation/prerequisites).

+For manual install you need to copy directory `custom_components` or `custom_components/keymaster` to your Home Assistant config dir so that
 the following path `HOME_ASSISTANT_CONF_DIR/custom_components/keymaster` exits. Now restart Home Assistant so that integration is loaded.
+
 Open HACS and select the Integrations tab.  Click the + icon at the bottom right and search for `keymaster`.  You will then get a message that the int
egration requires Home Assistant to be restarted.  Please do so.

 You need to create an integration for each lock you want to control.  Select Configuration | Integrations.  Click the + icon at the bottom right and s
earch for **keymaster** and select it.
@@ -53,4 +55,4 @@ This sensor returns the command number just executed by the lock.  Schlage value

 #### 9.  **Path to packages directory**

-The default `packages/keymaster` should suffice.
\ No newline at end of file
+The default `packages/keymaster` should suffice.

HACS is the perfered method as it 1) easier to use and 2) makes updating simple with handy notifications when updates are published.

Thank you! I'll update the link.


Wondering if there will be support for the mac M2 chip

+1

came here wondering the same thing, incredulous that nobody has gotten it working already even though it's been out for so long
I guess I've been too online, since it's only been out for one day

I did some changes as workaround to get the Step 1 and Step 2 running on my Mac with M1 with a pretty good performance.
What I did:

Cloned main
On gradio_app.py changed rng = torch.Generator(device=memory_management.gpu).manual_seed(int(seed)) to use device=memory_management.mps
On memory_management.py I...
3.1) replaced cpu = torch.device('cpu') with mps = torch.device('mps')
3.2) changed torch.zeros((1, 1)).to(gpu, torch.float32) to use mps instead of gpu
3.3) modified load_models_to_gpu function to:

def load_models_to_gpu(models):
    global models_in_gpu

    if not isinstance(models, (tuple, list)):
        models = [models]

    models_to_remain = [m for m in set(models) if m in models_in_gpu]
    models_to_load = [m for m in set(models) if m not in models_in_gpu]
    models_to_unload = [m for m in set(models_in_gpu) if m not in models_to_remain]

    if not high_vram:
        for m in models_to_unload:
            with movable_bnb_model(m):
                m.to(mps)
            print('Unload to MPS:', m.__class__.__name__)
        models_in_gpu = models_to_remain

    for m in models:
        m.to(mps)  #
        print('Load to MPS:', m.__class__.__name__)

    models_in_gpu = list(set(models_in_gpu + models))
    torch.cuda.empty_cache()
    return

I know step 3.3 is quite dirty, but it does the trick for me.


Add release announcement workflwo

This adds a new workflow to test the release announcement workflow once in a
production state sending to the Ansible-Talk channel.
Fixes #14

I tested the current setup with my repo mrolli/testytestness, which lead to the latest message card in our Webhook Test teams channel.


Memory leak in Revwalk?

<EMAIL_ADDRESS><EMAIL_ADDRESS>Step to reproduce:
I simply loop the following part
walker = repo.createRevWalk();
walker.push(firstCommitOnMaster.sha());
walker.sorting(nodegit.Revwalk.SORT.Time);
walker.fileHistoryWalk(historyFile, 500)
      .then(()=>walker.free()); // I added this line but still didn't help

in the walk-history-for-file example 3000 times in async in node repl and saw the memory usage of node process built up.
Neither calling global.gc(), Revwalk#free nor Repository#free made any different.

Have the same problem here.
RSS Memory jumps in 500MB for every file and never collected!

We have the same issue but we also cleanup and free the repo and free the commit

Were you able to resolve it? I see RSS memory is increasing and never being released during the repo commits walk.


Roll Flutter from 6656356bbf23 to ae6ace3f69cc (10 revisions)

https://github.com/flutter/flutter/compare/6656356bbf23...ae6ace3f69cc
2022-02-11<EMAIL_ADDRESS>[integration_test] Fix official example for Android (flutter/flutter#98162)
2022-02-11<EMAIL_ADDRESS>Reland "[EditableText] honor the "brieflyShowPassword" system setting #97769 " (flutter/flutter#98150)
2022-02-11<EMAIL_ADDRESS>Fix flutter gen-l10n help message (flutter/flutter#98147)
2022-02-11<EMAIL_ADDRESS>Update Xcode version to 13a233 (flutter/flutter#98144)
2022-02-11<EMAIL_ADDRESS>[flutter_tools] Make variants of Pub have consistent method signatures (flutter/flutter#98119)
2022-02-11<EMAIL_ADDRESS>Add keyLog and connectionFactory to HttpClient implementations (flutter/flutter#98045)
2022-02-11<EMAIL_ADDRESS>Update Material tokens to v0.81. (flutter/flutter#98149)
2022-02-11<EMAIL_ADDRESS>Fix NavigationRail Indicator alignment for NavigationRailLabelType.none (flutter/flutter#98028)
2022-02-11<EMAIL_ADDRESS>[framework] use platform touchslop on Android (flutter/flutter#97971)
2022-02-11<EMAIL_ADDRESS>bed591d39 Roll Skia from 3e6e29cf2e1b to a424b619bc40 (3 revisions) (flutter/engine#31349) (flutter/flutter#98142)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Improve Archive Region Calculation (#211)

Previously, GCToolkit recognized archive regions but did not use this data in its calculations. This change improves the accuracy of tenured generation metrics for G1GCPause events.

Syntactic changes LGTM - @kcpeppe or @dsgrieve to confirm semantic changes. I'm also curious as to whether this change should have triggered a test failure or if we're missing coverage.

I can implement some test cases for this PR

@kcpeppe and I had some discussion about this. We want to verify whether archive regions are counted as old gen regions which may mean that these regions are being accounted for twice.

@kkdlau - I found "OpenArchive" and "CloseArchive" as region types in jdk-11 and jdk-17. But I don't see an archive region type in jdk-8 or jdk-21. The only evidence I see of "Archive" appearing in a log trace is in the remembered set stats, but only in jdk-17.
Do you have a log file with the pattern "Archive regions:"?

@dsgrieve one of examples I found from <repo root>/gclogs/rolling/jdk14/rollinglogs/short_restart_4.log.1 line 106:

[15.034s][info ][gc,heap        ] GC(0) Archive regions: 2->2

But it is quite surprising to know that the way JDK handles the archive region is changing over generations. I will also look at OpenJDK's G1CollectedHeap implementation to understand more about the issue.

@dsgrieve I just walked through OpenJDK and found that Archive Regions are only available in JDK 14 and 17.
JDK 14: https://github.com/openjdk/jdk/blob/ae39310243b0486f5a6f1049c6ec5f29db31170c/src/hotspot/share/gc/g1/g1HeapTransition.cpp#L174-L177
JDK 17: https://github.com/openjdk/jdk/blob/dfacda488bfbe2e11e8d607a6d08527710286982/src/hotspot/share/gc/g1/g1HeapTransition.cpp#L175-L178
  log_info(gc, heap)("Archive regions: " SIZE_FORMAT "->" SIZE_FORMAT,
                     _before._archive_length, after._archive_length);
  log_trace(gc, heap)(" Used: " SIZE_FORMAT "K, Waste: " SIZE_FORMAT "K",
      usage._archive_used / K, ((after._archive_length * HeapRegion::GrainBytes) - usage._archive_used) / K);

JDK 8 doesn't have Archive Region type:
https://github.com/openjdk/jdk/blob/9a9add8825a040565051a09010b29b099c2e7d49/hotspot/src/share/vm/gc_implementation/g1/g1HRPrinter.hpp#L48-L56
  typedef enum {
    Unset,
    Eden,
    Survivor,
    Old,
    SingleHumongous,
    StartsHumongous,
    ContinuesHumongous
  } RegionType;

When I look at how the JVM collects region info, archive regions are not counted as old regions:
https://github.com/openjdk/jdk/blob/dfacda488bfbe2e11e8d607a6d08527710286982/src/hotspot/share/gc/g1/g1HeapTransition.cpp#L89-L94
    if (r->is_old()) {
      _usage._old_used += r->used();
      _usage._old_region_count++;
    } else if (r->is_archive()) {
      _usage._archive_used += r->used();
      _usage._archive_region_count++;

I think, even though archive regions are only available in JDK 14 and 17, we can still subtract the numbers from the total. Since the size of archive regions is 0 in other versions, there is no double counting for old regions and archive regions.


how to move the drawSvg.elements integrally

Discussed in https://github.com/cduck/drawSvg/discussions/72

Originally posted by lythhh April  9, 2022
I make a simple drawSvg.elements(a triangle), what i want to do is to move the triangle a certain distance, and finally form a series of ordered triangles. So i want to konw is there any function can help me? Thank you very much
the code is as follows:
import drawSvg as draw
def mm2pt(x):
return 2.835x
dw=mm2pt(3)
dh=mm2pt(3)
d = draw.Drawing(dw, dh, origin=(0, 0), displayInline=False)
r = draw.Lines(dw/4,dh/4,dw/43,dh/4,dw/2,dh/4*3,close=True,fill='none',stroke='black',stroke_width=mm2pt(0.1))
r.appendTitle("角砾")
d.append(r)
d.setPixelScale(5)
d
#def move(drawSvg,w,h): #i want to construct a function with a drawSvg object and certain distance on x and y, to realize the movement of drawSvg objects.

Please do not double-post.  GitHub issues are to let me know potential problems with the library and discussions are for questions.


Statistical code value set

Define a ValueSet for which code shall be mapped for the care communication and hospitalnotification

Value set is merged on release branch

Value set is merged on release branch


drivers: haptics: Fix unchecked return value in drv2605

Fix unchecked return value scanned by Coverity.
Closes: #81983

Hello, thanks for your contribution. However I do not want to duplicate the efforts being made on #82682. I should have linked that PR to the original issue so we all know it is being handled.

I will leave this issue to others. Thank you for your reminder.


perf: revert workaround for PySet memleak

Includes other changes:

scoring disabled for the stat collector
Added unique_docs_frames_unzipped to return (Vec, Vec)
Removed dead code
Improved API to be more fieldname agnostic

NOTE: THIS WILL REQUIRE CHANGES IN CHRYSALIS CODE TO USE.

@CodiumAI-Agent can you review this change providing any suggestions, as well as optimizations, taking into consideration especially the comments already on this pull request

Preparing answer...

Question: can you review this change providing any suggestions, as well as optimizations, taking into consideration especially the comments already on this pull request
Answer:
This PR includes several changes, mainly focused on performance improvements and code cleanup. Here are some observations and suggestions:

Scoring Disabled for the Stat Collector: The change from true to false in the requires_scoring function of the StatsCollector struct is a significant one. It means that the scoring feature is disabled for the stat collector. If scoring is not necessary for your use case, this change could improve performance. However, if scoring is required in the future, you might need to revert this change.

Removal of Dead Code: The removal of the Fruit enum and its related code is a good step towards cleaning up the codebase. If this code is not used anywhere, removing it makes the codebase cleaner and easier to maintain.

Change in Return Types: The return types of unique_docs, unique_frames, and unique_docs_frames have been changed from Vec to BTreeSet. This change could improve performance as BTreeSet ensures that the elements are sorted and duplicates are removed. However, it's important to note that the order of insertion is not preserved in a BTreeSet.

Addition of unique_docs_frames_unzipped Method: This new method returns a tuple of two vectors, which is an optimization for Python callers to obtain vectors without having to iterate over them. This is a good addition, but it would be better to add some comments explaining why this method is needed and how it improves performance.

Refactoring of the search Method: The search method has been refactored to take optional parameters for filter_fastfield_name and filter_fastfield_values. This change makes the method more flexible. However, it's important to handle the case where filter_fastfield_values is provided but filter_fastfield_name is not. The current code does handle this case, but it would be better to add a comment explaining why this check is necessary.

Test Updates: The tests have been updated to reflect the changes in the code. It's good to see that tests are being updated along with the code changes. However, it would be better to add some new tests to cover the new unique_docs_frames_unzipped method and the refactored search method.

Overall, this PR includes some good changes that could improve performance and clean up the codebase. However, it would be beneficial to add more comments explaining the reasoning behind some of the changes and to add more tests to cover the new and refactored code.

@cjrh did you review the upcoming changes in 0.20.0 for fastfields https://github.com/quickwit-oss/tantivy/releases/tag/0.20 and whether this change will work with that?


Reset Page Orientation on Navigation

Currently, when a page is taller than a device's viewport, if a user navigates to another page (i.e. home to campaign, etc.) the viewport will remain at the coordinates of the previous page.
You may want to learn about the DOM window property.

https://www.geeksforgeeks.org/html-dom-window-document-property/
https://renatello.com/vue-js-scroll-top/

To fix:

[ ] Write a function to reset each page before render.
[ ] Test that function if necessary

Do we have example screenshots of this issue being reproduced? I want to make sure I understand what the issue is and haven't been able to reproduce on my machine.

@DietBepis1 Do we have example screenshots of this issue being reproduced? I want to make sure I understand what the issue is and haven't been able to reproduce on my machine.

@smgraywood Until Glenn responds, you can try reproducing it in the following manner:

Scroll down while on the home page.
Click a campaign.
Notice the scroll location from the home page is retained (which is what we want to avoid).

Thank you @paramsiddharth , that helped me a lot! Ok, I submitted a fix: https://github.com/ProgramEquity/amplify/pull/480

This issue has been resolved: https://github.com/ProgramEquity/amplify/pull/480


xdp: Implement xdp action test cases

See details in commits.

I ran all xdp test with 'canonical 0001-com-ubuntu-server-focal 20_04-lts-gen2 latest', 2 failed, 1 skipped, 2 passed

I ran all xdp test with 'canonical 0001-com-ubuntu-server-focal 20_04-lts-gen2 latest', 2 failed, 1 skipped, 2 passed

I had a test, 1 skipped and 4 passed. Can you show me the log, if it's failed? The skipped test case needs more time to investigate. Will fix it later.


Tags and very long search inputs

When I do have some Tags in my multiselect and then add a long text, the tags are being pushed out of the visible area. Instead I would expect the input to change to a textarea and if necessary force a break into the next line.
I do consider this more an improvement than a bug, as this is probably a rare occurrence out in the wild
Following a collage of what can be observed:

entering a long text

tags are being pushed out

tags are now out of sight

clicking outside of the span. Notice how the height remains at 2 columns and the tags are still cut off to the left

upon clicking in the tags are still hidden

only when typing again, everything is rendered again and displayed correctly

Hello. I have an identical problem that you can try on Select2 demos.
When I use the ability to insert more than one option by text inserted mannualy, if that text lenght is bigger than HALF of the width of the select2 , after inserting (clicking ENTER), it goes to a new line, making my entry ''disappear to upside". If i click on any char key it starts the input after my already entered text in the same line (so it revert the line break).
So i'm using the ability to insert ONLY one option manually, and that option sometimes will be longer than half of the width. How can I make it to do not create a line break after entering my entry only if it's bigger than half of the width?


Account tile balance not updating(within app session)

The balance on the account tiles of the home screen isn't updating after the account refresh.
Here's the scenario:

Went into the Checking account, got a receive address from the receive screen, and made a tx from a test faucet.
Refreshed the Checking account and the tx appeared and the balance was also properly updated.

However, when I got back to the home screen(by pressing the back button on the checking account's screen), the checking account tile was still showing the previous balance. It didn't update.

It is only after logging in again into the app that the home tile showed the correct balance

This issue got resolved as a part of the following issue: https://github.com/bithyve/hexa/issues/2448

This issue got resolved as a part of the following issue: https://github.com/bithyve/hexa/issues/2448


[BUG] 'Package' does not contain a definition for 'targetFramework'

Describe the bug
Well. A bug
Library\PackageCache\org.kumas.nuget-importer@e006d74628\Editor\PackageOperation\OperatePackage.cs(251,112): error CS1061: 'Package' does not contain a definition for 'targetFramework' and no accessible extension method 'targetFramework' accepting a first argument of type 'Package' could be found (are you missing a using directive or an assembly reference?)

I won't open a pr that I've removed ZIP_AVAILABLE in my fork


breadcrumb: react 16

desc yo

Warnings

:warning:
:pencil2: Please add a description.

  Generated by :no_entry_sign: dangerJS against 275788707ce97465189916810c793d988652749e


Refactor CourseCreationDossier to CourseCreationRequest

Rename CourseCreationDossier to CourseCreationRequest, and the removal of creation of dossier in the InitiateCourseCreation() function, which led to some logic changes in the dbcontext, controller, service, and repository.
This PR closes #56.

Looks good overall, nice job, just some concerns to address.

Also, role-based auth is implemented, so the method should be limited to only users with the Initiator role. It's a 1 line change, just add the annotation above the method, see: https://github.com/JamesPartsafas/ConcordiaCurriculumManager/blob/main/Server/ConcordiaCurriculumManager/Controllers/DossierController.cs#L63

Also, role-based auth is implemented, so the method should be limited to only users with the Initiator role. It's a 1 line change, just add the annotation above the method, see: https://github.com/JamesPartsafas/ConcordiaCurriculumManager/blob/main/Server/ConcordiaCurriculumManager/Controllers/DossierController.cs#L63

Someone already added it for the controller method InitiateCourseCreation.


the backup file for ifcfg-eth0 contains "dhcp" and that lets the benc…

the backup file for ifcfg-eth0 contains "dhcp" and that lets the benchmark fail.

Making the auditors happy by dropping a backup file of /etc/sysconfig/network-scripts/ifcfg-eth0.


How to make more complex queries?

Hi, I want to completely change the structure of my database (mongodb), so the four methods provided by the API will not suffice. How do I use the mongodb API from within migrations?
I am currently stuck with this:
exports.up = (db, callback) ->
    db.connection._db.collection('users').find().toArray((err, arr) ->
        console.log err
        console.log arr
        callback()
     )

Which yields:
{ [MongoError: Connection Closed By Application] name: 'MongoError' }
null

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11880648-how-to-make-more-complex-queries?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github).

This is currently not possible, requests are encapsulated by the _run method which fistly builds the connection. I have planned anyway to make a bit of rework on some drivers, including this one.
So as we're on this topic I can ask this question, I've forgot to ask so often.
@toymachiner62 Could you tell me why exactly you establish the connection not when loading the driver and instead open a new connection for every _run call?
Does this have a specific reason that you choosed this design?

If i remember correctly I was having a helluva time with the connection stuff b/c it's different than most other sql connections. I can't remember the specifics off the top of my head though..
Either that or I was being dense at the time...

@toymachiner62 Ok thanks for your response :)
Then I will check out the details about the connection and see what happens if there is a persisten connection.
@thokari I will schedule this at a later time, I will invest more time in May again and hope to finish the v0.10.x. Either I will schedule this for the v0.10.x release or going to put into the next release, depends on how fast next month everything moves forward.


remove invalid flag 'o'

free command has no flag 'o' and it keeps throwing error under django log.

Thanks, but this was fixed in #47 .


GEOMESA-1403 Fix GeoMesa Command Line tools docs typos and formatting…

… issues.
Fixed some inconsistent formatting and typos that prevented the examples from working.
Signed-off-by: Austin Heyne<EMAIL_ADDRESS>

Apart from the crossref, looks good to me.

I've updated the cross ref.

+1

Looks good to me.


Vertex shader function could not be compiled into pipeline

When trying to run Mortal Kombat XL on MacOS running Wineskin/crossover/MoltenVK I get the following errors.
[mvk-error] VK_ERROR_INITIALIZATION_FAILED: Shader library compile failed (Error code 3):
Compilation failed:

program_source:473:9: error: no member named 'gl_CullDistance' in 'main0_out'
    out.gl_CullDistance[0] = out.o5;
    ~~~ ^
.
[mvk-error] VK_ERROR_INVALID_SHADER_NV: Vertex shader function could not be compiled into pipeline. See previous logged error.
err:   DxvkGraphicsPipeline: Failed to compile pipeline
err:     vs  : VS_48c64a754f51018d85d9719a2c5751764481a5d6
err:     fs  : FS_bae6d9d5b407ddd219b7dcaeb457942a161fc78e
[mvk-info] Converting SPIR-V:

and
[mvk-error] VK_ERROR_INITIALIZATION_FAILED: Shader library compile failed (Error code 3):
Compilation failed:

program_source:686:9: error: no member named 'gl_CullDistance' in 'main0_out'; did you mean 'gl_ClipDistance'?
    out.gl_CullDistance[0] = out.o0.z;
        ^~~~~~~~~~~~~~~
        gl_ClipDistance
program_source:114:11: note: 'gl_ClipDistance' declared here
    float gl_ClipDistance [[clip_distance]] [2];
          ^

I read that gl_CullDistance is not supported in Metal. https://github.com/KhronosGroup/SPIRV-Cross/issues/1403. If that's the case, what is the work around?
The original downstream issue which contains more logging output is here
Many thanks.

I read that gl_CullDistance is not support in Metal. KhronosGroup/SPIRV-Cross#1403. If that's the case, what is the work around?

That is correct. As is the SPIRV-Cross discussion you quote.
However, it might make sense to modify SPIRV-Cross to at least support compilation and setting an output value, even if it can't be used as an MSL built-in, and will have no actual affect on the rendering behavior.
Can you file this, along with my comments, on the SPIRV-Cross issue list, please?

However, it might make sense to modify SPIRV-Cross to at least support compilation and setting an output value, even if it can't be used as an MSL built-in, and will have no actual affect on the rendering behavior.

Can you file this, along with my comments, on the SPIRV-Cross issue list, please?

Ok many thanks.  Filled https://github.com/KhronosGroup/SPIRV-Cross/issues/1840

Filled KhronosGroup/SPIRV-Cross#1840

Thanks. Closing this here.


feat(source): add NATS source consumer parameters

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This PR adds the 20 missing parameters supported in the async_nats crate of the consumer created in NATS sources. The DDL of a NATS source would now look like:
CREATE SOURCE (
...
) WITH (
    connector='nats',
    server_url='{{ env_var("SERVER") }}',
    subject='risingwave.test.source',
    stream='risingwave-test-source',
    scan.startup.mode='earliest',
    connect_mode='user_and_password',
    username='{{ env_var("USER") }}',
    password='{{ env_var("PASSWORD") }}',
    consumer.durable_name='risingwave-test-source',
    consumer.description='desc-test-source',
    consumer.ack_policy='all',
    consumer.ack_wait=10,
    consumer.max_deliver=10,
    consumer.filter_subjects='demo.subject.filter.*',
    consumer.filter_subjects='demo.subject.filter.1,demo.subject.filter.2',
    consumer.replay_policy='instant',
    consumer.rate_limit=100000000000,
    consumer.sample_frequency=1000000000,
    consumer.max_waiting=10,
    consumer.max_ack_pending=10,
    -- consumer.idle_heartbeat=60, not available in async_nats crate
    consumer.max_batch=1000,
    consumer.max_bytes=1000000000,
    consumer.max_expires=3600,
    consumer.inactive_threshold=10000000,
    consumer.memory_storage='false',
    consumer.backoff='10,30,60',
    consumer.num_replicas=1
) FORMAT PLAIN ENCODE JSON

All parameters are optional. If left empty, it falls back to the async_nats default values. All are eventually parsed as strings in RisingWave and casted to the appropriate types before being passed to the build_consumer method.
Checklist

[x] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Release note
Now, all the parameters supported by the async_nats crate are supported in the RisingWave NATS source connector. In particular, users can now configure few important aspects of consumers such as the number of replicas that previously would default to 1, hereby causing Jetstream issues of synchronization across replicas. It also allows setting parameters such as max_waiting or max_ack_pending that can be critical for performance optimization of NATS clusters.
No backward incompatibility would be introduced by these changes.

Hi. Can you help to rebase the commits first?

Just added a commit to address the comments from previous reviewers! Sorry for coming back late on this 🙏


GH1653: Adding Before and After options

Adding Before and After options for NUnit3Labels enum with tests.
Relates to #1653.

Thanks for putting this in @phillipsj! 😄

@phillipsj are you in a position to rebase this PR?  If not, let me know, and I will do it.

@gep13 I may be able to do it in the morning.
On Mon, Oct 23, 2017, 2:11 AM Gary Ewan Park<EMAIL_ADDRESS>wrote:

@phillipsj https://github.com/phillipsj are you in a position to rebase
this PR? If not, let me know, and I will do it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cake-build/cake/pull/1869#issuecomment-338557148, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABfthRWRhIqJ42YCzQxjVDy5qQzRipgxks5svC4kgaJpZM4Px0US
.

@gep13  rebased.

@phillipsj your changes have been merged, thanks for your contribution 👍


Fares addition (prod)

PR checklist:

[ ] Changes previewed and QC'ed
[ ] In PR, mention team member for FYI, review, or approval

New content (if applicable):

[ ] Date updated
[ ] Tags and keywords updated
[ ] Feature box updated
[ ] Adding the new content to related or relatedData for things it's related to

New data (if applicable)

[ ] Added to Recently Updated
[ ] Added to 311-crosswalk

Thanks! FYI I made a slight code tweak - please take a look. Generally it's best to avoid using  and instead use css classes for margins - in this case, mb-2 (margin-bottom size 2).


HDFS-14455:Fix typo in HAState.java

There are some typo in HAState
destructuve -> destructive
Aleady -> Already
Transtion -> Transition

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
30
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1079
trunk passed

+1
compile
57
trunk passed

+1
checkstyle
43
trunk passed

+1
mvnsite
62
trunk passed

+1
shadedclient
718
branch has no errors when building and testing our client artifacts.

+1
findbugs
119
trunk passed

+1
javadoc
50
trunk passed

_ Patch Compile Tests _

+1
mvninstall
59
the patch passed

+1
compile
54
the patch passed

+1
javac
54
the patch passed

+1
checkstyle
37
the patch passed

+1
mvnsite
57
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
686
patch has no errors when building and testing our client artifacts.

+1
findbugs
127
the patch passed

+1
javadoc
49
the patch passed

_ Other Tests _

-1
unit
4923
hadoop-hdfs in the patch failed.

+1
asflicense
30
The patch does not generate ASF License warnings.

8242

Reason
Tests

Failed junit tests
hadoop.hdfs.TestErasureCodingExerciseAPIs

hadoop.hdfs.TestUnsetAndChangeDirectoryEcPolicy

hadoop.hdfs.server.datanode.TestDirectoryScanner

hadoop.hdfs.server.namenode.TestAuditLogs

hadoop.hdfs.TestErasureCodingPoliciesWithRandomECPolicy

hadoop.hdfs.server.namenode.TestFSDirectory

hadoop.hdfs.server.namenode.TestFSEditLogLoader

hadoop.hdfs.TestErasureCodingPolicyWithSnapshotWithRandomECPolicy

hadoop.hdfs.server.sps.TestExternalStoragePolicySatisfier

hadoop.hdfs.server.namenode.TestCheckpoint

hadoop.hdfs.server.namenode.TestAuditLoggerWithCommands

hadoop.hdfs.TestEncryptedTransfer

hadoop.hdfs.server.namenode.TestFSNamesystemMBean

hadoop.hdfs.server.namenode.TestNamenodeRetryCache

hadoop.hdfs.server.namenode.sps.TestStoragePolicySatisfierWithStripedFile

hadoop.hdfs.server.namenode.TestListCorruptFileBlocks

hadoop.hdfs.TestErasureCodingMultipleRacks

hadoop.hdfs.server.namenode.metrics.TestNameNodeMetrics

hadoop.hdfs.TestDFSShell

hadoop.hdfs.web.TestWebHdfsTimeouts

hadoop.hdfs.tools.TestDFSAdmin

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-764/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/764

JIRA Issue
HDFS-14455

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux 66072746aba7 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 64f30da

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-764/1/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-764/1/testReport/

Max. process+thread count
3405 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-764/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

+1

LGTM +1(non-binding)

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
0
Docker mode activated.

-1
patch
12
https://github.com/apache/hadoop/pull/764 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/764

JIRA Issue
HDFS-14455

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-764/2/console

versions
git=2.7.4

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
0
Docker mode activated.

-1
patch
11
https://github.com/apache/hadoop/pull/764 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/764

JIRA Issue
HDFS-14455

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-764/3/console

versions
git=2.7.4

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
0
Docker mode activated.

-1
patch
9
https://github.com/apache/hadoop/pull/764 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/764

JIRA Issue
HDFS-14455

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-764/4/console

versions
git=2.17.1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
0
Docker mode activated.

-1
patch
11
https://github.com/apache/hadoop/pull/764 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/764

JIRA Issue
HDFS-14455

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-764/5/console

versions
git=2.17.1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

Duplicates HDFS-14398. Close this one.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
0
Docker mode activated.

-1
patch
14
https://github.com/apache/hadoop/pull/764 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/764

JIRA Issue
HDFS-14455

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-764/6/console

versions
git=2.7.4

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.


[Peer-Bug] @nuxt/kit 3.3.0 breaks nuxt v2.16

Environment
See here: https://github.com/nuxt/nuxt/issues/19717
Reproduction
See here: https://github.com/nuxt/nuxt/issues/19717
Describe the bug
Pinia-ORM depends on Nuxt Kit. Its last version 3.3.0 breaks Nuxt2 with Nuxt-bridge. Pinning to @nuxt/kit 3.2 solves the issue.
The root issue is here: https://github.com/nuxt/nuxt/issues/19717
The work around I've used to fix this was to add this block in package.json:
  "resolutions": {
    "**/@nuxt/kit": "^3.2.0 < 3.3.0"
  }

Additional context
No response
Logs
No response

Thanks for the notice @mackwic. I don't know if i have to do something. I think not because it's not related to pinia-orm/nuxt.
Feedback is welcome if there is something i can change for this.

I agree, I think there's nothing to do on your part. This issue is only
here to point people to a workaround if they stumble upon the same bug
Le lun. 20 mars 2023 à 14:25, Gregor Becker @.***> a
écrit :

Thanks for the notice @mackwic https://github.com/mackwic. I don't know
if i have to do something. I think not because it's not related to
pinia-orm/nuxt.
Feedback is welcome if there is something i can change for this.
—
Reply to this email directly, view it on GitHub
https://github.com/CodeDredd/pinia-orm/issues/1091#issuecomment-1476229317,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEIIC2PJ6ARKQ3AMVSS6MLW5BLDFANCNFSM6AAAAAAWA6F24M
.
You are receiving this because you were mentioned.Message ID:
@.***>


Automated cherry pick of #8253: fix: qcloud eip with bandwidth attr

Cherry pick of #8253 on release/3.3.
#8253: fix: qcloud eip with bandwidth attr

Codecov Report

Merging #8255 into release/3.3 will decrease coverage by 2.70%.
The diff coverage is 0.30%.

@@              Coverage Diff               @@
##           release/3.3   #8255      +/-   ##
==============================================
- Coverage         6.78%   4.08%   -2.71%
==============================================
  Files              676     281     -395
  Lines           118955   59443   -59512
==============================================
- Hits              8076    2429    -5647
+ Misses          110267   56822   -53445
+ Partials           612     192     -420

Flag
Coverage Δ

#aFlag
4.08% <0.30%> (-2.71%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/cloudcommon/notifyclient/notify.go
2.22% <0.00%> (-0.36%)
:arrow_down:

pkg/compute/models/buckets.go
1.10% <0.00%> (-0.07%)
:arrow_down:

pkg/compute/models/capabilities.go
0.00% <0.00%> (ø)

pkg/compute/models/cloudproviderregions.go
0.00% <0.00%> (-0.27%)
:arrow_down:

pkg/compute/models/cloudproviders.go
0.87% <0.00%> (+0.01%)
:arrow_up:

pkg/compute/models/cloudregions.go
1.57% <0.00%> (-0.05%)
:arrow_down:

pkg/compute/models/cloudsync.go
0.00% <0.00%> (ø)

pkg/compute/models/dbinstance_accounts.go
2.05% <0.00%> (-0.03%)
:arrow_down:

pkg/compute/models/dbinstance_backups.go
2.78% <0.00%> (-0.17%)
:arrow_down:

pkg/compute/models/dbinstance_databases.go
3.33% <0.00%> (-0.07%)
:arrow_down:

... and 710 more

Continue to review full report at Codecov.

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

/lgtm
/approve


SPIKE: Figure out demand for Azure

Describe the issue
Investigative work to figure out what our clients want, by scanning existing emailed & other inquiries, getting the info from Jon.  This is to assist Jon with ticket 946.
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context

PRE-WORKSHOP if possible

How does NRM Files fit in this?

Audra has a ticket about who are the existing tenants

Definition of done

Jon to share email info, Sid to ask for them.  Related to the work Jon is doing with Audra about existing clients
Information on who wants what in Azure
Define a set of tickets based on the feedback we get, well-defined, estimated

What is the project?
Bonus is: what Services are they using?


[WIP] fix the issues caused when clean_nodes enabled

when clean_nodes enabled

In tasks/install_os.yml, after the overcloud delete the overcloud nodes are still in cleaning state and then move to available state. In meantime, if badfish triggers power-cycle on these nodes it throws error, So I have added a task to ensure the nodes move to available state
In tasks/delete_introspection_failed_nodes.yml, after the introspection for the nodes to move to available state it takes longer then default deployment, so I have added a wait time of 360 seconds

Do not merge, I still need to test this PR


Implement map smart move in Cesium.

Implement map smart move in Cesium.

Test case:
Use the API tester and enable the smart motion lock mode. Right click the borders of the map and keep the mouse down. The test is a success if the map moves as follows:
right side - moves to the right
left side - move left
top side - move top
bottom side - move bottom
top Left corner - move top left
top right corner - move top right
bottom Left corner - move bottom left
bottom right corner - move bottom right

Reintegrate

Merged into the dev branch. Closing this issue.


Couldn't set zoom extent to 22

Hi everybody,
I know this might be a duplicate question but I wasn't able to find any solution. I'm working on a tool with the help of this package which is actually a platform to convert a geojson file to mbtiles with the help of tippecanoe. The problem is actually not about conversion because it works successfully but the problem is with zoom levels in the Mapbox studio.
I need 22 zoom level which is stated with -z22 parameter. But even though I did state, it automatically defines 15 zoom level.

I'm developing this with JavaScript and my converter currently works with these parameters:
        tippecanoe(
            [inputGeojsonPath],
            {
              "maximum-zoom": 22,
              "full-detail": 10,
              "low-detail": 10,
              simplification: 10,
              layer: "buildings",
              output: outputMbtilesPath,
              description: "Building footprints in the municipality of Hobbiton.",
              force: true,
              "drop-densest-as-needed": true,
              "extend-zooms-if-still-dropping": true,
              pk: true,
              pf: true,
            },
            { echo: true }
          );

Also, command version of it goes like this:
tippecanoe --maximum-zoom=22 --full-detail=10 --low-detail=11 --simplification=10 --layer=buildings --output=/tmp/sample.mbtiles --description='Building footprints in the municipality of Hobbiton.' --force --drop-densest-as-needed --extend-zooms-if-still-dropping -pk -pf /tmp/sample.geojson
Are there any workarounds or suggestions that you can give? Is there anybody that had the same issue and solved it? I really need exact zoom extent level 22.
Thanks in advance.

I don't think Mapbox will let you upload an mbtiles with a maxzoom beyond 16.


HTML

HTML
User Story
As a boot camp student,
I want a Prework Study Guide website,
so that I can review all the information I learned throughout Prework.
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow
WHEN I open the console in my browser
THEN I can see the four topics I learned listed along with a suggestion on what I should study first
WHEN I visit the website using the URL
THEN I can access my website from any browser

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Javascript
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information

README
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

Deployment
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN a Prework Study Guide website,
WHEN I visit the website using the URL,
THEN I can access my website from any browser