OTP 27 will break coverage support

Before OTP 27, cover worked by inserting code to do counters:add(CRef, LineIndex, 1) around any significant code. The CRef is a counter for the original module's coverage, so Horus only needed to preserve those instructions in the standalone module in order to have the function's coverage count for the original module.
OTP 27 introduces "native" coverage (introduced in https://github.com/erlang/otp/pull/7856). The compiler now emits executable_line instructions when passed the force_line_counters (or line_coverage option, depending on the system's code:get_coverage_mode/0). The JIT turns that that instruction into an atomic add or literal mov that updates a section of the BeamCodeHeader's coverage buffer (see x86, arm). BeamCodeHeader is a member of each module instance (erl_module_instance) and the BIF that cover uses to read coverage information when native coverage is enabled reads from this new buffer from the current module. (As a consequence, old coverage is now no longer accessible after upgrading/reloading a module.)
The old cover code is still around for architectures that use the emulator but we can't turn off native coverage on a JIT-enabled build of OTP.
Since coverage information is now attached to module instances, I don't believe we can trick cover into counting executions of horus functions anymore.

we can't turn off native coverage on a JIT-enabled build of OTP.
+JPcover false?

I brought this up upstream in https://github.com/erlang/otp/issues/8214
But it's not trivial to make native coverage configurable. Even if we can configure it, we would need to also modify rebar to be able to configure how rebar calls cover:compile_beam/1, or at least modify its application env. Being able to change how cover works at runtime can also break some assumptions within cover - it assumes that it's either using native or EMU coverage per node. Using native coverage for some modules and EMU for others can break assumptions if we don't make larger changes to cover.
So for now I will put this down since I don't think it's worth the time investment. We can always call the original functions used in Khepri transactions in unit tests for coverage if needed. We will just need to gate the coverage tests when OTP 27 is released.


MikroORM website bug: v4.5 api docs "page not found"

Describe the bug
A clear and concise description of what the bug is.
When navigating to the v4.5 API docs the website displays the following error message:
"Page Not Found"
To Reproduce
Steps to reproduce the behavior:

Navigate to the following URL:
https://mikro-orm.io/api/4.5/

Expected behavior
A clear and concise description of what you expected to happen.
The older v4.5 API docs should be displayed.

There is some bug in the docusaurus typedoc plugin, the page https://mikro-orm.io/api/4.5 works fine if you open it in anonymous window btw.
Reported here: https://github.com/milesj/docusaurus-plugin-typedoc-api/issues/22

Thanks! :)
Congrats on the v5 release. Amazing work.

Should be fixed for now.


Are there send out RUA reports by Microsoft Office 365?

[Enter feedback here]
Are there send out RUA reports by Microsoft Office 365?

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

ID: 7861a800-7643-dca5-1046-232a51d6e61e
Version Independent ID: d78fc416-516b-15ab-4719-068f7d13ecca
Content: Use DMARC to validate email - Office 365
Content Source: microsoft-365/security/office-365-security/use-dmarc-to-validate-email.md
Service: o365-seccomp
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@aavdberg Thank you for your feedback. A DMARC record invites DMARC reporting organizations to send DMARC aggregate report to the email address mentioned in RUA. It is not sent by Microsoft office 365.
Hope this helps!
Thanks
Sri

@aavdberg  - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by @msbemba so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Added tests for the AnimationTime component

I added missing tests for a std::chrono::steady_clock::duration component after doing something similar in #401.
Signed-off-by: Ashton Larkin<EMAIL_ADDRESS>

A few other things to consider:

Should we use uint64_t instead of int64_t for de-serialization, or can std::chrono::duration::count() return a negative value if a jump back in time occurs?

https://github.com/ignitionrobotics/ign-gazebo/blob/aea1accff6d8daee0358ae9025e2a9797d292844/include/ignition/gazebo/components/Actor.hh#L58-L71

The serialization/de-serialization are only as accurate as milliseconds, since that is what we are currently casting the durations to. If we want to serialize time that is more precise than milliseconds, then the information that exceeds millisecond precision will be lost. Should we use a more precise duration cast? For example, something like std::chrono::nanoseconds?

can std::chrono::duration::count() return a negative value if a jump back in time occurs?

The specific duration type we're using is std::chrono::steady_clock::duration, which is equivalent to std::chrono::nanoseconds, which under the hood is std::chrono::duration<int64_t, std::nano>. Note the int64_t there. So it can be negative.
You're right that we use negative durations for jumps back in time. For example, dt will be negative when a log playback is rewinded:
https://github.com/ignitionrobotics/ign-gazebo/blob/da2ddbc14bc487d53e49de5b9fdea3272797698c/src/systems/log/LogPlayback.cc#L484-L486
Now, will it ever be negative for AnimationTime? I don't think it should. But I was trying to be general when implementing the serialization.

Should we use a more precise duration cast?

Yeah I think that's a good idea. For the animation it probably isn't necessary, but I think it's good to be general.

I went ahead and modified the serialization to use nanoseconds instead of milliseconds in 02ad2db. This should be ready for another review now.


Add notes function

You have to say "note hello world". It will create a file called "Notes.txt" on your desktop and will save "Hello world." to it.


[beta] d3 barchart tooltip: wrong expectations?

Challenge barchart has an issue.
The tooltip tests check for id and data-date attributes. In the process of learning the basics of d3 on fcc, the way to do tooltips is taught through appending a title element to the data elements. It looks like the tests expect a html tooltip instead.
Excerpt from my generated chart:
<rect x="434" y="143.88309797560993" width="2" height="206.11690202439007" class="bar" data-date="2001-04-01" data-gdp="10638.4">
   <title id="tooltip" data-date="2001-04-01">2001-04-01: 10638.4 billion USD</title>
</rect>

I imagine all titles having the same id will cause problems for their lookup, but from a experiential point of view, the user does get the tooltips with the wanted data and they have the proper id. But the tests don't pass.
I'm not sure adding another exercise, just to show an alternative way to do tooltips would make much sense. Maybe the test should be changed? Maybe the challenge description? Both?

What was the resolution, WONTFIX?


fix(examples): react-native: update .gitignore to exclude lib folder

This fix the missing src/lib folder.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-alpha.8 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The frame should be deleted

Compared with old version of Flyt, the frame should be deleted.

Fixed in #446


password reset please

Is your feature request related to a problem? Please describe.
a way to reset passwords becaue I am a dumbass
Describe the solution you'd like
email link to password reset token/page is fine
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Released in v0.16.0


added org/project endpoint with search parameter testing to all the proj...

...ect endpoint with search parameter tests, and made them better for both. Closes #214

@ondrae please review & merge!


Missing buttons in buttonText should keep default value, not "undefined"

Describe the bug
If I want to change the text of only one button, I have to define the text for all buttons in buttonText, otherwise they are set to "undefined". This is especially inconvenient for previousMonth, nextMonth and reset, which I'd have to find the original SVG to copy/paste.

To Reproduce

Set option {buttonText: {apply: "OK", cancel: "Annuler"}}
Open the widget.

Expected behavior
When an option is undefined, it should keep the default value.

Fixed in 1.4.4


[Popover]: Hydration error when popover is open by default of forceMount is used

Bug report
Current Behavior
When either the open prop of the popover root component is true or the portal component has been force mounted next.js will show an hydration error.

Expected behavior
The hydration error should not be there.

Reproducible example
GitHub repository: https://github.com/Haberkamp/nextjs-radix-popover
CodeSandbox Template
Suggested solution

Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/popover
1.0.0

React
n/a
18.2.0

Browser
n/a
n/a

Assistive tech
n/a
n/a

Node
n/a
16.14.0

npm/yarn
yarn
1.22.19

Operating System
MacOS
12.5

Hi @Haberkamp , thanks for raising. This is a known issue and being tracked in https://github.com/radix-ui/primitives/issues/1386
A workaround is to wrap your app in an element and pass the node reference to portal using the container prop.


Typos in Execute a script on the VM using SourceScriptUri parameter section

The example in the section: "Execute a script on the VM using SourceScriptUri parameter" is incomplete and contains issues with quotation that makes the example difficult to read and understand.

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

ID: 04865202-8e09-967f-a7a7-b18ab6845b2e
Version Independent ID: 73268057-c2f4-c7be-6149-a8172bf68e35
Content: Run scripts in a Windows VM in Azure using managed Run Commands - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/run-command-managed.md
Service: virtual-machines
GitHub Login: @nikhilpatel909
Microsoft Alias: jushiman

@MikeKemmerer
I've delegated this to @ericd-mst-github, a content author, to review and share their valuable insights.

@kobulloc-MSFT
Please review it.

The content in question would need to be reviewed by the document author:
https://learn.microsoft.com/en-us/azure/virtual-machines/windows/run-command-managed#execute-a-script-on-the-vm-using-sourcescripturi-parameter
Set-AzVMRunCommand -ResourceGroupName -VMName -RunCommandName -SourceScriptUri “< SAS URI of a storage blob with read access or public URI>" -OutputBlobUri “< SAS URI of a storage append blob with read, add, create, write access>” -ErrorBlobUri “< SAS URI of a storage append blob with read, add, create, write access>”

@MikeKemmerer Thanks for the feedback! I made the command more readable in this PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/279773
#please-close


Make map render jobs transient

Looks like we need to wait for https://smartos.org/bugview/MANTA-1870


Async/Await Broken In Default Config

Version
3.0.0-beta.6
Reproduction link
https://github.com/stolinski/vueBrokenAsync
Steps to reproduce
npm run serve
check console.
async code is added to HelloWorld.vue
What is expected?
Async method to be recognized as a function.
What is actually happening?
Runtime errors stating "TypeError: this.fetchData is not a function"

Also unexpected token errors in eslint. This same code works out of box with the CLI v2 and no mods to the .babelrc file

I'm dumb. Please ignore.

What was it in the end?

It was a typo. I had "method" as my methods object property instead of "methods"

Ok got it, thanks. Asking because all of a sudden this morning I updated my project (which was using beta 1 I believe) and now I'm getting all types of weird errors like 'process' is not defined, 'require' is not defined, and Parsing error: Unexpected token after async/await usage - trying to figure out what happened. Thanks for getting back.

If you are using typescript, make sure to check your tsconfig (especially the compiler options - look at libs).

Thank you for the suggestion @Akryum. Definitely a tough issue - I searched hard for a while today and couldn't find anyone mentioning such weird issues.
In the end I found the issue! I had initialized the project using vue-cli 3 beta 1, and had upgraded the vue-cli on my system and project to vue-cli beta 15 - it was the eslint configs! After scouring a project made with vue-cli beta 15 I found a small difference in the eslint section of the package.json. There was a small definition that wasn't allowing the webpack variables to come through in the app (at least eslint thought so.)
Thank you both for the help.


Initial benchmarks for Nova vs other systems

Problem
We want to have confidence that a Nova-based ZK-VM would be significantly better than existing Halo2 based VMs.
This is also useful for wider ecosystem to warrant more investment into Nova and similar proof systems in certain conditions.
While we primarily care about prover time, memory/ptau constraints are also relevant, especially for big circuits. We are especially interested in what parallelism unlocks.
Notes
There are also other benchmark efforts in the wider ecosystem that makes sense to hook into. E.g.:

https://github.com/celer-network/zk-benchmark
https://github.com/zkCollective/zk-Harness

Currently we are basing structure based on Celer network.
We can run both on laptops as well as a powerful server. While running on a distributed network would be desirable, this involves more setup and would likely be a natural next step after initial benchmarks are done.
Also see README for more details on current thinking and observations.
Acceptance criteria
Benchmarks for Nova against Circom (naive) and Halo2 that shows different prover time per depth.
Ideally find a benchmark that shows where Nova has a x10 performance improvement.

Running Nova native benchmarks https://github.com/microsoft/Nova/blob/main/benches/compressed-snark.rs#L33 to check only one step circuit, no recursion.
Constraints vs prover time:
10k 0.4s
100k 1.1s
1m 7s
10m 105s

We don't run out of memory so in Nova Scotia likely due to witness generation. CPUs maxed out so GPU/CUDA would be useful.

This is random constraints so not comparable with SHA256, unless looking at order of magnitude of constraints.

Bitcoin benchmarks https://hackmd.io/0gVClQ9IQiSXHYAK0Up9hg?view=

Above numbers was actually CompressedSNARK which seems wrong.
RecursiveSNARK Nova native (https://github.com/microsoft/Nova/blob/main/benches/recursive-snark.rs#L79):
10k 90ms
100k 148ms
1m 650ms
10m ...
Parallel Nova native - maybe something wrong in it, seemed a bit off (https://github.com/privacy-scaling-explorations/Nova/pull/9):
10k 300ms
100k 400ms
1m 1.3s
10m 11.3s
cc @aleph-v

Both run with this diff (to get x10 orders of magnitude)
-  let num_cons_verifier_circuit_primary = 9819;
+  let num_cons_verifier_circuit_primary = 10000;
   // we vary the number of constraints in the step circuit
   for &num_cons_in_augmented_circuit in
-    [9819, 16384, 32768, 65536, 131072, 262144, 524288, 1048576].iter()
+    [10000, 20000, 110000, 1010000, 10010000, 100010000].iter()

Summary of benchmarks and conclusions https://hackmd.io/0gVClQ9IQiSXHYAK0Up9hg?both=

The Parallel Nova is a bit disappointing but not that surprising, it does three times as many folds per step. So you can see that the overhead goes from 3.3x to around 1.9 as the extra folding becomes the overhead. Either that or the memory bug makes this slower. I think best way to lower this overhead is to optimize the F' in parallel for the foldings used.

I'll close this issue as we've done initial benchmarking above, especially in context of ZK Residency. There's still room for additional benchmarks, see "future benchmarks" in hackmd above.


panic in redisScaler

Report
Running a brand new keda 2.9.3 install (my first keda install) on EKS, monitoring many redis celery queues, and getting occasional panics in redisScaler.
I believe it is happening only when I am applying changes to the scaler configuration (though I was making no changes to the redis trigger.)
Expected Behavior
No panics
Actual Behavior
Rarely, during application of updates to a ScaledObject with a redis trigger, I would get the panic. The operator would restart fine after a few seconds.
Steps to Reproduce the Problem

Run a lot of simple redis ScalerObjects
Apply minor changes to a lot of them

Logs from KEDA operator
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x5eeea0]

goroutine 20089 [running]:
github.com/go-logr/logr.Logger.Error({{0x0?, 0x0?}, 0x2d6241e?}, {0x40930a0, 0xc000029c80}, {0x396abad, 0x19}, {0x0, 0x0, 0x0})
	/workspace/vendor/github.com/go-logr/logr/logr.go:279 +0x80
github.com/kedacore/keda/v2/pkg/scalers.(*redisScaler).GetMetricsAndActivity(0xc01a6b8540, {0x40bf988?, 0xc01abfcd00?}, {0xc019c75b40, 0xc})
	/workspace/pkg/scalers/redis_scaler.go:266 +0x1a5
github.com/kedacore/keda/v2/pkg/scaling/cache.(*ScalersCache).GetScaledObjectState(0xc01a6b85c0, {0x40bf988, 0xc01abfcd00}, 0xc01bdd2000)
	/workspace/pkg/scaling/cache/scalers_cache.go:136 +0x6fa
github.com/kedacore/keda/v2/pkg/scaling.(*scaleHandler).checkScalers(0xc000e97ef0, {0x40bf988, 0xc01abfcd00}, {0x3873b80?, 0xc01bdd2000?}, {0x40acd18, 0xc019c74928})
	/workspace/pkg/scaling/scale_handler.go:360 +0x510
github.com/kedacore/keda/v2/pkg/scaling.(*scaleHandler).startScaleLoop(0xc000e97ef0, {0x40bf988, 0xc01abfcd00}, 0xc019513680, {0x3873b80, 0xc01bdd2000}, {0x40acd18, 0xc019c74928})
	/workspace/pkg/scaling/scale_handler.go:162 +0x32a
created by github.com/kedacore/keda/v2/pkg/scaling.(*scaleHandler).HandleScalableObject
	/workspace/pkg/scaling/scale_handler.go:118 +0x71d

KEDA Version
2.9.3
Kubernetes Version
1.24
Platform
Amazon Web Services
Scaler Details
redis
Anything else?
No response

Hi,
Thanks for reporting, it seems that the logger isn't properly assigned IDK why (yet).

Okey, I found the issue, the logger isn't properly assigned during the scaler generation and on any error (which should be just logged) the operator panics.

Hey @veep
The issue has been fixed, it'll be included in the next release


Add new function-color-channel rule

Closes #1057.

Thanks! I'll try to review this asap.

Thanks! Looks really good, just one minor comment.

Oh yeah one more thing that I almost forgot. Could you please also add a link to this rule to the main README?

Could you please also add a link to this rule to the main README?

Done in f6af81f.


[SPARK-39282][SQL] Replace If-Else branch with bitwise operators in roundNumberOfBytesToNearestWord

What changes were proposed in this pull request?
use bitwise operators to avoid If-Else branch and improve computation performance.

Hi @cloud-fan @ueshin , Please give a review, thanks!

Can one of the admins verify this patch?

Thanks @cloud-fan @srowen ,  can it be merged?

The GA job didn't pass, can you check?

**continuous-integration/appveyor/pr ** — AppVeyor build

Hi @cloud-fan @srowen
All Three GA jobs passed. Thanks

The GA job didn't pass, can you check?

Hi @cloud-fan @srowen
All Three GA jobs passed. Thanks

I think the appveyor error is unrelated. I'll merge shortly

MErged to master


f<T>(a:T,b:T) not rejects passing different Record types for a and b (but not when a and b are values).

Bug Report

🔎 Search Terms

Generic values
Generic types
Geneics in parameters

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about all

⏯ Playground Link

Playground link with relevant code
💻 Code

function concat<T>(a:T, b:T): string{
    return a + ' ' + b;
}

var res1 = concat("hola", 3);
  // ERROR: Argument of type 'number' is not assignable to parameter of type 'string'.(2345)

var res2 = concat({name:"hola", value:3}, {name:/Hola/, value:3});
/* typeof concat here:
function concat<{name: string; value: number;} | {name: RegExp; value: number;}>
            (a: {name: string; value: number;} | {name: RegExp; value: number;},
             b: {name: string; value: number;} | {name: RegExp; value: number;}): string
*/

🙁 Actual behavior
The second call was accepted by TC, deducing that the property name is of type string | RegExp,
albeit rejects the first.
🙂 Expected behavior
In my opinion concat must reject it, because the types are not the same (they aren't in the first call also).
I think there is a degree of inconsistency.
use case
compareDbRows<T>(row1:T, row2:T): boolean{
   // ...
}

In my opinion concat must reject it, because the types are not the same (they aren't in the first call also).

I think there is a degree of inconsistency.

The consistency argument can only go so far, since concat(0, 1) is also a call of two types (0 and 1). Inference attempts to infer what you wanted to happen; it can't know for sure so has to make some judgment calls in both directions.

I understand that the union can be always computed. But in concat("hola", 3) of my example TS founds an error (and could have understood the union type here).
May be this is no so bad as it apears because it seems that this only happens when I passing literals to the function. In:
var left = {name:"hola", value:3};
var right = {name:/Hola/, value:3};
var res2 = concat(left, right);

It works as I expect (rejecting).
As you can see in Playground


PodInterface should support Watch

PodInterface should have Watch method behaving in the same way as ServiceInterface.Watch does.
Use case: a monitoring system (namely Prometheus) expects to get the list of monitored servers performing a  DNS-SD query. Special service (similar to kube2sky shipped with Kubernetes) mirrors status of running Pods into etcd, which is in turn monitored by SkyDNS server and exported as DNS-SD. Schematically the data flow looks like this: Kubernetes -> kube2sky -> etcd -> SkyDNS -> Prometheus.
The biggest challenge is to deliver list of all running pods from Kubernetes to kube2sky. Currently kube2sky can only poll Kubernetes with PodInterface.List() each N seconds, which is not efficient. Having a Watch method would be much better.

cc @lavalamp: Do you know why we do not support watch?

Every object should support Watch.

Watching status doesn't currently work -- that's #2726.

I agree with OP. This is just an accident of history; some boilerplate in the client isn't completely filled out. As Brian mentions, not every field is currently watchable but that's being addressed elsewhere.

@sdminonne This is relevant to your interests potentially w/ your work on #560

This is a pretty simple change if someone wants to get their feet wet. :)

I'll work on it.


update to new libdparse

libdparse changes quite a lot - it's impressive just changing the module path works.
It might be a good idea to lock the version ...

contains the fix for #34 too

oh just realized that Travis is failing as it tries to build with 2.066 - updated to the "native" travis solution ;-)


Restructure tests

Hi all,
I find myself stumbling across the structure of the tests more and more frequently.
The idea behind the structure as is was to keep implementation and test close to each other in the filetree. In practice, it is often a hassle to determine if there are existing tests and where to find them.

I suggest to move all tests into the top-level typhon/tests/ directory.

Every subpackage gets an own test module:
tests/
|-- test_atmosphere.py
|-- test_geographical.py
|-- test_latex.py
|-- test_spectroscopy.py
-- test_workspace.py

The structure within the sub-packages could be emulated by classes (TestXML, TestXMLMatpack, ...).
What are your opinions? Any arguments to stick to the old system? Or has anyone an even better idea how to structure our tests in general?
Cheers,
Lukas

I agree with the new structure. Experience so far has shown that having the tests scattered across all package subdirectories is not helpful.

Yes, I also think that there should be only one subtree for all the tests. But why not mirror the structure
of the source tree?
tests/
|-- typhon
    |-- arts
        |-- test_workspace.py
        |-- ...

For me that's the simplest mapping from source to test files.

I wanted to limit the number of directories and files for now :wink:
But I guess you are right. Mapping the full project structure in typhon/tests  seems to be more intuitive and future-proof.
I will work on the new structure and submit the pull request.


Fix missing requirements for Python 3.6 64bit

sdist might need to be first, but without deps it will fail on missing Cython, thus no wheels for 3.6 64b
ref: https://ci.appveyor.com/project/KivyOrg/kivy/build/1.0.2460/job/u6bsnlrwwe5n5yy2#L202
Perhaps kivy.deps.* might need to be moved to the original place, I'm not sure.

We don't want any of the kivy.deps.xxx to be installed when we do the sdist, otherwise they may get into the sdist. So we should only install the non kivy related packages before the sdist.


docs: elaborate on k8s compatibility

Motivation

this has been asked about by users several times on Slack

as well as at least once on GitHub: #12695

I also asked about our k8s upgrade policy in a Contributor Meeting and the contributors Slack

and I clarified later in the thread about k8s's guarantees and client-go's matrix, etc

Modifications

this PR is effectively equivalent to my clarifications on Slack: specify that k8s is backward compatible with all clients and link to client-go's docs

also link to the CI tests for concrete references on k8s version testing

minor wording improvements, simplifying some sentences as well as specifying Argo version vs k8s version to clear up ambiguous wording

reference k8s and Argo consistently (the docs do not use Argo Workflows or Kubernetes anywhere else)

Verification

make docs passes
Notes to Reviewers

NOTE: this will need to be modified when backported -- since we have versioned docs now (#11390), those should reference their own branch (i.e. release-3.5, release-3.4) and that branch's tests

this is important since currently the earlier versions of docs say that "the main branch is tested on" even though they are referring to their own branch now and not main anymore
I can handle backporting that myself once merged

Backported to release-3.5 in https://github.com/argoproj/argo-workflows/commit/848bfba74938c75558b70117e122d15d0b3a9abf and release-3.4 in https://github.com/argoproj/argo-workflows/commit/30d053c321139392f4e011e8ac2350d77d2ff425


Fix MP4 videos saving result in 3GP on Android API <=28

Fixed the issue that the video format changes to 3gp when saving mp4 videos to the album on Android 9

Could you provide a test case in which saving 3gp will fail? Your changes don't tell the idea and why it will fix the issue well.

pubspec.yaml
photo_manager: ^3.3.0
photo_manager_image_provider: ^2.1.1

use my code

In MI 8 Lite android 9

It is still unclear why the PR fixes the issue. Could you provide a minimal reproducible example?

var appCacheDir = await getApplicationCacheDirectory();
String filePath = '${appCacheDir.path}/aaa.mp4';
File result = File(filePath);
var entity = await PhotoManager.editor.saveVideo(
result,
title: path.basename(result.path),
);
print('assetEntity===${entity!.title}===${await entity.file}');

example code
var appCacheDir = await getApplicationCacheDirectory();
String filePath = '${appCacheDir.path}/aaa.mp4';
File result = File(filePath);
var entity = await PhotoManager.editor.saveVideo(
      result,
     title: path.basename(result.path),
     );
print('assetEntity===${entity!.title}===${await entity.file}');

On Android 9, the source file is mp4 and converted to 3gp format through saveVideo

Is this happening on all videos or just one specific video? Could you file an issue to summarize all the details? And provide a minimal runnable example so we can run to reproduce.

All videos on android9
demo: https://github.com/yishangfei/test1

I added it in line 455 according to your latest code.

Still working on this?

now there is no problem. I only modified the version below Android 10. It’s just that the timestamp needs to be used when saving. If the original name is used a second time, the entity returned by PhotoManager.editor.saveVideo will be null.

If the original name is used a second time, the entity returned by PhotoManager.editor.saveVideo will be null.

We need to figure out why.
Also please do another rebase on the latest changes and remove unnecessary changes.

The automatic renaming function (for example, video (1).mp4) is implemented in Android 11 and above. In earlier versions, if you repeat cr.insert in the insertUri method, an exception will be thrown. "Cannot insert the new asset."
 val uri = cr.insert(contentUri, values) ?: throw RuntimeException("Cannot insert the new asset.")

Just like the pictures I provided before，versions below Android Q use timestamp naming to insert, or write a renaming logic that is the same as that of higher versions.

The automatic renaming function (for example, video (1).mp4) is implemented in Android 11 and above.

TIL. Are there any docs referencing this?

I have not been able to find any official documentation.
I found the solution on stackoverflow.
ContentResolver.insert always returning null
When using the ContentResolver.insert() method to insert a file with the same name, an exception will indeed be thrown if the file with the same name already exists. This is because in Android 10 and below, MediaStore does not have the automatic renaming function.

add the following comment to the code. Is it OK?

How about:
Using a duplicate filename that already exists on the device will cause the insertion to fail on Android API 30-.

add the following comment to the code. Is it OK?

How about:
Using a duplicate filename that already exists on the device will cause the insertion to fail on Android API 30-.

i submitted

I've also added some format changes and fixed the nullable saving result.


Alamofire 4.0.1 podspec problem Xcode8.0

Hi, I'm migrating from 3.1.5 to 4.0.1 I'm using a podspec creating my own library, Using Xcode 8.0 and cocoapods 1.1.1 , but when I update to this version I get this error
[!] Unable to satisfy the following requirements:

- `ExpSwift (from `../`)` required by `Podfile`
- `ExpSwift (from `../`)` required by `Podfile`

Specs satisfying the `ExpSwift (from `../`)` dependency were found, but they required a higher minimum deployment target.

I did updated the deployment target to 10.0 but I get the same error. Do I need to update something else, here us my podspec and podfile
 s.platform     = :ios, '10.0'
  s.requires_arc = true

 s.source_files = 'Pod/Classes/**/*'
 #s.resource_bundles = {
 #  'ExpSwift' => ['Pod/Assets/*.png']
 #}

 # s.public_header_files = 'Pod/Classes/**/*.h'
 # s.frameworks = 'UIKit', 'MapKit'
  s.dependency 'PromiseKit','4.0'
  s.dependency 'Alamofire','4.0.0'
  s.dependency 'Socket.IO-Client-Swift','8.0.2'
  s.dependency 'JSONWebToken','2.0.1'
end

And my podfile just reference to my local pod
source 'https://github.com/CocoaPods/Specs.git'
use_frameworks!
platform :ios, '9.0'

target 'ExpSwift_Example' do
  pod "ExpSwift", :path => "../"
end

target 'ExpSwift_Tests' do
  pod "ExpSwift", :path => "../"
end

This looks more like a CocoaPods issue and we use our GitHub project for bug reports and feature requests. In the future, you should open questions like this on Stack Overflow.
Cheers. 🍻

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.

@jshier I think this is a problem with Alamofire beacuse if I try with version 3.5 it works with no problem, the problem is just with the last version.

It's not a problem with Alamofire, many people can install from CocoaPods just fine. You need to update your specs repo.

@jshier OK I found the problem is that you need to add platform :ios, '10.0' to your local podfile, I was using 9.0. Are you guys going to support ios 9??


Dataset downloaded undistorted images not fully downloaded as image_index.txt

After extracting the undistorted images (1000x1000), I found there are multiple images missing, e.g. 00000004/undist_images/00000000.jpg, 00000005/undist_images/00000417.jpg and so on. Totally there are 5631 images missing according to my observation.
I tried to download and extract the data twice, with the same issue. Could you check whether it is an error or my problem? Thanks!

I also have this problem.Have you solved it?


Add other peak and event level pairing plugins

coverage: 53.597% (-2.1%) from 55.662%
when pulling 302a26db245be952076618e1e19ee15e5a544c2f on events_paired
into 7ab745fa150e76e8fa624770ec9fb7fb29b5a779 on main.


[PE-D][Tester A] meetingUpdate is also not working as well

Similar to the meetingRemove, invoking meeting update does not work
Here are the exact steps to reproduce the error:
•	add n/Betsy Crowe t/LifeInsurance<EMAIL_ADDRESS>a/Newgate Prison p/1234567
•	meetingAdd 7 md/Meeting with Betsy ms/30-03-2025 12:30 me/30-03-2025 13:30
•	delete 7
•	meetingAdd 5 md/Meeting with Ibrahim ms/30-03-2026 15:30 me/30-03-2026 16:30
Then trying meetingUpdate does not work

Labels: type.FunctionalityBug severity.High
original: ARPspoofing/ped#21

Also an error with user's understanding of how to use the meetingUpdate command. Will rectify by updating UG


sed: can't read s@zzLAT@48.051762@g: No such file or directory

Problem
Fresh installation, results in sed: can't read s@zzLAT@48.051762@g: No such file or directory
To Reproduce
Please paste a screenshot or steps to reproduce the behavior:

Use Option 1 -- Quick Start
Select Tesla Cloud (2)
Run as Current User (Y)
Leave Time Zone Default
Enter email address for Tesla Account
Setup Dash for Solar-Only (N)
Do you want to setup Weather Data (Y)
Place API Key
Set Latitude
Set Longitude
Set Imperial
12.....error

sed: can't read s@zzLAT@48.051762@g: No such file or directory
Screenshots
If applicable, add screenshots to help explain your problem.
Host System

Hardware: Raspberry Pi 4
OS: Ubuntu Server v24

Additional context
Add any other context about the problem here.

Thanks, @wagisdev - I push a hotfix ASAP.

Thanks for reporting this @wagisdev!  This was introduced in the last slate of bug fixes (isn't that the way it is?). I've updated the logic to the sed -i.bak that should work for all OS variants.


Add a key with the type to the error message

For example:
{
  error: {
     type: "OAuthException",
     message: "Session has expired at unix time<PHONE_NUMBER>. The current unix time is<PHONE_NUMBER>."
  },
}

Will be released in the 6th version.


RFC: Key Visualizer: The visualized hotspot profiler

Summary
Add an enabled-by-default KeyVisualizer component to PD, providing a visual webpage to show key heatmap of the TiKV cluster, which is useful for troubleshooting and reasoning inefficient application usage patterns. KeyVisualizer is accessible in-browser via the URL from PD.
Motivation
At present, if someone wants to troubleshoot a performance issue in TiKV, he will usually make use of some existing diagnostic tools, such as pd-ctl, Prometheus and Grafana, which are hard to learn, intuitive, not efficient, unable to recognize the pattern of application and hard to make correct judgment basing on them.
Google BigTable has similar problems. So they provided a heatmap profiler called BigTable KeyVisualizer (Video), which is recommended that the user first use to troubleshoot performance problems. After investigation, we realized that it has
good features that also suits TiKV, thus we added some of those features to our design.
Rendered
Rendered (zh-CN)

@dcalvin Can you help with translation? I believe convention is that RFCs are in English.


Is it possible to have early stopping in lax.scan?

I am considering a case in which there is a loop of operations, each one of them being expensive; one can bound the worst case number of operations with a fixed N but one can have a stopping condition and on average the loop terminates in K operations with K << N.
With lax.while_loop it is relatively easy to implement this efficiently but unfortunately one loses reverse mode differentiation. Would it be possible to have a version of lax.scan that supports such an early stopping?

You could do this with lax.scan and lax.cond -- just apply the identity function if the loop terminates early.
The downside is that you will pay the price of memory allocation for all N steps. This a limitation of XLA's memory model: all memory allocation must use statistically known shapes.

This came up in one of our chat rooms yesterday, and it was also observed that because of how we currently implement vmap-of-cond (namely we always turn it into a select), using cond for this might lead to inefficient batching.
I think adding an early stopping version of lax.scan is a good idea. It's not trivial though. Notes to self: we'd probably need to add a jaxpr for the early-stopping function, and also add 'start' and 'stop' arguments to the scan for the linearized case.

Until we add this (or figure out a better plan), you might be able to work around the issue using jax.custom_vjp around your loop.

Hi, I am trying to have an early break within lax.scan since lax.while_looptakes more time to be compiled and slower. However, I did not understand how to use lax.cond exactly to achieve that. Could you elaborate more i.e giving a small example code.

Hey @mattjj! Late to the party, but I am facing the same limitations.
Is there any plan to implement this?

Sorry for bothering but IMO this feature will have a wide range of uses, e.g. stop auto-regressive generation of LLM when encountering EOS. Would really appreciate it if this could be implemented.


ESP32-C6 for both Matter over Thread and Matter over WiFi in single chip (CON-1361)

Is this possible to use ESP32-C6 for both Matter over Thread and Matter over WiFi in single chip? we want to include a button which can be selected before bootup to select either matter over Thread or Matter over WiFi. Is this possible?
Also is there any document for ESP32-C6 for battery life estimation for Matter over Thread case?

@bilalmalik76 It is possible, and we have a proof of concept. You can find the documentation here: https://gist.github.com/shubhamdp/76723cdedf6cdf88d7e0c49602ccab2. This PoC allows you to choose the interface after each factory reset and not on every boot-up.

Also is there any document for ESP32-C6 for battery life estimation for Matter over Thread case?

We have some sampled power consumption data, which you can find here: https://github.com/espressif/esp-matter/tree/main/examples/icd_app#4-power-usage. These samples are with Matter ICD feature. Please note that these numbers are not sampled using a battery power source.

Hi @shubhamdp Thanks for sharing but this is not accessible: https://gist.github.com/shubhamdp/76723cdedf6cdf88d7e0c49602ccab2

@shubhamdp power consumption figures shown on this link are seems to be very high for a battery power device.
https://github.com/espressif/esp-matter/tree/main/examples/icd_app#4-power-usage

Hi @shubhamdp Thanks for sharing but this is not accessible: https://gist.github.com/shubhamdp/76723cdedf6cdf88d7e0c49602ccab2

Sorry for the incorrect link, please check https://gist.github.com/shubhamdp/76723dcedf6cdfe88d7e0c49602ccab2
I have also updated the link in the above comment.

@bilalmalik76 please feel free to close the issue if question is answered.


CHANGELOG: update change log for 0.2.4 release

Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

Coverage remained the same at ?% when pulling 9f3701128d57be74c031823cad04b9da899f848e on ffmancera:changelog024 into 5e8d62985a6fc6915d3015aa922effe5c5723373 on nmstate:master.


Add consistent_comment_rule option for CSS

consistent_comment_rule option is false by default.
This change is not only in javascript, but also in python codes and regressions.
This shall be an option to be used for brackets-beautify project as an extension for Brackets.
https://github.com/leoliu1313/brackets-beautify
https://github.com/drewhamlett/brackets-beautify
http://brackets.io/

./jsbeautifier/tests/shell-smoke-test.sh - PASSED.
Post-build git status check...
Ensuring no changes visible to git have been made to '' ...
ERROR: Post-build git status check - FAILED.
Git status reported changes to non-git-ignore'd files.
TO REPRO: Run 'git status '.  The use git gui or git diff to see what was changed during the build.
TO FIX: Amend your commit and rebuild. Repeat until git status reports no changes both before and after the build.
OUTPUT FOR 'git status ':
HEAD detached at FETCH_HEAD
Changes not staged for commit:
(use "git add ..." to update what will be committed)
(use "git checkout -- ..." to discard changes in working directory)

modified:   js/test/beautify-css-tests.js
modified:   python/cssbeautifier/tests/test.py

no changes added to commit (use "git add" and/or "git commit -a")

I still don't know how to fix this issue. My codes are clean. And my result of git status is also clean. Can you help me pass this issue? Or, can I simply remove ./test/git-status-clear.sh file? Thanks a lot!

What happens when you make from the root?
Once you do that, you'll see this error as well.

thanks for the reply. what do you mean by make from the root? can you provide more info on that? really appreciate the help. I am really new on doing pull request. there are only a few materials on the Internet related to this issue and none of them is working. what is the purpose of that sh file check? my codes pass regression tests. it should be okay to merge them. thanks.

On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean


Enabled drag & drop to an empty section

My previous comment was partially incorrect. It appears that a section footer or header is not required for this to work.

This is a great improvement!
Can we take action and make a new PodSpec? There seem to be 6 pending pull requests, yet no changes in 2 years...


PR: Add an option to sort editor tabs alphabetically

Description of Changes

[X] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)
[X] Included a screenshot (if affecting the UI)

Added a small feature that sorts the editor tabs alphabetically. Sometimes having multiple files open at the same time can become overwhelming to sort through to find a file of interest to continue editing or to reference it contents, and although the file explorer can simply be used to re-select the file, sometimes the file explorer is navigated away from the location of the other files.
A menu item has been added to the editor tabs "option" menu at the top right corner of the editor tab widget stack.

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: Alexander L. Deweert

Hello @AlexDeweert! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/editor/widgets/editor.py:

Line 484:39: E231 missing whitespace after ','
Line 498:58: E231 missing whitespace after ','
Line 1576:29: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
Line 1577:29: E231 missing whitespace after ','
Line 1578:20: E201 whitespace after '('
Line 1578:80: E501 line too long (86 > 79 characters)
Line 1578:84: E202 whitespace before ')'
Line 1584:80: E501 line too long (83 > 79 characters)

@CAM-Gerlach, could you review the functionality of this PR? Thanks!

@CAM-Gerlach I'm seeing two failures but I'm not sure why exactly. Any help would be appreciated.

@ccordoba12 I think this fixes Issue https://github.com/spyder-ide/spyder/issues/5911

Thanks for noticing it :+1:


Feat: Witchess Daily Puzzle Solver in Quests menu

Adds an item to the Quests menu that will attempt to solve the user's 5 daily Witchess Puzzles if they haven't already.
States handled:

If the user has already solved all 150 puzzles (Puzzle Champ bonus == 20)
If the user has already solved today's puzzles (_witchessBuff = true)
If the user has already solved some of today's puzzles (Puzzle will have "Solved Today" text)
If the user does not have a Witchess Set (Campground does not contain "chesstable.gif")

To-do: (Open to suggestions on how to do each of these things)

Test cases

Some done: Tests if the html request is made and parsed correctly, and tests 1 of the 4 scenarios where the request isn't made because we're aware of the user being in an incompatible state. Includes the html for 15 unsolved and solved puzzles to test against. I've also been running it for myself during each leg of my looping, so can get 10 more puzzles to test the state of each day, but it seems unnecessary to have that many.

Handling if we failed to solve the puzzle somehow

Technically handled, will show an error to the user that it failed, generally trying to say why, and stop executing the script

Improve the check for having Witchess
Add gCLI command to run the solver.

Just extended the witchess gCLI command to allow running the witchess solver. In order to prevent breakage in existing code, the command now accepts witchess or witchess buff to get the buff, and witchess solve to try and solve the player's daily puzzles.


191 router bid error

Fixes #191

Update: the root cause is the provider.getBalance calls taking >6s (the auction period).  I think the caching might need to be done slightly differently, like cached per block number. Also with the new architecture, cache belongs as its own adapter.


fix(errors): support error checking methods for new problem structures

Adds previously returned errors to the "Unwrap" chain for checking against in methods like "errors.Is()" and adds an implementation of "Is" to Problem types to enable equality checking against problems.
Also, adds all of the previously returned errors back into the problem structures to maintain full compatibility with downstream code.

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


4 integration with huggingface datasets

Thanks for the review! Since this PR was already merged, I resolved these points in branch 9-update-api-based-on-deliverable-document


Did not work in Net Core3.1 PublishSingleFile mode

It work well using 'dotnet publish'. But I publish using 'dotnet publish /p:PublishSingleFile=true', I found it has no logs.

See https://github.com/NLog/NLog/issues/3517
ergo: use ${basedir:fixtempdir=true} and NLog 4.6.8+

thank's for 304NotModified
I change the path in the nlog.config and it work .

great! :)

@304NotModified
hi, I get this issue as well
and my Nlog 4.6.8 :
<target xsi:type="File" name="f1" fileName="${basedir:fixtempdir=true}/../logs/Normal/${shortdate}.txt"
           layout="${longdate} ${newline} ${uppercase:${level}} ${message} ${newline}" />

but it is not work in linux service

@longskikondi Please create new issue and fill out the template: https://github.com/NLog/NLog/issues/new/choose
Think the most important part is to provide the output from the InternalLogger. You need to provide the details of where Microsoft have decided to handle unzipping of single-file-publish-app on Linux (On Windows they used the temporary directory. Maybe another hack is used on Linux).
Please investigate yourself and if you find the issue, and can see NLog could better handle single-file-publish-app on Linux, then please create a pull-request.


Error in impoerting tensorflow 1.9

Hello!
I had installed tensorflow 1.5 for using g2p_seq2seq. But for some error using this, I decided to upgrade my tensorflow version to 1.9. when I use pip install --upgrade tensorflow everything is ok and I have Successfully installed tensorboard-1.9.0 tensorflow-1.9.0 in my CMD, but when I want to import tensorflow I have below error :

import tensorflow
Traceback (most recent call last):
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 14, in swig_import_helper
return importlib.import_module(mname)
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 658, in _load_unlocked
File "", line 571, in module_from_spec
File "", line 922, in create_module
File "", line 219, in _call_with_frames_removed
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 17, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 16, in swig_import_helper
return importlib.import_module('pywrap_tensorflow_internal')
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_pywrap_tensorflow_internal'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "<pyshell#0>", line 1, in
import tensorflow
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_init_.py", line 22, in
from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python_init_.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 14, in swig_import_helper
return importlib.import_module(mname)
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 658, in _load_unlocked
File "", line 571, in module_from_spec
File "", line 922, in create_module
File "", line 219, in _call_with_frames_removed
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 17, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 16, in swig_import_helper
return importlib.import_module('pywrap_tensorflow_internal')
File "C:\Users\ida\AppData\Local\Programs\Python\Python36\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_pywrap_tensorflow_internal'
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/install_sources#common_installation_problems
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.
what should I do now???
I really need to help.
my platform is windows 10, my python version is 3.6.5 and I'm using cpu tensorflow.
Thanks in advance.

You can try to uninstall tensorflow and reinstall tensorflow CPU version from tensorflow website.

I've done it, but I've gotten the same error.

A few users have claimed using conda instead of pip install helped with Windows python3.5 issues. Can you try that?

No I've never worked with it.
I uninstalled 1.9.0 version of tensorflow and installed 1.5.0 version of it to use the older verson of g2p_seq2seq.

Nagging Assignee @mrry: It has been 16 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @mrry: It has been 31 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @mrry: It has been 46 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @mrry: It has been 61 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @mrry: It has been 76 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Still There is issue you have downgrade installation to tensorflow 1.5.
is there any proper solution ?

Nagging Assignee @mrry: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I tried to debug this or a related issue a little more: https://github.com/tensorflow/tensorflow/issues/27035#issuecomment-479392652


Libxml rebuild

{package} {version} {:snowflake:}
Destination channel: {Snowflake | defaults}
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_189ws_5kgu/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_51lug0l1kn/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_774obi4ii1/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_b5_4c_uz_h/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_2ahgg9yx5e/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_ad4683y73n/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Sorry, my mistake. There's a run_exports in libxml2, which will take care of the compatibility. so it should be either simply - libxml2 in run, or you can leave it out completely. Putting your own bounds in here will override the run_exports (i think).
The important thing is that there's an upper bound because there's ABI breakage between minor versions.

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_45kqv5jc69/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_84r_z08w8u/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_d4ukweuyh4/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found


use @storybook published deprecated dependencies

Issue: temporarily relieves #2303
What I did
forked all the komposer/mantra deprecated packages and published version without peer dependency warning
How to test
Is this testable with jest or storyshots?
n
Does this need a new example in the kitchen sink apps?
n
Does this need an update to the documentation?
n
If your answer is yes to any of these, please make sure to include it in your PR.

Nice! LGTM! Do I need to test before we merge?

@shilman I tested on vue, cra, and crna. We'll see if this netlify thing works too.

I'm really confused about the build yarn failure. I just ran that same command and it didn't change anything.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2314   +/-   ##
=======================================
  Coverage   21.19%   21.19%
=======================================
  Files         283      283
  Lines        6156     6156
  Branches      724      723    -1
=======================================
  Hits         1305     1305
- Misses       4302     4305    +3
+ Partials      549      546    -3

Impacted Files
Coverage Δ

lib/ui/src/index.js
13.33% <ø> (ø)
:arrow_up:

lib/ui/src/compose.js
33.33% <ø> (ø)
:arrow_up:

app/react/src/server/config/babel.js
0% <0%> (-100%)
:arrow_down:

app/react/src/server/babel_config.js
0% <0%> (-77.42%)
:arrow_down:

app/react/src/server/utils.js
0% <0%> (-53.58%)
:arrow_down:

addons/knobs/src/components/PropField.js
10.63% <0%> (ø)
:arrow_up:

lib/ui/src/modules/shortcuts/actions/shortcuts.js
6.25% <0%> (ø)
:arrow_up:

...res__/update-addon-info/update-addon-info.input.js
0% <0%> (ø)
:arrow_up:

lib/components/src/navigation/menu_link.js
0% <0%> (ø)
:arrow_up:

.../src/manager/containers/CommentsPanel/dataStore.js
34.04% <0%> (ø)
:arrow_up:

... and 31 more

Continue to review full report at Codecov.

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


Emscripten ClearRenderTarget() issue

It seems this API accepts float RGBA, while the RTV could be in integral format...
WebGL complains about calling the float version API on integral texture backed render target.
Not sure what's the "correct" way to fix this...

seems duplicate of #287, sorry.

Duplicate of #287


update lftp package to 4.8.1 please

Can somebody update the lftp package to the latest 4.8.1
seems there are some issues when compiling with the new globfree.c in MSYS2
Can you help @mashir43 or @Alexpux ?
Thanks a lot!

Any news on this one @Alexpux ?

New package in repo


Update web-sites-configure-ssl-certificate.md

Adding clarification on multiple IP SSL bindings. Knowing this would have saved me a lot of time, and avoided this unfortunate bug: https://social.msdn.microsoft.com/Forums/en-US/8c0305ae-7f51-40c6-ae36-762c04e2030a/adding-multiple-ip-ssl-bindings?forum=windowsazurewebsitespreview
Basically the portal allows you to add multiple IP SSL bindings to an app, but it just doesn't work.

Hi @lorenpaulsen, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@lorenpaulsen : Thanks for your contribution to the Azure documentation! The author, @cephalin, has been notified to review your proposed change.

@lorenpaulsen Thanks for taking the time to make the change. And sorry that it took you time to figure it out. #sign-off


feat(BottomSheet): add suport dismissOnBackButton

Added functionality to not discard the sheet when pressing on the android back button

@vallemar very nice thank you !!!


Add guards where specific query param is required

E.g. list events and RSVP require calendar_id

Only event related endpoints currently would need this and API returns a nice error if calendar_id is not included.


[Support Request]: Update Error

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
Hello,
I tried to update from 21.10.3 to 21.11 and got the following error:
Script @php artisan package:discover --ansi handling the post-autoload-dump even                                      t returned with error code 1
$ ^C
$ php artisan cache:clear
php artisan config:clear
php artisan view:clear
In TestDatabases.php line 160:

  syntax error, unexpected ')'
$
In TestDatabases.php line 160:

  syntax error, unexpected ')'

Please find a screenshot here:
https://i.imgur.com/xQGX329.png
I don't know how to fix this. Other updates before were no problem.
Thanks in advance for any help.
Best, Daniel
Exact BookStack Version
21.11
Log Content
No response
PHP Version
7.4.26
Hosting Environment
Plesk Obsidian v18.0.39_build1800211117.18 os_Ubuntu 18.04

Hi @xdaniel2063,
Can you provide the exact output of the following two commands:
php -v
composer -V

Hello Dan,
here it is:
$ php -v
PHP 7.2.24-0ubuntu<IP_ADDRESS> (cli) (built: Oct 25 2021 17:47:59) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.2.0, Copyright (c) 1998-2018 Zend Technologies
    with the ionCube PHP Loader + ionCube24 v10.4.5, Copyright (c) 2002-2020, by ionCube Ltd.
    with Zend OPcache v7.2.24-0ubuntu<IP_ADDRESS>, Copyright (c) 1999-2018, by Zend Technologies

But when I run phpinfo it is this:
PHP Version 7.4.26
$ composer -v
   ______
  / ____/___  ____ ___  ____  ____  ________  _____
 / /   / __ \/ __ `__ \/ __ \/ __ \/ ___/ _ \/ ___/
/ /___/ /_/ / / / / / / /_/ / /_/ (__  )  __/ /
\____/\____/_/ /_/ /_/ .___/\____/____/\___/_/
                    /_/
Composer version 1.10.23 2021-10-05 09:44:27

Best, Daniel

Hi @xdaniel2063,
You likely have two versions of PHP installed on your system, where php7.2 is the default active for the command line.
You can often choose to run things with a specific version of php. For example:
php7.4 artisan cache:clear

Hello @ssddanbrown,
thank you.
/opt/plesk/php/7.4/bin/php artisan cache:clear
worked. Should I run all commands of the updates this way?

Should I run all commands of the updates this way?

Ideally, since php7.2 is not supported. Alternatively, if not in use anymore, you could look to remove php7.2 to prevent confusion.
Will therefore close this off.


Failure to plot more than 16384 points

Hi Sigplot team,
I am investigating an issue right now where Sigplot refuses to plot more than 16384 points (I am running in XY mode). I am going to continue to investigate, but if anyone on your side has any insight here in the meantime I would greatly appreciate it.
-Kevin

I am just a user, but I am able to display over 1.6 million points in 1D.  In 2D mode I can display 60,000 wide by 200 as a static raster.  I do have an older version of sigplot.  This may not be helpful since I'm not displaying in XY (not sure what XY mode is, unless that's for a surface).

Hey thanks for your engagement @kmun - yeah, sadly I have a feeling it is specific to my use of XY mode which allows for non-static xdelta (aka, arbitrary plotting of XY points)

I found that I was always ending up in this case:

https://github.com/spectriclabs/sigplot/blob/main/js/sigplot.layer1d.js#L264

So as new data was loaded it was never actually going and getting new data into and from the buffers.
The below (hacky feeling) change seemed to get me past that:
            if ((this.mode !== 'XY') && (imin >= this.imin) && (imin + npts <= this.imin + this.size) && (this.ybuf !== undefined)) {
                // data already in buffers
            }

by preventing me from dropping into that case while in XY  mode. But then the next boundary I seem to run into is at 32768 points with everything beyond that being cut off.
Will continue digging, but any and all help / comments are welcome!

I am assuming this comment is relevant here, but I can not determine where / how this referenced behavior is enforced in code:
https://github.com/spectriclabs/sigplot/blob/fa92189585cdc2daf0ea8de374ab48a5f8f214c2/js/sigplot.js#L8833

wellp. not sure if it is the best way to solve my issue but I just upped options.bufmax to 1e5 and then it worked 🤷♂


Send a string by default over microbit UART when program removed

When a Bluetooth component is added:

Start a UART service and write (microbit UART RX) onProgramRemoved code to send a string delimiter "#". Keep the starting and stopping delimiters for now. The whole message will read "$#|"

The MakeCode program will look for the "#" delimiter to act on the message sent when a program is removed.
No additional UI needed. Need to note its existence in the documentation, as well as the functions we added to abstract some of the data types for sending to and from the microbit.

OK, this is done. I tested in a simple case with a small change to the microbit code.

When a BLE component is removed, we send a microbit specific UART message. It looks like this: $#45|  where

$ is the start delimiter
# is the 'removal' delimiter
45 is the number of the program removed (so this can be any number)
| is the end delimiter

I added this bit in makecode to test it

Note this feature is specific to microbit UART RX.

@brettfiedler would you like to verify anything for this or are there any other changes needed here?


[REQUEST] DeepSpeed Zero3, gathering, updating and repartitioning gradients

For DeepSpeed stage 3, is there a way that I can gather gradients, update them and partition again, before moving them to the optimizer step function? this should be something similar to the GatheredParameters context. Can anyone please help with this?

same problem + 1


Dibi - date(time) columns return type

Hello @staabm
the dibi driver returns date/datetime/timestamp columns as \DateTimeImmutable object.
diff --git a/vendor/staabm/phpstan-dba/src/TypeMapping/MysqlTypeMapper.php b/vendor/staabm/phpstan-dba/src/TypeMapping/MysqlTypeMapper.php
index 43d929b70..548ba4927 100644
--- a/vendor/staabm/phpstan-dba/src/TypeMapping/MysqlTypeMapper.php
+++ b/vendor/staabm/phpstan-dba/src/TypeMapping/MysqlTypeMapper.php
@@ -10,10 +10,12 @@ use PHPStan\Type\IntegerRangeType;
 use PHPStan\Type\IntegerType;
 use PHPStan\Type\IntersectionType;
 use PHPStan\Type\MixedType;
+use PHPStan\Type\ObjectType;
 use PHPStan\Type\StringType;
 use PHPStan\Type\Type;
 use PHPStan\Type\TypeCombinator;
 use PHPStan\Type\UnionType;
 use staabm\PHPStanDba\QueryReflection\QueryReflection;
 use staabm\PHPStanDba\Types\MysqlIntegerRanges;

@@ -144,11 +146,13 @@ final class MysqlTypeMapper implements TypeMapper
                 case 'STRING':
                 case 'VAR_STRING':
                 case 'JSON':
+                    $phpstanType = new StringType();
+                    break;
                 case 'DATE':
                 case 'TIME':
                 case 'DATETIME':
                 case 'TIMESTAMP':
-                    $phpstanType = new StringType();
+                    $phpstanType = new ObjectType(\DateTimeImmutable::class);
                     break;
                 default:
                     $phpstanType = new MixedType();

I could make it work properly by these changes.
How do you think we could integrate this into existing codebase?
I was thinking i could pass third parameter into public function getResultType(string $queryString, int $fetchType): ?Type, eg ?string $calledFromClass = null
and call it from DibiConnectionFetchReturnDynamicType with self::class parameter
If you agree, I will open a PR

Since dibi supports more then just mysql we need to come up with something db independent.
I imagine other dbal layers having similar magic for other types as well.
I have to think about it .. suggestions welcome.

I will create PR with the logic described above for you to look at

Fixed with https://github.com/staabm/phpstan-dba/pull/480


dsp/window: refactor for GaussianComplex, revised test suite

As discussed on gonum-dev, this PR removes the GaussianComplex type, moving its Transform method
to Gaussian.TransformComplex.  It also implements TransformComplex for Tukey.
The test suite was re-written for improved clarity by disentangling the testing of real and complex windows.  Unfortunately, the same was not true of the parametric windows.  The asymmetry bothers me.
The tests rely on subtle Go behavior, which I added a comment for in the test file.  var a complex128 = 1 has value 1+0i.  It isn't insensible for the language to have that behavior, it just surprised me that untyped constants are valid as complex values.
I removed the test bodies that make a Values out of each window.  I do not see the utility in doing it twice -- the test already checks weights against a unity input.  It feels like the previous way mixed testing of Values and other windows which, to me, is not the way to go.
The new complex tests rely somewhat on (x + yi)(u + vi) = (xu-yv) + (xv + yu)i.  It has y=v=0 which zeros three of the terms.  It is modest busywork to change that, the process for which means updating the truth tables.
The test file defines five interfaces or types, which is the essential value to this PR:

type transformFunc func([]float64) []float64
type transformFuncComplex func([]complex128) []complex128

type transformer interface {
	Transform([]float64) []float64
}

type transformerComplex interface {
	TransformComplex([]complex128) []complex128
}

type dualTransformer interface {
	transformer
	transformerComplex
}

The previous Transform([]complex128) []complex128 interface on GaussianComplex overlapped with the Transform method on Gaussian.  This made the dualTransformer interface in the test file impossible.  The composability is not so important as the improved clarity and homogeneity with the non parametric windows, to me.

It looks like my choice of branching commit lead to a big overlap with the other PR and a mess in the commits :(
Only 53373c14559a4d268ae33ecde7844f9c943da359 is relevant

Yeah, branch management is important. I can look at that commit, but this won't merge as it is due to the conflicts. Can you please clean that up?.

Yeah, branch management is important. I can look at that commit, but this won't merge as it is due to the conflicts. Can you please clean that up?.

How?  I chose this branch point specifically because if the previous PR and this fanned out, there would be a conflict.  The extra commits here -- I think -- will evaporate once gonum/gonum:master contains them.

The previous PR has been merged. There are conflcts with that merged PR. I could resolve those with the GitHub UI, but I won't because it is a terrible experience and you better understand the changes where the conflicts exist.
To clean up the conflicts, either rebase this branch onto master and resolve the conflicts that arise or make a temporary duplicate of this branch reset this branch hard to master and then cherrypick the relevant commit onto that.

Tone does not carry well in purely textual communication; is my read of hostility in your words as you intend?

No, not intended at all, sorry if it came across that way; the emphatics were to counterpoint the claims that the merging of the previous PR would resolve the merge conflict. The "terrible experience" is attempting to resolve conflicts with the GitHub UI and I don't think in general that resolving conflicts should be done by third parties if avoidable given that it depends on the intention of the merging author.
I can understand the reason for you having chosen to do what you've done with the branches, but I think the squash merge brought that undone.

I've had a look through the relevant commit and I'm not convince that the testing changes are an improvement overall. We lose testing of NewValues* and, significantly, that using a Values* gives identical results to using the window functions directly (this is a design goal of the package since using a precomputed window is a significant performance benefit in tight loops), we also now have two sources of truth for the window functions (one for real and one for complex windows) that are readily interchangeable (I know I said this - I was wrong).
My preferred option would be just to do the work on the Gaussian and Tukey types placing them in a new file, "window_parametric.go" and then clean up the changes that were deferred in #1437. I would also roll the parametric tests into the others using received-curried method functions, e.g.
	{
		name: "Gaussian_1.2", fn: Gaussian{1.2}.Transform, fnCmplx: Gaussian{1.2}.TransformComplex,
		want: []float64{
			0.730981, 0.778125, 0.822578, 0.863552, 0.900293, 0.932102, 0.958357, 0.978532, 0.992218, 0.999132,
			0.999132, 0.992218, 0.978532, 0.958357, 0.932102, 0.900293, 0.863552, 0.822578, 0.778125, 0.730981,
		},
	},

In spending some time thinking about the package again, there is potential room for improving the complex window functions' performance by replacing direct complex multiplies with explicit scalar multiplication, e.g.
func SineComplex(seq []complex128) []complex128 {
	k := math.Pi / float64(len(seq))
	for i, v := range seq {
		f := math.Sin(k*(float64(i)+0.5))
		seq[i] = complex(f*real(v), f*imag(v))
	}
	return seq
}

This should be a separate PR from the work above and should have benchmarking to support it (though it has correctness benefits in rare cases; when the sequence element is Inf which probably justifies it without benchmarking).

withdrawn


Page with read-only warning.

Theres need for page with info about read only demo. We want to move it from dashboard panel info separate view.

A) gallery - with randomly displayed pictures
B) full screen (a bit in  wetransfer spirit) 4 to 8 random illustrations with Saleor Pirate and things,
every time user click sth without action will see different pics
Thoughts?

Feature implemented.
Try changing password to get awesome splash screens.


fix: warnings upon importing qsharp

fixes #185
Importing the qsharp python package in the Jupyter notebook example
resulted in various warnings since the Microsoft.Quantum.Chemistry Q#
package was not loaded yet.
This is fixed by instructing to autoload the package in the Q# project
file.

ping @guenp

Just out of curiosity - how long does it usually take for these jobs to start?

/azp run


Add info panel to complexity section

similar to the asset costs, explaining how the value is calculated.

@kamransaleem


Initial sketch for presenting cedar as cedar-10

We discussed various ways to do this, and it seems
the least obtrusive way is to do this at the boundaries
(i.e. dashboard / cli) as opposed to changing internals.

lgtm


Ruff: enable all pyflakes and perf rules

Summary of changes
Setuptools conforms to all the Ruff-implemented Pyflakes and Perflint rules except PERF203 and a single instance of PERF102. So we can be less picky about their selection.
PERF203 is worth looking at, but is often a micro-optimisation with false-positives, let's not block the CI on that. I applied it on some tests.
Pull Request Checklist

[x] Changes have tests (these are new static tests)
[x] News fragment added in newsfragments/. (no public-facing change)
(See documentation for details)

It's been a couple PRs where the PyPy test gets randomly cancelled, that's weid

Thank you @Avasam.


Japanese dictionary

When translating Ajin we needed to look up some Japanese words, so this feature could be handy sometime in the future.
For example, assuming ! is the command prefix, one could look up the meaning of a Japanese term like this:
!translate 教室

The command name (translate) is subject to change.

This feature could be implemented by querying other sites like Tanoshii Japanese.

Implemented into go-lolicon using http://jisho.org/.


Improvements to Slack Reader

Feature Description
Wondering if the Slack Reader should be improved. The context doesn't seem to have the real channel name nor the users names who post there.
Context information is below.
---------------------
channel: CAMPGT6R3

hi

<@U062YH5FXCG> has joined the channel
---------------------
Given the context information and not prior knowledge, answer the query.
Query: <query here>
Answer:

Reason
No response
Value of Feature
No response

Thanks for the suggestion @mulka ! The Slack reader probably needs some love.
AFAIK, we have a more up-to-date version via LlamaHub: https://llamahub.ai/l/slack
Maybe check if that satisfies your use-case.
We always welcome community contribution to continuously improve these!


Better Way of Server's sending Errors

All the error responses: are unclear and and difficult to use.
Solution
An example to improve the response is:
{
  "error": "A detailed error message",
  "publicError": "A less detailed error message",
  "code": 400
}

 Better Way of Server's sending Errors


Failed to handle the resource file on Win7+VS2013

Maybe the path for rc.exe is different on the other OS like Windows XP. I just tested this path on Win7 x64.

Sorry, your PR somehow fell off my radar. I've incorporated your suggested changes in https://github.com/dlang/visuald/commit/a09c183c0127f800fa656b801b57523de55ff53f.

It's so sad. On Windows 10.0.15063 + VS 2017, the path for rc.exe is C:\Program Files (x86)\Windows Kits\10\bin\x86\rc.exe. VisualD-v0.45.0 can't locate it.

The default has changed for new installations, but Visual D doesn't tamper with existing settings. You can verify this by starting VS with a different configuration suffix:
"c:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\devenv.exe" /RootSuffix Test

To fix it for your installation, just add \x86 to the executable search path entry $(WindowsSdkDir)bin.


How does "go to definition" exclude specified directories, such as "./history"?

How does "go to definition" exclude specified directories, such as "./history"?

Add search exclude in your settings : "search.exclude": { "**/.history": true },
(But this doesn't work with all extensions like php #40)
Move .history in another specific folder using local-history.path


fix arrays

Codecov Report

Merging #4 (844f8da) into main (ffe2839) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head 844f8da differs from pull request most recent head 426acba. Consider uploading reports for the commit 426acba to get more accurate results

@@          Coverage Diff          @@
##            main      #4   +/-   ##
=====================================
  Coverage   8.69%   8.69%
=====================================
  Files          3       3
  Lines        161     161
=====================================
  Hits          14      14
  Misses       144     144
  Partials       3       3

Impacted Files
Coverage Δ

sdk/src/ssi/crypto.go
0.00% <0.00%> (ø)

sdk/src/ssi/did.go
17.02% <0.00%> (ø)

sdk/src/ssi/util.go
17.64% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

One more capitalized Items in VerificationMethodSetArray. Other than that, looks good! I don't have permissions to approve PRs in this org.

@amika-sq updated and added you as a maintainer here. will merge.


ImportError: cannot import name 'SerializationError' from 'azure.core.exceptions'

Hi,
I updated both azureml-sdk and azureml-core to their latest version and still get the same error.

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

ID: f14a0671-51a0-d24f-fcdb-0177703bf507
Version Independent ID: 442c80bf-177d-6667-4929-e93666a75cc7
Content: Troubleshoot SerializationError - Azure Machine Learning
Content Source: articles/machine-learning/how-to-troubleshoot-serialization-error.md
Service: machine-learning
Sub-service: core
GitHub Login: @Blackmist
Microsoft Alias: larryfr

@zanakr
Thank you for bringing this to our attention.
I've delegated this to content author @Blackmist, who will review it and offer their insightful opinions.

@Blackmist
Could you please review add comments on this, update as appropriate.

@zanakr Thanks for contacting us with this feedback.
Unfortunately, this github repo is for documentation feedback and we can't provide assistance troubleshooting errors. This document provides information on known causes and resolutions for this error. If you're encountering this error and these steps aren't helping, contact Azure support for assistance.
If, through support, a new cause/resolution for this error is determined, the support group or product group will contact my team with the details and we can update the article with that information.
#please-close


fix csv precision

dbt 0.9.2 RC had a bug where we miscalculate the precision required for decimal values like 0.001. we were attempting to create a numeric(1,3) which doesn't make sense -- according to the Postgres docs:

NUMERIC(precision, scale)
The scale of a numeric is the count of decimal digits in the fractional part, to the right of the decimal point. The precision of a numeric is the total count of significant digits in the whole number, that is, the number of digits to both sides of the decimal point. So the number 23.5141 has a precision of 6 and a scale of 4. Integers can be considered to have a scale of zero.

So the precision should be 4, not 1. This branch fixes that.

Used the file in #683 to test, also wrote unit tests for that max_digits fn

@cmcarthur lgtm! Nice job on that test

not a fan of that code climate "duplicate code" check that's currently failing btw. duplicate code is not always bad, trying to solve it often makes for more complicated code.

right there with you @joevandyk -- sometimes no abstraction is better than a bad abstraction

I confirmed this fixed my issue with loading the csv file.  I think getting rid of the limits is a good idea, as it allows for the truncate/insert strategy with different lengths of data.


ci-11791

Add ffs for dlc

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65fc684a64b8ad40a0fbe800--harness-developer.netlify.app


TypeScript - Property is missing in type DndComponent

Hi,
Thanks for the react-dnd library, very nice work.
I have the following two typing issues with the following sample code. I appreciate if someone could tell me what am i doing wrong ?
export interface TestProps {

}
@DropTarget<TestProps>("TEST", {
    drop(props: TestProps) {
    }
}, (connect, monitor) => {
    return { dragTarget: connect.dropTarget() };
})
export class TestComponent extends React.Component<TestProps, undefined> {
    constructor(props: TestProps) {
        super(props);
    }
    someHelper() {
        return (
Test
);
    }
    render() {
        return (this.props as any).dragTarget(this.someHelper());
    }
}

Is there a better way of doing
        return (this.props as any).dragTarget(this.someHelper());
The following code produces the error below, any ideas?
 TS1238: Unable to resolve signature of class decorator when called as an expression. Type 'DndComponentClass<TestProps>' is not assignable to type 'typeof TestComponent'.  Type 'DndComponent<TestProps, any>' is not assignable to type 'TestComponent'.    Property 'someHelper' is missing in type 'DndComponent<TestProps, any>'.

Thanks

Any luck? I get the same issue error on @DragDropContext.

same here, I only managed to use constructor and render method inside Component. Any other function (like componentwillreceiveupdateprops) are throwing this error

if you uninstall the typings for react-dnd, the error is gone and it works. However, of course, you loose intellisense, just a workaround

I'm having the same issue. Anything added inside the React Component class causes an error - even the render function. An empty class is the only thing that doesn't produce an error. As far as I can tell it happens with DragSource, DropTarget, and DragDropContext decorators.

Yeah, I have the same issue.. has anyone solved this?

I gave up on the decorator style. I just use them as functions. You can use the lodash flow function to avoid the nesting.

@sundriver , @lukescott , I do face this issue, I removed decorator. Still component method not defined issue arise. Could you please share you sample code.
Interface Props {
    // tslint:disable-next-line:no-any
    connectDropTarget?: any;

}

interface InjectedProps extends Props {
  dashboardStore: DashboardStore;
}

const dashboardDropTarget = {

    // tslint:disable-next-line:typedef
    drop(props, monitor, component) {
        const item = monitor.getItem();
        component.addDropedItem(item);
        return { moved: true };
    }
};

// tslint:disable-next-line:typedef
function collect(connect, monitor) {
    return {
        connectDropTarget: connect.dropTarget(),
        canDrop: monitor.canDrop(),
        itemType: monitor.getItemType()
    };
}

@inject('dashboardStore')
@observer
class EditDashboardGrid extends React.Component <Props, {}> {
    static contextTypes = {
        addDropedItem: PropTypes.func
    };

    dashboardStore: DashboardStore;

    constructor(props: Props) {
        super(props);
        this.dashboardStore = this.injected.dashboardStore;
    }

    get injected() {
        return this.props as InjectedProps;
    }

    // tslint:disable-next-line:no-any
    addDropedItem(item: any) {
        alert(item);
    }
    render() {

        if (!this.dashboardStore.isLayoutLoading) {
            const { connectDropTarget } = this.props;
            return connectDropTarget  (

            );
        }
        return (<Loader/>);

    }
}
export default DropTarget(['WIDGET', 'SIGNAL'], dashboardDropTarget, collect)(EditDashboardGrid);

Thank you.

I don't use react-dnd anymore. I bit the bullet and implemented my own dnd solution as react-dnd was overkill for my needs, and I needed more direct control. But my code looked something like this:
export default
flow(DropTarget<Props>(...), DragSource<Props>(...))(
class Foo extends React.Component {
// ...
})


chore(deps): Bump object_store to 0.10

Close #4588

Would you like to create a list of unsupported features to help us track them more effectively?


Extending pool with X units, double the existing units + X units

Env: testnet
Branch: development_capacitypool

Expected Result
cus = 8
sus = 8

The extend object

shouldn't be an issue anymore.


Factor CONTRIBUTING.md out of README.md.

As the title says -- give this project a CONTRIBUTING.md file.

Coverage remained the same at 100.0% when pulling ea7a6b67fde1dea1c78965ab9768cd90234353e7 on contributing into 375afc39465403da4e365917bc7cb94e2e21c41a on master.


Fix some tests

Sadly, not all of the tests pass yet, but this is progress I hope.

Thank you!
There is a thing about raise ValueError('missing file_ argument in get_thumbnail()') we need to silent that in DEBUG=False probably, but maintain the error. It's a common problem.
I was thinking to enforce this in the api, instead of the Base image class; however, it worries me to add a file check, as it may run multiple disk reads

Hmm, I admit I don't quite understand what the ValueError is for. It seemed to me that this was the only way to get {% empty %} to work as intended when the image variable was an empty string. The commit makes test_empty in tests/thumbnail_tests/test_templetags.py pass, does the test need to be changed?

I merged and rebase all the commits in this pull request, except for the one about missing file__ argument.


Open RocksDB too slow

When there are a large number of data, recover from manifest takes a long time.
➜  [115] db du -sh ./
878G	./

2016/10/23-13:09:32.172179 7f97193f8a40                Options.report_bg_io_stats: 0
2016/10/23-13:13:32.557831 7f97193f8a40 Recovered from manifest file:/data/var/store/db/MANIFEST-101922 succeeded,manifest_file_number is 101922, next_file_number is 142492, last
_sequence is<PHONE_NUMBER>, log_number is 0,prev_log_number is 0,max_column_family is 3

@hhkbp2 handled this before https://github.com/pingcap/tikv/pull/1079
Can it not work?

@zhangjinpeng1987
What is the actual size of the manifest file, and the corresponding start time?

12MB

@zhangjinpeng1987 If it takes a number of minutes for the db to re-open, it seems all right to me.
If a short re-open time is needed, please modify the max size of the manifest file to a smaller value.

https://github.com/pingcap/tikv/pull/1222 fix this issue


Convert the various colors in web/viewer.css to CSS variables

To make re-skinning the viewer easier, the various repeated colors in web/viewer.css should be converted to CSS variables. This way they should be easier to tweak for third-party deployments of PDF.js.

Would like to work on this!

Would like to work on this!

As already mentioned in https://github.com/mozilla/pdf.js/issues/11572#issuecomment-583142672, this would be best done as part of other CSS modernization/improvement work:

Since there's already issues tracking the modernization/clean-up of the viewer UI, it'd probably make the most sense to introduce CSS variables as part of these improvements (which really need to happen iteratively, since changing the entire UI in one big commit is essentially impossible to test/review).

@Snuffleupagus Got it!

I am new to mozilla. Can I work on this?

Fixed by #11077.


⚠ Cioos National CKAN has degraded performance

In 4b23246, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 9101 ms

Resolved: Cioos National CKAN performance has improved in cc45a59 after 5 minutes.


[CSV node] Add support for parsing empty strings and null values

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes
Database replication use case
Current databases state
Database 1

name
e-mail
Language
Country

tmdoit
<EMAIL_ADDRESS>PL
Poland

Database 2

name
e-mail
Language
Country

tmdoit
<EMAIL_ADDRESS>PL
Poland

After "Database 1" update
Database 1

name
e-mail
Language
Country

tmdoit
""
NULL
NULL

Database 2

name
e-mail
Language
Country

tmdoit
<EMAIL_ADDRESS>PL
Poland

To reflect changes in "Database 2" I use CSV string: '§,"",,' where:

§: means field doesn't changed
"": means field is set to empty string
,,: means field is set to NULL value
,: means field is set to NULL value

Now using CSV node it's impossible to get null values and empty strings - in both cases there is no value returned.

Checklist

[ ] I have read the contribution guidelines
[x ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.02%) to 77.498% when pulling 305b2609fa22a0a339ef7e386db38d0104ead02d on digitaloak:dev into 6912dec166e23e38d9fc1e0a626512b4ca988358 on node-red:dev.

Thanks - there was no need to change all the variable names - just the labels/translations - but thanks !


Scope based authorization with Keycloak 4

Background
After version 4, the specification around scopes/roles was changed.
"Client Scope" was newly created.
And if a client specifies scopes in the query string when requesting an access token, these scopes are included in the "scope" claim.
Before Keycloak 3:
  "realm_access": {
    "roles": [
      "foo", "bar"
    ]
  }

or
  "resource_access": {
    "someone": {
      "roles": [
        "foo", "bar"
      ]
    }
  }

After Keycloak 4:
  "scope": "foo bar"

Of course, we can still specify roles dynamically in Keycloak 4 by relating the client scope and roles in the Scope tab in each Client Scope.
However, since the scope was distinguished clearly from the role in Keycloak 4, we should basically use the "scope" claim for scope based authorization.
Concerns
How to deal with this.

create a new policy like "keycloak scope check" policy
improve the existing "keycloak role check" policy
do nothing for now

@y-tabata I'd try to change current Keycloak Role Check policy into generic JWT claim check policy, that you could tell which claims to check and how to treat them (split string by spaces etc.).
That should be generic enough to cover both Keycloak 3 and 4 case as well as other IDPs too.

@mikz since it's keycloak original specification that access token includes scope information, if you'd like to cover other IdPs, you may need to get scope information from introspection endpoint.

@y-tabata I think that the JWT claim check policy introduced in https://github.com/3scale/APIcast/pull/1070 should cover your use case. Let us know if it does not.


Bridgy source should rename blacklist and similar offensive terms

Bridgy had a file called domain_blacklist.txt which was renamed to domain_blocklist.txt in a recent commit. This issue is for searching / tracking any other similar terms as identified by Microsoft in Chrome Issue 981129.
While we don’t have access to Microsoft’s PoliCheck tool mentioned therein, we can at least use the results they came up with in their searches of Chrome source as a first level approximation of terms to search for. E.g.:

change "master"/"slave" to "writer"/"reader"

Other terms may require manual inspection for context to evalute whether they are offensive or not. E.g.:

cracker
ho

Feel free to note other text profanity, geopolitical, or diversity issues in comments on this issue as discovered by PoliCheck analysis or other tools. (Please refrain from general offensive brainstorming though, instead, link to existing work and lists by others. Thanks.)
(Originally published at: https://tantek.com/2020/008/b1/bridgy-source-rename-offensive-terms)

thanks again for the nudge!
i've confirmed that none of the words mentioned here appear in bridgy, granary, oauth-dropins, or webutil. i've also checked for the standard english curse words, and only found a couple instances of hell in test data in granary:
https://github.com/snarfed/granary/blob/2f62e7ff6ccfed5d65c2db09241a77698dfa2277/granary/tests/test_facebook.py#L1157
https://github.com/snarfed/granary/blob/2f62e7ff6ccfed5d65c2db09241a77698dfa2277/granary/tests/testdata/facebook.m.post.html#L226
as mentioned, PoliCheck is private, and similar public tools i've looked at focus more on profanity, as opposed to offensive words. so, i'm tentatively closing, but i'm happy to reopen if anyone wants to suggest a specific tool and usage details. (or even better, feel free to run it and send pull requests!)