npm installed node-xlsx does not provide xlsx.utils. Use the npm 'xlsx' package instead.

Hi - Writing for a friend.. hope it helps someone else.
I was (and am still) mixing up libraries when I found myself banging my head against all examples pointing to npm install node-xlsx and then simply cite xlsx.utils.XYZ, but for me, .utils was not an object.  Elsewhere I found this debug snippet:
...
xlsx = require('node-xlsx'),
...
console.log(Object.keys(xlsx));  // not even down to .utils, just main xlsx

Output of this is:
[ 'parse', 'build', 'default' ]
Ah hah.  So I was in fact using the wrong npm.  Instead of node-xlsx I needed just node install --save xlsx and all works as expected.  console.log(Object.keys(xlsx)); gives this and .utils gives that.
This:
  'stream',
  'parse_xlscfb',
  'parse_ods',
  'parse_fods',
  'write_ods',
  'parse_zip',
  'read',
  'readFile',
  'readFileSync',
  'write',
  'writeFile',
  'writeFileSync',
  'writeFileAsync',
  'utils',
  'SSF',
  'CFB' ]

That:
  'encode_row',
  'encode_cell',
  'encode_range',
  'decode_col',
  'decode_row',
  'split_cell',
  'decode_cell',
  'decode_range',
  'format_cell',
  'get_formulae',
  'make_csv',
  'make_json',
  'make_formulae',
  'sheet_add_aoa',
  'sheet_add_json',
  'aoa_to_sheet',
  'json_to_sheet',
  'table_to_sheet',
  'table_to_book',
  'sheet_to_csv',
  'sheet_to_txt',
  'sheet_to_json',
  'sheet_to_html',
  'sheet_to_dif',
  'sheet_to_slk',
  'sheet_to_eth',
  'sheet_to_formulae',
  'sheet_to_row_object_array',
  'consts',
  'book_new',
  'book_append_sheet',
  'book_set_sheet_visibility',
  'cell_set_number_format',
  'cell_set_hyperlink',
  'cell_set_internal_link',
  'cell_add_comment',
  'sheet_set_array_formula' ]```
Thanks for listening.

@bronius tell your friend that node-xlsx and xlsx are in fact different projects. It's confusing.  Someone else also published js-xlsx on npm so it gets even more confusing.

Ok thanks for validating my observation, psychological framework, and even including a little humor. :D I assure you my friend feels better.


operator: add ccnp event watcher to cilium operator

Add kvstore status watcher for CiliumClusterwideNetworkPolicies
similar to that of CiliumNetworkPolicies.

Signed-off-by: Deepesh Pathak<EMAIL_ADDRESS>

This change is

Coverage decreased (-0.03%) to 46.064% when pulling a907e3e39584beb40dbee9a3aea06ff2024d26c2 on fristonio:pr/fristonio/clusterwide-policies into 66cccbb08a754ae46c03e63b35e833bc1590bcc9 on cilium:master.

test-me-please

test-me-please

test-me-please

test-focus K8sPolicyTest K8sHealthTest

test-me-please

test-me-please

test-me-please - VM provisioning failed in the last test.

test-me-please

test-me-please

test-me-please

test-docs-please


Fix sha sum for Poedit

Thank you for the contribution. It was merged directly as 93f1852a9a63b16adf80e3af38dfcd17e33ea6c5 to keep commit history cleaner. Your contribution is still credited to you.


Do not delete EXTENDED_ARG instructions that have no arg

Thanks for the great library!  I'm using it here with much enjoyment.
I've run into one issue where large if/elif trees fail parsing and I'll get an error similar to this:
  File "/Users/thomassaunders/Workshop/bytecode/bytecode/tests/test_concrete.py", line 526, in test_label3
    code = Bytecode.from_code(code)
  File "/Users/thomassaunders/Workshop/bytecode/bytecode/bytecode.py", line 133, in from_code
    return concrete.to_bytecode()
  File "/Users/thomassaunders/Workshop/bytecode/bytecode/concrete.py", line 373, in to_bytecode
    label = labels[jump_target]
KeyError: 280

This PR fixes that issue, and I've added a test for this.  Would be interested in any feedback!

Codecov Report

Merging #24 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
+ Coverage   93.21%   93.22%   +0.01%
==========================================
  Files          17       17
  Lines        2475     2481       +6
==========================================
+ Hits         2307     2313       +6
  Misses        168      168

Impacted Files
Coverage Δ

bytecode/tests/test_concrete.py
99.67% <100%> (ø)
:arrow_up:

bytecode/concrete.py
94.29% <100%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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

👍

Thanks @localhuman

Thanks for merging!
Any chance of a pypi deploy :)

We'd also appreciate a pypi deploy with this fix!

Will try to do one this week-end. Do not hesitate to ping me should I forget.

Oh yes a pypi release would be great! I encountered a similar issue as OP in another project.

The release is ready but I do not have the rights to push to PyPI. @vstinner could you push the release to PyPI ? I will take care of making the post-release work.


chore(components): implement path aliases

I've implemented path aliases and optimized the import of the package.json, so its definition does not end up in every single component output file.
Needed to update stencil to get a fix, which enables the wanted feature: https://github.com/ionic-team/stencil/issues/2319#issuecomment-2037146011

Duplicates #2998


Fix VectorMemtableIndex#update logic to compare vectors

Fix https://github.com/riptano/VECTOR-SEARCH/issues/88
This PR changes the buffer comparison logic to use the index's validator. The previous logic did not work correctly because it assumed all positions started at 0.

With this fix, the VectorUpdateDeleteTest passes with memtable_allocation_type: heap_buffers. I'm not sure that we want to change this setting for all of the tests though.


ignore STOP_WAITINGs, derive lower bound for packets to include in ACK

Fixes #964.
This PR implements the logic to derive the lower bound of which packets to include in an ACK frame. It does so by keeping track of ACKs for ACKs. This will be needed when STOP_WAITING frames are removed (which will happen in IETF QUIC, and a future PR will already make that change for VersionTLS).
We're now still sending STOP_WAITING frames, since Chrome will need them. However, we're only parsing and then ignoring any STOP_WAITING that we receive.

I must have closed this by mistake. Reopening.

Codecov Report

Merging #1025 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1025      +/-   ##
==========================================
+ Coverage   90.56%   90.57%   +0.01%
==========================================
  Files         103      103
  Lines        7087     7096       +9
==========================================
+ Hits         6418     6427       +9
  Misses        435      435
  Partials      234      234

Impacted Files
Coverage Δ

ackhandler/packet.go
100% <ø> (ø)
:arrow_up:

ackhandler/sent_packet_handler.go
98.31% <100%> (+0.05%)
:arrow_up:

session.go
83.55% <100%> (+0.06%)
:arrow_up:

packet_packer.go
90.05% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Investigate why git-upload may hang instead of failing if there is no auth data

First discovered in #1859: a URL with no authentication token was provided to the git-upload script, and it went like this:
Cloning into '/tmp/tmpnuea65j3'...
warning: redirecting to https://github.com/neondatabase/zenith-coverage-data.git/
remote: Enumerating objects: 287938, done.
remote: Counting objects: 100% (3485/3485), done.
remote: Compressing objects: 100% (810/810), done.
remote: Total 287938 (delta 2608), reused 3200 (delta 2323), pack-reused 284453
Receiving objects: 100% (287938/287938), 1.34 GiB | 47.52 MiB/s, done.
Resolving deltas: 100% (219754/219754), done.
Updating files: 100% (207676/207676), done.
$ git add .

... snip ...

$ git fetch origin main
warning: redirecting to https://github.com/neondatabase/zenith-coverage-data.git/
From https://github.com/neondatabase/zenith-coverage-data
 * branch                  main       -> FETCH_HEAD
$ git -c user.name=vipvap -c<EMAIL_ADDRESS>rebase origin/main
Current branch main is up to date.
$ git push origin main
Username for 'https://github.com': fatal: could not read Username for 'https://github.com': Success
failed to update branch `main`: Command '['git', 'push', 'origin', 'main']' returned non-zero exit status 128.
$ git fetch origin main
warning: redirecting to https://github.com/neondatabase/zenith-coverage-data.git/
From https://github.com/neondatabase/zenith-coverage-data
 * branch                  main       -> FETCH_HEAD
$ git -c user.name=vipvap -c<EMAIL_ADDRESS>rebase origin/main
Current branch main is up to date.
$ git push origin main
Username for 'https://github.com': fatal: could not read Username for 'https://github.com': Success
failed to update branch `main`: Command '['git', 'push', 'origin', 'main']' returned non-zero exit status 128.
$ git fetch origin main
warning: redirecting to https://github.com/neondatabase/zenith-coverage-data.git/
From https://github.com/neondatabase/zenith-coverage-data
 * branch                  main       -> FETCH_HEAD
$ git -c user.name=vipvap -c<EMAIL_ADDRESS>rebase origin/main
Current branch main is up to date.
$ git push origin main
Username for 'https://github.com':

Too long with no output (exceeded 10m0s): context deadline exceeded

Note that the script has tried to fetch/push three times. The first two failed after some time, but the third lasted for over 10 minutes and got the job killed.
I'm confused as to why the first two runs got stopped earlier, even though they have requestions the username in the same fashion.

Note: this issue may become outdated once we migrate to GitHub, see #1414

@zoete is this relevant or should be closed as we are on GH now

closing as this is not relevant anymore (coverage report has been reworked).


Hashie::Mash#dup blindly tries to initialize subclasses with two arguments.

The fix for #1 caused some unfortunate side effects that breaks subclasses of Hashie::Mash that override initialize to take different parameters than Hashie::Mash. (Specifically, I encountered this issue within Redfinger's link.rb )
class Foo < Hashie::Mash
  def initialize(message)
    self[:message] = message
  end
end

f = Foo.new('hello')
f.dup

yields
ArgumentError: wrong number of arguments (2 for 1)
    from [...]/hashie-1.0.0/lib/hashie/mash.rb:97:in `initialize'
    from [...]/hashie-1.0.0/lib/hashie/mash.rb:97:in `new'
    from [...]/hashie-1.0.0/lib/hashie/mash.rb:97:in `dup'
    [snip]

because Hashie::Mash redefines dup as
alias_method :regular_dup, :dup
def dup
  self.class.new(self, self.default)
end

I'm having the same problem when using in conjunction with mockumentary.  I went back to hashie commit f299f63637 and the problem goes away, so it seems it started at the next commit.  This is when Mash.new(self, self.default) was changed to self.class.new(self, self.default).


improveUsing Awaitility instead of Thread.sleep (#37812)

Proposed changes
Awaitility: Waits until a specific condition is met, which makes the tests more reliable and less prone to intermittent failures. It ensures that the test only proceeds when the expected condition is true. Thread.sleep: Introduces a fixed delay, which can lead to flaky tests. If the condition is met sooner than the sleep time, the test unnecessarily waits. Conversely, if the condition is met after the sleep time, the test will fail.
(cherry picked from commit 8d249a2562b15825a06a53a5a530a532a0ca2454)
Proposed changes
Issue Number: close #37812

run buildall


allow ties

The current implementation doesn't allow ties. Some games allow ties so this functionality should be added. Ideally in a way that's backwards compatible.
Not sure of the best way to do this without large rewrites to the code. Current syntax to get new ratings is:
elo.get_new_ratings([rating_1, rating_2, rating_3, rating_4])

Suppose players 2 and 3 tie. Maybe allow syntax like this?
elo.get_new_ratings([rating_1, (rating_2, rating_3), rating_4])

Then we'd have to figure out how ties should effect rating updates. It's pretty easy for 1v1 matchups -- each player gets an actual score of 0.5 -- but maybe a bit more complicated for multiplayer matchups. I'll have to think about it a bit.

Going with a slightly different syntax. If players 2 and 3 tie, then calculate new ratings like this:
elo.get_new_ratings([rating_1, rating_2, rating_3, rating_4], result_order=[1, 2, 2, 3])

(Note: it doesn't matter if you use [1, 2, 2, 3] or [1, 2, 2, 4])
Can also handle ties in the Tracker object. The dataframe for Tracker.process_data should look like this if there is a tie:
|   date | 1st                        | 2nd               | 3rd   | 4th |
|--------|----------------------------|-------------------|-------|-----|
|      1 | Lisa                       | ('Bart', 'Marge') | Homer |     |

(Similarly, it doesn't matter if Homer is in the 3rd column or the 4th)
PR coming soon.


Make it harder for Mean() and StdDev() to overflow

This is my proposal for solving issue #20

This looks great, but the admittedly very touchy tests don’t pass. Feel free to round the stddev and I’ll gladly merge this. Thanks for the patch!

bumping this @cassava .
The tests are still flaky. How do you want to proceed with it? :
:~/go/src/github.com/HdrHistogram/hdrhistogram-go$ make test
GO111MODULE=on go get -t -v ./...
GO111MODULE=on go fmt ./...
GO111MODULE=on go test -count=1 ./...
--- FAIL: TestStdDev (0.01s)
    hdr_test.go:80: StdDev was<PHONE_NUMBER>682712, but expected<PHONE_NUMBER>682715
FAIL
FAIL    github.com/HdrHistogram/hdrhistogram-go 0.153s
FAIL
make: *** [Makefile:35: test] Error 1

@ahothan  we can approve this assuming we merge #40 first.

@ahothan I've confirmed that all tests pass with the current master. If we want we can merge it. wdyt?

@filipecosta90
I was wondering if this could introduces the same kind of issue at the other end of the spectrum of possible values, when total count is extremely high and values are extremely small, but it looks like float64 can handle that in all realistic cases as the smallest number for float64 is ~10**(-308)
So this patch looks good.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1258 ms

Resolved: Bofrost.at performance has improved in 7a9d263.


Update Challenge-00-DASH.md

I updated the content of Challenge 0 to include the URL for the lab environment and to include a line to ensure the students copy the VM username and password.

Added a new commit and removed the following from C1, after speaking with Prashant to confirm.

Create a new Monitor configuration that suppresses alerts from the scale set and virtual machines


Link to DataDog/zstd breaks Cygwin

Versions
Sarama Version: Checkin e42dc76c4e0af and later
Kafka Version: N/A
Go Version: go version go1.9 windows/amd64
Configuration
Logs
$ go build                                                                                # github.com/mistsys/location/dtest                                                                                c:\home\nealc\go\pkg\tool\windows_amd64\link.exe: running gcc failed: exit status 1                                                                                /usr/lib/gcc/x86_64-pc-cygwin/7.3.0/../../../../x86_64-pc-cygwin/bin/ld: cannot find -lmingwex                                                                                /usr/lib/gcc/x86_64-pc-cygwin/7.3.0/../../../../x86_64-pc-cygwin/bin/ld: cannot find -lmingw32                                                                                collect2: error: ld returned 1 exit status
Problem Description
Data Dog and it's use of embedded "C" is causing some weird issue with "Go" linking the sarama package.
DataDog was added to the Sarama package in e42dc76c4e0af.
The only resolution I've found is to install mingw, and remove "gcc", which would cause a bunch of issues.

Sorry, I wrote the version wrong zstd was added in:
49c4c9a35917a18e15b9f0706f9dc91f6f07a89b

Same issue here... Sarama should not include dependencies that embed C code !

disable CGO, can install on the windows env, please see #1182

nothing we can do right now as this was merged, hopefully we can get rid of cgo in the future.
Also, the zstd support is not working right now :(


add the portability sample and allow for enabling globally during instance creation

Description
Add portability sample.  NB: this requires Vulkan 1.3.
Fixes #343
General Checklist:
Please ensure the following points are checked:

[x ] My code follows the coding style
[ x] I have reviewed file licenses
[ x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[ x] I have commented any added functions (in line with Doxygen)
[x ] I have commented any code that could be hard to understand
[x ] My changes do not add any new compiler warnings
[ x] My changes do not add any new validation layer errors or warnings
[ x] I have used existing framework/helper functions where possible
[ x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
[x ] My changes do not add any regressions
[ x] I have tested every sample to ensure everything runs correctly
[ x] This PR describes the scope and expected impact of the changes I am making

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[ x] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[x ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ x] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the samples readme

@gpx1000
I'm trying to build and test this. Portability is not getting enabled.
The portability/README.md says that both VK_ENABLE_BETA_EXTENSIONS and VKB_ENABLE_PORTABILITY define macros need to be enabled during build.
Unfortunately, I'm not a cmake guru. How exactly should those macros be defined within the documented Vulkan-Samples command line build instructions listed here?
cmake -H. -Bbuild/mac -DCMAKE_BUILD_TYPE=Release
cmake --build build/mac --config Release --target vulkan_samples -- -j4

Thanks.

cmake -DCMAKE_BUILD_TYPE=Debug  -DVKB_VALIDATION_LAYERS=ON -DVKB_ENABLE_PORTABILITY=ON -DVK_ENABLE_BETA_EXTENSIONS=ON ....
modify as your needs exhibit :)  i.e. you likely want to tell it something like a XCode generation is required or what you're targeting like ios toolchain what compiler etc.

cmake -DCMAKE_BUILD_TYPE=Debug -DVKB_VALIDATION_LAYERS=ON -DVKB_ENABLE_PORTABILITY=ON -DVK_ENABLE_BETA_EXTENSIONS=ON ....

Hmmm...I tried that...specifically...
cmake -H. -Bbuild/mac -DCMAKE_BUILD_TYPE=Release -DVK_ENABLE_BETA_EXTENSIONS=ON -DVKB_ENABLE_PORTABILITY=ON
cmake --build build/mac --config Release --target vulkan_samples -- -j4

but got the following results during the first command...
CMake Warning:
  Manually-specified variables were not used by the project:

    VKB_ENABLE_PORTABILITY
    VK_ENABLE_BETA_EXTENSIONS

and the portability sample continues to report onscreen VK_ENABLE_BETA_EXTENSIONS not enabled can't list portability feature set

cmake -DCMAKE_BUILD_TYPE=Debug -DCMAKE_DEPENDS_USE_COMPILER=FALSE -G "CodeBlocks - Unix Makefiles" /Users/swinston/Projects/holochip/vulkan_samples --target vulkan_samples -- -j4 -DVK_ENABLE_BETA_EXTENSIONS=ON -DVKB_ENABLE_PORTABILITY=ON
Another way to do it is above.  The -- at the end of the line tells CMake to pass remainder directly to the compiler.  That's the easy way.  The explicit way is to define the CMAKE_C_FLAGS and CMAKE_CXX_FLAGS to hold the values you want them to i.e. like this:
cmake -H. -Bbuild/mac -DCMAKE_BUILD_TYPE=Release -DCMAKE_C_FLAGS="-DVK_ENABLE_BETA_EXTENSIONS=ON -DVKB_ENABLE_PORTABILITY=ON" -DCMAKE_CXX_FLAGS="-DVK_ENABLE_BETA_EXTENSIONS=ON -DVKB_ENABLE_PORTABILITY=ON"
cmake --build build/mac --config Release --target vulkan_samples -- -j4
NB: the above will use the system defined code generator.  My original command up top is using CodeBlocks -Unix Makefiles for the generator.  There's a possibility that's not what you want and instead want to use XCode or target iOS.  If that applies to you, you'd want to modify the -G target correspondingly.  Also, you might need to supply a toolchain file depending upon how you have your system setup (-T then toolchain file).  So, if I wanted to target iOS, I might use -G XCode -T ios.toolchain.  This also describes how you'd target Android if you want but that's beyond the scope of this quick intro.

Thanks. Finally got this working with your second build option (CMAKE_C_FLAGS, etc).
Confirmed this sample works correctly with MoltenVK. The feature flags reported by the sample match those returned by MoltenVK.

@billhollings, if you're happy with this, I'm tempted to merge.  Anyone else you'd like to see review this first?

I'm fine with merging. I don't have an implementation to test this on, but code compiles, reports extension as not supported and both bode and documentation look fine :)

OK, I'll merge - this is a bit of a special case.  LMK if issues and we can revert if needed.


from .nms import nms

❓ Questions and Help
  File "/home/idea/model/maskrcnn-benchmark/maskrcnn_benchmark/layers/nms.py", line 3, in <module>
    from maskrcnn_benchmark import _C
ImportError: cannot import name '_C'

python3 demo/webcam.py --min-image-size 800

Hi,
Yes, there will be a conflict between pytorch version 0.4.1 and pytorch 1.0.
I'd recommend creating a new conda environment, as illustrated in the INSTALL.md file. This will make sure that there are no conflicts.

Thank you ,I  got it .


converting to fahrenhheit

I'm interested in showing temps in Fahrenheit. I found in extension.ts where the unit is set, however this only seems to change the label.  The actual temps still seem to be in celsius.  I see that there's a _toFahrenheit method, but it doesn't appear to be used.  If I can get any pointers or direction on where to implement that helper method, I'd appreciate it!
Thanks for this extension as well, it's really nice.

Should be possible now


Spy Number Checeker

A spy number is a number where the sum of its digits equals the product of its digits. For example, 1124 is a spy number, the sum of its digits is 1+1+2+4=8 and the product of its digits is 112*4=8.
Enter any number and know whether it is a spy number or not.

/assign


Blank page when starting the app after recent changes

Pulling from main now result in a blank page when starting the app:

From our technical meeting today, we suspect that it might be due to recent updates to the dev environment.

I can reproduce the blank page issue when starting the web app with nx serve web-app. The API and API DB are running using docker.

The browser console shows the following errors. This is probably because KC is not up.
ERROR
Object { error: "Timeout when waiting for 3rd party check iframe message." }
<EMAIL_ADDRESS>Object { error: "Timeout when waiting for 3rd party check iframe message." }


New Release?

Hello,
I wanted to know if maybe you could release a new version of LibPQ? I noticed that the latest release 1.17.1 restricts LibPQ_jll to major version 14, which in turn pulls in an OpenSSL < 3. This causes a few problems for my project because I don't have access to the old OpenSSL. If I restrict LibPQ_jll to >=16 via compat, installation of LibPQ pulls in version 1.4, which is apparently the last version without the LibPQ_jll dependency. This however brings its own set of problems...
Since compat-tables in downstream projects can only restrict packages to full semver versions (as far as I know), I don't have a way of specifying that downstream consumers of my code shall pull in a non-released version of LibPQ either. Otherwise I would just take that, add a compat entry for the most recent commit, and live with the solution for the time being, but I can't. :(
Thanks in advance!

It looks like the patch version (with LibPQ_jll >= 16 compatibility) was bumped but a release was never tagged nor  registered in the General registry -- commit
Could you please cut a release on this commit @iamed2 (or any other maintainer) when you're able to? Thank you!

I will take some time today or tomorrow to make sure the doctests run and then do so.

I'll register current master branch after https://github.com/JuliaRegistries/General/pull/113343 is merged, shouldn't be too long

Done

Done

Much appreciated, thank you!


Snapchat like video preview

@remirobert
I would like to be able to instantly show a preview of a video taken by the user - just like Snapchat and more impressive, Instagram stories. Somehow once a user stops capturing a video on both apps, a preview automatically starts playing in an endless loop seemingly within the same view.
Can I achieve that with CameraEngine?

Hi @otymartin , did you found some solution?

hi @otymartin , Did you found a solution?


adds powershell code to setup smb share

This script will create the smb share as described including settings of the right permissions.

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


"Reader to InputStream" should recommend Commons IO's ReaderInputStream

The article "Java – Reader to InputStream" should use and recommend Commons IO's ReaderInputStream since that is currently the most efficient solution available.
See also https://github.com/google/guava/issues/5376

Hi @Marcono1234 ,
Thanks for the feedback. We've updated the code and the article.

Thanks!


rejects other tenders ones accepted one

Dont accept im merging to main the wrong branch

closed without merging, PR made into wrong branch.


ValueError: not enough values to unpack (expected 4, got 1)

`import cv2
trained_face_data = cv2.CascadeClassifier('haarcascade_frontalface_default.xml')
img = cv2.imread('am.jpg')
bwimg = cv2.cvtColor(img, cv2.COLOR_BGR2GRAY)
face_crdnts = trained_face_data.detectMultiScale(bwimg)
(x, y, w, b) = face_crdnts       #problem in this line
cv2.rectangle(img, (x,y), (x+w, y+b), (0, 300, 0), 1)
cv2.imshow("That's u!!",img)
cv2.waitKey()
print("I am done!")`

This is a face detection code snippet I am trying to run, but it is giving the "ValueError". I tried to use the for loop to get the variables from the list being created in the coordinate detect function, but it still doesn't work.
Can anyone help me with this code.

face_crdnts only have 1 value.
you should check type and shape of face_crdnts...


Jetplate isn't usable in Anvil

Made full flux armour and went the Ender IO Route for the Jetplate.
Only to discover, it isn't usable in an Anvil or Auto-Anvil (MFR)
Testing shows only Enchant Table and Auto-Enchanter (MFR) work.

After some investigation, I have come to the following conclusion:
MFR's Auto-Anvil works on any item that is damageable, and doesn't work with any items that aren't. This causes problems with items that have infinite durability, but also with SJ's jetpacks (stored energy does not count as damage in this case). It may very well be a balance move on MFR's part though.
@skyboy: is this intended or an oversight?

@Tonius I've corrected the logic, but due to bad MCP naming it looks wrong. You will have to override isItemTool and return true for this to work in the next version, that method is used to determine if an item can be enchanted.

ItemJetpack already does this, so that should be fine.
https://github.com/Tonius/SimplyJetpacks/blob/master/src/main/java/tonius/simplyjetpacks/item/ItemJetpack.java#L194

Should be working in the latest MFR dev builds.


VS Code: unnecessary cross for the Serverless workflow diagram

In some cases, the rendered Serverless workflow diagram has unnecessary cross

Example Serverless workflow
id: VSCodeUnecessaryCorssCross
specVersion: "0.8"
version: "1.0"
name: test
description: show the unnecessary cross issue
start: StartAction

functions:
  - name: log
    type: custom
    operation: sysout:INFO

events:
  - name: BeginEvent
    source: ""
    type: begin-event
  - name: ChangeEvent
    source: ""
    type: change-event

states:
  - name: StartAction
    type: event
    onEvents:
      - eventRefs:
          - begin-event
        actions:
          - name: begin
            functionRef:
              refName: log
              arguments:
                message: Start executing
    transition: parallelCreation

  - name: parallelCreation
    type: parallel
    branches:
      - name: create1
        actions:
          - functionRef:
              refName: log
              arguments:
                message: Creating1
      - name: create2
        actions:
          - functionRef:
              refName: log
              arguments:
                message: Creating2
    numCompleted: "2"
    transition: next

  - name: next
    type: operation
    actions:
      - functionRef:
          refName: log
          arguments:
            message: finishing
    end: true

  - name: changeEvent
    type: event
    onEvents:
      - eventRefs:
          - ChangeEvent
        actions:
          - functionRef:
              refName: log
              arguments:
                message: changing
    transition: hasFlag

  - name: hasFlag
    type: switch
    dataConditions:
      - name: Checking if there is falg.
        condition: .hasFlag
        transition: hasFlagYes
    defaultCondition:
      transition: hasFlagNo

  - name: hasFlagYes
    type: operation
    actions:
      - functionRef:
          refName: log
          arguments:
            message: Has flag
    end: true

  - name: hasFlagNo
    type: operation
    actions:
      - functionRef:
          refName: log
          arguments:
            message: Has no falg.
    end: true

Expected https://serverlessworkflow.io/editor


Updated module-info.java to use com.azure.core.v2 if com.azure.core is used.

Added two recipes to rewrite.yml:

Recipe to change requires com.azure.core to requires com.azure.core.v2 in module-info.java
Recipe to change requires transitive com.azure.core to requires transitive com.azure.core.v2 in module-info.java

OpenRewrite's FindAndReplace recipe for changing plain text was used due to OpenRewrite's lack of support for converting Java module declarations to LSTs.
Fixes #6

Pull request was closed due to renaming the branch that was going to be merged.