Set-RegString -data doesn't allow for an empty or null value.

Tried to use Set-RegString to remove the data from a value but the data parameter is set to only accept string data.
Be nice if it would allow the passing of a null or empty ('') value so you could remove the data but leave the key value there.

Thanks for the submission! I'm not actively maintaining this module at the
moment but if you want to submit a pull request I'll review it.
On Thu, Jan 18, 2018 at 7:34 AM NH2PSCode<EMAIL_ADDRESS>wrote:

Tried to use Set-RegString to remove the data from a value but the data
parameter is set to only accept string data.
Be nice if it would allow the passing of a null or empty ('') value so you
could remove the data but leave the key value there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JustinGrote/PoshRegistry/issues/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/AOjVUtAIIcuyZRjqw9XcM4myT7mubftMks5tL2SZgaJpZM4RjG-u
.

Kinda embarrassing to admit, that while I've written literally hundreds of PS scripts and tools for work, I've not learned how GIT works. I've taken a few cracks at it but never had a use-case that forced me to pick it up. When I get some free time I'll have to take another shot at it.
Thanks for getting back with me.

From: Justin Grote<EMAIL_ADDRESS>Sent: Tuesday, February 6, 2018 6:50:39 PM
To: JustinGrote/PoshRegistry
Cc: NH2PSCode; Author
Subject: Re: [JustinGrote/PoshRegistry] Set-RegString -data doesn't allow for an empty or null value. (#5)
Thanks for the submission! I'm not actively maintaining this module at the
moment but if you want to submit a pull request I'll review it.
On Thu, Jan 18, 2018 at 7:34 AM NH2PSCode<EMAIL_ADDRESS>wrote:

Tried to use Set-RegString to remove the data from a value but the data
parameter is set to only accept string data.
Be nice if it would allow the passing of a null or empty ('') value so you
could remove the data but leave the key value there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JustinGrote/PoshRegistry/issues/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/AOjVUtAIIcuyZRjqw9XcM4myT7mubftMks5tL2SZgaJpZM4RjG-u
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363617590, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARN4VPurYq7IcW5jcHdDm-1PbtWAChPfks5tSPNfgaJpZM4RjG-u.

If you modify the file directly you can just send me the modified file and
I can commit it, or use this as an opportunity to learn. You can do a pull
request on the file directly from the web interface, no installing or
learning GIT required too:
https://github.com/WGBH/pbucore/wiki/Contributing-to-the-project-through-Github-web-interface
On Wed, Feb 7, 2018 at 6:28 AM NH2PSCode<EMAIL_ADDRESS>wrote:

Kinda embarrassing to admit, that while I've written literally hundreds of
PS scripts and tools for work, I've not learned how GIT works. I've taken a
few cracks at it but never had a use-case that forced me to pick it up.
When I get some free time I'll have to take another shot at it.
Thanks for getting back with me.

From: Justin Grote<EMAIL_ADDRESS>Sent: Tuesday, February 6, 2018 6:50:39 PM
To: JustinGrote/PoshRegistry
Cc: NH2PSCode; Author
Subject: Re: [JustinGrote/PoshRegistry] Set-RegString -data doesn't allow
for an empty or null value. (#5)
Thanks for the submission! I'm not actively maintaining this module at the
moment but if you want to submit a pull request I'll review it.
On Thu, Jan 18, 2018 at 7:34 AM NH2PSCode<EMAIL_ADDRESS>wrote:

Tried to use Set-RegString to remove the data from a value but the data
parameter is set to only accept string data.
Be nice if it would allow the passing of a null or empty ('') value so
you
could remove the data but leave the key value there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JustinGrote/PoshRegistry/issues/5, or mute the
thread
<
https://github.com/notifications/unsubscribe-auth/AOjVUtAIIcuyZRjqw9XcM4myT7mubftMks5tL2SZgaJpZM4RjG-u
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363617590>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/ARN4VPurYq7IcW5jcHdDm-1PbtWAChPfks5tSPNfgaJpZM4RjG-u

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363785416,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOjVUu4G2Q4H-Z2lchYE8hPgVWgj5dABks5tSbL9gaJpZM4RjG-u
.

Thanks for the information and the offer. I'll use this opportunity to figure this stuff out, finally.

From: Justin Grote<EMAIL_ADDRESS>Sent: Wednesday, February 7, 2018 9:20:46 AM
To: JustinGrote/PoshRegistry
Cc: NH2PSCode; Author
Subject: Re: [JustinGrote/PoshRegistry] Set-RegString -data doesn't allow for an empty or null value. (#5)
If you modify the file directly you can just send me the modified file and
I can commit it, or use this as an opportunity to learn. You can do a pull
request on the file directly from the web interface, no installing or
learning GIT required too:
https://github.com/WGBH/pbucore/wiki/Contributing-to-the-project-through-Github-web-interface
On Wed, Feb 7, 2018 at 6:28 AM NH2PSCode<EMAIL_ADDRESS>wrote:

Kinda embarrassing to admit, that while I've written literally hundreds of
PS scripts and tools for work, I've not learned how GIT works. I've taken a
few cracks at it but never had a use-case that forced me to pick it up.
When I get some free time I'll have to take another shot at it.
Thanks for getting back with me.

From: Justin Grote<EMAIL_ADDRESS>Sent: Tuesday, February 6, 2018 6:50:39 PM
To: JustinGrote/PoshRegistry
Cc: NH2PSCode; Author
Subject: Re: [JustinGrote/PoshRegistry] Set-RegString -data doesn't allow
for an empty or null value. (#5)
Thanks for the submission! I'm not actively maintaining this module at the
moment but if you want to submit a pull request I'll review it.
On Thu, Jan 18, 2018 at 7:34 AM NH2PSCode<EMAIL_ADDRESS>wrote:

Tried to use Set-RegString to remove the data from a value but the data
parameter is set to only accept string data.
Be nice if it would allow the passing of a null or empty ('') value so
you
could remove the data but leave the key value there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JustinGrote/PoshRegistry/issues/5, or mute the
thread
<
https://github.com/notifications/unsubscribe-auth/AOjVUtAIIcuyZRjqw9XcM4myT7mubftMks5tL2SZgaJpZM4RjG-u
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363617590>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/ARN4VPurYq7IcW5jcHdDm-1PbtWAChPfks5tSPNfgaJpZM4RjG-u

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363785416,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOjVUu4G2Q4H-Z2lchYE8hPgVWgj5dABks5tSbL9gaJpZM4RjG-u
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363802154, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARN4VJujYJBZcUTTGHi-KwA0bIGisJL8ks5tSb9NgaJpZM4RjG-u.

I can't seem to create a pull request on the website. The button is grayed out. When I try to publish the branch I created from the GitHub desktop program, it gives me an 'Authentication failed. You may not have permission to access the repository or the repository may have been archived. Open options and verify that you're signed in with an account that has permission to access this repository.'
I've attached the updated cmdlet.

From: Justin Grote<EMAIL_ADDRESS>Sent: Wednesday, February 7, 2018 9:20:46 AM
To: JustinGrote/PoshRegistry
Cc: NH2PSCode; Author
Subject: Re: [JustinGrote/PoshRegistry] Set-RegString -data doesn't allow for an empty or null value. (#5)
If you modify the file directly you can just send me the modified file and
I can commit it, or use this as an opportunity to learn. You can do a pull
request on the file directly from the web interface, no installing or
learning GIT required too:
https://github.com/WGBH/pbucore/wiki/Contributing-to-the-project-through-Github-web-interface
On Wed, Feb 7, 2018 at 6:28 AM NH2PSCode<EMAIL_ADDRESS>wrote:

Kinda embarrassing to admit, that while I've written literally hundreds of
PS scripts and tools for work, I've not learned how GIT works. I've taken a
few cracks at it but never had a use-case that forced me to pick it up.
When I get some free time I'll have to take another shot at it.
Thanks for getting back with me.

From: Justin Grote<EMAIL_ADDRESS>Sent: Tuesday, February 6, 2018 6:50:39 PM
To: JustinGrote/PoshRegistry
Cc: NH2PSCode; Author
Subject: Re: [JustinGrote/PoshRegistry] Set-RegString -data doesn't allow
for an empty or null value. (#5)
Thanks for the submission! I'm not actively maintaining this module at the
moment but if you want to submit a pull request I'll review it.
On Thu, Jan 18, 2018 at 7:34 AM NH2PSCode<EMAIL_ADDRESS>wrote:

Tried to use Set-RegString to remove the data from a value but the data
parameter is set to only accept string data.
Be nice if it would allow the passing of a null or empty ('') value so
you
could remove the data but leave the key value there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JustinGrote/PoshRegistry/issues/5, or mute the
thread
<
https://github.com/notifications/unsubscribe-auth/AOjVUtAIIcuyZRjqw9XcM4myT7mubftMks5tL2SZgaJpZM4RjG-u
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363617590>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/ARN4VPurYq7IcW5jcHdDm-1PbtWAChPfks5tSPNfgaJpZM4RjG-u

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363785416,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOjVUu4G2Q4H-Z2lchYE8hPgVWgj5dABks5tSbL9gaJpZM4RjG-u
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/JustinGrote/PoshRegistry/issues/5#issuecomment-363802154, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARN4VJujYJBZcUTTGHi-KwA0bIGisJL8ks5tSb9NgaJpZM4RjG-u.


feat: add global appConfig to MST environment

In @emcelroy's work to port the CODAP graph into CLUE we have encountered the need to configure the behavior of the graph to accommodate the differing requirement of CLUE and CODAP. The natural place to handle such configuration is via the AppConfigModel. This works great for React components and other entities that have ready access to the AppConfigModel via the stores, but some MST models whose behavior should be configurable do not have such ready access. This PR addresses this by making the AppConfigModel available via the MST environment.

The documentEnv: IDocumentEnvironment already has the appConfig in it.

Good point -- I'm going to close this.


error in vCPU calculation documentation

See https://github.com/kata-containers/documentation/blob/master/design/cpu-constraints.md
The formula is listed as:
(quota + (period -1)) / period

This doesn't match the code, and it is not correct.  vCPU = quota / period
This is related to https://github.com/kata-containers/runtime/issues/1767

/cc @devimc

Just to clarify. Is the following understanding correct?
For containers in a POD the formula is vCPU = quota / period
For the POD itself the formula is : (quota + (period -1)) / period
/cc @gkhanna79

The calculation is as follow:
The containers in a pod are converted to milli cpus
func CalculateMilliCPUs(quota int64, period uint64) uint32 {

        // If quota is -1, it means the CPU resource request is
        // unconstrained.  In that case, we don't currently assign
        // additional CPUs.
        if quota >= 0 && period != 0 {
                return uint32((uint64(quota) * 1000) / period)
        }

        return 0
}

Example:
//  This is 1500
milliCPUs:= CalculateMilliCPUs(150000, 100000)

When a pod has mutiples containers the milliCPUs are sum:
millCPUs := 0
milliCPUs += CalculateMilliCPUs(150000, 100000)
milliCPUs += CalculateMilliCPUs(150000, 100000)
// This result in 3000 millcpus

After the milliCPUs  for a POD are calculated this is converted to CPUs with:
//CalculateVCpusFromMilliCpus converts from mCPU to CPU, taking the ceiling
// value when necessary
func CalculateVCpusFromMilliCpus(mCPU uint32) uint32 {
       return (mCPU + 999) / 1000
}

VCPUS:
(3000 + 999 ) / 1000 = 3

@jcvenegas overall, though, the documentation is wrong.
It should be  vCPU = ceiling( quota / period )
Can you send a PR to update the doc?  WDYT?


Create native Benchmarks for SDK

Background
Create a simple benchmark that we can use to estimate TPS on a sov-sdk rollup, ignoring DA costs. Basically, we want to make "realistic enough" STF which has some transactions that contend with each other for the same state (think sequential trades on Uniswap) and some transactions which operate on random state (think "simple transfers between random accounts"). Create a Benchmark running this suite of transactions, so that we can start measuring and optimizing performance

@dubbelosix is this one ready? or is there something left?

@@dubbelosix @preston-evans98  is this one still in progress?


Fix SepParser* buffer overrun bug when column count is equal to internal capacity and new row after (issue #108)

Fixes #108
Bug likely present since 0.4.0 when parsing multiple rows at a time was introduced.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.09%. Comparing base (da18795) to head (134fbc7).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #109      +/-   ##
==========================================
- Coverage   99.21%   99.09%   -0.12%
==========================================
  Files          45       45
  Lines        3296     3323      +27
  Branches      356      356
==========================================
+ Hits         3270     3293      +23
- Misses         23       27       +4
  Partials        3        3

Flag
Coverage Δ

Debug
98.40% <100.00%> (-0.17%)
:arrow_down:

Release
99.04% <100.00%> (+<0.01%)
:arrow_up:

macos-latest
98.40% <100.00%> (-0.45%)
:arrow_down:

ubuntu-latest
99.04% <100.00%> (+0.22%)
:arrow_up:

windows-latest
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ETM improvements

Thank you for your contribution!
We appreciate the time and effort you've put into this pull request.
To help us review it efficiently, please ensure you've gone through the following checklist:
Submission Checklist 📝

[x] I have updated existing examples or added new ones (if applicable).
[x] I have used cargo xtask fmt-packages command to ensure that all changed code is formatted correctly.
[x] My changes were added to the CHANGELOG.md in the proper section.
[x] I have added necessary changes to user code to the Migration Guide.
[x] My changes are in accordance to the esp-rs API guidelines

Extra:

[x] I have read the CONTRIBUTING.md guide and followed its instructions.

Pull Request Details 📖
Description

ETM types are no longer prefixed
GPIO ETM types are no longer generic
GPIO ETM now works with signals instead of pins, allowing better flexibility.

Wrong base, depends on  #2418, oops


What does this mean?

Just use the default settings and the given configurations?
For instance,
"ActorCast": "0x012c",
"ActorControl": "0x017a",
"ActorControlSelf": "0x007c",
"AddStatusEffect": "0x0204",
"RequestUseAction": "0x02b3",
"RequestUseAction2": "0x0250",
"SkillResultResponse01": "0x02d0",
"SkillResultResponse08": "0x0199",
"SkillResultResponse16": "0x02d1",
"SkillResultResponse24": "0x01a4",
"SkillResultResponse32": "0x016e"
for Korean server.

please help :(

Yes as long as you use the default settings you are fine

Yes, as long as you didn't change game installation path, you can just use the defaults.


Feature/1936 no queryclient set in devtools

closes #1936

Nice!
I have no idea if there is any extra build config that is needed to support this as well. When I went back to my original PR for hydration, I saw there was some external and globals-config stuff in the rollup config that might be relevant.
I haven't checked if that's the case, but might be worth double checking if this PR doesn't bundle the entire react-query into the devtools package. 😄

so I downloaded: https://pkg.csb.dev/tannerlinsley/react-query/commit/6a704898/react-query
and it doesn't look like the devtools package contains a copy of react-query - it is also the same size as before.
The rollup config changed significantly with v3 and there is now no globals-config anymore
do you think we can merge this? Is the eslint-disable acceptable?

That's good! This warning when building doesn't look great though:

According to this article (now pretty old) Rollup tries to add external and globals for you automatically for unresolved dependencies, but prints a warning. In this case, it seems as if it's guessing the wrong name for the global name though (reactQuery instead of ReactQuery). I think this means this PR can break in some cases (when loading React Query/devtools from a CDN?).
Since it's a one line change and needs the exact same fixes, maybe you'd like to fix the relative import in the hydration package in this PR as well? In that case, I could verify that it works, otherwise I'll open a separate PR for that. 😄

I think maybe a rollup config like this would work and be flexible to fix the relative imports that exists in other packages as well?
import babel from 'rollup-plugin-babel'
import { terser } from 'rollup-plugin-terser'
import size from 'rollup-plugin-size'
import externalDeps from 'rollup-plugin-peer-deps-external'
import resolve from 'rollup-plugin-node-resolve'
import commonJS from 'rollup-plugin-commonjs'
import visualizer from 'rollup-plugin-visualizer'
import replace from '@rollup/plugin-replace'

const inputSrcs = [
  ['src/index.ts', 'ReactQuery', 'react-query'],
  ['src/core/index.ts', 'ReactQueryCore', 'react-query-core'],
  [
    'src/devtools/index.ts',
    'ReactQueryDevtools',
    'react-query-devtools',
    {
      extraExternals: ['react-query'],
      extraGlobals: { 'react-query': 'ReactQuery' },
    },
  ],
  [
    'src/hydration/index.ts',
    'ReactQueryHydration',
    'react-query-hydration',
    {
      extraExternals: ['react-query'],
      extraGlobals: { 'react-query': 'ReactQuery' },
    },
  ],
  [
    'src/persistQueryClient-experimental/index.ts',
    'ReactQueryPersistQueryClientExperimental',
    'persistQueryClient-experimental',
  ],
  [
    'src/createLocalStoragePersistor-experimental/index.ts',
    'ReactQueryCreateLocalStoragePersistorExperimental',
    'createLocalStoragePersistor-experimental',
  ],
  [
    'src/broadcastQueryClient-experimental/index.ts',
    'ReactQueryBroadcastQueryClientExperimental',
    'broadcastQueryClient-experimental',
  ],
]

const extensions = ['.js', '.jsx', '.es6', '.es', '.mjs', '.ts', '.tsx']
const babelConfig = { extensions, runtimeHelpers: true }
const resolveConfig = { extensions }

export default inputSrcs
  .map(
    ([input, name, file, { extraExternals = [], extraGlobals = {} } = {}]) => {
      const external = ['react', 'react-dom', ...extraExternals]

      const globals = {
        react: 'React',
        'react-dom': 'ReactDOM',
        ...extraGlobals,
      }

      return [
        {
          input: input,
          output: {
            name,
            file: `dist/${file}.development.js`,
            format: 'umd',
            sourcemap: true,
            globals,
          },
          external,
          plugins: [
            resolve(resolveConfig),
            babel(babelConfig),
            commonJS(),
            externalDeps(),
          ],
        },
        {
          input: input,
          output: {
            name,
            file: `dist/${file}.production.min.js`,
            format: 'umd',
            sourcemap: true,
            globals,
          },
          external,
          plugins: [
            replace({
              'process.env.NODE_ENV': `"production"`,
              delimiters: ['', ''],
            }),
            resolve(resolveConfig),
            babel(babelConfig),
            commonJS(),
            externalDeps(),
            terser(),
            size(),
            visualizer({
              filename: 'stats-react.json',
              json: true,
            }),
          ],
        },
      ]
    }
  )
  .flat()

Since it's a one line change and needs the exact same fixes, maybe you'd like to fix the relative import in the hydration package in this PR as well? In that case, I could verify that it works, otherwise I'll open a separate PR for that. 😄

let's do a separate PR as a follow up please. from what I can see, hydration imports multiple things relatively ...

Since it's a one line change and needs the exact same fixes, maybe you'd like to fix the relative import in the hydration package in this PR as well? In that case, I could verify that it works, otherwise I'll open a separate PR for that. 😄

let's do a separate PR as a follow up please. from what I can see, hydration imports multiple things relatively ...

Of course! I'll take care of that tomorrow. 😄

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cannot open math menu on ipad air 2

It seems that we cannot open the math menu on ipad air 2 device
ios version: 13.1.2
I tried upgrading the mathjax version to the latest with no luck (i was using 2.0.0) .
Also tried the double tab and hold but it is not working also.

OK, it looks like Apple has changed the way they handle touch events in iOS 13 from how they worked in previous versions.  So the way that we detect the menu tap no longer works.  I don't see a way to access the MathJax menu in iOS 13 using current versions of MathJax.  I will look into it further, and see what I can figure out for a way forward.


FYI - AP_IF needs to be inactive for requests to work

Hello,
I just discovered that due to this issue multicast packets don't get sent unless the AP interface is disabled, i.e. network.WLAN(network.AP_IF).active(False). Just posting this in case anyone else encounters the same issue. :)
Cheers,
Matt

Thanks for providing that information Matt.
It looks like more strictly multicast packets don't get sent on the STA interface if the AP  interface is active. The packets are still getting sent, just not on the interface on which you want them :-(
It also looks like it might be possible to fix Micropython to allow support for setting the multicast interface. I will try to get some time to try that fix myself. I'll leave this issue open until the underlying problem is fixed and reference the resolution here when one exists.


Search API

AFAICT the cs3 apis do not talk about search.
For now we could implement searching in file metadata and content based on elasticsearch, however I think we need a search api in cs3 as well because the storage itself might have search capabilities. At least optionally.
@labkode what are your thoughts on search?
also see https://github.com/owncloud/nexus/issues/17

@butonic Yes, I think the storage provider could provide a find endpoint for this purpose. We just need to be careful on what query filters we allow to not force storage providers to include complicate logic, like "give me all the files where @butonic has written a comment on the 1st of April".

also see https://github.com/cernbox/cs3apis/issues/8


Add anova-like functionality for terms

This is not currently a priority.  Setting to 0.2 milestone, but may decide not to do it then or at all.


Document WHERE pushdown

Description

Is this change a fix, improvement, new feature, refactoring, or other?

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

How would you describe this change to a non-technical end user or system administrator?

Related issues, pull requests, and links

Documentation
( ) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.
Release notes
( ) No release notes entries required.
( ) Release notes entries required with the following suggested text:
# Section
* Fix some things. ({issue}`issuenumber`)

Continued on https://github.com/trinodb/trino/pull/13814


feat: add lyric font settings

P.S: in the now playing window, click on singer and album doesn't seem to work.

I think we'd better change font size/weight text input to select option, it's easy for users.
PS: If there's a preview under that would be nicer, user can preview lyric font size/weight in real time. You can refer qq music settings of lyric.

I changed font weight to select, however I don't think change the font size to select is a good option:
There's no conventional name mapping for font size and I want user to choose from any accurate value of font size they want not just " small" " medium" "large" "x-large", also I will be really redundant if I create a dropdown on different pixel value.
Anyway, I think user a least have better knowledge on font size compared to font weight.


[Feature] new lint rule to require a comment #. at end each python block.

Create a new rule to require that each block must end with # or #.
This would work as a curly braces blocks found in C style languages.
Also, this could also be used to reformat python code by code editor to fix any indentation issues.
Example:
import string
def atbash(sequence: str) -> str:
    """
    Example
    """
    output = ""
    for i in sequence:
        extract = ord(i)
        if 65 <= extract <= 90:
            output += chr(155 - extract)
        #.
        elif 97 <= extract <= 122:
            output += chr(219 - extract)
        #.
        else:
            output += i
        #.
    #.
    return output
#.

Python extension does not implement any particular linting rule. Please file this on a specific linter that you use.
Closing this since this has nothing to do with python extension.
ref: https://github.com/microsoft/vscode-pylint/issues/564


[HOLD #11768] mWeb / Android - Popovers aren't closing on navigating back in native - reported by @thesahindia

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Back button

Go to any chat
Open emoji picker
Press native back button (back button of your device)

Swipe back

Open any chat that has a message
Long press on a message
Without letting your finger leave the screen during the long press, swipe right
The app goes to the previous screen but context menu stays active

Expected Result:
Popover should close
Actual Result:
Popover don't close
Workaround:
unknown
Platform:

Where is this issue occurring?

Mobile Web

Version Number: 1.1.79-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/178182147-8c0dcb4b-a013-4ea5-ab31-d24eb9da3455.mp4
https://user-images.githubusercontent.com/43996225/214363094-a18a9cb7-fdd6-4ae4-a7a2-4aed3ff06b21.mov
Upwork job URL: https://www.upwork.com/jobs/~01402faaf5968f784d
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1656497620210449
View all open jobs on GitHub

Definitely a contributor-worthy issue!

Job posted to Upwork - https://www.upwork.com/jobs/~010c2cbc9e40b3a07a

I don't have an android device to test it myself, but let's ask @mvtglobally if she has a moment to do so!

btw @jeet-dhandha were you unable to reproduce it yourself when you tried most recently?

@michaelhaxhiu Just tried few minutes ago not able to reproduce it. If you have a macbook then you can try it on you android emulator too. Please tell me if you need steps for that.

@michaelhaxhiu I can repro it on production . ( mWeb )
https://user-images.githubusercontent.com/21364901/179274346-30c58e40-d6d6-408c-8bd5-05c95cc0d4ae.mp4

Proposal
Handle this problem for the web platform (we don't see this bug on the android native platform/apk/google play) using the library react-navigation.
The final behavior of the web version is identical to the android native version, and fixes the bug for all popovers.
NavigationRoot.js
if(Platform.OS === 'web') {
    if(this.props.modal?.willAlertModalBecomeVisible && this.lastState.current) {
        Navigation.navigationRef.reset(this.lastState.current);
        const event = new Event('close-modal');
        document.dispatchEvent(event);
        return;
    }
    this.lastState.current = state;
}

If navigation occurs when a modal is open (wrong behavior), revert that navigation and send an event to close the modal.
Correct behavior: The modal should be closed, and no navigation should occur. The next navigation may be accepted, if a model is not visible.
NOTE: https://reactnavigation.org/docs/preventing-going-back/
A similar solution to this would not work, I've tried it, with variations and listening for other events.
BaseModal.js
this.handleShouldClose = this.handleShouldClose.bind(this);
document.addEventListener('close-modal', this.handleShouldClose);

Add the code for handing the modal-close event to the constructor.
handleShouldClose() {
    if(this.props.isVisible) {
        this.props.onClose();
        return true;
    }
    return false;
}

Code of the event handler. More than one modal exist at the same time, but only one modal is visible at any given time. For example out of 10 modals, 1 is visible and 9 others are not visible. This is not my own assumption, I picked this up from the code and comments for the modal object properties (willAlertModalBecomeVisible , isVisible) in Onyx.
Return false: Did not handle, some other handler should handle this
Return true: This event has been handled.
Expensify.js
Onyx.merge(ONYXKEYS.MODAL, {willAlertModalBecomeVisible: false});

Add this to the constructor.
willAlertModalBecomeVisible is correct, but there is a bug where the app may be closed while a modal is open. When the app is opened again, Onyx keeps the data and thinks that willAlertModalBecomeVisible:true, while it should be reset to false, as there is no modal open initially.
**The property isVisible in the codebase is not correct. (bug??)
**
Root Cause
The back button does not cause an event on the web platform, it only causes a change in navigation.
https://stackoverflow.com/questions/57764033/custom-event-for-android-back-button-in-pwa
The only way to deal with this on any web framework is to use navigation. (Android back button/back or forward in browser all cause changes in navigation)
Attachments
Demonstration: A video demonstrating that the bug has been fixed.
https://user-images.githubusercontent.com/56079110/180610788-bd564e73-1747-4d0b-b9c9-1b454ac8697b.mp4
Explanation.zip: A zip containing a video explaining the code, watch this after reading all of the above text.
Explanation.zip
JS.zip: The updated files.
JS.zip

Updating job bounty to $500

Still waiting for proopsals mevlin. But also @parasharrajat did you see the proposal above?

Checking on it. Still need to review a couple of things. I am not sure if we want to take that path.

Issue not reproducible during KI retests. (First week)

Interesting - so this doesn't need assignment it seems! Definitely holding off on doubling the price any more.

Mates, I don't believe it has been fixed. I've checked staging and it is still reproducible. And I have reproduced it by visiting both the staging site and compiling the code before.
Additionally, I would like to make an argument about one part of the philosophy section of the Readme. In its 5.ii section (Cross Platform 99%) it is said that  "If there is a feature that simply doesn't exist on all platforms and thus doesn't exist in RN". This is obviously NOT true. Check the below links, and it has been explicitly stated that these features are not available on all platforms, and yet they both exist in RN.
https://reactnative.dev/docs/modal#onrequestclose
https://reactnative.dev/docs/backhandler
Now if you check the source code for react-native-modal, it uses both of the aforementioned features of react-native. The android back button is simply not handled in a web version. This back button would only cause backward navigation (similar to the back button on your browser). Consequently, to deal with the modals remaining visible on the web version you would have to do something when navigating somewhere else, with the even simpler and minimal solution being closing the modals.
https://snack.expo.dev/@nimamt/a411aa
You can see for yourself by testing the above code on your Android phone's browser, and the Android app version.

Proposal 2
We should use an invisible helper screen to also reflect the popovers/modals in react-navigation, as we are using react-native-modal and not following the recommended way of react-navigation docs.
!!!IMPORTANT See this: https://reactnavigation.org/docs/modal
Note that the recommended way to open a modal is to use a screen in react-native, that is to go to another screen when a modal is opened. And this is also true for popovers/modals that are not full-screen! See https://stackoverflow.com/questions/48992961/react-navigation-modal-height
React-navigation should navigate to a screen/route called "Popover" when a BaseModal (Popover essentially) has been opened.
When we leave the "Popover" screen by pressing the back and forward browser buttons or the android back button or any other way that would be handled by react-navigation, the react-navigation-modal modal is also closed.
When the back button is handled on native versions of the app or the backdrop is pressed or any other way that would not close the "Popover" screen, we ensure that the screen would also be closed.
Again, the result is identical to the native version. Also, It is based on the recommended way to handle modals by react-navigation, and it is cross-platform.
Code
Navigation.js
function handleBaseModalOpen() {
    navigationRef.current.dispatch(StackActions.push("Popover"));
}

function handleBaseModalClosed() {
    const routes = navigationRef.current.getState().routes;
    const lastRouteName = routes[routes.length-1].name;
    if(lastRouteName === 'Popover')
        navigationRef.current.dispatch(StackActions.pop());
}

As mentioned before, we use an invisible "Popup" screen on top of our existing screens when a modal is opened.
When a modal is closed by react-native-modal listeners, our screen in react-navigation is also closed to reflect this.
(And when the react-navigation "Popover" screen is closed, the react-native-modal modal is also closed to reflect this)
BaseModal.js
constructor(props) {
    super(props);

    this.handleShouldClose = this.handleShouldClose.bind(this);
    **this.internalOnClose = this.internalOnClose.bind(this);**
    this.hideModal = this.hideModal.bind(this);
    document.addEventListener('close-modal', this.handleShouldClose);
}

internalOnClose() {
    this.props.onClose();
    Navigation.handleBaseModalClosed();
}

If the modal is closed (internally) by react-native-modal, then the app must ensure the relative screen would be closed, other than having to call this.props.onClose()
AuthScreens.js
class ModalScreen extends React.Component {

  render() {
    return (
      <View style={{ flex: 1 ,flexDirection: 'column', justifyContent: 'flex-end'}}>
      </View>
    );
  }
}

const baseModalHelperListeners = {
    beforeRemove: () => {
        const event = new Event('close-modal');
        document.dispatchEvent(event);
    },
};

<RootStack.Screen
    name="Popover"
    options={modalScreenOptions}
    component={ModalScreen}
    listeners={baseModalHelperListeners}
/>

The code for the "Popover" screen. Read the first part if you skipped it and you are clueless.
Regarding the listeners, this part is because when the react-navigation "Popover" screen is closed, the react-native-modal modal should also be closed to reflect this.
Attachments
As I've said the behavior is basically identical to the native version, also I've provided a video of this correct behavior in a previous proposal.
All the edited js files are in this zip.
Github Servers: Files.zip
Mirror: https://www.mediafire.com/file/bs06dy4r64tsmxa/Files.zip/file

Issue not reproducible during KI retests. (First week)

Hey @parasharrajat any update on this one? Should we double price?
Also @mvtglobally why can't you reproduce it and others claim they can?

Issue not reproducible during KI retests. (Second week)
@michaelhaxhiu Not sure why this is not consistently repro, maybe it is device specific or smth. We have different testers running validations each time, so devices might differ

@michaelhaxhiu let's double this?

@parasharrajat did you review the proposals?

"Definitely a contributor-worthy issue!"
"This is not yet fixed. It would be relatively complex to achieve but I am open to ideas."
"We already closed a similar issue in the past due to a complex solution."
Well, you did know it was complex from the beginning, why waste my time? :-)

Besides, I've also given you a simple solution: Close the modal when navigation happens

@parasharrajat what's the status of this?

Issue not reproducible during KI retests. (Third week)

Bump again @parasharrajat

No update. Unplanned leaves last week. There is an open proposal that I need help finalizing. I will discuss this internally. (will post a link if it is slack discussion)

Issue not reproducible during KI retests. (Forth week)
We are unable to repro consistently.

I will check it today and see if I can still reproduce this. I will also post a video for the same, if reprducible.

Perfect, will keep an eye for the reproduction attempt. If you can't reproduce let's close it.

@parasharrajat were you able to reproduce? If not, should we request a hand in #expensify-open-source to do with an Android?

Going to decide today if we (A) can't reproduce & close this GH or (B) keep open & double it.

Thanks @thesahindia. Doubled.

Do you need more proposals for this issue yet?

Happy to hear if the proposals are different from older ones. For now, working on a high-priority issue.

👋 Appreciate the polite approach of asking @1notchdev - in the future, if you see the help wanted label on a GH in this repo, that means it's available for hire and we are accepting proposals.
@parasharrajat mentions a savvy tip about ensuring your proposal is different from others provided (mainly to save you time from creating a potential proposal that's already shared).

Price doubled!

@parasharrajat  I believe this bug is related to the comment you left on this issue https://github.com/Expensify/App/issues/7010#issuecomment-1018194823 referencing another issue on react-native-web https://github.com/necolas/react-native-web/pull/1738. also this is affecting all the modals, not just the emojipicker.

No update.

I am requesting a buddy check in #expensify-open-source here. @parasharrajat you may be able to unblock my question 🤓

In discussions on the Slack. Close to conclusion.

What is the discussion? The one linked by @michaelhaxhiu ?

Based on the latest discussion https://expensify.slack.com/archives/C01GTK53T8Q/p1668198693474999?thread_ts=1668027443.464419&cid=C01GTK53T8Q, we decided to do close the modal/popovers as well as navigating the user to the previous page in history.
This will keep the default browser behavior and fix the issue of stuck popovers/modals.
@michaelhaxhiu Can you please update the expected behaviour on the description?

This issue is now taking proposals actively. Please reshare your proposals based on the new expected behavior https://github.com/Expensify/App/issues/9823#issuecomment-1317686264 if you already proposed.

Nice conclusion here! @michaelhaxhiu I figure we can move this forward with a fresh post to #expensify-open-source with the updated issue?

Looked into this and pretty sure this is actually related to the Freeze component used on the ReportScreen. When we go back using the browser while a modal is up it gets stuck since the modal visibility is dependent on state. since the component is frozen (which stops it from re-rendering) the modal never closes.
This fixes the issue:
diff --git a/src/pages/home/ReportScreen.js b/src/pages/home/ReportScreen.js
index 972dc0210f..b6d69bc234 100644
--- a/src/pages/home/ReportScreen.js
+++ b/src/pages/home/ReportScreen.js
@@ -218,7 +218,7 @@ class ReportScreen extends React.Component {
         const isLoadingInitialReportActions = _.isEmpty(this.props.reportActions) && this.props.report.isLoadingReportActions;
         return (
             <Freeze
-                freeze={this.props.isSmallScreenWidth && this.props.isDrawerOpen}
+                freeze={false}
                 placeholder={(
                     <ScreenWrapper
                         style={screenWrapperStyle}

However, disabling the Freeze component will re-introduce a bug where the previous screen shows briefly before the next screen when switching chats.
We are gonna get rid of the Freeze components as soon as we improve the react navigation stuff (it's kind of a hack IMO). But if we do it now this is what we'll see on native:
https://user-images.githubusercontent.com/32969087/202563174-c59f05ec-db92-4ddc-ba99-1a5d92be360f.mp4

@iwiznia To help clear out the extensive backlog of /App bugs, we're putting the spotlight on all bugs older than 4 weeks old. To help unblock the roadmap and get our bug pipeline back in equilibrium, can you:

Decide whether any proposals currently meet our guidelines and can be approved as-is
For any that can't, please take this issue internal and treat it as one of your highest priorities
If you have any questions, don't hesitate to start a discussion in #bug-zero

Thanks everyone!

P.S. I equally valid solution is to link this issue to react navigation issue and then put this issue on hold until then.

I think this issue is really not so bad and could just wait. Worst case scenario you tap once more to close the menu. Normally, I think people would do the tap on mobile web to exit the modal before going back - so it's a little edge case. Let's pull it into the navigation stuff.

Nice, I agree with that. I'll proactively do that and also assign myself now since I'm curating the tracking issue.

Not overdue, this is on hold

Still on hold for navigation reboot.

This is an internal issue and also on HOLD. We are not accepting proposals for it.

Still held on the larger navigation re-write.

Yep, thanks Melvin. Updated to monthly so you leave us alone for a bit.

Still on hold for navigation.

Still on hold for navigation. No ETA.

Same same

Still on hold

Coming off hold very soon!

Going on sabbatical, please reassign another engineer if this goes off hold

You're fine. I still need to get around to testing.

Hold

@thesahindia Would you mind re-testing this issue that you reported and confirm if it's still happening?

Perfect, thank you! $250 payment due to @thesahindia for issue reporting.

Given that you are on NewDot payments, I'm going to approve and close this issue.

$250 payment approved for @thesahindia based on summary above.


Server websocket over wss

I've developed a web app called RTR-OBSERVER that can be installed as a Progressive Web App (PWA) and used offline on smartphones.
When running it locally via HTTP, it works well. However, for real-world deployment, HTTPS is required to make the PWA work. This also means that the WebSocket connections need to use a secure connection (wss://).
@flemming-n-larsen, have you ever considered this issue?

Nice work with the RTR-OBSERVER :+1:😊
Regarding WSS, I have considered this, but not prioritized it yet. 🙂
But I will try to set up a server to use WSS and see how it goes. It should be easy to handle on the client side, but will require more work on the server side, e.g. setting up a certificate + UI and documentation stuff.

@SirStone I have had a look into this, and it should not be a big problem to support WSS. 😊
But I will fix #96 first, so the configuration and setup is better prepared for this feature when selecting the server.

@flemming-n-larsen no rush, I'm just glad you considered my request for implementation.

@SirStone
With release 0.26.0, remote servers are now supported. Lots of work has been put into this release (mostly on the GUI side) to support both remote and local servers, and it is a much better foundation for incorporation WSS in a coming release.

Great! Now I'm curious about the changes on the GUI :)
Il Dom 20 Ott 2024, 11:38 Flemming N. Larsen @.***> ha
scritto:

@SirStone https://github.com/SirStone
With release 0.26.0
https://github.com/robocode-dev/tank-royale/releases/tag/v0.26.0,
remote servers are now supported. Lots of work has been put into this
release (mostly on the GUI side) to support both remote and local servers,
and it is a much better foundation for incorporation WSS in a coming
release.
—
Reply to this email directly, view it on GitHub
https://github.com/robocode-dev/tank-royale/issues/106#issuecomment-2424780910,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ7Z3HN3PRMD2Y774ZIATTZ4N2ZLAVCNFSM6AAAAABN5L6EZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRUG44DAOJRGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


[Deploy] TickTick

Service
TickTick is a to-do organizer.
Link to your repository
https://github.com/jonhil/recipe-franz-ticktick

any news on enabling Ticktick plugin? thx

You can download my repo and install it manually as mentioned here.


Cannot paste plain text in a textarea inside of a droparea.

First off, thank you for your work!
I found this issue when trying to implement a fullpage droparea.  I've re-created a similar situation here:
http://jsfiddle.net/L6wkb54g/
I narrowed the issue down to the paste event listener.  As a temporary workaround, I added a conditional that skips evt.preventDefault() if it finds a custom attribute ngfAllowPasteText.  However, I believe you added it for a reason.
    elem[0].addEventListener('paste', function (evt) {
      if (elem.attr('disabled') || disabled) return;
      if (!$parse(getAttr(attr, 'ngfAllowPasteText'))()) { evt.preventDefault(); } // Changed this line
      if (stopPropagation(scope)) evt.stopPropagation();
      extractFiles(evt, function (files, rejFiles) {
        updateModel($parse, $timeout, scope, ngModel, attr,
          getAttr(attr, 'ngfChange') || getAttr(attr, 'ngfDrop'), files, rejFiles, evt);
      }, false, getAttr(attr, 'multiple') || $parse(getAttr(attr, 'ngfMultiple'))(scope));
    }, false);

Let me know if I can provide any more detail to help.  Thanks again!

Yea I will separate the paste image directive from drop in the next version. For now use the previous version 6.0.2

Got it.  Thanks for the heads up.  :+1:

Ok it is a bit of work to separate it into another directive, so for now I just removed the preventDefaults and stopPropagation lines.
Should be working at version 6.0.4


Best option for including image generation as a side feature?

Hello!
I'm looking into adding diffusion based image generation into a video game for fairly simple stuff like decals.  The game will be running under some pretty heavy resource constraints.  If I may ask, what would you recommend as the lightest option that give any sort of good result?  Are there stable diffusion quants like there are flux quants?
On a related note, do you intend to support BitFusions 2 bit stable diffusion model if Snap Research ever OKs it for release?
https://snap-research.github.io/BitsFusion/
Thanks!

Stable diffusion doen't handle GGUF quantization as well as Flux does. Q8_0 is still good, but smaller quants quickly become much worse.
About the BitFusion thing, I'm very sceptical, because from their demo images, it looks like they are claiming that the images generated by their heavily quantized distillation of SD1.5 look much better then the default SD1.5? This can't be right...

I believe they did post quantization training.  That might account for it.  A presentation on what they did is here:
https://www.youtube.com/watch?v=cBMrc1cY4bs
The lead author is also pretty approachable, if you want to reach out and email him:
https://eclipsess.github.io/yangsui.github.io/


Bluetooth: controller: Implement CSA #2 for SubEvents

Implement the functions required to calculate the SubEvent 1
and SubEvent n mapped channel indices.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

@wopu-ot please review.

@heml-ot @mtpr-ot please comment on how this CSA#2 implementation squares with our local implementation.

@kruithofa please re-review.


Question about the "increment of Brownian motion"

Can someone explain what exactly does the parameter W (The increment of the Brownian motion over the interval [t0, t1]) in BrownianInterval mean mathematically?
Let's say that I have a Brownian motion with E[dB_t dB_t^T] = Qdt. Is the parameter W related to Q? If yes, how should I set the value of W given that I have Q?

The parameter W is equal to B_{t_1} - B_{t_0}. For example it can be set to zero, so that BrownianInterval samples from a Brownian bridge.
In particular that should mean that it is unrelated to your use case. I believe sampling uncorrelated Brownian motion (via BrownianInterval) and then multiplying by sqrt(Q) should give you the distribution you're after.

Thanks a lot for the clarification, @patrick-kidger.
A small follow-up:
Setting the diffusion function (g) equal to cholesky(Q)/sqrt(Q) in the SDE should produce the same effect, right? Assume that my state_dim and brownian_dim are equal and the SDE that I am interested in looks something like this:
dy(t) = f(t, y(t)) dt + dW(t)  with       y(t0) = y0    and    E[dW(t)dW(t)^T] = Qdt

Yep, that sounds correct to me.

Awesome, thanks! Closing this.


fix(kitty): image shown when switching to different tmux session #198

fixes #198
steps to verify PR on issue above.
Approach:

detect and clear image when attached session has changed
detect and clear currently active pane tty across sessions

Hey, thank you for your work, checking it out!
I'll probably merge it into a different branch first to replace the awk/xargs/etc part with Lua, and we can get it in <3

This is a bit problematic with multiple attached sessions, looking into it.

@shurensha I think something like this should work https://github.com/3rd/image.nvim/tree/pr/199
Could you please try it out?

#199 still does not work for me. Doing some quick print debugging on the code it looks like utils.tmux.get_current_session() always returns the name of the session where the editor that lost focus lives. So it always is the same as initial_tmux_session when losing focus due to switching sessions, and therefore nothing gets cleared. Using tmux 3.4 and Neovim 0.10.1.


chainntnfs: leverage batched JSON-RPC requests to speed up spend/conf rescans

With this PR btcd has now gained support for being able to respond to batched JSON-RPC requests. The client has also been updated to allow several requests to be fired off in a single message as well. bitcoind has supported this feature for some time itself. With this new client-side capability, we can being to take advantage of the now available request pipelining to be able to speed up several interactions within lnd itself.
The goal here is to use the batch JSON-RPC requests to fetch several blocks/transactions/hashes in a single message where relevant across the codebase. Porting over the chainntnfs package to be aware of this new capabilities may serve to significantly speed up long running rescans, spend notification rescans, and also confirmation rescans. We can also extend this into btcwallet itself to speed up the initialization of the wallet, as well as seed import and other on demand rescan attempts.

This PR in btcwallet may resolve this issue: https://github.com/btcsuite/btcwallet/pull/807


Preparation for testing of integrated CoinSelection module.

Issue Number
ADP-1037
Summary
This PR performs some groundwork in preparation for testing the integrated CoinSelection module. It:

Extracts out internal functions from CoinSelection.performSelection to clarify the control flow and make testing easier.
Moves prepareOutputsWith to the top-level CoinSelection module.
Moves minted and burned assets from SelectionSkeleton to TransactionCtx.
Populates the assetsTo{Mint,Burn} field within selectAssets.

bors r+


peers doc iteration

Signed-off-by: Josh Horton<EMAIL_ADDRESS>Type of change

New feature
Documentation update

Description
Another iteration on the Peers topic for new Fabric Gateway service.
Related issues
https://github.com/hyperledger/fabric/issues/2807

So the "issue" with this page/topic is that it was originally written for Peers, and then goes through the relationship in detail of a Peer to EACH other element in a Fabric channel. Hence, the sense of duplication across this very long page/topic regardless of the gateway service adds. Especially in the later Orderers and Peers section, which is very long. I cleaned up that section e2e, BUT also left the Phase 1, 2, 3 discussions in there. ... With Phase 1, 2, 3 discussions ALSO in the previous Applications and Peers section, there remains some duplication - albeit from different perspectives (Apps vs. Orderers).
So rather than just delete legacy text - Orderers and Peers does have some interesting comments in there about security for example - I suggest we leave what I left in for Monday without further re-strategizing the entire page/topic. (But I have deleted substantial parts, such as those on determinism, per the review.)
In summary, the whole Peers page has gotten my e2e today and I think it's pretty good for Monday. Notable exception being the missing graphics for Figures 6 and 10, which are commented out for now. I'll read it again Friday in the formatted view to be sure of all details, and can do another PR per you reviews as necessary.

@joshhus No changes are reflected in this PR!

Replaced by https://github.com/hyperledger/fabric/pull/3079


Server don't start from manager

Hello, as the topic say... The server don't start or there might be something else messing with my server.
I hope someone here might be able to help me...
`
@2023-11-10T13:32:17.248Z | IMPORTANT | Manager      | Starting DZSM Version: 3.4.0 - []
@2023-11-10T13:32:17.255Z | INFO      | Database     | Database Setup: node 16.16.0 : v93-win32-x64 - []
@2023-11-10T13:32:17.262Z | IMPORTANT | Config       | Trying to read config at: C:\DayzServer\server-manager.json - []
@2023-11-10T13:32:17.278Z | IMPORTANT | Config       | Successfully read config - []
@2023-11-10T13:32:20.248Z | IMPORTANT | Bootstrap    | Initially checking SteamCMD, Server Installation and Mods. Please wait. This may take some minutes... - []
@2023-11-10T13:32:20.249Z | IMPORTANT | SteamCMD     | Checking/Installing SteamCMD - []
@2023-11-10T13:32:20.758Z | INFO      | SteamCMD     | Progress:   0% - Checking for available updates... - []
@2023-11-10T13:32:21.261Z | INFO      | SteamCMD     | Progress: ---- - Verifying installation... - []
@2023-11-10T13:32:22.236Z | IMPORTANT | Monitor      | Starting to watch server - []
@2023-11-10T13:32:22.237Z | WARN      | Discord      | Not starting discord bot, because no bot token was provided - []
@2023-11-10T13:32:22.259Z | IMPORTANT | RCON         | Listening on <IP_ADDRESS>:60196 - []
@2023-11-10T13:32:22.262Z | IMPORTANT | REST         | App listening on the http://<IP_ADDRESS>:2313 - []
@2023-11-10T13:32:22.266Z | IMPORTANT | IngameREST   | Ingame Rest API listening on the http://<IP_ADDRESS>:2312 - []
@2023-11-10T13:32:22.267Z | IMPORTANT | Bootstrap    | Server Manager initialized successfully - []
@2023-11-10T13:32:22.267Z | IMPORTANT | Bootstrap    | Waiting for first server monitor tick.. - []
@2023-11-10T13:32:22.896Z | IMPORTANT | Monitor      | Server not found. Starting... - []
@2023-11-10T13:32:22.899Z | INFO      | ServerStarte | Skipping to write server cfg because it is not configured - []
@2023-11-10T13:32:22.902Z | INFO      | ServerStarte | Adjusting dayzsetting.xml - []
@2023-11-10T13:33:53.231Z | IMPORTANT | Monitor      | Server not found. Starting... - []
@2023-11-10T13:33:53.234Z | INFO      | ServerStarte | Skipping to write server cfg because it is not configured - []
@2023-11-10T13:33:53.237Z | INFO      | ServerStarte | Adjusting dayzsetting.xml - []
@2023-11-10T13:35:23.957Z | IMPORTANT | Monitor      | Server not found. Starting... - []
@2023-11-10T13:35:23.962Z | INFO      | ServerStarte | Skipping to write server cfg because it is not configured - []
@2023-11-10T13:35:23.963Z | INFO      | ServerStarte | Adjusting dayzsetting.xml - []
@2023-11-10T13:36:54.707Z | IMPORTANT | Monitor      | Server not found. Starting... - []
@2023-11-10T13:36:54.712Z | INFO      | ServerStarte | Skipping to write server cfg because it is not configured - []
@2023-11-10T13:36:54.714Z | INFO      | ServerStarte | Adjusting dayzsetting.xml - []
`
This is how my console looks... at the moment I have disabled the manager server cfg part for testing but without success...
Where can I start looking?
DayZServer don't seem to start either or do I miss something.
Best regards

Fixed it, Manager needed to be in a different folder to works...


Update mechanism for OCIR regions by using the region variable itself

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Add support for Johannesburg.
https://docs.oracle.com/en-us/iaas/Content/General/Concepts/regions.htm

New or Affected Resource(s)

Potential Terraform Configuration

# Copy-paste any Terraform configurations for how the requested feature may be used.

References

@karthicgit Just discussed this with @snafuz, he has a slick proposal which will make maintaining this easier. I'm reassigning it to him

@hyder Thanks for the update.


Superset 3.0.2 Public Googlesheet can be read without service account credentials but throws error when you add creds

A clear and concise description of what the bug is.
How to reproduce the bug

Create a public google sheet
Add the google sheet as database to your Superset 3.0.2. It works just fine and data shows up in SQL Lab
Now delete this and this time when adding google sheet database select Public and Private Sheets and provide your service account credentials (by file or by text, doesn't matter)
See error

Expected results

Our sheet should be read because it is public
-- Providing json credentials seem to cause an error that prevents a con being created
-- Logs cannot distinguish between invalid input file, credentials failed, failure to read sheets or failure to create a connection

Actual results
On adding the credentials, you see on the interface The URL could not be identified. Please check for typos and make sure that ‘Type of Google Sheets allowed’ selection matches the input

In superset logs you see:
superset  | 2024-01-10 15:10:23,177:WARNING:superset.views.base:SupersetErrorsException
superset  | Traceback (most recent call last):
superset  |   File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1823, in full_dispatch_request
superset  |     rv = self.dispatch_request()
superset  |   File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1799, in dispatch_request
superset  |     return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
superset  |   File "/usr/local/lib/python3.9/site-packages/flask_appbuilder/security/decorators.py", line 95, in wraps
superset  |     return f(self, *args, **kwargs)
superset  |   File "/app/superset/views/base_api.py", line 127, in wraps
superset  |     raise ex
superset  |   File "/app/superset/views/base_api.py", line 121, in wraps
superset  |     duration, response = time_function(f, self, *args, **kwargs)
superset  |   File "/app/superset/utils/core.py", line 1526, in time_function
superset  |     response = func(*args, **kwargs)
superset  |   File "/app/superset/utils/log.py", line 255, in wrapper
superset  |     value = f(*args, **kwargs)
superset  |   File "/app/superset/views/base_api.py", line 93, in wraps
superset  |     return f(self, *args, **kwargs)
superset  |   File "/app/superset/databases/api.py", line 1433, in validate_parameters
superset  |     command.run()
superset  |   File "/app/superset/databases/commands/validate.py", line 72, in run
superset  |     raise InvalidParametersError(errors)
superset  | superset.databases.commands.exceptions.InvalidParametersError: [
    SupersetError(
        message='The URL could not be identified. Please check for typos and make sure that ‘Type of Google Sheets allowed’ selection matches the input.',
        error_type=<SupersetErrorType.TABLE_DOES_NOT_EXIST_ERROR: 'TABLE_DOES_NOT_EXIST_ERROR'>,
        level=<ErrorLevel.WARNING: 'warning'>,
        extra={
            'catalog': {'idx': 0, 'url': True},
            'issue_codes': [
                {'code': 1003, 'message': 'Issue 1003 - There is a syntax error in the SQL query. Perhaps there was a misspelling or a typo.'},
                {'code': 1005, 'message': 'Issue 1005 - The table was deleted or renamed in the database.'}
            ]
        }
    )
]

Environment
browser type and version: Microsoft Edge Version 120.0.2210.91 (Official build) (64-bit)
superset version: 3.0.2 from official docker image
any feature flags active:
SECRET_KEY = "SUPERSET_SECRET_KEY"
WTF_CSRF_ENABLED = True

FEATURE_FLAGS = {
    "ENABLE_TEMPLATE_PROCESSING": True,
}

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

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

Additional context

It seems that the credentials are not being properly read by superset/db_engine_specs/gsheets.py
In python, json.loads on this json does not work because it cannot read \n characters
I tried to replace gsheets.py with strict = False for json.loads but that did not work on Superset while it works on python: refer this stackoverflow answer
I also tried replacing \n with \\n but that too throws the same error while it works on python: refer to this stackoverflow answer

Are you still facing this? Tempted to close as stale, assuming you figured something out. I know we're using a ton of google sheets with credentials at Preset, so I suspect there's something in the credentials causing a problem (maybe a character like an @ or something), or this was an intermittent bug that came and went. As a side note, you should set a new secret key, too ;)
Let's see if our shiny new bot has more advice:
@dosu-bot

Are you still facing this? Tempted to close as stale, assuming you figured something out. I know we're using a ton of google sheets with credentials at Preset, so I suspect there's something in the credentials causing a problem (maybe a character like an @ or something), or this was an intermittent bug that came and went. As a side note, you should set a new secret key, too ;)
Let's see if our shiny new bot has more advice:
@dosu-bot

This issue still exists for me in Superset 3.1.2. Our "workaround" is not ideal, we have to schedule a script to regularly update another database with the googlesheet contents
If I don't provide any credentials.json for a pulicly shared sheet, the connection works fine. As soon as credentials are specified I get the error "The URL could not be identified. Please check for typos and make sure that ‘Type of Google Sheets allowed’ selection matches the input." which is the same error that I get for private sheets.

These credentials work with a python script using gspread so there are no auth or access issues
I have tried using the credentials.json file as is as downloaded when the key is created for the service account
I have also tried to escape /n as //n but that does not work either. It works in the python script either way

@dosu
I also tested using a public google sheet that was also shared with the service account email address. The connection works fine without the credentials. If I supply any credentials, the error that I mentioned earlier shows up.

Seeing this behavior on 4.0.2 as well.


Should we just assume pandas dataframes as input?

A lot of the data inputs you have in the boot_algo3 function in R assumes a dataframe. Should we just do the same and assume pandas dataframes in the python version? We can then turn them into numpy arrays in the function for performance.

Yes, this sounds like a good idea to me =) Though I do think that in most cases, there is no good reason to assume a DataFrame, so we could also be lenient and accept both.


PUD-1282  API: document detail to be non-blank for version gt 1

API: require generated document detail to be non-blank for all document versions except the first, version 1

Probably I should add a test to DocumentComponentTest to confirm the response to the client.


python generate.py --save_metadrive

Thanks for your work, when I try to use save_metadrive and run the above code in terminal I encountered with the problem in this picture, I am wondering which meta drive version you are using, I just run pip install meta drive to use this package.

Hi, I fixed the issue with latest metadrive. Please pull TrafficGen and see if everything work. Please feel free to inform me the result and reopen the issue. Thanks!


any plans and results for the SE154

I experiment the HTC model using the senet154 backbone models from cadene. And the result is only 42.3 which is lower than your resnext101 model which is 43.9. In the experiment, I used the DCN and multi-scale training. So if convenient,could you share your SENet154 pretrained model in your HTC paper?

You may expect it in a month.

Hi，any plans for the SE154 results?


New CLI command for install patches API

Resource Provider
Compute
Description of Feature or Work Requested
We need a new command for installPatches API, which is used to maintain security compliance on azure VMs
Command: az vm install-patches -g rg -n vm
Other input parameters as mentioned below:
NOTE: some parameters vary depending on the VM OS Type (i.e. Windows or Linux)

maximumDuration (Required for both Windows and Linux VMs)
Specifies the maximum amount of time that the operation will run. It must be an ISO 8601-compliant duration string such as PT4H (4 hours)

rebootSetting (Required for both Windows and Linux VMs)
Defines when it is acceptable to reboot a VM during a software update operation.
Possible values are:
- IfRequired
- Never
- Always

classificationsToInclude (Optional for both Windows and Linux VMs however the possible values vary)
The update classifications to select when installing patches. Possible values differ for Windows and Linux.
Possible values for Windows VM:
- Critical
- Security
- UpdateRollUp
- FeaturePack
- ServicePack
- Definition
- Tools
- Updates
Possible values for Linux VM:
- Critical
- Security
- Other

kbNumbersToInclude (Applicable to Windows VM only and is an optional parameter)
KBs to include in the patch operation.

kbNumbersToExclude (Applicable to Windows VM only and is an optional parameter)
KBs to exclude in the patch operation.

excludeKBsRequiringReboot (Applicable to Windows VM only and is an optional parameter)
Filters out KBs that don't have a reboot behavior of 'NeverReboots' when this is set.

packageNameMasksToInclude (Applicable to Linux VM only and is an optional parameter)
Packages to include in the patch operation. Format: packageName_packageVersion.

packageNameMasksToExclude (Applicable to Linux VM only and is an optional parameter)
Packages to exclude in the patch operation. Format: packageName_packageVersion.

Minimum API Version Required
2020-12-01
Swagger Link
https://github.com/Azure/azure-rest-api-specs/blob/compute-2020-12/specification/compute/resource-manager/Microsoft.Compute/stable/2020-12-01/compute.json#L4038
Target Date
March 1st Public Preview (Ignite)

2020-12-01 is not released yet. We have accumulated so many features that depend on it. March 1st, not a realistic time.

@qwordy Any updates on when this will be picked up?

@rane-rajasi Added to S185


Incorrect Command to Update Tag

Should this:
docker tag mcr.microsoft.com/azuredocs/azure-vote-front:v1 /azure-vote-front:v2
Be this?:
docker tag /azure-vote-front:v1 /azure-vote-front:v2
Given that previous instructions where to do this:
"Replace mcr.microsoft.com with your ACR login server name. You can find the image name on line 60 of the manifest file. The following example shows the default image name:"

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

ID: 56e9075f-9310-5e1f-72d4-a5bd061cdb1b
Version Independent ID: 8afbfa3d-1cc3-5326-502a-291f72a716d8
Content: Kubernetes on Azure tutorial - Update an application - Azure Kubernetes Service
Content Source: articles/aks/tutorial-kubernetes-app-update.md
Service: container-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@moehoward003
Thanks for the feedback! We will check and update the document as appropriate.

@moehoward003
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


pip install -r requirements.txt ERROR

pip install -r requirements.txt
ERROR: Invalid requirement: 'pytest chardet yacs termcolor' (from line 7 of requirements.txt)

Thank you for point that out. I have updated the requirements.txt.


[!!!][BUGFIX] Check if processes were successful

Symfony's Process class swallows errors from executed commands unless
the exit state explicitly checked. This is a rather unexpected but
documented behavior, and may lead to broken releases.
All Process usages are now guarded by a isSuccessful check.

❗ .env command for md5 has been changed - individual overrides have to updated locally
-CHECKSUM_MD5_COMMAND="md5sum %2\$s"
+CHECKSUM_MD5_COMMAND="md5sum %1\$s"

cc @andreasfernandez @bmack Please make sure to update your local .env concerning md5 command


Filtering images

I'd like to describe the issue on the next line.
#Filters
public function filterData($files, $filter)
this method takes the filter to files ok its good
But  this returns an array of the filters rules
  $filters = config('filemanager.filters', []);
What does method check inside an array if the $filter is true or false?
issue
I always get $filtered  as undefined
file: "/nova-filemanager/src/Http/Services/GetFiles.php"
message: "Undefined variable $filtered"

Fixed in https://github.com/stepanenko3/nova-filemanager/releases/tag/v5.0.1


Need to get back to English  I accidently selected chinnesse for my langauge is it any way possible you can reset my Vs back to englsih

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

I tried clicking all sections but do not speek chinnese so I could not reset back to english

Press F1 to open the command palette.
Type "Configure runtime arguments" and select that command.
Change the locale value from zh-cn or zh-tw to en, or take out that key-value pair entirely.
Close and reopen VS Code.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Get editor instance by id / Get all editor instances

Version
1.4.6
Test Environment
Mac OS, Version: 76.0.3809.100
I'm using jquery to centrally initialize all the editors on the page like shown in the code below

<link rel="stylesheet" href="https://uicdn.toast.com/tui-editor/latest/tui-editor.css"></link>
<link rel="stylesheet" href="https://uicdn.toast.com/tui-editor/latest/tui-editor-contents.css"></link>
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/codemirror/5.33.0/codemirror.css"></link>
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.12.0/styles/github.min.css"></link>

The editors are initialised perfectly well and function fine.
Now is there anyway (any method available) that I can get the tui-editor instance corresponding to the div with id = editor_2. ..so that I can wire up some event on it?
Also, is there any method to get a list/array of all editor instances initialised on the page?
Thanks

@jaydeepk
You can use getInstances static method to get all created instances. Just call this method to get all instances and call APIs for each instance.
const editors = tui.Editor.getInstances();

editors[0].getValue(); // id="editor_1"
editors[1].getValue(); // id="editor_2"
...

And "so that I can wire up some event on it?" I'm not sure what this sentence means.
Did you ask how to register events for each instance? Sorry, can you explain it again? 😅

Thanks @seonim-ryu .
tui.Editor.getInstances() does give me a list of all the editor instances.
The other thing  that I wanted to do was to retrieve a particular instance and attach events on it.
So lets say I have 10 editors on the page :

.
.

Now I want to attach 'change' event on editor_3.
So now instead of fetching the editor instance by index like:
 const editors = tui.Editor.getInstances();
 var editor_3  = editors[2];

is there a way I could fetch the editor instance using the id set on the div:
   // something like
   var editor_3 = tui.editor.get('editor_3');

   // so that I can wire a change event on that specific instance
   editor.eventManager.listen('change', function ( evt) {
          // do something
    });

Thanks

@jaydeepk
I understand! It would be nice to have a function like getInstanceById. I'll consider adding this API and let you know when it's decided. Thanks.

Thanks @seonim-ryu , that's exactly what I need.

Same here xD Would be cool to get an instance of editor.

Is the getInstances method removed from the current version? How can we do this now?


Make StatusPageList Public

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Im editing the code and redesigning the Status Page, id like to put another status page under the status page.
In the following image below, I edit the code but its static button
Id like to ask a help on how I can reproduce the list of status pages and put it on the active status page.

I  saw the statusPageList in the code however its available on authenticated users.
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.0-beta.1
💻 Operating System and Arch
Mac M2 2020
🌐 Browser
Google Chrome
🐋 Docker Version
No response
🟩 NodeJS Version
No response

Id like to ask a help on how I can reproduce the list of status pages and put it on the active status page

I don't quite understand what you mean. What do you need help reproducing?
I would currently suggest that these links could be put in the description of the status page or all status pages being merged into one.
⇒ Currently, the feature to list other status pages from one status page does not exist.

Yes, the feature to list other status pages from one page does not exist.
Id like to edit the code, do we have a feature request on this?

do we have a feature request on this

I was unable to find such an issue.
Please run a duplication search too: https://github.com/louislam/uptime-kuma/issues?q=is%3Aissue+is%3Aopen+status+page+

Id like to edit the code

Here is our contribution/development guide: https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md


i found spmthing that may help

[assistant]=(#message|)
thats what i got when it malfunctioned i think it's the new identifier ?
tell me if this helps (since the old one in the app doesn't work : system )
i will inform you if  [system]=(#additional_instructions|) works for me

Thank you for sharing your findings. It might be helpful for official copilot, but since we have no problem sending prompt-injected context in SydneyQt, it may not make any difference.


[WFLY-4647] RemoteOutboundConnectionReconnectTestCase, EJBClientRecon…

…nectionTestCase configured as clustered test cases
These two cases test outbound connections from one server A to another server B, and connection from a client to a server A when the server is stopped and started. There is no clustering functionality being tested, hence the standalone.xml profile should be used.
https://issues.jboss.org/browse/WFLY-4647

Can one of the admins verify this patch?

This is ok to test

Linux with security manager Build 1903 is now running using a merge of 18c9977154bb441fbddd40ad00919f16c07f0e39

Windows Build 2082 is now running using a merge of 18c9977154bb441fbddd40ad00919f16c07f0e39

Linux Build 7027 is now running using a merge of 18c9977154bb441fbddd40ad00919f16c07f0e39

Linux with security manager Build 1903 outcome was SUCCESS using a merge of 18c9977154bb441fbddd40ad00919f16c07f0e39
Summary: Tests passed: 871, ignored: 280 Build time: 0:06:26

Windows Build 2082 outcome was SUCCESS using a merge of 18c9977154bb441fbddd40ad00919f16c07f0e39
Summary: Tests passed: 2944, ignored: 344 Build time: 0:48:38

Linux Build 7027 outcome was SUCCESS using a merge of 18c9977154bb441fbddd40ad00919f16c07f0e39
Summary: Tests passed: 2942, ignored: 344, muted: 2 Build time: 0:52:03


BEST PRESET FOR ALL NIGHTER?

I tried the new features on yesterday night, but couldn´t find the best preset for having the bot running all night. I woke up with a soft ban. Which do you think is the best config to have the bot sniping all night and preventing a soft ban?

run for 50 searches and take 1 minute pauses

Thanks. Hope I get unbanned soon to try.

ejecutar 50 búsquedas y hacer pausas de 1 minuto

Do you think I may use 1-2 wait time? Im using 2-4 for security, but loosing a lot of snipes.

how long the soft ban continues?

how long the soft ban continues?

Just a day.


Roundtrip error while parsing "1000000000000000010E5"

Google OSS-Fuzz found an error executing fuzzer-parse_json.cpp:

input "1000000000000000010E5" is parsed to JSON value j1.
j1 is dumped to string s1; result is "1e+23".
s1 is parsed to JSON value j2; result is 9.9999999999999991E+22 (Xcode debugger).
j2 is dumped to string s2; result is "9.999999999999999e+22".
s1 and s2 are different.

This is an error in the roundtrip check.
Any ideas on this? (maybe related to #378, #362, and #454)

I can reproduce the problem: "1e+23" roundtrips to "9.999999999999999e+22" with the new code, and back to "1e+23" with the old code.
I think this is the problem:
            static constexpr auto fmt = d == 6  ? "%.7g"
                                        : d == 15 ? "%.16g"
                                        : d == 16 ? "%.17g"
                                        : d == 17 ? "%.18g"
                                        :           "%.19g";
            // I'm not sure why we need to +1 the precision,
            // but without it there's a unit-test that fails
            // that asserts precision of the output

If I change the the values within the format string to the values of d, as one would expect in the first place, then the problem-case round-trips correctly. On the other hand, such change triggers a unit-test failure:
src/unit-inspection.cpp:257: FAILED:
  CHECK( ss.str() == "3.141592653589793" )
with expansion:
  "3.14159265358979" == "3.141592653589793"

After changing the precision, "3.141592653589793" roundtrips stably to "3.14159265358979". Not sure if this is correct though.

It appears that it's the right thing to do, and it is like a problem related to the the unit-test.
    const char* d_str = "3.141592653589793";
    const double d = strtod(d_str, nullptr);
    const auto digits10 = numeric_limits<double>::digits10;
    cout << "digits10:" << digits10 << endl; //15
    cout.precision(digits10);
    cout << d << endl; // 3.14159265358979 (no 3 at the end)

    // with old code that used operator << internally
    // and with the code that uses snrpintf("%.15g");
    const nlohmann::json j = nlohmann::json::parse(d_str);
    std::cout << j << endl; // 3.14159265358979 (no 3 at the end)

So the code
            static constexpr auto fmt = d == 6  ? "%.7g"
                                        : d == 15 ? "%.16g"
                                        : d == 16 ? "%.17g"
                                        : d == 17 ? "%.18g"
                                        :           "%.19g";

would be changed to
            static constexpr auto fmt = d == 6  ? "%.6g"
                                        : d == 15 ? "%.15g"
                                        : d == 16 ? "%.16g"
                                        : "%.17g"; // we already assert that d can only be 6, 15, 16, or 17

?

Yes.

OSS-Fuzz marked the issue as fixed. I shall close this ticket.


Support javascript transformations across related documents

I don't think this was ever fleshed out enough to be actionable


[Disclaimer] Inactivity

Hello!
I owe you apologizes. 2018 has been such a great year and I've been busy all year long, working on a lot of projects. Because of that, this project has become inactive. It's true there is a branch for a new version, but I would like to work on a lot of stuff and I just can't do it right now. Feel free if you want to create a fork of this project and keep up with it, since, for the moment, I just can't keep working on this.
Best regards,
Arthur A. Bergamaschi.

Hi @arthurbergmz!  Would you be willing to allow development to continue in this repository and with this npm module name?  I'd be willing to watch the repo and keep up with PRs and issues -- to maintain the project.

@doug-wade I would be glad to make you a collaborator!
I'll be coming back to the open-source world very soon, I'm just really busy since a couple of months ago because of college and this new job in Montréal. I'm still trying to keep up with side-projects, but it has been kinda hard.
I'll add you as a collaborator. I would also be happy if you continue the "next" branch, if possible! The idea is just recoding it to have a faster and clear code.
Thank you for your support!

@arthurbergmz im willing to help as well! Im new to dev but I can do the small stuff/anything you guys want while I learn more

Hi @csantiago132!  My biggest concern is #56 and #32, which I think are duplicates.  It'd be really helpful to have a minimal repro (like in a repository somewhere) or a failing unit test as a PR, if you're looking for somewhere to start.  The same can be said for #32 -- it'd be easier if we had a repro or failing unit test.  If you're looking for somewhere to start, that might be a good place.
Personally, I plan on picking up #61 next and then doing a release to pick up the crossorigin stuff I just merged into a 3.7.0 release.

@doug-wade ok will do my best. ill create one on my end and have a repo

I'd love to see #56 and #66 and #32 being worked on. I'm not familiar with how webpack plugins work, but I'd imagine it's possible to somehow emit new files to webpack, and webpack passes them to the appropriate loaders etc. so these files will be handled like any other asset/dependency.

Hello, everyone!
Branch next just got update after a while. I'm working again on this project ;)

By the way, thanks everyone who contributed in the meanwhile! You guys nailed it.


ci: use expeditor environment to get package idents

In expeditor, we have access to the exact package identifiers that
were included in the build group.  We can use these identifiers to
avoid eventual consistency issues we've seen with the Habitat Depot.
We are currently unsure on the exact cause of the inconsistent manifest
creation.  While we think it is the result of caching in the Depot API,
we have been unable to confirm this.
To facilitate hunting it down, I've also added logging for cases where the expeditor
environment doesn't match what is found in the depot:
2019-08-30 11:18:37 +0100 WARN: Found mismatch between sources:
2019-08-30 11:18:37 +0100 WARN:    chef/automate-gateway/0.1.0/20190829202630 from expeditor environment
2019-08-30 11:18:37 +0100 WARN:    chef/automate-gateway/0.1.0/20190829202631 from Habitat Depot
2019-08-30 11:18:37 +0100 WARN: Using chef/automate-gateway/0.1.0/20190829202630

This script was growing rather large, so I've broken it up a bit:

Added a number of PackageQuerier classes each of which knows how to
look up packages from a single source.  The MustExistQuerier is maybe a
bridge too far, but oh well, I think the rest are fine.

Added a ManifestGenerator class just to provide a line between "setup" logic and
generation logic since there were a LOT of state variables in the main script.

Added a logger so that we get timestamps in the logs

Add some more log statements so we can see what configuration was used to generate
a given manifest from looking at the logs.
Signed-off-by: Steven Danna<EMAIL_ADDRESS>

It is probably better to just read the file anew than trying to parse this diff.

We could also avoid Habitat completely by always starting with the most recent release manifest.


Test launch failing

Hi,
I am facing below error while running tests.
SwiftMonkeyExampleUITests-Runner.app (74021) encountered an error (Timed out waiting for AX loaded notification)
Tests run on setup:
Xcode Version 10.0 (10A255)
iOS Deployment target: iOS 11.0
Target simulator: iPhone 8 plus
Please help.

Hey there, and sorry for a very late come back. Is this the example app you are testing or your own app in which you integrated SwiftMonkey?

I’m closing this issue because it has been inactive for a few months. Please reopen it if you think it should be addressed.


It is impossible to add the names of vulnerabilities in Latvian

Ordinary Latvian letters like āčēģīķļņōŗšūž and their respective capital letter versions are impossible to be used in vulnerability titles. There is regexp check that excludes these chars and error is "name and locale value must match /^[A-zÀ-ú0-9 []'()_-]+$/i".

see https://github.com/pwndoc/pwndoc/issues/371

Fixed on pwndoc https://github.com/pwndoc/pwndoc/commit/74deb01c27686daa71f2e8f68d0041861b06b1c5

Fixed with https://github.com/pwndoc-ng/pwndoc-ng/pull/102


I am unable to pull image from  private trusted registry

Hi,
I am unable to pull image from  private trusted registry.
here is the error i am getting:
Error response from daemon: Get https://dtr.abcddevpoc.com/v2/myservice/manifests/latest: unauthorized: unable to get identity token from client token: invalid_grant: no such account for refresh token: id:
Thanks & Regards,
Sai.

Closing. You need to check with the authorizer that grants access to that registry; this has nothing to do with the code in this repo


Oracle 23c

I was wondering when we can expect support for Oracle 23c database container?

It is already available
container-registry.oracle.com/database/free:latest

@yunus-qureshi  Sure but what about the usual script in this repository needed to create a customised container?

That will be made available when 23c production release goes out.

@yunus-qureshi why wait?  Users are obviously using containers for Free already, and we want testing and usage to be easy.  @gvenzl already has his image available.

@cjbj ok sure.  @IshaanDesai45 please make the build scripts for for 23.2 available here


Edit Klaxon Runs

Hi there!
Two observations:

Is there a way to edit Klaxon monitors after having launched them?
In the overview of running Klaxon monitors ("Add ons in progress") every job has the same title which makes it confusing if I want to stop a monitor.
Best,
Martin

Scheduled Add-Ons, including Klaxon, can be edited by clicking on Schedule & history from the left sidebar. You can toggle through the pages of  the scheduled Add-Ons to find the one that corresponds to the one you want to edit and then click on the pencil icon next to the one you do want to edit.
Thanks for the feedback on the latter, I'll investigate

Many thanks!

The latter request isn't unique to Klaxon, but all Add-Ons, so I've added it as a request here:
https://github.com/MuckRock/documentcloud-frontend/issues/629


Nuxt is throwing Uncaught ReferenceError: exports is not defined on transpiling an es6 library

Version
v2.10.2
Reproduction link
https://codesandbox.io/s/codesandbox-nuxt-typescript-ch8jz?fontsize=14
Steps to reproduce

Setup a Nuxt project.
Install and import the bip39 library.
Add the library to the transpile array in the build section of nuxt.config.js.

What is expected ?
For the library to transpile correctly
What is actually happening?
Uncaught ReferenceError: exports is not defined error is being thrown at  Object.defineProperty(exports, "__esModule", { value: true });

This bug report is available on Nuxt community (#c9991)

Because its not es module but a commonjs.

I think it is, due to the fact that it would through a syntax error on IE when it's not transpiled.
I even contacted the developers of the package and they said that it requires transpiling in order to work on IE. You can also check the package, there are arrow functions (IE usually breaks around there).

see https://github.com/babel/babel/issues/9187

I'll look into it, thanks for the reply.

@aldarund Thanks, managed to fix the issue. For those who encounter a similar problem, this is my implementation:
build: {
    babel: {
      sourceType: 'unambiguous',
      presets({ isServer }) {
        const targets = isServer ? { node: 'current' } : { ie: 11 };
        return [[require.resolve('@nuxt/babel-preset-app'), { targets }]];
      },
    },
    transpile: ['bip39'],
  },

@SDVII Thanks dude, you helped me a lot! I no longer knew what to do, deeply thankful.


[Snyk] Security upgrade zipp from 3.15.0 to 3.19.1

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

requirements-test.txt

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

@DaveSkender  Do we need this? 🤔

@DaveSkender Do we need this? 🤔

Looks like an override for a sub package to one of the pytest dependencies.  You’d think pytest will update this soon, and it’s only part of a dev dependency, so we can skip it.


Adjust oauth flow to use the middleman

Wait with merging this until the backend supports the new flow and we test it out fully


Limit number of matrix notifications

From @jeroenpraat on November 4, 2014 13:50
Now with Diaspora federation I have sometimes more than 1000, 2000 or more new matrix (/network) notifications. How higher the number how longer it takes to open them and mark them as seen. What is the function? I can understand that there is a function with lower numbers, but now it's getting useless IMHO.
I suggest to limit the number to, let's say, 100. More than that will read as 100+. Another way to do it is to make it an option of course.
Copied from original issue: friendica/red#673

From @ghost on November 4, 2014 14:46
It just counts the number of unseen items.  If you want it to do something else, that's a rewrite of the code.  You're probably better just paginating the results.

So the only logical thing is to provide a user option to disable matrix notifications completely. Let's wait what others have to say.

My issue is primary about the large number, that's growing and growing when you don't mark it as seen. The only time I viewed the results was when I had only a few connections.

From @pafcu on November 4, 2014 16:29
I think there should be settings for enabling/disabling different notifications. @jeroenpraat complains about matrix notifications, I complained about events notifications, and there have been complaints about excessive popup notifications (e.g. remote auth successful) too. While notifications are often useful, in some cases you do want to get rid of them, because they are more distracting than they are useful.

From @friendica on November 5, 2014 2:23
ba7f1bb8eee5

From @ghost on November 4, 2014 15:52
Well, it happens in mod/ping.php 352-367
If you hijack $result['network'] you can change it according to user settings...but you're going to end up with a permanent $value the first time somebody doesn't read all the notifications and ends up with more than $count unseen a few pages old.  There's no obvious way to escape marking all seen - unless you went back to the Friendica way of doing things, and marking the lot seen on the first view, but that would be inconsistent with all the other notifications.

This is great! Even better as wished for :-)


Merge to master

https://ledgerhq.atlassian.net/browse/B2CA-969

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 27.01%. Comparing base (74e65a4) to head (1c74157).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #111   +/-   ##
=======================================
  Coverage   27.01%   27.01%
=======================================
  Files           1        1
  Lines         211      211
=======================================
  Hits           57       57
  Misses        154      154

Flag
Coverage Δ

unittests
27.01% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Ops! Incomplete Completion. The request requires 640 tokens and you only have 500. Add more tokens in your CodeGPT Settings.

image

Go to settings -> Extensions -> CodeGPT and add more tokens: