Versions of python bindings

Hi,
Could you please specify the version of OGR bindings you use to run the script successfully?
Currently getting this traceback:
WARNING:
 This script assumes a polygon shapefile in spherical mercator projection.
 If any of these assumptions are not true, don't count on the results!
Traceback (most recent call last):
  File "./create_utfgrids.py", line 108, in <module>
    create_utfgrids(shppath, minzoom, maxzoom, outdir, fields)
  File "./create_utfgrids.py", line 44, in create_utfgrids
    polygon_symbolizer = mapnik.PolygonSymbolizer(mapnik.Color('#f2eff9'))
Boost.Python.ArgumentError: Python argument types in
    PolygonSymbolizer.__init__(PolygonSymbolizer, Color)
did not match C++ signature:
    __init__(_object*)

That error is related to mapnik bindings, not OGR
At the time this was written, mapnik 2.x was used. It's not been tested on 3.x. What version of mapnik are you using?

Thanks for getting back so fast. I see. That's the problem!


Add Telco Churn Tutorial with XGBoost and Feast

Adds a Telecom Churn Prediction tutorial using Feast. This tutorial will hopefully be a little bit more realistic compared to the "basic" example that we have.

Can we host the file instead of pushing it to the repository?

/retest

Can we host the file instead of pushing it to the repository?

Sure, I was planning to drop it on nbviewer like this: https://nbviewer.jupyter.org/github/woop/feast/blob/add-churn-prediction-tutorial/examples/feast-xgboost-churn-prediction-tutorial/Telecom Customer Churn Prediction (with Feast and XGBoost).ipynb
Or did you have something else in mind?

No, I mean the data file.

No, I mean the data file.

Sure, I can push it to GCS. It's pretty small though. 1mb vs the Feast repo that is 16mb.

No, I mean the data file.

Sure, I can push it to GCS. It's pretty small though. 1mb vs the Feast repo that is 16mb.

You're making it 1mb bigger though 🤔 it's already brutal pulling this repository on a bad connection

@zhilingc I've moved the data out. Will merge this.


⚠ GitLab Server has degraded performance

In c1b5a18, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 3988 ms

Resolved: GitLab Server performance has improved in d93b05b.


✨ Add a middleware to a list of message types

When using the use function, you can specify a message type. So you can add the same middleware to a collection of message types with mutilple use call. From a DX perspective this is not great, this operation have to be done with a single call !

Done in #25


Add **kwargs to all legacy provider classes, pass to super().init

This allows some of the new facilities that are setup through keyword args
through the drivers to operate in legacy (not alias, needs other changes).

...playing little games, like not looking at it for a while, then looking... to see if I still liked it.  I did.


Migration tools

Is there any migration tool to transfer messages of users to this platform?.
Currently Im having my old messages  (tons of)  in Firestore now what I want is to transfer all those messages from firestore to CC?
Or atleast provide any handling to database etc so we can write our self.

you can export your data from Firestore and turn to our support team<EMAIL_ADDRESS>and they together with the backend team will try to help you to import your data to the Connectycube DB.

How can i do that?
On Tue, Jun 13, 2023 at 1:01 PM TatankaConCube @.***>
wrote:

you can export your data from Firestore and turn to our support team
@.*** and they together with the backend team will try
to help you to import your data to the Connectycube DB.
—
Reply to this email directly, view it on GitHub
https://github.com/ConnectyCube/connectycube-flutter-samples/issues/268#issuecomment-1588758708,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5C5RMJ5IGWHBYFN4LJE453XLAM5BANCNFSM6AAAAAAZDJYB3I
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

I see this in a next way: you export your data to the JSON file using Firestore tools, then you convert your models into Connectycube-style models according to the specification provided in our Server API doc, then you share this result with our server-side team and they try to import it to the Connectycube DB.
Unfortunately, we don't have yet ready for using a solution for migration from the Firestore to the Connectycube(((

no activity for a long time. closing...


fix: match letterbox exoplayer version and version upgrade changes

Fork main repo because we needed this library to use the exact same version as letterbox --> 3.14.0
Some changes were necessary following the upgrade

my bad, this PR was supposed to be on my fork but github automatically selected the destination to be this repo and not mine which i saw too late


Multi-level subdomain

setup a new proxy host.
nvr.fulton.example.com
going to that link it wont find the host.
i added nvr.example.com to the same host.
going to this link works fine.
do we not support multilevel subdomains?

I'm using a lot of 4th level domains in nginx-proxy-manager without problems...

This is just a wrapper for Nginx so it will do whatever nginx allows, I too am using something.somehere.mydomain.com just fine. My guess is that your DNS is not correctly configured for the FQDN.

I setup a new instance and it seems to be working.

For those who have been searching for a solution to a similar problem, here's a bit of info (specific on my problem, but i hope it will help someone ;-) ):
I encountered problems with NPM using multi-level subdomains. However, it turns out the issue isn't with NPM itself; it's the classic "it's always DNS" scenario. It seems that some DNS servers provided by my local internet service are having trouble with multi-level subdomains (for reasons unknown).
Furthermore, it appears that propagating DNS entries worldwide is a bit slower compared to standard (sub-)domains, which have only one level, so to speak.
To make matters worse, I've found many issues related to DNS in German router forums, which can be traced back to the use of Fritz!Box routers. It seems that these devices occasionally suffer from bugs.
I personally experienced a problem where, even after several days, my DNS from the internet provider "EWEtel" did not correctly output the DNS entries. Additionally, my Fritz!Box seemed to distort the response from the DNS server of EWE (and others).
So, if you can't reach a subdomain or multi-level subdomain, try to troubleshoot your DNS using tools like nslookup or online services such as dnschecker.org.


AV Detected!!!

Built and ran the output file and a cookies file sent to the webhook
But the output.exe file was detected by Windows Defender!
Please help!

Exclude the output.exe file from Windows Defender scanning:
Open Windows Security by clicking on the shield icon in the taskbar.
Go to "Virus & threat protection" and click on "Manage settings."
Scroll down to the "Exclusions" section and click on "Add or remove exclusions."
Click on "Add an exclusion" and select "File."
Navigate to the location where the output.exe file is located and select it.

Since so many people are using this, antivirus softwares are beginning to detect it. I won't be fixing this, as it's a relatively simple project and I don't feel like making it complicated. You can learn how to manually edit and compile the main.py file if you'd like, as this can help with detections. I'd advise learning more about programming before trying this though.


Carthage Xcode 11 beta 2 error

Error with carthage update AFNetworking --platform iOS any idea ?:
`*** Building scheme "AFNetworking iOS" in AFNetworking.xcworkspace
Build Failed
Task failed with exit code 65:
/usr/bin/xcrun xcodebuild -workspace /Users/xxx/Desktop/ios/Carthage/Checkouts/AFNetworking/AFNetworking.xcworkspace -scheme AFNetworking\ iOS -configuration Release -derivedDataPath /Users/xxx/Library/Caches/org.carthage.CarthageKit/DerivedData/11.0_11M337n/AFNetworking/3.2.1 -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive -archivePath /var/folders/mg/jj2l1dpn0bb7kt9v9spfzq8m0000gn/T/AFNetworking SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO (launched in /Users/xxx/Desktop/ios/Carthage/Checkouts/AFNetworking)
This usually indicates that project itself failed to compile. Please check the xcodebuild log for more details: /var/folders/mg/jj2l1dpn0bb7kt9v9spfzq8m0000gn/T/carthage-xcodebuild.ShOP4X.log`

I'm seeing the same issue - did you find a solution?

The full error can be seen in the carthage xcodebuild log:
$ cat /var/folders/jq/0vvn04111f57vtyhmgnd19xw6w3gfm/T/carthage-xcodebuild.2b0BIG.log | xcpretty
▸ Processing Info.plist
▸ Compiling AFURLSessionManager.m
▸ Compiling AFURLResponseSerialization.m
▸ Compiling AFSecurityPolicy.m
▸ Compiling AFURLRequestSerialization.m

❌  /Users/emjaku/YLNetworking/Carthage/Checkouts/AFNetworking/AFNetworking/AFURLRequestSerialization.m:574:26: implicit conversion changes signedness: 'AFHTTPRequestQueryStringSerializationStyle' (aka 'enum AFHTTPRequestQueryStringSerializationStyle') to 'NSInteger' (aka 'long') [-Werror,-Wsign-conversion]

    [coder encodeInteger:self.queryStringSerializationStyle forKey:NSStringFromSelector(@selector(queryStringSerializationStyle))];
    ~                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

❌  /Users/emjaku/YLNetworking/Carthage/Checkouts/AFNetworking/AFNetworking/AFURLRequestSerialization.m:1298:26: implicit conversion changes signedness: 'NSJSONWritingOptions' (aka 'enum NSJSONWritingOptions') to 'NSInteger' (aka 'long') [-Werror,-Wsign-conversion]

    [coder encodeInteger:self.writingOptions forKey:NSStringFromSelector(@selector(writingOptions))];
    ~                    ^~~~~~~~~~~~~~~~~~~

❌  /Users/emjaku/YLNetworking/Carthage/Checkouts/AFNetworking/AFNetworking/AFURLRequestSerialization.m:1384:26: implicit conversion changes signedness: 'NSPropertyListFormat' (aka 'enum NSPropertyListFormat') to 'NSInteger' (aka 'long') [-Werror,-Wsign-conversion]

    [coder encodeInteger:self.format forKey:NSStringFromSelector(@selector(format))];
    ~                    ^~~~~~~~~~~

▸ Compiling AFHTTPSessionManager.m
▸ Compiling AFNetworkReachabilityManager.m
▸ Compiling AFNetworking_vers.c

The fix seems to be explicitly casting a few arguments to NSInteger.
diff --git a/AFNetworking/AFURLRequestSerialization.m b/AFNetworking/AFURLRequestSerialization.m
index a4d5d9d..0fd987d 100644
--- a/AFNetworking/AFURLRequestSerialization.m
+++ b/AFNetworking/AFURLRequestSerialization.m
@@ -571,7 +571,7 @@ - (void)encodeWithCoder:(NSCoder *)coder {
     dispatch_sync(self.requestHeaderModificationQueue, ^{
         [coder encodeObject:self.mutableHTTPRequestHeaders forKey:NSStringFromSelector(@selector(mutableHTTPRequestHeaders))];
     });
-    [coder encodeInteger:self.queryStringSerializationStyle forKey:NSStringFromSelector(@selector(queryStringSerializationStyle))];
+    [coder encodeInteger:(NSInteger)self.queryStringSerializationStyle forKey:NSStringFromSelector(@selector(queryStringSerializationStyle))];
 }

 #pragma mark - NSCopying
@@ -1295,7 +1295,7 @@ - (instancetype)initWithCoder:(NSCoder *)decoder {
 - (void)encodeWithCoder:(NSCoder *)coder {
     [super encodeWithCoder:coder];

-    [coder encodeInteger:self.writingOptions forKey:NSStringFromSelector(@selector(writingOptions))];
+    [coder encodeInteger:(NSInteger)self.writingOptions forKey:NSStringFromSelector(@selector(writingOptions))];
 }

 #pragma mark - NSCopying
@@ -1381,7 +1381,7 @@ - (instancetype)initWithCoder:(NSCoder *)decoder {
 - (void)encodeWithCoder:(NSCoder *)coder {
     [super encodeWithCoder:coder];

-    [coder encodeInteger:self.format forKey:NSStringFromSelector(@selector(format))];
+    [coder encodeInteger:(NSInteger)self.format forKey:NSStringFromSelector(@selector(format))];
     [coder encodeObject:@(self.writeOptions) forKey:NSStringFromSelector(@selector(writeOptions))];
 }

A viable workaround is to build this framework with GCC_TREAT_WARNINGS_AS_ERRORS=NO. AFNetworking.xcodeproj has it set to YES, and that's what Carthage uses to build it by default.

Why was this issue closed? This is still a problem in the later Xcode 11 betas, where the latest 3.2.1 release of AFNetworking fails to build with this error...

Same here. This issue should not be closed.

Until this issue is actually addressed, for those using AFNetworking as a managed dependency and building using Xcode 11.x (but also wanting to avoid forking to edit the project file to ignore the warnings shown above), it turns out you can workaround this issue when using Carthage quite easily.
Firstly, save a new xcconfig file with the following content:
CLANG_WARN_IMPLICIT_SIGN_CONVERSION = NO

Then you can provide this to Carthage when calling carthage build or similar:
XCODE_XCCONFIG_FILE=path/to/AFNetworking_workaround.xcconfig carthage build …

Hopefully this is of some use to anyone hitting this in Xcode 11.x with Carthage builds!

Still an issue.

the workaround working great.


Simplify import / reference tags using snippets and autocomplete-plus

[ ] snippets for /// <reference path="$1"/> and import ${1:alias} = require('${2:path}')
[ ] change the grammar to identify these
[ ] based on grammar scope in autocomplete requestHandler populate these with relative file paths

Currently this is the only reason I am running grunt-ts while working on this editor.


How to use Server-Side Row Model for ag-Grid

This example of how to use ipyaggrid shows the initialization of the agGrid with a fully loaded data (frame), which corresponds to the client-side row model.
g = Grid(grid_data = data, grid_options=grid_options, export_mode="buttons", theme='ag-theme-balham')

Does ipyaggrid support agGrid in the Server-Side Row Model? If so, where can I read about it? If not, could you give me some guidelines on what I should do to enable it?
I've studied the sources a little. Seems like that this use case is not supported. Inside widget_builder.js seems to be the only place where you inject the data:
    // instantiate grid
    const grid = new Grid(gridDiv, gridOptions);
    window.grid = grid;

    // inject gridData
    gridOptions.api.setRowData(gridData);

Maybe under your guidance I could implement the support for the SSR mode for agGrid and create a PR? Since agGrid's SSR model basically requires just a datasource object with getRows method, we could create it in python env and pass it to Grid during the instantiation
g = Grid(datasource= ds,

You're correct that ipyaggrid currently doesn't support agGrid's Server-Side Row Model. It's an area that we think would benefit the package, and we appreciate your interest in contributing.
Given the potential complexities of implementing this feature, we think that creating a new widget specifically for this functionality would be the most prudent approach. This would allow us to maintain compatibility for users who are already using the existing Grid widget.
While our team may not have the bandwidth for extensive guidance, we're definitely open to receiving a PR.

To clarify, the new widget could live in the ipyaggrid package as a new widget, it does not need to be a new ipywidget library/package.


Bug of hoisting function

Function hoisting sometimes causes a bug of calling bind method of undefined. Figure out later.

When function A is declared inside function B, function A will be hoisted and its ast tree node will be set as null, to avoid re-declaration. If function B is called several times, the hoist method will meet a null tree node, producing this bug.
Fix: do not set the ast tree node as null after hoist.


Added Apache MXNet deep learning framework support

Imerative, Symbolic and Hybrid (gluon) modes supported.  CPU only for now -- CUDA support is still not production-quality, so I'll add that in a later PR.

+1 LGTM


Get rid of Lombok dependency

It is more like question than issue. I would prefer the project without dependency on Lombok since it forces me to install additional IDE plugins.
Are you interested in PR that replaces Lombok with regular Java code?

Thanks but no, we'd like Lombok to stay.

Ok, in case if anyone interested I'm planning to maintain parallel fork without Lombok and sync it regularly with the upstream: https://github.com/jreznot/micronaut-microservices-poc


Properly recover previously throttled orphaned jobs when using Sidekiq Pro's SuperFetch

Without this fix, previously throttled orphaned jobs would remain throttled after being recovered and placed back on the queue.
One downside is that this necessitates the usage of setup! once more, as the orphan recovery handler must be registered after SuperFetch has been initialized with config.super_Fetch!.
Also fixed the bundle exec rake script when running Sidekiq Pro tests with the BUNDLE_GEMS__CONTRIBSYS__COM environment variable set.
NOTE: .rspec-local does not work (at least with rspec 3.12), you have to override .rspec instead.

@ixti something I ran into when trying out version 1.3.

Not sure how the tests are failing, they succeed locally for both .rspec and .rspec.sidekiq-pro 🤔

@gstokkink Sidekiq::Throttled.setup! is deprecated and will be removed in v2. We don't call it and haven't run into issues. Do you still experience the issue if you remove the call to Sidekiq::Throttled.setup!?

@gstokkink reading through your PR, do you need to keep .setup!? I see that you're removing the deprecation warning, so I'm wondering if you're using .setup! for more intricate setup than just reordering middlewares than what could be accomplished otherwise, like mentioned in this comment https://github.com/ixti/sidekiq-throttled/pull/167#issuecomment-1906739350

Last thing, your specs pass for me locally once I add this fix to spec_helper.rb #181
I also noticed that .rspec-local is not being respected and to update .rspec itself

@mnovelo looks like I spoke too soon, I was accidentally still running the 1.0.0 release which did have this bug. In 1.3.0 it seems to not be an issue, because with the new implementation a job cannot block itself. Thus, if a job is orphaned and recovered, and it held a slot, it will be allowed to run again, using that slot. I will investigate if this works correctly in 100% of all cases.

oh awesome! Thanks anyway for your help identifying that the Sidekiq Pro specs weren't working as expected.
Re: your question

The only problem that I can foresee is that the slot will remain occupied if the job is never recovered, but that would mean SuperFetch is broken I guess. Or will it expire?

SIdekiq-throttle's slots expire after 15 minutes by default.. Sidekiq-throttle calls .finalize! in an ensure block, so if that block never gets called, the slot will be released in about 15 minutes, depending on the TTL set


Fix some typos

What is the purpose of the change
fix #5500
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #5501 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5501      +/-   ##
============================================
- Coverage     61.15%   61.13%   -0.03%
+ Complexity      424      421       -3
============================================
  Files           921      921
  Lines         37496    37496
  Branches       5443     5441       -2
============================================
- Hits          22930    22922       -8
- Misses        12067    12072       +5
- Partials       2499     2502       +3

Impacted Files
Coverage Δ
Complexity Δ

...va/org/apache/dubbo/common/utils/ReflectUtils.java
70.03% <ø> (ø)
0 <0> (ø)
:arrow_down:

...n/java/org/apache/dubbo/common/utils/NetUtils.java
68.35% <100%> (ø)
0 <0> (ø)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0%> (-6.9%)
0% <0%> (ø)

...mmon/threadpool/support/AbortPolicyWithReport.java
83.78% <0%> (-5.41%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
58.69% <0%> (-4.35%)
0% <0%> (ø)

...he/dubbo/remoting/transport/netty/NettyServer.java
70.17% <0%> (-3.51%)
8% <0%> (-1%)

...c/main/java/org/apache/dubbo/rpc/RpcException.java
80% <0%> (-3.34%)
0% <0%> (ø)

...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java
88.57% <0%> (-0.96%)
16% <0%> (-1%)

...org/apache/dubbo/registry/redis/RedisRegistry.java
48.15% <0%> (-0.29%)
31% <0%> (-1%)

...org/apache/dubbo/config/AbstractServiceConfig.java
100% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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


chore: bump to 1.0.22

coverage: 57.892% (+0.09%) from 57.804%
when pulling 470d88ed8a71ffc64d0c57c29d593981fa114268 on carlosmiei:bump-22
into 4b641a3c2aeeecc267d710ab938b2492605c52af on sammchardy:master.


Migrate from tslint to eslint

As tslint gets deprecated (see https://medium.com/palantir/tslint-in-2019-1a144c2317a9), we will need to migrate to eslint. Nothing is urgent, so we have to carefully evaluate the gap and what (if) we will be missing some catches by the move.
See https://github.com/typescript-eslint/typescript-eslint for reference.

I try to take a look in my free time...
I'm also interested in migrate all my recent project to ESLint


First Release Checklist - v0.1

Some suggestions actions we can consider:

[ ] Explore Github Actions for CI/Lint and Build
[ ] Maybe use cobra & viper for config management and cli options
[ ] Merge -benchmark and -create output
[ ] Add concurrency to benchmark
[ ] Add more network types
[ ] Setup logging level at differnt logs and set level according to verbosity setting
[ ] Review configuration and use
[ ] Document usage guide in csbench's wiki (Github)

cc @harikrishna-patnala @vishesh92 @kishankavala @vladimirpetrov


feature: setup ci build

This PR is to add a build/smoke test for the pio package. Nothing fancy.

Thanks for review @soypat now merging.


Scale Node Title with size.

https://github.com/tchayen/markdown-links/blob/045dd4fc9c0013da0c69198a7d7f3e645e7c1d18/static/graphs/obsidian/graph.js#L258
I think it would be good to scale the size of the node in relation to the size, to give it more importance. Maybe set the scaling factor as an option?

I don't think making it an option would be a good idea. There are custom graph views, now we would be adding configuration to them. I think it would better to hear more opinions hear and decide whether or not we should alter the current styling.
@dobrovolsky What do you think about it?

I don't think making it an option would be a good idea. There are custom graph views, now we would be adding configuration to them.
I agree with you.
Imagine a situation when there will be 10s different graphs with different options. It will create lots of mess.
But on the other hand, we could create one settings option with a JSON object and specify what we want to change in our graph.

I also agree that making it a user-defined option probably isn't good.
Either way, I still think it would be interesting to scale the labels, even if you can't choose the scale, the same way that the nodes are scaled and can't be changed.
Implementing this change is relatively simple

@tchayen @dobrovolsky @Am3ra perhaps, at the very least, we could get the user's configured font size and base the scaling off of that? The option is editor.fontSize for reference.

@ingalless that sounds much better (and accessible too). We should connect font size to the font size of the editor I guess, and in case of obsidian make scaling based also on that base value.

@ingalless  Regarding the obsidian version, I think ${ nodeSize[d.id] * editor.fontSize / 8}px would be a good way to scale it.
nodeSize is a value between 8 and 24, so dividing by 8 would scale to a value between 1 and 3.
Multiplying by the base fontSize I think would achieve a good scaling range. I'm testing that out now. Any thoughts?

@Am3ra feel free to open a PR for the change! I'm not actually too familiar with graph generation code, so I can't comment on your method. Just so long as it is easily understandable (either through a comment or extracting it into a function) then I'm fine with it!
@tchayen will probably be able to add some more comprehensive thoughts 😆

Created the PR, wasn't able to link it to this issue 😢 . However, it very simply changed the behaviour of the graph labels.

In this image, it's pretty clear to see the difference in label sizes corresponding to node size.

Closed by #62


[HOLD #15362] [reportActionCount] Don't show DM's with only 1 reportAction in options list (unless it is the currently viewed report)

Please see the tracking issue for more. This is held until the penultimate step of the roll-out plan is merged.

Still on hold.

Closing since this solution is not needed anymore - see tracking issue.


hide logged in user amongs searched users

closes #379

[x] Hides the current user from/amongst searched users.

This is so that the logged-in users cannot follow themselves or access the follow button for themselves via the user search listing.
@codyseibert
@Mr-spShuvo

disable-button-add-loader-#351
hide-current-user-on-search-#379
master
register-button-disable-add-loader-#351
self-follow-button-#352

This pull request was created but the branch was not up to date with the master branch.
I rebase but thought that caused much confusion and decided to create another PR.


Azure Subscription Access - Fix for Users

Description
Fixes a limitation in v1.0 that only returns the first 100 users.
Issues Resolved
n/a
Contribution Check List

[x] All tests pass.
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] New functionality has been documented in CHANGELOG.MD

Fixed some of the markdown that was identified by a markdown linter


EthernetC33 debugging

the missing volatile on bool dhcp_acquired in CNetIf.h caused problems for WiFi DHCP too. The waiting for DHCP ended only on timeout, but status() check then was OK because the IP address was set. So WiFi.begin just took long time.
I use Uno R4 to test Renesas Core LwIP networking. For WiFi I have an esp32 with the Hosted firmware on SPI and for Ethernet I replaced the EthernetDriver implementation with one over the W5500 driver (Ethernet 2 shied) from the esp8266 core. So I use Ethernet.begin methods with the MAC address parameter where some of this bugs appeared. The missing restartAsyncRequest(); in setMacAddress stopped the periodic task so DHCP then timed out. After that I had in EthernetDriver a buffer smaller than the DHCP packet and the code just hang. It was the the assert in CEth::output).
tested with LegacyEthernetTest
EthernetDriverW5500.cpp should work on Portenta C33 with MKR ETH shield too

Looks like this branch is kept from merging by the license/cla check not bring signed. Can you sign the CLA agreement? Likely you've already signed it, but maybe you made this commit with a different email address? (This is a known cause for CLA errors).

Update: wasn't a CLA issue but a merge conflict with main. All is resolved now.

Why the force-push? Now there's another merge-conflict :wink: . You'd have needed to pull --rebase, then add a new commit on top (uncommenting the include). Do you want to clean this up or shall I?

sorry, the force push was just to start the check for the CLA

Hi @JAndrassy :coffee: :wave: can you check that your merges are in? (They should be).

looks OK. thank you


Increment version for storage releases

Increment package version after release of azure-storage-file-share

I will do this manually.


findAndModify returns old document's value

When I tried to update a field using findAndModify, it returned the old document in the callback.

You have to include the option new: true to get modified documents instead of old ones

@saintedlama Thank you! It works!!


use https while downloading binaries

What changes were proposed in this pull request?
./build/mvn file was downloading binaries in non HTTPS mode. This PR tends to fix it.
How was this patch tested?
By running ./build/mvn clean package locally

Can one of the admins verify this patch?

@infynyxx could you add a JIRA ticket and create a ticket like [SPARK-xxxx][BUILD] use https while downloading binaries (see the titles of other PRs).

@hvanhovell I didn't create JIRA ticket because as per Contributing Guide creating JIRA ticket is not required for trivial changes. But I can create one, if it's required ;)

The change itself LGTM

Jenkins test this please

Test build #55019 has started for PR 12182 at commit 49963ee.

Test build #55020 has started for PR 12182 at commit 49963ee.

Test build #55020 has finished for PR 12182 at commit 49963ee.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55020/
Test PASSed.

@infynyxx I think this one is on the borderline. I would just create one so everyone is happy.

@andrewor14 @hvanhovell updated PR with JIRA ticket

Merged to master


Question Regarding NTLM/Negotiate

I was wondering what level of encryption does NTLM/Negotiate give? 256 bit? Haven't been able to find this searching the web.

I think it's AES-256 Symmetric keys and it's better to use Negociate with an AD (aka Kerberos). NTLM has some weaknesses.

https://foxdeploy.com/2017/02/08/is-winrm-secure-or-do-i-need-https/

Thank you


Tag

Can this be tagged at 0.0.1? Bit scared about using this in production without a tag... :-)

Done


GH-36990: [R] Expose Parquet ReaderProperties

Rationale for this change
Expose the ReaderProperties class in R so that the thrift size settings can be altered.
What changes are included in this PR?
Add R6 class, link it up to the C++ class, use it when reading Parquet files.
Are these changes tested?
Yes
Are there any user-facing changes?
Nope

Closes: #36990

I've opened #36997 for the follow-up work of exposing this functionality (and other functionality) in the read_parquet() function signature.


Game Over! Player red has won!

change checkDiagonalBLtoTR to:
function checkDiagonalBLtoTR() {
return checkWin({i: 1, j: -1}, {i: -1, j: 1});
}
has solved it for me


Implement __concat, table.concat, and add Lua polyfills for table.sort and table.move

At some point it would be good to have table.move and table.sort implemented in Rust, but for now a Lua implementation works.
This PR also implements the __concat metamethod, and uses it to implement table.concat; unlike PRLua's implementation, this version supports tables with the __concat metamethod.
For example, table.concat({ a, b, c }, sep) is equivalent to a .. sep .. b .. sep .. c, and respects the right-associativity of the .. operator.
Semantics questions on table.concat:

Should table.concat({ x }) == x, regardless of type?  It currently does, as it never tries to call the __concat metaoperator.
Is the complexity of this worth it over only supporting string/integers in table.concat?

This is really good, thank you!

Should table.concat({ x }) == x, regardless of type? It currently does, as it never tries to call the __concat metaoperator.

I think so, I don't think that the __concat metamethod should be called with a single argument definitely, and it would be weirder to call something like __tostring. I think it makes sense because it makes table.concat({ table.concat({ x }), y}) the same as table.concat({x, y}).

Is the complexity of this worth it over only supporting string/integers in table.concat?

This I'm not as sure about, but it makes complete sense to me to try to use the same implementation of the concat metaop for table.concat

We're accumulating a lot of stdlib functions which sort of have unbounded time usage, but that unbounded time usage is proportional to some allocated memory (number of arguments on the stack, size of a table, etc).
I'm not fussed about solving this now, especially varargs because I think there might need to be a more centralized solution that limits varargs to something reasonable (I would be fine with a lower limit than PUC-Rio's limit even, which is something silly like a million). I'm mostly pointing this out to LET IT BE KNOWN that I am aware of it, and that it is a temporary problem.

What would be the current best approach to fuel consumption for concat?  Consume fuel proportional to the output size at each step?  (This runs into the issue of not being able to efficiently break up large allocations, though -- would need to repeatedly consume fuel / yield while doing nothing, and then once enough fuel has been consumed, make the full large allocation.)

I'm sorry it took me so long to merge this, I've had a rough week so I've been a little out of it.
I ended up moving the support code for async sequences used by meta ops into meta_ops.rs and making it private. I know this is the opposite of what we discussed before, but I didn't realize it was something ONLY used by meta_ops.rs, so this made more sense to me. This was the only meaningful feedback I had after sitting on it for a week (and it was a tiny change) so I just went ahead and merged the PR rather than bother you about it.
Thank you for all of your hard work. With this, every metamethod is implemented other than __gc!

Thank you!  There's no problem with the wait on the merge (after all, I'm responding with a significantly longer delay; life happens).
On the async support code: I agree, but I think it would still be worth making something like it available in the future (potentially in piccolo_util), since it helps a lot with the verbosity of async sequences.  It would need more planning to determine which helpers/API would work best, and at this point I don't have enough usage examples to come up with a good design.


[accessibility] Keyboard and screen reader support for the close button

The control is not focusable, it should be a <button type="button"> or role="button" (with tabindex="0" or <a href="#">).
aria-label or title can be used to allow screen reader users to understand what the control is for.
For example title="Close sidebar".

Other resources that you may find helpful:

Leaflet guide to accessibility: Test your maps
Leaflet Plugin Authoring Guide: Content Accessibility


 This means that not every building occupancy type from the CEA is included (e.g., Industry and Hospitals are not included). In the current CEA calculation, the target for these uses is set to zero!

Short term solution for the MIBS: use the values for industry from InduCity and set the values for all other occupancy types to the target for "other services" from InduCity.

Long term solution: need to calculate the targets for each use type following the methodology from Lenzlinger (2010) “SIA Effizienzpfad: Bestimmung der Ziel- und Richtwerte mit dem Top-Down Approach,” as done for InduCity. As described in this report from Jimeno:

The estimation of goals is based on the Swiss prospects of energy reference area by Haag, 2004 [7], the grey emissions of the Swiss building stock by Lenzlinger, 2009 [8] and the Swiss energy consumption statistics of Switzerland 2000-2011 by Kemmler et al., 2012 [9]. The result is a set of goals for reduction of CO2 emissions and primary energy consumption of the average Swiss building stock. These values are compared to those of the buildings in the area of study, obtaining goals for each category of use in the area on the time line of the suggested transformation (Table 1).

@amraladdin This is the issue I was referencing the other day when we talked about setting the 2000W targets. For the MIBS course we already used the values we assigned ourselves with Stefan. Longer term we should really calculate the targets as done in the literature.

@martin-mosteiro
Yeah, true. So now we have the new values, however, we also made a number of assumptions and we need to verify them. I don't think we can do this before this milestone. We will need to postpone the deadline of this issue. What do you think?

This issue was resolved along with #464 . The missing values were completed based on the standard calculation procedure when possible and based on simple assumptions otherwise. These assumptions were double checked and considered reasonable by @amraladdin . The missing values are completed as follows:
(1) 'INDUSTRY': operation and embodied energy and emissions calculated based on the bottom-up approach from SIA 2040; due to lack of data on mobility, the same values as for 'OFFICE' were assumed, as the demand consists mainly of workers' daily commutes
(2) 'HOSPITAL': operation and embodied energy and emissions estimated based on the bottom-up approach from SIA 2040 coupled with energy demand statistics per sector from the Bundesamt für Energie; due to lack of data on mobility, the same values as for 'HOTEL' were assumed (some people going to work, mainly people going for non-daily travel purposes)
(3,4) 'GYM', 'SWIMMING': operational energy calculated from SIA 2024, then the corresponding primary energy and CO2 emissions were estimated as an average of the values for all other uses; due to lack of information, embodied energy and mobility were assumed to be the same as for 'RETAIL' (mainly people attended leisure activities); targets were then set based on these values through the same approach as SIA 2040
(5) 'SERVERROOM': calculated the operational energy the same way as (3,4); due to lack of information embodied energy was assumed to be the same as 'OFFICE' (highly technical office space); mobility was assumed to be negligible; the targets were then set the same way as (3,4)
(6) 'COOLROOM': calculated the operational energy the same way as (3,4); due to lack of information embodied energy was assumed to be the same as 'HOSPITAL' (building with lots of technical installations); mobility was assumed to be negligible; the targets were then set the same way as (3,4)
While these assumptions are fairly rough (especially for 3 - 6), since these occupancy types are relatively rare in urban areas the impact of the uncertainty created is expected to be rather small.
The issue will be closed as soon as Pull request #464 is accepted.


OpenBSD 7.6 build does not work

     - name: Cross-platform test
        uses<EMAIL_ADDRESS>        with:
          operating_system: openbsd
          version: 7.6"

This fails (build):
  Downloading disk image: https://github.com/cross-platform-actions/openbsd-builder/releases/download/v0.8.0/openbsd-7.6-x86-64.qcow2
  Downloading hypervisor: https://github.com/cross-platform-actions/resources/releases/download/v0.11.0/qemu-system-x86_64-linux.tar
  Downloading resources: https://github.com/cross-platform-actions/resources/releases/download/v0.11.0/resources-linux.tar
  /usr/bin/truncate -s 40m /tmp/resourcesDmSzzb/res.raw
  /usr/bin/ssh-keygen -t ed25519 -f /tmp/resourcesDmSzzb/id_ed25519 -q -N
  /usr/bin/sudo losetup -f --show --offset 0 --sizelimit 0 /tmp/resourcesDmSzzb/res.raw
  /dev/loop3
  /usr/bin/sudo mkfs.fat -F32 /dev/loop3
  mkfs.fat 4.2 (2021-01-31)
  /usr/bin/sudo mount -o uid=1001 /dev/loop3 /tmp/resourcesDmSzzb/mount/RES
  /usr/bin/sudo umount /tmp/resourcesDmSzzb/mount/RES
  /usr/bin/sudo losetup -d /dev/loop3
  Error: Unexpected HTTP response: 404
  Downloaded file: /home/runner/work/_temp/3e912f2a-5ff7-4dec-a8d1-69ea77a79c62
  Downloaded file: /home/runner/work/_temp/54200c75-fcf5-4c6c-be61-50f8d5c27130

After reading #6, I've rechecked that os version is quoted in YAML
OpenBSD 7.5 work fine (build), as do FreeBSD 14.1, so the problem may be specific to OpenBSD 7.6

uses<EMAIL_ADDRESS>does not support OpenBSD 7.6. You can use the latest git: uses: cross-platform-actions/action@595d543e5d1aeb6d48c0a176965028afe56c7018.

Yes, a new version of an operating system always requires a new release of cross-platform-actions.


Don't exit 1 on benchmark error

Signed-off-by: Raul Sevilla<EMAIL_ADDRESS>Description
We shouldn't directly exit 1 on benchmark error. return 1 instead will let the parent script perform the benchmark cleanup before exiting.
Fixes

/rerun all

Results for e2e-benchmarking CI Tests

Workload
Test
Result
Runtime

router-perf-v2
./ingress-performance.sh
PASS
00:06:09

etcd-perf
./run_etcd_tests_fromgit.sh
PASS
00:03:18

scale-perf
./run_scale_fromgit.sh
PASS
00:15:06

network-perf
./run.sh
PASS
00:06:18

kube-burner
WORKLOAD=cluster-density ./run.sh
PASS
00:05:32

kube-burner
WORKLOAD=node-density ./run.sh
FAIL
00:00:19

kube-burner
WORKLOAD=node-density-heavy ./run.sh
FAIL
00:00:18

kube-burner
WORKLOAD=pod-density-heavy ./run.sh
PASS
00:06:13

kube-burner
WORKLOAD=max-services ./run.sh
PASS
00:04:51

kube-burner
WORKLOAD=max-namespaces ./run.sh
PASS
00:05:24

upgrade-perf
./run_upgrade_fromgit.sh
PASS
00:00:57

/rerun all

Results for e2e-benchmarking CI Tests

Workload
Test
Result
Runtime

router-perf-v2
./ingress-performance.sh
PASS
00:05:57

etcd-perf
./run_etcd_tests_fromgit.sh
PASS
00:02:56

scale-perf
./run_scale_fromgit.sh
PASS
00:12:38

network-perf
./run.sh
PASS
00:05:35

kube-burner
WORKLOAD=cluster-density ./run.sh
PASS
00:05:59

kube-burner
WORKLOAD=node-density ./run.sh
PASS
00:05:06

kube-burner
WORKLOAD=node-density-heavy ./run.sh
PASS
00:06:00

kube-burner
WORKLOAD=pod-density-heavy ./run.sh
PASS
00:07:02

kube-burner
WORKLOAD=max-services ./run.sh
PASS
00:05:07

kube-burner
WORKLOAD=max-namespaces ./run.sh
FAIL
00:05:10

upgrade-perf
./run_upgrade_fromgit.sh
PASS
00:00:58

The failures in the runs are from touchstone not any code changed by this PR. They are also "random" to when they fail and need to be looked at outside this PR. Merging this as the code is good.


fix: allow building eigen for oss contributors

This PR resolves MOPLAT-6
Description

Resolves a few issues preventing building and running for OSS contributors.

Adds a workaround for fonts by adding some phony font files and using system fonts instead
Adds an OSS flag
Adds instructions for getting a Mapbox token
Don't initialize firebase app for open source contributors
Don't initialize unleash for open source contributors
Don't initialize sentry for open source contributors

Still to do:

[ ] Add back the google files + add some phony files to allow compilation for OSS
[ ] Add the old open source client id and secret to .env.example
[ ] Clean up some of the handling, don't love the unleash handling and scattered if checks
[ ] Handle failing relay requests once logged in (friendly error message?)
[ ] Document known issues with running this way

PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Allow compiling and running app for open source contributors - Brian

Need help with something? Have a look at our docs, or get in touch with us.

just adding this issue that was opened yesterday cause its related

👏👏👏

🔥 Nice one! Thanks @brainbicycle for taking this on.


fix: Opensearch serverless vector search collections - remove default _id

Feature or Bugfix

Bugfix

Detail

Remove redundant _id generation as it is generated by OpenSearch by default.
Additionally, passing _id is not supported  by OpenSearch vector search collections (see PUT <index>/_create/<id> (for search collection types only):

opensearchpy.helpers.errors.BulkIndexError: ('3 document(s) failed to index.', [{'index': {'_index': 'dummy-vector-search-1', '_id': '32300c88-a52c-4a14-95f6-aad4919b1b7b', 'status': 400, 'error': {'type': 'illegal_argument_exception', 'reason': 'Document ID is not supported in create/index operation request'}, 'data': {'name': 'John'}}}, {'index': {'_index': 'dummy-vector-search-1', '_id': '4c5f5d5d-6337-460f-9efc-31f4c1dd5df1', 'status': 400, 'error': {'type': 'illegal_argument_exception', 'reason': 'Document ID is not supported in create/index operation request'}, 'data': {'name': 'George'}}}, {'index': {'_index': 'dummy-vector-search-1', '_id': '0f924638-c83b-43b1-8c21-de42f6e3a22f', 'status': 400, 'error': {'type': 'illegal_argument_exception', 'reason': 'Document ID is not supported in create/index operation request'}, 'data': {'name': 'Julia'}}}])

Relates

Community Slack convo

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 598b9921b18af15415281a29f9047b386d5779ec
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 2c50c448ba9f9f027dc42e5b957714c65ec8eaaa
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 598b9921b18af15415281a29f9047b386d5779ec
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 2c50c448ba9f9f027dc42e5b957714c65ec8eaaa
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Fractional Differencing support in TimeSeriesDifferentiator

Many time series are both non stationary and have a long memory. For these types of processes an order 1 difference removes a lot of the useful information for forecasting. This can be overcome with fractional differencing e.g.:
https://github.com/fracdiff/fracdiff
Can fractionally differencing be supported by TimeSeriesDifferentiator?

Hi @aptperson,
Thanks for the suggestion. At skforecast we are always happy to try new ideas.
I am not familiar with fractional differentiation. Do you know what the process is for undoing it once the predictions have been generated? This would be key to integrating it into skforecast.


Add glide

Changes Unknown when pulling eb9ab9b834f2c40943c53f1a95aa74c7423f92a5 on add-glide into ** on master**.

close #9


[ja] add translation of the index page of collector

Docs PR Checklist

[X] This PR is for a documentation page whose authoritative copy is in the
opentelemetry.io repository.

@open-telemetry/docs-maintainers et al. - I suggest that we hold back on merging this PR until I've addressed:

#4825

We could just patch things for this PR (by adding the ja page path to the appropriate script), but I'd rather take a little extra time to fix this for all localizations.

@katzchang yoroshiku desu


Root CMake structure does not reflect content of g3sinks and builds only a subset of the sinks

done in https://github.com/KjellKod/g3sinks/pull/108


Flutter debugger bug with bool value null

With the following code that triggers exception "Failed assertion: boolean expression must not be null", the exact code line where exception happened is not indicated. When debugging under Android Studio, set breakpoint at the line commented with "// crashes here under debugger!", then press F8 to step through the line. Bam! Crash! Never got to the real offending line just below that. One might think: "What's wrong, does the string contains('...') function have a bug???"
Also remove the above breakpoint and set a breakpoint just below the commented line, where the actual crash actually happens due to testBool being null, then press the FAB + button - the crash happens apparently before this breakpoint is hit, the debugger never stops on the offending line.
Very confusing in a more complex code.
class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;
  static bool testBool;

  @override
  void initState() {
    super.initState();
    SharedPreferences.getInstance().then((sharedPrefs) {
      testBool = sharedPrefs.getBool("testBool"); // testBool will be null
      setState(() {});
    });
  }

  void testBug(String type) {
    String otherStr = '';
    if (type.contains('application/atom+xml')) {
      otherStr = 'abc';
    } else if (type.contains('image/')) { // crashes here under debugger!
      otherStr = testBool ? 'one' : 'two';
      print(otherStr);
    }
  }

  void _incrementCounter() {
    testBug('image/png');
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {...}
}

I also have a demo project zipped, don't see anything to attach a file to bug report. You may download the ZIP file instead from:
http://hyperionics.com/downloads/test/bool_null_bug.zip
Flutter doctor output from my machine (I'm working from master channel, but tested dev and beta channels, same problem):
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel master, v0.4.4-pre.8, on Microsoft Windows [Version 10.0.16299.431], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[√] Android Studio (version 3.1)
[!] IntelliJ IDEA Ultimate Edition (version 2018.1)
X Flutter plugin not installed; this adds Flutter specific functionality.
X Dart plugin not installed; this adds Dart specific functionality.
[!] VS Code, 32-bit edition (version 1.19.3)
[√] Connected devices (2 available)
! Doctor found issues in 2 categories.

The assertion is generated from the ternary expression in testBug, where testBool is null.  bools in Dart are objects, so they can be null however null itself is not considered 'falsy'.
To avoid situations like this, you can initialize testBool with a default value and avoid setting it if the returned preference is null

I learned about this the hard way yesterday, wasting 2 hours due to the bug in the debugger described.

I understand this was frustrating for you - it's very hard to catch errors that one isn't even aware are possible (especially with type names like bool instead of Boolean).  I would recommend checking out the dart language tour for more information.
Ideally, additions to the type system like nullability and non-null by default could catch errors like this more easily in the future, but we're not quite there yet.

Thank you, I did it once, I'll do it again. However I don't think you've read my bug report beyond the first paragraph. So here is the gist: with these two lines, and the problem with testBool value being null actually in the 2nd line below, the debugger crashed in the first one. If I set the  breakpoint on the 2nd line below, the breakpoint was never reached before the crash:
} else if (type.contains('image/')) { // crashes here under debugger!
  otherStr = testBool ? 'one' : 'two';

This is what was VERY confusing. I thought that there is something wrong with String contains() function, replacing it with other variants (e.g. indexOf(...) > -1) and racking my brain in vain.

Can be closed


WIP: Test

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Test something against prow.
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
Does this PR introduce a user-facing change?

None

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 41.27%. Comparing base (11d77f4) to head (6c9dd7c).
Report is 206 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2259      +/-   ##
==========================================
- Coverage   45.50%   41.27%   -4.23%
==========================================
  Files          79      109      +30
  Lines        7782    15653    +7871
==========================================
+ Hits         3541     6461    +2920
- Misses       4099     8724    +4625
- Partials      142      468     +326


Example code for the Collaborative Logo builder?

The collaborative logo builder does exactly what I need to do, but I can't find the source code for that example.
Can someone point me in the right direction?

Hello @waynebloss , we didn't open sourced the collaborative logo builder yet. I need to do some code cleanup first and I'll open source it before the end of week.


krr version information hasn't been updated since v1.1.1

Describe the bug
The krr version information stored in the follow locations has not been bumped since the release of v1.1.1:

https://github.com/robusta-dev/krr/blob/main/robusta_krr/__init__.py#L3
https://github.com/robusta-dev/krr/blob/main/pyproject.toml#L3

This results in wrong package metadata and incorrect output of the version command:
~/git/krr ((HEAD detached at v1.2.1))$ python krr.py version
1.1.1

To Reproduce
Steps to reproduce the behavior:

Clone the git repo
Checkout a specific version tag > 1.1.1 via git checkout tags/v1.2.1
Run the version command: python krr.py version

Expected behavior
The version information and package metadata should be updated on every release.

Hey, @danielhass !
I think we are only updating the version in pipelines before packing it in standalone app, but that's confusing.
Thanks from pointing it out, we will fix it


qshape in Alpine Linux

Hello,
Having a boky/postfix container running as part of a docker based software solution, I stumbled upon a situation where e-mails sent by the container got "throttled" by a Microsoft Exchange relay.
What I need to do now is manage the mail queue and, although I can use the mailq, postqueue, postcat and postsuper commands, I couldn't find a way to have the qshape command available on Alpine Linux. Issuing  the command apk info -a postfix reveals that, indeed, the qshape command is not provided by the package.
Even though the available commands do allow me to manage the postfix queue, they are missing the global overview qshape would give me on the situation. I've been looking around the Internet to figure out whether the qshape command, which is part of the postfix package in most distros, can be installed on Alpine Linux but couldn't find much info about it.
Could anyone let me know whether it is possible to have qshape on Alpine Linux?
Thanks a lot.

It seems that it's going to be added to Alpine soon: https://gitlab.alpinelinux.org/alpine/aports/-/issues/11783
You could probably try installing it manually via the attached patch.

Oh, Great!
And, once again, thank you @bokysan for your incredibly swift answer!
Best regards.


Use Linux type 'Other' instead of 'Ubuntu' for Ubuntu Bionic

Uploading an Ubuntu Bionic stemcell will lead to changed kernel boot parameters on alicloud for Ubuntu Bionic images. For example, the kernel boot parameter ipv6.disable=1 will be removed.
As discussed with alicloud team we will change the Linux type from 'Ubuntu' to 'Other' so that the original kernel boot parameters will remain.
Related to https://github.com/cloudfoundry/bosh-linux-stemcell-builder/issues/132
/cc @xiaozhu36

HI @StefanWutz This issue has been fixed by #124 and contained by release 34.0.0. Please have a check.


Fix doctests and examples and add them to CI

Currently the CI will only run code tests
cargo test --tests --all-features
But it should probably be changed to run
cargo test --all-features
This requires that the all the documentation compiles, and that the examples compiles.
This would allow unit testing of small helper functions in doctests, eg.
    /// Returns `true` if `needle` is a suffix of the Vec.
    ///
    /// Always returns `true` if `needle` is an empty slice.
    ///
    /// # Examples
    ///
    /// ```
    /// use heapless::Vec;
    /// use heapless::consts::*;
    ///
    /// let mut v: Vec<_, U8> = Vec::new();
    /// v.push("a").unwrap();
    /// v.push("b").unwrap();
    ///
    /// assert!(v.ends_with(&[]));
    /// assert!(v.ends_with(&["b"]));
    /// assert!(v.ends_with(&["a", "b"]));
    /// assert!(!v.ends_with(&["a", "b", "c"]));
    /// assert!(!v.ends_with(&["b", "a"]));
    /// assert!(!v.ends_with(&["a"]));
    /// ```
    pub fn ends_with(&self, needle: &[T]) -> bool
    where
        T: PartialEq,
    {
        let (v, n) = (self.len(), needle.len());
        v >= n && needle == &self[v - n..]
    }

Note that some code in doctests can be hidden from the docs: https://doc.rust-lang.org/rustdoc/documentation-tests.html Can be quite useful if the test requires setup code that you don't want to show in the docs.

Ahh, thats a neat feature! Will remember that, when i come around to fixing this issue..
I think the examples are gonna be the worst one though.. I am not sure what is wrong, but it seems they are not catching the compile target? is .cargo/config unused for examples?

Ohh nvm. The problem is of course that running tests, the target becomes my host, where i can't compile RTFM for.
Any way to exclude single examples from tests?

You can run the tests with a --target parameter. That should work, right?
You could also create a rtfm Cargo feature that would be required for those tests, see https://github.com/stm32-rs/stm32l0xx-hal/blob/master/Cargo.toml#L273-L275.

--target thumbv7m-none-eabi doesn't seem to work here.
The required-feature thing was new to me, thats a nice feature!


renaming err as error to log createServer errors correctly

err is defined as "err" but being logged to the console as "error" on line 164

Thanks! I'll release 3.0.1


Fix INSERT_FIRST bug for http filter

HTTP_FILTER don't support INSERT_FIRST due to implement bug
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[] Test and Release
[x] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@zyfjeff: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
21bcab56428d7c3e7fc57a615ac2a584c75945b7
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zyfjeff: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
21bcab56428d7c3e7fc57a615ac2a584c75945b7
link
/test release-notes_istio

integ-multicluster-k8s-tests_istio
21bcab56428d7c3e7fc57a615ac2a584c75945b7
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mandarjog ping

LGTM

@mandarjog ping you

@ramaraochavali if we need to check for matches in insert-first I think we need to do the same for “add”. Wdyt?

@mandarjog @ramaraochavali
INSERT_BEFORE and INSERT_FIRST have a lot of similar processing logic, so they are combined for processing. It's also possible to do things separately like ADD. I could do it this way.

if we need to check for matches in insert-first I think we need to do the same for “add”. Wdyt?

Agree.

@mandarjog Please review it again. Now we're doing the same thing as "ADD".

@zyfjeff can you help me understand why this is needed when processing INSERT_FIRST.
By the time this code is called, filterchain and everything higher level is already matched.
This code is iterating over the list of http filter inside the currently selected hcm to find a position to insert. INSERT_FIRST only cares about hcm and does not care about trying to find a subFilter name within hcm.    ADD is already doing the right thing, and INSERT_FIRST is ADD at the other end of list.   ADD (append) and INSERT_AFTER are related in a similar way to INSERT_FIRST and INSERT_BEFORE.
			// find the matching filter first
			insertPosition := -1
			for i := 0; i < len(hcm.HttpFilters); i++ {
				if httpFilterMatch(hcm.HttpFilters[i], cp) {
					insertPosition = i + 1
					break
				}
			}

https://github.com/istio/istio/blob/cfb74ed8abf1f644e74ebcbefc685512b5e6f0c6/pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch.go#L434
So we do not need to check subFilter matches in INSERT_FIRST, just like we do not need it for ADD.
I think INSERT_FIRST implemented like ADD is much cleaner.

@ramaraochavali Is there anything that needs to be improved?


[Issue] Bug concerning the Expandable attribute

When I select multiple gameobjects of the same type, the references to the scriptableObjects they contain are corrupted. This happens in case I use the Expandable attribute.
Example on this link: https://fromsmash.com/Expandable-Issue

Fixed in 3944fec482fe883c70394bc8b2c680bb720578bd


Remove unnecessary Option wrapping in Config.rs

Expanding on #367
The following fields have type Option<bool> and Option<Vec<_>> in various types in config.rs.
We can drop them all and use serde::default for all of them and make our Config a lot more compact.
pub interface: Option<bool>
pub scalar: Option<bool>
pub list: Option<bool>
pub required: Option<bool>
pub list_type_required: Option<bool>
pub omit: Option<bool>
pub list: Option<bool>
pub required: Option<bool>

pub implements: Option<Vec<String>>
pub variants: Option<Vec<String>>
pub match_path: Option<Vec<String>>

/bounty $150

/attempt #371

/attempt #371

/attempt #371

/attempt #371


Add datepicker polyfill

Closes #49.
It does spit a error out that window is not defined but it still works 🤷♂

lol "still works"

Maybe I need to fix that error because Travis does not like it...

@isiah-lloyd did npm test fail locally?


License

This code is under what license? Would you license it under GPL v3?

this code is under no license, you can use for free.

Having no license doesn't mean it is free to use. You must specify under what license your code can be used. My suggestion is to go for GPL to make sure your code remains free and wont be used on any non-free project.

@ahangarha thx, I add MIT LICENSE for this project

Tnx
On جمعه ۱۴ دسامبر ۲۰۱۸، ۱۹:۲۴ 梅本金<EMAIL_ADDRESS>wrote:

@ahangarha https://github.com/ahangarha thx, I add MIT LICENSE for this
project
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/meibenjin/GoogleSearchCrawler/issues/13#issuecomment-447366431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKuHY3hp7f_MJGZEiSmT6Hm7W03onIoUks5u48mdgaJpZM4ZNEBl
.

@ahangarha  can you help me test this tool？ I can't access google for now，just type python2.7 gsearch.py 'google' in terminal

It works for me. This is what I get:
➜  GoogleSearchCrawler git:(master) python gsearch.py "GoogleSearchCrawler"
url	-> https://github.com/meibenjin/GoogleSearchCrawler
title	-> GitHub - meibenjin/GoogleSearchCrawler: a tool for crawl Google ...
content	-> Google Search Crawler.

 This is a simple google search results crawler.

 ... A html parser to extract search results from Google.

url	-> https://github.com/duyetdev/google-search-crawler/blob/master/crawl.py
title	-> google-search-crawler/crawl.py at master · duyetdev/google-search ...
content	-> Input list of keywords, crawler top (url, title, description) data from Google Search.

- duyetdev/google-search-crawler.

url	-> https://www.google.com/webmasters/tools/crawl-stats
title	-> Google Search Console
content	-> Use Search Console to monitor Google Search results data for your properties.

...

I will do it tonight. I need to build a system on top of it. So I will test if it works
…
On جمعه ۱۴ دسامبر ۲۰۱۸، ۱۹:۴۷ 梅本金 @.***> wrote: @ahangarha https://github.com/ahangarha �can you help me test this tool？ I can't access google for now，just type python2.7 gsearch.py 'google' in terminal — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#13 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AKuHY2qzk4Krel4Vt8C4uSGnKYmUhrrJks5u488ngaJpZM4ZNEBl .

could you tell me more details about the system you are build？

I want to make a simple script to monitor the position of a particular website for a list of keywords. It is to track the efficiency of SEO related activities.


Replace iTextSharp with another option

Is your feature request related to a problem? Please describe.
Replace the dependency of iTextSharp.
Describe the solution you'd like
The solution should be based on the result of this POC-dotnet-ExtractPdfContent

Use DocNet

Use PdfPig


An error occurred while installing .NET (6.0): .NET Acquisition Failed: Installation failed:

Does this occur consistently?  Yes
Repro steps:
Opened a bicep file and visual studio code automatically initiated an attempt to install .NET runtime v6.0
Output:
2022-01-25T01:21:47.726Z info: Current log level: debug.
2022-01-25T01:21:47.743Z info: Launching Bicep language service...
2022-01-25T01:21:47.743Z info: Acquiring dotnet runtime...
Error: Failed to install .NET runtime v6.0.
2022-01-25T01:22:01.362Z error: Failed to install .NET runtime v6.0.
2022-01-25T01:22:01.362Z error: Failed to install .NET runtime v6.0.
Action: bicep.activate
Error type: Error
Error Message: Failed to install redacted:url runtime v6.0.
NET Acquisition Failed: Installation failed: C:\Users\XXXXXXX.vscode\extensions\ms-dotnettools.vscode-dotnet-runtime-1.5.0\dist\install
scripts\dotnet-install.ps1 : Exception calling "ExtractToFile" with "3" argument(s): "The process cannot access the
file 'C:\Users\XXXXXX\AppData\Roaming\Code\User\globalStorage\ms-dotnettools.vscode-dotnet-runtime.dotnet\6.0.
1\shared\Microsoft.NETCore.App\6.0.1\System.Linq.Expressions.dll' because it is being used by another process."
Version: 0.4.1124
OS: win32
OS Release: 10.0.19043
Product: Visual Studio Code
Product Version: 1.63.2
Language: en

Call Stack
extension.js:2:1960109extension.js:2:1960109
o extension.js:2:1945025
processTicksAndRejections task_queues.js:93:5

Closing as dup of #2483. Please follow the steps there to report this issue to the vscode-dotnet-runtime repo


Coalesce non-vectored get with vectored code (use vectored get for single key gets)

Anticipate some performance work to make vectored get equally fast for single key case (e.g. https://github.com/neondatabase/neon/issues/6932)
### Ordered list of changes
- [ ] https://github.com/neondatabase/neon/pull/7486
- [ ] https://github.com/neondatabase/neon/pull/7490
- [ ] https://github.com/neondatabase/neon/pull/7477
- [ ] https://github.com/neondatabase/neon/pull/7498

Last week:

this shook out a bug that was fixed.
investigated why it's slower in some cases.

This week:

A fix for difference in how we handled aux files
Continue optimizing

Last week:

Merged aux file handling fix
Subtle fix for materialized page cache usage
Optimised
Benchmarking - looks on par with legacy read path (even a bit better)
Ran with validation in staging over the weekend - no correctness issues

This week:

Run in staging without validation to qualify perf
Start staged roll out if above is ok

Last week:

Qualified perf & correctness in staging/pre-prod

This week:

Deploy to ap-southeast regions & monitor

Last week:

Deployed to ap-southeast-*. Small latency increased, but tput was much higher. CPU utilisation was better

This week:

Deploy to us-east-*

#7939 should be reverted when the default is changed.

Rollout was completed a few weeks ago. At this point, the non-vectored read path is no longer in use (apart from validation which will be removed by #8005)


option to replace attestations

Description
Provide a way to replace (vs. append) attestations.
Currently the same attestation type can be appended multiple times. A proposal is to support a --replace which would change the behavior to replace an attestation of the same type instead of appending it to the existing list of attestations.
Slack discussion at: https://sigstore.slack.com/archives/C01PZKDL4DP/p1633463916305100.

Thanks, @dlorenc! Please assign to me and I will work on a PR.

amazing thought @JimBugwadia, we (w/@dentrax @erkanzileli) are available for a while, so, we can work on this issue if you let us do that 🤩

@developer-guy  please proceed, thanks!


Training with 10 epochs

As a first step, tried training with 10 epochs :
Got the following error after each step
('::Step::9,10558', '::total_loss::80117.11', '::time::0.235623836517')
('::Step::9,10559', '::total_loss::29414.088', '::time::0.234266996384')
('::Step::9,10560', '::total_loss::104685.22', '::time::0.23069190979')
('::Step::9,10561', '::total_loss::26441.73', '::time::0.237238883972')
('::Step::9,10562', '::total_loss::9810.764', '::time::0.237290143967')
('::Step::9,10563', '::total_loss::226181.88', '::time::0.236012935638')
('::Step::9,10564', '::total_loss::62022.016', '::time::0.237781047821')
('::Step::9,10565', '::total_loss::88922.95', '::time::0.235167980194')
('::Step::9,10566', '::total_loss::243382.31', '::time::0.234927177429')
('::Step::9,10567', '::total_loss::77744.95', '::time::0.2362241745')
('::Step::9,10568', '::total_loss::86635.64', '::time::0.236945867538')
('::Step::9,10569', '::total_loss::100394.62', '::time::0.237506866455')
('::Step::9,10570', '::total_loss::117557.22', '::time::0.236845970154')
('::Step::9,10571', '::total_loss::23898.738', '::time::0.234770059586')
('::Step::9,10572', '::total_loss::107994.164', '::time::0.241539955139')
('::Step::9,10573', '::total_loss::110407.42', '::time::0.23698091507')
('::Step::9,10574', '::total_loss::114145.55', '::time::0.235699892044')
('::Step::9,10575', '::total_loss::61390.625', '::time::0.234794855118')
('::Step::9,10576', '::total_loss::45022.875', '::time::0.236536026001')
('::Step::9,10577', '::total_loss::95963.21', '::time::0.237265110016')
('::Step::9,10578', '::total_loss::93546.28', '::time::0.235880851746')
('::Step::9,10579', '::total_loss::97628.5', '::time::0.237339019775')
('::Step::9,10580', '::total_loss::79147.49', '::time::0.236726045609')
('::Step::9,10581', '::total_loss::15524.831', '::time::0.238878011703')
('::Step::9,10582', '::total_loss::21033.553', '::time::0.237198829651')
ERROR

Try printing out the error specifically by changing the exception code to:
except Exception as e:
print("ERROR:: " + str(e))


[5.1] Revert iron changes

Because of this the documentation needs to be updated. Firstly the version it says is ~2.0|~4.0. Which will put you on 4.0 which simply doesn't work because of #10828. Secondly I can't even find the documentation on how to use their old API on their site or anywhere so far. Which means with Laravel 5.1 you can't use iron.io if you start a new project in iron.io now.

Thanks @robclancy. We'd accept a PR to the docs to point out that the laravel collective package can be used for newer versions, and L5.1 won't ship with support, and L5.2 has had all support dropped.
NB That's assuming laravel collective have actually dealt with that stuff as was the pan.

Laravel 5.1 is LTS. You can't just have something stop working and say "nope we don't have that issue in the next version".

Not to mention that https://github.com/LaravelCollective/iron-queue has this exact issue so upgrading to 5.2 isn't even an option (wasn't doing it anyway, I am on LTS for a reason).
It needs to be fixed in both places, why was the original fix removed?

The original fix was removed because it was a mess and was broken, mainly due to the terrible versioning system and backwards structure of the ironmq php library.

Is this also the reason support was dropped? Do you have an issue where the issues you had are explained? I need to recreate this fix properly.

Unfortunately, it's scattered about a bit, sorry.

Maybe just talk to Taylor on slack about this?

Are the people at LaravelCollective very active? Best solution is it being fixed there and 5.1 docs updated to explain what happened and to use the package instead.

Are the people at LaravelCollective very active?

I can't really comment, lol.

Best solution is it being fixed there and 5.1 docs updated to explain what happened and to use the package instead.

As I said, you'll have to convince Taylor, not me. ;)

I think they are fairly active over there.


OSX arm64 build?

Comment:
There's no arm build for this feedstock, so it can't be installed on Apple Silicon chips. (brew provides an arm build, so one should be feasible, but is beyond me.) Any chance of an arm build?

Sorry I missed this one. I just opened a PR here: https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/5992

closed by #12


[1.16.5] Disabling Ore Generation doesn't actually disable it. (though only sometimes)

Describe the bug
When disabling the generation of Copper or Tin it doesn't disable it. (Though only sometimes, which is weird.)
To Reproduce [Optional]
Steps to reproduce the behavior: (Not really sure how to reproduce as its not happening 100% of the time.

Disable Copper, Tin Ore generation in configs.
Make a world
Still able to find copper and tin.
Annoying.

Expected behavior [Optional]
That they're disabled like in the config.
MC: 1.16.5
Forge: 36.0.13
Crossroads version: 2.6.3
- Gbergz

I can confirm. The issue doesn't appear to be seed dependent, as I was able to create two worlds with the same seed and only one generated Crossroads ores.
If it helps, I've uploaded two worlds. One has ores, one doesn't. They were made with the same pack and the same seed. Maybe there's some useful info hidden in the level.dat file or something.
No ores.zip
Yes ores.zip

For the world which generated with ores even when they were disabled, did you load any other worlds before creating that one? For example, did you open a pre-existing world, play it for a bit, close it, then create the new world that shouldn't have had ores, all without restarting MC?
Trying to narrow down the possible cause.

The bug triggers upon reloading a world without restarting a pack. After closing to the title screen and entering a world, every world will generate ores. Old and new worlds are both affected.

Also found out that a .bak file gets generated each time a worlds generate. Might have something to about this issue.

Gets generated each time I enter the world.

If anyone needs/wants a mod file with this bugfix before I've published it on CurseForge, message me on the Crossroads discord and I can send you a compiled file.

Any ETA when it's uploaded to CurseForge?

@Gbergz Now. https://www.curseforge.com/minecraft/mc-mods/crossroads-mc/files/3246384

Seems to not be fixed. Still happening.

Is the ore generation fixed? Because that should be working now. It's possible it was two separate issues.

Not fixed. Config only works when creating a new world on first launch. If you quit from that world and create another. Disabled Ores generate again. (Config isnt respected) Like I originally reported.

Also there's a server config in normal config folder + a server config in the serverconfig folder?

I am on the latest version. Forge 36.1.2 as am writing this.
Crossroads version: 2.6.4

Also, which config am I suppose to even edit. The one that gets generated when creating a world inside the "serverconfig" folder (crossroads-server.toml). Or the exact same one in the regular config folder (crossroads-server.toml) ?. Right now I've edited the files in both locations. (Carried over the one in serverconfig to defaultconfigs folder)

It's the new Forge config system that keeps confusing people. Every world has its own set of config files (in the serverconfig folder).  When you make a new world, it copies the config in the defaultconfigs folder into the newly created world's serverconfig file.
So, for existing worlds, modify the serverconfig, and for changing the config before creating a new world, modify the defaultconfig file.
Is this unnecessarily confusing? Yes. Is it completely unexplained? Yes. It's the new Forge config system that allows every world to have its own set of configs.

Yeah thanks. I know and understand the new config system. But there's two of the exact same file. The one in /config/ like old times. And another one in /serverconfig/. (Both are crossroads-server.toml) Which is a first. Usually it's client config in /config/ & server side configs in serverconfig. Or there's nothing in the serverconfig folder and everything is in /config/ like old times. Hope you understand.

Totally unable to reproduce this issue at this point.
All I can suggest is that the \config\crossroads-server.toml is not correct. Correcting message, and large number of .bak files, are generally associated with incorrect config files. It's possible the config files themselves are malformed; deleting every crossroads config file (/serverconfig/, world specific, general, /config/, .toml, and .toml.bak; all of them) might fix it for you, but that's more of a guess than a guarantee.

They're completely newly generated with only Tin Ore & Copper Ore disabled. I'm at a loss aswell and don't know why it's happening. Oh well!

Disabled ores are now generating even on fresh worlds right after a pack restart. I can't find any situation where the config setting applies correctly.

I am aware that multiple times in the past, I've said 'it's fixed, for real this time', and each time I was wrong.
However, hubris notwithstanding, it's fixed for real this time.
This issue should be fixed with Crossroads 2.6.5.
If updating doesn't fix the error, please re-open this issue.

I'm still having the same issue...
Crossroads 2.6.5
Forge 36.2.33
I disable tin and copper gen in the main crossroads-server.toml file in the config folder, but they still generate in the world.
I've started multiple new worlds to test.
In each of the worlds serverconfig/crossroads-server.toml files it shows the ores as enabled, not disabled, despite having it disabled in the main settings file.

can confirm the problem, but on 1.19.2


Give the possibilities to change nickname

Fix the problem that doesnt want to change the nickname of the player

Lol., fail..
Thanks!

Not working for me :( SQL Error
Message: SQLSTATE[HY093]: Invalid parameter number: parameter was not defined
Query-Code:UPDATE uni1_users SET username = BassX, uctime =<PHONE_NUMBER>t WHERE id = 1;
File: /includes/classes/Database.class.php
Line: 109
URL: http://ddclan.mrtv.biz/game/game.php?page=settings
PHP-Version: 5.5.9-1ubuntu4.14
PHP-API: fpm-fcgi
MySQL-Cleint-Version: mysqlnd 5.0.11-dev - 20120503 - $Id: bf9ad53b11c9a57efdb1057292d73b928b8c5c77 $
2Moons Version: 1.8.git
Debug Backtrace:
#0 /includes/classes/Database.class.php(160): Database->_query('UPDATE %%USERS%...', Array, 'update')
#1 /includes/pages/game/ShowSettingsPage.class.php(283): Database->update('UPDATE %%USERS%...', Array)
#2 /includes/pages/game/ShowSettingsPage.class.php(138): ShowSettingsPage->sendDefault()
#3 /game.php(57): ShowSettingsPage->send()
#4 {main}

Search for timestampt and replace it with timestamp on the SettingsPage.php

working now perfect! But how can I change it back again?

Maybe you can create a pull reqest for this fix, too.?


Grey image view backgrounds are visible in carousel on home screen

This happens when the images have not been cached before.

@katarina: That's just happening before the image has been downloaded. It could be blank white if you prefer, but our default around the entire app is to show a grey box while the image is being downloaded. Once downloaded, it should be cached and available immediately.

@1aurabrown At least for images after the first one we could definitely download the image before automatically going to the next image, no?

We could, the view doesn't exist at that point in time so there's nowhere
to put those images in the UI, but we could hit all those image URLs
beforehand to ensure that they're cached.
On Thu, Mar 5, 2015 at 2:12 PM, Eloy Durán<EMAIL_ADDRESS>wrote:

@1aurabrown https://github.com/1aurabrown At least for images after
the first one we could definitely download the image before
automatically going to the next image, no?
—
Reply to this email directly or view it on GitHub
https://github.com/artsy/eigen/issues/251#issuecomment-77430448.

--
[A.]
Laura Brown
Artsy http://artsy.net/
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Is this still a problem? Even with the network link conditioner set to "Very Bad Network", I still can't reproduce this. Images appear to be completely downloaded when swiped.

Its only when the images have never been downloaded/cached. Possibly yours are cached?Sent using CloudMagicOn Tue, Mar 10, 2015 at 15:32,  artsy/eigen<EMAIL_ADDRESS>wrote:            Is this still a problem? Even with the network link conditioner set to "Very Bad Network", I still can't reproduce this. Images appear to be completely downloaded when swiped.
—Reply to this email directly or view it on GitHub.

Ah, I see. Thanks!


"recv failed: CryptoFail" error happend sometimes

I use quiche to send request. Sometimes it receive  CryptoFail error. I am in windows with IntelliJ. Not always.

I found an error message is happening here. file is examples http-client.rs. execute target\debug\examples\http3-client.exe https://<IP_ADDRESS>:443/Photo.html
many times. error happen.
`// Process potentially coalesced packets.
let read = match conn.recv(&mut buf[..len]) {
Ok(v) => v,
            Err(e) => {
                println!("recv failed: {:?}", e);
                continue 'read;
            },
        };

`

I found an error message is happening here. file is examples http-client.rs. execute target\debug\examples\http3-client.exe https://<IP_ADDRESS>:443/Photo.html
many times. error happen.
// Process potentially coalesced packets. let read = match conn.recv(&mut buf[..len]) {             Ok(v) => v,             Err(e) => {                 println!("recv failed: {:?}", e);                 continue 'read;             },         };


Hello there, good day!
More context
It would be very cool to also implement an encryption for the database at a file level, I found this explanation about it from another embedded database of the C# world.

This is an advanced feature. Will try this in the future.


Confusing statement

Hi,
Are this benefits included in W10 E3 or not?
These benefits include planning services, Microsoft Desktop Optimization (MDOP), Windows Virtual Desktop Access Rights, Windows-To-Go Rights, Windows Roaming Use Rights, Windows Thin PC, Windows RT Companion VDA Rights, and other benefits.
Thank you

@dcpopescu - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.

Hi Joyce,
This is the article:
https://docs.microsoft.com/en-us/windows/deployment/windows-10-enterprise-e3-overview
This part of the document is unclear:
How does the Windows 10 Enterprise E3 in CSP program compare with Microsoft Volume Licensing Agreements and Software Assurance?

Microsoft Volume Licensinghttps://www.microsoft.com/licensing/default.aspx programs are broader in scope, providing organizations with access to licensing for all Microsoft products.
Software Assurancehttps://www.microsoft.com/Licensing/licensing-programs/software-assurance-default.aspx provides organizations with the following categories of benefits:
*   Deployment and management. These benefits include planning services, Microsoft Desktop Optimization (MDOP), Windows Virtual Desktop Access Rights, Windows-To-Go Rights, Windows Roaming Use Rights, Windows Thin PC, Windows RT Companion VDA Rights, and other benefits.
*   Training. These benefits include training vouchers, online e-learning, and a home use program.
*   Support. These benefits include 24x7 problem resolution support, backup capabilities for disaster recovery, System Center Global Service Monitor, and a passive secondary instance of SQL Server.
*   Specialized. These benefits include step-up licensing availability (which enables you to migrate software from an earlier edition to a higher-level edition) and to spread license and Software Assurance payments across three equal, annual sums.

In addition, in Windows 10 Enterprise E3 in CSP, a partner can manage your licenses for you.
In summary, the Windows 10 Enterprise E3 in CSP program is an upgrade offering that provides small- and medium-sized organizations easier, more flexible access to the benefits of Windows 10 Enterprise edition, whereas Microsoft Volume Licensing programs and Software Assurance are broader in scope and provide benefits beyond access to Windows 10 Enterprise edition.
Windows 10 Enterprise E3 doesn’t include MDOP?
Thank you,

From: Joyce Y<EMAIL_ADDRESS>Sent: Wednesday, October 23, 2019 2:27:20 AM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Dan Popescu<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Confusing statement (#5244)
@dcpopescuhttps://github.com/dcpopescu - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/5244?email_source=notifications&email_token=ANPSRYYDX3P4HZZP7TBIMGTQQADGRA5CNFSM4JDRHS22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECARLCI#issuecomment-545330569, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANPSRYYU4PVSFXIQAFJ6LWDQQADGRANCNFSM4JDRHS2Q.

Hello @dcpopescu,
unfortunately no, Windows E3 in CSP doesn't provide listed benefits. It only allows users who have Windows 10 Pro licenses to get Windows 10 Enterprise features via subscription (benefits listed by you are not Windows 10 Enterprise features, they are Software Assurance features).
Thank you

@dcpopescu - From our understanding, the issue you raised has been answered by MaratMussabekov. If you feel it hasn't been answered, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!


[CARBONDATA-4147] Fix re-arrange schema in logical relation on MV partition table having sort column

Why is this PR needed?
After PR-3615, we have avoided rearranging catalog table schema if already re-arranged.
For MV on a partition table, we always move the partition column to the end on a MV partition table. Catalog table will also have the column schema in same order(partition column at last). Hence, in this case, we do not re-arrange logical relation in a catalog table again.
But, if there is a sort column present in MV table, then selected column schema and catalog table schema will not be in same order. In that case, we have to re-arrange the catalog table schema. Currently, we are using  rearrangedIndex to re-arrange the catalog table logical relation, but rearrangedIndex will keep the index of partition column at the end, whereas, catalog table has partition column already at the end. Hence, we are re-arranging the partition column index again in catalog table relation, which leads to insertion failure.
Example:
Create MV on columns: c1, c2 (partition), c3(sort_column), c4
Problem:
Create order: c1,c2,c3,c4
Create order index: 0,1,2,3
Rearranged Index:
Existing Catalog table schema order: c1, c3, c4, c2 (for MV, partition column will be moved to Last)
Rearrange index: 2,0,3,1
After Re-arrange catalog table order: c4,c2,c2, c3(which is wrong)
Solution:
Change MV create order as below
New Create order: c1,c4,c3,c2
Create order index: 0,1,2,3
Rearranged Index:
Existing Catalog table schema order: c1, c3, c4, c2 (for MV, partition column will be moved to Last)
Rearrange index: 1,0,2,3
After Re-arrange catalog table order: c3,c1,c4,c2
What changes were proposed in this PR?
In MV case, if there is any column schema order change apart from partition column, then  re-arrange index of only those columns and use the same to re-arrange  catalog table logical relation.
Does this PR introduce any user interface change?

No

Is any new testcase added?

Yes

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3798/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5564/

@Indhumathi27 please change description with example as discussed

LGTM, @ajantha-bhat please review this once as you have worked on rearrange logic in insert optimization. Please see if there is any impact.

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5573/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3807/

LGTM

retest this please

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3824/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5590/

retest this please

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3839/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5605/

retest this please

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5606/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3840/


update and test migrate_dataset

Issue #, if available:
Description of changes:

include already_migrated_ds creation in migrate_dataset
update deployment_config["target_permission"]["datasetpermission"] in migrate_dataset`
create test_deployment to test migrate_dataset. pass test on cross account migrate dataset

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Merged to feature branch


Add ReceiveSelectAsset page before the address generation

This PR uses the AssetListScreen component from send flow to receive flow in order to add a step before generating wallet addresses

abstract the SelectAsset screen in order to reuse it in send-select-asset.tsx and receive-select-asset.tsx.
add ButtonList component (= use the same scrollbar)
some CSS changes on Home and Transactions views (due to some abstraction in ButtonList).
ATM, the addresses derivation paths do not include the asset type. So all the assets buttons have the same behavior now in receive-select-asset.tsx.

it closes #226
@tiero please review

Lint/prettier

What do you mean by ATM, the addresses derivation paths do not include the asset type. So all the assets buttons have the same behavior now in receive-select-asset.tsx.

I mean that theoretically if u click on BTC or USDT u will get the same address. Same behavior in tdex-app, just want to notice that our Mnemonic Identity does not handle the asset type during address generation.


Use Zend\Authentication\AuthenticationService instead of ZfcUser's

Starting from ZendFramework 2.1, a Zend\Authentication\AuthenticationService should always be available. We should probably move away from the ZfcUser auth service or add a config key to switch the used auth service.

How can I override the AuthenticationIdentityProviderServiceFactory?

@Doltsche ovveride this key of configuration
https://github.com/bjyoungblood/BjyAuthorize/blob/fb81997c2d904da7b7620da78c0962989ea53dec/config/module.config.php#L74


Reconnect to all previous networks

Hi,
This PR make watchtower reload the updated containers with all the previous network it was connected to. (very useful while updating containers launched by docker-compose for example, that are connected to db/backend networks and proxy/frontend networks)
I based my work on your fork because I hoped that you changing the docker client lib to the official would solve my problem automagically. It didn't…  Sorry I couldn't give this any attention sooner. I thought I might get to it over the holidays but no such luck!

No problem, it was holidays afterwards. As long as it is merged somewhere else than my own tiny repo, it will make it available to more people. That's what counts.
Thank you for your patch again. Without you, I think it would have taken much more time.


Update shared workflow branches

This PR updates the branch reference used for our shared workflows.

/merge

Codecov Report

:exclamation: No coverage uploaded for pull request base (branch-23.04@e391989). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff               @@
##             branch-23.04     #510   +/-   ##
===============================================
  Coverage                ?   92.89%
===============================================
  Files                   ?      131
  Lines                   ?     9903
  Branches                ?        0
===============================================
  Hits                    ?     9199
  Misses                  ?      704
  Partials                ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Simplify quota reservation when preemption while borrowing

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
It is a follow up to https://github.com/kubernetes-sigs/kueue/pull/1399 to narrow down the impact of that PR
to only the scenario which is tested by integration tests. See discussion.
It is still not clear what should happen in the scenario when preemption with borrowing is used. In particular, there is a trade-off that letting the borrowing workload B get admitted may prolong waiting time for the workload A in the scenario depicted here.
So, I think it is better to simplify the code (which is already complicated) until we have feedback from users.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/assign @yaroslava-serdiuk @alculquicondor @tenzen-y

With your change, the workload in B would be blocked, whereas it wouldn't in the existing code (because A would only block up to 3 units).
I don't think this is right. A is not entitled to block more than 3 total.

With your change, the workload in B would be blocked, whereas it wouldn't in the existing code (because A would only block up to 3 units).
I don't think this is right.

That is correct, however, there is a trade-off because letting B run we may prolong the wait time for A, possibly even starving A, if there is a sequence of workloads like B.

A is not entitled to block more than 3 total.

On main A would block: nominal+borrowing-cqUsage=0+3-2=1, on the PR it would block usage=3 (so not more that 3 in any case).

That is correct, however, there is a trade-off because letting B run we may prolong the wait time for A, possibly even starving A, if there is a sequence of workloads like B.

And that is the intended behavior. A is already past its nominal+borrowing.

I see, if this is indeed the intended behavior then the code is indeed needed to support it.

I would appreciate a unit test for scheduleOne to clarify the intent.

You mean a new test case in TestSchedule? I think this makes sense, the test seems to provide a reliable way to test Kueue scheduler in more complex scenarios, without the need for integration tests (which are much heavier).

/close
The code cannot be simplified in the proposed way, because it is intended behavior to let B schedule, and keep A waiting. Going to follow up with an additional unit test for the scenario which captures the intention.


ageReferenceSystem

From @dennereed: "maximumChronometricAgeReferenceSystem is really long :-). Perhaps just ageReferenceSystem, and perhaps just a single term that covers both max and min? Will they every really be different? In the case of AD/BC group them as a single value. E.g acceptable entries could  be Ma, Ka, mya, kya, AD-BC, BCE, etc."
Opening this up for further discussion.

Agreed that this is long. I think this one may be up to discipline experts to decide if grouping them is ok, or if we want to keep separate ones for max & min? If we consider "AD/BC" as one unit, the only time I could see them being different would be different dating assays, which would be a different instance of a chronometricDate/Age extension record. However, one caveat to this that I could foresee being a reason to keep them separate is because @tucotuco had mentioned working on building into VertNet a way to make different dates comparable so users could search for age ranges, despite if they are reported using different reference systems.
Also, from our collaborator Michelle LeFebvre: "Would this change have an impact on someone reporting from an age model (I think that is what it is called). Remember Ed's [Ed Davis] concern about age modelling and being able to possibly report this type of data for dates?"

We have decided against this suggestion because if we combine "AD/BC" into one value for a single unit field, how are users supposed to know if the max or minAge is AD or BC? Are they both AD? Are they both BC? Are they one and the other? It is unclear if we combine them into one term.