When I save a program(source member) it erases all the date stamps from SRCDAT

Describe the bug
The date stamp column is all set to 000000 for all lines of code in the source member I edit, loosing all history.
Los problemas en espanol estan bien.
To Reproduce
Steps to reproduce the behavior:

Open a source member
Save the source member
Open on the IBM i using wrkmbrpdm (or any other source editing tool)
A. Or query the code using SQL
i.  Create Alias CODE for library/sourceFile(MemberName)
ii. Select * from Code
All likes date field(SRCDAT) is set to 000000

Expected behavior
The freshly edited lines should be updated  to today's date stamp and the old ones should remain the same.
Screenshots

Environment (please complete the following information):
|Code for IBM i version|2.2.1|
|Visual Studio Code version|1.82.2|
|Operating System|win32_x64|
Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.3.1
Db2 for IBM i (vscode-db2i): 0.4.1
Emmet (emmet): 1.0.0
Git (git): 1.0.0
Git Base (git-base): 1.0.0
GitHub (github): 0.0.1
GitHub Authentication (github-authentication): 0.0.2
IBM i Project Explorer (vscode-ibmi-projectexplorer): 1.2.2
JSON Language Features (json-language-features): 1.0.0
Merge Conflict (merge-conflict): 1.0.0
Microsoft Account (microsoft-authentication): 0.0.1
Node Debug Auto-attach (debug-auto-launch): 1.0.0
TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0
WSL: Recommender (remote-wsl-recommender): 0.0.19
Additional context

Do you have source dates enabled in the connection settings?

@gtank1
We had the same issue, until we read the documentation about the source dates:
Can be found here: Source dates prerequisites

@gtank1 We had the same issue, until we read the documentation about the source dates: Can be found here: Source dates prerequisites

Sorry to piggyback onto this issue, but: I have a couple suggestions for those docs (are they pull-request-able?):

It's not a hard prereq for QCSSID to be set properly. I am on a system with QCSSID = 65535, and I have source dates enabled. It seems to work as well as it would on a properly configured system. It would be more accurate to say that proper QCSSID is "strongly encouraged" and/or that 65535 is "not supported; use at your own risk".
We're well past the stage where diff mode is a test enhancement. It is here to stay, and from my experience, it is hands-down better than edit mode. Even users who want to exactly replicate SEU's behavior regarding dates will not get it with edit mode, despite its description. (Conceptually edit mode is SEU-style, but in practice, it doesn't work that well because the mechanism for multi-line edits is so drastically different.)

@jkyeung

It's not a hard prereq for QCSSID to be set properly. I am on a system with QCSSID = 65535, and I have source dates enabled. It seems to work as well as it would on a properly configured system. It would be more accurate to say that proper QCSSID is "strongly encouraged" and/or that 65535 is "not supported; use at your own risk".

You're correct, the documentation is a bit off talking about the system value QCCSID must be set. It is more accurate to say that the system value  or the userprofile must have a CCSID value different from 65535.
Code for IBM i uses SQL to read and write source date for source members, and SQL does not work when running in a job having CCSID 65535. The job CCSID comes from the userprofile for the job, which can point to system value QCCSID (*SYSVAL), but also can specify a specific CCSID.
So there must be a CCSID value different from 65535, but it does not need to be the system value QCCSID that is set.
Btw, the GitHub repository for the documentation is open for Pull Requests, and you are welcome to suggest some changes to the docs, if you find them insufficient or inaccurate.

Closing due to lack of response.


Watershed Algorithm added

Readme for watershed algorithm updated.
input and resultant(output) images added
Jupyter notebook for the code added. (if required can add .py file as well?)

kindly review and merge the PR.

I removed one library that I wasn't using in the code. Rest each and every Library has been used in the code.
Added sample input and output images in the notebook.

Have a look. @geekquad


lorri: unstable-2019-10-30 -> unstable-2020-01-09 (backport to nixos-19.09)

Motivation for this change
Backport of https://github.com/NixOS/nixpkgs/pull/77380 via git cherry-pick -x c976dc165b69c30f54d8fd42ec541285c598a345.
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.

@grahamc @Profpatsch PTAL


adjust_ranges never called

I'm not so sure but i guess adjust_ranges function is never called in the entire code.

Hi sdroy101. Thank you for the interest in the DashCandlestickCharting project on github. I am not supporting this code. It is just provided as is for anyone that may be struggling with having the ability to scale the view using the plotly library, but is provided as is. Sorry for not responding sooner, I have not been monitoring this code to provide support. It solved a one time issue for me where I needed to scale the view for a candlestick chart and has been working for me, but is no supported, but I share it freely. Thank you for your interest.


Fix some correctness issues in target triple validation and improve test coverage

Fixes "off by one" type error in watchOS validation - 32-bit archs should be blocked for watchOS >= 9.0, not >= 8.7. Fixes an incorrect fallthrough where invalid tvOS archs would issue errors about 32-bit iOS archs not being supported on iOS 11 and later. Fixes labeling of "macOS" as "OS X" in an error. Fixes an issue where arm64_32 would incorrectly be blocked for watchOS >= 8.7. Generally simplifies the validation code. Improves the test coverage to cover all of these scenarios.

I'd like @cyndyishida to comment on the watchOS changes though, since she's the expert here.

@swift-ci please test

@swift-ci please test

@swift-ci please test

I'd like @cyndyishida to comment on the watchOS changes though, since she's the expert here.

We worked on this together so I'm also confident in the changes in this area. Since she's OOO, I want to fix the regression from the previous PR right away.


Write a Java program to find square of a number

Description
Write a Java program to find square of a number
Input  : 5
Output : 25

How to contribute

Save the solution in FindSquareOfANumber.java file
Add FindSquareOfANumber.java file in find-square-of-a-number folder

!assign


Accordion arrow is highlighted after mouse click in Chrome

Environment
macOS 10.14.3
Chrome 71.0.3578.98
Detailed description

What version of the Carbon Design System are you using?

7.2.1, but happens also on carbon-components site.

What did you expect to happen?

Clicking with the mouse on the accordion's heading or arrow should not show any focus highlight  (AKA outline, ring) around the arrow.
Tabbing with the keyboard shows a focus highlight, as intended, though.

What happened instead?

Clicking the accordion's heading or arrow highlights the arrow SVG in Chrome (but not in FireFox).
The behavior of the arrow should be similar to that of the heading (no highlight on mouse click); it works only for heading now.
Steps to reproduce the issue

Create a simple accordion (include JS; accordion styles are not enough).
Click with a mouse on one of the accordion panes. Notice how in Chrome a rectangle highlight around the arrow is shown.

Additional information
See _accordion.scss:
.bx--accordion__item:focus .bx--accordion__arrow

Hi @Tal-Daniel thank you for reporting! As you may have imagined, this goes beyond the purview of a specific component (accordion). And while differentiating focus outline behavior between mouse vs. keyboard would definitely desirable in many contexts, doing so requires newer web standard to make the implementation straight-forward, and we are not going with non-straightforward solution for now. Hope this makes sense to you!


Add symfony/process dependancy

Fixes #24.
Ready for code review!

[x] Add basic unit tests for git_version() and is_windows().
[x] Add composer test script for running unit tests locally.

Closing in favour of #26.


Musicmanager.upload "TypeError: ord() expected string...but int found"

Encountering two (possibly unrelated) errors when uploading songs to Google Music, but songs still make it onto the service. This is the first of the two:
The error happens every time after calling MusicManager.upload(). No trancoding.
Traceback (most recent call last):
  File "/usr/lib/python2.7/logging/__init__.py", line 859, in emit
    msg = self.format(record)
  File "/usr/lib/python2.7/logging/__init__.py", line 732, in format
    return fmt.format(record)
  File "/usr/lib/python2.7/logging/__init__.py", line 471, in format
    record.message = record.getMessage()
  File "/usr/lib/python2.7/logging/__init__.py", line 331, in getMessage
    msg = str(self.msg)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/internal/python_message.py", line 983, in __str__
    return text_format.MessageToString(self)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 129, in MessageToString
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 172, in PrintMessage
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 205, in PrintField
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 239, in PrintFieldValue
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 167, in PrintMessage
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 205, in PrintField
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 239, in PrintFieldValue
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 172, in PrintMessage
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 205, in PrintField
    float_format=float_format)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_format.py", line 258, in PrintFieldValue
    out.write(text_encoding.CEscape(out_value, out_as_utf8))
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_encoding.py", line 79, in CEscape
    return ''.join(_cescape_byte_to_str[Ord(c)] for c in text)
  File "/media/sdv1/azhou890/.local/lib/python2.7/site-packages/google/protobuf/text_encoding.py", line 79, in <genexpr>
    return ''.join(_cescape_byte_to_str[Ord(c)] for c in text)
TypeError: ord() expected string of length 1, but int found
Logged from file shared.py, line 244

Code:
import os
from gmusicapi import Musicmanager
api = Musicmanager()
api.login()

musicdir = '/media/sdv1/azhou890/directory/directory/data/sync/Music'

print('Adding files')
for subdir, dirs, files in os.walk(musicdir):
  musiclist = []
  for file in files:
    _, file_ext = os.path.splitext(file)
    if file_ext == '.mp3':
      filename = os.path.join(subdir, file)
      musiclist.append(filename)
  if musiclist:
    print('Uploading Music')
    _,_,failed = api.upload(musiclist)

    print('Checking Music')
    for failed_upload in failed:
      print(failed_upload)

api.logout()

Client Info:
Python Version: 2.7.9
Pip Freeze:
Cheetah==2.4.4
Mako==1.0.0
MarkupSafe==0.23
MechanicalSoup==0.4.0
PAM==0.4.2
Pyste==0.9.10
Twisted==14.0.2
Twisted-Conch==14.0.2
Twisted-Core==14.0.2
Twisted-Lore==14.0.2
Twisted-Mail==14.0.2
Twisted-Names==14.0.2
Twisted-News==14.0.2
Twisted-Runner==14.0.2
Twisted-Web==14.0.2
Twisted-Words==14.0.2
appdirs==1.4.0
argparse==1.2.1
beautifulsoup4==4.4.1
cffi==0.8.6
characteristic==0.1.0
chardet==2.3.0
colorama==0.3.2
cryptography==0.6.1
decorator==4.0.9
deluge==1.3.11
funcsigs==1.0.1
future==0.15.2
gmusicapi==9.0.0
gpsoauth==0.2.0
html5lib==0.999
httplib2==0.9.2
iotop==0.6
mock==2.0.0
mutagen==1.31
ndg-httpsclient==0.3.2
oauth2client==2.0.2
pbr==1.9.1
ply==3.4
proboscis==<IP_ADDRESS>
protobuf==3.0.0b2
pyOpenSSL==0.14
pyasn1==0.1.7
pyasn1-modules==0.0.5
pycparser==2.10
pycrypto==2.6.1
pycryptodome==3.4
pyserial==2.6
python-apt==<IP_ADDRESS>
python-dateutil==2.5.3
python-libtorrent==1.0.5
pyxdg==0.25
requests==2.4.3
rsa==3.4.2
service-identity==1.0.0
six==1.8.0
urllib3==1.9.1
validictory==1.0.1
wheel==0.24.0
wsgiref==0.1.2
zope.interface==4.1.1

Do you still get the same traceback with the develop branch?

Yup, same traceback.

Same error here ...
▶ python --version
Python 2.7.9

▶ pip freeze
MechanicalSoup==0.4.0
Pillow==2.6.1
Pygments==2.0.1
SOAPpy==0.12.22
appdirs==1.4.0
argparse==1.2.1
beautifulsoup4==4.4.1
cffi==0.8.6
chardet==2.3.0
colorama==0.3.2
cryptography==0.6.1
decorator==4.0.9
defusedxml==0.4.1
dnspython==1.12.0
docutils==0.12
funcsigs==1.0.2
future==0.15.2
gmusicapi==10.0.0
gpsoauth==0.2.0
html5lib==0.999
httplib2==0.9.2
lxml==3.4.0
meld==3.12.1
mock==2.0.0
mutagen==1.32
ndg-httpsclient==0.3.2
numpy==1.8.2
oauth2client==2.1.0
pbr==1.9.1
pexpect==3.2
ply==3.4
proboscis==<IP_ADDRESS>
protobuf==3.0.0b2
pyOpenSSL==0.14
pyasn1==0.1.7
pyasn1-modules==0.0.8
pycparser==2.10
pycrypto==2.6.1
pycryptodome==3.4
pygobject==3.14.0
python-apt==<IP_ADDRESS>
python-dateutil==2.5.3
python-debian==0.1.27
python-debianbts==1.11
reportbug==6.6.3
reportlab==3.1.8
requests==2.4.3
roman==2.0.0
rsa==3.4.2
six==1.10.0
urllib3==1.9.1
validictory==1.0.1
virtualenv==1.11.6
wheel==0.24.0
wsgiref==0.1.2
wstools==0.4.3

Indeed, the songs are still uploaded.

No update on this? I'd really like to avoid having unnecessary error message when using the package. Makes real error detection harder.

I did some little digging about the issue.
It appears that the method called ord gets called numerous of time with strings, but once is called with a:
b'<25 bytes>' # text
<class 'future.types.newbytes.newbytes'> # type(text)

So I did some dirty lib editing and the single change in /usr/local/lib/python2.7/dist-packages/google/protobuf/text_encoding.py:
  Ord = ord if isinstance(text, six.string_types) else lambda x: x
  text = str(text)
  if as_utf8:
    return ''.join(_cescape_utf8_to_str[Ord(c)] for c in text)
  return ''.join(_cescape_byte_to_str[Ord(c)] for c in text)

fixes the issue.
I wanted to fix this at a higher level, but what bothers me is that at no point in the stacktrace a gmusicapi specific code is called. Only /usr/local/lib/python2.7/dist-packages/google/protobuf/internal/python_message.py.
One possibility though is that it became fixed in version 3.0.0b3 as specified here while gmusicapi is still using 3.0.0b2 as specified there.
But from my understanding, this is a protobuf-specific bug.

Hm. Can you try upgrading protobuf locally to see if that fixes the issue? It does seem like it's coming from protobuf - unless we're passing it something unexpected - so upgrading seems reasonable.

I wish but I'm not a pythonist and I have no idea how to do that.

pip install protobuf==3.0.0b3 ought to do it (from inside your virtualenv, if you're using one).

I get the same error and don't want to do this lib editing as @aug-riedinger mentioned.
$ python --version Python 2.7.11+

$ python --version
Python 2.7.11+

I also did the upgrade for protobuf:
$ pip freeze | grep proto protobuf==3.0.0b3

I gave the bug another try after reinstalling my system.
For sure, the issue is python 2.7 vs 3 related.
Here's another guy that has faced the issue: https://groups.google.com/forum/#!topic/protobuf/bF-KiCHwkzk
In the source code, we can now read:
  # PY3 hack: make Ord work for str and bytes:
  # //platforms/networking/data uses unicode here, hence basestring.

as if we were executing python3 code...
@simon-weber could you explain what is the point of this lib, what it is supposed to do and how it is used?
Thanks

what is the point of this lib

Protocol buffers itself? It's a Google serialization format; gmusicapi uses it to format data going to the Music Manager apis.

In the source code, we can now read:

Can you link to that? I'm not sure what you're referring to.

I wanted to fix this at a higher level, but what bothers me is that at no point in the stacktrace a gmusicapi specific code is called. Only /usr/local/lib/python2.7/dist-packages/google/protobuf/internal/python_message.py

This seems to imply it's a bug in protobuf, rather than our compiled pb2s or client code. It looks like they released version 3.0.0 recently -- maybe that's worth a try? Or we could try and ask the protobuf folks directly.

So I've run down the starting point of this in gmusicapi. I think this is something we could be able to work around on our end since it seems like protobuf is having an issue with a Python 3-like bytes object appearing on Python 2, namely the difference in indexing. I think we also end up butting our head against a protobuf issue I found. Protobuf has an open issue regarding bytearray support on Python 2, which has the same indexing behavior as bytes on Python 3.
So, I'll look into what the proper solution will be for gmusicapi this weekend.

The # PY3 hack: make Ord work for str and bytes: is there:
https://github.com/google/protobuf/blob/master/python/google/protobuf/text_encoding.py#L74
I considered opening an issue in google/protobuf, but I am absolutely incapable of reproducing the bug in another context, and I felt awkward asking for support to google for the unofficial google play music api ...

This may actually propagate up to the six module, as protobuf makes use of six.indexbytes which is what is failing them on bytearrays. If there's no proper way to solve it in gmusicapi, it will likely just require waiting for it to be solved upstream.
Or, deprecate Python 2 support and schedule removal at the end of 2017 ^_^
Wishful thinking is wishful : P

@iondulgheru ran into this just now in the linked issue.
Can you try upgrading protobuf to the latest and see if that fixes it? Hopefully they've landed a fix since we last saw this.

It seems that I have the latest version
[root@play gmusic]# pip install --upgrade protobuf
Requirement already up-to-date: protobuf in /usr/lib/python2.7/site-packages/protobuf-3.2.0-py2.7.egg

[root@play gmusic]# pip freeze | grep proto
protobuf==3.2.0

Huh. @thebigmunch do you remember anything about this? I'm at a loss.

Apparently I never linked the related protobuf issue. This is it. As I said before, that issue is for a Python 2 bytearray which has the same behavior as Python 3 bytes and bytearray objects when getting an index, it returns an integer. This is tripping us up here due to python-future backporting a Python 3-alike bytes object that gets used.
This isn't really an issue, though, as six should really just do the better thing here, which is to use a 1-length slice (i:i+1) which works on Python 2 bytes and bytearray objects and also on python-future's Python 3-alike bytes objects.
I never followed up because I was busy and assumed something would get done by someone on protobuf's end. But I should know better than that : P

@jimwd just saw another example of this in the linked issue.


icudtl.dat size

Hi, I'm looking at a few of the commands and how elinux is extending the default Flutter ones. --analyze-size caught my eye.
Understandably, it doesn't work with flutter-elinux however this is interesting:
icudtl.dat with flutter-elinux has a size of 10MB:
➜  data git:(main) ✗ ls -lh
total 10M
drwxr-xr-x 1 pak0stpc pak0stpc 140 Nov  3 12:36 flutter_assets
-rw-r--r-- 1 pak0stpc pak0stpc 10M Nov  3 12:36 icudtl.dat

Comparing this with the generic GTK wrapper:
➜  ecommerce_app git:(main) ✗ flutter build linux --analyze-size
Running "flutter pub get" in ecommerce_app...                    1,473ms

💪 Building with sound null safety 💪
Building Linux application...
▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒▒
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
  bundle                                                                   62 MB
  bundle/
    ecommerce_app                                                          28 KB
    data/
      icudtl.dat                                                          796 KB

Why would there be such a drastic difference in the size between the two files?

Thanks. Both issues need to be investigated.

Understandably, it doesn't work with flutter-elinux however this is interesting:

Fixed.

icudtl.dat is one of the flutter engine build artifacts. The engine build options may have an effect. Current build options: https://github.com/sony/flutter-embedded-linux/wiki/Building-Flutter-Engine-from-source

I'm waiting for a build to complete with:
https://github.com/flutter/engine/blob/main/ci/builders/linux_host_engine.json#L184
Not sure if that's comparable but it seemed like a good idea to start with whatever the main CI builder uses and then drone to the next option.

Hmm.... I don't get it - I always get 10M for icudtl.dat in the resulting local build regardless of architecture or options for any Linux output.
How is it that the officially released GTK build has a footprint of just a few KB :confused:

Update:
I believe this is the source of information relevant to the above observations: https://flutter-review.googlesource.com/c/recipes/+/9740
Ohhh... the GTK wrapper doesn't need to bundle fontconfig as it's already part of the GTK deps - https://github.com/flutter/engine/pull/23663#issuecomment-759848028
Anecdotal comparison with fontconfig package on my x64 tower (Plasma) being roughly 1100 KiB in size, appears to be in the right ball-park in terms of size.
I'm not sure if the above would have anything to do with a potential Wayland session for flutter-elinux but it might have something to do with the fonts not being rendered in debug mode.
I don't believe there is something else that may be discovered by this ticket, @HidenoriMatsubayashi .
Closing it down.

Thanks, @psstoyanov


Menu & MenuItem missing from archive

the compiler complains not finding import org.dominokit.domino.ui.menu.Menu; and import org.dominokit.domino.ui.menu.MenuItem;
Are they removed? If not how to resolve, if ye what are the replacements?

It was replaced by Tree, check it out here https://dominokit.github.io/domino-ui-demo/index.html?theme=indigo#tree, kindly note that you can fin the repository for domino-ui under this url https://github.com/DominoKit/domino-ui because this repository is for domino-mvp.

Yah, it took me a while to figure out which section to go to; but it is pretty cute.
Thank you guys. Keep it up.


crypto/cipher: NewCTR is too expensive for small runs

Hi,
SRTP, the security mechanism used by WebRTC, encrypts each packet individually with a different IV.  Current implementations use AES in CTR mode, with the IV derived from the packet sequence number, the track's identifier (SSRC), and the phase of the moon.  The Pion WebRTC library implements this by calling cipher.NewCTR for each packet.  This turns out to be one of he main sources of memory allocations in the library.
Pull request https://github.com/pion/srtp/pull/76 works around the problem by implementing a function that performs the equivalent of cipher.NewCTR followed by XORKeyStream without allocating the Stream structure.  However, the Pion maintainers appear to believe that the client library is not the right place to perform this kind of manipulation.
I humbly suggest that the library should include a function
func findAGoodName(block cipher.Block, iv []byte, dst, src []byte)

which performs the equivalent of NewCTR followed by XORKeyStream.  An alternative would be to add a function
func (ctr *ctr) Reset(block Block, iv []byte)

which resets a Stream to its initial state without reallocating the buffer.  Of course, due to backwards compatibility, this would not be added to the existing Stream interface, the client would need to check for it explicitly.

CC @FiloSottile

The AES-IGE mode is used in telegram:
cipher, err := aes.NewCipher(key[:]) // allocation
if err != nil {
	return nil, err
}
plaintext := make([]byte, len(data))
d := ige.NewIGEDecrypter(cipher, iv[:])
d.CryptBlocks(plaintext, data)

Telegram uses different iv and key for each message.
We can't reuse Cipher because there are no Reset method.
Should I create another issue for this?

@FiloSottile Any chance you could consider this, now that the tree is open again?

I think, that CTR mode is slow on small runs, because it precalculates 512 bytes:
https://github.com/golang/go/blob/go1.22.0/src/crypto/cipher/ctr.go#L28
Can the size of precalculated buffer be set to BlockSize? Method refill() fills the buffer with BlockSize runs of Encrypt() method anyway. If streamBufferSize is always set to BlockSize, if would improve performance of small runs (e.g. 16 byte) significantly, but should not damage performance of large runs.

I made an experiment: reduced streamBufferSize from 512 to 32 bytes. It caused ~5% slowdown.
diff --git a/src/crypto/cipher/ctr.go b/src/crypto/cipher/ctr.go
index eac8e266cf..9b93f43c69 100644
--- a/src/crypto/cipher/ctr.go
+++ b/src/crypto/cipher/ctr.go
@@ -25,7 +25,7 @@ type ctr struct {
        outUsed int
 }

-const streamBufferSize = 512
+const streamBufferSize = 32

 // ctrAble is an interface implemented by ciphers that have a specific optimized
 // implementation of CTR, like crypto/aes. NewCTR will check for this interface

Comparison of benckmarks:
$ benchstat /tmp/old.txt /tmp/new.txt
goos: linux
goarch: amd64
pkg: crypto/cipher
cpu: AMD EPYC Processor (with IBPB)
           │ /tmp/old.txt │            /tmp/new.txt             │
           │    sec/op    │   sec/op     vs base                │
AESCTR1K-3    1.485µ ± 0%   1.559µ ± 1%  +4.98% (p=0.000 n=100)
AESCTR8K-3    11.84µ ± 0%   12.46µ ± 1%  +5.25% (p=0.000 n=100)
geomean       4.193µ        4.407µ       +5.11%

           │ /tmp/old.txt │         /tmp/new.txt         │
           │     B/s      │     B/s       vs base        │
AESCTR1K-3   654.2Mi ± 0%   623.4Mi ± 1%  -4.72% (n=100)
AESCTR8K-3   659.6Mi ± 0%   626.7Mi ± 1%  -4.99% (n=100)
geomean      656.9Mi        625.0Mi       -4.85%

Now that 57d0551 has landed, it's fairly easy to work around the issue.  Please see https://github.com/pion/srtp/blob/master/crypto.go.


feat: implement migration up command

What kind of change does this PR introduce?
feature
What is the current behavior?
follow up https://github.com/supabase/cli/issues/1055#issuecomment-1535019708
What is the new behavior?
$ supabase migration up
Applying migration<PHONE_NUMBER>3745_full.sql...
Local database is up to date.

Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

40 of 61 (65.57%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 60.914%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/migration.go
6
13
46.15%

internal/migration/up/up.go
20
34
58.82%

Files with Coverage Reduction
New Missed Lines
%

internal/testing/fstest/open.go
1
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
4398

Relevant Lines:
7220

💛  - Coveralls


Created a snippet for a bunch of code for tableviews with core data to simplify coding.

Snippet will require adding IBOutlet and IBAction to code already set
up, also a button type “add” in the nav controller tab bar.

@bytor99999 thank you for this PR but I believe it's not generic enough to be merged. It may work great in your case but in my opinion it defines too much logic instead acting only as a scaffolding.


⚠ ClamAV has degraded performance

In 094c45d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 204 ms

Resolved: ClamAV performance has improved in 7f472a7.


Outline doesn't work on Jupyter Notebooks

Describe the bug
When using the default VSCode outline, we can see move into the document.
But Outline Map shows "The active editor cannot provide outline information"
To Reproduce

Create a notebook file (extension is .ipynb)
write some markdown into it

Expected behavior
I expected to get the same thing as the default VSCode outline
Screenshots

Desktop (please complete the following information):

OS: Windows 10 Enterprise
VScode version : 1.80.1
Outline Map version : 1.3.2

Edit
the outline works only when editing the markdown, and on the part I am editing

Here you can see that my title Aide mémoire VSCode isn't shown but the block I'm editing is.
Moreover, we can see the # characters, it would be awesome if they were not shown

When I use jupyter notebook in vscode, I can only display local functions and variables of the current cell.

On the left : the VSCode default outline
On the right : Outline Map extension
We clearly see the difference


delegate broadcast to cable_ready_channels

Type of PR (feature, enhancement, bug fix, etc.)
Bug fix
Description
@josefrichter discovered a bug in the CableReadyChannels proxy:
cable_ready["timeline"].inner_html(...)
cable_ready.broadcast # nothing happens

I realized that we need to delegate broadcast along with []. @hopsoft will this have unintended side-effects?
Why should this be added
Devs should be able to call cable_ready.broadcast in a Reflex and expect it to work consistently.
Checklist

[x] My code follows the style guidelines of this project
[x] Checks (StandardRB & Prettier-Standard) are passing
[x] This is not a documentation update

I'd like to take the opportunity of this PR to point out a general smell:
We're making liberal use of delegate in our codebase because it's convenient, but tends to be brittle.
Consider adding more methods to CableReady::Channels, which is likely to occur, we'll have to add it here, too. Classic shotgun surgery.
Using SimpleDelegator or Forwardable could have prevented this.

StimulusReflex::CableReadyChannels exposes both of the following methods.

#cable_ready_channels - a thread local singleton that holds a set of operations for N number of ActionCable identifiers
def cable_ready_channels
  CableReady::Channels.instance
end

#stimulus_reflex_channel - a representation of a single ActionCable identifier and its enqueued operations (the default StimulusReflex channel)
def stimulus_reflex_channel
  CableReady::Channels.instance[@stream_name]
end

Both of these objects expose broadcast and broadcast_to. Consider the following.
# returns an instance of a CableReady::Channel identified by "foo"
CableReady::Channels.instance["foo"]

# example of using the thread local singleton
CableReady::Channels.instance["foo"].inner_html(...)
CableReady::Channels.instance["bar"].inner_html(...)
CableReady::Channels.instance["baz"].inner_html(...)

# calling broadcast on a CableReady::Channel instance will only emit the broadcast for that specific identifier
# this line will only broadcast operations enqueued for the "foo" identifier
CableReady::Channels.instance["foo"].broadcast

# calling broadcast on the thread local singleton iterates all encapsulated
# channel instances and calls broadcast on each of them
# this line will broadcast all enqueued operations for the "foo", "bar", and "baz" identifiers
CableReady::Channels.instance.broadcast

StimulusReflex::CableReadyChannels currently delegates any unimplemented method calls to the default StimulusReflex channel stimulus_reflex_channel. That means that when we call cable_ready.broadcast in a reflex, it's broadcasting to the single default StimulusReflex channel instance.
I'm still undecided on what makes the most intuitive sense for developers regarding the reported problem. For example, the following examples would all work for the scenario outlined above.
cable_ready["timeline"].inner_html(...).broadcast

cable_ready["timeline"].inner_html(...)
cable_ready["timeline"].broadcast

# this is basically what's currently proposed in the PR
cable_ready["timeline"].inner_html(...)
cable_ready_channels.broadcast
# this would have the side effect of broadcasting all
# operations enqueued for all identifiers held in the thread local singleton

I'm pretty certain we don't want the behavior described in that last example, but I'm curious to hear what others think.

I agree that we don't want the behavior described in the last example.
Spontaneous thought: What about a broadcast_all or similar method?

It seems like we’re hitting some limitations of delegation here.
FWIW I think it’s just confusing that the usual CableReady methodology (that’s mentioned in the CR 101) doesn‘t work.
I think this really calls for SimpleDelegator where we could put some additional logic into the broadcast method.

Thank you for spelling this out in plain language, Nate.
My $0.02 is that given the above, I should close this PR and explain in the documentation that when using the special "Reflex version" of the cable_ready method, broadcast has to be at the end of a chain.
Given that there's nothing stopping them from using cable_ready[stream_name] if it suits them, I have no problem at all with how things currently work.
Keep in mind that they can still chain addition operations on separate lines. It's just that one of those will have to [eventually] call broadcast.

I agree with @leastbad. I think it's better to defer this in favor of a later, more exhaustive delegation refactor. We should keep an eye on it, though.


Add quick "add all" / "remove all" students from a class

Currently, the admin has to go through all the class students one by one to remove them from the class. Same goes for addition.

This would be troublesome if there are lots of students in the DB and the admin wants to create a class with all the students except a selected few.
@llaske and @tarunsinghal92 if you think it's nice feature, I can add a PR.

I don't think we need this. Classrooms are by definition small and, at worse, the teacher could just remove the classroom itself.


Add methods for setting up SQS subscription of a SNS topic

Adds class AmazonSNSSubscriptionSetup with a single public method subscribeSQSQueueToSNSTopic that sets up a SQS queue so that it receives all messages posted to a SNS topic.
Requires the topic and queue to exist before creating the subscription.
Sets up required permissions on the queue so that it can receive messages from the topic.

Ping @nzroller @mikaellothman @xiaodong-izettle

nice @oskarwiksten !

Beatiful @oskarwiksten!

Great! :+1:


Better guard ergonomics

I've started using guards in my codebase and it would be nice if they could be a little more flexible.
Things I found a bit annoying so far:

Taking a path instead of an ident so I can fully/partially scope guards instead of having to import them.
Having to provide empty parentheses when the guard does not take any parameter.

What I'm going for is something like this:
#[graphql(guard(guard::Session))]

Instead of:
#[graphql(guard(SessionGuard()))]

Thanks!

Oh also having to import async_graphql::guard::Guard for the proc macro to work is kind of annoying.

I've started using guards in my codebase and it would be nice if they could be a little more flexible.
Things I found a bit annoying so far:

Taking a path instead of an ident so I can fully/partially scope guards instead of having to import them.
Having to provide empty parentheses when the guard does not take any parameter.

What I'm going for is something like this:
#[graphql(guard(guard::Session))]

Instead of:
#[graphql(guard(SessionGuard()))]

Thanks!

Because of the limitation of procedural macros, I haven't thought of a better way to remove this annoying empty parentheses. 🙂

Ah, damn, that's understandable then, what about supporting paths instead of just an ident tho? :panda_face:

It should work, I am trying.

I have implemented it on the master branch. 🙂

You're the best! Thanks!


Convention not followed

Describe the bug
https://github.com/DataDog/datadog-api-client-go/blob/v1.1.0/api/v1/datadog/docs/SyntheticsApi.md#GetAPITest
CreateSyntheticsAPITest and CreateSyntheticsBrowserTest are the only two functions that contain Synthetics in the function name. Other functions don't have this redundancy, like GetAPITest and GetBrowserTest.
To Reproduce
Look at the docs
Expected behavior
They should all have Synthetics in the name, or don't. Don't make it unexpected.
Screenshots

Additional context
Looks like this is going to be another breaking change...

Related to https://github.com/DataDog/datadog-api-client-go/issues/1012, I don't understand how there are so many discrepancies when this library is autogenerated.
For example, this delete call doesn't accept an ID and require some kind of "body":
https://github.com/DataDog/datadog-api-client-go/blob/v1.1.0/api/v1/datadog/docs/SyntheticsApi.md#DeleteTests
But others act more like typical APIs:
https://github.com/DataDog/datadog-api-client-go/blob/v1.1.0/api/v1/datadog/docs/PagerDutyIntegrationApi.md#DeletePagerDutyIntegrationService
https://github.com/DataDog/datadog-api-client-go/blob/v1.1.0/api/v1/datadog/docs/MonitorsApi.md#DeleteMonitor
It's just so inconsistent which makes working with this library extremely difficult, especially given the fact that the docs and godocs for it are completely unnavigateable..


No INotifyPropertyChanged in MSBuild C# client generator

Is there a possibility to configure the C# client generator (from MSBuild) to generate INotifyPropertyChanged?

Use the template INPC?


scoped queries support

added support for scoped queries (select * where ...)
two bug fixes

dest_s3.connection.close() is not accesible in airflow 1.9
workaound for write_objcect_to_file when sending empty result set

@codeinvain Thanks for the PR! Lot's of good stuff here, but I also want to make sure we maintain backwards compatibility with anything that is currently relying on this hook and operator.
@benjamingregory What are your thoughts on merging this vs splitting into a new hook and operator?

@andscoop  how do i setup tests (unit) around features i fix/develop ?

@andscoop thanks , I see how to do it inside the airflow project but I'm interested on how to setup testing inside the plugin.

@andscoop No timeline right now. Warrants a larger conversation around how we want to handle across all plugins.


Implement Queue using Linked List in Java

🚀 Feature
(A clear and concise description of what the feature is.)
Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
Contribute to GSSOC'21
Pitch
In a Queue data structure, we maintain two pointers, front and rear. The front points the first item of queue and rear points to last item using Linked List Data Structure.
enQueue() This operation adds a new node after rear and moves rear to the next node.
deQueue() This operation removes the front node and moves front to the next node.

/assign


strkey: Check for leftover character to be in the expected alphabet

Found this while working on #3527

@tamirms PTAL


Support refactored DBXWebUIEnvWrapper (old DatabricksClient)

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have updated the relevant documentation.
[ ] I have added tests for new logic.

@michelhe can you approve I can safely upgrade utils here? the only new commit will be add auth method params


Interstate meeting

Issue
Interstate/intercity/international meeting locations are not working.
Expected behavior
Locations in a non-habitable place (s.a. water) are pulled to the nearest land location
Actual behavior (of issue)
The app pulls it in the middle of no where.
Steps to reproduce (issue)
Meet with a friend across state/ national lines and over a sizable body of water so that the calculated equidistant point is in water.
Proposed solutions, including designs, code, other relevant information / files
We need to have the application extend the "radius" and search for the highest rated places when the equidistant meeting location is in the "middle of nowhere"

I wrote a very long, detailed comment about this issue... and then shut down my computer. So, too long, didn't want to write:
To do this, we need to exhaust our server resources, or otherwise place strain on the client's user experience.
Essentially, what we will have to do is check the equidistant location's pixel color (source 1, 2, 3, which in many cases is CPU demanding and cumbersome.
Fortunately, that is relatively easy; the difficult part is finding the nearest coastline location. On a high level, I understand how this works: you draw coastline polygons and save it server side until you need to check for which polygon vertex is nearest the location. In practice, I have no idea where to begin; there is very little literature on the matter (only found one article), but there is a nice webapp that has completed this objective. Unfortunately, its protocol is http (not accessible by our secure app) and its API is private 😕.
Ultimately, then, we must consider the following:

Is it worth dedicating the time and resources needed to solve this problem in our current application state?

We must ponder the use cases and scenarios in which people will be searching for meeting locations across state and national lines - how common will that be, and if it occurs, how often will the bug appear? Personally, I do not feel that a person in Santa Monica will often be searching for a equidistant location with someone from Belarus - if these are businesses, the consultant almost always flies out directly to the client. Yes, it would be a nice bug to fix, but I'm not sure it's worth the resources given the negligible practical solution it invokes (though the UI would look better). That is why I have given it the label low priority, and if no other input is given within two days, this issue will be locked to outside respondents.

this is indeed a very practical issue to focus on. One of our value propositions is how effective and accurate our software is. Refining things like this is primary and should take precedence over anything else. At the end of the day, our calculations and accuracy is going to bring customers. We must establish ourselves as THE app to meet. Therefore, I think this should take precedence over any current issues that we have right now. ( Also, i get nailed with this question  a lot, so its apparent that people care)

Zipper and I have put it on the back burner for now. Of course, it would be a nice problem to solve, but we have more pressing issues and do not have the time to delegate 15 days towards solving this issue. Zipper and I will revisit implementation of it after all our objectives have passed critical state.

Requirements:

Database w/ coastlines

2 GB ram (upgrade Heroku)

client-side optimization (for CPU usage) or polygon storage (memory)


Add raw/cooked label like the text in "item identification"

Is your feature request related to a problem? Please describe.
I always can't know which pie is cook cuz the color is similar
Describe the solution you'd like
Add text similar to "item identification" to food/pie
Describe alternatives you've considered
Non
Additional context

something like this?

a current workaround you could use would be to use the inventory tags plugin and tag one of them to have a highlight

Oh yea inventory tag . Ya something like that will help a lot to know raw or cooked by a glance.


Profile tab content (first pass)

From morpho-app created by neema-orual: Radiant-Industries/morpho-app#3
Still need the UI components for user lists.

Have all the main timelines for each user done.

Note to self: The "Back" button appears too high currently. Fix this.

Comfy calling this part done (at least for a first pass, will update with lists once that UX gets done) as of today's big commit


[Draft] [#639] Remove the dependency on systemd

Proposed changes
The commit as described #639.
How to test?

Copy configuration/contrib/system/systemd/system.toml to /etc/tedge.
Try sudo tedge connect c8y and sudo tedge disconnect c8y.

What is defined in system.toml file?
Read configuration/contrib/system/README.md.
Possible error cases

system.toml is invalid toml file.
system.toml doesn't contain all required field.
typo in config (executable name): e.g. is_available = ["/bin/systemc", "--version"]
typo in config (invalid argument): e.g. is_available = ["/bin/systemctl", "----version"]
no placeholder (start, stop, enable, disable, is_active)

Decisions that is not described in #639

If the file is not given, should return error? Or should we prepare default (probably systemctl)? -> Give default using systemctl.
If case 1. gives an error, the current implementation of BuildContext is a problem. The commands like tedge cert, that doesn't require service definition, will fail due to lacking of system.toml file. (It's the reason of unit test failure) -> Move service_manager creation only in connect and disconnect.
If multiple placeholders {} are defined in the config file, should it replace all of this? -> this PR replaces all {}. Wait for the user feedback. Multiple {} sounds a user mistake.
How to test with other system managers? -> Prepare example configs with initd and openrc. Create a test with a dummy service executable and validate the output of the dummy service binary by tedge connect.
To make it general, I removed all errors related to systemd exit code. That causes a regression that the error is not well understandable. For example,

$ ./tedge connect c8y
Checking if from_file is available.

Checking if configuration for requested bridge already exists.

Validating the bridge certificates.

Saving configuration for requested bridge.

Restarting mosquitto service.

Error: failed to connect Cumulocity cloud.

Caused by:
    Service command <"/bin/systemctl restart mosquitto"> failed with code: Some(4).

-> Remove Some().
6. The file permission and owner of system.toml file? -> root:root and 444.
7. Should we remove old system_services/managers/ files? e.g. openrc.rs, bsd.rs.. I think yes. -> Yes. Translate them to the new config file format.
Types of changes
What types of changes does your code introduce to thin-edge.io?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[x] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

I did some manual tests. The feature is working fine - except for error handling that can be improved.
When there is no config, systemd is used
This is working as expected.
$ sudo rm /etc/tedge/system.toml
$ sudo tedge connect c8y
$ sudo tedge connect c8y --test
$ sudo tedge disconnect c8y
$ sudo tedge connect c8y --test

I would just prefer to see a message along the lines
"Config file `/etc/tedge/system.toml` not found => Use systemd"

rather than:
$ sudo tedge connect c8y --test
ConfigFileNotFound("/etc/tedge/system.toml")
Sending packets to check connection. This may take up to 10 seconds.

Try 1 / 2: Sending a message to Cumulocity.
Received expected response message, connection check is successful.

system.toml defines how to start/stop daemons
This is working as expected with systemd configured by system.toml
$ sudo cp configuration/contrib/system/systemd/system.toml /etc/tedge/
$ sudo tedge connect c8y
$ sudo tedge connect c8y --test
$ sudo tedge disconnect c8y
$ sudo tedge connect c8y --test

Using a custom configuration:
$ cat /etc/tedge/system.toml
name = "system V"
is_available = ["/usr/sbin/service", "--version"]
restart = ["/usr/sbin/service", "{}", "restart"]
stop = ["/usr/sbin/service", "{}", "stop"]
enable =  ["/usr/sbin/update-rc.d", "{}", "enable", "4"]
disable =  ["/usr/sbin/update-rc.d", "{}", "disable", "4"]
is_active = ["/usr/sbin/service", "{}", "status"]

and proper init scripts:
$ sudo cp configuration/init/initd/tedge-* /etc/init.d
$ sudo /usr/sbin/update-rc.d tedge-mapper-c8y defaults
$ sudo /usr/sbin/update-rc.d tedge-mapper-sm-c8y defaults
$ sudo /usr/sbin/update-rc.d tedge-agent defaults

then tedge connect is working as expected:
$ sudo tedge connect c8y
Checking if system V is available.
...

$ sudo tedge connect c8y --test
$ sudo tedge disconnect c8y
$ sudo tedge connect c8y --test

Note: this test is somehow incomplete. A proper test should be done a non-systemd box.
Error handling
This should be fixed. When the system.toml file is unreadable, the default systemd configuration is used.
An error should be reported.
$ sudo chmod a-r /etc/tedge/system.toml
$ sudo tedge connect c8y
ConfigFileNotFound("/etc/tedge/system.toml")
Checking if systemd is available.
...

Idem when the file is broken, an error should be reported and the command stopped.
Currently, the default systemd configuration is used:
$ sudo tedge connect c8y
InvalidSyntax { reason: "invalid type: boolean `false`, expected a sequence for key `is_available` at line 2 column 16" }
Checking if systemd is available.
...


Sumofsquare

closes #1190

All the PR's you made, showing same changes along with the actual PR requirement, Kindly review it

@sairish2001  Actually that might seem to be because of branching in git that every new PR includes previous commits too but in actual every PR got one feature extra. I have added the screenshots. The last commits are different in each PR. Since PR was not getting merged, and I had to complete it before deadline, so I opted for branching. :)


< warnings removed

a .. <b can now be replaced with a ..< b and x.len - 1 can be replaced with x.high, but this will do for now. thanks!

didn't know about ..<, thanks


Rendering completely broken

https://media.lidl-flyer.com/28e23461-8bf2-11e8-ab94-005056ab0fb6/Wochenprospekt-Unsere-Angebote-der-Woche-01.pdf
FF 61.0.1 Windows x64 builtin PDF.js
https://screenshots.firefox.com/YBs0YTEavfPfQ9Er/media.lidl-flyer.com
Console (not sure if related):
PDF 0a571803d6ac463387e7927265b551dc [1.6 Adobe PDF Library 15.0 / Adobe InDesign CC 2015 (Macintosh)] (PDF.js: 2.0.491) viewer.js:1628:7
Warning: Unsupported header type<PHONE_NUMBER> (pclr) pdf.worker.js:349:5
Warning: Unsupported header type<PHONE_NUMBER> (cmap) pdf.worker.js:349:5
Warning: Unsupported header type<PHONE_NUMBER> (pclr) pdf.worker.js:349:5
Warning: Unsupported header type<PHONE_NUMBER> (cmap) pdf.worker.js:349:5
Warning: Unsupported header type<PHONE_NUMBER> (pclr) pdf.worker.js:349:5
Warning: Unsupported header type<PHONE_NUMBER> (cmap) pdf.worker.js:349:5

Yes, those warnings are very much related. It looks like this is thrown by the JPX decoder (https://github.com/mozilla/pdf.js/blob/d21eecefc19907540c686c3635390150b53893ee/src/core/jpx.js#L119). Thank you for reporting this.


Add portability

This change allows code to be more portable instead of using "/n" you can use "%n" in linux it will print "/n" in windows will print "/r/n"

Hi @danrivcap
That's great! Thank you for these improvements 👍
Kr,
Mahmoud


[WIP] ZEPPELIN-138 SPARK_CLASSPATH is deprecated.

Removed exporting SPARK_CLASSPATH only.

Close this PR because of not using anymore


Remove recursion in Queue Mode

2 customers noticed that sometimes the knapsack_pro ruby process hangs on CodeShip and CircleCI and does not end work.
My bet it is because of recursion that sometimes causes ruby to hang.
In this PR we replace recursion with just while loop.

Thanks!


Question

[ X] I am running the latest version
[ X] I checked the documentation and found no answer
[ X] I checked to make sure that this issue has not already been filed
[ X] I'm sure that question is related to the library itself and not Bluetooth Low Energy or Classic in general.

Question
I'm trying to unpair the ble device from the phone.
i.e. I can scan, connect the ble device. It´s OK! No news for now. I know that there is the cancelConnection but this function cancel the connection between ble device-App (But the device is still in the paired device list in the smartphone).
I would like to know if there is any way to remove/unpair the ble device from the smartphone.
Thanks! Regards

Hi,
I have the same question
is there a way for the lib or an api to unpair a device from the system bluetooth settings?
Thanks

Hi,
You should be able to unpair the device in Bluetooth settings on your phone.

Hi @mciechanowicz,
Yes, I can do it,  but I would like to know if it is possible to unpair the device using the ble library.
Is this possible or should I do it from phone bt setting?

No, it is not possible to unpair the device using this library, so you have to do it manually.
As far as I know, I don't think iOS allows you to unpair your device through the app so it's not possible to implement.


Last navigation dropdown is out of the window

In dev17, on medium size screen:
Open the Accessories navigation dropdown. Then the dropdown content is out of the window:

The original screenshot is outdated as we have moved away from ng-bootstrap at this point, but the issue still remains and will keep coming back if not approached on a larger scope.
There is little that can be done with CSS in this case, other than customizing the columns to stack at breaking points (as per following screenshot) but then it will also depend on the customer's data:

If we take the Accelerator as a reference, there we have a dropdown menu that dynamically repositions itself depending on the viewport, but that's jQuery, we would need similar behaviour done with TypeScript for Spartacus. Do note that the solution in Accelerator doesn't comply with Accessibility Guidelines.

Provided that the Main Navigation also must include major Accessibility features as commented in GH-2853, I suggest a coordinated effort focused on requirements instead of getting in a cycle of patching the navigation here and there/now and then.
cc @Xymmer

Unsure why I was assigned - removing my assignment

Tested on dev2. It fits in the window now for different viewports.

And MyAccount dropdown menu is also ok.

Solution to this issue is JS heavy. If a css solution is possible, it would be cleaner.


Terminal profiles: should set the default profile

1) VSCode Smoke Tests (Web)
       Terminal
         Terminal Profiles
           should set the default profile:
     Error: Timeout: get element '.quick-input-widget' after 20 seconds.
      at poll (/Users/runner/work/1/s/test/automation/out/code.js:41:19)
      at async Code.waitForElement (/Users/runner/work/1/s/test/automation/out/code.js:149:16)
      at async QuickInput.waitForQuickInputClosed (/Users/runner/work/1/s/test/automation/out/quickinput.js:28:9)
      at async Terminal.runCommandWithValue (/Users/runner/work/1/s/test/automation/out/terminal.js:73:9)
      at async Context.<anonymous> (out/areas/terminal/terminal-profiles.test.js:35:13)

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=144921&view=logs&j=3792f238-f35e-5f82-0dbc-272432d9a0fb&t=00d0ae10-f5a4-5438-a3b8-c58076933cab&l=109

only failed on mac/linux so running locally on mac to see if this is because of the recent change I made (selecting powershell on windows and not providing a value for the others)

cannot repro the failure locally


Next steps for WG Chaos Engg

Hi folks - this issue is to discuss the state of the Chaos Engineering WG, with home repo at https://github.com/cncf/chaosengineering-wg. It seems there's been no activity in the WG for a long time. Is its work done and should we close it?
@prithvi1307 @umamukkara @ksatchit can you give us a quick report on the state of the WG - how are meetings going, what projects and goals are you pursuing and on what timelines? Are several chaos and resilience testing frameworks contributing - do you need help involving them?

In the last TAG general meeting we agreed to close WG Chaos now.

Need to first create a PR to archive the group, how about that PR will close/fix this issue?

We'll not following up on this issue, therefore closing it.


Remove create-hash-manifest script

GUS: https://gus.lightning.force.com/lightning/r/ADM_Work__c/a07AH000000MKi6YAG/view
Description
Small PR to clean up the create-hash-manifiest script as it is no longer used.
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

(change1)

How to Test-Drive This PR

(step1)

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

The sdk-create-hash-manifest script will be unavailable for those project who upgrades... would we consider this a breaking change?

@kevinxh I think removing the script is fine. Technically it is a breaking change but, according to the GUS, nothing calls this script after an upgrade and it has been that way since the move from JSDOM ssr to Node ssr several years back.

Instead of closing it, can we label this as breaking change and do-not-merge and park it?


[HUDI-2208] Support Bulk Insert For Spark Sql

Support bulk insert for spark sql & fix the bug for combine before insert
What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Hi @leesf , I have introduced another config key to enable the bulk insert for sql. Please take a review when you have time~

@vinothchandar @nsivabalan The PR has updated with the follow changes:
1、Allow bulk insert for pk-table.
I introduce a config:  hoodie.sql.insert.mode . If set to "strict", we will do the pk uniqueness guarantee. If set to "non-strict", we will ignore the uniqueness guarantee for pk table. The bulk insert is support in the case. By default the value is "non-strict".
2、CTAS use bulk insert by default.

@hudi-bot run azure

@hudi-bot run azure

Hi @nsivabalan , The PR has updated with the follow chanes:
add "upsert" mode for insert.mode. Currently we have 3 insert mode:

upsert
In upsert mode for insert into, duplicate record on primary key will be updated.This is the default insert mode for pk-table.
strict
In strict mode for insert into, we do the pk uniqueness guarantee for COW pk-table. For MOR pk-table, it has the same behavior with "upsert" mode.
non-strict
In non-strict mode for insert into, we use insert operation to write data which allow writing the duplicate record.

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot  run azure

thanks for your contribution! Good job 👍


Fix for double inclusions while in dev (#318)

I left the example I threw together to track down the issue in the repo. I can, of course, take it out if that seems sloppy.

Fix for #318 :point_left: to get the nice linking!

I can, of course, take it out if that seems sloppy.

I think it seems a bit sloppy, sorry! I’ll just baleet it really quick though, no need to make you do it.
Otherwise :+1: on the changes though, verified the test fails prior to the fix as well. Thanks! Will merge and publish shortly.

Published as v5.0.1. Thanks!


Upgrade to (pre-release) 1.4.0 results in all lost apps and all killed tasks

I recently upgraded a test cluster from 1.3.0-1.0.506 to 1.4.0-0.1.20161021221011.snap19 at Yelp.
It is a 3 marathon node cluster, I upgraded the cluster one node at a time.
The last node was the "important one", because after upgrading it, a new leader was elected for the first time that was on 1.4.0.
After the last node, the marathon UI starts timing out. I give it a bit, and then restart all the marathon servers to get them to re-elect or whatever, but when the interface loads, there are no apps :(
Here is my timeline:

23:30: paasta1-uswest1adevc is upgraded and restarted
23:33: paasta1-uswest1cdevc is upgraded and restarted
23:37: paasta2-uswest1adevc is upgraded and restarted
23:38: UI timeouts all around
23:43-44: I restart marathons all around
23:43: paasta1-uswest1adevc is elected leader and the UI starts responding again
23:43:22+: paasta1-uswest1adevc expunges all apps and kills all tasks with messages like:

bla exists in InstanceTracker, but not store. The run spec was likely terminated. Will now expunge
...
Killing 1 tasks for reason: Orphaned (ids: instance [kibana.platform.git1

23:45: the paasta control plane starts launching new apps to get things back in sync (~300 apps)
23:49: paasta2-uswest1adevc becomes a new leader? Not sure why.

Of course 1.4.0 is not released yet, and looking at zk it looks to have a new "on disk" state layout. I bet this is the root reason for why this upgrade failed this way. I could have taken some zk backups, but honestly Marathon works so fast, a zk restore wouldn't bring my processes back :(
I have the full log in all its glory, I can easily share via google drive if requested.
Let me know if you would like me to try to reproduce, or if you would like more data.
All apps are "flat" on the "/" namespace. (no groups)

Hi @solarkennedy,
thanks for reporting this. It would be great if you could try to reproduce this with dummy apps on your test cluster and share this logs. Please try RC3 as well, as the snapshots had known issues with migration.
Thanks

Alright, but we only saw this bug on a real cluster. (we don't do itests for marathon upgrades).
And because RC3 has a hard requirement on mesos 1.1, it means we'll have to test and roll out that upgrade first. (starting with https://github.com/Yelp/paasta/pull/942)
And on real clusters, we install marathon via debs. Can you build a trusty RC3 deb for us to use?

You can download RC3 here: https://s3.amazonaws.com/repos.mesosphere.com/ubuntu/pool/main/m/marathon/marathon_1.4.0-1.0.562.ubuntu1404_amd64.deb

@unterstein I think in 1.4.0 the runnable-jar stuff broke.
For the itests we have now (using snap19) I had to hand-make the deb and had to separate out the jar from the bin file.
Do you know if anyone else has used this deb before? Does it even work?
Here is the output when I try to run this thing:
root@41e5c43359a3:/# marathon --zk zk://zookeeper:2181/marathon --master zk://zookeeper:2181/mesos-testcluster --no-logger --env_vars_prefix MARATHON_ --mesos_authentication_principal marathon --mesos_authentication_secret_file /etc/marathon_framework_secret --mesos_authentication true
Error: Could not find or load main class mesosphere.marathon.Main

Here is the docker image used in this:
https://github.com/Yelp/paasta/blob/4e0efaf073835057eb6d180b5569c268d93eae54/yelp_package/dockerfiles/itest/marathon/Dockerfile
I've looked at this for a couple hours before hand-building the snap19 deb. I couldn't figure out why the runnable jar technique didn't work. The jar zip headers looked like they were there and everything.

@solarkennedy the migration issue might be related to https://github.com/mesosphere/marathon/issues/4873.

#4873 affects tasks with Marathon-based HealthChecks and does not explain or fix anything re orphaned tasks.

@solarkennedy,
we have issues with the linux packages, but addressed and fixed this today in https://github.com/mesosphere/marathon/pull/4890 and will include this changes to the next RC.
We also experienced this Could not find or load main class mesosphere.marathon.Main issue while trying to export the jar file from the binary manually. When downloading the jar file directly, e.g. from this package https://downloads.mesosphere.com/marathon/v1.4.0-RC3/marathon-1.4.0-RC3.tgz and extracting it afterwards, you should experience a starting marathon version with you arguments :)
Thanks for reporting and testing :)

I believe we fixed the root cause of this issue (the migration being incorrect) in a later snapshot. I started digging to find where it was yesterday but couldn't remember where exactly it happened.
We hit an issue later where the zip file for the shell script required zip64 and I think that was ultimately fixed as well.

I retested this with this setup:

Start Marathon 1.3.6
Start 1 simple cmd app with 10 tasks
Start 1 complex app with health checks with 1 task
Stop Marathon
Start Marathon 1.4.0-RC4

I see this in the logs:
Starting migration to 1.3.6
Finished 1.3.6 migration
Migration for storage: Version(1, 3, 6, LEGACY) to current: Version(1, 4, 0, LEGACY): apply change for version: Version(1, 4, 0, PERSISTENCE_STORE)
Migrated (tasks,11)(deployment plans,0)(task failures,0)(root + app versions,8)(framework-id,1)(event-subscribers,0) to new format

No task was killed.
All apps are available.

@unterstein to (finally) close the loop here, I can confirm that the 1.4.0-rc3 deb provided does work and I have a green build against it in https://github.com/Yelp/paasta/pull/943. Thanks!


Simplified code for Leviathan hard BodySlam AOE

realized that I can simplify the code for this a lot. functionality unchanged

accidently added new module to this pull request, hope it doesnt matter


Make runtime s3_bucket to hard-coded value

Temporary solution for #220 issue.
Hard-code runtime s3 region to us-west-2.
This make #220 fixed for now.

Fixed in #220, Close PR.


How to correctly use wait for animations?

I have a made animation with an object fading in and out by fading out with - 10 with 200ms wait before next fade. How would that work on different FPSes? Is there a way to use wait after every frame passed?
Here is my code

            while (takeOffSound.IsPlaying())
            {
                foreach (int i in Enumerable.Range(1, 20))
                {
                    tardis.Opacity -= 10;
                    Wait(50);
                }
                foreach (int i in Enumerable.Range(1, 20))
                {
                    tardis.Opacity += 10;
                    Wait(50);
                }
            }

The issues page is for bug reports or feature requests. Scripting questions should go to other forums (see README).
The value you pass in Wait is in milliseconds and it will pause the script for that amount of time, unaffected by FPS.


OCR for Linux

Tested and got some encouraging results; one of the documents shows lots of text. More work is needed.


Test Suit Failure

Compiling and running the latest commit in master branch.
System Info

Distro - Alpine Linux 3.2
LibC - MUSL
CC - GCC 4.9.2
Shell - ZSH 5.0.7

Log
======================================
   fwup 0.3.0: tests/test-suite.log
======================================

# TOTAL: 28
# PASS:  26
# SKIP:  0
# XFAIL: 0
# FAIL:  2
# XPASS: 0
# ERROR: 0

.. contents:: :depth: 2

FAIL: 004_env_vars
==================

fwup: can't open file-resource '/home/admin/src/fwup/tests/work/${TEST_ENV_VAR}'
FAIL 004_env_vars.test (exit status: 1)

FAIL: 005_define
================

fwup: can't open file-resource '/home/admin/src/fwup/tests/work/${TEST_DEFINE}'
FAIL 005_define.test (exit status: 1)

Do you know which version of libconfuse that you're using? See https://github.com/fhunleth/fwup#building. It really sounds like the libconfuse bug.

@fhunleth current version of confuse is 2.7 on Alpine. I take it it's a more recent revisions that fixes the bug? Should I build from their repository?

@systmkor I just checked https://github.com/martinh/libconfuse, and it still doesn't look like there's a new release tag (over a year since the fix was integrated). I've submitted an issue martinh/libconfuse#23 in hopes that the maintainer will make an official release. For now, you'll need to build from their repository.

@fhunleth  Thanks :+1:  I tried compiling from the github libconfuse head commit however it fails (i.e. the searchpath test results in a segmentation fault) when running make check

@systmkor Ugh. Are you running ./configure like at http://git.alpinelinux.org/cgit/aports/tree/main/confuse/APKBUILD? (Sorry - I'm just guessing right now since I haven't used Alpine Linux.)

@fhunleth tl;dr yes. I didn't check the configuration options since I didn't want to break other programs that depended upon that package. :clap: you google-fu :smile: I assume I need to change the ./confugre options. Also the libconfuse make check fails with and without those ./configure options.

Closing old issue. Hopefully fwup works on Alpine now, but if not, I would assume it to be a new issue since libconfuse is at 3.0 upstream now.


Using Webpack

Could you put together a very basic example using webpack?
Maybe three link example.  Two pages with the second with an update able section.
I thought I had seen one somewhere but can't find it now.

It installed fine with the package json.  But,  I am having problems getting it to work.

I'll work on an example. Can you let me know what exactly isn't working?

package.json
{
  "name": "moon-moon-router-webpack-example",
  "version": "1.0.0",
  "description": "Moon + Moon-Router + Webpack",
  "main": "scripts.js",
  "dependencies": {
    "moon-router": "0.0.5",
    "moonjs": "^0.10.0"
  },
  "devDependencies": {
    "webpack": "^2.6.1"
  }
}

webpack.config.js
module.exports = {
  entry: './scripts.js',
  output: {
    filename: 'bundle.js'
  }
}

index.html
<!DOCTYPE html>

	<title>Moon + Moon Router + Webpack</title>

<router-link to="/">Home</router-link>

<router-link to="/about">About</router-link>

<router-link to="/contact">Contact</router-link>

		<router-view></router-view>

scripts.js
const Moon = require("moonjs");
const MoonRouter = require("moon-router");

Moon.use(MoonRouter);

const router = new MoonRouter({
	"default": "/",
	"map": {
		"/": "Home",
		"/about": "About",
		"/contact": "Contact"
	}
});

Moon.component("Home", {
	template: "

Home

"
});

Moon.component("About", {
	template: "

About

"
});

Moon.component("Contact", {
	template: "

Contact

"
});

new Moon({
	el: "#app",
	router
});

Build with webpack

Ok thanks,  I was having problems getting the script.js written correctly.  I am going to get this working and practice some more.  My next question will be about component importing,  I see here that you have the "template" /HTML calls built into script.js  I think you have something written about that in the docs.  I will look it up.

This works perfectly,  Only took a sec to get it going.

FYI I am using the header component example,  so maybe you could show me how to pull a different header component in for home and and another header for about and contact?

Awesome! I'm not sure what you mean by pulling a different header component, but you can use slots as you would in Vue (not documented in Moon yet, but they work), or props (as used in the example) to change the text of the header.

I'm sorry,  I want to be able to work with the .moon like this
<template>

{{content}}

This is starter code for a multi-page app.

</template>

That above example should be working...what exactly is wrong with it?

It does work,  but in the script.js file that we have now for the pages it has this:
Moon.component("Home", {
	template: "

Home

"
}); ```
I want to be able to import or require the header.moon  in the page not "home"
you know like /home should load index.moon and so on..

Ahh, you want to use the header component in the router components. You can actually define your router components (home, about, contact) with .moon files. .moon files export a function that takes one parameter--Moon. It will register the component there.
You can do this:
home.moon
<template>

</template>

scripts.js
// at the top of the file...
require("./components/header.moon")(Moon); // register header
require("./components/home.moon")(Moon); // register home

That way you don't have to define your components in the scripts.js file, but instead you register them with .moon files.
By the way, component names are determined by their file name (or alternatively a name option). They are also case sensitive.

Let me see if I can get this working,
PS,  I am loving this project of yours and thanks for the help.

No problem, glad you like it!

so do I just change this:
Moon.component("Home", {
	template: "

Home

"
});

to this?
Moon.component("Home", {

});

Nope, you would just remove it all together, as the .moon file will register the component for you.

oh crazy easy,  let me try

I am doing something wrong:
require("./js/components/home.moon")(Moon); // register header

const Moon = require("moonjs");
const MoonRouter = require("moon-router");

Moon.use(MoonRouter);

const router = new MoonRouter({
	"default": "/",
	"map": {
		"/": "home.moon",
		"/about": "About",
		"/contact": "Contact"
	}
});

Moon.component("About", {
	template: "

About Findborg

"
});

Moon.component("Contact", {
	template: "

Contact

"
});

new Moon({
	el: "#app",
	router
});

You should register the header and home components (at least if you are using the header component), and you should add the require lines after including Moon and Moon-Router. Sorry if I didn't make that clear.
Next, change your map back to "/": "home", as it is mapping a route to the component, which is named after the file name, excluding the extension.

YES!!  I got it.

Here is what I have so others can see the simple require function:
/*=============================
  Primary Application Code
=============================*/

const Moon = require("moonjs");
const MoonRouter = require("moon-router");

require("./components/home.moon")(Moon); // register header

Moon.use(MoonRouter);

const router = new MoonRouter({
	"default": "/",
	"map": {
		"/": "home",
		"/about": "About",
		"/contact": "Contact"
	}
});

Moon.component("About", {
	template: "

About Findborg

"
});

Moon.component("Contact", {
	template: "

Contact

"
});

new Moon({
	el: "#app",
	router
});

Awesome!

Sorry so long since last seen.  Anyway,  I am back playing with this.  I created an About.moon component so that I could just require it instead of trying to do it in one file.  However, it is not loading.  Have a look.
/*=============================
  Primary Application Code
=============================*/

const Moon = require("moonjs");
const MoonRouter = require("moon-router");

require("./components/home.moon")(Moon); // register header
require("./components/About.moon")(Moon);

Moon.use(MoonRouter);

const router = new MoonRouter({
	"default": "/",
	"map": {
		"/": "home",
		"/about": "About",
		"/contact": "Contact"
	}
});

Moon.component("Contact", {
	template: "
<router-link to=\"/about\">About</router-link>
Contact

"
});

new Moon({
	el: "#app",
	router
});


fix: check cloud init status

Applicable spec: N/A
Overview
hotfix: check cloud-init status to see if job is already done before health check.
Rationale

Juju Events Changes

Module Changes

Library Changes

Checklist

[x] The charm style guide was applied
[x] The contributing guide was applied
[x] The changes are compliant with ISD054 - Managing Charm Complexity
[x] The documentation is generated using src-docs
[x] The documentation for charmhub is updated.
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

Do you need to fix the target branch?

Do you need to fix the target branch?

What do you mean by fixing the target branch please?

I'm guessing this isn't needed anymore?

@jdkandersson Yup, thanks for the comment, closing!


[WIP] Scene Import

For conceptual description refer JS PR: https://github.com/tangrams/tangram/pull/278
TODO:

[x] Maintain relative paths on imported assets, example normalize texture paths with respect to the import scene file, etc
Network request for imported scenes and handle downloading of all the referenced assets in the scene. (to be done in a separate PR, as this will require more work on the networking side of things)
[ ] Unit tests

@karimnaaji you want to have a look at this?
The only thing left for the basic import (non network) is to add unit tests.

Sure, I will start to look at this today.

@tallytalwar , this looks good to me! Last thing would be unit tests for some edge cases and clarifying the comments above.

https://github.com/tangrams/tangram-es/commit/e7c3b696cb6d9da4ba19d2f45f1794a6163aa690 looks good!

Closing in favor of #776.


use get-job to ensure that exists, before create it

every time that puppet agent run, I have an error, but I create the job from command line, it works
+++ /tmp/puppet-file20170810-14928-v875wx	2017-08-10 17:44:00.679981309 +0200
@@ -34,4 +34,3 @@

-

This one can be closed


[POC] Inject via marshaller and unmarshaller

This PR demonstrates injecting the JIT without adding any explicit hooks in Marshmallow. The tests in this test suite all pass, but I didn't patch the upstream vendored tests, as the configuration mechanism is now different.
This allows eliminating the custom Marshmallow fork.

Oops, accidentally deleted the branch. Didn't mean to do that. This PR is still as intended.

Got the benchmarks working too.
I'm not sure how to run the Marshmallow test suite with this API, though. I'm sure I could figure something out.
Still, how does this approach look? This seems easier to integrate, as it removes the "must uninstall Marshmallow" requirement.

@rowillia Any thoughts here?

@rowillia Ping! Does this seem like an interesting avenue to pursue?


v1.10.0 throws "To is not a function"

Hi all,
I'm facing a problem with the v1.10.0, it is throwing:
TypeError: To is not a function
    at eval (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:227343)
    at di (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:21359)
    at Generator.eval [as _invoke] (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:21109)
    at Generator.Ii.forEach.e.<computed> [as next] (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:21733)
    at Kn (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:19318)
    at o (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:19522)
    at eval (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:19583)
    at new Promise (<anonymous>)
    at eval (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:19462)
    at eval (webpack-internal:///./node_modules/commerce-sdk-isomorphic/lib/index.esm.js:19:227606)
    at eval (webpack-internal:///./app/commerce-api/index.js:171:35)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)

Steps to reproduce:

Fresh install retail app using pwa-kit-create-app
Remove package-lock.json
Remove node_modules directory
Run npm i
Run npm start

A workaround is to lock the version specifically to 1.9.0

Thank you @sorioinc for raising the issue, I am working to fix this.

Hello @sorioinc , please consume the v1.10.1 patch version that fixes this bug. https://www.npmjs.com/package/commerce-sdk-isomorphic


Android device as  a beacon [Question]

Is there the way to use Android device as beacon?

Yes, with Android L it is possible to act as a beacon. It is called peripheral mode.
See Android L changelog: https://developer.android.com/about/versions/android-5.0.html

Can I realize it in my app with your SDK?

At the moment no. You can use our iOS app to do so.


view pdf with hight quality

Attach (recommended) or Link to PDF file here:
compressed.tracemonkey-pldi-09.pdf
Configuration:

Web browser and its version: Chrome, firefox, ...
Operating system and its version: Ubuntu 18.04
PDF.js version: 2.2.117
Is a browser extension:

Steps to reproduce the problem:

How I can render pdf file with high quality like

not like

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension): https://mozilla.github.io/pdf.js/web/viewer.html
How I can do that?
Thankiu so much !

Zooming above 400% triggers canvas scaling, which makes it blurry because rendering canvasas this large causes memory issues. One of the ways to solve this is to use the SVG back-end which can zoom practically infinitely just like Adobe Reader, but that is experimental at this point in time.


feat(icons): add miniPinIcon

Overview

adds miniPinIcon

PR Checklist

[x] Related to designs: https://www.figma.com/file/kUthvk7lekAv5tk2BDDtLZ/🍎-Dashboard-updates?node-id=46%3A14159&mode=dev
[ ] Related to JIRA ticket: [ABC-123]
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Go to the Storybook icon preview
Search for miniPinIcon

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

🚀 Styleguide deploy preview ready!https://65fb32534405a80896ae6eac--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@55.14.2-alpha.973f7f.0@codecademy/gamut-icons@9.19.1-alpha.973f7f.0@codecademy/gamut-kit@0.6.390-alpha.973f7f.0@codecademy/styleguide@66.14.2-alpha.973f7f.0


[Bug] Can't destroy 1 specific item with Dejunk

Tried to add this item to Dejunk but it wouldn't let me add it for some reason https://www.wowhead.com/item=169687/fragment-of-zemlans-lost-treasure-map
I can destroy it if I do it myself though, but not through Dejunk.

I see these items that has the same border can't actually be manually destroyed so that's maybe why it was blocked in Dejunk

Yep. Dejunk only allows for specific item qualities to be added, and all others are denied (i.e., Legendary, Artifact, and WoWToken). As far as I know, there isn't a way to check if an item can actually be destroyed or not, so I just based it on item quality.
https://github.com/moody/Dejunk/blob/0082ae9bcf1c8725e50479e96af9db0827c745e7/src/items.lua#L187-L208

I see, well can close this issue now. I don't have the items anymore anyway and don't think many people will run into this scenario again :P


Crash : java.lang.ArithmeticException: divide by zero

Description of the problem:
Which library version are you using? 3.5.0.
Which phone/tablet are you using, and which Android version does it run? (e.g. Samsung Galaxy S5,
Android 5.0)
Samsung - Android 7
Does the same happen on other devices or an emulator? No
Can you reproduce the issue in the sample project included with the library? If not, can you
provide your own sample project or sample code that produces this error? No
In the case of an error do you have a stack trace or adb logs?
Fatal Exception: java.lang.ArithmeticException: divide by zero
       at com.journeyapps.barcodescanner.CameraPreview.calculateFrames(SourceFile:389)
       at com.journeyapps.barcodescanner.CameraPreview.previewSized(SourceFile:461)
       at com.journeyapps.barcodescanner.CameraPreview.access$300(SourceFile:59)
       at com.journeyapps.barcodescanner.CameraPreview$3.handleMessage(SourceFile:199)
       at android.os.Handler.dispatchMessage(Handler.java:98)
       at android.os.Looper.loop(Looper.java:154)
       at android.app.ActivityThread.main(ActivityThread.java:6692)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1468)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1358)

Maybe same problem.
I got the bug report from Crashlytics at Nov 17 2017 18:36 (UTC).
I'm not sure it was caused by the library or a mistake in my code.
Description of the problem:
Which library version are you using? 3.5.0.
Which phone/tablet are you using, and which Android version does it run? (e.g. Samsung Galaxy S5,
Android 5.0)
Galaxy A8 - Android 6.0.1
Does the same happen on other devices or an emulator? No
Can you reproduce the issue in the sample project included with the library? If not, can you
provide your own sample project or sample code that produces this error? No
In the case of an error do you have a stack trace or adb logs?
Fatal Exception: java.lang.ArithmeticException: divide by zero
at com.journeyapps.barcodescanner.CameraPreview.calculateFrames(CameraPreview.java:8389)
at com.journeyapps.barcodescanner.CameraPreview$3.handleMessage(CameraPreview.java:199)
at android.os.Handler.dispatchMessage(Handler.java:98)
at android.os.Looper.loop(Looper.java:148)
at android.app.ActivityThread.main(ActivityThread.java:7325)
at java.lang.reflect.Method.invoke(Method.java)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)

I am also facing this issue. Any fix for this.

Please let me know if is there any fix for this issue

Hello @rkistner , should we wait for changes?

Same problem here. Library version: 3.6.0
Fatal Exception: java.lang.ArithmeticException divide by zero com.journeyapps.barcodescanner.CameraPreview.calculateFrames (SourceFile:385) com.journeyapps.barcodescanner.CameraPreview.previewSized (SourceFile:469) com.journeyapps.barcodescanner.CameraPreview.access$300 (SourceFile:60) com.journeyapps.barcodescanner.CameraPreview$3.handleMessage (SourceFile:200) android.os.Handler.dispatchMessage (Handler.java:101) android.os.Looper.loop (Looper.java:164) android.app.ActivityThread.main (ActivityThread.java:6944) java.lang.reflect.Method.invoke (Method.java) com.android.internal.os.Zygote$MethodAndArgsCaller.run (Zygote.java:327) com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1374)

facing the same problem

+1
Same issue here. Are you accepting pull requests?

same here.

We are facing exactly the same bug at random times with Samsung Galaxy S8 Plus(Android version : 8.0.0) and Samsung Galaxy J7 Prime(Android version : 8.1.0) phones.
Is there any update on this issue?
BRS / Kostas

Any news about this issue ?

Any plan to merge these fixes? Is this library still maintained?

Should be fixed in v4.0.2.

Can you please answer?
If you plan to completely abandon lower android api then 24 could you at least write how you fixed it so maybe I could fork your repo and fix it there at least?

The fixes won't be backported.
You can use the latest versions with Android API 14+, either by:

Downgrading zxing:core - see https://github.com/journeyapps/zxing-android-embedded#older-sdk-versions
OR by using AGP - see https://github.com/journeyapps/zxing-android-embedded/pull/612


Announcements Template

Trying out the look for making announcements in a manner separate to the release notes where we can go deeper into a topic.

#2428 will implement this


Rendering slow when displaying back to a Mac.

Describe the bug
When running VisIt on one of the Livermore Computing systems and displaying back to a Mac, the rendering is very slow. When you rotate the object you can see it copy the image from the top to the bottom of the Window. When doing a rubber band zoom in 2D or 3D it takes a long time for each iteration of lines to get drawn, most likely because it is rendering the entire image each time. This is with VisIt 3.0. VisIt 2.13.3 is also slow, but you can't see the rendering of the image as you rotate the image, it is just slow. The rubber banding is also slow, but you don't see the lines. 3.0 is probably an improvement in this regard.
Impact

[X] Medium - User productivity partially degraded

Likelihood

[X] Medium - neither low nor high likelihood

Desktop

OS and version: Linux displaying to a Mac
VisIt Version: 2.13.3, 3.0

Trying to add move of what it looks like...
foo.zip

This is an issue with Linux, X11, and Qt5: https://bugreports.qt.io/browse/QTBUG-47069.
Also, tried some suggestions like setting the QT_XCB_NATIVE_PAINTING environment variable without success. The other suggestion was configure QT with --feature-xcb-native-painting (https://www.mail-archive.com/development@qt-project.org/msg32470.html).

I changed to impact low because of multiple mitigating work-arounds (client-server, vnc)


docs: Fix a few typos

There are small typos in:

docs/faq.rst
taggit/managers.py
tests/tests.py

Fixes:

Should read stored rather than storred.
Should read intermediary rather than intermeidary.
Should read function rather than funtion.

Semi-automated pull request generated by
https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

Maybe time for me to use a spell checker (at least locally :laughing: )


it doesnt work and i have no idea what this error is.


can't convert pcap to csv

I haven't tested this tool on windows, but it said tcpdump is missing, you can install it and retry.

Hi, I have a similar issue when I'm using it on winsows where tcpdump is not available. I believe it can't properly run on windows.

Please refer to this document as cicflowmeter depend on this library for capture packets.
https://scapy.readthedocs.io/en/latest/installation.html#windows

I have tried reinstalling npcap and scapy but it is still prompting the same error.

@840869522 hi did u manage to solve the issue? Would love to hear your solutions


Azure Service Bus trigger for Azure Functions PeekLock maxAutoRenewDuration maximum value

According to the current documentation the maximum value for maxAutoRenewDuration is 5 minutes but can be increased to 10 minutes. This seems to be wrong because the default value is 5 minutes but in can be increased indefinitely and also should be increased if the function timeout is higher than the maxAutoRenewDuration. This should be also stated in the documentation e.g. something like this:

The maxAutoRenewDuration is configurable in host.json, which maps to OnMessageOptions.MaxAutoRenewDuration. The default for this setting is 5 minutes according to the Service Bus documentation, whereas you can increase the Functions time limit from the default.

See MessageHandlerOptions.MaxAutoRenewDuration, ClientPumpRenewLockTimeout (default) and MessageReceivePump
Also the service bus lock can be renewed an unlimited time. See SO

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

ID: 1d50399c-0873-e903-35ca-1845c84b6c6c
Version Independent ID: 1a3150b4-5eb9-0ac5-ee2b-d1bb6b6fd63e
Content: Azure Service Bus trigger for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus-trigger.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Thanks for the feedback! We will review and get back to you shortly.

I have assigned the issue to the doc author to update appropriately.

I believe this can be closed. Fixed by #98591


Reorganize api.Version?

I don't see why this should be in it's own package when our implementation of semver builder.Version is in the builder package.
_Originally posted by @ekcasey in https://github.com/render_node/MDE3OlB1bGxSZXF1ZXN0UmV2aWV3MjgzODkzOTQw/pull_request_reviews/more_threads

Closing for now given it's part of out internal structure.


[BUG] Cannot find vertex via pk on Groot

Describe the bug
When running pk scan on Groot, it seems some vertices are not found.
For example:

Here, the vertex with 'id' as 899523 cannot be found.
From the log, it seems that the get_vertex_id_by_primary_keys() API didn't return the correct global id, which is -3857893039329826533, while the expected value is<PHONE_NUMBER>3659807.

This error was caused by incorrectly assigning the type i32 to the id 899523, when according to the schema definition, it should be an i64 value.
A query with g.V().hasLabel("PERSON").has("id", 899523L) worked.
Is it better to accommodate this during pk queries?


Does monk supports inference by directly using pre trained models? I don't want to train the models.

I have a requirement where I wanted to perform experiments on multiple pre-trained image classification models.  I was wondering if I can use monk and perform these experiments, monk seems to provide features and solve issues which I am facing.
Please note I don't want to train the models. I have created a notebook for pre-trained models available in the different libraries (Tensorflow, Keras & Pytorch).

By pre-trained models do you mean one trained on coco dataset?

@abhi-kumar - I mean pre-trained classification models trained on the ImageNet database.

Haven't enabled that feature yet, inference directly on pretrained models provided by frameworks. Will soon add it to the pipelines.

Ok, Thanks for the update @abhi-kumar.


Unit tests for swift-format CLI tool and for Plugins

Currently, swift-format test suite covers Rules and PrettyPrint. Several components are not covered:

generate-pipeline that builds RuleRegistry and friends
swift-format the CLI executable target, including Configuration, Frontend, and the whole argument-parser based thing.
Plugins.

Disclaimer: my understanding and knowledge are very high-level and basic still, I'm only diving in.
I think with some guidance, we could split these components up into smaller chunks, and each of those chunk-unit-test-issues could be a great good-first-issue for beginners.
UPD: Stumbled at this when I was looking to add tests for executable generate-pipeline target: https://forums.swift.org/t/argumentparser-and-xctest/34523

I introduced a test for the plugin in #608: PluginRunTests.swift. I expect there is some Gherkin style BDD library that could be used to write the test in a nicer way, but I've never used one in Swift and I don't know the policy on introducing third party libraries to the project (I notice all dependencies currently are Apple projects).

Tracked in Apple’s issue tracker as rdar://126948360


create_index_if_missing doesn't exist

>>> from pyes import ES
>>> es = ES('<IP_ADDRESS>:9200')
>>> es.create_index_if_missing(index)
Traceback (most recent call last):
  File "<console>", line 1, in <module>
AttributeError: 'ES' object has no attribute 'create_index_if_missing'

use:
es.indices.create_index_if_missing

thanks, I ended up rolling back to a previous version!

yeah, most pyes docs, including official one are not for new version, it makes confusion!


Objects are always sorted by the last key instead of the matching key only (if ranks are same)

match-sorter version: v4.0.1
node version: v10.15.0
npm (or yarn) version: yarn v1.17.3

Relevant code or config
console.log(matchSorter([
        {country: 'Italy', counter: 3},
        {country: 'Italy', counter: 2},
        {country: 'Italy', counter: 1},
      ],
      'Italy',
      {keys: ['country', 'counter']}));
//  =>    {country: 'Italy', counter: 1}, {country: 'Italy', counter: 2}, {country: 'Italy', counter: 3}

What you did:
We're using match-sorter for searching through lists of real estate objects (pre-sorted with most recent ones first).
What happened:
When several items match and have the same rank, they are resorted in an unexpected order – the value of the last key.

Reproduction repository:
See code example above. I'll also create a test showing that behavior in the PR.

Problem description:
In v4.0, a change was introduced to sort results alphabetically. When using multiple keys for matching, the last key in the list is used for sorting equally ranked items, which leads to unexpected results.
Suggested solution:
Sort by the value of the key that actually matched instead of the last one.

This change was introduced by @sdbrannum and was released in 3.1.0. I think I agree with you @bernharduw, and the recommendation for people will be to sort things before sending them into match-sorter. Thanks for the suggestion and PR. I'll merge that and we'll make this a breaking change (because it is).

:tada: This issue has been resolved in version 4.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: