digitalconcerthall extractor broken

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 reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.10.13 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[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
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Slovakia
Provide a description that is worded well enough to be understood
It seems the m3u8 url format has been changed.
Manual download using m3u8 link still possible:
yt_dlp https://world-vod.dchdns.net/shlss/dch/XXXXX-1/,h264_LOW_TWO,h264_LOW_ONE,h264_MEDIUM_TWO,h264_MEDIUM_ONE,h264_HIGH,h264_VERY_HIGH_ONE,flac_AUDIO_STEREO_LOSSLESS,eac3_AUDIO_IMMERSIVE_LOSSY,.mp4.urlset/master.m3u8?token=exp=123456687~acl=/shlss/dch/55029-1/*~hmac=xxxx.xxxx.xxx
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] 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
[debug] Command-line config: ['-o', '%%(playlist)s/%%(playlist_index)s-%%(title)s.%%(ext)s', '--username', 'PRIVATE', '--password', 'PRIVATE', 'https://www.digitalconcerthall.com/en/concert/55029', '-f', 'wv+audio0-flac', '--postprocessor-args', '-strict -2', '-vU']
[debug] Encodings: locale cp1250, fs utf-8, pref cp1250, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b634ba742]
[debug] Lazy loading extractors is disabled
[debug] Python 3.11.3 (CPython AMD64 64bit) - Windows-10-10.0.22621-SP0 (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: ffmpeg 4.2.3
[debug] Optional libraries: certifi-2023.07.22, sqlite3-3.40.1
[debug] Proxy map: {}
WARNING: Post-Processor arguments given without specifying name. The arguments will be given to all post-processors
[debug] Loaded 1890 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[DigitalConcertHall] Obtaining token
[DigitalConcertHall] Logging in
[DigitalConcertHall] Extracting URL: https://www.digitalconcerthall.com/en/concert/55029
[DigitalConcertHall] 55029: Downloading webpage
[DigitalConcertHall] 55029: Downloading JSON metadata
[download] Downloading playlist: A Mozart concert with Riccardo Minasi
[DigitalConcertHall] 55029-1: Downloading JSON metadata
WARNING: [DigitalConcertHall] Failed to obtain m3u8 URL; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[DigitalConcertHall] 55029-2: Downloading JSON metadata
WARNING: [DigitalConcertHall] Failed to obtain m3u8 URL; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[DigitalConcertHall] 55029-3: Downloading JSON metadata
WARNING: [DigitalConcertHall] Failed to obtain m3u8 URL; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[DigitalConcertHall] 55029-4: Downloading JSON metadata
WARNING: [DigitalConcertHall] Failed to obtain m3u8 URL; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[DigitalConcertHall] 55029-5: Downloading JSON metadata
WARNING: [DigitalConcertHall] Failed to obtain m3u8 URL; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[DigitalConcertHall] Playlist A Mozart concert with Riccardo Minasi: Downloading 5 items of 5
[download] Downloading item 1 of 5
ERROR: [DigitalConcertHall] 55029-1: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1567, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2028, in __process_iterable_entry
    return self.process_ie_result(
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1782, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2751, in process_video_result
    self.raise_no_formats(info_dict)
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1076, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [DigitalConcertHall] 55029-1: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

[download] Downloading item 2 of 5
ERROR: [DigitalConcertHall] 55029-2: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1567, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2028, in __process_iterable_entry
    return self.process_ie_result(
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1782, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2751, in process_video_result
    self.raise_no_formats(info_dict)
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1076, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [DigitalConcertHall] 55029-2: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

[download] Downloading item 3 of 5
ERROR: [DigitalConcertHall] 55029-3: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1567, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2028, in __process_iterable_entry
    return self.process_ie_result(
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1782, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2751, in process_video_result
    self.raise_no_formats(info_dict)
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1076, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [DigitalConcertHall] 55029-3: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

[download] Downloading item 4 of 5
ERROR: [DigitalConcertHall] 55029-4: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1567, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2028, in __process_iterable_entry
    return self.process_ie_result(
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1782, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2751, in process_video_result
    self.raise_no_formats(info_dict)
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1076, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [DigitalConcertHall] 55029-4: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

[download] Downloading item 5 of 5
ERROR: [DigitalConcertHall] 55029-5: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1567, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2028, in __process_iterable_entry
    return self.process_ie_result(
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1782, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 2751, in process_video_result
    self.raise_no_formats(info_dict)
  File "E:\progs\yt-dlp_202311\yt_dlp\YoutubeDL.py", line 1076, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [DigitalConcertHall] 55029-5: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

[download] Finished downloading playlist: A Mozart concert with Riccardo Minasi

Editing extractor/digitalconcerthall.py line 100 to this do the trick for now.
m3u8_url = traverse_obj(
                stream_info, ('channel', 'temporary_webapp_channel', 'stream', 0, 'url'), get_all=False)

It seems that the json data structure has changed, but not sure if it is temporary or permanent.
New structure shown below:
{
  "updated":<PHONE_NUMBER>,
  "channel": {
    "temporary_webapp_channel": {
      "codec": "mixed_h264_flac_eac3",
      "streaming_technique": "hls_abr_temp",
      "has_audio_only": true,
      "max_resolution": "2160p",
      "dynamic_range": "sdr",
      "is_encrypted": true,
      "stream": [
        {
          "quality": null,
          "audio_language": null,
          "video_language": null,
          "url": "https://world-vod.dchdns.net/shlss/dch/55031-4/,h264_LOW_TWO,h264_LOW_ONE,h264_MEDIUM_TWO,h264_MEDIUM_ONE,h264_HIGH,h264_VERY_HIGH_ONE,flac_AUDIO_STEREO_LOSSLESS,.mp4.urlset/master.m3u8?token=exp=1699481098~acl=/shlss/dch/55031-4/*~hmac=4fa03f1740c3b59c14fddec0d3df8518604a864f77f73311e4121fdfbab2d9e8"
        }
      ]
    }
  },
  "_links": {
    "self": {
      "href": "//api.digitalconcerthall.com/v2/streams/55031-4"
    }
  }
}

Hey can I pick this issue up? It is my "first issue". Is that okay?

Hey can I pick this issue up? It is my "first issue". Is that okay?

Yes

I can confirm that the json has returned to the original, and works with current release. Presumably no fix should be deployed.


Speculative processing

Description
This PR adds support for speculative processing on top of end-to-end timeout to improve availability within timeout
Introduces Threshold based speculative execution
Only for Reads / Queries
If we don't get a successful response (either delayed response or error response) within the threshold value, we go to all preferred regions in parallel.
- We can try one region at a time, with some linear timeline.
- Whichever responds first, we will return, and cancel all the other ongoing requests.
Benefits:
1. Better chances of getting a response from either of those regions.
2. This provides Maximum availability at RUs / compute tradeoff (increased cost)
flowchart TD
    A[Request] -->B(Base Request with timeout policy)
    B --> Threshold{timespent < speculativeThreshold}
    Threshold -->| Yes | C{success <= EndToEndTimeout}
    Threshold --> | No | RemoteProcessing[Start Parallel remote requests to other regions]
    C -->|Yes| D[Return result]
    C -->|No response| E[Cancel and timeout]
    C --> |error response| F[Retry Flow]
    F --> G{timespent < EndToEndTimeout}
    G --> |No| E
    G --> |Yes| F

If speculative retries are enabled, based on the probability threshold Speculative retries may increase RU costs
When we have sparse data, the algo adds more variance around mean and this may increase the exploration even though the lower mean value is selected until the perf guarantee is increased. (The algo has proofs that this keeps cost under consideration and does only necessary exploration)
Current algo is tuned for 10sample requests. It can be tuned based on time/ number of requests and can be adjusted accordingly
Initial test resuls

Tests:

Read
document 100 times
30s of
delay faults on the local region

| P99 in   ms | P95 in   ms | P75 in   ms |
-- | -- | -- | -- | --
No   Speculation | 2078.12109375 | 2028.05859375 | 2016.05859375 |
Threshold   based Speculation | 1134.046875 | 616.015625 | 298.0 |
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-cosmos

/azp run java - cosmos - tests

/check-enforcer override

/check-enforcer override

/check-enforcer override

/check-enforcer override

/check-enforcer override

/check-enforcer evaluate

/check-enforcer reset

/check-enforcer override


Problems with gitcommit for a repository in a sub-directory

I'm attempting to use gitcommit against a repository in a sub-directory within my project. The cwd option doesn't seem to be working properly.
My gitcommit task looks like this:
gitcommit: {
	dist: {
		options: {
			expand: true,
			cwd: 'dist',
			message: 'v<%= pkg.version %>'
		},
		files: [{
			expand: true,
			cwd: 'dist',
			src: ['**/*']
		}]
	}
},

And when I run it, I get something like this:
Running "gitcommit:dist" (gitcommit) task
Warning: fatal: pathspec 'dist/about.html' did not match any files Use --force to continue.

Aborted due to warnings.

Or, with verbose mode:
Running "gitcommit:dist" (gitcommit) task
Verifying property gitcommit.dist exists in config...OK
Files: dist/whatsnew.html -> whatsnew.html
Options: message="v0.0.84", ignoreEmpty=false, noVerify=false, noStatus=false, expand, cwd="dist"
Options: verbose=false, expand, cwd="dist", message="v0.0.84"
Warning: fatal: pathspec 'dist/about.html' did not match any files Use --force to continue.

Aborted due to warnings.

It appears that the dist/ portion is still being included despite my cwd, given that the file is in the appropriate location:
djmccormick at computer in ~/Projects/some-category/some-project on some-branch
± ls dist/

about.html

Am I doing something incorrectly or is this a bug?
node v0.10.28
grunt-cli v0.1.13
grunt v0.4.4
grunt-git v0.2.14

any news on this issue? grunt is still giving us the cwd relative to the gruntfile?

I've found another possible workaroud, which is ugly too:
var rootPath = process.cwd();

grunt.registerTask("cd", function(subpath) {

	var path = require('path');

	if(env === 'restore') {
		return grunt.file.setBase(rootPath);
	}

	return grunt.file.setBase(path.resolve('my/subdir/' + subpath));

});

grunt.registerTask("taskName", function(env) {
		grunt.task.run("cd:" + env);

		grunt.task.run("gitreset:" + env);
		grunt.task.run("gitpull:" + env);

		grunt.task.run("cd:restore");

		grunt.task.run("othertasks...");
});


Support CustomError inheritance in vm.expectRevert(abi.encodeWithSelector(Parent.myCustomError()))

Component
Forge
Describe the feature you would like
Given parent and child contracts:
contract Child {
  error myCustomError()
}

contract Parent is Child {
  function revertingFunction() {
   revert myCustomError();
  }
}

The following throws a compiler error vm.expectRevert(abi.encodeWithSelector(Parent.myCustomError())). With large nested systems, would be very helpful to not force specific call out of Child (e.g. vm.expectRevert(abi.encodeWithSelector(Child.myCustomError())).
Additional context
No response

I'm not sure I can follow here.
Isn't this the same error? Does the selector differ here?
FYI, we also just made expectRevert() catch any reverts regardless of selector

Ah thanks for the clarification @mds1 - you're right, looks like even though you can inherit a parent and call the error without the parent reference, in Solidity you do need to specify the exact child contract name when pulling the selector (e.g. Child.myCustomError.selector).
I think for now, going with the explicit reference to child will do.


Cannot find module 'bridge' on Android

Currently, npm install inside the nodejs project directory creates a symlink to the bridge module included in the capacitor-nodejs package. When syncing the application with npx cap sync android, the symlink is copied unchanged to the native project, resulting in a symlink pointing to a non-existent path.
A possible solution would be to pack the bridge module and provide a tarball for installation, but this would impact the current implementation of the Electron platform.
As a workaround, install the bridge module with the --install-links option.

Should be fixed in v1.0.0-beta.4


🛑🛑🛑Please contact customer support🛑🛑🛑

This GitHub repository is dedicated to handling issues and feedback for the latest Developer Preview releases of the AL development tools for Dynamics 365 Business Central.
We suggest that you open a support case or file a bug in Collaborate, to ensure that all Business Central users benefit from your catch as soon as possible.
You can open Support Request to CSS through PartnerSource portal or contact your Service Account Manager (SAM) in the local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also step by step direct you how to open a support request or how to get credentials if this is the first time for you / your company.

Aged Accounts Payable report format changed with Dynamics BC update v
Aged Accounts Payable Print Screens.docx
17.1.
1000 Separator (,) missing from totals and Subtotals.
Trailing zeros after decimal not showing.


Improve TermsSuggestions to not flag "By using"

This will close issue #514
Happy Hacktoberfest!

Hopefully good now.


A KeyUpEvent is dispatched, but the state shows that the physical key is not pressed. If this occurs in real application, please report this bug to Flutter. If this occurs in unit tests, please ensure that simulated events follow Flutter's event model as documented in HardwareKeyboard. This was the event: KeyUp Event#a91ee(physicalLy: PhysicalKeyboardKey#e6f4b(usbHidUsage: "0x0007002a", debugName: "Backspace"), logicalLy: LogicalKeyboardKey#147d9(keyId: "0x100000008", keyLabel: "Backspace", debugName: "Backspace"), character: null, timeStamp: 0:00:04.455957, synthesized)

it happens when i press the backspace button to delete some character

Hi @sheby460
There are also known issues in the database but I'm not sure which one is similar to this. So please try to provide all or some of the information indicated in the template so that we can address this issue properly (Report a bug).
And you should also look through filed issues in this repository (see flutter/issues) to see if there are any similar issues (reproducibility steps, target environment, stack trace, etc). Once you found it, please close this issue and follow it. Thanks!


[Bug]: Permission error when installing v0.4.0 on Windows

What happened?
pip install chromadb==0.4.0
Installing collected packages: chroma-hnswlib, starlette, fastapi, chromadb
ERROR: Could not install packages due to an OSError: [WinError 5] Access is denied: 'C:\\Users\\kaedenbrinkman\\AppData\\Local\\Packages\\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\\LocalCache\\local-packages\\Python311\\site-packages\\hnswlib.cp311-win_amd64.pyd'
Check the permissions.

I verified that the file exists and that my account has full permissions to it.
Versions
Chroma v0.4.0, Python v3.11, Windows 11 22H2
Relevant log output
Installing collected packages: chroma-hnswlib, starlette, fastapi, chromadb
ERROR: Could not install packages due to an OSError: [WinError 5] Access is denied: 'C:\\Users\\kaedenbrinkman\\AppData\\Local\\Packages\\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\\LocalCache\\local-packages\\Python311\\site-packages\\hnswlib.cp311-win_amd64.pyd'
Check the permissions.

I should mention that pip install chromadb==0.3.29 works without any issues.

Hi @kaedenbrinkman, I just tried this on a machine with Windows 11 22H2, however I used python 3.10.11, as we don't support 3.11 currently. It worked correctly for me and chroma-hnswlib installed with no fault. Can you maybe try installing in a clean virtual environment, this seems like an environment issue and not an issue with chroma, or its packaging (although I could be wrong!)

Can you try the permission workaround mentioned here. https://github.com/chroma-core/chroma/issues/250#issuecomment-1540934224

I have 'MSVC v143 - VS 2022 C++ x64/x86 build tools' installed, just tried running pip install chromadb==0.4.0 --user and had the same issue.

Sorry, missed your first comment about switching to python 3.10.11 - will try that and will let you know how it goes. Thanks!

Closing this as it is stale. Please let me know if anything else pops up here and we can re-open it.


Assigning nbt to variable doesn't compile

$var = storage::var;

doesn't work

Should be fixed in https://github.com/WingedSeal/jmc/commit/e00d6df869eaea4388e169b0318ebe21ba7d66f9


fix: Documantation

Close: https://github.com/alexcrichton/bzip2-rs/issues/68

fixed by https://github.com/trifectatechfoundation/bzip2-rs/pull/114
The situation is a bit more nuanced than just deleting the paragraph, so i did not cherry-pick this commit.


feature request: have option for autoplaying audio and video on cell evaluation

As in title.

Hmmm, this is a bit tricky to do (I dont really think it's possible without changing the audio tag to use html + javascript). We are at the mercy of ipython's API here.


add community changes, remove cart

-show fields in install.JSON without email present
-remove tax description
-add in custom amount option for every button type
-allow setting of default donation amount
-make subscription payment information read more clearly
-remove cart option
-fix minor css errors


Added formatters "state" and "stateAbbr" support in Simplified Chinese(zh_CN).

Also formatted the random data.

Thanks!


Lombok

Replaced all getters and setters (and some constructors) with Lombok. Equals methods have not been touched because there are in classes with extend another class, in which case generating the equals is disencouraged. Constructors which call super have also not been changed because I'm not sure how that works with Lombok.

All hail Lombok 💯 . No seriously, the code is much cleaner and more readable this way. I can understand that the equals methods would give problems if you would generate them, these are harder to generate than the getters/setters. Lets merge this PR for cleaner code! :)

This indeed removes the boilerplate code of the classes and therefore it is now easier to see which part of a class is really relevant / important. I think we can merge this.

Yes, now our code is more concise, now it is easier to find the code that we are looking for, it's nice to merge this PR.


Failed to print an element

The example in User Guide 7.7.1 cannot be run successfully. The code is
import cytnx
T = cytnx.UniTensor(cytnx.arange(9).reshape(3,3))
print(T.at([0,2]).value)

and here is the error message.
Traceback (most recent call last):
File "test.py", line 4, in
print(T.at([0,2]).value)
File "/home/chiamin/miniconda3/envs/cytnx/lib/python3.8/site-packages/cytnx/utils.py", line 6, in wrapper
return func(self,*args, **kwargs)
File "/home/chiamin/miniconda3/envs/cytnx/lib/python3.8/site-packages/cytnx/UniTensor_conti.py", line 216, in at
tmp_hclass = self.c_at(locator);
TypeError: Unable to convert function return value to a Python type! The signature was
(self: cytnx.cytnx.UniTensor, locator: List[int]) -> cHclass

In Cytnx version 0.9.1 this example works fine for me

My version is 0.9.0. Is there an easy way to update the version? I have run conda update cytnx but still in 0.9.0. Here is the output of conda list cytnx
cytnx                     0.9.0            py38h3fd9d12_0    kaihsinwu

I had to set up a new environment from scratch, conda update did not work for me either.

Actually I don't know why conda update won't work sometimes if anyone wants to investigate feel free

We should create another issue to address conda packaging. I will close this one


Small test fixes

Removes the example-go app that was accidentally checked in to version control. Also stops checking for optional DEIS_WORKFLOW_SERVICE_PORT and doesn't hard-code the location of the deis CLI.

thanks @mboersma


How to remove space after if(..), loops and functions before the opening brackets?

"pad-header": false,
"unpad-paren": true,
http://astyle.sourceforge.net/astyle.html#_pad-oper


Support time-group feature

Related:

https://github.com/coq-community/docker-coq/issues/4
https://github.com/erikmd/docker-coq-github-action-demo/pull/2

Also:

Add .gitignore
Add dockerignore

(I can do a rebase -i if need be as several commits are just fixups...)

(I can do a rebase -i if need be as several commits are just fixups...)

done

With the last commits the init_opts=$-; set +x noise caused by start/end-Group internals is fixed:

(you may remark that sometimes the STDOUT/STDERR logs are not exactly sync, but this seems to be a general issue with GitHub actions logging buffering, so there's nothing we could do in docker-coq-action, except possibly an exec 2>&1, but maybe that'd be overkill for the kind of issue at stake here(?))
Anyway the PR looks ready so @Zimmi48 I propose to merge it tonight.


Encrypted ClientHello - Missing?

System Details

OS:  Windows 10
Thorium Version 122.0.6261.132 (Official Build) (64-bit)

Problem
Cannot pass Secure SNI on cloudflare test using Thorium when using dnscrypt-proxy, works fine with Firefox, Chrome and Chromium
Check for Encrypted ClientHello flags which exist in Chrome/Chromium - but none exist in Thorium

Issue also affects Thorium for legacy Windows versions https://github.com/Alex313031/thorium-legacy/issues/72

This issue seems to be related to the new algorithm recently deployed by Google, please try going to chrome://flags and turn enable-tls13-kyber off

This issue seems to be related to the new algorithm recently deployed by Google, please try going to chrome://flags and turn enable-tls13-kyber off

Afraid its a still no goer.....

Have you updated to version M123? In addition, related problems may not be improved until the M124 version.
At the same time, this issue may also be related to some patches we use, and I need Alex to verify this issue.
@Alex313031

Have you updated to version M123? In addition, related problems may not be improved until the M124 version.
At the same time, this issue may also be related to some patches we use, and I need Alex to verify this issue.
@Alex313031

Just tried latest (M123), no change sorry

@gz83 @narinishi @adrianmmiller @eltociear I think this is related to the two DNS patches we use. One is from Ungoogled, the other is from Bromite.
They are always enabled and cannot be disabled except at the source code level. I don't want to remove them, because for the majority of cases, it works fine and hardens security. But what I will do (especially since you guys are not the first to report DNS problems in Thorium), is put them behind a chrome://flags flag. Something like "Disable Thorium DNS Config". This way it can be disabled via GUI.

@gz83 @narinishi @adrianmmiller @eltociear I think this is related to the two DNS patches we use. One is from Ungoogled, the other is from Bromite.
They are always enabled and cannot be disabled except at the source code level. I don't want to remove them, because for the majority of cases, it works fine and hardens security. But what I will do (especially since you guys are not the first to report DNS problems in Thorium), is put them behind a chrome://flags flag. Something like "Disable Thorium DNS Config". This way it can be disabled via GUI.

Cant ask for a better response than that, cheers

@gz83 @narinishi @adrianmmiller @eltociear Here we go > https://github.com/Alex313031/thorium/commit/840ec41e774442da669a5a863b5a1bec31523951
Also, @narinishi I added this to the thorium-legacy repo as well, so it will be present in the next builds.


DownloadPackageApiV2.webtest is broken on NuGet.Jobs

This test is failing. The ExpectedResponseUrl should be updated to whatever https://dev.nugettest.org/api/v2/package/Newtonsoft.Json/6.0.1-beta1 redirects to now.

@xavierdecoster , I see that this is your code. Can you explain the purpose of this test?


Add a type-guard overload of Array.every

This adds a type-guard aware overload of Array.every.
Fixes #14963.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: dmitri-gb sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

I've wanted this feature too, but it was pointed out in another issue that Array.prototype.every differs from Array.prototype.filter in that it does not return a new array, and other references mutate the underlying array in an incompatible way.
However, TS has other unsound narrowings, so maybe this should be a pragmatic exception?  I'm not sure.  Seems easy enough to wrap if you need it in a more sound way.

Overall LGTM. Can you fix the merge conflict so we can run some user tests?

@typescript-bot user test this
@typescript-bot run tests

@typescript-bot run dt

@typescript-bot rwc test this

@typescript-bot test this

User tests are fine -- we caught vs code during a build break, but otherwise there's nothing there.


Attributes of grozi.csv

Hey Anton,
What's gtbboxid in grozi.csv? How is it obtained and significant?

Hi, as far as I remember it is just a unique ID of a bounding box.

In that case, shouldn't it be continuous?

It originiates from the annotation process, and over time it became less continuous because of cleaning and filtering


qustion: How to ping host target from Zephyr?

Hi, I want ping a target device from Zephyr, and check if the target alive. But the API net_icmpv4_send_echo_request is declared in subsys/net/ip, not exported for us. Is there has a easy way to do that?

My personal idea is based on PR #10665 and PR #10660 .
static bool network_check(int arg)
{
	char cmd[64] = "net ping " CONFIG_NET_CONFIG_PEER_IPV4_ADDR;

	return !shell_execute_cmd(NULL, cmd);
}

This maybe not the best way, but it just works.
If we can export some ICMP APIs to public, or implement SOCK_RAW?

Is there need to do anything here? I thought after IRC conversation that there is some progress when working with shell_execute_cmd().


Add support for system CA certificate store on Windows

This will load CA certificates from the Windows system store, if required.
The feature is activated by specifying "wincert" in options.tls.cacert. Quite happy to take a more cross-platform approach here. Please advise.

@michael-grunder  could this be merged to master?

Closing in favor of #889


@craigspaeth => Team Channel QA

Info:
Closes https://github.com/artsy/force/issues/205
channel_nav is now a component (shared by team channel landing page + team channel article page).
Screenshots:

Nice refactor! I have some minor comments, but nothing blocking. Feel free to merge when you're ready. 👍

Thanks for the comments! I addressed the points you mentioned. Mergin'


Revert "[REVIEW] Add Async Support to XCTest"

Reverts apple/swift-corelibs-xctest#326
This breaks swift-corelibs-foundation on Windows; given that the retrench is so close, I think that reverting it temporarily is safer unless someone can immediately spot what is going on.
CI logs: https://dev.azure.com/compnerd/3133d6ab-80a8-4996-ac4f-03df25cd3224/_apis/build/builds/54170/logs/148

@swift-ci please test

CC: @briancroom - sadly this is somewhat urgent in light of the rebranch

@compnerd I'll go ahead and land this revert. I think we may need assistance debugging the Windows linker failure — it definitely looks related to this change, but it's not clear to me at first glance what may be going wrong.
(Side note: can you clarify what you mean by "in light of the rebranch"?)

@stmontgomery - the swift repositories are meant to update the LLVM base (rebranching) for updates, and it would be nice to keep everything green across that large change.
I should be able to help with the debugging, I suspect that there is a silly bug in the compiler (with the async function pointer being referenced incorrectly across module boundaries).


Wrong and Misleading Notifications Icons Docs

Those docs: https://github.com/arnesson/cordova-plugin-firebase/blob/master/docs/NOTIFICATIONS.md
seem to be totally wrong.
Both style.xml and colors.xml are not used at all.
The style.xml is not copied at all to the cordova android platform folder after build, and same for the icons under /res/native/android/res/..
The colors.xml which appears in the cordova android platform folder comes from /plugins/cordova-plugin-firebase/src/android/colors.xml and the /res/native/android/res/values/colors.xml is again unused.
Any thoughts on this?
Using cordova-plugin-firebase version 0.1.25.
android 7.1.4.
cordova 8.1.2.

I concur.  I imagine the documentation matched the code at some point but no longer works...

I updated the documentation after experimenting with my plugin.  @guylando let me know if the new documentation makes sense or not.  I can definitely make changes if something is confusing.


error QS7106: Unable to serialize large circuit with QDK v0.15, used to work with v0.7

Describe the bug
The dotnet build command fails with error EXEC : error QS7106: Unable to serialize the built compilation. when compiling the largest circuit from https://github.com/microsoft/grover-blocks ("LowMC L5"). This used to work under version 0.7 of the QDK, but fails under version 0.15.2101125897.
To Reproduce
Steps to reproduce the behavior:

Fetch the example source from https://github.com/fvirdia/QS7106-Unable-to-serialize
Try building the source under /qlowmc/ (it may require > 16GB of RAM and ~16 hours, this was also the case under v0.7 of the QDK)
See error output by the compiler

Expected behavior
The circuit should build and run. A related circuit (essentially a scaled down version) successfully compiles and runs.
System information

OS: Debian 10

CPU: two Intel Xeon E5-2690 v4 CPUs (2.60GHz, each with 14 physical cores), 256GB RAM

.NET Core Version: 3.1

QDK version 0.15.2102128318

@cesarzc Of the top of your head, do you have any idea what might be causing it and/or how to best gather additional data/confirm?

@bettinaheim of the top of my head I don't know what might be causing this. I'll start with your suggestion of checking whether the pre-bond serialization still works with this source code.
Thank you @fvirdia for providing the source code to reproduce this issue!

No problem! As a heads up if you are trying this out, I found that 8GB of RAM were not enough to compile (the kernel would eventually kill the compiler). On the server it run fine, I'm not sure how much memory was strictly necessary.


Add JSON report

This pull request adds the JSON report feature.
#45

Summary by CodeRabbit

New Features

Added the ability to generate reports in JSON format.
Introduced a new JSON report generation capability in the analysis command.
Implemented a function to clean and standardize data by removing NaN values.

Enhancements

Expanded the configuration options to include JSON report settings.

Documentation

Updated struct definitions to support JSON formatted reports.

@Halleck45 Fixed

Thanks for your contribution! I've been a bit busy lately. I'll test it out as soon as I can and get back to you.
Thanks again for your help!

@julian776 I've added commits to your PR https://github.com/julian776/ast-metrics/pull/1
Can you (if you're agree) accept these modifications, then rebase your Pull Request ?

Thanks for your efforts @julian776, that's a great first contribution  :clap:  !


fix: Add economy EnvoyStorageMode

Apparently there is also an "economy" storage mode on some Envoys/Ensemble systems.
https://github.com/home-assistant/core/issues/103209

In our initial testing we got savings-mode as a mode but economy is showing up on systems with 7.3.517 firmware when Enlighten shows savings mode, not sure why the discrepancy...

It appears that savings-mode and economy both map to the "Savings" profile in the Enlighten app, but when changing the profile from Enlighten it is setting the value to economy

Updated.. new enum value EnvoyStorageMode.LEGACY_SAVINGS (savings-mode) and EnvoyStorageMode.SAVINGS changed to economy
If the Envoy returns a storage mode matching EnvoyStorageMode.LEGACY_SAVINGS (savings-mode) we automatically convert it to EnvoyStorageMode.SAVINGS (economy)


Request blocked

Mixed Content: The page at 'https://mysite.com/websocket' was loaded over HTTPS, but attempted to connect to the insecure WebSocket endpoint 'ws://mysite.com:8080/'. This request has been blocked; this endpoint must be available over WSS.
In my live server (Ubundu)

Off course this is mixed content you are running the socket server on an insecure connection. You need wss://mysite.com:8080. With the appropriate certificates attached to it.


Lazy example broken

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?
I ran cargo r in examples/lazy.
I then get:
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', C:\<path>\iced\lazy\src\lazy.rs:221:22

I tried the current main @ 7ceb8b8d406cb72ea0e0c341ad755d69165d95f9 ,
as well as using iced 0.6 + iced_lazy 0.3 in a standalone project (copy pasting the lazy example).
What is the expected behavior?
The example to run.
Version
master
Operative System
Windows
Do you have any log output?
No response

One simple solution is to use  iced 0.5 + iced_lazy 0.2

I did a git bisect and found it the issue was introduced in this commit: https://github.com/iced-rs/iced/commit/f1ada7a803998ac3fb2c1bedc6d6650264f3e603
The panic happens in the Rc::get_mut(...).unwrap() part of fn overlay(...).
@tarkah maybe you know what's going on in this part?

@torsteingrindvik I took a quick glance a couple weeks ago and I wasn't able to please the borrow checker. ouroboros makes debugging errors rather difficult.

Issue is we hold an RC in two places when trying to RC::get_mut which panics.
I took a look a couple weeks back but couldn't get the compiler happy. I tried dropping the RC before the overlay is built and then manually implementing Drop for the overlay to reclone the RC, but ouroboros doesn't like manual Drop implementations.
I'll try to take another look tomorrow.

Sounds like the same thing I ran into with Modal: https://github.com/iced-rs/iced_aw/issues/73


Fix incorrect api usage, not Time rather TIME

http://www.nodegit.org/api/revwalk/#SORT

I decided to force the merge despite the AppVeyor build failure since it's a change to an example and not any code that is part of the build.
Thanks for the pull request, @cdosborn!


Update instance type/size in endpoint model_config example

Update instance type/size in endpoint model_config example.
See: https://huggingface.co/docs/inference-endpoints/en/pricing

Maybe I should also update the instances in https://github.com/huggingface/lighteval/blob/main/examples/model_configs/test.yaml?

Hm that file is an accidental upload on my side haha, you can even remove it imo


Fails to map snapshot on Ampere Altra with 64k page size

When building Flutter on my Ampere Altra running NixOS and using a 64k page size, I get this scary abort:
../../runtime/bin/snapshot_utils.cc: 154: error: Failed to memory map snapshot: /nix/store/gjvyzckkyq2s57ajcnxy3682bj06r9gx-flutter-3.22.0-unwrapped/bin/cache/flutter_tools.snapshot

===== CRASH =====
si_signo=Segmentation fault(11), si_code=SEGV_MAPERR(1), si_addr=0x10
Aborting reentrant request for stack trace.
/nix/store/yja2i691gbdxl6v1hd7ccnym3zqgl11i-stdenv-linux/setup: line 1582:   282 Aborted                 (core dumped) $out/bin/flutter config --android-studio-dir $HOME

When I build Flutter applications, I get no errors and no messages to the terminal at all. This seems like Dart isn't set up to handle a 64k page size which my system uses for better compiling performance. Meanwhile, Flutter and Dart compile easily on my MBP running NixOS which is using a 16k page size. In this case, I was building/using Flutter 3.22.0 but it seems every version of Flutter in Nixpkgs has this problem. It's best we get this issue fixed as more hardware which utilizes ARM may be running large page sizes (beyond 4k or even 16k).
To reproduce, use CONFIG_ARM64_64K_PAGES in the kernel config and try building Flutter or try building any Flutter applications.

//cc @rmacnak-google

This was fixed in https://dart-review.googlesource.com/c/sdk/+/368525.
Please try Flutter 3.24 or later.


Enable session sharing in sentinelhub tasks via execute arguments

This is a suggestion on how we can do session sharing in eo-learn.
My first idea was to add it as an init parameter, but it is easier to add it as an execution argument. It should be no different were it an __init__ parameter, so I am willing to adjust that.
I checked that it works with multiprocessing:
thread = SessionSharingThread(session)
thread.start()

try:
    with ProcessPoolExecutor(max_workers=3) as executor:
        futures = [executor.submit(task.execute, None, bbox, time_interval, collect_shared_session) for _ in range(3)]

finally:
    thread.join()

And it also works with Ray:
@ray.remote
class RaySessionActor:
    def __init__(self, session):
        self.session = session

    def get_valid_session(self):
        token = self.session.token
        return SentinelHubSession.from_token(token)

@ray.remote
def remote_execution(actor):
    task.execute(None, bbox, time_interval, lambda: ray.get(actor.get_valid_session.remote()))

actor = RaySessionActor.remote(session)

futures = [remote_function.remote(actor) for _ in range(3)]
ray.get(futures)

Codecov Report

Merging #411 (3871425) into develop (4186a2a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #411   +/-   ##
========================================
  Coverage    83.05%   83.05%
========================================
  Files           56       56
  Lines         5282     5283    +1
========================================
+ Hits          4387     4388    +1
  Misses         895      895

Impacted Files
Coverage Δ

io/eolearn/io/sentinelhub_process.py
86.81% <100.00%> (+0.04%)
: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 4186a2a...3871425. Read the comment docs.

Another question - would it perhaps make sense to put the implementation of RaySessionActor also into eo-learn?
The best place for it would actually be eolearn.io.extra.ray, which means we would have to add a Ray extension to eo-learn-io package. However, since this actor class really has only a few lines of code I'm not sure if it justifies creating a package extension. :thinking:

Another question - would it perhaps make sense to put the implementation of RaySessionActor also into eo-learn?
The best place for it would actually be eolearn.io.extra.ray, which means we would have to add a Ray extension to eo-learn-io package. However, since this actor class really has only a few lines of code I'm not sure if it justifies creating a package extension. thinking

I would wait until we are approached about it. It described in sh-py and easy to do. The thread solution is more complex so it makes sense to provide it for users.


Handle overloading better in the VSCode extension

At the moment, overloading looks like this when you highlight a function:

And like this when you're typing a function:

I would prefer it if it didn't show every overload on highlight and provided options when typing. For an example, look at how the TypeScript extension handles it:

Should be supported.


Update build to support macOS arm64/x86_64

For issue #80. This is my first attempt at using cibuildwheel or github actions so forgive me if it has issues.
The main change is using os [macos-13, macos-14] to build the x86_64 and arm64 wheels on their respective systems.
Note that I had to remove cp37 for it to build and test successfully but there is probably a way to fix that.

Updated to fix failing tests due to https://github.com/actions/setup-python/issues/696

It's working as expected.
Thank you for fixing the problem.

"Co-authored-by" was removed.


Slow database size for organizations

Measuring organization database size now involves running cartodb.CDB_UserDataSize. This is slow for big organizations. Normal user usage doesn't need measuring organization size, only some operations of organization owner are affected.
cc @Kartones @javisantana

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


dag_drawer should check the existence of filename extension

Information

Qiskit Terra version: 0.10.0.dev0+831d942
Python version: 3.7
Operating system: Mac

What is the current behavior?
If a filename without extension is passed to the function dag_drawer, this line reports two errors:
nxpd.pydot.InvocationException: Program terminated with status: 1. stderr follows: Format: "XXXXX" not recognized. Use one of: ......
During handling of the above exception, another exception occurred:
qiskit.visualization.exceptions.VisualizationError: 'dag_drawer requires GraphViz installed in the system. Check https://www.graphviz.org/download/ for details on how to install GraphViz in your system.'

This is confusing because the second error thrown by Qiskit is not the cause of the problem.
Steps to reproduce the problem
Try dag_drawer(dag, filename='abc')
What is the expected behavior?
Make the error catching better.
Suggested solutions
We could either catch this error by reading and filtering the error message, or we could check the existence of the filename's extension, and provide a default one.

Hi @we-taper , thanks for reporting, and I agree the conflicting error messages are confusing. It looks like the error catching around https://github.com/Qiskit/qiskit-terra/blob/d090eca91dc1afdb68f563885c4ccf13b31de20e/qiskit/visualization/dag_visualization.py#L91 could be refined to either inspect the raised nxpd.pydot.InvocationException to try to determine the cause of the error, or at least include the generated stderr in the VisualizationError that's raised by qiskit.

Hi @kdk, I see you started working on this problem a couple of days ago... I was wondering if you're still on it or if I could try my hand at fixing this bug as well :)

I can have a look!

Newcomer here, recently started learning QisKit. Would not mind taking a crack at this.

go for it @Rajan117! I've assigned to you, let us know if you have any questions

Thanks.
I am using this code to reproduce the error.
`from qiskit import QuantumRegister, ClassicalRegister, QuantumCircuit
from qiskit.dagcircuit import DAGCircuit
from qiskit.converters import circuit_to_dag
from qiskit.visualization import dag_drawer
q = QuantumRegister(3, 'q')
c = ClassicalRegister(3, 'c')
circ = QuantumCircuit(q, c)
circ.h(q[0])
circ.cx(q[0], q[1])
circ.measure(q[0], c[0])
circ.rz(0.5, q[1]).c_if(c, 2)
dag = circuit_to_dag(circ)
dag_drawer(dag, filename='abc')`
But I instead get this error.

I assume I have done something wrong in setting up my test environment but unsure how, followed the official QisKit YouTube guide.

I will be honest, I have had a lot of issues in setting up test environments to attempt fixing issues. Is there an easy way to do it?

I will be honest, I have had a lot of issues in setting up test environments to attempt fixing issues. Is there an easy way to do it?

It's best to follow the steps outlined here for the best way to set up your environment (including pip install -e .)

Thanks! Will give this another go over the weekend then :)

unassigning due to lack of activity.

from qiskit import QuantumRegister, ClassicalRegister, QuantumCircuit
from qiskit.dagcircuit import DAGCircuit
from qiskit.converters import circuit_to_dag
from qiskit.visualization import dag_drawer

q = QuantumRegister(3, 'q')
c = ClassicalRegister(3, 'c')
circ = QuantumCircuit(q, c)
circ.h(q[0])
circ.cx(q[0], q[1])
circ.measure(q[0], c[0])
circ.rz(0.5, q[1]).c_if(c, 2)

dag = circuit_to_dag(circ)
dag_drawer(dag, filename='abc')

"dot" with args ['-Tabc', '/tmp/tmp9ngv_7rl'] returned code: 1

stdout, stderr:
 b''
b'Format: "abc" not recognized. Use one of: bmp canon cmap cmapx cmapx_np dot dot_json eps fig gd gd2 gif gv ico imap imap_np ismap jpe jpeg jpg json json0 mp pdf pic plain plain-ext png pov ps ps2 svg svgz tif tiff tk vdx vml vmlz vrml wbmp webp xdot xdot1.2 xdot1.4 xdot_json\n'

Traceback (most recent call last):
  File "/tmp/ipykernel_78/1404195018.py", line 15, in <module>
    dag_drawer(dag, filename='abc')
  File "/opt/conda/lib/python3.8/site-packages/qiskit/visualization/dag_visualization.py", line 170, in dag_drawer
    dot.write(filename, format=extension)
  File "/opt/conda/lib/python3.8/site-packages/pydot.py", line 1828, in write
    s = self.create(prog, format, encoding=encoding)
  File "/opt/conda/lib/python3.8/site-packages/pydot.py", line 1956, in create
    assert process.returncode == 0, (
AssertionError: "dot" with args ['-Tabc', '/tmp/tmp9ngv_7rl'] returned code: 1

Use %tb to get the full traceback.

I think the message is pretty clear now.

thanks for checking if this is still valid @iuliazidaru! However I think the error is still not very user friendly and it would be good if we could catch it better in the dag_drawer

@javabster Please assign the issue to me.


feat: .alva files

Rationale
This PR introduces a changed persistence system where everything regarding a Alva project is saved into a single, shareable .alva file. For the time being this file is a .yml file with the .alva file extension. The intended improvements for our users are

Files that can be shared easily
No need to do technical setup on a component library for trying out alva
Better support for first steps without a own component library

Done

[x] Graph: Decouple pattern-property from pattern
[x] Graph: Decouple element-property from element
[x] Graph: Decouple element from content
[x] Graph: Decouple pattern-property from pattern-property
[x] Restore component search
[x] Reintroduce component library analyzer as "component import"
[x] Fix selected element highlighting in preview
[x] Remove notion of multiple projects
[x] Decouple styleguide from analyzer
[x] Add toJSON and static from methods to all classes for serialization/deserialization
[x] Move all interaction with the file system to the main process
[x] Add full typings for messages between render process, main and preview
[x] Model slots in elements as distinct entity: ElementContent
[x] Model properties in elements as distinct entity: ElementProperty

Dropped

[ ] Remove unused model and store methods
[ ] Associate the .alva file extension with Alva

Notes

There is weak encapsulation between the Alva data structures and the preview renderer. Hardening this boundary would result in a sturdier system. The client-side preview API should be powered by a  data structure initializied via Project.from
Moving to a graph where all data structures are loosely coupled via uuids would make most changesets smaller by avoiding sprawling, tree-spanning changes. We might consider using a graph modelling library to offload the infrastructure work for this
element-property should be split into value-specific classes analogous to pattern-property
Consider adding a purge functionality that collects orphaned elements from the project registry
Consider splitting ElementLocationCommand into smaller abstractions
Consider abstracting away the *Command pattern inside the remaining ViewStore

Fixes #440
Fixes #19
Fixes #199

Currently it's not possible to duplicate elements via Cmd+D / Ctrl+D

Currently it's not possible to duplicate elements via Cmd+D / Ctrl+D

Fixed via https://github.com/meetalva/alva/pull/454/commits/1f9ce1e66e4f5f03adfdf26b0d05c49be0bf42e4


Fix: Single Hotel Details - see picture detail instead of gallery

Who asked for this change? I really don't like it.
First of all, if I have the thumbnails, I can find and enlarge the fotos that interests me. Probably I would want to see 2 or 3 out of 120 photos.
Second, it does not make sense to directly open a modal with images, then when I close it, I am at the grid. I would expect to end up where I was when I clicked the photo.

So after our discussion from this morning, what do you think we should be doing about this one?

🤷♂ One one replied to it yet. I would say two options

Leave it as is (my preferred way)
Open the modal (🙈) then when you close it, you are on hotel detail screen, and just delete gallery screen.

Okay, let's wait for a reply from someone.

After discussion with designer:

Well okay, let’s keep gallery view
So closing this issue.


Update README with contributions info and unit test setup

I added a "Contributions" section to the README stating that pull requests require unit tests and the requirements to set up the project locally to run and create rspec tests. I did this as I figured it would help others contribute knowing what is needed to get started. I had some trouble figuring it out as I haven't used rspec before. Hope this helps.

Updated the list to contain just the commands to run and now using bundle install rather than the gems that need to be installed.


The documentation does not mention that Stream Action tags can be rendered inside any HTML

The way Turbo is currently implemented, stream tags will be processed and executed when they are rendered anywhere in HTML.
Here is a minimal example with a static HTML page:

    <turbo-stream action="append" target="list">
      <template>

list element

      </template>
    </turbo-stream>

    This list has an element added via the stream action:

This will work anywhere where stream tags are rendered into the page DOM:

Anywhere inside the initial page HTML.
As part of the turbo frame response as long as it is rendered inside the frame tag. Since content from outside the frame is not rendered into the actual document, the tag needs to be inside the frame for this to work.
If any javascript at any point rendered a turbo stream element anywhere inside the document DOM.
If a stream action renders HTML content (like it does in the example above) and that content in turn also has a stream action tag inside it, the second action tag will also work.

Not all of those are smart ways to use it but I was surprised that the documentation makes no mention of the feature. At least I couldn't find any mention anywhere in the documentation or old release notes.
I am guessing that the reason is that this is a side-effect of how StreamElements are implemented: They are defined as customElements with the stream tag execution being triggered by the connectedCallback implementation of the StreamElement class.
Perhaps it's just me and most other people find this clear without being mentioned but I think it would still be worth adding a few sentences and a documentation and a test to cover it. This can be a very useful feature in some cases.
I'd be very happy to mention this in the documentation but wanted to first check if there is something I am missing?
Can someone confirm that this is an intentional Turbo feature that should be mentioned explicitly in the documentation? If not, can we make it an official feature. Considering how it's implemented, one would have to do add explicit hacks just to stop it from working.

Can someone confirm that this is an intentional Turbo feature that should be mentioned explicitly in the documentation?

I've personally relied on the behavior that you've described. For example, you can nest a <turbo-stream> element inside a <template> element, then append that content to the page after a <button> click (try it out on JSFiddle):

<button type="button" id="call-to-action">
  Click me to insert the template's contents after this button!

  <template>
    <turbo-stream action="after" target="call-to-action">
      <template>
You clicked the button!
</template>
    </turbo-stream>
  </template>
</button>

I've personally relied on the behavior that you've described. For example, you can nest a <turbo-stream> element inside a <template> element, then append that content to the page after a <button> click (try it out on JSFiddle):

Glad to hear that. :) I've used, in production, stream tags inside frame response (for extra side-effects) and inside the initial page load HTML (for interacting with some legacy JS).

Can someone confirm that this is an intentional Turbo feature that should be mentioned explicitly in the documentation? If not, can we make it an official feature.

I think outlining these possibilities at a high level in the Streams Handbook section and in greater detail in Streams Reference section would be a big improvement!

Can someone confirm that this is an intentional Turbo feature that should be mentioned explicitly in the documentation? If not, can we make it an official feature.

I think outlining these possibilities at a high level in the Streams Handbook section and in greater detail in Streams Reference section would be a big improvement!

Great, I'll prepare something soon. 👍

@seanpdoyle I've opened two PRs to address this.
One to extend the documentation: https://github.com/hotwired/turbo-site/pull/192
And one to add a test ensuring no regressions of this functionality: https://github.com/hotwired/turbo/pull/1263
Happy to iterate on both based on feedback. :)


How to select relation in Attention while Testing?

Hi, some thing about the attention confused me a lot.

the r is the query vector with relation r (the relation representation).
In train phase,  is it r is the target relation label?  if so,  when in test phase, which r should be chosen to calculate the attention weight for the instances in a bag?
Do I misunderstand something about the paper?
Thanks.

While testing, you need to calculate all P(r|s) and just use the query vector with relation r' when calculating P(r'|s).

@Mrlyk423
Thanks for your reply.  It is quite clear. That is to say, the formula (10)

It is just a stack of each relation's score
o_r = M_r s + d_r
which is calculated separately rather than an extra soft-max layer.
Is that right?

Yes

@Mrlyk423
Thanks, your reply helps me a lot in understanding the paper.
Best.

@Mrlyk423 but in test phase how to calculate alpha(instance attention), and how to choose the y_pred, max(p(r|s))?

@Mrlyk423  Thanks very much. By the way, do you have other versions of PCNN+ATT such as tensorflow or pytorch?   I try to reproduce PCNN+ATT with pytorch but got a quite worse result comparing with your C++ version.  But the pytorch version of PCNN+ONE got a similar result as yours and Zeng2015. This issue confused me for a long time, What may be the possible reason?
Thanks.

Is vector r in equation (8) obtained from Matrix M mentioned in equation (10)? Or there are two separate trainable parameters for r in eq (8) and M in eq (10)?

In my opinion, the vector r in equation (8) shares with Matrix M

@Mrlyk423 Thanks very much. By the way, do you have other versions of PCNN+ATT such as tensorflow or pytorch? I try to reproduce PCNN+ATT with pytorch but got a quite worse result comparing with your C++ version. But the pytorch version of PCNN+ONE got a similar result as yours and Zeng2015. This issue confused me for a long time, What may be the possible reason?
Thanks.

I am also facing same issue with pytorch implementation. F1 score is very low when applying attention over PCNN features. Have you been able to fix it? Have you released your pytorch implementation?

Yeah,  My implementation is in https://github.com/ShomyLiu/pytorch-relation-extraction
The dataset is used in this repo, which is slightly different from the newer version dataset in OpenNRE.

For relation extraction, we only focus on the top predict results. If you want to get the all predict results, just change 2000 to the number you need.

Is the Precision-Recall curve in the paper based on to 2000 predicted results?


returning domino to the same place in the hand if it cannot be placed…

… on the board

Current coverage is 100% (diff: 100%)

Merging #65 into master will not change coverage

@@           master   #65   diff @@
===================================
  Files           8     8
  Lines         300   300
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits          300   300
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update cf1e344...ffdb7dd


Numbers represented as strings

Using v0.9.9, I notice that numeric values in the BigQuery API are represented as strings in the generated classes.
For example, the API documentation for Tables shows time and number values as long or unsigned long:
  "numBytes": long,
  "numLongTermBytes": long,
  "numRows": unsigned long,
  "creationTime": long,
  "expirationTime": long,
  "lastModifiedTime": unsigned long,

However, in the generated Table class they are all documented as String. String values are also what I see returned at runtime:
NoMethodError: undefined method `/' for "1468434950787":String
    /Users/quartzmo/code/google/codez/gcloud-ruby/lib/gcloud/bigquery/view.rb:198:in `created_at'

This is hardly a blocking issue, as it is easy to convert strings to numeric values. However, it does make upgrading from 0.8.x a bit more error-prone. Also, I am just wondering about the reason for this behavior.

No good reason, just a bug/oversight. While the date/time strings are coerced, the numeric ones aren't.
At this point I can't change it without breaking compatibility, so it'll have to wait a little bit until there's a good opportunity to fix it (can do when I go to 1.0 though)

@sqrrrl That's what guessed, thanks for the quick confirmation! I'll convert these values in gcloud-ruby for the time being.

@sqrrrl Not sure what your timeline is for 1.0, but how about a 0.10.0 release that fixes the type coercions?

Possible to do it as part of .10.0. Technically since we're < 1.0 we can break things at any time. I already did that with 0.8 -> 0.9, though it bit apps that didn't have their dependencies locked down tight enough.

Will be in the 0.10 release.

Has this fixed on the 0.10 release? The object is returning a string and documentation still shows them as strings.

No, it's not. We're in the process of transferring ownership and there was a little mixup on 0.10. I had a milestone and branch with some other backward compatible changes like this, but it didn't get merged in before 0.10 was released.
@saicheems -- there's a bunch of stuff in the v0.10 that will need to be merged in at some point. I was waiting on finishing one other large change (breaking the services into separate gems) to avoid churning too much on potentially breaking changes, but it's not critical.
LMK how you want to proceed with those changes.

Yeah, I was also surprised the dependency on hurley was continued in the 0.10.0 release.
FWIW, we are fully supportive of breaking changes if these issues can be fixed.


[BUG] Simple Wikipedia Demo error - missing imports

Describe the bug
Missing imports in Simple Wikipedia Demo
errors:
NameError: name 'math' is not defined for numpy, math, copy
TypeError: Index.add_documents() got an unexpected keyword argument 'batch_size'
To Reproduce

Run Marqo
Run  python3 simple_wiki_demo.py

Expected behavior
Missing imports error
Desktop (please complete the following information):

OS: MacOS on M-series Macbook air (M2)
Marqo version: 0.0.8

Additional context
Will be submitting a PR shortly

Thanks @georgewritescode ! A PR would be great. This was my fault, I did not update the demo :(

@jn2clark , no problem 👍
PR created https://github.com/marqo-ai/marqo/pull/216

closing for now. fixed in https://github.com/marqo-ai/marqo/pull/216


Date warnings appear twice

With this fix, we basically block the display of duplicate notifications by using a hash which is generated by the notification message.
This fixes issue [#814].
How to test the feature:

Select Cambodia
Enable layer X
Select a date that is out of range for a layer Y
Enable layer Y along with X
Expect to see a single notification saying there is no data for that date

Screenshot/video of feature:
https://github.com/WFP-VAM/prism-app/assets/24255041/a9a924c0-1bd0-47d7-831a-4c94f25af8c9

@tboulis great find. Just a quick note, when possible, try to link your PRs to the opened issue through GitHub for easier tracking :)

@tboulis great find. Just a quick note, when possible, try to link your PRs to the opened issue through GitHub for easier tracking :)

Of course Eric, I've already done this both in the PR description and the commit message

Yep but it's actually a bit picky in terms of formatting and I had to link it from the development pannel on the right side of the page. No big deal, just a note. In the PR body for example, "This PR fixes #834" needs to be the very first line and should not have the brackets []


Setup nginx + php-fpm instead apache

Hi,
Can you add configuration for setup nginx + php-fpm instead apache and exclude apache, please.
Or how I can configure it for myself, and care about next updates?

Unfortunately, the default php-fpm clean install for ubuntu is broken: https://github.com/serghey-rodin/vesta/issues/1238
nginx + apache + fcgi is very similar anyway (as more flexible) as php-fpm is alternative fcgi.   I will see what I can do to implement php-fpm in this container.

nginx + php-fpm will have to be it's own branch or docker build after the fix to the vesta issue reference above.  Instead, I did the next best thing.  I added php-fpm configuration to this container: https://github.com/niiknow/vestacp/wiki/php-fpm-support
Even though Apache2 is installed, it's not using it; and instead, nginx is configured to go directly to php-fpm.  Follow the wiki page above to config/switch between different version of php-fpm.
I will consider this is complete and close this ticket 7 days from now.


Docker platform

m-ld for Docker is coming soon. Please +1 this issue to vote for it.
m-ld for Docker bundles the Java m-ld engine with a REST API, so that it can be used in microservice architectures. For example it can be used as a backup clone in scenarios that require a high degree of data safety.

Additional motivation: using a backend m-ld engine as an interim measure when developing the idiomatic API for new platforms. E.g. 👇

The core m-ld protocol is still in motion, so our idea here is to learn as much as we can with a simulated engine before diving into a full implementation. However, the simulation itself should be as useful as possible right from the start!
We can learn a lot with a local un-synced JSON-file backend, but we should move as fast as possible to something with some sync, to shake out how the API behaves with concurrent changes.
So one idea is to implement a back-end sync service, deployable locally with Docker and also on the cloud. This service would use m-ld-js internally and expose the clone JSON API via HTTP. Since this introduces the possibility of concurrent clients, we'd implement some naive optimistic locking just to ensure data integrity. So some transaction commits may incur a lock violation; which you wouldn't see if you used a local m-ld engine.
Originally posted by @gsvarovsky in https://github.com/m-ld/m-ld-spec/issues/97#issuecomment-1254895898

Having containerised the timeld time-tracking application for Docker, the ground is now prepared for generalising this to m-ld itself.


recommendation in the Note is not clear

I'd like to make sure if "We recommend a size of /24 with 256 addresses." in the Note is for VNet. I assume so since "We recommend /25, which has 128 addresses and can handle a maximum-sized ASE." for a subnet is in 8.c.

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

ID: 7e31ca36-9aa0-f429-f2fd-65ba2758c359
Version Independent ID: e201f645-18c4-f420-167c-6431d114ef5b
Content: Create an External Azure App Service environment
Content Source: articles/app-service/environment/create-external-ase.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@kazshi, Yes, its the size of the VNet. Please do let us know if this helps or you have any further questions or clarifications. Thank you!

@AjayKumar-MSFT Thanks for your clarification.


Cannot run Arch with or without WSL1/2

running the installation with WSL2, gives
Installing...
ERROR:Installation Failed!
HRESULT:0x80070002
Press any key to continue..
running installation with WSL1 runs fine!
not untill i run it...
----- ERROR -----
This distribution only works with WSL2.
This is running in WSL1 mode now.
Please use WSL2 mode.
i have tried upgrading it...
The system cannot find the file specified.
I

Nevermind!
turns out the fix for this is updating windows, and the wsl2 kernel.
i hope maybe just maybe this will help someone


Create function to get the time statistics and the return code from the optimization solver

Hello Alex!
I needed to get access to the return code and the time statistics from IPOPT and currently they are only printed. In this commit, I created functions to access these variables. If you found useful, please merge into master.
Best,
Vivian.

Cool, thanks!


CurrentStateMonitor does not correctly report attached collision objects, but PlanningSceneMonitor does.

Description
We had an issue with MoveIt Servo that was fixed in https://github.com/ros-planning/moveit2/pull/2747, where collision checking with attached objects was failing since the presence of the object was not correct.
Simply switching the line:
robot_state_ = planning_scene_monitor_->getStateMonitor()->getCurrentState();

to
robot_state_ = planning_scene_monitor_->getPlanningScene()->getCurrentState();

fixes the issue, so I wanted to file a bug tracking this mismatch.

I think #2747 closes this.

Is this actually a bug? My understanding here is that:

CurrentStateMonitor just monitors the robot itself - the API doc is "Monitors the joint_states topic and tf to maintain the current state of the robot" - neither of those topics would provide information about attached objects.
PlanningScene is then the full planning scene, with attached objects and octomap, etc.

Perhaps we just need to update the API docs to be more clear about the differences between these two classes?

Good point. I agree that should be sufficient.

Probably also a good idea to do a quick search of the codebase and make sure there aren't other cases where the wrong function has been used...


Add findDocumentSymbols

A question was asked in gitter if VS Code has an equivalent of WebStorm's "File Structure Popup" in HTML. The closest that vscode has is "Show document symbols", but the html language service does not provide that.
To attempt parity with webstorm in this area, I have implemented findDocumentSymbols. I have augmented the parser to track tag attributes and their values (I could see nowhere before that attributes were being used) and combine the tag name with id and class attributes to build a symbol name.
For example,

 will have the symbol name div#id.my.classes. For child elements, I set the containerName to be the symbol name of their parent tag. So in

, the span element would have the symbol name span and the container name div#container1
I set the SymbolKind to field just to have something as I am unsure what the convention is for non-code related symbols.
For this to appear in vscode, it would necessitate a change there as well. I have a pull request ready to submit if this is accepted.

Awesome, look good @hoovercj !

Changing attributeNames to attributes is strictly speaking an API breakage. I added attributeNames back but only in the implementation as to stay compatible.
I removed attributeNames from the d.ts given that I just went 2.0 a few days ago and I don't anticipate that many users.
@hoovercj Actually all that's missing is a full test. I'll add these.

@aeschli what tests are missing? I tried to add tests to avoid putting extra effort on the team to accept the PR, so I'd like to know what I missed for future reference.

Np, see my commit above.
I'll adopt the change to VSCode so that it gets in today.

@aeschli how is the test you added different than the one I added in src/test/documentSymbols.test.ts? Yours are organized slightly differently than mine, but they essentially cover the same thing:
    test('Simple', () => {
        testSymbolsFor('

', [<SymbolInformation>{ containerName: '', name: 'div', kind: <SymbolKind>SymbolKind.Field, location: Location.create(TEST_URI, Range.create(0, 0, 0, 11)) }]);
        testSymbolsFor('
<input checked id="test" class="checkbox">
', [{ containerName: '', name: 'div', kind: <SymbolKind>SymbolKind.Field, location: Location.create(TEST_URI, Range.create(0, 0, 0, 53)) },
            { containerName: 'div', name: 'input#test.checkbox', kind: <SymbolKind>SymbolKind.Field, location: Location.create(TEST_URI, Range.create(0, 5, 0, 47)) }]);
    });

Given that you've added src/test/documentSymbols.test.ts, though, I recommend removing my src/test/symbols.test.ts

@hoovercj Sorry, I completely didn't see these tests. I must have been sleeping. Sorry.

:-) Glad we both had similar ideas. Thanks for accepting the pull request, cheers!

I merged the tests


topk panics when param < 1

		{
			name: "bbb",
			load: `load 30s
http_requests_total{pod="nginx-1", series="1"} 1+1.1x40
http_requests_total{pod="nginx-2", series="2"} 2+2.3x50
http_requests_total{pod="nginx-3", series="3"} 6+0.8x60
http_requests_total{pod="nginx-4", series="3"} 5+2.4x50
http_requests_total{pod="nginx-5", series="1"} 8.4+2.3x50
http_requests_total{pod="nginx-6", series="2"} 2.3+2.3x50
`,
			query: `topk without () (0.5, vector(time()))`,
		}

Using float value for the k param is actually valid in the old Prometheus engine. Same query won't hit panic in the old engine but panics in the new engine.
Old engine: https://prometheus.demo.do.prometheus.io/graph?g0.expr=topk without () (0.5%2C vector(time()))&g0.tab=1&g0.stacked=0&g0.range_input=1h
=== RUN   TestInstantQuery/disableOptimizers=true/bbb/disableFallback=false
panic: runtime error: index out of range [0] with length 0

goroutine 108 [running]:
github.com/thanos-community/promql-engine/execution/aggregate.(*kAggregate).aggregate(0x14000546a90, 0xc350, 0x14000077de8, 0x0, {0x1400032ce30, 0x1, 0x1?}, {0x140007d00e0, 0x1, 0x0?})
	/Users/benye/hub/gowork/promql-engine/execution/aggregate/khashaggregate.go:167 +0x414
github.com/thanos-community/promql-engine/execution/aggregate.(*kAggregate).Next(0x14000546a90, {0x1012b3ec8, 0x14000115f80})
	/Users/benye/hub/gowork/promql-engine/execution/aggregate/khashaggregate.go:114 +0x594
github.com/thanos-community/promql-engine/execution/exchange.(*concurrencyOperator).pull(0x14000801a10, {0x1012b3ec8, 0x14000115f80})
	/Users/benye/hub/gowork/promql-engine/execution/exchange/concurrent.go:80 +0xc0
created by github.com/thanos-community/promql-engine/execution/exchange.(*concurrencyOperator).Next.func1
	/Users/benye/hub/gowork/promql-engine/execution/exchange/concurrent.go:56 +0xa0

A very similar issue related to this param https://github.com/prometheus/prometheus/blob/main/promql/engine.go#L2334.
The old engine will check the whether the param is within the int64 range.
bottomk by (series, pod) ((time() ^ time()), {series="2"} offset -1m52s)
    compatibility_test.go:109:
        	Error Trace:	/Users/benye/hub/gowork/promqlsmith/example/thanos-engine/compatibility_test.go:109
        	Error:      	Not equal:
        	            	expected: *errors.errorString(&errors.errorString{s:"Scalar value +Inf overflows int64"})
        	            	actual  : <nil>(<nil>)
        	Test:       	TestPromQLCompatibility
--- FAIL: TestPromQLCompatibility (5.12s)

We don't have that check in the new engine so error mismatched.

Prometheus truncates to int64 and errors if its not possible: https://github.com/prometheus/prometheus/blob/44fcf876caad9a5d28b92a728d2c98c34015d377/promql/engine.go#L2235


Taking to Survey causing crash (2.8.1)

I'm setting up Mixpanel as a dependancy of ARAnalytics using Cocoapods (0.37.2), is it something on the radar?
Other specs:
Xcode 6.3.1
Simulator iPhone 6 iOS 8.3 (12F69)
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Could not find a storyboard named 'MPSurvey' in bundle NSBundle </Users/james/Library/Developer/CoreSimulator/Devices/BA4646B9-6CE0-4BB0-8474-847311AE12B3/data/Containers/Bundle/Application/086F3357-147A-409F-905F-399EEAFF51DE/Today.app> (loaded)'
*** First throw call stack:
(
	0   CoreFoundation                      0x0000000105057c65 __exceptionPreprocess + 165
	1   libobjc.A.dylib                     0x00000001041f9bb7 objc_exception_throw + 45
	2   UIKit                               0x00000001060866b8 -[UIStoryboard name] + 0
	3   Mixpanel                            0x00000001037603f1 -[Mixpanel presentSurveyWithRootViewController:] + 161
	4   Mixpanel                            0x0000000103760dec __43-[Mixpanel showSurveyWithObject:withAlert:]_block_invoke915 + 172
	5   UIKit                               0x0000000105db49e1 -[UIAlertController _fireOffActionOnTargetIfValidForAction:] + 55
	6   UIKit                               0x0000000105db4f23 __85-[UIAlertController _dismissAnimated:triggeringAction:triggeredByPopoverDimmingView:]_block_invoke + 30
	7   UIKit                               0x0000000105c588f9 -[UIPresentationController transitionDidFinish:] + 1118
	8   UIKit                               0x0000000105c5ab7c __56-[UIPresentationController runTransitionForCurrentState]_block_invoke_2 + 133
	9   UIKit                               0x00000001062997c0 -[_UIViewControllerTransitionContext completeTransition:] + 110
	10  UIKit                               0x0000000105bc51d3 -[UIViewAnimationBlockDelegate _didEndBlockAnimation:finished:context:] + 326
	11  UIKit                               0x0000000105bac136 -[UIViewAnimationState sendDelegateAnimationDidStop:finished:] + 209
	12  UIKit                               0x0000000105bac46c -[UIViewAnimationState animationDidStop:finished:] + 76
	13  QuartzCore                          0x00000001059ca882 _ZN2CA5Layer23run_animation_callbacksEPv + 308
	14  libdispatch.dylib                   0x00000001086f3614 _dispatch_client_callout + 8
	15  libdispatch.dylib                   0x00000001086dba1c _dispatch_main_queue_callback_4CF + 1664
	16  CoreFoundation                      0x0000000104fbf1f9 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 9
	17  CoreFoundation                      0x0000000104f80dcb __CFRunLoopRun + 2043
	18  CoreFoundation                      0x0000000104f80366 CFRunLoopRunSpecific + 470
	19  GraphicsServices                    0x0000000108bf4a3e GSEventRunModal + 161
	20  UIKit                               0x0000000105b4c900 UIApplicationMain + 1282
	21  Today                               0x00000001031c6307 main + 135
	22  libdyld.dylib                       0x0000000108728145 start + 1
	23  ???                                 0x0000000000000001 0x0 + 1
)
libc++abi.dylib: terminating with uncaught exception of type NSException

hey @jamztang - i believe this is due to our library not playing particularly nice when integrated as a sub dependency. the SDK assumes the Storyboard to be in the main app bundle (which here it is not) - you can copy the resource over manually or change a line of code to point to the correct location. see #207 and PRQ #250 for the relevant code changes - i personally think copying the resource over manually is the best route here.

@jamztang It looks like the podspec should properly copy over the storyboard files. Are you still experiencing this issue? Can you provide a Podfile that reliably reproduces the issue you're having?

I didn't manage to try on a fresh project yet but this is what I'm specifying in my Swift application
use_frameworks!
inhibit_all_warnings!

target "Today" do
  pod "ARAnalytics", '~> 3.6.1', :subspecs => ["Mixpanel", "Countly", "ParseAnalytics"]
  pod 'JazzHands', '~> 0.2.1'
  pod "Spring"
end

This is Podfile.lock
PODS:
  - ARAnalytics/CoreIOS (3.6.2)
  - ARAnalytics/Countly (3.6.2):
    - ARAnalytics/CoreIOS
    - Countly
  - ARAnalytics/Mixpanel (3.6.2):
    - ARAnalytics/CoreIOS
    - Mixpanel
  - ARAnalytics/ParseAnalytics (3.6.2):
    - ARAnalytics/CoreIOS
    - Parse
  - Bolts/Tasks (1.2.0)
  - Countly (3.0.0)
  - JazzHands (0.2.1)
  - Mixpanel (2.8.1):
    - Mixpanel/Mixpanel (= 2.8.1)
  - Mixpanel/Mixpanel (2.8.1):
    - Mixpanel/MPCategoryHelpers
  - Mixpanel/MPCategoryHelpers (2.8.1)
  - Parse (1.7.5):
    - Bolts/Tasks (>= 1.2.0)
  - Spring (1.0.3)

DEPENDENCIES:
  - ARAnalytics/Countly (~> 3.6.1)
  - ARAnalytics/Mixpanel (~> 3.6.1)
  - ARAnalytics/ParseAnalytics (~> 3.6.1)
  - JazzHands (~> 0.2.1)
  - Spring

SPEC CHECKSUMS:
  ARAnalytics: 697797d915d0afadf27f79563b262557cbe3d378
  Bolts: d176cb1e0012175589e389a9db49b85e27787576
  Countly: 4212967fc5739ffccfbff3b0fe863be45c984879
  JazzHands: d2343c3f2f9d70598d0cad8f521ce3893842a44a
  Mixpanel: 93c4825025e6380ced273ed7178496282e385077
  Parse: 7c0f115eff3e0cea3fac08d50bda47154edb8832
  Spring: 3f02b5dc3308572ef177b3cad0fa7bbc4b0706f7

COCOAPODS: 0.37.2

This will be resolved in v2.8.2

This should be resolved now if you want to do a pod install.


docs(readme): add punchout and schemas to 'in the wild'

Added 2 projects to "In the Wild" section:

punchout: takes the suck out of logging time on JIRA
schemas: lets you inspect postgres schemas in the terminal

Love these. Also punchout is such a great name. Thanks for the PR!


Refactor for shims 320+

This contributes #4447
Signed-off-by: Chong Gao<EMAIL_ADDRESS>

This fixes the comments from @tgravescs in #4447
Add a new Spark320PlusShims; then Spark322PlusShims and Spark320until322Shims extends the Spark320PlusShims

build

LGTM


.?queueclear and 2nd song adding non response

Bugreport:
Bug reported by Dadi
What is the title of your bugreport?: .?queueclear and 2nd song adding non response
Is it a bugreport for the website or for the bot?: bot
How can you reproduce the bug? (Explain in steps): STEP 1: Type in ?play songname
Then add another song --> you don't get the response of the bot that you have added another song (http://prntscr.com/ky93hz)
Which result would normally have to be?: STEP 2: Type in ?queue (after you have done the first and 2nd step) --> works
Then try to use ?queueclear --> No response and ?queue doesn't work too (http://prntscr.com/ky94kb)
You can only stop the bot
(could not get the whole message in one question)
What happens if you do the action now?: look first and second step
ReportID: 46

➤ LenoxBot commented:
Approve:
Could be reproduced
Nickkel#6515

➤ LenoxBot commented:
Approve:
could be reprocued
Monkeyyy11#0001

➤ LenoxBot commented:
Approve:
reproducible
Fabiooo#3984


missing async caused confusion

When copy & pasting the example code it never worked the first time. Now it does.

Good catch! Thank you 🙏


ERROR: ParseError at .../bbb-yocto-gatesgarth/meta-bbb/images/console-image.bb:4: Could not include required file images/basic-dev-image.bb

Here is another blunder I ran into while fixing a yocto-setup.sh script:
https://github.com/ZoranStojsavljevic/bbb-yocto/blob/master/yocto-setup.sh
While executing $  . ./yocto-setup.sh dunfell, everything ran smoothly.
I did the same routine with the $  . ./yocto-setup.sh gatesgarth, and approximately after:
Parsing recipes: 9% |##########
The following message pops up!
Loading cache: 100% |                                                                                | ETA:  --:--:--
Loaded 0 entries from dependency cache.
ERROR: ParseError at /home/vuser/projects_yocto/bbb-yocto-gatesgarth/meta-bbb/images/console-image.bb:4: Could not include required file images/basic-dev-image.bb
Summary: There was 1 WARNING message shown.
Summary: There was 1 ERROR message shown, returning a non-zero exit code.
If you peek into the following github repos:
https://github.com/jumpnow/meta-bbb
https://github.com/jumpnow/meta-jumpnow
You'll see that in later  images/basic-dev-image.bb does exist, as well as in the dunfell branch:
https://github.com/jumpnow/meta-jumpnow/blob/zeus/images/basic-dev-image.bb
As well as in gatesgarth branch:
https://github.com/jumpnow/meta-jumpnow/blob/gatesgarth/images/basic-dev-image.bb
What I see upon the logic, something changes in poky/ setup, while transitioning from dunfell to gatesgarth.
I would appreciate any ideas or hints... You can all try it yourselves, and see the same!
Thank you,
Zoran

The meta-jumpnow layer is not included. Verified in bblayers.conf. : https://github.com/ZoranStojsavljevic/bbb-yocto/blob/a6e8e8ce491705816d02df58fa0ef9378b18ef83/yocto-setup.sh#L114 where the gatesgarth check is accidentally missed.
The issue is solved.


GKE Documentation incompatible with automatic sidecar injection

https://istio.io/docs/setup/kubernetes/quick-start-gke-dm.html
Has no mention of whitelisting for Kubernetes 1.9.2 which I'm not sure how to do.  The docs as is crater badly when bookinfo is deployed with the sidecar.

a pr (https://github.com/istio/istio/pull/2997) is waiting to make it work, one of the issue is it's still using istio 0.3 yet fetches files from master on github

@ldemailly sure - aware of that PR (but thanks for the pointer, as I couldn't find it), however, the docs also need a rev after that merges.

totally agree the doc needs update too

GKE 1.9 should be GA now, so no white listing is required.
cc @salrashid123 @selmanj for GKE doc updates

thanks.  I'll submit the doc bug PR over the weekend
@selmanj   will remove the enableKubernetesAlpha: true setting in the DM template thats there now

this looks resolved.


Set correct content type for posts and patches

Check of content type is json for posts and patches
.... Otherwise the post will stop working with recent api changes

It seems like we dont set any content type

Doesn't seem to be a problem; posting reviews works just fine


Fix errors reported by vimhelplint

Mostly about line-width, but is also has some hidden treasures.
https://travis-ci.org/neomake/neomake/jobs/158598921#L335

opened PR #685


Wrong variable

_indexHandle was using a wrong variable on its typeof test

@fantoine good catch! We are using a Apache 2 CLA for ArangoDB and ArangoJS, which can be found here: https://www.arangodb.com/documents/cla.pdf
We have not been able to set of an electronic system yet. Can you sign, scan and email it to cla (at) arangodb.com?
Thanks a lot
Willi


DO NOT MERGE

Test PR

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-3.9%) to 80.392%

Files with Coverage Reduction
New Missed Lines
%

repository/IngSoft2-Model/Dummy.class.st
4
0%

Totals

Change from base Build<PHONE_NUMBER>:
-3.9%

Covered Lines:
82

Relevant Lines:
102

💛  - Coveralls


callback saves

If the system contains a callback, then additional values are saved to the solution?
I think several of the snippets assume that no such extra values are present.

We would need to make sure that the callbacks all don't save, or error.


Feature: support circuitbreak on kitex proxyless client

Is your feature request related to a problem? Please describe.
Kitex 当前支持配置熔断策略。Istio 目前的治理策略还不支持，envoy proxy 是支持的，可以通过 envoyfilter 进行 patch。
Describe the solution you'd like
Kitex 重试使用参见：https://www.cloudwego.io/zh/docs/kitex/tutorials/service-governance/circuitbreaker/
envoy proxy xDS 中的配置如下：
{
  "consecutive_5xx": {...},
  "interval": {...},
  "base_ejection_time": {...},
  "max_ejection_percent": {...},
  "enforcing_consecutive_5xx": {...},
  "enforcing_success_rate": {...},
  "success_rate_minimum_hosts": {...},
  "success_rate_request_volume": {...},
  "success_rate_stdev_factor": {...},
  "consecutive_gateway_failure": {...},
  "enforcing_consecutive_gateway_failure": {...},
  "split_external_local_origin_errors": ...,
  "consecutive_local_origin_failure": {...},
  "enforcing_consecutive_local_origin_failure": {...},
  "enforcing_local_origin_success_rate": {...},
  "failure_percentage_threshold": {...},
  "enforcing_failure_percentage": {...},
  "enforcing_failure_percentage_local_origin": {...},
  "failure_percentage_minimum_hosts": {...},
  "failure_percentage_request_volume": {...},
  "max_ejection_time": {...},
  "max_ejection_time_jitter": {...},
  "successful_active_health_check_uneject_host": {...}
}

其中参数 failure_percentage_threshold 和 failure_percentage_threshold 可以和 Kitex 中的熔断策略对齐
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

This feature has been merged. Thanks @whalecold for the contribution.


Add ability to take in entityset or entityset args for Featuretools Component

Extension of issue 470. PR 1454 addresses adding the FeatureTools component, but only handles single dataframes/datatables. In order to use FeatureTools fully, we want to be able to use it to combine multiple datasets.
This issue tracks the implementation of that feature. We want to either allow EvalML to take in EntitySets as args, or we want to take in the appropriate args such that we can combine the datatables and create the EntitySets under the hood.
Original quip doc here

After discussion with @dsherry, we decided on

Merge Featuretools (single dataframe/datatable support) into Evalml components (not AutoML), create functionality for ingesting EntitySets (as support for multiple datatables/dataframes),
afterwards, we have two potential options

do component caching for just the featuretools component
create caching for pipelines once Pipelines as DAGs is finished

Next steps: Discuss options with FT team, redo perf tests on initial featuretools component implementation, update quip doc


Increase contrast on the shown video statistics

When looking at the video statistics, the drawn curve can be hard to see.
I suggest to make the curve black, or at least much darker than it is right now.

Should be possible, I'll look into it.
On 06/14/2016 09:26 AM, Henrik Skov Midtiby wrote:

When looking at the video statistics, the drawn curve can be hard to see.
I suggest to make the curve black, or at least much darker than it is
right now.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DanThrane/tekvideo.sdu.dk/issues/80, or mute the
thread
https://github.com/notifications/unsubscribe/ABsypY_aMOdpR8Wfy11hCuFUSCY3SRoVks5qLlezgaJpZM4I1CF1.


License?

Hi, I spotted your license is "free".
First though is "Hey let's troll the guy, asking for a takover of the project" ^^
If you don't really know how to choose a license, I see two simple ways:
Do as most people do in the Python community: Choose the MIT (Expat) license, quoting gnu.org:

This is a lax, permissive non-copyleft free software license, compatible with the GNU GPL. It is sometimes ambiguously referred to as the MIT License.

In case you don't like the expat licence, you can choose one using https://choosealicense.com/.
I personally think you should choose a real license, at least for the "THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY […]

@JulienPalard Many thanks for your hints. I'll get to that this week.

@JulienPalard Done! :)


What is the purpose of duplicate text-decoration under abbr[title]

L87 and L88 both define text-decoration under abbr[title], what's the reason for this? Could it be included in the 'Extended details and known issues' of the README?
https://github.com/necolas/normalize.css/blob/fc091cce1534909334c1911709a39c22d406977b/normalize.css#L88

Here's the PR where it was merged #468
Seems it was necessary at the time, per the discussion there and in the other linked issues.

See https://caniuse.com/#feat=text-decoration


Add our playground to the project again

This PR addresses Issue #475.

Add Sample-iOS.playground to the project.
Set Build Active Scheme of the playground to true to enable importing Swinject package as mentioned in this post.

@1ucas @mpdifran Would one of you review this PR? 🙏

Sure!

I sent a PR to your branch from my Fork adding the XcodeGen property so we don't lose consistency and the playground's reference the next time we generate the project.
Here's the link: XcodeGen Fix

Thanks 1ucas! I merged your PR 👍 😃

Thank you for the review! Merged it😃


1.1.0 version has SyntaxError in build script. Does not show customization screen.

Customization screen does not show up in the left.

in my server.cfg
setr fivem-appearance:customization 1
ensure fivem-appearance

The code I'm using to call this
function open_clothing_store()

    local config = {
        ped = false,
        headBlend = false,
        faceFeatures = false,
        headOverlays = false,
        components = true,
        props = true
    }

    TriggerEvent("cd_drawtextui:temp_hide_text")
    exports['fivem-appearance']:startPlayerCustomization(function(appearance)

        if appearance then
            exports["em_fw"]:update_skin(json.encode(appearance))
            TriggerEvent("cui_wardrobe:open")
        end
        TriggerEvent("cd_drawtextui:temp_show_text")

    end, config)

end

I have no issue with the 1.0 build, but of course, that one is missing config.

I had no success trying to reproduce your issue on my end. Looks like it's something related to parsing the locales file? Try to add this line into your config, before ensuring the resource.
setr fivem-appearance:locale "en"

Alright. I had that line before, it didn't quite make a difference on my end.
Adding that line once more, doesn't clear up the syntax issue nor the issue with the interface being gone.
setr fivem-appearance:customization 1
setr fivem-appearance:locale "en"
ensure fivem-appearance

Not sure what's going on.

I've cleared server and client cache, which didn't help. I've switched the fivem build from stable to canary, no dice there.
Exports are working fine, just no interface. I've had other clients report the same issue.