$model->getPrimaryKey() returns wrong value when yii\log\DbTarget is used for loging

###setup
Model table has "id" column set as primary key, autoincrement
What steps will reproduce the problem?
$model = new Model();
$model->name = 'model name';
$model->save();
//this should return the id of newly inserted row to Model table,
//but returns the id of row inserted  to Log table instead
echo $model->getPrimaryKey();
What is the expected result?
expected is a newly assigned id according to AutoIncremen PK
What do you get instead?
returns the id of row inserted  to Log table instead
Additional info

Q
A

Yii version
2.0.9

PHP version
7.0.5

DB version
MariaDB 5.5

Operating system
Linux apache-data-it11-113 3.14.27-031427-generic #201412170558 SMP Wed Dec 17 11:00:01 UTC 2014 x86_64

config/web.php:
'log' => [
           'traceLevel' => 0,//YII_DEBUG ? 3 : 0,
           'flushInterval' => YII_DEBUG ? 1 : 100,
           'targets' => [
           	[
           		'class' => 'yii\log\DbTarget',
           		'exportInterval' => YII_DEBUG ? 1 : 10,
           		'levels' => ['error', 'warning', 'profile' /*, 'info', 'trace', */],
           		'logVars' => [],
           		'categories' => [
           			'application',
           			'yii\db\*',
           			'yii\web\HttpException:*',
           		],
           		'except' => [
           			/*
           			'yii\web\HttpException:404',
           			'yii\web\HttpException:429',
           			*/
           		],
           		'prefix' => function ($message) {
           			$user = Yii::$app->has('user', true) ? Yii::$app->get('user') : null;
           			$userID = $user ? $user->getId(false) : '-';
           			return json_encode(['userid' => $userID]);
           		},
           	]
           ],
       ]

This is a limitation of PDO: http://php.net/manual/en/pdo.lastinsertid.php not sure how to work around this. One problem could be the exportInterval of 1, which means that the message is written to the database immediately. If you set it to a higher value you can avoid this problem to some extent.

thanks @cebe for verifying. However I think, that setting a higher exportInterval might cause even more weird behavior as this error would arise randomly when the log stack is flushed to DB. Would it be possible to add some log restriction to core to avoid such setting? Or at least echo some warning that this setting causes problems when the $model->id is used as FK for saving subsequent related models?

to solve the issue completely the best way would be to exclude database queries from the logging of the database target.

Note that databases which use INSERT ... RETURNING such as Oracle and PostgreSQL are not affected by this issue.

What do you think about moving getLastInsertID before Yii::endProfile($token, _METHOD_); ?

Using another connection makes sense as a workaround. Does moving getLastInsertID solve issue for all possible cases?

Does moving getLastInsertID solve issue for all possible cases?

No, definitely not. It was not a very good suggestion.

Using another connection makes sense as a workaround.

So, is this about target configuration, right? Or may be we can make a workaround inside of DbTarget which will instantiate new connection?

@AnatolyRugalev yes, if you configure $db property of the DB target to not use 'db' application component but a separate DB connection.

I think the only way to solve this would be to document it so users can adjust the configuration to avoid the problem.

At the Schema::insert() we are passing $sequenceName to the PDO::lastInsertId():
https://github.com/yiisoft/yii2/blob/master/framework/db/Schema.php#L456
However for MySQL it is always empty. Perhaps if we could fetch full sequece name for the table and pass it here there will be such confusion. Although I am unsure if it will work or if fetching sequence name from MySQL is possible.

As far as I know, MySQL PDO does not have a concept like sequence name. last insert id is maintained per connection, not per table or db:

https://dev.mysql.com/doc/refman/5.7/en/getting-unique-id.html
https://dev.mysql.com/doc/refman/5.7/en/information-functions.html#function_last-insert-id


Define general verification flow over Header interface

Verification involves checking untrusted Header against trusted. There are lots of common verification points for any Header implementation:

Chain-id check
Untrusted time after trusted
Untrusted time before now with 10 secs clock drift
Untrusted height is after trusted

Closed by #90


SAK-29168 - Login tool needs updating for Morpheus

I've added a little explanation with screenshots in JIRA
https://jira.sakaiproject.org/browse/SAK-29168
--
I've modified xlogin.vm and access styles on this branch https://github.com/SedueRey/sakai/tree/SAK-29168 to improve a little our login form.
Summarizing:

I've removed all tables to layout the form with proper tags.
I've added an animation if you type an email in user name. The animation loads the user gravatar if available and animate it.

--
Any comments would be appreciated

This looks really good, great work. A couple of minor comments above. And there seems to be a build failure, might be unrelated but I'm happy to merge this once that goes green :)

I'm just updated xlogin. Now you can "xlogin" from gateway without leave the page.

I think this PR is ready to merge.

This looks great @SedueRey .  I am very impressed.
My one question: I see lots of Gravatar code (md5 calculations) in this commit. What is the relevance of Gravatar with this login modification?

There is an animation that shows your gravatar If you type an email as user id, pretty awesome. This is how its look https://jira.sakaiproject.org/secure/attachment/42790/animation_finised.png

Hi,
I though add support to gravatar at first step as long as we  support adding gravatar image on profile preferences.

But I have been thinking in go a step further and try to add, later, a little support for an API that take avatar profiles if they're public but it would be a big big change so I though start through this and habituate users to watch this.
What do you think?

I don't think gravatar should have gone in here. The majority of students
will have no gravatar profile. The integration looks nice but only applies
to a select few people.
On May 22, 2015 2:49 AM, "Juanjo Meroño"<EMAIL_ADDRESS>wrote:

Merged #536 https://github.com/sakaiproject/sakai/pull/536.
—
Reply to this email directly or view it on GitHub
https://github.com/sakaiproject/sakai/pull/536#event-311403369.

I'm wondering if it's worth to make a property to enable or disable this behaviour. Is only affecting instances without SSO (or with but only for xlogin users), with complete emails as user id and, finally, users that has gravatar profile... So, it looks great but also it has sense remove it completely. I'm going to file a jira to remove it, ok?

Okay sounds good
On May 22, 2015 6:45 AM, "Juanjo Meroño"<EMAIL_ADDRESS>wrote:

I'm wondering if it's worth to make a property to enable or disable this
behaviour. Is only affecting instances without SSO (or with but only for
xlogin users), with complete emails as user id and, finally, users that has
gravatar profile... So, it looks great but also it has sense remove it
completely. I'm going to file a jira to remove it, ok?
—
Reply to this email directly or view it on GitHub
https://github.com/sakaiproject/sakai/pull/536#issuecomment-104616034.


fix data race problem

Issues associated with this PR

from the picture we can find the curClientStreams value change to -1, so here change to atomic operation.
Solutions
You should show your solutions about the issues in your PR, including the overall solutions, details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #1319 into master will decrease coverage by 0.12%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1319      +/-   ##
==========================================
- Coverage   54.71%   54.59%   -0.13%
==========================================
  Files         331      331
  Lines       30394    30394
==========================================
- Hits        16630    16593      -37
- Misses      12181    12213      +32
- Partials     1583     1588       +5

Impacted Files
Coverage Δ

pkg/module/http2/mhttp2.go
0.00% <0.00%> (ø)

pkg/network/dns.go
54.76% <0.00%> (-23.81%)
:arrow_down:

...ream/faulttolerance/regulator/default_regulator.go
81.81% <0.00%> (-18.19%)
:arrow_down:

pkg/stream/stream.go
46.66% <0.00%> (-13.34%)
:arrow_down:

pkg/module/http2/transport.go
78.62% <0.00%> (-0.44%)
:arrow_down:

pkg/module/http2/frame.go
84.17% <0.00%> (-0.31%)
:arrow_down:

pkg/module/http2/server.go
81.85% <0.00%> (-0.08%)
:arrow_down:

Continue to review full report at Codecov.

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


feat: Upgrade to eslint v9

This has been put off for a while, finally got around to it
Use the new flat config setup for eslint config

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
172

Relevant Lines:
179

💛  - Coveralls


Database modifications

New "length_tracks" column in database
Empty playlist holds the value "[ ]". Becoming easier to manipulate.

Changed database in commit https://github.com/diogoditorr/discord-bot-py/commit/04bed37b601af956babeceaa42ed04ceae65d57f


create-env manifest validation doesn't catch missing properties - render job templates fails with obtuse erb_render error

When properties block is midding from the job block in manifest, bosh create-env fails during Rendering job templates... step with error
Deploying:
  Building state for instance 'concourse/0':
    Rendering job templates for instance 'concourse/0':
      Rendering templates for job 'unpack/213b9aaf8ee2b59dd4900c6c5435bafc281b8e34':
        Rendering template src: ignoreme, dst: ignoreme:
          Rendering template src: /Users/pivotal/.bosh/installations/3525263c-b10f-4905-6620-381612ee4cf6/tmp/bosh-release-job056632948/templates/ignoreme, dst: /Users/pivotal/.bosh/installations/3525263c-b10f-4905-6620-381612ee4cf6/tmp/rendered-jobs562517277/ignoreme:
            Running ruby to render templates:
              Running command: 'ruby /Users/pivotal/.bosh/installations/3525263c-b10f-4905-6620-381612ee4cf6/tmp/erb-renderer573538776/erb-render.rb /Users/pivotal/.bosh/installations/3525263c-b10f-4905-6620-381612ee4cf6/tmp/erb-renderer573538776/erb-context.json /Users/pivotal/.bosh/installations/3525263c-b10f-4905-6620-381612ee4cf6/tmp/bosh-release-job056632948/templates/ignoreme /Users/pivotal/.bosh/installations/3525263c-b10f-4905-6620-381612ee4cf6/tmp/rendered-jobs562517277/ignoreme', stdout: '', stderr: '/Users/pivotal/.bosh/installations/3525263c-b10f-4905-6620-381612ee4cf6/tmp/erb-renderer573538776/erb-render.rb:11:in `merge!': no implicit conversion of nil into Hash (TypeError)

but passes Validating deployment manifest... step.

@xtreme-gavin-enns

@adi-mishra properties key should either be on the job level or not. cannot have it as nil. if that's not what's happening, you'll have to provide manifest that causes this problem.

Yeah, exactly. The cli just doesn't catch this error at the validation step. We weren't able to track down which hash was unexpectedly nil except through visual validation of our yaml files.

Story: https://www.pivotaltracker.com/story/show/150620709

I ran into this issue when deploying a job that was not expecting any properties, and whose deployment manifest omitted the job properties block. Adding a non-empty properties block in the job section solved the issue. Thanks for the workaround documentation.
    properties:
       dummy: dummy # Just to keep bosh-cli happy, see https://github.com/cloudfoundry/bosh-cli/issues/240

Alternatively, I think you should be able to say properties: {} (i.e. can be more concise than a dummy key, if you want).


ErrorMessageResolver uses non "ng-" prefixed attribute as default and can cause conflict with browser defaults (IE9)

I ran across an issue today where one of my inputs had ng-maxlength="100". Every time I saw the error message, it would have the value<PHONE_NUMBER> instead of 100. As it turns out, the code below checks for the attribute matching the errorType by default. maxlength is the browser property that limits the physical amount of characters that can be entered into the field. In IE9, this became a problem because even though I did not physically specify maxlength, the browser defaulted the property. Because of this code, the parameter was found and it never looked at my ng-maxlength attribute.
parameter = el.attr(errorType);
if (parameter === undefined) {
    parameter = el.attr('data-ng-' + errorType) || el.attr('ng-' + errorType);
}

Because this is specifically angular, could we not alter the code to look for ng- version of the attribute first and have the non-prefixed attribute as a fallback? Such as:
parameter = el.attr('ng-' + errorType);
if (parameter === undefined) {
    parameter = el.attr('data-ng-' + errorType) || el.attr(errorType);
}

I had already created a custom message resolver for my app for other reasons so I just altered this code in my version, however, I think that this should be a change in the DefaultMessageResolver.

This seems like a sensible plan - I'll get the code change in tomorrow.  :-)

Sorry for the delay.  Issue fixed in commit 888ecd042affb391e57591a2f18f22395ba1f520 and available on bower V1.15.22
Thanks!!!!

@jonsamwell Thanks! I appreciate your efforts. Now, if I can just get people to use Ember instead of Angular...

Ha, not sure about using Ember!


Cache Policy is not set when calling request method with parameters and method

If I create a NSMutableURLRequest and set cache policy and then call request(.GET, mutableRequest, parameters:someParams) the cache policy in callback request param is different from what is being set.I think there is some bug in request building process..

Hi @sai-prasanna, thankfully there is not a bug here. Unfortunately you are just misusing the API. Here's the signature of the global method you are calling:
public func request(
    method: Method,
    URLString: URLStringConvertible,
    parameters: [String: AnyObject]? = nil,
    encoding: ParameterEncoding = .URL)
    -> Request
{
    // implementation...
}

You'll notice that the mutableRequest you are passing in conforms to a URLStringConvertible. In the implementation, the URLString for the convertible is extracted to create a new NSMutableURLRequest that will have the "default" cache policy. This is absolutely the intended behavior in this case.
Instead, you need the other request method which is as follows.
public func request(URLRequest: URLRequestConvertible) -> Request

This method will use your mutableRequest directly keeping your cache policy in place.

Thanks, I saw that the URLString was used to create new NSMutableURLRequest, didn't know whether  it was intended behaviour


cleanup

What
Cleanup stuff.
Why
because quite a lot of copy & paste and duplicate implementations, also so many folders of old code, and i want to rename Kiwi to something else too.

pretty happy with the cleanup; duplicate implementations are gone and the pull/push input/output functions are much cleaner.


Fix commented method name

Coverage remained the same at 26.699% when pulling f1f163ebc5aec9b963cfd76053a2dbd4766fe2cb on aunpyz:typo into 6d64e31fed6a008473ed3f692c85cb94e45be17e on tomv564:master.


Improve versioning from git tags with setuptools_scm

One possible way to close https://github.com/icubam/icubam/issues/121 using https://github.com/pypa/setuptools_scm which determines version from git tags + commit hash.
This also adds pyproject.toml as a recommended way to using that package, since setuptools_scm now needs to be available when installing, and this ensures it. This part is not absolutely necessary, can be reverted.
The workflow is just to add tags from time to time, and the version will be automatically generated, e.g.,

0.1.8rc6 - if it's the tag exactly
0.1.8rc4.dev0+g6dea7f7.d20200401 -  a few commits after the 0.1.8rc4 tag.

One limitation is that the tag needs to follow semver, and incorrect tag scheme (for the last tag) will result in an installation error due to regex parsing such as,
      prefix, tail = re.match(r"(.*?)(\d+)$", version).groups()
  AttributeError: 'NoneType' object has no attribute 'groups'

So for instance, v0.1.8-deploy is not a valid tag, while, v0.1.8.deploy might be OK (need to check).
Another limitation is that this package now needs to be installed before it can be used.

For instance, the version currently installed in CI is 0.1.dev1+g80a4266 which should follow the following pattern (https://github.com/pypa/setuptools_scm#default-versioning-scheme),
{next_version}.dev{distance}+{scm letter}{revision hash}

the hash is different from that of the last commit because CI merged with upstream internally I think. It does work as expected for me locally.
If you don't like setuptools_scm and the reliance on git tags, as simpler solution could be just indicate the version in setup.py, and it all other places it would be retrieved from there (as done in this PR).

@rprimet this would require us to change the deployment rule on staging.  What do you think?  This deffinitelly satisfies the 'centralize our versioning to one place' need.

@dulacarnold I like it, I'll amend the versioning scheme in the deployment repository when this is merged

what happens if someone deploys w/out git (e.g. docker, pip install from pypi or from tgz)?

if you install it from PyPi, it means the the .tar.gz was previously generated with python setup.py sdist, which currently for me generates icubam-0.1.8rc7.dev3+g7c1fe35.tar.gz and the version is internally included in the package metadata (icubam.egg-info/PKG-INFO). So it will work.
if you install from Github with git support,pip install git+https://github.com/rth/icubam.git@impr-version

e.g. to install the branch in this PR, it will also work, since you have access to git history including tags there.
If you install from Github without git support,https://github.com/rth/icubam/archive/impr-version.zip

it will fail with the following error,LookupError: setuptools-scm was unable to detect version for '/tmp/pip-req-build-toutfj9_'.

Make sure you're either building from a fully intact git repository or PyPI tarballs. Most other sources (such as GitHub's tarballs, a git checkout without the .git folder) don't contain the necessary metadata and will not work.

For example, if you're using pip, instead of https://github.com/user/proj/archive/master.zip use git+https://github.com/user/proj.git#egg=proj

which is a reasonable and explicit error message.

So overall it should work.


[SPARK-9876][SQL]: Update Parquet to 1.8.1.

What changes were proposed in this pull request?
This includes minimal changes to get Spark using the current release of Parquet, 1.8.1.
How was this patch tested?
This uses the existing Parquet tests.

Test build #59214 has started for PR 13280 at commit 022dd6b.

Test build #3015 has started for PR 13280 at commit 022dd6b.

in the past parquet upgrades brought perf regressions. Any idea about this release?

Test build #59214 has finished for PR 13280 at commit 022dd6b.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #3015 has finished for PR 13280 at commit 022dd6b.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

@rxin, I agree that we shouldn't upgrade if there are perf regressions. I would like to know what they are so we can fix them in Parquet upstream though. This should be a big performance improvement for  selective queries on sorted data because we can't currently push down any string predicates. It seems like a good thing to pair with the new {{DataFrameWriter#sortBy}} feature.

I'm not sure what should be done to fix the dependency test failure. Looks like there's a list of dependencies that needs to be updated. Is that something I should include in this PR?

Yea you would need to explicitly update the dependency list. We added that as a safe-guard to accidentally changing dependencies.

cc @liancheng who might have idea about past parquet perf regressions.

The dev/test-dependencies script can auto update the deps files for this purpose.
One thing we ask people to investigate are changes between old and new version so we can think through benefits vs potential incompatibilities

I had once tried to upgrade Parquet to 1.8.1, and one more change needs to be done for the upgrade: https://github.com/apache/spark/pull/9225/files#diff-b4108187503e0f3ac64c1630d266b122R115
For the performance regression, here is the full thread of previous discussion<EMAIL_ADDRESS>I had tried to bisect, but failed to find anything useful. I thought 1.8.2 would probably be release soon at that time, so didn't try hard to dig it...

@rdblue Is there any perf evaluation of this new version that we can refer to ?

I think we should upgrade Parquet to 1.8.1 in Spark 2.0 due to the following reasons:

Get PARQUET-251 fixed so that we no longer write corrupted statistics for string/binary columns, and can re-enable filter push-down for these columns (1.8.1 ignores corrupted statistics automatically).
The performance regression was discovered by a micro benchmark that simply scans the flat TPC-DS store_sales table (scale factor 15). However, we are not using our own vectorized Parquet reader for flat tables now. So at least table scan for flat schemata shouldn't be affected at the moment.

My only concern for this PR is that we should add the hack done in PR #9225. Otherwise there would be noticeable performance regression for queries like SELECT COUNT(*) FROM t.

Test build #59422 has started for PR 13280 at commit 30769bd.

@liancheng, thanks for pointing out that fix, I've added it. I thought that was already committed since it has been a while since we fixed the Parquet side.
I've also updated the dependency files, thanks @srowen.
I think given the fix for PARQUET-251, updating is a good idea. There isn't a perf evaluation for 1.8.1 that I can point to, but we've been using it without a noticeable change for months.

@rdblue LGTM pending rebasing and Jenkins. Thanks for fixing this!

@liancheng: rebased. Sorry I missed that earlier.

Test build #59430 has started for PR 13280 at commit d1c79c7.

Test build #59422 has finished for PR 13280 at commit 30769bd.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

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

Build finished. Test FAILed.

Test build #59441 has started for PR 13280 at commit af3957f.

Test build #59430 has finished for PR 13280 at commit d1c79c7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #59441 has finished for PR 13280 at commit af3957f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #59500 has started for PR 13280 at commit 85e03f9.

Test build #59500 has finished for PR 13280 at commit 85e03f9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

@liancheng, fixed. Yeah, IntelliJ has a few annoyances like that with scala. Imports are a mess.

Test build #59505 has started for PR 13280 at commit 40241dc.

Test build #59506 has started for PR 13280 at commit 37b4978.

Test build #59508 has started for PR 13280 at commit 33e07ee.

Test build #59505 has finished for PR 13280 at commit 40241dc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #59506 has finished for PR 13280 at commit 37b4978.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #59506 has finished for PR 13280 at commit 37b4978.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #59508 has finished for PR 13280 at commit 33e07ee.

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

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

Merged build finished. Test PASSed.

Great, I'm merging this to master and branch-2.0. Thanks for working on this!

Thanks @liancheng! It will be great to have predicate push-down for strings in 2.0!

Hello @rdblue, we are pretty late in this release cycle. I am afraid that we cannot actually upgrade Parquet to 1.8.1 because of the following two reasons:

Since this change was merged pretty late, I am not sure this change can be thoroughly tested. All of our previous testing was based on the Parquet 1.7.0. Also, some of our internal jobs started to fail after this upgrade.
Parquet 1.8.1 potentially introduces performance regression.  I am not sure we can upgrade Parquet before finish the investigation of this regression.

So, I'd like to propose to revert this upgrade. We can try to upgrade Parquet in the early development cycle of 2.1 (assuming we have figured out the regression). So, we can have more time to test this change.

+1 on reverting for the reasons Cheng mentioned. It's very risky to do dep updates at this point for 2.0, and I was surprised this got merged without at least verifying the prior performance regression we found went away.

@rxin Huh?  The merge was to master, not branch-2.0.  Doesn't that put it on the 2.1 track and not into 2.0.0?  I think that is all that Yin was saying, that @rdblue was mistaken about this change going into 2.0.

Oh, wait... sorry, I just realized that @liancheng said he also merged to branch-2.0.  +1 on reverting that.

I'm working on verifying the regression using the micro benchmark I did before. Sorry for the troubles.

As I said on PR #13445: It sounds reasonable, but we should follow up on this. If we revert the change I suggest that we only revert it in 2.0 or add it to master as soon as 2.0 is branched. That way we don't adversely affect 2.0, but we do get this addressed. Is that a reasonable path forward?

@yhuai, what started failing?


Tokenize the StarCoder dataset

Either tokenize from scratch or convert the arrow files to megatron format
Mayank has a script for converting to megatron, whill share


'reply_to_message' section missing from a reply-to message?

What prevents a bot from receiving the 'reply_to_message' section in a message?
My bot uses the username of the user from the original message that was responded to, but this section from the message is missing for this in some telegram groups.
user1: hi
user2: (reply to user1's message) /welcome Have a look here:  xxx
nodejs console.log(msg):
Message structure in a working group:
message_id:
from:
chat:
date:
reply_to_message:
text:
entities:
Message structure in a non-working group:
message_id:
from:
chat:
date:
text:
entities:
I have even added the bot as an 'admin' and also removed the privacy with the /setprivacy (disable) command in BotFather, but all to no avail.
Why can I see the 'reply_to_message' section in one group, but not in other groups?
Thanks

To clarify, the bots only needs to listen to the slash commands (/xxx) and therefor does not need group admin rights.
I created a two person group and added the bot into it and there it works, but if I add the bot into another group of around 50 people, there it does not work, because the 'reply_to_message' secton of the message is missing.
Is this something to do with privacy in a specific group that the original message that was responded to cannot be seen and only the response itself?

Also wondering about this. I can only see reply_to_message when a reply is to admin.

This issue still appears with todays telegram servers and node-telegram-bot-api v0.50.0

Workaround

Open group settings
Change the chat history for new members setting (it doesn't matter which of the two the group is using)
Hit "Save" on the chat visibility menu
Hit "Save" on the group settings menu
Repeat the above steps one more time

After editing these settings your bot should start receiving reply_to_message data in that group.
I tested this with my bot as NOT admin and privacy mode ON

Hi guys, any updates on this issue?
Sometimes I receive the reply_to_message some times not...


[SPARK-14704][CORE] create accumulators in TaskMetrics

What changes were proposed in this pull request?
Before this PR, we create accumulators at driver side(and register them) and send them to executor side, then we create TaskMetrics with these accumulators at executor side.
After this PR, we will create TaskMetrics at driver side and send it to executor side, so that we can create accumulators inside TaskMetrics directly, which is cleaner.
How was this patch tested?
existing tests.

cc @rxin @JoshRosen @andrewor14

Test build #56065 has started for PR 12472 at commit 9eb2a87.

Test build #56065 has finished for PR 12472 at commit 9eb2a87.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #56193 has started for PR 12472 at commit 6226058.

ready for review :)

Test build #56193 has finished for PR 12472 at commit 6226058.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

We should be able to remove almost all the methods on InternalAccumulators.scala, shouldn't we? All that includes create, createAll, createShuffleReadAccums, ...

Test build #56204 has started for PR 12472 at commit b82e90e.

Test build #56204 has finished for PR 12472 at commit b82e90e.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #56207 has started for PR 12472 at commit bfa39ff.

Test build #56207 has finished for PR 12472 at commit bfa39ff.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

looks pretty good now

Test build #56211 has started for PR 12472 at commit eff1790.

Test build #56211 has finished for PR 12472 at commit eff1790.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #56227 has started for PR 12472 at commit 7f00198.

Test build #56227 has finished for PR 12472 at commit 7f00198.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

retest this please

Test build #56300 has started for PR 12472 at commit 7f00198.

Test build #56300 has finished for PR 12472 at commit 7f00198.

This patch fails build dependency tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

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

Test build #56306 has started for PR 12472 at commit b80f3d0.

Test build #56310 has started for PR 12472 at commit 471c1e3.

Test build #56306 has finished for PR 12472 at commit b80f3d0.

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

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

Merged build finished. Test PASSed.

Test build #56310 has finished for PR 12472 at commit 471c1e3.

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/56310/
Test PASSed.

Thanks - merging in master.


Victor VosMottor: Watch ttpuretech.com

Victor VosMottor requests the watch of the watch_keyword ttpuretech\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ttpuretech\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ

Later removed because of false positives.


Can Snowflake be used as metadata database in Superset?

Can Snowflake be used as metadata database in Superset?

The short answer is no. Snowflake is very limited in functionality found in typical OLTP databases (e.g. only supports constraints and primary/foreign keys for documentation purposes), and in general performs poorly for low-latency low-volume queries. You're best off keeping your metadata in Postgres or MySQL, and using Snowflake for analytical workloads.


Ticks transition sometimes produces console errors when used with tickValues

There is issue we have which probably is related to [#24].
Here is example for reproduction.

You are specifying tick values that are not defined in your scale’s domain. For example:
domain = [316, 225, 392, 377, 336, 295, 382, 270, 391, 224, 251, 319, 207, 204, 383]
tickValues = [363, 276, 249]

Ordinal scales (including band and point scales) are only defined on their domain, so it’s not possible to render tick values outside of the domain.


Add config as a parameter

There is currently no way to pass the plotly config through to the frontend as config does not exist as an attribute on the Plotly Figure, which is what the react component is currently expecting.
This change adds a config dictionary as a parameter so it can be passed directly through to the frontend.
I also added some config to one of the test examples.

Are there any updates on this merge?

can i use it on map instead of x,y from bar

This would be exceptionally useful if it can be merged. Does anyone know if there is another fork with this implemented?


which lookup is executed when using Multiple lookup editors for the same Entity when use getlookup()

I create two lookup Editors for the same entity, one returns all record and another one has a complex filter.
now ,when I write this code
xxxRow.getlookup();
which lookup will executed .
thanks

you may consider checking log, where you will see SQL statement

I duplicated the xxxRow.cs class and create lookup for each class and then I called the right xxxRow.getlookup() method.


JBPM-5338: Remove kie-config-cli

Follow up on (should be merged with) https://github.com/droolsjbpm/kie-wb-distributions/pull/361

removed chapter about the tool & replaced one reference to it in RuntimeManagement chapter
added explanation about why it was removed to N&N

Jenkins please retest this.

Build successful! See generated HTML docs (may take a while till they are available):
http://ci.kiegroup.org/job/kie-docs-pullrequests/188/artifact/drools-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/188/artifact/jbpm-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/188/artifact/optaplanner-wb-es-docs//target/generated-docs/html_single/index.html


can't get temperature to stop going down to lowest setting

To Reproduce
Steps to reproduce the behaviour:

Easy to reproduce,  it happens  immediately now and every single time not just sometimes.  Android phone currently not useable
using the premade windows binary 1.12.14
pinesam  app buttons +/-  continue to go  up  or all way down.
windows 11 & iphone seem good.

android phone screen goes all way up to Max temp still but now it's also doing it in reverse Minus and going all way to lowest.   before it was only doing it to maximum (+)  now it's also doing it to minumum then get warning  out of range (10-450).
Tested two ways.  This happens even if I  start browser fresh, and do not touch any Preset buttons first.  I just start changing the temperature with +/- screen,   Before it happened on both iphone and android and now the iphone seems ok.  I tested by  not even touching any Preset first and just using the  +/-  buttons; it shoots all way  up if I preset (+).  then I restart browser fresh  without restarting the  servers, and  press only  (-)  and it shoots all way down without stopping, even if I then try to hit the preset then it will temporaraly go to preset number and start  moving down rapidly again (following my increments of 10 from settings).
New behavior is now it's happening on minus  as well.
Specs:

OS: [ Windows 11]
Browser: [chrome]
PineSAM Version:  1.12.14
IronOS build/date: [ 02-04-23 , 2.20.9C6ED68]

Additional context
related to this issue https://github.com/builder555/PineSAM/issues/119

new test:
Windows  browser it is all good.
iphone browser and it's all good.

This is only happening on android phone, and happening for both +/-   .  restarting android chrome browser does not fix it.
happening on both Chrome and Samsung Internet browser on android.
restarting android does not  fix.

If  we can't get this fixed I'm willing to mail you this Android phone  if it's not too much shipping. this is  my spare phone that  is not in sevice that I use just  displaying  PineSAM or Joric API  while soldering.  it sits in a stand on the side so it's not getting harmed at all.

@River-Mochi can you check if this is still the case? I believe it was addressed in one of the earlier releases.


DEP: remove skbio.parse/format subpackages

Coverage increased (+0.19%) to 99.02% when pulling 66ad7ce919bf024c48ea409e6f99c8f909aa8873 on jairideout:0.3.0-dep-removal into c9116e70970920627ebde2fda738b7596b33864d on biocore:master.


add PlayerPedia to repo

PlayerPedia is an android project (native) that will show you an Football Player Data

@kevinmf1 nice!


Create the minos-http-aiohttp plugin

[x] #209

Closing this issue as it is closely related to #209


Version 4.0.0 for Azure.Batch management

Description

The new Swagger specification is here: https://github.com/Azure/azure-rest-api-specs/blob/master/arm-batch/2017-01-01/swagger/BatchManagement.json
This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

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

Testing Guidelines

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

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as a link to the swagger spec, used to generate the code.
[x] The project.json and AssemblyInfo.cs files have been updated with the new version of the SDK.

Note for reviewers
This is a breaking change, so the version has been bumped to 4.0.0


Provide unique HTCondor users for each coffea-casa user

Related to: https://github.com/CoffeaTeam/coffea-casa/issues/150

From the corresponding email thread, Brian's advice on configs:

IIRC, you must (a) set USER_IS_THE_NEW_OWNER = true in the schedd's condor_config and (b) ensure the setting of UID_DOMAIN is distinct from the domain in the generated subject in the token.
So, if UID_DOMAIN=unl.edu and my token claims I'm<EMAIL_ADDRESS>then (with USER_IS_THE_NEW_OWNER):

The User attribute should remain<EMAIL_ADDRESS>The Owner should be set to nobody

All jobs will therefore launch the condor_shadow as nobody and similarly the spool directory will be owned by nobody.

Access to modifying the jobs will be limited to<EMAIL_ADDRESS>someone coming in as<EMAIL_ADDRESS>(even if they're mapped to Owner "nobody" as well) will not be able to condor_rm my jobs.

Discussion:

This implies we'll need to change how we generate IDTOKENS (coffea-casa jupyter config).
Can start by hand-generating a token, copy/pasting into the container, and trying to hand-submit to the schedd.  Can even do this on T3.

If owner is set to nobody (which it appears is not the case) how will we have to modify our startds to start them as we use a number of owner. = stuff to steer jobs.

Waiting on Condor pull request that's under review.

I'm at a loss as to how to proceed on this task.  Are tokens supposed to manage the usernames in queue?  Confused

@clundst is it still a valid issue?

This is almost solved, what I need (I think) is for the secret_creation_hook.py to assign condor_user to something unique for each user.  Right now all the users are still cms-jovyan and able to kill of other cms-jovyan jobs.  I can't try anything in the dev instance as condor is not working there.


urllib3 exceptions

Traceback (most recent call last):
File "./dlcharts.py", line 111, in
content = get_url(url)
File "/home/ed/projects/equalog/src/web.py", line 118, in get_url
referrer=referrer)
File "/home/ed/projects/equalog/src/web.py", line 81, in get_content
content = response.content
File "/usr/lib/python2.7/dist-packages/requests/models.py", line 694, in content
self._content = bytes().join(self.iter_content(CONTENT_CHUNK_SIZE)) or bytes()
File "/usr/lib/python2.7/dist-packages/requests/models.py", line 627, in generate
for chunk in self.raw.stream(chunk_size, decode_content=True):
File "/usr/lib/python2.7/dist-packages/urllib3/response.py", line 242, in stream
data = self.read(amt=amt, decode_content=decode_content)
File "/usr/lib/python2.7/dist-packages/urllib3/response.py", line 200, in read
raise ReadTimeoutError(self._pool, None, 'Read timed out.')
urllib3.exceptions.ReadTimeoutError: HTTPConnectionPool(host='....', port=80): Read timed out.

What version of requests are you using? This should have been patched in 2.4.0.

I'm not sure what version it was. I was using the system package instead of my virtaulenv one, and that fixed it. Most likey diferent versions.


1.3.18 -  [jqLite:onargs] jqLite#on() does not support the selector or eventData parameters

Hi, I'm using Angular 1.3.18.
I haven't build my project in a while. Today when i started the production pipeline, I saw that there is an error after i concatenate angular with all the other js file. When the files are not concatenated but just used like in developing stage it does work.
It's a long shot, but let see if you have any suggestions here:
It is basically a
Error: [jqLite:onargs] jqLite#on() does not support the selector or eventData parameters
http://errors.angularjs.org/1.3.18/jqLite/onargs
With a stack trace of:
    at REGEX_STRING_REGEXP (http://localhost:9000/js/app.min.js:63:12)
    at jqLiteOn (http://localhost:9000/js/app.min.js:3067:39)
    at Object.JQLite.(anonymous function) (http://localhost:9000/js/app.min.js:3285:17)
    at link (http://localhost:9000/js/app.min.js:49514:12)
    at invokeLinkFn (http://localhost:9000/js/app.min.js:8290:9)
    at nodeLinkFn (http://localhost:9000/js/app.min.js:7800:11)
    at compositeLinkFn (http://localhost:9000/js/app.min.js:7149:13)
    at compositeLinkFn (http://localhost:9000/js/app.min.js:7152:13)
    at compositeLinkFn (http://localhost:9000/js/app.min.js:7152:13)
    at publicLinkFn (http://localhost:9000/js/app.min.js:7028:30) <nav ui-nav="" class="navi clearfix">

The starting error, the tag mentioned at the bottom of the stack, is this tag:
<!-- nav -->
<nav ui-nav class="navi clearfix" ng-include="'tpl/blocks/nav.html'"></nav>
<!-- nav -->

That is some UI code for the navigation menu of my site.
uiNav is this directive:  https://github.com/TracklistMe/client/blame/master/src/js/directives/ui-nav.js
Now, if instead of declaring the directive in that way, i do something like:
<ui-nav class="navi clearfix"  ng-include="'tpl/blocks/nav.html'"></ui-nav>

It works.

When using the element form (<ui-nav ...></ui-nav>), it still doesn't work, because the directive is restricted to attribute and class selector (restrict: 'AC').
So using it as an element, it doesn't get recognized as a directive => it doesn't get compiled/linked by Angular => it's link function is never called => you don't get the error.
(But it doesn't work either.)
--
It is next to impossible to provide a definitive answer with the info (not) provided, but based on the error you get, this seems a possible explanation (still a stab in the dark):

The directive uses .on(<EVENT>, <SELECTOR>, <CALLBACK>).
This syntax is not supported by jqLite (Angular built-in, stripped-down version of jQuery).
Since you say that it works with the unconcatenated files, I presume you have jQuery loaded.
In order for Angular to use jQuery (instead of it's jqLite), the former has to be loaded before Angular.
A possible cause is that in the concatenated file, jQuery is placed after Angular, thus force Angular to fall back to jqLite.

That said, this is a general support question and would be better suited for one of the suport channels.
GitHub issues are reserved for bug reports and feature requests.
I am closing this (since it's not actionable on our part).
Feel free to reopen if you have any indication there is a bug in Angular core that causes the issue.


Support for distinct in functions and the BETWEEN WHERE-clause construct

Dear @Zaid-Ajaj, thank you for all the wonderful work you are doing for the open-source community!
I am considering using the SQL parser part of your library in a reference implementation I am building.
Currently, it is missing two features I would rely on, namely:

the ability to use the distinct keyword in a function (such as count(distinct column), and
the BETWEEN keyword for WHERE-clauses (WHERE age BETWEEN 1 and 20)

Would you welcome pull-requests to add these features?
If yes, are there any particular things I should keep in mind while adding these features? Say some non-obvious aspect of your library that doesn't immediately spring to mind but needs to be considered?

Close as duplicate of https://github.com/Zaid-Ajaj/Npgsql.FSharp.Analyzer/issues/22
Github at first told returned a 404 when I created the issue. Must have been a race condition of sorts. Hence the double issue.

Close as duplicate of https://github.com/Zaid-Ajaj/Npgsql.FSharp.Analyzer/issues/22
Github at first told returned a 404 when I created the issue. Must have been a race condition of sorts. Hence the double issue.


Is there a way to have “AutoZoom” (or “Fit”) enabled for embedded codeblocks?

The use case is probably the same as most people’s, if not all, use case for the embedded codeblock, which is to display the current note’s locations.
I am generating a note of my locations automatically from a tracking app. Most of the time a Zoomlevel of 13-15 works fine if I’m not travelling. If I’m travelling or flying there is no way to know the proper zoom level apriori without a lot of guess-work and programming on my end. Wondering if there is an easy “Auto” or “Fit” setting for the “mapZoom” key, as this is already a feature in the plug-in for the right-pane map.
If not, maybe you can give me the algorithm to determine zoom from the range in lat and lon?

Sounds super useful and I'd like to add this soon. However, I'm also seeing some potential for confusion and trouble here, as the "fit" setting will somehow need to work intuitively when users move and zoom the embedded map, then click 'Save'. How would you define such a case to behave -- should "fit" turn off when a user makes manual changes to the map?

My suggestion would be just that: the “Fit” option makes the view automatically fit (zoom and position) upon first draw and then turns off if the user adjusts the map. The button can be available to re-“fit” to starting zoom and position if desired but it’s now a user-initiated action, not automatic.
Thanks for considering!

This proved to be considerably more complicated than I expected. I added the basic feature in 4.0.0, so you can set autoFit in the code block, but there is not yet a UI for it and the user experience isn't perfect. See here: https://github.com/esm7/obsidian-map-view#advanced-additional-options


Use aiohttp for retrieving races, add a command_prefix kwarg

Firstly, I added add command_prefix kwarg to the command processor.  It defaults to "!"
Also, switched to aiohttp in the refresh_races loop instead of requests.  This lets token refreshes work asynchronously, which will prevent the bot from crashing if the request takes longer than normal to return.  The existing function that authorizes the bot still uses requests.
Feel free to reject this PR if you don't think aiohttp should be used here, I'll be happy to resubmit with just the command_prefix changes.

Tagged and released as 1.4.0

Nice, thanks for the tip on the .gitignore_global, didn't even know that was a thing.


Updatewindow

The document explains how to configure custom update windows for appsmith

@Pranay105 Can you please review this document.

I'm merging this in because we need to fast track this feature.


Plex refactor Part 1 - Update plexapi to 3.0.3

Description:
Updates plexapi to v3.0.3
This also requires updating requests to 2.18.4
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[X] New files were added to .coveragerc.

If the code does not interact with devices:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] Tests have been added to verify that the new code works.

Awesome, I was planning to update this. Let me know if there is anything we can do the plexapi end.

When will this be merged? I have big issues with current Plex

@Hellowlol Cheers
@frelev No idea

Did an upgrade to latest dev.
Unfortunately I now get:
2017-10-25 18:26:01 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up platform plex
How can I debug it?

As reference, a similar response:
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 170, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=self.hass.loop)
  File "/usr/lib/python3.4/asyncio/tasks.py", line 372, in wait_for
    return fut.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/plex.py", line 112, in setup_platform
    hass, config, add_devices_callback
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/plex.py", line 231, in setup_plexserver
    update_devices()
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/util/__init__.py", line 306, in wrapper
    result = method(*args, **kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/util/__init__.py", line 306, in wrapper
    result = method(*args, **kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/plex.py", line 198, in update_devices
    update_sessions, plexserver)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/plex.py", line 305, in __init__
    self.refresh(device, session)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/plex.py", line 368, in refresh
    if self._device.machineIdentifier is not None and \
AttributeError: 'NoneType' object has no attribute 'machineIdentifier'

Now it works for me, thanks


Handling multilabel classification

Hey,
I have a multilabel classification problem I'd like to solve using distance metric learning. For runtime evaluation of OOD samples and "weird" images, going with a distance metric learning based classifier backbone is favorable to me, so I can tell whether a new sample is far off my cluster centroids per class.
The issue is, that I have a multilabel classification problem, as multiple relevant objects can be in an image. How would you handle this currently? Do i just add another row for the same filename in the data.csv file but with the other class label assigned?
Or do I just leave ambiguous labels out of the training?
Thanks
EDIT:
I noticed, that in your example CSVs provided in the documentation (https://open-metric-learning.readthedocs.io/en/latest/oml/data.html), there is one CSV df_sop.csv which contains a new column "super_class_id" which is not documented anywhere. Did you write a custom dataset and sampler for that? I can't find any reference to it in your documentation.

Hey,
If your image has 3 objects on it, I recommend you to have 3 different rows in df.csv with the corresponding bounding boxes

super_class_id is a legacy thing, it's just an enum for the category column.
Thank you for pointing this out, I will remove it from the examples :)

Hey,
If your image has 3 objects on it, I recommend you to have 3 different rows in df.csv with the corresponding bounding boxes

Unfortunately, the objects are also overlapping or intersecting most of the time. What is your recommendation?

leave out ambiguous examples with intersecting / overlapping examples
do as you proposed and still keep 2 rows for the same image if its multilabel

By the way

The issue is, that I have a multilabel classification problem, as multiple relevant objects can be in an image.

It's an object detection problem. "multilabel classification problem" means that you have an image of a car and it has few labels: "car", "red", "4-wheels"

It's not in my case. Yes, there are objects and I also have bounding box labels but I do not care about coordinates of objects in an image in my scenario. I just want to know whether certain things are present in an image, without caring for where exactly they are present.

Got it.


Dependet Column on Windows is empty

On Windows the column with dependent patterns is always empty, even when the parent pattern shows the pattern as a dependency.
Windows 10
Node: v4.2.2

According to my latest test this is no longer an issue. Can you validate @LKuechler?


Readme states that Cpp2IL uses .NET 5 [Quick fix]

Title states the small problem

It does indeed, though it doesn't say so on the development branch. I'll copy the wording from there. Thank you for pointing this out!


Questions about using this with an existing Firebase database

I am working on an app that already makes extensive use of Firebase, and I'm evaluating this library as a means to add chat capabilities for our users.
My main concern is to make sure that I don't accidentally do anything with this code that will alter or delete our existing Firebase nodes. When I call firebaseRootPath() on the Configuration Builder, how does that work with my existing database nodes? I.e., if I have an existing database that looks like:
mycompany

users
organizations
groups
stuff
etc.

And I call firebaseRootPath('chat') on the Builder, will that create a new chat node underneath the mycompany node, without affecting any of the other nodes?
Thanks for your help!

Hi @rnstewart,
The root path is essentially a namespace for the Chat SDK data. Here's an example from a real database:

You can see each root path has it's own chat instance. This means that if you set the root path to chat the Chat SDK library wouldn't make any changes to any path outside root/chat.

OK, perfect. That's what I thought, but thank you for clarifying.


generate_cpi_crate proc macro panicked

Just a newbie to rust and solana.
I'm finding ways to let my program interact with other programs. While there is interface in solidity and you only need to know the address that you wanna call and apply the appropriate interface to that address, I may need IDL to do the similar thing in solana.
I've read the guide of anchor for cross program invocation. Just really don't like it. It means you have to clone other program's source code into your project. Then I found IDL to work like an interface that could solve the problem of source code. And then I found this tool for generating rust code from the IDL json file. So I'm trying to split the examples in anchor guide puppet and puppet-master into two seperate repos. And I'm trying to use anchor-gen within puppet-master repo in order to invoke puppet. But I ran into this error
puppet within puppet-master repo
anchor_gen::generate_cpi_crate!("idl.json");

declare_id!("9ceHBqVGkYRNJ2uRZG3xvbuU4BQbRKESSs8Q3sWQ74Z9");

what i'm using
[dependencies]
anchor-gen = "0.3.1"
anchor-lang = "0.30.1"

puppet idl json
{
  "address": "9ceHBqVGkYRNJ2uRZG3xvbuU4BQbRKESSs8Q3sWQ74Z9",
  "metadata": {
    "name": "hello_anchor",
    "version": "0.1.0",
    "spec": "0.1.0",
    "description": "Created with Anchor"
  },
  "instructions": [
    {
      "name": "initialize",
      "discriminator": [
        175,
        175,
        109,
        31,
        13,
        152,
        155,
        237
      ],
      "accounts": [
        {
          "name": "new_account",
          "writable": true,
          "signer": true
        },
        {
          "name": "signer",
          "writable": true,
          "signer": true
        },
        {
          "name": "system_program",
          "address": "11111111111111111111111111111111"
        }
      ],
      "args": [
        {
          "name": "data",
          "type": "u64"
        }
      ]
    },
    {
      "name": "set",
      "discriminator": [
        198,
        51,
        53,
        241,
        116,
        29,
        126,
        194
      ],
      "accounts": [
        {
          "name": "new_account",
          "writable": true
        }
      ],
      "args": [
        {
          "name": "data",
          "type": "u64"
        }
      ]
    },
    {
      "name": "sol_transfer",
      "discriminator": [
        135,
        254,
        247,
        202,
        217,
        48,
        184,
        165
      ],
      "accounts": [
        {
          "name": "from",
          "writable": true,
          "signer": true
        },
        {
          "name": "to",
          "writable": true
        },
        {
          "name": "system_program",
          "address": "11111111111111111111111111111111"
        }
      ],
      "args": [
        {
          "name": "lamport",
          "type": "u64"
        }
      ]
    }
  ],
  "accounts": [
    {
      "name": "NewAccount",
      "discriminator": [
        176,
        95,
        4,
        118,
        91,
        177,
        125,
        232
      ]
    }
  ],
  "types": [
    {
      "name": "NewAccount",
      "type": {
        "kind": "struct",
        "fields": [
          {
            "name": "data",
            "type": "u64"
          }
        ]
      }
    }
  ]
}

Same error, have you solved?


speed up service view loading

it turns out to be a load balancer (LB) issue. If the k8s API server behind a LB and the client is with the same IP with the API server, the LB will not route the request from client to API server (a known Azure LB issue)


ruff should respect targeting Python 3.6 for UP036

My project (cocotb) will continue to support Python 3.6 for the foreseeable future. However ruff doesn't seem to respect this. With the following in pyporject.toml:
[project]
requires-python = ">=3.6"

And this code in my project:
# On python 3.7 onwards, `dict` is guaranteed to preserve insertion order.
# Since `OrderedDict` is a little slower that `dict`, we prefer the latter
# when possible.
if sys.version_info[:2] >= (3, 7):
    insertion_ordered_dict = dict
else:
    import collections

    insertion_ordered_dict = collections.OrderedDict

I get a UP036 error (Version block is outdated for minimum Python version) and --unsafe-fixes attempts to remove the version ifdef. Clearly ruff is internally bumping the version to python 3.7 and then complaining about this ifdef rather than respecting the actual version. I don't much care if Python 3.6 gets no official support, but this isn't a very good behavior =/

I'm honestly not sure what the right behavior is here! It's good, at least, that these are gated behind --unsafe-fixes, since users are at least required to opt-in to explicit, unsafe changes. But I agree that we can do better.
It would be straightforward to either (1) log a warning if you're using an unsupported requires-python range, or (2) throw an error. I'm not sure that we'd want to go through and audit all rules to determine whether they require Python 3.7 or above though -- I mean, we definitely could, I just don't know if we'd prioritize it.
Any opinion on the behavior you'd expect, assuming Ruff doesn't support Python 3.6?

The output of ruff assuming 3.7 works as is in 3.6 for us. I'd be happy if there was at least "unofficial" support for 3.6. As in you can give 3.6 as the version, ruff respects it, but maybe there isn't the full range of checks there should be. This would be accompanied by a "consider all fixes unsafe unless proven otherwise" warning. Existing rules that are gated to version 3.7 could be brought back to 3.6 until proven otherwise by a user.
Personally I don't care about anything before 3.6. The only reason I care about 3.6 is because it's the standard Python version on RHEL8, which a lot of our users are stuck on, and it's at least being unofficially supported by Red Hat. Everything before that can and should be damned.

My two cents: I don't think rules need to have consideration for EOL Python past a certain point. But UP036 doesn't need special consideration, it should be able to just compare the number configured against the number found in code.


Variadic Macros on Microsoft C++ compiler does not work like gcc.

When used in Visual Studio 2015/2017 I get compiler errors  that say "OPERATOR has no type identifier class" etc.
I tried to come up with  a fix myself but I am not that savy with c++ macros. If I am doing something wrong in visual studio, can you point me to the right direction? thank you!

Can you get me an error log? I need to see the actual error message and not a abridged description. The compiler usually provides much more technical and useful diagnostics.

Sorry about that, I am currently at work and on a mac, will update the issue with the actual error log from my windows pc later. For now, all the info I can provide is the exact line of the error:
 which is included in multiple places but it specifically crashes at this #include:

and the fact that all single variable macros seem to work, it is the OPERATOR with 3 arguments causing the issue.

Also, it worked when I compiled in here => , so possibly some visual studio setting I am missing?

Again thanks for taking the time to reply.
Cheers.

Closing this problem since I cannot reproduce it.


runtimeCaching not working?

Hello,
Sorry this probably is not a bug but my head is going to explode.
I have this app: https://edu.carre-project.eu/ and a service worker file in https://edu.carre-project.eu/service-worker.js generated by this conf :
{
  "staticFileGlobs": [
    "dist/public/app/*",
    "dist/public/manifest_icons/*",
    "dist/public/fonts/*",
    "dist/public/*"
  ],
  "stripPrefix": "dist/public/",
  "runtimeCaching": [{
    "urlPattern": "/^https:\/\/edu\\.carre-project\\.eu\/api\/",
    "handler": "networkFirst"
  }]
}

from the cli command : sw-precache --config=sw-precache-config.json
My main problem is that it doesnt cache the api...
I have also tried this regex /(.*)\/api\/(.*)/ and single request such as 'api/config.json' but again with no luck
Is there any obvious mistake in my conf?

I just gave it a try, and that generates toolbox.router.get(/^https://edu\.carre-project\.eu/api/, toolbox.networkFirst, {});, which is an invalid RegExp.
String escaping in JSON is coming into play here, and you need to actually escape all the '' characters in the string, so that they make it into the final RegExp. (You're only double-escaping some of them.)
Try "urlPattern": "/^https:\\/\\/edu\\.carre-project\\.eu\\/api\\//", which leads to toolbox.router.get(/^https:\/\/edu\.carre-project\.eu\/api\//, toolbox.networkFirst, {}); in the output.


Log in MongoDB

Today I went to test write the log using MongoDB and found that it's not possible.

Invalid Configuration – yii\base\InvalidConfigException

"mongodb" refers to a yii\mongodb\Connection component. yii\db\Connection is expected.

I honestly do not even know MongoDB enough to understand why.
It is a limitation of MongoDB or Yii2?
Or, ask them: save the log in MongoDB is not a good idea, thinking about performance?

Have you configured yii\mongodb\log\MongoDbTarget as target?
Because \yii\log\DbTarget is sql-specific

You should use MongoDbTarget:
https://github.com/yiisoft/yii2-mongodb/blob/master/log/MongoDbTarget.php


[WIP] Deprecate bcolz support

[x] Closes #8330
[x] Tests added / passed
[x] Passes pre-commit run --all-files

The failing windows-3.8 test seems to be related to this PR.

I think the failing macos-python-3.8 test isn't related to this PR

FWIW I would be fine just ripping this out.


Backport #43

Add full index support for v2

@brettlaishley I've forgotten an important line :disappointed: I've fixed it up now


Optimisation: Hogan Mustache templates precompiled at build time (hgn! RequireJS processor)

It looks like we should be able to optimize the build by using the stubModules and excludeHogan directives?
https://github.com/readium/readium-js-viewer/blob/develop/grunt/requirejs.js
    stubModules : ['text', 'hgn'],

    pragmasOnSave : {
        excludeHogan : true
    }

Official requirejs-hogan-plugin:
https://github.com/millermedeiros/requirejs-hogan-plugin/blob/master/test/build.js
Readium's hgn.js:
https://github.com/readium/readium-js-viewer/blob/develop/lib/thirdparty/hgn.js
Readium's hogan.js:
https://github.com/readium/readium-js-viewer/blob/develop/lib/thirdparty/hogan.js

This is already done seamlessly by hgn.js. It's a requirejs plugin. When you run everthing in the browser expanded, the templates are compiled on the fly. When you run requirejs (or run it via grunt in our case), it precompiles the template into javascript.
Is that what you had in mind or am I missing something?

The stubModules and excludeHogan configuration directives are recommended by hgn.js (see links in the descripton above). It looks like we can trim the Readium app payload by telling the r.js optimizer to skip the unnecessary Hogan bits (given that the templates are already compiled). In the expanded case (no RequireJS optimization / aggregation), naturally these r.js settings have no effect, and the full Hogan compiler needs to be part of the runtime.

Fixed in RequireJS refactoring:
stubModules: ['hgn', 'i18n']
https://github.com/readium/readium-js-viewer/blob/develop/build-config/RequireJS_config_single-bundle.js#L32
https://github.com/readium/readium-js-viewer/blob/develop/build-config/RequireJS_config_single-bundle_LITE.js#L32


Sort Breakpoints by Creation date

Issue Type: Feature Request
I have many breakpoints setted in my environment. I would like to sort my breakpoints by Creation Date because it would make it easier to see the most recents.
VS Code version: Code 1.56.2 (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:13:13.157Z)
OS version: Windows_NT x64 10.0.18362

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog and we might add this in the future. Thanks

@isidorn Thanks! It also would be good if we could group the breakpoints by filename or even a label, but I understand that this may be much more complex.


Add spoc merge Command

What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR adds a spoc merge command which can be used to merge multiple security profiles via the CLI.
Does this PR have test?
Yes.
Does this PR introduce a user-facing change?
Added a `spoc merge` command to merge multiple security profiles from the command line.

Codecov Report

Merging #2136 (cdd50cc) into main (11d77f4) will decrease coverage by 4.55%.
Report is 55 commits behind head on main.
The diff coverage is 54.35%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2136      +/-   ##
==========================================
- Coverage   45.50%   40.95%   -4.55%
==========================================
  Files          79      102      +23
  Lines        7782    15106    +7324
==========================================
+ Hits         3541     6187    +2646
- Misses       4099     8482    +4383
- Partials      142      437     +295

Thank you for the prompt review! 🍰  Tests are just flaky here, right?

Yes, restarted the OLM test.

Thanks - looks like we are extra lucky today or we have CI breakage. 🙃

Thank you for the super quick review! 🍰 ⚡

Thank you for working on this @mhils! :pray:


Client Allocation Request for: Stefan Magdalinski

Client Allocation Questions
Core Information

Name: Stefan Magdalinski
Website / Social Media:
Region: North America
DataCap Requested: 32GiB
Addresses to be Notarized: f1zp5opmzrtx6j3iqb3sdodfu2q6sn62sfx44ycha
Notary Requested: dannyob

@smagdali Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Closing these issues as they're multiple requests (due to https://github.com/keyko-io/filecoin-verifier-frontend/issues/454 I believe)


[BP-1.16][FLINK-32392][ci] Invalidate Maven repo cache on Azure every year to reduce its disk usage

Unchanged back port of #22858 on release-1.16

Since CI fails with
Jun 26 12:40:26 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.22.1:test (end-to-end-tests) on project flink-end-to-end-tests-aws-kinesis-streams: Unable to generate classpath: org.apache.maven.artifact.resolver.ArtifactResolutionException: Failure to transfer org.opentest4j:opentest4j:jar:1.1.1 from https://maven-central-eu.storage-download.googleapis.com/maven2/ was cached in the local repository, resolution will not be reattempted until the update interval of google-maven-central has elapsed or updates are forced. Original error: Could not transfer artifact org.opentest4j:opentest4j:jar:1.1.1 from/to google-maven-central (https://maven-central-eu.storage-download.googleapis.com/maven2/): Entry [id:28][route:{s}->https://maven-central-eu.storage-download.googleapis.com:443][state:null/] has not been leased from this pool
Jun 26 12:40:26 [ERROR] org.opentest4j:opentest4j:jar:1.1.1

I  wonder if there is a way to say -U for the case when it's the build with empty cache?

@snuyanzin I don't think adding -U could help, as the root cause is actually
Could not transfer artifact org.opentest4j:opentest4j:jar:1.1.1 from/to google-maven-central (https://maven-central-eu.storage-download.googleapis.com/maven2/): Entry [id:28][route:{s}->https://maven-central-eu.storage-download.googleapis.com:443][state:null/] has not been leased from this pool

which happened when downloading the dependency.
I found FLINK-16947, which shows the same error messages. Probably it's another bug in the HTTP client that Maven Wagon depends on 😞

@flinkbot run azure

I checked the log in the second CI run, and there is cache hit already. I'm gonna merge it as this instability should be very rare and higher Maven version on master branch has already fixed it.


Why change value of leftPanDisabled / rightPanDisabled in dealloc?

- (void)my_dealloc
{
    AMSlideMenuMainViewController *mainVC = [AMSlideMenuMainViewController getInstanceForVC:self];
    mainVC.leftPanDisabled = NO;
    mainVC.rightPanDisabled = NO;

    [self my_dealloc];
}

Any view controller that includes this category will now mess with the mainVC's pan behavior when the VC is deallocated. This seems really problematic, and indeed is causing a lot of problems for me. I want to disable the pan gestures while showing a tutorial or a details screen, but any VC anywhere that uses the category and gets deallocated is overriding this important property.

same issue...


[stable/sensu] add repo archive notice and deprecate chart

See #24030

Manually merging as this is a required action. The tests are failing because the chart includes a resource version that is no longer supported in Kubernetes.


Add account_name related utility functions

Added three utility functions to both eosiolib and the chain library:
// Gets a length * 5 bit mask
static constexpr uint64_t name_mask(size_t length);
// Gets part of a name
static constexpr uint64_t name_mask(size_t length);
// Gets the length of name (normalized)
static constexpr size_t name_length( uint64_t n );

Unit tests have been written for these functions.

Thanks for your pull request. This has moved to eosio.cdt repo and is no longer needed.


Consider integrating template engine

Maybe here is a low hanging fruit waiting for us to pick it:
There may be a way to grant ProjectMaker the super powers of a full-blown template engine.
This could make our code much simpler as we could just outsource the current file creation logic to an external library. It also would endow ProjectMaker with templating abilities going beyond mere string substitution. Namely iteration, conditionals and inheritance would enhance usability.
Questions:

Does it make sense at all? Do we want this?
Do we recreate the wheel? (like coding another cookiecutter, at least another GCnP)
Which templating engine to use? (likely Jinja2)
Should we stick with boost libraries' format string delimiters (${...}) or those native to the templating engine chosen? (Jinja2:  {{...}}, {%...%})

I've actually been working on another project inspired by this. https://github.com/bit101/tinpig
It is kind of reinventing the wheel. Maybe I'm the only one who will use it. But it's been fun. Hope to release v1 by the weekend. Maybe sooner.

But don't let that stop you from taking this project other places.

@bit101 I am busy with other stuff right now. It will take me a couple of weeks to again attend to further development. Then I am going to make this Package 2.0.: #53

I am not moving forward with this idea. Closing.


Add tic tac toe and YouTube Downloader

Have you read the Contributing Guidelines ?
yes
Description
Intermediate level web app that uses sockets to make connections between clients and plays the game.
Fixes #issue_no
#315 and #351
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
https://github.com/Kushal997-das/Project-Guidance/issues/315

@sudip-mondal-2002 One PR at a time .


Add script to run repl

Since there's a repl, it would be really nice to be able to run on our laptops. I think that the easiest way would be for it to run LD_PRELOAD with the unit test shim. Another option would be to build a Linux kernel and run an initramfs via qemu, but that seems like it would lead to a slow dev cycle.

Done. Run make repl


fix(Icons): substitute favorite icon

What:
New favorite icon

Why:
Requirements

How:

Checklist:

[ ] Documentation n/a
[x] Tests
[x] Ready to be merged

Looks like there are snapshot failures in this PR as well.


Use GB units instead of GiB units.

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

@wongma7 PLAL :). Thanks


Refactor: New res Folder

Created new res/ folder to contain any static files (e.g. FactoryUtils, etc.). Also cleaned up minor code nits.

This change is

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 2 unresolved discussions.

res/factory_utils.js, line 1 at r1 (raw file):

/**

Leave this in src.

res/list_element.js, line 2 at r1 (raw file):

/**
 * @license

Leave this in src.

Comments from Reviewable

Moved FactoryUtils and ListElements back.

Review status: 0 of 6 files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable


⚠ SkipTheCommericals has degraded performance

In 08638c3, SkipTheCommericals ($STC) experienced degraded performance:

HTTP code: 200
Response time: 3148 ms

Resolved: SkipTheCommericals performance has improved in 0296ad6.


Update harness-variables.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://655e9bab63efde10e2754d89--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://655e9c24fd1a4117c13e7b31--harness-developer.netlify.app


[Backport 2.9] Bump zstd version to 1.5.5-5 (#9431)

Signed-off-by: Sarthak Aggarwal<EMAIL_ADDRESS>(cherry picked from commit ba78d93acf1da6dae16952d8978de87cb4df2c61)

Description
Backport bumping zstd to 1.5.5-5
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta this might not have the changelog right?

@sarthakaggarwal97 could you please fix changelog? thank you

@sarthakaggarwal97 could you please rebase? thank you


I need help installing SnapWrap

I did the following process in a linux terminal to install it
git clone https://github.com/Rob--/SnapWrap
cd SnapWrap
python setup.py install
python
import snapchat

And I get the following error
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'snapchat'

What am I doing wrong?
Should the folder be in $PATH?
Please help!

I figured this Module is outdated and has outdated dependencies. It need to be updated in order to make it work. Snapchat API has probably also changed in the last 6 years.

Yeah, this wrapper is outdated. Should probably archive the repo!


Update how-to-connect-install-existing-tenant.md

Fixing misleading sentence. Only one attribute needs to match, either UPN or ProxyAddress, not necessarily both.

Docs Build status updates of commit 4113ffa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/hybrid/how-to-connect-install-existing-tenant.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@billmath

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Removing "files before account reset" folder fails

This issue might be related to #661 .
When deleting my "files before account reset" folder I get an Input/output error
➜ ~ $ ls -la /keybase/private/
total 0
dr-x------ 1 elexx root 0 Aug 24 07:07  elexx
dr-x------ 1 elexx root 0 Aug 24 07:07 'elexx (files before account reset 2017-10-29)'

➜ ~ $ ls -la "/keybase/private/elexx (files before account reset 2017-10-29)/"
ls: cannot open directory '/keybase/private/elexx (files before account reset 2017-10-29)/': Input/output error

➜ ~ $ rm -rf "/keybase/private/elexx (files before account reset 2017-10-29)/"
rm: cannot remove '/keybase/private/elexx (files before account reset 2017-10-29)/': Input/output error

Versions:
➜ ~ $ keybase --version
keybase version 2.6.0-20180817210538+0b2dcf191f
➜ ~ $ uname -a
Linux elexxBook 4.15.0-33-generic #36-Ubuntu SMP Wed Aug 15 16:00:05 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Try rmdir.

Thanks for the fast reply, unfortunately I get the same error.
➜ rmdir /keybase/private/elexx (files before account reset 2017-10-29)/
rmdir: failed to remove '/keybase/private/elexx (files before account reset 2017-10-29)/': Input/output error

whereas ls private/elexx works fine:
➜ ~ $ ls -la /keybase/private/elexx
total 0

Can you ignore it in the files tab of the Keybase GUI?

Yes. And now it's also gone from the folder:
➜ ~ $ ls -la /keybase/private/
total 0
dr-x------ 1 alexander root 0 Aug 28 15:31 elexx

Is this a bug, do you want me to keep the issue open?

Just uploaded the logs. The log ID is 06707deaca9cfb720e51c41c

In case it help, I was using rmdir (GNU coreutils) 8.28 when I got the error.

@elexx I just merged #1763, which should fix the issue with rmdir not working for pre-reset folders, in the next Keybase release.


raft: fix race in LeaderAddr call

Without mutex lock it's possible that we'll hit State() call during
Stop() and it will hang forever.
ping @aaronlehmann

Current coverage is 55.43% (diff: 0.00%)

Merging #1391 into master will increase coverage by 0.10%

@@             master      #1391   diff @@
==========================================
  Files            80         80
  Lines         12660      12662     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           7004       7019    +15
+ Misses         4696       4681    -15
- Partials        960        962     +2

Powered by Codecov. Last update fdbd9df...dd55a27

Thanks for finding this.
How about replacing the call to leader with Leader (which uses the lock) instead?

@aaronlehmann I think ErrNoRaftMember is nicer than ErrNoClusterLeader in this particular case.

LGTM

LGTM


feature: makes poly-commit compatible with snarkOS

Ports to the snarkos traits, and uses the new serialization derives

I've resolved the nits, this should be good to merge once we get CI working.


[WIP] Migrate canary helm chart updates to Emissary

Description
Updates to the helm chart that will allow for the optional creation of a new deployment for canary pods that can have a separate image, and additional environment variables from the main deployment while using all of its other resources.
New values:
new canary section to hold all of the following values.

canary.enabled: When true creates a canary deployment that creates canary pods and a canary Ambassador service that will only route traffic to pods from the canary deployment. Everything excluding the following new values are shared with the main deployment.

canary.mixPods: When true allows for requests to the main Ambassador service to hit EITHER the pods from the main deployment or pods from the canary deployment.

canary.replicaCount: Determines the number of Canary pods to be created.

canary.image.repository: Allows for setting the repository for the canary image.

canary.image.tag When set allows for the canary deployment to use a different tag for the image.

canary.envRaw: Takes the place of envRaw for the canary deployment. This ensures that the main env will be copied over to the Canary deployment, but gives a way to provide additional environment variables for only the canary deployment.

NOTE:  do not put a duplicate environment variable in canar.envRaw if it exists in env

_helpers.tpl:
New ambassador.canaryImage vaue that  will set the imgae for the canary deployment. Gives you the ability to overwrite the image.tag and/or the image.repository for the canary deployment.
Testing
Manual testing is all that has been done at the moment until a reviewer can look over this PR.
Checklist

[ ] I made sure to update ./ambassador/CHANGELOG.md.
Remember, the CHANGELOG needs to mention:

Any new features
Whenever the Envoy version changes
Any non-backward-compatible changes
Any deprecations

[x] This is unlikely to impact how Ambassador performs at scale.
Remember, things that might have an impact at scale include:

Any significant changes in memory use that might require adjusting the memory limits
Any significant changes in CPU use that might require adjusting the CPU limits
Anything that might change how many replicas users should use
Changes that impact data-plane latency/scalability
Any change to how we talk to Redis

[ ] My change is adequately tested.
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[x] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently.

Squashed into different branch targeting 2.0


Calendar command not work well

this is buggy for me.
:Calendar 2022 11 18
give me this:

what is 5/24 ?

Please refer to :h calendar-options-date_endian.


added  windows 11

added  windows 11 to this article

Learn Build status updates of commit edced08:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows/security/threat-protection/security-policy-settings/secpol-advanced-security-audit-policy-settings.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Encryption API 2.0

I noticed, that the builder pattern for the encryption/signing API was a bit complex and inflexible.
Most importantly, the user was expected to define which algorithms to use when encrypting. Furthermore sign-only operations would not allow change of algorithms at all (see #75).
This PR introduces a new API which moves options for signing and encrypting into separate builders.
Instead of calling
        EncryptionStream encryptor = PGPainless.encryptAndOrSign()
                .onOutputStream(envelope)
                .toRecipients(recipientPub)
                .usingAlgorithms(SymmetricKeyAlgorithm.AES_256, HashAlgorithm.SHA512, CompressionAlgorithm.ZIP)
                .signWith(keyDecryptor, senderSec)
                .signBinaryDocument()
                .noArmor();

the API has been simplified to
        EncryptionOptions encOpt = new EncryptionOptions()
                .addRecipient(recipientPub);
        SigningOptions signOpt = new SigningOptions()
                .addInlineSignature(keyDecryptor, senderSec, type);

        EncryptionStream encryptor = PGPainless.encryptAndOrSign()
                .onOutputStream(envelope)
                .withOptions(ProducerOptions
                        .signAndEncrypt(encOpt, signOpt)
                        .setAsciiArmor(armor));

The new API evaluates the keys used to sign/encrypt and automatically negotiates algorithms.
It is still possible to use custom algorithms though:
        encOpts.overrideEncryptionAlgorithm(AES_128);
        signOpts.overrideHashAlgorithm(SHA_256);
        prodOpts.overrideCompressionAlgorithm(UNCOMPRESSED);

Default algorithms and fallbacks can be controlled via PGPainless.getPolicy().
Lastly the new API also allows control over which subkeys are used for encryption.
Instead of encrypting to all available encryption subkeys of a keyring, the API will now per default only encrypt to the first encryption capable subkey that is being found.
This behaviour can for example be modified by calling
        encOpts.addRecipient(aliceKey, EncryptionOptions.encryptToAllCapableSubkeys())

Other implementations are possible by implementing EncryptionKeySelector.
All in all, the new API should be way more flexible. We now have better separation of concerns, which should also make the implementation more maintainable and easier for external contributors to understand.
TODO:

[x] squash

Thank you - this looks to be an improvement :+1:

Pull Request Test Coverage Report for Build 501

568 of 809 (70.21%)  changed or added relevant lines in 33 files are covered.
112 unchanged lines in 12 files lost coverage.
Overall coverage increased (+0.3%) to 74.237%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pgpainless-core/src/main/java/org/pgpainless/signature/cleartext_signatures/MultiPassStrategy.java
0
1
0.0%

pgpainless-core/src/main/java/org/pgpainless/util/SignatureTree.java
0
1
0.0%

pgpainless-sop/src/main/java/org/pgpainless/sop/Print.java
5
6
83.33%

pgpainless-core/src/main/java/org/pgpainless/exception/KeyValidationException.java
0
2
0.0%

pgpainless-core/src/main/java/org/pgpainless/key/SubkeyIdentifier.java
2
4
50.0%

pgpainless-sop/src/main/java/org/pgpainless/sop/commands/Armor.java
0
2
0.0%

pgpainless-core/src/main/java/org/pgpainless/algorithm/negotiation/SymmetricKeyAlgorithmNegotiator.java
18
21
85.71%

pgpainless-sop/src/main/java/org/pgpainless/sop/SopKeyUtil.java
6
9
66.67%

pgpainless-core/src/main/java/org/pgpainless/encryption_signing/EncryptionResult.java
30
34
88.24%

pgpainless-core/src/main/java/org/pgpainless/key/info/KeyAccessor.java
19
23
82.61%

Files with Coverage Reduction
New Missed Lines
%

pgpainless-core/src/main/java/org/pgpainless/decryption_verification/OpenPgpMetadata.java
2
84.47%

pgpainless-core/src/main/java/org/pgpainless/encryption_signing/EncryptionBuilder.java
2
76.67%

pgpainless-core/src/main/java/org/pgpainless/key/SubkeyIdentifier.java
2
40.0%

pgpainless-core/src/main/java/org/pgpainless/util/Passphrase.java
2
90.2%

pgpainless-core/src/main/java/org/pgpainless/util/selection/key/impl/NoRevocation.java
2
40.0%

pgpainless-core/src/main/java/org/pgpainless/util/selection/key/impl/SignatureKeySelectionStrategy.java
2
83.33%

pgpainless-sop/src/main/java/org/pgpainless/sop/Print.java
3
50.0%

pgpainless-core/src/main/java/org/pgpainless/encryption_signing/EncryptionStream.java
4
83.58%

pgpainless-core/src/main/java/org/pgpainless/util/Tuple.java
6
0%

pgpainless-core/src/main/java/org/pgpainless/util/selection/key/impl/And.java
9
47.37%

Totals

Change from base Build 475:
0.3%

Covered Lines:
3674

Relevant Lines:
4949

💛  - Coveralls


[Extensibility Request] Page 5050 "ContactCard" - Protected variables & Protected procedure

Hello, we need the following currently global variables defined as protected for our page extension. Can you please check that?
protected var
    CompanyGroupEnabled: Boolean;
    IsOfficeAddin: Boolean;
    OrganizationalLevelCodeEnable: Boolean;
    ParentalConsentReceivedEnable: Boolean;

We also need the following currently Local procedure to be set as protected, for the same purpose.  Can you please check this one too? Thank you!
protected procedure SetParentalConsentReceivedEnable()

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


General lab policies: off-boarding guidelines?

we could add things like expectations and steps for leaving the lab.
one of the things that could be added is updating the lab members tab on the website

Additional points from #40 (@rios-jaime):

making sure their analyses/data have the appropriate permissions on sherlock or are transferred to russ so he has sole control
return room keys to harry


TLO probing inaccurate offset

sender 2.0.41p8
In TLO probing circle offset aplyed with G10L20P1Z0 which use current position instead of probbing result. I's give some speed and acceleration depended error relaive to actual probbing results.
In my case I get 0.02 - 0.04 mm difference to edge-finder Z mesurement results with reasonable slow feed rate.
I also found, G10L20P1Z0 applyed twice ( shown in terminal window)
@terjeio could you please set the same algoithm as for egde finder

Is this in from the Center finder tab or the Tool length offset tab?
FYI I am on holiday and do not have access to a machine to test with so I cannot look into this before I am back home.

Hello @terjeio  Wish you happy New Year!
It's from TLO tab. Edge finder and center finder works correctly.

Happy new year!
You are correct in that the current position is incorrectly used instead of the probed position. I'll fix this and retest when I am back home.

I also found, G10L20P1Z0 applyed twice ( shown in terminal window)

I cannot immediately see this happening from looking at the code, so this has also to wait until I am back.

I have now uploaded a new edge version with a fix.

Verifyed with 2.0.43p3


NSUnknownKeyException', reason: '[<UITabBarItem 0x7ff41c40b3b0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key animation.

I followed all the 5 steps carefully but I don't know why this error is coming.
I tried to search for the solution in stackoverflow and some other websites, but it was not helpful.
Please help

@satyam04 @Akamajin Hey try this NMAnimatedTabBarItem

if you have this issue this class is not key value coding-compliant for the key animation you have a problem with key codding in storyboard