Add notice to canjs npm package to use can instead

This has confused some people.

I think @justinbmeyer added a deprecation message when you install canjs.

janimikkonen@RASJani ~/src/mixradio/web/tvapp (upgrade)$ npm install canjs
<EMAIL_ADDRESS>node_modules/canjs
janimikkonen@RASJani ~/src/mixradio/web/tvapp (upgrade)$
Latest canjs does not warn about deprecation.
ALSO do note that<EMAIL_ADDRESS>does contain  dist/cjs  but<EMAIL_ADDRESS>doesn't so switching to can package will break builds for those who use webpack/browserify etc ..

@rasjani are you sure you don't have a caching problem?  I just installed<EMAIL_ADDRESS>and did get a dist/cjs.  Maybe try npm cache clean?

Yeah, it was caching issue..

Added the NPM deprecation warning to the https://www.npmjs.com/package/canjs package.


Upgrade mockito test dependency to be compatible with new Java version

Make unit test compatible with Java 17

force merging as macos build are failing for older virtual images.


Refactor date inputs for DWP check

Users were getting errors when inputting two digit years
Pivotal story https://www.pivotaltracker.com/story/show/91439958
Added tests and handlers to ensure wider type of inputs accepted.

Looks fine :+1:


Show an error when size for type does not correspond to the tag described in a struct tag

I was testing the library using this schema https://github.com/s1na/fastssz/blob/8f8e2c18f1c85e157e7b8d22bde491831c5ebc8d/tests/codetrie.go#L15
Some times showed me this error: [ERR]: failed to encode Hash: []byte expects either ssz-max or ssz-size, and sometimes worked just fine.
The error occurred when the first element to be processed is Hash. When processing the Metadata structure first it was correctly finding the size for Hash by inspecting the tag and when getting to Hash again it detected it already "knew" the size.
In this change, when processing Hash after the structure, it will check if the size of the array type definition corresponds to the one specified in the structure's field tag. When processing Hash type definition first it will show the same error but also indicating the possibility to fix this problem by just adding a length to the type definition.

Hi, Thank you for digging deep on the generator!
The issue is that it is trying to generate the Hash struct which is not an SSZ object itself. Even when it does not fail and tries to generate the functions for the Hash struct, it produces nothing.
The solution should be to use the '--objs' flag to specify the objects to generate. Thus, it will not try to generate Hash and fail. So, I do not think this PR is worth it the complexity it includes.
Another option for the future is to add another flag '--avoid' so that some structs can be avoided if there is no '--objs' flag.

@ferranbt thanks for the reply, I confirm using the --objs avoids the behavior I described in the comment above. I'm closing this PR.

@ferranbt thanks for the reply, I confirm using the --objs avoids the behavior I described in the comment above. I'm closing this PR.


Make the SpanAPI write-only

The Tracing SDK makes it clear that the Tracing API defines a Span as being write-only.
However, the current API definition certainly makes things readable.
The following issues in the SDK will then provider read access:

https://github.com/deno-otel/otel-tracing-sdk/issues/2
https://github.com/deno-otel/otel-tracing-sdk/issues/1

There is one exception:

GetContext


Add Client library tests

basic tests for the domains should be added, e.g. type serialization roundtrips and conversion tests

Type extensions tests have been added. direct client tests are for another day, as the client is basically tested in the consuming ARCValidationPackages library already


How to change employee`s status (en, ru)

Description of the Change

Alternate Designs

Benefits

Possible Drawbacks

Verification Process

Applicable Issues

rename permalinks in front matter in russian versions of the articles


Verification: Sealer supports mixed clusters of arm64 and x86

Issue Description
Type: feature request
Describe what feature you want
Additional context
Add any other context or screenshots about the feature request here.

sealer 0.9.2 is implemented this feature


Roll Flutter Engine from ced8f1a71206 to 73f55dc25674 (1 revision)

https://github.com/flutter/engine/compare/ced8f1a71206...73f55dc25674
2023-09-01<EMAIL_ADDRESS>[Linux] Fix channel buffers control commands error handling (flutter/engine#45056)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add improved probabilities to smirky

We have created options users to input odds for smirky, however there are also probabilities for certain move types built into the code. These need to be further fine tuned for the sampling to be more efficient.
The first priority here will be how frequently entire "types" should be destroyed. The first decision made in sample_types is if an environment should be destroyed which is still happening 50% of the time, which is too frequent considering how many steps it takes to build up a good environment and that intermediate moves can also be reversed, decorators and atoms can be removed.

We've done a lot of updating to odds on input files in SMIRKY for the paper. However, I was thinking about something for future move proposal engines this week.
It would be ideal if odds could be based on training molecules or learned without needing the human intuition input. Something I was imagining was using substructure searches to pick these numbers, as in how many single, double, aromatic bonds are there in this set. Where the set being considered might be all of the molecules or only the bonds that match the SMIRKS pattern currently being changed. You could extend this to any of the options, how many element numbers are included or how many atoms match any of the given decorators.
I'm not suggesting we do any of this for SMIRKY, but as a thought for how to get at least starting odds for the future move proposal engine tool.

In reality I have expanded this idea quite extensively. If you want to see what we're currently working on checkout chemper


Add Astro, Tailwind CSS and Flowbite tutorial

Hey @one-aalam,
Thanks for this awesome resource! I've added a tutorial about installing and configuring Astro with Tailwind CSS and the UI components from the Flowbite Library.
Cheers,
Zoltan

Hey @one-aalam,
Appreciate the merge! Gatsby is indeed awesome :)
Also I've connected with you via Twitter - you're doing some pretty cool stuff.
Cheers,
Zoltan


Update cloud-on-aws-faq.md

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Thanks, @scjane :)


Fix readiness and liveness probes scheme when ssl is enabled

Signed-off-by: Erwan Miran<EMAIL_ADDRESS>What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area falco-chart
What this PR does / why we need it:
When falco.webser.sslEnabled is true, readiness and liveness probes fail as performed with http scheme (default)
Which issue(s) this PR fixes:
None that I know of
Special notes for your reviewer:
I didn't check the "Variables are documented in the README.md" item as no change has been made. Don't know if that was wrong...
Checklist

[x] Chart Version bumped
[ ] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: mirwan
To complete the pull request process, please assign bencer after the PR has been reviewed.
You can assign the PR to them by writing /assign @bencer in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

falco/OWNERS

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

Welcome @mirwan! It looks like this is your first PR to falcosecurity/charts 🎉

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, mirwan
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:

falco/OWNERS [leogr]

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

LGTM label has been added.  Git tree hash: 0d79c5281ea6b2e9c322c354a29e599183512735


Unit test and utility function created.

Creation of a unit test for a new utility function and updating the Makefile, tsconfig.json. Below is a screenshot of the test.

Remove all files from dist folder and add it to gitignore.

/dist removed. Sorry for that.


[Card group] v2 Epic

Epic to track Card group v2
Review notes
### Design
- [ ] https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/10455
- [ ] Website
- [ ] Figma

### Dev
- [ ] https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/10453
- [ ] QA

closing epic in favor of new organization strategy


support Ticket-17055 - fix expiration time

Reduce expiration calculation time from 2 hours to 5 mins.

[x] Node SDK
[x] TypeScript SDK

This is done and released https://github.com/commercetools/commercetools-sdk-typescript/pull/313


0.8 xstyle

Improvements to cross-scope styling.

Support for rules using calc, e.g. calc(var(--a) + var(--b))
performance improvments

support for @extends (https://tabatkins.github.io/specs/css-extend-rule/)

Closing this here - we'll have to rebase this on the latest master and merge into the master branch


Plugin request: add magit-delta support

Describe the feature

If the feature involves new plugins, include links to them
Link to plugin magit-delta.
If the feature involves replacement of current functionality, how the new
functionality is better?
Magit-delta makes it easier to distinguish changes in magit diffs.

System information

((system
  (type . gnu/linux)
  (config . "x86_64-pc-linux-gnu")
  (shell . "/usr/bin/zsh")
  (uname . "Linux 5.8.14-zen1-1-zen #1 ZEN SMP PREEMPT Thu, 08 Oct 2020 00:01:25 +0000 x86_64")
  (path "~/.cargo/bin" "/opt/draw.io" "~/.local/bin" "/other/things/in/path" "/usr/local/bin" "/usr/local/sbin" "/usr/bin" "/usr/lib/jvm/default/bin" "/usr/bin/site_perl" "/usr/bin/vendor_perl" "/usr/bin/core_perl" "/var/lib/snapd/snap/bin" "~/.emacs.d/bin/" "/usr/lib/emacs/28.0.50/x86_64-pc-linux-gnu"))
 (emacs
  (dir . "~/.emacs.d/")
  (version . "28.0.50")
  (build . "Oct 17, 2020")
  (buildopts . "--prefix=/usr --sysconfdir=/etc --libexecdir=/usr/lib --localstatedir=/var --mandir=/usr/share/man --with-gameuser=:games --with-sound=alsa --with-modules --without-gconf --without-gsettings --with-nativecomp --with-x-toolkit=gtk3 --without-xaw3d --without-m17n-flt --with-cairo --without-compress-install 'CFLAGS=-march=x86-64 -mtune=generic -O2 -pipe -fno-plt -g -fuse-ld=gold' CPPFLAGS=-D_FORTIFY_SOURCE=2 LDFLAGS=-Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now")
  (features . "XPM JPEG TIFF GIF PNG RSVG CAIRO SOUND GPM DBUS GLIB NOTIFY INOTIFY ACL GNUTLS LIBXML2 FREETYPE HARFBUZZ LIBOTF ZLIB TOOLKIT_SCROLL_BARS GTK3 X11 XDBE XIM MODULES NATIVE_COMP THREADS LIBSYSTEMD JSON PDUMPER LCMS2")
  (traits batch envvar-file))
 (doom
  (dir . "~/.doom.d/")
  (version . "2.0.9")
  (build . "HEAD -> develop bba035a1a 2020-10-16 23:42:59 -0400")
  (elc-files . 0)
  (modules :completion (company +childframe) (ivy +fuzzy) :ui doom doom-dashboard doom-quit hl-todo indent-guides modeline nav-flash ophints (popup +all +defaults) vc-gutter vi-tilde-fringe workspaces :editor file-templates fold multiple-cursors snippets word-wrap :emacs (dired +icons) electric undo vc :term vterm :checkers syntax (spell +flyspell) grammar :tools (debugger +lsp) docker (eval +overlay) lookup (lsp +peek) magit pdf :os tty :lang (cc +lsp) data emacs-lisp (latex +lsp) (markdown +grip) (org +attach +babel +capture +export) (python +lsp +pyright) (scala +lsp) (sh +lsp) (yaml +lsp) :email (mu4e +gmail) :config (default +bindings +smartparens))
  (packages (drag-stuff :disable t) (mu4e-maildirs-extension) (mu4e-alert) (color-identifiers-mode) (cmake-font-lock) (eldoc-cmake) (gitconfig-mode) (gitignore-mode) (goto-addr) (goto-line-preview) (flycheck-clang-tidy) (whole-line-or-region) (gitlab-ci-mode) (gitlab-ci-mode-flycheck) (beacon))
  (unpin "n/a")
  (elpa "n/a")))

For reference, #3021 was a previous PR for magit-delta.

How does this differ from (setq magit-diff-refine-hunk 'all)?

There's also (setq magit-diff-refine-hunk 'all) if you want more fine grained diff-highlighting.

Thanks for your suggestions! I did not know about PR #3021. Also, I did not know magit had built-in support for that. While some features cannot be in doom's code, it should be easier to discover them. Maybe some configuration options could be suggested in the module's documentation for users to experiment with and check if they are useful?

The main differences between magit-delta and magit-diff-refine-hunk 'all for me are that magit-delta adds syntax highlighting, is a bit less slow (huge diffs can still take a while to fontify), and that the word diffs are a bit better and easier to read.

From the little time I have used delta, I can agree with you about the word diffs and that the text is easier to read.

This is a duplicate of #3021.


Create function to construct @odefun from DynaSim model

Create DynaSim2Odefun()
Use @odefun with functions for nullcline and bifurcation analysis

This was accomplished using dsDynasim2odefun.m.


The shared folder from macOS not shown in windows 7 virtual machine

BEFORE SUBMITTING YOUR ISSUE, PLEASE LOOK AT THE PINNED ISSUES AND USE THE SEARCH FUNCTION TO MAKE SURE IT IS NOT ALREADY REPORTED. ALWAYS COMMENT ON AN EXISTING ISSUE INSTEAD OF MAKING A NEW ONE.
Describe the issue
I shared a folder in my host macOS 13.1. But I could not find it in my Windows 7 Virtual machine. Where to find it ? In networking?

By the way, I installed the Windows Guest Tool already.
Configuration

UTM Version: 4.0.9 (68)
macOS Version: Ventura 13.1
Mac Chip (Intel, M1, ...):  M1

Here is the settings:

It mentioned to install SPICE daemon. But it didn't mention how to install and where to install?

I even run the map-drive.bat:

Still can't find the shared folder.


Make mocking easier by specifiying virtual on appropriate methods in the library

External references, such as a standards document, or specification?
https://code.google.com/r/iancox-google-api-dotnet-client/source/list

Target platform (e.g. Windows, Mono, Silverlight, WP7, All)?
All

Please describe the feature requested.
Mocking can be tricky at times because quite a lot of the api classes do not
have virtual methods. So far this has affected me in the ResumableUpload and
ClientServiceRequest classes. I am sure there are other places where a
judicious use of virtual will be helpful.

I've made changes in my own clone (see above) so you can see what I mean.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 24 Sep 2013 at 2:33

No activity here, so closing. Pull request with suggested changes is welcome though.


NoMethodError: undefined method `consolidate' for Logging:Module

Logging just got upgraded to v2.0.0, in the process they removed logger consolidation. https://github.com/TwP/logging/blob/logging-2.0.0/History.txt
Now I am getting this error:
NoMethodError: undefined method `consolidate' for Logging:Module
~/.rvm/gems/ruby-2.1.4/gems/blazing-0.4.1/lib/blazing/logger.rb:29:in `<module:Logger>'
~/.rvm/gems/ruby-2.1.4/gems/blazing-0.4.1/lib/blazing/logger.rb:4:in `<module:Blazing>'
~/.rvm/gems/ruby-2.1.4/gems/blazing-0.4.1/lib/blazing/logger.rb:3:in `<top (required)>'
~/.rvm/gems/ruby-2.1.4/gems/blazing-0.4.1/lib/blazing.rb:1:in `require'
~/.rvm/gems/ruby-2.1.4/gems/blazing-0.4.1/lib/blazing.rb:1:in `<top (required)>'

It looks like there is already a PR open for this issue. https://github.com/effkay/blazing/pull/84


[fix] Import records could be empty

Summary
An error caught by unit tests in traject.
Test Plan

add a test case
ci

add a test case

Codecov Report

Merging #38 (029f5e3) into base/donaldong/fix_import_recordsc688d4f (579edcd) will not change coverage.
The diff coverage is 100.00%.

@@                            Coverage Diff                            @@
##           base/donaldong/fix_import_recordsc688d4f      #38   +/-   ##
=========================================================================
  Coverage                                     97.10%   97.10%
=========================================================================
  Files                                            32       32
  Lines                                          1933     1933
=========================================================================
  Hits                                           1877     1877
  Misses                                           56       56

Impacted Files
Coverage Δ

lib/redis_memo/memoize_query/invalidation.rb
96.29% <100.00%> (ø)

Continue to review full report at Codecov.

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


homepage main logo

simplify the main logo on the home page so it is a fixed size. It's a good size when it first appears, but then shifts to something else when the screen gets a little wider.

Done


SQL_CALC_FOUND_ROWS and offset / limit usage using Active Record

Hi,
I want to notify a problem if you use a SELECT SQL_CALC_FOUND_ROWS and offset / limit using Active Record.
my_table is a table with more than 50 rows. For example 100
$this->db->select('SQL_CALC_FOUND_ROWS *', false);
$this->db->from('my_table');
$this->db->offset(0);
$this->db->limit(50);
$query = $this->db->get();
$result = $query->result(); //first 50 records
$total = $this->db->query("SELECT FOUND_ROWS() AS count")->row('count');  //return 50! Should be 100
The error is in system/database/drivers/mysql/mysql_driver.php
function _limit($sql, $limit, $offset)
{
if ($offset == 0)
{
$offset = '';
}
else
{
$offset .= ", ";
}
	return $sql."LIMIT ".$offset.$limit;
}

should be:
function _limit($sql, $limit, $offset)
{
return $sql . "LIMIT " . $offset . "," . $limit;
}

There's nothing wrong with that _limit() code and we don't accept non-security reports about CI 2.x anymore anyway.


readme?

Would be nice to have a readme with a brief description of the project and it's features, how to implement and deploy to discord, and how to contribute! :)

I created a basic README with basic instructions, it is pretty simple to run it, take a look to see if that is enough for now.

yes, absolutely! thanks :)


The underscore is not displayed at the BOTTOM of terminal

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

VS Code Version: 1.59.0
Commit: 379476f0e13988d90fab105c5c19e7abc8b1dea8
Date: 2021-08-04T23:13:20.182Z
Electron: 13.1.7
Chrome: 91.0.4472.124
Node.js: 14.16.0
V8: 9.1.269.36-electron.0
OS: Linux x64 5.4.0-70-generic

Steps to Reproduce:

Turn on vscode, open an internal terminal.
Enter and enter... until the cursor reach the bottom line.
Type in the underscore character '_', it will disappear.

As the screenshot shows, the external terminal display fine.

works for me

Does this issue occur when all extensions are disabled?: Yes
VS Code Version: 1.59.0
Commit: 379476f
Date: 2021-08-04T23:13:20.182Z
Electron: 13.1.7
Chrome: 91.0.4472.124
Node.js: 14.16.0
V8: 9.1.269.36-electron.0
OS: Linux x64 5.4.0-70-generic

Environment information is shown above.
I saw your working case, seems like you've tested on Windows, but my problem occurs on Ubuntu OS. Could you debug on Ubuntu too?

@meganrogge

let me know of any special settings you have or how I can reproduce the issue

I saw many issues opened because underscore in terminal is not displayed correctly on Linux. You had closed all issues except https://github.com/microsoft/vscode/issues/123077, saying they are duplicated. Now, you closed that issue, too, saying that you cannot reproduce it anymore.
Many reported that out-of-the-box vscode terminal cannot display underscores correctly (1) on Linux, (2) at the last line. You must try to reproduce this issue on Linux as these issues described so. You showed that you cannot reproduce the same issue on Windows and it is unclear whether you reached the last line in the terminal.
This is how you can reproduce this issue:

Install any Linux distribution that vscode can run. (I recommend Ubuntu Desktop.)
Install vscode and open vscode, open terminal.
Press Etner key until the cursor reaches the last line in the terminal
Type texts with underscores

I can still reproduce the same rendering error on Debian 10 using out-of-the-box vscode 1.59.0.

So this actually appears to be a separate issue than https://github.com/microsoft/vscode/issues/123077, you will notice that in your font the underscore is abnormally low which causes it to get cut off since it's below the cell. This image shows the dimensions of what the cells would be, since the underscore appears below the cell, for the last row it ends up below the terminal.

The bottom 2 pixels are padding that separates the terminal canvas from the status bar.
It's also worth mentioning that https://github.com/microsoft/vscode/issues/123077 is about the canvas renderer which the underscore issue is now an issue we're not planning on fixing, but you're definitely using the webgl renderer because the underscore is printing beyond its own cell.
I have an idea on fixing this, perhaps we should measure the underscore character in particular and measure if it would get cut off from the line and if so we would draw it custom (this could even be rolled into the upcoming custom box drawing/block element glyph setting). This custom drawing could be maybe 1 pixel higher than the underline style such that links and underscore styles can still be differentiated from underscores.
Tracking upstream in https://github.com/xtermjs/xterm.js/issues/3423

If anyone seeing this issue could let me know which font they're using that would be helpful (terminal.integrated.fontFamily, which falls back to editor.fontFamily).

If anyone seeing this issue could let me know which font they're using that would be helpful (terminal.integrated.fontFamily, which falls back to editor.fontFamily).

"editor.fontFamily": "'Droid Sans Mono', 'monospace', monospace, 'Droid Sans Fallback'"

This is my config in vscode.


KAFKA-14805 KRaft controller supports pre-migration mode

Based on work by @cmccabe #13003
Pre-Migration mode in QuorumController
This patch adds the concept of pre-migration mode to the KRaft controller. While in this mode, the controller will only allow certain write operations. The purpose of this is to disallow metadata changes when the controller is waiting for the ZK migration records to be committed.
The following ControllerWriteEvent operations are permitted in pre-migration mode

completeActivation
maybeFenceReplicas
writeNoOpRecord
processBrokerHeartbeat
registerBroker
unregisterBroker

Raft events and other controller events do not follow the same code path as ControllerWriteEvent, so they are not affected by this new behavior.
New metric from KIP-866
The kafka.controller:type=KafkaController,name=ZkMigrationState metric is now supported by the KRaft controller.
Upgrade after 3.4 ZK migration
In 3.4.0, we wrote a single ZkMigrationStateRecord(PRE_MIGRATION) at the beginning of the migration, but nothing after it completed. This means in later versions, we need to tolerate the stray record. This patch adds a tagged field PreMigrationSupported to ZkMigrationStateRecord which allows us to handle this scenario.
System tests
Two new system tests are included in this patch:
zookeeper_migration_test.py::TestMigration.test_pre_migration_mode_3_4 Starts a KRaft cluster in 3.4 with migrations disabled. It then performs a rolling upgrade to 3.5 with migrations enabled. The test ensures that a migration does not happen and checks for a log message

ZkMigrationState is NONE which means this cluster should not be migrated from ZooKeeper

zookeeper_migration_test.py::TestMigration.test_upgrade_after_3_4_migration Starts a ZK cluster and KRaft controller in 3.4 and does a migration. The KRaft controller is upgraded to 3.5 still with migrations enabled. The test ensures that the migration does not happen again and checks for the log message

Read a ZkMigrationState of PRE_MIGRATION from a ZK migration on Apache Kafka 3.4.

The release artifacts from 3.4.0 have been uploaded to Confluent's public S3 bucket in order to support these new tests.
TODO:

[x] Add system test for upgrading from older KRaft version to 3.4, ensure we write the correct ZkMigrationStateRecord
[ ] Additional unit tests for completion event

@mumrah thanks for this patch. pardon me that I don't dig in this patch before. Do you think the bug related to both #13405 and ZkMigrationIntegrationTest#testDualWrite is fixed by this patch?

@chia7712 it's quite possible that #13405 is another source of failures for ZkMigrationIntegrationTest#testDualWrite. This patch will fix the main source of failures that I've seen which is that some metadata (ProducerId block) is written before
the ZK migration completes. I was hoping to take a look at your PR this week, but I've not found the time yet. Hopefully I can look tomorrow.

@mumrah thanks for the response. I raised the question because 3.5.0 will get released on 4/26. The bugs happen when testing Zk migration so it would be nice to get fix into 3.5.0
Feel free to close #13405 if this PR can fix #13405 :)


CI: buildah to build images

Fixes the issue with containerization.
Images manifests are changing with the transition to buildah.
For the reference paritytech/scripts#246

This project is not maintained @TriplEight

gotcha


Passing additional variable data into the function

I'm using a DB select to search a model, it works fine but I want to pass in the _id of a belongs to so I can return options that are groupedBy 'name', 'some_id' - I can't seem to pass @this->some_id into the scope of the field function trying:
`
        $someId = $this->some_id; // Get some id from the current resource works and dumps '1'

        TextAutoComplete::make('Value', 'name')->items(function($search) use ($someId) {
            dump($someId); // always NULL not getting into the field scope
            return DB::table('products')
                ->select('name', 'some_id')
                ->where('some_id', $someId)
                ->where('name', 'like', '%' . $search . '%')
                ->groupBy('some_id', 'name')
                ->get()
                ->pluck('name');
        })->help('* Enter the value'),

`

Hello,
it should be possible now. Please check out v0.0.3.


Remove debugging prints from util/add-depends.pl

Oops...

It also leaves Makefile.$$ behind.

"rm Makefile.$$" if 1;
approved.

As it stands now one can execute nmake on Windows only once. After that it's syntax error about colon being unexpected. This is because compiler warnings are blended in...

As it stands now one can execute nmake on Windows only once. After that it's syntax error about colon being unexpected. This is because compiler warnings are blended in...

...  I don't understand how that happens.  Lines that do not start with Note: including file: should be discarded ($line = undef) ...

I think I've fixed the stray temp Makefile.$$ issue

Good idea.
Merged.  4ba5ce651790264cdd0e1af3564cf68f8be9ac81 to d3313a1f4cc6cb641c3bb68fd40625c118a4b7b7


Fix GC Error for FhirIO Integration Tests

Fixes #28635, https://github.com/apache/beam/issues/28505

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

retest this please

r: @bvolpato
I think it's worth disabling these for now, and then seeing what we have to work with once the migration is complete.

Thanks! Missed an import
/runner/_work/beam/beam/sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/healthcare/FhirIOSearchIT.java:159: error: cannot find symbol
  @Ignore("https://github.com/apache/beam/issues/28505")
   ^
  symbol:   class Ignore
  location: class FhirIOSearchIT

yes, sorry, pushing in next commit

R: @ahmedabu98

Run Java_GCP_IO_Direct PreCommit


Update PULP_CONTENT_HOST to PULP_CONTENT_ORIGIN

This is based on:
https://github.com/pulp/pulp_ansible/commit/e8a9176cef731b7646aa770608ec2276802ccd10
which is based on pulpcore change:
https://github.com/pulp/pulpcore/pull/370
which implements:
https://pulp.plan.io/issues/5629

@bmbouter, @daviddavis Mind taking a look

CONTENT_ORIGIN needs to have a scheme, host, and port in it. It's more than a name change. See the docs here:  https://github.com/pulp/pulpcore/blob/master/docs/installation/configuration.rst#content_origin
When setting it you put PULP_ on the front because that's how dynaconf layers settings onto Pulp. Docs on that are here: https://github.com/pulp/pulpcore/blob/master/docs/installation/configuration.rst#by-environment-variables

Also I believe @cutwater already applied a change like this somewhere?

Oh actually I think that was in the other repo, not the galaxy-api repo. Perhaps that's where the other change occurred.

Also I believe @cutwater already applied a change like this somewhere?

Hmm, maybe, but I don't see anything in origin/master to indicate that.

@alikins I agree I also don't see it. I had heard about if from slack so I'm not sure.

It's applied for production config. For development this change is still needed.

But this is not the right place to apply this change. It's a config for galaxy, not pulp.
PULP_CONTENT_HOST and PULP_CONTENT_PORT parameters are not related to pulp configuration. These indicate connection port and host for galaxy.
You should update docker/pulp/settings.py and set CONTENT_ORIGIN there.

PR that fixes this issue:
https://github.com/ansible/galaxy-dev/pull/171

@cutwater you likely correct about this because I haven't worked with the galaxy code closely like you have. FYI, prepending pulp options with PULP_ is how to set a Pulp setting via environment variable. I wanted to share some information in case it's helpful.

@bmbouter I'm aware of PULP_ prefix usage in environment variables.
We use GALAXY_ prefix for galaxy-api service, similar to how pulp is configured.
This configuration is part of galaxy-api configuration and has no direct relation to pulp services.


Incorrect link in Tutorial

The link to create an Apache Spark pool is incorrect. It takes users to a document describing how to create a dedicated SQL pool instead. For a newbie like me, this is an issue.
Current URL: https://docs.microsoft.com/en-us/azure/synapse-analytics/quickstart-create-sql-pool-studio
Needs to be: https://docs.microsoft.com/en-us/azure/synapse-analytics/quickstart-create-apache-spark-pool-studio

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

ID: 111f270d-937d-a682-0b35-93732c34c82b
Version Independent ID: 6e26539c-aa93-320e-016f-fa9b7a5f11bd
Content: Tutorial: Sentiment analysis with Cognitive Services - Azure Synapse Analytics
Content Source: articles/synapse-analytics/machine-learning/tutorial-cognitive-services-sentiment.md
Service: synapse-analytics
Sub-service: machine-learning
GitHub Login: @NelGson
Microsoft Alias: negust

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

PR is raised ,. so closing the issue


fix: Many recent PRs which broke tests and site

Schedule Calendar switcher broken in #2121
Track layout and direct link broken in #2157

Finally fixed. Two obscure selenium bugs remain


bug about prepareBath in MetricsPersistentWorker

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ X] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

https://github.com/apache/skywalking/blob/c5df0760f257e523902b641755b1cf7e6cee4476/oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/analysis/worker/MetricsPersistentWorker.java#L133
Math.max make maxBatchGetSize meaningless. And from the context , should it be Math.min ?

Bug

Which version of SkyWalking, OS and JRE?
8.1.X

Which company or project?

What happened?
If possible, provide a way to reproduce the error. e.g. demo application, component version.

Requirement or improvement

Please describe your requirements or improvement suggestions.

What do you mean meaningless? Could you explain?

https://github.com/apache/skywalking/blob/c5df0760f257e523902b641755b1cf7e6cee4476/oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/analysis/worker/MetricsPersistentWorker.java#L133-L147
If use Math.max, bathSize will always be astCollection.size() when astCollection.size() is grather than 2000. And then flushDataToStorage  all lastCollection。
If use Math.min, bathSize will  be 2000 when astCollection.size() is grather than 2000. And then flushDataToStorage  2000 size of
lastCollection every time.

Make sense, could you send the pull request to fix this?

Of course.


initial prefetch for simple single chunked dim

POC Prefetch Generator:
This is a draft pr to articulate one possible approach to "prefetching" dask arrays or xarray arrays with dask.
The goals were to simultaneously:

support batching over the first dimension, which is the only chunked dimension
support prefetching by making sure prefetch number of batches are always loading
use the typical dask mechanics to transfer data by submitting tasks, working with futures and as_completed
super basic profile of Gb/s. I want to be able to feed my gpu at around a few gb/s.
additional considerations are written in the docstring.

I also tried one approach using a Queue on the workers. This felt weird and found myself reinventing features that dask already has.
Results
Using helm to deploy a cluster on kubernetes with 8 workers (4cpu and 16gb each and relatively standard network configurations), I am able to see:

1.6Gb/s without any adjustment in prefetch or array size
data transferring between workers on the daskui

What Next?
No clue. I would like to investigate

[ ] does gb/s scale as number of workers increases?
[ ] how do we determine a good prefetch size and avoid memory issues?
[ ] does this work with tensorflow? Will I get an error about things being pickled?
[ ] Can the deterministic batch idx approach be generalized easily?
[ ] Does this even belong in xbatcher? I feel like it was functionality I was hoping xbatcher would support.

note:

pre-commit is being funny--I skipped verification on commit for now.
running the script will save a 33gb zarr dataset if you specify a remote_path!

In another attempt to simplify an example and profile transferring data from multiple workers to a single worker (where an ml tasks would iterate over batches) I have created this example:
from more_itertools import chunked
import dask.array as da
from distributed import Client, get_client, wait
import seaborn as sns
import pandas as pd

# 16 workers (4cores, 16gb each) on kubernetes; no special network configuration
client = Client("...")

chunk_gbps = {}
for max_gather in [.5e9, 1e9, 2e9, 4e9, 6e9]:
    for chunk in [128, 256, 512, 1028, 2048]:
        print(f"looking at chunk {chunk}, {max_gather / 1e9}")
        _ = client.restart()
        array = da.random.random((25000, 100, 100, 9), chunks=(chunk, 100, 100, 9)).persist()
        wait(array)
        del client.datasets['test']
        client.publish_dataset(test=array)

        # determine block batch size to control transfered data with gather
        ex = array.blocks[0]
        batch_size = max(int(np.floor(max_gather / ex.nbytes)), 1)

        def compute_bytes():
            client = get_client()
            array = client.get_dataset('test')
            blocks = list(array.blocks)
            nbytes = 0
            t0 = time.time()
            for block_batch in chunked(blocks, batch_size):
                fs = [client.compute(b) for b in block_batch]
                arrays = client.gather(fs)
                for array in arrays:
                    nbytes += array.nbytes
            elapsed = time.time() - t0
            return (nbytes / elapsed) / 1e9

        # blocks = client.submit(get_blocks, pure=False)
        f = client.submit(compute_bytes, pure=False)
        chunk_gbps[(max_gather / 1e9, chunk, batch_size)] = f.result()

# plot for some trends
data = [(*k,v) for k,v in chunk_gbps.items()]
df = pd.DataFrame(data, columns=['gb_gather','chunk_size', 'actul_batch', 'gbps'])
sns.lineplot(x="gb_gather", y="gbps",hue="chunk_size",data=df)

@jhamman @maxrjones this is sort of the approach am considering developing.
I think 2gbps should be fine, but I was able to get 8+gbps with https://github.com/NVlabs/tensorcom, which uses msgpack with pyzmq. I am trying to avoid using that and stick with the dask mechanics, but I am tempted to mock up a quick profile script of using zmq to bypass dask entirely, but within dask tasks.
This all might not belong in xbatcher, but I wanted to put it out there to get ay feedback people might have.

Here is an example of using the prefetch generator with tf.data.Dataset
from xbatcher.prefetch_generators import PrefetchBatchGenerator
array = ...
batch_size=128
shape = array[:batch_size].shape
def do_tf_ml():
    batch_gen = lambda : PrefetchBatchGenerator(array=array, batch_size=batch_size, prefetch=20)
    ds_counter = tf.data.Dataset.from_generator(batch_gen, output_types=tf.int32, output_shapes=(shape))
    nbytes = 0
    t0 = time.time()
    for count_batch in ds_counter.repeat().take(128):
        nbytes += count_batch.numpy().nbytes
    elapsed = time.time() - t0
    return nbytes / elapsed

f = client.submit(gen_ex)
f.result() / 1e9

Can the test at the bottom be wrapped as a function? I'm guessing it's not supposed to run for everyone.

Actually I think I was confused. I read if __name__ == "__main__" as the entry point rather than as a conditional entry point. Pythonisms are not for forte lol


Rich Dad Wallet

Dear Reviewer,
In this final capstone project, I have applied all the requirements. please see the details below.

[x] - Splash screen
- A simple page with the name of your app (yes, you need to choose one), and links to the sign up and log in pages.

[x] - Sign up and log in pages
-The user should be able to register in the app with full name, email and password (all mandatory).
- The user can log into the app using email and password.
- If the user is not logged in, they can't access pages that require the user to be logged in (all the pages described below).

[x] - Home page (categories page)
- When the user logs in, they are presented with the categories page.
- For each category, the user can see their name, icon and the total amount of all the transactions that belongs to that category.
- When the user clicks (or taps) on a category item, the application navigates to the transactions page for that category.
- There is a button "add a new category" at the bottom that brings the user to the page to create a new category.

[x] - Transactions page
- For a given category, the list of transactions is presented, ordered by the most recent.
- At the top of the page the user could see the total amount for the category (sum of all of the amounts of the transactions in that category).
- There is a button "add a new transaction" at the bottom that brings the user to the page to create a new transaction.
- When the user clicks on the "Back" button (<), the user navigates to the home page.

[x] - "Add a new category" page
- The user fills out a form to create a new category, indicating their name and icon (both mandatory).
- The user clicks (or taps) the "Save" button to create the new category, and is taken to the home page on success.
- When the user clicks on the "Back" button (<), the user navigates to the home page.

[x] - "Add a new transaction" page
- The user fills out a form to create a new transaction with:
- name (mandatory)
- amount (mandatory)
- categories (mandatory at least one)
- The user click (or taps) the "Save" button to create the new transaction, and is taken to the transactions page for that category.
- When the user clicks on the "Back" button (<), the user navigates to the transactions page for that category.

Hi @abeltsew,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because you did not follow the right git-flow workflow thus creating the PR branch from a descriptive feature branch into the dev branch
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


tf1.13 can be installed with cuda8.0, cudnn5.1 but tf1.6 cannot

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version:
Python version:
Installed using virtualenv? pip? conda?:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Ubuntu 16.04
Cuda 8.0
Cudnn 5.1
tensorflow pip3 install tensorflow install tf1.13 sucessfully
but I cannot install tf1.4 with pip install tensorflow-gpu==1.6.0 (https://pypi.org/project/tensorflow-gpu/1.6.0/)

@yangjingyi Please find instructions in this link which help you to install Tensorflow GPU on Linux Ubuntu platform. Please let me know how it progresses.

@yangjingyi You can download the tensorflow 1.6 wheels from this page. Further you can pip install the downloaded wheels to install tf. Hope this helps. Thanks!


Feature: Make integration config enrichment respect user config

Changes so that the EnrichIntegrationConfigStrategy respects the user configuration.
Fixes issue https://github.com/stormpath/express-stormpath/issues/106.

Looks great! Merging


Auto/autoware utils colcon ignore

Description
remove COLCON_IGNORE in autoware_utils
Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Read pull request guidelines
[x] Code follows coding guidelines
[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code follows coding guidelines
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR / build-and-test-pr: Required to pass before the merge.
Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the guidelines.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@kenji-miyake @TakaHoribe
Auto-merge was set in the autoware_utils porting PR, and the PR was merged automatically though I'm still working on it.
I opened a new one, just removing COLCON_IGNORE, and that's all for autoware_utils porting.
https://github.com/tier4/AutowareArchitectureProposal.iv/pull/489

@kenji-miyake @takayuki5168
require dependency?
ERROR: the following packages/stacks could not have their rosdep keys resolved
to system dependencies:
autoware_utils: Cannot locate rosdep definition for [autoware_lint_common]

@taikitanaka3 I've mentioned to Miyake-san on Slack.

You have to remove autoware_lint_common/COLCON_IGNORE.


7 audio ml to frontend

Assignee Tasks

[ ] added unit tests (or none needed)
[ ] written relevant docs (or none needed)
[ ] referenced any relevant issues (or none exist)

Guidelines
Please add a description of this Pull Request's motivation, scope, outstanding issues or potential alternatives, reasoning behind the current solution, and any other relevant information for posterity.

Your PR Notes Here

closed as updates were made


lucet-module, which is part of the runtime pulls in all of cranelift-codegen which is large

Context
This is part of a series of bugs that I spoke to @tyler @pchickey about. We are currently using Lucet to sandbox libraries in C++ applications. The idea behind this is that using a wasm sandboxed version of the library allows ensuring that a memory safety issue in the library does not automatically result in a memory safety vulnerability in the full application. One of the consumers of this work is the Firefox web browser.
Problem
lucet-module, which is part of the runtime pulls in all of cranelift-codegen. So, from code comments here, this is already on people's radar. I am just opening this bug to express that we have an interest in solving this.
Actions
Could you let me know if this is a change that is being planned anytime soon internally, or if it would be better if I work on a patch that can be accepted? (cc @froydnj)

We use cranelift-codegen's types for signatures in lucet-module, which I think is desirable over defining types in lucet-module that mirror cranelift's, and impl to/from conversions in lucetc and lucet-runtime. So, I think the right thing to do here is to make a PR to cranelift that factors the signatures into a separate module, similar to what cranelift-entity did for the entity datatypes. Once that exists, we can switch the dep to cranelift-signatures.

We brought this up with the cranelift team. See here. The suggestion was to see if we could perform the cranelift IR types to lucet type translation during compilation of the module instead of the runtime.
At this time, I'm not well versed enough in the lucet code to know if this is feasible?
@pchickey - any thoughts?

Ah, I didn't see that, I never look at bugzilla for Cranelift things, just the github issue tracker.
The solution they're proposing is basically for lucet-module to have its own version of the small collection of cranelift-codegen::ir types it uses: the integers types, AbiParam, and the ir::Argument... types.
I just went and looked through the lucet-module::types and  cranelift-codegen::ir types actually get generated from the meta crate, I no longer think a separate cranelift-signatures crate is feasible. And we already have our analog to signatures defined. So, scratch the suggestion I made in my prior post.
The way forward is for lucet-module to move everything from lucet-module/src/types.rs that mentions cranelift-codegen::ir out into lucetc. The only things that should remain in lucet-module::types are pub struct Signature and pub enum ValueType.

Makes sense. In the interest of next steps, could you let me know if this is something that would be handled internally (given that this involves a bit of surgery in the code base) or something I should contribute a patch for (compared to some of the other open bugs, I'm a bit hesitant on this one, as it is touching parts of the code base that I don't know as well, but I could probably figure it out)?

I don't think anyone is going to have time to get to this in the next few days, so if you want to do a PR I'm happy to review it and help however you need.


Licensing question

Hi,
I'm interested in porting LZO to another language as well, but see it's GPLv2. My understanding was that this was quite pervasive, covering even "work derived". However, I see you have this under MIT; which is what I'm after as well. I am a license novice, so just wanted to understand better if relicensing from implementation is acceptable or if you maybe contacted the author directly?
Thanks for the help!

You can't relicense, but MIT is permissive enough that you can use it as long as you credit Jack and include the original license.

I don't know how accurate that is, GPL basically says it can't be used in proprietary software, and this spans to include all work, even derived from the original source. MIT allows, of course, use in proprietary software. What you're describing is BSD genre of licensing.

Maybe you're thinking I want to re-implement LZO again from this C++ implementation; which is under MIT. My question was how did this project manage to get it MIT licensed when the original C implementation was GPLv2 licensed. ref: http://www.oberhumer.com/opensource/lzo/

From the source code, it seems like the original implementation wasn't used, but docs from some broken URL: https://github.com/jackoalan/lzokay/blob/546a9695271e8a8b4711383f828172754fd825f2/lzokay.cpp#L6 I think maybe this is how it was done, as I'm pretty sure implementing something with a restrictive license is okay if it's instead implemented from papers/something else other than project itself which implements it with the restrictive license.
Will assume this is how it was done then, and close this issue. Thanks! Please feel free to clarify if I have it wrong.


New dark themed Dimension Hole

It's my first time doing game update in Swarfarm, so let me know if I'm missing something.

Added new dark themed dimension
Added new 2A monsters (Howl, Grim Reapers)
Fixed language for Wave info

resolves #183

Looks good!


Model cannot be saved.

I tried to use model.save() to save the model after training but an error happens. Why?

I tried to use model.save() to save the model after training but an error happens. Why?

you could save model by model.save_weights


WIP: FINERACT-2114: Interest Rate Modification as on Date

DISCLAIMER: THIS PR. still a work-in-progress status.
Description
This P.R. applies an Interest Rate Change through the Advanced Payment Schedule Processor and reprocesses the installments if interest rate recalculation is enabled.

@janez89 @Nayan @rvs @jimjag @raboof  are the contributions still active for this project??

I'm not sure why you pinged me, but https://github.com/apache/fineract/pulls?q=is%3Apr+is%3Aclosed show contributions are being merged for this project.
What makes you think they might not be? This PR has only been open for 14 hours. Even for active projects, it can sometimes take days, weeks or even months for a PR to be reviewed - as projects are driven by volunteers after all.

@raboof I wanted to know about the status of the contribution even I am a contributor wanted to know further about this project ? since its a community driven project I wanted to know further about the contributions?

You can find more information on how to contribute at https://fineract.apache.org/#contribute https://github.com/apache/fineract/blob/develop/CONTRIBUTING.md


chore: Migrate Card to gamut

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.4-alpha.c3394c.0@codecademy/gamut-labs@10.5.3-alpha.c3394c.0@codecademy/gamut@26.0.2-alpha.c3394c.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6044539893462c2dd567f713--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.4-alpha.e50760.0@codecademy/gamut-labs@10.5.3-alpha.e50760.0@codecademy/gamut@26.0.2-alpha.e50760.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60445775f32a8d259c6880d6--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.7.7-alpha.e0d05f.0@codecademy/gamut-kit@0.2.4-alpha.e0d05f.0@codecademy/gamut-labs@10.5.3-alpha.e0d05f.0@codecademy/gamut-styles@8.3.2-alpha.e0d05f.0@codecademy/gamut-tests@2.2.2-alpha.e0d05f.0@codecademy/gamut@26.0.2-alpha.e0d05f.0@codecademy/markdown-overrides@0.5.9-alpha.e0d05f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6045a16c5caa1495d89825dd--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.7.7-alpha.4f89db.0@codecademy/gamut-kit@0.2.4-alpha.4f89db.0@codecademy/gamut-labs@10.5.3-alpha.4f89db.0@codecademy/gamut-styles@8.3.2-alpha.4f89db.0@codecademy/gamut-tests@2.2.2-alpha.4f89db.0@codecademy/gamut@26.0.2-alpha.4f89db.0@codecademy/markdown-overrides@0.5.9-alpha.4f89db.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6045a39793462ce62067f901--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.7.7-alpha.7dad42.0@codecademy/gamut-kit@0.2.4-alpha.7dad42.0@codecademy/gamut-labs@10.5.3-alpha.7dad42.0@codecademy/gamut-styles@8.3.2-alpha.7dad42.0@codecademy/gamut-tests@2.2.2-alpha.7dad42.0@codecademy/gamut@26.0.2-alpha.7dad42.0@codecademy/markdown-overrides@0.5.9-alpha.7dad42.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6045dd6e707d1c89d5a3e053--gamut-preview.netlify.appDeploy Logs


use https as submodule registry

Codecov Report

Merging #1261 (57bc4ac) into develop (492d477) will increase coverage by 0.27%.
The diff coverage is 37.30%.

@@             Coverage Diff             @@
##           develop    #1261      +/-   ##
===========================================
+ Coverage    57.94%   58.22%   +0.27%
===========================================
  Files          407      408       +1
  Lines        15371    15270     -101
  Branches      2931     2876      -55
===========================================
- Hits          8907     8891      -16
+ Misses        6187     6103      -84
+ Partials       277      276       -1

Impacted Files
Coverage Δ

src/cascader/cascader.tsx
64.00% <ø> (ø)

src/date-picker/DatePicker.tsx
26.74% <ø> (ø)

src/date-picker/DateRangePicker.tsx
15.82% <ø> (ø)

src/date-picker/date-picker-panel-props.ts
100.00% <ø> (+90.00%)
:arrow_up:

src/date-picker/date-range-picker-panel-props.ts
100.00% <ø> (+90.90%)
:arrow_up:

src/date-picker/date-range-picker-props.ts
42.85% <0.00%> (-11.69%)
:arrow_down:

src/date-picker/props.ts
53.84% <0.00%> (-16.16%)
:arrow_down:

src/range-input/props.ts
50.00% <ø> (ø)

src/range-input/range-input-popup-props.ts
25.00% <0.00%> (-75.00%)
:arrow_down:

src/range-input/range-input-popup.tsx
100.00% <ø> (ø)

... and 26 more

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


Custom file extensions

I'm using eslint-import-resolver-alias along with eslint-import-resolver-typescript.
I have the following alias:
["@dpStores", path.resolve(__dirname, './src/stores')]
But have the following eslint error when I try to import a types.tsx file without specifying the extension:

If I specify the extension it works just fine.
Importing files without an extension works fine when not using an alias.
This is my resolver settings in .eslintrc:
 "settings": {     "import/resolver": {       "node": true,       "eslint-import-resolver-typescript": true,       "alias": [         ["@dpStores", path.resolve(__dirname, './src/stores')]       ]     }   }

@Tsury  Actually, eslint-import-resolver-alias uses Node.js native package resolution to find a module and does not support custom file extensions now. That's why it works fine when you specify the extension. I'm going to add this new feature recently and hope that will be helpful.

I have same problem with .jsx extensions.

@johvin Got it, thanks!

Version 1.1.0 is released to support custom file extensions. @Tsury  You can try again with the following configuration in your .eslintrc.js file.
"settings": {
  "import/resolver": {
    "alias": {
      "map": [
        ["@dpStores", path.resolve(__dirname, './src/stores')]
      ],
      "extensions": ['.js', '.ts', '.tsx', '.d.ts', '.json']
    }
  }
}

@dhlavaty You can solve your problem with a similar configuration as above, like "extensions": ['.jsx', '.js', '.json']

Works. Thank You


VQE optimization history has unexpected plateaus

Environment

Qiskit version: 0.37.2
Qiskit Terra version: 0.21.2
Qiskit Aer version: 0.10.4
Python version: 3.9.0
Operating system: macOS Big Sur 11.6.4

What is happening?
When using the statevector simulator, the ADAM optimizer, and a callback function in VQE, the energy stored by the callback function displays unexpected plateaus. In an example below, every 9 adjacent iterations have very similar energies.
How can we reproduce the issue?
The code is modified from https://qiskit.org/documentation/tutorials/algorithms/04_vqe_advanced.html
from qiskit import Aer
from qiskit.opflow import X, Z, I
from qiskit.utils import QuantumInstance, algorithm_globals
from qiskit.algorithms import VQE
from qiskit.algorithms.optimizers import ADAM
from qiskit.circuit.library import TwoLocal
import matplotlib.pyplot as plt

H2_op = (-1.052373245772859 * I ^ I) + \
        (0.39793742484318045 * I ^ Z) + \
        (-0.39793742484318045 * Z ^ I) + \
        (-0.01128010425623538 * Z ^ Z) + \
        (0.18093119978423156 * X ^ X)

seed = 50
algorithm_globals.random_seed = seed
qi = QuantumInstance(Aer.get_backend('statevector_simulator'), seed_transpiler=seed, seed_simulator=seed)

ansatz = TwoLocal(rotation_blocks='ry', entanglement_blocks='cz')
optimizer = ADAM(maxiter=10)

intermediate_info = {
    'nfev': [],
    'parameters': [],
    'energy': [],
    'stddev': []
}

def callback(nfev, parameters, energy, stddev):
    intermediate_info['nfev'].append(nfev)
    intermediate_info['parameters'].append(parameters)
    intermediate_info['energy'].append(energy)
    intermediate_info['stddev'].append(stddev)

vqe = VQE(ansatz, optimizer=optimizer, quantum_instance=qi, callback=callback)
result = vqe.compute_minimum_eigenvalue(operator=H2_op)

Plotting intermediate_info['energy'] versus intermediate_info['nfev'] gives

Every 9 adjacent iterations have very similar energies, which seems unexpected with the Adam optimizer. When the number of qubits of the Hamiltonian changes, the length of each plateau can also change (from 9 in the current example).
What should happen?
The expected optimization history of energy with an Adam optimizer should be smoother, without artificial plateaus.
Any suggestions?
I guess that some repeated evaluations are done in optimization, or the callback function is not working properly.

The CG optimizer also yields a similar issue

The GradientDescent optimizer does not yield plateaus but also yields some repeated pattern

If you look at the plots in this tutorial https://qiskit.org/documentation/tutorials/algorithms/02_vqe_convergence.html you will see similar plateaus. These are where optimizers using gradients are computing say a finite diff gradient using the same objective function - the callback is from the objective function and cannot discriminate how the optimizer is using it. Since the delta around the point is so small when the gradient is computed at the scale of the plots they end up with the staircase like look.
Adam uses a finite diff gradient by default. TwoLocal defaults to reps=3 and with 2 qubits that 2 parameters per layer (rep) plus 2 more in a final block so 8 params in total. So I believe the 9 is 1 + 8 i.e one eval at the current point and 8 around it to compute the gradient to go to the next point (the finite diff like scipy only does a small epsilon in one direction in each dimensions)
And the number of points will vary by size of the hamiltonian, number of qubits given the same ansatz structure. Hopefully you can see from my detailing where the 9 comes from in the above.

As you gave a thumbs to my prior comment I will assume the question/matter is solved, so I am closing this.
I will finally however note that if you use a gradient with VQE then the gradient computation does not go through its objective function and the callback will reflect each iteration as you expected.

Thank you so much! Your explanation makes sense to me. Yes, the question was solved. I further checked the source code of ADAM and VQE. Without using the gradient argument in VQE, with the ADAM optimizer, the first evaluation is at the center, then consecutively at n_params points in all directions of the parameter space by eps. Each step's evaluation is passed into the VQE callback function. The last iteration in the plot is only at the center, since no further update is needed. That is why there is only one point in the last "step" of the staircase.
A fairly similar process happens with SPSA and QNSPSA.
Thank you for your note on gradient, and since you mentioned, I am also trying the VQE gradient argument exlicitly with different types of optimizers, but sometimes the callback does not work as I have expected. Sometimes only the final result is passed into the callback. Just like the page you attach https://qiskit.org/documentation/tutorials/algorithms/02_vqe_convergence.html, the last plot "Energy Convergence using Gradient" contains only one point, I think (It is hard to eyeball) consistent with my tests.
Sometimes, when changing the type of the optimizer, and the grad_method in Gradient, the callback might give all the evaluations.
Do I need to submit a separate issue regarding the gradient?

That last plot used to look like a curve - is there really a point in it. I noticed that it seemed empty and just assumed it did not plot for some reason. Hopefully it ran more than one point and stopped, then plotted that. The optimizer itself always works with the objective function for each iteration. There is also a callback on the scipy optimzers - you could also see what is happening using that. Its hard to imagine if the objective function is called that a callback is not done. If you look further and there does seem to be a problem then please feel free to raise another issue. A different one would be better more targetted to whatever the problem seems to be.
I will note that algorithms like VQE are currently being rewritten to use the (runtime) primitives and there are now gradients under algorithms folder here in main that are new and will be used instead on the older ones as these too work with primitives. So things are going to change in this area anyway - well they are changing already and the plan is this is all in the next upcoming release (the existing code will be marked pending deprecation, then be deprecated in a subsequent release and finally removed leaving just the new primitive based algos.)

Thank you. There is actually a single point on that last plot. The plot does not set the marker, so it may not display. The center of the energy axis of the "empty" plot is at that final energy value.
I will raise another issue with more details.


Turn off preview OTel metrics and remove the macro

Describe the pull request

What does your PR fix?

OpenTelemetry C++ provided preview version of metrics which is outdated as the stable version of metrics released in v1.7.0 (v1.7.0 is the current version in vcpkg, see here). This PR removes setting WITH_PREVIEW_METRICS=ON which turns off the preview version of metrics, and back to the new metrics implementation in this version which is the expected behavior.
BTW, the setting WITH_PREVIEW_METRICS was completely removed in https://github.com/open-telemetry/opentelemetry-cpp/pull/1735/files.

Which triplets are supported/not supported?

Yes
If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Please bump the port version. See documentation.
Also please run command vcpkg x-add-version opentelemetry-cpp then commit the changes.

Please bump the port version. See documentation. Also please run command vcpkg x-add-version opentelemetry-cpp then commit the changes.

Thanks @FrankXie05. Updated the port number accordingly.

Close this fixes for now as it requires change in the source opentelemetry-cpp repo. This fix was in the latest version (v1.8.1) and I'll update this based on that.


Unable to get running virtual services

Describe the bug
A clear and concise description of what the bug is.
I am running istio using consul , While deploying any configuration/application, lets say the official bookinfo sample, after successfully deploying the required yaml files when I run istioctl get virtualservices, I get an error  Error: the server could not find the requested resource (get virtualservices.networking.istio.io)
Expected behavior
A clear and concise description of what you expected to happen.
This command should give a list of all running virtualservices
Steps to reproduce the bug
Steps to reproduce the behavior.

Install istio
2.Go to samples/bookinfo
Run - docker-compose -f samples/bookinfo/consul/bookinfo.yaml up -d
docker-compose -f samples/bookinfo/consul/bookinfo.sidecars.yaml up -d
run istioctl get virtualservices

Version
What version of istio and Kubernetes are you using? Use istioctl version and kubectl version
istioctl - 0.80
Running on consul
Is Istio Auth enabled or not?
Did you install the stable istio.yaml, istio-auth.yaml.... or if using the Helm chart please provide full command line input.
Yes
Environment
Which environment, cloud vendor, OS, etc are you using?
Running locally using consul

not stale


Build error

I downloaded the whole project. Tried to build macOS demo project. Xcode returned the error "could not read data from '/XXXX/Swifter-master/Swifter/Info.plist': The file “Info.plist” couldn’t be opened because there is no such file."
Seems to me that when moved source files to "source" directory, something went wrong. Workaround is make "Swifter" directory and copy info.plist file from "source" directory.

@Goro-Otsubo it missing plist attach with it framework. Just go to Project Target:  Example: I build it on iOS, so select SwifteriOS( In Target ) -> General -> Choose plist file in the Source Directory.
Hope it help.

It worked! Thanks..


Keybindings with 'cmd' won't work on Linux

On Linux, packages that specified 'cmd' in their keybinding sequences won't respond to any keyboard shortcuts, because there's no cmd key on Linux.
I see the platform-specific solution like .platform-linux, but I don't think package authors would like to add extra keybindings.
Can we solve this issue elegantly, like mapping cmd to ctrl on platforms other than darwin? I'm aware there's ctrl key on darwin, too, but it doesn't seem to be so useful (except in Terminal), like the corresponding super key on Linux.

Who closed this? The issue is hardly resolved.

Is this issue really just a "package" problem?  I decided to give Atom a try, because it looks promising and is associated with github.  However the lack of Atom Linux support for (an alternative to) the cmd- key quickly lead to frustration.  Even in a vanilla install (version 1.0.10), I had real problems with the basics.  For instance, trying to follow section 2.2 of the flight manual (Moving in Atom) to edit my first file was difficult and frustrating.  Specifically, according to the manual, "To go up and down a single character, you can use ctrl-P and ctrl-N".  But these commands, respectively, invoke the fuzzy finder and open a new pane having an empty "untitled" buffer.
Perhaps this behavior occurs because the Linux version of Atom binds some of the standard cmd- chords to ctrl- versions, crowding out the standard ctl- chords.  I haven't dug deeply enough to verify this speculation, but it does seem that the lack of cmd- key support in Linux causes problems for core Atom functionality and makes the flight manual untrustworthy.  Perhaps it is arguable whether this is an Atom, Chromium or Linux problem.  But it is undoubtedly a problem for me as a new user.   I have lots of work to do, and a text editor is supposed to make life easier rather than harder, so I find myself reluctantly concluding that my brief flirtation with Atom is unlikely lead to any lasting relationship.
My vote for a cmd- key replacement for Linux would be ctrl-alt- or ctrl-shift-.  Is such a "triple chording" approach viable?

@MichaeLeroy, correct. The lack of cmd- key causes a lot of problems for linux users and as you mentioned; makes life for linux users harder rather than easier.
The problem with your "triple chording" approach is that some packages have already started using these combinations to support linux users. This would inevitably result in conflicts, however, I agree that it would at least reduce conflicts. For example,:
package A uses cmd+t to open a new tab
package B uses ctrl-shift-t to open a a new window
"triple chording" cmd- to ctrl-shift- would create a conflict here.
The only perfect solution I could think of is giving linux users a cmd key.

I really hope this issue will eventually be re-opened and atom will allow us to use more key-bindings. Closing it and saying "not my problem" is fair, but at the same time, atom is the only place where this can be fixed.

@t1bb4r thanks for your reply to my post.  It has encouraged me to think a bit more on this issue.  A bit of research hints that it may well be possible, and perhaps not too hard, to "add a cmd- key" to Linux.  It might even be possible to provide the cmd- key specifically to instances of Atom, leaving other applications undisturbed.
I'm not an expert on this topic, but I do know my way around Linux and am willing to give it a try.  However, I am an Atom newbie, so I need a bit of help in that department.  Is there a quick and easy way to tell the Linux version of Atom to adopt the "standard" key-bindings?  Ideally I'd be able to switch back and forth easily and unambiguously; is it as simple as dropping in the right configuration file?  Doing so would help me in testing my work and, if all goes well, writing up a How-To.

It looks as if I have managed to "add a cmd- key" to my Linux system.
As I had expected, it wasn't terribly difficult to do so.  My quick and easy solution (under KDE) was: system settings >> hardware >> keyboard >> advanced >> Alt/Win key behavior >> meta is mapped to left Win (see screenshot below).  (A production solution might be different, more general and perhaps user transparent.)  After making this assignment, I was able to bind the action core:move-to-bottom to cmd-end and found that this keying did send me to the bottom of the buffer.  I attempted to get cmd-down, the flight manual keystroke for this action, to work but failed.  Perhaps this standard binding fails because cmd-down is mapped to "native!", whatever that means.
OK, I think that I have a Linux system with a cmd- key.  But my Atom installation has its idiosyncratic Linux without cmd- bindings, where ctrl-p opens the fuzzy finder rather than moving up a line.  Is there a way for me to get to the standard vanilla, with cmd-, bindings?  Are all of those "native!" bindings (there are dozens) going to be an obstacle to achieving this goal?

I didn't read everything you posted yet, but I can share some information I've gathered in my struggles.
Linux has and can set a "cmd" or "meta" -key, however, the problem is that atom doesn't see this as a modifier. eg.,
Atom interprets a + b as a then b, since 'a' isn't a modifier. More specifically ctrl-cmd-t fires the events ctrl+cmd and ctrl+t instead of what you would expect (I found this out by logging all events fired). That's the real issue right! We need the extra modifier to allows us to make smart key-bindings. The modifier key can be any key as long as Atom registers it as a modifier and not just another key. This is where one of my previous comments regarding chromium becomes relevant.
My explanation is probably inaccurate, it's been a year since I've played around with Atom, but I'm fairly sure the core issue is still the same.

Is this issue really just a "package" problem? I decided to give Atom a try, because it looks promising and is associated with github. However the lack of Atom Linux support for (an alternative to) the cmd- key quickly lead to frustration. Even in a vanilla install (version 1.0.10), I had real problems with the basics.

The issue here is not that Atom on Linux lacks support for the Cmd key, but that the documentation needs to be updated to display platform-specific keybindings (instead of just OS X keybindings) - it's an unfortunate vestige from when Atom was just an OS X beta. You can subscribe to https://github.com/atom/docs/issues/16 for updates on fixing this.
All of the commands that have keybindings on OS X should have corresponding keybindings on Linux (and Windows) - there should be absolutely no need to hack your system to support keybindings that will not work on your OS anyway, even with a system hack. (All of those Cmd shortcuts, at least for Atom core and bundled packages, are scoped to OS X, so they cannot be used on Linux, even if you hack a Cmd key together.)

Back to the original post:

I see the platform-specific solution like .platform-linux, but I don't think package authors would like to add extra keybindings.

This is the solution here - no one should be expecting OS X-specific keybindings using Cmd to work on any other platform. If a package lacks non-OS X keybindings, then the author should be made aware of that.

@t1bb4r, your additional comments are very helpful in clarifying the issue.  If I understand them, it means that "double chorded" keystrokes containing cmd- may work if the system is properly configured but that "triple chorded" cmd- keystrokes are garbled/misinterpreted as pairs of "double chorded" keystrokes.  A quick look at cmd-ctrl-t on my system (using xvt) yields:
KeyPress event, serial 40, synthetic NO, window 0x3000001,
root 0xd5, subw 0x0, time 1192899, (1147,456), root:(1147,493),
state 0x0, keycode 37 (keysym 0xffe3, Control_L), same_screen YES,
XLookupString gives 0 bytes:
XmbLookupString gives 0 bytes:
XFilterEvent returns: False

KeyPress event, serial 40, synthetic NO, window 0x3000001,
root 0xd5, subw 0x0, time 1201738, (1147,456), root:(1147,493),
state 0x4, keycode 133 (keysym 0xffe7, Meta_L), same_screen YES,
XLookupString gives 0 bytes:
XmbLookupString gives 0 bytes:
XFilterEvent returns: False

KeyPress event, serial 40, synthetic NO, window 0x3000001,
root 0xd5, subw 0x0, time 1205229, (1147,456), root:(1147,493),
state 0x44, keycode 28 (keysym 0x74, t), same_screen YES,
XLookupString gives 1 bytes: (14) ""
XmbLookupString gives 1 bytes: (14) ""
XFilterEvent returns: False

KeyRelease event, serial 40, synthetic NO, window 0x3000001,
root 0xd5, subw 0x0, time 1205348, (1147,456), root:(1147,493),
state 0x44, keycode 28 (keysym 0x74, t), same_screen YES,
XLookupString gives 1 bytes: (14) ""
XFilterEvent returns: False

KeyRelease event, serial 40, synthetic NO, window 0x3000001,
root 0xd5, subw 0x0, time 1206081, (1147,456), root:(1147,493),
state 0x44, keycode 133 (keysym 0xffe7, Meta_L), same_screen YES,
XLookupString gives 0 bytes:
XFilterEvent returns: False

KeyRelease event, serial 40, synthetic NO, window 0x3000001,
root 0xd5, subw 0x0, time 1206130, (1147,456), root:(1147,493),
state 0x4, keycode 37 (keysym 0xffe3, Control_L), same_screen YES,
XLookupString gives 0 bytes:
XFilterEvent returns: False

Or more compactly ⇓ctrl-⇓meta-⇓t-⇑t-⇑meta-⇑ctrl.  The analogous ctrl-alt-t yields a similar press and release sequence, ⇓ctrl-⇓alt-⇓t-⇑t-⇑alt-⇑ctrl.  In both cases, these appear (to my non-expert eye) to be single keystrokes, a t with states 0x44 and 0xc respectively.  This seems pretty reasonable, but who knows (without digging into the source code) how Chromium responds to such inputs?  It's understandable that Chromium might have (fixable) issues with this keystroke but does seem weird that it would translate ctrl-meta-t to the character pair ctrl-t + meta-t.

@mnquintana, thanks for providing your perspective on this issue.  I encountered Atom for the first time only a few days ago and certainly agree that having a significantly inaccurate introductory manual is an impediment to adoption.  However, as somewhat of an application adoption trailblazer amongst those I work and play with, I'm accustomed to getting past poor documentation and assaulting difficult learning curves.  If the issue were merely one of a platform specific manual, I'd have likely shrugged it off and proceeded to learn this intriguing tool, not bothering to check much less post to an issues forum.
This issue strikes me a going deeper than a few platform specific differences in key bindings.  Well considered and implemented key-bindings are critical to a good text editor.  My impression, from looking at the manual, is that Atom's developers did an excellent job of choosing key-bindings that should be easy to learn for new users who have experience with other applications.    I placed myself in the analogous Linux situation a few days ago and my reaction was, "this is crazy, nothing makes sense".  It was only after this frustrating initial impression that I turned to the flight manual to learn what I was doing wrong.  The flight manual didn't help me much with editing, but it did convince me that "if only I had a cmd- key and a standard set of bindings, things would be cool".  That's why I picked up this old thread and played at hacking to "give my system a cmd- key".  As I hinted in my third post here, I didn't consider such hacking a production solution to the issue, rather it was a way to explore the issue and see if achieving good key-bindings consistent with those on other platforms is doable.  From what I've seen so far, it looks as if implementing a good analog of the cmd- key for Atom on Linux is feasible.  My testing hinted that it is quite practical to do so, though there are some potential sticking points in the "native!" key-bindings and the issues with triple chording noted by @t1bb4r.
Sorry that last paragraph is rather wordy.  I'll try to put the issue more succinctly (at the risk of coming off as arrogant and inflammatory, which I wish to avoid).  Atom's standard key-bindings are quite good, and they make rather heavy use of the cmd- key.  Linux has been deemed not to have an appropriate functional equivalent of the cmd- key.  Atom's Linux key-bindings work around this deficiency, cramming bindings in a smaller palette, and compromising their quality.  It would be wonderful for Linux users and developers alike to have a fully functional analog to the cmd- key, and it appears to be possible to provide such a key.  The feasibility of this solution is as yet unclear; some potential obstacles have been identified, but I've yet to run up against them.
I'm not satisfied with Atom on Linux.  It appears to be a somewhat hobbled version of Atom on other platforms.  If this issue really is just one of documentation and that one is expected to accept Linux Atom more or less as is, I'm inclined to cut my losses and move on to other tools.

I got it working with gnome 3.18.3.
In tweak tools I've set "Alt and Meta are on Alt keys" , which makes the "Windows" key act as cmd- .

Hello guys!
Sorry, I don't buy that Linux doesn't have a meta/cmd key. For example emacs has always had a meta key. And I can't find any keyboards in store that lacks a cmd/win-key. I know we have some keyboards at home from before mid-90 that lacks this key, but in that case it should be possible to use ESC or something.
We can't afford to lose 1/3rd of the modifier keys!  And we can't expect that package creators will add shortcuts for platforms they rarely or never use. Maybe it would be most correct, but problem is that it is never gonna happen. Can't we just be pragmatic and realize that? I really don't buy any arguments presented so far on this. It would be so much easier if we had the same keyboard layout (or at least a choice to do so) on every platform.
But at least it should be possible to use the meta-key on linux, even with a little different layout. (I can see that for some key bindings, it is preferable to go for the platform default one, for example ctrl+c vs. cmd+c for copying)
If it is true what @t1bb4r says, this should be easy to fix. And if it is a Chromium issue, we should fix it in Electron/Chromium. It is so extremely useless and painful to only have alt and ctrl on Linux while the same editor has three modifier keys on your Windows-box and OS X laptop.
Atom is so good! It has replaced Emacs for me. But Atom is also the single peace of software vendor-locking me to Apple/OS X now.
Am I the only one with this view? Sorry for just repeating everyone else, but this gives me so much frustration!

Well there is this chromium issue https://code.google.com/p/chromium/issues/detail?id=95874 maybe we can convince the chromium developers to have another look at the meta key for linux.

After reading the non-solution above and the apparent level of investment by the Atom team and upstream Chromium developers I was able to arrive at the following solution for all Debian-based platforms I maintain:
sudo apt-get remove atom

Have to agree with @cschooley and it's now 8 months later. Basic keybindings don't work in Ubuntu and the difficulty in changing even basic keybindings is too overwhelming when it should be dead simple. Back to Sublime. I'll check back with atom in another year I guess.

Following in the footsteps of @cschooley and @wtgee due to this issue. Moving on to Emacs.

Back to sublime, after wasting hours on this.


git-init-submodules: make sure to always complete

Don't ignore if git-init-submodule fails to initialize submodules or is
interrupted by the user and keep trying to complete.
For that we keep the .git/modules-ok control file to be sure if
submodule initialization process is sanitized.
We force a [re]initialization case .git/modules-ok is not found.
Signed-off-by: Leandro Dorileo<EMAIL_ADDRESS>

:+1:

:+1:

ok, applied... thanks.


[Request] Ideas for New Challenge Modes

I do not know how likely/easy to implement any of these are but after playing these modes for about 1 year I have LOVED this module and have some suggestions for more ideas.
No Cash
Player gold is always locked at 0gp 0sp 0cp. This means you cannot pay for repairs or buy items. Makes hunter especially interesting as must craft own ammo.
No Loot
Disable the ability for the player to loot corpses and chests. This will challenge the player by removing many quests and all loot. You can still do some quests and you can still get items from quest rewards/vendors.
Snail
Player movespeed is set to 0.5 or 0.75 or something that feels about right. You cannot kite/run from mobs anymore. Positioning must be extremely careful.
No Respawns
NPCs, objects, chests, etc do not respawn in the world. Once an NPC is killed it stays dead forever.
Eventually zones will be empty/safe, but if you didn't loot a low % quest item (bad luck) there's nowhere else to get it.
Once you do a dungeon, the dungeon is done.
Hunted Faction
Set player character's faction to one that is hostile to all factions. Player is attacked on sight by all NPCs, all mobs, etc.
This is the ultimate grinding/survival mode. No town is safe.
You'd need a way to train new skills to make this work.

Once you reach max level, the challenge "ends" and you can now die and resurrect. Challenges player to get to 80 without dying but doesn't punish once you are max.
Item Qualities
Inspired by Item Quality (White/Grey only) - maybe a bitmask for item qualities allowed
Item Quality: White/grey/green only.
Item Quality: Blue only (basically naked until lvl 20 then you have to figure out where/how to get some gear)
Item Quality: Purple only (basically naked until lvl 35 then you have to figure out how to gear)

Wonderful ideas


feat: support anthropic (non-streaming only)

:rocket: This PR description was created by Ellipsis for commit 584eb5c8eefbac185322f50a043060f5d65fd1a6.

Summary:
This PR adds support for the Anthropic AI model to the Instructor library by introducing a new mode and utility functions for handling XML, and includes tests for the new functionality.
Key points:

Added ANTHROPIC_TOOLS mode in instructor/mode.py
Modified instructor/function_calls.py to handle the new mode
Added utility functions in instructor/anthropic_utils.py for JSON to XML conversion and vice versa
Modified instructor/process_response.py and instructor/retry.py to handle the new mode
Added new dependencies in pyproject.toml and requirements.txt
Added tests for the new functionality in tests/anthropic/test_simple.py

Generated with :heart: by ellipsis.dev

let me know when this is good to merge :)

let me know when this is good to merge :)

should be good now

the follow does not work
from pydantic import BaseModel
from typing import List
import anthropic
import instructor

# Patching the Anthropics client with the instructor for enhanced capabilities
anthropic_client = instructor.patch(
    create=anthropic.Anthropic().messages.create, mode=instructor.Mode.ANTHROPIC_TOOLS
)

class Properties(BaseModel):
    key: str
    value: List[str]

class User(BaseModel):
    name: str
    age: int
    properties: List[Properties]

user_response = anthropic_client(
    model="claude-3-haiku-20240307",
    max_tokens=1024,
    max_retries=0,
    messages=[
        {
            "role": "user",
            "content": "Create a user for a model with a name, age, and properties.",
        }
    ],
    response_model=User,
)  # type: ignore

print(user_response.model_dump_json(indent=2))

@shreya-51 mind skipping the tests using pytest.mark.skip and changing the model to use haiku?

@shreya-51 what about streaming messages support? This is very important for user experience.


WIP: Add Wizard component

@LightOfHeaven1994 there is already a PR #117 from @Victoremepunto. What's the point to create another one?

@quarckster I can't remove Draft mark from his MR, for example, so I think I can't push changes to his branch then. That's why I opened this one and going to finish with the Wizard this/next week.

Closing this one as we agreed that it's a very complex to have it here. @mshriver is going to implement basic part of wizard without extra logic. After this we can use it in platform_ui repo to implement customizable part if needed


mirrord: 3.127.0 -> 3.128.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mirrord is: Run local processes in the context of Kubernetes environment
meta.homepage for mirrord is: https://mirrord.dev/
Updates performed

Ran passthru.UpdateScript

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 3.128.0 with grep in /nix/store/xgv45d6cx02yy9lh6m6z7xa3h9b5hhc3-mirrord-3.128.0
found 3.128.0 in filename of file in /nix/store/xgv45d6cx02yy9lh6m6z7xa3h9b5hhc3-mirrord-3.128.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

mirrord

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/xgv45d6cx02yy9lh6m6z7xa3h9b5hhc3-mirrord-3.128.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached 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 mirrord https://github.com/r-ryantm/nixpkgs/archive/bf513d5b2c2d77c9bfbd0ac0189ad7d428d2c9eb.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/bf513d5b2c2d77c9bfbd0ac0189ad7d428d2c9eb#mirrord

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xgv45d6cx02yy9lh6m6z7xa3h9b5hhc3-mirrord-3.128.0
ls -la /nix/store/xgv45d6cx02yy9lh6m6z7xa3h9b5hhc3-mirrord-3.128.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 result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

mirrord

Maintainer pings
cc @aaronjheng for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


[TIR] More consistent TIR printing

If you print the same primfunc twice it can output different results if they have the same name.. This is due to iterating over unordered collections. This fixes this.
Example:
https://github.com/AndrewZhaoLuo/TVM-Sandbox/blob/main/tir/test_inconsistent_tir_printing.py
Before the hashed text would be occasionally different, now it is all the same.

Thanks for doing this :-)


win32api.GetFileAttributes(f_file) error and Jupyter demo fix

The bug fix looks good, I am currently still running the consecutive parts to see whether there are other issues.
The Setting path part looks a bit weird though; especially when putting "not recommended" first.
I think we could do:
Setting Paths
Option 1: In Yaml File (Recommended)
...
Option 2: In Python
...
I feel like if it's really "not recommended", then should better just hide it

The consecutive parts finished, looks good.

Updated the jupyter notebook according to Yida's suggestion. @Yida-Lin do you think we are good to go now?


bug: telescope example in documentation doesn't work

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.2
Operating system/version
Ubuntu 22.04
Describe the bug
Adding a file called telescope.lua to my local plugins folder, as per the example.lua file, provides a new keymapping which works on initial boot, but subsequently fails.
-- telescope.lua
return {
  "nvim-telescope/telescope.nvim",
  keys = {
      -- add a keymap to browse plugin files
      -- stylua: ignore
      {
        "<leader>fp",
        function() require("telescope.builtin").find_files({ cwd = require("lazy.core.config").options.root }) end,
        desc = "Find Plugin File",
      },
  },
  -- change some options
  opts = {
    defaults = {
      layout_strategy = "horizontal",
      layout_config = { prompt_position = "top" },
      sorting_strategy = "ascending",
      winblend = 0,
    },
  },
  -- add telescope-fzf-native
  {
    "telescope.nvim",
    dependencies = {
      "nvim-telescope/telescope-fzf-native.nvim",
      build = "make",
      config = function()
        require("telescope").load_extension("fzf")
      end,
    },
  },
}

When I first fire up neovim, the new keybinding works as expected, albeit the "p" character is added into search box. If I change the keybinding, whatever character follows the <leader>f is similarly added into the search box.
After this, however, the keybinding is no longer recognised and doing <leader>fp resutlts in the same behaviour as fp.
I suspect the keybinding being lost and the final character being pre-entered into the search box are related, but I cannot workout why this is happening.
Steps To Reproduce

Add custom telescope.lua file to local plugins folder.
Open vim, use keymap <leader>fp to trigger new keymap. It works, but note that p is entered into the search box for you
Open any random file, and re-try to use  the keymap - it is no longer recognised.
nmap <leader>fp points to the generic which-key entry for "+file/find"

Expected Behavior
When using the new keymap it always works and is not overriden. Bonus characters aren't entered into the search box.

Your spec is wrong. Toplevel spec is telescope and under that you add another for telescope native. Fix your spec and next time, please use discussions for user questions

Should be something like this:
return {
  {
    "telescope.nvim",
    dependencies = {
      {
        "nvim-telescope/telescope-fzf-native.nvim",
        build = "make",
        config = function()
          require("telescope").load_extension("fzf")
        end,
      },
    },
    keys = {
      {
        "<leader>fp",
        function()
          require("telescope.builtin").find_files({
            cwd = require("lazy.core.config").options.root,
          })
        end,
        desc = "Find Plugin File",
      },
    },
    opts = {
      defaults = {
        layout_strategy = "horizontal",
        layout_config = { prompt_position = "top" },
        sorting_strategy = "ascending",
        winblend = 0,
      },
    },
  },
}

I'm sorry - but this is literally copied and pasted from the example file:
https://github.com/LazyVim/starter/blob/main/lua/plugins/example.lua#L53-L85
and was not working - hence my assumption it was a bug.

Upon closer inspection ti appears that a pair of brackets were indeed lost during the copy and paste - the example from the example file does work if they are included.