Manipulating strings of size greater than ~1.5 Gb consumes all available resources

I don’t know if it's a bug or not, but I was able to write a PowerShell script that consumed all available CPU time and memory.
When I ran the script my mouse pointer got slower and slower and eventually completely stopped. Both mouse and keyboard became unresponsive and the only thing a could do was pressing the reset button.
I suspect that it happened when the script’s internal string (one of its variables) got about 1.5 Gb in size.

How much RAM do you have?

What are you doing, and what do you expect to happen?

I didn’t expect that a PowerShell script that doesn’t have instructions increasing the priority of its process is able to make a computer freeze.

The key thing with strings is that they are, fundamentally, immutable. .NET does not allow you to modify a string. All string operations involve copying portions of or the entire original string, and then saving the new string to a new memory address.
So if you're doing repeated operations on very large strings, it's certainly possible to end up consuming all available resources. Strings aren't designed for that kind of operation to happen repeatedly with enormous strings.
This isn't really a PowerShell problem, it stems from .NET itself. But really, it's probably moreso that you're trying to use tools that weren't designed for the kinds of operations you're trying to do.
If you need to manipulate that much data in detail you're probably better off managing the strings as [char[]] arrays, being careful to allocate enough space to do what you need from the beginning. There is also the option of StringBuilder which is much more efficient for building strings without excessive allocation.
The tool you should use depends on exactly what you're needing to do, though.

What surprised me the most is that a script was able to make entire Windows completely unresponsive. If it was only the PowerShell terminal got unresponsive, I’d never consider that situation worth reporting. So, the question is:
Can a trivial PowerShell script make Windows completely unresponsive?
$bigString = "" $aFiles = Get-ChildItem -Path "a folder with a lot of files" -File foreach ($aFile0 in $aFiles) {     $afc0 = Get-Content -Path $aFile0 -Raw     $bigString = -Join($bigString, $afc0)   } } Set-Content -Path "OutputFile.txt" -Value $bigString -Encoding Unicode

Can a trivial PowerShell script make Windows completely unresponsive?

Yeah for sure it's possible.

You can try to wrap the script with
$currentProcess = Get-Process -id $pid
$originalPriority = $currentProcess.PriorityClass
$currentProcess.PriorityClass = 'Idle'
try {
  # your code goes here
}
finally{
   $currentProcess.PriorityClass = $originalPriority
}

But your real problem is that you have a fundamentally inefficient way of solving your original problem.
A red flag should always go up when you see code like
$bigString = -Join($bigString, $afc0)
or
$bigString += "some more text"

because, as @SeeminglyScience explained, strings are not designed to be mutated. Other data structures should be used.

For example:
Clear-Content OutputFile.txt
$aFiles = Get-ChildItem -Path "a folder with a lot of files" -File
foreach ($file in $aFiles)
{
    $fileContent = Get-Content -Path $file -Raw
    Add-Content -LiteralPath OutputFile.txt -value $fileContent -NoNewLine
}

Should do the same thing. If any single file is larger that available memory, you could review stream-based solutions.

because, as @SeeminglyScience explained, strings are not designed to be mutated. Other data structures should be used.

(that was @vexx32 btw)


wuapi scanned update id's are not found on wsus / microsoft update catalog

Hello,
i've got the same problem. My client reports some UpdateIdentity.UpdateId 's (which are detected via wuapi). But it look's like that these Id's are not used to find an Update on WSUS or https://www.catalog.update.microsoft.com/.
Client scan
$session = (New-Object -ComObject 'Microsoft.Update.Session')
$history = $session.QueryHistory("",0,50)

Scan with an ms-wsusss implementation and also check id on microsoft update catalog
used endpoint: https://sws.update.microsoft.com

Originally posted by @Kilika in https://github.com/MicrosoftDocs/windows-itpro-docs/issues/4327#issuecomment-926393061

@Kilika Which article are you referring

@RAJU2529 https://docs.microsoft.com/en-us/windows/client-management/mdm/device-update-management this one.

Hello,
i've got the same problem. My client reports some UpdateIdentity.UpdateId 's (which are detected via wuapi). But it look's like that these Id's are not used to find an Update on WSUS or https://www.catalog.update.microsoft.com/.
Client scan
$session = (New-Object -ComObject 'Microsoft.Update.Session')
$history = $session.QueryHistory("",0,50)

Scan with an ms-wsusss implementation and also check id on microsoft update catalog
used endpoint: https://sws.update.microsoft.com

Originally posted by @Kilika in #4327 (comment)

The IDs are global, valid in both WSUS and MU Catalog, but the updates were expired from the Catalog.
Please use an update that is more recent and also available in the Catalog.

Maybe, ideally we shouldn't keep superseded updates for too long:
https://aka.ms/wsusmaintenance
Please share if you have a more recent example of an update ID/title/KB which is not found.


git-branchless: 0.3.9 -> 0.3.10

Description of changes

New upstream release: https://github.com/arxanas/git-branchless/releases/tag/v0.3.10
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 166013 run on x86_64-linux 1

1 package built:

git-branchless


Does the instruction FLI.S need to exhibit floating-point values at the assembly level?

hi @aswaterman ，
I have some doubts about the fli instruction.
I'm not sure. If the number 0-31 is used instead of the value  in Chapter 25 of "Table 25.1:  Immediate values loaded by the FLI.S instruction", how can this reflect the actual floating point of the load at the instruction level? Is it more appropriate to use fli.h ft1, -1.0 instead of fli.h ft1, 0?
Perhaps both options should be taken into account. While supporting the lookup of the values in the table by passing 0-31(RS1), it should also support the loading of accurate floating-point values to reflect the actual floating-point operation at the assembly level.
I think it may be more appropriate to embody the specific decimal floating-point number at the assembly level.
This has a similar example on arm architecture.
ARMv8 Instruction Set Overview

Looking forward to your reply, thanks :)

hi @a4lg
This seems to be different from what I understand :)

@jinma-work
Current draft spec says:

The preferred assembly syntax for entries 1, 30, and 31 is min, inf, and nan, respectively. For entries 0 through 29 (including entry 1), the assembler will accept decimal constants in C-like syntax.

Preferred syntax supports:

Decimal constants (0-29)
min (for 1)
inf (for 30)
nan (for 31)

and my zfa branch supports all of these (and any constant expression that are evaluated to 0-29 as a GNU extension).
So, I think @aswaterman is pointing that 30 and 31 are not supported.

I think that the confusion is about what "decimal constant" means.  The assembler syntax is to use the decimal value in the "Value" column and not the 5b decimal number in the rs1 column.  There are special cases for the three entries that don't have decimal representations (infinity and NaN) and the minimum positive normal number (which might not have a finite decimal representation, I'm not sure).

What @pdonahue-ventana said.  I didn't think the spec was ambiguous, but if you have a suggestion for how to clarify it, I'm all ears.

the minimum positive normal number (which might not have a finite decimal representation, I'm not sure).

A rational number $p/q$, in lowest terms, has a terminating expansion in base $b$ if each prime factor of $q$ divides $b$. In the case of a binary floating-point number, $q$ is a product of $2$s, and $2$ divides $b=10$. (In the case $q=1$, it's an empty product, so divisibility holds vacuously.)

I think the minimum positive normal numbe :

When the type is _Float16 , it is equal to 6.104E-5.
When the type is _Float32 , it is equal to 1.1754944e-38.
When the type is _Float64 , it is equal to 1.1754943508222875E-38.
I got it through floating-point binary, and although I don't know if my understanding is correct, I think the compiler output assembly should include similar specific decimal floating-point values.

@jinma-work I don't agree.
Despite that it is technically possible, it suffers from small numbers (min and around... 2 through 5) because of interpretation of inexact floating point numbers.

@aswaterman @a4lg
Well, then I can understand that, except for a few special cases, should the rest be represented using decimal floating-point values? For example fli.h ft1, -1.0 , fli.h ft1, 0.0625 , etc.

min (for 1)
inf (for 30)
nan (for 31)

:)

@jinma-work Mixing different but looking similar semantics - table index numbers and actual numbers are definitely what I don't want to proceed (your original proposal I don't agree - using actual numbers (only) is far better than that).
For instance, such confusing cases will happen:
fli.s fa0, 2    # to load the value by index 2 (actual value: 2^(-16))
fli.s fa0, 2.0  # to load the value 2.0 (table index 25)

When the type is _Float64 , it is equal to 1.1754943508222875E-38.

The minimum value for _Float64 should be 2.2250738585072014 × 10−308 with a lot more decimal places.

@topperc Yeah, I realized you were right :)
In addition, @a4lg can we consider the binutils of zfa to be combined into the upstream? As far as I know, llvm is basically combined into the upstream, except for the FLI instruction.

@mar-jin
If that's possible, i would but... currently, upstream merge to Binutils / GCC requires an extension to be at least frozen.  There's a discussion about relaxing the requirement but I think this is not happened yet.

@mar-jin If that's possible, I would but... currently, upstream merge to Binutils / GCC requires an extension to be at least frozen. There's a discussion about relaxing the requirement but I think this is not happened yet.

My llvm-mc patches for Zfa have been accepted and merge into llvm upstream.
https://reviews.llvm.org/D140460
https://reviews.llvm.org/D141984
There are some differences between my implementation and your binutils/gcc implementation.
First, it seems that you didn't support 30 and 31 as an index number for FLI while llvm-mc can accept.
Second, as @topperc said, the minimum positive normal for _Float64 should be 2.2250738585072014 × 10−308. It seems that Binutils didn't add support for these minimum values.

@joshua-arch1
LLVM experimental support is helpful but... really?  Supporting two different format (some values conflict!) seems too confusing as I mentioned earlier and I felt only supporting table index is sufficient.

I removed the index support from LLVM due to it being ambiguous. Only floating point values with decimal point are supported.

@topperc Okay, then (I am not a fan of floating point immediates but) I can understand the LLVM's position.
To be sure, I want at least @aswaterman and Kito to hear about what operand we should support for FLI (table index [as my GNU Binutils' test patchset] or floating point immediates [as LLVM's experimental support]).

@a4lg Sorry that I haven't had time to revisit this thread recently.  I've spoken recently with both @topperc and Kito on this topic, so my input is probably redundant with theirs.

@aswaterman @majin2022 @topperc
Okay, I'll reflect this later (I will make my GNU Binutils patchset to support floating point immediates instead).


Create API endpoint to return list of blog posts for blog page

Removing for now because can be achieved with static json file


Add meta option

Only reason I still keep around a custom template

could this also be a key value pair?

A single meta tag can have one or more attributes, so not really

I was just thinking because we could set meaningful defaults if it would be named... but right now I don't have a good idea for that..
Maybe we can come up with something - otherwise I will merge this one

Here is a list of all possible header values:
https://github.com/joshbuchea/HEAD#meta

What about:
{
   'charset:utf8': true,
   'http-equiv:x-ua-compatible': "ie=edge",
}

to get
<meta charset="utf-8">
<meta http-equiv="x-ua-compatible" content="ie=edge">

How would you infer content for example in that sample? Some sort of list of specific attributes?
This implementation works for every possible permutation of the meta tag, at the expense of a slightly clunky API. But the logic is minimal, and very easy to grok.

I'm with you on making it easy to add these (I think this should perhaps remove the ones from the default template, and add them through this API), but the current approach represents a simpler implementation to understand and maintain.

Yes the api is kind of ugly ;)
According to the list https://github.com/joshbuchea/HEAD#meta every setting is placed into content.
   {  'http-equiv:x-ua-compatible':                "ie=edge" }
               ↑                    ↑                               ↑
<meta  http-equiv="x-ua-compatible" content="ie=edge">

So the key could ble split on colon, and the value could be placed in content?  If so, I very much agree :smile: I took a look at how meta works now, and it seems to be the case

So you want
new HtmlWebpackPlugin({ meta: { 'charset:utf-8': null, 'http-equiv:x-ua-compatible': 'ie=edge' } })

?

Yes I guess that would be nice - maybe we use = instead of :
new HtmlWebpackPlugin({ meta: { 'charset=utf-8': null, 'http-equiv=x-ua-compatible': 'ie=edge' } })

what do you think?

@jantimon Updated now.

Hmm this is quite nice and you are right we should probably push it to the beginning of head.
The change is not backwards compatible and everyone who is using a custom template will have to adjust the template or the config.

We could drop changing the default template here, and release that as a separate thing (keep a PR open with the change until you're ready to release a v3

Maybe we should just not use defaults and go with your initial proposal.

Hmm but then the base template would be mixed

What do you want to happen? 😄

Hmm I really like the idea but I would like to try a different approach let me prepare a draft

Okay sorry seems to take longer than I hoped because of https://github.com/webpack/webpack/issues/2978

Any status on this ?

Since v3 is coming, maybe merge this as is?

FWIW, I still prefer the original API, with an array of objects. Clean and easy, no inferrence or magic delimiters

Yeah magic is quite bad and this would be a good feature for 3.x
What about a mix of both of the ideas so we get defaults, an easy configuration for name:content pairs and full power if needed:
  {
      // <meta charset="utf-8">
      charset: { charset: 'utf-8' },
      // <meta name="google" content="notranslate">
      // <meta name="google" content="nositelinkssearchbox">
      google: ['notranslate', 'nositelinkssearchbox'],
      // <meta name="referrer" content="no-referrer">
   }

if the value is an object we ignore the key (but we can set a default)
if the value is an string or array we use the key as name and the value as content

Why not just simply a [Template] String ?
// {String}
new HTMLPlugin({
  meta: `
    <meta charset="utf8">
    <meta ...>
 `
})

// {Function}
new HTMLPlugin({
  meta: (ctx) => `
    <meta charset=${ctx.charset}>
    <meta ...>
    <meta ...>
 `
})

Sorry for postponing the topic for so long.
The request came again in #897 .
What about the following structure:
{
  charset: { 'charset': 'utf-8' },
  viewport: 'width=device-width, initial-scale=1, shrink-to-fit=no',
  'Content-Security-Policy': { 'content': 'default-src "self"', 'http-equiv': 'Content-Security-Policy'},
}

This would allow a very extensible api with defaults.
The default might be (here I need some feedback):
{
  charset: { 'charset': 'utf-8' },
  viewport: 'width=device-width, initial-scale=1, shrink-to-fit=no'
}

And could easily be overwritten e.g.
new HtmlWebpackPlugin({
  meta: {
    viewport: false,
    // or
    viewport: width=500, initial-scale=1
  },
}

And the default could easily be extended:
new HtmlWebpackPlugin({
  meta: {
    referrer: 'no-referrer'
  },
}

The following function would convert it internally into the structure you proposed initally:
function getMetaTagArray(meta) {
  if (meta === false) {
    return [];
  }
  const metaObjects = Object.entries(meta).map(([metaName, value]) => {
    return (typeof value === 'object') ? value : {
      name: metaName,
      content: value
    };
  });
  return metaObjects;
}

Hah, completely forgot about this. 😀
How would you do meta tags without name, like <meta http-equiv="X-UA-Compatible" content="IE=Edge"> or Open Graph tags: <meta property="og:image:width" content="1200"> with that API? I suppose extra name doesn't hurt (although I don't know), like your example with charset would be

@SimenB haha me too 😉
Just like the 'Content-Security-Policy' example:
{
  'og:image:width' : {
     property: 'og:image:width',
     content: 1200
  }
}

I created an interactive example to play around:
https://codepen.io/anon/pen/RMprbX?editors=0010

Ah, so if the value is an object, that's used as is, but if the value is a string, its key becomes name? Sounds good 🙂

What about having an array directly: less moving parts, easier for the user to understand
new HtmlWebpackPlugin({
  meta: [
    {
      "charset": "utf-8"
    },
    {
      "name": "viewport",
      "content": "width=device-width, initial-scale=1, shrink-to-fit=no"
    },
    {
      "property": "og:image:width",
      "content": "1200"
    }
  ]
}

A bit longer tough.
Either way I'm fine with the object based one :)

Yeah, that's the API of my original commit if you look at the history. 🙂 FWIW I still think that'd be the cleanest solution, but I have nothing in particular against @jantimon's latest suggestion either

I totally agree that the array looks easier but it’s harder to modify as it it number based.
So we could not provide any defaults or offer only a all or nothing preset.

Moved to #906.


Unable to build lumen on Arch.

Hey all, tried building from master on Arch
Linux devbox 5.7.3-arch1-1 #1 SMP PREEMPT Wed, 17 Jun 2020 19:42:12 +0000 x86_64 GNU/Linux
I followed the instructions for grabbing a prebuilt LLVM and had that stored at the correct prefix.
After cloning lumen I ran make -j20 build and got the following error.
jakob@devbox~/t/lumen> make -j18 build
Checking for lumen-tblgen..not found!

~/tmp/lumen/target/debug/build/lumen_tblgen ~/tmp/lumen
-- CMake Version   : 5.7.3-arch1-1
-- Compiler        : /usr/bin/cc
-- Target          :
-- Operating System: Linux-5.7.3-arch1-1
-- Build Compiler  : ON
-- Build Tests     : OFF
-- Building with -fPIC
-- Configuring done
-- Generating done
-- Build files have been written to: /home/jakob/tmp/lumen/target/debug/build/lumen_tblgen
[1/1] Linking CXX executable lumen-tblgen/lumen-tblgen
FAILED: lumen-tblgen/lumen-tblgen
: && /usr/bin/c++ -Wno-unused-parameter -Wno-undef -fno-rtti  -fPIC -fvisibility-inlines-hidden -Werror=date-time -w -fdiagnostics-color -ffunction-sections -fdata-sections -O3  lumen-tblgen/CMakeFiles/lumen-tblgen_lumen_tblgen.dir/lumen-tblgen.cpp.o -o lumen-tblgen/lumen-tblgen  -Wl,--whole-archive  lib/lumen/EIR/Tools/liblumen_EIR_Tools_TableGen.a  -Wl,--no-whole-archive  /home/jakob/.local/share/llvm/lumen/lib/libMLIRSupport.a  /home/jakob/.local/share/llvm/lumen/lib/libLLVMSupport.a  /home/jakob/.local/share/llvm/lumen/lib/libLLVMTableGen.a  -lMLIRTableGen  /home/jakob/.local/share/llvm/lumen/lib/libLLVMSupport.a  -lrt  -ldl  -lpthread  -lm  /home/jakob/.local/share/llvm/lumen/lib/libLLVMDemangle.a && :
/usr/bin/ld: cannot find -lMLIRTableGen
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
make: *** [Makefile:27: build] Error 1

My cc is
jakob@devbox~/t/lumen> cc --version
cc (GCC) 10.1.0

And my c++ version is
jakob@devbox~/t/lumen> c++ --version
c++ (GCC) 10.1.0

Any advice?

@jkbbwr the version of LLVM to download was out-of-date in the README with what we're using locally and in the CI.  It should be updated now.  THe bit change is the LLVM upstream made TableGen a library, which is why you had MLIRTableGen missing when using an older version.


ERROR: NYI flatten types of generator or nodef module, for Linebuffer?

bash-3.2$ ~/CppWorkspace/coreir/bin/coreir -p rungenerators,flattentypes -i ~/CppWorkspace/CGRAMapper/examples/conv_3_1.json -o veri_conv_3_1.json --load_libs ~/CppWorkspace/coreir/lib/libcoreir-commonlib.dylib
In Run Generators
ERROR: NYI flatten types of generator or nodef module
{commonlib.Linebuffer}.out Is not a flattened type!
Type is: Bit[16][1][3]
Assertion failed: (isBitOrArrOfBits(rpair.second)), function runOnInstanceGraphNode, file flattentypes.cpp, line 69.
Abort trap: 6

@dillonhuff Can you confirm that this is fixed in dev and close the issue?

Yes, this is fixed.


singup error

We can't continue with the signup process for the following reason:
Your phone number did not pass security checks.
MS-CorrelationId: 00000000-8323-4b19-8aa7-31d1172a6769
Code: 40315

Hi, sorry to hear you were having issues and hope they're resolved now! I'm closing this issue; if you still need assistance, contact<EMAIL_ADDRESS>As you can see, this repo isn't monitored regularly. We're working on that!


[macOS export] Add support for embedded provisioning profile and Installer PKG format and macOS min version.

Adds support for embedded provisioning profile.
Adds support for Installer PKG export format (without Notarization).
Adds export option to set macOS min. version.

Fixes https://github.com/godotengine/godot-proposals/issues/6360
Fixes https://github.com/godotengine/godot/issues/73876
Fixes https://github.com/godotengine/godot/issues/74154

Great to see a PR so quickly to resolve these issues!
With these added additional export options and modes the complexity and possibilities to make mistakes will however also increase. So perhaps it would also be a good idea to add some additional hints/warnings to help users set up the correct combinations of these signing parameters for different exporting intentions. For example indicate the proper signing identity to use if embedding a provisioning profile for an app store pkg vs. when exporting for distribution outside of the appstore in a signed dmg with notarization instead.
Or maybe to feature some sort of "Export intent" parameter the user can set and then have the export dialogue warnings provide guidance based on that to the appropriate signing settings and formats for the selected export intent.

Labeling with cherry-pick as this is required to fix https://github.com/godotengine/godot/issues/74154.

Should this still be merged or is it better to focus on #74644?

Should this still be merged or is it better to focus on #74644?

74644 include all of the changes, and definitely superior to this.


PH Sensor API library (E-201C-Blue)

Embedded Library:
PH Sensor API library (E-201C-Blue)
This library aims to create a comfortable and semantic experience to read PH values from the E-201C-Blue sensor.

Library complete - see branch firmware_e201c


[1pt] Create website header with empty nav - Setup (group task)

Download the free image for the app logo.
Create empty navigation (this will be filled in in separate tasks).

https://github.com/Dorcas126/space-travelers/pull/26


Synthesis failed for java-translate

Hello! Autosynth couldn't regenerate java-translate. :broken_heart:
Here's the output from running synth.py:
 DO NOT EDIT!' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/DetectLanguageResponseOrBuilder.java.
2020-07-12 20:58:01,624 synthtool [INFO] > Replaced '// Generated by the protocol buffer compiler.  DO NOT EDIT!' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/InputConfigOrBuilder.java.
2020-07-12 20:58:01,624 synthtool [INFO] > Replaced '// Generated by the protocol buffer compiler.  DO NOT EDIT!' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/DetectedLanguageOrBuilder.java.
2020-07-12 20:58:01,624 synthtool [INFO] > Replaced '// Generated by the protocol buffer compiler.  DO NOT EDIT!' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/DeleteGlossaryMetadata.java.
2020-07-12 20:58:01,624 synthtool [INFO] > Replaced '// Generated by the protocol buffer compiler.  DO NOT EDIT!' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/GetGlossaryRequestOrBuilder.java.
2020-07-12 20:58:01,624 synthtool [INFO] > Replaced '// Generated by the protocol buffer compiler.  DO NOT EDIT!' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/ListGlossariesResponse.java.
2020-07-12 20:58:01,625 synthtool [INFO] > Replaced '// Generated by the protocol buffer compiler.  DO NOT EDIT!' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/DeleteGlossaryRequestOrBuilder.java.
2020-07-12 20:58:01,625 synthtool [INFO] > Replaced '/\\*\n \\* Copyright \\d{4} Google LLC\n \\*\n \\* Licensed under the Apache License, Version 2.0 \\(the "License"\\); you may not use this file except\n \\* in compliance with the License. You may obtain a copy of the License at\n \\*\n \\* http://www.apache.org/licenses/LICENSE-2.0\n \\*\n \\* Unless required by applicable law or agreed to in writing, software distributed under the License\n \\* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express\n \\* or implied. See the License for the specific language governing permissions and limitations under\n \\* the License.\n \\*/\n' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/LocationName.java.
2020-07-12 20:58:01,626 synthtool [INFO] > Replaced '/\\*\n \\* Copyright \\d{4} Google LLC\n \\*\n \\* Licensed under the Apache License, Version 2.0 \\(the "License"\\); you may not use this file except\n \\* in compliance with the License. You may obtain a copy of the License at\n \\*\n \\* http://www.apache.org/licenses/LICENSE-2.0\n \\*\n \\* Unless required by applicable law or agreed to in writing, software distributed under the License\n \\* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express\n \\* or implied. See the License for the specific language governing permissions and limitations under\n \\* the License.\n \\*/\n' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/proto-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/GlossaryName.java.
2020-07-12 20:58:01,627 synthtool [INFO] > Replaced '^package (.*);' in /tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/grpc-google-cloud-translation-v3-java/src/main/java/com/google/cloud/translate/v3/TranslationServiceGrpc.java.
2020-07-12 20:58:01,646 synthtool [WARNING] > No files in sources [PosixPath('/tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/gapic-google-cloud-translation-v3-java/samples/src')] were copied. Does the source contain files?
2020-07-12 20:58:01,646 synthtool [WARNING] > No files in sources [PosixPath('/tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/gapic-google-cloud-translation-v3-java/samples/resources')] were copied. Does the source contain files?
2020-07-12 20:58:01,646 synthtool [WARNING] > No files in sources [PosixPath('/tmpfs/tmp/tmpns92bjgj/google-cloud-translation-v3-java/gapic-google-cloud-translation-v3-java/samples/src/**/*.manifest.yaml')] were copied. Does the source contain files?
2020-07-12 20:58:01,647 synthtool [INFO] > Running java formatter on 45 files
2020-07-12 20:58:05,498 synthtool [INFO] > Running java formatter on 1 files
2020-07-12 20:58:07,664 synthtool [INFO] > Running java formatter on 57 files
2020-07-12 20:58:14,310 synthtool [INFO] > Running java formatter on 0 files
2020-07-12 20:58:14,669 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
.github/CODEOWNERS
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.github/trusted-contribution.yml
.github/workflows/ci.yaml
.kokoro/build.bat
.kokoro/build.sh
.kokoro/coerce_logs.sh
.kokoro/common.cfg
.kokoro/common.sh
.kokoro/continuous/common.cfg
.kokoro/continuous/dependencies.cfg
.kokoro/continuous/integration.cfg
.kokoro/continuous/java11.cfg
.kokoro/continuous/java7.cfg
.kokoro/continuous/java8-osx.cfg
.kokoro/continuous/java8-win.cfg
.kokoro/continuous/java8.cfg
.kokoro/continuous/lint.cfg
.kokoro/continuous/propose_release.cfg
.kokoro/continuous/samples.cfg
.kokoro/dependencies.sh
.kokoro/linkage-monitor.sh
.kokoro/nightly/common.cfg
.kokoro/nightly/dependencies.cfg
.kokoro/nightly/integration.cfg
.kokoro/nightly/java11.cfg
.kokoro/nightly/java7.cfg
.kokoro/nightly/java8-osx.cfg
.kokoro/nightly/java8-win.cfg
.kokoro/nightly/java8.cfg
.kokoro/nightly/lint.cfg
.kokoro/nightly/samples.cfg
.kokoro/populate-secrets.sh
.kokoro/presubmit/clirr.cfg
.kokoro/presubmit/common.cfg
.kokoro/presubmit/dependencies.cfg
.kokoro/presubmit/integration.cfg
.kokoro/presubmit/java11.cfg
.kokoro/presubmit/java7.cfg
.kokoro/presubmit/java8-osx.cfg
.kokoro/presubmit/java8-win.cfg
.kokoro/presubmit/java8.cfg
.kokoro/presubmit/linkage-monitor.cfg
.kokoro/presubmit/lint.cfg
.kokoro/presubmit/samples.cfg
.kokoro/release/bump_snapshot.cfg
.kokoro/release/common.cfg
.kokoro/release/common.sh
.kokoro/release/drop.cfg
.kokoro/release/drop.sh
.kokoro/release/promote.cfg
.kokoro/release/promote.sh
.kokoro/release/publish_javadoc.cfg
.kokoro/release/publish_javadoc.sh
.kokoro/release/snapshot.cfg
.kokoro/release/snapshot.sh
.kokoro/release/stage.cfg
.kokoro/release/stage.sh
.kokoro/trampoline.sh
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
java.header
license-checks.xml
renovate.json
samples/install-without-bom/pom.xml
samples/pom.xml
samples/snapshot/pom.xml
samples/snippets/pom.xml
2020-07-12 20:58:14,832 synthtool [DEBUG] > existing pom file found (samples/pom.xml) - keeping the existing
2020-07-12 20:58:14,833 synthtool [DEBUG] > existing pom file found (samples/snippets/pom.xml) - keeping the existing
2020-07-12 20:58:14,833 synthtool [DEBUG] > existing pom file found (samples/install-without-bom/pom.xml) - keeping the existing
2020-07-12 20:58:14,833 synthtool [DEBUG] > existing pom file found (samples/snapshot/pom.xml) - keeping the existing
2020-07-12 20:58:14,834 synthtool [DEBUG] > Wrote metadata to synth.metadata.
2020-07-12 20:58:14,897 autosynth [INFO] > Changed files:
2020-07-12 20:58:14,897 autosynth [INFO] > M synth.metadata
2020-07-12 20:58:14,897 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at 44df12e ci(java): run dependency test on Java 8 and 11 (#236)
2020-07-12 20:58:14,904 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2020-07-12 20:58:14,910 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 657, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 506, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 637, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 364, in synthesize_loop
    if change_pusher.check_if_pr_already_exists(fork.branch):
  File "/tmpfs/src/github/synthtool/autosynth/change_pusher.py", line 117, in check_if_pr_already_exists
    prs = self._gh.list_pull_requests(repo, state="open", head=f"{owner}:{branch}")
  File "/tmpfs/src/github/synthtool/autosynth/github.py", line 61, in list_pull_requests
    return cast(List[Dict], _get_json_or_raise_exception(response))
  File "/tmpfs/src/github/synthtool/autosynth/github.py", line 478, in _get_json_or_raise_exception
    json = response.json()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/requests/models.py", line 898, in json
    return complexjson.loads(self.text, **kwargs)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


Spurious syntax highlighting of code blocks in site documentation

Please make sure that this is a documentation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:doc_template
System information

TensorFlow version: N/A
Doc Link: Various, but see for example /community/documentation, /guide/debugger, or /guide/datasets_for_estimators

Describe the documentation issue
Many of the code blocks in this guide on Writing Tensorflow Documentation are meant to represent markdown, but they're syntax highlighted as... something else. e.g. the word "from" is highlighted as if it were a keyword in:
Flips an image horizontally from left to right.

These blocks should either have no syntax highlighting, or markdown syntax highlighting (if the library you're using for syntax highlighting allows it).
In some cases, authors have explicitly used a language identifier of 'none' in fenced code blocks, presumably to try to suppress syntax highlighting, but it doesn't look like that's respected when rendering. For example, the first two code blocks in /guide/debugger are tagged 'none' in the corresponding md file, but the "Accuracy at step..." block is still lit up like a Christmas tree.
We welcome contributions by users. Will you be able to update submit a PR (use the doc style guide) to fix the doc Issue?
I wish I could, but as far as I can tell, the markdown -> html part of the docs pipeline isn't part of any public repo.

Any thoughts @lamberta @MarkDaoust ? Here is a screenshot for reference.

Thanks for reporting this @colinmorris.
It uses prettify to do the syntax highlighting. There doesn't seem to be a markdown option.
You can use a <pre> tag without the "prettyprint" class to render as plain text.
<pre>
Flips an image horizontally from left to right.
</pre>

produces on the site.
Flips an image horizontally from left to right.

Any chance you could send a PR fixing those problems?
(And maybe mention this, and that link to prettify link to community/documentation?)

Sure. How do I generate documentation locally from https://github.com/tensorflow/docs?

@MarkDaoust , sure. How do I generate documentation for specific pages locally?

There's no way to preview the final product :(. A lot of the pipeline is built into the site infrastructure, so there's no way to opensource it.


Paper layout different on Arxiv

Hey all.
I noticed that Arxiv layouts the paper slightly differently than my local sphix/buildsystem. Now I am somewhat disappointed, because I spent a significant time on engineering the paper such that it fits on 8 pages (which is already oversize, sorry for that!) But now Arxiv layouts it to 8.5 pages and then expands it to 10 with 1.5 blank pages.
Did someone else experience this kind of discrepancy? My local paper (Docutils 0.12, Sphinx 1.4):
https://dl.dropboxusercontent.com/u/50299996/Garbage Collection in JyNI.pdf
(Tex-file: https://dl.dropboxusercontent.com/u/50299996/paper.tex)
Compared to Arxiv-variant:
http://arxiv.org/pdf/1607.00825v1.pdf
The euroscipy_proceedings build-script does not leave room for layout configuration, does it?
Can I configure it such that it matches Arxiv-output?
Is it possible to fix this in Arxiv-repo?

I have noticed this as well, but did not have time to investigate.


It's stuck to drag the window

Version: 1.0.126
Platform: win32 10.0.18363
Plugins: none
Look this gif.

Same issue
Version: 1.0.126
Platform: win64 10.0.19041
Plugins: none

Same issue
Version: 1.0.126
Platform: win64 10.0.19041
Plugins: none

This problem has been solved in Version 1.0.127

This problem has been solved in Version 1.0.127


Support array-like object in argument of almost functions

Compatibility with CuPy and NumPy is important. I hope CuPy functions accept array-like object.

I recognize that we are in the following situation.

CuPy function with CuPy arguments: OK
NumPy function with NumPy arguments: OK
CuPy function with NumPy arguments: NO -> OK
NumPy function with CuPy arguments: NO (discussed at #589)

cupy.testing.numpy_cupy_* functions test that 1st and 2nd results are same. How about extending this and testing the 3rd one too? In this case, we need xp_func and xp_arg instead of xp.

OK, I think it's good.
Cloud you design small sample or PR?


on-chain transaction to external wallet stuck

Background
Sent transaction with low fee to external wallet
https://www.blockchain.com/btc/tx/575bb7b9d2e0eaeef4e75c0c912a2e19b65db3072f7fabd4e9087a2bdd855d74
now wallet balance is
{
"total_balance": "0",
"confirmed_balance": "0",
"unconfirmed_balance": "0"
}
Your environment
0.11.0-beta commit=basedon-v0.11.0-beta-fresh-btcpay
using up2date btcpay docker deployment with my first lnd config where providing private keys was not possible
Steps to reproduce
tried with command wallet bumpfee tx:id --sat_per_byte xx
but of course the output does not belong to this wallet, works only for channel transaction bumps for me
whats my line? thanks for your time & help

It looks like the transaction never made it to the network because it's trying to spend an output that's already spent. Sounds to me like your wallet is out of sync with the chain.
Can you try to update to lnd v0.12.1 and use the reset-wallet-transactions=true configuration to do a full rescan?

thank you, will try as soon as lnd is updated to v0.12.x, afaik reset-wallet-transactions=true is only available with v0.12.x right?
https://github.com/btcpayserver/btcpayserver-docker - will probably be updated soon

is only available with v0.12.x right?

Correct.


appSettings documentation is missing

Cannot locate  element documentation
While trying to redirect my old msdn links to the new ones for breaking changes.. I noticed that I could no longer find up-to-date documentation for  element.
I found these two outdated ones (that say they are archived):

https://msdn.microsoft.com/en-us/library/aa903313(v=vs.71).aspx
https://msdn.microsoft.com/en-us/library/ms228154(v=vs.85).aspx

but when I try to follow the link to the recommended one, it does not exist.  Nor can I find it on docs.microsoft.com in the configuration schema. :/

@conniey, the appSettings element appears to have last been published in the .NET Framework 4.0 doc set. In the current doc set, it has been "paved over" and replaced with the "This topic does not exist page" boilerplate. I'm trying to investigate the reason for this, if there was one, with the ASP.NET team, which owned the topic. Unfortunately, the topic has been deleted from CAPS, the authoring system that we used for .NET Framework 4.5 through 4.7 content, and therefore was not migrated to GitHub.
My suspicion is that this was part of a rather unfortunate initiative to reduce the size of our doc set by eliminating low page-view pages. I've noticed that the configuration settings documentation seems to have some major gaps caused by the haphazard removal of topics.
I've opened an item in the Migration project to try to investigate this and what I think may be some other significant gaps in the configuration documentation.

AFAIK the entire documentation for ASP.NET got removed from MSDN and then moved to the asp.net site but I don't which content was ported and which one was deleted.
Do we need to restore those topics?
/cc @Rick-Anderson @tdykstra @wadepickett

I think that we at least need to restore the basic configuration settings documentation.
In terms of appSettings, I may be wrong, but I don't think that it was used exclusively by ASP.NET; I distinctly seem to recall using it myself in non-ASP.NET applications.

@rpetrusha  what's the process to get at least this restored? I'm concerned there might be new settings that are not documented in the older versions.

At this point, the restore process might have to be manual. These probably only existed back in DxStudio.

Restoration will have to be manual, since the files weren't migrated to CAPS. Fortunately, manual migration is fairly simple, and there are only a handful of topics.

only a handful of topics

@rpetrusha Is there an easy way to isolate the missing topics, or should I dig through https://msdn.microsoft.com/en-us/library/1fk1t1t0(v=vs.71).aspx and perform a manual comparison to what we have in the repo to find the MIA topics?

@GuardRex, there's no easy way to isolate the missing topics. I can't think of any way to do it other than performing a manual comparison.

Ok, thx. I'll get them on a 🐠 Big Fishing Expedition! 🎣

@GuardRex, along with the config documentation, there's a legacy compat document that we need to recreate as well: .NET Framework 4 Migration Issues.

@rpetrusha Do they have a script for migrating markup content into markdown? If so, can I open a SiteHelp request to have .NET Framework 4 Migration Issues converted?

Probably the best thing to do is to use Internet Explorer to save it as HTML, and use an HTML to markdown conversion tool. Whenever I've had to convert XML or HTML to markdown, I've done it manually, so I don't have a sense of what tool is better than the others.

I'll work a few sections and tables with https://domchristie.github.io/to-markdown/ ... we'll see how that goes.


No posts were found - with newest version

Hey,
I get this error:
2022-11-07 20:10:22,447 - facebook_page_scraper.driver_utilities - CRITICAL - No posts were found!
running: python3 posts.py
posts.py file (unchanged from readme.md suggestion):
**#import Facebook_scraper class from facebook_page_scraper
from facebook_page_scraper import Facebook_scraper
#instantiate the Facebook_scraper class
page_name = "metaai"
posts_count = 10
browser = "chrome"
proxy = "IP:PORT" #if proxy requires authentication then user:password@IP:PORT
timeout = 600 #600 seconds
headless = True
meta_ai = Facebook_scraper(page_name, posts_count, browser, proxy=proxy, timeout=timeout, headless=headless)
json_data = meta_ai.scrap_to_json()
print(json_data)
**
Tries master and 4.x branches with the same result.  Checked the code - I can't find that css selector on the facebook by myself as well, so will this be working, or did facebook change everything ? Thanks

Hey, It works for me. I tried both UI, the new one as well as the old one.

Are you using a proxy properly? I can see you're using a proxy in the example but it seems to be hardcoded instruction.
What region are you using it? It might happen that you're from some strict European country.

I think I found the issue. A dialogue to accept cookies appears for the people who are running the crawler in the European region. It was mentioned in this issue I remember #36. You can set headless=False and watch crawler running, can confirm if that's the case for you too

Hey thanks for the answer. Removed proxy from arguments and used headless = False
Weird, that https shows insecure, and it seems that it gets stuck on cookies ppopup

Is it the popup, or in my country (Lithuania) it is having different layout?

I've pushed the changes. to close that popup. Seems like there's another popup that shows up to alert for login, I was using the German location for scraping.
In case if login popup shows up you can use the browser_profile argument for bypassing that. You will need to have a Facebook account logged into the browser then copy the browser profile path for that browser session and pass it to the constructor. Something like this Facebook_scraper('mysql', headless=False, browser='firefox', browser_profile='YOUR PATH') it will use that FB account for scraping but beware that it might get blocked as well.


Adding Description under the blog titles

Is there a way to add a description under the blog titles in the /blog?

Something like whats present in Ramium?

Hey @SirCharan , there's been an open PR (https://github.com/jakewies/hugo-theme-codex/pull/75) for this feature for a few months now, but it has gone pretty stale and I don't have much time to work on it at the moment. If there were more support for this I'd do my best to make it a priority but it doesn't seem to be.
That being said, if you'd like to take a peak at the PR and see how it's done, you could integrate with your own project. If you'd like to contribute this feature to this project / take over that PR that would be most welcome.


Convert to basic extension

frame time graph is kinda weird + frame time display is broken
also really spaced out
i dont think i really wanna have this primarily as an ext, also, i'd rather keep it on the page :thinking:

unplanned


Add a more detailed explanation of the test & build framework to the README

This should detail how the framework works at a high level and include info on the build step.

@modeswitch This seems like something that should belong in CONTRIBUTING. Should I put it there?

In fact, the grunt stuff already sorta lives there.


a11y: textfield with maxLength and obscureText includes hint in its character count

Internal: b/141386361
If you have a TextField with obscureText: true and a maxLength, the characters from the hintText will be counted in the announcements which announce how many characters the text field contains.
Tested only on TalkBack, haven't checked VoiceOver
Repro instructions:
Build and deploy the code below. When you first focus on the field, it reads "Editing, password, password, Edit box". I think it's an error for it to read password twice.
Enter a character into the text field, then screenreader navigate away. When you screenreader navigate back, it will read "Editing, password, 11 characters, Edit box" even though you only entered one character. Delete the character, navigate away, and navigate back. Now it reads "Editing, password, 8 characters, Edit box". Shouldn't that be 0 characters?
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'textfield repro',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: Scaffold(
        body: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text('some text'),
            TextField(
              obscureText: true,
              enableInteractiveSelection: true,
              maxLength: 5,
              decoration: InputDecoration(
                hintText: 'password',
              ),
              autocorrect: false,
              onChanged: (_) {},
            ),
          ],
        ),
      ),
    );
  }
}

b/141386361

This works fine with VoiceOver, investigating fix for TalkBack

In TalkBack, it only triggers if the TextField is in edit mode, and the incorrect number is reports is always lengthOfHintText + numberOfBullets + 2 (which only happens when obscureText is true). I have not found a connection between the lengthOfHintText (or the 2) and the Semantics. The describeSemanticsConfiguration of RenderEditable: https://github.com/flutter/flutter/blob/27321ebbad/packages/flutter/lib/src/rendering/editable.dart#L1156 only deals with the _plainText String

flutter doctor -v
[✓] Flutter (Channel dev, 1.20.0-7.1.pre, on Mac OS X 10.15.6 19G73, locale
    en-GB)
    • Flutter version 1.20.0-7.1.pre at /Users/taha/Code/flutter_dev
    • Framework revision 7736f3bc90 (6 days ago), 2020-07-10 16:33:05 -0700
    • Engine revision d48085141c
    • Dart version 2.9.0 (build 2.9.0-21.2.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version
    30.0.0)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google
      Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.2

[✓] Connected device (5 available)
    • SM M305F (mobile)      • 32003c30dc19668f          • android-arm64  •
      Android 10 (API 29)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            •
      iOS 13.6
    • macOS (desktop)        • macos                     • darwin-x64     •
      Mac OS X 10.15.6 19G73
    • Web Server (web)       • web-server                • web-javascript •
      Flutter Tools
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 83.0.4103.116

• No issues found!

Do we have any update on this ? I am facing the same issue.

I just tried reproducing this, and the behavior seems to have improved significantly.
Now instead of saying the number of typed characters, it tells you the number of characters remaining. I consider this fixed


Create anonymous public demo

Since the paper reviews are double-blind our demo must be anonymous, too.
@sgratzl Could you please configure the nginx on AWS that we can use the Public IP of our cluster. The result would be something like http://01.23.456.789/taggle.

I think @anita-steiner could also take a look at this issue.

actually, one just need to add the port of the taggle app to the security group such that it can be accessed externally and then you can use: http://cluster-ip:xxxx Tho, if we are not going to change all caleydo related things, I don't see the points. Even tho it is Caledyo, it is not directly related to a specific person

-> http://<IP_ADDRESS>:32793/


Integrate Coveralls

https://coveralls.io/
https://github.com/kt3k/coveralls-gradle-plugin

Multiple uploads with same commit hash problematic - last one wins instead of being merged in.
https://github.com/lemurheavy/coveralls-public/issues/613


Logo and Home

Fixed logo and Home reloading page.

Great! thanks for your interest @Pongy-jr.
I think you didn't notice that the logo in the footer has the same issue


Kobukson doesn't return all top cards with a castle

The echo effect also isn't working

I fixed both of these things very recently! Just re-tested with the latest code and they are working fine.


ci: lint all the things

This does a few things - has to be in one PR otherwise builds will fail

Moves all CI linting out of individual actions and puts them in their own action
Lints everything that is left

@kyle-ssg I need some checks on this commit https://github.com/Flagsmith/flagsmith/pull/2446/commits/68740a2ef26845fdaf2f0d0b7eab4da325ec6ce7 also need to go through some of the files excluded:
frontend/web/components/tags/TagValues.tsx
frontend/web/components/pages/EnvironmentSettingsPage.js
frontend/common/utils/base/_utils.js
``` in eslintignore

Codecov Report
Patch and project coverage have no change.

Comparison is base (d150c7f) 95.42% compared to head (f1f1850) 95.42%.

:exclamation: Current head f1f1850 differs from pull request most recent head 20056a5. Consider uploading reports for the commit 20056a5 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2446   +/-   ##
=======================================
  Coverage   95.42%   95.42%
=======================================
  Files         963      963
  Lines       27044    27044
=======================================
  Hits        25807    25807
  Misses       1237     1237

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Can't add repository

Hi,
I'm currently trying to get the git_hosting working. Already fixed a previous issue (#420) but still can't get it to work. In my git hosting log, the following entry came up:
root@debian:/opt/redmine/redmine-3.0/log# cat git_hosting.log
# Logfile created on 2015-04-26 23:01:40 +0200 by logger.rb/31641
2015-04-26 23:01:40 +0200 [INFO] User 'stefan' created a new repository 'test2'
2015-04-26 23:01:40 +0200 [INFO] Accessing gitolite-admin.git at '/tmp/redmine_git_hosting/git/gitolite-admin.git'
2015-04-26 23:01:40 +0200 [INFO] add_repository : repository 'test2' does not exist in Gitolite, create it ...
2015-04-26 23:01:40 +0200 [INFO] add_repository : commiting to Gitolite...
2015-04-26 23:01:40 +0200 [INFO] add_repository : let Gitolite create empty repository 'repositories/test2.git'
2015-04-26 23:01:40 +0200 [INFO] Execute Gitolite Plugins
2015-04-26 23:01:41 +0200 [INFO] Create README file for repository 'test2'
2015-04-26 23:01:41 +0200 [ERROR] Error while creating README file for repository 'test2'
2015-04-26 23:01:41 +0200 [ERROR] Early EOF
2015-04-26 23:01:41 +0200 [ERROR] Git exited with non-zero status : 128
root@debian:/opt/redmine/redmine-3.0/log#

When I look in my config files, my gitolite.conf looks like this:
repo    @all
  RW+                            = redmine_gitolite_admin_id_rsa

repo    gitolite-admin
  RW+                            = redmine_gitolite_admin_id_rsa

repo    test
  R                              = DUMMY_REDMINE_KEY gitweb
  config redminegitolite.projectid = test
  config redminegitolite.repositoryid =
  config redminegitolite.repositorykey = KGFEO1dJRtHTCqFpnHSPQb2h6J0KRCFcYX2ia0AyKUwoeBWqABidOFQI6dHlMTy
  config http.uploadpack = true
  config http.receivepack = true
  config multimailhook.enabled = false

repo    test2
  R                              = DUMMY_REDMINE_KEY gitweb
  config redminegitolite.projectid = test2
  config redminegitolite.repositoryid =
  config redminegitolite.repositorykey = Y9H5MHSwy8o5kFxTZXpnyKcZwfFxPKKHm528JFZNc2TizRd6M6ftZ0iznU8XwMR
  config http.uploadpack = true
  config http.receivepack = true
  config multimailhook.enabled = false

repo    testing
  RW+                            = @all

The authorized_keys contains:
# gitolite start
command="/opt/git/gitolite/src/gitolite-shell redmine_stefan_3",no-port-forwarding,no-X11-forwarding,no-agent-forwarding,no-pty ssh-rsa [valid_key] stefan@debian
command="/opt/git/gitolite/src/gitolite-shell redmine_gitolite_admin_id_rsa",no-port-forwarding,no-X11-forwarding,no-agent-forwarding,no-pty ssh-rsa [valid_key] redmine@debian
# gitolite end

Could you please give some guidance on solving this issue?

Hi @n-rodriguez ,
Did you find time to look into this by any chance? If you need any more info, please let me know. I can even provide access to the VM this is happening in.
Kind regards,
Stefan

Hi there!
Actually it seems to work as repositories are created. The issue here is README file creation that is not working. If you uncheck 'create README file' on repository creation you should not have these errors.
This might be a SSH key permission issue on Gitolite side : check that redmine_stefan_3 and redmine_gitolite_admin_id_rsa are truly 2 different SSH keys. If not, Gitolite will pick up the first SSH key it'll find. So if the keys are the same Gitolite will use redmine_stefan_3 as the current user to check the permissions against the repository. This will end in failure since at this moment redmine_stefan_3 don't have the RW permission yet.

Check that you can clone those repositories by hand (even if they are empty).
If you have an error like 'repository don't exist' check your .gitolite.rc config file : http://redmine-git-hosting.io/get_started/#step-5-gitolite-must-accept-hook-keys and do a project resync.

Hi there! Any news?

Hi,
Sorry for the late response. I retraced all steps and everything is working now. I probably forgot something.


drivers: gpio: Remove redundant GPIO dep. from GPIO_HT16K33

GPIO_HT16K33 is defined in drivers/gpio/Kconfig.ht16k33, which is
source'd within an 'if GPIO' in drivers/gpio/Kconfig.

See this section on the Kconfig tips page for how to check for redundant dependencies.
Unrelated tip (just copy-pasting it into all of these):
if DEP

config FOO
	...

endif # DEP

is never needed (though it might be fine if it's preparation for adding other symbols within the if). It's the same as
config FOO
	...
        depends on DEP

if is just shorthand (or longhand) for depends on.


Load code from file

Hi, It's possible to load code from file using jquery and then run codeflask to edit that code?
Example here: http://plnkr.co/edit/lwy2Zk11UJ8GVWX1JgKD?p=preview
In the example the html code is rendered as HTML...
NS

You could try out some AJAX. This appears to work.
$(document).ready(function() {
  var flask = new CodeFlask();

  $.get('code.html', function(data) {
    console.log(data);
    flask.run('#my-code-wrapper', { language: 'html' });
    flask.update(data);
  })
});


SM2 failed to generate public key from private key

Problem Description:
SM2 failed to generate public key from private key of 3.0 and 3.2.
command
1. generate sm2 private key file by OpenSSL commands of 3.0 and 3.2:
openssl ecparam -genkey -name SM2 -out sm2-private.key
2. failed to generate public key file with sm2 private key file
openssl ec -in sm2-private.key -pubout -out sm2-public.key
Preliminary analysis:
private key file
file generated by 1.1 as
-----BEGIN EC PARAMETERS-----
-----END EC PARAMETERS-----
-----BEGIN EC PRIVATE KEY-----
-----END EC PRIVATE KEY-----
file generated by 3.0 or 3.2 as
-----BEGIN SM2 PARAMETERS-----
-----END SM2 PARAMETERS-----
-----BEGIN SM2 PRIVATE KEY-----
-----END SM2 PRIVATE KEY-----
error in generating public key
read EC key
unable to enable public key encoding
4076A04EF87F0000:error:030000A3:digital envelope routines:EVP_PKEY_set_params:invalid key:crypto/evp/p_lib.c:2386:
analysis
Guess whether the high version is not compatible with the low version, or update the usage rules of related commands, because this group of commands is correct in version 1.1.
### Tasks

Would these commands work for you:
openssl genpkey -algorithm SM2 -out sm2-private.key
openssl pkey -in sm2-private.key -pubout -out sm2-public.key

Is the sm2 key generated in this way provided by you the same as that generated in version 1.1 in the way mentioned earlier?

Yes, the encoding of the key is different. The parameters are defined by the SM2 OID in the encoding of the key.

How to get the SM2 key I need in the version 3.0 or above?

Is this problem a bug in the current version 3.0 and above?


Trigger the 'latest' image push on GH triggers

The image is built but not pushed:
https://github.com/Apicurio/apicurio-registry-mt-ui/actions/runs/3063841642/jobs/4946361427

Ah, I missed out on this one. Good catch. thanks you @andreaTP


npm run start complaining about options

After  npm install and npm run start I got the following issue on my system and another fresh Windows install:
(libertem310) C:\Users\weber\Documents\src\LiberTEM\client>npm run start
npm WARN config global `--global`, `--local` are deprecated. Use `--location=global` instead.

><EMAIL_ADDRESS>start
> react-scripts start

Browserslist: caniuse-lite is outdated. Please run:
  npx browserslist@latest --update-db
  Why you should do it regularly: https://github.com/browserslist/browserslist#browsers-data-updating
Invalid options object. Dev Server has been initialized using an options object that does not match the API schema.
 - options.allowedHosts[0] should be a non-empty string.

As a workaround one can use tox -e build_client to get the GUI compiled.


Consider pyproject.toml for bandit

Checked that both .bandit and pyproject.toml are respected, with preference given to .bandit.

Can I add the flag conditionally in the run command based on whether the file exists? If not, I can replace the command with a script.

We don't have any way to add flags conditionally based on if config files exist, although it is a common case that we are aware of. If bandit doesn't error when a -c arg doesn't exist, you can add it, but otherwise it will run into issues. I will take a closer look at this tomorrow to see what our options are

Unfortunately bandit fails if the config file doesn't exist, so we can't support this run command change OOTB, and we'd prefer not to layer this with an additional script. Instead, users will have to override this execution in their trunk.yaml if they want bandit to check their pyproject.toml. I've created a feature request you can upvote to handle this in the future: https://features.trunk.io/check/p/support-configuration-files-in-run-commands-if-they-exist
This does highlight a bug, however, that we are not respecting pyproject.toml as an affects_cache element. I will put up a separate PR that fixes this.

This does highlight a bug, however, that we are not respecting pyproject.toml as an affects_cache element.

bandit does not read pyproject.toml if not explicitly specified on the command line, so it does not affect the cache.


Unzip map items

Rebased

Awesome. Thanks!

it is synced to the main branch now https://github.com/cloudflare/pingora/commit/2a449ac305a4573dbb463ac638b63ccfd2653191. Cheers!


[FEATURE] manage systemd overrides

Is your feature request related to a problem?

Describe the solution you'd like

A way to manage systemd overrides in pillar.
Describe alternatives you've considered

Additional context

dropin: true creates a salt-override.conf file and reloads systemd  --> good
But if you want to override an existing entry, you need two entries.  For example, if you're overriding ExecStart   First a blank row that clears it followed by your override.  (what an odd override system...)
ExecStart=
ExecStart= myapp -d

caveat: Type: simple can only have one ExecStart line and requires the above.  Type: oneshot can have multiple ExecStart and they are executed in series.


Update package.xml

Hi,
I just filled a bug report to package this for Debian ( https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=768041 )
One way of easily doing it is to use the package.xml file used to build the PECL extension.
I never had to update such a file, but here is my try. Would it be ok for you to release a new PECL version ?
Best regards,

Merged, thank you!


Add class_name keyword to syntax highlighting

This adds the class_name keyword that was added to Godot 3.1.
PS: What's the difference between GDScript.full.tmLanguage.json and GDScript.tmLanguage.json? Are these used in different contexts?

The GDScript.full.tmLanguage.json is not use now.
Thanks.


Hide images after keypressed

when drop down is clicked and options are visible, after any keypress (A-Z , del , ...) images of options will disappear.

I just checked this on http://websemantics.github.io/Image-Select/
The images are disappearing when typing something in the search input!
I don't see why this issue is closed?


[Bug]: After creating an export task in the desktop version, the log file first prompts that the task has expired, and then displays normally after a while.

ODC version
ODC4.2.3
OB version
independent
What happened?
After creating an export task in the desktop version, the log file first prompts that the task has expired, and then displays normally after a while.

What did you expect to happen?
can execute normally
How can we reproduce it (as minimally and precisely as possible)?
chat privately
Anything else we need to know?
No response
Cloud
No response

pass


Allow to specify certificates

This PR is related to #5. Expected usage is as follows:
helm install --generate-name . --set ... --set-file azIotEdgeConfig=config.toml,rootCA=root.ca.cert.pem,deviceCACert=device.full-chain.cert.pem,deviceCAKey=device.key.pem

@levi106 looking at this PR now (back from leave), thank you for your contribution.

Thanks @suneetnangia for the feedback, I've updated Readme.


CardWidth is not working

in my layout :
<android.support.v7.widget.RecyclerView             android:id="@+id/pileLayout"             android:layout_width="match_parent"             android:layout_height="250dp"             android:layout_below="@+id/textview_wod_quantity"             android:layout_marginBottom="16dp"             android:layout_marginTop="32dp"             android:clipChildren="false"             android:overScrollMode="never"             app:layoutManager="com.ramotion.cardslider.CardSliderLayoutManager"             app:viewUpdater="com.mvince.wodrandom.ui.customView.CardsUpdater"             app:cardWidth="400dp"/>
the cardWith is set but not effective in the app

Cant reproduce. It works in example.

I am facing the same issue, I even tried setting item width, height to match_parent as it is done in the sample. The app:cardWidth attribute works in the xml preview but not when ran in the app.
Seems like this works only in the sample.


After I add a listen on the on_pointer_event event, the mouse drag event is disabled.

I want to do somethint when the mouse click on the scene, but I found when I add a listener on the click event, I cannot rotate and translate the screen by drag the left and right mouse. It seems the base operation is disabled.
class Viewer:
  def __init__(self):
    self.server = viser.ViserServer()
    @self.server.on_client_connect
        def _(client: viser.ClientHandle):
            @client.scene.on_pointer_event(event_type="click")
            def _(event: viser.ScenePointerEvent):
              print(f"event.ray_direction")

the manipulation in viser/examples/20_scene_pointer.py is okay, I will check there difference firstly.

client.scene.remove_pointer_callback() is missing.
class Viewer:
  def __init__(self):
    self.server = viser.ViserServer()
    @self.server.on_client_connect
        def _(client: viser.ClientHandle):
            @client.scene.on_pointer_event(event_type="click")
            def _(event: viser.ScenePointerEvent):
              print(f"event.ray_direction")
              # Important.
              client.scene.remove_pointer_callback()


Version bump to 2.4.0

Includes:

Update ext.js download URL to Cloudera, per @jeremyhahn ( Issue: #265 )
Restrict Gem versions on older Ruby ( Issue: #266 )
Set ZOOPIDFILE ( Issues: #267 COOK-105 )
Split client and service JAAS files ( Issues: #268 COOK-106 )

LGTM


Add ASV

Adds a basic benchmarking suite, hopefully giving an idea of how to write something similar for other components. Still a number of things to do, tracking in #136

AppVeyor is failing in the install process somewhere, which is not touched by this PR.

Thanks for working on this Kai!  It will definitely be very useful to have.
I had not used asv before, but I like how you can manipulate the plots through the web browser.
I agree using the latest packages from upstream is reasonable for development purposes.  In that case I guess the workflow would be to run asv on a proposed PR vs. master to check that no unexpected performance regressions are being introduced.

+1 for using the latest released versions of numpy and cython.

AppVeyor is failing in the install process somewhere, which is not touched by this PR.

Is a pip issue, looks like it's fixed by pip install --upgrade pip (works in gh-161). You could add that here, or gh-161 could be merged first.

asv run
asv publish
asv preview

works as expected. Merging this PR without sorting out hosting etc. first would be fine with me.

My main concern with this at the moment is that ASV is incredibly slow. It takes about 20 minutes to run asv run --quick for one environment. This has an average of 99% CPU utilization, but I don't see which of the benchmarks could be taking this long. pywt.dwtn(np.ones((256,) * 3, 'sym20') is the slowest and takes about 10 seconds when run manually.
Is anybody else encountering this?

I have not tried with --quick, but it definitely took a long time to run for the 2 python versions and 4 commits shown in #162.  I didn't measure the time, but I think it was at least 1 hour.

That matches my experience, most of the time seems to be spent in initialization rather than actual timing. This is possibly still useful to control for regressions, but it's too slow to get any kind of feedback during development so I'd prefer another alternative.

initialization may be a little faster if you are able to use 'conda' rather than 'virtualenv' in the ASV configuration.  I think it would be fetching a prebuilt numpy in that case.

Since the envs don't get cleaned up even with git clean -xdf, the initial build time is a one-time cost. And with asv run --python=same, the total runtime of the benchmarks is about 5 min for me. Could be better, but it's not that bad considering that there are actually quite a few benchmarks here. The first DwtTimeSuiteBase param definition alone is 7 values of n times 3 wavelets times 6 modes x 4 time_ methods = 504 benchmarks.

I think it would help to steal benchmarks/run.py and the benchmark related stuff in runtests.py from Scipy. That avoids having to reinvent stuff like --python=same.

Yeah, 5 minutes would be a reasonable time for full benchmarks. What do you get with --quick enabled as well? I can't get asv run --python=same to work at the moment (ditto for asv dev):
kai@kjw53-laptop ~/D/c/p/asv> asv run --python=same
· Discovering benchmarks
·· Error running /usr/bin/python3.4 /usr/lib64/python3.4/site-packages/asv/benchmark.py discover benchmarks /tmp/tmp40ddvr65
             STDOUT -------->

             STDERR -------->
             Traceback (most recent call last):
               File "/usr/lib64/python3.4/site-packages/asv/benchmark.py", line 658, in <module>
                 list_benchmarks(benchmark_dir, fp)
               File "/usr/lib64/python3.4/site-packages/asv/benchmark.py", line 639, in list_benchmarks
                 for benchmark in disc_benchmarks(root):
               File "/usr/lib64/python3.4/site-packages/asv/benchmark.py", line 624, in disc_benchmarks
                 for module in disc_files(root, os.path.basename(root) + '.'):
               File "/usr/lib64/python3.4/site-packages/asv/benchmark.py", line 613, in disc_files
                 module = import_module(package + filename)
               File "/usr/lib64/python3.4/importlib/__init__.py", line 109, in import_module
                 return _bootstrap._gcd_import(name[level:], package, level)
               File "<frozen importlib._bootstrap>", line 2254, in _gcd_import
               File "<frozen importlib._bootstrap>", line 2237, in _find_and_load
               File "<frozen importlib._bootstrap>", line 2226, in _find_and_load_unlocked
               File "<frozen importlib._bootstrap>", line 1200, in _load_unlocked
               File "<frozen importlib._bootstrap>", line 1129, in _exec
               File "<frozen importlib._bootstrap>", line 1471, in exec_module
               File "<frozen importlib._bootstrap>", line 321, in _call_with_frames_removed
               File "/home/kai/Documents/code/pywt/asv/benchmarks/dwt_benchmarks.py", line 5, in <module>
                 class DwtTimeSuiteBase(object):
               File "/home/kai/Documents/code/pywt/asv/benchmarks/dwt_benchmarks.py", line 11, in DwtTimeSuiteBase
                 pywt.Modes.modes)
             AttributeError: 'module' object has no attribute 'Modes'
Traceback (most recent call last):
  File "/usr/lib/python-exec/python3.4/asv", line 9, in <module>
    load_entry_point('asv==0.1.1', 'console_scripts', 'asv')()
  File "/usr/lib64/python3.4/site-packages/asv/main.py", line 36, in main
    result = args.func(args)
  File "/usr/lib64/python3.4/site-packages/asv/commands/__init__.py", line 47, in run_from_args
    return cls.run_from_conf_args(conf, args)
  File "/usr/lib64/python3.4/site-packages/asv/commands/run.py", line 122, in run_from_conf_args
    skip_existing_commits=args.skip_existing_commits
  File "/usr/lib64/python3.4/site-packages/asv/commands/run.py", line 197, in run
    benchmarks = Benchmarks(conf, regex=bench)
  File "/usr/lib64/python3.4/site-packages/asv/benchmarks.py", line 285, in __init__
    for benchmark in benchmarks:
  File "/usr/lib64/python3.4/site-packages/asv/benchmarks.py", line 325, in disc_benchmarks
    dots=False)
  File "/usr/lib64/python3.4/site-packages/asv/environment.py", line 429, in run
    self._executable] + args, **kwargs)
  File "/usr/lib64/python3.4/site-packages/asv/util.py", line 431, in check_output
    raise ProcessError(args, retcode, stdout, stderr)
asv.util.ProcessError: Command '/usr/bin/python3.4 /usr/lib64/python3.4/site-packages/asv/benchmark.py discover benchmarks /tmp/tmp40ddvr65' returned non-zero exit status 1

[Current coverage][cc-pull] is 84.80%

Merging [#158][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #158   diff @@
==========================================
  Files            17         17
  Lines          2047       2047
  Methods           0          0
  Messages          0          0
  Branches        208        208
==========================================
  Hits           1736       1736
  Misses          251        251
  Partials         60         60

Powered by Codecov. Last updated by [2af1e46...42b7033][cc-compare]
[cc-base-branch]: https://codecov.io/gh/PyWavelets/pywt/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/PyWavelets/pywt/compare/2af1e46a1396f1c10a7f34e83f0439c105a49073...42b703328000f3c0fa7fb8679136f28be5e5bd74
[cc-pull]: https://codecov.io/gh/PyWavelets/pywt/pull/158?src=pr

While we're in PR merging mode: would be good to get this in. We can worry about a site for displaying results and fine-tuning later, but once we have the beginning then new PRs can add benchmarks quite easily for new functions or performance improvements.
asv had a new release, so hopefully --python=same works for you now. With asv run --quick things work partially for me, takes about 10 min including creating 2.7 and 3.4 envs and building pywt in those. The 2.7 benchmarks all work as expected, the 3.4 ones fail - that's not my default Python, so it's probably something in my environment.
To get this in, I suggest:

address the couple of open review comments
rename asv/ to benchmarks/
add a benchmarks/README with basic instructions (a simple pip install asv and then asv run --python=same will do)

The full run indeed takes forever, not yet sure why.

just tried this again using asv 0.2.
asv run --python=same --quick worked for me and took < 5 mins
Setting "environment_type": "conda" in the config also worked and tests completed for 2.7 and 3.4

continued in gh-264, so closing this PR.


Uploaded YogaInfo

Have you read the Contributing Guidelines ?
Yes.
Description
I have uploaded YogaInfo folder
Fixes #975

Checklist

[x] I've read the contribution guidelines.
[ ] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

@Yashoda2003 Update on this.


Device back button Doesn't count previous screen / page which wrapped with the widget

while pressing back button on the device it doesn't store previous screen from the cache history instead it always pops into very start of the page that app starts with, in my case it skipped 3 screens!

Yes. Currently, it doesn't support it. I will work on that feature and will add it soon. Thank you for reporting an issue.

Yes. Currently, it doesn't support it. I will work on that feature and will add it soon. Thank you for reporting an issue.

NP, hope you give an update as my App is completely done except for that package is a huge stepdown for me unless a fix is on the way i would really appreciate it.

I'm working on it and will provide a new dev version soon.

The new version has no support for the back button yet. But the previous screen is stored, and when it is online, you can see the old screen instead of opening the very first screen.
For example:
You navigated to,

Splashscreen
Login Screen
Home Screen
Accounts Screen

Now let's say there is no internet after navigating to the Accounts screen, and a new full screen (No internet screen) is displayed.
** After a few seconds, the internet is back. **
You will see the "Accounts screen" rather than the Home Screen.


Fix to not allow mixed dash and slash in parameters prefix

PR Summary
Proposed fix for issue #12013 to not allow \- as legal prefix for a command line parameter.

PR Context
Make sure that both /- and -/ are not legal command line parameters prefix, with added tests to ensure -, -- and / are legal prefixes.

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

@davidBar-On Thanks for your contribution!


Resolve Name Conflict

The Instantiate Extension functions were conflicting with the actual
static methods in the UnityEngine.Object class, making calling the
functions impossible.
Since the objects were already loaded in memory by the time the functions
are called. I renamed them to Copy instead (since that's basically what
the functions are doing at this point).

Can you give an example of how would the conflict happen? If you're inside a MonoBehaviour method and do: Instantiate, it should resolve to the static one unless you said this.Instantiate.

public static class Visuals
{
    public static void CreateEffect(ParticleSystem effect prefab)
   {
        effect.Instantiate ();
   }
}

It is a nice shortcut when not working strictly in a MonoBehaviour or ScriptableObject context.

Well then, I'd use InstantiateNew (or NewInstance) instead of Copy, cause Copy implies that you have an source and you're copying to it from a destination.

Would Clone be OK? or would that be confusing with ICloneable implementers?
If not, I'll just use NewInstance.

Clone could be used as an extension method to do a serialize/deserialize to an object (deep copy) and ICloneable like you said. Just use InstantiateNew or NewInstance. I'd use the former cause it's sort of obvious what it's doing: Instantiate as opposed to NewInstance which could imply it's doing something different, just my opinion. Both are fine.

It should be to InstantiateNew now.


subscribers.find(id) returns array of group objects instead of array of group ids as documented

The documentation for subscribers.find(id) indicates that it returns response containing a SubscriberObject. This object indicates that the groups should be returned as string array, however when I use the SDK it actually returns an array of group objects:

The API does the same so I assume this is due to the ambiguity in the API docs about the structure of the groups field (esp since on create/update it's described as a list of IDs)

Fix released in 1.2.2


Calling stage() when already in a transaction should be a no-op

When writing code that uses transactions, applications need to keep track of whether the client they are using is in a transaction so that they don't accidentally start a new one before committing.

public void methodA() {
    concourse.stage();
    try {
        // do some work
        concourse.commit();
    }
    catch(TransactionException e) {
        concourse.abort();
    }
}

public void methodB() {
    concourse.stage();
    try {
        // do some work
        methodA();
        concourse.commit();
    }
    catch(TransactionException e) {
        concourse.abort();
    }
}

In the example above, the work that methodA() is done when called from methodB is an extension of the work methodB is doing, so the fact that methodB is already in a transaction should already be considered by the client when methodA is called
I can't think of a reason why the behaviour of calling stage when already in a transaction is to create a new transaction. I think its best to simply make it a no-op and continue with the current transaction, if it exists. This should be a small change.
Each client should just to see if the transaction token is not null, and if that is the case just return immediately.
We could implement this on the server if we edited the thrift API to take a transaction token in the stage method.

One thing to think about:
In the example above, methodA calls commit, which can prematurely commit the work that methodB is doing if the transactions are automatically merged. To handle this we would need to add logic in the server to keep track of how many times stage has been called and only truly commit when commit has been called enough times to match how often stage has been called.
I'm not crazy about doing that kind of work on the server, but am open to it. At a certain point it seems lik we're starting to implement nested transactions, which may or may not be a good thing. In general I think the semantics of nested transactions are complicated and I prefer to keep things simple, even if that means that the client should maintain its logic about whether it is in a transaction or not. But I can be convinced otherwise here.

@jtnelson Could you please guide me on this?

We can assigning a static`` counter inside stage()orcommit()``` to check how many times stage has been called. This can be somewhat like this-
stage() called then-> counter ++
commit() called then-> counter --
if counter == 0:
either transactions haven't started yet or are completed.
else:
nested transactions in process.
specific logic needed here

@iamutkarshtiwari We can't use a static counter, because we'd need to keep track of transaction calls PER client. If we're going to have a counter, it'll need to either be something that embedded in the transaction token itself OR held within a collection that maps the transactiontoken to a counter

@as577 take a look at this as first bug fix

@jtnelson, just made a pull request for this issue


Add cask for kullo app

tested locally and audited
For the interested: Kullo is a new app/ecosystem for encrypted communication similar to e-mail.

Thank you for the contribution. It will be merged in a different pull request due to some simple recommended alterations. Your contribution is still included (and still credited to you), with the appropriate modifications, when applicable.
Changes:

Facilitated future updates.
License change.


Add view encapsulation attributes to generated elements to support view encapsulation in Ionic 2

Short description of the problem:
Ionic 2 components doesn't support view encapsulation, the custom view encapsulation attribute generated by angular is not inserted into elements generated by ionic. (Maybe  because view encapsulation is turned off for ionic components? ref)
When I write
<ion-title> Title </ion-title>

ion-title .toolbar-title {
  color: green;
}

it will be generated into:
<ion-title _ngcontent-ily-4="">

 Events

</ion-title>

So the div miss the generated scope for view encapsulation. This means I have no way to manipulate ionic components appearance from encapsulated css code, but only from globally applied one.
What behavior are you expecting?
Ionic components support view Encapsulation.
So the output should be:
<ion-title _ngcontent-ily-4="">

 Events

</ion-title>

Steps to reproduce:

Check out plnkr example
See the scenario described in the intro is valid

insert any relevant code between the above and below backticks

Which Ionic Version? 1.x or 2.x
Ionic 2
Plunker that shows an example of your issue
http://plnkr.co/edit/lPzp95sNdJ9lYViDE0PI?p=preview
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 6.2.0
Gulp version:  CLI version 1.2.1
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.10
Ionic CLI Version: 2.0.0-beta.31
Ionic App Lib Version: 2.0.0-beta.17
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v6.2.1
Xcode version: Xcode 7.3.1 Build version 7D1014

platform:android and platform:windows is affected as well.

Hello, thanks for opening an issue with us! We have turned off view encapsulation with ionic 2 as there are some performance concerns, especially on ios, but because of how our sass is configured, there is no need for it at this time. To get encapsulated styles for each page you can just put your styles in the sass file for that page. I hope this makes sense, thanks for using Ionic! Since this is expected behavior i will be closing this issue for now.

how about styling an isolated custom component? The encapsulated styling is desired to gain reusability.

It will be great if view encapsulation can be enabled on components.
I understand to encapsulated styles for each page make sense but not really for custom components to easily reuse it.

@jgw96 I don't understand how styles being encapsulated per component/page? Here is an example of a background being overridden even thought it's wrapped by the component/page selector:

@jgw96
I know this is an old issue but I'm really expecting view encapsulation feature in order to build scalable apps. The sass configuration you mentioned helps in some degree but not perfect. If my custom component A uses my custom component B, the sass of A can simply overrides the sass of B.
Please consider enable view encapsulation!

I agree. It would be really useful to add the css view encapsulation in Ionic. By the way, what is the exact SASS configuration that mimic the view encapsulation?

bump
This was extremely disappointing news for my team. Really looking forward to this being enabled in the future.

Supports Ionic 3 view encapsulation?


how multi user use a config file

hi
in openvpn multi user can use a certificate file for connect to server
in wireguard  multi user can use a file?

You cannot have peers share the same keys, a new key must be generated for each client.
However, once you generate those keys, all the peers can be stored in a single file that all the clients share, see: https://github.com/pirate/wireguard-docs#sharing-a-single-peersconf-file


Only react-native init gives a deprecation warning, other commands simply fail with no warning

Description
We're trying to integrate the latest react-native main into react-native-windows, and we were pulling our hair out trying to find why our platform CLI commands weren't running anymore.
Our integration PR: https://github.com/microsoft/react-native-windows/pull/13508
We get that the react-native command is being deprecated, but what's the point of the code calling warnWithDeprecated() if that function only gives a warning if you're calling init? Literally every other command could use that notification too (since we use those way more often than init).
Steps to reproduce

Build an app against a 0.76 nightly (say 0.76.0-nightly-20240701-9f6cb21ed)
Run react-native config

React Native Version
0.76.0-nightly-20240701-9f6cb21ed
Affected Platforms
Build - Windows
Output of npx react-native info
yarn run v1.22.22
$ E:\code\rnw\node_modules\.bin\react-native info
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Stacktrace or Logs
yarn react-native config
Debugger attached.
yarn run v1.22.22
$ E:\code\rnw\node_modules\.bin\react-native config
Debugger attached.
Waiting for the debugger to disconnect...
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Waiting for the debugger to disconnect...

Reproducer
https://github.com/Yajur-Grover/react-native-windows/tree/yg-int-7-1
Screenshots and Videos
No response

This looks like a duplicate of https://github.com/facebook/react-native/issues/45461 from @tido64
We already addressed this and fixed it in https://github.com/facebook/react-native/pull/45464
Any nightly after July 17th should behave correctly. Please let us know if this is not the case

Looking at the [yarn.lock


[SPARK-39604][SQL][TESTS] Add UT for DerbyDialect getCatalystType method

What changes were proposed in this pull request?
Add complemented UT for DerbyDialect
Why are the changes needed?
Add UT for existed key function.
If conversion from original data source to Spark data type is needed, the getCatalystType method will be added to specific ${data-source}Dialect class. It's needed to add UT for each of them. Currently, all *Dialect class has UT to cover the method except DerbyDialect.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Unit Test

@srowen It's appreciated if it can be reviewed in your convenience, thanks!

Can one of the admins verify this patch?

Merged to master.


Button to approve or disapprove new screenshot if there is a mismatch

Hi Sudharsan , first of all great work. Second here is my question is there a way to use this library without this adapter part because I already have the S3 bucket part working and the screenshots as well as the report are deployed there. I just want to import the PlayShotMatcher and use it directly is that possible ?
import { test as BaseTest, expect } from '@playwright/test';
import { PlayShot } from 'playwright-cloud-visuals';
const playshot = new PlayShot({
adapter: s3Adapter, // or sshAdapter for SFTP
remotePathDelimiter: 'screenshots',
});
const test =
BaseTest.extend <{ playShot: PlayShotMatcher }>
{
playShot: async ({ page }, use, testInfo) => {
await use(playshot.createMatcher(page, testInfo));
},
};
export default test; Also does the button to disapprove or approve the new different screenshot appear on the playwright report or are you generating a customized report ? Thanks in advance for your reply

Hey @zinaelnahel, we don't have a sophesticated way to updated the screenshots via reporter the this point.
What this library currently does is, users can pass in a parameter during the assertion to let the library know to automatically replace the current screenshot to the s3 bucket.
await playShot.assertPage({
  update: true,
});

Ah ok so we can not see it in the report and then choose to approve or disapprove via a button in the playwright report

Ah no, it not feasible with the current playwright reported. We need to create a custom reporter to able to achieve that.

I managed to overwrite the index.html of the playwright report and add a button when there is an image difference. Do you think it is possible to use your library when clicking the button to trigger your update screenshot function? Or in your library the screenshot is always updated whenever there is a difference?
My problem is I want to only run the update screenshot when the button is clicked. Would appreciate your thoughts


Fixing typos and prettying up some of the documentation.

deferring this until I've added some other changes to the PR


Execute k8s command directly

Hi, How can i execute a kube command directly to get certificate status?

Hi @ahmadabd
Late response, but maybe it'll help someone else.
You can create a CRD:
final class Certificate extends K8sResource implements InteractsWithK8sCluster
{
    use HasStatus;
    use HasAnnotations;

    protected static $kind = 'Certificate';
    protected static $defaultVersion = 'cert-manager.io/v1';
    protected static $namespaceable = true;
}

And then you can query certificates like so:
Certificate::register();
$certificates = $this->cluster->certificate()->whereNamespace('default')->all();


feat: add doctest integration

Description
Describe your changes in detail.
Motivation and Context
Why is this change required? What problem does it solve?
If it fixes an open issue, please link to the issue here.
You can use the syntax close #15213 if this solves the issue #15213

[ ] I have raised an issue to propose this change (required for new features and bug fixes)

Add xdoctest integration and update examples accordingly.
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[X] New feature (non-breaking change which adds core functionality)

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[X] I have read the CONTRIBUTION guide. (required)
[X] My change requires a change to the documentation.
[X] I have updated the tests accordingly. (required for a bug fix or a new feature)
[X] I have updated the documentation accordingly.
[X] I have reformatted the code using make format. (required)
[X] I have checked the code using make lint. (required)
[X] I have ensured make test pass. (required)

Codecov Report
Base: 94.04% // Head: 94.04% // No change to project coverage :thumbsup:

Coverage data is based on head (a0aca34) compared to base (4dac679).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #23   +/-   ##
=======================================
  Coverage   94.04%   94.04%
=======================================
  Files           4        4
  Lines         319      319
=======================================
  Hits          300      300
  Misses         19       19

Flag
Coverage Δ

unittests
94.04% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

optree/ops.py
90.85% <ø> (ø)

optree/registry.py
96.92% <ø> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.