feat: Refactored template, workspace settings, and profile pages

Description
This PR includes template, workspace settings, and profile pages
Fixes #19387
Fixes #19388
Fixes #19389
Media
https://user-images.githubusercontent.com/87797149/232612403-f9a0e962-7182-438a-8514-733cc59573ba.mov
Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

Manual
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/build-deploy-preview env=release

Also left [6] comments about the dp itself in your DM
https://theappsmith.slack.com/archives/D03K37HRU00/p1681768452166859


Fix infinite loop in MavenPomDownloader

What's changed?
Fix an infinite loop in MavenPomDownloader
What's your motivation?
Since this MR https://github.com/openrewrite/rewrite/pull/4472 , there is an infinite loop in MavenPomDownloader.download() when root pom have a parent.
Checklist

[ ] I've added unit tests to cover both positive and negative cases
[ ] I've read and applied the recipe conventions and best practices
[ ] I've used the IntelliJ IDEA auto-formatter on affected files

I think this fix would break projects that use a revision property and where the module structure is more than one level deep. I assume @marcel-gepardec has an example for that. The code wants to traverse to the root-pom, but apparently the condition for reaching the root is not good enough.

Thanks for the quick work here @philippe-granet ; I'll give @marcel-gepardec a little time to chime in, but ideally we work this in before tomorrow's release.

Thanks for the quick work here @philippe-granet ; I'll give @marcel-gepardec a little time to chime in, but ideally we work this in before tomorrow's release.

I also ran into this infinite loop with some local testing :)
imo, if we don't land on a root fix in time for that release, then #4472 should just be reverted for the release until a thorough fix is available.
(but I am grateful for the intended enhancement; I've tried to fix the ${revision} pattern in the past and it keeps coming back to haunt me)

@marcel-gepardec wont be available today before noon CET. Its unlikely that he can provide a fix today especially since the code is really complicated imho. In my opinion the code would need a more in depth rework.

Thanks for that context @ErhardSiegl ; then I propose we merge the change already, given that the same tests still pass, it fixes an issue for two folks (at least), and might only break when the module structure is deeper. That last case can then be picked up separately with a matching test.
A test for the scenarios @philippe-granet & @nmck257 encountered would also be helpful to avoid regressions at that point, but let's not hold up this fix and the release for that. Thanks all!

well this is odd -- one of the tests from that PR (shouldNotThrowExceptionForModulesInModulesWithRightProperty) is already failing on my machine at main, but I see CI is happy.
I'm on windows; Java URI syntax isn't OS-specific?...
org.opentest4j.AssertionFailedError: Unexpected exception thrown: java.lang.IllegalArgumentException: Illegal character in path at index 52: file:///C:/Users/me/.m2/repository/test/test2/${test}/test2-${test}.pom

	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:152)
	at org.junit.jupiter.api.AssertDoesNotThrow.createAssertionFailedError(AssertDoesNotThrow.java:84)
	at org.junit.jupiter.api.AssertDoesNotThrow.assertDoesNotThrow(AssertDoesNotThrow.java:75)
	at org.junit.jupiter.api.AssertDoesNotThrow.assertDoesNotThrow(AssertDoesNotThrow.java:58)
	at org.junit.jupiter.api.Assertions.assertDoesNotThrow(Assertions.java:3259)
	at org.openrewrite.maven.internal.MavenPomDownloaderTest$WithNativeHttpURLConnectionAndTLS.shouldNotThrowExceptionForModulesInModulesWithRightProperty(MavenPomDownloaderTest.java:675)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
Caused by: java.lang.IllegalArgumentException: Illegal character in path at index 52: file:///C:/Users/me/.m2/repository/test/test2/${test}/test2-${test}.pom
	at java.base/java.net.URI.create(URI.java:906)
	at org.openrewrite.maven.internal.MavenPomDownloader.download(MavenPomDownloader.java:560)
	at org.openrewrite.maven.internal.MavenPomDownloaderTest$WithNativeHttpURLConnectionAndTLS.lambda$shouldNotThrowExceptionForModulesInModulesWithRightProperty$5(MavenPomDownloaderTest.java:675)
	at org.junit.jupiter.api.AssertDoesNotThrow.assertDoesNotThrow(AssertDoesNotThrow.java:71)
	... 7 more
Caused by: java.net.URISyntaxException: Illegal character in path at index 52: file:///C:/Users/me/.m2/repository/test/test2/${test}/test2-${test}.pom
	at java.base/java.net.URI$Parser.fail(URI.java:2974)
	at java.base/java.net.URI$Parser.checkChars(URI.java:3145)
	at java.base/java.net.URI$Parser.parseHierarchical(URI.java:3227)
	at java.base/java.net.URI$Parser.parse(URI.java:3175)
	at java.base/java.net.URI.<init>(URI.java:623)
	at java.base/java.net.URI.create(URI.java:904)
	... 10 more

@nmck257 @timtebeek I fixed the issue with the infinity loop and the failing test in #4609


Remove k mem pool defrag

Remove references to k_mem_pool_defrag and any related bits associated
with mem_pool defrag that don't make sense anymore.

This should address #1597


Check if we can use new GSON version

Check if we can use new GSON version. Maybe by doing so the code in the command encoders can be simplified.

Consider overhauling all dependencies, not just GSON, such as
groovyVersion     = 2.4.11
gsonVersion       = 2.8.1
guavaVersion      = 22.0
jacocoVersion     = 0.7.9
rxjavaVersion     = 1.3.0
slf4jVersion      = 1.7.25
springBootVersion = 1.5.4.RELEASE

Running the dependencyUpdatesgradle task at the root level ( gradle :depUp) yields
The following dependencies have later milestone versions:
 - javax.enterprise:cdi-api [1.1 -> 2.0]
 - cglib:cglib-nodep [3.2.4 -> 3.2.5]
 - org.kt3k.gradle.plugin:coveralls-gradle-plugin [2.6.3 -> 2.8.1]
 - org.apache.deltaspike.core:deltaspike-core-api [1.7.1 -> 1.8.0]
 - org.apache.deltaspike.core:deltaspike-core-impl [1.7.1 -> 1.8.0]
 - com.avast.gradle:docker-compose-gradle-plugin [0.3.13 -> 0.4.3]
 - com.jfrog.bintray.gradle:gradle-bintray-plugin [1.7.1 -> 1.7.3]
 - org.ajoberstar:gradle-git [1.3.2 -> 1.7.2]
 - com.github.ben-manes:gradle-versions-plugin [0.13.0 -> 0.15.0]
 - com.google.code.gson:gson [2.7 -> 2.8.1]
 - com.google.guava:guava [19.0 -> 22.0]
 - com.hazelcast:hazelcast-client [3.6.2 -> 3.8.2]
 - org.hibernate:hibernate-validator [5.1.3.Final -> 5.4.1.Final]
 - javax:javaee-api [6.0 -> 7.0]
 - org.glassfish:javax.el [3.0.0 -> 3.0.1-b08]
 - javax.servlet:javax.servlet-api [3.1.0 -> 4.0.0-b07]
 - org.jmockit:jmockit [1.25 -> 1.32]
 - com.github.dblock:oshi-core [3.2 -> 3.4.0]
 - org.reflections:reflections [0.9.10 -> 0.9.11]
 - io.reactivex:rxjava [1.1.7 -> 1.3.0]
 - org.sonarsource.scanner.gradle:sonarqube-gradle-plugin [2.2 -> 2.5]
 - org.spockframework:spock-core [1.0-groovy-2.4 -> 1.1-groovy-2.4]
 - org.springframework.boot:spring-boot-starter-test [1.5.3.RELEASE -> 1.5.4.RELEASE]
 - org.springframework.boot:spring-boot-starter-web [1.5.3.RELEASE -> 1.5.4.RELEASE]
 - net.nemerosa:versioning [2.4.0 -> 2.6.0]

Thx for the Gradle  hint. I think we already have a general issue for that. Will link it

Working on.
Would try to update all dependencies.
@hendrikebbers is there a generell issue for that?

@TimRiemer yes we have https://github.com/canoo/dolphin-platform/pull/644

ah, you already found it :D


ErrorException: tempnam(): file created in the system's temporary directory

Getting "ErrorException: tempnam(): file created in the system's temporary directory" after dropping file in filepond area. I am on windows 10 and using Laragon server with php 7.4.1.
package FilepondController.php line 62
$tempPath = '/tmp';
$filePath = tempnam($tempPath, 'nova-filepond-');

It will be best if you use sys_get_temp_dir() function instead of hardcoded '/tmp' folder.
Or using '@' symbol to ignore warning in windows 10. Thanks
Please also see same issue here Sopamo/laravel-filepond#5
Please help. Thanks

Can you try modifying your local file with your suggestion, and see if it does fix this problem? If it does I can quickly submit a PR

This is an issue for windows machine for php /tmp path. Developing docker container does not have this issue. Thanks


[QUERY] How can I download a blob into a Stream passing some BlobRequestOptions's properties?

Query/Question
Hi. We've been using the Microsoft.WindowsAzure.Storage.Blob version <IP_ADDRESS> and used a BlobRequestOptions to set some properties before downloading and existing blob to a stream, as follows:
public void Test()
{
  var blob = StorageFileManager.GetBlobReference(
  file.StorageArquivo.Storage.CaminhoFisico,  //connectionString
  file.StorageArquivo.Container,                      //containerName
  blobName                                                     //blobName
  );

  blob.FetchAttributes();
  var blobSize = blob.Properties.Length;
  long blockSize = (5 * 1024 * 1024); // 5MB
  blockSize = Math.Min(blobSize, blockSize);

  var blobRequestOptions = new BlobRequestOptions
  {
    RetryPolicy = new ExponentialRetry(TimeSpan.FromSeconds(5), 3),
    MaximumExecutionTime = TimeSpan.FromMinutes(60),
    ServerTimeout = TimeSpan.FromMinutes(60)
  };

  long currentPointer = 0;
  long bytesRemaining = blobSize;
  int partNumber = 0;
  do
  {
      var bytesToFetch = Math.Min(blockSize, bytesRemaining);
      using (MemoryStream ms = new MemoryStream())
      {
          blob.DownloadRangeToStream(ms, currentPointer, bytesToFetch, null, blobRequestOptions);
          ms.Position = 0;

          /*Do some more things with the stream*/
      }
    }
    while (bytesRemaining > 0);
}

I know that in the new library we have the DownloadTo method, and it receives an BlobRequestConditions, but it does not contain any of the properties I'm looking for.

I wanted to know where can I find this properties to configure it before making the download.
Environment:
- .Net Version: Full Framework 4.8
- Original version: Microsoft.WindowsAzure.Storage.Blob <IP_ADDRESS>
- New version: Azure.Storage.Blobs v12.8.0
- IDE and version : Visual Studio 16.8.5

Hi,
You can use the overload of DownloadTo() that allows users to specify non-default StorageTransferOptions.

Hi @amnguye, thanks for the feedback! I've changed the title of my question, think it makes more sense now.
Sorry about that :).
So, I've read the "retry policy" and the "timeout" examples. As I can tell:

Retry Policy: I should set it when creating a new BlobServiceClient (as bellow)

Timeout: I should create a custom policy and set the timeout in the BlobClientOptions before passing it to create a container (as bellow)

With that in mind, I still have a question. I'm currently updating an old library to the new one. In the old, we were using the CloudBlob class and its method DownloadRangeToStream, where we configured the BlobRequestOptions. In the new, I'm using a BlobClient class and trying to use the DownloadTo method to download it to a stream, as we used before (see bellow).

With that in mind, my questions are:

There is a way to set the RetryPolicy in the container, since I'm not using a BlobServiceClient? Should I create a custom policy as it is done in the timeout example?

The only way to configure those two options is in the container's creation? Can't I do it only when making the download to a stream?

Thanks again for all the help :)

Please re-open if you have further questions.


Concept Idea for canvas size control

I was thinking we have the stand logo functions which can be found on https://docs.python.org/2/library/turtle.html
I was thinking of adding a custom one to this parser like canvassize width height to let you dynamically control the p5 canvas at loading time
What do you all think about this idea @

Something like
canvassize 500 500

in code we can do something like the setxy function

Sure! Is this part of the original logo? I like the idea of being an "emulator" as close as possible to Logo from the 70s/80s.

I couldn't find any reference to the canvas size in the logo reference:
https://people.eecs.berkeley.edu/~bh/docs/usermanual.pdf
So, I think they think in an infinite canvas where size doesnt matters, so, I guess the best fit could be a canvas adapted to the screen, where 0,0 its in the center of the canvas, and because we can move now with the mouse, it would give the impression of an infinite canvas... What do you think?

Maybe I should close this now that the size is "auto-generated" and there's an auto-fit button?

Yes this can get closed now
Sent from my iPhone
On Nov 13, 2018, at 6:34 AM, Daniel Shiffman <notifications<EMAIL_ADDRESS>wrote:
Maybe I should close this now that the size is "auto-generated" and there's an auto-fit button?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/CodingTrain/Logo/issues/32#issuecomment-438287274, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABZ1YGKQk9NJbeKgytUDajkM1tAWLYUbks5uutiLgaJpZM4YXdC-.


The examples in docs/get_started/ are outdated

The simple.mlir file was split in 32b3101ba74d0283b49304d1e8062ba9a2db2585, there is no longer a simple.mlir file.

duplicate of https://github.com/google/iree/issues/4667


Protective Order (Alaska) - Guided Assist Error

There are 2 paths to this defect.
First path: 1. "If you or someone . . . . " Answer "No."  2. "What is your situation?"  Answer  "I or my child(ren) have . . . "  3.  "Is your protective order . . . " Answer "Alaska."   4. "Do you have a . . . "  Answer "Long-term"   5. "Has the other side violated the order?"  Answer "No."   6.  Broken question.
Second path:  1. "If you or someone . . . . " Answer "No."  2. "What is your situation?"  Answer  "I or my child(ren) have . . . "  3.  "Is your protective order . . . " Answer "Alaska."   4. "Do you have a . . . "  Answer "Short-term"   5. "Did you also request a long-term . . . . "  Answer "No."   6.  "Has the other side violated the order?"  Answer "No."   7.  Broken question.

Fixed.


ci: update to latest osmosis

Codecov Report

Merging #647 (2c2c80b) into main (141b05b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #647   +/-   ##
=======================================
  Coverage   10.35%   10.35%
=======================================
  Files          57       57
  Lines        8101     8101
=======================================
  Hits          839      839
  Misses       7200     7200
  Partials       62       62

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 141b05b...2c2c80b. Read the comment docs.


Completion script for aws-vault

Fixes #20

I have taken the completion script in https://github.com/reegnz/aws-vault-zsh-plugin and updated it to work with the latest aws-vault version.

Thanks @reegnz - this is great. One question - do you know how we can unify the current completion (https://github.com/blimmer/zsh-aws-vault/blob/3d5ad1ebfc12f91b157f7bf2a38027fc4f48bb76/_av_completion) with your improved completion script from this PR?
I'm not super-familiar with how completions work in zsh. Basically, what I'd love to see is how we could update the plugin file (https://github.com/blimmer/zsh-aws-vault/blob/main/zsh-aws-vault.plugin.zsh) to delegate to your completions.
I was toying around with this locally and I think we want something like this:
function avl() {
  aws-vault login $@
}
compdef avl=aws-vault

That's close, but we really want to alias the completions of avl to aws-vault login and I'm not sure how to do that. @reegnz, do you know how to do this
Thanks again for taking time to contribute!

I put it in a script called _aws_vault as that's a common way of shipping completion scripts for zsh.
The great thing about zsh is it's smart about aliases, so if you do alias avl=aws-vault login completion will still work, without having to compdef.
So I'll have a look in replacing the avl function with a plain alias.

So avl, avll, ave can be replaced with a plain alias. avsh and avli are a bit harder nut to crack, I think for those it makes more sense to keep a separate completion file (as they're not possible to replace with just an alias, they're actually new commands built on top of aws-vault).

@blimmer I did some changes, now all aliases (except for avsh and avli) are using the _aws_vault completion.
I think given that avsh and avli are new commands, it's much harder for them to adopt the completion of _aws_vault, so I didn't take a run on doing those.

@reegnz - this is great. Thank you so much for taking the time to merge in your work to this repo!


if folder exists, remove

Description

I had a similar error in reco https://github.com/microsoft/recommenders/pull/985, right now your pipeline is not failing because you have continueOnError skip the error when the folder cv does not exist, however, the current pipeline won't detect if there is a different error. This change just check whether there is a folder called cv and then erase it
Related Issues

Checklist:

[ ] I have followed the contribution guidelines and code style for this project.
[ ] I have added tests covering my contributions.
[ ] I have updated the documentation accordingly.
[ ] This PR is being made to staging and not master

@miguelgfierro Hi Miguel! Do you want us to merge this for you?

yes, if you agree with the changes, please merge.

@Jiata Can you sign off please if this looks good (lgtm)

yeah looks good. i'm going to merge this.


Modified Lua version

Thanks catwell for simple.lua . Closed the other pull request and started this.

Now I think it's done

I've decided to go with https://github.com/benhoyt/countwords/pull/69 due to the conflicts / rebasing issues in this PR. The code looks quite similar, though. Thanks for your contribution, anyway!


Add ignore for ruff C409 rule

It's true that there's a (big!) cost difference between the two, with the rule pushing us towards the slower alternative:
$ python -m timeit 'tuple(i for i in range(1))'
1000000 loops, best of 5: 340 nsec per loop
$ python -m timeit 'tuple([i for i in range(1)])'
2000000 loops, best of 5: 133 nsec per loop

At the same time, I still think the rule has a point: For most cost-insensitive spots, I'd prefer to go along with the rule, especially since it covers more than just comprehensions.

Replaced by #51.

It's true that there's a (big!) cost difference between the two, with the rule pushing us towards the slower alternative:
$ python -m timeit 'tuple(i for i in range(1))'
1000000 loops, best of 5: 340 nsec per loop
$ python -m timeit 'tuple([i for i in range(1)])'
2000000 loops, best of 5: 133 nsec per loop

At the same time, I still think the rule has a point: For most cost-insensitive spots, I'd prefer to go along with the rule, especially since it covers more than just comprehensions.

That's more or less what I'm seeing too performance-wise (about 3 times slower). From what I can tell, that difference gets smaller for larger lists.
$ python -m timeit 'tuple(i for i in range(1))'
2000000 loops, best of 5: 186 nsec per loop
$ python -m timeit 'tuple([i for i in range(1)])'
5000000 loops, best of 5: 67.2 nsec per loop
$ python -m timeit 'tuple(i for i in range(1000))'
20000 loops, best of 5: 17.8 usec per loop
python -m timeit 'tuple([i for i in range(1000)])'
20000 loops, best of 5: 11.4 usec per loop

I'll ago ahead and modify the other PRs too. Probably should have asked before.. sorry for the noise! :grin:


How to change number of characters?

Hello.
I have try jquery.terminal today. It's cool. But I ran into a problem.
I resize the term to 426px width, but only got 35 characters per line. It's the half of one line.
I found that the number of characters relative to num_rows.
I want to fill up the whole line. Is there any way or trick to solve this problem?
sorry for my English.

You should have about 55 characters. Do you have a demo where this is happening, where I can try things out? Maybe jsfiddle?

I copy my code to jsfiddle. http://jsfiddle.net/sumy/f7qdrp1y/

You need to set the same font on .terminal because it's used for character calculation:
.terminal {
  font-family: fantasquesansmono;
}

You will have 67 characters http://jsfiddle.net/f7qdrp1y/3/

Thanks a lot.
I add the css, and clear browser cache, then it works.

Thanks for your suggestion. I try this method in my blog, but it didn't work. /(ㄒoㄒ)/~~
You notice that the problem is cause by font-family,  I want to say what I understand.
In the function char_size() function, a fake terminal created calcualte the dimention of   representing one character. the normal <span> width is 6, but my is 12.
font-family from jquery.terminal.css is
.terminal, .cmd {
    font-family: FreeMono, monospace;
}

I don't have FreeMono and monospace in my computer. so a default font which have a big   would be selected.
My solution is using a higher priority selector to make my font-family be selected.
.terminal span{
    font-family: fantasquesansmono !important;
}

monospace is meta name same as sans or sans-serif it will pick the font of that type.


Connection to the server

any documentation / examples about the connection to XMPP server

Hello @aitbaali, there is documentation on SO, or other places. Github issues that doesn't relate to code aren't allowed here, so I'm closing this.

Hello Paul,
I'll do that,
and what about this issue "
https://github.com/processone/xmpp-messenger-ios/issues/40"
2016-05-10 0:34 GMT+01:00 Paul Lemaire<EMAIL_ADDRESS>
Hello @aitbaali https://github.com/aitbaali, there is documentation on
SO, or other places. Github issues that doesn't relate to code aren't
allowed here, so I'm closing this.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/processone/xmpp-messenger-ios/issues/48#issuecomment-218020351

--
A.Abderrahmane(+212) 0642 246 504


Normalize initial Composer Electron browser window size

Composer Electron should not start maximized, and should instead start up as a window that takes up a large portion of the screen.

https://github.com/microsoft/BotFramework-Composer/pull/2769


Fix Typo in learner.cc

Codecov Report

Merging #3902 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #3902      +/-   ##
============================================
- Coverage     56.35%   56.34%   -0.01%
  Complexity      203      203
============================================
  Files           185      185
  Lines         14679    14679
  Branches        495      495
============================================
- Hits           8272     8271       -1
- Misses         6169     6170       +1
  Partials        238      238

Impacted Files
Coverage Δ
Complexity Δ

src/learner.cc
27.17% <0%> (ø)
0 <0> (ø)
:arrow_down:

src/objective/multiclass_obj.cu
92.63% <0%> (-1.06%)
0% <0%> (ø)

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 fe999bf...bfc855f. Read the comment docs.

@theycallhimavi Thanks.


error message about completion.bash in Kitty (fixable by adding set +o posix)

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
I frankly don't know if this is an fzf issue, a Kitty one or something unique to me (though I've checked all the obvious shell problems).
This was the error, when launching new windows in Kitty shell (but not in iTerm2 or PhpStorm terminal):

-bash: /opt/homebrew/opt/fzf/shell/completion.bash: line 279: syntax error near unexpected token <' -bash: /opt/homebrew/opt/fzf/shell/completion.bash: line 279:       __fzf_orig_completion < <(complete -p "$orig_cmd" 2> /dev/null)'
-bash: `fzf-file-widget': not a valid identifier

I tried with both the old (Apple) and newest (homebrew) versions of bash.
I've read about similar problems with scripts running via sh when they should be using bash, but as you can see it's a bash error message.
I added set +o posix line 14 as per this suggestion https://stackoverflow.com/a/63397594/763419 and that has worked…
Any ideas? Thank you.
macOS Ventura 13.6, Kitty 0.31, fzf 0.44.1 (brew), bash (currently) 3.2.57

I also use kitty but I don't have the problem and I have no idea what's going on. You might want to ask the maintainers of kitty, after replacing the fzf script with a simple line of code that can reproduce the problem. Something like:
cat <(echo)

Youa re probably running the outdated bash provided by apple: https://github.com/kovidgoyal/kitty/issues/6926


Issue with use of BINOMIAL_LOG_LIKELIHOOD loss

Hi
I have moved from 0.2.3 to main (latest) and have now found an issue with my code:
[INFO 2023-01-26T12:20:50.093951+00:00 gradient_boosted_trees.cc:452] Default loss set to BINOMIAL_LOG_LIKELIHOOD
[FATAL 2023-01-26T12:20:50.0948034+00:00 YggdrasilWrapper.cpp:177] INVALID_ARGUMENT: No class registered with key "BINOMIAL_LOG_LIKELIHOOD" in the class pool "class yggdrasil_decision_forests::model::gradient_boosted_trees::AbstractLoss". Registered classes are "". Add as a dependency the cc_library rule that defines this class in your BUILD file
I can see that the loss functionality has changed quite a bit since 0.2.3 but I'm not sure what I'm missing to get this to work again.
Thanks!

Sorry for coming back to this again but I'm still having this issue.
So far I've tried adding "//yggdrasil_decision_forests/learner/gradient_boosted_trees/loss:loss_imp_binomial" to the cc_library_ydf at \yggdrasil-decision-forests\yggdrasil_decision_forests\learner\gradient_boosted_trees\BUILD and building the lib again but this doesn't fix the issue. Perhaps I should add it somewhere else?
I have also tried adding:
CreateLoss(proto::Loss::BINOMIAL_LOG_LIKELIHOOD, proto::Task::CLASSIFICATION, myFeatureCol, config)
to my code (although probably this is not needed, as I'm not using early stopping). This doesn't help either...

Hi JoseAF,
Sorry to hear about this.
As you noted, the part of the code related to losses has changed. Losses are now using a registration mechanism similarly as for the learner, models and dataset formats. In most existing code, I would expect no changes in the user code. But I could imagine corner cases where you would have to register those losses manually (that is to link :loss_imp_binomial, or one of its dependents).
Following is the path used to inject :loss_imp_binomial in most user code. It could be useful to identify if you need to inject the loss manually:
The loss implementation "BINOMIAL_LOG_LIKELIHOOD" is defined in the file yggdrasil_decision_forests/learner/gradient_boosted_trees/loss/loss_imp_binary_focal.h and compiled in the rule :loss_imp_binomial of the file https://github.com/google/yggdrasil-decision-forests/blob/main/yggdrasil_decision_forests/learner/gradient_boosted_trees/loss/BUILD. Your error suggest that none of the losses are registered. For our tools, those losses are registered in the build rule :all_implementations of in file https://github.com/google/yggdrasil-decision-forests/blob/main/yggdrasil_decision_forests/learner/gradient_boosted_trees/loss/BUILD. ;all_implementations is used in yggdrasil_decision_forests/learner/gradient_boosted_trees, which is itself used in the build rule :all_learners or https://github.com/google/yggdrasil-decision-forests/blob/main/yggdrasil_decision_forests/learner/BUILD.
If this does not solve this issue, please try the bazel query tool. Notably, identify the path from your binary to :loss_imp_binomial. If such path does not exist, you have to register :loss_imp_binomial (or one of its decedents) manually.

Thanks for the reply Mathieu - I'll work on the above and let you know...

Hi guys
I haven't got this working yet. It seems all the dependencies are correct in the BUILD files as far as I can see. I have built a lib with tensorflow in case I was missing something from there but the error persists. I have also added a CreateLoss call and a set_loss call to try to trigger registering of the BINOMIAL_LOG_LIKELIHOOD loss but no luck. I'll keep trying, but any ideas are welcomed here...
Just for your information, in case it helps, the code I have works perfectly well with 0.2.3 and I haven't made changes to the BUILD files that I get from github.
Thanks


Stop health checks running until the underlying resource enters the running state.

Description
Adds a service to the container which replaces the predicate on HealthCheckPublisherOptions and only enables checks when a resource associated with it is in a Running state (once enabled they stay enabled). It also reduces the period to 5 seconds.
Checklist

Is this feature complete?

[ ] Yes. Ready to ship.
[x] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[x] No (existing tests cover this, if it isn't working the tests will fail)

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[ ] No

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


Removing uglify

Your documentation implies that I can edit the flow for just the js, without affecting the css.  I found that the following didn't work:
    useminPrepare: {
		html: '<%= buildDir %>/index.html',
		options: {
			dest: '<%= buildDir %>',
			flow: {
				html: {
					steps: {
						js: ['concat']
					}
				}
			}
		}
	}

First, I had to add post: {}, which is ridiculous.  How are users supposed to figure this out?
Then, I was getting "No cssmin targets found", and fixed that by adding back the steps: { css: ['concat', 'cssmin'].  And so the above turns into:
    useminPrepare: {
		html: '<%= buildDir %>/index.html',
		options: {
			dest: '<%= buildDir %>',
			flow: {
				html: {
					steps: {
						js: ['concat'],
						css: ['concat', 'cssmin']
					},
					post: {}
				}
			}
		}
	}

Am I missing something?

Closing as duplicate of #249.

OK reopen this with docs label. Need to document this issue.


color problems/questions

Describe your problem here.
is there any way how to set the color of an object with setProperty
I looked at a example but it didn't work
how to set an object color to white with doColorTween()
what does 0x(hex color) mean?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No (other than the accuracy 'ratingStuff'), just modded in the "example_mods" folder

You can tween an object's color by  doTweenColor(tag:String, vars:String, targetColor:String, duration:Float, ease:String)
else idk

setProperty(getProperty('[the object].color'), '[hex color value here]')

change [the object]  to the object name
change [hex color value here] to the color you want (needs to be hex color code, all caps)

Why is getProperty there ????

And by that you're able to use that property anywhere on that file now
Em sáb, 17 de set de 2022 20:17, Bernardo Gameplays E mais 2021 <
@.***> escreveu:

To get the color property out of the object
Em sáb, 17 de set de 2022 19:05, frantastic24 @.***>
escreveu:

Why is getProperty there ????
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/10559#issuecomment-1250147340,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYFVJYAG7QNOZGAUGRK7EI3V6Y6BPANCNFSM6AAAAAAQOUGCVE
.
You are receiving this because you commented.Message ID:
@.***>

that is not how it works

It works that way for me atleast
Em sáb, 17 de set de 2022 20:19, frantastic24 @.***>
escreveu:

that is not how it works
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/10559#issuecomment-1250154459,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYFVJYACL4X2AA6KBCMCAJLV6ZGXRANCNFSM6AAAAAAQOUGCVE
.
You are receiving this because you commented.Message ID:
@.***>

You do not need getProperty

Ok, thanks.
Em sáb, 17 de set de 2022 20:23, frantastic24 @.***>
escreveu:

You do not need getProperty
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/10559#issuecomment-1250154855,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYFVJYBWDI5VMP2UJXLH56LV6ZHIFANCNFSM6AAAAAAQOUGCVE
.
You are receiving this because you commented.Message ID:
@.***>

To get the color property out of the object
Em sáb, 17 de set de 2022 19:05, frantastic24 @.***>
escreveu:

Why is getProperty there ????
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/10559#issuecomment-1250147340,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYFVJYAG7QNOZGAUGRK7EI3V6Y6BPANCNFSM6AAAAAAQOUGCVE
.
You are receiving this because you commented.Message ID:
@.***>


[Windows - Bazel] ERROR undeclared inclusion(s) in rule '//tensorflow/core/kernels:fake_quant_ops_gpu'

Windows 10
CUDA 8.0
cuDNN 5.1
TensorFlow at e8a34420ce06c7f99049b4e4fec4308aff8dd058
Bazel at e565230
Msys2
During build Bazel failed with
ERROR: C:/users/adriano/documents/tensorflow/tensorflow/core/kernels/BUILD:2809:1: undeclared inclusion(s) in rule '//tensorflow/core/kernels:fake_quant_ops_gpu':
this rule is missing dependency declarations for the following files included by 'tensorflow/core/kernels/fake_quant_ops_gpu.cu.cc':
  'C:/windows/temp/tmpxft_0000426c_00000000-8_fake_quant_ops_gpu.cu.cudafe1.stub.c'
  'C:/windows/temp/tmpxft_0000426c_00000000-6_fake_quant_ops_gpu.cu.fatbin.c'
c:\tmp\_bazel_adriano\piryq3n9\execroot\tensorflow\external\eigen_archive\eigen\src/Core/products/Parallelizer.h(20): warning: variable "m_maxThreads" was set but never used
c:\tmp\_bazel_adriano\piryq3n9\execroot\tensorflow\external\eigen_archive\eigen\src/Core/ArrayWrapper.h(93): warning: __declspec attributes ignored
c:\tmp\_bazel_adriano\piryq3n9\execroot\tensorflow\external\eigen_archive\unsupported\eigen\cxx11\src/ThreadPool/SimpleThreadPool.h(139): warning: "constexpr" is ignored here in Microsoft mode
c:\tmp\_bazel_adriano\piryq3n9\execroot\tensorflow\external\eigen_archive\unsupported\eigen\cxx11\src/ThreadPool/NonBlockingThreadPool.h(151): warning: "constexpr" is ignored here in Microsoft mode
c:\tmp\_bazel_adriano\piryq3n9\execroot\tensorflow\external\eigen_archive\eigen\src/Core/products/Parallelizer.h(20): warning: variable "m_maxThreads" was set but never used
c:\tmp\_bazel_adriano\piryq3n9\execroot\tensorflow\external\eigen_archive\eigen\src/Core/ArrayWrapper.h(93): warning: __declspec attributes ignored
c:\tmp\_bazel_adriano\piryq3n9\execroot\tensorflow\external\eigen_archive\unsupported\eigen\cxx11\src/ThreadPool/SimpleThreadPool.h(139): warning: "constexpr" is ignored here in Microsoft mode
c:\tmp\_bazel_adriano\piryq3n9\execroot\tensorflow\external\eigen_archive\unsupported\eigen\cxx11\src/ThreadPool/NonBlockingThreadPool.h(151): warning: "constexpr" is ignored here in Microsoft mode
nvcc warning : option '--relaxed-constexpr' has been deprecated and replaced by option '--expt-relaxed-constexpr'.
Target //tensorflow/tools/pip_package:build_pip_package failed to build
INFO: Elapsed time: 2078.109s, Critical Path: 104.55s
FAILED: Build did NOT complete successfully
+ exit 1

Steps to reproduce

configure TensorFlow accordingly without XLA
export BUILD_OPTS='--cpu=x64_windows_msvc --host_cpu=x64_windows_msvc --copt=/w --verbose_failures --experimental_ui'
./tensorflow/tools/ci_build/windows/gpu/pip/build_tf_windows.sh

@mrry Guy with the correct Windows CUDA versions is encountering what appears to be either a Bazel sandbox or strict deps issue.

Alas support for Bazel on Windows is still limited, and it will probably be easier to build with CMake.
@meteorcloudy Have you seen this error before?

I was expecting to leverage gcc but was warned by @meteorcloudy I might face issues if building TF.
I am aware bazel on Windows is experimental but decided to post the issue anyways to make the problem known for future improvements.

Hmm, did those errors appear when using GCC as the compiler on Windows? (Like @meteorcloudy, I'd be surprised if that worked, especially for the GPU build, but also for the various dependent libraries that might not have GCC-aware build configurations for Windows in our build files.)

As in https://github.com/bazelbuild/bazel/issues/2753#issuecomment-292818313

Someone else had a compiling issue related to fake_quant_ops_gpu
http://stackoverflow.com/questions/43482642/tesnsorflow-1-1-build-breaks-on-cuda-code

Thanks for pointing that out! @cwhipkey Are you aware of any issues with building fake_quant_ops_gpu (which I think you added, but feel free to redirect :)...)?

(Looks like this has also surfaced in #9296.)


codecs.tar.TarStream: ValueError: invalid literal for int() with base 8: b'00000057115\x00'

On Windows:
$ python -c 'import platform; print(platform.python_version()); print(platform.uname())'
3.10.10
uname_result(system='Windows', node='DESKTOP-89OPGF3', release='10', version='10.0.19044', machine='AMD64')

$ python -m pip show range-streams
Name: range-streams
Version: 1.2.11
Summary: Streaming via range requests.
Home-page: https://github.com/lmmx/range-streams
Author: Louis Maddox
Author-email<EMAIL_ADDRESS>License: MIT
Location: c:\users\pp\appdata\local\programs\python\python310\lib\site-packages
Requires: aiostream, httpx, python-ranges, pyzstd, tqdm
Required-by:

$ python -c 'from range_streams.codecs import TarStream; tar_stream = TarStream(url="https://cdn.watchguard.com/SoftwareCenter/Files/WSM/2_2_1/watchguard-dimension_2_2_1.ova")'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "C:\Users\pp\AppData\Local\Programs\Python\Python310\lib\site-packages\range_streams\codecs\tar\stream.py", line 114, in __init__
    self.check_header_recs()
  File "C:\Users\pp\AppData\Local\Programs\Python\Python310\lib\site-packages\range_streams\codecs\tar\stream.py", line 136, in check_header_recs
    file_size = self.read_file_size(start_pos_offset=scan_tell)
  File "C:\Users\pp\AppData\Local\Programs\Python\Python310\lib\site-packages\range_streams\codecs\tar\stream.py", line 187, in read_file_size
    file_size = int(file_size_b, 8)  # convert octal number from bitstring
ValueError: invalid literal for int() with base 8: b'00000057115\x00'

Same in WSL:
pp@DESKTOP-89OPGF3:~$ python3 -c 'import platform; print(platform.python_version()); print(platform.uname())'
3.10.6
uname_result(system='Linux', node='DESKTOP-89OPGF3', release='<IP_ADDRESS>-microsoft-standard-WSL2', version='#1 SMP Fri Jan 27 02:56:13 UTC 2023', machine='x86_64')
pp@DESKTOP-89OPGF3:~$ python3 -m pip show range-streams
Name: range-streams
Version: 1.2.11
Summary: Streaming via range requests.
Home-page: https://github.com/lmmx/range-streams
Author: Louis Maddox
Author-email<EMAIL_ADDRESS>License: MIT
Location: /home/pp/.local/lib/python3.10/site-packages
Requires: aiostream, httpx, python-ranges, pyzstd, tqdm
Required-by:
pp@DESKTOP-89OPGF3:~$ python3 -c 'from range_streams.codecs import TarStream; tar_stream = TarStream(url="https://cdn.watchguard.com/SoftwareCenter/Files/WSM/2_2_1/watchguard-dimension_2_2_1.ova")'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/pp/.local/lib/python3.10/site-packages/range_streams/codecs/tar/stream.py", line 114, in __init__
    self.check_header_recs()
  File "/home/pp/.local/lib/python3.10/site-packages/range_streams/codecs/tar/stream.py", line 136, in check_header_recs
    file_size = self.read_file_size(start_pos_offset=scan_tell)
  File "/home/pp/.local/lib/python3.10/site-packages/range_streams/codecs/tar/stream.py", line 187, in read_file_size
    file_size = int(file_size_b, 8)  # convert octal number from bitstring
ValueError: invalid literal for int() with base 8: b'00000057115\x00'

Good catch, yes this reproduces
>>> from range_streams.codecs import TarStream
>>> url = "https://cdn.watchguard.com/SoftwareCenter/Files/WSM/2_2_1/watchguard-dimension_2_2_1.ova"
>>> tar_stream = TarStream(url=url)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/louis/dev/range-streams/src/range_streams/codecs/tar/stream.py", line 114, in __init__
    self.check_header_recs()
  File "/home/louis/dev/range-streams/src/range_streams/codecs/tar/stream.py", line 136, in check_header_recs
    file_size = self.read_file_size(start_pos_offset=scan_tell)
  File "/home/louis/dev/range-streams/src/range_streams/codecs/tar/stream.py", line 187, in read_file_size
    file_size = int(file_size_b, 8)  # convert octal number from bitstring
ValueError: invalid literal for int() with base 8: b'00000057115\x00'
>>>

The test case is:
from range_streams.codecs import TarStream

data_dir_URL = "https://github.com/lmmx/range-streams/raw/master/data/"
EXAMPLE_TAR_URL = f"{data_dir_URL}data.tar"
tar_stream = TarStream(url=EXAMPLE_TAR_URL)

I also found another tarball which reproduces the error:
tar_stream = TarStream(url="https://storage.googleapis.com/kubernetes-release/gci-mounter/mounter.tar")

Breakpointing there in each case shows that in the working example case the file_size_b is:
>>> from range_streams.codecs import TarStream
>>> tar_stream = TarStream(url="https://github.com/lmmx/range-streams/raw/master/data/data.tar")
> /home/louis/dev/range-streams/src/range_streams/codecs/tar/stream.py(193)read_file_size()
-> return file_size
(Pdb) p file_size
5124
(Pdb) p file_size_b
b'00000012004 '
(Pdb) p len(file_size_b)
12

Whereas in both of the failure cases its value is 0
Your tarball:
>>> tar_stream = TarStream(url="https://cdn.watchguard.com/SoftwareCenter/Files/WSM/2_2_1/watchguard-dimension_2_2_1.ova")
> /home/louis/dev/range-streams/src/range_streams/codecs/tar/stream.py(193)read_file_size()
-> return file_size
(Pdb) p file_size_b
b'00000057115\x00'
(Pdb) p int(file_size_b.rstrip(b"\x00"), 8)
24141
(Pdb) p start_pos_offset
0

Kubernetes tarball:
>>> tar_stream = TarStream(url="https://storage.googleapis.com/kubernetes-release/gci-mounter/mounter.tar")
> /home/louis/dev/range-streams/src/range_streams/codecs/tar/stream.py(193)read_file_size()
-> return file_size
(Pdb) p file_size_b
b'00000000000\x00'
(Pdb) p file_size_b.rstrip(b"\x00")
b'00000000000'
(Pdb) p int(file_size_b.rstrip(b"\x00"), 8)
0
(Pdb) p start_pos_offset
0

Now the file size offset is 0, so the range is being taken at position 124 for 12 bytes.
(Pdb) p file_size_rng
Range[124, 136)

as confirmed from Wikipedia: https://en.wikipedia.org/wiki/Tar_(computing)#Header
Field offset 	Field size 	Field
0 	100 	File name
100 	8 	File mode (octal)
108 	8 	Owner's numeric user ID (octal)
116 	8 	Group's numeric user ID (octal)
124 	12 	File size in bytes (octal)
136 	12 	Last modification time in numeric Unix time format (octal)
148 	8 	Checksum for header record
156 	1 	Link indicator (file type)
157 	100 	Name of linked file

I'm a bit stumped by this! I began downloading the Kubernetes mounter.tar tarball and confirmed it is indeed not 0 bytes.

One possible answer (though not sure it is correct) is given in the tar man pages:

The size field is the size of the file in bytes; linked files are archived with this field specified as zero.

So a size field of value 0 may indicate "linked files" in the Kubernetes tarball case. This doesn't fit the description of the case you reported though.
I asked ChatGPT (no luck) and GPT-4 gave a futile suggestion. Then I asked it to try again and it replied that one option was:

Encodings: Ensure that the TarStream class handles different character encodings correctly when reading the file size and other header fields. For example, POSIX tar format uses the ASCII encoding, and the file size field is stored as a null-terminated octal string.

This matches the description, and implies rstripping the file_size_b is indeed the correct approach.
From that, we might expect the fix to be simply adding this rstrip(b"\x00") call to the line
https://github.com/lmmx/range-streams/blob/4d7385e0f71c57486e990af5ae9b94e8ed43c626/src/range_streams/codecs/tar/stream.py#L187
        try:
            file_size = int(file_size_b, 8)  # convert octal number from bitstring
        except ValueError:
            file_size = int(file_size_b.rstrip(b"\x00"), 8)  # may be null-terminated

and with that, it works :tada:
>>> from range_streams.codecs import TarStream
>>> url = "https://cdn.watchguard.com/SoftwareCenter/Files/WSM/2_2_1/watchguard-dimension_2_2_1.ova"
>>> tar_stream = TarStream(url=url)
>>> tar_stream
TarStream ⠶ [0, 24141), [25088, 25438), [26112, 128512), [129024, 960447488), [960448000, 960520704) @@ 'watchguard-dimension_2_2_1.ova' from cdn.watchguard.com

I'll ship the fix now, thanks for reporting.

:ship: 571433a

Released in v1.3.0

@lmmx:

        try:
            file_size = int(file_size_b, 8)  # convert octal number from bitstring
        except ValueError:
            file_size = int(file_size_b.rstrip(b"\x00"), 8)  # may be null-terminated

and with that, it works 🎉

Yeah, should work in most cases. You could do file_size = int(file_size_b.rstrip(b"\x00"), 8) directly, because rstrip will just return the string unchanged if it doesn't end with \x00, and int(..., 8) will always fail if it contains unknown digits for the given base (in this case 8), but that's a detail.
The only potential discrepancy I can see is that rstrip doesn't really handle "null-terminated" strings in the sense of the C language semantics (I guess this also applies to filenames, for example), which most likely the format authors had in mind. In C, a character string is represented just as a pointer to the beginning and its length is only determined by scanning it byte by byte from the start until the NUL byte ('\0') is found. But rstrip doesn't do that - it only strips the NUL bytes from the end of the string, but some zero bytes may still remain somewhere in the middle of the string, "protected" by other bytes from being reached when scanning from the end. So for such archives, you'd be probably getting different values than the rest of applications working with the tar format.

Oh! I didn't think of that. So I suppose file_size_b[:file_size_b.index(b"\x00")] would work then?

So I suppose file_size_b[:file_size_b.index(b"\x00")] would work then?

Yes. But bytes.index raises ValueError if the searched sequence is not found, so now it really makes sense to keep the try..except statement and apply the null termination only if int() on the original file_size_b fails.
And as I mentioned, this should be probably used for other fields (like file names) too.


Convert PLY to OBJ

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D455

Firmware Version
<IP_ADDRESS>

Operating System & Version
{Win 1

Platform
PC

SDK Version
<IP_ADDRESS>   }

Language
{C++ }

Segment
Robot

Issue Description
Hi,
Am actually creating an app using c++ and Qt. So far I have been able to display the camera stream on my application and apply the necessary post processing filters to get a better rendering and then generate my Cloud Point and export it. Now, I'm looking for a way to export the cloud point but in OBJ format or convert the PLY to OBJ directly, I tried to do that following this example: https://github.com/FZzzz/PlyToObj but I didn't get any result, in output I manage to generate an .obj but there is nothing inside, no mesh.
first of all,Is my approach correct ?
Can someone help me to solve it and understand it ?

Hi @Amayess  There is not a straightforward method to convert pointcloud data to a solid mesh in the RealSense SDK from within a self-created program.  It would likely be best to best to use specialized 3D libraries with a RealSense compatibility wrapper such as Open3D to do so.  One method of generating a mesh from a pointcloud is to compute a convex hull triangle mesh, like in the tutorial for Open3D at the link below.
http://www.open3d.org/docs/release/tutorial/geometry/pointcloud.html?highlight=convex hull#Convex-hull
If your project is able to make use of external conversion tools to convert  ply to mesh, you could use the open-source 3D tool MeshLab to convert a RealSense ply to a mesh and export it as an .obj file, as described in the tutorial at the link below.
https://www.andreasjakl.com/capturing-3d-point-cloud-intel-realsense-converting-mesh-meshlab/

Hi @MartyG-RealSense,
I am trying to do it with this solution [Picutre below], if it works i'll share it with the community !
Actually, i get a compilation error LNK1104: cannot open file ‘libboost_thread-vc142-mt-gd-x64-1_79.lib’ when i add #include <pcl/io/obj_io.h> to my project.,e ven if boost is used correctly in this project for other cases. I am trying to fix it !

I have not seen that partcular error with librealsense before, though a non-librealsense discussion at https://github.com/introlab/rtabmap/issues/678 has advice about dealing with it.
As your error refers to vc142 and boost 1_79, I assume that you would adapt the discussion's advised solution to link the following directory path to your project's config in Visual Studio:
C:\Libraries\boost_1_79_0\lib64-msvc-14.2

Beside the fact that i have a problem with the linkers, is my approach correct to generate a .obj mesh with the PCL Mesh ? @MartyG-RealSense

I do not have enough knowledge of PCL programming to comment on whether your approach is correct.

Hi @Amayess  Do you require further assistance with this case, please?  Thanks!

Hi @MartyG-RealSense , didn't found an optimal solution yet but you can close it.
Thank you.

Thanks very much @Amayess - as you suggest, I will close the case.  Thanks again!


Remove required dependencies from multi-ledger code

Also adjusts the return value from get_ledger_for_identifier for consistency.
Fixes #1549

@dbluhm -- can you please do the reviewing and testing of this, since you have the use case to try?
Others are welcome to jump in.  A lot more code changed than expected :-).

@dbluhm -- can you please do the reviewing and testing of this, since you have the use case to try?
Others are welcome to jump in.  A lot more code changed than expected :-).

Will do, thanks for working to address this one!

Codecov Report

Merging #1550 (04be014) into main (7422b3c) will increase coverage by 0.06%.
The diff coverage is 92.45%.

@@            Coverage Diff             @@
##             main    #1550      +/-   ##
==========================================
+ Coverage   95.69%   95.75%   +0.06%
==========================================
  Files         527      527
  Lines       32400    32306      -94
==========================================
- Hits        31005    30936      -69
+ Misses       1395     1370      -25


Better migration removal idea

So I practised removing migrations and found I could not do this so I went to SO and found using the -v option allowed me to see the error being generated which said I...
"cannot access the file ... ContosoUniversity.dll because it is being used by another process." Accidently via (Ctrl + F5).
Instead of dotnet ef migrations remove, you should consider dotnet ef migrations remove -v so user can see the error and is saved a trip to SO.

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

ID: e56e675b-5964-a75b-7e45-6f6d1a607e65
Version Independent ID: 78e47218-58a3-f8d9-e091-306126871a67
Content: Tutorial: Using the migrations feature - ASP.NET MVC with EF Core
Content Source: aspnetcore/data/ef-mvc/migrations.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@LayersOfAbstraction  good suggestion thanks. I made a similar change in https://github.com/aspnet/EntityFramework.Docs/pull/2011


Crosshair used with zoom/pan cause box flickering

As seen in our meeting, using the crosshair while drawing a box to zoom/pan causes the box to flicker. This problem doesn't happen when crosshair isn't selected.

I've reproduced this on my laptop, but am not sure when I'll be able to dive into what's causing. Thanks for the report @j-booth4!


ENV DEBIAN_FRONTEND=noninteractive

Why do not you use a variable DEBIAN_FRONTEND?
Because of this, during the installation using the apt-get errors similar to the following:

debconf: unable to initialize frontend: Dialog
debconf: (TERM is not set, so the dialog frontend is not usable.)
debconf: falling back to frontend: Readline
debconf: unable to initialize frontend: Readline
debconf: (This frontend requires a controlling tty.)
debconf: falling back to frontend: Teletype
invoke-rc.d: unknown initscript, /etc/init.d/systemd-logind not found.
invoke-rc.d: policy-rc.d denied execution of start.

If Dockerfile specify the following line, the error will disappear:
ENV DEBIAN_FRONTEND=noninteractive

It doesn't error out the build so I didn't bother incase people use the
files as a base later on
On Sunday, April 5, 2015, Evsyukov Denis<EMAIL_ADDRESS>wrote:

Why do not you use a variable DEBIAN_FRONTEND?
Because of this, during the installation using the apt-get errors similar
to the following:
debconf: unable to initialize frontend: Dialog
debconf: (TERM is not set, so the dialog frontend is not usable.)
debconf: falling back to frontend: Readline
debconf: unable to initialize frontend: Readline
debconf: (This frontend requires a controlling tty.)
debconf: falling back to frontend: Teletype
invoke-rc.d: unknown initscript, /etc/init.d/systemd-logind not found.
invoke-rc.d: policy-rc.d denied execution of start.
If Dockerfile specify the following line, the error will disappear:
ENV DEBIAN_FRONTEND=noninteractive
—
Reply to this email directly or view it on GitHub
https://github.com/jfrazelle/dockerfiles/issues/12.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

Ok, thank you!

if you use docker, you can write below line in the Dockerfile
ARG  DEBIAN_FRONTED=noninteractive

The only way I've been able to get this to work is by using ENV DEBIAN_FRONTEND noninteractive after the FROM line.
I then set it back to teletype a la ENV DEBIAN_FRONTEND teletype after the CMD line.
If you use the ARG command, where does that go in command order of the Dockerfile?

Sorry I don't really know what you are trying to say...? can you give an
example and which dockerfile you are referring to. I do not use the ARG
command in any of mine
On Fri, Oct 7, 2016 at 9:21 AM, Will Pracht<EMAIL_ADDRESS>wrote:

The only way I've been able to get this to work is by using ENV
DEBIAN_FRONTEND noninteractive after the FROM line.
I then set it back to teletype a la ENV DEBIAN_FRONTEND teletype after
the CMD line.
If you use the ARG command, where does that go in command order of the
Dockerfile?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jessfraz/dockerfiles/issues/12#issuecomment-252296543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYNbITXw4QBnUOvfuPZBbsaWO-r5cFGks5qxnF-gaJpZM4D6vHI
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3


Reuse FFT plan for convolve/correlate

Follow-up of https://github.com/cupy/cupy/pull/3560#issuecomment-656301909. Tests passed locally.

@emcastillo Could you post the script you used to generate https://github.com/cupy/cupy/pull/3560#issuecomment-655941892, or could you run it for me?

Both complex and real inputs are supported! For the latter, it makes sense to only reuse the forward plan; the backward one is run only once, so we may just let cupy.fft.irfft handle it.

Let me crunch the numbers

becomes 200us faster in all cases. drops to total time of 1200 us.

If we will have a plan cache soon, the only advantage of this PR would be to avoid accessing the cache.

Jenkins, test this please

Jenkins CI test (for commit a2cb2184ffa5925ccaef701177b457986bfad804, target branch master) failed with status FAILURE.

Hmm...The failures actually make sense. result_type(float16, complex128) returns complex128, but internally a float16 array is converted to a complex64 one before FFT runs. Not sure how my local tests and the other CIs got around this issue...

Yes, I was a bit puzzled

I think the problem may be when the two input arrays are not of the same type?

What's the NumPy version in cupy-py35?

1.15.4

Holy *!@#, cupy-py35 is the only one not on CUDA 10.0, and so the only one actually ran these tests...

I can locally reproduce the error by running this snippet:
import cupy as xp
from cupy import testing

dtype1 = xp.complex128
dtype2 = xp.float16

a = testing.shaped_random((200,), xp, dtype1)
b = testing.shaped_random((100,), xp, dtype2)
out = xp.convolve(a, b, mode='full')
print(out, out.dtype)

@emcastillo Fixing this requires addressing #3603 first I am afraid...

I wil find a way to work around #3603.

I worked around the dtype issue (#3603) under the assumption that reusing the plan gains more, and the bonus coming with this change is that the precision issue that we previously encountered is gone, so the tests no longer need to be skipped in CUDA 10.0.
@emcastillo We need to confirm if we still get the 200us reduction in time, otherwise we should just abandon this and start working on a plan cache.

Jenkins, test this please

Jenkins CI test (for commit 59a4346b2f3ad04b4e5c10a1d10af17a99ec60b6, target branch master) succeeded!

I think the problem is that you are using a complex plan for a non-complex datatype,
you should only reuse plans if both datatypes are the same

you should only reuse plans if both datatypes are the same

Or when we know both dtypes would be cast to the same dtype when calling fft.

Let's just add the comment and this will be good to go :)

Jenkins, test this please

Jenkins CI test (for commit 8fdc6a9d0c42f1d886d0c729ee6d270a64cc746d, target branch master) succeeded!

On my system I saved 400us CPU time for two complex128 inputs. GPU time fluctuates a lot from shot to shot, so I couldn't tell the impact, but I think it's a good sign.


[v0.7.0] Pipeline content has invalid ARGS param

Describe the Bug
There is a mismatch between the example pipeline content in the docs and the version on github. The content in the docs has a single string value for the ARGS parameter, whereas the github content in pipelines-tutorial has an array.
Steps to Reproduce

Copy the content in Assembling a pipeline into a YAML file.
Create/apply the pipeline YAML
Start a pipeline run as specified in Triggering and Running a Pipeline

Expected Behavior
Pipeline runs successfully.
Actual Behavior
Pipeline fails to run due an invalid value for the ARGS parameter.

@adambkaplan thank you for raising the issue. The content has been updated as per the latest changes in the tutorial, this was true of 0.7 release, the current release 0.10.5 is up to date. I am closing this issue for now, please feel free to reopen it in case you see anything missing.
Also the contributing doc has been updated with instructions for contributing and publishing docs, we would be happy to receive PRs.


No Deploy Button

I'm getting a broken link to the status image and no "Deploy" button.

Developer error - I was using the wrong API ID value - sigh!


Commit composer.lock and platform version

By locking the platform version we encourage a good practice which prevents people from messing up with a prod server that has a different version than the one on the dev's machine that ran composer up.
By committing the composer.lock file, we optimize a lot the installation process.
Here is a comparison of a create-project with/without the lock file:

Without lock
With lock

Nb of files downloaded
167
40

Peak Memory
274MB
120MB

Duration on regular network
19s
12s

The difference of duration can be much worse on slower networks.
Process-wise, this will require us to ship a new lock file with each new Symfony version. This feels like something we can automate in the releasing process, and will give us the opportunity to synchronize skeleton's version numbers with Symfony'.
Here is a Blackfire comparison with the website-skeleton:

https://blackfire.io/profiles/compare/42c40cf5-46d0-40df-acf3-94e929482889/graph
This illustrates how much this will improve the experience of installing Symfony for newcomers.

While it's a very interesting approach, I don't like the idea of being locked into a specific version (say 7.0.8) if I'm running 7.1 or 7.2, it might lead to receiving "outdated" versions of the dependencies. I'm not exactly sure how else to do this neatly though. Would it help to specify the minimum patch versions in the requires instead? Perhaps a --super-optimized flag?
It seems like the original issue is somewhat related to this: https://github.com/composer/composer/issues/7273

This PR is for branch 3.4.
We will bump the platform to 7.1 on 4.0.
Then, of course, this file is just the starting point for projects and will be edited.
It's still useful to encourage people to actually use that setting.

I don't think this is a good idea. When I run create-project now, I will actually get wrong dependencies. Your composer.lock will give me dependencies for PHP 7.0.8 but I have 7.2.7 so I would get different library version combinations. If you commit that I have to run composer create-project, adjust the composer.json and then run composer update again. Nothing about this process helps to save memory, it just makes things more complicated.
I agree that configuring the PHP version in the platform config is a good idea to make sure it matches the production system but you have to do that in your local project not in the skeleton.

@Toflar what matters is the first time to success. You will not have "wrong" dependencies, you will have dependencies that work on 7.2 also. And the skeleton is just the starting point, which we lock here to make it stable and fast to bootstrap.
I think that will improve the experience of installing Symfony.

If I were to use phpunit as dependency (which it is, because my tests depend on the public API of it), I would get an old version of phpunit though, @Toflar does have a valid point on that: https://github.com/sebastianbergmann/phpunit/blob/c1a87c0a4d9457c8ca051ef8b0b8aac0339207a9/composer.json#L24
For developers less experienced with package management/composer or development in general, this might cause more problems than it solves in performance, as they will not receive the dependencies they'd expect.

No, that's not true. Think about a dependency that defines requires like so:
"require": {
    "php": ">7.0 <7.2",
}

because it uses something that explicitly only works for PHP 7.0.* and 7.1.*. I would get the dependency even though it fails on my system when I have PHP 7.2.*.
Sure, this example is probably very rare but that's what constraints are about. Any combination is possible.

you will have dependencies that work on 7.2 also

This is an assumption and you don't know if that's true.
It's not a good idea to commit the composer.lock for a skeleton. It will make it faster to bootstrap, yes but at the cost of probably providing a combination that doesn't work for your system. This approach will very likely cause issues at some point.
Regarding the platform config. I think it would be better if Flex checked the composer.json and showed a warning/recommendation to add the PHP version there to ensure you're not installing dependencies that do not match the target platform.

That's a skeleton. Dependencies are not random but minimal. This issue doesn't exist with any deps of the skeleton. About platform, putting a recommendation in Flex will add complexity for something that is unrelated to its business. Then, if people don't know how to deal with deps, its even more important to lock the platform, because they will just mess up with local vs remove versions of PHP.
Which means I'm still pushing to commit this lock file.

Here is a Blackfire comparison when with the website-skeleton:

https://blackfire.io/profiles/compare/f77c9501-c71a-493d-bdba-bde6a9debc56/graph
This illustrate how much this will improve the experience of installing Symfony for new comers.

For newcomers that means it all installs faster now but it probably installs packages with security issues or incompatibilities.
For experienced devs it means you have to run create-project, adjust the composer.json to match your platform and then run update.
Don't get me wrong, I love your efforts on trying to make this experience better but what I think it actually does is that it prefers performance over integrity and that's not a good decision. But that's my 2c, let's see what other devs think 😄

probably installs packages with security issues or incompatibilities.

impossible: we manage the lock file and we release new version of the skeleton as part of each new version of Symfony. Again, that not random deps, but deps of the skeleton.

For experienced devs it means you have to run create-project

That's a skeleton. You WILL have to use composer very soon. Nothing new here.

What would the performance be in the following scenarios if you have this composer.json?

The current situation without lock file for the first process that installs the packages
The new situation for the first time running composer update

    "require": {
        "php": "^7.0.8",
        "ext-ctype": "*",
        "ext-iconv": "*",
        "symfony/console": "^3.4.12",
        "symfony/flex": "^1.0.80",
        "symfony/framework-bundle": "^3.4.12",
        "symfony/lts": "^3",
        "symfony/yaml": "^3.4.12"
    },
    "require-dev": {
        "symfony/dotenv": "^3.4.12"
},

I have no problem with it, because after starting a project the composer.json file is often modified.
If this feature can convince users and guide them to good practice, then why not

@iltar I'm not 100% sure to understood your question, but if you're wondering how performance would change by narrowing the version windows (^3.4.12), the answer is not at all: solving the dependencies it not the resource consuming part. What is resource intensive is 1. downloading the json repositories from packagist 2. keeping them in memory. And this has to be done whatever the version windows are.

probably installs packages with security issues or incompatibilities.

impossible: we manage the lock file and we release new version of the skeleton as part of each new version of Symfony. Again, that's not random deps, but deps of the skeleton.

Well that means you won't get any updated packages when no new Symfony version is released. There could very well be a dependency of Symfony that does have a security issue and you're not getting it.
On the same page, you won't get Doctrine DBAL 2.7, because it only works with PHP 7.1. So the Doctrine version you get won't (fully) support MySQL 5.7…

Not sure this will work but, did you consider to use some composer events, like pre-package-install to make this option configurable?
For example you can ask user what version to set to composer.json before resolving dependencies:

use pre-defined version
use current system version if it fulfils minimum-requirements
set custom one, that developer can get from the production server, for example

What we did to avoid this issue in the deprecated Symfony Installer was to automatically remove the config.platform option from composer.json after the project was created (see https://github.com/symfony/symfony-installer/blob/ff32aacce264076f26b6924e5dde0ff9ffb36773/src/Symfony/Installer/Manager/ComposerManager.php#L18). Maybe we can do the same here?

@javiereguiluz I didn't know about this, thanks for the reminder.
I'm mixed on this: on one side locking the platform prevents installing e.g. Doctrine DBAL 2.7 as @aschempp noticed, on the other side removing config.platform feels like a missed opportunity to set the option and encourage a best practice that prevents some nasty mistakes (we don't want to make Flex interactive so we don't want to go down with @anyt proposals.)
Still, OK to remove config.platform on my side.

By the way, I forgot to say that I like this a lot!! I agree with Nico that the first installation time is a critical metric. A minor question related to this: should we remove ext-ctype and ext-iconv from composer.json and replace them by Symfony's related polyfills?
PS: I also hope that some day the entire PHP community organizes a joint effort to massively improve Composer's performance. It's sad that Yarn takes a few seconds to do anything and Composer takes so long even for small apps. Of course I'm not complaining about Composer's maintainers. Thanks to them for everything they do 🙏

First of all, making things faster is a laudable goal. I like that a lot. Now, how to achieve that (without re-introducing an installer)?
Several issues raised in the PR.
The first issue is about deps that are not part of "symfony/symfony". They might be updated between two releases, and the lock won't be updated (that was also the case with the Standard Edition). This issue does not really exist for the bare skeleton (this repo) as we only depend on a sub-part of symfony/symfony (+ the polyfill), but it definitely exists for the website skeleton (which depends on Twig, Swiftmailer, Doctrine, ...).
The second issue is if you would have had more recent/different dependencies because your version of PHP is different locally (we compute the lock with 7.0.8, but you have PHP 7.2.4). I think this is marginal and not something we want to take into account. We can reconsider if that's really a problem later on. What's important is to not have the platform hardcoded in the composer.json file.
With that in mind, I'd like to propose something different from this PR:

We don't change the composer.json file and we only commit the composer.lock file (and we make sure that the right version of PHP is used to generate it like 7.0.8 for Symfony 3.4).

We setup a cron that each hour tries to update the composer.lock. If there are some changes, it commits them and tag a new version (for Symfony 4.1 for instance, the next version would be <IP_ADDRESS>, and if a third-party dep is updated before 4.1.14, we would release <IP_ADDRESS>).

By using the cron feature of Travis (where we can nicely get all the PHP versions we need), that should be completely automated.
That way, people creating a project will always have the latest versions of all deps, including transitive ones. And a simple composer update gives you whatever updates depending on your local PHP version. Huge gains without too many drawbacks
WDYT?

That way, people creating a project will always have the latest versions of all deps, including transitive ones.

That assumes they do run with the outdated PHP version (like Doctrine ORM 2.7 does not …)

It's sad that Yarn takes a few seconds to do anything and Composer takes so long even for small apps.

This is comparing apples and oranges. PHP needs a SAT solver because you can use one class only ever once. Javascript doesn't because you can include the same modules in multiple versions if you work with modules 😄 Optimizing it in the current state is really challenging, I've tried for many, many hours...but this is a discussion for the composer/composer repository. I just wanted to make sure we do not compare PHP's package manager with npm or yarn in this issue 😊

@Toflar I don't want to go off-topic too much, but just to give you some more info. Executing composer create symfony/skeleton on my machine:

Called strtolower() 429,669 times
Made 72 non-parallel HTTP requests to packagist.org (compare that to Flex, which makes 1 HTTP request)
Called VersionParser::parseConstraints() 117,076 times
Constructed 117,075 Composer\Package\Link() objects
Garbage collector took 7.47 seconds (12% of total)
The 139 calls to json_decode() took 310 MB of memory
etc.

It only a guess ... but there could be massive opportunities to improve performance here at all levels: I/O, CPU, memory, etc.

@javiereguiluz and all these should be discussed in composer/composer.

I like that plan @fabpot :)
Here is a first step: https://github.com/symfony/skeleton/tree/composer-update
This branch contains a .travis.yml that is now setup to run daily (the max frequency on Travis.)
Here is an example build: https://travis-ci.org/symfony/skeleton/builds/400979967
This computes the next version number by incrementing its 4th part, prefixed by the last tag from https://symfony.com/versions.json
We now need to make it push the update + tag it somehow.

A minor question related to this: should we remove ext-ctype and ext-iconv from composer.json and replace them by Symfony's related polyfills?

I didn't answer this: the answer is no. ctype and iconv are always enabled, except on maybe on Alpine & custom builds. We really want ppl to have a great experience from start, so that even ppl that do custom things have great perf.
So the current situation is by design and desired.

well, for ctype, it is also related to the fact that the ctype polyfill is quite new. So this might be reconsidered.
But I agree about not using the iconv polyfill (thus, it is huge)

This computes the next version number by incrementing its 4th part

That's not semver-compliant, as I've raised in #70


Nats Jetstream example

Add a simple example for data and images using NATS Jetstream.
this will automatically make the example do real time sync between all users using the app. When an image or data changes all users will get the change either online  ( instantly )  or offline ( when they go back on line , it will push the message updates they missed ).
A nats leaf server can be embedded for full offline editing and sync. https://docs.nats.io/running-a-nats-service/configuration/leafnodes/jetstream_leafnodes explains it.
migrations:
Stare is stored on the NATS server.
if the data types change then the data stored in NATS must be migrated.
The typical solution to this is to use protobufs for data.
For images it’s fine because they are just binary , but we should sniff the image type and switch in the code .
web:
gio supports WASM and so does nats.go. So web works fine too

Yes 👍

Suggest we use this as a base
https://github.com/oderwat/go-nats-app
The gui is WASM , and so is gio for web at least

Then add marmot nats for SQLite replication as every app needs this and we reuse the nats server.
https://github.com/maxpert/marmot
this allow event streaming with gio which is very powerful because a mutation event can update the DB as well as for example a bleve faceted search store . Check bleve ( a golang store for free text and facet search ) at https://github.com/blevesearch/bleve

this will be added to a different repo as it brought in too many dependencies.
not sure which repo yet.

I close this issue as we have not plan on this example.


Start time should always be parsed before end time calculation

Context
PV-672
How Has This Been Tested?
Tested manually (checked immediate and selected dates)
Manual Testing Instructions for Reviewers
In Webshop UI, check both "Start immediately" and "From" options. Both should run without issue.
Screenshots

Codecov Report

Merging #405 (11669a9) into develop (4a6584b) will increase coverage by 0.01%.
Report is 3 commits behind head on develop.
The diff coverage is 72.72%.

@@             Coverage Diff             @@
##           develop     #405      +/-   ##
===========================================
+ Coverage    80.95%   80.96%   +0.01%
===========================================
  Files          149      149
  Lines         8783     8796      +13
===========================================
+ Hits          7110     7122      +12
- Misses        1673     1674       +1

Files
Coverage Δ

parking_permits/customer_permit.py
72.44% <72.72%> (+0.03%)
:arrow_up:

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Strange 'Element is not visible' error

Are you requesting a feature or reporting a bug?
Bug
What is the current behavior?
Error 'Element is not visible' is shown when clicking on second div.
What is the expected behavior?
Scroll to the second div and click on it.
How would you reproduce the current behavior (if this is a bug)?
Run the test
Provide the test code and the tested page URL (if applicable)
Tested page:

<title></title>

Span1

Span2

Test code
fixture `123`.page `belym-w8:8080`

test('test', async t => {
    await t
        .click('body > div:nth-child(2)')
        .click('body > div:nth-child(3)');
});

Specify your

operating system: Windows 10
testcafe version: 0.9.0-rc2
node.js version: 6.5.0

An another example of the page:

<title></title>

Attempt to do a click on a div that is not on visible part of the page gives 'Element is not visible error', instead of performing scroll to the element.

The problem was caused by incorrect document.documentElement.clientHeight usage in scrollAutomation. This property represents height of the whole page in Chrome, not just the viewport height. The document.body.clientHeight must be used for this purpose in Chrome.

Working on it.


Error in youtube again.

I'm using Windows + Brave, with Ghostery + Tampermonkey addon for youtube premium, and some hours ago i still having the error, the windows get black.

Same for me

Same for me

If you open it in a new window, it will play, but if you just play the video, only a black screen will appear and it will not play.

It working today!, mistake was only yesterday.

It stopped working completely for me. Not sure when was the last time I went on YouTube; most likely in the past week.

Open in a private window and ensure your extensions are enabled on private windows.


Size of the test set of the Informer datasets

Hi Nate!
Just scanned through you marvelous work. I found that the precomputed output of Autoformer on the Informer datasets are substantially smaller than the original test set. As you mentioned in your paper that the test set has been narrowed, but what is the actual size of the test set?

HI Astor,
The test set has one chunk per covariate and they are predicted independently. For the datasets with a small number of covariates, you could probably run evaluation with the original test set, but with many covariates it is intractable, as the number of series to predict will be (number of covariates) x (number of temporal chunks).
Nate


Fixed typo with the direction of how to use custom sanitizers.

Good catch! Thanks!

This fix is in 2.14.2

Thanks!


[DCM KIT] Release Checks

This issue tracks all relevant release checks for your KIT.
Important
Follow the guidance on how to use the templates.
Release Info
Please provide information on what you want to be included in the Eclipse Tractus-X release. If you are not owner of this issue, please provide the information as comment to the issue. Have your prepared content reviewed by a professional peer group before submitting it for committer approval. Best practice: have a professional Peer review your prepared content before you submit for expert approvals. Please link the open or merged PR that includes the changes.
Version to be included in Eclipse Tractus-X release: version placeholder

KIT semantic version of change log.
Date of KIT publication must be aligned with overall release date.

If changes to your KIT are not yet live on our website, link the open PR introducing the changes for this release.

DCM PR #981
DCM PR #984
DCM PR #985

If changes are already live, also point out the PRs witch included the changes for this version if possible.
General

[ ]  CHANGELOG available and maintained
[ ]  Tractus-X Release Guidelines fulfilled

Adoption View

[ ]  Vision and Mission statement present
[ ]  Business Value statement present
[ ]  Semantic Models linked and described
[ ]  Logic / Schema available and described
[ ]  Business Process documentation available
[ ]  Access and Usage Policies documented
[ ]  Standards submitted / published via the Catena-X e.V. process and linked with current version (e.g., semantic models)
[ ]  Tutorials available
[ ]  White Paper available
[ ]  Notice CC BY 4.0 licence statement available

Development View

[ ]  API Specification published on SwaggerHub
[ ]  Protocols available (e.g., dataspace protocol)
[ ]  Sample Data available
[ ]  Architecture integration into the overall Catena-X architecture and / or product architecture (e.g., Arc42) available.
[ ]  Reference Implementation available

Operation View

[ ]  Setup Guide available (e.g., installation guides)
[ ]  Deployment Script available (e.g., helm charts)

Helpful Links

[CX-0128 Demand and Capacity Management Data Exchange 2.0.0]

@ClosedSourcerer: Can you please have a look if anything is missing to have the release ticket? Thanks.

@jSchuetz88 We urgently need support for the Eclipse IP Clearance for PR #981 mentioned above. Already requested in the PR support from @danielmiehle  and @maximilianong  but still pending and it was mentioned it takes 14 days. Thanks.

@ClosedSourcerer: Can you please have a look if anything is missing to have the release ticket or if we can already tick some checkboxes? Thanks.

General

[X]   CHANGELOG available and maintained
[ ]   Tractus-X Release Guidelines fulfilled

They change all the time, so I am not signing this off.

Adoption View

[X]   Vision and Mission statement present
[X]   Business Value statement present
[X]   Semantic Models linked and described
[X]   Logic / Schema available and described
[X]   Business Process documentation available
[X]   Access and Usage Policies documented
[X]   Standards submitted / published via the Catena-X e.V. process and linked with current version (e.g., semantic models)
[ ]   Tutorials available
[ ]   White Paper available
[X]   Notice CC BY 4.0 licence statement available

According to the TRGs this needs to be everywhere, not just within the adoption view and according to the tempalte the adoption view does not exist.

Development View

[X]   API Specification published
[ ]   Protocols available (e.g., dataspace protocol)
[X]   Sample Data available
[X]   Architecture integration into the overall Catena-X architecture and / or product architecture (e.g., Arc42) available.
[ ]   Reference Implementation available

FOSS Application references, however since it has no released package for 24.08 I do not consider it a reference implementation

Operation View

[ ]   Setup Guide available (e.g., installation guides)
[ ]   Deployment Script available (e.g., helm charts)

Within the FOSS repository

Helpful Links

[CX-0128 Demand and Capacity Management Data Exchange 2.0.0]

And just fyi: Nothing really plays nicely together. The Eclipse Foundation Rules, the association release process, the KIT template, this release checklist. There are HUUUUUGE gaps everywhere and sometimes even direct contradictions ;) (Business View vs. Adoption View)

Hi @LukasSchmetz ,
thank you for all your efforts :), as all three PRs has been approved and already merged, I changed the status to done.

Hi @jSchuetz88 small fix still outstanding in PR #1010 to move Onboaring Content to the right place in Eclipse Tractus-X


TypeError: webpack_require(...) is not a function

Bug report

What is the current behavior?
TypeError: __webpack_require__(...) is not a function

If the current behavior is a bug, please provide the steps to reproduce.
it's a really simple situation that takes you 5 minutes to check the entire codes.
// ./src/index.js
require('./other');

// ./src/other.js
module.exports = { a: 1 };

// ./babel.config.js
module.exports = {
    "presets": ["@babel/preset-env"],
    "plugins": ["babel-plugin-source-map-support"]
};

// ./webpack.config.js
const path = require('path');
const { CleanWebpackPlugin } = require('clean-webpack-plugin');
const webpackNodeExternals = require('webpack-node-externals');

module.exports = {
    mode: 'development',
    target: 'node',
    devtool: 'source-map',
    entry: './src/index.js',
    output: {
        filename: 'index.js',
        path: path.resolve(__dirname, 'dist'),
    },
    module: {
        rules: [{ test: /\.js$/, use: 'babel-loader' }],
    },
    plugins: [
        new CleanWebpackPlugin(),
    ],
    externals: [webpackNodeExternals()],
};

// package.json
{
  "name": "webpack-bug",
  "version": "0.0.1",
  "description": "",
  "main": "./dist/index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "build": "webpack"
  },
  "author": "",
  "license": "UNLICENSED",
  "devDependencies": {
    "@babel/cli": "^7.5.0",
    "@babel/core": "^7.5.0",
    "@babel/preset-env": "^7.5.0",
    "babel-loader": "^8.0.6",
    "babel-plugin-source-map-support": "^2.0.1",
    "clean-webpack-plugin": "^3.0.0",
    "webpack": "^4.35.2",
    "webpack-cli": "^3.3.5",
    "webpack-node-externals": "^1.7.2"
  },
  "dependencies": {
    "source-map-support": "^0.5.12"
  }
}

you can clone the repo for checking.
$git clone git@github.com:zimtsui/webpack-bug.git
$cd ./webpack-bug
$npm i
$npm run build
$node .
TypeError: __webpack_require__(...) is not a function

babel-plugin-source-map-support is a simple babel plugin, which just add a single line at the beginning.
$npx babel ./src/index.js
"use strict";

require("source-map-support/register"); // the plugin adds this line

require('./other');

if you disable the plugin and manually add that line in source file, everything will be ok:
// ./babel.config.js
module.exports = {
    "presets": ["@babel/preset-env"],
    // "plugins": ["babel-plugin-source-map-support"]
};

// ./src/index.js
require('source-map-support/register');
require('./other');

if you use exports instead of module.exports to export, everything will be ok as well:
// ./src/other.js
exports.a = 1;
// module.exports = { a: 1 };

What is the expected behavior?
to exit normally and output nothing.

Other relevant information:
webpack version:  4.35.2
Node.js version: 10.16.0 lts
Operating System: ubuntu 18.04
Additional tools:

Problems in webpackNodeExternals, it is package has invalid regexp

O(∩_∩)O xie xie

Problems in webpackNodeExternals, it is package has invalid regexp

emmm...that seems unrelated with webpackNodeExternals
if you comment externals, the problem still exists
// webpack.config.js
const path = require('path');
const { CleanWebpackPlugin } = require('clean-webpack-plugin');
// const webpackNodeExternals = require('webpack-node-externals');

module.exports = {
    mode: 'development',
    target: 'node',
    devtool: 'source-map',
    entry: './src/index.js',
    output: {
        filename: 'index.js',
        path: path.resolve(__dirname, 'dist'),
    },
    module: {
        rules: [{ test: /\.js$/, use: 'babel-loader' }],
    },
    plugins: [
        new CleanWebpackPlugin(),
    ],
    // externals: [webpackNodeExternals()],
};

@zimtsui can you create minimum reproducible test repo?

@zimtsui can you create minimum reproducible test repo?

maybe that's already simplest repo. 2 source files of single line only.
https://github.com/zimtsui/webpack-bug
git@github.com:zimtsui/webpack-bug.git
something seems wrong with webpack/buildin/harmony-module.js
in the compiled file, __webpack_require__(...) returns an empty object {}, but used as a function.

babel-cli makes the same output for these 2 situations:

manually adding require("source-map-support/register"); on the top of source file
adding require("source-map-support/register"); with babel-plugin-source-map-support

therefore, theoretically, babel-load should output the same, and the bundler should run in the same context.
but actually, webpack used webpack/buildin/harmony-module.js for situation 2, but didn't for situation 1
// for situation 2
[./node_modules/webpack/buildin/harmony-module.js] (webpack)/buildin/harmony-module.js 670 bytes {main} [built]
[./src/index.js] 101 bytes {main} [built]
[./src/other.js] 84 bytes {main} [built]

// for situation 1
[./src/index.js] 60 bytes {main} [built]
[./src/other.js] 46 bytes {main} [built]

see https://babeljs.io/docs/en/options#sourcetype
babel-plugin-source-map-support seem to insert imports

see https://babeljs.io/docs/en/options#sourcetype
babel-plugin-source-map-support seem to insert imports

but import will be compiled to require by @babel/preset-env
just try:
npx babel ./src/index.js

@zimtsui should be not, please provide your babel config

@zimtsui should be not, please provide your babel config

https://github.com/zimtsui/webpack-bug/blob/master/babel.config.js
module.exports = {
    "presets": ["@babel/preset-env"],
    "plugins": ["babel-plugin-source-map-support"]
};

Can you try without babel-plugin-source-map-support?

that'd be correct. see the content above.
but i dont think it's problem of that plugin, because the output of babel-cli goes well

You should not content import to require fro webpack@4, you should keep import, if plugin change this, please look in options to disable this behavior or open issue about this in plugin repo, thanks

You should not convert import to require for webpack@4, iif plugin change this, please look in options to disable this behavior or open issue about this in plugin repo

webpack@4 does support commonjs exactly.
furthermore, if source-map-support has to be import as es6 module, nodejs should catch a runtime- error from the plugin, but not a type-error from webpack

but import will be compiled to require by @babel/preset-env
just try:
npx babel ./src/index.js

The behavior for babel-cli and babel-loader differs. babel-loader doesn't compile to commonjs because webpack understands ESM.

Set sourceType: "unambiguous" as I recommended.

but import will be compiled to require by @babel/preset-env
just try:
npx babel ./src/index.js

The behavior for babel-cli and babel-loader differs. babel-loader doesn't compile to commonjs because webpack understands ESM.

since webpack understands esm, it should work if i make esm everywhere, just like:
// ./src/index.js
import './other';

// ./src/other.js
export default { a: 1 };

// ./babel.config.js
module.exports = {
    "presets": ["@babel/preset-env"],
    "plugins": ["babel-plugin-source-map-support"],
};

but in fact it doesn't work.
even though I set "sourceType": "unambiguous" ("target": "node" has been set), webpack includes all build-in modules such as fs path ...
$ npx webpack

Hash: aca1fd6fa5a958740feb
Version: webpack 4.35.2
Time: 1093ms
Built at: 07/09/2019 3:53:54 PM
       Asset     Size  Chunks             Chunk Names
    index.js  131 KiB    main  [emitted]  main
index.js.map  217 KiB    main  [emitted]  main
Entrypoint main = index.js index.js.map
[./src/index.js] 162 bytes {main} [built]
[./src/other.js] 155 bytes {main} [built]
[fs] external "fs" 42 bytes {main} [optional] [built]
[module] external "module" 42 bytes {main} [optional] [built]
[path] external "path" 42 bytes {main} [built]
    + 14 hidden modules

im sure there is something wrong within webpack loading mechanism.

im not a free rider who's saying "hey, debug this for me"
this repo is what i made specially for this bug.
sincerely, i suggest webpack members debug this problem FOR WEBPACK

@zimtsui it is not webpack problem as we said above, also i can't reproduce problem using your reproduce test repo

I could reproduce the problem with your repo.
Adding sourceType: "unambiguous" to babel.config.js fixes the problem.

webpack includes all build-in modules such as fs path ...

These are modules source-map-support/register are using, so that's fine

I could reproduce the problem with your repo.
Adding sourceType: "unambiguous" to babel.config.js fixes the problem.

webpack includes all build-in modules such as fs path ...

These are modules source-map-support/register are using, so that's fine

that really makes sense. thank you very much O(∩_∩)O


Update limits-specifications-teams.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/4359
Added note.

@David-Rosenthal please review.

@LolaJacobsen I actually wasn't aware of this limit, but if this came from the owning team of this functionality I'm fine with this edit. Can you let me know where this info came from so I can follow-up after the fact to check on why this limit exists in GCC but nowhere else? Thanks!

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LolaJacobsen PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@David-Rosenthal it came from this GitHub issue:
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/4359


Print as landscape with middleware

Hello,
I'm using PDFKit on my rails application and I need to set the pdf as landscape but I'm using the middleware.
Is this possible?
I tried this css but doesn't work (maybe wkhtmltopdf issue?):
@page {
  size: landscape; }

Sure, just use the 'orientation' option for wkhtmltopdf.
when setting up middleware
config.middleware.use PDFKit::Middleware, :orientation => 'Landscape'
or as a meta tag in the HTML


Does not work with srcset/sizes

When the sizes attribute is used on an image, Firefox and Chrome (at least) appear to be using the calculated size based on the sizes attribute for image.naturalWidth and image.naturalHeight. This means the scale factor is wrong.

That is not currently supported. Would welcome a PR.

also is there a onChange event for area coords to change and fire another event?
cheers.

Do you have an example I could look at?

I think this is now fixed by a PR


Flutter Webview upload image button not working

Hello,
I have a button to upload images in the web page I call in the Webview plugin. When I press this button, there is no loading screen at all. Flutter webview does not support this feature?
My project is interrupted.
Can you please help?

Hi @atamturk
It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you