Take operator not completing if count is a string

Bug Report
Current Behavior
When using the take operator and providing a string instead of a number for the first and only argument ("count"), everything behaves the same way with the exception of the observable not completing after emitting n times.
Reproduction
https://stackblitz.com/edit/rxjs-kenahb?devtoolsheight=60
const mouseClick = fromEvent(window, 'click');
mouseClick.pipe(
  take('2'),
  tap(() => console.log('CLICK')),
  finalize(() => console.log('completed'))
).subscribe();

Expected behavior
The above code should write 'CLICK', 'CLICK' and 'completed' to the console after clicking twice. Because '2' is a string, the output is 'CLICK', 'CLICK'.
Environment

Runtime: Chrome
RxJS version: 6.6.3

I'm not sure if I understood issue correctly, take explicitly requires to supply number https://github.com/ReactiveX/rxjs/blob/master/src/internal/operators/take.ts#L53 . Are you saying it should try to parse given argument as number if string is provided and it's number?

Exactly. It kind of does that already, but completed is never called. Took me forever to find out that this was the issue. It should either work as expected or not work at all.

I'm not sure if I understood issue correctly, take explicitly requires to supply number https://github.com/ReactiveX/rxjs/blob/master/src/internal/operators/take.ts#L53 . Are you saying it should try to parse given argument as number if string is provided and it's number?

Exactly. It kind of does that already, but completed is never called. Took me forever to find out that this was the issue. It should either work as expected or not work at all.

The real code that caused this issue without raising any alarm bells:
connect(inputs: Observable<any>[]) {
  const taken = inputs[1].pipe(
    first(),
    switchMap((count: number) => inputs[0].pipe(take(count))),
  );
  return [taken];
}

count ended up not being a number in runtime.

It's because === is used in the v6 implementation:
https://github.com/ReactiveX/rxjs/blob/6.x/src/internal/operators/take.ts#L93
This won't happen in v7 - which is the implementation linked in a comment above.

Perfect, thanks!

@cartant Can I close this issue?

@christophbuehler Leave it open for the moment, I think. The team can decide what to do with it tomorrow.

Core Team Meeting: We'll deal with these on a case-by-case basis. Adding a test at the time. We will not be adding runtime type checking that throws.


Remove resize listener, extend iron-resizable-behavior

Connected to #3
Remove resize listener, extend iron-resizable-behavior

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Very nice syntax:

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

vaadin-nav.html, line 112 at r2 (raw file):

       * @mixes Vaadin.NavListMixin
       * @mixes Vaadin.ThemableMixin
       * @mixes Polymer.IronResizableBehavior

Should we drop @mixes annotations for ThemableMixin and IronResizableBehavior? Otherwise their apis will end up in our docs: https://www.dropbox.com/s/l9wyrdtcmbi87az/Screenshot 2017-10-05 09.37.31.png?dl=0

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

vaadin-nav.html, line 112 at r2 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Should we drop @mixes annotations for ThemableMixin and IronResizableBehavior? Otherwise their apis will end up in our docs: https://www.dropbox.com/s/l9wyrdtcmbi87az/Screenshot 2017-10-05 09.37.31.png?dl=0

We always include the ThemableMixin in all our elements, it should not introduce any public API in the doc, but I think the reference that the element implements the mixin is useful.
Related with the resizable, I think it's nice that the user knows that this element needs resizing notifications and works well in a resizable tree. But we are not documenting it in other documents, so I think we should align with them.

Comments from Reviewable

vaadin-nav.html, line 112 at r2 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
We always include the ThemableMixin in all our elements, it should not introduce any public API in the doc, but I think the reference that the element implements the mixin is useful.
Related with the resizable, I think it's nice that the user knows that this element needs resizing notifications and works well in a resizable tree. But we are not documenting it in other documents, so I think we should align with them.

Yep, there's already a fix for the first issue https://github.com/vaadin/vaadin-themable-mixin/pull/19. So drop @mixes Polymer.IronResizableBehavior and we're good

Comments from Reviewable

Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion.

vaadin-nav.html, line 112 at r2 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Yep, there's already a fix for the first issue https://github.com/vaadin/vaadin-themable-mixin/pull/19. So drop @mixes Polymer.IronResizableBehavior and we're good

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


Honor $EXTRA_TSC_ARGS when building

Allows us to pass extra options to tsc when running from within e.g.
lerna. Good to try out new compiler options.
Also fixed some super strict linter warnings.

Codecov Report

Merging #1284 into master will decrease coverage by 0.13%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1284      +/-   ##
==========================================
- Coverage   52.95%   52.81%   -0.14%
==========================================
  Files         260      249      -11
  Lines       23386    23108     -278
  Branches     5176     5125      -51
==========================================
- Hits        12383    12204     -179
+ Misses      11003    10904      -99

Impacted Files
Coverage Δ

...ils/lib/projection/TransverseMercatorProjection.ts
93.68% <100%> (ø)
:arrow_up:

@here/harp-test-utils/lib/TestUtils.ts
81.17% <100%> (ø)
:arrow_up:

...re/harp-text-canvas/lib/rendering/TextMaterials.ts
42.85% <0%> (-34.07%)
:arrow_down:

@here/harp-mapview/lib/ThemeHelpers.ts
6.55% <0%> (-0.68%)
:arrow_down:

...rp-mapview/lib/text/TextElementsRendererOptions.ts
100% <0%> (ø)
:arrow_up:

@here/harp-datasource-protocol/lib/Outliner.ts
100% <0%> (ø)
:arrow_up:

...ere/harp-materials/lib/LuminosityHighPassShader.ts
100% <0%> (ø)
:arrow_up:

@here/harp-test-utils/lib/WebGLStub.ts
100% <0%> (ø)
:arrow_up:

@here/harp-omv-datasource/lib/clipPolygon.ts
100% <0%> (ø)
:arrow_up:

...p-geoutils/lib/tiling/QuadTreeSubdivisionScheme.ts
100% <0%> (ø)
:arrow_up:

... and 29 more

Continue to review full report at Codecov.

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


Update record based on name field

I am trying to understand how to update the 'count' field of a particular record based on the 'name' field.
Here's the setup for my collection. Each record represents a player, which has a name (string) and a count (int).

I can successfully create new records, but I can't figure out how to edit an existing record to increase the count.
In the code below, I check whether item, a string representing the name of the current player, is in a list of the names in the database called namesinDB. If if is, I try to increment the count by 1. If it is not, I create a new record with the name=item and count=1. That latter part successfully creates new records, but once a record exists, I can't seem to update the count. I believe the update function searches for a record by it's ID. Is there a way to search by a different value (name, in this case)?

console.log(item);
//If the record already exists, increment the count
if (namesinDB.includes(item)){
console.log("Updating the count for " + item);
var record1 = await pb.collection('men_player_selections').update('name' = item, {
count: +1,
});
}
//Otherwise, create the record
else{
//console.log("Creating a record for " + item);
var record1 = await pb.collection('men_player_selections').create({
name: item,
count: 1,
});

In order to update a record you need to provide its id (aka. update(id, data)).
If you want to update a record using a different identifier, you can first fetch it with getFirstListItem(filter).

Thanks for the help!
The problem I'm having with this method is using getFirstListItem to select a record in which the name isn't hard coded.
If I hard code a name in like "Test Player", it works fine:
var record = await pb.collection('men_player_selections').getFirstListItem('name="Test Player"');
But what I really want is to by able to search for the name stored in the variable called item. When I try it this way, I get an error. Using double quotes around "item" also wouldn't work because it would search for a record whose name is literally "item". What's the correct syntax in this case?
var record = await pb.collection('men_player_selections').getFirstListItem('name=item');

Sorry, here's the error:

You can use Template literals (backticks) to embed your dynamic value, something like:
let name = 'item';
let record = await pb.collection('men_player_selections').getFirstListItem(`name="${item}"`);

(note that the name value must be wrapped in a single or double quotes, otherwise it will be treated as identifier)

That works perfectly.  Thank you so much for the help!!! :)


Performance issues on Android Kitkat

For some reason .parse takes 10x more time to complete on devices running kitkat
What could be the problem? This has been confirmed on all official kitkat devices, it doesn't matter how the html looks like.

I also find jsoup about 10x slower on Android 4.4 KitKat and v5 Lollipop than on older versions (tested 4.3 and 4.2.2). Under KitKat I tested with both ART and Dalvik VMs, practically no difference. Any idea what could be done about it?

What jsoup version -- can you try with Mainline (not 1.8.1)? I optimized a
lot of things that can cause GC. I didn't see a 10x speedup but it did
improve.
Are you looking at parsing or serialization? Can you give me your example
parsing HTML files?
On Wed, Apr 1, 2015 at 5:38 PM, Greg Kochaniak<EMAIL_ADDRESS>wrote:

I also find jsoup about 10x slower on Android 4.4 KitKat and v5 Lollipop
than on older versions (tested 4.3 and 4.2.2). Under KitKat I tested with
both ART and Dalvik VMs, practically no difference. Any idea what could be
done about it?
—
Reply to this email directly or view it on GitHub
https://github.com/jhy/jsoup/issues/383#issuecomment-88675283.

Hey, thanks for the answer! I was using before ver. 1.7.4, and after reading your release notes about Android optimizations in 1.8.2, I instantly built it form the latest GitHub source (last build yesterday) and tested this. The issue persists, version 1.8.x is very fast under Android 4.3 and lower, e.g. loads a long chapter from ePub book into my app in about 2 seconds even on slow old devices, while it takes about 20 seconds to load on Galaxy Note 3 - lots of memory and decent processor, with KitKat before, now upgraded to Lollipop. Same on other devices I own and in the emulators. It's probably the inefficient use of memory, as the VM doing garbage collection often? I'm going to try it now with a setting requesting a bigger heap, will report shortly what happens then.
What my app (https://play.google.com/store/apps/details?id=com.hyperionics.avar) does with jsoup: runs my modified version of Readability algorithm to extract text for reading aloud or on screen , reject junk like menus, ads, unrelated links etc.Then, it processes further the cleaned-up jsoup doc e.g. downloading and inserting style nodes and doing further cleanup to have styled HTML text for reading aloud and highlighting the sentences it reads, e.g. it traverses all nodes extracting text, but also keeping text attributes like headers, paragraphs, divs and styles. This last step, which is also slow on Android 4.4 and higher, is based on your text extractor and node visitor. It no longer manipulates the Document object, just visits the nodes and accumulates my "simpliefied HTML" in a StringBuilder object.
Greg

Update: just tried android:largeHeap="true" in AndroidManifest.xml of the app, no difference on Android 5 test (Galaxy Note 3 phone).

Could you send me (directly<EMAIL_ADDRESS>a sample of that epub
doc? I'm seeing like 10ms parse times for jsoup.org, 155ms for CNN sites on
Lollipop Nexus 5.
I need to find an old phone to compare pre-kitkat.
On Thu, Apr 2, 2015, 05:11 Greg Kochaniak<EMAIL_ADDRESS>wrote:

Update: just tried android:largeHeap="true" in AndroidManifest.xml of the
app, no difference on Android 5 test (Galaxy Note 3 phone).
—
Reply to this email directly or view it on GitHub
https://github.com/jhy/jsoup/issues/383#issuecomment-88880188.

I was not sure what exactly you need, but assume an ePub chapter. Will try to extract and email it... The jsoup parse is probably pretty fast, the other operations I do take longer. I looked at the jsoup code, and for example Element hasClass(String className) method calls classNames() method that creates a lot of strings and a String table, and then a Set - a lot of objects created on the heap, to give us a simple yes/no answer, and they all have to be garbage collected soon after. Imagine now iterating through all elements in a doc...
I will try to replace hasClass() with just a RegEx "\bclass_name\b" expression lookup and see if that makes a difference in my app. Then will continue searching for other places like that to optimize memory management. If you have suggestions where to look, let me know. If I do get something that works really well, will issue a merge request to your repo.
Greg

I dug into why CharsetEncoder#canEncode() is so slow on Android. I can't find why it's changed so much between ICS to KK where it got really slow, but I do see that it's implemented natively and involves creating arrays and copying through the native interfaces. However in the JDK, for at least UTF and Ascii, there are simple one-liner implementations for canEncode. I expect somewhere between ICS and KK, Android moved from using OpenJDK to a native (ICU) implementation and that's where this speed regression comes from.
I think the fix will be to roll some specific encode checks, referring to the simple implementations in the OpenJDK character sets.

Thanks to @gregko and everyone else for getting to the bottom of this. The Android implementation of canEncode doesn't use the OpenJDK simple methods, and ends up being very slow. I've built simple tests for ascii and UTF, similarly to Greg's impl. We can extend it simply if there are other charsets in use that get impacted.

Here's data on the improved Document.html() time:
OLD impl v New (ms)
Wiki: 1895, 16 (118x)
CNN: 6378,  (115x)
Alterslash: 3013, 28 (107x)
Jsoup: 167, 2 (83x)

Yes, that was pretty much my concussion as well. Wrapping chars in character arrays (lots of junk on the heap to dispose) sending them over VM/native boundary, and do this for the file I sent you over 200,000 times... hopefully you can find a more general solution, what I purposed is still the best optimization for the social case of dealing with Unicode. Good luck!
Greg
-------- Original message --------
From: Jonathan Hedley<EMAIL_ADDRESS>Date: 04/06/2015  5:22 PM  (GMT-05:00)
To: jhy/jsoup<EMAIL_ADDRESS>Cc: Greg Kochaniak<EMAIL_ADDRESS>Subject: Re: [jsoup] Performance issues on Android Kitkat (#383)
I dug into why CharsetEncoder#canEncode() is so slow on Android. I can't find why it's changed so much between ICS to KK where it got really slow, but I do see that it's implemented natively and involves creating arrays and copying through the native interfaces. However in the JDK, for at least UTF and Ascii, there are simple one-liner implementations for canEncode. I expect somewhere between ICS and KK, Android moved from using OpenJDK to a native (ICU) implementation and that's where this speed regression comes from.
I think the fix will be to roll some specific encode checks, referring to the simple implementations in the OpenJDK character sets.
—
Reply to this email directly or view it on GitHub.

I didn't really find a more generic approach. I looked at reimplementing
the encoding loop, or catching an exception on unsupported characters and
replacing with an escape. The problem with these is that we aren't actually
doing the encoding at this phase.
This implementation should be easy to extend if we see issues with other
common encodings.
On Mon, Apr 6, 2015 at 7:53 PM, Greg Kochaniak<EMAIL_ADDRESS>wrote:

Yes, that was pretty much my concussion as well. Wrapping chars in
character arrays (lots of junk on the heap to dispose) sending them over
VM/native boundary, and do this for the file I sent you over 200,000
times... hopefully you can find a more general solution, what I purposed is
still the best optimization for the social case of dealing with Unicode.
Good luck!
Greg
-------- Original message --------
From: Jonathan Hedley<EMAIL_ADDRESS>Date: 04/06/2015 5:22 PM (GMT-05:00)
To: jhy/jsoup<EMAIL_ADDRESS>Cc: Greg Kochaniak<EMAIL_ADDRESS>Subject: Re: [jsoup] Performance issues on Android Kitkat (#383)
I dug into why CharsetEncoder#canEncode() is so slow on Android. I can't
find why it's changed so much between ICS to KK where it got really slow,
but I do see that it's implemented natively and involves creating arrays
and copying through the native interfaces. However in the JDK, for at least
UTF and Ascii, there are simple one-liner implementations for canEncode. I
expect somewhere between ICS and KK, Android moved from using OpenJDK to a
native (ICU) implementation and that's where this speed regression comes
from.
I think the fix will be to roll some specific encode checks, referring to
the simple implementations in the OpenJDK character sets.
—
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub
https://github.com/jhy/jsoup/issues/383#issuecomment-90326225.

Hey, thanks! I've been waiting for this for year!
I even reimplemented escaping in a very hacky way just for one site to work properly.
Glad to see you didn't forget your open projects! It brings me some money from time to time - where can I donate few bucks to you?


fix: change library layout

fixes #221

Pull Request Test Coverage Report for Build 1066

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.32%

Totals

Change from base Build 1048:
0.0%

Covered Lines:
585

Relevant Lines:
761

💛  - Coveralls


Point to correct stream9 image tag

stream9-development was last updated Oct 9 2022, while thestream9 tag is being actively updated
https://quay.io/repository/centos/centos?tab=tags

@ttuffin If I remember correctly you used centos9-development because the image was ligther. Is that correct? If it is not maintained we should consider to move to regular stream9 or probably better to stream9-minimal.

@Alex-Izquierdo thats correct. Using stream9 instead of stream9-development adds an extra ~600mb to the image size. I didn't realise that tag is no longer being updated though. Lets see how much reduction we get in #639.


Google auth service hardcoded data

It must be refactored and the Google secret and id client must be deleted from the Google console!!
The service must be able to get those data from environment variables!
This is a big mistake.. :(

client_secret and client_id removed from the Google developer console...

Fixed in #105


[iwyu] rest of the owl: src/core/ext

@nicolasnoble

Excluded binder transport for a bit, but otherwise this looks ready to go.


checkpoint-prof.py fails when using QueueInputTrainer

Full example at https://github.com/zsc/tensorpack-exp/blob/master/mnist/base.queue/main.py
For comparison, https://github.com/zsc/tensorpack-exp/blob/master/mnist/base/main.py works like a charm.
The command line used:
python3 ~/zsc-tensorpack/scripts/checkpoint-prof.py --model train_log/main/checkpoint --meta train_log/main/graph-0625-115015.meta --input input:0=1,28,28 label:0=1 --output cross_entropy_loss --print-flops

In training, the input tensor is called input_dequeued:0, the output tensor is loss:0
It also builds the graph for validation, where input tensor is named e.g. input:0, and output tensor is towerp0/loss:0.
You can print(tensor.name) to see the name. This is currently very unintuitive, I'll fix it in the future.

For that specific instance, the name seems to be input_deque, and a working command line is as follows:
checkpoint-prof.py --model train_log/main/checkpoint --meta train_log/main/graph-0625-115015.meta --input input_deque:0=1,28,28 input_deque:1=1 --output cross_entropy_loss:0 --print-flops


High contrast: Icons for block categories now have colors in Minecraft

All of our other targets still have the colors on the icons for the block categories, so we want Minecraft to match that.
Most of the categories are super straightforward because you can just change the icon color via code. However, the Agent category is a bit tricky because the icon that we're using doesn't have an associated icon in Semantic; we're using a static png. Because of this, changing the color of the Agent icon via code isn't really possible. That's where the new Agent_icon_orange file comes in. I just alternate between the two files to make the color changes work. I wasn't sure if referencing files across the repos like this would cause problems in production, so we might have to find another solution, but it works locally. One thing to note, though, is that when you first switch to high contrast mode, it takes a second for the agent icon to load in, so there is definitely room for improvements.

Will get an upload target on Monday.
Closes: https://github.com/microsoft/pxt-minecraft/issues/2062

UPDATE
I've decided to go the CSS route so it's a bit easier to interpret going forward. There are still a couple things that I want to fiddle with, but I wanted to push up my current changes.
The following has changed on this first pass:

Hover and not hovering for inverted toolbox is no longer handled in JS, but is handled by the :hover property in CSS
Styling for inverted and colored toolboxes is moved to toolbox.less
CSS variables made for block meta color, the fade color that applies to toolbox categories for inverted toolbox hover, and the border properties that we use for our colored toolbox. Looking at it now, I will likely just move  this fully into CSS and pass in the --block-meta-color variable for it's color property.
I removed the logic for flipping the border side when the editor changes between right to left and left to right orientation for colored toolboxes. From what I can tell, we sort this out programmatically. When adding the coloredToolbox styling to CSS, I noticed that the styling in CSS that specified border-left automatically turned into border-right.

What I want to change still:

The icon image style. Right now, we are using a style element to apply more specific styling to blocks categories that use images as their icons. I think a majority of this styling can just live in CSS, and in the case of the backgroundImage url, I'll use a CSS variable as was done with the meta color.

Things that need clarification:

The animation stuff that was in the styling. Thanks to the git blame, I was able to find this PR: https://github.com/microsoft/pxt/pull/6774, showing what the animation stuff should be giving us. This is broken as far as I can tell. I'm wondering if this is something that I should open a bug for or if I should just get rid of the animation logic.

Hover and not hovering for inverted toolbox is no longer handled in JS, but is handled by the :hover property in CSS

I might be missing something obvious, but I couldn't find where this is happening. Can you point me to it?

The animation stuff that was in the styling. Thanks to the git blame, I was able to find this PR: https://github.com/microsoft/pxt/pull/6774, showing what the animation stuff should be giving us. This is broken as far as I can tell. I'm wondering if this is something that I should open a bug for or if I should just get rid of the animation logic.

Personally, I'd recommend just opening a bug for now. No need to remove the code; it could be helpful for whoever ends up debugging.

I might be missing something obvious, but I couldn't find where this is happening. Can you point me to it?

Yeah, no problem, https://github.com/microsoft/pxt/blob/8aa26877869c51363dd0e3d8b77f71ee7a301e22/theme/toolbox.less#L285

Hover and not hovering for inverted toolbox is no longer handled in JS, but is handled by the :hover property in CSS

I might be missing something obvious, but I couldn't find where this is happening. Can you point me to it?

The animation stuff that was in the styling. Thanks to the git blame, I was able to find this PR: #6774, showing what the animation stuff should be giving us. This is broken as far as I can tell. I'm wondering if this is something that I should open a bug for or if I should just get rid of the animation logic.

Personally, I'd recommend just opening a bug for now. No need to remove the code; it could be helpful for whoever ends up debugging.

Unless it's causing a crash or something, that is. In that case, I'd say we should remove it and reference the commit that removed it in the bug. If it's just not doing anything then I think it's fine to leave it in.

Personally, I'd recommend just opening a bug for now. No need to remove the code; it could be helpful for whoever ends up debugging.

Unless it's causing a crash or something, that is. In that case, I'd say we should remove it and reference the commit that removed it in the bug. If it's just not doing anything then I think it's fine to leave it in.

Yep, not causing any issues, just not adding anything.


Support for PSCore 7

Is your feature request related to a problem? Please describe.
Supporting officially PSCore v7, possibly v6
Describe the solution you'd like
Clean usage as in PS Desktop v5
Describe alternatives you've considered
maybe only ps core v7?

Blocked by issue regarding ADAL libraries and PS Core not able handling auth-popup without credential
https://www.enowsoftware.com/solutions-engine/powershell-core-is-ga-but-not-ready-for-office-365

Possible now as most modules seem to be support somehow by PSCore.
Unfortunately, there are plenty of restrictions, different options and switches that need to be considered by the different modules MS-modules in different ways.


How shall I add codes if I want to add some actions?

Now I want to add new actions such as  OFPAT_PUSH_GTPU and OFPAT_POP_GTPU. Are there any file I should add codes in the project?
I have added some codes there:
ofp_action.c:
switch(tlv.type):
add case OFPAT_PUSH_GTPU:
add case OFPAT_POP_GTPU:
openflow13.h:
enum ofp_action_type:
add OFPAT_PUSH_GTPU = 28;
add OFPAT_POP_GTPU = 29;
openflow13packet.c:
ofp_action_type_str(enum ofp_action_type val):
add case OFPAT_PUSH_GTPU:
add case OFPAT_POP_GTPU:
group.c:
get_group_features:
add (1 << OFPAT_PUSH_GTPU)
add (1 << OFPAT_POP_GTPU)
flowdb.c:
static int action type[]:
add OFPAT_PUSH_GTPU, OFPAT_POP_GTPU
flow_dump: dump_actions:
add case OFPAT_PUSH_GTPU:
add case OFPAT_POP_GTPU:
flowdb_show.c:
show_action:
add case OFPAT_PUSH_GTPU, case OFPAT_POP_GTPU
Please help to develop new actions :)

Sorry for late reply.
Thank you for your contribution to Lagopus project.
We are waiting your pull request :)


config: added ava prefix

This pull request is fixing:
1:1  error  Definition for rule 'no-identical-title' was not found                                    no-identical-title
1:1  error  Definition for rule 'no-invalid-end' was not found                                        no-invalid-end
1:1  error  Definition for rule 'no-only-test' was not found                                          no-only-test
1:1  error  Definition for rule 'no-skip-assert' was not found                                        no-skip-assert
1:1  error  Definition for rule 'no-skip-test' was not found                                          no-skip-test
1:1  error  Definition for rule 'test-ended' was not found                                            test-ended
1:1  error  Definition for rule 'test-title' was not found                                            test-title

According to the docs this is not correct: http://eslint.org/docs/developer-guide/working-with-plugins#rules-in-plugins
You're the one that has to prefix rules with ava/, not this plugin.

@sindresorhus The error happens with the recommended settings.
I can replicate the error when using "extends": "plugin:ava/recommended" in my eslintrc.

@sindresorhus That's true, but this PR changes the recommended config, not the rules themselves.
See also the recommended configs of eslint-plugin-import, eslint-plugin-react.

I can replicate this, and this fix makes it ok, so this looks good to me. Thanks @solcik!

Oh, sorry. I thought I was looking at a different part of that file.


Old PR editors show error after revisiting

Version: 1.77.0-insider
Commit: a5bd844beddef8ca5f9ccfdb1bb7b3b4f8e5204e
Date: 2023-03-22T10:41:38.157Z
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 22.3.0
Sandboxed: Yes
Steps to Reproduce:

Finish a PR yesterday
Close window (with PR editor open) yesterday
Come back to it today:

I have the PR extension installed and am able to interact with it just fine (create new PRs etc)

I have not seen this in a while so marking as verified.


Changed Action schema and streamlined code. Fixed toast not appearing.

Use factory pattern to ensure all actions are valid or throw error. All actions are conforming now.
Update schema to reflect new Action schema:

onTap:
  invokeAPI:
    options: ...

@vusters can you fix the menu as well. adding @imjaadFRT
Menu should be -
ViewGroup:
  menu:
    BottomNavBar:
      icon:
      label:
      page:
      selected:
      onTap:

etc

@imjaadFRT done


Added information about npm package version of the test suite

This PR updates the README with a Node.js section that provides a link to an npm package of the JSON Schema Test Suite. The package facilitates testing by validator implementors without requiring that git submodule be added to their repositories.
Relates to Issue #96

Closing this PR since I've submitted #99 instead to merge the Node.js additions into this project.


Example not work: The argument type 'List' can't be assigned to the parameter type 'List'.

audioplayers-main/packages/audioplayers/example/lib/components/tgl.dart:
The argument type 'List' can't be assigned to the parameter type 'List'.
Missing type arguments for generic method 'map'.
The getter 'name' isn't defined for the type 'T'.

I just tried it locally, it is definitely working with no errors whatsoever

Also works for me. Try upgrading flutter...


Create a Spring Boot/Spring MVC E2E sample to demonstrate usage of Spring Azure modules

@backwind1233 any idea on this, should we close this one and use the petclinic to implement these feature support?

closing this one, reopen if needed


Login to anime game account, displays error code 9006. any suggestions?

Yaagl version 0.1.5
g.i wine 1.2
on ventura 13.4.1

apfs external disk needed.


Preferred Editor

Which editor/IDE do you currently use to edit the JavaScript source files? I know among the free tools Atom used to be the preferred tool of choice for text editors but personally I'm partial to the new kid on the block from Microsoft: VS Code. It has better support for JavaScript and git built in, and plugins for everything else I've needed so far including ES Lint/Prettier to keep the quality/formatting of my code correct in real time. Plus the development on it has been very active and constantly improving while Atom seems somewhat stagnant lately.
If you're interested in using VS Code, or at least not otherwise bothered, I would like to commit some files I have locally that provide a launch configuration for running the tests in milsymbol with debugging support in your editor as well as some general config items for ES Lint/Prettier/etc. If you're not OK with that, then I'd like to add an ignore for these files so they won't keep showing up as changes and might accidentally get commited.

I have historically used BBEdit, but a week ago or so I tried out VS Code and really like it. I have set it up with Prettier (format on save) and ESLint.
If it is a limited number of files I think we could keep them in the repository to make it easier for other that want to contribute. If you update Contributing.md with information about how to use VS Code you can commit the files.

Pull request #102 has an initial pass at this.


Please publish a github release for JitPack to access

Jitpack allows developers to reference builds directly from github by tag or branch,
Currently though it looks like the auto build does not support JDK 16
This results in the below log when using JitPack
https://jitpack.io/com/github/Auties00/WhatsappWeb4j/master-whatsappweb4j-1.2-gd6edb3d-77/build.log
https://jitpack.io/#Auties00/WhatsappWeb4j/master-SNAPSHOT
Not sure if this would be something you are interested in supporting really

Jitpack allows developers to reference builds directly from github by tag or branch,
Currently though it looks like the auto build does not support JDK 16
This results in the below log when using JitPack
https://jitpack.io/com/github/Auties00/WhatsappWeb4j/master-whatsappweb4j-1.2-gd6edb3d-77/build.log
https://jitpack.io/#Auties00/WhatsappWeb4j/master-SNAPSHOT
Not sure if this would be something you are interested in supporting really

Looks interesting, I'll add it before version 2.0 comes out. There's a problem though as the unit tests for the build require a WhatsApp connection to exist, which is not really something that can be automated. I've not managed to come up with a fix for this for Github's Maven CI, so I guess I'll do the same for jitpack. Le me know if you have better ideas


Support refreshing credentials on a deployment

When the credentials in the broker have changed for a service, it would be nice to have a straightforward way to refresh any bound secrets.
When @arschles first suggested this in the original svcat repo, he approached this from the cli. For example:
svcat rebind --name BINDING_NAME

The above command would create a new binding and then update the original secret with the new values, and then restart anything that was using it.
That would work but I do wonder if perhaps it would be best to have service catalog support this scenario at the API level, instead of filling in functionality gaps through the cli.

This issue was migrated from the original svcat repo: https://github.com/Azure/service-catalog-cli/issues/17

Does this unbind the original binding?
Does rebinding a service instance make the binding count n+1?

@arschles This was originally your suggestion (I just copied it over from the original svcat repo before archiving it). Maybe we can back up a bit off the implementation and go back to understanding when someone would need to refresh credentials and how we'd like that interaction to look? Can you provide some more context around the original idea?

Imagine that the service broker wished to rotate the credential that is encapsulated within a binding.  Ideally the service catalog would poll the binding information and update the Secret resource accordingly.  When a Secret is updated, Kubernetes automatically updates the secret data in the pod (volume-based secrets only AFAIK).  For example, given a binding that includes a private key file, the file would automatically be updated in all associated pods whenever the broker updated the binding info, without human intervention.

I think a key rotation feature is something that works with service catalog rather being a service catalog feature directly. It depends on where you want to draw the line, is service catalog enabling building blocks or is it an application? I lean towards building blocks.

@n3wscott the 'building block' that the catalog provides is that it manages a Secret resource that represents the binding info.   Key rotation is application-specific, yes, but what I described is appropriate given the functionality that is intrinsic to the catalog.  I do wonder whether OSBA specifies the updatability of binding information.

Bindings are not updatable in the current spec.


process_queue triggered an exception

Hi,
I get this error when trying to run the bot.

Thanks for the bug report! Can you please post your yml configuration file? Or, at the very least, what version of OpenTTD the server is running?

server:
invite_code: "+KkZEhfU" # either this or server_host is REQUIRED
player_name: "doopekk" # REQUIRED
company_name: "WoodyWoodPacker LLC" # either this or company_id is REQUIRED
server_port: 3979
server_password: # default: unset
company_password: "somePassword!" # default: unset
bot:
Go into spectator mode if the bot is the only player currently playing.
This is useful if you'd like the game to be paused when nobody is playing.
spectate_if_alone: true
Connect back to the server if connection is lost or bot got kicked.
auto_reconnect: true
How long to wait before reconnecting. Applicable only if auto_reconnect is set.
auto_reconnect_wait: 30
How many times to try to reconnect before giving up and exiting.
reconnect_count: 3
Should the bot reconnect if the game password is incorrect?
auto_reconnect_if_wrong_game_password: false
Should the bot reconnect if the company name does not exist?
auto_reconnect_if_company_not_found: false
Should the bot reconnect if it can't move to the specified company?
This can happen if the company does not exist, or password is incorrect.
auto_reconnect_if_cannot_move: false
Should the bot reconnect if the server is shutting down?
auto_reconnect_if_shutdown: false
Should the bot reconnect if the bot got banned?
auto_reconnect_if_banned: false
Should the bot reconnect if the server is restarting?
Note that after restart, the company will, most likely, no longer exist.
auto_reconnect_if_restarting: false
Bot log level. See https://docs.python.org/3/library/logging.html#levels for levels.
log_level: DEBUG
ottd:
network_revision: "12.2"
revision_major: # default: derived from network_revision
revision_minor: # default: derived from network_revision
revision_stable: true
coordinator_host: coordinator.openttd.org
coordinator_port: 3976

Any news on this?

Any news on this?

I will let you know when I do some investigations on this.

Ok, no problem. I am sorry, but I am just addicted to ottd :)

I don't know if that might be useful, but I run ottd on GOG

The problem is due to NewGRFs being present on the server, the bot chokes on them. I'll try to get the bot to support (and ignore) NewGRFs.

Knowing what's the problem is a first step to solve it 😊 That's a great news, can't wait to use it :)

Should be fixed now, please let me know the result.

Seems fine, thank you very much! :)


Optimize chest images

Fixes #64

Hey @MaherFa could you take a look at this please? I think it looks good but as you did this work, maybe you have a comment or something. Thanks!

@AmirSavand thank you.
@martincarrera this pull is ok to merge, thank you for checking with me


Enable simulation with Metrics DSim

Changes to dsim.hjson to enable use of the Metrics simulator with OpenTitan.
One small change to code in dv_base_env.sv: DSim enforces assignment rules strictly, and does not perform an implicit conversion when assigning an integer to a real number.
Please contact me if you need help testing these changes.

Adding @udinator who is aware of this work.

Thanks for this PR @aimeepsutton .  This means you agree with the CLA policy as outlined in https://github.com/lowRISC/opentitan/blob/master/CLA. After that, the CI check should be green.

Thanks for this PR @aimeepsutton .  This means you agree with the CLA policy as outlined in https://github.com/lowRISC/opentitan/blob/master/CLA. After that, the CI check should be green.

Done, sorry I missed that!

Adding @slaval-google to the discussion.

I believe I have implemented all the feedback and resolved conflicts. I think this should merge cleanly with lowRISC:master, but please let me know if that is not the case. I can do a git rebase if required.

I believe I have implemented all the feedback and resolved conflicts. I think this should merge cleanly with lowRISC:master, but please let me know if that is not the case. I can do a git rebase if required.

somehow, it still shows conflicting. Can you do a rebase or just create another fresh PR? Thanks

After git rebase there is still an issue even though everything looks clean from my side. I'll close this PR and open a new one. Sorry for this!


feat(battery_plus): add save power mode getter

Description
Adding save power mode getter  in order to disable some functionalities if the device is in battery-save mode.
Only Android & iOS are done for now, need some help for other platforms.
I try to run melos run format but without success even in a clean clone.
Related Issues
#229
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

This is very nice, thank you, however, I have two concerns
1- Since you have introduced this feature, it has just been implemented in Android, I think we should take care of other platforms too as much as possible for example, iOS has LOW Battery Mode. I understand that might now be your expertise but we need to ensure that we have a good understanding of what will happen next if we merge this PR.
2- can you please also check why some of the CI tasks fail.

Thanks for the review 👍 !
1-  iOS has been done in this PR, only linux, macOS and windows are missing. For macOS, API are not ready for now (only macOS 12 beta and 13)
2- yes, I will check 🥇

thanks @v-apparence this LGTM. Thank you for adding this.
@jpnurmi @miquelbeltran, please leave a comment if you have any?

LGTM as well! the version needs to be bumped for a new release, on both the platform interface and the main plugin projects

I can add Windows support. Should I do it in a new PR once this gets merged or to this PR via @v-apparence's branch?

Thank to @mzdm, windows support added 🥇


Fix flag() with rest() support with no arg()

Rest was triggered when all defined arguments were set, but in a scenario where there are no defined arguments and only a rest() to vacuum up arguments flags were never processed even if they were first.
The Parser now attempts to parse first and if the command falls through to interpreting the result as an argument, then the check for the number of defined arguments found is run. This means the parser needs to be walked back one place as it has already processed the first argument of the rest at that point.
It is assumed that the first non-defined argument found is the start of the rest arguments and all remaining arguments belong to the rest. This means even valid flags after the first rest argument are considered rest arguments, not flags.

Nice! @davidmarkclements can you review?

@lejeunerenard v.1.1.4 released - thank you


[BR] gc safepoint not set correctly when there is log backup fatal error

Bug Report

What version of TiKV are you using?
TiKV
Release Version:   6.1.0-alpha
Edition:           Community
Git Commit Hash:   f05af56d12b93c74ac5a54a28f77d2f6dc7d6ad8
Git Commit Branch: inflight-detect
UTC Build Time:    2022-05-17 08:48:36
Rust Version:      rustc 1.60.0-nightly (1e12aef3f 2022-02-13)
Enable Features:   jemalloc mem-profiling portable sse test-engine-kv-rocksdb test-engine-raft-raft-engine cloud-aws cloud-gcp cloud-azure
Profile:           release
What operating system and CPU are you using?
Cent OS, 16U
Steps to reproduce

Start a log backup task
Simulate fetal errors like storage can't be access

What did you expect?
When error happens, GC safepoint should be set appropriately to prevent GC scheduler delete logs that is not backed up.
What did happened?
Currently we use start_ts to set the GC safepoint, which doesn't take effect actually, and will result in data loss.

fix by https://github.com/joccau/tikv/pull/13

/close


[AutoPR datafactory/resource-manager] Add support for HDIinsight cluster with Enterprise Serurity Package

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4780

Build output:

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/2136


Intro to Ember

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #6654

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

@08-dev-verma 👋  What topic suggestion does this PR link to?

Linked PR to Topic Suggestion correctly. @08-dev-verma, please remember to remove the backticks in the future.

Mam what type of backticks , I didn't get it.

Linked PR to Topic Suggestion correctly. @08-dev-verma, please remember to remove the backticks in the future.

Mam what type of backticks , I didn't get it.

By default, the line to link your PR to your topic suggestion is surrounded by backticks to show it as an example. You need to remove this when you add your topic suggestion number.

Linked PR to Topic Suggestion correctly. @08-dev-verma, please remember to remove the backticks in the future.

Mam what type of backticks , I didn't get it.

By default, the line to link your PR to your topic suggestion is surrounded by backticks to show it as an example. You need to remove this when you add your topic suggestion number.

Sure mam , from next time I will keep that in mind.

Plagiarism check complete

Plagiarism check complete

Mam can u pls tell me what's the status of my article ?

@Jethro-magaji sir what's the status of my article ?

@Jethro-magaji sir what's the status of my article ?

It's ready for final review by a CM.

Noted - we will get to it asap - we have a few other in the queue

@Jethro-magaji sir what's the status of my article ?

It's ready for final review by a CM.

Sir it's been more than 2 weeks and I'm also working on other article too for section so can you please tell me what 's the status ?

@Jethro-magaji sir what's the status of my article ?

It's ready for final review by a CM.

Sir it's been more than 2 weeks and I'm also working on other article too for section so can you please tell me what 's the status ?

Please, calm down.
There's a queue, when it gets to your turn it would reviewed and then published.
Thanks.

@Jethro-magaji sir can you please give me more information about the latest commit as this is my first article?

?

Sir , I'm little confused about it.

@Jethro-magaji sir can you please give me more information about the latest commit as this is my first article?

What seems to be the issue?

Sir, I'm little confused about it.

@Jethro-magaji sir can you please give me more information about the latest commit as this is my first article?

What seems to be the issue?

Sir, I'm little confused about it.

What clarification do you need?

@Jethro-magaji sir can you please give me more information about the latest commit as this is my first article?

What seems to be the issue?

Sir, I'm little confused about it.

What clarification do you need?

Sir clarification about the latest commit as I'm new here

@Jethro-magaji sir can you please give me more information about the latest commit as this is my first article?

What seems to be the issue?

Sir, I'm little confused about it.

What clarification do you need?

Sir clarification about the latest commit as I'm new here

Your article is on the queue, please be patient while it gets to your turn.
Thanks

@08-dev-verma
Don't believe there is an issue (w/ the PR) - just going through the queue.
Have noticed that this PR is a (3x) duplicate - we will be adding a white label to this PR because it was re-submitted on the 19th after being closed on the 10th.
FYI

@hectorkambow
Sir can you please tell me about the white label for this PR?

White label indicates that the PR was submitted/approved after the payout transition announcement.
It means the PR will not be eligible for a payout.

Sir as I was told to wait for my article to get published I waited patiently and now you are saying it's not eligible for payout?? , I don't understand it. And even my article is published or not I don't have any surety after this long wait.

White label indicates that the PR was submitted/approved after the payout transition announcement.
It means the PR will not be eligible for a payout.

Sir as I was told to wait for my article to get published I waited patiently and now you are saying it's not eligible for payout?? , I don't understand it. And even my article is published or not I don't have any surety after this long wait.

@08-dev-verma
I believe you were instructed to revise your PR a couple of times or so.
It was closed and re-submitted 2-3 times.

"I found that we might need some major revisions before we can proceed with the next round of review. The changes primarily deals with the markdown formatting."

I have not gone over this new PR (to assume its quality).
But our policy was that PR should be submitted error-free and fully polished (ready to be published).
As our Peer reviewer team does not provide revision services.
That is why we are adding the white label.

We are working our way down the queue asap - this should be reviewed within the week or so.

👏
Well done @08-dev-verma
I'd recommend adding a skills section to your author page- that will go a long way with future potential employers.


"Fight" message when starting a match.

Hello! My issue is that the Fight message when starting a fight comes really quick (before the voice announcer says FIGHT) and it is quite confusing and annoying, is there any fix to that so the voice announcer and the message can be matched? Thanks in advance.

Duplicate of: https://github.com/TekkenOverlay/TekkenOverlay/issues/62


[DPE-4279] Revamp Charmed Spark documentation

I have revamped and updated the documentation, given that we the latest updates we have changed and improved the UX quite substantially.
In the text, you will often find "TODO". These represent placeholder that will be populated once we will create the Discourse pages.

superseded by #23


Allow any color to be passed through as primary to checkbox

Hey,
We had the issue where we needed a white checkbox, and needed to specify a custom color
Cheers,

Thanks for this


TREC Microblog 2013-14

We can add the TREC Microblog 2013-14 benchmarks using the Tweets2013-IA collection. This isn't the exact same document collection, but it's close enough as explained here.
The tricky is supporting document lookups. The collection is rather large (79GB compressed, 250M tweets). We'll need to decide whether to go with: (a) lz4pickle docstore, or (b) lookups from the source files.
The main problem with (a) is the large number of documents. The current doc_id lookup structure may not scale well to this. The size could be managed by treating the tweet IDs as integers. Compression may also be a limiting factor. Since tweets are short, there may be little to compress, inflating the size of the data segment.
Looking up from the source files (b) is feasible, given that the source files are organized by date and the tweet IDs include timestamps. This will be slower than using the lookup structure, but will probably save storage space. And the transparent caching should help subsequent reads.

Using the file structure from the archives for lookups is imperfect. While timestamps can be extracted from the tweet IDs, they do not always translate perfectly to the files.
This code identifies the correct file ~99.7% of the time:
def sf2dt(sfid):
  ts = ((sfid >> 22) +<PHONE_NUMBER>657) / 1000 # "magic" numbers from https://github.com/client9/snowflake2time/blob/master/python/snowflake.py#L24
  dt = datetime.datetime.fromtimestamp(math.floor(ts))
  return dt
def sf2file(sfid):
  dt = sf2dt(sfid)
  dt -= datetime.timedelta(hours=2) # not sure why there's this offset... maybe a problem with the magic number? Or TZ weirdness?
  return f'{dt.month:02d}/{dt.day:02d}/{dt.hour:02d}/{dt.minute:02d}.json.bz2'

This can be slightly improved by also checking the next and previous files, which happens occasionally (but in a seemingly unpredictable way).
There's some files that end up just being completely wrong, such as 03/10/01/54.json.bz2 and 03/10/01/40.json.bz2, with timestamps indicating 03/10/02/54.json.bz2 and 03/10/02/40.json.bz2, respectively. And there's occasionally oddball cases as well.
(Note: this was only looking at the March 2013 archive; Feb 2013 could be different.)

An alternative approach would be to re-build a similar (but correct) file structure as the files are downloaded.
I think this is the way to go because it also gives us the chance to:

Change the compression to lz4 for much faster decompression
(possibly) change the granularity of the structure, if we want more or fewer tweets per file on average
(possibly) remove unnecessary records (e.g., delete records) or fields (e.g., redundant id and str_id files) from the source.
(possibly) introduce sorting by ID (within lz4 blocks) to allow for early exits

I don't think doing the work above would even add any real overhead to getting started with the collection. Downloading these files is pretty slow (ranging from a few hundred KB/s, at most 1-2MB/s) and all this work could be done while streaming the content.


Indy Java Wrapper eats the exception when it can't load libindy via jna

https://github.com/hyperledger/indy-sdk/blob/master/wrappers/java/src/main/java/org/hyperledger/indy/sdk/LibIndy.java#L186
	static {

		try {

			init();
		} catch (UnsatisfiedLinkError ex) {

			// Library could not be found in standard OS locations.
			// Call init(File file) explicitly with absolute library path.
		}
	}

The exception is getting eaten and no log output or anything, this results in a null pointer exception later on but we don't know what the cause is. Ideally we should at least print this out using a logger or system.out.

@kchan628 it's better to have this issue as ticket in https://jira.hyperledger.org/projects/IS/
Would you like to create it from yourself or I may do it?

hi @jovfer you can create it. I just wanted to report the issue to hopefully smooth out usage for later. What I ended up doing is building from source and removing that catch to get at the exception to find the root cause of why it wasn't linking.


substr() throws if pos == size

For example:
    string_view x("abc");
    string_view y = x.substr(3);
    ASSERT_TRUE(y == "");

I would expect an empty string to be returned.
Compare https://en.cppreference.com/w/cpp/string/basic_string_view/substr

@AdmiralCurtiss : Have you thought about forking this repo and fixing this yourself? It looks like it hasn't been touched in 3 years.


Add blade directives to clean up views

Any other directives I should add?

Was on WIP in case somebody had a different name in mind or more directives.

better to have new directives on a separate PR.. :) merging this


#548 Safari won't load video

recognize that safari won't load videos, keep showing the spinning animation and doesn't preload the video.
sometimes after pause and play again the video loads.
same mac, used chrome and #548 same settings works without problems
safari Version 10.0.3 (12602.4.8)

I can confirm this issue. Same YTC and Safari version.


Kubeconfig rotation does not work as expected

How to categorize this issue?

/area usability
/kind bug
/priority critical
What happened:
After the rotation of the kubeconfig of a shoot the new kubeconfig is (sometimes?) not synced into the project namespace in the garden cluster, hence, the downloadable kubeconfig does not work and the end-user cannot interact with his cluster.
What you expected to happen:
The kubeconfig in the project namespace is up to date.
How to reproduce it (as minimally and precisely as possible):
n/a yet, sorry
Anything else we need to know?:
I've seen occasions in which even the kubecfg secret in the shoot namespace in the seed was not correct, i.e., in those cases not even a second reconciliation helps, but only trying to rotate the credentials again.
Environment:

Gardener version: v1.11.3

After some investigation I found several weird things

the secrets in the shoot namespace in the seed are re-generated
the shootstate in the garden is never updated and keeps the old static-token
sometimes the kubeconfig synced to the garden clusters is containing the deleted token, sometimes it is properly updated
not every trigger of kubeconfig rotation reproduces the issue.

Just wild guessing, it might be related to https://github.com/gardener/gardener/pull/2798 and to the cached clients, but I have no proofs about it. I will keep looking into it trying to understand more what exactly is happening.

/cc @plkokanov @swilen-iwanow

I can confirm that the shoot state is never updated and this is always reproducible.
The reason seems to be the reflect.DeepEqual that does not detect differences correctly.
I have added some logs:

diff --git a/pkg/apis/core/v1alpha1/helper/shootstate_list.go b/pkg/apis/core/v1alpha1/helper/shootstate_list.go
index bc60271bf..dde5b29ec 100644
--- a/pkg/apis/core/v1alpha1/helper/shootstate_list.go
+++ b/pkg/apis/core/v1alpha1/helper/shootstate_list.go
@@ -85,6 +85,15 @@ func (g *GardenerResourceDataList) Get(name string) *gardencorev1alpha1.Gardener
 	return nil
 }

+// Keys returns the names of the items in the list
+func (g *GardenerResourceDataList) Keys() []string {
+	keys := []string{}
+	for _, resourceDataEntry := range *g {
+		keys = append(keys, resourceDataEntry.Name)
+	}
+	return keys
+}
+
 // Upsert inserts a new element or updates an existing one
 func (g *GardenerResourceDataList) Upsert(data *gardencorev1alpha1.GardenerResourceData) {
 	for i, obj := range *g {
diff --git a/pkg/operation/botanist/secrets.go b/pkg/operation/botanist/secrets.go
index 624fd2dfe..cab4fbe14 100644
--- a/pkg/operation/botanist/secrets.go
+++ b/pkg/operation/botanist/secrets.go
@@ -47,9 +47,12 @@ func (b *Botanist) GenerateAndSaveSecrets(ctx context.Context) error {
 	gardenerResourceDataList := gardencorev1alpha1helper.GardenerResourceDataList(b.ShootState.Spec.Gardener)

 	if val, ok := common.GetShootOperationAnnotation(b.Shoot.Info.Annotations); ok && val == common.ShootOperationRotateKubeconfigCredentials {
+		b.Logger.Info("[ROTATION] Will try to rotate secrets")
 		if err := b.rotateKubeconfigSecrets(ctx, &gardenerResourceDataList); err != nil {
+			b.Logger.Error("[ROTATION] Failed to rotate secrets")
 			return err
 		}
+		b.Logger.Info("[ROTATION] Secrets rotatet successfully")
 	}

 	if b.Shoot.Info.DeletionTimestamp == nil {
@@ -82,14 +85,18 @@ func (b *Botanist) GenerateAndSaveSecrets(ctx context.Context) error {
 	if shootWantsBasicAuth {
 		secretsManager = secretsManager.WithAPIServerBasicAuthConfig(basicAuthSecretAPIServer)
 	}
+	b.Logger.Infof("[ROTATION] ResourceList before generation: %v", secretsManager.GardenerResourceDataList.Keys())

 	if err := secretsManager.Generate(); err != nil {
 		return err
 	}
+	b.Logger.Infof("[ROTATION] ResourceList after generation: %v", secretsManager.GardenerResourceDataList.Keys())

 	if reflect.DeepEqual(secretsManager.GardenerResourceDataList, gardencorev1alpha1helper.GardenerResourceDataList(b.ShootState.Spec.Gardener)) {
+		b.Logger.Info("[ROTATION] Generated and existing state secrets are the same.")
 		return nil
 	}
+	b.Logger.Info("[ROTATION] Generated and existing state secrets differ.")
 	return b.SaveGardenerResourcesInShootState(ctx, secretsManager.GardenerResourceDataList)
 }

and find out that when there is a differences in the resources list, reflect does not detect it
time="2020-10-27T21:30:01+02:00" level=info msg="[ROTATION] Will try to rotate secrets" operation=reconcile shoot=foo/bar
time="2020-10-27T21:30:01+02:00" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=foo/bar task="Generating etcd encryption configuration"
time="2020-10-27T21:30:02+02:00" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=foo/bar task="Deploying Kubernetes API server service in the Seed cluster"
time="2020-10-27T21:30:02+02:00" level=info msg="[ROTATION] Secrets rotatet successfully" operation=reconcile shoot=foo/bar
time="2020-10-27T21:30:02+02:00" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=foo/bar task="Waiting until Kubernetes API LoadBalancer in the Seed cluster has reported readiness"
time="2020-10-27T21:30:02+02:00" level=info msg="[ROTATION] ResourceList before generation: [ca ca-etcd ca-front-proxy ca-kubelet ca-metrics-server kube-apiserver kube-apiserver-kubelet kube-aggregator kube-controller-manager kube-controller-manager-server kube-scheduler kube-scheduler-server cluster-autoscaler gardener-resource-manager kube-proxy kube-state-metrics prometheus prometheus-kubelet gardener gardener-internal cloud-config-downloader monitoring-ingress-credentials monitoring-ingress-credentials-users ssh-keypair service-account-key etcd-server-cert etcd-client-tls metrics-server alertmanager-tls grafana-tls prometheus-tls dependency-watchdog-internal-probe dependency-watchdog-external-probe vpn-shoot vpn-seed vpn-seed-tlsauth vpa-tls-certs etcdEncryptionConfiguration]" operation=reconcile shoot=foo/bar
time="2020-10-27T21:30:02+02:00" level=info msg="[ROTATION] ResourceList after generation: [ca ca-etcd ca-front-proxy ca-kubelet ca-metrics-server kube-apiserver kube-apiserver-kubelet kube-aggregator kube-controller-manager kube-controller-manager-server kube-scheduler kube-scheduler-server cluster-autoscaler gardener-resource-manager kube-proxy kube-state-metrics prometheus prometheus-kubelet gardener gardener-internal cloud-config-downloader monitoring-ingress-credentials monitoring-ingress-credentials-users ssh-keypair service-account-key etcd-server-cert etcd-client-tls metrics-server alertmanager-tls grafana-tls prometheus-tls dependency-watchdog-internal-probe dependency-watchdog-external-probe vpn-shoot vpn-seed vpn-seed-tlsauth vpa-tls-certs etcdEncryptionConfiguration static-token]" operation=reconcile shoot=foo/bar
time="2020-10-27T21:30:02+02:00" level=info msg="[ROTATION] Generated and existing state secrets are the same." operation=reconcile shoot=foo/bar

For regular reconciliation there is no difference, which is expected
time="2020-10-27T21:40:47+02:00" level=info msg="[ROTATION] ResourceList before generation: [static-token ca ca-etcd ca-front-proxy ca-kubelet ca-metrics-server kube-apiserver kube-apiserver-kubelet kube-aggregator kube-controller-manager kube-controller-manager-server kube-scheduler kube-scheduler-server cluster-autoscaler gardener-resource-manager kube-proxy kube-state-metrics prometheus prometheus-kubelet gardener gardener-internal cloud-config-downloader monitoring-ingress-credentials monitoring-ingress-credentials-users ssh-keypair service-account-key etcd-server-cert etcd-client-tls metrics-server alertmanager-tls grafana-tls prometheus-tls dependency-watchdog-internal-probe dependency-watchdog-external-probe vpn-shoot vpn-seed vpn-seed-tlsauth vpa-tls-certs etcdEncryptionConfiguration]" operation=reconcile shoot=foo/bar
time="2020-10-27T21:40:47+02:00" level=info msg="[ROTATION] ResourceList after generation: [static-token ca ca-etcd ca-front-proxy ca-kubelet ca-metrics-server kube-apiserver kube-apiserver-kubelet kube-aggregator kube-controller-manager kube-controller-manager-server kube-schedulerkube-scheduler-server cluster-autoscaler gardener-resource-manager kube-proxy kube-state-metrics prometheus prometheus-kubelet gardener gardener-internal cloud-config-downloader monitoring-ingress-credentials monitoring-ingress-credentials-users ssh-keypair service-account-key etcd-server-cert etcd-client-tls metrics-server alertmanager-tls grafana-tls prometheus-tls dependency-watchdog-internal-probe dependency-watchdog-external-probe vpn-shoot vpn-seed vpn-seed-tlsauth vpa-tls-certs etcdEncryptionConfiguration]" operation=reconcile shoot=foo/bar
time="2020-10-27T21:40:47+02:00" level=info msg="[ROTATION] Generated and existing state secrets are the same." operation=reconcile shoot=foo/bar

Just a note: I've been trying to reproduce it with gardenlet from v1.12.0-dev-cf9a793c4cdce4a64cb329868bd0ebad7c31b8fd and haven't been able to for the past 40 minutes (I did around 10-12 rotations)
How often does it happen with 1.13?

The code that syncs the credentials back to the Garden cluster is looking fine to me and I do not have any suspicious.
I will keep on eye on

sometimes the kubeconfig synced to the garden clusters is containing the deleted token, sometimes it is properly updated

in the next weeks and if it is not reproduced, I will close the issue.

@vpnachev , do you have any observations on it, can it be closed now?
I tried to look into the test results for the integration test that you added. But after 20m of search for test results I found that the commit that adds the integration test was not cherry-picked into the release branch - only the fix commit. Ref https://github.com/gardener/gardener/commits/release-v1.12.

I faced the issue only once, but the shoot still was not reconciled by the fixed version. I would like observe it next week and if we do not see it, we can close this issue.

And, yes - the test was not cherry-picked as it is more like a feature. I was not sure whether we should do it or not.

Thank you for the update!

I faced the issue only once, but the shoot still was not reconciled by the fixed version. I would like observe it next week and if we do not see it, we can close this issue.

Any new observations?

Nope.
/close
fixed by #3069


1.17?????

Is this mod also for 1.17????
I hope so 😇😇😇

Hi there! Thanks for asking! Currently there is no 1.17 version as there is no 1.17 forge version available; however once there is, I'll be sure to update as soon as possible!


version of apk file

Is there a way to name the apks by version? We are archiving apk files by version number for clients from china. So version number as part of the output filename would be really neat. Basically if we put this to crontab, even we do not know the version without looking up to web.

Hi, you can add the version to the name of the saved file. First you need to save the version by modifying this dictionary (add the following 2 lines): https://github.com/ClaudiuGeorgiu/PlaystoreDownloader/blob/efcefa2de8b77633710fc31dcea6157b212ddfe1/download.py#L97-L101
"versionString": app.details.appDetails.versionString,
"versionCode": app.details.appDetails.versionCode,

Then you can change this code to include the version in the name:
https://github.com/ClaudiuGeorgiu/PlaystoreDownloader/blob/efcefa2de8b77633710fc31dcea6157b212ddfe1/download.py#L111-L113
"{0} by {1} - {2} - v. {3}.apk".format(
    details["title"], details["creator"], details["package_name"] , details["versionString"]
),


Prevent stray output from reaching stdout

prevents prints that happen after (user's) main returns from polluting stdout.
Context: https://stackoverflow.com/questions/75714151/kotlin-coroutines-order-of-printing-logs/75760394#75760394

On second thought, not switching System.out back may cause stray prints become data races as the main thread reads from ByteArrayOutputStream and the thread the prints writes to it.
But if that is the reason for calling setOut, System.err should also be switched back.

Or if the writes synchronize over the BAOS as I suggested in #619, the data race can be prevented by
synchronized(outputStream) {
    outputObj.text = outputStream.toString()
        .replace("</errStream><errStream>".toRegex(), "")
        .replace("</outStream><outStream>".toRegex(), "")
}

Hi, is it possible to add a test snippet?

possible to add a test

Is the modified test sufficient?
https://github.com/JetBrains/kotlin-compiler-server/pull/618#discussion_r1148465908

Following up on this.
I wasn't very clear in my previous comment. I have changed the test which now guards against the bug. I don't think a new additional test is needed.
The test that I modified, base coroutines test 7 used to only test for substring match.
Now it is testing for full equality.

This bug has confused another person.
https://stackoverflow.com/questions/77619172/why-does-the-kotlin-coroutine-output-a-b-even-with-a-delay

I found another SO question caused by this bug.
Imagine how many people are confused by this but didn't ask a question.
https://stackoverflow.com/questions/71918773/why-does-coroutine-in-background-execute-before-the-main-thread-in-this-kotlin-c

@phiSgr, thank you for bringing this up!
We are reviewing the PR, and I hope we can merge it this week.

No objections from my side.
@phiSgr, thank you for the contribution!

It seems to me that it's now deployed to play.kotlinlang.org only in version 2.0.0-RC2. Will it be back-ported to older versions as well?

@phiSgr,
1.9.24 version released today contains the fix ✅
There are no plans to back-port it to all the previous versions; their usage is relatively low.
Thank you for your participation ❤


Connection aborts after firmware update

Since I updated my Raspberry (Linux 4.19.42-v7+ #1219; newest released kernel), the Miflora sensor is no longer accessible after some time. In the beginning, when I started the service, it works fine for some time, but after a while the Miflora sensor is no longer accessible. It looks similar to what @aymeric106 described in #80.
The sensor no longer returns any values. Even a restart of the service or even of the complete system does not result in the connection being restored.
By rebooting several times, connecting to the Android App, removing the battery, etc. I made it work again somehow. But it is not an unique problem. The battery is 99% full and it has happened several times after the update.
I still have no idea what it could be. Do any of you have a similar problem after upgrading the kernel?

I just noticed that if the sensor is not reachable again and I execute the following command, it returns an error message:
sudo hcitool lescan

LE Scan ...
62:2B:4F:27:9D:6C (unknown)
62:2B:4F:27:9D:6C (unknown)
C4:7C:8D:67:3A:CC (unknown)
C4:7C:8D:67:3A:CC Flower care
Disable scan failed: Input/output error

Hi
I have same issue but the Android app. I cannot bind the sensor since yesterday.
But my pi 0W still received data from sensor :/
Weird

Hi @Markkuuss,
I can confirm this behaviour, same kernel. I also rebooted, restarted and so on, but no difference – I only got this:
[2019-06-17 00:39:43] Initial connection to Mi Flora sensor "Konferenzpflanze" (C4:7C:8D:67:4C:9E) failed.

[2019-06-17 00:39:43] Announcing Mi Flora devices to MQTT broker for auto-discovery ...
[2019-06-17 00:39:43] Retrieving data from sensor "Konferenzpflanze" ...
[2019-06-17 00:40:13] Retrying ...
[2019-06-17 00:40:44] Failed to retrieve data from Mi Flora sensor "Konferenzpflanze" (C4:7C:8D:67:4C:9E), success rate: 0%

[2019-06-17 00:40:44] Sleeping (300 seconds) ...

Interesting: I have a running Home Assistant instance, miflora configured as a sensor; it stopped working, too – HA only reports messages like these:
2019-06-17 00:15:35 WARNING (MainThread) [homeassistant.helpers.entity] Update of sensor.orchidee_battery is taking over 10 seconds
2019-06-17 00:16:05 WARNING (MainThread) [homeassistant.helpers.entity] Update of sensor.orchidee_conductivity is taking over 10 seconds
2019-06-17 00:35:36 WARNING (MainThread) [homeassistant.helpers.entity] Update of sensor.orchidee_moisture is taking over 10 seconds

The iPhone App Flower Care detects this miflora device, connects it and retrieves the data! So what – at least for the moment – helped was this: placing miflora within spitting distance to RPi. I mean: not more than 10 or perhaps up to 20cm. So I finally got it working again (both miflora-mqtt-daemon and Home Assistant instance!).
Adding sensor to device list and testing connection ...
Name:          "Konferenzpflanze"
Internal name: "Konferenzpflanze"
Device name:   "Flower care"
MAC address:   C4:7C:8D:67:4C:9E
Firmware:      3.1.9
[2019-06-17 00:46:01] Initial connection to Mi Flora sensor "Konferenzpflanze" (C4:7C:8D:67:4C:9E) successful

[2019-06-17 00:46:01] Announcing Mi Flora devices to MQTT broker for auto-discovery ...
[2019-06-17 00:46:01] Retrieving data from sensor "Konferenzpflanze" ...
[2019-06-17 00:46:01] Result: {"moisture": 0, "light": 110, "battery": 100, "conductivity": 112, "temperature": 29.3}
[2019-06-17 00:46:01] publishing to mqtt topic "homeassistant/sensor/konferenzpflanze/state"

[2019-06-17 00:46:02] Sleeping (300 seconds) ...

That's really crazy. Does distance make a difference for you, too? I just started to search for 4.19.42-v7+, bluetooth and problem and there seem to be some... so probably it's a more general problem. But it's way too late now for further researching... :sleeping:
Cheers,
Marianne

Hey guys, Hey Marianna!
This is crazy! In general be aware that updating the RPi firmware (rpi-update) is not generally a good idea. It's recommended to stay with the default. Could you try reverting? I believe these instructions might help: https://www.raspberrypi.org/documentation/linux/kernel/updating.md

I have changed the firmware of my Raspberry back to the kernel with the last kernel version 4.14:
https://github.com/Hexxeh/rpi-firmware/commit/a08ece3d48c3c40bf1b501772af9933249c11c5b
So it works again.
The command for this is:
sudo rpi-update a08ece3
@ThomDietrich: But what happens if I download the current Raspbian Image? Then the default firmware has a kernel higher than 4.14 and Miflora won't work anymore?

I tested it again yesterday. With the current stable kernel 1.20190517, which comes with the standard Raspbian update/upgrade process (sudo apt-get dist-upgrade), miflora-mqtt-daemon no longer works.
The previous stable kernel version 1.20190401 (https://github.com/raspberrypi/firmware/releases/tag/1.20190401) is available with a downgrade via rpi-update:
sudo BRANCH=stable rpi-update 45a2e771e7272781e62ae92322734c2b90e0268a
So it works fine again.
@sysadmama: I didn't notice any difference in the distance. In the beginning it runs with the current kernel at exactly the same distance as with the old kernel. But if it runs longer, after a certain time no connection can be established. This is also reproduceable.

it looks like this is the problem why it is not working on my brand new raspberry pi 4...

it looks like this is the problem why it is not working on my brand new raspberry pi 4...

Probably that's the problem. Post your Linux version - the output of the command "uname -a".

i don't understand why, but it started working again... so forget my post...
[18:55:25] root@raspi4:/opt/miflora-mqtt-daemon# uname -a Linux raspi3plus 4.19.50-v7l+ #895 SMP Thu Jun 20 16:03:42 BST 2019 armv7l GNU/Linux

I had similar problem with not working connections or working for a while an then not anymore. Since the downgrade (like @Markkuuss) suggested, everthing seems fine. Will do further testing...

I am experiencing the same problem on a raspberry pi v3 with Stretch since a long time, but it seems to me that my kernel is older than the one reported here.
My kernel is
Linux raspberrypi2 4.14.79-v7+ #1159 SMP Sun Nov 4 17:50:20 GMT 2018 armv7l GNU/Linux
I implemented a rule in OpenHAB  (apt-get version 2.4 installed on raspbian stretch) that restarts bluetooth and miflora services.
I happens more ore less daily and it seems related to the weakness of the signal (with low batteries it occurs more often?)
I updated right now to the version suggested a few post above, and in my case it does not seem a downgrade, rather it is an upgrade.
Linux raspberrypi2 4.14.98-v7+ #1200 SMP Tue Feb 12 20:27:48 GMT 2019 armv7l GNU/Linux
I will see how it proceed and will report back

Let me add that I am experiencing the same issue (miflora not reporting status after a while) with an older version of the kernel on a rasberry pi v3 runinng with raspbian stretch
Linux raspberrypi2 4.14.79-v7+ #1159 SMP Sun Nov 4 17:50:20 GMT 2018 armv7l GNU/Linux
So far I implemented an OpenHAB rule (apt-get installed, version 2.4) that restarts the bluetooth service when the miflora does not give data for a prescribed time.
I updated the kernel with the one suggested in this thread (which is an upgrade in my case)
Linux raspberrypi2  4.14.98-v7+ #1200 SMP Tue Feb 12 20:27:48 GMT 2019 armv7l GNU/Linux
and will report back how it goes

in my case the daemon works on a RaPi with Raspbian (no OH installed) and kernel:
Linux raspberrypi 4.19.57-v7+ #1244 SMP Thu Jul 4 18:45:25 BST 2019 armv7l GNU/Linux
Success rate 100%.
the daemon does not work if activated on another RaPi with openhabian and kernel:
Linux openhab 4.19.66-v7+ #1253 SMP Thu Aug 15 11:49:46 BST 2019 armv7l GNU/Linux
the 2 RaPis (both are model 3b+) are positioned close to each other so position is not part of the problem.
I even tested switching the SD card of the two RaPi, and the behaviour becomes switched, so it does not depend on the hardware either.

Any news here? Has anyone made it stable with the latest firmware?
I'm afraid my attempts were all negative. After some time no more data will be read.

Any news here? Has anyone made it stable with the latest firmware?
I'm afraid my attempts were all negative. After some time no more data will be read.

Tried it on Pi 0, 3 and 4. Same issue. Its even the same in Node Red.

I have an error too. I installed this library first time
raspberry pi zero,
uname -a
Linux 4.19.75+ #1270 Tue Sep 24 18:38:54 BST 2019 armv6l GNU/Linux
cat /etc/*-release
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
...
Adding sensor to device list and testing connection ...
Name:          "flower_3"
[2020-05-02 19:19:45] Initial connection to Mi Flora sensor "flower_3" (C4:7C:8D:6A:65:12) failed due to exception:

[2020-05-02 19:19:45] Announcing Mi Flora devices to MQTT broker for auto-discovery ...
Traceback (most recent call last):
  File "miflora-mqtt-daemon.py", line 293, in <module>
    'sw_version': flora['firmware']
KeyError: 'firmware'

UPD. I found my mistake. I must run it with sudo!
Make errors clear, please! It took many hours of my life.
Any way, big thank you for your work!

Hey @a-x-
thanks for the exhaustive analysis. Let me just clarify that root privileges are not needed for the daemon. Also see the service definition: https://github.com/ThomDietrich/miflora-mqtt-daemon/blob/master/template.service#L8 I wonder if there were changes in buster leading to missing permissions, e.g. users not being added to the bluetooth group. Would you be willing to check this?
Quick note: The KeyError: 'firmware' was just fixed in https://github.com/ThomDietrich/miflora-mqtt-daemon/pull/118 but isn't really related to your issue.

Make errors clear, please!

I thought we already do that. Which error message is missing in your opinion? A pull request would be highly appreciated!

I had similar problem with not working connections or working for a while an then not anymore. Since the downgrade (like @Markkuuss) suggested, everthing seems fine. Will do further testing...

Were you able to find anything else in the meantime, @cropab and @milkpirate ?
I'm currently at kernel version 4.19.66-v7+ #1253. Unfortunately it doesn't work after a while. Then I have to do a reset every time: sudo hciconfig hci0 reset

I think I have now found the solution. With the firmware, which is currently distributed as a stable with Raspbian Stretch (1.20190819 / 4.19.66), the Bluetooth does not work correctly on my Raspberry Pi 3b+. The problem existed since the first update to 4.19.
I have now updated to the last available version 4.19.118 (1.20200512: kernel: Latest 4.19). Since then miflora runs stable again, at least it runs 24h without problems. This was unthinkable before.
The command for the update to the last 4.19 is
sudo rpi-update 866751bfd023e72bd96a8225cf567e03c334ecc4


Adds a "<> rdf:type memento:TimeMap" triple to the body of TimeMap (L…

Adds a "<> rdf:type memento:TimeMap" triple to the body of TimeMap (LDPCv - /fcr:versions)  resource.

JIRA Ticket: https://jira.duraspace.org/browse/FCREPO-3006

Other Relevant Links (Mailing list discussion, related pull requests, etc.)

What does this Pull Request do?
It simply adds the aforementioned triple to the timemap body by adding the required rdf:type to the jcr model via the mixin mechanism. It includes an integration test.
How should this be tested?
Create a resource
curl -u fedoraAdmin:fedoraAdmin http://localhost:8080/rest/test -X PUT

GET the timemap
curl -u fedoraAdmin:fedoraAdmin http://localhost:8080/rest/test/fcr:versions -X GET

Verify the <> rdf:type memento:TimeMap triple is present in the response.
Additional Notes:
Any additional information that you think would be helpful when reviewing this PR.
Example:

Does this change require documentation to be updated? No
Does this change add any new dependencies?  No
Does this change require any other modifications to be made to the repository (ie. Regeneration activity, etc.)?  No
Could this change impact execution of existing code? No

Interested parties
Tag (@ mention) interested parties or, if unsure, @fcrepo4/committers

This seems to be due to larger changes in Travis, I fixed this in a test branch by adding
dist: trusty

at the top of the .travis.yml file


Any word on continued support for downloading Silverlight plugin

The document gives a good explanation of the different applications supported and not supported by IE mode (Silverlight, Flash etc.).  However it does not mention whether the Silverlight plugin will still be installable once Silverlight reaches end of life.  IE mode means nothing if new PC's cannot download and install the Silverlight plugin because the installer is not available anymore.  Can this be clarified?

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

ID: 418ea780-28cd-c78d-8855-df5188dd4be9
Version Independent ID: 15be2361-06cf-5683-9213-8aec3aebef67
Content: About IE mode
Content Source: edgeenterprise/edge-ie-mode.md
Product: microsoft-edge
GitHub Login: @dan-wesley
Microsoft Alias: kvice

@cjlotz Hello - Based upon the Silverlight documentation (https://support.microsoft.com/en-us/windows/silverlight-end-of-support-0a3be3c7-bead-e203-2dfd-74f0a64f1788) the installer will not be available after the end of support date.

I would recommend reaching out to the Silverlight team for additional questions/clarification.  Thanks!


ROPC: suggest and review Blazor updates

Review changes in #33749

Associated WorkItem - 320926

Offline email, #33805,


Feature: Make Channel Mod modification easier

Description
Channel name, workspace, and display name should be sufficient to do the rest. This could even be an admin slash command or an interactive form.
Purpose
If something happens and we need more channel mods, we want this to be easy.

I'm assuming we can just scrap this feature as it didn't really take off?

Right now they just gave people admin. But I think in the future formal channel mods would be great.


SAMZA-2147: Improve the documentation of the samza release process.

Currently the RELEASE.md which outlines the samza release process in open-source has the following problems:

There is no clear separation between the steps that has to be followed before and after the successful vote completion.
There are formatting errors in the bash commands that has to be executed to validate the release.

This PR fixes the above problems.

@dxichen Please take a look when you have a chance.

@dxichen
Here is the screen shot of the RELEASE.md file rendered after this patch.


[Bug] Randomly misaligned Player [Introduced Dev 503, only random since 507]

At first I thought dev 507 has fixed #1712 but I noticed that it does still randomly appear. I'll call back if I find out any regularities.

Yeah facing same issue


Compatible Reader

Hi
I'm interested in playing with this. Knowing nothing about the technology I searched for an Android HCCB reader to try out your sample. I found the Microsoft Tag app, but this fails to recognise your sample HCCB - just keeps refocusing. It does recognise other (simpler?) codes (though resulting in a screen saying "this code is no longer active" message, then forwarding to ScanLife), such as this.
Do you have a reader that works with your codes?

Hi Sam, the reader may actually work, but because I haven't implemented the
ECC (error correcting code) portion, it probably can't establish a correct
read. I have not created a reader at this time.
On Sep 24, 2017 5:48 AM, "Sam Smith"<EMAIL_ADDRESS>wrote:

Hi
I'm interested in playing with this. Knowing nothing about the technology
I searched for an Android HCCB reader to try out your sample. I found the
Microsoft Tag app, but this fails to recognise your sample HCCB - just
keeps refocusing. It does recognise other (simpler?) codes (though
resulting in a screen saying "this code is no longer active" message, then
forwarding to ScanLife), such as this
https://en.wikipedia.org/wiki/High_Capacity_Color_Barcode#/media/File:High_Capacity_Color_Barcode_Tag.svg
.
Do you have a reader that works with your codes?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/canadaduane/elm-hccb/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAAgebr_S80CqnTnne5SArLT3ghk2haks5slkF_gaJpZM4Ph3LY
.

Any hopes for this? Pretty please?

I don't have the specs for HCCB. Does anyone know how it does ECC?
On Tue, Feb 5, 2019 at 4:00 PM stelonix<EMAIL_ADDRESS>wrote:

Any hopes for this? Pretty please?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/canadaduane/elm-hccb/issues/1#issuecomment-460838526,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAAgYULufqQ0clmGuxajtYhSePQIeDjks5vKg0JgaJpZM4Ph3LY
.

Is this still alive?

Not really. Feel free to use, adopt, or repurpose though.
On Sat, Aug 17, 2019 at 9:02 AM Donald Tsang<EMAIL_ADDRESS>wrote:

Is this still alive?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/canadaduane/elm-hccb/issues/1?email_source=notifications&email_token=AAAABAKSAYKXL5AR5LNSVV3QFAHHVA5CNFSM4D4HOLMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4QNIPQ#issuecomment-522245182,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAABAKBOJCB5CJ5GHINVZTQFAHHVANCNFSM4D4HOLMA
.

According to the pattern System and method for encoding high density geometric symbol set, I think they are using Reed-Solomon error correction and the correction is put at the head to the code


How to cancel authentication by a user during authentication and authorization flow

Test modules FAPIRWID2UserRejectsAuthenticationWithMTLS and FAPIRWID2UserRejectsAuthenticationWithPrivateKeyAndMTLSHolderOfKey requires cancellation of authentication by the user.

I found a solution. By enabling "Consent Required" in the client configuration, the user can cancel the authentication flow. I tried the test case with keycloak 10.0.1 and the latest conformance suite in this pull request: https://github.com/jsoss-sig/keycloak-fapi/pull/29
The results were all clear!! 🎉

@tnorimat Please close this issue.

@wadahiro Great! I'll close this issue.


/r/place

Dont really want to have another thing to moderate
If I was to do this it'd be some separate webapp, not a desktop icon here, it might conflict with the experience here if its just random squares.


BUG - Max/Min Value Increase causes dialog to close

Can you determine, which version was used?
Version: 1.3.39
Describe the bug
When altering the Maximum and Minimum Values of an Element's Output through the arrows, the dialog window gets closed after 2 clicks.
Additional context
If the solution is laborious, just kill those arrow buttons. Entering numbers works as well :)

fixed in 1.3.40-alpha.4


WT does not respect the initial rows and columns

By default I see that
"initialCols": 120,
"initialRows": 30,

but truly they are cols=129, rows=28. See
 I tested this also with the original profile.json.
May be you intend to include also the tabs row etc. but I think it is wrong. When one sets those parameters in Command prompt (that comes with Windows) the number of rows and columns is respected. See

The original profiles.json includes 8,8,8,8 padding on cmd.  Could this be the problem?

/dup #2061


[Feature Request] Add short names for CRDs

Description of problem
The names of CRDs in cartographer are quite long and typing them out constantly can be tedious.
Proposed solution
Add Short names to the CRDs of cartographer to allow for quick access to crds via easy to understand short names
Example
Clustersupplychain -> csc
Clustertemplate -> ct
Clusterimagetemplate -> cit
Clusterconfigtemplate -> cct
Clustersourcetemplate -> cst
Clusterdeploymenttemplate -> cdt
Clusterruntemplate -> crt
clusterdelivery -> cd
runnable -> run
Workload -> wld
Delivery -> dlv
Additional Context
The shorter names such as workload,runnable and deliverable are less important to have these but for the templates and supply chain it would enhance the UX greatly.

I love this idea. I feel we should check for collisions with popular or in-platform crd's and then proceed

to get the list of existing shortnames in a cluster, it is not a simple kubectl command as they dont support json output for the api-resources command but ive built i simple command with sed to do the same and ran this against a TAP cluster running on EKS so it has all the platform CRDs as well as some additional ones from EKS:
the command is:
kubectl api-resources --sort-by=name | awk '{print $1 "  " $2}' | sed '///d' -
on a TAP 1.0 on EKS with the Full profile installed the list is:
NAME  SHORTNAMES
accelerators  acc
apiexportrolebindings  aerb
apiresourceimports  ari
bindings  v1
builders  bldr,bldrs
builds  cnbbuild,cnbbuilds,bld,blds
certificaterequests  cr,crs
certificates  cert,certs
certificates  kcert
certificatesigningrequests  csr
channels  ch
clusterapigroupimports  agi
clusterbuilders  clstbldr,clstbldrs
clusterdomainclaims  cdc
clusterresourceexportmonitors  emon
clusterresourceimportmonitors  imon
componentstatuses  cs
compositecontrollers  cc,cctl
configmaps  cm
configurations  config,cfg
cronjobs  cj
customresourcedefinitions  crd,crds
daemonsets  ds
decoratorcontrollers  dec,decorators
deployments  deploy
domainmappings  dm
endpoints  ep
events  ev
events  ev
extensionservices  extensionservice,extensionservices
gitrepositories  gitrepo
helmcharts  hc
helmrepositories  helmrepo
horizontalpodautoscalers  hpa
httpproxies  proxy,proxies
images  img
images  cnbimage,cnbimages,img,imgs
ingresses  ing
ingresses  ing
ingresses  kingress,king
inmemorychannels  imc
limitranges  limits
namespaces  ns
networkpolicies  netpol
nodes  no
packageinstalls  pkgi
packagemetadatas  pkgm
packagerepositories  pkgr
packages  pkg
persistentvolumeclaims  pvc
persistentvolumes  pv
pipelineruns  pr,prs
podautoscalers  kpa,pa
poddisruptionbudgets  pdb
pods  po
podsecuritypolicies  psp
podtemplates  v1
priorityclasses  pc
replicasets  rs
replicationcontrollers  rc
resourceclaimpolicies  policies
resourceclaims  claims
resourcequotas  quota
revisions  rev
routes  rt
secretexports  se
secretimports  si
secrets  v1
securitygrouppolicies  sgp
serverlessservices  sks
serviceaccounts  sa
services  svc
services  kservice,ksvc
statefulsets  sts
storageclasses  sc
subscriptions  sub
tanzunetdependencyupdaters  dependencyupdater,dependencyupdaters
taskruns  tr,trs
tlscertificatedelegations  tlscerts
vspherebindings  vsb
vspheresources  vss
it does not seem to have any overlapping shortnames with platform CRDs nor have i come accross such short names but they may exist out in the wild.


fix opening the admin console

Add dedicated ways to open the admin console link for different operating systems (x-www-browser does not work on fedora). I reused the code from here https://github.com/SoMuchForSubtlety/f1viewer/blob/09b0db789bf62343a5628b2a4c1b8f1e54c97a50/internal/util/util.go#L14-L30
Also fixes the executable function to actually check the passed command

Thank you