Deepnest

Project description
Open-source nesting engine
Metadata

homepage URL: https://deepnest.io/
source URL: https://github.com/Jack000/Deepnest
license: Unlicensed
platforms: unix

FWIW the SW wasn't updated since 2018, but this fork seems to maintain it up until 2019 https://github.com/InfernoEmbedded/Deepnest
CC @Jack000 Can you clarify the maintenance status of this software or up-to-date fork?

{ pkgs ? import <nixpkgs> { } }:
pkgs.appimageTools.wrapType2 {
	name = "deepnest";
	src = /home/kreyren/Downloads/Deepnest-1.0.5-linux/Deepnest-1.0.5-x86_64.AppImage;
}

Results in
[kreyren@leonid:~/.config/nixpkgs/packages/deepnest/result/bin]$ ./deepnest

(deepnest:2): Pango-ERROR **: 00:46:46.509: Harfbuzz version too old (1.2.

@patka-123 Even if it's unmaintained it's still a good idea to package it as it's to my knowledge the only decent option to do nesting on linux for e.g. laser cutting as FreeCAD currently lacks a good ML model to do it[https://forum.freecad.org/viewtopic.php?t=23484] which makes it unusable as it's wasting too much material and it's unable to manage scenario where the parts don't fit on the set zone + the software itself is not complex to update if needs be.

(deepnest:2): Pango-ERROR **: 00:46:46.509: Harfbuzz version too old (1.2. -- @Kreyren (https://github.com/NixOS/nixpkgs/issues/130585#issuecomment-883755206)

It seems that this issue is tracked at https://github.com/Jack000/Deepnest/issues/161, i though it was a NixOS issue :thinking:
It seems that the software is now getting updates on https://github.com/deepnest-io/Deepnest i try to investigate it to see if it can be packaged


💖 Use @layer base to lower precedences

Context
No response
Current
a {
    cursor: pointer;
}

Expected
@layer base {
    a {
        cursor: pointer;
    }
}

https://github.com/master-co/normal.css/releases/tag/v3.0.0


AngularCompilerPlugin hostReplacementPaths option does not work

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions
node: v9.2.1
npm: 6.3.0
ng: 6.1.2
macOS (High Sierra)
Repro steps

ng eject with ng-cli 1.7
update to Angular6.1.2
try to run webpack-devserver

Desired functionality
Mention any other details that might be useful
new AngularCompilerPlugin({
  ...
  hostReplacementPaths: {
    'environments/environment.ts': environmentFiles[NODE_ENV]
  },
  ...
})

does not replace the environment.ts file anymore.
because here it tries to normalize to absolute path and then comparing it with relative one - https://github.com/angular/angular-cli/blob/master/packages/ngtools/webpack/src/virtual_file_system_decorator.ts#L188
Here also question on SO - https://stackoverflow.com/questions/51676035/webpack-angularcompilerplugin-hostreplacementpaths-option-does-not-work

This seems like a bug but we'll need to look at a reproduction to find and fix the problem. Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.

@stevermeister The bug was introduced moving from 6.1.0-rc.1 to 6.1.0-rc.2, probably with this commit: https://github.com/angular/angular-cli/commit/86a62adbe8faeb4628296d5d6915c54e6dbfd85b
I just downgraded to 6.1.0-rc.1 for the time being

Same issue for me.


External Hostname

Version: master
I add via terraform a lambda function with an environment variable to a queue that gets also created via terraform. The queue url is then http://localhost:5000/123456789012/componenttest_rcsws_generate_feed which is not resolvable from inside the lambda function. I guess I need to set it to the internal IP of the host that runs the tests (e.g. http://<IP_ADDRESS>:5000/123456789012/componenttest_rcsws_generate_feed).
Is it possible to change the external displayed hostname?

I'm going to assume this was resolved - let  us know if there are still any issues @benkeil


Ketcher issue

Hi,
ChEBI uses Ketcher for drawing chemical structures. One of our users has recently reported an issue with Ketcher in the way it generates SMILES for certain compounds (see https://github.com/ebi-chebi/ChEBI/issues/4109). Can this be fixed? Thanks

Dear @amalik01,
In order to proceed with your issue could you provide additional information:

steps to reproduce
version of Ketcher you use
provide original mol file from the end user

Regards,
Ketcher team

Ketcher version 1.1-beta
Here is the mol file:
ACD/Labs03252216163D
32 36  0  0  1  0  0  0  0  0  2 V2000
11.6797   -7.9720    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
11.6797   -9.3020    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
10.5278   -7.3070    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
10.5278   -9.9670    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
9.3760   -7.9720    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
9.3760   -9.3020    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
8.1111   -7.5610    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
7.3294   -8.6370    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
8.1110   -9.7130    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
12.8315   -7.3070    0.0000 N   0  3  0  0  0  0  0  0  0  0  0  0
12.8315   -9.9670    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
13.9833   -9.3020    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
14.1224   -7.9793    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
15.1984   -9.8430    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
15.4233   -7.7028    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
16.0884   -8.8545    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
17.4111   -8.9935    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
15.4749  -11.1440    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
16.7398  -11.5550    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
17.7282  -10.6651    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
17.0163  -12.8560    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
18.9931  -11.0762    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
18.2812  -13.2670    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
19.2696  -12.3771    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
20.1449  -10.4113    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
21.1332  -11.3012    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
20.5924  -12.5161    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
16.0279  -13.7459    0.0000 N   0  3  0  0  0  0  0  0  0  0  0  0
14.7630  -13.3349    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
16.3044  -15.0469    0.0000 O   0  5  0  0  0  0  0  0  0  0  0  0
12.8026   -5.9773    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
14.2134   -6.9224    0.0000 O   0  5  0  0  0  0  0  0  0  0  0  0
1  2  1  0  0  0  0
1  3  2  0  0  0  0
2  4  2  0  0  0  0
3  5  1  0  0  0  0
4  6  1  0  0  0  0
5  6  2  0  0  0  0
5  7  1  0  0  0  0
7  8  1  0  0  0  0
6  9  1  0  0  0  0
8  9  1  0  0  0  0
1 10  1  0  0  0  0
2 11  1  0  0  0  0
12 11  1  6  0  0  0
12 13  1  0  0  0  0
12 14  1  0  0  0  0
13 15  1  0  0  0  0
14 16  1  0  0  0  0
15 16  1  0  0  0  0
16 17  2  0  0  0  0
14 18  1  1  0  0  0
18 19  1  0  0  0  0
19 20  1  0  0  0  0
19 21  2  0  0  0  0
20 22  2  0  0  0  0
21 23  1  0  0  0  0
23 24  2  0  0  0  0
22 25  1  0  0  0  0
22 24  1  0  0  0  0
25 26  1  0  0  0  0
24 27  1  0  0  0  0
26 27  1  0  0  0  0
21 28  1  0  0  0  0
28 29  2  0  0  0  0
28 30  1  0  0  0  0
10 31  2  0  0  0  0
10 32  1  0  0  0  0
M  CHG  4  10   1  28   1  30  -1  32  -1
M  END
Try generating the Daylight SMILES  for the above mol file in ketcher. It seems to be missing the Hydrogens.

HI @amalik01 ,
Unfortunately we can't open the Mol file you've provided (even using the Ketcher 1.1. on the ChEBI). We receive the error about Molfile version 0.0.0. Could you, please, try to save and provide mol one more time?
Thank you.

please try this (you may need to leave a blank space at the top):
ACD/Labs03252216163D
32 36  0  0  1  0  0  0  0  0  2 V2000
11.6797   -7.9720    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
11.6797   -9.3020    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
10.5278   -7.3070    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
10.5278   -9.9670    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
9.3760   -7.9720    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
9.3760   -9.3020    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
8.1111   -7.5610    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
7.3294   -8.6370    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
8.1110   -9.7130    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
12.8315   -7.3070    0.0000 N   0  3  0  0  0  0  0  0  0  0  0  0
12.8315   -9.9670    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
13.9833   -9.3020    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
14.1224   -7.9793    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
15.1984   -9.8430    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
15.4233   -7.7028    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
16.0884   -8.8545    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
17.4111   -8.9935    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
15.4749  -11.1440    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
16.7398  -11.5550    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
17.7282  -10.6651    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
17.0163  -12.8560    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
18.9931  -11.0762    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
18.2812  -13.2670    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
19.2696  -12.3771    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
20.1449  -10.4113    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
21.1332  -11.3012    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
20.5924  -12.5161    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
16.0279  -13.7459    0.0000 N   0  3  0  0  0  0  0  0  0  0  0  0
14.7630  -13.3349    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
16.3044  -15.0469    0.0000 O   0  5  0  0  0  0  0  0  0  0  0  0
12.8026   -5.9773    0.0000 O   0  0  0  0  0  0  0  0  0  0  0  0
14.2134   -6.9224    0.0000 O   0  5  0  0  0  0  0  0  0  0  0  0
1  2  1  0  0  0  0
1  3  2  0  0  0  0
2  4  2  0  0  0  0
3  5  1  0  0  0  0
4  6  1  0  0  0  0
5  6  2  0  0  0  0
5  7  1  0  0  0  0
7  8  1  0  0  0  0
6  9  1  0  0  0  0
8  9  1  0  0  0  0
1 10  1  0  0  0  0
2 11  1  0  0  0  0
12 11  1  6  0  0  0
12 13  1  0  0  0  0
12 14  1  0  0  0  0
13 15  1  0  0  0  0
14 16  1  0  0  0  0
15 16  1  0  0  0  0
16 17  2  0  0  0  0
14 18  1  1  0  0  0
18 19  1  0  0  0  0
19 20  1  0  0  0  0
19 21  2  0  0  0  0
20 22  2  0  0  0  0
21 23  1  0  0  0  0
23 24  2  0  0  0  0
22 25  1  0  0  0  0
22 24  1  0  0  0  0
25 26  1  0  0  0  0
24 27  1  0  0  0  0
26 27  1  0  0  0  0
21 28  1  0  0  0  0
28 29  2  0  0  0  0
28 30  1  0  0  0  0
10 31  2  0  0  0  0
10 32  1  0  0  0  0
M  CHG  4  10   1  28   1  30  -1  32  -1
M  END

structure.zip

Hi @amalik01 ,
Thank you for providing the details.
This issue is not reproduced in the latest version of Ketcher, it was fixed quite a time ago. In the 2.4 version of Ketcher the SMILES produced does not add radicals to the structure.
Our suggestion would be to update your version of Ketcher to latest one to have this problem fixed.  The 2.4 demo version is available for you to explore it's abilities here: Ketcher Demo
2.4.2 Release resides here: Ketcher 2.4.2.
You can refer to Discussion for the information on the deployment.
Please, note that Ketcher now has 2 builds provided - Standalone and Remote. The difference would be in the need of having separate Indigo server deployed for some functions to work in Remote version. Standalone version has those functions packed together with Ketcher code. Please, note that Ketcher Remote 2.4.2 will need at least Indigo 1.6 to be used with it, it will not work correctly with lower version.
From user's perspective the only functional difference between Standalone and Remote will be function Recognize (turning picture to the molecule on the canvas): it is not supported in the Standalone version, but is supported in the Remote one. If you don't need it - Standalone build will be sufficient for your needs.
We realize that Ketcher API from 2.4.2 will be dramatically different from the one you are using right now with 1.1.
We are planning to provide API description for the upcoming Ketcher version in April. Also, please, let us know if some specific questions will appear on the API and Ketcher functions.

@Alena-Guznischeva Thanks for the info. Hopefully we will update to the latest ketcher version in the near future.

The issue has been closed as fixed in previous releases.


Tutorials maintenance

Goals:

[x] Migrate references from module READMEs to individual tutorials
[x] Revise any unclear language in tutorials

Status:

[x] Ready to merge

@loriab it's intentional that you and I both have write access (as well as the psi4/psi4 maintainers) to take the load of maintaining this repo solely off Daniel's shoulders.  I'll leave the PR open so that @dgasmith can weigh in, though.


Add support for parsing complex license expressions

Make use of the license-expression library to parse complex license expressions in SPDX documents.
Signed-off-by: Xavier Figueroa<EMAIL_ADDRESS>

@xavierfigueroav ping.... do you thing this could be rebased?

@xavierfigueroav ping.... do you thing this could be rebased?

Hi, PhiIippe :) I've rebased the code.

I am keeping this around as this will eventually be merged.

@pombredanne, is this still valid? Do you know when the "eventually" you managed might be?
As this PR is quite massive, it will cause a lot of conflicts in the future.

I'd like to see a rebase of this PR to get a better sense of what actually changed as the multiple merge commits make this rather incomprehensible.
@xavierfigueroav, are you willing to do this? Else I'll gladly take over from here :)

Looking at the size, age and type of changes in this PR, I think the only viable course of action is to start from scratch based on current main and potentially use similar logic as in this PR (I didn't actually see the implementation details at first glance, since they are drowned in a sea of formatting changes...).
I will wait for a few days to see if anyone disagrees, then close this. The corresponding issue is https://github.com/spdx/tools-python/issues/10

Closing this as announced earlier.


Enforce stricter linting with biome

e.g. https://biomejs.dev/linter/rules/use-naming-convention/
Help @richard-to  out :)

OK I tried it out and there's a lot of violations, some of which are false positives. I'm going to say it's not worth it, and @richard-to, you'll just need to remember 😛
yarn run v1.22.17
$ /Users/will/Documents/GitHub/mesop/node_modules/.bin/biome lint mesop --apply-unsafe
mesop/web/src/component_renderer/type_to_component.ts:66:3 lint/style/useNamingConvention ━━━━━━━━━━

  ✖ This object property name should be in camelCase.

    64 │   'select': SelectComponent,
    65 │   'radio': RadioComponent,
  > 66 │   'content_slide_toggle': SlideToggleComponent,
       │   ^^^^^^^^^^^^^^^^^^^^^^
    67 │   'progress_spinner': ProgressSpinnerComponent,
    68 │   'progress_bar': ProgressBarComponent,

  ℹ The name could be renamed to `contentSlideToggle`.

mesop/web/src/component_renderer/type_to_component.ts:67:3 lint/style/useNamingConvention ━━━━━━━━━━

  ✖ This object property name should be in camelCase.

    65 │   'radio': RadioComponent,
    66 │   'content_slide_toggle': SlideToggleComponent,
  > 67 │   'progress_spinner': ProgressSpinnerComponent,
       │   ^^^^^^^^^^^^^^^^^^
    68 │   'progress_bar': ProgressBarComponent,
    69 │   'icon': IconComponent,

  ℹ The name could be renamed to `progressSpinner`.

mesop/web/src/component_renderer/type_to_component.ts:68:3 lint/style/useNamingConvention ━━━━━━━━━━

  ✖ This object property name should be in camelCase.

    66 │   'content_slide_toggle': SlideToggleComponent,
    67 │   'progress_spinner': ProgressSpinnerComponent,
  > 68 │   'progress_bar': ProgressBarComponent,
       │   ^^^^^^^^^^^^^^
    69 │   'icon': IconComponent,
    70 │   'divider': DividerComponent,

  ℹ The name could be renamed to `progressBar`.

mesop/web/src/component_renderer/type_to_component.ts:77:3 lint/style/useNamingConvention ━━━━━━━━━━

  ✖ This object property name should be in camelCase.

    75 │   // component / API, but the implementation is almost identical as Input.
    76 │   'textarea': InputComponent,
  > 77 │   'content_button': ButtonComponent,
       │   ^^^^^^^^^^^^^^^^
    78 │   'content_checkbox': CheckboxComponent,
    79 │   'text': TextComponent,

  ℹ The name could be renamed to `contentButton`.

mesop/web/src/component_renderer/type_to_component.ts:78:3 lint/style/useNamingConvention ━━━━━━━━━━

  ✖ This object property name should be in camelCase.

    76 │   'textarea': InputComponent,
    77 │   'content_button': ButtonComponent,
  > 78 │   'content_checkbox': CheckboxComponent,
       │   ^^^^^^^^^^^^^^^^^^
    79 │   'text': TextComponent,
    80 │   'markdown': MarkdownComponent,

  ℹ The name could be renamed to `contentCheckbox`.

mesop/web/src/component_renderer/type_to_component.ts lint ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ The file contains diagnostics that needs to be addressed.

mesop/web/src/component_renderer/component_renderer.ts:349:3 lint/style/useNamingConvention ━━━━━━━━━━

  ✖ This class property name should be in camelCase.

    347 │   }
    348 │
  > 349 │   SelectionMode = SelectionMode;
        │   ^^^^^^^^^^^^^
    350 │
    351 │   getComponentName(): string {

  ℹ The name could be renamed to `selectionMode`.

mesop/web/src/component_renderer/component_renderer.ts lint ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ The file contains diagnostics that needs to be addressed.

mesop/web/src/services/editor_service.ts:6:3 lint/style/useNamingConvention ━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ This enum member name should be in PascalCase.

    5 │ export enum SelectionMode {
  > 6 │   DISABLED = 0,
      │   ^^^^^^^^
    7 │   SELECTING = 1,
    8 │   SELECTED = 2,

  ℹ The name could be renamed to `Disabled`.

mesop/web/src/services/editor_service.ts:7:3 lint/style/useNamingConvention ━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ This enum member name should be in PascalCase.

    5 │ export enum SelectionMode {
    6 │   DISABLED = 0,
  > 7 │   SELECTING = 1,
      │   ^^^^^^^^^
    8 │   SELECTED = 2,
    9 │ }

  ℹ The name could be renamed to `Selecting`.

mesop/web/src/services/editor_service.ts:8:3 lint/style/useNamingConvention ━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ This enum member name should be in PascalCase.

     6 │   DISABLED = 0,
     7 │   SELECTING = 1,
   > 8 │   SELECTED = 2,
       │   ^^^^^^^^
     9 │ }
    10 │

  ℹ The name could be renamed to `Selected`.

mesop/web/src/services/editor_service.ts lint ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ The file contains diagnostics that needs to be addressed.

mesop/web/src/dev_tools/dev_tools.ts:30:3 lint/style/useNamingConvention ━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ This class property name should be in camelCase.

    28 │   componentConfigs!: readonly ComponentConfig[];
    29 │
  > 30 │   Panel = Panel; // Make it accessible by template.
       │   ^^^^^
    31 │
    32 │   get selectedPanel(): Panel {

  ℹ The name could be renamed to `panel`.

mesop/web/src/dev_tools/dev_tools.ts lint ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ The file contains diagnostics that needs to be addressed.

mesop/web/src/services/channel.ts:37:3 lint/style/useNamingConvention ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ This enum member name should be in PascalCase.

    36 │ export enum ChannelStatus {
  > 37 │   OPEN = 'OPEN',
       │   ^^^^
    38 │   CLOSED = 'CLOSED',
    39 │ }

  ℹ The name could be renamed to `Open`.

mesop/web/src/services/channel.ts:38:3 lint/style/useNamingConvention ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ This enum member name should be in PascalCase.

    36 │ export enum ChannelStatus {
    37 │   OPEN = 'OPEN',
  > 38 │   CLOSED = 'CLOSED',
       │   ^^^^^^
    39 │ }
    40 │

  ℹ The name could be renamed to `Closed`.

mesop/web/src/services/channel.ts lint ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ The file contains diagnostics that needs to be addressed.

mesop/web/src/utils/diff_state_spec.ts:25:9 lint/style/useNamingConvention ━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ This object property name should be in camelCase.

    23 │         'value': null,
    24 │         'type': "<class 'NoneType'>",
  > 25 │         'old_type': "<class 'bool'>",
       │         ^^^^^^^^^^
    26 │       },
    27 │       {'path': ['val1'], 'action': 'values_changed', 'value': 'VAL1'},

  ℹ The name could be renamed to `oldType`.

mesop/web/src/utils/diff_state_spec.ts:86:9 lint/style/useNamingConvention ━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ This object property name should be in camelCase.

    84 │         'value': true,
    85 │         'type': "<class 'bool'>",
  > 86 │         'old_type': "<class 'str'>",
       │         ^^^^^^^^^^
    87 │       },
    88 │     ]);

  ℹ The name could be renamed to `oldType`.

mesop/web/src/utils/diff_state_spec.ts lint ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ The file contains diagnostics that needs to be addressed.

The number of diagnostics exceeds the number allowed by Biome.
Diagnostics not shown: 11.
Fixed 89 file(s) in 22ms
lint ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Some errors were emitted while running checks.

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

LMAO - That's fine. Thanks for giving it a try.


No module named pyaudio

What is the current behavior?
Missing dependency

Steps to reproduce
Run project in virtual environment, install requirements.txt but missing pyaudio

Expected behavior
pyaudio must be in the requirements.txt file

Please tell us about your environment
 Windows 11
Other information
DEBUG:deepgram.clients.live.v1.client:LiveClient.start LEAVE
ERROR:root:Error: No module named 'pyaudio'
This leads to:
ERROR:root:Error: 'NoneType' object has no attribute 'start'
ERROR:root:Error: 'NoneType' object has no attribute 'start'

Have you installed the dependencies?

Yes, I did create and activate the virtual environment beforehand and then
installed dependencies. It appears pyaudio is not listed in the
requirements file:
[image: image.png]
On Fri, Feb 16, 2024 at 12:48 PM Luke Oliff @.***>
wrote:

Had you enabled the virtual environment before you installed your
dependencies?
—
Reply to this email directly, view it on GitHub
https://github.com/deepgram/deepgram-python-sdk/issues/302#issuecomment-1948158975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A55A6TV6TVQWE5JNQCYUSWTYT42P3AVCNFSM6AAAAABDLZ2GE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBYGE2TQOJXGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
[image: CallCabinet Corporation]
Nic Markram
Data Scientist
@.*** | +1<PHONE_NUMBER> <+1%20%28561%29%20559-2716>
CallCabinet Corporation |
17895 Cadena Drive
33496
Boca Raton
US
FL
--
This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error please notify the system manager.
Please note that any views or opinions presented in this email are solely
those of the author and do not necessarily represent those of the company.
Finally, the recipient should check this email and any attachments for the
presence of viruses. The company accepts no liability for any damage caused
by any virus transmitted by this email.

It's intentional in order to not place the dependency as a hard requirement for install. Pyaudio is only required if you run one of the microphone examples. If you look in the microphone code, it is imported when executed.
The README in the Microphone example should give details on requirements.

Here is the README in case you need it:
https://github.com/deepgram/deepgram-python-sdk/blob/main/examples/streaming/microphone/README.md

Closing as resolved. If you still have any questions, please reopen the issue or drop us a line in discord.


go/text: gotext update fails on type incompatibility issues

What version of Go are you using (go version)?

$ go version
go version go1.11.2 darwin/amd64

Does this issue reproduce with the latest release?
This is the latest release today.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/fgm/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/fgm/src/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/Cellar/go/1.11.2/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.11.2/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/fgm/src/OSInet/net/plusvite/kurz/src/code.osinet.fr/fgm/kurz/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/0g/p780bc554njc4qj110_8rmbr0000gn/T/go-build677218822=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Store https://play.golang.org/p/7y5rpr2MpL6 as a simple go app in main.go
run gotext -srclang=en extract -lang=fr,en .

What did you expect to see?

Nothing on the output
Directories locales/fr and locales/en created with an out.gotext.json file in each, ideally containing message definitions for NoError and Unimplemented strings.

What did you see instead?

On the first invocation:

errors on the output:

/Users/fgm/src/go/src/golang.org/x/text/internal/number/number.go:150:42: cannot convert t (variable of type golang.org/x/text/language.Tag) to golang.org/x/text/internal/language/compact.Tag
/Users/fgm/src/go/src/golang.org/x/text/feature/plural/plural.go:259:42: cannot convert t (variable of type golang.org/x/text/language.Tag) to golang.org/x/text/internal/language/compact.Tag
gotext: extract failed: : : couldn't load packages due to errors: golang.org/x/text/feature/plural, golang.org/x/text/internal/number

a locales/en-US directory created with an out.gotext.json file containing only this:

{
    "language": "en-US",
    "messages": null
}

On subsequent invocations:

nothing on the output
same locale/en-US directory with the same file in it

Note that on a local build of x/text at commit 17bcc049122f272a32787ba38073ee47433023e9 (current master HEAD), the error does not appear, and 3 directories are created: the two expected ones, plus the unexpected en-US one. Messages files are similarly empty.


Directory for instances??????

Where idea came from: https://github.com/fnic-lemmy/lemmy-directory/issues/11#issuecomment-2170654149
I doubt we will be doing it but let's have the discussion on that somewhere far in the future where the directory is finished and we're bored.
Was about to submit the issue and realised that there isn't that many instances actually so I'm not against the idea by that much anymore. We still need to discuss if the directory will be useful though.

Copying this link in here so it's all together:
https://github.com/chrisacheson/instance_list_management
We can easily either fork or submodule that and push it up in a post in the directory.
Might be worth talking to the maintainer, their last update was 7 months ago.

Submodule?

Basically pulls in that repo as a directory in our one.

Wait a moment, @chris-y is this what we needed to avoid hitting API rate limits?

https://shields.io/badges/lemmy
I don't have this thing in my memory at all. lol


messages flushing afer handling 10 messages

After receiving 10 requests and ack correctly i get the following.
2019-06-09T06:36:43.851Z [PRODUCER] info: Message published to exchange PRODUCER.EVENT
2019-06-09T06:36:44.041Z [PRODUCER] warn: Message has been flushed, after 10 times requeued.

Yes I will fix it asap ! Sorry for that

Hi, I've just published the version 1.0.11 to npm, with a fix for this issue, I hope it will work fine for you.
Please feel free to give any ideas to improve the lib ...

Thanks very much :) I definitely will let you know if I think of something.


Video Live Streaming in React Native using Amazon S3 and Clouad Front

Question
Hello everybody. I'm creating a video live streaming application in React.js using Amazon S3 and Amazon Cloudfront service. But I don't know how to use these services in React. Can you help me please?

Hi,
You should check that post which is exactly about that:
http://facebook.github.io/react-native/blog/2018/03/05/AWS-app-sync

@hamedmardanpour  Did you find any ressources regarding this?


WebGL support

Whisper.cpp supports webasm and even has some online demos running into browser.
I didn't research that, but it seems that Unity support native C++ plugins for WebGL builds. So in theory, you can use cmake to compile whisper.cpp into .bc and embed it into web unity build.
Also Unity doesn't support microphone input for WebGL out of the box and you need third-party plugin to record anything.

Did some tests today. Here is rough list of tasks that need to be done:

[ ] Unity doesn't support C# multithreading. Whisper blocks main unity thread and app hangs until inference complete. The good news: Unity 2022.2+ supports C++ multithreading. If I add parallel job creation in whisper.cpp code - this should work.
[ ] Bump Unity project to 2022 LTS
[ ] Inference time is very slow comparing with official whisper.cpp demo. Unity transcribe "jfk.wav" for 15 seconds, while official demo works for 3 seconds. Researching this.
[ ] No microphone support. Need to look for some MIT license compatible microphone library for Unity.

If you want to play around with it, check webgl branch.

Hi Macoron, any updates on webgl support. Did you get it running?

Not much, biggest blocker is that Unity doesn't support C# multithreading. But you can checkout webgl branch. It's a bit outdated, but still working.

Not much, biggest blocker is that Unity doesn't support C# multithreading. But you can checkout webgl branch. It's a bit outdated, but still working.

Based on this source, Async/Await is possible for WebGL with this addon: https://github.com/Cysharp/UniTask

sorry about the previous issue . Now Import is __internal . but I am getting this new error undefined symbol: _ZNSt3__212basic_stringIcNS_11char_traitsIcEENS_9allocatorIcEEE17__assign_externalEPKc (referenced by top-level compiled C/C++ code). using Unity 2021.3.3f1

Got it working  on Unity 2022.3.39f1. Thanks

The Webgl library  does not implement whisper_init_from_file_with_params . It would be of immense help to have the Latest library for webgl.

can i just check if this has been resolved and it can now work with webgl? if so how can i get it to work, sorry i am new to all this

No microphone support. Need to look for some MIT license compatible microphone library for Unity.

Is it possible to add https://github.com/bnco-dev/unity-webgl-microphone in the meantime

There is a framework, which supports async operations in unity: https://github.com/Cysharp/UniTask.git


Make the Objective-C APIs play slightly better with platform conventions

Change property names in structs to be lowerCamelCased.  This requires
three interesting changes:

Deal with property names that collide with NSObject properties by
renaming them.
Deal with property names that start with several all-uppercase
letters by lowercasing all but the last one.
Define getter names for some of the properties, because some start with
'new', which makes the compiler complain.

Also change all the properties to be nonatomic.
Problem
See above.
Change overview
See above.
Testing
Ran existing tests. No behavior changes intended in this PR.

Fast tracking: platform specific changes (darwin)


ARROW-11159: [Developer] Consolidate pull request related jobs

Demo: https://github.com/kou/arrow/pull/10

Demo: https://github.com/kou/arrow/pull/10

This may not address the slow queue... We don't know without try...

This may not address the slow queue... We don't know without try...

My hypothesis for a slow queue is that it is driven by cost: we simply have too many expensive (but unnecessary) jobs running.

Every push triggers a 30m+ job (integration tests). Only about 5m of this job is the actual testing
jobs are not canceled if there is a push after the push that triggered the job, even though those jobs are very likely not relevant once there is a new commit / rebase.
there are jobs from language X running on very push to a user's repo, even when code for X does not change.

These 3 aspects very quickly fill any queue.

My hypothesis for a slow queue is that it is driven by cost: we simply have too many expensive (but unnecessary) jobs running.

Every push triggers a 30m+ job (integration tests). Only about 5m of this job is the actual testing
jobs are not canceled if there is a push after the push that triggered the job, even though those jobs are very likely not relevant once there is a new commit / rebase.
there are jobs from language X running on very push to a user's repo, even when code for X does not change.

These 3 aspects very quickly fill any queue.

It seems that we need to speed up integration test job. (Caching built binaries and reusing not changed binaries?)
https://github.com/apache/arrow/graphs/commit-activity shows that we have about 10 pushes in apache/arrow master per day. If we can fix 1., this may not be a critical.
It seems that we need auto cancel feature that exists on Travis CI and AppVeyor.

It seems that we need to speed up integration test job. (Caching built binaries and reusing not changed binaries?)
https://github.com/apache/arrow/graphs/commit-activity shows that we have about 10 pushes in apache/arrow master per day. If we can fix 1., this may not be a critical.
It seems that we need auto cancel feature that exists on Travis CI and AppVeyor.


fail to apply custom-electron-titlebar on my demo app

i follow the usage ,but found below error :
Uncaught TypeError: Cannot read property 'Menu' of undefined
at Object. (D:\temp\full_Electron_Puppeteer\node_modules\custom-electron-titlebar\lib\titlebar.js:31)
at Object. (D:\temp\full_Electron_Puppeteer\node_modules\custom-electron-titlebar\lib\titlebar.js:451)
at Module._compile (internal/modules/cjs/loader.js:1152)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1173)
at Module.load (internal/modules/cjs/loader.js:992)
at Module._load (internal/modules/cjs/loader.js:885)
at Function.f._load (electron/js2c/asar_bundle.js:5)
at Module.require (internal/modules/cjs/loader.js:1032)
at require (internal/modules/cjs/helpers.js:72)
at Object. (D:\temp\full_Electron_Puppeteer\node_modules\custom-electron-titlebar\lib\index.js:17)
and I review the \node_modules\custom-electron-titlebar\lib\titlebar.js ,
found this code occur error : menu: electron_1.remote.Menu.getApplicationMenu(),

I think what you're experiencing is exactly issue #115 and #101.
When constructing a new BrowserWindow, you need to enable the remote module usage in the webPreferences:
webPreferences: {
    enableRemoteModule: true
}

Sadly the usage documentation is not really up to date, but you can always have a look at the example provided in this repository.

I think what you're experiencing is exactly issue #115 and #101.
When constructing a new BrowserWindow, you need to enable the remote module usage in the webPreferences:
webPreferences: {
    enableRemoteModule: true
}

Sadly the usage documentation is not really up to date, but you can always have a look at the example provided in this repository.


System.MissingMethodException in Django

I got the following error message after opening up a Django project after upgrading to Visual Studio 2015 Upgrade 1:
System.MissingMethodException: Method not found: 'Microsoft.VisualStudio.Text.Editor.ITextView Microsoft.Html.Editor.Document.HtmlEditorDocument.get_PrimaryView()'. at Microsoft.PythonTools.Django.TemplateParsing.TemplateClassifier.GetClassificationSpans(SnapshotSpan span) at Microsoft.VisualStudio.Text.Classification.Implementation.ClassifierTagger.d__5.MoveNext() at Microsoft.VisualStudio.Text.Tagging.Implementation.TagAggregator`1.d__38.MoveNext()

Thanks for this. It appears someone made a breaking change they shouldn't have in an update. I'm investigating now, but I'm not aware of any workaround just yet.

The change is in the HTML editor - they removed that property late during Update 1 and we didn't notice in time.
Currently discussing with that team whether they'll fix their public API or maybe we'll do another PTVS release to fix this.

Same problem here, I can't use Django Template Editing on Visual Studio 2015 Update 1 with PTVS 2.2. When I try to open some html file in a Django Project, multiple exceptions are thrown (same as @aewhatley 's one) and I can't get syntax highlighting and Intellisense in templates. The issue seems to be only linked to templates. Please, let us know when you'll have a workaround! Thanks.

Any progress on this? I am getting the same error message

It's fixed in our latest dev builds, and will be in our next release.


Question: Why does card use so many HashMaps?

Why does the Card Struct use so many HashMaps? For example the legalities.
For the programmer it annoying, because he need to assume that the needed format is present.
Using a format struct like:
struct Format{
    standard: bool,
    modern: boo,
    ...
}

would have the advance, that it will not break with scryfall updates. Since new variants will be simple ignored.

@LuckyTurtleDev can this issue be closed? Or are there more hashmaps you think could be removed?

Not sure.  This should still fail if at least one image field exist, but not all image types. Since the crate does support batch download, it would be a good idea to test deserialize against all cards.

see https://github.com/mendess/scryfall-rs/pull/36#discussion_r1450408558


Localization

This adds localization possibilities.

Added prop localization with all the texts to input translations.
Added prop switchWidth so user can customize toggle switch width to fit his text.
Added prop hourFormat24 to support 24 hour clock format.

Thanks. I'll take a look at this as soon as I can. Hopefully this weekend.

Appreciate the contribution @Drackokacka !

Gladly :)


Using in a C++/CLI project

Hello,
I wanted to use this in my C++/CLI project. I imported CloudFlareUtilities as a dll into my project, but it doesn't give response.
client.GetStringAsync(target).Result;
It's stuck at this line. How can I use it in my project? Thank you.

I believe C++ has the async/await pattern now (according to here). So, to get the result, you'd need to do something like auto x = client.GetStringAsync(target);.
I don't have any prior experience with async in C++, but you could check out an MSDN page on the topic. Note that if you want to make use of the async / await pattern, you'll probably need to compile with Visual Studio as opposed to GCC or an alternative.

Sorry,  I have almost no experience with C++ and absolutely no experience with C++/CLI. Stack Overflow or a similar site is probably a better place to get your question answered.


add keywords

hi could  you support keywords display in beginning of poster

@LYB030 I have added support for Keywords. Check out how it looks - example.pdf. If you think it can be improved in any ways, do let me know.


Change to javascript action

Thanks @mheap for this action. I noticed that this is set to run as a docker action which winds up taking a lot more time to run since each action run requires that the docker container has to be built and then run. Switching to running nodejs is significantly faster (several seconds vs 1+ minutes).

Hey @ejhayes! The development version of the action uses docker, but when we tag a release it's converted to a JS action using https://github.com/mheap/github-action-required-labels/blob/main/.github/workflows/release.yml#L12
If you use mheap/github-action-required-labels@v1 rather than @main you'll see the increased performance


Correct Rethinkdb buggy version in setup.py

The version of RethinkkDB post1 in setup.py makes impossible to run bigchaindb-benchmark load because of error:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/pkg_resources.py", line 444, in _build_master
    ws.require(__requires__)
  File "/usr/lib/python3/dist-packages/pkg_resources.py", line 725, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/usr/lib/python3/dist-packages/pkg_resources.py", line 632, in resolve
    raise VersionConflict(dist,req) # XXX put more info here
pkg_resources.VersionConflict: (rethinkdb 2.2.0-1 (/usr/local/lib/python3.4/dist-packages), Requirem
ent.parse('rethinkdb==2.2.0.post1'))

I changed to 2.2.0-1 as required by the package.

Current coverage is 72.84%

Merging #40 into develop will increase coverage by +72.84% as of 398ae04

@@            develop     #40   diff @@
=======================================
  Files             0      14    +14
  Stmts             0     744   +744
  Branches          0       0
  Methods           0       0
=======================================
+ Hit               0     542   +542
  Partial           0       0
- Missed            0     202   +202

Review entire Coverage Diff as of 398ae04

Powered by Codecov. Updated on successful CI builds.

@Mec-iS apparently this is related to a bug in pip. Which version of pip do you use?

The documentation for packaging states:

It is not considered best practice to use install_requires to pin dependencies to specific versions, or to specify sub-dependencies (i.e. dependencies of your dependencies). This is overly-restrictive, and prevents the user from gaining the benefit of dependency upgrades.

I don't know if that will resolve this issue, but we may consider following the above recommended practice.

@sbellem I agree, this PR wasn't meant to be a permanent fix, but a way to make installation working asap.
@vrde You were right. setuptools is too old also.

see #41


Unclosed client session

Hi!
With the following code:
from adlfs import AzureBlobFileSystem
fs = AzureBlobFileSystem(account_name=..., tenant_id=..., client_id=..., client_secret=...)
fs.ls("")

I am running into the following error:
Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x7fccf5206df0>
Unclosed connector
connections: ['[(<aiohttp.client_proto.ResponseHandler object at 0x7fccf51fee20>, 26399.290502918)]']
connector: <aiohttp.connector.TCPConnector object at 0x7fccf4a4fa90>

Kindly look into this. Thanks!

Is that the full traceback that's printed when running fs.ls("")?

Yes. The output of ls comes out to be the desired.
Although, please correct me if I am wrong, it seems to be much of a warning rather than an error. Ref: here


Show saved Time Spent Entries, for the current date

The default load should show the previously saved time entries.
Save data should get populated after saving new entries(after UPDATE)

This is now covered through the Weekly view card. so closing this.


Pull isinf/isnan/signbit and isfinite from std namespace

... if we're compiling with clang which (as of r258880) provides device-side wrappers for them.
This is needed to provide clang with access to both host and device-side variants of those functions.
Global scope will not have host-side variants available if they were implemented as preprocessor macros.

we have to be careful with defined(__clang__) because on OS X nvcc uses clang as a host compiler, and thus defined(clang) returns always true. Before committing, I suggest to verify that this doesn't introduces errors and new warnings.

Hence (defined(CUDA) && defined(clang))
CUDA is set by clang (very recent one at that) only when we compile CUDA files. It will not be set if you use clang as a host compiler used by nvcc because in that case it will be compiling C++ inputs.

Thanks Artem & Evghenii.
Evghenii, could you test this change on a OS X install of nvcc? If you are satisfied with Artem's change, I will merge it.

Should be safe to merge.


tower cell id

Any starting instructions how to get basestation cellId ?

hi, such feature is not present in the plugin. maybe if you inspect android API you will find it can be added as easily as I added the signal strength. In such case you are welcome with a pull request adding such feature :)


Expressions should also evaluate on statistics

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Physical expressions should be aware of the rules on how to propagate statistics. For instance, for the row-wise max of two columns, the maximum statistics is the greatest of the maximum of the two input input columns. This would allow the ProjectionExec node to provide much better statistics.
Describe the solution you'd like
We should extend the PhysicalExpr trait with a method that handles statistics:
fn stats_eval(&self, stats: &Statitistics) -> Result<Statistics>

Describe alternatives you've considered
This could be implemented in the ProjectionExec with downcasting on various expression types.
Additional context
This is a follow up of #962

That's a great idea 👍

I think PhysicalExpr.html::evaluate_bounds https://docs.rs/datafusion/latest/datafusion/physical_expr/trait.PhysicalExpr.html#method.evaluate_bounds
Implements this partly (though it is in terms of bounds, not statistics) 🤔


Add preference to restart automatically without crash reporter

An update with just the hidden default per the conversation at https://github.com/quicksilver/Quicksilver/pull/1837

Thanks for this @clarkewd I've created a new pull request adding in Rob's suggestion but building on your work.
If you have any more things you want to change, then feel free to submit another pull request


Explain the need for a NetworkPolicy on k8s

Description
Sometimes users need a NetworkPolicy when using flannel etc for the webhook.
Might be worth explaining this in the docs.
Does it fit in a k8s troubleshooting section? Somewhere else?
apiVersion: http://networking.k8s.io/v1
metadata:
  name: allow-same-svc-call
  namespace: abcd-poc
spec:
  podSelector:
    matchLabels:
      http://app.kubernetes.io/name: "kong-mesh"
  ingress:
  - from:
    - ipBlock:
        cidr: x.x.x.x/28
    ports:
    - port: 5443

Worth adding securityCobtext requirements too


fix(rosetta): use of keyword 'lambda' produces invalid Python code

Replace all 'lambda' identifiers with 'awslambda'
fixes https://github.com/aws/aws-cdk/issues/9169

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 29806e67f3fc1ea1928f2fea7e6f938c99508587
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: c83f39a02ae4dfc6340da14dbd6729cd76873a5e
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 526d9d16bc46d697e2a0e9789a9e7bd76de39c4c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


rename MapView#removeAllMarkers to MapView#clear

According to docs public final void clear () "Removes all markers, polylines, polygons, overlays, etc from the map." not just markers.

Changed removeAllMarkers to just remove markers.  Added clear method for removing everything.

@dapriett  can you show me an example hwo to remove all markers from map?

mapView.removeAllMarkers();


[RFC] Admin modules

This PR reimplements my old admin modules PR #890 for recent versions of Wagtail (using the new namespaced URLs and generic views features).
The idea is to build the admin site up using "modules". These provide a url config, menu items, permissions and other things for Wagtail. Modules are all class based and are easy to subclass.
I think there's a few good reasons for modularising the admin interface like this:

Reduces duplicated view code
Makes it easier to add/remove/modify/duplicate sections of the admin
Moves model view permission logic to a higher level (currently each view is responsible for permissions, which is a source of mistakes)
Also allows each component of a module (urls, menu item, dashboard panel, etc) to share permission logic

This PR adds modules into Wagtailadmin and refactors the sites section to use one.
To use admin modules to register a model in the admin, just add the following to a wagtail_hooks.py file:
# app/wagtail_hooks.py
from wagtail.wagtailadmin.modules.model import ModelModule

from .models import MyModel

@hooks.register('register_admin_module')
def register_admin_module():
    return ModelModule('mymodel', model=MyModel, icon='the-icon')

TODO:

Permission checks
Permissions (for the groups interface)
Menu items
Search areas
Dashboard panels

This looks good! I came up with something very similar here https://github.com/gasman/wagtail/commits/feature/generic-model-admins (but didn't pursue it, as the generic-views work was only a spinoff from collections in the first place, and I didn't want to get too derailed :-) ). I particularly like the approach of passing parameters through the as_view method... I struggled to find a good way to pass settings from the module to the CBVs while keeping the CBVs agnostic about the whole module pattern, and ended up with a pretty horrible monkeypatching approach (https://github.com/gasman/wagtail/commit/5f8f74f08872ee9538937e80ce3c303f07035666).
A couple of things I'd pick up on:

As it stands the base 'Module' class seems like needless abstraction to me - either a module follows the CRUD pattern (in which case it should inherit from ModelModule), or it doesn't, in which case it might as well use plain Django views / urlconfs rather than wrapping it in a new kind of object that provides no added functionality. I can see how it might potentially be used in future as somewhere to hang the MenuItem object off (for example), but I'd be inclined to hold off on this until it's clear that there's a definite advantage from doing so.
If we are keeping 'Module', I'd like to look for a better name, since it clashes with the Python concept of a module, and isn't very descriptive of what it really does (namely, provides a collection of related views). ViewSet perhaps?
Having a single 'name' property that serves as the URL namespace and template path probably isn't sufficient - for example, the collections settings views have a URL namespace of wagtailadmin_collections and a template path of wagtailadmin/collections. Ideally we'd have separate url_namespace and template_path properties that default to name if not specified - we can cross that bridge when we come to it, though.

Thanks @gasman!
I like the name ViewSet. Django REST Framework uses that name for pretty much the same concept (http://www.django-rest-framework.org/api-guide/viewsets/).
I'd like it if modules/viewsets weren't tied into models as I think someone might want to add a non-model module (eg a uwsgi dashboard). Also, the page explorer and search promotions probably won't fit in nicely with ModelModule as it makes lots of assumptions about how you want the views to be layed out/permissions to work/etc.

OK, that sounds good. I agree that there will be plenty of admin areas that don't fit the ModelModule pattern. My original thoughts went something along the lines of: "this change is primarily about eliminating boilerplate repetition for things that do follow the standard ModelModule conventions; for the non-standard things, there is no generic boilerplate to eliminate, so there's no advantage over using plain views / urlconfs - it's just adding extra abstraction to solve a non-problem."
However, given the more descriptive name, and seeing how it fits into DRF, I'm now persuaded that the concept of "a bundle of related views that can be treated as a single unit" is a useful building block in its own right, so happy to keep it.

:+1:

Superseeded by #2113


Support obographs in ontology documents

See https://github.com/geneontology/amigo/issues/473

Isn't it a bit...odd...to support two graph formats like this?

NSWYM, we should have one JSON for ontologies.
This ticket seems duplicative of #473, closing...


Assess Performance of The Stack

I have durations in HoneyComb, I should do a full performance trace for requests and a proper analysis if this is a viable tool.

Possibly could use fuse filters to enable a config store setup like what was done here with password hashes: https://www.fastly.com/blog/detecting-compromised-passwords-with-haveibeenpwned-and-fastly-kv-store


Test

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: t-dasong sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Fixes  #343 My Account Page - Save Button - Add Toast

@hameleonka i think the Materialize.toast() invocation should be moved to
https://github.com/Code4SocialGood/C4SG/blob/master/C4SGWeb/app/user/account/user-account.component.ts#L64
not just onclick.

@russon77  Corrected

Corrected warnings


nginx repeatedly deleting cache.flush on certain sites

Hi,
We're running a (slightly old) version of ngx_pagespeed on a server with approx 150 sites.
Nginx configuration :
nginx version: nginx/1.7.9
built by gcc 4.4.7 20120313 (Red Hat 4.4.7-11) (GCC)
TLS SNI support enabled
configure arguments: --prefix=/usr/share --sbin-path=/usr/sbin/nginx --conf-path=/etc/nginx/nginx.conf --error-log-path=/var/log/nginx/error.log --http-log-path=/var/log/nginx/access.log --lock-path=/var/lock/nginx.lock --pid-path=/var/run/nginx.pid --http-client-body-temp-path=/var/lib/nginx/body --http-fastcgi-temp-path=/var/lib/nginx/fastcgi --http-proxy-temp-path=/var/lib/nginx/proxy --http-scgi-temp-path=/var/lib/nginx/scgi --http-uwsgi-temp-path=/var/lib/nginx/uwsgi --user=nginx --group=nginx --with-ipv6 --with-file-aio --with-http_ssl_module --with-http_realip_module --with-http_sub_module --with-http_dav_module --with-http_gzip_static_module --with-http_stub_status_module --add-module=../ngx_pagespeed-release-<IP_ADDRESS>-beta
We pre-populate the PagespeedCache directory with a cache.flush file with FTP user permissions to allow the cache to be flushed manually via FTP, on some sites this file is repeatedly being deleted by Nginx (so presumably by Pagespeed).
There doesn't appear to be any consistency across which sites are affected, other than its recurring on specific sites as far as we can see.  It may be related to the frequency with which this file is touched, but we haven't seen a specific pattern.
Is this deliberate behaviour? If so is there anything we can do to prevent it / why?  We can get it around it fairly easily but it strikes me it may be a bug / undesired behaviour,
Paul.

That's strange.  PageSpeed shouldn't be deleting that file.
For our debugging, did you set EnableCachePurge on?

Sorry for the delay in replying, I meant to post the configs originally, no we don't, we have the following configuration :
pagespeed on;
pagespeed FileCachePath /XXX/ngx_pagespeed_cache;
location /ngx_pagespeed_statistics { allow XXX.XXX.XXX.XXX;deny all; }
location /ngx_pagespeed_message { allow XXX.XXX.XXX.XXX;deny all; }
location /pagespeed_console { allow XXX.XXX.XXX.XXX;deny all; }
location ~ ^/pagespeed_admin { allow XXX.XXX.XXX.XXX;deny all; }
pagespeed StatisticsPath /ngx_pagespeed_statistics;
pagespeed MessagesPath /ngx_pagespeed_message;
pagespeed ConsolePath /pagespeed_console;
pagespeed AdminPath /pagespeed_admin;
location ~ "\.pagespeed\.([a-z]\.)?[a-z]{2}\.[^.]{10}\.[^.]+" { add_header "" ""; }
location ~ "^/ngx_pagespeed_static/" { }
location ~ "^/ngx_pagespeed_beacon$" { }
#Without this pagespeed only works for non-www
pagespeed Domain www.XXX;
pagespeed Domain ipv4.XXX;

#Work for https
pagespeed MapOriginDomain http://XXX https://XXX;
pagespeed MapOriginDomain http://www.XXX https://www.XXX;

We identified that nginx was doing it through audit if this helps :
time->Tue Jul 21 09:49:11 2015
type=PATH msg=audit(1437468551.772:1523243): item=1 name="/XXX/ngx_pagespeed_cache/cache.flush" inode=1452477 dev=08:05 mode=0100644 ouid=10058 ogid=494 rdev=00:00 nametype=DELETE
type=PATH msg=audit(1437468551.772:1523243): item=0 name="/XXX/ngx_pagespeed_cache/" inode=1443702 dev=08:05 mode=040755 ouid=495 ogid=494 rdev=00:00 nametype=PARENT
type=CWD msg=audit(1437468551.772:1523243):  cwd="/root"
type=SYSCALL msg=audit(1437468551.772:1523243): arch=c000003e syscall=87 per=400000 success=yes exit=0 a0=7f5274008cb8 a1=7f5274008cb8 a2=1651af0 a3=7f5274008cb8 items=2 ppid=1 pid=709516 auid=504 uid=495 gid=494 euid=495 suid=495 fsuid=495 egid=494 sgid=494 fsgid=494 tty=(none) ses=148299 comm="nginx" exe="/usr/sbin/nginx" key=(null)
I can't find anything significant that matches that time in the nginx logs...
Paul

It is possible the file is being deleted as part of cache cleaning.
Usually i think it should be OK because it would only be deleted if all the
older files were also deleted.  But it would spoil your hack for setting
it's permissions for ftp access.
A workaround is to set the flush filename outside the directory tree for
the cache.
Josh
On Jul 21, 2015 7:27 AM, "Paul Grant"<EMAIL_ADDRESS>wrote:

Sorry for the delay in replying, I meant to post the configs originally,
no we don't, we have the following configuration :
pagespeed on;
pagespeed FileCachePath /XXX/ngx_pagespeed_cache;
location /ngx_pagespeed_statistics { allow XXX.XXX.XXX.XXX;deny all; }
location /ngx_pagespeed_message { allow XXX.XXX.XXX.XXX;deny all; }
location /pagespeed_console { allow XXX.XXX.XXX.XXX;deny all; }
location ~ ^/pagespeed_admin { allow XXX.XXX.XXX.XXX;deny all; }
pagespeed StatisticsPath /ngx_pagespeed_statistics;
pagespeed MessagesPath /ngx_pagespeed_message;
pagespeed ConsolePath /pagespeed_console;
pagespeed AdminPath /pagespeed_admin;
location ~ ".pagespeed.([a-z].)?[a-z]{2}.[^.]{10}.[^.]+" { add_header "" ""; }
location ~ "^/ngx_pagespeed_static/" { }
location ~ "^/ngx_pagespeed_beacon$" { }
#Without this pagespeed only works for non-www
pagespeed Domain www.XXX;
pagespeed Domain ipv4.XXX;
#Work for https
pagespeed MapOriginDomain http://XXX https://XXX;
pagespeed MapOriginDomain http://www.XXX https://www.XXX;
We identified that nginx was doing it through audit if this helps :
time->Tue Jul 21 09:49:11 2015
type=PATH msg=audit(1437468551.772:1523243): item=1
name="/XXX/ngx_pagespeed_cache/cache.flush" inode=1452477 dev=08:05
mode=0100644 ouid=10058 ogid=494 rdev=00:00 nametype=DELETE
type=PATH msg=audit(1437468551.772:1523243): item=0
name="/XXX/ngx_pagespeed_cache/" inode=1443702 dev=08:05 mode=040755
ouid=495 ogid=494 rdev=00:00 nametype=PARENT
type=CWD msg=audit(1437468551.772:1523243): cwd="/root"
type=SYSCALL msg=audit(1437468551.772:1523243): arch=c000003e syscall=87
per=400000 success=yes exit=0 a0=7f5274008cb8 a1=7f5274008cb8 a2=1651af0
a3=7f5274008cb8 items=2 ppid=1 pid=709516 auid=504 uid=495 gid=494 euid=495
suid=495 fsuid=495 egid=494 sgid=494 fsgid=494 tty=(none) ses=148299
comm="nginx" exe="/usr/sbin/nginx" key=(null)
I can't find anything significant that matches that time in the nginx
logs...
Paul
—
Reply to this email directly or view it on GitHub
https://github.com/pagespeed/ngx_pagespeed/issues/988#issuecomment-123272121
.

Cheers, I wasn't aware of that being a possibility (I did look!) and after some digging have found it, should anyone else end up here :
https://developers.google.com/speed/pagespeed/module/system
"You can change the polling interval and name of the cache-flush file in the configuration file. If you set the polling interval to 0, the cache flushing feature will be disabled. If you specify the cache flush filename as a relative path, PageSpeed will look for that file in the FileCachePath directory. If you use an absolute path, then the caches associated with multiple virtual hosts can be flushed all at once. "
pagespeed CacheFlushFilename alternate_filename;
Paul.

Reopen if using an absolute path doesn't work out?


Suggestion of syntax to terrascript: "ts.write"

I started work on documentation to upgrade to version 0.5.0, and I'm think that declare ts += a bit tedious, specially when we will use the variable another time. In this case is necessary two lines.
ts = Terrascript()

var_access_key = variable('access_key')
ts += var_access_key
var_secret_key = variable('secret_key')
ts += var_secret_key
var_region = variable('region', default='us-east-1')
ts += var_region

Than, my suggestion is another syntax to Terrascript. Something explicit and more easy to use.
I think that is possible take advantage of multiple assign of python.
We would use a setter property, for example ts.write or ts.w(less verbose).
I'm not advocating to remove the current syntax, but add another.
ts = Terrascript()

ts.write = var_access_key = variable('access_key')
ts.write = var_secret_key = variable('secret_key')
ts.write = var_region = variable('region', default='us-east-1')

To apply this syntax, Is need just add some lines:
class Terrascript(object):
    """Top-level container for Terraform configurations."""

    ....

    def get_write(self):
        return None

    def set_write(self, value):
        self += value

    write = property(get_write, set_write)

Full example for comparision
Syntax: ts +=
from terrascript import Terrascript
from terrascript import provider
from terrascript import output
from terrascript import variable
from terrascript.aws.r import aws_instance

ts = Terrascript()

var_access_key = variable('access_key')
ts += var_access_key
ts.write = var_secret_key = variable('secret_key')
ts += var_secret_key
ts.write = var_region = variable('region', default='us-east-1')
ts += var_region

ts += provider(
    'aws', access_key=var_access_key,
    secret_key=var_secret_key,
    region=var_region
)

example = aws_instance('example', ami='ami-2757f631', instance_type='t2.micro')
ts += example

ts += output('example_public_ip', value=example.public_ip, description='Public IP of example')

print(ts.dump())

Syntax: ts.write or ts.w
from terrascript import Terrascript
from terrascript import provider
from terrascript import output
from terrascript import variable
from terrascript.aws.r import aws_instance

ts = Terrascript()

ts.write = var_access_key = variable('access_key')
ts.write = var_secret_key = variable('secret_key')
ts.write = var_region = variable('region', default='us-east-1')

ts.write = aws_p = provider(
    'aws', access_key=var_access_key,
    secret_key=var_secret_key,
    region=var_region
)

ts.write = example = aws_instance('example', ami='ami-2757f631', instance_type='t2.micro')

ts.write = output('example_public_ip', value=example.public_ip, description='Public IP of example')

print(ts.dump())

I only added the += syntax in version 0.5.0 which was a major change. This is of course implemented as the Terrascript.__add__() method. There is actually a Terrascript.add() method which is simply an "alias" for   Terrascript.__add__(). Since it is not documented I suggest changing the behaviour of Terrascript.add() so it can be used as follows.
from terrascript import Terrascript
import terrascript
ts = terrascript.Terrascript()
var_access_key = ts.add(terrascript.variable('access_key'))

The Terrascript.add() method would simply return its argument.

I have a test case for what I suggested in my previous post: https://github.com/mjuenema/python-terrascript/blob/issue33/tests/test_issue33.py

Great!! I will use the syntax ts.add in the docs in this case.

As this syntax was implemented on branch issue33, I'm closing this issue.


Request: Start an installed app like Amazon Video or Kodi

I'm wondering if it would be possible to start an App from within HA.
It would be cool to start for example the VLC media player on the TV showing the RTSP stream from the Babycam automatically.
Or if the preferred steaming App would be started whenever the living room enters the "Couch potato mode".
Maybe "setActiveApp" could help here:
https://pro-bravia.sony.net/develop/integrate/rest-api/spec/service/appcontrol/v1_0/setActiveApp/

See my comment at #3 regarding the Android Debug Bridge addon. With that addon this is all possible.

See my comment at #3 regarding the Android Debug Bridge addon. With that addon this is all possible.


@eessex => Remove unnecessary date casting

Thanks to your PR, we can now remove the Date casting on this moment call.

woohoo! merge when ready


v1.2.3

Addresses https://github.com/ZencashOfficial/arizen/issues/407 by introducing a label change.
Unable to replicate the report of not seeing all transactions up to the limit.

Travis osx job 945.1 finished, you can download the results here.

Travis linux job 945.2 finished, you can download the results here.


[Identity] update mapping for browser for azd

Packages impacted by this PR
@azure/identity
Issues associated with this PR
Bug fixes
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.

@tyteen4a03 Yes, I will release the official fix for it this week in 3.3.2


Develop an OBJ-viewer, non-blocking requests' handling

Every request should be handled using Kotlin coroutines;

Done.

Done.

Done.


Add search feature

Allow searches of users and organizations by name

Must the search be restricted by location too? or I can search by name users located in any location?

@psantos10 Restricted by location which is Angola and any other provinces.

Assigne this issue to me. Working on it


Added Git Hooks

I've added a pre-commit hook and a commit-msg hook placed inside the hooks folder #9 .
Pre-commit Hook:
It consists of 2 function.
The first function analyses the code and provides feedback on potential errors, coding style violations, and other issues.
The second function runs tests from the Django test suite. Both of these functions will not allow the person to proceed further without resolving the issues.
Commit-msg hook:
It ensures that the commit message length is not more than 50 characters and it does not end with a period.

I've added more linters alongside pydocstyle but the output generated was a lot and therefore I stored them in a linters.log file located inside the logs folder in the project root and It will only contain logs from last time a person tries to commit.

since this is a pre-commit hook we need to ensure it doesn't check files such as README,etc

All the linters that I've used perform checks on files ending with .py unless specified, However I excluded all the files in migrations folder since they are automatically generated.

There are too many files as part of the recent PR, let's review them and get it in-place before merging it.
cc: @neelansh27

I've made the requested changes, please review them.

Thanks for incorporating the changes, @neelansh27
I'll work with you to test it on the local environment first before merging the above changes.

At a high level, this would be our next high level action item.
-- identify all the files on the user's local branch that have been modified w/ respect to the branch off point from master. This should identify the following types of files, and implement linting correctly to them (e.g., should not lint deleted or untracked files): staged files, moved/renamed files, committed files, deleted files, untracked files
-- Files that have not been modified by the user should not raise linters errors
cc: @neelansh27

I've added the setup file. If someone runs into permission issues, it can be fixed by
chmod +x <file_name>
THE SETUP FILE MUST BE RUN WITHIN THE _hooks FOLDER
The pre-commit hook now prompts user to install dependencies if they are not met.


Use buf.subarray instead of buf.slice

Refs: https://github.com/tshemsedinov/Drop-Nodejs16

[x] tests and linter show no problems (npm t)
[ ] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[ ] description of changes is added in CHANGELOG.md
[ ] update .d.ts typings

Landed in https://github.com/metarhia/metautil/commit/810f494e2332bcf697577254a4268c88d4b522f8


how many colors can detect?

Hi ,thanks for your work, it helps me a lot. The question is how many colors can detect, can you give me a list ?

Hello there @xuweidongkobe . Following the documentation Example, ColorDetect takes a default of 5 most dominant colors. If you want more, just parse the argument to the relevant method. The limit of this is up to you.


Follow should work with Posting key.

eSteem Surfer v1.0.2
steps to reproduce:
set up account in Surfer with just Posting key, try to follow someone - Surfer prompts for stronger key, even though the same action on steemit.com works just fine

It's true. Please fix.


Display credits for tossups and bonuses

Showing the author of the tossup, if available, would be beneficial for three reasons:

players may grow to like an author, and may want to see more of their content
a writer might want to see how another, more distinguished writer words certain
crediting the creators is generally a cool thing to do

not sure if this is really feasible


R.unionWith documentation doesn't say which one of equal elements will be included

const l1 = [{a: 1, b: 'b'}];
const l2 = [{a: 1, c: 'c'}];
R.unionWith(R.eqBy(R.prop('a')), l1, l2);

It's not clear from the documentation whether this should return [{a: 1, b: 'b'}] or [{a: 1, c: 'c'}].

Since Ramda focuses on value equality, we don't make the call.  I'm not sure if we still do, but at one point, at least one of our Set-like array operations varied in that behavior based upon which of the two supplied lists was shorter (a performance optimization that was perhaps not the best idea.)
I'm not sure Ramda would want to commit to choosing a or b here.  My feeling is that if you're saying these two are equal, then you shouldn't care which one is included, or even if it's a different object altogether that also tests equal (not that we would do the latter, of course, but it shouldn't matter.)

Ok, that applies to R.union, but for R.unionWith the results are different values.
My use case is I want to combine two arrays but prefer values from one of the arrays if there are duplicates as determined by the predicate function. I just want to know if I can rely on the order of the array arguments, or if the behavior is explicitly unspecified and I should look for another solution.

It seems to me that the same thing applies to unionWith.  But if you want to make a PR to clarify the documentation, pointing to the current behavior (taking the first-found value), it will definitely get a fair hearing.
If you do, I would love to see the example changed to use a lambda for the predicate rather than R.eqBy(R.prop('a'), something like (x, y) => x.a == y.a.  I think the function examples should be as stand-alone as we can reasonably make them.

For uniqWith we already have the following in its documentation: "Prefers the first item if two items compare equal based on the predicate". So a similar sentence could be added for unionWith.
I might do a PR at some point if I get time, but if somebody wants to pick this up before that, feel free.

Two comments to unionWith function:

The order of the params for the predicate function looks wrong to me. First parameter corresponds with the second object. and second parameter corresponds with the first object. This makes the first example to fail and the second to work.

R.unionWith((l, r) => l.date.substr(0,10) === r.date, [{date:'2020-01-02T00:00:00'}],  [{date:'2020-01-02'}])
//?  [{date:'2020-01-02T00:00:00'}, {date:'2020-01-02'}]

And this works:
R.unionWith((l, r) => l.date === r.date.substr(0,10), [{date:'2020-01-02T00:00:00'}],  [{date:'2020-01-02'}])
//? [{date:'2020-01-02T00:00:00'}]

I am not sure about this one... but shouldn't it better to take the second item when there is a match? Last parameter usually has more preference considering (data last parameter rule). If we were talking about a function called update, modify or something similar which reinforces the idea of first params modifying the master... I would understand the case... but union doesn't give strength to one side... so being even by the operation, then last parameter is the master by default.... it is like the main stream of data (last parameter) only accepting new data and rejecting duplicated data.

@ josuamanuel:

The order of the params for the predicate function looks wrong to me.

But the point of this function is that the user is saying that the two values are equal.  Equality is transitive.  If there are any a and b values such that
fn (a, b) ≠ fn (b, a)

then fn is not a suitable predicate to pass to functions such as unionWith.  Any function that treats a and b differently is suspect.

[S]houldn't it better to take the second item when there is a match?

While there's an interesting argument there about API consistency, the user should never care.  If we're doing a union of two distinct types that are close enough, the user shouldn't care which of the values is returned.
Similarly to the above, things should be symmetric.  If fn is our equality function then the user should not ever need to care about the difference between unionWith (fn) (xs, ys) and unionWith (fn) (ys, xs).  If a difference between them could cause problems further in the codebase, then unionWith is not suitable for that usage.
There was at some point an interesting performance enhancement which removed some consistency in how the parameters were ordered.  We could satisfy such requests by undoing that, but it seems to me that there is no gain, only loss.  If the order of the parameters to the predicate or the order of the last two parameters to union* matters to a codebase, I would suggest that the user find a different tool than union*.

@CrossEye It looks really restricted in both sides fn (a, b) must equal fn (b, a) and  unionWith(fn) (xs, ys) must equal (fn) (ys, xs)
Documentation is a little bit short in these aspects.
I understand now... I ended up using Map that probably is a better tool for my purpose and with much much better performance.
Thanks for the great explanation!!!

@josuamanuel:

Documentation is a little bit short in these aspects.

We'd love to hear suggestions, either here or as a pull request!

I understand now... I ended up using Map that probably is a better tool for my purpose and with much much better performance.

I would always choose Maps/Sets for primitive types.  Ramda allows for value equality as well, and it useful for those cases where Map/Set won't do.  (For instance, new Set ([{x: 1}, {x: 1}]) has size of 2, where the Ramda tools would see only one.)  But it is much less performant for the primitive cases, as it has to call Ramda's equals function for many different pairs.


ABOUT could have unique build number...

Version
alpha6
What is the issue?
With the sometimes-rapid-iteration, it would be useful to see a completely unique build number in the About box.
(In my projects, every build runs a script that increments version.h, so the about box or --help shows things like "build 4037".)
Can also include __DATE__ and __TIME__ to get a very unique number.
Ideally the build number is also on the file name, like FIRMWARE_5.6_4037.333 or whatever. (Right now I have two quick iterations in my downloads which are distinguished by an extra period in the filename :) )

Which builds are those? I usually use numbers for significant changes…. And letters for minor fixes. I am pretty lazy about ensuring the about page gets upgraded consistently, mostly because the quick iterations, but typically ensure the file names are reflective.

@Frudah I think @distrakt is referring to an automatically generated number (by a script in the build process) that will be incrementing with every build. I have similar mechanisms in Platformio to maintain this number and generate a build version string from it. Once implemented you will not have to care about it any more.


[Feature request]: About postponement

Version

[X] I'm using version 1.7.0

Existing requests

[X] I've checked Existing requests

What operating system are you using?
macOS
Operating System Version
Catalina 10.15.7
Problem description
When I am working, stretchly may sometimes pop out in an awkward timing for example I want to finish this part before my thoughts went away. The postponement can help but sometimes I want more postponement time like 5 minutes or even 10 minutes. Could you add a feature that I can customize the postponement time for each break?
Possible solution
Add a feature that I can customize the postponement time for each break, better if I can set the postponement time with a simple keyboard binding with pre-set time period (5mins, 10mins).
Alternative solutions
Not really
Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi there,
there is more complex system of postponments, read more here https://github.com/hovancik/stretchly#editing-postpone-functionality- about how to set it up :)
It's not exactly what you ask for, but I hope it will be sufficient for you. For, I would prefer not to complicate it further with different postpone times and shortcuts.


How to pass THEME_KEY parameter to CLI dev script

I could not work out the correct syntax for adding the THEME_KEY parameter to npm dev or npx astro dev scripts (as suggested in the features list). I tried a few conventional ways including:
npm run dev -- THEME_KEY=indigone
npm run dev -- --THEME_KEY=indigone
I was able to make the change by direct edit of the default section of tailwind.theme.config.cjs but how does it work through CLI?

Hey @camedgell the THEME_KEY needs to go before the command you wanna run. You can pass THEME_KEY with the dev or build commands like THEME_KEY=purpleheart astro build/dev to change the color theme being used.

Thanks but the suggested syntax does not work at my end (running on Windows 11). Error code is EINVALIDTAGNAME with error message Invalid tag name "THEME_KEY=purpleheart" of package "THEME_KEY=purpleheart": Tags may not have any characters that encodeURIComponent encodes.

You should be able to pass it direct in the package.json
eg.
  "scripts": {
    "dev": "THEME_KEY=indigone astro dev",
    "build": "astro build",
    "check": "astro check",
    "postbuild": "node ./scripts/search/prepare-index.js",
    "preview": "astro preview",
    "check:type": "tsc --project tsconfig.json --pretty --noEmit",
    "lint": "biome lint src",
    "format": "biome format src --write",
    "cz": "cz"
  },

Thanks @lovettbarron but that does not work either. However, I have found that I can change the theme by direct edit of the default section of tailwind.theme.config.cjs, so will proceed with that.


Add more examples for post requests

Could use some more examples of specifying the parameter for post requests.
For example, all the controllers given here are for get requests only.

It's very simple
// Create a device
// @Tags User
// @Summary Create Device
// @Description Create Device
// @Accept json
// @Produce json
// @Param message body request.Device true "Device Info"
// @Success 200 {object} request.Device
// @Failure 400 {object} response.Error
// @Failure 503 {object} response.Error
// @Router /device [post]
func (h *device) Create(w http.ResponseWriter, r *http.Request) {
}

Thank you, perhaps adding it the examples might help future folks looking for the same thing? I've opened a PR #832 .

I believe more examples should be written into a separate repository ...

Closing this since #832 is now merged.


pause_on_unhandled_async_exceptions2_test/service_1 went from flaky to Timeout on app-kernel-linux-release-x64

https://dart-ci.appspot.com/log/app-kernel-linux-release-x64/app_jitk-linux-release-x64/11359/service_2/pause_on_unhandled_async_exceptions2_test/service_1

https://dart-ci.appspot.com/log/app-kernel-linux-release-x64/app_jitk-linux-release-x64/11530/service_2/pause_on_unhandled_async_exceptions2_test/service_1

Happened on vm-kernel-linux-debug-ia32 https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/vm-kernel-linux-debug-ia32/7612/overview


after auto back when take picture

when i was take picture ,camera auto goback html ,i want   like android camera can continuity take,help

Can you provide more information and/or a sample app where the issue can be reproduced?

I believe I'm seeing the same issue. When I successfully take a picture or select from saved pictures, the success callback fires, then the application behaves as if window.history.back() was executed. (I don't believe it's actually calling window.history.back(), but the effect is the same, transitioning the view to the previous view on the stack.)

Hi, I have this problem.
You can see on this video : https://streamable.com/xz6xm
Sometimes camera auto go back when i trigger open camera.

closing since the original reporter didn't answer and the problem is unclear.
please, when reporting issues provide all the information that the template request instead of deleting it


Editor: Cannot Load an Image on New ImageTexture of a Sprite.

Godot version:
3.1 alpha #bde3e88
Mac OS X, High Sierra
Issue description:
Cannot Load an Image on New ImageTexture of a Sprite.

In 2D you don't need to import textures, you can use images directly by clicking the Texture property, Load and select your png. Did you try that?

I think the issue is that ImageTexture is the only one in the list with no UI... I guess it must be interacted with only by code? Why is there no Load Path or something like some other options in the list?
But I just found out this is intentional. :)
https://docs.godotengine.org/en/stable/classes/class_imagetexture.html
"An ImageTexture is not meant to be operated from within the editor interface directly, and is mostly useful for rendering images on screen dynamically via code. If you need to generate images procedurally from within the editor, consider saving and importing images as custom texture resources implementing a new EditorImportPlugin."
Maybe could add a message saying "this space intentionally left blank" or "use from code" or something. Idk...


Updating the website

In overall, our site is not up to date. Before considering a global redesign of the site, it would be nice to have the following features :

[ ] Updates the presentation of the project

[ ] Adds a section listing articles where the project is mentioned, events in which Pyronear has participated
non-exhaustive list (discuss on communication channel to be up to date):

Poster lors de la Participation au PyTorch Ecosystem Day
Présentation de la solution au dataXdays, Organisé par les équipes de Publicis Sapient
Enregistrement d’un podcast avec horizonIA
Article pour le blog de Qarnot
Interventionau MonthlyMarty de DataForGood
Intervention au Ministère de la Transition Écologique, lors de la 1ère journée de la donnée au service de la transition écologique
Others: le parisien  / AEFinfo
podcastdatagen
hebdo Ardèche
lauréat Epe
article TF1 epe

[ ] Introduces section related "nos partenaires" (SDIS 07, etc)

[ ] Updates consequently section "ils nous soutiennent " (D4G, Atraksis, Fondation M33)

[ ] Adds link for donation

[ ] Presentation of devices / Photos

[ ] Team photo vs member photos ?

Happy to discuss it ! 😃

I agree with most of what you said :)
Only part I have doubts is "team photo vs. member photos":

going for member photos can end up in a "badge" for some people and not others. So I'd argue against
now for team, the same applies but especially it "exposes" people (for people looking for customer support let's say). Additionally, I do think that the project should feel like going beyond the core contributors (e.g. also the people opening a PR only once on GitHub, etc.).

What do you think?

Sorry about the late reply, I agree, let's do a general team picture if people on it are alright with that :smiley:
If we agree on the other modifications, I can start opening PRs in the coming days :+1:


UI does not allow edition on Response Policy Zones

Hi there,
While testing  to use Response Policy zones management (via APIs), which is perfectly working for me.
I discovered that UI does not allow to create a CNAME to '.'  only .
If you are unfamiliar with Reponse Policy zones , please have a look here
And please find some zone examples of policy zone  here
If you try to add a CNAME manually that has a '.' for destination, you will end up with an error (if I remember well).
I think it could be fixed easily by adding a simple test on utils.py   : like
test if value is '.' .. and return value  else .. perfom the rest of the function...
If you think there is a better way to handle this, please feel free to suggest.
Best regards,

@WhatshallIbreaktoday you're welcome to submit a PR but I don't know exactly when I'll get to reviewing it. January is a busy month for me so I will possibly lag behind a bit.
Thank you for participating!


[test] Update llama2-model.libsonnet

Description
Fix pt-nightly-llama2-train-spmd-func-v4-8-1vm's issue Check failed: dim1 == dim2 || dim1 == 1 || dim2 == 1 || dim1 == xla::Shape::kUnboundedSize || dim2 == xla::Shape::kUnboundedSize according to @vanbasten23 suggestion
Tests
Please describe the tests that you ran on Cloud VM to verify changes.
Instruction and/or command lines to reproduce your tests: ...
List links for your tests (use go/shortn-gen for any internal link): ...
Checklist
Before submitting this PR, please make sure (put X in square brackets):

[x] I have performed a self-review of my code.
[ ] I have run one-shot tests and provided workload links above if applicable.

since we modify config file directly, let's close this PR for now until we need it again


Add command page, including DestroyWildDinos and SaveWorld

This PR adds a command page, two commands have been added as of now:

DestroyWildDinos
SaveWorld

There is the possibility for expansion in the future. Though, additional commands may require an input box for args.
Closes #86

Looks great, only I think this list of commands should be under the console, and for #86  I don't think this is implemented here, that issue wants a dedicated wipe button, so you can wipe the server with one button and without manually needing to delete files.

Ah I understand. I'll move them below the console and see about implementing the other feature in a different PR.

Small change included, changed "clear input" to "clear console" to more accurately represent what the button does.


Review and Validate the Completed Resume

Ensure that the final resume is thoroughly reviewed for readability, usability, and code cleanliness. Verify that all components have been successfully uploaded to GitHub and are displaying correctly.

Please double-check all hyperlinks, formatting, and alignment to confirm that the resume presents well on various devices and browsers.


Fix details/summary visual glitch in Firefox

This removes the details/summary arrow in Firefox as it's overlapping the custom '+' one.
Compare

with

Thanks!

No problemo :)


Set the default pathToDestination for cli

I'm wondering if this would make it less likely for people to realise that a path can be provided to this tool 🤔
If you give it a default, someone might never try to do it with a custom path. I'm not sure there should be a default for this.

I'm wondering if this would make it less likely for people to realise that a path can be provided to this tool 🤔
If you give it a default, someone might never try to do it with a custom path. I'm not sure there should be a default for this.

I agree with you, but I was too lazy to use write that much. And I don't care where the output is, I just want the file and decide later. Dunno. You can drop this

Dropping this for now, maybe we can reconsider the default API of this some time


[RFC]: add support for exporting API docs to a PDF

Description
This RFC proposes adding support for exporting API docs to a PDF.
PDF documentation could be useful outside of the website and in the context of offline usage. PDF docs are common in, e.g., R and TeX documentation.
Generation of the PDF could be done either on-demand (client side; see htm2pdf.js) or pre-compiled on the server and simply sent on request.
HTML to PDF can be problematic, as a common technique is to capture HTML as an image, which prevents the generated PDF text from being searchable or selectable.
This RFC would benefit from gh-27, where JSON could be used to automatically generate TeX which could then be compiled to a PDF.
Related Issues
Related issues #27.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

Alternatively, could use https://github.com/bpampuch/pdfmake in combination with html-to-pdfmake, which I have been using with some success.

Yeah, my sense is that anything HTML-to-PDF will be tricky/imperfect given our heavy reliance on CSS, and that a more robust conversion pipeline over the long term would be JSON-to-TeX.

While not getting us near the ability to export the entire docs as a PDF document, I tweaked our CSS a bit per https://github.com/stdlib-js/www/commit/8eec861b0fce7171b814d9132474c3838679879c to allow one to nicely print a single README.md without any interface element.

Here is an example:
array_shape_readme.pdf
We could consider having additional elements that are only visible when printing, e.g. some attribution / date of export. What do you think?

Also a added a button to print the currently selected README.md per  a9495a12bc6b96f12a1a5f9536cebe695b20aba2.
Dark theme:

Light theme:

Yeah, adding a copyright notice might be one thing to add, along with the date of export.

And maybe the version of the docs for which the README.md was exported...

Yeah, that makes sense, as well.


Mention OAuth related to refresh token to educate the reader

In "Add Azure AD authentication"
"However, this token is short-lived. The token expires an hour after it is issued. This is where the refresh token becomes useful. The refresh token allows the app to request a new access token without requiring the user to sign in again."
Suggest we add a mention of OAuth for those who may not be familiar with refresh token
"However, this token is short-lived. The token expires an hour after it is issued. The OAuth specification introduces a refresh token to help here. The refresh token allows the app to request a new access token without requiring the user to sign in again"

what is the significance of refresh_token expire time 90 days, even every hour have to make a new request to get access_token ? is code available?

This is all part of the OAuth spec, and you can read a lot more authoritative sources on this than what I can provide. However I'll give my two cents:

The access token grants access to the resource, and is short-lived. This is to take care of situations where the user revokes access, or if somehow the token is leaked to a bad actor - the access will not be forever.
The refresh token doesn't give you access to anything, but can be used by the app to get a fresh access token. The refresh token by itself cannot be used - you also need the app's ID and secret.

The code in this sample does all of this for you using the simple-oauth2 library. I'm in process of removing that library and using the msal-node package instead, which again, does all of this token management for you.

Fixed by #52


[release/v2.6] Bump rancher/system-agent to v0.0.1-alpha36

https://github.com/rancher/rancher/issues/34036

https://github.com/rancher/rancher/issues/34027


[bug] vela top cannot switch theme

Describe the bug

In vela top mode, press ctrl+T can not switch theme
To Reproduce

run vela top
press ctrl+T, choose any theme
press <ESC> to exit
rerun vela top, nothing changed

Expected behavior

UI change
Screenshots

KubeVela Version

CLI Version: 1.7.4
Core Version:
GitRevision: f3cdbcf203ba68bbbfec491e1e692ce808ba873f
GolangVersion: go1.19.6

Cluster information

Additional context

Root Cause
theme dir ~/.vela/theme doesn't have executable permission, so theme config cannot be written.

after running chmod +x theme, works fine to me.


chore: maxHeight for panels

Should address #1664, although I think it's already addressed? I am not sure where this issue is actually happening in elements.
I did notice that the request sample panel was unnecessarily long and put a maxHeight on it in the meantime.

request sample fix still should be merged, no?

@paulatulis FYI I've changed both this one and #1710 from "chore" to "fix" because the distinction is not how small the PR is, but whether they have an impact on customers and thus need to be released. Chores might include, fixing typos in the README or changing the CI pipeline to be faster or something. Both of these changes make a difference to how the application works from a customer perspective!

@paulatulis FYI I've changed both this one and #1710 from "chore" to "fix" because the distinction is not how small the PR is, but whether they have an impact on customers and thus need to be released. Chores might include, fixing typos in the README or changing the CI pipeline to be faster or something. Both of these changes make a difference to how the application works from a customer perspective!

Thank you! I get into the habit of using chore and forget to update. Will try to make it a point to be more mindful going forward.


Feature: allow to exclude some properties from knobs

In certain components we have public properties that don't make any sense in knobs:

autofocus (only processed once and if set to true by default)
tabIndex (not a component-specific API that we don't want to highlight)

Proposed syntax:
<api-viewer exclude-knobs="autofocus,tabIndex">

Done by #23 and will be included to the upcoming 0.4.0 release.


Webpack 5 compatibility

When upgading to Webpack 5, offline-plugin is not compatible.
So we forked to work on a fix here => https://github.com/NekR/offline-plugin/pull/499
Yours faithfully,
LCDP

Closed and reopened the pull request here => https://github.com/NekR/offline-plugin/pull/499

We really need offline-plugin to work with webpack 5 so our team forked and worked on a release candidate that work with Webpack 5, it's available here => https://www.npmjs.com/package/@lcdp/offline-plugin
Any feedback are welcome, we need you to improve the compatibility with webpack 5 !

We really need offline-plugin to work with webpack 5 so our team forked and worked on a release candidate that work with Webpack 5, it's available here => https://www.npmjs.com/package/@lcdp/offline-plugin
Any feedback are welcome, we need you to improve the compatibility with webpack 5 !

I am still getting the exact same error as with offline-plugin:
nx run back-ui:build:production
>  NX   Invalid options object. Terser Plugin has been initialized using an options object that does not match the API schema.
       - options has an unknown property 'minimizer'. These properties are valid:
         object { test?, include?, exclude?, terserOptions?, extractComments?, parallel?, minify? }
      Pass --verbose to see the stacktrace.


amp create cluster doesn't take in account the SendGrid key

if the fichier amplifier.yml containing a SendGrid key is created in the directory platform/secrets, then the deploiment script deploy, take it in account to create the right docker secret in order to allow the mailer to find the key and be able to send email.
But,
if we launch amp cluster using the CLI, it doesn't work the amplifier.yml file is not taken in account

@freignat91 I think #1310 clarified the situation. Please reopen if needed.


fix "Error: Extra data:" error

When using 1Password 4, it seems that I get this error on some entries:
1pass: Error: Extra data: line 1 column 800 - line 1 column 816 (char 800 - 816)

I tracked it down to there be extra non-printing characters at the end of the decrypted json data.  This change fixed it for me.

I can confirm this issue, however I would use "rstrip" instead of "replace"


Duplicate event fix for master

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

On another note: If we are getting master up to date with develop in this PR, wondering why travis CI file still shows up. Develop branch has completely moved to GHA.

Codecov Report

Merging #838 (121da2b) into master (0121f2d) will decrease coverage by 0.17%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #838      +/-   ##
==========================================
- Coverage   69.47%   69.30%   -0.18%
==========================================
  Files          32       32
  Lines        1661     1668       +7
==========================================
+ Hits         1154     1156       +2
- Misses        507      512       +5

Impacted Files
Coverage Δ

src/KinesisVideoStream.cpp
47.77% <0.00%> (-3.42%)
:arrow_down:

src/KinesisVideoStream.h
100.00% <ø> (ø)

tst/ProducerTestFixture.h
84.53% <0.00%> (+1.20%)
:arrow_up:

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 0121f2d...121da2b. Read the comment docs.