Duplicated table names for Apache Kyuubi / Spark

There is know issue of using dropbox/PyHive to connect Kyuubi/Spark.
A forked acryldata/PyHive fixed the table name issue, when I try to use this one to connect Kyuubi/Spark Thrift Server, I see duplicated table names in the list.
How to reproduce the bug
Follow the docker README to launch apache/superset:1.5.0 in local.
Replace dropbox/PyHive by acryldata/PyHive
docker exec -it -u root superset pip uninstall PyHive
docker exec -it -u root superset pip install acryl-PyHive

Start a Spark Thrift Server. (Got same result when connect to Kyuubi)
Create database connection in Superset

Refresh table list.
Expected results
No duplicated table name in list.
Actual results

Environment

browser type and version: Latest Edge
superset version: superset version: official docker image, apache/superset:1.5.0
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
When click refresh table list button, I see the Spark Thrift Server receives two SHOW TABLES queries in log.
22/05/24 15:00:10 INFO SparkExecuteStatementOperation: Submitting query 'SHOW TABLES IN `tpcds_s1`' with 3e98efb4-780f-43db-afb1-1d204f1009cc
22/05/24 15:00:10 INFO SparkExecuteStatementOperation: Running query with 3e98efb4-780f-43db-afb1-1d204f1009cc

...

22/05/24 15:00:10 INFO SparkExecuteStatementOperation: Submitting query 'SHOW TABLES IN `tpcds_s1`' with cf3fdb93-cad5-43c8-977e-aa18f44068fa
22/05/24 15:00:10 INFO SparkExecuteStatementOperation: Running query with cf3fdb93-cad5-43c8-977e-aa18f44068fa

cc @shirshanka and thanks for maintaining the forked acryldata/PyHive

@bkyryliuk any thoughts on this one?

i also had this problem when Using superset to connect spark with acryl-PyHive，my Environment is：superset 2.1.0、acryl-PyHive 0.6.13、spark 3.2.4

We also have the same problems when connecting to Databricks (superset 2.0.1, spark 3.2.x). Any news about this?

I'm not sure if this is still an issue in current versions of Superset (3.x). If it is, we can re-open this, or feel free to open a new issue with updated context and a reproducible case using example data. We're no longer supporting Superset 2.x or prior, and it's been a while since this thread saw any activity, so I'm closing this as stale.


Google Drive link not working

The link seems to have died. If possible update the link?

Sorry, looks like I accidentally deleted the file on my GD.
Not sure when I can find time to retrain it and post the model. Can you train it, maybe?

sorry sir, I wish I had resources. I will let you know if I manage to
retrain it.
On Mon, Sep 24, 2018 at 7:18 PM Mike Kroutikov<EMAIL_ADDRESS>wrote:

Sorry, looks like I accidentally deleted the file on my GD.
Not sure when I can find time to retrain it and post the model. Can you
train it, maybe?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mkroutikov/tf-lstm-char-cnn/issues/23#issuecomment-423980602,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWEW9mH2-d5BVz8JaaxAJSJiXhl8o730ks5ueOKdgaJpZM4W2D-m
.


Compiler bug when mixing number and Float: [rank = 1]

Quick Summary: It seems it's possible to incorrectly "assert" a number to be a Float in the type annotation below. Instead of noting (in numerous places) that there's a type mismatch, the compiler errors out with the message at the bottom of this issue.
SSCCE
module Main exposing (..)

import Html

main = Html.text ""

sum : List number -> Float
sum items =
  case items of
    [] ->
      0

    first :: rest ->
      sumHelper 0 0 0 rest

sumHelper : number -> number -> number -> List Float -> Float
sumHelper s c t items =
  case items of
    [] ->
      s + c

    first :: rest ->
      let
        correction =
          if s >= first then
            0
          else
            0
      in
      sumHelper 0 0 0 rest

Elm: 0.19.1
Browser: N/A
Operating System: MacOS Catalina 10.15.4 Beta

Additional Details
I was attempting to port a more precise implementation of List.sum from js that can be found here. I eventually ended up in this situation because I wanted to:
sum items / Basics.toFloat (List.length items)

to calculate the mean of a list. Of course, the initial annotation for sum was List number -> number which upsets the / operator. Simply changing the annotation to List number -> Float seemed to work until I tried testing the module!

Here is the compiler error:
> import Test
elm: You ran into a compiler bug. Here are some details for the developers:

    ? [rank = 1]

Please create an <http://sscce.org/> and then report it
at <https://github.com/elm/compiler/issues>

CallStack (from HasCallStack):
  error, called at compiler/src/Type/Solve.hs:206:15 in main:Type.Solve

I don't think I can reduce the SSCCE any further. I'll outline the some small modifications to the example that will trigger the correct type mismatch compiler error below.

Modifying s + c to a constant 0
Modifying either call sumHelper 0 0 0 rest to sumHelper 0 0 0 []
Modifying correction to a constant 0
Removing correction entirely
Removing sum entirely

Possibly a duplicate of #2035.


Minion segment conversion framework

Added interface for record partitioner, transformer, aggregator
Added mapper and reduce record reader
Added segment converter that puts all the logics together

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2e90c64). Click here to learn what that means.
The diff coverage is 79.76%.

@@            Coverage Diff            @@
##             master    #2781   +/-   ##
=========================================
  Coverage          ?   69.27%
=========================================
  Files             ?      908
  Lines             ?    43197
  Branches          ?     5855
=========================================
  Hits              ?    29924
  Misses            ?    11292
  Partials          ?     1981

Impacted Files
Coverage Δ

...data/readers/MultiplePinotSegmentRecordReader.java
73.63% <ø> (ø)

.../core/minion/segment/DefaultRecordPartitioner.java
100% <100%> (ø)

...din/pinot/core/data/readers/RecordReaderUtils.java
69.23% <100%> (ø)

...m/linkedin/pinot/core/minion/SegmentConverter.java
60.78% <60.78%> (ø)

...din/pinot/core/minion/SegmentConverterBuilder.java
82.6% <82.6%> (ø)

.../pinot/core/minion/segment/MapperRecordReader.java
83.33% <83.33%> (ø)

...pinot/core/minion/segment/ReducerRecordReader.java
92% <92%> (ø)

Continue to review full report at Codecov.

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


Adds jenkinsfile used for rebuilding virtualbox instances

Issues:
Fixes #761
Problem:
The jenkinsfiles that are used to support the CI/CD are not in this
repo and need to be added
Analysis:
This adds the first of the files; the one that bootstraps the set
of bigips twice a month
Tests:
none

@zancas


btcdeb not work for P2SH puzzle transaction, show unexpected stack info.

hi @kallewoof
I create a puzzle P2SH output and make a transaction to redeem it.
the redeemScript is just 93016387, and scriptPubKey is [OP_HASH160 hash160(93016387) OP_EQUAL]=(a9143f58b4f7b14847a9083694b9b3b52a4cea2569ed87)
and spend it also just send back to itself.

#redeemScript
  93     01     63      87
  OP_ADD 1Byte  int:99  OP_EQUAL

#scriptSig
  51  01 62    04   93  01  63  87

because this is not normal transaction, and can not create by signrawtransaction bitcoin core cli,
I create by python https://github.com/passionofvc/raw-tx-tool/blob/master/P2SH/redeem_P2SH.py
and broatcast to testnet
https://live.blockcypher.com/btc-testnet/tx/e8a9df4f3b5f5ff454ab96083fca8f8375359df26051ef2831bbc07e93788b27/
./btcdeb --tx=0100000001e2d4fe69fbfa62beac61da2a07ae84d96e3e1c14cdee348cf3c84efd20eda37d00000000085101620493016387ffffffff0160ceab070000000017a9143f58b4f7b14847a9083694b9b3b52a4cea2569ed8700000000 --txin=02000000019729eadf29e14ea589a2ebc7a2341b7d8abcdfe321c253d3234316e76369bc8c000000006a4730440220562f08fb8da9449d69a20ea9009a5f3372c2c01f57072593bceced7dfb10e88902204cf3d2ceff3b4537dad9e06fde49f58d2e02ad10c6786dd867ca65d94fd5f40d01210396a4b78bd85494a32e7189cd597fc4d3f6ad4e8cfc76356eec7339bdd0a9efdbffffffff010055ad070000000017a9143f58b4f7b14847a9083694b9b3b52a4cea2569ed8700000000

btcdeb -- type `./btcdeb -h` for start up options
got transaction:
CTransaction(hash=e8a9df4f3b, ver=1, vin.size=1, vout.size=1, nLockTime=0)
    CTxIn(COutPoint(7da3ed20fd, 0), scriptSig=5101620493016387)
    CScriptWitness()
    CTxOut(nValue=1.28700000, scriptPubKey=a9143f58b4f7b14847a9083694b9b3)

got input tx #0:
CTransaction(hash=7da3ed20fd, ver=2, vin.size=1, vout.size=1, nLockTime=0)
    CTxIn(COutPoint(8cbc6963e7, 0), scriptSig=4730440220562f08fb8da944)
    CScriptWitness()
    CTxOut(nValue=1.28800000, scriptPubKey=a9143f58b4f7b14847a9083694b9b3)

input tx index = 0; tx input vout = 0; value = 128800000
warning: ambiguous input 51 is interpreted as a numeric value; use 0x51 to force into hexadecimal interpretation
warning: ambiguous input 62 is interpreted as a numeric value; use 0x62 to force into hexadecimal interpretation
warning: ambiguous input 93016387 is interpreted as a numeric value; use 0x93016387 to force into hexadecimal interpretation
3 op script loaded. type `help` for usage information
script                                   |   stack
-----------------------------------------+---------
OP_HASH160                               | 43518b05
3f58b4f7b14847a9083694b9b3b52a4cea2569ed |       3e
OP_EQUAL                                 |       33
#0001 OP_HASH160

This should now work correctly as of latest master.
Would you mind double checking and closing the issue if things seem to be correct?

hi @kallewoof
very thanks!, It’s work!

script                                   |   stack
-----------------------------------------+---------
OP_HASH160                               | 93016387
3f58b4f7b14847a9083694b9b3b52a4cea2569ed |       62
OP_EQUAL                                 |       01
#0001 OP_HASH160
btcdeb> step
		<> POP  stack
		<> PUSH stack 3f58b4f7b14847a9083694b9b3b52a4cea2569ed
script                                   |                                   stack
-----------------------------------------+-----------------------------------------
3f58b4f7b14847a9083694b9b3b52a4cea2569ed | 3f58b4f7b14847a9083694b9b3b52a4cea2569ed
OP_EQUAL                                 |                                       62
                                         |                                       01
#0002 3f58b4f7b14847a9083694b9b3b52a4cea2569ed
btcdeb> step
		<> PUSH stack 3f58b4f7b14847a9083694b9b3b52a4cea2569ed
script                                   |                                   stack
-----------------------------------------+-----------------------------------------
OP_EQUAL                                 | 3f58b4f7b14847a9083694b9b3b52a4cea2569ed
                                         | 3f58b4f7b14847a9083694b9b3b52a4cea2569ed
                                         |                                       62
                                         |                                       01
#0003 OP_EQUAL
btcdeb> step
		<> POP  stack
		<> POP  stack
		<> PUSH stack 01
script                                   |                                   stack
-----------------------------------------+-----------------------------------------
                                         |                                       01
                                         |                                       62
                                         |                                       01
btcdeb> step
Drop-in P2SH redeem script
		<> POP  stack
Restoring stack:
- 01
- 62
Script:
- OP_ADD
- 63
- OP_EQUAL
script                                   |                                   stack
-----------------------------------------+-----------------------------------------
OP_ADD                                   |                                       62
63                                       |                                       01
OP_EQUAL                                 |
btcdeb> step
		<> POP  stack
		<> POP  stack
		<> PUSH stack 63
script                                   |                                   stack
-----------------------------------------+-----------------------------------------
63                                       |                                       63
OP_EQUAL                                 |
btcdeb> step
		<> PUSH stack 63
script                                   |                                   stack
-----------------------------------------+-----------------------------------------
OP_EQUAL                                 |                                       63
                                         |                                       63
btcdeb> step
		<> POP  stack
		<> POP  stack
		<> PUSH stack 01
script                                   |                                   stack
-----------------------------------------+-----------------------------------------
                                         |                                       01
btcdeb> step
script                                   |                                   stack
-----------------------------------------+-----------------------------------------
                                         |                                       01
btcdeb> step
at end of script
btcdeb>

I think this may be the bitcoin core daemon bug,
it can not parse non normal script, show below minus number
"scriptSig": {
    "asm": "1 98 -123928979",
    "hex": "5101620493016387"

},
On Tue, Jun 19, 2018 at 3:26 PM kallewoof<EMAIL_ADDRESS>wrote:

Very cool test! First off, I fixed the bug where it interpreted the script
data as numeric, even though it knows it's hex data. The execution still
doesn't work, but the equality check succeeds this time.
I am seeing this as the script sig for tx, is that correct?
  "scriptSig": {
    "asm": "1 98 -123928979",
    "hex": "5101620493016387"
  },

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kallewoof/btcdeb/issues/14#issuecomment-398288338,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWoDpZsexHvG7KmRwoBFA8SpPbxex-ngks5t-JmHgaJpZM4Uq2Z4
.

Glad it works! Btw btcdeb will repeat the last command if you hit enter so you don't have to type out 'step' every time. ;)


make video tutorial of package installation

Closing. See https://github.com/neuropoly/spinalcordtoolbox/issues/592#issuecomment-708090189 for rationale.


Plane summaries

Implements a series of updates:
nd_pixel_positions: A generalisation of the pixel position and coordinate functions

Rebuild the functions in utils/coordaintes.py to merge all of the pixel positions and coordinate functions into a single nd-version
Updates the corresponding code base to use this new method

plane-summaries: Adds a series of summary and plotting method designed to help examining of the big systems to help with debugging.

Adds a general cartesian and angular unit conversion method to be used in the summary and plotting methods
Adds display and summary method to all Spectra, Sources, Optics and Detector classes
Adds plot and summarise methods to the core classes

Also expands the unit conversion functions in utils/units.py


Bump digitalmarketplace-test-utils from git to PyPI

We want to install digitalmarketplace-content-loader from PyPI instead of VCS, so we can rely on Dependabot to update our apps automatically when a new version is available.

@lfdebrux did you mean to include those other commits?

@lfdebrux did you mean to include those other commits?

@lfdebrux did you mean to include those other commits?

Yeah, as it needed to be based off #379

@lfdebrux did you mean to include those other commits?

Yeah, as it needed to be based off #379


Better user experience for setting the flashing address

The default flashing address is 0x1000 which is valid for the ESP32. Newer chips expect the binary to be flashed at 0 address. We could automatically update the address field after detecting the chip.

Thank you for your comment @Jason2866.
Indeed, can also generate merged binaries using esptool.py merge_bin command, they need to be flashed at the address 0.
There has been a feature request for esptool.py to produce .hex files, which would be self-descriptive — a user who got such a file wouldn't need to specify the address at all. Maybe that would be better user experience than automatically setting the flashing address.

Nice, generated firmware files which includes the info on which address the should be flashed would be indeed a great enhancement.

Yes, Intel hex. I did an experiment using this script to generate a .hex file from an IDF build directory. With the get-started/hello_world example:
-rw-r--r--  1 ivan  staff    26K Jan 11 18:47 build/bootloader/bootloader.bin
-rw-r--r--  1 ivan  staff   174K Jan 11 18:48 build/hello_world.bin
-rw-r--r--  1 ivan  staff   558K Jan 13 17:15 build/hello_world.hex
-rw-r--r--  1 ivan  staff   238K Jan 13 17:04 build/merged.bin
-rw-r--r--  1 ivan  staff   3.0K Jan 11 18:47 build/partition_table/partition-table.bin

558kB .hex file size is almost 3 times larger than the sum of the .bin file sizes. It's also twice as big as the "merged" binary. Is that going to be a problem? I don't think so, most people probably wouldn't notice much difference. When downloading such a file, it is likely to be gzip-encoded during transfer. So the only difference is storing 558kB vs 238kB on the PC.

Yep, that's true indeed.
Most browsers will try to request a GZipped transfer and if the server accepts it, it will probably be about the same size as the bin files GZipped.


ServiceDiscovery not working on WiFi connection

Hello,
using this lines of code
[TestMethod]
public void DiscoverServiceInstanceRemote()
    {
        var service = new ServiceProfile("bar", "_foo._tcp", 4711);
        var done = new ManualResetEvent(false);
        var sd = new ServiceDiscovery();
        sd.ServiceInstanceDiscovered += (s, e) =>
        {
            if (e.ServiceInstanceName == service.FullyQualifiedName)
            {
                Assert.IsNotNull(e.Message);
                done.Set();
            }
        };
        try
        {
            sd.QueryServiceInstances(service.ServiceName);
            Assert.IsTrue(done.WaitOne(TimeSpan.FromSeconds(5)), "instance not founde");
        }
        finally
        {
            sd.Dispose();
        }
    }

are working pretty fine as long as the machine is connected via cable. As soon as I switch to Wifi it doesn´t.
What I´m doing wrong?

I am having a similar issue. I cannot advertise or discover over my Wi-Fi Interface. Tested with the example programs in the repo ("Spike"). Windows 10, Visual Studio 2022. When running the programs Windows Firewall prompts me to give the process access to the Wifi interface.

I had the same issue for a while. My solution was that Windows automatically set the WiFi to public instead of private which in turn blocks mdns. Go to Settings => Network and Internet and then on the top it should tell you what kind of network it thinks you have.

Thank you! That makes so much sense.


Restore package-lock

Fixes #242
Fixes #241
and prevents breakages like it in the future.

This was caused by gitignoring and removing package-lock.json in https://github.com/punkave/sanitize-html/pull/240/files#diff-32607347f8126e6534ebc7ebaec4853d. What is the reason this was done? It seems that without the lockfile, some devDependency was silently updated, which changed the build output. This is exactly what lockfiles are there for to prevent - making the build reproducible and deterministic, so breakages can't suddenly happen without any changes to master. Please keep a lockfile checked in.

#244  should still be merged.

I didn't find that this was necessary. With the "browser" keyword removed, I get a good build even if I remove package-lock.json. And it never makes sense to suggest package-lock.json is going to be honored in a library because it won't be (project level npm install will ignore any package-lock other than its own). So I think this can be closed, but let me know if 1.18.4 doesn't do the trick for you.

It won't be honored for dependencies, which is why dependencies should be updated often (best automatically), but it will be for devDependencies. As a user, it makes me nervous to use this library if between releases a bug in e.g. Babel or Browserify could cause a faulty build.
If the bug was introduced by #226, how come it was introduced between 1.18.2 and 1.18.3? 1.18.3 does not include any change to the browser field, it already existed before: https://github.com/punkave/sanitize-html/compare/1.18.2...1.18.3
If the tags are not wrong, this must have been caused by a silent dependency update through removal of the lockfile.

Apologies for commenting on closed bugs, but just to resolve the mystery, it turns out adding "browser": "dist/sanitize-html.js" to package.json does actually cause the build of sanitize-html.js to fail in this way, presumably because browserify tries to import the package in the working directory, honouring the browser key, so this explains how the bad build happened.

If the bug was introduced by #226, how come it was introduced between 1.18.2 and 1.18.3? 1.18.3 does not include any change to the browser field, it already existed before: 1.18.2...1.18.3

@felixfbecker see the 2nd problem over in https://github.com/punkave/sanitize-html/issues/241#issuecomment-409881724 - #226 was only actually published to npm in 1.18.3.

Ah, that makes sense, so it was the tags being wrong. Sorry for the confusion.


Unable to resolve - hiltMavericksViewModelFactory

From where do I get the dependency for - hiltMavericksViewModelFactory.
The docs do not have any reference to it.

Do I need to copy the whole mavericks-hilt folder to my app? or is there a Gradle dependency that I can add?

It's on Maven Central just like the main dependency
https://mvnrepository.com/artifact/com.airbnb.android/mavericks-hilt


Refresh the table on enter key

Instead of exiting the program and having the user to run it again every time, It's better to keep the window open and just wait for an enter key to update the table, this seems more convenient.
Note: the presence function is renamed to get_presence so that it does not collide with the local variable's name (Python complains about this)

I'll rename the variable but on the key is bad idea because it adds keyboard input delay.

I'll rename the variable but on the key is bad idea because it adds keyboard input delay.

Sorry, i didn't really get what you mean by keyword input delay here. The table is initially displayed once and then it just stays there until the user presses enter key, then it just refreshes itself.

hmmmm I thought you would like press keybind in-game and it would refresh. Alright I'll commit in few hours

hmmmm I thought you would like press keybind in-game and it would refresh. Alright I'll commit in few hours

Thanks! sounds good.
Also in-game keybind sounds like a good separate future, but I doubt it would help much though because most users would need to alt-tab out of the game to access this table anyways, so just hitting enter key inside the console window of this app sounds like a simpler and more user friendly option.

ye but thing about in-game keybinds is that they add a lot of input delay

Alright added the feature tutorial video coming soon


Set assets.export_concurrent to false

There seems to be issues with Sprockets version 4, errors like [1]
occur when precompiling assets.
A suggestion I've seen to avoid this is to add this configuration [2].
1: SassC::SyntaxError: Error: error in C function font-path: undefined
method `[]' for nil:NilClass
2: https://github.com/rails/sprockets/issues/581#issuecomment-486984663

I'm still seeing this error in CI:
[SassC::FunctionsHandler] undefined method `[]' for nil:NilClass
rake aborted!
SassC::SyntaxError: Error: error in C function font-path: undefined method `[]' for nil:NilClass
        on line 2:11 of ../../bundles/ruby/2.6.0/gems/bootstrap-sass-3.4.1/assets/stylesheets/_bootstrap-sprockets.scss, in function `font-path`
        from line 2:11 of ../../bundles/ruby/2.6.0/gems/bootstrap-sass-3.4.1/assets/stylesheets/_bootstrap-sprockets.scss, in function `twbs-font-path`

It looks like all recent builds of Contacts have failed and it's blocked a GOV.UK Content Schemas PR.

I'm hoping this change will also help https://github.com/alphagov/contacts-admin/pull/663


Using Parallel WaveGan as vocoder

Hi. I trained the tacotron2 model and can successfully synthesize voice by using WaveGlow as vocoder. However, when I turned to the Parallel WaveGan (https://github.com/kan-bayashi/ParallelWaveGAN) , the synthzised waveform is quite strange:
.

(In the training time, the hop_size, sample_rate and window_size were set as the same for the tacotron, WaveGlow and waveGan model.)
Previously, I successfully used WaveGan as vocoder to synthesize speech from the FastSpeech acoustic model. The only difference here is that in FastSpeech, mel-spectrogram features were normalized to have zero mean and unit variance before training.
My question is, in your tacotron2 implementation, except audio_norm = audio / self.max_wav_value , is there any other preprocessing of the input mel-spectrogram features? Or, could you kindly give me some advice?
Thank you very much!

Take a look at this and this

Thank you. For anyone who tried tacotron2+WaveGan and also encountered this problem, see https://github.com/kan-bayashi/ParallelWaveGAN/issues/169


How to load init script before Stryker run

Hi. I have one init script which is required to load some env variables before executing mocha. Here's that script in my package.json.
...
"test": "nyc mocha -r ./tests/setup-mocha.js --recursive tests/unittests/**/*.test.js --exit",
"test-stryker": "mocha -r ./tests/setup-mocha.js tests/unittests/schemas/someSchema.test.js --exit",
...
Now that I want to run Stryker, this setup-mocha.js is not loaded anymore and my env variables are undefined.
module.exports = function(config) {
config.set({
mutator: "javascript",
packageManager: "npm",
reporters: ["clear-text", "progress"],
testRunner: "command",
commandRunner: {
command: "npm run test-stryker",
},
transpilers: [],
mutate: ["schemas/someSchema.js"]
});
};
How to force that script to be loaded before starting the Stryker?
Thanks in advance.

Hi @Saro-DA thanks for opening an issue.
Your config looks fine to me. Could you give some more info?

Does npm run test-stryker work as expected?
Does it also work if you run it from inside the sandbox directory?
The sandbox directory can be found in the .stryker-tmp folder. You should be able to just run npm run test-stryker from there as well

Hi @nicojs. Thanks for replying.
Yes it does work as expected. And even the Stryker run works fine if I manually set those environment variables inside my *.test.js files.
Inside my setup-mocha.js file, I'm using dotenv module to load environment variables from a config file. And the issue seems to be that this module is not able to do that when is executed with 'Stryker run'. It works fine with npm run test-stryker though.

I'm using dotenv module to load environment variables from a config file.

Aha, I think that is your smoking gun. Is your .env file included in the list of files inside sandboxes?
To see that, run npx stryker run --logLevel debug and look for: 18:19:11 (8560) DEBUG InputFileResolver All input files: [
There, all files are listed. Is your .env one of them?

#1343 might be a duplicate of this issue. It contains a workaround.

@nicojs, @simondel: Is there any way how to execute command before standard stryker run. Basically, I want to convert paths form @account to ../../account. I can do it by script, but it will modify the files. I would like to do it in a sandbox before the mutation starts.
commandRunner: {
command: "npm run modify-paths",
},
it doesn't work because it is just an alternative for testRunner.
Thank you

How do you run your tests right now @Mrna1? You may be able to use that command in the test command test runner:
commandRunner: {
command: "npm run modify-paths | npm test",
},

(the command above will probably only work on windows. Perhaps npm test is good enough for your use case)

@simondel: Thank you, unfortunately, it fails, because the paths are not modified :( So there possibility to get logs from my modification functions? Maybe there are different paths in the sandbox
11:33:17 (25716) DEBUG ChildProcessProxy Disposing of worker process 7916
11:33:17 (25716) DEBUG ChildProcessProxy Kill 7916

I don't know what your modifications do exactly and what they usually log. You could try running stryker with log level 'trace': https://www.npmjs.com/package/@stryker-mutator/core#loglevel--string-


Add ORD links to the CAPs defaults index page, next to the list of APIs and Events

It would be great to have at least the ORD configuration endpoint (./well-known/open-resource-discovery listed on the default index page. Potentially one could also expand it to also include the ORD document directly....

+1

It would be great to have at least the ORD configuration endpoint (./well-known/open-resource-discovery listed on the default index page. Potentially one could also expand it to also include the ORD document directly....

Could you please explain what you mean by the "default index page". Something like the following:

It would be great to have at least the ORD configuration endpoint (./well-known/open-resource-discovery listed on the default index page. Potentially one could also expand it to also include the ORD document directly....

Could you please explain what you mean by the "default index page". Something like the following?

Yes, exactly this page, I meant.


it always report "Please install python and pip first" even if I already installed

Issue Type: Bug
In my win10 system I already installed python2.7 and pip19.0.1 in anaconda. But the alios-studio always report cannot find python and pip and report "Please install python and pip first". It cannot recognize my anaconda python system . Is there any way to setup the python path manually?
Extension version: 0.10.8
VS Code version: Code 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-12T02:20:54.427Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz (4 x 2400)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.59GB (2.25GB free)

Process Argv

Screen Reader
no

VM
0%

negative, this has nothing related with the #50286

Please file issue against the alios-studio extension. Thanks!


Update Data Uses for 1.4

Closes #104
Reference Confluence Document
Code Changes

[x] Top-Level Changes

[x] add analytics
[x] add employment
[x] add finance
[x] add operations
[x] add sales
[x] remove advertising at the top-level (becomes a sub-category under marketing)
[x] rename advertising to marketing
[x] rename provide to essential

[x] Second level changes

[x] Advertising is now under marketing
[x] add legal_obligation to the new essential top-level category

[x] Advertising Restructure

[x] first_party.personalized -> marketing.advertising.third_party.targeted
[x] add marketing.advertising.third_party.targeted (takes the place of the following 3 removed ones)
[x] remove advertising.third_party.personalized
[x] remove third_party_sharing.personalized_advertising
[x] remove advertising.third_party

Steps to Confirm

[ ] list any manual steps taken to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
[ ] Documentation Updated
[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
Write some things here about the changes and any potential caveats

@NevilleS @mfbrown ready to get y'all's eyes on this 🙂 I did my best to translate the document into exactly the code changes required above, so please double-check that I didn't misunderstand the requirements!

@NevilleS I'm actually not sure how to generate the CSV versions...I'll poke around though, is this a feature/script we have somewhere?

OK I pushed some local fixes to get the taxonomy visualization updated 👍

Oh- one thing I just realized. When I exported our worksheet to CSV the ~strikethrough~ styling is lost so you won't see that two of the data uses are meant to be deleted entirely:

third_party_sharing.payment_processing
third_party_sharing.fraud_detection

@ThomasLaPiana One requested change... We've updated the descriptions one more time for these. You can find them here: https://docs.google.com/spreadsheets/d/1FNg8vHBmEmWx8SGNxFOutSGxqGULwftzytrsXdBDM2k/edit?usp=sharing
Can you update the descriptions to match, then we're good to go from my side.

@ThomasLaPiana added a few notes on some minor typos or stylistic changes. Can we also add one last use: essential.service.authentication with the description: "Authenticate users to the product, service, application or system."
After that, I think we're good.

confirmed by @mfbrown via slack this is the final change and is good to ship


Interface 'CheckBoxProps' cannot simultaneously extend types 'DataProps' and 'HTMLAttributes'.   Named property 'defaultChecked' of types 'DataProps' and 'HTMLAttributes' are not identical.

Problem description
使用create-react-app --scripts-version=react-scripts-ts创建的空项目
使用readme的代码
编译时出现的问题
 "dependencies": {
    "react": "^16.2.0",
    "react-dom": "^16.2.0",
    "react-scripts-ts": "2.10.0",
    "react-uwp": "^1.1.8"
  },
  "scripts": {
    "start": "react-scripts-ts start",
    "build": "react-scripts-ts build",
    "test": "react-scripts-ts test --env=jsdom",
    "eject": "react-scripts-ts eject"
  },
  "devDependencies": {
    "@types/jest": "^22.0.1",
    "@types/node": "^9.3.0",
    "@types/react": "^16.0.34",
    "@types/react-dom": "^16.0.3"
  }

Is there a workaround for this issue?

no，type declaration document(*.d.ts) of this project is wrong.
i gave up create-react-app

Thanks for the response. So is this a create react app issue or a react-uwp issue?

Hopefully someone will let users know when this will be usable in typescript projects. They really should put a disclaimer saying that typescript definitions are messed up.

tsconfig.json
{ "compilerOptions": { ... "skipLibCheck": true } }


Remove deprecated arg in M.object.align_to()

Overview: What does this pull request change?

Motivation and Explanation: Why and how do your changes improve the library?

Links to added or changed documentation pages

Further Information and Comments

Closes #3162.

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Thanks for cleaning this up!

Related issue: #113


ci: correct indentation on mini-e2e-helm

The trailing '}' needs to be one the next line.
Signed-off-by: Yug<EMAIL_ADDRESS>

/retest ci/centos/job-validation

/retest ci/centos/job-validation

@mergifyio rebase


feat(website): Updated landing page and added style-guide route

Added tailwind for styling
Added google fonts
Added image assets
Added headlessui for additional components
Added heroicons as icon set
Added initial attempt at landing page
Added /style-guide route for quick access to new style guide

@thantos Please review and let me know if any changes are needed


Docker metrics support is currently broken

Describe the bug
When using the --docker-metrics flag, we see a bunch of errors. Firstly, there is an error of the form
4858 ERROR [main] DockerHelper Image prom/graphite-exporter not found, unable to automatically pull image. Check docker images

This can be resolved by manually downloading the requisite images and re-tagging without the docker.io/ component at the front of the image name.
Running again yields another error:
3881 ERROR [main] ERRORHANDLER {"message":"Cannot link to a non running container: /prom AS /grafana/prom"}

   3881 ERROR [main] ERRORHANDLER for the full stack trace, run with --show-stacktraces

Still looking for a workaround on that.
Additional context
RHEL 7, nb version 4.15.47.

So it looks like this was just because I was using RHEL 7, which has a fairly old kernel and doesn't support docker very well. Closing.

Thanks for the follow-up. Maybe we can find a way to make this fail-fast with a helpful message.


Intermittent Audio Recognition Failure in Real-Time Transcription

Description
We are experiencing intermittent failures in audio recognition when using the Google Speech-to-Text v1 library for real-time transcription in our .NET project. The issue manifests as follows:

The frontend opens a WebSocket connection and sends audio data every 500ms to our API.
Sometimes, the audio is not recognized, and the frontend doesn't receive any transcription.
After one minute of inactivity, our backend closes the WebSocket connection from the finally block.
Errors are visible in the Cloud Speech-to-Text API dashboard, but these errors are not captured in our backend code.

Current Implementation

We use StreamingRecognize for real-time audio transcription.
Audio is processed in chunks sent over WebSocket.
A cancellation token is used to limit the transcription time to 6 minutes.
The backend attempts to handle various scenarios, including websocket closure and error reporting.

Problem
Errors reported in the Cloud Speech-to-Text API dashboard are not propagated to our backend code.
Additional Information

.NET version: 8
Google.Cloud.Speech.V1 library version: 3.8.0

Current Code
public async Task TranscribeAudioAsync(WebSocket webSocket)
{
    using var cts = new CancellationTokenSource(TimeSpan.FromMinutes(6));

    try
    {
        await TranscribeAudioInternalAsync(webSocket, cts.Token);
    }
    catch (OperationCanceledException)
    {
        logger.LogInformation("Transcription cancelled after 6 minutes");
        await CloseWebSocketWithMessageAsync(webSocket, "Transcription time limit reached", WebSocketCloseStatus.NormalClosure, cts.Token);
    }
}

private async Task TranscribeAudioInternalAsync(WebSocket webSocket, CancellationToken cancellationToken)
{
    string conversationId = null;
    var transcription = new StringBuilder();

    try
    {
        var streamingConfig = new StreamingRecognitionConfig
        {
            Config = new RecognitionConfig
            {
                Encoding = RecognitionConfig.Types.AudioEncoding.WebmOpus,
                SampleRateHertz = 16000,
                LanguageCode = "en-US",
            },
            InterimResults = true,
        };

        using var streamingCall = _speechClient.StreamingRecognize();
        await streamingCall.WriteAsync(new StreamingRecognizeRequest
        {
            StreamingConfig = streamingConfig
        });

        conversationId = await GetConversationIdAsync(webSocket, cancellationToken);

        var receiveTask = ReceiveAudioDataAsync(webSocket, streamingCall, cancellationToken);
        var responseTask = ProcessTranscriptionResponsesAsync(webSocket, streamingCall, transcription, cancellationToken);

        await Task.WhenAny(receiveTask, responseTask);

        await streamingCall.WriteCompleteAsync();

        if (!string.IsNullOrEmpty(conversationId))
        {
            await SaveUserResponseToDatabase(conversationId, transcription.ToString());
        }
    }
    catch (Exception ex)
    {
        logger.LogError(ex, "Error during audio transcription for conversation {ConversationId}", conversationId);
        await CloseWebSocketWithErrorAsync(webSocket, "Internal server error occurred", cancellationToken);
    }
    finally
    {
        if (webSocket.State == WebSocketState.Open)
        {
            await webSocket.CloseAsync(WebSocketCloseStatus.NormalClosure, "Transcription completed",
                cancellationToken);
        }
    }
}

private async Task<string> GetConversationIdAsync(WebSocket webSocket, CancellationToken cancellationToken)
{
    var buffer = new byte[1024];
    var result = await webSocket.ReceiveAsync(new ArraySegment<byte>(buffer), cancellationToken);

    if (result.MessageType != WebSocketMessageType.Text)
    {
        await CloseWebSocketWithErrorAsync(webSocket, "First message must be conversationId", cancellationToken);
        return null;
    }

    var conversationId = Encoding.UTF8.GetString(buffer, 0, result.Count);

    if (string.IsNullOrEmpty(conversationId))
    {
        await CloseWebSocketWithErrorAsync(webSocket, "Invalid conversationId", cancellationToken);
        return null;
    }

    var transcriptionResult = new TranscriptionResult
    {
        IsFinal = false,
        Text = "Transcription started"
    };
    await SendTranscriptionResultAsync(webSocket, transcriptionResult, cancellationToken);

    return conversationId;
}

private async Task ReceiveAudioDataAsync(WebSocket webSocket, SpeechClient.StreamingRecognizeStream streamingCall, CancellationToken cancellationToken)
{
    while (webSocket.State == WebSocketState.Open && !cancellationToken.IsCancellationRequested)
    {
        var buffer = new byte[BufferSize];
        var result = await webSocket.ReceiveAsync(new ArraySegment<byte>(buffer), cancellationToken);
        logger.LogInformation("Received message of type: {MessageType} Bytes received: {BytesReceived}",
            result.MessageType, result.Count);

        if (result.MessageType == WebSocketMessageType.Binary)
        {
            await streamingCall.WriteAsync(new StreamingRecognizeRequest
            {
                AudioContent = Google.Protobuf.ByteString.CopyFrom(buffer, 0, result.Count)
            });
        }
        else if (result.MessageType == WebSocketMessageType.Close)
        {
            await webSocket.CloseAsync(WebSocketCloseStatus.NormalClosure, "Closing", cancellationToken);
            break;
        }
    }
}

private async Task ProcessTranscriptionResponsesAsync(WebSocket webSocket,
    SpeechClient.StreamingRecognizeStream streamingCall, StringBuilder transcription,
    CancellationToken cancellationToken)
{
    await foreach (var response in streamingCall.GetResponseStream().WithCancellation(cancellationToken))
    {
        foreach (var result in response.Results)
        {
            var transcriptionResult = new TranscriptionResult
            {
                IsFinal = result.IsFinal,
                Text = result.Alternatives[0].Transcript
            };

            if (result.IsFinal)
            {
                transcription.Append(transcriptionResult.Text).Append(' ');
                transcriptionResult.Text = transcription.ToString();
            }

            await SendTranscriptionResultAsync(webSocket, transcriptionResult, cancellationToken);
        }
    }
}

private async Task SendTranscriptionResultAsync(WebSocket webSocket, TranscriptionResult result,
    CancellationToken cancellationToken)
{
    var json = JsonSerializer.Serialize(result);
    var buffer = Encoding.UTF8.GetBytes(json);
    await webSocket.SendAsync(new ArraySegment<byte>(buffer), WebSocketMessageType.Text, true, cancellationToken);
}

private async Task CloseWebSocketWithErrorAsync(WebSocket webSocket, string reason,
    CancellationToken cancellationToken)
{
    if (webSocket.State == WebSocketState.Open)
    {
        await webSocket.CloseAsync(WebSocketCloseStatus.InternalServerError, reason, cancellationToken);
    }
}

private async Task CloseWebSocketWithMessageAsync(WebSocket webSocket, string message, WebSocketCloseStatus status, CancellationToken cancellationToken)
{
    if (webSocket.State == WebSocketState.Open)
    {
        try
        {
            await webSocket.CloseAsync(status, message, cancellationToken);
        }
        catch (OperationCanceledException)
        {
            // If the cancellation token is triggered during the close operation,
            // we'll have to close the connection abruptly
            await webSocket.CloseOutputAsync(WebSocketCloseStatus.EndpointUnavailable, "Operation timed out", CancellationToken.None);
        }
    }
}

private async Task SaveUserResponseToDatabase(string conversationId, string transcription)
{
    var languageAssessment = await _dbContext.LanguageAssessments
        .SingleOrDefaultAsync(x => x.ConversationId == conversationId && x.UserProfileId == _userContext.Id);

    if (languageAssessment == null)
    {
        throw new EntityNotFoundException("Language assessment not found.");
    }

    var userLanguageAssessmentResponse = new LanguageAssessmentResponse
    {
        Role = LanguageAssessmentRole.User,
        CreatedAt = DateTime.UtcNow,
        LanguageAssessmentId = languageAssessment.Id,
        Text = transcription
    };

    dbContext.LanguageAssessmentResponses.Add(userLanguageAssessmentResponse);
    await dbContext.SaveChangesAsync();
}

The similarity to the long-standing Ruby issue (https://github.com/googleapis/google-cloud-ruby/issues/19236) suggests this might be a broader problem with the Speech-to-Text API itself, rather than a client library-specific issue. I would appreciate any insights or potential solutions of this problem

Hi @jskeet,
Thank you.
Here is screenshot form Google Сould dashboard

It would be nice to be able to reproduce this without any web sockets involved. Are you able to reproduce this in a console app, ideally with just a file providing the audio data, to make it easily reproducible?
(I do wonder whether this is partly due to your use of Task.WhenAny - you're basically completely ignoring the result of whichever task completes second, so if that "second to complete" task is faulted, you won't see that in your code. If you're not getting any responses for a minute, then closing the web socket, the eventual error response from the API may be invisible for that reason.)

Hi @jskeet,
I’ll try to reproduce it in a console app after my vacation. Thank you for your response! :)

@datvladik I'll be closing this issue as we cannot really work on it until we have a repro. When you are ready to provide that, leave a comment and we'll reopen. Thanks!


JobServiceClient: createBatchJobs

Hi, I'm opening this issue to report a problem I have using the JobServiceClient provided by the PHP client of Google Cloud Talent (V4Beta1)
I am following the documentation as I understand it :
$operationResponse = $this->jobClient->batchCreateJobs($this->formattedParent, $jobs);   $operationResponse->pollUntilComplete();   if ($operationResponse->operationSucceeded()) {             $result = $operationResponse->getResult();             dump($result);    } else {             $error = $operationResponse->getError();             dump($error);    }
And this raises a Google\ApiCore\ValidationException: Failed to build request, as the provided path (google.longrunning.Operations/GetOperation) was not found in the configuration. in /srv/api/vendor/google/gax/src/RequestBuilder.php:80
I tried running it having grpc installed, and didn't see a change
Thanks in advance

Great find @dave-web, thanks for the report. There is definitely an issue with the REST transport here, it looks like the config we use to generate the client is missing some key data regarding operations calls that we need. We'll work on getting it updated right away.

I tried running it having grpc installed

I attempted the call with gRPC as well and it succeeded. Could you please try forcing gRPC as the transport by passing in the following configuration to the client and let me know if it helps?
new JobServiceClient([
    'transport' => 'grpc'
]);

Works for me !
Thank you very much !

Glad to hear it @dave-web. We'll keep this open to track fixing the REST transport config. 😃

Closed by: https://github.com/googleapis/google-cloud-php/pull/2495


Add simple regex & orgBody check to account for

Assume that if a client is putting in a

 Story Continues button producing a  red block instead of displaying content


Merge smithy-rs-release-1.x.y into main

Merge it with gh pr merge --admin --merge or manually merge it with the merge commit (not squash merge).

semver hazards failure is transient since aws-sdk-s3 in versions.toml in aws-sdk-rust specifies 1.30.0 but the actual crates is in the process of being uploaded to crates.io. Once the upload is done, we can re-run the failed job.

semver hazards failure is transient since aws-sdk-s3 in versions.toml in aws-sdk-rust specifies 1.30.0 but the actual crate is in the process of being uploaded to crates.io. Once the upload is done, we can re-run the failed job.


Linting for Python

Partly closes #6
Use flake8 and isort for linting. I moved the tox file to an extra file.
@Luthaf if you don't agree we can also move it back...
Also bumped the version of the GH checkout actions to version 3
The rules for flake8 are pretty strict but the code is already in really
good shape. What is leftover are only doc string issues and
too long lines. I will fix this within the next days.

Looking at the test failure, I think we can allow D101 Missing docstring in public class for now

I made flake8 happy. I hope the changes also are fine for you @Luthaf. I tried not to split docstrings too much. Let me know if there is anything to be improved.

Ok, I pushed some more small changes, if you are happy with them we can merge this!

Ok, I pushed some more small changes, if you are happy with them and the two final review comment, we can merge this!

I am happy. If you are with my changes and the linter is also, we can merge! 🚀


recording all moves

Current coverage is 100% (diff: 100%)

Merging #96 into master will not change coverage

@@           master   #96   diff @@
===================================
  Files           8     8
  Lines         339   346     +7
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits          339   346     +7
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 3bb5c23...89f07e3


Support for Conda directory layout, libiomp5md.dll

Description
The MKLConfig.cmake file of 2023.2 version has already support conda directory layout for the mkl dll files. However, it missed the libiomp5md.dll file, which causes "Could NOT find MKL (missing: OMP_DLL_DIR)" error.
This pr hopes fixes the issue.
Checklist
All Submissions

[ ] Do all unit tests pass locally? Attach a log. The pr only add one line in the cmake file, no tests are needed indeed.
[ ] Have you formatted the code using clang-format? The pr only add one line in the cmake file, no tests are needed indeed.

Bug fixes

[ ] Have you added relevant regression tests?
[x] Have you included information on how to reproduce the issue (either in a
GitHub issue or in this PR)?
Related issue: https://github.com/oneapi-src/oneMKL/issues/368

I found that the new cmakefile had already fix the issue, so I closed the PR.


Italian translation.

Attached, if you need, the translation in italian language.
activeusers.php.txt

thanks for that.
I just added it and tagged a new version.


Add support for juju 3.2

Now is been block by:
https://github.com/canonical/prometheus-juju-exporter/pull/66

Closed by #50


[Web] Diacritic test fails on web with html renderer.

As part of the skip test audit:
Attempting to reenable the 'Diacritic' test:
https://github.com/flutter/flutter/blob/9664a109d94699fcc91a97a59e332cc7e5f27a45/packages/flutter/test/painting/text_painter_test.dart#L978
It fails with:
00:02 +11 ~12 -1: Diacritic [E]
  Expected: <17>
    Actual: <16.64>

  ../dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 251:49      throw_
  ../packages/test_api/src/expect/async_matcher.dart.js 164:22                         fail
  ../packages/test_api/src/expect/async_matcher.dart.js 159:12                         _expect
  ../packages/test_api/src/expect/async_matcher.dart.js 86:12                          expect$
  ../packages/flutter_test/src/deprecated.dart.js 7250:12                              expect$
  text_painter_test.dart.js 2601:21                                                    <fn>

Closing as a duplicate of https://github.com/flutter/flutter/issues/99933 which gives more context.


Radius Client isn't working PwCrypt not defined in the Client Class.

Hi I am trying to run the client following the documentation.
It looks like the PwCrypt function is not defined in the pyrad.client.Client.
I am receiving the errors below, the code is:
Please advice if things changed on how to create a password using PwCrypt.
create request
req = srv.CreateAuthPacket(code=pyrad.packet.AccessRequest,
User_Name="wichert", NAS_Identifier="localhost")
req["User-Password"] = req.PwCrypt("password")
/scripts$ ./clientradius.py
Traceback (most recent call last):
File "./clientradius.py", line 16, in
req["User-Password"] = req.PwCrypt("password")
File "build/bdist.linux-i686/egg/pyrad/packet.py", line 189, in setitem
File "build/bdist.linux-i686/egg/pyrad/packet.py", line 126, in _EncodeKeyValues
KeyError: 'User-Password'

It seems that User-Password is not defined in your dictionary.


BLING package update

What changes does this PR introduce?
(Bug fix, feature, docs update, ...)
BLING default package is unchanged. But more options added (Silica cycling, new chem, new MLD diagnostic), and overall structure significantly cleaned.
BLINGv1 (Galbraith et al., 2010) and the new NBLING are cleanly separated.
BLINGv1 is consistent with DIC package inputs.
What is the current behaviour?
(You can also link to an open issue here)
BLING BGC package
What is the new behaviour
(if this is a feature change)?
BLINGv1 and NBLING packages are cleanly separated.
Does this PR introduce a breaking change?
(What changes might users need to make in their application due to this PR?)
Other information:
CHANGES:

Obsolete files removed: bling_production.F, bling_remineralization.F, bling_dvm.F
gchem_forcing_sep.F is revised to call BLING or NBLING
bling_carbon_chem: Replacing apressure with AtmosP, which is default=1 and set to apressure in bling_fields_load if using EXF
bling_carbonate_init: Use silica tracer value instead of a constant value if using SiBLING
bling_carbonate_sys: Use silica tracer value instead of a constant value if using SiBLING
bling_diagnostic_init: Adding diagnostic for MLD, Adding diagnostics for silica when using SiBLING
bling_light: Adding new option to calculate light penetration. Dealing with ice here instead of in bling_readparms
bling_min_val: Keeping track of corrections to tracer fields for closing budgets
bling_mixed_layer: Changing the way MLD is calculated. For old threshold criterion, #define BLING_USE_THRESHOLD_MLD
BLING_OPTIONS: New options for updated carbon chemistry (from J. Lauderdale), mixed layer and light calculation.  Removed option to use EXF atm pressure and atm pCO2; these are now always used if available through EXF. Adding option for SiBLING (silica tracer)
Added bling_bio: Original 6-tracer bling model: Combines bling_production.F and bling_remineralization.F
Added bling_bio_nitrogen: Current bling model, with nitrogen cycling (default has 8 tracers, options for additional tracers for silica and phytoplankton). Combines bling_production.F and bling_remineralization.F
BLING_VARS: Adding variable MLmix_max (max depth over which light and biomass can be mixed; default = 200m)
bling_main: Adding nutrients in runoff
Different order of tracers if using original 6-tracer bling code (USE_BLING_v0)
Call either bling_bio.F or bling_bio_nitrogen.F, depending on which version of the model we’re using.
Adding option for SiBLING (silica tracer)
bling_fields_load: Wind, fice, atmosp, atm_pco2 are read either from file or EXF

Suggested addition to tag-index
(To avoid unnecessary merge conflicts, please don't update tag-index. One of the admins will do that when merging your pull request.)

@mmazloff  I cleaned up few things there. I looked briefly to the code and have 2 comments:

the new CPP option "USE_BLING_V0" is not in BLING_OPTIONS.h ; should appear there (I guess
as #undef) with 2 line of comment. Did not check if there is other new CPP options that are not
documented but if there are, they should be added to BLING_OPTIONS.h too.
we will need to update the reference output of experiment "global_oce_biogeo_bling" in i
verification_other repo. We could decide to submit a PR there once this one is merge ?
And just to open the discussion: Might be good, as some point, to change the order of S/R argument
in pkg/bling, from "myIter, myTime, myThid" to "myTime, myIter, myThid" like every where in the remaining MITgcm code (except in pkg/dic, that we could change later). And since you updated all
pkg/bling code in this PR, we might take this opportunity to make this switch ?

@jm-c

#undef USE_BLING_V0  is in the new checked in BLING_OPTIONS.h. It is on line 14
We will fix to the order myTime, myIter, myThid
Thanks!

@mmazloff you are right, I missed this one.

@jm-c
the ordering of the S/R arguments have been brought into consitency with the rest of the MITgcm. It is now "myTime, myIter, myThid"

@mmazloff I run testreport with debug options that check S/R arguments so I think it's good now.
Just a detail: since you did not provide a suggested "tag-index addition", may be I could let you do this
(i.e., add documentation in tag-index) when we are ready to merge. Let me know.

@jm-c Great! I am ready to see this merged. I can ad documentation to tag-index at that time. How much detail do you recommend. Here are the major changes. Is putting all this too much detail for a tag-index?

bling_carbon_chem: Replacing apressure with AtmosP, which is default=1 and set to apressure in bling_fields_load if using EXF
bling_carbonate_init: Use silica tracer value instead of a constant value if using SiBLING
bling_carbonate_sys: Use silica tracer value instead of a constant value if using SiBLING
bling_diagnostic_init: Adding diagnostic for MLD. Adding diagnostics for silica when using SiBLING
bling_light: Adding new option to calculate light penetration. Dealing with ice here instead of in bling_readparms
bling_min_val: Keeping track of corrections to tracer fields for budgets
bling_mixed_layer: Changing the way MLD is calculated. For old threshold criterion, #define BLING_USE_THRESHOLD_MLD
BLING_OPTIONS:  New options for updated carbon chemistry (from J. Lauderdale), mixed layer and light calculation.  Removed option to use EXF atm pressure and atm pCO2; these are now always used if available through EXF. Adding option for SiBLING (silica tracer)
bling_bio: Original 6-tracer bling model. Combines bling_production.F and bling_remineralization.F
bling_bio_nitrogen: Current bling model, with nitrogen cycling (default has 8 tracers, options for additional tracers for silica and phytoplankton). Combines bling_production.F and bling_remineralization.F
BLING_VARS: Adding variable MLmix_max (max depth over which light and biomass can be mixed; default = 200m)
bling_main: Adding nutrients in runoff. Different order of tracers if using original 6-tracer bling code (USE_BLING_v0). Call either bling_bio.F or bling_bio_nitrogen.F, depending on which version of the model we’re using.  Adding option for SiBLING (silica tracer)
bling_fields_load: Wind, fice, atmosp, atm_pco2 are read either from file or EXF
gchem_forcing_sep.F: updated for compatibility
ordering of the S/R arguments have been brought into consitency with the rest of the MITgcm. Now "myTime, myIter, myThid"


Document how to configure what commit types are included in the changelog

This is a follow up on #459 which has been closed.
Yet, I still don't know how to configure release-please in order to include or exclude some commit types from the changelog generation.

I wanted to add a comment here based on implementing release-please in a few different repositories recently that may help with closing out this issue... or at the very least help someone finding this issue in the future!
It seems the configuration of changelogSections, at the moment, is heavily dependent on using the GitHub Action implementation. Using the action, you can simply configure changelog-types, which will map your configuration to the changelogSections option parsed by the factory.
When using release-please (via CLI) on its own, there doesn't appear to be a way to configure this value – this seems to be blocked by the usage of yargs.strict (and the missing option for changelogSections), in addition to the argument likely being too large for most default terminal sessions (see [1] below).
It is possible configure the value when using release-please programmatically:

Example
const ReleasePlease = require('release-please');
const chalk = require('chalk');

// @see https://github.com/googleapis/release-please/blob/master/src/bin/release-please.ts#L182
function handleError(err) {
  let status = '';
  const command = argv._.length === 0 ? '' : argv._[0];
  if (err.status) {
    status = '' + err.status;
  }
  console.error(chalk.red(`command ${command} failed${status ? ` with status ${status}` : ''}`));
  if (argv.debug) {
    console.error('---------');
    console.error(err.stack);
  }
  process.exitCode = 1;
}
const rp = ReleasePlease.ReleasePRFactory.build('node', {
  repoUrl: 'my-org/my-repository,
  packageName: 'my-package',
  defaultBranch: 'trunk',
  token: process.env.SENSATIVE_VALUE,
  changelogSections: [
    {type: 'feat', section: 'Features'},
    {type: 'fix', section: 'Bug Fixes'},
    {type: 'perf', section: 'Performance Improvements'},
    {type: 'deps', section: 'Dependencies'},
    {type: 'revert', section: 'Reverts'},
    {type: 'docs', section: 'Documentation'},
    {type: 'style', section: 'Styles'},
    {type: 'chore', section: 'Miscellaneous Chores'},
    {type: 'refactor', section: 'Code Refactoring'},
    {type: 'test', section: 'Tests'},
    {type: 'build', section: 'Build System'},
    {type: 'ci', section: 'Continuous Integration'}
  ]
});
rp.run().catch(handleError);

I'm currently using the above to trigger release-please with custom changelog sections using Google Cloud Build.

[1] In general, it seems like the release-please CLI might benefit from using yargs.pkgConf to support this feature from both the action and the CLI utility... but I'm not sure that is in line with the roadmap here.
I'd be happy to contribute toward documentation or potentially an update to the CLI with a little better understanding of planned changes. Regardless, I hope someone finds this information useful, and if I've misinterpreted any of the implementations do let me know and I'll update (or remove) this comment!

@jbottigliero would happily take your contributions 😄
It's not intentional that the CLI and the action have drifted apart, it was just that I wanted to get the feature shipped for someone who needed it in the action... you're probably right that we would do better to follow the pkgConf spec.

Greetings, we're closing this due to inactivity. Please let us know if the issue needs to be reopened.

Now that this has been refactored, I know even less about where the default changelog sections are (nor how to override them) https://github.com/googleapis/release-please/pull/1104#pullrequestreview-820573534


regarding new dynamic programming algorithms

I would like to add the dynamic programming approaches into the project algorithms like The Levenshtein distance (Edit distance) problem.
Matrix Chain Multiplication.
so please i would like to request the admin to assign me this issue

Please check the issue history once before raising an issue


ff not defined if _STRICT_1CH = 0

in file _txRx.ino this line was commented.
//const float ff		= root["txpk"]["freq"];			// eg 869.525
This throw an not defined error if _STRICT_1CH = 0

Please verify that this issue has been resolved in the last patch update of today.

I will close this issue. I think it has been solved by the latest release. Thanks!


python37Packages.pex: 1.5.3 -> 1.6.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.7-pex/versions.
meta.description for python37Packages.pex is: '"A library and tool for generating .pex (Python EXecutable) files"'.

Checks done (click to expand)

built on NixOS
/nix/store/qw8v254q58rhmq8fnn59gjq75nypl37x-python3.7-pex-1.6.0/bin/.pex-wrapped passed the binary check.
/nix/store/qw8v254q58rhmq8fnn59gjq75nypl37x-python3.7-pex-1.6.0/bin/pex passed the binary check.
2 of 2 passed binary check by having a zero exit code.
2 of 2 passed binary check by having the new version present in output.
found 1.6.0 with grep in /nix/store/qw8v254q58rhmq8fnn59gjq75nypl37x-python3.7-pex-1.6.0
directory tree listing: https://gist.github.com/cba5fea2f52a98bdbea4d6072f55c04c
du listing: https://gist.github.com/bde29c54ee75b3849cbebb0456cf6430

Rebuild report (if merged into master) (click to expand)

20 total rebuild path(s)
5 package rebuild(s)
5 x86_64-linux rebuild(s)
5 i686-linux rebuild(s)
5 x86_64-darwin rebuild(s)
5 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
pants
python27Packages.pants
python27Packages.pex
python37Packages.pants
python37Packages.pex

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qw8v254q58rhmq8fnn59gjq75nypl37x-python3.7-pex-1.6.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python37Packages.pex https://github.com/r-ryantm/nixpkgs/archive/a5d62c958d740ea26e67d51586fad0f482582e32.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qw8v254q58rhmq8fnn59gjq75nypl37x-python3.7-pex-1.6.0
ls -la /nix/store/qw8v254q58rhmq8fnn59gjq75nypl37x-python3.7-pex-1.6.0/bin

cc @copumpkin for testing.

@GrahamcOfBorg build python27Packages.pants python37Packages.pants

It looks like pants needs an update also, but it looks like they changed a lot of things.
It currently doesn't build because of a version pin that didn't get bumped when the dep in nixpkgs got bumped.
I couldn't get either pex or pants to execute so this looks broken.


Wrong content selected for playback - same title

If you haven't already, be sure to check the How-to Report an Issue page before reporting an issue: https://docs.synclounge.tv/how-tos/report-an-issue/
Please complete the following when reporting a bug/an issue.
Web App
You can remove this section if your issue isn't related to the Web App
System Details

Operating System: Windows 10, Ubuntu 18.04
Web Browser name and version: Firefox 76.0.1, Chromium 81.0.4044.122

Plex Media Server details

Version: <IP_ADDRESS>64
Operating System (Windows/Linux/Mac): Docker on Ubuntu 18.04 (linuxserver.io latest image)
Running in Docker? (Yes/No): Yes
Location (Local/Remote): Both
Remote Access Enabled (Yes/No)?: Yes

Plex Client details

Name: Plex Web Client
Version: 4.30.2
Network connection (Wired/Wifi): Both
Platform: Windows 10, Ubuntu 18.04

Self-Hosting
You can remove this section if your issue isn't related to Self-Hosting
System Details

Operating System: Ubuntu 18.04

SyncLounge Details

Running in Docker? (Yes/No): Yes
Using a Reverse Proxy (Yes/No): Yes
If Yes, which Reverse Proxy (Nginx, Apache, Caddy, etc.): Traefik 2

Issue information
To Reproduce
Steps to reproduce the behavior:
(Untested)

Create a movie and a tv Plex library, each with a piece of content with identical titles
Attempt to play the tv episode, observe that the movie plays instead
Click the X to close the movie, note that the app switches to playing the tv episode as intended

Describe your bug/issue
Please describe the bug/your issue below. If applicable, add screenshots to help explain your problem.
I can do more testing if unable to reproduce, but I'm guessing that the bug is that the content is selcted by name/title, and not some other unique identifier within Plex, causing the movie to play instead of the tv episode (perhaps because "Movies" comes before "TV Shows" alphabetically?).
Curiously, this affected only 3 of 5 users in our party. All were using Firefox, one of those unaffected was streaming from a different, local server. I feel like there must be a factor I'm missing here that affected the 3, but I'm not sure what it could be.

Currently, the media selection is affected by the response times of servers which is why you are seeing differences in users. I have been working on a better media selection algorithm that will choose the same content if the user has access to the same server / content as the host.

#220 has guests play the same media (ratingKey) from the same server as the host if the guest can and then falls back to the media matching algorithm otherwise. The media matching algorithm is also improved in this PR.


Ensure csrf cookie is readable by javascript

javascript cannot read a cookie loaded by 'HttpOnly', since we are only
allowing the cookie to be set after successful https login, we should be
fine to allow the csrf cookie be created without the 'httponly' setting.

Please have one of your security engineers review this and consider alternatives... I'm highly skeptical when I see stuff like this being done, especially as their are other ways.

The CSRF cookie needs to be read by JavaScript so we can send it back to the server as part of the CSRF protection framework. This is similar to the double submit and encrypted token pattern (https://www.owasp.org/index.php/Cross-Site_Request_Forgery_(CSRF)_Prevention_Cheat_Sheet). If malicious JavaScript code is reading this cookie, then we would have a XSS issue which would invalidate any CSRF mechanism we have in place. Please note, our session cookie and other sensitive cookies are still protected with both secure and httponly flags.


fix: better description and explanation of how to use the dual themes of Shiki

Description (required)
When trying to implement the dual themes of Shiki, it's necessary to use the CSS snippets that their docs provided. But because of how Astro implements the Shiki under the hood, it's necessary to use the class .astro-code instead of .shiki that their docs stated.

I'm proposing a new section in the Syntax highlighting of Astro docs to make clear to new users how to implement the feature of dual theming. I think that dark/light mode is a key feature and having this new section would make everything clear with Shiki.
The correct CSS to make the CSS dual theme work should be
@media (prefers-color-scheme: dark) {
  .astro-code,
  .astro-code span {
    color: var(--shiki-dark) !important;
    background-color: var(--shiki-dark-bg) !important;
    /* Optional, if you also want font styles */
    font-style: var(--shiki-dark-font-style) !important;
    font-weight: var(--shiki-dark-font-weight) !important;
    text-decoration: var(--shiki-dark-text-decoration) !important;
  }
}

html.dark .astro-code,
html.dark .astro-code span {
  color: var(--shiki-dark) !important;
  background-color: var(--shiki-dark-bg) !important;
  /* Optional, if you also want font styles */
  font-style: var(--shiki-dark-font-style) !important;
  font-weight: var(--shiki-dark-font-weight) !important;
  text-decoration: var(--shiki-dark-text-decoration) !important;
}

Related issues & labels (optional)

Closes: 8163 | 8152
Suggested label: CSS Configuration of Shiki with dual themes

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/markdown-content.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Hi @gxlpes ! Thank you for picking up on some outstanding issues in the docs repo!  💪
We actually already have an open PR very close to merging to address this very topic: #7218 You can follow its progress, see my comment to the existing proposal, and comment there!
Since that PR is already open and being worked on for this very thing, I will close this PR. But I encourage you to participate in the other PR! I have a proposed suggestion for the author, and have been waiting to hear back. Would be happy to have your thoughts there.


docs(integration): add k8sgpt

@john-cloudnatix , this is just a stub. please feel free to add more. thanks!


Add an image for the Implementation and CLI wormhole tool

This PR is about (re-)creating a Docker image for the Magic Wormhole implementation and CLI tool only (w/o mailbox and transit) and publish it in Docker Hub to replace the old unmaintained version we have there.
The first use case for this PR is to be found in this issue: LeastAuthority/mw-projects#68
For which we are considering to define and publish a GitHub actions using the Magic Wormhole to exchange secrets with GitHub runner to mitigate the risk of having those secrets stored within the GitHub repository.
Using an existing public image for this action could speedup the step which today can takes a few minutes to build the magic-wormhole.
There was such an image before, but it has been removed from upstream once the mailbox and transit had there own repositories.
Least Authority also had a similar image, but I could not find from where it was provided. This PR should update this later one only.

Code Review Checklist (to be filled out by reviewer)

[x] Description accurately reflects what changes are being made.
[x] Description explains why the changes are being made (or references an issue containing one).
[x] The PR appropriately sized.
[x] New code has enough tests.
[x] New code has enough documentation to answer "how do I use it?" and "what does it do?".
[x] Existing documentation is up-to-date, if impacted.

@wuan, this did worked as expected: the new image has been published on your name.
However, I did noticed the (compressed) size of it went from 292 to 525 MB!
Though the final size of this image is similar to the other components (relay and transit), 1,5 GB seems to invite us to optimize...

There's probably a smaller base image to start with?
Also, for the client I wouldn't use PyPy: you pay a startup-time penalty for using PyPy and it's unlikely to reap any JIT rewards in the (relatively) short time that it runs. (Contrast with a server that'll be running the same code for hours or days or more).


Fix TestDiff failure with docker

TestDiff started failing with target docker after moving cmd/container to a subpackage.
The actual reason for the failure is unclear to me, but the bottom-line is that it does not work with Parallel, which suggests it is interacting with another parallelized test.
As for why this is only showing up (more) now, the answer is simply the reduction in size of the parallelized test pool (since we only parallelize per sub-package now), which makes it much more likely that the interacting test is actually running in parallel with TestDiff.
Anyhow, this will unblock #3383 and can / should be merged now.

Thanks, you might be hitting some bug of Docker

Very likely, yes.


WORLD CORRUPTION AND BLUE SCREENING

Okay this mod is just the worst. every world i played on with this mod it ended in desaster. the wirst world had corrupted chunks, and now on the next world after loading it back up the world unloaded, i fell into the void and my pc had a blue screen. and now the world is corrupt and unplayable. incompetent mod devs istg

such a mod shouldnt be allowed for download


Make port server python3 compatible

Currently port_server doesn't work when run with python3. This should fix it. Tested manually with python3 on windows and linux and it seems to be working fine.

Based on the errors, looks like this actually breaks something - I'll try to fix.

Maybe try "foo".encode("utf-8")?

Known failures:
https://github.com/grpc/grpc/issues/23466


Store raw responseBody and decompress when needed

Description
Based on idea and discussion here, this PR make it work
Motivation and Context
For every single requests which is compressed, JMeter always decompresses the data and stores the uncompressed data in responseData. This causes high memory usage per thread and adds cpu time for decompression when the responseData is never used in an assertion, post processor or listener.
How Has This Been Tested?
First of all, the tests from the build succeeded (after they failed for deflate which needed some extra care). Then I've ran some of my scripts against various websites and all worked.
With debugging I've checked that the responseData is actually only decompressed when accessed and it worked. I'd like to do a benchmark later under load and compare impact on memory and cpu
Screenshots (if appropriate):
Types of changes
Removed decompression from HC4 and Java HTTP implementation and doing the decompression in getResponseData(). This way, it is only decompressed when the data is accessed.
Checklist:

[ ] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.

So I've done some benchmarking with this code, comparing it with default 5.6.3.

Results are surprisingly good! Somehow, the benefit is relatively more on cpu rather than memory.
I've ran a benchmark, running identical JMX script against the same environment at the same time. Both doing 100 threads, 300 requests per second. It was a recording of 5 transactions of a generic website, with a running time of about 80s per threadgroup.
Ran it on 2 separate aws ec2 instances, 2cpu/2gb mem, max heap at 1500m
The one without updated code, did about 15% cpu average during the run, while the updated code where we only decompress when the body is actually used (assertion/postprocessor), did about 10% on average during the run. Also the total memory usage of the VM was about 10% less, meaning the JVM didn't spiked as high.
I think it has less impact on memory, because the responseData and previousResult gets overwritten every next sampler, so it only holds the latest responseData. But not having to decompressing all responses (for nothing) and not needing to do much heavier garbage collections, it saves more on cpu usage.


Rename folder/album improvement

Is your feature request related to a problem? Please describe.
Currently, the rename folder/album feature seems to create a new folder/album and then move the files into it, or at least that's what I believe happens, as it took about an hour to rename an album with 30 videos stored on the SD card.
Describe the solution you'd like
Would it be possible to simply use the "mv" command to rename the folder? In a Linux system, moving (mv) or renaming a folder merely updates the folder's name (metadata), it doesn’t actually move the files. The underlying inode remain the same, so the operation is instantaneous as no files are physically relocated.
Describe alternatives you've considered
Currently, I use Termux to execute a mv album1 album1_n command, which completes the operation instantly rather than taking hours.

Oh, didn't know that, thanks for the clarification!


[Talk to]: icons of models are 'blinking' when switch from one model to another

EPAM AI DIAL chat version
0.18.0
How to reproduce
Steps:

Click "+ New conversation" button
switch one by one models in Talk to section

Actual result
icons of all models in 'Talk to' are reloaded
comparing to version 0.15.2 in 0.18.0 'blinking' became more obvious
https://github.com/user-attachments/assets/3fcc2e55-cf1a-4cbb-a959-72385caf75d8
Expected result
need reduce "blinking" of the icons when switching between models
Confidential information

[X] I confirm that do not share any confidential information

need to test all possible types of files which were included for icon

Selection of avif type failed with error

related issues #2412, #2411

issue verified on staging


Add SHA1 and SHA256 algorithms for EC2 fingerprint calculation

Resolves localstack issue
@viren-nadkarni is recommended to make changes in moto in this comment

hey @bblommers , could you restart test 3.9 CI stage? Looks like if failed because of connection issue


Run tests with Twig 3.15

It seems that this test fails on Twig 3.15:
Failed asserting that two strings are identical.
Expected :'File is invalid: Unexpected token "end of template" of value "".'
Actual   :'File is invalid: Unexpected end of template.'
<Click to see difference>

/Volumes/CS/opensource/Twig-CS-Fixer/tests/Runner/LinterTest.php:372

Will be fixed with https://github.com/VincentLanglet/Twig-CS-Fixer/pull/331.
Did you wanted to implement something ? ^^

Fixed


Add Drizzle

drizzle-orm

That sounds like a good solution, giving the user some amount of choice, but not too much. We might need to think if and how we can test this adder.
I want to avoid getting in a state we had with svelte-add@v1 where changing anything could have resulted in breaking everything without even noticing. I'm not so worried about the cli itself, as these are quite easily testable by hand, but nobody wants to test 7 adders multiplied by x options by hand. We had that in the past, and it was awful.
We should probably be able to test sqlite quite easily, but we might need to check if the same applies for the other providers.

I think the providers all have a common portion of the setup, which is const db = drizzle(client) and anything after that. However, creating the client varies for each database. Maybe we can have a createClient method which gets filled in with different code based on the client. We could even simplify it further and have only one or two clients instead of three
I think the other part of this is how do we help people setup and test their database locally. I've never used sqllite, so am not sure what's involved there. When using something like postgres I would usually run it on Docker locally so that I don't have to install the database on my machine. It could help to provide a docker-compose.yml in this case so that folks don't need to install the database. This is from a project I setup a couple of years ago, so might be slightly outdated, but could be a helpful starting point:
version: "3.0"
services:
  postgres:
    image: postgres:12.2
    ports:
      - "5432:5432"
    environment:
      - POSTGRES_DB=${POSTGRES_DB}
      - POSTGRES_USER=${POSTGRES_USER}
      - POSTGRES_PASSWORD=${POSTGRES_PASSWORD}
  adminer:
    image: adminer
    restart: always
    ports:
      - "4000:8080"


Modify the ION-ITEM css

Feature Request
Please, change ion-item css above:
:host(.item-label-floating) .item-native, :host(.item-label-stacked) .item-native {
align-items: start;
}
to:
:host(.item-label-floating) .item-native, :host(.item-label-stacked) .item-native {
align-items: inherit;
}
My ion-item have always been centered now I can no longer align correctly
Ionic version:
[x] 4.x

Hi there,
This change was made to fix https://github.com/ionic-team/ionic/issues/16375. Can you provide a code example of how you are using ion-item?
Please be sure to include what the expected behavior is and what the actual behavior is.

In #16375 comments
"
I think the solution here (in Ionic) is to change the alignment for stacked/floating items like this:
.item-label-stacked,
.item-label-floating {
align-items: start;
}
and/or move the css for this rule to the parent item and inherit it in the inner one so the user has more control over it.
"
This is a solution.
But configure align-items in INNER, we are not access

Hi there,
I understand what the proposed solution is. It would be very helpful if you could provide an example of how you are using ion-item so we can better understand your use case.

I'm using it in many ways. But the default usage is in Forms or List.
An example of my code:
  <ion-item *ngIf="ip.editing || ip.item.dateOfBirth" @fade>

      <ion-label position=stacked i18n>Data de nascimento (aniversário)</ion-label>
      <ion-input readonly (click)="selectBirth($event)" [ngModel]="ip.item.dateOfBirth?(ip.item.dateOfBirth|date: 'dd/MM'):'Selecione'" name="dateOfBirth"></ion-input>
    </ion-item>

    <ion-item *ngIf="ip.item.type=='student' && (isParent || isSchool)" @fade>

      <ion-label i18n>Sexo</ion-label>
      <ion-button slot=end (click)="changeSex()" [attr.style]="'--color: '+ip.item.female?'lightpink':'lightblue'" fill="outline" type="button">
        {{ip.item.female?'Feminino':'Masculino'}}
      </ion-button>
    </ion-item>

Thanks for the follow up! I discussed with another team member, and we agree that we should have align-items inherit.

Thanks

It should be
:host(.item-label-stacked),
:host(.item-label-floating) {
align-items: start;
}

Agreed this is a much better solution, so it can still be custumized!

@brandyscarney what chance do we see this change in the next version of Ionic?

PR created here: https://github.com/ionic-team/ionic/pull/19278
It just needs review and verification that it didn't have effects on other elements.

+1 on this one. I need to set flex alignment on a ion-item button, but it's impossible


Connection duration

Hello and sorry if I bother you. I would like to switch from Firebase to Supabase. Libraries that run Firebase have an issue with the token expiring after 60 minutes. This blocks the procedure and forces the processor to restart. I wanted to know if even in this case there is a problem with the duration of the connection, beyond which it is necessary to restart. I have to develop a procedure linked to an ESP32CAM which must always work and must acquire data from the database for operational choices. Can all this be done with this library?
Thanks for the kind cooperation.

The JWT token (if you use RLS) still be expixing in 60 minutes (you can change the setting in Supabase Web UI). But this library handle this stuff automatically by requesting new token every 60 minutes, or whatever you set in Supabase Web UI.


Unable to define ASP.Net Core MVC controllers in .csproj class libraries

I have an issue with a Web Api project created from the "Asp.Net Core Web Application (.Net Framework)" template which references a .Net 4.6.1 class library (.csproj) that uses the Microsoft.AspNetCore.MVC nuget package. Controllers defined in the class library are not picked up by the web project.
I have found some issues similar to this, but they have been fixed or closed and seem to be related to xproj project references only.
I'm not sure if this scenario should be supported right now, however it is something I would like to do to help me crack on with serious development in ASP.Net core.  The tooling around xproj projects is not yet mature enough to easily fit into our development environment (source control, code analysis and unit testing all have problems). Whilst this is the case, the more code I can keep in .csproj the better!
To reproduce in VS2015 (update 3):

Create a new project from the "Asp.Net Core Web Application (.Net Framework)" template
Select "Web API" from the "Asp.Net Core Templates" Dialog
Confirm this will run by hitting F5 - a browser will load and navigate to api/values
In the same solution create a new .Net 4.6.1 class library (csproj)
apply the Microsoft.AspNetCore.MVC nuget package to the class library (v1.0.0)
Copy the ValuesController.cs from your web project into your class library.
Delete ValuesController.cs from your web project.
Add a reference to the new class library from your web project
Hit F5 to Debug the web project again
You get 404 Not found when navigating to api/values

Workaround for this as provided by @pranavkm in the issue referenced above:
In your startup:
services
.AddMvc()
.AddApplicationPart(typeof(ValuesControllerFromClassLib).Assembly)

It looks like the Dependencies node for Microsoft.Extensions.DependencyModel.Library entries for csproj based references is empty (and consequently we don't look for controllers in them). @pakrym would you know if there's a work item addressing this in the CLI?

@pranavkm I don't think so, we might need to file a new one.

Filed https://github.com/dotnet/cli/issues/3804 to track this.

@pranavkm @pakrym so this is an external issue?

Appears so. We might want to verify that it works correctly once the CLI issue is fixed.

https://github.com/dotnet/cli/issues/3804 isn't getting fixed so there's not anything more to do here.

Maybe this is a feature? That all controllers from referenced libraries do not activate by default.

Hi, it looks like you are posting on a closed issue/PR/commit!
We're very likely to lose track of your bug/feedback/question unless you:

Open a new issue
Explain very clearly what you need help with
If you think you have found a bug, include detailed repro steps so that we can investigate the problem

Thanks!

This is an old issue from 2016. Defining a controller in another class library that you reference in your app should just work.

Yes it works, tried that later. Thanks.


bump version to 0.4.0

Changes

Regenerate v2beta1 endpoint to reflect a routing change.
Updated docstrings wording to DialogFlow instead of API.AI

Codecov Report

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

@@          Coverage Diff          @@
##           master    #73   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           5      5
=====================================
  Hits            5      5

Continue to review full report at Codecov.

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


Null points on Chart Line

I would like to know if there's a way to DRAW the null point namely, that the line doesn't break on the null value and continue drawing until the next not null value.

One way you can do is by not passing NULL values at all.
Use the below code to send the x & y values explicitly (without null values).
					data: [{
						x: newDateString(0),
						y: randomScalingFactor()
					}, {
						x: newDateString(5),
						y: randomScalingFactor()
					}, {
						x: newDateString(7),
						y: randomScalingFactor()
					}, {
						x: newDateString(15),
						y: randomScalingFactor()
					}],

May be this is not the perfect solution, but alternate way to achieve it.

Thanks for the answer. It didn't work, the chart doesn't show nothing but the lines of the Y values I mean the "background" of the chart. Do I need to specify a certain property in the config array? Or in the options?

Sorry, my mistake, I was putting your solution in a bad way, but the original issue persist, how can I put a NULL value, I cannot put a 0 because the line goes down to the zero, if I left the Y value empty, the chart simply doesn't appear, and if I put a NULL, the linea breaks, so what is the value that I have to put if I don't have a Y value for that point in X.

Holy God!!! Issue solved! spanGaps was the light!!!

closing as resolved. spanGaps is the way to go


replace texts in original pdf

I need one question, for example: have one pdf for "model", i need replace some texts in original pdf.
Original: "I'm 10 years old".
After change: "I'm 9 years old".
In Vue It would be like one variable like {{y}}, how would it look to do that in the pdf using Vue?

Hello @fabioselau077!
You can do this pretty easily if your original PDF has form fields. See the form filling example.
If your original PDF does not use form fields, then you won't be able to achieve this with pdf-lib right now. See https://github.com/Hopding/pdf-lib/issues/177 for more details.


Add Icon Component

Summary
Part one of adding the background around the icon inside the Bento. This adds the Icon component.
List of notable changes:

Added Icon Component to behave similar to Card.Icon component
To come: Update Bento to accept new Icon component.

What should reviewers focus on?

Icon should match Card.Icon visually

Steps to test:

View Icon component in storybook
Compare to Card.Icon
Examine API to verify it matches other components.

Supporting resources (related issues, external links, etc):

Towards https://github.com/primer/brand/issues/773
Related https://github.com/github/marketing-platform-services/issues/3553

Contributor checklist:

[ ] All new and existing CI checks pass
[ ] Tests prove that the feature works and covers both happy and unhappy paths
[ ] Any drop in coverage, breaking changes or regressions have been documented above
[ ] New visual snapshots have been generated / updated for any UI changes
[ ] All developer debugging and non-functional logging has been removed
[ ] Related issues have been referenced in the PR description

Reviewer checklist:

[ ] Check that pull request and proposed changes adhere to our contribution guidelines and code of conduct
[ ] Check that tests prove the feature works and covers both happy and unhappy paths
[ ] Check that there aren't other open Pull Requests for the same update/change

Screenshots:

Please try to provide before and after screenshots or videos

 Before  After

One other thing I just spotted. I'm not entirely sure why, but --brand-Icon-color-default doesn't appear to be defined. If you look at this story, then you'll see the icon is defaulting to black and so blends in with the page's black background.

The other thing we'll need, alongside this implementation, is documentation.
That might be dependent on us updating the Figma as we'll likely need some images from Figma to use in the documentation.
Just flagging so we don't forget @danielguillan

Thank you so much @joshfarrant for the help with prettier and with checking that accessibility! 👏🏼 ♿ ! I left a response to the span comments and I think I covered the other suggestions you made.
Thanks, again!

Awesome, thanks for addressing those comments @rfearing 🙌
Regarding the span, I had a play around myself and I'm keen to get your thoughts.
I branched off your branch and created a PR against your branch to illustrate the changes
https://github.com/primer/brand/pull/787
I think this is the same approach you mentioned (using padding to achieve the background). I can't see any issues with this approach immediately, but keen to hear what you think.

@joshfarrant That looked good! Merged it into mine. 😄


Implement custom RPC Provider with Fallback

The Problem

The Solution

Update June 7
Work on this is deprioritized temporarily (will likely resume next week).
Currently blocked by an issue with the ethprovider image. Specifically, when running integration tests the contracts fail to deploy because the image does not recognize the symlinked types package.
When creating the ethprovider image, there are dependencies installed that are listed here which the image has access too. The symlinked packages are generally only accessible by the vector_builder image, which is the image used by the server-node package in the duet and trio stacks.
To move forward, we have to give the images used in the chains stack access to the symlinked packages. This means replacing the ethprovider image with the builder image within the start-chains script.


[QA] - Retire (de-register) an existing stake pool

retire (de-register) an existing stake pool (you can create it first)
check that the stake pool was correctly retired

https://github.com/input-output-hk/cardano-node-tests/blob/167bae76cc17faea23c2e2aacba6f5cdba83750d/cardano_node_tests/tests/test_staking.py#L568


setup.py: explicitly specify cython's language_level

this series restructures get_extensions(), and also silences the warning like:
/tmp/build-env-p23crgfa/lib64/python3.11/site-packages/Cython/Compiler/Main.py:384: FutureWarning: Cython directive 'language_level' not set, using '3str' for now (Py3). This has changed from earlier releases! File: /home/kefu/dev/scylladb/tools/cqlsh/cqlshlib/copyutil.py
  tree = Parsing.p_module(s, pxd, full_module_name)

@fruch hi Israel, could you help review this change as well?


Moved all prox and solver source code

Second iteration for the reorganization and flattening of tick, see #124

Moved all prox and solver python modules and C++ source code
Updated config files for compilation with mkn
Updated and reorganized documentation

Without these files I get
ImportError: No module named 'tick.prox.build.prox' error as well.
I think you have these files on your computer but you need to tell git to track them manually as they are ignored. Either by typing git add -f tick/prox/build/__init__.py either by tweaking .gitignore file, this would be cleaner I don't know how to set a general rule for all __init__.py under build folders.

Is this superseded  by #143 ?

This is superseded by #145


Added some referrences to the android.md file

Signed-off-by: infiniteoverflow<EMAIL_ADDRESS>Describe your changes
I have added some more referrences to some Free android courses in Udacity to the android.md file.

Please refer to the original repo here.
This repo is a fork of the original as I meant to contribute to the latter.