Forked domtokenlist

I don't understand why the domtokenlist is a fork.
Do you really need the UMD handling?
https://github.com/jwilsson/domtokenlist/compare/master...rodneyrehm:4ccd85f5662e674bd9a7d1a9e121eb49de42a14a?expand=1
We got some firewall issues which do not allow to install directly from github.
Do you think we can get your fork merged and use npm again?

Do you really need the UMD handling?

yes. The tests (using intern) require AMD. The build (using browserify) requires CommonJS. Until there is a decent support for ES6 modules in a test framework like Intern, I fear I'm stuck with this module mess.

We got some firewall issues which do not allow to install directly from github.

interesting.

Do you think we can get your fork merged and use npm again?

My code already landed in the original project, but was reverted before they released 1.2.0. Because of that I kept the fork alive.
As I was generally unhappy with domtokenlist-shim, I went ahead and implemented my own tokenlist. However, I did not want to just swap the implementations, without making use of the additional functionality my own implementation.
The domtokenlist-shim is currently only loaded in focus-source.js and focus-within. We currently only need .classList.add() and .classList.remove().
It is entirely possible to drop the polyfill altogether and use one of the older approaches to this (i.e. do the string manipulation ourselves). I believe this would be the best course of action for now.

Are you using the bundle or individual modules? Maybe it would make sense to publish the bundle separately without dependencies, as they're already included anyway. That doesn't help if you're using individual modules, though…

Right now we are using a small helper ts file which picks directly the focus source file out of ally.js.
/**
 * Allow to distinguish between tab and mouse focus interactions
 * @see http://allyjs.io/api/style/focus-source.html
 *
 * Usage:
 * ```css
 * [data-focus-source='key'] :focus {
 *   background: orange;
 * }
 * ```
 */
const focusSource = <() => void> require('ally.js/style/focus-source.js');
module.exports = focusSource();

The current solution is a small helper ts file which picks directly the focus source file out of ally.js

Does that mean you were able to work around the problem?

Only during development as npm install fails on the production machine

If you like I can prepare a pull request which uses string methods to add/remove the classes

#148 is now ready for review

Merged! Thanks for your efforts! :)
I imagine you'd like a new release asap. As I'm still not in a position to release the other changes in master, I'll have to create another release based off v1.1.1 only containing this fix. Since we're removing a dependency that had a global effect (DOM prototype), I'm not sure if this should become 1.2.0 rather than 1.1.2…

You mentioned somewhere that you weren't able to run the tests locally. What was the problem?

Yes sure it would be cool to release 1.2.0 soon.
It's either a feature release 1.2.0 (works without domtoken plugin) or a breaking change release 2.0.0. (doesn't include domtoken plugin)
Local tests:

Your contribution is now known as release 1.2.0 - Adios DOMTokenList - thank you for your efforts! :)

regarding your ability to run tests…

Are you by any chance using npm v3?

Thanks for the release 🎉🎉🎉
Moved the local testing problems into a new issue: #149
However although it is very slow it is also fine to work on a feature branch and let travis run the tests.


[tests-only] Fix posixfs flakiness

Fix posixfs flakiness

@aduffeck Interesting. How did that fix it?

@micbar the problem was that the integration tests create new users for each test, with different ids but the same username. That in return lead to the space roots having the same path according to the "users/{{.User.Username}}" path template, leading to clashes and confusion in the cache. With the "users/{{.User.Id.OpaqueId}}" template it is guaranteed that there's a clean state for each test run now.


CREATE PROCEDURE with morethan 1 parameter and WITH-clause is not working as expected

Description
Earlier syntax error is being raised when tried to create a procedure which contains more than 1 parameter and WITH-clause in it. This was happened because the remove of COMMA is not handled properly in case of create or alter procedure.
This commit fixes the above issue by removing the COMMA in WITH-clause but not in the parameters.
Issues Resolved
[List any issues this PR will resolve]
BABEL-3843
Test Scenarios Covered

Use case based - NA

Boundary conditions - Tested whether the create procedure accepts more than 1 parameters and WITH-clause in it.

Arbitrary inputs - NA

Negative test cases - NA

Minor version upgrade tests - NA

Major version upgrade tests - NA

Performance tests - NA

Tooling impact - NA

Client tests - NA

Signed-off-by: vasavi suthapalli<EMAIL_ADDRESS>Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Changes for procedures looks good. Don't we need to do same for functions as well?


Theme suggestions page should be linked in the sidebar

Currently, the only way to the suggestions page I can find is in the blog post. It should be reachable from the sidebar from every page.

It will be in the title bar, once event signups are working (i.e. how we make teams)

Okay, I put it in the title bar for now. When signups (and actives) are live, it'll replace them.


[AVM Module Issue]: Sql Server Module should add the creation of Microsoft.KeyVault/vaults/secrets

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
Module Name
avm/res/sql/server
(Optional) Module Version
No response
Description
Please add the creation of Microsoft.KeyVault/vaults/secrets.
In azd, we allow the user to pass a keyvault reference when creating a sql server. If they do provide one, then we set a keyvault secret that contains the sqlAdminPassword, appUserPassword, connectionString. Refer to this issue: https://github.com/Azure/bicep-registry-modules/issues/632
For example (azd):

https://github.com/Azure/azure-dev/blob/main/templates/common/infra/bicep/core/database/sqlserver/sqlserver.bicep#L100-L126

CC: @jongio
(Optional) Correlation Id
No response

This is connected to

https://github.com/Azure/bicep-registry-modules/issues/1934
https://github.com/Azure/bicep-registry-modules/issues/1780

Hey @NanaXiong00 & @jongio, this module avm/res/sql/server is currently orphaned 😢
Do yourselves or anyone you know want to take ownership of this module and make the the required changes to the module as per: https://github.com/Azure/bicep-registry-modules/issues/1934#issuecomment-2223349183 ?
#RR

@jtracey93 We will try to fix this issue.
@jongio for notification.

@jtracey93 PR is ready, please review.

https://github.com/Azure/bicep-registry-modules/pull/2859

as commented in the PR, this depends on https://github.com/Azure/bicep-registry-modules/issues/1934

According to the comments here: https://github.com/Azure/bicep-registry-modules/pull/2859#discussion_r1693915323, it seems more appropriate to set KeyVault Secrets outside the module. Should this issue continue?

According to the comments here: #2859 (comment), it seems more appropriate to set KeyVault Secrets outside the module. Should this issue continue?

Hey @zedy-wj,
please note: The comment only relates to the addition of a secret value to the interface.
The idea of the issue should be that the module can store all the service's credentials (e.g., access keys, connection strings, etc.) in a given Key Vault, if the user opts in. This is something that we already agreed on implementing, and I just need to maintainers to greenlight the suggested spec so that we can go ahead and implement the interface.
The commented line goes a step beyond that and not only does the above, but also would have the user be able to add custom secrets to that mix which I'd say should be done outside the module. In this case specifically, a password is added which could/should just as well be set outside the module's scope.
If we don't, I'm of the (subjective) opinion, that we're bloating the key vault secrets feature beyond the module's scope as one could essentially start setting any secrets via the module, even if they're completely unrelated.
I hope that makes sense. Happy to discuss :)


update the Slice bean which id and max should be accumulate in global

The number of slice should be  (numAllDocs - 1) / maxDocsPerPartition + 1 ,not the number of the shard / maxDocsPerPartition.

Can you explain why do you think this change is necessary ?
We compute the number of partitions per shard since each map is responsible for a slice inside a specific shard. So we need to ensure that the mappers responsible for a specific shard handle all partitions created for that shard (from 0 to max slice id). With your modification we'd miss documents since all slices wouldn't run on all shards.

Thank you very much for your reply.
I just made a test.
When slice.max < slice.max, only slice.id = (shardId % slice.max) has data.
When slice.max > shards.size, only (slice.id % shards.size) =shardId  has data.
I don't think that every shard needs to start so many map, many of which are unable to get data and end the task directly.
And I think slice.max should be caculate globally by (numAllDocs - 1) / maxDocsPerPartition + 1 .
Am i right ?

Thanks for explaining @wankunde . The fact that some slices get data and some don't is a bug in es that we fixed in https://github.com/elastic/elasticsearch/pull/29533. Each slice query targets a single shard but the slicing is computed as if all shards were part of the query and that leads to empty bucket. The fix will be released in 6.4 and should resolve the issue with empty slices.

No we can't, this code has the same issue than this pr. Each shard must see all partitions created for that shard otherwise you'll miss documents. A working solution would be to decorrelate partitions and shards so that each partition targets all shards but that's a big change for 5x. Since this issue is fixed in 6.4 I'd advise to wait for this release and in the meantime you can reduce the cost of the empty slices by reusing the mapper jvm:
mapred.job.reuse.jvm.num.tasks: 10

OK, Thank you @jimczi  for your help!


Tabs not properly expanded

Line 86 has
text = text.replace('\t', ' ' * tab)

This is the wrong way to this. For example, consider the following:
txt = 'a\tf\n\tf'

With the current implementation, this will result in f not being aligned
Should instead use the .expandtabs(tab) command

Thanks.


Make it case sensitive when get partition columns

Pull Request Description
Make is case sensitive when get partition column, since kusto take schema as case sensitive if I understand this right.
If I specify a partition column while the schema itself has upper letters, the query will fail.

Future Release Comment

None

Breaking Changes:

None

Features:

None

Fixes:

None

We can do that - although partitioning is not supported - this was some triage and its on hold until kusto will be able to partition on export


Moorhen should take whatever space is given when rendered inside a  not the whole window

I don't think we want this behaviour in every use case, only when instructed in props passed to MoorhenContainer. It will be particularly difficult to get this behaviour with the MUI drawers as they use the window as an anchor.


bug: Recent attempt to fix the double Lua LSP workspace loading progress seems to break Lua indexes in certain situations

Did you check docs and existing issues?

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

Neovim version (nvim -v)
NVIM v0.10.0-dev-2135+g98a4ed0a1
Operating system/version
Windows 11
Describe the bug
Related references / discussion:

https://github.com/folke/neodev.nvim/commit/c6be05aab078827e51aabdc64cc9fba7c06d27b7#r137665412
https://github.com/LuaLS/lua-language-server/issues/1596#issuecomment-1905757027
https://github.com/folke/neodev.nvim/commit/e7ca4a2ea0da5e39a639c08c3edb352b9355f09e

As mentioned here (https://github.com/folke/neodev.nvim/commit/c6be05aab078827e51aabdc64cc9fba7c06d27b7#r137665412)

I have problems using lspsaga peek definitions as it loads the peeked item in a floating window and then loads the lsp for that floating window... it seems to conflict with whatever changes were made in this PR as lua language server then reloads the lua workspace files specific to just that floating window (I think basically just loads the few files related to whatever I am peeking)... so then what happens is when I close the floating window my original file seems to have reference to just those limited lua workspace loaded files and all the sudden I get a bunch of diagnostics for missing references for vim.xxx global definitions. To fix this I have to run a LspRestart on that buffer.

As further context, @liskin workaround posted here (https://github.com/LuaLS/lua-language-server/issues/1596#issuecomment-1855087288) does work as expected and does not have this issue.
Steps To Reproduce

See below my .neoconf.json (main thing to notice is I have disabled importing of all plugins and only have a few imported with my lua workspace.
Add a line to one of your lazy plugin imports for setting a keymap like: vim.keymap.set("n", "<leader>TEST", "TEST", { desc = "TEST" })
Restart neovim
Use LspSaga to do a peek definition on the keymap keyword from step 2 using :Lspsaga peek_definition
Notice in my final screenshot below I (1) peek def keymap (2) loads def from neodev types nightly (3) the diagnostic of undefined globals now pops up in my original buffer

.neoconf.json
{
  "neodev": {
    "library": {
      "enabled": true,
      "runtime": true,
      "types": true,
      "plugins": [
        "nvim-treesitter",
        "plenary.nvim",
        "telescope.nvim"
      ]
    }
  },
  "neoconf": {
    "plugins": {
      "lua_ls": {
        "enabled": true
      }
    }
  }
}

Expected Behavior
The lua lsp workspace loader should not be duplicated, which is now working as expected... but now when I peek into other buffers, the lua workspace indexing seems to lose references most notably it seems to the core vim api stuff?
Repro
No response

I meet this issue..
the situation is ..
everything is ok at first time I open the file.
and after I do some operations , the lua_ls is reload double for this buffer.
and everything has gone..  lsp is no effective ..

I meet this issue.. 我遇到这个问题.. the situation is .. everything is ok at first time I open the file. and after I do some operations , the lua_ls is reload double for this buffer. and everything has gone.. lsp is no effective ..一切都消失了.. lsp 无效..

it's volar's issue

Is there any known workaround? The plugin is basically useless because of this


Android: .webp images are displayed black/partially when loaded from server

I noticed another issue when images loaded from server display partially or completly black in CachedImage. It seems to be related to .webp processing.
When I load .jpg instead of .webp from the server I have no issues.
Heres the log from loading .webp files:
This occurs very often on low-end devices (with less than 1GB ram).

D/ReuseBitmapDrawableCache(17126): OnEntryAdded(key = http://imageserver.com/picture46.webp;60x0)
D/dalvikvm(17126): WAIT_FOR_CONCURRENT_GC blocked 0ms
D/dalvikvm(17126): GC_EXPLICIT freed 3729K, 20% free 20948K/25991K, paused 5ms+10ms, total 61ms
D/ReuseBitmapDrawableCache(17126): OnEntryAdded(key = http://imageserver.com/picture65.webp;60x0)
D/STATUSBAR-NetworkController( 2203): refreshSignalCluster: data=-1 bt=false
E/WifiHW  ( 2035): ##################### set firmware type 0 #####################
D/WifiStateMachine( 2035): setWifiEnabled : WifiNative.setFwType(0)
I/wpa_supplicant( 2698): Scan requested (ret=0) - scan timeout 30 seconds
D/WifiService( 2035): startScan: false pid=2505, uid=10016
W/AlarmManager( 2035): FACTORY_ON= 0
W/PowerManagerService( 2035): Timer 0x3->0x3|0x0
D/SelfDisposingBitmapDrawable(17126): OnFreeResources
D/skia    (17126): --- decoder->decode returned false
D/SelfDisposingBitmapDrawable(17126): OnFreeResources
D/skia    (17126): --- decoder->decode returned false
D/SelfDisposingBitmapDrawable(17126): OnFreeResources
D/skia    (17126): --- decoder->decode returned false
D/ReuseBitmapDrawableCache(17126): OnEntryAdded(key = http://imageserver.com/picture14.webp;480x0)
D/ReuseBitmapDrawableCache(17126): Reuse pool is not full, refusing reuse request
D/ReuseBitmapDrawableCache(17126): OnEntryAdded(key = http://imageserver.com/picture32.webp;60x0)
D/ReuseBitmapDrawableCache(17126): Reuse pool is not full, refusing reuse request
D/ReuseBitmapDrawableCache(17126): OnEntryAdded(key = http://imageserver.com/picture53.webp;480x0)

To reproduce:

Load some .webp files from web (I use a Resolution of 480x480)
I started an Android Emulator and set the Ram Limit to 512MB.
The images appear to be partially displayed

http://s23.postimg.org/xk4hexpyz/webperror.png

It may be related to this:

https://bugs.chromium.org/p/webp/issues/detail?id=170
http://stackoverflow.com/questions/18842748/certain-webp-image-not-able-to-load-in-android-4
https://github.com/square/picasso/issues/177#issuecomment-32117960

@coonmoo Could you please indicate your Android version and also deliver an image that fails?

Thanks for the fast answer.
You are probably right, it's related to the issue because some Android versions do not support alpha.
Android Version is 4.1.2.
I'll send the Images to daniel-luberda because I can't attach them here.

Please have a look at: http://developer.android.com/intl/es/guide/appendix/media-formats.html
WebP lossy (Android 4.0+), so similar to JPG
WebP lossless, transparency (Android 4.2.1+), so similar to PNG
If you target a 4.1.2 device make sure your WebP image is lossy/without alpha.

I think we could close the issue.
BTW: Does anyone know a way for full support of webp in older Android versions?

@coonmoo This issue doesn't seem to be related to FFImageLoading.
Feel free to reopen it if you believe the opposite.

Thank you very much for the information.
I solved it with lossy Images for Android < 4.2.1

I'm not completly sure if not FFImageLoading causes some issues here.
I encoded all webp files not lossy without transparency.
I see some improvements here, but occassionally some images are still only partially displayed in CachedImage.
The reported issues:
https://bugs.chromium.org/p/webp/issues/detail?id=170
http://stackoverflow.com/questions/18842748/certain-webp-image-not-able-to-load-in-android-4
state that the images get not loaded at all.
So maybe there is an issue in FFImageLoading?

Could you send me a few pics that cause trouble?
fabien.molinet AT gmail.com

Hello fabien,
the issue is definitely with pictures loaded from a webserver.
I created a new xamarin.forms project and added 2 CachedImage controls. One displaying the Image from a local file and one displaying the same Image loaded from the Server.
Image
Code:
namespace ImageWebpTest
{
    using System;

    using FFImageLoading.Forms;

    using Xamarin.Forms;

    public class App : Application
    {
        public App()
        {
            var uri = new Uri("http://www27.online-convert.com/download-file/6ea211540e900e7c8b89f8c57fec724a/converted-f92e4295.webp");

            var img = new CachedImage { HeightRequest = 200 };
            img.Source = ImageSource.FromFile("imgwebp.webp");

            var cachedImg = new CachedImage { HeightRequest = 200 };
            cachedImg.Source = ImageSource.FromUri(uri);

            this.MainPage = new ContentPage { Content = new StackLayout { VerticalOptions = LayoutOptions.Center, Children = { img, cachedImg } } };
        }
    }
}

@coonmoo Sorry for the delay. Could you reupload the picture somewhere? The link no longer works.

@molinch thanks for looking into this issue.
Yeah the host only allows like 10times Access to the link.
I just uploaded the "webptest.webp" I sent you via mail through this formular: http://image.online-convert.com/convert-to-webp.
So when the below link expires you can upload it again.
Here is another link http://www24.online-convert.com/download-file/7935cd65a3a769c42ac186439e05d934/converted-c9066a69.webp

Ok I reproduce the issue on a Nexus 4 with Jelly Bean.
I confirm that it's related to your webp format: if I use a lossy webp then there's no issue.
lossy.webp.zip

Thanks for looking into this.
But if it's an Encoding issue, why does "ImageSource.FromFile("imgwebp.webp");" perfectly load, whereas the ImageSource.FromUri fails?

I really don't know why from disk it works, the only thing I'm sure is that it's related to Android version.
I tested on 4.4 and Lollipop and it worked perfectly there.


Update GCE resource reader

What type of PR is this?
/kind feature
What this PR does / why we need it:
Import resources needed by GCE provider.
Which issue(s) this PR fixes:
Fixes #87
Does this PR introduce a user-facing change?:
None

Wait until we finalized on how we should introduce gateway and ingress extension. See https://github.com/kubernetes-sigs/ingress2gateway/pull/148#discussion_r1591871746


chore: release main

:robot: I have created a release beep boop
plugin: 0.4.0
0.4.0 (2024-01-12)
⚠ BREAKING CHANGES

add full project details to plugins

Features

add full project details to plugins (17404c4)

Bug Fixes

bump detect-indentation from 6.7.0 to 6.8.0 (808a0bd)

Dependencies

The following workspace dependencies were updated

devDependencies

@rmtc/project bumped from ^0.0.0 to ^0.1.0

plugin-biome: 0.2.6
0.2.6 (2024-01-12)
Bug Fixes

bump @biomejs/biome from 1.4.1 to 1.5.1 (7c7a05c)

Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0

plugin-contributing: 0.1.4
0.1.4 (2024-01-12)
Bug Fixes

use the new plugin.project property (ff9e400)

Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0

@rmtc/plugin-eslint: 0.1.9
Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0

plugin-ignore-files: 0.1.2
0.1.2 (2024-01-12)
Bug Fixes

use the new plugin.project property (ff9e400)

Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0

plugin-mit-license: 0.1.3
0.1.3 (2024-01-12)
Bug Fixes

use the new plugin.project property (ff9e400)

Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0

@rmtc/plugin-mocha: 0.3.1
Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0

@rmtc/plugin-npm-scripts: 0.4.3
Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0

@rmtc/plugin-types-in-jsdoc: 0.2.7
Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0

project: 0.1.0
0.1.0 (2024-01-12)
Features

add a new project package (37c8087)

toolchain: 0.1.9
0.1.9 (2024-01-12)
Features

pass project details to the plugin loader (a19442e)

Dependencies

The following workspace dependencies were updated

dependencies

@rmtc/plugin bumped from ^0.3.0 to ^0.4.0
@rmtc/project bumped from ^0.0.0 to ^0.1.0

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-v0.4.0 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/project-v0.1.0 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/toolchain-v0.1.9 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-biome-v0.2.6 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-contributing-v0.1.4 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-eslint-v0.1.9 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-ignore-files-v0.1.2 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-mit-license-v0.1.3 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-mocha-v0.3.1 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-npm-scripts-v0.4.3 :sunflower:

:robot: Release is at https://github.com/rowanmanning/toolchain/releases/tag/plugin-types-in-jsdoc-v0.2.7 :sunflower:


Numbers and Wildlife panels no info

I'm, not sure how or why, but the first time I open the Numbers tab, it shows all the info of my pawns. After closing the tab, everything freezes for a moment, then resumes. I then check the Wildlife tab, and for a moment I can see the listed animals, but then all the info disappears, and I get this red error running over and over again:
Exception filling window for Numbers.MainTabWindow_NumbersWildLife: System.NullReferenceException: Object reference not set to an instance of an object
at Numbers.WorldComponent_Numbers+<>c.<.cctor>b__8_5 (Verse.Pawn pawn) [0x00000] in <33d2e6fa131c457b96140df2c637cc9e>:0
at Numbers.MainTabWindow_Numbers+<>c__DisplayClass23_0.<get_Pawns>b__0 (System.Func2[T,TResult] validator) [0x0000c] in <33d2e6fa131c457b96140df2c637cc9e>:0 at System.Linq.Enumerable.All[TSource] (System.Collections.Generic.IEnumerable1[T] source, System.Func2[T,TResult] predicate) [0x0002c] in <351e49e2a5bf4fd6beabb458ce2255f3>:0 at Numbers.MainTabWindow_Numbers+<get_Pawns>d__23.MoveNext () [0x0008f] in <33d2e6fa131c457b96140df2c637cc9e>:0 at System.Linq.Enumerable.Count[TSource] (System.Collections.Generic.IEnumerable1[T] source) [0x00052] in <351e49e2a5bf4fd6beabb458ce2255f3>:0
at Numbers.MainTabWindow_Numbers.DoWindowContents (UnityEngine.Rect rect) [0x00049] in <33d2e6fa131c457b96140df2c637cc9e>:0
at Verse.Window.InnerWindowOnGUI (System.Int32 x) [0x00165] in <36108d57f84d49cabdae818c2a53fc38>:0
Verse.Log:Error(String, Boolean)
Verse.Window:InnerWindowOnGUI(Int32)
UnityEngine.GUI:CallWindowDelegate(WindowFunction, Int32, Int32, GUISkin, Int32, Single, Single, GUIStyle)
I do have a ton of mods (including Fluffy's relationship mod, Better Pawn Control and the Zombieland mod. The Relationships mod is loaded after Numbers).
Are there any mentioned mod conflicts? I can check if those mods are in my list.

Do you have a Hugslib log?

Do you have a Hugslib log?

Sure, there were two errors, so I clicked on each and shared both logs (I'm not sure if they're the same one, but the second one was repeating over and over).
https://gist.github.com/HugsLibRecordKeeper/bcd60d608d975bab6c31eabc8561c850
https://gist.github.com/HugsLibRecordKeeper/b4e27973398de5b71094f7a7af7af095

Does it happen on a new colony as well?
If so, could you go into Steam => RimWorld (right-click) => Properties => Local files => Verify file integrity.

I made a new colony after verifying the file integrity (there were to issues found), and so far Numbers seems to work... I'll keep playing and see if anything changes.


embedded_frameworks always overwritten each time app.pods is called

Here's what i'm running into on macos.  In order to get the Fabric.framework to work correctly, it needs to be statically linked in.
In motion-fabric, we added the following lines for osx:
Motion::Project::App.setup do |app|
   ...
  if osx?
    # staticlly link the frameworks
    app.vendor_project('./vendor/Pods/Fabric/OSX/Fabric.framework', :static, products: ['Fabric'])
    app.vendor_project('./vendor/Pods/Crashlytics/OSX/Crashlytics.framework', :static, products: ['Crashlytics'])

    # make sure they are not copied into the Frameworks folder
    app.embedded_frameworks.delete_if {|item| item.to_s.include?('Fabric.framework')}
    app.embedded_frameworks.delete_if {|item| item.to_s.include?('Crashlytics.framework')}

This ensured that the frameworks were statically linked, and that they were not copied into the app bundle.  This worked great with motion-cocoapods 1.8.x
Now in 'motion-cocoapods1.9, theembedded_framework` variable is getting reset to all frameworks, causing RM to copy it into the executable.
I'm not sure how to fix this within motion-cocoapods.  I even tried deleting the framework before codesigning but get permission errors.
For now I've reverted back to 1.8.x...

This has now become a blocking problem for me in submitting to the Mac App Store, so now I need to dig into this.
@Watson1978 do you have any ideas on this?

What seems to be happening is that config.embedded_frameworks gets overwritten each time app.pods is used.  So when the motion-fabric gem (for osx), marks the frameworks as static and removes them from the embedded_frameworks, https://github.com/HipByte/motion-fabric/blob/7d18a19a0703422846656f509e5987347657ff32/lib/motion-fabric.rb#L96-103, they just get added back if app.pods is called again.
The quick fix for now is to add the line to remove them at the end of my Rakefile.  I'm not sure what, if any, fix would need to be in motion-cocoapods

As this is an old issue, I'm closing this out.


Update the Policy Generator to v1.2.3

This is to include the following bug fix:
https://github.com/open-cluster-management/backlog/issues/18236

/cherrypick release-2.4

/retest


config.lobby.newMember invoked before config.clientEvents.begin?

Looking at the sequence of server side events, it seems like lobby.newMember gets called before clientEvents.begin (newly connected user) does. This doesn't make sense to me but I'll work with it if that is indeed the way things are. Can you confirm? Thanks.

Yup looks like it: https://github.com/incompl/cloak/blob/master/src/server/cloak/index.js#L133-L138
I could see the case for swapping those in a new version. What do you think, @jugglinmike?

To clarify, this is obviously not a deal breaker but I was thrown off by the fact that one could be seen as entering a room (newMember) before one had established a first handshake with the server (begin). Hence my post. I could do a PR if you want.

The begin, disconnect and resumeclient events are related to the connectivity layer (e.g. web sockets), while the newMember is an application level concept (e.g room/lobby).
Maybe the documentation needs to be updated to make the distinction clearer.

@gamecubate I'd be happy to look at a PR for that!

Well, judging from @vstirbu's answer, this seems normal. I wouldn't want to rock the boat for so trivial a matter. Will close issue for now. Forget the PR. I have another question though, coming right up. :)


Spotify segment doesn't work on Windows

Prerequisites

[x] I have read and understand the CONTRIBUTING guide
[x] I looked for duplicate issues before submitting this one

Description
The spotify-segment doesn't work on Windows, at least with Microsoft Store version. Not only that, but the segment also breaks oh-my-posh3 when Spotify-process is not running.
Environment

Oh my Posh version: 3.34.0 beta
Theme: jandedobbeleer (and custom theme)
Operating System: Windows 10 20H2
Shell: PowerShell 7.1
Terminal: Windows Temrinal 1.4.3141.0

Steps to Reproduce

Install Spotify using Microsoft Store (haven't tested standalone version)
Open PowerShell and set theme to jandedobbeleer
Verify spotify icon shows stopped-icon
Play song in spotify - should still show stopped-icon
Close/exit spotify and write ex. cls
Oh-my-posh prompt is gone. Only showing PS>

Expected behavior:
Song playing: Spotify segment should show song
Song stoppped: Spotify segment should show stopped icon
Spotify not running: Spotify segment should not be visible
Actual behavior:
Song playing: Spotify segment shows stopped icon
Song stoppped: Spotify segment shows stopped icon
Spotify not running: Oh-my-posh prompt missing

The broken prompt when spotify isn't running might be same as issue mentioned PR #152, but it doesn't mention song-detection bugfix. MS Store-version of Spotify also shows song in title, so the concept should work

@fflaten merged the bugfix, can you check when that release drops? Should be pretty fast.

Great. It fixed the crash (missing prompt) and song is detected.
Would still like the Spotify-segment to be invisible when the process is not running. Currently shows stopped icon. New issue?

That's by design on Windows if I'm not mistaken.

I'd expect it to hidden when not applicable (no process running), just like git, python virtenv (iirc) etc.

I'd expect it to hidden when not applicable (no process running), just like git, python virtenv (iirc) etc.

That's how it works on MacOS. I'll adjust 👍🏻

I fixed it in my pull request

@fflaten @lnu merged 👍🏻

What a quick response - thanks! Now it's hidden when spotify is closed 🎉

I fixed it in my pull request
Please note that on windows, we can get the title and artist only when playing. There is no paused status as in macOS. So stopped is basically the same as when Spotify is closed.

That's correct, but we could still show stopped-icon. Did we go too far this time? Now paused/stopped song is no longer detected even though it is possible.
Stopped/paused can be identified by a spotify.exe process with a windowtitle that does not contain the track s- (and/or matches "Spotify*")
# Paused/stopped
Get-Process spotify | ? MainWindowTitle | ft ProcessName, MainWindowTitle

ProcessName MainWindowTitle
----------- ---------------
Spotify     Spotify Premium

# Playing
Get-Process spotify | ? MainWindowTitle | ft ProcessName, MainWindowTitle

ProcessName MainWindowTitle
----------- ---------------
Spotify     Phlake - Angel Zoo

How is this in MacOS? I'd guess the paused/stopped stated is shown with an icon as well. Windows should be as similar as possible since this is cross-platform. In this case I'd expect a stopped-icon since we can't show the song during pause/stop anyway.
Sorry for being so picky 😄

MacOS gives the ability to get the track title when it's paused. That doesn't seem possible on Windows. We could however resort to only show the playing state everywhere and disable the prompt in all other cases. Would make sense.

If a platform offers more capabilities like macos does, let’s keep it rather than going to the lowest common denominator?
What do you think?

rather than going to the lowest common denominator

Agreed. This way at least it's working as you'd expect. There's bound to be platform restrictions anyways.

I agree that MacOS should not be downgraded. It's the ideal scenario which other OSes should try to offer, as close as possible.
Closed vs stopped/paused is a useful distinction, so adding the stopped-icon on stop/pause is the best we can do on Windows - for now.
If you guys don't like it, then feel free to close the issue. The original bugs are solved 👍

The problem is there's no other info rather than a stopped icon which is rather sad (IMHO). Let's close this for now, if more people stumble upon this we'll revisit 😉


Add support for unit type literals ()

@Zett98 It doesn't need to be, it could be its own type, but unit is isomorphic to an empty record, no?

@Zett98 It doesn't need to be, it could be its own type, but unit is isomorphic to an empty record, no?

i haven't thought about it this way. I thought that with explicit Unit variant we would have an unboxed int representing this type, but with empty record we will have block with an int param


[Feature Request] Mass importer

Pretty straight forward after I export all of my databases to a folder I'd like a way to import them with the same easiness as I exported them. This makes life so much easy to just clone the a ravenDB without the database somewhere else.

@ogreenz Makes sense. However, is there something specific missing from the Raven.Backup tool that you would require in this mass-smuggler tool?

I know this is an old issue, but I would just like to point out that with the backup tool, not only does it makes very easy to import, with ravendb cloud (don't know if many people are using it), make it kinda pointless.
We are using mass smuggler for a older ravendb 3 instance. But even with that huge old database, we never found a use for an "Mass importer". Can we close this? :)

Agreed. Closing now because Raven 4 has built-in functionality for it.
Although, as you mention yourself @Malavos, it's still useful for 3.x instances.


Create the forms epic

Write an outline for the Forms area.
Create tickets for the work.
outline currently looks something like this:
HTML forms (25-30 hrs) (already available at https://developer.mozilla.org/en-US/docs/Learn/HTML/Forms, but this could really do with some improvement; in fact, I think this could do with being in a separate place to HTML, CSS, or JS, as it covers aspects of all three. The below article titles describe what I think the course should cover, not what it necessarily does cover)

    HTML forms overview

    Your first HTML form

    Making selections: radio buttons, checkboxes, selects, and datalists

    Special text fields: search, passwords, urls, and more

    Collecting numbers, choosing from ranges and dates (also include meters/progress)

    How to structure an HTML form

    Sending form data

    Form validation

    How to build custom form widgets

    Sending forms through JavaScript

    Basic HTML form styling

    Other articles to cover styling “the more difficult ones”

    Assessments

[ ] - check https://developer.mozilla.org/en-US/docs/Learn/HTML/Forms/Your_first_HTML_form "Smashing Magazine has very good articles about forms UX, but perhaps the most important is their Extensive Guide To Web Form Usability.
UXMatters is also a very thoughtful resource with good advice from basic best practices to complex concerns such as multi-page forms." to see if this is still good advice.


'link[rel*="alternate"]' seems to be way too strong assumption

https://github.com/igorlogius/list-feeds/blob/3f39814d19a13d5373fddc7d9f60a00b4e8112f1/content-script.js#L8
I might be wrong about JavaScript, but if I understand correctly, this requires the link to RSS/atom to looks like
<link rel="alternate" href="https://www.example.com/" />

However, I have encountered a lot of people just throw a plain <link href="/rss.xml"> link to their website and nothing more. Using rel="alternate" is for sure more "standard", but not everyone knows that.

The one used by livemarks seems to be better
https://github.com/nt1m/livemarks/blob/82666a51a53e7c6b2186b38789217b2998105808/contentscripts/feed_finder.js#L14-L17

I like standards, but i can see that many websites would not add the 'alternate' attribute
Although this will reduce the accuarcy but i guess that is not a huge dealbreaker.
The livemark xpath ... is nice ... but a little to "generous" for my taste.
I would expect a page to at least partially correctly identify a link with a mimetype.
I'll see what i can do.

fixed in version 1.7.7


No iterator type

Hi,
there is no Iterator type in @see https://esdoc.org/tags.html#type-syntax

ok, there is an undocumented Generator type


Rewrite of Windows console

This change, which is larger than I like, reorganizes the Windows console code. While it fixes some issues, it does not yet address the ANSI handling problems. We would like to get this merged into master and then tackle the ANSI code.
Signed-off-by: Brendan Dixon<EMAIL_ADDRESS>

@tiborvass @jfrazelle I really apologize for the size of this pull request. But restructuring the code in pieces would have been much messier.
The goals of the change are to:

Separate related code into stand-alone files (much code was intermingled)
Adopt more Go idioms
Standardize / improve invoking the Windows API (pattern now follows existing Go syscall)
Remove duplicate code / simplify logic as much as possible

I hope you'll find the code easier to understand. It does improve, a bit, on that which we shipped in 1.6, but the ANSI problems remain. Once this code is merged, another developer (who I am working closely with) will rewrite the ANSI handling to use a robust state-machine. Many of the remaining panics / bugs occur because of errors handling the ANSI sequences.

@brendandixon hey I'm not sure if it's still possible to do it, but it would be so much easier to review if you had one commit that simply moved files around without changing any content (apart from package name) and have other commits that actually change things (like i saw in the ANSI code).

@tiborvass Yah, sorry about that. Submitting multiple now would be very hard since I've continually rebased off of master (tossing the intermediate commit records). I appreciate the challenge the one large commit presents. What else could I (now) do to ease the review?

@brendandixon I guess it's for next time then :)
I tried to manually diff between term/winconsole and term/windows but it was cut up in smaller files (which is actually good). What could actually REALLY help, is if you could take master, and recut the files in the same files that are in this PR, so that I can do a diff file by file. Do you think that's possible?

@tiborvass I made the few changes discussed in the comments so far. And I sent you, via mail, a .zip with an attempt to map the original files into the new set. Sigh. I hope it helps. Again, I apologize for this. The old code was intermingled to the point that step-wise surgical changes were nearly not possible.

@brendandixon sorry needs a rebase :(

@tiborvass PR with changes based on comments.

@tiborvass I had to add back the pkg/term/windows/windows_test.go file to pass the Docker tests.

@brendandixon I tested the binary itself and I saw a couple of regressions.

Upon TAB, when autocomplete would list suggestions, it seems to not do anything if it can find multiple matches. It does autocomplete if there's only one match (e.g. ec -> echo). It's even more problematic because it was the best way I could test buffer problems.

When you do docker run -it busybox vi, the status line No file 1/1 100% should be at the bottom, it's however on the first line.

Ctrl+R doesn't work in the shell. On busybox it quickly blinks (reverse-i-search), and on ubuntu it's like no-op.

Ctrl+A (go to beginning) is buggy. On busybox it goes to beginning of displayed line, not to the beginning of prompt (after PS1), on ubuntu it does not insert the typed characters.

So I tried other things like Ctrl+H: it is not working properly on ubuntu (works on busybox). It takes the changes into account but does not display them.

On ubuntu, Ctrl+W displays things as expected, but when I paste with Ctrl+Y it only pastes the last word instead of the last consecutively eaten words. echo hello^W^W^Y -> should be echo hello instead of echo. On busybox, Ctrl+W is uber slow, and Ctrl+Y doesn't work.

Better news: the ^C behavior is working, so that's great! Minor difference between the OS X and Windows: on docker run --rm -it ubuntu if i do a ^C, there should be a newline after outputting ^C. If I hit Enter, I see then two newlines:

root@88b142428bc5:/# ^C

root@88b142428bc5:/#

I wonder whether it's about flushing a buffer. On busybox it works as expected (not outputting ^C, only a newline). Not sure how this is related to your PR (could not be).

That's all I got for now. If you could fix at least the TAB problem I could test the buffer overflows we used to have. Thanks!

@tiborvass If you really want, I can try to figure out why the TAB

@tiborvass If you want, I can try to figure out why the TAB is broken, but I strongly suggest we allow @gabrielhartmann to complete his rewrite of all the ANSI handling. Anything that worked before did so because it got lucky. I have shown this at the bits-and-bytes level. My re-worked, which correctly implements a few things, seems to have removed the luck that made other things work. The correct answer is correct code.
What do you think?

@brendandixon My problem is just that it's hard to justify merging something that breaks the user experience even more so. As much as it pains me to say it, but even if the code is cleaner now, it's less useful for a user than it was before. So I think we should fix at least:

vi (busybox), vim.tiny (ubuntu)
tabs

@tiborvass This puts me / us in a bit of a tough spot. The 1.6 code never worked; it got very lucky. Handling ANSI characters requires a state-machine (e.g., as the VT100 parser state diagram shows -- http://vt100.net/emu/vt500_parser.svg). The 1.6 code used a simple loop. I can break that code by starting vi after connecting a Ubuntu container. The byte stream (in that case) showed a sequence the code could not handle due to bad assumptions. I am surprised the code behaved as well as it did.
@gabrielhartmann is busily writing VT100 ANSI emulation that will work well. (In fact, we're writing it in hopes of getting it into Go as a package.) Attempting to make something that "worked" in 1.6 work again is actually a step backwards for us. If we must ensure no regressions at all, then we may need to pull this PR and resubmit with all the new ANSI code. But that would be a larger PR for you folks.
What do you feel is best? If we can merge this code, then @gabrielhartmann's life gets easier. I wish we were not in this position. :(

@brendandixon I understand the situation is complicated :S
It's awesome to have a VT100 ANSI emulation package, i really can't wait for it. I also suggest we merge that in Docker before waiting for it to be in Go (that may be tough, but totally worth it).
I also think a basic VT100 emulation would be more than welcome, so as to get the problems mentioned above work. If possible would love to have that in this PR as two separate commits: 1 for the package, 1 for the lines that use the package. Realistically this PR being pretty isolated, should not be conflicting with other PRs that we merge.
When do you think a very basic VT100 emulation package could be ready?
Again this is my personal opinion and others are welcome to voice their opposite opinions (/cc @jfrazelle @icecrime).
Either way, I'm super happy to have you guys contribute. It's gonna be huge. Not just for Docker users, but for the whole Go community :)

@tiborvass Ok. Here's what we'll do. I made @gabrielhartmann a collaborator on this fork. He'll add, as a new commit, all his ANSI work. It'll take him a week or two as there is a lot to it. So, let's let this sit for now.
Ok?

@brendandixon works for me! Really sorry about this, I should have started by testing the binary.
Anyway, I'm sure it will work out well :) Just remember that the priority, for this PR at least, is to have something usable, not feature-perfect. As soon as we have something usable we can merge this PR and open more PRs to increase ANSI support.
If it will take him a week or two, should we close this PR and reopen it when it's ready for another review? if you close it, then you can reopen it anytime (gotta do so before pushing commits though).

@tiborvass I'd prefer leaving it open for now. Let's see where we are by the end of the week. @gabrielhartmann is dedicated full-time to the work.

@brendandixon ok

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "win-cli/redo" git@github.com:brendandixon/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~59
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

This will update the existing PR, so DO NOT open a new one.

@tiborvass We're going to cancel this PR for now. We're making good progress on the ANSI, but it'll be easiest if we pull this until we're ready.
The code already looks much better and is fixing some latent ANSI issues.

@brendandixon glad to hear that! Will you open a new PR then or reopen this one and force push?

@tiborvass I feel like a new PR may be cleaner. Do you have a preference?

@brendandixon i would usually say let's continue on this one, but since it already reached the 120 messages I think we can start clean :)


Hscript help

Describe your problem here.
I'm trying to add this line of code into a lua script but it never works. How do I fix it?
addHaxeLibrary('FlxMath', 'flixel.math')
runHaxeCode([[
     if (curBeat % 4 == 2 && game.SONG.notes[Math.floor(curStep / 16)].mustHitSection && game.dad.animation.curAnim.name == 'idle')
		game.dad.playAnim('idle');
]])

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

Messing with the haxe code directly imo is very finnicky, use lua instead to do what you wanna do. If you know how to code in haxe, it doesn't hurt to spend some time reading PlayState.hx or FunkinLua.hx if what you're trying to do can't be coded with lua.
I haven't used addHaxeLibrary before so take this with a grain of salt, but you could also try replacing "Math" with "FlxMath" instead. All of what you're doing here can also be used with lua, you can look around in the documentation to try and make what you have in that code.


Exclude merge commits

I've compared against the GitHub UI, and my results seem to line up, except for a few small variations that could be caused by slight differences in the edges of the query time:
GitHub results

My results

The results are different by 10 commits for Ladislav Thon,  which I can't explain and am a bit worried about, but I think overall the data is looking basically fine.

This PR also fixes a UI regression introduced by #421.

Merging, although a UI problem (perhaps introduced by #421?) means the community information and sponsor are missing from the UI.


Do we have these templates for the gov environment?

Hi , I am trying to automate the deployment of Crowdstrike in our AWS infrastructure, While going through the template I see that the CrowdStrike cloud listed does not have Gov cloud. CrowdStrikeCloud: Default: us1 Description: The CrowdStrike Cloud your CID is hosted in us1, us2, eu Type: String AllowedValues: - us1 - us2 - eu Is it supported?
I think it would be great for the Gov cloud users to have these templates available for them as well.
https://github.com/CrowdStrike/Cloud-AWS/tree/main/Control-Tower/log-archive-acct
Thank you.
Regards,
Pradipna Gautam

Control Tower is now a part of AWS Builtin... For issues and questions, please see https://github.com/aws-ia/cfn-abi-crowdstrike-fcs


Can I get the map and the marker out of the box?

I wanted to add reverse geocoding by taking the lat, lng from the user and send it to the geocomplete .. which must give me the full address of this lat, lng.
The issue is that the geocomplete doesn't compute the address precisely .. so I needed to give the user another way to get the right full address. But, I am not able to get the map or the marker that geocompelete work with.
Does anyone have any idea about how to get the map and the marker that geocomplete use.

You can dig out the actual instance of the class with the real implementation given an element el with $(el).data('plugin_geocomplete')

Thanks, it works :thumbsup:


lab-specific calibration trials?

A handful of labs have an idiosyncratic name for their first calibration. Can we figure out if this is the star calibration or something else? The labs are:

labUNAM
cecBYU
childdevlabAshoka

working assumption is that this is likely lab-specific, not star calibration.

This is a wider issue, as we currently remove the "Eyetracker Calibration" media name, but I think some labs might have called their star calibration that


Binance us version

I have been trying for several days now to get this to work with binance.us, I found a thread that someone figured out how tweak this to make it work. I tried to replicate what was shown and I still keep running into API issues.
Anyone help me with this? I tried changing binance.js, package.json as well as package-lock.json with still the same API issues.
<3 plz send halp

It's duplicated issue - https://github.com/chrisleekr/binance-trading-bot/issues/146
Let's keep only one open.
Sorry, I haven't got a chance to take a look Binance US.


Uncaught TypeError: Cannot read property 'stdout' of undefined

[Enter steps to reproduce below:]

happens directly after installing
...

Atom Version: 0.201.0
System: C415M5CB3410TQN
Thrown From: autocomplete-elixir package, v1.2.6
Stack Trace
Uncaught TypeError: Cannot read property 'stdout' of undefined
At C:\Users\MRAUSC\.atom\packages\autocomplete-elixir\lib\alchemide\wrapper.coffee:19

TypeError: Cannot read property 'stdout' of undefined
    at Object.exports.init (C:\Users\MRAUSC\.atom\packages\autocomplete-elixir\lib\alchemide\wrapper.coffee:36:21)
    at exit (C:\Users\MRAUSC\.atom\packages\autocomplete-elixir\lib\alchemide\wrapper.coffee:27:22)
    at triggerExitCallback (C:\Users\MRAUSC\AppData\Local\atom\app-0.201.0\resources\app.asar\src\buffered-process.js:213:47)
    at ChildProcess.<anonymous> (C:\Users\MRAUSC\AppData\Local\atom\app-0.201.0\resources\app.asar\src\buffered-process.js:235:18)
    at emitTwo (events.js:87:13)
    at ChildProcess.emit (events.js:169:7)
    at Process.ChildProcess._handle.onexit (child_process.js:1057:12)

Commands

Config
{
  "core": {}
}

Installed Packages
# User
atomatigit, v1.5.1
autocomplete-elixir, v1.2.6
go-plus, v3.3.11
language-elixir, v0.4.0

# Dev
No dev packages

On it

It's probably an effect of issue #11

I don't see autocomplete-plus installed in Your packages. Are You sure You've got that installed?

No, I'm new to Atom. I assumed that it did dependency resolution itself, just like any other IDE...

@Hades32
Nope. It doesn't. autocomplete-elixir is a resolver for autocomplete-plus.
Please install autocomplete-plus and it should be fine.
Hit me up if it fixed the issue :smile:

Yeah, it works now! Thanks!
I'm pretty disappointed by Atom though, that it doesn't support plugin dependencies...

Give them some time. It's still young.
And to be honest, easiness of writting of plugins is absolutely amazing.
They should fix MacOS and Windows integration though, beacuse I see a lot of errors on these.
Especially MacOS.
Have a good day and peace from alchemists' community :cocktail: :grinning:

I still get this with the latest version 1.3.7

autocomplete-plus is installed (bundled with atom 1.0), getting this error since upgrading to 1.3.7 and 1.3.8

@fibric Really? I thought I fixed potential issue in 1.3.8

It is something related to failing to spawn the

what information from my machine do you need to find the root cause of this issue?

Stack trace tells not that much here. But I guess it could help. I regret they don't send local variables with that

https://discuss.atom.io/t/send-local-variables-with-stack-trace-when-sending-issue-report/18172?u=iraasta

/Users/fibric/.atom/packages/autocomplete-elixir/lib/alchemide/wrapper.coffee:31
Hide Stack Trace
TypeError: Cannot read property 'stdout' of undefined
  at Object.exports.init (/Users/fibric/.atom/packages/autocomplete-elixir/lib/alchemide/wrapper.coffee:31:19)
  at exit (/Users/fibric/.atom/packages/autocomplete-elixir/lib/alchemide/wrapper.coffee:19:53)
  at triggerExitCallback (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:213:47)
  at ChildProcess.<anonymous> (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:235:18)
  at emitTwo (events.js:87:13)
  at ChildProcess.emit (events.js:169:7)
  at Process.ChildProcess._handle.onexit (child_process.js:1057:12)

atom/atom/issues/6956

Does it happen anymore?
Also I've added Troubleshooting to the README.md file
Please check out if anything works for you from there


import error with tornado

I'm getting an import error when I have tornado version 6.0.1 and 6.0.2 installed. If I downgrade it to 5.1.1 it stops appearing.

Traceback
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-3-137cb941d741> in <module>
----> 1 import hvplot.dask

/usr/local/lib/python3.7/site-packages/hvplot/dask.py in <module>
      1 from . import patch
      2
----> 3 patch('dask', extension='bokeh')

/usr/local/lib/python3.7/site-packages/hvplot/__init__.py in patch(library, name, extension, logo)
    153         setattr(intake.source.base.DataSource, name, patch_property)
    154     if extension and not getattr(_hv.extension, '_loaded', False):
--> 155         _hv.extension(extension, logo=logo)
    156
    157

/usr/local/lib/python3.7/site-packages/param/parameterized.py in __new__(class_, *args, **params)
   2540         inst = class_.instance()
   2541         inst.param._set_name(class_.__name__)
-> 2542         return inst.__call__(*args,**params)
   2543
   2544     def __call__(self,*args,**kw):

/usr/local/lib/python3.7/site-packages/holoviews/ipython/__init__.py in __call__(self, *args, **params)
    121             else:
    122                 from IPython import nbformat # noqa (ensures availability)
--> 123             from .archive import notebook_archive
    124             holoviews.archive = notebook_archive
    125         except ImportError:

/usr/local/lib/python3.7/site-packages/holoviews/ipython/archive.py in <module>
     14     # Jupyter/IPython >=4.0
     15     from nbformat import reader
---> 16     from nbconvert import HTMLExporter
     17
     18     from nbconvert.preprocessors.clearoutput import ClearOutputPreprocessor

/usr/local/lib/python3.7/site-packages/nbconvert/__init__.py in <module>
      5 from . import filters
      6 from . import preprocessors
----> 7 from . import postprocessors
      8 from . import writers

/usr/local/lib/python3.7/site-packages/nbconvert/postprocessors/__init__.py in <module>
      3 # protect against unavailable tornado
      4 try:
----> 5     from .serve import ServePostProcessor
      6 except ImportError:
      7     pass

/usr/local/lib/python3.7/site-packages/nbconvert/postprocessors/serve.py in <module>
     17
     18
---> 19 class ProxyHandler(web.RequestHandler):
     20     """handler the proxies requests from a local prefix to a CDN"""
     21     @web.asynchronous

/usr/local/lib/python3.7/site-packages/nbconvert/postprocessors/serve.py in ProxyHandler()
     19 class ProxyHandler(web.RequestHandler):
     20     """handler the proxies requests from a local prefix to a CDN"""
---> 21     @web.asynchronous
     22     def get(self, prefix, url):
     23         """proxy a request to a CDN"""

AttributeError: module 'tornado.web' has no attribute 'asynchronous'

As the error indicates this error has nothing to do with hvplot, it's an issue in nbconvert which has already been fixed in recent releases.

you're right, sorry my mistake.


[$250] Search - Search section loads infinitely when opened after logging in

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.68-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Open the Expensify app.
Log in with any Gmail account.
When logged in, tap on "Search" on the bottom of the screen.
Verify that search section and all the content in it, loads smoothly.

Expected Result:
Search section and all the content in it, should load smoothly when opened.
Actual Result:
Search section loads infinitely when opened just after logging in.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/0cf5eee6-fa3b-4a31-81c1-b32c71adcd7b

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021863519972369952192
Upwork Job ID:<PHONE_NUMBER>369952192
Last Price Increase: 2024-12-02

Issue OwnerCurrent Issue Owner: @akinwale

Proposal
Please re-state the problem that we are trying to solve in this issue.
Search section loads infinitely when opened after logging in
What is the root cause of that problem?
We called the search API with this jsonQuery, but the response returned an empty onyxData array.
jsonQuery: {"type":"expense","status":"all","sortBy":"date","sortOrder":"desc","filters":null,"inputQuery":"type:expense status:all sortBy:date sortOrder:desc","hash":243428839,"recentSearchHash":422547233,"offset":0}

The reason for the empty onyxData array might be that the backend validated the query as having the wrong format or something similar.
And since onyxData is an empty array, shouldShowLoadingState always returns false, which is why the loading state is not hidden.
https://github.com/Expensify/App/blob/9933888bd1576e2723560b938945b805b8a8d3d3/src/components/Search/index.tsx#L222
https://github.com/Expensify/App/blob/9933888bd1576e2723560b938945b805b8a8d3d3/src/components/Search/index.tsx#L309
What changes do you think we should make in order to solve the problem?
To resolve this issue, we need to do two things:

Double-check the JSON query to match the backend's requirements.
Handle the case where onyxData is an empty array.

https://github.com/Expensify/App/blob/9933888bd1576e2723560b938945b805b8a8d3d3/src/components/Search/index.tsx#L308
//src/components/Search/index.tsx#L308
+    if (searchResults === undefined && !isOffline) {
+        setShouldShowStatusBarLoading(false);

+        return (
+            <View style={[shouldUseNarrowLayout ? styles.searchListContentContainerStyles : styles.mt3, styles.flex1]}>
+                <EmptySearchView type={type} />
+            </View>
+        );
+    }

POC

What alternative solutions did you explore? (Optional)

cc @luacmartins @lakchote can you have a look into this?

@daledah the requestID shared here shows no errors for that request. Do you have expense data in that account?

@huult Could you clarify what you mean by "Double-check the JSON query to match the backend's requirements." in your proposed solution?

@akinwale Yes, I mean we need to check with the backend why this JSON query has returned Onyx with an empty error, and this resolved it

This is entirely internal. So gonna unassign BZ and C+

This is fixed


ERROR: can't find the current version. Please try again later

For the past week whenever I try to use youtube-dl.exe it has been failing. In the past I would just update it by running youtube-dl -U but now when I try to update it says it cannot find the current version and to check back later. Is there something wrong on my end?

Please instead follow my issue which does the regular template and has verbose output. Please close this ticket.
#26260


[Redesign] Change degree planning toolbar color

Make toolbar in this screenshot same as buttons on home page. Toolbar can be found in [...planId.tsx]

Is this what you're after?

Yes that's perfect!


[fix](routine load) fix be core dump while use routine load

Proposed changes
Issue Number: close #17221
be core dump while use routine load, _file_reader in be/src/vec/exec/format/csv/csv_reader.cpp::145 is nullptr, because of stream_load_ctx->pipe is not be initialized.
This problem was introduced by pr https://github.com/apache/doris/pull/16996
routine load will try to start a fragment to read data, fragment will read data from pipe, but pipe is not set.
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

cloud you  explain the reason?

@dataroaring this bug is introduced by me. And I add explanation in issue decription.

cloud you explain the reason?

The cause of the problem is explained in the issue #17221

run buildall


Windows cmd.exe and Powershell support

How can this be run on Windows cmd.exe and Powershell?

I'm not sure, it'll likely require changes to run in a Windows env. From what I've read, cmd.exe may not support ANSI escape codes before Windows 10, and in Windows 10 you may need to configure ANSI support yourself.

Ok. But Powershell does support ANSI escape codes. Check https://github.com/PoshCode/Pansies.

Ok. Have you tried installing or using on Windows? What errors did you get?

Please specify the steps to install the dependencies on Windows.

According to the code dependencies are installed by setup.py
clone the repository cmd or powershell  with :
git clone https://github.com/google/gif-for-cli.git
Change directory to gif-for-cli :
cd gif-for-cli
Then install the dependencies using setup.py:
python setup.py install

If you do that you'll end up with:

Traceback (most recent call last):
  File "setup.py", line 33, in
    os.symlink('../third_party', symlink_path)
AttributeError: 'module' object has no attribute 'symlink'

Due to some peculiarities with the way Python packing works, the install script has to temporarily create a symbolic link to that dir. It'll probably have to change for Windows.
If you guys want to work on a PR for Windows support it would be much appreciated. :)

Why is a symbolic link needed for third party dependencies at all. Can't it just be packaged as a single folder?

I needed to include a font for the export feature, and it's Google policy to put all 3rd party code in the third_party dir. I tried a number of different ways, but I couldn't get that directory to consistently be included in the .whl and .tar.gz distributions. The best way to ensure the needed files would be available after install was to make it a Python module, but I didn't want third_party to be a top level module name, as it's too generic, I wanted it to be gif_for_cli.third_party. After several failed attempts to map the third_party dir to that module name, it seemed the easiest/only way was to just add a temporary symlink.

Seems a bit daft but I guess you could add Windows support for symlink...though that would require an elevated command prompt to install.

if os.name == "nt":
    def symlink_ms(source, link_name):
        import ctypes
        csl = ctypes.windll.kernel32.CreateSymbolicLinkW
        csl.argtypes = (ctypes.c_wchar_p, ctypes.c_wchar_p, ctypes.c_uint32)
        csl.restype = ctypes.c_ubyte
        flags = 1 if os.path.isdir(source) else 0
        try:
            if csl(link_name, source.replace('/', '\\'), flags) == 0:
                raise ctypes.WinError()
        except:
            pass
    os.symlink = symlink_ms

As per Windows 10 Creators Update, elevated command prompt is not mandatory anymore to create symlinks. Refer, this.

Linux subsystem on Windows works with this at present.Type bash in cmd.exe (yeah it's still in cmd.exe) and deploy it like Linux.


Feature Request: English number.

Current version I saw that there is no feature for English Number in the calendar, only Nepali number is visible. Is there is possible to change those number to English.

The english calendar view has been added.
Use Option:
{
language: "english"
}
Example:
http://nepalidatepicker.sajanmaharjan.com.np/#languageEnglish


Address some yarn audit issues (for toolkit v8) (#199)

These target the 8.x branch; I'll make the corresponding change for "master" (v9) separately.
See the individual commit messages for the details of each change.
Related: https://github.com/liferay/liferay-js-themes-toolkit/issues/199

That's the low-hanging fruit, at least. Overall this PR takes us from:
111 vulnerabilities found - Packages audited: 144859
Severity: 42 Low | 33 Moderate | 36 High

to:
92 vulnerabilities found - Packages audited: 135399
Severity: 33 Low | 27 Moderate | 32 High

going to go through the same exercise now for v9 (ie. "master").


[LLVMCPU] Compilation failure for small softmax

What happened?
#executable_target_static = #hal.executable.target<"llvm-cpu", "static", {cpu = "generic", cpu_features = "", data_layout = "e-m:e-p:32:32-p10:8:8-p20:8:8-i64:64-n32:64-S128-ni:1:10:20", link_embedded = false, link_static = true, native_vector_size = 8 : index, target_triple = "wasm32-unknown-emscripteN", ukernels = false}>
#pipeline_layout = #hal.pipeline.layout<push_constants = 0, sets = [<0, bindings = [<0, storage_buffer, ReadOnly>, <1, storage_buffer>]>]>
#device_target_llvm_cpu = #hal.device.target<"llvm-cpu", {executable_targets = [#executable_target_static]}>
module attributes {hal.device.targets = [#device_target_llvm_cpu]} {
  hal.executable public @predict_dispatch_2 {
    hal.executable.variant public @static, target = #executable_target_static {
      hal.executable.export public @predict_dispatch_2_softmax_1x10xf32 ordinal(0) layout(#pipeline_layout) {
      ^bb0(%arg0: !hal.device):
        %x, %y, %z = flow.dispatch.workgroup_count_from_slice
        hal.return %x, %y, %z : index, index, index
      }
      builtin.module {
        func.func @predict_dispatch_2_softmax_1x10xf32() {
          %c512 = arith.constant 512 : index
          %c0 = arith.constant 0 : index
          %0 = hal.interface.binding.subspan set(0) binding(0) type(storage_buffer) alignment(64) offset(%c512) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<1x10xf32>>
          %1 = hal.interface.binding.subspan set(0) binding(1) type(storage_buffer) alignment(64) offset(%c0) : !flow.dispatch.tensor<writeonly:tensor<1x10xf32>>
          %2 = flow.dispatch.tensor.load %0, offsets = [0, 0], sizes = [1, 10], strides = [1, 1] : !flow.dispatch.tensor<readonly:tensor<1x10xf32>> -> tensor<1x10xf32>
          %3 = tensor.empty() : tensor<1x10xf32>
          %4 = iree_linalg_ext.softmax dimension(1) ins(%2 : tensor<1x10xf32>) outs(%3 : tensor<1x10xf32>) -> tensor<1x10xf32>
          flow.dispatch.tensor.store %4, %1, offsets = [0, 0], sizes = [1, 10], strides = [1, 1] : tensor<1x10xf32> -> !flow.dispatch.tensor<writeonly:tensor<1x10xf32>>
          return
        }
      }
    }
  }
}

Fails to compile on cpu with the error
sources/module_predict_dispatch_2.mlir:16:8: error: operand #1 does not dominate this use
        %4 = iree_linalg_ext.softmax dimension(1) ins(%2 : tensor<1x10xf32>) outs(%3 : tensor<1x10xf32>) -> tensor<1x10xf32>
       ^
sources/module_predict_dispatch_2.mlir:16:8: note: operand defined here (op in a parent region)
// -----// IR Dump After LLVMCPUTileAndFuse Failed (iree-llvmcpu-tile-and-fuse) //----- //

Steps to reproduce your issue
The failure can be reproduced with the following compile command
iree-compile sources/module_predict_dispatch_2.mlir \
        --iree-input-type=none \
        --iree-hal-target-backends=llvm-cpu \
        --iree-llvmcpu-target-triple=wasm32-unknown-emscripten \
        --compile-from=flow \
        -o /tmp/dispatch.vmfb

What component(s) does this issue relate to?
No response
Version information
Fails on current ToM IREE.
Additional context
No response

I think what is happening here is the softmax gets decomposed and produces and op like
consumerOp: %9 = linalg.generic {indexing_maps = [affine_map<(d0, d1) -> (d0, d1)>, affine_map<(d0, d1) -> (d0)>, affine_map<(d0, d1) -> (d0)>, affine_map<(d0, d1) -> (d0, d1)>], iterator_types = ["parallel", "parallel"]} ins(%3, %6, %8 : tensor<1x10xf32>, tensor<1xf32>, tensor<1xf32>) outs(%2 : tensor<1x10xf32>) attrs =  {lowering_config = #iree_codegen.lowering_config<tile_sizes = [[0, 0], [0, 0], [0, 0], [2, 2]]>} {
^bb0(%in: f32, %in_1: f32, %in_2: f32, %out: f32):
  %10 = arith.subf %in, %in_1 : f32
  %11 = math.exp %10 : f32
  %12 = arith.divf %11, %in_2 : f32
  linalg.yield %12 : f32
} -> tensor<1x10xf32>

Where the tensor<1xf32> cause the outer unit dimension to be treated as a "fastest varying dimension" and thus it gets a tile size of 2.

Oh not landed yet https://github.com/openxla/iree/pull/15021


Makes checkServerIdentity a valid tls_options.

This is useful for testing HTTPS access to a self-signed cert server.

Where is this option documented? I couldn't find it in Node's HTTPS documentation:

It came from tls.connect() document:
https://nodejs.org/api/tls.html#tls_tls_connect_options_callback

But are you sure it works? The documentation explicitly says that the following options from tls.connect() can also be specified.

A test or two would also help proving me wrong. :)

Yes, I verified. It works.

I'm not sure if just by adding the key to needle's tls_options string would work, because we currently check whether that option is undefined or not. And the documentation, in fact, says that if passed the value should be undefined.

I think, it says if hostname looks good for you return undefined other values means the hostname is bad.
Here is my testing code:
var tls_option = {
    ca: [fs.readFileSync(__dirname + "/server.pem")],
    checkServerIdentity: function checkServerIdentity(host, cert) {return undefined;},
  }

Oh yes, you're right. Thanks for the heads up.


Fix Bug Loading Remote Source Files

We've encountered an issue with loading source code examples from a remote end point.  In our case the playground is not rendered on the client initially.  When we do open the playground, we want the code visible immediately.  On the first interaction to open the playground the code is displayed as expected.  However, if we close the playground and attempt to reopen it, the code is not displayed.  Clicking the hide code/show code button to toggle the Playground code editor allows the code to be visible again.
The code seems to be properly loaded from the endpoint, yet its not displayed in the editor pain.
Consider the following video example:
https://github.com/freewheel/code-kitchen/assets/12767362/1ec76c91-63ea-4f39-9875-0abc2b10ba27
Solution: TBD different commit

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Shea Leslein seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing due to bad auhtor for first commit,  reopening in #23


Unable to build app after integrating, profiler_build_phase.sh script failure

Description
Having followed the integration steps (added linker flags, added run script), after building, i get the following error.
/Applications/Detox Instruments.app/Contents/SharedSupport/Scripts/profiler_build_phase.sh: line 16: syntax error near unexpected token `('
Running detox instruments version v1.11.b10669
Steps to Reproduce
Steps to reproduce the behavior:

Integrate detox instruments with an app
build the app

Environment*

Model: MacBook Pro 2017
OS: macOS 10.14.5
Xcode: Xcode 10.1

iOS Device (please complete the following information):

Model: iPhone XR simulator
OS: 12.2
Xcode: Xcode 10.1

Additional Context
Add any other context about the problem here.

Apologies, I found the error. I tested the commands on zsh but not on bash, and in this instance, the two shells deviated in these commands.

Please update to v1.11.10670 and let me know.

All good now!


event when the user input is rejected?

I would like to make the input blink red when the user hits a key that is invalid. For example: when they try to add a letter on a number masked input.
Is there an event that I can listen to do this?

found it that you can use postValidation with currentResult. It will be false com a rejected input.


Couldn't launch container after installing librairies

Hello I have aleady run an image of ml-worskpace to create a container. I am developping a webapp in python in this container so I have installed streamlit and bigquerry-api libraires and pycharm IDE.
Now i try to start this container but it gaves me this error :
Traceback (most recent call last):

File "/opt/conda/bin/jupyter-notebook", line 11, in <module>

sys.exit(main())

File "/opt/conda/lib/python3.8/site-packages/jupyter_core/application.py", line 254, in launch_instance

return super(JupyterApp, cls).launch_instance(argv=argv, **kwargs)

File "/opt/conda/lib/python3.8/site-packages/traitlets/config/application.py", line 844, in launch_instance

app.initialize(argv)

File "/opt/conda/lib/python3.8/site-packages/traitlets/config/application.py", line 87, in inner

return method(app, *args, **kwargs)

File "/opt/conda/lib/python3.8/site-packages/notebook/notebookapp.py", line 2085, in initialize

self.init_webapp()

File "/opt/conda/lib/python3.8/site-packages/notebook/notebookapp.py", line 1757, in init_webapp

success = self._bind_http_server()

File "/opt/conda/lib/python3.8/site-packages/notebook/notebookapp.py", line 1764, in _bind_http_server

return self._bind_http_server_unix() if self.sock else self._bind_http_server_tcp()

File "/opt/conda/lib/python3.8/site-packages/notebook/notebookapp.py", line 1790, in _bind_http_server_tcp

self.http_server.listen(port, self.ip)

File "/opt/conda/lib/python3.8/site-packages/tornado/tcpserver.py", line 143, in listen

sockets = bind_sockets(port, address=address)

File "/opt/conda/lib/python3.8/site-packages/tornado/netutil.py", line 168, in bind_sockets

sock.bind(sockaddr)

OSError: [Errno 99] Cannot assign requested address

2021-04-27 14:53:34,733 INFO exited: jupyter (exit status 1; not expected)

2021-04-27 14:53:35,735 INFO gave up: jupyter entered FATAL state, too many start retries too quickly

2021/04/27 14:55:01 [error] 113#0: *1 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: , request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:8090/", host: "localhost:8080"

2021/04/27 14:55:08 [error] 113#0: *3 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: , request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:8090/", host: "localhost:8080",

@ahmedbendebba1 Thanks for reporting and sorry for the issue. It seems like Jupyter is trying to use a port that is already in use by another service. Did you specify any custom Jupyter arguments/configuration? If you like, you can also share the Dockerfile with your modification, which might help to find out the issue.

@LukasMasuch Thanks for your response
Did you specify any custom Jupyter arguments/configuration. I am useing the default minimal version of mlworkspace by running docker run -p 8080:8080 mltooling/ml-workspace-minimal:0.12.1. And after runing the image I run a container and in this container I have develeopped an app without running the dockerfile of the app in ml-workspace. But i have run
pip install -r requirements.txt

and the requirements file is
streamlit==0.50.2
pandas==0.23.4
altair==3.3.0
chart_studio
pyarrow


BBC Documentaries is empty

http://bbc-rss.herokuapp.com/rss/both/upcoming/genre/PT006.rss

Apologies for losing sight of this issue. Links containing "PT0..." should contain format not genre. The app should be updated now.


Create Sidenav (rui-sidenav)

Directive
** Options
HTML
CSS

Pull Review: @tobalsgithub

@jdl031, what's your progress on this?


Ctrl keys to behave like Esc when tapped

I just wait for this functionality for available asap. This is a bit more important for vim users and spacemacs users.
Is it possible for now? How can i config it on Karabiner-Elements?
Thanks.

This functionality has been implemented, or work arounds found.  See these issues for more info.
https://github.com/tekezo/Karabiner-Elements/pull/247
https://github.com/tekezo/Karabiner-Elements/pull/170#issuecomment-257091685
https://github.com/tekezo/Karabiner-Elements/issues/165#issuecomment-258762846
https://github.com/tekezo/Karabiner-Elements/issues/103#issuecomment-258765038

@walt-r It looks like all these solutions require building Karabiner-Elements from source. Is this true?

@IanEdington Yes. Karabiner-Elements is under active development, so all releases are unstable anyway. FWIW I've beeng using #247 for a couple of weeks now without any problems.

gtk, ty.

I believe these are related in terms of implementation https://github.com/tekezo/Karabiner-Elements/issues/697


Method call parameters and casts unexpectedly coloured

Describe the bug
Curley braces working well but the colours also apply to items in C# such as casts and method calls. Maybe by design but my preference would be to exclude (perhaps Enhancement rather than Bug).
To Reproduce
The following statements show orange for the method calls and red for the int cast:
var item = await GetSingleAsync((int)id, cancellationToken).ConfigureAwait(false);
Other examples include class and method attributes including their parameters, and interpolated strings:
[HttpGet]
[Route("/register")]
$"{firstName} {lastName}";

Expected behavior
Use the default colour for curly braces and related brackets if not used for code blocks.

PS: Loving this extension and working well with Dark+ theme.

I've likewise noticed colorization of brackets for array declarations:
Object[] arrayVariable;

A low priority to be sure, I'm excited to see this coming together.


How to tune the code so as to get results only for say persons and car classes?

Hello @dbolya,
Yours is an amazing piece of work.
However, in my application, I need detection and segmentation only on a few classes like say person and car. Can you please let me know where to tune the code so as to get the desired results?
Thanking you in anticipation!

Hello @PareshKamble. Did you find a solution to your issue? If yes, it would be great if you could share it!

Hi @Param2302 ,
Say, if I want only person and car class and filter out the rest of the classes.
Kindly follow the below given steps:

From data/config.py line# 31 COCO_CLASSES, find out the class index of person and car classes which are 0 and 2 resp. You may also directly search in COCO_LABEL_MAP dictionary but, remember, you need to subtract 1 from the value (class index like class_index-1) and not the key of the dict.
Add the below given lines at line# 57 of layers/output_utils.py.

boxes = torch.cat((boxes[classes==0], boxes[classes==2]),dim=0)
scores = torch.cat((scores[classes==0], scores[classes==2]),dim=0)
masks = torch.cat((masks[classes==0], masks[classes==2]),dim=0)
classes = torch.cat((classes[classes==0], classes[classes==2]),dim=0)

classes should come at the end. It is important.
Remember, if you need more classes, go on adding it in boxes = torch.cat((boxes[classes==0], boxes[classes==2], boxes[classes==12], ...),dim=0). Similarly, do it for scores, masks and classes.
Optional: If you get any error saying that boxes, scores, masks or classes are empty,
then wrap the code between line# 58 - 120 in layers/output_utils.py by an if condition if classes.nelement()!=0:, just to let the code run even if the input image does not have the desired classes.
Note: I do not claim this is the only way. There may be a better one.

@PareshKamble  getting this error following your solution :
Traceback (most recent call last):
  File "./yolact/eval.py", line 1105, in <module>
    evaluate(net, dataset)
  File "./yolact/eval.py", line 885, in evaluate
    evalimages(net, inp, out)
  File "./yolact/eval.py", line 623, in evalimages
    evalimage(net, path, out_path)
  File "./yolact/eval.py", line 600, in evalimage
    img_numpy = prep_display(preds, frame, None, None, undo_transform=False)
  File "./yolact/eval.py", line 151, in prep_display
    score_threshold   = args.score_threshold)
  File "/content/yolact/layers/output_utils.py", line 57, in postprocess
    boxes = torch.cat((boxes[classes==0]),dim=0)
TypeError: cat() received an invalid combination of arguments - got (Tensor, dim=int), but expected one of:
 * (tuple of Tensors tensors, name dim, Tensor out)
 * (tuple of Tensors tensors, int dim, Tensor out)

@Marwen-Bhj what is the version of pytorch you are using?

Hi @areebsyed,
This seems to be altogether different RuntimeError.
I am not aware of this yet.
I suggest you to check the shapes of masks and boxes and see if its crop function is working fine for a few cases.
All the best!

Hi @areebsyed,
This seems to be altogether different RuntimeError.
I am not aware of this yet.
I suggest you to check the shapes of masks and boxes and see if its crop function is working fine for a few cases.
All the best!

Hi just figured out the issue; your solution above works for me on images having persons and cars only for example. But when there is a frame with none of the specifically selected classes, then it gives out the error I wrote above and as following:
Traceback (most recent call last):   File "eval.py", line 1105, in <module>     evaluate(net, dataset)   File "eval.py", line 881, in evaluate     evalimage(net, args.image)   File "eval.py", line 600, in evalimage     img_numpy = prep_display(preds, frame, None, None, undo_transform=False)   File "eval.py", line 151, in prep_display     score_threshold   = args.score_threshold)   File "/home/user/Documents/yolact/layers/output_utils.py", line 79, in postprocess     masks = crop(masks, boxes) RuntimeError: strides[cur - 1] == sizes[cur] * strides[cur] INTERNAL ASSERT FAILED at /opt/conda/conda-bld/pytorch_1573049310284/work/torch/csrc/jit/fuser/executor.cpp:175, please report a bug to PyTorch.  The above operation failed in interpreter, with the following stack trace:

@areebsyed I suggest use a try catch to evade the issue.

@areebsyed I suggest use a try catch to evade the issue.

@PareshKamble thankyou so much for your concern and quick replies.
I found the solution. It may also interest you!
https://github.com/dbolya/yolact/issues/218#issuecomment-577963416

@areebsyed Thank you.
The solution is interesting and simple.

Hi @Param2302 , Say, if I want only person and car class and filter out the rest of the classes. Kindly follow the below given steps:

From data/config.py line# 31 COCO_CLASSES, find out the class index of person and car classes which are 0 and 2 resp. You may also directly search in COCO_LABEL_MAP dictionary but, remember, you need to subtract 1 from the value (class index like class_index-1) and not the key of the dict.
Add the below given lines at line# 57 of layers/output_utils.py.

boxes = torch.cat((boxes[classes==0], boxes[classes==2]),dim=0)
scores = torch.cat((scores[classes==0], scores[classes==2]),dim=0)
masks = torch.cat((masks[classes==0], masks[classes==2]),dim=0)
classes = torch.cat((classes[classes==0], classes[classes==2]),dim=0)

classes should come at the end. It is important. Remember, if you need more classes, go on adding it in boxes = torch.cat((boxes[classes==0], boxes[classes==2], boxes[classes==12], ...),dim=0). Similarly, do it for scores, masks and classes.
Optional: If you get any error saying that boxes, scores, masks or classes are empty, then wrap the code between line# 58 - 120 in layers/output_utils.py by an if condition if classes.nelement()!=0:, just to let the code run even if the input image does not have the desired classes. Note: I do not claim this is the only way. There may be a better one.
Hope this helps you!
@PareshKamble，Hi, I am getting this error using your method
Traceback (most recent call last):
File "/home/hermione/src/yolact-mask/eval_mask.py", line 1115, in
evaluate(net, dataset)
File "/home/hermione/src/yolact-mask/eval_mask.py", line 895, in evaluate
evalimages(net, inp, out)
File "/home/hermione/src/yolact-mask/eval_mask.py", line 633, in evalimages
evalimage(net, path, out_path)
File "/home/hermione/src/yolact-mask/eval_mask.py", line 610, in evalimage
img_numpy = prep_display(preds, frame, None, None, undo_transform=False)
File "/home/hermione/src/yolact-mask/eval_mask.py", line 157, in prep_display
idx = t[1].argsort(0, descending=True)[:args.top_k]
TypeError: 'NoneType' object is not subscriptable

Hi @wafaer ,
It appears you are not getting anything in t variable.
Please check the output of t.

Hi @wafaer , It appears you are not getting anything in t variable. Please check the output of t.
@PareshKamble，Thank you very much for your reply, because I am a beginner in semantic segmentation, and I need to study the error report slowly. I use yolact to detect people and cars in the video. For frames with no people and cars, the pixel value is 0. In response to what you said, If you get any error saying that boxes, scores, masks or classes are empty, then wrap the code between line# 58 - 120 in layers/output_utils.py by an if condition if classes.nelement()!= 0:, I put if classes.nelement()!=0: in front of 58-120 and this error occurs. If you are interested, can you tell me the specific ideas, thank you very much.

Dear @wafaer you may also try solution suggested by @areebsyed here.

@PareshKamble，Thank you for your reply again, I'm very sorry to bother you.  I used the suggestion you found for me, and added [cur_scores[1] *= 0]to the line yolact/layers/functions/detection.py  83, the previous error still occurs。I am in output_utills.py, Add
boxes = torch.cat((boxes[classes==0], boxes[classes==2]), dim=0) after line py56
scores = torch.cat((scores[classes==0], scores[classes==2]), dim=0)
masks = torch.cat((masks[classes==0], masks[classes==2]), dim=0)
classes = torch.cat((classes[classes==0], classes[classes==2]), dim=0) to achieve only detection of people and cars, what do you have for my modification and the result I want Suggestions?

Dear @wafaer
Sincere apologies, but I do not have any solution on this at the moment.


new_benchmark

Matbench Pull Request Template
Thanks for making a PR to Matbench! We appreciate your contribution (like, a lot). To make things run smoothly, check out the following templates,
depending on what kind of PR you are making.
If you are making a benchmark submission (i.e., you have tried an algorithm on Matbench and want to appear on the leaderboard),
please use the template under Benchmark submissions.
If you are making changes to the core matbench code, data, or docs, please use the template under Core code/data/docs changes.
Benchmark submissions
Benchmark submissions can include a full benchmark on any of the benchmarks Matbench submits, as well as any subset of tasks within a benchmark (e.g., 3 Matbench v0.1 tasks your algorithm supports).
Brief description of your algorithm
You should a brief description of your algorithm in the pull request body. This can include any details you'd like.
Included files
If you are making a benchmark submission, please only include the submission as a folder in the /benchmarks directory with the format <benchmark_name>_<algorithm_name>. Your PR should have no other changes to the core code.
The submission should have these three required files, as indicated in the
docs:
Example
-- benchmarks
---- matbench_v0.1_my_algorithm
------ results.json.gz             # required filename
------ notebook.ipynb              # required filename
------ info.json                   # required filename

Please make sure each of these files has the information specified in the docs.
If you have other short/small files required for the notebook, please give a brief overview of what each one is used for and how to use it.
Label the pull request
Label the pull request with the new_benchmark label.
Core code/data/docs changes
Brief description of changes
Please include a brief description of the changes you are making, in bullet point format.
Tests
Indicate if your code requires new tests and whether they are included with your PR. ALL core code/data/docs changes adding new features must have new tests for them.
Closed issues or PRs
Indicate if your PR closes any currently open issues or supersedes any other currently open PRs.
Label the pull request
Label the pull request with the code or docs labels, depending on which one (or both) applies.

@cheng7258002 thanks for the submission, looks like one of the tests is failing though :(
Seems like it has to do with including a task that had no data recorded for it. You should be able to remove that task and (matbench_dielectric) and resubmit though! If that doesn't work please post the problems here and we can debug together

Note I have fixed the issues above by removing the tasks with empty data, regenerating the hash, and rewriting the file to disk (along with some other fixes, like data point values being lists not floats). But the underlying data is unchanged from @cheng7258002 original submission. I will merge here to keep the commit history but see cheng branch for more details.

See #248


populateFeaturesFiles.py: Syntax, comments, and python3 compatibility

Looking over the populateFeaturesFiles, I found several small things I could clean up.
Initially I'd wanted to be able to run it with my default python3, but it appears that kango doesn't support 3.x yet. At least this file runs fine though. ;-)

Yup, Python is definitely not my native language. Thanks!


refactor(opentrons-ai-client): refactoring api call and inputprompt component

Overview
refactoring api call and inputprompt component
close AUTH-402

Test Plan

Changelog

add custom hook
export a function as a util function

Review requests

Risk assessment

[ ] add test for useGetAccessToken
[ ] fix fetch functinality

@koji please rebase


Add a quicklook recipe and diagnostics for the EMAC model

Add quicklook recipe and diagnostic for the EMAC model

Tasks

[ ] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[ ] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[ ] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[ ] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[ ] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[ ] Please use yamllint to check that your YAML files do not contain mistakes
[ ] (Only if really necessary) Add any additional dependencies needed for the diagnostic script to setup.py, esmvaltool/install/R/r_requirements.txt or esmvaltool/install/Julia/julia_requirements.txt (depending on the language of your script) and also to meta.yaml for conda dependencies (includes Python and others, but not R/Julia)
[ ] If new dependencies are introduced, check that the license is compatible with Apache2.0
[ ] Add documentation for the recipe to the doc/sphinx/source/recipes folder and add a new entry to index.rst

If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.

Closes {Link to corresponding issue}

If this is still relevant to anyone, please feel free to pick it up.


📃 docs: Two typos

correct typos.
https://github.com/babyfish-ct/jimmer-doc/issues/18

@Enaium github真方便，提供了点击的方式就更新了PR


deploy dns cluster add on: stuck with pods pending

followed all steps to get to here: https://github.com/kelseyhightower/kubernetes-the-hard-way/blob/master/docs/12-dns-addon.md
when i try to List the pods created by the kube-dns deployment, i see my pods pending, has been that way for over an hour. I noticed at somepoint, for an unknown reason, the scheduler was down and i've since restarted it. But i have been unable to get these pods created and have absolutely no idea how to resolve it.

re generated and redistributed config files to nodes, seemed to fix it.


arm64 or armhf images

This is general question, what is the difference between arm64 and armhf builds in terms of usage?
Which is better (more stable) e.g. for creating a desktop environment?
Thanks

You should prefer to use arm64, but due to big RPI popularity and armhf some software might be only available for armhf. However, it is possible to extend arm64 with armhf userspace and have this binary compatibility.

@ayufan Thanks for that, I have an issue on latest 0.6.25, tried xenial-minimal
on armhf, I can do apt update && apt upgrade and install_desktop.sh mate everything is ok
on arm64 system is unbootable after apt upgrade or  install_desktop.sh mate without upgrade.
What is the best way to debug it (installing ~200 packages one by one with reboot in between seems unfeasible)?


Add a pkgconf file

This allows easy detection of the package through many common build systems.  For example from autotools you might use:
PKG_CHECK_MODULES([LIBKRUN], [libkrun >= 1.5.0])
to define $(LIBKRUN_CFLAGS) and $(LIBKRUN_LIBS).

Yeah, this is definitely a good idea, thanks!

@rwmjones could you please rebase so we get the greenlight in the CI?


[FIX #51] Allow to use different streams for Redis adapter

@davydovanton I've added specs and made changes according to review.

great work!


iOS :: First registration request on iOS fails, subsequent ones work

When opening the Safari on iOS (14.3+), the first request does not complete successfully.
If the user attempts again (using a different username), registration works as expected.

Solved in https://github.com/TechWebAuthn/webauthn-components/commit/97abde4b7a447a1453e0318dc443d12afe22a1ab


// jshint ignore:line not working

Just tried this in the demo on the home page as well and it seems to have no effect.
ignore:start / ignore:end does work

I'm having this issue with<EMAIL_ADDRESS>and the unused rule.
for ( var x of this.gen ) { // jshint ignore:line
}

       	Unused Variables:
       		x(175),

Oh, never mind my previous comment, it was just the --show-non-errors that displays unused variables even when ignoring the unused rule violation.


Add support for /about/spam and /about/edited endpoints.

I hope I committed to the right branch. If not, I'll re-add it on the correct one... :/
See also: #88 and #89

Sweet thanks!
Closes #89