Umbrella issue about prompt

[x] Customizable prompt2 #35

[x] lexer status in prompt2

[x] allow newlines in prompt #15
[x] Use % instead of \ #15

Because \ is treated as a special character in many context, especially go-flags needs to escape \.

[x] Showing system variables in prompt

Related issue in original spanner-cli

https://github.com/cloudspannerecosystem/spanner-cli/issues/183
https://github.com/cloudspannerecosystem/spanner-cli/issues/186
https://github.com/cloudspannerecosystem/spanner-cli/issues/187
https://github.com/cloudspannerecosystem/spanner-cli/issues/188

Close this issue because I have finished planned features.


Load JSGF grammar from file?

I quickly went through the code and I couldn't find any class or function that loads
a JSGF grammar from a file.
Here's the use case: I already have a JSGF grammar, but my speech recognition component does not
support grammars, e.g., deep speech. Thus, my SR outputs strings --- now I'd like to use pyjsgf
to match the output strings against my existing grammar that already exists in a file.
Is there a way of doing this?

Unfortunately no, there isn't a way to do this yet. I ran into a similar problem today actually. The way to do it would be to write a JSGF parser to read grammar files and populate Grammar objects with rules. I'll see what I can do about this.

That would be awesome! Thanks

Okay, I have a work in progress parser up on the feat/parser branch. It's actually mostly done, I'm just having trouble getting expansions parsed properly.
By the way, I've switched to using setuptools in setup.py so that the pyparsing package can be installed automatically. You might need to remove the old distutils version of pyjsgf.

Is there something I do wrong or is the parser not able to parse rules yet? Always getting an issue.

No it doesn't work yet unfortunately. I'm going to get it working soon, it's been an issue for way too long.

Parsers for grammar files, grammar strings, individual rules and rule expansions are complete and are in the develop branch. Sorry this took so long @warp1337 @embie27! I hope you find it useful. Let me know if you have any issues.
An example for using parse_grammar_string:
from jsgf import parse_grammar_string
parse_grammar_string("#JSGF V1.0 UTF-8 en;"
                     "grammar example;"
                     "public <greet> = hello world;")

A Grammar object with the name example containing the greet rule will be returned. You can use parse_grammar_file(path) to parse grammar files.
I've tried to stick to the JSGF spec while implementing this. C++ style comments (/* ... */ and // ...) are supported. I'll have some examples up soon and hopefully a readthedocs project.
I'll release this in v1.4.0 in the next week, I want to add a few more things first, such as an import resolver and make the character set attribute in the grammar header useful.

Sorry I took almost 3 weeks to release v1.4.0. I'll make separate issues for the import resolver and grammar header attributes for later releases.
I wanted to say here that the parser has limitations with long sequences and alternative sets. If a sequence is very long or an alternative set has a lot of alternatives (between 50-100 expansions/alternatives from my testing), the parser will crash with a RecursionError. This is a consequence of how sequences and alternative sets are parsed and how pyparsing does the recursive descent. The workaround is to split problematic sequences and alternative sets into multiple rules and use rule references instead. I will also make a note about this in the documentation when I get around to writing it.
I think this issue is resolved now.


Setup Tests

Add basics tests to for:

[x] Images API
[x] Context API
[x] Session API

Fixed in 042c445


Implement buffer reordering

Closes #118
TODO:

[x] Cleanup unnecessary changes
[x] Documentation

@mengelbrecht Ready to review at your leisure 😄

@dkaszews thanks for your work. A very nice feature 🙂. The code looks good.


Fix isort

@anapaulagomes I hope you don't mind me spamming with those things. :)

No worries. :) Is it ready to be reviewed? @amureki

@anapaulagomes yeah. Pydocstyle seems to add quite some new checks, so it fails in plenty of places. We'll check it separately later.


fix: error if player doesn't have selected profile

Description
Fixes error User not found in selected profile. This is probably due to a declined co-op invite. if player doesn't have selected profile (hasn't joined after Hypixel's change from last_save to selected)
Example
https://sky.shiiyu.moe/stats/zCris1xV2

LGTM, never tested


[Feat] - Migration of Docs from Old Site

Description:

Migration of Files with respect to https://docs.google.com/document/d/1wYOLvhuWt50M0sMPGN2TlA5JMICzv5BSczPmV4GsilQ/edit

I found that you missing WasmEdge runner here. https://wasmedge.org/book/en/cli/wasmedge_tool.html

@alabulei1 Made the changes and Resolved Conflicts as well :)

@adithyaakrishna  Please check it out again. Thanks.

LGTM! Thanks for your contribution.


Support different types in simple_set

Following on from #7 ... the idea to have a set of ints, doubles or any other type would certainly be interesting.
Do you have any idea how to achieve that? I'm looking at this article for some ideas:
https://levelup.gitconnected.com/using-templates-and-generics-in-c-968da223154d
and wonder if the technique could be applied to the the C code.
Code that has char *key would need adaptation, which occurs in a few places.

The issue is in the hashing function needing to take the int/float/etc in and hashing it. The char * key values would also need to be updated, but that would be much simpler than changing the hashing function.
Another option is to turn the float/int/etc into a c-string and use that directly. Just an idea.

Yes, thanks for the suggestion. I'm going to go with that idea for the hashing function, and see about a small API to take care of the conversion.


fix(live): ensure non-class service id can be used

Q
A

Bug fix?
yes

New feature?
no

Tickets
n/a

License
MIT

Without this fix, you cannot register live components with a non-class service id.

Thanks Kevin!


How can I tell who is NOT using Teams?

[Enter feedback here]

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

ID: fd6683c0-a685-d279-538f-f0df86113911
Version Independent ID: efe3afd1-f9c2-9db4-509f-8caaad6f0be0
Content: Monitor usage and feedback in Microsoft Teams - Microsoft Teams
Content Source: Teams/get-started-with-teams-monitor-usage-and-feedback.md
Service: msteams
GitHub Login: @cichur
Microsoft Alias: v-cichur

@cfizz34 Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Omnipay 3.x support

It's about time this was supported.
The 2.x release will go into its own 2.x branch. The master branch will them move to Omnipay 3.x (3.0-beta.1 at this time).

I'm sure I've closed this one three times now. Closing again - 3.x is supported.


Fio no log hist

do not merge yet, in process of running fio CI tests, but tested this on a small scale
This PR turns off histogram logging by default, while allowing the user to turn it back on for later development
This shrinks the amount of log data being fetched by fio-client pod, hopefully reducing chance of fio inflate error

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1cc1216). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #666   +/-   ##
=========================================
  Coverage          ?   53.48%
=========================================
  Files             ?        8
  Lines             ?      316
  Branches          ?        0
=========================================
  Hits              ?      169
  Misses            ?      147
  Partials          ?        0

Flag
Coverage Δ

gha
53.48% <0.00%> (?)

python-3.9
53.48% <0.00%> (?)

unit
53.48% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_fiod
Fail
3
01:28:22

@bengland2 something weird happened during CI tests. fio client took a lot of time
[root@ip-172-31-40-110 ~]# oc logs  fio-client-5aec2453-v85vv  | wc -l
918162

yeah I saw this running on my laptop, the final tests/test_crs/fiod_hostpath.yaml took almost a half-hour, so the CI timed it out:
11:39:02 + timeout -k 1200s 1200s bash -c 'wait_for_benchmark fiod-hostpath-benchmark Complete'
11:39:02 + echo 'Timeout exceeded for: fiod-hostpath-benchmark'
11:39:02 Timeout exceeded for: fiod-hostpath-benchmark

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_fiod
Pass
0
00:12:37


PR: Unix interrupt

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Add an interrupt signal to make the kernel process the messages while processing.
pendent of https://github.com/spyder-ide/spyder-kernels/pull/117
Issue(s) Resolved

Fixes #9403
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

Hello @impact27! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/app/tests/test_mainwindow.py:

Line 1003:13: E225 missing whitespace around operator

In the file spyder/plugins/editor/plugin.py:

Line 616:20: E128 continuation line under-indented for visual indent
Line 1256:45: E128 continuation line under-indented for visual indent

In the file spyder/plugins/editor/widgets/editor.py:

Line 691:39: E127 continuation line over-indented for visual indent
Line 692:39: E127 continuation line over-indented for visual indent
Line 693:39: E127 continuation line over-indented for visual indent

In the file spyder/plugins/ipythonconsole/comms/tests/test_console_kernel.py:

Line 77:1: E305 expected 2 blank lines after class or function definition, found 1
Line 79:1: W391 blank line at end of file

In the file spyder/utils/icon_manager.py:

Line 148:80: E501 line too long (95 > 79 characters)
Line 149:80: E501 line too long (117 > 79 characters)

As an alternative that would be supported on window, we could modify ipykernel.kernelbase._input_request:
It is now getting data with:
while True:
    try:
        ident, reply = self.session.recv(self.stdin_socket, 0)
    except Exception:
        self.log.warning("Invalid Message:", exc_info=True)
    except KeyboardInterrupt:
        # re-raise KeyboardInterrupt, to truncate traceback
        raise KeyboardInterrupt
    else:
        break

Instead we could do:
reply = None
while reply is None:
    try:
        ident, reply = self.session.recv(self.stdin_socket, zmq.NOBLOCK)
        self.do_one_iteration()
    except Exception:
        self.log.warning("Invalid Message:", exc_info=True)
    except KeyboardInterrupt:
        # re-raise KeyboardInterrupt, to truncate traceback
        raise KeyboardInterrupt

So do_one_iteration can be called while waiting for input. What do you think?

I added the option to start debugging in a running kernel.
For example let's say the code is stuck on a mutex / in an infinite loop, but you don't know where, and you want to start debugging from there, you just need to press the debug button and the code will stop there.

Except if you have windows because windows doesn't support signals.

Retriggering as the corresponding spyder-kernels PR is updated

Retrigger tests after updating spyder-kernels PR

Superseded by #10438


Add Stadler Form Eva humidifier

I opted to set the fan speed to a dropdown as the speeds are 1-5. I couldn't find a way to use a slider that did not start with 0.


Request languageServerProvider-any.request failed with message

Issue Type: Bug
This happens every time one of us saves a file when working together in Live Share.
Request languageServerProvider-any.request failed with message: args is not iterable (cannot read property undefined)

It's hard to tell what's causing this or how to reproduce it. Another coworker sees the same message pop up at the same time, but the host doesn't.

Not fixed

We’ve made improvements in this area since this issue was filed. We believe this may have improved your experience and are closing this issue. If this issue still persists for you, please comment on the issue and let us know. Thank you.


TODO: Client Interaction/Colliders

Problem: Client should not be throwing a exception for something not being implemented. Instead should write to the console / log

Is this still a problem?

All the references of notimplementedexception have been blown away so this is solved


Uncaught (in promise) ParseJSONFailed: Fail to parse the received json document. Has MSR updated its json structure? InnerEx: TypeError: Cannot read properties of null (reading 'promotionType')

Works on some of my accounts but a few i'm getting this error

Please update to the latest push.

issue fixed in the latest release


[2.2] Test HTTPS enforced via HttpsToken WS-SecurityPolicy does not work

related #628

Merged manually 57c363e8d10c4f02035be56bb72b65994a9b70d6


Event Search Expression update

We need to modify the SearchExpression under EventSearchSettings to allow users to search both by event type and by event name.
FYI @LindsayThomas

[ ] Event Calendar
[ ] Events Locator

@LindsayThomas  Updated configEventLocator.js for State Gov.

@lindsayking did we test if this is affected by the AGOL security setting issue we had for the other search expression?

@LindsayThomas
Tested with the "Allow only SQL queries" security setting in the org both checked and unchecked.  Search results were not affected by the org setting.

LG commit - https://github.com/Esri/configurable-locator/commit/6e4bee76f1211bc561cfd87a680c6ed76c557201


Question: enum name generation

Hi,
I'm not sure, maybe it's an issue with the swagger generator I'm using, but currently when I have an enum inside of an element, the definition is directly included in the element and not as an external definition.
Then when I generate the typescript code, the enum is prefixed with the element name.
For example enum "ContactType" in my Contact element is named "ContactContactType".
I suppose this prevent name collision but is there any way to disable this ?
More over when I'm using the same ContactType definition in another element A, I have an other definition named "AContactType".
Do you think it's an issue with the swagger definition tool I'm using ? or could this be handled in nSwag ?
Thanks a lot.

The problem is that because the enum is not referenced from the "definitions" section, no type name can be inferred... and thus the parent class name + property name is used... When using NSwag as assembly, you can provide an own ITypeNameGenerator (ie implement own class which inherits from DefaultTypeNameGenerator) and selectively change type names...

Hi,
Ok thanks I'll try to have a look on this. I'll first try to check if it's possible to change the generated swagger definition but I'm not sure about it.
Thanks.

In the best case, every special type (object, enums) should be referenced in the "definitions" section, so that the type name is defined... however there are some cases where this is not possible (https://github.com/NSwag/NSwag/issues/328)

I have submitted a PR to the project which generate the swagger definition. It's now working as I would expect. Thanks.

What project are you using? Swashbuckle?

I'm using Happi-swagger (https://github.com/glennjones/hapi-swagger)


Move common arguments to their own structure

Description of changes:
This is a pre-work needed for implementing #2440 as a subcommand. Today, the only other subcommand we have is assess and the way it takes arguments is rather broken as explained in #1831.
For the playback sub-command, I expect that we will need at least CargoArgs and CommonArgs to control the build process and to control the process verbosity.
We would probably benefit from properly isolating the build arguments, and not allowing arguments when subcommands are provided, but that will be a problem for another day since it will require changes to assess logic and Kani sesssion. I am hoping this will be enough for a first working version of the playback subcommand (famous last words... 🙂).
Resolved issues:
Related to #2440
Related RFC:
Call-outs:
I created an args/ folder for all argument related logic because I believe it will be easier to maintain and it creates a better separation of responsibility and cleaner dependency tree (i.e.: args module no longer depends on assess module).
Update: I'll probably need to create one more PR that enables args_conflicts_with_subcommands to fix the issue with the argument position.
Testing:

How is this change tested?

Is this a refactor change?

Checklist

[ ] Each commit message has a non-empty body, explaining why the change was made
[ ] Methods or procedures are documented
[ ] Regression or unit tests are included, or existing tests cover the modified code
[ ] My PR is restricted to a single feature or bugfix

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

Question: If I understand this right, we plan to have two subcommands (assess and concrete-playback) and one default one for verify after this PR, is that correct? That makes sense. Does this also mean that running concrete-playback, will block the verification flow and just print the unit test?

I have just one comment, which is about the regression test that was changed. I didn't understand the motivation behind removing --dry-run. Is it because you moved it into the ValidateArgs and so dry-run doesn't work now as expected, but it will with the upcoming PR?

Great question! The test at hand had two errors:

Conflicting unwinding arguments (which is what we want to test).
Use of --dry-run which has been removed already.

Before this change, we checked # 1 first, then we checked # 2. With this change, the order has changed, and the test was failing now because of # 2. So I removed the --dry-run argument so it still tests # 1.

Question 1: This change is a simple refactor to put all the args into two structures, one for common ones (Which will be used by the subcommands) and one structure for the verification related arguments, am i understanding that correct?

Yes! I was thinking that we could even add a subcommand verify, which is the default one when no subcommand is given.

Question 2: The plan to have two subcommands (assess and concrete-playback) and one default one for verify after this PR, is that correct? That makes sense. Does this also mean that running concrete-playback, will block the verification flow and just print the unit test after the coming PR?

Oh, I guess you already answered the first question here. 😉 So the testcase generation won't change, and it will still be part of the "verify" subcommand. The new playback subcommand will compile and run this testcase.

Question 3: I didn't understand the motivation behind removing --dry-run in the test case. Is it because you moved it into the ValidateArgs and so dry-run doesn't work now as expected, but it will with the upcoming PR?

Per my previous answer, --dry-run doesn't work today. We should really just remove it completely, but I didn't want to make that change as part of this PR.


feat: pub use

This change is

@spapinistarkware @orizi this PR is a following up of my previous PR to add pub visibility to items. Please have a review.


How can i use lib in my react-native tv app

Hello
I have tried to install react-natve-media-console@next and tested in my tv app. but controls not working in my tv app.
here is installed version in my app
   "react-native-media-console": "^2.0.2-0",
please help

Hello! Why @next? Specifying @next will install the latest release candidate. Currently there is no release candidate, the latest version is 2.0.6.
npm install react-native-media-console
or
yarn add react-native-media-console


re-implement cloud aware function for e2e test

Which issue this PR addresses:
Fixes e2e test failures in USGovCloud environments. #15119913

What this PR does / why we need it:
Fixes a previously hard-coded string that was causing test failures.

Test plan for issue:
Run test in USGovCloud environment. Test now re-implements cloud aware logic from controller so confidence is high that it will work correctly.

Is there any documentation that needs to be updated for this PR?
N/A

/azp run ci


Update CI to tests multiple node versions, use node 16 locally

Run CI against node 12 / 14 / 16. Following a similar pattern to quilt and loom
Use node 16.13.0 (LTS version) locally

The node versions we test against in CI are: 12.14 - our lowest target for packages, and 14.x and 16.x - whatever the latest releases for these branches are - this matches what quilt does.
Locally we run against 16.13 - the first LTS release in the v16 line.
We attempted part of this previously, but bundled it with a bunch of stylelint updates that needed to be reverted in #298

Updated the branch protection rules so now they require the three new test checks (Test (12.14) etc) instead of the old singular "Test"


Update example project to latest pod version

Ran a pod install on the Example project

Looks good to me - merge in (/rebase onto) master and if the tests pass merge away :)


feat: support "Response.error()" for mocking network errors

Closes #413
Prerequisite for https://github.com/mswjs/msw/pull/1730

Released: v0.25.0 🎉
This has been released in v0.25.0!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


Custom Matcher (oldMatcher) does not work on AJAX Remote Data

Hi -- thanks for your work on this.  I enjoy using it.
It appears the matcher is never called when using select2 with remote ajax data.  See the JSBin located here, where I took the remote data example from the examples page and wrapped it using documented matcher backwards compatibility technique.  Note: For simplicity, I trimmed the formatRepo method simply to display the repository name rather than all the other stuff.
Entering in the search text "ght", I would expect that a search done in this manner would limit the results to only repositories beginning with "ght". However, the first repository returned is AerialX/PSL1GHT, and others that do not begin with "ght".
Windows 7
Google Chrome 48
Select2 4.0.1
JQuery 2.2.1
Hope that helps!

Here's an additional demo.

While typing anything I would expect it to cap the suggestions that only matches our term. And also matching results should come first in the order. Instead it shows every thing in that is in the data-result array.

i have the same problem.

any updates on this?

I have the same problem too. Did you solve this?

no basically you are fucked.

For the moment, matcher will only support local data sources.  This has been clarified in the documentation: https://select2.org/searching#customizing-how-results-are-matched

I have same problem in angular 5 , i solved my problem with this code
<select2 (valueChanged)="Changed($event,KeyPress())"></select2>
  KeyPress() {
    var keynum = 0;
    var e: any = window.event;

    if (e) { // IE
      keynum = e.keyCode;
    } else if (e.which) { // Netscape/Firefox/Opera
      keynum = e.which;
    }

    return keynum
  }

  Changed(e: any, keyNum: number) {

    //if option not choose with enter key or e is undefined
    if ((keyNum && keyNum != 13) && (!e.data || !e.data[0] || !e.data[0].selected)) return

    //data is choosed
    if (e.data.length > 0) {
        //do something

    }
  }


i18n: multiple-folders create invalid image references

Describe the bug
when using:
i18n:
  structure: multiple_files
  locales: [ de, en]

- name: "portraits"
    folder: "src/pages/portraits"
    media_folder: ''
    public_folder: ''
    path: '{{slug}}/index'
    i18n: true
    fields:
      - {label: "title", name: "title", i18n: duplicate, widget: "string"}
      - {label: "Profile Picture", name: "profilepicture", widget: image, i18n: duplicate}

A new (localized) item in a collection with an image widget results in the following folder structure:
/src/pages/portraits/de/lord-voldemort/index.md
/src/pages/portraits/en/lord-voldemort/index.md
/src/pages/portraits/lord-voldemort/image.jpg
both of the mardown files look something like that (after creating a new item in the UI and uploading an image as profilepicture)
---
title: Lord Voldemort
profilepicture: image.jpg
---
## Some markdown

My problem is the invalid reference of the profile picture, because this markdown and the described folder structure breaks the Gatsby GraphQL because now the profilepicture is not a "image" object node in the graph but some string. If i change profilepicture to "../../lord-voldemort/image.jpg" in the MD file and restart gatsby, everything is ok again.
To Reproduce
Create a Collection like described and create an entry via CMS UI.
Expected behavior
profilepicture with a proper reference to the image file like so:
---
title: Lord Voldemort
profilepicture: ../../lord-voldemort/image.jpg
---
## Some markdown

Screenshots
Applicable Versions:
"gatsby-plugin-netlify-cms": "^4.7.0",
"netlify-cms-app": "^2.14.8",

Git provider: GitHub

CMS configuration
as seen above.

Is there any update on this issue? Are there any workarounds or fixes available since the last discussion?


Client Fatal error after pairing

Description of the issue
Addon crashes after I paired my whatsapp.
Maybe because of outdated Whatsapp-web.js?
Bumped Verison in #178
Addon version
2.3.9
Platform
Core 2024.9.2
Supervisor 2024.09.1
Operating System 13.1
Logs of the issue (if applicable)
9/14/2024, 14:29:38 [error]: 	[Whatsapp] Client fatal error
Error: Evaluation failed: TypeError: Cannot read properties of undefined (reading 'default')
    at __puppeteer_evaluation_script__:5:95
Error: Evaluation failed: TypeError: Cannot read properties of undefined (reading 'default')
  at __puppeteer_evaluation_script__:5:95
  at ExecutionContext._evaluateInternal (/app/node_modules/puppeteer/lib/cjs/puppeteer/common/ExecutionContext.js:221:19)
  at async ExecutionContext.evaluate (/app/node_modules/puppeteer/lib/cjs/puppeteer/common/ExecutionContext.js:110:16)
  at async Client.initialize (/app/node_modules/whatsapp-web.js/src/Client.js:323:9)
[14:29:38] WARNING: Halt add-on with exit code 1
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service base-addon-timezone: stopping
s6-rc: info: service base-addon-log-level: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service base-addon-timezone successfully stopped
s6-rc: info: service base-addon-log-level successfully stopped
s6-rc: info: service base-addon-banner: stopping
s6-rc: info: service base-addon-banner successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

Hi! Check if it has been fixed in the latest version.

The build of 2.4.0 failed

Fixed in v2.4.1


[lodepng] Update to commit e34ac0@2020-03-15

I'm currently preparing a cmake integrated bgfx port (requested here #1303) which requires a newer lodepng version. Additionally I added proper cmake import configs, because otherwise the debug library build cannot be found (the release binary always takes precedence).

/azp run

/azp run


Address feedback from #17510

Added/changed comments as mentioned.
Improved the ugly resize_and_overwrite hack into the STL.
Add Write functions for xterm's window API.
The only reason we needed a move operator for VtIo::Writer
is because we used it in a ternary in CONSOLE_INFORMATION.
Ternaries are like if branches with hidden move assignments.
Instead, we simply construct each Writer in place.
No ternary = No move = No problems in life.
The best benefit of this is that this makes calling GetVtWriter
a hundred times cheaper.

Otherwise, I still need to extend a few tests in VtIoTests,
but I'm planning to do that later.

leftovers from #17510 are:

⚠ Intentionally unresolved changes/quirks:

To force a delayed EOL wrap to wrap, WriteCharsLegacy emits a
\r\n if necessary. This breaks text reflow on window resize.
We cannot emit  \r the way readline does it, because this would
overwrite the first column in the next row with a whitespace.
The alternative is to read back the affected cell from the buffer
and emit that character and its attributes followed by a \r.
I chose to not do that, because buffer read-back is lossy (= UCS2).
Unless the window is resized, the difference is unnoticeable
and historically, conhost had no support for buffer reflow anyway.

If ENABLE_VIRTUAL_TERMINAL_PROCESSING is set while
DISABLE_NEWLINE_AUTO_RETURN is reset, we'll blindly replace all
LF with CRLF. This may hypothetically break DCS sequences, but it's
the only way to do this without parsing the given VT string and
thus the only way we can achieve passthrough mode in the future.

ENABLE_WRAP_AT_EOL_OUTPUT is translated to DECAWM.
Between Windows XP and Windows 11 21H2, ENABLE_WRAP_AT_EOL_OUTPUT
being reset would cause the cursor position to reset to wherever
a write started, if the write, including expanded control chars,
was less than 100 characters long. If it was longer than that,
the cursor position would end up in an effectively random position.
After lengthy research I believe that this is a bug introduced in
Windows XP and that the original intention was for this mode to be
equivalent to DECAWM. This is compounded by MSDN's description
(emphasis mine):

If this mode is disabled, the last character in the row is
overwritten with any subsequent characters.

⚠ Unresolved issues/quirks:

Focus/Unfocus events are injected into the output stream without
checking whether the VT output is currently in a ground state.
This may break whatever VT sequence is currently ongoing.
This is an existing issue.

VtIo::Writer::WriteInfos should properly verify the width of
each individual character.

Using SetConsoleActiveScreenBuffer destroys surrogate pairs
and extended (VT) attributes. It could be translated to VT pages
in the long term.

Similarly, ScrollConsoleScreenBuffer results in the same and
could be translated to DECCRA and DECFRA in the near term.
This is important because otherwise vim output may loose
its extended attributes during scrolling.

Reflowing a long line until it wraps results in the cooked read
prompt to be misaligned vertically.

SCREEN_INFORMATION::s_RemoveScreenBuffer should trigger a
buffer switch similar to SetConsoleActiveScreenBuffer.

Translation of COMMON_LVB_GRID_HORIZONTAL to SGR 53 was dropped
and may be reintroduced alongside UNDERSCORE = SGR 4.

Move the OSC 0 ; P t BEL sequence to WriteWindowTitle
and swap the BEL with the ST (ESC \).

PowerShell on Windows 10 ships with PSReadLine 2.0.0-beta2
which emits SGR 37/40 instead of 39/49. This results in black spaces when typing and there's no good way to fix that.

A test is missing that ensures that FillConsoleOutputCharacterW
results in a CSI n J during the PowerShell shim.

A test is missing that ensures that PtySignal::ClearBuffer
does not result in any VT being generated.

and #1765 too.

@mominshaikhdevs Could you either explain why you keep linking #1765 in a lot of places, or stop doing so? 2 words are insufficient for me to understand what you're trying to say. That issue hasn't been fixed either.


implement mock that accepts ETH and accepts/rejects ERC223 transfers

Why we do it
Due to Istanbul fork we cannot use SimpleExchange to test ETH transfer contract to contract. (we had event there that now crosses 2300 gas stipend of fallback function)
We need to reintroduce those e2e tests
What we do
We need a contract that will be deployed on dev (loclahost) env against which we can test ETH transfer. On top of that let's add ERC223 callback for testing ERC223 transfer for which we do not have coverage at all
Acceptance criteria

[ ] implements payable callback function
[ ] function above can be called via another contract (test!)
[ ] implements IERC223Callback
[ ] implements IERC223LegacyCallback (to test NEU which requires old callback ;/)
[ ] provides a mock method to accepts/reject erc223 transfers transfers in the callback function

Deployment

[ ] make it a part of deployment pipeline so it gets deployed only on dev.

Basically the conract should also Accept
Ether
Plus work with TransferAndSend method provided by ethertoken


Update SetDefaultSampler to revert to default sampler

Hey there!  I was looking through the issues for one that would get me into the code and found #197
I'm not 100% sure this is what is needed, but, it's fun to get in and see how it is built!

Thanks for the PR! We haven't agreed this should be the expected behavior yet but @jcd2 can provide feedback.


the trait bound &url::Url: IntoClientRequest is not satisfied

bug from conmmit 8f9bec21d7a0d1e095039c1ff9ecd7c532d9e74e
error[E0277]: the trait bound `&url::Url: IntoClientRequest` is not satisfied
   --> C:\Users\duong\.cargo\git\checkouts\librespot-06fda9f186b35c32\8f9bec2\core\src\dealer\mod.rs:412:66
    |
412 |     let (mut ws_tx, ws_rx) = tokio_tungstenite::client_async_tls(address, stream)
    |                              ----------------------------------- ^^^^^^^ the trait `IntoClientRequest` is not implemented for `&url::Url`
    |                              |
    |                              required by a bound introduced by this call
    |
    = help: the following other types implement trait `IntoClientRequest`:
              httparse::Request<'h, 'b>
              ClientRequestBuilder
              tokio_tungstenite::tungstenite::http::Request<()>
              tokio_tungstenite::tungstenite::http::Uri
              std::string::String
              &'a str
              &'a std::string::String
              &'a tokio_tungstenite::tungstenite::http::Uri
note: required by a bound in `client_async_tls`
   --> C:\Users\duong\.cargo\registry\src\index.crates.io-6f17d22bba15001f\tokio-tungstenite-0.23.0\src\tls.rs:161:8
    |
156 | pub async fn client_async_tls<R, S>(
    |              ---------------- required by a bound in this function
...
161 |     R: IntoClientRequest + Unpin,
    |        ^^^^^^^^^^^^^^^^^ required by this bound in `client_async_tls`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `librespot-core` (lib) due to 1 previous error
warning: build failed, waiting for other jobs to finish...

Weird, so it builds on the CI but not on your system?

You're building with tokio-tungstenite v0.23.0, which is not the version in Cargo.lock (v0.21.0).
If I build with a clean checkout of the dev branch which is locked to tokio-tungstenite v0.21.0 it builds fine.
I then made a clone of the repo/build directory, ran cargo update tokio-tungstenite and, sure enough, it's now broken. So, it seems that tokio-tungstenite v0.23.0 is broken.
timw in 🌐 ubuntu-server in librespot on  dev is 📦 v0.5.0-dev via 🦀 v1.78.0
❯ cargo build --release
   Compiling libc v0.2.153
   Compiling autocfg v1.1.0
...
   Compiling webpki v0.22.4
   Compiling librespot v0.5.0-dev (/home/timw/git/other/librespot)
    Finished `release` profile [optimized] target(s) in 3m 57s

vs
timw in 🌐 ubuntu-server in librespot-tmp on  dev is 📦 v0.5.0-dev via 🦀 v1.78.0
❯ cargo update tokio-tungstenite

    Updating crates.io index
    Updating rustls v0.22.2 -> v0.23.7 (latest: v0.23.9)
    Updating tokio-rustls v0.25.0 -> v0.26.0
    Updating tokio-tungstenite v0.21.0 -> v0.23.0
    Updating tungstenite v0.21.0 -> v0.23.0
note: pass `--verbose` to see 217 unchanged dependencies behind latest

timw in 🌐 ubuntu-server in librespot-tmp on  dev [!] is 📦 v0.5.0-dev via 🦀 v1.78.0 took 2s
❯

timw in 🌐 ubuntu-server in librespot-tmp on  dev [!] is 📦 v0.5.0-dev via 🦀 v1.78.0
❯ cargo build --release
  Downloaded tokio-rustls v0.26.0
  Downloaded tungstenite v0.23.0
  Downloaded tokio-tungstenite v0.23.0
  Downloaded rustls v0.23.7
  Downloaded 4 crates (505.1 KB) in 0.50s
   Compiling rustls v0.23.7
   Compiling rustls-webpki v0.102.2
   Compiling tungstenite v0.23.0
   Compiling tokio-rustls v0.26.0
   Compiling tokio-tungstenite v0.23.0
   Compiling librespot-core v0.5.0-dev (/home/timw/git/other/librespot-tmp/core)
error[E0277]: the trait bound `&url::Url: IntoClientRequest` is not satisfied
   --> core/src/dealer/mod.rs:412:66
    |
412 | ...ws_rx) = tokio_tungstenite::client_async_tls(address, stream)
    |             ----------------------------------- ^^^^^^^ the trait `IntoClientRequest` is not implemented for `&url::Url`
    |             |
    |             required by a bound introduced by this call
...

https://github.com/snapview/tokio-tungstenite/commit/a53454b16d9eff3fbaa7445541c7fe1c6ccad4d7 looks like the cause of the breakage.


New Word Builder

Before jumping into Pr first comment for assign.All Problem from Edabit. Link is mandatory to add
Problem statement: Create a function that builds a word from the scrambled letters contained in the first list. Use the second list to establish each position of the letters in the first list. Return a string from the unscrambled letters (that made-up the word).
Problem Link: https://edabit.com/challenge/R5F99DeuhqYxbGyMM

Before submitting a PR please Check some details.Check Edabit for problem task

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Add label for Problem
[ ]  In file there should be problem statement and link to probelm
example -

'''
    Probem Task : This program will add two numbers
    Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
'''
*/
def main():
    return a + b

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

@aayushi-droid Can I work on it?

yes sure

[✓]  Task assigned
@Matheusqz

Hello @aayushi-droid I would like to work on this.

Please assign this issue to me


Does the project have the ability to manipulate the attribute style?

It is mentioned in the paper that AttGAN could manipulate the attribute style, such as bangs' or glasses' style. Does the project hace the function or not?

@chenghan1995 No, this project doesn't include the style part.

Thank you a lot.


Added functionality to connect increase of stats to level, stat reset…

… on class change

Just a note. It's generally helpful to put a brief description in the start of the MR to explain what the changes are to help set context for the reviewer(s)


Support other inputs besides devices

Is your feature request related to a problem? Please describe.
The input of the video source can be another rtsp stream or stdin input from ffmpeg or libcamera-vid. I haven't seen any non-device options, should be easy to add support.
Describe the solution you'd like
--stdin option or --rstp-input option.
Describe alternatives you've considered
N/A

Hi,
v4l2 stand for video4linux, the approach of this project is to use this standard interface. In order to feed with something else you can use v4l2loopback.
Best Regards,
Michel.


Outdated Format

I think Alacritty uses .toml now?

It appears that @Incompleteusern is correct:
https://github.com/alacritty/alacritty#configuration
But I think we should leave the yml files in for backwards compatablity.

@isabelroses  @Incompleteusern Hello there. It looks like doing alacritty migrate -c <path-to-catppuccin-yaml> works fine. Just tried that on v0.13.0 and it worked for me. The only caveat is the warning for
Unused config key: footer_bar.
After commenting this part of migrated toml, the warning disappeared
#[colors.search.footer_bar]
#background = "#6C6F85"
#foreground = "#EFF1F5"

@nekowinston thanks a lot!

When I comment out the following code:
#[colors.search.footer_bar]
#background = "#6C6F85"
#foreground = "#EFF1F5"

I get an error. I'm trying to remove the warning "Unused config key: footer_bar", but am unable to.
Anyone have any idea what's going on?


Sessions

Hi Mr Juliuss, i'm new using laravel and of course your package and i really love it, However, i want to control the session time for the users, but i wanna ask you something... what's  the best way to add session expiration for the users?. Please I just want to know your opinion or an idea. Thanks for reading.

Sentry 2 is no longer maintained. I think this problem will not be fixed...


Fix missing ln -r flag in Centos 6

The ln version included in Centos 6 doesn't have the '-r' flag which makes links relative. Therefore we workaround this issue by steeping in the AppDir folder and generating links from there.
fix #28

Seems reasonable, although I'm not completely sure about that loop. I'll have to run a test.


YQL-17766 Fix decimal incompatibility between yqlrun and yt

Bugfix

@rvu1024 is it Ok to change the signature of WriteYsonContainerValue here ?
arg nativeYtTypeFlags is always 0

@kinash-varvara TYtCodegenCodec.TestReadContainerRow crashed. The reason: signarure of ReadYsonContainerValue was changed.


You'll need to recreate this PR if it's still valid for the master branch since we moved to Java 8 a long while ago now.


Multiple same cards with different settings

Ex: Weather with different city

Done in 4e78012a71920f19d19583b0da7411077a8aafaf
Allow Weather and LastFm to be multiple


Update proxy.go delete db stat core properties

delete db stat core properties

Codecov Report

Merging #32 (670d4a4) into master (0299444) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   71.65%   71.65%
=======================================
  Files          17       17
  Lines        1905     1905
=======================================
  Hits         1365     1365
  Misses        356      356
  Partials      184      184


Add Invoice class to generate valid bank slip references

Banks have certain rules about the format of invoice references.
This information is not easily available but I found enough for:

Finland
Estonia
Norway

The implementation should be easy to append for other countries as well

Pull Request Test Coverage Report for Build 2025

13 of 39 (33.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 33.926%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

test/test_determinism.rb
0
1
0.0%

test/test_invoice.rb
13
38
34.21%

Totals

Change from base Build 2023:
0.002%

Covered Lines:
1489

Relevant Lines:
4389

💛  - Coveralls

I'll fix the rubycop errors which travis mentioned soon.

@onnimonni  sure. Just run bundle exec rubocop -a and we should be ok.


Integrate code with fmu-dataio and fmu-sumo.uploader and thereby improve metadata

So far all metadata are generated by this code, and bypass fmu-dataio completely.
This should be changed,
Whether this should be done prior to integration with aggregation code is a point of discussion
Integration with fmu-dataio would simplify code, and iron out some bugs

Class Meta something could be removed, only needed because something needs to take care of what AggregatedData does for fmu-dataio
There are sections in source._sumo that are not being populated, this is done either by fmu-dataio or by the uploader.

Code moved to https://github.com/equinor/sumo-fmu-aggregation-service
Issue re-created there: https://github.com/equinor/sumo-fmu-aggregation-service/issues/246


ANMS_FUN_AGT_002 (Manage Agents): Clear Tables option

The Agents tab option to "Clear Agent Tables" has not yet been implemented, but it is available for selection.  Suggest this option be greyed out as is the "Write Agent Reports" to file option.

referencing old agent tab  updated with https://github.com/NASA-AMMOS/anms/pull/78


State Snapshot restore script fails

OS
Ubuntu focal
Versions
11.0.3
Build/Install Method
docker-compose running the following services:

postgres:11.5-alpine
inputoutput/cardano-node:1.30.1 (bind mound for directory /state-dir)
inputoutput/cardano-db-sync:11.0.3 (bind mound for directory /state-dir)

The sync runs fine.
Problem Report
I stop the cardano-db-sync service first as described in the documentation.
Then I try to restore the ledger state from the snapshot provided in version 11.0.3 for the mainnet.
I am running the command
PGPASSFILE=/root/cardano-db-snapshot/pgpass bash /root/cardano-db-snapshot/postgresql-setup.sh --restore-snapshot /root/cardano-db-snapshot/db-sync-snapshot-schema-9-block-5796064-x86_64 /root/code/141x/setup/cardano-db-sync/state-dir
and getting the error
ERROR:  cannot drop objects owned by role postgres because they are required by the database system
Additional  Context
My pgpassfile: localhost:5432:cexplorer:postgres:<mypass>
Maybe I am doing something wrong but I am stuck on this and would appreciate some help.

ERROR: cannot drop objects owned by role postgres because they are required by the database system

It looks like you have another process access the database at the same time you are trying to restore the database. Postgres says no.
This is not a db-sync issue.

Thank you very much @erikd.

Turns out I am supposed to restore the db snapshot in another way if I use docker-compose.
@rhyslbw @jbgi To my understanding the commit should make it possible to start the cardano-db-sync and perform a snapshot restore using the environment variables RESTORE_SNAPSHOT and RESTORE_SNAPSHOT but it's not officially documented here and I am not sure if I am doing it correctly.
My docker compose is standard and looks like this:
version: "3.5"

x-141x-node-config:
  &141x-node-config
  type: bind
  source: ./node-config
  target: /node-config
x-141x-node-ipc:
  &141x-node-ipc
  type: bind
  source: ./node-ipc
  target: /node-ipc
x-141x-state-dir:
  &141x-state-dir
  type: bind
  source: ./state-dir
  target: /state-dir

services:
  postgres:
    image: postgres:11.5-alpine
    environment:
      - POSTGRES_LOGGING=true
      - POSTGRES_DB_FILE=/run/secrets/postgres_db
      - POSTGRES_PASSWORD_FILE=/run/secrets/postgres_password
      - POSTGRES_USER_FILE=/run/secrets/postgres_user
    secrets:
      - postgres_password
      - postgres_user
      - postgres_db
    volumes:
      - postgres:/var/lib/postgresql/data
    ports:
      - 5432:5432
    restart: on-failure
  cardano-node:
    image: inputoutput/cardano-node:1.30.1
    volumes:
      - <<: *141x-node-config
      - <<: *141x-node-ipc
      - <<: *141x-state-dir
      - node-db:/data/db
    restart: on-failure
    command: [
    "run",
    "--topology /node-config/mainnet/mainnet-topology.json",
    "--config /node-config/mainnet/mainnet-config.json",
    "--socket-path /node-ipc/node.socket",
    "--database-path /data/db"
    ]
  cardano-db-sync:
    image: inputoutput/cardano-db-sync:11.0.3
    env_file:
      - mainnet.env
    environment:
      - POSTGRES_HOST=postgres
      - POSTGRES_PORT=5432
      - DB_SYNC_NETWORK=${DB_SYNC_NETWORK}
      - NETWORK=${DB_SYNC_NETWORK}
      - PGPASSFILE=/node-config/${DB_SYNC_NETWORK}/pgpass
      - CARDANO_NODE_SOCKET_PATH=/node-ipc
      - RESTORE_SNAPSHOT=/root/cardano-db-snapshot/db-sync-snapshot-schema-11-block-6278640-x86_64.tgz
      - RESTORE_RECREATE_DB=Y
    depends_on:
      - cardano-node
      - postgres
    secrets:
      - postgres_password
      - postgres_user
      - postgres_db
    volumes:
      - <<: *141x-node-config
      - <<: *141x-node-ipc
      - <<: *141x-state-dir
      - db-sync-data:/var/lib/cdbsync
    restart: on-failure
secrets:
  postgres_db:
    file: ./cardano-db-sync/config/secrets/postgres_db
  postgres_password:
    file: ./cardano-db-sync/config/secrets/postgres_password
  postgres_user:
    file: ./cardano-db-sync/config/secrets/postgres_user
volumes:
  db-sync-data:
  postgres:
  node-db:
  schema:
networks:
  default:
    name: 141x
    driver: bridge

after starting the compse file i get the following error:
cardano-node_1     | [739d59bd:cardano.node.ChainDB:Notice:35] [2021-09-30 17:47:48.35 UTC] Chain extended, new tip: b775d4d9a1fa88ac3de790bcc515b55994c3a04c10ddf893d1f2242a6135b4c3 at slot 257440
cardano-node_1     | [739d59bd:cardano.node.ChainDB:Notice:35] [2021-09-30 17:47:48.82 UTC] Chain extended, new tip: 71275d3a2bfa27b54c3ad036ab90a78e2b524d3de9882f15b92d7148bd9bb9b6 at slot 257602
cardano-node_1     | [739d59bd:cardano.node.ChainDB:Notice:35] [2021-09-30 17:47:50.08 UTC] Chain extended, new tip: 4b72449da9fdbc04eca22d8285e8bb6ec94373e52278be617439501381a35677 at slot 258178
cardano-node_1     | [739d59bd:cardano.node.ChainDB:Notice:35] [2021-09-30 17:47:51.33 UTC] Chain extended, new tip: 0d9270a71358185b0ad5e3507690d4600239c20e767bf5d361ffc31821615b83 at slot 258757
cardano-db-sync_1  | ERROR:  cannot drop objects owned by role postgres because they are required by the database system
postgres_1         | 2021-09-30 17:47:51.951 UTC [82] ERROR:  cannot drop objects owned by role postgres because they are required by the database system
postgres_1         | 2021-09-30 17:47:51.951 UTC [82] STATEMENT:  DROP OWNED BY CURRENT_USER;
cardano-db-sync_cardano-db-sync_1 exited with code 1

Is there something more I can try?

postgres_1         | 2021-09-30 17:47:51.951 UTC [82] ERROR:  cannot drop objects owned by role postgres because they are required by the database system
postgres_1         | 2021-09-30 17:47:51.951 UTC [82] STATEMENT:  DROP OWNED BY CURRENT_USER;

This really looks like a Postgres management issue. LIke I said it looks like there is another process using the database when you try to drop it.

@ggcaponetto Currently the restore process drop all object belonging to the configured user if the DB already exist. But this cannot work if the user is the system user postgres (usually used with docker). In this case you need to set the environment variable
https://github.com/input-output-hk/cardano-db-sync/blob/25b88e8e2aba7e43e3fffcc0b589119377dcc530/docker-compose.yml#L45
in order to disable this process. In this case you are responsible for the db to be already existing and empty before the restore process (eg. by dropping postgres volume).

Thank you all!


How to load style files?

I've recently got the new version of library and it's not like the way it used to be. In older versions, there was ConfigurationManager to do this.

See issues #5. You can find more information about the new way to set styles in the documentation.


[T2A2][W14]Ang Ray Yan

Good work. You might consider putting the while true loop of main function into the newly created function as well.
Remember to name your pull request correctly next time: [T2A2][W09-A1]James Yong


Fix broken headings in Markdown files

GitHub changed the way Markdown headings are parsed, so this change fixes it.
See bryant1410/readmesfix for more information.
Tackles bryant1410/readmesfix#1

I've made an equivalent change in master.


pr#158 problem with darwin_amd64_gfortran

When I try to compile with -mpi using the new optfile darwin_amd64_gfortran
(using brew installed openmpi on Sierra 10.12.6) genmake2 crashes with this stop:
MPI_HOME is not set and pkg-config not available, aborting
If I go back to the previous version of the optfile before PR#158, works fine. I don't know enough how the brew install of openmpi is being set/used so not sure where to start looking for the problem.
FWIW @edoddridge is not having this problem using Mojave

Seems that @raphaeldussin offered a good solution for this particular problem. Now it's time to close this issue.


[TODO] Create developer documentation

To help facilitate new contributors to contribute to our project, we need to create documentation on:

how to clone or fork the project
installing the project client and server
creating the database and environment variables
setting up the project in ediors/ides
how to submit PR's
basic git usage
how to start the server, client and database
how to run lint and prettier the code
how to run tests
list of client and server dependencies
links to documentation for dependencies
links to documentation for React
link to CONTRIBUTING.md doc

This should also exist in the CodeBooker GitHub Wiki which is clone-able as a separate project and uses markdown files.

Hi, I'm newish tech writer looking to get more documentation experience. I'd be happy to help with any of the above documentation tasks. I have some experience with JavaScript ES5/ES6/ES7 as wells as git, Angular etc. If you are interested, please get back to me with next steps. Thanks,
John

Hi @jchansen925
I started creating some basic documentation, but its not that solid.   Feel free to work on it and include the things above.
I think the best way is to put the majority of the documentation in the /docs folder as markdown.
I also think a great addition to this is to use the GitHub wiki which is cloneable at git clone https://github.com/gbowne1/codebooker.wiki.git and you can make all the files in markdown and make separate files for each heading.

Hi @gbowne1 .  Thanks for the reply, and cloning the GitHub wiki and making a markdown file for each heading is a good start. I'll start this process tomorrow and get a placeholder/starter page for each heading beginning tomorrow.
John

Alright,  @jchansen925, Feel free to ask questions  here or in Discussions. Myself and or @LOGESH-B can answer them.
The main objective is to support a developer to get started easily working on this project. Later on the documentation can be elaborated on with more details such as editor specifics, language translations,  etc..

@gbowne1 Thanks, will be cloning the wiki today and starting on the markdown files.

@gbowne1
Hi, I have forked this repo and made docs_update branch there. I have made .md files for this list of topics above-- how to clone or fork the project...
I am trying to commit my changes using HTTP, but am getting 403 errors I can't seem to get around, should I use SSH instead?

@gbowne1
These are the errors I am getting: johnhansen@Johns-MacBook-Pro docs % git commit --all
[master 031a03e] Added fork and clone instructions with images
5 files changed, 57 insertions(+)
create mode 100644 docs/CreateFork.png
create mode 100644 docs/CreateForkBtn.png
create mode 100644 docs/ForkClone.md
create mode 100644 docs/ForkedBranch.png
create mode 100644 docs/fork.png
johnhansen@Johns-MacBook-Pro docs % git push --all
Username for 'https://github.com': jhansen925
Password for<EMAIL_ADDRESS>remote: Permission to jchansen925/codebooker.git denied to jchansen925.
fatal: unable to access 'https://github.com/jchansen925/codebooker.git/': The requested URL returned error: 403

@gbowne1
These are the errors I am getting: johnhansen@Johns-MacBook-Pro docs % git commit --all [master 031a03e] Added fork and clone instructions with images 5 files changed, 57 insertions(+) create mode 100644 docs/CreateFork.png create mode 100644 docs/CreateForkBtn.png create mode 100644 docs/ForkClone.md create mode 100644 docs/ForkedBranch.png create mode 100644 docs/fork.png johnhansen@Johns-MacBook-Pro docs % git push --all Username for 'https://github.com': jhansen925 Password for<EMAIL_ADDRESS>remote: Permission to jchansen925/codebooker.git denied to jchansen925. fatal: unable to access 'https://github.com/jchansen925/codebooker.git/': The requested URL returned error: 403

Hii @jchansen925
This may help you
https://github.com/trilbymedia/grav-plugin-git-sync/issues/39


lyxal: Watch forrest[\W_]+inn[\W_]+camping[\W_]*+resort

lyxal requests the watch of the watch_keyword forrest[\W_]*+inn[\W_]*+camping[\W_]*+resort. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

forrest[\W_]*+inn[\W_]*+camping[\W_]*+resort has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


compile_index_url_fallback_prefer_primary test doesn't seem to test anything

The following test:
https://github.com/astral-sh/uv/blob/116d47ed03695c5fb2ce8084663ff79bc87b6b54/crates/uv/tests/pip_compile.rs#L7808-L7844
indicates that it relies on the "extra" index having 3.1.2 as the newest version. However, looking at https://download.pytorch.org/whl/jinja2/, the index includes 3.1.3 as well, so the test doesn't really test anything.
I suppose it would be better to actually use a index that you control here.

Thanks!

My guess is that the index version was updated.

Are we not using exclude newer here?

Huh we explicitly turn that off, seems unnecessary.

No, it's intentional -- we can't use --exclude-newer with the PyTorch index.

Ah because they don't have timestamps? 😭

But the solution is to not use the PyTorch index :joy:


Display module and line number of crash or exception in CRASH DUMP

It is sometimes very difficult to diagnose where a program has crashed without having to add printf's to the code.  This becomes increasingly difficult with third party and core libraries.
If we could identify the module and  line number of the crash, that would make diagnosing errors a lot easier.

Fixed in PR #254


flutter doctor -v  run wrong

flutter doctor -v
The flutter tool cannot access the file or directory.
Please ensure that the SDK and/or project is installed in a location that
has read/write permissions for the current user.

#0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
#1      _throwFileSystemException
(package:flutter_tools/src/base/error_handling_io.dart:820:3)
#2      _handlePosixException
(package:flutter_tools/src/base/error_handling_io.dart:761:3)
#3      _runSync
(package:flutter_tools/src/base/error_handling_io.dart:614:7)
#4      ErrorHandlingProcessManager.runSync
(package:flutter_tools/src/base/error_handling_io.dart:711:12)
#5      _DefaultProcessUtils.runSync
(package:flutter_tools/src/base/process.dart:373:51)
#6      AndroidStudio._init
(package:flutter_tools/src/android/android_studio.dart:458:39)
#7      new AndroidStudio
(package:flutter_tools/src/android/android_studio.dart:40:5)
#8      AndroidStudio.fromMacOSBundle
(package:flutter_tools/src/android/android_studio.dart:90:12)
#9      AndroidStudio._allMacOS.<anonymous closure>
(package:flutter_tools/src/android/android_studio.dart:316:68)
#10     MappedListIterable.elementAt
(dart:_internal/iterable.dart:413:31)
#11     ListIterator.moveNext (dart:_internal/iterable.dart:342:26)
#12     WhereTypeIterator.moveNext (dart:_internal/iterable.dart:869:20)
#13     new _GrowableList._ofOther
(dart:core-patch/growable_array.dart:219:26)
#14     new _GrowableList.of (dart:core-patch/growable_array.dart:155:26)
#15     new List.of (dart:core-patch/array_patch.dart:51:28)
#16     Iterable.toList (dart:core/iterable.dart:470:12)
#17     AndroidStudio._allMacOS
(package:flutter_tools/src/android/android_studio.dart:318:10)
#18     AndroidStudio.allInstalled
(package:flutter_tools/src/android/android_studio.dart:247:34)
#19     AndroidStudioValidator.allValidators
(package:flutter_tools/src/android/android_studio_validator.dart:33:55)
#20     _DefaultDoctorValidatorsProvider.validators
(package:flutter_tools/src/doctor.dart:80:35)
#21     Doctor.validators (package:flutter_tools/src/doctor.dart:192:47)
#22     Doctor.startValidatorTasks
(package:flutter_tools/src/doctor.dart:198:45)
#23     Doctor.diagnose (package:flutter_tools/src/doctor.dart:309:47)
#24     DoctorCommand.runCommand
(package:flutter_tools/src/commands/doctor.dart:50:48)
#25     FlutterCommand.verifyThenRunCommand
(package:flutter_tools/src/runner/flutter_command.dart:1320:12)
<asynchronous suspension>
#26     FlutterCommand.run.<anonymous closure>
(package:flutter_tools/src/runner/flutter_command.dart:1161:27)
<asynchronous suspension>
#27     AppContext.run.<anonymous closure>
(package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#28     CommandRunner.runCommand
(package:args/command_runner.dart:209:13)
<asynchronous suspension>
#29     FlutterCommandRunner.runCommand.<anonymous closure>
(package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>
#30     AppContext.run.<anonymous closure>
(package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#31     FlutterCommandRunner.runCommand
(package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>
#32     run.<anonymous closure>.<anonymous closure>
(package:flutter_tools/runner.dart:62:9)
<asynchronous suspension>
#33     AppContext.run.<anonymous closure>
(package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#34     main (package:flutter_tools/executable.dart:94:3)
<asynchronous suspension>

@dd9913762113
As the error message indicates :
The flutter tool cannot access the file or directory. Please ensure that the SDK and/or project is installed in a location that has read/write permissions for the current user.
So the solution is to ensure that the SDK and/or project is installed in a location that has read/write permissions for the current user.
Closing from here.


Formatting?

Have you considered using a formatter to make code review / merges less painful?

The codebase now adapted the ruff formatter.


Remove redundant param retrieval

There's a variable named param for this already inside the function scope.

LGTM

Never mind. This is not correct since at this point


stop passing credential provider to interceptor factory

When initializing the pubsub client interceptors, we have been passing the credential provider to the PubsubClientInterceptorFactory. We should instead grab the values we need (just authToken I believe) off of the credential provider and pass them in instead.

Addressed by #37


[BUG?] - RSS link to issue results in error "could not find in my database"

From version 1.0.11 January 10th, 2015 (perhaps already corrected, if so - sorry for duplicate)
When accessing inside KB:
http://SERVER/kanboard/?controller=task&action=show&task_id=61**&project_id=12**
When accessing link found in RSS feed:
http://SERVER/kanboard/?controller=task&action=show&task_id=61
RESULT when accessing link found in RSS feed:
"could not find in my database"

it was a bug should be fixed in the dev version  #543 #523

Closing (duplicate)


Can i install it on endless os? its ostree based too

probably, only if endless can run rpm-ostree


mysql 数据库切换导致cdc 消费binlog出错，是否支持gtid同步？

dba进行数据库备库切换操作，导致binlog出错
dba反馈数据库已经支持gtid的同步方式，目前cdc这边好像还不支持？
org.apache.kafka.connect.errors.ConnectException: The connector is trying to read binlog starting at GTIDs 3fa7d5bb-65f3-11eb-9413-b0262879b560:1-730774004 and binlog file 'mysql-bin.000650', pos=521310219, skipping 2 events plus 1 rows, but this is no longer available on the server. Reconfigure the connector to use a snapshot when needed.

Closing this issue because it was created before version 2.3.0 (2022-11-10). Please try the latest version of Flink CDC to see if the issue has been resolved. If the issue is still valid, kindly report it on Apache Jira under project Flink with component tag Flink CDC. Thank you!


DOCFIX: Remove misleading attribute from table doc

The resource documentation for snowflake_table includes the owner attribute in its example code in a way that makes it seem like the attribute is writable. However, the owner attribute is read-only, and its value is assigned at resource creation.
This pull request removes the owner attribute from the example code.

/ok-to-test sha=d7d7d11

/ok-to-test sha=b8e081b


Compile-time static : void should unregister static

The problem in tokay.tok with the Token1 could also be eliminated by allowing to set Token : void previously, and unregister it this way. This is an extension of the void-paradigm described in #82.

Removing this from Tokay v0.6 milestone as it requires further consideration.


Fix formatting in insult constant

Correct me if I'm wrong, but it seems like you were missing a comma?

No comma missing, they're part of the same value. It's just formatted like that as part of my IDE's integrated PEP standards (PEP 8 E501 if you're interested). Rejecting this.

Thanks for clarifying! However, while PEP 8 E501 addresses line length, it doesn't mandate formatting values this way. To improve readability, especially in lists of long strings, PEP 8 recommends implicit line joining with parentheses over IDE-specific formatting. This avoids accidental concatenation and makes each item clearer, as discussed in PEP 8's guidelines on line continuation.

The preferred way of wrapping long lines is by using Python’s implied line continuation inside parentheses, brackets and braces. Long lines can be broken over multiple lines by wrapping expressions in parentheses. These should be used in preference to using a backslash for line continuation.

Would you consider updating for consistency?

    ),
    (
        "Ew, do you smell that? Oh, that's just {arg}."
    ),
    (
        "We thought they went to Mordor to throw the One Ring in Mount Doom, "
        "but it was actually to get away from {arg}."
    ),
    (
        "If {arg} was a Pokémon, they would be the only one Ash wouldn't catch."
     Meanie."
    ),
    (
        "{arg} is proof light travels faster than sound. They appear bright until they speak."

    ),
]

I have pushed this to my branch, anyway, so feel free to reopen and use, if you want. But no worries, if not.


Move MD5-SHA1 digest completely to the default provider

This leaves minimal implementations of EVP_md5_sha1, which is now only
there to provide a name for implicit fetches.

This will fail if an ENGINE implements this digest.

Or to be clearer, if an EVP_MD_CTX is used with an ENGINE, and then the same EVP_MD_CTX is reused with this digest (even if the engine doesn't implement it) then it will fail.
Of if EVP_DigestInit_ex is called with an ENGINE that doesn't implement this digest it will also fail.

Or (perhaps less likely with this particular digest - but still possible), if it is used as part of a DigestSign operation.

Of if EVP_DigestInit_ex is called with an ENGINE that doesn't implement this digest it will also fail.

Doesn't it anyway?  If someone specifies an ENGINE, then it's because they fully expect that engine to implement the digest they want, and if the ENGINE doesn't, then DigestInit should fail, no?  It sure looks like it does, here:
https://github.com/openssl/openssl/blob/fac9200a881a83bef038ebed628ebd409786a1a6/crypto/evp/digest.c#L220-L228

Or to be clearer, if an EVP_MD_CTX is used with an ENGINE, and then the same EVP_MD_CTX is reused with this digest (even if the engine doesn't implement it) then it will fail.

Same thing here, surely?

Doesn't it anyway? If someone specifies an ENGINE, then it's because they fully expect that engine to implement the digest they want, and if the ENGINE doesn't, then DigestInit should fail, no? It sure looks like it does, here:

Ah...yeah I think in that case it just fails.

Or to be clearer, if an EVP_MD_CTX is used with an ENGINE, and then the same EVP_MD_CTX is reused with this digest (even if the engine doesn't implement it) then it will fail.

Same thing here, surely?

If an EVP_MD_CTX has previously been used with an engine then ctx->engine is set. That's enough to send us into the legacy handling. If "impl == NULL" then we check if there is an ENGINE can handle this digest (see tmpimpl)...if not then we just end up in the standard legacy handling

If an EVP_MD_CTX has previously been used with an engine then ctx->engine is set. That's enough to send us into the legacy handling.

Ah, yeah, I can see that.  This is because the code currently always sees the context as legacy if there was an engine involved, regardless of if it implements the new desired algo or not.  The 1.1.1 code simply drops the currently attached engine when a new type is given, and starts over the process of figuring out if there is an engine that implements the type or not, which is slightly different semantics...
I'm thinking that the portion of the legacy code that drops the engine should appear higher up and not be tucked away in the legacy portion.  Do you agree with that line of thinking?

I'm thinking that the portion of the legacy code that drops the engine should appear higher up and not be tucked away in the legacy portion. Do you agree with that line of thinking?

So there's two section of legacy handling code. That between then comment /* TODO(3.0): Legacy work around code below. Remove this */ and the comment /* TODO(3.0): Start of non-legacy code below */. And then everything below the comment /* TODO(3.0): Remove legacy code below */. My expectation is that, eventually, both sections will be removed and we'll just have providers. In the interim, I'm fine with moving code around between the two legacy sections as we see fit to streamline things.
Once ENGINEs are seen as just another provider then a lot of the existing legacy code will be able to be removed.
That doesn't solve the other problem though which is the DigestSign stuff. That code hasn't been implemented yet so we're forced into the legacy handling in that case.

See #9077

Ok, #9077 is in, are your immediate concerns cleared?

Like I said:

That doesn't solve the other problem though which is the DigestSign stuff. That code hasn't been implemented yet so we're forced into the legacy handling in that case.

I'll have to look more closely at that tomorrow.  Too tired for the moment.

I think this PR will hang around until after PKEY is moved to the provider.

I think this PR will hang around until after PKEY is moved to the provider.

Why?  If DigestSign is the problematic part, that's what we need to wait on, but the rest of PKEY?  Nah...

Rebased.  It tests cleanly on my laptop...

Merged.
f50958b8c80ae267c02b2d534da8bde091010707 Move MD5-SHA1 digest completely to the default provider


PreTag    InsecureRequestWarning: Unverified HTTPS request is being made

While executing the test , we are seeing logs throwing up on console and log output
.tox/palmetto/local/lib/python2.7/site-packages/requests/packages/urllib3/connectionpool.py:821: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.org/en/latest/security.html
InsecureRequestWarning)
This is a debatable topic.. they are really genuine warning coming from urllib library packaged which comes with the python however for the test case output this seems redundant and flooding the log is not desirable...
This warning as per the discussion over the forums seems to fix post 2.7.xx Python version, however there are dependencies with robot framework which creates more configuration and setting issues..
There is a work around hack for this.. but will wait for Chris to discuss and close on this one if this really needed to be fixed..

I've already added       PYTHONWARNINGS="ignore:Unverified HTTPS request" as env var so that you will not a problem if Python 2.7 used with tox.
You will hit this issue if 2.6 version of python used or if you are not having PYTHONWARNINGS set to ignore warning.

We are aware of that tox.ini having that parameter.. It does work now... I have a version 2.7.11 installed as well
Python 2.7.11 (default, Dec 15 2015, 16:48:05)
This is seen with other folks as well...

I have a simplified fix working solution for this.. but will talk to Chris Austen on it to see if he is ok with the fix..  Will keep posted

The fix will look like this.. This would to suppress almost or stop it from propagating on the console those COnnections, URLLIB warnings..
lib/disable_warning_urlib.py
#!/usr/bin/python
import logging
import warnings
import httplib
warnings.filterwarnings("ignore")
Hijack the HTTP lib message and Log only once
#httplib.HTTPConnection.debuglevel = 0
requests_log = logging.getLogger("requests.packages.urllib3")
requests_log.setLevel(logging.CRITICAL)
requests_log.propagate = False
class disable_warning_urlib():
def do_nothing():
return
*** Setting ***
Library                 ../lib/disable_warning_urlib.py
in the lib/rest_client.robot
That's it..

Can you please make sure whether this works for python 2.6 as well? if yes - send the PR with proposed changes.

Python 2.6.9 (default, Oct 22 2014, 19:58:14)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.

I have to tweak the tox.ini to pick up the 2.6.9
rango@ubuntu:~/FTC_OPENBMC_TEST/openbmc-automation$ ls .tox/barreleye/bin/
activate      activate.fish     easy_install      pip   pip2.6  python   python2.6  robot
activate.csh  activate_this.py  easy_install-2.6  pip2  pybot   python2  rebot
rango@ubuntu:~/FTC_OPENBMC_TEST/openbmc-automation$ .tox/barreleye/bin/python
Python 2.6.9 (default, Oct 22 2014, 19:58:14)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.

and it runs good like it ran on 2.7.11
We will still see some Warning but not recurring  in a test case any more..

SAMPLE  TEST CASE 1  : o/p

BEFORE :
==============================================================================
Test Generic Conf :: This suite will verifiy the Generic Configuration Rest...
Get the boot_flags :: GOOD PATH                                 URI:https://<IP_ADDRESS>/org/openbmc/settings/host0//attr/boot_flags, method:Get, args:{}
/home/rango/FTC_OPENBMC_TEST/openbmc-automation/.tox/barreleye/lib/python2.6/site-packages/requests/packages/urllib3/util/ssl_.py:318: SNIMissingWarning: An HTTPS request has been made, but the SNI (Subject Name Indication) extension to TLS is not available on this platform. This may cause the server to present an incorrect TLS certificate, which can cause validation failures. You can upgrade to a newer version of Python to solve this. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#snimissingwarning.
SNIMissingWarning
/home/rango/FTC_OPENBMC_TEST/openbmc-automation/.tox/barreleye/lib/python2.6/site-packages/requests/packages/urllib3/util/ssl_.py:122: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. You can upgrade to a newer version of Python to solve this. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
InsecurePlatformWarning
/home/rango/FTC_OPENBMC_TEST/openbmc-automation/.tox/barreleye/lib/python2.6/site-packages/requests/packages/urllib3/connectionpool.py:821: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.org/en/latest/security.html
InsecureRequestWarning)
/home/rango/FTC_OPENBMC_TEST/openbmc-automation/.tox/barreleye/lib/python2.6/site-packages/requests/packages/urllib3/connectionpool.py:821: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.org/en/latest/security.html
InsecureRequestWarning)
Response code:200, Content:{
"data": "Default",
"message": "200 OK",
"status": "ok"
}
Get the boot_flags :: GOOD PATH                                 | PASS |
Test Generic Conf :: This suite will verifiy the Generic Configura... | PASS |
1 critical test, 1 passed, 0 failed
1 test total, 1 passed, 0 failed
AFTER ::
==============================================================================
Test Generic Conf :: This suite will verifiy the Generic Configuration Rest...
Get the boot_flags :: GOOD PATH                                 URI:https://<IP_ADDRESS>/org/openbmc/settings/host0//attr/boot_flags, method:Get, args:{}
Response code:200, Content:{
"data": "Default",
"message": "200 OK",
"status": "ok"
}
Get the boot_flags :: GOOD PATH                                 | PASS |
Test Generic Conf :: This suite will verifiy the Generic Configura... | PASS |
1 critical test, 1 passed, 0 failed
1 test total, 1 passed, 0 failed

SAMPLE  TEST CASE 2  : o/p

BEFORE :**
==============================================================================
Test Ssl :: This testsuite is for testing SSL connection to OpenBMC
[ WARN ] Retrying (Retry(total=2, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0xa535e6c>: Failed to establish a new connection: [Errno 111] Connection refused',)': //list
[ WARN ] Retrying (Retry(total=1, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0xa544aec>: Failed to establish a new connection: [Errno 111] Connection refused',)': //list
[ WARN ] Retrying (Retry(total=0, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0xa544c2c>: Failed to establish a new connection: [Errno 111] Connection refused',)': //list
Test non-SSL Connection to port 80 :: This testcase is for test to... | FAIL |
ConnectionError: HTTPConnectionPool(host='<IP_ADDRESS>', port=80): Max retries exceeded with url: //list (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0xa544d4c>: Failed to establish a new connection: [Errno 111] Connection refused',))
Test Ssl :: This testsuite is for testing SSL connection to OpenBMC   | FAIL |
1 critical test, 0 passed, 1 failed
1 test total, 0 passed, 1 failed
AFTER:
==============================================================================
Test Ssl :: This testsuite is for testing SSL connection to OpenBMC
Test non-SSL Connection to port 80 :: This testcase is for test to... | FAIL |
ConnectionError: HTTPConnectionPool(host='<IP_ADDRESS>', port=80): Max retries exceeded with url: //list (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x9cdd08c>: Failed to establish a new connection: [Errno 111] Connection refused',))
Test Ssl :: This testsuite is for testing SSL connection to OpenBMC   | FAIL |
1 critical test, 0 passed, 1 failed
1 test total, 0 passed, 1 failed

The code is merged and available on the main repo.. Closing this task..


server.py: handle possible case where only 1 completion causes error

Merged.

@mallochine
Can you provide a test case for this?
@Shougo
You should not merge things that break it more in strange ways.
I assumed you want to see steps to reproduce it always?!

I should revert the changes?

I assumed you want to see steps to reproduce it always?!

I have read the code and thought it is just simple fixes.
Sorry.

Reverted.

@mallochine
Please shed some light.

the problem was an exception occurred in self.parse_completion. The error was some encoding thing like utf8. Can we just have a %s there to fix it? I don't have time for a simple test case.

the problem was an exception occurred in self.parse_completion. The error was some encoding thing like utf8. Can we just have a %s there to fix it? I don't have time for a simple test case.

The test case is needed.

OK how to add test case?

Please provide the error cases.

The minimal init.vim
The minimal script
The reproduce ways from neovim starting


Fix build break from VS 2022 17.8.2

Resolves: #3094
Description
eBPF for Windows fails to build using Visual Studio 2022 version 17.8.2
Testing
CI/CD
Documentation
No.
Installation
No.

@Alan-Jowett this PR would also complete #3091, please add it to the description for closure.


Consolidate bosh-init and normal deployment type files

After #24, we should be able to remove the differences between normal bosh and bosh-init template files, and not have to merge/gen differently

We don't have to merge/gen them differently, but we do have to deploy them differently.  Closing.


need indication that more than one instance of a nested template has been filled in, when reopening resource

(see also #1249)
for nested templates, if you add data to more than one nested chunk, when you later open the resource, only the first chunk shows. you have to expand the second chunk to see the data, but there's no indication that the second chunk has data in it.
example from stage, RT = ld4p:RT:bf2:Monograph:Work:Un-nested
here shows two Related Work chunks filled out

then I save, and then go to Search and search by the URI https://trellis.stage.sinopia.io/repository/cornell/39e7dacb-ba24-493d-b63d-acf278afc3a5 and open the resource, and now here's how Related Works looks

In Stanford cataloging tests, catalogers thought the second chunk of info had disappeared

so we need either an indication that there's something hiding in that second chunk, or can we just always open everything that has data in it?
@astridu


Multiple Fonts in one theme

Hi!
I met a need of using multiple fonts in one theme (i.e. I have multiple components that use a Monospaced font instead of Regular), but since  has set number of variants and it's impossible to add new, what is the proper way to add support for the second font in theme file?
Don't want to set fontFamily in component, cause it kills the whole idea of theming, imo

Hi @oliviertassinari, I have trouble setting the fontFamily on a variant:
let themeOptions = {
      typography: { fontFamily: 'Droid Sans', h1: {fontFamily: 'some other font, e.g. Tahoma'} },
      overrides: {
        MuiCssBaseline: {
          '@global': {
            '@font-face': fontList?.[0],
          } as any,
        },
      },
    };

const theme = createMuiTheme(themePalette);

h1 never gets the fontFamily change.

Nothing prevents you from using two different fonts. You can override different part of the theme:

theme.typography.fontFamily: this is the default font, Roboto, changing it will impacts all the components.
theme.typography.VARIANT_NAME.fontFamily: each variant can have a different font, it will impact the built-in typography.
theme.typography.fontFamilyMonospaced: you can add a new property in the theme, and use it where needed.

this is the kind of thing that goes in FAQ of the theme....


[Exports] New export structure

Currently
import * as Nessie from 'nessie-ui'
...
<Nessie.Button />

or
import { Button }  from 'nessie-ui'
...
<Button />

Problem
Too inflexible:

no good way to export non-components (HOCs, etc.)
can't categorize components

Wanted
Keep this old way:
import * as Nessie from 'nessie-ui'; // still works but DISCOURAGED
...
<Nessie.Dropdown />

continue to export all individual components at top level:
import { Dropdown } from 'nessie-ui';
...
<Dropdown />

Export all components as default export:
import Nessie from 'nessie-ui';
...
<Nessie.Dropdown />

The above can be combined:
import Nessie, { Dropdown } from 'nessie-ui';
...
<Nessie.Dropdown /> or <Dropdown />

Export submodules per component (here also containing a non-component export - the withDropdown HOC):
import Dropdown, { withDropdown } from 'nessie-ui/dropdown';
...
<Dropdown />
...
const MyComponentWithDropdown = withDropdown( MyComponent );

might also be useful to export categories of components:
import Layout, { Column } from 'nessie-ui/layout';
...
<Layout.Row><Column> ... </Column></Layout.Row>

Due to the folder structure of the repo it might be tricky to achieve the from 'nessie-ui/XXX' bit (e.g. from 'nessie/dropdown', from 'nessie/layout').
It would be easier to implement  from 'nessie-ui/lib/XXX' (from 'nessie/lib/dropdown', from 'nessie/lib/layout').


fix(ui): Fix segmenter yup validation schema

What this PR does / why we need it:
This PR introduces an additional fix to the yup validation schema, as a follow up from #80. In particular, the fix in this PR addresses this change:

When using the same Schema.when function but passing a function directly as an argument instead of the builder object, the signature of the expected function has also been updated from (value, schema)=> Schema): Schema to (values: any[], schema) => Schema): Schema.
Existing schemas that do not follow this convention now have been updated to reflect the new expected array:
Example:
// before
some_field: yup.string().when("some_other_field", (some_other_field, schema) =>
  some_other_field ? yup.string().required("this is needed") : schema
),

// after
some_field: yup.string().when("some_other_field", ([some_other_field], schema) =>
  some_other_field ? yup.string().required("this is needed") : schema
),

Which issue(s) this PR fixes:

Fixes #

Thanks for the quick review @tiopramayudi, I'll merge this after rebasing this branch once the other PR with the CICD fixes is merged!


Close Fit shadow artefacts

Close Fit shadows produce incorrect results compared to Stable Fit

Resolved in version v1.0.6


Pixel means in config file

In the config file there is a PIXEL_MEANS parameter.
Is this just the BGR pixel mean of the training set used for normalization?

It is RGB means actually, it will be converted into BGR in the code.

Do I need to adjust the means according to my own dataset?

It is up to you. We use the default ImageNet means. You can try with the means of your own dataset. And please let me know whether it is better to use dataset specific means.


119 5.6 group accepted users by color

US_5.6 (connects to #119)
Implemented the ability to set and sort by groups with colors on the participants page.

Coverage decreased (-0.1%) to 99.882% when pulling e5bc0232a8ef527c7df25743ef12c28c4a3c0456 on 119_5.6_groupAcceptedUsersByColor into fa5ab5007842a61dbddc852cd10f543711e62051 on dev.

Coverage remained the same at 100.0% when pulling 98161228f814e8b3a8ead774e17db4c338026f18 on 119_5.6_groupAcceptedUsersByColor into fa5ab5007842a61dbddc852cd10f543711e62051 on dev.

Coverage remained the same at 100.0% when pulling 3574b4380600f43f9709c6ec267a772d7cd55db9 on 119_5.6_groupAcceptedUsersByColor into 8a87c2632a266ee8f32b19e7d2615b5c6db56c40 on dev.

Coverage remained the same at 100.0% when pulling b9e1b7f6a3e259a59d1e04470915fc4c8a36ef69 on 119_5.6_groupAcceptedUsersByColor into 8a87c2632a266ee8f32b19e7d2615b5c6db56c40 on dev.

Coverage remained the same at 100.0% when pulling 3160461d19921024d4095c1d4fdd8fec6f2170dd on 119_5.6_groupAcceptedUsersByColor into 7bd7a034f6bb655a431c50df6bc98ee7e559db3d on dev.

Coverage remained the same at 100.0% when pulling 69441f9afb3d93006f9391550c50db57d48eb19b on 119_5.6_groupAcceptedUsersByColor into d6d1af329b01da0a5494b2cc30aa33591a8d4819 on dev.

Coverage remained the same at 100.0% when pulling c3bb78433397d3668846d9c25f354406c301323f on 119_5.6_groupAcceptedUsersByColor into d6d1af329b01da0a5494b2cc30aa33591a8d4819 on dev.

Coverage remained the same at 100.0% when pulling f399046afe5fe6589e53f8224fd2a25644d8a08a on 119_5.6_groupAcceptedUsersByColor into d6d1af329b01da0a5494b2cc30aa33591a8d4819 on dev.

Coverage remained the same at 100.0% when pulling 8fe2f3c6788adafce768aaa3560de76d5e9b4a53 on 119_5.6_groupAcceptedUsersByColor into d6d1af329b01da0a5494b2cc30aa33591a8d4819 on dev.


JavaScript: Refactor a few access path computation predicates.

I haven't done a full evaluation of this PR in isolation, but here (internal link) is a comparison between runtimes for 1.20.2, rc/1.21 (before https://github.com/Semmle/ql/pull/1449), and rc/1.21 with this PR and https://github.com/Semmle/ql/pull/1449. As you can see, taken together these two PRs fix major performance regressions on doctored, fiddlesalad, goojs and orion.client. I think this PR fixed the first three and the other the last one.
I have also verified that there are no result differences introduced by this PR and https://github.com/Semmle/ql/pull/1449.

Very nice. Thanks.


Update the .NET Aspire dashboard page to reflect preview-3 changes

We need updated screen captures:

[ ] Trace length radial icon
[ ] Resource details view
[ ] Telemetry improvements

Metrics filter UI
Histogram metrics count
Span events

@alistairmatthews could you take this one please?

@IEvangelist Yes! On it now.

PR #399 should fix all these. I've also revised some other screenshots that had changed.


Adding a non-existing or no longer existing blog silently fails

Describe the bug
Adding a non-existing or no longer existing blog silently fails without error message or the like.
To Reproduce
Steps to reproduce the behavior:

Enter a no longer existing blog address in the url field (e.g. urmeimu.tumblr.com)
Click on 'Add Blog'
Nothing visible happens

Expected behavior
At least a message should be shown, some error label, a log entry or the like.
Desktop (please complete the following information):

TumblThree version: <IP_ADDRESS>
OS: Windows 10
Browser: Chrome
Version [e.g. 22]

Additional context

As before this one is for first-timers-only. That means that we will only accept a PR for this one from someone who's never contributed to open source before. This one is particularly easy (but don't make that statement make you feel bad if you have a hard time with it, there's more to contributing to open source than changing lines of code, especially if it's your first time). We walk you through it if you need help. :-) Here are the steps to get a PR merged (see also our guide).
When trying to add a non-existing or no longer existing blog to TumblThree, nothing visible happens, no error message is shown and the blog is not added to the blog list. But the blog's database file and download directory are already created. The problem is that the blog cannot be added because it just don't exist. So a message in the error message bar should be shown to the user. Additionally it either should be prevented that the file and directory are created or they should be cleaned up afterwards.

[x] Create a fork of the repository
[x] Clone it to your local disk and create a new branch (e.g. 112-adding-non-existing-blog-fails)
[x] Look what class and methods are used to add a new blog inside the TumblThree.Applications project. Does it look more easy to prevent the file and directory creation or to remove it afterwards? How can the code be changed to accomplish it?
[x] Add your solution
[x] Build the app and run it
[x] Type in the url of a non-existing blog and try to add it
[x] Check if your error message occurs
[x] Commit your changes mentioning "issue #112" somewhere in your commit message title or description
[x] Push your changes to your fork, create a PR, get merged, celebrate??

We are happy to support you if you need help.

As before this one is for first-timers-only. That means that we will only accept a PR for this one from someone who's never contributed to open source before. This one is particularly easy (but don't make that statement make you feel bad if you have a hard time with it, there's more to contributing to open source than changing lines of code, especially if it's your first time). We walk you through it if you need help. :-) Here are the steps to get a PR merged (see also our guide).
When trying to add a non-existing or no longer existing blog to TumblThree, nothing visible happens, no error message is shown and the blog is not added to the blog list. But the blog's database file and download directory are already created. The problem is that the blog cannot be added because it just don't exist. So a message in the error message bar should be shown to the user. Additionally it either should be prevented that the file and directory are created or they should be cleaned up afterwards.

[x] Create a fork of the repository
[x] Clone it to your local disk and create a new branch (e.g. 112-adding-non-existing-blog-fails)
[x] Look what class and methods are used to add a new blog inside the TumblThree.Applications project. Does it look more easy to prevent the file and directory creation or to remove it afterwards? How can the code be changed to accomplish it?
[x] Add your solution
[x] Build the app and run it
[x] Type in the url of a non-existing blog and try to add it
[x] Check if your error message occurs
[x] Commit your changes mentioning "issue #112" somewhere in your commit message title or description
[x] Push your changes to your fork, create a PR, get merged, celebrate??

We are happy to support you if you need help.

Has this been taken up? I'm contributing to open source for the first time but this looks like a good place to start

Hello,
thanks for your interest to work on this issue. It is free and I will assign it to you.
Please ask if you have a question or get stuck.

Do you need any help?

Hi. I'm interested in contributing to open source for the first time and I'd give this problem a good go. Would you be willing to reassign this to me?

Hello,
thanks for your interest to work on this issue. As we haven't received any reaction for over 14 days (e.g. asking for more time), according to our rules, this issue can be reassigned.
I assign it to you.

I could not reproduce the issue. Entering the example URL and pressing "Add Blog" resulted in an error message, as shown.

Oops, I double checked, it slipped in with the last version. So it's already solved.
Can you please tackle one of the other open issues?

The issue has been fixed and closed.


404 error when onboarding FTD with LTP possible

404 errors can happen when onboarding an FTD with LTP due to a race condition. Fix is to add polling to the ftd.py file when attempting to retrieve the specific device after adding the device to CDO.

Fixed in pull https://github.com/CiscoDevNet/ansible-cisco-cdo/pull/30 and released version 1.1.7 to Ansible-Galaxy


Onboarding for Barcelona

Application submitted 2021-07-01T21:39:53.000Z by Kevin van Reenen

[ ] Location email access
[ ] Set up event page

if new location

[ ] Set up new location page
[ ] Set up new location email address

Duplicate of #15 #16


Deploy and check REACT_APP_SPENDING_LIMIT_MODULE_ADDRESS before release

For testing spending limits we are using a module deployed to this rinkeby address:

0x9e9Bf12b5a66c0f0A7435835e0365477E121B110

We need to deploy the spending limit module to mainnet and change configuration before releasing to production

@dasanra, if we happen to get the AllowanceModule artifact from the safe-modules package, we can retrieve the module's address by searching for it in the networks key.
With that, we wouldn't need the REACT_APP_SPENDING_LIMIT_MODULE_ADDRESS env constant.
Am I right?

closing this as it was released already.