[WIP] Added concurrent reading through SDK

Added an API  CarbonReader.split to enable concurrent reading of carbondata files through SDK.
List<CarbonReader> multipleReaders = CarbonReader.split(maxSplits)

For detailed information on how to use this API for concurrent reading, please refer ConcurrentSdkReaderTest.java
Performance Metrics:

configured table block: 1 MB
configured table block size: 10 MB
configured table block: 100 MB

# rows: 1e6Store: 7.6 MB
# files generated: 11Sequential Read: 274 msParallel Read: 123 ms
# files generated: 1Sequential Read: 247 msParallel Read: 248 ms
# files generated: 1Sequential Read: 252 msParallel Read: 254 ms

# rows: 1e7Store: 78 MB
# files generated: 104Sequential Read: 2685 msParallel Read: 1230 ms
# files generated: 9Sequential Read: 2499 msParallel Read: 1357 ms
# files generated: 1Sequential Read: 2527 msParallel Read: 2597 ms

# rows: 1e8Store: 865 MB

# files generated: 95Sequential Read: 27069 msParallel Read: 16082 ms
# files generated: 15Sequential Read: 25841 msParallel Read: 13256 ms

[ ] Any interfaces changed?
[x] Any backward compatibility impacted: No
[ ] Document update required?
[x] Testing done
- New unit test case have been added.

Can one of the admins verify this patch?

@NamanRastogi I think we can further optimize this function.

We can enable the parallel reading and set the parallelism while creating a CarbonReader;
Inside CarbonReader, we handle the concurrent processing;
The interfaces for CarbonReader should be kept the same as before, there is no need to add more interfaces. By calling hasNext or next, user can get the next record and will not care about which RecordReader does this record belong to.

The user interface looks like below:
CarbonReader reader = CarbonReader.builder(dataDir).parallelism(3).build();
while (reader.hasNext()) {
  reader.next();
}
reader.close();

To keep it simple, by default the parallelism can be 1 which means we will process each RecordReader one by one. Setting this parallelism to a higher value means that we will go process the RecordReaders in a thread pool with size 3.

@xuchuanyin
Since CarbonReader is iterator based, we can only read a line when user wants. So, even if we keep the parallelism internally before the build method, it will still read the file row by row (even though they are in different threads) and reading multiple rows will still be accessed sequentially. Please notice the test file ConcurrentSdkReaderTest.java, the reading is happening in the thread itself, and multiple threads are reading the files (line-by-line) concurrently.
The actual reading is happening inside the CarbonReader.readNext() method, as long as that is sequential, the actual reading performance is not going to get better, so we have to have CarbonReader.readNext() inside different threads,.

emm, but in your implementation, most of the work has to be done by the user (multi-thread handling). CarbonData itself only split the input data and return multiple readers. If this is the solution, why not just tell the user to generate multiple CarbonReaders by passing only part of the input dir each time they create the reader?
Addition to my proposal, I think we can add a buffer for the records. When CarbonReader.next is called, we can retrieve the record from the buffer and fill the buffer asynchronously. WhenCarbonReader.hasNext is called, we can first detect this from the buffer, if it is empty, we will then detect this from the recordReader and fill the buffer asynchronously.

@xuchuanyin
Using this API the overhead of reading concurrently is on the consumer of CarbonReader. So yes, what you said what you said is right. But this API was made to consider this case only when a user wants to have more manual control over the concurrent reading. Consider the scenario when the user wants to read different files in different machines. If we take care of the concurrent reading internally (using a buffer as you suggested), the user cannot do that.
Another thing that you mentioned was user providing the location of carbondata files in the SDK reader itself, and that does not work in the current implementation of SDK.
And for faster reading, there is one more pull request: 2816, it supports batch reading for a better reading performance. It is still not concurrent reading, but better than iterator based row-by-row reading.

@NamanRastogi Hi, customer required carbon provide the same order between using one thread to read and use multiple threads to read data.

Yes, data coming from one file will always be in order. Please check the split method, it splits the list of CarbonRecordReader into multiple CarbonReader s.
Suppose there are 10 carbondata files, and the user wants to get 3 splits, so he will get a list like this:

Please check the split method, it splits the list of CarbonRecordReader into multiple CarbonReaders. It does not jumble the order of CarbonRecordReader, it still keeps them sequential.
Suppose there are 10 carbondata files and thus 10 CarbonRecordReader in CarbonReader.readers object and the user wants to get 3 splits, so he will get a list like this:
CarbonReader reader = CarbonReader.builder(dataDir).build();
List<CarbonReader> multipleReaders = reader.split(3);

And the indices of CarbonRecordReaders in multipleReaders will be like:
multipleReaders.get(0).readers points to {0,1,2,3} indices of carbondata files
multipleReaders.get(1).readers points to {4,5,6} indices of carbondata files
multipleReaders.get(2).readers points to {7,8,9} indices of carbondata files
Now, if you read the rows like following code, the rows will still be in order.
for (CarbonReader reader_i : multipleReaders) {
    reader_i.readNextRow();
}

Earlier, you were getting data from 5th CarbonRecordReader only after you have exhausted the 4th. But now, you are getting it earlier, maybe even before 0th. So the user has to make sure he consumes it after he has used up the 4th file if order is important for him/her, otherwise he/she can use it earlier also if order is not important. So, for example to count the total no. of rows, user does not need the original order.

retest this please

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1244/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9509/

LGTM


Use shortcodes for adding files instead of adding code to markdown directly

What would you like to be added:
Add a shortcode that lets us add file content in markdown without having to copy code content in markdown.
Why is this needed:
With out this we will have to maintain the copy of code in multiple places.
Completion requirements:
Be able to include file instead of copying code
This enhancement requires the following artifacts:

[ ] Design doc
[ ] API change
[x] Docs update

The artifacts should be linked in subsequent comments.

/assign


Order choices by vote count

This also has a side-effect of increasing the performance since we're not performing a count query for each choice, but instead an aggregate query.

@kylef Who should review and give you the ok for this one?
Can you also rebase this?

Rebased.
I don't mind who reviews and merges, anyone who understands Python and cares enough to review :balloon:.


clientv3/yaml: change CAFile to TrustedCAfile

To be consistent with etcdmain? We're deprecating ca-file in favor of trusted-ca-file.

this breaks compatibility. we can add a field though...

@xiang90 Added TrustedCAfile field without breaking the compatibilities. PTAL.

lgtm


Ials improvement

Use the frequency based regularization

Codecov Report

Merging #89 (80cdec6) into main (5ce0b4b) will decrease coverage by 7.63%.
The diff coverage is 97.43%.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   94.66%   87.02%   -7.64%
==========================================
  Files          72       62      -10
  Lines        3690     3585     -105
==========================================
- Hits         3493     3120     -373
- Misses        197      465     +268


Q: Is it possible to query the state of other accessories through hc?

If I have other HomeKit enabled devices, would it be possible to query the state of those, and build new aggregated services from those devices?
Say I want to create a thermostat using the the already HomeKit connected temperature sensor in Elgato Eve Room, would that be possible via HomeKit or would I have to get data from the Bluetooth transfer directly?

No, HomeKit accessories don't know anything about other accessories.


Experimenting with adapting the linkerd labels module for general use

Heavy WIP. The linkerd module is good and has nice patterns, but it is not super focused to fitting our interfaces, plus it does not have a way to serialize the query string for the API so am trying to write that plus make the builder interface on expression more ergonomic. Stashing here as WIP, will maybe get to it on the weekend.

[x] Selector extend with stringifiers for ListParams
[x] Extend ListParams::labels_from taking a Selector
[ ] Builders for Expression (done 1)
[ ] Labels revamp

Allow converting from ResourceExt::labels() to something we can compare against
remove Labels altogether? commented out for now (arc use seems unnecessary for us)

[ ] combine Operator and Values? now there's the awkward representable case of In / NotIn but empty values...
[ ] Feature flag in jsonschema derives? If we want to embed selectors inside ObjectMeta.
[ ] Alternatively, more converters for selectors that can be injected into ObjectMeta

Maybe on the side note… I ended up implementing another version similar to linkerd in crust-gather: - https://github.com/crust-gather/crust-gather/blob/main/src/gather/selector.rs but skipping having a LabelSelector.
So this version also lacks conversion back to original LabelSelector and the use-case it was meant for is just an ability to deserialize the definition from a string, not a tuple of matchExpressions and matchLabels keys with values… I feel like the ideal state would be to leave on user to preserve the original LabelSelector, add a trait capable of matching the stuff over it, and a default implementation like the Selector presented in this PR + the ability to convert from LabelSelector to this custom Selector (like Into/From). WDYT?

So this version also lacks conversion back to original LabelSelector and the use-case it was meant for is just an ability to deserialize the definition from a string, not a tuple of matchExpressions and matchLabels keys with values… I feel like the ideal state would be to leave on user to preserve the original LabelSelector, add a trait capable of matching stuff over it, and a default implementation like the Selector presented in this PR + the ability to convert from LabelSelector to this custom Selector (like Into/From). WDYT?

Yeah, I also wanted a way to go from some arbitrary labels / ObjectMeta / K: Resource. Maybe some kind of SelectorExt trait would make sense here, as we could impl SelectorExt for ObjectMeta and hook in getters/setters that way. At any rate, do feel free to take this over. I have clearly not made a lot of progress on this issue in the last month.

#1539 is a successor PR. closing this.


Collating ideas from the meeting 5/31/2022 Meeting

I threw together this from the comments on the last meeting.  Everything is open for comment or criticism.  A couple of notes:

I think we are going to have to support #pending return types especially when we get eth and BTC integration, so I updated the variant there.
#noop is there in case a ledger wants to not charge a fee for an update call of some kind.  All should probably have a fee of some kind to prevent spamming.
I've obviously updated this with namespacing because I think it is a good idea. All the functions end in _ic1. What if you are implementing two standards on one canister that use decimals?  For example, a ledger that supports trading some kind of trade interface that has a decimals() call that returns the number of supported decimals in the price field.  Won't someone think of the decimals?
I've put a single principal to get the notification in there on transactions. If you leave it blank then no transaction. To do this we all need to agree on a standard function and payload. I haven't really looked at the block payload yet, but that probably deserves some debate as well.
Approving has a fee and suggests moving funds to a separate sub-account, although I think that could technically be an implementation detail, but I think it should be a best practice.
I considered changing the TOs and FROMS to a {#subaccount: {principal: Principal; subaccount: Blob;}; #account: Text} format, but held off for now as I'm not sure it is necessary.  Once we can dump subnet blocks people will be sifting through this info and I guess we should consider if we want that to be easy or hard.
Before I forget, we need a batch send/approve/transferFrom, but I haven't added it yet.

@skilesare I'm closing this as we moved on from the proposals


Examples/friend list

Why was this PR closed, @TylorS ?

It's just stale, and I think we can do it better

I have an idea :tongue:


[style]: Enhance Main Heading with Dynamic Hover Effect

What's the style idea?
The main heading on the homepage currently appears static and could benefit from a more engaging design. The goal is to implement a hover effect that captures users' attention and encourages interaction.
Proposed Changes:
Background Transition:

Add a background color or gradient transition that changes on hover.
Consider a shadow effect to create depth.

Kindly assign me this issue @SurajPratap10
Add screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@sanchitc05 GTG


[MM-25789] - Update default settings for new installations

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Summary
Adds the showing of tray icon and minimise icon to tray default options for Linux and Mac OS as well. For the variable minimizeTray I only saw that it's used in Linux context that's why I updated the default options directly. But if I missed something let me know.
Issue link
https://mattermost.atlassian.net/browse/MM-25789

/update-branch

Thanks @Willyfrog I think I understand the issue you mentioned now. Sorry for the confusion :(

seems like my editing caused a missmatch of number of spaces, which prevents linting from running properly, sorry about that


feat: bengali language

Maybe it would be better to not include the strings for which we don't have the bengali translation instead of spanish?

Maybe it would be better to not include the strings for which we don't have the bengali translation instead of spanish?

I think we should add a template, mastema who did this I think used the spanish one at the time as a template.

Maybe it would be better to not include the strings for which we don't have the bengali translation instead of spanish?

I think we should add a template, mastema who did this I think used the spanish one at the time as a template.

It's ok to leave out strings without translations. The English file should be used as the template, remove the strings with @. It's not super intuitive though but not sure if there's a better solution.


drivers/serial: stm32: LPUART max baud rate is 9600

LPUART instances do not support baud rates higher than 9600.
Return and error if configured with value higher than 9600.
Signed-off-by: Erwan Gouriou<EMAIL_ADDRESS>

Please do not merge yet, it seems limitation doe not apply in all cases.

Codecov Report

Merging #14198 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14198      +/-   ##
==========================================
- Coverage   52.68%   52.67%   -0.01%
==========================================
  Files         307      307
  Lines       45473    45473
  Branches    10530    10530
==========================================
- Hits        23956    23955       -1
  Misses      16640    16640
- Partials     4877     4878       +1

Impacted Files
Coverage Δ

arch/posix/core/posix_core.c
90.9% <0%> (-1.02%)
:arrow_down:

Continue to review full report at Codecov.

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

9600 bauds limitation is only valid when used LPUART clock is LSE, which is not the default case.
Will push and updated version of the patch when I have some time.

If you desire this fix in 1.14, please open or associate this PR with an Issue.

@erwango Please create a gitHub "bug" issue, labeled as "low priority", so it can be properly tracked.
I remove the "priority:low" label from here, as PRs priorities are not looked at by anybody following our current way of working.

Any plans here?

Closing. Will re-open when required


fix Issue 14197 - replace was removed from std.string

add public aliases to std.array.replace/replaceInPlace
std.string should at least cover any basic string
manipulation needs

Issue 14197 – "replace" was moved from std.string without alias added

thanks, this annoyed me last week as well.

Auto-merge toggled on


LISAMZA-27395 removing the current recursive call prevention logic

Issue:
We observed logging data loss happens in StreamAppender
The RC is the current implementation of the “append()” method uses one AtomicBoolean variable to detect if the thread is called recursively, which could lead to race conditions when "append()" is invoked by multiple threads and could cause log data loss.
we have reproduced the data loss issue locally in the multithread logging scenarios.
When the current implementation was introduced in samza, log4j1 framework didn't have recursive call prevention logic at the framework layer. More context can be found in SAMZA-723
However, the current implementation in samza has the side effect that data loss might occur in the parallel logging scenario. We believe it is a bug existing for a long time but never got noticed.
Change:
Samza now is upgraded to log4j2 framework which applies recursive call prevention at the framework layer
So we don't need recursive call prevention logic at Samza
API Changes:
Remove a metric recursiveCalls from StreamAppenderMetrics since it is no longer used
Test Done:
./gradlew build

Is the default support to prevent recursion not available when this change was introduced or did we upgrade to newer version of log4j that adds this prevention at the framework layer?

Is the default support to prevent recursion not available when this change was introduced or did we upgrade to newer version of log4j that adds this prevention at the framework layer?

log4j1 doesn't have this prevention that's when we added the change in samza. Later we upgraded to log4j2 which has he prevention at the framework layer.

@jia-gao As discussed offline, it would be great if we can add a unit-test for validating these changes in this patch.

validating

@jia-gao As discussed offline, it would be great if we can add a unit-test for validating these changes in this patch.

Added unit tests

Thanks for adding the tests @jia-gao . After going through the linkedin documents and JIRA tickets, it seems like we are just proceeding with the approach finalized as a part of the investigation.
We had just removed the recursive call check from the log4j2-appender implementation and we're relying upon native-recursive check from log4j2. Mostly the changes look good to me, let us wait for the build to pass and we can merge this in!


some markdown files are lost

can not open follow files:

3b_sefr-cut_pretokenize.md
4_run_mlm.md

1


⚠ Bofrost.lu has degraded performance

In 35b5aa4, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2077 ms

Resolved: Bofrost.lu performance has improved in 8ceed5a after 11 minutes.


Camera conflict

I'm connect two users in a room in my app
and camera conflict.
How to disable HMSVideoCapturer and only one user is allowed to use the camera?

@anvanvo Can you explain your use case a bit more?
At any moment you can join only one room although there can be multiple instances of the SDK


Add Pod PID Limit in Kubelet Configuration

What this PR does / why we need it:
This PR adds Pod PID Limit in Kubelet Configuration. We can Pass the CIS Benchmark 4.2.13 Ensure that a limit is set on pod PIDs (Manual)
Which issue(s) this PR fixes:

Fixes #3351
What type of PR is this?

/kind api-change
/kind feature
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

The Kubeone HostConfig support Pod PID Limits in its Kubelet Config

Documentation:

T.B.D

I built a Kubeone binary and created an AWS cluster to run CIS Benchmark:

/retest

/lgtm
/approve


Add police-force and police-neighbourhood to discovery

should this be police-force, not policy-force?

This looks fine but you will also need to create a PR for this bit - https://github.com/openregister/deployment#6-update-terraform-config. I'm happy to approve this one and you raise another, or you can add both to this PR and I will approve this one?

@michaelabenyohai I was going to create another PR but, everything is here now

Sorry! I will just approve next time!

@michaelabenyohai  No worries, it was a fair question :)


ci: add flake auto-update workflow

Just like everywhere else

Merging, because this is blocking #70
The workflow has been tested in multiple other repositories and there are no other changes
This should have been a shared workflow


Compile the test2

Compile the test2 please...

:) I have all week off now and it will be finished:)

On 24 Dec 2021, at 12:27, M-K-C @.***> wrote:
Compile the test2 please...
—
Reply to this email directly, view it on GitHub https://github.com/kapitainsky/RcloneBrowser/issues/193, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADXJHOWYPZ3I5TVUEV3THM3USRRJJANCNFSM5KWR7GUA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.

Do you have a date for the new release?

anytime before 4th of January when my holidays end:)

anytime before 4th of January when my holidays end:)

Just enjoy your holidays and forget about RB for awhile. You deserve it! Please don't let the entitled whiners make you feel guilty.

Thank you. I actually enjoy working on it:) And need to set some deadline for myself:)

I've been using RB and following this repo long enough now to know how dedicated you are. You do only work on RB in your spare time right?
I don't think many users really understand that updating and maintaining RB isn't your full-time job. Since I do understand, I know you're doing the best you can to juggle everything you have going on in your life, of which RB may only be a small part.
So give yourself a break. Spend time with your family or whatever, and come back from holiday refreshed and recharged and ready to finalize the new RB release asap.
Cheers!

The new release of Rclone Browser, I hope will come today!

I hoped too but still not happy with my testing. 1-2 days I should iron it. good is I have off all this week too so I can work on it.

Have you iron it, with your testning?
If not please release a compile beta tests release!

@M-K-C what OS you are using? I will compile beta for you

Windows x64
Linux x64
Linux arm (Raspberry Pi 3b+)
😉👍😉

haha - I will create BETA branch - so you and anybody else can compile what they need (RPi will be a bit more tricky as you have to compite Qt). Me personally I do not like "beta" experience. But let it be

@kapitainsky
Can't compile it self, got error on software needed to compile!!!Did you not write that you will compile to me or ???
But most l use Windows X64 or Linux X64

@kapitainsky
You wrote to me 2 days ago: what OS you are using? I will compile beta for you
I did not receive any compiled file from you, even though I wrote to you that I used Win11 x64 and Linux x64

@kapitainsky
I'd much rather see you focus all your attention on getting the final release ready as opposed to compiling beta versions for different OS's for one specific individual.
Just my personal opinion obviously. I don't mean for this to offend OP, but I suspect it will.

I have a great solution, just study and learn how to program and finish the job.
Success guaranteed. :)

I couldn't agree more.

I have a great solution, just study and learn how to program and finish the job.
Success guaranteed. :)

I couldn't agree more.²


Domain != Presentation

I really l liked your architecture - but intent of a domain is to be agnostic to outside world, and why do we have a key-word like json in the domain struct?
https://github.com/bagashiz/go-pos/blob/f17b11a84d98dd5c05a9f6fc788c2abfb722efa6/internal/core/domain/category.go#L7

Thanks for pointing that out, your feedback is much appreciated!
I initially added the json tag for potential API response handling but then I have since resolved this by creating a separate struct dedicated to API responses. I will remove it if it contradicts the domain's agnostic principle.


version 1.0 binaries?

When I go to https://cloudbase-init.readthedocs.io/ I see "cloudbase-init 1.0 documentation", but https://github.com/cloudbase/cloudbase-init/releases has no 1.0 version. What am I missing?

@rgl the update to show the correct version in the readthedocs page is under way:
https://review.opendev.org/#/c/694535/

@rgl the docs have been updated for the latest version (I cannot update the previous docs, because it would break the mutability of docs). https://cloudbase-init.readthedocs.io/en/latest/
When 0.9.12 version is released, the stable docs will change to the current latest docs.

@rgl this issue has been fixed by https://github.com/cloudbase/cloudbase-init/commit/e74570e01e219a7e246161ba76a3c53bb1a65293
If all good, can you please close this issue?


Cache update helpers

This is a first complete draft of the helpers for mutations and subscriptions. To test it, you can use this branch of the showcase application: https://github.com/aerogear/ionic-showcase/pull/171
Usage:
Mutations:
const helperOptions: MutationHelperOptions = {
  // see interface
};
const mutationOptions = createMutationOptions(helperOptions);
apollo.mutate<T>(mutationOptions);

The create mutation options will then create some useful stuff like the optimistic response, the update function for the cache etc.
Subscriptions:
There are two ways to use this. the createSubscriptionOptions file exports a function of the same name and it also exports subscribeToMoreHelper.
To use createSubscriptionOptions you can do the following:
const myObservableQuery = apollo.watchQuery<T>({options});
const options = createSubscriptionOptions({
// see interface
});
myObservableQuery.subscribeToMore(options);

This can be either run multiple times for different SubscriptionOptions such as create, update, delete etc. or you can use the subscribeToMoreHelper function. This function takes two parameters. One is the myObservableQuery from above and the other is an array of SubscriptionHelperOptions. This function calls subscribeToMore for each set of options it receives.

Since we moved towards the variable and doc for operation input we can also refactor 	optimisticResponseHelper to accept these types instead of strings.
Then we can simply create another wrapper method like offlineMutation() where we will supply update and optimisticResponse parameters that are generated.
UX FTW!

Verified on showcase using following cases:
Offline flow (when the optimistic response is added)

Creation
Updates
Deletion
App restart

Data replication

Creation
Updates
Deletion
App restart

It worked for all the cases apart from deletion. For deletion, the device did not actually added anything to offline queue. It may be related with some unrelated bug but is best to check that out.

@wtrocki this should be fixed now. It was the optimistic response checking the wrong operation types.


can't find ppn

Flutter says No file or variants found for asset: assets/android/pico%20clock_android.ppn.
how to create???

Please follow the issue template and provide enough information for us to understand the issue and assist. If you do that, re-open the issue and we'll continue.


chore) regenerate fake

approve

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is 85.71%.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   86.66%   86.66%
=======================================
  Files           1        1
  Lines          15       15
=======================================
  Hits           13       13
  Misses          2        2

Impacted Files
Coverage Δ

vos.go
86.66% <85.71%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Infinite backpack duplication glitch.

Dying while the backpack is in your inventory causes you to drop it but also keep it in the inventory upon death, meaning you can infinitely generate new backpacks as long as its in the backpack slot by just dying.

Missed this since this was in the repo for the old version, but this has been fixed and is now working in properly in 0.5-v2. (Still broken in 1.20.2-0.10-v2 if curios/trinkets is installed but port coming soon)


Fix version typo in eventcontext_v03.

Incorrect version in AsV1()

LGTM, thanks for seeing that.

Forgot to fix the signoff on this one, also found 2 more typos on master.

yikes, thanks.LGTM


Update type converters to treat emtpy string the same as null

Empty strings pass through to the base TypeConverter and for enumerable
types at least, this results in a NotSupportedException.
cc90af9 added support for accepting null values, and in pretty much
every case, empty string should be treated as null: no meaninful data
can be gleaned from it for conversion.
EnumConverter works without the check because the empty string gets
pased to the base ConvertFrom method which just returns the default enum
value. But we might as well be explicit about the expected return result
for empty strings.
Fixes #58

Merging this into my current branch will cause trouble if done automatically as I've deleted the ConverterHelper class in favour of a base TypeConverter. I'll merge things manually then close this. Thanks for the code! :+1:

Cool. Glad to help!


Animation Duration for animateTo

I would really like it if for each call to animateTo it was possible to set its own duration, which by default will be equal to the one specified in the AnimatedMapController, do you plan to implement this?

Hi the feature has been merged and is available on the main branch, will waiting for a new version on pub.dev you can point to this commit reference!: acdcf6bc9483a16193e0996238f637806c5625cd

Hi
I didn’t expect it to be so fast) Thank you!


RC1 calicoctl runs the wrong calico/node

The RC1 calicoctl runs the following when doing calicoctl node run:
Running command to load modules: modprobe -a xt_set ip6_tables
Enabling IPv4 forwarding
Enabling IPv6 forwarding
Increasing conntrack limit
Removing old calico-node container (if running).
Running the following command to start calico-node:

docker run --net=host --privileged --name=calico-node -d --restart=always -e CALICO_NETWORKING_BACKEND=bird -e NO_DEFAULT_POOLS= -e CALICO_LIBNETWORK_ENABLED=true -e ETCD_AUTHORITY=<IP_ADDRESS>:2379 -e ETCD_SCHEME=http -e ETCD_ENDPOINTS= -e NODENAME=drogon -v /var/run/calico:/var/run/calico -v /lib/modules:/lib/modules -v /var/log/calico:/var/log/calico -v /run/docker/plugins:/run/docker/plugins -v /var/run/docker.sock:/var/run/docker.sock calico/node:latest

So its running calico/node:latest.  Is that correct?  Should it not be a specific calico/node version?

@lwr20 Useful to include output from:
calicoctl node run --help
calicoctl version

OK looks like we aren't passing in the version environment to the containerized build.

$ ./calicoctl version
Version:      v1.0.0-rc1
Build date:   2016-12-01T06:00:06+0000
Git commit:   ab22efb

$ ./calicoctl node run --help
Usage:
  calicoctl node run [--ip=<IP>] [--ip6=<IP6>] [--as=<AS_NUM>]
                     [--name=<NAME>]
                     [--log-dir=<LOG_DIR>]
                     [--node-image=<DOCKER_IMAGE_NAME>]
                     [--backend=(bird|gobgp|none)]
                     [--config=<CONFIG>]
                     [--no-default-ippools]
                     [--dryrun]
                     [--init-system]
                     [--disable-docker-networking]

Options:
  -h --help                Show this screen.
     --name=<NAME>         The name of the Calico node.  If this is not
                           supplied it defaults to the host name.
     --as=<AS_NUM>         Set the AS number for this node.  If omitted, it
                           will use the value configured on the node resource.
                           If there is no configured value and --as option is
                           omitted, the node will inherit the global AS number
                           (see 'calicoctl config' for details).
     --ip=<IP>             Set the local IPv4 routing address for this node.
                           If omitted, it will use the value configured on the
                           node resource.  If there is no configured value
                           and the --ip option is omitted, the node will
                           attempt to autodetect an IP address to use.  Use a
                           value of 'autodetect' to always force autodetection
                           of the IP each time the node starts.
     --ip6=<IP6>           Set the local IPv6 routing address for this node.
                           If omitted, it will use the value configured on the
                           node resource.  If there is no configured value
                           and the --ip6 option is omitted, the node will not
                           route IPv6.
     --log-dir=<LOG_DIR>   The directory containing Calico logs.
                           [default: /var/log/calico]
     --node-image=<DOCKER_IMAGE_NAME>
                           Docker image to use for Calico's per-node container.
                           [default: calico/node:latest]
     --backend=(bird|gobgp|none)
                           Specify which networking backend to use.  When set
                           to "none", Calico node runs in policy only mode.
                           The option to run with gobgp is currently
                           experimental.
                           [default: bird]
     --dryrun              Output the appropriate command, without starting the
                           container.
     --init-system         Run the appropriate command to use with an init
                           system.
     --no-default-ippools  Do not create default pools upon startup.
                           Default IP pools will be created if this is not set
                           and there are no pre-existing Calico IP pools.
     --disable-docker-networking
                           Disable Docker networking.
  -c --config=<CONFIG>     Path to the file containing connection
                           configuration in YAML or JSON format.
                           [default: /etc/calico/calicoctl.cfg]

Description:
  This command is used to start a calico/node container instance which provides
  Calico networking and network policy on your compute host.

verified this is fixed in RC2


Server Overview Suite setup fail due to "Unknown Host" error

OpenBMC GUI :: Main initialization file for the test cases contained in this
==============================================================================
OpenBMC GUI.Test
==============================================================================
OpenBMC GUI.Test.Server Overview
==============================================================================
OpenBMC GUI.Test.Server Overview.Server Overview :: This test suite will va...
==============================================================================
Verify Title Text Content At OBMC Power Off State :: Verify displa... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
Verify Display Text Server Information At OBMC Power Off State :: ... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
Verify BMC Information Should Display At OBMC Power Off State :: V... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
Verify POWER Information Should Display At OBMC Power Off State ::... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
Verify High Priority Events Should Display At OBMC Power Off State... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
Verify High Priority Events Can Be Operated At OBMC Power Off Stat... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
Verify Launching Of Serial Over LAN Console At OBMC Power Off Stat... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
Verify High Priority Events Can Be Operated At OBMC Power Running ... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
Verify Launching Of Serial Over LAN Console At OBMC Power Running ... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host
------------------------------------------------------------------------------
OpenBMC GUI.Test.Server Overview.Server Overview :: This test suit... | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host

9 critical tests, 0 passed, 9 failed
9 tests total, 0 passed, 9 failed
==============================================================================
OpenBMC GUI.Test.Server Overview                                      | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host

9 critical tests, 0 passed, 9 failed
9 tests total, 0 passed, 9 failed
==============================================================================
OpenBMC GUI.Test                                                      | FAIL |
Parent suite setup failed:
herror: [Errno 1] Unknown host

9 critical tests, 0 passed, 9 failed
9 tests total, 0 passed, 9 failed
==============================================================================
#(IST) 2018/02/22 21:43:10.111234 -    0.571922 - OBMC_GUI Testing ==> [Finished]
====================================================================================================
OpenBMC GUI :: Main initialization file for the test cases contain... | FAIL |
Suite setup failed:
herror: [Errno 1] Unknown host

9 critical tests, 0 passed, 9 failed
9 tests total, 0 passed, 9 failed
==============================================================================
Output:  /Users/sivas/openbmctest21Feb2018/openbmc-test-automation/gui/OutputResult/output.xml
Log:     /Users/sivas/openbmctest21Feb2018/openbmc-test-automation/gui/OutputResult/log.html
Report:  /Users/sivas/openbmctest21Feb2018/openbmc-test-automation/gui/OutputResult/report.html  ```

we may have to validate if hostname is already there need to skip executing the keyword Get Hostname From Ip Address ${OPENBMC_HOST}

Full Name: | OpenBMC GUI
-- | --
Main initialization file for the test cases contained in this directory and setting up the test environment variables.
/Users/sivas/openbmctest21Feb2018/openbmc-test-automation/gui/obmc_gui
20180222 21:41:57.593 / 20180222 21:41:58.735 / 00:00:01.142
9 critical test, 0 passed, 9 failed9 test total, 0 passed, 9 failed
Suite setup failed: herror: [Errno 1] Unknown host

Good catch,  lets fix it up @sivassrr

This bug only on Mac OS but working fine for both Linux headless browser and Windows header browser with all type of browser (FireFox and Chrome)


Unexpected error & behavior when cancel a delay transaction using owner permission

Expected Behavior:
the owner should be able to cancel the delayed transaction.
Repro steps:

Originally, the account has two different keys for active and owner, one for each.

permission settings as follows:

account: aaa123aaa123
active '{
"threshold": 2,
"keys": [
{"key": "EOSXXXXXXXXXXXXXXXXXXXXXXXXXX", "weight": 1},
{"key":"PUB_R1_XXXXXXXXXXXXXXXXXXXXXXXX", "weight":1}],
"accounts": [],
"waits": [{"wait_sec": 120, "weight": 1}] }'
owner just one key: EOSYYYYYYYYYYYYYYYYYYYYYY
./cleos -u https://eos.greymass.com set account permission aaa123aaa123 active '{     "threshold": 2,      "keys": [         {"key": "EOSXXXXXXXXXXXXXXXXXXXXXXXXXX", "weight": 1},         {"key":"PUB_R1_XXXXXXXXXXXXXXXXXXXXXXXX", "weight":1}],      "accounts": [],      "waits": [{"wait_sec": 120, "weight": 1}] }' -p aaa123aaa123@owner

remove one active key from the wallet

./cleos wallet remove_key PUB_R1_XXXXXXXXXXXXXXXXXXXXXXXX

send a transaction
file send1eos.json:
{
"delay_sec": 600,
"actions": [{
"account": "eosio.token",
"name": "transfer",
"authorization": [{
"actor": "aaa123aaa123",
"permission": "active"
}],
"data": {
"from": "aaa123aaa123",
"to": "bbb123bbb123",
"quantity": "1.0000 EOS",
"memo": ""
}
}]
}
./cleos -u https://eos.greymass.com push transaction ./send1eos.json

try to cancel above transaction using different auth.

using owner permission to cancel the transaction,
./cleos -u https://eos.greymass.com system canceldelay aaa123aaa123 owner 12345txtxtxtxtxttxxtnoxxxxxxxxxxxxxxxxxxxxx
Error 3050000: Action validate exception
Error Details:
canceling_auth in canceldelay action was not found as authorization in the original delayed transaction

using active permission to cancel the transaction (one key)
Error 3090003: Provided keys, permissions, and delays do not satisfy declared authorizations
Ensure that you have the related private keys inside your wallet and your wallet is unlocked.
Error Details:
transaction declares authority '{"actor":"aaa123aaa123","permission":"active"}', but does not have signatures for it.

./cleos -u https://eos.greymass.com system canceldelay aaa123aaa123 active 12345txtxtxtxtxttxxtnoxxxxxxxxxxxxxxxxxxxxx

Add the other active key back to wallet and using active permission to cancel the transaction (two active keys included in the wallet)
Error 3090002: Irrelevant signature included
Please remove the unnecessary signature from your transaction!
Error Details:
transaction bears irrelevant signatures from these keys: ["PUB_R1_XXXXXXXXXXXXXXXXXXXXXXXX"]

Workaround:
Change the active permission to one key, and using active permission to cancel the delay transaction. (tried using owner permission, failed).

There is another way to work around this. Set keys with higher weight in active permission, and use active permission to cancel the transaction.

Has this issue solved? Please ask in eosio.stackexchange.com if you have further questions.


add kpanda offline upgrade

Check the preview: https://deploy-preview-2134--daocloud-docs.netlify.app/kpanda/intro/offline-upgrade


Allow setting labels on adapters

We have OPA policies in place for Deployments, Pods, and Services. I don't entirely follow the conversation in https://github.com/triggermesh/triggermesh/issues/705 but if this is already possible please let me know how.
If not possible it'd be great if that could be added. I had tried to add a label to GoogleCloudPubSubSource but I didn't see label transfer in the code like I did with the annotation for workload identity.

All correct, although thread in #705 is relevant to your question, the adapterOverrides implementation does not include adapter labels override at this moment. We could probably extend adapter overrides to include labels or just make adapter deployments inherit the parent object meta attributes.

@tzununbekov related: https://github.com/triggermesh/triggermesh/issues/768

I leave the implementation to you all obviously, but I do think the adapterOverrides is the more intuitive location.
I've recently started learning to program and I started with golang. I was reading some devs are against overloading context with too many values. Purely from wanting to learn, but if you do go with that solution for labels I'd like to hear thoughts on that statement.

"Label overrides" support added in #1278, and will be available in our next release. I'll post links to dev builds after we merge it with the upstream.
Regarding the context, I personally agree with the opinion that it should not be abused as a values storage, however, there are a lot of examples where this is practiced without any problems.

https://app.circleci.com/pipelines/github/triggermesh/triggermesh/3505/workflows/93ba47c9-99f0-4cc0-afc8-f2cc590112e2/jobs/9720/artifacts
@snuggie12 please let us know if this helped in your case

yeah that's working, @tzununbekov
thanks
do you have an expected release date?

Glad that it helped. I don't have the date for our next release, but you can join our slack where somebody from the team will help you out with that:
https://join.slack.com/t/triggermesh-community/shared_invite/zt-1kngevosm-MY7kqn9h6bT08hWh8PeltA

1.24 should come out on April 3rd, but we will see about doing a patch release next week if that can help you

@snuggie12 any thoughts on this fix https://github.com/triggermesh/triggermesh/issues/1299 that allows individual SA per adapter on the same namespace ?

this is now in v1.24 which is just released so I am closing it.
@snuggie12 @automaticgiant-ml let us know how it goes. Don't hesitate to reach out on Slack.

Cool, I'll give it a go soon. Thanks for all the hard work.


Check for stateless function before using ref

What's this PR do?
Adds a check for a stateless function component before applying the ref property in the bindInput method. The check is simply whether there is a render method in the current instance. Solution found here: https://stackoverflow.com/questions/41488713/react-how-to-determine-if-component-is-stateless-functional
Where should the reviewer start?

Check the bindInput method in FormContainer.tsx

How should this be manually tested?

Apply the bindInput method to a stateless function component.

What are the relevant tickets / issues?
#28

Right, I see the problem now that I've raised a PR. Seems we're retaining the ref in state but if we use a stateless function we won't have a ref which means we won't have it available in inputs.
The options I see are...

Remove inputs. Though this would also mean that the ref prop is now useless so it would also be scrapped.
Only use inputs for class components. This would mean we have an inconsistency where some inputs are available in inputs and some are not. I don't like this.
Force the inputs to only use class based inputs.

I'm leaning towards 1, but open to ideas.

have a HOC to wrap stateless ones, e.g. using recompose lifecycle

can this be an option?

Yeah, that's an interesting option. I had a quick go at implementing it but it didn't succeed. Will have to look into it further.

Codecov Report

Merging #31 into master will increase coverage by 0.21%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
+ Coverage    82.5%   82.71%   +0.21%
==========================================
  Files           4        4
  Lines          80       81       +1
  Branches       13       13
==========================================
+ Hits           66       67       +1
  Misses          9        9
  Partials        5        5

Impacted Files
Coverage Δ

src/FormContainer.tsx
84.31% <100%> (+0.31%)
:arrow_up:

Continue to review full report at Codecov.

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

What do you recommend for this: https://codeclimate.com/github/vitkon/form-container/pull/31
Seems like a valid duplication to me...


STYLE: Change some declarations to use trailing return type

This fixes some of the compile errors on VS2017.
They were introduced around the time C++17 was enabled. The first error message was:
m:\dashboard\itk\modules\numerics\statistics\include\itkHistogram.hxx(474): error C2244: 'itk::Statistics::Histogram<TMeasurement,TFrequencyContainer>::GetMeasurementVector': unable to match function definition to an existing declaration
m:\dashboard\itk\modules\numerics\statistics\include\itkHistogram.hxx(473): note: see declaration of 'itk::Statistics::Histogram<TMeasurement,TFrequencyContainer>::GetMeasurementVector'
m:\dashboard\itk\modules\numerics\statistics\include\itkHistogram.hxx(474): note: definition
m:\dashboard\itk\modules\numerics\statistics\include\itkHistogram.hxx(474): note: 'const Histogram<TMeasurement,TFrequencyContainer>::MeasurementVectorType &itk::Statistics::Histogram<TMeasurement,TFrequencyContainer>::GetMeasurementVector(const itk::Statistics::Histogram<TMeasurement,TFrequencyContainer>::IndexType &) const'
m:\dashboard\itk\modules\numerics\statistics\include\itkHistogram.hxx(474): note: existing declarations
m:\dashboard\itk\modules\numerics\statistics\include\itkHistogram.hxx(474): note: 'const Histogram<TMeasurement,TFrequencyContainer>::Sample<itk::Array<TValue>>::MeasurementVectorType &itk::Statistics::Histogram<TMeasurement,TFrequencyContainer>::GetMeasurementVector(const itk::Statistics::Histogram<TMeasurement,TFrequencyContainer>::IndexType &) const'
m:\dashboard\itk\modules\numerics\statistics\include\itkHistogram.hxx(474): note: 'const Histogram<TMeasurement,TFrequencyContainer>::Sample<itk::Array<TValue>>::MeasurementVectorType &itk::Statistics::Histogram<TMeasurement,TFrequencyContainer>::GetMeasurementVector(Histogram<TMeasurement,TFrequencyContainer>::Sample<itk::Array<TValue>>::InstanceIdentifier) const'

PR Checklist

[x] No API changes were made (or the changes have been approved)
[x] No major design changes were made (or the changes have been approved)

Thanks @dzenanz, for doing a follow-up to PR #2780 commit c54ad6715a68570f9db14f002c239ec7ea06383d
Did you do same the regular expression as in c54ad6715a68570f9db14f002c239ec7ea06383d ?
Find what: ^typename (\w+<.+>::)(.+)\r\n\1(.+)\r\n{\r\n
Replace with: auto\r\n$1$3 -> $2\r\n{\r\n
Filter: itk*.hxx
[v] Match case
(*) Regular expression

Honestly I don't really understand why theses errors reappear now, as I thought they were solved by PR #2780.

They were introduced around the time C++17 was enabled

Thanks but that was already 3 weeks ago, with pull request #3969 commit 513507e4c70172d5431563f89dc77c6bae1ad5b1 How frequently is VS2017 still being tested? And is there a reason to assume that the compile errors are related to enabling C++17?

No regex, I went by compiler error messages. VS2017 is tested nightly, and the errors started appearing on April 1st. I looked at the log and that was about time we flipped the switch on C++17, and you did a first round of refactoring. I did not narrow it down to a specific commit, though.

Locally tested: those VS2017 compile errors do indeed appear with pull request #3969 commit 513507e4c70172d5431563f89dc77c6bae1ad5b1, the upgrade to C++17 😢
Nevertheless it's just fine to use trailing return types in the cases addressed by your PR. It is also in accordance with the ITKSoftwareGuide:

https://github.com/InsightSoftwareConsortium/ITKSoftwareGuide/pull/162

OK, let's merge this one. The remaining use cases for trailing return types can always be addressed afterwards. Thanks @dzenanz

Follow-up in #4026.


Generalize CartesianGrid for multiple dimensions and variable cell lengths

This is false advertising.  You are doing more than moving things around.  You did changes to the class.  Please provide a description.

This is false advertising. You are doing more than moving things around. You did changes to the class. Please provide a description.

Sorry, did not notice it took the title from the first commit.

@dalg24 OK, I templated the CartesianGrid on the dimension instead of the Box. The PR had to be expanded to also include DBSCAN working with different Box types (new and old geometries). But everything works now. Please re-review

I'll need to run some DBSCAN experiments to make sure no performance penalty occurred.

Confirmed no performance penalty running few DBSCAN and MST experiments using master (aca4aba5) and this branch (b23049cd) on OACISS Saturn (Nvidia A100).
Merging with HIP skipped (not started due to timeout).


Readme should point to releases instead of dev versions

What do you think?

ye ig godo.

ye ig godo.

Ok, making a PR

Ok, making a PR


make update_python_requirements.bsh support binary only packages not on pypi

With the update_python_requirements.bsh:PIP_BINARY_ONLY env variable, it is possible to specify which packages from pypi should be treated as binary only. I suggest extending this to support packages from other locations (e.g., PyTorch).

tl;dr
This is more synonymous to how some packages need numpy installed before installing. So it's up to the user to install torch first before calling the update script. Perhaps adding it to a virtualenv like you would numpy would work?
RUN virtualenv /opt/venv; \
    PS1= source /opt/venv/bin/activate; \
    pip install --no-cache-dir setuptools==35.0.2 pip==9.0.1; \
    pip install --no-cache-dir numpy==1.12.1

PIP_BINARY_ONLY is a pip feature, not vsi_common feature.
The intended use of update_python_requirements is to update the python requirements.
So if I said I need "scipy=0.19.1" as a requirement, then I want to update all the requirements to the latest versions of the package, say "scipy=1.0.0". This update process is done by asking pypi via pip what are the latest versions.
For packages such as pytorch, that are not download from a repository like pypi, rather a simple url, asking the question "What is the newest version" does not compute. This is why a package like pytorch's url should be added to the custom requirements file.
http://download.pytorch.org/whl/cu80/torch-0.3.1-cp36-cp36m-linux_x86_64.whl

Now packages like torchvision are a problem, as the are in pypi, and require torch, which is not. This is essentially a broken package. For this reason, you can not run
pip download -d /tmp/foo torchvision

as some torch-0.1.2.post1.tar.gz release will be downloaded, and have setup.py run, which subsequently fails by design.
Without having manually installed torch first, there's just no other way, so barring a better solution, torchvision would need to be added as a custom package too, as it is indeed broken
The correct thing to do is to have torchvision remove the requirement for torch


[question] would you provide example configs of the preview screenshot?

Let's say, I'm more nosy than lazy, (and my grandma has no time, btw).
But I can't imaging what functionality the first three widgets (after the username), at the top bar of your screenshot could have, so I just kindly wanted to ask, if you're possibly willing to share that configs as a bit more extended examples?
As some kind of 'proof' that I not just want to use your configs, I made a screenshot of my current bar config:

If you insist, I could upload my configs here, and we could do some kind of swap, if you like 😉
PS: Completely off topic, but I just wanted to let you know, that there seems to be a (Wayland targeted and GTK using) Hybridbar, already - if you shouldn't be be aware of it, already; and even if I know that:
HybridBar =! Hybridbar ✌

Yeah sure, I'll attach it below.
Here's how it's structured:

Username
Hyprland: Current Workspace / Max Workspaces
Hyprland: Currently focused window
Cava
Speaker Volume
Current Time
Custom hot-reload

config.json
{
    "hybrid": {
        "cava_sed": "s/;//g;s/0/▁ /g;s/1/▂ /g;s/2/▃ /g;s/3/▄ /g;s/4/▅ /g;s/5/▆ /g;s/6/▇ /g;s/7/█/g;",
        "stylesheet": "style.css",
        "cava_framerate": 360,
        "expand_right": false,
        "expand_left": false,
        "update_rate": 69,
        "cava_bars": 7,
        "r": 10,
        "g": 10,
        "b": 10,
        "a": 0.5
    },
    "variables": {
        "@reload": "killall -I hybrid-bar -9 ; hyprctl dispatch exec hybrid-bar ; hyprctl dispatch exec \"HYBRID_CONFIG=config_bottom.json hybrid-bar\"",
        "@get": "jq -r",
        "$+s": "<small>",
        "$-s": "</small>"
    },
    "left-label_username": {
        "text": "  visions",
        "tooltip": "$+sCurrent Username$-s"
    },
    "left-spacing_separator": {
        "spacing_start": 5,
        "spacing_end": 7
    },
    "left-label_current_workspace": {
        "text": " 𧻓 ",
        "command": "hyprctl monitors -j | @get '.[].activeWorkspace.id'",
        "tooltip": "$+sCurrent Hyprland Workspace (0-10)$-s"
    },
    "left-label_max_workspaces": {
        "text": "/10"
    },
    "left-spacing_1": {
        "spacing_start": 5
    },
    "centered-label_active_window": {
        "text": "  ",
        "command": "hyprctl activewindow -j | @get '.title'",
        "tooltip": "$+sCurrently focused window$-s"
    },
    "centered-spacing_2": {
        "spacing_start": 5
    },
    "right-spacing_3": {
        "spacing_start": 5
    },
    "right-box_4": {
        "?": "This exists because spacings didn't work for this use-case.",
        "width": 5
    },
    "right-cava_5": {},
    "right-spacing_separator_vol": {
        "spacing_start": 5,
        "spacing_end": 7
    },
    "right-label_volume": {
        "text": "  ",
        "command": "echo $(pactl get-sink-volume @DEFAULT_SINK@ | rg -o '[0-9]{1,3}%' | head -n 1 | cut -d '%' -f 1)%",
        "tooltip": "$+sVolume percentage (%)$-s"
    },
    "right-spacing_separator": {
        "spacing_start": 5,
        "spacing_end": 7
    },
    "right-label_time": {
        "text": "  ",
        "command": "date +%H:%M",
        "tooltip": "$+sCurrent hour:minute$-s"
    },
    "right-spacing_separator_ending": {
        "spacing_start": 5,
        "spacing_end": 7
    },
    "right-button_reload": {
        "text": "",
        "command": "@reload",
        "tooltip": "$+sReload$-s"
    }
}

style.css
/* Label Color. Config default is white. */
label {
    color:         white;
}

#root {
    border-bottom: 1px solid;
}

#left {
    border-right: 1px solid;
    border-left: 1px solid;
}

#right {
    border-left: 1px solid;
    border-right: 1px solid;
}

#separator, #separator_ending, #separator_vol {
    background-color: white;
}

/* Button Configuration */
button {
    font-weight:    normal;
    box-shadow:     none;
    margin-bottom: -8px;
    margin-top:    -8px;
    padding-right:  0;
    padding-left:   0;
}

PS: Completely off topic, but I just wanted to let you know, that there seems to be a (Wayland targeted and GTK using) Hybridbar,
Yeah I noticed this after seeing "Hybridbar" listed on GitHub a few weeks ago. Although that project seems dormant

Wow, thanks a lot for that (very structured) inspirations. You showed me yours, I'll show you mine 😉
-> configs.zip

which font is it?

Wow, thanks a lot for that (very structured) inspirations!
You showed me yours, I'll show you mine 😉 -> configs.zip
Preview:
Structure (from left to right):

[right] Hyprland dynamic workspace number list, with active workspace in square brackets
[centered] Hyprland window titles, wich trims static browser names at the end, (e. g.: " - Mozilla Firefox")
'# CPU usage (with fixed length of 2 numbers), followed by system temperature (°C) in brackets
Button_command: opens htop | Tooltip_command: shows the 10 most active processes
☰ Memory used
Tooltip_command: details about memory usage
⛁ Disk used
Tooltip_command: details about disk usage
⚟ Volume label (toggles if muted), followed by audio volume
∅ Microphone volume label (toggles if muted), followed by mic volume
⚙ run audio management (pavucontrol in this case)
直 run wifi stats (wavemon in this case), followed by label wich toggles depending on wifi connection status
𧻓 Hyprland workspace count
Tooltip_command: active windows count
祥 run calendar (cal), followed by time
Tooltip_command: date
⏻ run power menu (wlogout in this case)

Hope this helps someone 🤞

which font ??

which font ??

Oof, it's been a long time... But I'm pretty sure if was one of them:

Nerd Fonts (FiraCode or DroidSansMono)
Noto Fonts

Thanks

which font ??

Oof, it's been a long time... But I'm pretty sure if was one of them:

Nerd Fonts (FiraCode or DroidSansMono)
Noto Fonts

Thanks for the reply, anyways I've left Wayland and shifted to xorg Xmonad my long time fav twm


Enzyme Internal Error with setupTests.js Adapter Not found

Bug related to running TEST with create-react-app

Though Im using React 16, I get following error:
Enzyme Internal Error: Enzyme expects an adapter to be configured, but found none. To
          configure an adapter, you should call `Enzyme.configure({ adapter: new Adapter() })`
          before using any of Enzyme's top level APIs, where `Adapter` is the adapter
          corresponding to the library currently being tested. For example:

          import Adapter from 'enzyme-adapter-react-15';

I have tried deleting node_modules & fresh run npm install, just to be safe.
Package.json
https://pastebin.com/uECbKXR7
Environment
node version: v9.10.1
npm version: 5.6.0
yarn version: 1.5.1
os: MACOS
src/setupTests.js
import * as enzyme from 'enzyme';
import * as Adapter from 'enzyme-adapter-react-16';

enzyme.configure({ adapter: new Adapter() });

Steps to Reproduce
yarn test
Im using CI=true otherwise I get following error
➜  order-web-2 node scripts/test.js --env=jsdom
2018-04-03 11:57 node[7470] (FSEvents.framework) FSEventStreamStart: register_with_server: ERROR: f2d_register_rpc() => (null) (-22)
2018-04-03 11:57 node[7470] (FSEvents.framework) FSEventStreamStart: register_with_server: ERROR: f2d_register_rpc() => (null) (-22)
2018-04-03 11:57 node[7470] (FSEvents.framework) FSEventStreamStart: register_with_server: ERROR: f2d_register_rpc() => (null) (-22)

Project structure

You ejected your app. In this case if you didn’t already have src/setupTests.js it won’t be automatically configured for you.
Please follow Enzyme documentation: https://github.com/airbnb/enzyme/blob/master/docs/guides/jest.md


Close the pyodbc Google Group?

Just FYI, the pyodbc Google Group is still up-and-running:
https://groups.google.com/forum/#!forum/pyodbc
Perhaps it's time to retire it?  It's rarely used these days and I'm thinking it would better if people raised issues here on Github.  I don't know who the administrator for it is though.

Good idea.  I've posted a message pointing people here.  I could not find a way to archive it, so I simply set it so only owners can post.  Thanks.


Missing command: "gort role info"

We could use a gort role info command, which displays information about roles,. including (but not necessarily limited to):

Role name
Permissions granted
Groups added to the role, maybe?

This actually requires a bit more work on the DAL to implement than I thought.


404 error

[Enter feedback here]
The link under automatic mdm enrolment is broken

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

ID: 74d09c81-d205-dbd0-ea6a-aaf469eadf5c
Version Independent ID: 458db848-1b30-1115-8814-3a639a57c70b
Content: Configure Device-based Conditional Access on-Premises
Content Source: WindowsServerDocs/identity/ad-fs/operations/Configure-Device-Based-Conditional-Access-on-Premises.md
Product: windows-server
Technology: identity-adfs
GitHub Login: @billmath
Microsoft Alias: billmath

Specific details: Paragraph section Automatic MDM enrollment

https://docs.microsoft.com/windows-server/identity/ad-fs/operations/configure-device-based-conditional-access-on-premises#automatic-mdm-enrollment

Automatic MDM enrollment
To enable automatic MDM enrollment of registered devices so that you can use the isCompliant claim in your access control policy, follow the steps here.
Broken URL: https://blogs.technet.microsoft.com/ad/2015/08/14/windows-10-azure-ad-and-microsoft-intune-automatic-mdm-enrollment-powered-by-the-cloud/

From experience, most of the TechNet blog pages are now hosted on a slightly different domain structure:

https://techcommunity.microsoft.com/t5/custom/page/page-id/Blogs

So after doing a quick search, based on this knowledge, the working, up-to-date blog link URL is this one:

https://techcommunity.microsoft.com/t5/azure-active-directory-identity/windows-10-azure-ad-and-microsoft-intune-automatic-mdm/ba-p/244067

I will get back to this later today and open a Pull Request to have this link URL updated.

Link corrected from
https://blogs.technet.microsoft.com/ad/2015/08/14/windows-10-azure-ad-and-microsoft-intune-automatic-mdm-enrollment-powered-by-the-cloud/
to https://docs.microsoft.com/en-us/windows/client-management/join-windows-10-mobile-to-azure-active-directory
in commit https://github.com/MicrosoftDocs/windowsserverdocs/commit/60b7e505e0580ba8eeff6ee96c9172e8c42ed630 as of July 22, 2020 (direct file link https://github.com/MicrosoftDocs/windowsserverdocs/commit/60b7e505e0580ba8eeff6ee96c9172e8c42ed630#diff-6df6593c8c6582bf1db0a1347e16f2c57a2e89703ed6233d83d3de2512a7f5c2).

#please-close


New Python binding

Since the legacy binding is deprecated, needs a new binding

I can potentially create the python binding using either PyO3 or RustPython if you're ok with that 🙂 .

I just read this link: https://github.com/PyO3/pyo3/issues/435, and I think your disclosure is right. Let's use PyO3.

Would be happy to help if the work starts regarding the python binding implementation ! (contact<EMAIL_ADDRESS>

Hi - does anyone know if there is any progress on this issue?  @vincentdchan @DataHearth


Allow compiling on Android (Termux)

Currently when trying to build on Android (using Termux) it fails because auth_client cannot be found. This fixes that problem

Thanks for the patch (and for testing on Android!)
Looks good to me.

Needs cargo fmt:
 --- a/src/kbs2/agent.rs
+++ b/src/kbs2/agent.rs
@@ -205,10 +205,7 @@ impl Agent {

     // TODO(ww): These can be replaced with the UnixStream.peer_cred API once it stabilizes:
     // https://doc.rust-lang.org/std/os/unix/net/struct.UnixStream.html#method.peer_cred
-    #[cfg(any(
-        target_os = "linux",
-        target_os = "android",
-    ))]
+    #[cfg(any(target_os = "linux", target_os = "android",))]
     fn auth_client(&self, stream: &UnixStream) -> bool {
         use nix::sys::socket::getsockopt;
         use nix::sys::socket::sockopt::PeerCredentials;

Thanks again!


Showcase Submission for challenges/182-apollonian-gasket

Thank you Thanos Psaridis for your contribution! A member of the Coding Train team will review it shortly.

Apollonian Gasket with a twist
Thanos Psaridis

lovely!


Visualise a module with the plan task

Bring back the capability to visualise a module based on the new api.

Closed with #212


Update Hero section Figma design As per round 1 feedback

Summary
Design the Hero Section for the 'Le Jardin' website landing page on Figma, incorporating changes based on feedback received during Design Testing Round 1.
Acceptance Criteria:

Feedback Implementation:

Implement changes as per the feedback provided by manual design testers.

Web Accessibility:

Follow the Web Content Accessibility Guidelines (WCAG) to ensure the website is accessible to all users.

Visibility Enhancement:

Make the Hero text more visible.

Header Text Optimization:

Reduce the number of words in the header text.

Component Library:

Utilize a Component Library like MUI and NextUI for a consistent and effective design.

Brand Archetype:

Adhere to the Brand Archetype persona 'Explorer' in the Hero Section design.

Color Selection:

Use the Adobe color wheel for selecting colors for the website components. Refer to Adobe Color Wheel.

Note: Ensure that the redesigned Hero Section not only addresses the feedback but also aligns with accessibility standards, the brand identity, and provides an engaging and visually appealing user experience.

Updated design:

figma link: https://www.figma.com/file/puog0pDY0mW5VGzuivGCi1/Final-Group-Project--Bakery?type=design&node-id=1%3A2&mode=design&t=7Ie8TMMu8EvLugrn-1


Accessing auto generatedprops.className inside styled() template

const StyledDiv = styled.div`
  ${(props) => `
    .${props.className}-some-element {
      ...
    }
  `}
`

At the moment the className prop is undefined

Please open at the appropriate typing repo, we don’t maintain the types.
On Mon, Oct 21, 2019 at 6:16 PM Kilian Ciuffolo<EMAIL_ADDRESS>wrote:

const StyledDiv: React.FC<{ className?: string }> = styled.div ${(props) =>    .${props.className}-some-element {      ...    }  }
At the moment the className prop is undefined
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/styled-components/styled-components/issues/2825?email_source=notifications&email_token=AAELFVTEUVOOVI5ZYVEXWS3QPYS3LA5CNFSM4JDGGOTKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HTK6A2A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAELFVSVOJN27GAKP7G6Z7TQPYS3LANCNFSM4JDGGOTA
.

@probablyup I removed the types, but the question has nothing to do with typescript!

This seems to work: https://codesandbox.io/s/styled-components-using-classname-prop-dynamically-7dkeh

@ali's example works... what exactly are you trying to do @kilianc?

Where is this documented? And my example was with a custom component. Let me see if I can repro.

I can confirm this does not work with custom components: https://codesandbox.io/s/styled-components-using-classname-prop-dynamically-stqcb

I think the confusion is that an interpolated function only receives the direct prop values of the rendered component, and it doesn't receive the generated className for the instance of the styled component. Here's a Sandbox that shows that behavior more directly: https://codesandbox.io/s/styled-components-using-classname-prop-dynamically-8ky16
From my perspective it seems like the behavior is working as intended, but I'm merely a user of styled-components and not a maintainer :)

@ali I think there is a thin line, the balance between using CSS as they were supposed to be used and everything components. It is quite impractical that I have to create a component to set a margin or a couple of properties. This is particularly annoying when prototyping and changing styles around. Passing inline styles seems to be the current best approach to this but looking at this beautiful API that we have, honestly: why not?
Every className that starts with a dash can be scoped using the generated className as prefix. I am sure it gets complicated when there are more than one className attached but the last "generated" one should work.

@kilianc if you want to make a couple adjustments but not a whole component, that's the general idea behind the "css prop". https://www.styled-components.com/docs/api#css-prop

@probablyup very close, but still not ideal, as it couples the structure to the styles. Is it possible to pass a new prop called styledClassName (or something similar) to the template? This would unlock few patterns and best practices that I think are valuable.
const Picture = ({ className }) => (

    <div className={`${className}-foo`}>hi!

)

const scn = (className) => ({ styledClassName }) => `${styledClassName}-${className}`

const StyledPicture = styled(Picture)`
  .${scn('foo')} {
    /* this is isolated now */
  }
`

See also https://github.com/styled-components/styled-components/issues/2113


[18.09] Bump Golang 1.10.7

go1.10.7 (released 2018/12/14) includes a fix to a bug introduced in Go 1.10.6
that broke go get for import path patterns containing "...".
See the Go 1.10.7 milestone for details:
https://github.com/golang/go/issues?q=milestone%3AGo1.10.7+label%3ACherryPickApproved

@tiborvass @andrewhsu


Error: ENOENT: no such file or directory, open 'E:\Projects\Code-Crafters\node_modules\timeago.js\src\format.ts'

How to solve this error show on terminal :
WARNING in ./node_modules/timeago.js/esm/format.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from 'E:\Projects\Code-Crafters\node_modules\timeago.js\src\format.ts' file: Error: ENOENT: no such file or directory, open 'E:\Projects\Code-Crafters\node_modules\timeago.js\src\format.ts'

In my case, I resolved it by adding GENERATE_SOURCEMAP=false to the .env.local file and then restarting the server.

@kimch321 it works, thanks, but why is this required?


OC-493 ~ Fixes protocol for new devices

 You should sign our Contributor License Agreement in order to get your pull request merged.

SonarQube analysis reported 5 issues

 5 minor

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 OsgpAdminClientSoapService.java#: Replace all tab characters in this file by sequences of white-spaces.


[Task]: Stabilize self hosted runners

What needs to happen?
Beam currently uses dynamically provisioned runners on top of K8s, and they have significant stability issues. Replace dynamic runners with static VMs hosting runners or a more stable solution. Ensure that all workflows are stable and correct with the static runners.
Issue Priority
Priority: 2 (default / most normal work should be filed as P2)

.take-issue


socket: too many open files

I'm having an issue - the cachet-monitor seems to leave the sockets opened.
At the beginning everything works fine but the number of used file descriptors constantly increases till it reaches the soft limit (1024) then I'm getting an error:
... level=warning msg="Could not log metric! ... socket: too many open files
Thank you for your time,

I have the same issue, somebody, please comment about it.

same problem. It seems it never closes the socket when it checks.

Probably related to https://github.com/CastawayLabs/cachet-monitor/blob/0e93d140e87a52a84218171cd7e94496caa42916/http.go#L63

I'm seeing this issue again using the per-compiled binarys (cachet_monitor_linux_amd64 and cachet_monitor_linux_386) with v3.0 running on dreamhost.

Same issue here

Me too

Oh I see, the binary that has this issue fixed wasn't released......

Can anyone help me build the cachet-monitor binary to include the latest commits in master to fix this issue?

I added to the crontab a restart of the service each night, that fixed the issue. It's not the best solution ever, but given the time I had, it was a working and effective solution.

I've been restarting every hour with a crontab because of the php memory limitation of the shared host I'm using.


Make it possible to run oci_image and oci_image_index targets directly

Current dependencies on/for this PR:

main

PR #33   👈

PR #34

This comment was auto-generated by Graphite.


availability on conda-forge

After we make sure the initial release and publish workflow is in order, we need to set up a spopt-feedstock so that spopt available via conda-forge.

step-by-step instructions here

IIRC we need to list at least 2 recipe maintainers (here's spaghetti's as an example). I will be one. @xf37 and/or @knaaptime would you also like to be recipe maintainer?

IIRC we need to list at least 2 recipe maintainers (here's spaghetti's as an example). I will be one. @xf37 and/or @knaaptime would you also like to be recipe maintainer?

im happy to do it

im happy to do it

Started in conda-forge/staged-recipes#13708.

Started in conda-forge/staged-recipes#13708.

We're getting errors in the staged-recipe. See here.
Traceback (most recent call last):
  File "/home/conda/staged-recipes/build_artifacts/spopt_1610828040194/test_tmp/run_test.py", line 2, in <module>
    import spopt
  File "/home/conda/staged-recipes/build_artifacts/spopt_1610828040194/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_/lib/python3.9/site-packages/spopt/__init__.py", line 4, in <module>
    from .region import MaxPHeuristic
ModuleNotFoundError: No module named 'spopt.region'

We're getting errors in the staged-recipe. See here.
Traceback (most recent call last):
  File "/home/conda/staged-recipes/build_artifacts/spopt_1610828040194/test_tmp/run_test.py", line 2, in <module>
    import spopt
  File "/home/conda/staged-recipes/build_artifacts/spopt_1610828040194/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_/lib/python3.9/site-packages/spopt/__init__.py", line 4, in <module>
    from .region import MaxPHeuristic
ModuleNotFoundError: No module named 'spopt.region'

@knaaptime I'm not sure what's going on with the staged-recipe failures. Can you take a look if you have some time?

@knaaptime I'm not sure what's going on with the staged-recipe failures. Can you take a look if you have some time?

im equally confused

im equally confused

Looks like there is a problem with the build up on PyPI:
$ conda create -n spopt_pip python=3.9 ipython pip
$ conda activate spopt_pip
$ conda install geopandas
$ pip install spopt

Python 3.9.1 | packaged by conda-forge | (default, Jan 10 2021, 02:52:42)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.19.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import spopt
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-1-44f6f8e47c0d> in <module>
----> 1 import spopt

~/miniconda3/envs/spopt_pip/lib/python3.9/site-packages/spopt/__init__.py in <module>
      2
      3
----> 4 from .region import MaxPHeuristic
      5 from .region import RegionKMeansHeuristic
      6 from .region import Skater

ModuleNotFoundError: No module named 'spopt.region'

Looks like there is a problem with the build up on PyPI:
$ conda create -n spopt_pip python=3.9 ipython pip
$ conda activate spopt_pip
$ conda install geopandas
$ pip install spopt

Python 3.9.1 | packaged by conda-forge | (default, Jan 10 2021, 02:52:42)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.19.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import spopt
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-1-44f6f8e47c0d> in <module>
----> 1 import spopt

~/miniconda3/envs/spopt_pip/lib/python3.9/site-packages/spopt/__init__.py in <module>
      2
      3
----> 4 from .region import MaxPHeuristic
      5 from .region import RegionKMeansHeuristic
      6 from .region import Skater

ModuleNotFoundError: No module named 'spopt.region'

the staged-recipe is now passing

the staged-recipe is now passing

the recipe was merged on conda-forge, so the recipe should be available in a few hours. This is good to close

the recipe was merged on conda-forge, so the recipe should be available in a few hours. This is good to close

IIRC

Sorry to miss this issue. I would also like to be a recipe maintainer. Please add me in. Thanks. @jGaboardi

IIRC

Sorry to miss this issue. I would also like to be a recipe maintainer. Please add me in. Thanks. @jGaboardi

@xf37 You can create a PR for this in spopt-feedstock. You'll need to update two things in recipe/meta.yaml:

add your GitHub handle - here
bump the build number to 1 - here

You should wait unit after the spopt v0.1.0 is up on conda-forge to do this. I will make another comment here when that is complete.

@xf37 You can create a PR for this in spopt-feedstock. You'll need to update two things in recipe/meta.yaml:

add your GitHub handle - here
bump the build number to 1 - here

You should wait unit after the spopt v0.1.0 is up on conda-forge to do this. I will make another comment here when that is complete.

@xf37 is spopt v0.1.0 is now up on conda-forge.

@xf37 is spopt v0.1.0 is now up on conda-forge.

xf37 I took care of this in conda-forge/spopt-feedstock#4.

xf37 I took care of this in conda-forge/spopt-feedstock#4.


Support AWS Terraform Provider > 5.0.0

I have tested these changes locally, and the terraform plan passes with zero changes mooted for the bastion, as expected, under AWS TF provider v5.67.0.


[docs] make latest tag as default and remove the legacy versions.

as titled.

Closing as resolved by #2803


Tapping message emojis doesn't do anything

Reaction emojis show up for me, but clicking on the emojis button next to send button does nothing. Is this expected behavior?

That's strange. I cannot reproduce on the demo: https://antoine92190.github.io/vue-advanced-chat/
Can you? Otherwise which browser/device are you using?

That's strange. I cannot reproduce on the demo: https://antoine92190.github.io/vue-advanced-chat/
Can you? Otherwise which browser/device are you using?

That's strange. I cannot reproduce on the demo: https://antoine92190.github.io/vue-advanced-chat/
Can you? Otherwise which browser/device are you using?

The demo works fine, I'm using Chrome.

That's strange. I cannot reproduce on the demo: https://antoine92190.github.io/vue-advanced-chat/
Can you? Otherwise which browser/device are you using?

The demo works fine, I'm using Chrome.

I also tested with Firefox and Edge, just nothing happens when tapping on it.

I also tested with Firefox and Edge, just nothing happens when tapping on it.

Thru debugger I confirmed the function,
openEmoji: function openEmoji(ev) {
      this.$emit('open-emoji', true);
      this.setEmojiPickerPosition(ev.clientY, ev.view.innerWidth, ev.view.innerHeight);
    },

does get called, but nothing happens.

The difference when clicking on the message emojis is that the code ends up running here:

In reactions case, roomFooterRef is defined!

So the demo works fine on Chrome, Firefox and Edge?
If yes, then I guess the issue comes from your code.
Can you share a minimal component code snippet that reproduces your issue?

So the demo works fine on Chrome, Firefox and Edge?
If yes, then I guess the issue comes from your code.
Can you share a minimal component code snippet that reproduces your issue?

Well I'm using it as a component:
<template>
    <v-ons-page>
        <chat-window
            :theme="theme"
            :current-user-id="myUserId"
            :rooms="topics"
            :single-room="false"
            :is-mobile="true"
            :messages="messages"
            :messages-loaded="messagesLoaded"
            :show-add-room="true"
            accepted-files="*"
            :message-actions="messageActions"
            :menu-actions="menuActions"
            :loading-rooms="loadingRooms"
            :rooms-loaded="topicsLoaded"
            @fetch-messages="fetchMessages"
            @send-message="sendMessage"
            @edit-message="editMessage"
            @delete-message="deleteMessage"
            @menu-action-handler="menuActionHandler"
            @message-action-handler="messageActionHandler"
            @send-message-reaction="sendMessageReaction"
            @room-info="topicInfo"
            @typing-message="hmessages.sendKeyPress"/>
    </v-ons-page>
    <!--    @open-file="openFile"-->
    <!--    @add-room="addRoom"-->
</template>

Looks like one strange bug/conflicting code.

Well I'm using it as a component:
<template>
    <v-ons-page>
        <chat-window
            :theme="theme"
            :current-user-id="myUserId"
            :rooms="topics"
            :single-room="false"
            :is-mobile="true"
            :messages="messages"
            :messages-loaded="messagesLoaded"
            :show-add-room="true"
            accepted-files="*"
            :message-actions="messageActions"
            :menu-actions="menuActions"
            :loading-rooms="loadingRooms"
            :rooms-loaded="topicsLoaded"
            @fetch-messages="fetchMessages"
            @send-message="sendMessage"
            @edit-message="editMessage"
            @delete-message="deleteMessage"
            @menu-action-handler="menuActionHandler"
            @message-action-handler="messageActionHandler"
            @send-message-reaction="sendMessageReaction"
            @room-info="topicInfo"
            @typing-message="hmessages.sendKeyPress"/>
    </v-ons-page>
    <!--    @open-file="openFile"-->
    <!--    @add-room="addRoom"-->
</template>

Looks like one strange bug/conflicting code.

I meant a full working component so I can reproduce the issue on my side. Without reproducing, it will be difficult for me to help.
Also, I see that you modified the chat code, as is-mobile prop doesn't exist in my code.
Have you confirmed the issue appears in the original chat component?

I meant a full working component so I can reproduce the issue on my side. Without reproducing, it will be difficult for me to help.
Also, I see that you modified the chat code, as is-mobile prop doesn't exist in my code.
Have you confirmed the issue appears in the original chat component?

It looks like the emoji picker on your screenshot is very tall. Have you overridden the css to achieve that?

It looks like the emoji picker on your screenshot is very tall. Have you overridden the css to achieve that?

It looks like the emoji picker on your screenshot is very tall. Have you overridden the css to achieve that?

I investigated further, the problem is this top attribute:

It's very strange to be added there, I looked into your CSS files there's no top defined there for vac-emoji-picker, and the one added to the html (visible in the image) is also weirdly commented, yet chrome still - for another strange reason - applies it.
I tested the component alone and it works fine, I guess that make it 99% something in my code, I'll investigate more and if so will close this.

It looks like the emoji picker on your screenshot is very tall. Have you overridden the css to achieve that?

I investigated further, the problem is this top attribute:

It's very strange to be added there, I looked into your CSS files there's no top defined there for vac-emoji-picker, and the one added to the html (visible in the image) is also weirdly commented, yet chrome still - for another strange reason - applies it.
I tested the component alone and it works fine, I guess that make it 99% something in my code, I'll investigate more and if so will close this.


JeffH - Updated Sphinx Framework Guide

This request is in support of the following issues that I submitted:
https://github.com/cloudflare/cloudflare-docs/issues/1443
https://github.com/cloudflare/cloudflare-docs/issues/1444
Updated to ensure the Python 3.7 requirement is more noticeable, additional detail on the installation and usage of pipenv, improved examples of using git, added a table for the environment variables, and various other minor enhancements.
The git examples should be usable for other framework guides.

i should be awaited were you do so. therefor you also disputed their on merging blocks.

@signalnerve - I see this was already merged. I was unsure of what needed to be done. My apologies for not getting it done in a more timely fashion.
There was one section where the revised version was not accurate:
Line 51 was deleted: "Installing Sphinx"
Replaced with line 58: "Creating a Sphinx project directory"
The original "Installing Sphinx" is technically correct. The content that was on line 89 got pushed down to line 151 - that is technically where "Creating a Sphinx Project Directory" should have been used.
I can do another pull, then revise it and push the changes. It's a quick change.

Hi @jeffh-cloudflare , I opened a revised PR. https://github.com/cloudflare/cloudflare-docs/pull/2021 Can you please check when you have the time? :)


Feature request: JsDiff.serializePatch(structuredPatch)

What do you think about adding a JsDiff.serializeStructuredPatch(structuredPatchObject) method,
which is basically the reverse of JsDiff.parsePatch(diffStr).
I'd even provide a pull request =P

Name is a little bit lengthy, but glad to review and PRs sent in.

@kpdecker How do we generate the diffStr out now? Does it come from structuredObject?

Hmm. It looks to me like the formatPatch method we already export from patch/create.js essentially is the serializePatch method contemplated here, and all I need to do is

add tests showing it's the inverse of parsePatch
expose it on the top-level Diff object exported by the library, and
document it

I would also prefer serializePatch as a name over formatPatch but I suppose it doesn't matter much so I'll just keep the old name for consistency.


[v6] Initializing the form unregisters all the fields

Dispatching initialize unregister all of the form fields, causing errors upon submission.
Also the third parameter fields (from the most recent docs) doesn't seem to apply to v6 since redux-form no longer needs to define the form fields in the form config.
Steps to reproduce:

Load a form
Make sure all fields were registered to the form
Dispatch initialize
Notice that the form state no longer no longer has registeredFields
Submitting the form causes an error

The methods reset and destroy are doing the same. My use case is: I have a "Reset" button that resets the form and then automatically submits it. Because registeredFields disappear, we get:
Uncaught TypeError: Cannot read property 'map' of undefined
```

[at this line](https://github.com/erikras/redux-form/blob/v6/src/reduxForm.js#L181).

This appears to be fixed in recent versions, closing it


Generate Tokyo Night bat colorscheme automatically

Either in this repo or https://github.com/joshuadavidthomas/dotfiles

Probably dotfiles... Doesn't make sense to have it in the base image. Dotfiles already have the generation script.


Portable Grid Huge FPS loss

Issue description:
When holding the Portable my frames drop from 250 to 70-125.
 This is with and with out a disk of any size in it.  Tested in void world also.
What happens:
Frames drop in half or worst depending on surroundings.
What you expected to happen:
no loss?
Steps to reproduce:

Hold portable grid in had or have it on the hot bar.

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>05
Refined Storage: 1.5.34

Does this issue occur on a server? [yes/no]
Yes
***Notes its fine when placed or not rendered.

https://refinedstorage.raoulvdberge.com/wiki/profiling
Thank you.

Here is an export:
PortableGrid.zip

Should be fixed in 1.6.


Add a nice background to file cards and media only modals

💬 Description
using some base64 encoding since importing the image path breaks on app
✅ Checklist

[ ] Tests written
[ ] Browser tested
[ ] Added to documentation
[ ] Added to storybook

 The enlarged/preview should have the same transparent background (repeating icon)


New docker image libraries

Can you please update the references to the new docker image libraries and which one to use, BC OnPrem, BC Cloud, Dynamics NAV, future releases.
"bcsandbox will move to mcr.microsoft.com/businesscentral/sandbox"
https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/
http://www.waldo.be/2019/01/24/where-to-find-the-microsoft-dynamics-365-business-central-and-nav-docker-images/

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

ID: 6979fc73-1395-b63f-50bf-b2fb1bfda4df
Version Independent ID: eaf8c4d6-15a5-6444-bc72-48bd0a2e326a
Content: Running a Container-Based Development Environment - Business Central
Content Source: dev-itpro/developer/devenv-running-container-development.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for your feedback. We will not include all the library information in this topic, but we've added a link to the blog post for more information, which will be pushed live with the next refresh.


python -m decoding-contour-next-link.read_minimed_next24 fails

Calling the module from the parent directory as described in the readme fails:
tgordon:~$ python3 -m decoding-contour-next-link.read_minimed_next24
/usr/lib/python3.8/runpy.py:126: RuntimeWarning: 'decoding-contour-next-link.read_minimed_next24' found in sys.modules after import of package 'decoding-contour-next-link', but prior to execution of 'decoding-contour-next-link.read_minimed_next24'; this may result in unpredictable behaviour
  warn(RuntimeWarning(msg))
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 193, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/tgordon/decoding-contour-next-link/read_minimed_next24.py", line 1571, in <module>
    downloadPumpSession(pumpDownload)
  File "/home/tgordon/decoding-contour-next-link/read_minimed_next24.py", line 1465, in downloadPumpSession
    mt.openDevice()
  File "/home/tgordon/decoding-contour-next-link/read_minimed_next24.py", line 764, in openDevice
    self.device.open( self.USB_VID, self.USB_PID )
  File "hid.pyx", line 66, in hid.device.open
OSError: open failed

I tested with two different Contour Next Link 2.4 units on Ubuntu 20.04. The error is identical before and after unplugging either CNL.

Try with python2

Try with python2


Fix Infinity css warning

closes https://github.com/apache/incubator-superset/issues/6190

bonus bugfix

Codecov Report

Merging #6280 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6280   +/-   ##
=======================================
  Coverage   77.01%   77.01%
=======================================
  Files          64       64
  Lines        9508     9508
=======================================
  Hits         7323     7323
  Misses       2185     2185

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 124c55c...57726ce. Read the comment docs.


Fix an issue where multiple handlers weren't registering correctly

The use case here is this:
@listen_to("(.*)")
def do_some_fallback(msg):
    ...

@listen_to("(.*)")
def do_some_other_fallback(msg):
    ...

We want >1 function attached to one regex "(.*)". However due to the way Python compiles regexs, these handlers were overwriting themselves here:
PluginsManager.commands['respond_to'][re.compile(regexp, flags)] = func

To convince yourself what's happening:
$ python
Python 2.7.13 (default, Dec 18 2016, 07:03:34)
[GCC 4.2.1 Compatible Apple LLVM 7.0.2 (clang-700.1.81)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import re
>>> mapping = {}
>>> mapping[re.compile('(.*)', 0)] = 1
>>> mapping[re.compile('(.*)', 0)] = 2
>>> mapping[re.compile('(.*)', 0)] = 3
>>> mapping
{<_sre.SRE_Pattern object at 0x10f418580>: 3}

cc @gotlium

Hi, we need to test with our existing code. Or you can add some settings to enable this feature. Because we have a backward compatibility with slackbot.


Are MULTI/EXEC transactions thread-safe?

I was looking at the code that handles MULTI/EXEC. This comment seems to say that issuing transactions with MULTI/EXEC is not thread-safe. I.e., if you have multiple connections issuing transactions in different threads, it could result in commands being received in the wrong order (e.g. redis server receiving MULTI twice before receiving EXEC). I just wanted to make sure. Is the comment still correct?

Pretty sure that just means that you can't share the multi object itself between concurrent goroutines. As far as I'm aware, under the hood that multi reserves a single connection for it's use and thus cannot collide with any other goroutine using the redis connection pool.

What @justonia said.
I don't mind to make MULTI/EXEC thread-safe (goroutine-safe?) if there is a real use case. But I can't imagine any.

Okay. That makes complete sense, thank you. I can't imagine a use-case where you would need to share a Multi object between multiple goroutines.


Images with jewel 10.2.9

Being completely new to all ceph and docker thing, so sorry for this question.  Other manually rebuilding the base is there a way to get 10.2.9 image? According to the release notes 10.2.8 has a serious regression.


[Doc]bugfixes-in-loading-from-mysql

Signed-off-by: hellolilyliuyi<EMAIL_ADDRESS>What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[x] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[ ] 2.2

run starrocks_admit_test

@mergify backport branch-2.5

@mergify backport branch-2.4

@mergify backport branch-2.3


Update ZipOutputStream.cs : PutNextEntry()

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the SharpZipLib open source product (the "Contribution"). My Contribution is licensed under the MIT License.

Updating entry.CompressedSize in CloseEntry() function in file.
its create issue while extracting zip.

What issue is this attempting to prevent?

Please refer the following issue
https://github.com/icsharpcode/SharpZipLib/issues/139

I am closing this as I cannot reproduce the issue.