Remove upcoming deprecations

We were a bit delayed by the SpikeInterface release, but since we're changing the import structure anyway we might as well clean up other stuff ahead of the upcoming release.
The icephys module still has the save_path deprecation warning it since we caught that one much later than the others - should we remove it here as well?

@CodyCBakerPhD I am curios how much the coverage will increase with this one.

@CodyCBakerPhD I am curios how much the coverage will increase with this one.

Lol. We probably should have used # pragma on that originally.

But it's meaningless coverage either way so it's just artificial inflation/deflation

But it's meaningless coverage either way so it's just artificial inflation/deflation

Don't be a party popper, @CodyCBakerPhD  : P

This branch got tangled with the ROI centroid stuff somehow. Re-doing


refactor OpenAPI paths

This PR follows the spirit OGC API - Tiles and refactoring OpenAPI Paths as building blocks.  This allows downstream applications to bundle/profile their own OpenAPI definitions according to their support/requirements.

SWG 2022-01-12: Agreed to merge this PR.


QUICKSTEP-65 Fix Quickstep build failure on Mac OSX 10.12

Quickstep build on Mac OSX 10.12 fails due to deprecation of certain system APIs in the new OSX version. The Quickstep code does not rely on these deprecated APIs, but it is the protobuf library which uses these deprecated APIs. [See: https://github.com/google/protobuf/issues/2182]
Although, this has been resolved in the latest master of protobuf [See: https://github.com/google/protobuf/pull/2337], Quickstep protobuf cmake integration file has to be updated resolve this.
As this is turning out to be a blocker for the dev environment, this JIRA proposes a quick short-term fix for this issue by specifically turning off deprecation errors on OSX 10.12 platform. The more stable long-term fix would be to port the protobuf_cmake to the newest protobuf version. However, that requires some non-trivial effort and will be done through a separate JIRA.

@cramja Could you please verify if this fix works on your Mac as well? Thanks.

compiles on my mac, +1

It is also looking good to me. Merging.

Thanks @hakanmemisoglu  and @cramja  for reviewing this.


[SPARK-41533][CONNECT] Proper Error Handling for Spark Connect Server / Client

What changes were proposed in this pull request?
This PR improves the error handling on the Spark Connect server and client side. First, this patch moves the error handling logic on the server into a common error handler partial function that differentiates between the internal Spark errors and other runtime errors.
For custom Spark exceptions, the actual internal error is wrapped into a Google RPC Status and sent as trailing metadata to the client.
On the client side, similarly, the error handling is moved into a common function. All GRPC errors are wrapped into custom exceptions to avoid presenting the user with confusing GRPC errors. If available the attached RPC status is extracted and added to the exception.
Lastly, this patch adds basic logging functionality that can be enabled using the environment variable SPARK_CONNECT_LOG_LEVEL and can be set to info, warn, error, and debug.
Why are the changes needed?
Usability
Does this PR introduce any user-facing change?
No
How was this patch tested?
UT

I'm working on implementing centralized PySpark error messages from https://github.com/apache/spark/pull/39137, and it will includes whole errors generated by PySpark packages including Spark Connect.
Seems like the improvement errors from Spark Connect side here is not conflicts with my current work.
So, I think we can merge them separately, and I'll integrate the Spark Connect errors into the centralized error framework when the both PR is get merged. (Seems the merging order is not matter from my look)

I'm absolutely looking for suggestions on designing the error structure. If it doesn't make sense please let me know.

Can one of the admins verify this patch?

Updated the documentation.

Merged to master.


cleanup(libsinsp): fix warnings on e2e tests

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind cleanup

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area libsinsp
/area tests

Does this PR require a change in the driver versions?
No

What this PR does / why we need it:
This is necessary to add sanitizers and warnings enabled for our new e2e tests
Which issue(s) this PR fixes:

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

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

cc @therealbobo

/milestone 0.15.0

LGTM label has been added.  Git tree hash: dfb1557275db6dc8751272535192865fa1473f2d

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,LucaGuerra,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


a small error in train.py

https://github.com/jnulzl/PRNet/blob/bbdc3c95266aa4ce6379cd343121616244964872/train.py#L39
this should be 'label', instead of 'img', like as:
label_array = np.array(label,dtype=np.float32)
by the way, thanks for your train code!

label_array = np.array(label,dtype=np.float32)
can you tell me the datasets where i can get:?

@destiny311 I have met the same problem, have you solved it? What is "--train_data_file“

@destiny311 @LucienXian I get the dataset by search "300W-LP" in Google, and you can find the download link of 300W-LP in the  web http://www.cbsr.ia.ac.cn/users/xiangyuzhu/projects/3DDFA/main.htm.
I generate groundtruth position map just using  generate_posmap_300WLP


feat(eigen-client-extra-feature): rlp encoding

What ❔
Implement rlp encoding for the dispersed blob.
Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[ ] Code has been formatted via zkstack dev fmt and zkstack dev lint.

Looks great but add some unit tests for RLP encoding/decoding.

A simple test that encodes and decodes a blob has been added, do you think it's enough?


xyzrgb2dotspdb produces points on a plane

Using xyzrgb2dotspdb.py
python utils/xyzrgb2dotspdb.py examples/VALSARTAN.xyzrgb

to convert a xyzrgb file into a pdb file for visualization with PyMOL produces points on a plane:

@douguet Thanks for sharing the code on GitHub; I'm evaluating it for my research and opened a few issues and PRs that I hope might be useful for the wider community.
Really sorry for the ping, but I just noticed that no-one is watching the repository and most of the latest commits are tagged as "Add files via upload" which makes me think about an automated sync, so I thought you might have not received any notification about open issues and PRs.

Dear RMeli, thanks for your feedback.  I fix this issue. Regards

Thanks @douguet . I will close my PR #2 .


Functional test rebase

By default the functional tests are not run when executing cargo test. To run them, run cargo test -- --ignored.
Before running the functional tests, start the docker containers first by running docker-compose up in the tests directory.
Docker creates and mounts a volume in the data_dir directory. The owner of this directory is the root user, so ownership has to be changed before running the functional tests with sudo chown -R $USER data_dir.
Alternatively, the regtest setup can be run directly on the host, with the tests expecting the following endpoints:

Bitcoind rpc cookie at ./tests/data_dir/regtest/.cookie
Bitcoind at http://localhost:18443
Electrs at tcp://localhost:50001
Monerod at http://localhost:18081 run with arguments --regtest --offline --fixed-difficulty 1
Two instances of Monero-wallet-rpc at http://localhost:18083 and htp://localhost:18084 run with arguments --disable-rpc-login --wallet-dir wallets

I added some documentation and ensured that the tests are only run when explicitly mentioned by annotating them with #[ignore].


How does NavigationService.ClearCache() work?

ClearCache seems to preserve the current size of the cache, set it to another value and then set it back to the previous value.
The intermediate value is one of the following:

0 if removeCachedPagesInBackStack is true
1 if BackStackDepth is 0
BackStackDepth otherwise

Is there some magic (undocumented) behaviour of setting CacheSize to these values? According to https://www.jayway.com/2012/05/25/clearing-the-windows-8-page-cache/, setting it to 0 clears the cache of all pages but I'm struggling to understand how the other two values behave.

This is a good question. What should it do?

Here's where it was born.
https://github.com/Windows-XAML/Template10/pull/974

That is where the code is being called from but I'm not sure it is the birth of it.
git blame breaks the changes down as follows:
078dcd2b Template10 (Library)/Services/NavigationService/NavigationService.cs      (Brandon Hurlburt     2015-10-12 15:58:41 -0500 357)         public void ClearCache(bool removeCachedPagesInBackStack = false)
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 358)         {
e25fa59a Template10 (Library)/Services/NavigationService/NavigationService.cs      (jerrynixon           2016-02-06 08:05:47 -0700 359)             DebugWrite($"Frame: {FrameFacadeInternal.FrameId}");
8a5e5881 Template10 (Library)/Services/NavigationService/NavigationService.cs      (jerrynixon           2016-01-16 10:19:42 -0700 360)
e25fa59a Template10 (Library)/Services/NavigationService/NavigationService.cs      (jerrynixon           2016-02-06 08:05:47 -0700 361)             int currentSize = FrameFacadeInternal.Frame.CacheSize;
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 362)
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 363)             if (removeCachedPagesInBackStack)
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 364)             {
e25fa59a Template10 (Library)/Services/NavigationService/NavigationService.cs      (jerrynixon           2016-02-06 08:05:47 -0700 365)                 FrameFacadeInternal.Frame.CacheSize = 0;
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 366)             }
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 367)             else
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 368)             {
e25fa59a Template10 (Library)/Services/NavigationService/NavigationService.cs      (jerrynixon           2016-02-06 08:05:47 -0700 369)                 if (FrameFacadeInternal.Frame.BackStackDepth == 0)
e25fa59a Template10 (Library)/Services/NavigationService/NavigationService.cs      (jerrynixon           2016-02-06 08:05:47 -0700 370)                     FrameFacadeInternal.Frame.CacheSize = 1;
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 371)                 else
e25fa59a Template10 (Library)/Services/NavigationService/NavigationService.cs      (jerrynixon           2016-02-06 08:05:47 -0700 372)                     FrameFacadeInternal.Frame.CacheSize = FrameFacadeInternal.Frame.BackStackDepth;
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 373)             }
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 374)
e25fa59a Template10 (Library)/Services/NavigationService/NavigationService.cs      (jerrynixon           2016-02-06 08:05:47 -0700 375)             FrameFacadeInternal.Frame.CacheSize = currentSize;
4cd848d3 Template10 (Library)/Services/NavigationService/NavigationService.cs      (Jerry Nixon          2015-11-30 19:51:01 -0700 376)         }

so it looks like the method was initially started on 12th October 2015 by Brandon Hulburt. I can't find this commit, though, nor the other commits in order to drill down further to understand what Brandon wrote or what you wrote, Jerry.

Seeking some clarification - what is this a duplicate of?


 It looks fine, but I don't notice a difference in the fonts for the notification message.

@katevoss There should be a visible change, you might to do a full refresh if you're switching branches. The intended behavior is for the notifications to use Montserrat, instead of Arial.

When I view this on my local machine the change works as expected. I can see the difference between the fonts and confirmed the ordering of the CSS in the source. However, when I go to bvdp-saturn-dev.appspot.com or app.terra.bio I am seeing the Montserrat font right now.
The CSS is minified and loaded in chunks on these two sites, but this does not seem to be the case in the local dev environment. I looked at the CSS source in bvdp-saturn-dev and found that the easymde css (added to chunk 2) is loaded before the react-notifications CSS (added to chunk 3).
My only concern is that this change, when fully built could have the reverse of the intended effect.

It does look like the CSS is bundled differently in local vs prod, so this issue likely only affects local builds. The PR site uses the same build process as prod, so you can check that everything looks good there.


Simple Example(s)

I am hoping to use this library since I have to integrate an api that uses Hmac/SHA-512 and the base crypto library for Dart doesn't support that (?????).  I am having a hard time finding an end to end example of how to emulate the api documentation.  Can someone point me to a place with an example?
Thanks!!!

I have found a couple of things but I am having a hard time getting the same result as my working examples in Postman.  Does anybody have any examples?

Specifically trying to duplicate this (JS) in PointyCastle, and I cannot get it :(
CryptoJS.enc.Hex.stringify(CryptoJS.HmacSHA512(MESSAGE, KEY));

Sigh, ok, at the end of the day I think my lack of understanding led to, what I am sure, is a very easy concept to understand.  Essentially I was using a block size of 64 (which seems to be the default of the SHA-512 algo, but it seems that it actually wants 128 to match the output of the JS above.
So for whatever person runs into the same issue here is a very concise example of how to do a SHA-512 HMAC using pointycastle
Uint8List hmacSHA512(Uint8List key,Uint8List data) {
  final _tmp = new HMac(new SHA512Digest(), 128)..init(new KeyParameter(key));
  return _tmp.process(data);
}

String formatBytesAsHexString(Uint8List bytes) {
  var result = new StringBuffer();
  for( var i=0 ; i<bytes.lengthInBytes ; i++ ) {
    var part = bytes[i];
    result.write('${part < 16 ? '0' : ''}${part.toRadixString(16)}');
  }
  return result.toString();
}

Uint8List createUint8ListFromString( String s ) {
  var ret = new Uint8List(s.length);
  for( var i=0 ; i<s.length ; i++ ) {
    ret[i] = s.codeUnitAt(i);
  }
  return ret;
}
Uint8List message = createUint8ListFromString("12345");
Uint8List key = createUint8ListFromString("test");
String digest = formatBytesAsHexString(hmacSHA512(message, key)));

You could use the algorithm registry and just done new Mac("HMAC/SHA-512"). And use the hex package for hexadecimal encoding and decoding.

Cool, I will give that a shot, thank you

@stevenroose @bagintz  I am struggling with this. Cannot find any example in ReadMe or anywhere else in order to implement SHA-512 encryption. It's really urgent, please help me out if possible.
Need to implement something like this
const encryptedHash = CryptoJS.SHA512(paramsList);
This is a JS snippet.

I'm not longer able to actively maintain this package. All I can do is look at PRs and merge them if I have the time to verify them or if they have been confirmed working by other contributors.
Every once and a while someone comes by and complaints about lack of example code. I sometimes give them some pointers and invite them to contribute their examples into the repo. I never hear again from them.
I think I'm finally gonna put up a message on top of the README that it's no longer maintained.

String digest = formatBytesAsHexString(hmacSHA512(message, key)));

you twisted the message and key parameter for correct results


A11y_VSCode_Problems Panel_AI4D:Name of the filter edit box is provided as "No problems have been detected in the workspace so far" which is incorrect.

Environment Details:
VSCode Version : 1.44.2
OS Version : Win10
Additional Details:
Screenreader: NVDA[2019.3.1], JAWS[2020.70]
GitHubTags: #A11yMAS; #A11yTCS; #Win32; #DesktopApp; #A11y_VSCode; #MAS1.3.1; #MAS4.2.1; #A11ySev2; #AI4D
Repro Steps:
1)Open Visual studio code editor
2)Press ctrl+shift+M
3)Validate it's possible to navigate between items in the problems view
4)Open Accessibility Insights for desktop and check the name of the filter edit box present besides to debug console tab after click on Problems Tab.
Actual:
Name of the filter edit box is provided as "No problems have been detected in the workspace so far" which is incorrect.
Expected:
Name should be provided as "Filter" and control type should be edit.
User Impact:
Screen reader user unable to get the purpose of the control.
Attachment For Reference:

Does this issue occur when all extensions are disabled?: Yes

Assigning to @isidorn for insights.

@sandy081 thanks for the ping. The issue is that you are setting aria-labelledby=markers-panel-arialabel on the filter input. Just remove that and it will be good since you set a proper aria-label on the element
So this line should be removed https://github.com/microsoft/vscode/blob/8f0f44cd0f718e9c2277758b400b7b34676d7bf6/src/vs/workbench/contrib/markers/browser/markersViewActions.ts#L326

Thanks.

@sandy081 The issue is still reproduced that the name is provided as No problems have been detected in the workspace so far". The name should be Filter.

Verified and Closed .Please refer attachment below
**

**

Thanks for verifiying, adding verified label.


Test failure with ruby 2.2.2 / Gentoo

ruby 1.9, ruby 2.0 and ruby 2.1 seems to work, but the tests fail in ruby 2.2. Do you have any idea why this happens?
>>> Test phase: dev-ruby/redcarpet-3.3.2
 * Running test phase for ruby20 ...
/usr/bin/ruby20 -I"lib:lib:test" -I"/usr/lib64/ruby/gems/2.0.0/gems/rake-10.4.2/lib" "/usr/lib64/ruby/gems/2.0.0/gems/rake-10.4.2/lib/rake/rake_test_loader.rb" "test/*_test.rb"
Run options:

# Running tests:

Finished tests in 0.259141s, 420.6208 tests/s, 744.7689 assertions/s.
109 tests, 193 assertions, 0 failures, 0 errors, 0 skips

ruby -v: ruby 2.0.0p645 (2015-04-13 revision 50299) [x86_64-linux]
 * Running test phase for ruby21 ...
/usr/bin/ruby21 -I"lib:lib:test" -I"/usr/lib64/ruby/gems/2.1.0/gems/rake-10.4.2/lib" "/usr/lib64/ruby/gems/2.1.0/gems/rake-10.4.2/lib/rake/rake_test_loader.rb" "test/*_test.rb"
Run options:

# Running tests:

Finished tests in 0.246710s, 441.8150 tests/s, 782.2963 assertions/s.
109 tests, 193 assertions, 0 failures, 0 errors, 0 skips

ruby -v: ruby 2.1.6p336 (2015-04-13 revision 50298) [x86_64-linux]
 * Running test phase for ruby22 ...
/usr/bin/ruby22 -I"lib:lib:test" -I"/usr/lib64/ruby/gems/2.2.0/gems/rake-10.4.2/lib" "/usr/lib64/ruby/gems/2.2.0/gems/rake-10.4.2/lib/rake/rake_test_loader.rb" "test/*_test.rb"
Loaded suite /usr/lib64/ruby/gems/2.2.0/gems/rake-10.4.2/lib/rake/rake_test_loader
Started
..........................................................................F
===============================================================================
Failure:
  <//> expected to be =~
  <"
A ==simple== fixture file -- with a link.
\n">.
test_enabling_a_parse_option(RedcarpetBinTest)
/var/tmp/portage/dev-ruby/redcarpet-3.3.2/work/ruby22/redcarpet-3.3.2/test/redcarpet_bin_test.rb:74:in `assert_output'
/var/tmp/portage/dev-ruby/redcarpet-3.3.2/work/ruby22/redcarpet-3.3.2/test/redcarpet_bin_test.rb:30:in `test_enabling_a_parse_option'
     27:   def test_enabling_a_parse_option
     28:     run_bin("--parse", "highlight", @fixture_path)
     29:
  => 30:     assert_output ""
     31:     refute_output "=="
     32:   end
     33:
===============================================================================
F
===============================================================================
Failure:
  </\[link\]/> expected to be =~
  <"
A ==simple== fixture file -- with a link.
\n">.
test_enabling_a_render_option(RedcarpetBinTest)
/var/tmp/portage/dev-ruby/redcarpet-3.3.2/work/ruby22/redcarpet-3.3.2/test/redcarpet_bin_test.rb:74:in `assert_output'
/var/tmp/portage/dev-ruby/redcarpet-3.3.2/work/ruby22/redcarpet-3.3.2/test/redcarpet_bin_test.rb:37:in `test_enabling_a_render_option'
     34:   def test_enabling_a_render_option
     35:     run_bin("--render", "no-links", @fixture_path)
     36:
  => 37:     assert_output "[link]"
     38:     refute_output ""
     39:   end
     40:
===============================================================================
.F
===============================================================================
Failure:
  <//> expected to be =~
  <"
A ==simple== fixture file -- with a link.
\n">.
test_legacy_option_parsing(RedcarpetBinTest)
/var/tmp/portage/dev-ruby/redcarpet-3.3.2/work/ruby22/redcarpet-3.3.2/test/redcarpet_bin_test.rb:74:in `assert_output'
/var/tmp/portage/dev-ruby/redcarpet-3.3.2/work/ruby22/redcarpet-3.3.2/test/redcarpet_bin_test.rb:56:in `test_legacy_option_parsing'
     53:   def test_legacy_option_parsing
     54:     run_bin("--parse-highlight", "--render-no-links", @fixture_path)
     55:
  => 56:     assert_output ""
     57:     refute_output "=="
     58:
     59:     assert_output "[link]"
===============================================================================
...............................

Finished in 0.270107798 seconds.
------
109 tests, 155 assertions, 3 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
97.2477% passed
------
403.54 tests/s, 573.84 assertions/s
rake aborted!
Command failed with status (1): [ruby -I"lib:lib:test" -I"/usr/lib64/ruby/gems/2.2.0/gems/rake-10.4.2/lib" "/usr/lib64/ruby/gems/2.2.0/gems/rake-10.4.2/lib/rake/rake_test_loader.rb" "test/*_test.rb" ]

Tasks: TOP => test:unit
(See full trace by running task with --trace)

Hi @mrueg,
This seems pretty strange. I'm not able to reproduce this with Ruby 2.2.2 on my computer ; are you using 2.2.2 or a previous version ? I can't find anything that could explain this inside 2.2's changelog.

@robin850 I'm using 2.2.2, too. I've tested redcarpet-3.3.0 and 3.3.1, which show the same test failure.

Hum, and if you try the following:
$ echo "A ==simple== fixture file" >> fixture
$ redcarpet --parse highlight fixture

What is the output please ? I'm wondering whether this is the bin that doesn't work or something wrong inside the test case.

ruby22 -S redcarpet --parse highlight fixture
A simple fixture file
ruby20 -S redcarpet --parse highlight fixture
A simple fixture file

Okay, I think I got it.
The issue is caused by https://github.com/vmg/redcarpet/blob/master/bin/redcarpet#L1 and likely a downstream issue.
In Gentoo one is able to install multiple rubies in parallel, and one changes the symlink of /usr/bin/ruby to the ruby implementation that is desired (ruby 2.0, 2.1, 2.2).
Tests are run in all variants (we don't change the symlink here, but call the testsuite with each implementation, e.g., /usr/bin/ruby21 directly). This won't work the testsuite calls env to get the path to the ruby binary (which happens here) and caused the issue. I don't know if there's a "proper" upstream fix available, otherwise it works now on Gentoo again.
Thanks for your help!

Sorry if I'm missing something but I do not understand why this would be a problem ; it is supposed to work with Ruby 1.9.3+ so even if the "wrong" one is picked, it should not output any error.

I guess it is caused by the fact, that the test suite is called with e.g. ruby22 but the binary is using another ruby version.

Yep but this should not be a problem if this version is compatible with Redcarpet. Anyway, I will close this one ; feel free to comment if this comes back later and I will reopen! Thanks! :-)

@mrueg  How did you solve it, actually? We are facing the same in Debian.

@balasankarc https://gitweb.gentoo.org/repo/gentoo.git/tree/dev-ruby/redcarpet/redcarpet-3.3.2.ebuild#n32
/usr/bin/ruby is actually a symlink to ruby19, ruby20, ruby21 etc. so we'll just fix the shebang line per ruby version


Upgrade Linux Kernel for flatcar-3602 from 5.15.144 to 5.15.145

Subject says it all.

Closing in favor of https://github.com/flatcar/scripts/pull/1557


Code coverage badge problem

For the time being the code coverage is 10. Why?

Thanks for reporting an issue to us! We're glad you are using and invested in Enzyme.
Before submitting, please read over our commonly reported issues to prevent duplicates!
All common issues

common issues

Current behavior
The coverage is 10%
Expected behavior
The coverage reported could not be right.
Your environment
master of Github
API

[ ] shallow
[ ] mount
[ ] render

Version

library
version

enzyme

react

react-dom

react-test-renderer

adapter (below)

Adapter

[ ] enzyme-adapter-react-16
[ ] enzyme-adapter-react-16.3
[ ] enzyme-adapter-react-16.2
[ ] enzyme-adapter-react-16.1
[ ] enzyme-adapter-react-15
[ ] enzyme-adapter-react-15.4
[ ] enzyme-adapter-react-14
[ ] enzyme-adapter-react-13
[ ] enzyme-adapter-react-helper
[ ] others ( )

Thanks, our coveralls setup has been having some issues.

Fixed in 12f324abbfcf362156e3909b99a095ed876a6841.


How to validate Policy Document best practices.

I need to test the policy documents attached to the IAM Role, I found the below link but dont see nay such function iam.go,
https://github.com/gruntwork-io/terratest/pull/130

Looks like that PR never got wrapped up and merged, unfortunately. If you'd like to contribute some code to test IAM policy documents, that would be much appreciated!

Is there any chance to merge this PR? I need this urgently.

It looks like the existing PR is quite stale. It hasn't been worked on since 2018, it has merge conflicts and unaddressed review comments. So we can't merge until someone either addresses the issues in the existing PR, or opens an new PR.

Can you please take this as a feature request.

Sure, this feature is definitely something that the codebase could benefit from! Having said that, we don't have any availability to work on this at the moment. So if you'd like to see it done sooner rather than later, a PR would be very appreciated!


⚠ Cioos National CKAN has degraded performance

In ecd3ab8, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 8519 ms

Resolved: Cioos National CKAN performance has improved in 795f825 after 5 minutes.


Added the 'focus_on_fit' variable to exclude entities from fitting map and more

focus _on_fit - when adding entities such as zones to the map, there may be cases where you want to exclude these entities from the fit map target.
Add 'Reset focus' button on the map. This button icon uses the @mdi package
Set padding value to 0.1 when fitting the map.
Lexical fix for gradualOpacity in README.md
Add z_index_offset for Overlapping markers

I'll submit a new pull request later with the bug fix


Function return type inference fails when the possible return types have different structures for a field with the same name, and one is returned as a promise

🔎 Search Terms
function inference, promise, return type, union
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about function return type inference

⏯ Playground Link
https://www.typescriptlang.org/play?ts=5.4.0-beta#code/IYZwngdgxgBAZgV2gFwJYHsL3egFAShgG8AoGcmAJwFNkFKtSKKAjYSgLmLOYqgAscXAET9qAG3HphPZgF9ysmHJIqSoSLEQoMWNpQLdmNOgyO9y+rkwu8BQmKIlThAGiW2YLB8IDu6SnEAExlPFXlVEnVwaHgkKDRMLwQAc0MbchN6LABZYGR+ADpKYAgg9ABbQwAeGAAGQoBWGAB+JTgcQw4lfQIAblUgA
💻 Code
async function foo() {
    return {
      bar: {
        choo: "hello"
      }
    }
}

function bar() {
    return {
        bar: {
            choo: "hello",
            boo: "world"
        }
    }
}

async function bug() {
    return Math.random() < 0.5 ?
    await foo() :
    bar();
}

🙁 Actual behavior
bug's return type is
Promise<{
    bar: {
        choo: string;
    };
}>

🙂 Expected behavior
bug's return type should be
Promise<{
    bar: {
        choo: string;
    };
} | {
    bar: {
        choo: string;
        boo: string;
    };
}>

Additional information about the issue
I tested across a number of versions on the playground, including the oldest and nightly
It looks like playing with the awaits can resolve this issue, but that doesn't seem to be ideal.

It's working as intended and called "subtype reduction". There are plenty of issues about it.

Seems not easy to find if you don't know the name for it. Doesn't seem like a great solution honestly, but if this is known I'll close this ticket.


Remove Control Spacers from editor and dialogs

Godot version
4.x
System information
Windows 10
Issue description
Control Spacers are used in the editor instead of using Control size flags and BoxContainer's separation custom constant, you can find more related info here https://github.com/godotengine/godot-proposals/discussions/7444#discussioncomment-6653563
Steps to reproduce
N/A
Minimal reproduction project
N/A

@AThousandShips I have spent 2 years using godot Control nodes most of the time and have found many hacks for the UI, I'm sorry if this makes you feel annoyed but this should be fixed soon and will give the editor a nice boost ^^, Godot is the best Engine for me and I will do all I can to help it even with small commit like this one.

I have no idea why you are pinging me over a "hmm" reaction... Nowhere have I expressed any annoyance, please don't project your assumptions onto me

I have no idea why you are pinging me over a "hmm" reaction... Nowhere have I expressed any annoyance, please don't project your assumptions onto me

It looks like a sad face for me, sorry am not expert with emoji meanings.

I am simply unconvinced by this providing a significant performance boost, and concerned by the other unforseen impacts of this, and would like to see some independent verification of the performance boost

the performance boost will happen mainly to ram, simply because the Engine is using lot's of Dialogs and it adds 2 spacers arroung the first Dialog button spacer - button - spacer and if you add an extra button it will add one more spacer spacer - button - spacer - button - spacer and so on, so if we have like 60 dialogs exists in editor surely they exist in memory too, and the sum of the useless buttons inside the memore will be minimum of 120 Control nodes which exists only to offset the button, and each control node has nearly 3500 lines of code and when they are visible they will have an effect on GPU and CPU, the other files which was changed may not make much changes yes but they also save some lines in the script makes it easier to read and the fact that those removed lines can be replaced with just one or two lines seems good.

It looks like a sad face for me, sorry am not expert with emoji meanings.

Well just hover over it and you'll see it says "confused emoji", but the point stands, you're projecting assumptions of intent and opinion onto me and I ask that you stop that, it's inappropriate
And as I said, I would like to see some confirmation from someone else that this improves performance in a significant enough way, and that it doesn't change the appearence and usability of the interface
The amount of code in a control has zero relevance to their memory usage, an individual control takes up relatively little memory, though it is not zero

Well just hover over it and you'll see it says "confused emoji", but the point stands, you're projecting assumptions of intent and opinion onto me and I ask that you stop that, it's inappropriate

I'm sorry about that.

And as I said, I would like to see some confirmation from someone else that this improves performance in a significant enough way, and that it doesn't change the appearence and usability of the interface

there will be tests before merging and someone surely will compare the performance and startup time of the editor or i can do it but will require some more search about how to correctly compare both execs.

The amount of code in a control has zero relevance to their memory usage, an individual control takes up relatively little memory, though it is not zero

the comparison here says something about it, and it was done before and after removing the spacers, so i already have tested it and if it was useless i may have not spent some hours on this fix

I think it's pretty obvious that almost 400 MB of ram cannot be taken up by 120 spacers when there's thousands of GUI elements across the editor, so some more rigorous testing is needed as something else differs here
I have not said or implied that any of your work is useless

I think it's pretty obvious that almost 400 MB of ram cannot be taken up by 120 spacers when there's thousands of GUI elements across the editor, so some more rigorous testing is needed as something else differs here

nothing was changed in the source code except the current changes in the fix commit and github action already compiled the code and you can download the editor and check the differences, I was running the same project with both execs and nothing else have been changed in both. and also i have said if 60 dialogs exists then a minimum of 120 spacers will be available but if each dialog have 2 buttons then it will be 180 Control nodes, and this is not a real number of the actual control spacers count but maybe you can tell the editor node count in the current artifact to know how many Control nodes was removed.

But you do understand that it's completely unreasonable to think that 80% of the process ram is taken up by spacers right? And that some rigorous, repeatable testing needs to be done, and that this throws some doubt on the validity of the assumptions of improvements here, right?

But you do understand that it's completely unreasonable to think that 80% of the process ram is taken up by spacers right? And that some rigorous, repeatable testing needs to be done, and that this throws some doubt on the validity of the assumptions of improvements here, right?

Actually if something else was changed in the source code between both versions i may have not said that, but the difference between the two comparison was only the remove of the spacers.

And that if removing spacers provide a significant performance boost then there's probably something wrong in the spacer code that should be fixed instead

maybe this ? it will already be merged with this fix.

I'm not saying that the source is different, I'm saying that the testing conditions do not match, or are you really claiming that spacers take up four times the memory of everything else in the editor?

Many modern UI frameworks do not use spacers anymore, so not using them in Godot will follow this direction.
While nothing visible is rendered (or maybe there is?), it is still an additional node in the tree and it still needs to be calculcated in everywhere.
And in pretty much every UI framework I used (including Godot), you can achieve the same by using the correct flags and properties.

In HTML, many years ago the <spacer> tag was completely removed for this reason. It is recommended to use margin, padding or gap instead (in conjunction with flexbox or grid). In HTML this is even more critical as you want to keep your HTML file small and rendering time as low as possible
JavaFX is pretty similar here, you always use margin, padding and spacing

And Godot also does provide some properties to control the layout e.g. a spacing between elements (e.g. BoxContainer) or size flags. There is also the content margin.
So why I don't think it will improve the performance that much, it is still a good, modern approach to not render invisible objects as spacers and use the different properties instead.

we can achieve all of those styles without the spacers.

we can achieve all of those styles without the spacers

Note that this is only true for two children. If the container has 3 or more children of different sizes, then the same behavior as with spacers cannot be achieved using size flags, as far as I know.

we can achieve all of those styles without the spacers

Note that this is only true for two children. If the container has 3 or more children of different sizes (and use 2 or more spacers between them), then the same behavior as with spacers cannot be achieved using size flags, as far as I know.

this is wrong, for 2 years I use containers and control nodes and never have used any spacers for UI layout! don't underestimate the Godot Control Nodes! they are the reason why i can't switch to any other engine :D
https://github.com/godotengine/godot/assets/53877170/46b02c98-63c0-4304-83d5-430a5d7c3157

@WhalesState I mean the following layout:
Child elements of fixed different size and responsive spacers of equal size between them.

As far as I know, with size flags you can't make equal spacing between buttons.

@WhalesState I mean the following layout:
Child elements of fixed different size and responsive spacers of equal size between them.

As far as I know, with size flags you can't make equal spacing between buttons.

let's assume that someone will create a button called Press me so i can confirm! and another one called "cancel" then we should loop over all the buttons and give them set their minimum width to the maximum width of the largest button. all programs do that already because having such small child and another bigger one then a smaller one makes the UI looks awful and to change this behavior will require us to extend the Dialog class and access it's hidden children and change them.

but actually in actual real scenarios we will create buttons like this ok, cancel, confirm, exit, close, with respect to some other languages that may have longer width for each of those names after translating, then we should first define the minimum width of each button as this max(96, maximum_width) where maximum width is updated when about to popup or translation changes and it loops over all children to just update the maximum width if it's greater than 96 or the current maximum width! a for loop for 2 or 3 buttons then a beautiful UI as a result.

In any case, I gave an example where the size flags are not appropriate and require spacers or a custom container. I don't remember the details, but I've definitely run into this limitation before and a spacer was the right solution for me.
This is not directly related to this issue, I was just responding to your suggestion to deprecate the method (in the initial discussion) and the implied statement that spacers are not needed and can always be replaced with size flags.
As for the issue and PR, I'm not against these changes in general, as long as the behavior is really equivalent to the old one. But I doubt these changes really improve performance and/or reduce RAM consumption that much. Your measurements look questionable to me too, I think that the reason is most likely something else.
If performance gain is not confirmed, then PR should be considered minor style changes. Relatively harmless, but in any case requiring review, testing and discussion. In my opinion spacers are a little more clear than size flags, especially when reading code. At least, it's easier for me to imagine it this way.

i will keep working on this in my free time, and will also make sure to provide many style options for the Dialog class based on all the current used styles in modern UI and popular programs, then after finishing this the BoxContainer.add_spacer() will not be used in editor and you can ask on twitter if anyone is using it then we can deprecate because it's replacement is just 3 lines of code when needed again and I have seen many cases in the UI that creates the spacers manually which will give better control over the spacer. The spacer is useful for sure but for the editor it creates lots of control nodes and I will make sure to make a detailed comparison after fixing this issue. Thank you ^^

having such small child and another bigger one then a smaller one makes the UI looks awful

The key thing is that the elements are different sizes (the absolute difference doesn't have to be big, they can be slightly different sizes), and you want to place the elements with equal padding between them. This cannot be achieved with size flags, in general.

The spacer is useful for sure but for the editor it creates lots of control nodes

Buttons were an unfortunate example. Imagine hyperlinks, icons, or some other element without a background, so you don't want to stretch them.

The key thing is that the elements are different sizes (the absolute difference doesn't have to be big, they can be slightly different sizes), and you want to place the elements with equal padding between them. This cannot be achieved with size flags, in general.

but devs main task is problem solving and finding all possible solutions, that's why windows for example moved the HboxContainer of the buttons to use (EXPAND | SHRINK_END) and others makes the width of all buttons the same to achieve beautiful look for the UI, the issue here is not related to this task but if you align buttons horizontally while each of them has a different size they will look bad even if they are padded equally! we should make the size equal for Dialogs
it seems like only Godot is using different button sizes and i already have this issue of the bad look of the dialogs since godot 3.x

I just wanted to mention that we definitely misuse spacers in places and whenever they can be replaced with a theme definition, they should. But the feature itself is useful, is sometimes required, and is definitely not a source of any performance issue. So a PR that simplifies appropriate cases would be appreciated and welcomed. But not as a solution to a performance problem, which, as outlined before, should be tested more rigorously.
You need to actually benchmark the code and see what takes RAM and what takes time. The reporting done by the operating system in the task manager is not a good source and is superficial at best.

I appreciate your understanding and commitment to following advice. It's important to prioritize clean code over immediate performance concerns. We want to avoid any potential mistakes or shortcomings in the UI, such as lacking knowledge of Container's size flags or Hbox separation. Whenever you point out issues, I will promptly address them without argument. I value the opportunity to learn from you and have grown fond of working with Godot. My aim is to assist with smaller tasks, allowing you to focus on more complex ones. Once I complete my work, we can discuss the new changes and find better ways to write code, rather than engaging in unproductive arguments or premature performance tests. It's crucial to measure performance before merging the code, rather than during unfinished work. Thank you all for your support, and I apologize if I make any mistakes or say anything incorrect.


Cosmetic fixes to examples

Fixes https://github.com/scikit-optimize/scikit-optimize/issues/253

Current coverage is 84.97% (diff: 100%)

Merging #258 into master will not change coverage

@@             master       #258   diff @@
==========================================
  Files            25         25
  Lines          1331       1331
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1131       1131
  Misses          200        200
  Partials          0          0

Powered by Codecov. Last update 01aafdb...6f707a8

Can't see the diff in the browser but it looks like all you changed is the figure size. If the answer is yes, feel free to merge.

Thanks but however the output from plot_evaluations and plot_objective still does not scale to a single page because it seems that the functions return an array of axes?

They set the figure size "by hand". The idea was that you want a minimum size for each square in the grid otherwise it becomes impossible to look at.


Adding new content for learning web skills

Hey there ,
Adding new content for learning web skills.

Hey @RitikPatni ,
I added  a good resource for learning a web skill..
please check, if you like it then please merge it.

Thank you
On Sun, Oct 3, 2021, 9:04 PM Ritik Patni @.***> wrote:

Merged #152
https://github.com/RitikPatni/Front-End-Web-Development-Resources/pull/152
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RitikPatni/Front-End-Web-Development-Resources/pull/152#event-5399968324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATFAIBGSAWURMEU7XFGDANTUFBZZBANCNFSM5FHWBOKA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Fix master 2

Change Summary
Proposal for https://github.com/samuelcolvin/pydantic/pull/1682

Related issue number

Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

thank you.


Incorrect transform in preprocessing.py

With transform in preprocessing.py

With correct transform

Fix
https://github.com/PetroWu/AutoPortraitMatting/pull/32
Discussion
The bottom center image of each example is the mean mask aligned with the given portrait. They're intended to be rough estimates of what the final mask should look like. The misalignment in the first example is due to using an incorrect transform.

Hi，I am doing my homework, I have some questions about it. Hope to communicate.

@MLjian What's your question?

Did you run the example? How is the result?

I'm stuck at 90% IoU on a validation set, while the authors got 96% on the test set. This is with my custom code, though.
I haven't ran this repo's code. I only looked at the code to compare my implementation of the paper with PetruWu's implementation. I noticed an error, and so I posted here.

You are so passionate. Thank you very much.I saw the paper Automatic  Portrait Segmentation for Image Stylization carefully. And what's the architecture in deep learning for implement? Tensorflow or other.

@MLjian
The paper uses a custom Caffe build from 2014. I'm using PyTorch.

Hi,guy!
The datasets folder are so complex for me. Now I want to just use the FCN without additinal channels to complete the matting. So I want to know whether the Images_mask folder is the annotation that the paper author labeled.


nanoflann: 1.5.5 -> 1.6.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/nanoflann/versions.
meta.description for nanoflann is: Header only C++ library for approximate nearest neighbor search
meta.homepage for nanoflann is: https://github.com/jlblancoc/nanoflann
meta.changelog for nanoflann is: https://github.com/jlblancoc/nanoflann/blob/v1.6.0/CHANGELOG.md
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.6.0 in filename of file in /nix/store/hskkxp4jmjwdmznl50vd2ligymlg89y5-nanoflann-1.6.0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

nanoflann
opensplat
opensplatWithCuda

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hskkxp4jmjwdmznl50vd2ligymlg89y5-nanoflann-1.6.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nanoflann https://github.com/r-ryantm/nixpkgs/archive/9cd16b5e0143adcef1e8f710fcda4ffecf1ddd3d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9cd16b5e0143adcef1e8f710fcda4ffecf1ddd3d#nanoflann

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hskkxp4jmjwdmznl50vd2ligymlg89y5-nanoflann-1.6.0
ls -la /nix/store/hskkxp4jmjwdmznl50vd2ligymlg89y5-nanoflann-1.6.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 120m and timed out

Maintainer pings
cc @AndersonTorres for testing.

@NixOS/nixpkgs-merge-bot merge


Vanity - keyword in starting

Vanity Addresses normally have desired string in the starting. Keyword in middle doesn't mean much to the user.
Now, desired keyword will start from second place only. However, this may increase time required to generate the vanity address. But it will be worth it. ;D
Please see https://docs.ark.io/docs/contributing for details before opening your pull request.

@kalgoop conflicts


branch name on push refers as ref-heads-main

what do we wanna do here?
trim the name ? if we have refs-heads-main then print main?

yes if main then trim refs-heads! would be good,


Compiler errors on missing files in uavcan/protocol directory

Hi.  Really new to this and managed to get it into a format for use in Teensy 3.5 sketches using the Arduino IDE.  But as I am going through it to see what has to change if anything I was getting a bunch of errors on missing files used in the include statement's.  All the errors were in the protocol directory:

GetDataTypeInfo..hpp not found - referenced in data_types_info_provider.hpp
There if no "file" directory and associated files referenced in file_server.hpp
GlobalTimeSync.hpp not found - referenced in global_time_sync_slave.hpp
No "debug" directory
GetNodeInfo.hpp not found ref. in node_info_retreiver.hpp
Node_Stats.hpp not found - ref in node_stats_monitor.hpp
Panic.hpp not found - panic_broadcaster.hpp
8 No "param" directory of associated files
no Restart_Node.hpp - ref in restart_request_server.hpp
No Get_Transport_stats.hpp - ref in transport_stats_provider.hpp.

Any help would be appreciated.
Thanks
Mike

The missing files are supposed to be auto-generated by the DSDL compiler. An example of its invocation can be found in the CMake files or here https://github.com/UAVCAN/libuavcan_stm32#building or just run libuavcan_dsdlc --help

Yep.  They were generated.  Just didn't realize they were autogenerated.
Thanks


New version

Please make a new release :)

Certainly.  Stay tuned.


Azure OpenAI API not supported for embeddings

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Linux X64 6.8.0
- Continue: 0.8.40
- IDE: 1.90.1

Description
Currently, the OpenAIEmbeddingProvider does not support Azure OpenAI Endpoints.
To reproduce

Open the configuration file.
Add the Azure OpenAI Endpoint to the configuration.
Attempt to update the vector database.
Observe the following error message: Error updating the vectordb::text-embedding-3-large index: Error: {"error":{"code":"404","message": "Resource not found"}}

Log output
Error updating the vectordb::text-embedding-3-large index: Error: {"error":{"code":"404","message": "Resource not found"}}

I'm also encountering this issue. The Azure tutorial (shown below) in the docs does not work and swapping the provider to OpenAI as a hail-mary does not work. "embeddingsProvider": [{     "provider": "azure",     "model": "text-embedding-3-large",     "apiBase": "<YOUR_DEPLOYMENT_BASE>",     "engine": "<YOUR_ENGINE>",     "apiVersion": "<YOUR_API_VERSION>",     "apiType": "azure",     "apiKey": "<MY_API_KEY>" }]


added  socket 4/5 support!

I really want to improve the options exactly for this we're missing a bunch and it's not so great to edit so many files just to add one.
@cazala You can merge this if you agree with the changes (I do) but we should wait a couple of days before issuing a new release (because at this rythm binary arguments will be a mess :P).

@soyuka Good observation. I suggest a good parser https://github.com/substack/minimist

Exactly, also I want to refactor the constructor arguments to an object see #7.
Just need time to do so and dunno if I'll do it tonight haha.

Hi @tailot, after merging #11 this has conflicts 👇

H @cazala, done!


Composer installs datatables in the wrong dir.

In my local environment, this problem does not occur. On a webserver I get this:
ReflectionException in Container.php line 736:
Class datatables does not exist
I did update composer and checked /vendor/ - DT seems to be installed here:
vendor/yajra/laravel-datatables-oracle/src/yajra/Datatables/Datatables.php
is this path correct?
This is how the DT is called: return yajra\Datatables\Datatables::of($selection) - why does it not find the Datatables class on the web server and does find it locally?
The installation of laravel on the server is ~/laravel/ and the public files are in ~/public_html/ (the paths in autoload.php are adjusted)
Any idea?

i have same problem.. how did you solve it? thanks!

i have same problem But how can i solve that?

I found the solution after hours of googling and try different ways :
1.first rename my project to a new name.
2.use composer update
3.run php artisan config:cache
4.and run php artisan cache:clear
and try your Code again.

I overlook the line below; after this everything works well..
php artisan vendor:publish


Add Network Switcher component.

Overview
Add a reusable Network Switcher UI component (components/NetworkSwitcher.tsx), along with a Network Context Provider (contexts/NetworkContextProvider.tsx).
Features

The Network Switcher component is a selector currently added to the AppBar component, under the settings dropdown.
Changes to the network are propogated site-wide without need for a reload, and are saved to localStorage for later retrieval.
Notifications after performing a transaction link to the correct cluster on Solana Explorer.

Notes:
localhost is yet to be added as a network option.

Related documentation
https://user-images.githubusercontent.com/14112766/174336976-b33f226a-7de5-4e60-8c3e-cb49a61b2261.mp4

noted network switching logs twice in the console - > could be fixed.


Bug in this documentation?

"Possible values" section says: "If th Account lockout duration is set to 0, the account will remain locked until an administrator unlocks it manually.
It is advisable to set Account lockout duration to approximately 30 minutes. To specify that the account will never be locked out, set the value to 0"
So which is it? Setting 0 means it's locked forever, or never locked? These are very different things..

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

ID: 403fdccd-6988-3a5d-d175-c8f30ffddf31
Version Independent ID: 5fd127aa-d224-559a-c7ad-00a1257397b7
Content: Account lockout duration (Windows 10)
Content Source: windows/security/threat-protection/security-policy-settings/account-lockout-duration.md
Service: unspecified
GitHub Login: @brianlic-msft
Microsoft Alias: justinha

@Justinha -- Can you take a look at this issue?

@untilted-document thanks for opening this. That should say set Account Lockout Threshold to 0 if you don't want any account to get locked out. Also, our security baselines recommend 15 mins for the duration. I submitted both corrections. They should appear on 04/02/18.


Are Encounters / Tokens added to maps as well?

I did not find much information except for "Scenes are converted to maps with an encounter for assinged tokens.".
For the module I exported (adding the -c flag) there are no tokens on the map.
Is this something that's not yet implemented or did I do something wrong here?

If there were tokens on the map to be converted, they would be made into an encounter under the map. See the attached photo.

Support was added in v1.8.1


Issue with track subscription on iOS

Hi
The following error is logged while joining
unable to subscribe because MediaStreamTrack is ended. Do not call MediaStreamTrack.stop()
The video view is black but can hear audio from the remote track.
client-sdk-react-native -> latest
react-native-webrtc -> 106.0.0-beta.5
iOS -> 16.1
react-native -> 0.69.5
rolling back to react-native-webrtc to https://github.com/livekit/react-native-webrtc.git#dl/wip-transceiver worked, no longer gave any errors, and was able to subscribe to tracks.
Tried using a couple of other beta versions of react-native-webrtc, the error did not appear but the issue persists.
Thank you for the great work.

@gladbhex in your app, do you call MediaStreamTrack.stop() at any point?

No, not calling this anywhere.

Thanks, we'll investigate.

@davidliu the issue is here, where it's returning Live when it's checking for lowercase live here

@davidzhao The line before that, they turn it to lowercase, so I don't think that's the cause.

Related: https://github.com/react-native-webrtc/react-native-webrtc/issues/1219

Fixed as of react-native-webrtc 106.0.0-beta-6


Clear error messages if required environment variables aren't set

As a first pass only checking environment variables that use non-null assertions.
Later should check all environment variables or use json configs instead.
Libraries worth considering:
https://www.npmtrends.com/config-vs-convict-vs-envalid-vs-nconf


"this is an associated function, not a method" error with RwLockUpgradableReadGuard::upgrade

Seeing a compilation error of "this is an associated function, not a method"  when trying to call upgrade() on a RwLockUpgradableReadGuard. I've narrowed this down to parking_lot version 0.6.0; version 0.5.5 compiles the below files with no error (just irrelevant warnings about unused vars).
Was this breakage intentional? If so, what's the ergonomic way of calling upgrade() now? Frankly I have no idea how to call this method with >=0.6.0; even the help note from rustc isn't working out for me (maybe I'm dumb though).
Repro rust source file:
extern crate parking_lot;

use parking_lot::RwLock;

fn main() {
    let lock = RwLock::new(());
    let guard1 = lock.upgradable_read();
    let x = guard1.upgrade();
}

Cargo.toml:
[package]
name = "rusty-playground"
version = "0.1.0"

[dependencies]
parking_lot = "=0.6.3"

Compilation error:
$ cargo build
   Compiling rusty-playground v0.1.0 (file:///home/valloric/repos/rusty-playground)
error[E0599]: no method named `upgrade` found for type `lock_api::rwlock::RwLockUpgradableReadGuard<'_, parking_lot::raw_rwlock::ParkingLotRwLock, ()>` in the current scope
 --> src/main.rs:8:20
  |
8 |     let x = guard1.upgrade();
  |             -------^^^^^^^
  |             |      |
  |             |      this is an associated function, not a method
  |             help: use associated function syntax instead: `lock_api::rwlock::RwLockUpgradableReadGuard<'_, parking_lot::raw_rwlock::ParkingLotRwLock, ()>::upgrade`
  |
  = note: found the following associated functions; to be used as methods, functions must have a `self` parameter
  = note: the candidate is defined in an impl for the type `lock_api::rwlock::RwLockUpgradableReadGuard<'_, _, _>`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0599`.
error: Could not compile `rusty-playground`.

To learn more, run the command again with --verbose.

OK, I figured out how to call upgrade():
extern crate parking_lot;
extern crate lock_api;

use parking_lot::RwLock;
use lock_api::RwLockUpgradableReadGuard;

fn main() {
    let lock = RwLock::new(());
    let guard1 = lock.upgradable_read();
    let x = RwLockUpgradableReadGuard::upgrade(guard1);
}

[package]
name = "rusty-playground"
version = "0.1.0"

[dependencies]
parking_lot = "0.6.3"
lock_api = "0.1.3"

This required an explicit dependency on lock_api and another use import. It's also very verbose.
Please tell me this isn't the intended way of using the API. :) The whole lock_api split is IMO hugely problematic because just navigating the docs is extremely difficult now ("which crate has the info I need?"); there's also an explosion of types with similar names. Knowing where to find the right method in the docs is very painful... but if this now also means that what used to be a simple method call requires this much ceremony, it's become rather difficult to swallow.
This is one of the most impressive crates in the Rust ecosystem but it's getting harder and harder to use. :(

You don't need to import the lock_api crate, you can use the re-exports from parking_lot (use parking_lot::RwLockUpgradableReadGuard;).
The reason this was changed to an associated method is to avoid conflicts with methods from Deref. This is actually specified in the Rust API Guidelines.
As for the documentation issue, I don't know of a good solution for this at moment.

Yeah, I figured out that use parking_lot::RwLockUpgradableReadGuard is sufficient. Could this be mentioned in the docs somehow? It's not at all obvious how this method should be called; the rustc error is very confusing.

The reason this was changed to an associated method is to avoid conflicts with methods from Deref. This is actually specified in the Rust API Guidelines.

The kind people on #rust pointed this out to me as well. I see why it's being done, I understand that it's necessary, but it just sucks in terms of UX.

As for the documentation issue, I don't know of a good solution for this at moment.

This is the biggest issue IMO. I'll close this issue because it's not related to that, but you may want to track that separately. It's a substantial problem.
Thanks again for being open to feedback! :)


Start to bring the parser up-to-date with Ruby 3.2

Every commit in this PR corresponds to a commit on the whitequark parser for Ruby 3.2. It will probably be easiest to review this PR by going commit-by-commit and comparing to the corresponding whitequark commit, which I've linked in the commit messages.
Motivation
The primary motivation of this PR is to group together a bunch of small changes/refactors to the parser that reflect the current state of the whitequark parser. A couple of larger changes will be created in separate PRs.
Test plan
New whitequark parser tests have been imported using the tools/scripts/import_whitequark.sh script.
Note
There are two sets of changes that I have not brought over in this PR:

whitequark/parser@820cc04b8d176c6f46ccc9a010c55f6113f4b2ad or whitequark/parser@1d275fc21227f538a2dda74a54939cd55b1c9200 -- there are some aspects of translating them to the Sorbet parser grammar that I don't understand yet, but I'll try to address them in a future PR.

Forwarding anonymous rest and kwrest args -- this requires changes in the Desugarer and probably in the resolver, so I will make those changes separately.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_Na1r7Sx11oUYKG
→ https://go/builds/bui_Na1rFatvZJBoNC

Is the intention that attempting to use these new node types will generate an "Unsupported node type" error from desugar at the moment?

Nope :) I assumed any issues further down the pipeline would be captured in the tests but obviously the parser tests just test the parsing. 🤦🏻♀  I'll look into it!

@jez I modified this PR to only include the changes outside of forwarding anonymous rest args. I'll open another PR with the forwarded rest arg work, including Desugaring changes.


feat: add global navigation loader

verified on devnet
navigation never fails even when it is slow 3g speed
Screencast from 12-21-2023 11:48:20 AM.webm

Verified:

Devnet
000eea2


Switch over to using six.PY{2,3} when possible

Summary of changes
Per the reasoning in #1415, I think that we should use six.PY{2,3} whenever possible and outsource how to determine the major version to them.
The one concession I made to #1413 is that I changed one sys.version_info.major to sys.version_info[0] in the version logic itself, because that is more consistent with how six does it. I'm not saying that we guaranteed that we'll never use sys.version_info.major in the future, just that this is another equally good way to check the value that doesn't trigger the Jython bug.
Closes #1415
Pull Request Checklist

Changes have tests N/A
[x] News fragment added in changelog.d. See documentation for details

@jaraco Any objections? I know you didn't want to explicitly handle the Jython case.

I'm going to merge this.


Module release upcoming?

I notice the current version of OVF published to PowershellGallery is 1.0.1, however there have been two significant branches merged in to master with no corresponding releases, is there any plan to publish these changes soon?
Most notably I'm finding OVF is pretty unusable with Pester 4.1.1 due to the OVF -IncludePesterOutput parameter not supporting the Pester deprecated use of its -Quiet parameter, this fix is included in master and it's a shame it can't be easily consumed by people already using OVF.
I'm happy to help set-up a CI pipeline if required.

@JamesWTruher The current version in master should be pushed to PSGallery. Can you do this or add me as an owner in PSGallery so I can push it?

Thanks @JamesWTruher! Here is the doc on how to add an owner in PSGallery.
https://docs.microsoft.com/en-us/powershell/gallery/psgallery/managing-item-owners

@devblackops hopefully I did this right - you should have an email if I did, otherwise, you'll need to send me your username for the gallery

Got it @JamesWTruher. Thanks!

@lholman Just published v1.1.0. I'll be making a few changes soon to add support for macOS/Linux as well.
https://www.powershellgallery.com/packages/OperationValidation/1.1.0

Just checked back to see if this had been done and happy to see it has :) thanks all.


Refreshing Edge with checkboxes ignores v-model

Version
2.5.21
Reproduction link
https://jsfiddle.net/andrewlongmore/50wL7mdz/821270/
Steps to reproduce
Since you need to refresh in Edge, I suggest you create a .html and .js file from the code in https://jsfiddle.net/andrewlongmore/50wL7mdz/821270/.
What is expected?
There are three check-boxes, with text "first", "second", "third".  The first two check-boxes are checked, the third one isn't.
The paragraph text shows that the selectedValues array is ["first","second"].
Refreshing the page doesn't change anything.
What is actually happening?
On FireFox, this is what happens.
On Edge, refreshing the page causes the first two check-boxes to become unchecked.  This happens whether you do F5 or ctrl-F5.  Refreshing the page again causes the first two check-boxes to become checked again.  The paragraph text always shows that the selectedValues array is ["first","second"].

Checked against latest Edge on Windows and OSX and cannot reproduce. It was probably a bug on Edge that got fixed

@posva please re-open.  I have just checked against latest Edge on Windows and the issue is still there.  To be clear, download test.zip attached above, unzip, and open test.html.  If you refresh the page in Edge, the first and second checkboxes become unchecked.  This should not happen.

You are probably on an old version of Edge. The bug was present on Edge 15 but it's not on the Insiders build. As I said, it was probably a bug on Edged that got fixed

I am on the May 2019 Windows update, Microsoft Edge 44.18362.1.0, Microsoft EdgeHTML 18.18362, which is the current stable version, and the bug is still present.  However, I can imagine that the bug is fixed in the upcoming Chromium-based release of Edge.


Add support for pg12

This PR adds support for pg12

Looks good and the Molecule tests were successful on both CentOS and Ubuntu. Are you proposing to tag this as 5.0.1 as the logic is effectively in-place in the role already and this is only  testing + documenting the new PSQL version?
Also before merging and tagging, should we handle https://github.com/ome/ansible-role-postgresql/pull/19#issuecomment-622948376 ?


To fix this, just rerun assh config build > ~/.ssh/config, but after you connect enough hosts, the problem will occur again.

Hi @wd, thank you for your feedback,
Can you give me the related part of your assh.yml (or included) file(s) ?

$ cat assh.yml
hosts:
  '*.com':
    IdentityFile: ~/.ssh/id_rsa

$ cat config
# This file was automatically generated by assh v2.5.0
# on 2017-01-10 17:16:51 +0800 CST, based on ~/.ssh/assh.yml
#
# more info: https://github.com/moul/advanced-ssh-config

# host-based configuration
Host *.com
  IdentityFile ~/.ssh/id_rsa
  # KnownHosts: [a.com]

Host a.com
  IdentityFile ~/.ssh/id_rsa
  # KnownHostOf: *.com

# global configuration
Host *
  ProxyCommand assh connect --port=%p %h

When you have lots of host ended with *.com, assh will add lots at line # KnownHosts: [a.com].

Hello,
I can confirm the same, i have a lot of wildcards guessing like :
Host sc*-comp
IdentityFile ~/.ssh/rsa
Port 22
StrictHostKeyChecking no
User root
Inherits: [root-template]
We're talking several thousand hosts matched by this. Also this breaks when trying to use something like ansible on top, because i can't re-run assh build config all the time.
Is there any easy fix, or should we wait for a new release ?

I just pushed a fix for this (#192); can you check that it fixed your problem?

I've tested, it's fixed. I see you split # KnownHosts to several lines. Thanks.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Gmail login not working

Describe the bug
Unable to login via Google when CircuitVerse runs on Gitpod
To Reproduce
Steps to reproduce the behavior:

Go to README.md
Click on Gitpod
Give permission to expose all end points
Go to Sign In
Log in with Google

Expected behavior
Should log in successfully as it's an old account
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome

Additional context
Add any other context about the problem here.
Are you working on this issue? (Yes/No)
For GCI students under difficult

It is not expected to work as Oauth is generally used in production only. We can make it work but this would cause us to leak private keys into docker configuration.


Hide stats when no stats are available

I think the interface would look cleaner if no stats were shown when all stats are empty.

@d0ctr This is done in v1.3.0 and should be available on the store sometime this week and should be on the testing store later today.


Settings Sync should perform a basic cross-platform translation

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.68.1 (Universal)
OS Version: Mac OS 12.4 "Monterey" / Darwin arm64 21.5.0

Settings Sync is an ingenius thing.
However it falls short in one regard: Imagine you're happen to either switch systems (i.e. from Win to Mac) for once, or you happen to have different systems at hand (e.g. different work stations) which differ in their OS.
Sync will fail to properly reuse the carefully defined shortcuts and just use the default.
From a user perspective this is bad. I expected to have all my shortcuts work, which was not the case.
I think there are some different ways to handle this:

Settings Sync could ask to auto "translate" the keyboard shortcuts (so control would become command ⌘, etc.)
There could be a dedicated command: "Please translate the shortcuts as good as you can"
Both above.
User made "translation" settings, where the user can define what e.g. "command" means.
... probably you'll have even better ideas for this.

Of course there must be some safeguard system in place for those circumstances where the shortcuts shall deliberately be different. BUT, and this is a big But, I still think that people out there would rather have their settings actually in sync, and not have to many differences between different OS's.

Please check the setting settingsSync.keybindingsPerPlatform to avoid synchronising keyboard shortcuts across platforms.

this checkbox is ticked for me.

It means keyboard shortcuts are synced per platform. Mac shortcuts are synced only to Mac machines and same for Windows machine.
We do not auto convert keyboard shortcuts say, changing from CTRL to CMD or vice versa.
Please refer to this issue for OS specific keybindings - https://github.com/microsoft/vscode/issues/8962


[BUG] App Runner cannot handle Windows Path properly

(Thanks thibault.pelletier for reporting this issue!)
Describe the bug
We haven't tested the package in the Windows environment (Python on Windows for developing and Running it on Docker).
Building package is working but App runner cannot launch it properly.
Creating the docker image has worked fine but when trying to execute it I run into the following problem :
monai-deploy run simple_app:latest input output
Checking dependencies...
--> Verifying if "docker" is installed...

--> Verifying if "simple_app:latest" is available...

Checking for MAP "simple_app:latest" locally
"simple_app:latest" found.

Reading MONAI App Package manifest...
docker: Error response from daemon: mkdir C:UsersThibaultAppDataLocalTemptmpb8re0le4: Access is denied.
See 'docker run --help'.
ERROR: Failed to fetch MAP manifest.
Execution Aborted

After fixing the particular issue by the following code, there are still remaining issue to handle:
--- a/monai/deploy/runner/utils.py
+++ b/monai/deploy/runner/utils.py
@@ -41,8 +41,7 @@ def run_cmd(cmd: str) -> int:
     Returns:
         output: child process returncode after the command has been executed.
     """
-    args = shlex.split(cmd)
-    proc = subprocess.Popen(args, universal_newlines=True)
+    proc = subprocess.Popen(cmd, universal_newlines=True, shell=True)
     return proc.wait()

After upgrading the package, re packaging the app, and running from windows I get the following error :

Traceback (most recent call last):
  File "/opt/monai/app/app.py", line 53, in <module>
    App(do_run=True)
  File "/home/monai/.local/lib/python3.8/site-packages/monai/deploy/core/application.py", line 129, in init
    self.run(log_level=args.log_level)
  File "/home/monai/.local/lib/python3.8/site-packages/monai/deploy/core/application.py", line 429, in run
    executor_obj.run()
  File "/home/monai/.local/lib/python3.8/site-packages/monai/deploy/core/executors/single_process_executor.py", line 125, in run
    op.compute(op_exec_context.input_context, op_exec_context.output_context, op_exec_context)
  File "/opt/monai/app/sobel_operator.py", line 34, in compute
    data_in = io.imread(input_path)[:, :, :3]  # discard alpha channel if exists
  File "/home/monai/.local/lib/python3.8/site-packages/skimage/io/_io.py", line 48, in imread
    img = call_plugin('imread', fname, plugin=plugin, plugin_args)
  File "/home/monai/.local/lib/python3.8/site-packages/skimage/io/manage_plugins.py", line 207, in call_plugin
    return func(*args, kwargs)
  File "/home/monai/.local/lib/python3.8/site-packages/skimage/io/_plugins/imageio_plugin.py", line 10, in imread
    return np.asarray(imageio_imread(*args, kwargs))
  File "/opt/conda/lib/python3.8/site-packages/imageio/core/functions.py", line 265, in imread
    reader = read(uri, format, "i", kwargs)
  File "/opt/conda/lib/python3.8/site-packages/imageio/core/functions.py", line 172, in get_reader
    request = Request(uri, "r" + mode, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/imageio/core/request.py", line 124, in init
    self._parse_uri(uri)
  File "/opt/conda/lib/python3.8/site-packages/imageio/core/request.py", line 260, in _parse_uri
    raise FileNotFoundError("No such file: '%s'" % fn)
FileNotFoundError: No such file: '/var/monai/\var\monai\input'

From https://discord.com/channels/854377741825998868/874812709210828830/891021849742417982
Even though we do not have a plan to support Windows (non-WSL) yet, it would be nice to handle the issue properly.
Steps/Code to reproduce bug
Both building image and launch the packaged app on Windows (non-WSL).
Expected behavior
Should work without error.
Environment details (please complete the following information)

OS/Platform: Windows 10
Python Version: 3.8
Method of MONAI Deploy App SDK install: pip
SDK Version: 0.1.0

I hope @Thibault-Pelletier is you(thibault.pelletier on Discord)
Trying to push PR to resolve this issue:
https://github.com/Project-MONAI/monai-deploy-app-sdk/pull/153
Currently, the new package is in Test PyPI.
pip install -i https://test.pypi.org/simple/ monai-deploy-app-sdk==0.1.1rc2

Couldn't test on Windows yet (having trouble installing Docker Desktop for Windows).
Please help test it if you are available.
Or, I would resolve Docker installation issue and test on Windows.

Hi @gigony,
Thank you for the update.
I will try the new test version today or tomorrow and let you know how it goes on my end!

Hi @gigony,
I just ran the simple_app from scratch on Windows with the new release candidate and now everything works perfectly!
Thanks!

Thank you @Thibault-Pelletier for the feedback!


Switch to storefront-api, set the vue-storefront-api deprecated

What is the motivation for adding / enhancing this feature?

With the 1.13rc1 we should switch from vue-storefront-api to storefront-api. It's a vue-storefront-api refactored and TypeScripted. 100% compliant with VSF
@ResuBaka what do you think? Will we be able to release the current develop (including packages) till this time? I mean realistically it will be July :)

"100% compliant with VSF" I would currently more say 90% to 99%. When it comes to the api extensions, but when it comes to the external api e.g. Products/Category/... these are 100% compliant with the current vue-storefront-api.
For getting the storefront-api to 1.0.0 we should over all just look what still needs to be done so that is in a state where it is in a state where it is fine to release to every body. So is some basic docs, there are all the APIs internally that developers need to extend it to there needs.

We need to update:

[ ] docs (regarding the installer and all the other sections),
[ ] installer,
[ ] cli,
[ ] put some notes to the vue-storefront-api repo


Vial does not work with keyboard

Describe the bug
I get the following output when I launch Vial 0.6.0 with a keyboard (CTSC40) connected via USB:
I've got a mechanical keyboard that can only be configured via the open source application Vial, which is packaged in NixOS, at the latest version (0.60 in unstable).
I installed it via my configuration.nix, adding it to the package field like I have any other package.
My next stop was to check upstream, which suggests adding a udev rule that is already present in default.nix, but adding it doesn't resolve the issue.
I added the udev rule to configuration.nix with the following:
  # set udev rule for Vial
  services.udev.extraRules = ''
    'KERNEL=="hidraw*", SUBSYSTEM=="hidraw", MODE="0666", TAG+="uaccess", TAG+="udev-acl"'
    '';

However, with and without this custom rule, identical to the one in the package's default.nix, I get the following output:
INFO:root:Matching VID=586A, PID=0040, serial=vial:f64c2b3c, path=b'/dev/hidraw0' - vial serial magic
WARNING:root:is_rawhid: b'/dev/hidraw0' does not match - usage_page=0001 usage=06
INFO:root:Matching VID=586A, PID=0040, serial=vial:f64c2b3c, path=b'/dev/hidraw1' - vial serial magic
WARNING:root:is_rawhid: b'/dev/hidraw1' does not match - open_path error open failed
INFO:root:Matching VID=586A, PID=0040, serial=vial:f64c2b3c, path=b'/dev/hidraw2' - vial serial magic
WARNING:root:is_rawhid: b'/dev/hidraw2' does not match - usage_page=0001 usage=80
INFO:root:Matching VID=586A, PID=0040, serial=vial:f64c2b3c, path=b'/dev/hidraw2' - vial serial magic
WARNING:root:is_rawhid: b'/dev/hidraw2' does not match - usage_page=000C usage=01
INFO:root:Matching VID=586A, PID=0040, serial=vial:f64c2b3c, path=b'/dev/hidraw2' - vial serial magic
WARNING:root:is_rawhid: b'/dev/hidraw2' does not match - usage_page=0001 usage=06
INFO:root:Matching VID=7070, PID=5342, serial=, path=b'/dev/hidraw5' - VIA stack
WARNING:root:is_rawhid: b'/dev/hidraw5' does not match - usage_page=0001 usage=06
INFO:root:Matching VID=7070, PID=5342, serial=, path=b'/dev/hidraw6' - VIA stack
WARNING:root:is_rawhid: b'/dev/hidraw6' does not match - usage_page=0001 usage=80
INFO:root:Matching VID=7070, PID=5342, serial=, path=b'/dev/hidraw6' - VIA stack
WARNING:root:is_rawhid: b'/dev/hidraw6' does not match - usage_page=000C usage=01
INFO:root:Matching VID=7070, PID=5342, serial=, path=b'/dev/hidraw6' - VIA stack
WARNING:root:is_rawhid: b'/dev/hidraw6' does not match - usage_page=0001 usage=06

but am unable to change settings on the keyboard.
Steps To Reproduce
Steps to reproduce the behavior:

Install Vial via configuration.nix
Plug keyboard in
start Vial application

Expected behavior
The keyboard should be recognized by the Vial application, display onscreen, and allow configuration
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
I confirmed it was not the keyboard by testing in Windows and ArchLinux.  I manually applied the udev rule recommended by upstream's documentation as well:
  services.udev.extraRules = ''
    'KERNEL=="hidraw*", SUBSYSTEM=="hidraw", MODE="0666", TAG+="uaccess", TAG+="udev-acl"'
    '';

The same issue occurs in either state.
Notify maintainers

@Kranzes
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
arin@nixdesk:~/Downloads$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.68, NixOS, 22.11 (Raccoon), 22.11pre411457.62228ccc672`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.0`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Oof, there's an egg on my face, but I'm not the only one to ever have this issue so I'll share the solution:
https://get.vial.today/manual/linux-udev.html
{ config, pkg, ... }:
{
services.udev.packages = with pkgs; [
vial
];
}
solved this problem for the vial package.


Oracle - Timeout

Oracle connection takes to long to load and most of the time it raises timeout.
I think it's related with oracle owner with a big dictonary.
I don´t know how the extension load the list of table/objects for the explorer/navigator, but,
if it´s using dba_* or all_* views, i would suggest to use user_* views and ignore public synonyms.

SQLTools Version 0.19.3
VSCode Version: 1.33.1
OS: Windows

hey @mcardia ,
extension uses queries from https://github.com/mtxr/vscodesqltools/blob/master/packages/core/dialect/oracle/queries.ts file to load tables: fetchTables, columns in tables fetchColumns, functions fetchFunctions, etc...

I must take a look on #218.
I think we are having a feel issues because of loading all data after connecting

Not supported within this repository anymore.


Break up strategies_and_params.jl

Move strategies to separate files
Move docstrings before type definitions (fixed in JuliaLang/julia#37048)

Pull Request Test Coverage Report for Build 901548151

105 of 198 (53.03%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 80.698%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/strategies_and_params/reportingstrategy.jl
69
80
86.25%

src/strategies_and_params/memorystrategy.jl
7
24
29.17%

src/strategies_and_params/shiftstrategy.jl
3
68
4.41%

Totals

Change from base Build 901522339:
0.03%

Covered Lines:
2337

Relevant Lines:
2896

💛  - Coveralls


Add instructions to set nightly as the default tool chain

I stumbled a bit when I had the stable tool chain configured. It would have been useful to know that cargo build is supposed to work even if it is not strictly required to do the tutorial. So I thoght it was a good time to make my first contribution. I hope this is helpful.

Thanks @johlrogge 🎉

Actually I tried to testcompile pretty early and just assumed cargo didn't
work. It would have helped me as soon as I started writing rust-code.
On Wed, 30 May 2018, 16:05 Michael Gattozzi<EMAIL_ADDRESS>wrote:

Hey @johlrogge https://github.com/johlrogge thanks for the fix! Is
there a certain point in the tutorial that you ran into this issue? Like
when running cargo test for instance? If so we should move these
instructions as is there because it feels a bit out of place with the setup
instructions. When that happens we can get it merged in :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang-nursery/rust-wasm/pull/183#issuecomment-393174861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADja4AGkNuT7QJyJxXjBKP0qq2RM1-nks5t3qccgaJpZM4UQiM3
.

@mgattozzi Is this fine, do you have any comments on this else we can merge this.

I thought we had previously agreed upon using cargo +nightly build/test/etc so that

We didn't mess with users' global configuration state, and
Regardless of where you start reading and if you skip setup chapters, you know that you need to use nightly.

I think we should continue following this convention, and fix places where we do raw cargo build into cargo +nightly build, rather than setting rustup default nightly.

Perhaps this should be in a troubleshooting section and not in the maintext?

@rustwasm/book team: does anyone have strong opinions on this subject?

We'll want to reopen this PR on rustwasm/book I'd be fine with a troubleshooting section just in case people run into issues. I think they're important enough to cover, but not necessarily something that every user will come across depending on their setup. Sorry for dropping the ball on this PR I've just been having one hell of a week with work since I got back from vacation.

Troubleshooting page looks like a better option. +1 from me.

I also think it would be a better option to collect these kind of fringe isdues somewhere out of the way but easy to find when you need it.
Do you want me to do something? I'm happy to help but pretty new to this stuff so I don't really know what goes where and when to do stuff.

@johlrogge so this got buried in my notifications. Let's do this:

Close this PR
Open up a new one with this content in a new Troubleshooting section

The book is located now at https://github.com/rustwasm/book

No worries.
I will do this during the weekend.
/J
On Tue, 12 Jun 2018, 17:19 Michael Gattozzi<EMAIL_ADDRESS>wrote:

@johlrogge https://github.com/johlrogge so this got buried in my
notifications. Let's do this:

Close this PR
Open up a new one with this content in a new Troubleshooting section

The book is located now at https://github.com/rustwasm/book
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rustwasm/team/pull/183#issuecomment-396628254, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADjaw7aG_y3MEUplq2tFCJfulnstZ3pks5t79v4gaJpZM4UQiM3
.

@adityac8 Nice! I also learned that you can override the defaults for the current directory only. It seems the instructions should be the same for both tutorials or link to a common page from both tutorials?
@mgattozzi Should we maybe link to the setup-page in the book in the same manner as is done with rustup in the previous section. That is, just have the bare minimum "happy path" in the tutorials and link to the main chapters in the book for more details so the reader can easily find them if needed but is not distracted by them if all goes well?
I'm thinking this page: https://rustwasm.github.io/book/setup.html for a more elaborate discussion, or even something like https://rustwasm.github.io/book/troubleshooting.html as a place for more verbose instructions on how to solve common problems.
I'm closing this PR since my first solution is clearly not in the correct place.


ScrollController.animateTo not firing the first time

I'm making the FlutterChat app from Codelabs, doing some modifications in the layout. I'm trying to scroll down automatically when I send a message. I've achieved this using the ScrollController.animateTo in the following way:
_scrollController.animateTo((_messages.length + 1/_messages.length)*_scrollController.position.maxScrollExtent,duration: const Duration(milliseconds: 300),curve: Curves.easeOut);
Where _messages is a listview containing the messages and this function is called on every submit.
The problem is that the first time scrolling is not working. I deduced that is because when I insert the message, scrolling in the listview is still not enabled, so the function is called before.

We can see that the scrolling hasn't took effect. But then, if I write another message:

Scrolling is working perfectly.
Any possible solution to this? Am I missing something?

Is this still an issue?

Hi,
I've tried it and still not working for me

Please add the output of flutter doctor -v
and if possible a minimal reproduction.

Hi. I've just had 0 luck with animating to position in ListView until I've found this issue.
Using Future.delayed with Zero duration fixed the issue but this is non-self-explanatory. Maybe it would be good to add a small comment into the documentation?
Thank you for the Future info. 👍
Regards.

This depends no where you call the code.
If you call it in build() you instruct the scroll view to scroll before the view is rendered and before it is even known if there is overflowing content and how much.

@polmp please post a minimal running reproduction and the output of flutter doctor -v.

Using Future.delayed:

Flutter doctor:

[✓] Flutter (Channel master, v0.8.2-pre.17, on Mac OS X 10.11.6 15G21013, locale es)
• Flutter version 0.8.2-pre.17 at /Users/USER/Documents/Apps/flutter
• Framework revision ed698b8165 (8 days ago), 2018-09-07 16:46:44 -0700
• Engine revision af42b6dc95
• Dart version 2.1.0-dev.1.0.flutter-ccb16f7282

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
• Android SDK at /Users/USER/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 27.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
• All Android licenses accepted.

[!] iOS toolchain - develop for iOS devices
✗ Xcode installation is incomplete; a full installation is necessary for iOS developmen.
Download at: https://developer.apple.com/xcode/download/
Or install Xcode via the App Store.
Once installed, run:
sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
✗ libimobiledevice and ideviceinstaller are not installed. To install, run:
brew install --HEAD libimobiledevice
brew install ideviceinstaller
✗ ios-deploy not installed. To install:
brew install ios-deploy
✗ CocoaPods not installed.
CocoaPods is used to retrieve the iOS platform side's plugin code that responds to your plugin usage on the Dart side.
Without resolving iOS dependencies with CocoaPods, plugins will not work on iOS.
For more info, see https://flutter.io/platform-plugins
To install:
brew install cocoapods
pod setup

[✓] Android Studio (version 3.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 28.0.1
• Dart plugin version 173.4700
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] Connected devices (1 available)
• Android SDK built for x86 • emulator-5554 • android-x86 • Android 6.0 (API 23) (emulator)
! Doctor found issues in 1 category.

I'm having a lot of trouble because sometimes works and sometimes no. I don't have any idea of how to do it properly.

I'm making the FlutterChat app from Codelabs, doing some modifications in the layout. I'm trying to scroll down automatically when I send a message.

The easiest way would be to use reverse: true, then the list does that automatically.


ModuleNotFoundError: No module named 'azure.storage.cloudstorageaccount'

I'm always getting this error when I'm trying to integrate Ansible with Azure:
File "/tmp/ansible_azure_rm_resourcegroup_payload_qkhbne1k/ansible_azure_rm_resourcegroup_payload.zip/ansible_collections/azure/azcollection/plugins/module_utils/azure_rm_common.py", line 249, in
from azure.storage.cloudstorageaccount import CloudStorageAccount
ModuleNotFoundError: No module named 'azure.storage.cloudstorageaccount'
fatal: [localhost]: FAILED! => {
"changed": false,
"invocation": {
"module_args": {
"ad_user": null,
"adfs_authority_url": null,
"api_profile": "latest",
"append_tags": true,
"auth_source": "auto",
"cert_validation_mode": null,
"client_id": null,
"cloud_environment": "AzureCloud",
"force_delete_nonempty": false,
"location": "eastus",
"log_mode": null,
"log_path": null,
"name": "myResourceGroup",
"password": null,
"profile": null,
"secret": null,
"state": "present",
"subscription_id": null,
"tags": null,
"tenant": null
}
},
"msg": "Failed to import the required Python library (ansible[azure] (azure >= 2.0.0)) on vmi576440.contaboserver.net's Python /home/yf/miniconda3/bin/python. Please read the module documentation and install it in the appropriate location. If the required library is installed, but Ansible is using the wrong Python interpreter, please consult the documentation on ansible_python_interpreter"
}
This is my requirements file:
packaging
requests[security]
azure-cli-core==2.0.35
azure-cli-nspkg==3.0.2
azure-common==1.1.11
azure-mgmt-authorization==0.51.1
azure-mgmt-batch==5.0.1
azure-mgmt-cdn==3.0.0
azure-mgmt-compute==4.4.0
azure-mgmt-containerinstance==1.4.0
azure-mgmt-containerregistry==2.0.0
azure-mgmt-containerservice==4.4.0
azure-mgmt-dns==2.1.0
azure-mgmt-keyvault==1.1.0
azure-mgmt-marketplaceordering==0.1.0
azure-mgmt-monitor==0.5.2
azure-mgmt-network==2.3.0
azure-mgmt-nspkg==2.0.0
azure-mgmt-redis==5.0.0
azure-mgmt-resource==2.1.0
azure-mgmt-rdbms==1.4.1
azure-mgmt-servicebus==0.5.3
azure-mgmt-sql==0.10.0
azure-mgmt-storage==3.1.0
azure-mgmt-trafficmanager==0.50.0
azure-mgmt-web==0.41.0
azure-nspkg==2.0.0
azure-storage==0.35.1
msrest==0.6.1
msrestazure==0.5.0
azure-keyvault==1.0.0a1
azure-graphrbac==0.40.0
azure-mgmt-cosmosdb==0.5.2
azure-mgmt-hdinsight==0.1.0
azure-mgmt-devtestlabs==3.0.0
azure-mgmt-loganalytics==0.2.0

same issue and it can repro with a simple scenario running in cloud shell. https://learn.microsoft.com/en-us/azure/developer/ansible/getting-started-cloud-shell?tabs=ansible

I was hitting the same issue. It seems to be because I had the wrong version of azure-storage or azure-storage-common installed.
The Azure collection docs say that you have to install the included pip requirements-azure.txt file:

Install dependencies required by the collection (adjust path to collection if necessary):
pip install -r ~/.ansible/collections/ansible_collections/azure/azcollection/requirements-azure.txt

The latest version at time of writing is 1.13.0, which includes this requirements file: https://github.com/ansible-collections/azure/blob/v1.13.0/requirements-azure.txt
Weirdly azure-storage is the same version as in your requirements.txt, though perhaps the issue is azure-mgmt-storage, which is 19.0.0 instead of the 3.1.0 you are using? Or maybe you aren't using version 1.13.0 of the collection?
For me clearing out all the pip packages in my venv and then installing the ones from the collections requirements-azure.txt fixed the issue.


Documentation needed for node types

It is not clear what the role of each of the nodes are for e.g. master, worker, service.

I would agree with the service node, as that is in some ways particular to kubeinit and how it sets up the provisioning environment, but how far should kubeinit be going to explain basic kubernetes concepts like master and worker nodes or describing all of the differences in the distributions, etc.?  Other than a link to some generic "What is kubernetes?" site what would you expect us to say in the kubeinit docs?

For me it just needs two things:

What the service node is for (and what's installed on it e.g. HAProxy, Apache).
Importance of the node naming convention (this should also be in the inventory file)

Everything is much clearer now that the nodes are named controller/compute.


Making it impossible to call Policy.open() on an already opened policy.

Similar with Policy.close().
Fixes #4488.

I'll hold off on merging until we can chat about the possibility of returning the "removed" future in Policy.close().