500 Server Error after pulling new image

Describe the bug
As of lately, whenever an updated image is pulled (from the latest branch), navigating to the container returns a 500 server error:
The only way to get around this issue is to manually restart the container after an updated image has been pulled/the container has started.
To Reproduce
Steps to reproduce the behavior:

docker compose pull && docker compose up -d to pull and deploy updated image
Once up, navigate to the container in any browser, which returns the 500 server error
docker restart speedtest to restart the container after which the login screen appears as expected

Expected behavior
A clear and concise description of what you expected to happen.
Environment (please complete the following information):

OS: Ubuntu 22.04.3 LTS
Architecture: amd64
Browser Edge chromium
Version: Speedtest-Tracker 0.13.0 (App version)

Screenshots

Logs
<IP_ADDRESS> - - [03/Oct/2023:16:08:52 +0000] "GET / HTTP/1.1" 500 6622 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/117.0.2045.47"
[2023-10-03 16:08:52] production.ERROR: No application encryption key has been specified. {"exception":"[object] (Illuminate\\Encryption\\MissingAppKeyException(code: 0): No application encryption key has been specified. at /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php:79)
[stacktrace]
#0 /var/www/html/vendor/laravel/framework/src/Illuminate/Support/helpers.php(307): Illuminate\\Encryption\\EncryptionServiceProvider->Illuminate\\Encryption\\{closure}()
#1 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(77): tap()
#2 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(60): Illuminate\\Encryption\\EncryptionServiceProvider->key()
#3 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(32): Illuminate\\Encryption\\EncryptionServiceProvider->parseKey()
#4 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(908): Illuminate\\Encryption\\EncryptionServiceProvider->Illuminate\\Encryption\\{closure}()
#5 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(795): Illuminate\\Container\\Container->build()
#6 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(937): Illuminate\\Container\\Container->resolve()
#7 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(731): Illuminate\\Foundation\\Application->resolve()
#8 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(922): Illuminate\\Container\\Container->make()
#9 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(1066): Illuminate\\Foundation\\Application->make()
#10 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(982): Illuminate\\Container\\Container->resolveClass()
#11 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(943): Illuminate\\Container\\Container->resolveDependencies()
#12 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(795): Illuminate\\Container\\Container->build()
#13 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(937): Illuminate\\Container\\Container->resolve()
#14 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(731): Illuminate\\Foundation\\Application->resolve()
#15 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(922): Illuminate\\Container\\Container->make()
#16 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(255): Illuminate\\Foundation\\Application->make()
#17 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(213): Illuminate\\Foundation\\Http\\Kernel->terminateMiddleware()
#18 /var/www/html/public/index.php(55): Illuminate\\Foundation\\Http\\Kernel->terminate()
#19 {main}
"}

Additional context
Add any other context about the problem here.

I'm assuming since I don't have a persisted key, this issue is coming up whenever a new image is pulled. I'll set one now and will see if the same happens with the next update.

Feel free to close this out should that fix the issue

Try running php artisan optimize:clear in the container and then restart it. It's probably in the wrong order.

I thought perhaps when a new image is pulled the cache would need to be cleared for those that don't have APP_KEY defined within this function.

My apologies for linking to the wrong file.
An update: I have another remote system that I hadn't updated to 0.13.0 yet, so I was able to use it as a test. I entered into the container, generated a key, and then set the APP_KEY to it. I then pulled the updated image and sure enough no 500 error.  So, it looks like the key is the root cause of the 500 error when updating the image for users that don't have the APP_KEY set.

Cool thanks for confirming, I'm going to tag this as a bug and get the caching order fixed.

Appreciate the quick response!

@RobertD502 pipeline is building v0.13.1 now and the cache will be fixed in that release. Also you should see a significant page performance as the cache default has been changed to the file driver.
All changes in #841 with a link to the docs.

Did not seem to get fixed:

I commented out the APP_KEY prior to pulling the latest image. Once it was pulled and the container was running, navigating to it still results in a 500 error. Should the cache be cleared after the new key is generated? The threads I've seen about this issue usually involve resetting the cache after key is generated.

Same log as before:
<IP_ADDRESS> - - [04/Oct/2023:16:57:39 +0000] "GET / HTTP/1.1" 500 6622 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/117.0.2045.47"
[2023-10-04 16:57:39] production.ERROR: No application encryption key has been specified. {"exception":"[object] (Illuminate\\Encryption\\MissingAppKeyException(code: 0): No application encryption key has been specified. at /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php:79)
[stacktrace]
#0 /var/www/html/vendor/laravel/framework/src/Illuminate/Support/helpers.php(307): Illuminate\\Encryption\\EncryptionServiceProvider->Illuminate\\Encryption\\{closure}()
#1 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(77): tap()
#2 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(60): Illuminate\\Encryption\\EncryptionServiceProvider->key()
#3 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(32): Illuminate\\Encryption\\EncryptionServiceProvider->parseKey()
#4 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(908): Illuminate\\Encryption\\EncryptionServiceProvider->Illuminate\\Encryption\\{closure}()
#5 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(795): Illuminate\\Container\\Container->build()
#6 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(937): Illuminate\\Container\\Container->resolve()
#7 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(731): Illuminate\\Foundation\\Application->resolve()
#8 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(922): Illuminate\\Container\\Container->make()
#9 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(1066): Illuminate\\Foundation\\Application->make()
#10 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(982): Illuminate\\Container\\Container->resolveClass()
#11 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(943): Illuminate\\Container\\Container->resolveDependencies()
#12 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(795): Illuminate\\Container\\Container->build()
#13 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(937): Illuminate\\Container\\Container->resolve()
#14 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(731): Illuminate\\Foundation\\Application->resolve()
#15 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(922): Illuminate\\Container\\Container->make()
#16 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(255): Illuminate\\Foundation\\Application->make()
#17 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(213): Illuminate\\Foundation\\Http\\Kernel->terminateMiddleware()
#18 /var/www/html/public/index.php(55): Illuminate\\Foundation\\Http\\Kernel->terminate()
#19 {main}
"}


[4.4.x] CAMEL-20873: Handle potential failures in VertxPlatformHttpSupport.writeResponse

Test robot failed due to missing SNAPSHOTs. I verified the tests pass locally.


[Issue-359] Direct Dictionary Query Flow handling as per the latest impl & [Issue-374] Direct dictionary dataload failure with lenient date parsing

[Issue-359] Direct Dictionary Query Flow handling as per the latest impl
Corrected the UT failure
[Issue-374] Direct dictionary dataload failure with lenient date parsing

Please rebase

rebased

retest this please

LGTM, Merged to master.


JVM crashes when LMDB is used after the env is closed

If an LMDB cursor, cursorIterable, txn or dbi is used after the env is closed then a JVM crash is likely.  The code should not be trying to do this but where we have multiple threads interacting with the same env and closure of envs is part of the process then this is a possibility.
Ideally the lmdbjava lib would protect us from this as we can't risk the JVM crashing.  Have raised an issue/PR on lmdbjava lmdbjava/lmdbjava#185 to see if it can be addressed there.  If not then we may need to build in our own protection.

Fixed in b.157


Update shared-experiences-in-mixed-reality.md

Docs Build status updates of commit 18339e4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/shared-experiences-in-mixed-reality.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


How To Pre-generate Specific DB configurations?

Operating system (distribution) and version: macOS (Mojave and above)

DBeaver version: Community 7.2.1

Database name and version: Snowflake

Hi,
I'm working to automate database connections for users, meaning that when a user opens his laptop - opens DBeaver - he'll see all the database connections ready on his Database Navigator list. This doesn't mean they'll work, as the user will need to receive username/password separately to access it.
I've noticed I can export Projects, but not DB navigator configuration. Is this something that can be done / how can this be done?

This should be the file you're looking for:
~/Library/DBeaverData/workspace6/General/.dbeaver/data-sources.json

@p91paul Thank you! unfortunately, it seems there's only a "workspace2" folder, with /Scripts inside it and doesn't have the General folder as mentioned.
I tried looking for data-sources.json directly, but it was not found.

@p91paul Funny enough, it works with Jamf's MDM solution Composer when capturing it, but still not found in that directory. It seems to be coming from the ROOT folder.
Thanks again!


Disable Swipe Gesture of XLtabstrip

Is there any way to disable the page swiping gesture. So the user will only be able to move the pages by clicking the tabs in the tab strip.

self.containerView.isScrollEnabled = false
put this line in viewDidLoad() block in your base class file

It would be nice to have this documented in the README


[PERSISTENCE] Ruby Float should correspond to Elasticsearch double

The Ruby Float class is in fact a double-precision floating point value, not single-precision.  Because of this, when mapping the attribute information for elasticsearch-persistence, the Float class should become 'double', rather than 'float' in order to avoid loss of precision
Ruby Float documentation: https://ruby-doc.org/core-2.2.0/Float.html
Line of code mapping the Ruby Float type to the Elasticsearch float type: https://github.com/elastic/elasticsearch-rails/blob/a714d5931182501531d2ce6b767bcb3c97ce56a7/elasticsearch-persistence/lib/elasticsearch/persistence/model/base.rb#L76

Closing as the ActiveRecord persistence pattern has been deprecated and removed.


[Bug]: Misleading "Stake" Button Text in Tezos Delegation Feature

Impacted Library name
@ledgerhq/coin-tezos
Impacted Library version
0.3.12
Describe the bug
In the Tezos protocol, it is possible to "Delegate" the baking/validation rights of our account to a network "Baker". Baker is the Tezos term for a POS Validator.
In the Ledger Live Wallet App, there is a call-to-action button labeled "Stake". The use of the word stake in this context up until now has been a misnomer, as "Stake" and "Delegate" are occasionally substituted for each other.
On Tuesday, June 4th, the Tezos blockchain protocol will update to a new protocol named "Paris" (more details here).
This protocol introduces a new feature named "Staking" that exists alongside "Delegation".
A quote from the Paris proposal description is as follows:

The Paris proposals introduce a new role — staker — in addition to delegate (baker) and delegator. Stakers contribute to their chosen baker’s security deposit without the baker taking custody of their funds, for which they are allocated a proportional share of rewards automatically by the Tezos economic protocol. Unlike liquid delegation, staked funds are frozen and are subject to slashing, in proportion to their contribution to their chosen baker’s deposit.

Given this new feature, the misnomer of using the word "Stake" in place of "Delegation" becomes more problematic.
Expected behavior
The "Stake" button text should be renamed to "Delegate" so that Ledger Live Wallet users are not misled into thinking they would put their tokens at risk when delegating.
Additional context
This change is critical with the introduction of the new "Staking" feature in the Tezos "Paris" protocol update. Clarifying the terminology will prevent confusion and potential misuse of the delegation feature by users.

This needs to be a high priority


There HAS to be a bug in Brackts because my javasript event "onmouseover" is not working in JavaScript! SOMEONE HELP!!!

I downloaded Brackets TODAY and converted all my files to use on there. When I literally run the exact same code in NotePad++ the "onmouseover" event DOES NOT WORK PROPERLY. So basically i converted from Notepad++ to Brackets today and the issue is when i use the "onmouseover" event the image transition is immediate, but when i use the "on-click" event the transition is smooth. However when i use the exact same code in Notepad++ either event executes my function properly. I even RECORDED A VIDEO of whats going on and upload it to my repository  folder titled "Brackets_Bug" to show whats happening with my event. The onmouseover event either doesn't work properly or there is something wrong with Brackets. HOW do i contact them directly for their support OR can SOMEONE PLEASE HELP!!!

Hi @CheckLife,
are you using the Live Preview feature of Brackets?
Brackets the code runs on actual browsers and shouldn't have anything to do with Brackets itself. However there might be differences on how you run the code: running the code from the FileSystem (with Notepad++) might behave differently than over the remote debugging protocol (like through Brackets Live Preview).

@petetnt Did you see the video of whats happening in my repository? I literally run both events in NotePad++ and it works fine. What is a solution to this!!!???? because "onmouseover" should not produce a faster transition if i haven't changed anything in my function or CSS

@CheckLife Please watch your tone. I've checked your video, cloned your code and the problem has nothing to do with Brackets, I can reproduce it even when changing code through Notepad++.
Your problem is that by moving the mouse across button, onmouseover is executed multiple times. You can try to write
function slow() {
console.log('slow executed');

and check the developer tools for how many times it outputs slow executed text.
Simple solution to this is:
    var slowHasRun = false;
    function slow() {
        if (slowHasRun) { return; }
	slowHasRun = true;
        document.getElementsByClassName("tracy")[0].style.opacity = 0;
        ... rest of your code

Closing this as it's not a Brackets issue.


Pass fields labels trough translation filter in control panel

Sometimes fields name values are not human readable. I propse that they should be passed trought |t filter to give developers ability to trsbalate them.
So, in _templates/submissions/_show.twig would have this:
        {% for key, value in messageObject %}

{{ key|ucfirst|t }}

            {% if value is iterable %}
                {% for item in value %}

{{ item|nl2br }}

                {% endfor %}
            {% else %}

{{ value|nl2br }}

            {% endif %}
        {% endfor %}

To avoid accidentally matching some already existing translation strings, some prefix could be added:
        {% for key, value in messageObject %}

{{ 'contactField`~key|ucfirst|t }}

            {% if value is iterable %}
                {% for item in value %}

{{ item|nl2br }}

                {% endfor %}
            {% else %}

{{ value|nl2br }}

            {% endif %}
        {% endfor %}

I'll gladly accept a PR that changes this!
I don't like a prefix as then you're required to translate them. I'd just use |t('contact-form-extensions') which will then allow people to translate it using the contact form translations file

Closing as PR open for this item.


I'm having a typerrore : abytes like object is required not str .

Line 89. Change with   output_file.write(ablob.encode())

Change the  line 89
output_file.write(ablob)
to
output_file.write(ablob.encode())

I don't think that it will solve the issue, I think that he is trying to use maps directly from openstreetmap,without processing as suggested on the documentation.
Including these lines of code it will solve the exceptions, but the output are not png files.

I have the same problem as @Mohammad-ezzidden
I'm using a .mbtiles file which is a custom extract from OpenMapTiles.com. Changing the code as suggested generated the files, but as @philippe78 said not valid .png files.
Are the files from OpenMapTiles.com raster or vector files? I tried to (just in case) to convert from vector .mbtiles file to raster using this https://github.com/bjornreppen/rasterize-vector-tiles but then I got an error regarding a missing property in the source .mbtiles

I can't believe I missed these comments. My appologies for not seeing them to address them earlier.
@Swepilot
OpenMapTiles.com states it offers both raster and vector files, but the downloads appear to only offer MBTiles (PBF). PBF is a vector format. This project will only work if the MBTiles are in raster format.

I've updated the readme to be clearer on rasterised .mbtiles usage only. Closing issue as I am certain this error is when using vectored based .mbtiles.


In later steps, map port

File: get-started/part2.md
should be:
In later steps, you will map port

@jimshowalter: Could you please be more specific where the line needs to be fixed? Thanks.

you will map port
should be:
you will map a port

The document has been changed, i even can not find this sentence anymore "you will map port". This issues should be closed.


Remove size mixin

Closes #25752.
This PR removes the obsolete size mixin.

OK, so how about showing a warning when the mixin is used? And what's the policy for breaking stuff? Only in major version changes?

And what's the policy for breaking stuff? Only in major version changes?

Semantic versioning rules say only major versions can have breaking changes. It'd have to be deprecated, meaning the code left in and a warning added.

I'm closing this.
Discussion about mixins documentation here: #25977


Vulnerability fix for DownloadPackageV1

Task name: DownloadPackageV1
Description:
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N)
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected
[x]  node make.js build --task DownloadPackageV1
[x]  node make.js test --task DownloadPackageV1

/azp run

@rajnish-byte 2 questions:

Why did this get merged without our team's approval as code owners? This created a livesite incident.
Why was the task version not bumped in this PR?

cc @phil-hodgson

Incident 330053957 : TypeError in Download Package step for Build Pipelines


feat: unit test of task.py

just a test , do not know meet the issue or not .

The PR in the forked repository is currently facing some issues with action execution. We will merge your PR once we resolve them.


Document auto-link syntax for Erlang extras

ExDoc 0.29.1 contains a change that supports non-Elixir extras: elixir-lang/ex_doc#1617. This just changes the syntax parsed by the autolinker. Previously the markdown code-blocks would need to be written in Elixir+ExDoc syntax like :gen_server.call/3, c::gen_server.handle_call/3 or t::gen_server.from/0. Now when the proglang is erlang (which is set by rebar3_ex_doc here), the syntax is Erlang with the ExDoc prefixes (c: for callbacks and t: for types). Those examples are now written as gen_server:call/3, c:gen_server:handle_call/3 and t:gen_server:from().
This PR updates the ex_doc dependency to 0.29.1 and adds some documentation on the autolinking syntax. (There's also a small typo fix in the README.)
What do you think?

Beautiful! ❤


Update the guides in section "Other Guides"

https://www.kubeflow.org/docs/other-guides/upgrade/
Documentation is out of date (still refers to v1alpha1 TFJobs). Also the page makes references to ksonnet which is deprecated.
We should just remove the section completely until upgrade is supported.
Similarly, these pages also contain references to ksonnet:
https://www.kubeflow.org/docs/other-guides/troubleshooting/
https://www.kubeflow.org/docs/other-guides/usage-reporting/
https://www.kubeflow.org/docs/other-guides/advanced/

I think we should update the guides, rather than removing the section. The guides are needed, but are out of date.


Beacon search exception on Sony phone

Exception occurs when it runs on Sony F3316 device (Android 6.0)
10-27 17:35:50.356 28763-29567/ W/MessageQueue: Handler (com.estimote.sdk.BeaconManager$IncomingHandler) {a3f529d} sending message to a Handler on a dead thread
                                                            java.lang.IllegalStateException: Handler (com.estimote.sdk.BeaconManager$IncomingHandler) {a3f529d} sending message to a Handler on a dead thread
                                                                at android.os.MessageQueue.enqueueMessage(MessageQueue.java:555)
                                                                at android.os.Handler.enqueueMessage(Handler.java:707)
                                                                at android.os.Handler.sendMessageAtTime(Handler.java:609)
                                                                at android.os.Handler.sendMessageDelayed(Handler.java:579)
                                                                at android.os.Handler.sendMessage(Handler.java:516)
                                                                at android.os.Handler$MessengerImpl.send(Handler.java:797)
                                                                at android.os.Messenger.send(Messenger.java:57)
                                                                at com.estimote.sdk.service.BeaconService.invokeCallbacks(BeaconService.java:644)
                                                                at com.estimote.sdk.service.BeaconService.access$4800(BeaconService.java:91)
                                                                at com.estimote.sdk.service.BeaconService$6.onScanCycleCompleted(BeaconService.java:1042)
                                                                at com.estimote.sdk.service.BeaconService$3$2.run(BeaconService.java:412)
                                                                at android.os.Handler.handleCallback(Handler.java:815)
                                                                at android.os.Handler.dispatchMessage(Handler.java:104)
                                                                at android.os.Looper.loop(Looper.java:207)
                                                                at android.os.HandlerThread.run(HandlerThread.java:61)

I'm not sure if it happens on other Sony device but my application works well for Samsung Galaxy S6 (6.0.1)
It happens everytime found a beacon device. I try to use SDK 0.12.0 and latest 0.13.0 but nothing changed.
Anyone faced this problem?

I got the same problem. do you have solve it yet?

Hello @chkchk, @ducva
It looks like your BeaconManager object was deleted, but never disconnected from underlying service, which tries to send scan data to "dead" beacon manager. I want you to make sure that your BeaconManager object is not deleted without calling disconnect() method. If this won't help, we will proceed further.
Have a great day!

@pawelDylag, I didn't call disconnect() method but I didn't delete BeaconManager, my demo application just called Connect then StartRanging, I don't know why but it runs perfect on my Samsung device but the Sony device didn't

Hello @chkchk @ducva
Could you confirm that this is still happening in version 1.0.2?
Have a nice day!


Inconsistent separated theme in interactive window.

Environment data

VS Code version: 1.54.3
Jupyter Extension version (available under the Extensions sidebar): v2021.3.619093157
Python Extension version (available under the Extensions sidebar): v2021.3.658691958
OS (Windows | Mac | Linux distro) and version: Local Mac and Remote Ubuntu 18.04
Python and/or Anaconda version: python 3.8.8 and conda 4.9.2
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Remote

Expected behaviour
I'd like to set jupyter interactive windows with separated theme (Light) to my vscode dark theme with the setting:Jupyter: Ignore Vscode Theme enabled.

This is what it looks as I manually switch vscode to light theme and disable that option.

Actual behaviour
As enabling that option and set vscode theme to dark, the background color and highlight of code syntax in interactive windows do switch to Light theme, but others like the icons in the top and toolbars above code box are still the color in dark theme.

Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Set vscode theme to dark theme.
Enable that option in Setting.
Reload vscode window.
Open the notebook.

Thanks for your feedback. This issue will be addressed once VS Code adds native support for editing notebooks. Please see https://github.com/microsoft/vscode/issues/91987.


Added carthage scripts

Added carthage script
Added frameworks from carthage subdirectories

CI script is using Xcode 10.2.
Xcode 11 is required.


Lyrics in mini player

Describe the bug
Non-synced lyrics in the media player cannot be scrolled.
To Reproduce
Steps to reproduce the behavior:

Enable mini player
Enable lyrics

Expected behavior
Not sure if non-synced lyrics should be shown, since scrolling would be better set to control the volume.
Screenshots

System Information:

OS: Windows 10
Affected App Version: 1.0.0-stable

What if I only show lyrics in the Mini Player if the song only has synchronized lyrics?
If it only has unsynced lyrics, a message saying No Synced Lyrics Found is displayed instead of the unsynced lyrics.

Confirmed


Provide instructions on using and accessing example data

Closes #119, #145, and #146.
Here I have included a README file for the example folder, providing instructions on how to access and download both the example input data and example output files as well as perform a test run using the input files. In the instructions I included the download links that can be used to access the files and then have instructed the users to create a metadata file for the example data, providing a table with an example of what to fill in for each column.
In doing this I also updated the example_metadata.tsv to reflect only the columns that are required or optional and are listed in the section on preparing the metadata file in the external instructions. Specifically this meant removing the library_method column. I also added an example of a bulk sample, since we do mention that in the description of the column contents in the section. What I did not do was update the path to the example data on S3, since we want people to download the files locally and then test their setup rather directly using the files on S3. Correct me if I'm wrong on this and should correct this.
However, in order to run the workflow using our config setup, I did input the path to the files on S3, and that metadata file that was used can be found on S3 in s3://scpca-references/example-data/example_metadata.tsv. That file contains just a single row with the path to the example outputs on S3. Let me know if you think this should be updated in anyway.
Questions for reviewers:

Should we provide instructions on using either wget or curl to download the data? Or do we assume that users who want to use that will know how?
Is there anything that is unclear about the instructions? Or things that I missed?

Also to note that the zip file for the input files is still in the process of uploading to S3 but should be there soon and I used the --acl public-read to ensure the file was set to public.

I went ahead and updated the instructions based on @jashapiro's comments. In doing this I updated the link to the input fastq files to be the link directly provided by 10X, rather than the input files that we have stored.
I also updated the ID's in the example table that I included. I would agree with Josh's original comment that we should update the example_metadata.tsv file to also utilize a different naming pattern than the SCPCX pattern but thought that change better fit in a separate PR/issue.
One thing to note is that we should probably make sure the output data contains the same ID's that we are suggesting here, rather than the SCPCX id's as the folder names like they are now. If others agree, I will file an issue to track updating the output folder to reflect the naming chosen in this example table in the instructions here.
Since Josh is out, going to re-route this PR to @jaclyn-taroni for review, but let me know if there is someone else I should have review.

Thanks for the review @jaclyn-taroni. I did a little bit of rearranging, making the beginning section about setting up the configuration file and making sure that you had the necessary pieces before proceeding on to using the example data. I also added a paragraph explaining each of the arguments for the nextflow run command and then a note about the size of the download for the input files.
I added a line about needed the config file at the beginning and then changed the note that I had at the end to a warning. Do we need another warning at the beginning of the testing your setup section? I also thought about relocating the warning to the beginning of that section, but no matter where I put it, it still felt like it was slightly repetitive to the information before it. Let me know if you had something else in mind or if things are still unclear.

@jaclyn-taroni I moved the warning to be at the beginning of the section. I interpreted your comment that it should be moved and not both at the beginning and the end of the section, let me know if I interpreted that incorrectly. I also reworded the description of the arguments, emphasizing when relative vs. full paths are necessary and adding in the link to the example file like you suggested.


Allow adding meta data when registering service

I like to add meta data to my services, this allows just that. It's a variation of #222

PR seems to be fine. Thank you @ojongerius.

@matusvalo can you please release this new feature?


failure to display music with Mopidy-Local-SQLite

After change the library from JSON to SQLITE, moped fail to display the song.
Other web clients works fine.
Flavio

@F74 could you please confirm that this issue is fixed?


Plugin APIs and implementations

When CivGradle was introduced, each plugin that was switch over to it had all of its code moved to a paper subfolder. This was ostensibly done to allow support for multiple NMS versions and potentially even other platforms:

However, there has been zero progress towards this. Indeed, we've just surpassed NameLayer's second anniversary of being Gradle'd, and yet it's further from that vision now than before. NameLayer had code to make it work over multiple versions, and even on Bungee, an entirely different platform; NameLayer should've been the shining example of this vision, and it remains conspicuous in its nonfeasance.
There is a real opportunity to push towards this vision, or abandon it. NameLayer needs a good clean up, and NameColors' going to be merged into CivChat2, and CivChat2 itself needs a good clean up to move away from legacy colours and formatting.

Personally, I am of the opinion that the api/implementation split should be abandoned, here are my 13 reasons why:

The likelihood of ongoing support for older versions is minimal at best. Even prior to NameLayer's Gradle-ification, new features were not tested with ANY of its other supposedly supported NMS versions. Indeed, NameLayer now includes some use of Kyori components, which makes NameLayer more or less incompatible with Minecraft below Paper 1.16.5 (473). It's a safe bet that this was never tested for, or even a consideration. This will likely continue. (CivModCore's README says: "No explicit backwards support is offered to any previous version, whether it be major, minor, or patch.")

The likelihood of other platform implementations are extremely low. I would hazard a guess that anyone committed enough to do such a drastic refactor just make an entirely new and separate project. CivCubed did this without changing the platform.

The likelihood of contributions from fellow civ-iterations are minimal at best. CivMC, Civclassics, DevotedMC, CivEx, CivRev, etc, all had/have their own distinct organisations with their own forks, and it's immediately obvious why: why would you forego a neat collection of your own permanent forks, for a random collection of temporary forks while you wait for some crusty long-quit dev to merge your upstream PR? And devs aren't going to go out of their way to make nice and concise upstream PRs, otherwise why hasn't CivMC done exactly that to Civclassics or DevotedMC?

For these reasons, I don't think it's worth the time, effort, and gatekeeping that'll come with this. It would be best if civ-plugins went back to being provided as-is.


Using MetroNavigationWindow

There's nothing in the docs that I can see about using the MetroNavigationWindow, I've tried the following:
<controls:MetroNavigationWindow x:Class="PackageCreator.MainWindow" />

But I get MahApps.Metro.Controls.MetroNavigationWindow' cannot be the root of a XAML file because it was defined using XAML

@ClintPearson-KPMG  MetroNavigationWindow it's not like a control. You have to use it the same way as the MetroWindow.

@robertstefan I gathered, I also tried to have the .xaml.cs file inherit from MetroNavigationWindow as well, problem is the XAML error still happens, then changing the XAML to MetroWindow but leaving the derivation as MetroNavigationWindow causes an error about differing base classes.

So, in the end; how does your code look like?

Scenario 1
<MetroNavigationWindow ...>

The MainWindow.xaml file

public partial class MainWindow : MetroNavigationWindow
{
  // Content Here
}

The MainWindow.xaml.cs file

Produces error: ... cannot be root of XAML file because it is defined using XAML
Scenario 2
<MetroNavigationWindow ...>

public partial class MainWindow : MetroWindow
{

}

Produces error ... partial class cannot have differing base class

So, taken from the example application provided by the community, you will have to use it like this:
var navWin = new MetroNavigationWindow();
navWin.Title = "Navigation Demo";
navWin.Show();
navWin.Navigate(new Navigation.HomePage());

where HomePage you can find it here.

Oh, so it doesn't support being used in XAML in the slightest? This framework keeps seeming less and less MVVM friendly. Thanks for your help though!

@ClintPearson-KPMG i you can make something better MVVM "friendly", so do it!

@ClintPearson-KPMG the issue with the MetroNavigationWindow is that the XAML designer in Visual Studio doesn't support loading NavigationWindow subclasses.

Yep, the problem is, that MahApps.Metro\Themes\MetroNavigationWindow.xaml defines content of the MetroNavigationWindow. The solution is either to define the content in the code only, or define the content as a Style Template, see http://svetoslavsavov.blogspot.gr/2009/09/user-control-inheritance-in-wpf.html.

@ckpearson why is this issue closed? There's no way to solve the problem from MapApps side?

For anyone that would like to go with my crappy code, here is a version that works for me and is XAML compatible.
Essentially a rewrite of existing XAML to code.
NavigationWindow.cs.txt


[LLC] [Feature] Determine where LLC-generated model files will go

For RCv2, the LLC will generate model types where input models can be implicitly cast from model to RequestContent, and output models can be implicitly cast from Response to model.
We expect these models to change over time, either from service changes, or changes for the convenience layer that may be added to the library based on business needs.  In order to guarantee that we will not make breaking changes to our libraries, these models need to be generated to a target outside a library package.  For them to be useful, we need to provide the ability for customers to copy the model code into their projects and take ownership of this code.
This issue tracks the work to define that process.

@annelo-msft:  This seems like it has been superseded by DPG changes.   Should we close this out?

@m-nash can share whether it is still needed.  This was a post-GA feature for value-add over currently generated models, so it might make sense to put it on the backlog.


iOS build failure in 1.1.3?

Thanks for taking the time for reporting an issue!
Describe what happened
I'm seeing a bunch of undefined symbols when I try to build on iOS. I saw this with 1.1.2 and 1.1.3. Am I doing anything wrong?

Steps to reproduce the issue:
Integrate the SDK, install pods and attempt to build.
Describe what you expected:
The build to succeed
Additional context

1.1.2 and 1.1.3
React native 0.67.4, building on iPhone 14

Hey @louiszawadzki thanks for getting back to me! Here are the Datadog deps in Podfile.lock
...
  - DatadogSDK (1.12.0)
  - DatadogSDKCrashReporting (1.12.0):
    - DatadogSDK (= 1.12.0)
    - PLCrashReporter (~> 1.10.1)
  - DatadogSDKReactNative (1.1.3):
    - DatadogSDK (~> 1.12.0)
    - DatadogSDKCrashReporting (~> 1.12.0)
    - React-Core
....

- PLCrashReporter (1.10.2)
...

Anything specific you are looking for in the Podfile? Thank you!

Also, does the library support simulator? Was curious if this was an architecture (x86_64) problem

Hey @louiszawadzki, I fixed it, thanks! We use use_frameworks and the issue was in my Podfile. I had to add it as a static lib. All good now.

@sushilr I'm facing the same issue you were facing last week, we are also using use_frameworks in our Podfile and encountering the missing symbols issue you mention above. Could you specify here the change you made to your Podfile? Which library did you have to make static? Thanks

Actually don't worry, I think I've got it working now, the bit I wasn't sure about was which Pod to make static but turns out to be DatadogSDKReactNative, so this is basically what I have now
static_libraries = ['DatadogSDKReactNative', ...]
  pre_install do |installer|
    installer.pod_targets.each do |pod|
      if static_libraries.include?(pod.name)
        puts "Overriding static framework for #{pod.name}"
        def pod.static_framework?;
          true
        end
        def pod.build_type;
          Pod::BuildType.static_library
        end
      end
    end
  end


Parse migration version as a long value instead of int

Hi! I'm thrilled to use this library in my project, but unfortunately, it does not support migration versions with the long type.
I'm using the 'Flyway Migration Creation' IntelliJ IDE plugin, which creates files as follows:
V20240309222903__create_new_example_table.sql
V20240309222927__insert_initial_example_data.sql

When I run the application, it throws a NumberFormatException because it cannot parse this long value as an int in name.nkonev.r2dbc.migrate.core.FilenameParser#getVersion. Could you please take a look at it?
Thanks in advance!

Just released 2.12.0, it should be available after some time. It's recommended (but it still works for low numbers without this change at least in case PostgreSQL) to make id column bigint.


Incomplete docs

I was looking for a way to use phinx within my phpunit integration tests and found this page: http://docs.phinx.org/en/latest/commands.html.
Unfortunately no example code is provided how to use phinx within your tests.
In the meanwhile I will look into the tests of phinx self and look for a way to use it without the command line. If I have time I could update the readme.

@MadeRelevant Could you give a bit more information as to what you were specifically trying to do please?

Hi JayPHP,
Sure. At the moment the docs are updated and it is much clearer how to use Phinx programmatically.
Before the docs were updated I came up with the following:
<?php

class SomeTest extends SomeAbstractMigrationTest {

function setUp() {
$migration = new class(1) extends AbstractMigrator {
            public function up()
            {
                   /* Your usual stuff */
            }

            public function down()
            {
                   /* Your usual stuff, probably just a drop table within tests */
            }
        };

        $input = new StringInput(null);
        $dbAdapter = new MysqlAdapter([
            'charset' => 'utf8',
            'name' => getenv('MYSQL_DATABASE'),
            'host' => getenv('MYSQL_HOST'),
            'user' => getenv('MYSQL_USER'),
            'pass' => getenv('MYSQL_PASSWORD')

        ], $input);
        $dbAdapter->setSchemaTableName('patchhistory_global');
        $migration->setAdapter($dbAdapter);

        $migration->up();
        // register the migration so it can be rollbacked automatically in tearDown()
        $this->migrations[] = $migration;
}

protected function tearDown()
    {
        parent::tearDown();
        foreach ($this->migrations as $migration) {
            $migration->down();
        }
    }

}

I have some EAV implementation where new entity types require some meta tables so in my tests I wan't to fake having X number of entity types with their corresponding tables, thats the reason behind this requirement.
In case this could be done much better or easier please let me know. Until further response I'll close the issue.


run our cluster finder on the entire DES survey

The results do not need to be good to close this issue—it’s an achievement to have processed the entire survey. The first step is to download all the relevant images (coadds presumably). They can be stored under /nfs/turbo/lsa-regier/scratch/.

Done!

Great job @kapnadak and @gapatron


Add Forum Pinner module

This module automatically pins the first message of newly created forum posts.
It also adds the /pin command that pins the first message it fetches (probably needs rework?)

Mainly not sure what the intended use of the /pin command is. Is it staff only? If not, I'm wondering if it's worth being a separate PR so that it can be evaluated independently of the more important functionality here (automatic pinning).

The pin command is useful for threads that were made prior to this module running, it is of course only executable by members with the MANAGE_MESSAGES permission.

If the command is staff-only then it's all good in the current state :+1:


How to specify the gpus used

I tried to add os.envrions in the code but it doesn't make sense.

Here is the bug

GPU is alrealy in use, but it encounters an OOM error, try to reduce your batch size.


v6 — Changing initialValues should update per-field dirty/pristine

I'm using redux-form 6.0.0-alpha.15. I'm enjoying using it, but there's a problem with the 'dirty' state.
In a form I'm working on, I added a visual indicator next to each field to show whether that field is currently dirty or pristine. After I submit the form and the server accepts the changes, the form's initialValues object is updated to the values I just submitted. Because the initialValues I'm passing to the form now matches the new field values, I would expect fields to become pristine, but that is not happening; the changed fields remain dirty. It turns out redux-form ignores changes to initialValues until I reinitialize or remount the form.
I could reinitialize the form, but that is not an acceptable solution because the user may have changed field values after submission and I don't want to overwrite their changes. Instead, I added a temporary workaround that computes dirty state by comparing the latest initialValues with the current values.
I looked at the cause and I see that redux-form copies initialValues to Redux state. I wonder if it would make more sense to keep initialValues in component props only and never copy it to the Redux state where it can fall out of sync with component props. Would that change conflict with anything? An alternative is to provide some kind of changeInitialValues() method, but that would be a lot less intuitive.

Maybe you could dispatch an action, which removes the dirty flag?
http://redux-form.com/6.0.0-alpha.15/docs/api/ReducerPlugin.md/
@erikras would this be the right use case for the ReducerPlugin?
PS: sideNote question: Why is the flag "anyTouched" in the Store but "dirty" is not?

This will be solved in the next release with an enableReinitialize config parameter. #1121.

PS: sideNote question: Why is the flag "anyTouched" in the Store but "dirty" is not?

@ms88privat dirty is calculated from the values in the store by comparing value and initial.

@erikras Thanks. Could you please show me the code snippet for this? I can't find it.
I need this dirty flag on a different place, is there a possibility to expose your selector / computation on this?

This was solved in 6.x.


Prepare HFSM to be agnostic

jam: 19 Jun 2019
This is related to the Player as owner & StateMachine references on State we talked about earlier today. With this PR there's no need for a setup method to pass in the Player & StateMachine objects that needs to be called manually.
This goes more towards a "behind the scenes" type operation, but I think it fits well with the way the HFSM system is in place.
I've moved StateMachine.gd & State.gd to the new src/Agnostic folder as these can be used now on any object as long as it follows the rules (yet to be written in a doc of sorts) of the HFSM.
In State.gd now we rely on onwer to get the "top node of the scene", in the case of Player scene, this is Player. Mind you, the owner isn't ready by the time this variable is assigned, which means that all of the onready vars in Player scene aren't ready to be used. To circumvent this problem we connect to the owner's "ready" signal to do basic setup since we can't rely on _ready in the individual States.
There's a new _setup function that behaves like _ready, but it's called only when, in our case Player emits the ready signal. So now there's no _player variable any longer, we operate directly on owner.
The StateMachine node is found through get_hfsm method from the State.gd script. It relies on StateMachine being part of a group called hfsm, but to reliably do this StateMachie.gd assigns itself to this group in the _init function so this isn't something that the user/programmer needs to remember to do manually.
With this PR that moves us in the direction of an Agnostic system should be pretty simple to extend Hook with HFSM so that PR will rely on this - to be updated in that PRs description.

I'd only move things to a descriptive folder like StateMachine instead of "agnostic". Aside from that, it's looking good, merging.


Allows can-import to be used for dynamic imports

This adds the ability of can-import to be used for dynamic imports. The
static form is still supported, but dynamic imports are allowed by
including a closing </can-import> tag and nesting content within.
The viewModel for can-import is a Promise that will resolve with a
value once the module has been imported.

Oh also, tests can't be written in CommonJS format yet (but probably should eventually although I'd prefer ES6).


kernel/timer: fix period argument clamp handling

Commit 3e729b2b1c52 ("kernel/timer: Correctly clamp period argument")
increased the lower limit to 1 so that it wouldn't conflict with a
K_NO_WAIT. But in doing so it enforced a minimum period of 2 ticks.
And the subtraction must obviously be avoided if the period is zero, etc.
Instead of doing this masquerade in k_timer_start(), let's move the
subtraction and clamping in z_timer_expiration_handler() right before
registering a new timeout. It makes the code cleaner, and then it is
possible to have single-tick periods again.
Whith this, timer_jitter_drift in tests/kernel/timer/timer_behavior does
pass with any CONFIG_SYS_CLOCK_TICKS_PER_SEC value, even when the tick
period is equal or larger than the specified timer period for the test
which failed the test before.
Fixes #54755

Looks like CI failure is related to bug #54720 which has nothing to do with
this PR.

Repushed to restart CI now that the ARC failure workaround is merged.

@peter-mitsis @ceolin @andyross @teburd Your reviews were dismissed. Please re-approve.


ci: no concurrent publish

I think the recent issues we faced with publishing were related to having the changesets/action running in parallel doing two different things. 🤔
Anyway, publishing shouldn’t be executed in parallel and existing jobs shouldn’t be canceled, so let’s add this to the CI workflow.

Yes! We can publish to multiple registries in parallel, but we can’t publish to the npm registry in parallel. Or actually it’s more about not running the changeset/action in parallel, which creates/updates the release branch (= trouble when running in parallel).

Yes! We can publish to multiple registries in parallel, but we can’t publish to the npm registry in parallel. Or actually it’s more about not running the changeset/action in parallel, which creates/updates the release branch (= trouble when running in parallel).

This sounds good to me 👍


TASK 1e: ensure CMake compile groups only include what is necessary

"Ensure CMake compile groups are structured to only include what is absolutely
necessary. Ideally we would like to be able to install the basic MCMC stack using only
eigen, boost, and HDF5. If a c++ compile group is turned off, make sure both its python and test
equivalents are turned off as well."

compile groups have been fixed and can build individually


SI-2707-vehicle-by-sd

find vehicle a synthetic device is connected to

@Allyson-English Giving this a nudge.

@elffjs thanks for nudging! I made those edits on friday but forgot to push them up because I got distracted by cases of if/ else for the before/ after queries. I removed the use of it here because you pointed out that it's legal but noticed that we're also doing it in these spots and was wondering if we wanted to use this opportunity to make both options allowable in those instances too: 1, 2, 3

That's a good catch. I can do that in another PR, no worries.
I don't think it's a very useful option but unlike combining first and last it doesn't seem to do any harm.


Can't start vite dev-server for Cypress Component testing

Description
Adding component test for Cypress and running cypress vite-ssr throws an error
module.exports = defineConfig({
chromeWebSecurity: false,
component: {
devServer: {
bundler: 'vite',
framework: 'react',
},
},
Error Stack
Error<EMAIL_ADDRESS>You stumbled upon a bug in vite-plugin-ssr's source code. Reach out at https://github.com/brillout/vite-plugin-ssr/issues/new or https://discord.com/invite/qTq92FQzKb and include this error stack (the error stack is usually enough to fix the problem). A maintainer will fix the bug (usually under 24 hours). Do not hesitate to reach out as it makes vite-plugin-ssr more robust.
at Object.configResolved (/.../node_modules/vite-plugin-ssr/dist/cjs/node/plugin/plugins/devConfig.js:33:36)
at file:///.../node_modules/vite/dist/node/chunks/dep-c6273c7a.js:62630:39
at Array.map ()
at resolveConfig (file:///.../node_modules/vite/dist/node/chunks/dep-c6273c7a.js:62630:10)
at async Module.createServer (file:///.../node_modules/vite/dist/node/chunks/dep-c6273c7a.js:59000:20)
at async AsyncFunction.createDevServer [as create] (/Users/alya/Library/Caches/Cypress/10.3.1/Cypress.app/Contents/Resources/app/packages/server/node_modules/@cypress/vite-dev-server/dist/devServer.js:34:16)
at async devServer (/Users/alya/Library/Caches/Cypress/10.3.1/Cypress.app/Contents/Resources/app/packages/server/node_modules/@cypress/vite-dev-server/dist/devServer.js:14:20)
From previous event:
at Object.wrapChildPromise (/Users/alya/Library/Caches/Cypress/10.3.1/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/util.js:58:23)
at Object.wrap (/Users/alya/Library/Caches/Cypress/10.3.1/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/dev-server.js:18:8)
at RunPlugins.execute (/Users/alya/Library/Caches/Cypress/10.3.1/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/run_plugins.js:150:26)
at EventEmitter. (/Users/alya/Library/Caches/Cypress/10.3.1/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/run_plugins.js:257:12)
at EventEmitter.emit (node:events:527:28)
at EventEmitter.emit (node:domain:475:12)
at process. (/Users/alya/Library/Caches/Cypress/10.3.1/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/util.js:33:22)
at process.emit (node:events:527:28)
at process.emit (node:domain:475:12)
at process.emit.sharedData.processEmitHook.installedValue [as emit] (/Users/alya/Library/Caches/Cypress/10.3.1/Cypress.app/Contents/Resources/app/node_modules/@cspotcode/source-map-support/source-map-support.js:613:40)
at emit (node:internal/child_process:936:14)
at processTicksAndRejections (node:internal/process/task_queues:83:21)

Thanks for opening the ticket. I'll have a look at it.

It seems like some Vite plugin is setting optimizeDeps.entries?
If you remove the assertion at node_modules/vite-plugin-ssr/dist/cjs/node/plugin/plugins/devConfig.js:33, does it work then?

Yes that works now and does not throw the error


make footer white not gray

(and make sure this turns off gray bg on safari mobile)

maybe add an hr to delineate


Update README.md example file

Adding better examples for our examples/README.md file
Moving original file used by the e2e suite to a new fixtures location.

@degrammer do you have lifecycle identity disabled?
diff --git a/examples/README.md b/examples/README.md
index aaaa4cf..ee8da78 100644
--- a/examples/README.md
+++ b/examples/README.md
@@ -18,7 +18,7 @@ You can author and execute Shell scripts inside a Runme Runbook, making them acc

 Let's get started by running a simple script:

-```sh
+```sh {"id":"01HTZ1P0FQ96BS5P97WJVTDYVK"}
 echo "What are you waiting for 👀 🕰"
 sleep 2
 echo "to write Runbooks 📔"
@@ -32,7 +32,7 @@ Now you get the idea, (isn't that cool ?) let's be creative and think about more
 - Memory usage
 - CPU load

-```sh {"name":"shell-script","terminalRows":"20"}
+```sh {"id":"01HTZ1P0FQ96BS5P97WN0D6BR9","name":"shell-script","terminalRows":"20"}
 #!/bin/bash

 check_disk_usage() {
@@ -77,7 +77,7 @@ Feel free to run the following shell script example, it's similar to the above e
 - Memory usage
 - CPU load

-```sh {"background":"true","name":"interactive-shell-script","terminalRows":"25"}
+```sh {"background":"true","id":"01HTZ1P0FQ96BS5P97WRQY125W","name":"interactive-shell-script","terminalRows":"25"}
 #!/bin/sh

 check_disk_usage() {
@@ -146,7 +146,7 @@ Take a look at the following example:

 ### Install kubectl

-```sh {"background":"true","name":"install-kubectl","terminalRows":"25"}
+```sh {"background":"true","id":"01HTZ1P0FQ96BS5P97WRZMZX2T","name":"install-kubectl","terminalRows":"25"}
 curl -v -o /dev/null "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/darwin/arm64/kubectl"

@@ -157,14 +157,14 @@ You can confidently execute the provided curl command, as the downloaded file wi
Single line
-sh {"name":"set-kubeconfig","promptEnv":"yes","terminalRows":"2"} +sh {"id":"01HTZ1P0FQ96BS5P97WT2SHZG9","name":"set-kubeconfig","promptEnv":"yes","terminalRows":"2"}
#!/bin/bash
export KUBECONFIG=Insert kubeconfig file path

Verify the provided value for the environment var **KUBECONFIG**

-```sh {"name":"check-kubeconfig","terminalRows":"2"}
+```sh {"id":"01HTZ1P0FQ96BS5P97WXEWSKDA","name":"check-kubeconfig","terminalRows":"2"}
echo "KUBECONFIG: $KUBECONFIG"

@@ -172,7 +172,7 @@ echo "KUBECONFIG: $KUBECONFIG"
Runme also supports multiple lines where the export is just somewhere in between:
-sh {"excludeFromRunAll":"true","name":"set-tokens","promptEnv":"yes","terminalRows":""} +sh {"excludeFromRunAll":"true","id":"01HTZ1P0FQ96BS5P97WZF1ACZ9","name":"set-tokens","promptEnv":"yes","terminalRows":""}
echo "Auth token for service foo"
export SERVICE_FOO_TOKEN="foobar"
echo "Auth token for service bar"
@@ -181,33 +181,33 @@ export SERVICE_BAR_TOKEN="barfoo"
verify:
-sh {"interactive":"false","name":"check-tokens"} +sh {"id":"01HTZ1P0FQ96BS5P97X2D50H3D","interactive":"false","name":"check-tokens"}
echo "SERVICE_FOO_TOKEN: $SERVICE_FOO_TOKEN"
echo "SERVICE_BAR_TOKEN: $SERVICE_BAR_TOKEN"

You can also change existing environment variables, like adding changes to `$PATH`:

-```sh {"interactive":"true","name":"export-path"}
+```sh {"id":"01HTZ1P0FQ96BS5P97X4WNDF1B","interactive":"true","name":"export-path"}
export PATH="/some/path:$PATH"
echo $PATH

Supports piping content into an environment variable:
-sh {"name":"export-license"} +sh {"id":"01HTZ1P0FQ96BS5P97X7JZHZGR","name":"export-license"}
export LICENSE=$(cat ../LICENSE)

verify:

-```sh {"interactive":"false","name":"check-license"}
+```sh {"id":"01HTZ1P0FQ96BS5P97XAW73Y97","interactive":"false","name":"check-license"}
echo "LICENSE: $LICENSE"

Support multiline exports:
-sh {"name":"export-privatekey"} +sh {"id":"01HTZ1P0FQ96BS5P97XD0ADQHP","name":"export-privatekey"}
export PRIVATE_KEY="-----BEGIN RSA PRIVATE KEY-----
MIIEpAIBAAKCAQEA04up8hoqzS1+
...
@@ -217,7 +217,7 @@ l48DlnUtMdMrWvBlRFPzU+hU9wDhb3F0CATQdvYo2mhzyUs8B1ZSQz2Vy==
verify:
-sh {"interactive":"false","name":"check-privatekey"} +sh {"id":"01HTZ1P0FQ96BS5P97XDXZXW0B","interactive":"false","name":"check-privatekey"}
echo "PRIVATE_KEY: $PRIVATE_KEY"

@@ -227,7 +227,7 @@ You can use the `interpreter` annotation to use different programming languages

Take a look at the following YAML file for creating a Kubernetes Deployment object:

-```yaml {"interpreter":"cat"}
+```yaml {"id":"01HTZ1P0FQ96BS5P97XEFXXJ89","interpreter":"cat"}
apiVersion: apps/v1
kind: Deployment
metadata:
@@ -256,13 +256,13 @@ Non-shell scripts can also access environment variables, and are run from the cu

Export the following environment variable called __YOUR_NAME__

-```sh {"interactive":"false","promptEnv":"yes"}
+```sh {"id":"01HTZ1P0FQ96BS5P97XFVQD76A","interactive":"false","promptEnv":"yes"}
export YOUR_NAME=enter your name

Run the following python script that reads the environment variable specified in the previous step:
-python {"interpreter":"/usr/bin/python3"} +python {"id":"01HTZ1P0FQ96BS5P97XK2NA9Q0","interpreter":"/usr/bin/python3"}
import os
def main():
@@ -280,7 +280,7 @@ You can visualize static and dynamic images inside your Runbook.
Run the following curl command to render a Kubernetes cluster Grafana dashboard.
-sh {"interactive":"false,","mimeType":"image/png"} +sh {"id":"01HTZ1P0FQ96BS5P97XKKD8Q9T","interactive":"false,","mimeType":"image/png"}
curl -s https://grafana.com/api/dashboards/6417/images/4128/image

@@ -290,13 +290,13 @@ With [`antonmedv/fx`](https://github.com/antonmedv/fx) you can inspect JSON file

Ensure you have **fx** installed (it requires [go](https://go.dev/)), run the following command:

-```sh
+```sh {"id":"01HTZ1P0FQ96BS5P97XP4GTWNZ"}
go install github.com/antonmedv/fx@latest

Now you can explore any json file interactively, run the following command to explore the weather at Berlin 🍻
-sh {"background":"true","promptEnv":"no","terminalRows":"20"} +sh {"background":"true","id":"01HTZ1P0FQ96BS5P97XP4Y4DJ2","promptEnv":"no","terminalRows":"20"}
export FX_THEME="2"
curl -s "https://api.marquee.activecove.com/getWeather?lat=52&lon=10" | fx


Header check

Does GoPostStuff do a header check on the files it posts?

nope

Any plans to implement it?

This is a duplicate of #12

Dupe #12


Fix bug: Line-height issue in [type='file']

@incuna/frontend please merge
It looks like input[type="file"] got there accidentally.
Its sass is in the same file, here:
https://github.com/incuna/incuna-sass/blob/fbf074cf2171e43485148dd8a95dd4f79cb227d3/incuna-sass/components/_forms.sass#L133

Seems like it was intentionally added to match the size of text inputs. See here:
https://github.com/incuna/incuna-sass/commit/6c590756b01d02620d1a18214080bdb346de7ae1#diff-ac0a8f9c204d0a0973a8e3861f58d098R117
What was the reason you needed to remove it?

All the comments there relate to select;
input[type="file"] is set  in another place.
Setting line-height to 28px on file input makes the element misaligned

After removing the 28px line-height on file input it looks like this:

this whole file needs some work really, I have no problem with this being merged

The initial comment refers to select and file elements, so that looks to have been intentional.
The sub comments do not mention file elements, but thats probably an error, as they were added at the same time.
There are input[type="file"] properties elsewhere, but that doesn't mean that these aren't necessary. file has likely been grouped with select as one selector because they both share the same CSS properties. The individual file element section does not seem to set height for example.
My concern is that it looks like this was originally added for some sort of cross browser consistency. If we are removing, we should make sure we aren't removing something that is fixing a problem in other projects or browsers?
Maybe we could do some browser testing on a simple test case of a page importing just these form styles, as we could then be sure these initial ones were correct?

@grahamgilchrist I'll do IE testing and will report it back here


Dbeaver today with error exit code 13 from 8/02/23

Dbeaver has been running for more than 1 year without problems, but today, when opening it, it gives Java exit code 13 error. I have just installed the latest version 6th Feb, which still gives the error.

Thanks for the bug report. This is a known issue in the latest release.
As a workaround, you can either download DBeaver from https://dbeaver.io/files or install JRE 11 or greater.
Closing as a duplicate of #18960

Thanks,  but I had this problem before installing the last version

This issue is reproducible on 22.3.4. As I can see by the attached image, you have this version.

yes, because I installed the latest version when today when I opened the program it gave me an error.  I'm going to do as you said and I will let you know if I have some error. Thanks a lot for your support


[New Feature]: Introduce on Type formating language feature to LS

Description
Format code when typing. The format request textDocument/onTypeFormatting  is trigger by special characters like {, }, ;
Refer https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_onTypeFormatting

+1

Sample use case,
function getJsonProperty(json|error data) returns json|error? => <cursor>

or
function getJsonProperty(json|error data) returns json|error? => <cursor> data is error ? data.message() : data.'worker;

When we press enter at the cursor position above,

current behavior,

function getJsonProperty(json|error data) returns json|error? =>
data is error ? data.message() : data.'worker;

expected behavior,

function getJsonProperty(json|error data) returns json|error? =>
    data is error ? data.message() : data.'worker;

In coding styles, we encourage using expression-bodied functions over block-function-body whenever possible. Therefore, I expect this to be a quite frequent scenario.

We are closing this issue due to inactivity. If you need further assistance or have additional input, feel free to reopen it. Thank you for helping us maintain a relevant and focused issue list!


When creating or editing feedback, weight number is off the screen

On the PickupFeedback form (can be accessed with http://localhost:8080/#/group/1/give-feedback and similar), when entering a weight, the weight number is weirdly moved off the screen:

It should be shown inside the form field.

Came here to say this is a pattern in design called "content jumping" - will try and take a look how to discontinue the issue


mpf.round() returns mpf

Probably this will look non-intuitive, but underlying floating-point arithmetic is binary, not decimal.  Thus, round(x, n) sometimes will pick up a decimal representation with more than n digits.  This is something common for CPython < 2.7 and < 3.1:
Python 2.6.6 (r266:84292, Aug 12 2014, 07:57:07) [GCC 4.4.5] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> round(0.42754287856598971, 2)
0.42999999999999999

And with this patch as well:
Python 3.12.5+ (heads/3.12:0181aa2e3e, Aug 29 2024, 14:55:08) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from mpmath import *
>>> round(mp.mpf(0.42754287856598971), 2)
mpf('0.42999999999999999')
>>> mp.pretty = True
>>> round(mp.mpf(0.42754287856598971), 2)
0.43

See also https://github.com/python/cpython/issues/45921
Closes #455

I'm going to merge this after a week.  But I would appreciate @cbm755 feedback, as new feature might look surprising.


implement a new API

Fixes #568, fixes #564, fixes #563.
Closes #566, closes #506 (obsolete now).

Codecov Report

Merging #613 into master will decrease coverage by 0.19%.
The diff coverage is 80.39%.

@@            Coverage Diff            @@
##           master    #613      +/-   ##
=========================================
- Coverage   90.88%   90.7%   -0.19%
=========================================
  Files          79      79
  Lines        5484    5460      -24
=========================================
- Hits         4984    4952      -32
- Misses        320     328       +8
  Partials      180     180

Impacted Files
Coverage Δ

server.go
91.25% <100%> (+0.52%)
:arrow_up:

client.go
81.68% <61.11%> (-2.63%)
:arrow_down:

h2quic/client.go
78.87% <78.57%> (-9.54%)
:arrow_down:

h2quic/server.go
62.87% <83.33%> (+1.68%)
:arrow_up:

session.go
87.05% <90%> (+0.3%)
:arrow_up:

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 6628994...2c76af3. Read the comment docs.

@lucas-clemente: Done.

LGTM modulo my mutex & renaming comments.
Please also update quic-conn at the same time as merging this.

Merged. I just noticed we forgot to update the CHANGELOG.
We should probably also have another look at our godoc.


downloading from ipfs doesn't work

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
ffmpeg itself supports IPFS protocol - and can deal with ipfs:// links. There is problem with python library - not willing to accept ipfs:// link and pass it to ffmpeg for download.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -vU ipfs://bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8
[debug] Command-line config: ['-vU', 'ipfs://bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8']
[debug] Portable config "C:\ProgramData\scoop\apps\yt-dlp\current\yt-dlp.conf": []
[debug] Encodings: locale cp1250, fs utf-8, pref cp1250, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19044-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 6.0-full_build-www.gyan.dev (setts), ffprobe 6.0-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.17, brotli-1.0.9, certifi-2022.12.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-10.4
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 5590c57bd0433ed239a2deaaf92e2ad6f37fe50f53664c821575cafe106a9421
yt-dlp is up to date<EMAIL_ADDRESS>[generic] Extracting URL: ipfs://bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8
[generic] index_1080p: Downloading webpage
ERROR: [generic] None: Unable to download webpage: <urlopen error unknown url type: ipfs> (caused by URLError('unknown url type: ipfs'))
  File "yt_dlp\extractor\common.py", line 694, in extract
  File "yt_dlp\extractor\generic.py", line 2371, in _real_extract
  File "yt_dlp\extractor\common.py", line 839, in _request_webpage

  File "yt_dlp\extractor\common.py", line 821, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 3742, in urlopen
  File "urllib\request.py", line 525, in open
  File "urllib\request.py", line 547, in _open
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 1425, in unknown_open
urllib.error.URLError: <urlopen error unknown url type: ipfs>

Adding native implementation for an ipfs node is out of scope. We could try supporting this with dependencies. But, would redirecting to https://ipfs.io/ipfs/bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8 be an acceptable solution? That would be way easier to do.

Or https://cloudflare-ipfs.com/ipfs/bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8, etc.
Although an IpfsIE could redirect to a proxy URL as suggested, it would be subject to the operational vagaries of the selected gateway.
Is it not the case that anyone using IPFS seriously already has some sort of proxy, either local or cloudy, and anyone else can form a public proxy URL like the above?
One gateway (ipns.co) returned a page comprising a frameset with a single <frame ...> with the valid HTTPS M3U8 link. Mozilla, at least, treated that as a redirect, but the generic extractor has to be told how:
        frame = self._search_regex(
            r'(?s)<html\b[^>]*>\s*(?:<head\b.*?/head>)?<frameset\b[^>]*>\s*(<frame\b[^>]+>)\s*</frameset>\s*</html>',
            webpage, 'captured frame', default='')
        new_url = url_or_none(extract_attributes(frame).get('src'))
        ...

Or cloudflare-ipfs.com/ipfs/bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8, etc.

Correct me if I am wrong, but I believe ipfs.io is the official site of the protocol/project - hence why I chose it.

It now redirects to ipfs.tech, which still looks like the home page, but offers no explanation of the variable TLD.
I guess that CloudFlare might be better resourced. My etc linked to a list of gateways in the ipfsProxyHTTP project, whereas the IPFS site's list of gateways doesn't seem to be maintained.

ffmpeg and mpv doesn't do native ipfs either, it redirects to http gateway thing. https://github.com/FFmpeg/FFmpeg/blob/master/libavformat/ipfsgateway.c
https://blog.ipfs.tech/2022-08-01-ipfs-and-ffmpeg/
ffmpeg have problems with playing m3u8 from ipfs - https://trac.ffmpeg.org/ticket/10325 - probably links are constructed from m3u8 file incorrectly.
cloudflare ipfs gateway often detect and blocks mp4 movies based on file content, they don't block webm. Detection is not reliable, some mp4 have no problems with that. Also gateways throttle requests. To work around gateway restrictions You can run own ipfs node (kubo). ffmpeg defaults to using own local node instead of public gw.
Problem is that later versions of ipfs started sometimes send back redirects to url like: http://bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq.ipfs.localhost:8080 which only works inside web browser - outside its host not found error. You can't really currently run own ipfs node and then yt-dlp from localhost:8080 without some kind of internal ipfs support in yt-dlp.
So I propose to add some ipfs support using redirect to gateway (local and remote) like ffmpeg have, to gain ability download m3u files from ipfs which are currently pretty painful to download because it have lot of parts and fast ipfs gws will block you for sending too many requests.

ffmpeg makes some effort to find a gateway. Are its environment variables some sort of IPFS convention that should be respected, or could a --ipfs-gateway option with value a URL (or comma-separated URL list?) be sufficient, perhaps with a default?

there are 2 standards how to get file from ipfs gateway old one is using /ipfs/cid url and newer one is cid as subdomain https://github.com/ipfs/in-web-browsers/issues/89 Problem is that sometimes ipfs gw instead of returning requested data it will return redirect to "cid as subdomain" format which will fail unless you have dns wildcard set for localhost.
This prevents software like vlc from working with ipfs while using it as normal url: vlc http://localhost:8080/ipfs/xxx
for example if I sent this: https://cf-ipfs.com/ipfs/bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq I will get back redirect to this: https://bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq.ipfs.cf-ipfs.com
which works because cf-ipfs.com have wildcard dns record but it won't work for localhost outside browser where this magic url format is handled by ipfs browser plugin.

So a local gateway detected according to https://github.com/ipfs/specs/pull/280 may return an unusable proxy URL, but an external gateway won't have this problem (though maybe others, such as media blocking)?

In my test yt-dl code I have this
  Network Options:
    ...
    --ipfs-gateway URL                   Use the specified IPFS gateway

plus a function that gets a default according to the docs linked above, if that  option wasn't passed. A local gateway is meant to prepend its name to the global gateways file while running and should be found from that: there should probably be a liveness check as well as a URL format check in case a gateway disappears without cleaning up. I set up a local gateways file.
$ cat ~/.config/ipfs/gateways
https://ipns.co
$ python -m youtube_dl -v -F 'ipfs://bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8'
[debug] System config: [u'--prefer-ffmpeg']
[debug] User config: []
[debug] Custom config: []If you had a script
[debug] Command-line args: [u'-v', u'-F', u'ipfs://bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: d7b502a72
[debug] Python version 2.7.18 (CPython) - Linux-4.4.0-210-generic-i686-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 4.3, avprobe 4.3, ffmpeg 4.3, ffprobe 4.3
[debug] Proxy map: {}
[IPFS] Extracting URL: ipfs://bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8
[generic] Extracting URL: https://ipns.co/bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8
[generic] index_1080p: Requesting header
WARNING: Falling back on generic information extractor.
[generic] index_1080p: Downloading webpage
[redirect] Following redirect to https://ipfs.eth.aragon.network/ipfs/bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8
[generic] Extracting URL: https://ipfs.eth.aragon.network/ipfs/bafyjvzacdkmdv5iy5ldtibqc6yesvwg3cqmfpdo4clwfrpbc7xdq/index_1080p.m3u8
[generic] index_1080p: Requesting header
[generic] index_1080p: Downloading m3u8 information
[info] Available formats for index_1080p:
format code  extension  resolution note
0            mp4        unknown
$

ipns.co may redirect to various real servers, some giving 404 for the test URL. There's no special retry handling in the code, but a script could add that.

--ipfs-gateway URL                   Use the specified IPFS gateway

Imo, it should just be an extractor-arg since it's for a specific extractor. I know ytdl doesn't yet have them, but they are easy to support and somewhat helps with option bloat.


Variables as Topic

Hi, I'm trying to use variables as topic. I've tryed several syntax without success. Is it a problem related to Grafana or to the plugin itself?
I've installed  the officilia mqtt datasource plugin directly from within grafana 9.3.2
Thanks

That I know, there is no progress on this topic unfortunatly :-(

Same here. I want to use dashboard variables as a topic, but doesn't seem to be supported yet.

Does anybody know if this official Grafana Labs plugin is still supported?
I was trying to use dynamic topic and found this thread... and it look like, as everybody mentionned here so far, there's no way to use variable in the topic.
I already start working on a quick MQTT to websocket bridge (not MQTT over websocket that already exist in many broker implementations) so I can use the websocket datasource plugin instead until the dynamic topic is officially implemented.
Let me know if somebody has more news or ETA about this


Exception when shutting down TensorFlow after running some ops

Error
The following error appears when exiting the Python session after a TensorFlow op has been run in a session:
Exception TypeError: "'NoneType' object is not callable" in <bound method Context.__del__ of <tensorflow.python.eager.context.Context object at 0x7fcd7f98b450>> ignored

The Python session still exits cleanly (status 0).
Repro
$ cat test.py
import sys
import tensorflow as tf
tf.Session().run(tf.square(0))
sys.stderr.write("Done.\n")
$ . tensorflow-nightly-20170814/bin/activate
$ python test.py 2>&1 | grep -Fv 'could speed up CPU computations'
Done.
$ . tensorflow-nightly-20170815/bin/activate
$ python test.py 2>&1 | grep -Fv 'could speed up CPU computations'
Done.
Exception TypeError: "'NoneType' object is not callable" in <bound method Context.__del__ of <tensorflow.python.eager.context.Context object at 0x7f9d8795c450>> ignored

Most ops seem to trigger this: I tried tf.square, tf.add, and tf.summary.scalar and tf.summary.text, and they all do. However, tf.constant does not.
Introduction
Bug introduced between 20170814 and 20170815 nightlies. (In particular, build 588 does not have the bug, while build 589 does.) The changelog for these two builds is b93fd37e143bcdd6339f8e6081c948384a262e0b...1d33a59a9d554be863cfb06e9aa0ce1fa33ce9e6, which does include changes to eager.

Maybe not related, but shouldn't you close() the tf.Session object to ensure safe termination?

@carlthome: Unfortunately, python -c 'import tensorflow as tf; sess = tf.Session(); sess.run(tf.square(0)); sess.close()' exhibits the same behavior, as does python -c $'import tensorflow as tf\nwith tf.Session() as sess: sess.run(tf.square(0))'.

This behavior no longer happens in tensorflow-nightly-20170816:
$ python -c 'import tensorflow as tf; tf.Session().run(tf.square(0))'
2017-08-16 09:48:29.949554: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA

Build 591 has the bug, while build 592 does not. The diff is d480ac587a0040acfc21bc2f63faa12cfd3194cc...3686ef0d51047d2806df3e2ff6c1aac727456c1d.
From the commit description, I suspect that 9e6183e4a fixed this.


cpu serial

"To Be Filled By O.E.M."
Found on an ordinary machine, definetely not VM/sandbox

I will edit the list manually.

Thanks for the comment

Frank VM machine actually uses "to be filled by oem" for cpuid: but I have removed it from the list. It might get on the list later again...


Search: add tests for XSS

In https://github.com/readthedocs/addons/commit/33989f1ff2962bd197b08096b9122cba3dcf1302 we fixed a XSS vulnerability, we should have tests for it, so it doesn't happen again.

I tried to quickly write this test but it's not easy. We don't have any test currently for the search addon and they are not trivial to write. I will need to spend some time working on the initial test pattern for this addon before being able to write this particular test case.


Error invalid option

I get the following error:
./configure: error: invalid option "no-camellia"

I close this, cause I will just use the arch linux nginx, which is up to date.


Make ExternalStudioHost be its own resource

The following is from @petero-dk in #1.
Why would I want this:
The same sanity project is used in a couple of different deployment pipelines (think staging / prod / dev) and for the Sanity project only one would be able to have the resource of the project (which i don't want) so I have linked the Sanity project as a data object in terraform.
However for my production pipeline, I would like to set the external host name as that is dependent on other resources in that pipeline.
Currently that is not possible because it is available only on the project resource.

My initial concern with extracting ExternalStudioHost into its own resource is that there can only be one external studio host per project. Creating multiple instances would interfere with each other.
However, this is exactly how the aws_sqs_queue_redrive_allow_policy works ref. If you create two instances one overwrites the other, and if you delete one, it removes the policy from the queue altogether.
I think it is reasonably to think that people with the use case described in the original write-up will be aware if there are more than one ExternalStudioHost resource is created (or at least, they will figure out out really quickly).

I was also concerned that the change here would not be backwards compatible, and it was not clear to me how many people would be affected.
However, in doing some initial research, I have found a bug in the underlying go-sanity library that has prevented anyone from actually using the ExternalStudioHost option to-date.
While the proposed change will technically break backwards compatibility, it won't in practice.

Here is a short example of what I think the API should look like.
resource "sanity_project" "my_blog" {
  name        = "My blog"
  color       = "#0000ff"
  studio_host = "my-cool-blog"
}

resource "sanity_project_external_studio_host" "my_blog" {
  project = sanity_project.my_blog.id
  host    = "https://studio.example.com"
}

project is already used in other resources, and host is similar with studio_host in the project resource.
Does this make sense to you, @petero-dk?

I think that looks excellent, just to make sure that would work with a data project object also?

Yes, good call out, thanks.
I will use this to start going through your PR later this evening.


Benchmark different databases

lmdb vs rocksdb and more

lmdb seems to be noticeably faster for me than leveldb when I was woking on katnip v.2.
When I observed the process of importing data from kaspad to katnip it seemed to me it seemed to me that the bottleneck was reading data from leveldb (rather than writing to lmdb).


Add correct MIT licensing on all files

Add the correct licensing on files in this repo.

This is fixed.


What is the difference between a product's "purchaseCost" and its variants' "cost price"?

What I'm trying to achieve
I am working with saleor release 2.8.0.
I am trying to build up my products list in a consistent way. In particular, I need a clear relationship between what a product cost me and how much it will cost my buyers.
In the GraphQL API, I can find the field purchaseCost in the product query. That field is protected by the manage_products permission. Is that purchaseCost the amount I paid as a vendor to get that product in my shop?  To be more specific, let's say I have an e-commerce platform grouping the products I get from some people. I want to gain money on those products, so that I sell them on my platform a bit more expensive than what those people sold them to me. Is the purchaseCost the price those people sold me their products?
I can also find a field costPrice in the variants field of the product query. That field is not protected with the manage_products permission. I expect this field to be related to the product's purchaseCost. Therefore I expect that field to be protected by the same permission.
But I might've misunderstood the purpose of those fields. What is the difference between a product's purchaseCost and a its variant's costPrice?
My assumption until now is that purchaseCost has the same relationship to the variants' costPrice as the product priceRange has to the variants' pricing. If that holds true, however, I don't understand why the variants costPrice isn't protected with the product.manage_products permission.

purchaseCost is a money range. The lowest purchase cost to the highest one of the product variants.
They should definitively be protected, yes. Thank you for the issue!


Trying to download SQL Server on Linux

I get Unable to locate package mssql-server
Can you help me out?

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

ID: 5634d572-0020-808b-693a-13addc8861d9
Version Independent ID: 136bab7d-d4d1-1e82-ba89-be2020bc784b
Content: Get started with SQL Server on Ubuntu
Content Source: docs/linux/quickstart-install-connect-ubuntu.md
Product: sql
GitHub Login: @rothja
Microsoft Alias: jroth

@jszazzing Hello. I'm sorry you're having trouble getting SQL Server. Can you paste in the commands you run and the error you get into this thread? I'll see if there is something wrong with the steps in this Ubuntu quickstart. Thank you.

root@ubuntu:/home/steven# sudo apt-get install -y mssql-server
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package mssql-server
On Mon, Dec 10, 2018 at 4:52 PM Jason Roth<EMAIL_ADDRESS>wrote:

@jszazzing https://github.com/jszazzing Hello. I'm sorry you're having
trouble getting SQL Server. Can you paste in the commands you run and the
error you get into this thread? I'll see if there is something wrong with
the steps in this Ubuntu quickstart. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/1396#issuecomment-446005049,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ags3JwFGOzBB7ipwvI05Cs0lrwnd4rYrks5u3uWzgaJpZM4ZLP-d
.

root@ubuntu:/home/steven# sudo add-apt-repository "$(wget -qO-
https://packages.microsoft.com/config/ubuntu/16.04/mssql-server-2017.list)"
root@ubuntu:/home/steven# sudo apt-get update
Hit:1 http://us.archive.ubuntu.com/ubuntu xenial InRelease
Get:2 http://security.ubuntu.com/ubuntu xenial-security InRelease [107 kB]
Hit:3 http://us.archive.ubuntu.com/ubuntu xenial-updates InRelease
Get:4 http://us.archive.ubuntu.com/ubuntu xenial-backports InRelease [107
kB]
Hit:5 https://packages.microsoft.com/ubuntu/16.04/mssql-server-2017 xenial
InRelease
Ign:6 https://download.webmin.com/download/repository sarge InRelease
Hit:7 https://packages.microsoft.com/ubuntu/16.04/mssql-server-preview
xenial InRelease
Hit:8 https://download.webmin.com/download/repository sarge Release
Fetched 214 kB in 0s (290 kB/s)
Target Packages (main/binary-amd64/Packages) is configured multiple times
in /etc/apt/sources.list:55 and /etc/apt/sources.list.d/mssql-server.list:1
Target Packages (main/binary-all/Packages) is configured multiple times in
/etc/apt/sources.list:55 and /etc/apt/sources.list.d/mssql-server.list:1
Target Translations (main/i18n/Translation-en_US) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
Target Translations (main/i18n/Translation-en) is configured multiple times
in /etc/apt/sources.list:55 and /etc/apt/sources.list.d/mssql-server.list:1
Target DEP-11 (main/dep11/Components-i386.yml) is configured multiple times
in /etc/apt/sources.list:55 and /etc/apt/sources.list.d/mssql-server.list:1
Target DEP-11-icons (main/dep11/icons-64x64.tar) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
Reading package lists... Done
W: Target Packages (main/binary-amd64/Packages) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target Packages (main/binary-all/Packages) is configured multiple times
in /etc/apt/sources.list:55 and /etc/apt/sources.list.d/mssql-server.list:1
W: Target Translations (main/i18n/Translation-en_US) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target Translations (main/i18n/Translation-en) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target DEP-11 (main/dep11/Components-i386.yml) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target DEP-11-icons (main/dep11/icons-64x64.tar) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target Packages (main/binary-amd64/Packages) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target Packages (main/binary-all/Packages) is configured multiple times
in /etc/apt/sources.list:55 and /etc/apt/sources.list.d/mssql-server.list:1
W: Target Translations (main/i18n/Translation-en_US) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target Translations (main/i18n/Translation-en) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target DEP-11 (main/dep11/Components-i386.yml) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
W: Target DEP-11-icons (main/dep11/icons-64x64.tar) is configured multiple
times in /etc/apt/sources.list:55 and
/etc/apt/sources.list.d/mssql-server.list:1
root@ubuntu:/home/steven# sudo apt-get install -y mssql-server
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package mssql-server
On Wed, Dec 12, 2018 at 8:00 AM Steven Bailey<EMAIL_ADDRESS>wrote:

root@ubuntu:/home/steven# sudo apt-get install -y mssql-server
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package mssql-server
On Mon, Dec 10, 2018 at 4:52 PM Jason Roth<EMAIL_ADDRESS>wrote:

@jszazzing https://github.com/jszazzing Hello. I'm sorry you're having
trouble getting SQL Server. Can you paste in the commands you run and the
error you get into this thread? I'll see if there is something wrong with
the steps in this Ubuntu quickstart. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/1396#issuecomment-446005049,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ags3JwFGOzBB7ipwvI05Cs0lrwnd4rYrks5u3uWzgaJpZM4ZLP-d
.

@jszazzing. Thank you. I've got a machine setup to test now. Are you on Ubuntu 18.04? 16.04? Something else? Thanks.

@jszazzing, on my Ubuntu machine, I don't get this error. I'm not sure what the difference could be. I see you're running the commands as root. You could try repeating all the commands run as a regular user. The sudo is suggested in places where you need the elevated permissions. Sorry!

@jszazzing One other thing to try is to run SQL Server in a Docker Container. There is almost no difference between running in a Docker container versus running locally, and the Docker container should always run consistently no matter what. Check out the Docker quickstart in the documentation as well as the Docker Configuration Guide that it points to. If you do use this for your SQL Server running on your local machine, you'll want to use one of the persistence options for your data. That way if your container is removed, you still have your data, and you can start a new container that is attached to that data directory.

@jszazzing , I'm sorry we couldn't get your deployment working. If you continue to have issues on your machine, our support team might be able to further investigate. Or you might be able to find help on the forums. However, I think it is also possible that you'll find the Docker containers with Developer Edition to be a good solution for your scenario. Are you OK if I close this Git Issue out at this time? Thank you!

https://github.com/MicrosoftDocs/sql-docs/labels/Issue-Status-2-Author-will-triage


feat(#784): Skip Already Translated Files

In this PR I added some sort of caching. If assemble,disasseble or unroll goals see already translated
file (target last modification time > source last modification time), then they just skip transformation.
Related to #784.

PR-Codex overview
This PR focuses on refactoring the Disassemble and Assemble classes to use a Transformation interface for better abstraction and maintainability. It also introduces caching for transformations to avoid redundant processing.
Detailed summary

Refactored apply methods in Disassemble and Assemble to use Transformation interface.
Introduced Caching, Disassembling, and Assembling classes implementing the Transformation interface.
Updated BatchedTranslatorTest to accommodate changes in transformation logic.
Modified Unroll class to implement Transformation.
Added CachingTest for testing cached transformations.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 12min).


Final fix for loading Accucor corrected sheets from csv

Summary Change Description
This makes a csv or the accucor corrected sheet actually work.
Details:

A isotopeLabel column is added to the corrected sheet, based on the C_Label column.
The Original sheet was removed from AccucorLoader.OrigDataRequiredHeaders.
"formula" is only added to the get_or_create for the PeakGroup model if it has a value.
medMz, medRt, and formula were removed from the required headers and values of the Unicorr format.

Affected Issues/Pull Requests

Resolves no documented issue
Merges into PR #1309

Review Notes

See comments in-line.
Checklist

This pull request will be merged once the following requirements are met.  The
author and/or reviewers should uncheck any unmet requirements:

Review requirements

Minimum approvals: 1
No changes requested
All blocking issues resolved by reviewers
Specific reviewers: @add_username_here

Review period: 2 days

Associated issue/pull request requirements:

[x] All requirements in affected issues marked "resolved" are satisfied
[x] All required pull requests are merged (or none)

Basic requirements

[x] All linters pass
[x] All tests pass
[x] All conflicts resolved

Overhead requirements

[x] New/changed method tests implemented/updated (or no change)
[ ] Updated Unreleased section of changelog.md (or no change)
[x] Migrations created & committed (or no change)

These changes are pretty straightforward, but I know @fkang-pu has more experience with pandas methods and could catch issues where perhaps I didn't use the best strategy, so I am tagging you for a review here.

Hey @fkang-pu - I fixed it. Could you have another look?  The running of the load in my sandbox is waiting on Michael's assessment of an issue in the data, but I ran the load of Lingfan's data in dry run mode and it succeeds with this code:


Update Outlook.Recipients.Add.md

Fixed code formatting in the example.

Docs Build status updates of commit 357b751:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Outlook.Recipients.Add.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 93151d6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Outlook.Recipients.Add.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Demo site does not send user to User Home after login

Steps to replicate

on demo site, click on "log in with Tonomy ID"
Scan QR code to log into Demo website
Press Next on consent screen

expected behaviour

User is logged into the demo site and can now navigate around the user homepage

actual behaviour

The SSO page redirects the user back to the Demo site, to the home page which tries to request the Block chain account repeatedly from the API

script: https://www.notion.so/tonomy-foundation/Sales-pitch-video-76032e6afee74add904dee481f68d905
slides: https://www.canva.com/design/DAFkSNyyOB4/l-QgnQVwgYwapjXO-WgxdQ/edit