Bug fix folder and file validation

Description
Folder validation
To fix bug #393.
If root path is missing:

Will have a log message about the root path is missing.

The project will still be visible, but the total number of images is 0. Letting the project stays visible will prevent duplication of project naming.

If root path is restored:

Images will be restored by refreshing the browser.

Extra to have?[Not Implemented Yet]
Change the project status, for example to [Project Path Missing] to be more descriptive. Requires frontend support

Image validation
To fix bug #395
Changes the method used to calculate the total number of images in folder
from https://github.com/CertifaiAI/classifai/blob/v2_alpha/classifai-core/src/main/java/ai/classifai/util/data/FileHandler.java#L69-L83
to https://github.com/CertifaiAI/classifai/blob/sum_v2a_folderFileValidation/classifai-core/src/main/java/ai/classifai/util/data/ImageHandler.java#L252-L255.
Better logging for not supported image
Types of files are recorded in the log.

Changelog

Not related to this pr as these are fixes for developing features.
Added for auto pr for frontend

Fixes # (issue)
#393
#395
Type of change
Please delete options that are not relevant.

[x] 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)
[ ] This change requires a documentation update

Tested on?

[ ] Windows
[x] Linux Ubuntu
[ ] Centos
[ ] Mac
[ ] Others  (State here -> xxx )

Checklist:

[x] My code follows the style guidelines of this project
[x] 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
[x] 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
[ ] Any dependent changes have been merged
[x] CHANGELOG.md has been updated.

@codenamewei the bug is fixed.

Using this dataset image_fail_display.zip
log gist: here

Found a concern in general. When a data point is not supported, its saved to the database, checked and not loaded everytime. For example, if the image is webp and not supported at the current point, should it dont save in database in the first place?
That way the tool dont need to check and failed to load the image everytime loading the project, which spend the computing time.
the message of log.info is printing alot of messages in the terminal when there's alot of project. Do we want to consider changing it to log.debug?

there is concern of log.info and all the printing in terminal will actually slow down the performance of the tool (cause printing in terminal is always slower but this problem does not have in log.debug and log.warn cause it only writes to the file without printing it)
Example of log message in terminal when i have multiple projects: here
since end user wont see terminal if they install using package, printing it in terminal does not matters cause they dont see it. If want user to notice what is wrong, backend will have to send the error content to frontend to print it out as message prompt

The invalid images is not saved in database. Instead, the logic during metadata retrieval always checks the project root path to get the number of images.

From the point 1, we see that files checking always happens when metadata is retrieved, therefore there're alot image invalid messages.

Changing the log info message to log debug, did not solve users' concern when image is not loaded into classifai. Prompt it to frontend is a good idea if log debug is used.

Suggestion

Change log.info to log.debug + frontend promt message -> to tell user which images is not imported and why it is not imported.
Keep the metadata retrieval logic first since we do not want to cause any potential logic breaking. And keep this in note to change the image number retrieval logic in the next bug fix / version release

Using this dataset image_fail_display.zip
log gist: here

Found a concern in general. When a data point is not supported, its saved to the database, checked and not loaded everytime. For example, if the image is webp and not supported at the current point, should it dont save in database in the first place?
That way the tool dont need to check and failed to load the image everytime loading the project, which spend the computing time.
the message of log.info is printing alot of messages in the terminal when there's alot of project. Do we want to consider changing it to log.debug?

there is concern of log.info and all the printing in terminal will actually slow down the performance of the tool (cause printing in terminal is always slower but this problem does not have in log.debug and log.warn cause it only writes to the file without printing it)
Example of log message in terminal when i have multiple projects: here
since end user wont see terminal if they install using package, printing it in terminal does not matters cause they dont see it. If want user to notice what is wrong, backend will have to send the error content to frontend to print it out as message prompt
@codenamewei

The invalid images is not saved in database. Instead, the logic during metadata retrieval always checks and loops through the project root path to get the number of images.
From the point 1, we see that files checking always happens when metadata is retrieved, therefore there're alot image invalid messages.
Changing the log info message to log debug, did not solve users' concern when image is not loaded into classifai. Prompt it to frontend is a good idea if log debug is used.

Suggestion

Change log.info to log.debug + frontend promt message -> to tell user which images is not imported and why it is not imported.
Keep the metadata retrieval logic first since we do not want to cause any potential logic breaking. And keep this in note to change the image number retrieval logic in the next bug fix / version release

i agree with the suggestion. kindly work on the details..

Will open another pull request as the frontend has changed.


Update testing-other-browsers.md

changed cloud-based testing list introduction.

Docs Build status updates of commit b30b3c9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/device-mode/testing-other-browsers.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

mistake


Understanding convolution kernels in dilation layers

Hi @ibab,
I'm a bit late to the wavenet paper implementation party, but I'm reading the paper and your code and trying to understand where dilated convolution kernels are present. Your ASCII diagram shows

               |-> [gate]   -|        |-> 1x1 conv -> skip output
               |             |-> (*) -|
        input -|-> [filter] -|        |-> 1x1 conv -|
               |                                    |-> (+) -> dense output
               |------------------------------------|

        Where `[gate]` and `[filter]` are causal convolutions with a
        non-linear activation at the output. Biases and global conditioning
        are omitted due to the limits of ASCII art.

The Wavenet paper diagram shows a single 'Dilated Conv' fed into both tanh and sigmoid functions.
From your ASCII diagram and code (which agree), it seems there is in fact not not one dilated convolution but two dilated convolutions, one for the tanh (defining the 'filter'), and one for the sigmoid (defining the 'gate'). Is this correct and is this what was in fact intended in the Wavenet paper?
Additionally could you give justification for the parameter choices not mentioned in the paper?
  '''Implements the WaveNet network for generative audio.

    Usage (with the architecture as in the DeepMind paper):
        dilations = [2**i for i in range(N)] * M
        filter_width = 2  # Convolutions just use 2 samples.
        residual_channels = 16  # Not specified in the paper.
        dilation_channels = 32  # Not specified in the paper.
        skip_channels = 16      # Not specified in the paper.
        net = WaveNetModel(batch_size, dilations, filter_width,
                           residual_channels, dilation_channels,
                           skip_channels)
        loss = net.loss(input_batch)
    '''

Thanks in advance.

Answering this for myself from looking through the literature, yes it looks like there are in fact two distinct dilated convolutions passed to the 'gated activation unit'- the original wavenet paper diagrams appear misleading.

@redwrasse, I agree that the original paper misses some details here and there. Take a look at Gated PixelCNN by WaveNet's main author and you will find that he "copies" the gated activation from there. Also, it seems like they stacked them along the output function dims to spare a conv1d.
For the later, have a look here
https://github.com/cheind/autoregressive/blob/e1f9b72b0f9764f9b4d6b6f65f028cd50db6940e/autoregressive/wave.py#L63

Thanks @cheind, I'll take a look. A side project I'd like to get back into.

@redwrasse, same for me :) I just figured that it works nicely on 2D images as well (without the special architecture of PixelCNN, just plane WaveNet with unrolled images). In addition, once you have the joint you might start to query all kind of things from the model (like given a wavenet conditioned on the speaker id, what is the probability that this speech was spoken by speaker X).
In case you are interested, I have a quite elaborate presentation + code here
https://github.com/cheind/autoregressive/tree/image-support


🔧 add more skill item and localize them

resolves #16
add more skills related to various non-mainstream tech and new tech like deno .

Hi @rohanprasadofficial, please rebase.

Hi @rohanprasadofficial, please rebase.


feat: scrap checkpoint and parallelize s3 uploads

Description of proposed changes
Uploads to s3 currently consume 6hr, which is about half of the total time taken by ingest.
The main reason for upload taking so long is that uploads are currently sequential.
This PR changes the Snakemake workflow to parallelize uploads. Uploads are done as soon as a file is ready.
I have also removed a checkpoint that does not seem to save any time: if I understand correctly the goal of the checkpoint is to avoid uploading a file if it is unchanged - but in our upload-to-s3 function we already check if the file is identical or not. If it is identical upload is skipped.
The checkpoint means uploads are deferred and the workflow is more complicated. It's cleaner snakemake without.
I tested the Snakefile in dry run mode and it seemed to work fine.
We can cut another hour with PR #354.
Test run:  nextstrain build --aws-batch --no-download --attach 85c4cf28-27bd-457d-9fc2-12b911c30849  .
Test worked well, shaving off 3 hours by parallelizing uploads.

New test runs:
nextstrain build --aws-batch --no-download --attach cf1de17f-21f0-4f33-9ee9-f00c1d8e89d6 .
nextstrain build --aws-batch --no-download --attach 268c8c4d-8f96-46a8-b1a3-84cbde65ccec  .

I have also removed a checkpoint that does not seem to save any time: if I understand correctly the goal of the checkpoint is to avoid uploading a file if it is unchanged - but in our upload-to-s3 function we already check if the file is identical or not. If it is identical upload is skipped.

I haven't looked through the changes yet, but the checkpoint is to check if we need to run Nextclade on any sequences. By removing the checkpoint, the pipeline will always run Nextclade even when it doesn't have new sequences.

I haven't looked through the changes yet, but the checkpoint is to check if we need to run Nextclade on any sequences. By removing the checkpoint, the pipeline will always run Nextclade even when it doesn't have new sequences.

Since we cache running Nextclade on no files takes 1 minute, which isn't something we need to avoid. The checkpoint complicates parallel uploads and in general complicates the workflow logic. Could it be that the checkpoint predates Nextclade caching?

Since we cache running Nextclade on no files takes 1 minute second, which isn't something we need to avoid. The checkpoint complicates parallel uploads and in general complicates the workflow logic. Could it be that the checkpoint predates Nextclade caching?

Ah, this might be a Nextclade v1 issue where an empty FASTA file caused an error?

Context:
https://github.com/nextstrain/ncov-ingest/pull/237#issuecomment-993674154
(Found using GitHub search: https://github.com/nextstrain/ncov-ingest/search?q=checkpoint+is%3Apr&type=issues)
The mutation summary is no longer a thing if I remember correctly, so it seems reasonable to remove the checkpoint.

I don't however understand most of the changes other than changing the checkpoint to the rule. In particular, the commented code and mv thingy. Some explanation might help a review. As well as commit messages better than
Update workflow/snakemake_rules/nextclade.smk

Thanks for the comment @ivan-aksamentov, I've added some context.
The main change is to start upload (and hence compression) of each file as soon as it's ready, and not do it sequentially in the end.
Compression is slow and there's no reason to do things sequentially. It takes an extra 3 hours to do it this way.
The pattern I use to parallelize is a standard Snakemake pattern.

@corneliusroemer could you add the context comments to the commit messages? It would be easier for future us to understand context in the commits rather than having to track down the comments in this PR.


Unable to fill Polygons

First of all, thank you very much for the incredible work you are doing with this package.
The problem I am experiencing is that I'm not able to fill the polygon with the color passed in the props even after setting style="fill".  The problem can also be reproduced from the "PointsDemo" example in the docs (setting style="fill" doesn't change anything).
import { Canvas, Points, vec } from "@shopify/react-native-skia";

const PointsDemo = () => {
  const points = [
    vec(128, 0),
    vec(168, 80),
    vec(256, 93),
    vec(192, 155),
    vec(207, 244),
    vec(128, 202),
    vec(49, 244),
    vec(64, 155),
    vec(0, 93),
    vec(88, 80),
    vec(128, 0),
  ];
  return (
    <Canvas style={{ flex: 1 }}>
      <Points
        points={points}
        mode="polygon"
        color="lightblue"
        style="fill"
        strokeWidth={4}
      />
    </Canvas>
  );
};

I don't know if I'm doing something wrong, but I couldn't find references on the docs on how to fill in the polygon.
Versions

React Native: 0.69.1
React: 18.0.0
React Native Skia: 0.1.136

I just came across this, and also am wondering if style="fill" would have done anything differently compared to ="stroke"?


[type:fix]fix  huaweicloud lts in  scheme.sql of h2

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4869 (fd93649) into master (594447e) will decrease coverage by 0.13%.
The diff coverage is n/a.

:exclamation: Current head fd93649 differs from pull request most recent head 72c6aae. Consider uploading reports for the commit 72c6aae to get more accurate results

@@             Coverage Diff              @@
##             master    #4869      +/-   ##
============================================
- Coverage     64.50%   64.37%   -0.13%
+ Complexity     8221     8211      -10
============================================
  Files          1161     1161
  Lines         34153    34153
  Branches       3061     3057       -4
============================================
- Hits          22031    21987      -44
- Misses        10367    10410      +43
- Partials       1755     1756       +1

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


flashplayer: <IP_ADDRESS>3 -> <IP_ADDRESS>4

Motivation for this change
The release notes are not updated yet.
https://helpx.adobe.com/flash-player/release-note/fp_32_air_32_release_notes.html
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Only tested with chromium and standalone versions.
Neither Firefox 71 nor 72 recognizes flash plugins. It is not shown in about:plugins.
See also #75436.

Is there a backport I should also merge?


Handle closure actions

Handle closure actions but also continue to handle string actions via sendAction [DEPRECATED].
Allows preventing the following deprecation warning:
VM2169 vendor.js:25085 DEPRECATION: You called <app@component:link-to::ember406>.sendAction("invokeAction") but Component#sendAction is deprecated. Please use closure actions instead. [deprecation id: ember-component.send-action] See https://emberjs.com/deprecations/v3.x#toc_ember-component-send-action for more details.
        at logDeprecationStackTrace (http://localhost:4200/assets/vendor.js:25071:29)
        at HANDLERS.(anonymous function) (http://localhost:4200/assets/vendor.js:25173:17)
        at raiseOnDeprecation (http://localhost:4200/assets/vendor.js:25095:17)
        at HANDLERS.(anonymous function) (http://localhost:4200/assets/vendor.js:25173:17)
        at invoke (http://localhost:4200/assets/vendor.js:25182:17)
        at deprecate (http://localhost:4200/assets/vendor.js:25154:34)
        at Class.sendAction (http://localhost:4200/assets/vendor.js:66419:49)
        at Class._sendInvokeAction (http://localhost:4200/assets/vendor.js:218714:14)
        at sendEvent (http://localhost:4200/assets/vendor.js:51258:20)

Hey @scottkidder, thank you for PR :heart:
Could you also fix the unit test related to this change?
not ok 15 Chrome 68.0 - [1 ms] - Unit | Mixin | link action: _sendInvokeAction sends `invokeAction` action
    ---
        actual: >
            null
        stack: >
                at Object.<anonymous> (http://localhost:7357/assets/tests.js:253:21)
                at processModule (http://localhost:7357/assets/test-support.js:5381:16)
                at module$1 (http://localhost:7357/assets/test-support.js:5402:4)
                at Module.callback (http://localhost:7357/assets/tests.js:243:21)
                at Module.exports (http://localhost:7357/assets/vendor.js:111:32)
                at requireModule (http://localhost:7357/assets/vendor.js:32:18)
                at TestLoader.<anonymous> (http://localhost:7357/assets/test-support.js:10369:11)
        message: >
            Expected 2 assertions, but 0 were run
        negative: >
            false
        Log: |
    ...

Sure thing @Kuzirashi! Thanks for the useful addon.

Thanks!!! I'll draft new release soon.

New release 0.1.2 which includes this change has been published. Thank you @scottkidder once again! :heart:

Thanks for the addon @Kuzirashi! :+1:


Button name "Refresh" is not visible behind window controls[BUG]

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
No response
Expected Behavior
No response
Steps To Reproduce
No response
Environment
- Mod Assistant Version: 2.4.0
- OS: w10
- Virtualized:

Anything else?
No response

If possible, I'll ask here: Crop Calendar is not implemented yet? Why is this in FSModAssistant? )

If possible, I'll ask here: Crop Calendar is not implemented yet? Why is this in FSModAssistant? )

Pull up a detail window of a map (double click), and scroll down. It should be right below the description (moddesc) stuff. I should have done a better announcement
Fwiw, I didn't send it to crowdin, but I did grab the month names and the crop names direct from the game files for all current translations

Clearly, I'm developing a vanilla map and I don't have any mod maps.
I have already translated months and crops on Crowdin. ) They are quite a bit different from those accepted in the game: for several months in Russian they are not reduced to three letters and somewhere added "ё".

Crop Calendar Example

Refresh Button Tooltip fixed

Between July and October there are two Septembers (in Russian and English):

OH!  ok, took a bit - but it's not a duplicate exactly, it was a poor job of copy-paste.  I set the translation of AUG to the word for September, and then failed to copy over September at all.  This is already fixed upstream when I run the next build.

Closing these as complete, i'm sure there will be more soon.  The "look inside the mod" feature is pretty cool, and bound to throw all sorts of issues around.


Short term fix for the static build of Fractal

What does it do?
Manually copy our theme CSS and Images into the appropriate dirs to make
the static build of Fractal work, previously it was missing out theme assets.
I spoke to the Fractal team and they think this should happen automatically,
which makes sense for theme assets to be auto-copied. In the mean time
this allows us to use static builds if we want to.
Also ensures that the fractal theme assets have been compiled from source
before running the static build. This would've only worked if the fractal
server had run and the compiled assets happened to be there.
Note: The static build only work if the static output is served from /, not
directly from file system, or hosted subdir. But would be fine on Heroku/GH Pages
What type of change is it?

New feature (non-breaking change which adds functionality)

Closing this as there's a proper fix in the works. I'll open a new PR


Fix loop variable reuse in database close

Required for all non-trivial PRs

[x] Rebased/mergable
[ ] Tests pass
[x] CHANGELOG.md updated

Fixes #6650

LGTM 👍

Confirmed locally to fix the data race in #6650. 👍


Fix errors when submodules are from different URL

Errors when submodule is from github.com and repo from gitlab both clone over ssh

@jessesuen I added ignore host key to the SSH now, it's now working in all use cases. It's a hard problem to kinda fix securely. Even if we allow then to add a list of keys and dynamically get it it's gonna be the same level of security as InsecureIgnoreHostKey.
I do believe this is an ok solution.


Added note to run Enable-CsComputer

Added note to run Enable-CsComputer after CsAuthConfig as per https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/1266 @AndreaBarr

@jambirk @CarolynRowe @MicrosoftHeidi Can I merge this?


Result of alluxio metadata calculation is incorrect

Alluxio Version:
What version of Alluxio are you using?
branch-2.3-fuse
Describe the bug
A clear and concise description of what the bug is.
After the fix #12315 , we've found two unexpected things happened to us.
~1. Results of all the alluxio command(e.g. alluxio fs du -sh / and alluxio fs ls -R /) are doubled.~ (Resolved)
For example, we mount a UFS(about 143.67GB and 1157 files in total) to Alluxio, and run the following commands:
bash-4.4# alluxio fs du -sh /
File Size     In Alluxio       Path
287.34GB      0B (0%)          /

bash-4.4# alluxio fs ls -R / | wc -l
2314

The result is twice as the actual number, while we expect to see the two are same.
2. When mounting a UFS with large amounts of files(about 92GB and 3.8million files), the result of alluxio fs du -sh is incorrect.
This is the result we got when running alluxio fs du -sh / on the UFS mentioned above:
File Size     In Alluxio       Path
168.58GB      0B (0%)          /

Divide 168.58GB by 2 due to problem 1, we got 84.29GB which is still inconsistent with the actual size of our UFS (about 92GB).
We also found that 84.29GB is exactly the same result we got before the fix, this may provide additional info about this problem.
bash-4.4# alluxio fs du -sh /
File Size     In Alluxio       Path
SLF4J: Failed toString() invocation on an object of type [java.util.ArrayList]
java.lang.OutOfMemoryError: Java heap space
	at java.util.Arrays.copyOf(Arrays.java:3332)
	at java.lang.AbstractStringBuilder.ensureCapacityInternal(AbstractStringBuilder.java:124)
	at java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:448)
	at java.lang.StringBuilder.append(StringBuilder.java:136)
	at java.lang.StringBuilder.append(StringBuilder.java:131)
	at java.util.AbstractCollection.toString(AbstractCollection.java:462)
	at org.slf4j.helpers.MessageFormatter.safeObjectAppend(MessageFormatter.java:304)
	at org.slf4j.helpers.MessageFormatter.deeplyAppendParameter(MessageFormatter.java:276)
	at org.slf4j.helpers.MessageFormatter.arrayFormat(MessageFormatter.java:230)
	at org.slf4j.impl.Log4jLoggerAdapter.warn(Log4jLoggerAdapter.java:463)
	at alluxio.AbstractClient.retryRPC(AbstractClient.java:372)
	at alluxio.client.file.RetryHandlingFileSystemMasterClient.listStatus(RetryHandlingFileSystemMasterClient.java:228)
	at alluxio.client.file.BaseFileSystem.lambda$listStatus$9(BaseFileSystem.java:274)
	at alluxio.client.file.BaseFileSystem$$Lambda$71/825249556.call(Unknown Source)
	at alluxio.client.file.BaseFileSystem.rpc(BaseFileSystem.java:531)
	at alluxio.client.file.BaseFileSystem.listStatus(BaseFileSystem.java:270)
	at alluxio.cli.fs.command.DuCommand.runPlainPath(DuCommand.java:94)
	at alluxio.cli.fs.command.AbstractFileSystemCommand.runWildCardCmd(AbstractFileSystemCommand.java:92)
	at alluxio.cli.fs.command.DuCommand.run(DuCommand.java:207)
	at alluxio.cli.AbstractShell.run(AbstractShell.java:137)
	at alluxio.cli.fs.FileSystemShell.main(FileSystemShell.java:66)
84.29GB       0B (0%)          /

To Reproduce
Steps to reproduce the behavior (as minimally and precisely as possible)

To reproduce problem 1, just launch an Alluxio master and mount any WebUFS to it. Either check the total size of UFS or list all the files in UFS can reproduce it.

To reproduce problem 2, launch an Alluxio master and mount an UFS with millions of files. Check the total size of it may reproduce it.

Expected behavior
A clear and concise description of what you expected to happen.
Returned results should be consistent with the actual number.
Urgency
Describe the impact and urgency of the bug.
Urgent
Additional context
Add any other context about the problem here.
This is an issue related to #12127.
@apc999

/ping @apc999

@apc999 Hi, here are some few steps for you to reproduce problem 1:

run Docker container with Alluxio Image (branch-2.3-fuse, commit id: 75a8e27) and login

docker exec -it alluxio-master /bin/bash

set ALLUXIO_JAVA_OPTS with some alluxio properties like:

export ALLUXIO_JAVA_OPTS=' -Dalluxio.user.metadata.cache.enabled=true -Dalluxio.master.hostname=localhost '

start Alluxio Master:

/entrypoint.sh master-only

Create some dir and file

alluxio fs mkdir /aaa && alluxio fs touch /aaa/bbb

list files

alluxio fs ls -R /

There should be two /aaa and /aaa/bbb file status in returned results.

@TrafalgarZZZ  thanks for the detailed instruction! I have created PR #12423 to fix this with unit tests added

Hi @apc999 Thanks for the fix! The fix did resolve problem 1, but problem 2 seems still unresolved, so would you mind maybe reopen this issue?

@TrafalgarZZZ my understanding is that problem 2 you described that the results are doubled has been resolved.
You were reporting the case where the result is no more doubled but just different (with some files missing). Am I correct?
If that's the case, I would suggest creating a new issue documenting the new behavior, but keep this issue closed.

@apc999 yes, just like what you said. I will close this and open a new issue later. Thanks for your help.


Added bracket matching pattern for Qute if section

Added bracket matching pattern for Qute if section

Fixes #490
Signed-off-by: Alexander Chen<EMAIL_ADDRESS>

Thanks @AlexXuChen !


Crash after ~20 seconds

Unity crashes after recording for more than ~20 seconds.
How to reproduce:

Open Test scene.
Set Record Length on camera to 60.
Unity crashes after around 20 seconds.

Unity version: 5.5.2f1

I paused this project for a while due to other tasks. I'll look into it when resuming the project. Thanks for reporting.

This is happening to me too. So far it seems like the ffmpeg process
doesn't close properly.
If you wait for it in a different thread Unity doesn't crash, but the
thread never exits.
On Fri, 24 Mar 2017 at 07:53, Keijiro Takahashi<EMAIL_ADDRESS>wrote:

I paused this project for a while due to other tasks. I'll look into it
when resuming the project. Thanks for reporting.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/keijiro/FFmpegOut/issues/1#issuecomment-288947205,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABmDsp4DgZzfD5SIpL_MeDRh9c7V8UTIks5ro2h4gaJpZM4Mnnm8
.

I can confirm as well

Hi all,
I guess that it crashed because ffmpeg.exe is missing. I removed it intentionally because it's too big (over 40MB) to add to the repository.
I'll add instruction about it to README later. Please reopen this issue even if it reproduces after adding ffmpeg.exe.

I am experiencing the issue with ffmpeg.exe being present. It actually starts the process (it's visible in the windows task manager) and exports short videos fine.

I can't reopen - but it definitely still happens with ffmpeg

Hmm, okay. Reopened.
I'm refactoring the whole code now. Please verify it after the overhaul.

I've refactored the entire code and added an installation note to README.
Please download the following executable and copy it into Assets/StreamingAssets/FFmpegOut/.
https://github.com/KeatsPeeks/ffmpeg-static/blob/master/bin/win32/x64/ffmpeg.exe
I think it's almost the same as you used, but I'd like to make sure that it's not an issue depending on the version of ffmpeg.

Thank you Keijiro for your fast response&fix, I will test it soon

Seems like more stable, however I am getting some funny results. Big percentage of the .mov s im producing are presenting a picture that is constantly offsetting on the x or y axis (I think depending on the sizing of the window). I am using the ffmpeg.exe you provided a link for. If you want I can share the .mov here

Thank you very much for the quick feedback. That's a big progress because I have a clue about that problem (funny constant offsetting). Could you provide information about your graphics card?

(updated my comment witht the error message that I missed at posting).
I have two 755Ms in a Lenovo Y510P, I have a desktop with an R9 serie if you want I can run a quick test on that too.

Thanks for the info. I guess that it's caused by a non-regular screen resolution. I added "Set Resolution" property to avoid this problem. Could you give a try at it?

yes, it did solve the running picture issue! however now during recording the there is no camera on the Display 1, is that intentional?

Yes. It grabs the output of the camera, so that the game view goes blank. It's a little bit difficult to resend the output to the same game view. I added a preview area to the inspector as a workaround.
Personally, I'm okay with this design because mainly I'm using this for non-interactive projects. What do you think about it? Any suggestion is welcomed.

By the way, I added a workaround for the problem. Now the game view keeps displaying during recording.

I have yet to test it with VR. I would render an additional camera that does not render to the eyes (openvr), so I guess recording that will have to work perfectly. I will get back to you with my findings.

I'm closing this issue. Please feel free to reopen (or poke with @keijiro ) when reproducing this with the latest version.


Chunks exist in my stats file but not in LOADABLE_REQUIRED_CHUNKS on client.

I have been trying the last few days to get code splitting working with SSR and after fixing a bunch of webpack/babel related issues (upgrading to webpack v4 & babel v7 with updated babel plugins), I am now facing an issue with the ChunkExtractor and loading of links/scripts.
In my routes file I have a loadable component
const Discover = loadable(() => import('../pages/discover.js'));

and I am seeing the the chunkName in my stats-loadable.json
"assetsByChunkName": {
    "main": [
      "main.d1605868d5f74e36b617.css",
      "main.d1605868d5f74e36b617.js"
    ],
    "pages-discover": "pages-discover.271ab8a69db56a13d8d5.js"
  },

But after running the example code from the documentation for the ChunkExtractor, I'm still not seeing the "pages-discover" chunk in the DOM when server side rendering.  The weird part is that I am seeing the main chunk being rendered though.
<script>window.__LOADABLE_REQUIRED_CHUNKS__ = [];</script>
<script async data-chunk="main" src="/static/main.d1605868d5f74e36b617.js"></script>

Here is the relevant server code I'm using for SSR:
 const AppComponent = () => (
          <ApolloProvider client={client}>
            <Provider store={store}>
              <I18nextProvider i18n={req.i18n}>
                <AppDelegate>
                  <StaticRouter location={req.url} context={context}>
                    <Route component={props => <Routes {...props} />} />
                  </StaticRouter>
                </AppDelegate>
              </I18nextProvider>
            </Provider>
          </ApolloProvider>
        );

const webExtractor = new ChunkExtractor({
          statsFile: webStats,
        });

        const jsx = webExtractor.collectChunks(<AppComponent />);

        let initialState = {};

        try {
          await getDataFromTree(jsx);
          initialState = client.extract();
        } catch (e) {
          console.log('catching getDataFromTree');
        }

        const html = renderToString(jsx);

        const scriptTags = webExtractor.getScriptTags();

        const linkTags = webExtractor.getLinkTags();

        const styleTags = webExtractor.getStyleTags();

Then I pass all these variables to a render function to load the content that is sent to the user.
Any help or insight would be much appreciated!

Hello @garrettsherm, I don't see anything wrong in your setup. Without being able to reproduce it, I can't help you. Can you provide a repro repository?

@neoziro Ok np I can't share this specific repo but I'm going to try and create another repo where I replicate the issue when I get time.

Outdated.

I have the same problem. Have you solved it?

I have the same problem. Have you solved it @neoziro

@zhoushuanglong please submit a repro repository and a new issue to describe your problem. Also please check that your setup is correct before submitting it.

The library is stable and widely used, if you experience this issue, it is probably a wrong setup.

@zhoushuanglong Sorry for the late response but I solved my issue a while back, and like @neoziro said it had to do with my webpack config being wrong. Instead of using @loadable/component as an external I was using @loadable/server (see https://github.com/smooth-code/loadable-components/blob/master/examples/server-side-rendering/webpack.config.babel.js).
Once I changed that it worked. Hope that can you help you out.


mkPopulateRequestsForPromotionEdges: clean up duplicate statements

These logging cases are already handled by getPromotionCandidates.
Also, remove a duplicate check of if dp.PqinDigestMatch ... within
getPromotionCandidates.

/approve
/lgtm


Wrong amount in Send actionsheet

I tried to send 5 DAI, but it displays my total balance under "Amount".

Vlad: could you include a test case for this too. The screen's too fragile


ARM templates publicly accessible

[Enter feedback here]
It's unclear what is meant by "publicly accessible" in the following statement on the page.
"If you currently have your templates in a GitHub repo or storage account, you run into several challenges when trying to share and use the templates. To deploy the template, you need to either make the template publicly accessible or manage access with SAS tokens."
Does it mean that the repositories have to be made public in order to deploy ARM templates stored in a repo?

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

ID: 37f85feb-ae78-c454-8c26-c21d474bbf7a
Version Independent ID: 769971a1-5891-dbde-dff8-9fee76780506
Content: Create & deploy template specs - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/template-specs.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@borgewi, thank you for the query.
It means that the template has to be made available by either of the following

If you are using Github repo to store the template-  make the repo public, or

If you are using Storage account, you can use Shared Access Token.

@AnuragSingh-MSFT
In the guide to deploying ARM templates from a pipeline it states that the repo can be either public or private. These documentations seem to contradict each other, or am I missing something here?

@borgewi, thank you for bringing it to our attention. This is being assigned to the content owner to review and makes updates accordingly.

@tfitzmac, could you review the following 2 links which seems to be containing contradicting information:

Why use template specs states that "To deploy the template, you need to either make the template publicly accessible or manage access with SAS tokens."
"Create Github repo" under "Tutorial: Continuous integration of ARM templates with Azure Pipelines" states that the repo can be either public or private.

If you get the raw URL on a template in a private repo, it will include an access token. The options are still the same. At deployment time, access to the template happens either through a public URL or a URL with a token. Template specs are a better option because you manage access through RBAC.
#please-close


Website Redesign

Depends on
What needs to be done before we can do this?

N/A

What success / done look like?
What does it look like for this project to be done?

New website launched

Requirements
What resources are needed for this project (e.g skills or funds)?

[x] Designer
[x] Copy Editor

RACI
Who is involved in this project? Use the RACI model to identify the roles

Responsible
@JosephMcarthur
Accountable
@chealsye
Consulted
@markmacgillivray, @chealsye, Jess
Informed
Team@

Timeline:
Comms and website drafts completed on a rolling basis but no later than Saturday, 24 September.

Copy to get done by Friday: Homepage, About main page, History, Team, Alumni, Press (doesn't have to change right now and can go up), Contact, Create a Request main

I think this is good enough to close now. We've done lots of the hard work here. Remaining stuff is well covered by things in to-doist and we have time allocated for it.


Fix the required styles on title fields

I noticed that in the latest release candidates the required asterisk wasn't appearing for the title fields.
Wagtail 2.5.1:

Wagtail 2.6rc1

There wasn't any reason other than that's where I tried and it fixed the issue I was having. I am more than happy to make the suggested change.

👌 thanks! Yes I think I’d rather do the other change, feels a bit less risky


Add README and config files for flexible/pubsub.

Also, remove the text describing how to create the topic and
subscription, because that no happens automatically.

This looks great! We should definitely add an e2e test for this, though.

Moved to https://github.com/GoogleCloudPlatform/php-docs-samples/pull/124


Set the environment to release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.849%

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
2351

Relevant Lines:
2707

💛  - Coveralls


Preview feature feedback: Prisma field reference support (fieldReference)

Please share your feedback about the fieldReference functionality released in v4.3.0 in this issue.

If you encounter a bug, please open a bug report in this repo.
If the feature is working well for you, please share this in a comment below or leave a 👍 on this issue.

If you have any questions, don't hesitate to ask them in the #prisma-client channel in the Prisma Slack.

Request for references across tables from @danshapir:
Assume we have 2 tables.
Accounts and Transactions
We have a field on the Accounts called LinkTime, a datetime.
We want to find all transactions where their createdAt is larger than the LinkTime of their parent account.
That's for the simple query, just comparison of fields from a join.
For the more complex scenario, but something we all do a thousand times using SQL, we create a dynamic property as part of the select (computed of multiple fields or any other logic, or maybe aggregated SUM/MIN) and we want to only fetch items that are larger or equal to it.

hope it can do it with cross models.
A.updatedAt > B.updatedAt

@matthewmueller with this new feature is the following valid? See #11646 for context!
// For the given class (with ID `some-class-id`):
// - Get what school it's in
// - Get all classes in that school in the same year as the given class
prisma.db.class.findUnique({
  select: {
    year: true,
    name: true,
    school: {
      select: {
        name: true,
        classes: {
          select: { name: true },
          where: { year: { equals: prisma.class.fields.year } }
        }
      }
    }
  },
  where: { id: `some-class-id` }
})

As far as I can tell, this doesn't seem to work with NOT and not. I get an error thrown about an invalid type:
  'Argument originalBlockId: Got invalid value prisma.block.fields.id on prisma.findManyBlock. Provided BytesFieldRefInput<Block>, expected BytesFilter or Bytes.',
  'type BytesFilter {',
  '  equals?: Bytes | BytesFieldRefInput',
  '  in?: List<Bytes>',
  '  notIn?: List<Bytes>',
  '  not?: Bytes | NestedBytesFilter',
  '}',

Any update for this feature?

It is in preview. You can use it. It is documented here: https://www.prisma.io/docs/reference/api-reference/prisma-client-reference#compare-columns-in-the-same-table

Thanks for the response! I meant to ask about using this feature for comparing columns in separate tables (as mentioned above). I guess this will be a separate issue/feature. Both are much needed.

I think I found a bug: https://github.com/prisma/prisma/issues/16715

Hum looks like fieldReference it's not working with enums
enum Locale {
  en
  ja
  es
  fr
 ...
}

  // ...

  localeDefault Locale
  locale        Locale

if I try to match those
.findMany({
    where: {
      locale: { equals: db.table.fields.localeDefault  },
    }
})

/*
Argument equals: Provided value prisma.table.fields.localeDefault of type LocaleFieldRefInput<Table> on prisma.findManyTable is not a Locale.
*/

Looks like a field reference is not a valid enum value, or it doesn't pass some form of validation.
If I do exactly the same query with 2 strings, it works fine (I created a view which cast my fields from enum to string)

How about updating the value of a field from a referenced field?
For example, for the table:
model Table {
  id String @unique
  limit Int
  remains Int
}

set value from the limit field to the remains field, if remains is greater than limit:
await prisma.table.updateMany({
  where: {
    remains: {
      gt: prisma.table.fields.limit
    }
  },
  data: {
    remains: prisma.table.fields.limit
  }
});

It would be very helpful to update a such case.

It looks like the value of a referenced field can only be used directly right now. But I would like to do something like this:
prisma.job.updateMany({
  data: {
    status: JobStatus.ABORTED,
    stoppedAt: new Date(),
  },
  where: {
    startedAt: {
      lt: Date.now() - prisma.job.fields.timeoutMs,
    },
    status: JobStatus.STARTED,
  }
});

Would it be possible to add support for such computations that make use of referenced fields?

Hum looks like fieldReference is not working with enums

@kefniark: Thanks for bringing this to our attention. It should be fixed in the next release.

How about updating the value of a field from a referenced field?

@aarbmx6s: We agree this is a valuable feature. While we don't plan on adding this feature just yet as we've got other priorities, feel free to persist this in a separate feature request so that it gets more visibility 🙏

As far as I can tell, this doesn't seem to work with NOT and not.

@tslater: It's hard for me to tell without A code snippet but the TS definition you've shared suggests to me that you're trying to do the following:
await prisma.model.findMany({
  where: { field: prisma.model.otherField }
})

But this is not supported (yet). Try using the explicit equals filter instead:
await prisma.model.findMany({
  where: { field: { equals: prisma.model.otherField } }
})

It looks like the value of a referenced field can only be used directly right now. But I would like to do something like this:

@livthomas: We agree this is a valuable feature. See similar issue here: https://github.com/prisma/prisma/issues/16715#issuecomment-1557103894. While we don't plan on adding this feature just yet as we've got other priorities, feel free to persist this in a separate feature request so that it gets more visibility 🙏

Simply adding fieldReference to my previewFeatures causes typescript to fail building my project with
RangeError: Maximum call stack size exceeded
    at recursiveTypeRelatedTo (/src/flike/node_modules/typescript/lib/typescript.js:66353:44)
    at isRelatedTo (/src/flike/node_modules/typescript/lib/typescript.js:65942:25)
    at eachTypeRelatedToType (/src/flike/node_modules/typescript/lib/typescript.js:66275:41)
    at unionOrIntersectionRelatedTo (/src/flike/node_modules/typescript/lib/typescript.js:66154:25)
    at structuredTypeRelatedToWorker (/src/flike/node_modules/typescript/lib/typescript.js:66549:34)
    at structuredTypeRelatedTo (/src/flike/node_modules/typescript/lib/typescript.js:66474:30)
    at recursiveTypeRelatedTo (/src/flike/node_modules/typescript/lib/typescript.js:66439:30)
    at isRelatedTo (/src/flike/node_modules/typescript/lib/typescript.js:65942:25)
    at isPropertySymbolTypeRelated (/src/flike/node_modules/typescript/lib/typescript.js:67205:24)
    at propertyRelatedTo (/src/flike/node_modules/typescript/lib/typescript.js:67248:31) {
  type: 'RangeError'

This is before making any other code changes.
Unfortunately I'm working with a large-ish project that makes it hard to distill this down into a minimal repro :confused:

@he-la Is there any way you could share your Prisma Schema with us, even privately if needed?

const workflowsToTrigger = await db.workflow.findMany({
    where: {
      builderId: auditLog.builderId,
    },
    include: {
      steps: {
        where: {
          workflowVersion: {
            equals: db.workflow.fields.version,
          },
        },
      },
    },
  })

I am trying to do something like this, find steps in a workflow that match the workflow's version number.
It would be great to be able to reference the parent as that's where I see the most benefit.
Type 'FieldRef<"Workflow", "Int">' is not assignable to type 'number | FieldRef<"WorkflowStep", "Int">'.   Type 'FieldRef<"Workflow", "Int">' is not assignable to type 'FieldRef<"WorkflowStep", "Int">'.     Type '"Workflow"' is not assignable to type '"WorkflowStep"'.

It looks like the value of a referenced field can only be used directly right now. But I would like to do something like this:
prisma.job.updateMany({
  data: {
    status: JobStatus.ABORTED,
    stoppedAt: new Date(),
  },
  where: {
    startedAt: {
      lt: Date.now() - prisma.job.fields.timeoutMs,
    },
    status: JobStatus.STARTED,
  }
});

Would it be possible to add support for such computations that make use of referenced fields?

I would also love to be able to do this.

Thank you everyone for the feedback 🙌🏼
We’re excited to announce that the fieldReference Preview feature is now stable and Generally Available. This means you can use this feature without the Preview feature flag in your Prisma schema.
We first introduced this feature in 4.5.0 to add the ability to compare columns on the same table. For example, the following query returns records where the quantity value is less than the warnQuantity of a product:
await prisma.product.findMany({
  where: {
		quantity: { lte: prisma.product.fields.warnQuantity }
	},
})

See Prisma 5.0.0 release notes.

Does this feature work for a deeply related relation as well?
For ex
model A {
  field1 Int @id
  field2 Int @unique
  field3 Int
  field4 String @db.char(5)
  B      B[]
}

model B {
  fieldX Int @id
  fieldY Int
  fieldZ Int

  rel A @relation(fields: [fieldY], references: [field2])
}```

Can I do

prisma.a.findMany({ where: { field4: x, B: { some: { fieldZ: prisma.A.field3 } } } })

Please open a discussion to ask this question. Thanks.

The best way to achieve this is with TypedSQL:

https://www.prisma.io/docs/orm/prisma-client/using-raw-sql/typedsql
https://www.prisma.io/typedsql


Create content card 15

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

Follow  figma design and to make the button use Button Design System
After building the component please raise a PR with a screenshot of the component and add the component in content-card-15.html.
If you need to use any icon please use it from Hero Icons
Good luck.

I would like to work on this.

I want to contribute

I will complete it by tomorrow.

I want to work on this issue please assign me this issue.

@RavindraP04 Yeah sure. Can you assign it under JWOC?


Failed methods do get an updated ddp response

To fix, look here:
https://github.com/mondora/asteroid/blob/master/src/core/asteroid-methods.js#L135
The updated promise of a call or apply call must always be resolved. It would probably be useful to resolve it with info on the success or failure of the call.

No longer relevant


MultiGPU finetuning

Hi,
Did you try using multi-GPU for training?
I am hoping to finetune your model with 1/10 of the learning rate.
Wondering what is the change needed if using 8 GPU.

Has anyone had any luck with Multi GPU training?
I installed NCCL 2 but the speed for 10 iterations is 20 seconds on 1 P100 and the same on 2 P100's and yet it looks like both are being used:
[0] Tesla P100-PCIE-16GB | 50'C,  98 % |  6434 / 16280 MB | madhavajay(6341M) root(82M)
[1] Tesla P100-PCIE-16GB | 53'C,  98 % |  6171 / 16280 MB | madhavajay(6161M)
Can anyone help?


[Broken] amcnetworks - Unable to extract media url

Checklist

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

Verbose log
U:\>c:\support\youtube-dlc.exe --verbose https://www.amc.com/shows/the-salisbury-poisonings/episodes/season-1-the-salisbury-poisonings-episode-1--48143
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.amc.com/shows/the-salisbury-poisonings/episodes/season-1-the-salisbury-poisonings-episode-1--48143']
[debug] Loading archive file None
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dlc version 2020.10.09
[debug] Python version 3.8.6 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg N-92986-gaceb9131c1, ffprobe N-92986-gaceb9131c1
[debug] Proxy map: {}
[AMCNetworks] season-1-the-salisbury-poisonings-episode-1--48143: Downloading webpage
ERROR: Unable to extract media url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dlc -U  to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "youtube_dlc\YoutubeDL.py", line 830, in extract_info
  File "youtube_dlc\extractor\common.py", line 532, in extract
  File "youtube_dlc\extractor\amcnetworks.py", line 66, in _real_extract
  File "youtube_dlc\extractor\common.py", line 1010, in _search_regex
youtube_dlc.utils.RegexNotFoundError: Unable to extract media url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dlc -U  to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.

U:\>c:\support\youtube-dlc.exe --verbose https://www.wetv.com/shows/life-after-lockup/episodes/season-3-threes-a-crowd--48289
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.wetv.com/shows/life-after-lockup/episodes/season-3-threes-a-crowd--48289']
[debug] Loading archive file None
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dlc version 2020.10.09
[debug] Python version 3.8.6 (CPython) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg N-92986-gaceb9131c1, ffprobe N-92986-gaceb9131c1
[debug] Proxy map: {}
[AMCNetworks] season-3-threes-a-crowd--48289: Downloading webpage
ERROR: Unable to extract media url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dlc -U  to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "youtube_dlc\YoutubeDL.py", line 830, in extract_info
  File "youtube_dlc\extractor\common.py", line 532, in extract
  File "youtube_dlc\extractor\amcnetworks.py", line 66, in _real_extract
  File "youtube_dlc\extractor\common.py", line 1010, in _search_regex
youtube_dlc.utils.RegexNotFoundError: Unable to extract media url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dlc -U  to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.

Description
Unable to extract media url.  Both of the listed URLs should be available without login.

the first time i tried this link - https://www.amc.com/shows/the-salisbury-poisonings/episodes/season-1-the-salisbury-poisonings-episode-1--48143 with the Firefox addon called the stream detector, it got the m3u8 files. closed the tab and reopened the tab to attempt  to download the episode it  now says it has DRM protection.

Cannot really test anything here. Need a vpn for that.
There are also non drm files available it seems
https://amc350885sec-a.akamaihd.net/thumbnail/AMC_Networks/1012/663/pre_start_preferred_formats__v1_0_2.min.js

i was able to download & playback this episode just fine  https://www.amc.com/shows/the-salisbury-poisonings/episodes/season-1-the-salisbury-poisonings-episode-1--48143
this episode - https://www.wetv.com/shows/life-after-lockup/episodes/season-3-threes-a-crowd--48289 requires a tv provider

These appear to be free and available (as of post time).
https://www.amc.com/shows/fear-the-walking-dead/episodes/season-6-the-end-is-the-beginning--48698
https://www.wetv.com/shows/love-after-lockup/episodes/season-3-2-put-a-sweater-on-and-chill--48584
Yes, I too, can get the m3u8 in the browser, but thats not as easily automated.

Cannot really test anything here. Need a vpn for that.
There are also non drm files available it seems
https://amc350885sec-a.akamaihd.net/thumbnail/AMC_Networks/1012/663/pre_start_preferred_formats__v1_0_2.min.js

Is there some output I can get for you?  I am in the US and have a subscription.


Issue 969

Description:
This just removes the unused dependency that's making a distracting notification pop up during linking protobufs and such.
Related issue(s):
#969
Fixes #
removed dependency
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Can you rebase this onto develop. I did force push a few times :)

I think I can easier just create a new pr with the needed change. see https://github.com/hashgraph/hedera-sdk-js/pull/1010


Expose spotify client id and client secret in config.yml

[x] Using latest version as provided on the master branch
[x] Searched for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug
[x] Feature Request
[ ] Question
[ ] Other

Description
Just like we allow users to set a custom Youtube Data API key, we should also allow users to set their own Spotify client id and client secret in config.yml.

Closed by #501.


height option?

Would it be possible to specify the height of the select box?
Right now I'm using it together with bootstrap but it seems much smaller than the bootstrap inputs / selects.

You're going to have to handle styling of Select2 through CSS. Though for Bootstrap I'd personally recommend the Select2 Bootstrap theme
https://github.com/select2/select2-bootstrap-theme


Upgrade dependency

Upgrade<EMAIL_ADDRESS>to version 3.1.3

Added by OX Security


DE_de translation error, missing quote

You have cancelled the order. Please try again", "Sie haben den Auftrag storniert. Bitte versuchen Sie es erneut"
should be
"You have cancelled the order. Please try again", "Sie haben den Auftrag storniert. Bitte versuchen Sie es erneut"
see char 1

Hi Sean,
Thanks for letting us know. You can always do the changes by yourself and do a pull request.
Regards,
Rik
Adyen

sorry that is true; was almost to small an error to report ....
On Thu, Aug 27, 2015 at 10:53 AM, Rik ter Beek<EMAIL_ADDRESS>wrote:

Hi Sean,
Thanks for letting us know. You can always do the changes by yourself and
do a pull request.
Regards,
Rik
Adyen
—
Reply to this email directly or view it on GitHub
https://github.com/Adyen/magento/issues/393#issuecomment-135346108.


Error running with api-version=2: unknown flag: --api-version

Delve version:

$ dlv version
Delve Debugger
Version: 0.11.0-alpha
Build: v0.11.0

Go version:

$ go version
go version go1.6.2 darwin/amd64

What operating system and processor architecture are you using?

OSX 10.11.2
15.2.0 Darwin Kernel Version 15.2.0: Fri Nov 13 19:56:56 PST 2015; root:xnu-3248.20.55~2/RELEASE_X86_64 x86_64

What did you do?

Tried running this, as instructed by https://github.com/derekparker/delve/tree/master/Documentation/api :
dlv debug --headless --api-version=2 --log --listen=<IP_ADDRESS>:8181

What did you expect to see?

Building and debugging of the program.

What did you see instead?

`Usage:
dlv debug [flags]
Global Flags:
--build-flags="": Build flags, to be passed to the compiler.
--headless[=false]: Run debug server only, in headless mode.
--init="": Init file, executed by the terminal client.
-l, --listen="localhost:0": Debugging server listen address.
--log[=false]: Enable debugging server logging.
Error: unknown flag: --api-version`

My guess is that you need to update your delve
$ cd $GOPATH/src/github.com/derekparker/delve/
$ git pull
$ CERT=dlv-cert make install

Maybe - this was installed with https://github.com/derekparker/delve/blob/master/Documentation/installation/osx/install.md instructions (via homebrew). But the version seems to be the same as the latest version tag.

We haven't made a new release in a while.

I'll try when I get a chance, thanks. Maybe a new release is warranted, given at least this feature (new api version)?

That works, thanks

@aarzilli or whoever, how does one update their delve? I"m receiving the following error when I try to set the api version:
unknown flag: --api-version
My dlv looks out of date:
$dlv version
Delve Debugger
Version: 0.11.0-alpha
Build:

But I can't figure out how to upgrade it. I've tried go get -u github.com/go-delve/delve/cmd/dlv and go get -u github.com/derekparker/delve/cmd/dlv, but neither worked. I also tried the suggestion to make install:
$ git clone https://github.com/go-delve/delve.git $GOPATH/src/github.com/go-delve/delve
$ cd $GOPATH/src/github.com/go-delve/delve
$ make install

But that appears to have no effect:
$ dlv version
Delve Debugger
Version: 0.11.0-alpha
Build:


[PE-D] UG not well-formatted

I feel that the UG is too cluttered as in too much information under a section e.g. Adding an entry: add section as shown in the image below. The various subsections of the add command i.e., module, group, professor, student, event and meeting can be extracted and formatted as a proper section (like a subsection for adding a module including its examples) using the markdown so that the information shown to the readers is not that hard to read/find.

Labels: severity.Medium type.DocumentationBug
original: flairekq/ped#11

currently fixing in another issue


Update from the discussion on Issue #1

Points that need to improve:

Change package name from +neuropixels to +npxtoolkit
ClassDiagram 1 - 1...0 relationship should be 1 - *, also updated UsecaseDiagram to resolve some confusion
Add an Assembly class as the top level object, also changed Pipeline to Session in order to avoid concept conflicts
Now the object level will be Assembly->Session->Stage->JobBase
Change function assemble() to add_session() to avoid confusion
Include log4m at +npxtoolkit/+internal/+logging/
Will work on the 'non-fork' repo

@dimitri-yatsenko @iamamutt Please take a look when you guys are available, I made those changes followed by the discussion in Issue 1


mark /comp/README.md as un-owned

Currently all new comps implicitely need a review from ASC, even if created in existing domains owned by other teams.
This comes from the requirement to edit /comp/README.md which is owned by ASC.
This PR proposes to mark that specifc file as un-owned to avoid that bottleneck. If this is intended and desirable it's also fine to reject it.

Closing for now since based on the above


System.IO.FileNotFoundException

When to call
using (var db = new LiteDatabase(SettingsManager.DbFilePath))
an error occurs
A first chance exception of type 'System.IO.FileNotFoundException' occurred in mscorlib.dll
but despite that, everything works

Hi @ambyte? This DbFilePath is in local machine? It´s a valid path? Are directory exists? When you open a database that do not exists datafile, LiteDB creates a new data file.

Hi
I understood the reason.
In the constructor LiteDatabase in line this.Recovery.TryRecovery() LiteDb try open journal file, but it is missing, and in class RecoveryService in method OpenExclusiveFile this error appears.
But i don`t disable journaling.

in my case directories did not exist. so
        var fulldir = System.IO.Path.GetDirectoryName(dbFilePath);
        System.IO.Directory.CreateDirectory(fulldir);
        using (var db = new LiteDatabase(dbFilePath))
       {


Fetch abi from AnyABI as first source and leave Etherscan as fallback

Working on this draft to fetch abi from AnyABI API, to not have to deal with Etherscan API limits.
Current Etherscan abi fetch stays as fallback option in case AnyABI fails to fetch the abi.

[x] Create function to fetch abi from AnyABI
[x] Set fetchContractABIFromAnyABI as first option in ContractDetailPage
[x] Set fetchContractABIFromAnyABI as first option in Homepage

In https://github.com/BuidlGuidl/abi.ninja/pull/35/commits/c8b4af94e84f398ee21c647cdde43358fb4e7504 I've also added the abi load into the global state. The reasoning behind it was that we were already fetching Etherscan API to know if the abi was available, so with that change we save a fetch in ContractDetailPage, but probably duplicating the code in both pages isn't the way to go, let me know what do you think 🙌
Closes #26

Thanks for the awesome review @portdeveloper and @technophile-04 !! Learnt a lot from them ♥🙌

Tysm !! Looking great ! Just pushed a couple of commits to fix a bug where if the errror was thrown from  fetchContractABIFromAnyABI it would not fallback to etherscan instead it was going directly inside catch block. checkout in below video I blocked the anyAbi domain (just to simulate error) ideally it should have been fallback to etherscan but instead it just throwed an error :

Ouch sorry!! During my tests I did the error handling totally wrong and was having the opposite effect, so deleted it and thought it was safe, because I prepared a wrong failure test, so it was working fine 😓. Thanks for investing so much time in showing the right test and the way to place the catch block correctly ♥

Ouch sorry!! During my tests I did the error handling totally wrong and was having the opposite effect, so deleted it and thought it was safe, because I prepared a wrong failure test, so it was working fine 😓. Thanks for investing so much time in showing the right test and the way to place the catch block correctly ♥

No worries at all !! Lol even I am bad at getting proper error handling control flow correctly it gets tricky  sometimes
Tysm @Pabl0cks and @portdeveloper for review, merging this 🙌


Is there a built-in api to convert the schema with $data reference

like:
const schema = {
  type: 'object',
  properties: {
    test: {
      type: 'number'
    },
    target: {
      type: 'number'
      minimum: { $data: '/test' }
    }
  }
};

const data = {  test: 5 };

func(schema, data);
// output:

{
  type: 'object',
  properties: {
    test: {
      type: 'number'
    },
    target: {
      type: 'number'
      minimum: 5
    }
  }
}

Thanks

@Airkro no, ajv doesn't transform schemas, the compiled validation function uses actual runtime data - it would have been very inefficient otherwise if the schema needed to be re-compiled every time.
If you need to do schema transformation like you describe you can use json-schema-traverse package to iterate through all the subschemas and replace $data reference in all the keywords.


Hide production publish btn, prohibit metadata editing for non-admin editors

…rom editing metadata
Pre-Flight checklist

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

What are the relevant tickets?
Fixes #696
What's this PR do?

Hides the production->publish button from non-admin site editors
Tweaks backend permissions class for WebsiteContent to prevent non-admins from creating/updating objects of type sitemetadata

How should this be manually tested?

Create a new website
Create metadata for the website
Add another user as a site collaborator of type 'Editor'
Log in as that other user
You should not see 'Metadata' listed in the left sidebar menu
You should still be able to create/edit pages and resources
Click the 'Publish' button, you should only see the option for "Staging"
Log back in as the website owner, change the other user's role from 'Editor" to "Admin"
Log back in as the other user.  You should now be able to edit the metadata and publish to both staging and production.

Codecov Report

Merging #702 (68598c2) into master (8bef5e8) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #702   +/-   ##
=======================================
  Coverage   94.37%   94.38%
=======================================
  Files          82       82
  Lines        2330     2333    +3
  Branches      485      487    +2
=======================================
+ Hits         2199     2202    +3
  Misses        129      129
  Partials        2        2

Impacted Files
Coverage Δ

static/js/util/factories/websites.ts
100.00% <ø> (ø)

static/js/components/PublishDrawer.tsx
93.65% <100.00%> (+0.10%)
:arrow_up:

static/js/components/SiteSidebar.tsx
100.00% <100.00%> (ø)

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 8bef5e8...68598c2. Read the comment docs.


improve doc of stack address

@Thykof can you also create a documentation page with the whole explanation on how the stack works ?

@Thykof can you also create a documentation page with the whole explanation on how the stack works ? (not just the documentation of those 3 functions)

@damip don't you think this documentation should be on Massa's read the doc ?

@gregLibert here is the doc https://docs.massa.net/en/latest/web3-dev/smart-contracts/address-stack.html


Change project name

Ok, I think "Blink" has a conflict with Google‘s browser kernel name “Blink”.
So, we should change project name.

“Blink”  -> "Tlink" -> "Tink"


Added wasp-lang to user showcase.

Motivation
Adding a project (Wasp) that is built with Docusaurus v2.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Related PRs

Thanks
I think you interverted Wasp and Wisdom no?
The PR is failing but it's not your fault, will fix this asap.

The PR issue on our side should be fixed.
However still seem to have interverted 2 sites's data, and the uploaded filename is wrong

Just updated your PR, now ready to merge

@slorber thanks a lot for fixing it! I was just about to do it and then noticed you've just done it :)


affected/package:   https://github.com/golang/website

websit
How to pop up the survey pop-up box at the bottom right of the website?

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Adding animation support to parallax

Description

adds animation support to our parallax API
Fix a bug which made the layer not respect the alignment correctly

https://user-images.githubusercontent.com/835641/120415341-c02e3680-c331-11eb-9709-a03244745ede.mov
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors and are passing (See Contributor Guide).
[ ] My PR does not decrease the code coverage, or I have a very special case and explained on the PR description why this PR decreases the coverage.
[ ] I updated/added relevant documentation (doc comments with ///) and updated/added examples in doc/examples.
[x] I have formatted my code with ./scripts/format.sh and the Flame analyzer (./scripts/analyze.sh) does not report any problems.
[x] I read and followed the Flame Style Guide.
[x] I have added a description of the change under [next] in CHANGELOG.md.
[ ] I removed the Draft status, by clicking on the Ready for review button in this PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flame users to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md).
[ ] No, this is not a breaking change.

Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR.

I think you must have forgot to push some files? Can't find ParallaxImageData and ParallaxData.
EDIT: Nvm, found them, the file was hidden because of "too many" changes.

That file is getting a little big, maybe we should start splitting it?


Move Pagermon Server into the root folder

Description
This completes the task of separating the repos and allows us to move forward with automated testing
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
Nothing needs to be tested - its simply a dir change in the repo
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have updated changelog.md with changes made

+1 We'll do the same to the v0.3.0 branch and knex branch once master is completerized


java.lang.UnsupportedOperationException: com.alibaba.com.caucho.hessian.io.MapDeserializer

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.7
Operating System version: xxx
Java version: 1.8

Steps to reproduce this issue
消费者请求服务提供者，报错 ，版本均为2.7.7 ，两边PO均实现序列化接口，并且没有不可序列化的字段
Pls. provide [GitHub address] to reproduce this issue.
Expected Behavior
正常
Actual Behavior
Caused by: java.util.concurrent.ExecutionException: org.apache.dubbo.remoting.RemotingException: Fail to decode request due to: java.lang.UnsupportedOperationException: com.alibaba.com.caucho.hessian.io.MapDeserializer@31cf246f
java.lang.UnsupportedOperationException: com.alibaba.com.caucho.hessian.io.MapDeserializer@31cf246f
If there is an exception, please attach the exception trace:
Just put your stack trace here!

传输对象里边如果有枚举，枚举的名字不能带下划线分割 否则报错

Could you pls provide a demo?


Save core tools logs to output log file

This fixes https://github.com/NVIDIA/spark-rapids-tools/issues/735
This PR introduces improvements to the tools by enhancing the handling of log4j configuration files and saving the logs to the output file for future analysis if needed.
Changes

Added log4j.properties to capture and configure logging.
Updated pyproject.toml to include .properties files in the package data to ensure log4j.properties is available during execution.
Added a function modify_log4j_properties to dynamically create a temporary log4j.properties file with a unique log file path. The unique log file path is to save the file in output_directory. Added cleanup method to delete the temporary file after java command execution.
Updated ToolUserArgModel to initialize the path to the log4j.properties file, making it configurable via the tool arguments.
If the user provides --verbose argument, then the current PR will display stdout and stderr both on console and capture in logfile as well.

Testing
spark_rapids qualification --verbose --platform=onprem  --eventlogs=<PATH_TO_EVENTLOGS>

core tools logs are saved in : output_directory/rapids_4_spark_qualification_output/rapids_4_spark_qualification_stderr.log
Example: `qual_20240808200632_fF9721ba/rapids_4_spark_qualification_output/rrapids_4_spark_qualification_stderr.log
Sample output:
2024-08-08 15:34:11 main WARN NativeCodeLoader:60 - Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
2024-08-08 15:34:11 main WARN AutoTuner:72 - No file found for input workerInfo path: ./worker_info.yaml
2024-08-08 15:34:11 main INFO PlatformFactory:60 - Using platform: dataproc
2024-08-08 15:34:11 main INFO PluginTypeChecker:60 - Trying to read operators scores with platform: dataproc
2024-08-08 15:34:11 main INFO Qualification:60 - Threadpool size is 3
2024-08-08 15:34:11 main INFO ToolTextFileWriter:60 - RAPIDS Accelerator for Apache Spark's Build/Runtime Information output location: /home/nartal/spark-rapids-tools/user_tools/qual_20240808223406_24290be1/rapids_4_spark_qualification_output/runtime.properties
2024-08-08 15:34:11 main INFO RuntimeUtil:60 -
RAPIDS Accelerator for Apache Spark's Build/Runtime Information
-- listing properties --
runtime.spark.version=3.5.0
build.spark.version=3.5.0
build.hadoop.version=3.3.6
build.java.version=1.8.0_402
build.version=24.06.2-SNAPSHOT
build.scala.version=2.12.15

2024-08-08 15:34:11 Qualification-0 INFO QualificationAppInfo:60 - Start Parsing Event Log: file:/home/nartal/eventlogs/SingleEventlog1
2024-08-08 15:34:12 Qualification-0 WARN EventUtils:72 - ClassNotFoundException while parsing an event: DBCEventLoggingListenerMetadata
2024-08-08 15:34:39 Qualification-0 INFO QualificationAppInfo:60 - Total number of events parsed: 406432

Added a function modify_log4j_properties to dynamically create a temporary log4j.properties file with a unique log file path. The unique log file path is to save the file in output_directory. Added cleanup method to delete the temporary file after java command execution.

Why is this needed?

If the user provides --verbose argument, then the current PR will display stdout and stderr both on console and capture in logfile as well.

What is behavior without verbose?

I think we should come up with a different name for the log file rapids_4_spark_tools_jar.log doesn't easily stand out to me among all the other files.   Is this both stdout and stderr?  putting stdout and/or stderr in the name of it would be more obvious.

Thanks @cindyyuanjiang for the review. I have addressed review comments. PTAL.


Remove feature.coordinates

We are currently outputting both feature.coordinates and feature.geometry.coordinates. The former is the result of layer.feature(i).loadGeometry(); which is a custom structure of x,y in tile coordinates. The latter is geojson coordinates. I think we should output only the latter by removing this line:
@@ -69,7 +78,6 @@ function readTile(args, buffer, callback) {
         var layer = tile.layers[layerID];
         for (var i = 0; i < layer.length; i++) {
             var feature = layer.feature(i).toGeoJSON(args.x, args.y, args.z);
-            feature.coordinates = layer.feature(i).loadGeometry();
             collection.features.push(feature);
         }
     });

/cc @jfirebaugh who might remember if there was a reason for feature.coordinates = layer.feature(i).loadGeometry();.

No reason that I'm aware of.

This was resolved in #12 and has been released in<EMAIL_ADDRESS>


mutations return data: null

Hello, I wanted to use the endpoint create under user.
I have added a button on the homePage. When I click the button, the following code will be executed.
const createUser = trpc.user.create.useMutation()

const handleCreateUser = async () => {
    if (!user?.email) {
      toast.show('Please sign in first')
      return
    }

    createUser.mutate(
      {
        email: user.email,
        id: user.id,
      },
      {
        onSuccess: (res: any) => {
          console.log('res', res)
        },
      }
    )
  }

In my browser console I can see that the req was successful, but data from the res is null and I don't see any console.logs in my terminal.

Thanks for your help!

after restarting my pc everything works

after restarting my pc everything works


auth: add ldap support

Looks like a good start, thanks.

Codecov Report

Merging #13 into master will decrease coverage by 1.27%.
The diff coverage is 51.87%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
- Coverage   59.35%   58.07%   -1.28%
==========================================
  Files          10       11       +1
  Lines        1176     1319     +143
==========================================
+ Hits          698      766      +68
- Misses        353      411      +58
- Partials      125      142      +17

Impacted Files
Coverage Δ

pkg/api/ldap.go
34.52% <34.52%> (ø)

pkg/api/controller.go
75.67% <50%> (-6.68%)
:arrow_down:

pkg/api/config.go
70.83% <63.15%> (-29.17%)
:arrow_down:

pkg/api/auth.go
73.26% <76.47%> (+0.25%)
:arrow_up:

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 be7ce56...6295e0c. Read the comment docs.


docker run has error: standard_init_linux.go:220: exec user process caused "exec format error"

Hello,I use docker to run hue:
docker pull gethue/hue:latest
docker run -it -p 8888:8888 gethue/hue:latest

But has error:
standard_init_linux.go:220: exec user process caused "exec format error"
libcontainer: container start initialization failed: standard_init_linux.go:220: exec user process caused "exec format error"

My runtime platform is arm64, is this images built for x86_x64 platform only?

Weird, this is the image used by https://gethue.com. Here is the docker file https://github.com/cloudera/hue/blob/master/tools/docker/hue/Dockerfile

Weird, this is the image used by https://gethue.com. Here is the docker file https://github.com/cloudera/hue/blob/master/tools/docker/hue/Dockerfile

I try to build the image based on the Dockerfile, but the same error appeared in step 4:
Step 4/18 : RUN apt-get update -y && apt-get install -y   build-essential   libkrb5-dev   libmysqlclient-dev   libssl-dev   libsasl2-dev   libsasl2-modules-gssapi-mit   libsqlite3-dev   libtidy-dev   libxml2-dev   libxslt-dev   libffi-dev   libldap2-dev   libpq-dev   python-dev   python-setuptools   libgmp3-dev   libz-dev   software-properties-common   curl   git   rsync   sudo   maven   gcc   swig   xmlsec1   libxmlsec1-openssl   krb5-config   krb5-user   && rm -rf /var/lib/apt/lists/*
 ---> Running in 9f38eae41544
standard_init_linux.go:220: exec user process caused "exec format error"
libcontainer: container start initialization failed: standard_init_linux.go:220: exec user process caused "exec format error"

What shoud I do now?

disappointed

This is too difficult. I met the same problem.
By the way,how to use Dockerfile

@Mashark-star @hui-250 @tulingfeng Sorry for the slow progress on this. We thought that the issue was resolved but it seams that it is still a problem. We are gonna look into this. Thank you for reporting.


Setting "suggestedMax" seems to be ignored for charts with actual maxes near zero

Using v 2.2.1 of charts.js.
I had an earlier issue where I had to figure out how to configure a set of related charts with the same Y-Axis range.  They all have slightly different data, but they're all in the same domain, so I wanted them to have the same range.  I learned that I needed to set the "suggestedMax" (and "suggestedMin") properties.  This appeared to fix my problem.
However, I've since noticed that there are some cases where the rendered chart appears to ignore the value I've set.
I'm attaching one picture that shows this happening with one chart among several on the same page.

Looking at the four charts whose title starts with "Hourly", all of them have a max Y value of 400, except for the last one, which shows a max Y value of 300.  In the actual data, the max value was 242, and I verified with the debugger and "console.log()" calls that I'm setting "suggestedMax" to "242" for all four charts.
It's not good form to create an issue without providing any code, so I will, but I can't provide a fully reproducible test case, so this code might not be that helpful.
The following is an excerpt of my HTML:

        <canvas id="last30Minutes-chart-{{dataCenterData.dataCenter}}"
            class="chart chart-line" chart-data="dataCenterData.data"
            chart-labels="dataCenterData.labels"
            chart-series="dataCenterData.series"
            chart-colors="dataCenterData.colors"
            chart-options="dataCenterData.options"></canvas>

        <canvas id="hourly-chart-{{dataCenterData.dataCenter}}"
            class="chart chart-bar" chart-data="dataCenterData.data"
            chart-labels="dataCenterData.labels"
            chart-series="dataCenterData.series"
            chart-colors="dataCenterData.colors"
            chart-options="dataCenterData.options"></canvas>

    <canvas id="aggregate-chart" class="chart chart-bar"
        chart-data="aggregateData.data"
        chart-labels="aggregateData.labels"
        chart-series="aggregateData.series"
        chart-colors="aggregateData.colors"
        chart-options="aggregateData.options"></canvas>

The following is a function I use to configure the "suggested" values for each chart data block:
function determineAndSetYAxisRange(dataList) {
    var minVal  = -1;
    var maxVal  = -1;
    console.log("dASYAR: first title[" + dataList[0].options.title.text + "]");
    angular.forEach(dataList, function (dcData) {
        console.log("Processing dataList. title[" + dcData.options.title.text + "]");
        angular.forEach(dcData.data, function(countsArray) {
            angular.forEach(countsArray, function(count) {
                if ((minVal == -1) || (count < minVal)) {
                    minVal  = count;
                }
                if ((maxVal == -1) || (count > maxVal)) {
                    maxVal  = count;
                }
            });
        });
        console.log("After processing dataList. minVal[" + minVal + "] maxVal[" + maxVal + "]");
    });
    console.log("After all dataLists. minVal[" + minVal + "] maxVal[" + maxVal + "]");
    angular.forEach(dataList, function (dcData) {
        dcData.options.scales.yAxes = [{
                                            ticks: {
                                                suggestedMin: minVal,
                                                suggestedMax: maxVal
                                            }
                                        }];
    });
}

Note that this app updates its charts every minute or so to reflect changing circumstances, and an hour or so after I took the above image, the latest renderings showed all of the charts using the same max Y-Axis value (up to 1000 when I checked it).  So it's clear this symptom is very data-dependent. The actual maximum value was 610 at this point in time.

@davidmichaelkarr I think what happens is that the number of ticks is auto determined for the linear axis. The charts that go up to 400 have rotated x labels which makes the smaller vertically. This means they cannot support as many grid lines and so the size of each division increases. The next division up from 100 is 200. I'm not sure what the best approach is in this scenario.
@simonbrunel @tannerlinsley @zachpanz88 any thoughts?

Curious that the "last 30 minutes" chart just above the "hourly" one that only goes up to 300 doesn't have the same symptom, but I understand what you're saying.
I suppose the only way I could guarantee perfect uniformity is to ensure that all the charts in the same domain have the exact same x-axis points. In this situation, that isn't worth the trouble.
It would be good to have a solution for this, but I understand if it's not practical.

yeah. It's probably right at the limit.
There is a ticks.maxTicksLimit but it sets an upper limit on the number of ticks.
You could actually overwrite the function that generates the number of ticks. The number of ticks is done with Math.min(11, Math.ceil(height / (2 * tickFontSize))). This is just an approximation which you could change. Maybe 1.5 is a better multiplier.
See: https://github.com/chartjs/Chart.js/blob/master/src/scales/scale.linear.js#L135

What if there was a way to force the X axis labels to be slanted?  Wouldn't that produce exactly what I'm looking for, and at the same time make the X axis labels render the same way in each chart?

there is a ticks.minRotation setting which defaults to 0. You could try setting something other than that. The value is in degrees and needs to be in the range [0, 90].

This is looking good. I need to see it in a few more variations, but it seems to work.

This works for me.  This seems like something that would be good to elaborate on a bit in the doc.

Good call, I will tag this as documentation so that we can add something.

Fixed in #3751


defaultTabIndex turns things a but funny

I'm quite happy with tabbed-card, but yesterday I felt to make use of
Options:
defaultTabIndex: 1
it's main function works as declared but I got 2 gauges sitting on TAB 0 and since I start with default tab being 1 what happens is that these gauges don't show values any more. Simply the gauge but the name for example 'temperature' but no value nor the unit.
Any idea?
All get's cured if I remove the defaultTabIndex or turn this to 0.

Well, duplicated the card, on the detail tag is a plotly graph. So I simply swapped the code so that Klima is the first one on the left side card. And in there I defined defaultTabIndex: 1 so that I had to manually select the left tab named "KLIMA".
As you see the value is gone. Sometimes it gets lost on both gauges, something one of them remains there.
If I wait for a while the missing value/unitofmeasurement comes back.
Looks to me as if there's a missing fresh or such like fetching the last value. Might also be a gauge problem itself, who knows.
pic and the beginning code snippet .....

type: custom:tabbed-card
options:
  defaultTabIndex: 1
tabs:
  - attributes:
      label: Klima
    card:
      type: horizontal-stack
      cards:
        - type: gauge
          entity: sensor.esp32computerraum_bme680_temp
          name: Temperatur
          severity:
            green: 0
            yellow: 0
            red: 0
        - type: gauge
          entity: sensor.esp32computerraum_bme680_humi
          name: Luftfeuchte
  - attributes:
      label: Detail
    card:
      type: custom:plotly-graph
      entities:

its nothing to do with defaultTabIndex. you can see here that the humidifier card also has issues when used this way:

there are other cards with the issue like thermostat and map. though, map may have been recently fixed. i will look into a way around it. cant promise a timeline though.

Yep trying to use it for the Humidifier and Thermostat, but things won't render, sometimes... tis very odd. Loved the concept of this addon though, but it won't work for this use-case I suppose.

Yep trying to use it for the Humidifier and Thermostat, but things won't render, sometimes... tis very odd. Loved the concept of this addon though, but it won't work for this use-case I suppose.

one trick around that is to make sure cards with this issue are not defaultTabIndex. they will be rendered later.

then we need a switchToIndex   😃
no don't do that ... it's unwise to establish routines to bypass issues, better try to find out why it happens. And yes I'm pretty sure the issue got introduced by HA itself, since I run into similar issues since some time. I was unsure it resulted from a HA update or my switch from a pi into the NUC .... of if it's simply browser-related. Edge gives me less trouble but chrome for example. But I wasn't able to track it down what causes the misery.

from what i can tell, the most recent versions of ha(2023.1.*) no longer have this render issue. testing on the map and thermostat cards. can you all whom are subscribed confirm?

closing this as fixed by home assistant but reopen if necessary.


git_pull

I can't seem to clone my github into my config file.
[Error] can't clone https://github.com/username/package into /config

This doesn't sound as an issue or bug in home assistant. Please open a new issue and fill in the issue template if you still think so.


Trying to install a non-existent version of PHP does not give an error

Describe the bug
Action reports success even if the version of PHP you ask for does not exist.
Version

[ ] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
ubuntu-latest (Ubuntu 18.04.5)
PHP versions
I was trying to use 7.5, which does not currently exist in the Ubuntu 18.04.5 images.
To Reproduce
      - name: Setup PHP
        uses: shivammathur/setup-php@v2
        with:
          php-version: 7.5
          extensions: mbstring, pgsql, mysqli

Expected behavior
Either I get PHP 7.5, or the Step fails with a clear error message, so it is easy to see why my build failed.
Actual behavior
Script tries to continue with PHP 8.0.0, and then fails later because that is not yet supported, but the error message is not very obvious.
Screenshots/Logs
Run shivammathur/setup-php@v2
  with:
    php-version: 7.5
    extensions: mbstring, pgsql, mysqli
/bin/bash /opt/hostedtoolcache/linux.sh 7.5 /home/runner/work/_actions/shivammathur/setup-php/v2/dist

==> Setup PHP
/home/runner/work/_actions/shivammathur/setup-php/v2/dist/../src/scripts/common.sh: line 250: php7.5: command not found
✓ PHP Installed PHP

==> Setup Tools
✓ composer Added composer 2.0.8

==> Setup Extensions
✓ mbstring Enabled
✓ pgsql Enabled
✓ mysqli Enabled

==> Support this project
✓ setup-php https://setup-php.com/support

Note, in particular, the like that says ✓ PHP Installed PHP which is not really right. (The line above is a helpful message.)
Additional context
Thank you very much for making this step and sharing it. Obviously, now that I see what the problem is, i can work around it (don't ask for PHP 7.5 when it does not exist!) so fixing this is not urgent. I just wanted to make you aware of the issue I had spotted.
Are you willing to submit a PR?
Sorry, I am quite new to GitHub actions. I am unlikely to try to submit a fix any time soon.

Fixed in https://github.com/shivammathur/setup-php/commit/ce02569aea1cd6a8e466c1b108631baca269c4d2

Brilliant! Thank you very much.


Exception thrown when connecting to empty DB

This only occurs if you forget to execute populate.sql before quarkus dev.
2022-11-02 12:45:15,414 INFO  [io.deb.con.mys.MySqlStreamingChangeEventSource] (debezium-mysqlconnector-us-east-change-event-source-coordinator) Waiting for keepalive thread to start
2022-11-02 12:45:15,414 INFO  [io.deb.uti.Threads] (blc-localhost:3306) Creating thread debezium-mysqlconnector-us-east-binlog-client
2022-11-02 12:45:15,415 ERROR [io.deb.con.mys.MySqlStreamingChangeEventSource] (blc-localhost:3306) Error during binlog processing. Last offset stored = null, binlog reader near position = binlog.000002/2772
2022-11-02 12:45:15,415 INFO  [com.git.shy.mys.bin.BinaryLogClient] (blc-keepalive-localhost:3306) threadExecutor is shut down, terminating keepalive thread
2022-11-02 12:45:15,416 ERROR [io.deb.pip.ErrorHandler] (blc-localhost:3306) Producer failure: io.debezium.DebeziumException: Client requested master to start replication from position > file size Error code: 1236; SQLSTATE: HY000.
	at io.debezium.connector.mysql.MySqlStreamingChangeEventSource.wrap(MySqlStreamingChangeEventSource.java:1229)
	at io.debezium.connector.mysql.MySqlStreamingChangeEventSource$ReaderThreadLifecycleListener.onCommunicationFailure(MySqlStreamingChangeEventSource.java:1274)
	at com.github.shyiko.mysql.binlog.BinaryLogClient.listenForEventPackets(BinaryLogClient.java:1079)
	at com.github.shyiko.mysql.binlog.BinaryLogClient.connect(BinaryLogClient.java:631)
	at com.github.shyiko.mysql.binlog.BinaryLogClient$7.run(BinaryLogClient.java:932)
	at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: com.github.shyiko.mysql.binlog.network.ServerException: Client requested master to start replication from position > file size
	at com.github.shyiko.mysql.binlog.BinaryLogClient.listenForEventPackets(BinaryLogClient.java:1043)
	... 3 more

This definitely never happened before as originally we didn't even have the populate.sql file. Let me see if I can reproduce.

@ryanemerson Can you provide the entire CLI output? I am unable to reproduce, the only thing that should be different is I have some schema updates, but you should have that already with the setup.sql script running.


Match not found

Microsoft PowerToys version
0.36
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Using a widescreen monitor from LG as primary monitor and secondary monitor from phillips. All connected with a thunderbolt.

✔ Expected Behavior
showing me normal grids.
❌ Actual Behavior
always runs match not found error. Grids are wrongly placed. Problem occurs only when connecting with external displays. Just on the laptop display, it works properly.
Other Software
dadf

Adding Information
I got the same error under this configurations.

Display Settings: Multiple monitor setup (both active monitors)
Taskbar settings: Taskbar must be located Left
Taskbar settings: Turn OFF Automatically hide the taskbar in desktop mode
PowerToys: This FancyZones settings must be enabled Allow zones to span across monitors (all monitors must have the same DPI scaling)

Then if you try to access FancyZones Editor you get

And all the FancyZones settings get lost, even if you try to save them again , those settings won't be saved.
Mitigation
After changing any of this the behavior is fixed
. Taskbar settings: Turn ON Automatically hide the taskbar in desktop mode
. Taskbar settings: Taskbar must NOT be located Left

Fixed in 0.41


Version 3.2.1 package doesn't have updated sass files

The updated _flag-icon-list.scss is not updated.
also the old _flag-icon-more.scss is still inside the package.
We can't use the $include variable.
Also maybe it's possible to use a more distinct property name instead of $include.
Keep up the good work!
Ramon

3.3.0 is released.. can you verify if it's all good?


amd-hybrid-graphics: fix race condition

Motivation for this change
#19775
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I'm really not sure about the correctness of my implementation. Feedback is very welcome!

I removed them because they hid the fact that it was failing, which made the actual problem less evident. I don't know why they were there.

LGTM, @wkennington can you take a look? OR someone with this hardware?

Since nobody else seems to be using this module and able to review the change, perhaps it would make sense to remove the module entirely? Nobody else has complained about it being broken, so either nobody else has encountered the race condition (seems unlikely to me) or nobody else is using it and it would be better off living only in my personal config.


Create data-access services for rooms, desks and facilities

Closes #31

For demo one these data-access services are read-only, i.e. adding rooms/desks/facilities is not supported.


New command: aad license list

Usage
m365 aad license list [options]
Description
Lists commercial subscriptions that an organization has acquired
Options
None!
Examples
List all licenses within the tenant
m365 aad license list

Default properties

id
skuId
skuPartNumber

Additional Info
API docs: https://learn.microsoft.com/en-us/graph/api/subscribedsku-list?view=graph-rest-1.0&tabs=http

nothing to add 👍. looks good to me ✅

Can I work on this one?

Sure