[Samsung Galaxy] fix naming for W20/21 5G

:grey_question: About
Rename Samsung Galaxy W21 5G to Galaxy W21 5G

:tickets: Related stuff

https://github.com/endoflife-date/endoflife.date/pull/2353
https://github.com/endoflife-date/endoflife.date/pull/2336
https://github.com/endoflife-date/endoflife.date/pull/2156

Samsung seems to disagree with the name, the W20 and W21 aren't labeled as a Galaxy phone here: https://security.samsungmobile.com/workScope.smsb
Also not labeled as a Galaxy phone on the support page for the phone: https://www-samsung-com.translate.goog/cn/support/model/SM-W2020ZWACHC/?_x_tr_sl=auto&_x_tr_tl=en&_x_tr_hl=en&_x_tr_pto=wapp

:information_source: These are the foldables ones

ℹ These are the foldables ones

Wait what? There's a foldable and non foldable W20/W21?
Due to these phones being released in only China there's admittedly not that many resources, but I am convinced that outlets naming these under the "Galaxy" brand are mistaken, Samsung doesn't seem to put them under the same brand: https://www.samsung.com/cn/smartphones/all-smartphones/?other

:rofl:

Wait what? There's a foldable and non foldable W20/W21?

Nope, they both are foldable devices:

Samsung W20 5G : un Galaxy Fold 5G équipé du Snapdragon 855+
Samsung W21 5g

:bulb: Could we call them both:

Galaxy Fold W20 5G
Galaxy Fold W21 5G

* [Samsung W20 5G : un Galaxy Fold 5G équipé du Snapdragon 855+](https://leclaireur.fnac.com/article/16684-samsung-w20-5g-un-galaxy-fold-5g-equipe-du-snapdragon-855/)

This source doesn't name it under the Galaxy marketing brand either: https://leclaireur-fnac-com.translate.goog/article/16684-samsung-w20-5g-un-galaxy-fold-5g-equipe-du-snapdragon-855/?_x_tr_sl=fr&_x_tr_tl=en&_x_tr_hl=en&_x_tr_pto=wapp

* [Samsung W21 5g](https://www.sammobile.com/samsung/w21-5g/specs/)

Also doesn't place it under the Galaxy name.

💡 Could we call them both:
* `Galaxy Fold W20 5G`

* `Galaxy Fold W21 5G`

I disagree with this, every Samsung official marketing I've found for the phone explicitly doesn't include it under Galaxy, and the Samsung lifecycle page for phone which is our main source for these phones support cycles also doesn't have Galaxy in its name while it does for every single other phone.
I'm ok with naming it W20 5G and W21 5G respectively, which is how it is named in every official Samsung wording I've found.
Sources for the name:
Lifecycle page: https://security.samsungmobile.com/workScope.smsb
Support page for the phone (not sure what you meant by this being wrong, it's using google translate if that confused you): https://www-samsung-com.translate.goog/cn/support/model/SM-W2020ZWACHC/?_x_tr_sl=auto&_x_tr_tl=en&_x_tr_hl=en&_x_tr_pto=wapp
Store: https://www.samsung.com/cn/smartphones/all-smartphones/?other

Am I going mad or am I missing something here? Damn these ridiculous phone marketing departments

@Evernow : what about discarding the current PR in favor of this one :

https://github.com/endoflife-date/endoflife.date/pull/2387

@Evernow : what about discarding the current PR in favor of this one :
* [[Samsung] Add missing devices #2387](https://github.com/endoflife-date/endoflife.date/pull/2387)

That simplifies things, just a matter of renaming the following:
"Samsung W21 5G" -> "W21 5G"

:heavy_check_mark:  Done

Closing this.

I :sponge: the branch :zap:


Integration couldn’t be loaded since updating Home Assistant to core-2021-12-0.

 The Log shows the following error...
KeyError: 'allow_cors'

Okay, sorry. I saw there was already an issue opened.
Changing allow_cors to allow_all_cors in in utils.py worked!


cocomac: Watch jobiworld.com

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

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

Approved by Spevacus in Charcoal HQ


[SPARK-15688][SQL] RelationalGroupedDataset.toDF should not add group by xpressions that are already added in the aggregate expressions

What changes were proposed in this pull request?
The dataset aggregate methods (agg) computes aggregates and produces
the resulting dataset. Currently the resulting data set includes the
grouping columns even though the aggregation expression already
contains the columns resulting in duplicate columns.
val df = Seq(1 -> "a", 2 -> "b", 3 -> "b").toDF("col1", "col2")
val df1 = df.groupBy("col1").agg($"col1", count("*")).show()
+----+----+--------+
|col1|col1|count(1)|
+----+----+--------+
|   3|   3|       1|
|   1|   1|       1|
|   2|   2|       1|
+----+----+--------+

This PR addresses the problem by only including grouping columns
in the dataset if they were not already contained by the aggregate
expressions.
val df1 = df.groupBy("col1").agg($"col1", count("*")).show()
+----+--------+
|col1|count(1)|
+----+--------+
|   3|       1|
|   1|       1|
|   2|       1|
+----+--------+

Normal Aggregate plan.
Aggregate [col1#5], [col1#5,count(1) AS count(1)#15L]
+- Project [_1#2 AS col1#5,_2#3 AS col2#6]
   +- LocalRelation [_1#2,_2#3], [[0,1,1800000001,61],[0,2,1800000001,62],[0,3,1800000001,62]]

Plan with window expressions in aggregate.
Project [sum(earnings)#21,grouping_id(course, year)#22,RANK() OVER (PARTITION BY grouping_id(course, year) ORDER BY sum(earnings) ASC UnspecifiedFrame)#23]
+- Project [sum(earnings)#21,grouping_id(course, year)#22,_w0#24,_w1#25,RANK() OVER (PARTITION BY grouping_id(course, year) ORDER BY sum(earnings) ASC UnspecifiedFrame)#23,RANK() OVER (PARTITION BY grouping_id(course, year) ORDER BY sum(earnings) ASC UnspecifiedFrame)#23]
   +- Window [rank(_w1#25) windowspecdefinition(_w0#24, _w1#25 ASC, ROWS BETWEEN UNBOUNDED PRECEDING AND CURRENT ROW) AS RANK() OVER (PARTITION BY grouping_id(course, year) ORDER BY sum(earnings) ASC UnspecifiedFrame)#23], [_w0#24], [_w1#25 ASC]
      +- !Aggregate [course#4,year#5], [sum(earnings#6) AS sum(earnings)#21,grouping_id(course#4, year#5) AS grouping_id(course, year)#22,grouping_id(course#4, year#5) AS _w0#24,sum(earnings#6) AS _w1#25]
         +- LogicalRDD [course#4,year#5,earnings#6], MapPartitionsRDD[1] at apply at Transformer.scala:22

How was this patch tested?
Added tests in DatasetAggregatorSuite

Test build #59896 has started for PR 13483 at commit 393f589.

Test build #59896 has finished for PR 13483 at commit 393f589.

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

Please update the PR description and post the analyzed plan. Also added another example to show the root node is Window. Thanks!

Test build #59917 has started for PR 13483 at commit 6e59f9d.

@gatorsmile added comments and also updated the PR description.

LGTM except a few comments in test cases. : )
@cloud-fan @yhuai Could you please review this PR? Thanks!

Test build #59922 has started for PR 13483 at commit f010f94.

Test build #59917 has finished for PR 13483 at commit 6e59f9d.

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

As the setting spark.sql.retainGroupColumns is set to true by default, GroupedData.agg will retain group by columns. So the problem is you explicitly ask the column col1 when calling agg method. That is like when users explicitly do SELECT a, a, but we automatically convert it to SELECT a. Looks unreasonable to me.

Test build #59922 has finished for PR 13483 at commit f010f94.

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

@viirya Based on my understanding, we want to avoid having duplicate output columns generated by our APIs. If users want to explicitly have two duplicate output columns, they can specify them in agg. That  For example,
df.groupBy("col1").agg($"col1", $"col1", count("*"))

In addition, spark.sql.retainGroupColumns is an internal parameter. The default value is true. That means, users might not realize they can disable it. Thus, it makes more sense to avoid generating duplicate columns that are caused by retaining group-by expressions.

@gatorsmile Thanks a lot for your explanation. I agree that if we internally generate the aggregate expressions that is different than user specified one , then we should eliminate the duplicate expressions as opposed to simply adding the the grouping expressions like we do today.

I don't think it's a big problem. After users understand the semantic, they can easily work around it, by not putting grouping columns in agg list, or turning off the config.
Well, it looks better if we can deduplicate it automatically, but seems it's not that easy, according to this PR. Maybe it doesn't worth.
cc @marmbrus

@gatorsmile this automatically deduplicate the group by columns will cause confusion. Using your example:
df.groupBy("col1").agg(count("*"))

When users try the above API call, they will find that groupBy will retain group by columns. Then as they try:
df.groupBy("col1").agg($"col1", count("*"))

They will figure out that adding group by columns will make duplicate ones.
However, if we automatically deduplicate them. The users first think that groupBy will include group by columns. Then when adding group by columns explicitly, the duplicate group by columns are not shown up. But as they add more, the group by columns show up again...
Personally I think the behavior above is inconsistent.

@viirya This is a design decision. So far, both ways are not perfect.
In my mind, we have to consider the use cases here. If users want to have duplicate columns, they should not use the duplicate names. Do you think this makes sense?
That means, we should not remove the duplicate in the following scenario:
df.groupBy("col1").agg($"col1".as("col1_replica"), count("*"))

The expected output schema:
+----+------------+--------+
|col1|col1_replica|count(1)|
+----+------------+--------+

If they do not change the column name, I am unable to find any usage senario for duplicating the columns.
df.groupBy("col1").agg($"col1", count("*"))
df.groupBy("col1").agg(count("*"))

The expected output schema of the above two:
+----+--------+
|col1|count(1)|
+----+--------+

What is your opinions?

@gatorsmile Your last example just shows the inconsistency. Given two different parameters, $"col1", count("*") and count("*"), you get the same output. I think this confuses users. In contrast, original API usage has consistent behavior: group by columns are retained, and when you add group by columns in agg API call, it means you want to add duplicate ones.
Of course, that is my opinion. I am not against this change if others think it is ok.

@viirya I am just wondering why users need a dataframe with duplicate column names? Could you give me a usage scenario?

@gatorsmile I am not stand for it due to actual use cases, but the API behavior consistency. If we disallow duplicate group by columns, then we should do filtering them all. Let
df.groupBy("col1").agg($"col1", $"col1", count("*"))

show duplicate group by columns but disallow
df.groupBy("col1").agg($"col1", count("*"))

looks confusing...

@viirya You know, as I said above, both ways are not perfect. My inputs are just based on my previous design experiences. All the design decisions I made are based on usage scenarios.
This PR is to refine the behavior of spark.sql.retainGroupColumns. I think that might be why @yhuai opened this JIRA.
The problem you mentioned above is another issue. I think we should not deduplicating the columns, if users really want to do something absurd. : )

I haven't looked at this PR in detail, but I'm the originator of this suggestion.  It seemed odd to me that we were auto adding columns when I was already specifying them.  I agree if the user manually specifies a column twice, then it needs to show up twice.
Additionally, we are now too late in the 2.0 process to make changes to the semantics.  So its probably okay to just close this as wont fix.

@marmbrus Thanks Michael. I am going to close this.

Thank you @marmbrus @dilipbiswal and @viirya !


Installation failure on ubuntu 10

It seems like binding.gyp has dos/windows line endings.  This in conjunction with the default level of python in ubuntu 10 results in a failure during install.
Running dos2unix on binding.gyp seems to resolve the issue.
See the following for more details: https://developer.ibm.com/answers/questions/167322/node-gyp-rebuild-fails-during-compile-phase.html#answer-168535
It would be good to fix as ubuntu 10 is used as the default environment for many PaaS environments.

@mdawsonibm , can you please fork last commit Convert line endings to LF eaae2c7e45c383d6704e0ccf43fe92439d406460 and submit, that all works correctly.

I added some fixes to v8-profiler and v8-debug. Also I added prebuild binaries for Linux and OSX.
Please try to reinstall node-inspector now.

Sorry I did not get to this earlier.  I just did an npm install in my unbuntu environment and both v8-profiler and v8-debug reported as installing ok.  It reports:
ubuntu@ub10:~/test4$ npm install v8-profiler
|

<EMAIL_ADDRESS>install /home/ubuntu/test4/node_modules/v8-profiler
node-pre-gyp install --fallback-to-build

[v8-profiler] Success: "/home/ubuntu/test4/node_modules/v8-profiler/build/profiler/v5.2.2/node-v11-linux-x64/profiler.node" is installed via remote
<EMAIL_ADDRESS>node_modules/v8-profiler
âââ<EMAIL_ADDRESS>I'll ask the original person who reported the issue on dw Answers to validate it works for them (I just tested the install as opposed to using the modules)


Test patched Drupal modules

This is an attempt at running the Drupal unit tests for Flysystem (or rather, seeing if we can get any module's tests to run).

🙌 I don't think we need this anymore now we have tests running 🥳


/ is only for floats

$ starlark-go -c '0/(1/2)'
cmdline:1:2: this Starlark dialect does not support floating point (use //)
(exit 1)

$ starlark-go -float -c '0/(1/2)'

(exit 0)

$ starlark-rust -c '0/(1/2)'
error[CV08]: Cannot divide by zero
 --> [command flag]:1:1
  |
1 | 0/(1/2)
  | ^^^^^^^ Division by zero

(exit 2)

starlark-rust should either reject this program because floating point support is not enabled or return 0 without error.

Filed https://github.com/bazelbuild/starlark/issues/31 to clarify the spec


[Bugs] DMP CMS

Unexpected token '<', "<!doctype "... is not valid JSON

Current open threads:

IDinsight & Shikshalokam-2 tickets not showing up
The above Arghyam ticket PRs not showing up
Certain project pages crashing instead of showing the ticket
Weekly goals for certain projects (as raised on the dmp contributor group) not showing up
Weekly learnings not being displayed correctly (for certain projects as raised on dmp contributor group)
Progress bar not moving even though checkboxes are ticked in one project (as raised on the dmp contributor group)

Arghyam - PRs raised before 11th June
IDinsight: https://github.com/IDinsight/aaq-core/issues/128 changed to https://github.com/IDinsight/ask-a-question/issues/128
Shikshalokam: https://github.com/ELEVATE-Project/sl-backend-automation/issues/19 repo in db is test-automation-portal

IDinsight & Shikshalokam-2 tickets not showing up - db error - tested and closed
The above Arghyam ticket PRs not showing up - TBP
Weekly goals for certain projects (as raised on the dmp contributor group) not showing up - tested and closed
Weekly learnings not being displayed correctly (for certain projects as raised on dmp contributor group) - tested and closed
Progress bar not moving even though checkboxes are ticked in one project (as raised on the dmp contributor group)- tested and closed

Arghyam ticket PRs do not contain issue number in the PR title, so they aren't getting picked

Issue ticket numbers to be added to PR titles @MohitNSamagra


How to install default packages to all sessions ?

How would this feature be useful?
I'm working in an Azure devOps company environment so we use a custom feed for our private Python packages. In my CI I set an environment variable  PIP_EXTRA_INDEX_URL to have access to this feed.
As this is an env variable it will crash if I don't start by installing both keyring and artifacts-keyring for authentication.
At the moment I'm forced to run the following in each session definition:
@nox.session(reuse_venv=True)
def something(session):
    session.install("keyring", "artifacts-keyring") # need to be the first thing I do
    session.install("pre-commit")
    session.run("pre-commit", "run", "--a", *session.posargs)

I think what is missing is the inverse of session.notify("other_session") + share the venv. Right now you can construct a chain of things that are done after, but not what should be done before.
Something like session.require("common", share_venv=True), which would invoke the "common" session where you define whatever you need as preparation BEFORE the rest of this session, and it is executed on the venv of the invoking session. The notify alone does not cut it because from the point of view of common you have no idea of what should be done after.
The share_venv would also be useful for .notify, so you can pile up actions happening on the same environment?


DOC: Fix broken url to Scientific Python Development Guide

At some point urls were changed to use '-' over '_'.

I'm going to approve and merge this myself given it is the only link in the README that is broken. As always, PRs approved by a single core dev can be reverted as needed by the rest of the team.


Show hint if user tries to click phrases

In annotation editor, the phrases can only be drag'n'dropped, which may be unclear to users. If the user clicks a phrase, a hint (such as tool tip or popover) should be shown (e.g. "Drag'n'drop phrases onto the submission.")

Done, pending review


A new mode to call Update at the same timings as Draw

Aside from UncappedTPS (= -1), we might need one more special TPS.
CC @eihigh

Oops this is exactly what UncappedTPS indicates... My bad.


Can't find credentials for demo website

Hey. Love this project, you're doing amazing work!
I've been looking for the last couple minutes to find the credentials to access the demo at https://demo.unfoldadmin.com/admin/login/?next=/admin/. I know I managed to access it in the past, but I can't find it anymore.
If they are still there somewhere, they probably need to be more obvious (maybe even in the help_texts of the username and password field).
Thanks

Hey, tomorrow I will redeploy demo website with proper settings. Thanks for pointing at this issue.
If you want you can try demo site locally:
https://github.com/unfoldadmin/formula

Fixed.


slack link problem on landing page

There is a typo in the link: "hhttps://chainsfr.slack.com"

Switch from slack channel to rocket chat.


Windows 11: app stops working after sometime and the taskbar reverts to auto-hide even on desktop

Does it always stop working after running for a while?

@ChanpleCai Yes. After a working as expected a few times, it just reverts to auto-hide. Also  double clicking the overflow menu icon doesn't toggle auto mode, it just switches between always show and auto-hide. Is that expected?

I don't think that is the case, because it is highly unlikely that some invisible full-screen application is running some time after logging in and then staying on-screen indefinitely.
I can see where you're coming from, though. Wallpaper Engine used to cause this issue for the same reason you're implying, but I can assure you, that's not running.
I think this is a different issue. Are there any logs you'd be interested in checking out?

Looks like this was resolved in a subsequent update. Thanks!


Update README.md

Update readme to reflect Navie release.

:tada: This PR is included in version 0.66.0 :tada:
The release is available on:

v0.66.0
GitHub release

Your semantic-release bot :package::rocket:


Support for streaming RAW images

Question about an existing feature
RAW images take up a lot of memory. Is there a way to stream RAW to reduce memory requirements?
What are you trying to achieve?
Stream RAW input to reduce memory requirements.
When you searched for similar issues, what did you find that might be related?
#220
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this question
Example from https://sharp.pixelplumbing.com/api-constructor has the entire image in memory instead of streaming.
// Read a raw array of pixels and save it to a png
const input = Uint8Array.from([255, 255, 255, 0, 0, 0]); // or Uint8ClampedArray
const image = sharp(input, {
  // because the input does not contain its dimensions or how many channels it has
  // we need to specify it in the constructor options
  raw: {
    width: 2,
    height: 1,
    channels: 3
  }
});
await image.toFile('my-two-pixels.png');

Please provide sample image(s) that help explain this question
N/A

No, sorry, this is something that would probably be part of the future possible enhancement detailed at https://github.com/lovell/sharp/issues/179

Let's continue to track this at #179


Typescript integration fails on beforeRoute* guards implemented on the class

Version
3.0.1
Reproduction link
https://gist.github.com/silkentrance/9caa45481329f6ef911bd5eae5dbb7c3
Steps to reproduce
Just npm install vue-router, vue, and vue-class-component or integrate the gist into an existing project, adjusting the import paths where necessary.
Then run the server and navigate to root '/'.
In the console log there should now be multiple entries, with the outputs from beforeRouteEnter/beforeRouteUpdate missing.
What is expected?
The guards implemented on the class as methods must be called.
What is actually happening?
The guards implemented on the class are never called.

Registering the beforeRouteEnter and beforeRouteUpdate and beforeRouteLeave hooks with Component.registerHooks() and then implementing these hooks on the class will not work as expected as vue-router still tries to access these guards directly from def.options, see below
https://github.com/vuejs/vue-router/blob/d539788df8394efe41c1534e1e9d1555aa2edbe2/src/history/base.js#L262
The solution will be to also query the methods property of these options and return the available guards from there, e.g.
  if (def.options[key]) {
    return def.options[key];
  }
  return def.options.methods ? def.options.methods[key] : null;

I can prepare a PR for this, if you like.

Darn, so much work. Yet, the error lies in vue-class-component.
Closing.

I ran into the same issue and ask to reopen the issue, so it can be fixed.

@babielmam I issued this over at https://github.com/vuejs/vue-class-component/issues/193
The solution is to register the hooks in a separate file, see the updated gist.


Power BI REST API - documentation structure

Power BI API documentation is very unfriendly and illegible.
E.g. https://docs.microsoft.com/en-us/rest/api/power-bi/datasets
Table of content of "In this article" contains only "Required scopes" which does not make much sense nor it is useful. The same applies for all other APIs, not only Datasets.

Hi @romanzdk, thanks for your feedback. I know that the REST APIs dealing with datasets have been undergoing some work recently, so it is possible some of what you are seeing is the result of that, but in any event I am forwarding your comment along to the person who monitors the REST API documentation. Thanks again for reaching out!


Minor correction in the opening sentence on Tuples

I was reading the documentation and discovered a minor issue. The first sentence needed to be corrected.

Hi @IEvangelist, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@IEvangelist Thank you for spotting this and submitting the fix.  We really appreciate it.
I'll merge this now, and it should show up on the site by the end of the week. Thanks again. :shipit:


Opam package calls the 'install' tool with an illegal option on OSX

When installing the fstar opam package on OSX $ opam install fstar
there is yet another boring incompatibility between Linux and Unix tools...
# install -m 755 -D -T fstar.exe /Users/bbeurdouche/.opam/system/bin/fstar.exe
### stderr ###
# [...]
# Warning 26: unused variable for1_sig.
# File "FStar_Extraction_ML_ExtractExp.ml", line 547, characters 8-10:
# Warning 26: unused variable t0.
# install: illegal option -- D
# usage: install [-bCcpSsv] [-B suffix] [-f flags] [-g group] [-m mode]
#                [-o owner] file1 file2
#        install [-bCcpSsv] [-B suffix] [-f flags] [-g group] [-m mode]
#                [-o owner] file1 ... fileN directory
#        install -d [-v] [-g group] [-m mode] [-o owner] directory ...
# make: *** [install] Error 64

Urgh. Any ideas Benjamin?

Unfortunately, I am not sure there is a generic way to circumvent this kind of problems...
That said, everyone that has installed coreutils through Homebrew or Macports has all those utilities in the GNU version available by prefixing the tools with g (ginstall -D will work properly while install -D fails).
A fix could be to simply search if ginstall exists and use it if it is available.
It's not generic, still it will be a good fix a lot of people...

@msprotz, I think, I'll patch the Makefile this way in order to fix (#468) ...

Yes that would be good.
That sounds like one more argument in favor of a configure... see the last few lines of https://github.com/protz/mezzo/blob/master/configure to see how to detect this kind of stuff

Fixed by 85f3ba4e


Use the "linux" component type within the cgmanifest whenever appropriate (e.g. clang-format)

For example, the following is currently marked as "other":
https://github.com/Azure/azure-sdk-for-c/blob/75a4910e7e034949897e9cc2cb88f4a168391f3d/cgmanifest.json#L86-L96
https://docs.opensource.microsoft.com/tools/cg/features/cgmanifest/

The component type defined the package type it can be npm, nuget, maven, etc ... in this case this is an ubuntu package for what I can see, when the packages are from ubuntu, debian alpine or other linux distribution we recommend to use the component type "linux"

cc @RickWinter, @jcfiorenzano

See https://github.com/Azure/azure-sdk-for-c/pull/1902#discussion_r703913334


Atom icon does not look good [small], colours are confusing

Before I downloaded atom I was confronted with this view, which I think is really appealing.

After using atom I often struggled to find the tab in my app bar (I'm using Windows btw) and furthermore I think the icon for files doesn't look good [when small]
In the taskbar there is a small light green ball, the atom is not visible at all for me. I do not associate any of this with the atom I was introduced to

I often use the details view in windows and i also think the atom icon does not come across well here

I suggest to invert the atom icon, making it green with a white atom or applying the (brown, bronze) colours from the website. Same
For the app bar I suggest either thicker lines, no background and/or different colours

You mean something similar to this!

Wow, this looks really good, great work!
Since I'm a new atom user, I might be missing some history: why is the icon green? Is this what the atom brand used to be? I am only familiar with the current website, which does not make a green impression on me, wouldn't it be good if both were the same?
Meaning either the website go green or the logo go brown/bronze?

Yeah that would be sweet!

Yeah that would be sweet!

Sehr geehrte Damen und Herren,
leider bin ich bis zum 10.01.2021 nicht erreichbar, werde Ihre Nachricht jedoch nach meiner Rückkehr schnellstmöglich bearbeiten. Wenden Sie sich bitte in dringenden Fällen an<EMAIL_ADDRESS>Vielen Dank für Ihr Verständnis.
Mit freundlichen Grüßen
Jay Trees

Sehr geehrte Damen und Herren,
leider bin ich bis zum 10.01.2021 nicht erreichbar, werde Ihre Nachricht jedoch nach meiner Rückkehr schnellstmöglich bearbeiten. Wenden Sie sich bitte in dringenden Fällen an<EMAIL_ADDRESS>Vielen Dank für Ihr Verständnis.
Mit freundlichen Grüßen
Jay Trees


⚠ Bofrost.fr has degraded performance

In fd1a72c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1992 ms

Resolved: Bofrost.fr performance has improved in 798dbd7 after 6 minutes.


refactor(execution): move bouncer-related code to its file

This change is

Codecov Report
Attention: Patch coverage is 97.14286% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 76.06%. Comparing base (20a6a9e) to head (6d0f1be).
Report is 1 commits behind head on main.

Files
Patch %
Lines

crates/blockifier/src/bouncer.rs
97.14%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1829   +/-   ##
=======================================
  Coverage   76.06%   76.06%
=======================================
  Files          60       60
  Lines        8556     8556
  Branches     8556     8556
=======================================
  Hits         6508     6508
  Misses       1622     1622
  Partials      426      426

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


stmgr did not quit by LOG(FATAL)

We observed stmgr stuck in a particular scenario.
In the heron-executor log, the stmgr restarted 3 times, but in the last restart, stmgr seems stuck and did not quit.

I expected exited with status 134. at 10:14:11, but there was not. Thus I think that stmgr stuck at  OG(FATAL).

any thoughts to fix this @srkukarni

LOG(FATAL) is supposed to kill the process.
One thing that we could do is maybe add an explicit ::exit() after FATAL?

This rarely happens so I'd like to keep the current state and wait if we see more of this error.

I believe LOG(FATAL) calls exit() to quit. If the process freezes, it seems the code is not executed. Calling ::exit() doesn't help.
The root cause is to solve the freezing issue itself.

should be fixed in #2908, cannot reproduce in the recent releases


These tests require additional libraries to run

They are currently just not running via travis because
they require additional libraries to run and hence they appear to
be passing. But in fact - if necessary client libraies are installed
these tests are failing.

This is just a short term fix until we can prioritize getting all the tests to pass.


fix: make it iOS 13 compatible

UInt32 is deprecated on iOS 13.

@mirkancal thanks for your contribution!


[Bug]: SQLTalk demo is using VertexAI SDK, not Gemini SDK but is branded as GeminiAI

Description of the bug:
SQLTalk, a sample Streamlit app demonstrating Vertex AI's function calling with BigQuery was recently shared by Google: Video, Tutorial, Code, Live demo.
Even though the code lives in Gemini repo (see the full path), it is actually using Vertex AI SDK, not Gemini SDK. The Github page actually says that this repo is for Vertex AI:

But, the Tutorial and Video mention Gemini AI, not Vertex AI.
In summary, A sample app, which was branded as Gemini AI in the video and docs , is actually using vertexai SDK but is present under "gemini" folder in a repo that is for VertexAI only.
Actual vs expected behavior:
Provide a version of SQLTalk app that uses Gemini SDK, not VertexAI SDK;
This is not a simple matter of renaming vertexai.generative_models to google.generativeai because there are subtle differences in both the SDKs.
While this line from SQLTalk app sample code works fine:
from vertexai.generative_models import FunctionDeclaration, GenerativeModel, Part, Tool

This fails:
from google.generativeai import FunctionDeclaration, GenerativeModel, Part, Tool

because the GeminiSDK does not have FunctionDeclaration, Part and Tool but does have GenerativeModel.
To someone new to GeminiAI/VertexAI, all this is highly confusing.
Any other information you'd like to share?
No response

Cross-posted to GoogleCloud/VertexAI organization's repo: https://github.com/GoogleCloudPlatform/generative-ai/issues/756


ARK server info

What are your thoughts on either having an info command or integrating into the status command basic server info (i.e. server name including version, and number of players)?
So far, the only cross-distribution way that doesn't involve compiling a program uses either python or perl.  Both can take their scripts as multi-line strings arguments.
I looked at bash networking, but it's compiled out on a lot of distributions.
nc isn't universal, and sometimes ncat will be installed instead of nc.
If python or perl are not present, then the distribution must be very limited indeed.
python:
python -c "
import socket

try:
  sock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
  sock.settimeout(1.0)
  sock.sendto('\xff\xff\xff\xffTSource Engine Query\x00', ('<IP_ADDRESS>', ${ark_QueryPort}))
  data, addr = sock.recvfrom(1400)
  servername, mapname, game, fullname, rest = data[6:-1].split('\0', 4)
  players = ord(rest[2])
  maxplayers = ord(rest[3])
  print('Server Name: {0}'.format(servername))
  print('Players: {0} / {1}'.format(players, maxplayers))
except:
  print('Unable to query server')
"

perl:
perl -e "my \$port = ${ark_QueryPort}" -e '
use Socket;

socket(my $socket, PF_INET, SOCK_DGRAM, 0);
setsockopt($socket, SOL_SOCKET, SO_RCVTIMEO, pack("i4", 1, 0, 0, 0));
my $sockaddr = pack_sockaddr_in($port, inet_aton("<IP_ADDRESS>"));
send($socket, "\xff\xff\xff\xffTSource Engine Query\x00", 0, $sockaddr);
my $data = "";
recv($socket, $data, 1400, 0) or (print "Unable to query server\n" and exit(1));
my ($servername, $mapname, $game, $fullname, $rest) = split(/\x00/, substr($data, 6), 5);
my $players = ord(substr($rest, 2, 1));
my $maxplayers = ord(substr($rest, 3, 1));
print "Server Name: $servername\n";
print "Players: $players / $maxplayers\n";
'

I think I'd prefer the perl version, I'm not sure if it should be part of arkmanager or be a different script of the ARK Server Tools tho

Maybe we could even prepare a small node.js web app to provide a web interface to show these values. Without have to ask our users to configure nginx and so on.

We should wait for official RCON command to do that.

@klightspeed do you know how to perform the same query with Node.js? I'm writing a little widget for NodeBB and I'm kinda stuck :/

RCON is coming.
http://steamcommunity.com/app/346110/discussions/0/594820656447032287/

yey :D

And an administration UI even! I like what i'm seeing.

This can be closed, doesn't it?

Yes, the main feature in this issue has been fulfilled.
The Admin UI suggested by @FezVrasta would be a separate feature.


Music keeps on jumping (stop-play)

Compatible Device

[X] I'm running Raspotify on a compatible Device

Are you sure?

[X] I'm not trying to run Raspotify on a ARMv6 device

Compatible OS

[X] I'm running Raspotify on a compatible OS

Compatible Configuration

[X] I'm running Raspotify on a compatible Configuration

Latest Version

[X] I'm running the latest version of Raspotify

Are you sure?

[X] I'm not running Raspotify <IP_ADDRESS> on a Pi v1 or Pi Zero

Due Diligence

[X] I have done my due diligence

What happened?
Music keeps on jumping. Already restarted the service but still the same.
Please see attached sceenshot from systemd status

Relevant log output and/or the contents of /etc/raspotify/crash_report if any ( sudo journalctl -u raspotify -b and sudo cat /etc/raspotify/crash_report )
Jul 10 12:29:41 raspberrypi librespot[22891]: [2024-07-10T04:29:41Z WARN  librespot_playback::audio_backend::alsa] Error writing from AlsaSink Buffer to PCM, trying to recover, ALSA function 'snd_pcm_writei' failed with error 'EPIPE: Bro
ken pipe'
Jul 10 12:29:41 raspberrypi librespot[22891]: ALSA lib pcm.c:8570:(snd_pcm_recover) underrun occurred
Jul 10 12:29:42 raspberrypi librespot[22891]: [2024-07-10T04:29:42Z WARN  librespot_playback::audio_backend::alsa] Error writing from AlsaSink Buffer to PCM, trying to recover, ALSA function 'snd_pcm_writei' failed with error 'EPIPE: Bro
ken pipe'
Jul 10 12:29:42 raspberrypi librespot[22891]: ALSA lib pcm.c:8570:(snd_pcm_recover) underrun occurred
Jul 10 12:30:02 raspberrypi librespot[22891]: [2024-07-10T04:30:02Z WARN  librespot_playback::audio_backend::alsa] Error writing from AlsaSink Buffer to PCM, trying to recover, ALSA function 'snd_pcm_writei' failed with error 'EPIPE: Bro
ken pipe'
Jul 10 12:30:02 raspberrypi librespot[22891]: ALSA lib pcm.c:8570:(snd_pcm_recover) underrun occurred
Jul 10 12:30:15 raspberrypi librespot[22891]: [2024-07-10T04:30:15Z WARN  librespot_playback::audio_backend::alsa] Error writing from AlsaSink Buffer to PCM, trying to recover, ALSA function 'snd_pcm_writei' failed with error 'EPIPE: Bro
ken pipe'
Jul 10 12:30:15 raspberrypi librespot[22891]: ALSA lib pcm.c:8570:(snd_pcm_recover) underrun occurred
Jul 10 12:30:16 raspberrypi librespot[22891]: [2024-07-10T04:30:16Z WARN  librespot_playback::audio_backend::alsa] Error writing from AlsaSink Buffer to PCM, trying to recover, ALSA function 'snd_pcm_writei' failed with error 'EPIPE: Bro
ken pipe'
Jul 10 12:30:16 raspberrypi librespot[22891]: ALSA lib pcm.c:8570:(snd_pcm_recover) underrun occurred

same problem here

Same problem here. This suddenly occurred. No changes made to the system.

Does latest release resolve this?

Does latest release resolve this?

nice seeing you.
I only occasionally update upgrade and will say that for whatever reason, the latest actually became unuseable.
I also run shairport alongside and am using it instead at the moment..
dmesg revealed that raspotify would open and close the audio device repeatedly until it gave up after maybe 30 sec.

@giggywithit will you please post those logs from raspotify and dmesg?

Initially, I did an update and had the same issues, then did a full reinstall and still have the same issues...
The only thing I think it might not like is my config?
service raspotify status
○ raspotify.service
Loaded: masked (Reason: Unit raspotify.service is masked.)
Active: inactive (dead)
For whatever reason, this time there was no event in trying to open the audio device... But this is the way it looked the other day.. dmesg
[ 1946.741086] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_unmute: un-muting amp using GPIO22
[ 1951.958341] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_mute: muting amp using GPIO22
[ 1956.908575] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1956.908731] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1956.909423] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1956.909643] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1956.910032] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1956.910863] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_unmute: un-muting amp using GPIO22
[ 1961.942116] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_mute: muting amp using GPIO22
[ 1967.082001] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1967.082162] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1967.082860] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1967.083084] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1967.083476] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1967.084308] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_unmute: un-muting amp using GPIO22
[ 1972.181875] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_mute: muting amp using GPIO22
[ 1977.279098] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1977.279250] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1977.279950] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1977.280172] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1977.280561] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 1977.281391] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_unmute: un-muting amp using GPIO22
[ 1982.421638] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_mute: muting amp using GPIO22
[ 2245.570677] pcm512x 1-004c: No SCLK, using BCLK: -2
[ 2245.573188] snd-rpi-iqaudio-dac soc:sound: snd_rpi_iqaudio_gpio_unmute: un-muting amp using GPIO22

It would be helpful if you post your config if you think the config is the problem.
Also missing logs from raspotify itself. These dmesg messages don't look like there is an issue really.

It was the config
For some reason, it didn't like the
LIBRESPOT_AUTOPLAY=
being left empty or something...
I just commented it out and it's working so far
cat /etc/raspotify/conf
LIBRESPOT_QUIET=
LIBRESPOT_ALSA_MIXER_DEVICE="hw:0"
LIBRESPOT_MIXER="alsa"
LIBRESPOT_ALSA_MIXER_CONTROL="Digital"
LIBRESPOT_NAME="COX-GhostAmp"
LIBRESPOT_BITRATE="320"
LIBRESPOT_INITIAL_VOLUME="20"
Volume control scale type {cubic|fixed|linear|log}.
Defaults to log.
LIBRESPOT_VOLUME_CTRL="cubic"
#LIBRESPOT_AUTOPLAY=
LIBRESPOT_DISABLE_AUDIO_CACHE=
LIBRESPOT_DISABLE_CREDENTIAL_CACHE=
LIBRESPOT_ENABLE_VOLUME_NORMALISATION=
TMPDIR=/tmp

Alright, now I have discovered that I have the same issues as the above users.
My player will start and stop on it's own, usually stop, mid-song or otherwise..
My amplifier is controlled by audio being used or not, and at times it will just turn on all by itself.
I have also experienced the underrun conditions as above..
raSpotify is useless at the moment, had to uninstall

Had to reboot to get it working again.
Oct 12 20:04:39 RPi4Spotify systemd[1]: Started raspotify.service - Raspotify (Spotify Connect Client). Oct 12 20:06:44 RPi4Spotify librespot[13068]: [2024-10-12T18:06:44Z ERROR librespot_connect::spirc] user attributes update selected, but none received Oct 12 20:07:14 RPi4Spotify librespot[13068]: [2024-10-12T18:07:14Z ERROR librespot_connect::spirc] user attributes mutation selected, but none received Oct 12 20:07:44 RPi4Spotify librespot[13068]: [2024-10-12T18:07:44Z WARN  librespot_connect::spirc] Cannot flush spirc event sender when done.

Please try the 0.45.0 release which contains librespot 0.5.0.

Please try the 0.45.0 release which contains librespot 0.5.0.

Seems good so far..
Thank you

@andrewhamili it seems other users are having luck with the proposed upgrade, please confirm whether this works for you.

fingers crossed 🤞


Travis PR build "endless"

Expected Behaviour
When travis build is in devel or master branch, it gets "endless" because the registered IPs there doesn't exist anymore.
*Current Beahaviour
Endless builds
*Steps to reproduce unexpected behaviour
Just commit in master or devel, or open a PR.
Checklist

[x] Issue has labels.
[x] Issue has screenshots if required.
[x] Issue has an significant name.

Boa tarde pessoal, posso pegar essa tarefa?


Add i2c2 clock init

IMPORTANT INFORMATION
Contributor License Agreement (CLA)

The Pull Request feature will be considered by STMicroelectronics after the signature of a Contributor License Agreement (CLA) by the submitter.
If you did not sign such agreement, please follow the steps mentioned in the CONTRIBUTING.md file.

This pull request has been refused, the Contribution License Agreement must be signed.


Document peer name

Doc preview will be available at https://dev-weavewww.pantheonsite.io/docs/net/2233-peer-name-docs/ipam/ once the CI build is complete.
CC @abuehrle
Fixes #2233.

@matthias now with added linkage


es-modules

Hi, thanks for your help on migrating these. js-timers has already been updated in the official contrib repo https://github.com/purescript-contrib/purescript-js-timers. So I think we can close this one here.


Showing manf# as number in Excel

Hello,
First of all, thanks for this nice piece of software. It's very useful!
I'm using a csv bom file and version 1.1.15 of kicost.
I'd like to report two specific parts for which the tool is unable to complete part search:

BAT54H,115: diode from Nexperia. If I replace the ',' by '-' it finds it but it's not the correct manf# information. I guess a simple regexp in the manf# parse could solve this issue.
9551670000000: DB9 connector. Excel somehow represents the number in exponential (9.55E+12) and fails to search.

Thanks.
Sébastien

See section Parts With Subparts in https://xess.com/KiCost/docs/_build/singlehtml/index.html.
The , is used to indicate multiple manufacture part numbers. If it is present a single part number, it must be preceded by escape character: \,.
The other issue, please attache a BOM example and the commands execute to call KiCost.

github won't let me upload a csv file... :(
here is the xlsx:
github-issue423.xlsx
In the csv, here are the values used:

manf#
refs

9.55167E+12
J6

The DB9 connector (Mouser ref) is<PHONE_NUMBER>741 and I used this value within the csv file.
Excel shows it in scientific mode.

The command line is the following:
python3 -m kicost -i github-issue423.csv --exclude arrow lcsc tme --overwrite

Is this spreadsheet generated by a CSV or XML file?
I could not reproduce this on Linux-LIbreOffice.
Please, attach the file example (changing the extension for GitHub ignore it).

Here it is.
The file has been created with MS Excel 2016.
github-issue423.csv.xlsx

The problem here seems to be a wrong part number. The correct part number is "09551667812741", not "9551667812741". Using the correct value I get data from KitSpace API.
KiCost writes it as an string, I guess the "9.55167E+12" comes from some mistake when importing the CSV in Excel. I tried loading the KiCost XLSX in Excel from Office 365 and it shows the correct P/N.


Which base62 algorithm does this implement?

My understanding is that there is no formal spec for base62, but that the "glowfall" implementation (despite its lack of stars) has become the de facto implementation (used the most across the most repos).
Does this follow that spec? Or a different one? Or create a new one?

Have not heard of glowfall before. This library implements mathematical byte by byte base conversion of arbitrary data. There is really only one way to do it. Standards such as base85 which has extra rules for compressing spaces etc are not pure base conversions.
By default this library uses<PHONE_NUMBER>ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz as character set. It is the same as The GNU Multiple Precision Arithmetic Library uses. GMP has been around since early 1990's. I personally like this character set most since the encoded base62 strings preserve the sort order of the encoded values

I think I was mistaken. The GMP, GnuPG, and Saltpack implementations seem to be the most recognized.
Would you mind giving example output for your implementation compared to these references?
Raw   : "Hello, 世界"
Base64: SGVsbG8sIOS4lueVjA (18 chars)
Base62: 1wJfrzvdbuFbL65vcS (18 chars)

Raw   : "Hello World"
Base64: SGVsbG8gV29ybGQ (15 chars)
Base62: 73XpUgyMwkGr29M (15 chars)

Raw   : [ 0, 0, 0, 0, 255, 255, 255, 255 ]
Base64: AAAAAP____8 (11 chars)
Base62: 000004gfFC3 (11 chars)

Raw   : [ 255, 255, 255, 255, 0, 0, 0, 0 ]
Base64: _____wAAAAA (11 chars)
Base62: LygHZwPV2MC (11 chars)


New XML API endpoint: Char/ContactNotifications

Base on current master.

Thanks! Cherry-picked as 05e3e5e619886b9df6de2acc33280f3f1c809758.


Bosean FS-5000 Issues

I've been experiencing some small issues while using Radpro on my Bosean FS 5000:

Random glitch: The device starts constantly beeping with a red LED for about 3 seconds, then it returns to normal. Not a big issue.

Playing the chess game often randomly crashes and turns off the Geiger Counter.

I've also noticed that after the update, the J321 tube is now affected by UV light from the sun and reacts a lot to my UV 365nm torch. Before the update, I tried my UV light and it didn't react.

Thank you for your hard work on the software. I really enjoy using it!

Hello there,
thanks for the feedback! Could you post a short video illustrating the first and second issue? If the file doesn't fit in GitHub discussions, use a file sharing service.
Regarding the third issue: on the FS-5000, the high voltage is controlled by hardware, not by software. So this issue is most likely caused by some other factor (perhaps humidity, temperature or device wear).

It would be greatly helpful if you could capture the first bug.
Also, can you please post a second video showcasing the problem. Could you try pressing the buttons more slowly? I have a suspicion, this second video would greatly help in finding the culprit!

I flashed the firmware and found that the original firmware is now also reacting to UV light, which is strange. I'm 100% sure that it didn't do that before, so it must have something to do with humidity, temperature, or device wear, as you were saying. I tried to capture the game crashing by pressing the buttons slowly, but it wouldn't crash. So I guess I was just pressing the buttons too fast, which led to the crash. I still couldn't catch the first bug on video; it happened twice but went away before I could catch it on video.

Noahtheg, your report was very useful: I think the problem is related to the keyboard handler, not the game.
Once I find the time, I'll take a look at this.

Awesome, thanks for all your hard work on this firmware!

I have been unable to replicate the problem locally. If someone is able to further diagnose this, please post here.


Compilation error

fatal error: termios.h: no such file or directory

Install header files for libc in order to be able to compile. E.g. for Debian Jessie:
apt-get install libc6-dev

what about windows
?

windows is for sissies

You need cygwin.....not sure of the effort needed to replace such
dependencies, perhaps something in boost libs can make this lessened. Think
it's sort of lame that a project that wants to be a basis for larger text
editor projects can only be built on linux.... :/
On Wed, Aug 31, 2016, 9:44 AM Ali H. Fardan<EMAIL_ADDRESS>wrote:

windows is for sissies
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/antirez/kilo/issues/31#issuecomment-243768145, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AS1YNVg5hmrrrK_5XnRE7ClVCeUxlpXmks5qlYUogaJpZM4JONsn
.


⚠ SceneTime has degraded performance

In fa87582, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1304 ms

Resolved: SceneTime performance has improved in 2638afa after 7 minutes.


test: populate tables for system test using pytest fixtures #51

Looks like we have a lint error.

There still seems to be something wrong with this. It's deleting / not creating the dataset before the UDF tests
___________________________________ test_udf ___________________________________

client = <ibis_bigquery.client.BigQueryClient object at 0x7fc33545de20>
alltypes = ref_0
BigQueryTable[table]
  name: ibis-gbq.testing.functional_alltypes
  schema:
    index : int64
    Unnamed_0 : in...

        Args:
            timeout (int):
                How long (in seconds) to wait for the operation to complete.
                If None, wait indefinitely.

        Returns:
            google.protobuf.Message: The Operation's result.

        Raises:
            google.api_core.GoogleAPICallError: If the operation errors or if
                the timeout is reached before the operation completes.
        """
        kwargs = {} if retry is DEFAULT_RETRY else {"retry": retry}
        self._blocking_poll(timeout=timeout, **kwargs)

        if self._exception is not None:
            # pylint: disable=raising-bad-type
            # Pylint doesn't recognize that this is valid in this case.
>           raise self._exception
E           google.api_core.exceptions.NotFound: 404 Not found: Dataset ibis-gbq:testing was not found in location US
E
E           (job ID: 002827b4-1526-4839-b3d7-c56b964b5aa0)
E
E                          -----Query Job SQL Follows-----
E
E               |    .    |    .    |    .    |    .    |    .    |
E              1:CREATE TEMPORARY FUNCTION my_add_0(a FLOAT64, b FLOAT64)
E              2:RETURNS FLOAT64
E              3:LANGUAGE js AS """
E              4:'use strict';
E              5:function my_add(a, b) {
E              6:    return (a + b);
E              7:}
E              8:return my_add(a, b);
E              9:""";
E             10:
E             11:SELECT my_add_0(`double_col`, `double_col`) AS `tmp`
E             12:FROM (
E             13:  SELECT *
E             14:  FROM `ibis-gbq.testing.functional_alltypes`
E             15:  WHERE `bigint_col` IN (10, 20)
E             16:  LIMIT 10
E             17:) t0
E             18:LIMIT 10000
E               |    .    |    .    |    .    |    .    |    .    |


Dependencies between configurations

This addresses the following issues:

When a predicate's root is not a pre-existing root in Sitecore a warning shown on the Unicorn page. On sync the items are deserialized fine.
If a solution has multiple configurations and one configuration is relying on templates and root items from another configuration there needs to be a way to explicitly set up dependencies.

This branch adds the following:

Fixed the root paths to check the parent item and other configurations
Added the ability to have dependencies between configurations
Added explicit dependencies added in Configuration
Added implicit dependencies based on root path

I love this PR. This is an excellent feature. I've been hacking around not having dependencies by using config declaration order so far (I define my content and code configs in the same file so that's easy).
My only issues are stylistic - the comments above, and that it looks like the code is formatted with spaces and Unicorn uses tabs (this makes reading the diffs particularly hard). Fix that and I'll merge it right up :)

Good to hear.
The feature is a necessity for sitecore/habitat where all modules own their own items in Sitecore, but rely on other modules for templates or root folders.
Sorry for not writing tests - not my forte - but I'll definitely fix the formatting issues asap.

On 4 Nov 2015, at 3:42 PM, Kam Figy<EMAIL_ADDRESS>wrote:
I love this PR. This is an excellent feature. I've been hacking around not having dependencies by using config declaration order so far (I define my content and code configs in the same file so that's easy).
My only issues are stylistic - the comments above, and that it looks like the code is formatted with spaces and Unicorn uses tabs (this makes reading the diffs particularly hard). Fix that and I'll merge it right up :)
—
Reply to this email directly or view it on GitHub.

I think this latest commit should fix your comments. Please let me know if you have additional suggesting before merging.

Any news on this?

I've done a partial merge of this and opened #95 because there are still some todo items left before it gets fully merged. Going to close this and take the discussion to #95.


update timestamp of question on episode 304

Fixes a small typo on the timestamp: 45:17 instead of 47:17.

thanks!


Catch errors when instantiating components with initAll

Initialise all of the components using a loop in initAll, catching any errors thrown by the components.
Closes #4010

@colinrotherham I think I've addressed all your comments – can you take another look?

@36degrees Nice, thanks. Looking now


indexing (not working currently)

@adamnemecek ping. Are you interested to land this one day?

@adamnemecek ping. Are you interested to land this one day?

I’ll revisit this.

I’ll revisit this.


[Microsoft.Azure.ServiceBus] Test Failures: Management Tests

Summary
A set of management client tests has begun consistently failing in the nightly runs, following the changes made in #16488 or #16526.   The impacted tests are:

ManagementClientTests.QueueDescriptionParsedFromResponseEqualityCheckTest
ManagementClientTests.BasicQueueCrudTest
ManagementClientTests.AuthRulesEqualityCheckTest

Scope of Work

Update the identified tests such that they pass deterministically and consistently.

Mark the test as enabled rather than skipped.

Success Criteria

The tests identified in the summary have been updated to pass consistently across all platforms.

The tests identified in the summary have been enabled and is executed as part of the nightly runs.

The existing test suite continues to produce deterministic results and pass reliably.

References

Example run with failures

Closing this since the related PR has been merged.

@axisc: This shouldn't be closed.  The referenced PR was to disable these tests as a temporary work-around.  This issue is to track the need for one of the library owners root cause the failure and either fix the underlying issue with the test or to decide the test is not necessary and remove it.

@jsquire these should be tracked on the internal tracking board please.
Closing it here.


If match-phase limiting has concluded that a post filter is most effi… MERGEOK

…cient,
we must only generate posting lists if it is actually benefiscal. If not the fixed cost is too high.
@geirst PR

@havardpe PR
@geirst note last commit


snowplowctl: add ability to target existing persistent cluster

Instead of spinning up a new cluster.

Covered by #1173?

Hi all,
I am facing with this problem :
Before merging, remember that the contents of HDFS [1] [2] is not cleared out between "runs" in a persistent cluster, so you could easily end up reprocessing the same data...
[1] hdfs:///local/snowplow/enriched-events/
[2] hdfs:///local/snowplow/raw-events/
So the next running was  re-processing the old data. Did any one solve this issue ?
Thanks for read and help.

Hi @ngocthanhit - my suggestion would be to add some startup code into Scala Hadoop Enrich and Hadoop Shred to clear out those file paths. You should be able to use something like fullyDelete: https://hadoop.apache.org/docs/r1.2.1/api/

Thanks so much @alexanderdean. I need to implement those code now to use in my existing cluster.

Yep, please open a PR when you have it working! This code will give you a flavor of how to do one-time setup tasks in our Hadoop jobs:
https://github.com/snowplow/snowplow/blob/master/3-enrich/scala-hadoop-enrich/src/main/scala/com.snowplowanalytics.snowplow.enrich.hadoop/EtlJob.scala#L72

Yes sure i will :+1:

Actually - what I suggested won't work... What I suggested will delete precisely the files that you want to process - the ones that S3DistCp just copied into hdfs for this run.
Alternative suggestions:

Update the hdfs pathing in EmrEtlRunner to include the run ID, or
Add a scripting step after enrich and shred to delete everything from hdfs

In fact, with option 1 you would still need 2, because otherwise your HDFS cluster will run out of space. So I would jump straight to step 2.
Resources:

http://docs.aws.amazon.com/ElasticMapReduce/latest/DeveloperGuide/emr-hadoop-script.html
http://stackoverflow.com/a/16797599/255627

Can you send me the file which process for  after enrich and shred, so i will try to add the setup for delelete everything from hdfs? I am too new with scala so still investigating your code now :)
Thanks

You don't need to write any Scala. Instead, you:

Write a Bash script which uses the hadoop hdfs command line tools to empty the HDFS path
Update the EmrEtlRunner to add a new job step which invokes your Bash script using Amazon's  script-runner.jar step

All as per the links I've shared.

I will try to apply to my current cluster now.

@alexanderdean  I can run with an existing cluster using feature/existing-emr branch and add the customer jar step for empty the HDFS path. So will i create an PR for your to process ?
Thanh/

Sounds good to me! Thanks @ngocthanhit

Bringing forwards

done in #3945 , closing


Add StepProgressView

Project URL
https://github.com/yonat/StepProgressView
Description
Step-by-step progress view with labels and shapes. A good replacement for UIActivityIndicatorView and UIProgressView.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

:no_entry_sign:
Found 2 link issues

Link issues by awesome_bot

Line
Status
Link

2244
301
https://github.com/zwaldowski/BlocksKit redirects tohttps://github.com/BlocksKit/BlocksKit

2633
Error
http://swiftsandbox.io  getaddrinfo: Name or service not known

  Generated by :no_entry_sign: Danger

Hey @yonat, thanks for contributing again! 👍


Popover: Implemented flipping/overflow logic and minor fixes

Note
@henon @mckaragoz @just-the-benno i had go thru like ~20 files and do allot of changes to solve all merge conflicts, i have tested and compared, but if you could just double check your changes that's been made to autocomplete, select etc i would sleep better allot better tonight.
Description
Developed by  @just-the-benno and me. (Mostly Benno) i pointed he coded.
Added some functionality that we missed with the popover reworked PR #2833

Added flipping/overflow functionality (when popover is opened to close to window border it opens in another direction.
Removed the Pickers own flipping code and now uses the Popover code.
Changed default z-index to:

Drawer: 1100
Popover: 1200
Appbar: 1300

this is to prevent selects, pickers, autocompletes, popovers to overlapp the AppBar when scrolling. But popovers inside main content can still overlap Drawer.
Popovers inside Drawer AppBar Dialog will instead of using the popover z-index use the respective components z-index. This applies to all components that are using Popover internally as well.
Issues:
fixes #1342
None Issue fixes
fixes this (reported by @mikes-gh in discord channel, previously it would display under the drawer)

Added missing functionality from #2833
Popover flipping when to close window border:

Overflow is a property in Popover itself.

How Has This Been Tested?
New tests
Manual inspection, tested reported issues
Compared functionality and behavior to current stable
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Did you get autocomplete to scroll again too?

commited update to tooltip and update my previuse comment. only thing left now as far as i know is try get that delay going but i have big problems with the tooltip flickering.


fix official sample notebook for vector search

Summary

Fix the format of the source file for the vector index (allow_list → allow)
Fix folder name for Vector Search sample in CODEOWNERS (/matching_engine → /vector_search)

Checklists for official notebook modification

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[x] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[x] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

/gcbrun

/gcbrun

@gericdong
Sorry to bother you while you're busy!
Could you please run /gcbrun again?

/gcbrun

@gericdong
All the checks have passed.
I would greatly appreciate it if you could kindly review and approve it.


update "validDomains" after adding authentication to message extension

[Enter feedback here]
Could you remind the user to add the "token.botframework.com" to "validDomains" after adding authentication to the message extension?

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

ID: 503e124f-2b80-d6ae-581a-ca77f0a1999f
Version Independent ID: 66d2bd8a-536b-1a45-f71c-eb2a0199de46
Content: Add authentication to your message extension - Teams
Content Source: msteams-platform/messaging-extensions/how-to/add-authentication.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: anclear

Thanks @moira2k for your feedback.
@surbhigupta12 - It would be better if we add a note in this doc saying the user to add the "token.botframework.com" to "validDomains" section in manifest after adding authentication to message extension


Tooltip trigger text size is different to the label size of form controls, like Select, TextInput

Now I need to create a Select control with label and a tooltip icon near the label. Since Select component does not support, so I wanted to use Tooltip together with Select (hide its own lable). But I found the font-size of "bx--tooltip__trigger" is 1rem, but font-size of "bx--label" is 0.875rem. So when I put this field with other normal fields with label, its label looks bigger.
Can you change the font-size of "bx--tooltip__trigger" to 0.875rem? Or provide tooltip in form controls?

This seems like a common use case, and we should modify the bx--form-item to accept a tooltip as well as a label

@tw15egan agree

This is resolved in #314 .


Flag for sql logging in VoidRelationalResult - used for temp table

What type of PR is this?
Bug Fix

What does this PR do / why is it needed ?
Fixes logging in VoidRelationalResult in cases where server is configured to not log SQL because of sensitive values

Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
Logging fix, no tests, manual checks only
Does this PR introduce a user-facing change?
No

/easycla


layers.TimeDistributed does not change number of weights or network architecture

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras):
OS Platform and Distribution: Windows 10 21H1
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):  2.5.0
Python version: 3.9.6

Describe the problem.
import tensorflow as tf
from tensorflow.keras import layers
from tensorflow import keras

model1 = keras.Sequential()
model1.add(layers.LSTM(10, input_shape=(10, 1), return_sequences=True))
model1.add(layers.Dense(1))
model1.summary()

model2 = keras.Sequential()
model2.add(layers.LSTM(10, input_shape=(10, 1), return_sequences=True))
model2.add(layers.TimeDistributed(layers.Dense(1)))
model2.summary()

Running the above code will show that the output shapes of each layer and the total number of parameters in both networks is exactly the same. So layers.TimeDistributed is not doing anything. The docs say that TimeDistributed should result in the same weights for the dense layer to be applied to every timestep, but clearly this is not happening since the number of parameters in the network says the same regardless of whether TimeDistributed is used or not.

@hdavis472 Could you please refer to link  and let us know if it helps. Thanks!

I have read the stack overflow post you linked and tried running my own experiments based off the answer, but I still don't understand what using TimeDistributed dense does. I believe the original poster of the stack overflow thread was also still confused even after having read the answer. Would it be possible for someone to explain it clearly here please?

@hdavis472 please take alook at these explanations solution1 and solution2 and let me know if it helps you understand timedistributed vs dense layer. Thanks!


[Other issue] Confusion in coherence time calculation.

Hello!
I was trying to replicate the coherence times reported in PRX 11, 011010 (2021) with the following lines of code fluxonium = scq.Fluxonium(EJ = 9.0, EC = 2.5, EL = 0.52, flux = 0.5, cutoff = 110, truncated_dim = 4)
print(r'T1:',fluxonium.t1_effective(total=True, common_noise_options=dict(T=0.00019))*(1e-3),' us')
print(r'T2:',fluxonium.t2_effective(common_noise_options=dict(T=0.00019))*(1e-3), ' us')
This is what I get: T1: 4445.823719536628 us, T2: 2910.6219635808525 us. I immediately checked the documentation page for Calculations and Visualizations and it says that the times are given in units of the system units (which I have set to GHz). But going over here - scqubits.Fluxonium.t1_effective, it says that the decoherence times are given in units of angular frequency, which means that I'll need to multiply a factor of 2pi. This results in unrealistic coherence times (like 27ms or so).
Please help me in this regard.

Thanks for your question. I think maybe the doc strings are confusing; you do not have to multiply by factors of 2pi to get the coherence times in natural units. To be concrete, the thing that the t1 or t2/tphi methods of scqubits spit out is the inverse rate that would normally go directly into a master equation.
So to give a specific example: let's say we're looking at two lowest levels of a fluxonium circuits - that's our qubit, and we care about some t1 type decay process. The effective master equation of such a system would read (setting hbar=1):
$$ \dot ρ = -i [ 2 \pi (\frac{E_q}{2}) \sigma_z, ρ ]  + T_1^{-1} D[\sigma_{-}]ρ $$
where E_q would be the energy splitting between the two lowest levels, and 1/T_1 the corresponding downwards decay rate.
One could obtain E_q from sqcubits, and by default it would assume that all the energies, and thus the reported E_q, are given in GHz. Furthermore, the obtained T_1 (through t1_XXX type methods) would be be in nanoseconds.
(Note, to just get the downward-only contribution required for this example, the relevant t1 based method should get total=False passed in).
Hope things are clearer now. I will add it to our list to make the doc strings reflect this better.
Note also that in your example you have:
print(r'T1:',fluxonium.t1_effective(total=True, common_noise_options=dict(T=0.00019))*(1e-3),' us')
you would want to put the total=True into the common_noise_options dictionary, as t1_effective would not process it otherwise (here it doesn't actually matter as total=True is the default for all the t1_XXX methods that t1_effective calls, anyway).

Thank You @petergthatsme for your response. Things are a bit clearer now.

But a small question, why does the Fluxonium times seem to be high as per scqubits? Why am I unable to replicate the values that PRX 11, 011010 (2021) reports?

can you send the code you're using to try to replicate their numbers? one thing to point out would be that perhaps some of the deafult parameters that go into those calculations (e.g. quality factots, noise strenghts, etc) that scqubits uses, may not be identical to what the paper assumes. Hence one might have to exlicitily set the values taken from the paper.

Like, in calculating the relaxation from flux noise:

h = 6.62606957e-34
h_bar = h/2*np.pi
e = 1.6e-19 #C
flux_quantum = h/(2*e)
k_B = 1.3806488e-23

### RELAXATION FROM FLUX NOISE ###
R_q = h/e**2
R = 26
M = 625
temp = 0.042
beta = 1/(k_B*temp)
L = 1257.4e-9

def flux_noise_spectral_density(omega):
    spect = (np.pi**3)*(h_bar**2)*(R_q/R)*(M**2/L**2)*omega*(1/np.tanh(beta*h_bar*omega/2))
    return spect

fluxonium.t1(1, 0, fluxonium.phi_operator(), flux_noise_spectral_density, 0.042)

I get 6.112e+33 ns.
For 1/f flux noise:
def one_f_flux_noise_spectral_density(omega):
    #spect = 8*(np.pi**3)*(fluxonium.EL**2)*((5.21e-6)**2)/omega
    spect = 2*np.pi*(5.21e-6*flux_quantum)**2/omega
    return spect

fluxonium.t1(1, 0, fluxonium.phi_operator(), one_f_flux_noise_spectral_density, 0.042)

I am getting infinity. (paper reports 2.4ms)
For chargeline noise:

Q_c = 7.4e4 #R_q/(16*np.pi*R)

def chargeline_noise_spectral_density(omega):
    spect = (omega/Q_c)*(1/np.tanh(beta*h_bar*omega/2))
    return spect

fluxonium.t1(1, 0, fluxonium.n_operator(), chargeline_noise_spectral_density, 0.042)

I am getting 0.2894 ns (the paper reports 10us)
Please correct me. Where am I going wrong?

I've only had a very quick chance to glance at the paper, but here are a few comments:
t1 capacitive/inductive noise
to reproduce those from fig 3 in the paper, one could run something like:
fluxonium = scq.Fluxonium(EJ = 3.395, EC = 0.479, EL = 0.132, flux = 0.5, cutoff = 210, truncated_dim = 6)

fig, ax = fluxonium.plot_coherence_vs_paramvals(
                     param_name='flux',
                     param_vals=np.linspace(0.25, 0.5, 25),
                     noise_channels = [('t1_capacitive', dict(Q_cap=1/(4e-6)))],
                     common_noise_options=dict(T=0.042),
                     label = "t1_capacitive",
                     scale=1e-6,
                    color='red',
                    linestyle='--'
                    );

fluxonium.plot_coherence_vs_paramvals(
                     param_name='flux',
                     param_vals=np.linspace(0.25, 0.5, 25),
                     noise_channels = [('t1_inductive', dict(Q_ind=8e9))],
                     common_noise_options=dict(T=0.042),
                     label = "t1_inductive",
                     scale=1e-6,
                    color='green',
                    linestyle='--',
                    fig_ax = (fig, ax)
                    );
ax.set_title("")
ax.set_ylabel(r"$T_1$($ms$)")
ax.set_xlabel(r"$\Phi_x/\Phi_0$")
ax.set_ylim(1e-1, 2e1)
ax.legend()

which results in something like:

purcell t1
scqubits does not include the type of calculation that's done in the paper for a purcell t1 (i.e. from appendix G5); that could be done "manually", or alternatively one define an appropriate S(omega), which would be related to the admittance of the oscillator circuit that their fluxonium couples to, shown in fig 1b. At some stage we will provide an explicit example of how this can be done in the docs somewhere.
1/f flux dephasing
note that scqubits currently does not implement a tphi from 1/f noise obtained via an echo type experiment, so this is something you'd have to do manually (using results from the Ithier paper, for example) in order to reproduce fig 3b (also worth pointing out that there seems to be a typo in their x label - should probably have a factor of 1e-4 and not 1e4).
we do provde a 1/f calculation for 1/f Ramsey type experiment, and that could be for by something along the lines of:
fig, ax = fluxonium.plot_coherence_vs_paramvals(
                     param_name='flux',
                     param_vals=np.linspace(-0.05, 0.05, 51),
                     noise_channels = ['tphi_1_over_f_flux'],
                     label = "tphi_1_over_f_flux",
                     scale=1e-3,
                    ylabel=r"$\mu$",
                    xlabel=r"$\Phi_x/\Phi_0$",
                    color='red',
                    linestyle='--'
                    );
ax.set_ylabel(r"$T_{\phi}$($\mu s$)")
ax.set_ylim(1, 3e1)
ax.legend()

it's worth pointing out, however, that currently we only calculate the leading order correction, resulting in a zero rate at the optimal point (flux=0.5 here, befause the qubit-energy flux dervative is zero there)... so that's why the tphi value will go into infinity at this flux point. We hope to add a 2nd order correction in the next version of scqubits.
other; your manual calculations.
I haven't tried to run your code, but you seem to have a bug in how you define hbar: should read hbar = h/(2*pi) and not hbar = h/2*pi. So maybe try to fix that and look for other typos, and let me know if you still have issues.

Thank you so much for the detailed response. I'll close this issue.


Fix UploadField in v4

seems to be a general bug with handling sub-routes. TreeDropdownField is also effected.
though for UploadField there is a workaround: pressing attach file will open a dialog where a file can then be selected or uploaded

seems to be a general bug with handling sub-routes. TreeDropdownField is also effected.
though for UploadField there is a workaround: pressing attach file will open a dialog where a file can then be selected or uploaded


EventSource.from_config does not work correctly when passed parent

The EventSource.from_config method does not work correctly when passed a parent object.
It extracts the config attribute from the parent and creates an event source without parent, thus removing the hierarchy, which results in e.g. the logging not being correctly setup.
I think the for the from_config method, i.e. the inability to set the input url via the config was fixed in https://github.com/cta-observatory/ctapipe/pull/1503, so maybe this method is not needed anymore / can be simplified to correctly set the parent in the created event source.

Fixed in #1514


Upgraded dependency

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

Comparison is base (32d7590) 94.14% compared to head (82cfbd1) 94.14%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   94.14%   94.14%
=======================================
  Files          12       12
  Lines         785      785
=======================================
  Hits          739      739
  Misses         46       46

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Is it possible to get calendar cell size ?

Hi,
I try to add event image using delegate here
func calendar(calendar: FSCalendar, imageForDate date: NSDate) -> UIImage?  {         return UIImage(named: "image.png")  }
but it seem that my UIImage is too large for each cell and collapse with another cell.
My guess is that I have to resize the UIImage to match cell size. but I wonder how can I get the actual cell size from calendar ?
Thank you,

[calendar frameForDate:date];


When "git checkout" moves a file, the analysis server doesn't process the removal of the old file

I noticed this while trying to repro https://github.com/Dart-Code/Dart-Code/issues/2329 (I'm not sure if it's related to that issue yet). I ran all this on macOS - I'm not sure if it's OS-specific.
When changing Git branch, files that are moved are retained in the analysis server memory. This means you can have invalid code (eg. importing a file that's been since moved) but not get any errors for it until you restart the IDE/force reanalyze.
These steps will create a repo that has two files (a.dart and b.dart, where a.dart imports and calls a function from b.dart):
mkdir /tmp/my_repro_folder
cd /tmp/my_repro_folder
git init
echo "import 'b.dart'; main() {foo();}" > a.dart
echo "foo() {}" > b.dart
git add .
git commit -m "Added A and B"

This creates a second branch where b.dart has been moved to a folder (without the import being updated).
git checkout -b branch_two
mkdir b
mv b.dart b/b.dart
git add .
git commit -m "Moved B into B/"

Finally, switch back to the master branch:
git checkout master

Now open this project in VS Code (to avoid any confusion with priority files, no not open either of the Dart files) and run the Dart: Open Analyzer Diagnostics command. Note that the "Added files" section contains both files as expected.
Now change branch in a terminal:
git checkout branch_two

You'll notice two things:

The editor is not showing an error in a.dart for the (now-invalid) import.
Reloading the diagnostics page still shows b.dart under "Added files", but the file does not exist anymore

Here's the relevant part of the instrumentation file:
1586166259731:Req:{"id"::"4","method"::"diagnostic.getServerPort","clientRequestTime"::1586166259730}
1586166259742:Res:{"id"::"4","result"::{"port"::51182}}
1586166269724:Watch:/private/tmp/my_repro_folder:/private/tmp/my_repro_folder/.git/index:modify
1586166269724:Watch:/private/tmp/my_repro_folder:/private/tmp/my_repro_folder/.git/logs/HEAD:modify
1586166269724:Watch:/private/tmp/my_repro_folder:/private/tmp/my_repro_folder/.git/HEAD:modify
1586166269725:Watch:/private/tmp/my_repro_folder:/private/tmp/my_repro_folder/b/b.dart:add
1586166269730:Noti:{"event"::"server.status","params"::{"analysis"::{"isAnalyzing"::true}}}
1586166269743:Noti:{"event"::"analysis.errors","params"::{"file"::"/private/tmp/my_repro_folder/b/b.dart","errors"::[]}}
1586166269743:Noti:{"event"::"server.status","params"::{"analysis"::{"isAnalyzing"::false}}}

What's interesting is that there doesn't seem to be any event for the file being deleted (there's an add for b/b.dart - I don't know if that's a move/rename being interpreted wrong, or if an event is being missed here).
(@bwilkerson fyi - I don't have powers to label issues)

We received reports from flutter framework engineers in the past which map to something like this issue - a series of changing git branches will confuse the analysis server.
We may need to investigate changes in how we use file watching? Improvements to the file watching package?

This might be an issue in the watcher package. I created a script that just prints watch events and switches between my branches:
import 'dart:io';

import 'package:watcher/watcher.dart';

final repoFolder = '/tmp/my_repro_folder';

void main() async {
  await checkout('master');

  print('Setting up watcher...');
  DirectoryWatcher(repoFolder).events.listen((event) {
    if (event.path.contains('.git')) return;
    print('    ${event.type}: ${event.path}');
  });

  await Future.delayed(Duration(seconds: 5));
  await checkout('branch_two');
  await Future.delayed(Duration(seconds: 5));
  await checkout('master');
  await Future.delayed(Duration(seconds: 5));
  await checkout('branch_two');
  await Future.delayed(Duration(seconds: 5));
}

Future<void> checkout(String branch) async {
  print('');
  print('Checking out branch $branch...');
  await Process.run('git', ['checkout', branch], workingDirectory: repoFolder);
  await Future.delayed(Duration(seconds: 2));
  final files = await Directory(repoFolder)
      .list(recursive: true)
      .where((f) => !f.path.contains('.git'))
      .toList();
  print('  Current files:\n    ${files.join('\n    ')}');
}

This is the output when I run it:
Checking out branch master...
  Current files:
    File: '/tmp/my_repro_folder/two.txt'
    File: '/tmp/my_repro_folder/b.dart'        // Note this file exists and disappears without any event
    Directory: '/tmp/my_repro_folder/folder'
    File: '/tmp/my_repro_folder/a.dart'
Setting up watcher...

Checking out branch branch_two...
    add: /tmp/my_repro_folder/b/b.dart
  Current files:
    File: '/tmp/my_repro_folder/two.txt'
    Directory: '/tmp/my_repro_folder/folder'
    File: '/tmp/my_repro_folder/a.dart'
    Directory: '/tmp/my_repro_folder/b'
    File: '/tmp/my_repro_folder/b/b.dart'

Checking out branch master...
    remove: /tmp/my_repro_folder/b/b.dart
  Current files:
    File: '/tmp/my_repro_folder/two.txt'
    File: '/tmp/my_repro_folder/b.dart'                // And it's back, without any event
    Directory: '/tmp/my_repro_folder/folder'
    File: '/tmp/my_repro_folder/a.dart'

Checking out branch branch_two...
    add: /tmp/my_repro_folder/b/b.dart
  Current files:
    File: '/tmp/my_repro_folder/two.txt'
    Directory: '/tmp/my_repro_folder/folder'
    File: '/tmp/my_repro_folder/a.dart'
    Directory: '/tmp/my_repro_folder/b'
    File: '/tmp/my_repro_folder/b/b.dart'

@DanTup  - I cannot reproduce this locally.
Does this only happen on windows?

I found the bug causing a problem for this specific reproduction case - it happens because of a shared prefix between /tmp/my_repro_folder/b and tmp/my_rpro_folder/b.dart. I'll send out a PR shortly.


After update laravel to 6.18.35, i got error in model that has $guarded attribute

Laravel-mongodb Version: 3.6.4
PHP Version: 7.3.21
Database Driver & Version: mongodb 1.5.5

Description:
Recently, was merged https://github.com/laravel/framework/pull/33777. Since then, I'm getting an error when I try to use the create method with a model that has $guarded attribute
Steps to reproduce

Have a Model that has $guarded attribute:

class Tester extends \Moloquent
{
    protected $guarded = [
        'test',
    ];
}

call Tester::create(['foo' => 'bar']);

Expected behaviour
It should create a object with property 'foo' in database
Actual behaviour
PHP Error:  Call to a member function compileColumnListing() on null in /home/project/vendor/laravel/framework/src/Illuminate/Database/Schema/Builder.php on line 136

PRs welcome 👍
I might take a closer look but probably on Sunday.
Thanks!

Same issue here. i have to downgrade to  laravel/framework v6.18.32
with composer require laravel/framework:6.18.32
But be patient this release has the vulnerability wich is fixed with the v6.18.35 and actual not working with this package
laravel/framework (v6.18.32)
----------------------------
 * [CVE-NONE-0001][]: Guard bypass in Eloquent models
[CVE-NONE-0001]: https://blog.laravel.com/security-release-laravel-61834-7232

I had the same problem here.
I fixed the version.

The exact fix that causes this error, is mentioned [on the Laravel blog] by the way.

Today's patch fixes this and other potential unexpected behaviors by comparing the column that is being updated with an actual list of database columns that exist on the database table.

Since this involves a security fix that fixes some protected JSON assignments (like foo->bar whilst foo is guarded), it's good to cross-check this against code known to exploit stuff like this. Maybe mocking such a request in a unit test.
An easy fix is also suggested:

As a personal recommendation, I recommend always using $fillable instead of $guarded


Adding MM eval tests

Context
What is the purpose of this PR? Is it to

[ ] add a new feature
[x] fix a bug
[x] update tests and/or documentation
[ ] other (please add here)

Please link to any issues this PR addresses.
Changelog
What are the changes made in this PR?

Adds eleuther eval tests for dummy MM models. These tests are running on GPU since it's super slow on CPU.
Fixes a bug introduced by https://github.com/pytorch/torchtune/pull/1961 which missed expanding KV-cache tensors for cross-attention layers.

Test plan
Please make sure to do each of the following if applicable to your PR. If you're unsure about any one of these just ask and we will happily help. We also have a contributing page for some guidance on contributing.

[x] run pre-commit hooks and linters (make sure you've first installed via pre-commit install)
[ ] add unit tests for any new functionality
[ ] update docstrings for any new or updated methods or classes
[x] run unit tests via pytest tests
[x] run recipe tests via pytest tests -m integration_test
[ ] manually run any new or modified recipes with sufficient proof of correctness
[ ] include relevant commands and any other artifacts in this summary (pastes of loss curves, eval results, etc.)

UX
If your function changed a public API, please add a dummy example of what the user experience will look like when calling it.
Here is a docstring example
and a tutorial example

[ ] I did not change any public API
[ ] I have added an example to docs or docstrings

Codecov Report
Attention: Patch coverage is 37.68116% with 43 lines in your changes missing coverage. Please review.

Project coverage is 67.50%. Comparing base (e1caa9f) to head (b1c283d).
Report is 5 commits behind head on main.

Files with missing lines
Patch %
Lines

tests/recipes/test_eleuther_eval.py
37.25%
32 Missing :warning:

tests/recipes/utils.py
21.42%
11 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1989      +/-   ##
==========================================
- Coverage   67.61%   67.50%   -0.12%
==========================================
  Files         318      318
  Lines       17597    17674      +77
==========================================
+ Hits        11899    11930      +31
- Misses       5698     5744      +46

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Parasite axes

Added a class MatplotlibHostParasiteAxes for plotting multiple axes.

This pull request just contains the class. To run tests I have just been adding "MatplotlibAxes = MatplotlibHostParasiteAxes" to the end of _matplotlib_axes.py.

The next step would be to add a ParasiteAxes Model and move some of the "brains" here into that Model. I have a PR on the way (maybe today, maybe later this week) that will simplify the interaction between View-models and the Models, so it probably worth waiting until that lands before proceeding. I'll check in here when it has.

The next step would be to add a ParasiteAxes Model and move some of the "brains" here into that Model. I have a PR on the way (maybe today, maybe later this week) that will simplify the interaction between View-models and the Models, so it probably worth waiting until that lands before proceeding. I'll check in here when it has.

The work I referred to in my previous comment is now merged from https://github.com/bluesky/bluesky-widgets/pull/97. This will need to be rebased on master to absorb those changes, but fortunately it should require only a minor adjustment to what you have done so far.
Next steps:

Add a ParasiteAxes model in bluesky_widgets.model.plot_specs inheriting from Axes and extending it with the concept of  a host and sense of which axis it is sharing --- x, y, or both (if that's allowed?)
When the view receives a bluesky_widgets.model.plot_specs.Axes model it should make normal matplotlib.axes.Axes. When it receives a bluesky_widgets.plot_specs.ParsiteAxes model it should make matplotlib...[blah blah blah].ParasiteAxes.
Work with example data from LIX as a real test case for plotting a scan with one detector (y) over two motors (sharing x). I have some, just need to find it and pack it up for you.

The work I referred to in my previous comment is now merged from https://github.com/bluesky/bluesky-widgets/pull/97. This will need to be rebased on master to absorb those changes, but fortunately it should require only a minor adjustment to what you have done so far.
Next steps:

Add a ParasiteAxes model in bluesky_widgets.model.plot_specs inheriting from Axes and extending it with the concept of  a host and sense of which axis it is sharing --- x, y, or both (if that's allowed?)
When the view receives a bluesky_widgets.model.plot_specs.Axes model it should make normal matplotlib.axes.Axes. When it receives a bluesky_widgets.plot_specs.ParsiteAxes model it should make matplotlib...[blah blah blah].ParasiteAxes.
Work with example data from LIX as a real test case for plotting a scan with one detector (y) over two motors (sharing x). I have some, just need to find it and pack it up for you.

P.S. I am happy to do a call to check in on this if it would be helpful, though this week is :banana: :banana: :banana: so it would be best to plan for next week.

P.S. I am happy to do a call to check in on this if it would be helpful, though this week is :banana: :banana: :banana: so it would be best to plan for next week.

Yes I think it would be good to discuss over a call - next week works for me too. In the meantime I will look into the next steps you suggested.

Yes I think it would be good to discuss over a call - next week works for me too. In the meantime I will look into the next steps you suggested.

Path forward here is:

Rebase
Get the model side ParasiteAxes and HostAxes solid.
Rework the view side on that.

Good question. Answer:
from bluesky_widgets.models.plot_specs import Axes

axes = Axes()
figure = Figure([axes], title="...")
line1 = Lines(..., axes=axes)
line2 = Lines(..., axes=axes)

Does that address the question?

The way the colors work for non-parasite plotting is that there is a color for each scan. Should I keep this behavior for parasite plotting and make it so that each line within a scan corresponds to a different line type?

Oops, somehow I missed the notification of your comment 12 days ago, @bsobhani. Thanks for your patience with my slow reply.
I don’t understand the question yet. To take the LIX use case as an example, if I do a scan of one detector moving two motors, I’d have two x axes (one of them being a “parasite”) and one line. To take a different example, if I have one detector scanned against one motor that I was to display against two y axes—say, Celsius and Fahrenheit—I’d still have two lines. In what scenario do we get two lines representing the same scan?

No problem @danielballan.
I think what you describe is that is probably the most useful application of this.
The example I had in mind is if you want to compare two sensors that measure the same thing but in different units. At ESM for example they have measured beam intensity in two different ways: one way using gold foil and another way using their electron analyser. One is measured in amps but the other in counts but it may still be useful to view both measurements in one plot if for example you want to compare their performance.
It might be rare that someone would want to use this though so maybe it is better to keep it simple for now. Over the next few days I will test how what I have now works and also make a few other changes that were mentioned.

Oh, that’s a great use case to include. Thanks for explaining.
Then to answer your original question

The way the colors work for non-parasite plotting is that there is a color for each scan. Should I keep this behavior for parasite plotting and make it so that each line within a scan corresponds to a different line style?

I would say yes. I imagine they would need to have separate entries in the plot legend and be visually distinguishable.


Concatenate output layers of keras yolov3 trained model.

Hi,
Thanks for this implementation.
I have converted yolo weights to keras model.It seems that it had 3 outputs   of different shapes at
conv2d_59, conv2d_67, conv2d_75
I tried merge and concatenate method but they didn't work.
I just need a stack of these output layers into list but as a single output only.
conv2d_59          conv2d_67        conv2d_75
\                              |                            /
\                           |                          /
\                       |                        /
\                   |                      /
\             |                /
output = [ conv2d_59, conv2d_67, conv2d_75 ]
It can be done stacking layers into list but I is it possible to get withing the layer.
Thanks!!

model = Model(inputs=input,outputs=x)
x0 = model.output[0]
x1 = Reshape(target_shape = (1,), input_shape=model.output[1].shape)(model.output[1])
x2 = Reshape(target_shape = (1,), input_shape=model.output[2].shape)(model.output[2])

y = Concatenate(axis = 1)([x0, x1, x2])

tensor_info_input  = tf.saved_model.utils.build_tensor_info(model.input)
tensor_info_output = tf.saved_model.utils.build_tensor_info(y)
signature = (
    tf.saved_model.signature_def_utils.build_signature_def(
        inputs={"input":   tensor_info_input},
        outputs={"output": tensor_info_output},
        method_name=signature_constants.PREDICT_METHOD_NAME)
)


can't autocomplete new import packages

package main
import (
       "fmt"
)
func main(){
        fmt.Println("hello world!")
}

here is the start code.
when i import a new package like "math/rand"
the rand package can't be auto completed
package main
import (
        "fmt"
        "math/rand"
)
func main(){
        fmt.Println("hello bingo!")
        rand.
        (it will bot be auto completed here.)
}

when i close the file and reopen it.it works again.
Editor:neovim 0.3.4
lsp:w0rp/ale

I am also having this issue.
After importing, if you type a symbol from the package manually then save the file, autocompletes will work going forward.

@chenjie199234 @malexdev
The root cause of this problem is: bad import code

@chenjie199234
Please use the enhanced version of gopls, it has fixed this issue.

@saibing: just to confirm, in order to use your updates to gopls we need to install your version, not the official one?
If so: Do you and the Go team have plans to integrate your fixes to gopls, or is this probably going to stay the way it is?


Usage Swift Documentation ?

You said Documentation is coming soon but when ?

@X901, I've rewritten this project to swift a couple of days ago, you can check on it here: https://github.com/drougojrom/CSStickyFlowLayoutHeaders

@drougojrom Thanks!. I have a question about your Project.
I set section count = 1 and section for Items = 300.  While Scrolling up, the sticky header is disappeared. I found visibleHeaders array is to be an empty array. Could you have the plan to solve this issues?
visibleHeaders = visibleHeaders.flatMap { (header) in
self.updateHeaderAttributes(header,
lastCellAttributes: lastCells[header.indexPath.section])
}