Hot Restart not working on Android Simulator

Describe the bug
Since the beta-6 version, whenever I try to restart the app in the Android simulator, the app crashes with the log output below. Have not evaluated yet, if this a simulator problem only. Have you experienced this?
Environment (please complete the following information):

OS: Android
Version:
cbl: 1.0.0-beta.8
cbl_flutter: 1.0.0-beta.8

Additional context
Log output
F/libc    ( 9615): FORTIFY: pthread_mutex_lock called on a destroyed mutex (0xbb647ab0)
F/libc    ( 9615): Fatal signal 6 (SIGABRT), code -1 (SI_QUEUE) in tid 9723 (DartWorker), pid 9615 (atsinthefoto.de)
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'google/sdk_gphone_x86/generic_x86:10/QSR1.190920.001/5891938:user/release-keys'
Revision: '0'
ABI: 'x86'
Timestamp: 2021-11-07 23:22:05+0100
pid: 9615, tid: 9723, name: DartWorker  >>> some.app <<<
uid: 10141
signal 6 (SIGABRT), code -1 (SI_QUEUE), fault addr --------
Abort message: 'FORTIFY: pthread_mutex_lock called on a destroyed mutex (0xbb647ab0)'
    eax 00000000  ebx 0000258f  ecx 000025fb  edx 00000006
    edi e6e0633e  esi ba8fbb40
    ebp eb5b4ad0  esp ba8fbae8  eip eb5b4ad9
backtrace:
      #00 pc 00000ad9  [vdso] (__kernel_vsyscall+9)
      #01 pc 00092328  /apex/com.android.runtime/lib/bionic/libc.so (syscall+40) (BuildId:<PHONE_NUMBER>8016ad14f4b98c3ab6c65c)
      #02 pc 000ad651  /apex/com.android.runtime/lib/bionic/libc.so (abort+193) (BuildId:<PHONE_NUMBER>8016ad14f4b98c3ab6c65c)
      #03 pc 0011c44a  /apex/com.android.runtime/lib/bionic/libc.so (__fortify_fatal(char const*, ...)+58) (BuildId:<PHONE_NUMBER>8016ad14f4b98c3ab6c65c)
      #04 pc 0011b96a  /apex/com.android.runtime/lib/bionic/libc.so (HandleUsingDestroyedMutex(pthread_mutex_t*, char const*)+58) (BuildId:<PHONE_NUMBER>8016ad14f4b98c3ab6c65c)
      #05 pc 0011b812  /apex/com.android.runtime/lib/bionic/libc.so (pthread_mutex_lock+258) (BuildId:<PHONE_NUMBER>8016ad14f4b98c3ab6c65c)
      #06 pc 007e0f80  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #07 pc 0002adb9  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #08 pc 00033984  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #09 pc 00033931  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #10 pc 000331eb  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #11 pc 0003345c  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #12 pc 00033571  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #13 pc 004fef7d  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #14 pc 004fefc5  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #15 pc 00027aea  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (CBLListener_Remove+74) (BuildId: df740a80caa6a80a970888797b4be93c40921838)
      #16 pc 00019c0b  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblitedart.so (BuildId: 5c7fe917ddb3db18b8c693a09d71d3b8f115706c)
      #17 pc 00011cdc  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblitedart.so (BuildId: 5c7fe917ddb3db18b8c693a09d71d3b8f115706c)
      #18 pc 00011b08  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblitedart.so (BuildId: 5c7fe917ddb3db18b8c693a09d71d3b8f115706c)
      #19 pc 01c2db63  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libflutter.so (BuildId: 85974401eedd76aedb89e0e20d8d510d538e57d4)
      #20 pc 0190e036  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libflutter.so (BuildId: 85974401eedd76aedb89e0e20d8d510d538e57d4)
Lost connection to device.
Performing hot restart...

Thanks for reporting this. I haven't come across this yet.
Which edition (cbl_flutter_ce or cbl_flutter_ee) and version is the backtrace from?
Looks like that during hot restart, finalizer for the different objects, like Database, are called.
Seems like this is more specifically related to change listeners, because of this line:
#15 pc 00027aea  /data/app/some.app-VZiwYqGY9ys4MG5w41XnSQ==/lib/x86/libcblite.so (CBLListener_Remove+74) (BuildId: df740a80caa6a80a970888797b4be93c40921838)

It has very likely something to do with a use after free, since the crash is triggered by using a destroyed mutex.

The trace is from the ee edition... I have double checked that the crash is only happening on the Android simulator, not on real devices or the iOS simluator. We are using change listeners to subscribe to document changes. At the moment we are not unsubscribing the listeners, because we want to hold the subscriptions open for the complete session of the app. Do we have to do an unsubscription handling for hot reload?

I found that the issue is caused by how cbl is interacting with CBL C. Whether this is incorrect usage of the C API or should be fixed in CBL C is a bit unclear. I have contacted Couchbase to resolve this ambiguity.
The problem is that garbage collection is not deterministic, and runs finalizers in an uncontrollable order.
It's possible that a database has been closed and freed before an associated listener token could be removed.

This is probably something that can happen on every OS and deployment target, and not just when hot restarting, but anytime garbage collection runs and some specific set of finalizers is called in the wrong order.
There is nothing you need to do. I don't think there is a workaround.

It's a bug in CBL C.

Fixed by #235


Add support for listening on given address.

This is done by the --listen command-line argument.
I find this very useful for servers that have multiple network addresses (we don't all have servers dedicated to monitoring!). In my case, I have a publically-accessible eth0 and an internal eth1, as this server also hosts a VPN. While it's certainly possible to use a firewall to reject external connections (and I do that in addition), this seems cleaner to me.
Thanks for a wonderful collectd frontend!

This looks great to me. Thanks! :rocket:


Update package-lock.json

I was trying to set-up from the repository today and had issues installing the dependencies. Odd errors about missing files and unmet dependencies everywhere.
After removing the package-lock.json file everything installed like a charm.
Looking at my diff, some dependencies had never resolved to a URL in the repo’s lock file. The lock file is also still for version 0.7.0 of Omnibear. I feel like it has somehow gotten out of date with your actual package.json.

After rm -rf node_modules package-lock.json (possibly my favourite node debugging command :wink:) doing npm install followed by npm run build I seem to get a functioning extension. But…
I also get a few warnings pertaining to micropub-helper (ping @grantcodes):
><EMAIL_ADDRESS>build /Users/martijn/Sources/omnibear
> webpack --config webpack.prod.js

Hash: dedf1fb3ffbd3b641d58
Version: webpack 2.7.0
Time: 6546ms
                   Asset       Size  Chunks             Chunk Names
             icon-32.png  779 bytes          [emitted]
                index.js    66.7 kB       0  [emitted]  index
                 page.js    45.1 kB       2  [emitted]  page
  fonts/jaldi-bold.woff2    13.4 kB          [emitted]
   fonts/lato-bold.woff2    14.1 kB          [emitted]
fonts/lato-regular.woff2    13.9 kB          [emitted]
            icon-128.png    2.68 kB          [emitted]
             icon-16.png  419 bytes          [emitted]
           background.js    22.8 kB       1  [emitted]  background
             icon-48.png    1.05 kB          [emitted]
             icon-64.png    1.31 kB          [emitted]
                icon.svg    1.31 kB          [emitted]
              index.html  322 bytes          [emitted]
           manifest.json  718 bytes          [emitted]
                page.css  231 bytes          [emitted]
              styles.css    7.77 kB          [emitted]
   [0] ./~/preact/dist/preact.js 23.5 kB {0} [built]
   [1] ./src/util/utils.js 1.52 kB {0} {1} [built]
   [3] ./src/util/micropub.js 723 bytes {0} {1} [built]
  [11] ./~/micropub-helper/src/main.js 15.5 kB {0} {1} [built]
  [16] ./src/util/url.js 1.79 kB {1} [built]
  [18] ./src/util/draft.js 860 bytes {0} [built]
  [19] ./src/util/settings.js 1.54 kB {0} [built]
  [20] ./src/background/authentication.js 2.16 kB {1} [built]
  [22] ./src/page/entry.js 3.04 kB {2} [built]
  [23] ./src/background.js 2.92 kB {1} [built]
  [37] ./src/index.js 368 bytes {0} [built]
  [38] ./src/page.js 1.22 kB {2} [built]
  [39] ./src/page/dom.js 1.32 kB {2} [built]
  [40] ./~/microformat-shiv/microformat-shiv.js 110 kB {2} [built]
  [41] ./~/parse-uri/index.js 994 bytes {1} [built]
    + 27 hidden modules

WARNING in ./~/micropub-helper/src/main.js
9:4-25 "export 'FormData' (imported as 'dependencies') was not found in './dependencies'

WARNING in ./~/micropub-helper/src/main.js
10:20-41 "export 'FormData' (imported as 'dependencies') was not found in './dependencies'

WARNING in ./~/micropub-helper/src/main.js
12:4-26 "export 'DOMParser' (imported as 'dependencies') was not found in './dependencies'

WARNING in ./~/micropub-helper/src/main.js
13:21-43 "export 'DOMParser' (imported as 'dependencies') was not found in './dependencies'

WARNING in ./~/micropub-helper/src/main.js
15:4-20 "export 'URL' (imported as 'dependencies') was not found in './dependencies'

WARNING in ./~/micropub-helper/src/main.js
16:15-31 "export 'URL' (imported as 'dependencies') was not found in './dependencies'

It doesn’t seem to break anything. But I also have no clue where this is coming from. If some node developer knows more, I’d love to learn.

I’ve updated the package-lock file. Let me know if you have any more troubles.
I’ve seen the warnings you mention, and they’re coming from micropub-helper. I’m note exactly sure what the issue is internally there.


⚠ Bofrost.lu has degraded performance

In 60f96c8, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2295 ms

Resolved: Bofrost.lu performance has improved in 1ff91d0 after 36 minutes.


Parallelize the GapEncoder

Implements parallelization of the GapEncoder column-wise, as it is currently a simple for loop.
This could speed up the process significantly for wide datasets (intuitively, to be benchmarked).

We might want to chose n_jobs based on the number of columns and the number of available core.

We never set parallelism on by default, because it can too easily lead to overcommit (eg with nested objects that have parallelism).
We have progressively evolved toward using n_jobs=None, which opens the door to joblib to make clever choices.
Our long-term view is that library users should rely on joblib.parallel_config to set parallel computing. But this is new and still to be experiment.
@TomMoral is the expert on these questions.

We never set parallelism on by default, because it can too easily lead to overcommit (eg with nested objects that have parallelism).

Ok, so what about the predict methods of RandomForest in sklearn?
https://github.com/scikit-learn/scikit-learn/blob/main/sklearn/ensemble/_forest.py#L866
In case the user does set n_jobs, wouldn't it be good to provide some heuristics?

In case the user does set n_jobs, wouldn't it be good to provide some heuristics?

That's a good point. The code in RandomForest is there to create bunches of estimators. This is useful when there are many more iterations than CPUs. I do not think that it will be useful here.

Aha not sure if I am an expert but here is my 2cts:
joblib is auto batching so it should already  be not too bad. But if you find instances on which auto batching does not work well, please report them so we can try to improve the heuristic for every joblib users 😁
Also note that if possible, it is best to not change the n_jobs too much in different parts of the pipeline as this might cause respawning/resizing the executor, which is slow for fast running examples. If you really have an heuristic that works well, it is better to keep n_jobs the same but create smart batches (except if you intend to play with lower level parallelism, but in this case, tell me more 🙃)


C++\error\reversearray.cpp

Reverse the array
inp -- >  1234
out -- >  4321
Find the error in the code🚀
NOTE:- Screenshot of the code and output is required

Can you please assign this issue?

Please assign this issue to me !!

can you please assign me this issue?

This issue is already assigned to someone...try new

Hey, I have resolved the error in the reversearray.cpp file. It now works correctly! Please check my PR #300


TDK-9019 Adding Bug fix in build.sh for Opensourced L1 HALs

Reason for change: Adding bug fix in build.sh for L1 LPA HAL Suite
Risks: Low
Test Procedure: Check the ticket

 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.


Renovate and more

This PR adds Renovate to the chart.
Renovate only watches the Chart.yaml files in the ./config folders with custom managers to work with the weird release tags upstream.
The .sh scripts collect the versions (updated by renovate as explained above) and continues as before.
I also added a small script at the end to ask the user if they want to update the chart's version number as we never do it otherwise.
The new method for updating this chart should be as follows (explained in README):

Renovate opens a PR.
git checkout to the PR's branch
run make all

[!NOTE]
If we want to do things right (even though unlikely that it causes issues), we only release if both the CPI and CSI versions officially support the same Kubernetes version (and update the table in the README would be nice).

Let's see what happens #yolo


tried to run it and ran into that issue:

TypeError: Cannot read property 'rx_bytes' of undefined
    at NetworkIO.update (/usr/src/widgets/network.js:24:47)
    at /usr/src/index.js:217:45
    at Array.map (native)
    at /usr/src/index.js:217:22
    at Request.<anonymous> (/usr/src/utils/index.js:49:14)
    at emitOne (events.js:77:13)
    at Request.emit (events.js:166:7)
    at IncomingMessage.<anonymous> (/usr/src/node_modules/request/request.js:1184:12)
    at emitOne (events.js:77:13)
    at IncomingMessage.emit (events.js:166:7)

+1

Same for me+

+1

For me also. +1

Same here:
$ docker version
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:12:04 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:12:04 UTC 2015
 OS/Arch:      linux/amd64

Yep, +1
git commit 75f4f25adb0a94080495c29a4418a6550bf365ae
Client:
Version:      1.10.2
API version:  1.22
Go version:   go1.5.3
Git commit:   c3959b1
Built:        Thu Feb 25 09:29:05 2016
OS/Arch:      linux/amd64
Server:
Version:      1.10.2
API version:  1.22
Go version:   go1.5.3
Git commit:   c3959b1
Built:        Thu Feb 25 09:29:05 2016
OS/Arch:      linux/amd64

+1

+1
Client:
Version:      1.10.3
API version:  1.22
Go version:   go1.5.3
Git commit:   20f81dd
Built:        Thu Mar 10 15:54:52 2016
OS/Arch:      linux/amd64
Server:
Version:      1.10.3
API version:  1.22
Go version:   go1.5.3
Git commit:   20f81dd
Built:        Thu Mar 10 15:54:52 2016
OS/Arch:      linux/amd64

I managed to fix that locally changing statItem.network.rx_bytes here to statItem.networks.eth0.rx_bytes (same for following lines)
All I did was add console.log(statItem) before the breaking line, if networks.eth0 do not work for you try that.


Add possibility to pass extra parameters in getTree

It would be usefull to be able to pass parameters that can be retrived in the method addFlatTreeConditions.
For example if you need to add conditions depending on the user rights or others...

PR was rebased here: https://github.com/KnpLabs/DoctrineBehaviors/pull/358
Closing this one.


Standing on object you are holding sends you into space

I had step offset set to 0.4 and when I held the object I was holding towards my body space it assumed I was standing on it, and so moved me up, then it moved up the object I was holding, and repeated this until I was high in space.

If you want to avoid collision with objects you will need to setup the proper collision layers and masks to fit your gameplay logic. You can find some documentation on layers and masks here: https://docs.godotengine.org/en/stable/tutorials/physics/physics_introduction.html

How does this help?  Don't you want to collide with with same objects that you can pick up?

In the general case no. You want the falling feature only to collide with your level geometry. This is common practice for (simple) game collision. You usually setup your (named collision) layers to have finegrained control over what collides with what.
Of course if you want to implement a fully physics based game like boneworks this is different but then you will need to implement a very different and much more complex system than what the toolkit can provide.

Closing this for now. This is more of a problem related to the game and physics system that someone implements and not something that can be solved in a general way in the toolkit. Using collision layers and setting them according to the desired gameplay logic is one potential solution. Maybe there is a way to setup better default behavior; if someone has an idea let me know.


Hide labels based off distance from camera

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Labels show on every node even when there so small they're unreadable. This can have a big impact on performance
What is the new behavior?
Labels don't render when they're so small they can't be read. This is based off zoom and distance from camera
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
BEFORE
https://github.com/reaviz/reagraph/assets/40581813/e2880203-ead6-4f21-accc-18c50e055a28
AFTER
https://github.com/reaviz/reagraph/assets/40581813/9de77a72-4602-41e2-85f1-997bf1769cce
https://github.com/reaviz/reagraph/assets/40581813/f32d7f90-23e5-4afc-81f5-30469ceb88df

This doesn't hide edge labels. Edges don't have a position so we'd prob grab either the source or target node to decide whether to show the label
I can add hiding edge labels to this PR or a follow up


KOGITO-1225: Bump to KIE 7.33.0.Final

All checks have failed
well this is disappointing

@mariofusco is fixing this


New package: OceanLight v1.0.0

Registering package: OceanLight
Repository: https://github.com/haoboatlab/OceanLight.jl
Created by: @tayT0T
Version: v1.0.0
Commit: 7983266a7badbe941667a72bdadf7ef8ab43cbe8
Reviewed by: @tayT0T
Reference: https://github.com/haoboatlab/OceanLight.jl/issues/3#issuecomment-2412028544

[noblock] Documenter should probably not be a dependency of your project. Please make sure to only list the minimum number of dependencies


Possible vanity URL/domain for this project? (html5plea.se)

I checked the site/app http://domai.nr/
Turns out html5plea.se is available.
OVH are doing really nice prices for these domains (.SE)
It should be totally bought, and hooked up to Github pages.
Just a thought.

Cool cool.


removed virtualhost provider settings

related to : https://github.com/cloudfoundry-incubator/service-fabrik-broker/pull/471

@pranjaljain Can you please remove it from templates/deployment.yml


[sourcekitd] Add a request to get statistics from the service

... and add a few basic statistics about the number of requests, ASTs
built, etc.  The Statistic type is loosely based on the one from LLVM,
but suitable for using without DEBUG macros and using SourceKit UIdents
to identify the statistic.  The easiest way to add a new statistic is to
add it to SwiftStatistics.def in the SwiftLangSupport.

@swift-ci please smoke test


[cxx-interop] Add missing dependency to CxxStdlib

The CxxStdlib module now relies on the libcxxshim modulemap, so we need to make sure that the modulemap is copied into the correct location before CxxStdlib is built.
rdar://108007693

@swift-ci please smoke test


10414 DEV Create folder structures and add scripts to bring up the Admiral container on a fresh RHEL 7 machine

https://github.com/Shippable/pm/issues/10414

This is complete.
Closing


Title Case or Localized title case?

Hey @gaearon a typography question here!
React's docs use “reasonable title case” (pronouns, conjunctions, etc. are usually lowercase). This is not a de-facto English standard, but is commonplace in the press and has become widespread in English tech docs.
However, title casing is pretty foreign to other roman-letters languages, such as French.
Is this an intentional design choice that you wish roman-letters languages would preserve? Or was that a sort of reflex, and you don't mind locale-aware title casing?
French translations use the same casing as English so far, but it does feel foreign.
Let us know!
/cc @JeremiePat @shinework @tesseralis

@tdd I'd say do whatever looks normal in French.

@tesseralis awesome, I'll do an overhaul pass.

@reactjs/reactjs-org-french-translation attention à tous ! La politique de capitalisation des titres a changé, pour revenir à la casse française normale (seule la première lettre de la phrase est majuscule).
Je vais faire une PR dédiée pour les textes déjà fusionnés ou en passe de l'être. Faites juste attention lors de vos prochaines PR !


[admin] (Links) Adding moniker to Common API link from Excel page

Docs Build status updates of commit f5937b9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-add-ins-error-handling.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Misc fixes

waits for https://github.com/ceph/cn/pull/26

Also fix WorkingDirectory for multi cluster so they don't share the same one.

Need to fix the functional tests:
The command "sudo make tests" exited with 0.
after_script.1
0.02s$ ./cn stop
Error: unknown command "stop" for "cn"
Run 'cn --help' for usage.
unknown command "stop" for "cn"
after_script.2
0.02s$ ./cn purge --yes-i-am-sure
Error: unknown command "purge" for "cn"
Run 'cn --help' for usage.


translate Rule Engine DSL to English

The features inside the Rule Engine DSL should preferably be abstracted over and have an English translation implemented as well.

Has there been any recent progress on this story?

I don't think so. Feel free to pickup the branch and translate some stuff. We can look at how it fits V's plans together. (Or maybe he'll pick it up since I heard he was a bit bored) ^_^

Hi There!
Currently looking at rules for a scala project I'm working on. What's the approx revised timeline for a release in English? I notice the branch is stale, but obviously a lot of effort had already gone into it?

The finance DSL is actually done, I'm going to open a PR for it today. We were revamping the translation of the rules-part of the DSL since we just submitted a CFP-proposal for it.
When would you need it? We'll gladly put it on the top of our list for you :)

Quick reply:) I'm hoping to put it in as an extension  proposal to a
project I'm working on in the travel industry. Obviously I'd be keen to see
the DSL released, but I can't honestly say I need it yesterday! Happy to
get a rough idea though:)
On Aug 22, 2016 6:12 PM, "Jan-Hendrik Kuperus"<EMAIL_ADDRESS>wrote:

The finance DSL is actually done, I'm going to open a PR for it today. We
were revamping the translation of the rules-part of the DSL since we just
submitted a CFP-proposal for it.
When would you need it? We'll gladly put it on the top of our list for you
:)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/scala-rules/rule-engine/issues/4#issuecomment-241322684,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAianTwTp9SFV3MMVJ4APl6sY9gcf0eCks5qiT3YgaJpZM4IG0mO
.

We talked about it today and decided to make this our top priority. Tuesday or Wednesday you'll see a brand new branch pop up for this. (The current branch is too outdated, so we'll be better off cherrypicking what we have onto a new branch). Then we'll try to get it done in the next two or three weeks.

Fantastic!
Tony Culshaw MA Cantab
Director
Mentation Limited - Custom Software Development
Simpson Desert Ride - Sponsor Me!
https://www.mycause.com.au/page/128393/snow2sand
On 23 August 2016 at 06:46, Jan-Hendrik Kuperus<EMAIL_ADDRESS>wrote:

We talked about it today and decided to make this our top priority.
Tuesday or Wednesday you'll see a brand new branch pop up for this. (The
current branch is too outdated, so we'll be better off cherrypicking what
we have onto a new branch). Then we'll try to get it done in the next two
or three weeks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/scala-rules/rule-engine/issues/4#issuecomment-241510535,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAianVqNEu9VR5hovHgxai9axTF-DP2tks5qie53gaJpZM4IG0mO
.

I have integrated the changes made into the Translation branch. I now have everything compiling and ready to build further. There are a few new additions to the Dsl that I still need to incorporate into the English translation.
I want to give it a small overhaul before I push these changes. I expect to do a push on saturday the 27th of august, with compiling and working DSL in English. Not sure if I can incorporate all the new features yet, but will do my best.

On leave next week, but will give it a go on my return. If it goes smoothly I'd be happy to contribute any examples I build.

As soon as you've pushed it, we'll run it against our client's project to see if maybe you've broken anything ;)

I have pushed the merge result where everything compiles and all tests are green.
There are some things still left to do.

Rename some translations, like Determination to Calculation
Translate DslLoopWordTrait
Translate DslListFilterWord
Translate AfrondingsWordsTrait

I will pick them up one by one, starting from the top

I would appreciate a review. I made some choices regarding translations and would like to see what you guys think.

It's going to sound like a cop-out but for now I don't have any remarks. I think we will need to keep a close look at how the english DSL will actually be used in practice, to see if any of the keywords need adjustment or alternatives...


[Outlook] (manifest) update SSO articles with JSON manifest

Learn Build status updates of commit 4a79361:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/outlook/authentication.md
:warning:Warning
View
Details

docs/outlook/authenticate-a-user-with-an-sso-token.md
:white_check_mark:Succeeded
View

docs/outlook/authentication.md

Line 30, Column 30: [Warning: file-not-found - See documentation] Invalid file link: 'json-manifest-overview.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 78ba0b9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/authenticate-a-user-with-an-sso-token.md
:white_check_mark:Succeeded
View

docs/outlook/authentication.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit eca4ad7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/authenticate-a-user-with-an-sso-token.md
:white_check_mark:Succeeded
View

docs/outlook/authentication.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Install fails on os x

Versions
OS version: macOS (Hight Sierra 10.13.1)
Output from: `ng --version`.
    _                      _                 ____ _     ___
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/

Angular CLI: 1.5.3
Node: 8.9.1
OS: darwin x64
Angular: 5.0.2
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.5.3
@angular-devkit/build-optimizer: 0.0.33
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.36
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.3
@schematics/angular: 0.1.5
typescript: 2.4.2
webpack: 3.8.1

Repro steps

Download and install Node JS
run: npm install -g @angular/cli

Observed behavior
The below repeats without stopping
gyp verb command install [ '8.9.1' ]
gyp verb install input version string "8.9.1"
gyp verb install installing version: 8.9.1
gyp verb install --ensure was passed, so won't reinstall if already installed
gyp verb install version not already installed, continuing with install 8.9.1
gyp verb ensuring nodedir is created /Users/esben/Develop/Angular/my-app/node_modules/node-sass/.node-gyp/8.9.1
gyp WARN EACCES user "root" does not have permission to access the dev dir "/Users/esben/Develop/Angular/my-app/node_modules/node-sass/.node-gyp/8.9.1"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/Users/esben/Develop/Angular/my-app/node_modules/node-sass/.node-gyp"
gyp verb tmpdir == cwd automatically will remove dev files after to save disk space

Desired behaviour
A flawless install
Mention any other details that might be useful (optional)
A workaround was found by creating the folder:
/Users/esben/Develop/Angular/my-app/node_modules/node-sass/.node-gyp/
I think there might be some thing like a 'mkdir -p', or the node equivalent missing in the install script.

This seems like a problem with your node/npm and not with Angular CLI.
Please have a look at the fixing npm permissions page, common errors page, npm issue tracker, or open a new issue if the problem you are experiencing isn't known.


Feature/415 ftn unlockable vested coins

:tada: This issue has been resolved in version 0.25.0-development.3 :tada:
The release is available on:

npm package (@development dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Make GraphQL list items non-nullable

Add NonNullList that inherits from graphene.List and automatically adds non-null constraint on contained items.
Apply the new type to all list fields in the schema.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-6315 [schema] Lists should not have null item values


Failed GPU optimizer test with mixed precision

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): CI
TensorFlow version and how it was installed (source or binary): CI
TensorFlow-Addons version and how it was installed (source or binary): CI
Python version: CI
Is GPU used? (yes/no): yes

Describe the bug
GPU test is broken after https://github.com/tensorflow/addons/commit/bd5bbfc66e750cba94360236e7f85481662ba4de.
https://source.cloud.google.com/results/invocations/147c0642-7973-429b-87b1-b8f1b1d2cb14/log

cc @bhack

https://github.com/tensorflow/tensorflow/issues/39775

In the maintime check if we could bypass with https://github.com/tensorflow/addons/pull/1943

We can close this and just add https://github.com/tensorflow/addons/pull/1951#issuecomment-651004300

We can close this and just add #1951 (comment)

Agree, this appears to be solved in upstream for 2.3


when i running showing this

thread 'main' panicked at 'unexpected error: InitializationFailed', /home/runner/.cargo/git/checkouts/vulkano-debee48ff796f830/4bc33c4/vulkano/src/extensions.rs:73:18
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

You're probably running an unsupported driver, graphics card, or distro. Please include which driver you are using (only the packaged driver is supported), which card, and which distro.


If long service name, the dropdown of services on the detailed service page has the carrot on service name

PR waiting on review

Build Master
Verified Fixed


Default/fallback translation improvements

Fixes #31
Proposed Changes

Implement alternative translation format for unknown IDs
Implement shared default translations for unknown IDs
Various other fixes to style and docs

Codecov Report

Merging #51 (c4dc2ae) into master (e80e919) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #51   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        24    +1
  Lines          733       777   +44
  Branches       115       116    +1
=========================================
+ Hits           733       777   +44

Impacted Files
Coverage Δ

src/rics/translation/_from_config.py
100.00% <100.00%> (ø)

src/rics/translation/_translator.py
100.00% <100.00%> (ø)

src/rics/translation/offline/__init__.py
100.00% <100.00%> (ø)

.../rics/translation/offline/_default_translations.py
100.00% <100.00%> (ø)

src/rics/translation/offline/_format_applier.py
100.00% <100.00%> (ø)

src/rics/translation/offline/_magic_dict.py
100.00% <100.00%> (ø)

...rics/translation/offline/_placeholder_overrides.py
100.00% <100.00%> (ø)

src/rics/translation/offline/_translation_map.py
100.00% <100.00%> (ø)

src/rics/translation/offline/types.py
100.00% <100.00%> (ø)

src/rics/utility/misc.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Leaderboard

Leaderboard
TODO:
Add submission guidelines, we should discuss in next meeting.

Looks great, we can clear this probably on Monday/Tuesday together.
A couple notes from my side:

[ ] submission instructions, for now I guess can be PR to this table
[ ] add links to leaderboard at the top to click quickly to certain task
[ ] table edit, like adding publication / author associated with the model
[ ] add leaderboard to website
[ ] try to remove the dvs gesture stuff from this PR if possible
[ ] verify that the numbers are correct between the paper results and these tables

Notes:

[ ] Change leaderboard to md file from rst
[ ] Include in website neurobench.ai (a github.io site), in addition / rather than here in this repo
[ ] TODO: can the website repo point to the md file from this repo?
[ ] Move website repo into organization instead of account?
[ ] How should results be submitted? PR with table filled in for new results?

@jasonlyik I found this interesting package that renders ".MD" files
https://www.npmjs.com/package/react-markdown/v/8.0.6

@korneelf1 Here's the preprints for the biocas challenge winners
https://arxiv.org/abs/2409.01762
https://arxiv.org/abs/2410.22283
https://arxiv.org/abs/2409.04428

@korneelf1 Cleaned up just a little more, thanks for putting this together!


Make the demo page look pretty

The developer doesn't know any css, so please make the demo page look pretty.
The demo page can be viewed by running yarn start and pointing your favourite browser to http://localhost:2000.

😍


Sheer Bali Blinds

any way to get the preset "home" position incorporated ?  Trying to adjust using %'s i cant get the sheer blinds to exactly align , where they open in morning and align the sheer to add light to house

I'm not sure what you mean by home. I don't have sheer blinds. Can you post a link to the blinds you have?

Home , or preset position is what I was referring to
Sent from my iPhone

On Mar 29, 2019, at 10:47 AM, ericvitale<EMAIL_ADDRESS>wrote:
I'm not sure what you mean by home. I don't have sheer blinds. Can you post a link to the blinds you have?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Does this thread shine light
https://community.smartthings.com/t/partial-failure-with-somfy-blinds-no-more-preset/154249/4


Archives do not generate with Jekyll 3.0.4

No Errors are apparent, unfortunately.
Reproduction Steps:

Install latest version of Jekyll via bundle (Currently this is 3.0.4)
Install latest version of jekyll-archives (Currently 2.1.0)
Attempt jekyll build and no index.html files will be included were they should be

Workaround is rolling Jekyll back to 3.0, but that prevents the github-pages Gem (And likely others) from installing as it specifies 3.0.4 currently.
Relevant section of my _config.yml:
jekyll-archives:
  enabled: all
  layout: archive
  permalinks:
    year: /:year/
    month: /:year/:month/
    day: /:year/:month/:day/
    tag: /tag/:name/
    category: /category/:name/

gems:
  - jekyll-archives
  - jekyll-sitemap

Hi! The issue here is actually in your Gemfile. When you're using group: :jekyll_plugins, you have to add the Jekyll plugins you want to the group, like so:
gem 'github-pages', group: :jekyll_plugins

# ...

group :jekyll_plugins do
  gem 'jekyll-sitemap'
  gem 'jekyll-paginate'
  gem 'jekyll-archives'
end

Alternatively, you can just remove the group from the github-pages gem.

That indeed was the problem! It makes perfect sense now that you mention it, as I commented out that first line when I rolled back jekyll to stop it from giving me error messages related to the version not being high enough for github-pages.
Thanks so much for helping me with this! Hopefully this helps someone else in the future.


User Profile Popup to Simple Sheet Modal

With the full user page added, the popup will be changed to a sheet modal, and only show some basic information, and link to the user's full profile.
Gives the user the target users basic information without having to load their full profile.
Name, Job, Department, basic description.
https://framework7.io/docs/sheet-modal.html

Sheet Modal is a Framework 7 v2 addition. We are currently using v1 so we would have to migrate to v2 before doing this.
Going to leave the function where it would open the sheet, but have it move straight to the full page profile. That way there is room to either do something else, or do this when on v2.

any update on this card @bmrgould ?

Going to take a look after the current milestone is completed. Going to need to make sure it handles well for accessibility. Ill add it to the next milestone.

Accessibility for the sheet is similar to other on-page F7 components. Focus will need to be set on open/close to move Screen-reader and keyboard users to the correct html elements.
For implementing it, the data required is present in most (all?) entities with their UserDetails passed in with the api request.

Currently setup for only list wires, inside the User-Profile-Sheet branch, but looking like this:


How do I create a dry-run?

I'd like to know if there are any ways to simulate the execution of the program to run with sh?
What I mean is that sh prints out the command it is going to run, but doesn't actually execute it.

You can print the path of the command without running it
import sh
print(sh.ls)

You can also partially apply arguments to a command and print the result
import sh
my_ls = sh.ls.bake("-a", "-l", "--colors")
print(my_ls)

But there is no real dry-run feature.  Are you trying to debug arguments?

Ah the solution you provided works perfectly! Thanks @amoffat for the fast reply! Greatly appreciated!
And thank you for creating sh. It is amazing! 😃

@amoffat On a related note, what would you think about adding a special kwarg to print the command before execution?  As you noted above it can be done with baking, but it would be nice to simplify it.
Current:
cmd = sh.echo.bake('hello')
print(cmd)
cmd()

Proposed:
sh.echo('hello', _print_prior=True)

@carlwgeorge I like the idea of making command debugging more clear.  A good place to expand on this would be the logging functionality that is somewhat already there in sh, but lacking, and not very well documented:
import logging
import sh

logging.basicConfig(level=logging.INFO)
sh.echo("hello")

The output definitely needs some love (seeing duplicate completion messages currently), but I see this as the place to expand on for better print debugging going forward:
INFO:sh.command:<Command u'/bin/echo hello'>: starting process
INFO:sh.command:<Command u'/bin/echo hello', pid 30646>: process started
INFO:sh.command:<Command u'/bin/echo hello', pid 30646>: process completed
INFO:sh.command:<Command u'/bin/echo hello', pid 30646>: process completed
INFO:sh.command:<Command u'/bin/echo hello', pid 30646>: process completed
hello


Feature Request: Ability to issue an action to players that are within the worldguard region.

Hey,
I got a pretty useful idea that would be appreciated if it added.
Currently you can only use "To Actions" types for limited things. Sometimes the Range method does not work like intended.
Therefore, it would be a better option if there was a "to region" action type that checks a player in the region and anybody in that region will be given the commands if they meet the condition as per usual.

You can currently do that by using the "to_condition" action, to check for all players on the server if they accomplishes some conditions and execute the actions for all of them.


MergeOne-Regular.ttf: AssertionError on glyph entries when saving hotfixed TTF

When saving the final hotfixed TTF file for github_google/fonts/ofl/mergeone/MergeOne-Regular.ttf
FontBakery crashes with an AssertionError related to the glyphOrder/glyphs entries. This may mean some mistake in the way we hotfix these entries. I'll take a look at this one tomorrow morning. First step will be identifying exactly which check has hotfix routines that cause the bad data and then I'll fix it.
Traceback (most recent call last):
  File "./fontbakery-check-ttf.py", line 2685, in <module>
    main()
  File "./fontbakery-check-ttf.py", line 2677, in main
    font.save(font_file_output)
  File "/home/felipe/devel/github_felipesanches/scope-one/src/fonttools/Lib/fontTools/ttLib/__init__.py", line 202, in save
    self._writeTable(tag, writer, done)
  File "/home/felipe/devel/github_felipesanches/scope-one/src/fonttools/Lib/fontTools/ttLib/__init__.py", line 628, in _writeTable
    self._writeTable(masterTable, writer, done)
  File "/home/felipe/devel/github_felipesanches/scope-one/src/fonttools/Lib/fontTools/ttLib/__init__.py", line 631, in _writeTable
    tabledata = self.getTableData(tag)
  File "/home/felipe/devel/github_felipesanches/scope-one/src/fonttools/Lib/fontTools/ttLib/__init__.py", line 644, in getTableData
    return self.tables[tag].compile(self)
  File "/home/felipe/devel/github_felipesanches/scope-one/src/fonttools/Lib/fontTools/ttLib/tables/_m_a_x_p.py", line 45, in compile
    self.recalc(ttFont)
  File "/home/felipe/devel/github_felipesanches/scope-one/src/fonttools/Lib/fontTools/ttLib/tables/_m_a_x_p.py", line 64, in recalc
    self.numGlyphs = len(glyfTable)
  File "/home/felipe/devel/github_felipesanches/scope-one/src/fonttools/Lib/fontTools/ttLib/tables/_g_l_y_f.py", line 195, in __len__
    assert len(self.glyphOrder) == len(self.glyphs)
AssertionError

I just figured out that this is triggered by the hotfix of the following check:
"Font contains glyphs for whitespace characters?"

Forget hotfixing, I don't use them, I report problem upstream
On 28 June 2016 at 14:58, Felipe Corrêa da Silva Sanches <
<EMAIL_ADDRESS>wrote:

I just figured out that this is triggered by the hotfix of the following
check:
"Font contains glyphs for whitespace characters?"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/816#issuecomment-229148253,
or mute the thread
https://github.com/notifications/unsubscribe/AAP9y1ehKgCHJScmmuy3rqVG-SLhogvdks5qQW7ugaJpZM4I_n-f
.

--
Cheers
Dave

ok, good

MergeOne is temporarily blacklisted in the "Font contains glyphs for whitespace characters?" check.

fixed at 05e6ee82ef7d0f48fca1c169cf808c72c5bae480


@conda-forge-admin, please add user @borchero

Additional comment:
Would love to be able to merge PRs in this repo to get access to new versions as early as possible 🙏🏼

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @borchero in conda-forge/sqlfluff-feedstock#32.


Rigado

Thank you for contributing to the Azure IoT Plug and Play Models repository
This project has adopted the Microsoft Open Source Code of Conduct. For more information see the Code of Conduct FAQ or contact<EMAIL_ADDRESS>with any additional questions or comments.
The repository pipeline will ensure minimum validation of incoming DTDL models.

PR validation steps are described in the tools Wiki.
The same validation can be run locally via the dmr-client CLI.

Requested info template for model(s) submission
When submitting models to the repository we ask that you provide as much of the following meta information around your models and related devices as possible. This info will be used to improve Plug and Play.
:star2: Please replace the markdown comment examples with your own values.
Company Info

PnP Device Info

Model Submission Goals

Code Owners & Reserved Names

Have not yet processed certification on these models. and they are not used anywhere yet. can we update them?
Thanks


Implement SnowflakeReader

Discussion on #257 has lead me to to try and implement a SnowflakeReader using Snowflake's dbapi instead of SQLAlchemy.
Here's what I have so far:
import re

from snowflake.connector import connect

from recap.types import BoolType, BytesType, FloatType, IntType, StringType, StructType

class SnowflakeReader:
    def __init__(self, **kwargs):
        self.con_params = kwargs

    def struct(self, table: str) -> StructType:
        """
        Return a StructType for the given table.
        """
        fields = []

        with connect(**self.con_params) as con, con.cursor() as cur:
            cur.execute(f"DESCRIBE TABLE {table}")
            for row in cur:
                col_type = row[1]
                params = self._parse_parameters(col_type)
                col_type = col_type.split("(")[0]

                if col_type in {
                    "NUMBER",
                    "DECIMAL",
                    "NUMERIC",
                    "INT",
                    "INTEGER",
                    "BIGINT",
                    "SMALLINT",
                    "TINYINT",
                    "BYTEINT",
                }:
                    col_type = BytesType(
                        logical="build.recap.Decimal",
                        bytes_=32,
                        variable=False,
                        precision=params[0] if params else 38,
                        scale=params[1] if params else 0,
                    )
                elif col_type in {
                    "FLOAT",
                    "FLOAT4",
                    "FLOAT8",
                    "DOUBLE",
                    "DOUBLE PRECISION",
                    "REAL",
                }:
                    col_type = FloatType(bits=64)
                elif col_type in {
                    "VARCHAR",
                    "STRING",
                    "TEXT",
                    "NVARCHAR",
                    "NVARCHAR2",
                    "CHAR VARYING",
                    "NCHAR VARYING",
                }:
                    col_type = StringType(bytes_=int(params[0]) if params else 16_777_216)
                elif col_type in {"CHAR", "CHARACTER", "NCHAR"}:
                    col_type = StringType(bytes_=int(params[0]) if params else 1)
                elif col_type in {"BINARY", "VARBINARY"}:
                    col_type = BytesType(bytes_=int(params[0]) if params else 8_388_608)
                elif col_type == "BOOLEAN":
                    col_type = BoolType()
                elif col_type == "DATE":
                    col_type = IntType(bits=32, logical="build.recap.Date", unit="day")
                elif col_type == "TIME":
                    unit = self._get_time_unit(params)
                    col_type = IntType(bits=32, logical="build.recap.Time", unit=unit)
                elif col_type in {
                    "DATETIME",
                    "TIMESTAMP",
                    "TIMESTAMP_NTZ",
                    "TIMESTAMPNTZ",
                    "TIMESTAMP WITHOUT TIME ZONE",
                }:
                    unit = self._get_time_unit(params)
                    col_type = IntType(
                        bits=64,
                        logical="build.recap.Timestamp",
                        unit=unit,
                    )

                fields.append(col_type)

        return StructType(fields)

    def _parse_parameters(self, col_type: str) -> list[str] | None:
        """
        Parse types that have parameters.
        """
        match = re.search(r"\((.*?)\)", col_type)
        if match:
            return [p.strip() for p in match.group(1).split(",")]
        return None

    def _get_time_unit(self, params: list[str] | None) -> str:
        match params:
            case [unit, _] if int(unit) == 0:
                return "second"
            case [unit, _] if int(unit) <= 3:
                return "millisecond"
            case [unit, _] if int(unit) <= 6:
                return "microsecond"
            case _:
                return "nanosecond"

It supports nearly all of Snowflake's types. I opted not to deal with TIMESTAMP_TZ and TIMESTAMP_LTZ at the moment.
I've come across another issue, though. Snowflake has a VARIANT type, which is basically "Any" in Python, "Object" in Java, etc. Recap doesn't currently support this. The VARIANT type is important for Snowflake because both the OBJECT and LIST types use it (OBJECTs have a VARCHAR key and a VARIANT value to emulate JSON objects, and LIST items are all VARIANTS).
So, question is how to deal with this? Three potential ideas come to mind:

Add an any type to Recap's type system.
Change list's (and map's) values attribute from required to optional. An unset values would be treated as a a variant type.
Do nothing. This would prevent OBJECT and ARRAY support in SnowflakeReader.

I'm partial to (2). It solves the immediate problem without adding the baggage of an any type. I really don't want to overcomplicate Recap's type system. I feel like an any type would have a lot of  unintended consequences that I haven't thought through right now.
@cpard @adrianisk Looking for your thoughts/feedback.

Sorry I missed this somehow... I agree that adding an any type seems like overkill, and could cause problems down the line. Option 2 seems like the best idea, as it also fits the Postgres JSON types which I believe can only store arrays/objects, not primitives.
The potential problem I see with option 2 for Snowflake is that VARIANT really is an any type, there's nothing preventing someone from creating a VARIANT column and only storing strings in it, or even a mix of types. Going from Recap array/map -> VARIANT makes sense, but going from Snowflake VARIANT -> Recap still won't have a type that 100% makes sense. That being said, in practice I've really only seen VARIANT columns being used to store objects & arrays, so that's probably fine.

Actually I was wrong, took a closer look at the documentation I linked, and you can store primitives in Postgres' json type
-- Simple scalar/primitive value
-- Primitive values can be numbers, quoted strings, true, false, or null
SELECT '5'::json;

VARIANT type of snowflake is an interesting one.
I've primarily seen it used as part of ingestion pipelines during the "staging" of the data. You can see an example of how this works by checking the Airbyte connector for Snowflake. All data is inserted into a table with a column of type VARIANT, after this has happened, a DBT model is automatically generated based on schema information that Airbyte has and it gets applied to the VARIANT column, generating new tables and columns that the user can actually use.
The main benefit of having a VARIANT type is that your pipelines will not break because of schema conflicts. Obviously you are just pushing the problem down further the pipeline stages because at some point you will have to figure out how to deal with the schema changes. BUT, the ingestion process does not break and in some cases like the ones Airbyte deals with, moderate to low volume but expensive extraction, it does make sense to ensure that data gets delivered to the warehouse and the let data engineer deals with the issue later on.
Having said that, VARIANT makes sense for Snowflake but I don't believe that it makes sense for a system like Recap. Being expressive is good but becoming too expressive turns you into something that nobody understands. My recommendation would be to keep the VARIANT type out of Recap's type system for now.

Cool, so we're agreed on not supporting VARIANT in Recap.
I'm inclined to punt on this for now. This will cause VARIANT to fail for now, which I think is OK for the time being.
Another thought occurred to me: just treat VARIANT as a string no matter what. That will work reasonably well for mixed primitives in a variant col (e.g. "foo", 1, true, 3.0) but not for complex types. Again, I think that's fine.
I'm still going to punt on this for now, but leaving it as a note. I'll open a separate GH issue for VARIANT.


Bug: Config $userProvider is ignored

PHP Version
8.1.13
CodeIgniter4 Version
4.2.9
Shield Version
v1.0.0-beta.3
Which operating systems have you tested for this bug?
Linux
Which server did you use?
apache
Database
8.0.31
Did you customize Shield?
No
What happened?
I created a model and named it UserModel.php and placed it under the /app/Models/ directory. I wanted to use this model for dealing with non auth transaction like saving user first name and last name or displaying users on a list.
I was thinking of creating another model to be used for Shield and call it AuthModel.php.
I left the default value for the  $userProvider in the /app/Config/Auth.php to CodeIgniter\Shield\Models\UserModel;
When the creating a new user using the register page this error occurs:
TypeError
CodeIgniter\Shield\Controllers\RegisterController::getUserProvider(): Return value must be of type CodeIgniter\Shield\Models\UserModel, App\Models\UserModel returned

Steps to Reproduce
Register for a new account
Expected Output
I am not sure if my thinking is correct but I expected Shield to still use the CodeIgniter\Shield\Models\UserModel which is the value of the $userProvider in the config  and not the App\Models\UserModel
Anything else?
No response

I haven't checked this but I'm sure you haven't considered section user-providers.

If you think you have done the steps correctly, please upload the minimum code so I can check.

An example of correct operation in case of setting public string $userProvider = \App\Models\UserModel::class;.

This is not a bug. It is the specification of CI4.
Read https://codeigniter4.github.io/CodeIgniter4/concepts/factories.html#example

Thank you for the replies!


feat: Added users system table

Codecov Report

Merging #498 (cabdeae) into main (5593ad6) will increase coverage by 0.32%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #498      +/-   ##
==========================================
+ Coverage   29.57%   29.89%   +0.32%
==========================================
  Files          79       80       +1
  Lines        9594     9669      +75
==========================================
+ Hits         2837     2891      +54
- Misses       6496     6509      +13
- Partials      261      269       +8

Impacted Files
Coverage Δ

server/metadata/dictionary.go
51.63% <ø> (ø)

server/metadata/key_generator.go
11.90% <0.00%> (ø)

server/metadata/schema.go
68.85% <ø> (ø)

server/metadata/subscope_registry.go
50.00% <50.00%> (ø)

server/metadata/users.go
67.60% <67.60%> (ø)

server/metadata/key_encoder.go
45.76% <70.00%> (ø)

server/metadata/tenant.go
34.90% <83.33%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


CustomPainter inside Stack goes to zero

Imagine you have a Stack inside a Container with a fixed height such that the Stacks incoming constraints are tight height, but bounded loose width. The stack will give its non-positioned children the same constraints.
If you then put a CustomPainter inside the stack, it'll go to zero width, which is weird and there's no obvious way to debug it.
Should we maybe make all the objects that go to smallest size (constrain size.zero) instead go to biggest size unless the incoming constraint is unbounded?
We could have BoxConstraints have a computeDefaultSize function that returns a Size that tends to zero for unbounded axes and tends to infinity for bounded axes. If we make every render box use it, it would be something you would come to expect throughout the platform. (It's what we already do for Flex.)

Looks like RenderCustomPaint inherits this behavior from RenderProxyBox, which inherits it from RenderBox's default performResize.

This is probably too late to change. Closing.


[Enhancement] in SQL query, treat -- inside quotes not as comment

behavior now
--within quotes is also treated as comment. This brings unexpected truncation of query.

This happens when store Pytest case results in Clickhouse and then query. In Pytest, when parametrize cases with sets of values [('name1', 'id1', 'score1'), ('name2', 'id2', 'score2'), ('name3', '', 'score3')], case name will become like name1-id1-score1. Say if id is omitted for the 3rd case, case name then becomes name3--score3, then in query:

when query: SELECT * FROM my_table WHERE case='name3--score3'
then generated SQL: SELECT * FROM my_db.my_table WHERE case='name3

desired behavior

when query: SELECT * FROM my_table WHERE case='name3--score3'-- some comment
then generated SQL: SELECT * FROM my_db.my_table WHERE case='name3--score3'

proposed change
https://github.com/delphid/clickhouse-grafana/blob/commentRe/src/scanner.ts#L304

demo
const comment_reg = '--(([^\"\']*[\"\']){2})*[^\"\']*[$\n]';
const query = 'query content"--actually not comment" \'--also not comment\'--some comment"comment in quotes" \'also comment\'\na new line of query';

console.log(
    query.replace(new RegExp(comment_reg, 'g'), '')
);

// result: query content"--actually not comment" '--also not comment'a new line of query

If that seems alright I'd like to submit a PR

Simplified use case: see the behavior now part in Pytest related contents.
My real use case looks like this:
select timestamp, suite, case, duration, result, log, failure from my_db.my_table
where
  timestamp >= $from and timestamp <= $to
  and business in ($business)
  and system in ($system)
  and suite in ($suite)
  and case in ($case)
  and result in ($result)
  and env='$env'
order by timestamp desc
limit $limit

It happens in the and case in ($case) part. When ($case) is something like ('aaa-5-mtdnn_xxx', 'bbb-10-bert_xxx', 'ccc--bert_yyy')
It's a Pytest feature to create name for parametrized case by connecting parameter values with -. When one parameter is omitted, will have '-' + '' + '-' -> '--'
cases are like:

tokens
branch_id
model_tag

aaa
5
mtdnn_xxx

bbb
10
bert_xxx

ccc

bert_yyy

Thanks. I'll fill some test cases and run

What can be published for this special character? It seems that the clickHouse plugin has not been published for a long time


Caliper can not have multiple orderer definitions in a network configuration file

Context

Caliper can not have multiplemultiple orderer definitions in a network configuration file, or it will report a issue that the hosturl of the orderer is not defined when it is trying to create a channel.  The issue only is found on Caliper Composer but did not test on other adapters yet.
In the channels section of composer.json
{
  ...
  "channels": {
    "mychannel": {
      "config": "/etc/hyperledger/configtx/mychannel.tx",
      "mspconfig"<EMAIL_ADDRESS>      "orderers": [
        "orderer0.example.com",
        "orderer1.example.com",
        "orderer2.example.com"
      ],
      "peers": [
        "peer0.org1.example.com",
        "peer0.org2.example.com"
      ]
    }
  }
  ...
}

The orderers in channels section is a JSON array but only can have one single orderer defined there. it is confused to who want to apply Caplier to a network with a cluster of orderers.
Expected Behavior

Successfully create a channel across all peers and orderers as that is defined in network configure file (composer.json).
Actual Behavior

Failed when it try to create a channel and mention that hosturl of orderer is not defined, which is actually defined in orderers section
Possible Fix

With a quick investigation of the issue by myself, I refer the problem file below.
https://github.com/hyperledger/caliper/blob/27ec5f94e70a4cb3856f2b1dfd26b3e2f95b087f/packages/caliper-composer/lib/composer_utils.js#L68-L102
Take a look at line 85 above, it might include potential problems somehow. This is the direct place which causes the issue.
Steps to Reproduce

Add one more orderer record in the channel definition in the network configuration file (composer.json)
Re-run the benchmark, it will show the node.js issue

Your Environment

Version used:  Caliper with the commits on Jun 22, 2019 (the latest version when the issue is found)
Environment name and version: node.js (8.16.0)
Operating System and version (desktop or mobile): Linux 5.1.14-arch1-1-ARCH

@Jamesxu182 Does it work if you change the line to this:
let orderer = config.composer.network.orderers[channel.orderers[0]].hosturl;

Note, that Composer support is currently limited, and improving it is not a high priority right now. But every fix and enhancement is welcome of course!


ARROW-6506: [C++] Fix validation of ExtensionArray with struct storage type

https://issues.apache.org/jira/browse/ARROW-6506

@pitrou on apveyor their is a segfault during the python csv tests (which does not seem related to this PR). Is that a known failure?

It's the first time I see that. I'm gonna retry the build just in case.

@jorisvandenbossche Can you fix the C++ lint failure on Travis? The Java failure looks unrelated (I got it on another PR)

Can you fix the C++ lint failure on Travis?

I need to make it a habit to run those locally .. (or configure my editor to do that)

It will soon become muscle memory :-)

I run the following bash function every time before I git commit
function arrow_preflight {
    ARROW_PREFLIGHT_DIR=$ARROW_ROOT/cpp/preflight-build
    mkdir -p $ARROW_PREFLIGHT_DIR
    pushd $ARROW_ROOT/cpp
    python build-support/lint_cpp_cli.py src || return

    pushd apidoc
    doxygen || return
    popd

    popd
    pushd $ARROW_PREFLIGHT_DIR
    arrow_cmake
    ninja format
    ninja lint || return
    popd
    pushd $ARROW_ROOT
    ./run-cmake-format.py
    popd
    flake8 $ARROW_ROOT/dev || return
    pushd $ARROW_ROOT/python
    flake8 --count pyarrow || return
    flake8 --count --config=.flake8.cython pyarrow || return
    popd
}

There were flakes on Joris's Travis CI so I'll just wait for Travis to run on here later today

The remaining travis failure seems to be java related.

Thank you @jorisvandenbossche !


yarn upgrade in website

Motivation

GitHub's Security Alerts were yelling at us.
This only fixes some of the problems. Two of them have been reported and not
fixed upstream yet.
Test plan

See included automated tests.

why does the site have so many dependencies :-(

Welcome to JavaScript.


TH 🍋

This is a tricky one sorta since lemon is เลมอน, and while มะนาว is lime, it's often used as a stand-in for lemon as the distinction between the two isn't really made often. I think I could compare it to me saying basil is basil, but Thais always make the distinction between Thai basil and holy basil (which doesn't include the sweet basil); I used to use Thai basil when cooking in the US since the Asians store sold it for cheaper than chain basils haha.  ชามะนาว is a lime tea, but is people often translate it to lemon tea. I'm just guessing since lemon tea is much more common in the West that's what the translations bots say and it's all over the signs, but it's definitely lime. I usually try to ask if it's lemon or lime if I order off a menu, and people seem puzzled that I'd even ask. That said, I much prefer lime tea. /story
@turboMaCk

Wow! I don't think I've ever learned so much from a PR before 😀


Fix the display of utf8 characters in the code editor

Work done by @baloone during his internship at OCamlPro. This fixes the display with invalid syntax (display in strings and comments was already OK), but it shouldn't get scrambled either way.
NOTE: should be rebased and squashed once the 4.12 PR is merged

Rebased

Thanks a lot @AltGr ! I was precisely about to ask this to you 🙂

(Let's just wait for the CI though :)

Hello @erikmd, Thanks the merge!
I think using smiley inside string has always worked fine, this fixes the display of Unicode characters outside of comments or string i.e. when the ocaml lexer throws an "illegal character" error.


Chore: Clear network error message

Overview
This PR removes the Monaco webpack plugin and the getWorkersUrl call as they aren't since we use the @monaco-editor/react package. This package does not need us to set up any plugins or modify configuration files for Webpack
Closes #2613

Opted to fix the failing snapshots in this PR as well. I noticed some were outdated.


Update LICENSE.md

LICENSE.md

albertanastasia:patch-1

albertanastasia:patch-1LICENSE.md

.patch

.diff

Yes, I agree.
On Thu, Aug 29, 2024, 3:02 AM github-actions[bot] @.***>
wrote:

CLA Signature Action:
Thank you for your submission, we really appreciate it. Like many
open-source projects, we ask that you read and sign our Contributor
License Agreement https://roblox.github.io/cla-bot-store/ before we can
accept your contribution. You can sign the CLA by just by adding a comment
to this pull request with this exact sentence:
I have read the CLA Document and I hereby sign the CLA
By commenting with the above message you are agreeing to the terms of the
CLA. Your account will be recorded as agreeing to our CLA so you don't need
to sign it again for future contributions to our company's repositories.
0 out of 1 committers have signed the CLA.
❌ @albertanastasia https://github.com/albertanastasia
—
Reply to this email directly, view it on GitHub
https://github.com/Roblox/roblox-blender-plugin/pull/62#issuecomment-2316857461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BK3QRQQZU4C4PGDFMQBDEY3ZT3BPZAVCNFSM6AAAAABNJX6O7KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWHA2TONBWGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


LinkageError when the path matches ...

Take again, as an example, kotlin-fullstack-sample (Thinkter)
Adding tasks to copy jars:
task install(type: Copy) {
    dependsOn build
    from configurations.runtime
    from jar
    into "${project.projectDir}/lib"
}

Execute the task and run the backend:
java -cp "lib/backend-0.0.1-SNAPSHOT.jar;lib/*" org.jetbrains.ktor.jetty.DevelopmentHost`

Full path: D:\kotlin-fullstack-sample\lib
Backend was successfully started.
But if you just change kotlin-fullstack-sample to thinkter and the server does not start:
Exception in thread "main" java.lang.reflect.InvocationTargetException
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)
        at kotlin.reflect.jvm.internal.FunctionCaller$Method.callMethod(FunctionCaller.kt:98)
        at kotlin.reflect.jvm.internal.FunctionCaller$StaticMethod.call(FunctionCaller.kt:108)
        at kotlin.reflect.jvm.internal.KCallableImpl.call(KCallableImpl.kt:107)
        at kotlin.reflect.jvm.internal.KCallableImpl.callDefaultMethod(KCallableImpl.kt:147)
        at kotlin.reflect.jvm.internal.KCallableImpl.callBy(KCallableImpl.kt:111)
        at org.jetbrains.ktor.host.ApplicationLoader.callEntryPointFunction(ApplicationLoader.kt:257)
        at org.jetbrains.ktor.host.ApplicationLoader.instantiateAndConfigure(ApplicationLoader.kt:228)
        at org.jetbrains.ktor.host.ApplicationLoader.instantiateAndConfigureApplication(ApplicationLoader.kt:189)
        at org.jetbrains.ktor.host.ApplicationLoader.createApplication(ApplicationLoader.kt:107)
        at org.jetbrains.ktor.host.ApplicationLoader.getApplication(ApplicationLoader.kt:60)
        at org.jetbrains.ktor.host.ApplicationLifecycle$DefaultImpls.ensureApplication(ApplicationLifecycle.kt:22)
        at org.jetbrains.ktor.host.ApplicationLoader.ensureApplication(ApplicationLoader.kt:22)
        at org.jetbrains.ktor.jetty.JettyApplicationHost.start(JettyApplicationHost.kt:164)
        at org.jetbrains.ktor.jetty.JettyApplicationHost.start(JettyApplicationHost.kt:27)
        at org.jetbrains.ktor.host.ApplicationHostStartable$DefaultImpls.start$default(ApplicationHostStartable.kt:6)
        at org.jetbrains.ktor.jetty.DevelopmentHost.main(CommandLine.kt:9)
Caused by: java.lang.LinkageError: loader constraint violation: when resolving method "org.jetbrains.ktor.application.ApplicationFeatureKt.install$default(Lorg/jetbrains/ktor/pipeline/Pipeline;Lorg/jetbrains/ktor/application/ApplicationFeature;Lkotlin/jvm/functions/Function1;ILjava/lang/Object;)Ljava/lang/Object;" the class loader (instance of org/jetbrains/ktor/host/OverridingClassLoader$ChildURLClassLoader) of the current class, org/jetbrains/demo/thinkter/ApplicationKt, and the class loader (instance of sun/misc/Launcher$AppClassLoader) for the method's defining class, org/jetbrains/ktor/application/ApplicationFeatureKt, have different Class objects for the type kotlin/jvm/functions/Function1 used in the signature
        at org.jetbrains.demo.thinkter.ApplicationKt.main(Application.kt:21)
        ... 20 more

I still can not understand what coincides with thinkter, which leads to an error.
I thought this was related to the main class (Application.kt) package, where there is a thinkter:
package org.jetbrains.demo.thinkter
But using as a directory name, for example, a demo does not lead to an error.

I found a match: project directory and parameter watch.
Disabling automatic reload of the application autoreload = false helps to get around this error.
Since I was creating the application distribution for production, this solution suited me.

Thanks for the tip, had the same issue and indeed getting rid of watch fixed the problem for me too.
How to disable autoreload just for production build, is it possible without creating some custom Gradle tasks? Can a specific developer application configuration file can be defined?
Possible to use environment variables in the application.conf to configure something?

http://ktor.io/servers/configuration.html

Thanks for the pointer, I already saw, I'll build something around this.


[Bug]: Firmware updates time out if the target device is not directly connected to the controller

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Zwavejs2Mqtt version
6.15.2
ZwaveJS version
9.6.2
Describe the bug
I've experienced an issue while attempting to update firmware on multiple Zooz ZEN77 (dimmer switches) and ZSE44 (temperature/humidity sensor) devices, but only when they are not connected directly to the controller (these devices are connected via a "1 hop" connection -- that is, there is one neighbor node between the device and the controller).
The devices start a firmware update as would be expected, albeit slower than you might expect, but eventually, the firmware update times out without an error being presented in the UI, and the "FW" column within the Z-Wave JS list display reverts from a progress bar to the FW/SDK versions without any error showing either in the UI or in the logs. The only "error" I see is a "firmware update finished" event in the "events" tab of the device with a "-1" argument.
I've been experiencing this issue ever since I found new firmware updates for my devices on July 30. Over the past month, I've updated zwavejs2mqtt multiple times, healed my network to try and get the devices a better connection to the controller, and ultimately upgraded from a Aeotec Z-Stick Gen5+ to a Zooz 700-series Z-Wave USB stick, none of which helped fix the issue directly.
That being said, after upgrading to a Zooz 700-series Z-Wave USB stick from the Aeotec 500-series controller, I did notice that, after healing my network, many more of the devices on the network were able to communicate directly with the controller. When the devices in question were connected to my 500-series Z-Wave stick via a 1-hop connection, their firmware updates failed, but after upgrading to the Zooz 700-series controller and healing the network, all but 3 devices in my network were connected directly to the controller.
After this upgrade, every single one of the devices that previously had a firmware update fail on the 500-series controller (while connected via a "1 hop" connection) that were now directly connected to the controller were able to complete the firmware update expeditiously and without issue. However, the two Zooz devices (out of the three total nodes that are still connected on a "1 hop" connection) that are still not directly connected to the controller are still experiencing the same behavior, where the firmware update fails partway through.
For reference, I've carried out the same update procedures on every device (whether battery or mains-powered), and haven't seen any change in behavior unless the connection to the node swaps from a "1 hop" connection to a direct connection.
While attempting to export debug logs, I did not see any logs produced from the firmware update, but I did see "events" showing within the device in the UI pertaining to the firmware update. I've attached a snippet of the last few events when the firmware update was cancelled. For reference, node 11 is the one depicted in the image, but I've also experienced an issue with updating node 43, as it is also 1 hop away from the controller.
Node dump:
nodes_dump.zip
Node events:

Appreciate any assistance that you all able to provide! Please let me know if any more information would be helpful in debugging this issue. Thanks!
To Reproduce

Connect a device to the controller in a manner where it connects with a "1 hop" connection -- meaning one and only one neighbor between the device and the controller.
Attempt a firmware update on the device by selecting the "begin firmware update" option within the Z-Wave JS UI, and select a valid firmware file for the device.
The firmware update begins and the device starts reporting progress.
The firmware update cancels without error after progressing anywhere between 3-30% through the firmware update (in my experience).

Expected behavior
A device should successfully be able to complete a firmware update regardless of how it is connected to the controller.
Additional context
I've experienced this issue so far with Zooz devices -- I have other devices on my network, but none have needed firmware updates so far. I've corresponded with Zooz support over the past month in order to try and figure out this issue in the event that it was a hardware/firmware issue with the devices, but I have not yet been able to ascertain any other cause for failed firmware updates aside from the devices only failing to update whenever they are not directly connected to the controller.

Hi, seems like you've done quite a research here, but this issue should be filed with the driver node-zwave-js.
I think @AlCalzone can move it within the organization.
You should also produce a debug log for zwave-js (not for zwavejs2mqtt). I didnt download your nodes.zip, so it might be you already have it in there?


test_processing unit tests should check its results

The test_processing unit tests presently generates plots and csv files and successfully checks that these are in place. However, the contents of the csv file (the result of the processing) is not checked.
Adding an explicit check to the results of the processing would improve the test substantially.

line 120-130 of test_process checks the contents of the STATS file

Ah, then I missed that! Then this can probably be closed.


How to configure azkaban to send emails via postfix?

Hi,
I have installed and configured postfix on my hosted ubuntu server to be able to send notification emails to my personal (@outlook.com) account.
This is the content of my /etc/postfix/main.cf
mydomain = o365weve.com
smtpd_banner = $myhostname ESMTP $mail_name (Ubuntu)
biff = no
append_dot_mydomain = no
readme_directory = no
smtpd_tls_cert_file=/etc/ssl/certs/ssl-cert-snakeoil.pem
smtpd_tls_key_file=/etc/ssl/private/ssl-cert-snakeoil.key
smtpd_use_tls=yes
smtpd_tls_session_cache_database = btree:${data_directory}/smtpd_scache
smtp_tls_session_cache_database = btree:${data_directory}/smtp_scache
smtp_generic_maps = hash:/etc/postfix/generic
smtpd_relay_restrictions = permit_mynetworks permit_sasl_authenticated defer_unauth_destination
myhostname = o365weve.com
alias_maps = hash:/etc/aliases
alias_database = hash:/etc/aliases
myorigin = o365weve.com
mydestination = o365weve.com, WeveJobs01.o365weveclusterjobs.c1.internal.cloudapp.net, localhost.o365weveclusterjobs.c1.internal.cloudapp.net, localhost
relayhost =
mynetworks = <IP_ADDRESS>/8 [::ffff:<IP_ADDRESS>]/104 [::1]/128
mailbox_size_limit = 0
recipient_delimiter = +
inet_interfaces = localhost
inet_protocols = all
These are the contents of /etc/postfix/generic
@WeveJobs01.o365weveclusterjobs.c1.internal.cloudapp.net<EMAIL_ADDRESS>
These are the settings in azkaban.properties
mail settings
<EMAIL_ADDRESS>mail.host=o365weve.com
<EMAIL_ADDRESS><EMAIL_ADDRESS>
When I send emails directly using mail command, I get the emails correctly
echo "This is the body of the email" | mail -s "This is the subject line"<EMAIL_ADDRESS>But I dont get any emails from azkaban when a job succeeds or fails.
What am i doing wrong?

I was able to figure this out by wading through internet pages and reading the source code on github.
A better documentation is a must for this feature.
Here is what I did to make it work.

Configure postfix to server as a hotmail/outlook smtp relay.
This is what I added to my /etc/postfix/main.cf file to use outlook.com as relay

SASL parameters
smtp_use_tls=yes
smtp_sasl_auth_enable = yes
smtp_sasl_password_maps = hash:/etc/postfix/sasl_passwd
smtp_tls_CAfile = /etc/ssl/certs/ca-certificates.crt
smtp_sasl_security_options =

The /etc/postfix/sasl_passwd file contains user name and password like this
[smtp.live.com]:587  username@outlook.com:Password

Next run postmap and re-start posffix
postmap /etc/postfix/sasl_passwd
service postfix restart

These are the changes I made to conf/azkanban.properties to make it work
<EMAIL_ADDRESS><EMAIL_ADDRESS>mail.password=Password
mail.host=smtp.live.com
mail.tls=true
<EMAIL_ADDRESS><EMAIL_ADDRESS>jetty.hostname=dnsname.cloudapp.net

The canonical way to do this is "you send email to localhost, and whatever MTA is configured on there will then send it where it should go".
MTAs pretty much exist so that every service ever written doesn't need to do anything but talk to localhost.


Read in a range as list?

I am aware of the possibility to override the excel.set_xl_array_conversion with something like excel.set_xl_array_conversion(convert_to_list). But is there a way to leave the default as DataFrame, and just occasionally get a list, or is the idea to convert the DataFrame back to a list to do this?

It's a bit of a hack, but xl() function signature supports **kwargs, so you can pass additional parameters which are then pass to the conversion function, i.e.  xl("A1:B5", headers=True, to_list=True)
Then check that to_list parameter as part of our convert_to_list and implement your logic accordingly.

Tagging as documentation (for when we enable support for editing of the Initialize pane).

Actually, I think overriding the default converter is easier. This is what I came up with:
def myconverter(x, headers=False, convert=None, **kwargs):
    if convert is None or convert == pd.DataFrame:
        return excel.convert_to_dataframe(x, headers=headers, **kwargs)
    elif convert == np.array:
        return np.array(x)
    elif convert == list:
        return x

excel.set_xl_array_conversion(myconverter)

This keeps the default unchanged, but allows for these:
xl(..., convert=np.array)
xl(..., convert=list)

As long as the initialize pane isn't editable, this can be placed in A1 of the first sheet.


Adding dedup on images already assessed

Skipping images with an evaluation already available in the time windows specified.

Logic LGTM, but two suggestions:

Let's make this optional (CLI flag), and turned off by default.  Reasoning is container vulns endpoint is extremely verbose and even small lacework accounts can sometimes hit 100 pages (maximum pagination) of data returned for a 24 hour query.  This takes a long time to paginate through (5-10 minutes) and may be an incomplete dataset. Don't want to negatively affect existing users.

Can you add some documentation to the readme regarding the above mode and how it works?  Also some word on how it compares to using the local cache.


On the Admin Service Learning page: get the Email All button working and move it to the top right of the table, across from the search bar.

PR merged, closing issue.


diffbrowsers: render a html page for user wordlists

If users include use the --user-wordlist arg, diffbrowsers will now output a test html page containing the user words.
cc @vv-monsalve

Thanks! I've just tested this and is working to produce the diffbrowsers_user_strings.html with the provided wordlist.
Could the text be displayed as a paragraph, like in text.html?  This would be necessary to check for the font's default line spacing for languages with many marks or complex scripts.


Move docs on HOCON config from docs/ to comments in reference.conf

That's where it'd be more expected to be found, from what I've seen across the libs in Scala ecosystem

improve existing comments inside reference.conf - especially for the workspace.path.cause

Partly already done on #252 right? in fact, just the part about removing the docs from docs/ can be considered (to remove duplication)

Yes

Note: examples from https://github.com/VirtusLab/ide-probe/blob/master/docs/custom-resolvers.md are not valid (outdated?). They contain invalid paths like probe.resolvers.intellij.repository

could be done together with https://github.com/VirtusLab/ide-probe/issues/262 and https://github.com/VirtusLab/ide-probe/issues/265 - all three issues are for reference.conf comments


Help wanted: why does the README not appear

For other charts, READMEs appear, but not for Argo, e.g.:
https://hub.helm.sh/charts/argo/argo-cd/0.5.2
Here is the template:
https://github.com/argoproj/argo-helm/tree/master/charts/argo-cd

Hi @alexec, I checked that issue with Kubeapps (in which the issue is also present). It seems a problem with the way the tarball is packaged. The service extracting the README is failing to do so. When I try to extract the tarballs for argo I get several warnings which doesn't appear for other charts:
▶ tar zxf argo-0.5.2.tgz
tar: Ignoring unknown extended header keyword 'LIBARCHIVE.creationtime'
tar: Ignoring unknown extended header keyword 'SCHILY.dev'
tar: Ignoring unknown extended header keyword 'SCHILY.ino'
...

Maybe there is an issue with the packaging procedure?

@alexec I concur with @andresmgot on this. The Helm Hub extracts the README.md from the archive. It's not able to do that which is why it can't display.
On Mac I cannot reproduce what he found but I can on Linux.
I suspect the problem is with the publish.sh script. Instead of using tar to create the archive might I suggest using the helm package command.

OK. I've updated the script:
https://github.com/argoproj/argo-helm/blob/master/scripts/publish.sh#L11

Hmm.. still does not work?
Version 0.5.4 should have README.
https://hub.helm.sh/charts/argo/argo-cd

I'm still seeing the unknown extended header error when extracting on Linux... looks like this might be an issue when creating tar archives on macOS: http://lifeonubuntu.com/tar-errors-ignoring-unknown-extended-header-keyword/.
What platform does your packaging script run on?
Based on that article, Monocular/Kubeapps should just ignore the warnings and extract the README anyway.

Mac. Run on Docker maybe?

Could error be red herring?

ah, I think I've found the issue. The problem is not related to the headers warning. The issue is that your tarball doesn't contain a folder (all the files are in the root). It's expected to have a folder with all the files within it. Instead of ./README.md, it should be located at argo/README.md.

Note to self: 0.1.4 should be updated https://hub.helm.sh/charts/argo/argo-ci

On Linux I use the helm package command.The README not appear.

I think that your case @liwenhe1993 is different because I am able to see the README in https://hub.kubeapps.com/charts/liwenhe1993/clickhouse (so I am not sure why it's failing for the helm hub).


Facilitation, 12th June meeting (Q2)

Facilitation and any needed agenda tweaking for meeting on 10th June 2024
Done by LeeLoo
See agenda here https://docs.google.com/document/d/1s7UfdRsw2CuZ1FQ_cse73lEfu-Bc2gViEeAP8jpr5lk/edit?usp=sharing

@VanCardui who facilitated this meeting?

Like it says "Done by LeeLoo"
I've just asked her if she wants to join this GitHub organisation, and if she is on Dework. She might prefer not to - and, since she is employed by the Foundation, she might prefer not to also be in the Ambassador Program treasury system as well. In which case, I guess we would just close this issue unpaid..?

later: from LeeLoo via DM:
"Please put the money back in treasury. Let me know if there needs to be an official letter of denial:) "

Alright, thanks


Doc: pooch install for xarray dataset

As specified here

Dear @Ishaanj18 ,
I am writing to express my frustration regarding this pull request.  I made an effort to start the other PR, and observingClouds suggested adding this to the docs. I wanted to implement that, only to find that you had already made those changes.
While I understand that you are keen to contribute, and take the initiative, but not like this. Similar behavior can make it for starters like me very difficult to feel motivated to contribute.
I believe that collaborative work should involve clear communication and mutual respect.

Really sorry for the inconvenience that I have caused, even I am new to open source contribution. It has been pointed out to me now that the cooldown period for open source is 2 weeks. I made the request as there had been no updates/changes made on the prior PR and had been marked as closed. I will keep this in mind in future.
Looking forward to collaborating with you.
Regards

Hi @Ishaanj18,
No worries, I understand. The issue was not yet stale, but I appreciate your apology. I'm looking forward to collaborating with you too.
Thanks


The prune model cannot reimplemented

I downloaded your pruned model and identified the model achieved delta1 >= 0.77.
I implemented the pruned model and trained it from the pretrained model of imagenet.
But its accuracy is around 0.6.
lr = 0.01,
weight decay = 0.0001
SGD with a momentum 0.9
=> That's is I didn't change these parameters at all.
Could you change those parameters while training using NetAdapt?

Hi @songya
In our work, we first trained our (unpruned) model and then applied network pruning to it, aiming to maintain accuracy. We fine-tuned the pruned model but did not train it from scratch. If you are training the pruned model from scratch, the achieved accuracy may very well differ from our reported metrics as we had not tried that.
Yes, it is possible to change parameters when training or fine-tuning using NetAdapt. For more detail, please refer to the NetAdapt code repo at https://github.com/denru01/netadapt, especially the customization section.


not able to run the test pipeline

Hi,
After creating the environment and pip install everything as described in the readme, I get the following error when I call the function run_microMatch_test():

complete error file:
error.txt
Does anyone get the same error or knows how to fix this?

Hello,
Sorry for the late reply to this. This is a run-time error in the neural network (I assume during training). You might try playing with the parameters of the network (learning rate etc) and also the ls_regularizer term in line 17 of /main/micro_match/preprocessing/deep_functional_maps/operations.py


Improve AveragePrecisionResult

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (d321b78) to head (a25d833).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #98   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           35        35
  Lines          823       827    +4
  Branches        87        88    +1
=========================================
+ Hits           823       827    +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Bug Report #5

Case #5
Reported By: Humud Abdalla
Date Reported: March 15, 2023
Program name: VirtualRestaurantAssistant
Release number: Iteration 2
Configurations: Mac OS 64-bit
Report type: code issue
Can reproduce: yes
Severity: High
Priority:
Problem summary: As a shop owner i am not able to see the orders that were placed through out the day
Problem description: I was not able to see the orders that were placed through out the day as it was described in the user story for iteration 2:
Suggested fix: Implement a frame in which where a shop owner can see the orders that were placed throughout the day
Status: Open
Resolution:
Resolved by:

Thank you for reporting, Humud. We have prioritized this issue as High and I have identified and added the feature.
We have implemented a panel to switch to and from Inventory display and Manager Sales History for shop owners to view the orders placed over time, as you have suggested in the User story for Iteration 2. You can find the solution in the pull request #41 . Feature is also in the latest build of main, ready to pull.
Thank you again for bringing this report, and please let us know if this fits as per your request. Any feedback is appreciated. You can also see an attached screenshot of the feature.

Ali Amin


Issue 1769 - part 1 - Adding of handling for AggregateExceptions

Added handling of AggregateException exceptions generated in agent code.
Currently AggregateException exceptions are displayed alike "System.AggregateException: One or more errors occurred". The changes correct the layout to following:
One or several exceptions have been occurred.
<text of exception 1>
<text of exception 2>
Fix wasn't tested manually.

Using of IF-ELSE statements inside of CATCH were replaced with using of several CATCH to handle different types of exceptions.


fix: upgrade to version 3.1.138

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign garethjevans
You can assign the PR to them by writing /assign @garethjevans in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign garethjevans
You can assign the PR to them by writing /assign @garethjevans in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test kube

vault
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test kube

vault
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test kube

vault
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test vault

gsm
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test kube

vault
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test vault

gsm
e68f26a626f5f8876ff54136facb03b4a305d766
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest

/retest


Add Fish shell completions and setup

Adds argument and subcommand completions for the Fish shell. I didn't add fzf support because I'm personally happy with the "basic" completions.
Closes #99

merged


refactor

Any change to the endpoint spec relative to the local cluster/spoke should contain within itself. Do not use the global variable

/retest

/cherrypick release-2.10