fix: selectedOption not working properly. unable to…

…  empty the selectedoption useState while giving input to the search onscroll unnecessary data loading because of that wrong data mapin. i have fix this search issues
#1585
Related Issue

Description
In search input while searching for book keyword  we get 3 book related results  while on scroll this result get changes we are getting unnecessary result because of that our result get changes and unnecessary api call.
in code if you see we are unable to empty the  selectedOptions because of that some tag or search keyword triggering. I have fix this issue please check and merge my pr.
Screenshots

@biplavmz avoid creating PR first

Thanks @priyankarpal for merged my Pr


textMask still have problem on ion-input

creating a mask for price with angular2-text-mask v4.0.0 and text-mask-addons v3.0.0 but still have this error on ion-input ionic v2 final and mask won't work
Cannot read property 'selectionStart' of undefined
Someone fix it pls

+1

Confirmed, the recent fix done by adding ngAfterViewInit:
  ngAfterViewInit() {
    if (!this.textMaskInputElement) {
      // the element was not found when ngOnInit ran, let's try to find it again
      this.setupMask()
    }
  }

should be written instead as:
  ngAfterViewInit() {
    if (!this.inputElement) {
      // the element was not found when ngOnInit ran, let's try to find it again
      this.setupMask()
    }
  }

I tried and tested it on my ionic rc5 project by changing the corresponding code on the dist js file:
    MaskedInputDirective.prototype.ngAfterViewInit = function () {
        if (!this.inputElement) {
            // the element was not found when ngOnInit ran, let's try to find it again
            this.setupMask();
        }
    };

@PerfectPixel Shall I make a PR on this?

@marcocarosi yes please do submit a PR. Thanks for taking the time to figure out a fix.

@ihadeed created a branch, committed locally but I guess I don't have access to the repository to created a PR. I guess you would need to give me access. But I have no issue if you create the same PR on my behalf.

@marcocarosi you can fork the repo and submit a PR.

Requested #347

Just published the changes in #347 as<EMAIL_ADDRESS>which should hopefully fix this.


Allocating memory inside prange fails

Knn workload in dpbench fails since it allocates memory inside prange as shown below. To successfully executee knn, numba-dpex needs to add support for allocating memory inside prange loops.
    for i in nb.prange(test_size):
        queue_neighbors = np.empty(shape=(k, 2))
         ...

How to reproduce:
Follow instructions to setup dpbench
Run knn - python -c "import dpbench; dpbench.run_benchmark("knn")

@adarshyoga Can the allocation (https://github.com/IntelPython/dpbench/blob/64651cc2c364a3a004e5b38caa447a87938ed3bb/dpbench/benchmarks/knn/knn_numba_dpex_p.py#L25) be moved out of the prange?
I am in the process of designing the interface for our prange feature and starting a discussion on what a prange means for numb-dpex. In general, we have to clearly define the scope of what is supported and what is not supported. The goal also is to make it possible to use loop analysis and optimization by limiting the expressiveness of a prange.
Right now a prange is completely lowered as a kernel and there is no easy way to allow user functions (such as numpy.empty or dpnp.epmty) in a kernel and be able to deterministically optimize it way.

It is possible to move the allocation out for knn. It will require some changes to the prange loop's body to adjust indexing, which should not be difficult.
But the main implication could be the size of the allocation that can be made outside the prange. We are allocating queue_neighbors which is of size k*2 for each work item. If it is moved out, we will be making a total allocation of size test_size*k*2. If test_size is large this allocation might fail. We will need to test it out.

We are allocating queue_neighbors which is of size k*2 for each work item.

Is not the queue_neighbors overwritten inside the prange for every loop iteration? Would be able to allocate it once outside and reuse every iteration?
queue_neighbors = np.empty(shape=(k, 2))

for j in range(k):
x1 = x_train[j]
x2 = x_test[i]

distance = 0.0
for jj in range(data_dim):
    diff = x1[jj] - x2[jj]
    distance += diff * diff
dist = math.sqrt(distance)

queue_neighbors[j, 0] = dist
queue_neighbors[j, 1] = y_train[j]

Every iteration uses a private local copy of queue_neighbors. Just moving the allocation outside the prange will make it shared across all work-items and result in a race condition.


Add error fields for cluster and refactor machine status

add validation of cluster spec
add errorReason and errorMessage to cluster Status
refactor machine status

/cc kashifest
/cc xenwar

@xenwar


java.lang.NoSuchFieldError: No field search_view_shadow

Hi,
I'm trying to run super simple example, and App is crashing with following exception:
java.lang.NoSuchFieldError: No field search_view_shadow of type I in class Lcom/lapism/searchview/R$id; or its superclasses (declaration of 'com.lapism.searchview.R$id' appears in /data/app/com.agilityscales.app.dev-2/split_lib_slice_2_apk.apk)
                                         at com.lapism.searchview.SearchView.initView(SearchView.java:884)

com.lapism.searchview.R is getting generated, but search_view_shadow not inside it.
Maybe I'm missing something? Is there any working sample code?
Thanks in advance )

Facing the same issue.
here is the error log -
http://paste.ubuntu.com/25911280/
gradle -> http://paste.ubuntu.com/25911285/

Nevermind, I had another search view library.. after removing that this library works fine..Maybe some conflicts could have occurred


use LOG_ABORT not just abort()

abort() has the unfortunate effect that nothing is seen in the log,
just an event (which is usually not displayed); so ops people don't
see that the program is crashing at all.
LOG_ABORT("message") will log an error with the message (and the file
and line) before calling abort(), so it's easy to see what happened.
add or move <vespa/log/log.h> include and LOG_SETUP lines before
LOG_ABORT is used (or included).

@baldersheim please review and merge
@toregge @havardpe @geirst @vekterli  FYI

added a special HDR_ABORT to avoid needing LOG_SETUP in/before header files.
also done some more cleanup.
@baldersheim please take another look now


Inconsistent results with the official website

I am using the sequence to be run on http://opig.stats.ox.ac.uk/webapps/newsabdab/sabpred/anarci/ QVQLVESGGGVVQPGRSLRLSCAASGFTFSVYGMNWVRQAPGKGLEWVAIIWYDGDNQYYADSVKGRFTISRDNSKNTLYLQMNGLRAEDTAVYYCARDLRTGPFDYWGQGTLVTVSSASTKGPSVFPLAPSSKSTSGGTAALGCLVKDYFPEPVTVSWNSGALTSGVHTFPAVLQSSGLEIVLTQSPDFQSVTPKEKVTITCRASQSIGSSLHWYQQKPDQSPKLLIKYASQSFSGVPSRFSGSGSGTDFTLTINSLEAEDAAAYYCHQSSSLPFTFGPGTKVDIKRTVAAPSVFIFPPSDEQLKSGTASVVCLLNNFYPREAKVQWKVDNALQSGNSQESVTEQDSKDSTYSLSSTLTLSKADYEKHKVYACEVTHQGLSSPVTKSFNRGEC
The result was given as two alignments, one is human_H, one is human_K.

But when I debugged locally on my laptop(mac OS), alignments are rat_H and human_K. Only when I set allowed_species = ["human"], it worked, and I don't think it would be the correct way.

Can you guide me how to run the program to output the same results as the website do?

@sarah-robinson @oxpig/website

@broncio123 @sarah-robinson @oxpig/website Hi, guys， I also found the same problem, such as sequence： EVQLVESGGGLVQPGRSLRLSCAASGFTFNDYAMHWVRQAPGKGLEWVSTISWNSGSIGYADSVKGRFTISRDNAKKSLYLQMNSLRAEDTALYYCAKDIQYGNYYYGMDVWGQGTTVTVSSASTKGPSVFPLAPGSSKSTSGTAALGCLVKDYFPEPVTVSWNSGALTSGVHTFPAVLQSSGLYSLSSVVTVPSSSLGTQTYICNVNHKPSNTKVDKKVEP
Why are the local and web results inconsistent ，hope to get a quick reply, thanks!

The web server currently runs the older python 2 version of ANARCI, and most likely is using slightly different germlines to peoples' local installations, which may explain the differences. @sarah-robinson can you take a look and see if there's anything else that might be implicated here?

Thanks for raising this. I agree with Fergus - I'll look into the germlines that the web server is using, to make sure that this is what's causing the discrepancy.

Hey guys
Any findings?
Many thanks.

We are still investigating - whilst there are germline differences between the webserver and the ANARCI depository on GitHub, I think the main difference is the web server reports back the V gene and J gene assignments (human) and not the species HMM with the highest score (alpaca), whereas when running locally the immediate output is the species HMM. We are looking into why the alpaca HMM is scoring so highly for these sequences.

Hey guys,
I digged into the source code. I understood that one thing, the ALL.hmm was built from ALL.stockholm by command hmmbuild --hand $DIR/HMMs/ALL.hmm $DIR/curated_alignments/ALL.stockholm in RUN_pipeline.sh. ALL.stockholm was downloaded from http://www.imgt.org/IMGT_GENE-DB and rebuilt using IMGT.
Finally, I compared the ALL.stockholm after I ran 'python setup.py install',  a lot of rows were shown differently. Which means different time to build ALL.hmm would cause ANARCI behaves differently.

If my guess is correct, one best way is to provide the same HMM that the server using.
Many thanks.

Thanks for the info - this matches what we've found in testing, and indeed we plan on synching the HMMs between different versions of ANARCI to make things more consistent.

Hi Guys,
Any findings?
Best regards,
Owen


Sort by Surname

How to Sort by Surname in alpha order in a table column that contains for example:
Dr Fred Smith
Mr Joe Day
Mary Allen

so default load would be
    Mary Allen
    Mr Joe Day
    Dr Fred Smith

thank you

If you have a way to display the name without the title, you could use the data-sort helper, as such: https://jsfiddle.net/ns3zxd93/
If you don't have an easy way to display the names without the titles, I think you are going to need to play with JavaScript quite a lot.

Or just use the surname in the data-sort attribute?

Or just use the surname in the data-sort attribute?

I think I lost my thought when making the example. But yeah @coldes, do it as @tristen said.

...thank you, yes by adding surname to data-sort has worked

Mr Joe Day

Mary Allen

Dr Fred Smith


⚠ ClamAV has degraded performance

In 7e01a86, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 1d59bf5.


PP 1494 ePDQ auth support

WHAT
A brief description of the pull request:

Built out the class hierarchy for ePDQ authorisation responses, mapping all supported status codes to the following internal statuses:

Status 5 to AUTHORISATION SUCCESS
Status 50, 51 to AUTHORISATION SUBMITTED
Status 2 to AUTHORISATION REJECTED
Any other status to AUTHORISATION ERROR

Introduced a new transient waiting state AUTHORISATION_SUBMITTED, which we should not see many occurrences of as authorisation are almost never deferred in real life. The rare instances of non-immediate authorisation (ePDQ status 50 and 51) are treated as errors, hence AUTHORISATION_SUBMITTED is mapped to EXTERNAL_ERROR_GATEWAY.
The state engine has been updated to handle the new AUTHORISATION_SUBMITTED status that was introduced to cater for situations when ePDQ defers authorisation which we then treat as an error. The precondition for a charge to transition to AUTHORISATION_SUBMITTED is to be in a state of AUTHORISATION_READY although it is not mandatory for a charge to transition through AUTHORISATION_SUBMITTED in order to proceed to AUTHORISATION_SUCCESS, AUTHORISATION_REJECTED, AUTHORISATION_CANCELLED or AUTHORISATION_ERROR. In other words AUTHORISATION_SUBMITTED is optional and transient.
Transaction that are deferred during authorisation return a 400 bad response with an error message: This transaction was deferred.
The initial implementation of EpdqPaymentProvider offers support for authorisation operation only at this stage. When building the authorisation request, the ORDER ID submitted to the payment provider in this implementation is going to be the external charge id for now until we are provided with more insight by HMCTS as to what sort of reference would be preferable instead in order to aid reconciliation.
Credentials in GatewayAccount has been extended with a SHA passphrase which is required for any request payload signature.
The configuration has a new section under epdq for urls and jerseyClientOverrides settings. Note that the Jersey Client settings are provisional and subject to change once we have collected real metrics and in a position to set sensible default values.
New utility class TestTemplateResourceLoader used in tests to facilitate loading all request/response payloads that are made available as resources under src/test/resources/templates.
Added argument check ensuring the passphrase is not blank for EPDQ message payment signature.

There were errors, for info, please see...

There were errors, for info, please see...

There were no errors, go have a cup of coffee...


update to new compilers /py37

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@mingwandroid @isuruf @jakirkham any ideas how how to solve OSX:
  No known features for CXX compiler

  "Clang"

  version 4.0.1

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jakirkham thanks for the hint, but still the same problem.
CMake Error in CMakeLists.txt:
  No known features for CXX compiler

  "Clang"

  version 4.0.1.

It seems another CMakeLists.txt may be generating the error. Netgen perhaps?

hmm, there is no CMakeLists.txt in the netgen osx-package. Maybe another CMakeLists.txt in a subdirectory?

Right, that's what I was meaning.

After some local iterations it seems like the mentioned problem is correlated to linking to vtk:
target_link_libraries(SMESHControls ${VTK_LIBRARIES})
Where ${VTK_LIBRARIES} looks like this:
vtksysvtkCommonCorevtkCommonMathvtkCommonMiscvtkCommonSystemvtkCommonTransformsvtkCommonDataModelvtkCommonColorvtkCommonExecutionModelvtkCommonComputationalGeometryvtkFiltersCorevtkFiltersGeneralvtkImagingCorevtkImagingFouriervtkalglibvtkFiltersStatisticsvtkFiltersExtractionvtkInfovisCorevtkFiltersGeometryvtkFiltersSourcesvtkRenderingCore$PREFIX/lib/libz.dylib$PREFIX/lib/libfreetype.dylibvtkRenderingFreeTypevtkRenderingContext2DvtkChartsCorevtkDICOMParser$PREFIX/lib/liblz4.dylibvtkIOCorevtkIOLegacy$PREFIX/lib/libexpat.dylibvtkIOXMLParservtkDomainsChemistryvtkglewvtkRenderingOpenGL2vtkDomainsChemistryOpenGL2vtkIOXMLvtkParallelCorevtkFiltersAMRvtkFiltersFlowPathsvtkFiltersGenericvtkImagingSourcesvtkFiltersHybridvtkFiltersHyperTreevtkImagingGeneralvtkFiltersImagingvtkFiltersModelingvtkFiltersParallelvtkFiltersParallelImagingvtkFiltersPointsvtkFiltersProgrammable$PREFIX/lib/libpython3.7m.dylibvtkWrappingToolsvtkWrappingPythonCorevtkFiltersPythonvtkFiltersSMPvtkFiltersSelectionvtkFiltersTexturevtkFiltersTopologyverdictvtkFiltersVerdictvtkmetaio$PREFIX/lib/libjpeg.dylib$PREFIX/lib/libpng.dylib$PREFIX/lib/libtiff.dylibvtkIOImagevtkImagingHybridvtkInfovisLayoutvtkInteractionStylevtkImagingColorvtkRenderingAnnotationvtkRenderingVolumevtkInteractionWidgetsvtkViewsCorevtkproj4vtkGeovisCore$PREFIX/lib/libhdf5.dylib/usr/lib/libpthread.dylib/usr/lib/libdl.dylib/usr/lib/libm.dylib$PREFIX/lib/libhdf5_hl.dylibvtkIOAMRvtkIOEnSight$PREFIX/lib/libnetcdf.dylibvtkexoIIcvtkIOExodusvtkgl2psvtkRenderingGL2PSOpenGL2vtklibharuvtkIOExportvtkIOExportOpenGL2vtkIOGeometryvtkIOImport$PREFIX/lib/libxml2.dylibvtkIOInfovisvtkIOLSDynavtkIOMINC$PREFIX/lib/libtheoraenc.dylib$PREFIX/lib/libtheoradec.dylib$PREFIX/lib/libogg.dylibvtkIOMovievtknetcdfcppvtkIONetCDFvtkIOPLY$PREFIX/lib/libjsoncpp.dylibvtkIOParallelvtkIOParallelXMLvtksqlitevtkIOSQLvtkIOTecplotTablevtkIOVideovtkImagingMathvtkImagingMorphologicalvtkImagingStatisticsvtkImagingStencilvtkInteractionImagevtkPythonInterpretervtkRenderingContextOpenGL2vtkRenderingImagevtkRenderingLODvtkRenderingLabelvtkRenderingMatplotlibvtkRenderingVolumeOpenGL2vtkViewsContext2DvtkViewsInfovis

But I still have no idea what this is about.

To me it seems there are two bugs involved:

cmake is reporting nonsense instead of library  can not be found
This can be reproduced by adding this command:
target_link_library(<lib> libReallyNonExisting)

vtkLibraries is missing the version suffix:
target_link_library(<lib> vtkCommonCore) # not working
target_link_library(<lib> vtkCommonCore-8.1.2) # works

@mingwandroid is it possible this is related to the new-compilers? I have the impression that it is not possible to link to symlinks:
target_link_library(<lib> vtkCommonCore) doesn't work
target_link_library(<lib> vtkCommonCore.1) works
with libvtkCmmonCore.dylib beeing a link to libvtkCommonCore.1.dylib
maybe you have an idea what is going on here?

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [15]

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

is it possible this is related to the new-compilers

I do not have time to look into that, anything is possible. You need to figure out a way to determine this for yourself though.

I do not have time to look into that, anything is possible. You need to figure out a way to determine this for yourself though.

Yes it's a problem with cmake not detecting the compiler-features. But finding the root of the problem is absolutely beyond my knowledge/skills. I will work around by simple setting those features with:
SET(CMAKE_CXX_COMPILE_FEATURES "cxx_nullptr;cxx_override")

Hopefully it's possible to add proper compiler-feature detection with cmake in the future. Also I have no idea where to report if conda-build is the wrong place.

next problem: zlib in netgen-feedstock is used from the build-environment 👎

Hopefully it's possible to add proper compiler-feature detection with cmake in the future. Also I have no idea where to report if conda-build is the wrong place

Since you are running CMake here, Occams Razor would suggest this is an issue with CMake. But for you to prove that you need to demonstrate that the problem happens outside of the Anaconda Distribution (so official CMake, official Apple compilers). If it doesn't reproduce there then it's either a problem with our CMake packages, our compiler packages or an interaction between the two. Either way, it's got nothing to do with conda-build!

But for you to prove that you need to demonstrate that the problem happens outside of the Anaconda Distribution (so official CMake, official Apple compilers).

I guess it will work with official compilers. Searching for this error gave some reports of problems with feature-detection with cmake-minimum-requirement < 3.3. But with setting the cmake-policy CMP0025 to  NEW, this was resolved. So my guess is that this problem is specific to conda. If it is caused by cmake or the compilers, I don't know.

Please stop guessing. Investigate. You will learn that way.

Please stop guessing. Investigate. You will learn that way.

Here is the minimal example. No problems with official compilers. Feel free to try it yourself:
https://github.com/looooo/cmake_tests
The C compiler identification is AppleClang <IP_ADDRESS>01145
The CXX compiler identification is AppleClang <IP_ADDRESS>01145
Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
Detecting C compiler ABI info
Detecting C compiler ABI info - done
Detecting C compile features
Detecting C compile features - done
Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -- works
Detecting CXX compiler ABI info
Detecting CXX compiler ABI info - done
Detecting CXX compile features
Detecting CXX compile features - done
Configuring done
Generating done

But problems with conda:
-- The C compiler identification is Clang 4.0.1
-- The CXX compiler identification is Clang 4.0.1
-- Check for working C compiler: $BUILD_PREFIX/bin/x86_64-apple-darwin13.4.0-clang
-- Check for working C compiler: $BUILD_PREFIX/bin/x86_64-apple-darwin13.4.0-clang -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: $BUILD_PREFIX/bin/x86_64-apple-darwin13.4.0-clang++
-- Check for working CXX compiler: $BUILD_PREFIX/bin/x86_64-apple-darwin13.4.0-clang++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
CMake Error in CMakeLists.txt:
  No known features for CXX compiler

  "Clang"

  version 4.0.1.

Thanks for doing that. I'm in the process of making Cmake work better within the ecosystem BTW.

@looooo, do you have the CMakeError.log and CMakeOutput.log for the small project?

Related to https://gitlab.kitware.com/cmake/cmake/issues/17712
Need the cmake_minimum_required before the project command. Try swapping the first 2 lines of your example project.

From, https://cmake.org/cmake/help/v3.10/command/cmake_minimum_required.html

Note

Call the cmake_minimum_required() command at the beginning of the top-level CMakeLists.txt file even before calling the project() command. It is important to establish version and policy settings before invoking other commands whose behavior they may affect.

@isuruf Thanks for the information. Great, this really solves this issue!


git setting core.excludesFile is ignored if placed inside an include.path of ~/.gitconfig

I have multiple per-machine gitconfig files with sections like this:
[include]
           path = ~/.local/lib/private-scripts/common.gitconfig

and inside common.gitconfig I have this section:
[core]
           excludesFile = ~/.gitignore

This core.excludesFile setting is ignored by vscode. The setting does take effect if I copy the "core.excludesFile" section into ~/.gitconfig itself. All of this only happens with search.useGlobalIgnoreFiles=true.
The setting is also ignored in git config --global core.excludesFile but does appear in git config --include --global core.excludesFile and takes effect for commands like git status.
I think vscode should always process includes when handling user settings, reading settings with the equivalent of git config --include --global.

This is related to https://github.com/microsoft/vscode/issues/59364

It seems that this might be from ripgrep itself because it doesn't handle gitconfig includes.


esp_event_loop_deinit() should declare in esp_event_legacy.h file. (IDFGH-2367)

I am getting this error when using esp_event_loop_deinit() function. Once I declare in esp_event_legacy.h, the error get resolved. Technically it should be declare in esp_event_legacy.h because the definition of this function is in event_loop_legacy.c

 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.

@ankur2854 Thanks for the contribution.

Apparently this got replaced by https://github.com/espressif/esp-idf/pull/4525, closing.
@ankur2854 FYI, you don't need to create a new PR after changing the code. Just push the new commit to the same branch, the PR will be updated automatically.

Ah, actually https://github.com/espressif/esp-idf/pull/4525 is submitted against the release/v4.0 branch, so will keep this one open.
There is one minor request that still needs to be addressed. If you don't have time, let me know and we will add the deprecation attribute when pulling this change.

 All committers have signed the CLA.

Closing this PR as the esp_event_loop_legacy has been removed.


After running spectrum-scale-driver-snap.sh ,in output ibm-spectrum-scale-csi-describe-all-by-label file having some incorrect information

Describe the bug
ran spectrum-scale-driver-snap.sh , in output file ibm-spectrum-scale-csi-describe-all-by-label got following output at the end
Error from server (NotFound): clusterroles.rbac.authorization.k8s.io "external-provisioner-runner" not found
Error from server (NotFound): clusterrolebindings.rbac.authorization.k8s.io "csi-provisioner-role" not found
Error from server (NotFound): clusterroles.rbac.authorization.k8s.io "external-attacher-runner" not found
Error from server (NotFound): clusterrolebindings.rbac.authorization.k8s.io "csi-provisioner-role" not found
Error from server (NotFound): clusterroles.rbac.authorization.k8s.io "csi-nodeplugin" not found
Error from server (NotFound): clusterrolebindings.rbac.authorization.k8s.io "csi-nodeplugin" not found

To Reproduce
Steps to reproduce the behavior:

Run spectrum-scale-driver-snap.sh file
In output directory , open ibm-spectrum-scale-csi-describe-all-by-label file and look at the end

Expected behavior
Server should not give error,  clusterrolebinding names maybe incorrect

@vrushch Could you please verify the same with latest snap script ?

Tried with latest snap script , Above problem seems fixed ( above error messages are not there).
closing the issue


Add information about ports to odo create output

This PR adds message of the form:
"Component $componentName was created and ports $containerPort/$containerProtocol,.... were opened."
fixes #608
Signed-off-by: anmolbabu<EMAIL_ADDRESS>

Will change fmt.Print statements to glog statements once https://github.com/redhat-developer/odo/pull/590 is in

Will change fmt.Print statements to glog statements once #590 is in

user messages like this should remain fmt.Print

odo-bug-id: redhat-developer/odo#608

Please use "fixes #608" GitHub will automatically close that issue when we merge this PR. Otherwise, it is easy to forget to close the issue.

@kadel Done incorporated your comments

@kadel Incorporated comments
Please review
!! Verified !!

@cdrage @surajnarwade @syamgk @mik-dass Please review

@kadel @cdrage @surajnarwade @syamgk @mik-dass Incorporated your comments. Please review

@surajnarwade Incoporated your comments Please revisit the PR
@kadel @cdrage @surajnarwade @syamgk  @mik-dass Please review

Verified as in : https://gist.github.com/anmolbabu/7a908af61cb46e633471eebe370a37db

@cdrage @surajnarwade please check if your comments were addressed so we can merge it

@cdrage Done.Can you push this in if looks good?

Tests were a false-positive. I'm re-running them to see if it passes and then we'll merge!

Travis looks red, I am retriggering the build. if it get passed, I'll merge it :)


[dotnet] installed nugets are not found in build

Description
I have an action that pulls a dotnet core 3.1 solution from a GitHub repo and then restores and builds it.  Turning on "normal" logging, I can see the nuget packages from my private GitHub Packages feed installing, but the build fails with errors that would indicate the packages aren't found or the DLLs are missing.
I need to be careful about pasting as the projects and Nuget include my employers name, so please bear with me.
The build error is:
 Using shared compilation with compiler from directory: /home/runner/.dotnet/sdk/3.1.414/Roslyn/bincore
     2>Constants.cs(1,18): error CS0234: The type or namespace name 'Entities' does not exist in the namespace 'Employer.Cs' (are you missing an assembly reference?) [/home/runner/work/mercury-middleware/mercury-middleware/src/common/Employer.Cs.Mercury/Employer.Cs.Mercury.csproj]

This reference is to Employer.Entities, which comes from the Nuget Employer.Cs.Core
Installation of packages:
[...]
OK https://nuget.pkg.github.com/<org>/download/employer.cs.core/index.json 1562ms
[...]
GET https://nuget.pkg.github.com/<org>/download/employer.cs.core/21.24.0/employer.cs.core.21.24.0.nupkg
[...]
OK https://nuget.pkg.github.com/<org>/download/employer.cs.core/21.24.0/employer.cs.core.21.24.0.nupkg 954ms
[...]
Installed employer.Cs.Core 21.24.0 from https://nuget.pkg.github.com/<org>/index.json with content hash 276giT0IuhtgSsOZAQtVMaWwD1mFC3CtkfX5AwopsFZ/XNCDdzTe0lcYgJt2S+eiBC5zNUwOfe3Bap/MzPEUbg==.

Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Environment: ubuntu-20.04
Version: 20211011.1
Included Software: https://github.com/actions/virtual-environments/blob/ubuntu20/20211011.1/images/linux/Ubuntu2004-README.md
Image Release: https://github.com/actions/virtual-environments/releases/tag/ubuntu20%2F20211011.1
Is it regression?
No response
Expected behavior
The logs indicate the nuget package was found and installed.  Expected behavior is the solution should build.  The solution builds fine locally even when nuget caches are cleared.
Actual behavior
Build fails.
Repro steps
Not sure how to repo this.

@mmulhearn , Could you check?
- uses: actions/checkout@v2
- uses: actions/setup-dotnet@v1
  with:
    dotnet-version: '3.1'
    source-url: https://nuget.pkg.github.com/<org>/index.json
  env:
    NUGET_AUTH_TOKEN: ${{secrets.GHP_READ}}
- name: restore
      run: dotnet restore
- name: build
      run: dotnet build --configuration Release --no-restore

@mmulhearn i suspect we have a differences between<EMAIL_ADDRESS>and DotNetCoreCLI@2 namely something related to these:
https://docs.microsoft.com/en-us/nuget/consume-packages/migrate-packages-config-to-package-reference
https://docs.microsoft.com/en-us/nuget/consume-packages/package-references-in-project-files
While i am trying to reproduce the problem can you please send nuget.config,  packages.config and *.cproj files?
OR, even better, try to create hello world project in the same way as the original project was created with the more or less same dependencies - this i that i am trying to do by myself.


[OPPRO-203]Add a scala example for TPC-H testing

What changes were proposed in this pull request?
Add a scala code as an example to run TPC-H testing.
How was this patch tested?
N/A for README and example code change.

If no other update, please help to approve the PR and I can merge it.


Document settingsmeta.yaml

Per this comment from @forslund, documenting yaml as the preferred format for the settingsmeta file. If this is approved, I'll go ahead and update mycroft-skills-kit as well.

Thanks @ChristopherRogers1991. looks good! Merging


no_vnc_heartbeat param to enable ws ping/pong

This PR fixes #23.

Hello @patlachance
I'll merge your PR and my changes from the next_release branch after I'll get your answer about the testing results (see the issue #23).
I really appreciate that you've thought even about the readme file. Thanks.
Regards
accetto


Fix resolving Doctrine Target Entities to known Classes only

Hi!
I forgot these changes in my last PR (#111). Apologies for that.
In order to resolve only to known classes, we need to replace all calls to $classAnnotationMatcher->resolveTagFullyQualifiedName() to $classAnnotationMatcher->resolveTagToKnownFullyQualifiedName().

Thank you both 👍


refactor: extract client Chat API to seperate file

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
in current design, all  client use API in their components, like ChatRoot.tsx:
  async function createConversation() {
    const response = await fetch("/api/chatgpt/conversation", {
      method: "POST",
      body: JSON.stringify({
        action: "create_conversation",
        name: "Default name",
      } as RequestCreateConversation),
    });
   ...
  }

I'll take it


[ci] Remove repo tooling

Removes the repo tooling, and updates to use the published version rather than the local version. This is the other half of https://github.com/flutter/packages/pull/3198, migrating these tools to flutter/packages.
Part of https://github.com/flutter/flutter/issues/113764

This will need updates in flutter/flutter and probably the Dart repo.

The Dart script uses tool_runner.sh, so should be okay. Flutter is an issue though.


Keyboard workflow enhancements

Fixes https://github.com/ploi-deploy/roadmap/issues/89

Create new item command added
Autofocus on first input in the create item modal

Bonus

Fixed typo

🙏✅


Update main challenge reference input files with correct ionic strength

In this PR:

Updated the files for the main SAMPL challenge reference calculations (the files for the SAMPLing challenge are untouched). The protocol is exactly the same as the old one. The only difference is the number of ions in the solvation box.
Fixed a typo and added a short description of the buffers composition in host_guest_description.md.

@jmichel80, @Steboss a heads up that the buffer used for the octa acids experiments was made by mixing Na3PO4 and Na2HPO4 so the experimental ionic strength should be about 41mM, according to my calculations. If I remember correctly, you mentioned you were using an ionic strength of 60mM in a previous PR.

Thanks, @andrrizzi .
@mrshirts - these should be final files.


Design logo and T-shirt

If conversion to vector graphics is needed, ask Nico

Andreas asked that we could incorporate the SumUp logo into our logo.

Here you can find the table to put on your size for T-Shirts:
https://docs.google.com/spreadsheets/d/1nkOh5xcn1sA9c-8aCx9MyUHdG88xRGl_kjxZt3iZisM/edit?usp=sharing

Hi Berkay @rbbozkurt ,
I think we do not have write access yet.

Besides our t-shirt size table, according to homework 1, every individual also need to submit t-shirt preferences to the survey.

According to responses chart below, the most picked logo is "Logo 3" with 50%:

Logo 3 (on white background):

Logo 3 (on black blackground):

You can check the t-shirt designs with the link.


upvote is downvoting

When clicking on upvote the counter is moving backwards below is the code where needs fix
if(commentModel.userHasUpvoted) {
newUpvoteCount = previousUpvoteCount - 1;  //this should be previousUpvoteCount+1
} else {
newUpvoteCount = previousUpvoteCount + 1; //this should be previousUpvoteCount-1
}

oh ok i see this needs to be implemented ,
set userHasUpvoted to true fist time when the same user upvotes that next time check if userHasUpvoted is true if so then downvote I guess that is what the code is doing
if(commentModel.userHasUpvoted) {
newUpvoteCount = previousUpvoteCount - 1; //this should be previousUpvoteCount+1
} else {
newUpvoteCount = previousUpvoteCount + 1; //this should be previousUpvoteCount-1
}

Yes, user is only allowed to upvote once so the upvote button toggles the value. Thus userHasUpvoted needs to be set to true whenever the user has actually upvoted the comment. This works as intended.


Implement a daisy-chain of 74HC165 modules on the DE0-Nano

Add another 74HC165 module to the top-level AHDL top.tdf and connect it to the original module as well as the necessary I/O pins.

Satisfied with commit 460e619d978d0220f13c8161a0fd2c5e8b7217c0


DataCache on macOS?

Is it possible to update UIKit part conditionally with AppKit to make this package work on macOS projects too?

I will consider it in the future, but not now. If you can, please help to implement it then make a pull request. Thank you!

Thanks for the response, well I can mark UIKit with this
#if os(iOS)
    import UIKit
#elseif os(OSX)
    import AppKit
#endif

and mark UIImage with a #if os(iOS), if that is good enough for you, I'll send a pr right away.


Feature/au 685 oma profiili

AU-685

What was done

Own Profile DOM was fixed to have better accessibility

How to install

Make sure your instance is up and running on correct branch.

git checkout AU-685-oma-profiili
make fresh

Run make drush-cr

How to test

[ ] Check that everything looks normal and works
[ ] Check that code follows our standards and makes sense

Sadly my FF hangs when authenticating.
But the paging seems to be a bit off?

Also profile data fields with bullets?


sidecar inject annotation istio-cni not work in istio 1.7.3

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities/
To ask questions about how to use Istio, please visit https://discuss.istio.io
)
Bug description
Affected product area (please put an X in all that apply)
[ ] Docs
[X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version --short and helm version if you used Helm)
licheng@admindeMacBook-Pro  ~  istioctl version --remote
client version: 1.7.3
control plane version: 1.7.3
data plane version: 1.7.3 (2 proxies)
licheng@admindeMacBook-Pro  ~  kubectl version --short
Client Version: v1.19.1
Server Version: v1.18.3
How was Istio installed?
istio operator
Environment where bug was observed (cloud vendor, OS, etc)
opehshift-4.5
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

fix: because of I have two different configurations for sidecarInjectorWebhook in my yaml to install the istio.


generate single sitemap.xml

Is it possible to generate single sitemap.xml with just the page urls and not generate individual sitemap files?
Thank you!

@s-kris for sure! Feel free to make a PR just make sure to account for sitemaps larger than the limit as that is why it is broken out.
Would love that functionality, let me know if you get stuck. Thanks.

Perfect. I'll be back with a PR.


remove botservice from index.json

Removing the botservice extension because all features of the extension are already in the main botservice command module in the Azure-CLI repo or are outdated.
@williexu, @tjprescott, @yugangw-msft

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

~[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)~
~[ ] Have you run python scripts/ci/test_index.py -q locally?~

For new extensions:

~[ ] My extension description/summary conforms to the Extension Summary Guidelines.~

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/stevengum/azure-cli-extensions.git@delist-botservice#subdirectory=src/$EXT&egg=$EXT"


⚠ Bofrost.de has degraded performance

In 390bfff, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1219 ms

Resolved: Bofrost.de performance has improved in e3f26df after 9 minutes.


how to use source conf

my .tmux.conf:
'source  ~/.config/.tmux/.tmux.conf'
'source  ~/.config/.tmux/.tmux.conf.local'
but No effect

Hello @yuxing1994,
This configuration needs ~/.tmux.conf to exist, and ~/.tmux.conf.local to optionally exist.
There's no way to place them in different directories.
Please follow https://github.com/gpakosz/.tmux#installation


can Centos7 run newest version of Vscode?

os:centos 7
version of vscode :1.75
can you release some compatibility version to support Old users to run the newest VSCODE?

What's the problem you're seeing?

vscode version:1,77
OS:centos7
i run the binary file code in the folder of VSCode-linux-x64,but it shows some error message just like below.
$ ./code .
[114504:0406/085249.244404:FATAL:setuid_sandbox_host.cc(157)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /home/user/Downloads/VSCode-linux-x64/chrome-sandbox is owned by root and has mode 4755.
[1]    114504 trace trap  ./code --version

[114517:0100/000000.259916:ERROR:zygote_linux.cc(650)] write: Broken pipe (32)

OK. It's done!


Order transaction is unclear

What are you trying to achieve?

Manual transaction with a message created two "events" - success and info, that contains only message. Technically its one event. "Capture" type can be enriched with additional message
"Info" from previous point is visible as order event, but capture event is not. This event doesn't have any context (like message "Manual transaction captured with message: asdf")
API doesn't respond with capture event and custom message event doesn't contain amount of money

Steps to reproduce the problem

Use transaction payments in channel
Create order
Capture a transaction for full amount

What did you expect to happen?
I expect clear information about transactions, mainly merged events into single one
Logs
No response
Environment
Saleor version: 3.16.1

@lkostrowski  Dashboard uses transactionCreate to create manual transactions.
The mutation has multiple fields for amounts: authorized, charged, refunded and canceled.  Ref:https://docs.saleor.io/docs/3.x/api-reference/payments/inputs/transaction-create-input
If you provide any of these amounts (you can also provide all of them), Saleor will create a separate TransactionEvent for each. To keep the correct flow of recalculation process: Ref https://docs.saleor.io/docs/3.x/developer/payments#event-types-explanation
Saleor creates a separate info event when fields from https://docs.saleor.io/docs/3.x/api-reference/payments/inputs/transaction-event-input are provided. We didn't want to propagate a psp-reference from the input to events, as it could have an impact on re-calculations (we rely on matching the events based on the pspReference).
This is why we decided to keep a separate event: INFO, which can contain the details, that don't have an impact on amounts. (We didn't propagate message, as it could look strange, when you would have 4 amount events, with the same message).
I believe that these events should be better represented on dashboard side (for example, having amount field there, doesn't make sense).
Also. There is mutation: OrderMarkAsPaid which creates the Payment or TransactionItem(depends on channel settings: ref https://docs.saleor.io/docs/3.x/api-reference/miscellaneous/objects/payment-settings#code-style-fontweight-normal-paymentsettingsbdefaulttransactionflowstrategybcodetransactionflowstrategyenum--) to mark if order is paid.

Created the task on dashboard side for using orderMarkAsPaid - https://github.com/saleor/saleor-dashboard/issues/4421

Maybe we can somehow nest INFO within parent even. Agree that this can look better on the frontend side. cc @andrzejewsky

Closing, as I believe this should be resolved on the dashboard side. The INFO event is created only when you provide the TransactionEventInput.  Additionally, dashboard should use a dedicated mutation for marking as paid.
@lkostrowski If you don't agree, please re-open the issue.

@korycins I am reopening and assigning myself for future refinement


doc for write() and end()

Some documentation on how and when to use write() and end() would be really helpful.
Thanks

Can you elaborate a bit? Those are just the normal node.js writable stream methods. You shouldn't need to use them though if you are feeding busboy data from another (readable) stream, in which case you can use something like pipe() to automatically handle writes and backpressure.

The only usual hangup people have with cloud functions is realizing that the request body is already buffered somewhere by the cloud function SDK, so pipe() ends up doing nothing (when using the examples).
Aside from that, just make sure you always call .end() to be on the safe side.

I thank you for the amazing library you provide, but I had the same problem and lost 4 hours of work before finding about the .end().
In the end I found the correct code on a stackoverflow request (https://stackoverflow.com/questions/73217059/busboy-missing-file-data) about something else.
I'm a Front-end developer mainly, so I didn't know about .write() and .end()

@Lonolf Just as a heads up, write() and end() are a standard part of node.js streams, so you may find yourself using them again if you work with streams.


Example for solis inverter config

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I have configured your amazing card with a solis inverter. I thought it'd be useful to have my config as an example somewhere for others to use.
Expected behaviour
Example of Solis inverter config in documentation
Possible Solutions
This is my config using (https://github.com/wills106/homeassistant-solax-modbus to read the inverter)
type: custom:sunsynk-power-flow-card
cardstyle: lite
show_solar: true
large_font: true
inverter:
  modern: true
  autarky: energy
battery:
  energy: 14400
  shutdown_soc: 20
  full_capacity: 100
  empty_capacity: 20
  show_daily: true
solar:
  show_daily: true
  mppts: 1
load:
  show_daily: true
  additional_loads: 2
  load1_name: PC
  load1_icon: mdi:desktop-classic
  load2_name: TV
  load2_icon: mdi:television
grid:
  show_daily_buy: true
  show_daily_sell: true
  show_nonessential: false
entities:
  grid_power_167: sensor.solis_inverter_meter_active_power
  essential_power: sensor.solis_inverter_house_load
  essential_load1: sensor.pc_socket_power
  essential_load2: sensor.smart_socket_3_power
  energy_cost_buy: sensor.octopus_energy_electricity_xxxxx_xxxxxx_current_rate
  energy_cost_sell: >-
    sensor.octopus_energy_electricity_xxxxxx_xxxxx_export_current_rate
  remaining_solar: sensor.energy_production_today_remaining
  radiator_temp_91: sensor.solis_inverter_inverter_temperature
  use_timer_248: switch.sunsynk_toggle_system_timer
  inverter_voltage_154: sensor.solis_inverter_inverter_voltage
  load_frequency_192: sensor.solis_inverter_inverter_frequency
  inverter_current_164: sensor.solis_inverter_inverter_current
  inverter_power_175: sensor.solis_inverter_active_power
  day_battery_charge_70: sensor.solis_inverter_battery_charge_today
  day_battery_discharge_71: sensor.solis_inverter_battery_discharge_today
  battery_voltage_183: sensor.solis_inverter_battery_voltage
  battery_soc_184: sensor.solis_inverter_battery_soc
  battery_power_190: sensor.battery_load
  battery_current_191: sensor.solis_inverter_battery_current
  day_grid_import_76: sensor.solis_inverter_grid_import_today
  day_grid_export_77: sensor.solis_inverter_grid_export_today
  grid_ct_power_172: sensor.solis_inverter_meter_active_power
  day_load_energy_84: sensor.solis_inverter_house_load_today
  day_pv_energy_108: sensor.solis_inverter_power_generation_today
  pv1_power_186: sensor.solis_inverter_pv_total_power
  pv1_voltage_109: sensor.solis_inverter_pv_voltage_1
  pv1_current_110: sensor.solis_inverter_pv_current_1

Mode
lite
Context / Reason

Thanks for sharing, I'll add it to the docs

No worries, thanks for creating this card :D

Added https://slipx06.github.io/sunsynk-power-flow-card/examples/solis.html


Logs ingested from AppEngine Standard environment (Java11) do not correlate with the respective request log

In AppEngine (Gen2) that runs Java11 applications in Standard Environment the application logs are not grouped under the respective request log entry like it worked in Gen1 with Java8.
The use of the auto-population of the http request and trace context does not solve the problem.

It looks like the fix was applied in Log Explorer and the logs are grouped as expected.


:wrench: [Under investigation] Fixed navigation to work properly on both iOS and Android.

Issue

close #953

Overview (Required)

Links

Screenshot (Optional if screenshot test is present or unrelated to UI)

Before
After

Movie (Optional)

Before
After

It seems that the change isn't released yet. 👀

Now that we can access the issues, it seems there are some workaround and the workaround 2 looks easier.
https://github.com/JetBrains/compose-multiplatform/issues/4735#issuecomment-2090235055

This commit is probably unnecessary. 🤔
Even if you revert this commit, it seems that the iOS issue has been resolved. 🥳
I'll revert it. 🏃


Make whiskers' end state visible and toggle-able

PCB-Sizes branch now how UI functionality to disable the simulation while maintaining the information of the whiskers. Also comes with an end state button.


LottieView caused builds to fail

LottieView (previously named Lottie) is defined in this package but wrapped in a #canImport statement.
I managed to get the app running from a simulator using LottieView from this framework, but the build seemed to fail.
I'm not sure the way we are conditionally importing Lottie is correct?

Think Lottie may support support SwiftUI directly now? https://github.com/airbnb/lottie-ios/discussions/2189


EmptyStaticCreds: static credentials are empty

Cool project, but i'm getting EmptyStaticCreds on a new build from src. Any idea?
panic: EmptyStaticCreds: static credentials are empty
goroutine 1 [running]:
panic(0x705b40, 0xc420018400)
/usr/local/go/src/runtime/panic.go:500 +0x1a1
main.perror(0x7f27da9489d0, 0xc420018400, 0xc420113980)
/root/updateIP/dyndns_route53/dyndns_route53.go:37 +0x10a
main.main()
/root/updateIP/dyndns_route53/dyndns_route53.go:118 +0x1119

fixed.. updated config.json values to values in code.. also removed static hostname in code..

@rjgough did you had to adjust any code? If so, please make a pull request. About config.json, maybe the readme is not clear enough? Any feedback is more then welcome!

Sure will sent a pull request later on today
Sent from my iPhone

On 11 Jul 2017, at 07.21, Gert Leenders<EMAIL_ADDRESS>wrote:
@rjgough did you had to adjust any code? If so, please make a pull request. About config.json, maybe the readme is not clear enough? Any feedback is more then welcome!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Proto: Rename in RequestType enum according to Feng-Shui.

Issue #, if available:
N/A
Description of changes:
Taken from #220

renames like Hvals to HVals
renames like HashDel to HDel
renames of the corresponding functions
{G,S}etString renamed to {G,S}et

All renames (except HashSet) were done with Ctrl+Shift+H magic
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Fixed, thank you for review.
@barshaul your turn.


How to use django-vote in templates ?

The doc doesn't have explanation of how to use this package in templates!
I will be very grateful if you give me example.... Thanks

Any updates on this? I'm also having hard time figuring out how to use it in templates. Some description with examples might be super helpful. If I can't figure this out, then unfortunately it'll prevent me from using this solution and switch to something else.

@apetryla you can use vote_count and vote_exists in template now, the README is also updated to add example code.

@shellfly Thank You for the update, now it's getting much clearer. However, currently I'm getting an interesting problem.
In my project, using the template
{% load vote %}
{% vote_exists post %}

results in the returned boolean as expected, however
{% load vote %}
{% vote_exists post %}
{% vote_count post %}

raises as
TemplateSyntaxError at /t3/
Invalid block tag on line 4: 'vote_count'. Did you forget to register or load this tag?

So only part of templatetags gets loaded correctly? Can't get my head around it.
I tried to experiment with a project as close to Your testing config as I could and with the exact same comments.html I'm getting
TemplateSyntaxError at /comments/

Invalid block tag on line 5: 'vote_count', expected 'empty' or 'endfor'. Did you forget to register or load this tag?

when using python manage.py runserver.
Could You reproduce it?


Meetup - 5th Monday

There is an interesting edge case in the meetup problem:
some months have five Mondays.
March of 2015 has five Mondays (the fifth being March 30th), whereas
February of 2015 does not, and so should produce an error.

Thanks, @JKesMc9tqIQe9M for pointing out the edge case.
See exercism.io#2142.

That's not a problem here, the exercise doesn't allow for passing the fourth week. :stuck_out_tongue:
I'll go and fix it.

Hah, that certainly solves that problem :)

As someone mentioned in the original issue:

You can't regularly schedule a meetup on the fifth week of every month because February usually doesn't have a fifth week at all (and no month ever has a full fifth week). It's nonsense to even support "fifth" as an input in the first place. I think that first-fourth, last, teenth are the only scheduling inputs that should be supported.

What do you think, should we just bail on the fifth week idea?

All things considered I think that's best, yes. The consequence of including it would be to introduce much extra complexity.
Katrina Owen<EMAIL_ADDRESS>schreef op 22 maart 2015 21:37:08 CET:

As someone mentioned in the original issue:

You can't regularly schedule a meetup on the fifth week of every
month because February usually doesn't have a fifth week at all (and no
month ever has a full fifth week). It's nonsense to even support
"fifth" as an input in the first place. I think that first-fourth,
last, teenth are the only scheduling inputs that should be supported.

What do you think, should we just bail on the fifth week idea?

Reply to this email directly or view it on GitHub:
https://github.com/exercism/xelixir/issues/38#issuecomment-84697072

--
Verzonden van mijn Android telefoon met K-9 Mail.

Cool, I'll close the issue and the PR. Thanks!


Startup time

Here is my result with a basic Dart program:
$ cat a.dart
main () {
   print('sunday');
}

$ dart --version
Dart VM version: 2.7.1 (Thu Jan 23 13:02:26 2020 +0100) on "windows_ia32"

$ time dart a.dart
sunday

real    0m0.390s

Dart seems to have a startup about 3 times slower than other options:
$ cat a.jl
println("sunday")

$ julia -v
julia version 1.5.0-DEV

$ time julia --compile=min a.jl
sunday

real    0m0.140s

$ cat a.rb
puts 'sunday'

$ ruby -v
ruby 2.6.4p104 (2019-08-28 revision 67798) [x86_64-cygwin]

$ time ruby a.rb
sunday

real    0m0.140s

$ cat a.py
print('sunday')

$ python3 -V
Python 3.6.9

$ time python3 a.py
sunday

real    0m0.078s

$ cat a.php
<?php
echo "sunday\n";

$ php -v
PHP 7.3.7 (cli) (built: Jul 21 2019 18:10:35) ( NTS )

$ time php a.php
sunday

real    0m0.062s

$ cat a.pl
print("sunday\n")

$ perl -v
This is perl 5, version 26, subversion 3 (v5.26.3)

$ time perl a.pl
sunday

real    0m0.047s

$ cat a.lua
print('sunday')

$ lua -v
Lua 5.3.5  Copyright (C) 1994-2018 Lua.org, PUC-Rio

$ time lua a.lua
sunday

real    0m0.031s

Please use dart2native. @cup
$ time dart ./a.dart
sunday

real	0m0,118s
user	0m0,122s
sys	0m0,063s
$ dart2native a.dart
Generated: /a.exe
$ time ./a.exe
sunday

real	0m0,010s
user	0m0,004s
sys	0m0,007s

$ type dart
dart is /dart-sdk/bin/dart

$ dart --version
Dart VM version: 2.7.1 (Thu Jan 23 13:02:26 2020 +0100) on "windows_ia32"

$ type dart2native
sh: type: dart2native: not found

https://github.com/dart-lang/sdk/issues/40545

dart2native is the intended tool for this:
https://dart.dev/tools/dart2native

On IA32 machines you can instead generate a non AOT snapshot, because AOT snapshots are not available.
$ dart --snapshot=a.snapshot --snapshot-kind=kernel a.dart
$ dart a.snapshot

This would not be as fast as AOT compiled code but would remove CFE overheads from the equation.


sftp fails with Could not resolve hostname sftp

sftp backend support appears to be broken on current master, at least on on macOS 10.14.6 and likely on Ubuntu 18.04.4 LTS too.
Output of restic version
restic 0.9.6-dev (compiled manually) compiled with go1.14 on darwin/amd64

How did you run restic exactly?
mbp:restic igor$ ./restic -r 'sftp:igor@<IP_ADDRESS>:/data/var/tmp/restic-test-repo' restore --target=/tmp/restic-restore-output latest
subprocess ssh: ssh: Could not resolve hostname sftp: nodename nor servname provided, or not known
Fatal: unable to open repo at sftp:igor@<IP_ADDRESS>:/data/var/tmp/restic-test-repo: unable to start the sftp session, error: EOF

What backend/server/service did you use to store the repository?
sftp
Expected behavior
Expected restic to connect to remote repository and restore the backup from there.
Actual behavior
Restic fails to start sftp session
Steps to reproduce the behavior
Run restic restore using sftp backend
Do you have any idea what may have caused this?
Almost certainly caused by e1969d1e3360fd268184b810b5f08ad327725c66, reverting that commit makes the problem go away.  I also don't see any ssh version that supports -- host syntax on Ubuntu.  @greatroar fyi.

Confirming this on Ubuntu 18.04. Putting the host last makes sense, but I don't see anything about -- in man ssh. It might be a very recent addition.

Please pull #2633 (or revert some other way).


Store metadata

Create a file storing: training range, f-min, f-max, seglen, frequency nodes reduction factor.
This way the PE sampler will know exactly which basis it needs to interface with.

Done for everything except for training range.

Moved to new repository: https://github.com/GCArullo/JenpyROQ/issues


pypy3.9: reports missing branch from function-to-decorator

Describe the bug
in pypy 3.9 coverage reports a missing branch from a function to its decorator (though that should be impossible?)
To Reproduce
How can we reproduce the problem?  Please be specific. Don't link to a failing CI job. Answer the questions below:
What version of Python are you using?
$ ./venv/bin/pypy3  --version --version
Python 3.9.10 (8276b505180f70c5784a698a510f0a17317a85c3, Feb 19 2022, 16:50:47)
[PyPy 7.3.8 with GCC 10.2.1 20210130 (Red Hat 10.2.1-11)]

What version of coverage.py shows the problem? The output of coverage debug sys is helpful.
coverage debug sys

$ ./venv/bin/coverage debug sys
-- sys -------------------------------------------------------
               coverage_version: 6.3.1
                coverage_module: /tmp/y/venv/lib/pypy3.9/site-packages/coverage/__init__.py
                         tracer: -none-
                        CTracer: unavailable
           plugins.file_tracers: -none-
            plugins.configurers: -none-
      plugins.context_switchers: -none-
              configs_attempted: .coveragerc
                                 setup.cfg
                                 tox.ini
                                 pyproject.toml
                   configs_read: -none-
                    config_file: None
                config_contents: -none-
                      data_file: -none-
                         python: 3.9.10 (8276b505180f70c5784a698a510f0a17317a85c3, Feb 19 2022, 16:50:47)[PyPy 7.3.8 with GCC 10.2.1 20210130 (Red Hat 10.2.1-11)]
                       platform: Linux-5.13.0-28-generic-x86_64-with-glibc2.31
                 implementation: PyPy
                     executable: /tmp/y/venv/bin/pypy3
                   def_encoding: utf-8
                    fs_encoding: utf-8
                            pid: 6693
                            cwd: /tmp/y
                           path: /tmp/y/venv/bin
                                 /home/asottile/opt/pypy3.9-v7.3.8-linux64/lib/pypy3.9
                                 /tmp/y/venv/lib/pypy3.9/site-packages
                    environment: HOME = /home/asottile
                                 PYTHONSTARTUP = /home/asottile/.pythonrc.py
                   command_line: ./venv/bin/coverage debug sys
                sqlite3_version: 2.6.0
         sqlite3_sqlite_version: 3.7.17
             sqlite3_temp_store: 0
        sqlite3_compile_options: DISABLE_DIRSYNC; ENABLE_COLUMN_METADATA; ENABLE_FTS3
                                 ENABLE_RTREE; ENABLE_UNLOCK_NOTIFY; SECURE_DELETE
                                 TEMP_STORE=1; THREADSAFE=1

What versions of what packages do you have installed? The output of pip freeze is helpful.
(this is just what comes with pypy + coverage)
$ ./venv/bin/pip freeze
cffi==1.15.0
coverage==6.3.1
greenlet==0.4.13
hpy==0.0.3
readline==<IP_ADDRESS>

What code shows the problem?  Give us a specific commit of a specific repo that we can check out. If you've already worked around the problem, please provide a commit before that fix.
here's a minimal bit of code:
def dec(g):
    def dec_inner(f):
        return f
    return dec_inner

@dec(1)
def f(x):
    print(x)

f(1)

What commands did you run?
$ ./venv/bin/coverage erase && ./venv/bin/coverage run --branch -m t && venv/bin/coverage report --show-missing
1
Name    Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------
t.py        8      0      2      1    90%   8->7
---------------------------------------------------
TOTAL       8      0      2      1    90%

Expected behavior
I expect 100% coverage from that script
Additional context
n/a

related, a decorated function with a branched return will show a missing branch as well:
def dec(f):
    return f

@dec
def f(x):
    if x:
        return x

f(0)
f(1)

$ ./venv/bin/coverage erase && ./venv/bin/coverage run --branch -m t && venv/bin/coverage report --show-missing
Name    Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------
t.py        8      0      4      2    83%   5->4, 6->exit
---------------------------------------------------
TOTAL       8      0      4      2    83%

Confirming this issue also affects property decorators:
class Foo:
    @property
    def bar(self):
        return 42

print(Foo().bar)

$ coverage run --branch foo.py
42
$  coverage report --show-missing
Name     Stmts   Miss Branch BrPart  Cover   Missing
----------------------------------------------------
foo.py       5      0      4      2    78%   1->3, 3->2
----------------------------------------------------
TOTAL        5      0      4      2    78%

This is new behavior in PyPy3.9.  Coverage adapted in commit 9d90ebd1.

This is now released as part of coverage 6.3.2.

I see same behavior on latest version of PyPy3.8, though. So it's not just affecting PyPy3.9.


Mutators as a path among others

I have an example use case that either does not work currently, or I am deeply misunderstanding the documentation ;-)
#[derive(Debug, TypedBuilder)]
struct CalendarEvent<Tz: chrono::offset::TimeZone> {
    #[builder(setter(into))]
    title: String,

    // Goal: User should be able to set start(...), then may EITHER:
    //   - set end(...)
    //   - call with_duration(chrono::Duration) which would automatically add duration to start and set end.
    #[builder(
      mutators(
        fn with_duration(&mut self, duration: chrono::Duration) {
            self.end = self.start.clone().checked_add_signed(duration).unwrap();
        }
      )
    )]
    start: chrono::DateTime<Tz>,
    #[builder(???)]
    end: chrono::DateTime<Tz>,
}

The "goal" comment describes the use case, i.e. two different paths to the completed builder.
Of course, there may be multiple other paths.
I have tried several combinations but none work. Is it because I am using generics? Or am I trying to do something that is currently unsupported?

Mutators change the value of the builder payload, but they cannot change its type. Having mutators that can change the type is a more complex feature (which would also require more complex syntax)


Introduce RHEL family

merges the files for rockylinux and almalinux into a rhel dockerfile
decided not to add fedora into this since, it requires to add the epel packages, which might make fedora too close to rhel for our community users. But wonder what you think, is it a good/bad idea to try to also merge that one?

Clean up fedora dockerfile

remove packages which are already installed by default "coreutils" and "curl"
make it similar to the other dockerfiles for easier maintenance

decided not to add fedora into this since, it requires to add the epel packages, which might make fedora too close to rhel for our community users. But wonder what you think, is it a good/bad idea to try to also merge that one?

IMHO I would merge all of them. What packages come from the epel repos? Only the needed ones like zfs? I think merging them all under the same family would be ok as in the end they will have the minimal packages required for a base Kairos flavor, but they can expand it with whatever afterwards, so they can also drop the epel repos on their custom dockerfiles no?

I tried doing a pure rhel but it's a lot more work :( lots of packages are not available, not sure if they are named differently or if I need to enable some repo


To convert and pipe live images

Hello,
I want to stream a real time video to a browser. I have a camera API which generates images on the fly. What I want to do is as soon as the camera grabs and saves an image to the directory, I want to take that image, convert it and pipe it.
In my camera API, I have a buffer of 5 and after the fifth image it again writes to the first image. I have written code for doing something but it doesn't work correctly.
The code is here -
app.get('/video', function(req, res){
	res.contentType('flv');
	var proc = ffmpeg('/root/flycapture.<IP_ADDRESS>_armhf/bin/14135894-%d.jpg')
		.format('flv')
		.videoCodec('libx264')
  		.loop()
  		.fps(15)
  		.on('end', function() {
    		console.log('file has been converted succesfully');
  		})
  		.on('error', function(err) {
    			console.log('an error happened: ' + err.message);
  		})
		.pipe(res, {end:true});
});

For watching the directory where the image is saved, I do following -
var previousImage = '';
  var imageName = '14135894-0.jpg';

  // Watching the directory bin for changes of file
  fs.watch('bin/', function(event, filename){
    // When the file changes, it will run the if condition
    if (filename != imageName){
      // Change the imageName to filename and emit with help of socket
      previousImage = imageName;
      imageName = filename;
      uploadImage();
    };

I can call something as I called uploadImage() when a new image is generated. Or anything other that this would also work.
I would really appreciated if you can help in this.
Thanks

Why would you want to convert each image to flv ? Why not just send each image directly to the client and display it ?

I am doing the same thing. My uploadImage function with the help of sockets sends images to client directly. But that is generating soo much of network traffic. The image gets saved in the browser cache and then browser gets it. It happens every 60 ms seconds or soo. And I think this is a bad.
Also if someone doesn't have a good network speed or is far away, it might get problem.

I don't think ffmpeg has a way of generating a flv dynamically with new files appearing all the time. It will generate a flv with all files present when it started, but will stop then.
The "right" way to have a web client display live video is:

run ffserver or equivalent, configure it to serve a RTP/RTSP stream
run ffmpeg with a real video capture device (not captured files) as an input and make it output to ffserver
point the client  tag to the rtp/rtsp stream URL

And you don't need nodejs/express for this.
If the only thing you get from the camera is image files, what you could do is generate video by chunks of a certain length (say 5 seconds for example). Every 5 seconds, you run a ffmpeg command to convert all images from the last 5 seconds into flv and send the URL to this flv to your client via a websocket. Of course you will have a 5 second latency in this case. Also you will need to move files around so that you're sure to keep files for 5 seconds.
But you should really look into using a real capture device here. Writing an image on the SD card every 60ms is definitely a bad idea.

Thanks for the suggestions. I am a beginner and still trying to learn stuff but I got what you said in "right" way. This means I can run my node server and ffserver simultaneously on RaspberryPi and have a live stream as well as different things I might require.  But I would like to add in here.
The camera I am using is a Point Grey research grade camera and it has its own GUI which uses some libraries to display the video and has several other sample code to acquire images.  It doesn't expose me to /dev/video0.
Coming to if I have only access to image files, 5 seconds would be a lot of delay. I know its very bad to convert each pixel and then save it at disk because it consumes time as well.
But my professor told me to have a live stream and then if I would like to capture an image, it should let me do that too.
Is there any other suggestion you can give me so that I can work with this set up.
Someone pointed out to use v4l2loopback to create a fake /dev/video0 , feed in the image there and then use ffmpeg and ffserver. But i don't know how to feed in image through Node to v4l2loopback.


a question about expected training time

hello, xl-sr. thank you for your great project.
when i train the stem, although the resolution is 16, but when i train it on 1-v100, batch_size = 8,  got 120sec/kimg
is it right?? but when i train the same zip using stylegan2-ada, batch_size = 32, got 8sec/kimg.
maybe the stylegan-xl is too slow???
can you answer my question? thank you very much. I really need your help

can you release your expected training time for us?
i think this will help us.

I have another question, you mentioned in your paper that it is best to set the batch_size to 2048 when the resolution is between 16 and 64, but when I set the batch_size to 32, 2 gpu's and a separate batch_size of 16, it already shows that a single GPU with 16 video memory is not enough.
Is it possible that the batch_size you mentioned in your paper requires too much video memory?

StyleGAN-XL is a larger than StyleGAN2/3. If you want to have the same model size, you can pass:
--cbase 16384 --cmax 256 --syn_layers 4
and for superresolution stages:
--head_layers 4
Re. batch size. If you train with a single GPU and a batch-size of 2048, you do this via:
--batch=2048 --batch-gpu 16
This way you do batch accumulation, ie., accumulate gradients 128 times to get to a final batch size of 2048. If you have only a single GPU available, you can also start with a batch size of 256, the results won't be much worse. Also, for smaller / less diverse datasets, you can directly start with much lower batch sizes.

thank you for your help


Pointer scan for this address

Great project!
For a complete gentleman's set, the "Pointer scan for this address" functionality is required, but it does not work yet.
Are you planning to add this feature?

It is planned, I just haven't started on it yet.

I tried to resolve this part, on my fork but i couldn't. I couldn't find the hook which i should to work on it. I tried to inject kernel dll to DMA target via DMA but it crashed the target memory, each time. I hope Metick will need it ASAP, or can resolve this for us ^^

I resolved pointer scanning!! not within Metick's plugin though.  I haven't tried to debug Metick's implementation but looking at the source code, I am sure this is one of the problems (for debugging a 64-bit process):

Since the hk_openprocess returns an invalid handle, the call to isWow64bitProcess below in the newkernelhandler unit would return False.

Which would set processhandler.is64bit to false and mess up pointer map generation by passing the 32bit stop address $0xFFFFFFFF to stop in processhandler unit.

This would further mess up the pointer map and pointer scanning in pointervaluelist unit.
However, since the current version of ce plugin sdk does not export the isWow64process function, I ended up solving this by adding it to the exported functions in the ce source code. If anyone has a better solution around this, please let me know.
Here is the link to my plugin.


3DCityDB Docker Healthcheck

Docker can test if a container is still working using healthchecks. This is especially important when orchestrating services and one services depends on a functional state of another services, like for instance the WFS requires an operational 3DCityDB instance.
Here is an example for a PostgresDB:

# ...
# ...

    healthcheck:
      test: ["CMD-SHELL", "pg_isready -U postgres -d postgres"]
      interval: 30s
      timeout: 5s
      retries: 10
      start_period: 30s

# ...
# ...

For the 3DCityDB this means, that:

The Postgres/Oracle DB inside the container is up and running
The 3DCityDB Schema setup has completed

The second point leads to my question. At which point can I safely assume that 3DCityDB schema is fully set up. In the install script, the last operation (for Postgres) is to set the search path:
https://github.com/3dcitydb/3dcitydb/blob/20e1d0ab4c9c5d27461a40fbf73025a5ac7aa4b1/postgresql/SQLScripts/CREATE_DB.sql#L89
Do you think checking for the correct search_path is suitable as a heathcheck test?
psql -U postgres -d postgres -c 'show search_path;' | grep -q "citydb"

Hm, good question. If you create a new schema and want to directly import data, I would say that you should also wait for the SELECT citydb_pkg.change_schema_srid(:SRSNO,:'GMLSRSNAME'); process to be finished. But I am not sure how to test this in your healthcheck. Maybe you could grep for some specific output in the log (like Done - or we could add a more specific output).

Hmm, I think grepping from log files is not a good idea. The healthcheck should be able to determine if the 3DCityDB works at any time. Log files could be rotated after a while causing the check to fail.
However, it might be too much to ask for a healthcheck to test if the entire 3DCityDB schema is setup correctly and has not been corruped, e.g. by malicious user queries.
So probably a combination of pg_isready to check server availability and a small check that the schema setup has finished would be a reasonable compromise. We could e.g. set a variable/file/flag in the entrypoint.sh script to indicate a successful setup and sue that for the check.
With pure SQL this could work:
The last statement in the setup process is this:
https://github.com/3dcitydb/3dcitydb/blob/20e1d0ab4c9c5d27461a40fbf73025a5ac7aa4b1/postgresql/SQLScripts/CREATE_DB.sql#L100
This is the function that is called:
https://github.com/3dcitydb/3dcitydb/blob/20e1d0ab4c9c5d27461a40fbf73025a5ac7aa4b1/postgresql/SQLScripts/CITYDB_PKG/SRS/SRS.sql#L219-L243
Thus, if I check if the container env var SRID matches the SRID of all 3DCityDB geometry columns SRIDs in public.geometry_columns we know that the 3DCityDB is ready, right?
As all columns of the schema despite implicit_geometry, texture_coordinates, relative_other_geom are set to SRID, this could do the trick:
SELECT AVG(gc.srid) srid
FROM public.geometry_columns gc
WHERE gc.f_table_schema = 'citydb'
  AND gc.f_geometry_column NOT IN ('implicit_geometry', 'texture_coordinates', 'relative_other_geom');

-- OR
SELECT AVG(gc.srid) srid
FROM public.geometry_columns gc
WHERE gc.f_table_schema = 'citydb'
  AND f_geometry_column <> 'implicit_geometry'
  AND f_geometry_column <> 'relative_other_geom'
  AND f_geometry_column <> 'texture_coordinates';

Is there a way to check whether the entry script has finished its job? I think that would be the easiest way to make sure the 3DCityDB is completely set up. For periodic health checks afterwards, maybe it's really sufficient just to use pg_isready or to check whether you can log into the database.

I did some testing and found that pg_isready alone is not sufficient. A container will already be listed as ready/healthy during the setup process of the schema, as it is possible to connect to the database while the entrypoint.sh scripts are applied.
Currently, I don't see a suitable way to check, if entrypoint.sh has successfully finished. One option is to write a small file during initialization we can use for the check later on. However, this breaks Docker`s paradigm that containers should be ephemeral and can be disposed at any time.
An example: If we create a small file in entrypoint.sh at /3dcitydb-init.log this file needs to survive an update of a 3DCityDB container using docker rm oldcontainer and docker run --name newcontainer 3dcitydb:3dcitydb-pg:newversion to be available for future healthchecks.
This would not be the case, as the file is located in the containers file system which is dumped during the update.
An alternative would be to put this file in $PGDATA, which lives inside a volume.
This way the container would be disposable and we could use the file in healthchecks even after an update. However, it somehow feels wrong to write data in the Postgres data folder. I don't know if this causes problems for Postgres, I'd have to look this up.
As the same issue applies for the Oracle version, I think an "SQL-only" approach would be the best, as it is guaranteed to work independently of the container file system or version.
I suggest:

Create a healthcheck.sql with a query that verifies that the schema has been setup correctly. We could provide that e.g. in SQLScripts/UTIL/HEALTHCHECK/ for Postgres and Oracle.

We could extend this multiple healthcheck for different scenarios in the future, if requried.

Execute this SQL in a Docker healthcheck to verify database connectivity and schema status.
An alternative could be an additional table for status information of the 3DCityDB instance.

@clausnagel @yaozhihang
I am thinking of implementing this for the upcoming release (if there is enough time, otherwise, wll include it in the next release.) What do you think about the suggestions in my last post?
This is important when starting up orchestated containers, that e.g. rely on a 3DCityDB instance to be "up and ready for data import", e.g. when running automated tests. Currently, the Importer/Exporter container would exit with an error, if the 3DCityDB is not ready for import, when ist starts.
(Maybe this is something we should consider implementing for the CLI version of the ImpExp too. Retry connecting to the given database until a given timeout, that has a reasonable default value, that can be overwritten by CLI ARG or ENV var. This would be handy for automation with the ImpExp in general).

I think such kind of heathcheck function is good. We coud extend this function in future and use it to do heath check of migrated or upgraded 3DCityDB instance.

Closing due to inactivity. Feel free to reopen with a corresponding PR.


Code cleanup plus added a getter

cleaned up some of the unnecessary code and added a getter method to get is the button is showing or not. Also there is a bug fix for if you try to show.hide the button while the animation is in progress. if you try to do one or the other during an animation the button does not return to the proper original position

Connet


Blink on Hoobs show cameras as Privacy Mode or Camera Offline and no doorbell

I just installed Hoobs and successfully put my Blink system on. I can see everything but the doorbell in HomeKit, but 1) the cameras don't turn on; I have two accessories named Switch and Switch 2 for each of my four cameras, but I don't know what those are and whether to set them on or off. Any help here? Are there any detailed instructions on how use the Accessories?

The Switch is for using Motion Activated and Switch 2 is for Privacy Mode.


unable to run demo tests against empty maven repository

unable to download all dependencies against empty .m2 repo due to the missed
17:38:00 [ERROR] Failed to execute goal on project carina-demo: Could not resolve dependencies for project com.qaprosoft:carina-demo:jar:1.0: Could not find artifact javax.media.jai:com.springsource.javax.media.jai.core:jar:1.1.3 in central (https://repo.maven.apache.org/maven2) -> [Help 1]
17:38:00 [ERROR]
to be able to download it automatically we have to register "Spring plugins" repo:
https://mvnrepository.com/artifact/javax.media.jai/com.springsource.javax.media.jai.core/1.1.3

can't reproduce


Directory remove/rename issue

Hey!
All methods from xcodebuild.project.manager works good, except directory moving/renaming. Each time when i try to call delete_group or move_or_rename_group i getting some warnings like:
Renamed "TestFolder" successfully
Could not find "TestFolder" in project

and changes not reflected in xcode project

Hi! What is the structure of your project? Single module/multiple modules/SPM modules?
Does it also happen if you use Project Manager or integration with nvim-tree/oil.nvim?

Single .xcodeproj, with SPM. Im not using nvim-tree/oil, so i can't check that.
Are you using xcodegen behind? Because it looks like deletion process falling on last step, when we needs to re-generate .xcodeproj. Because directory delete is fine (i mean it deletes from filesystem)

It is using Xcodeproj (ruby tool) underneath. If you use the function that you mentioned, they don't perform any disk operations, so I guess it's something else?

Could you try performing these operations using built-in interface? Call :XcodebuildProjectManager

If i navigate to inside target folder and then do :XcodebuildProjectManager -> Delete current group then it works

That's interesting, because in my integrations with nvim-tree and oil.nvim I also react on AFTER events and it works well. If you physically delete files/folders it shouldn't have any effect on your xcodeproj therefore the tool should be also able to perform actions and update xcodeproj.
Could you show your code? Is it the one from the other issue?

And just one thought. Maybe you are subscribing twice for events? And the second call fails. Actually your output confirms that, because you've got 2 messages. I think there should be only one.

Could you show your code? Is it the one from the other issue?
yep

Yeap, I have just found the issues.
As you said, it happens after the file is deleted so you can't rely on vim.fn.isdirectory because it check physical files. So the solution is:
    api.subscribe({
      event = "file_deleted",
      handler = function(path)
        local extension = vim.fn.fnamemodify(path, ":e")
        local isDir = extension == ""
        if isDir then
          if shouldUpdateProject(path) then
            projectManager.delete_group(path)
          end
        else
          if shouldUpdateProject(path) then
            projectManager.delete_file(path)
          end
        end
      end,
    })

With rename/move you just need to check the final path, not the source (which doesn't exist anymore):
local isDir = vim.fn.isdirectory(paths.destination) == 1

It should solve your problem. Let me know if it works.

All working as expected, thanks!


🐛 Bug Report: aperture theme is not user friendly

📜 Description
Aperture theme is not user friendly
👍 Expected behavior
The backstage header contains the href's/ links which is not visible
👎 Actual Behavior with Screenshots

👟 Reproduction steps
Enable "aperture" theme
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Definitely not intentional, good spot @db-cataloge-db! Fixes are referenced above, one component correction in the main repo and some additional overrides on the demo site.


payload_on/off being ignored in binary_sensor

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.33.1
Python release (python3 --version):
Python 3.5.2
Component/platform:
mqtt
Description of problem:
Ignoring the payload_on/off values
Expected:
Should show as "open" or "closed". Instead presents as "on" or "off"
Problem-relevant configuration.yaml entries and steps to reproduce:
binary_sensor:
  - platform: mqtt
    state_topic: "/smartthings/South Living Room Window/contact"
    name: "South Living Room Window"
    payload_on: "open"
    payload_off: "closed"
    sensor_class: opening

Traceback (if applicable):

Additional info:

The payload is not ignored. Your configuration works fine with mosquitto. There is no issue but I guess that with "show" you  mean the representation in the frontend.

https://community.home-assistant.io/t/customize-text-of-binary-sensor-on-off-state/4653

BTW, usually topics don't start with a /.


index ready time

Hello
Running on a 8core 8gb ram system... since 30min ago
curl http://localhost:8000/readyz
{"detail":"Index not ready."}
anyone knows aproximatley how much time it takes to finish it?
Thank you

That's too much time. Can you access the log stream for the yente container and share it here?

it's finished. took about 1h

Just to follow up here:

Did you enable the statements API? That thing takes forever, the entities should be much faster.
Maybe consider upping the memory available to ElasticSearch a bit (in docker-compose.yml, ES_JAVA_OPTS). For your spec, I'd dedicate almost half your memory to the index - also for better runtime performance.

Thanks alot for the help and this repo.


How to add multiple kafka clusters with single kafka-UI

Is kafka-UI for multiple clusters, It seems for multiple clusters with single dashboard. But how can we configure that, there is no proper information about it

I think I found how to configure multiple cluster with same UI

Hi, Team
I have configure multiple cluster with Kafka-UI, But it is showing only 3 cluster, Is it not allowing more than 3 clusters..?
Please suggest me if it will not support more than 3 clusters

Hi, @vangahari. Thanks for creating issue. There is no such limit. Could you please share your config?

@germanosin Sorry, I gave number same in the env variable as shown in the following screen shot

I changed it to 3, its working now
environment:
#dev cluster config
KAFKA_CLUSTERS_0_NAME: "dev-cluster"
KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS: ""
#Staging cluster config
KAFKA_CLUSTERS_1_NAME: "stg-cluster"
KAFKA_CLUSTERS_1_BOOTSTRAPSERVERS: ""
#ewf cluster config
KAFKA_CLUSTERS_2_NAME: "preprod-cluster"
KAFKA_CLUSTERS_2_BOOTSTRAPSERVERS: ""
#demo cluster config
KAFKA_CLUSTERS_3_NAME: "demo-cluster"
KAFKA_CLUSTERS_3_BOOTSTRAPSERVERS: ""
@germanosin Thanks for the quick reply

Great, thank you for provided info!


[Paper+1] Garbage Collection in JyNI – How to bridge Mark/Sweep and Reference Counting GC Stefan Richthofer

This paper is still somewhat incomplete. I already submitted this version, because no exact submission deadline time for September the 15th was defined. Main things that remain to be done is writing the introduction and a chapter about weak reference support. I am going to finish these tasks today (i.e. on the 15th), so this is actually in time if I can assume that the 15th is entirely within the deadline.

Also note that my fork of euroscipy_proceedings still contained my proceedings paper from 2013. I pulled all updates from euroscipy_proceedings into my fork and replaced the 2013-paper with the 2015 one. However, now I noticed that my pull-request also lists actions done to the 2013 paper.
For current EuroSciPy just ignore all but the last item "Updated paper to EuroSciPy 2015".
For some reason my paper is still rendered with the page header "PROC. OF THE 6th EUR. CONF. ON PYTHON IN SCIENCE (EUROSCIPY 2013)". I don't understand how this comes, since I recently pulled all updates from euroscipy_proceedings into my fork. Any advice how to fix this?

Hi Stefan,
Your master branch is in conflict with the current repository, see https://github.com/Stewori/euroscipy_proceedings/blob/master/euroscipy_proc.json#L8 where the 2013 edition is still referenced.

Alright. Confusing that github did not report this when I merged the 2015 variant into my fork. Actually I wanted to create a different fork for the 2015 proceedings, but github does not seem to allow multiple forks of the same project within one account. However - thanks for the hint!

I tried to solve the conflict with the 2013 variant and it appears, there actually is no conflict; https://github.com/euroscipy/euroscipy_proceedings/blob/master/euroscipy_proc.json is already outdated in the base-repository. Or is there another EuroSciPy proceedings repository than euroscipy/euroscipy_proceedings that I should have forked?
For now I'll fix euroscipy_proc.json in my fork manually up to my knowledge and include it in the next pull request.

Indeed, the current master has an outdated file euroscipy_proc.json. My local version was not in sync. I'll let @NelleV manage this then.

Comments from a cursory first glance:

Please do a grammar check on the whole paper.
The abstract is an important part of the paper.  You can improve it by providing a better idea of what JyNI is, and by clarifying the purpose of this specific paper.
Add references to claims such as claims that Python considers switching to mark-and-sweep.
Cite the other projects mentioned, such as CFFI.
The logo figure is superfluous.

I updated the abstract and added references for many mentioned projects. I think it is not required to reference extremely popular projects like NumPy, PyPy, IronPython or Python itself, is it?

I removed the logo figure like you suggested, but actually would like to have the logo on the paper in some way, so if there is a proper way to do so, please let me know. If logos are not wanted in principle I will of course accept that.

The claim that Python considers switching to mark-and-sweep is from Larry Hasting's talk at 2015 Python Language Summit and was already referenced (https://lwn.net/Articles/640179). I think I made carefully clear that these are rather vague considerations, still underlining the relevance of the topic. I did not mention the reference in the abstract originally, because I somehow thought that abstracts should not contain references. I did not yet check for other claims that might need references (but will do so soon); if you have some specific in mind, please feel free to point me to it.

I checked the paper for grammar issues, but it mostly appears okay to me. Still I improved various minor things, e.g. I am having a hard time to decide whether to use a "-" to connect words or not. I queried several terms on the internet to look up common style but still didn't really get the rule (e.g. it appers to be "problem-specific", but on the other hand "Python specific".) Also apologize if I still overlooked spelling issues. Auto-correction is messy, because the text contains so much computer-science specific vocab. However, I will give it another look.

The logo can be incorporated into a diagram, if such a diagram exists.  It's just a bit strange to have it hanging around without a caption.
I wouldn't reference NumPy, but I would reference PyPy, CFFI, and other projects that do similar things.
Vague claims can still be cited--you're not stating that Larry promised anything.  The sentence outlining the claim is one of those for which the grammar bothered me.  "Vague claim", as you use here, is very acceptable.
Hyphenation preferences vary quite a bit; I don't think it's a big deal.  The correct use of an em-dash (rather than an en-dash) to connect sentences would be good.

Sorry for the delay.  As far as I can tell, this is similar to ideas we're playing with for cpyext, which is PyPy's CPython C extension module compatibility layer.  This paper goes into more depth than we do, e.g. supporting reference cycles, but on the other hand cpyext is more practically useful right now and we're seeing issues that occur in practice.  There are a lot of them that the authors seem to ignore so far, and numpy is particularly troublesome, so I'd be careful when presenting the work here as getting most of the way towards numpy support.  The approach is still interesting and well-presented, though.

For reference, CFFI is very different: it is a layer to access generic C code.  This is instead about C code that is written specifically for one VM, and we're running a different VM for the same high-level language.

Thanks very much for taking time to review the paper, @arigo.  @Stewori If you can update the paper with this latest feedback, then we're good to go from my point of view.

Also sorry for the delay; I am very busy right now and hope to find time for the final editing soon.
My intention was to present numpy as the driving motivation for JyNI, which is simply plain truth. I'll check, where I might have stated this in a too optimistic manner.
So far I could hardly work with numpy itself, because essential dependencies were not yet working (e.g. ctypes is currently gaining support). I am highly interested to learn in detail about the practical issues you mention (maybe we could get in contact about this topic independently from this thread). It's not that I am ignoring them; such issues were - so far - just out of scope due to higher priority on improving the general concept - supporting gc, weak references etc.
That said, keep in mind that JyNI is at least internally very different from cpyext, mainly indicated by the fact that it is (in principle, modulo uncompleted API) binary compatible to extensions, while cpyext requires recompilation with different headers. Given the tons of macros Python is using, it is literally impossible to tell how issues from cpyext would transfer to the JyNI concept.
No doubt - there will be issues with numpy and other extensions - gonna face them one by one.
Regarding CFFI, it was never my intention to compare it with JyNI; I rather target CFFI as one of the key extensions to support. From CPython's point of view CFFI (i.e. the CPython version) is an ordinary C-extension, isn't it (one that accesses gcc though)?

Hi Stefan,
On Thu, Dec 3, 2015 at 2:05 AM, Stefan Richthofer<EMAIL_ADDRESS>wrote:

Regarding CFFI, it was never my intention to compare it with JyNI; I
rather target CFFI as one of the key extensions to support. From
CPython's point of view CFFI (i.e. the CPython version) is an ordinary
C-extension, isn't it (one that accesses gcc though)?

Ah, sorry, I didn't get that.  Then I'm pretty sure that a better idea
would be to natively port CFFI to Jython.  It is not a lot of  work; in
fact the design of CFFI is such that it is meant to be ported in this
way, as is done with PyPy.  It doesn't expose anything to the user that
only makes sense on CPython.  Moreover the interesting parts of such a port
would be to tweak it in order to improve performance inside Jython; that's
how CFFI is much faster on PyPy than on CPython, after the JIT kicks in.
Of course, it is more interesting to think about general solutions, but in
this specific case I believe we can achieve much, much better results with
a native rewrite.
A bientôt,
Armin.

Then I'm pretty sure that a better idea would be to natively port CFFI to Jython

I agree, but this would be a distinct project and I don't have capacity to do this too. AfaIk https://github.com/jythontools/jiffy (c.f. http://bugs.jython.org/issue2148) is exactly about that, but I don't know about its current state as the page is currently just a stub (which is why I didn't mention it in the paper).
However the JyNI variant will provide an alternative that is potentially available earlier; "works unperformant" is still way better than "unsupported".
Another sorry for the delay; I am currently at ICMLA and was busy with preparation before. As soon as I am back from this conference I'll follow up on this.

I applied the changes suggested by @arigo, updated the ctypes-section, corrected some typos, added references for PyPy and IronPython, added a footnote about CFFI and applied various other improvements.


Add Audit info to Kafka header fields

The audit trail of who set / unset an active alarm (or shelved or created an alarm definition) should be included in the message headers.  Other audit info could also include the host name of the message origin.  A timestamp is already built-into the Kafka raw message, but we may need to confirm if it is sufficient.
Note: Including audit info in the payload will NOT work for tombstone messages, since by definition they must have a null payload.
See: https://cwiki.apache.org/confluence/display/KAFKA/KIP-82+-+Add+Record+Headers

Added in release 0.2.0