PDF417 max 255 characters

I try to print a 2d code with the following command :
impresion.Print2DCode(TwoDimensionCodeType.PDF417, documento.Ted, Size2DCode.LARGE, CorrectionLevel2DCode.PERCENT_15)
but it returns an error "string has more than 255 characters", I need to print a string of 350 characters which is the official stamp for electronic ballots in my country, I am generating the pdf stamp as an image and then I print it with:
print.PrintImage(bytesPDF, true, true) , but the quality is blurry.
I take this opportunity to thank you for the great work you do, really thank you very much.

Hi, that 255 char cap is hard coded, and the reason is that EPSON's manual says it can't be larger than that, see below

Right now there's no way of overriding that setting, but you could pull the code locally, and change it at least to see if it works for your printer.

I see you mentioned something about your country's official ballots.  Do they have to be PDF417?  The reason I ask is if PDF417 is supposed to support larger data areas than ESCPOS currently supports then maybe we're doing something wrong.
Another possible interpretation of the above is that you can store more than 255 bytes, but you can only write 255 bytes to the symbol storage area at once, and pL/pH are used as offsets, so the true max size is then 65535.  If that's the case and that's the right way to interpret this, then I think you would split your byte array on every 255 bytes, so byte 256 would be in a second call, and byte 512 would be in a third call, etc.  And you would increment pH each time by one (so pL would be 0 and pH would be 0, then 1, then 2). I'm not sure if that's really how it works, but I don't know why they would have an offset based system like this if it didn't work that way though.
If you try this out and you're able to use the offset to print, then we could update the code around this QR format to take in longer bytes and auto split it and set offsets for the user (basically bake this into the code itself).  That seems like a really helpful feature.  We could have test coverage around this too.
Let me know if you try this and if you run into any issues.

This document from Star makes it sound like the true data max size is 1024, and that you have to write it 255 bytes at a time to the buffer.  This is probably why it's a buffered command as well and not an immediate print command - essentially there's a 1024 byte (1KB) staging area set aside for this.  you write your data into it with multiple commands, and then you issue a print command.  If the max was 255 then there probably wouldn't be a way to feed data into a buffer at all and it would print directly from the command.
Makes me wonder if this is a general thing across the other 2d code family or if it's only relevant to PDF417 because it stores more data?
PRs are very welcome so if you get this working, please consider contributing back as a PR so others can use it!

This document from Star makes it sound like the true data max size is 1024, and that you have to write it 255 bytes at a time to the buffer. This is probably why it's a buffered command as well and not an immediate print command - essentially there's a 1024 byte (1KB) staging area set aside for this. you write your data into it with multiple commands, and then you issue a print command. If the max was 255 then there probably wouldn't be a way to feed data into a buffer at all and it would print directly from the command.
Makes me wonder if this is a general thing across the other 2d code family or if it's only relevant to PDF417 because it stores more data?
PRs are very welcome so if you get this working, please consider contributing back as a PR so others can use it!

So you think we need to run the store command multiple times (255 chars each) and then in the end print out the barcode? It does make sense, but I would assume the printer just overwrites the stored bytes with a new command being sent... I would like someone to test it, if it works I can easily submit a PR with that change.

Actually, I refreshed myself on ESCPOS notation a bit and I think this is the proper interpretation of the following image:

the confusing parts are the reuse of d1...dk and d in the definition, and the reuse of k in dk vs the k in the command specification. they also define d as being between 2 and 255 (and Epson specifies as 0 to 255, which is probably more correct?), but the important point is - this is the definition of d, not the definition of k - i.e. they are saying a byte is a byte, and d can't be > 255, which is obvious because that's the max size of an 8-bit byte, so it doesn't even really need to be said but they said it anyway and confused the issue.  Similarly they declare the value of pL and pH as minimum 0 and maximum 255 in decimal, because that's the maximum value a single byte can hold.  If you ignore all the a byte is a byte stuff, then I think the implementation becomes clearer:

The command must start with 0x1D 0x28 0x6B (Command Group)
The next 2 bytes are the data length (including a fixed overhead of 3 bytes for Sub-Command identifier)
The next 3 bytes are fixed as 0x31 0x50 0x30 (Sub-Command)
The maximum data size is 65535 minus 3

The data size (k) has a "calculated size - 3" because it is implying the above constraint that the fixed sub-command length is always included in the size length (i.e. pL + pH include the 3 bytes of the Sub-Command, they are not just the data length).
Given a data length of 629 characters as an example, this would be the calculation in pseudocode:
string data = "some ... string of 629 bytes in length"; // assume this is actually the right length
int k = data.Length;
int dataLength = k + 3; // should be 632
byte pH = dataLength / 256; // implies int division. should be 2
byte pL = dataLength % 256; // should be 120
var command = $"\x1D\x28\x6B{pL}{pH}\x31\x50\x30" + data;

the reason pL is 120 and pH is 2 is because dataLength = 629, and with the fixed sub-command, the length is 632.  2 x 256 = 512 and 512 + 120 = 632.
Hopefully that makes sense!  I have a printer I can test this on if you want to do an implementation, but I don't know what valid PDF417 data looks like.  I do see that it is some type of structured data, so we could potentially provide helpers on building the PDF417 headers.

Hmm, that actually makes sense! Ok, I can fix it then, and probably double-check other bar codes they may have mistakenly similar caps in place.

Indeed, the electronic stamps of official documents must come stamped with
a PDF417, which contains a variable paragraph of the electronic signature
of the same document, this paragraph varies between 350 and 450 characters,
in previous systems the way to solve it was to generate the PDF417 as a BMP
graph and then drawing this on the receipt as an image.
El dom, 14 ene 2024 a las 20:08, Luke Paireepinart (<
@.***>) escribió:

I see you mentioned something about your country's official ballots. Do
they have to be PDF417? The reason I ask is if PDF417 is supposed to
support larger data areas than ESCPOS currently supports then maybe we're
doing something wrong.
Another possible interpretation of the above is that you can store more
than 255 bytes, but you can only write 255 bytes to the symbol storage area
at once, and pL/pH are used as offsets, so the true max size is then 65535.
If that's the case and that's the right way to interpret this, then I think
you would split your byte array on every 255 bytes, so byte 256 would be in
a second call, and byte 512 would be in a third call, etc. And you would
increment pH each time by one (so pL would be 0 and pH would be 0, then 1,
then 2). I'm not sure if that's really how it works, but I don't know why
they would have an offset based system like this if it didn't work that way
though.
If you try this out and you're able to use the offset to print, then we
could update the code around this QR format to take in longer bytes and
auto split it and set offsets for the user (basically bake this into the
code itself). That seems like a really helpful feature. We could have test
coverage around this too.
Let me know if you try this and if you run into any issues.
—
Reply to this email directly, view it on GitHub
https://github.com/lukevp/ESC-POS-.NET/issues/253#issuecomment-1891104289,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOSEIR2WXSABIZGMAK5PKJDYORQOXAVCNFSM6AAAAABA66Y42CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJRGEYDIMRYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Luis Perez Vasquez
*Especialista en Proyectos y Soluciones *Tecnológicas
www.tokem.cl http://www.tokem.cl

@lukevp, I submitted PR #259
I double checked all barcodes, there was just a couple more that had different values based on EPSON's manual, However I've also had to use STAR's manual to get GS1 Barcode limitations (not sure way, but it's not part of EPSON's manual that I have).
I also noted the library doesn't implement (0 <= m <= 6) barcodes, which seems to be a way to have unlimited length for CODABAR, ITF and CODE39, for it seeks a NUL byte (0x00) of the command to know when it ended. However, I'd suggest testing it out to see if that tracks. And that might not even be possible when dealing with regular printing (portrait), because once the barcode exceeds the printing area (paper width when portrait), printer will simply feed paper ( per "If the horizontal size exceeds printing area, the printer only feeds the paper").


[Flow layouts] FlowRow/FlowColumn crossAxisSpacing is applied even when there is only one row/column

Description
Same as title
Steps to reproduce

Create a FlowRow and set a cross axis spacing
Add a single element to this flowRow

Expected behavior
No cross axis spacing should be applied. If we keep adding more elements to introduce a second row, then only crossAxisSpacing should be applied.
Additional context
I'll raise a PR to fix this.

Closing this as it was a misunderstanding


Create wfu.txt

Add WuFeng University Domain

@york-sun First stage of verification passed, official domain is wfu.edu.tw as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@york-sun Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Elisa/5899 upgrade storybook v7

BACKEND PULL REQUEST
Related Issue

Why is this being done? Link to issue, or a few sentences describing why this PR exists

Changes Proposed

Detailed explanation of what this PR should do

Additional Information

decisions that were made
notice of future work that needs to be done

Testing

How should reviewers verify this PR?

DATABASE PULL REQUEST
Related Issue

Why is this being done? Link to issue, or a few sentences describing why this PR exists

Changes Proposed

Detailed explanation of what this PR should do

Additional Information

decisions that were made
notice of future work that needs to be done

Testing

How should reviewers verify this PR?

DEVOPS PULL REQUEST
Related Issue

Why is this being done? Link to issue, or a few sentences describing why this PR exists

Changes Proposed

Detailed explanation of what this PR should do

Additional Information

decisions that were made
notice of future work that needs to be done

Testing

How should reviewers verify this PR?

FRONTEND PULL REQUEST
Related Issue

Why is this being done? Link to issue, or a few sentences describing why this PR exists

Changes Proposed

Detailed explanation of what this PR should do

Additional Information

decisions that were made
notice of future work that needs to be done

Testing

How should reviewers verify this PR?

Screenshots / Demos

For large changes, please pair with a designer to ensure changes are as intended

closing for now: work documented in https://github.com/CDCgov/prime-simplereport/issues/5899


fix Mac OS German demo installation "master" branch

Description
on Mac OS with language German, the default branch is "master" and not "main".
Lets set this specifically when initialising the repositories.

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[X] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Related Issue # (issue)
Closes # (issue)
Fixes # (issue)

Remove if not applicable

Screenshots

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Added to documentation?

[ ] 📜 README.md
[x] 🙅 no documentation needed

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

I was still testing this. :D But lucky for you, Gerald, it worked. :D


Missing creation panel after maya crash while moving panels

Here's the error:
# RuntimeError: Internal C++ object (PySide2.QtWidgets.QFormLayout) already deleted.
# Traceback (most recent call last):
#   File "C:/Users/Brendan/Documents/maya/2018/scripts\icarus\ui\signals.py", line 58, in publish
#     res = func(*args, **kwargs)
#   File "C:/Users/Brendan/Documents/maya/2018/scripts\icarus\ui\module.py", line 92, in _on_selection_changed
#     self._update_ui()
#   File "C:/Users/Brendan/Documents/maya/2018/scripts\icarus\ui\module.py", line 245, in _update_ui
#     clear_layout(self.form)
#   File "C:/Users/Brendan/Documents/maya/2018/scripts\icarus\ui\utils.py", line 2, in clear_layout
#     item = layout.takeAt(0)
# RuntimeError: Internal C++ object (PySide2.QtWidgets.QFormLayout) already deleted.

Removing the preference file fixed the issue.
Maybe ignore the reset window layout if we get this error?

这是错误：
# RuntimeError: Internal C++ object (PySide2.QtWidgets.QFormLayout) already deleted.
# Traceback (most recent call last):
#   File "C:/Users/Brendan/Documents/maya/2018/scripts\icarus\ui\signals.py", line 58, in publish
#     res = func(*args, **kwargs)
#   File "C:/Users/Brendan/Documents/maya/2018/scripts\icarus\ui\module.py", line 92, in _on_selection_changed
#     self._update_ui()
#   File "C:/Users/Brendan/Documents/maya/2018/scripts\icarus\ui\module.py", line 245, in _update_ui
#     clear_layout(self.form)
#   File "C:/Users/Brendan/Documents/maya/2018/scripts\icarus\ui\utils.py", line 2, in clear_layout
#     item = layout.takeAt(0)
# RuntimeError: Internal C++ object (PySide2.QtWidgets.QFormLayout) already deleted.

删除首选项文件修复了该问题。
如果我们收到此错误，可能会忽略重置窗口布局？

你好，我也遇到这个问题，请问你是如何解决的？？

@niuzhenjiang English please, we can't help you otherwise.

@niuzhenjiang英文请，我们不能帮助你。

sorry,I mean, I also encountered this problem. How can you solve it? Can you elaborate on it? Thank you.

Try to delete the content of C:\Users\Brendan\AppData\Roaming\Holistic Coders and restart maya.
This isn't ideal but should do the trick for now


Read header without loading entire file?

I'm noticing that using Filterbank to extract header parameters takes significantly longer on larger .fil files than on smaller ones. I'm using the Filterbank class to read header information from spliced 0002-resolution .fil files (all 8 files spliced together) and I've noticed that, for example, extracting fch1 takes between 0.5 and 0.6 seconds per file. In contrast, extracting fch1 from a smaller file, i.e. a non-spliced file of the same frequency resolution, takes about 0.1 seconds.
Am I correct in suspecting that if one desires to extract header information without interacting with the data, the speed should be independent of the size of the data part of the file? If so, the difference in speed suggests the entire file is being read before the header information is returned. Is there any way of getting around this?

Hi @umranhaji, the latest version should address this. The Filterbank class creates an array of frequency values, which takes more time if there are more channels. Isabel and I changed this behavior recently for extracting the 21cm data.
The quickest way would be to just use the read_header function and skip generation of the Filterbank object:
https://github.com/UCBerkeleySETI/filterbank/blob/master/filterbank.py#L290

Thank you @telegraphic!


feat(join): add join operator

Closes #1341.

I've thought about bit if it's possible to make this without exposing outerIndex as public using innersubscription management looks like
interface Context<T1, T2> {
  observable: Observable<T1 | T2>;
  subscription: Subscription;
}

class join {
  private left: Context<T1, T2>;
  private right: Context<T1, T2>;

  protected _next(value: Observable<T1 | T2>): void {
    (this.left) ? this.right = { observable: value, subscription: null } : this.left = { observable: value, subscription: null};
  }
}

rough proof of concept can be seen at https://github.com/kwonoj/RxJS/commit/1e06664eb56856ba4c9ee9edbacc5b0381e002fc, what do you think? since it's rough test code, still few topics I'd like to improve such as

not using map for managing values?
better duration completion handling?

and some others I couldn't think of.

clarification : I'm not suggesting to update PR as comment. Just kicking off discussion. Not even sure if innersubscription management is efficiently perform compare to original PR.

@kwonoj I think I can improve the solution with some of your suggestions. ;) I'll update this PR as soon as I can.
Answering some specific parts:

rough proof of concept can be seen at kwonoj@1e06664, what do you think? since it's rough test code

I was not very happy with exposing outerIndex as a public member (maybe a getter would be better?), but I think this is a useful information to have on notifyComplete() the same way it is available in notifyNext(). Is there a reason not having it on notifyComplete()?

not using map for managing values?

The items can be removed in any order depending on duration. The first item added can be the last to be removed, i.e. there is no relation between when the item is added to the list and when it is removed. We could use an array instead, but it'll require searching for the item. Map seems a more approriate solution.

better duration completion handling?

I tried to keep it as similar as possible to how RxJS 4 is handling completion to make sure it behaves similarly.

Is there a reason not having it on notifyComplete()?

I don't have clear visibility on those too. I think you may able to file an issue with compelling usecases. As similar, notifyNext didn't have innerSub as parameter and it's added recently, my poc branch utilized it.  Personally, yet not sure about having idx in error / complete.

Map seems a more approriate solution.

In general I agree, that question was just came out of curiosity if it's possible to do so.

Looks like this will need to be rebased.

Also, I think I'd like to block this until #1364 is merged. After that you'll need to convert your tests to TypeScript. Cool?

Okay, this just needs rebased now, I think.

I'll update it later today.

Just updated the PR.
Changes:

Remove D1 and D2 type parameter
Expose outerIndex of InnerSubscriber as a getter instead of make it public
Add a Context class to encapsulate the state related to each observable that is being joined
Move durationSelector subscription to a separate function (try/catch optimization)
Completion handling is simpler now due to the use of Context
Change join declaration on Observable and CoreOperators to use JoinSignature
Move unit tests to TypeScript

@kwonoj I'm still using the index/id approach but I think it's simpler now with Context.

I'm still in quite favor of not exposing outerindex completely, but that's purely my own :) change looks ok to me. Once @blesh approves, I'll check in this.

@kwonoj I kept using the index/id approach for two reasons:

I don't like very much having an Map where the key is a reference type
Without something that can identify the observable that is completing we have to "guess" in which Map the Subscription is trying to delete it from the left first and if it's not there, deleting from the right.

The first is not a strong argument as it's a matter of taste/style. But the second have performance implications. If the subscription is in the right Map, it'll walk through the whole left Map looking for a subscription that is not there.
That said, I have no problem in changing the implementation if you guys prefer not to expose the outerIndex from InnerSubscription.

@luisgabriel Yes I understand, reason I mentioned it was 'my favor', and not insist to change it :)

This is a pretty in-depth implementation. It's going to take more than a while to review. Can you please add perf tests? It's important that we make sure this hits the perf numbers well.

@blesh sure. No problem.

Hey, @luisgabriel ... sorry I haven't gotten back to this one yet.  Hey, @trxcllnt, can you give this a review if you have the time?

@blesh No problem. I didn't have time to add the perf tests yet. I'll add them as soon as a can.

Sorry, @luisgabriel, this is going to need rebased again. While this PR looks fine to me, the major hold up is I'm looking for someone that actually uses join a lot to review the PR. @mattpodwysocki, can you lend a hand? Also pinging @trxcllnt again.

@blesh I'll take some time this weekend to work on this. I have to rebase it and change the interface to accept Subscribable instead of Observable.

Ping @luisgabriel ... just checking in.

@blesh I'm having trouble with the tests after rebasing. I'll try to fix it.

@blesh Sorry for the long delay. I'm no sure when I can work on this. So I'm going to close this for now. As soon as I can, I'll revisit this patch and submit a new PR.


[Help] Setup takenote from source encounter patch-package error

Thank you for writing a wonderful tutorial on React - https://www.taniarascia.com/getting-started-with-react/
It makes me feel I want to learn from takenote project, so that I can create some "real" application using react.
I try to download master branch source code in Windows 10 environment. I then perform the following setup.
`
C:\yocto\takenote>npm i

><EMAIL_ADDRESS>postinstall C:\yocto\takenote
> patch-package

patch-package 6.2.0
Applying patches...

**ERROR** Failed to apply patch for package react-codemirror2

  This happened because the patch file patches\react-codemirror2+6.0.0.patch could not be parsed.

  If you just upgraded patch-package, you can try running:

    cd patches\react-codemirror2+6.0.0.patc
    patch -p1 -i h
    npx patch-package react-codemirror2
    cd ../..

  Otherwise, try manually creating the patch file again.

  If the problem persists, please submit a bug report:

    https://github.com/ds300/patch-package/issues/new?title=Patch+file+parse+error&body=%3CPlease+attach+the+patch+file+in+question%3E

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: `patch-package`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\yccheok\AppData\Roaming\npm-cache\_logs\2020-08-07T02_38_21_652Z-debug.log

C:\yocto\takenote>

`
The 2020-08-07T02_38_21_652Z-debug.log can be found at https://gist.github.com/yccheok/5902982d2485f46e23a66f6938a02ce6
Seem to me patch-package is the root cause, but I have 0 idea how to deal with it.
Can you provide me some hint? Thanks.
p/s I cannot find .env file after checkout the source code. Should I create that file manually in root folder, then input my CLIENT_ID and CLIENT_SECRET ? Thanks.

yes you will have to create the .env file as mentioned in the readme on setting up development enviroment


[Enhancement] Force enable replicated storage when ingestion need shuffle

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.0
[x] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 35 / 35 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/loadv2/LoadingTaskPlanner.java
5
5
100.00%
[]

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/LoadPlanner.java
22
22
100.00%
[]

:large_blue_circle:
com/starrocks/sql/InsertPlanner.java
7
7
100.00%
[]

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


dbc_dark dropdown searchable text input change

For the dbc_dark version of the dropdown, when you try to search for something, the input text is black and therefore not visible. If you add the following to the dark version css, it fixes the issue for the dark theme.
.Select input{
color:white;
}

Nice catch!  I'll fix that.
Thanks for the feedback :+1:

This is live now.  Thanks again.


Resolves Issue #1493- Add more functionalities in Typing Speed Calculator

Fixes Issue🛠
Closes #1493
Description👨💻
I added the following functionalities:-

3 levels of difficulty- easy, medium, hard. Introducing digits, special characters based on difficulty level.
Test duration of 1min, 3mins, 5 mins
Display the result in standard units- wpm(words per minute) .
Display the level of accuracy and no. of corrections made while typing.
Introduce a 'Retake Test' button. After taking the test, the results will be shown for as long as the 'Retake Test' button is not clicked. Earlier, the result of this calculator disappeared in 1.5 secs, leading to inconvenience.
Introduce a 'Reset' button, which clears the input by users so they could retype the sentence as desired.
JS covers all edge cases. Eg. - "You did not type."

Type of Change📄

[X] New feature (non-breaking change which adds functionality)

Checklist✅

[X] I am an Open Source contributor
[X] I have performed a self-review of my code
[X] My code follows the style guidelines of this project
[X] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Rakesh9100 This PR is reviewed and can be merged.

Thanks @Nihal4777 .  Over to you, @Rakesh9100

It will look good if you provide a dropdown to choose the option for both @Nidhi-Satyapriya

It will look good if you provide a dropdown to choose the option for both @Nidhi-Satyapriya

Also, we can Start the test without selecting difficulty and duration, so handle this as well!

It will look good if you provide a dropdown to choose the option for both @Nidhi-Satyapriya

Also, ensure the user selects both the difficulty and duration before starting the test!

Are you working @Nidhi-Satyapriya

Yes sir, I am. Will be present with the changes asap. @Rakesh9100

@Rakesh9100 I made the recommended changes. kindly look over it.

@Rakesh9100 Hi!  Do you approve of this now?!

@Rakesh9100 It's been long. GSSoC end is nearing. How did you find it?

@Nihal4777 @Rakesh9100 Can you review this once. It's been three weeks. There are only few days left for GSSOC to get over.

Hii @Nidhi-Satyapriya, Thank you so much for your contribution.
We really apologise for the delay caused. Don't worry your PR will get reviewed very soon within GSSoC period.


Fetch examples from their new site-webdev home

Fixes #21
cc @kwalrath

@thso - when do you expect to be able to review and merge?

Thanks!


Helm repo publication does not work

A merge should trigger publishing the chart to the gh-pages branch but it does not appear to work.

Fixed by #22


Provide authentication through an AuthenticationProvider interface

@thoas noticed that in it's current state the Authentication is a mutable process. He suggested we could enhance this by creating an AuthenticationProvider interface, implemented a TravisAuthProvider and GithubAuthProvider, to make the authentication process more plain and immutable.

Implementation Idea:
interface
type AuthenticationProvider interface {
    Authenticate() (AccessToken, error)
}

usage
travis.NewClient(TRAVIS_API_DEFAULT_URL, GithubAuthProvider("mytoken"))
// or
travis.NewClient(TRAVIS_API_DEFAULT_URL, TravisAuthProvider())

A possible implementation of it would consist in the following:
interface implementation
type GithubAuthProvider struct {
        GithubToken string
}

type TravisAuthProvider struct {
        TravisToken string
}

func (*GithubAuthProvider) Authenticate() (AccessToken, error) {
    // get and return a travis token from the api using the github token
}

func (*TravisAuthProvider) Authenticate() (AccessToken, error) {
    // return the TravisToken
}

client implementation
func (c *Client) Authenticate(auth AuthenticationProvider) error {
    c.client.Headers["Authorization"] = "token " + auth.Authenticate()
}

Spotted a problem in the implementation of this design: the GithubAuthProvider would need the client instance to query travis for a travis token. Which would actually lead to an impossible design as the client constructor would need to pass himself to the provider.

A workaround could be to add a client attribute to the provider when relevant. To pass the provider to the constructor with its client attribute uninitialized. And when calling Client.Authenticate initializing the provider client attribute, and calling its Authenticate method.
But this sounds super hacky, and introduces mutability, which was exactly the aspect of current implementation we are trying to get rid of.

Hey @tcr-ableton, do you think it would make sense to close this?


Provide option to hide values

When the user clicks the browser extension, the extension shows their account's total USD value. This can be a security issue for users who are in public locations or in a location where they don't want others to see their USD value. Trust Wallet on iOS is a good example where clicking the balance masks all balances as asterisks. Would you please add a feature where the user clicks the USD value to hide/unhide the amounts in the wallet?

Good idea! We've been discussing this recently as well 🙏

A contributor has implemented this feature and it will be available in the next release.
Thanks again @adrianocola 👏


Assigning things to things

On pp. 273-274, the book says that I[d/c] is "just like the interpretation I except it also assigns the name c to the object d". In the more official statement on the top of p. 274, it is not said c should be a new/newly interpreted name. But I was more surprised about the idea of interpretations assigning names to objects. I had expected to read it the other way round, namely, that I[d/c] also assigns to the name c the object d. At first I thought perhaps my English is deficient here, but then the book repeatedly says that truth-values are assigned to sentence letters. By analogy then, interpretations should assign objects to names, right? This is only a minor point. But it may create a little confusion in the students. For instance, an object may have several names, but in logic a name refers only to a single object – assignments are functional. (I am wondering whether it would be good to phrase the definition of an interpretation on p. 266 a little bit more formally. The "we care to consider" phrase means that interpretations are partial functions, right? Perhaps clause 2 should substitute "an assignment of a truth value" for "a truth value" in order to align it better with the other clauses.

fixed in https://github.com/rzach/forallx-yyc/commit/034ff3de5f79708339a389beb32bd8cd1604052b; thanks!


openblas: 0.3.5 -> 0.3.6

Motivation for this change
Fixes, especially for #61611.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

duplicate of https://github.com/NixOS/nixpkgs/pull/60573

Closing in favour of #60573.


Add docs for connector and download

This PR adds documentation for data access and download together with GHGA connector.

@dontseyit Thanks, I've done some more refactoring also to the repo structure and committed to this branch, good to go from my end!


High CPU cost and High Network TIME_WAIT in 0.18.20

Hi Daniels,
I'm trying the new version 0.18.20, trifecta_ui. But I found very high CPU cost and high network TIME_WAIT. I have 70+ topics. In the last version, the CPU load is less than 1, but the new version will reach to 4. And the TIME_WAIT reach 15,000 from 30. Is there any configuration I should update when upgrading?

Try using the latest version (0.19.0). No configuration properties need to change when migrating.


SSO: Create Load Testing cases

To do

implement load test cases based on the guidance from the load testing plan

Definition of Done

a set of load test cases that are runnable

Update:

functional test cases create as of https://github.com/bcgov/ocp-sso/pull/103

MVP test cases created and tested. Will continue to improve and create more. Closing this ticket.


drawio: 13.9.9 -> 14.1.5

Motivation for this change
Changelog:
https://github.com/jgraph/drawio-desktop/releases/tag/v14.1.5
https://github.com/jgraph/drawio-desktop/releases/tag/v14.1.4
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108122 run on x86_64-linux 1

1 package built:

drawio

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108122 run on x86_64-linux 1

1 package built:

drawio


Full dot found in the beginning for the csv

I don't why there is always fulldot found in the 1st word of csv in punkProse data, such as:
for test_groundtruth\0.txt
word|punctuation_before|pos|pause_before|pause_before_norm|f0_mean|f0_range|f0_birange|f0_sd|i0_mean|i0_range|i0_birange|i0_sd|speech_rate_norm|sent_tag
and|.|CC|0.1098|-0.3633|-3.6178|1.4643|3.8865|0.5531|4.3286|1.572|9.2383|0.774|-0.0917|
for test_samples\0.txt
and i was certain that's what my life would be . and when i went to college at the university of nevada , las vegas when i was eighteen , i was stunned to find that there was not a pop star onezeroone , or even a degree program for that interest .
no '.' at the beginning, I don't know why they are not sync together. Thanks for then kindly help
-dick

Hi!
I think the author(s) of https://github.com/alpoktem/punkProse can better answer this question.
Best,
Ottokar


feat: git single clone of chosen branch

run external nuts against a non-main branch.
also speeds up git cloning by using --single-branch

QA'd as part of https://github.com/forcedotcom/source-deploy-retrieve/actions/runs/4547246860


Printing object Object instead of the object and all it's properties

Challenge Waypoint: Delete Properties from a JavaScript Object has an issue.
User Agent is: Mozilla/5.0 (iPad; CPU OS 9_2 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Version/9.0 Mobile/13C75 Safari/601.1.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
var ourDog = {
  "name": "Camper",
  "legs": 4,
  "tails": 1,
  "friends": ["everything!"],
  "bark": "bow-wow"
};

delete ourDog.bark;

var myDog = {
  "name": "Happy Coder",
  "legs": 4,
  "tails": 1,
  "friends": ["Free Code Camp Campers"],
  "bark": "woof"
};

// Only change code below this line.

delete myDog.tails;

// Only change code above this line.

(function(z){return z;})(myDog);

Whats the issue?

The lesson I was playing with was how to add and remove properties from an object.  You want the print to do a toString and print all the properties to see your new property got added or removed.
Andrew
Sent from my iPhone

On Dec 24, 2015, at 10:56 AM, Mrugesh Mohapatra<EMAIL_ADDRESS>wrote:
Whats the issue?
—
Reply to this email directly or view it on GitHub.

This display function should be moved to the tail section of the JSON and hand-coded to display something friendly to the console since Object.prototype.toString just displays [object Object]

Closing as duplicate of #5097
Please see https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/Searching-for-Your-Issue-on-Github
Happy Coding!


Flutter Web failing to compile

Has happened on the latest upgrade, initially happened on master, and has now happened after reverting to beta
Logs:
C:\Users\Kamson\AndroidStudioProjects\ECS>flutter run -d chrome --verbose
[   +7 ms] executing: [C:\src\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +44 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] 2738a1148ba6c9a6114df62358109407c3ef2553
[        ] executing: [C:\src\flutter/] git tag --contains HEAD
[ +164 ms] Exit code 0 from: git tag --contains HEAD
[   +1 ms] 1.18.0-11.1.pre
[   +8 ms] executing: [C:\src\flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +25 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/beta
[        ] executing: [C:\src\flutter/] git ls-remote --get-url origin
[  +22 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +55 ms] executing: [C:\src\flutter/] git rev-parse --abbrev-ref HEAD
[  +24 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] beta
[  +42 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +2 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +13 ms] executing: C:\Users\Kamson\AppData\Local\Android\sdk\platform-tools\adb.exe devices -l
[  +30 ms] List of devices attached
           ce03171383c3c8ad0c     unauthorized transport_id:6
[   +5 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +78 ms] Found plugin cloud_firestore at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore-0.13.5\
[   +6 ms] Found plugin cloud_firestore_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore_web-0.1.1+2\
[   +6 ms] Found plugin firebase_auth at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_auth-0.16.0\
[   +2 ms] Found plugin firebase_auth_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_auth_web-0.1.2\
[   +1 ms] Found plugin firebase_core at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core-0.4.4+3\
[   +2 ms] Found plugin firebase_core_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core_web-0.1.1+2\
[   +1 ms] Found plugin firebase_storage at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_storage-3.1.5\
[   +4 ms] Found plugin flutter_keyboard_visibility at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_keyboard_visibility-0.7.0\
[   +1 ms] Found plugin flutter_plugin_android_lifecycle at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_plugin_android_lifecycle-1.0.7\
[   +4 ms] Found plugin image_picker at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\image_picker-0.6.6+1\
[   +5 ms] Found plugin path_provider at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider-1.6.8\
[   +1 ms] Found plugin path_provider_macos at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider_macos-0.0.4+2\
[  +57 ms] Found plugin cloud_firestore at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore-0.13.5\
[   +2 ms] Found plugin cloud_firestore_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore_web-0.1.1+2\
[   +5 ms] Found plugin firebase_auth at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_auth-0.16.0\
[   +2 ms] Found plugin firebase_auth_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_auth_web-0.1.2\
[   +1 ms] Found plugin firebase_core at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core-0.4.4+3\
[   +1 ms] Found plugin firebase_core_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core_web-0.1.1+2\
[   +1 ms] Found plugin firebase_storage at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_storage-3.1.5\
[   +2 ms] Found plugin flutter_keyboard_visibility at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_keyboard_visibility-0.7.0\
[        ] Found plugin flutter_plugin_android_lifecycle at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_plugin_android_lifecycle-1.0.7\
[   +3 ms] Found plugin image_picker at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\image_picker-0.6.6+1\
[   +4 ms] Found plugin path_provider at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider-1.6.8\
[   +1 ms] Found plugin path_provider_macos at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider_macos-0.0.4+2\
[  +38 ms] Generating C:\Users\Kamson\AndroidStudioProjects\ECS\android\app\src\main\java\io\flutter\plugins\GeneratedPluginRegistrant.java
[  +69 ms] Launching lib\main.dart on Chrome in debug mode...
[  +37 ms] Updating assets
[  +96 ms] Syncing files to device Chrome...
[   +8 ms] Found plugin cloud_firestore at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore-0.13.5\
[   +1 ms] Found plugin cloud_firestore_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore_web-0.1.1+2\
[   +3 ms] Found plugin firebase_auth at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_auth-0.16.0\
[   +1 ms] Found plugin firebase_auth_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_auth_web-0.1.2\
[   +1 ms] Found plugin firebase_core at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core-0.4.4+3\
[   +1 ms] Found plugin firebase_core_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core_web-0.1.1+2\
[        ] Found plugin firebase_storage at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_storage-3.1.5\
[   +2 ms] Found plugin flutter_keyboard_visibility at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_keyboard_visibility-0.7.0\
[        ] Found plugin flutter_plugin_android_lifecycle at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_plugin_android_lifecycle-1.0.7\
[   +2 ms] Found plugin image_picker at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\image_picker-0.6.6+1\
[   +3 ms] Found plugin path_provider at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider-1.6.8\
[        ] Found plugin path_provider_macos at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider_macos-0.0.4+2\
[  +13 ms] Found plugin cloud_firestore at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore-0.13.5\
[   +1 ms] Found plugin cloud_firestore_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore_web-0.1.1+2\
[   +2 ms] Found plugin firebase_auth at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_auth-0.16.0\
[   +1 ms] Found plugin firebase_auth_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_auth_web-0.1.2\
[   +1 ms] Found plugin firebase_core at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core-0.4.4+3\
[   +1 ms] Found plugin firebase_core_web at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_core_web-0.1.1+2\
[        ] Found plugin firebase_storage at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_storage-3.1.5\
[   +1 ms] Found plugin flutter_keyboard_visibility at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_keyboard_visibility-0.7.0\
[        ] Found plugin flutter_plugin_android_lifecycle at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_plugin_android_lifecycle-1.0.7\
[   +2 ms] Found plugin image_picker at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\image_picker-0.6.6+1\
[   +3 ms] Found plugin path_provider at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider-1.6.8\
[   +1 ms] Found plugin path_provider_macos at C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider_macos-0.0.4+2\
[  +11 ms] Generating C:\Users\Kamson\AndroidStudioProjects\ECS\android\app\src\main\java\io\flutter\plugins\GeneratedPluginRegistrant.java
[  +48 ms] <- reset
[   +5 ms] C:\src\flutter\bin\cache\dart-sdk\bin\dart.exe C:\src\flutter\bin\cache\artifacts\engine\windows-x64\frontend_server.dart.snapshot --sdk-root C:\src\flutter\bin\cache\flutter_web_sdk/ --incremental --target=dartdevc --debugger-module-names -Ddart.developer.causal_async_stacks=true --output-dill
C:\Users\Kamson\AppData\Local\Temp\flutter_tool.96a29a83-964d-11ea-824a-dcf5054656e8\app.dill --libraries-spec file:///C:/src/flutter/bin/cache/flutter_web_sdk/libraries.json --packages .packages -Ddart.vm.profile=false -Ddart.vm.product=false
--bytecode-options=source-positions,local-var-info,debugger-stops,instance-field-initializers,keep-unreachable-code,avoid-closure-call-instructions --enable-asserts --track-widget-creation --filesystem-root C:\Users\Kamson\AppData\Local\Temp\flutter_tools.96a29a84-964d-11ea-824a-dcf5054656e8 --filesystem-scheme org-dartlang-app --initialize-from-dill
build\cache.dill --platform file:///C:/src/flutter/bin/cache/flutter_web_sdk/kernel/flutter_ddc_sdk.dill
[   +8 ms] <- compile org-dartlang-app:/web_entrypoint.dart
[+1197 ms]
                    Compiler message:
[        ] /C:/src/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_chips_input-1.8.1/lib/src/chips_input.dart:65:7: Error: The non-abstract class 'ChipsInputState' is missing implementations for these members:
[        ]  - TextInputClient.currentAutofillScope
[        ] Try to either
[        ]  - provide an implementation,
[        ]  - inherit an implementation from a superclass or mixin,
[        ]  - mark the class as abstract, or
[        ]  - provide a 'noSuchMethod' implementation.
[        ] class ChipsInputState<T> extends State<ChipsInput<T>>
[        ]       ^^^^^^^^^^^^^^^
[        ] /C:/src/flutter/packages/flutter/lib/src/services/text_input.dart:769:21: Context: 'TextInputClient.currentAutofillScope' is defined here.
[        ]   AutofillScope get currentAutofillScope;
[        ]                     ^^^^^^^^^^^^^^^^^^^^
[+13908 ms] Syncing files to device Chrome... (completed in 15,237ms, longer than expected)
[   +1 ms] Synced 0.0MB.
[        ] Failed to compile application.
[   +3 ms] "flutter run" took 15,770ms.

#0      throwToolExit (package:flutter_tools/src/base/common.dart:14:3)
#1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:570:7)
<asynchronous suspension>
#2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:797:18)
#3      _rootRunUnary (dart:async/zone.dart:1192:38)
#4      _CustomZone.runUnary (dart:async/zone.dart:1085:19)
#5      _FutureListener.handleValue (dart:async/future_impl.dart:141:18)
#6      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:686:45)
#7      Future._propagateToListeners (dart:async/future_impl.dart:715:32)
#8      Future._completeWithValue (dart:async/future_impl.dart:526:5)
#9      _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:36:15)
#10     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:303:13)
#11     RunCommand.usageValues (package:flutter_tools/src/commands/run.dart)
#12     _rootRunUnary (dart:async/zone.dart:1192:38)
#13     _CustomZone.runUnary (dart:async/zone.dart:1085:19)
#14     _FutureListener.handleValue (dart:async/future_impl.dart:141:18)
#15     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:686:45)
#16     Future._propagateToListeners (dart:async/future_impl.dart:715:32)
#17     Future._completeWithValue (dart:async/future_impl.dart:526:5)
#18     Future._asyncCompleteWithValue.<anonymous closure> (dart:async/future_impl.dart:560:7)
#19     _rootRun (dart:async/zone.dart:1184:13)
#20     _CustomZone.run (dart:async/zone.dart:1077:19)
#21     _CustomZone.runGuarded (dart:async/zone.dart:979:7)
#22     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1019:23)
#23     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#24     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#25     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#26     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:169:5)

Doctor:
C:\Users\Kamson\AndroidStudioProjects\ECS>flutter doctor -v
[√] Flutter (Channel beta, 1.18.0-11.1.pre, on Microsoft Windows [Version 10.0.18363.778], locale en-NZ)
    • Flutter version 1.18.0-11.1.pre at C:\src\flutter
    • Framework revision 2738a1148b (28 hours ago), 2020-05-13 15:24:36 -0700
    • Engine revision ef9215ceb2
    • Dart version 2.9.0 (build 2.9.0-8.2.beta)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at C:\Users\Kamson\AppData\Local\Android\sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    X Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.6)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] IntelliJ IDEA Community Edition (version 2019.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2019.3.3
    • Flutter plugin version 44.0.3
    • Dart plugin version 193.6494.35

[√] Connected device (2 available)
    • Web Server • web-server • web-javascript • Flutter Tools
    • Chrome     • chrome     • web-javascript • Google Chrome 81.0.4044.138

! Doctor found issues in 1 category.

@jaxnz use https://pub.dev/packages/flutter_chips_input/versions/1.9.0-dev.1

@jaxnz use https://pub.dev/packages/flutter_chips_input/versions/1.9.0-dev.1

This has fixed the issue, I have requested flutter_form_builder to update the package dependencies

I'm having Failed to compile application erring (web). It was working perfectly until last friday. Did the new update from last week have any impacts toward web (for debugging mode on chrome)?

I'm having Failed to compile application erring (web). It was working perfectly until last friday. Did the new update from last week have any impacts toward web (for debugging mode on chrome)?

(just a follow up, the application is running on beta channel, and also I did all flutter upgrade and  flutter config --enable-web. Flutter doctor is all good too. Not sure, why suddenly having Failed to compile application.

If you are using flutter_form_builder and now not being able to compile -  adding the below to your pubspec file will fix the issue
flutter_chips_input: ^1.9.0-dev.1

Ohh adding that solved it! Thanks!! I wasn't thinking about that adding previously because I wasn't using flutter_chips_input.

yeah, the package flutter_form_builder runs an older version of flutter_chips_input so to get the fix you have to manually update that package to get the fix


[FEATURE] Let an extension query OpenSearch for all initialized extensions

Is your feature request related to a problem?
#108 will require extensions to be aware of what other extensions are initialized.
#102 (resolving #52 and #93) changed the initialization to only send one extension rather than the whole list.  This is good because that list didn't have any status of the other extensions.
We need to be able to have an extension query OpenSearch to either obtain a list of initialized extensions or determine whether a particular extension is initialized.
What solution would you like?
This should be a simple Request/Response transport call fetching the values in the List<DiscoveryExtension> extensionsInitializedList; and returning it.
Alternately (see #151) accept a list of uniqueIds and return a shortened list containing just those extensions.
Bonus points: do both.
What alternatives have you considered?
The ExtensionsOrchestrator could broadcast the list to all extensions after initialization is complete.  The transport would essentially be the same with request/response reversed.   However, in the case of dependent extensions, an extension may not fully respond to the initialization request until after its dependency has been initialized.
Do you have any additional context?
Eventually this feature can be embedded as part of initialization, with a polling feature to enforce initialization order of dependencies.   And hope there's no circular dependency loop!

Implementation should be similar to sendClusterSettingsRequest except the response would be a List<DiscoveryExtension> object rather than a Settings object.

[ ] Review the Meta Issue and its sub issue (#151 and #152) to get the idea about context.
[ ] Think about any possible use case will happen.
[ ] Establish a completion priority for each sub issue (#151 -> #149 ->  #152)

[x]  Establish a completion priority for each sub issue ([FEATURE] Update DiscoveryExtension to include dependency information #151 -> [FEATURE] Let an extension query OpenSearch for all initialized extensions #149 ->  [FEATURE] Relay Request/Response actions between extensions #152)

The reason why should start #151 first is because the purpose of #149 is to get initialization information based on extension's dependency, before that we should make sure the extension's dependency information which is #151 do.

The reason why should start #151 first is because the purpose of #149 is to get initialization information based on extension's dependency, before that we should make sure the extension's dependency information which is #151 do.

I like this line of thinking!  When I originally wrote the issues I tried to make them independent of each other, but doing them in the order you propose will reduce the re-writing of code.

The purpose of the #149 is to generate a request for dependency information, and send it from SDK to Opensearch thought TransportService. After the Opensearch receive the request, it will use extensionIdMap to find the List<ExtensionDependency> based on requirement, then generate a response with the List<ExtensionDependency> and send the response to SDK though TransportService. After we received the response from Opensearch, we transfer the response to List.
Opensearch-sdk-java

[ ] Add Request class ExtensionDependencyRequest for  generate a request
[ ] Add Response class ExtensionDependencyResponse for transfer the response the List
[ ] Add Handler class ExtensionDepdenceRequestHandler to prove logic process for sending dependency information request to Opensearch and receive dependency information response from Opensearch.
[ ] Register the request though TransportService by RegisterRequestHandler in ExtensionRunner

Opensearch (feature/extensions)

[ ] Add new RequestType REQUEST_EXTENSION_DEPENDENCY_INFORMATION in ExtensionsManager
[ ] Register the request though TransportService by RegisterRequestHandler in ExtensionsManager
[ ] Add case for extension dependency information request in handleExtensionRequest method in ExtensionsManager
[ ] Add Response class DependencyInformationResponse to generate response
[ ] Add logic process for using extensionIdMap to get the List<ExtensionDependency> in DependencyInformatuonResponse
[ ] Add handleResponse method for DependencyInformationResponse to use countdown latches to delay the handler return until the response is available

Add Request class ExtensionDependencyRequest for generate a request
If you create a new class for this, yes. If you use the ExtensionRequest as indicated later, then you don't need a new one; but see comments below.

Add Handler class ExtensionDepdenceRequestHandler
typo :)  Also similar to the above comment, if you use the ExtensionRequest the handler already exists and you just need to add a case to it.

Register the request though TransportService by RegisterRequestHandler in ExtensionRunner
I think the handler needs to be registered in OpenSearch, since that is the node responding to the request.  On the ExtensionsRunner side (probably in the ExtensionInitRequestHandler you will create a send....Request() method to do the transport sending and awaiting response.

Add new RequestType REQUEST_EXTENSION_DEPENDENCY_INFORMATION in ExtensionsManager
You will need both a String with a discovery prefix, and add to the enum.

Add case for extension dependency information request in handleExtensionRequest method in ExtensionsManager
The problem with using ExtensionRequest for this is that it currently only passes an enum value in the request which is used as a case in the handler.  I see two alternatives here:

Create your own request type and handler as the first two bullets, and create a separate handler for them
Modify ExtensionRequest to add an Optional<String> parameter for the uniqueId.  When it is needed, extensions can provide it; otherwise they can omit it.  @joshpalis interested in your thoughts here.

Add handleResponse method for DependencyInformationResponse to use countdown latches to delay the handler return until the response is available
This may already be taken care of inside the existing handleExtensionRequest.

@dbwiddis I agree with your first alternative of modifying ExtensionRequest and adding an Optional parameter. I think it would be best to reuse more of our existing architecture rather than add a completely new request just for the unique ID.

Also @mloufra note in the original issue we can enable returning the whole list (if we pass null instead of the uniqueID, that unfiltered list should be the default) or a filtered list, so that can work into your logic process.

Looks good with two tweaks:

Try to make the sendWhateverNameRequest() method and WhateverNameResponse class have the same name.
THe list returned should be a List<DiscoveryExtensionNode>.  It is currently the ExtensionManager field extension on line 88.  You will either return the whole list (if no uniqueID supplied so it's null in the request) or return the list filtered to only dependencies.

@joshpalis @mloufra I've been thinking about this a bit more and I think restricting to a single String overcomplicates this.
Consider that on initialization, the ExtensionsManager has already sent the ExtensionsRunner a copy of its own DiscoveryExtensionNode, which is stored in its own class variable extensionNode.  This means that the extension already knows the uniqueId values of its dependencies.
I'm thinking rather than adding an Optional<String> to the ExtensionRequest, we can send a List<String>.  This simplifies things:

I'm always iffy about passing null around and this avoids that by allowing an empty list to be passed instead (and the input/output streams already have methods for lists of strings.
In the case where a future implementation of ExtensionRequest wants a single string, we can always pass a list with size 1.
But in this case, we can pass a list of strings, specifically a list of uniqueID of extensions we want to know are initialized.  So the logic would simplify to:

start with the list of initialized extensions
if the extension request string list is empty, just return that list
if the extension request string list is not empty, filter the list such that stringList.contains(extension.getUniqueID()).

After more thought, I realized we need more than the strings; we'd need the whole object including ID and versions and potentially other information.
This information is already in a map on the ExtensionsManager object so there is really no need to spend bandwidth sending it over transport; just send the uniqueID string as a key to get exactly this information from a map.  So ignore the previous comment... I think that Optional<String> uniqueId implementation is still the best option.

Last Version of the Plan

[ ] Create a String with a discovery prefix in ExtensionManager and add this prefix in enum RequestType
[ ] Modify ExtensionRequest for Dependency request

add a new class variable private Optional uniqueId = null; (default)
add as second constructor that takes a String id parameter and assigns it this.uniqueId = Optional.of(id);
add a getter for uniqueId that does a null check (return null) or returns the optional orElse(null)
add writeable readers and writers to the stream constructor and writeTo(). The writeOptionalString() and
readOptionalString() methods already exist.

[ ] Add send...Request method in ExtensionRunner (which need a handler class to handle the response form Opensearch)
[ ] Send Asynchrom request to Opensearch threadpool. (which need to create a task for Dependency Request in thread pool)
[ ] handleExtensionRequest method will handle the request. (which need add case for Dependency prefix and need a new response class with extension, extensionIdMap, uniqueId as argument to generate response to Extension(SDK))

Looks good, @mloufra!   Only minor clarification is that the constant names you're calling "prefix" are two separate but same-named things. One is a string constant and one is an enum.    It might not have been the best idea for us to use the same names for both, but you're fine to continue this pattern.

Reopening as https://github.com/opensearch-project/OpenSearch/pull/5658 was only part of the code. Needs #300 to be complete.


got the various files when executre vidclip

execute
!vidclip --start_timestamp 0 --end_timestamp 3 --outname 'vidclip.mp4' --crf 23 'ForBiggerBlazes.mp4'
result
got the various files
e.g.
rw-r--r-- 1 root root  320950 Apr 12 01:19 vidclip_alt.mp4
-rw-r--r-- 1 root root  320950 Apr 12 01:18 vidclip.mp4

expected result
just the mention file when execute the command 'vidclip.mp4'

Could not reproduce. Please provide a repro case.

This seems to be working as intended.

no, got this after upgrade it
you need to run it twice,
solution
perhaps use -y argument to static_ffmpeg
e.g.
static_ffmpeg -y

Fixed in version 1.4.100


Hours determination fix

If value of hours is >11, then mark it's parsedTime.isFixed as true.

Done here.


Changes to implement resolution to issue-112

Merged through<PHONE_NUMBER>e437f939a77a14a46d43f6bfc39979

Renamed a few things for brevity. Otherwise great patch! Thank you very much.
@temyers all yours.

Great, thanks guys.
I'll kick off a release build.

It should be in Maven Central shortly.


Introduce new configure interface with non-optional completion

Jira issue:
https://glia.atlassian.net/browse/MOB-xxx
What was solved?
New configure interface with non-optional completion was introduced
PR also includes several improvements:

filtering out empty queueIds;
deprecating of old configure method;
replace checking interactor existence with configuration one to ensure SDK is configured;
improve error handling in TestingApp

Release notes:

[ ] Feature
[ ] Ignore
[x] Release notes (Is it clear from the description here?)
[x] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

[x] Tests fixed, added? Unit, acceptance, snapshots?
[ ] Logging necessary for future troubleshooting of customer issues added?

Screenshots:

!squash


Updates to allow custom HTML in select elements

These changes allow you to pass a placeholder and HTML content through data attributes in each option tag for select elements. When used in this manner, the value of data-placeholder will act as the text to indicate which option is selected when the select form is closed, and you can specify your own escaped/stringified HTML in data-content which will act as the selectable element when the select form is open.
Putting HTML in string form inside a data attribute is similar to the way HTML is used in tooltips/popovers in bootstrap.
Fully backwards compatible.
Example usage:

  <select id="primaryItem" class="icons">
    <option value="" disabled selected>Click to choose a product...</option>
    <option value="product1" data-icon="product1.png" data-placeholder="Awesome Product" data-content="
Awesome Product
by: Brazen Tech Mfg." class="left circle">Awesome Product</option>
    <option value="product2" data-icon="product2.png" data-placeholder="Some Other Awesome Product" data-content="
Some Other Awesome Product
by: Brazen Tech Mfg." class="left circle">Some Other Awesome Product</option>
  </select>
  Featured Product

Any questions let me know!

https://github.com/Dogfalo/materialize/issues/2709

When will this be merged/put into prod (via npm)?

@isaachinman You can easily use an override if you need this functionality right.
See gist: https://gist.github.com/dbnoble/4dbdf4a6d9be60e00674
Just load after materialize.js

Been a couple months now, any chance of this being merged soon?

:+1:

Also looking to add this type of functionality -- would like to be able to use ReactToolTip (data-tip="Text") with option dropdowns.

can confirm the override code does not work.


trailingSlash and redirects

What problem does this feature solve?
Right now if you force trailingSlash to true, all your url's will be appended a trailing slash, which is a good feature to have for SEO. But the downside of this is that if somebody links to https://example.com/directory and you have trailingSlash set to true, it will give a 404 error (thus removing all the link juice from a SEO perspective, which is bad). As people generally don't pay much attention to trailing slashes when linking, it would be awesome if this feature would offer a redirect option.
What does the proposed changes look like?
All people going to https://example.com/directory would be redirected to https://example.com/directory/ when trailingSlash set to true.

That's the trade-off (also documented in the PR and the pages).
Your webserver or hoster (e.g. Netlify) should have configurations for that, otherwise you can use sth. like https://github.com/nuxt-community/redirect-module

The point is that is very cumbersome to actually write a redirect from non-slash to slash as there are some quirks when you deal with parameters. I have tried with redirect-module, but still haven't found a way to handle property all cases (https://stackoverflow.com/questions/56100693/nuxt-js-add-trailing-slash-to-urls-with-params)
If you already have implemented the logic for the url creation, maybe it could also be used to deal with redirects. It would be awesome if trailingSlash could have an option like redirects:true (and I think it would really fit there).

redirects: true sounds like a nice idea 👍🏻

This is our latest plan: https://github.com/nuxt/nuxt/issues/15462#issuecomment-1418944206. We can track this in the linked issues.


fix(setState): Updated setState documentation

The PR is addressing this issue: https://github.com/flutter/flutter/issues/99304
Only docstring changes so no tests are needed
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

Thank you for your review. It took me some time to reword the sentences but I think that now the text gives more the idea that "setState is good, but not always"

I am going to close this for now. I don't think this is the direction we want to take.


Use multiple Where clauses when using fetch()

ReActiveAndroid Version:
latest
Bug or Feature Request:
Use multiple Where clauses when using fetch()
Description:
There is still an issue where I wish to return all (or paginated) data filtered by the query using fetch() which returns the error of Cannot bind argument at index 2 because the index is out of range. The statement has 1 parameters
All the documentation advises only of using fetchSingle() when using multiple Where clauses
I wish not to return only the first hit, but all (or some) data items.

Maybe fetch() has not been implemented for using multiple Where clauses
fetch() with multiple clauses would require List<> return type

Ok funny thing as per below, two args passed through and got results from fetch() ...
List<MyTableClass> MyTableClassList = Select.from(MyTableClass.class).where(
"id BETWEEN ? AND ?",
firstId,
lastId)
    .orderBy("id " + defaultSortBy)
    .limit(limit)
    .fetch();

however, using LIKE '?&' causes crash ...
Ill investigate further ..

@Bennith yeah, ReActiveAndroid has a problem with LIKE clause. I will try to catch and fix this bug.


How to set a virtual environment, updated version

Hello Santiago and well done! I have some remarks at your commit. Please have a look.


When I install this using my pod file and try to build it, it literally gives me like 20 errors which is a shame as it looks good

closing this as it fails to provide any constructive feedback


fix(pythonapi): fix int, float in yaml

This fixes an issue where integers and floats where put into YML as strings.

FYI your second and third commits fix issues that should have been found by the pre-commit hooks.
From the readme:
pre-commit install  # Required for developers

FYI your second and third commits fix issues that should have been found by the pre-commit hooks.
From the readme: pre-commit install # Required for developers

Yes, I am aware of this.
As I have mentioned multiple times, mypy's pre-commit hook locally behaves differently to the Action's mypy, so this is the only way I can guarantee that all checks pass. I have to ignore the local pre-commit hook

FYI your second and third commits fix issues that should have been found by the pre-commit hooks.
From the readme: pre-commit install # Required for developers

Yes, I am aware of this. As I have mentioned multiple times, mypy's pre-commit hook locally behaves differently to the Action's mypy, so this is the only way I can guarantee that all checks pass. I have to ignore the local pre-commit hook

which mypy
If you are using the global system package manager version of mypy, then there could be a problem.
If you are using the version of mypy that gets installed with the wic conda environment (i.e. the same one that gets installed in the github action) then there should not be a problem.


feature request: enable only the animations we need

It would be great to be able to pass the animations we need, like:
useAutoAnimate(['add', 'remove']);

My app uses react-beautiful-dnd and it doesn't work very well with 'remain', so in cases like this when we use other libraries with animations that might interfere with the behavior of the element, this function will be really useful.

I think this #118 is related

@yojona I use a similar library (dndkit) and solved this issue by disabling autoAnimate when updating the DOM after a dropped drag. It's a little tricky to get right, but it works, and as a nice side-effect, it preserves the "remain" animation when the list order is changing for other reasons.

It would be nice to have a bit more control over it and it's defaults.
Eg.: I want to override just add animation and keep others as default.
In theory we could modify the config to have: onAdd, onRemove, onRemain and if omitted it would pick the defaults for it. Probably solves two things at once, if user pass false to any of these, than it wouldn't animate that action.

So is there any easy way to do this now?

So is there any easy way to do this now?
I'm also trying to disable the add and remove animations and just keep the remain default animation.

I ended up forking it on our private monorepo and added the callbacks I said earlier, most likely the pull request wasn't going to be accepted so I didn't even tried to make one.


[FEATURE REQ] Return object for SendMessageAsync with success or failure state

Library name
Azure.Messaging.ServiceBus
Please describe the feature.
We had some issues on our Cloud environment where messages were not sent to the service bus. This was in fact entirely our fault, we bursted the messages towards the service bus without closing our Sender and Client connection (not using DisposeAsync()).
At a certain point, no messages were sent as described above, but also no feedback or error message was given from the call to SendMessageAsync().
We would expect if connection exhausting (which was the issue in our case) or general failure occurs in sending the message, we would get some feedback. Now we could only deduct it from watching the metrics of our service bus in Azure portal.
It would be nice to have some sort of object containing a success state when send a message.

Hi @GeertVerthe.  Thank you for reaching out and we regret that you're experiencing difficulties.  The contract for SendMessageAsync is that the sender invokes the service operation and waits for acknowledgement from the service that it has accepted the message and, therefore, responsibility for it.   If the call completes and you have not seen an error, then the operation was successful, and the service has acknowledged the message.   For any other scenario, whether a failure on the client related to the network or a failure/rejection from the service, the sender will throw, and your application will observe an exception.
The scenario that you describe above sounds like one where the sender had to wait for resources to be available for its work to be scheduled and was otherwise in an awaiting state.  Once the host had the resources to allow the sender to complete the end-to-end service operation, your application would have observed either a timeout or another network-related failure if the message was not able to be sent.


Getting a timeout issue, while trying to publish a message.

Here is my code to publish the message
Kafka::publishOn('email_topic')
      ->withBodyKey('foo', 'bar')
      ->withHeaders([
          'foo-header' => 'foo-value'
      ])
      ->send();

And I'm using credentials from confluent cloud.
This is my broker address KAFKA_BROKERS="pkc-12576z.us-west2.gcp.confluent.cloud:9092"
When I'm trying to send a message, I get this timeout error while my default timeout setting is 50000.
"message": "Your message could not be published. Flush returned with error code -185: 'Local: Timed out'",
"exception": "Junges\\Kafka\\Exceptions\\CouldNotPublishMessage",
"file": "/opt/lampp/htdocs/office/quotation-ms/vendor/mateusjunges/laravel-kafka/src/Exceptions/CouldNotPublishMessage.php",
"line": 17,

Thanks you

which version of this package and laravel are you using?

@mateusjunges
"laravel/framework": "^10.10"
"mateusjunges/laravel-kafka": "^1.13"

@mateusjunges could you please help me with that?

any one solved thisissue??

Hi 👋 I'm taking a look on this one now

Ok, I installed this package on a fresh laravel app. Created a basic consumer and was able to consume messages from confluent cloud:

This is my consumer code:
$consumer = Kafka::createConsumer(brokers: 'pkc-12576z.us-west2.gcp.confluent.cloud:9092')
    ->subscribe('confluent')
    ->withHandler(function (ConsumedMessage $message) {
        $log = [
            'body' => $message->getBody(),
            'partition' => $message->getPartition(),
            'key' => $message->getKey(),
            'topic' => $message->getTopicName(),
        ];
    })
    ->withOption('topic.metadata.refresh.interval.ms', '2000')
    ->withSasl(
        new Sasl(
            username: 'QEZLZVT5OZOAYEKI',
            password: 'is/knYsCmPzNECWuI++/B+ygHs6AjKwDhZzlS7TbHDO/aGtq+/fzVTr8pjmr/L1r',
            mechanisms: 'PLAIN',
            securityProtocol: 'SASL_SSL'
        )
    )
    ->build()
    ->onStopConsuming(function () {
        $this->line('Stop consuming now');
        $this->newLine();
    });

$consumer->consume();

I also tested producing messages from my laravel app to the kafka cluster and it works just fine. I think you are probably missing some SASL configuration.

Closing this now since I can't reproduce it.


Using cache instance instead of creating a new one always when requested

Helping applications with tons of requests to reuse created LinkTo instances. And since LinkTo
always use local variables, there are no concurrent issues.

Are these objects piling up? Is the creation of these objects costly? Did you measure any performance enhancement with this PR? It's still creating Linker instances, that has state and is way more costly than the linkToProxy.
Besides that, code looks :ship: , I'd just consider extracting the proxy instance creation to a method.

Did you measure any performance enhancement with this PR?

+1


Added next.js component hero section with 3d coin

Fixes Issue🛠

Closes #1348
Description👨💻
Important disclaimer
Added the 3d coin hero section
This is the component for next.js so we cannot preview this but someone can understand the structrure from it and can copy this structure from the github to use the component in there project...

Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[ ] New feature (non-breaking change which adds functionality)
[x] Style (non-breaking change which improves website style or formatting)
[ ] Documentation (non-breaking change which updates or adds documentation)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Rakesh9100
Kindly mrege this PR before GSSOC ends 😌

This project is not using next.js in any component addition, so this PR will not be merged @Utsavladia


Python extension loading

Issue Type: Bug
Python doesn't work. The extension is just keep loading, and it never gets loaded. How do I fix the problem?
Extension version: 2020.8.109390
VS Code version: Code 1.49.1 (58bb7b2331731bf72587010e943852e13e6fd3cf, 2020-09-16T23:21:17.533Z)
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-1000NG4 CPU @ 1.10GHz (4 x 1100)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 3

Memory (System)
8.00GB (0.14GB free)

Process Argv
--crash-reporter-id 1cc40d11-5cd6-41d6-a15e-69b8ff62efad

Screen Reader
no

VM
0%

Looks like you might have a bad update. Do the following to remove stale versions of the extension:

Uninstall Python extension (if you have pylance uninstall it first).
Close all instances of VS Code.
Go to, %USERPROFILE%/.vscode/extensions (on windows) or ~/.vscode/extensions on Linux/Mac.
Delete any folder with the name starting with ms-python.python*
Start VS Code, and install Python extension (also pylance if you uninstalled it in step 1).

If this does not fix it, we will re-open this issue.


Error due to olly.richify.URLs being null when there are no URLs to embed

This gives a "Cannot read property 'length' of null" error:
for (var urlIndex=0; urlIndex<URLs.length; urlIndex++) {
                    var url = URLs[urlIndex];
                    for (otbIndex=0; otbIndex<OTBs.length; otbIndex++) {
                        OTB = OTBs[otbIndex];
                        nOTBs.push(OTB.replace(url, "" + url + ""));
                    }
                }

when we use
olly.richify("just text, no embeddable content", document.getElementById("blob"));

The richify method is really bad and needs a complete test suite, so I've removed it from Olly until further testing can be done.


Simple Mistype

Fix the commit message, and I'll merge it in.

@Meistercoach83 the actual commit message needs to be changed, not the title of the PR. See https://github.com/meanjs/mean/blob/master/CONTRIBUTING.md for what it should look like.

@Meistercoach83 Can you change the commit message?


moving FileTarget archival methods to their own namespace.

This change is 0% functional, just code shuffled around to new smaller files.
I found a corner case defect in FileTarget with ConcurrentWrites and Rolling archival at my day job. I want to fix it, but I'm having sincere trouble reasoning about FileTarget's archival code because it feels like a spahgetti monster. Trying to refactor / extract it down to a mere ravioli monster so future work on defects in this class can be clearer.

Thanks, but I'm afraid this is in a bad timing with #1040

The type or namespace name 'Compression' does not exist in the namespace 'System.IO' (are you missing an assembly reference?) [C:\projects\nlog\src\NLog\NLog.sl4.csproj]
Command exited with code 1

Your are missing some #if
Did you used a refactoring tool? (eg resharper)

I'm patient, will be glad to wait until #1040 is complete. (So far it looks like I'll just need to carefully merge a few !WINDOWS_PHONE directives when it comes out).
No, I did not use Resharper this time (my goal was to keep the code identical to original, at least for this pull request. Resharper tends to encourage code massage which is good for code but hard on reviewers)
No good excuse for #if mistakes... I will thoroughly correct them.

#1040 has been merged and this PR has now a conflict as expected ;)
Can you fix the conflict and the also Travis error (it does work on master)

AppVeyor failed, looks like a configuration blooper. Drop me a line when it's a good time to push a whitespace-change commit to make it build again. =)

it fails due to the appveyor RegisterXamarinLicense
See: https://github.com/NLog/NLog/blob/master/appveyor.yml#L11
not sure how to make that conditional...


Safely handle servers with in-memory logs recovering from the leader

When servers with in-memory logs crash and recover and rejoin a cluster with the same leader, in-memory leader state becomes inconsistent:
https://github.com/atomix/copycat/blob/master/server/src/main/java/io/atomix/copycat/server/state/LeaderState.java#L1204
This is because the leader does not allow followers to forcefully decrement their matchIndex. This will result essentially in infinite recursion since the leader can't decrease nextIndex to the follower's initial log index (0) and so AppendRequest is never consistent with the follower's log.
It should be safe to allow followers to specify their true last log index. There is a risk that doing so can mask bugs in the algorithm, but should not itself be a bug.

I lied :-)
This is properly handled in the actual method that the leader calls in response to a failed AppendRequest:
https://github.com/atomix/copycat/blob/master/server/src/main/java/io/atomix/copycat/server/state/LeaderState.java#L1266


turns off deserialization for ints and strings

integers and strings are not wrapped in an EntityProperty.

#13420

/azp run python - tables - tests


feat(Intersections): inspect intersection between TriangulatedSurface…

…s and EdgedCurves

:tada: This PR is included in version 2.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


#6796 test failure 4: UnknownState("internal error: let statement unparseable"

This is not an issue in the main nushell repo
This issue is for a class of test failures currently affecting https://github.com/nushell/nushell/pull/6796
Failure:
thread 'filters::where_::test::test_examples' panicked at 'test parse error in `let a = {$in > 3}; [1, 2, 5, 6] | where -b $a`: UnknownState("internal error: let statement unparseable", Span { start: 0, end: 17 })', crates
/nu-command/src/example_test.rs:274:13
stack backtrace:
   0: rust_begin_unwind
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/core/src/panicking.rs:142:14
   2: nu_command::example_test::test_examples::parse
             at ./src/example_test.rs:274:13
   3: nu_command::example_test::test_examples::eval_pipeline_without_terminal_expression
             at ./src/example_test.rs:312:34
   4: nu_command::example_test::test_examples::check_example_input_and_output_types_match_command_signature
             at ./src/example_test.rs:111:17
   5: nu_command::example_test::test_examples::test_examples
             at ./src/example_test.rs:40:17
   6: nu_command::example_test::test_examples
             at ./src/example_test.rs:6:5
   7: nu_command::filters::where_::test::test_examples
             at ./src/filters/where_.rs:277:9
   8: nu_command::filters::where_::test::test_examples::{{closure}}
             at ./src/filters/where_.rs:274:5
   9: core::ops::function::FnOnce::call_once
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/core/src/ops/function.rs:248:5
  10: core::ops::function::FnOnce::call_once
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/core/src/ops/function.rs:248:5

Note the trailing space character in "into ".
Repro
git remote add dandavison git@github.com:dandavison/nushell.git
git fetch dandavison input-output-syntax-shape
git checkout dandavison/input-output-syntax-shape

cargo test -p nu-command filters::where_::test::test_examples

does the issue also repro with let a = {$in > 3}; or let a = {$in > 3}?
Could help narrow what parts of the parser might be having issues.

Thanks for looking at these @jntrnr.
Yes it does repro the same with both of those. E.g.
                example: "let a = {$in > 3};",

Narrowing down a bit more, the problem here is that here in parse_let
working_set.find_decl(b"let", &Type::Any)

is returning None.
Perhaps something to do with the way I'm setting up engine_state in test_examples?

This was simply caused by Let not being in the working_set in example_test.rs. We didn't notice, because where was accidentally missing its tests.


Fix StreamInvocationsBlockOtherInvocationsUntilTheyStartStreaming

Fixes https://github.com/aspnet/AspNetCore-Internal/issues/1874
Fixes https://github.com/aspnet/AspNetCore/issues/8006

/azp run AspNetCore-ci


Add word-is-not-own-anagram-case-insensitively

My implementation of anagrams-for suffered from this bug.

This is a rebasing of #129


Add short explanation on where to deploy a database

This MR adds a short explanation about where to deploy a Database in the cloud. There are no details about which database to deploy, or how, as it is very well explained already in the databases documentations.

@attesillanpaa what do you think? I tried to keep it simple but useful


Extractors -> listeners, trigger listeners on saved feeds automatically

This renames extractors to listeners (and creates a legacy endpoint for old extractors) as well as includes basic implementation of job feeds (saved searches) and listeners on those feeds. Lots of decisions here we should discuss.
See test_feeds.py for implementation example, still in progress and very rough.

I think the failing test might be due to ListenerIn needing to be a superclass of ExtractorInfo instead of BaseModel, or perhaps the other way around.

Will you be adding https://github.com/clowder-framework/clowder2/blob/workflow-documentation/backend/docs/source/listeners.md to this PR?

Are you still in favor or renaming Listeners to EventListeners to be a bit more specific. When looking through the code and documentation I am still wondering if Listeners is still too generic, even for a developer.

Can you add docstrings to all functions and models? We need to start requiring this on all PRs. Thanks!

Will you be adding https://github.com/clowder-framework/clowder2/blob/workflow-documentation/backend/docs/source/listeners.md to this PR?

Added with minor updates as well.

Are you still in favor or renaming Listeners to EventListeners to be a bit more specific. When looking through the code and documentation I am still wondering if Listeners is still too generic, even for a developer.

I renamed to EventListeners across the app, although "listeners.py" is still the filename for simplicity.


fix: Fixing unwanted GRID_DENSITY_MIGRATION_V1 based configs.

Description
Have made a mistake of using  GRID_DENSITY_MIGRATION_V1 to update the then config file used to declare default dimensions of widgets. removing those references since it is just a constant whose usage was supposed to be deprecated after the migration.
also have added validation checks to make sure no float values are added for a widget dimension as default.
Fixes #9946

if no issue exists, please create an issue and ask the maintainers about this first

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.

Test A
Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: fix/grid-constant-misused

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
54.97 (-0.04)
36.6 (-0.01)
34.82 (0)
55.46 (-0.04)

:red_circle:
app/client/src/sagas/OnboardingSagas.ts
16.46 (-0.21)
1.32 (0)
10.91 (0)
16.72 (-0.24)

:red_circle:
app/client/src/selectors/commentsSelectors.ts
83.61 (-1.64)
61.76 (-2.95)
73.33 (0)
88.24 (-2.35)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

:red_circle:
app/client/src/widgets/SkeletonWidget.tsx
93.33 (-0.42)
100 (0)
75 (0)
92.86 (-0.47)

@marks0351 Tested and LGTM.

/ok-to-test sha=b8b7b25

/ok-to-test sha=20d3aa5

/ok-to-test sha=20d3aa5


How to train parallel adapters?

**Could you confirm if parallel training for adapters is possible or not with the latest version?
If parallel training is not possible, what do you suggest I should do? Do I need to override a specific class / method in the package?
If parallel training is possible, would the following be the correct way to set it up?**
self.adapter_model = AutoModelWithHeads.from_pretrained("bert-base-uncased")

self.adapter_model.add_adapter('task1')
self.adapter_model.add_classification_head('task1', num_labels=len(self.id_to_task1), id2label=self.id_to_task1)
self.adapter_model.train_adapter("task1")

self.adapter_model.add_adapter('task2')
self.adapter_model.add_classification_head('task2', num_labels=len(self.id_to_task2), id2label=self.id_to_task2)
self.adapter_model.train_adapter("task2")

self.adapter_model.add_adapter('task3')
self.adapter_model.add_classification_head('task3', num_labels=len(self.id_to_task3), id2label=self.id_to_task3)
self.adapter_model.train_adapter("task3")

# load parallel adapters
self.adapter_model.set_active_adapters(ac.Parallel("task1", "task2", "task3"))

The notebook suggests that is is possible (https://github.com/adapter-hub/adapter-transformers/blob/master/notebooks/Parallel_Adapter_Inference.ipynb)
The documentation suggests that is it not (https://docs.adapterhub.ml/adapter_composition.html)

Thanks for your help!

Hey @sallymoon10, you can train parallel adapters. We will update the documentation on parallel compositions. To train a parallel composition you need a slightly different code to set it up:
self.adapter_model.add_adapter('task1')
self.adapter_model.add_classification_head('task1', num_labels=len(self.id_to_task1), id2label=self.id_to_task1)

self.adapter_model.add_adapter('task2')
self.adapter_model.add_classification_head('task2', num_labels=len(self.id_to_task2), id2label=self.id_to_task2)

model.train_adapter(Parallel("task1", "task2"))

Thanks for the help, seems to be working!
Just FYI for others, it seems like you also need to run the following to get the code running:
self.adapter_model.set_active_adapters(ac.Parallel("task1", "task2", "task_3"))


feat: migrate workspace to Nx and split migration into a separate library

Hey! I've split the migration script into a different package.
I have a deep experience with Nx, but I don't have much experience with the default Angular's splitting mechanism into projects. Tho Nx comes with everything out of the box: Jest, ESLint, Cypress, ng-packagr and lots of other handy stuff.
I've also switched from npm to yarn because it's a pain for me how npm works with binaries...

@arturovt I'm too busy to go over everything. I trust you.


Document transparent representation

This supersedes #353.

No, I was waiting to see if anyone else has anything to say.


Find matches redo

This looks much nicer I think! I even had to use edit inside git add -p.
I believe this is what you want me to do, by I'm still wondering about your initial concern about the +690 −124, which is still unchanged.

Perhaps I'm adding to the 'this PR is too big' issue, but oh well!

I also learned how important it is to make sure your tests fail when they are supposed to fail.

You should always see a test fail first before making it pass!

As far as this PR is concerned, I asked you to re-org it into modular commits so you could push them up one at a time, and ease the burden for review. Go ahead and set your HEAD to b7ec098, and check out a new branch from there like indMatches-redo-part-one, and push up a new PR.