XML code comment tooltips error on extended characters (e.g., >)

Describe the bug
XML Code comments that contain extended characters cause "An error occurred when parsing the doc comment" error when hovering over the commented symbol.
Steps to reproduce

Create an F# script file (bugreport.fsx)
Define a code commented function:

///
/// function fn
///
/// <example id="example-1">
///
/// fn (fun i -> i <= 3) 2
///
/// Evaluates to "true"
/// </example>
let fn p x = if p x then "true" else "false"

Hover over the function definition and observe the error tooltip

Workaround
HTML Encode characters such as > as >
///
/// function fn'
///
/// <example id="example-1">
///
/// fn' (fun i -> i <= 3) 2
///
/// Evaluates to "true"
/// </example>
let fn' p x = if p x then "true" else "false"

Expected behaviour
Should correctly render the comments without requiring encoding of characters.
Code comments on the fsharp lang codebase are not encoded to compensate for these types of errors:
https://github.com/dotnet/fsharp/blob/main/src/FSharp.Core/string.fsi
/// <example id="collect-2"> How to show the ASCII representation of a very secret text
///
/// "Secret" |> String.collect (fun chr -> int chr |> sprintf "%d ")
///
/// The sample evaluates to <c>"83 101 99 114 101 116 "</c>
/// </example>

Screenshots

Machine info

OS: Windows 10
.NET SDK version: 7.0.102
Ionide version: v7.5.2

bugreport - rename to fsx.txt

The problem is caused by that specific character <.
You can use > without escaping it.
This is because, we are using an XML parser to parse the doc comment and it detect < as an opening tag.
I see 2 ways of fixing this problem:

Be able to identify < that are not part of the doc comment structure, and escape them before the parser
Implement a doc comment parser which doesn't rely on an XML parser

Unfortunately, neither of these solutions are easy to implements. Last time I checked, other editors had the same limitation as Ionide right now and I decided at the time that it was better to make the user use escaped syntax when having this problem otherwise the doc comment is not useful to the end user.

That all makes sense. Thank you.
Given the use of a standard XML parser, I've changed my code block to be a CDATA section as follows:
///
/// function fn
///
/// <example id="example-1">
/// <![CDATA[
/// fn (fun i -> i <= 3) 2
/// ]]>
/// Evaluates to "true"
/// </example>
let fn p x = if p x then "true" else "false"

This avoids the parsing error and renders a usable tooltip. The only issue is that CDATA enclosing mark-up is also rendered.

Could there be a simpler fix to render the contents of a CDATA section when detected?
If not, I'm happy to see this issue closed anyway as I have a workaround that I'm happy to use.

I don't think we'd support CDATA here given that no where else in the .NET space that I'm aware of supports that syntax. The generally-accepted solution for all .NET languages is to escape the angle bracket when you need it to appear as a literal.


Accept Arbitrary Attribute Values for Loop

WIP - Draft PR to track progress and information
Below is direction from @Demonthos:

Change the key type in the node: https://github.com/DioxusLabs/dioxus/blob/master/packages/core/src/nodes.rs#L51
Make arbitrary attribute Hash here: https://github.com/DioxusLabs/dioxus/blob/master/packages/core/src/nodes.rs#L558
Change the parsing here https://github.com/DioxusLabs/dioxus/blob/6a16df835d232a5565c10f35a9319e8df7907d2d/packages/rsx/src/element.rs#L16 and https://github.com/DioxusLabs/dioxus/blob/6a16df835d232a5565c10f35a9319e8df7907d2d/packages/rsx/src/component.rs#L59

Design Considerations:

arbitrary values should probably not need to implement hash. They are used to allow custom elements to accept other types. We can just panic instead.
f64 does not implement hash. We can either panic or create a custom implementation

The main use case I see for this is raw indexes for arrays as keys
It might be better to create a different type that does not allow custom values and does not include f64. We should be able to reuse most of the code for arbitrary attribute value
The code for arbitrary attributes in core and rsx could be useful here. The main thing they do is create a trait that can convert from a normal attribute to a arbitrary value here: https://github.com/DioxusLabs/dioxus/blob/master/packages/core/src/nodes.rs#L839
Then create attributes here in the macro: https://github.com/DioxusLabs/dioxus/blob/master/packages/rsx/src/element.rs#L248, which accepts anything that can implements that trait here https://github.com/DioxusLabs/dioxus/blob/master/packages/core/src/scopes.rs#L517

I can deal with the hash trait issues tomorrow. You mentioned this in directions, I will take a gander in the morning.

I am away for weekend, but I am bringing laptop. I will try to tackle this when I have down time (hopefully today).

@Demonthos sorry for the delay, I didn't realize I could just run Cargo Build at the root of the core directory. This is my first Open Source Rust PR. I was just using rust analyzer until I relaized that.

Closing to start over


Additional improvements for Arm/RISC-V platforms

The following proposed changes:

Add more supported baud rates to SPCR
(Hopefully) make DBG2 language on NamespaceString clearer.
Add NamespaceString support to SPCR.

Note: these changes do not bump the SPCR revision or modify the revision log.

@barrygolden - The serial port updates look fine to me. Will assign to you for ACPI bring up.


add git_add action

thanks @4brunu, this is awesome! just a couple small things from the comment above and we can :rocket:

Sorry, I forget that.
Should I open another pull request?

thanks for making that change so quickly! one more small thing that was missed.
no need to create another pull request, but if you could squash the commits once you have made that last change that would be great!

thanks for making that last change! finally, would you mind squashing your commits into 1?

Was this that you were asking for?

close, but here are some better instructions, sorry about that i probably should have included this sooner https://davidwalsh.name/squash-commits-git

I think now it's done.
Sorry, but I was having some difficulties squashing the commits.

@ohwutup is anything wrong with the commit now?

yup! this looks great! :rocket:


Report schema validation - please add for source and target datatypes

If I run schema validation i have report with this column:
run_id,validation_name,validation_type,start_time,end_time,source_table_name,target_table_name,source_column_name,target_column_name,aggregation_type,source_agg_value,target_agg_value,status
One column is missing is the source and target datatypes - for me if I do a UnitTest this information I need to have i acceptable criteria.
Thanks,
Greg

We could include the source and target data types in the source_agg_value and target_agg_value

Similar to Issue #555 . This will be closed by PR #558

Thanks 👍


Tours for DUB packages

Currently we can place only a single example for a DUB package. It would be nice to have a tour per package, at least for math related stuff.

We could simply add it as a new language?
Languages are completely independent as they use their own index.yml files.
And if wanted we can use subdomains like tour.dlang.science or whatever works best.
Would that work?

Awesome! One yml for all mir packages should be fine. Please let me know how I can pay for subdomain and server or transfer the payment to you.

I have used https://www.gandi.net previously

Oh I already own dlang.science - at Namecheap they had an offer: 5 years for 5$ in total :)
I will get to creating a mir repo later today, but using it in "standalone" locally will essentially be
dub run dlang-tour -- -l <path-to-mir-language-repo>

@9il
https://github.com/dlang-tour/mir is now online at
https://tour.dlang.org/tour/mir/welcome/welcome-to/mir
I simple copy/pasted the DUB examples for now. There's no restriction on the structure, i.e. the index.yml define which chapters and sections are loaded.
I might look into mapping https://tour.dlang.org/tour/mir/welcome/welcome-to/mir to mir.dlang.science, but I currently don't have enough time for this. Sorry.


Support for spcifying VR IP in shared networks

Depends on ACS code: https://github.com/apache/cloudstack/pull/4503

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/872 (JID-3695)

@Pearl1594 can you also add screenshots?


WH Hang In compile program

WH frequently hangs for me during program compile, multiple threads are all stuck waiting on mutex inside of detail::async from build->build.
Run the following test a few times:
python tests/ttnn/integration_tests/unet/test_ttnn_shallow_unet.py

For some reason this only seems to happen on WH machines, though the bug itself seemingly should be device agnostic since this is host side compile flow.
I can workaround the issue entirely by commenting out all of the detail::async calls and just do compiles on the main thread.

@vtangTT , are you working on this issue?

sorry. my question was not super clear. Is this supposed to be on your plate? This seems to be a runtime thing. Want to make sure this sit in the right queue even it is not being worked on right now. thanks!
c.c. @TT-billteng

sorry. my question was not super clear. Is this supposed to be on your plate? This seems to be a runtime thing. Want to make sure this sit in the right queue even it is not being worked on right now. thanks!
c.c. @TT-billteng

we made this change to avoid a gazillion unique threads showing up in the tracy GUI. Also, if we have an existing thread pool, why create new threads?

@TT-billteng, I'm not sure I understand your point.  This issue is reported against vanilla main, it is using the existing thread pool mechanism and I was seeing a deadlock.  My workaround is to remove the thread pool entirely and just compile on the main thread, single threaded.

after rebasing, wasn't able to repro hang on n150 after 100 iterations, waiting for if @nsmithtt can repro on his machine

Seems to have gone away, can no longer repro.  Will reopen if I run into it again


small error in step 6

In step 6 under Access your blob container from Azure Databricks,
the python code should be df = spark.read.text("/mnt/<mount-name>/<file-name>").
It's missing a "/" in front of "mnt".

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

ID: c2223cfd-6cd9-d637-d750-36156acf3438
Version Independent ID: c65ea061-d499-3449-4f1c-fb2af4b9b27b
Content: Tutorial - Access blob storage using key vault using Azure Databricks
Content Source: articles/azure-databricks/store-secrets-azure-key-vault.md
Service: azure-databricks
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@youcc Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will update the document as appropriate.

@youcc The PR for the fix has been created and it will be merged soon. Thanks again for pointing this out, I am closing the issue based on this update, please let us know if there is something else that we can help you with.


Adding include template for images

Currently generated as

Figure 2. Grace notes

One question to address is figure numbering. Do we want to keep it?

as PDF is not our main output (anymore), I'm happy to drop image numbering. The images directory should be cleaned up, yes. But I also agree that this can happen at a later point…

So if we want to postpone it then we need to have an distinct image set for each version. I'll take care of this. It does actually not matter for the template, which will be
{% include figure img="modules/cmn/grace-300.png" caption="Grace notes" %}

OK?

We need to remove in the XSLT the "/Images" we have in the path in order to have distinct version of the images for each version. So the result should be as above and not img="Images/modules/cmn/grace-300.png"

I fixed this one. I made a few changes directly in the canonicalized3.0.0.xml. However, because this is only to migrate the guidelines it will not cause any problem in the future. @bwbohl just make sure you use that file too.


replace Post with Put

Since post is an insert, it does not make sense to pass in an "id" to a post, but it does to a put.
When creating a new PR, please do the following and delete this template text:

Reference the issue number if there is one:
Fixes #Issue_Number

The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.

@pkellner Nice catch! Thank you for fixing this.


[Bug]: ROWS_AFFECTED column in table FCT_DBT__MODEL_EXECUTIONS shows always 1

I added dbt_artifacts in my project and then run one model.
after this when I query this table for specific model run I can see rows_affected is always showing as 1 and not actual number of rows loaded as part of run.
as part of my run, there are 100 records loaded in my table so I am expecting same 100 value should show in column rows_affected present in table FCT_DBT__MODEL_EXECUTIONS .
please help to fix this as this will be big saver to quickly see how many records got loaded as part my latest model execution.
Thanks a lot in advance for all your help  !

Hi @sushildjoshi are you able to provide any more context on this error:

Version of dbt artifacts
Version of dbt
The database you're using
The type of materialisation you're using for the table

any more info you can provide to help us look into this for you


Custom estimates in pipeline

Adds a custom dataset and test for earnings estimates data.

Is this branch forked off of one of your other branches? It seems like there are a lot of files that were touched here that aren't related to estimize. Would it be possible separate those out and do a separate PR?

No, this is off master. Since this is almost exactly like the earnings calendar dataset, I realized I was going to be duplicating a bunch of code, so I ended up refactoring.

Coverage increased (+0.2%) to 84.428% when pulling 3fd90a5885336c9f3473ef219af2204d3aa7cfaa on custom-estimates-in-pipeline into 85a2f6fe007a2a22fe29cc9586d8da247e8883f1 on master.

More generally in the tests, is there a way to represent the "this will not return bad things" case? I'm trying to model the error Seong was running into, where with the default loader, we would have gotten back multiple results from any number of quarters, and now we should get back only one.

For that case, we want to make sure that the data we were getting back was based on today and the last revision that we knew about <= today and not on the asof_date (updated). So what I'm missing in this test is an asof_date. I'll add that.

@joshpayne- @mtydykov and I were discussing the naming of this dataset in pipeline. In their API, we take elements from their releases endpoint and also estimates endpoint. In our database/DBZ API, we refer to this as revisions, but in the store it's Consensus Estimates.
Personally, I find earnings estimates confusing, but I'm willing to be overruled if you guys feel that best reflects the data. Though I would note that there are revenue estimates in there as well.

About my previous comment to add asof_date - never mind. At this point in the pipeline (distinct from Pipeline) we can only provide the columns that the loader expects; asof_date is not one of them, so it isn't an issue.

Coverage increased (+0.1%) to 84.174% when pulling 78cdbe549d54c63929135aed382f22bc57d23f93 on custom-estimates-in-pipeline into 620d2a8b2294986003940ace7a06e019bdc911f9 on master.

Can we do a fixup on the commit entitled "MAINT: rename" since the Travis build failed for that one, and it wouldn't be a valid state?

👍  from me.


Route param information not being propagated to mounted APIs

When mounting an API within a route_param, the attributes of the route param are not propagating downward in the route.params property.
Attaching a failing test.   #1620
This seems to touch on #1177 as to what the documented behavior should be for mount.  I would expect type information etc. to propagate down.
Note this has downstream effects; when we use grape-swagger and look at the documentation for the API, the parameter types are wrong for these route parameters (which I believe is https://github.com/ruby-grape/grape-swagger/issues/532)  It always shows type of Integer, and does not display description information etc.  I believe this is the root cause of that failure but cannot be sure.

This is still an issue. Route params with type: 'String' revert to type: 'Integer' inside mounted APIs.

This is still an issue. Route params with type: 'String' revert to type: 'Integer' inside mounted APIs.

Give a shot at fixing it?

Sure, but I'm unclear how this configuration works. Any suggestions on how to start?


Clarification on Storage.Delete

What is your question?
Under what conditions should Delete return an error? This and this seem to contradict each other.

I'd be fine with either, though semantically your proposed error essentially boils down to "something I expected to exist doesn't, so inform the caller". So maybe it should return an error in both cases? idk it's not quite the same thing.
In terms of my implementation, what would you guess would be the best behavior to have for now to maximize compatibility with the current release of certmagic?

I'd have to think on it more. (I'll be busy with a conference presentation for the next week though.)
For now, I'd say just return no error, even if the parent dir(s) does/do not exist, but I want to revisit this thinking at some point.


Added changes

@shaikkhajaibrahim Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Fix broken certificaterequest yaml

"d" is not a valid character for a duration
/kind cleanup

/lgtm

/test pull-cert-manager-website-verify

/test pull-cert-manager-website-verify


feat: colors command

This is not something I'd like to add to tokyonight.
You can just view the colors in the repo.
Thank you anyway!


Can't download from Weibo

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.04.26
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

.\youtube-dl.exe https://weibo.com/1053932022/KfbfupgTV?ref=home"&"rid=0_0_1_5046670436339028950_0_0_0
[Weibo] KfbfupgTV: Downloading webpage
[Weibo] KfbfupgTV: Generating first-visit data
[Weibo] KfbfupgTV: Running first-visit callback
[Weibo] KfbfupgTV: Revisiting webpage
ERROR: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

I tried downloading from Weibo for the first time using ytdl, and I got this error. It was on the list of supported sites, so I tried using it. The link is playable even when not logged in.

Missing --verbose flag. I add it twice just in case even if it's redundant.
You should double-quote the whole address, not only the chars under all circumstances.

But yeah, it still doesn't work:
With 2x --verbose:
# youtube-dl --verbose --verbose "https://weibo.com/1053932022/KfbfupgTV?ref=home&rid=0_0_1_5046670436339028950_0_0_0"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '--verbose', 'https://weibo.com/1053932022/KfbfupgTV?ref=home&rid=0_0_1_5046670436339028950_0_0_0']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2021.04.26
[debug] Python version 3.8.8 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: none
[debug] Proxy map: {}
[Weibo] KfbfupgTV: Downloading webpage
[Weibo] KfbfupgTV: Generating first-visit data
[Weibo] KfbfupgTV: Running first-visit callback
[Weibo] KfbfupgTV: Revisiting webpage
ERROR: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "c:\programdata\anaconda3\lib\site-packages\youtube_dl\YoutubeDL.py", line 806, in wrapper
    return func(self, *args, **kwargs)
  File "c:\programdata\anaconda3\lib\site-packages\youtube_dl\YoutubeDL.py", line 827, in __extract_info
    ie_result = ie.extract(url)
  File "c:\programdata\anaconda3\lib\site-packages\youtube_dl\extractor\common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "c:\programdata\anaconda3\lib\site-packages\youtube_dl\extractor\weibo.py", line 76, in _real_extract
    title = self._html_search_regex(
  File "c:\programdata\anaconda3\lib\site-packages\youtube_dl\extractor\common.py", line 1021, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File "c:\programdata\anaconda3\lib\site-packages\youtube_dl\extractor\common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Without -verbose:
# youtube-dl "https://weibo.com/1053932022/KfbfupgTV?ref=home&rid=0_0_1_5046670436339028950_0_0_0"
[Weibo] KfbfupgTV: Downloading webpage
[Weibo] KfbfupgTV: Generating first-visit data
[Weibo] KfbfupgTV: Running first-visit callback
[Weibo] KfbfupgTV: Revisiting webpage
ERROR: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

same issue, any idea? Thanks in advance!


Exception/Error Handling with negative tests for Mux

Fixed in PR  #343

@coot you'll have to explain what this was reopened for :-)


Due to overly strict limitations, some requests with a content-type of CBOR were not  intercepted.

ic-inspector has indeed provided great help in the development of IC, but I have encountered an issue. When the content-type is already application/cbor and the message body content is also in cbor format, some cbor content is not properly captured due to the following code limitations:
export function shouldCapture(event: chrome.devtools.network.Request): boolean {
return (
isBoundaryNodeURL(event.request.url) && event.request.method === 'POST'
(isBoundaryNodeURL(event.request.url) || hasICHeaders(event)) &&
event.request.method === 'POST'
);
}
Can we relax the code to only check the content-type and body content?

Yes, I think we could potentially relax to contentType=cbor as the only requirement. Would you be interested in creating a PR @ljx199?

This has been rolled out.


Pull requests / updates?

Hi @hashtagchris, I was wondering whether you'd be willing to review the work that @gvheertum and @vestervang have been doing, and merge it?
I mean the caching PR (#15) and the commits in @vestervang's fork (https://github.com/hashtagchris/DotNet-BlueZ/compare/main...vestervang:DotNet-BlueZ:main)
I'd like to look into a dotnet 7 upgrade, and am wondering whether a PR would be accepted?

Hi @hansmbakker i'm currently maintaining this fork as I use it at work. If you want to improve the library you can make a PR there and i'll have a look at it and push the changes to nuget.

@vestervang please see https://github.com/ProrepubliQ/DotNet-BlueZ/pull/7
Could your repo somehow take over the "lead role" in the sense that the issue reporting is here but the most recent updates are in your repo?

So I've just cloned the repository that @vestervang holds, it looks like this was updated to .NET 7, not a huge deal but is there a reason we want to target that as opposed to netstandard like this upstream project does?
FWIW out of the box the current Ubuntu LTS (Jammy) does not have dotnet-sdk-7 in its default apt repos which is what I'm currently targeting.
I also noticed that the branch doesn't have the ability to open issues; do we want to still try an coordinate with this upstream project or do we want to attempt a hard fork?

Hi @aolszowka,
I have enabled issues on the repo now and there is no reason as far as i can see that we should target dotnet7 specifically.

Ok I am going to start coordinating work over there on that repository, please let me know if it becomes too much. I've got some free time today to grind this!


Could not invoke RNAndroidLocationEnabler.pro mptForEnableLocation Needed

fixed : https://github.com/Richou/react-native-android-location-enabler/issues/99


Bug when mixing implicit arguments, recursion and totality

The following code:
assume type A : Type
assume type B : Type
assume val f : #a:A -> b:B -> Tot bool
val bug : a:A -> b:B{f #a b} -> Tot unit
let rec bug a b = ()

returns the strange error Expected expression of type "B"; got expression "a" of type "A" at line 4, chars 10-11.
It works well if either:

I do not use implicits;
I remove rec; or
I do not ask bug to be total.

See examples/bug-reports/bug139.fst .

This is still broken.


1.22.0 is broken

Can't build project after upgrading version from 1.21.1 to 1.22.0.
node_modules/@types/domutils/index.d.ts:6:10 - error TS2614: Module '"../../../../../../../../Users/UserName/projects/project/node_modules/domhandler/lib"' has no exported member 'DomElement'. Did you mean to use 'import DomElement from "../../../../../../../../Users/UserName/projects/nc/namp-cms-content-provider/node_modules/domhandler/lib"' instead?

6 import { DomElement } from "domhandler";
           ~~~~~~~~~~

node_modules/@types/htmlparser2/index.d.ts:17:10 - error TS2614: Module '"../../../../../../../../Users/UserName/projects/project/node_modules/domhandler/lib"' has no exported member 'DomElement'. Did you mean to use 'import DomElement from "../../../../../../../../Users/UserName/projects/project/node_modules/domhandler/lib"' instead?

17 export { DomElement, DomHandlerOptions, DomHandler, Element, Node } from 'domhandler';
            ~~~~~~~~~~

node_modules/@types/sanitize-html/index.d.ts:17:10 - error TS2459: Module '"../../../../../../../../Users/UserName/projects/project/node_modules/htmlparser2/lib"' declares 'Options' locally, but it is not exported.

17 import { Options } from "htmlparser2";
            ~~~~~~~

  node_modules/htmlparser2/lib/index.d.ts:5:14
    5 declare type Options = ParserOptions & DomHandlerOptions;
                   ~~~~~~~
    'Options' is declared here.

@boutell it looks like a similar issue to https://github.com/apostrophecms/sanitize-html/issues/320
Look at it, please.

All tests are passing — need to know how to reproduce this issue, as a failing test PR or as a simple test project demonstrating it.

(All tests are passing in node 12.)

I am also getting this issue. From the output from @nightnei, it seems like both you and I are using typescript with @types/sanitize-html v1.20.2. So I believe the issue is with the typed library, not this module itself

Looks like Options in the typed lib should be changed to Parser
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/1c82c17ce10411086f2bb56aa91a1c9246463bfc/types/sanitize-html/index.d.ts#L67
@boutell do you also mantain typed version?

@dmattia, @boutell  I have found the problem.
"@types/sanitize-html" uses dependency - "@types/htmlparser2": "*".
but now latest verison of htmlparser2 -4.1.0 uses own typeshttps://github.com/fb55/htmlparser2/compare/v4.0.0...v4.1.0#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R24
so now you can remove @types/sanitize-html and everything will be fine :)

solution above does not work for us because our monorepo has but htmlparser2 and sanitize-html installed. if we remove @types/sanitize-html then we get Could not find a declaration file for module 'sanitize-html'.

Also chiming in -- @nightnei's solution unfortunately also doesn't resolve this for us. Without @types/sanitize-html, tsc spits the dummy when compiling: Could not find a declaration file for module 'sanitize-html'
This is with Typescript 3.8.3. I'd guess that the proposed solution relies on a JS inference option being set?
Either way, once I install<EMAIL_ADDRESS>the issue then becomes a mess of:
Module '"../../domhandler/lib"' has no exported member 'DomElement'
Module '"../../htmlparser2/lib"' declares 'Options' locally, but it is not exported.

For now I'll keep the installed version pinned prior to 1.22.

can this issue be re opened plz?

Sure, reopened and the title clarified. This isn't something we can resolve in-house as we don't do typescript but happy to help release whatever is decided upon.

our solution here was to set "skipLibCheck": true, in our tsconfig. This is obviously not ideal but for those that want a quick fix it works

I've made a PR to @types/sanitize-html https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43870, which should resolve the issue with htmlparser2 types.


如何下载聊天中的图片和视频？

How to download the pic and audio from the message?

Link to: https://github.com/wechaty/PMC/issues/14


ytmapi-rs: Wrong value in album, duration if song has multiple artists (with my crappy solution)

Hello. I'm apologizing in advance for my bad English since I'm not a native speaker.
During the development I saw that SearchResultSong.duration return an artist or album name instead of actual duration and this is only happening to song with multiple artists (Example: Sleeping In The Cold Below by Keith Power & Alan Doyle AND All Falls Down by Alan Walker, Noah Cyrus & Digital Farm Animals)
Solution

ytmapi-rs > src > parse > parse_song_search_result_from_music_shelf_contents

// TODO: Type safety
// TODO: Tests
fn parse_song_search_result_from_music_shelf_contents(
    music_shelf_contents: JsonCrawlerBorrowed<'_>,
) -> Result<SearchResultSong> {
    let mut mrlir = music_shelf_contents.navigate_pointer("/musicResponsiveListItemRenderer")?;

    let title = parse_flex_column_item(&mut mrlir, 0, 0)?;
    let mut artist: String = parse_flex_column_item(&mut mrlir, 1, 0)?;

    let mut run_idx = 1;
    loop {
        let cont: String = parse_flex_column_item(&mut mrlir, 1, run_idx)?;
        run_idx += 1;

        // Since next section seperator is always " • "
        // We can check if it's not equal to that instead of ", " and " & "
        if cont != " • " {
            artist.push_str(&cont);
            artist.push_str(&parse_flex_column_item::<String>(&mut mrlir, 1, run_idx)?);

            run_idx += 1;
        } else {
            break;
        }
    }

    let album = parse_flex_column_item(&mut mrlir, 1, run_idx)?;
    let duration = parse_flex_column_item(&mut mrlir, 1, run_idx + 2)?;
    let plays = parse_flex_column_item(&mut mrlir, 2, 0)?;
    let explicit = if mrlir.path_exists(BADGE_LABEL) {
        Explicit::IsExplicit
    } else {
        Explicit::NotExplicit
    };
    let video_id = mrlir.take_value_pointer(PLAYLIST_ITEM_VIDEO_ID)?;
    let thumbnails: Vec<Thumbnail> = mrlir.take_value_pointer(THUMBNAILS)?;
    Ok(SearchResultSong {
        artist,
        thumbnails,
        title,
        explicit,
        plays,
        album,
        video_id,
        duration,
    })
}

PSA: I'm happy to create a pull request if you are happy with my solution but since I'm a bad programmer I decided to create an issue instead

G'day there,
Thanks for the report - I'm glad to see somebody using this library!
I can replicate this locally, it's a faulty assumption from my behalf.
Let me see what I can do, there are a couple of things I'd like to investigate before committing to a solution, ie can this be done declaratively and have I made this same mistake elsewhere.

Yes. Thank you for your hard work. Have a good day.

I have resolved this in #171 and will merge and release as a new version shortly. I couldn't find a better solution than yours, so that's what I've gone with, the only change is to abstract it out to a function. Well done tracking this down and finding the solution :).


Problem in updating app info

I have a problem in update app info in daily update app code. It seems that if the app_id exists, the code doesn't update any information of app information

this problem happened because of missing in commit query of get or update id.
"conn.commit()"


migrated app still requires -e for app run even with only 1 extension

Following the instructions for migration, a user reported that when attempting to do aio app run they were told to explicitly choose which extension to run.  There was only 1 extension in the app, but someone, leaving some previous files around left the tooling to think that there were 2 ( dx/excshell/1 and application )
I suspect this is not deleting the .aio file ...

Determining the number of extensions is by config https://github.com/adobe/aio-cli-plugin-app/blob/c00e8af1c911bfcc6e27923069eb245cfda0939f/src/commands/app/run.js#L40 but unless we can repro I don't think we can do much with this issue


Fix build / Minor cleanup

After cloning the project, I ran into a few issues when trying to get it to run. This was mostly due to version incompatibilities between devDependencies.
This PR fixes that issue along with a few minor house cleaning tasks including:

Updated dependencies and devDependencies to their latest compatible versions

survey-vue has been updated to 1.9.23. Any later and there's styling issues with bootstrap / WET

Linted the project using the lint script
Fixed minor TypeScript issues in src\store.ts
Disabled incomplete unit test
Updated store.spec.ts to use jsdom as jest-environment

@MrDeshaies Is this repository still being maintained?


Wall disappears in 2D SLAM

I'm trying cartographer's 2D SLAM and met the problem that the wall, which once has been recognized, be disappeared when the robot returns.
It had not been happend with Gmapping, so I want the cartographer's result to be the same level.

cartographer

gmapping

photo of the area, taken from the entrance, left side in the upper images (the way is about 1m wide):

How can I fix this?
I think the wall's smooth surface is causing problem (It may be hard to recognize for laser sensor).
But I believe the effect can be mitigated in software side.
Sensors:

2D laser sensor (/scan)
wheel odometry (/odom)
IMU (/imu)

forked repo
bagfile
result of rosbag_validate
Thanks in advance for your help!

I'd look at the submaps display instead of occupancy grid, The occupancy grid node of Cartographer just blends all submaps into an image, so it could be that it just looks as if the wall is gone.
The submaps plugin of cartographer_rviz is able to show you the individual submaps instead, which might be better for checking the result here. Note that you also have similar laser artifacts ("spray" of free space) in the gmapping result, maybe because of the reflective (?) surface of your walls.

Thanks for the response.
First, I checked the individual submaps on rviz and found that the first submap, no 0, still holds the wall and others don't. I show the all submaps below (total 6 with default num_range_data =90).
Is there a way to use the better result of the submap in navigating the robot? (I'm considering to use the Navigation Stack)
0
1
2
3
4
5
Second, as you mentioned, "spray" of free space can also be found in gmapping result (and also in the result of official data checker tool of HOKUYO) and I think it's caused by the reflective surface.
I guess gmapping tends to keep what is recognized to be a wall as a wall and cartographer not.
It seems to be a tradeoff between erasing what have to be kept and failing to erase what have to be erased.
p.s.
I found that setting max_range to around 5m reduces the probrem, but I want to utilize the 20m view range of the sensor (HOKUYO UST-20LX-H01).

@wkwkw for the reflective surface, I suggest to filter some points based on their intensity.

@wkwkw I am interested to hear that you are using Hokuyo lidar, have you ever confused by the problem that this lidar get inaccurate at near distance(0.2m - 1.0m)?

@Yu-Xiaoxian Hi, I've just been trying the way you mentioned and getting better results.
I'm now feeling more confident in taking this way by your reply, thank you.

@wkwkw for the reflective surface, I suggest to filter some points based on their intensity.

And for the question about Hokuyo lidar, no. I've never been met a problem by the inaccurateness at near distance so far.

@wkwkw great, I'm glad to hear you making progress. Thanks for your info about the Hokuyo Lidar, I will contact my supplier for a solution.


Namespace 'NodeJS' has no exported member 'WriteStream'.

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ ] 3.x
[x] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Current behavior:
I am just trying to serve my application and it throws this:
<EMAIL_ADDRESS>Namespace 'NodeJS' has no exported member 'WriteStream'.
Namespace 'NodeJS' has no exported member 'ReadStream'.
Expected behavior:
Serve the application.
Steps to reproduce:
`ionic serve`

Other information:
I am not sure if that could be the problem, but I recently changed from Windows to Mac.
Ionic info: :
Ionic:

   ionic (Ionic CLI)  : 4.0.1 (/Users/eduardogodoy/node_modules/ionic)
   Ionic Framework    : ionic-angular 3.9.2
   @ionic/app-scripts : 3.1.6

Cordova:

   cordova (Cordova CLI) : not installed
   Cordova Platforms     : none

System:

   NodeJS : v8.11.3 (/usr/local/bin/node)
   npm    : 5.6.0
   OS     : macOS High Sierra

Same error with ionic 3.X
` Ionic:
ionic (Ionic CLI)  : 4.1.1 (/usr/local/lib/node_modules/ionic)
Ionic Framework    : ionic-angular 3.9.2
@ionic/app-scripts : 3.2.0
Cordova:
cordova (Cordova CLI) : 8.0.0
Cordova Platforms     : android 6.4.0, ios 4.5.5
Cordova Plugins       : no whitelisted plugins (21 plugins total)
System:
Android SDK Tools : 26.1.1 (/Users/pisix/Library/Android/sdk)
ios-deploy        : 2.0.0
ios-sim           : 7.0.0
NodeJS            : v8.9.4 (/usr/local/bin/node)
npm               : 6.4.0
OS                : macOS High Sierra
Xcode             : Xcode 9.4.1 Build version 9F2000
`


Enhance ftp delete

This enhance change the logic of FTP Upload splitting the delete of camera storage after upload as requested in #166

Thank you for your time.
Please also edit the check_conf.sh file to check for the presence of the parameter at startup, with the default value.

I've changed the check_conf.sh
For default I've set to "yes" for defaut. I hope this semplify the retrocompatibility.
I need this modify too. ;)

Thank you.


use robot framework : appium flutter library but can't use commands with webview

my app is developed using flutter but some pages are webview ...
ex. run robot framework  use appium flutter library can't click or input page webview
How do you have a way to do it? appiumflutterlibrary can be used for webview page ?

I'm facing the same issue as well, do we have any solution for this, since this is critical in testing some of the pages which are in webview


Which size should be the input image inorder to detects all its objects

First I tried to inset this image, it detects null

Then when I cropped it, it detects 3 objects, great!!

Also this image it detects objects without cropping (one object).

But while I cropped it it detects 0 objects

So what should I do with this issue ?

There are 3 constructor methods for an InputImage:

from file path
from file
from bytes

Only when you create an InputImage from bytes you need to set the size.  And from what you have share it seems that you are using the path, so, you do not need to set the size. I guess your issue is another: you are not setting the correct settings for the object detector:
Since you are processing only one image you need to set the detector options in single mode:
final options = ObjectDetectorOptions(mode: DetectionMode.single, classifyObjects: true, multipleObjects: true);

Interesting... I uploaded your images and tested with our example app and I am getting the same result, when the image is cropped it is not detecting the object, but so far I think it is the Google API not detecting. What I would recommend you to do is: try Google's sample app and try your images, if you still can reproduce the issue, then we need to report the issue to Google.
You can get their example app here: https://github.com/googlesamples/mlkit/tree/master/ios/quickstarts/vision


description in SubParsers

Is it possible to specify some kind of description of the sub-command?
For example in the example I would like to tell the user what <program> foo start will do, what <program> foo stop will do etc.
I searched for a parameter to tap.SubParser like description but found none.  Other "natural" choices where to place such a description were (in my opinion)

the docstring of the business logic functions (e.g. run_foo_start here ) (as click does it)
the docstring of the argument class derived from TypedArgs (e.g. FooStartArgs here )

Yes, this is a frequently requested feature, just requested in the lasted opened issue #32 as well.
I've just added some minimal support in #34 which adds:

help in subparsers
description in subparser groups

That's basically what argparse allows. I hope this resolves the issue?
New version 0.2.8 is available now on PyPI.


xml with xi:include should not be allowed into the draft repository

For us to truly know what was submitted (for note well purposes) internet-drafts in the repository really need be self-contained. They should not rely on external content, and in particular, should not have any xi:includes remaining.
However, authors need to be able to manage source that carries pointers to, for instance, reference bibxml for the "current" version of a thing being referenced.
We need to consider tooling that prepares a document for submission, fully expanding the includes in such a way that a future user of the submitted xml file can trivially reverse the preparation to recover the pointers to the external content. The result should be such that a diff before and after the preparation makes it trivially easy to see that the includes were expanded and where they were expanded from.
This tooling should exist both for authors locally (where the expansion can fetch from arbitrary places) and in the submit tool itself (where expansion will be limited to well-known "safe" places to fetch from).
We may also want to consider moving back to the ability to indicate a reference with a token with a well-known structure rather than providing a URL.
This has repercussions for almost all tooling involved in authoring, so a plan needs to be built with a timeline allowing such tooling to adapt. In particular @cabo and @martinthomson need to weigh in.

see also #3570 and #3576

On 2022-03-04, at 21:31, Robert Sparks @.***> wrote:

This has repercussions for almost all tooling involved in authoring, so a plan needs to be built with a timeline allowing such tooling to adapt. In particular @cabo and @martinthomson need to weigh in.

Kramdown-rfc (*) generates fully resolved XML, so for the immediate issue there is not a problem.
Kramdown-rfc makes its own fetches from bibxml servers.  There is a default server that kramdown-rfc imposes, and the tool can be redirected to different sources only by environment variables (i.e., not from the document).
Kramdown-rfc itself does have a {::include feature, which can be switched off by KRAMDOWN_SAFE (as is done on author-tools.ietf.org, AFAIK).  This is a very useful feature when constructing documents from multiple files in the repo (and it also generates fully resolved XML, so there is no submission problem), so it probably should remain on by default.  We might want to restrict the places from which included text can be pulled, however it is hard to avoid that attacks with symlinks in repos remain possible.
Clearly, embedding URLs in the authoring document that are indiscriminately resolved upon processing is not the way to go.
(Requiring authors that author in XML to type in full URIs where reference tokens sufficed in v2 also was one of the more obviously author-hostile decisions in v3, but was easily solved by using the v2v3 function.)
Note that SYSTEM entities (**) are another way to dereference URLs in RFCXML, in addition to xi:include.
Grüße, Carsten
(*) Unless stand_alone is not set to true.
stand_alone: true is the normal way of running kramdown-rfc (also now implied by “v: 3”, but not yet default for backwards compatibility reasons).
(**) which are also generated by kramdown-rfc with stand_alone: false, while losing some of the functionality of reference processing such as setting anchor names.
Example (from rfc6775.xml as we used it during AUTH48):

]>

[…]

I currently rely fairly extensively on @cabo's kramdown-rfc for most drafts I'm involved with, as do many others that I'm aware of.  There are a few cases where external references are used, but those are generally XML format drafts, like RFC 6775 from this comment.
I think that the plan of resolving entities and xi:include during submission is a good one.  I've observed that the v2v3 tool does not resolve xi:include piece, but it does resolve entities.  I already run v2v3 for most things in CI (which would undo the work of standalone: false done by kramdown-rfc if you are using that...).
If there was another option on that tool that pulled in xi:include imports, I would be very happy to run that in CI so that the datatracker does not have to.  It's probably less reliable (failed fetches are the number 1 reason for failing builds in my experience), but it would mean less divergence between what is submitted and what is used, which I think is worthwhile trying to achieve.
@reschke's tools generally favour (but don't require) a fully standalone document with resolved references.  He might also provide some insights here.

@reschke's tools generally favour (but don't require) a fully standalone document with resolved references. He might also provide some insights here.

It's more like that: I prefer standalone documents, for all the reasons discussed. My tools process includes as far as possible; that is, if XSLT runs in a browser, it's limited to the browser's security constraints.
In general, I think that standalone plus the ability to refresh what has been included before is the way to go.


quoting issue breaking the resource

Hello,
this line breaks the resource:
https://github.com/zarplata/concourse-git-bitbucket-pr-resource/blob/master/assets/check#L79
We had a PR with quotes in it (title: This is a "test"). The resource then errors on that line because it is not properly escaped

I think that's been fixed in commit 88b115b2d98a2b609832a78620b12e5fc46f187a.


do not resolve gvk if override is defined

If the path is overwritten, the gvk is not needed to be evaluated.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.0%) to 93.548%

Totals

Change from base Build<PHONE_NUMBER>:
1.0%

Covered Lines:
203

Relevant Lines:
217

💛  - Coveralls


com.gargoylesoftware.htmlunit.ScriptException: missing formal parameter (https://www.mohavecounty.us/Scripts/app/password_reset/index.js?v=2#100)

With Firefox on Ubuntu 20.04 (latest daily build), I am able to read the contents of https://www.mohavecounty.us/ContentPage.aspx?id=111&cid=869&parcel=31344502 without any problem, regardless of the existence of script errors. However, when I try to do the same with HTMLUnit, using the following Java code and the latest release of HTMLUnit (version 2.37.0, which I downloaded from https://jar-download.com/artifact-search/htmlunit along with all its dependencies).
final WebClient webClient = new WebClient();
webClient.getOptions().setCssEnabled(false);
webClient.getOptions().setThrowExceptionOnScriptError(false);
webClient.setRefreshHandler(new RefreshHandler() {
	public void handleRefresh(Page page, URL url, int arg) throws IOException {
	    System.out.println("handleRefresh");
	}
});
HtmlPage page = (HtmlPage) webClient.getPage("https://www.mohavecounty.us/ContentPage.aspx?id=111&cid=869&parcel=31344502");

I get the following exception:
com.gargoylesoftware.htmlunit.ScriptException: missing formal parameter (https://www.mohavecounty.us/Scripts/app/password_reset/index.js?v=2#100)
I understand that whoever has developed the website has messed up. But Firefox is able to handle that problem gracefully. I would like the next release of HTMLUnit to handle the same problem just as gracefully as Firefox. In the meantime, I would appreciate any workarounds and tips on how I can change my code to get it to open the web page without throwing the above referenced exception.

Sorry, but i can't reach the page at https://www.mohavecounty.us/ContentPage.aspx - got a server not found. Without being able to reproduce your problem i can't really help.
In general
webClient.getOptions().setThrowExceptionOnScriptError(false);
should make the javascript execption handling similar to the browsers. But even with this set to false the errors are visible in the logs.

https://www.mohavecounty.us/ContentPage.aspx does return a server not found. But that's not the page I am trying to access. The page I am trying to access is:
https://www.mohavecounty.us/ContentPage.aspx?id=111&cid=869&parcel=31344502
And https://www.mohavecounty.us/ContentPage.aspx?id=111&cid=869&parcel=31344502 does NOT return a server not found, and does return data.

Am 10. Februar 2020 22:01:41 MEZ schrieb AbdenourAchab<EMAIL_ADDRESS>
https://www.mohavecounty.us/ContentPage.aspx does return a server not
found. But that's not the page I am trying to access. The page I am
trying to access is:
https://www.mohavecounty.us/ContentPage.aspx?id=111&cid=869&parcel=31344502
And
https://www.mohavecounty.us/ContentPage.aspx?id=111&cid=869&parcel=31344502
does NOT return a server not found, and does return data.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/HtmlUnit/htmlunit/issues/129#issuecomment-584352121

Strange, have tested it again, same result - server not found. Any idea?

Are you in the United States or Canada ? ... And are you using a VPN ?
I am in the United States. I access https://www.mohavecounty.us/ContentPage.aspx?id=111&cid=869&parcel=31344502 using my laptop without a VPN, and using a server in Canada without a VPN. But, I just tunneled to France using a VPN and, now, I cannot access the web page either.
Please make sure you are not trying to access the page from outside the United States or Canada. Whether by being physically outside the United States without a VPN, or by using a VPN to tunnel outside the US or Canada.

No, i'm from the good old europe. So far i have no access to any VPN - will try if i can find some spare time.

The following import statement:
import com.gargoylesoftware.htmlunit.javascript.*;

and adding the following code, suggested by rbri while helping me with issue # 127
webClient.setJavaScriptErrorListener(new SilentJavaScriptErrorListener());

did fix the problem.
Thank you very much rbri for your help. I am going to close this issue.


Resize metal /efi to 300MiB

What this PR does / why we need it:
Garden Linux images have a default /efi size of 64MiB. This is not sufficient for more than one kernel, which often prevents Kernel Updates from working. We experienced this with our images and customers also experienced it. I propose to enlarge the /efi partition to 300MiB have no issues if the need for two (or even three) images arises.
Which issue(s) this PR fixes:
Fixes #1893
Special notes for your reviewer:
Although this is against the "create a small/minimal immutable image" rule, but i think practically this would help many users in the future.

@dnan0s I do not see this in the metal feature directly yet. I assume you have your own Garden Linux image build pipelines and can add a custom feature?
I suggest that you move the changes to a new feature and call it something like _bigefi (maybe you can come up with a better name) and include it in the cname when building it, like metal-kvm_bigefi_prod. This way you can still consume our changes from metal and other features without code conflicts. Still, you would need to test this of course if your changes still are applied and not "overwriten" by some other feature.
This PR should not be merged yet to gardenlinux/gardenlinux. If you have a new _bigefi feature (or a feature with a better name), you are very welcome to open a PR to contribute it.


Lasmerge added

I finally figured out a method to put LAStools under lasmerge module, in setup.py (once the LAStools is under lasmerge/ ), however it's too big to push to git, so i will try delete some files and push again later.
The lasmerge module will call LAStools through subprocess
the unittest is under tests directory, it would not only test the existance of the file, but also the size of the file

@weiqin61 looking through this, we are still currently using PDAL to convert the PLY files to LAS files.
do we have an alternative way to do this? if not, the science package will still have a dependency on PDAL even if we refactor the heightmap functions to use las2dem instead of PDAL. maybe that's unavoidable.

@max-zilla I finally pushed the LAStools to our package as well, so now we don't need to worry about setting up the environment, please try python setup.py install to see whether this would work on your end.
As to the ply2las, I would be happy if that can get rid of PDAL, things will get easier for other users as well. And I can help with that once we make a decision.
And the las2dem, I am still thinking what we can do through pure python programming.


Update requirements.txt

just adds pillow to the requirements list

I've used the following commands in Anaconda to see if the current requirements.txt need modifications:
git clone https://github.com/1hubert/genshin-dialogue-autoskip.git
cd genshin-dialogue-autoskip
conda create -n test
conda activate test
conda install python=3.12
pip install -r requirements.txt
autoskip_dialogue.py

As you see, the win32api is able to correctly detect my screen resolution. The rest of the script also works as intended. I appreciate your help as it's clearly intended to save other people's time which I respect a lot, but I don't want to add an extra module to requirements.txt without direct evidence that it's indeed needed for the script to work.

@1hubert hey im not sure if this helps but when i was changing to the most recent version of the script and had just installed the new requirements i got this error
pyautogui.PyAutoGUIException: PyAutoGUI was unable to import pyscreeze. (This is likely because you're running a version of Python that Pillow (which pyscreeze depends on) doesn't support currently.) Please install this module to enable the function you tried to call.
I'm running python 3.12.1 and all the most recent versions of the other packages. The script immediately worked after ran pip install pillow.


Check alter column for MSSQL

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
#11

Fixed by yiisoft/db-mssql#282


Syntax Error

When I run checkpe.py I get the syntax error
 File "checkpe.py", line 188, in <module>     data = extract_infos(args.FILE)   File "checkpe.py", line 108, in extract_infos     res['SectionsMeanEntropy'] = sum(entropy)/float(len(entropy)) TypeError: object of type 'map' has no len()
I am a noob, so can someone help me fix this. Oh yeah and entropy = map(lambda x:x.get_entropy(), pe.sections)

just put
len(list(entropy)).
this solves the issue of len.
but its giving an error of giving division by zero error.

The zero error is because the entropy variable is not getting generated correctly.
You can change those lines to this :
list_pe_sections = []
for i in pe.sections:
list_pe_sections.append(i)
entropy = list(map(lambda x:x.get_entropy(), list_pe_sections)).

You will get similar errors at many places with the map function. Just put the map function in list().
Making these changes will remove the syntax errors, but the code is predicting legitimate files as Malicious.
Still need to look into that.


Bug when preparing data for finetuning

I've run into a bug that I can't fix when trying to prepare a dataset for finetuning.
Here's the code:
def data_generator() -> Generator[dict[str, Any]]:
    yield {
        "target": df['Weekly_Sales'].to_numpy(),
        "start": df.index[0],
        "freq": pd.infer_freq(df.index),
        "item_id": "1",
    }

features = Features(
    dict(
        target=Sequence(Value("float32")),
        start=Value("date32")),
        freq=Value("string"),
        item_id=Value("string"),
    )

hf_dataset = Dataset.from_generator(data_generator, features=features)

hf_dataset.save_to_disk(Path("sales_dataset/"))

df = hf_dataset.to_pandas()

df.to_csv('sales_dataset/sales_data.csv', index=False)

Then, when I run python -m uni2ts.data.builder.simple sales_data sales_dataset/sales_data.csv --offset 40 --dataset_type long  , I get the error:
IndexError: index 0 is out of bounds for axis 0 with size 0. Not sure why that happens, as my df is not empty, and the .csv is not empty either.
Here's the CSV I'm using: https://raw.githubusercontent.com/marcopeix/FoundationModelsForTimeSeriesForecasting/main/data/walmart_sales_small.csv
I'm only using data for Store==1 (143 rows of data) and the first three columns only (Store, Date, Weekly_Sales). Prior to running the function, I set the index as the Date column.
What am I missing?

didn't look too deeply into this, but I'm guessing it's due to the format (column names) of your data frame?
https://github.com/SalesforceAIResearch/uni2ts/blob/2ba614de8878d350c62835c942b450d2f4d5a711/src/uni2ts/data/builder/simple.py#L58

Hi @marcopeix, have you solved this issue? If so, I will close this issue


[18.03] docs cherry-picks

docs cherry-picks for 18.03; more commits can be added to this, but we can merge at any point.

https://github.com/docker/cli/pull/900 docs: mention sctp
https://github.com/docker/cli/pull/909 Update run.md --restart to include unless-stopped

cc @gbarr01


List of deployable locations in the helper wizard should be auto-populated

Is your feature request related to a problem? Please describe.
Currently the list of locations that can be deployed to via the helper wizard is hardcoded. E.g.
https://github.com/Azure/AKS-Construction/blob/88be947c9ada7c92805b3251a931e2c7aee634e3/helper/src/components/deployTab.js#L307-L328
Describe the solution you'd like
This list should be auto-populated based on the available locations that support AKS.
Describe alternatives you've considered
n/a
Additional context
If someone can point me in the direction of a published JSON file/API that includes the locations and their service availability matrix, I can put a PR together.

@khowling - perhaps we can generate the json during the release workflow. Alternatively we can just maintain the json in the repo, but i think creating it on demand in the release might be cool.

@fireblade95402 - This one up your street?

Just noticed this one now. As we have a huge number of locations. I have previously created a file that populates the locations. As this file is also used to generate the VM SKU list.
I suggest we keep the current logic and update the location  files with any new required locations for the helper.
Current list is:
[
{ "key": "europe", "text": "Europe", "itemType": 2},
{ "key": "WestEurope", "text": "West Europe"},
{ "key": "NorthEurope", "text": "North Europe"},
{ "key": "UKSouth", "text": "UK South"},
{ "key": "UKWest", "text": "UK West" },
{ "key": "middleeast", "text": "Middle East", "itemType": 2 },
{ "key": "UAENorth", "text": "UAE North" },
{ "key": "america", "text": "North America", "itemType": 2 },
{ "key": "CentralUS", "text": "Central US" },
{ "key": "EastUS", "text": "East US" },
{ "key": "EastUS2", "text": "East US2" },
{ "key": "WestUS", "text": "West US" },
{ "key": "WestUS2", "text": "West US2" },
{ "key": "WestCentralUS", "text": "West Central US" },
{ "key": "canada", "text": "Canada", "itemType": 2 },
{ "key": "CanadaCentral", "text": "Canada Central" },
{ "key": "CanadaEast", "text": "Canada East" }
]


Exception and Pagination refactor, major Integration test cleanup

Moved exceptions from the experiments-objects module to the exceptions module.
Moved specific exceptions to their respective packages, leaving only exceptions which are not dependent on other modules behind.
Adjusted pagination to follow Java style more closely (converting enums to UPPER_CASE).
Simplified ExceptionProvider unit tests and added exception providers to avoid some "500: null" exceptions.
As a result had to change a couple of integration tests, and while I started I couldn't stop. Hence here we go:

Restructured the messy service package into multiple meaningful packages.
Cleaned up the TestNG XML files and moved all general tests to the main file, while keeping the ability to run smoke and perf tests individually.
Updated the module README.
Enforced camelCase method names in all service files. Simplified code where IntelliJ's inspections told me to take action.
Unified license headers in all integration test service files.
Resolved a lot of FIXME's around multiple tests, many of which were automatically resolved thanks to the exception changes from 500s to more useful ones.
Removed dead code (commented out or only called by a test but no where else).
Simplified the APIServerConnector to only have one request method (code dedup).

Instead of ignoring the EventLog tests, I removed one of them as it basically tested try-finally capability, and improved one to a mockito timeout
simplified some code in the EventLog with the potential of speed-up through parallel streams

Changes Unknown when pulling 68a3da2c9231e68b6d75f671bde0ccea1e80fabe on feature/exception_and_pagination_refactor into ** on develop**.

I did:
git clone git@github.com:intuit/wasabi wasabi-fresh && cd wasabi-fresh
git checkout feature/
./bin/wasabi.sh stop remove bootstrap build start test

Bootstrap upgraded Docker, Wasabi built properly, the UI built properly, during startup I got lots of chill 'ax though: needed to restart it. It's maybe an issue with our bootstrap script.
So started Docker manually and continued from
./bin/wasabi.sh start test

which worked fine:
===============================================
IntegrationTestsMeta
Total tests run: 789, Failures: 0, Skips: 0
===============================================

I also ran
mvn clean test # (because build does not run unit tests, see #86 )

and all passed.

Ok I tried everything as you described it. Got also the beer-problem in the start step (will take a look at that since it still works in my branch). So normally a
./bin/wasabi.sh stop remove build start

should be enough when I switch from one branch to the other, right? The tests are running through now with /bin/wasabi.sh test and mvn test but for some reason when I use IntelliJ some of them are failing. This could be either an IntelliJ internal -not refreshed everything properly- thing or maybe the problem is that something is not properly cleaned up and when I run the tests via IntelliJ it fails during to that.
Either way: I am not sure whether this is really blocking your merge, but I will investigate a little bit further.

Can you tell me which ones are failing? We observed the SegmentationProfileTest failing due to how IntelliJ picks up the simple-json dependency, but that happens on several branches right now.

Ok - here is a list (I am afraid it's a bit more than that):

ForcedFailureTest
ExperimentTest

failPostExperiments (the one with the 'unique constraint')
failPutExperiments (same)
failPutExperiments (checking whether the sampling percent are between 0.0 and 1.0)

PaginationTest

experimentPaginationFilterByName
experimentPaginationSortByIllegalKey

ContextMutexTest
ContextTest
FavoritesTest
MutualExclusionTest

prepareBucketsForMutualExclusionAssignment
startBatchExperiments

SetupPerformanceTest
SmokeTest
TearDownTestExperiments
ExperimentTest

checkIfExperimentsAreDeletedProperly

PagesTest

deletedExperimentPages

SegmentationOnHttpHeaderTest

cleanUp

@felixgao thank you, I will go through the code and update all those isNull/nonNull's, it reads a little nicer and is less likely to cause errors by accidentally assigning null to a variable.

@felixgao I went through all (*.java) files (in modules) and replaced all null != ..., null == ..., ... == null, ... != null with Objects.nonNull(...) and Objects.isNull().

Coverage increased (+0.3%) to 63.745% when pulling 297b81b039f6712f2473463c5a1482130ffe2688 on feature/exception_and_pagination_refactor into 90ad1769a0cbf1c98e8ed73d70bf893b04b14c67 on develop.


Release 3.3.2

👍

👍

Coverage increased (+0.7%) to 89.416% when pulling bcb80698a4683e389b4f5120e5af08f4561363be on czpython:release/3.3.2 into 1442d74a62ade9990a21b689771fc9e0209c2e48 on divio:release/3.3.x.


build: fix building on mac

Get os type and use different cfgs to build thirdparty.
If we build on local machine, not the gitflow, pack_xx.sh outputs are in the root path.
We can use ./pack_all.sh <work_dir> <tars_dir> to determine the pack work dir and the output tarballs' dir.
We will release two zk pkgs, thirdsrc.tar.gz & thirdsrc-mac.tar.gz, they're the same. It's to make Mac user easier to download all pkgs, they won't forget the thirdsrc tarball.

doxygen will be downloaded, but won't be built.

doxygen will be downloaded, but won't be built. Shall we delete it?

Yep

doxygen will be downloaded, but won't be built. Shall we delete it?

Yep

Done.

Do we have any other todos?

Do we have any other todos?

Yes, updated in the first comment.

gflags: cmake -DBUILD_SHARED_LIBS=OFF
glog: ./autogen.sh && ./configure --enable-shared=no

All finished, plz review this pr.

We'll get release like this: https://github.com/vagetablechicken/hybridsql-asserts/releases/tag/v1.1.2
We download zetasql, but have removed the build step. So I want to delete the download of zetasql, is it OK? @jingchen2222 @aceforeverd

Good Job!


support setting the last resource version in the reflector

Support setting the resource version in the reflector before resource synchronizatiion.
In the weak network environment, the start and stop of resource synchronization may be more frequent,
so setting the last resource version can avoid the full list

We need to refactor the reflector to use watch and then list to prevent the list from getting the full amount of resources in case of errors


Provide the commit message in the zip deploy API

The commit message exists in GITHUB_CONTEXT
Sample repo:
https://github.com/nertim/github-action-testapp-node
YML:

Commit message in output (line 231):

@AmrutaKawade @JasonFreeberg @spellegrino021 FYI

@Balaga-Gayatri is looking into this issue

Hi @nertim released the changes with this feature.


import problems

I've just encountered a problem when imporing the library:
import 'package:country_code_picker/country_code_picker.dart';
Your application could not be compiled, because its dependencies could not be established.
The following Dart file:
  C:\IT_Projects\ISALON\isalon_app\lib\ScreenRegister.dart
...refers, in an import, to the following library:
  package:country_code_picker/country_code_picker.dart
That library is in a package that is not known. Maybe you forgot to mention it in your pubspec.yaml file?

heres my pubspec:
dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^0.1.2
  flutter_statusbarcolor: any
#  barcode_scan: ^1.0.0
#  qr_mobile_vision: ^0.2.2
  country_pickers: 1.0.0

the package appears as:

any ideas how to fix and use it properly ? Id like to use dropdown selector. Thanks

Hi @itelixgithub The import looks problematic. Maybe you tried another package earlier. Here is the proper import to use this package:
import 'package:country_pickers/country_pickers.dart';

Dida a Flutter lean, manually deleted DArt package, then restarter, reinstalled and now it's importing properly. Thanks! 👍


Experiencing 401 Bad Credentials when credentials are valid

While synchronising data for our github organisation, we've been experiencing consistent HTTP 401 errors with the reason "Bad credentials" which should occur when an invalid token is provided.
Although, this happens after a lot of time into the synchronisation, after performing multiple successful API calls with the same credentials. According to this line of code a similar behaviour has been experienced (random 401 errors - triggering a retry), but not with the reason "Bad Credentials".
All failures experienced were while fetching team members stream, but always in different teams/members. Not consistently on the exact same call.
While trying a curl call to this endpoint using the same token and endpoint call that cause the failure, we receive a 200 OK.
Yet another detail which makes the behaviour even weirder is that if I do an execution with --select teammembers.*, the execution is successful without any failure. With the exact same token.
While an easy solution could be to changethis line of code to retry on any 401 call (not excluding bad credentials), this would cause the tap to retry, even when a bad token is effectively provided.
So, the only solution I see would be to retry on every 401 call, as long as an X amount of API calls was already successfully performed with the same credentials.

@dlouseiro Can you confirm if this is still an issue in the latest version of the tap? Thanks!


Differences between Windows Virtual Desktop and VMware Horizon Cloud?

[Enter feedback here]
I don't understand the difference between them. When to use VMware Horizon Cloud. When to use Windows Virtual Desktop.

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

ID: d2eeca14-80ef-c0b4-2719-80ceb286ba3d
Version Independent ID: b7d3df23-3158-2807-eecc-73e7230c2033
Content: What is Windows Virtual Desktop? - Azure
Content Source: articles/virtual-desktop/overview.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@cuongdnv2307 Thank you for sharing your query! We are currently investigating and will get back to you shortly on this.

@cuongdnv2307 Apologies for the delay in response and all the inconvenience caused because of the issue.
Did you got a chance to go through below links:
https://azure.microsoft.com/en-in/services/virtual-desktop/vmware-horizon-cloud/#documentation
https://www.vmware.com/in/products/horizon-cloud-virtual-desktops.html
https://www.nakivo.com/blog/key-differences-between-vmware-horizon-and-vdi/
Hope it helps!!!

@cuongdnv2307 Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


Set up presentation files

Intended change management:

Create upcoming presentation branch
Contribute to that branch until presenting
Merge to main after presentation to incorporate any live fixes

Files:

app/index.html <-- serves toc.js and defines TOC
app/toc.js <-- redirects to prev / next .md, uses index.html DOM as page order config
docs/index.md <-- title page with next link
docs/topic/some-detail.md <-- presentation "slides" with prev / next links

Prev / next links all target the app with the current file name, e.g. Prev | Next.  If no match, default to display the TOC.

Opted for a simple MD collection of files without magic redirecting.


Fix @liveblocks/react lint script

Description
Linting fails in package @liveblocks/react when running npm run lint due to the lint path being src/;.
Before
@liveblocks/react:lint:
@liveblocks/react:lint: ><EMAIL_ADDRESS>lint
@liveblocks/react:lint: > eslint src/; npm run check:exports
@liveblocks/react:lint:
@liveblocks/react:lint: Oops! Something went wrong! :(
@liveblocks/react:lint:
@liveblocks/react:lint: ESLint: 8.38.0
@liveblocks/react:lint:
@liveblocks/react:lint: No files matching the pattern "src/;" were found.
@liveblocks/react:lint: Please check for typing mistakes in the pattern.
@liveblocks/react:lint:
@liveblocks/react:lint: npm ERR! Lifecycle script `lint` failed with error:
@liveblocks/react:lint: npm ERR! Error: command failed
@liveblocks/react:lint: npm ERR!   in workspace<EMAIL_ADDRESS>
After
@liveblocks/react:lint:
@liveblocks/react:lint: ><EMAIL_ADDRESS>lint
@liveblocks/react:lint: > eslint src/ && npm run check:exports
@liveblocks/react:lint:
@liveblocks/react:lint: ><EMAIL_ADDRESS>check:exports
@liveblocks/react:lint: > tsc scripts/check-factory-exports.ts && node scripts/check-factory-exports.js
@liveblocks/react:lint:

See https://github.com/liveblocks/liveblocks/pull/950#issuecomment-1596608871.

Use changes in #950


Configuring component passed to Route

Currenly I have basic code like this:
const appProps = {
    title: 'my title',
}
React.render(<Application {...appProps} />, document.getElementById('app');

Now I am confused how to apply router to this. How can I pass these props to Application component?
const app = (<Router>
    <Route path="/" component={Application} />
</Router>);

React.render(app, document.getElementById('app'));

I tried <Route path="/" component={Application} {...appProps} /> which makes these props available on this.props.route.title, but that's not that useful, because at some other point I might want to render Application directly without Router.
I also tried using Application directly as children of a Router instead of using Route component, but adding path. Unfortunately my component is not even mounted that way.
Is there some other option to manage this?

Alright I've found a way thanks to this ... http://stackoverflow.com/a/29720488/902813
function createApplication(options = {}) {
    return class extends React.Component {
        render() {
            return (<Application
                title='my title'
                {...options}
            />);
        }
    };
}

export function renderApplication(options) {
    return (
        <Router>
            <Route component={createApplication(options)} path="/" />
        </Router>
    );
}


⚠ SceneTime has degraded performance

In b9a48d1, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 4226 ms

Resolved: SceneTime performance has improved in 39cd7ee after 6 minutes.


Add "about language" documentation

See https://github.com/exercism/exercism.io/issues/2299#issuecomment-123798119
/cc @mixolidia @sguermond
Each language track should optionally contain a file docs/ABOUT.md, which can contain the short intro to the language.
This might be enough to return an "about" key from the endpoint defined in lib/app/routes/docs.rb: http://x.exercism.io/docs/ruby
To do this, we'll need to add an ABOUT.md file to test/fixtures/problems/fake/docs/ and then work in these files:

lib/xapi/docs.rb
test/xapi/docs_test.rb

Thanks!


Exception handlers cause warnings when exceptions are disabled

C++ exceptions in my opinion (and in the opinion of others) are subpar and I usually have them turned off along with RTTI. However,
the presence of try/catch blocks in doctest causes warnings (at least on MSVC - C4530) which can't be suppressed (and I'm building with /W4 /WX so this breaks my build).
I simply deleted / commented out all usages of try/catch in the my copy of the library but it will be helpful to only use exceptions handling if the user's exceptions are enabled. This can be detected by checking for

__cpp_exceptions on gcc
_HAS_EXCEPTIONS on MSVC

try the dev branch and let me know if everything works fine - also checkout these docs in the dev branch

@onqtam Works as expected!


Fixes #6 by changing regex for grid row units and removing rounding

I believe this fixes the issue @custa1200 was experiencing but I've assumed the rounding for grid row units isn't needed anywhere else, please check.

Grid row units were limited to integers with the idea that Gridlover could guarantee that the grid always stays in sync, but I guess it can't be guaranteed because really anything could thow the grid off some amount. This is good, thanks!


Neovim missing libstdc++.so.6

Describe the bug
I'm on nixos unstable and I installed neovim 0.5 from the repository
To Reproduce

install neovim 0.5
create a profile with telescope in it
start a telescope function

Expected behavior
No error message
Additional context
Error executing vim.schedule lua callback: Failed to load parser: uv_dlopen: libstdc++.so.6: cannot open shared object f
ile: No such file or directory

Something similar happens when I edit a file:
Error detected while processing FileType Autocommands for "*":
E5108: Error executing lua Failed to load parser: uv_dlopen: libstdc++.so.6: cannot open shared object file: No such fil
e or directory

Notify maintainers
@Ma27 @rvolosatovs @manveru
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.47, NixOS, 21.11pre301044.87807e64a5e (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.12`
 - channels(root): `"nixos-21.11pre301044.87807e64a5e, home-manager"`
 - channels(asdf): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: neovim
# a list of nixos modules affected by the problem
module:

I guess something has to have had happened during compilation of the treesitter parsers because after cleaning out the packer folder and recompiling everything, all works well

Hi @yourfavouriteuncle. I'm facing the same problem here. I've tried removing the contents of .local/share/nvim but the issue persists. Could you share a more detailed explanation of what you did to get it fixed. Thanks in advance.

I have found a horrible workaround for this:
LD_LIBRARY_PATH=$(ls -d /nix/store/*gcc-10.3.0-lib)/lib nvim

I am also facing this issue. Can we please reopen?

Before, I only tried removing some of the directories from ~/.local/share/nvim, which didn't work. But now when I did like @lobre suggested and deleted (or moved, in my case) the entire folder, everything seems to work like it should after reinstalling packer.nvim and all the plugins and language servers again.

I'm seeing this problem, and removing ~/.local/share/nvim and re-installing treesitter parsers does not fix the problem. I'm not on NixOS, but using nixpkgs unstable on Ubuntu 11. Does anybody have any other fixes? Testing a compiled parser with ex. ldd ~/.local/share/nvim/site/parser/markdown.so shows:
        linux-vdso.so.1 (0x00007ffe107f1000)
        libstdc++.so.6 => /lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007f4a91a6a000)
        libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f4a918a5000)
        libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f4a91761000)
        /lib64/ld-linux-x86-64.so.2 (0x00007f4a91d04000)
        libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f4a91747000)

All those paths are valid, but I still see issues with using any parser linked against libstdc++.so.6 and ONLY parsers linked with libstdc++.so.6.
I tried just using nvim-treesitter.withPlugins (plugins: pkgs.tree-sitter.allGrammars), but there's query incompatibilities with vim.treesitter right now, specifically with markdown.

I'm in a similar position to @duckpuppy , running neovim via home-manager on Ubuntu 20.04.

Seeing this problem since updating to neovim 7.0

Seeing this problem since updating to neovim 7.0

Same here, did you find any workaround?


[main] Upgrade to latest dependencies

release -dprotaso
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing 89b588f...0310bf9:
  > 0310bf9 Fixing docs eventing spec details (# 6082)
  > 3b3eac1 Enable features configmap watching and fix mt broker filter subscriptions API implementation (# 6089)

/lgtm
/approve

/retest


Disconnect ISCSI Target should check for VMs on datastore before continuing.

Before disconnecting or removing an iscsi target there should be a check to validate that there are no VMs currently registered and using the datastore.  This causes orphaned objects and run command failures/timeouts.

@khensler-msft is there specific commandlet that's known to cause this or we just want to make sure it's broadly addressed?

It was reported via fasttrack that when disconnecting ESAN the tasks times out when attempting this operation when a VM exists on the datastore.  I believe it is the Disconnect iscsi target commandlet that fails.  It for sure does not have a check.  The disconnect NVME targets should already have the check in place.  Can probably use that as a template


ansible-playbook main.yml fails at Ensure Pihole is running step.

I changed the host list in the \internet-pi\config.yml file and per another post on this board, tried to re-run ansible-playbook main.yml to reload the list of hosts.  When I try running that command, it fails with the error below.  I am running the command from the internet-pi subdirectory.
TASK [Ensure Pi-hole is running.] **********************************************
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "errors": [], "module_stderr": "Starting pihole ... \nHost is already in use by another container\nStarting pihole ... error\n", "module_stdout": "", "msg": "Error starting project Encountered errors while bringing up the project."}
When I go to my grafana dashboard I get the new hosts I added to the config.yml file but I also still see the hosts that were in the sample config file as well as a couple host names I put in the Pinghosts.yaml file under prometheus directory.  Even when I remove the pinghosts.yaml file entries and completely remove the sample config.yaml file from the directory, I still see all the old and the new host in the grafana dashboard. I copied a picture.  The only hosts in the config.yml file are the https://<IP_ADDRESS> and <IP_ADDRESS>.
Thank you in advance for any help you can provide.

Please disregard.  It was showing the old ones because there was still data in the view.  As soon as I got more history, just the 2 IP addresses are showing up in the panel now.  If you know what to do to fix the ansible-playbook main.yml error for future updates that I would need, please let me know.

@david99352 The pihole issue you are experiencing is likely due to port 53 being already in use by dnsmasq. If you disable the dnsmasq service, Pihole will be able to utilize that port and start successfully.


ProcessBuilder::inheritIO works differently between Gradle and Java

I discovered today that if I run a Java 11 app on macOS that uses the following code...
new ProcessBuilder("echo", "hello")
    .inheritIO()
    .start()
    .waitFor();

...and if I run an equivalent custom Gradle task...
tasks.register("echo") {
    doLast {
        ProcessBuilder("echo", "hello")
            .inheritIO()
            .start()
            .waitFor()
    }
}

...then the Java app prints "hello" to the console, whereas the task does not.
Is this behaviour intended? If anything, I'd personally expect the two pieces of code to produce the same result.
Expected Behavior

Running either ./gradlew echo or ./gradlew run should output "hello" to the console.
Current Behavior

Running ./gradlew run outputs "hello" to the console, but running ./gradlew echo outputs nothing.
Context

I'm working on a closed-source Android project where we have a Bash script that spins up an Android emulator. I'm trying to create a Gradle task that runs this script before our emulator-related tests, so that we don't have to remember to start the emulator up first.
However, the Bash script has this instruction:
emulator -avd $EMULATOR_NAME -no-audio > /dev/null &

...and when I run the script with this Exec task:
tasks.register<Exec>("setupEmulator").configure {
    commandLine = listOf("$projectDir/scripts/emulator-setup.sh")
}

...it spins up the emulator just fine, but then it freezes. I assume this is because Gradle is waiting for the emulator child process to finish even though it's in the background, but I'm not sure. I'll describe this in more detail in another bug report.
So as a workaround, I'm using ProcessBuilder directly. However, when running in Gradle, it fails to print the echos in the rest of our Bash script, which is how I narrowed down the problem in Current Behavior.
Steps to Reproduce

git clone https://github.com/jbduncan/gradle-processbuilder-repro
cd gradle-processbuilder-repro
Run ./gradlew echo and then ./gradlew run. Both should output "hello" but only run does.

Your Environment

Build scan URL: https://scans.gradle.com/s/jj3gw7aompkv6

The aforementioned bug report is: https://github.com/gradle/gradle/issues/16721

I'd love for this to be tackled at some point.

In AndroidX we encountered this too: we eventually discovered that the output of the process wasn't getting output by Gradle when there was a case where our subprocess failed. See https://android-review.googlesource.com/c/platform/frameworks/support/+/2221307/1/buildSrc/private/src/main/kotlin/androidx/build/studio/StudioTask.kt#223


visibleDates call after reloadData with comletionHandler error

I'm using 6.1.2 version of JTAppleCalendar
The problem:
I'm using something like this in viewDidLoad()
loadDataFromServer {
        self.calendar.reloadData() {
                self.calendarView.visibleDates { (visibleDates: DateSegmentInfo) in
                    .....
                }
        }
}

This line of code crashes the app. It worth to mention I was on swift 3.0 and there was no problem, then I updated it to 3.0.2 and JTCalendar too. Since then I got this crash. I attach images of that crash

OK, this should not crash your app.
Your loadDataFromServer takes about how many seconds?
I am trying to re-create this on my end in my example application.
If you can recreate the issue there on the example application, this will be very helpful as well.
But if you cant, its cool. I'm investigating.
Also, I have swift 3, not swift 3.0.2. I'm not sure of the changes in there yet.
Will have to look into it.

I recreated it
calendarView.reloadData {             self.calendarView.reloadData {                 self.calendarView.visibleDates { (visibleDates: DateSegmentInfo) in                     self.setupViewsOfCalendar(from: visibleDates)                 }             }         }
in the viewDidLoad at example app

OK. I can re-create the error now. Looking into it

I think this issue is now fixed from this commit -> https://github.com/patchthecode/JTAppleCalendar/commit/f3685534671d3dc79b1092df6ed37ca13dfbce70
Let me know if you are still experiencing this.


Add braces to staging table name to make is safe for reserved characters

The NO_DUPLICATES strategy has been problematic for some as this strategy uses the appId to create staging tables.
The appId commonly contains dashes ( - ) and this then results in a failure: Incorrect syntax near '-'.
Adding braces around the name of the table will help mitigate this issue.
Relates to https://github.com/microsoft/sql-spark-connector/issues/22

Duplicate


⚠ Bofrost.es has degraded performance

In 08e810f, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1801 ms

Resolved: Bofrost.es performance has improved in 43cad9c after 43 minutes.


ImportError when test name is too long

I have a kivy project stored on my machine at /Users/............../Personal/grocery (my name .-ed out, but with the right number of .s), and my unit tests are stored in this project in the tests folder.  I had one test called 'test_recipe_availability.py' (so the full file path is /Users/............../Personal/grocery/tests/test_recipe_availability.py) and when I run python -m kivyunittest --folder tests I get the following error:
python -m kivyunittest --folder 'tests'
======================================================================
| test_recipe_availabilit                                        LOG |
======================================================================
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/joshuabrowning/anaconda/lib/python2.7/unittest/loader.py", line 91, in loadTestsFromName
    module = __import__('.'.join(parts_copy))
ImportError: No module named test_recipe_availabilit

When I rename the test to test_recipe_avail.py, everything works fine.

I have no limitation set there, but I think it's the Windows that interferes. There's a limitation for the length of a filepath and I think you might have reached it. Try to put it into e.g. C:\ and keep the long name for your test, it should work.


Invalid seeds, address must fall off the curve

there is en error when i debugger the code.
const tokenAccountAddress = await getAssociatedTokenAddress(
mint,
owner,
false
);

I would suspect either the mint or owner address is incorrect, but it’s hard to say more without having more information.


Validate that either Version Name or Version Code is provided

If neither is provided then the step is pretty much just a "no-op" and can be confusing why it's Green even though it does not do anything.
Additionally, it's possible to just set the Offset input, expecting that it will offset the value set in the build.gradle file, but of course that offset is a different offset, that's not how the step works (the offset it only applied on the value provided as the step's input, the build.gradle file's original version infos are not used at all).
E.g.
+------------------------------------------------------------------------------+
| (2<EMAIL_ADDRESS>                       |
+------------------------------------------------------------------------------+
| id: change-android-versioncode-and-versionname                               |
| version: 1.0.0                                                               |
| collection: https://github.com/bitrise-io/bitrise-steplib.git                |
| toolkit: go                                                                  |
| time: 2018-02-07T15:35:14Z                                                   |
+------------------------------------------------------------------------------+
|                                                                              |

Configs:
- BuildGradlePth: REDACTED/build.gradle
- NewVersionName:
- NewVersionCode:
- VersionCodeOffset: 100000000

Updating versionName and versionCode in: REDACTED/build.gradle

0 versionCode updated
0 versionName updated
|                                                                              |
+---+---------------------------------------------------------------+----------+
| ✓ |<EMAIL_ADDRESS>            | 2.16 sec |
+---+---------------------------------------------------------------+----------+

What is the point of using a VersionCodeOffset if doens't use the gradle version code value?
If my current version code is 57 and I set:

NewVersionCode: 58
VersionCodeOffset: 1

What should I expect? I just want to increment +1 on my current version code automatically.

AutoIncrement Shouldn't be sommething like this?
(...)
regexp.MustCompile(`^versionCode (?P<version_code>.*)`): func(line string, lineNum int, match []string) string {
			oldVersionCode := match[1]
			//finalVersionCode = oldVersionCode
			updatedLine := ""

			if configs.VersionCodeOffset != "" { //if configs.NewVersionCode != "" {
				//finalVersionCode = strconv.Itoa(newVersionCode)
                                //updatedLine = strings.Replace(line, oldVersionCode, finalVersionCode, -1)
				updatedLine = strings.Replace(line, oldVersionCode, oldVersionCode + offset , -1)
				updatedVersionCodes++
				log.Printf("updating line (%d): %s -> %s", lineNum, line, updatedLine)
			}

			return updatedLine
		},

(...)

@KunamiPT this step is not designed to read/evaluate the build number, nor to bump it. It's meant to set it to a specific value.
Usual use case is to set it to $BITRISE_BUILD_NUMBER, which is the build number on bitrise.io
More infos why: https://stackoverflow.com/questions/33821137/build-versioning-in-continuous-delivery/33821876#33821876

The offset is mainly used to specify an offset to this base build number, for simplicity (e.g. something like: $BITRISE_BUILD_NUMBER +100)

We have released a new version (1.1.0) of Change Android versionCode and versionName
Changes:

If neither NewVersionCode or NewVersionName are provided the step will fail

Thanks @BirmacherAkos , closing it! ;)


Microsoft Azure integration

[ ] T2 should be able to create AKS clusters
[ ] Please check if this is an option for us to save some money: https://opensource.microsoft.com/azure-credits#credits-apply

@lfrancke Do we have an Azure account already? I had a look at the link to the "open source credits" stuff. It says

Applications to the program will be reviewed and you will receive a notification of the decision within 3-4 weeks.

Maybe we don't want to wait that long?

We do. Please ask @razvan for access.
No, we don't want to wait that long but if you could take a look at the requirements and decide whether it makes sense to apply anyway that'd be great.

Thanks, I'll ask him.
Regarding the credits: I looked into the questionnaire and it contains some business-related questions that are preferably answered by someone else.

https://docs.microsoft.com/en-us/azure/developer/terraform/authenticate-to-azure?tabs=bash