RFC 8738 support?

Does/will dehydrated support RFC 8738 to issue certificates to IP addresses over ACME? ZeroSSL currently supports this over their REST API, but I am unable to test it with ACME because I am unable to find a client that supports both RFC 8738 and the EAB now requried by ZeroSSL. Looks dehydrated might be a possible choice, is this feature planned? Thanks!

This will be integrated for the upcoming release :+1:
But I'm not sure if ZeroSSL supports it on their ACME API.

Thanks! I have confirmed ZeroSSL only supports it over their REST API but not ACME yet. Either way, it's great to see more client support for this feature :+1:

I have an experimental implementation locally but since I want to release the new version today and didn't yet have an opportunity to test the code I'll postpone it for the next release (which will follow much quicker than the current one).

So that I understand, that would mean this would work?
# echo 'CA="https://acme.zerossl.com/v2/DV90"' >> /etc/dehydrated/config
# dehydrated --register --accept-terms
# INFO: Using main config file /etc/dehydrated/config
# INFO: Using additional config file /etc/dehydrated/conf.d/hooks.sh
# INFO: Using additional config file /etc/dehydrated/conf.d/neuropoly.sh
+ Generating account key...
+ Registering account key with ACME server...
  + ERROR: An error occurred while sending post-request to https://acme.zerossl.com/v2/DV90/newAccount (Status 400)

Details:
HTTP/2 400
server: nginx
date: Tue, 20 Apr 2021 07:53:13 GMT
content-type: application/problem+json
content-length: 159
replay-nonce: zbqMLe3MhuyqQXP2ZUZ2MEZ5XZJAVw_EZOJxUr6cO_g
cache-control: max-age=0, no-cache, no-store
access-control-allow-origin: *
link: <https://acme.zerossl.com/v2/DV90>;rel="index"
cache-control: max-age=-1

{"type":"urn:ietf:params:acme:error:externalAccountRequired","status":400,"detail":"The request must include a value for the \"externalAccountBinding\" field"}

So that I understand, that would mean this would work?
# echo 'CA="https://acme.zerossl.com/v2/DV90"' >> /etc/dehydrated/config
# dehydrated --register --accept-terms
# INFO: Using main config file /etc/dehydrated/config
# INFO: Using additional config file /etc/dehydrated/conf.d/hooks.sh
# INFO: Using additional config file /etc/dehydrated/conf.d/neuropoly.sh
+ Generating account key...
+ Registering account key with ACME server...
  + ERROR: An error occurred while sending post-request to https://acme.zerossl.com/v2/DV90/newAccount (Status 400)

Details:
HTTP/2 400
server: nginx
date: Tue, 20 Apr 2021 07:53:13 GMT
content-type: application/problem+json
content-length: 159
replay-nonce: zbqMLe3MhuyqQXP2ZUZ2MEZ5XZJAVw_EZOJxUr6cO_g
cache-control: max-age=0, no-cache, no-store
access-control-allow-origin: *
link: <https://acme.zerossl.com/v2/DV90>;rel="index"
cache-control: max-age=-1

{"type":"urn:ietf:params:acme:error:externalAccountRequired","status":400,"detail":"The request must include a value for the \"externalAccountBinding\" field"}

Wrong thread. RFC 8738 is about getting certificates signed directly for IP addresses instead of domains. If you're having issues with zerossl please make sure to have the newest dehydrated version and open a new issue if the problem persists.

Thank you @lukas2511, installing 0.7.0 made ZeroSSL work. Debian stable has 0.6.5 packaged; testing has 0.7.0 so in like a year when the next Debian stable comes around this will work more easily.
For reference for anyone who stumbles across this thread, my install script is now:
ssh root@server apt-get install dehydrated
scp ./usr/bin/dehydrated root@server:/usr/bin/ # where ./usr/bin/dehydrated is from https://github.com/dehydrated-io/dehydrated/releases/tag/v0.7.0
echo 'CA="https://acme.zerossl.com/v2/DV90"' >> /etc/dehydrated/conf.d/site.sh

And this seems to work fine for getting ZeroSSL working on Debian stable.

RFC 8738 support is now implemented on the rfc8738 branch. There are still a few issues, e.g. comparison on already signed certificates is missing normalization for now so certificates especially for IPv6 addresses might be refreshed on every run. Also signing of external CSRs hasn't been tested yet.
I don't think any public CA supports this feature yet, but e.g. step-ca can be used for testing or whatever internal use-cases one might have (e.g. etcd or ssh).

RFC 8738 support is now implemented on the rfc8738 branch. There are still a few issues, e.g. comparison on already signed certificates is missing normalization for now so certificates especially for IPv6 addresses might be refreshed on every run. Also signing of external CSRs hasn't been tested yet.
I don't think any public CA supports this feature yet, but e.g. step-ca can be used for testing or whatever internal use-cases one might have (e.g. etcd or ssh).

zerossl allows to have ip based certificates, can you please share a command line example? Thanks in advance

RFC 8738 support is now implemented on the rfc8738 branch. There are still a few issues, e.g. comparison on already signed certificates is missing normalization for now so certificates especially for IPv6 addresses might be refreshed on every run. Also signing of external CSRs hasn't been tested yet.
I don't think any public CA supports this feature yet, but e.g. step-ca can be used for testing or whatever internal use-cases one might have (e.g. etcd or ssh).

zerossl allows to have ip based certificates, but probably only in rest api... anyway, can you please share a command line example? Thanks in advance

Unfortunately they are not supporting ip certs via acme.
Example usage (for now) would be dehydrated -c -d IP:<IP_ADDRESS> --ca zerossl (but it will fail). The IP: prefix may be dropped, not entirely sure about that yet.

@lukas2511 Is this done or its still in implementation stage? I have a similar requirement where both ip and fqdn are being added as SANS in a certificate.

@lukas2511 Is this done or its still in implementation stage? I have a similar requirement where both ip and fqdn are being added as SANS in a certificate.

It's implemented, but basically untested as afaik there are no public CAs supporting this (yet). It should probably work fine with private CAs like smallstep. Also the normalization issue with IPv6 addresses still exists but that can be worked around.

Is this code available in v0.7.1 release?

@lukas2511  I think there is a typo here in the code:
https://github.com/dehydrated-io/dehydrated/blob/master/dehydrated#L1812
Should probably be ( Address)*. This showed up in shellcheck, which you really should be running.


Shendy/develop/migration

@leogdion I believe I have done SwiftDoc for all, and cleared most of swiftlint issues .. just few swiftlint issues ones that need your help.


Do not show weight or cash on delivery if us zero.

Also add currency to cash on delivery.

And also add pieces.


Template action

Fixed template reducer to stop getting called on every redux state update.

@mattben @chaosphere2112 closing until I've got the kinks worked out.


THRIFT-5074: Crosstest client/server command line [skip ci]

This matches the documentation to the actual interface.
The parameters have form --protocol=binary not --protocol binary

[ ] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[x] If your change does not involve any code, add  [skip ci] at the end of your pull request to free up build resources.

Committed.


Scope error

A user raised this issue over in the Johnny-Five repo.
The problem appears to be caused by this block in interchange.js:
try {
    const manifest = JSON.parse(fs.readFileSync(manifest_file));
} catch (e) {
    console.error('Manifest file incorrect');
    throw e;
}

let manifest_objects = (options.useFirmata ? manifest.firmata : manifest.backpack);

const is scope limited in try blocks so an error is thrown when trying to read manifest.firmata or manifest.backpack.

This is being resolved in the new test build out branch.

This has been fixed in the new 2.0.0 build. Will close and if there's any follow up then please reopen.


Changed mouse action to 'released'

I think this is a better way to go about this, because it generates two redraws at the moment!

Great change!

Thanks!


Errors on installation on php8

Hi.
Local web serwer (OpenServer_5_3_8) on Win 8.1, PHP8, all extensions intalled (Ctype, iconv, JSON, PCRE, Session, SimpleXML,Tokenizer, sqlite)
An attempt to run the command "symfony new . --demo" fails:

And other errors from web path:

Why?

IIRC, symfony new --demo uses the released version of the demo. Proper support for PHP 8 was added after the last release and is only in the dev version for now.

rosier, do I understand correctly that I cannot install a demo on PHP8 until this pull request is approved?
well, what doing this commit? - [Allow PHP 8 to run the project] (https://github.com/symfony/demo/commit/782b08d3ee6360a75422a87a7f80560843757699)

do I understand correctly that I cannot install a demo on PHP8 until this pull request is approved?
well, what doing this commit? - [Allow PHP 8 to run the project] (https://github.com/symfony/demo/commit/782b08d3ee6360a75422a87a7f80560843757699)

At moment symfony new --demo and composer create-project symfony/symfony-demo pick an older version that doesn't work on PHP 8. The PR #1194 plus a new release should fix that.
For now you can try the following to install the demo
git clone https://github.com/symfony/demo.git demo
cd demo
composer install

and to start the demo
symfony serve -d
symfony open:local

Hi.
Local web serwer (OpenServer_5_3_8) on Win 8.1, PHP8, all extensions intalled (Ctype, iconv, JSON, PCRE, Session, SimpleXML,Tokenizer, sqlite)
An attempt to run the command "symfony new . --demo" fails:

And other errors from web path:

Why? problems with downloading from another web (expetionproblems with downloading from another web (expetion)

directly out of the web when I have a download from soul browswer

Let's close this old issue because we haven't received similar reports from other users. Thanks.


chore: add PR template

Description
Adds PR template. Updates README
There are a few specific sections for Faency, which would require more attention:

breaking changes
good PR checkboxes
good review checkboxes

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Creat a donate now button

Hide the #razorpay button and instead of that create a #donate now button, and by clicking the button paymane getway should be opened.

NA


Full installation of Xcode.app is required

Upon trying to install via Homebrew, there comes an error:
brew install ohueter/tap/autokbisw autokbisw: A full installation of Xcode.app is required to compile this software. Installing just the Command Line Tools is not sufficient.
Therefore installation steps shall be updated.

Thanks for bringing this up. This requirement seems to be unexpected for many users, so we should definitely add this to the Readme. I think the Readme could generally use some improvements. I hope to get to this soon.

I pushed a revised README in 67db5a3. Please let me know if it's still missing relevant info! :-)

I appreciate your desire to make installing autokbisw as accessible as possible. In my opinion, when users are able to find a software on Github and are ready install it using the command line and Homebrew, they should be able to find out how to install Xcode on their own. I also don't want to maintain an external link in the documentation. But thanks again for your contribution, I really appreciate it!


"Cannot set percent" error when downloading folder using Get-SFTPItem

I'm not sure if this error has stemmed from the recent 2.2 update, but my folder downloads are failing when using Get-SFTPItem - it processes about a quarter of the files in question before crashing out.
I did some digging and found that its producing the following error on most of the files when it downloads them:
Cannot set percent because PercentComplete cannot be greater than 100.
Parameter name: value
Actual value was 25600

The actual value varies. I tried passing -NoProgress as a workaround, but no luck.

Thats a mew one I have not seen before. What ia the target server, version of SSH and size of the items in the folder so I can replicate
Sent from my iPhone

On Jun 17, 2019, at 2:32 AM, ryanjgillies<EMAIL_ADDRESS>wrote:
I'm not sure if this error has stemmed from the recent 2.2 update, but my folder downloads are failing when using Get-SFTPItem - it processes about a quarter of the files in question before crashing out.
I did some digging and found that its producing the following error on most of the files when it downloads them:
Cannot set percent because PercentComplete cannot be greater than 100.
Parameter name: value
Actual value was 25600
The actual value varies. I tried passing -NoProgress as a workaround, but no luck.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix Block.getLength() to be Unicode aware

As Block.getLength() is widely used, it should be Unicode aware.
It fixes not just Block.getLength() but also Modifier.applyEntity() which uses internally Block.getLength() to apply an Entity at a given range.
Let me know if you need more test coverage on this issue.

I'm not convinced that this is correct. This might be an annoying aspect of JS string handling, but it is consistent. Additionally, considering surrogate pairs as two characters matches the expectations of text node length in the DOM.
Are you encountering bugs related to this?

To clarify the point on DOM text node length, this is important for setting selection state at appropriate points within text nodes, handling autocorrect/spellcheck, and so on. We could instead translate back and forth from Unicode-aware to non-Unicode-aware for these purposes, but it seems simpler to me to deal with this kind of translation only when necessary, e.g. handling character deletion.

I get your point and I agree with you 👌
Also, as the documentation is explicit, we could use strlen(block.getText()) if we need to be Unicode aware.
But I'm still encountering several issues with surrogate pairs when using ContentState.createFromText() which uses block.length internally. See https://github.com/facebook/draft-js/blob/master/src%2Fmodel%2Fimmutable%2FContentState.js#L143
Should a surrogate pair be one or two characters in the 'Character List'?

Surrogate pairs are represented with two characters in characterList, to reflect the JS/DOM expectations of the string.
Can you describe the issues you're having with ContentState.createFromText()?

Thank you for the answer, I'll close this issue and open an other one once I have a clear script to reproduce the bug.


Only parse specific sheets in a workbook?

I'm wanting to extract data out of a certain sheet in a workbook.
The gem seems to process everything into memory. Some of the other sheets can be quite large.
Is there a way to tell the gem to only parse a certain sheet I am interested in?.

So far it is not possible, although I believe it can be implemented (although in that case the spreadsheet will become read-only, and it will not be possible to save it). Can you propose an API for that functionality?

Off the top of my head the same API etc would be cool, but perhaps just lazily evaluated?.
Eg returning the same workbook but only accessing the sheets when called?. :)

Lazy loading will require significant effort to implement.


Backfill tests for Title component.

What is the context of this PR?
Add test which asserts missing prop types errors for Title component.
This is the first attempt at writing a snapshot test which will fail if a component changes. Title seemed like the easiest component to begin with.
This is designed to prevent inadvertent changes to components as discussed here.
I will raise separate PR's for the other components.
How to review
Tests should pass and team should agree with approach of using snapshot tests to supplement component Unit tests.

 Storybook (5)


Add API for TaskProviders on tasks that possibly don't exist yet

A common task ordering papercut I run into is the need to depend on a task that I know will exist, but doesn't exist at the time I register it.
val knownTestTask = "testReleaseUnitTest"
val dependentTask = project.tasks.register("dependent") {
  // ...
}

// Wait for the task that matches to appear in tasks, configure then
project.tasks.matching { it.name == knownTestTask }
  .configureEach {
    val task = this
    dependentTask.configure {
      dependsOn(task)
    }
  }

This is a pain with ordering, and often a crash for unsuspecting plugin authors that try to use tasks.named(knownTestTask) only to be hit with an exception if it doesn't exist yet.
The worst case scenario is that an author may just punt it to an afterEvaluate {} block and hope for the best.
I'd like to propose a new API for a "future" task that returns a TaskProvider for a task that will eventually exist.
val knownTestTask = "testReleaseUnitTest"
val knownTestTaskProvider = project.tasks.futureNamed(knownTestTask)
val dependentTask = project.tasks.register("dependent") {
  // ...
  dependsOn(knownTestTaskProvider)
}

Naming suggestions definitely welcome!
Another consideration would be cases where the task never exists, which I think could be ok and just effectively a no-op. If it's mapped, it would just be an empty provider.
Another benefit of this is that it could allow reuse of the TaskProvider, whereas the current configureEach-based solution requires a new filter every time one wants to use it.

Another option could be to limit this just to task dependencies and make it an API on TaskProvider
val myTask = tasks.register(...) {}

myTask.maybeDependsOn("anotherTask")

Or use the maybe semantic for the provider name
val eventualTask = tasks.maybeNamed("anotherTask")

If you know the name of the task, you can just do:
project.tasks.register("dependent") {
  dependsOn("testReleaseUnitTest")
}

My understanding is that doesn't work if the task doesn't exist yet

Nope, the names are resolved while building the task graph.
tasks.register("foo") {
  dependsOn("bar")
}

tasks.register("bar")

sometimes I want to configure a task that doesn't exist yet (you know, Android build variants...)
I'm forced to use:

afterEvaluate

or

tasks.matching but when using matching means I can't know if there is any match, so I can't fail if there is none.

so, maybe it makes sense to have a "lazy named"
(another option would be rewriting AGP in a way it's not necessary but that's more difficult)

Just want to echo what @CristianGM mentioned above. We have this exact issue in a couple of projects, and with the new Android 7.x DSL we can no longer mix this with afterEvaluate in many cases.

AGP 7 provides some APIs that help in many cases, but anyway Gradle should provide that feature, not just expect that all plugins out there are perfect.
I mean, we use AGP as use case, but even if AGP fixed all the cases, not yet true, I think Gradle should still implement it.

Enjoyed @melix blog post on the matter
Dropped this in my build which cleans things up a bit, but does still waste generating the exception.
Would be great to have this added to Gradle model officially
fun <T> NamedDomainObjectSet<T>.findNamed(name: String):
    NamedDomainObjectProvider<T>? {
  try {
    return this.named(name)
  } finally {
  }
}

I don't follow how that helps?

I want to be able to find out, and get, a lazy task (TaskProvider) without an exception
every method I could find either returns a Task (not provider), or a TaskProvider but then throws an exception if not there
This just a bit of boilerplate so I don't have to litter my main Gradle scripts with it

For this you have a much simpler solution which is tasks.names.contains(...). This issue is really about being able to configure a task if it happens to exist at some point in the build.

@melix I guess I was thinking that the names method seems to do a bunch of creating of collections, including TreeSet, when called. So seems to me a new method around provider, that aligns with the existing  capability of find/get would make sense. Also see how that may not apply to this reported request - so I'll stop chiming in here ;-)
Thanks for the pointers nonetheless.

Here's a workaround that works for me. taskNamed(...) will lazily provide a task.
// build.gradle.kts
fun taskNamed(taskName: String): Provider<Task> = providers.provider {
  taskName
}.flatMap {
  tasks.named(it)
}

A quick demo. Even though 'world' is eagerly created before 'hello' is registered, the dependency still works.
tasks.create("world") {
  dependsOn(taskNamed("hello"))
//  dependsOn(tasks.named("hello")) // causes exception: Task with name 'hello' not found in project
  doLast {
    println("World!")
  }
}

tasks.register("hello") {
  doLast {
    println("Hello,")
  }
}

This should really be the default behaviour of tasks.named(...). If a user wants a task to be available immediately then they can do tasks.named(...).get() or tasks.getByName(...).

@aSemy for your workaround, I've seen issues where TaskProvider<Task> works to properly chain dependencies, but Provider<Task> does not. For example, I had 2x TaskProvider<Task> that I needed to map into a single Provider<Task> based on a gradle property, but the task dependency chain properly. Moving to a ValueSource<TaskProvider<Task>> solved this IIRC

Unfortunately, the project.provider workaround does not return a TaskProvider but a Provider<Task>, so calling configure is not possible.


Errors running got-your-back

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/got-your-back. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GYB release from https://github.com/jay0lee/got-your-back/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/got-your-back/wiki

Full steps to reproduce the issue:

Run command on Linux $ python3 gyb.py<EMAIL_ADDRESS>--action estimate > select Allow on opened browser > error appears:
Terminal error: 403: Insufficient Permission - insufficientPermissions.
Run command on Mac $ python3 gyb.py<EMAIL_ADDRESS>--action estimate > error: Traceback (most recent call last):
File "gyb.py", line 49, in
import sqlite3
File "/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/sqlite3/init.py", line 23, in
from sqlite3.dbapi2 import *
File "/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/sqlite3/dbapi2.py", line 27, in
from _sqlite3 import *
ImportError: No module name '_sqlite3'.

Expected outcome (what are you trying to do?):
Back up gmail emails and labels, so I can transfer emails and labels to a new gmail account.
Actual outcome (what errors or bad behavior do you see instead?): Errors as above.

This is fixed and now works.

Thanks for the update.


angularjs integration

Hi there,
Thank you for this amazing job.
How can in integrate the picker with angularjs application ?
i Hope to see an angular version.
Thanks !

+1

+1

+1

+1

+1

this can easily be done with a custom directive, I don't think it is within the scope of this library to support a integration into one specific framework

Yes... I created a filter who converts plain text in emoji using  emojione.shortnameToImage.
something like this:
app.filter('emojify', function() {   return function(input) {    return emojione.shortnameToImage(input);   } });
and used {{ input | emojify }}.
But... If you have to bind html, i suggest to create a custom directive, and link function will come like this..
`
link: function(scope, element, attrs){
        var template = '

'+emojione.shortnameToImage(scope.post.text[0])+'

';
        element.append(template);
    }

`
Hope it helps Someone :partly_sunny:

When I import wdt-emoji-bundle.min.js and emoji.min.js (I am using JSPM and es6 style of coding), and then, when I use wdtEmojiBundle.init, I get error "wdtEmojiBundle is not defined"

@Samar that is obvious, since wdtEmojiBundle is not imported. You've to find a way whereby you get access to all the methods inside wdtEmojiBundle object. In angular style for web, I would directly use it as $scope.wdtEmojiBundle.

@nageshd3 I am using controller as methodology and I have used  this.wdtEmojiBundle but still same error.

@SamarRizvi if you are using angular 2 with typescript:
you need to declare wdtEmojiBundle with
declare var wdtEmojiBundle:any;

@nageshd3 @SamarRizvi
I am having the same issue as @nageshd3 and I am using angular1. I'm trying to get the emoji picker to display inside of an input field that's found within an html template associated with a controller. When I try to init this bundle in the controller (by way of a simple this.wdtEmojiBundle.init() in the controller, I get the same error that @nageshd3 had.
As others have suggested, it seems this is not meant to be used within an angular app.

Hi, I'm having some issue, I'm trying to install for angular 2 & typescript, but I don't understand how to implement this :disappointed: someone can help me please ?

Same issue here .. ! I can't integrate

My issue is  when i want to execute wdtEmojibundle.init(''.my-input-selector") it tells me it's not a constructor :/ And I'm not sure where exactly i could be wrong? Did someone also had the same issue?

@VLazzarova Can you try with capital letter of B: wdtEmojiBundle.init

@needim It's actually how I wrote it,  wrote it with a typo inhere 🙈
So after i add the scripts and i call my function.. do i have to initialize the ** wdtEmojiBundle** Object somewhere else?

@VLazzarova Oh this is an issue about Angular :/ I'm not familiar with Angular :/

@needim I found sth. I think it throws me the error cause of the emojiConvertor. In the init function it calls this constructor, so it throws me the error it's not a constructor. It actually prints the emojiConvertor on the console as _  (and my error is exactly: " _ is not a constructor")
Some thoughts on that may be?

@VLazzarova hmm emoji.min.js is a dependency. It's in the repo. Maybe you missed that part?

@VLazzarova check the file emoji.min.js is being loaded when you initialize controller.  I was loading wdt-emoji-bundle.min.js first and then the emoji.min.js
Thank you for the help !  🙌🏼

@VLazzarova nice! I guess I'm not good at documentation section, I will work on that.  :/

@needim a getting this error what do you think could be the problem..?  Uncaught TypeError: Cannot read property 'querySelector' of null
at Object.h.init (wdt-emoji-bundle.min.js:16)
at

If anyone has been able to make an integration with AngularJS work for the emoji picker, please do share the code/repo.


update CCI biomass and NASA JPL

merge these to production :

https://github.com/MAAP-Project/biomass-dashboard-datasets/pull/106
https://github.com/MAAP-Project/biomass-dashboard-datasets/pull/107

ope


Azure Active Directory - Universal with MFA support never renews grants

Azure Data Studio Version:

Version: 1.27.0 (user setup)
Commit: 765c7064d46253d88393e0e1ff78874b6fe8fa64
Date: 2021-03-16T21:24:59.168Z
VS Code: 1.51.0
Electron: 9.4.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.17763
Steps to Reproduce:

Add a db connection using Azure Active Directory - Universal MFA support
Add user account within the connection
Eventually the connection fails with invalid_grant message
Only way to fix is remove Azure user and re-add, even though I'm bounced back to my login it never seems to actually renew the grant

Response error! - {"error":"invalid_grant","error_description":"AADSTS50173: The provided grant has expired due to it being revoked, a fresh auth token is needed. The user might have changed or reset their password. The grant was issued on '2021-03-11T13:11:19.3170000Z' and the TokensValidFrom date (before which tokens are not valid) for this user is '2021-03-23T09:26:23.0000000Z'.\r\nTrace ID: 1f3e2556-1ef8-45c6-9fbd-7d8d5d472200\r\nCorrelation ID: 0fd65cfe-73aa-4f08-80bb-bad346f29757\r\nTimestamp: 2021-03-24 10:05:03Z","error_codes":[50173],"timestamp":"2021-03-24 10:05:03Z","trace_id":"1f3e2556-1ef8-45c6-9fbd-7d8d5d472200","correlation_id":"0fd65cfe-73aa-4f08-80bb-bad346f29757","error_uri":"https://login.microsoftonline.com/error?code=50173"}

Same issue here. No workaround found yet and I cannot work with my databases, so is quite impactful.
I'm looking for some profile folder from where I can remove the previous grants. Any pointers where the folder is?
Version: 1.27.0 (system setup)
Commit: 765c7064d46253d88393e0e1ff78874b6fe8fa64
Date: 2021-03-16T21:24:59.168Z
VS Code: 1.51.0
Electron: 9.4.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

https://github.com/microsoft/azuredatastudio/issues/12637 duplicate


FluentD not sending application logs to open telemetry collector

Hello, I am trying to send all application logs of each pods to open telemetry collector . What configuration  I have to use for that.
Currently I am using this:
   <match *.*>       @type forward       <server>         host <OTEL_HOST>         port <OTEL_PORT>       </server>     </match>
This configuration is forwarding fluentD pods logs to open telemetry, but I need to send application logs running on EKS, which are  forwarded  to cloud watch too.
I am using fluentd-kubernetes-daemonset:v1.7.3-debian-cloudwatch-1.0 this image currently.

@saritatewari4 for me this works:
Stock config from here https://artifacthub.io/packages/helm/fluent/fluentd
Modified 04_outputs.conf section:
      <label @OUTPUT>
        <match **>
          @type forward
          send_timeout 60s
          recover_wait 10s
          hard_timeout 60s

          <server>
            name opentelemetry-collector
            host opentelemetry-collector.opentelemetry.svc.cluster.local
            port 8006
            weight 100
          </server>
        </match>
      </label>

Then using helm chart opentelemetry-collector https://artifacthub.io/packages/helm/opentelemetry-helm/opentelemetry-collector
I put this in values.yaml:
  ports:
    fluentforward:
      enabled: true
      containerPort: 8006
      servicePort: 8006
      hostPort: 8006
      protocol: TCP

and
  config:
    receivers:
      jaeger: null
      fluentforward:
        endpoint: ${env:MY_POD_IP}:8006

Hope it get you some inspiration


Addresses: #168 - Project Summary: Recently Viewed Items - Update Aft…

…er Ordering Panel closes
Recently Viewed Items in Project Summary only rearrange after closing the ordering panel, this is far less distracting than rearranging upon opening the panel.

Does this work if you click the View Project button after provisioning a service? I'm not sure the ordering panel is ever actually "closed" in that case.

Might need to add something to $scope destroy to add the item if the panel is open.

Might need to add something to $scope destroy to add the item if the panel is open.

Added $onDestroy() to update Recently Viewed if nav. away when ordering panel is open.

@spadgett when you get a moment, review/merge, thanks!


versioning

Is your feature request related to a problem? Please describe.
currently we have to change the version number in

include/yacx/main.hpp
doxyfile
CMakeList

Describe the solution you'd like
I would like to only change it in one location and have the other locations pull it from there
btw: I want to make a 0.4.2 and in general I want to keep track of changes, so if @LukasSiefke could write a changelog for the JNI in form of https://github.com/ZerataX/yacx/blob/master/CHANGELOG.md that would be super useful. I know it's just for me, but I do love it a lot

Changes

more java- and scala-examples
JNI

benchmark kernels
junit tests
throw java-execption when passing illegal arguments (such as null)
split up KernelArg-class into BooleanArg, ByteArg, ShortArg, IntArg, LongArg, FloatArg, DoubleArg-classes
executor-class for easily launch and benchmark kernels

going to create that as soon as possible, but after we've merged the whole sbt stuff we should create a v0.5.0


kubelet/container: Move ShouldContainerBeRestarted() to runtime.

This moves ShouldContainerBeRestarted to runtime, so it can be reused by rkt.
But I hate the function name, and the function input is ad hoc. Add a TODO to remind us to refactor this later.
@vmarmol @dchen1107 @yujuhong @jonboulle

LGTM, may have to be rebased once #7610 goes in, or the other way around if the opposite is true :)

Sure :)


Fixed how MediaTypeHeaderValue is created

It's not possible to create a MediaTypeHeaderValue from a string when it contains a charset — instead, the Parse method should be called (or MediaTypeHeaderValue should be created from a string containing only its content type, and then the charset should be set in the Charset property).
This PR fixes this issue.

Thank you very much @Thecentury for your work :) Have a nice weekend. Contributions are always welcome.

Thank you very much @Thecentury for your work :) Have a nice weekend. Contributions are always welcome.

Thank you for promptly merging the fix!


[Bug]: It does not work inside wizard

What happened?
When I try to use it within one of the wizard steps, everything appears fine, but the items seem unselectable, as if they are all disabled
How to reproduce the bug
try to use it inside one of the wizard steps
Package Version
v1.2.8
PHP Version
8.2
Laravel Version
11
Which operating systems does with happen with?
Linux
Notes
No response

@ashkesarmaa did you found a solution here? i have the same issue.

Hi all,
I have had the same issue just now. I was able to solve it by adding './vendor/jaocero/radio-deck/resources/views/**/*.blade.php', to the content array in my tailwind.config.js and rendering my assets again (npm run build).
For clarification, like so:
import preset from '../../../../vendor/filament/filament/tailwind.config.preset'

export default {
    presets: [preset],
    content: [
        // ...
        // ...
        './vendor/jaocero/radio-deck/resources/views/**/*.blade.php',
    ],
}

Hopefully this will solve this problem for you as well. Have a good day.


Add resource attributes

:+1: This becomes reality with https://github.com/signalfx/splunk-otel-js/pull/312
I will leave it open for now.

Codecov Report

Merging #316 (534e49b) into main (a762e2c) will decrease coverage by 0.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #316      +/-   ##
==========================================
- Coverage   94.91%   94.57%   -0.34%
==========================================
  Files          10       10
  Lines         295      295
  Branches       76       76
==========================================
- Hits          280      279       -1
- Misses         15       16       +1

Impacted Files
Coverage Δ

src/tracing.ts
97.95% <0.00%> (-2.05%)
:arrow_down:

Continue to review full report at Codecov.

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


Dynamic Data

HI，thank you for making this useful tool，I have some problems
{list.map((item, index) => {
     return (
        <div key={index}>
            slide
        </div>
      )
    }) : null
}

but swiper doesn`t work，I can't drag it，please tell me how to solve this problem，thank you！

Do you have any console errors. Have you imported the swiper css stylesheet from node_modules?

There are no errors here，I have imported the swiper css from node_modules. I find that if code like the example will be ok，Have you ever used Array.map to create elements？

Yes, using Array.map is fine as long as your wrap the slides with a <div>. You are doing this correctly. What are you doing with : null on line 7 of your example?

that is needless code, I have deleted it , but it doesn`t work. I will try again. thanks a lot!

I have the same problem.
With map(), the additionnal css are not applied to dynamic element generated by the map(), even if we using

Found my solution after 2 hours.
For dynamic data, I have to use : rebuildOnUpdate: true.
Because I use {this.props.children}.
So the divs are read by Swiper but Swiper don't apply anything on them. It's like he receive the divs after the initialization...
So rebuildOnUpdate, do the job once.

Set shouldSwiperUpdate={true}. Worked for me.


Trailing comma in the dataflow-configuration-metadata.properties causes undesired results

Whilst generating a docker image for a Spring Cloud Dataflow Stream Application, it was noticed that if there is a trailing comma in the dataflow-configuration-metadata.properties, the resultant image label org.springframework.cloud.dataflow.spring-configuration-metadata.json contains more entries than expected.
Expected Behavior
When generating a docker image the org.springframework.cloud.dataflow.spring-configuration-metadata.json should only contain entries for the filtered classes (defined in the property configuration-properties.classes).
Current Behavior
When there is a trailing comma in the list of configuration-properties.classes from the dataflow-configuration-metadata.properties file, the code does not handle an empty String and causes the filtering not to work as expected and it includes ALL properties. In our case 408 vs 16.
Possible Solution
Check for empty string after splitting the value for the property configuration-properties.classes. Currently the string is already trimmed, but it does not check for empty strings.
Steps to Reproduce
Add a trailing comma in the string stored in the property configuration-properties.classes in the  dataflow-configuration-metadata.properties file. This should result in the org.springframework.cloud.dataflow.spring-configuration-metadata.json label to contain ALL properties from the project instead of only the filtered properties defined in the configuration-properties.classes property.
Motivations
This issue has caused the listing of too many properties for a defined Spring Cloud Dataflow Stream app. This issue is subtle and easy to make. It is harder to find or root-cause without debugging the buildpack code or by knowing what to look for.

not sure if it's a blocker, but all these properties do have sourceType missing from the spring metadata json file, and that's why they are matched with an empty class name.
Might it be ever useful to include them? I don't believe it, but maybe I'm missing some context

@tinedel given that all of this works without the trailing comma, makes me think that by ignoring the empty class name will be a perfect solution.

@kohlmu-pivotal can you confirm if this issue is resolved? The PR with the fix was merged and should be released

I think we can mark this issue as resolved. I will raise a ticket on our internal systems to test this out, the next time we are in the project where we found this issue. Given that this issue was stumbled upon by accident, and the "fix"/ work around was to remove the trailing comma.


How do you add a custom PreProcessor?

Effectively the title.
https://python-markdown.github.io/extensions/api/#example describes how to define a custom PreProcessor, but I can't figure out how the heck you're supposed to then use it.
Looking around at some other issues it seems that people are defining a custom extension just to inject their preprocessor. That seems a bit janky of a solution. I could I suppose just append my preprocessor to Markdown.preprocessors after the main Markdown() class has been instantiated, but monkey-patching like that seems fragile.
What is the proper way to use a custom preprocessor? And would it be possible to get the documentation updated to show the new method?

So there's zero way to add a preprocessor without a extension?
That seems like a kind of arbitrary and actively unhelpful requirement.

That seems like a kind of arbitrary and actively unhelpful requirement.
That seems like a kind of arbitrary and actively unhelpful requirement.

This is literally how any library that has a workflow like this works. I don't see how this is unhelpful as we provide a supported way in which to run preprocessors. If you have trouble with the API, we offer support to help out in either understanding parts that are not clear or fixing issues that have been overlooked.  With that said, let's rephrase. This is the only way supported in the library.
Could you subclass the Markdown class and inject your own code to perform some preprocessing, sure, but it isn't supported, and any issues doing this will be yours to figure out.
Could you create your own wrapper script that processes the content and then hands it over to the Markdown class to do its thing, sure, but again, this is an external process completely disconnected from this library, and any issues you have doing this are yours to figure out.


Failing unit tests on macos

A non-deterministic failure of unit tests has been introduced with #387 and/or possible #389.
Current behavior
The PR checks were successful on #387, but the post merge checks once again failed the macos bmi c tests.
[  FAILED  ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_0_a
Returning unconverted value!
[  FAILED  ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_1_a
Unit conversion error:
[  FAILED  ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_1_b

With the following errors reported for these tests
[ RUN      ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_0_a
/Users/runner/work/ngen/ngen/test/realizations/catchments/Bmi_C_Formulation_Test.cpp:310: Failure
Value of: output
Expected: matches regular expression "-?0.000000,-?0.000000"
  Actual: "0.000000,-4.000000"
[  FAILED  ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_0_a (65 ms)

[ RUN      ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_1_a
/Users/runner/work/ngen/ngen/test/realizations/catchments/Bmi_C_Formulation_Test.cpp:326: Failure
Value of: output
Expected: matches regular expression "-?0.000000,-?0.000000"
  Actual: "-4.000000,0.000000"
[  FAILED  ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_1_a (57 ms)

[ RUN      ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_1_b
/Users/runner/work/ngen/ngen/test/realizations/catchments/Bmi_C_Formulation_Test.cpp:341: Failure
Value of: output
Expected: matches regular expression "-?0.000000,0.000001"
  Actual: "-4.000000,0.000001"
[  FAILED  ] Bmi_C_Formulation_Test.GetOutputLineForTimestep_1_b (133 ms)

Expected behavior
These three tests should return 0.000000 (or possibly -0.000000, though I'm starting to suspect that the -0.0 is related to the same bug???)
Steps to replicate behavior (include URLs)

Seems non-deterministic, and doesn't fail consistently (local testing on various macbooks haven't been able to reproduce yet...)

uh....hmm. Those values are much larger than -0.000000.

Introduction of a nondeterministic bug in #389 doesn't seem possible--triggering of it maybe...of course it could have been around longer... that's the problem with randomness

I think I have tracked down the issue with this (and why we didn't see it before, though the bug was still technically there...).
It is related to https://github.com/NOAA-OWP/ngen/blob/a099619ed9f54fbbeacbbebbbe9868cd9d76df38/include/realizations/catchment/Bmi_Module_Formulation.hpp#L454
There is a subtle bug here that actually makes #387 NOT work correctly in a non-multi BMI setting or, even in multi-bmi but when a module doesn't provide a certain variable...
I'll document that bug in more detail, and I have a patch that just needs cleaned up a bit and pushed.

See #393 for details.  I think the undefined behavior of the self referencing BMI variable is to blame of this failure.  I removed the -0.0 allowance in the unit tests on #394 to check.


Test phrase passed even though Test cases failed

Before opening, please confirm:

[X] I have searched for duplicate or closed issues and discussions.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

JavaScript Framework
React
Amplify APIs
Not applicable
Amplify Categories
hosting
Environment information

I am using Jest as the Test runner

Describe the bug
The Jest test cases fail on Test Phrase in Amplify pipeline, but it still continue to proceed instead of stopping.
Expected behavior
The Test Phrase should be failed and stop proceeding to Deploy Phrase
Reproduction steps

commit the changes in branch and push to remote
The pipelines in Amplify runs
The Jest test cases fail in Test Phrase, but the pipeline still continues
The application deploys successfully

Code Snippet
// Put your code below this line.

Jest.config.ts
export default {
  preset: 'ts-jest',
  coverageProvider: 'v8',
  globals: {
    'ts-jest': {
      isolatedModules: true,
    },
  },
  testEnvironment: 'node',
  transform: {
    'node_modules/variables/.+\\.(j|t)sx?$': 'ts-jest',
  },
  transformIgnorePatterns: ['node_modules/(?!variables/.*)'],
};

amplify.yml
version: 2
frontend:
  phases:
    preBuild:
      commands:
        - npm install
    build:
      commands:
        - npm run build
  artifacts:
    baseDirectory: build
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*
test:
  phases:
    preTest:
      commands:
        - npm install
    test:
      commands:
        - 'npm test'
  artifacts:
    baseDirectory: ./
    configFilePath: '**/mochawesome.json'
    files:
      - '**/*'

Log output

// Put your logs below this line
2022-05-10T09:14:52.261Z [WARNING]: FAIL src/utils/formatters/formatters.test.ts
2022-05-10T09:14:52.286Z [WARNING]: ● formatters test › decimalPlacesRegex test › should return true - when the decimal of the string equals to decimalPlaces
                                    expect(received).toEqual(expected) // deep equality
                                    Expected: false
                                    Received: true
                                    28 |     it('should return true - when the decimal of the string equals to decimalPlaces', () => {
                                    29 |       const result = decimalPlacesRegex(DEFAULT_AMOUNT, DEFAULT_DECIMAL);
                                    > 30 |       expect(result).toEqual(false);
                                    |                      ^
                                    31 |     });
                                    32 |     it('should return false - when the decimal of the string is different from decimalPlaces', () => {
                                    33 |       const result = decimalPlacesRegex(DEFAULT_AMOUNT, 2);
                                    at Object.<anonymous> (src/utils/formatters/formatters.test.ts:30:22)
2022-05-10T09:14:52.301Z [WARNING]: Test Suites: 1 failed, 1 passed, 2 total
                                    Tests:       1 failed, 16 passed, 17 total
                                    Snapshots:   0 total
                                    Time:        1.309 s
                                    Ran all test suites.
2022-05-10T09:14:52.376Z [WARNING]: npm
2022-05-10T09:14:52.377Z [WARNING]: ERR! Test failed.  See above for more details.
2022-05-10T09:14:52.380Z [INFO]: # Completed phase: test
2022-05-10T09:14:52.380Z [INFO]: ## Tests completed successfully
2022-05-10T09:14:52.382Z [INFO]: # Starting test artifact upload process...
2022-05-10T09:14:59.382Z [INFO]: # Uploading build artifact 'testConfig.json'...
2022-05-10T09:14:59.950Z [INFO]: # Build artifact is: 6MB
2022-05-10T09:14:59.951Z [INFO]: # Uploading build artifact 'testArtifacts.zip'...
2022-05-10T09:15:00.172Z [INFO]: # Test artifact upload completed
2022-05-10T09:15:00.175Z [INFO]: No custom headers found.
2022-05-10T09:15:00.194Z [INFO]: # Starting build artifact upload process...
2022-05-10T09:15:00.667Z [INFO]: # Build artifact is: 4MB
2022-05-10T09:15:00.668Z [INFO]: # Uploading build artifact '__artifacts.zip'...
2022-05-10T09:15:00.686Z [INFO]: # Build artifact is: 4MB
2022-05-10T09:15:00.686Z [INFO]: # Uploading build artifact '__artifactsHash.zip'...
2022-05-10T09:15:00.849Z [INFO]: # Build artifact upload completed
2022-05-10T09:15:00.849Z [INFO]: # Starting environment caching...
2022-05-10T09:15:00.849Z [INFO]: # Uploading environment cache artifact...
2022-05-10T09:15:00.928Z [INFO]: # Environment caching completed
Terminating logging...

Hi @andychoww 👋🏽  can you try implementing the workaround suggested here please: https://github.com/aws-amplify/amplify-hosting/issues/2409#issuecomment-1027401956

Hi @hloriana tried but the test phrase still passes
version: 4
frontend:
  phases:
    preBuild:
      commands:
        - npm install
    build:
      commands:
        - REACT_APP_OPTIFI_PROGRAM_ID=${REACT_APP_OPTIFI_PROGRAM_ID}
        - REACT_APP_OPTIFI_EXCHANGE_ID=${REACT_APP_OPTIFI_EXCHANGE_ID}
        - REACT_APP_CLUSTER=${REACT_APP_CLUSTER}
        - REACT_APP_GOOGLE_ANALYTICS_TRACKING_ID=${REACT_APP_GOOGLE_ANALYTICS_TRACKING_ID}
        - npm run build
  artifacts:
    baseDirectory: build
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*
test:
  phases:
    preTest:
      commands:
        - npm install
        - npm install pm2
    test:
      commands:
        - 'npm test'
    postTest:
      commands:
        - npx pm2 kill
  artifacts:
    baseDirectory: ./
    configFilePath: '**/mochawesome.json'
    files:
      - '**/*'

I even tried below but still not working
version: 5
frontend:
  phases:
    preBuild:
      commands:
        - npm install
    build:
      commands:
        - npm run build
  artifacts:
    baseDirectory: build
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*
test:
  phases:
    preTest:
      commands:
        - npm install
        - npm install pm2
    test:
      commands:
        - CI=true npm test
    postTest:
      commands:
        - npx pm2 kill
  artifacts:
    baseDirectory: ./
    configFilePath: '**/mochawesome.json'
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

@andychoww please add - npx pm2 start npm -- start under preTest like in this example for running Cypress tests:
test:
  phases:
    preTest:
      commands:
        - npm ci
        - npm install wait-on
        - npm install pm2
        - npm install<EMAIL_ADDRESS>mochawesome mochawesome-merge mochawesome-report-generator
        - npx pm2 start npm -- start
        - 'npx wait-on --timeout 60 http://localhost:3000'
    test:
      commands:
        - 'npx cypress run --reporter mochawesome --reporter-options "reportDir=cypress/report/mochawesome-report,overwrite=false,html=false,json=true,timestamp=mmddyyyy_HHMMss"'
    postTest:
      commands:
        - npx mochawesome-merge cypress/report/mochawesome-report/mochawesome*.json > cypress/report/mochawesome.json
        - npx pm2 kill
  artifacts:
    baseDirectory: cypress
    configFilePath: '**/mochawesome.json'
    files:
      - '**/*.png'
      - '**/*.mp4'

@hloriana I have added that in preTest but still not working

Thank you for testing that out @andychoww. We are tracking this as a bug and will update you once it's been fixed. In the mean time, we are still investigating a possible workaround. Thank you for your patience!

Solution from @hloriana is about Cypress test, but @andychoww report about Jest tests.
That's why all previous conversation is useless.

Have the same problem with the Jest test, that been adopted from E2E example tutorial,
I solved it just with removing artifacts from test > artifacts section inside amplify.yml.


Encoding issues on VDCs descriptions

Description
VDC's description may have an encoding problem generating de XML file sent through the API. The Java library is not able to decode the file and sets the pullElements attribute to null.
The unhandled exception is:
[Fatal Error] :1:2519: Invalid byte 2 of 3-byte UTF-8 sequence.
To Reproduce
Set a VDC description with an accent mark, ie. "máquina", and make a request to the API to get the VDC pool from the Javalibrary.
Details

Affected Component: XML RPC API
Version: 5.8.1

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Check if related to #3232

Seems duplicated and fixed


Membership on project pages

Something is weird here. Sage worked on BodyVis (with Seokbin) on a new breathing sensor (though never matured to publication). He is appropriately listed on the BodyVis project page but BodyVis is not listed under his thumbnail...

I think this might be because Sage never published on this project.
I wonder if we should have the projects stuff only pop-up on mouseover...


Better formatted code examples

Updated README.md to include GitHub flavoured markdown to get some syntax highlighting.

Old PR


CDVPlugin error in xcode / ios

I installed this plugin and am seeing the following error in xcode when trying to compile and run the app for ios.  Does anyone know what I can do to fix this?
CDVPlugin class SpeechRecognition (pluginName: SpeechRecognition) does not exist.
ERROR: Plugin 'SpeechRecognition' not found, or is not a CDVPlugin. Check your plugin mapping in config.xml.
I ran the following commands to install the plugin.  Maybe I missed something?
sudo npm install ionic-native --save
sudo cordova plugin add cordova-plugin-speechrecognition
The plugin seems to work an android, I'm only having issues with ios.
Any help is appreciated.

ME TOO


CLOUD-55989 GUI: 'manage security configurations' provider types' lab…

…els are missing
CLOUD-56131 GUI: 'manage recipes' plugins and their types' labels are missing
@doktoric or @mhmxs plz check

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2357/


Spark integration can cause YARN applications to crash if Spark context is explicitly stopped

Related to https://github.com/OpenLineage/OpenLineage/issues/2511
We have jobs that take a very long time to run buildInputDatasets (> 90s). This means the listener events are processed after spark.stop() (the Spark context is stopped in our application), which is problematic because it seems like the AM continues to request containers from YARN even after the Spark context is stopped. These new containers crash because they're unable to connect to the scheduler, which eventually cause YARN application to be marked as failed.
Caused by: org.apache.spark.rpc.RpcEndpointNotFoundException: Cannot find endpoint: spark://CoarseGrainedScheduler@{ip}:{port}

I believe this is because in https://github.com/OpenLineage/OpenLineage/blob/987e5b806dc8bd6c5aab5f85c97af76a87[…]n/java/io/openlineage/spark/agent/OpenLineageSparkListener.java a SparkListenerSQLExecutionEnd event is processed after the Spark context is stopped - I believe createSparkSQLExecutionContext is doing something undefined in https://github.com/OpenLineage/OpenLineage/blob/987e5b806dc8bd6c5aab5f85c97af76a87b1de0f/integration/spark/app/src/main/java/io/openlineage/spark/agent/lifecycle/ContextFactory.java#L45. I verified this was happening by adding log statements that printed whether the Spark context was stopped when receiving the event.
I'm not sure if this is defined behavior for the Spark context to be accessed after its stopped? After I skipped the event in onOtherEvent if the Spark context is stopped it no longer crashes trying to spin up new containers in some of our jobs.
+  private boolean isSparkContextStopped() {
+    return asJavaOptional(SparkSession.getDefaultSession()
+      .map(sparkContextFromSession)
+      .orElse(activeSparkContext))
+      .map(
+        ctx -> {
+          return ctx.isStopped();
+        })
+      .orElse(true); // If for some reason we can't get the Spark context, we assume it's stopped
+  }
+
   @Override
   public void onOtherEvent(SparkListenerEvent event) {
     if (isDisabled) {
       return;
     }
+    if (isSparkContextStopped()) {
+      log.warn("SparkContext is stopped, skipping event: {}", event.getClass());
+      return;
+    }

However I don't think this is sufficient to patch this, we still have jobs that take >200s to process events fail even with this patch. It seems like this is because the Spark context is stopped during processing onOtherEvent so this check is insufficient.
Could https://openlineage.io/docs/integrations/spark/configuration/circuit_breaker/ be used to trigger a circuit breaker if the Spark context is stopped?

Yeah - can confirm, we're suffering from a similar issue. The listener is attempting to access information after the spark context has been stopped, which in our case is causing the job to fail and restart.

@d-m-h are you also using Yarn?

Nope. Spark on K8s.

Yeah, then I think any Spark context access needs to be guarded if we can't guarantee that any Spark context is available. The problem is that complete event might miss any necessary information, but this is preferable to failing or hanging.

Just for additional information @d-m-h - this was happening before, right? It's not a new bug?

I don't understand why this could be an outcome of some recent changes. If it was like this, we would be able to address it with an immediate fix. Based on the description provided, it seems like this is something we have forever. We're relying on a closed context and accessing it causes timeouts which cumulate for all the visitors and dataset builder.
I think this could be addressed with:

implementing in java client CompositeCircuitBreaker which can contain many circuit breakers
create SparkContextStoppedCircuitBreaker to be added by default.

Secondly, openlineage-spark should always catch all the exceptions possible. We're doing this for dataset visitors and builders. We should do this as well in ContextFactory but I am not sure where exactly is the exception thrown and where to catch it.

I think this could be addressed with:

implementing in java client CompositeCircuitBreaker which can contain many circuit breakers
create SparkContextStoppedCircuitBreaker to be added by default.

I don't understand how a circuit breaker would prevent the connector from triggering a request to YARN (or in my case, Kubernetes)

@d-m-h circuit breaker would not prevent triggering yarn request. But it would prevent creating dozens of such request and it would stop execution for a hanging request once it detects a stopped context.
It is still not clear to me how is this killing a job.

@mzheng-plaid Are you able to provide an exception pointing to OpenLineage code resulting in RpcEndpointNotFoundException? Would love to see why isn't the exception caught?

@pawel-big-lebowski the exception is not in OpenLineage code - its an indirect effect of YARN thinking new containers need to be spun up after the Spark session is stopped because of OpenLineage, this exception is in the container startup code (it makes sense these can't start up because the scheduler is stopped). After enough containers crash on startup YARN thinks the application itself is failed.
The reason I think the circuit breaker will work is because it should always happen fast enough to prevent YARN from attempting to spin up new containers.

Hi, any updates here?

@mzheng-plaid https://github.com/OpenLineage/OpenLineage/pull/2535 was merged. However, while it might fix the immediate issue, I'm not sure it fixes the underlying problem - as you yourself correctly noticed:

I feel like it should be more obvious that something is wrong (without actually failing the application), if in my setup something is taking 90s to process an event I'd rather OpenLineage do nothing until I can remediate. The partial data doesn't seem useful without these facets. Ideally I'd also prefer OpenLineage stop everything and terminate if the application itself were done - I definitely wouldn't want https://github.com/OpenLineage/OpenLineage/issues/2375 to happen and the infrastructure to remain up.

There are few tiers to answer that question.
First, it's obvious that we shouldn't take that long to generate lineage. I think one of the possible causes is slow implementation of column level lineage or bugs when defining datasets like https://github.com/OpenLineage/OpenLineage/issues/2511
Second is whether we can do it:

Ideally I'd also prefer OpenLineage stop everything and terminate if the application itself were done
because, really, it can be miliseconds between finishing a job and calling spark.stop().

I think one solution would be to refactor integration to

generate basic lineage information, that could be accessed just by getting LogicalPlan properties and not doing any advanced processing or network calls,
then do more potentially time consuming stuff: generate column-level lineage, ask external catalogs about table schemas and much more
if spark context stops somewhere after 1), we terminate 2) and send basic event generated from 1).

I don't think generally skipping emitting COMPLETE event at all, and misleading OL consumers that the application is still running is the desired solution - but limiting what we do then might be.

Thanks for the detailed answer @mobuchowski , your suggestion on refactoring seems at a high level reasonable. Overall as a user, its hard for me to advocate for OpenLineage to be enabled on all our apps if there is even a 0.1% chance of it causing Spark job failure - its hard to justify that risk. For now, we're operating a fork with the Spark circuit breaker I pasted above to unblock usage.
re: refactoring integration timeline, this seems like it may take a while? What should other users do in the interim?

I don't think generally skipping emitting COMPLETE event at all, and misleading OL consumers that the application is still running is the desired solution - but limiting what we do then might be.

This would also happen on any circuit breakers being triggered right?

@mzheng-plaid Could you turn on debugFacet (see https://openlineage.io/docs/integrations/spark/configuration/spark_conf) for the job where building inputs take more than 90s? This should contain some short desc of logical plan and can be helpful to find visitor or dataset builder which has a bug.
Would adding a timeout to an existing circuit breaker solve the issue of breaking others jobs? To me these two separate problems: (1) assure OL is not failing jobs (2) OL always emits termination events. I don't mind solving (1) while giving more time for (2).

(@pawel-big-lebowski sorry missed email - I added a reproducible example in https://github.com/OpenLineage/OpenLineage/issues/2511#issuecomment-2031232924 for the slow code)
I liked your original suggestion of being able to compose circuit breakers together (seems more flexible), but I think adding a timeout that's configurable to existing circuit breakers would work too

@mzheng-plaid thanks for comment (I realised the other issue later too). I like @mobuchowski approach.
For now I prepared timeout PR https://github.com/OpenLineage/OpenLineage/pull/2609 as a short term solution.


⚠ SceneTime has degraded performance

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

HTTP code: 200
Response time: 2016 ms

Resolved: SceneTime performance has improved in 53a760e after 6 minutes.


fix account signer crash

#4028  👈
main

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

Join @jnsdls and the rest of your teammates on  Graphite


Improve Suggestion: The actor param and some parameters can be omitted for authenticated users

1. What could be improved

Now
import 'package:bluesky/bluesky.dart' as bsky;

Future<void> main() async {
  // Session of authenticated user.
  final session = await _session;

  final bluesky = bsky.Bluesky.fromSession(session);

  // You have to pass did or handle now.
  await bluesky.feeds.findFeed(actor: session.did);
}

Should
import 'package:bluesky/bluesky.dart' as bsky;

Future<void> main() async {
  // Session of authenticated user.
  final session = await _session;

  final bluesky = bsky.Bluesky.fromSession(session);

  // No need to pass for authenticated user's identifier
  await bluesky.feeds.findFeed();
}

2. Why should this be improved

3. Any risks?

4. More information

ActorsService

findProfile
findProfileAsJson

FeedsService

findFeed
findFeedAsJson
paginateFeed
paginateFeedAsJson
findActorFeeds
findActorFeedsAsJson
paginateActorFeeds
paginateActorFeedsAsJson
findActorLikes
findActorLikesAsJson
paginateActorLikes
paginateActorLikesAsJson

GraphsService

findFollows
findFollowsAsJson
paginateFollows
paginateFollowsAsJson
findFollowers
findFollowersAsJson
paginateFollowers
paginateFollowersAsJson
findLists
findListsAsJson
paginateLists
paginateListsAsJson


LayoutLM Mixin

What kind of change does this PR introduce?
Adds the LayoutLM ReplayStrategy Mixin class.  Addresses #187

Summary
Currently the only implemented method is the one for document question answering, as object detection in GUI screenshots is still being explored. Also, table transformers are also being explored as a possible solution to use in combination with LayoutLM to extract tables and the content within.

Checklist

[x] My code follows the style guidelines of OpenAdapt
[x] I have perfomed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. README.md, requirements.txt)

How can your code be run and tested?
Mixins are not meant to be run, however class can be individually instantiated and tested for document question answering.

Other information

```
    questions = ['What is the sender\'s phone number?',
                 'What is the sender\'s address?',
                 'What is surcharge?',
                 'What is the total cost?',
                 'What is the daily interest rate on overdue balances?']
```
Output:
321-956-7331,
MELBOURNE,
45.00
$ 3,165.00
0.5%

One step towards being able to obtain all the useful information from a document without knowing what to ask it beforehand may be: feeding the document type to another model and output a list of questions, loop over them and feed each one to the pipeline and create a dictionary of outputs this way.
@abrichr thoughts?

https://github.com/huggingface/transformers/issues/24531 Gonna have to wait a bit for the GUI screenshot example, HF pipeline isn't working as intended for some reason

questions = ["What is the URL of the page?",
             "Is interns a valid channel?",
             "What is the title of the Youtube video?",
             "How many buttons are there at the top of the page?",
             "is Direct messages a valid subheading?",
             "What is the name of the current logged in user?"]

Output
https://www.youtube.com/watch?v=ut5kp56wW_4&t=0s
random
Yannic Kilcher
40
Owais Zahid
https://www.youtube.com/watch?v=ut5kp56wW_4&t=0s

Gives the youtube video URL as the page's URL, and it output my name for a Yes/No question 😆

Actually, I'm going to try using Donut for GUI detection.

TODO:
add 4-5 tests.

@FFFiend please name branches more descriptively 🙏


[RFE / POC] Implement MiqExpression interpreter

Overview
MiqExpression can run in both sql and ruby.
When it runs in ruby, it produces a string, regular expressions replaces the string a few times and then it is run through eval.
A. Seems like the regular expression gsub in _subst shouldn't be necessary
B. The regular expression swapping takes a lot of time.
C. We want to drop eval
D. We want to better understand the code / make it more maintainable.
@Fryguy goal: Lets walk the miq expression tree with a visitor instead of eval(generated_string).
@kbrock goal: Lets first generate a ruby string that doesn't require the subst{_find} to put values into it.
This proposal I consider my goal a stepping stone to get the interpreter.
Mid Goals
Removing code and and reducing processing:

a single generated tree that does not need to be modified between each subst_match?
only parse field/value once per node and determining action.
fully understand/process intent of each operation once (at preprocess time?)

Steps

[ ] ? convert preprocess_exp!(exp.dup) to preprocess_exp(exp)
[ ] preprocess_exp! converts all operations to lowercase once (vs 15 operator.downcase)
[ ] preprocess_exp! adds Field object for field and value into the tree (vs repeated Field.parse, col_details, is_field?)

drop col_details cache and get_cols_from_expression
may need to cache something for includes_for_sql
drop MiqExpression.get_col_info, MiqReport.get_col_info from the ui if possible.

[ ] preprocess_exp! consume valid?.
[ ] evaluate_atoms no longer modify expression. (This will become the interpreter)
[ ] evaluate_atoms s/obj/rec/ in params for consistency

Aggressive changes

[ ] Do we use find operator (can we drop)?
[ ] pass rec to Condition.do_eval so value can be used directly instead of substituted
[ ] _to_ruby "find" - generate correct code and drop Condition.subst_find(str, rec)
[ ] _to_ruby generate correct code and drop Condition.subst(str, rec)
[ ] preprocess_exp! convert hashes into s-expression structs

Making an Interpreter

[ ] copy paste _to_ruby strings (without the "'s) into evaluate_atom_helper
[ ] Profit

If we can drop Condition#mode="tag_expr", then we can change Condition#subst to take in an MiqExpression#exp and we can drop all the gsub calls for temporary <find> / <value...> nodes. Possibly even transition some of this code over to MixExpression#to_ruby.
subst takes up a majority of our object creation and execution time (ignoring database lookups). Also a single generation of the ruby code will simplify the code enough to make an interpreter that much easier.

ok, "tag_expr" was transitioned to "tag_expr_v2" in 2008.
But at this time we had a bunch of policies in product/policy/ and they looked very different. Now, I'm pretty sure we no longer support script: and they all have condition:. So this is probably a 10 year old relic that can be put to rest.
commit<PHONE_NUMBER>1e0cc24e05adf13fa5bbc85fa19200
Author: Gregg Tanzillo
Date:   Wed Apr 2 17:08:04 2008 +0000

Changes were made

I also commented here (https://github.com/ManageIQ/manageiq/pull/22989#issuecomment-2052639385) with similar findings as you.  I agree we can drop tag_expr.


Biber on large inputs

I switched to Biber, because the features are nice. Unfortunately, it turned out almost unusably slow (>5 mins) on huge bib-files, e.g. cryptobib. I tried to work around this, by hacking yet another latex "fix". (A script, which produces a ".aux" compatible with bibtex, then runs bibexport to get only the relevant entries (this breaks aliases and probably more) and then runs latex/biber on this reduced dataset.)
However, this "fix" introduces a lot of overhead, inconveniences, and will probably cause problems in the long run. I'd prefer to avoid it.
Is there a good workflow for Biber with large files?
I guess/hope (knowing nothing about Biber/Perl) that it's possible to improve Biber's handling of large files relatively easily.
Biber should be lazier: There is no need to "INFO - LaTeX decoding ..." every entry. (That's the part which takes the longest by far.) Only the (transitively) referenced ones are important. So, if Biber first gets rid of all irrelevant entries, and then does its work on that subset, this should speed up large file handling by a lot, and hardly affect small files. Perhaps, Biber could preprocess with a separate (fast) tool that does this? (The (only) difficulty I see lies in resolving aliases/crossrefs. Especially between different files.)

Have you look at bibers tool mode? You can output (without tool mode) a subset of only referenced references also by using --output-format=bibtex and the other --output-* options which control the output details. This will give you a .bib file with only the referenced items.
biber can't really be speeded up internally much - I've already tuned it a lot. It has to decode everything into UTF-8 to make sure sorting works correctly and sorting itself is very complex and complete, taking a long time. biber does far more than bibtex does and so any comparison is fairly meaningless at this point.

Tool mode wouldn't solve the problem, it would just be a (probably better) replacement for bibexport. The "switch-bib-files-logic" of the script would still remain.[0] But I did not see an option, which exports just the entries referenced (in some .bcf file). If there is, and if it's faster than a full run of biber, wouldn't it be a useful addition to have this preprocess the input?
That biber does much more than bibtex is clear. That it will be slow(er) is expected (mostly due to Perl vs C). Nevertheless:
Does biber's sorting have to take into account entries which are not referenced? Does it have to UTF-8-decode everything[1], before picking out the referenced entries? If the answer to both is No, then I really expect that delaying UTF-8-decoding as long as possible should improve speed for sparsely used large inputs by a lot.[2]
That is first picking out the (transitively) referenced entries. Here no is used.  Now, discard all unreferenced entries. Finally, apply the full logic to this subset of data. In my use-case, this gets from 25MB to 20kB (plus a 300kB file with strings, which I did not resolve.
If so, then I see no way to speed things up. If not, why can't biber get rid of all entries, which do not need to be sorted, because they are not referenced?
Anyway, if you don't see a simple, manageable way to speed up processing large files, feel free to close this issue.
Also, thanks for your work!
[0] It's vitally important that compiling is as simple as possible. Optimally toolchain is supported by the editor. Otherwise, collaborating with other people gets very difficult. This setup is far from user-friendly and very fragile. It's a proof-of-concept.
[1] In particular, are names/keys/aliases which are UTF-8 encoded allowed? Do you promise to normalise such keys, so that different representations of the same string are correctly detected? Even if, normalised keys are just bytestrings with no real semantics. If I understand correctly, UTF-8 lexical comparisons are the expensive operations.
[2] How simple this is probably depends a lot on how the content is parsed.

What people normally do is to use the full .bib on the first run but output bibtex format instead of bbl and that gives you a small .bib with only the referenced items which can be used on subsequent runs. This accords with the general policy that you can the archive the .tex and the new .bib and they are a self-contained unit to reproduce the document.
You do this by running biber with the --output-format=bibtex option (not tool mode).
biber's sorting does not sort anything not referenced. However, it does preprocess every entry in the .bib to make it UTF8 first because the btparse library used is ancient and had to be modified to pass through glyphs properly. btparse and bibtex are famously bad at handling encodings and so we have to normalise everything into UTF-8 before entering the C routines for parsing. Another reason for this is because .bib files are often a horrible mess of things pasted from all over the place and in order to stand a chance of local-sensitive sorting, we have to ensure everything is in UTF-8. Unfortunately, this has to happen before parsing (and therefore reference selection) to make sure that the parsing library gives us something consistent and sensible.

Thanks for taking the time to answer and for the --output-form=bibtex option. In general, I guess, there's nothing short of a rewrite of btparse (and probably a lot more) you can do to speed things up, due to too much legacy and unpredictable use of .bib-files.


TabPanel with index > 0 breaks TextField variant outlined

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When using a TextField with the outlined variant inside of a TabPanel with a non-zero index, the outline doesn't shift to provide space for the label to move up when selected.

Expected Behavior 🤔

The TextField should function as expected, with the outline shifting over so that the when the label moves up it isn't clashing with the outline of the TextField, inside of all TabPanels regardless of index.

Steps to Reproduce 🕹

Steps:

Go to the codesandbox example
The Item One tab should have a working TextField
The Item Two and Item Three tabs should have a broken TextField

Context 🔦

I just noticed the issue while working on a project. Right now my solution is to just use a different variant of the TextField, but I would prefer if I could use the outlined variant.
Your Environment 🌎

Tech
Version

Material-UI
v4.3.0

React
16.8.6

Browser
Chrome

It's a duplicate of #14530


Adds npmE alias

Option to execute a command from closest ancestor node_modules folder
Example
$ls
node_modules
other_folder
$npmE gulp
No gulpfile found
$cd other_folder
$npmE gulp
No gulpfile found

Here's the correct way to add quotes:
 alias npmE='PATH="$(npm bin)":"$PATH"'

This way the alias uses the dynamic $PATH every time, and the npm bin folder will only be set once, not use the appropriate one depending on the folder you're in.

@mcornella Thanks, fixed the quotes

:+1:
/cc @robbyrussell


L inf normball

Codecov Report

Merging #246 (2faf493) into master (675f6f6) will increase coverage by 0.11%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #246      +/-   ##
==========================================
+ Coverage   69.67%   69.79%   +0.11%
==========================================
  Files          14       13       -1
  Lines        1837     1748      -89
==========================================
- Hits         1280     1220      -60
+ Misses        557      528      -29

Impacted Files
Coverage Δ

src/polytope_oracles.jl
97.53% <0.00%> (-2.47%)
:arrow_down:

src/types.jl
66.31% <0.00%> (-1.58%)
:arrow_down:

src/oracles.jl
86.59% <0.00%> (-1.14%)
:arrow_down:

src/active_set.jl
79.36% <0.00%> (-0.64%)
:arrow_down:

src/fw_algorithms.jl
77.30% <0.00%> (-0.26%)
:arrow_down:

src/function_gradient.jl
97.05% <0.00%> (-0.17%)
:arrow_down:

src/afw.jl
74.25% <0.00%> (-0.16%)
:arrow_down:

src/norm_oracles.jl
97.46% <0.00%> (-0.10%)
:arrow_down:

src/FrankWolfe.jl

... and 2 more

Continue to review full report at Codecov.

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


About CVUSA dataset csv files

Hi,
I noticed that there are some codes try to read train.csv and test.csv files.
I downloaded the CVUSA dataset, but I don't see these 2 files.
May I know how generate these 2 files?
Or, where are they?
Thank you!

This is included in the CVUSA dataset. Please ask the author to get the dataset.


Unload checkpoint pickle after use

Checkpoint pickle gets loaded, but never unloaded, causing an OOM error for runs that supposed to fit.
Using settings: python train.py --outdir=./training-runs/ --cfg=fastgan --data=pokemon256.zip --gpus=1 --batch=8 --mirror=1 --snap=10 --batch-gpu=8 --kimg=10000 --metrics=None I was able run training but not resume it while using a single 8GB GPU, all because the checkpoint pickle was wasting over 1GB of memory.

Relevant to this issue:

I can train kimgs as long as I want with batch-gpu=4, on resume however I ALWAYS get a cuda out of memory? With batch-gpu=2 the resume works then. Why is that so?

#45


Run own RGB-D sequence from ROS

The work is so interesting. It does two things: location and reconstruction. I'd like to replicate your code in our own data set. So I captured a series of RGB-D data sets using the ros camera.Then run the recommended tutorial. However, I found that the Re-rendering loss keeps increasing, and when Tracking Frame 691, the following error is reported.
Tracking Frame  689
Re-rendering loss: 61943169.13->32158803.15 camera tensor error: 2.3272->2.3433
Tracking Frame  690
Re-rendering loss: 14259613.90->6811814.82 camera tensor error: 2.4108->2.4126
Coarse Mapping Frame  690
Mapping Frame  690
Tracking: update the parameters from mapping
Tracking Frame  691
[W CudaIPCTypes.cpp:15] Producer process has been terminated before all shared CUDA tensors released. See Note [Sharing CUDA tensors]
Process Process-1:
Traceback (most recent call last):
File "/opt/conda/envs/nice-slam/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/opt/conda/envs/nice-slam/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(self._args, **self._kwargs)
File "/workspace/nice-slam/src/NICE_SLAM.py", line 266, in tracking
self.tracker.run()
File "/workspace/nice-slam/src/Tracker.py", line 233, in run
camera_tensor, gt_color, gt_depth, self.tracking_pixels, optimizer_camera)
File "/workspace/nice-slam/src/Tracker.py", line 107, in optimize_cam_in_batch
self.c, self.decoders, batch_rays_d, batch_rays_o,  self.device, stage='color',  gt_depth=batch_gt_depth)
File "/workspace/nice-slam/src/utils/Renderer.py", line 109, in render_batch_ray
far = torch.clamp(far_bb, 0,  torch.max(gt_depth1.2))
RuntimeError: max(): Expected reduction dim to be specified for input.numel() == 0. Specify the reduction dim with the 'dim' argument.
I don't understand why there is such a mistake, can you help to solve it.Thanks!

Hi,
Thanks for your interest in our work!
I think you can ignore this specific RuntimeError since it is caused by large camera tracking error. You can focus on making tracking not drifting away. One way to debug is to directly provide pseudo ground truth camera poses (e.g. calculated by Open3D redwood).
Best wishes!

Hi, thanks for your interesting work! I've encountered the same problem caused by large camera tracking error. I'm wondering if the model can generate satisfying resulting without annotations of pseudo ground truth camera poses, as we'd like to test the algorithm on a pre-recorded video. Also, is there any way to obtain pseudo GT camera poses from the video not recorded by Azure Kinect?

Hi,
Thanks for your interest in our work!
The pseudo ground truth camera poses will not affect the system's result, but would be great for debugging.
The capture device does not matter, as long as you can get the depth/color/intrinsics needed for the Redwood in Open3D. https://github.com/cvg/nice-slam#use-your-own-rgb-d-sequence-from-kinect-azure
Best wishes!

Hi, thanks for your interesting work! I've encountered the same problem caused by large camera tracking error. I'm wondering if the model can generate satisfying resulting without annotations of pseudo ground truth camera poses, as we'd like to test the algorithm on a pre-recorded video. Also, is there any way to obtain pseudo GT camera poses from the video not recorded by Azure Kinect?

@AlvinYH
Hi~Do you know how to use the author's tutorial (https://github.com/cvg/nice-slam#use-your-own-rgb-d-sequence-from-kinect-azure) with a different camera? I've used my own camera (Kinect2) to get depth/color/intrinsics, but I'm confused about how to do it next.

The work is so interesting. It does two things: location and reconstruction. I'd like to replicate your code in our own data set. So I captured a series of RGB-D data sets using the ros camera.Then run the recommended tutorial. However, I found that the Re-rendering loss keeps increasing, and when Tracking Frame 691, the following error is reported.
Tracking Frame 689
Re-rendering loss: 61943169.13->32158803.15 camera tensor error: 2.3272->2.3433
Tracking Frame 690
Re-rendering loss: 14259613.90->6811814.82 camera tensor error: 2.4108->2.4126
Coarse Mapping Frame 690
Mapping Frame 690
Tracking: update the parameters from mapping
Tracking Frame 691
[W CudaIPCTypes.cpp:15] Producer process has been terminated before all shared CUDA tensors released. See Note [Sharing CUDA tensors] Process Process-1: Traceback (most recent call last): File "/opt/conda/envs/nice-slam/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap self.run() File "/opt/conda/envs/nice-slam/lib/python3.7/multiprocessing/process.py", line 99, in run self._target(_self._args, **self._kwargs) File "/workspace/nice-slam/src/NICE_SLAM.py", line 266, in tracking self.tracker.run() File "/workspace/nice-slam/src/Tracker.py", line 233, in run camera_tensor, gt_color, gt_depth, self.tracking_pixels, optimizer_camera) File "/workspace/nice-slam/src/Tracker.py", line 107, in optimize_cam_in_batch self.c, self.decoders, batch_rays_d, batch_rays_o, self.device, stage='color', gt_depth=batch_gt_depth) File "/workspace/nice-slam/src/utils/Renderer.py", line 109, in render_batch_ray far = torch.clamp(far_bb, 0, torch.max(gt_depth_1.2)) RuntimeError: max(): Expected reduction dim to be specified for input.numel() == 0. Specify the reduction dim with the 'dim' argument.
I don't understand why there is such a mistake, can you help to solve it.Thanks!

@szgy66  Hi~
Would you mind sharing your dataset with me? Thank you very much!


slowMo disables or interferes with page.click()'s scroll-into-view behavior

Steps to reproduce

Puppeteer version: 1.9.0
Platform / OS version: macOS Mojave 10.14
Node.js version: v8.12.0

What steps will reproduce the problem?
Set slowMo to at least 50.
// jest-puppeteer.config.js
module.exports = {
  launch: {
    slowMo: 50
  }
}

Use page.click(). In my example, I also had page.hover(). I'm not sure if it's related—I had to use it to reveal the div in which the click target is located.
await page.hover('.product-details .dropdown:nth-child(4)');
await page.click('.product-details #widget3 a[data-value="AT&T"]');

What is the expected result?
page.click() scrolls element into view
What happens instead?
I had several instances of those page.click() calls, and when I disabled headless mode, I noticed that the ones which were below the fold of the page in headless had this error:
Node is either not visible or not an HTMLElement

It looks like your page has some delayed behavior that you aren't properly awaiting.

@JoelEinbinder If that's the case, why would there be issues when slowMo is enabled but not when it's disabled? I'm assuming with slowMo enabled, any delayed behaviour on the page is less of a problem. Unless slowMo actually lengthens any existing delay?

@kriskhaira it might be because with slow mo, page is given more time to do its operation; it looks like there's some logic in the page that removes the .product-details #widget3 a[data-value="AT&T"] element from the page.


CrScreenshotDxe: Work around TPL-related ASSERT within firmware compiled to assert

references acidanthera/OpenCorePkg#539

cc @mikebeaton


Support/test ddev on Windows 10 Home (Or with Docker Toolbox anywhere)

What happened (or feature request):
One of the issues identified in the conversations about Drupal sprints is that there are some sprinters that come to sprints with Windows 10 Home. I think we should be able to support Docker Toolbox on Windows 10 Home. We could look at managing the Docker VM on behalf of the user as well, but that might not be necessary out of the gate - documentation is also a valid approach to solving that problem.
This might also be useful for Windows 10 Pro users who don't want to/can't enable HyperV for whatever reason (some enterprise Windows deployments lock down HyperV, but Virtualbox still works fine). This is very common on government laptops as well.
What you expected to happen:
How to reproduce this:
Version: Please include the output of ddev version, docker version and the project's .ddev/config.yaml.
Anything else do we need to know:
Once we're done with Phase 0 and 1 of #621, it would be pretty straightforward to bring up a Windows 10 Home machine on MacStadium and add that to our fleet of test bots.
Related source links or issues:
Please use a complexity rating of 1-5 (5 is high) for a feature request. (High complexity implies more PR planning)

I tried win 10 home way back in the day with docker toolbox and it worked if you just set up the env var for docker host. That's basically the same as what you have to do to use wsl also. However... Who knows today.
Our decision to support docker for Windows was about limiting permutations, not really feasibility.

Yep, drud/mariadb-local#53 works fine on Win 10 Home with Docker Toolbox. But the performance (see the issue) is pretty abysmal. @cweagans suggests there might be tolerable workarounds on this.


SIGINT-859: Support multi version for CNC 2023.6 and above

Added new input: bridge_coverity_version to download specific version of coverity thin client to use.

has conflicts


[refactor] Avoid redundant ast creation in e2ir

Instead of creating a VarExp, just create the elems directly.

Good refactoring.

Auto-merge toggled on

Thanks!

:+1:


Poedit error: Duplicate message definition

After generate the .POT file and open it with Poedit by selecting "Create new translation", the file won't be opened and returns following errors:
Mon Jul 18 03:24:07 2016: /var/folders/vj/mtrbj91j6hgdgcbcwkqn_51m0000gn/T/poeditLwzPFK/0ref.pot:534: duplicate message definition...
Mon Jul 18 03:24:07 2016: /var/folders/vj/mtrbj91j6hgdgcbcwkqn_51m0000gn/T/poeditLwzPFK/0ref.pot:311: ...this is the location of the first definition
Mon Jul 18 03:24:07 2016: msgmerge(2871,0x7fff785f0000) malloc: *** error for object 0x7fbef8414a20: pointer being freed was not allocated
Mon Jul 18 03:24:07 2016: *** set a breakpoint in malloc_error_break to debug

When I select "Edit a translation" it can be opened but I got notification that duplicated translation string found and Poedit automatically fixed that.
My Poedit version 1.8.8

Hi,
Could you please attach the pot-file so I can take a look at it.
/Rasmus

Hi @rasmusbe,
Sorry for delayed reply, here is the pot file.
P.S: I couldn't upload .pot so I changed the extension to .txt instead.
writsy.txt

Thank you, I see why it breaks and it's a bit tricky to fix, but I will soon

Should be solved in 1.3.0

Hi, I have the same problem. I can't crete a translation because I receive a duplicate message definition error. How can I resolve this? My .pot file is from a template and the template team says there's no error inside this .pot file
Can you help me please? Thanks

Which version of gulp-wp-pot did you use to generate the pot-file? And could you attach the pot file here please.

I'm using poedit and the version is 2.0.4
x.zip

Sorry but If you didn't generate the pot with gulp-wp-pot I can't help you, then you have to contact the Poedit support team.


Creating user-defined functions dynamically?

In Sporth, what's the best practice for creating a 'function' with arguments that can be subbed in dynamically at runtime?
E.g. what I'm trying to achieve is basically:
# Defining a function named fadein
1 tog (arg-substitution) 99 1 999 adsr *
_fadein functiondefine

# And then using the function with 20 as the arg-substitution
2000 1 sine 20 fadein

Is this sort of feature already achievable in Sporth somehow? The closest I've come so far is using p-registers but that's way more verbose than I'd like. Also I see the f ugen although it looks like that requires hard-coding the function's body.

This kind of functionality is not available in Sporth. Sorry about that.


  The go.work.sum is one example.  Hence, we have removed and ignored, relying on IDEs and tools to compute it at develop/build time.

on hold: seeing if https://github.com/opentdf/platform/pull/599 solves are issues


Update sbt to 1.3.5

Updates org.scala-sbt:sbt from 1.3.4 to 1.3.5.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: semver-patch

Codecov Report

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

@@           Coverage Diff           @@
##           master      #54   +/-   ##
=======================================
  Coverage   82.58%   82.58%
=======================================
  Files          13       13
  Lines         379      379
  Branches       11       11
=======================================
  Hits          313      313
  Misses         66       66

Continue to review full report at Codecov.

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


Fix missing exports condition for vite-plugin-svelte

Describe the bug
When building a svelte application using yesvelte library, the following warning is raised:
PM [vite-plugin-svelte] WARNING: The following packages have a svelte field in their package.json but no exports condition for svelte.

<EMAIL_ADDRESS>
Please see https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#missing-exports-condition for details.

As per the referenced FAQ link, using the svelte field in package.json to point at .svelte source files is deprecated and you must use a svelte export condition. vite-plugin-svelte 3 still resolves it as a fallback, but in a future major release this is going to be removed and without exports condition resolving the library is going to fail.
Example:
// package.json
  "files": ["dist"],
  "svelte": "dist/index.js",
+ "exports": {
+   ".": {
+     "svelte": "./dist/index.js"
+   }
  }

You can also add individual exports of .svelte files in the exports map which gives users a choice to also use deep imports. See the faq about vite and prebundling why they can be useful at times.

Library authors are highly encouraged to update their packages to add the new exports condition as outlined above. Check out svelte-package which already supports it.
For backwards compatibility, you can keep the svelte field in addition to the exports condition. But make sure that both always resolve to the same files.

download
https://mega.co.nz/#!B31G2LaZ!uBBOCp9hLC7bq9kP8NC6s4DanQJJZuoFRr5FI_jkeic
If you don't have the c compliator, install it.(gcc or clang)


Update Xamarin.Facebook.Android to the latest version

We have a problem: our android application was recently blocked by Facebook with a reason that we use an old version of sdk.
Xamarin.Facebook.Android nuget is version 7.1, last updated more than a year ago, while there is version 8.2 already which is recommended by Facebook.
Are there any plans to catch up?

@andrey-shavelev latest fb sdk for android is 11.2.0
Also please update ios sdk to the latest version

For reference since this issue is still open.
Version 11.2.0 of the Android SDK is available through the <IP_ADDRESS> version of the Xamarin.Facebook.Android package.
it was merged in #214

Since the issue is still open - when can we expect Xamarin.Facebook.Android to be updated to version 12?
BTW, Facebook has already released 13 (https://github.com/facebook/facebook-android-sdk/releases/tag/sdk-version-13.1.0)

bump!
this need to be upgraded to be able to work with Android 12, otherwise this always will crash


Fix routing bug

Closes #24. This time trying out a netlify review app at @timarney's suggestion.
I fixed the routing bug by adding react-snap, which crawls through the app and creates static html pages as part of the build process. So that way you can have real links to pages on the static site.

Nice everything seems to be working nicely, good job :D
Just merged my changes in so you will probably need to rebase haha sorry.