"Query Canceled by User" when querying encrypted fields

Issue Type: Bug
When querying encrypted fields, query cancels itself, and "Query Canceled by User" message is returned.

Using Azure Key Vault
SSMS is working
Azure account is connected
"Always Encrypted" is Enabled
Secure Enclave is not being used

Steps to Reproduce:

Clean install of Azure Data Studio 1.22.0 (Also attempted with clean install of 1.19.0)
Enable "Always Encrypted"
Execute query with any encrypted field selected

Expected Results
Results are returned for selected encrypted fields
Actual Results
Query is canceled by system before results are returned. Following system message is returned:

Started executing query at Line 1
(X rows affected)
Query was canceled by user
Total execution time: X

No errors are returned in any logs
Azure Data Studio version: azuredatastudio 1.22
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz (8 x 1400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
1, 2, 2

Memory (System)
16.00GB (3.39GB free)

Process Argv
-psn_0_208947

Screen Reader
no

VM
0%

Extensions: none

Same issue here.

Using Azure Key Vault

SSMS is working

Azure account is connected

"Always Encrypted" is Enabled

Secure Enclave is not being used

macOS 11.2.2

ADS 1.26.1

I iterated through the archived versions, and the last time this work was ADS v1.20.1

@Xtrimmer Do you think this might be caused by the SqlClient issue that was recently fixed? We had to roll back the driver update due to another regression so the current driver being used doesn't contain that fix.

@Charles-Gagnon it is quite possible, especially if it is within a non-windows environment. We knew that without the Microsoft.Data.SqlClient driver version 2.1.0 or greater, there will be issues with the Always Encrypted feature on MacOS and Linux platforms.

I'm experiencing the same thing and I'm on Windows

Merged into release/1.27.


Can't suppress the log output

I have these options set:
silent: true,
logger: {
   debug: () => {},
   info: () => {},
   warn: () => {},
   error: () => {},
},

But when I send an email (like to gmail), I get the whole sending session logged to my console. Is there any option I'm missing to suppress that?

Nevermind, sorry. Had the config specified in the wrong spot.

Hey Kyle,
No worries - let me know if you have any other questions
Jason


For a story, merge content from multiple pages

For stories_pages.csv, each story consists of multiple pages (There are multiple rows of page_content for one story_id). The content of a story spread across multiple pages need to be merged against one story id. Finally each row (instance) in the csv file should represent the content of one story.
This issue is dependent on the following issue regarding html clean up:
https://github.com/DataKind-BLR/PrathamBooks-Sprint-2018/issues/1
End result should be:

Modified stories_pages.csv (Do NOT commit data in github. You may mention data file name in .gitignore so that it does not get committed to github)
Script which is used to generate the data (MUST be committed to github)

@siddjain24 Since you co-own #1 please check this one as well.

I think I have the script ready for this. Will submit a PR soon.

This was address as a part of #1 and #10. Hence closing.


Seprate ignore config for backend

Right now, I need to explicitly wrap all processes that do not run through the plug to get error handling.
I'd like to just use the backend that you wrote to catch everything, but then I'm getting double reporting with the plug. It would be nice to have another ignore key like :ignore_backend that could have a different function than ignore (e.g. ignore anything from MyApp.Endpoint module...).

Maybe to be a little more clear about the issue. The apps that I've written have a lot of processes running and it becomes difficult to monitor every process to ensure that a failure is reported. It also requires that every dev ensures that they are monitoring any process created.
My thought here is that I really want to be able to ensure that all failures are logged to airbrake without having to rely on devs putting in a monitor. The formatting for the plug is currently nicer than that of the backend logger, so I'd like to keep using the plug but also avoid getting two messages every time the endpoint process dies.
I imagine there may be other reasons why some one might want a different filter on the notifier than the backend logger so I thought it'd be nice to make it configurable.


Deploy TEAMNADO-4182 to prod-beta

Codecov Report
Base: 47.72% // Head: 48.28% // Increases project coverage by +0.56% :tada:

Coverage data is based on head (bf46212) compared to base (0bb6260).
Patch coverage: 51.89% of modified lines in pull request are covered.

:exclamation: Current head bf46212 differs from pull request most recent head 43830c5. Consider uploading reports for the commit 43830c5 to get more accurate results

Additional details and impacted files
@@              Coverage Diff              @@
##           prod-beta      #85      +/-   ##
=============================================
+ Coverage      47.72%   48.28%   +0.56%
=============================================
  Files             56       64       +8
  Lines            681      758      +77
  Branches         129      161      +32
=============================================
+ Hits             325      366      +41
- Misses           340      372      +32
- Partials          16       20       +4

Impacted Files
Coverage Δ

...eHostConfigPopover/AboutRemoteHostConfigPopover.js
0.00% <ø> (ø)

...vationKeysDocsPopover/ActivationKeysDocsPopover.js
100.00% <ø> (ø)

...alRepositoriesTable/AdditionalRepositoriesTable.js
0.00% <0.00%> (ø)

...ionalRepositoriesTable/NoAdditionalRepositories.js
0.00% <0.00%> (ø)

src/Components/ConnectLog/SystemsTable.js
0.00% <ø> (ø)

src/Components/Services/permissionsConfig.js
0.00% <ø> (ø)

src/Components/shared/breadcrumbs.js
0.00% <0.00%> (ø)

src/Routes.js
0.00% <0.00%> (ø)

src/Routes/Dashboard/index.js
0.00% <ø> (ø)

src/hooks/useFeatureFlag.js
0.00% <0.00%> (ø)

... and 5 more

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

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


[SPARK-23702][SS] Forbid watermarks on both sides of a streaming aggregate.

What changes were proposed in this pull request?
Forbid watermarks on both sides of a streaming aggregate. As detailed in the jira, we don't currently support this and doing so would require significant revision to the execution model.
How was this patch tested?
new unit test

Can one of the admins verify this patch?

Test build #88282 has started for PR 20840 at commit 9d742ab.

Test build #88282 has finished for PR 20840 at commit 9d742ab.

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


Did Subscriptions support Publish Composite?

Did React-Native-Meteor Subscription able to get data from collection which is published by Meteor Publish Composite package? (publish collections with it's relation) so it can get the relational collection?
btw here is the link of publish composite package https://atmospherejs.com/reywood/publish-composite

I have tested it, and I can't get the related collections published by publish composite. I think this is very important in production. Is it possible?

I do not know yet, but I know I had issues using https://github.com/CollectionFS/Meteor-CollectionFS
I will try your plugin when I'll be less busy. (maybe next monday :) )
Please note that I am thinking more and more about a V1.0 release equivalent to react-meteor plugin. (with startMeteorSubscriptions and getMeteorData mixins).

Please make sure to make it as similar as real meteor (docs.meteor.com) DDP usage. If it possible, then your plugin will be the best plugin out there, and I think that's possible when looking at your current plugin.
Currently for production I am using this plugin https://www.npmjs.com/package/ddp-client, and It works pretty well. But unfortunately their code is not really taste like meteor, for some case it's very difficult to be used.
Thanks

Hi @Mokto, when will you take a look at this? :dart:

wow 1.0 coming, I'll try now! and test if this already available!! Good job

:)
My next step is to check your PublishComposite plugin.
Soooon.

This is already worked on latest version 1.0.0-beta2. Thanks :)


The following assertion was thrown building BlogTile(dirty): 'package:flutter/src/painting/_network_image_io.dart': Failed assertion: line 23 pos 14: 'url != null': is not true.

`class BlogTile extends StatelessWidget {
  final String imageUrl;
  final String title;
  final String description;
  final String url;

  BlogTile(
      {@required this.imageUrl,
      @required this.title,
      @required this.description,
      @required this.url});
  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      onTap: () {
        Navigator.push(
            context,
            MaterialPageRoute(
                builder: (context) => Article(
                      blogUrl: url,
                    )));
      },
      child: Container(
        child: Column(
          children: <Widget>[
            ClipRRect(
              borderRadius: BorderRadius.circular(6),
              child: Image.network(imageUrl),
            ),
            SizedBox(
              height: 10,
            ),
            Text(
              title,
              style: TextStyle(
                color: Colors.black87,
                fontSize: 17,
                fontWeight: FontWeight.bold,
              ),
            ),
            SizedBox(
              height: 10,
            ),
            Text(
              description != null ? description:'',
              style: TextStyle(color: Colors.grey),
            ),
            SizedBox(
              height: 10,
            ),
          ],
        ),
      ),
    );
  }
}`

The relevant error-causing widget was
    BlogTile
lib\views\home.dart:98
When the exception was thrown, this was the stack
#2      new NetworkImage
package:flutter/…/painting/_network_image_io.dart:23
#3      new Image.network
package:flutter/…/widgets/image.dart:399
#4      BlogTile.build
package:NewsLe/views/blogs_tile.dart:31
#5      StatelessElement.build
package:flutter/…/widgets/framework.dart:4576
#6      ComponentElement.performRebuild
package:flutter/…/widgets/framework.dart:4502

...
════════════════════════════════════════════════════════════════════════════════

@Agrim1210
Please provide flutter doctor -v and a complete minimal reproducible code sample that shows the issue.
Thanks.

Hi @Agrim1210
Image.network(imageUrl), This imageUrl variable is null somehow.
please use null safety like imageUrl != null ? showImage : some other widget .
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Activate user account manually front

Migration task to add the user setting with account activation by default set to false
New tab user in Technical settings menu with checkbox to enable/disable auto activation of user accounts
email verification set user to inactive if user's don't activate by default setting is set to true / active otherwise

N.B: cannot test locally validation by clicking the verification email without adding hardcoded data - need to be tested

@cabch Not linked to an issue


add configFile option for nugget pack

add missing -ConfigFile option for nugget pack

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tools/cli-ref-pack.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fixes #15 - Adds ESLint

Fixes #15 - Adds ESLint along with relevant .eslintrc file (copied from the django repo so we have the same config)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
32

Relevant Lines:
42

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
32

Relevant Lines:
42

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
32

Relevant Lines:
42

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
32

Relevant Lines:
42

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
32

Relevant Lines:
42

💛  - Coveralls


⚠ Jenkins Plugins Artifactory has degraded performance

In 1450297, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1676 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 3c073c4 after 37 minutes.


use the . command to source setup-envtest.sh

Signed-off-by: Zbynek Roubalik<EMAIL_ADDRESS>
Fixing make test target which was failing after merging https://github.com/kedacore/keda/pull/1960. There was used source command which is not supported on a shell that's has been used in Ubuntu (which we run the test against). Using the . command to source setup-envtest.sh will fix the problem.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Superseded by https://github.com/kedacore/keda/pull/2003


Object Manager is behaving really weird

Sometimes it doesn't add everyone or anyone to the friendly object list and it requires me to rezone/relog to sometimes fix it. In generally it feels very clunky

Hello,
Please provide the unlocker details (FireHack; Genric).
Thanks, MTS.

I'm using firehack

Got some lua-errors
...AddOns\NerdPack\system\ObjectManager\ObjectCache.lua:273: attempt to call global 'UnitIsEnemie' (a nil value)

I've confirmed and worked around that error.
it seems like UnitIsEnime API is unstable...
Sometimes it works, others, not so much.
Let me know if that fixed the issues you had.

I'll give it a spin in tonights raid and let you know how it works out, thanks!

Seems to have fixed the issue but I haven't tried it in more then LFG, I'll try it in LFR and some heroic/mythic tonight and see how it goes, I'll get back to you, thanks!

Glad it got fixed, closing this error.
Re-open it acts up again :)


Fix three mistakes

Fixes #3370 .
Changes proposed in this pull request:

Fix jdbc getIndexInfo method and NullPointerException.
Fix catalog and schema method parameter error.
Fix resultSet.getString(IS_NULLABLE) and
resultSet.getString(IS_AUTOINCREMENT) return type error.

Pull Request Test Coverage Report for Build 851

4 of 42 (9.52%)  changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 68.522%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-jdbc/sharding-jdbc-core/src/main/java/org/apache/shardingsphere/shardingjdbc/jdbc/metadata/JDBCTableMetaDataConnectionManager.java
2
3
66.67%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/schema/ProxyTableMetaDataConnectionManager.java
2
3
66.67%

sharding-core/sharding-core-common/src/main/java/org/apache/shardingsphere/core/exception/UnrecognizeDatasourceInfoException.java
0
2
0.0%

sharding-core/sharding-core-common/src/main/java/org/apache/shardingsphere/core/metadata/datasource/DataSourceInfo.java
0
4
0.0%

sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/core/execute/metadata/TableMetaDataLoader.java
0
13
0.0%

sharding-core/sharding-core-common/src/main/java/org/apache/shardingsphere/core/util/DataSourceUtil.java
0
17
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/core/execute/metadata/TableMetaDataLoader.java
1
0.0%

Totals

Change from base Build 547:
-0.1%

Covered Lines:
11511

Relevant Lines:
16799

💛  - Coveralls

Hi, @beijing-penguin Appreciated your contrition. The point need to disscuss i think is that can we use jdbc interface to get url and username? Therefore we will not care about which connection pool user chooses. When get connection for jdbc url, username can be got as well. Do you think it is available？

reset pr


arcgis vector tile - doubt/help

Hi ,I am using arcgis vector map in my ol vector tile. there i have set max zoom to 15 in tilegrid. but some of the .pbf request at certain areas fails at zoom level 14 and map gets blurred. actually i am trying to display the building . when i set 14 as maxzoom that particular area building is displayed , but buildings are not displayed in other areas which requires zoom 15.
the below is the one i am using https://www.arcgis.com/sharing/rest/content/items/291da5eab3a0412593b66d384379f89f/resources/styles/root.json
same i have posted in olms  https://github.com/boundlessgeo/ol-mapbox-style/issues/47
waiting for valuable suggestions

The problem with the tileset linked above is that not all tiles are available for all zoom levels.
Vector image tiles will only be rendered for a view resolution from their associated source tiles. If source tiles are not available, than the vector image tile will also be marked as error tile, and so the pre-rendered images from a lower resolution will be used, instead of using lower resolution source tiles for a higher resolution vector image tile.
To fix this, VectorImageTile would need to be smarter to also consider source tiles from lower resolutions if the requested resolution is not available.

Closing due to lack of activity.


Commands Do Absoultey Nothing

After changing the map of my server to washington and adding permisions and changing so zombies dont spawn on the map. When i type commands on the server i get zero feedback. Nothing. It dosent say command not found. It says nothing.  Someone please help

RocketMod is probably not installed.

Missing issue template, create new issue and fill it out


Week 6 Hacks

Collegeboard Notes

Hello Soham,
Good job on having detailed notes for the 1.3. You wrote all the key details of the development process.

Hello Soham,
Good job on your college board notes for the week. You provided many explamples of why program development is important. You Deserve at least a 2.7 for this week's issue.


Translated reference-react.md and edited TRANSLATION.md

Translated the API Reference - React.
Edited TRANSLATION.md

Hey @mahdinba97
Good job! I will review it by tomorrow.

Can I translate the title of this section in the sidebar?

Can I translate the title of this section in the sidebar?

Sure, we also have to do it.

Thank for your review @zahrajoulaei. I hope I fixed them correctly.

Thank for your review @zahrajoulaei. I hope I've fixed the issues correctly.

Thank you Mahdi and sorry for the delay since I was on vacation. I am more active from now on 🎉


question about dataset filtering

Hi,
  Thank you and your team for open-sourcing this impressive work.I want to express my appreciation for your team's great work and for releasing it as open source. I'm really interested in understanding more about your data filtering strategy.
  1. I noticed that you only trained the model on monomeric CIF files with max_len < 512. Would it be beneficial to include single chains from multimeric CIF files as well? This approach would significantly increase the dataset size by 5x to 100k proteins, even though their structure may not be true in the monomer state.
  2. Have you considered expanding the dataset using representative clustered proteins in the high confidence AFDB? If not, what is your opinion on this approach?
Thank you for your help and I look forward to hearing back from you soon.

Hi,
Thanks for the message.

Yes we only trained on monomers to simplify the application of our method. A follow-up work would seek to train on multimeric chains and do more interest tasks (binder design, protein-protein interaction). Our paper was primarily concerned with the SE(3) diffusion methodology and theoretical groundings which was very non-trivial! We have found more data to lead to better results so I am optimistic more data in these regimes would help.
The thought of using AFDB high confidence proteins did cross my mind but was not high priority for the reasons I stated above; it was outside the scope of this work. I think it is a very interesting extension as we've seen that AF2 distillation in RFdiffusion help it tremendously. Here it will be interesting to see how a pure MSA-free generative model can learn from the same source of structures.

Happy to answer any more questions!

I view RFdiffusion as an extension of FrameDiff where pre-trainedRosettaFold is trained with the the SE(3) diffusion derived in FrameDiff. (I was involved in incorporating SE(3) diffusion into RFdiffusion.) There are some minor differences like the rotation loss. I think additional investigation into the benefits of pre-training is an important one (as we stated in the paper). What I meant by AF2 distillation is that RosettaFold uses a form of distillation from AFDB so I think it plays a large part in why RFdiffusion works well.
I am personally not working on this. I think it is a great application.

Btw I'm extremely busy with end of semester matters. My responses will likely be delayed.

I beleve a lightweight model like FrameDiff is better for methods research. It's easier to train and analyze rather than having to go through the RosettaFold pipeline which is not publicly trainable. Obviously larger more complex models will perform the best but uncovering the scaling principles will be done first through lightweight models.


chore(main): release 1.8.7

:robot: I have created a release beep boop
1.8.7 (2022-12-16)
Miscellaneous Chores

deps: bump brianium/paratest from 6.6.5 to 6.7.0 (9b5f70f)
deps: bump brianium/paratest from 6.6.5 to 6.7.0 (#238) (dff23fc)
deps: bump fakerphp/faker from 1.20.0 to 1.21.0 (a1252bd)
deps: bump fakerphp/faker from 1.20.0 to 1.21.0 (#240) (19e78a1)
deps: bump laravel-vite-plugin from 0.7.1 to 0.7.2 (9849b8f)
deps: bump laravel-vite-plugin from 0.7.1 to 0.7.2 (#243) (d6a95e4)
deps: bump laravel/dusk from 7.1.1 to 7.2.1 (c46139f)
deps: bump laravel/dusk from 7.1.1 to 7.2.1 (#247) (28f9a90)
deps: bump laravel/fortify from 1.14.0 to 1.14.1 (1b568da)
deps: bump laravel/fortify from 1.14.0 to 1.14.1 (#245) (e086da8)
deps: bump laravel/framework from 9.43.0 to 9.44.0 (063ad2a)
deps: bump laravel/framework from 9.43.0 to 9.44.0 (#246) (76f74f7)
deps: bump laravel/sail from 1.16.3 to 1.16.4 (fce75e1)
deps: bump laravel/sail from 1.16.3 to 1.16.4 (#241) (1fec1de)
deps: bump postcss from 8.4.19 to 8.4.20 (9201e2c)
deps: bump postcss from 8.4.19 to 8.4.20 (#239) (3062993)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/rotex1800/registration/releases/tag/v1.8.7 :sunflower:


[AutoRelease] t2-containerservicefleet-2023-10-20-78901(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4663
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=982
BuildTargetingString
azure-mgmt-containerservicefleet
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/4663


Spelling/Gramma mistake

Hi.
I was reading over the document https://github.com/MicrosoftDocs/msteams-docs/blob/live/msteams-platform/resources/schema/manifest-schema.md
Under the section "staticTabs" it looks as if someone accidentally deleted parts of a word:
"without the user adding theM manually."

accidentally capitalized the M - should just be them. Nothing's missing. Thanks!


Force array from Get-SqlInstance

This should work as seen in the screenshot below

Gonna try on v4

Works in 4 as well /cc @michalporeba

whaaaat! sweet

I agree, that is much cleaner solution.


As the Designer, I want to establish the license agreement so that Users of this software know their obligations.

[x] Use BSD.

I am adding this comment to verify the integration between GitHub and Slack is still enabled.   The last notification I received through the integration channel was 2016.12.30 @ 04:47 PST.

BSD-3-Clause enabled and integration with Slack re-enabled.


rescursiveUpdateUntil tries to merge non-set types

Describe the bug
recursiveUpdateUntil seems to naively merge values which are not sets and therefore fails.
Steps To Reproduce
nix-repl> :lf nixpkgs
nix-repl> lib.recursiveUpdateUntil (path: l: r: false) {foo = 1;} {foo = 2;}
error: value is an integer while a set was expected

Expected behavior
nix-repl> :lf nixpkgs
nix-repl> lib.recursiveUpdateUntil (path: l: r: false) {foo = 1;} {foo = 2;}
{foo = 1;}

Additional context
recursiveUpdate (without Until), does not have that problem and stops the recursion if something is not a set.
I propse that either:

the documentation of recusiveUpdateUntil should be updated to clearly state that the user needs to take care of stopping the recursion in any case
or recursiveUpdateUntil is updated to stop recursion when encountering anything else than an attrset, so it will behave similar to recursiveUpdate.

Notify maintainers

@nbp @infinisil
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.110, NixOS, 21.11 (Porcupine), 21.11.20220413.a62ce97`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.0pre20220318_963b8aa`
 - channels(root): `"nixos-22.05pre353535.48d63e924a2"`
 - nixpkgs: `/nix/store/ps3i3zj8c9swr3a39kybnsv7ys0pw1p3-source`

Note that since recently there's also the new lib.updateManyAttrsByPath which imo is often a better alternative to many of these other set updating functions. For your motivating use case that might be a better option

Thanks for the hint. Will definitely take that into consideration next time.
About the current issue, what would you suggest? Is there a problem that should be fixed or not?

Thinking more about this, yes I believe this is a bug that should be fixed, specifically by changing recursiveUpdateUntil to throw a better error message than the one it currently throws.


Appstudio update hacbs-test-project-pici

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-pici-on-pull-request-
creating pipelinerun hacbs-test-project-pici-on-pull-request- in namespace rhtap-demo-xwyu-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Bug: TypeError crashes Homebridge

Describe The Bug
TypeError crashes Homebridge
To Reproduce
TypeError crashes Homebridge
Expected behavior
Does not crash Homebridge
Relevant log output
[7/4/2023, 11:53:55 AM] TypeError: Cannot read properties of undefined (reading 'temperature')
    at LeakSensor.parseStatus (/usr/local/lib/node_modules/homebridge-resideo/src/devices/leaksensors.ts:178:67)
    at new LeakSensor (/usr/local/lib/node_modules/homebridge-resideo/src/devices/leaksensors.ts:80:10)
    at ResideoPlatform.Leak (/usr/local/lib/node_modules/homebridge-resideo/src/platform.ts:579:9)
    at ResideoPlatform.deviceClass (/usr/local/lib/node_modules/homebridge-resideo/src/platform.ts:430:14)
    at ResideoPlatform.discoverDevices (/usr/local/lib/node_modules/homebridge-resideo/src/platform.ts:401:24)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
[7/4/2023, 11:53:55 AM] Got SIGTERM, shutting down Homebridge...

Config for homebridge-resideo
N/A

Screenshots
No response
Device & Model
N/A
Node.js Version
Current
NPM Version
Current
Homebridge Version
Current
Homebridge Resideo Plugin Version
Current
Homebridge Config UI X Plugin Version
Current
Operating System
OSX

@tbaur Can you try disabling temperature using the config to bypass till I have time to fix this.

v1.4.0 has been released which should resolve this.


Remove unused variables from glsl shader code + pipeline parameters

Can we update pgcraft to remove unused glsl variables in GLSL shader code, and to remove unused variables from NGLI pipeline_desc_params?
Example: in unit test blending_all_diamond:
In the generated GLSL shader code (see below) ngl_uvcoord and ngl_normal are unused.
#version 450
#define ngl_out_pos gl_Position
#define ngl_vertex_index gl_VertexIndex
#define ngl_instance_index gl_InstanceIndex
layout(location=0) in  vec3 ngl_position;
layout(location=1) in  vec2 ngl_uvcoord;
layout(location=2) in  vec3 ngl_normal;
layout(std140,set = 0, binding = 0) uniform readonly ngl_vert_block {
mat4 ngl_modelview_matrix;
mat4 ngl_projection_matrix;
mat3 ngl_normal_matrix;
} ;
void main()
{
ngl_out_pos = ngl_projection_matrix * ngl_modelview_matrix * vec4(ngl_position, 1.0);
}

How would you detect that they are unused?
Also, why?

How do you decide which variables to include?  Isn't it from the node graph?  Then you shpul should know which variables are used?
Why: several reasons:

remove unnecessary additional complexity
simplify shader conversion
reduce likelihood of errors
performance benefit: don't send unused data to gpu

How do you decide which variables to include? Isn't it from the node graph? Then you should know which variables are used?

How do you know if the user fragment code uses them or not?

There's two options:

user specify input variables in the shader (architectural changes)
we parse the fragment code. Maybe we can leverage existing open source code like spirv cross for this option

There's two options:
1. user specify input variables in the shader (architectural changes)

This is really cumbersome for the user

2. we parse the user shader code. Maybe we can leverage existing open source code like spirv cross for this option

We can't: we don't have enough information to make a valid GPU preprocessor.

For Option 2: we can parse the generated glsl shader code (in pgcraft) and strip the unused variables, and remove them from the pipeline data structures too, etc.


SelectableText.rich breaks selection when scrolling during selection

Steps to Reproduce
(1) Paste the following into main.dart
(2) run for Linux or Web
(3) start a selection and scroll (e.g. with trackpad) while keeping the selection active (i.e. left mouse stays pressed)
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark(),
      home: Scaffold(
        body: SingleChildScrollView(
          child: Column(
            crossAxisAlignment: CrossAxisAlignment.stretch,
            children: [
              SelectableText.rich(
                TextSpan(
                  children: List.generate(
                    100,
                    (index) => TextSpan(
                      text: "[$index] This is a line\n",
                    ),
                  ),
                ),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

Expected results:
The start of the selection stays the same and the end expands to the new cursor position after scrolling.
Actual results:
One of two things can happen
(A)
The list scrolls back to the end of the previous visible section.
The selection reaches only to that point.
(B)
The start of the selection jumps and the selection reaches to the new cursor location:

doctor:
$ flutter doctor -v
[✓] Flutter (Channel master, 2.2.0-11.0.pre.274, on Linux, locale en_US.UTF-8)
    • Flutter version 2.2.0-11.0.pre.274 at /home/omni/repos/flutter_web/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d1f73a1065 (14 hours ago), 2021-04-24 00:44:02 -0400
    • Engine revision eba9efd1c3
    • Dart version 2.14.0 (build 2.14.0-37.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0-rc2)
    • Android SDK at /home/omni/Android/Sdk
    • Platform android-30, build-tools 30.0.0-rc2
    • ANDROID_HOME = /home/omni/Android/Sdk
    • ANDROID_SDK_ROOT = /home/omni/Android/Sdk
    • Java binary at: /opt/AppDev/SDK/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /snap/bin/chromium

[✓] Linux toolchain - develop for Linux desktop
    • clang version 6.0.0-1ubuntu2 (tags/RELEASE_600/final)
    • cmake version 3.10.2
    • ninja version 1.8.2
    • pkg-config version 0.29.1

[✓] Android Studio (version 3.5)
    • Android Studio at /opt/AppDev/SDK/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Linux
    • Chrome (web)    • chrome • web-javascript • Chromium 90.0.4430.85 snap

• No issues found!

Same Issue on mobile web

flutter doctor -v
[✓] Flutter (Channel master, 2.2.0-11.0.pre.274, on macOS 11.2.3 20D91 darwin-x64, locale en-AO)
    • Flutter version 2.2.0-11.0.pre.274 at /Users/pedromassango/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d1f73a1065 (2 days ago), 2021-04-24 00:44:02 -0400
    • Engine revision eba9efd1c3
    • Dart version 2.14.0 (build 2.14.0-37.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/pedromassango/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 55.1.5
    • Dart plugin version 211.7179

[✓] VS Code (version 1.55.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 90.0.4430.85

! Doctor found issues in 1 category.


Performance improvements

Hi Andreas,
Here's the final set of performance improvements to the event_handler. When I have time, I plan to look into
rule triggering, as that looks like the current bottleneck now.
Here are some numbers.
With 10,000 IceCream entities:
Create, within a transaction
Before: 47 secs  After: 30 secs
Destroy_all, within a transaction
Before: > 5 minutes After: 7 seconds
Cheers,
Vivek

Awesome !
I've been thinking another improvement - Identity Map
Maybe we can store the instances in a hash per local thread context and transaction.
I guess we could do this in the wrapper method which loads the Ruby wrapper around the Neo4j::Node object

The RDoc of the method like node_deleted is not correct. Do you have time to fix it ?

Sure,
I'll correct it and push in a little while
Vivek

I've updated the rdoc, could you please take a look?

Just in case you didn't see it, I'm working on the identity map : https://github.com/andreasronge/neo4j/tree/identity_map
Get even better performance running the RSpecs.

Great news! Do let me know if I can help in any way. Like I'd said earlier, the next bottleneck seems to be rule triggering. I'm thinking about whether it would be useful to broadcast event information like this from the event handler:
on_class_added(model_class,added_entities) #I'll think of a better name :-)
on_class_removed(model_class,removed_entities)
on_class_updated(model_class,updated_entities)
This would enable things like the _all rule node to do one single update for class count, for example.
BTW, I've been using the built in JRuby profiler for the performance tuning, it's really good.

Sounds like a good idea, and we can remove the other listener methods.
Not sure what impact on the rule feature code it will have. will  it simplify that code base ?

I'm not sure about the impact on the rule feature code, the approach I was describing might need to invert the way rules are triggered - I think it might be easier to do if rules nodes subscribe to specific kinds of events instead of the listener directly triggering rules.
For some kinds of rules, it might still be necessary to get notified about node addition/updates/deletion at a fine grained level, while for others (like the count function of the all node), it might be more useful to listen to coarser grained events. Another way would be to have specialized handling for 'system' rules like the _all rule. I'll try and look into this over the weekend

I'm currently using the event framework a lot, see the new neo4j add for export/import and replay/rewind of events: https://github.com/andreasronge/neo4j-replay
Btw, I'm little bit worried about the current rule :_all implementation which creates a relationship between each single instance of a class to a rule node. Not sure about the performance of loading the rule node if you have millions of instances (used to be a problem in neo4j, maybe that is solved).
Also, not sure how useful the rule feature is. Maybe it should be a separate gem ?
Maybe its possible to use lucene instead to find all the instances of a class (and subclasses).
Just some thoughts ...

You can review the identity map on master if you want
https://github.com/andreasronge/neo4j/commit/84650f9272750a16f810fd0229dd3c2d8951b17b
Thinking of doing a release in the next few days

Thanks Andreas, I'll take a look, it looks very interesting. I haven't had too much time to look into further performance improvements, unfortunately.
BTW, I remember reading on the Neo4j mailing lists that support for 'supernodes' (ie, nodes with lots of outgoing / incoming relationships) such as the rule node in Neo4jrb is still poor.


Setting up

Hi!
I am interested in using your method in a set of cortico-subcortical recordings. I was trying to run the commands in Windows 10 and it ran until Est_Data.py that it ran into an issue with the subprocess package. I figured out that this issue is due to incompatibility with Windows10 and tried running it in a Ubuntu virtual machine which seems to work. However, I ran into another issue and it is more into the amount of given data. I saw in the paper that you define the minimum criteria for estimating the connectivity matrix in Table 1 but, unfortunately, our recordings have sparse firing and are maximum 1 hour long. Additionally, I would like to split the experiment into the different treatments that I'm applying, which makes our registered activity even sparser.
Do you think that I could still use this method for my sparse data?
Do you have a suggestion for running the algorithms in Windows? (I've looked into Cygwin, which is a Linux environment for Windows, but it creates many incompatibilities with my conda environments)
Thank you very much in advance!
Emilio Isaías-Camacho

Thanks for your inquiry.

Do you think that I could still use this method for my sparse data?

In general, you can use this method for sparse data.
The sparser the data is, the more difficult detecting the connectivity is. Note that this issue is independent of the method.
You can also try our web application just to try!
https://s-shinomoto.com/CONNECT/

Do you have a suggestion for running the algorithms in Windows?

Can you tell me more detail?
For example, how you run the code using conda, the version of the conda and windows, and the error massage.

Thank you for your quick reply!
I will give it a shot in the web interface, thanks!
I am currently using:
Windows 10 version 20H2,
conda 4.9.2,
python 3.9.2,
scipy 1.6.1
numpy 1.20.1
matplotlib 3.3.4
The command that I run is
python Est_data.py simulation_data 20 sim
And the error that I get is

Best regards,
Emilio Isaías-Camacho

Thanks for your reply.
Please try to change the last lines in the python code "Est_Data.py".
===== before ==================
cmd = ['rm', "J_py_"+str(T)+".txt"]
proc.check_call(cmd)
===== after ==================
import os
os.remove("J_py_"+str(T)+".txt")
I hope it works.
Best regards,
Ryota Kobayashi

Thanks a lot for looking into this!
I'll do the change and test.
Meanwhile, I've gotten my hands on a Linux system to try out the code and it runs on the simulated data seamlessly! However, as I input my sparse data, the code exits with the following error:

I reviewed the firing rate of the given cells and the firing rate of the considered one is very low ~0.15 Hz. I've been thinking that maybe a set of bins with 0 counts in the cross correlograms might arise some issues with the logarithm operation. I don't know, what do you think?
Best regards
Emilio Isaías-Camacho

Thanks for your report.
I suggest you to exclude the neurons that fires too sparse (< 0.5 Hz).
I also agree with you that each bin in the cross-correlogram should be more than a spike.

Very well, thanks for your time!
I'll paste here the cross-correlogram of those cells which caused the math domain error in the logarithm sometime later.


tree-sitter: 0.17.3 -> 0.18.2

Motivation for this change
Update tree-sitter an grammars
Also include commit from #108639 , without many of the grammars are broken
Closes #110662
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 114405 at 66299575 run on x86_64-linux 1

1 package built:

tree-sitter

1 suggestion:

warning: unused-argument
Unused argument: fetchurl.
Near pkgs/development/tools/parsing/tree-sitter/default.nix:2:30:
  |
2 | , fetchgit, fetchFromGitHub, fetchurl
  |                              ^

thanks. apparently there is already a 0.19 available https://github.com/neovim/neovim/pull/14045

@teto right, I saw it today, I wanted to update this PR, but I'll create a new one instead


Null safe check if isBackgroundPlaybackEnabled property is set

Description
isBackgroundPlaybackEnabled was checked in an unsafe way which caused attaching a player to a view to crash if  isBackgroundPlaybackEnabled was not set.
Changes

make isBackgroundPlaybackEnabled check null safe

Checklist

[x] 🗒 CHANGELOG entry

@123mpozzi is it fine that the property is not set in this scenario or do we have to have a closer look why it is unset?


Profile recasts sorted in reverse

Recasted episodes in external profile should be sorted with most recent first!

Resolved with backend fix


upgrade: Upgrade Python version form 3.9 to 3.10.

Add version changes to of both python and pipenv to README.md.
Addbandit module as its also needed but not in Pipfile.

Hi Swangeon, could you please remove the tool-generated files as we discussed? Also, could you perform the soft syntax updates? To clean up the code. Then I can review all the changes together.
Best

Will do! This PR will probably be up for some time longer but I will get those done.

Everything to my knowledge has been changed to conform more with PEP8. There are a couple of things that I didn't decide on changing to conform to PEP8 like long strings or long paths as splitting those up to get to the 79 line length made the code harder to read in my opinion. Type hints should be there for most if not all variables and created some type aliases to refer to certain data structures that were nested and/or had a lot of types that could be returned. Added the runtime generated output files to the .gitignore (Will do the reports directory later as I am doing the revamp of the reporting system). Fixed minor things like the output of the port scanner as os became osmatch and hostnames became hostname and how the version check for Conpot is done.


"mockito-android" + "dexopener" - java.lang.NoClassDefFoundError: org/mockito/internal/configuration/plugins/Plugins

Reposted from: https://github.com/mockito/mockito/issues/1082#issuecomment-315628196
java.lang.NoClassDefFoundError: org/mockito/internal/configuration/plugins/Plugins
        at org.mockito.internal.configuration.GlobalConfiguration.tryGetPluginAnnotationEngine(GlobalConfiguration.java:55)

Since Mockito-android is so large, I had to use LinkedIn's dexmaker.
Version of dexmaker - 0.9.2
I was using my example project here: https://github.com/jaredsburrows/android-gif-example

I guess this is solved now, so I will close this.
If you are facing this error with the latest DexOpener, please reopen.

Thanks.


Insert failover when domain moves from active to passive

What changed?
Insert failover when domain moves from active to passive

Why?
Insert failover marker into replication queue and let the new active cluster consumes the markers.

How did you test it?
TODO

Potential risks
N/A

Coverage decreased (-0.1%) to 68.337% when pulling 7b7ba4db641d43e6ceb0b3d6726f94a078aa9a8c on insert_marker into a1a14830e4b16c26f8dadbfcba063632e691efd7 on master.


Mobile support

I opened a PR for this functionality here: #31
Also, here's my fork: https://github.com/juandjara/quill-blot-formatter-mobile

this change would be really useful.
The creator of this change also pushed a for to https://www.npmjs.com/package/quill-blot-formatter-mobile which works fine for me.


License missing from repo

Without a license, this project isn't open source and no one can use the code.
Please use something like http://choosealicense.com/ to decide what license to use. I recommend MIT or GPL.

A novel effort by Add a License Please

Karan,
I have been working on adding proper documentation to all of my repositories.
Thanks for the suggestion, I have added the MIT license.

Great. Thanks @trentrand


custatevec-cu11 fixes cuquantum version?

Before submitting
Please complete the following checklist when submitting a PR:

[x] All new features must include a unit test.
If you've fixed a bug or added code that should be tested, add a test to the
tests directory!

[x] All new functions and code must be clearly commented and documented.
If you do make documentation changes, make sure that the docs build and
render correctly by running make docs.

[x] Ensure that the test suite passes, by running make test.

[ ] Add a new entry to the .github/CHANGELOG.md file, summarizing the
change, and including a link back to the PR.

[x] Ensure that code is properly formatted by running make format.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Context:
L-GPU compiled libraries are still jumping to favour CUDA 12 over CUDA 11. We may need to favour building using custatevec-cu11 rather than the generic cuquantum package, which seems to also pull in 12.
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

It seems something is up in this logic:
https://github.com/PennyLaneAI/pennylane-lightning-gpu/blob/c963a8a0515ae06db9e33a1e88cbab7d435b9f10/CMakeLists.txt#L119-L154
@vincentmr Do you know how is ${Python_SITELIB} determined in your CMakeLists.txt? I suppose it's looking at the wrong site-packages hence not finding the cuStateVec wheel installation.

@vincentmr Do you know how is ${Python_SITELIB} determined in your CMakeLists.txt? I suppose it's looking at the wrong site-packages hence not finding the cuStateVec wheel installation.

I'm unsure, but one can usually just set the CUQUANTUM_SDK env var. I'll try that.

@vincentmr Do you know how is ${Python_SITELIB} determined in your CMakeLists.txt? I suppose it's looking at the wrong site-packages hence not finding the cuStateVec wheel installation.

I'm unsure, but one can usually just set the CUQUANTUM_SDK env var. I'll try that.

I can answer here --- the issue is likely now imposed by the newly added CI env clear functionality, which is required to avoid the GPU runner doesn't become too bloated across PRs. If more than 1 GPU runner is using the image, it will purge the env of the other (this is a limit we have to live it, unfortunately).
I think we can just rerun it and assuming it works fine, I'll fix this later and move the venv-clear code to be per PR, rather than global on the GPU runner we have set up.

In this case, the site-packages should be pulled correctly, but often are not due to the existence of the virtualenv or not.

Right, independent venvs per PR sounds about right to me. I don't expect any code change in your codebase is needed, since the pennylane-lightning-gpu package builds fine on conda-forge 🙂
If this works, then it'd be nice for your wheels to just depend on custatevec-cuXX, as your pip-install experience would become nicer (one less big package to install).

Right, independent venvs per PR sounds about right to me. I don't expect any code change in your codebase is needed, since the pennylane-lightning-gpu package builds fine on conda-forge slightly_smiling_face
If this works, then it'd be nice for your wheels to just depend on custatevec-cuXX, as your pip-install experience would become nicer (one less big package to install).

Yep, agreed. Definitely on the roadmap.


add the second example from #523 to the tests

In order to test the accuracy for the default method
The actual reference values might need to be adjusted.

closing this pr into a pr which is already merged now


General clean up

Removes unused functions
Removes more deps on FortranIntrinsics
Adds type restrictions
Cleans logic/use dispatch on combine_and_reorder, and removes allocations
Makes argument order of rte_lw and rte_sw more similar, and moves result to first argument
Removes old Fortran file that has been translated

bors r+

bors r+


ImageAugmentor and GeneratorEnqueuer do not share DatasetGenerator Attributes

Right now all operations on a DatasetGenerator are complicated with ImageAugmentor and GeneratorEnqueuers (and possibly more generators to come) since they do not share the same attributes of a DatasetGenerator. These classes need to be modified to allow their use in any place a normal DatasetGenerator might be used.
This might be achieved by subclassing the DatasetGenerator or creating a super class. This can also be achieved by creating matching attributes for GeneratorEnqueuer and ImageAugmentor defined by the attributes of DatasetGenerator.
Ideally, changing DatasetGenerator should not require changing any solution code for this issue.


Broken links to example keymap

Change example to examples in the file path.
Also changed ZMK_CONDITIONAL_LAYER in example to new syntax.

Lot's of good catches! Thanks for making the changes!


activate travis-ci for this repo

It would be nice to have travis-ci integration for this repo to only be able to merge pull requests when tests are passing.
@abingham Could you setup emacs-ycmd on travis-ci.org and then activate the travis service in the repo settings?

Good idea. I'll try to get to it soon.

Cool thanks!

@ptrv When you set up your travis build, did you have any problems with the dash package? It looks like my .travis.yml is the same as your (which builds successfully), but mine fails because it can't require dash: https://travis-ci.org/abingham/emacs-ycmd

Looks odd. I'll have a look.

It seems like some old elc file is sticking around or something.

In the travis.yml there is caching for the cask directory enabled. In my travis the builds were run successful because it didn't update dash for a while. I created a test branch and now it fails there too

Ah, right. I've started a build with a "make clean" at the start. If that doesn't do the trick, I'll disable the caching.

The builds are still failing. I'll have a deeper look when I have more time

Travis works fine now. Finally


tf.strings.to_number cannot convert positive integers prefixed with "+" when out_type is tf.int32 or tf.int64

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
No
Source
source
TensorFlow version
tf 2.11.0, 2.13.0, 2.14.0
Custom code
Yes
OS platform and distribution
MacOS 13.1
Mobile device
Macbook Pro
Python version
3.10.6
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Converting string to numbers with "+"  throws errors in TF 2.11.0, 2.13.0, 2.14.0 when out_type=tf.int64 or out_type=tf.in32. Not expecting error to be thrown and strings can be correctly converted into integers. For example, I would like to parse timezone information from a string (using substring)
t = tf.constant([
    "2023-05-07 17:32:25-08:00", # utc: next day
    "2023-05-07 05:32:25+11:00", # utc: previous day
    "2023-05-07 05:32:25-08:00", # utc: same date
    "2023-02-29 23:32:15-04:00", # leap year
    ]
)

Standalone code to reproduce the issue
Code to reproduce

import tensorflow

# these are all okay
tf.string.to_number(tf.constant("-11"), out_type=tf.int64)
tf.string.to_number(tf.constant("11"), out_type=tf.int64)
tf.string.to_number(tf.constant("+11"), out_type=tf.float32)

# this throws the error below
tf.strings.to_number(tf.constant("+11"), out_type=tf.int64)

Relevant log output
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/edward/opt/miniforge3/envs/testtf214/lib/python3.10/site-packages/tensorflow/python/util/traceback_utils.py", line 153, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/Users/edward/opt/miniforge3/envs/testtf214/lib/python3.10/site-packages/tensorflow/python/framework/ops.py", line 5888, in raise_from_not_ok_status
    raise core._status_to_exception(e) from None  # pylint: disable=protected-access
tensorflow.python.framework.errors_impl.InvalidArgumentError: {{function_node __wrapped__StringToNumber_device_/job:localhost/replica:0/task:0/device:CPU:0}} StringToNumberOp could not correctly convert string: +11 [Op:StringToNumber] name:

Hi @sapphire008 ,
I have replicated the reported behaviour in tf-nightly and attached gist here for reference.
We will look into it and update you. Thanks!

I have fixed the issue on local environment, but I can't open a PR on tsl repo since "An owner of this repository has limited the ability to open a pull request to users that are collaborators on this repository".


Create/Enter Idea Component (from Text Area)

Create a component to enter a Idea from the original text area (with buttons and style)


New error in compilation

Hello boys,
Yesterday I updated Nim (1.6.14), Piko-Sdk (1.5.1) and Picostdlib (3.3).
But now, when i compiling a progets (new an old), I get the following mistake:
/home/andrea/ProgPicoNim/pwmLed/src/random.nim(10, 18) Warning: imported and not used: 'gpio' [UnusedImport]
Hint: gc: arc; opt: speed; options: -d:release
56422 lines; 0.591s; 60.984MiB peakmem; proj: /home/andrea/ProgPicoNim/pwmLed/src/pwmLed.nim; out: /home/andrea/ProgPicoNim/pwmLed/csource/build/nimcache/pwmLed.json [SuccessX]
make: Entering directory '/home/andrea/ProgPicoNim/pwmLed/csource/build'
PICO_SDK_PATH is /home/andrea/ProgPicoNim/pico-sdk
PICO platform is rp2040.
PICO target board is pico.
Using board configuration from /home/andrea/ProgPicoNim/pico-sdk/src/boards/include/boards/pico.h
TinyUSB available at /home/andrea/ProgPicoNim/pico-sdk/lib/tinyusb/src/portable/raspberrypi/rp2040; enabling build support for USB.
CMake Error at imports.cmake:6 (target_include_directories):
  Cannot specify include directories for target "pwmLed.nim" which is not
  built by this project.
Call Stack (most recent call first):
  CMakeLists.txt:28 (include)

-- Configuring incomplete, errors occurred!
make: *** [Makefile:2467: cmake_check_build_system] Error 1
make: Leaving directory '/home/andrea/ProgPicoNim/pwmLed/csource/build'

Does anyone know what I have to do to be able to go back to work ??? Thank you

Serious mistake!
I compiled like Piconim Build XXX.NIM <- so no but I was getting used to!
instead that like Piconimbuild XXX <- and so it works !!


Can implement document or other else for udemy ?

Like tittle, this's awesome if yes, thank you so much or using number of playlist like udemy, ex here number of lecture 4 is 4 but your is 3, can fix that ! Thank you so much !
http://i.imgur.com/iL0ror9.png

Carefully read new issue template and provide all requested information.


MIC-146 retrieve csip record by uuid

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

Project coverage is 83.04%. Comparing base (7dca885) to head (e154772).

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #69      +/-   ##
============================================
+ Coverage     82.90%   83.04%   +0.14%
- Complexity      434      436       +2
============================================
  Files           107      108       +1
  Lines          1626     1628       +2
  Branches         48       49       +1
============================================
+ Hits           1348     1352       +4
+ Misses          258      257       -1
+ Partials         20       19       -1

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


Update copy and request translations

Task/Issue URL: https://app.asana.com/0/1204920898013511/1208577246155728/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5260  👈
#5237
#5171 : 4 other dependent PRs (#5172 , #5187 , #5213  and 1 other)
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CrisBarreiro and the rest of your teammates on  Graphite

Done!

@CrisBarreiro this file is wrong.
It's missing the smartling instructions at the top:
<!-- smartling.entity_escaping = false -->
<!-- smartling.instruction_attributes = instruction -->

And the file name is the other way around, should be strings-broken-site.xml, this is why the automation is not picking it up. Note when you change the file name, it will run through the automation.

@marcosholgado that's fixed now and the automation is working properly. Could you please review?


Issue while running a sample video

Hi I was trying to run videopose.py using python 3.7 in virtual env using ubuntu 18.04. The command i used was as follows
python3.7 videopose.py --video ./outputs/kunkun_cut.mp4  which results in the following  error
Traceback (most recent call last):
  File "videopose.py", line 174, in <module>
    inference_video('outputs/dance.mp4', 'alpha_pose')
  File "videopose.py", line 163, in inference_video
    main(args)
  File "videopose.py", line 67, in main
    split_video(args.viz_video)
  File "/sandbox/video-to-pose3D/common/utils.py", line 154, in split_video
    saved_path = os.path.dirname(save_path)
UnboundLocalError: local variable 'save_path' referenced before assignment

I am using hrnet as 2D detector .

First of all, you don't need to specify --video in your command line. python3.7 videopose.py is enough because the args.viz_video will be overloaded by the first parameter of inference_video.
The Error is caused by the
i = 1
while True:
    grabbed, frame = stream.read()

    if not grabbed:
        print(f'Split totally {i + 1} images from video.')
        break

    save_path = f'{save_folder}/output{str(i).zfill(length)}.png'
    cv2.imwrite(save_path, frame)
    i += 1

in which stream.read() don't read any frame. The while loop break directly.
You can run the ./test/opencv_capture_test.py to check whether the cv2.VideoCapture(path) works properly or not. Don't forget to change the path to your video path, and check if there are any problems of the working directory of python interpreter.
If it runs properly, the args.viz_video must be changed because of the args.viz_video = video_path in inference_video. Try to directly change the first parameter of inference_video in
if __name__ == '__main__':
    inference_video('outputs/dance.mp4', 'alpha_pose')

to your absolute video path.
By the way, I'm developing this project using PyCharm. It's recommended that using the same IDE to run the script instead of the command line.

Hi thanks for your clarification,
i did as you had suggested and tested the script ./test/opencv_capture_test.py  and it worked perfectly.  However i think there are some error in importing the hrnet model . Because i changed the following in videopose.py
    inference_video('/home/sandbox/video-to-pose3D/outputs/kunkun_cut.mp4', 'hr_pose')
It gave the following error
Traceback (most recent call last):
  File "videopose.py", line 174, in <module>
    inference_video('/home/pramah/sandbox/video-to-pose3D/outputs/kunkun_cut.mp4', 'hr_pose')
  File "videopose.py", line 163, in inference_video
    main(args)
  File "videopose.py", line 62, in main
    detector_2d = get_detector_2d(args.detector_2d)
  File "videopose.py", line 50, in get_detector_2d
    return detector_map[detector_name]()
  File "videopose.py", line 39, in get_hr_pose
    from joints_detectors.hrnet.pose_estimation.video import generate_kpts as hr_pose
  File "/home/sandbox/video-to-pose3D/joints_detectors/hrnet/pose_estimation/video.py", line 15, in <module>
    from lib.config import cfg
ModuleNotFoundError: No module named 'lib'

I seemed to have followed your instructions to place pretrained weights and yolo module . I  think there is just a dependency some where

As is said in my README

If you're using PyCharm, mark ./joints_detectors/Alphapose, ./joints_detectors/hrnet and ./pose_trackers as source root.
If your're trying to run in command line, add these paths mentioned above to the sys.path at the head of ./videopose.py

The bug is caused by the fact that these paths are not in your sys.path.
I've updated my code to add them into sys.path manually. You can pull the project and try again.

I will try again  . Sorry that i have missed it .

Hello again , I seem to move forward but seem to have some issues with CUDA  . I have an inbuilt GPU and i have setup the requirements by creating a virtual env . I double checked if torch deviceis detected and apparently it is .
print torch.cuda.is_available()
true

And the error i get is in the python package as follows .Do you have a certain version of torch ? I have the following packages installed in my virtual enviroment. I am using ùbuntu18.04`
cycler (0.10.0)
ffmpeg (1.4)
kiwisolver (1.1.0)
matplotlib (3.1.1)
nibabel (2.5.0)
numpy (1.17.0)
opencv-python (<IP_ADDRESS>)
Pillow (6.1.0)
pip (9.0.1)
pkg-resources (0.0.0)
pyparsing (2.4.2)
python-dateutil (2.8.0)
PyYAML (5.1.2)
scipy (1.3.1)
setuptools (39.0.1)
six (1.12.0)
torch (1.2.0)
torchsample (0.1.3, /home/pose3d/src/torchsample)
torchvision (0.4.0)
tqdm (4.34.0)
yacs (0.1.6)

created the virtual env with python3.7

May caused by the index of cuda devices:
os.environ["CUDA_DEVICE_ORDER"] = "PCI_BUS_ID"  # see issue #152
os.environ["CUDA_VISIBLE_DEVICES"] = "1"

I'm using multi-GPU machine in my environment. Try to change the os.environ["CUDA_VISIBLE_DEVICES"] into "0" in ./videopose.py

Thanks ! it works very good . Actually smoother than you had experienced and better. I suspect you used a wrong pretrained model that is why you got some fluctuating results. I would like to close this issue and perhaps reach you out for more details . Great work !


the question about accuracy in scene flow datasets

Hello，we test your model on scene flow dataset，but the EPE is only 0.97. Is there something wrong with my use of the code or is the EPE is just 0.97. Thank you very much.

I am not sure and I will check the record later. Did you get the correct cross-domain generalization accuracy for the pre-training model?

I am not sure and I will check the record later. Did you get the correct cross-domain generalization accuracy for the pre-training model?
ok，I will check the cross-domain generalization accuracy soon.

I am not sure and I will check the record later. Did you get the correct cross-domain generalization accuracy for the pre-training model?
OK, I just checked that there is no problem in cross-domain accuracy, so it seems that the accuracy of scene flow is indeed 0.97


Remove natron.rb

The download url requires a login now, which means install fails. Unable to find a download link that does not require login so removing

After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Never mind. Retested and it's working now.


Users/madhurig/m83javafixes

This needs to be against releases/m83, closing it and will create new one


Realm: prometheus support

This is kind of a crazy pie in the sky idea I had, but I think it might be useful if we instrumented the runtime with prometheus (https://prometheus.io/docs/introduction/overview/) and built a grafana dashboard ( https://grafana.com/grafana/) to tell us if individual nodes/ranks are making progress in the runtime during a run and to give us live feedback. This is one of the most useful tools we have debugging LCLS problems.
Here is an example LCLS dashboard:
https://confluence.slac.stanford.edu/download/thumbnails/295077738/image2021-5-7_14-27-46.png?version=1&modificationDate=1620422866000&api=v2
We might even be able to use this for automating debugging, where you have another process that watches prometheus and if certain ranks are not making progress in the runtime then maybe it decides to attach and dump stack traces or something. We could also potentially expose this interface to user applications as well so they could export their own metrics, although that is currently possible right now if someone just decided to implement it in their application directly, and maybe that might be a good place to start to see how something like this would actually work in practice.
If we do this in the runtime it should definitely be something that a user has to opt in to at compile time and then again at run time. This also may not actually be feasible at large scales due to the volume of data.

@syamajala that certainly sounds like a interesting for discussion. Could you please provide more details on how are you using it what exactly does it help to solve? Possibly the best would if could just throw one or two examples (use cases) here

Based on this blogpost the instrumentation libraries are only available GO, Java and Ruby:

https://developers.soundcloud.com/blog/prometheus-monitoring-at-soundcloud

In the context of SLAC/LCLS its being used as a sort of poorman's profiler right now:

In those screenshots you see we've defined metrics for things like how long is it taking a node to process data, what rate are we processing data at, when are we idle and waiting for data, when is data missing/corrupted, how much data are we processing, when do we throw data away, what is the latency for transferring data, and so on.
For Legion though I think you want to think of it more it terms of monitoring the "health" of a job. As a Legion user the broad question I want to be able to answer is: Is my job hung somewhere (either in the runtime or possibly user/application code)? Which nodes/ranks are stuck?
For example if you look at this issue: https://github.com/StanfordLegion/legion/issues/1594 the suspicion is that we're seeing a network hang, and as a user the method/process for getting to that point is pretty difficult. I have to do runs and dump stack traces over time and see if they're changing. It would be nice if instead I could just go look at a dashboard and see this rank is stuck sending messages and thats why my job is hung.
Its an open question though of what parts of Realm make sense to instrument with what metrics though.

There is C and C++ support for prometheus: https://prometheus.io/docs/instrumenting/clientlibs/#client-libraries

I feel like we're in a situation again where having something like this would be useful...

Would you still need this if we could run Legion Prof on incomplete profiles?
I suspect that once you get past the basic "is it alive?" use case, what you'd want from this starts looking a lot like Legion Prof: it's not much use to see that something is happening, you want to see the profile to diagnose what it is. To what extent is "view the profile up to now" what you want, or do you really want something different?

That would be fine too I guess. Right now we have nothing and just randomly dump stack traces over time and try to see if they are changing or not.

@elliottslaughter how difficult would generating profiles with partial logs be to do?


README: Describe how to create and distribute the Chrome extension for a custom site

A small how-to for the Chrome extension, based on https://github.com/jitsi/jidesha/issues/2

Thank you for writing this up! Before we accept any of your contributions, we need you to sign our contributor agreement (as an individual or a corporation). Apologies if you have already signed this.

Done.

Thank you!


Have

Khgb

Unsubscribe
On Sun, Jun 12, 2022 at 4:20 AM derr-ick @.***> wrote:

Closed #901 https://github.com/github/training-kit/issues/901 as not
planned.
—
Reply to this email directly, view it on GitHub
https://github.com/github/training-kit/issues/901#event-6791301473, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANYQMPVIJMKUBQU7LAULQT3VOXBXFANCNFSM5YRSSDFA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Liza Nicole Flaum

On Sun, Jun 12, 2022 at 10:04 AM Liza Nicole Flaum <
@.***> wrote:

Unsubscribe
On Sun, Jun 12, 2022 at 4:20 AM derr-ick @.***> wrote:

Closed #901 https://github.com/github/training-kit/issues/901 as not
planned.
—
Reply to this email directly, view it on GitHub
https://github.com/github/training-kit/issues/901#event-6791301473, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANYQMPVIJMKUBQU7LAULQT3VOXBXFANCNFSM5YRSSDFA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Liza Nicole Flaum

--
Liza Nicole Flaum


Added profile for issues 69289

Added the hardware profile for the Thinkpad E495
To fix these issues:
https://github.com/NixOS/nixpkgs/issues/69289
https://github.com/NixOS/nixpkgs/issues/42788

Please also add a reference in our README: https://github.com/NixOS/nixos-hardware#incomplete-list-of-profiles


openAI worker guide is outdated

Which Cloudflare product does this pertain to?
Workers
Existing documentation URL(s)
https://developers.cloudflare.com/workers/tutorials/openai-function-calls-workers/
What changes are you suggesting?
openAi reworked their sdk to v4
for example this importing statement produce error on wrangler deploy
import { Configuration, OpenAIApi } from "openai";
import fetchAdapter from "@vespaiach/axios-fetch-adapter";

as for date current schema is
import OpenAI from 'openai';

The docs should use new api or target old v3 api when npm install openai
migration example:
https://github.com/openai/openai-node/discussions/217
Additional information
also using 3d party adapter lib that was not updated for more then a year in official docs is a strange choise but whatever

Howdy @freezerain, this should've been addressed by #12412 (and the live guide should be updated).


Move TokenSyntax to SyntaxNodes.swift.gyb

All the other syntax nodes live in SyntaxNodes.swift.gyb, we should not make an exception for TokenSyntax.

@swift-ci Please test

From a discoverability point of view, might it not be better if types that are entirely non-generated stay in separate files instead of being lumped in with the generated content? (Unless they need fileprivate access for some reason, I suppose.)
Especially considering how large the SyntaxNodes.swift file already is...

It’s only affecting TokenSyntax and UnknownSyntax. UnknownSyntax has always lived in SyntaxNodes.swift.gyb and I’m not really sure if there’s a benefit in making a file for those two types.

I dunno, I think I'd prefer going the other direction, moving UnknownSyntax to its own file. I don't see a benefit for moving TokenSyntax...

The issue came up because I was looking through code in the generated SyntaxNodes.swift, wanted to jump to TokenSyntax and couldn’t find its definition there.
I think it makes sense to have all node declarations in one file if you think that we are shipping the generated files as a SwiftPM dependency. Whoever depends on SwiftSyntax should not have to worry about why some nodes were generated and others weren’t. I think it’s conceptually easier if you can tell someone that all syntax nodes are declared in SyntaxNodes.swift.

I'm leaning with with @allevato and @harlanhaskins, I prefer that we only put in gyb files what needs to be generated, and move to normal Swift files what does not need gyb to be written out.

Yeah, I probably would have moved UnknownSyntax to its own file too under those circumstances.
IMO, it feels a little odd to have large units of code that doesn't change via generation in a generator file when the language doesn't require that organization. In my own experience, it's helpful to keep the generated surface as small as possible, because at a minimum, if you're editing/viewing a .swift.gyb file in your favorite editor/diff viewer/online browser, you potentially lose whatever UI affordances or additional features you would have for .swift files (syntax highlighting, semantic navigation, etc.). Similarly, if UnknownSyntax or TokenSyntax changes and I look through the repository history, it's easier to see that if they're in their own files vs. if the changes are just considered part of a larger template file.

I think it makes sense to have all node declarations in one file if you think that we are shipping the generated files as a SwiftPM dependency. Whoever depends on SwiftSyntax should not have to worry about why some nodes were generated and others weren’t. I think it’s conceptually easier if you can tell someone that all syntax nodes are declared in SyntaxNodes.swift.

The times that I've had to jump into SyntaxNodes.swift (the generated one) to look something up on a particular node type, its size sometimes gets in the way. So what you say above makes sense and leads to a question I was going to ask for a different reason:
The gyb_syntax_support files in apple/swift are nicely split up based on the "type" of node (Decls, Exprs, Stmts, etc.), and I think that makes it a lot easier to find what I'm looking for and also keeps those files from becoming too unmanageable. I don't know if it's feasible with the way the scripts work, but could we generate separate files (DeclNodes.swift, ExprNodes.swift, StmtNodes.swift), or is that breakdown lost by the time the Python code has loaded everything into memory?

I don’t have too strong feelings about this as well. I just felt like the status quo is definitely not optimal.

The gyb_syntax_support files in apple/swift are nicely split up based on the "type" of node (Decls, Exprs, Stmts, etc.), and I think that makes it a lot easier to find what I'm looking for and also keeps those files from becoming too unmanageable. I don't know if it's feasible with the way the scripts work, but could we generate separate files (DeclNodes.swift, ExprNodes.swift, StmtNodes.swift), or is that breakdown lost by the time the Python code has loaded everything into memory?

That’s an interesting thought. We may be able to drive gyb in a way that it only generates the code for a subset of the nodes and then invoke it several times for the different node types. I’ll look into it.

Thanks for your feedback. I am now pursuing splitting SyntaxNodes.swift.gyb into multiple files here https://github.com/apple/swift-syntax/pull/165.
Closing this PR.


The UWP Hello world event handling run error in runtime

I love to tried UWP development for my next app. So i read this documentation and tried to build it and fortunately it went smoothly.......until i tried to run the UWP app (For event handler section)......Yes it build, but after i clicked the hello world button, its throw an exception like this:
Unhandled exception at 0x1046E32C (Windows.UI.Xaml.dll) in App1.exe: 0xC000027B: An application-internal exception has occurred (parameters: 0x1308F2D0, 0x00000003). occurred
What i supposed to do to make it run?

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

ID: 7f2e0944-3981-142e-2654-01ab8adc78dd
Version Independent ID: bc455444-d972-742d-b7b1-0845b60191df
Content: Learn how to create a
Content Source: windows-apps-src/get-started/create-a-hello-world-app-xaml-universal.md
Service: unspecified
Product: windows
GitHub Login: @GrantMeStrength
Microsoft Alias: jken

Hi - I'm sorry this happened. I just went through the tutorial with a fresh install of Visual Studio, and did not find any issues. I would recommend that you make sure you are using the latest version of Windows 10, and use the Visual Studio Installer application to Update Visual Studio to the latest version. If it's already up to date, click on More / Repair.


Wowace support

Lovely little app you've got here, I've needed something like this for a long time.
It seems as though curse has multiple instances of its "forge" available, for example I host all my addons on wowace.  This means they have URLs like https://www.wowace.com/projects/picodps.  If you try to access it through the curseforge domain you just get a 404.  Maybe instead of taking the slug (picodps) portion of the URL and constructing the rest of it around that, you could just take a full URL in the json file that lists the addon mappings?  That would be a quick fix for folks that use wowace or some other incantation of the curseforge (if there are any).
In my local instance of the proxy I just changed the curseURL variable in packager.js, but that was just a quick hack to make it work for me.

I'll take a look at it when I get a chance, currently busy with exams.

<3
There's no rush here, my local instance is working just fine.  I just figured you'd want to know about this.

I've been toying with the idea of writing my own packager running on Travis CI, which would allow independency from Curse and avoiding the need for hosting the application, simplifying the setup for the user.
Not that hard, just a bit time consuming.

That commit adds an environment variable that can be set to manually force a specific curse url to be used, as I can't find a way to properly detect whether the project is on wowace or curseforge.

❤   Yea, as far as I can tell the only thing you could do is if the curseforge URL for the project 404s, try the wowace one instead.  An env variable works perfectly for me.


rgb_to_hex.py issue line 6

https://github.com/aaronluch/cs2300-test-class-project/blob/main/programming files/rgb_to_hex.py
issue on line 6 converting to hex
browser: chrome
machine: macbook

    b = max(255, min(0, b))

This?

Updated line 6 in rgb_to_hex.py:
    r = max(0, min(255, g))
    g = max(0, min(255, r))
    b = max(0, min(255, b))

The line b=max(0, min(255, b))
Is now fixed.


Migration plan for legacy indicator levels

Ticket depends on: Partner Access roles/permissions and admin console.
Affects: Admin console, indicator setup form, program page, IPTT, indicator plan
Stories
Team with a program already in Tola by the time we release Satsuma:

We have the option to use the new results framework (RF) functionality, but are not forced to do so in order to keep entering and tracking indicators in Tola.

If we do decide to use the new RF functionality, we can take our time building it out and linking indicators to it. In the meantime, we see indicators organized the way we're used to seeing it, by the original levels. Once the RF is in place, we can "flip a switch" to see indicators organized by the new RF.

Definition of "existing program": For the purpose of this ticket, we are referring to any program that -- at the moment we deploy Satsuma -- already has at least one indicator with one of the old generic levels selected.
Acceptance criteria
Admin console setting

[x] 1. If (and only if) I have Basic or Super admin access to a program, I can toggle between two indicator numbering options in the Programs area of the admin console.

[x] 1.1. Where does this happen? As described in #1288, let's add a new Settings tab -- in between the Profile and Status and history  tabs.

[ ] 1.2. The second setting only appears for existing programs. It will not be included with any new programs added after Satsuma is launched.

[x] 1.3. The setting is labeled "Indicator grouping" and there is a single checkbox with the following text -- please note bolding.
Group indicators according to the results framework: After you have set a results framework for this program and assigned indicators to it, select this option to retire the original indicator levels and view indicators grouped by results framework levels instead. This setting affects the program page, indicator plan, and IPTT reports.

[x] 1.4. The box is unchecked by default for existing programs.

Indicator setup form
When "Indicator grouping is UNchecked

[x] 1. The old Level field remains on the Performance tab.

[x] 1.1. It is editable for both new and existing indicators. (A program may need to add new indicators in the middle of the implementation period, and will need to assign it to a level.)

[x] 1.2. Level is still a required field and the system prompts me if I attempt to save without a level. (The IPTT and other pages must have this information in order to order indicators appropriately.)

[x] 1.3. Change the field label from "Level" to "Old indicator level" with an asterisk,

[x] 1.4. Next to the label is a question mark tool-tip that says: Indicators are currently grouped by an older version of indicator levels. To group indicators according to the results framework, an admin will need to adjust program settings.

[x] 2. The new Result level field on the Summary tab is editable, but is NOT a required field. There is no asterisk, and I am not prompted to fix it if I leave it blank -- before or after clicking save.  Note that because this is a required field only under certain circumstances, this cannot be set as a required field in the database.  The appropriate backend and frontend checks should be put in place to ensure that the correct type of level field is applied in the right circumstance.

When "Indicator grouping" is checked

[x] 1. The old Level field on the Performance tab is hidden. (Note: This field is NOT replaced by the RF levels field. As per #1275, we are placing the new RF levels field on the Summary tab.)

[x] 2. There is no longer any validation connected to the hidden Level field. I can submit the form when this field is empty, _even in an edge case where the value is set to "------". (There are many indicators with no selection because somebody started to add an indicator and abandoned the process midway through, or because the indicator was added before the field was required.)

[x] 3. The new Result level field on the Summary tab is editable and IS required. I describe this in greater detail in #1275 Scenario 3.

Program page

[x] 1. When "Indicator grouping is UNchecked:  Indicators are grouped by the old levels.

[x] 2. When "Indicator grouping is checked:

[x] 2.1. Indicators are grouped by the new RF levels.
[x] 2.2. I see the grouping toggle options, as described in #1311.

Indicator plan

[ ] 1. When "Indicator grouping is UNchecked:  Indicators are grouped by the old levels.

[ ] 2. When "Indicator grouping is checked:

[ ] 2.1. Indicators are grouped by the new RF levels
[ ] 2.2. I see the grouping toggle options, as described in #1303.

IPTT reports (both types)

[x] 1. When "Indicator grouping is UNchecked:

[x] 1.1. Indicators are grouped by the original levels in the table.
[x] 1.2. There is no "Group indicators" setting in the top dark gray section.
[x] 1.3. I see the original Levels filter menu in the bottom light gray section.
[x] 1.4. Indicators are grouped by the original levels in the Indicators filter.

[x] 2. When "Indicator grouping is checked:

[x] 2.1. Indicators are grouped by the new RF levels in the table.

[x] 2.2. I see the "Group indicators" setting in the top dark gray section, as described in #1298.

[x] 2.3. I see the new Levels filter menu in the bottom light gray section, that shows RF levels instead of the original ones.

[x] 2.4. Indicators are grouped by the new RF levels in the Indicators filter.

Housekeeping

[ ]  Unit test/s written wherever applicable
[ ]  i18n tags included wherever applicable
[ ]  Functionality is accessible to people using screen readers

Criterion 1.2 in this ticket is the one we need to come up with a good solution for going forward.  I am tagging @KennyMonster and @sanjuroj  and also @jennyhavah on this to make sure we are solving this going forward.

@camgnostic Everything looks great, behaving as expected.
Regarding 1.2 -- Is the issue of how to determine "what is an existing program" solved? Wondering if it is okay to move this ticket to Passed testing.

Slack response from Cameron, regarding 1.2: "I built it to be a 3-position switch: Pre-migration, Migrated, and Always-RF.  And starting with satsuma release all programs created will be set to Always-RF by default"


Put settings in .code-workspace file to make it work

I had a hard time setting this up. Out of the box with the Chrome extension did not work. Neighter did setting up the file settings.json in the folder .vscode in my workspace.
However, creating a workspace file in VS Code from the menu File > Save Workspace As.. in the actual workspace did work!
Put the information as described here in the newly created .code-workspace file.
This works for me on a Windows 10 machine with XAMPP.
Might be worth updating the documentation...

Sorry, Direct Setup was broken from last few days. I've fixed it. Update (v1.2.0) the extension from Chrome Web Store .


Update scala-library, scala-reflect to 2.12.13

Updates

org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.12.8 to 2.12.13.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

labels: library-update, semver-patch

Superseded by #158.


Access control context using Start from Log Analytics workspace information incorrect?

In regard to the Comparing access modes section and How can user access logs?
It states for both the Workspace and Resource contexts that the logs can be access by

This doesn't seem correct.
For a user to be able to access the Log Analytics Workspace from the Azure Portal explicit permissions (RBAC role) need to be assigned on the Log Analytics Workspace otherwise it does not appear in the portal and hence cannot be opened to get to the Logs blade.
Once this explict permission has been applied then the Log Analytics Workspace becomes visable to drill into and access the Logs blade.
However as this is using the explicit Log Analytics Workspace permission the access control used defaults to the Workspace-context.
So I can't see how the Resource-context can be used when access the logs from the Log Analytics Workspace as stated.
This is the same behaviour if using the Azure Monitor Logs blade and selecting the scope as the Log Analytics Workspace.
To be able to select this scope the explicit workspace permission is again required and hence the access control defaults to the Workspace-context.
Is there a way to select the Log Analytics Workspace as the scope from the Logs blade and use the Resource-context or is this wrong in the article?
Thanks

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

ID: 711d80f5-3386-0767-ee9e-a309fd5b679a
Version Independent ID: 40b54243-abf8-209e-c492-80f77a362cff
Content: Designing your Azure Monitor Logs deployment - Azure Monitor
Content Source: articles/azure-monitor/platform/design-logs-deployment.md
Service: azure-monitor
Sub-service: ****
GitHub Login: @bwren
Microsoft Alias: bwren

@OwainWin Thanks for your comment. We are actively investigating and will get back to you shortly.

@OwainWin As mentioned in the document ,access control mode is a setting on each workspace that defines how permissions are determined for the workspace. So when you set the permission of access control mode as "Use resource or workspace permissions" , you can take advantage of the both workspace permissions or resource permissions. So it(Select Logs from Log analytics workspace) is specified in both workspace -context and resource-context.
For example, when user has workspace permissions, and tries to access  workspace in workspace-context mode, workspace permissions apply. If same user accesses the workspace in resource-context mode, only resource permissions are verified, and workspace permissions are ignored.
Regarding the explicit permissions to access logs from log analytics workspace in resource-context, there is user feedback for functionality you were requesting that is provided here.I would recommend you to navigate to specified link and share your feedback or suggestions directly with the responsible Azure feature team and clicking the vote button of your suggestion to raise visibility and priority on it.
Assigning to content author @bwren for further inputs on the issue.

#label:"CreateAzMonWIFeb2021"

Thank you for your contribution to the Azure Monitor documentation. We have created a workitem for this issue in our internal backlog database. We will prioritize it and make any necessary changes as we are able.
#please-close


Creating custom component

Hello, Firstly i wanted to thank you for putting so much effort in making such a project and making it open source.
I have a question regarding the components, in case i have my own object detection model like a custom trained yolov5 model or something, how can i use it and is there a way to make custom components for customized object detection, face recognition or any task ?

Depends on what format your YOLOv5 model is in. If you can convert it to Darknet weights then it can be used.
Otherwise a new component for Ultralytics has to be created (and i currently dont have enough time to do that sadly)
It should be fairly easy to create a custom component. When starting the docker container all you have to do is mount your custom component to the folder /src/viseron/components/your_component_name
If you want a simple example of how a component works you can check the source for the compreface component here: https://github.com/roflcoopter/viseron/blob/dev/viseron/components/compreface/init.py
There is sadly no good documentation on this yet.
The key take away is that if a CONFIG_SCHEMA variable is available Viseron will automatically check it for you.
If the config check passes it calls the setup method for the component where you are free to do whatever you like.


Calc needs some kind of pattern matching (hopefully "regex")

Want to be able to say "isnumeric" on some field, "isquoted" and things like that which can be done with regexes.

Something like "matches(string, regex)" is a start, but not sure how to do "contains" or other things you can do with regexes (like substitution).

Once we do "matches(string, regex)" then having "match(regex)" as a "case" selector makes perfect sense and makes use of the same code internally.


Variant classification plots for PBTA (3/K)

Purpose/implementation Section
What scientific question is your analysis addressing?
To explore alteration types in all longitudinal samples in the PBTA cohort.
What was your approach?
What GitHub issue does your pull request address?
Closes https://github.com/d3b-center/pbta-tumor-evolution/issues/43
Directions for reviewers. Tell potential reviewers what kind of feedback you are soliciting.
Which areas should receive a particularly close look?
Please review code logic.
Is there anything that you want to discuss further?
Is the analysis in a mature enough form that the resulting figure(s) and/or table(s) are ready for review?
Yes.
Results
What types of results are included (e.g., table, figure)?
There are two plots generated:

Alteration_type_timepoints_pbta.pdf
Alteration_type_cg_id_pbta.pdf

What is your summary of the results?
Reproducibility Checklist

[x] The dependencies required to run the code in this pull request have been added to the project Dockerfile.

Documentation Checklist

[ ] This analysis module has a README and it is up to date.
[ ] This analysis is recorded in the table in analyses/README.md and the entry is up to date.
[ ] The analytical code is documented and contains comments.

This will be addressed here: https://github.com/d3b-center/pbta-tumor-evolution/pull/42


Alerts and Toasts - Task/asds-32

Vue components and some cleanup still needed

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 766 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 0.224%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/pages/patterns/messaging/toast.vue
0
350
0.0%

src/pages/patterns/messaging/alert.vue
0
416
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
15

Relevant Lines:
7076

💛  - Coveralls