Response from prompts.choice not working

Hi,
I'm sending these choices to the user as prompts: 'Low 😊', 'Moderate 😐', 'High 😣'
And on Facebook I'm getting the response: I didn't understand. Please choose an option from the list.
It works fine in the emulator. Any ideas?
I'm logging the output of bot.use({receive to an external logging service and the response of event.text is Low �

I've just tested this by trying to use just an emoji as the buttons value (😊) and it doesn't work when it's just an emoji but does if it's just text. I've seen other frameworks that support the user sending emojis through. The strange thing is that some buttons with emojis work, in my previous example Moderate 😐 works.

I double that! I just couldnt get this to work on messenger, i however managed to work it out on the bot framework emulator, any suggestions on what you did around it @gilesbutler

@slidenerd removed the emojis 😩😔🙁👎

Hi people, I played around a bunch to get a workaround.
The problem: Basically, the emoji comes back wrong, as mentioned by @gilesbutler gilesbutler, and is counted two characters. Then the answer is only accepted if the best match is above the default threshold of 0.6 [correct characters divided by total]. Not to mention spaces are alway counted as incorrect.
Before getting into the workaround, I think all that needs to be done on the BotBuilder side is to set the payloads to be the index corresponding to that quick reply because I think it is the payload that gets evaluated when identifying the response. The payloads are already being set to something ("value") here on line 689, but I can't seem to figure out where action.value is set. Any help on this would be appreciated, such as by @Stevenic.
Workaround: You can reconstruct Prompts.choice using Prompts.text and your validation loop (not necessary). You can also change the threshold for matching this way. Caveat - the buttons don't appear in the emulator, but you can just enter 1,2,3 or the correct text to test it there. Also, I pass variables to the validation loop dialog using session.userData, but there's probably a better way.
session.quick_reply("your question", ['Low 😊', 'Moderate 😐', 'High 😣']);

// start validation loop
Object.defineProperty(builder.Session.prototype, 'quick_reply', {
	enumerable: false,
	value: function(text, options) {
		this.userData.context.options = options;
		this.userData.context.text = text;
		this.beginDialog('/check-for-response');
	}
});

// send quick replies
Object.defineProperty(builder.Session.prototype, 'quick_replies', {
	enumerable: false,
  value: function(text, options) {
		var qrs = options.map(function(option, ind) {
			return {
				"content_type": "text",
				"title": option,
				"payload": parseInt(ind+1)
			};
		});
		var msg = new builder.Message(this)
      .text(text)
      .sourceEvent({
        facebook: {
          "quick_replies": qrs
        }
      });
		builder.Prompts.text(this, msg);
	}
});

// validation loop to check quick reply responses
bot.dialog('/check-for-response',
  new builder.SimpleDialog(function(session, results) {
		if (results && results.response) {
			var rsp = results.response;
			//console.log(rsp);
			var options = session.userData.context.options;
			var threshold = .6;
			var match = builder.EntityRecognizer.findBestMatch(options,
				rsp, threshold);
			if (match) {
				var num = match.index;
				var score = match.score;
			} else if(!isNaN(rsp) && rsp > 0 && rsp <= options.length) {
				var num = Number(rsp);
				var score = 1.0;
			}
			if(num) {
				var res = {response:{
					'entity': rsp,
					'index': num,
					'score': score
				}};
	      session.endDialogWithResult(res);
			}	else {
				session.quick_replies(
					"I didn't understand. Please choose an option from the list.",
					options);
			}
		} else {
			session.quick_replies(session.userData.context.text,
				session.userData.context.options);
		}
  })
);

Thanks for the workaround @bmperrea - it would definitely be nicer to get a proper fix for this though, interested to hear what the MS guys think

agreed. It would be nice to

get the emojis to come back intact (somewhere along the line there must be a problem with Unicode support?)

be able to make use of payloads when routing to facebook so that the correct answer is always found when a quick-reply is clicked.

fix the EntityRecognizer not to penalize spaces.

add the option to set the threshold for built-in prompts.

Hey guys... Sorry for the delay on this. I agree that we should probably do a better job with this. Let me investigate this some.

This is a very early iteration as to how I'm getting around the issue:
The correct response with emoji appears on the context.message.sourceEvent message, so a cheeky little override, swapping in this message text if it exists, is doing the trick.
My buttons are emoji only, so the previous workaround is sadly of no use.
var builder = require('botbuilder');
var origPromptRecognize = builder.Prompts.prototype.recognize;
builder.Prompts.prototype.recognize = function (context, cb) {
  if (context.message && context.message.sourceEvent && context.message.sourceEvent.message && context.message.sourceEvent.message.text) {
    context.message.text = context.message.sourceEvent.message.text
  }
  origPromptRecognize.call(this, context, cb)
};

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botbuilder-channels/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


VK downloading error

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.07.18 (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 the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
I upgraded yt-dlp for today and it didn't work for any VK-URLs at all now.
I used May version before and it worked fine.
Youtube downloading works fine...
Ubuntu 22.04 OS
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
$ yt-dlp -f b --merge-output-format mp4 -o Видео/'%(title)s.%(ext)s' https://vk.com/video-173637871_456242073
[vk] -173637871_456242073: Downloading JSON metadata
ERROR: 'generator' object is not subscriptable

$ yt-dlp -vU https://vk.com/video-173637871_456242073
[debug] Command-line config: ['-vU', 'https://vk.com/video-173637871_456242073']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.07.18 [135f05ef6]
[debug] Lazy loading extractors is disabled
[debug] Python 3.10.4 (CPython 64bit) - Linux-5.15.0-33-generic-x86_64-with-glibc2.35 (glibc 2.35)
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 4.4.1 (setts), ffprobe 4.4.1, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, pyxattr-0.7.2, secretstorage-3.3.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.07.18, Current version: 2022.07.18
yt-dlp is up to date (2022.07.18)
[debug] [vk] Extracting URL: https://vk.com/video-173637871_456242073
[vk] -173637871_456242073: Downloading JSON metadata
ERROR: 'generator' object is not subscriptable
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1440, in wrapper
    return func(self, *args, **kwargs)
  File "/home/user/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1516, in __extract_info
    ie_result = ie.extract(url)
  File "/home/user/.local/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 665, in extract
    ie_result = self._real_extract(url)
  File "/home/user/.local/lib/python3.10/site-packages/yt_dlp/extractor/vk.py", line 452, in _real_extract
    return self.url_result(dailymotion_urls[0], DailymotionIE.ie_key())
TypeError: 'generator' object is not subscriptable

Work with:
--cookies
--cookies-from-browser
https://github.com/yt-dlp/yt-dlp/pull/4128#issuecomment-1203546033

$ yt-dlp  https://vk.com/video-173637871_456242073
[Cookies] Extracting cookies from firefox
[Cookies] Extracted 496 cookies from firefox
[vk] -173637871_456242073: Downloading JSON metadata
[vk] -173637871_456242073: Downloading m3u8 information
[info] -173637871_456242073: Downloading 1 format(s): hls-3467
[download] Необычный освежитель воздуха [-173637871_456242073].mp4 has already been downloaded
[download] 100% of 5.26MiB

$ lh *.mp4
-rw-r--r-- 1 live live 5,3M авг  6 17:33 'Необычный освежитель воздуха [-173637871_456242073].mp4'

Must be compiled no later than Jul 27, 2022.
https://github.com/yt-dlp/yt-dlp/commit/59f63c8f0facb71208c8c131935fc4317e96f8b4

full exhaust:
$ yt-dlp -vU https://vk.com/video-173637871_456242073
[debug] Command-line config: ['-vU', 'https://vk.com/video-173637871_456242073']
[debug] User config "/home/live/.config/yt-dlp/config": ['--cookies-from-browser=firefox:jvjddxlz.default-1626613399792']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.07.18 [135f05ef6] (linux_exe)
[debug] Lazy loading extractors is disabled
[debug] Python 3.10.5 (CPython 64bit) - Linux-5.18.0-pf3-x86_64-with-glibc2.36 (glibc 2.36)
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.0.1 (setts), ffprobe 5.0.1
[debug] Optional libraries: certifi-2022.06.15, sqlite3-2.6.0
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/home/live/.mozilla/firefox/jvjddxlz.default-1626613399792/cookies.sqlite"
[Cookies] Extracted 498 cookies from firefox
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.07.18, Current version: 2022.07.18
yt-dlp is up to date (2022.07.18)
[debug] [vk] Extracting URL: https://vk.com/video-173637871_456242073
[vk] -173637871_456242073: Downloading JSON metadata
[vk] -173637871_456242073: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] -173637871_456242073: Downloading 1 format(s): hls-3467
[debug] Invoking hlsnative downloader on "https://vk9-3.vkuser.net/expires/1660058016821/srcIp/<IP_ADDRESS>/pr/40/srcAg/CHROME/ms/<IP_ADDRESS>/type/3/sig/XF5ZN06Bxxo/ct/8/urls/<IP_ADDRESS>%3B<IP_ADDRESS>/clientType/13/zs/12/id/2113275169400/video/"
[download] Необычный освежитель воздуха [-173637871_456242073].mp4 has already been downloaded
[download] 100% of 5.26MiB

ERROR: 'generator' object is not subscriptable

Compiled yt-dlp, today's git slice. Got the same error.
$ ./yt-dlp -vU https://vk.com/video-173637871_456242073
[debug] Command-line config: ['-vU', 'https://vk.com/video-173637871_456242073']
[debug] User config "/home/live/.config/yt-dlp/config": ['--cookies-from-browser=firefox:jvjddxlz.default-1626613399792']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.07.18 [135f05ef6] (zip)
[debug] Lazy loading extractors is disabled
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python 3.10.5 (CPython 64bit) - Linux-5.18.0-pf3-x86_64-with-glibc2.36 (glibc 2.36)
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.0.1 (setts), ffprobe 5.0.1
[debug] Optional libraries: sqlite3-2.6.0
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/home/live/.mozilla/firefox/jvjddxlz.default-1626613399792/cookies.sqlite"
[Cookies] Extracted 502 cookies from firefox
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.07.18, Current version: 2022.07.18
yt-dlp is up to date (2022.07.18)
[debug] [vk] Extracting URL: https://vk.com/video-173637871_456242073
[vk] -173637871_456242073: Downloading JSON metadata
ERROR: 'generator' object is not subscriptable
Traceback (most recent call last):
  File "/mnt/sda3/GIT/yt-dlp/./yt-dlp/yt_dlp/YoutubeDL.py", line 1440, in wrapper
    return func(self, *args, **kwargs)
  File "/mnt/sda3/GIT/yt-dlp/./yt-dlp/yt_dlp/YoutubeDL.py", line 1516, in __extract_info
    ie_result = ie.extract(url)
  File "/mnt/sda3/GIT/yt-dlp/./yt-dlp/yt_dlp/extractor/common.py", line 665, in extract
    ie_result = self._real_extract(url)
  File "/mnt/sda3/GIT/yt-dlp/./yt-dlp/yt_dlp/extractor/vk.py", line 452, in _real_extract
    return self.url_result(dailymotion_urls[0], DailymotionIE.ie_key())
TypeError: 'generator' object is not subscriptable

Works:
$ git describe --long --tags
2022.07.18-26-gedebb6517

Does not work:
$ git describe --long --tags
2022.07.18-78-g061a17abd

git clone https://github.com/yt-dlp/yt-dlp.git
cd yt-dlp
git checkout 59f63c8f0facb71208c8c131935fc4317e96f8b4
make yt-dlp

$ ./yt-dlp https://vk.com/video-173637871_456242073
[Cookies] Extracting cookies from firefox
[Cookies] Extracted 508 cookies from firefox
[vk] -173637871_456242073: Downloading JSON metadata
[vk] -173637871_456242073: Downloading m3u8 information
[info] -173637871_456242073: Downloading 1 format(s): hls-3467
[download] Необычный освежитель воздуха [-173637871_456242073].mp4 has already been downloaded
[download] 100% of 5.26MiB

I wish you had mentioned in OP that you are on the master branch

$ git checkout master

$ git pull
remote: Enumerating objects: 15, done.
remote: Counting objects: 100% (15/15), done.
remote: Compressing objects: 100% (1/1), done.
remote: Total 8 (delta 7), reused 8 (delta 7), pack-reused 0
Распаковка объектов: 100% (8/8), 1.46 КиБ | 373.00 КиБ/с, готово.
Из https://github.com/yt-dlp/yt-dlp
   061a17abd..43aebb7db  master     -> origin/master
Обновление 061a17abd..43aebb7db
Fast-forward
 yt_dlp/extractor/foxnews.py  |  9 ++++-----
 yt_dlp/extractor/francetv.py |  2 +-
 yt_dlp/extractor/heise.py    |  2 +-
 yt_dlp/extractor/vk.py       | 12 ++++++------
 4 files changed, 12 insertions(+), 13 deletions(-)

$ make yt-dlp

$ ./yt-dlp https://vk.com/video-173637871_456242073
[Cookies] Extracting cookies from firefox
[Cookies] Extracted 509 cookies from firefox
[vk] -173637871_456242073: Downloading JSON metadata
[vk] -173637871_456242073: Downloading m3u8 information
[info] -173637871_456242073: Downloading 1 format(s): hls-3467
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 2
[download] Destination: Необычный освежитель воздуха [-173637871_456242073].mp4
[download] 100% of 5.53MiB in 00:02 at 1.97MiB/s
[FixupM3u8] Fixing MPEG-TS in MP4 container of "Необычный освежитель воздуха [-173637871_456242073].mp4"

$ lh *.mp4
-rw-r--r-- 1 live live 5,3M авг  7 01:12 'Необычный освежитель воздуха [-173637871_456242073].mp4'

Checked in LiveCD Mint 20.2 to exclude my settings. Works without cookies. )

git clone https://github.com/yt-dlp/yt-dlp.git
cd yt-dlp
git checkout 59f63c8f0facb71208c8c131935fc4317e96f8b4
make yt-dlp

Works great!!!
$ ./yt-dlp https://vk.com/video-173637871_456242073 [vk] -173637871_456242073: Downloading JSON metadata [vk] -173637871_456242073: Downloading m3u8 information [info] -173637871_456242073: Downloading 1 format(s): hls-3467 [hlsnative] Downloading m3u8 manifest [hlsnative] Total fragments: 2 [download] Destination: Необычный освежитель воздуха [-173637871_456242073].mp4 [download] 100% of 5.53MiB in 00:00 at 10.99MiB/s [FixupM3u8] Fixing MPEG-TS in MP4 container of "Необычный освежитель воздуха [-173637871_456242073].mp4"

git checkout 59f63c8f0facb71208c8c131935fc4317e96f8b4

It was fixed yesterday. There is no need to revert to an old commit.

Master is now working

Ok.
Great job!
Tanks!


Minimum Y-value greater than zero?

Sorry, another question. Is there a way to have the graph's y-axis not start at zero? I have a data set that ranges from 20000-25000, and don't want all of the excess displayed.

You can do it, but you need a bit of pre-processing before. If you know the value you want your chart to start, you can offset all the values by 20000 (in your case), and then in the labelForValue block, you offset the value that is going to be displayed by 20000.
Example:
lineChart.labelForValue = ^(CGFloat value) {     return [NSString stringWithFormat:@"%f", value + 20000]; };

Oh, fantastic. Thanks a bunch.

On Aug 15, 2016, at 5:11 PM, Arthur<EMAIL_ADDRESS>wrote:
You can do it, but you need a bit of pre-processing before. If you know the value you want your chart to start, you can offset all the values by 20000 (in your case), and then in the labelForValue block, you offset the value that is going to be displayed by 20000.
Example:
lineChart.labelForValue = ^(CGFloat value) {
return [NSString stringWithFormat:@"%f", value + 20000];
};
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ArthurGuibert/FSLineChart/issues/35#issuecomment-239930541, or mute the thread https://github.com/notifications/unsubscribe-auth/AL77bWyTzLpRpwVkQcuNTfSXhbWxfmcBks5qgNYBgaJpZM4JkxH5.

@canyondellomo can you tell me how you did this exactly in the code? Thanks!

Hey Arthur-
So I'm not around the code I used for that project, but conceptually you do the following:

Wherever in your project you create your data matrix, add in the Y-axis adjustment- this is as simple as subtracting a hard coded value from the entire matrix.

In your plotting method, when you label your Y-axis, simply add the adjustment back in to the value when printing your string.

That seemed to work for me. If you're still having trouble ping me again and I'll go find the actual code.
Best of luck!
C

On Nov 1, 2016, at 4:30 AM, Alifar<EMAIL_ADDRESS>wrote:
@canyondellomo can you tell me how you did this exactly in the code? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@canyondellomo yes please if you could be so kind 👍
This is the code I have now, but same as you I know the minimum value, where do I need to put the minimum value and take it with the grid?
NSMutableArray* chartData = [NSMutableArray arrayWithCapacity:self.lineData.count];
    NSMutableArray* months = [NSMutableArray arrayWithCapacity:self.lineData.count];
    for(int i=0;i<self.lineData.count;i++) {

        Kobject *z = [self.lineData objectAtIndex:i];
        chartData[i] = [NSNumber numberWithFloat:[z.oPrice floatValue]];
        months[i] = z.oDate;
    }

    self.lineChart = [[FSLineChart alloc] initWithFrame:CGRectMake(15, 188, [UIScreen mainScreen].bounds.size.width - 30, 120)];
    self.lineChart.verticalGridStep = 6;
    self.lineChart.horizontalGridStep = 7;
    self.lineChart.fillColor = [UIColor colorWithRed:0.18f green:0.67f blue:0.84f alpha:0.20f];
    self.lineChart.color = [self.lineChart.dataPointColor colorWithAlphaComponent:0.3];
    self.lineChart.valueLabelPosition = ValueLabelLeftMirrored;
    self.lineChart.indexLabelFont = [UIFont fontWithName:@"HelveticaNeue-Light" size:10];
    self.lineChart.valueLabelFont = [UIFont fontWithName:@"HelveticaNeue-Light" size:9];
    self.lineChart.labelForIndex = ^(NSUInteger item) {
        return months[item];
    };
    self.lineChart.labelForValue = ^(CGFloat value) {

        return [NSString stringWithFormat:@"€ %.2f", value];
    };
    [self.lineChart setChartData:chartData];

Ah okay.
So self.lineData contains the numerical information. Wherever that gets originally populated, subtract your adjustment from each value.
So for instance, in my code, in viewDidLoad, self.visVals, an NSMutableArray, gets filled by a series of JSON objects. Each value it loads in, I subtracted 120 from it because I wanted my Y-axis to start at 120.
int vis=[[metrics objectAtIndex:2] intValue];
vis=vis-120;
[self.visVals addObject:[NSNumber numberWithInt:vis]];

Super simple there. Then, in the plotting area (which you included above), under the labelForValue component, simply call out the value and add your adjustment back to it.
So in mine:
self.visChart.labelForValue = ^(CGFloat value) {
        value=value+120;
        return [NSString stringWithFormat:@"%.f",value];

    };

See if that works!

@canyondellomo it works, thanks a million! 👍

@canyondellomo @ArthurGuibert would you guys mind take a look at the following issue I have?
see: https://github.com/ArthurGuibert/FSLineChart/issues/41
Thanks in advance.

What's wrong with it specifically? Are the values displayed incorrect? Or simply the placement of the X-axis?
Like is your first value actually ~$4.08 or whatever? Or should it be in the negatives and it's not displayed properly?
C

On Jun 23, 2017, at 5:36 AM, Alifar<EMAIL_ADDRESS>wrote:
@canyondellomo @arthurguibert would you guys mind take a look at the following issue I have?
see: #41
Thanks in advance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

 On on mobile and haven't worked on this in a while... BUT
Check where you actually plot the chart data. Pretty sure you only do the value adjustment (in this case your lowFloat value) in the labeling section. All you're doing with our workaround is tricking the Y-axis labeling... not actually changing where the points are plotted.
Check if that helps?
C

On Jun 23, 2017, at 5:36 AM, Alifar<EMAIL_ADDRESS>wrote:
@canyondellomo @arthurguibert would you guys mind take a look at the following issue I have?
see: #41
Thanks in advance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you for your time C,
The whole code looks as followed:
float n = [[sort objectAtIndex:0] floatValue];

        NSString* forFloat = [NSString<EMAIL_ADDRESS>n];
        lowFloat = [forFloat floatValue]-0.001;

NSMutableArray* chartData = [NSMutableArray arrayWithCapacity:self.lineData.count];
            NSMutableArray* months = [NSMutableArray arrayWithCapacity:self.lineData.count];
            for(int i=0;i<self.lineData.count;i++) {

                Kobject *z = [self.lineData objectAtIndex:i];
                chartData[i] = [NSNumber numberWithFloat:[z.oPrice floatValue]-lowFloat];
                months[i] = z.oDate;
            }

            self.lineChart = [[FSLineChart alloc] initWithFrame:CGRectMake(15, 44, [UIScreen mainScreen].bounds.size.width - 30, 120)];
            self.lineChart.verticalGridStep = 6;
            self.lineChart.horizontalGridStep = 7;
            self.lineChart.fillColor = [UIColor colorWithRed:0.24 green:0.72 blue:0.68 alpha:0.20];
            self.lineChart.color = [self.lineChart.dataPointColor colorWithAlphaComponent:0.3];
            self.lineChart.valueLabelPosition = ValueLabelLeftMirrored;
            self.lineChart.indexLabelFont = [UIFont fontWithName:@"HelveticaNeue-Light" size:10];
            self.lineChart.valueLabelFont = [UIFont fontWithName:@"HelveticaNeue-Light" size:9];
            self.lineChart.labelForIndex = ^(NSUInteger item) {
                return months[item];
            };
            self.lineChart.labelForValue = ^(CGFloat value) {

                return [NSString stringWithFormat:@"$ %.2f", value+lowFloat];
            };
            [self.lineChart setChartData:chartData];
            [view addSubview:self.lineChart];

Would you mind pointing me what I need to add more and where? Also, it only shows the strange behaviour when the values are above 10.
Thanks in advance.

If I’m correct,
chartData[i] = [NSNumber numberWithFloat:[z.oPrice floatValue]-lowFloat];
Should just be
chartData[i] = [NSNumber numberWithFloat:[z.oPrice floatValue]];

On Jun 23, 2017, at 9:49 AM, Alifar<EMAIL_ADDRESS>wrote:
Thank you for your time C,
The whole code looks as followed:
float n = [[sort objectAtIndex:0] floatValue];
    NSString* forFloat = [NSString<EMAIL_ADDRESS>n];
    lowFloat = [forFloat floatValue]-0.001;

NSMutableArray* chartData = [NSMutableArray arrayWithCapacity:self.lineData.count];
NSMutableArray* months = [NSMutableArray arrayWithCapacity:self.lineData.count];
for(int i=0;i<self.lineData.count;i++) {
            Kobject *z = [self.lineData objectAtIndex:i];
            chartData[i] = [NSNumber numberWithFloat:[z.oPrice floatValue]-lowFloat];
            months[i] = z.oDate;
        }

        self.lineChart = [[FSLineChart alloc] initWithFrame:CGRectMake(15, 44, [UIScreen mainScreen].bounds.size.width - 30, 120)];
        self.lineChart.verticalGridStep = 6;
        self.lineChart.horizontalGridStep = 7;
        self.lineChart.fillColor = [UIColor colorWithRed:0.24 green:0.72 blue:0.68 alpha:0.20];
        self.lineChart.color = [self.lineChart.dataPointColor colorWithAlphaComponent:0.3];
        self.lineChart.valueLabelPosition = ValueLabelLeftMirrored;
        self.lineChart.indexLabelFont = [UIFont fontWithName:@"HelveticaNeue-Light" size:10];
        self.lineChart.valueLabelFont = [UIFont fontWithName:@"HelveticaNeue-Light" size:9];
        self.lineChart.labelForIndex = ^(NSUInteger item) {
            return months[item];
        };
        self.lineChart.labelForValue = ^(CGFloat value) {

            return [NSString stringWithFormat:@"$ %.2f", value+lowFloat];
        };
        [self.lineChart setChartData:chartData];
        [view addSubview:self.lineChart];

Would you mind pointing me what I need to add more and where? Also, it only shows the strange behaviour when the values are above 10.
Thanks in advance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ArthurGuibert/FSLineChart/issues/35#issuecomment-310670436, or mute the thread https://github.com/notifications/unsubscribe-auth/AL77bXhWMhNpo0I4zRL3oHk7xhoEslYzks5sG8JxgaJpZM4JkxH5.

That corrected the linechart, but gives the wrong values, see screenshot:

Change this
return [NSString stringWithFormat:@"$ %.2f", value+lowFloat];
To this?
return [NSString stringWithFormat:@"$ %.2f", value-lowFloat];

On Jun 23, 2017, at 10:34 AM, Alifar<EMAIL_ADDRESS>wrote:
That corrected the linechart, but gives the wrong values, see screenshot:
https://user-images.githubusercontent.com/6464812/27486725-c8ff89d6-5831-11e7-8ec0-f1871727f7b6.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ArthurGuibert/FSLineChart/issues/35#issuecomment-310681776, or mute the thread https://github.com/notifications/unsubscribe-auth/AL77bVU9xor_Reu_6IGHpbL4ktMOw2vLks5sG8zngaJpZM4JkxH5.

Hmmm now it shows this, getting a bit confused here lol

If I remove the lowfloat completely it shows the right line.

So are you set now? I’m confused as well haha.

On Jun 23, 2017, at 10:43 AM, Alifar<EMAIL_ADDRESS>wrote:
If I remove the lowfloat completely it shows the right line.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ArthurGuibert/FSLineChart/issues/35#issuecomment-310684305, or mute the thread https://github.com/notifications/unsubscribe-auth/AL77bVlgR9gFKR89LYesRIeySVL6Dmosks5sG88MgaJpZM4JkxH5.

Well, when I remove it on the values that are set lower then 10 it will have pretty much no change in the linechart, see:

So, I will make an if statement if the value goes above then do it your way 👍
Thanks again man!

Yeah no worries. Sorry I couldn’t be super helpful, it’s been a while.

On Jun 23, 2017, at 10:47 AM, Alifar<EMAIL_ADDRESS>wrote:
Well, when I remove it on the values that are set lower then 10 it will have pretty much no change in the linechart, see:
https://user-images.githubusercontent.com/6464812/27487266-862add0c-5833-11e7-8d23-66fd480e17e1.png
So, I will make an if statement if the value goes above then do it your way 👍
Thanks again man!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ArthurGuibert/FSLineChart/issues/35#issuecomment-310685418, or mute the thread https://github.com/notifications/unsubscribe-auth/AL77baY70sH-zVDC9tY-IJgseqU-wNLtks5sG8_6gaJpZM4JkxH5.


remove deprecated code for v0.10.0

Description
search our code base, find all deprecated code, remove them all.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Has the Feature been Requested Before?
Please link any other
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.

Finished.


Unable to process input arrays that have no non-empty values

When I pass a []string or []sql.NullString array to a PL/SQL function (using a TABLE OF VARCHAR2 input parameter), I consistently see errors whenever the array does not contain any non-empty values: ORA-03101: invalid input data for occurs when the array has zero elements, and ORA-03146: invalid buffer length for TTC field occurs when the array has one or more elements but they're all empty.
I'm also seeing ORA-03146: invalid buffer length for TTC field when I pass an empty []byte array to a PL/SQL function (using a RAW input parameter).
This Gist demonstrates the problems I mentioned above:
https://gist.github.com/robstradling/ca8619f832e19b0785a4c89502cfedbf
Are these bugs in go-ora?  Or am I doing something wrong?

these are bugs and I am fixing

fixed in v2.7.4

Many thanks @sijms !!


Issue while compiling the client code

I have been following the steps listed in the tutorial about building the client code and I seem to run into the following error.
CMake Error at CMakeLists.txt:106 (find_package):
  By not providing "Findmicrocdr.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "microcdr",
  but CMake did not find one.

  Could not find a package configuration file provided by "microcdr"
  (requested version 1.2.0) with any of the following names:

    microcdrConfig.cmake
    microcdr-config.cmake

  Add the installation prefix of "microcdr" to CMAKE_PREFIX_PATH or set
  "microcdr_DIR" to a directory containing one of the above files.  If
  "microcdr" provides a separate development package or SDK, be sure it has
  been installed.

-- Configuring incomplete, errors occurred!
See also "/home/arsh/Micro-XRCE-DDS-Client/build/CMakeFiles/CMakeOutput.log".
Makefile:160: recipe for target 'cmake_check_build_system' failed
make: *** [cmake_check_build_system] Error 1

The error is exactly similar to the one that was asked here.
Even after following the intended work around I still seem to have the same problem.
It would be great if I could get some help here.

Hello, I just have followed the instructions and everything works ok:
git clone -b develop https://github.com/eProsima/Micro-XRCE-DDS-Client
cd Micro-XRCE-DDS-Client
mkdir build && cd build
cmake .. -DUCLIENT_BUILD_EXAMPLES=ON
make

Here you have a screencast of my testing session:
https://asciinema.org/a/sV52py9wwGDlwSfJFaiYmHAkl
Let us know if following these instructions everything works.

Hi, this issue occurs when following https://github.com/eProsima/Micro-XRCE-DDS-Apps/tree/microxrce_dev/Zephyr,
clone the git, install zephyr and try to build,..
CMake Error at CMakeLists.txt:110 (find_package):
Could not find a configuration file for package "microcdr" that exactly
matches requested version "1.2.0".
The following configuration files were considered but not accepted:

using the linux default gcc compiler is fine ,but the smae issue occurs when switching the arm toolchain. find lib seems to have the problem,and attempt to manually add lib links did not work. The following is the error message, I will be appreciate that provide idea to fix it
-- Install configuration: "Release"
-- Up-to-date: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/lib/libmicrocdr.a
-- Up-to-date: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/include/ucdr
-- Up-to-date: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/include/ucdr/visibility.h
-- Up-to-date: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/include/ucdr/microcdr.h
-- Up-to-date: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/include/ucdr/config.h
-- Up-to-date: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/share/microcdr/cmake/microcdrTargets.cmake
-- Installing: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/share/microcdr/cmake/microcdrTargets-release.cmake
-- Up-to-date: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/share/microcdr/cmake/microcdrConfig.cmake
-- Up-to-date: /home/next/Workspace/Micro-XRCE-DDS-Client/build/temp_install/microcdr-1.2.1/share/microcdr/cmake/microcdrConfigVersion.cmake
[ 31%] Completed 'microcdr'
[ 50%] Built target microcdr
Scanning dependencies of target uclient
[ 56%] Performing configure step for 'uclient'
loading initial cache file /home/next/Workspace/Micro-XRCE-DDS-Client/build/uclient-prefix/tmp/uclient-cache-.cmake
-- Setting build type to 'Release' as none was specified.
CMake Error at CMakeLists.txt:122 (find_package):
By not providing "Findmicrocdr.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "microcdr",
but CMake did not find one.
Could not find a package configuration file provided by "microcdr"
(requested version 1.2.1) with any of the following names:
microcdrConfig.cmake
microcdr-config.cmake

Add the installation prefix of "microcdr" to CMAKE_PREFIX_PATH or set
"microcdr_DIR" to a directory containing one of the above files.  If
"microcdr" provides a separate development package or SDK, be sure it has
been installed.

Could you provide instructions to reproduce using the ARM toolchain?

Hi @pablogs9 ,I encountered this problem when compiling agents using the NDK

git clone -b develop https://github.com/eProsima/Micro-XRCE-DDS-Client
cd Micro-XRCE-DDS-Client
mkdir build && cd build
C:\Android_SDK\cmake\3.18.1\bin\cmake.exe -DCMAKE_SYSTEM_NAME=Android -DCMAKE_TOOLCHAIN_FILE=C:\Android_SDK\ndk-bundle\build\cmake\android.toolchain.cmake -DANDROID_ABI=x86 -DANDROID_PLATFORM=android-25 -DANDROID_NDK=C:\Android_SDK\ndk-bundle -DCMAKE_MAKE_PROGRAM=C:\Android_SDK\cmake\3.18.1\bin\ninja.exe -DCMAKE_BUILD_TYPE=Debug -DCMAKE_SYSTEM_NAME=Android -DCMAKE_ANDROID_ARCH_ABI=x86 -DANDROID_NATIVE_API_LEVEL=9 -DCMAKE_SYSTEM_VERSION=25 -GNinja -DUCLIENT_BUILD_EXAMPLES=ON -DCMAKE_ANDROID_NDK_TOOLCHAIN_VERSION=clang -DANDROID_TOOLCHAIN=clang -DCMAKE_GENERATOR="Ninja" ..
C:\Android_SDK\cmake\3.18.1\bin\ninja.exe

Hi, I had the same issue pop up when cross compiling for a raspberry-pi 0W using a buildroot generated toolchain.
The issue was:
When previously building, I succeeded and then tried to install the built binaries onto my buildroot target using the cmake install prefix flag. Apparantly the flag was ignored for some reason and the binaries instead got installed on my host system which included the cmake Find files. This was causing the new cross-compile build to look for the binaries/libraries on my host system.
The issue went away  when I manually deleted the installed binaries from /usr/local/lib
For reference, the cmake toolchain file I am using is:
`
set(CMAKE_SYSTEM_NAME Linux)
set(CMAKE_TRY_COMPILE_TARGET_TYPE STATIC_LIBRARY)
set(CMAKE_SYSTEM_VERSION 1)
list(APPEND CMAKE_MODULE_PATH ${CMAKE_CURRENT_LIST_DIR})
set(CMAKE_C_FLAGS_DEBUG "" CACHE STRING "Debug CFLAGS")
set(CMAKE_C_FLAGS_RELEASE " -DNDEBUG" CACHE STRING "Release CFLAGS")
Build type from the Buildroot configuration
set(USE_NO_MAYBE_UNINITIALIZED 1)
set(USE_NO_UNINITIALIZED 1)
set(CMAKE_C_FLAGS "-D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -O2 -g0 -D_FORTIFY_SOURCE=1" CACHE STRING "Buildroot CFLAGS")
set(CMAKE_EXE_LINKER_FLAGS "" CACHE STRING "Buildroot LDFLAGS for executables")
set(CMAKE_SHARED_LINKER_FLAGS "" CACHE STRING "Buildroot LDFLAGS for shared libraries")
set(CMAKE_MODULE_LINKER_FLAGS "" CACHE STRING "Buildroot LDFLAGS for module libraries")
set(CMAKE_SYSROOT "$ENV{TOOLCHAIN_DIR}/arm-buildroot-linux-gnueabihf/sysroot")
set(CMAKE_FIND_ROOT_PATH ${CMAKE_SYSROOT})
set(CMAKE_LIBRARY_ARCHITECTURE arm-buildroot-linux-gnueabihf)
set(ENV{PKG_CONFIG_SYSROOT_DIR} ${CMAKE_SYSROOT})
This toolchain file can be used both inside and outside Buildroot.
set(CMAKE_C_COMPILER "$ENV{TOOLCHAIN_DIR}/bin/arm-buildroot-linux-gnueabihf-gcc")
set(CMAKE_CXX_COMPILER "$ENV{TOOLCHAIN_DIR}/bin/arm-buildroot-linux-gnueabihf-g++")
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-maybe-uninitialized -Wno-unused-parameter -Wno-unused-function -lpthread")
set(CMAKE_FIND_ROOT_PATH_MODE_PROGRAM NEVER)
set(CMAKE_FIND_ROOT_PATH_MODE_LIBRARY ONLY)
set(CMAKE_FIND_ROOT_PATH_MODE_INCLUDE ONLY)
I am setting the TOOLCHAIN_DIR to buildroot'soutput/host`
Also note the additional linkages to libpthread. I found that the linkage set in the CMakeLists.txt doesn't agree with the cross-compile system and had to edit the following out:
line 369, comment out "find_package(Threads REQUIRED)"
While I haven't attempted to build the client libraries myself because I am instead using microROS static libraries built using ROS2, I suspect a similar problem might be causing the missing microcdr error.


Enable multiple authenticators and customizable auth

About the tests:
there is a test in go-template for the symphony style roles enforcements and in hybrid for the oauth2 style enforcement.


Move to fsspec org?

fsspec related repos are gathering at https://github.com/fsspec . We should probably move this repo too - although it integrates nicely with Dask, it doesn't strictly need to be used in conjunction with Dask at all.

Makes sense.

+1

done!


readme: change syntax highlighting from html to js

A portion of the code wasn't being highlighted. This fixes it.
Here's the rich diff.

:+1:


docs: outline usage of condition based export/import maps

Oh, MDX, I never would of thought of that. I was trying html 🙈

Just realizing this didn't get sent last night...
Pulled some prose into a list and hid some stuff in a <details> element, let me know what you think...
The linter went crazy on when using "```html" in the one example. If you have thoughts on how to escape that, happy to clean it up some more, but working seemed more important the highlighted, here.
Separately, I couldn't get npm run dev:website to run a preview locally, so I ended up thrashing Netlify to test the build.


[Enhancement] Add docs for mutateExisting policies

Description
KDP: https://github.com/kyverno/KDP/pull/4.
Implementation: https://github.com/kyverno/kyverno/pull/3669.
To highlight this feature:

the mutateExisting policy will be triggered on trigger resource or policy updates (CREATION, UPDATE, DELETE)
the annotation policies.kyverno.io/last-applied-patches will be added to the target resource indicating the last applied patches
an event will be generated for policy application

3h57m       Normal    PolicyApplied       secret/test-secret-1                policy test-post-mutation/mutate-deploy-on-configmap-update applied to Secret/staging/test-secret-1 successfully

A note on this:

the permissions need to be granted to Kyverno service account properly, like how generate works. Currently, there's no default ClusterRole and ClusterRoleBinding in Kyverno blueprint as Kubernetes doesn't allow "empty" ClusterRole.

A few sample policies for testing:

add a label to the existing secret using patchStrategicMerge:

apiVersion: kyverno.io/v1
kind: ClusterPolicy
metadata:
  name: "test-post-mutation"
spec:
  rules:
    - name: "mutate-deploy-on-configmap-update"
      match:
        any:
        - resources:
            kinds:
            - ConfigMap
            names:
            - dictionary-1
            namespaces:
            - staging
      preconditions:
        any:
        - key: "{{ request.operation }}"
          operator: Equals
          value: DELETE
      mutate:
        targets:
        - apiVersion: v1
          kind: Secret
          name: test-secret-1
          namespace: "{{ request.object.metadata.namespace }}"
        patchStrategicMerge:
          metadata:
            labels:
              foo: bar

add a label defined with a JMESpath variable using JSON patch:

apiVersion: kyverno.io/v1
kind: ClusterPolicy
metadata:
  name: "test-post-mutation"
spec:
  rules:
    - name: "mutate-deploy-on-configmap-update"
      match:
        any:
        - resources:
            kinds:
            - ConfigMap
            names:
            - dictionary-1
            namespaces:
            - staging
      preconditions:
        any:
        - key: "{{ request.operation }}"
          operator: Equals
          value: DELETE
      mutate:
        targets:
        - apiVersion: v1
          kind: Secret
          name: test-secret-1
          namespace: "{{ request.object.metadata.namespace }}"
        patchStrategicMerge:
          patchesJson6902: |-
            - op: add
              path: "/metadata/labels/env"
              value: "{{ request.userInfo.username }}"

restart the deployment's pod on configmap updates:

apiVersion: kyverno.io/v1
kind: ClusterPolicy
metadata:
  name: "test-post-mutation"
spec:
  rules:
    - name: "mutate-deploy"
      match:
        any:
        - resources:
            kinds:
            - ConfigMap
            names:
            - dictionary-1
            namespaces:
            - staging
      mutate:
        targets:
        - apiVersion: apps/v1
          kind: Deployment
          name: test-deploy
          namespace: staging
        patchStrategicMerge:
          spec:
            template:
              metadata:
                annotations:
                  kyverno.io/since-last-update: "{{ time_since('', '{{ request.object.metadata.creationTimestamp }}', '') }}"

Slack discussion
No response

Adding example policies for variables support, PR https://github.com/kyverno/kyverno/pull/3998:

policy with {{@}}

apiVersion: kyverno.io/v1
kind: ClusterPolicy
metadata:
  name: sync-cms
spec:
  mutateExistingOnPolicyUpdate: false
  rules:
  - name: concat-cm
    match:
      any:
      - resources:
          kinds:
          - ConfigMap
          names:
          - cmone
          namespaces:
          - foo
    mutate:
      targets:
        - apiVersion: v1
          kind: ConfigMap
          name: cmtwo
          namespace: bar
        - apiVersion: v1
          kind: ConfigMap
          name: cmthree
          namespace: bar
      patchStrategicMerge:
        data:
          key<EMAIL_ADDRESS>

Policy referes target variable:

apiVersion: kyverno.io/v1
kind: ClusterPolicy
metadata:
  name: sync-cms
spec:
  mutateExistingOnPolicyUpdate: false
  rules:
  - name: concat-cm
    match:
      any:
      - resources:
          kinds:
          - ConfigMap
          names:
          - cmone
          namespaces:
          - foo
    mutate:
      targets:
        - apiVersion: v1
          kind: ConfigMap
          name: cmtwo
          namespace: bar
        - apiVersion: v1
          kind: ConfigMap
          name: cmthree
          namespace: bar
      patchesJson6902: |-
        - op: add
          path: "/metadata/labels/env"
          value: "{{ target.data.key }}"


DEPRECATION WARNING *** You are calling a method named ul

Hi,
I'm getting the warning when trying to access 'ul':
block_element.ul(index: position).lis
The warning looks like:
*** DEPRECATION WARNING
*** You are calling a method named ul at /Users/jordipons/Documents/repos/repo_one/block.rb:59:in `block_choices'.
*** This method does not exist in page-object so it is being passed to the driver.
*** This feature will be removed in the near future.
*** Please change your code to call the correct page-object method.
*** If you are using functionality that does not exist in page-object please request it be added.

I've tried to use this: block_element.unordered_list(index: position).lis but then I get the following error:
NoMethodError: undefined method `unordered_list' for #<Watir::Div:0x007ffe1dad6880>

How can I solve the warning?
Thanks

Hi @jordiponsllaurado ,
lis is not supported in page-object. You can try using,
block_element.unordered_list(index: position).list_item_elements
Let me know if the issue is solved or not.

I moved the unordered_list outside the method, so at the beginning of the class and now it works


ion-nav-bar out-of-sync with nested tabs

See an example in the following plunker (using RC2):
http://plnkr.co/edit/mR41D2plb796LXcXFl5E?p=preview
The tab views are broken down as:
- Home
  - Home Tab 1
  - Home Tab 2
- About
  - About Tab 1
  - About Tab 2

Perform the following steps:
Select "About" tab (bottom-right)
Select "Learn" tab (top-right)
Select "Home" tab (bottom-left)
Notice that the header title is now showing "About Tab 1", but the content displayed is "This is Home Tab 1"
Now, continue with the following steps:
Select "Analytics" tab (top-right)
Select "Dashboard" tab (top-left)
Notice that the header title correctly matches the content.

Greetings @jeremyalan!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Tabs within tabs!
Tabs expects the parent state to be abstract. This means you'll need to update your links dynamically through a service or something, but I think that's an acceptable complication as nested tabs isn't something we officially support (I'm actually pretty amazed it works as well as it does).
This should set you on the right path, but I assume you want to add some conditional logic regarding what page the bottom tab links to
http://plnkr.co/edit/hhi31o9T2J84172Ug4wq?p=preview

Thanks for the info, @perrygovier. Is that an official stance for Ionic 1.0 on whether nested tabs are supported? It makes sense, I just haven't heard anyone from the team make that sort of blanket statement on other similar issues.
Luckily, we have re-designed the navigation in our app, so this is no longer a show stopper for us. :-)
Thanks again!


lock healthcare administrators out of /admin/

We will be giving the is_admin property to healthcare administrators and using is_superuser for CDS folks.

Closing because this went in in #65


[CPAOT] Make sure any methods marked as MethodImpl.AggressiveOptimization is not compiled even if it is a dependency of some other methods.

@trylek pointed out offline that #7748 might be incomplete. In particular, a method marked as MethodImpl.AggressiveInlining might not initially be rooted, but it could be added to the graph as the dependency of another method (e.g. it is being called), so we still end up compiling them.

I believe that it should be easiest to model these methods akin the already existing "uncompilable methods" in CPAOT. Certain constructs (HW intrinsics depending on a particular processor version, some helpers not supported by R2R today etc.) throw RequiresRuntimeJitException during method compilation that subsequently gets caught in CompileMethodInternal and marks the method in question as "uncompilable"; such methods are subsequently excluded from the MethodEntryPoints R2R header table.
In your case, the only difference is that we don't actually need / want to throw a RequiresRuntimeJitException because we don't want to spin up the native JIT at all. I believe we should basically short-circuit the logic in CompileMethodInternal pertaining to uncompilable methods (see the catch clause at line
https://github.com/dotnet/corert/blob/946883b1fd62ab972ee83d484503e2fa9b4e2f7a/src/JitInterface/src/CorInfoImpl.cs#L192
to basically do the same without running JIT at all for aggressively compiled methods. I still claim you should copy around the code comment from CoreCLR and mark it with TODO-PERF because I believe the comment clearly indicates room for improvement in exactly the space where CPAOT is expected to be able to outperform the legacy Crossgen. Once you're done, you should be able to delete the two lines in ReadyToRunLibraryRootProvider.

AggressiveInlining on a method does not alter how the method so-marked is jitted (with perhaps one exception: in some cases we may not  propagate jit-deduced noinline back to the metadata).
It is also not a guarantee; inlining can fail for various reasons even for callees marked with AggressiveInlining.
So I don't really see how it should factor into AOT logic.

@AndyAyersMS The title of the issue was wrong. It should have said AggressiveOptimization

Thanks for the clarification, and sorry for the typo.


iCloud3 v3, Beta 19.1 (7/10/2023)

iCloud3 v3, Beta 19.1 was Uploaded
Changes:

Bug Fix - Fixed a problem that may be encountered if you have a watch paired to an iPhone and are tracking the iPhone but not the watch.
New Sensors:

[devicename]_travel_time_hhmm - Displays the travel time to the zone in an hh:mm format.
[devicename]_arrival_time - The expected arrival time to the zone (current time + travel time) (5:25p, 17:25) or the time you arrived in the Home or other Tracked From zone (@5:25p, @17:25)

Simplified the sensors associated with track_from_zones other than Home.

Important Links
Installing using HACS - iCloud3 is not available on the HACS base. You have set it up as a custom repository.
See here for instructions on how to do that and then install it. If you just go to HACS and do the installation, you will be installing v2.4.7, not v3.
Download and Install - Go to the iCloud3 Releases page here. Scroll down past the notes, then select Assets, then select icloud3.zip. Download the zip file and unzip it into the /config/custom_components/icloud3 directory. Then restart HA.
Migrating from v2.4._ - See here for instructions on migrating from from an older version.
iCloud3 v3 Documentation - It is still in process, does not include the latest updates to iCloud3 and can be found here

I’m on this beta and stationary zones aren’t working for me anymore.

There haven’t been any changes to Stationary zones for a long time. I was out last night and went into a StatZone without any problems. The Event Log tracking Results line shows when you should go into one and the message about any delays due to movement appeared to be working.
What problems do you see? Does the Event Log show the StatZone activity correctly? Have with checked th streets on the Configured Settings > Special Zones screen?

I dug through the logs this morning and I think it might have just been a GPS drift issue where my wife was yesterday. She was there for hours but I kept seeing old location events and events saying she moved too far, sometimes meters, sometimes kilometers but she was in the same spot and it was resetting the stationary timer. I’m guessing the gps accuracy kept changing and showing her in different location.

The last line of the error message indicates the error is coming from the HA Proximity component, not iCloud3.
File "/usr/src/homeassistant/homeassistant/components/proximity/init.py", line 293, in async_check_proximity_state_change
iCloud3 does everything the Proximity component does and a lot more. There are many sensors available for distances between devices and zones that can be used in automations and scripts.
Remove the Proximity component and you should be all set.


Update Tags docuemntation content

Tags content needs to be updated to align with the latest design spec.
Note: The anatomy section is not part of this update so it can be left out.

I can't find the spec design document - it seems it's moved or expired - so is this valid still?
@hippee-lee


Site not found?

I have deployed the completed web app to firebase, however when I go there, I get a message saying 'Site not Found'. Is there any way to fix this?

Could you paste the output of the firebase deploy command and provide the URL of your deployed app?


Services Page: Extra spacing on the right side of the services page

When you navigate to any service page, the page is scrollable left to right because there is extra white space on the Righthand side of the page. This is on all browsers and all screen sizes.

@Wholloran can you provide a screenshot of this? I'm not able to see the extra spacing at the right at all screens, only to the smaller ones and due to the footer button, this is what you mean?

If you go to any services page, on any size desktop or browser and scroll to the right you will see this extra spacing. it seems like the page it self-doesn't match the nav bar width

No longer Any extra space, Closing ticket


how to use instance group

Hi
I have a question about instance_group.
In TIS, i'm using instance_group in config.pbtxt.
instance_group [{
count: 3,
kind: KIND_GPU
}]
i want to use instance_group count 3 in single gpu in pytriton.
Is there a way to use instance_group as above in pytriton?

Hi,
it can be achieved by passing multiple inference callables to the infer_func paramter of the bind method. Please take a look at
https://triton-inference-server.github.io/pytriton/0.1.5/deploying_models/#multi-instance-model-inference

then, if i wanna two infer_func in 0 gpu device and one infer_func in 1 gpu device,
should create a devices list as follows?
infer_func=_infer_function_factory(devices=["cuda:0", "cuda:0", "cuda:1"])

right

@taehwakkwon can you share more context on your model environment? how do you currently start your model parts - as single process or separate processes run by mpirun/srun/torchrun?
For single process job you can use same approach as above:
DEVICES = [("cuda:0", "cuda:1"),("cuda:2", "cuda:3"),("cuda:4", "cuda:5"),("cuda:6", "cuda:7"),]

class _InferFuncWrapper:
    def __init__(self, model: torch.nn.Module, device: str):
        self._model = model
        self._devices = devices

    @batch
    def __call__(self, **inputs):
        (input1_batch,) = inputs.values()
        input1_batch_tensor = torch.from_numpy(input1_batch).to(self._devices[0])
        output1_batch_tensor = self._model(input1_batch_tensor)
        output1_batch = output1_batch_tensor.cpu().detach().numpy()
        return [output1_batch]

def _infer_function_factory(devices: List[Tuple[str]]):
    infer_funcs = []
    for d1, d2 in devices:
        model = load_model(d1, d2)
        infer_funcs.append(_InferFuncWrapper(model=model, devices=devices))

    return infer_funcs

But there is a problem with setup with multiple processes as inference threads cannot communicate with them to perform call.
We're working on "remote mode" feature which could handle such use case. Each script will connect to already run Triton Inference Server and handle requests from it.  We think this feature running on single node should be ready on end of July 2023.

Hi, it can be achieved by passing multiple inference callables to the infer_func paramter of the bind method. Please take a look at https://triton-inference-server.github.io/pytriton/0.1.5/deploying_models/#multi-instance-model-inference

when infer_func=[func_1, func_1, func_1],  does it means 3 requests can be processed at the same time? (parallel=3)
(And if I want to make the server parallel=100, just repeat func_1 100 times?  that is so strange)
in other words, if func_1 takes 10 ms,   then the client sends 3 requests to the server, it will get the result in 10 ms + socket_time ?


Fixing the location for config file. Look for the installation dir in registry. Then fallback to existing hardcoded location.

Approved.


[Feature] Spawner type name in display name

I think the spawner name should be "{Spawner_type} Spawner" would be better. Ex: "Allay Spawner"
If you are not going to change this please contact with me and explain me where can i get the spawner name as string and add it to the spawners display name. Thanks

If this ever gets added, could we still keep functionality of the old named spawners while using the new name for new spawners?

@tbm00 we're still considering adding this feature but yes, we would likely make it backwards compatible.


a11y: disabled textfield not announced as textfield

TextField(
   enabled: false,
),

On Android with TalkBack enabled this should be announced as a disabled editbox. It is currently not announced at all. (It looks like the TextField flag is missing from the semantics tree when the TextField is disabled).
I haven't tested on iOS (and I am not 100% of what the native behavior is there), but I assume it's also broken there.
/cc @dnfield for a11y
/cc @chunhtai @justinmc for TextFields

TextField wraps an EditableText within an IgnorePointer:
https://github.com/flutter/flutter/blob/24f483cf25aab43a389a64f15bf9cc1aee130f47/packages/flutter/lib/src/material/text_field.dart#L1026
RenderIgnorePointer has the following TODO:
https://github.com/flutter/flutter/blob/24f483cf25aab43a389a64f15bf9cc1aee130f47/packages/flutter/lib/src/rendering/proxy_box.dart#L2975-L2982
Which is probably to blame here. When the textfield is disabled, IgnorePointer is enabled and all semantics from EditableText (like isTextField, etc) is no longer included in the tree.

I tried on iOS with VoiceOver enabled and you're right that it's also broken there.  For an enabled CupertinoTextField, it's announced correctly as "text field"., and for a disabled CupertinoTextField, it's not announced at all.

Hi @justinmc @goderbauer and probably @Hixie,
Can you take a look at my prototype pr?
https://github.com/flutter/flutter/pull/37199

design doc https://docs.google.com/document/d/1WuVBc58_wV6__ZCe1NgYsnvTJtSdOtVcwSzPwM_YT74/edit?usp=sharing

Is this still being actively worked on? If we have addressed it, please close the bug.

No, I have a draft up, but I has been distracted due to other higher priority issues. This is currently sitting in my back log, and i don't think i can get to it in the near future. Has the priority of this issue shifted?

@chunhtai no change on priority. I am simply triaging old issues and noticed this one.

https://github.com/flutter/flutter/issues/59402 related
@goderbauer  The input decorator can take in prefix and suffix widget that can respond to pointer event, but is it something we should worry about in this case? If it is the user passes in the widget, should the textfield disable the gesture on that widget?

What's today's behavior when you do that and disable the text field?

I just tested. the whole thing is exclude from semantics, including the suffix button. The IgnorePointer is wrapping everything in the textfield.

flutter doctor -v
[✓] Flutter (Channel dev, 1.20.0-7.1.pre, on Mac OS X 10.15.6 19G73, locale
    en-GB)
    • Flutter version 1.20.0-7.1.pre at /Users/taha/Code/flutter_dev
    • Framework revision 7736f3bc90 (6 days ago), 2020-07-10 16:33:05 -0700
    • Engine revision d48085141c
    • Dart version 2.9.0 (build 2.9.0-21.2.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version
    30.0.0)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google
      Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.2

[✓] Connected device (5 available)
    • SM M305F (mobile)      • 32003c30dc19668f          • android-arm64  •
      Android 10 (API 29)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            •
      iOS 13.6
    • macOS (desktop)        • macos                     • darwin-x64     •
      Mac OS X 10.15.6 19G73
    • Web Server (web)       • web-server                • web-javascript •
      Flutter Tools
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 83.0.4103.116

• No issues found!

Verified this on latest versions and confirmed the behavior is still replicable. Talkback / voiceover doesn't announce / read disabled textfield.

 stable, master flutter doctor -v
[✓] Flutter (Channel stable, 3.3.7, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.3.7 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e99c9c7cd9 (2 days ago), 2022-11-01 16:59:00 -0700
    • Engine revision 857bd6b74c
    • Dart version 2.18.4
    • DevTools version 2.15.0

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.5.0-12.0.pre.175, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.5.0-12.0.pre.175 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 92f10ed712 (3 hours ago), 2022-11-08 17:57:52 -0800
    • Engine revision e7d7edab98
    • Dart version 2.19.0 (build 2.19.0-374.0.dev)
    • DevTools version 2.19.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Verified this on the latest versions and it is replicable but not fixed yet, for android it read disabled, and for iOS, it read dimmed not with the text field or edit box.


BUG: printed_every_n_steps and keep_every cannot be equal

When print_every_n_steps and keep_every are set to equal, the network is not updated properly
Following the notebook example:
model = Bohamiann(print_every_n_steps=50)
model.train(x[:, None], y, num_steps=20000, num_burn_in_steps=2000, keep_every=50, lr=1e-2, verbose=True)

results in :

The reason is because the seemingly innocent print_every_n_steps does an important job in the background : by calling network_predict, implicitly the current weights is reset to the latest sampled_weights via the line : self.network_weights = weights
Upon removing the line where it prints the network performance, we can log and notice the drifting loss weights prior shown as WP here:

good catch thanks, could you  send a PR to fix it?

I don't know how to fix it unfortunately !..


Mochawesome reporting

Summary
Use the cypress module to configure cypress and run tests programatically
Integrate mochawesome reports (pretty HTML reports, with integrated screenshots)
To run, just do npm run test, afterwards you should have a mochawesome-report directory. If you open up mochawesome.html in your browser, you can view the report.
Ticket Link
JIRA

This is great!
Could you please check once again the generated link for screenshot? It's broken on me when I ran it. There were / that's missing in the link.

Ah! Good catch! I was stripping out / from the final filename, I just needed to strip it from the parent names.

@saturninoabril @prapti Everything should be up to date now!

Should I be looking for the mochawesome-report folder inside e2e/results?
I am however, getting the following error while trying to run the test. Am I missing anything?

Error: Command failed: /bin/sh -c npm run test:ci
at makeError (/Users/prapti/go/src/github.com/mattermost/mattermost-webapp/e2e/node_modules/start-server-and-test/node_modules/execa/index.js:174:9)
at /Users/prapti/go/src/github.com/mattermost/mattermost-webapp/e2e/node_modules/start-server-and-test/node_modules/execa/index.js:278:16
at processTicksAndRejections (internal/process/task_queues.js:85:5) {
code: 1,
stdout: null,
stderr: null,
failed: true,
signal: null,
cmd: '/bin/sh -c npm run test:ci',
timedOut: false,
killed: false
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ test: start-test start:webhook 3000 test:ci
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ test script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.


Persistent connections via connection pool to a single redis server?

Hi,
How can I have this client maintain a persistent connection (via a connection pool) to a redis server? Is this possible? I looked at the RedisClientPool but that looks like it round robins requests to a set of servers.

Thanks for posting these questions & answers :- )  It wasn't clear to me when & why to use the connection pool, and when not to — so I searched all GitHub issues for "connection" and then I found this topic.
I think it'd make sense to rewrite & include the stuff in this topic in the main readme.
(Some stuff from this topic: https://github.com/etaty/rediscala/issues/22  about how to avoid bad performance when writing async requests, could also be in the main docs perhaps)


How to pushVideo?

Hi,
Thanks for LFLiveKit, it is great!  :-)
If I want to do a video overlay, how do I push the video to LFLiveSession.pushVideo?  Should I invoke this method 30 times per second on my own timer?  Or is there a delegate or callback that I can use to be notified when you need a frame and I can supply it?
Also, will any CVPixelBuffer that I supply here by composited with the existing video source (i.e. blended) or will it replace the frame entirely?
I want to blend in an overlay on top of parts of the captured video, but I'm not sure how to do that here.
I understand how to create a CVPixelBuffer, just not when or how I should invoke pushVideo with the buffer ...
Thanks!

LFLiveSession  pushVideo

Have you been able to upload the video? Could you provide me with your sample code? I have tried to upload it, but I'm experiencing audio issues. The video, however, is working fine.


JDK 1.6 Compatibility

Ran into an issue with the 2.3.0+ releases.  Not sure if this is the only issue, but at the very least OperationCachingEquivalence.java is using java.util.Objects causing ClassNotFoundExceptions.  Any chance you could switch to using Guava MoreObjects instead or just inline the functions you need?

I think it's been fixed in 2.3.1. Could you try that version

Thanks, I must have missed that there was a 2.3.1 release.  Changing to that resolved the issue and it's running with JDK 1.6 now.


Enhancement: Houdini: Unable to review/Render using Lopimportcamera

Is there an existing issue for this?

[X] I have searched the existing issues.

Please describe the feature you have in mind and explain what the current shortcomings are?

Currently OP is publishing without errors using SOP level import camera. Its thowing error when loading the LOP camera
How would you imagine the implementation of the feature?
The OP should successfully publish, using Lopimportcamera SOP node as well
Are there any labels you wish to add?

[X] I have added the relevant labels to the enhancement request.

Describe alternatives you've considered:
No response
Additional context:
No response
[cuID:OP-6365]

Related but doesn't solve the issue.
Publishing render supports marking as review.


Confirm allocated disk space is correct.

so I started my build with the following bytes<PHONE_NUMBER>328. I wake up this morning to it complaining about disk space and I now see 3.47TB used in my drive. Why did it go so far past 3TB?

brandon 4:26 PM (in 'dev')

Hmmm thats right,<PHONE_NUMBER>328 / 1024 / 1024 / 1024 = 3072 or 3 TiB

Well the next version should fix that. Because we allow users to say GB or TB instead of bytes. Note there is going to be an error of 1 height(128 MB) when building.

We changed the way that we pass build sizes as of 6f3f6d896e990c78c4057b5ecd89d7f1144f5a4e. When the next update goes out, please verify that your issue is gone. If not then reopen issue.


[Extensibility Request] Table 113 "Sales Invoice Line" - New Procedure GetInvoiceHeader

Hallo,
I would like to use the function GetHeader(), but unfortunately this is not possible because the function does not return the header and is local. Best just like in the "Sales Line" there it works great.
Old Code
local procedure GetHeader()
    begin
        if SalesInvoiceHeader."No." = "Document No." then
            exit;
        if not SalesInvoiceHeader.Get("Document No.") then
            SalesInvoiceHeader.Init();

        if SalesInvoiceHeader."Currency Code" = '' then
            Currency.InitRoundingPrecision
        else
            if not Currency.Get(SalesInvoiceHeader."Currency Code") then
                Currency.InitRoundingPrecision;
    end;

New Code:
  procedure GetInvoiceHeader(): Record "Sales Header"
  begin
      GetHeader();
      exit(SalesInvoiceHeader);
  end;

  local procedure GetHeader()
  begin
      if SalesInvoiceHeader."No." = "Document No." then
          exit;
      if not SalesInvoiceHeader.Get("Document No.") then
          SalesInvoiceHeader.Init();

      if SalesInvoiceHeader."Currency Code" = '' then
          Currency.InitRoundingPrecision
      else
          if not Currency.Get(SalesInvoiceHeader."Currency Code") then
              Currency.InitRoundingPrecision;
  end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 45938.


OpenPipeline Implementation

OpenPipeline was released in 1.295.0, is there a timeline for when support will be added to the provider? I see they have support for spans/metrics/topology planned, but is there work in place to get log/event pipelines supported in the Terraform provider?
https://docs.dynatrace.com/docs/platform/openpipeline

Hi @KevinThompsonYCRX, we are currently targeting the beginning of September 2024 for initial OpenPipeline support in the Terraform provider.


[TEMP] Update to crate-docs-theme==0.34.0.dev1

About
This RTD build gives you the chance to preview and review the documentation, using a pre-release of the modernized crate-docs-theme, based on sphinx-basic-ng/Furo. Please do not merge.
Preview
Rendering.
References

https://github.com/crate/crate-docs-theme/pull/390

This is no longer applicable. crate-docs-theme==0.34.0 is GA.


Remove v prefix from Helm charts and container images

Issue #, if available:
https://github.com/aws-controllers-k8s/community/issues/1715
Description of changes:
This pull requests updates all CD scripts that release Helm charts, or rely on the versions defined within Helm charts, to remove or disregard the v prefix at the beginning of the Git tag.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/hold
Deploying all v prefix changes at once

/unhlod

/unhold

/unhold


close possible avenue for racing and panic'ing/sending traces multiple times

It looks to be (theoretically, I haven't verified by testing it) possible for multiple goroutines to end up in this particular switch arm at the same time.
Use a sync.Once to make sure that even if that happens they can't both try to close the channel and/or send the trace.

There's another close() of the cancel channel in func (i *InMemCollector) sendAfterIdleTimeout, but that doesn't seem to be called anywhere?  Can I nuke that function?

Nice use of sync.Once there. I like it.


Nargo runs all tests with the same name

Aim
The nargo test command should be able to run exactly one test in a specific file.
Expected Behavior
Currently, if you have multiple test functions with the same name (e.g. in different files in your project), the nargo test <TEST_NAME> command will run all those tests.
Bug
There should be a way to run exactly one test in a specific file, otherwise running nargo test <TEST_NAME> via the LSP might run multiple tests.
To Reproduce

Create a project with 2 files that have tests with the same name
Use > Run Test as provided by #1835
Notice that the output says 2 tests ran:

Installation Method
None
Nargo Version
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

I think we should copy rust testing workflow here and do module scopes in the regex to pinpoint a file.
Leaving the bug label because test names need to be named properly in that case

@f01dab1e do you want to take this on?

I'll take this task


A road to better configuration

For allowing apps to have their own configuration and not overwrite freepy.

@n9986 any consequences to this?

@lylepratt None.


cannot build from release archive

Tried .tar.gz distribution for both 1.6.1 and 1.6.5.
[dan@host Ammonite-1.6.1]$ sbt clean compile package
[info] Loading project definition from /home/dan/tmp/Ammonite-1.6.1/project
[info] Updating {file:/home/dan/tmp/Ammonite-1.6.1/project/}ammonite-1-6-1-build...
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[info] Done updating.
[info] Set current project to ammonite-1-6-1 (in build file:/home/dan/tmp/Ammonite-1.6.1/)
[info] Executing in batch mode. For better performance use sbt's shell
[success] Total time: 0 s, completed Mar 29, 2019 11:53:47 PM
Generating Scalatex Sources...
[info] Updating {file:/home/dan/tmp/Ammonite-1.6.1/}readme...
[info] Updating {file:/home/dan/tmp/Ammonite-1.6.1/}ammonite-1-6-1...
[info] Resolving jline#jline;2.14.4 ...
[info] Done updating.
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[info] Done updating.
java.util.NoSuchElementException: key not found: CONSTANTS_FILE
at scala.collection.MapLike$class.default(MapLike.scala:228)
at scala.collection.AbstractMap.default(Map.scala:58)
at scala.collection.MapLike$class.apply(MapLike.scala:141)
at scala.collection.AbstractMap.apply(Map.scala:58)
at $fc996592160a746a5ce3$$anonfun$readme$6.apply(build.sbt:17)
at $fc996592160a746a5ce3$$anonfun$readme$6.apply(build.sbt:17)
at sbt.std.Transform$$anon$3$$anonfun$apply$2.apply(System.scala:44)
at sbt.std.Transform$$anon$3$$anonfun$apply$2.apply(System.scala:44)
at sbt.std.Transform$$anon$4.work(System.scala:63)
at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:228)
at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:228)
at sbt.ErrorHandling$.wideConvert(ErrorHandling.scala:17)
at sbt.Execute.work(Execute.scala:237)
at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:228)
at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:228)
at sbt.ConcurrentRestrictions$$anon$4$$anonfun$1.apply(ConcurrentRestrictions.scala:159)
at sbt.CompletionService$$anon$2.call(CompletionService.scala:28)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
[error] (readme/compile:unmanagedSources) java.util.NoSuchElementException: key not found: CONSTANTS_FILE
[error] Total time: 0 s, completed Mar 29, 2019 11:53:47 PM
[dan@host Ammonite-1.6.1]$

Ammonite isn't built using SBT

Apparently. Thanks.
Here is why someone might think that it was:

assuming sbt is a good default option for scala projects
there is a file called build.sbt in the unpacked release directory
sbt is mentioned in the readme to run the integration tests

Maybe it would be useful to add a short section on how to build the project to the readme.

Thanks for letting future readers know how to kick off a local build...

To future readers, this project uses mill another project by the primary author.
This blog post got me started: https://www.lihaoyi.com/post/MillBetterScalaBuilds.html
I wanted to publish artifacts locally for scala 2.13.6 to see if I can find a proposal to fix #1206. This did the trick for me and might help another who has never heard of mill before.
./mill '__[2.13.6].publishLocal'


MAINT: type sph vor function signatures

add type hints to _spherical_voronoi.py module
function signatures

expansion of the ckdtree.pyi stub module with a few
more call signatures was needed to allow type checking of
the Cython calls without errors

doc build fail is related, somewhat surprisingly

@pmla @person142 thoughts? I haven't looked into this deeply yet, but hopefully an easy fix.

I was able to reproduce the doc build error and fix locally using approach described by an astropy dev.
In short, sphinx can't find Union[numpy.ndarray, List, None]', though I don't think that is surprising?!
Hopefully CI agrees..

Just a question; might have been answered elsewhere: When does scipy use type annotations directly in the file and when a .pyi file? What is the reason for mixing both styles?

@MartinThoma in Python code it can be inline, in compiled code it can't.

I'll let someone else drive this forward if it hasn't already been taken care of. A type hint function that has been open for three years might as well be closed, and it is mine anyways so shouldn't annoy anyone.


parquet.hadoop.ParquetOutputCommitter.commitJob() throws parquet.io.ParquetEncodingException

I'm running an example combining Avro, Spark and Parquet (https://github.com/massie/spark-parquet-example), and in the process of updating the library versions, am getting the warning below.
The version of Parquet-Hadoop in the original example is 1.0.0. I am using 1.6.0rc3
The ParquetFileWriter.mergeFooters(Path, List) method is performing a check to ensure the footers are all for files in the output directory. The output directory is supplied by ParquetFileWriter.writeMetadataFile; in 1.0.0, the output path was converted to a fully qualified output path before the call to mergeFooters, but in 1.6.0rc[2,3] that conversion happens after the call to mergeFooters. Because of this, the check within merge footers is failing (the URI for the footers starts with file:, but not the URI for the root path does not)
Here is the warning message and stacktrace.
Oct 30, 2014 9:11:31 PM WARNING: parquet.hadoop.ParquetOutputCommitter: could not write summary file for /tmp/1414728690018-0/output
parquet.io.ParquetEncodingException: file:/tmp/1414728690018-0/output/part-r-00000.parquet invalid: all the files must be contained in the root /tmp/1414728690018-0/output
at parquet.hadoop.ParquetFileWriter.mergeFooters(ParquetFileWriter.java:422)
at parquet.hadoop.ParquetFileWriter.writeMetadataFile(ParquetFileWriter.java:398)
at parquet.hadoop.ParquetOutputCommitter.commitJob(ParquetOutputCommitter.java:50)
at org.apache.spark.rdd.PairRDDFunctions.saveAsNewAPIHadoopDataset(PairRDDFunctions.scala:936)
at org.apache.spark.rdd.PairRDDFunctions.saveAsNewAPIHadoopFile(PairRDDFunctions.scala:832)
at com.zenfractal.SparkParquetExample$.main(SparkParquetExample.scala:72)
at com.zenfractal.SparkParquetExample.main(SparkParquetExample.scala)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:134)
Reporter: Chris Albright
Original Issue Attachments:

PARQUET-124-test

Note: This issue was originally created as PARQUET-124. Please see the migration documentation for further details.

Ryan Blue / @rdblue:
Chris, I'm attaching a test that produces the same error that you've posted. I ended up making the mergeFooters method accessible within the package to add it. Could you check your PR and push a new version with this test included so we can see it passes tests in Travis?
Also, is there a better way to check that the path than to use String#startWith? If there is a Path or URI method that does it, that would be preferred. That way we don't run into problems trying to merge hdfs:/ and file:/ URIs. That shouldn't ever happen though.

Ryan Blue / @rdblue:
Looks like the problem is that we're matching a URI against a path. The check should be URI.getPath().startWith(dirPath). I'll submit a PR today unless you want to take care of it, Chris. The fix should be on line 422 of ParquetFileWriter.

Ryan Blue / @rdblue:
Great, now it'll look bad if I don't include a test. :)

Chris Albright:
I was trying to create a breaking test for it last night, but to no avail. Probably need to get a bit more experience with the codebase. I went ahead and submitted the PR, but without a test I don't feel great about it.

swetha k:
<EMAIL_ADDRESS>I still see the issues. Please see the Warning below: Which version to be used to avoid this? I use 1.6.0 of parquet-avro.
    <dependency>
        <groupId>com.twitter</groupId>
        <artifactId>parquet-avro</artifactId>
        <version>1.6.0</version>
    </dependency>

How is this issue resolved? I still see the following issue when I try to save my parquet file.
Nov 8, 2015 11:35:39 PM WARNING: parquet.hadoop.ParquetOutputCommitter: could not write summary file for active_sessions_current
parquet.io.ParquetEncodingException: maprfs:/user/testId/active_sessions_current/part-r-00142.parquet invalid: all the files must be contained in the root active_sessions_current
at parquet.hadoop.ParquetFileWriter.mergeFooters(ParquetFileWriter.java:422)
at parquet.hadoop.ParquetFileWriter.writeMetadataFile(ParquetFileWriter.java:398)
at parquet.hadoop.ParquetOutputCommitter.commitJob(ParquetOutputCommitter.java:51)
at org.apache.spark.rdd.PairRDDFunctions$$anonfun$saveAsNewAPIHadoopDataset$1.apply$mcV$sp(PairRDDFunctions.scala:1056)
at org.apache.spark.rdd.PairRDDFunctions$$anonfun$saveAsNewAPIHadoopDataset$1.apply(PairRDDFunctions.scala:998)

Chris Albright:
@rdblue, I was not able to find a way compare to the path prefixes without using String#startsWith. It does seem very unlikely that there would ever exist a situation where the footers come from a different filesystem than the root, since the footers are read out of the root to begin with :)
I thought of comparing the scheme as well, but I was not able to make that work since the scheme was missing from the root path URI. If its important, I've got some other ideas, but I'm not sure its within scope of this issue. I'll go ahead and create a new one and work off of that.
Pull request is updated, and passing in Travis.

Alex Nastetsky:
Still seeing this issue, at least when running locally. I think it needs to do something like FileSystem.get(new URI("file:///"), new Configuration()).resolvePath(X) on both "root" and "new Path(footerPath)", except using the correct file system.

Ryan Blue / @rdblue:
[~swethakasireddy], it looks like this wasn't completely addressed by the fix above. [~terrasect] had a problem with it as well. Would one of you be willing to open a new issue for the current problem? Then we can work on getting it fixed. Thanks!

swetha k:
@rdblue
Following is the new issues.
https://issues.apache.org/jira/browse/SPARK-11620

Ryan Blue / @rdblue:
Issue resolved by pull request 79
https://github.com/apache/incubator-parquet-mr/pull/79

swetha k:
@rdblue
I can create a JIRA  issue for this. Just to confirm, this is just a Warning message right and would not affect the functionality in any case?


Demote confusing errors that are not really errors

Fixes #468

If you're seeing this message are you guaranteed to be properly connected to and ETL source?
If the ETL source is configured such that the ledger will never be available will it get out of this loop?

@legleux The original implementation is already flawed so that the loop does not exits. While this is a potential problem and what you noted is true - this "issue" is not about fixing those behaviours but just demoting the severity of the messages so that people who see them don't panic. Those "errors" are something clio should be able to recover from if ETL sources are configured correctly. On the other hand "error" is not "fatal" either. But people still find it scary so I rather have it as a warn/info for this case.
Before trying to go into the loop there is a check (and wait) for the required seq to be validated by the network. This does not mean however that the required seq is available for fetching - it only checks that the latest seq validated is at least the one we ask for or newer. So in a way, yes, we should not be going into this loop until we know that ETL source connection is ok.
However, the indefinite loop itself will be fixed when we get to redesign and update the ETL subsystem.

If I understand correctly, clio will continue trying and it might succeed later. Indeed , it is not an error.

This comment was more about the error message itself. I'm familiar with this message and get that it's not really an "error".
I meant to get some info to maybe help to make the message more helpful as when I saw it, I couldn't tell if the problem was with Clio or rippled config or if it would resolve itself once rippled was synced or...

This comment was more about the error message itself. I'm familiar with this message and get that it's not really an "error". I meant to get some info to maybe help to make the message more helpful as when I saw it, I couldn't tell if the problem was with Clio or rippled config or if it would resolve itself once rippled was synced or...

If you have some better wording - please let me know and i'll apply it. I don't mind what wording we use, i just don't want it to say "error".
Thanks


fireplace: nix-darwin bugfix

The Darwin stdenv uses clang instead of gcc. When referring to the compiler $CC or cc will work in both cases. Some builds hardcode gcc/g++ in their build scripts, that can usually be fixed with using something like makeFlags = [ "CC=cc" ]; or by patching the build scripts.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This is working great for me; mind squashing your second commit into your first one?

This is working great for me; mind squashing your second commit into your first one?

Thank you for trying it out! Just squashed the second into the first one 🐛

Result of nixpkgs-review pr 348547 run on x86_64-darwin 1

1 package built:

fireplace

Result of nixpkgs-review pr 348547 run on x86_64-linux 1

1 package built:

fireplace


[test:unit] bonding UI flows

balances
calculator results
cost of bond based on the asset & behavior for max available bonds
approval validation/display
assigned: biwano


The number of boxes displayed is different from the value returned by bbox

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
draw_pred=True shows 50 boxes, but the bbox that returns the value has 50 more results (say 55), and I use OpenCV to display the images in the check box individually to show that there are indeed 55
Reproduction

What command or script did you run?

inferencer = DetInferencer(model=r'work_dirs\full\yolox_l_8xb8-300e_coco\yolox_l_8xb8-300e_coco.py', weights=r'work_dirs\full\yolox_l_8xb8-300e_coco\epoch_300.pth', device='cuda', palette=(0,255,0))
A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
modify
What dataset did you use?

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

@CyanMystery There will be a threshold filter before visualization.


Request: Option to add a main application file when initiating a crystal app project

So I was wondering what exactly was the difference between crystal init lib Name and crystal init app Name and at least with the version I currently have installed they don't share any differences. Same project structure and files.
So i was wondering maybe the application version could create an app.cr file automatically in the root folder which adds some comments from the compiler (generated by crystal) and require's the applications base project file?
filename: app.cr
# Generated By Crystal v.0.21.1
require "./ProjectName.cr"

The contents of shards.yml should be different.

Two things differ:

The shards.yml file contains a target section for an app (and not for a lib). You can try it, run shards build and it will build that file.
The .gitignore adds shards.lock for a lib

@exts Do you think something else is missing?

@asterite I never knew the shards command existed. Thanks. Yeah I guess I can close this as I didn't notice the target before or the shards command. I somehow overlooked that.


SystemReady 1.0 Rel Changes

Support to build IR ACS live image : build scripts and build instructions in documentation
Upgrading ACS image to v5.13 kernel
Refinement of UEFI-SCT Sequence files for IR and ES and verification
Migrate to the version V21.08.00 of FWTS
BBR: Migrate to edk2-test-stable202108 tag of SCT
General improvements in scripts
Bug Fixes

Co-authored-by: Edhaya Chandran<EMAIL_ADDRESS>Co-authored-by: Chetan Rathore<EMAIL_ADDRESS>Co-authored-by: Jiss Jose<EMAIL_ADDRESS>Signed-off-by: jisjos01<EMAIL_ADDRESS>

approved


Compatibility with internet explorer

https://stackoverflow.com/questions/44648758/uncaught-referenceerror-vendor-is-not-defined-when-using-dllplugin-and-dllrefer
and
https://stackoverflow.com/questions/42601721/webpack-vendor-dll-reference-error
SCRIPT1002: Syntax error
vendor_0a29a6cf2b71efe024aa.js (118983,18)
SCRIPT5009: 'vendor_0a29a6cf2b71efe024aa' is undefined

Your plugin seems to set library itself so I'm not sure why it's still not working. Any ideas?

@switz  I have the same issue. Did you figure out what is the fix for it?


Keep mtenn pinned until we can upgrade asapdiscovery to pydantic 2.0

@kaminow seeing as we upgraded mtenn to pydantic 2.0+ we have to keep it pinned here until we upgrade to pydantic 2.0 asapdiscovery side.

yep good catch, thanks! I'll take care of this


[Bug]: Please add a check for "develop" in CiderCollective.Cider.Nightly

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hi, first of all, I love your work, thanks for automating winget package making for our app.
I made this issue because we have added releases for our other branches and it has also been added to the winget repo even though it was never meant to be there. Please add a check to see if the name of the release ends with (develop), if not, please do not add to the winget repo.
Thanks so much,
Yaz

Hi, first of all, I love your work, thanks for automating winget package making for our app.

Thank you🙏🎉

I made this issue because we have added releases for our other branches and it has also been added to the winget repo even though it was never meant to be there. Please add a check to see if the name of the release ends with (develop), if not, please do not add to the winget repo.

I think we can add releases with "main" to the stable package identifier "CiderCollective.Cider.Nightly". If yes, you can make an initial PR for it at winget-pkgs repository so that I can add that to the automation too after that PR is merged.

Hi, first of all, I love your work, thanks for automating winget package making for our app.

Thank you🙏🎉

I made this issue because we have added releases for our other branches and it has also been added to the winget repo even though it was never meant to be there. Please add a check to see if the name of the release ends with (develop), if not, please do not add to the winget repo.

I think we can add releases with "main" to the stable package identifier "CiderCollective.Cider.Nightly". If yes, you can make an initial PR for it at winget-pkgs repository so that I can add that to the automation too after that PR is merged.

Hi, sorry it took a while for me to respond, I hope you may reopen this issue, My PRs have been merged to winget-pkgs, also updated the version number in the packages

Hi, the check does not work i think, both cider and cider.nightly pull the same package...