Escape-free, error-free, parser-based chalk-template

Hey @sindresorhus and @Qix- - please check out this enhanced chalk-template function. This started out as a contribution to https://github.com/google/zx/pull/635 - but after thinking about it, it would be more appropriate upstreamed here.
I happen to enjoy writing parsers - weird hobby, I know.
Background: When creating that PR for zx, I realized that I couldn't just send arbitrary text through chalkTemplate - it is very picky about escaping, and matching parenthesis, and styles, etc. I started to write a pre-processor for the template, to adopt a different start/end syntax. I was planning using new new prefix like ct:{ but that wasn't short enough, and it might have been common enough to require escaping, which is a PITA to handle correctly. Then I realized - hey I already have the parser that is pre-processing the template - I could just go all the way and do the actual styling too. This would also solve another problem I had with chalkTemplate - you couldn't use a template as an argument to a template, since it would be automatically escaped by chalkTemplate.
Then I realized - hey - we don't need to handle escapes at all. The syntax of {style  or {rgb(  or {#  was already unique enough to not conflict with most arbitrary text. And, since its a parser, I didn't need to throw errors for mismatching { }s or anything - the fallback would be to just not style the unmatched text.
And so, the parser, AST, and renderer was born. This was written with typescript annotations as well, so I added typescript and prettier as dependencies.
Additionally, I changed the project layout a bit: created /src and /dist. Lastly, the exports were manipulated a little bit. There is no more default-export. There is one that uses chalk, one that uses chalkStderr, and one that you can use to bring your own chalk. There is also an 'experimental' export that exposes the AST. This is really just used to help play/develop the parser, but maybe its useful to someone. Its experimental as this is v1 of the AST and it may need some time to bake before being "API."
Technically, the parser is flexible to allow changing the 'prefix' of the template, currently set to {- this could be configurable, but currently it is not. The suffix is always '}' but that too could probably be configureable without much work - not sure if its a good idea.
Anyways, I'm wondering if there is any appetite for merging this PR, especially since this is basically a rewrite. I'm open to all comments/critiques - especially around the actual exported functions. And naming. Naming is hard. I'm bad at it.
This PR probably needs more documentation, more tests, and more general refinement. But I wanted to start the discussion before deciding if it was worth any more effort.

Hi there, this is a rather large departure from the current status quo, and it's also breaking a lot of expected functionality (e.g. (chalk`${'{bold escaped}'}`) is now bolded instead of being escaped). Since this might be formatting user input, and chalk might be extended in some cases, this leads to the potential for code injection to be performed, which was expressly why we chose to escape all interpolated tagged template values.
I'm not opposed to a more robust parser implementation but as of now I'm not really seeing the motivation other than the beginning/ending delimiters.
Instead, it might be simpler to construct the regular expression objects with the delimiters interpolated directly into the pattern using \Q and \E. Then the entire template function can be turned into a constructor that returns an instanced templater, with the default having { and } being the default delimiters.
Perhaps a bit of a hack to your particular problem, but as of now the change of base behavior is a non-starter for me, personally. Happy to discuss further, though :)

Hey! Thanks for your time and comments. I appreciate and share your concern for security. I’d be happy to work through your concerns to make sure that value is being added in a constructive way. To that end, before I do any more work, I’d like to talk through the use case, the concerns, and the mitigations and proposals - to make sure I'm on the right track.
Use Cases:

Be able to output json in the same template as styles.

import {ct} from 'chalk-template'
log(ct`{bold {“hello":"there","brave":"world”}})

Be able to nest templates

const header = '{bold.bgBlue  UPDATE }'
log(ct`${header} I just updated {green ${thingone}} to {green ${thingtwo}}`)

Paraphrasing your comment: args must continue to be escaped

for security, because the styles directly invoke functions on chalk ${'{bold escaped}'}
because its too different from current behavior

Constraints
Assume the user does not have direct access to chalk or chalk-template, herein known as the "template fn." So all configuration must be via the string. i.e. assume the user just has access to a tagged template function called echo which takes a styled chalk-flavored string and args.
This is playing on hard-mode. Doing it within opts on the template fn is probably fine for most people.
Analysis
Point 1
Changing the delimiters of a style to a different character could work, but that would like require access to the template fn. Unless we take a page from sed, and allow the delimiter to be specified in the text it self. i.e. sed s/replace/me/g === sed s|replace|me|g.
How I solved [1] was to just not treat a single { as the delimiter. Instead, it is the combination of { plus valid chalk templates, verified against the chalkInstance. That means {hello} will just be {hello} but {bold hello} will be hello. However, note {notexisting style} will not be rendered since notexisting is looked up from the chalkInstance (unless, of course, a custom chalk had defined a style called notexisting). I don't think unintentional rendering would happen too often, since {style, {rgb(1,2,3, or {#000000 is pretty uncommon, but you know what they say about assumptions...
Mitigations:
1a. Changing the { to something else, even multi-chars: i.e. c>bold, c>rgb(1,2,3), c>#:000000
1b. Make it (even) more unlikely by adding a leading style: i.e. {chalk.bold, {chalk.rgb(1,2,3), ${chalk.#000000
Point 2
The extra constraint of only using strings is a bummer. I realized it could have been solved if I had just pre-rendered the template to ANSI and then just inserted the ANSI in the next template. i.e.
const header = ct`{bold.bgBlue  UPDATE }`;
log(ct`${header} I just updated {green ${athing}} to {green ${bthing}}`)

Point 3
I totally agree that invoking arbitrary functions from template strings is risky. But breaking it down - its not always risky. Its not risky if you are in full control of the argument (i.e. did not come user). Its not risky if chalk hadn't been customized. But, lets be secure-by-default and make the default be escaping all arguments - but lets give the user an ability to opt-in to rendering args.
Mitigations:
3a. Create a new function nameTbd, which will cause the template fn to render the selected args.
log(ct`${nameTbd(header)} I just updated {green ${athing}} to {green ${bthing}}`)

or better yet, just use the template again
log(ct`${ct(header)} I just updated {green ${athing}} to {green ${bthing}}`)

-- which is basically an application of [2]
3b. Create a new keyword {nameTbd ${arg1}} but not ${arg2} which will render arg1 but not arg2.
Proposal
I'll do 1a and 1b. These will be options on the template builder fn.
import {ctbuilder} from 'chalk-template'
const ct = ctbuilder({ startDelimeter: 'c>', stylePrefix: 'chalk'})
log(ct`c>chalk.bold {"hello":"there"} c>chalk.green ${thing}}}`)

For 3 I will go ahead and make escaping of args true by default, to match previous behavior and to make it secure.
For 3a, will actually already work with no changes to the PR.
For 3b, this is playing on hard mode - and i think its pretty cool. A new keyword/directive that will cause the template renderer to render the template arg. The user would only use these on args that are known to be secure (i.e. not user input)
// note - no deps on chalk or chalk-template - just a function
// that hides them and exposes the chalk-flavored template
// via the third-party tagged template function `echo`
const header = '{bold.bgBlue  UPDATE }'
echo`{render ${header}} I just updated {green ${athing}} to {green ${bthing}}`)

All that being said, this definitely deserves a bump to 2.0.0.


Evaluate ASR metrics for noisy and non-noisy subsets

Implements this functionality through:

[x] an alternate entry point via CLI - eevee asr --noisy
[ ] a cleaning method to separate out the info tags from every utterance
[ ] optionally, a yaml to define the noisy and non-noisy subsets
[ ] add documentation for usage

tests failing is intentional. when the missing code is added, tests should/will not fail

Ready to review and merge

@greed2411 made changes. please check and merge whenever

looks good to me, thanks for the PR @sachinkum & @janaab11.
great work! it means a lot.


WIP:  Techdebt: Replace sure with regular assertions in Utilities

See https://github.com/getmoto/moto/issues/6035

@bblommers  test_paginator.py is a bit problematic in terms of pylint errors when using a less curated set of pylint checks.  I addressed some of them, but I wanted to let you know about the ones I ignored.
Some of the ones I addressed (I think I cleaned up some others before deciding to let you know):
tests/test_utilities/test_paginator.py:193:4: W0102: Dangerous default value [] as argument (dangerous-default-value)
tests/test_utilities/test_paginator.py:261:14: W0612: Unused variable 'token' (unused-variable)

The ones I ignored are listed below.  I believe I understand what's going on, but I thought I'd show you in case they should be addressed.
test_paginator.py:234:12: E1123: Unexpected keyword argument 'next_token' in method call (unexpected-keyword-arg)
test_paginator.py:248:12: E1123: Unexpected keyword argument 'next_token' in method call (unexpected-keyword-arg)
test_paginator.py:267:15: C0121: Comparison 'self.custom_limit == None' should be 'self.custom_limit is None' (singleton-comparison)
test_paginator.py:278:22: E1120: No value for argument 'resources' in method call (no-value-for-parameter)
test_paginator.py:281:22: E1123: Unexpected keyword argument 'next_token' in method call (unexpected-keyword-arg)
test_paginator.py:281:22: E1120: No value for argument 'resources' in method call (no-value-for-parameter)
test_paginator.py:288:22: E1123: Unexpected keyword argument 'next_token' in method call (unexpected-keyword-arg)
test_paginator.py:302:12: E1123: Unexpected keyword argument 'next_token' in method call (unexpected-keyword-arg)
test_paginator.py:184:8: W0201: Attribute 'custom_token' defined outside __init__ (attribute-defined-outside-init)
test_paginator.py:189:8: W0201: Attribute 'custom_limit' defined outside __init__ (attribute-defined-outside-init)

The ones I ignored are listed below. I believe I understand what's going on, but I thought I'd show you in case you want therm addressed.

Looks like a combination of our @paginate approach being a bit wonky, and pylint maybe not understanding what's going on.
The tests are readable, IMO, so I don't really mind that pylint might not like them.
The defined outside __init__ can be a code smell, and a source for bugs, but sometimes also just unavoidable (without major refactoring). In this case, I don't really see an issue with the test approach.
Thanks for letting me know!


Updated codecov dependency with "blessed" package.

<EMAIL_ADDRESS>-><EMAIL_ADDRESS>One of the reasons:

And codecov is what they have on their page:

Sounds good to me.
Hey, is this one capable of sending reports to CodeCov without being run in a CI environment?  I'd love to be able to test this locally at some point.

@isaacs I think so, this is what they tell you to install when you register new project and they have local execution examples.

Thanks!

Can you teach me the magic you just did here? I mean merged without merging. :)
And commit list is showing this:

Thanks.

I usually use this bash shortcut to merge pull requests: https://github.com/isaacs/dotfiles/blob/master/.extra.bashrc#L431-L473
If it's a fast-forward commit, then it'll autoclose.  If not, I close manually.  I'm usually not a fan of merge commits.

@isaacs do you mind to release these changes on npm? Thank you.

Oh, I've found this one :) https://github.com/tapjs/node-tap/commit/5af48879ab7c15d1ff607e4b810fbc56dd2c63b6 nvm then :)


package.json module pointing to wrong file

Describe the bug
package.json:8 "module" is pointing to "dist/lazyload.esm.min.js" -> which does not exist.
A non minified version exists but not the lazyload.esm.min.js.
This breaks compiling (production), when importing with e.g. import LazyLoad from "vanilla-lazyload";
Saw that minification of esm-version was disabled, with this one:
https://github.com/verlok/lazyload/commit/44e1403#diff-b9e12334e9eafd8341a6107dd98510c9
If you want, I could create a PR for pointing to non-min-version or enable minifying again?! :)
LazyLoad version

Version 11.0.4

Yes please open a pull request to point to the non minified version.
Just remember to bump the version number to 11.0.5.
Thanks

Hey @MarciD,
I found 5 minutes to do that on the train and I fixed it.
Thank you for reporting!
verlok


Auto scroll feature in Output window does not work

Type: Bug
Auto scroll feaure in Output window has stopped working after an update, probably a few weeks back.
Even with the lock icon in open state, it does not auto scroll when a new entry is added to the Output window.
VS Code version: Code 1.74.2 (e8a3071ea4344d9d48ef8a4df2c097372b0c5161, 2022-12-20T10:26:09.430Z)
OS version: Darwin x64 21.6.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
3, 2, 2

Memory (System)
16.00GB (0.02GB free)

Process Argv
--crash-reporter-id 7d6eb5c4-e43d-4d81-8d3c-c91dc610cd59

Screen Reader
no

VM
0%

Extensions (22)

Extension
Author (truncated)
Version

ng-template
Ang
15.0.3

vscode-eslint
dba
2.2.6

gitlens
eam
13.2.0

vscode-electricimp
ele
0.2.0

go
gol
0.37.0

Angular2
joh
13.0.0

vscode-docker
ms-
1.23.3

csharp
ms-
1.25.2

data-workspace-vscode
ms-
0.3.0

mssql
ms-
1.16.0

sql-bindings-vscode
ms-
0.3.0

sql-database-projects-vscode
ms-
0.19.0

isort
ms-
2022.8.0

python
ms-
2022.20.1

vscode-pylance
ms-
2022.12.20

jupyter
ms-
2022.11.1003412109

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.12

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

remote-containers
ms-
0.266.1

angular-console
nrw
17.27.1

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
vsccc:30610678
pyindex848cf:30577861
nodejswelcome1cf:30587006
3biah626:30602489
gswce1:30612156
3d0df643:30613357
f6dab269:30613381
fim-prod:30623723

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add ability to set output PDF filename

@gutschilla Changes have been made, added one test. I check the filename and that the file exists. I don't know if checking both is overkill. I can remove the check for File.exists? if you prefer.

@gutschilla is this okay to merge now or would you like some more changes?

@gutschilla Make sense, I removed the Zarex dep

@praveenperera Merged this, thanks a lot for your contribution!


[bitcoin]: inscritpion offset mismatched

output of rooch statedb genesis-verifyt:
offset in data source(height is 85566(included)): 50228297
offset in statedb(rooch relay after 852202(included): 49643926
[inscription] mismatched: exp: "Inscription { txid: d8cf4fe99687989c52a2d69908b4e32c0f728fb0d2e1464c9bb74e0182e46975, index: 0, offset: 50228297, sequence_number: 447319, inscription_number: 447106, is_curse: false, body: [123, 34, 112, 34, 58, 34, 98, 114, 99, 45, 50, 48, 34, 44, 34, 111, 112, 34, 58, 34, 109, 105, 110, 116, 34, 44, 34, 116, 105, 99, 107, 34, 58, 34, 112, 117, 110, 107, 34, 44, 34, 97, 109, 116, 34, 58, 34, 49, 34, 125], content_encoding: MoveOption { vec: [] }, content_type: MoveOption { vec: [text/plain;charset=utf-8] }, metadata: [], metaprotocol: MoveOption { vec: [] }, parents: [], pointer: MoveOption { vec: [] }, rune: MoveOption { vec: [] } }", act: "Inscription { txid: d8cf4fe99687989c52a2d69908b4e32c0f728fb0d2e1464c9bb74e0182e46975, index: 0, offset: 49643926, sequence_number: 447319, inscription_number: 447106, is_curse: false, body: [123, 34, 112, 34, 58, 34, 98, 114, 99, 45, 50, 48, 34, 44, 34, 111, 112, 34, 58, 34, 109, 105, 110, 116, 34, 44, 34, 116, 105, 99, 107, 34, 58, 34, 112, 117, 110, 107, 34, 44, 34, 97, 109, 116, 34, 58, 34, 49, 34, 125], content_encoding: MoveOption { vec: [] }, content_type: MoveOption { vec: [text/plain;charset=utf-8] }, metadata: [], metaprotocol: MoveOption { vec: [] }, parents: [], pointer: MoveOption { vec: [] }, rune: MoveOption { vec: [] } }"

result in https://ordinals.com/inscription/447106(2024/8/15 21:21 utc+8)

The inscription #7569e482014eb79b4c46e1d2b08f720f2ce3b40899d6a2529c988796e94fcfd8i0 spent as gas fee at height 855412, and final transfer at height 855629.
Therefore, it is necessary to synchronize Rooch to the corresponding height before comparing the offset. In the above case, before Rooch synchronizes to the height 855412,  both the offset and owner of Rooch and Bitcoin should be inconsistent.

spent as gas fee tx https://ordiscan.com/tx/74b2c06b6a85b4d1897dcf14e7dbe9239e25bfe80f2c33734dbad19560e8dc58
final transfer tx https://ordiscan.com/tx/df662337ec2622a3df79e2e708c063b9eb7bef3aa028bd8444b4eea9588ea0d6

Use this tool to view the records of inscription transfers, https://ordiscan.com/inscription/447106. Please note that some inscription data may be inaccurate sometimes.

@baichuan3 rooch has sync height at 855666, so it should be consistent

The inscription result in rooch:
rooch rpc request --method btc_queryInscriptions --params '[{"inscription_id":{"txid":"7569e482014eb79b4c46e1d2b08f720f2ce3b40899d6a2529c988796e94fcfd8","index":0}},null,null,null]'

{
  "data": [
    {
      "id": "0xd23c49fb9a742624498390a5b59c90e968c11d52b6437278a0e2eb9b82824103aa2a6f19511f00f4ebe2a5057cde8d78aff9e568becd4fb4df341ae9f0a4e63a",
      "owner": "rooch1fhey2uf6vpqm6fdmz4sxrscd4gltl90d5lthz5w4uqvq9l3ztkrqzl5nwm",
      "owner_bitcoin_address": "1Fu41MmbZLFHebUsDya8B3oqoG2MMZGiML",
      "flag": 0,
      "state_root": "0x5350415253455f4d45524b4c455f504c414345484f4c4445525f484153480000",
      "size": "0",
      "created_at": "1722815681000",
      "updated_at": "1722936378000",
      "object_type": "0x4::ord::Inscription",
      "value": {
        "txid": "0xd8cf4fe99687989c52a2d69908b4e32c0f728fb0d2e1464c9bb74e0182e46975",
        "bitcoin_txid": "7569e482014eb79b4c46e1d2b08f720f2ce3b40899d6a2529c988796e94fcfd8",
        "index": 0,
        "offset": "49643926",
        "sequence_number": 447319,
        "inscription_number": 447106,
        "is_curse": false,
        "body": "0x7b2270223a226272632d3230222c226f70223a226d696e74222c227469636b223a2270756e6b222c22616d74223a2231227d",
        "content_encoding": null,
        "content_type": "text/plain;charset=utf-8",
        "metadata": "0x",
        "metaprotocol": null,
        "parents": "",
        "pointer": null
      },
      "tx_order": "13656655",
      "state_index": "2101"
    }
  ],
  "next_cursor": {
    "tx_order": "13656655",
    "state_index": "2101"
  },
  "has_next_page": false
}

should be resolved after #2500


Could you tag a release ?

I already packaged kakoune-cr for alpine using a commit reference but it would be lovely if you could tag a release instead

I’m not fond of releases. 😭

What do you mean ? You could just tag and not "release" if you prefer :)


Font Overlapping on Obfuscated Java Code (long strings)

Error

JEB3 output

Sample
sample.apk.zip

JEB adds the following to avoid font overlapping: [...]
public final class bf {
public static String a = "e6a291c4b3f780d56c4d5e6f708192a3b4c5d6e7fb3d5e680f791a2c43b5d6e087f192a4c9180c4d5a2b3f7e619084c5d2a3b7f6e9485e7f6d0c1a3b21c0d6f7e58492b3a1c0d6f7e58492b3a9485e7f6d0c1a3b2b2f6d085e7a3c1943a7e580d6 [...] ed3021120374655647b8a99a8bfceddecf74deed47309aa903fc5665cfb812218b8b9aa9b8cfdeedfc0312213047566574";
static {
}
}

I don't have problems displaying this line and the f3306a variable using the latest Jadx version via Java 11.0.3 on Windows. I assume the problem is the line length - the hex string is more than 40k characters long.

fyi -- my box is Linux Ubuntu 18.04.2

Can you check the class xxxxxx.obfuscated.bp? bi? bj? It doesnt work for me. However, some are displayed okay.


OpenSource firmware for BLDC motor controller, chinese ultra cheap

Hi.
I wounder why there isn't a shield for Arduino to control BLDC motors and using the advanced FOC!!
Great project!!
I am being devoloping this OpenSource firmware and I adopted cheap chinese controllers from EBikes: they are not Arduino, they use STM8 or STM32 and there is now STM8 and STM32 Arduino versions.
See my firmware here: https://opensourceebikefirmware.bitbucket.io/

Thanks, your site is insanely dense and a great read.  The Chinese controllers that I bought did not have a ST micro on them just some Chinese 8 bit controller.


Settings: Model is disposed

Found this while running "out of sources", not sure how I was getting into that state, but I had settings opened and probably closed it. Might also be related to snippets?
[23396:0514/094110.853577:INFO:CONSOLE(273)] "Model is disposed!: Error: Model is disposed!
    at Model.TextModel._assertNotDisposed (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/editor/common/model/textModel.js:56:23)
    at Model.TextModelWithDecorations.changeDecorations (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/editor/common/model/textModelWithDecorations.js:141:18)
    at InsertSnippetController.stopAll (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/editor/contrib/snippet/common/snippetController.js:287:24)
    at file:////Users/bpasero/Development/Microsoft/monaco/out/vs/editor/contrib/snippet/common/snippetController.js:154:23
    at CallbackList.invoke (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/callbackList.js:58:43)
    at Emitter.fire (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/event.js:91:40)
    at CodeEditor.CommonCodeEditor.setModel (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/editor/common/commonCodeEditor.js:147:36)
    at TextFileEditor.clearInput (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/parts/files/browser/editors/textFileEditor.js:189:31)
    at SideBySidePreferencesWidget.clearInput (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/parts/preferences/browser/preferencesEditor.js:341:48)
    at PreferencesEditor.clearInput (file:////Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/parts/preferences/browser/preferencesEditor.js:122:46)", source: file:////Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/electron-browser/shell.js (273)

Looks to me related to Snippets
@jrieken FYI

it's the old code. it's not in use anymore


deviation service

Add the deviation logic in the config-server to act upon the information in the GRPC stream from the data-server.

for unhandled/unintended config -> create a deviation object per target
update logic through a channel


Question about Filters

Do you have a way of hiding filter options till a different option is selected?
Example:
I have Enclosures on my site, they come in 14 different sizes, I don't want 14 options shown unless they select the option enclosures.
If this is not possible, is there plans to do this?

Can you put some links to your example - I'm not sure I'm 100% following the challenge.

I have not set them all up yet.
But I'll try to explain better.
https://www.mdius.com/liquid-level-controls
This shows a LOT of filter options.
Would there be a way to maybe only show the Float Style filter only if you check Fiber Optic Switching under Switch Style?
This is how it looks right now.

This is how I'd like to see if I could do.

Then when you click an option.

Did that help?
I did notice another issue, but I'll post that 1 later.

Yes, I think I'm following now.
The issue I'm seeing is how you define how each attribute filter interrelates.
In your case, I can see why you'd want the other options hidden first, as with these type of products it doesn't really make sense to select one of the lower filters until you've picked the main type of switching style.
But in other uses of the filter on other shops, there might not be this kind of thing at all, where a customer would just as likely filter by some option lower down the list.
I believe what you are after would be able to be achieved though with a custom block template for the filter block.
You'd put in some logic in the block template to output the first group, then only output the other groups of options if the first group has at least one thing ticked.  So the short answer is to tackle it with PHP in a custom block template, instead of it being with javascript, or some feature that needs to be added to Community Store itself.

I think I'm following what you're suggesting.
Like when I said I saw another issue, it was that is was showing options with a Mini Float (0).
On line 53 of the Filter block, you have <?php if ($show) { ?>
I added <?php if ($show && $count != 0) { ?> to filter out the options with 0 counts.
It would be similar to this, only show the next block below if there are options available. That actually sounds better than what I was thinking. I'll let you know how it goes.

https://www.mytinytown.com/liquid-level-controls
So it seems I'm on the right track, I am just missing something. I just clicked the products with the highest amount, it seemed to be working till the 3rd group, then it gets weird, but still seems to kind of work. I'm not a fan of kind of though.
But then I question myself, would this be the best way, or would it be better to put them all in an accordion.
Click only the category you want to see. That would reduce the overwhelming selections, but still give all the options.
So, I'm going to give the accordion idea a try before I go any further. Also, with my fix above <?php if ($show && $count != 0) { ?> makes it so you cannot click 2 options in the same category.

Yeah, I think the accordion is the best idea as of now. I might pick back up on the other idea later.
https://www.mytinytown.com/liquid-level-controls

I have another question about filter/page list.
https://www.mytinytown.com/liquid-level-controls
On my test site I have added a Product Per Page feature that works, except:
I have is display 18 product by default, if I click 5 it works. Until I select a filter option then it defaults back to 18.  But if I select a Sort By option, it remains selected after selected a filter option. I narrowed it down to the word sort in the input select in blocks/community_product_list/view.php.
If I rename my Products Per Page input select name to "sort" it works, but then the Sort By obviously does not.
But I cannot figure out what triggers the sort option to stay.
I am guessing it is someplace in the Concrete core?


WebUI plots way too many data points when zooming out

Note: related to #1240 - "Replace plotly.js with echarts" has not fixed the problem of too many data points
System information

Have I written custom code (as opposed to using zenbot vanilla):
No

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 16.04

Zenbot version:
8bcd969

NodeJS version:
v8.9.4

Python version:
Python 2.7.12

Exact command to reproduce:
Open WebUI. Expand date range sliders to zoom out. Very quickly way too many data points are being plotted into the graph.

Describe the problem
Chrome starts to become unresponsive quickly. It's impossible to get the sliders to expand to cover the full range.
Source code / logs
See screenshot:

What are you specs?
Performance was the main reason i ported this from plotly to echarts as even my workstation (with a i7 hexa-core) was staling will loading the page and made navigation almost impossible.
With echarts even my phone can handle a couple of thousand datapoints without any problem.

@defkev Macbook 2017 brand-spanking-new with latest OS and Chrome.
The issue arises when the bit has been online for say 48hrs and when I load the web ui it's fine (but only because it is by default zoomed in on the last hour or so). If then try to zoom out and expand the graph to include all 48hrs it totally breaks. I can't even drag the slider to 24hrs let alone 48hrs.

A 2017 MBP should definitely be able to handle this.
By default we load the last 500 trades.
I never tried this with Chrome as i am personally using Fire/Waterfox everywhere.
Can you try one of the man echart demos at https://ecomfe.github.io/echarts-examples/public/index.html and see if they perform equally bad?

@defkev I actually had a look at (all) these demoes yesterday night before filing this issue.
These demoes mostly work fine (some simply don't run. If they run, they don't seem to crash the browser. Some with loads of data might be slow but it keeps working)
That said none of these demoes do exactly what you seem to be doing: It's a little bit comparing apple & pears.

Here's a screenshot of my web console on Chrome.


[Ready for review] Retrieve original run tasks and clone successful ones directly

For reference and posterity: the way we use resume is in one of two ways:

the usual: "clone all tasks and continue executing in runtime.py"
a distributed manner where a scheduler (argo/airflow) "resumes" the run (the execution frontier of tasks is relaunched by the scheduler and then all those tasks try to "resume") and then each task will execute its own thing and the scheduler will then continue. This is effectively a resume but without a runtime.py to drive it.

This PR is the second in the line to improve that second behavior:

the first elected a leader to do the cloning as opposed to doing it in a distributed fashion as we used to do
this one makes it also a lot faster to clone the tasks (ie: the job of the leader is simplified).


Reload share gallery when re-entering the app

This is so the images refresh to see the new ones

sorta kinda accomplished with the swipe to refresh.... if you disagree feel free to reopen


Chromium difference without script / with script

When I use the browser without launching puppeteer script, the web page works well and does not show me any security captcha. but when I do it with my puppeeter script, it sends me a captcha when entering the we. Should I configure something at the beginning of my scripts?

Duplicates of https://github.com/GoogleChrome/puppeteer/issues?q=is%3Aissue+Captcha+is%3Aclosed


Test the wallet CLI

There is now a good amount of code in the wallet, and it will continue to grow. It needs to be tested in an automated way.
I'm not exactly sure how to test CLI app code. This wallet interacts with user input, a filesystem, and a running node via RPC connection. That seems harder to test than simple functions.

We currently have a simple wallet integration test in CI, to ensure that the wallet connects to the local node:
https://github.com/Off-Narrative-Labs/Tuxedo/blob/36d7baca4ae159d77ea545d7cb999bd1e715557d/wallet/test.sh#L1-L6
The CI job fails if the wallet can't connect to the node because the process writes to stderr.
I think there might be cleaner ways to test the CLI, so I don't recommend using this pattern for the commands.

I like both approaches. We definitely need something elegant for testing each subcommand systematically.
But I also think we should expand the existing E2E test to send a series of wallet commands to the node to make sure transactions are still working properly. For example we should at least run through the commands in the wallet's Guided Tour.


VS Code frontend looking for files in wrong directory

Describe the bug
Occasionally (if not all the time), the VS Code frontend looks in the extension's directory instead of the one you're in when trying to locate files. This only seems to happen upon first creating the notebook. Closing and re-opening it seems to be the only workaround I've discovered for the issue.
Please complete the following:
Which version of .NET Interactive are you using?
1.0.160204+9a96ad3eee7b38d6db907a97513009e04f967825

OS

[x] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[x] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[x] Visual Studio Code
[ ] Other (please specify)

Screenshots

@jonsequitur @colombod I'm interested in your thoughts here.
The issue arises when we create a new notebook.  At that point in time the notebook doesn't exist on disk and we have no idea where to set the working directory, so it defaults to our extension's global storage path, e.g., c:\Users\<username>\AppData\Roaming\Code - Insiders\User\globalStorage\ms-dotnettools.dotnet-interactive-vscode.  I don't think there's anything else we can currently do here because we can't divine their desired location.  We could, however, add an extension variable defaultNotebookLocation or similar that we could default to.  My issue with that is one of discoverability and actual utility; I don't think it'll get used.
The next step is when the user saves the notebook for the first time.  Currently we do nothing so the working directory is still the global storage path.  At this point we can easily detect that it's a new file being saved for the first time and issue a ChangeWorkingDirectory command to the kernel, and that may be desirable unless the user has already set it on their own, but we have no real way of discovering that.
The final step is if the notebook is closed and reopened then the working directory is set as expected.  This part is fine.
My suggestion is paragraph 2: if we're saving a notebook for the first time, issue a ChangeWorkingDirectory command to the kernel.  This will work in most cases, except where the first thing a user does with a new notebook is something with the current directory and we come along and modify it out from under them, but I imagine this will be a tiny set of scenarios, certainly smaller than those that create a new notebook, save it somewhere, then do stuff that depends on the working directory.

The issue arises when we create a new notebook. At that point in time the notebook doesn't exist on disk and we have no idea where to set the working directory, so it defaults to our extension's global storage path

I think the ideal experience would be the directory where VS Code was started. For example, if I do this:
> code /some/path

and then I create a new document and save it, the file save dialog defaults to /some/path. Can we access that path and use it as the working directory for a new blank notebook?

does this repro with latest version of the extension?

We now explicitly set the working directory to whatever directory contains the notebook, or if it's a new untitled one, we use the vscode workspace directory.  Only if all of those fail do we use the extension directory, but at that point that's the only thing we can do.


Refactor size() function

Currently the size() function will traverse the list and count each node, returning the accumulated total. Unfortunately, this means that any call to size() results in O(n) time complexity.
If we can create a member variable called SIZE, and update it when nodes are deleted or added we can just return the current size and the time complexity will be constant.
This size member will need to be initialized to zero in the constructors, and updated in each of the modifier functions.

Hi @AlexanderJDupree I have submited a pull request for this issue.

Great job @dsolisherrera thanks for the help. I am completing a data structures course at my University now and will have long task list of projects and feature additions for this and other repos I have made. So, if you want to keep helping out I will be active on my repositories.


added image

please merge

Hello @Krish6170 ... thanku for ur contribution... it seems like there are some minor mistakes... please do fetch upstream and paste the code to the place as said in the readme.md
Also don't forget to add your data to the particular card

is this issue solved or not

@coder483 No.. I didn't rectify it simply because he just pasted the code given in the readme.md


FCREPO-2384

Addresses https://jira.duraspace.org/browse/FCREPO-2384
Adds apple trees as a dependency.

This is an instance of Maven's DependencyConvergence rule getting in the way of things. You will definitely want org.clojure/tools.logging (so don't exclude it), but in order to play Maven't game here, you'll need something like this:
    <dependency>
      <groupId>edu.amherst.acdc</groupId>
      <artifactId>acrepo-apple-trees</artifactId>
      <version>${apple-trees.version}</version>
      <exclusions>
        <exclusion>
          <groupId>commons-codec</groupId>
          <artifactId>commons-codec</artifactId>
        </exclusion>
      </exclusions>
    </dependency>
    <dependency>
      <groupId>org.clojure</groupId>
      <artifactId>tools.logging</artifactId>
      <version>0.3.1</version>
      <exclusions>
        <exclusion>
          <groupId>org.clojure</groupId>
          <artifactId>clojure</artifactId>
        </exclusion>
      </exclusions>
    </dependency>

Ordinarily, maven will take the "nearest" version of clojure, namely 1.8.0, but given that the enforcer plugin is turned on, it is very strict about such things.

@acoburn Awesome, thank you.  I didn't think of trying to import tools.logging explicitly and put the exclusion there.

@acoburn I just pushed up a change to address your feedback.


Root system theory

Mathlib contains a definition of root systems. We should develop this theory further including:

A root system induces a canonical non-degenerate bilinear form
The dual of a root system is a root system
Define the automorphism group, Weyl group
Define bases and prove they exist
Define positive roots
Cartan matrices
Classify root systems

I'm delighted to see some interest in this from @ScottCarnahan
Please feel free to split this issue up into smaller pieces!

It looks like @DeeproChoudhury already has made some progress on this, so I'll try to work in an orthogonal direction.
My interest in this issue mostly comes from the infinite-dimensional setting, because the real roots of a Borcherds-Kac-Moody algebra form a root system in the mathlib4 sense minus the [Finite ι] hypothesis.  There is an axiomatic treatment by Moody, Pianzola "On infinite root systems" Trans. AMS v.315 (1989).  Anyway, I've started a branch where I removed finiteness from the basic lemmas, and I hope it doesn't make things too cumbersome if merged.

@ScottCarnahan Please do carry out this generalisation if you have the appetite, and ping me on the PR as soon as you are ready.
I don't really know the theory of infinite root systems and I would be extremely happy to learn it via Mathlib.


Add/Change - The 'Sit' system could use ArmorStands instead of Arrows

ArmorStands are magnificent things. They can be invisible, unhittable(using the Marker tag, mind you this will change where entities sit while mounting it to near the feet instead of on the head), and perhaps most importantly they can sit or float anywhere, entirely still, never despawning.
Relevant NBT tags for the stands to make them pretty much uninteractable+invincible:
{Marker:1,Invisible:1,Invulnerable:1,NoGravity:1,DisabledSlots:2039583}
Could even use the CustomName tag to easily define each armorstand based on the NPC's name/ID that is sitting in it (if that would be useful in any way, I don't know...)
The current arrow system is far from reliable. ArmorStands are rock solid and awesome.
Screenshots of myself using an ArmorStand to sit on: http://imgur.com/a/tk47d

@Blue0Dwarf But the armorstand has a bigger hitbox and cancels block-placing.

The block placing seems like an absolute non-issue to me given the fact that the NPC itself will cancel block placing. That's already a flaw with the current system. You just need to teleport your NPC to what they're meant to be sitting on. They only stop block placing where the base would be (if the Marker flag is set), and that is right where the NPC would be sitting anyway.
An armorstand with the Marker attribute pretty well has no hitbox... I'm not entirely convinced it DOES have a hitbox, I certainly can't seem to hit them. It'll stop blocks being placed in the block that its base occupies, but I'm not seeing it have any sort of "Hitbox blocking interaction" issues.

@Blue0Dwarf Maybe you're right ... :smile::smiley:

it makes me wonder, how is this not implemented yet O_o Extremely better solution than arrows (affected by gravity so the citizens are not able to be suspended in mid air sitting (meditating - levitating))

That. Is an excellent question. >.>
This is actually the current system for /ex animate <npc> animation:sit
But not for /npc sit
0.o
Hopefully someone'll remember to make it work for both sometime soon >.>

this uses Citizens sit animation, which uses armor stands


remove the override for aggregate query creation

This should address the issue noted in #4. Been using my fork for some time in relation to Mappable, and Mutable at least and everything seems to work just fine.

not sure what it does and there's no test


(Probably) Action reports cache not found when s3 credentials misconfigured

Hi,
We're evaluating runs-on and experimenting with the cache.
We think (based on evidence below) that:
When s3 is misconfigured and accessing gives a 403, the action repots that the key was not found, but it should error or report auth problems.

The cache action detected a local S3 bucket cache. Using it.
Cache not found for input keys: /home/runner/_work/_tool/Python/3.10.14/x64-acfb00df7caf8b5650137b8f855e203b58ab4784bf6518061fc092eb576068e4

Due to misconfiguration on our end, we know that the runner can't access that bucket . Trying it more directly gives a 403.
It did take a few hours to figure that out. A clearer error message here would be helpful for future users

Hi @labeleddata, if this is a config error, you should get a message starting with Error listing objects with prefix in the logs. Did you see this line?

Thanks.
Oh I am not sure if it's in the logs, it's been many jobs since then.
The ask here is a clearer and more explicit error message at the point of failiure.
Niether "Error listing objects" or "cache not found" aren't telling me the root error.
The reason we want explicit errors is that  we have a decently sized dev org, and all the engineers write GHA, sometimes.
We want them all to use this, but don't want them all to become "runson" experts. Clear error messages means they can either solve it or know what/who to ask fast

Sure, it’s just that currently it’s supposed to be the message that gets spit out when there is a permission issue, and I don’t see it in the logs your shared so I’m wondering what was the root cause here. Also note that v2.6.3 now ships with a magic cache so that you don’t need to use this action anymore. https://runs-on.com/caching/magic-cache/

Closing this issue cause I don't remember the details and we got magic cache working


Payments page

Implemented in https://github.com/lvlup-pro/spawn-frontend-material/commit/213c6f3dab9450f27e6e215111975f1dcb775e49


[Feature Request] Show hostile settings for non owned

I would like an option to show the hostile settings for non owned tokens.
I would like my players to see their tooltip normally but for enemies and allies a different one.

An option to show just owned tokens and hostile would already be pretty nice.

A full rewrite of the settings editor will be done in the future. But for now I'm taking a 1-2 weeks brake from active development.
So this will take some time. But I plan on implementing full control over each actor type in a system and between what GMs see vs what players can see

Does the rewrite include an option for the GM to see the hostile version of the tooltip?
For example, as GM: For Hostile I want to see HP, AC, and speed, but for Friendly I only want to see HP and AC.

The rework will include a separation between GM's tooltip and player. And a separation between actor types.
E.g. In dnd5e you have characters and NPC, you will now be able to set up a different tooltip for each one as a GM. For the player side of things, you will be able to set a different tooltip depending on the tokens DISPOSITIONS (hostile, neutral, friendly)...
Complicated, but with the new UI, I hope it will make sense and be usable

Your plan makes sense, but I think for me it would be more helpful to have:
GM view: Friendly / Neutral / Hostile
Player view: Friendly / Neutral / Hostile
For some of my bad guy actors, I make them characters and not NPCs so I can be more specific with their sheet.

@jessev14 This will be the new settings menu. Hope it makes more sense now.

Wow this is perfect! Can't wait for the update!


Obtain funding for ARM CI GH runners

As per https://github.com/open-quantum-safe/liboqs/issues/1789#issuecomment-2105730555

@hartm What would the process be for getting approval to spend money Github ARM runners?

We just had a staff discussion about this.  @ryjones has created a new thread, which he plans to bring to the TAC next week:  https://github.com/PQCA/TAC/issues/20
The basic idea is that we first need an estimate of the budget needed.  Ry is the expert here and can help figure things out.  Then we will discuss at the TAC meeting and get approval at the board meeting.  Hopefully we can get this all done next week.
From our rough estimates, this should be very feasible and we should definitely have the budget.

done


feat(Select): rework Select component

This PR contains a rework of the Select input field. The old version was far from complete and wasn't applicable to all the use cases we need. This new version enables some features planned for the next quarter.
Highlights
Options grouping
We can now nest options into separate logical groups to help users quickly navigate through the options list.

Styled multi-select variant
The original version contained a multi-select variant but it was completely unstyled.

Custom menu options
We allow appending custom actions to the dropdown menu. Those actions are getting general info about the select state as an argument.

Option creation
The new options can be now created from the select itself. This component is exported as a separate <CreatableSelect /> due to type differences but it still keeps the exact same styling and default behaviour as a normal select.

@ajkl2533 Can be reused in Filters?


/g does not working

The command /g does not working in the update.
1.0-SNAPSHOT-git-CardinalPGM-624c987

Dupe https://github.com/twizmwazin/CardinalPGM/issues/174
Please check the issue tracker of already submitted issues before creating one please

Please go to #144
There we are talking about that.
El dia 15/02/2015 16.49, "Shipi1"<EMAIL_ADDRESS>va escriure:

The command /g does not working in the update.
—
Reply to this email directly or view it on GitHub
https://github.com/twizmwazin/CardinalPGM/issues/178.

Nothing faillll
El dia 15/02/2015 16.50, "Elespawol GMail"<EMAIL_ADDRESS>va
escriure:

Please go to #144
There we are talking about that.
El dia 15/02/2015 16.49, "Shipi1"<EMAIL_ADDRESS>va escriure:

The command /g does not working in the update.
—
Reply to this email directly or view it on GitHub
https://github.com/twizmwazin/CardinalPGM/issues/178.

ok.
ty Elliot_


Knight's Tour Problem Added under Backtracking

Have you read the Contributing Guidelines on Pull Requests?
YES
Description
As assigned by @shraddhavp for issue #1475, I have added Knight's Tour problem under Backtracking algorithms.
The solution to the problem uses a recursive approach which places the Knight in each and every possible cell one by one
and repeat this process until all cells are visited.
Problem Link:- Knight's Tour
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#1475  #1504

LGTM @charlie219

@iamrajiv Link to the problem is added to the solution code


Hi, can you make it runnable on cpu-only?

Hi,
I have a simple request to make the code runnable in cpu-only machine.
It is more testing the Python notebook.
Thank you.

Not sure if it's possible - might need to find equivalents for the in place abn functions that currently rely on CUDA...I'm interested in finding out the answer to this question too.

@billcai what do you mean by the ifs and is it just in the module/src/inplace_abn.cpp file that this is the case?

Removed this if else statement and just leave the cpu version for all the functions in inplace_abn.cpp
For example, from:
if (x.is_cuda()) {
return mean_var_cuda(x);
} else {
return mean_var_cpu(x);
}
To'
return mean_var_cpu(x):

Thanks a lot @billcai . Your input helped me after days of struggle on the same. One more important thing I had to do was comment out or delete the "inplace_abn_cuda.cu" in the functions .py script in the modules. Ultimately, then the TernausNetV2 was imported.


use enable-target-optspace

It helps to cutdown some roughly ~500KB to 1MB~ up to 300KB of occupied space of EBOOT.PBP.
I did some tests of samples in website and they are all working.

I wonder if this has any side effects, but it does seem interesting.

Interesting, right? but the problem with this was optimized for code size instead of speed. This will turn -O2 to -Os based from here and disable the others. This is all i know the effects of that changes, What do you think? I want to post this in discord but this is non-prio and they really want is speed. 😅


Add support for generic k8s cluster

It will be good to support generic k8s cluster, contents can be

kubectl cp from local to remote pod (may use ksync)
use pvc from shared local to remote pod

If needed, the remote pod (kaniko) can be run as permanent.

Hey :) I didn't think about using kbuild for dev workflows, but this might be an interesting use-case. So you're basically thinking of syncing your local files to a volume (can be any volume right?) and thus we need some mechanism to rebuild+push the image when they change. This would require some pod (not only kaniko) to run permanently in the cluster, correct.
One part I don't really get, is that a "generic k8s cluster" support would be good. As of now, I just use an empty dir volume to copy the build context to. This should work on every cluster from minikube/kind to even GKE clusters. Maybe I just got this part wrong...

so far it states for GKE in readme, i thought it was not for other k8s instance. (I use rancher, it doesn't work, may raise another issue)

Sure just raise another issue. I expected it to work on every k8s cluster but I can have a look :)


No C# support even with C# extension installed

VSCode Version: Code 1.5.2 (66f37fd2a99eb9d628dd374d81d78835b410c39b, 2016-09-12T13:30:03.456Z)
OS Version: Windows_NT ia32 10.0.14393

VSCode has stopped listing C# as an available language.  It opens all *.cs files as plain text and C# is not selectable as a file type.  I have the C# extension installed. I have attempted uninstalling and reinstalling the current extension, VSCode in its entirety, and the legacy C# extension.
Steps to Reproduce:

Have C# extension installed
dotnet new a new project
Open Program.cs from Explorer or via code . in the directory or code Program.cs

Tried and it works for me. I have only the current C# extension installed, not the legacy one.
Could you check the console in the dev tools (Help > Toggle Dev Tools) for errors/hints?

I am getting two errors concerning the thread service and extension service:

mainThreadExtensionService.ts

threadService.ts

Fixed.
I forced VSCode to re-download and reinstall all my extensions by rm *-ing inside the .vscode directory in my user directory.  I then reinstalled the extensions and they worked correctly.

@joaomoreno Anything we can do here? Maybe improve the logging?

For the record, those error messages were no longer present after I cleared .vscode/extensions.

Yeah @chrmarti logging is definitely something we need to tackle together pretty soon.

Clearing the "bug" label as there is nothing to verify


White bar into the top

Hello folks!!
Thanks for your work, I was really looking for a similar idea and I think that you are doing a really good job.
I don't understand why we have this white bar in the top of the screen when we start a record.

Can you help explain me why? :) Thanks a lot. It is also into the final video

Hey @gianarb!
Do you mean this white bar? http://i.imgur.com/ZujV39X.png

Hello @doot0 no the white in the bottom

Okay, this is interesting. Let's clear up a few things:

I presume this screenshot is taken while recording or before starting it?
Could you please share the actual exported file?
What version is your OS?

Thank you for the kind words!

When I select the size of the recording all is fine.. When I start to record in the bottom I have this bar.. And it's also in the final result..

You can see a bar :) I am happy to know that it's not a feature but my problem :D

Will investigate this with @matheuss as the problem may possibly be upstream in https://github.com/wulkano/aperture.js.
Do you have any software installed on your machine that modifies screen output or anything similar? This is very strange. 🤔

We don't officially support OS X Yosemite. That being said we know quite a few people haven't upgraded yet, but we have not heard of this issue before.
Just for the record, these bars on on the top and bottom of your screen, they show both before you start recording when using the crop tool to select the area you wish to record, and while recording as well as being present in the export as shown above?

The bar is only in the bottom of my screen.
It appears when I start to record.
it is into the screen (I can see it) and also into the final file.
When I use the cropt tool I can not see the bar.

Thank you for clarifying! We'll update you here if and when we're able to reproduce and fix it. In the meantime you might want to sign up for the newsletter to see if the next beta solves the issue.

@gianarb I'm having a lot of trouble trying to reproduce this 👎.
Can you try an older version of Kap for me? You should uninstall your current version of Kap beforehand.

I tried yesterday with the version 0.2.0 and version 0.3.0 and I have the same behavior

@gianarb In that case the issue is probably exclusive to your machine.
Is there anything you're aware of about your Macbook related to your screen that could be interfering with the output?

mm I am thinking but I can not see nothing strange.. I have a look and come back if I can notice something.. In the meantime I close the PR :)
Thanks anyway!

Just in case somebody else ends up having this issue we're gonna leave this one open for now so we've got a bit of searchable paper trail on it. Thanks for your report!

👍 thanks

@gianarb @doot0 Exactly same issue here.
cc @matheuss

@alisonmonteiro @matheuss This is beyond weird, could you give us some more details about your setup please?

i have the exact same problem on yosemite.

added a video of the capture process. tried to delete the html, but looks like a wrapper/window problem (electron?): https://youtu.be/O0P5RycZaPs

@polarity @alisonmonteiro @gianarb Would you be willing to test a new build of the app to see if that solves the issue? Make sure to not replace your current working version of the app, this build is in development and has bugs and weird behaviour. Don't expect to rely on it for everyday use! https://65-65411043-gh.circle-artifacts.com/0/tmp/circle-artifacts.fxBO5IZ/Kap-0.3.0.dmg

@skllcrn the problem persists in the new build ( new video: https://youtu.be/GSi9SDO2UY8 )

@polarity Argh... Thank you for testing it. We've got one last thing to try, I'll let you know when it's ready!

I'm also experiencing this issue. Also on Yosemite (10.10.5), experienced it with both Kap 0.2 and Kap 1.0.

We should have a method to fix this soon. I'll be looking into it tonight.

Hey guys,
My co-worker used Kap and he experienced the same thing. He then upgraded from Yosemite to Sierra, and it works now without the bar. So highly likely that it's something with Kap on Yosemite.

I've also upgraded to Sierra and no longer have this issue.
On Thu, Jan 26, 2017 at 4:22 AM, onbjerg<EMAIL_ADDRESS>wrote:

Hey guys,
My co-worker used Kap and he experienced the same thing. He then upgraded
from Yosemite to Sierra, and it works now without the bar. So highly likely
that it's something with Kap on Yosemite.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/wulkano/kap/issues/67#issuecomment-275366048, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMs-8yPAfr5FO02LsRWHd1i6lVDgbzYfks5rWIH4gaJpZM4KoI6G
.

Also experiencing this issue on Yosemite. Tried building Kap from master and still see the issue (at https://github.com/wulkano/kap/commit/f9cc3b1cc98b8243012149962b9640e05e03cf24).
Maybe will give me incentive to finally upgrade to Sierra 😄 .

Upgrading to Sierra also fixed the problem for me.

Are you able to reproduce the issue on the latest master @hanxue? If you're still seeing the issue, one alternative workaround seems to be to update your OS.

Are you able to reproduce the issue on master @gianarb, @alisonmonteiro, @polarity, @parkerpearson @onbjerg and @joehand? If you're still seeing the issue, one alternative workaround seems to be to update your OS.

@skllcrn upgrading my OS to Sierra fixed the problem for me

That's great @alisonmonteiro. Going forward we will support macOS 10.12 (Sierra) and up.


Darts models (V1, V2) are the same?

Describe the bug
A clear and concise description of what the bug is.
If you look at the V1 and V2 of your DARTS implementation, it seems like they are exactly the same. Is this intended?
V1 and V2

Yes, their models are the same, the only difference is optimization: one using first-order approximation, and another using the second-order calculation for the architecture encoding. I implement their optimization in other python files instead of the model files:
https://github.com/D-X-Y/NAS-Projects/blob/master/exps/algos/DARTS-V2.py#L46


Exclude tests from build in library.json for PlatformIO

Hi, thank you for the great library.
This PR fixes the build when using the PaltformIO system, where this library is listed as well (https://platformio.org/lib/show/5421/tiny-AES-c)
I added a srcFilter directive to the build section in the library.json file, to exclude the test programs when the project is compiled into a library. Without this change, the library clashes with the main user code:
Compiling .pio/build/uno/lib2ed/tiny-AES-c/aes.c.o
Compiling .pio/build/uno/lib2ed/tiny-AES-c/test.c.o
Compiling .pio/build/uno/lib2ed/tiny-AES-c/test.cpp.o
Archiving .pio/build/uno/lib2ed/libtiny-AES-c.a
Indexing .pio/build/uno/lib2ed/libtiny-AES-c.a

In addition, I added the most recent version number (1.0.0) to the library manifest file. This version bump should automatically update the version hosted on PlatformIO.

Hi @StarGate01 and thanks for your kind word! :smiley:
I have no clue how PlatformIO really works, I just accepted this PR https://github.com/kokke/tiny-AES-c/pull/94 on good faith.
Maybe @lucifurtun or @pjunni has a comment? (They have made contributions to the file).

Thank you! If you want to have a look at the applicable section of the PlatformIO docs, you can find it here: https://docs.platformio.org/en/latest/librarymanager/config.html#srcfilter .


Write guide for setting this up

Add an example of using Cloud Scheduler for running the function periodically.

https://www.simoahava.com/google-cloud/lighthouse-bigquery-google-cloud-platform/


ESLint Flat Config

Upgrade to eslint 9, which needs us to migrate to an eslint "flat config" file.

Fixed by #178.


Regression - Sample HandleEventsYourself not downloading

Hi,
There seems to be a regression between  2.0.9-preview20220106001 and 2.1.0.
I used code based on HandleEventsYourself, which worked with preview version and noticed the InitAndBeginDownload doesn't trigger the download.
I then downloaded and tested the original HandleEventsYourself code from tag 2.1.0, and it won't download the update.
I can't find the preview tag anymore to perform a compare. Any hint to fix?
I also checked version 2.0.9 and it worked fine, it seems to be something introduced with version 2.1.0
Regards,
Emanuele

Hi @emazv72,
Thank you so much for reporting this problem. I really appreciate it. I was able to replicate the bug that you experienced, and a fix is rolling out to NuGet via CI/CD now -- v2.1.1.
Please note that I did find a small issue in the sample -- it is technically possible that if all the right race conditions happen and your internet is SUPER FAST, the download finished event can be called before download started, and the download started event is still called. This...could be fixed in the future, but since most will likely never run into that, but wanted to let you know.
Again, thank you!!

Hi @Deadpikle,
Thank you for the quick fix, I've integrated the new release and I can confirm it works just fine, great job!


⚠ Bofrost.at has degraded performance

In 049ca76, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2126 ms

Resolved: Bofrost.at performance has improved in 51b3766.


Merge pull request #1414 from rmohr/memory-overcommit-v2

What this PR does / why we need it:
Allow separating guest memory from resource request
(cherry picked from commit 1628128e6fda45d06b9eb22fbd2689c652ba3017)
Signed-off-by: Roman Mohr<EMAIL_ADDRESS>Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Allow real memory overcommit on a node by adding a field spec.domain.memory.guest, which sets the visible memory for the guest and which can be higher than the amount of memory than requested from the k8s scheduler.

ci test please

Tests look much better now, all storage tests fail on OS environments. but it does not relate to the PR.


Don't use the Exception base class

https://github.com/vipyrsec/dragonfly-loader/blob/909810f18dfb9d0218c73b7e3fb229127222d102/src/loader/settings.py#L33-L43
This should implement a custom exception class.
Or even better, use pydantic-settings.

This should be resolved now, Robin's PR (#16) switches to pydantic-settings.


X-point search needs to try to look for multiple x-points

Currently, neutpy works for single-null reactors and only searches for 1 x-point in _get_sep_lines(). Can/should be made more robust in the future.

Fixed


Fix non-fatal pester test failures

On another PR, I see a bunch of red error messages, but they appear to not actually fail the tests, but makes it hard to determine what actually failed.  Cleaned up the tests.  It appears that with the refactoring of the test modules, some lines of code were missed.  There were also cut and paste issues in the test code so cleanup doesn't happen correctly.

@mirichmo can you merge?


feature request: add CORS support

Add CORS support, loaded from CORS_ORIGINS in env. I've opened a WIP pull request in #47. This is useful for enabling browsers to directly request to the local faster-whisper-server, as currently they are blocked unless they use an intermediate server endpoint.
Invocations from the browser are currently blocked unless CORS is set. For now, I'm proxying requests via a rust backend function (I'm using Tauri).
Any help with how to integrate and pass environment variables in the docker images? I'm not a docker expert and am unsure how to best continue.

#50

Thank you very much!


Horizontal bar profile pic (logo) not centered at all :(

This is so sad, lets cry.

Sotchapp issue!! just margin lol 😮


Fix no implicit any in utils.ts

@oyejorge This should fix no implicit any in utils, opening a PR early since it's my first one here. This unfortunately does break one test, I believe it's the addEvent(document,'mousedown', (e) => doc_mousedown(e as MouseEvent)); change but have no idea why. Also fixed settings.d.ts and removed unused value variable in loadCallback.

Thanks!


Mac-Android specific salt states are running on mac-ios test beds

Mac ios machines do not have android phones attached making the android-sdk step fail and preventing some other steps from running. We prevent this problem we need to separate how the states are assigned to mac-ios and mac-android test beds.
\cc @keyonghan

This is fixed by configuring salt.

For this type of issues should we track until they land to prod?

Good point. Reopen.

Blocked on the salt configs propagating to prod master.

@keyonghan can we close this one?


Add disableWarnings option

Addresses #21 and #23.

awesome. thanks.
why not pass the option directly through the template here:
https://github.com/okcoker/babel-plugin-universal-import/blob/afb100920a1af8116eb773e93d370b027496fd7c/index.js#L130

I actually did that at first, but I thought I'd save the couple bytes and extra diffs within the snapshots.

and by save a couple bytes, I mean not turning importCss('./Foo') into importCss('./Foo', {}) when disableWarnings is not provided.

feel you. still probably most idiomatic just to pass a 4th key/val to the template.

I can update it.

Ok updated. I basically cmd + z'd a few times. Originally I had it like the latest push so you could easily add additional options in the future but then I thought that might be too forward so I had undid that.
Let me know if this works.

Very nice. Would be nice if the options not just for CSS (if we were ever to have any) were handled by this, but I like having this in the code for future reference. I like how you made it its own function.

Oh yea that was another reason I got rid of cssOptions in the first place because disableWarnings could potentially apply to non css-y things in the future as well.
Ah well. Thanks for the quick merge!

just one thing for future reference; use npm run cm to make commits. it formats them a special way that will automatically deploy to NPM from travis when merged to master.


Issues with Android SDK

Installing via gradle on a brand new project.  Seeing a bunch of the following errors

Error:(9, 5) error: resource android:attr/dialogCornerRadius not found.

I believe this is from a package mismatch, but I'm unable to find it.  Pretty easy to replicate by creating a new project in Android Studio.
Using the following SDK targets

I don't believe this is related to this SDK.  Seemed to be a gradle issue.


add csuid note

Description
Added an echo message to configure custom domain verification ID in the script
Checklist

[ ] This pull request was tested on both of:

[ ] PowerShell 5.1 (Windows)
[ ] PowerShell 6.x
[x] PowerShell 7.x (Latest PowerShell)

[x] Scripts do not contain static passwords or other secret tokens.
[ ] All Azure resource identifiers which must be universally unique are guaranteed to be so.

Testing information

Platform and PowerShell version: Linux 5.4.0-1094-azure #100~18.04.1-Ubuntu SMP Mon Oct 17 11:44:30 UTC 2022              7.2.7
Az version: 9.1.0

#sign-off

@tamram please take a look and merge


Table 6 :  Multi-Stage

Is there a code where I can reference the Multi-Stage experiment conducted on Table 6?
It looks like the experiment itself is mentioned in issue #11

You can just train a supervised model on the labeled data and then retrain a model using both labeled data and unlabeled data.

Thank you for your reply,
I have question regarding your reply.
If I train a supervised model on labeled data (which is same as baseline) and retrain a model using both types of data,
isn't it simply inserting a trained model into a E2E method? How is it same as Multi-Stage of Table 6?

Multi-Stage here means you have to do semi-supervised learning in train a model on labeled data->get label on unlabeled data->train on both labeled data and unlabeled data.

If both stages of training are conducted with same training code, do they have same data augmentation? From your paper, it seems strong augmentation is used to train student. (While teacher infers weakly augmented images)

No, you'd better refer to the paper of STAC. In the multi-stage setting, the same setting is used. ( strong augmentation is used to train student While teacher infers weakly augmented images)

Thank you for your reply!


position mismatch when soft wrap option is enabled

position calculation is not considering the soft-wrapped sentenses by atom.

Confirmed in 0.4.1.

already fixed, use the latest version


Enum not deserialized

I am using solita for the first time in my anchor program. I am trying to fetch the following account and it's throwing an error.

while trying to fetch from address using

It's not able to deserialize the enum BlockType and throwing the following AssertionError

Turns out it was my bad, I was passing wrong address to the function.


chore(deps): update golang.org/x/exp digest to 9b4947d

This PR contains the following updates:

Package
Type
Update
Change

golang.org/x/exp
require
digest
778ce7b -> 9b4947d

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 63.005%

Files with Coverage Reduction
New Missed Lines
%

pkg/config/env.go
4
79.65%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.2%

Covered Lines:
1233

Relevant Lines:
1957

💛  - Coveralls


Is an SMS sent when there is no communication network?

If the device has no network available then the sms won't get delivered but I think you might be able to see the sms in the default sms app.

I closed the WIFI & Mobile Data in my Android Phone.
I sent SMS using Messages app and received in my other Phone.
I sent SMS using Telephony application and the SMS is not in Messages app and was not received in my other Phone

I am not sure how the SmsManager on android works in situations like this. I'll have a look.
Would be great if you could also share the code snippet that you are using.

Thanks for your cooperation,
I can't share the code but can do a test with code you will share with me
Kobi
On Sun, Jun 13, 2021 at 7:33 AM Shounak Mulay @.***>
wrote:

I am not sure how the SmsManager on android works in situations like this.
I'll have a look.
Would be great if you could also share the code snippet that you are using.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/shounakmulay/Telephony/issues/94#issuecomment-860149621,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARIYHPANESRPUVVSPMIXZJLTSQYITANCNFSM46KYYTWQ
.

hi
why i get the error
Restarted application in 3,034ms.
E/flutter ( 6235): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: MissingPluginException(No implementation found for method requestPhoneAndSmsPermissions on channel plugins.shounakmulay.com/foreground_sms_channel)
E/flutter ( 6235): #0      MethodChannel._invokeMethod
E/flutter ( 6235):
E/flutter ( 6235): #1      _SendSMSstate.initPlatformState
E/flutter ( 6235):
E/flutter ( 6235):
E/flutter ( 6235): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: MissingPluginException(No implementation found for method openDialer on channel plugins.shounakmulay.com/foreground_sms_channel)
E/flutter ( 6235): #0      MethodChannel._invokeMethod
E/flutter ( 6235):
E/flutter ( 6235): #1      Telephony.openDialer (package:telephony/telephony.dart:514:5)
E/flutter ( 6235):
E/flutter ( 6235): #2      _SendSMSstate.build.
E/flutter ( 6235):
E/flutter ( 6235):
i'm using the code
import 'package:flutter/material.dart';
import 'dart:async';
import 'package:telephony/telephony.dart';
onBackgroundMessage(SmsMessage message) {
debugPrint("onBackgroundMessage called");
}
//void main() {
// runApp(MyApp());
//}
class SendSMS extends StatefulWidget {
@override
_SendSMSstate createState() => _SendSMSstate();
}
class _SendSMSstate extends State {
String _message = "";
final telephony = Telephony.instance;
@override
void initState() {
super.initState();
initPlatformState();
}
onMessage(SmsMessage message) async {
setState(() {
_message = message.body ?? "Error reading message body.";
});
}
onSendStatus(SendStatus status) {
setState(() {
_message = status == SendStatus.SENT ? "sent" : "delivered";
});
}
// Platform messages are asynchronous, so we initialize in an async method.
Future initPlatformState() async {
// Platform messages may fail, so we use a try/catch PlatformException.
// If the widget was removed from the tree while the asynchronous platform
// message was in flight, we want to discard the reply rather than calling
// setState to update our non-existent appearance.
final bool result = await telephony.requestPhoneAndSmsPermissions;

if (result != null && result) {
  telephony.listenIncomingSms(
      onNewMessage: onMessage, onBackgroundMessage: onBackgroundMessage);
}

if (!mounted) return;

}
@override
Widget build(BuildContext context) {
return MaterialApp(
home: Scaffold(
appBar: AppBar(
title: const Text('Plugin example app'),
),
body: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: [
Center(child: Text("Latest received SMS: $_message")),
TextButton(
onPressed: () async {
await telephony.openDialer("xxxxxxxxxxxx");
},
child: Text('Open Dialer'))
],
),
));
}
}


Suggestion: student-set living name / preferred name

Some students (for a variety of reasons) do not go by the legal name on their Brown account. It would be nice if they could set their preferred name so that we use the name they want to go by when calling them for hours.

Thanks for submitting this issue!
Students are able to edit the name field if they want to when joining the queue. Their preference isn't stored in the database, but it's possible right now. Does that seem to be the answer to the problem?

Yep. Not sure how that appears to students - just something I thought worth bringing up after two cases of people saying 'call me ____' in as many days.

Here's what it looks like:

But I agree that there seems to be a usability issue that makes it unclear to students that they can edit their name. I'll definitely think about it!


Specify GitLab branch

Is there an existing issue or pull request for this?

[X] I have searched the existing issues and pull requests

Feature description
The API calls used for GitLab currently do not specify the branch - which means the default branch is always used.  There should be a way (configuration setting and/or environment variable) which specifies which branch / reference should be used when looking for commits.  This would allow GitLab metadata to be retrieved correctly if a changelog was being generated for a branch other than the GitLab default branch.
Desired solution
Provide a configuration option such as:
[remote.gitlab]
branch = "other_branch"

or an environment variable such as: GITLAB_BRANCH which, when specified, sets the ref_name parameter of the GitLab Commits API call which will return the commits specific to that branch instead of the default branch.
Alternatives considered
I don't believe there are any other viable alternatives, other than limiting GitLab-integrated changelog generation to only the default branch.
Additional context
Unlike the commits API call, the Merge Request list API call returns all project MRs regardless of branch.  Thanks!

Thanks for reporting! I agree that we should be able to specify the branch for the request. I'll look into this after the release.


MM-25562 Fix dismiss reaction list crash

Summary
Applies to both iOS and Android. Fixes the Relaunch Alert and the crash when swiping the reaction list from the top to half way.
Note: Also dismissing all the modals when tapping on the Alert Relaunch option. <- no need to test this
Ticket Link
https://mattermost.atlassian.net/browse/MM-25562

Failed to check ref. @mattermost/core-build-engineers have been notified. Error:
no match found for this ref

@metanerd @jaydeland ☝

@enahum can you try without the slash in the branch name please? We already get pinged by the @mattermost/core-build-engineers jfyi.

@metanerd we've built previous branches with a slash before.. having it without a slash makes it so that I need to close this PR and open a new one... so I rather not

Ok, issue is rate limit. I am working on it currently.

Failed to check ref. @mattermost/core-build-engineers have been notified. Error:
no match found for this ref

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/6341
https://circleci.com/gh/mattermost/mattermost-mobile/6339

Artifact links:
https://6341-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa
https://6339-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk

/cherry-pick release-1.32


Can't deploy?

I'm not sure what I'm doing wrong.
Recently (over the past week) I've been getting an error deploying to the gcloud:

I've tried several ways of fixing it, even the firestore specific configuration and can't get it to deploy with ext-grpc.
I'm sure this worked a month or so ago because I was deploying; it is a recent change to gcloud.
I'll also note, when you install this program you get a warning

It seems pretty silly for demo code to be outdated. Is there someplace else I should be looking for PHP examples? Is PHP getting unsupported?

Seems to be working now. I assume there was some issue with the gcloud machine bootup :/
Still has the PHP 7.3 warning. Would be nice it the demo code wasn't complaining about unsupported code.

Hi @cdcline! We have a runtime out for PHP 8.1, so if you'd like, you can change runtime: php73 to runtime: php81 in your app.yaml to remove the warning.

@bshaffer I'm still having issues:

are you expecting me to do composer stuff too?
I was initially able to just clone it into the Google Shell and gcloud deploy but now I get all these errors.

This repo hasn't been updated for some time, so it's not compatible with PHP 8.1 in it's current state. Sorry for the incorrect advice. We would need to update the dependencies and code for it to run on PHP 8.1


Could you please provide the file of svc_preprocess_ids.py?

@Robinatp i need time to rewrite it, https://github.com/PlayVoice/VI-Speaker

Thank you so much.


Terminal Package Generates Unnecessary 403 Requests When User Lacks Required Policy

Issue Description:
Overview:
The Shel.Neos.Terminal package in Neos currently generates a 403 Forbidden error in the backend when a user lacks the necessary policy. Given that the Terminal button is hidden in such cases, it seems redundant and counterintuitive for the package to trigger this error.
Expected Behavior:
Ideally, when a user does not have the required policy for the Terminal, the package should prevent the generation of a 403 Forbidden request, especially considering that the Terminal button is already hidden in such scenarios.
Steps to Reproduce:

Create a user without the required policy for the Terminal (e.g. Editor).
Log in to the Neos backend.
Observe the network requests, where a 403 Forbidden request is generated by the Shel.Neos.Terminal package.

Actual Result:
The Shel.Neos.Terminal package triggers a 403 Forbidden request even though the Terminal button is appropriately hidden when the user lacks the required policy.

Expected Result:
The package should suppress the generation of a 403 Forbidden request when the user does not have the necessary policy, given that the Terminal button is hidden in such cases.
Additional Information:
Neos version: 7.3
Terminal package version: 1.1

Could resolve one part of the issue, but did not know before that the request is used to know if someone has permissions.
Need to think about it :D

I am not sure if you like it, but I now have a new command that has permissions for everybody to check if the user has available commands and that returns true or false.
With that approach we save the errors in the console but have one extra request.


Thumbnailing

I had some problems when doing this one:

The date in the filename (created by multer) may differ from the one created at the db upload key.
As the thumb and original filename are identic, the client should just point it at /thumb/ if it wants thumbnails, adding a thumb key to db would be redundant.

I'm going to try and fix a few of the problems, and clean it up a bit before we merge it.

The probelem with gm is that it requires extra dependencies. I found this module, and it seem like it doesn't need external dependencies.

That seems good, I'll give it a try.

So it works, but there's a problem.
This is the thumbnail generated by lwip and this is the one created by gm
Maybe adjusting the width and height could help, but gm creates a more "natural one".

Maybe cropping could help?

I don't think so.
I don't know how exactly GraphicsMagick and Imagemagick do it, but they preserve the proportion of the image when resizing.

I believe it's because lwip uses the height if the width isn't specified,
resulting in square images if not both a calculated accordingly.
On 20 Mar 2015 23:49, "Under-sama"<EMAIL_ADDRESS>wrote:

I don't think so.
I don't know how exactly GraphicsMagick and Imagemagick do it, but they
preserve the proportion of the image when resizing.
—
Reply to this email directly or view it on GitHub
https://github.com/phikal/4jhan-server/pull/7#issuecomment-84174254.

I fixed the image ratio problem by using @enagorny's from EyalAr/lwip#93 solution to scaling with lwip. And I removed the naming by date. I didn't understand the point behind that anyway.

I didn't understand the point behind that anyway.

It's because mostly IBs use UNIX timestamps on naming, but that's not really relevant.


Correcting the Samples

Removed test libraries for java in local folder.
Renaming powershell output binding functions to correct ones
Correcting nuget package version for javscript

a nit pick is there that needed to be fixed otherwise LGTM


Research python concurrency

How do we run the fastapi server to be truely concurrent to the simulation. Is a thread sufficient, do we need to start it via Popen of multiprocessing?

should the whole class RedisDocker run as a seperate process or only the server thread (or soon to be process) within that class?

only the server itself will be started as a seperate process. The API server will also be seperated from the energy system interface. The ESI only ought to communicate with the API Server with GET and PUT requests.The new API server should be located in vessim/sil


Fix class emit in converted loop body

Fixes class emit in a converted loop body when the class contains TypeScript syntax (i.e., fields).
Fixes #31330

@DanielRosenwasser: Once @sandersn has reviewed, should we cherry-pick this for 3.8?

No, it's several versions old and not worth the risk of a late cherry-pick at this point.


Add GHA workflow for Sphinx previews

This adds a basic workflow to build Sphinx docs and push them to a dedicated preview branch on local PRs. This should make it easier to review proposed changes.

@rabernat any idea why the build CI is failing here? I only added a workflow file.

@rabernat any idea why the build CI is failing here? I only added a workflow file.

Never mind, assuming this is because the CI is using the older setup-minconda that the refactor updates.

Never mind, assuming this is because the CI is using the older setup-minconda that the refactor updates.


book: Use glib Priority enum

Update to work with gtk-rs-core commit f32fd06
"glib: Move priority constants to associated ones"
https://github.com/gtk-rs/gtk-rs-core/commit/f32fd0608a7066366eef9e8aca63ba01752adf1c
I was following along with the book and these two examples wouldn't compile with Cargo until I made this change; I tested in a new Cargo package and it works as expected from what it says on the chapter. using Priority::default() also works and returns the same value, but seemed like a less clear way of writing it.

Update to work with gtk-rs-core commit f32fd06 "glib: Move priority constants to associated ones"
gtk-rs/gtk-rs-core@f32fd06
I was following along with the book and these two examples wouldn't compile with Cargo until I made this change; I tested in a new Cargo package and it works as expected from what it says on the chapter. using Priority::default() also works and returns the same value, but seemed like a less clear way of writing it.

CI fails with your changes. Have you tried building the listings locally? You might have to update the lockfile with "cargo update"

Update to work with gtk-rs-core commit f32fd06 "glib: Move priority constants to associated ones"
gtk-rs/gtk-rs-core@f32fd06
I was following along with the book and these two examples wouldn't compile with Cargo until I made this change; I tested in a new Cargo package and it works as expected from what it says on the chapter. using Priority::default() also works and returns the same value, but seemed like a less clear way of writing it.

CI fails with your changes. Have you tried building the listings locally? You might have to update the lockfile with "cargo update"

That is expected as those changes are not part of any stable release yet.

Then let's wait until they are :)

I am able to build in a new package with the two changed main files, using the cargo toml and lock files that result from following along with the book.
I haven't been able to build listings locally with or without the changes because it requires "gtk4 => 4.7" and my OS is on GNOME 42.5 which doesn't seem to want to upgrade past 4.6.6, so I can't recreate that test sorry.

It's fine. I'll just test it myself after the next release.


Avoid explicit resolve/reject functions in some cases in Promise polyfill

Avoid the creation of explicit resolve/reject functions for some basic cases in the Promise polyfill, to reduce memory usage. Remove the .prototype object of resolve/reject functions as unnecessary (they are not constructable). Also reuse queued reactions as Promise jobs which works in the polyfill (but not necessarily in a native implementation).

[x] Optimizations
[ ] Add Promise.prototype.then() override test case
[ ] Run tests
[ ] Run memory test comparison to master

TestMasterThis pull

test-promise-instance-resolve-reject.js134.1 MB113.1 MB

test-promise-pending.js35.48 MB35.54 MB

test-promise-rejected.js17.51 MB17.48 MB

test-promise-resolved.js17.40 MB17.46 MB

test-promise-then-chain.js1.93 MB1.02 MB


Theme Submission: gatsby-theme-intro

Submitting a new Gatsby theme - personal website for developers.
On-line demo is available at:
https://weeby.studio/intro/preview/
Repository:
https://github.com/wkocjan/gatsby-theme-intro

@wkocjan Hi, really nice theme! I just wanted to check, is the demo link your personal site or the theme demo? The demo link must link to a demo of the theme as it would be if you just installed it.

@wkocjan Hi, really nice theme! I just wanted to check, is the demo link your personal site or the theme demo? The demo link must link to a demo of the theme as it would be if you just installed it.

Thanks @JugglerX
The demo link is a theme demo, not my personal page. A link contains sample data which is copied when initially running gatsby develop command.


[Bug] CI workflow shows tput: No value for $TERM and no -T specified

Is there an existing issue for this?

[X] I have searched the existing issues

What projects are you seeing the problem on?
No response
Current behavior
This issue has been there for a while and is not new.
The issue is likely because of tput statements used in dev-env.sh to use bold and italic font.
bold=$(tput bold)
italic=$(tput sitm)
reset=$(tput sgr0)

orange=$(tput setaf 166)

Expected behavior
No response
Anything else?
No response
Commit ID
No response
Are you developing inside the dev container?

[X] I am using the dev container

Code of Conduct

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

From inside the local devcontainer:
vscode@684f605b152f:/workspaces/sage-monorepo$ echo $TERM
xterm-256color

This comment could be a solution.


Preparation or improvement of the guide for introduction to CoCalc environment and programming in Python (issue #987 related)

The goal is to prepare or improve a guide or report that is generated by the following code:
from dynpy.utilities.documents.guides import IntroToCocalcGuide
IntroToCocalcGuide()

Minor changes introduced. The code was checked with the following call:
from dynpy.utilities.documents.guides import IntroToCocalcGuide
IntroToCocalcGuide()


Testing loadable-components with Jest snapshots

Hi there,
Currently to use loadable with Jest and snapshots. Is there any strategies you could see to do this, running into lots of issues and there isn't a lot of Stack Overflow on this aha!
Thanks for your help

Oudated, I close it. Feel free to open a new issue.

The issue is outdated I mean. The method described is not. I recommend testing library to test them. All tests in the repository use it, you can inspire from it.

@gregberge sorry, I can't see any test in this repo... can you point me to them?

any solution ?
i'm trying to make it work for many hours
Thanks :)

Me too trying to test loadable imports for improving the unit test coverage.  But no test is covering the import part of the code. Is there any mock option available for 'loadable' so that all the imports done via loadable can be covered as in the code?

My test case code is given below;
jest.mock("loadable", () => () => ({}));
test("Should render correctly", () => {
render();
});

@benedict333
it work for you for regular loadable import with Jest ?
I cannot make it work
exemple with this :
import {NotificationsDisplay} from './components/general/NotificationsDisplay';
import {Index} from './components/routes/Index';
// this break my JEST
const {Gcu} = loadable(() => import('./components/routes/Gcu'), {fallback: <LoadingScreen /> })

did you add something to babel or webpack config ?

@benedict333 I am also trying to get coverage on the same lines that are in your screenshot.
My snapshot tests do not cover them. Did you manage to get coverage on the loadable import lines and if so, how?
Thanks in advance for your help


hyphenate those fancy looks

Coverage remained the same at 100.0% when pulling 1d3be08d83e236e3cf18fab2138d97f1dda39c57 on michaelbeil:patch-1 into e3a8694b3507764af46b8bed7f48f76363e4c58b on austinpray:master.

:100:

:gem:


Restricting the number of incoming requests to wiremock server

Throttling issue
How can we restrict the number of concurrent incoming requests to wiremock server? I tried using .containerThreads() but this doesn't seem to work. Is there any other possible way to implement this so the server doesn't break when there are too many requests triggered at the same time. Is there any method provided by wiremock to solve this? Also what is the maximum number of concurrent requests wiremock can handle?
WireMockServer wireMockServer = new WireMockServer(options().port(8087));
I am trying to implement throttling in the above line of code.

I think while .containerThreads() limits the size of the tread pool of the embedded Jetty server used by WireMock the server can still continue to accept HTTP connections but they are queued. You would probably want to limit the size of the queue. Apart form that there may be OS level settings related to TCP/IP that may need to be tweaked to prevent the OS form accepting TCP connections and queuing them.
If you don't want to mess around with TCP/IP settings of the OS and are looking to throttle the max requests per second that Jety should process then you might want to take a look at the DOS Servlet Filter - https://www.eclipse.org/jetty/javadoc/9.4.20.v20190813/org/eclipse/jetty/servlets/DoSFilter.html
However I am not sure how to configure a custom servlet filter on the embedded Jetty server when initializing WireMock.

Closing this due to age and inactivity.


npm start fails because .eslintrc is missing; ENOENT no such file or directory

What should happen?
Following the introduction for running the foundation-sites locally, I hit an error.
`Error: ENOENT: no such file or directory, open '/home/myself/loopy/node_modules/foundation-sites/.eslintrc'
What happens instead?
No error should be present.
Possible Solution

echo {} > node_modules/foundation-sites/.eslintrc
Test Case and/or Steps to Reproduce (for bugs)

Test Case: no test case; it's easy to reproduce.

How to reproduce:

npm install foundation-sites
cd node-modules/foundation-sites
npm install
npm start -- this is where the error happens

Context

Running the foundation-sites documentation site, locally.
Your Environment

Foundation version(s) used: latest install from npm at this date
Browser(s) name and version(s): n/a
Device, Operating System and version: Ubuntu 16.04
Link to your project: n/a

Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] There are no other issues similar to this one.
[x] The issue title and template are correctly filled.

npm install foundation-sites
cd node-modules/foundation-sites
npm install
npm start -- this is where the error happens

Why would you want to do that? npm and yarn already install all the dependencies.
<EMAIL_ADDRESS>The file is not published. Please use the Git remote URL or zurb/foundation-sites if you want to develop in a Node module - but this is discouraged.

You might want to use https://github.com/zurb/foundation-zurb-template or the Foundation CLI.

That is definitely my oopsie. I misunderstood the documentation; it felt weird cd-ing to the node_modules and doing something in there. I misread the readme.md at the root of this project. My mistake


Error 208 on createAccount

Log:
 ChatClient: createAccount: Test, test
 LOG  register: fail {"code": 208, "description": "Registration failed."}

Code:
     ChatClient.getInstance()
      .createAccount(username, password)
      .then(() => {
        console.log('register: success');
        setWarnText('register: success');
      })
      .catch(reason => {
        console.log('register: fail', reason);
        setWarnText('register: fail' + reason);
      });

the reason is app is not open registration by default (can create account with app token via REST api), now I update it to open registration, now you can try the createAccount API again