[Probably concerning upstream, but need reference where to create this report] SCSS Module color format gets changed during processing

Describe the bug
Colors imported from SCSS Modules to JS code (and maybe also when used in SCSS, that would need further testing) behave erratically. They behave different during any of these combinations: During (react-scripts) build and production (start), depending on the color (at least black compared to other primary colors, and primary colors when compared to non-primary), depending on the original color format (seems mostly hex-based colors vs others like hsl, rgb or rgba), depending on if they come from an scss-variable or are directly used.
Potentially also influenced by whether Typescript is used or not, didnt test that, I use typescript. Edit: Test by robv8r show that the usage of typescript or not has no significance
This problem probably doesnt concern you directly, but I'm not completely aware of what will get processed how, and as the end-user facing project I'd like you to at least forward me to the correct projects, as finding out the concerned projects isnt trivial and I might miss some, and/or annoy projects that dont have issues at all.
Did you try recovering your dependencies?

This behavior can be reproduced in a fresh setup, as will be shown below.
Which terms did you search for in User Guide?

None. I followed the official guides regarding scss usage and css modules to get here tho.
Environment

Executing this lead to errors. The one line without errors:
current version of create-react-app: 3.4.1.
Elsewhise I use Manjaro Linux, x64, running these commands from a zsh, with yarn 1.x. The build errors were originally observed from the build generated by the docker-container node:lts-alpine (using already present yarn) that was later served by an nginx container. I looked at the pages with Firefox, Chrome and IE11.
Steps to reproduce

npx create-react-app scss-module-error --template typescript
yarn add node-sass
create these files/ replace the original ones with them:

# ./test.module.css
:export {
    simple-hash: #f00;
    extended-hash: #ff0000;
    extended_hash_black: #000000;
    extended_hash_blue: #0000ff;
    extended_hash_green: #008000;
    hsl_green: hsl(120, 100%, 25%);
    hsl: hsl(0, 100%, 50%);
    rgb: rgb(255, 0, 0);
    rgba: rgba(255, 0, 0, 1);
};

# ./test.module.scss
:export {
    simple-hash: #f00;
    extended-hash: #ff0000;
    extended_hash_black: #000000;
    extended_hash_blue: #0000ff;
    extended_hash_green: #008000;
    hsl_green: hsl(120, 100%, 25%);
    hsl: hsl(0, 100%, 50%);
    rgb: rgb(255, 0, 0);
    rgba: rgba(255, 0, 0, 1);
};

# ./test_with_vars.module.scss
$simple_hash:#f00;
$extended_hash: #ff0000;
$extended_hash_black: #000000;
$extended_hash_blue: #0000ff;
$extended_hash_green: #008000;
$hsl_green: hsl(120, 100%, 25%);

:export {
    simple-hash: $simple_hash;
    extended-hash: $extended_hash;
    extended_hash_black:$extended_hash_black;
    extended_hash_blue:$extended_hash_blue;
    extended_hash_green:$extended_hash_green;
    hsl_green: $hsl_green;
    hsl: hsl(0, 100%, 50%);
    rgb: rgb(255, 0, 0);
    rgba: rgba(255, 0, 0, 1);
};

# ./index.tsx
import React from 'react';
import ReactDOM from 'react-dom';
import './index.css';
import App from './App';
import * as serviceWorker from './serviceWorker';
import * as css_imp from "./test.module.css";
import * as scss_imp from "./test.module.scss";
import * as scss_var_imp from "./test_with_vars.module.scss";

console.log("pure css:\n", css_imp, "\nscss:\n", scss_imp, "\nscss from vars:\n", scss_var_imp);

ReactDOM.render(
  <React.StrictMode>
    <App />
  </React.StrictMode>,
  document.getElementById('root')
);

Execute yarn run start and yarn run build && npx serve build

Open the respective pages (localhost:3000 and :5000) in a browser and look at the log output.

Expected behavior

Colors preferably dont get processed, or if they have to at least reproducible between developement and production, as well as between scss-variables and different color formats.
Actual behavior

Left side: start, right side: react-scripts build && npx run serve build

Reproducible demo

Edit:
Github repos provided by robv8r below:
(TS)
https://github.com/robv8r/scss-module-error
(JS based)
https://github.com/robv8r/scss-module-error-js
I think above steps should suffice, though if neccessary I can still do this later.

At the end of index.tsx, did you remove serviceWorker.unregister();

Not really, it was distracting me on here and I did not believe that it was of significance. My tests were made with that in place. Maybe I was wrong about it being insignificant to this demonstration? But AFAIK these are mostly for PWA features, and shouldnt really matter on first execution.
Did you try this example yourself, and did it work as described or not?

...AFAIK these are mostly for PWA features, and shouldn't really matter on first execution.
That's my understanding as well.  I'm glad that it's an opt-in feature now.
Did you try this example yourself, and did it work as described or not?
I followed your example and got the same result.  I can confirm that you, sir, have found a bug.

Just for fun, I've created a repo that reproduces the error.
https://github.com/robv8r/scss-module-error

Well, it was more than fair to point out, I should've mentioned it or not omitted it just because.
Thank you very much for confirming my observations, testing for typescript influence and providing the repos, I've added the informations to my opening post.

Goddamnnit.... Too damn easy to misclick on the "comment and close" button IMAO.


[ContainerApp] az containerapp replica count: app/revision scope

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

ContainerApp

@zhoxing-ms this is ready for review!

Please resolve the code conflicts.

Please add test for the new added command.

If you want to release a new version of the extension, please change version in setup.py file.

If you want to release a new version of the extension, please change version in setup.py file.

Hi @yanzhudd
Generally, we will release versions manually periodically, so this PR no need to update set_up.py.
But if @Juancpani really want to release a new extension for containerapp in this PR, please rerun all tests in containerapp with --live，then request review version change and approve again for me and @Juliehzl  for the release:
azdev test containerapp --live --no-exitfirst

If you want to release a new version of the extension, please change version in setup.py file.

Hi @yanzhudd Generally, we will release versions manually periodically, so this PR no need to update set_up.py. But if @Juancpani really want to release a new extension for containerapp in this PR, please rerun all tests in containerapp with --live，then request review version change and approve again for me and @Juliehzl for the release: azdev test containerapp --live --no-exitfirst

I'm not looking to request a review version change.

Please fix CI issues

Rebase issue, merged conflicts and confused git. Will open new PR.


XTDB connector fails CTS - Retype instance

Using egeria 3.15-SNAPSHOT (container, built ~30 Jan 23) & XTDB main, both
the automated CTS and manual run failed.
The manual run used

OpenShift 4.11
3 x 64GB / 8 CPU nodes

Results: (manual - action can be reviewed above)
              Metadata sharing MANDATORY_PROFILE    CONFORMANT_FULL_SUPPORT [  71895 /      0 ]
             Reference copies  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   8543 /      0 ]
         Metadata maintenance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  14091 /      0 ]
                Dynamic types  OPTIONAL_PROFILE             UNKNOWN_STATUS [      0 /      0 ]
                Graph queries  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [    528 /      0 ]
            Historical search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [    531 /      0 ]
               Entity proxies  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2782 /      0 ]
      Soft-delete and restore  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2598 /      0 ]
               Undo an update  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   1218 /      0 ]
          Reidentify instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2655 /      0 ]
              Retype instance  OPTIONAL_PROFILE             NOT_CONFORMANT [    621 /     69 ]
              Rehome instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   1593 /      0 ]
                Entity search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  60710 /      0 ]
          Relationship search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   8233 /      0 ]
       Entity advanced search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  40384 /      0 ]
 Relationship advanced search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   9280 /      0 ]

FAIL [225662/69]

These were failing prior to recent changes to fix graph, and add some more tests in the CTS

The configuration used was https://github.com/odpi/egeria-test-cts/blob/main/etc/xtdb.yaml


Crossed connections in pipeline editor

Sometimes the connections cross each other in the pipeline editor (on their way to the ports)

Fixed by #836


Change the main branch to main

I don't have a strong feeling about it but I find it confusing that in this repository it's called master, so can we change it to main? @jan-janssen

Done - this was mainly for historic reasons.


fix(datepicker): screen reader reading out role twice

Currently the markup of the calendar is div[role="grid"] > table > td[role="gridcell"]. Because we've got both a grid and a table, NVDA ends up reading out something like "Dialog, table, table, 31st of March 2019, column X, row Y". These changes set the table to be a presentation role so that "table" is only read out once.

@crisbeto this passes presubmit, just needs a rebase

Rebased.


Wrong code generation with multipart files

What are the steps to reproduce this issue?

Using React query generator option
When desire multiple documents to be sent once

What happens?
Orval is generating a method that wraps all the files in JSON.stringfy
What were you expecting to happen?
Instead of
export const uploadMultipleDocuments = (
  uploadMultipleDocumentsBody: UploadMultipleDocumentsBody,
  options?: SecondParameter<typeof customInstance>
) => {
  const formData = new FormData()
  formData.append('files', JSON.stringify(uploadMultipleDocumentsBody.files))
  formData.append('submissionName', uploadMultipleDocumentsBody.submissionName)

  return customInstance<void>(
    {
      url: `/documents/multiple`,
      method: 'post',
      headers: { 'Content-Type': 'multipart/form-data' },
      data: formData,
    },
    options
  )
}

We might have
export const uploadMultipleDocuments = (
  uploadMultipleDocumentsBody: UploadMultipleDocumentsBody,
  options?: SecondParameter<typeof customInstance>
) => {
  const formData = new FormData()
  uploadMultipleDocumentsBody.files.forEach(file -> formData.append('files', file))
  formData.append('submissionName', uploadMultipleDocumentsBody.submissionName)

  return customInstance<void>(
    {
      url: `/documents/multiple`,
      method: 'post',
      headers: { 'Content-Type': 'multipart/form-data' },
      data: formData,
    },
    options
  )
}

Any other comments?
If I'm doing something wrong with my specification, please, help me out.
/documents/multiple:
    post:
      tags:
        - document-controller
      operationId: uploadMultipleDocuments
      description: Uploads multiple documents to the server
      parameters:
        - in: header
          name: X-Request-ID
          required: true
          schema:
            $ref: '#/components/schemas/XRequestIDHeader'
        - in: header
          name: X-Platform
          required: true
          schema:
            $ref: '#/components/schemas/XPlatformHeader'
        - in: header
          name: X-Version
          required: true
          schema:
            $ref: '#/components/schemas/XVersionHeader'
        - in: header
          name: Authorization
          required: true
          schema:
            type: string
      security:
        - jwt: ['secret']
      requestBody:
        content:
          multipart/form-data:
            schema:
              type: object
              properties:
                files:
                  type: array
                  items:
                    type: string
                    format: binary
                submissionName:
                  type: string
                  minLength: 1
              required:
                - files
                - submissionName
      responses:
        '200':
          description: OK
        '400':
          description: Invalid user request
        '401':
          description: Unauthorized
        '403':
          description: User not allowed to perform the requested action
        '500':
          description: Internal server error

What versions are you using?
Operating System: MacOS
Package Version: "orval": "^6.8.1"

I found a way to override it, but, I still think this should be the default behavior.
Orval config:
module.exports = {
  'v2/mw-api': {
    output: {
      mode: 'split',
      target: './src/services/v2/mw-api.ts',
      schemas: './src/services/v2/model',
      client: 'react-query',
      mock: true,
      prettier: true,

      override: {
        mutator: {
          path: './src/services/custom-instance.ts',
          name: 'customInstance',
        },
        operations: {
          uploadMultipleDocuments: {
            formData: {
              path:  './src/utils/custom-upload-multiple-documents.ts',
              name: 'customUploadMultipleDocuments',
            },
          }
        }
      },
    },
    input: {
      target: './src/services/open-api/mobile_api_v2.yaml',
    },
    hooks: {
      afterAllFilesWrite: 'prettier --write',
    },
  },
}

Function to override:
export const customUploadMultipleDocuments = (param: {files: Blob[], submissionName: string}): FormData => {
  const formData = new FormData()

  param.files.forEach(file => formData.append('files', file))
  formData.append('submissionName', param.submissionName)

  return formData;
};

Result:
/**
 * Uploads multiple documents to the server
 */
export const uploadMultipleDocuments = (
  uploadMultipleDocumentsBody: UploadMultipleDocumentsBody,
  options?: SecondParameter<typeof customInstance>
) => {
  const formData = customUploadMultipleDocuments(uploadMultipleDocumentsBody)
  return customInstance<void>(
    {
      url: `/documents/multiple`,
      method: 'post',
      headers: { 'Content-Type': 'multipart/form-data' },
      data: formData,
    },
    options
  )
}

will be added in the next release

Released with 6.9.0


Can we have assets in a separate directory to the main html file?

So my previous webpack config would generate something which looked like this. html-webpack-plugin
public
├── index.html
└── ui-assets
    ├── assets-manifest.json
    ├── main.d2fe0910b8.js
    ├── main.d2fe0910b8.js.map
    ├── vendor.67dfc8f6a1.js
    └── vendor.67dfc8f6a1.js.map

This was useful to us because we didn't want the main entry point (index.html) to be hashed. it also allowed us to serve static assets from a different place.
<EMAIL_ADDRESS>generates this:
public
├── 1-25MTWOSN.js
├── 1-25MTWOSN.js.map
├── 1-LCOOVKTY.css
├── 1-LCOOVKTY.css.map
└── index-LJMJ3LED.html

We don't want the main entry point hashed, and we would like the assets (JS and css files) to be in a separate directory.
Is this possible for you to implement?

Hello @jasonwilliams! Would you give a try to the version 0.15.3 of the plugin? It should now respect the entryNames pattern for HTML entrypoints, so you can use assetNames for assets.

The output is exactly the same, also its printing a huge array of files out when i run build. Did you leave a console.log in there?
how do i utilise entryNames to get the result I want? Do you have any docs on this? Im still unsure how i set a folder name for assets? Could you explain how to do this? The docs don't really cover this. https://github.com/chialab/rna/tree/main/packages/esbuild-plugin-html#usage

In the main esbuild config i set assetNames: 'assets/[name]-[hash]' but this didn't help.
The output was this:
public
├── 1-LCOOVKTY.css
├── 1-LCOOVKTY.css.map
├── 1-Y53U5IJE.js
├── 1-Y53U5IJE.js.map
├── assets
└── index-8c4db01e.html

It generated an assets folder but but it was empty

I would definitely update the documentation as im sure this would be useful to others too. Right now it's a bit hidden knowledge

If you use images or fonts references in the index.html entrypoint or in you CSS, you have to keep the assetNames configuration. More in general, esbuild uses the assetNames for resources that are imported with the file loader.
Thanks for your feedback, I certainly will update the usage section.

Does this new configuration solved https://github.com/chialab/rna/issues/35?

Yes and this issue too thanks

Hey sorry to bring this back up, but watch mode fails now. Do you happen to know why?
✘ [ERROR] [plugin html] Build failed with 1 error:
src/1.css:1:8: ERROR: Could not resolve "./ui-assets/1-27BSLWHE.css"

I am not sure the html plugin can even work in serve mode. Probably, the error is given by the fact you have previously build the application. If you remove the public folder and then start the server, you can see that esbuild does not generate the index.html file at. That because esbuild does not write anything to fs in server mode. I'll add this notice to the doc.

If you remove the public folder and then start the server

I tried that, it made no difference sadly. Its true no index.html gets generated but the assets do

I’m going to close this in favour of https://github.com/chialab/rna/issues/44 as the original issue here was fixed

Hey @edoardocavazza I just noticed in watch mode (not serve), its generating new assets but the html file isn't being updated to point to the new ones. I think this may be a bug


Enable access_log customisation per virtual host

Following on from #216 and #230
Removing access_log from default nginx.conf enables per-virtual host logging configuration.
Is there any argument against modifying default nginx.conf in this way, considering it's already being done with the server_names_hash_bucket_size? If so, why is server_names_hash_bucket_size added in the Dockerfile instead of in nginx.tmpl?
Paging Dr @md5

These changes seem reasonable to me.
To be honest, it probably makes sense to just replace the entire nginx.conf. That's what the Nginx folks suggested. I'm not sure that there's much being gained by relying on the default configuration of the nginx image.

Agreed. I'd say there's much to be gained by writing an essentially blank nginx.conf with the interesting bits imported in the nginx.tmpl or user entered conf.d / vhost.d entries.

I've just stumbled into this while debugging the upload of a big file.

413 Request Entity Too Large

But that's clearly not the point here.
One should probably mirror all mounting options into this README from the nginx base image, including the HTML directory for standard splash pages and the log directory for getting access to the logs
https://github.com/dockerfile/nginx#attach-persistentshared-directories

@almereyda There isn't a log directory since the logs go to STDERR and STDOUT. In the base nginx image, /var/log/nginx/access.log and /var/log/nginx/error.log are symlinks.

Also, there shouldn't be a standard splash page for this image when it's working properly.

@27Bslash6 Sorry to necro this, but since I think this repo is dead, I need to merge these changes into my own fork.
How does this enable per-host loggin?  I see that you've factored access_log out to the root, but this doesn't add support for any new environment variables in other containers?
I would expect something like ACCESS_LOG_FORMAT and ACCESS_LOG_PATH get added?  Or is your intention simply to lay the groundwork for that later?

@subdavis Since access_log isn't defined in the server block any more, it enables you to define per-virtual-host access logging in any /etc/nginx/vhost.d/* files as per normal nginx server {} syntax


Categories for software packages

Related to #1
Should we have a README.md or similar in each of our software-tips-and-tricks packages e.g. version-control which gives an overview of how the different options for that particular thing rates. We could have stuff like:

recommend (highly, somewhat, never ever ever use)
ease of use
ease of installation
target audience

It seems to me that having an 'ease of use'/'review' component to our stuff could be a good way to distinguish this repo from others which are more like enormous lists of stuff...

Also might be worth adding pre-requisites or so


EchoNode: Add helper methods for EchoType checks

Small QoL pull request that adds a set of helper methods for EchoType checks.

Neat! Thank you for the PR! This is now available in the 1.8.0 release


Add Bosnian and  Croatian localization

Pull request checklist

[x] All tests pass. Demo project builds and runs.
[x] I have resolved any merge conflicts.
[x] I have followed the coding style, and reviewed the contributing guidelines. Confirmation: ____

💪😎👊
This fixes issue
What's in this pull request?
Bosnian and Croatian localization. Translated by Admir Ireiz (admir.ireiz@gmail.com).

Thanks @iadmir ! 👍


postgresql: fix regress tests after tzdata update

Description of changes
postgresql is currently broken on staging, after the tzdata update (#339913) due to failing regression tests.
This was discussed upstream here:
https://postgr.es/m/0a997455-5aba-4cf2-a354-d26d8bcbfae6%40technowledgy.de
This will be in the next minor releases, until then apply the patch. Applying the patch from github via fetchurl is not possible, because the upstream patch includes some doc changes, which cause our build to fail. We could fix that by adding new dependencies to rebuild the docs... but it's easier to just keep the patch in nixpkgs for 1-2 months.
TLDR: The patches have been slightly modified from upstream, removing the docs changes.
@Ma27

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

Thanks!
Will try to review/merge both of your PRs today :)


Implement rate-limiting of GitHub API in CI/CD

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

Expected Result:
Our GitHub API requests should be successful.
Actual Result:
GitHub flagged us for API abuse and started rate-limiting us. This caused a number of our API requests to fail arbitrarily. See this workflow run for an example.
Action Performed:

Close the StagingDeployCash when a large number of PRs have not yet been deployed to production
When commenting on each, the GH API started rate-limiting us.

Workaround:
n/a in this case.
Platform:
Where is this issue occurring?
GitHub
Upwork job: https://www.upwork.com/jobs/~01b6f4487984aa7229

Reopening, will close upon payment.

Paid.


iam-policy template

Hi,
I would like to request a CFT template for https://cloud.google.com/resource-manager/reference/rest/v1/projects/setIamPolicy.
It is a non-trivial update as it involves updatemasks and policy etags. Our main use case is to ensure all possible audit logs are being generated.
Currently we implement it via custom template: https://github.com/GoogleCloudPlatform/healthcare/blob/master/deploy/templates/data_project.py#L539
cc @ocsig @morgante

It seems like terraform has a single template for both iam member bindings and audit configs: https://www.terraform.io/docs/providers/google/d/google_iam_policy.html which seem simpler to use. So I wonder if the existing iam_member template could be expanded to support this use case.

Currently we have IAM member binding for IAM roles on the project level here:
https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/tree/master/dm/templates/iam_member
We are actively working on extending this for the folder, organization level as well as the support of organization policies. I will add audit log policies to the feature request.


Require unique benchmark names in add_benchmark and list_benchmark

currently it is allowed to have multiple benchmark with the same name.
This seems unexpected to me, in case there is multiple benchmark how can the HBS template file works ?
For me user should use unique names for each benchmark.
WDYT @shawntabrizi
EDIT: my assumption is wrong, we still need the path to the pallet in some way.

ah but this doesn't work because it will generate stuff like:
impl<T: frame_system::Config> pallet_collective_council::WeightInfo for WeightInfo<T> {

Where we want:
impl<T: frame_system::Config> pallet_collective::WeightInfo for WeightInfo<T> {

to solve this issue we might need to give an extra optional argument to add_benchmark/list_benchmark and have a different field for the benchmark name "pallet_collective_council" and the trait path "pallet_collective".

another idea would be to do this:
diff --git a/frame/benchmarking/src/lib.rs b/frame/benchmarking/src/lib.rs
index fb602f0732..18297f412f 100644
--- a/frame/benchmarking/src/lib.rs
+++ b/frame/benchmarking/src/lib.rs
@@ -1334,7 +1334,7 @@ macro_rules! add_benchmark {
                        verify,
                        internal_repeats,
                } = config;
-               if &pallet[..] == &name_string[..] {
+               if &pallet[..] == &instance_string[..] {
                        let benchmark_result = $( $location )*::run_benchmark(
                                &benchmark[..],
                                &selected_components[..],

Then I guess we need to modify the bench bot a bit to be able to pass the output file directly. But this would solve the issue with 2 pallet being triggered together

or thinking again it seems the error might just be in benchmarking-cli writer. maybe fixing it would solve https://github.com/paritytech/polkadot/pull/3511

this all comes from a missunderstanding of the cli writers, in case there is multiple batch it will writes to multiple files.
I'll try to find the issue and fix it for https://github.com/paritytech/polkadot/pull/3511


LibreBoard logo design

@yasaricli I will make the logo. How do you want it to be?

Variations: I thought material design would be good for this. So made it something like this.
1

2

3

4

5

6

Hello;
a friend of mine is also working on log. Thanks.

@yasaricli Okay.

Thank you my Friend Ekrem.
All logos
Social Avatar :+1:

Site logo :+1:

I think it's fancy :) i liked it

@orges :+1:

@yasaricli Loved it. :+1: Cool.

I tend to uppercase the letter B to make the name easier to understander Libreboard — LibreBoard
This is also what other do:

Let's pick a convention and stick with it!

I think it's a good point! When i write libreboard i always feel writing it as LibreBoard is better. It emphasis on the Libre as an adjective and Board as the main functional unit..
But also if we write it like Libreboard it sounds like a board of liberty, emphasing Liberty and a place to show it...
Also i vote for LibreBoard with capital "B" but i can't imagine how it will look like...

Also, I believe there are some copyright issues — http://designshack.net/design/pigeon

What about using another font (while matching the core identity values) to add a +1 on the Trello differentiation effort?

@yasaricli "Pigeon, Logo for political party." https://dribbble.com/shots/746215-Pigeon

:+1:

@yasaricli it appears your friend did not make that logo. Leaving aside the fact that this is dodgy practice, that logo needs to change asap before you guys get into legal troubles again.
How about a stack of cards/post-its? Here are some free examples (so you can use them as-is or modify them if you'd like).
https://openclipart.org/detail/183333/cube-notes-by-crisg-183333

https://openclipart.org/detail/45841/sticky-note-green-folded-corner-by-palomaironique

https://openclipart.org/detail/140227/stack-of-unlined-colored-index-cards-by-snifty

Hey guys! I'm not a designer, but I've tried to explore some ideas (the feeling/meaning of "Liberty" and a "Tool to explore & deliver" and a "Journey from point A to B" ) and came up with the following  logo design (proposal). So, if anybody likes this idea and can design a logo upon it, I'd be very happy. :)

Quick adaption of the Cube Notes:

:+1:

I think if the current logo really is a problem, we should sort it out before the next release, as else it will just confuse more if we change the logo not at the same time as we changed the overall design.

@ePirat I definitely agree.

I think @neynah may be interested in this thread.

Thanks @ocdtrekkie
Hi all! I am the designer at Sandstorm.io. This is a very relevant discussion to me as I also found the existing LibreBoard logo on a Dribbble user's portfolio and decided that I should not use it to represent the app on Sandstorm. Here are the variations I have come up with (sharing for the first time).
Option 1: New design altogether

Option 2: Existing plain text logo (which I assume does not have copyright issues) with a nice gradient bg

Option 3: I'm very much open to suggestions and am happy to volunteer my time to design something so we can get this on the new App Market ASAP. :)

I wanted to make this announcement in due form at the v0.9 release time, but I will share my idea now (2am here in France :)) in favour of saving everyone's time. Here is the thing, I would like to re-brand the project from “LibreBoard” to “Wekan”.
Yasar the originally author of this application originally named it Metrello because it was a Trello clone made in Meteor. Trello wasn't happy with that and asked us to rename the project. I first thought of "OpenBoard" (because well it's an open-source board) but because the dot com domain name was already taken, I replaced the word “Open” by “Libre” which is free as in freedom in some Latin derived languages (including French). The whole naming process took something like 10 minutes.
It turns out this name is pretty difficult to remember (when I talk to some people about this project, they always forget how it was called) and I've seen quite a few person on reddit/twitter (and also #sandstorm IRC if I recall correctly) that says we should stop naming our project "LibreSomething".
I think that my proposition, Wekan, is shorter to type and more memorable (of course if you are used to the name LibreBoard you may dislike it :)). This is a name includes both the idea of collaboration "We" on a "Kan"ban software. I also bought this logo:

The slogan/catchphrase could be something like “Wekan — organize anything”
Any feedback about this re-branding idea? If people don't like it, I can very well change my mind. Also about the logo I'm open to alternatives, especially since I currently don't have a good “square icon”.
Of course the project will stay open-source, just because a Firefox isn't called LibreBrowser doesn't mean it is not free as in freedom :)

Also @neynah, I really like all your work, both here and on sandstorm :-) The re-branding is not a blocker, I could publish at least a beta under current LibreBoard name using either option 1 or 2.

@mquandalle Wow I think that is a great idea and the proposed logo looks great and catchy!

@mquandalle I appreciate that. Perhaps we can wait a day or so to see what the consensus is. If you'd like to go with Wekan then I think we should publish the app under Wekan. The downside to publishing the app as LibreBoard with LibreBoard branding is that once it's updated to Wekan, it will be difficult to find/identify. Some users may think "what happened to LibreBoard?"
If you could share the source file for the logo I'd be happy to provide some square logo options based on what you have.

That's super kind to you Néna, I sent you the AI source on a sandstorm grain. And if you have a genius idea, please share, I'm completely open to changes.

You know, when I am repeating this "WeKan"  in my head, the image I see is Obama's 2008 campaign slogan "Yes we can". I am not sure if it only me or rest of folks in US feel the same. I am not sure if it is good or bad thing in terms of branding a open source kanban board - I mean in the right instances it might be a good thing as there is already name recognition
I got this idea of using some kind of metaphors instead that creates images of organizing, like Hive or OpenHive or kanHive or kanbanHive... There is this french word that chefs use to organize the ingredients "Mise en place" , we can appropriate that word in some fashion and kind of make as a brand. Just my 2 cents...

FWIW: when I and others at Sandstorm read "WeKan," Jade said "Oh, like a kanban board." I also made the same connection in my head and hadn't thought of Obama's campaign- though if it had the word "Yes" in front of it I definitely would have. Maybe this is different for others.
@mquandalle  If we go with WeKan branding, we can always improve the graphics later. The important part(in the context of Sandstorm) is to have it in the App Market by Sunday so that it will be ready for Launch. However, if we get the SPK package sooner we will have time to test it thoroughly.
For the sake of time, I created a preview of the app graphics using the source file you sent me. I'll create a pull request shortly for 2 sets of graphics so that you'll have graphics available whether you decide to go with LibreBoard branding or Wekan branding.

Looks really neat @neynah :smile:

Thanks @fjaguero !
I also came up with a slight improvement for LibreBoard which I think is pretty cool- but my opinion is biased.

This is indeed very cool @neynah :-) Would that concept work with the Wekan name?

These are really great!!! @neynah

awesome work, @neynah!
love the second concept...

@mquandalle
I rounded the edges of the square outlines to better match the type. My only issue with this is that "We" is not super legible due to the e being attached. But the source file has the entire type logo outlined so I won't be able to change that without some surgery. (But maybe you think it's OK as it is.)

Nice work @neynah  Looks good.  How about using 'Wk' instead, if 'e' is not legible....

@nata-goddanti I gave it a shot but I don't think the k flows as smoothly. (Probably because the typography wasn't originally designed for this.)

)

oh ok...I see what you are saying. I confess  the word it is kind of growing on me :-)

+1 for the "We" since the logo has more whitespace as @neynah said.

Thanks for all the feedback everyone. I'm in a rush so I went ahead and created PR #247 with the blue tron-like graphic sets for LibreBoard and Wekan. I exported for Sandstorm purposes but they're editable so feel free to play around with them for other purposes. If you're having trouble, let me know. :)
@mquandalle we're excited to see this on the App Market soon! :D

awesome work @neynah  :+1:  :+1:

Awesome Néna, I'll merge the wekan logo as part of the upcoming re-branding (by the way I'm dubious we had the right to use the Libreboard font).
I'm also in a rush to fix remaining bugs, but that will be ready for the Sandstorm app market launch.
Thank you very much :)

@mquandalle So it's official now? Can I rename Libreboard to Wekan in the Wiki?

Yes. I'll probably re-brand tomorrow.

Done.


Hooks error: Uncaught ReferenceError: AutoForm is not defined

Hi there,
when I try to add Hooks to autoform-ionic I get the error message:
Uncaught ReferenceError: AutoForm is not defined
I think this is because in your package.js file aldeeds autoform package is just added but not exported of some kind. Is this right?
When I add aldeeds autoform additionally to this package it works fine, but I am wondering whether it makes sense to have both packages installed and also two different autoform versions then.
Wouldn't it be better to export the autoform object from this package so it can be used like so
AutoForm.hooks(...
Regards

I have the same issue. What works for me:

remove autoform-ionic
add<EMAIL_ADDRESS>add autoform-ionic


Have uxm_relock run take_noise even if tracking is bad

Quietly returning without noise being taken is actually more annoying that waiting for bad noise.

We removed the lines that exit out if there are too many bad channels in tracking, so this should be solved


DNS testing validation scenarios and automation

DNS testing validation scenarios Like below and other scenarios
{ }
null

Getting addressed by multiple commits:

https://gerrit.openbmc-project.xyz/#/c/openbmc/openbmc-test-automation/+/23910/

automated and merged. closing this


Threshold link in Risk Assessment still current?

Your Question

Website URL Path: https://www.coronawarn.app/en/faq/#encounter_criteria

Line / Paragraph: 2nd paragraph
"If the calculated risk exceeds a certain threshold specified by the Robert Koch Institute, users will receive a notification regarding an increased risk."

Question: Does the link threshold  to document solution_architecture.md Section #risk-score-calculation cover the v2 risk calculations? It seems like the document solution_architecture.md is out-of-date as far as risk score calculations are concerned.

v2 calculations are described in https://www.coronawarn.app/en/faq/#encounter_19_calc

I can answer my own question:
The link threshold to document solution_architecture.md Section #risk-score-calculation only describes the v1 risk calculations. The document solution_architecture.md is out-of-date as far as risk score calculations are concerned (see https://github.com/corona-warn-app/cwa-documentation/issues/453).
Suggested change
In
https://www.coronawarn.app/en/faq/#encounter_criteria and
https://www.coronawarn.app/de/faq/#encounter_criteria

Remove the link in the 2nd paragraph to
https://github.com/corona-warn-app/cwa-documentation/blob/master/solution_architecture.md#risk-score-calculation. The text can remain without the link.
Optionally, link this article, https://www.coronawarn.app/en/faq/#encounter_criteria, to the detailed v2 risk calculation description in https://www.coronawarn.app/en/faq/#encounter_19_calc for both languages respectively.

I can answer my own question:
The link threshold to document solution_architecture.md Section #risk-score-calculation only describes the v1 risk calculations. The document solution_architecture.md is out-of-date as far as risk score calculations are concerned (see https://github.com/corona-warn-app/cwa-documentation/issues/453).
Suggested change
In
https://www.coronawarn.app/en/faq/#encounter_criteria and
https://www.coronawarn.app/de/faq/#encounter_criteria

Remove the link in the 2nd paragraph to
https://github.com/corona-warn-app/cwa-documentation/blob/master/solution_architecture.md#risk-score-calculation. The text can remain without the link.
Optionally, link this article, https://www.coronawarn.app/en/faq/#encounter_criteria, to the detailed v2 risk calculation description in https://www.coronawarn.app/en/faq/#encounter_19_calc for both languages respectively.

These documents are still supposed to be updated, but this process is still ongoing. It might be okay to remove them for now and add the links back later. If there are no news on this on Thursday, I'll remove the links temporarily like you suggested.

Corona-Warn-App Open Source Team

These documents are still supposed to be updated, but this process is still ongoing. It might be okay to remove them for now and add the links back later. If there are no news on this on Thursday, I'll remove the links temporarily like you suggested.

Corona-Warn-App Open Source Team

@heinezen
https://github.com/corona-warn-app/cwa-documentation/issues/453 has the label Fix 1.12, so I assumed that the documents would not be updated so quickly.

@heinezen
https://github.com/corona-warn-app/cwa-documentation/issues/453 has the label Fix 1.12, so I assumed that the documents would not be updated so quickly.

@MikeMcC399 PR is ready. See https://github.com/corona-warn-app/cwa-website/issues/803 for the changes.

@MikeMcC399 PR is ready. See https://github.com/corona-warn-app/cwa-website/issues/803 for the changes.

@heinezen
#813 LGTM
Thank you for implementing! 🙂

@heinezen
#813 LGTM
Thank you for implementing! 🙂

Merged! I suggest we leave this issue open so that we don't forget to reinegrate the new doc links once they are ready.

Merged! I suggest we leave this issue open so that we don't forget to reinegrate the new doc links once they are ready.


✨ Implement ILogger for supporting libraries

Is there an existing issue for this?

[X] I have searched the existing issues

Suggestion
There are a few places where we emit data and instead of using Console.Write(), we can implement an ILogger for this.
Possible Implementation
No response
Anything else?
No response

Further, it would be useful to be able to log to the file system or optionally a remote target while the application is being used.


Put app in foreground incoming call #343

1- any incoming call should put the application in foreground
2- there are 4 types of notification
-> listening for events
-> message one
-> incoming call
-> call in progress
3- fix some issues

EventStreamService
Notification IDs line 110: unify naming with a prefix such as NOTIF_ID_.. (ex: NOTIF_ID_MESSAGE_NOTIFICATION)
onIncomingCall(): same conditon seems to be tested twice (state vs IMXCall.CALL_STATE_CREATED)
checkNotification() line 1025: add inforation to log (Log.d(LOG_TAG, "checkNotification " + "SessId="+mNotificationSessionId + " - " + "RoomId="+mNotificationRoomId + " - " + "EventId="+mNotificationEventId);)
prepareNotification() line 819: log to be updated
NotificationUtils
buildIncomingCallNotification() comment header: add info that this method also starts the VectorHomeActivity which is in charge of centralizing the incoming call flow.
buildIncomingCallNotification(): some sanity checks may be missing..
VectorCallViewActivity
getActiveCall(): add some light comments to explain the code where hideCallNotifications() may be called if the call cannot be resumed..


Fax.File.Pdf.Bytes returns JSON if API returns error

If the API returns an error when an attempting to access the fax file (say, 429 rate limit exceeed), fax.File.Pdf.Bytes apparently returns the JSON.  It should raise an exception instead

ETA on this is US Saturday.

@jnankin I'm not able to reproduce this issue - I added a test that covers this case and the code works as excepted, throwing an exception if a JSON error is returned.

Ok thanks
On Oct 20, 2017 11:08 PM, "Noel Herrick"<EMAIL_ADDRESS>wrote:

@jnankin https://github.com/jnankin I'm not able to reproduce this
issue - I added a test that covers this case and the code works as
excepted, throwing an exception if a JSON error is returned.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phaxio/phaxio-dotnet/issues/11#issuecomment-338362287,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAhFEZLMX8qeK18zpWh9nHPwh_q-1Rydks5suW4zgaJpZM4P56wa
.


Shield icon for new modmail

@Jasius The first one looks pretty flush to me.

I might stick with the old icon whenever old modmail gets depreciated.


Skypack turbo down

Just heard a report from one of our customers, it sounds like this may have happened as early as Thursday. Didn't spot it earlier as we use it for a kind of external customer portal section of our app.
We were just using the unversioned link from the Turbo handbook docs: https://cdn.skypack.dev/@hotwired/turbo
Cheers,
Oli
/*
 * [Package Error] "<EMAIL_ADDRESS>could not be built.
 *
 * No build output found.
 *
 * How to fix:
 *   - If you believe this to be an error in Skypack, file an issue here: https://github.com/skypackjs/skypack-cdn/issues
 *   - If you believe this to be an issue in the package, share this URL with the package authors to help them debug & fix.
 *   - Use https://skypack.dev/ to find a web-friendly alternative to find another package.
 */

console.warn("[Package Error] \"<EMAIL_ADDRESS>could not be built. ");
throw new Error("[Package Error] \"<EMAIL_ADDRESS>could not be built. ");
export default null;

Quick fix:
<!-- <script type="module"> import Turbo from 'https://cdn.skypack.dev/@hotwired/turbo';  </script> -->
<script src="https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>async type="text/javascript"></script>

Same thing here.

Sadly this is an issue on Skypack's part.
See hotwired/turbo#731 and skypackjs/skypack-cdn#319 for context.

Hey @oliwoodsuk, were you able to resolve this? The latest version on skypack seems to be working fine again!

Hey @marcoroth ,
Seems to be working again now, but have since moved to using Jsdelivr.
Great work on the Outlets API for Stimulus by the way!

Thank you @oliwoodsuk!
Can we close this issue?


Lots of 504s, confused about server settings

Hi, firstly thanks for making this, it's been invaluable in our attempts to make our site rank well.
We are getting a lot of 504 errors with the reason being given in console that PhantomJS restarted in the middle of the request. Is there a way to avoid this happening?
Here is my server config:
PRERENDER_SERVER_PORT=4000
REDIS_URL="redis://localhost:4001/0"
PAGE_TTL=604800
RESOURCE_DOWNLOAD_TIMEOUT=10000
PAGE_DONE_CHECK_TIMEOUT=300
WAIT_AFTER_LAST_REQUEST=2000
JS_TIMEOUT=10000
JS_CHECK_TIMEOUT=2000
NO_JS_EXECUTION_TIMEOUT=3000
EVALUATE_JAVASCRIPT_CHECK_TIMEOUT=300
PHANTOM_CLUSTER_NUM_WORKERS=16
PHANTOM_WORKER_ITERATIONS=500
PHANTOM_WORKER_SOFT_ITERATIONS=100

I'm confused between the iterations and softIterations settings and how they work together/against each other, would you mind clarifying how these can be set optimally?
Is it possible to stagger PhantomJS restarting so they don't all restart at a similar time? The 504s seem to come in batches and it's when all the PhantomJS instances are restarting.
Do any of the settings jump out at you as being the cause?
I'm happy to send you some server logs if that helps.
Server is running Windows Server 2012 with the following specs:
Intel E3-1220 v1 processor (quad core 3.1GHZ from 2011)
32GB RAM
2 X 500GB SSD (RAID 1)

softIterations and iterations both mean how many pages are rendered before the PhantomJS server is restarted. The difference is that softIterations tries to wait until there are no pages currently being rendered so that it doesn't interrupt anything. iterations does not wait and will shut down PhantomJS while it's rendering a page and cause it to return a 504 for any pages currently being rendered during shutdown. These restarts are to free up RAM that PhantomJS holds onto.
So I would suggest setting softIterations to something low like 10 (restart PhantomJS often if no requests are in flight) and set iterations to something like 50-100 depending on how much RAM your machine has.
I would recommend setting the number of workers equal to the number of CPUs you have on your machine. If you are still seeing lots of 504s after that, you might be sending too many requests to that Prerender server and might need to start a few Prerender servers behind a load balancer.
That being said, are you using an older version of the Prerender server? Those configs looks like configs from old version of our server.

@thoop I don't think this problem is caused by iterations and softIterations.
I have this problem too. I believe there is something wrong with phridge (PhantomJS)
Here is my problem info:
By just serving one request correctly other request will face 504 errors:
2017-05-05T20:44:49.721Z getting https://manmamanam.com/explainpost.html?PostId=13090&name=میم مثل مد&category=مد و زیبایی&categoryId=13&pageid=62#!/post/12920/کلینیک پوست دکتر زارعی/مد و زیبایی/13/2
2017-05-05T20:44:58.707Z Experiencing infinite javascript loop. Killing phantomjs...
2017-05-05T20:44:58.803Z got 504 in 9297ms for https://manmamanam.com/explainpost.html?PostId=286&pageid=54&category=خانه و خانواده&categoryId=24&name=میم مثل مزرعه#!/post/12257/میم مثل مهربانو/خانه و خانواده/24/52
2017-05-05T20:44:58.851Z PhantomJS restarted in the middle of this request. Aborting...

But when i use service.prerender.io all pages will render.
Im using master branch by the way. last commit: cb73d8605e
Thanks

I think my problem is caused primarily by the number of requests incoming
to the server, our site has a huge amount of pages to index. The repeated
requests mean that there are not enough processes to handle them all and
still have time to restart phantom when it's needed.
On 6 May 2017 8:23 pm, "Alisina Bahadori"<EMAIL_ADDRESS>wrote:

@thoop https://github.com/thoop I don't think this problem is caused by
iterations and softIterations.
I have this problem too. I believe there is something wrong with phridge
(PhantomJS)
Here is my problem info:
By just serving one request correctly other request will face 504 errors:
2017-05-05T20:44:49.721Z getting https://manmamanam.com/explainpost.html?PostId=13090&name=میم مثل مد&category=مد و زیبایی&categoryId=13&pageid=62#!/post/12920/کلینیک پوست دکتر زارعی/مد و زیبایی/13/2
2017-05-05T20:44:58.707Z Experiencing infinite javascript loop. Killing phantomjs...
2017-05-05T20:44:58.803Z got 504 in 9297ms for https://manmamanam.com/explainpost.html?PostId=286&pageid=54&category=خانه و خانواده&categoryId=24&name=میم مثل مزرعه#!/post/12257/میم مثل مهربانو/خانه و خانواده/24/52
2017-05-05T20:44:58.851Z PhantomJS restarted in the middle of this request. Aborting...
But when i use service.prerender.io http://service.prerender.io all
pages will render.
Im using master branch by the way. last commit: cb73d86
https://github.com/prerender/prerender/commit/cb73d8605e1c4d13c1faad7ffbbe4f5044131625
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prerender/prerender/issues/396#issuecomment-299656689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxaQYicb88ZV6JgTOgd8xtoCuRW3dwOks5r3Lb_gaJpZM4NKA0G
.

@alexstanbury did you increase iteration time? If so did it work?
Is there any other errors before PhantomJS restart. (In my case there is a infinity loop exception)

Increasing iteration values does help but putting it too high causes
phantom to crash more than it already does. I do see phantom errors and
crashes still but I am having to live with the 504s for now in the hope
that next time the page is crawled it will be ok. Not ideal but better than
nothing.
On 6 May 2017 8:58 pm, "Alisina Bahadori"<EMAIL_ADDRESS>wrote:

@alexstanbury https://github.com/alexstanbury did you increase
iteration time? If so did it work?
Is there any other errors before PhantomJS restart. (In my case there is a
infinity loop exception)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prerender/prerender/issues/396#issuecomment-299662599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxaQZdligpZSRLPeZ40shbfK32frY9kks5r3ND8gaJpZM4NKA0G
.

@alexstanbury I would suggest using more Prerender servers behind a load balancer. Sounds like you are just sending too many requests to the PhantomJS server. Feel free to use our hosted service as we can handle any scale you need :)
@alisinabh Feel free to open a new issue, I don't think what you are seeing is related to @alexstanbury 's problem

@thoop Do you mean using more physical servers or using something like nginx to balance the load between multiple instances of the node app?

I mean more physical servers. One prerender server already does start multiple instances of PhantomJS to load balance itself but we recommend only using a number of workers equal to the number of CPUs on the machine. Anything more than that wont improve performance. So if you hit performance issues after scaling workers, you'll need to scale physical servers behind a load balancer.

Ok yeah that's what I thought. I'll definitely have to look into it.
Prerender.io is a bit out of our reach at the moment unfortunately or else
we'd be using that.  Thanks for your help!
On 8 May 2017 22:18, "Todd Hooper"<EMAIL_ADDRESS>wrote:
I mean more physical servers. One prerender server already does start
multiple instances of PhantomJS to load balance itself but we recommend
only using a number of workers equal to the number of CPUs on the machine.
Anything more than that wont improve performance. So if you hit performance
issues after scaling workers, you'll need to scale physical servers behind
a load balancer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prerender/prerender/issues/396#issuecomment-299993897,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxaQati1uKkBmtua9PKwVFoaqh2Gefiks5r34amgaJpZM4NKA0G
.

@thoop I've checked and yes the problem seems to be insufficient memory. Thank you


Questions about the page

How to implement multi-page Linux applications, is there any related demo, and how to manage pages?

Flet Docs on navigation and routing:
https://flet.dev/docs/guides/python/navigation-and-routing
Packaging with pyinstaller:
https://flet.dev/docs/guides/python/packaging-desktop-app


Obscure error message for flutter run when ios/ directory missing

My app doesn't have an iOS directory.  When I run my app, I get this obscure error message:
2016-02-20 23:00:47.058 defaults[7720:4714481]
The domain/default pair of (/[redacted]/ios/Info, CFBundleIdentifier) does not exist
2016-02-20 23:00:47.114 defaults[7721:4714485]
The domain/default pair of (/[redacted]/ios/Info, CFBundleIdentifier) does not exist

$ ../../third_party/flutter/bin/flutter --version
Flutter from https://github.com/flutter/flutter.git (on master)
flutter revision: 4ec960847219837d017dde64e41d849e176d4ebc (55 minutes ago)
engine revision : 6e736e7a76600640c5bd854c2fc9ed0f664ee26e

[-] The iOS toolchain is partially installed; more components are available.
[✓] The Android toolchain is fully installed.

Run "flutter doctor" for information about installing additional components.

/cc @chinmaygarde @devoncarew

flutter create should create the iOS directory for you now. Did you manually nuke this directory?

In my case, I'm working on an old app that never had an ios/ directory.  However, we should be robust to people nuking the ios (and android) directories if they don't care about ios (or android).

We can create a manifest of all files created by flutter create and check for their existence on run. Or at least add a doctor check.

I think the ideal behavior would be to just say nothing until I attempt to run the app on iOS.  At that point, the tool should tell me that my ios directory is missing or corrupt and offer to create one for me.

flutter run with a connected Android device works fine even if I delete my ios/ directory.
When an iOS device is connected, I get the message...
$ flutter run
No application found for TargetPlatform.ios.
Is your project missing an ios/Runner/Info.plist?
Consider running "flutter create ." to create one.

Running flutter create fixes the problem.

this problem is so long ,but flutter owner not see it ,


BREAKING CHANGE: refactor -  showcase support

This is great!

christ - thanks @rjanjua  @Mnargh @Mnargh 😂


Get numpy arrays out of .tf_record

I've been trying (unsuccessfully) for the last hour or so to transform your dat/PeerRead/proc/arxiv-all.tf_record into numpy arrays that one can actually read.
(1) I don't think your ReadMe currently points to your array_from_dataset.py file which I had to dig around for awhile to find.
(2) When I run python array_from_dataset.py from src/PeerRead/dataset/ folder I get errors like ModuleNotFoundError: No module named 'bert' which makes me think I'm running it from the wrong folder. Where should I be running this from or what command should I use?
Do mind adding some documentation or cleaning up the code to make it easier to transform .tf_record into numpy arrays? Thanks!

Thanks for the feedback -- we'll update/improve the documentation to describe how to create arrays from tf_record objects. As for your question, you should call the script from the src folder directly as:
python -m PeerRead.dataset.array_from_dataset

We've updated the readme to clarify this now.
Note that this file creates tsv files to use as input to our baselines. In particular, the output is already tokenized and cleaned. Depending on your use case (i.e., if you want to do different pre-processing) you might be better off downloading the original PeerRead data and adapting our data cleaning scripts.

Great! Thank you so much for the quick response time!


fix model_zoo.md

Thank you for your contribution to the MindYOLO repo.
Before submitting this PR, please make sure:

[x] You have read the Contributing Guidelines on pull requests
[x] Your code builds clean without any errors or warnings
[x] You are using approved terminology
[ ] You have added unit tests

Motivation
fix model_zoo.md
Test Plan
(How should this PR be tested? Do you require special setup to run the test or repro the fixed bug?)
Related Issues and PRs
(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

Create an issue and associate it with PR

Modify "Supported Algorithms" in file mindyolo/README.md.


MWR-39 Implement BlockExplorer (Transaction history) from X-Networking

XNetworking is already used in FW since we implemented history for SORA assets


Resolved #26 Png's not Loading

Resolved issue by adding d.ts in demo root to type and export png's.

LGTM!


Example doc for random seed

Add an example doc to
https://github.com/near/near-sdk-rs/blob/bb8a7f8c796e3dd2bd22b4066c97251288f7d84b/near-sdk/src/environment/env.rs#L261
The doc should showcase how to construct a struct that implements SeedableRng from the random_seed and uses Rng::gen_range to get a random value in a range with the seed.
The doc should also contain a warning that the random seed should not be used as input to cryptographic primitives.

The doc should also contain a warning that the random seed should not be used as input to cryptographic primitives.

Random seed in NEAR is quite well-thought, so it can be used for security-critical needs, but the public nature of the blockchain might be the blocker anyway. So I would not add that warning.

Thanks for raising this @frol , the issue warrants more discussion, so I wrote it up in detail.
As explained in the paper, the goal of the NEAR randomness beacon is twofold:

Unpredictable. No participant should be able to predict the resulting
number or reason about any properties of it before the number is created.
Unbiasable. No participant should be able to influence the resulting
number in any way.

These goals work well (with some caveats) for use cases such as on-chain betting where the quality of the randomness only matters during the execution of the transaction by the block producer.
Many cryptographic primitives such as encryption algorithms rely on random values as inputs. These random values are generated with cryptographically secure pseudorandom number generators (CSPRNG). The requirement on CSPRNGs is that their output should be undistinguishable from a truly random distribution by an adversary.
CSPRNGs are called pseudorandom, because their outputs depend on a seed value. If the attacker learns about this seed value, they can distinguish the CSPRNG output from a random distribution, so the security of the CSPRNG and all the cryptographic primitives that depend on it is broken.
Poor seeding of a CSPRNG is a common source of vulnerabilities. The best practice to avoid this is to use the secure random number generator API exposed by the OS (e.g. getrandom on Linux) to seed the CSPRNG used by the application. The expectation with values returned from the OS API is that they cannot be recovered once they're dropped from memory by the application.
My understanding is that the value returned in near_sdk::env::random_seed is the same within a block, it's known by the producer, and we should assume that the seed values are recoverable for historical blocks. (I don't understand well enough how it works to give a solution for all blocks, but trivially, it could be logged by a contract.) This means that if near_sdk::env::random_seed is used to seed a CSPRNG, all the outputs of the CSPRNG will be recoverable by an adversary. This is very different from the expectation on seeds for CSPRNGs which is why I thought it appropriate to issue a warning that the random seed should not be used as input to cryptographic primitives.
To give a concrete example of how things could wrong, consider an NFT where some secret property of the NFT is only revealed to the owner, but the owner can selectively reveal it to other blockchain users. A naive implementation could solve this by generating a data encryption key (DEK) within the NFT contract using a CSPRNG seeded with near_sdk::env::random_seed, then encrypt the secret property with the DEK, then encrypt the DEK with the public key of the owner and then store the encrypted DEK in the contract. The owner could then retrieve the encrypted DEK and encrypt it with the public key of another user and store the newly encrypted DEK in the contract to let the other user access the secret property.
Now this scheme could be trivially broken, since the block producer knows the seed value, other contracts executed within the same block also know the seed value and historic seed values are recoverable. But if you turned the smart contract into a Linux program and replaced the call to near_sdk::env::random_seed with a call to getrandom, the Linux program would be secure. This is why I think some users might be surprised by how near_sdk::env::random_seed behaves.
Question to me is how shall we best communicate the difference? Maybe we could explain that seed values are public and this can be a problem for use in cryptographic primitives?

Maybe we could explain that seed values are public and this can be a problem for use in cryptographic primitives?

@agostbiro I feel this is the main blocker and not the lack of a cryptographically secure random generator property.

there should also be a link to https://docs.near.org/develop/contracts/security/random
Consider mentioning https://lib.rs/crates/near-rng (I did not look into the code, but from the docs it is promissing)
https://stackoverflow.com/questions/71092791/how-to-select-a-random-key-from-an-unorderedmap-in-near-rust-sdk


Add support for handling multiple TouchEvents in one OnTouchEvent notification

Fixes #972 and it is based on #973
This PR is ready for review.
Risk
This PR makes no API changes.
Testing Plan
Unit tests will be added to test converting one OnTouchEvent that has multiple TouchEvents to multiple MotionEvents
Summary

Update VideoStreamManager to be able to convert one Sdl OnTouchEvent to multiple Android MotionEvents
Update SdlProxyBase to be able to convert one Sdl OnTouchEvent to multiple Android MotionEvents
Pass list of all active pointers to MotionEvent so the developer can later get the correct number of active pointers from the MotionEvent and the coordinates for each pointer
Update the converted MotionEvent actions to exactly match the native Android ones. For example, if the user touches the screen using two fingers at the same time, the action will be ACTION_POINTER_DOWN (1) instead of just ACTION_POINTER_DOWN for the second touch

Tasks Remaining:

[x] [Update VideoStreamManager]
[x] [Update SdlProxyBase]
[x] [Write new tests]

CLA

[x] I have signed the CLA

Codecov Report

Merging #1071 into develop will increase coverage by 0.13%.
The diff coverage is 30.76%.

@@              Coverage Diff              @@
##             develop    #1071      +/-   ##
=============================================
+ Coverage      45.89%   46.02%   +0.13%
- Complexity      3686     3689       +3
=============================================
  Files            433      433
  Lines          20224    20273      +49
  Branches        2180     2182       +2
=============================================
+ Hits            9281     9330      +49
+ Misses         10476    10466      -10
- Partials         467      477      +10

Impacted Files
Coverage Δ
Complexity Δ

...n/java/com/smartdevicelink/proxy/SdlProxyBase.java
6.42% <0%> (-0.05%)
16 <0> (ø)

...tdevicelink/managers/video/VideoStreamManager.java
30.93% <63.41%> (+19.51%)
8 <3> (+3)
:arrow_up:

...smartdevicelink/encoder/VirtualDisplayEncoder.java
24.43% <0%> (-4.55%)
8% <0%> (ø)

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 6419642...da44d19. Read the comment docs.

@shiniwat can you confirm this fix works for you? This PR is based off your work in #972.

@joeygrover Sorry for the slow response, but I found a bunch of implementation difference between our PR and feature/multi_touch_events branch, including unit test. I am trying to run the equivalent unit test code in multi_touch_events branch right now...

We're double-checking this change on our end. Please allow us to review this in next couple of days. Thanks.

@shiniwat we will need to merge this PR by 2019-06-07 in order to stay on track with our 4.9 release timeline.  Please provide any feedback by that time.  Thank you!

We double checked this change, running the unit test we had in our PR, along with running our test app with this change. Finally finished verification and the change works fine. Thanks.

Awesome. Thanks for confirming that.


Username and password not acceptable

I don't understand please help me out and if I remove the @gmail.com from the address it still print out another error saying email provider not supported

 I don't understand please help me out and if I remove the @gmail.com from the address it still print out another error saying email provider not supported

I HAVE PROGRAMMED THE VERY SIMILAR TOOL
I have Release It Just Now It's also Generates Phishing emails and also allow users to send them. You don't need to own an email already Configured! Click Here  Or Just go manually
https://github.com/Z3R0SEC/Bait

Gmail has stopped the third party ability to send emails, in version 2 of phishmailer it will have its own dedicated mailservers to send out, it is in the works


OutOfMemory Exception

Hi, when using the new ChunkDownloader with default 4 threads on a large dataset I get an OutOfMemoryException in GetChunkContentAsync. The underlying JsonSerializer blows up when trying to create a List
If I set it to 1 thread no issues. Same request.

Do you have any more details for this?  E.g. a call stack trace? Do you know the approximate size of the data you were trying to retrieve? Thanks.

Can you remember if your dataset had large binary columns in it? And if so do you an approximate size of the largest binary results? Thanks.

Hi, the issue was the large payload, the result set was above 45M rows no binary data just varchar, timestamp and number. The OOM exception was triggered internally in the JsonSerializer more specifically when trying to new up a List in the ChunkDownloader class. The problem has been made worse by the new parallel chunk downloader as is using multiple threads increasing memory pressure. If I set the chunk downloader to 1 thread then it worked fine but obviously is much slower. I've changed my implementation to pagination to avoid massive datasets.

@georgebarbu Hi,
I'm gonna release new version with @colgreen great improvements in memory allocations - this might help.

The update looks fine


jax_debug_nans doesn't work with complex numbers

Somehow JAX can't detect nans coming from operations on complex numbers:
import jax.numpy as jnp

def nan_with_complex(x):
    return x * 0 / 0

print(nan_with_complex(jnp.array([42.j])))

JAX_DEBUG_NANS=true python script.py

[nan+nanj]

Closing in favor of #2612


[BUG] Ancient Tome of the Gods can't be used again after disenchanting it via Grindstone

When using a looted Ancient Tome of the Gods with an enchantment in the grindstone in order to remove the enchantment and re-use the book in another enchantment, the book becomes unusable for crafting other enchantments.
This is because when disenchanting an enchanted Ancient Tome of the Gods leaves it as a "minecraft:enchanted_book" item, while the naturally crafted one is a "minecraft:book".
When you use a regular minecraft enchanted book on a grindstone, however, it goes back to being a regular book. A vanilla Enchanted Book shares the same item type ("minecraft:enchanted_book"), but after grinding it, it goes to ("minecraft:book").
I believe this is a minor oversight that could be fixed. If I find a Ancient Tome of the Gods with an enchantment I already have/don't need and try to disenchant it, the book goes back to normal "non-enchanted" book, but it is still an enchanted book. This means I can't recycle it into enchantments that I really want, but theoretically should be possible.

Example of naturally crafted Ancient Tome of the Gods. Notice the item tag/name is "minecraft:book".

Example of a disenchanted Ancient Tome of the Gods. Notice the item tag/name is "minecraft:enchanted_book".

Example of a Vanilla Enchanted Book which has been disenchanted via a grindstone. It goes back to being a "minecraft:book".

Step 1 of disenchanting

Step 2 of disenchanting

I am not sure if this is relevant or not, but when I first had disenchanted the AtotG, it had lost the "enchantment glint", characteristic of enchanted items. It was still an enchanted book, but it had lost the "glint/glow" effect.
When I spawned another AtotG via commands, however, it had this glint/glow and somehow the old book also got it back. I am not sure how, since I had put it away in a chest (so the items wouldn't stack, which now I know they wouldn't, because minecraft:enchanted_book don't stack, only minecraft:book).
As far as I could tell, this was only a visual issue and it had no effect in recipes not working. The whole issue seems to originate in the fact of the books being different items (minecraft:enchanted_book vs minecraft:book).

A fix for this will be present in the upcoming 1.21.3 release


(1.2) uninitialized constant Flipper::CLI::IRB (NameError)

Excited to check out the 1.2 CLI but running into this error around IRB colorization. flipper --help runs correctly but list, show, enable and disable are giving the same NameError.
I'm on Ruby 3.2.2. via asdf. Any ideas?
❯ flipper list
/Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/lib/flipper/cli.rb:213:in `colorize': uninitialized constant Flipper::CLI::IRB (NameError)

      IRB::Color.colorize(text, options)
         ^^^^^^^
Did you mean?  ERB
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/lib/flipper/cli.rb:152:in `block in feature_summary'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/lib/flipper/cli.rb:149:in `map'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/lib/flipper/cli.rb:149:in `feature_summary'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/lib/flipper/cli.rb:68:in `block (2 levels) in initialize'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/lib/flipper/cli.rb:232:in `run'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/lib/flipper/cli.rb:115:in `run'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/lib/flipper/cli.rb:6:in `run'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/flipper-1.2.0/exe/flipper:5:in `<top (required)>'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/bin/flipper:25:in `load'
	from /Users/mikemanewitz/.asdf/installs/ruby/3.2.2/bin/flipper:25:in `<main>'

The same error here on Ruby 2.7.8 and 3.1.4 via rbenv. Manually requiring irb in the cli code in the gem does NOT solve it and moves the error to a point where IRB::Color::WHITE is referenced.

here's mine:
❯ rails --version
Rails 7.1.2
❯ irb --version
irb 1.11.1 (2024-01-08)
❯ gem --version
3.5.4

Ok, I was finally able to duplicate this in a Rails 7.0 app. Rails 7.1 added a dependency on the irb gem, which is where new development is happening to try to get parity with pry.
So we either need to make Flipper depend on irb gem as well, or just make colorization optional if IRB::Color is defined. The latter would probably be my preference.


Compatibility with Keras predict API and allow to export to SavedModel

These little fixes allow GLOW to run with tf.keras API like predict, and export to SavedModel

Great updates! The commits seem good to me.

Great updates! The commits seem good to me.


Add better colour support

Love this plugin, would be great if it had better colour support for different themes.

Yes! It would be nice to be have highlight groups for things like the selection row, which is kinda hard to see atm.

Hello, greate plugin. But what's that weird background (with some kind of opacity) in UI? I've checked with all popular themes like gruvbox, onedark, oceanic and it's there.

Yep would be great to customise I can't even see whats selected with mine. I think its just the highlights.

I definitely need to work on this, thanks for all the feedback. I will prioritize it this coming week. 😊

legend

Addressed in: https://github.com/camspiers/snap/commit/ee87275c0f99491fe3804bfa9fb76b7e42b778cc
Please let me know if you need any more customization.
You can define your own colors with:
SnapSelect
SnapMultiSelect
SnapNormal
SnapBorder
SnapPosition
SnapPrompt

But they all should now have more reasonable defaults from whatever colorscheme you are using.

Please feel invited to provide more color feedback here, I will keep watching this issue, and in fact I will leave it open for a while. The defaults I have chosen seem reasonable and I have checked in a few themes where it seems to work well.

Hello, It's a lot better now, but it's still has vertical line

I've tried playing around with all the highlight settings, but I seem to be having this issue still. Not sure if its just me. Not able to see where the cursor is. https://github.com/camspiers/snap/issues/9#issuecomment-864380570

@LoydAndrew that might be cursorcolumn highlighting. I'll have a look into it when I get time.

@beauwilliams Are you sure you have updated? Unless your theme doesn't set a highlight for WildMenu it should no longer be an issue, and if your theme doesn't set a highlight for WildMenu it should (but otherwise you can set a default).

I have just done a git pull and checked with :highlight, very odd seems that the group is set correctly and my wildmenu colours should be visible. That must not be the issue then

@LoydAndrew I have resolved your cursorcolumn issue in https://github.com/camspiers/snap/commit/c9d6dcc516d096f022bd30d0bb2640a3253d44ea

Yeah, thanks it works. There is however a new issue C-u C-d keybindings stop working after new changes but that's separate issue.

@LoydAndrew fixed in https://github.com/camspiers/snap/commit/bbd864b40e96b22459ff660e696a07e2f333392d


Issue with Angular 2 template when using d3

Hello,
Hopefully this question is not too off topic, but I'm getting a really weird issue with the yo aspnetcore-spa template when trying to use d3.js with it. The issue seems to be somehow linked to something in the template because the same code doesn't have the behaviour in an Angular CLI project.
I'm using d3.js to display a simple bar chart I found as an example online. It's a component that imports d3 and uses it to add an SVG and create the bar chart. When I access it through a link I added to the template's left menu, everything works fine, the chart is displayed just fine. But, if I try to access it directly through the URL I've configured as route for the component, I get this exception thrown:
Exception: Call to Node module failed with error: TypeError: Cannot read property 'createElementNS' of undefined
at Object.<anonymous> (C:\sources\d3NetCoreAngular2\ClientApp\dist\main-server.js:9352:33)
at Object.<anonymous> (C:\sources\d3NetCoreAngular2\ClientApp\dist\main-server.js:10129:39)
at Selection.selection_select [as select] (C:\sources\d3NetCoreAngular2\ClientApp\dist\main-server.js:9561:53)
at Selection.selection_append [as append] (C:\sources\d3NetCoreAngular2\ClientApp\dist\main-server.js:10128:18)
at BarchartComponent.createChart (C:\sources\d3NetCoreAngular2\ClientApp\dist\main-server.js:474:39)
at BarchartComponent.ngOnInit (C:\sources\d3NetCoreAngular2\ClientApp\dist\main-server.js:460:15)
at AppView._View_BarchartComponent_Host0.detectChangesInternal (BarchartComponent_Host.ngfactory.js:28:86)
at AppView.detectChanges (C:\sources\d3NetCoreAngular2\ClientApp\dist\vendor.js:12652:19)
at AppView.detectContentChildrenChanges (C:\sources\d3NetCoreAngular2\ClientApp\dist\vendor.js:12670:24)
at AppView.detectChangesInternal (C:\sources\d3NetCoreAngular2\ClientApp\dist\vendor.js:12662:19)

I'm really lost as to why there would be a difference between accessing the component through a link or directly. Could it be because of a server-side pre-rendering? Is there something else in the template that would make a difference between the two ways of accessing? I've tried adding a breakpoint in the typescript, if I access through the link, the breakpoint is triggered and the code stopped, but if I access directly (or if I refresh the page), then the breakpoint is not triggered and I get the exception.
Here are the important parts of the component's typescript:
...
@Component({
    selector: 'barchart',
    template: require('./barchart.component.html'),
    styles: [require('./barchart.component.css')],
    encapsulation: ViewEncapsulation.None
})
export class BarchartComponent implements OnInit, OnChanges, AfterViewInit {

    @ViewChild('chart')
    private chartContainer: ElementRef

    ...

    ngOnInit() {
        this.htmlElement = this.chartContainer.nativeElement;
        this.host = d3.select(this.htmlElement);
        let svg = this.host.append('svg')
            .attr('width', this.width + this.margin.left + this.margin.right)
            .attr('height', this.height + this.margin.top + this.margin.bottom);
        ...
    }
    ...
}

The html is pretty simple:

I've uploaded the whole project in this project: https://github.com/Gimly/d3NetCoreAngular2

Can you try wrapping the logic inside of the ngOnInit with the code below and see if it works? D3 doesn't readily get rendered on the server so you might have to make sure it's only rendered on the client.
import { isBrowser } from 'angular2-universal'; // add this uptop

if (isBrowser) {
   // your d3 code
}

Thanks! It does work indeed!
I fought that issue for nearly a week, and it's fixed in just a line... 👍 Oh well, that's how you learn.
Does it mean that my guess was correct? When I access the URL directly, the code is pre-rendered server-side?
Care to explain what in the template activates that server-side rendering?

Well when you refresh, or type an address and press enter you're hitting the server side, which in turn tries to render that specific section of the application with Angular Universal. Once that's rendered to a string and loaded on the page, the client-side bootstrap starts and then takes over, and the rest of the your time navigating is like a normal SPA. You just get that initial server side performance boost, SEO and deep-linking (think social media sharing), that you now get because of the SSR :)

Thanks for resolving this @MarkPieszak!

Thanks a lot for your help and explanation @MarkPieszak.

FYI isBrowser() has been switched to isPlatformBrowser.  See Angular Universal Gotchas
import { Component, Inject, PLATFORM_ID, OnInit } from '@angular/core';
import { isPlatformBrowser, isPlatformServer } from '@angular/common';

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.css'],
})
export class AppComponent implements {

  constructor(
    @Inject(PLATFORM_ID) private platformId: Object
  ){
  }

  ngOnInit(){
    if (isPlatformBrowser(this.platformId)) {
     //Client only code.
    } else {
     //Server only code.
    }
  }
}


crash in k4a_device_open() due to index out of range

When an app calls k4a::device::open(0) with no sensors connected...
the app crashes due to a bug in your wrapper near...
https://github.com/orbbec/OrbbecSDK-K4A-Wrapper/blob/84741df67a7f5d71ce061b72da7b0814f93e0573/src/orbbec/ob_k4a_impl.cpp#L267
with
[12/29 05:39:30.540786][info][18268][Context.cpp:67] Context created with config: default config!
[12/29 05:39:30.540810][info][18268][Context.cpp:72] Context work_dir=C:\njs\dp.kinect3\build\orbbec-out?
Loaded 'C:\Windows\System32\winusb.dll'. Symbol loading disabled by Include/Exclude setting.
Loaded 'C:\Windows\System32\hid.dll'. Symbol loading disabled by Include/Exclude setting.
Loaded 'C:\Windows\System32\CompPkgSup.dll'. Symbol loading disabled by Include/Exclude setting.
Loaded 'C:\Windows\System32\Windows.Media.dll'. Symbol loading disabled by Include/Exclude setting.
Loaded 'C:\Windows\System32\Windows.ApplicationModel.dll'. Symbol loading disabled by Include/Exclude setting.
[12/29 05:39:30.586484][info][18268][DeviceManager.cpp:15] Current found device(s): (0)
[12/29 05:39:30.586542][warning][18268][ObException.cpp:5] ob_device_list_get_device_serial_number: index out of range!
SDK-K4A-Wrapper\src\orbbec\ob_type_helper.cpp(43)P2: orbbec_sdk_log(). [12/29 05:39:30.586542][warning][18268][ObException.cpp:5] ob_device_list_get_device_serial_number: index out of range!

Exception thrown at 0x00007FF906C9567C in Max.exe: Microsoft C++ exception: libobsensor::invalid_value_exception at memory location 0x00000000007DAAB0.
Exception thrown at 0x00007FF906C9567C in Max.exe: Microsoft C++ exception: [rethrow] at memory location 0x0000000000000000.
SDK-K4A-Wrapper\src\orbbec\ob_type_helper.cpp(21)P1: check_ob_error(). Inner Orbbec SDK error: ob_device_list_get_device_serial_number: index out of range!, function: ob_device_list_get_device_serial_number, exception: 3
SDK-K4A-Wrapper\src\orbbec\ob_type_helper.cpp(21)P1: check_ob_error(). Inner Orbbec SDK error: ob_device_list_get_device_serial_number: index out of range!, function: ob_device_list_get_device_serial_number, exception: 3
Critical error detected c0000374

this is likely because some part of your wrapper does not check for the scenario of there being NO sensors connected.
Setup

Microsoft Windows [Version 10.0.22631.2861]
VS2022 community v17.8.3
orbbec wrapper v1.8.3

Repro

have no sensors connected to computer
compile app that calls k4a::device::open(0)
run app

Result
App crashes.
When app runs in debugger, I can see errors that your code is not handling the out-of-bounds scenario of there being NO sensors connected.
[12/29 05:39:30.540786][info][18268][Context.cpp:67] Context created with config: default config!
[12/29 05:39:30.540810][info][18268][Context.cpp:72] Context work_dir=C:\njs\dp.kinect3\build\orbbec-out?
Loaded 'C:\Windows\System32\winusb.dll'. Symbol loading disabled by Include/Exclude setting.
Loaded 'C:\Windows\System32\hid.dll'. Symbol loading disabled by Include/Exclude setting.
Loaded 'C:\Windows\System32\CompPkgSup.dll'. Symbol loading disabled by Include/Exclude setting.
Loaded 'C:\Windows\System32\Windows.Media.dll'. Symbol loading disabled by Include/Exclude setting.
Loaded 'C:\Windows\System32\Windows.ApplicationModel.dll'. Symbol loading disabled by Include/Exclude setting.
[12/29 05:39:30.586484][info][18268][DeviceManager.cpp:15] Current found device(s): (0)
[12/29 05:39:30.586542][warning][18268][ObException.cpp:5] ob_device_list_get_device_serial_number: index out of range!
SDK-K4A-Wrapper\src\orbbec\ob_type_helper.cpp(43)P2: orbbec_sdk_log(). [12/29 05:39:30.586542][warning][18268][ObException.cpp:5] ob_device_list_get_device_serial_number: index out of range!

Exception thrown at 0x00007FF906C9567C in Max.exe: Microsoft C++ exception: libobsensor::invalid_value_exception at memory location 0x00000000007DAAB0.
Exception thrown at 0x00007FF906C9567C in Max.exe: Microsoft C++ exception: [rethrow] at memory location 0x0000000000000000.
SDK-K4A-Wrapper\src\orbbec\ob_type_helper.cpp(21)P1: check_ob_error(). Inner Orbbec SDK error: ob_device_list_get_device_serial_number: index out of range!, function: ob_device_list_get_device_serial_number, exception: 3
SDK-K4A-Wrapper\src\orbbec\ob_type_helper.cpp(21)P1: check_ob_error(). Inner Orbbec SDK error: ob_device_list_get_device_serial_number: index out of range!, function: ob_device_list_get_device_serial_number, exception: 3
Critical error detected c0000374

Expected
Your orbbec implemntation shoudl return a K4A error code which then the c++ method in k4a.hpp which convert into an exception.

Thank you for your feedback. We fix this issue in the 'develop' branch.

Great. Please link your commit or PR to this issue so we can track your progress

Great. Please link your commit or PR to this issue so we can track your progress

https://github.com/orbbec/OrbbecSDK-K4A-Wrapper/tree/develop

That is not a commit. A commit is a SHA hash that uniquely identifies a change.

That is not a commit or PR. A commit is a SHA hash that uniquely identifies a change. And a PR is a github-specific group of commits that is used to implement the github workflow.

3aee22ade209a60012fe936df7e3c3fbdb45762c

great. How do yopu want me to give feedback on that commit? You did not use the github PR process so I can now use that standard method.

great. How do you want me to give feedback on that commit? You did not use the github PR process so I can not use that standard method that has a review workflow, comments on code lines, etc. https://docs.github.com/en/get-started/quickstart/github-flow#create-a-pull-request

Thank you for your excitement! PR is not our primary workflow currently, but we should learn to consider using gitflow and PR as a workflow in the future.
This issue I will close as it is now fixed.


How to enable tracking?

Hello Conezi,
I am interested in enabling face tracking. Google has provided the following code. Could you please guide me on how to achieve this?
FaceDetectorOptions options = FaceDetectorOptions(
        performanceMode: FaceDetectorMode.accurate, enableTracking: true);
    faceDetector = FaceDetector(options: options);

    final List<Face> faces = await faceDetector.processImage(inputImage);

    for (Face face in faces) {
      // If face tracking was enabled with FaceDetectorOptions:
      if (face.trackingId != null) {
        final int? id = face.trackingId;
        print("face id is = $id");
      }
    }

Thank you for bringing this to our attention, @VivekVithlani. I appreciate your inquiry. Currently, face tracking functionality is already integrated and enabled by default. In our upcoming release, we'll ensure that this feature remains seamlessly accessible, allowing you the flexibility to enable or disable it simply by passing it as a parameter. If you have any further questions or concerns, please don't hesitate to reach out. We're here to help!


STORM-4017 Fix for isAnyWindowsProcessAlive with multiple pids

What is the purpose of the change
Fixes STORM-4017 I raised recently so that isAnyWindowsProcessAlive works correctly in Windows.
What prompted this was that we are evaluating Storm 2.5.0 under Windows and noticed that when workers are restarted the child processes are left running.
We noticed logs reporting that "None of the processes , ,  are alive" when they were in fact still running.
Further investigation showed why this was happening, hence raised STORM-4017
How was the change tested
Existing unit test 'testIsAnyProcessAlive' in ServerUtilsTest.java did not work under Windows but now does with this fix.
We have confirmed that we only now see the "None of the processes ... are alive" when the processes are genuinely not running
See attached unit test run before and after fix

This request is to merge to 2.4.x-branch. However, we are actaully working with 2.5.0 but notice there is no corresponding 2.5.x-branch to merge in to. Can this be added? (We are working on 2.5.0 as this is the latest version to work with Java8, which we are currently restricted to)

Thanks for the PR. I definitley see it in the next release of Storm. However, given the limited amount of volunteers working on Storm (it was discussed to move it to the attic a couple of months ago due to limited resources / volunteers willing to contribute), there might be no intention / capacity to acutally release a 2.5.x version of Storm. You are free to bring up this topic on the dev@ mailing list.

Therefore, I suggest to target main with the change and post a related mail on dev@


Management of a hierarchy of sub-folders

Thank you very much for this very useful and powerful add-on!
There is currently only one feature that I would like to see improved.
I have a local folders hierarchy with many sub-folders (which doesn't create problems) but with several hierarchical levels, typically:
Local Folders

Family

Fred
Joyce
...

Friends

Ann
Mary
...

Purchases

Amazon
Bookshops
...

The behaviour is fully correct for "Local Folders", "Fred", "Joyce", "Amazon", etc. but not for "Family", "Friends", "Purchases", etc., for which I get 0 messages. Formally, this is true as there are no messages directly in these folders, but what is needed is an option to rather get the statistics of their sub-folders.
Thanks in advance!

Thank you very much for your quick reply. Yes, you correctly understand my suggestion! This is of course the behaviour for the "Local Folders" root itself, but this will be very useful to have an option for that for intermediate levels when we manage a hierarchy of sub-folders.
P.-S. My apologizes for the template not used. I note it.

Alright, I see when I have time to implement this.


Internal node name shown in error

Description:

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This is a duplication of this: https://github.com/ballerina-platform/ballerina-lang/issues/29168


Short solution needed: "How to add watermark to image" (python-pillow)

Please help us write most modern and shortest code solution for this issue:
How to add watermark to image (technology: python-pillow)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to add watermark to image


Rename xstrided_view to xstridedview

to be consistent with the file naming and other view types.

xstridedview.hpp has been renamed in xstrided_view.hpp, the same pattern has been applied to other classes and files. See #565


nextLast() instead of next("last")

fix #11

As https://github.com/tc39/proposal-iterator-helpers/issues/122#issuecomment-1176946383 , iterator helpers do not pass the protocol, so we have to use nextLast().


test: add test case for compacting res0 children

Children of a res 0 cell should compact back to that cell successfully.

It looks like around h3Index.c line 344 it tries to take the parent of a res 0 cell at res -1 (res -1 was calculated on line 325), which is where the failure is coming from.

I'm uncertain of this solution, but it passes existing tests and the new failing test.
Basically if the compaction process reaches a base cell, we skip the parent reserved-bit-hit processing and move on to the rest of the logic.

Coverage decreased (-0.03%) to 99.008% when pulling 3b25b706742e174f18c8ed8bd94c6fe002a86ea6 on jogly:gilley/test-compact-to-res0 into e5209fc714293008aeea5d7bd074d2ffd80d679b on uber:master.


Is it compatible with Delphi Rio 10.3 ?

I have compiled it with delphi rio 10.3 but I have some errors
I can't compile android application using TALText or TALMemo
unit ALFmxCommon;
[DCC Error] ALFmxCommon.pas(5951): E2003 Undeclared identifier: 'ConvertPixelToPoint'

You need to include in your search path the delphi patched file located in \Alcinoe\references\embarcadero\rio\10_3_1\patched. I guess you still have in your path the delphi tokyo source file


Add badges for the montly and total downloads

I thought it can give motivation to see the amount off downloads.

Are they currently down?

Are they currently down?
Yes, just after creating the PR 😞

I'm not a fan of this kind of things, especially if they come from providers that tend to go down ;)
However, here's an alternative: https://shields.io

I've used the ones from shield.io in the past.

Then it is the question, do I change them to the shields.io version, keep it this way or close the PR?


the-gauntlet

An all-in-one plugin for The Gauntlet.

Custom resource tracking
Resource image and outline overlays
NPC outline overlays
Starting-room utilities outline overlays
Timer

You code doesn't seem to compile because you've use Set.of, which was only added in Java SDK 1.9. Try using ImmutableSet.of from com.google.common.collect instead, such as seen in ChatMessageManager.java, for example.