".{less}" does not match less files, whereas ".less" does

Description
Using a pattern like  "*.{less}" with only 1 extension in curly brackets does not match files with that extension.  "*.less" works correctly.
Steps to reproduce
Add a pattern with only one extension in curly brackets to the lint-staged config:
    "lint-staged": {
        "*.{less}": [
            "prettier --write",
            "lesshint --max-warnings 0",
            "git add"
        ]
    },

Logs
$ npx lint-staged
  ↓ Stashing changes... [skipped]
    → No partially staged files found...
  ✔ Running linters...

Environment

OS: macOs Mojave
Node.js: 10.14.0
lint-staged: 8.1.0

Should it? Anyways it’s not an issue of 🚫💩 lint-staged since were using micromatch. Please consult to their docs and if you still think it’s a bug please report to them. Thanks!

Thanks for the hint. I consulted the docs, and it is indeed intended behavior (weird though):

Single items
Following bash conventions, a brace pattern is also not expanded when it contains a single character:
console.log(braces.expand('a{b}c'));
//=> ['a{b}c']

@fabb Single items in braces are not expanded, per the "Single items" heading in the "Escaping" section of the Braces docs. If you really want to use braces around a single item (for future additions, I guess?), preface it with a comma:
"*.{,less}"

No that‘s not necessary. It‘s just unintuitive, and I spent a few minutes trying to find out why my lint-staged rule didn‘t work.


Provide custom layout panels

Provide custom layout panels similar to those supported by the Lima project:
https://github.com/HTTP-RPC/Lima

LayoutPanel - abstract base class for layout panels
RowPanel - arranges sub-components in a horizontal line
ColumnPanel - arranges sub-components in a vertical line
Spacer - provides either fixed or flexibly sized filler
StackPanel - arranges sub-components in layers (z-axis)

Add factory methods for creating these components:
row(horizontalAlignment, verticalAlignment, spacing, alignToBaseline, insets, cells): Cell<C>
column(horizontalAlignment, verticalAlignment, spacing, alignToGrid, insets, cells): Cell<C>
spacer()
spacer(size)
stack()

LayoutPanel should support the following properties:

horizontalAlignment (LEADING, TRAILING, CENTER, FILL)
verticalAlignment (TOP, BOTTOM, CENTER, FILL)

RowPanel should additionally support the following property:

alignToBaseline

ColumnPanel should additionally support the following property:

alignToGrid

Both row and column panels will support a "weight" property that will be added to Cell and implemented as a constraint (a double value representing the percentage of leftover space that should be allocated to the cell).
RowPanel will support a notion of "constrained preferred size". This will allow sub-components to wrap. To implement this, the row layout manager will first set a sub-component's size (specifically width) before calling getPreferredSize().

Note that all layout panels must take insets into account. When a component has a border, the insets will be provided by the border.

This is done.


UI Enhancement and Bug Fixed in Story Page

Description

Hey @apu52

issue closes #1027

I have enhanced the UI and fixed the bug in the story page
Video/Screenshots (mandatory)

Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[X] I have performed a self-review of my code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have gone through the  contributing.md file before contributing

Additional context:

##Are you contributing under any Open-source programme?

[X] I am contributing under GSSOC'24
[ ] I am contributing under VSOC'24

@apu52 Please merge it

@apu52 Please merge this PR


DatePicker not working in browser

The DatePicker works in IOS and IOS Safari Browser but does not work on windows desktop browsers

This depends on the browser being used, Chrome for example would work, but IE will not. You are free to add your own date picker as required.

In my opinion, we should be supporting modern versions of IE with the XControls project, setting the bar at IE 9 or IE 10.

Let's discuss on Skype.


(refactor) merge .dockerignore files

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
.dockerignore file working only in the root context Any .dockerignore files in subfolders are ignored.
Tests performed by the developer:
I've run the test suite.
Tips for QA testing:

Create a file that should be ignored in a subfolder e.g. conf/sensitive.yml
Build the docker container on the development branch

docker build -t hummingbot .

Verify that conf/sensitive.yml is present in the container with

docker run --rm -it  hummingbot ls -la conf/

Build this branch with

docker build -t hummingbot .

Verify that conf/sensitive.yml absent in the container with

docker run --rm -it  hummingbot ls -la conf/

@shibanovp you are right but I don't think that the data was copied to the image since the Dockerfile is not moving that folder.
@rapcmia @nikspz I think that we can test this one next week based on the steps mentioned and merge it

I see that from b32b585795de895d0b74249ab546861fca891e60 conf/ not in the container.


Layout/MultilineHashBraceLayout auto-correct syntax error

Hi, I noticed that the auto-correct for Layout/MultilineHashBraceLayout sometimes causes syntax errors. This seems to happen when

the closing } is on a new line
the hash is inside a function call
the last hash entry ends with a comment

Expected behavior
Don't introduce syntax errors.
Actual behavior
Auto-correct for Layout/MultilineHashBraceLayout can introduce syntax errors when a line ends with a comment.
Steps to reproduce the problem
Run  rubocop --only Layout/MultilineHashBraceLayout --auto-correct test.rb where test.rb is the following file:
h = f({ x => 1, # foo
        y => 2, # bar
})

This is auto-corrected to the following
h = f({ x => 1, # foo
        y => 2,} # bar)

Notice that the closing ) of the function is now after the comment.
RuboCop version
Include the output of rubocop -V. Here's an example:
$ rubocop -V
0.57.1 (using Parser <IP_ADDRESS>, running on ruby 2.4.1 x86_64-linux)

Thanks for the feedback. It was reproduced at 0.57.2.


Adding support for syncing „all“ devices with one publication

This pull requests adds the option to sync all devices with one publication option, which makes the setup on our end a lot easier.

We also set this option as a default publication, so the initial setup is directly working without any specific publication. In combination with the %deviceId% placeholder, this gives us the flexibility we needed.
Would be great to see this merged and the plugin updated. As for now, we roll this out manually.

Thanks @Edubits for merging this. Are you planning to release a new version with this feature enabled?

Thanks @Edubits for merging this. Are you planning to release a new version with this feature enabled?


Aeon Labs Multisensor Temperature Displayed Incorrectly

I created HomeKit Devices for my several Aeon Labs Multisensors to display motion, temperature, humidity, and luminosity. The motion detection devices show correctly, and the humidity and luminosity values display correctly in the Home App. The temperature values, however, are incorrect. I have set the server to Fahrenheit which is the setting of the Multisensors. The temperature values displayed in Indigo are correct and in Fahrenheit (for example 67.9). The values displayed in the Home App are incorrect (for example 154). The difference indicates that the sensor temperature (already in Fahrenheit) was assumed to be in Celsius and therefore converted to Fahrenheit (i.e.; 67.9*(9/5) + 32 = 154).

Please post the results from Simulate HomeKit For Server Item so I can see.  If it's converting then I think one of your config settings may be backwards.  There are two settings for temperature, one is in the server config and the other is when you config the device there is a checkbox that says "Temperature is in Fahrenheit" and if this is NOT checked then it will assume it's Celsius and pass it to HomeKit without converting.

Sorry about that. I didn't see the individual device temperature settings. I checked the "Temperature is in Fahrenheit" boxes and now the displayed temperatures are correct.

Woot!


Manually detect/outline faces

Describe the enhancement you'd like
An option to manually select an area on an image with a face or some way to set a mark that there is a face.
Describe why this will benefit the LibrePhotos
If this could be implemented and connected with the facial recognicion to help it train, that could improme the accuracy of it.

Looks similar to #431


List all load balancers

This PR adds functionality LBList() for listing all Load Balancers
Co-authored by : Sayandeep Sen<EMAIL_ADDRESS>Signed-off-by: Palani Kodeswaran<EMAIL_ADDRESS>

@vtolstov Addressed your comments. Please take a look. Thanks.

merged as it simple change

@vtolstov Thank you for your reviews and quick response.


Added basic test for LoomProvider

Removed unused variables
Added web3js dep on devDeps

Rather than relying on there being a "SimpleStore" already contract in place on the loomChain, it might be an idea to use eth_sendTransaction to deploy one.


Tech Debt: some prod data must be a migration instead

###Description of the Tech Debt###
Right now, we have form data deployed after each migration, in schema/data/prod/form_001
This causes issues if some migrations rely on the proper form to be in the database.
Solution: make that data part of the migration, in a schema/deploy/data/form_001
This will prevent us from releasing migrations relying on data not present in the db

@dleard flagging this as blocking for a next release


Missing Information - Features - Readme

Could you please add more features in the features section in Readme.md.
For instance:

specific directory can be specified for the output.
stylesheet functionality

Mentioning all the features would make the project look more better and would be able to assist the user in knowing the functionality for getting deeper towards the code and commands.
Thank you.

Thanks for the feedback, and added more information to two functionalities!


cert_file param on conjur class errors when attempting to reassign ssl_certificate

Summary
Using the cert_file param on the conjur class (which only happens on the server) errors when attempting to reassign ssl_certificate. The module tries to reassign the ssl_certificate variable with the contents read from cert_file. Puppet variables are immutable once set, and so an error is thrown.
Note that if you use just ssl_certificate param it works since the logic above is circumvented
Steps to Reproduce
Steps to reproduce the behavior:

Run a puppet agent using the cert_file parameter on the server (Hiera or hardcoded conjur class in the manifest)

Expected Results
cert_file should be used as the source of the SSL certificate for Conjur
Actual Results (including error logs, if applicable)
The puppet agent run errors with:
Unfortunately I got an error when using cert_file :disappointed:
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Cannot reassign variable '$ssl_certificate' (file: /etc/puppetlabs/code/environments/production/modules/conjur/manifests/init.pp, line: 15, column: 22) on node 7c0b1985e297

Reproducible

[x] Always
[ ] Sometimes
[ ] Non-Reproducible

Version/Tag number
Unreleased commit on master at the moment of creation of this issue.
Environment setup
Can you describe the environment in which this product is running? Is it running on a VM / in a container / in a cloud?
Which cloud provider? Which container orchestrator (including version)?
The more info you can share about your runtime environment, the better we may be able to reproduce the issue.

Agent v5 on Windows Server 2019 (Windows container on a Windows VM in EC2)
Server v5 (In a container as a docker-compose service)

Additional Information
Add any other context about the problem here.

Reopened since the PR was reverted https://github.com/cyberark/conjur-puppet/pull/158

Code that caused this issue is no longer present in main branch. Closing.


Overlay requests

Hello. I have problem. For example, two requests with params :
first -
"expression": "/api/projects/1/documents/1/rename",
"method": "POST"
second -
"expression": "/api/projects/1/documents",
"method": "POST"
Waiting behavior that is each of them should get their response.But i have result, that each of them have response from second request.Can you tell me how to resolve this?
Best regards.

I found decision : "^/Reporting/api/projects/1/documents$". That was my sick knowledge of regular expressions and patterns :)

Hi @Zima04  I'm facing similar issue, could you please share the solution that you have applied? It would be greatly appreciated.


Zotero paper import modal

Implement decent UI for picking papers to import from zotero.
See figma for designs: https://www.figma.com/file/YxjVX46hwvsnxtOoA38bqwp7/Citation-Gecko?node-id=20%3A59

mvp now up and running on zotero-front-end branch


Seemingly incorrect parsing error

Using the attached SWC file (I had to change the extension to upload it here) and this snippet:
>>> import morphio
>>> morphio.Morphology("BasketCell.swc")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
morphio._morphio.MissingParentError:
BasketCell.swc:5164:error
Sample id: 5164 refers to non-existant parent ID: 0

But these are lines 5163-5165, there is not a single line in the entire morphology with parent id 0:
5163 2 -69.74903869628906 -21.408447265625 -0.26000022888183594 0.33000001311302185 5162
5164 2 -69.80905151367188 -21.6484375 -0.020000457763671875 0.3400000035762787 5163
5165 2 -69.82904052734375 -21.688446044921875 0.0 0.3199999928474426 5164

The morphology parses just fine with my own SWC parser.
BasketCell.txt

Problem solved by upgrading from morphio 3.3.3 to 3.3.6


Fix 0003-iteration Decision return type

Based on test case, the return type of the decision is a collection/list
of feel:number.

Please use this handy-link to evaluate the changes proposed with this PR, netting the tabs-to-spaces change: https://github.com/agilepro/dmn-tck/pull/16/files?w=1

cc/ @etirelli

OK, looks alright, and necessary.
I am mystified by the changes to the tabs.   Only the first tab on a line was change, but subsequent tabs left in.  Why?   I have a small preference for always avoiding tabs, but why modify only some of the tabs, and not all of them?

@agilepro sorry I believe I didn't used the IDE functionality properly, I've added commit 6170432 to address that.

@tarilabs thank you!


How to debug DataFire Actions?

I know this question will sound stupid, but I am developing a DataFire.Action using VSCode. How do I properly debug the handler? Thanks in advance.

What exactly are you trying to debug? I'm not familiar with VSCode, but a good old console.log inside of the handler should work.

I am trying to debug the execution of the handler. I know how to run it through command line, but using VSCODE (or any other IDE), I am able to debug step by step.
I found a "solution". I implemented a main that imports the action and calls the handler as myaction.handler()

OK cool. FYI you can also use action.run(input, context), which will validate input and do some error handling.
You can absolutely chain actions together. If you're using async/await, that might look something like this:
const getHubspotAccounts = require('./get-hubspot-accounts');
const addAccountToDB = require('./add-account-to-db');
const datafire = require('datafire');

module.exports = new datafire.Action({
  handler: async (input, context) => {
    let accounts = await getHubspotAccounts.run({}, context);
    for (let account of accounts) {
      await addAccountToDB.run(account, context);
    }
    return `Added ${accounts.length} accounts`;
  }
})

Thank you!


Remove coverage badge

Removes coverage badge since coverage.gitea.io is unused.

LGTM

drone fail is unrelated to this pr

#102 will fix the build

Build passes


[FEATURE MASTER] Close Account

Master ticket for Account Profile Update Password .

[x] Implement Facade GH-1187
[x] Implement UI GH-1188
[x] Implement e2e tests GH-1189

Created branch feature/GH-1579 as the feature branch.

[X] "Close Account" is located below "Email Address" in the MyAccount nav menu.

[X] 2 buttons on "Close Account" page, "Cancel" and "Close my account".

[X] Clicking on "Cancel" brings the user to homepage without closing the account.

[X] Clicking on "Close my account" presents the user with a confirmation dialog.

[X] Clicking on "Cancel" or the "X" button on the confirmation dialog or clicking outside the popup window brings the user back to the "Close Account" page.

[X] Clicking on "Close Account" on the confirmation dialog takes the user back to homepage, with a successful message, as an anonymous shopper.

[X] If user tries to login or register with the account that's closed, s/he gets an error message.

[ ] Signing out on "Close my account" page leaves the user on the same "Close my account" page. And if user clicks on "Close my account" from there, s/he even gets the popup window to confirm it. But s/he cannot actually close the account from there. Updated ticket https://github.com/SAP/cloud-commerce-spartacus-storefront/issues/1957


Missing VSIX file

Hello everyone,
I was about to test this SDK, but the VSIX file is missing or hidden. Is it on purpose?
Thank you for your answer.
Štěpán

Hi @tirnovar:
Could you have a try of the https://github.com/microsoft/vscode-powerquery-sdk/releases
And let us know whether it still got hidden from you or not
Thanks

Yes! It works. But it seems like an error on the cache level because I can see releases on mobile and not on PC. Thank you. I am going to test it.


Simplify snapshotter architecture for faster performance

In current snapshotter architecture, if building a snapshot fails for some reason, it is queued and tried again with next epoch and so on multiple time (there is a max limit) until it succeeds or reaches that limit.
With the new architecture and even with  current offchain-consensus architecture, since there are multiple snapshotters snapshotting for each project and there is a small submission window for each epoch, there is no need to retry snapshot generation after the submission window is closed as it doesn't serve any other useful purpose and missing data can be handled by self healing.
Simplifying the architecture and removing retry mechanism should make snapshotting much faster and simpler, reducing load on RPC node as well.

Fixed in https://github.com/PowerLoom/pooler/pull/20, closing.


Spend without 'Subtract fee from amount' check is not shown on transaction tab

Actual behaviour
Spend without 'Subtract fee from amount' check is not shown on transaction tab
Even for manyspend where one of operation with 'Subtract fee from amount' unchecked.
They are shown only after restart.
Desired behaviour
Spend without 'Subtract fee from amount' check should be shown on transaction tab
Version where reproduce: v.<IP_ADDRESS>-01a50d393
Environment: regtest
Steps to reproduce

Mint coins

Make spend without subtract fee from amount

Check transactions tab:

Fixed in https://github.com/zcoinofficial/zcoin/pull/581


Abstract repository

We're going to need to push to more services that just Github such as Redmine and Jira.  We need to extract the object that IssuePack::push() pushes to.

There's good information on implementation approaches in this SO article: http://stackoverflow.com/questions/29480569/does-ecmascript-6-have-a-convention-for-abstract-classes

@leedavidr Would you say that your recent work completes this task?


indentation for sublevels

it would be nice if the sub levels would be indented on a smartphone like in the desktop version of the drop down.
don't know though if this is supported on ios and similars.

it's just the demo that didn't work, at least for me


MAINT: rename s3_path --> s3_key and bucket --> bucket_name

To close #11
This is to be more consistent with aws/boto3 naming conventions.

I go ahead and merge this as there is already a conflict on the other PR, requiring it to be rebased.


Fix object storage path handling

Object storage path rules:

No single / or ., use empty string for root path
Need to use trailing / for a list prefix to distinguish a "dir/"

I was unable to create a backport for 1.20. @wxiaoguang, please send one manually. :tea:
go run ./contrib/backport 27024
...  // fix git conflicts if any
go run ./contrib/backport --continue


Prediction result from Azure image classification is rediculous

This is because the normalizer layer is missing from onnx model

To fix this, we need to set enable_onnx_normalization to true in AmlJsonSettingString when kicking off aml job, for both object detection and image classification.

Now we do not have AmlJsonsetting , we use the sdk, hence the sdk should have a way  to enable  enable_onnx_normalization


Added another recipe for removing trailing slashes

This one is using a before_request hook to strip the trailing slash from request.environ['PATH_INFO']. I prefer it to the WSGI middleware method because it leaves the API of the app intact. For example, in the StripPathMiddleware snippet, you can't later add a route to myapp because you'll get the following error: AttributeError: 'StripPathMiddleware' object has no attribute 'route'.

The middleware-problem can be fixed this way:
app.wsgi = StripPathMiddleware(app.wsgi)

but your example is still helpful. merging, thanks :)

Ah! I had no idea you could apply middleware to just the WSGI! Good to know.


Support all valid collection names

See #125.
We should store a mapping between PostgreSQL and MongoDB names in a settings table. While we don't necessarily need it for this issue, we will need it later, so let's add it now, in a relatively small task.
Let's create a table/collection _ferretdb_settings per MongoDB database / PostgreSQL schema. It will contain a single fjson-encoded *types.Document with the following structure:
{
  "collections": {
    "collection1": "table1",
    // ...
  }
}

($f fields omitted for brevity but should be present – that's a normal fjson-encoding of a *types.Document value).
_ferretdb_ prefix is reserved for internal FerretDB collections. Tables/collections with that prefix in the name should be omitted everywhere where we return a list of collections, etc.
Let's add a method to pgdb.Pool type to return a PostgreSQL table name for the MongoDB collection name. It should query _ferretdb_settings table and return the existing value if present. If not, it should produce a valid PostgreSQL table name for a given collection name. The result should be stable (i.e., should not depend on other collections, the total number of them, random numbers, etc.) and should be readable to make debugging easier. For example, it could use a shorted version of a name together with a checksum/hash of a full name (e.g. values_a4gf). That result then should be saved back in the _ferretdb_settings table in the same transaction.
We should then use that method in all places where it is needed.
Add a test for a very long collection name.
Validating collection names is not in the scope. Tigris is not in scope.
Create PR as early as possible to get early feedback.

We probably also could simplify our helpers that produce "valid" collection/table names in tests


[Bug]: The puppeteer error while building

Discussed in https://github.com/atapas/react-play/discussions/112

Originally posted by atapas April 28, 2022
@erayalkis
I am seeing this intermittent error in the log and it fails the build. Do you think, we have a fix for it? Starting this discussion, if you want to get to a root-cause, welcome. When you have a fix, let's convert this one to an issue.

I'll get on it ASAP, thank you for opening an issue for it!

@erayalkis One doubt I got so clarifying. A play creator can add play cover in 3 ways.

They provide a link to an online image in the play metadata while creating the play.
They drop a cover.png in the play folder.
They don't provide anything and framework add one default image.

I know in the case of 2 and 3 the social share of the play displays the cover image correctly on all the platforms. Have we taken care of the first case too in the code?

@erayalkis One doubt I got so clarifying. A play creator can add play cover in 3 ways.

They provide a link to an online image in the play metadata while creating the play.
They drop a cover.png in the play folder.
They don't provide anything and framework add one default image.

I know in the case of 2 and 3 the social share of the play displays the cover image correctly on all the platforms. Have we taken care of the first case too in the code?

Yep! The first case should work fine as well, but if I missed an edge case I'll create a PR to fix it ASAP!

If the new changes don't fix the error, I've read that downgrading react-snap usually fixes the error, so we could also try that!


feat(headless): Notify Triggers

https://coveord.atlassian.net/browse/KIT-765

Erhm I think this PR is polluted with the changes for query execution triggers ?
And is not strictly "notification" ?
Please fix the PR so that reviewer can get a clean diff.
Either by merging master back, or by targeting a different branch.

Once the Query triggers gets merged to master, I will clean up this PR by merging master into it.


feat: add optional attr support

Fixes #74
Adds support for appending a ? to the end of an attribute config entry to make it optional. Optional attributes will be checked if they exist, but if they are missing and rule::whitelist is set it will not fail validation.
I'll update docs once this looks good to a maintainer.

Oops, checks on node 12 failed, probably because I used ??. I'll update here in a bit.

I'll update docs once this looks good to a maintainer.

Do you still plan on adding documentation @tivac?

I'll take a stab at it! Will try to get to that in a little bit here.


Bug: Rowspan appears to not work

It appears that the rowspan feature does not work currently. Trying the example from the README looks like this:

Either it is my fault or or the API changed. I didn't set the rowspan option to true. For using rowspan, it should look like this:
// ...
.use(require("markdown-it-multimd-table"), {
    // Other options...
    rowspan: true,
})
//...


[SPARK-27247][SQL] If the logicPlan get from cached is invalid, invalid it and retry instead of throwing an assert error directly.

What changes were proposed in this pull request?
Catalyst cache some logic plan.
During relationConversions(Relation conversion from metastore relations to data source relations for better performance), if the logicPlan get from cached is invalid, it would touch an assert error.
However, if the cached logicPlan is invalid, we should remove it from cache and retry instead of throwing an assert error directly.
How was this patch tested?
UnitTest

@cloud-fan @gatorsmile

Can one of the admins verify this patch?

Could you add some tests?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Do you know how this can happen? Using assert means it should never happen unless we have a bug. I'd like to find the root cause instead of hiding the bug.

Could you add some tests before Jenkins run?

Do you know how this can happen? Using assert means it should never happen unless we have a bug. I'd like to find the root cause instead of hiding the bug.

I would find the cause.

Could you add some tests before Jenkins run?

I try my best.

Why this pr closed?


Slow input menu opening

Environment
On Firefox and Safari (not tested on Chrome)
MacBook Pro 2019
Proc: 2,4 GHz Intel Core i9 8 cœurs
Graphic Card:

AMD Radeon Pro 5500M 8 Go
Intel UHD Graphics 630 1536 Mo

Is this bug related to Nuxt or Vue?
Nuxt
Version
v3.0.0-alpha9
Reproduction
On https://ui3.nuxt.dev/components/input-menu demo page, just open any input menu and see the latency between the click and the menu opening.
Description
On any menu opening, small latency is feel-able and make the component very uncomfortable to use, especially on the nuxt ui website.
Additional context
Code causing the long style calculation identified are following :
(~40ms)
            const l = window.innerWidth - document.documentElement.clientWidth,

And following: (~40ms)
                i.value && (L.value = window.getComputedStyle(i.value).zIndex)

(code coming from bundles)
Finally, ~80ms
Logs

Would you mind opening this in https://github.com/unovue/radix-vue repository? 😬

@benjamincanac I have the same issue, and it does feel to me like there's a difference between using Reka UI directly, and using Nuxt UI
This is with Reka UI (500 items)
https://stackblitz.com/edit/gg5k1l?file=src%2FApp.vue
A similar setup in Nuxt UI (https://pkg.pr.new/@nuxt/ui@695224f) feels slower.
<template>
	<USelectMenu :items="numbers"	/>
</template>

<script lang="ts" setup>
const numbers = [...Array(500).keys()].map(x => 'num' + x);
</script>

It is indeed way slower with Nuxt UI, not sure why exactly. Will look into it!


ast: add Walk and WalkFunc

Codecov Report

Merging #9 into master will decrease coverage by 6.76%.
The diff coverage is 2.76%.

@@            Coverage Diff             @@
##           master       #9      +/-   ##
==========================================
- Coverage    69.3%   62.54%   -6.77%
==========================================
  Files          10       12       +2
  Lines        1541     1706     +165
==========================================
- Hits         1068     1067       -1
- Misses        410      576     +166
  Partials       63       63

Impacted Files
Coverage Δ

ast/ast.go
0% <0%> (ø)

ast/decl.go
0% <0%> (ø)
:white_check_mark:

ast/expr.go
25.49% <0%> (+1.63%)
:white_check_mark:

parser/expr.go
75.08% <0%> (-0.27%)
:x:

ast/walk.go
0% <0%> (ø)

ast/type.go
0% <0%> (ø)
:white_check_mark:

parser/parser.go
85.66% <100%> (-0.03%)
:x:

Continue to review full report at Codecov.

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


Set Hot test cases timeout to 5s

What kind of change does this PR introduce?
bugfix
Did you add tests for your changes?
no
If relevant, link to documentation update:
n/a
Summary
Timeout wasn't applied to dynamic tests resulting in random failures on the CI.
Does this PR introduce a breaking change?
no

Thanks


Add labels to run container

Adds labels that can be useful for finding the containers spawned on a host by this plugin. This can also enable automated tools to associate these docker-compose containers with the agent that started them.
Tested on our infra:

Question for maintainers: should this be behind a plugin config? I don't think so since it doesn't affect the behaviour of the container itself, but I'm happy to make it conditional if needed.

This is a great addition! :clap:
I would indeed add it behind a configuration, but have it on by default... the reason is two-fold:
1- whoever doesn't want/need the behaviour will be able to turn it off
2- it will make it simpler to not have to change all 75 tests and only turn it on in the feature-specific tests :wink: :wink:

Thanks for the feedback @toote, made it conditional (default true), updated the existing tests to disable the option, and added a new test.


Limit the use of the duplicate bazaar project title when adding a project

Hey, I just made a Pull Request!
When adding a project in a bazaar plugin its should not accept the duplicate project name (which already exist)
limiting the duplicate bazaar project title will help user to avoid the confusions
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

@freben @camilaibs please review and reopen it

@Rugvip can you justify then how other plugins (shortcuts , playlists) have duplicate validations?? And its approved.


New version: MiniTool.PartitionWizard.Free version 12.8

Description

Related #124954
Also remove following versions: 12.5, 12.6, 12.7

Checks

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

According to the worktree, there is only one version in the package folder.
How should I fix this problem?

The policy is one manifest per change conflicts with bulk manifest change.

Re-create the history with Git locally. This pull request will only submit version 12.8 of this package after force-push action.
I'll create more PRs for removing all previous versions.


Restore Plugin Toolbar Button Tooltips

Restores the tool-tips for the plugin toolbar buttons which were accidentally removed in https://github.com/runelite/runelite/commit/7025a9fc84cd8e70c694432638f3f3d91a0cc416#diff-eccf96162c7889418564d750d1e2d099.

I thought about different ways of restoring this feature but I decided to go with this way for now as this is the way it was done before.
Perhaps it may be better to make the NavigationButton name non-final and make the tool-tip final, and then add the tool-tips at each builder call? Currently the name isn't really used as far as I am aware.

I would probably remove the name or tooltip completely.


Return project IDs instead of names

A user reported that they were trying to load a project from their workspace but because it returned the names of the projects (which you can't do anything with) vs their ids which you can look up.

This is a quick fix to make that work more intuitively. But I think we should determine whether we actually want to return a list of Project objects vs strings. We have those available in this method.

@yeldarby can you rebase on master (so you have all the up to date code and the git hooks) ? :)


How I can login this page at frontend?I try to use sql.And user_login_histry table tells me I login success.But page and log tell me throws a exception.

I can konw what I login success。

Dear issue owner,
Thanks for your interest in WhereHows. We have recently announced DataHub which is the rebranding of WhereHows. LinkedIn improved the architecture of WhereHows and rebranded WhereHows into DataHub and replaced its metadata infrastructure in this direction. DataHub is a more advanced and improved metadata management product compared to WhereHows.
Unfortunately, we have to stop supporting WhereHows to better focus on DataHub and offer more help to DataHub users. Therefore, we will drop all issues related to WhereHows and will not accept any contribution for it. Active development for DataHub has already started on datahub branch and will continue to live in there until it's finally merged to master and project is renamed to DataHub.
Please check the datahub branch to get familar with DataHub.
Best,
DataHub team


Unclear why "Ipykernel setup required for this feature" is showing up for new setups and trying to run-by-line

Testing https://github.com/microsoft/vscode/issues/133903
Both Brigit and I hit this when testing via the test plan item steps.
For Brigit, things just started working, perhaps after updating the ipykernal version.
For me, things started working after I did a Run All and accepted the prompt to install ipykernal.
Might be an issue with the test plan steps but it is also confusing that the "Ipykernel setup required for this feature" message has a "Setup" button that takes you to documentation, while the "Run All" message allows you to install the missing components.

This would only affect older environments, not going to address it


Build error on Android

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':onesignal_flutter:generateReleaseRFile'.

In project 'google_ml_vision' a resolved Google Play services library dependency depends on another at an exact version
(e.g. "[10.2.1, 17.3.99]", but isn't being resolved to that version. Behavior exhibited by the library will be unknown.

Dependency failing: com.onesignal:OneSignal:3.16.0 -><EMAIL_ADDRESS>17.3.99], but fire
base-messaging version was 17.3.4.
The following dependencies are project dependencies that are direct or have transitive dependencies that lead to the art
ifact with the issue.
-- Project 'google_ml_vision' depends on project 'onesignal_flutter' which depends onto<EMAIL_ADDRESS>-- Project 'google_ml_vision' depends on project 'onesignal_flutter' which depends onto com.onesignal:OneSignal@{strictl
y 3.16.0}
-- Project 'google_ml_vision' depends on project 'onesignal_flutter' which depends onto com.google.firebase:firebase-mes
saging@{strictly 17.3.4}
For extended debugging info execute Gradle from the command line with ./gradlew --info :google_ml_vision:assembleDebug t
o see the dependency paths to the artifact. This error message came from the strict-version-matcher-plugin Gradle plugin
, report issues at https://github.com/google/play-services-plugins and disable by removing the reference to the plugin (
"apply 'strict-version-matcher-plugin'") from build.gradle.

everything is ok if I delete the following lines in gradle:
classpath 'com.google.android.gms:strict-version-matcher-plugin:1.2.1'
apply plugin: 'com.google.android.gms.strict-version-matcher-plugin'
why are they needed and what do they do?


Doesn't handle last,first

Does not properly parse a name formatted as "last,first" with no spaces around the comma. This results in the full name being classified as just the last name. Interestingly, if you add a middle name onto this, it does correctly parse all of the names: "last,first middle".
Simple fix for this is to change the following in the Parse method:
if (name.IndexOf(' ') == -1)
to
if (name.IndexOf(' ') == -1 && name.IndexOf(',') == -1)

Thanks for the report and fix.


Allow no location header with post response with not location specified.

Fixes #83.

Can you elaborate a bit on the underlying issue?

So I encountered this issue when trying to improve the validation of the restup a bit, but it's really a separate issue which is why I created a separate pull request.
Basically a response with HttpResponseStatus.Created is allowed to have no Location specified according to the HTTP spec. This is what we allow for as well in the PostResponse constructor.
The code was always creating a Uri using the LocationRedirect property, the Uri constructor would then throw an exception and cause a BadResponse from the server.
The fix is simply to do a string.isNullOrEmpty check in the code so that the request is returned successfully without the Location header present.
This is basically the underlying issue of #83
Hope this clarifies things a little bit :)


Use Ardalis.Specification and repository

Added ItemsByBaseFilterSpec and ItemsByPaginationFilterSpec
built on Ardalis.Specifications.
I used them in Brands and Producs search requests for now.
Tell me what you think!
If ok, I'll update everything to this (specifications and repository) and then we can throw away all our repository and specifications stuff which is a bit all over to place right now according to me ;-)
Only 2 extension methods were needed (SpecificationBuilderExtensions) to implement everything that is now scattered across all the files in Persistence/RepositoryAsync.
On top of it, it uses "sql like" functionality on the database (built into the ardalis library)...
There's also caching built in the library, but I'll have to check that out how to use it (or anybody that feels like it ;-))

I think it's done... please review carefully!
Whoaw it felt so good deleting those folders "repositoryasync" and "specifications" ;-)

Maybe the specs can even move to the Domain project? Not sure what's usually done here...

Maybe the specs can even move to the Domain project? Not sure what's usually done here...
Hmm... Ardalis's CleanArchitecture (and eshop on web) puts everything in one "Core" project... no split between application and domain. JasonTaylordev's CleanArchitecture does make the split, but doesn't make use of specifications...

In my opinion if the domain has the task of representing the context of the domain of the entities there would be no specs.
also taking into account that the classes contained in for the most part do not have much to do (apart from IEntity ...

Maybe the specs can even move to the Domain project? Not sure what's usually done here...
Hmm... Ardalis's CleanArchitecture (and eshop on web) puts everything in one "Core" project... no split between application and domain. JasonTaylordev's CleanArchitecture does make the split, but doesn't make use of specifications...

hmm, BaseSpecification depends on DN.WebApi.Application.Common.Models so not sure

You're looking at old code... BaseSpecification is no more ;-)
But anyways, I don't think the specs belong in domain either...

You're looking at old code... BaseSpecification is no more ;-)
But anyways, I don't think the specs belong in domain either...

it's still there in the repo...

but not in this pr ;-)

ah...doh!


Add 'Deployer' as owner of Function App Managed Identity (Enterprise App) during deployment

In order to stay compliant with certain Microsoft directives, we must regulate app permissions and authentication settings. Many of these configuration are done at the function app managed identity-level (or enterprise application). This identity is created indirectly during deployment and, consequently, is not created with an actual 'owner' who can manage the settings of the principal.
I'm requesting that we research how to add the 'deployer' of the instance in question as an owner of this enterprise application/managed identity.
AB#39994210

We have not control over the ownership. Closing.


add TextDocument tests for invalid inputs

adds more tests for TextDocument dealing with positions/offsets around line endings.
Ref #1285

@dbaeumer For you to review...

I'm not sure if this repo follows semver or not, but just a heads up that this PR (being released as 1.0.12 of vscode-languageserver-textdocument) ended up breaking things for us downstream
Probably that we were relying on broken behaviour, but still, lost a few hours tracking this PR, not helped by the title being "add tests" when it in fact, does not just add tests 😅

@Princesseuh sorry for the bad title. We do use semver but since this is a bug fix for a broken behavior (https://github.com/microsoft/vscode-languageserver-node/issues/1285) it is not a major version update.


navigate by hash results in context.DeadlineExceeded sporadically

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.8.2
$ go version
go version go1.18.3 darwin/amd64

What did you do? Include clear steps.

1. chromedp.Navigate(url) // like https://google.com
2. chromedp.Navigate(urlWithHash) // like https://google.com#hash1. this hash must be on html content of this url.
3. take screenshot
4. chromedp.Navigate(urlWithHash)
5. take screenshot
6. chromedp.Navigate(urlWithHash)
7. take screenshot

What did you expect to see?
3 screenshots
What did you see instead?
step 2 will result in context.DeadlineExceeded sporadically

I can not reproduce the issue. Can you provide a minimal reproducer?
Please note that, most of the time, chromedp.Navigate() won't return until the page is loaded (namely, it will wait for the Page.loadEventFired event). It's very likely that step 1 takes too long and there is not much time left for other steps. I think you should provide a reasonable  timeout for the context.

	dir, err := mkdirTemp("user-data-dir")
	if err != nil {
		return nil, fmt.Errorf("screenshot: create user data dir: %w", err)
	}
	defer os.RemoveAll(dir)

	allocCtx, cancel := chromedp.NewExecAllocator(ctx, append(chromedp.DefaultExecAllocatorOptions[:],
		chromedp.DisableGPU,
		chromedp.UserDataDir(dir),
		chromedp.Flag("disable-web-security", true),
	)...)
	defer cancel()

	taskCtx, cancel := chromedp.NewContext(allocCtx)
	defer cancel()

	err = chromedp.Run(taskCtx, chromedp.Tasks{
		page.SetBypassCSP(true),
		chromedp.EmulateViewport(t.ViewportWidth, t.ViewportHeight,
			chromedp.EmulateScale(t.Scale)),
		emulation.SetTimezoneOverride(t.TimezoneID),
		chromedp.Navigate(t.URL),
	})
	if err != nil {
		return nil, fmt.Errorf("screenshot: initial navigate %q: %w", t.URL, err)
	}

	var imgs []image.Image
	for _, f := range t.Frames {
		err := chromedp.Run(taskCtx, chromedp.Tasks{
			chromedp.Navigate(fmt.Sprintf("%s#%s", t.URL, f.Hash)),
		})
		if err != nil {
			return nil, fmt.Errorf("screenshot: frame navigate by hash %q: %w", f.Hash, err)
		}

		var buf []byte
		err = chromedp.Run(taskCtx, chromedp.Tasks{
			chromedp.Screenshot(t.Selector, &buf, chromedp.ByQuery),
		})
		if err != nil {
			return nil, fmt.Errorf("screenshot: frame screenshot by query %q: %w", t.Selector, err)
		}

		imgPNG, err := png.Decode(bytes.NewReader(buf))
		if err != nil {
			return nil, fmt.Errorf("screenshot: decode png: %w", err)
		}
		imgs = append(imgs, imgPNG)
	}

The timeout happens for screenshot: frame screenshot by query.
I set the timeout to be 60s. The avg time to finish this is 7s. I suspect that there might be an error that chromedp didn't return, and it is busy polling for a result that will never happen.

Though the important information (the URL and the selector) is missed, that makes more sense now.
I think the element that the selector pointing to does not exist in the page. So it will keep trying until the context times out.
https://github.com/chromedp/chromedp/blob/ac112ec8eaa42607d85f349c75405957ce01d2fe/query.go#L151-L153
Try to dump the page to check whether the element exists. See https://pkg.go.dev/github.com/chromedp/chromedp#example-package-RetrieveHTML.

Try something like this:
	var buf []byte
	err = chromedp.Run(taskCtx,
		// 1. "chromedp.Tasks" is not necessary.
		// 2. chromedp.AtLeast(0) makes return immediately even no element is found.
		chromedp.Screenshot(t.Selector, &buf, chromedp.ByQuery, chromedp.AtLeast(0)),
	)
	if err != nil {
		if strings.HasSuffix(err.Error(), "did not return any nodes") {
			// try to dump the html
		}
		return nil, fmt.Errorf("screenshot: frame screenshot by query %q: %w", t.Selector, err)
	}

can chromedp.Navigate wait for networkIdle0 or networkIdle2?
it doesn't seem like we can do it by wrapping chromedp.Navigate.

chromedp.Navigate does not provide such options. But I think it should be easy to implement if in the user code. You can run the action page.Navigate() directly and use chromedp.ListenTarget() to wait for the desired event.
See similar issues below:

#757
#927
#987
#1009

@ZekeLu thanks for your help.
so after putting in more debugging code in production, I found it is actually possible that it is caused by internal code in some rare conditions (<1%, and a part of that 1% is caused by deploy/restart.)
It is not a chromedp's bug.


When running example server/client, can't see avatars for other players connected

I can see the blocks that are added/removed via the other client connection on another computer, but the avatars are invisible. Is this expected?

Yup! We decided to leave player characters to be highly customizable. You can utilize the Peers class and do something like:
import { Peers } from '@voxelize/client'

class CustomPeers extends Peers {
   onPeerJoin: (id: string) => void;
   onPeerUpdate: (peer) => void;
   onPeerLeave: (id: string) => void;
}

...

// Make the custom peer update `controls.object` to the server.
network.register(new CustomPeers(controls.object));

Let me know if you have any questions! @balta-z-r and I are working on a tutorial now, should be published soon :)

After a lot of code inspection and tinkering, I got it to work! Diff below.
However, it seems rotation data isn't in the peer metadata that is sent from the server to the client? It seems to have "direction" which is apparently dx dy dz of the position, if I understand correctly. Are you planning on implementing rotations so that peers can see each other rotate, to see which direction they are facing, even if they don't move?
diff --git a/examples/client/src/components/customPeers.ts b/examples/client/src/components/customPeers.ts
new file mode 100644
index 0000000..9554877
--- /dev/null
+++ b/examples/client/src/components/customPeers.ts
@@ -0,0 +1,50 @@
+import { Peers } from '@voxelize/client'
+import { MessageProtocol, PeerProtocol } from "@voxelize/transport/src/types";
+import * as VOXELIZE from "@voxelize/client";
+
+console.log('It works here');
+
+var myWindow = window;
+
+
+export class CustomPeers
+   extends Peers {
+   public name: string;
+   private head: VOXELIZE.Head;
+   private world: VOXELIZE.World;
+
+   public setWorld(world) {
+      this.world = world;
+   }
+
+   // onPeerJoin: (id: string) => {
+   // }
+   // onPeerUpdate: (peer) => {
+   // }
+   // onPeerLeave: (id: string) => {
+   // }
+
+   public onPeerUpdate = (
+      peer: PeerProtocol<{ direction: number[]; position: number[] }>
+    ) => {
+      const metadata = peer.metadata;
+      console.log("metadata: ", JSON.stringify(metadata, null, 2));
+
+      if (this.world && !this.head) {
+         const head = new VOXELIZE.Head({
+            headDimension: 1,
+            headColor: "black"
+          });
+
+          this.world.add(head.mesh);
+          this.head = head;
+      }
+      const position = metadata.position;
+      const direction = metadata.direction;
+      this.head.mesh.position.set(position[0],position[1],position[2]);
+      // this.head.mesh.direction.set(direction[0],direction[1],direction[2]);
+
+
+    }
+
+}
diff --git a/examples/client/src/containers/App.tsx b/examples/client/src/containers/App.tsx
index 42e63c5..9e01dd4 100644
--- a/examples/client/src/containers/App.tsx
+++ b/examples/client/src/containers/App.tsx
@@ -14,6 +14,7 @@ import * as THREE from "three";
 import { setupWorld } from "src/core/world";
 import { ColorText, Peers } from "@voxelize/client";
 import { sRGBEncoding } from "three";
+import { CustomPeers} from "../components/customPeers";

 const GameWrapper = styled.div`
   background: black;
@@ -69,6 +70,7 @@ const App = () => {
     sky.box.paint("top", VOXELIZE.drawSun);
     world.add(sky);

+
     const clouds = new VOXELIZE.Clouds({
       uFogColor: sky.uMiddleColor,
     });
@@ -268,8 +270,12 @@ const App = () => {
       { identifier: "BRUH" }
     );

-    const peers = new Peers(controls.object);
-
+    const peers = new CustomPeers(controls.object);
+    peers.setWorld(world);
+    // peers.onPeerJoin = function(id: string) {
+    //   console.log('here');
+    //   return;
+    // }
     ColorText.SPLITTER = "$";

     inputs.bind(

Ah I guess I misunderstood - it comes from the quaternion. So I guess the rotation data is being sent.
    const {
      x: dx,
      y: dy,
      z: dz,
    } = new Vector3(0, 0, -1)
      .applyQuaternion(this.object.quaternion)
      .normalize();

To be honest, the example code is really really messy hahaha, since we haven't had the chance to put serious work to https://docs.voxelize.io, we also haven't reorganized the example code yet.
But I believe you can get the quaternion with Peers.directionToQuaternion. Yea... this really needs a lot of documentation hahaha

Thanks, I didn't notice that at first. And no worries about the documentation!
However, it doesn't seem to work properly for whatever reason. Maybe the math is off?
The rotations I visually observe spin way too fast and don't match at all what I'm actually doing. My code is below:
public onPeerUpdate = (
      peer: PeerProtocol<{ direction: number[]; position: number[] }>
    ) => {
      const metadata = peer.metadata;
      console.log("metadata: ", JSON.stringify(metadata, null, 2));

      if (this.world && !this.head) {
         const head = new VOXELIZE.Head({
            headDimension: 1,
            headColor: "black"
          });

          this.world.add(head.mesh);
          this.head = head;
      }
      const position = metadata.position;
      const direction = metadata.direction;
      this.head.mesh.position.set(position[0],position[1],position[2]);

      const q = Peers.directionToQuaternion(direction[0], direction[1], direction[2]);

      this.head.mesh.applyQuaternion(q);
    }

I think the issue is that applyQuaternion isn't what you want. Maybe try head.quaternion.set or even better, head.quaternion.slerp

Ah yes, you are correct. head.quaternion.slerp did the trick!


The remote server returned an error: (500) Internal Server Error.

Hi,
I have a site online and was working fine, using the Image Processor.
All of a sudden, remote images were not working and nothing has been changed from the file system on the server. For images being served remotely, an internal error is being thrown: https://www.remaxcollection-malta.com/remote.axd?http://www.remax-malta.com/UserImages/24/large/L_878263c333524b61bb336ed6eba363a2_iList.jpg?anchor=center&mode=crop&width=1358&height=800&rnd=131081472090000000?entropycrop=250&width=1358&height=800&mode=crop

Any help would be much appreciated.
Thank you in advance.
Kind Regards

Hi @simoazzo
What steps have you done to debug this so far. The error message looks to me like a malformed url is somehow being sent to your remote server. Attach a debugger to a local build of the requesting server. If you are using ImageProcessor v2.4.0 or later (ImageProcessor.Web v4.6.0 or later) you will receive debug logs to your output pane in visual studio which will help you gather details.
There must have been an environmental change for things to break. Has your Cloudflare setup changed? Have there been any server updates? Have you recently updated any part of the software or configuration?
I have to ask... Why are you using the wrong, legacy API calls for remote images?
http://imageprocessor.org/imageprocessor-web/imageprocessingmodule/#remote
The url is a bit mess. you've duplicated the following parameters width, height, mode.
center is the default anchor point. You don't need to add this.
https://www.remaxcollection-malta.com/remote.axd?http://www.remax-malta.com/UserImages/24/large/L_878263c333524b61bb336ed6eba363a2_iList.jpg?anchor=center&mode=crop&width=1358&height=800&rnd=131081472090000000?entropycrop=250&width=1358&height=800&mode=crop

Should be
https://www.remaxcollection-malta.com/remote.axd/www.remax-malta.com/UserImages/24/large/L_878263c333524b61bb336ed6eba363a2_iList.jpg?&mode=crop&width=1358&height=800&rnd=131081472090000000?entropycrop=250

If I navigate to the remote server directly the server is serving the image correctly so we know that ImageProcessor.Web is working well there.
Hopefully we can get to the bottom of this quickly.
Thanks
James
P.S. Please list version numbers when creating issues as described in the contributing guidlines, it saves us a few steps.

Closing due to lack of response.


I've edited the external_download function

Hello, I've modified a bit of the external downloader code so it can handle some simple errors so far. It works as normal and everything. It should recover on linux if aria2c needs root access to do some I/O and even create directories now.
Thanks for this awesome repo though. 😁😁

I don't believe that running shit as root is in the best interest of the end user as that could potentially brick or cause a malfunction in the system, especially if the user isn't particularly adept at computers.
For example aria could overwrite an essential file if the user is trying to download to a restricted directory.

I don't believe that running shit as root is in the best interest of the end user as that could potentially brick or cause a malfunction in the system, especially if the user isn't particularly adept at computers.
For example aria could overwrite an essential file if the user is trying to download to a restricted directory.

You think a linux user wouldn't be adept at computers?

I don't believe that running shit as root is in the best interest of the end user as that could potentially brick or cause a malfunction in the system, especially if the user isn't particularly adept at computers.
For example aria could overwrite an essential file if the user is trying to download to a restricted directory.

You think a linux user wouldn't be adept at computers?

yes, some might. Don't underestimate the power of the human brain, and plus installing linux is much easier now as it can basically just be 8 1/2 clicks and you're installed


Typo in architecture in sidebar

Typo in the naming of "architecture" in the navigation pane
]

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

ID: 58e0c8fc-6697-6a35-ba3f-0986916db4e6
Version Independent ID: 7dd13d5d-314e-97a4-5154-cea0e588f99e
Content: Master planning system architecture - Supply Chain Management | Dynamics 365
Content Source: articles/supply-chain/master-planning/master-planning-architecture.md
Service: dynamics-365-supply-chain
GitHub Login: @t-benebo
Microsoft Alias: benebotg

@Vanseeuw , thank you for reporting this typo. I am fixing it now and it should go live very soon.


Add new CELTS logo and change the application name to "CELTS Link"

We have been told that the name they came up with for the application is "CELTS Link" so we will want to change the title and we need to find a place to work in their new logo.
The new CELTS logo:

 Add new CELTS logo and change the application name to "CELTS Link" #736


Introduce support for seccomp in VMMs

Introduce support to enable or disable seccomp filters for the VMM process that executes the unikernel. The seccomp filters will disallow access to system calls that are not required by the VMMs.  Seccomp support is targeting Hvt, Qemu and Firecrakcer. Moreover, the seccomp filters can be deactivated by using the --security-opt seccomp=unconfined command line option in nerdctl.
For the time being, we do not support custom seccomp filters, since we only want to reduce the system calls the VMM can do and not the ones of the application.

Thank you @ananos for the approval.

thanks @cmainas, looks good and thorough.
A minor comment is whether we could add tests that determine whether the seccomp filter actually works. Not sure how to test that, maybe by removing a syscall in the filter and assert we get an error? maybe this is for a separate iteration.

Yes that is a good point. However, for the case of Qemu and FC we rely on the seccomp mechanisms that these two VMMs provide. Therefore, maybe one idea would be to use a "dirty" version of them which try to access some blocked system calls. In the case of hvt we could address it with a simple unit test.

Another minor comment is about SPT. Should we include this as well?

Do you meant in the tests, to check if seccomp has been enabled?


Fix left simplebar positioning

Simplebar positioning:
Before:

After:

Border radius of first stream filter:
Before:

After:

@SiddharthVarshney can you write commit messages explaining the reasoning for your changes a bit more?  These seem promising, but it'd be a lot faster to review if you explained why you made this specific set of changes.

sure I will write description of each commit shortly.

@timabbott made changes in the commit messages
needs your final review

@vinitS101 I have fixed those
Is it good to go?

@andersk can you review this?  I feel like you're a lot more likely to notice subtle issues with simplebar than I am.

@andersk can you review this?

I would love to have more uniformity in the sidebar implementations here.  That's going to be a rabbit hole (which I'd love the right person to go down at some point), but probably shouldn't block this more tactical fix.

I have updated the changes,
Another thing I noticed here is simplebar for private messages container still lies inside the container like the stream filters

Merged, thanks @SiddharthVarshney!  Please submit a new PR for that issue; I just wanted to get this integrated since it seems critical for the ellipses migration.


Add an explicit opt-in step to join Classroom

Right now, here is the flow:

the student clicks the "join" link.
Student is taken to the Join page on the classroom app.
Request to the mock API endpoint which returns the uuid

Ideally, after the student clicks the join link, auth0 redirects to freecodecamp.org.  The student gives permissions to share information with the classroom app.  Student is redirected to the classroom app.
Edit (Mrugesh):
Every user (student) of freeCodeCamp should opt-in to sharing their data with the Classroom app. It requires some development work on the freeCodeCamp (main) project as well. Here is a list of action items that we want to do:

Classroom app gets a verified user from our IdP (which is Auth0) and hands off the request to Classroom's backend API.
The backend API checks with freeCodeCamp API if the user has opted-in for sharing data with the Classroom app (more on this later).
The freeCodeCamp API can respond with status as the case may be.
The Classroom UI app can display a message which is either the user had opted-in already or a link to the settings page on freeCodeCamp where they can check off a box to opt-in.
The Classroom UI and Backend can repeat step 2 as many times as needed to confirm when the user has opted-in.

We'll need to complete Issue #56 before we can work on this issue.

@raisedadead's example Auth0 app would utilize two sets of keys:

Development ==> Will be shared with the classroom team
Production  ==> Will not be shared with the classroom team

Hey @utsab
I updated the OP to reflect the actual need for this.
I mulled over this a bit and came up with a workflow that should cover different scenarios and how all the stakeholders like freeCodeCamp API, Classroom UI, Classroom API, and our various IdP (GitHub, Google, Email via Auth0) tie together.
It's best we always define the responsibility of the stakeholders.
While it could be more convenient that a user is taken to freeCodeCamp to opt-in, I think it's better long-term that freeCodeCamp remains agnostic of apps that get built upon its core platforms.
So, I propose a more simple and modular logic in the OP.
Let me know what you think, and we can discuss the specifics in more detail over our next call.

Now being tracked inside of #479


Emit the compiler flags in CI again

fixes #2886

@bors r+

:hourglass: Testing commit 67d4fafd0ea7290cb0261aa8f40190c57f684cd8 with merge 4b65dcffb200a9632bf28d862f309c584d25ebcb...

:sunny: Test successful - checks-actions
Approved by: oli-obk
Pushing 4b65dcffb200a9632bf28d862f309c584d25ebcb to master...


Change Colors.fromARGB() to Colors.fromRGBA()

This is a potential usability improvement.

Usually when designers/developers specify color, the order is RGBA.
This is also true for CSS.
Alpha value is also usually from 0-100 or 0.0-1.0 instead from 0-255.
This will reduce cognitive overhead for designers/developers coming from the web world.

Examples:
Chrome:

Sketch:

Thanks for filing! Dupe of https://github.com/flutter/flutter/issues/5205 :)


Exploring why the [!NOTE] isn't correctly rendered

Docs Build status updates of commit 499271a:
:clock10: Full build: the average full build time is 17 min(s) 55 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 499271a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Remove-OMEConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

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


Consider dedicating some flash for configs

So things like IP and MAC address can be changed without re-flashing firmware.
Add some config update commands to the CLI for setting them.
Other potentially useful things to have:

max log level (i.e. trace vs info)
last panic message
last reset reason from panic
env-config constants are used to build up the default config struct values

Boot and application configs in sector 3 (0x0800_C000) as of adb15738a05430141ecf259f89fdc9bfbfa76ee9
Not currently exposing any configs yet, just bootloader flags

Probably use something like TicKV from Tock for a kv store.


Plotly can't find the xvfb-run executable when running from a Gunicorn Flask application.

Hi. I have installed Plotly and Orca with conda install -c plotly plotly-orca and xvfb with sudo apt-get install xvfb.
When running a Python script from the terminal that prints or saves a simple figure to file, it works, but when trying the same code from a Flask application running with Gunicorn and Nginx, it fails on .to_image

File "/home/ubuntu/anaconda3/lib/python3.7/site-packages/plotly/io/_orca.py", line 1482, in to_image
ensure_server()
File "/home/ubuntu/anaconda3/lib/python3.7/site-packages/plotly/io/_orca.py", line 1344, in ensure_server
validate_executable()
File "/home/ubuntu/anaconda3/lib/python3.7/site-packages/plotly/io/_orca.py", line 1067, in validate_executable
formatted_path=formatted_path
ValueError:
The plotly.io.orca.config.use_xvfb property is set to True, but the
xvfb-run executable could not be found on the system path.
Searched for the executable 'xvfb-run' on the following path:
/home/ubuntu/anaconda3/bin

Inside /home/ubuntu/anaconda3/bin I have an orca.sh script with the following code:
#!/bin/bash
xvfb-run -a /home/ubuntu/anaconda3/bin/orca-1.2.1-x86_64.AppImage "$@"

The orca-1.2.1-x86_64.AppImage file is also present on that directory
Related: running orca on the terminal does not print anything, but
xvfb-run orca graph '{ "data": [{"y": [1,2,1]}] }' -o fig.png does work.
here is my $path contents:
/home/ubuntu/bin:/home/ubuntu/.local/bin:/home/ubuntu/anaconda3/bin:/home/ubuntu/anaconda3/condabin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
Any help?
I'm a newbie with unix shell commands, btw.
tl;dr: Plotly can't find xvfb when running inside a Flask-Gunicorn app. Works fine when running from a simple python script from the terminal

I just found the solution. The issue was that my Gunicorn service configuration file was not receiving the correct path content.
In the section [Service], the Environment setting was set with only "PATH=/home/ubuntu/anaconda3/bin"
I updated the Environment with my whole $PATH content and now it works:
"PATH=/home/ubuntu/bin:/home/ubuntu/.local/bin:/home/ubuntu/anaconda3/bin:/home/ubuntu/anaconda3/condabin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin"


Xarray returns duplicate Longitude/Latitude values with sel method

What is your issue?
I am trying to select air temperature from Netcdf file and polgon regions from NUTS3 shape file with the aim of masking both to extract the values. However, for some regions, xarray returns a data frame with the same latitude or longitude values which cannot be masked by geopandas. The shape file and the netcdf I am using are zipped here
Here is the code:
import os
import math
import numpy as np
import pandas as pd
import matplotlib.pyplot as plt
import cartopy.crs as ccrs
import cartopy.feature as cfeature
import seaborn as sn
import geopandas as gpd
import earthpy as et
import xarray as xr
import regionmask
from sultan.api import Sultan
#Plotting options
sn.set(font_scale=1.3)
sn.set_style("white")
nc_file = xr.open_mfdataset('year2019.nc') #, chunks = {'time': 10})
shapefile = "NUTS_RG_60M_2021_4326/NUTS_RG_60M_2021_4326.shp"
countries = gpd.read_file(shapefile)
querry = 3
countries = countries[countries['LEVL_CODE'] == querry]
#Select the longitude and latitude coordinates of the region of interest in the shape file.
#These values will be passed to xarray
german_state = countries[countries.NUTS_ID.isin(["DE255"])]
german_state.plot()
boundaries = german_state.total_bounds
print(boundaries)
ger_mask = regionmask.mask_3D_geopandas(german_state, nc_file.longitude, nc_file.latitude, method="pygeos")
print(ger_mask)
state_lat = [float(boundaries[1]), float(boundaries[3])]
print("Boundary Latitude", state_lat)
state_lon = [float(boundaries[0]), float(boundaries[2])]
print(" Boundary Longitude", state_lon)
print(state_lat, state_lon)
#Slice the data by time and spatial extent by passing the values selected from the shape file
two_months_255 = nc_file["tg"].sel(
longitude=[state_lon[0], state_lon[1]],
latitude=[state_lat[0], state_lat[1]], method="nearest")
two_months_255.where(ger_mask)
Error:
File /opt/homebrew/lib/python3.9/site-packages/xarray/core/indexes.py:449, in PandasIndex.reindex_like(self, other, method, tolerance)
    445 def reindex_like(
    446     self, other: PandasIndex, method=None, tolerance=None
    447 ) -> dict[Hashable, Any]:
    448     if not self.index.is_unique:
--> 449         raise ValueError(
    450             f"cannot reindex or align along dimension {self.dim!r} because the "
    451             "(pandas) index has duplicate values"
    452         )
    454     return {self.dim: get_indexer_nd(self.index, other.index, method, tolerance)}

ValueError: cannot reindex or align along dimension 'latitude' because the (pandas) index has duplicate values

ask.zip

Hi @rogomichael, thanks for raising your issue.
Could you possibly turn your problem into a minimal reproducible example? That would make it much easier for us to help you.
You've included a lot of code, but from looking at it it seems to me that the only lines which matter are
nc_file = xr.open_mfdataset('year2019.nc')

nc_file["tg"].sel(
    longitude=[state_lon[0], state_lon[1]],
    latitude=[state_lat[0], state_lat[1]],
    method="nearest"
)

This suggests to me that your original file has duplicate values, as all xarray has done here is open it.
(However I still am not going to be able to reproduce this error without access to the file year2019.nc.)

@TomNicholas ,
I have reduced the code to a smaller portion. The reason I include a long piece of code is because the longitude and latitude values are read from the shape file and passed to xarray to read from year2019.nc. I previously attached the file in the issue above but you can still access it here.
https://1drv.ms/u/s!AvYOTGjStibEmGGsJwRlpoBfLvHm?e=LBLm95
I hope it is accessible.
import xarray as xr
#Plotting options
sn.set(font_scale=1.3)
sn.set_style("white")
nc_file = xr.open_mfdataset('year2019.nc') #, chunks = {'time': 10})
two_months_255 = nc_file["tg"].sel(
longitude = [49.294, 49.376],
latitude = [10.971, 11.087])

Hi @rogomichael,
It is expected in your example that sel with your given values and method='nearest' returns coordinates with duplicate longitude & latitude values.
The resolution of the grid is too coarse compared to your ROI, which is why the two provided values for latitude (longitude) both select the same grid point.
If you want to use sel for extracting a range over the latitude and longitude dimensions, it is better to use slices. It better illustrates that your ROI is smaller than the grid resolution:
ds.sel(
    longitude=slice(49.294, 49.376),
    latitude=slice(10.971, 11.087)
)
# <xarray.Dataset>
# Dimensions:    (time: 365, longitude: 1, latitude: 0)
# Coordinates:
#   * time       (time) datetime64[ns] 2019-01-01 2019-01-02 ... 2019-12-31
#   * longitude  (longitude) float64 49.38
#   * latitude   (latitude) float64
# Data variables:
#     tg         (time, latitude, longitude) float32 ...

Hi @benbovy ,
As you can see, the slice option returns an empty latitude. From your explanation above, does it mean that it is not possible to select the latitude values and in some cases longitude values? I needed both the longitude and latitude values.


change private fields to protected in HiveMetadataWriter for impl to use

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-XXX

Description

[ ] Here are some details about my PR, including screenshots (if applicable):

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3282 (24309cc) into master (f72523c) will decrease coverage by 37.52%.
The diff coverage is 0.00%.

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

@hanghangliu  Can you do following as part of regular practices for PR:

Create a JIRA ticket in the Apache board.
Adding the JIRA ticket number as part of the PR title and attach the JIRA link in the PR description template as well.
Briefly introduce what's included in the PR in the description section.


High Level Kafka Consumer

PR checklist:

Did you run ClangFormat ? Yes
Did you run CheckStyle ? Yes
Did you check the comment / log / exception conventions in Engineering code process wiki page ? Yes
Did you import unnecessary headers ? No
Did you surround Daisy : starts/ends for new code in existing ClickHouse code base ? Yes

Please write user-readable short description of the changes:
Refactor existing KafkaWAL and add high level kafka consumer to support large scale topics consumptions

Init regression tests / load tests look good. Merge the changes.


ublox_dgnss: 0.4.2-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository ublox_dgnss to 0.4.2-1:

upstream repository: https://github.com/aussierobots/ublox_dgnss
release repository: https://github.com/ros2-gbp/ublox_dgnss-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.3.5-4

ntrip_client_node

No changes

ublox_dgnss
* fixed dependc
* Contributors: Nick Hortovanyi

ublox_dgnss_node

No changes

ublox_nav_sat_fix_hp_node

No changes

ublox_ubx_interfaces

No changes

ublox_ubx_msgs

No changes

@hortovanyi presumably this PR is superseded by https://github.com/ros/rosdistro/pull/37362 and can be closed?

I am closing these PRs under the assumption that the latest release is the preferred one. Conflicts prevent these from being merged as-is anyway.


Home Assistant will not start due to cast component

Home Assistant release with the issue: 0.84.6

Last working Home Assistant release (if known): Unknown. Have been using HA since approx 25/11/18, hope that helps a little.
Operating environment (Hass.io/Docker/Windows/etc.): Hass.io on Linux Docker host

Component/platform: cast

Description of problem: Home Assistant freezes on boot if cast component is "configured" and discovery is enabled.
If I remove it's entry from .storage/core.config_entries and reboot HA then everything other than cast will function fine and I can use HA. If I go Configuration>Integrations and configure Google Cast, it will then hang upon next reboot. All logs stop at ERROR (SyncWorker_2) [pychromecast.socket_client] Failed to connect, retrying in 5.0s.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
homeassistant:
  # Name of the location where Home Assistant is running
  name: [redacted] Home
  # Location required to calculate the time the sun rises and sets
  latitude: [redacted]
  longitude: [redacted]
  # Impacts weather/sunrise data (altitude above sea level in meters)
  elevation: 67
  # metric for Metric, imperial for Imperial
  unit_system: metric
  # Pick yours from here: http://en.wikipedia.org/wiki/List_of_tz_database_time_zones
  time_zone: Europe/London
  # Customization file
  customize: !include customize.yaml

# Show links to resources in log and frontend
#introduction:

# Enables the frontend
frontend:
  themes: !include custom/themes.yaml

# Enables configuration UI
config:

# Uncomment this if you are using SSL/TLS, running in Docker container, etc.
# http:
#   base_url: example.duckdns.org:8123

# Checks for available updates
# Note: This component will send some information about your system to
# the developers to assist with development of Home Assistant.
# For more information, please see:
# https://home-assistant.io/blog/2016/10/25/explaining-the-updater/
updater:
  # Optional, allows Home Assistant developers to focus on popular components.
  # include_used_components: true

# Discover some devices automatically
discovery:

# Show the hassio tab
# hassio:

# Allows you to issue voice commands from the frontend in enabled browsers
conversation:

# Enables support for tracking state changes over time
history:

# View all events in a logbook
logbook:

# Enables a map showing the location of tracked devices
map:

# Track the sun
sun:

# Text to speech
tts:
  - platform: google

# Cloud
cloud:

group: !include groups.yaml
automation: !include automations.yaml
script: !include scripts.yaml

# HASS Configurator addon - Sidebar link
panel_iframe:
    configurator:
        title: Configurator
        icon: mdi:wrench
        url: http://smarthome01.int.[redacted]:3218

# Custom includes
sensor: !include custom/sensor.yaml
switch: !include custom/switch.yaml
light: !include custom/light.yaml
zone: !include custom/zone.yaml
proximity: !include custom/proximity.yaml
device_tracker: !include custom/tracking.yaml

# Third party smart home services
## SmartLife
tuya:
  username: [redacted]
  password: [redacted]
  country_code: 44
  platform: smart_life

# Self hosted services
mqtt:
  broker: <IP_ADDRESS>
  username: homeassistant
  password: [redacted]

media_player:
  - platform: plex

# Lights on when home after sunset
device_sun_light_trigger:
  light_group: group.[redacted]_room
  light_profile: relax
  device_group: group.[redacted]_devices
  disable_turn_off: 0

Traceback (if applicable):

Additional information:
I read through #14956 but there was no real solution that I could find, other than changes to a file (cast.py) that has since been updated a fair bit, and those modifications appear to no longer work. I have reverted those changes since trying them.
I've also tried removing all cast devices from .storage/core.entity_registry and rebooting. This works great for the first reboot, my Chromecasts are rediscovered, added back to the entity registry, and Home Assistant boots perfectly. If I reboot HA again after that, it suffers the same issue as described above.

I was having the same issues. After removing the cast entry from .storage/core.entity_registry HA booted.
Unsure if this is related, but I set up cast again but this time there is a computer being detected as a Chromecast Ultra

@ClayF I don't think your issue is related, but I'm not familiar enough with this project to say for sure really. I've also configured the cast component in configuration.yaml temporarily, but this is really not a solution for me, only an annoying workaround.

I have a Sony Soundbar with Chromecast built-in which sometimes becomes unavailable and i have to pull the plug.
When this happens Home Assistant wont start which is a huge issue, especially when not at home so i cant reboot the soundbar

I have a Sony Soundbar with Chromecast built-in which sometimes becomes unavailable and i have to pull the plug.
When this happens Home Assistant wont start which is a huge issue, especially when not at home so i cant reboot the soundbar

Same here, did not found the issue but you are right i also have as sony soudnbar and this issue.

Same Airserver problem here. Jeez, I reinstalled hass so often to nail it down to Chromecast/Airserver.
I removed Airserver from my PC as ist does not work very well anyway. Now Home assistant is alive again.
Maybe a hint in the documentation or a fix in the component would be nice.

Maybe a hint in the documentation or a fix in the component would be nice.

You can modify the documentation yourself and then create a pull request. Write it within the guidelines and in the right place, and they'll most likely accept it.

Can you please give the fix in #21565 a try? Please note you also need to patch pychromecast with balloob/pychromecast#271

Closed by #21565


replace argonaut with foreign-generic

As far as I understand everyone uses foreign-generic rather than argonaut these days. I forked this because

I want foreign-generic instances
I don't want a dependency on argonaut

Also use spago and hopefully it can go into the package set one day.
Oh I also formatted with purty without even thinking so the diff has a few whitespace changes, I hope that's ok.

Foreign is different from Argonaut's Json type.
Foreign covers all possible JS types (ex. DOM objects) where Argonaut's Json only covers valid Json constructs (string, number, arrays, objects).
For this reason I still stick to Argonaut and as such, prefer those to still be in place.

@jvliwanag there is https://pursuit.purescript.org/packages/purescript-foreign-generic/10.0.0/docs/Foreign.Generic#v:decodeJSON etc for dealing with JSON, at least the documentation suggests that everything should be valid JSON. If that is not the case then perhaps these functions should be altered.
Is this possible inconsistency an issue in datetime-iso?


[INFINITY-813] only run the upgrade/downgrade test when a test repo is present

cc @susanxhuynh

This is blocked on some dcos-tests bugs that mehmet is working to fix

blocked on https://github.com/mesosphere/dcos-tests/pull/188

retest this please

Blocked until kafka master is fixed

retest this please

Is this PR still waiting docs-tests to pass? If so, please git rebase master.
Parent commit is 7d6c93 from Dec 18, before Kafka is upgraded to docs-commons:0.8.x
Jenkin automated testing is getting the dcos-tests from its master branch.   dcos-tests is ahead of your base commit which is on dcos-commons 0.7.12 version.


Custom package isn't installed

The enviroment.yml file is very nice way to get everyone setup but it doesn't install the custom package in the repo. Apparently you can add something like pip install -e . to the environment.yml as a fix. Other options are to make it a real conda package or just put some kind of sys.path.append(...) calls before imports in the notebooks.

It is already a real conda package:
https://anaconda.org/SymPy/scipy2017codegen
But if you are making changes to it locally you should remove the last line of environment.yml and run e.g. python setup.py develop (for some reason pip install -e . failed on one of the CI servers, can't remember which). Or do export PYTHONPATH=$(pwd):$PYTHONPATH, or as you said: sys.path.append(...)

Oh good. So I'm not sure why it didin't get installed when I ran the environment.yml command. I'll dig around.

I just updated the README to hopefully clarify this.


Deploying a CSAR with contained Plans Does Not Work

If a CSAR contains plans that have been generated, e.g., using Winery's Generate Plans button, the deployment does not work as the container generates the plans again and deploys both, the generated ones as well as the already contained ones.

container_1         | o.o.c.c.impl.service.CsarStorageServiceImpl       :196  : Successfully stored Csar as PetClinic_MySQL-OpenStack-w1.csar
container_1         | o.o.c.control.winery.WineryConnector              :87   : Exception while checking for availability of Container Repository:
container_1         | iceTemplateBoundaryPropertyMappingsToOutputHandler:173  : ServiceTemplate has no Properties defined
container_1         | iceTemplateBoundaryPropertyMappingsToOutputHandler:64   : Couldn't generate mapping, BuildPlan Output may be empty
container_1         | o.o.p.c.b.t.BPELBuildProcessBuilder               :238  : Created 1 build plans for CSAR PetClinic_MySQL-OpenStack-w1.csar
container_1         | o.o.p.c.b.t.BPELTerminationProcessBuilder         :211  : Created 1 termination plans for CSAR PetClinic_MySQL-OpenStack-w1.csar
container_1         | o.o.p.c.b.t.BPELBackupManagementProcessBuilder    :289  : Created 1 backup plans for CSAR PetClinic_MySQL-OpenStack-w1.csar
container_1         | o.o.c.p.d.plugin.bpel.BpelPlanEnginePlugin        :146  : Deploying Plan: PetClinic_MySQL-OpenStack-w1_backupManagementPlan.zip
container_1         | o.o.c.p.d.plugin.bpel.BpelPlanEnginePlugin        :146  : Deploying Plan: PetClinic_MySQL-OpenStack-w1_buildPlan.zip
container_1         | o.o.c.p.d.plugin.bpel.BpelPlanEnginePlugin        :146  : Deploying Plan: PetClinic_MySQL-OpenStack-w1_terminationPlan.zip
container_1         | o.o.c.p.d.plugin.bpel.BpelPlanEnginePlugin        :146  : Deploying Plan: PetClinic_MySQL-OpenStack-w1_buildPlan.zip
container_1         | o.o.c.p.d.plugin.bpel.BpelPlanEnginePlugin        :146  : Deploying Plan: PetClinic_MySQL-OpenStack-w1_terminationPlan.zip
container_1         | o.o.c.p.d.plugin.bpel.BpelPlanEnginePlugin        :146  : Deploying Plan: PetClinic_MySQL-OpenStack-w1_backupManagementPlan.zip
container_1         | o.o.container.api.controller.CsarController       :333  : Uploading and storing CSAR "PetClinic_MySQL-OpenStack-w1.csar" was successful
container_1         | o.o.container.api.controller.CsarController       :337  : Csar handling took 0:0:26

seems to be fixed


Allow .subscribe() to accept array of topics

As discussed in these comments,
https://github.com/tulios/kafkajs/issues/289#issuecomment-585497627
https://github.com/tulios/kafkajs/issues/289#issuecomment-585591295
by @ThisIsMissEm and @tulios.
The limitation on KafaJS is that it doesn't let you subscribe to multiple topics at once with an array of topics. That fixes the issues of having multiple consumers for multiple topics.
It's be something like
await consumer.subscribe([
	{ topic: 'topic-a', fromBeginning: true },
	{ topic: 'topic-b', fromBeginning: true }
])

Implementation
It can be possible to pass the array of topics to .subscribe() method, and it will be used as const topicsToSubscribe = [] here https://github.com/tulios/kafkajs/blob/master/src/consumer/index.js#L163

@WaleedAshraf it's not possible to pass an array but you can call subscribe multiple times or use subscribe with a regexp.

@tulios, sorry, maybe I misunderstood the comment in other issues.
You mean it is possible to do this:
const kafka = new Kafka({
  clientId: 'my-app',
  brokers: ['kafka1:9092', 'kafka2:9092']
})

const consumer1 = kafka.consumer({ groupId: 'test-group' })

await consumer1.connect()
await consumer1.subscribe({ topic: 'topic-a', fromBeginning: true })
await consumer1.subscribe({ topic: 'topic-b', fromBeginning: true })
await consumer1.subscribe({ topic: 'topic-c', fromBeginning: true })

The groupId is the same, and the consumer is subscribed to multiple topics.

Yes, this is possible and used in production. You can also do:
await consumer1.subscribe({ topic: /topic-*/, fromBeginning: true })

Thanks, 👍  I'll close this issue.


BUG: groupby().size gives casting error on 32 bit platform

In [1]: df = pd.DataFrame({"id":[1,2,3,4,5,6], "grade":['a', 'b', 'b', 'a', 'a', 'e']})

In [2]: df.groupby("grade").size()
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-3-d8e387418f9d> in <module>()
----> 1 df.groupby("grade").size()

/home/joris/scipy/pandas/pandas/core/groupby.pyc in size(self)
    818
    819         """
--> 820         return self.grouper.size()
    821
    822     sum = _groupby_function('sum', 'add', np.sum)

/home/joris/scipy/pandas/pandas/core/groupby.pyc in size(self)
   1380         """
   1381         ids, _, ngroup = self.group_info
-> 1382         out = np.bincount(ids[ids != -1], minlength=ngroup)
   1383         return Series(out, index=self.result_index)
   1384

TypeError: Cannot cast array data from dtype('int64') to dtype('int32') according to the rule 'safe'

In [4]: pd.__version__
Out[4]: '0.17.0rc1+108.g3fb802a'

cc @behzadnouri

Hmm, although this is maybe more an issue with my numpy installation?
As I don't really see a reason why np.bincount would fail when getting int64 values (the docs says it needs integers and not floats, but does not say anything about integer size)

Possibly an issue with bincount itself: https://github.com/numpy/numpy/pull/4366

since ids are compressed, they cannot exceed memory address space. so i think it should be safe to cast on pandas side, i.e.: ids[ids != -1].astype(np.intp).

with copy=False arg: ids[ids != -1].astype(np.intp, copy=False)

I setup a vm for linux-32
these break for the same reasons as well.
https://github.com/pydata/pandas/blob/master/pandas/core/groupby.py#L2580
https://github.com/pydata/pandas/blob/master/pandas/core/groupby.py#L1815

both of them u added int64 not me. xref https://github.com/pydata/pandas/pull/10988
def group_info(self): indeed requires to return int64_t values but not def nunique. Either case, the right way to do it would be to leave internal computations alone, but only change the output as in:
diff --git a/pandas/core/groupby.py b/pandas/core/groupby.py
index e72f7c6..06b1105 100644
--- a/pandas/core/groupby.py
+++ b/pandas/core/groupby.py
@@ -1808,15 +1808,17 @@ class BinGrouper(BaseGrouper):
     @cache_readonly
     def group_info(self):
         ngroups = self.ngroups
-        obs_group_ids = np.arange(ngroups, dtype='int64')
+        obs_group_ids = np.arange(ngroups)
         rep = np.diff(np.r_[0, self.bins])

         if ngroups == len(self.bins):
-            comp_ids = np.repeat(np.arange(ngroups, dtype='int64'), rep)
+            comp_ids = np.repeat(np.arange(ngroups), rep)
         else:
-            comp_ids = np.repeat(np.r_[-1, np.arange(ngroups, dtype='int64')], rep)
+            comp_ids = np.repeat(np.r_[-1, np.arange(ngroups)], rep)

-        return comp_ids, obs_group_ids, ngroups
+        return comp_ids.astype('int64', copy=False),
+               obs_group_ids.astype('int64', copy=False),
+               ngroups

     @cache_readonly
     def ngroups(self):
@@ -2565,8 +2567,8 @@ class SeriesGroupBy(GroupBy):

         # group boundries are where group ids change
         # unique observations are where sorted values change
-        idx = com._ensure_int64(np.r_[0, 1 + np.nonzero(ids[1:] != ids[:-1])[0]])
-        inc = com._ensure_int64(np.r_[1, val[1:] != val[:-1]])
+        idx = np.r_[0, 1 + np.nonzero(ids[1:] != ids[:-1])[0]]
+        inc = np.r_[1, val[1:] != val[:-1]]

         # 1st item of each group is a new unique observation
         mask = isnull(val)
@@ -2577,7 +2579,7 @@ class SeriesGroupBy(GroupBy):
             inc[mask & np.r_[False, mask[:-1]]] = 0
             inc[idx] = 1

-        out = np.add.reduceat(inc, idx)
+        out = np.add.reduceat(inc, idx).astype('int64', copy=False)
         return Series(out if ids[0] != -1 else out[1:],
                       index=self.grouper.result_index,
                       name=self.name)

I changed them because they didn't work on window. I will try your diff and let u know.

Hello,
I have issue with minlength which seems to be fix with this solved issue... Are we going to have pandas 0.17.1 version tag soon? It really unreliable without it to pass to 0.17.x which seems to bring many benefit despite this ".size() minlength" issue...
And if not is using master consider "safe" (as read stable as an official tagged commit)?
Thanks

Sorry, I think I am wrong... This bug fix seems to be include in 0.17.0
But I have this error : <type 'exceptions.ValueError'> minlength must be positive
File "/usr/local/lib/python2.7/dist-packages/pandas/core/groupby.py", line 820, in size
return self.grouper.size()
File "/usr/local/lib/python2.7/dist-packages/pandas/core/groupby.py", line 1383, in size
out = np.bincount(ids[ids != -1], minlength=ngroup)
ValueError: minlength must be positive
Which seems related...
I just do :
grouped.size()
Which were working just fine prior to upgrade from 0.15.0 to 0.17.0
Could it be related?

Downgrading to 0.16.2 solve my issue...
:(

you would have to show a copy-pastable example along with pd.show_versions()

I will try to reproduce with dummy data, because I can't include the actual data... Give me a couples of days.
Thanks


Issues from Slack Channel

This issue is to track issues raised in the Slack Channel

:warning: BUG UPDATE: Hey guys! I know some of you have been facing issues with the lab. Very sorry about that. I checked with my backend team and have some updates. I was informed that it's platform wide issue with the lab platform I am using (katacoda). There seems to be an issue with http://socket.io|socket.io module that keeps kicking out sessions randomly.  A fix/solution is being worked on, however it's going to take a few more days. I am promised a resolution by Monday. Please bear with me until then. Apologies for the inconvenience caused. Thank you for understanding.
View in Slack

<@UDQHGHJQ0> good day!
Issue with practice lab for cluster is not disconnecting, but now it throws errors after upgrading cluster
View in Slack

<@UDQHGHJQ0> hi, is it possible to add Helm in the administrator course.
View in Slack

Hi All,
During the Challenge 1 from CKAD training I failed only with secret. I used:
echo 'admin' | base64
with result:
YWRtaW4K

But in your yaml file, Mumshad, I can see different value: YWRtaW4=
And that is also 'admin':
[vagrant@k8s01 ~]$ echo "YWRtaW4=" | base64 -d
admin[vagrant@k8s01 ~]$
The question is: how it can happens that same word encoded to different values?
View in Slack

<@UDQHGHJQ0> there is a section about initcontainer in the course ?
View in Slack


Finding more NLP rules

@infvie is going to work on collecting more knowledge elements from the cookbook text (beyond the definitions) and coming up with odin patterns that will find those knowledge elements.
The result should be captured in a CSV table similar to PR #15 examples/knowledge_graph/relations.csv is a goo place to store it.

Closed by https://github.com/jpfairbanks/SemanticModels.jl/tree/master/examples/knowledge_graph/manual


Updating packages, adding new configuration for KCL

Updates all packages to newest versions. 1.1.1 SBT builds.
One exception - I could not update the KCL version fully to 1.9.0 as the localstack dependency, kinesisalite, does not support KCL's newest dependency on the ListShards Kinesis API. This resulted in failed integration tests.

Coverage increased (+0.3%) to 90.16% when pulling f1fa51806cb94c20ab5aedc6e0a103efde799020 on etspaceman:updates into 6ae4b06504f396eed898ef11ae498c7311829ff1 on WW-Digital:master.

Scala was just updated to 2.12.5
Should we include that in this PR? @etspaceman
cc: @markglh

Done @agaro1121 - the only place that it was being defined was in the Travis CI build and it seems to have run just fine.

I'll wait for @markglh to give the final word here


x-offset is to low

I'm not sure if this is only a problem on non-retina monitors, but the characters were very squashed together and hard to read.  I had to change the x offset from 2.0 to 3.0 in the default config file.

The default monospace font changes between Linux distros. Or are you on macOS?

I'm on linux, but two different fonts on two different linuces were too squished together.  It's probably a retina thing.

We've improved our use of font metrics so that 0 offsets can typically be used.


Option to set a custom class name for poppers

It would be tremendously useful, as a way to style different poppers differently, to be able to specify an additional class for the popper markup. Right now, it's possible by specifying the template option but it's needlessly verbose, requiring the entire template to be specified again simply to add a class.
I'm happy to provide a PR with this feature if it is deemed that it should be part of the library.

Up!

Hi, I'm interrupting the development of the tooltip.js library to focus on the development of @popperjs/core (Popper.js v2).
Please consider switching to Tippy.js, which is a way better alternative to Tooltip.js and is well maintained. Tippy.js still uses Popper.js under the hood, so you will find all your favourite features.
I'll close this issue.