PrefixSpan can't find certain patterns

Currently Prefix-Span won't generate all possible sub-sequences of a pattern, which means that it is not a correct implementation of the algorithm as put forward by Pei et al.
To illustrate, say we have as output sequence [node1, node2, node3, node4, node5]. In addition to the sequence itself, Prefix-Span will find all sub-sequences except:

[node2, node3, node4, node5]
[node2, node3, node4]
[node3, node4, node5]

It will find entire sequences, but not all sub-sequences. So at the moment, it is sufficient to generate test cases. However this should eventually be fixed as it is not a proper implementation of the algorithm, and does not function as it should/how we expect it to.

I've experimented a bit further with longer and smaller paths, and this seems to be the pattern (no pun intended): The algorithm finds all subpatterns of lengths 1 and 2, without issues. For all subpattern-lengths greater than 2, it only finds the subpatterns starting on the first node of the frequent sequence (as opposed to those that start somewhere in the middle). This means that it finds exactly one of length 3, 4, and so on. I haven't found a solution yet, but this might help :)

I had found something along those lines as well, but thank you for sharing! I will look into it further soon.

It seems that the issue is that the condition pathContainsPrefix checks for both the prefix plus the item and the last item of the prefix plus the item. However the body if the if makes no distinction between these cases meaning the new prefix is always the entire prefix plus the item and never the last item of the prefix plus the item.

If I modify parts of the PrefixSpan algorithm to read like this:
    private fun runAlgorithm(prefix: List<Node> = emptyList(), projectedPaths: Collection<List<Node>> = allPaths) {
        frequentItems.forEach { frequentItem ->
            if (projectedPaths.any { pathContainsPrefix(it, prefix, frequentItem) || pathContainsLastOfPrefix(it, prefix, frequentItem) }) {
                val newPrefix = prefix + frequentItem
                frequentSequences += newPrefix

                runAlgorithm(newPrefix, extractSuffixes(prefix, allPaths))
            }
            if (projectedPaths.any { pathContainsLastOfPrefix(it, prefix, frequentItem) }) {
                val newPrefix = prefix.last() + frequentItem
                frequentSequences += newPrefix

                runAlgorithm(newPrefix, extractSuffixes(prefix, allPaths))
            }
        }
    }

    private fun pathContainsPrefix(path: List<Node>, prefix: List<Node>, frequentItem: Node) =
        pathContainsSequence(path, prefix + frequentItem, comparator)

    private fun pathContainsLastOfPrefix(path: List<Node>, prefix: List<Node>, frequentItem: Node) =
        prefix.isNotEmpty() && pathContainsSequence(path, listOf(prefix.last(), frequentItem), comparator)

... we get 14 out of 15 patterns. The pattern we're missing for length 5 is (2, 3, 4, 5). These are more patterns but unfortunately the change also produces a StackOverflow exception elsewhere. I haven't found the cause yet...

Hmm this does seem weird. It seems to be that the second if will never be true if the first if wasn't true. And the condition of the first if is functionally identical to the current one, with the only difference being that the function has been split in two (pathContainsPrefix and pathContainsLastofPrefix) and that the or operator has been lifted out.
I can't really figure out why more patterns are found here.

As PrefixSpan is no longer part of this repository, I have created an issue there and I'll this one.


WebSharper.CodeMirror <IP_ADDRESS>9/<IP_ADDRESS>8  require inexistent versions of WebSharper

Hi guys, there seems to be something amiss with the WebSharper dependencies.
Paket has trouble resolving dependencies:

Paket reports that:

WebSharper.CodeMirror <IP_ADDRESS>9 requires WebSharper <IP_ADDRESS>
WebSharper.CodeMirror <IP_ADDRESS>8 requires WebSharper <IP_ADDRESS>

However, the latest available WebSharper seems to be <IP_ADDRESS>

Questions:

Have the WebSharper.CodeMirror packages newer than <IP_ADDRESS>7 been published by mistake?
Should WebSharper <IP_ADDRESS> (and the other corresponding packages) have been published but haven't?

For now I am manually restricting WebSharper.CodeMirror versions to be < 3.6, but I wanted to let you know there may be an issue lurking in package management.

Note that I did try to post this on the forums first, but posting repeatedly failed with "Oops... Something unexpected happened" (using Chrome - logging in does not work in IE 11).

Thanks!

Just pulled the trigger on a public release, should be out there shortly.
Not sure what happened with fpish, restarted it, now seems better.

Thanks @granicz - the nuget package dependencies are OK now.


[InstanceMethodStasher] Say method is owned by class when the owner i…

…s the object
Closes #1143

Nice find, thank you, would you be able to provide just a bit more assistance and then I'll get this merged.

Given this works around a 1.8.7 bug and we will drop support for 1.8.7 in RSpec 4, it would be nice if this part of the conditional was applied only on 1.8.7 so we don't keep it indefinitely.

@myronmarston I was thinking the extra comment would be sufficient for us to remove it in 4.

Or would you prefer something like:
diff --git a/lib/rspec/mocks/instance_method_stasher.rb b/lib/rspec/mocks/instance_method_stasher.rb
index 8a113db6..679bd0d1 100644
--- a/lib/rspec/mocks/instance_method_stasher.rb
+++ b/lib/rspec/mocks/instance_method_stasher.rb
@@ -114,6 +114,8 @@ module RSpec
         # `#<MyClass:0x007fbb94e3cd10>`, rather than the expected `MyClass`.
         owner = owner.class unless Module === owner

+        return true if owner == @klass
+
         # On some 1.9s (e.g. rubinius) aliased methods
         # can report the wrong owner. Example:
         # class MyClass
@@ -128,7 +130,9 @@ module RSpec
         # Hence, we verify that the owner actually has the method defined.
         # If the given owner does not have the method defined, we assume
         # that the method is actually owned by @klass.
-        #
+
+        return true unless method_defined_on_klass?(owner)
+
         # On 1.8, aliased methods can also report the wrong owner. Example:
         # module M
         #   def a; end
@@ -138,7 +142,9 @@ module RSpec
         # end
         # The owner of M.b is the raw Module object, instead of the expected
         # singleton class of the module
-        owner == @klass || owner == @object || !(method_defined_on_klass?(owner))
+        return true if RUBY_VERSION < '1.9' && owner == @object
+
+        false
       end
     end
   end

If we do modify this with a conditional, simply adding:
diff --git a/lib/rspec/mocks/instance_method_stasher.rb b/lib/rspec/mocks/instance_method_stasher.rb
index 8a113db6..679bd0d1 100644
--- a/lib/rspec/mocks/instance_method_stasher.rb
+++ b/lib/rspec/mocks/instance_method_stasher.rb
@@ -138,7 +142,9 @@ module RSpec
         # end
         # The owner of M.b is the raw Module object, instead of the expected
         # singleton class of the module
-        owner == @klass || owner == @object || !(method_defined_on_klass?(owner))
+        return true if RUBY_VERSION < '1.9' && owner == @object
+        owner == @klass || !(method_defined_on_klass?(owner))
       end
     end
   end

Should suffice

@JonRowe done!

Thanks!


Adding option to enable insights for replica instances

GCP now has option to enable Query Insights also for replica instances and this PR will enable it to set this via terraform

Thanks for the PR! 🚀✅ Lint checks have passed.


Warrior simulations end with an error

Describe the bug
Every time I run a simulation on either the fury or arms warrior sims the sim finishes (or apperas to finish the iterations) and then comes up with an error that says "Something went wrong running your raid sim. Reload the page and try again."
To Reproduce
Steps to reproduce the behavior:

Go to 'https://wowsims.github.io/cata/warrior/arms/' OR 'https://wowsims.github.io/cata/warrior/fury/'
select any tab
click run sim
See error in bottom right

Expected behavior
the simulation finishes and the dps output is displayed in teh bar on the right with the usual save reference options, results tab display results of which abilites were used and how much damage and dps they did
Sim Links and Screenshots
https://i.imgur.com/H0NG6fO.jpeg
Desktop (please complete the following information):

OS: Windows 10 Pro
Browser Chrome
Version 124.0.6367.210 (Official Build) (64-bit)

I believe this issue is tied to selecting Blood Elf as the race; other races appear to function fine.


Does ComputeLibrary support be built on windows system

Hi, I saw the supported system of ComputeLibrary was not include windows system. Is it possible to build ComputeLibrary with cmake and virtual studio 2019 on windows system. If not, whether there are plans to support it, since the arm-based PC will release soon.

Hi @laoniangjiu
It is possible to build on windows but I would suggest using Linux or MacOS because all the tools used to build the library are unix tools.
Please take a look at the documentation: https://arm-software.github.io/ComputeLibrary/v21.05/how_to_build.xhtml#S1_6_windows_host
Hope this helps.


Unable to use Storage Explorer on laptop

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
latest
Regression From
first time download
Architecture
x64
Storage Explorer Build Number
No response
Platform
Windows
OS Version
No response
Bug Description
I get the following message when I click on any resource in the explorer to get details on the resource:
Unable to retrieve child resources
ProducerError:{
"name": "FetchError",
"message": "request to https://axwaysftp.file.core.windows.net/?comp=list failed, reason: self signed certificate in certificate chain",
"stack": "FetchError: request to https://axwaysftp.file.core.windows.net/?comp=list failed, reason: self signed certificate in certificate chain\n    at ClientRequest. (C:\Users\T4263SA\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\node_modules\@storage-explorer\file-extension\dist\src\index.js:2:154871)\n    at ClientRequest.emit (node:events:513:28)\n    at TLSSocket.socketErrorListener (node:_http_client:502:9)\n    at TLSSocket.emit (node:events:513:28)\n    at emitErrorNT (node:internal/streams/destroy:151:8)\n    at emitErrorCloseNT (node:internal/streams/destroy:116:3)\n    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)",
"type": "system",
"errno": "SELF_SIGNED_CERT_IN_CHAIN",
"code": "SELF_SIGNED_CERT_IN_CHAIN"
}
Steps to Reproduce
1.Launch storage explorer
2.Launch Blob store
Actual Experience
I get the following message when I click on any resource in the explorer to get details on the resource:
Unable to retrieve child resources
ProducerError:{
"name": "FetchError",
"message": "request to https://axwaysftp.file.core.windows.net/?comp=list failed, reason: self signed certificate in certificate chain",
"stack": "FetchError: request to https://axwaysftp.file.core.windows.net/?comp=list failed, reason: self signed certificate in certificate chain\n    at ClientRequest. (C:\Users\T4263SA\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\node_modules\@storage-explorer\file-extension\dist\src\index.js:2:154871)\n    at ClientRequest.emit (node:events:513:28)\n    at TLSSocket.socketErrorListener (node:_http_client:502:9)\n    at TLSSocket.emit (node:events:513:28)\n    at emitErrorNT (node:internal/streams/destroy:151:8)\n    at emitErrorCloseNT (node:internal/streams/destroy:116:3)\n    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)",
"type": "system",
"errno": "SELF_SIGNED_CERT_IN_CHAIN",
"code": "SELF_SIGNED_CERT_IN_CHAIN"
}
Expected Experience
I was supposed to see what data the blob container has
Additional Context
I need to startusing Storage explorer

Have you tried to diagnose the issue following the steps in the Storage Explorer Troubleshooting Guide?  https://learn.microsoft.com/en-us/troubleshoot/azure/azure-storage/storage-explorer-troubleshooting?tabs=Windows#ssl-certificate-issues

Are you using a proxy? You may find that changing your Proxy Configuration setting to "Use system proxy" will also help:

Closing due to lack of response. If you are still encountering this issue we recommend opening an Azure support ticket. Alternatively you can open a new issue here. This issue will no longer be monitored.


fix(stack/proxy): ensure wsProxy and httpProxy have correct type

Both, httpProxy and wsProxy inside Proxymanager have been assigned
the value of new Proxy(...).serve() which is an instance of express.
This was wrong and causes runtime errors when proxies are being stopped as
they don't have a stop() API.
What did you refactor, implement, or fix?

How did you do it?

Is there anything that needs special attention (breaking changes, etc)?

Questions

Review

Cool Spaceship Picture
<!-- Star Trek/Wars, n/BSG, Voltron, Transformers... all welcome, but not all equally cool ;-) ->

@emizzle updated accordingly!

Closing in favour or #2318


Couldn't install assembly to GAC

The latest 2.2.3 is not signed and doesn't have a strong name, so couldn't be installed to GAC. I have verified all versions below with ILSpy and found that the latest signed is 2.1.6 after which the project was migrated to NETCORE. I suspect that migration has a side effect of loosing signing. Please verify.

I cannot sign my application that uses this library because it is not signed. Any chance on turning signing back on?

I've re-enabled signing with this commit: https://github.com/Vodurden/Http-Multipart-Data-Parser/commit/a9350c105b5baf00928d51fa3b83c64f3816b58c
You're right that we lost signing with the transition to .NET core. I've just published v2.2.4 which should be correctly signed.
I'm going to close this but if you find any issues around signing with 2.2.4 let me know and I'll re-open and investigate.


replace /licenses listings with spectrum

(on home page, non-hidden) -> (agplv3 to unlicense)

emphasize macro (spectrum of oss licenses) rather than micro (families/variations) structure
rm <100% accurate 'family'/'variant', and 'featured'/'hidden' fields no longer needed to control display
closes #331 (not a pure fix; feel free to open new issue)
reverse order as on home page, #335 claims order potentially privileges

@mlinksva would you be able to post a few screenshots and talk a bit more about the motivation? Suspect this is the right move, but hard to tell from a few low-level bullets.

The first bullet "emphasize macro (spectrum of oss licenses) rather than micro (families/variations) structure" is the big picture motivation.
The choices between GPLv*, LGPLv* and MIT/ISC/BSD* aren't ones someone who doesn't already know about should be confronted with when there's a big picture to get. Eclipse and Artistic aren't licenses anyone should choose unless they already know about them because they're in the relevant communities.
before (left) after (right) screenshots

Am I missing something? How does one get to the full listing of licenses? I believe the summary in terms of permissions/conditions/limitations is very useful, and even though I get the reasoning for not providing an exhaustive list (of the project as a hole, and of this change in particular), I think completely hiding such useful summarized views from visitors does a disservice to them, as that information is hardly available elsewhere.

@waldyrious there never has been a full listing of licenses presented on the site. I can't find it now, but I'm recall suggesting or reading a suggestion that every license we have a page for be linked at the end of /licenses for discoverability purposes. I don't feel strongly one way or the other whether that's a good idea so I haven't opened an issue or implemented, but it'd be easy to do.

every license we have a page for be linked at the end of /licenses for discoverability purposes.

:+1:

Ah awesome thanks. A few questions:

Why are we going from most restrictive to least restrictive? Assuming people have a short attention span, shouldn't we default to the most open?

It looks like we're no longer displaying some of the licenses currently displayed. What's the rationale behind that? Why those licenses? Are we now "recommending" a smaller subset of (family-agnostic) licenses?

Was there a discussion about Unlicense versus CC0 a while back? I can't seem to find it. Thinking maybe we should prefer CC0?

It'd require some small design effort, but what if we made the continuum explicit with a TLDR on the side of the delta between each licenses (or at last "more restrictive" and "least restrictive"). E.g., between unlicense and MIT, you retain copyright. Between MIT and Apache, you also grant patents, etc.

(a) first bullet "reverse order as on home page, #335 claims order potentially privileges"; re short attention span -- home page has correct order from your perspective -- but an aside: I wouldn't phrase as "more open", if I were trying to be neutral (b) unconditional licenses least well developed/accepted of spectrum (eg neither CC0 nor Unlicense on OSI list) so would rather not start there as much as I personally favor

Except for CC0 the ones that would no longer appear on /licenses are covered by above "The choices between GPLvN, LGPLvN and MIT/ISC/BSD-x aren't ones someone who doesn't already know about should be confronted with when there's a big picture to get. Eclipse and Artistic aren't licenses anyone should choose unless they already know about them because they're in the relevant communities." CC0/Unlicense choice is similar -- to the MIT/ISC/BSD-x one -- cultural baggage, not substantive or helpful to present learner with.

Unlicense is defaulted to in the present UI, so just going with that. CC0 is more problematic than Unlicense due to CC0's explicit patent non-grant.

I've got zero design skills, but would be happy to see something like that. I think the decreasing number of conditions is already a pretty strong visual indicator of what is happening.

I like the new design. It allows you to compare very quickly what you want and not just choose something you know (MIT or GPL2 for instance). Thanks!

I wish I could +100 instead of +1. This is a major improvement.

It takes what was a complex, confusing 2-factor system and makes it a single linear list with one factor.
On one side is the most protective of openness, on the other side is the most permissive.
Being the opposite order as the homepage provides a balance between which side of this list is shown first.
It's visually easy to compare
removing Artistic and Eclipse is definitely correct. There's ZERO sense in recommending those to anyone who isn't already planning to use them.
removing Public Domain in favor of Unlicense is alright

It would be ideal from a design perspective if the one new bullet for each item going from top to bottom was highlighted somehow to indicate "moving up to this adds this bullet" more visually. But I think that would probably fit into a more full redesign.
Note of concern: as much as I favor going with the updated latest licenses, it remains the case that GPLv2 is indeed still used and provides some compatibility, especially as GPLv2+, that GPLv3 doesn't. Incidentally, this is less relevant to AGPL (which had no v2) and LGPL (which is less common and less compatibility issues). So, I would recommend that in the one case of GPLv3, there be an extra little link that would flip open an explanation about GPLv2 saying that it's still in use, and there are some compatibility reasons to stick to GPLv2 even though it lacks the updates of GPLv3; and there could be a mention about the "or later" option for all the GPL family somewhere. I don't think this stuff belongs in the core list, but for someone trying to make sense of things from this site, at least having a link to explain about the version situation with GPL is warranted.

Another idea to consider: have a "similar licenses" dropdown under various things. This does not distract from the linear list because this would only be used where it really is somewhat inconsequential. Basically the idea would be to state that, e.g. BSD-2 and BSD-3 are effectively interchangeable with MIT, because this is not strictly about choosing a license, it can also be about learning to recognize licenses. I.e. I'd hope people can quickly see BSD licenses in use and recognize "oh, that's comparable to MIT in that spectrum of licenses". I'd only support this for BSD and maybe any other really popular licenses. And I'd not mind if this was rejected.

I don't really buy into any argument based on popularity. But I'd like to see a list of "similar lics" tags as well. However I'd simply link to wikipedia or whatnot because maintaining nitpicky differences between them will be hard.

@ionas agreed overall. @mlinksva work here is great, and for anything beyond this with more really thorough stuff, it could make sense to just link to the well-designed page at https://www.gnu.org/licenses/license-list.html#SoftwareLicenses

Don't get me wrong, I'd personally link gnu.org but I think to be on neutral ground I'd rather link to the license entries on Wikipedia. There is also multi language options available over there for most of them.

that particular gnu.org page is just superior to about any other in its clarity and thoroughness.
Anyway, to not get too tangential, my concern about GPL version from https://github.com/github/choosealicense.com/pull/386#issuecomment-207825053 above is outstanding too

I don't think educating users about why they might want GPLv2+ instead of GPLv3(+) is a net win, at least on /licenses. Talk about a lot to digest. I'd love to swap in copyleft-next at some point, when it has a version that meets criteria for including in the site. There's a related unfortunate issue that I have yet to open but will soon...

removing Artistic and Eclipse is definitely correct. There's ZERO sense in recommending those to anyone who isn't already planning to use them.

How? Surely the value of a license stands on the merit of what the license allows you to do/restricts, and if somebodies interest falls squarely in line with an artistic license, then..... surely, they should use artistic, because it reflects what they want.

@kentfredric you didn't provide any concrete reasoning here. Are you even aware of a single aspect of the Artistic license that would make it a better fit than others already on the list? Your comment could be applied to any of hundreds of licenses that you don't even bother to read. Most licenses are redundant. Do you actually know of a single real-world case where Artistic and Eclipse licenses do something someone wants over the licenses this change keeps, other than being the license a particular community already uses?

@kentfredric (posted a bit hasty initially), the trouble is that there are hundreds of licenses if someone were to pick just right one for each preference. This list is necessarily a subset and small selection. Having re-read Artistic v2, I see it has some merit as being between MIT and MozillaPL, but still not sure it really is valuable to be included.

Artistic v2, I see it has some merit as being between MIT and MozillaPL

Apache License 2.0 fills this space. Note at the granularity currently described by permissions/conditions/limitations fields here, these two licenses are exact matches. Of course the licenses have subtle differences, but I don't see any that are worth calling out for people who aren't license nerds, something which contributors to open source should not be required to become.
I've no objection to anyone using Artistic 2.0 if it's what they want, but listing it on /licenses just makes the page noisier.

Noted over in https://github.com/github/choosealicense.com/issues/434, but similar to https://github.com/github/choosealicense.com/pull/398, there's value in exposing the hidden field (and using it for the site's templating, rather than hard-coding preferences, to distinguish license prominence.

@mlinksva is there an issue already about re-adding CC0 to the UI? Removing it has killed the usefulness of the github new repo license UI for me and collabs. The unlicense is much more problematic than CC0 in its lack of fallback for jurisdictions where total public domain dedication is not possible. (happy to discuss in detail)


Add functions for returning a single tile

This adds two functions: getSingleTile and getSingleTileAtAnotherScale.  These functions take exactly the same parameters as tileIterator, but will return the first tile from the iterator.
tileIterator now takes an optional parameter tile_position which may be a 0-based integer which is functionally the same as iter(list(tileIterator(...))[tile_position]).  tile_position may also be a dictionary of either {i, j} or {x, y}, where {i, j} are the 0-based tile positions within the iterator, and {x, y} are the level-based locations of the tile at the iterated level.
The tile information yielded by the iterator now includes tile_position and iterator_range which include information about the tile position in the iterator and the ranges of x, y, i, j, and position that the iterator produces.
This resolves issue #100.

@cdeepakroy Let me know if you want any changes.  In the simplest case, call getSingleTile with the same parameters as tileIterator, and it will return the first tile.

@manthey Few comments. Rest LGTM!

LGTM!


Errors in reCAPTCHA validation: invalid-input-response

There appears to be a bug in 8.5.5 with reCAPTCHA. If a user takes more than a few minutes to fill out a form you get an error "Incorrect captcha code." and "Errors in reCAPTCHA validation: invalid-input-response" show up in the logs. This began happening on at least 2 sites I manage after upgrading from 8.5.4 to 8.5.5. At least one other person has seen this problem and reported on it in the forums.
https://forums.concretecms.org/t/errors-in-recaptcha/876

https://github.com/concrete5/concrete5/issues/9780

Thanks @ccmEnlil. Odd I didn't see that issue when I searched.


refactor: remove header theme

#454  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


Update free-programming-books.md

Added a new book of Coding interviews freely available on Research Gate.
Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

Please add author names where appropriate.
For Researchgate, we need evidence that the distribution is authorized.
When submitting PRs, please use a title that tell what it is doing


Wasm Telemetry 1.7: provide latency/cpu/memory data

Wasm telemetry is enabled in 1.7 builds to get test and stability mileage.
Provide latency, stability, cpu and memory data of this experiment.
Based on the results we will continue the new defaults or switch back to null vm.
Add release notes based on the outcome.

Seems like this issue is only about gather data for Wasm telemetry v2 vs Nullvm? I believe the data is already gathered and we decided to not ship it as default by 1.7 due to elevated memory (increased by 50%). I will close this, we can create a new issue to track performance optimization for Wasm extensions.


Ability to set resource attributes on the telemetry trait

Requirement
As I developer, I wish I could add resource attributes to my telemetry spans using the the telemetry add on trait. Currently I can do it using the camel trait:
camel:
  runtimeVersion: 3.2.3
 properties:
  - quarkus.otel.resource.attributes=service.name=My Service,product=My Product,tenantId=123

Problem
There isn't a field in the telemetry trait to do so
Proposal
Include a resourceAttributes field in the telemetry trait.
Open questions
No response

Same as expressed in https://github.com/apache/camel-k/issues/6002#issuecomment-2555341628 - I think that we should avoid just translating camel properties in the future in order to be able to simplify long term maintenance.


Fix unable to change directory in user access mode

Fix for #58605
Set it to use the user directory specified
Noticed that it currently ignores custom user data directory set in project settings, needs testing, might be a separate issue that affects more than Windows builds

The fact that other modes don't require explicit change_dir() suggests that the fix should be done somewhere else.

Yeah I think so but dir_access defaults to the current directory the game is started in, which works on Linux because that's what the user directory defaults to, but on Windows, it defaults to a folder in AppData for some reason. So without adding a case somewhere else, this is what I thought would be fine, unless we change the default folder on Windows.

Thanks!

Hi, I have this same bug in 3.5 beta3 !

Hi, I have this same bug in 3.5 beta3 !

Please open a new issue with a minimal reproduction project attached.


Remove html text replacer

The replace function that replaces newlines with break tags does not seem to have a good use and breaks other things (see issue #1663 for example). I think the replace function was meant to improve the table layout, but as markdown tables are not supported by commonmark, it only causes formatting issues right now.

I couldn't find a reason for the  replacement to be there. Not to say there isn't one, but it seems to have gone undocumented and is now lost to history.
The lines (of which there are now two) were originally added in SDK commit dc8498b57eec762f48f0b8826914141151960959, which message merely says "Fix the \n issue in markdown".
I looked for contemporary issues in github, but found none that applied.
The tables comment seems unrelated and was added later along with some code. See #166.
Let's remove this line and find out what happens on various phones and versions of Android. Worst case, we can simply put it back.


Create symlinks into psiflow_internal directory

see #54
Turn off with  make_symlinks: False in psiflow config.
Also removed option to supply custom path (fixed to psiflow_internal )

updated .gitignore, otherwise LGTM


How to ensure latest appium settings app(which is 4.0.0 now) is installed when initialize driver

I use latest Appium desktop and latest Appium-Python-Client
But after initializing driver, 3.4.0 version of io.appium.settings app is installed rather than latest 4.0.0

I know how to get current version. But I wanna know whether exist io.appium.settings apk version configuration item in Appium(eg. in desired capability)
or if I want to use latest version of io.appium.settings apk, I must clone this repo and build an apk.
And every time after I init driver webdriver.Remote("http://localhost:4723/wd/hub", desired_caps), need manual check version matched and then install?

it depends on the package version of uia2/espresso driver. Each package has proper settings app version as their dependencies, so basically they handle it.
If you'd like to install an arbitrary version, you need to manage the settings apk by yourself. Then, appium:skipDeviceInitialization allows you to skip the preparation steps by Appium.


fix Prevent data race when finishing s3 file writer's _put_object operation

Proposed changes
Issue Number: close #xxx
buf.set_val(_st); would decrement the condition variable waiting for the task to be finished, which might result the s3_file_writer to be destructed before the line LOG(WARNING) << _st; to take effect and turn out to be one heap-use-after-free for accessing the member variable _st.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Use yarn instead of npm in Dockerfile

There is yarn.lock in the repository, but there isn't package-lock.json in it.
Switch to use yarn in Dockerfile will fix #80

LGTM. THANK YOU


GH-79: Add AWS Integration tests module

Fixes GH-79 (https://github.com/spring-cloud/spring-cloud-stream-app-starters/issues/79)
The real testing against AWS is based on the Cloud Formation and Stack templating.

Add module spring-cloud-starter-stream-aws-integration-tests
Include maven-failsafe-plugin to run Integration Tests in safe mode and don't run them with the Surefire plugin.
Provide the ability to read AWS credentials from an external properties file. See README.adoc in the module.
Add Cloud Formation Stack template - AwsIntegrationTestTemplate.json
Add AwsIntegrationTestStackRule JUnit Rule to read AWS credentials, store them in the System.properties and create and remove the test Stack around target test class.
Add S3Source test: stores a content in the TestBucket and the S3Source application poll it with the mode=lines.
Fix AmazonS3SourceConfiguration to accept ResourceIdResolver to resolve the real S3 bucket against the AWS stack in which our application is ran.

Note: this new module is intended to utilize integration tests for any other AWS new SCSt applications.
More over their tests must be placed in the same SpringCloudStreamAwsApplicationsITCase to avoid extra time and, of course, payment for stack creation.
In other words: the stack must be created only once and only after that we should run all our integration tests.

Fixed, rebased, squeshed.
See the commit comments for more info,

rebased and merged.


tile-join to a directory

In the documentation for tile-join, there's mention of being able to export to a directory rather than mbtiles. I've tried running this with something like the following:
tile-join -e [merged-directory] [file-1].mbtiles [file-2].mbtiles
but it keeps popping up that -e is an illegal operation with the following help message:
Usage: tile-join [-f] [-i] [-pk] [-c joins.csv] [-x exclude ...] -o new.mbtiles source.mbtiles ...
Which seems to omit any mention of the directory operations. Was this capability removed?

Didn't realize this was a very new feature! Updated tippecanoe, seems to be working now. I need to buy a rubber duck.
Thanks for the great work!


customize theme via option or antd.customize.less not work

Hi, I need help.
Why customize theme not work as expected?
Am I missing some configuration?
I have configured craco.config.js as seen as below

and my package.json
{
  "name": "resist",
  "version": "0.1.0",
  "private": true,
  "dependencies": {
    "@craco/craco": "^3.2.2",
    "antd": "^3.11.2",
    "craco-antd": "^1.9.3",
    "craco-preact": "^1.1.0",
    "preact": "^8.3.1",
    "preact-compat": "^3.18.4",
    "react": "^16.7.0",
    "react-dom": "^16.7.0",
    "react-hot-loader": "^4.6.3",
    "react-scripts": "2.1.1",
    "unistore": "^3.1.0"
  },
  "scripts": {
    "start": "nodemon -w ./antd.customize.less --exec 'craco start'",
    "build": "craco build",
    "test": "craco test",
    "eject": "craco eject"
  },
  "eslintConfig": {
    "extends": "react-app"
  },
  "browserslist": [
    ">0.2%",
    "not dead",
    "not ie <= 11",
    "not op_mini all"
  ],
  "devDependencies": {
    "nodemon": "^1.18.9"
  }
}

Hi, your configuration looks fine to me. To see the customized theme, you will need to start using some of the Ant Design components. Right now you are just using the default React app component, and this is not affected by the Ant Design theme. (It's just a plain component generated by create-react-app.)
Hope that helps!

Oh sorry, I just saw that you are using an Ant Design button. The button is actually not affected by the @primary-color variable.
You need to set @btn-primary-color - https://github.com/ant-design/ant-design/blob/master/components/style/themes/default.less#L125-L159


get primary group?

Is there anyway to set a 'primary' group and just retrieve that?

And not completely related, but how do you retrieve the role name for the user? Right now I am doing
public function roles()
{
return $this->belongsToMany('Bican\Roles\Models\Role');
}
and something like $user->roles[0]->i want the rolename but i dont know to get it :)

I think you might need another field to mark a primary group. I think this package doesn't have this feature.

add another field call 'primary_role' in either your users database or roles database,

add function call setPrimaryRole($adminGroup) and getPrimaryRole() to mark and retrieve the primary group from the field primary_role.

$user1->attachRole ($adminGroup)
$user1->attachRole($managerGroup)
$user1->setPrimaryRole($managerGroup)
$user1->getPrimaryRole(); //return $managerGroup.


Only load concrete models on boot

Coverage remained the same at 100.0% when pulling 2ee051831cffcd2592c8eda00d81abfa9742095b on OnlyLoadModelsOnBoot into 59a31b7634669831f05398b362a3697e22dfe6d8 on master.

Coverage remained the same at 100.0% when pulling 2ee051831cffcd2592c8eda00d81abfa9742095b on OnlyLoadModelsOnBoot into 59a31b7634669831f05398b362a3697e22dfe6d8 on master.

Coverage remained the same at 100.0% when pulling 8f083c714647a58caaef348dff3986ef583da6e4 on OnlyLoadModelsOnBoot into 59a31b7634669831f05398b362a3697e22dfe6d8 on master.


Vertical tree with branch of nodes as vertical list

Question / Feature request
It is possible to make a vertical tree with branch of nodes as vertical list? I want to add this functionality to this library, but I can't find information about in d3 docs.
Expected view

+1

+1

Was anyone able to figure out how to implement this?


Seq to improve the mem footprint of Tree.clear?

https://github.com/mirage/irmin/pull/1171 recently expressed concern about the memory footprint of Tree.clear.
Currently, clear_info starts by calling StepMap.bindings on all 3 maps, would it be worthwhile to call StepMap.to_seq instead and concatenate these sequences using Seq.append?

That seems like a good idea to me :-) all the better if we can write a benchmark / collect a memtrace that indicates the improvement.

That seems like a good idea to me :-) all the better if we can write a benchmark / collect a memtrace that indicates the improvement.

Fixed by 4c90fa54f7a26f1a53cce55597fe044cd93be56a


3.16 and 3.17 replays don't work

I can't play replays from SC2 versions 3.16 and 3.17 with pysc2.bin.play. I am using python 3.6.1, pysc2 2.0.1 on Windows. I played all replays manually in game to download older client versions and they worked fine.
Relplays from various 4.x versions work fine when played by pysc2, but when I attempt to play one of  3.16 or 3.17 replays it displays replay's info normally, launches SC2 and closes it almost immediately with the following error:

E0619 16:22:04.923007  2652 _logging.py:54] close status: 16896
I0619 16:22:05.525105  2652 sc_process.py:197] Shutdown gracefully.
I0619 16:22:05.525105  2652 sc_process.py:182] Shutdown with return code: 0
Traceback (most recent call last):
File "C:\Users\User\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\Users\Michal\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 85, in _run_code
exec(code, run_globals)
File "C:\Users\Michal\AppData\Local\Programs\Python\Python36\lib\site-packages\pysc2\bin\play.py", line 212, in
app.run(main)
File "C:\Users\Michal\AppData\Local\Programs\Python\Python36\lib\site-packages\absl\app.py", line 274, in run
_run_main(main, args)
File "C:\Users\Michal\AppData\Local\Programs\Python\Python36\lib\site-packages\absl\app.py", line 238, in _run_main
sys.exit(main(argv))
File "C:\Users\Michal\AppData\Local\Programs\Python\Python36\lib\site-packages\pysc2\bin\play.py", line 157, in main
controller.start_replay(start_replay)
File "C:\Users\Michal\AppData\Local\Programs\Python\Python36\lib\site-packages\pysc2\lib\remote_controller.py", line 88, in _valid_status
return func(self, *args, **kwargs)
File "C:\Users\Michal\AppData\Local\Programs\Python\Python36\lib\site-packages\pysc2\lib\remote_controller.py", line 63, in _check_error
return check_error(func(*args, **kwargs), error_enum)
File "C:\Users\Michal\AppData\Local\Programs\Python\Python36\lib\site-packages\pysc2\lib\remote_controller.py", line 54, in check_error
raise RequestError("%s.%s: '%s'" % (enum_name, error_name, details), res)
pysc2.lib.remote_controller.RequestError: SC2APIProtocol.ResponseStartReplay.Error.MissingOptions: ''

Have the same problem on Ubuntu 16.04

That's not a very helpful error message... Unfortunately I'm not sure what's wrong either. I'd recommend getting newer replays using https://github.com/Blizzard/s2client-proto/tree/master/samples/replay-api and stop using 3.16/3.17.

Anyone find a fix to this? I am using the newest replays off the Blizzard github.


[$1000]  routing number paste to compose box instead of input field

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

delete all the workspace (if you have any)
click on FAB icon
select New workspace option
which will open RHP
go to settings and change the currency to non USD (if currency is USD)
click on back button to go previous page
click on bank account option which will open workspace currency popup
click on update to USD button
click on connect manually option
paste routing number in input field

Expected Result:
should paste routing number in input field
Actual Result:
routing number paste to compose box
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.42-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/f8a7996e-92dd-4545-873a-584a80ba0d70
https://github.com/Expensify/App/assets/43996225/dbf7fbcf-3f93-4bc6-85d3-f297f634b479
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689754408457479
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019ed0e16ea50369c3
Upwork Job ID:<PHONE_NUMBER>094263296
Last Price Increase: 2023-07-21

I will work on this issue

@abdulrahuman5196 bump for review of these proposals. Thanks

@abdulrahuman5196 bump for review pls :)

@abdulrahuman5196 bump for review ^^ thanks

Sorry for the delay from my end. Will close out the review today.

@laurenreidexpensify This seems to be a dupe happening at a different place of this issue - https://github.com/Expensify/App/issues/23184
Thanks @samh-nl for pointing it out.

Ah you're right. That one was opened first, so closing in favour of that


[Bug]: Nav bar logo does not change in dark theme.

Contact Details
No response
What happened?
When I change the theme color from light to dark in the properties, the logo changes to default logo?
const adminOptions = {         componentLoader,         rootPath: "/admin",         defaultTheme: dark.id,         availableThemes: [dark, light, noSidebar],         dashboard: {             component: Components.Dashboard,             handler: dashboardHandler,         },         branding: {             companyName: 'Admin Dashboard',             withMadeWithLove: false,             favicon: "http://localhost:4002/public/favicon.png",             logo: "http://localhost:4002/public/logo.png",         } }
Bug prevalence
Always
AdminJS dependencies version
^7.8.1
What browsers do you see the problem on?
No response
Relevant log output
No response
Relevant code that's giving you issues
No response

I'm Facing the same problem .. any solution yet


Fix/transaction functions

Fixes Transaction functions for node operations:

Fixes Bundlr Version
Updates Bundlr instance import in transaction function
Removal of File type as data input param from createTransaction function

The following utility has been tested:

createTransaction
signTransaction
postTransaction
createAndPostWBundlr

These work with both node18 and node16. For running the scripts in node16 you need to pass some experimental flags as follows which has been confirmed by the Arweave Team as well:
node --experimental-fetch --no-warnings file-name
Here is a repo for testing scripts: https://github.com/ropats16/arweavekit-examples-test
This must be linked with ArweaveKit via yarn link for testing.
However, the Bundlr fix work only in the node environment due the the way it is being imported. This import has been implemented after consultation with the Bundlr team. They are aware of the errors that persist and some incompatibility with arweave-js but there is no workaround at the moment. Hence, all our transaction functions work only in the node environment at the moment and as long as we continue to support Bundlr in the same file there is no way to test the other functions in web apps.
A potential solution is separating Bundlr functions and implementing them individually for node and web.

Latest update to the PR fixes the following:

All the transaction functions work in a node environment
Arweave based transactions work in web apps
Bundlr transaction functions trigger with a window.ethereum object but due to some underlying errors they always fallback to an arweave web transaction for now

This can all be tested with https://github.com/ropats16/arweavekit-examples-test via yarn link or packing the PR with yarn pack


Investigate spurious Bluetooth pairing requests for some Android-AppleDevices

Bluetooth pairing request on Android has been reported when an Android device (e.g. Samsung S10) is within range of some Apple devices (e.g. Apple TV and Mac computers). It is unclear if this is triggered by HERALD, because this has not been observed in the dev and test environments involving many Android and Apple devices running many different OS versions. However, it is worth investigating even if it is not caused by HERALD because the end user may assume HERALD is the cause and uninstall the app, eroding public trust.

Need to confirm if this issue occurs for :

A specific pair of Android-Apple device (e.g. S10-AppleTV)
All Android devices with a specific Apple device (e.g. AnyAndroid-AppleTV)
A specific Android device with any Apple device (e.g. S10-AnyAppleDevice)

Need to confirm if this is caused by other apps on the Android or Apple device :

Factory reset both Android and Apple device (e.g. S10, AppleTV) and check if issue occurs without HERALD
Install HERALD and test if this issue only occurs when HERALD is running on the Android device
Keep HERALD running in the background and test if Bluetooth pairing request only occurs once, or is recurring

If this is caused by HERALD, there are some theories and potential mitigations :

Specific Android or Apple device running specific OS version initiates Bluetooth pairing on (i) device discovery, (ii) connection, (iii) service and characteristic discovery. If (i), then the issue will arise for any app performing a BLE scan, thus there is no solution. If (ii) or (iii), then HERALD can avoid connecting to Apple device unless it is absolutely certain that it is an phone offering HERALD services. This will be non-trivial as the Apple background advert offers little clue in the advert (and subject to change) about the device, beyond the manufacturer ID. This is being actively investigated for connectionless Apple device filtering.
If it is triggered repeatedly by HERALD on connection, the current ignore device logic can be improved to mark devices to be ignored indefinitely (e.g. good connection confirms device does not offer service), rather than the current logic that retries at regular intervals (increases in duration) to ensure transient BLE faults does not prevent connection to a genuine device.

Accepted. May be caused by an adopter app. Still investigating.

Cannot replicate issue with latest Herald code on develop branch and reported devices. Also discovered a way to positively identify some device models over Bluetooth.


Is it possible to add flags for subcommand?

I have been struggling to find an example of how to use flags with a given subcommand.
Would love to know if it is possible to do something like the following.
mycli bump --major or mycli bump --minor or mycli bump patch

Yes, just define the options on the subcommand struct. For example: https://github.com/jessevdk/go-flags/blob/master/examples/add.go


Used glide to vendor dependencies

In order to introduce glog (and possibly other dependencies) we need
a flat vendor directory (glog panics if two versions of it are
imported). We decided to use glide. It seems to be developer-friendly
and also offers mechanism for consistend deploys
Some questions you may have:
Q: What with the size of repo
A: We don't store any additional code in repo. Only the hashes in
glide.lock
Q: How can I use my own k8s version.
A: Edit the glide.lock and put your commit hash there
Q: The difference between update and install
A: glide update will modify glide.lock. glide install only installs
stuff

This change is

Reviewed 6 of 6 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

Dockerfile, line 17 at r1 (raw file):

    mv /go/src/github.com/Mirantis/k8s-AppController/manifests /opt/kubeac/manifests &&\
    rm -fr /go

Unnecessary empty line.

README.md, line 120 at r1 (raw file):

You can also override this behaviour by using `-l` flag for `kubeac deploy` command available on AppController pod, but this should be only done for testing purposes and is not encouraged in production.

Unnecessary empty lines

Comments from Reviewable

Looks like travis seeing Makefile uses make to build project, before running tests. Maybe you should add script entry there as described in https://docs.travis-ci.com/user/languages/go/ ?

Hmmm... What is the reason behind using glide-vc if we are not vendoring anything in project?
IMO using it makes sense only when vendor/ is included in project, to minimize it's size to only necessary part of dependent files.

@jellonek smaller binary size, smaller file with deps. Isn't it?

Didn't noticed binary size change (have to check - possibly by eliminating files not used by our source but having init func).
Smaller file with deps? Which file? repo tgz?

I'd second jellonek as for getting rid of glide-vc

If it changes nothing then let's get rid of it. :)

Only change it now provides is increasing time of build :P

Testing travis manifest

Review status: 1 of 7 files reviewed at latest revision, 5 unresolved discussions.

glide.lock, line 317 at r5 (raw file):
Previously, zefciu (Szymon Pyżalski) wrote…
> It seems that glide.lock must contain all the flattened dependencies

Done.

Makefile, line 12 at r4 (raw file):
Previously, jellonek (Piotr Skamruk) wrote…
> Misspointed question. It's ok.

Done.

Makefile, line 11 at r5 (raw file):
Previously, zefciu (Szymon Pyżalski) wrote…
> Why? You can't run tests if the 'vendor' directory is not created.

Done.

Comments from Reviewable

The image seems to be really big. 700MB is much more than was before: https://hub.docker.com/r/zefciu/k8s-appcontroller/builds/bnvvkgpchasbhpmrfba2hz3/

Image size problem seems to be resolved. 👍

Resolved the problem.

Review status: 1 of 7 files reviewed at latest revision, 10 unresolved discussions.

Dockerfile, line 12 at r7 (raw file):
Previously, gluke77 (Illia Khudoshyn) wrote…
> Please do not use aliases for glide command

Done.

Dockerfile, line 17 at r7 (raw file):
Previously, gluke77 (Illia Khudoshyn) wrote…
> Ditto

Done.

Makefile, line 5 at r6 (raw file):
Previously, nebril (Maciej Kwiek) wrote…
> should glide.lock be here? it seems like it's already handled in dockerfile

Done.

Makefile, line 9 at r7 (raw file):
Previously, gluke77 (Illia Khudoshyn) wrote…
> So we run 'glide install'  twice while building a docker img: here and in the dockerfile. THat does not look right. We probably should do something with it.

Done.

Makefile, line 12 at r7 (raw file):
Previously, gluke77 (Illia Khudoshyn) wrote…
> Please use 'go test -run ...' to filter tests

Can I do it without negative lookaheads?

Comments from Reviewable

@gluke77 @nebril ptal

Reviewed 3 of 3 files at r5, 2 of 3 files at r6, 2 of 3 files at r7, 1 of 1 files at r8.
Review status: all files reviewed at latest revision, 7 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 8 unresolved discussions.

Comments from Reviewable


QRDAIII Validation - QRDAIII validation getting failed because of code system exist in the file

Hi Team
When we are validating qrdaiii with cypress, its throwing error as attached, if we remove that code system section, then cypress passing the qrdaiii files but as per QRDA-III-Eligible-Clinicians document, code system section coming under shall(screenshot attached). So it seems like this is the bug in the cypress. So please take a look and suggest us

Sorry for the delayed comment. The content of the languageCode element is actually dictated by the CDA XML schema that underlies the QRDA Implementation Guide.  The languageCode element in CDA prohibits the use of the CodeSystem attributes.  I've attached 2 screenshots from the CDA schema.


Opener can sometimes get stuck opening duplicate tabs.

Under exceptional circumstances, the opener may start opening more than one tab for the same url. Once this happens, it doesn't go back to normal, and the bug can often compound. A that bug occurs to create two tabs with the same url instead of one, can be triggered again to produce four tabs with the same url instead of one, etc.
Exceptional circumstances I think have caused this at one point or another:

Dragging previously opened/opening tabs from one window to another.
In Chrome DevTools, resuming script execution from a breakpoint pause.

I don't currently have a recording of this occurring. But I'll try and add one for completionism, even though that doesn't help solve this much.

I'm worried I've fundamentally misunderstood how some key part of my endless listener chain with thisListener works.
However, it's also possible that my solution to tab dragging errors is ill-conceived or incomplete. Maybe it's not impossible that multiple "Error: Tabs cannot be edited right now (user may be dragging a tab)." errors can occur fast enough that multiple such errors are on some stack. Then, instead of the last such error triggering createTab() without error and therefore continuing tab opening normally, the second to last error does, and so does the last one, meaning that there's two valid createTab() loops occuring.
https://github.com/Lemonsavery/openUrlsWhenLoaded/blob/79e9ca92c918a55eb895ec86b2488c497698f11d/page.js#L488-L491

I'm not going to spend more time trying to fix this right now. I'm just hoping that features I add don't make this bug far more common.

I am not sure maybe add in extension somthing to prevert open duplicate tabs like Duplicate Tab Closer do?


Updated react-add-implicitflow sample to SPFx GA

Q
A

Bug fix?
no

New feature?
no

New sample?
no

Related issues?
n/a

What's in this Pull Request?
Updated react-add-implicitflow sample to SPFx GA

thx Waldek, appreciated.


Fix example with-custom-babel-config: update to Babel 7

The option "decoratorsLegacy": true seems mandatory, fails without it:

The new decorators proposal is not supported yet. You must pass the "decoratorsLegacy": true option to @babel/preset-stage-0

Thanks!


Add link to Page Discussing Datafactory Linked Service Setup with MSI

In the Databricks Properties section of this page. I feel that a link should be added to this page here for the MSI row that discusses the Databricks Link Service Setup with MSI in more detail. Specifically it mentions that the ADF System MSI need to have Contributor permissions on the Databricks Resource RBAC in order for this setup to be successful which is not something that is currently mentioned on this page.

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

ID: 794a72a8-f145-dad9-1f80-16d8ce0b587a
Version Independent ID: be745cc3-bda5-ff52-e390-324c2a429155
Content: Compute environments - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/compute-linked-services.md
Service: data-factory
Sub-service: concepts
GitHub Login: @nabhishek
Microsoft Alias: abnarain

@pnarsi Thank you for feedback. We are actively validating it. We will get back to you with updates.

@pnarsi - I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hello @pnarsi
Thanks for reporting this item! I have updated the doc and it will be published soon. We appreciate the feedback!


ARCHICAD Converter: No BuildingMaterial for Windows and not all Components

Prerequisites

[x] I read the contribution guidelines
[x] I checked the documentation and found no answer.
[x] I checked existing issues and found no similar issue.
[x] I checked the community forum for related discussions and found no answer.
[x] I'm reporting the issue to the correct repository (see also speckle-server, speckle-sharp, specklepy, speckle-docs, and others)

What package are you referring to?
ARCHICAD Converter
Describe the bug
When sending a stream from ARCHICAD, the Components and material information for Windows are not correct. In the example, there is only one component with "0" as BuildingMaterial.
Expected behavior
We would expect at least two components (glass and frame) with a BuildingMaterial other than "0".

While the information is missing at the Component object, it is available in the display Values (see image to the right).

@ks-cph i was able to reproduce this. I will check in with @jozseflkiss and get back to you.

Hi @bimgeek,
sounds good. thanks for the update.

Hi @ks-cph,
Thanks for the report!
In Archicad Component Schedules are created based on materials used in construction elements. For example, a composite wall will have components based on the skins/layers of the Composite attribute, or a profiled beam will have components based on the Building Materials attached to the polygon parts of the Profile.
This functionality is also available for GDL-based elements (Object, Door, Windows, etc.): you need to define the Building Material in the GDL script and all subsequent geometries will belong to that component. E.g. you can model the window frame and glass in two separate components.
However, for most of the library elements in the Archicad Library shipped with Archicad this is not developed. They do not define building materials at all, so all part of the library element will belong to one component. This is why we export only one component to Speckle. (I realized that in the Archicad Component Schedule you cannot list the one and only component for windows, possibly because there is no real material defined. So it might be a problem in the Speckle Connector, we should just skip the component in these cases to be similar with Archicad).
Regarding the displayValue, you are right you can see some material information, but this is based on the Surface attribute not on Building Materials.
If you have any questions, please let us know!
Regards,
József


Process Semantics (2.1.1) could benefit from more illustrative examples of the concepts it introduces

I feel that 2.1.1 can benefit from having examples (or references to the implementation of the concept in question) in relation to the concept it introduces. Handling process state, for example, references only 2.6, which outlines the vfs structure, not how really process state is handled. The overall chapter can benefit from more examples on almost all of the things introduced.

Thanks for the comment. I agree we can improve this a lot. Will be working on it.


update to mermaid v10

fix #23153

ERROR in License<EMAIL_ADDRESS>has disallowed license EPL-2.0

Hmm…

If the lincense is okay, add it to the array in the webpack config.
Changelog here. The module has switched from CJS to ESM, it may be necessary to change how we import it, the default export is a crutch for CJS that is probably no longer necessary.

The problem is:
If I've seen it correctly, this license isn't compatible.
It would require every user to use the EPL2 or a related license as well.
MIT doesn't have this restriction, so they should be incompatible.

Actually, this should be a problem on Mermaid's side, since they are violating their own license with that package.
Or is my superficial license knowledge wrong about that?

I think generally most open source software does not take license violations seriously, so I think we may have to follow suit and just relax the (currently strict) validation.

Most open source software does not seem to take license violations seriously, so I think we may have to follow suit and just relax the (currently strict) validation, or add an override for this module.

But that may break those company users who care about this. Before this change, they could choose not to open source their special code.

Actually, this should be a problem on Mermaid's side, since they are violating their own license with that package

Do you know which part of that license is violated?

If I read Wikipedia correctly, it's not actually viral of you just depend on it:

Linking to code (for example to a library) licensed under EPL automatically does not mean that your program is a derivative work. Eclipse Foundation interprets the term "derivative work" in a way that is consistent with the definition in the U.S. Copyright Act, as applicable to computer software.[12]

So I think we can just allow it in the SPDX expression in webpack.

Yep, right.
If we modified the code, we would need to be EPL as well.
Simply using it should not be derivative work then.

Actually, this should be a problem on Mermaid's side, since they are violating their own license with that package

Do you know which part of that license is violated?
I think this license qualifies as a viral one, similar to GPL, is this the issue?

3.1 If a Contributor Distributes the Program in any form, then:
    a) the Program must also be made available as Source Code

If EPL is included, does it mean that all downstream Gitea users have to make their own-built Gitea open-source?

I don't think "running" a program qualifies as "distributing".


Operation fails with v2 emulation when API v3 authentication is enabled

We are trying to use the v2 emulation to use the etcd plugin of CoreDNS with v3 backend.
We found that any operation to the v2 emulation fails when v3 authentication enabled. We are testing with etcd v3.3.3. Is this the intended behavior?
# Check etcdctl version
$ ETCDCTL_API=3 etcdctl version
etcdctl version: 3.3.3
API version: 3.3

# Start etcd with v2 emulation
$ docker run -it --rm -p 2379:2379 gcr.io/etcd-development/etcd:v3.3.3 \
    etcd --advertise-client-urls="http://<IP_ADDRESS>:2379" --listen-client-urls="http://<IP_ADDRESS>:2379" --experimental-enable-v2v3="/v2"

# Enable authentication
$ ETCDCTL_API=3 etcdctl user add root
Password of root:
Type password of root again for confirmation:
User root created
$ ETCDCTL_API=3 etcdctl auth enable
Authentication Enabled

# We can write a key/value as root via v3 API
$ ETCDCTL_API=3 etcdctl --user root:test put /etcd "v3"
OK
$ ETCDCTL_API=3 etcdctl --user root:test get /etcd
/etcd
v3

# We can't write a key/value as root via v2 emulation
$ etcdctl -u root:test set /etcd "v2"
Error:  client: etcd cluster is unavailable or misconfigured; error #0: dial tcp <IP_ADDRESS>:4001: connect: connection refused
; error #1: client: etcd member http://<IP_ADDRESS>:2379 has no leader

error #0: dial tcp <IP_ADDRESS>:4001: connect: connection refused
error #1: client: etcd member http://<IP_ADDRESS>:2379 has no leader

# We can write a key/value via v2 emulation when authentication disabled
$ ETCDCTL_API=3 etcdctl --user root:test auth disable
Authentication Disabled
$ etcdctl set /etcd "v2"
v2

@summerwind thanks for your report. Currently the auth mechanism of v3 cannot care about the v2 emulation feature. Also it is hard to provide a unified auth mechanism which can cover both v2 and v3 keys because of their completely different data models.
As an adhoc fix, giving privilege of accessing to v2 key space only to root user would be possible (although it isn't easy because of the complex nature of the emulation layer). How do you think? Is it suitable for your use case?

@mitake Thank you for the suggestion!
In our use case, it is only the specific system (CoreDNS) that uses v2 emulation, so applying the privilege of v2 emulation to a specific user will fit.
However, if the user is shared with v2 emulation and v3, it will be difficult to separate privileges by applying it to root, so it is better to apply it to user dedicated to v2 emulation like emulation user.

@heyitsanthony v2 auth is built on basic auth of http. So it doesn't have credentials.
@summerwind I see. I'll consider the design of unified auth (errors in the design would introduce security holes so I need to be careful). Of course your PRs are welcomed :)


Can't !import_model: [WinError 183]

I'm trying to import the two new Runway SD 1.5 ckpts but unsuccessfully:
FileExistsError: [WinError 183] Impossibile creare un file, se il file esiste già: './configs\\new_config.tmp' ->
'./configs/models.yaml'

Translating: Impossible to create a file, if the file already exist.

@theRolento Sorry for the late response here. I believe our updated model loading handles this sitaution.
@lstein , Do we correctly handle this situation now?

@theRolento Sorry for the late response here. I believe our updated model loading handles this sitaution.
@lstein , Do we correctly handle this situation now?

Don't worry, yeah as of today everything works correctly

Good to hear, thanks for letting me know.


[[FIX]] Correct parsing of RegExp character sets

Do not interpret RegExp syntax characters using their specialized
semantics when they appear within a character set.
This is intended to resolve gh-3356

Coverage increased (+0.001%) to 98.472% when pulling 76db9f8db902dbd74496e71eb6856370ffa910a5 on jugglinmike:regexp-char-class into 0e85de5f33f3a4bb18833bbb87b4a08294f16899 on jshint:master.


feat(Enterprise Billing Units): add service to project

PR summary
Adds the Enterprise Billing Units service to the project.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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


added configurable alloy base directory

This is needed if your alloy project is not working out of the app directory.
For example, JAST.

Thanks!


Revert banned fields in GBIF Taxa sheet

The use of extra fields in the GBIF taxa sheet was permitted in the past but is now prevented. This was actually used a few times and is supportable, so we should re-enable it. This is at least partly to allow re-validation of legacy datasets

If this isn't already fixed, I think this should be fixed for 3.0.0


Multithreading issue on GzipChannel write header

When having multiple instances of WarcWriter the operations on
private static final ByteBuffer GZIP_HEADER = ByteBuffer.wrap(GZIP_HEADER_); are causing issues.
Some threads are writing the gzip header, some might not.
I think the issue could be fixed by removing the static part for the GZIP_HEADER.

Yikes. Good catch.
Suggested fix applied in 68575d42cf0c589b78923055864ab73640b5e78e and released as 0.28.3.


Live train running status

Aim

By entering the train number and date the script will show the live running status os train.

Details

The live running status will be fetched via API

@kaustubhgupta please assign it to me.

@Amit366 I don't think there is an official API for this. You can try web scrapping IRCTC for this issue.


Update copyright year to 2017

If we do a v1.0.5 release, we should probably update the copyright year before 2017 is over.

Looks good.


Unable to integrate Facebook Signup/Signin on Azure AD B2C

We were following the instructions since we wanted to integrate Facebook login with our applications (Web+ Mobile). But we got stuck at point #10 since we have not been able to find "from the menu, select the plus sign or Add Product link next to PRODUCTS. Under the Add Products to Your App, select Set up under Facebook Login". The URL to the document is: _https://learn.microsoft.com/en-us/azure/active-directory-b2c/identity-provider-facebook?pivots=b2c-user-flow_
Please share the correct documentation as soon as possible.

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

ID: aeb73901-3320-4d53-767c-bfd03b3b0354
Version Independent ID: 404e18f1-d6da-5654-3e68-8e888e11433e
Content: Set up sign-up and sign-in with a Facebook account - Azure AD B2C
Content Source: articles/active-directory-b2c/identity-provider-facebook.md
Service: active-directory
Sub-service: b2c
GitHub Login: @garrodonnell
Microsoft Alias: godonnell

Following is the screenshot for your reference.

@Prathavi1312
Thanks for your feedback! We will investigate and update as appropriate.

Hello @Prathavi1312
I have reproduce the issue, I got the step 10 in created app, Please refer below screenshot.

@Prathavi1312
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close


Fatal error with {} and {/}

Describe the bug
Hi, I have a problem when using the {} tag with the {/} ending. When I use it, the plugin throws a fatal error. I leave the log in the link below.
Environment (please complete the following information):

PhpStorm version 2021.3
Plugin version 1.1.4.

Code
error log
Additional context
Example:
{block title}     {_}hydra_title_admin_tools_accounts{/_} {/block}

Fixed in v1.1.5


Error in Lifecycle page loading for a created API Product

Description:
After a API Product is created, and deployed , an error occurred when trying to view the life cycle page. The below error occured.

This is fixed in APIM RC2 pack. Hence closing.


Add ability to dump the state of any volumes mounted on the pod

Is this a BUG REPORT or FEATURE REQUEST?: FEATURE
Requirements

Optionally, any volumes mounted on any of the containers in the pod should be dumped to the S3 as well.

Wondering if leveraging Velero - or extending Velero might be better for this use case - ability to include all related Objects (configMaps, Secrets, ingress, etc) in addtion to container image and volumes.


bump docker image to 0.25.0

Hey,
great tool! so there is a 0.25.0 release, but the docker image is still 0.24.4. Will there be an upgrade of the docker image at some time?
Thanks!

0.26.1 pushed


Add conversation settings

Add conversation settings, which is a list of SettingKV pairs for each conversation. They live on ConversationMetadata and ConversationInfo sibling to Status.

old


Connection to remote SSH not working, python extension not loading

What: Connection to remote SSH vscode server not working, python extension also not loading

version: 1.43.1
Error message in console:
(node:7728) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
I see similar issues that have been reported. But i am not able to see the fix.
Please let me know if you additional information.

I had the same problem this morning.
The password prompt started to loop, asking for the password over and over.
Out of curiosity, the same problem appeared both in vscode and vscode-insiders.
One of the errors I was getting looked like this:
[10:21:56.330] Showing password prompt
[10:25:19.361] Got password response
[10:25:19.362] "install" wrote data to terminal: "**********"
[10:25:19.372] >
>
[10:25:19.382] > Connection closed by <IP_ADDRESS> port 22
> The process tried to write to a nonexistent pipe.
>
[10:25:19.653] "install" terminal command done
[10:25:19.653] Install terminal quit with output: The process tried to write to a nonexistent pipe.
[10:25:19.653] Received install output: The process tried to write to a nonexistent pipe.
[10:25:19.653] Resolver error: The process tried to write to a nonexistent pipe

In my case I solved removing completely the folder .vscode-server on the server (Ubuntu server 18.04).
For vscode:
sudo rm -R .vscode-server
For vscode-insiders:
sudo rm -R .vscode-server-insiders

David-it - Thanks for sharing your comment. For me problem started with the password prompt starting to loop. I did not have to remove the .vscode-server. Problem resolved itself. However, I lost half day.
Were you able to find the root cause?

David-it - Thanks for sharing your comment. For me problem started with the password prompt starting to loop. I did not have to remove the .vscode-server. Problem resolved itself. However, I lost half day.
Were you able to find the root cause?

No, I have no idea about the cause. I just noticed that the problem showed up after the updates of yesterday morning of both vscode and vscode-insider
vscode
Version: 1.43.2 (user setup)
Commit: 0ba0ca52957102ca3527cf479571617f0de6ed50
Date: 2020-03-24T07:38:38.248Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
vscode-insiders
Version: 1.44.0-insider (user setup)
Commit: 073a24de05773f2261f89172987002dc0ae2f1cd
Date: 2020-03-24T05:37:12.592Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363


Vulnerabilities display issue when scanned multiple images

I invoke Trivy task twice in the pipeline and while the report contains tabs of both scans, when switching between the tabs the list of vulnerabilities in not refreshed.
On the screenshot below, there are two scan results, one with 12 vulnerabilities and the other one with 0 vulnerabilities. When I switch to the second tab, it still shows vulnerabilities list from the first tab.

Here are the tasks:
- task: Docker@2
  displayName: Login to Container Registry
  inputs:
    containerRegistry: "testmanagement"
    command: "login"
- task: trivy@1
  displayName: Scanning XXXXXX-client:$(DOCKER_TAG)
  inputs:
    image: XXXXXX/XXXXXX-client:$(DOCKER_TAG)
    docker: false
    exitCode: 0
- task: trivy@1
  displayName: Scanning XXXXXX-flows:$(DOCKER_TAG)
  inputs:
    image: XXXXXX/XXXXXX-flows:$(DOCKER_TAG)
    docker: false
    exitCode: 0

I can confirm this problem. A workaround is switching the sub-tab (Vulnerabilities -> Misconfigurations) and back to cause a refresh of the table. It then shows the correct vulnerabilities of the selected scan.
Therefore, the cause might be some missing event handling in the UI.

Thanks for sharing the workaround, as I just find out the same problem.
Hopefully, this would be resolved in the next version.

I can confirm this problem. A workaround is switching the sub-tab (Vulnerabilities -> Misconfigurations) and back to cause a refresh of the table. It then shows the correct vulnerabilities of the selected scan.
Therefore, the cause might be some missing event handling in the UI.


syntactic revision of the documentation

Brief revision of the documentation to provide a README.md with a syntax
easier to maintain, than the previous version.  Edits account for suggestions
by markdownlint / ruby-mdl (version 0.12.0).[1]
[1] https://github.com/markdownlint/markdownlint/
[2] https://tracker.debian.org/pkg/ruby-mdl

For me, 80 characters per line (maximum) is helpful if there is not much space available, the restraint may become a constraint with multiple panes/buffers simultaneously open (e.g.,kdiff3).  To some extent and time during the day, it then turns into a question of screen size/resolution and font parameters (same pixel height not necessarily equates same width, even among the non-proportional fonts).  Hence, fold -s my_file.txt > folded.txt with its (adjustable) threshold on 80 characters became something I like (though fmt is an interesting alternative because this can widen a column of text too narrow, too).  Either the road taken, this wasn't complete until a subsequent sed -i "s/ \+$//" input.txt to remove trailing spaces.
Obviously, the line including the grep command displays the limits of such an approach unaware of the context of a line.  Though the computer is fast to digest the file, the output still requires a check by a critical eye.
Thanks,
Norwid


JBPM-5082: Added users management configurations for the jbpm installer.

Hi @krisv , @mswiderski , @dgutierr ,
Can you please check if you agree on this change? If so, will create the PR for master as well.
Currently passwords on the jbpm-installer are on clear text, but both add-user script and our users management stuff do encrypt the passwords. So adding users from the UI adds the hashed password on the properties file, but the default passwords are on clear text.
I see two options:
1- Adding a configuration flag on the users management stuff to indicate wether passwords should be encrypted and use this configuration to set no encryption on the jbpm-installer.
2.- Moving the jbpm-installer to the use of encrypted passwords, as both add-user script and users management stuff expects.
This PR is the fix for option 2 -> Added users management configurations for the jbpm installer and disabled use of UserRoles login module for the other domain security (use of encrypted password using same RealmDirect module).
Note that the UserRoles login module used on the security domain is replaced by the RealmDirect one, which by default expects the encrypted passwords. This way:

The passwords are encrypted ( added comments in the password file with the clear text )
The configuration is centralized on just the realm conftiguration ( no need to specify the file paths on the security domain section as well )

I have checked the workbench and the kieserver, both connectivities works fine and now the users created from the UI can log in fine. Not sure if this change could have more implications...
PS: https://issues.jboss.org/browse/JBPM-5082
Thanks!!

+1

PS: If this gets metged, will update docs as it's necessary!

jenkins retest this


fix docs about observability exporter values; fix other minor doc bugs

Proposed Changes

Add notes about OBSERVABILITY_EXPORTER environment variable
Remove notes about "tracing services" (I didn't see any settings for distributed tracing specifically, but I maybe wrong)
Other minor doc fixes

Release Note

NONE

@googlebot I signed it!