WIP Discriminator

Summary
Ignis does not have a discriminator framework to go from measurement level 1 to measurement level 2.
Details and comments
This WIP PR introduce a way we might implement a discriminator in ignis. Main contributions are:

AbstractDiscriminator showing what functions a use must implement.
LinearIQDiscriminator showcasing how a discriminator may be implemented.

In keeping with the style of the other parts of ignis, there should be a circuit method (which generates the calibration circuits), a Fitter class (which takes in the calibration) and a Filter class (this is the actual discriminator). The Filter will take a result and return a new result (which has been discriminated).

I think the folder structure should be ignis/measurement/discriminator/ vs just ignis/discriminator, but I'm open to suggestions

@dcmckayibm Thanks for looking into this. I agree with all points. Regarding the calibration we should keep Pulse in mind so that users can specify their own pulse schedules usable for calibration.

@dcmckayibm Regarding the fitter: should this inherit from characterization.fitters.BaseFitter? If so we may want to move BaseFitter to another location higher up the folder hierarchy.

@taalexander @dcmckayibm What does the ExperimentResultData data structure look like for meas_level=1 and meas_return='single'? Would the data be stored in the memory instance?

@dcmckayibm @taalexander Please comment on the most recent commit which implements a functional LinearIQDiscriminator inherited from BaseFitter. If you think this structure is fine I'll go ahead and do the following:

generalize LinearIQDiscriminator(BaseFitter) to IQDiscriminator(BaseFitter)
implement LinearIQDiscriminatorIQDiscriminator)
implement QuadraticQDiscriminatorIQDiscriminator)
implement the Filter class which takes a Result and returns a Result

One question:

do you think this needs more sanity checks on the input to __init__ of LinearIQDiscriminator? E.g. checking that circuit_names and expected_states have the same length?

@dcmckayibm Thanks for looking into this. I agree with all points. Regarding the calibration we should keep Pulse in mind so that users can specify their own pulse schedules usable for calibration.

Good point, but I think you want to provide the qasm circuits (which can be converted to the users desired pulses with qasm->pulse).

Structure looks good to me

I've found it useful with the other ignis PRs to make a notebook in /examples/ that can serve the dual purpose of testing and as the eventual tutorial

@dcmckayibm and @taalexander do you guys want to have a look this? I'd like a bit of feedback/ideas how to improve it. I think we can then merge. I'm also working on a NB that uses the hardware (instead of making data up out of thin Aer) but am running into a snag with the meas_level the backend returns (see qiskit-pulse slack channel).

@taalexander Thanks for reviewing. I'm not sold on the idea of testing for a complete basis. This may restrict the user too much. Consider the following cases:

user is working in |1> to |2> space.
user is working in the |0> to |2> space.
user is working with qutrits or something else.
Perhaps the experiment only requires discriminating |01> from |10>.
etc...

Ok those, are all good reasons 😄. How do you think we should handle the input reference states? An integer representation only works for |0>, |1> space but this is the typical computational basis.

From the discriminator's point of view str or int is kind of irrelevant. When I look at get_memory(...) I see that for meas level 2 it returns List[str] ergo it would make sense to restrict the discriminator's reference state format to str to be consistent. If a user really wants to work with int or something else nothing would prevent him from giving the discriminator a str matching his int or his something else.

Ok those, are all good reasons 😄. How do you think we should handle the input reference state format? An integer representation only works for |0>, |1> space but this is the typical computational basis. If an integer is provided assume a qubit representation, or only accept strings? Also, it is important to remember that the returned information in get_memory is not necessarily the qubits, but the memory slots. How should we handle this mapping?

I think we have to assume the calibrations are mapped the same as data to be discriminated

From the discriminator's point of view str or int is kind of irrelevant. When I look at get_memory(...) I see that for meas level 2 it returns List[str] ergo it would make sense to restrict the discriminator's reference state format to str to be consistent. If a user really wants to work with int or something else nothing would prevent him from giving the discriminator a str matching his int or his something else.

I think the output should always be 0 or 1 but what those mean can be gamed by the calibration. Using Daniels example of |01> and |10> you give a |01> calibration with a label "0" and a |10> calibration with a label "1" and that these involve both qubits and the discriminator will push this outcome to memory slot 0.
Modifying slightly what I said above, the discriminated data should get pushed to the memory slots in the order the discriminators are added.

I would focus on 3 cases in the example notebook to see if they make sense (with 2Q)

Provided with the IQ data from preparing the |00> and |11>  discriminate Q0 into 0/1 into slot 0 and Q1 into 0/1 into slot 1

Provided with the IQ data from preparing |00>, |01>, |10>, |11> discriminate Q0 into 0/1 into slot 0 and Q1 into 0/1 into slot 1

Provided with the IQ data from preparing |00> and |11>, discriminate into 0/1 into slot 0

After playing around with some data from Tokyo and the discriminator that is already implemented I think I will add an option to perform rescaling of the IQ data. For each qubit I suggest taking all the data over all the calibration schedules (i.e. irrespective of weather it is in 0, 1 or whatever) and removing the mean and normalizing to the std. I saw that this helps the LDA a bit.

To be consistent the discriminator module should be renamed discrimination.

The new test folders were missing __init__ so I added and that opened up some new linter errors, could you fix?

Looks good, can we move the plotting code to the fitter class? For example see https://github.com/Qiskit/qiskit-ignis/blob/e9247de37801f11ad5bc0a0f129b59573f567da2/qiskit/ignis/mitigation/measurement/fitters.py#L203.
Can we also add a schedules/circuits.py which generates the schedule. Right now that code is in the example notebook. Following the example of https://github.com/Qiskit/qiskit-ignis/blob/master/qiskit/ignis/mitigation/measurement/circuits.py.
The function result_subset should also be wrapped by the fitter/discriminator class. So you give the discriminator the full result and the list of calibration circuits and it just uses those for the discriminator.

Even better for the schedule generation would be to modify the existing measurement calibration circuit code and give an option to return the schedule.


[FLINK-33638] Support variable-length data generation for variable-length data types

What is the purpose of the change
Extending datagen to generate variable length values (using a new option to enable it, e.g.,'fields.f0.var-len'='true').
Brief change log

introduce random length for variable-length types in RandomGeneratorVisitor

Verifying this change
org.apache.flink.table.factories.DataGenTableSourceFactoryTest#testVariableLengthDataGeneration
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? yes

@lincoln-lil Looking forward your review, thanks :)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@lincoln-lil I have updated based on your comments, now CI succeed :)

@liyubin117 Could you also synchronize the latest changes to the documentation to the Chinese version?

@flinkbot run azure


Gmail login (not imap login) throws this error

Error
Traceback (most recent call last):
  File "/sec/root/email-bot/main.py", line 72, in <module>
    with MailboxClass(m["server"], port=m["port"]).login(m["login"], m["password"]) as mailbox:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/imap_tools/mailbox.py", line 300, in init
    super().init()
  File "/usr/local/lib/python3.11/dist-packages/imap_tools/mailbox.py", line 32, in init
    self.client = self._get_mailbox_client()
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/imap_tools/mailbox.py", line 306, in _get_mailbox_client
    return imaplib.IMAP4_SSL(
           ^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/imaplib.py", line 1323, in init
    IMAP4.init(self, host, port, timeout)
  File "/usr/lib/python3.11/imaplib.py", line 202, in init
    self.open(host, port, timeout)
  File "/usr/lib/python3.11/imaplib.py", line 1336, in open
    IMAP4.open(self, host, port, timeout)
  File "/usr/lib/python3.11/imaplib.py", line 312, in open
    self.sock = self._create_socket(timeout)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/imaplib.py", line 1326, in _create_socket
    sock = IMAP4._create_socket(self, timeout)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/imaplib.py", line 302, in _create_socket
    return socket.create_connection(address)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/socket.py", line 827, in create_connection
    for res in getaddrinfo(host, port, 0, SOCK_STREAM):
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/socket.py", line 962, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
socket.gaierror: [Errno -2] Name or service not known

Can you please send the value of "mailbox.server" (from config.json)?

"server": "imap.google.com",
"port": 143,

imap.google.com is not a real domain. You probably meant imap.gmail.com. The IMAP port for imap.gmail.com is 993.
https://developers.google.com/gmail/imap/imap-smtp

Now, I get it. Though port 143 is default, google uses port 995
Thank you for your time. I'll try with this one and let you know the results.

Getting same error.
I think Gmail login is faulty. Have to try with IMAP credentials

I couldn't run the 2nd command to activate env
Ran this with IMAP. Got this error.

This error means that the requirements were not properly installed. Running pip install without venv is unlikely to succeed in modern Linux distributions.
If you use debian or ubuntu, you could try installing the python3-imap-tools package with apt.

The server hostname is wrong.

imap.google.com

You probably meant imap.gmail.com

Yay, it works! 🥳
Another query
If I want to increase the text characters, should I replace here?
2000 instead of 1000
{msg.text[:1000]}

I think emails with 2000 chars or more don't get forwarded to the discord channel because of this:
def idle(mailbox, callback):
    while True:
        logger.debug("idling")
        responses = mailbox.idle.wait(timeout=60*5)
        if responses:
            logger.debug("idle returned responses")
            callback()

But I want these emails to be forwarded, too.

Gmail login doesn't work
Traceback (most recent call last):
  File "/home/anonxpro/Downloads/email-bot/main.py", line 72, in <module>
    with MailboxClass(m["server"], port=m["port"]).login(m["login"], m["password"]) as mailbox:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/anonxpro/Downloads/email-bot/.venv/lib/python3.11/site-packages/imap_tools/mailbox.py", line 62, in login
    check_command_status(login_result, MailboxLoginError)
  File "/home/anonxpro/Downloads/email-bot/.venv/lib/python3.11/site-packages/imap_tools/utils.py", line 46, in check_command_status
    raise exception(command_result=command_result, expected=expected)
imap_tools.errors.MailboxLoginError: Response status "OK" expected, but "NO" received. Data: [b'[ALERT] Application-specific password required: https://support.google.com/accounts/answer/185833 (Failure)']

The error comes from GMail and it explains what you need to do to enable IMAP access to Google accounts

Application-specific password required: https://support.google.com/accounts/answer/185833

Please read the link from the error.

I have enabled it but still doesn't work.
Traceback (most recent call last):
  File "/home/anonxpro/Downloads/email-bot/main.py", line 77, in <module>
    with MailboxClass(m["server"], port=m["port"]).login(m["login"], m["password"]) as mailbox:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/anonxpro/Downloads/email-bot/.venv/lib/python3.11/site-packages/imap_tools/mailbox.py", line 62, in login
    check_command_status(login_result, MailboxLoginError)
  File "/home/anonxpro/Downloads/email-bot/.venv/lib/python3.11/site-packages/imap_tools/utils.py", line 46, in check_command_status
    raise exception(command_result=command_result, expected=expected)
imap_tools.errors.MailboxLoginError: Response status "OK" expected, but "NO" received. Data: [b'[AUTHENTICATIONFAILED] Invalid credentials (Failure)']

The Gmail app password is good for now. If you have free time, you can try fixing this.

Isn't it ironic?
This script uses a single IMAP account. The recipient address is also the same as the mail address. It can't be two.
And I can't filter email address of From field within the script.

The error comes from GMail and it explains what you need to do to enable IMAP access to Google accounts

Is it working on your end?

Yes, my gmail account with an app password seems to work ok with this script.

Yes, my gmail account with an app password seems to work ok with this script.

My bad, then. I thought you were implementing a feature called "Gmail account without app password"
And I confirm your script works flawlessly.


quill:0.3.0

I am submitting

[x] an update for a package

This release contains a complete rewrite of the layout algorithm, finally allowing for transparent gates. Also, it is now possible to place gates and co. explicitly, similar to table.cell(). This allows for more flexibility in the calling code and enables batch-production of gates or labels. Some minor improvements are included as well, in particular, equations numbers are now suppressed within the circuit, even for block-style equations.

Thank you!


Refactor to ApprovalUiQuery UC with Presenter

No new logic, tests still missing, but these can now be added as next step
Manually verified

Hmm, presenter does not add value. I will refactor to a return arg

I started work on a test, but again, this PR does not need to be blocked on that. Would be nice if https://github.com/ProfessionalWiki/PageApprovals/pull/44 can be merged to help with the new tests.


Update upgrader script to upgrade binaries only when there is a version diff

Issue #, if available:
Upgrader pod uses a upgrade.sh script that upgrades all the core kubernetes binaries with the binaries that are already baked into the pod. At present, we upgrade the binaries every time the script is invoked to upgrade a particular component. This change adds logic to the script to check if there is any version diff between the existing binary on the host and the binary baked into the pod before proceeding with the upgrade.
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

yeah def use checksums and not versions
Now, do we need this now?
My main concern atomicity. Skipping the upgrade logic for kubeadm if the existing binary matches the new binary is dangerous, the two binaries being the same doesn't guarantee that the upgrade process succeed before. The container can be restarted mid process: after the binary has been copied but before the upgrade logic is run.
IMO the risk here is not worth the optimization benefits. We should write the upgrade logic in a way that is idempotent, so it can be run multiple times. If you want to be able to skip it, we need something smarter than just checking the version/checksum. We need to check the results of the actual upgrade, which involve knowing things about the kubeadm internals, etc. Or we use a "commit" strategy that records the success of each upgrade step for a particular upgrader version.

That's a very valid point. Yeah we need a mechanism to actually verify the upgrade had happened. We were looking to see if we could optimize the upgrade logic but looks like it's more involved than what it appeared to me in the first glance. I will hold this PR for now and discuss on how and if we want to do this at all.

/hold

/close


There should be a possibility to set the server should fail to start when modrinth mods cannot get resolved

Enhancement Type
Improve an existing feature
Describe the enhancement
Since it is possible, that modrinth mods can be taken down (either by the owner, or the modrinth team), it should be possible to specify an environment variable to stop the server from starting if mod resolution using modrinth did fail for any mod.
If a server is run with an essential mod, that would otherwise cause data inconsistencies if not installed, this could cause alot of trouble and not running the server might be the solution. In my case this was an inventory synchronizing mod, that has not the possibility to merge data on startup and not starting the server would have helped.
I know that there is the REMOVE_OLD_MODS flag, which would also prevent this behaviour, but having a clean setup while trying to set up the server is kind of essential to me, so I always have this setting set to true.
A suggestion for the env could be MODRINTH_FAIL_ON_MOD_NOT_FOUND which defaults to the current behaviour false.

Oh, this might be a bug then. My intention was to fail-fast if any mod doesn't resolve. Can you point me to an example of this? Or is it hypothetical at this point?

Yeah youre right, if the project does not exist, the server will not start. I ment the case, that the mod does not exist in the given scope of the MODRINTH_ALLOWED_VERSION_TYPE, so there is no mod being downloaded from modrinth, altho the mod project exists on modrinth. For example:
    VERSION: 1.19.2
    MODRINTH_PROJECTS: fabricproxy-lite
    MODRINTH_ALLOWED_VERSION_TYPE: release

will result in these mods:
test    | [17:19:39] [main/INFO]: Loading 3 mods:
test    |       - fabricloader 0.14.12
test    |       - java 17
test    |       - minecraft 1.19.2

(FabricProxy-Lite as of today is only available in beta for 1.19.2, see https://modrinth.com/mod/fabricproxy-lite)

Thanks for the details. Yeah, I'm still thinking it's a bug that it doesn't fail when doing that final resolution.

This should now be fixed by #2128


request for gdal_trace_outline.cc

First thanks for this repository. This is not a bug report but a new feature request. There was some discussion on supporting this capability in released gdal tools which I would love to see. However I would like to see two more options.
(1) an option for a field to be created (e.g. filename) with the image's name stashed in the file. This helps later with merging into one file.
(2) To help with merging, if the shapefile already exists can the new polygon be appended to the file.
thanks,
Trent

Trent,
It pains me to say, since I usually try to maintain my utilities, but at the moment I'm stretched too thin.  I'll leave the issue open though, in case I have some down time in the future.

Dan


refactor: field to constructor injection in Image Service App

Description

Related Issue

Fixes #(issue)
Changelog

Add ...
Remove ...

Preview

@mdsage1 It looks like you didn't start the PR from the latest main.


Weird behaviour with Angular

I've been encountering a weird problem. I've written a little Angular + Material Design script that does some animations here and there. When I run that script in Firefox, Chrome and Safari, everything works. Even when I compile an Electron app with the Angular Script using electron-packager, or run it in a prebuilt provided on GitHub Releases page for Electron, it works. However, if I install electron-prebuilt from npm and run the script with it, it shows weird behaviour. For example, a dialog opens correctly, but takes really long to close, and after that, the interface remains unusable for some time, it's as if Angular is waiting and blocking interaction.
Here's a video showing the behaviour in Chrome:
https://drive.google.com/file/d/0B3S33WrzEUwMV3ladTFkc1RIYWc/view?usp=sharing
And here's one showing it in Electron using electron-prebuilt:
https://drive.google.com/file/d/0B3S33WrzEUwMZnREdHJiUE0wV1k/view?usp=sharing
The script is the exact same, it does not have bugs, and several electron-prebuilt versions were tested, all behave the exact same. OS is El Capitan, computer is MacBook Pro 13" early 15, most recent versions of Chrome, electron-prebuilt and Angular + Dependencies were used.

Hmm very weird. Only thing I can think of is that maybe disable-renderer-backgrounding is being enabled for some reason. It's the power saving feature that background chrome tabs use and it limits executions peed. Maybe try disabling it and see if that fixes the issue? e.g. https://github.com/maxogden/screencat/blob/bbde6e65ba960a34751f36eeb1ed4dc7ad6d3295/electron.js#L20

I just tried it, but it doesn't change anything, and even if it did, it would be weird that it's only needed in this prebuilt right? Something I did notice though is that this seems to be only happening on my machine, on an iMac (El Capitan too) this does not happen, and electron-prebuilt does its job. I already thought it could be related to broken npm packages, but I removed everything and installed everything again, no change.

The only difference between running your app with electron-packager vs electron-prebuilt is that prebuilt spawns the electron process from the command line, and packager runs it using a mac application. There will be some differences such as your PATH and process.cwd(). If you look at path.txt (which is autogenerated at install time) you can see exactly what electron binary is being spawned:

https://github.com/mafintosh/electron-prebuilt/blob/master/index.js
https://github.com/mafintosh/electron-prebuilt/blob/master/cli.js

If you can reproduce this by running electron yourself from the CLI then its probably some weird electron bug

I opened electron-prebuilts Electron executable using the CLI, and its the exact same behaviour. Again, if I download the prebuilt from the releases page and open it from the CLI (the exact same way), it does work. However, I noticed that if I copy the Electron.app from electron-prebuilt to somewhere else, like my Downloads folder, it does not show this behaviour, so it seems like it makes a difference in where it runs.

@maxrimue can you try one last thing:

download the prebuilt from the releases page, unzip it somewhere, and run the electron executable inside from the command line?
run the electron executable that electron-prebuilt downloaded and unzipped

im curious if theres a difference between 1 and 2 (it might be related to how i'm unzipping in electron-prebuilt)

Well, no. Both behave weird now. Another Electron.app prebuilt I downloaded and placed in /Applications works though, and zips I download always get unzipped by Safari when downloading, so it's not the zipping that's not working, as two prebuilts that behave differently were unzipped the same way.

@maxrimue Sorry for the late reply. I am running OSX 10.11.1, Electron Prebuilt 0.36.2.
Still, after working on the project several hours per day the only fix is to drag and drop the entire project folder into the electron version from the release page.

I'm having the exact issue.
with angular-material 1.0.4 and electron prebuilt v0.35.4.
running on osx 10.11.2.
I simply use this project and added a button to open a dialog on it : http://develephant.github.io/Amy/

Hi, I'm having the same issue. It's not reproducable:

in Chrome or any other recent browser.
when the app is built using electron-packager
when using electron-prebuilt on windows (win10)
when ngAnimate is removed from the app

it only occurs when using electron-prebuilt on osx (10.11.3) - it's also the same for electron-prebuilt 0.36.11 & 0.37.2.
just for info: my dep list (same for osx & win):

npm 3.7.12
node 5.9.1
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I believe this bug has been solved with electron-prebuilt 1.1.1

@lincolnwan Unfortunately, this bug is still present in<EMAIL_ADDRESS>for me.
Here is an example electron app I've put together to test whether someone is affected by this bug:
https://github.com/maxrimue/electron-prebuilt-bug

@maxrimue maybe my guess was too naive.. I used to be affected by this bug but just suddenly everything runs smooth, no mater on 1.1.1 or 1.2.1. I have no idea what did the magic that fixed it.

Tool
Version

MacOS
10.11.6

Electron
1.4.3

Node
6.4.0

I just tried out @lincolnwan's repo and was unable to reproduce the issue. See a short video here: https://cloudup.com/cc1zgZanP8V
I suspect this was fixed in a recent version of Electron or its Chromium dependency. Gonna close this out, but if anyone is still having problems, feel free to comment.

@zeke Unfortunately, I'm still having this problem. I too created a demo repo to test for this here, it uses the most current Electron and Angular 1 deps and for me still shows the weird behaviour. It doesn't however surprise me that it didn't occur for you, it always seemed like something that would only appear on a few machines.

@maxrimue so you only experience this issue when launching your app directly with electron-prebuilt but not when built with electron-packager?

For example, a dialog opens correctly, but takes really long to close, and after that, the interface remains unusable for some time, it's as if Angular is waiting and blocking interaction.

Are you able to capture a CPU profile in the dev tools and see what is happening during this lag?

@kevinsawicki Exactly. I also just went ahead and upgraded Electron and Angular1 in my aforementioned demo repository and this problem still occurs. I captured a CPU profile using the DevTools and it seems like the "gap" in which nothing happens Electron is in "idle", at least that's what it says. You can find the profile here.
On the left, you can see the stack that opens the dialog and on the right the stack that closes it, in between mostly just "idle":

+1 same problem with angular and angular material and dialog. It's fine in the build version.

The electron-prebuilt repo is being retired and its code has been moved into the electron/electron repo. For the sake of historical transparency, we will leave GitHub Issues enabled on this repository, but if you are still affected by the issue reported here, please open a new issue on electron/electron repo and reference this issue from it so people can get the full context. The electron repository has a large and active contributor community, so your issue is more likely to get the attention it deserves there. Thanks!


⚠ DesiTorrents has degraded performance

In ba064ca, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 7399 ms

Resolved: DesiTorrents performance has improved in 89887b6 after 1 hour, 9 minutes.


Add metadata editing

Added three functions for editing metadata. If necesarry tests can be added.

put_request: Make a PUT request using the rquest package, which is needed for editing
get_dataset_metadata: returns a dictionary containing the metadatablocks of a given dataset
edit_dataset_metadata: updates the dataset metadata

Additionally utilities function dict_to_json now ensures encoding by default. If there was a reason it should not, an additional boolean could be added to the function.
The documentation was written in the same style except for curl calls which was moved to own code-blocks and urls which now align with the official sphinx guide. If this style agrees with the auther, the whole documentation should be adapted.

Thanks, that's a great PR!
Will do the review in the next days, as I am in the middle of adding the metadata data models for Dataverse, Datasets and Datafiles for release 0.2.0. api.py will not be touched for this from my side, so maybe i will add your PR to it. But as this is the first time i work within a real open source development workflow, i have to do some research, who this is done best (release management, implement PRs, etc.). And need to finish the release before i leave to the Dataverse Community Conference, so I can present something nice.
Looking forward to hear and see more. :)

I have removed the get_dataset_metadata() function, because in the next release, there will be metadata manipulation functions included, which can be used to address the metadata directly. Thanks again for your contribution.


Unable to connect with Local Network while testing with real iOS Device

My app- QuickSupport connects with Local IoT Wi-Fi network and executes some SFTP Commands on the click of a button. This works fine with Simulator but when I test on Real Device using Testflight it fails and I get a pop-up message "quicksupport" would like to find and connect to devices on your local network. When I allow this and try again it works.
I tried to give permissions through my iPhone by going to Settings --> Privacy --> Local Networks but my app's name didn't appear. It appears after the crashes when the pop-up appears to Allow connection to Local Networks. Experts, How can I give permission to my app to be able to access Local networks or is there any other workaround?

I was able to resolve this issue. With iOS 14, there is a Security and privacy concern that is why it doesn't allow connection to Local network by default. But we can manually trigger it. Below is the link:
https://www.iosapplication.net/2020/12/22/local-network-in-ios-14/


Add some steps to be more explicit.

@codefoster : Thanks for your contribution! The author, @seguler, has been notified to review your proposed change.

Thank you @ktoliver! Looks great.

#sign-off


Search locks up browser

hugo version: v0.55.6
When typing into the search bar, the browser locks up and I don't see any errors in the console.

For someone as active as yourself on github I would have expected a somewhat higher quality issue post. There is zero useful information to use here.
Despite the fact this is not a support forum, let's start with the real basic diagnostics:
What site are you loading? Local server or online? Your own site or someone elses?
What browser are you using? Have you tried multiple browsers? Have you tried in private-mode? Have you disabled all extensions?
What does locks-up mean? Use full CPU? If so chrome / etc will kill the tab and give you diagnostic information inside developer tools.
You should also be aware (or could see in the code) that search is provided by a javascript file loaded from the same repository. The very same repository that supports the documentation site at https://learn.netlify.com/en/. And for me running a search in chrome and firefox is working fine.
Please do some basic troubleshooting yourself and provide something useful if you find anything.

For someone as active as yourself on github I would have expected a somewhat higher quality issue post. There is zero useful information to use here.

That's fair. FWIW, I searched all the exiting issues (open and closed) to try and resolve before giving up. I've also found it helpful when repos provide an Issue template so I know the type of info that is valuable when troubleshooting.

Despite the fact this is not a support forum

Is there somewhere else I should have looked for help?

What site are you loading? Local server or online? Your own site or someone elses?

This is an internal docs site we're working to convert to hugo. The issue is reproducible when running hugo server and when serving the static assets with nginx.

What browser are you using? Have you tried multiple browsers? Have you tried in private-mode?

The issue seems to happen only with Chrome (including incognito mode) but not with Safari. I do not have Firefox installed but will do so if it would help diagnose further.

Have you disabled all extensions?

I had not thought to try that but it did not resolve the issue.

What does locks-up mean? Use full CPU? If so chrome / etc will kill the tab and give you diagnostic information inside developer tools.

After I type at least 3 characters and pause long enough for it to run, the browser stops responding and looking at Activity Monitor shows CPU pegged at 99-100% for the "Google Chrome Helper" process. Chrome never kills the tab (which I found odd) and the developer tools also do not function once it happens so I can't really see what is happening. After several minutes, it finally gives the "Page Unresponsive, would you like to wait or exit page" prompt.
The only other thing to note that could help is the index.json file is ~3MB.
I apologize for not providing more info initially as I was very stumped after trying several things for quite some time today.

It was late in my day too so apologies on my side as well.
I haven't heard of this issue before, especially strange as it only affects a single browser. 3MB for the index.json seems on the large side. Is your site particularly big, and have you had a peek inside the json to check it looks sane (& valid json)?
At a high level, hugo generates the index.json, learn.js fetches index.json into the browser, and lunr.min.js is instantiated with the json.
At the time of writing we're only one minor version behind of lunr (https://cdnjs.com/libraries/lunr.js/2.3.5).
If I had to guess I'd say there might be something unexpected in the json file. You could verify by deleting half the content in your site and starting the local server, then doing the same with the other half, just to rule out that it might be an issue with the content.

Thanks for the help and suggestions to try and resolve this. We removed a lot of the content and search started working again. As we slowly add more things back maybe something will jump out but hopefully not.


Spawn objects from gz resource path

Basic Info

Info
Please fill out this column

Issue(s) this addresses
#324

ROS2 version tested on
Humble

Aerial platform tested on
Ignition

Description of contribution in a few bullet points

Look for objects in environment variable

Do we add our AS2 GZ Models path into de IGN_GAZEBO_RESOURCES path in the CMakelists?

Yes, we do. Indeed, we should only look into Gz resource path..

Codecov Report

Merging #325 (7dbb604) into main (aaa543c) will decrease coverage by 0.07%.
The diff coverage is n/a.

:exclamation: Current head 7dbb604 differs from pull request most recent head 1e0c14d. Consider uploading reports for the commit 1e0c14d to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff            @@
##            main    #325      +/-   ##
========================================
- Coverage   2.91%   2.85%   -0.07%
========================================
  Files        103     100       -3
  Lines       5524    5508      -16
  Branches     456     471      +15
========================================
- Hits         161     157       -4
+ Misses      5225    5197      -28
- Partials     138     154      +16

Flag
Coverage Δ

unittests
2.85% <ø> (-0.07%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 52 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


CK2 country names don't convert really well for some reason

So basically converted CK2 to EU4 and with the VIC2 save countries exclusively from CK2 (With that i mean countries that appear in EU4 work, like the byzantines) just have names like "X34" or "Y41". I have already read the FAQ but the solutions don't work.

Should be fixed by #240


fix: Incorrect locale information provided Error if accept-language is invalid

Any errors thrown by Intl.getCanonicalLocales should be catched and the default language should be returned.
I noticed this because cypress was setting the accept-language header to *.

Out in<EMAIL_ADDRESS>thanks again!


Set User-Agent header in PF Terraform provider

Looks good. Just be sure to include the ldflags in your goreleaser configuration when that is added.


Overlaying cartoons

Codecov Report

Merging #406 into master will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #406      +/-   ##
==========================================
- Coverage   89.14%   89.06%   -0.08%
==========================================
  Files          20       20
  Lines        1317     1317
==========================================
- Hits         1174     1173       -1
- Misses        143      144       +1

Flag
Coverage Δ

#unittests
89.06% <ø> (-0.08%)
:arrow_down:

Impacted Files
Coverage Δ

lineage/states/stateCommon.py
93.82% <0.00%> (-1.24%)
:arrow_down:

Continue to review full report at Codecov.

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

If this builds @jclagarde can you see if you're satisfied with the way the figures. In the future (on a new branch), ideally you can probably resize the original cartoons to fit their new positions in the figures. I think something that would be easy to do is make the text bigger, for starters, keeping the rest of the figure the same. On a new branch, the cartoons for figure 2,3,4,5 can be added to the figures 12,13,14,15.


⚠ Bofrost.de has degraded performance

In 0a21406, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 818 ms

Resolved: Bofrost.de performance has improved in 05d1e23.


Add member type to legends, subtitles, and collection logic

Resolves #382 and #898 and comment
First we added member and memberPlural as annotations for CollectionVariableTreeNode. Then we used those annotations to clarify collection filters, and add better info to the volcano visualization. One side effect was that to better position the subtitle for the OutputEntityTitle, I moved the <br> inside the component and modified the css.

oof i just had a thought that i forgot about the scatterplot. @jernestmyers dont worry about reviewing until i give you the all clear :)

Scatterplot updated :) @jernestmyers all ready for you!


ENH change plotted quantiles to (q3, q7)

This PR proposes to use (q3, q7) instead of (q1, q9) in the plots.
For some fast benchmarks, computational time can vary a lot between runs, and the log-plots are highly visually affected by some unfortunate outliers. Using quantiles closer to the median would alleviate this issue by providing a more robust estimate of the computational times.
In a Gaussian distribution with zero-mean and unit standard deviation, the range [-0.5, 0.5] corresponds to the percentiles [30.8, 69.2], so the range (q3, q7) does seem a reasonable choice to me.
WDYT?

Codecov Report

Merging #327 (81ac035) into main (60dcf6a) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #327   +/-   ##
=======================================
  Coverage   61.45%   61.45%
=======================================
  Files          41       41
  Lines        2278     2278
  Branches      418      418
=======================================
  Hits         1400     1400
  Misses        780      780
  Partials       98       98

I think that using quartile 0.25 and 0.75 would be more standard.

I agree that this would make it more robust.
I have a concurrent PR changing for 0.2 and 0.8 (see #287). I updated it to use quartile into account.
As the PR also add some new plot (iteration wise + interpolation to avoid weird loooking curves), I will close this one to avoid merge conflicts. Feel free to review theother one so we can merge it fast :)


Update from angular 11 to 12/13 error: Property 'styles' does not match the schema.

🐞 Bug report
Command

[x] build
[x] serve

Is this a regression?

 Yes, the previous version in which this bug was not present was: 11
Description
TLDR: How to build lazy styles files to a folder ?
I though about postbuild process but this doesn't solve my issue with "serve".
Previous shema.json (<12) allow bundleName to have slash.
With new pattern restriction this configuration doesn't work anymore :
"bundleName": "assets/theme/light",

Shema.json from angular-cli 11:
"bundleName": {
    "type": "string",
    "description": "The bundle name for this extra entry point."
},

Shema.json from angular-cli 12/13:
"bundleName": {
    "type": "string",
    "pattern": "^[\\w\\-.]*$", // <--
    "description": "The bundle name for this extra entry point."
},

🔬 Minimal Reproduction
Adding slash in bundleName under angular 12/13 lead to this error.
// build > options > styles
{
      "bundleName": "assets/theme/light",
      "inject": false,
      "input": "apps/my-app/src/assets/styles/theme/light.scss"
 },

🔥 Exception or Error
Property 'styles' does not match the schema.
{
  "oneOf": [
    {
      "type": "object",
      "properties": {
        "input": {
          "type": "string",
          "description": "The file to include."
        },
        "bundleName": {
          "type": "string",
          "pattern": "^[\\w\\-.]*$",
          "description": "The bundle name for this extra entry point."
        },
        "inject": {
          "type": "boolean",
          "description": "If the bundle will be referenced in the HTML file.",
          "default": true
        }
      },
      "additionalProperties": false,
      "required": [
        "input"
      ]
    },
    {
      "type": "string",
      "description": "The file to include."
    }
  ]
}'

🌍 Your Environment
Angular CLI: 13.0.2
Node: 14.18.0
Package Manager: npm 6.14.15
OS: linux x64

Angular: 12.2.13
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.1202.13
@angular-devkit/build-angular      12.2.13
@angular-devkit/build-ng-packagr   0.1002.3
@angular-devkit/core               12.2.13
@angular-devkit/schematics         12.2.13
@angular/cdk                       13.0.1
@angular/cdk-experimental          13.0.1
@angular/cli                       13.0.2
@angular/flex-layout               12.0.0-beta.35
@angular/material                  13.0.1
@schematics/angular                12.2.13
ng-packagr                         12.2.5
rxjs                               6.6.7
typescript                         4.3.5

This is one of breaking changes in version 12, were now we error when the bundleName is invalid. The bundleName is meant to be used to change the bundle filename.
More info: https://github.com/angular/angular-cli/issues/20360


Fix permissions on files that should not be executable

YAML and CSV files especially, but app/* doesn't need 'x'

@Fryguy @jrafanie @chessbyte Please review
cc @simaishi

Here's an example commit where the change occurred for one of the files dbc371d6282c0051bbc32a37755933d0f5f18c3a

@bdunne @simaishi is there a BZ for this?

@chessbyte none that I am aware of.


Ignore image_uri changes

Is your request related to a problem? Please describe.
Hi, I would like a feature to ignore_image_uri to deploy new versions of lambda function from CICD without doing changes in Terraform

This is a feature that has been requested many times: https://github.com/terraform-aws-modules/terraform-aws-lambda/issues?q=is%3Aissue+ignore+image_uri+is%3Aclosed
I would also like to second this, as it's become a bit annoying


docker-storage-setup: Fix typo and condition issues

If 'VG_SIZE' is empty then will hit a shell syntax issue, so it has better to
make sure the 'VG_SIZE' is non-null firstly. In addition, option '-L' of
lvcreate requires an argument to create a logical volume, in here, it
should make sure the 'META_SIZE' not 'META_LV_SIZE' is non-null too.
Signed-off-by: Alex Jia<EMAIL_ADDRESS>

@rhvgoyal, the VG_SIZE will be nil for the following case.
$ fdisk -l
Disk /dev/sda: 1000.2 GB,<PHONE_NUMBER>016 bytes,<PHONE_NUMBER> sectors
Units = sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 4096 bytes
I/O size (minimum/optimal): 4096 bytes / 4096 bytes
Disk label type: dos
Disk identifier: 0x0002d352
Device Boot      Start         End      Blocks   Id  System
/dev/sda1   *        2048   206768127   103383040   83  Linux
/dev/sda2       206768128   247728127    20480000   83  Linux
/dev/sda3       247728128   268208127    10240000   83  Linux
/dev/sda4       268208128<PHONE_NUMBER>   842657792    5  Extended
/dev/sda5       268212224   269236223      512000   82  Linux swap / Solaris
Disk /dev/mapper/docker-8:1-1574019-pool: 107.4 GB,<PHONE_NUMBER>00 bytes, 209715200 sectors
Units = sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 65536 bytes / 65536 bytes
$ systemctl start docker-storage-setup.service
Job for docker-storage-setup.service failed. See 'systemctl status docker-storage-setup.service' and 'journalctl -xn' for details.
$ journalctl -xn
-- Logs begin at Fri 2015-06-05 15:14:53 CST, end at Tue 2015-06-09 10:37:48 CST. --
Jun 09 10:37:13 intel-e52650-16-3.englab.nay.redhat.com systemd[1]: Unit docker-storage-setup.service entered failed state.
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com systemd[1]: Starting Docker Storage Setup...
-- Subject: Unit docker-storage-setup.service has begun with start-up
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
-- Unit docker-storage-setup.service has begun starting up.
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com docker-storage-setup[10686]: Volume group "sda1" not found
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com docker-storage-setup[10686]: Cannot process volume group sda1
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com docker-storage-setup[10686]: /usr/bin/docker-storage-setup: line 85: / 1000 + 1 : syntax error: operand expected (error token is "/ 1000 + 1 ")
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com docker-storage-setup[10686]: Volume group "docker-pool" not found
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com docker-storage-setup[10686]: Cannot process volume group docker-pool
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com systemd[1]: docker-storage-setup.service: main process exited, code=exited, status=5/NOTINSSTALLED
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com systemd[1]: Failed to start Docker Storage Setup.
-- Subject: Unit docker-storage-setup.service has failed
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
-- Unit docker-storage-setup.service has failed.
-- The result is failed.
Jun 09 10:37:48 intel-e52650-16-3.englab.nay.redhat.com systemd[1]: Unit docker-storage-setup.service entered failed state.

So there are two issues.

It is complaining about that volume group sda1 not found. How did it reach a stage where it thinks that volume group is sda1.

Secondly there is already a thin pool on the system created by docker. You need to stop docker to remove that pool.

Anyway, for the first issue, looks like you don't have any root volume group on your system, is that right.
If that's the case, then following code might create problem.
if [ -z "$VG" ]; then
VG=$ROOT_VG
VG_EXISTS=1
else
We are assuming that ROOT_VG is always there, We might have to check whether ROOT_VG is a valid value or not otherwise error out early.

Anyway, for the first issue, looks like you don't have any root volume group on your system, is that right.

Yes, you're right.

We might have to check whether ROOT_VG is a valid value or not otherwise error out early.

Definitely agree.

@rhvgoyal, I committed V2 patch based on your advise yesterday, I'm not sure if it is enough, please help review it, thanks.

bot, add author to whitelist

This PR is more than 1 year old with no update in last 1 year. Closing it for now. If issue still exists please reopen.


[Bug]: Can't debug server-side

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
App, Extension
Expected behavior
When you have an app created by Shopify/cli 3 using the rails template, you should be able to use byebug or pry to debug.
Actual behavior
Because the byebug commands never reach the backend server. If I call byebug, i can't give it any commands
Stack trace
-

Reproduction steps

Create a rails app with Shopify cli 3
call byebug or binding.pry in one of the controllers

Operating System
MacOS Montery
Shopify CLI version (check your project's package.json if you're not sure)
3.2.0
Shell
zsh
Node version (run node -v if you're not sure)
v18.5.0
What language and version are you using in your application?
ruby 3.1.2

Hi @wunksert! Thanks for raising this issue, i'm currently investigating, will get back to you with an update 👌

As a workaround while we fix it, you could run the rails server separately:

Replace the dev command in web/shopify.web.toml with something like dev = "echo done"
Run yarn/npm/pnpm run dev
The backend server won't start, so when you open the URL you should see an error similar to Error: connect ECONNREFUSED <IP_ADDRESS>:53361. Copy the port for the next step.
Open a new terminal and run SHOPIFY_API_KEY=xxx SHOPIFY_API_SECRET=xxx bin/rails server -p 53361, replacing the port and your API key and secret (you can find them in the Partners dashboard)

In the new terminal, you will be able to debug as expected. Hope it helps!

I am unable to load the app page when starting rails separately with these instructions.  It seems to stop at
Redirected to https://3398-116-255-63-53.ngrok.io/ExitIframe?embedded=1&hmac=4431de5757351a8546d23e1db2d89f1026984a5d05a4056765d1060b512cbc8f&host=cHVyZWJsYWNrc2JlYWNoLm15c2hvcGlmeS5jb20vYWRtaW4&locale=en&redirectUri=https%3A%2F%2F%2Fapi%2Fauth%3Fhost%3DcHVyZWJsYWNrc2JlYWNoLm15c2hvcGlmeS5jb20vYWRtaW4%26shop%3Dpureblacksbeach.myshopify.com&session=c8140065d638a6cd75243a5ba0a2f63bf31307c337bb5d17e9bff39973582b1b&shop=pureblacksbeach.myshopify.com&timestamp=1665314078
I am able to load my extension though.   Is there anything I can try to debug the rails app?  Thanks
Shopify CLI version: 3.16.3
node 16.17.1
ruby 3.1.2p20
Rails 7.0.4
npm  8.19.2

As a workaround while we fix it, you could run the rails server separately:

Replace the dev command in web/shopify.web.toml with something like dev = "echo done"
Run yarn/npm/pnpm run dev
The backend server won't start, so when you open the URL you should see an error similar to Error: connect ECONNREFUSED <IP_ADDRESS>:53361. Copy the port for the next step.
Open a new terminal and run SHOPIFY_API_KEY=xxx SHOPIFY_API_SECRET=xxx bin/rails server -p 53361, replacing the port and your API key and secret (you can find them in the Partners dashboard)

In the new terminal, you will be able to debug as expected. Hope it helps!

I am unable to load the app page when starting rails separately with these instructions.  It seems to stop at
Redirected to https://3398-116-255-63-53.ngrok.io/ExitIframe?embedded=1&hmac=4431de5757351a8546d23e1db2d89f1026984a5d05a4056765d1060b512cbc8f&host=cHVyZWJsYWNrc2JlYWNoLm15c2hvcGlmeS5jb20vYWRtaW4&locale=en&redirectUri=https%3A%2F%2F%2Fapi%2Fauth%3Fhost%3DcHVyZWJsYWNrc2JlYWNoLm15c2hvcGlmeS5jb20vYWRtaW4%26shop%3Dpureblacksbeach.myshopify.com&session=c8140065d638a6cd75243a5ba0a2f63bf31307c337bb5d17e9bff39973582b1b&shop=pureblacksbeach.myshopify.com&timestamp=1665314078
I am able to load my extension though.   Is there anything I can try to debug the rails app?  Thanks
Shopify CLI version: 3.16.3
node 16.17.1
ruby 3.1.2p20
Rails 7.0.4
npm  8.19.2


[#119269637] Move to cloud config

What
Move to cloud config. This PR includes:

new manifest directory structure
new, unified way to render manifests
cleanup of unnecessary properties and settings
various minor fixes to make move easier
move of cf-manifest to cloud config + manifest
update pipeline to facilitate cloud config rendering and stemcell upload (as this is now not done by BOSH anymore)
further simplify manifest where possible
adjust tests:

remove tests for things that are no longer in cf-manifest
add tests for cloud-config
add cloud-config and cf-manifest integration tests

ensure persistent data and job names are migrated
ensure interrupt-less future deploys

NOTE: THERE'S NO WAY BACK AFTER CLOUD CONFIG
if you want to go back to resource_pools and manifest only, you need to delete your BOSH (inc. database). Don't forget to run cleanup repeatedly
How to review
Do at least these 3 intependent tests:

Review commits and code
Build from scratch. All tests should pass.
Apply against existing deployment (built from master). Upgrade/migration takes about 1 hour, there is some downtime. Verify that persistent data for graphite and elasticsearch was migrated correctly. You should be able to see historical data in the deployment after migration. All tests should pass.

Who can review
Not @mtekel or @HenryTK or @saliceti

Squashed commit (see @keymon comment above), made zone name be picked up from TF outputs instead of hardcoded.

cf-deploy job failed with version miss-match for stemcell. Upload-stemcell task pushed 3232.6 to bosh:
  Started update stemcell > Uploading stemcell bosh-aws-xen-hvm-ubuntu-trusty-go_agent/3232.6 to the cloud. Done (00:00:07)

even if version 3232.4 was properly extracted from manifest and passed to the concourse pipeline. Then the deployment failed with:
Director task 13
  Started preparing deployment > Preparing deployment. Failed: Stemcell `bosh-aws-xen-hvm-ubuntu-trusty-go_agent/3232.4' doesn't exist (00:00:02)

Error 50003: Stemcell `bosh-aws-xen-hvm-ubuntu-trusty-go_agent/3232.4' doesn't exist

Reading the new schema, we could move the cloud_properties to the new vm_extensions.
With that we can define our ELBs and the security groups or AWS roles as "VM Extensions" which can be specified per job. If we do so, we can get rid of several vm_types that only exist to be able to custom those specific properties.
But I am OK to do this in a different story (if we do, we should create a story).

Reading the new schema, we could move the cloud_properties to the new vm_extensions.
With that we can define our ELBs and the security groups or AWS roles as "VM Extensions" which can be specified per job. If we do so, we can get rid of several vm_types that only exist to be able to custom those specific properties.
But I am OK to do this in a different story (if we do, we should create a story).

Sounds good, but definitely a separate story. This is already huge 😸

bosh-io-stemcell-resource check script has a bug:
{
  "source": {
    "name": "bosh-aws-xen-hvm-ubuntu-trusty-go_agent"
  },
  "version": { "ref": "3232.4" }
}

cat x.json | ./check
[
  {
    "version": "3232.6"
  }
]

Ok, that's not a bug. It seems that we misread documentation ( but it's confusing anyway ):

otherwise check returns all versions from the version specified on.

Will remove the stemcell resource & download stemcell every time. The micro stemcell file size is about 25KB. We can fix/extend stemcell resource in some other story, to let specify version of stemcell to get...

I'm deploying this to my existing dev environment. Have setup Pingdom checks with a frequency of 1min (the min, it seems) to get an idea of downtime.

My deployment from scratch succeeded and all checks passed. I'll destroy it now and re-apply on the deployment from master.

Deploy took 1h1m. Pingdom report 404s from the healthcheck app for 4m at the beginning of the deployment. I wasn't able to tell (from smoke tests or logsearch) how API operations were affected during the deploy, but I did see it update all of the jobs one-by-one except consul. Acceptance tests passed. So 👍  from me.
@combor is starting a test now.

Test reported 15mins downtime for apps but we have decided to give it a go so ... merging.


Uncaught SyntaxError: Unexpected token s

[Enter steps to reproduce below:]

...
...

Atom Version: 0.201.0
System: Microsoft Windows 7 旗舰版
Thrown From: sync-settings package, v0.2.2
Stack Trace
Uncaught SyntaxError: Unexpected token s
At file:///C:/Users/fcn0590/AppData/Local/atom/app-0.201.0/resources/app.asar/static/index.html#%7B%22locationsToOpen%22%3A%5B%7B%22pathToOpen%22%3A%22C%3A%5C%5CPython27%5C%5CLib%5C%5Csite-packages%5C%5Cpylint-1.4.3-py2.7.egg%5C%5Cpylint%22%7D%5D%2C%22bootstrapScript%22%3A%22C%3A%5C%5CUsers%5C%5Cfcn0590%5C%5CAppData%5C%5CLocal%5C%5Catom%5C%5Capp-0.201.0%5C%5Cresources%5C%5Capp.asar%5C%5Csrc%5C%5Cwindow-bootstrap.js%22%2C%22resourcePath%22%3A%22C%3A%5C%5CUsers%5C%5Cfcn0590%5C%5CAppData%5C%5CLocal%5C%5Catom%5C%5Capp-0.201.0%5C%5Cresources%5C%5Capp.asar%22%2C%22devMode%22%3Afalse%2C%22safeMode%22%3Afalse%2C%22apiPreviewMode%22%3Afalse%2C%22appVersion%22%3A%220.201.0%22%2C%22initialPaths%22%3A%5B%22C%3A%5C%5CPython27%5C%5CLib%5C%5Csite-packages%5C%5Cpylint-1.4.3-py2.7.egg%5C%5Cpylint%22%5D%7D:1

SyntaxError: Unexpected token s
  at Object.parse (native)
  at C:\Users\fcn0590\.atom\packages\sync-settings\lib\sync-settings.coffee:64:26
  at Object.proto.sendError (C:\Users\fcn0590\.atom\packages\sync-settings\node_modules\github\api\v3.0.0\index.js:32:13)
  at C:\Users\fcn0590\.atom\packages\sync-settings\node_modules\github\api\v3.0.0\gists.js:180:29
  at callCallback (C:\Users\fcn0590\.atom\packages\sync-settings\node_modules\github\index.js:743:17)
  at ClientRequest.<anonymous> (C:\Users\fcn0590\.atom\packages\sync-settings\node_modules\github\index.js:809:17)
  at emitOne (events.js:77:13)
  at ClientRequest.emit (events.js:166:7)
  at TLSSocket.socketOnEnd (_http_client.js:283:9)
  at emitNone (events.js:72:20)
  at TLSSocket.emit (events.js:163:7)
  at _stream_readable.js:891:16
  at process._tickCallback (node.js:357:13)

Commands
     -1:32 sync-settings:upload (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "atom-light-ui",
      "github-2014"
    ],
    "ignoredNames": [
      "*.pyc"
    ]
  },
  "sync-settings": {
    "personalAccessToken": "dbc5d05d6cda5b432bdc3f291bda0d05bd41f820",
    "gistId": "AtomSettings"
  }
}

Installed Packages
# User
github-2014, v3.5.0
highlight-selected, v0.9.3
language-glsl, v2.0.0
language-lua, v0.9.3
linter, v0.12.6
linter-lua, v0.1.5
linter-pylint, v0.2.2
project-manager, v1.15.7
sync-settings, v0.2.2
terminal-status, v1.6.7

# Dev
No dev packages

Related to https://github.com/Hackafe/atom-sync-settings/issues/51

Probably fixed in https://github.com/Hackafe/atom-sync-settings/commit/292b1654477341bdd344465dfee135bba3a516ee
Please reopen if still happens.

In the above error report you have shared your personalAccessToken. You must revoke it to prevent abuse.


ENH: add BSpline.insert_knot method

What does this implement/fix?

Add a BSpline.insert_knot method to parallel the module-level insert routine. The latter is a wrapper around FITPACK, and the new routine has a small cython snippet instead.
This can be further optimized, especially for multiple knots: ATM m > 1 incurs m allocations, each being linear in data. However for the first version let's keep it simple, and optimize on a user request.
To get feature-completeness, still need to port FITPACK's handling of periodic boundary conditions. this is done now.
Additional information

This PR only adds functionality, and does not remove anything Fortran, not just yet.

Core dev approved, has no backwards compat issues. Merging in. Thanks for tge reviews @j-bowhay , @h-vetinari


Do not disable GPIO interrupt while regestering MCPWM (IDFGH-14156)

Is your feature request related to a problem?
I'm always frustrated when an MCPWM capture disables my GPIO interrupts enabled earlier on the same pin.
This is more like a bug than a feature request really.
Describe the solution you'd like.
This line should be modified https://github.com/espressif/esp-idf/blob/024c6dee5f9d4d99878e71f47e42da2114ad4137/components/driver/mcpwm/mcpwm_cap.c#L293
ESP-IDF v5.1
Describe alternatives you've considered.
I see the master branch code is different:
https://github.com/espressif/esp-idf/blob/f420609c332fbd2d2f7f188c6579d046c9560e42/components/esp_driver_mcpwm/src/mcpwm_cap.c#L303
Is this issue fixed in v5.2 and so on?
Additional context.
For now, I re-enable my GPIO interrupt after I enable and start MCPWM capture. This hack requires knowledge of MCPWM's intrinsic realization.
For more info on why I need both MCPWM and GPIO ISR running on the same pin refer to https://github.com/espressif/esp-idf/issues/10327.

Hi @dizcza Master branch and release/v5.4 (also the incoming v5.4 tag release) should have resolved this issue. Please have a try.
This is a common issue that exist in many peripheral drivers, where the GPIO configurations may be overridden by the peripheral drivers. Since v5.4, we have refactored them to only touch the "input/output" enable, and signal connection.

Great!


Move prelude in top of Ace Editor

Fix #215
This PR move the prelude from the bottom of the Exercise pane to the begining of the tab editor.
We can hide the toplevel code with a button, the tab editor automatically adapts itself to the size of the toplevel code.
If the toplevel code is too big, use of the Toplevel window on the right to exercise is recommended.
A view with the prelude

A view without the prelude

Did you check that the layout is fine on a smartphone?

the last commit fixed the layout on the smartphone with one and two colunms.
this one with 2 columns and toplevel code

this one with 1 column and toplevel code

this one with 1 column and no toplevel code

this one with 2 columns and no toplevel code

My bad, this pull request does not refer to issue #215 but issue #168 .

This looks good, thanks!

I found some graphical issues in my pr, I'm working on them.
Don't merge my pr before I solved it !
Also I must add a scroll-bar to the toplevel code.

Are you done fixing the last issue of this PR @beobillly ?

It's finally done !
I add a scroll bar on the prelude toplevel code when the code is longer than ~ 8 lines
I fixed some ace graphical issues which appened when the prelude was shown.
I fixed some bugs which were occuring when the editor pane content and the toplevel code were using a scrollbar (the content pane editor wasn't resized correctly).
The prelude toplvel code is shown by default.
It works with one and two columns and with mobile interface !
with the prelude hidden

with the prelude shown

with prelude shwon and the scrolling at the end

with prelude hidden on mobile

with prelude shown on mobile

with prelude resized on mobile


Response model with default value of empty list not showing in docs

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
class User(BaseModel):
    username: str = None

class UserResponse(BaseModel):
    data: List[User] = []
    metadata: Dict

Description
I am having an issue with the docs displaying the response models sub models correctly when an empty list is the default value. I would expect the docs to display the array and the sub-model inside the array, but instead the docs just show an empty array.  If I set the default value to None, it displays as I would expect.  This happens on both redoc and swagger.
Any ideas on a workaround or on how to solve this issue?
This is how the docs appear with when an empty array is used as default value.

This is how the docs appear when setting the default value to None (This is what I want the docs to look like).

Operating System
macOS
Operating System Details
No response
FastAPI Version
0.75.0
Python Version
Python 3.9.10
Additional Context
No response

I agree that this generated example isn't ideal, but I'm not sure if there's a reason behind it or how to change it in the FastAPI source.
A workaround I've used is to override the schema_extra field of the response model to include a custom example, which comes with the disadvantage of having to fully specify every field:

from fastapi import FastAPI
from pydantic import BaseModel
from typing import List, Dict, Optional

class Item(BaseModel):
    id: Optional[str] = None

class ItemList(BaseModel):
    items: List[Item] = []
    metadata: Dict

    class Config:
        schema_extra = {
            "example": {
                "items": [Item(id="abc123")],
                "metadata": {"total_items": 1},
            }
        }

app = FastAPI()

@app.get("/")
async def endpoint(items: ItemList):
    pass

Which gives me the following in redoc:

Heres an alternative workaround which doesn't require static swagger-documentation.
Setting the default of the list field to None and utilizing a root-validator to set the field to contain an empty list ([]) when no data is provided for the field results in swagger showing the model contained in the list properly and for the API-response to return an empty list and not None
class User(BaseModel):
    username: str = None

class UserResponse(BaseModel):
    data: List[User] = None
    metadata: Dict

    @root_validator(pre=True)
    def validate_data(cls, data):
        if not data['data']:
            data['data'] = []
        return data


k8s: remove istio from elasticsearch

In order to test the hypothesis that istio
is causing some errors in pinging between pods
lets try removing it in production and staging

what do you mean by redeploy? I think this will mean a new release and new pods.


Allow unexpected_cfgs

Much like https://github.com/tokio-rs/tokio/pull/6538 I need to silence the new warnings introduced by https://github.com/rust-lang/cargo/pull/13571 about unexpected cfg condition names because the alternative is to use a build script which is undesirable and impractical as that introduces a warning about a build script syntax that isn't compatible with the MSRV.

introduces a warning about a build script syntax that isn't compatible with the MSRV.

You can use the : syntax that does work with your MSRV

Thanks Ed, I'll give that a try.

Weirdly this causes an entirely different warning. No idea why this is coming from the build script.
warning: missing documentation for the crate
 --> crates\libs\targets\build.rs:1:1
  |
1 | / fn main() {
2 | |     println!("cargo:rustc-check-cfg=cfg(windows_raw_dylib)");
3 | | }
  | |_^
  |
  = note: requested on the command line with `-W missing-docs`

So I was able to keep this for most of the crates and further refine code gen to elide code that depended on excluded features, which is great. 🎉
I still have to allow the warning for the windows-targets crate due to the unrelated https://github.com/microsoft/windows-rs/pull/3022#issuecomment-2096430603 warning that I've not yet found a workaround for.

Looks like the error is coming from the missing_docs workspace lint which is erroneously thinking the windows-targets crate lacks documentation when a build script is added. 🤷


Getting bbox from model.predict()

💡 Your Question
Hi,
I am using YOLO-NAS to perform some object detection and to compare it with YOLOv5 and YOLOv8. I am trying to get the bbox, confidence, label... of the predictions, like in the YOLOv5 or YOLOv8. How can I obtain these metrics?
Thank you,
Mikel
Versions
No response

Please check https://github.com/Deci-AI/super-gradients/issues/907 where this question has been answered

Hi @mikelmujika11 -
Thanks for opening an issue for SG. I'm gathering some feedback on SuperGradients and YOLO-NAS.
Would you be down for a quick call to chat about your experience?
If a call doesn't work for you, no worries. I've got a short survey you could fill out: https://bit.ly/sgyn-feedback.
I know you’re super busy, but your input will help us shape the direction of SuperGradients and make it as useful as possible for you.
I appreciate your time and feedback. Let me know what works for you.
Cheers,
Harpreet


chore(release): cherry pick improve engine API abstraction

Description
Cherry-pick #6871

closing in favor of revm 6.x bump


Thoughts on frames and containers

Frames
Frames are modular components that can run stand-alone or be combined into more complex applications.
Frames are more than components; they have the following framework-supported properties:

registered with DMG (local or global).
able to be dynamically discovered and loaded via KUBE.
able to be installed into an application container.
able to be developed and tested stand-alone.
able to be nested.
isolated from the main application container and other frames.
granted access control permissions to Halo and specific Spaces by the user (similar to Bots).
able to discover and communicate with other frames (e.g., broadcast selection state).

Frames have a host container that provides:

A viewport.
Information about the requested visual state (i.e., generic UX context that decouples the host from frame-specific params).
A context that provides access to the host (e.g., client, events).

Containers and Cards
There are 100s of different ways to organize information.
Apps are typically built around very narrow use cases and we may similarly have very specific frames (e.g., calendar/maps).
However, we also want to support generic frames (e.g., list, grids, tables) that are loosely coupled to data (perhaps selected by the user at runtime), and have components that can be composed (again possibly by the user) into more complex arrangements.
Therefore, to some extent, the UX framework should be aware of the capabilities of the data model. I.e., Spaces, Objects, Schema, and Queries (either by strong framework concepts or patterns).
The following is a list of generic and commonly used data-driven layouts.

Card/Form (e.g., associated with an individual ECHO object); Cards may be a pervasive pattern since they can be used in a multitude of different layouts (Grid, Kanban, Master/Detail)
List (virtualized)
Table (virtualized)
Text document
Grid
Stack
Masonry
Kanban
Master/Detail
Map
Calendar
Timelines
Technical drawing
Graphs (e.g., Force-directed, hypergraph, 100s of D3 visualizations, etc.)
3D visualization (Data visualization, AR, games, etc.)
App-specific visualizations (Gantt, Pie charts, 100s of third-party libs, etc.)

Kai contains some demos to rapidly experiment with container ideas (e.g., to explore thematic commonalities and constraints).
m kai
p storybook

Grid

Masonry

Stack

List

Kanban

Master/Detail

Ontology
[[App > Frames]] > [[Layouts > Tiles]] > Controls
Priorities

Controls
List Control

Flexibility to be a task list with checkbox and delete icon
Establishing gutters, line height, styling, editable, comfortable/compact
Informs Table control
Text Control

Tiles (mosaic)
Tasks tile
Document tile

Layout
Vertical Tile stack (part of mosaic)
Issues

Drag/reorder?
Insert/delete tile?
Collapse tile?
Locked/frozen (e.g., scrolling tiles when part of vertical stack)

Apps
Cardstack = Vertical Tile stack
Issues

Multiple per space?
Space selector?

Tiles
Mobile first (widths): adaptable based on available real-estate
Tangible / Arrangeable “Rectangular”
Associated with object (e.g., List)
Common UX properties (e.g., titles, grab handles)
Examples

Document (e.g., compact Note)
Map
Task LIST
Chess game
Table
Sketch

Layout

Arrangement (and selection: query + type) of Tiles + Chrome
Examples

Kanban
Document
Noteboard


logError does nothing with Crashlytics

The method logError does not log anything that is shown with Crashlytics. You can no longer see errors reported to Firebase Crash in Firebase, so this method is now useless.
However it would be very useful if we could use this method or a new one to report a non-fatal exception to Crashlytics as described here:
https://firebase.google.com/docs/crashlytics/customize-crash-reports

Trying to trigger Crashlytics with throwing errors: impossible as well ...

Same issue here. "Trying to trigger Crashlytics with throwing errors"

@herrskytte @StephanMeijer @tmvnk Is it happening on both platforms or just one?
@herrskytte please use the bug template as it helps gather relevant information so we don't have to ask follow up questions :)

Sorry for poor bug reports. This is valid for both platforms using v 2.0.2 of the plugin.
To recreate, call logError("anything") on either platform. Observe that no errors are shown in Firebase console under Crashlytics

@herrskytte Thanks.  I updated the bug report accordingly.
As far as resolving this issue, the cut over to Crashlytics was not as smooth as we had hoped.  The submission was a community submission and it was not something we could easily test.  With the fact that Firebase Crash was being sunsetted in a matter of days it was merged since FB Crash was not going to.
There are no full time developers, only community developers on the project so if someone who is affected can investigate, that would be the fastests way to resolve the issue.  There is some information in the Contribution Guidelines that talks about where the code lives to help as a starting point.
If it flat out isn't working, hopefully it shouldn't be too hard to diagnose (JS call isn't calling native code [doubt this but should confirm], native code isn't calling the proper Crashlytics method [should be very easy to find in the code]).

Anyone able to get error logging / stack trace to work with this plugin?

It works for android for me. logError in android has Crashlytics.logException(e) implemented and that works . As far as IOS is considered crashalytics has CLSNSLog which might not be supported with the current Fabric SDK
https://docs.fabric.io/apple/crashlytics/logged-errors.html
So implementing it should solve the problem.

Just attaching a screenshot of android error which was acheived

@rahulecomp
The logError function doesn't seem to be documented. Do you have an example of how to use it? Do you just implement a GlobalErrorHandler provider and send the error into logError?
Thanks!

logError is missing in documentation


Verify status of JBS issues in ProblemList_openjdk11.txt file

We would like to verify whether we still need exclude statements that relate to issues tracked in the JBS (Java Bug System) issues.  What this involves is to look up the JBS issues (the URLs containing https://bugs.openjdk.java.net), see if their Status indicates the issue is resolved.

If the Status is resolved, then we need to check if we can run the test on the platforms it is excluded on and pass.  This will be done by running a Jenkins test job called a Grinder.  If you are assigned to this issue, you will be given access to run Grinder jobs.  If the testcase passes consistently, we can remove the exclude line from the ProblemList file.  Information regarding running a Grinder can be found here: https://github.com/adoptium/aqa-tests/wiki/How-to-Run-a-Grinder-Build-on-Jenkins#common-scenario-rerun-an-openjdk-regression-test-that-was-failing
ProblemList files are typically in the following format:
path/of/test/class/to/exclude URLtoIssueReportingFailure platformsExcludedOn
For example:
java/beans/PropertyEditor/6380849/TestPropertyEditor.java https://bugs.openjdk.java.net/browse/JDK-8173082 macosx-all
For this example, the Grinder job would take parameters:
BUILD_LIST=openjdk
TARGET=jdk_custom
CUSTOM_TARGET=java/beans/PropertyEditor/6380849/TestPropertyEditor.java
PLATFORM=platformThatMapsToTheOnesFromTheExcludeLine
JDK_VERSION=11
JDK_IMPL=hotspot

First Header
Second Header

Content Cell
Content Cell

Content Cell
Content Cell

Grinder link for java/beans/PropertyEditor/TestFontClass.java retest:
https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1908/
See the parameters that were set for Grinder/1908:
https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1908/rebuild/parameterized
You can change the CUSTOM_TARGET value to be security/infra/java/security/cert/CertPathValidator/certification/LetsEncryptCA.java and rerun for a check of how that test behaves.  You can also change PLATFORM=x86-64_linux or some other value in the PLATFORM list, since the exclude line on line 162 says that the test is excluded for all platforms.

security/infra/java/security/cert/CertPathValidator/certification/LetsEncryptCA.java

@smlambert I don't have the rights to edit CUSTOM_TARGET value to anything or re-run in Grinder. Whenever I try I get this message I get
Access Denied
julian55455 is missing the Job/Build permission

You have not accepted the invitation to the outreachy_f2021 github team, once you do, you will have access.  Please check your email or https://github.com/notifications to accept the invite.

Un-assigning and moving into canosp project list, as it relates to the "Disabled tests pipeline" work.

Closing as this can now be covered by automation through Test Reenablement pipeline, https://github.com/adoptium/aqa-tests/issues/1853.


Support transformer from the current package

This issue was originally filed by @zoechi

I want to use the smoke package in my (client+server) application my_app.
I created the necessary transformer in my_app/lib/client/transformer.dart.
My pubspec.yaml contains
transformers:

polymer:
    entry_points:
    - web/index.html
my_app/client/transformer

Which results in
/home/user/darteditor/dart-sdk/bin/pub serve web --port=39553 --admin-port=38994
Error on line 55, column 3 of pubspec.yaml: "my_app" is not a dependency.

my_app/client/transformer
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Pub Serve terminated

This issue has been moved to dart-lang/pub#1157.


About install gulp problem

npm install --save-dev gulp
the node_modules folder has 300 folders，but I don't need so many modules，Is this normal？
node version 6.3
gulp version 3.9.1

This is completely normal!
You are using node 6 with npm 3. Gulp has dependencies to other modules, and these dependencies have other dependencies and so on...
npm 3 flattens that tree as much as possible in node_modules, this is why you see so much modules!


NVAccess.NVDA 2020.4 classifies as update for newer version of package

Originally posted in https://github.com/microsoft/winget-cli/issues/976#issuecomment-849361409
I have the following issue with the NVAccess.NVDA package. I have version 'alpha-22934,e05715d3' installed whereas winget tries to upgrade (or rather downgrade) to version 2020.4, which is the most recent stable version available in the winget repostiroy.
When I looked at how winget determines the installed version, I noticed that DisplayVersion in Add/Remove programs is used. It seems that the Version, VersionMajor and VersionMinor fields in the uninstall registry key are ignored. Also what might make it more difficult for winget is that NVDA has a 4 part version scheme for its full version, e.g. 2<IP_ADDRESS>34 for the version with display version alpha-22934,e05715d3.
Interesting enough though, A version string that starts with the word 'Alpha' which is non-numeric, shouldn't that be treated as newer than a version string that starts with a number (e.g. 2020)?
See also https://github.com/nvaccess/nvda/issues/12469

@leonardder can you share output from winget --info and winget list nvda? I'd like to make sure which version of the Windows Package Manager is installed, and how the "NVAccess.NVDA" package is displayed.

Thanks for your quick reply.
The relevant stuff from winget --info:
Windows Package Manager v1.0.11451
Windows: Windows.Desktop v10.0.19043.1081
Package: Microsoft.DesktopAppInstaller v1.11.11451.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Output from winget list nvda:
Name Id            Version              Available Source
--------------------------------------------------------
NVDA NVAccess.NVDA alpha-23168,b1a52939 2020.4    winget

That is an interesting version number "alpha-23168,bla52939" 😕
Do you know if the "alpha" is a different release channel?
If it is, do you know if it can be installed side by side with the stable version?
We don't support #147 release channels yet.

Versions are assumed to be number [extra string part] (dot number [extra string part])* as this is the most popular format.  What this means for the way that we parse is that “alpha-23168,b1a52939” is all version 0 with [extra string part] being the whole thing.  So the official version, which is 2020.4 will have its first version number be 2020 which is > 0.
The reason that we use DisplayVersion if it is available is due to multiple factors, but one is simply that this is the string that people will see and thus the one they are likely to put into the manifest.  It also means that the Add/Remove Programs UI and our display number match.
There has been a desire to add "hidden" data to manifests for correlative purposes, and I think if we were to do that for Name and Publisher, Version might be a good idea too.  Along with that I think that we could probably make the numeric versions the ones used for comparison.
In addition, channel tracking would probably aid you here, but without the version work being done as well it would be hard to actually compare "alpha-23168,b1a52939" to some theoretical future "alpha-14821,934745fa".

That is an interesting version number "alpha-23168,bla52939" 😕

Agreed. It is build from -,. It is, however, the user visible version.
As of writing, the current version is alpha-23168,b1a52939. The year.major.minor.build version string of this version is 2<IP_ADDRESS>68. In theory, it could be sufficient to write the 2<IP_ADDRESS>68 string to the DisplayVersion field in the registry, but I'd rather have Winget use something else that's less presentational.

Do you know if the "alpha" is a different release channel?

Yes. We have alpha, beta and stable channels.

If it is, do you know if it can be installed side by side with the stable version?

Nope, it can't.

We will be separating the "actual" version as reported in Add / Remove Programs #1073 into a different area in the v1.1 manifest schema so the "PackageVersion" field could be something different in the future. We've seen at least one other package with "Marketing" versions" different from "Add / Remove Programs" versions.
Our current best case scenario would be semantic versioning as it's deterministic for identifying the order of releases for a given package.
I think your suggestion would be the next best option.
For the release channels what would happen if you tried to install a stable on an alpha or vice versa?
Until we have support for channels in a single "package", maybe there should be a separate alpha package for NVDA.

For the release channels what would happen if you tried to install a stable on an alpha or vice versa?

Alpha replaces stable and stable replaces alpha. NVDA compares the installing copy against the installed copy based on the file version of the main executable. If the version to be installed is older, a downgrade warning is displayed which you can dismiss.
Honestly, I'm not intending to add/maintain manifests for Alpha versions, as they sometimes appear daily and their builds are dismissed by Appveyor in a month or six.

As it stands now, the authors from NVDA aren't willing to change their display version. It is possible to expand on other registry fields though, so as long as we could somehow tell winget to use the product version instead of the display version, that could improve things.

@denelon Given recent updates in winget, would it help to do the following?

Add the ProductVersion to the ARP entry in the registry
To the NVDA manifest, add something like this:AppsAndFeaturesEntries:
- DisplayName: NVDA
ProductVersion: 2<IP_ADDRESS>844

note that DisplayVersion and the PackageVersion in the manifest will still have the form of `alpha-22934,e05715d3`. Will Winget use the ProductVersion or the DisplayVersion for comparison in this case? In other words, will a current entry in ARP with a higher ProductVersion be ignored for updates?

@leonardder we're still working on the service side components, but they will enable matching for upgrade on the "AppsAndFeatures" "displayValue" and rendering the "packageVersion" to the user in command line output.

@denelon, thanks for your quick reply. Turns out I was wrong in my assumption that add/remove programs contains something like a ProductVersion string though.
Let me reformulate my question properly this time. Assuming the people at NVDA insist on keeping DisplayVersion set to the format outlined above (e.g. alpha-25875,d6a193ae, which is considered a more readable form than its file version 2<IP_ADDRESS>875. This means that winget considers version 2022.1 newer than this particular Alpha version even though it's older. Could we avoid this by defining AppsAndFeaturesEntries properly in the manifest for every release version if we add a Version dword to the registry? Will that Version dword in the manifest be compared against the current Version field in the registry? Or will changing the DisplayVersion format be our only option?

This is complicated to discuss, so some ground rules:

DisplayVersion will be the value that winget reads from the registry (which may come from DisplayVersion or the other locations that we read from).
PackageVersion will be the value that is defined in the manifest

The new feature attempts to map the detected DisplayVersion to a known PackageVersion, and then use the for all future considerations (output and/or comparison for upgrade, etc.).  If the mapping is strongly ordered, which I am using here to mean that the ordering of PackageVersion as winget defines it matches with the ordering of the mapped DisplayVersion, then winget will also map unknown DisplayVersion values into ranges between the known PackageVersion values.
Given that hashes are being used, I doubt you will ever get a strong ordering.  So the best you can do is have every released version defined so that there is no ambiguous mapping.  Even if you don't have every version, any one that we know of will start using the PackageVersion and be able to be compared against the incoming available versions.  If you only installed from winget, you should be on the golden path.
Alternately, I think that there are code changes that we could make, although I don't love any of the options that I have thought of.  Channels could be helpful too, but they don't solve the underlying problem.
First option would be to add custom version parsing and comparison rules.  The easiest way would be to add incremental quirking rules, like "Use the NVDA version format" or something.  Easiest in the short term, but opens the door to everyone wanting their version format quirked.  We also do not have a good story around package level data...
Second option would be to change the code to look at the various ways in which we attempt to form a comparable version string, and choose "the best" one.  Currently we take the first one that we find, as can be seen in the code.  We could look at all of them and decide which has the most number of version parts, using it instead.  I think that has a potential for strange behavior though, and would prefer not to use it (it also potentially impacts other packages that are working fine now).
https://github.com/microsoft/winget-cli/blob/8abc3252b4cf9109e370eff2bd9528e12b4e03ed/src/AppInstallerRepositoryCore/Microsoft/ARPHelper.cpp#L114

I guess the short story is that NVDA will need to change anyway. 😉 Thanks for elaborating!

Note: We've released Windows Package Manager 1.3, and support for version mapping against the "displayVersion" in the registry can happen while still allowing the "packageVersion" to be more of a "marketing" type version.

Thanks for pointing out @denelon. We eventually settled on saving the display version as year.major.minor.build in the DisplayVersion string in the registry, so that fixes the version comparison for us.


DOC-5969 update sslmode for npgsql

Addresses: DOC-5969
Hi Eric! This is the hello-world-csharp side of https://github.com/cockroachdb/docs/pull/16101, spawned by Artem's DOC-5969. Please reach out to Artem if his suggestion discounts context unknown!
Or let me know if there's more to be done in hello-world-csharp to accommodate this. Thanks much for the help!

LGTM, I just tested with those two changes and it works.

Thanks both!! I'll get this live.


Setup Endpoints for Geospatial Data

Overview
In order to do geospatial data analysis we need the PostGIS extension setup for Postgres + the RGeo gem.
Action Items

Setup PostGIS locally.
Setup RGeo in the backend repo.
Create a geospatial data column that matches what is available in the City of Boston dataset shapefile.
Append geospatial data to the imported CSV data using a script.

Resources/Instructions

Introduction to PostGIS by Paul Ramsay
Introduction to PostGIS
RGeo Gem

Dat explained they just need the latitude and longitude as imported from the Data team. It should be provided in two parts. First we need to add the lat/long to the homes. Secondarily we need to add a "start point" to the Assignment.

The assignment has a centroid location rather than “start point”. To get the start point, just find the lowest visit_order home with the appropriate assignment_id.


Resolves Issue #1689: Adding filter count to column headers

This PR resolves issue #1689. When a filter is selected for a column that is instantiated with filterable=True, this feature will automatically add a number for how many filters are applied to that column. For 0 filters, no number will be shown; for more than 9 filters, '9+' will be shown.

@SiddharthParatkar are you going to finish this or may I close this PR?

I don't think I'll be able to finish, sorry.


Changing the slider's value by code

I am trying to change the value of the slider by code, but it does not change and even it does not show any error.

A jsfiddle or some public link would be required to help you further.

Dear Sir,
I have further analyzed that once the slider has been rendered, even if we change the attributes of the slider (like value or scale), it gets reflected in the attribute variables but it does not affect the slider.
It would be great if you help me with some trick to get the changes reflected in the slider.
Or could you please help me with setting the maximum value in the slider once it has been rendered.
Any one of the above would be helpful to me.
Thanks in advance..

If you need to update the value of the slider then your should see https://github.com/nitinhayaran/jRange/issues/6 and https://github.com/nitinhayaran/jRange/issues/25
However if you want to update range all together, you could simply re-render complete range slider.

Sir,
Thank You, Now I am able to change the value of the slider,
But as I am new to jquery, I am not able to re-render the slider and I even do not find anything like this in the documentation,
Kindly help me with the re-rendering,
Thanks in advance..


update to the latest adafruit community code of conduct

updates the code of conduct to match the latest revision in adafruit/Adafruit_Community_Code_of_Conduct
fixes #230

Based on the comments in #230 it should be good to update this document to match.
I'm not sure how the existing repositories could be best updated to match this change.

wow I opened this against my repo I'm a dummy.


Implement transition shorthand compactor

Until added transition* properties cannot be merged or overridden.

Examples of overriding multi-value shorthands with single-value longhands: https://jsfiddle.net/w3qft0tm/
Similar to #860 and #933.


Problem with float values after update to Swift 4.1 Xcode 9.3

Hi, I was using a mappable model like this
struct BalanceData: Mappable {

    var balance = Float()

    init?(map: Map){
    }

    mutating func mapping(map: Map) {
        balance     <- map["balance"]
    }

}

And this is the response I am parsing into that model
{
  "status": 200,
  "message": "OK",
  "error": "",
  "data": {
    "balance": 797.76
  }
}

This was working just great until I update to Xcode 9.3 and macOS High Sierra this morning.
Now BalanceData.balance is getting 0 instead of 797.76, I have to change var balance = Float() to Double() in model in order to get the correct value.

we just changed our Floats to Doubles

This was quite a suprising bug! Also on Swift 4.1/XCode 9.3 here, and the app just completely stopped working because of this o_O
We also have a large number of Mappable objects and now I am sitting here and wondering are there more things that are broken? Did I find all the floats?

Hi, I just merged a PR which should address this issue. Please see v3.2.0


Update Coronal Temperature Response IDL Files with Updated Effective Area (Linear Interpolation)

Updating all coronal temperature response IDL files in the repository to reflect the new effective area calculations using IDL's linear interpolation method. The updated files ensure consistency and accuracy for temperature response calculations based on the latest IDL effective area data.

pre-commit.ci autofix

How are the IDL files created?

@nabobalis , @jslavin
The IDL text files used for testing were created in SSW-IDL using make_xrt_temp_resp to generate the temperature response values. However, the reason for updating all the temperature responses is that these values are calculated based on the effective area. Both the IDL effective area code and the XRTpy code have been updated to use linear interpolation. As a result, the temperature response IDL text testing files need to be updated as well while referecning the new Effective Area linear interpolation. (I hope this was clear)
I’m going to start a new PR shortly to ensure that the current temperature response calculations/values fail or are flagged as invalid when calculated with the updated effective area linear interpolation. This will highlight the need for updating the temperature response files.

Is that IDL code stored in the repo, so it can be updated in the future by running one pro file?

We currently have two IDL scripts in the repository that generate these testing files when run in IDL. However, there have been some updates—thank you for noting that! I’ll make sure to update the IDL files accordingly. I’ll also create an issue to ensure I don’t forget.


Add support for setting gid/uid on testfs

This allows to configure the target uid and gid when setting up the filesystem.
This PR also includes some small changes to the dockerfile to improve development speed.
Change-type: minor

@balena-ci I self certify!


GH-2709: Preserve existing JAAS configuration

Resolves https://github.com/spring-projects/spring-kafka/issues/2709
The existing JAAS configuration at the time the KafkaJaasLoginModuleInitializer:afterSingletonsInstantiated is now saved.
The InternalConfiguration class will now search the saved JAAS Configuration first before returning its own stored configurationEntries.
NOTE:  This mimics the approach used by Microsoft's SQL Server JDBC driver, which preserves any existing Configuration while still being able to return its own default JAAS Configuration.
I also added a unit test to KafkaJaasLoginModuleInitializerTests.java to ensure that an existing configuration can be found.
I welcome any feedback on the approach used or testing procedure.
Thanks!

Thanks; merged as 6a2f81cc4 with minor reformatting.

..and back ported to 2.9.x as 138cf1576

Thank you so much!  Does that mean it will be available in the next release of 2.9.x and 3.0.x, perhaps in a month?

No, thank you 😄
Yes, I will schedule them for July 17, as part of the next Boot release train. Snapshots are available now (2.9.10-SNAPSHOT, 3.0.9-SNAPSHOT).


Update FizzBuzz.java

SonarQube analysis reported 6 issues

 2 blocker
 3 major
 1 info

Watch the comments in this conversation to review them.


Fix tar creation to accept windows path

Actualy ADD and COPY failed in Docker file if we use windows path.

The bug is that the path are stored as absolute with full volume name and not as relative as it should be when using unix path

Hi @pleclech ! Thanks for the PR!
Can you give a little more detail on the problem, maybe a repro case we could play with to make sure we don't re-break this?
It's unclear from the PR description if this is a bug with the Tilt Windows binary, or with Windows-based container images.

Hi @nicks
Here the step to reproduce the issue:
OS: windows 10 pro
Tool: Docker for windows <IP_ADDRESS>
#tilt version (v0.12.1, built 2020-02-12)
.\tilt version
#create kubernetes cluster with k3d (https://github.com/rancher/k3d/releases/download/v1.6.0/k3d-windows-amd64.exe)
.\k3d-windows-amd64 version
.\k3d-windows-amd64.exe create
export kubeconfig
$env:KUBECONFIG=..\k3d-windows-amd64.exe get-kubeconfig --name='k3s-default'
#check cluster
kubectl get node -o wide
clone tilt demo
git clone https://github.com/windmilleng/tiltdemo
cd tiltdemo
#launch tilt up
..\tilt up
result in error :
error in ADD ./cmd/demoserver1 no such file or directory

check whats in the container
editing Dockerfile with
ADD . .
RUN ["ls", "-lrt"]
got
→ total 56
→ -rw-rw-rw- 1 root root   248 Feb 14 21:00 C:\work\kubernetes\tilt-windows-issue\tiltdemo\cmd\demoserver2\main.go
→ drwxrwxrwx 2 root root  4096 Feb 14 21:00 C:\work\kubernetes\tilt-windows-issue\tiltdemo\cmd\demoserver2
→ -rw-rw-rw- 1 root root   329 Feb 14 21:00 C:\work\kubernetes\tilt-windows-issue\tiltdemo\cmd\demoserver1\main.go
→ drwxrwxrwx 2 root root  4096 Feb 14 21:00 C:\work\kubernetes\tilt-windows-issue\tiltdemo\cmd\demoserver1

@nicks  Thanks.