Halt on error

Can we have an option to halt (stop submitting new jobs) when any of the running jobs fails?
It is useful when we are trying to debug if a process has many jobs, we don't have to wait until all jobs being submitted.

Now you can use proc.errhow = "halt" to halt the pipeline when any job fails.


Fetching wrong byte range for HLS with fMP4 using BYTERANGE

Have you read the FAQ and checked for duplicate issues:
yes. I don't see a duplicate.
What version of Shaka Player are you using:
Shaka Player v2.1.0-117-gaf252c9-debug
at https://nightly-dot-shaka-player-demo.appspot.com/demo
Can you reproduce the issue with our latest release version:
N/A
Can you reproduce the issue with the latest code from master:
N/A
Are you using the demo app or your own custom app:
Demo app.
If custom app, can you reproduce the issue using our demo app:
N/A
What browser and OS are you using:
Linux with Chrome
Version 59.0.3071.115 (Official Build) (64-bit)
What are the manifest and license server URIs:
(you can send the URIs to<EMAIL_ADDRESS>instead, but please use GitHub and the template for the rest)
HLS manifest at: https://storage.googleapis.com/wvtemp/rkuroiwa/hls_single_segment/single_segment.m3u8
(Just for comparison and repro) MPD manifest at: https://storage.googleapis.com/wvtemp/rkuroiwa/hls_single_segment/sintel_mpd_hls_compare.mpd
What did you do?
Used the HLS manifest and watched the network dev console.
What did you expect to happen?
For the second segment, the player fetched this (copy pasting HTTP request headers)
range:bytes=11316324-14165510
Where as it should be:
range:bytes=11316325-14165511
Looks like there is off by 1 error.
If you use the MPD manifest you get the second HTTP headers.
Although I see 3016 error, the MPD and the HLS playlist point to the same media file, so it looks like shaka-player is not appending the right buffer.
What actually happened?
Fetch range is "range:bytes=11316324-14165510"
Also the playback stops with an error after playing for 2 seconds.
It seems like after the first 2 seconds, the player tries to append the second subsegment to MSE.
The message in the console looks like this
"Shaka Error MEDIA.VIDEO_ERROR (3,,CHUNK_DEMUXER_ERROR_APPEND_FAILED: Append: stream parsing failed. Data size=131072 append_window_start=0 append_window_end=888.04)"

@rkuroiwa Thanks a lot for such a detailed repro that potentially saved me hours of debugging!
I think it is indeed an off by 1 error and will start working on the fix right away.

The fix has been cherry-picked for v2.1.6.


'ascii' codec can't encode character u'\u0421' in position 6: ordinal not in range(128)

<type 'exceptions.UnicodeEncodeError'>
'ascii' codec can't encode character u'\u0421' in position 6: ordinal not in range(128)

Fixed in d97dd26


Added Prompt Engineering in Json file and also added new course Closes #723

[X] I added the Prompt Engineering coursesin readme.mdwith proper links.
[X] Added them in allcourses json file with links.
[X] Created issue and starred the repo too.

Please accept this PR and instead of hactoberfest please mark this as label : feature .
Thankuu.

we are  not adding coursera courses as coursera do not provide free courses , to get free course we need to apply financial aid and it takes 16 days to verify

okay then can i create another issue regarding the same as Prompt enginerring is not mentioned in the .json file . i am participating in hacktoberfest and i need 2 pr ; ) . So can i make issue regarding that

@avinash201199 Thanks i understood , okay then can i create another issue regarding the same as Prompt enginerring is not mentioned in the .json file . i am participating in hacktoberfest and i need 2 pr ; ) . So can i make issue regarding that

okay ; )


EID-1666 Create string transformer that doesn't encrypt assertions

This is for use when returning a Response from saml-engine that wraps an
eIDAS countries SAML response and their encrypted keys. All the
sensitive data is already encrypted so we don't need to encrypt it
again.

Closing this as the non-matching unsigned assertions flow in the hub now encrypts assertions, and so can re-use existing transformers.


[qt5] qt5 fails to install if libpng port is already installed in the system

Problem
If a user tries to install the qt5 port in a vcpkg installation in which libpng has been previously installed, the installation fails with the following message:
-- Build x86-windows done
-- Installing x86-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:14 (message):
    Command failed: C:/vcpkg-qt-png/downloads/tools/jom/jom.exe;-j1;install
    Working Directory: C:/vcpkg-qt-png/buildtrees/qt5/x86-windows
    See logs for more information:
      C:\vcpkg-qt-png\buildtrees\qt5\install-x86-windows-out.log
      C:\vcpkg-qt-png\buildtrees\qt5\install-x86-windows-err.log

Call Stack (most recent call first):
  ports/qt5/portfile.cmake:74 (vcpkg_execute_required_process)
  scripts/ports.cmake:73 (include)

Error: building package qt5:x86-windows failed
Please ensure sure you're using the latest portfiles with `vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: qt5:x86-windows
  Vcpkg version: 0.0.65-dd8cbba38b9b2c6443dcd35bd1cd070a26ba7990

Additionally, attach any relevant sections from the log files above.

The last lines of the buildtrees\qt5\install-x86-windows-out.log file are:
	C:\vcpkg-qt-png\downloads\tools\jom\jom.exe -f Makefile.Debug install
	C:\vcpkg-qt-png\buildtrees\qt5\x86-windows\qtbase\bin\qmake -install sed -e "s,C:/vcpkg-qt-png/buildtrees/qt5/x86-windows/[^/][^/]*/lib,$$[QT_INSTALL_LIBS],g" -e "s,C:\\\\vcpkg-qt-png\\\\buildtrees\\\\qt5\\\\x86-windows\\\\[^\\\\][^\\\\]*\\\\lib,$$[QT_INSTALL_LIBS],gi" ..\..\..\lib\qtfreetyped.prl > C:\vcpkg-qt-png\packages\qt5_x86-windows\lib\qtfreetyped.prl
	copy /y ..\..\..\lib\qtfreetyped.lib C:\vcpkg-qt-png\packages\qt5_x86-windows\lib\qtfreetyped.lib
        1 file(s) copied.
	copy /y ..\..\..\lib\qtfreetyped.pdb C:\vcpkg-qt-png\packages\qt5_x86-windows\lib\qtfreetyped.pdb
        1 file(s) copied.
	C:\vcpkg-qt-png\downloads\tools\jom\jom.exe -f Makefile.Release install
	C:\vcpkg-qt-png\buildtrees\qt5\x86-windows\qtbase\bin\qmake -install sed -e "s,C:/vcpkg-qt-png/buildtrees/qt5/x86-windows/[^/][^/]*/lib,$$[QT_INSTALL_LIBS],g" -e "s,C:\\\\vcpkg-qt-png\\\\buildtrees\\\\qt5\\\\x86-windows\\\\[^\\\\][^\\\\]*\\\\lib,$$[QT_INSTALL_LIBS],gi" ..\..\..\lib\qtfreetype.prl > C:\vcpkg-qt-png\packages\qt5_x86-windows\lib\qtfreetype.prl
	copy /y ..\..\..\lib\qtfreetype.lib C:\vcpkg-qt-png\packages\qt5_x86-windows\lib\qtfreetype.lib
        1 file(s) copied.
	copy /y ..\..\..\lib\qtfreetype.pdb C:\vcpkg-qt-png\packages\qt5_x86-windows\lib\qtfreetype.pdb
        1 file(s) copied.
	cd gui\ && ( if not exist Makefile C:\vcpkg-qt-png\buildtrees\qt5\x86-windows\qtbase\bin\qmake C:\vcpkg-qt-png\buildtrees\qt5\src\qt-5.7.0\qtbase\src\gui\gui.pro -o Makefile ) && C:\vcpkg-qt-png\downloads\tools\jom\jom.exe -f Makefile install
	C:\vcpkg-qt-png\downloads\tools\jom\jom.exe -f Makefile.Debug install
	link /NOLOGO /DYNAMICBASE /NXCOMPAT /DEBUG /DLL /SUBSYSTEM:WINDOWS /VERSION:5.7 /MANIFEST:embed /OUT:..\..\lib\Qt5Guid.dll @C:\Users\TRAVER~1\AppData\Local\Temp\Qt5Guid.dll.512.1093.jom
LINK : fatal error LNK1104: impossibile aprire il file 'libpng.lib'

On the same machine, tryng to install qt5 in a clean installation of vcpkg works correctly.
Partial problem analysis
The problem seems related to the fact that the qt5 port is compiling its own copy of libpng. However for some reason if libpng is already installed the "external" version is used (perhaps as a consequence to this: https://github.com/Microsoft/vcpkg/commit/67303f15690629053d8a6191665131fde5829f73#diff-261ad181c16e17acae7baf8fb2b321d9R17 ? ) But then the link fails because the qt5 build system expects a library called libpng.lib, while the one installed by the libpng port is called libpng16.lib.
cc @barcharcraz @Barath-Kannan

I'm going to try and investigate this. Perhaps we should just explicitly request qt's internal copy. Long run we want to patch qt to use the vcpkg versions of everything, but it's quite a lot of work to do that.
Qt is hard to test because it takes so long to build.

Some additional data: I am getting a bunch of failures in my main enlistment (with lots of packages installed, lots of which are outdated) when trying to build qt5.
I ran some tests from a clean copy and it didn't hit the failure above. I do have integrate install on this machine from the first repo, so due to this working I think we can rule out integrate install cross-contamination.
Repro steps for success were:

Fresh clone + bootstrap
vcpkg install qt5
Succeeds.
vcpkg remove --purge qt5
vcpkg install libpng
Succeeds.
vcpkg install qt5
Succeeds.

I'm going to try adding two more libraries into the mix (pcre and libjpeg-turbo) which I had in my main enlistment.

fresh install of qt5 fails for me also...
vcpkg remove --purge qt5
vcpkg install libpng
Succeeds.
E:\src\github\vcpkg [master ≡]> .\vcpkg install qt5
-- CURRENT_INSTALLED_DIR=E:/src/github/vcpkg/installed/x86-windows
-- DOWNLOADS=E:/src/github/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=E:/src/github/vcpkg/packages/qt5_x86-windows
-- CURRENT_BUILDTREES_DIR=E:/src/github/vcpkg/buildtrees/qt5
-- CURRENT_PORT_DIR=E:/src/github/vcpkg/ports/qt5/.
-- Using cached E:/src/github/vcpkg/downloads/qt-<IP_ADDRESS>z
-- Testing integrity of cached file...
-- Testing integrity of cached file... OK
-- Extracting done
-- Applying patch E:/src/github/vcpkg/ports/qt5/set-shared-qmakespec.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch E:/src/github/vcpkg/ports/qt5/set-shared-qmakespec.patch done
-- Configuring x86-windows
-- Configure x86-windows done
-- Building x86-windows
-- Build x86-windows done
-- Installing x86-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:14 (message):
    Command failed: E:/src/github/vcpkg/downloads/tools/jom/jom.exe;-j1;install
    Working Directory: E:/src/github/vcpkg/buildtrees/qt5/x86-windows
    See logs for more information:
      E:\src\github\vcpkg\buildtrees\qt5\install-x86-windows-out.log
      E:\src\github\vcpkg\buildtrees\qt5\install-x86-windows-err.log

Call Stack (most recent call first):
  ports/qt5/portfile.cmake:74 (vcpkg_execute_required_process)
  scripts/ports.cmake:73 (include)

Error: building package qt5:x86-windows failed
Please ensure sure you're using the latest portfiles with `vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: qt5:x86-windows
  Vcpkg version: 0.0.65-835693ce97f271c0213f65c1c782e8df84cfb409

Additionally, attach any relevant sections from the log files above.

E:\src\github\vcpkg\buildtrees\qt5\install-x86-windows-out.log
link /NOLOGO /DYNAMICBASE /NXCOMPAT /DEBUG /DLL /SUBSYSTEM:WINDOWS /VERSION:5.7 /MANIFEST:embed /OUT:..\..\lib\Qt5Guid.dll @C:\Users\drize\AppData\Local\Temp\Qt5Guid.dll.33744.516.jom
LINK : fatal error LNK1104: cannot open file 'libpng.lib'

Ok, I think adding -no-libpng to the configure step in the qt5 profile fixes this issue. However, you must delete buildtrees\qt5 in order for configuration changes to take effect.

More in general, to avoid this non-determinism in the build I think we need add either -no-<dep>, -qt-<dep> or -system-<dep>  for all the dependencies listed in http://doc.qt.io/qt-5/configure-options.html . Apparently quite a few are missing.

It should be fixed by https://github.com/Microsoft/vcpkg/issues/456 .


License this code

@nottrobin would you license this code?  Currently there's no licence, can you apply one?

You're welcome to license the code however you want. I'm not precious about it. I tend to make my work public domain, but I don't mind if you want to retain more control than that. Feel free to take it.

Alright, when I get home I will apply CC0.


Inline Gtfslib

Inline the bits of Gtfslib I'm using. Mostly just the model classes (deserializing GTFS CSV files), and putting them into MapDB.
Paste Conveyal's license file.

The forbidden API checker has a problem due to the default locale:
[ERROR] Forbidden method invocation: java.lang.String#format(java.lang.String,java.lang.Object[]) [Uses default locale]


Update DG for view and Model and Storage

Included

Updated diagrams for Model
Updated links for Model and Storage API
Sequence diagram for parsing of view progress command.
Activity diagram for generation of DegreeProgressionData
Updated some parts of the implementation of view

bravo


Update parameter structure based on breaking changes

Update our template to use our new default and parameters keys inside of our input_parameters and output_parameters definitions.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Initial experiments of adding demo usage to the ProgressIndicatorDemo

Is this the kind of demos you expect? I'm just having some issue to get it "full screen size", seems holder.prefWidthProperty().bind(container.prefWidthProperty()); is not the correct way to achieve this...

Looks fine. I can finalise it if you want - otherwise keep going and let me know when to review :-)

Can you finalise please so I can see how you solve the full size problem. This is the most anyoing problem I always face with JavaFX. Can you add a chapter about that in your book? ;-)
Afterwards I'll check which other ProgressIndicator functions I can add with a demo.

I merged and made some improvements. Check them out and improve as desired (although it is probably good enough for now, so we can probably move on to another demo instead).

I did a follow-up push to show how to make the ProgressIndicator massive - it's just setting the max size.


Upgraded btcd package

BTW, I use Semantic Commit Messages to automatically generate a CHANGELOG for this project. It would be better if you could do the same :)

Sure


System time change causes Spyder error

Description
When the system time is changed to the past, Spyder opens an error dialog.
It has something to do with line 571 in:

C:\Users\user\Anaconda3\Lib\site-packages\spyder\widgets\ipythonconsole\client.py

What steps will reproduce the problem?

Open Spyder.
Change system time to 2017 or earlier.

What is the expected output? What do you see instead?
An error dialog as shown below

When the time is changed back, the error dialog goes away.
Please provide any additional information below
OS: Windows 10
Error message details:

File "C:\Users\angert\Anaconda3\lib\site-packages\spyder\widgets\ipythonconsole\client.py", line 571, in show_time
"" % (color, strftime(fmt, gmtime(elapsed_time)))
OSError: [Errno 22] Invalid argument
Traceback (most recent call last):
Traceback (most recent call last):
File "C:\Users\angert\Anaconda3\lib\site-packages\spyder\widgets\ipythonconsole\client.py", line 571, in show_time
"" % (color, strftime(fmt, gmtime(elapsed_time)))
OSError: [Errno 22] Invalid argument
File "C:\Users\angert\Anaconda3\lib\site-packages\spyder\widgets\ipythonconsole\client.py", line 571, in show_time
"" % (color, strftime(fmt, gmtime(elapsed_time)))
OSError: [Errno 22] Invalid argument
Traceback (most recent call last):
Traceback (most recent call last):
File "C:\Users\angert\Anaconda3\lib\site-packages\spyder\widgets\ipythonconsole\client.py", line 571, in show_time
"" % (color, strftime(fmt, gmtime(elapsed_time)))
OSError: [Errno 22] Invalid argument
File "C:\Users\angert\Anaconda3\lib\site-packages\spyder\widgets\ipythonconsole\client.py", line 571, in show_time
"" % (color, strftime(fmt, gmtime(elapsed_time)))
OSError: [Errno 22] Invalid argument
Traceback (most recent call last):
Traceback (most recent call last):

Version and main components

Spyder Version: 3.2.6
Python Version: 3.6.4
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.22.0 (OK)
numpy >=1.7      :  1.14.0 (OK)
sphinx >=0.6.6   :  1.6.6 (OK)
rope >=0.9.4     :  0.10.7 (OK)
jedi >=0.9.0     :  0.11.1 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.27.3 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.2.1 (OK)
pylint >=0.25    :  1.8.2 (OK)

Edit: Added image of error dialog box.

Thanks for reporting. This seems like an easy fix.

@ccordoba12 No problem, thank you for fixing it. I am not very familiar with Spyder patches so please excuse me if this is common knowledge, but is there an estimated release date of v3.2.9 ?

It isn't common knowledge (I actually don't know it myself); but given we haven't released 3.2.8 yet and we just released 3.2.7 it will probably be a bit, since we're currently unfunded.

but is there an estimated release date of v3.2.9 ?

Two months, perhaps a bit less.

FYI @angertdevsingh , since its made it into 3.2.8, looks like you'll get this a lot sooner than two months.

@CAM-Gerlach Thank you, that's great to hear!

In fact it should drop tomorrow morning, though it might take a few days for conda packages to be generated.

@ccordoba12, I don't want to clutter the thread too much, so I'll keep it brief but thank you so much for fixing the bug. It's allowed me to keep my workflow and my sanity!

Please thank @csabella, she was the one who fixed this bug.


PHBSB - ProvenHeadersBlockStoreBehavior

We need to create a new behavior- PHBSB which will be inherited from BSB.
It should be present only on PoS networks (instead of BSB).
It should override announcing logic- to PH peers we should announce with proven headers payload, for non-PH nodes- announce like we do it now.

In BSB we have
                case SendHeadersPayload sendHeadersPayload:
                    this.PreferHeaders = true;
                    break;

Same should be for SendProvenHeadersPayload except for we also will save height from where it's preferred over normal headers.
When we announce blocks we first check their height and if proven headers are preferred starting from before that height then we announce proven headers. If preferred height is after height of headers we will announce with normal headers.

Fixed


useTypescriptIncrementalApi increases initial build time

Hi,
I wanted to try the new option useTypescriptIncrementalApi. But our build time was as follows:
                                         initial                incremental
with 4 workers:                              31s                    10s
with one worker:                             50s                    30s
with useTypescriptIncrementalApi:           180s                    10s

Incremental was only change to one file, so a little strange to see the difference between 4 workers and 1 worker. Any insights?

Just to give more context - could you provide some info about your codebase:

how many files do you have and what is the total size?
is it possible for you to run directly tsc --watch and see how much time it takes?
are you using modules in your project?
if so, do you have circular dependencies between them?

Also, to give you some more history of useTypescriptIncrementalApi, it was inspired by compiling the sources of Visual Studio Code (which I encourage to do, just to have a better feeling). They are compiling the sources with tsc directly. Here are some very rough measurements that I have from my own repository and vscode repository (by old I mean useTypescriptIncrementalApi: false and 4 workers):

codebase
compiler setup
files
total size
initial compilation
incremental compilation
initial compilation (old)
incremental compilation (old)

my repo
webpack (dev-server)
150
1.5 MB
12 s
0.5-4s
8s
2s

vscode
tsc --watch
2500
20MB
20s
0.3-5s

Currently, I have a feeling that performance of typescript incremental compilation depends on how sources are organized: less globals -> faster incremental compilation.

Hi, are any updates on this issue ?
I have about 6k files in my repo. Compilation with "useTypescriptIncrementalApi: true" takes 300+s without linters, when with "useTypescriptIncrementalApi: false" about 90s with a single worker
TS v3.5.3
fork-ts-checker-webpack-plugin v1.5.0
webpack v4.40.2

Just FYI, I was recently trying to get some performance measurements for webpack+TS setup, but discovered that webpack watch mode time grows linearly with the number of files. I found this extremely discouraging and didn't do any further tests/investigation.
@Kavorka79, could you try running tsc --watch and see if it has better performance? (and, while we are at it, also try the tsc --incremental). This way we could know if there is still hope to fix something in the plugin.

There is a related issue in TypeScript repo: https://github.com/microsoft/TypeScript/issues/34119
If you have a problem with slow initial build with useTypescriptIncrementalApi, run tsc (with watch mode and without) independently and post the results in the issue. The more information we will give to the developers of TS the sooner they will resolve the issue.

@Kavorka79 , @IIIristraM
Please try fork-ts-checker-webpack-plugin@alpha - I've published a new version which should resolve this issue 🚀
I will close this issue to clean-up the backlog. If this release didn't solve the issue, we can re-open this :)


D:/librime-master/thirdparty/src/marisa-trie/lib/marisa/grimoire/io/reader.cc

Describe the bug
building marisa.
CMake Warning (dev) in CMakeLists.txt:
No project() command is present.  The top-level CMakeLists.txt file must
contain a literal, direct call to the project() command.  Add a line of
code such as
project(ProjectName)

near the top of the file, but after cmake_minimum_required().
CMake is pretending there is a "project(Project)" command on the first
line.
This warning is for project developers.  Use -Wno-dev to suppress it.
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.18363.
-- The C compiler identification is MSVC 19.29.30141.0
-- The CXX compiler identification is MSVC 19.29.30141.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: e:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.29.30133/bin/Hostx64/x86/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: e:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.29.30133/bin/Hostx64/x86/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
CMake Error at CMakeLists.txt:37 (add_library):
Cannot find source file:
D:/librime-master/thirdparty/src/marisa-trie/lib/marisa/grimoire/io/reader.cc

Tried extensions .c .C .c++ .cc .cpp .cxx .cu .mpp .m .M .mm .h .hh .h++
.hm .hpp .hxx .in .txx .f .F .for .f77 .f90 .f95 .f03 .ispc
CMake Error at CMakeLists.txt:37 (add_library):
No SOURCES given to target: marisa
CMake Generate step failed.  Build files cannot be regenerated correctly.
从github上下载的最新代码，第三方全部是最新的，其它的glog和yaml-cpp已经过了，boost也通过了。
To Reproduce
Steps to reproduce the bug:

Go to '...'
Click on '....'
Scroll down to '....'
See bug

Expected behavior
A clear and concise description of what you expected to happen.
Log
If applicable, add crash log to quick focus your problem.
Screenshots
If applicable, add screenshots to help explain your problem.
Flavor(please complete the following information):
Select your flavor:

[ ] ibus-rime
[ ] fcitx-rime
[ ] fcitx5-rime
[ ] Squirrel
[ ] Trime
[ ] Weasel

Package:

OS: [e.g. openSUSE Tumbleweed]
Version: [e.g. 5.0.7]
URI: [e.g. https://software.opensuse.org/package/fcitx5-rime?search_term=fcitx5-rime]

Additional context
Add any other context about the problem here.

請按照文檔說明的方法構建：
https://github.com/rime/librime/blob/master/README-windows.md


Display both Lent Total and Lent Lendable only if they are different

Description
Showing both of them just pollutes screen in case if they are equal.
TESTING STAGE

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read CONTRIBUTING.md
[x] I fully understand Github Flow.
[ ] My code adheres to the code style of this project.
[ ] I have updated the documentation in /docs if I have changed the config, arguments, logic in how the bot works, or anything that understandably needs a documentation change.
[ ] I have updated the config file accordingly if my change requires a new configuration setting or changes an existing one.
[ ] I have tested the bot with no issues for 24 continuous hours. If issues were experienced, they have been patched and tested again.

BTW, coding style covers only Python and regular contributors can not set labels (however CONTRIBUTING.md tolds them so)

Great, I wanted to do that.  I'm also thinking if the effective rate calculation needs to be changes to calculated compare to lendable amount and not total - not sure.

Don't worry about it, I would rather @rnevet created the JS style requirements. And I do realise the labels are unable to be set by non-contributors, but they are there to guide everybody.

@rnevet As I told before, it should not depend on lent amount. Problem solved, but you probably will disagree :-)

@Evanito new contributors might lose time trying to figure out how to add labels. I think it should noted that they are not expected to set them.

@yura-pakhuchiy being consistent with your own opinion is a good trait. :)
We can think about some url params for personal customization of the html.

lgtm

lgtm

We can think about some url params for personal customization of the html.

@rnevet Sounds good. I will make a patch.


Radix-2 Problem on Windows Intel HD 4400

Hi,
I ran into a problem with clFFT today: I'm doing 1D power-of-2 complex to complex transforms on Windows using an Intel HD 4400 in a Surface Pro 3 i5.
The problem only appears in power-of-2 transforms which seems to be wrong (see images below), all other supported FFT sizes seem fine. I was not able to reproduce the problem on any other platform (CUDA device on MacBook using OSx and Linux , Intel HD device on MacBook using OSx).
I have a reproducer in python (using gpyfft) but I see the exact same behaviour when using clFFT from a C++ application so I rule out gpyfft or pyopencl as a source of the problem. I suspect clFFT and its interaction with the OpenCL driver for Intel HD on Windows. Both the master and the develop branch of clFFT have this problem.
Below you see images generated with the same code, only the used device (CPU/GPU) is changed. If I compare the generated kernel files for both CPU and GPU, they are identical.
Update 4
It looks like Intel has confirmed this and is looking into it: https://software.intel.com/en-us/forums/topic/559915#comment-1828156 It is probably not a clFFT problem though. I'll update this issue with information from Intel as this might potentially affect other clFFT users.
Update 3
I homed in on the problem and could reduce the FFT kernel to a write-read on local memory problem. The gist that contains the reproducer can be found here https://gist.github.com/sschaetz/f37e15ec2f059e13777b I also submitted a request for help with Intel here: https://software.intel.com/en-us/forums/topic/559915
Update 2
It looks like if the access pattern to local memory is done the way it is done for powers of 2 FFTs, the memory barrier between the access is ignored. If I mix up the memory access pattern (for example by adding +1 everywhere) it works.
Update
I could gather more information:
If I run the same transform multiple times, the output differs each time on this particular GPU. The Surface 3 (not Pro) GPU does not have this problem. Transforms up to and including 32 do not have this problem. The problem starts to appear with a transform size of 64. I started debugging the OpenCL code. The major difference between 32 samples and 64 samples seems to be FwdRad8B1. I run this code in a stand-alone fashion and it seems to run correctly on CPU on GPU. Major difference between the two devices I can see:
CPU: max_work_group_size 8192, max_work_item_dimensions 8192 8192 8192
GPU: max_work_group_size 512, max_work_item_dimensions 512 512 512
Has anybody encountered this problem? Any suggestions how this problem could be tracked down? Here is the reproducer and the output:

# coding: utf-8

# In[1]:

get_ipython().magic(u'matplotlib inline')
get_ipython().magic(u"config InlineBackend.figure_format = 'retina'")
import numpy as np
import matplotlib.pyplot as plt
import os
import pyopencl as cl
import pyopencl.array
import pyopencl.array as cla
import scipy.io
import gpyfft
import matplotlib.pylab as pylab
import time

G = gpyfft.GpyFFT(debug=True)

print "clAmdFft Version: %d.%d.%d"%(G.get_version())

# In[11]:

# Dimensions.
d1 = 1024
d2 = 1024

# Context, queue, FFT plan.
## Step #1. Obtain an OpenCL platform.
platform = cl.get_platforms()[0]

## It would be necessary to add some code to check the check the support for
## the necessary platform extensions with platform.extensions

## Step #2. Obtain a device id for at least one device (accelerator).
device = platform.get_devices()[1]
print device

## Step #3. Create a context for the selected device.
ctx = cl.Context([device])

queue = cl.CommandQueue(ctx,
                properties=cl.command_queue_properties.PROFILING_ENABLE)

# data
bounds = (d1, d2)
size = d1 * d2

inputData = (np.random.randn(size) + 1.j*np.random.randn(size)).astype(np.complex64).reshape(d1,d2)
outputData = (10000 * np.ones(size) + 1.j*10000 * np.ones(size)).astype(np.complex64).reshape(d1,d2)

# clFFT plan
plan = G.create_plan(ctx, (d1, ))
plan.batch_size = d2
plan.strides_in = (1024, )
plan.strides_out = (1024, )
plan.distances = (1, 1)
plan.inplace = False
plan.bake(queue)

mf = cl.mem_flags
inputDataBuffer = cl.Buffer(ctx, mf.READ_WRITE, inputData.nbytes)
outputDataBuffer = cl.Buffer(ctx, mf.READ_WRITE, outputData.nbytes)
cl.enqueue_copy(queue, inputDataBuffer, inputData)
cl.enqueue_copy(queue, outputDataBuffer, outputData)
plan.enqueue_transform((queue,), (inputDataBuffer,), (outputDataBuffer,))
cl.enqueue_copy(queue, inputData, inputDataBuffer)
cl.enqueue_copy(queue, outputData, outputDataBuffer)
queue.finish()

fig, ax = plt.subplots(figsize=(10,10))

cax = ax.imshow(np.transpose(np.abs(outputData)), cmap='gray')
cbar = fig.colorbar(cax)
fig.show()

The output of this code, if I select the Intel HD 4400 can be seen in this image:

If I select the CPU as a device, the output is correct:

If anybody is curious, this was a OpenCL Intel driver issue that is supposedly fixed in the next OpenCL driver. Here is the entire interaction with Intel: https://software.intel.com/en-us/comment/1832865#comment-1832865


chore: print commands to start the client and remove unused --non-itera…

print commands to start the client and remove unused --non-iteractive in dockerfile
Background
What does this PR do?
As the pnpm start command will not start the web client in localhost:5173 but the log says visit it, so I changed the output log.
Also removed the --non-iteractive args in Dockerfile as it is no longer read by the agent.
What kind of change is this?
Bug fixes (non-breaking change which fixes an issue)

Documentation changes needed?

Testing
Where should a reviewer start?
Detailed testing steps

LGTM, but @yang-han you need to target the develop branch instead of main

LGTM, but @yang-han you need to target the develop branch instead of main

ok, will do

LGTM, but @yang-han you need to target the develop branch instead of main

in #1163


Incremental updates to an index

I'm interested in using LanceDB for an embedded application where vectors can be added over time and queried efficiently without having to do a full reindex every time. Is this feature supported, and if not is it on the roadmap? I'd be fine if the partitions/sub-vectors from the initial index remained constant, as they're likely to still be representative of the data.
Many thanks for your great work on this library!

that's definitely on the roadmap, just don't have a firm date on it yet.
right now if you add vectors after you create an index, that's still searchable. You shouldn't see a meaningful performance penalty until you accumulate more than like 20K new rows or smth

Oh that's awesome, I wasn't aware additions would be searchable. That's great to know! Do deletions work after indexing? For my use cases I can probably proceed with sporadic re-indexing now that I know this 🚀

yeah, deletions will introduce some additional latency (since we need to check the bit mask), but deleted rows will not show up in search results

We should be able to implement this for IVF_PQ right now.
We can use LanceDataset.index_statistics to get the current centroids and then pass those into LanceDataset.create_index as ivf_centroids.


Added jsonsecretstore

Description
Added JSON Secret Store as proposed in: #327
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #327
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Thanks @cmendible for the contribution, this will be very helpful for demos, samples and dev environments. Could you please add a doc how to PR as well?

Thanks @cmendible for the contribution, this will be very helpful for demos, samples and dev environments. Could you please add a doc how to PR as well?

@amanbha Should I open this PR in the docs repo?

Thanks @cmendible for the contribution, this will be very helpful for demos, samples and dev environments. Could you please add a doc how to PR as well?

@amanbha Should I open this PR in the docs repo?

Yes, please open PR in docs repo under https://github.com/dapr/docs/tree/master/howto/setup-secret-store


Use the displayName instead of a type for IdeCreate

Background:
Use the displayName instead of type for IdeCreate CDAI children component checking

Closing based on above comment


Nuxt 3 with multiple composables does not persist

I've tried multiple solutions but I cannot seem to get my state persisted within Nuxt 3, here is an example with the pinia Options store. The same occurs when I use the Composition Api:
https://github.com/rowanfuchs/nuxt3-pinia-persist
Is there something I'm doing wrong? what can I do different to make sure the state is persisted.

Hi, in the given repo, you are not defining stores properly.
If you want to use options API like in https://github.com/rowanfuchs/nuxt3-pinia-persist , the persist option should be at the same level as other options (state, actions...).
import { defineStore } from "pinia";

export const useAuthStore = defineStore("auth", {
    state: () => {
        return {
            authenticated: false
        };
    },
    getters: {
        getAuthenticated: (state) => state.authenticated
    },
    actions: {
        setAuthenticated: async function (payload) {
            this.authenticated = payload
        }
    },
    persist: true,
});

Thank you for taking the time to correct my mistake 😊. I will leave the repository up as a reference for others.


How do I change the datasource for a chart?

Make sure these boxes are checked before submitting your issue - thank you!

[X] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[X] I have reproduced the issue with at least the latest released version of superset.
[X] I have checked the issue tracker for the same issue and I haven't found one similar.

Superset version
0.30
Expected results
I should be able to pick from the list of datasources, or otherwise be able to switch to another pre-created datasource.
Actual results
I'm taking to the confusing Datasource Editor (per this PR). What do I do from here? How do I actually switch to another datasource?
Steps to reproduce

Open a Chart
Tap on the name of the Datasource

Fixed in master and 0.31.0rc17

Thanks to @michellethomas !

Wonderful, thank you!

Is it necessary to set up some authorisations to be able to change Datasource .
When I try it I get an "Access Denied" error message.
Here is what I can see in the logs:
URL: https://zzz.fr/datasource/get/table/49/
Status: 401 UNAUTHORIZED


Re-Implement Normal Map Generation

The old headless mode would be able to import a heightmap and generate a normal map from that. Functionality should be added so that on export the normal map gets generated if a heightmap has been imported.

Resolved by #111


Most jailbreak attempts kernel panics at oobPCI

The title says it all, most attempts at jailbreaking end up kernel panicking (screen freeze, flashes magenta, and reboots) Is there a way to reduce the chances of this happening or do i just have to spam that button until it works?

you have to retry until it works


memberlist DNS resolution error

I am running master to play with the new memberlist clustering in Kubernetes. I have an odd DNS resolution error... It doesn't appear to interfere with functionality but wanted to log it in case anyone else sees it and is confused too.
So this is my service and statefulset:
apiVersion: v1
kind: Service
metadata:
  name: alertmanager-webui
  namespace: monitoring
  labels:
    app: alertmanager
    component: core
  annotations:
    prometheus.io/scrape: "true"
spec:
  type: LoadBalancer
  ports:
    - port: 9093
      targetPort: web
      name: web
  selector:
    app: alertmanager
    component: core
---
apiVersion: v1
kind: Service
metadata:
  name: alertmanager
  namespace: monitoring
  labels:
    app: alertmanager
    component: core
spec:
  ports:
  - port: 8001
    name: cluster
  type: ClusterIP
  clusterIP: None
  selector:
    app: alertmanager
    component: core
---
apiVersion: apps/v1beta2
kind: StatefulSet
metadata:
  name: alertmanager
  namespace: monitoring
  labels:
    app: alertmanager
    component: core
spec:
  updateStrategy:
    type: RollingUpdate
  replicas: 3
  selector:
    matchLabels:
      app: alertmanager
  serviceName: alertmanager
  template:
    metadata:
      labels:
        app: alertmanager
        component: core
    spec:
      containers:
      - name: alertmanager
        image: quay.io/prometheus/alertmanager:master
        imagePullPolicy: IfNotPresent
        command:
        - "sh"
        - "-c"
        args:
        - /bin/alertmanager
            --config.file=/etc/alertmanager/config.yml
            --web.listen-address=<IP_ADDRESS>:9093
            --cluster.listen-address=<IP_ADDRESS>:8001
            --storage.path=/alertmanager
            --cluster.peer="alertmanager-0.alertmanager.monitoring.svc:8001"
            --cluster.peer="alertmanager-1.alertmanager.monitoring.svc:8001"
            --cluster.peer="alertmanager-2.alertmanager.monitoring.svc:8001"
            --log.level=debug
        ports:
        - containerPort: 9093
          name: web
          protocol: TCP
        - containerPort: 8001
          name: cluster
          protocol: TCP
        livenessProbe:
          httpGet:
            path: /api/v1/status
            port: web
            scheme: HTTP
          failureThreshold: 10
        readinessProbe:
          failureThreshold: 10
          httpGet:
            path: /api/v1/status
            port: web
            scheme: HTTP
          initialDelaySeconds: 3
          periodSeconds: 5
          successThreshold: 1
          timeoutSeconds: 3
        volumeMounts:
        - name: alertmanager-config-volume
          mountPath: /etc/alertmanager/
        - name: alertmanager-data-volume
          mountPath: /alertmanager/
      volumes:
      - name: alertmanager-config-volume
        configMap:
          name: alertmanager-server-conf
      - name: alertmanager-data-volume
        emptyDir: {}

This is the debug output from one of the alertmanagers:
addrs <IP_ADDRESS>:8001
level=info ts=2018-02-27T03:33:46.165947329Z caller=main.go:151 msg="Starting Alertmanager" version="(version=0.14.0, branch=master, revision=29e441f88f3e758ff6a1c8a536fdb752c5b32af9)"
level=info ts=2018-02-27T03:33:46.166043263Z caller=main.go:152 build_context="(go=go1.9.2, user=root@19dec1fcf7af, date=20180222-15:01:16)"
level=debug ts=2018-02-27T03:33:46.24821988Z caller=cluster.go:75 component=cluster msg="resolved peers to following addresses" peers=alertmanager-0.alertmanager.monitoring.svc:8001,alertmanager-1.alertmanager.monitoring.svc:8001,alertmanager-2.alertmanager.monitoring.svc:8001
level=debug ts=2018-02-27T03:33:46.262930727Z caller=cluster.go:347 component=cluster received=NotifyJoin node=01C7AN5GMFPHBMYV1YQK4D247D addr=<IP_ADDRESS>:8001
level=warn ts=2018-02-27T03:33:46.326634461Z caller=cluster.go:122 component=cluster msg="failed to join cluster" err="3 errors occurred:\n\n* Failed to resolve alertmanager-0.alertmanager.monitoring.svc:8001: lookup alertmanager-0.alertmanager.monitoring.svc on <IP_ADDRESS>:53: no such host\n* Failed to resolve alertmanager-1.alertmanager.monitoring.svc:8001: lookup alertmanager-1.alertmanager.monitoring.svc on <IP_ADDRESS>:53: no such host\n* Failed to resolve alertmanager-2.alertmanager.monitoring.svc:8001: lookup alertmanager-2.alertmanager.monitoring.svc on <IP_ADDRESS>:53: no such host"

HOWEVER - the cluster is working:

So I am not sure why it's telling me it can't resolve the endpoints... shrugs

It seems that this AlertManager instance tries to resolve the names while they are not yet registered in the k8s DNS server. I suspect that the cluster state is ok because another instance of the cluster has managed to connect to this instance and has broadcast the peer information.
@fabxc would it make sense to add more retries for the DNS resolution?

In another context but still using StatefulSet, I've investigated a similar configuration (eg deploying AlertManager using statefulet). Here is what happens using the OP example:

k8s launches the first pod (alertmanager-0). When the alertmanager process starts, it can't resolved any of the peer addresses hence the errors in the log.
once alertmanager-0 is running and ready, k8s launches alertmanager-1. This time when the alertmanager process starts, it resolves alertmanager-0.alertmanager.monitoring.svc and connects to it. alertmanager-2.alertmanager.monitoring.svc can't be resolved though so it will logged.
once alertmanager-1 is running and ready, k8s launches alertmanager-2. When the alertmanager process starts, it resolves both alertmanager-0.alertmanager.monitoring.svc and alertmanager-1.alertmanager.monitoring.svc, and it connects to them.
The cluster is now complete.

In the end, it works because the default pod management for statefulsets is to start pods sequentially. If you were to set podManagementPolicy: Parallel (more here), it would fall apart.

@jamtur01 @simonpasquier Thanks for this, works fine for me it seems.
Only issue is that prometheus seems to want to scrape on port 8001 as well as 9093 when looking at targets in Prometheus. Is there a way to avoid this?

@csghuser can you ask the question on the prometheus mailing list so that the discussion can benefit to everyone else? Also please provide the details on your Prometheus configuration.

I managed to fix it by adding the port:
---
apiVersion: v1
kind: Service
metadata:
  annotations:
    prometheus.io/scrape: 'true'
    prometheus.io/port: '9093'


fix(tests): enable EventCb for KubeEventManager test

Overview
Fix integration tests for KubeEventManager and update "kind" to v0.10.0

What this PR does / why we need it
It will fix failed jobs when merge to master: https://github.com/flant/shell-operator/runs/2251945392?check_suite_focus=true
fix #255

Special notes for your reviewer
Does this PR introduce a user-facing change?

Force push with a small fix in go.mod.


Outstanding work

Looking forward to seeing your paper and code soon.

The draft has been released. The code will be available within this week if everything goes well.


The server takes ~20 minutes to start

Brief Description:
The server requires about 20 minutes to fully start.  In the meantime, the client can connect to it, but can not get past "step 1: choose a language".   It certainly isnt obvious that one should wait 20 minutes before starting to use the client.
Operating System (OS and version):  linux
OpenBazaar version: master
Hardware: intel
Reproducible (Always / Almost Always / Sometimes / Rarely / Couldn't Reproduce):
always
Steps to reproduce:

start the server
start the client
The client will connect to the localhost server and present the step 1 - language menu
The problem is that after selecting my language and clicking "next", nothing happens.
wait ~20 minutes for the server to report several errors (see below) before it finally starts up

Observed Behavior:
The problem is that while the server is starting the client is stuck on step1 selecting my language - clicking "next", nothing happens.  There is no indication that the server is not yet initialized.
Expected Behavior:
At a minimum, the client should indicate that the server is still starting up.
Additional info (links, images, etc go here):
root@463fc6ac9e28:/ob/OpenBazaar-Server# python openbazaard.py start
________                      __________
\_____  \ ______   ____   ____\______   \_____  _____________  _____ _______
 /   |   \\____ \_/ __ \ /    \|    |  _/\__  \ \___   /\__  \ \__  \\_  __ \
/    |    \  |_> >  ___/|   |  \    |   \ / __ \_/    /  / __ \_/ __ \|  | \/
\_______  /   __/ \___  >___|  /______  /(____  /_____ \(____  (____  /__|
        \/|__|        \/     \/       \/      \/      \/     \/     \/

OpenBazaar Server 0.2.6 starting...
2017-07-22 18:15:40+0000 [Server] [WARNING] no known neighbors, so not writing to cache.
Unhandled error in Deferred:
2017-07-22 18:30:30+0000 [-] Unhandled error in Deferred:

Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/base.py", line 1194, in run
    self.mainLoop()
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/base.py", line 1203, in mainLoop
    self.runUntilCurrent()
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/base.py", line 825, in runUntilCurrent
    call.func(*call.args, **call.kw)
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/task.py", line 239, in __call__
    d = defer.maybeDeferred(self.f, *self.a, **self.kw)
--- <exception caught here> ---
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/defer.py", line 150, in maybeDeferred
    result = f(*args, **kw)
  File "/ob/OpenBazaar-Server/api/restapi.py", line 78, in _keep_sessions_alive
    session.touch()
exceptions.AttributeError: 'str' object has no attribute 'touch'
2017-07-22 18:30:30+0000 [-] Unhandled Error
	Traceback (most recent call last):
	  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/base.py", line 1194, in run
	    self.mainLoop()
	  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/base.py", line 1203, in mainLoop
	    self.runUntilCurrent()
	  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/base.py", line 825, in runUntilCurrent
	    call.func(*call.args, **call.kw)
	  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/task.py", line 239, in __call__
	    d = defer.maybeDeferred(self.f, *self.a, **self.kw)
	--- <exception caught here> ---
	  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/defer.py", line 150, in maybeDeferred
	    result = f(*args, **kw)
	  File "/ob/OpenBazaar-Server/api/restapi.py", line 78, in _keep_sessions_alive
	    session.touch()
	exceptions.AttributeError: 'str' object has no attribute 'touch'

2017-07-22 18:42:11+0000 [MarketProtocol] [WARNING] could not find any listings in the database

What type of machine are you running this on? Specs?

Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz
I dont think it was cpu-bound.  I didnt see any significant cpu, disk or network i/o.
Today I have better results.  The time from starting the server and client to when the client is able to move to the second welcome page is much faster - just a few minutes.  Still, I think a "please wait, server is connecting to the network" message would help a lot.


A dummy commit to capture the screenshot for pull request

Can you mark this as spam so it won't count toward my Hacktoberfest. I don't want this to count toward the PRs


Comment types checked out "Assignee" but still showing that activity/comment in issue page

Description
I checked off Comment type "Assignee" but still showing that activity/comment in issue page. Snapshots attached. It's happening in our existing version 1.20.5  &  In Gitea test Instance 1.22.0+dev-625-g3e8414179
https://try.gitea.io/yashmehta2nd/dasdasdad/issues/1
Gitea Version
1.22.0+dev-625-g3e8414179
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
No response
Operating System
Asustor Naas
How are you running Gitea?
Selfhosted via Docker on Asustor Naas v1.20.5
Database
None

But in the screenshot 1, "Assignee" isn't checked off?

Oh, it's opposite. Apologies team.


Add trade stations

Map was already done by ubaser / devi.
This just adds gridfill + a console to buy / sell goods.
draft coz lol. FTL engines in future PR.
Requires https://github.com/space-wizards/RobustToolbox/pull/4842
:cl:

add: Added trade stations for cargo instead of FTL.

Hello. Can you please show your changes in a video?


Access to the pretrained model for the QDMR parser

Hi,
Thanks for sharing the code!
I wonder whether is possible to access the pretrained model for the QDMR parser which is based on T5-large?
Thanks

Hello,
The checkpoint of our trained QDMR parser has been added to the README. The model can also be downloaded from here.


Merge p4ovs repository

https://cbjenkins-fm.devtools.intel.com/teams-nex-epgsw-ipdkci/job/nex-epgsw-ipdkci/job/GitHub_MultiBranch_Pipelines/job/ovs/job/PR-29/ Result:FAILURE JenkinsConsole:https://s3.ipdk.io/nex-epgsw-ipdkci/GitHub_MultiBranch_Pipelines/ovs/PR-29-2/index.html GitCommit:https://github.com/ipdk-io/ovs/commits/836c7877648335e02bcede799465d2b916b98111

Closing this PR. Merge has been done via https://github.com/ipdk-io/ovs/pull/30

This review to merge P4-OVS code to open source has not been taken forward, as this merge review has a build issue.
Hence merge is done via PR #30.


Initial README Version

Please check out the new README and suggest any changes as neccessary. Thank you!

I think everything is perfected

There is no mention of what kind of program this will be. Web, mobile or so on.

There is no mention of what kind of program this will be. Web, mobile or so on.

Good point. Will fix tomorrow.

@ago-ble Could it be in a separate "Goal" section as I think that "Vision" is more for the end user that doesn't care what type of app it is, while the "Goal" could be more related to technical development things? By the way, SPA screams WEB and it is commonly used nowadays to refer to the kind of app that we are going to build.

@ago-ble Could it be in a separate "Goal" section as I think that "Vision" is more for the end user that doesn't care what type of app it is, while the "Goal" could be more related to technical development things? By the way, SPA screams WEB and it is commonly used nowadays to refer to the kind of app that we are going to build.

Yes, I think now everything is understandable and clear.

It seems like there are no more disagreements at this point, so I have merged the PR. We may make further adjustments after the progress meeting.


Add WorkItem Type Task

Add in the list of the workitem type "Task" When this type is selected, show the "Activity" drop down, but we can type some other string like "Support" Thanks

We need to work item type Task and DCR (Design Change Request) , can you please have a look at that?
Thanks,Sachin


Ability to hide the zoomToExtent button on a zoom-restricted layer

Description
Currently we always add the zoomBtn if the layer is zoom restricted.
However, we should allow configuration to turn this off if the functionality is not required / contextually appropriate.
 // Create a div for the magnifying glass icon
  layer.zoomBtn = layer.tables
    && mapp.utils.html.node`<button
      data-id="zoom-to"
      title=${mapp.dictionary.zoom_to}
      class="mask-icon search"
      onclick=${() => zoomToRange(layer)}>`

I propose we instead do this, by making it an OR assignment we can make it false to hide it.
e.g.
"zoomBtn": false

 // Create a div for the magnifying glass icon
  layer.zoomBtn ||= layer.tables
    && mapp.utils.html.node`<button
      data-id="zoom-to"
      title=${mapp.dictionary.zoom_to}
      class="mask-icon search"
      onclick=${() => zoomToRange(layer)}>`

(Asana Ticket)
Type of Issue
Please delete options that are not relevant, and select all options that apply.

✅ New Feature

This shouldn't be shown by default and not shown with the false flag which is counter intuitive to mapp json.
The default should be to not show the button and show if configured.
This also opens a can of worms in regards to other elements. If there is an option for the zoomButton, should the zoomToExtentBtn be optional as well.
Should all of these flags be add to the layer JSON or perhaps should there be a config object.
layer.view === null, no view is created. Otherwise view will be the view element. We cannot change this.

The only option is see to not break the current behaviour without spamming more and more flags into the layer json is by creating a default object layer.viewConfig = {zoomBtn, zoomToExtentBtn, panelOrder} etc. And you can provide viewConfig taking away the options to change the behaviour.

All this should happen in a viewConfig method in the module to reduce the complixity of the view method exported from the module.


Banner options and osx features

Banners Option
Added ability to select from the list of banner messages using -i option.
First, list available banner messages using
$ ssh-honeypot -b

Then, select a banner message by index
$ ssh-honeypot -i 2 -r ...

Alternately, specify a custom banner message with a string
$ ssh-honeypot -b "my banner message here!" -r ...

OSX
Added ability to build on OSX. Readme and install files were updated to reflect this.

Thanks! I don't have a Mac right now to test it on, everything looks right.

Fixed a few minor formatting and grammar mistakes and a bug with signal(). bin/.ignore was also re-added, because it fails without bin/ directory being present if you clone it fresh.


Reaching paper performances on TSUNAMI and GSV

Hi,
At first ı want to say thank you for sharing your work.
I try to train PCD dataset with your network but the performances that ı get far from the performances that shared in the paper so is there anything that I should pay extra attention or any advices to reach that performances which shared in the paper ?
Thanks in advance

The F1 score implementation looks weird. Notice it should be run on two classes: "change" and "No change".
https://torchmetrics.readthedocs.io/en/stable/classification/f1_score.html


make HA Addon

just copying the existing code from my own repo

Maybe this can be put in separate repository ? putting inside currect directory all this files will be copied also to the juicepassproxy docker image.

that will be true if the target archive is updated. for now I just did a straight copy and the docker image build doesn't copy from the repo directly

Any chance we can find a way to merge this? Rather this or another repo it was the easiest way to integrate everything with home assistant. I'm happy to help if something needs to move forward

I gutted my juicebox for parts for an openevse - so I won't be maintaining this anymore.

I might take it on especially if we can merge it into juice rescue to keep it central. Thoughts @ivanfmartinez?
@wozz  How'd the openevse change go? I was considering it, was there a guide you followed?

I might take it on especially if we can merge it into juice rescue to keep it central. Thoughts @ivanfmartinez?
@wozz  How'd the openevse change go? I was considering it, was there a guide you followed?

there are docs on their site. the biggest issue was some of the gauge wires were bigger in the juicebox so I had to buy an extra plug to fit. I basically only ended up reusing the car plug/wire. If you had a custom case, or got things to fit in the juicebox case, you could reuse a bit more.


[Bug] use of unstable library feature 'matches_macro'

🐛 Bug

To reproduce

Compiling program
Compiling move-core-types v0.1.0 (/home/inplus/blockchain/libra/language/move-core/types)
error[E0658]: use of unstable library feature 'matches_macro'
--> language/move-core/types/src/identifier.rs:42:9
|
42 |         matches!(c, '_' | 'a'..='z' | 'A'..='Z' | '0'..='9')
|         ^^^^^^^
|
= note: for more information, see https://github.com/rust-lang/rust/issues/65721

error: aborting due to previous error
For more information about this error, try rustc --explain E0658.
error: could not compile move-core-types.
Code snippet to reproduce
# Your code goes here
# Please make sure it does not require any external dependencies

Stack trace/error message
// Paste the output here

Expected Behavior

System information
Please complete the following information:

Additional context
Add any other context about the problem here.

The matches macro was stabilized in rust 1.42.0. Are you on an old branch that still uses older compiler? https://github.com/libra/libra/pull/2922 was about a month ago.

Reopen if you get more detailed steps to reproduce.

i got the same issue.

Same error:
env :
Distributor ID: Ubuntu
Description: Ubuntu 18.04.4 LTS
Release: 18.04
Codename: bionic

I'm running into this same issue on the same environment (Ubuntu 18.04.4)
Seems like it may be related to that Ubuntu version for some reason?

Run cargo build at the project root, do not at sub crate.

For me the issue was that I needed to be on rust nightly

For my case, on Windows 10, it's solved after rustup update.

@bingli224 Also fixed for me in Linux by doing rustup update. My rustc was at 1.41, now at 1.47.


Missing attribute in Illuminate\Notifications\Messages\SlackAttachment

In class Illuminate\Notifications\Messages\SlackAttachment there is at least one missing attr -  pretext.
To prevent this in Illuminate\Notifications\Messages\SlackAttachment should be a options property:
     /**
     * The attachment's custom options.
     *
     * @var int
     */
    public $options = [];

and it should be merged here:
https://github.com/laravel/framework/blob/6a64c8d37143322c5a02b978c499d0e2b303c6f6/src/Illuminate/Notifications/Channels/SlackWebhookChannel.php#L84-L100

added here https://github.com/laravel/framework/pull/23075

@themsaid why you don't like idea with options prop? It's a common pattern for such cases, because there can be other missing attrs or/and slack API also can be changed any time.

I don't mind it, but saw all attributes assigned fluently so it made sense to me, you can PR the options field if you want.

When you close issue without any comment to my suggestion, for me it's mean you against.

Man I've opened a PR to add the missing attribute you want :) That's the only reason I closed the issue, because it's handled already.


Array .at() on readonly tuples

Unless there is a good reason for typescript not to return the correct type when accessing a tuple with .at it feels like a good candidate for this library.
This is how it is currently:
const a = [false, 1, '2'] as const

const b = a.at(0)
//    ^? const b: false | 1 | "2" | undefined

const c = a.at(-1)
//    ^? const c: false | 1 | "2" | undefined

But with this library it could be:
const a = [false, 1, '2'] as const

const b = a.at(0)
//    ^? const b: false

const c = a.at(-1)
//    ^? const c: "2"

As long as the index is positive, the following works pretty well:
interface ReadonlyArray<T> {
	/**
	 * Returns the item located at the specified index.
	 * @param index The zero-based index of the desired code unit. A negative index will count back from the last item.
	 */
	at<I extends number>(index: I): this[I];
}

We can detect (and extract from) negative numbers with something like:
interface ReadonlyArray<T> {
	/**
	 * Returns the item located at the specified index.
	 * @param index The zero-based index of the desired code unit. A negative index will count back from the last item.
	 */
	at<I extends number>(index: I): `${I}` extends `-${infer J extends number}` ? T : this[I];
}

But at that point I'm only typing it as T (which is the default implementation, meaning "the type of any member of the tuple") because I don't know how to index in reverse.

Hmmmm, I think I'm happy with the current workaround, which is to just use a[0] - I think the overhead isn't necessarily worth it here. I'll leave this open in case others disagree.

FYI here is the implementation in type-plus:
https://github.com/unional/type-plus/blob/main/ts/array/array.ts#L17

@mattpocock I don't understand why you don't want to add this? If I understood correctly, the point of ts-reset is make it so that TypeScript types reflect reality. ReadonlyArray:s contain the .at function, but TypeScript incorrectly says it doesn't. That just plain false. "improving types for common JavaScript API's" fits this exact case.
The DX improvement of being able to write arr.at(-1) instead of every time having to write arr[arr.length - 1] is so nice. I don't know why we would want to regress in DX?


cuelang.org playground "About" page is 404

The playground ( https://cuelang.org/play/ ) links to an "About CUE Playground" page ( https://cuelang.org/about-play ) which 404's.

Hm, I fail to find any such page in the new website. cc @myitcv @jpluscplusm

For reference:

Thanks @infogulch, indeed a 404.
Moving to the docs-and-content repo.


[Enhancement] Improve error messages

Motivation / Problem
Error messages are not really useful to the user. See access denied because of asset whitelists as an example below:
When logged in with an account that is not on whitelist the use section does not indicate any restrictions. There is no message for the user before buying, that access is not granted / account is not found on whitelist:

User is then able to buy datatokens (this is wanted behaviour - at least for ocean market) but the error message does not give any information as to why the compute job could not be started:

Solution

Provide information for user that access is restricted (maybe alert before buying or similar)
Improve error messages

Alternatives
Any alternatives are welcome to be discussed in this issue.

@moritzkirstein I was thinking of putting a short warning right before the "buy compute job" button (the actual message is up for debate)

And if the user decides to still try to start the job the error message will remind him why it's not working

Do you think the first warning should be more prominent (maybe a colored box or a bigger font size)?

I like the idea. Maybe let's use the alert component here and use a warning alert.
Warning:
Your account is not authorized to access this data service.

Better?


channels..patches doesn't work if supportedSystems contains more than 1 architecture

After adding a patch to channels.nixpkgs.patches, I get the following error:
error: a 'aarch64-linux' with features {} is required to build '/nix/store/is49g8n6gwyk91aa7pq6n1bnfy7pynmv-nixpkgs-patched-83667ff.drv', but I am a 'x86_64-linux' with features {benchmark, big-parallel, kvm, nixos-test}

If I then restrict supportedSystems to only include x86_64-linux, it works just fine on this machine. However, that's not an option in my case because I still need to support an aarch64-linux host.

OK, I think that makes sense now. I am getting this error when running nixos-rebuild, not just nix flake show or nix flake check. So I still think there is a bug somewhere.

Alright, it seems that the only solution here is to just have binfmt and/or build machines with required architectures since it is not possible to resolve this in any other way due to nix purity.
So the issue here is to patch nixpkgs we require patch command and since everything needs to be pure - patch needs to be taken from aarch64-linux packages which we can not execute since we are on the wrong architecture.


add missing keywords

fixes #111

Coverage: 96.714% (+0.2%) from 96.517% when pulling 7c603f227764fee3569789ffe22dbfb265299d27 on add_missing_keywords into 4f8de1f4114b8670e0b1af6eb1eee7687f53a453 on master.


Verify server identity

[x] After socket connected, server send identity to client
[x] If connection drops, during reconnect, use previous saved server identity to know is same server or not
[ ] Require #6 to continue
[x] If failed to verify server identity after connect

[ ] queue an external action defined in config
[x] Server will send a json string, if failed to parse

[ ] also treat it as failed (is unknown third party server).

[x] If failed to connect within certain retry times

[ ] queue an external action defined in config

[ ] Verify only preshared key, any other is not necessary because server may reboot

[ ] Remove verify of random string on client
[ ] Remove random string in response
[ ] Remove server version verify
[ ] Only keep server name verify, other property will be used to see if response can be parsed as JSON

Verify only preshared key
While implementing server verify, an idea is if preshared key is match,
allow to update server identity (name, version, random string),
realized soon only preshared key verify is necessary and server name is optional.


List of Multiples

Before jumping into Pr first comment for assign.All Problem from Edabit. Link is mandatory to add
Problem statement: Create a function that takes two numbers as arguments (num, length) and returns a list of multiples of num up to length.
Problem Link: https://edabit.com/challenge/BuwHwPvt92yw574zB

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

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

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

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

I call dibs

[✓]  Task Assigned
@gokulrajanpillai

@aayushi-droid  can you add me as contributor to make the PR?

@gokulrajanpillai  yes sure, I am inviting you in org, check your email.
Task - Create issue from Edabit problem, make sure to avoid duplicate question. use template for making issue.Assign people according to issues.
Best of Luck

I would like to work on this.


Support oidc auth provider

Support clouds (such as IBM Kubernetes Service) that use the oidc authorization provider.
Resolves https://github.com/aspenmesh/istio-vet/issues/40

Thanks for merging this!  Will this be pushed to the Docker image quay.io/aspenmesh/istio-vet:master?

yes as quay.io/aspenmesh/istio-vet:master and quay.io/aspenmesh/istio-vet:master-52. We need to make the tags more useful like commit SHAs if users want to not always use the latest.


Migrate code from codeplex to github

Forum thread: https://github.com/MarimerLLC/cslaforum/issues/2


Document OneMinusFidelity.__call__

Refer to https://www.sphinx-doc.org/en/master/usage/extensions/autodoc.html#directive-option-automodule-special-members

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

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 95.231%

Files with Coverage Reduction
New Missed Lines
%

qiskit_addon_aqc_tensor/simulation/explicit_gradient.py
1
93.68%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
599

Relevant Lines:
629

💛  - Coveralls


feat(lightspeed-backend)!: migrate to the new backend and remove deprecations

General

removes legacy backend system
removes deprecations

Fixes
RHIDP-4229

/cc @yangcao77


[SPARK-22912] v2 data source support in MicroBatchExecution

What changes were proposed in this pull request?
Support for v2 data sources in microbatch streaming.
How was this patch tested?
A very basic new unit test on the toy v2 implementation of rate source. Once we have a v1 source fully migrated to v2, we'll need to do more detailed compatibility testing.

/cc @zsxwing

Test build #85445 has started for PR 20097 at commit 19919de.

Test build #85445 has finished for PR 20097 at commit 19919de.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85445/
Test FAILed.

Test build #85478 has started for PR 20097 at commit e27d1db.

Test build #85478 has finished for PR 20097 at commit e27d1db.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85478/
Test FAILed.

Test build #85491 has started for PR 20097 at commit 9ffb92c.

Test build #85491 has finished for PR 20097 at commit 9ffb92c.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85491/
Test FAILed.

retest this please

Test build #85525 has started for PR 20097 at commit 9ffb92c.

Test build #85525 has finished for PR 20097 at commit 9ffb92c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85525/
Test PASSed.


Fixed #12090 -- Show admin actions on the edit pages too

Hi!
It fixes ticket 12090

New discussion thread: https://forum.djangoproject.com/t/admin-actions-also-in-the-object-details-view/22927/4

When save_on_top is True looks like:

To run tests please do: ./runtests.py admin_views.test_actions.AdminDetailActionsTest

Another point that is worth mentioning: Django Admin Inline Actions have the description attribute which can be used to give a more friendly human-readable name to the action.
e.g
@admin.action(description='Mark selected stories as published')
def make_published(modeladmin, request, queryset):
    queryset.update(status='p')

The problem this poses is that this attribute was always designed to be used as part of an action against multiple instances of a given model and thus over the years people (me..) would have used names written in the plural-form like "Regenerate contracts", or "Refresh indexes".
Therefore using those names in the context of the edit page would cause them to not make sense because it's a singular object being "actioned" rather than multiple. Others would have followed this naming pattern too, especially as the documentation example uses the wording "Mark selected stories as published" which would not work in singular form.
In an ideal world, inline actions would need two variables to do a good job of this - as Django does in many other places (verbose_name vs verbose_name_plural) etc. But this poses another problem in that the known-plural version is currently called description so it's not as simple as adding a new description_plural parameter.

@mgaligniana Do you have time to keep working on this?

@felixxm Yes! This week I'm going to add some progress! 🙏 Sorry, I've been busy these months.

Another point that is worth mentioning: Django Admin Inline Actions have the description attribute which can be used to give a more friendly human-readable name to the action.
e.g
@admin.action(description='Mark selected stories as published')
def make_published(modeladmin, request, queryset):
    queryset.update(status='p')

The problem this poses is that this attribute was always designed to be used as part of an action against multiple instances of a given model and thus over the years people (me..) would have used names written in the plural-form like "Regenerate contracts", or "Refresh indexes".
Therefore using those names in the context of the edit page would cause them to not make sense because it's a singular object being "actioned" rather than multiple. Others would have followed this naming pattern too, especially as the documentation example uses the wording "Mark selected stories as published" which would not work in singular form.
In an ideal world, inline actions would need two variables to do a good job of this - as Django does in many other places (verbose_name vs verbose_name_plural) etc. But this poses another problem in that the known-plural version is currently called description so it's not as simple as adding a new description_plural parameter.

Has been passed almost 1 year! Sorry djm, but finally I've returned to work on this! 🙏
As I defined other different variable for detail view: detail_actions. The description could be written in singular way! That solve the problem?

Hi @nessita!
I applied the description_plural paramenter and added a new test.
Two questions/doubts:

I pass down the cache parameter (to know if its a change or list view) until _get_action_description but not sure if it is the best approach. What do you think? Is there another clean way?
Following the forum discussion: The next step would be remove the Delete button?  (as it was replaced by the action inside the dropdown)

Thank you! And sorry the delay!

Hi @nessita! Just as a 🎗, no rush. Any thoughts about https://github.com/django/django/pull/16012#issuecomment-1741925611 and https://github.com/django/django/pull/16012#discussion_r1342654481 ? To move forward and add more tests cases if its necessary. Thank you!

Hi @nessita! Just as a 🎗, no rush. Any thoughts about #16012 (comment) and #16012 (comment) ? To move forward and add more tests cases if its necessary. Thank you!

Hi! This this definitely in my ToDo list, but I've been prioritizing release blockers for 5.0 and some DjangoCon PRs (to capitalize on the momentum). I should be able to review this soon! Thank you for your patience :100:

@sarahboyce Thank you Sarah!! Sorry for not answering previosly! For sure, I'm going back to work on this and update the Tarcker when is ready! 💪

This is how looks like the documentation I added:

I haven't added SeleniumTest but I added the label to check I didn't break anything. Let me know if add some would be required!

I installed my Django branch version. I recorded a happy path video:
happy_path_actions_in_detail_view.webm
I think we have things to test and define, but we are close! :crossed_fingers:


RepositorySail: wrapping third party in a Sail object?

Hi
While experimenting with SailRepository and LuceneSail for a project I'm working on, I've notice that while a SailRepository instance expose a getSail() method for obtain the underlying implementation, there seems to be no way to wrap a Repository back into a working Sail instance.
I know it could probably introduce a lot of issues in some cases, it could be anyway very useful on some specific scenario: for example I'm trying to adopt virtuoso into a LuceneSail, and then it seems that if I could write something like:
Sail sailVirtuoso = new RepositorySail(new VirtuosoRepository(...))

I could then try to log all the activities on the new sailVirtuoso object (a sort of naive transaction log) and also try to use it as a base sail for LuceneSail.
If I'm not wrong something similar was implemented in Ripple, but since as far as i know it was not updated to RDF4J, I'm trying to re-start a partial implementation for my tests.
What do you think about it?

Post questions to the mailing list. This is for requesting changes to the RDF4J project.

It is a request indeed, with attached a basic idea for an implementation.
My apologies if it looks not in the right form

Thanks for the code. However, I think this needs more discussion on the mailing list. You can post to the rdf4j-dev mailing list to get more detailed feedback.


Some files missing

Hi, I noticed that the .gitignore file excludes SUN files, which prevents the project from being compiled.
Is this intentional, or is it something that can be fixed?

Hey! The .SUN format is a custom engine format for cooked assets. I don't check these in for the same reason you don't usually check in the precompiled binaries for a project: they can be large and instead can be cooked locally.
Try building the CookerRun target in Visual Studio, as that will cook all demo and engine assets into their .SUN format equivalents.
I will add some instructions to that effect in the README, thanks for pointing it out!

Oh, it worked!
I was able to compile and run the demo.
Thank you!


Octopus - Discover kubernetes node in vSphere

Once vSphere Cloud Provider (VCP) running on master gets list of nodes, it need to Discover that node in vSphere. User can specify multiple Virtual Center details.  This can be a time consuming process.
For discovery, OS-UUID of the node should be used.

Issue is resolved in PR: https://github.com/vmware/kubernetes/pull/320


ld: warning: ignoring duplicate libraries: '-lproc'

I don't know where the error come.
when I go run main.go the message come ld: warning: ignoring duplicate libraries: '-lproc',just a warning.

For those who find this page when searching for ways to silence this obnoxious warning, I was able to get rid of it by setting this env:
export CGO_LDFLAGS="-Wl,-no_warn_duplicate_libraries"

@eaardal thank you so much!


IOS Developer

IOS Developer
Skill Set:
•	Knowledge of the Information Technology Market;
•	Knowledge of the Products and Services provided by the company;
•	Knowledge of Operating Systems, Database, Networks, Software Environment and Software Security, Designer;
•	Experience with Swift 3 or 4;
•	Experience with IOS.
CLT + Attractive salaries ( Bonus Program, Career path Plan
HR Buddy/Mentorship program, International Company, Health and Dental Plan (100% paid by the company), Education Benefits, Sport Benefits, Sponsored English classes, Sponsored Certifications
Location: Vila Olímpia -São Paulo, Brazil.
=><EMAIL_ADDRESS>

Vaga removida por violação da regra:

Todas as vagas devem conter o nome da empresa que está contratando
Se você é um recrutador, você deve identificar o nome da empresa em que o candidato irá trabalhar


Amplify status check fails with Github Merge Queue

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d2t0beklulssyh
AWS Region
us-east-1
Amplify Hosting feature
Backend builds, Build settings, Git providers
Frontend framework
React
Next.js version
N/A
Next.js router
N/A
Describe the bug
When enabling a github Merge Queue https://github.blog/2023-07-12-github-merge-queue-is-generally-available The github app that builds the pull request fails to build the app for the pr on the queue.
Expected behavior
I expect the build to succeed and the pr to automatically merge after the status is reported success.
Reproduction steps

Enable Github Merge Queue
Create a pull request

Build Settings
No response
Log output

# Put your logs below this line

Additional information
No response

This is the issue for the merge queue and the build never runs.

Hi @austinamorusoyardstick 👋 , thanks for your continued patience.
After doing some deep diving on the merge queue feature, we have decided to take this as a feature request as it would require additional feature implementation from our end to accommodate the merge-group webhook event which is required to successfully validate the status checks for the PR's in the merge-queue.

This feature would be very much appreciated: we are working on a huge monorepo with multiple open PRs, yet they’re blocked from merging as we require them to be updated to main branch prior to merging. This require sequential merges which significantly delays merge and deployment.
We also considered using merge queue but the inability of Amplify to generate preview branches from merge queue branches is blocking our efforts to do so.


Factory should warn if annotee have js.UndefOr field without js.undefined

I do not think this is valuable, since
@Factory
trait X extends js.Object {
  var y: js.UndefOr[String]
}

will generated a companion
object X {
  def apply(y: js.UndefOr[String] = js.undefined): X = ...
}

whose apply with js.undefined added.


Mourning's End Part II - Final pillar warning is saying wrong direction when you're about to leave

The capitalized text before you're about to leave says to turn the pillar to north, but you're supposed to turn the last pillar back to "east" before leaving.
I just got confused about that part, not like you can turn it anywhere else than west or east or up.. But just wanted to make sure that it's not too confusing for others because it's quite important step to get right.

Thanks, will be fixed with **https://github.com/Zoinkwiz/quest-helper/commit/a935e4e845bf59d71f11f2a1ae6f6bdd3679c45a


User feedback v.1.1.0

Hello

Thank you. I agree that you're taking giant leaps in the right direction.
I like the icons, they can be used for distinguishing between different steaks as well (mine, kids', wife's). You have gone with a UI mimicking a particular series (Forerunner)? To me the icons for functions and the on/off switch in settings are strange to look at since it does not match the UI on the Fenix series - but this is no biggie.

I'll take a look at the Fenix button layout but in the simulator it looks like a similar layout of buttons to that of the forerunners.

So my prioritized wish list is now:

Total time per steak, as well as time to next flip

Will be addressed in future version. We have an open issue for this.

Ability to save last flip time per "position", so that if the first steak last time had 1:10 per flip, and the second it the list had 1:30, the first will still show "1:10 (last time)" and the second "1:30 (last time) - since it's somewhat a pain to set a large number of different flip times each session (and they're always more or less the same for me)

This is a variant of what we have now. We need to track a "lastTimer" per steak in the steakList. Although I am not convincedt that this makes the experience any better than now.

And small stuff on top would make it better (maybe just for me)
Only show the amount of steaks active - (makes for easier overview), maybe with slightly larger font - are you using a custom font?
Make the UI in line with Fenix

Same as above. Will look into the Fenix button layout.

I think maybe he's referring to the icons, toggles, etc. We could update those to use Menu2 on newer devices where it's available.
I do like both of the other suggestions though - dynamic number of entries until the screen is full so we don't have to write any page scrolling for the list, and saving all the usage settings on exit and reloading on start. I think a lot of people probably end up with a couple of timers that they use for specific things (I know I do this) and it would be convenient for the app to save those settings.

Yes, he refers to the icons.

What I mean is the standard way of showing enabled/diabled options on a Fenix6 is like this:
https://hikingguy.com/wp-content/uploads/fenix-6-battery-saver-1536x864.jpg
This what I meant with non-standard UI. I just sort of expected that in the Dev kit there is some standard function that draws UI for different watches differently, but maybe I am mistaken?

Is there an easy way to support the Menu2? I remember we discussed this at some point but it sounded like we had to have a complex build environment.
Ok, I see what you mean with the frequently used timers. It makes sense. I do the same. There are some things I cooked regularly that are always the same.
We could hide the inactive timers, and replace the text with "New timer" always at the bottom. That way we grow until we run out of space.

Menu2 can be supported by testing for it with "has", so that one's not bad. Also, I've done a lot of research on the build stuff and it's not really that bad to get it working. However, unless there's a problem with memory usage we can implement that particular bit without dealing with the jungle file.
For integrating the bluetooth thermometer support we will have to use the jungle file, there's no way to avoid it. No point in building a bunch of extra code for devices that don't have the hardware or the CIQ version to support it.

I would put the Menu2 as a low priority.
I had a quick look at the jungle file description in the developer's page. If I understood correctly, jungle files are reusable/ libraries. This makes sense and make it cleaner.
My dongle was shipped today. Hopefully arrives this week.

I don't really mind looking at the menu thing. If you want to start looking at saving the item settings on exit and think a bit about how we fit the total time into the app, I can fix the outstanding bugs in development and see if how much time/if it's worth adding Menu2 support for CIQ 3.x devices.
When I'm done with that I'm going to take the bluetooth code I wrote and turn it into a monkey barrel (that's the reusable library) and add it to the project in low-n-slow. We will need to update the jungle file then (that's the "build script"). By then you should have your dongle and we'll be ready to start making some headway on the temperature and smoking stuff.
MonkeyBarrel (barrel): reusable library, like stuff you get from pip for python or .so in unix, .dll in windows, etc. except used as a code library, not a pre-built binary
Jungle file: kind of equivalent to a makefile, ant or gradle script. controls what files/resources/annotations get compile on a per-device/family/language level

Ok. I like the plan.

I'll bring the total time discussion to its issue.
I'll implement the saving of item settings per steak on exit.
I'll fix the activity saving bug when no timer has been initiated.

Closing, well identified improvements and new functions. Tracking in independent tickets.


Add cookbook entry for updating nested fields

We have an entry for filtering based on nested fields, but no entry for updating nested fields. Someone just asked this on IRC.

+1. This is a very common question that was asked again today.

The user was trying to do something like this:
r.table('data')('items').filter(lambda item:
  item['tags'].contains('foo')
).delete()

The solution turned out to be something like this:
r.table('data')
  .update(lambda row:
    {'items':
      row['items'].filter(lambda item:
        item['tags'].contains('foo').not())})

Linking to https://github.com/rethinkdb/rethinkdb/issues/2692 for posterity. (Once 1.16 is out we should see if we can schedule this to happen sooner rather than later)


Overridden Logo Height

Current behavior

Height of logo in a-logo is forcefully overridden
Expected behavior

SFUI default height works perfectly
Steps to reproduce the issue

Can you handle fixing this bug by yourself?

[ ] YES
[ ] NO

Environment details

Browser:
OS:
Code Version:

Additional information

Closing the PR, as this variable is needed in /error page


Useless!!

Useless if you can't publish to power bi service!! I need to use many to many relationships and I can only use it on Power Bi Desktop. Please don't release something if it's not going to work properly!! I was so excited about this new feature and I'm already using M:M but now I have to change everything!! So disappointed!!

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

ID: 477d54a5-8a27-ae63-9887-9801639962e8
Version Independent ID: 8a2c56c0-cf25-bd2d-8de9-207ccc1e83d1
Content: Many-to-many relationships in Power BI Desktop (Preview) - Power BI
Content Source: powerbi-docs/desktop-many-to-many-relationships.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi @JuanPerezHdz - thanks for your comment. Did you see that this is a Preview feature? That means it's not a generally available (GA) feature (yet), and is out there for feedback and to test with users. Limitations such as being unable to publish to the Power BI service may be removed when it reaches GA.

As my data model is cluttered with M2M relationships, I can't wait for the GA to be able to publish in the Power BI Service. Any planned date for a GA?


[Snyk] Upgrade @types/node from 20.4.4 to 20.4.6

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @types/node from 20.4.4 to 20.4.6.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 22 days ago, on 2023-08-02.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

#291 291


Have an online version of the tool for visitors

See https://github.com/adyen-examples/checkoutCreate/pull/65

Closing as this is now available and working


Render in HTML  with monospaced font

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.202
Where did you get the app from?
F-Droid
Android version
12
Device model
No response
Steps to reproduce
I (send and) receive an email containing:
--example.org=_3175999=_1666532806=_1948217805_A
Content-Type: text/html; charset=utf-8
Content-Disposition: inline

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<title></title>
You have received an updated invitation from <a<EMAIL_ADDRESS>B C

Summary	Hokus Pokus

Location	somewhereL2

Start	<tt>Thu, 27 Oct 2022 07:00 UTC

                 10:00 Europe/Sofia </tt>

Attendees	• <a<EMAIL_ADDRESS>
• <a href="mailto:1<EMAIL_ADDRESS>
Description	Long text, no meaning.

Expected behavior
I expect that the text bewenn <tt> and </tt> is rendered with monospaced font.
Actual behavior
No monospaced font, the lower line is aligned to the upper line
Logs
No response

Our HTML sanitizer removes all tags that are not explicitly allowed. <tt> is deprecated and hence wasn't on the allow list. I'll add it. But you might want to let the sender know that they should switch to alternatives. You could point them to https://developer.mozilla.org/en-US/docs/Web/HTML/Element/tt

When I replace <tt> with <samp> the text is also not monospaced and <samp> is not deprecated.  Looking at the change,
 and  seem also to be stripped.
Moreover, in
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: 8bit

ZZZ

the foreground and background colors are not displayed by K9 mail.

When I replace <tt> with <samp> the text is also not monospaced and <samp> is not deprecated.

<samp> was indeed not on the allow list. I'm guessing mainly because it is not very commonly used. Same goes for  and <var>. I'll add them to the list.

Looking at the change,
 and  seem also to be stripped.

Those two are not stripped. If you used "ZZZ" for your tests, it's hard to see the difference between regular and monospaced text, though.
To find out what the sanitized HTML looks like you could clone the repository, make sure you have a JDK installed, then run
./html-cleaner input.html in the root of the repository.

Moreover, in
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: 8bit

ZZZ

the foreground and background colors are not displayed by K9 mail.

Works fine for me. The values are overridden when using the dark theme, though. See #4198.

style="color:#000;background-color:#ffd700" is indeed honoured, when using light theme.
In my case the user can select any colour as background and the software sets black or white for the foreground text, depending on which of black/white has higher contrast to the background.
Since it cannot be assumed that emails are read/composed on white background, the background-color should either be always ignored, or always considered.  K9 shall assume that the sender knows what it is doing by setting style="color:#000;background-color:#ffd700" (as in my case).


Replace rest bed library with non-GPL equivalent

Replace rest bed library with non-GPL equivalent

@kirichoi How difficult would it be to integrate libcurl into Windows build?  Binaries are here https://curl.haxx.se/download.html.  I was successful using it to access SBOLValidator API, and it is distributed under MIT license

libcurl is already part of Windows static lib as a dependency of raptor2.

Converted to libCurl.


[MAS1.3.1][Screen Reader-Connect the bot] Incorrect position of the controls are announced by voiceover for the buttons available at the top most menu bar.

Solves xxx
Description
This pull request fixes how voiceover announces the position of the controls for the buttons at the editor menu bar.
Changes made
We added an aria-hidden attribute to the separator bar between the restart conversation button and the sub-menu button. This way voiceover skips this element which was causing the problem.
Additionally, we noticed that in windows, the last element of the menu bar was not being announced as 3/3. To solve this we removed a div around the save transcript button and moved the role label menuitem inside it.
Testing
In the next image, you can see the voiceover reading the order of the elements as intended.

Remember to put the Solves # NUMBER (with the space in-between) as we spoke last week. Also, I'd change

This pull request fixes how voiceover announces the position of the controls for the buttons at the editor menu bar.

To something like

Fixes how voiceover announces the position of the editor menu bar's buttons' controls.

Merged MS PR 1935