[certificate] Add certificate expiration badge

Hi!  I have this, and it is basically working (i.e. tests pass), but there is some caching going on that is causing problems.
If you try to get the same hostname within 60ish seconds, it will not call checkServerIdentity and you'll get an error.  It works again after a bit, and other hostnames will also work (but just once) during the interval.
I think it is something being cached or re-used in guts of got.js, but couldn't find a workaround.  I'm happy to bypass got.js and call node.js https.request directly, but wanted to check if this would be acceptable first.
Separate question: I have a website with a page to check certificates example.  Is it okay to use it for the link?

I haven't really reviewed this PR in any detail as I think it is not really finished yet. I have had a quick look at your core issue though.
Fundamentally, it seems like this is the issue you're hitting: https://github.com/sindresorhus/got/issues/2153
Anecdotally, this seems to work:
const resp = await this._request({
  // stuff
})

const cert = resp.res.req.socket.getPeerCertificate()

// do more stuff with cert - get cert.valid_to, or whatever

@chris48s Clarifying things:

The PR code works: the tests are real, and they pass.
I have worked with certificates in node.js before (successfully).
Something in the internals of shields.io code and/or got.js is caching/re-using something, that is making the 2nd attempt to verify the same cert fail for a period of time.

Example behavior:

check cert for shields.io (time t): gets date, passes
re-check cert for shields.io (time t+1 seconds): does not try to get date (i.e. checkServerIdentity is not called, though I can see that fetch does hit the webserver), throws
re-check cert for shields.io (time t+90 seconds): gets date, passes

I have already spent a fair bit of time without making much progress.  If you want to try to get this in:

I can rewrite it to use the native node.js https.request instead of got.js
I am willing to work with someone who understands the guts of shields.io's got.js usage to fix the issue.

Sorry if I'm being snippy: I really do like shields.io & would like to contribute.
Andrew

I'm aware of the issue you're trying to solve, and I'm trying to help you. In my previous post, I've linked you to a relevant upstream issue with more detail and proposed a solution.
The behaviour you're seeing is a behaviour of got, not shields. Lets take shields completely out of the equation. Here's a minimal script that reproduces the behaviour you're describing just using got directly:
import got from 'got'

for (let i = 0; i < 2; i++) {
  const resp = await got(
    'https://shields.io/',
    {
      https: {
        checkServerIdentity: (innerhost, certificate) => {
          console.log(certificate.valid_to)
        }
      }
    }
  )
}

Note we are running this twice in a loop, but you'll only get one line of output.
As noted in https://github.com/sindresorhus/got/issues/2153#issuecomment-1272536302 A got maintainer says this is not the advised way to do this (although admittedly there is not much detail on why).
Here's another minimal example using the approach I've suggested in my post above:
import got from 'got'

for (let i = 0; i < 2; i++) {
  const resp = await got('https://shields.io/')
  console.log(resp.req.socket.getPeerCertificate().valid_to)
}

Again, we're running this twice in a loop. Notice this time you should get 2 lines of output.
You should be able to use that approach to get yourself unstuck.

Your test program works only because the connection is still open.  If you read the response (which _request does), it fails.  I can certainly change it to call got directly and bypass using all the _request infrastructure, but then I might as well call https.request.
Failing example (just added a call to .text() to force reading the response):
import got from 'got'

for (let i = 0; i < 2; i++) {
  const resp = await got('https://shields.io/').text()
  console.log(resp.req.socket.getPeerCertificate().valid_to)
}

BaseService._request() doesn't call .text() (or .json() or .buffer()).
We do return the value of .body, so what we're doing is more like:
import got from 'got'

for (let i = 0; i < 2; i++) {
  const resp = await got('https://shields.io/')
  const body = resp.body
  console.log(resp.req.socket.getPeerCertificate().valid_to)
}

so you should be able to call socket.getPeerCertificate().
This:
const resp = await this._request({
  // stuff
})

const cert = resp.res.req.socket.getPeerCertificate()

does work for me (across multiple calls).
Is your concern with this that you think it doesn't work at all, or that it is in some way unreliable? If its the second one, can you explain further. If we need to, I have a couple of ideas about how we can pass the cert details back to the caller, but I don't think we need to.
In general, if we're going to take on the maintenance of this, we do need to do this via our usual process for calling got as this allows us to manage certain settings globally like making the request with our standard user agent, ensuring there is a limit on the size of response we will accept, etc. I don't want there to be one service which is bypassing those global settings by directly invoking got, https.request, or some other http client.

I updated the PR as you suggested.  The happy path is working, but the error which happens on redirects shows as "error" instead of getting the date.  A good example is cloudflare.com, which has a different certificate from www.cloudflare.com. (2024-12-31 vs 2024-05-23)
Is there anything I should be doing differently in the error handler or passing in options?
I would like to handle this case, since certificates on hosts that redirect are easy to overlook, but I suppose it isn't a hard requirement.  I am guessing (but haven't tested) that it would fail on sites that have basic auth as well.

We're throwing here
https://github.com/badges/shields/blob/84e29440ba2d6afeaccfe8678c76c555b5221b61/core/base-service/check-error-response.js#L16-L18
I would say if we're using the option followRedirect: false, we shouldn't be throwing there. If that option is set to false, a 301 or 302 is valid/expected. This has just never come up before because there are zero other badges where we use followRedirect: false.
I think the solution here is to modify BaseService._request() to pass options.followRedirect to checkErrorResponse() and then checkErrorResponse() shouldn't throw if followRedirect === false and res.statusCode === 301 or res.statusCode === 302.

Just having chewed this over slightly more, I reckon using error.response is probably the best option to look at first.

Update per your suggestion and it seems to work.  Once it is live, it is worth seeing if there are any of the caught errors without an expireStr.


BUG: list Multiple Constraint Dependencies Not Supported

poetry supports multiple constraint dependencies provided as arrays of inline tables in TOML (i.e. lists of dictionaries in Python):
[tool.poetry.dependencies]
foo = [
    {version = "<=1.9", python = ">=3.6,<3.8"},
    {version = "^2.0", python = ">=3.8"}
]

The current source errors in parsing these, leading to the following error when attempting to use the plugin with such a dependency:
'list' object has no attribute 'name'

To fix this, please use the method in the current poetry source:
_constraints = constraints if isinstance(constraints, list) else [constraints]  # Convert to list

for _constraint in _constraints:  # Iterate through each item in constraint list
    # Now parse dictionaries in list

👍 thanks for the report.
Are you interested in opening a pull request or would you rather I address it?

Only thing I don't know how to do yet is handle recombining the constraints again into one multi-constraint. e.g. foo in the above gets 2 entries in target_entries if you do:
target_dependencies: List[Dependency] = []
for group in groups:
    ...
    for name, constraints in dependency_config.items():
        if isinstance(constraints, list):
            _constraints = constraints
        else:
            _constraints = [constraints]

        for _constraint in _constraints:
            target_dependencies.append(
                Factory.create_dependency(name, _constraint)
            )

If you print out target_dependencies, you'll get:
[<Dependency foo (<=1.9)>,
 <Dependency foo (>=2.0, <3.0),
]

I'm trying to read the poetry source now to find out how they do it...

@zanieb I don't see that poetry does anything in particular. I think this should work (starting at line 266 in the current source):
for group in groups:
    dependency_config = extract_dependency_config_for_group(
        group, poetry_config
    )
    if dependency_config is None:
        continue

    for old_constraint, dependency in updated_dependencies[group]:
        # Mutate the dependency config (and consequently the pyproject config)
        name = dependency.name
        version = dependency.pretty_constraint

        constraints = dependency_config[name]

        # Support multiple constraint dependencies
        if isinstance(constraints, list):
            for ndx, _constraint in enumerate(constraints):
                if isinstance(_constraint, dict):
                    dependency_config[name][ndx]['version'] = version
                else:
                    dependency_config[name][ndx] = version
        else:
            if isinstance(constraints, dict):
                dependency_config[name]['version'] = version
            else:
                dependency_config[name] = version

        # Display the final updates since they can be buried by the installer update
        self.info(
            f"Updated <c1>{dependency.pretty_name}</> constraint from "
            f"<c2>{old_constraint}</> to <c2>{dependency.pretty_constraint}</>"
            f"{_pretty_group(group)}"
        )

👍 thanks for the report.
Are you interested in opening a pull request or would you rather I address it?

My pleasure. If you want to make the changes I suggested, I'd rather you address it if that's alright.

No problem, I'll look into it. Thanks for taking the time to research it!

Here's a failing test https://github.com/zanieb/poetry-relax/commit/11a4f28f300fa3ecdfa875798169cb3d35553fc8 — although it does not fail with the error you described so I may have set it up wrong.

#52 resolves this if you're interested in reviewing!

#52 resolves this if you're interested in reviewing!

Thank you! Love your fast response (you're a great maintainer)


Throw SparkArrayIndexOutOfBoundsException for Spark 3.3.0+

Throw SparkArrayIndexOutOfBoundsException when Array.getElementAt() tries to access out of bounds
Fixes #4039

build

build

build

build

GpuElementAt should also use this new exception shim, as Spark is using the same query execption method there.

Sorry, I should've caught that. I have added tests for element_at and threw the new exception

build

build


x/pkgsite: "other flags" section of fmt package documentation is garbled

What is the URL of the page with the issue?
https://pkg.go.dev/fmt#section-sourcefiles
What is your user agent?
Mozilla/5.0 (X11; Linux x86_64; rv:101.0) Gecko/20100101 Firefox/101.0
Screenshot

What did you expect to see?
The format flags as listed in the source documentation.
What did you see instead?
The - flag turned into a list indicator, newlines gone.

Duplicate of #53759


Custom Title?

I've tried both...
gpaViewController.title = NSLocalizedString("ENTERCITYNAME", comment:"Enter City")
gpaViewController.navigationItem.title = NSLocalizedString("ENTERCITYNAME", comment:"Enter City")

If you'd prefer not to use the master branch or wait for a new version, then the (admittedly clunky) way to achieve this with the current version is:
gpaViewController.gpaViewController.title = "..."


Make a script, that automatically sets up the server after updating the branch

Currently the following steps are required to set up the aws server:
pkill screen
cd ~/chat-team-11/rasa
conda activate rasaenv
rasa train
screen -S rasa -d -m rasa run --cors "*" --enable-api
screen -S rasa-actions -d -m rasa run actions
cd ~/chat-team-11/frontend
sudo cp -r * /var/www/html/
cd /var/www/html/
sudo sed -i -e "s/localhost/<IP_ADDRESS>/g" index.html # Probably better not to hardcode the IP and instead use source like in the provision.sh
sudo sed -i -e "s/localhost/<IP_ADDRESS>/g" scripts.js

The objective is to write a script, that reliably sets up the server and gives feedback, if sth goes wrong.

Optional function added in comments to enable Input forwarding from the commandline
Additionally it is required to start the script with source cause sudo ./setup.sh will result in an error. More on this in our wiki.
#IP=$1
#if [ -z "$IP" ]
#then
#  echo "IP param is unset! Aborting..."
#  exit
#else
#  echo "IP is set to '$1'"
#fi
pkill screen
cd ~/chat-team-11/rasa
conda activate rasaenv
rasa train
screen -S rasa -d -m rasa run --cors "*" --enable-api --log-file out.log --debug
screen -S rasa-actions -d -m rasa run actions
cd ~/chat-team-11/frontend
sudo cp -r * /var/www/html/
cd /var/www/html/
sudo sed -i -e "s/localhost/<IP_ADDRESS>/g" index.html
#sudo sed -i -e -r 's/(\b[0-9]{1,3}\.){3}[0-9]{1,3}\b'/"$1"/ index.html
#sudo sed -i -e -r 's/(\b[0-9]{1,3}\.){3}[0-9]{1,3}\b'/"$1"/ scripts.js
sudo sed -i -e "s/localhost/<IP_ADDRESS>/g" scripts.js

Issues in der Wiki: https://github.com/ID-Start-Winter22/chat-team-11/wiki/🏃♂-Ideation-und-Sprint-1#-issues


Built custom ubuntu18 image takes huge space

Hi there,
I tried to build a custom ubuntu18 image from souce (6f60a7d).
The docker images takes 40G. Even if I cut off source code repo, it's still 14G.
This is relatively large compared to those released on docker hub
Is it possible to further reduce the size of built OpenVINO?
B.R.,
Kelvin

Hello @KodeWorker,
Sorry for the delay. This Docker image is mainly intended for OpenVINO development and contains OpenVINO, Open Model Zoo and OpenCV built from source code and installed dependencies. I scanned built image with the dive tool and the most largest parts are OpenVINO repo and build files (28 GB), Python packages (5,9 GB) and OpenCV built from source (4,2 GB).
Can you describe your use case? The image size can be reduced by removing components unnecessary for you (like OpenCV or installed ML frameworks: tensorflow, mxnet etc).

Hi @ilya-naumov,
Thanks for your advices.
In my use case, the docker image is only for converting ONNX model to OpenVINO IR.
Does it means that I can remove all the OpenModelZoo, built OpenCV and some of the Python packages?
B.R.,
Kelvin

Hi @KodeWorker,
I think yes, you don't need OpenCV and some other components for this task. Currently the custom Dockerfile supports building an image without OpenCV (more info in the readme) - this should reduce image size by about 4,2 GB:
docker build --target openvino -t ubuntu20_custom_no_opencv:latest .

Another way to reduce the image size is to modify the Dockerfile manually. I created this file with the following changes:

Removed OpenVINO and OMZ source codes and build files, OMZ demos
Install ONNX-related Python dependencies only (I changed this line).
Removed OpenCV

The resulting image takes (uncompressed) 6.4 GB versus 5.72 GB for openvino/ubuntu20_dev:2021.4.2 from Docker Hub.
I think a good long term solution would be to add the ability (maybe build-arg) to build an image that doesn't include the repository files.

@KodeWorker You can also reduce the image size by building OpenVINO in Release configuration (instead of Debug specified by default). The CMake build options is specified in the openvino_cmake.txt file (more info can be found in the wiki page) :
-D CMAKE_INSTALL_PREFIX=install

-D CMAKE_BUILD_TYPE=Release
-D ENABLE_PYTHON=ON
-D PYTHON_EXECUTABLE=/usr/bin/python3
-D ENABLE_WHEEL=ON

Hi @ilya-naumov,
I build the image following your suggestion. The size of image is reduced to 1.88 G.
It is no longer an issue for me.
Thanks you so much :)
B.R.,
Kelvin


Transposed convolution improvements

New more efficient algorithm for strides <= 2 and large input channels
Minimizes scratch buffer needed for original algorithm

Nice work! Please check the warnings in arm_nnfunctions.h.


Better bibtex does not seem to start after update to 5.4.16

I open zotero almost everyday, and every day it works perfectly.
Except today :)
The problem is simple to reproduce:

Open zotero
Fail to see the familiar better bidlatex initialization popup
Fail to see the citekey column in the references list.
Fail to see the citekey field in the reference view.

I'm sure this was introduced in the latest version (5.4.16), since I've in the meantime reverted back to 5.4.15 and everything is working fine.
The error log consists of many messages, similar to this one:
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15

My app specs:
version => <IP_ADDRESS>, platform => Win32, oscpu => Windows NT 10.0; WOW64, locale => en-US, appName => Zotero, appVersion => <IP_ADDRESS>, extensions => Zotero LibreOffice Integration (5.0.25.SA.<IP_ADDRESS>, extension), Zotero Word for Windows Integration (5.0.19.SA.<IP_ADDRESS>, extension), Better BibTex for Zotero (5.4.16, extension)

I followed the instructions to send the better bibtex log report, and got nothing.

I have encountered the same issue.

Are you both on Windows? I have it reproducible on Windows.

Are you both on Windows? I have it reproducible on Windows.

I'm using the latest macOS.

Bummer. Maybe the windows problem gives a lead on the more general problem, but I'm on macos and I have no issue.

If you both just want to get on with your life, grab 5.4.14, but I'd hugely appreciate it if you both would be around to test, given that I can't reproduce on mac.

Thank you! I used the 5.4.15 version and it's working now.
I'd love to test it and offer more debugging info. Let me backup my files and test it with 5.4.16.

Build 954 solves the startup problem on Windows for me.

I had the same issue and it seems to be working now with test build <IP_ADDRESS>4. Thank you for the quick fix.

Build <IP_ADDRESS>4 solves the issue on macOS for me.

Test suite is running, release will be out in approx 10 minutes.

@clagms if you can confirm, we can close this issue.

Damn. Can you get me a full log? The actual problem happens before this.

Sure:
`[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
[JavaScript Error: "Error: Invalid field 'citekey" for base field" {file: "chrome://zotero/content/xpcom/data/itemFields.js" line: 280}]
getFieldIDFromTypeAndBase@chrome://zotero/content/xpcom/data/itemFields.js:280:13
Zotero.Item.prototype.getField@chrome://zotero/content/xpcom/data/item.js:246:19
Zotero.ItemTreeRow.prototype.getField@chrome://zotero/content/xpcom/itemTreeView.js:3500:10
Zotero.ItemTreeView.prototype.getCellText@chrome://zotero/content/xpcom/itemTreeView.js:1086:15
version => <IP_ADDRESS>, platform => MacIntel, oscpu => Intel Mac OS X 10.16, locale => pl-PL, appName => Zotero, appVersion => <IP_ADDRESS>, extensions => ZotFile (5.0.16, extension), Zotero Word for Mac Integration (5.0.36.SA.<IP_ADDRESS>, extension), Better BibTex for Zotero (5.4.17, extension), Zotero LibreOffice Integration (5.0.25.SA.<IP_ADDRESS>, extension, disabled)`

Yes! I think this has the info I need - on it!

I downloaded the test build, but I'm still having the same problem as @gkwiatk.
errors.txt
I was using Zotero beta when I first noticed this problem, then I downgraded to Zotero 5.0.96, but the problem is still present.
I'm also unable to use Zotero at all, and I get this message in the library view, instead of the files:

Zotero does work again if I remove the extension.

Actually mine looks exactly like @brienna

Can you go into the hidden prefs, look up extensions.zotero.translators.better-bibtex.citekeySearch, disable it, and restart Zotero?

Can you go into the hidden prefs, look up extensions.zotero.translators.better-bibtex.citekeySearch, disable it, and restart Zotero?

@retorquere It seems that nothing has changed.
Log.txt

Thanks for the instructions!
typescript.zip

I need a new log from 960. This will not fix it, I need more logging to see where the problem occurs.

<IP_ADDRESS>0 still doesn't work for me

Hi; I ran the log instructions to generate a file 'typescript'.  Thanks for your work. Doug
typescript.txt

typescript.zip

The builds that will drop here will only be debug builds for now -- I still don't know what causes this.
This is driving me up the wall -- neither of those logs show any activity from BBT before the error hits.

I appreciate you guys being patient in helping suss out this problem. I wish I could replicate the problem on one of my systems, but I haven't been able to so far.

slight change: it's easier for me if the file is uploaded as
To generate a full log, on macos or linux, you will need to do the following:
Type script <your username>.txt on the command line. You will see
Script started, output file is <your username>.txt

and will be back at the prompt. There, type
/Applications/Zotero.app/Contents/MacOS/zotero -ZoteroDebugText

reproduce the error, and quit zotero. You should once again see the command prompt. Type exit, you will see
Script done, output file is <your username>.txt

and attach the file <your username>.txt it generated here

xf15.txt

We have activity! I'm trying to bisect where the problem could have occurred, but this helps massively.

Icanhazlog?

xf15.txt

Ugh why is this so &^%@#&^%@# random! No BBT activity before the error hits.

Anyone around here on discord? We can do much quicker turnaround that way. https://discord.gg/D6QsyJ74

I've narrowed down the problem to two candidate areas, if you have time, please drop by in the morning (EU time) on discord. I need to catch some ZZs right now.

Are you both on Windows? I have it reproducible on Windows.

I'm using the latest macOS.

Just as an FYI: I was having the same problem on Linux, so this is not just a Windows/MacOS issue. Downgrade to 5.4.14 has restored functioning again.

Are you both on Windows? I have it reproducible on Windows.

I'm using the latest macOS.

Just as an FYI: I was having the same problem on Linux, so this is not just a Windows/MacOS issue. Downgrade to 5.4.14 has restored functioning again.

Did you try the latest patch 5.4.18? It fixes the issue for me on macOS.

OK, will give it a go... 🤞

Liftoff! 🚀
👍

That's because it's 5.8.14 (check the addons pane) in disguise so as to not get every single BBT user on here.
I'm on discord. There's only two candidates for the problem now, but since I cannot reproduce the problem on any of my systems, I'll need help for the diagnosis. I'm on discord right now.

I try to join you later on in Discord. Unfortunately, the problem still exists for me. I cannot set a citation key and exporting is not possible.
jonnythemonster.txt

Ah wait, I missed the new release. This one ist Working 🚀🚀🚀 Thanks man!
jonnythemonster.txt

That's because it's actually 5.4.14 in disguise. But we're really close on discord.

Thank you.  Working on PayPal contributions, your package is a major element of workflow in my group.

super thanks for helping diagnose this one people! Tests for release are running.

@retorquere: I was just looking at
https://github.com/retorquere/zotero-better-bibtex/compare/v5.4.15...v5.4.16
https://github.com/retorquere/zotero-better-bibtex/compare/v5.4.15...master
clicking on "Files changed". There's this line which is probably unrelated to this issue, but it seems wrong, so you might want to change it:
https://github.com/retorquere/zotero-better-bibtex/blob/10f74dea521005bee3aaf05477d2f6a26e356380/util/issue-state#L42
brances.remote -> branches.remote.
(I was still on 5.4.15 when I read about the issue, so I didn't experience it.)

Fixed, but indeed unrelated, it's a script I use the find stale branches.

10... 9... 8...

(no need to manually install, it will update automatically, or you can request Zotero to update now)

Thank you so much for all the work maintaining this plugin!

It is very much recommended to request to update now - while not catastrophic problems, fixing this issue also flushed out another bug, plus 5.4.19 has a rare addition of eye candy - so if you can find it ;)

Mind I have modest standards for what counts as eye candy - latex user after all.

Do you mean the "exporting" scroll bar near the top of the screen (for when it does auto-export)?

That and the auto-exports show their progress, but for small exports they disappear pretty much before you'd notice them. It's mostly useful for people with fairly large exports.


Missing several files from './build/'

Webpack.config uses a lot of files from that folder, that is not included.
PS: Port this to .NET Core 2.0?

up

@redbaty @zwei61 Will refactor it when antd 3.0.0 land.


Add experimental Prometheus metrics endpoints

Todo:

[x] Expose existing metrics
[x] Add metrics around resync
[x] Add config to enable.  Diasabled by default to avoid port binding clash.
[x] Docs.
[x] Package prometheus client library

[x] Ubuntu (already existed in Xenial, so used backport tool)
[x] RH (added to Jenkins)

[x] UT

So we're going to ship the 14.04 and EL7 versions of python-prometheus-client ourselves?

Looks like we are.  I think you'll also need to update libcalico (the frozen requirements .txt).
@tomdee might know of more places: for adding a new python dependency to Felix, is there anywhere else we need to update?

The frozen requirements are generated by running make update-frozen

@matthewdupre I added https://github.com/projectcalico/libcalico/pull/114 and updated the comments around the use of Gauge.


Question regarding pixel_to_object_mapping (zones) array

Hello,
I observed that some pixels characterized as belonging to some parcel in instance_ids array are mapped to a different one in pixel_to_object_mapping. May I ask what was the reasoning behind that?

Thank you for your answer, that crleared up things!


Consolidate path tests

xspec.bats (a test script for *nix) and xspec-bat.cmd (a test script for Windows) have 2 tests for path related issues (#84 and #119).
Though they need to check the same things, each of them runs XSpec individually.
This pull request consolidates them and runs XSpec just once.
No code change in XSpec core.

This pull request derives from #257. So needs to be handled after that.

This pull request just simplifies the test scripts. It doesn't touch the XSpec code. None of the existing tests are removed as a result of the consolidation.
The consolidated test is able to catch test failures. (demonstrated on Travis and AppVeyor)
This pull request makes it easy to handle some pending pull requests (towards #235 in particular).

So I think this pull request can be merged (after #257) safely without reviewing details, or I'd like to merge this by myself in a few days.


feat: support comments in vscode settings

🎉 Thanks for submitting a pull request! 🎉
Summary
Fixes #5407

This PR adds support for having comments in the settings.json file used by VSCode while using the netlify recipes vscode command.

For us to review and ship your PR efficiently, please perform the following steps:

[x] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[x] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[ ] Update or add tests (if any source code was changed or added) 🧪
[ ] Update or add documentation (if features were changed or added) 📝
[x] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

Thank you so much for taking that on! I'd love to see some tests to prevent any future regressions 🙏🏻

I'd love to see some tests to prevent any future regressions 🙏🏻

Will add a test for JSON with comments.

@kaelig added the test.

@danez I think we can merge this now. Also see: https://github.com/netlify/cli/pull/5436/files#r1096563844


Added missing translations

I'm currently rewriting ModManager (see #154), so many texts will change and so the translations. Which will be maintained here.

I'm currently rewriting ModManager

Yes, I'm following your project, I know about that too :)
But recently my laptop died, so now I spend much less time translating. Especially because of the difficulty of testing them. Yeah, Pojav Launcher helps a lot, but without VS Code it is very inconvenient to compare, edit and correct the translations. So... I may be late.

Oooops... I received your message from mail, and for some reason I thought that you were preparing about my fork with translations for the upcoming rewrite.


Quickstart failing

I can't start the quick-start deployment in minikube
Prerequisites

have minikube v0.25.0
have vault v0.9.3

Reproduction steps:

run setup.sh script found in quick-start folder

Error writing data to intermediate-ca/intermediate/set-signed: Error making API request.

URL: PUT http://<IP_ADDRESS>:8200/v1/intermediate-ca/intermediate/set-signed
Code: 400. Errors:

* no data found in PEM block

Would anyone know the problem that could be causing this?

This seems to be due to changes in the latest version of Vault. I am currently investigating and will fix the script.

Thanks @F21!


huggingface API

According to the the sample code in 'Use via API' link down at the bottom of the huggingface app page, I kept getting error:
{
  type: 'status',
  endpoint: '/generate_image',
  fn_index: 2,
  time: 2024-05-25T17:50:27.405Z,
  queue: true,
  message: null,
  stage: 'error',
  code: undefined,
  success: false
}

Here is the source code to invoke the endpoint:
import { Client } from "@gradio/client";
import path from 'path'
import fs from 'fs/promises'

// const response_0 = await fetch("https://raw.githubusercontent.com/gradio-app/gradio/main/test/test_files/bus.png");
// const exampleImage = await response_0.blob();

// const response_1 = await fetch("https://raw.githubusercontent.com/gradio-app/gradio/main/test/test_files/bus.png");
// const exampleImage2 = await response_1.blob();

 // Read the image file from the local file system
 const imagePath = path.join(process.cwd(), 'imgs/users/OIP.HtnbVUj5aYlTxYh8-eV2eQHaJ3.jpeg');
 const imageBuffer = await fs.readFile(imagePath);
const userImg = new Blob([imageBuffer]);

 // Repeat for the second image if necessary
 const imagePath2 = path.join(process.cwd(), 'imgs/styles/1ac930a169eab44e709b78f7009cc98b.jpeg');
 const imageBuffer2 = await fs.readFile(imagePath2);
 const refImg = new Blob([imageBuffer2]);

const app = await Client.connect("https://instantx-instantid.hf.space/--replicas/5s9fg/");

try {
	const result = await app.predict("/generate_image", [
		userImg, 	// blob in 'Upload a photo of your face' Image component
		refImg, 	// blob in 'Upload a reference pose image (Optional)' Image component
		"a nice photo", // string  in 'Prompt' Textbox component
		"", // string  in 'Negative Prompt' Textbox component
		"(No style)", // string  in 'Style template' Dropdown component
		30, // number (numeric value between 1 and 100) in 'Number of sample steps' Slider component
		0.6, // number (numeric value between 0 and 1.5) in 'IdentityNet strength (for fidelity)' Slider component
		0.3, // number (numeric value between 0 and 1.5) in 'Image adapter strength (for detail)' Slider component
		0.4, // number (numeric value between 0 and 1.5) in 'Pose strength' Slider component
		0.4, // number (numeric value between 0 and 1.5) in 'Canny strength' Slider component
		0.4, // number (numeric value between 0 and 1.5) in 'Depth strength' Slider component
		["pose", 'canny'], // undefined  in 'Controlnet' Checkboxgroup component
		5, // number (numeric value between 0.1 and 20.0) in 'Guidance scale' Slider component
		42, // number (numeric value between 0 and<PHONE_NUMBER>) in 'Seed' Slider component
		"EulerDiscreteScheduler", // string  in 'Schedulers' Dropdown component
		false, // boolean  in 'Enable Fast Inference with LCM' Checkbox component
		false, // boolean  in 'Enhance non-face region' Checkbox component
	]);

	console.log(result.data);
}
catch (e) {

	console.error('error;', e)
}

The API is turned to closed,  thus you cannot call it directly.

Can you make it a paid service? I'm ok with a paid plan.


typo corrected

@aftab10662,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@ggailey777 Please review these proposed changes. Thanks.

I think that both are technically correct--but yours might be better 😉 Thanks for the contribution.
#sign-off

@ggailey777 Hi Glenn Can you please resolve the merge conflict to enable a merge? It appears that the article might have recently become a redirect to another article. Thanks.

@aftab10662 thanks for your proposed fix, but this topic has been superceded by a new topic. #please-close


Question how to get the browser icon "Install App" in the url ?!

Hello,
Testing silverbullet on my home server I run it like this
/usr/local/bin/deno run -A --unstable-kv https://get.silverbullet.md --user mysuser:1234 /mnt/notes
Instance is running behind a nginx configured in reverse proxy.
For now there is no https, it's just local testing.
I've noticed that I don't have the icon that opens the application in a dedicated Window. That's anoying cause I have tons of tab on my browser And would like to have the notes on a separate one ...

On the other hand if I run the app locally on my laptop with http://<IP_ADDRESS>:3000/, I got the icon, even if there is no https connection.
Do I need to configure something to get this icon ?

You need HTTPS to be able to install it as a PWA. Surprise that it works
with localhost.
On Wed, Sep 4, 2024, 8:20 AM Mathieu A. @.***> wrote:

Hello,
Testing silverbullet on my home server I run it like this
/usr/local/bin/deno run -A --unstable-kv https://get.silverbullet.md
--user mysuser:1234 /mnt/notes
Instance is running behind a nginx configured in reverse proxy.
For now there is no https, it's just local testing.
I've noticed that I don't have the icon that opens the application in a
dedicated Window. That's anoying cause I have tons of tab on my browser And
would like to have the notes on a separate one ...
html-icon.png (view on web)
https://github.com/user-attachments/assets/f9fac89c-0f5c-43ee-a1d0-66fd01edc5b1
On the other hand if I run the app locally on my laptop with
http://<IP_ADDRESS>:3000/, I got the icon, even if there is no https
connection.
Do I need to configure something to get this icon ?
—
Reply to this email directly, view it on GitHub
https://github.com/silverbulletmd/silverbullet/issues/1077, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFC4R2RNIXTO576P4PB4EYDZU33KDAVCNFSM6AAAAABNUGFMZSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYDKMJZGQYTANQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This is a browser security feature that I have no control over. PWAs are only enabled on localhost or <IP_ADDRESS> over http, and over any other URL via https.


NonNegative

I do not stand behind this.  This is for discussion

Sketch of the non-negative type I half-heartedly advocated for in #6286.  This is necessary for the strictest standards compliance, but is quite impractical.  It matches everything with 1*DIGIT in the spec:

Max-Forwards: if you want to allow ten quintillion instead of nine quintillion

Content-Length.  Again, should not be more than nine quintillion, yet we:

MUST anticipate potentially large decimal numerals and prevent parsing errors due to integer conversion overflows or precision loss due to integer conversion

Retry-After: in case we want to tell people to come back in 30 billion years instead of 29

Range specifiers: makes sense that they can refer to whatver is in Content-Length

Without this, we fail to parse well-formed headers, but no backend can reasonably interpret these values that exceed Long.MaxValue.  I think this proposal is impractical.
We could prevent invalid constructions and cover the practical space with a ULong (see Spire for a robust one).  But unsigned long is not a standard JVM type, and we'll be doing range checks all over the backends to make sure nobody used that sixty-fourth bit.
We could also create a 63-bit Long, which is just a refined type.  This might be the PosLong that @danicheg refers to.  I think it's that or #6286.

I don't understand. The boilerplate we force to check values that probably are fine, and if not should rightfully blow up doesn't seem to be worth the overhead.

The main value proposition is that there valid headers we can't parse to a Long-backed model, and we need repeated logic to rule out invalid headers that Long can model.  But those values are all pretty absurd in practice.  I'm also :-1: on it after sketching it.
The only change I see as remotely practical is a NonNegativeLong, using some sort of opaque type.  That's basically refined without the refined dependency.  It would be similar to a whole bunch of String refinements we could do.  I'm also working with a junior developer who is super sharp and super positive until she starts talking about Refined, so, :shrug:.

Just a side note: Refined still has an advantage in some cases because it provides a macro-based compile-time literal parsing. Which can be somewhat cool.

The Literally library makes those macros easy as well.  The distinction is perhaps whether we want refined types (i.e., NonNegLong) vs semantic types (i.e., Length).


Notebook cell diagnostics not cleared when closing a notebook or deleting a cell

Related to https://github.com/microsoft/vscode-jupyter/issues/6130

Closing as stale


Atom Code of Conduct

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description

Steps to Reproduce

Expected behavior:

Actual behavior:

Reproduces how often:

Versions

Additional Information

is this a spam account?

Hi UziTech,
My name is Jeff, I am new to GitHub...working at home, trying to learn some coding.
If I have done anything inappropriate, please let me know--I am still just trying to learn.
Sincerely,
Jeff

No problem, just that the description doesn't say anything about what the issue is.

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


EN VCCV (Cz's) Phonemizer

This is a temporary workaround for the lack of phonemizer for Cz's VCCV method. Since it uses an arpabet dictionary, it lacks some vowels right now but it is fully working. Known issues:

There may be some more VC exceptions to add.
The phonemizer defaults to [V C] for parsing, maybe it should be [VC-]. Feedbacks welcome, please test it yourself!

Feel free to contribute. :)
The Lyric Parser plugin is still a sturdier alternative.

Currently adding some new logic with "prevWordConsonantsCount"

Didn't add Length by Oto in the end because it didn't sound good.


Update for device_tracker.kmd1027703 fails

Hello,
My router is scheduled to reboot every night, so the connection to it is down during few minutes.
Here is the log record:
Logger: homeassistant.helpers.entity
Source: custom_components/miwifi/device_tracker.py:260
Integration: MiWiFi (documentation)
First occurred: 3:58:04 AM (42 occurrences)
Last logged: 4:04:54 AM
Update for device_tracker.kmd1027703 fails
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 446, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 641, in async_device_update
raise exc
File "/config/custom_components/miwifi/device_tracker.py", line 260, in async_update
self._device["ip"] = device["ip"][0]["ip"] if "ip" in device else "0:0:0:0"
IndexError: list index out of range
Perhaps, list "_device[]" is empty in such cases, so it makes reason to frame it with "try / except".
Thanks

Yes thank you. I'll do it. There was no such case before.

Fixed in version 1.2.5. Please check

Thank you, seems to be fixed.
Got exception in another place:
File "/config/custom_components/miwifi/core/luci.py", line 322
Will check later and open new issue if confirmed.


Fix warning suppression for config.toml vs config compat symlinks

What does this PR try to resolve?
Background: the cargo config file is being renamed from .cargo/config to .cargo/config.toml.  There's code in new cargo to look for both files (for compatibility), to issue a warning when onliy the old filename is found, and also to issue a warning if both files are found.  The warning suggests making a symlink if compatibility with old cargo is wanted.
An attempt was made to detect when both the old and new files exists, but one is a symlink to the other, but as reported in #13667, this code is not effective.  (It would work only if the symlink had the precise absolute pathname that cargo has decided to use for the lookup, which would be an unnatural way to make the link.)
Logically, the warning should appear when both files exist but are different.  That is the anomalous situation that will generate confusing behaviour.   By "different" we ought to mean "aren't the very same file".
That's what this MR implements, where possible.  On Unix, we use the information from stat(2).  That's not available on other platforms; on those, we arrange to also tolerate a symlink referring to precisely config.toml as a relative pathname, which is also fine, since by definition the target is then in the same directrory as config.
Fixes #13667.
How should we test and review this PR?
I have interleaved the new tests with the commits that support them.  In each case, a functional commit is followed by a test which fails just beforehand.
(This can be observed by experimentally reordering the branch.)
I have also done ad-hoc testing.
Additional information
I'm making the assumption that a symlink containing a relative path does something sane on Windows.  This assumption may be unwarranted.  If so, "Handle config -> config.toml (without full path)" needs to be dropped, and the test case needs to be #[cfg(unix)].
But also, in this case, we should probably put some warnings in the stdlib docs!

Please add all test cases in the initial commit, passing.  This should show what the current behavior is.  As the commits are made that change behavior, the test cases will then be updated to pass with the behavior changes.
This has a couple of benefits

Helps test the tests
Makes it more clear to anyone reading this what the behavior changes are

Please add all test cases in the initial commit, passing. This should show what the current behavior is. As the commits are made that change behavior, the test cases will then be updated to pass with the behavior changes.

OK.  When should I rebase the branch then?  Normally I wouldn't do that without an OK from the reviewers.

OK. When should I rebase the branch then? Normally I wouldn't do that without an OK from the reviewers.

yes, please

OK.  Thanks everyone for the comments.  LMK about the samefile thing, and I'll probably provide a revised version tomorrow.

I've revised this and rebased/reordered it, as discussed.
While I was writing the code, I felt dirty writing if let Ok(...) = , because that's an ignored error bug.  But I don't think this MR is the right time to address this, so I just added a comment about it.  Ideally we'd change this here, but also  use clippy to forbid use of the buggy exists function throughout the codebase.  If folks agree, I could do that as a followup.

Err, I have discovered a mistake.  Give me a moment, will rewind this again.  Sorry.

I think this is ready now.  CI failures are unrelated.

:pushpin: Commit 2f168383858fac36b5829e777879474af6487eb9 has been approved by epage
It is now in the queue for this repository.

:broken_heart: Test failed - checks-actions

:hourglass: Testing commit 2f168383858fac36b5829e777879474af6487eb9 with merge 52dae0c1f650ac8a376df7ee402cb9a3ffd147d6...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 52dae0c1f650ac8a376df7ee402cb9a3ffd147d6 to master...


Avoid layout cycle changing SelectedBackgroundPath Path size within TabViewItem::OnSizeChanged

Updating TabGeometry asynchronously rather than synchronously in TabViewItem::OnSizeChanged to avoid possible layout cycles.

/azp run


Javascript Error SCRIPT5007 on IE8

When calling the iFrameResize function in IE8, it shows the following error in console:
SCRIPT5007: Unable to get value of the property 'call': object is null or undefined
iframeResizer.min.js, Línea 8 Carácter 5761
I replicated this error using the following initialization parameters with Internet Explorer 9 in IE8 compatibility mode:
iFrameResize({"log": false,
	"heightCalculationMethod": "grow",
	"resizedCallback": function() {
		//...
		}},
	"#form-iframe");

This is the iframe HTML code:
<iframe id="form-iframe" src="http://..."
style="margin:0; width:100%; height:100%px; border:none; overflow:hidden;" scrolling="no"></iframe>

Using version 2.8.4 - 2015-02-20

https://github.com/davidjbradshaw/iframe-resizer#browser-compatibility

Sorry, didn't read the browser compatibility section.

That's ok you just neeed to either use it with jQuery or loaded the polyfil.
On Tue, Mar 17, 2015 at 4:09 PM, robertux<EMAIL_ADDRESS>wrote:

Sorry, didn't read the browser compatibility section.
—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/iframe-resizer/issues/172#issuecomment-82437020
.

--
David J. Bradshaw )'<EMAIL_ADDRESS>


modules/amazon: apply some pokemon error handling

Gotta catch em all.
If there was an issue with the credentials supplied, before the module
would crash due to error[0].Error[0].
This ensures that will not crash the whole bot.
This also drops the 'new' keyword per upstream's documentation and
reduces logging verbosity a bit.
cc @gmackie

@LinuxMercedes nit addressed, LGTY?

LGTM!

Went through and eslinted since #174 merge-conflicted anyways.

Made irrelevant by #180

Yup, closing in favor of 180


Move Watch Connection interface to a published library

It's possibly very useful for other apps that want to support multiple watch platforms. We can separate it into libraries and publish to Maven Central

See https://github.com/boswelja/WatchConnectionLib for development progress

Moved, and implemented in #155


Document DynamicRegistration feature

You may turn DynamicRegistrationTests into the doc.
Check #143 for the context.

Documented here: https://github.com/dadhi/DryIoc/blob/master/docs/DryIoc.Docs/RulesAndDefaultConventions.md#unknownserviceresolvers


It looks like the description mismatch of the code

https://kindxiaoming.github.io/pykan/API_demo/API_9_video.html

you're right, this description was copied from the exp(sin(pi*x)+y^2) [2,5,1] case.


WebRequestMethod does not support CONNECT method

Summary of the new feature / enhancement
WebCmdlets do not support -Method CONNECT, that is supported by System.Net.Http.HttpMethod.
Proposed technical implementation details (optional)
Just add Connect to Microsoft.PowerShell.Commands.WebRequestMethod

Currently there is CustomMethod parameter for such values.
It makes sense to add Connect to Microsoft.PowerShell.Commands.WebRequestMethod only if users use it frequently in the web cmdlets.
A search yielded no examples. Therefore, I doubt the advisability of adding it.

If it does support CONNECT how would you use it in pwsh? I’ve only seen CONNECT used for proxy tunneling which means you need a way to create the inner HTTP or TLS stream which isn’t really feasible in pwsh. Are you needing it for some other purpose?

I don't have a use case, I just noticed it wasn't there and it was supported by System.Net.Http.HttpMethod. I think we can close with a won't fix


[Bug]: Import application redirects to url with no application id

Is there an existing issue for this?

[X] I have searched the existing issues

Description

Steps To Reproduce
Though this is difficult to reproduce when tried importing the golden app on release I faced this issue.
Public Sample App
No response
Version
1.6.17

Verified only by uploading the same Golden app and works fine now


Crash related to #1888

After following the steps from #1888:

Import a NetCDF dataset and save a session
Close Vapor
Move/rename the dataset
Open vapor and open the session

We get an error message, which fixes #1888.
However, now try to import a wrf file and vapor will crash.

This is not related to #1888

These bugs are related to a use case that we were not supporting.


Datasources tab generates a query which can cause ArithmeticException divide by zero

Description
Noticed this on the datasources tab in druid management console.
Datasources tab was not showing any data since the underlying SQL was running into java.lang.ArithmeticException divide by zero.

Here is the query generated by the view -
SELECT
datasource,
COUNT(*) FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) AS num_segments,
COUNT(*) FILTER (WHERE is_available = 1 AND ((is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1)) AS num_available_segments,
COUNT(*) FILTER (WHERE is_published = 1 AND is_overshadowed = 0 AND is_available = 0) AS num_segments_to_load,
COUNT(*) FILTER (WHERE is_available = 1 AND NOT ((is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1)) AS num_segments_to_drop,
SUM("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) AS total_data_size,
SUM("size" * "num_replicas") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) AS replicated_size,
MIN("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) AS min_segment_size,
(SUM("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) / COUNT(*) FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1)) AS avg_segment_size,
MAX("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) AS max_segment_size,
SUM("num_rows") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) AS total_rows,
(SUM("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) / SUM("num_rows") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1)) AS avg_row_size
FROM sys.segments
GROUP BY 1

Below snapshot showing total_rows reported by druid as 0 that causes the calculation of avg_row_size to break

multiple issues here which could be improved -

Improve the UI so that the generated SQL query do not run into divide by zero issue
Figure out why druid is returning num_rows as zero

https://github.com/apache/druid/pull/10496/files#diff-b424606a1aed0c701cf23b70d252bab3b445c334a46fd4f5909c7ced137816ccR230-R244
@nishantmonu51 Looks like this was fixed by #10496 ^

I also encountered this problem twice in my test cluster running 0.20.
It appears that this problem comes out when the cluster is starting while all other web console views work fine. It lasts for roughly 1 minute.

I checked the broker log and found that this problems happens when the historical nodes are starting or  have not been ready. Once the historical nodes are ready, there's no such problem. That's why I said that it lasted for about 1 minute in my cluster.
So it's better to improve the SQL to avoid zero divisor exception. I also checked the changes in #10496 pointed out by @suneet-s , the bug has already been fixed there.

This has been fixed in master and released as part of Imply 2020.11. It will be in Druid 0.21.0.


Release New Version to Hex

I'd love to be able to use different release stages with Bugsnag, which was added to the library last June (in 9a3e15d973ca04bd49b1036f4622415351abd43e). Would you consider releasing a new version to Hex?

@jarednorman ping! Friendly reminder to release new version to Hex:)

Ahh, I forgot. I will make an effort to cut a release this weekend but am travelling so no guarantees.


docs(integrations): Add a block with Gowebly CLI

Hi, @a-h 👋
This PR adds a block with Gowebly CLI to the Integrations page.
I recently added (yes, I am the author of this project) support for project generation with Templ for the built-in net/http and Go web frameworks Fiber, Gin, Echo, Chi and HttpRouter.
So it would be great to make a collaboration of our products, so that more developers will choose this way to start a new project! ✨

⭐ Repo: https://github.com/gowebly/gowebly
📖 Docs: https://gowebly.org
🙌 Article series on Dev.to: https://dev.to/koddr/series/24817

✨ Also, this is an introduction article about Gowebly CLI + Templ integration: https://dev.to/koddr/big-update-the-gowebly-cli-now-supports-templ-3gog

Implemented in #503


Update view-service-health.md

Made changes to reduce ambiguity on how to access Service health

@JohanFreelancer9 please copy review

issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7903

@kelleyvice-msft Please merge this PR. Thanks


Add Dockerfile for Moore Threads GPU

@thxCode

Also add musa to https://github.com/gpustack/gpustack/blob/main/.github/workflows/docker-ci.yaml#L32

Also add musa to https://github.com/gpustack/gpustack/blob/main/.github/workflows/docker-ci.yaml#L32

Done.


Document Egress feature

For #1924

/skip-all

Appended a newline at end of file to fix "Verify docs and spelling"

/skip-all


Add "extra-roles" flag to run, and fix bug in test command.

In #28 there was a flag added called "extra-roles". Per discussion,
adding it to the run command as well.
The test command did not have the "destination" flag,
which meant that containers launched using the run command with that
flag specified could not be tested. Added the flag and updated the
config.

Tested:
ansible-role-tester run -d /etc/ansible/roles/$(basename $(pwd)) -t ubuntu1804 --extra-roles /usr/local/etc/ansible/roles --name test2
ansible-role-tester test --name test2 -d /etc/ansible/roles/$(basename $(pwd)) --extra-roles /usr/local/etc/ansible/roles

its really common to see failed or timed out tests. I’ve restarted the test.


info needed

Hello I would like to know if anyone has a working example for Json read and write files with this library
Thanks

i thing you are looking in the wrong place. Hardware libraries does not determine the digest of a file. Look for a lib to digest JSON (which are not the best idea to use in CPP, better would be to chose xml, jaml, a csv...)

Hello, I encountered a problem writing data to a file, only the first letter is written. USB drive is formatted to fat32. I'm using Arduino mega serial1 115200. What can I do to fix this?
Thanks


Make all functions top level?

I was thinking, it may make for a more predictable API to avoid namespaces and just have every function be top-level, e.g. setParent() instead of parents.set(). That works better when you do import * from treecle . What do you think @adamjanicki2?

I support this!

This is also required for option 1 in #1 so I'm on it


Entry "Stars of Blood" exists twice

Describe the bug
Entry "Stars of Blood" exists twice and points to the same page
To Reproduce
Steps to reproduce the behavior:

Go to https://archive.thedatadungeon.com/index.html
Scroll down to 'Stars of Blood'
Find two entries 'Stars of Blood', both pointing to https://archive.thedatadungeon.com/stars_of_blood_cancelled.html

Okay, fortunately it's just a case that it accidently got added to the root.dd file twice it seems, obviously had a brain fart. Will amend after work.


Holder of composable

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Refactoring
[x] Bug fix

Needs QA check

@kodadot/qa-guild please review

@prury  affected areas are

requirements for drop
mint button (enabled/disabled) and label

all should work perfect both on holder of drop (reflection) and others (monotifs / refraction)
Context

[x] Closes #9635

Fixed bugs

label of mint button for holder drop was wrong
count refs were wrong inside isHolderAndEligible in MintButton. making the button disabled wrongly

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

thank you for fixing the holder of count/button state

trying to mint on the reflection of the sound drop, but I'm getting the failed to load image error(same happening on canary)
seems like a image worker problem
https://github.com/kodadot/nft-gallery/assets/36627808/eed71ad6-231b-4c74-bbcc-a170307e4432

Geolocation still needs to be implemented later on

collection is null error, prob causing latest nft mints not to load

@yangwao hey, can you send some funds to the testing account?
there are funds there, but most of them are locked in created collections on assethubs or existential deposit and the rest was spent during testing
addresses:
DOT: 1CBmjqJvV1ha8BPQ5JAXGEzeJBLmGHNG9gYg96eC1dnjH5W
KSM: CmWHiv7h4m9tEzKD94DH4mqwGTvsdYQe2nouWPF7ipmHpqA

two things:

changing wallet is not resetting the requirements properly(in the video it shows the previous ✔ from wallet used before, instead of loading icon)
connect wallet button disabled when disconnected

https://github.com/kodadot/nft-gallery/assets/36627808/62fb53f6-ca05-4145-b104-dce624aefe72

@prury

regarding geolocation, i don't see the issue, both refraction and alteration drops show same location in this PR and canary

collection is null error, prob causing latest nft mints not to load

fixed

mint button has see listing instead of minting(drop is open)

fixed

two things:

changing wallet is not resetting the requirements properly(in the video it shows the previous ✔ from wallet used before, instead of loading icon)
connect wallet button disabled when disconnected

fixed

1CBmjqJvV1ha8BPQ5JAXGEzeJBLmGHNG9gYg96eC1dnjH5W

sent https://polkadot.subscan.io/extrinsic/19866199-3

@prury

regarding geolocation, i don't see the issue, both refraction and alteration drops show same location in this PR and canary

How geolocation works: if you are in the specified location, minting button should be active and allow you to mint, if you are not in the specified location

@prury

@prury

i should have been more specific sorry, still waking up, verification works properly, but it lets me mint even though i'm not in colorado:
https://github.com/kodadot/nft-gallery/assets/36627808/0465d4cd-4a2c-48c0-ae8a-d53a59f157ca

found something:
on canary, if you exit the uploading to IPFS modal while its loading, the button will stay deactivated until it finishes uploading and cancel the operation
if i do the same on this PR, the button stays active and won't work, only after upload to ipfs action ends(end of video)
https://github.com/kodadot/nft-gallery/assets/36627808/9c2fe7e6-9e62-4209-b9d7-fa0aa2b3ca70

updated branch to trigger deployment from netlify

@prury sound like a diff in loading state handling
Thank you for finding it
I'll check

thank you!
another one:
Infinite checking when disconnected (holderof only)

@prury all done

@Jarsen136 @preschian
Review please 🙏

@prury is good to merge?

@prury is good to merge?

after dev reviews, yes

Thanks!
pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 11.01 USD/DOT ~ 4.541 $DOT
🧗 1cAsKZYNRb8dkSCpn4eVkEn6ycNZTGoDo5dGDgB8J1UUWK8
🔗 0xfb6eb5324763b5d69b46caf6fe2a61d83b754f8e116a06b8460201b2ec0f44e6
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Other related tools

People have pointed out other related tools to include in the Philosophy section of the docs:

https://github.com/semantic-release/release-notes-generator
https://github.com/sympy/sympy-bot

https://github.com/sqlalchemyorg/changelog

https://github.com/huntcsg/releasely (https://twitter.com/FoolOfGod/status/1308046683780141056)

https://pypi.org/project/git-changelog/ (https://twitter.com/pawamoy/status/1308176705178218501)

https://pypi.org/project/towncrier/ (haven't used either one, curious to know the difference)

@lofidevops thanks, I already have towncrier in the Philosophy section: https://github.com/nedbat/scriv/blob/master/docs/philosophy.rst   I'd be happy to add more detail if you think it is warranted.

:facepalm: Would help if I'd looked for it first. Easy to read, thanks!

changie Seems to be very similar to this project

https://github.com/changesets/changesets


Update miniconda on Windows from 4.6.14 to 4.8.3

Tool information

Tool name: Miniconda
Tool license: BSD 3-Clause License
Add or update? Update
Desired version: 4.8.3
Approximate size: 54.1MB
If this is an add request:

Brief description of tool:
URL for tool's homepage: https://docs.conda.io/en/latest/miniconda.html#windows-installers
Provide a basic test case to validate the tool's functionality:

Area for Triage:
Packages
Question, Bug, or Feature?:
Feature
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Ubuntu 20.04 LTS
[x] Windows Server 2016 R2
[x] Windows Server 2019

Can this tool be installed during the build?

Tool installation time in runtime

Are you willing to submit a PR?
Yes, if someone can guide me a bit on where to start :smile:
These lines seem to suggest that Miniconda was pinned to 4.6.14 (released April 2019) for some reason, is that reason still valid?
https://github.com/actions/virtual-environments/blob/b47ba413c9bd9411407b8a5cf18e2c14ea8bda78/images/win/scripts/Installers/Install-Miniconda.ps1#L10-L15
xref https://github.com/conda-incubator/setup-miniconda/issues/58#issuecomment-687670999

@weiji14 , no any reason to hardcode specific version. I think it may be upgraded to 4.8.3 or even latest (https://repo.anaconda.com/miniconda/Miniconda3-latest-Windows-x86_64.exe) to simplify future maintenance.
Do you want to submit PR?

Sure, I'll give it a go :smile:

Thank you for the PR,
we would like to get more context why 4.6.14 was hardcoded previously and I have pinged a few people. Hope it will take little time

Hello @weiji14 , Sorry for delay.
Changes were merged and new image will be deployed in 1-2 weeks

Awesome! Thanks for keeping on top of this, looking forward to the deployment!

Changes appear to deployed, close this?

Changes are still deploying (Some machines are already updated, some not). It should be finished today. After that we will close issue.
Everything works as expected for you?

@weiji14 the images were deployed!
Miniconda version is 4.8.3
https://github.com/actions/virtual-environments/blob/main/images/win/Windows2019-Readme.md#database-tools
I'm going to close the issue, thank you for your contribution!
Please feel free to contact us if you have any concerns.


Hot Restart not updating changes on web

Steps to reproduce

Create a new Flutter project
Implement a web application within the project.
Make changes to the code.
Execute a hot restart using the command flutter run -d web.
Observe that the changes are not reflected on the web.

Expected results
After performing a hot restart (flutter run -d web), the web application should update and reflect the recent changes made to the code.
Actual results
The hot restart does not update the changes on the web, and the application continues to display the previous version of the code.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _currentIndex = 0;

  final List<Widget> _tabs = [
    TabOne(),
    TabTwo(),
    TabThree(),
  ];

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Bottom Tab Bar Example'),
      ),
      body: _tabs[_currentIndex],
      bottomNavigationBar: BottomNavigationBar(
        currentIndex: _currentIndex,
        onTap: (index) {
          setState(() {
            _currentIndex = index;
          });
        },
        items: [
          BottomNavigationBarItem(
            icon: Icon(Icons.home),
            label: 'Tab 1',
          ),
          BottomNavigationBarItem(
            icon: Icon(Icons.search),
            label: 'Tab 2',
          ),
          BottomNavigationBarItem(
            icon: Icon(Icons.person),
            label: 'Tab 3',
          ),
        ],
      ),
    );
  }
}

class TabOne extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Center(
      child: Text('Tab 1 Content'),
    );
  }
}

class TabTwo extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Center(
      child: Text('Tab 2 Content'),
    );
  }
}

class TabThree extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Center(
      child: Text('Tab 3 Content'),
    );
  }
}

Logs
Logs
Performing hot restart...                                          426ms
Restarted application in 431ms.

Performing hot restart...                                        1,450ms
Restarted application in 1,451ms.

Flutter Doctor output
Doctor output
[√] Flutter (Channel beta, 3.19.0-0.2.pre, on Microsoft Windows [Version 10.0.22000.2538], locale en-US)
  • Flutter version 3.19.0-0.2.pre on channel beta at C:\Users\devfe\Downloads\flutter_windows\flutter
  • Upstream repository https://github.com/flutter/flutter.git
  • Framework revision c1df7d07ac (9 days ago), 2024-01-17 15:42:07 -0800
  • Engine revision 4f18bb4dcb
  • Dart version 3.3.0 (build 3.3.0-279.0.dev)
  • DevTools version 2.31.0

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
  • Android SDK at C:\Users\devfe\AppData\Local\Android\sdk
  • Platform android-34, build-tools 33.0.0
  • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
  • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
  • All Android licenses accepted.

[√] Chrome - develop for the web
  • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Enterprise 2022 17.3.4)
  • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Enterprise
  • Visual Studio Enterprise 2022 version 17.3.32901.215
  • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2021.2)
  • Android Studio at C:\Program Files\Android\Android Studio
  • Flutter plugin can be installed from:
    https://plugins.jetbrains.com/plugin/9212-flutter
  • Dart plugin can be installed from:
    https://plugins.jetbrains.com/plugin/6351-dart
  • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[√] VS Code, 64-bit edition (version 1.85.2)
  • VS Code at C:\Program Files\Microsoft VS Code
  • Flutter extension version 3.80.0

[√] Connected device (3 available)
  • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.2538]
  • Chrome (web)      • chrome  • web-javascript • Google Chrome 120.0.6099.225
  • Edge (web)        • edge    • web-javascript • Microsoft Edge 120.0.2210.144

[√] Network resources
  • All expected network resources are available.
  • No issues found!

Hot restart changes were updating/reflecting changes on web fine in my old flutter version

Hello @devfemibadmus
Thanks for filing this. I see you are on the beta channel and this is a known issue which has been fixed https://github.com/flutter/flutter/issues/141588.
Closing as a duplicate of that issue.


Hot fixes and updates

Empty string in phone number caused an issue and rest of properties was ignored while updating of profile.
Fixed wrong key for longitude on iOS side.
Removed redundant post-fix 'model' for KlaviyoProfile.
Added missing fields, such as organization, title, image and properties for custom purposes.

LGFM - Good job


Kickoff Camel K support for Camel Quarkus 2.x + Camel 3.10

To build this, first build camel-k-runtime from camel-main:
mvn install
And then:
make clean ; make IMAGE_NAME=you-registry.com/group/camel-k PACKAGE_ARTIFACTS_STRATEGY=copy RUNTIME_VERSION=1.9.0-SNAPSHOT codegen build images-dev images-push

Release Note
NONE

@squakez FYI


Update VMSS to Mariner with FIPS enabled

Which issue this PR addresses:
Covers improving bootstrap scripts in : ARO-6773
Covers move to Mariner OS for FIPS in: ARO-8989
Fixes
What this PR does / why we need it:
This moves our RP and Gateway VMSS instances to Mariner OS, which is compatible with FIPS.
This is also needed for a resilient to failure RP deployment.
Our current implementation has shown some faults over time with RP deployments, and need improvements in general.
Move all shared code into a commonly shared file to be sourced by all
bootstrapping scripts. This allows for code reuse, minimal duplication.
Change VMSS OS to Mariner with FIPS enabled by default.
Firewalls are configured at the Azure vnet level. I did not configure firewalld, because in my testing Mariner OS does not support firewalld out of the box.
Test plan for issue:
Testing during development by deploying a full service development RP environment.
Further testing will be done by deploying to INT. I will update with the successful INT deployment once completed.
If you would like to test these bootstrapping scripts out yourself, here are some helper functions.
Example Log Output
Abort
Error: Unable to find a match: azsec-clamav azsec-monitor azure-cli azure-mdsd azure-security
dnf_install_pkgs: failed to install required packages

Log
configure_sshd: setting ssh password authentication
configure_disk_partitions: extending partition table

Is there any documentation that needs to be updated for this PR?
No
How do you know this will function as expected in production?
Per testing mentioned in the test plan.

azp run ci,e2e

/azp run ci,e2e

Do we have understanding of when automatic updates are ran or is it just as-needed, outside of our control? Do we know how long they take to execute per-VM? It seems like we'd need that to make adjustments to our alerting.
I am in agreement that we should tolerate 1 unhealthy at a time, maximum, but only for a short period of time.

@cadenmarchese We could manually apply updates ourselves. I think that would require creating a pipeline to do so, which would give us more control. That's a topic I think is worth discussion amongst the team.
For them to be automatically applied, we have no real insight or control.
They don't provide a time estimate per machine. I have configured a 10 minute waiting period between updating machines, so an update will apply to instance 0, wait 10 minutes, proceed with the next update.
The OS updates are re-provisioning the instance with an updated OS image. I believe that means our custom script will have to run after this each time. That would mean we'd see two reboots before the update is completed. One for the updated OS re-image, our custom script runs, reboots post completion, then it's back to responding to requests.

/azp run e2e

/azp run e2e

/azp run e2e


Coop Information page: newsletter link is broken, minor content fixes

The page at: https://www.rchain.coop/coop-information-1/ has a newsletter subscribe button that doesn't respond. This seems to be the same problem as on the main page a few days back.
@patrick727 Is this page linked from somewhere? It's slightly out of date as the Activist Meeting is now after the Weekly Debrief. Also the main title is funky: the word devel-opment is broken into two lines (on my screen) and it's hard to read.
Measure of Completion:  the newsletter button and content items are fixed.
Suggested Budget: $250

@kitblake that's true, I tried signing up for newsletter but it isn't going through.

@David405 When you signed up, was that on the main page or the Information page? I believe the main page is fixed, I just tested it and the button seemed to send, which is what is what not doing a few days ago.

@kitblake did you fix this and can we close this out?

Yes, closing.


Fix depreciation

np.bool was depreciated and now throws an error on the latest numpy versions.
https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations

Oops, looks like this was already fixed


🐛 Fix and add test to capture a race w/ partial metadata in cache

Signed-off-by: Vince Prignano<EMAIL_ADDRESS>
Summary
Fix should go in kcp/kubernetes (maybe?), this test captures the sequence of events leading up to the issue
Related issue(s)
Related to #2278
/cc @ncdc @stevekuznetsov @sttts

@sttts is this ok to merge?

/milestone v0.11


[GEP-20] Zone pinning for shoots on multi-zonal seeds 📌

How to categorize this PR?

/area control-plane
/area high-availability
/kind enhancement
What this PR does / why we need it:
This PR adds the zone pinning requirement discussed by GEP-20, ref.
In essence it does the following:

Gardenlet labels namespaces which belong to non-HA or single-zonal shoots if it's responsible for a multi-zonal seed (seed which has worker across availability zones).
A pod-zone-affinity webhook in GRM adds a zone-aware affinity term for such control-plane pods so that those will be scheduled into a single zone only.
After etcd was deployed successfully, it's assumed that at least one stateful pod was scheduled. Gardenlet then extracts the zone information from that pod/node.
From that moment on, pods' affinity configurations is enhanced by a nodeAffinity to "pin" pods to a specific zone. It is especially important to schedule pods to the original zone when shoots are being woken up from hibernation.

Which issue(s) this PR fixes:
Fixes partially #6529
Release note:

Gardener has been being prepared for more shoot HA use-cases and thus some assumption about currently running landscapes are required: If you use a `multi-zonal` labelled seed and scheduled non-HA shoots onto it, this release of Gardener will potentially cause scheduling conflicts to the control-plane pods as it will try to locate all pods into a single zone only. Pods that can't be re-scheduled (mainly because of volume dependencies) will remain in `Pending` state.

Gardener is prepared to run non-HA and single-zonal shoots on multi-zonal seeds. In such a setup, control-plane pods of the mentioned shoots are scheduled into a single availability zone only to avoid any extra cross zonal traffic that would usually involve higher latency and cost.

/hold
I plan to improve the affinity terms after having a second look. With the current approach we rely on the fact, that any pod with a volume is created and scheduled first. This becomes especially important when a shoot is woken up from hibernation. If a stateless pod is scheduled first to any zone, then the remaining stateful pods are supposed to be scheduled to exactly this zone. Unfortunately, there is no guarantee that this stateless pod will run in very same zone that was used before the cluster was hibernated.
This PR will be transferred into draft mode as long as I'm working on the improvement.

Update: The PR is ready for review now.

/assign

/assign

@oliver-goetz thanks a lot for your feedback:
Concerning your questions: It's assumed that there are no seed clusters with worker nodes across multiple zones today that host "normal" shoot clusters. As discussed, I'll add a release note explaining the breaking change.

/rerun

Thanks for your feedback @rfranzke and @oliver-goetz. I addressed all your comments, open is still https://github.com/gardener/gardener/pull/6579#discussion_r962570312. PTAL.

Gardener has been being prepared for more shoot HA use-cases and thus some assumption about currently running landscapes are required: If you use a `multi-zonal` labelled seed and scheduled non-HA shoots onto it, this release of Gardener will potentially cause scheduling conflicts to the control-plane pods as it will try to locate all pods into a single zone only. Pods that can't be re-scheduled (mainly because of volume dependencies) will remain in `Pending` state.

Just to double-check: This is likely no problem because the "seed.gardener.cloud/multi-zonal" label must be added manually to the Seeds, right? I can imagine that there are folks in the wild that have seeds with multiple AZs, however I believe that nobody has added this label yet given that it's quite new and the whole HA story is still alpha/in development. In this case, I'm fine with it.

@rfranzke

Just to double-check: This is likely no problem because the "seed.gardener.cloud/multi-zonal" label must be added manually to the Seeds, right?

Yes, that's mostly right. We planned to enable automatic labeling for ManagedSeeds because there we have information about workers.

Thanks @rfranzke. I incorporated your feedback and in addition found some spots where I could improve the test handling regarding your suggestions: 56a86cf, b4a4869

/approve
@oliver-goetz Do you also want to have another look?


fix syntax in documentation comments

Documentation comments (those starting with /**) are interpreted as Markdown by modern IDEs.

This PR is escaping the method names with underscores for proper display support.

Thanks for your contribution.
The comments are made for gtkdoc, for automatic man page and HTML generation.
I am sorry to say that your PR breaks the HTML generation. You can try with
./configure --enable-gtk-doc --enable-gtk-doc-html
make
xdg-open docs/libpsl/html/index.html

What modern IDEs are you referring to? Are there possibly plugins for gtkdoc comment style available?

Thank you, I'll investigate that gtkdoc. although for now I'm hitting:

zsh: no such file or directory: ./configure

The screenshot is from Xcode.

Thank you, I'll investigate that gtkdoc. although for now I'm hitting:

zsh: no such file or directory: ./configure

The screenshot is from Xcode.

Are you using the instructions for building from a tarball, but downloading with git instead? Please follow the "building from git" instructions. :)

Also note that the autotools project files only produce Makefiles. You can build with meson instead, if you prefer, which has an option to actually generate xcode solutions.

configure: error: You requested libidn2|libidn but libunistring is not installed.
➜  libpsl git:(rockdaboot) brew install libunistring
Warning: libunistring 1.1 is already installed and up-to-date.

Is there possibly a development package for libunistring? Or does the already installed package have all C header files?


chore: add discord link

added discord link of Askdevs:
https://discord.com/invite/VgFMfMJn

Thanks @jayk-gupta for the PR.
I was thinking of adding link with image in a separate section. Example and code below. Can you please update. Place it after demo section.
👋 Join Us On Discord

Code
## 👋 Join Us On Discord

Thanks @jayk-gupta for the PR.
I was thinking of adding link with image in a separate section. Example and code below. Can you please update. Place it after demo section.
👋 Join Us On Discord

Code
## 👋 Join Us On Discord

Updated the discord task.
I don't know whether the json file was edited or not, I have updated that too.
Thanks for the help!

@jayk-gupta are you working on the comments? We need discord link change soon.

@jayk-gupta are you working on the comments? We need discord link change soon.

Comments on PRs? What do I need to do exactly? I can work on this.

Yes. The PR comments. I would suggest revert data/users file and send it separately.

What to send seperately? My PR of adding my info,right?

What to send seperately? My PR of adding my info,right?
Yes. Just add discord link in this PR. (README file change)

What to send seperately? My PR of adding my info,right?
Yes. Just add discord link in this PR. (README file change)

okay


need to export config file （backup）

Description of the new feature / enhancement
so many features need to setup up when install clean OS make my mind broken
export the config file to setup again easily
Scenario when this would be used?
every time need to backup or setup again
Supporting information
No response

/dup #586
You can find the config files under %USERPROFILE%\AppData\Local\Microsoft\PowerToys


"SQLTools Settings" tab is blank

Describe the bug
"SQLTools Settings" tab is blank

To Reproduce
Steps to reproduce the behavior:

Use the command "SQLTools Management: Add New Connection"
An "SQLTools Settings" tab opens, but it is blank. (No matter how long I wait and how many times I restart VSCode)

Expected behavior
There should be some settings in the "SQLTools Settings" tab.
Screenshots

Desktop (please complete the following information):

SQLTools Version 0.22.10
VSCode Version: 1.47.2
OS: windows 10
Driver:

[ ] PostgreSQL/Redshift
[ ] MySQL/MariaDB
[ ] MSSQL/Azure
[*] SQLite
[ ] Other? Which...

Database version: [e.g. MySQL v5.6, PostgreSQL vX...]

Additional context
Add any other context about the problem here.

Having same issue, with all the same versions listed above

Same here

Also having same issue.

Fixed on v0.22.11.


parameter change

Thankyou very much for the library.
in version 6 4 0 2419 of DynamicData

the TotalSize property corresponds with Page and vice versa
a change from 6 1 3 2378

Also perhaps worth mentioning in the readme that IChangeSet needs to be cast to IPageChangeSet for use in pagination (again a change from the version in Dynamic.Trader)

In the first image, you are using an observable list because no key is specified whereas the example in the second example from Dynamic Trader it is using an observable cache.
The error is that the list overload of page. I will fix the API (both issues) when I get a chance.

i thought i was missing something. Thanks for the info.


Serialization of int as double?

I have a basic POJO with a property of type int.
...
@Json(name = "partner_id") int partner_id = 1;
...

After serialization it logs as:
...
"partner_id":1.0
...

How can I prevent conversion to double and send partner_id as int, the way it's modeled?

It seems an implementation issue of the ResourceJsonAdapter. When serialize a Resource object, the adapter first convert attribute object into a JSON string. Then, the JSON is converted into a LinkedHashMap by the default Object converter implementation of moshi in which the value 1 is converted into 1.0 (a double value).
Newly implemented ResourceJsonAdapter eliminates the conversion and should work as expected.
Thanks for your help and the new release 1.0.13 is available on bintray.

Excellent work! Many thanks for the fix.


allow to set sideEffects from rules

What kind of change does this PR introduce?
feature

Did you add tests for your changes?
yes

If relevant, link to documentation update:
TODO

Summary
fixed #6065

Does this PR introduce a breaking change?
no

Other information

Wow. Segfault?! How is it even possible?