harssi_present Where is the value set?

I understand that you set the Input_number with a name but how do I adjust what the present and not present values are? Is this done with the input_number value?

This is a very nice project. I am using 2 of these. One is working great but the second one does not stay connected.

I understand that you set the Input_number with a name but how do I adjust what the present and not present values are? Is this done with the input_number value?

You have to create the input number helpers in your home assistant instance.

Great thanks! I did that now.
I am still having issues with my second sensor. I tried a ESP32 Dev and now an ESP32 mini, same issue. It becomes unavailable in HA most of the time.
Here is my log.
Should it be running the script timeout every few seconds even when I am next to watch?
[D][script:020]: Script 'presence_timeout' restarting (mode: restart) [D][api.connection:735]: Home Assistant 2021.9.6 (<IP_ADDRESS>): Connected successfully [D][ble_adv:072]: Found Apple Watch (mac 77:57:A4:72:A4:DD) rssi -51 [D][ble_adv:072]: Found Apple Watch (mac 77:57:A4:72:A4:DD) rssi -50 [D][ble_adv:072]: Found Apple Watch (mac 77:57:A4:72:A4:DD) rssi -50 [D][ble_adv:072]: Found Apple Watch (mac 77:57:A4:72:A4:DD) rssi -50 [D][sensor:118]: 'RandyB Apple Watch masterbath RSSI': Sending state -50.43729 dBm with 0 decimals of accuracy [D][sensor:118]: 'room_presence_debounce': Sending state 1.00000  with 1 decimals of accuracy [D][script:020]: Script 'presence_timeout' restarting (mode: restart) [D][ble_adv:072]: Found Apple Watch (mac 77:57:A4:72:A4:DD) rssi -50 [D][ble_adv:072]: Found Apple Watch (mac 77:57:A4:72:A4:DD) rssi -50

I removed the MQTT config and it seems to have helped the issues.
Thanks! Randy

Hi @rlust I am glad that you go it sorted. Maybe publishing events to MQTT was overloading the ESP?
Regarding

Should it be running the script timeout every few seconds even when I am next to watch?

Yes. What's happening is the timeout scripts is restarted (mode: restart) each time a valid rssi is received. Only after the timeout period (default 30 seconds) is a "not present" indication sent to HA.
For example, my logs show:
[19:39:59][D][sensor:127]: 'Dale Apple Watch bedroom RSSI': Sending state -72.52032 dBm with 0 decimals of accuracy
[19:39:59][D][sensor:127]: 'room_presence_debounce': Sending state 1.00000  with 1 decimals of accuracy
[19:39:59][D][script:020]: Script 'presence_timeout' restarting (mode: restart)
[19:40:00][D][ble_adv:061]: Found Apple Watch (mac 56:FF:87:2B:B1:0C) rssi -74
[19:40:00][D][sensor:127]: 'Dale Apple Watch bedroom RSSI': Sending state -72.96423 dBm with 0 decimals of accuracy
[19:40:00][D][sensor:127]: 'room_presence_debounce': Sending state 1.00000  with 1 decimals of accuracy
[19:40:00][D][script:020]: Script 'presence_timeout' restarting (mode: restart)
[19:40:01][D][ble_adv:061]: Found Apple Watch (mac 56:FF:87:2B:B1:0C) rssi -72
[19:40:01][D][sensor:127]: 'Dale Apple Watch bedroom RSSI': Sending state -72.67496 dBm with 0 decimals of accuracy
[19:40:01][D][sensor:127]: 'room_presence_debounce': Sending state 1.00000  with 1 decimals of accuracy
[19:40:01][D][script:020]: Script 'presence_timeout' restarting (mode: restart)


Unable to publish to gitlab package registry

Since this always assume that the registry is an OCI registry, it doesn't publish to gitlab.
"semantic-release-helm3",
      {
          "chartPath": ".",
          "isChartMuseum": false,
          "registry": "https://gitlab.com/api/v4/projects/12345678/packages/helm/release",
          "skipRegistryLogin": true
      }

and I get this error when i try and publish
An error occurred while running semantic-release: Error: Command failed with exit code 1: helm push /builds/test/devops/charts/test/gustpTest-1.1.1.tgz oci://https://gitlab.com/api/v4/projects/12345678/packages/helm/release
Error: invalid_reference: invalid repository

I am not familiar with what gitlab uses but might have to implement something similar to github
https://github.com/nflaig/semantic-release-helm/blob/b1647baeeab7f3f33f5699126688ce41d1b66ac7/lib/publish.js#L124

So i got this working with the following config
    {
          "chartPath": ".",
          "isChartMuseum": true,
          "registry": "https://gitlab.com/api/v4/projects/1234567/packages/helm/release",
          "skipRegistryLogin": true,
          "cmPushArgs": "-d"
      }

and just doing the helm registry login in my before_step of my pipeline since if it's gitlab you need the whole repo on it, not just gitlab.com

Greetings!
I got the same issue that @jwhitcraft had weeks ago since I'am not be able to publish on my gitlab registry, although I could register on gitlab  on "before_script":
    helm repo add --username ${REGISTRY_USERNAME} --password ${REGISTRY_PASSWORD} semantic-release-helm  ${REGISTRY_HOST}

This is the configuration that I have:
          {
            "chartPath": . ,
            "registry": "${CI_API_V4_URL}/projects/${CI_PROJECT_ID}/packages/helm/${CI_COMMIT_BRANCH}",
            "isChartMuseum": true,
             "skipRegistryLogin": true
          }

And still it doesn't works, this the error that I get
[7:19:25 PM] [semantic-release] › ✘  Failed step "publish" of plugin "semantic-release-helm3"
[7:19:25 PM] [semantic-release] › ✘  An error occurred while running semantic-release: Error: Command failed with exit code 1: helm cm-push . semantic-release-helm
.
.
.
. Error: 413: could not properly parse response JSON: Request Entity Too Large

[7:19:25 PM] [semantic-release] › ✘  Failed step "publish" of plugin "semantic-release-helm3"
[7:19:25 PM] [semantic-release] › ✘  An error occurred while running semantic-release: Error: Command failed with exit code 1: helm cm-push . semantic-release-helm
.
.
.
. Error: 413: could not properly parse response JSON: Request Entity Too Large

This looks like an error from the gitlab server, the payload you are sending is too large

For some reason did everything from the scratch and now it's seems that is working even the version increment,  nevertheless I noticed that although the tag and chart version works fine, the Chart.yaml is not updated in the source main project

the Chart.yaml is not updated in the source main project

Might be a configuration on semantic release itself, version update is independent from any registry, you can check your release logs for this
https://github.com/nflaig/semantic-release-helm/blob/b1647baeeab7f3f33f5699126688ce41d1b66ac7/lib/prepare.js#L12-L13
Note that it also part of prepare step, so if that would fail (e.g. due to file permissions) the publish wouldn't have worked.


MangaPlus chapters inside MangaDex delegation show incorrect dates

Steps to reproduce

Open a mangaplus licensed manga in mangadex
Observe the dates for the latest chapter released under mangaplus group

Expected behavior
The dates correspond to dates seen on mangadex website
Actual behavior
The date is set to end of unix time

Crash logs
No response
Tachiyomi version
528
Android version
13
Device
Samsung Galaxy Note 10 Lite
Other details
No response
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.9.3.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Ah I should have been more clearer
mangadex.org uses the readableAt date value instead of publishAt (Ref: Timestamps in MangaDex)
Neko and Tachiyomi Extension use the same but Sy's MangaPlus dates were never changed

I checked Tachiyomi extensions, and it uses the same as SY, publishAt

Neko does use readableAt though

Ah my bad, its not really required when you're filtering out external urls. Without external urls, there's no difference between readableAt and publishAt but I shall confirm

Will be fixed in the new build.


Add Django 3.0 support for CI (Fix #120)

As I can judge it's the only thing that must be changed.

Coverage remained the same at 100.0% when pulling ed4bf211d213ba240943220fabe01cf47b02e853 on lk-geimfari:master into 48da1cc4c7ed2acd2e80f2b9d840e8ef7939e314 on sobolevn:master.

Well, I have made all the requested changes. Looks like it works fine with Django 3.0.

Thanks a lot!


Why key can validate multiple use ?

when i validate key and success login , after i logout why it can use same key to validate
How can i implement it to expire key now after validate success.
thnak you

See this explanation

The explanation of @RobThree is super good.
You have to take care of that yourself. That's why there is a text in my README that says:

For a secure installation you have to make sure that used codes cannot be reused (replay-attack).


Option.isSome/isNone should be a type guard function in d.ts

We need https://github.com/Microsoft/TypeScript/pull/5906 (TypeScript 1.8) to resolve this.

Fixed by #97

Need to wait the result of #103

Instead, we uses boolean literal type in https://github.com/saneyuki/option-t.js/pull/157


Full screen Button Stops Working After Double Click

The Issue
Full screen button stops working after double clicking it. You have to use the keyboard or double click again to get out of it.
Steps to Reproduce
Double click a video to make full screen. Then try coming out of full screen using the full screen icon in the bottom right.
Expected Behavior
The icon should make the screen size normal again.
Actual Behavior.
Does nothing.
System Configuration
App | 0.31.1
Daemon (lbrynet) | 0.34.0
Windows 10

Thanks for filing and vv181 for finding this! Strange that it works fine if you full screen using the button, or exit full screen when double clicking!

Is this still an issue ?

I can't recreate this issue in the latest version:

App
0.32.2

Daemon (lbrynet)
0.37.1

Also tested with the new player on: https://beta.lbry.tv/

probably a windows issue ?

probably a windows issue ?

I discovered the bug on Arch.
Recently I haven't yet upgraded nor tried the latest LBRY app. Will do it if I've some free time.

Is this still an issue ?

App | 0.32.2
Daemon (lbrynet) | 0.37.1
Platform | Linux (Linux-5.1.4-arch1-1-ARCH-x86_64-with-arch)
I can confirm the issues haven't yet fixed.

Tested on both Windows and Linux the issue still occurs, except on the https://beta.lbry.tv/ its work as expected.

@vv181 Thanks for testing this issue, It's probably a bug on electron but new video player should fix this issue :+1:

Small update on this issue, the old toggleFullscreen was wrong implemented and it was requesting fullscreen mode for the container  instead of the video element.
This should be fixed on this branch: https://github.com/lbryio/lbry-desktop/tree/fullscreen


New version: JetBrains.WebStorm.EAP version 241.9959.35

This pull request is automatically generated by 🥟 Dumplings in #3254
Microsoft Reviewers: Open in CodeFlow

Quite normal for JetBrains packages.

I'll add them to an exception for this check.


fix Disable releases for Apache snapshots repo

What problem does this PR solve?
Problem Summary: while building fe, Maven looks for release artifacts in snapshots repo (and gets HTTP 404).
Repro:
$ mvn -DskipTests clean package
...
Downloading from snapshots: https://repository.apache.org/content/repositories/snapshots/com/google/cloud/first-party-dependencies/3.19.0/first-party-dependencies-3.19.0.pom
Downloading from cloudera: https://repository.cloudera.com/repository/libs-release-local/com/google/cloud/first-party-dependencies/3.19.0/first-party-dependencies-3.19.0.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/google/cloud/first-party-dependencies/3.19.0/first-party-dependencies-3.19.0.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/google/cloud/first-party-dependencies/3.19.0/first-party-dependencies-3.19.0.pom (3.2 kB at 269 kB/s)
Downloading from snapshots: https://repository.apache.org/content/repositories/snapshots/com/google/cloud/google-cloud-shared-config/1.6.1/google-cloud-shared-config-1.6.1.pom
Downloading from cloudera: https://repository.cloudera.com/repository/libs-release-local/com/google/cloud/google-cloud-shared-config/1.6.1/google-cloud-shared-config-1.6.1.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/google/cloud/google-cloud-shared-config/1.6.1/google-cloud-shared-config-1.6.1.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/google/cloud/google-cloud-shared-config/1.6.1/google-cloud-shared-config-1.6.1.pom (30 kB at 2.7 MB/s)
...

Repository list shows snapshots is configured for releases+snapshots:
$ mvn -N --batch-mode eu.maveniverse.maven.plugins:toolbox:list-repositories
...
[INFO] --- toolbox:0.6.0:list-repositories (default-cli) @ fe ---
[INFO] Remote repositories used by project org.apache.doris:fe:pom:1.2-SNAPSHOT.
[INFO]  * central (https://repo.maven.apache.org/maven2/, default, releases)
[INFO]    First introduced on root
[INFO]  * snapshots (https://repository.apache.org/content/repositories/snapshots/, default, releases+snapshots)
[INFO]    First introduced on root
[INFO]  * cloudera (https://repository.cloudera.com/repository/libs-release-local/, default, releases+snapshots)
[INFO]    First introduced on root
[INFO]  * apache.snapshots (https://repository.apache.org/snapshots, default, snapshots)
[INFO]    First introduced on root
[INFO]  * sonatype-nexus-snapshots (https://oss.sonatype.org/content/repositories/snapshots, default, snapshots)
[INFO]    First introduced on org.awaitility:awaitility:jar:4.2.0 (compile)

Excerpt from same list with the fix:
$ mvn -N --batch-mode eu.maveniverse.maven.plugins:toolbox:list-repositories
...
[INFO]  * snapshots (https://repository.apache.org/content/repositories/snapshots/, default, snapshots)
...

Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[x] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?


Gitter strategy?

The README has a link to Gitter chat - are we actively maintaining that as a discussion forum?
/cc @goodits

With the monorepo split, we made the call to remove Gitter chat from snowplow/snowplow as we no longer actively watch the rooms (https://github.com/snowplow/snowplow/issues/4418) and would prefer to send people to discourse, I believe the same would apply here too.

Makes sense. How about we come up with a nice button to drive to Discourse, and replace the Gitter button with that?

[![Discourse posts](https://img.shields.io/discourse/posts?server=https%3A%2F%2Fdiscourse.snowplowanalytics.com%2F)](https://discourse.snowplowanalytics.com/)


credentials table and migrations

Table to store different types of credentials for users(member users)
tablename: credentials

id column
secret column
expires column

Moving issue to a note


Does required packages include installing sdk 3.0?

Hi,
i am running the code the dataok always stays zero and no data is being gathered from the sensor , i wanted to know is there any other packages that needed to be installed other then following
numpy
pyqtgraph
pyserial

Hi, which sdk version are you using? Each sdk version has a different configuration. So  if you are using the version 3, you should use this other repository:
https://github.com/ibaiGorordo/AWR1843-Read-Data-Python-MMWAVE-SDK-3-

i have run the code in awr1843 with configuration file '6843.conf file' from the 'https://github.com/kirkster96/IWR6843-Read-Data-Python-MMWAVE-SDK'  but still same issue that no data is being collected.
i am using iwr6843ids osk module , i wanted to collect data from the sensor for doing further deep learning on it, i dont know about the sdk versions for the mmwave sensor , if you please give me short way out for doing this, like what are things i need to install to collect data after running the script.
Thanks for your reply

I guess it is either a problem with the sdk version or that the xonfiguration file does not match. My recommendation is to use the mmwave demo  visualizer first to generate a get a working .config file. Also, that way you can test that the sdk version is the correcr one.
Once you have a working config file, you can replace the config file in this repository to run the python script.

how can i generate .config file on mmwave demo visualizer , and also is there any way to run the mmwave demo visualizer on ubuntu,
one last question : after creating the .config file, do i need to install the sdk on my device to run the python script?

https://dev.ti.com/gallery/view/mmwave/mmWave_Demo_Visualizer/ver/3.5.0/
Press the save "config to pc"
No, you do not need the sdk installed, this repository only parses the data from the radar,  so you do not need the sdk.


nbsp normalization

Added nbsp (no-break space) normalization into PDFFindController. All pdfs that I came across generated by Google Docs have nbsp instead of regular space, so this comes in handy for searching them.

@dumyan Could you post an example PDF for verification?

Test document found in IRC backlog: https://drive.google.com/file/d/0B-4Gw6drMSs2OUVoQzJaRjNWUmM/view?usp=sharing

/botio-linux preview

/botio test

Really nice, thank you!


[WIP] interface for string rendering

Here I would like to discuss method name. It seems that toString is conflicting with Basics.toString. Any suggestions?
Corresponding PR https://github.com/evancz/virtual-dom/pull/32

@evancz What do you think about merging these to PRs?
I heard in your talk that Elm is focusing on providing better developer experience. Coping my fork all over the place instead of installing it through the packager is not the best experience I've ever had :)

IMO the conflict is not a big problem because we would refer it as Html.toString usually.
But splitting a module may be better as we usually import Html exposing (..)
How about you? @evancz


Update Composer dependencies (2021-10-14-21-17)

Created multidev environment pr-11 for circleci-orb-demo-d82.

Visual regression test passed!
View the visual regression test report


Convert from Greencard to C2HS.

Everything works now. Sorry for the spurious PR's before.

No problem, thanks so much for contributing this!

Thanks for merging. Any plans of a new release?


Performance regression in configuration cache for 8.1

After unconditionally enabling configuration cache encryption.
Context
This was observed during:

pull-request https://github.com/gradle/gradle/pull/24293

All Performance Tests
https://builds.gradle.org/buildConfiguration/Gradle_Release_Check_AllPerformanceTests_Trigger/63132832
Original performance regression issue: https://github.com/gradle/gradle-private/issues/3771

I've noticed this in our project as well. With Gradle 8.1 RC-2 and the configuration cache some of our tasks are taking over 2x their normal length, and frequently failing due to GC Overhead errors. A little under half the runtime is spent in GC, according to gradle enterprise.
Disabling the configuration cache solved this for us.

@inktomi Which previous version of Gradle did you compare 8.2 RC1 with?

@abstratt I think it really got worse in RC2. I don't recall seeing nearly as many memory issues with RC1, though we did see a few.

In your specific case (GC overhead failures, much slower execution, probably due to GC pressure), could you try increasing the max heap size (btw, what heap size did you have in place?)?

Due to how our builds execute (Gradle Client, Gradle Daemon, Kotlin's Dameons) we set both -Xmx and -Xms at 13G. That puts us at the 40G limit our CI builds get to use.

We are seeing something similar with our builds, post trying out Gradle 8.0 with Configuration cache. GC Overhead error and Memory usage spiking to 95%. It works pretty well with Gradle 7.6.1  + Configuration cache.
We also have tried updating max heap size from 16G to 32G

We expect 8.2 to be slightly faster than 8.1 with configuration cache enabled. Also, we expect 8.3 to faster than 8.2 - at least that is the case ATM.

Fixed by various PRs aiming to reduce object allocation rate/memory footprint that went into 8.2 and 8.3.

Reopening this to consider re-updating the performance baseline commit.

8.2 RC1 is unblocked.


.bat file will not run for the 11khz filter off

I  edited all the Stems files under the  configs file edited to "mask_extension":"average" & can get the 11khz filter turned off when manually running the file but when trying to make a drag & drop over a .bat shortcut, I always get the stems with the 11khz filter on.  How can this be fixed please?
I get the error -- Configuration file spleeter/configs/4stems/base_config11.json  not found even though I created a file (base_config11.json with "mask_extension":"average") with that name & instructions in the same place as the original "base_config.json". I even tried editing the original "base_config.json" with "mask_extension":"average" but when I run it from a ,bat, it always runs with the 11khz filter on.
Thanks, Roger

I was able to fix the issue.


Feature request: CreateWindowInBand, CreateWindowInBandEx, SetWindowBand, GetWindowBand

Motivation
https://blog.adeltax.com/window-z-order-in-windows-10/
Drawbacks
No response
Rationale and alternatives
No response
Additional context
No response

Will forward to the win32metadata repo for consideration. These aren't documented APIs and are unlikely to be included.


Add Cloud Blog for reference points

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @jessiielynn7, thank you for the PR! However, we do not accept PRs that only add links.
I'm going to close this PR, but feel free to open a new one with more content. Happy coding!


Is the project not maintained anymore?

Just asking if the project has been abandoned, though PhysX is by far the best engine I've seen, one would have to think about switching if the project suddenly doesn't work with new versions of Rust..surely hope it's not the case but if it is I guess that's just life.

Hi!
It's a bit out of date with upstream PhysX but we still use it. :) Anything in particular you were wondering about?

Ok nice to hear, thanks for replying


How to create isomorphic app?

I am wondering how to create isomorphic apps using snabbdom. Is it just a matter of rendering the same content server side? or there is something more to do?

Hi :)
You use the package snabbdom-to-html
Check this example https://github.com/cyclejs/cyclejs/tree/cda6a8e525b8062c6765027db4683afb21242e8a/examples/isomorphic


Assistant access doesn't work without OPENAI_API_KEY env variable, preventing from accessing assistants in different projects.

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
Assistant access doesn't work without OPENAI_API_KEY env variable, preventing from accessing assistants in different projects.
Why is it needed?
I have two assistant that i need to access from slack bot
they are located in different projects
I can't rely on the environment variable - as the api_key needed is different.
To Reproduce

Create an assistant
fill the details
run the code snippet

Code snippets
import os
import openai
from retrying import retry

def should_reject(run_status):
    return run_status.status != "completed"

@retry(retry_on_result=should_reject, stop_max_attempt_number=20, wait_fixed=3000)
def retrieve_run_status(thread_id, run_id):
    return openai.beta.threads.runs.retrieve(thread_id=thread_id, run_id=run_id)

def monitor_thread_status(thread_id, run_id):
    run_status = retrieve_run_status(thread_id, run_id)
    return run_status

def access_the_assistant(api_key):

    question = "What can you help with?"
    project_id = 'proj_..........'
    assistant_id = 'asst_...........'

    client = openai.OpenAI(project=project_id, api_key=api_key)

    the_thread = client.beta.threads.create()
    thread_id = the_thread.id

    _ = client.beta.threads.messages.create(
        thread_id, role="user", content=question
    )
    run = client.beta.threads.runs.create(
        thread_id=thread_id, assistant_id=assistant_id
    )

    run_status = retrieve_run_status(thread_id, run.id)

    thread_messages = client.beta.threads.messages.list(
        thread_id,
    )
    return ( str (thread_messages.data) )

def main():
    #load the api key
    api_key = os.getenv("OPENAI_API_KEY")

    # if uncommented fix the issue - caches the api_key
    # ret = access_the_assistant(api_key)
    # print(ret)

    del os.environ['OPENAI_API_KEY']

    ret = access_the_assistant(api_key)
    print(ret)

if __name__ == "__main__":
    main()

OS
Mac
Python version
3.9
Library version
1.35.7

Assistant access doesn't work without OPENAI_API_KEY env variable

what do you mean by "doesn't work", does it crash? if so can you share a stack trace?

My bug
I used :
return openai.beta.threads.runs.retrieve(thread_id=thread_id, run_id=run_id)
instead of
return client.beta.threads.runs.retrieve(thread_id=thread_id, run_id=run_id)
which caused a mess


Unexpected in-flight messages in SQS during restart/shutdown

Type: Bug
Component:
SQS
Describe the bug
io.awspring.cloud 2.3.1
spring-cloud-aws-messaging, spring-cloud-aws-dependencies, spring-cloud-aws-autoconfigure
spring boot 2.4.5
Problem: during app restarts, queued SQS messages go in flight without acknowledgement in the app. During app shutdown, it appears that the polling mechanism is still pulling messages from SQS after the handler has been shut down. So what we're seeing is:
Test is run in the AWS environment
A queue with thousands of messages is prepared
The @SqsListener method is set up to immediately log the message and then wait 1000ms to simulate latency of real world tasks
Queue is configured with wait time of 0 zero seconds (i.e., short polling is active)
App shutdown is initiated
During shutdown, clusters of messages go in flight without creating a log in the app
On completion of restart, the messages remain in flight until the visibility window expires
Restarting the app repeatedly produces more unprocessed messages in flight. Note: the issue is intermittent and appears somewhat dependent on timing of the shutdown command. To reproduce the issue, you may need to restart the app multiple times.
This presents an issue for us during deployments (we do rolling restarts on load balanced instances). We could set the queues to retry on failure, but there are some tasks (sending emails, etc) where automatic retrying is not the desired behavior.
Sample
I will add a github link here to a proof of concept app demonstrating the issue soon.

@merickson-vg can you prepare a sample that reproduces this issue and ideally verify if it still happens with 3.0-SNAPSHOT (or M2 once released)?

Close due to lack of feedback


ArangoSearch storedValues: include fields from objects under array, ie. array[*].field

My Environment

ArangoDB Version:        3.11.8
Deployment Mode:       Leader/Follower ("Master/Slave")
Deployment Strategy:   systemd
Infrastructure:               own
Operating System:       Ubuntu 22.04
Total RAM in your machine:        256GB
Disks in use:        SSD
Used Package:              Ubuntu .deb

Component, Query & Data
Affected feature:
ArangoSearch views
Dataset:
Documents of the form:
{
  "_key": "…",
  "images": [
    {
      "id": "foo",
      "…": "…"
    },
    {
      "id": "bar",
      "…": "…"
    }
  ],
  "…": "…"
}

Steps to reproduce

Try to create an ArangoSearch view with a link to the collection described above
Specify "storedValues": { "fields": [ "images[*].id" ], "compression": "none", "cache": false }
Get an error, view cannot be created

Problem:
How can one store values in a view, for fields that are under an array?
Thank you,
Best regards

Answering my own question here, by quoting a member of ArangoDB team (hope this is OK) :

As of the current capabilities of ArangoDB, storedValues can be used to store scalar values and arrays of scalars, but not for complex objects inside arrays. This means you can store individual scalar fields or arrays of scalars but not nested objects within arrays directly.

To handle objects inside arrays, a common approach is to flatten the structure or store individual scalar fields within the objects.

So it's not possible at this time, but can be worked around.

Thanks!


Integrate Go into aws-cdk build and publish workflows

whoops, duplicate of #1834

whoops, duplicate of #1834


security parameters set for addition-only

I want to test the performance of (B)FV encryption scheme and the operations on ciphertexts will be additions only. How can I set the following parameters in order to strike a good balance between performance and security in such a context?

Plaintext modulus
Polynomial modulus

Thanks in advance!

First, Plaintext modulus doesn't impact security at all, but it matters because your data type must be wide enough to contain the sum of the values. Security depends on poly_modulus_degree and coeff_modulus total bit count. By default SEAL won't allow you to set up insecure parameters at all, so that shouldn't be a concern.
One thing that is not clear from your message is whether your hope is to use batching. I would highly recommend this, as you'll be able to encrypt and add thousands of numbers in one operation. However, after adding all batched ciphertexts you'll still need to do a slot-wise addition, which requires log(poly_modulus_degree) many rotations. This requires parameters that support keyswitching, so you won't be able to use some of the smallest parameters.

First, Plaintext modulus doesn't impact security at all, but it matters because your data type must be wide enough to contain the sum of the values. Security depends on poly_modulus_degree and coeff_modulus total bit count. By default SEAL won't allow you to set up insecure parameters at all, so that shouldn't be a concern.
One thing that is not clear from your message is whether your hope is to use batching. I would highly recommend this, as you'll be able to encrypt and add thousands of numbers in one operation. However, after adding all batched ciphertexts you'll still need to do a slot-wise addition, which requires log(poly_modulus_degree) many rotations. This requires parameters that support keyswitching, so you won't be able to use some of the smallest parameters.

Thanks for your prompt and informative reply! I will try batching and will close this issue in a few days if there is no further related questions.

Thanks for your prompt and informative reply! I will try batching and will close this issue in a few days if there is no further related questions.

Hi, there are two issues regarding batching:
Currently I have 10 arrays of 1000 16-bit integers, and I need to add them together, resulting in an array of 1000 16-bit integers.

How to set poly_modulus_degree? I found poly_modulus_degree=8192 (and meanwhile plaintext_modulus=1964769281) produced correct results, however, poly_modulus_degree=2048 (and meanwhile plaintext_modulus=65537) gave apparently wrong results. Is this related to my number of integers in an array, or to the magnitude of my integers, or both?
How is the batch efficiency affected by valid parameters? I mean, given two sets of valid parameters (plaintext_modulus, poly_modulus_degree), will there be any difference between the size of their ciphertexts given the same plaintext? If yes, how can I determine the optimal parameters in order to achieve maximum computational performance?

Thank you for the attention!

Hi, there are two issues regarding batching:
Currently I have 10 arrays of 1000 16-bit integers, and I need to add them together, resulting in an array of 1000 16-bit integers.

How to set poly_modulus_degree? I found poly_modulus_degree=8192 (and meanwhile plaintext_modulus=1964769281) produced correct results, however, poly_modulus_degree=2048 (and meanwhile plaintext_modulus=65537) gave apparently wrong results. Is this related to my number of integers in an array, or to the magnitude of my integers, or both?
How is the batch efficiency affected by valid parameters? I mean, given two sets of valid parameters (plaintext_modulus, poly_modulus_degree), will there be any difference between the size of their ciphertexts given the same plaintext? If yes, how can I determine the optimal parameters in order to achieve maximum computational performance?

Thank you for the attention!

I want to test the performance of (B)FV encryption scheme and the operations on ciphertexts will be additions only. How can I set the following parameters in order to strike a good balance between performance and security in such a context?

Determine the bit-size of required Plaintext modulus, plain_modulus_bitsize. For example, if you Plaintext modulus is 17, and you are adding a bunch of 5, then 5+5=10, 5+5+5=15, 5+5+5+5=3 (mod 17). In either words, if you intend to add four 5s, you need a 6-bit Plaintext modulus to avoid overflow.

Start with the smallest poly_mouldus_degree = 1024, then call PlainModulus::Batching(poly_modulus_degree, plain_modulus_bitsize) and CoeffModulus::BFVDefault(poly_modulus_degree) to get both moduli.

Create your program and check invariant_noise_budget when decrypting. If budget is 0, double the poly_modulus_degree and repeat from step 2.

I want to test the performance of (B)FV encryption scheme and the operations on ciphertexts will be additions only. How can I set the following parameters in order to strike a good balance between performance and security in such a context?

Determine the bit-size of required Plaintext modulus, plain_modulus_bitsize. For example, if you Plaintext modulus is 17, and you are adding a bunch of 5, then 5+5=10, 5+5+5=15, 5+5+5+5=3 (mod 17). In either words, if you intend to add four 5s, you need a 6-bit Plaintext modulus to avoid overflow.

Start with the smallest poly_mouldus_degree = 1024, then call PlainModulus::Batching(poly_modulus_degree, plain_modulus_bitsize) and CoeffModulus::BFVDefault(poly_modulus_degree) to get both moduli.

Create your program and check invariant_noise_budget when decrypting. If budget is 0, double the poly_modulus_degree and repeat from step 2.

Hi, there are two issues regarding batching:
Currently I have 10 arrays of 1000 16-bit integers, and I need to add them together, resulting in an array of 1000 16-bit integers.

How to set poly_modulus_degree? I found poly_modulus_degree=8192 (and meanwhile plaintext_modulus=1964769281) produced correct results, however, poly_modulus_degree=2048 (and meanwhile plaintext_modulus=65537) gave apparently wrong results. Is this related to my number of integers in an array, or to the magnitude of my integers, or both?
How is the batch efficiency affected by valid parameters? I mean, given two sets of valid parameters (plaintext_modulus, poly_modulus_degree), will there be any difference between the size of their ciphertexts given the same plaintext? If yes, how can I determine the optimal parameters in order to achieve maximum computational performance?

Thank you for the attention!

Thanks WeiDaiWD for exactly addressing my primitive concerns! The answer even helps me understand the first follow-up issue quoted here.
I am still curious about how is the batching efficiency affected by the parameters. Is WeiDaiWD's answer also implying that batching runtime overhead goes up as poly_mouldus_degree increases, and this is why we need try-and-error by incrementing it? By the way, I am interested not only in runtime overhead, but also in space requirement of the resulting ciphertexts. May I have a thumb rule on how these two things affected by poly_mouldus_degree (or other parameters if any) in my scenarios, respectively?
Looking forward to your instruction!

Hi, there are two issues regarding batching:
Currently I have 10 arrays of 1000 16-bit integers, and I need to add them together, resulting in an array of 1000 16-bit integers.

How to set poly_modulus_degree? I found poly_modulus_degree=8192 (and meanwhile plaintext_modulus=1964769281) produced correct results, however, poly_modulus_degree=2048 (and meanwhile plaintext_modulus=65537) gave apparently wrong results. Is this related to my number of integers in an array, or to the magnitude of my integers, or both?
How is the batch efficiency affected by valid parameters? I mean, given two sets of valid parameters (plaintext_modulus, poly_modulus_degree), will there be any difference between the size of their ciphertexts given the same plaintext? If yes, how can I determine the optimal parameters in order to achieve maximum computational performance?

Thank you for the attention!

Thanks WeiDaiWD for exactly addressing my primitive concerns! The answer even helps me understand the first follow-up issue quoted here.
I am still curious about how is the batching efficiency affected by the parameters. Is WeiDaiWD's answer also implying that batching runtime overhead goes up as poly_mouldus_degree increases, and this is why we need try-and-error by incrementing it? By the way, I am interested not only in runtime overhead, but also in space requirement of the resulting ciphertexts. May I have a thumb rule on how these two things affected by poly_mouldus_degree (or other parameters if any) in my scenarios, respectively?
Looking forward to your instruction!

If the number of primes remain the same, runtime overhead is linearly to quasilinear, while ciphertexts' sizes are linear on poly_modulus_degree. If poly_modulus_dgree remain the same, runtime overhead is linearly to quadratic, while ciphertexts' sizes are linear on the number of primes. These are the two factors affecting runtime overhead and space.

Finally have your clear answers! Thanks a lot.


insert live into the PACE file during binary upload step

Resolves #204.

@mcevoypeter sorry I accidentally changed the base branch for the PR, will you please re-approve?

After this is merged, we should cherry-pick the commit onto the release branch so we have the code for the next release process (which means we won't have to worry any more about where the live binaries get uploaded to in our gcloud account).


SwiftNIO in Playground

Expected behavior
Use Import NIO and import NIOTransportServices in the playground
Actual behavior
NIO and NIOTransportServices are not listed in the playground
Steps to reproduce
I followed instrcutions from this link - https://forums.swift.org/t/cant-experiment-swiftnio-in-playground/15020/4
Using Xcode Version 11.0 beta 5 (11M382q)

Create a new Xcode project for macOS
pod init
edit Podfile and put

POD File
target 'ByteBuffer_Swift_NIO' do
  # Comment the next line if you don't want to use dynamic frameworks
  use_frameworks!

  # Pods for ByteBuffer_Swift_NIO
  pod 'SwiftNIO'
  pod 'SwiftNIOTransportServices'

end

pod install
close the Xcode project
open the Xcode workspace created by Cocoapods
File -> New -> Playground, select Blank
put import NIO and probably import NIOTransportServices in the playground
and you're done :)

===

After that you can run swift package generate-xcodeproj inside SwiftNIO and drag the generated Xcode projects into any of your Xcode projects and it will work. It will also work in a playground

This also doesnt work. I drag the generated swift-nio.xcodeproj in the any other xcode project but modules are not recognized.
Swift & OS version (output of swift --version && uname -a)
Apple Swift version 5.0.1 (swiftlang-1<IP_ADDRESS> clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin18.6.0

Try adding pod 'SwiftNIO' and pod 'SwiftNIOTransportServices' to the root of the pod file. I think dragging xcodeprojects around makes things get a bit weird with targets.

This also doesnt work. I drag the generated swift-nio.xcodeproj in the any other xcode project but modules are not recognized.

Can you elaborate on what happens here?

Try adding pod 'SwiftNIO' and pod 'SwiftNIOTransportServices' to the root of the pod file. I think dragging xcodeprojects around makes things get a bit weird with targets

Scenario - 1 :
In the pod test project - I did not drag any xcodeproj - but just generated with pod install and then tried creating the playground project as described.

This also doesn't work. I drag the generated swift-nio.xcodeproj in the any other xcode project but modules are not recognized.
Can you elaborate on what happens here?

Scenario - 2 :
Created swift-nio project with swift package generate-xcodeproject and then dragged and dropped the swift-nio.xcodeproj into a new xcodeproj (not built from pods) but from xcode > File > New > Project
Then created new playground but still  I do not get any of NIO modules in the import list.

@amitahire Apologies. I just tried with Xcode 11 beta 7 and it still works. I also made a screen recording.

@amitahire can you confirm that this works for you too?

Thanks for replying back. It works now!


Good to add manual step to copy/upload the files

with ref.: #30824

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

ID: 046582d4-a031-1e6a-fbdd-ff6f59245266
Version Independent ID: e73b7c20-3f12-5ab5-1def-460df469d16e
Content: Build ASP.NET app with Windows container (Preview) - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-windows-containers-custom-fonts.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

Thanks for the feedback! We are currently investigating and will update you shortly.

Hi @msinghm, we appreciate your feedback. The COPY line command you mentioned in #30824 is part of the docker file that gets generated after you choose Docker Compose as your container orchestrator. Therefore, I don't think adding this to the doc is necessary.


Suppress magic number for first arg of pow()?

Expected Behavior of the rule
The Magic Number rule exists to prompt for constants to be named so as to provide inherent documentation for the purpose of the number. But what if the only logical name for the constant is the name of the number itself?
Context
This code line:
        val base = pow(10.0, magnitude)

triggers the Magic Number rule. But I can't think of any name for 10.0 in this context other than TEN, and to add a literal constant const val TEN = 10.0 merely adds complexity without improving readability.
I'm open to suggestions on how to deal with this - suppressing the magic number check for the first argument of pow() is the first thing that comes to mind. Suppressing all warnings for 10.0 as is already done for 0, 1 and 2 is probably not a good idea.

How about using named arguments here for the 10.0 ?

pow() is a Java method (java.lang.Math.pow()) and so does not have named arguments.

@clydebarrow you can add 10 to the ignoreNumbers list in the config.
Maybe this should also be a default number.

I don't think whitelisting functions is the way to go... There are surely other cases where values other than 10 are used as the first parameter to pow, but if the function is whitelisted it would miss other usages and allow magic numbers to be used.
There's already suppression available within individual files, or 10 can be added to ignoreNumbers into the config file as mentioned - can you use either of those options instead?

Suppressing in that file is fine - I did not know that was supported.


Remove unnecessary _.map in the code for video call menu items

Tests & QA

Open a chat
Click on the video chat icon
Click the google meet option
Go back to the app
Verify that the video chat menu went away

[ ] Verify that no errors appear in the JS console

Screenshots

Web

https://user-images.githubusercontent.com/1228807/166560700-31a3ba6a-c65c-4b66-bd1f-8d1f24fb64e4.mp4
Mobile Web

https://user-images.githubusercontent.com/1228807/166560768-ac2c85a1-18cc-48e6-992e-c1dbea3d946a.mp4
Desktop

https://user-images.githubusercontent.com/1228807/166561066-1ca31c22-a5b0-4770-a380-d2e9accf7a51.mp4
iOS

https://user-images.githubusercontent.com/1228807/166560436-d81387bd-47c1-4b39-9a8a-4c106e79af0a.mp4
Android

https://user-images.githubusercontent.com/1228807/166559694-d53d670e-4bde-407f-aea0-70e7baae9cde.mp4

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Cherry-picked to staging by @sketchydroide in version: 1.1.57-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @chiragsalian in version: 1.1.57-17 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Incompatible with asyncpg 0.29.0

Describe the bug:

IndexError: tuple index out of range
  File "app/core/repositories/user.py", line 155, in find_all
    async with self.engine.begin() as conn:
  File "contextlib.py", line 199, in __aenter__
    return await anext(self.gen)
  File "sqlalchemy/ext/asyncio/engine.py", line 1061, in begin
    async with conn:
  File "sqlalchemy/ext/asyncio/base.py", line 125, in __aenter__
    return await self.start(is_ctxmanager=True)
  File "sqlalchemy/ext/asyncio/engine.py", line 270, in start
    await greenlet_spawn(self.sync_engine.connect)
  File "sqlalchemy/util/_concurrency_py3k.py", line 190, in greenlet_spawn
    result = context.throw(*sys.exc_info())
  File "sqlalchemy/engine/base.py", line 3268, in connect
    return self._connection_cls(self)
  File "sqlalchemy/engine/base.py", line 145, in __init__
    self._dbapi_connection = engine.raw_connection()
  File "sqlalchemy/engine/base.py", line 3292, in raw_connection
    return self.pool.connect()
  File "sqlalchemy/pool/base.py", line 452, in connect
    return _ConnectionFairy._checkout(self)
  File "sqlalchemy/pool/base.py", line 1269, in _checkout
    fairy = _ConnectionRecord.checkout(pool)
  File "sqlalchemy/pool/base.py", line 716, in checkout
    rec = pool._do_get()
  File "sqlalchemy/pool/impl.py", line 169, in _do_get
    with util.safe_reraise():
  File "sqlalchemy/util/langhelpers.py", line 146, in __exit__
    raise exc_value.with_traceback(exc_tb)
  File "sqlalchemy/pool/impl.py", line 167, in _do_get
    return self._create_connection()
  File "sqlalchemy/pool/base.py", line 393, in _create_connection
    return _ConnectionRecord(self)
  File "sqlalchemy/pool/base.py", line 678, in __init__
    self.__connect()
  File "sqlalchemy/pool/base.py", line 916, in __connect
    )._exec_w_sync_on_first_run(self.dbapi_connection, self)
  File "sqlalchemy/event/attr.py", line 473, in _exec_w_sync_on_first_run
    self(*args, **kw)
  File "sqlalchemy/event/attr.py", line 487, in __call__
    fn(*args, **kw)
  File "sqlalchemy/engine/create.py", line 704, in on_connect
    do_on_connect(dbapi_connection)
  File "sqlalchemy/dialects/postgresql/asyncpg.py", line 1247, in connect
    conn.await_(self.setup_asyncpg_json_codec(conn))
  File "sqlalchemy/util/_concurrency_py3k.py", line 125, in await_only
    return current.driver.switch(awaitable)  # type: ignore[no-any-return]
  File "sqlalchemy/util/_concurrency_py3k.py", line 185, in greenlet_spawn
    value = await result
  File "sqlalchemy/dialects/postgresql/asyncpg.py", line 1173, in setup_asyncpg_json_codec
    await asyncpg_connection.set_type_codec(
  File "asyncpg/connection.py", line 1339, in set_type_codec
    typeinfo = await self._introspect_type(typename, schema)
  File "asyncpg/connection.py", line 543, in _introspect_type
    rows = await self._execute(
  File "asyncpg/connection.py", line 1794, in _execute
    result, _ = await self.__execute(
  File "asyncpg/connection.py", line 1892, in __execute
    result, stmt = await self._do_execute(
  File "asyncpg/connection.py", line 1945, in _do_execute
    result = await executor(stmt, None)
  File "elasticapm/instrumentation/packages/asyncio/asyncpg.py", line 65, in call
    query = self.get_query(method, args)
  File "elasticapm/instrumentation/packages/asyncio/asyncpg.py", line 62, in get_query
    return args[0].query

To Reproduce

install asyncpg==0.29.0 and elastic-apm==6.19.0 together
perform any request to postgresql without params

Environment (please complete the following information)

OS: Linux 5.16
Python version: 3.10
Agent version: 6.19.0
asyncpg version: 0.29.0
sqlalchemy version: 2.0.23

Fix in #1935. :)

@basepi I encountered this similar issue, however it occurred intermittently, failing for some queries and working for most of them. Any insights as why it fails for some queries only , shouldn't it have failed for all the queries when i was using elastic-apm v==6.8.1

@slice-ArpitSharma the fix (#1935) was released on v6.20.0. I think it'd fail only for queries without args.
https://github.com/elastic/apm-agent-python/blob/main/CHANGELOG.asciidoc#6200---2024-01-10


Change DIR value after programming.

Let's begin with, congrats for the nice work in the library.
Imagine the following scenario:
I program de AS5600 to give max output value at 10 degrees and min output value at 170 degress. It works fine, however, shouldn't it give me min output at 10 degreens and max output at 170 if I change the value of the direction pin?
What is the expected behaviour in this case? I've been looking for such information in the technical documents of AS5600, however there is little information about how the DIR pin affects the behaviour of an already programmed sensor.
Thanks for any advice.

A quick first response
Changing the direction pin changes the direction of increase and from my head it works on the raw data. So 10 degrees become 350 and 170 becomes 190. It does not swap 10 and 170.
What is the underlying problem you try to solve?

I'm trying to design a replacement for a potentiometer based angle sensor.
In the specific model that I'm working in, this sensor can be mounted in its target machine in both polarities, it has 3 pins:
Wiring when used in place 1:
A- 5V
B- Output
C- GND

10 deg 0V
170 deg 5V

Wiring when used in place 2:
A- GND
B- Output
C- 5V

10 deg 5V
170 deg 0V

I'm using a low drop rectifier bridge in order to provide 5V to the AS56000, and I wanted the DIR pin to indicate to the IC in which way it is being mounted.
In order to keep the same behaviour as the original sensor that uses a potentiometer, I need the afore mentioned output curve.
I'm wondering if the AS5600's output would follow this mechanism of inverting its output ramp according to the DIR state, in special after programming.

@vbenso
Any progress made?
If you mount in place 2 what are the angles you get if you dont change anything?
(A simple mapping might then do the trick as you map a range of 160 degrees on another range of 160 degrees )

As there is no answer I assume the problem is solved, so I close the issue.


npx oxlint -c .oxlintrc.json not working

oxlint not working with .oxlintrc.json config.
Command npx oxlint -c .oxlintrc.json not found any cycle
Config example:
{
  "rules": {
    "import/no-cycle": "error"
  }
}

npx oxlint --import-plugin -A all -D no-cycle working fine
npx oxlint --import-plugin -A all -D no-cycle

  × eslint-plugin-import(no-cycle): Dependency cycle detected
   ╭─[src/test.ts:1:23]
 1 │ import { test2 } from "./App";
   ·                       ───────
 2 │
   ╰────
  help: These paths form a cycle:
        -> ./App - src/App.tsx
        -> ./test - src/test.ts

  × eslint-plugin-import(no-cycle): Dependency cycle detected
   ╭─[src/App.tsx:4:22]
 3 │ import "./App.css";
 4 │ import { test } from "./test";
   ·                      ────────
 5 │ console.log(test());
   ╰────
  help: These paths form a cycle:
        -> ./test - src/test.ts
        -> ./App - src/App.tsx

Finished in 8ms on 5 files with 1 rules using 10 threads.
Found 0 warnings and 2 errors.

Try npx oxlint -c .oxlintrc.json --import-plugin

@DonIsaac It seems unintuitive that --import-plugin needs to be supplied here along with -c 🤔

I agree, having plugins enabled in configs will assuage this.

Let's improve the documentation a little bit before closing.


Cmake: FetchContent vs ExternalProject_Add

CMake 3.11 (?) Introduced FetchContent as an alternative to ExternalProject_Add.

This module enables populating content at configure time via any method supported by the ExternalProject module. Whereas ExternalProject_Add() downloads at build time, the FetchContent module makes content available immediately, allowing the configure step to use the content in commands like add_subdirectory(), include() or file() operations.

I.e. for googletest:
include(FetchContent)

FetchContent_Declare(
  googletest
  GIT_REPOSITORY https://github.com/google/googletest.git
  GIT_TAG        release-1.8.0
)

FetchContent_GetProperties(googletest)
if(NOT googletest_POPULATED)
  FetchContent_Populate(googletest)
  add_subdirectory(${googletest_SOURCE_DIR} ${googletest_BINARY_DIR})
endif()

inplace of:
cmake_minimum_required(VERSION 2.8.2)

project(googletest-download NONE)

include(ExternalProject)
ExternalProject_Add(googletest
    GIT_REPOSITORY    https://github.com/google/googletest.git
    GIT_TAG           master
    SOURCE_DIR        "${CMAKE_CURRENT_BINARY_DIR}/googletest-src"
    BINARY_DIR        "${CMAKE_CURRENT_BINARY_DIR}/googletest-build"
    CONFIGURE_COMMAND ""
    BUILD_COMMAND     ""
    INSTALL_COMMAND   ""
    TEST_COMMAND      ""
    )

and
# START Download & Build GoogleTest
MACRO (download_gtest)
    configure_file(CMakeLists.txt.in googletest-download/CMakeLists.txt)
    # Run CMake generate
    execute_process(
        COMMAND ${CMAKE_COMMAND} -G "${CMAKE_GENERATOR}" .
        RESULT_VARIABLE result
        WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/googletest-download
        )
    if (result)
        message(WARNING
                "CMake step for googletest failed: ${result}\n")
    endif()
    # Run CMake build (this only downloads, it is built at build time)
    execute_process(
    COMMAND ${CMAKE_COMMAND} --build .
    RESULT_VARIABLE result
    WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/googletest-download
    )
    if (result)
        message(WARNING
                "Download step for googletest-build failed: ${result}\n"
                "Attempting to continue\n"
                "You may want to disable BUILD_TESTS")
    endif()
ENDMACRO()

IF(UPDATE_GTEST)
    download_gtest()
ENDIF()
IF(NOT EXISTS ${CMAKE_CURRENT_BINARY_DIR}/googletest-src)
    download_gtest()
    # Give option to disable trying to update after first run
    set(UPDATE_GTEST ON CACHE BOOL "Update GTest from Git")
ENDIF()

Seems ok, guessing it defers building gtest till build time too.
Might not work for all dependencies, e.g. the zip ones.
Should also work for freetext in vis.


Add liveness card in Discover

same process as other kyc card

do it later with new ID360

same ticket


Migrating TFOD to TensorFlow 2.x

Describe the solution you'd like
Training workflow using "Tensorflow Object Detection API" should be update to work with TensorFlow 2.x and the Saved Model format since frozen models were deprecated in TensorFlow 2.x
In the training workflow:

Training workflow should support tf2.
In CVAT
The folder and files of the saved model must be uploaded maintaining their folder structure or a single zip file must be accepted.
The automatic annotation process should work flawlessly with the old frozen model (tf1.x) and the new saved model (tf2). Or the saved model format should be used for both.

Additional context
Currently, CVAT is performing a validation that made it impossible to load the model in the Saved Model format.

@javoweb looks like onepanel/dl:0.17.0 supports --use-feature:
$ pip --use-feature test

Usage:
  pip <command> [options]

option --use-feature: invalid choice: '--help' (choose from '2020-resolver', 'fast-deps')

@javoweb looks like onepanel/dl:0.17.0 supports --use-feature:
$ pip --use-feature test

Usage:
  pip <command> [options]

option --use-feature: invalid choice: '--help' (choose from '2020-resolver', 'fast-deps')

@javoweb looks like onepanel/dl:0.17.0 supports --use-feature:
$ pip --use-feature test

Usage:
  pip <command> [options]

option --use-feature: invalid choice: '--help' (choose from '2020-resolver', 'fast-deps')

@javoweb looks like onepanel/dl:0.17.0 supports --use-feature:
$ pip --use-feature test

Usage:
  pip <command> [options]

option --use-feature: invalid choice: '--help' (choose from '2020-resolver', 'fast-deps')


sdk for azure spring cloud (appplatform)

Let me know when this is ready for review @jialindai thanks!
And please put the swagger review links in PR description.

@isra-fel , the swagger and .NET sdk is updated. Now it's ready for review.
Thanks!

@isra-fel , here is the swagger file link: https://github.com/Azure/azure-rest-api-specs/tree/master/specification/appplatform/
And here are swagger related PRs link: https://github.com/Azure/azure-rest-api-specs/commit/ab77bfc760b7830058616b49ee52a2ed8128b738#diff-58d78663e62b0ada2b49958c54fcb438, https://github.com/Azure/azure-rest-api-specs/commit/59998f796f05580bc1d43ba39f0170204113fa91#diff-58d78663e62b0ada2b49958c54fcb438, https://github.com/Azure/azure-rest-api-specs/commit/191aa6f4e91585f08acf453f47817a36f1db9c69#diff-58d78663e62b0ada2b49958c54fcb438

@isra-fel , the version is updated and AssemblyInfo.cs is added. Please help to review the PR.


Crash on Startup

As the name says, the program starts up, but as soon as it hits the loading screen, it closes. I tried to check the dump file, but I couldn't find anything.

Same, China's server takes longer to get newer updates.


Discuss: use of stormpath keyword

Our examples say that you shoud import this module like this:
var stormpath = require('express-stormpath');

This is proving confusing, because our node docs also do the same thing:
var stormpath = require('stormpath');

Alas, it's unclear which API you're actually working with - the framework integration, or the SDK.
I believe that our examples should use this form:
var expressStormpath = require('express-stormpath');

+1 for being explicit with expressStormpath.

I totally disagree. Users using this library will likely only have this be their 'stormpath' experience. Namespacing this differently doesn't make sense, IMO, as it makes your middleware codez look ugly.
app.get('/', stormpath.loginRequired, ...);

// vs

app.get('/', expressStormpath.loginRequired, ...);

The current stuff looks pretty elegant this way, imo.

I agree with it looking better, but isn't also many using the SDK in the integrations? But if there is no (or little) chance that they're going to be working with the SDK then I'd agree that just stormpath is nicer.

Most people start with express-stormpath, then move on to needing more advanced features that require the underlying SDK.  Some people are happy to pull the constructed client out of the integration state (aka req.app.get('stormpthClient') and use that to get work done, but others end up installing the SDK as a top-level library.  It gets confusing when helping people debug.
I'm really resistant to the "looks better" argument when the underlying API's are totally different.

If people are using express-stormpath, and want to access the underlying client, we should tell them how to do that in the docs.
They have two choices:
app.get('stormpathClient')

// or, inside of a route:
req.app.get('stormpathClient')

They should never ever import the stormpath library directly, as it'll create a duplicate client which causes 2x caching / all the issues we've had originally.

I agree that a creating two clients is problematic, if you're not aware of the caching issues.  I still feel dirty every time I call this export stormpath, but I do see how it makes things feel simpler.  Closing.


Can't delete components in Firefox

Clicking on the 'x' does nothing

Also can't rename components in Firefox


Show\Hide "nothing found"

Is there any way to prevent showing warning with no suggestions?

Looking on sources I can prevent showing "nothing found" by replace empty suggestions array with null.


release #1152 to new version

installing package is not possible now.
It seems #1152 has merged but not yet released.
Would you plan for more fix??
I hope to know when the new version is released

Hi @laggu , do you have problems installing only the latest version?

installing with github.com/hiero-ledger/hiero-sdk-go fail
but installing with github.com/hashgraph/hedera-sdk-go works

Ok, we will release a new version tomorrow and hopefully the go modules can now find the new import uri.

working well now


[TFLite Export] Adds TFLite support for LayoutLMv3

What does this PR do?

Adds support for exporting LayoutLMv3 to TFLite. Some tests for all quantization approaches were failing for some tasks, so I limited the supported quantization schemes for now.
It was primarily based on https://github.com/huggingface/optimum/pull/813 and the ONNX export code already part of the library.
It can be tested as:
pytest tests/exporters/tflite/test_*.py -k "layoutlmv3" -s --exitfirst

You can export a trained model as:
optimum-cli export tflite --model <input_dir> --task <> --sequence_length <> --width <> --height <> <output_dir>
optimum-cli export tflite --model microsoft/layoutlmv3-base --task token-classification --sequence_length 512 --width 224 --height 224 ~/tflite_conversion

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes? (Not needed as far as I can tell)
[ ] Did you write any new necessary tests? (Not needed as far as I can tell)

Hey @fxmarty @michaelbenayoun Would you like to merge this before it goes stale? Happy to update the PR in case it has

Hi @salmanmaq ,
Happy to do it once the tests related to LayoutLMv3 pass. I think some tests in export with quantization do not pass.

Hi @salmanmaq , Happy to do it once the tests related to LayoutLMv3 pass. I think some tests in export with quantization do not pass.

Thanks @michaelbenayoun. Is it something with the PR, or at your end? Would be checking it out.

@salmanmaq, since there is a suggestion the issue might not be related to your PR, I noticed that your fork is a bit behind, maybe you could pull the latest commits try again?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Config: keystore.path should be renamed keystore.file

Currently keystore.path is used to configure an existing keystore, however this is confusing as keystore.crtPath expects a directory path. To differentiate the two, we should change keystore.path -> keystore.file.

On second thought, I think keystore.file maybe more confusing as users might think that it's just the name of the keystore file not the complete path.


Please add DogeChain Support

Website
https://explorer.dogechain.dog/
ChainId
2000 - Main net
568 - Test net

I have created an issue to discuss this process and I would like to invite you to participate. The issue can be found at https://github.com/safe-global/safe-infrastructure/issues/75. Thank you for your consideration.

https://github.com/safe-global/safe-singleton-factory/pull/115


Genomics England data does not have required publication field

@elipapa commented on Mon Jan 08 2018
@emepyc commented on Fri Dec 01 2017
The genomics evidence schema needs a non-empty publications array but the Genomics England PanelApp data doesn't provide one always.
In the December release, to make the schema happy, a dummy publication has been added in the form:
      "evidence": {
        "provenance_type": {
          "literature": {
            "references": [
              {
                "lit_id": "NA"
              }
            ]
          },

If the field is not mandatory it should not be required in the schema instead of bypassing the validation by including a "NA" value. Having these hacks undermine the benefits of having an schema in the first place.
As a side effect this requires including weak code in the webapp or any other consumer of the data.

This issue is still valid. Of 11,079 Genomics England evidence strings, 6,059 contain 'NA' as literature ID. Examples with literature and without literature ID.
For Genomics England evidence, the actual gene panel is listed in the Evidence source column - I wonder whether it makes sense to say Genomics England in the Evidence source column and list the specific gene panel in Publications column and if a publication is available, that would also be listed there.

Reassigned to 20.11 pending further conversations with Genomics England

After conversations with GEL (see #38) the API querying has been optimized - being able to get all the Publications terms without any raised errors.
This issue no longer applies.


tintColor of the JGProgressHUDImageIndicatorView

Hello,
This is my first comment, sorry in advance for my english and the formatting.
Within JGProgressHUDErrorIndicatorView and JGProgressHUDSuccessIndicatorView you have the exact same two methods:
- (void)setUpForHUDStyle:(JGProgressHUDStyle)style vibrancyEnabled:(BOOL)vibrancyEnabled
- (void)tintColorDidChange
1 - Why aren't the moved within JGProgressHUDImageIndicatorView directly?
Imagine I'm using the error hud like so:
JGProgressHUD *jgHUD = [JGProgressHUD progressHUDWithStyle:JGProgressHUDStyleDark];
jgHUD.textLabel.text = title;
jgHUD.delegate = self;
jgHUD.indicatorView = [[JGProgressHUDErrorIndicatorView alloc] init];
[jgHUD showInView:self.view animated:false];
[jgHUD dismissAfterDelay:2.0];

The image should therefore be white (because the style used is JGProgressHUDStyleDark).
However, since there is a the:
- (void)tintColorDidChange {
    [super tintColorDidChange];
    self.contentView.tintColor = self.tintColor;
}

The image color changes to the one set on the UIViewController or Storyboard when the HUD appears.
2 - What is the purpose off using - (void)tintColorDidChange here? Is this a wanted behavior? Am I using it wrong?
Thanks in advance for your response.

@JonasGessner ? 🙂

You do make a point and I think I will change something here to make it simpler. But it's something that needs some thought and I will take a look sometime. Feel free to open a PR with a suggested change.


How can I use python to achieve the effect demonstrated by the demo, so that the mask of the corresponding area appears when hovering?

When I use the demo for image segmentation:
Demo url: https://segment-anything.com/demo
It shows very good results.
But when I use SamAutomaticMaskGenerator and SamPredictor, I need to enter each coordinate separately to obtain the mask map of the corresponding area.But this is too inefficient.
Is there any way to return the corresponding coordinates and mask map of the divided area, or can it only be achieved through the front end and Vue?
This is the best practice for image segmentation!

This repo contains an example of hovering and getting a masked area, inside of the demo folder. It also uses the browser/javascript, but uses react, in case that's something you're familiar with.
In general that hover + masking effect will require some sort of UI library, and can be a lot of code to implement. If you wanted to do it directly from Python, then OpenCV actually contains enough UI support (through window callbacks) to do it, but it's a lot of fiddly code to set up. Using the browser for handling UI is likely the most popular option, and there are existing repos that do this (like sam-webui) which might be worth checking out.


Crash with java.lang.IllegalArgumentException: Unique transitionNames are required for all sharedElements

Branch complete
On click on MaterialCardView of  HomeFragment#onEmailClicked get crash:
    java.lang.IllegalArgumentException: Unique transitionNames are required for all sharedElements
        at androidx.fragment.app.FragmentTransaction.addSharedElement(FragmentTransaction.java:595)
        at androidx.navigation.fragment.FragmentNavigator.navigate(FragmentNavigator.java:212)
        at androidx.navigation.fragment.FragmentNavigator.navigate(FragmentNavigator.java:58)
        at androidx.navigation.NavController.navigate(NavController.java:933)
        at androidx.navigation.NavController.navigate(NavController.java:867)
        at androidx.navigation.NavController.navigate(NavController.java:1006)
        at com.materialstudies.reply.ui.home.HomeFragment.onEmailClicked(HomeFragment.kt:117)
        at com.materialstudies.reply.databinding.EmailItemLayoutBindingImpl._internalCallbackOnClick(EmailItemLayoutBindingImpl.java:219)
        at com.materialstudies.reply.generated.callback.OnClickListener.onClick(OnClickListener.java:11)
        at android.view.View.performClick(View.java:6294)
        at android.view.View$PerformClick.run(View.java:24770)
        at android.os.Handler.handleCallback(Handler.java:790)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loop(Looper.java:164)
        at android.app.ActivityThread.main(ActivityThread.java:6494)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

Also got same issue on branch develop after step 4 of the codelab. The change made in HomeFragment.onEmailClicked() on this step seems to trigger the issue.

It seems android:transitionName="@{@string/email_card_transition_name(email.id)}" has not correct value

@Werder630 @IgnacioGarcia198 thanks for trying out the codelab!
When I check out the complete branch, run the app, and click on an email, the transition runs successfully and I don't see the crash. Same thing happens if I start from develop and add in the transition names, the navigate call with FragmentNavigatorExtras, and the MaterialContainerTransform sharedElementEnterTransition.
Can you make sure that the android:transitionName="@{@string/email_card_transition_name(email.id)}" is set on the MaterialCardView in email_item_layout.xml, as opposed to the MaterialCardView in fragment_email.xml? That's the important part for ensuring uniqueness.
Also, if you can fork the codelab repo and push your local changes to a branch, I can take a look to see if there's an issue.

It seems like assigning the email variable to the binding in the is not updating the property in the CardView itself.
One way of fixing it is assigning the transitionName directly in EmailViewHolder.bind:
binding.cardView.transitionName = itemView.context.getString(R.string.email_card_transition_name, email.id)

@IgnacioGarcia198 would you be able to push your changes that reproduce the crash to a branch?

@dsn5ft Of course. How should we name it, issue#4Reproduction?

@IgnacioGarcia198 thanks, sounds good to me!

@dsn5ft I created the pull request from my fork. Although mysteriously enough, now the issue is not happening anymore on any of the branches.

Hmm yeah I don't see anything wrong with your code. Maybe @Werder630 will be able to reproduce it and create a branch.

i haven't changed anything in codelabs, just cloned repo and checkout branch "complete", next launch app and click on card. App crashes with my stack

Gotcha. By any chance did you happen to upgrade any Gradle versions with Android Studio's suggestion popup? Can you do a git status to make sure that you don't have any local changes at all?
Also, does the crash happen when clicking on all of the cards? Or just some of them?

@dsn5ft I made no changes at all in Gradle, etc. As far as I know it looks like an issue that only happens under certain circumstances, probably something to do with the proper working of DataBinding in certain circumstances. For example, at the time I was getting the crash, text related to the DataBinding layouts was in red, not sure if it could have something to do.
I think it was happening when clicking on any card.
I checked my status, I have no local changes, after I cleared app data and uninstalled it I can still not reproduce it.

yes, i haven't local changes too. For exmaple,  if i change android:transitionName="@{@string/email_card_transition_name(email.id)}" to android:transitionName="@string/email_card_transition_name" - app works awesome (but the animation not entirely accurate), so i consider, the problem due to dataBinding
PS: i try on AS 4.1 and 4.2

Thanks for confirming. Random guess, but can you try doing a Build > Clean Project and then a Build > Rebuild Project in Android Studio, and then running the app without using Instant Run?

It is strange, when i rebuilt app several times, the crash disappeared...  @dsn5ft Could you clone app from scratch, and try launch app on branch complete?

I cloned app as new project again and run complete branch. Not happening anymore. I also tried wiping app data, cleaning project, installing again. Not reproducing at all.

Yes, it is strange floating bug, so i advise to close it

Same here, I wonder if it has to do with Studio caching an older version of the Gradle/AGP, and then the build gets into a bad state. I did recently update the repo's Gradle versions in https://github.com/material-components/material-components-android-motion-codelab/commit/d1d87ee3c4cf329d0ccbaba7ff59c10879b63a18, not sure if you originally cloned before that and then pulled. Anyway, closing for now but feel free to reopen if you find out more.

I just experienced this going through the codelab fresh.
AS 4.0.1
Coming in from building a project previously that uses...
classpath 'com.android.tools.build:gradle:3.6.4'```

Doing a project clean _seems_ to resolve the issue.

Ran into the same issue as well on Android Studio 4.0.1. Cleaning and rebuilding the project as suggested fixed the issue.

@Werder630 @IgnacioGarcia198 thanks for trying out the codelab!
When I check out the complete branch, run the app, and click on an email, the transition runs successfully and I don't see the crash. Same thing happens if I start from develop and add in the transition names, the navigate call with FragmentNavigatorExtras, and the MaterialContainerTransform sharedElementEnterTransition.
Can you make sure that the android:transitionName="@{@string/email_card_transition_name(email.id)}" is set on the MaterialCardView in email_item_layout.xml, as opposed to the MaterialCardView in fragment_email.xml? That's the important part for ensuring uniqueness.
Also, if you can fork the codelab repo and push your local changes to a branch, I can take a look to see if there's an issue.

exactly as mentioned, it was in constraint layout it supposes to be in cardview. all-star :)


Deprecated api used Firebase Messaging iOS Direct Channel

🐛 Bug Report
the firebase team has announced that Firebase Messaging iOS Direct Channel will be removed.  This plugin sets up direct connection:
https://github.com/CrossGeeks/FirebasePushNotificationPlugin/blob/master/Plugin.FirebasePushNotification/FirebasePushNotificationManager.ios.cs#L239
can't APNs be used instead?
Platform:

[x] :iphone: iOS

MY COMMENT IS NOT A SOLUTION, IT'S EMPHASIS
I Received the exact email from the firebase team not so long ago. Are there any plans in regard to this plugin to deal with the depreciation of the iOS Direct Channel ?


Proposal: dockerfile keyword: ASSERT

what we have:
docker has a great feature: ONBUILD
what we need:
how one can be sure that the dockerfile is working?

build success
test success!

how to do it:

implement a new keyword: ASSERT or TEST or ONVISIT or what-ever
this command should give the client all the answers he ask from his dockerfile.

benefits:

no need to actually test the dockerfile locally or build a new testing-system

god bless

Hello!
We are no longer accepting patches to the Dockerfile syntax as you can read about here: https://github.com/docker/docker/blob/master/ROADMAP.md#22-dockerfile-syntax
Mainly:

Allowing the Builder to be implemented as a separate utility consuming the Engine's API will open the door for many possibilities, such as offering alternate syntaxes or DSL for existing languages without cluttering the Engine's codebase

Then from there, patches/features like this can be re-thought. Hope you can understand.


fix prompt_formatting_fn

fixes #322 .
I don't know if this could break any undocumented legacy use, but Task should always return a Prompt, so the current implementation just discards prompt_formatting_fn silently and the type checks should be unnecessary here?
Code is tested for the problem I encountered in #322 .

Hi here @jphme thanks for the issue and the PR straight away! Our idea with this was mainly for the tasks that were not returning a Prompt so that one could define a custom Task and then override the prompt-processing function before it gets passed to the LLM.generate method. But fair, currently it has no point with the tasks that we provide as part of distilabel, so before merging we'll review the current situation of the prompt_formatting_fn arg and define whether it's needed or not. Thanks! 🤗

Hi here @jphme thanks for the issue and the PR straight away! Our idea with this was mainly for the tasks that were not returning a Prompt so that one could define a custom Task and then override the prompt-processing function before it gets passed to the LLM.generate method. But fair, currently it has no point with the tasks that we provide as part of distilabel, so before merging we'll review the current situation of the prompt_formatting_fn arg and define whether it's needed or not. Thanks! 🤗

I figured that this is the reason but in the current implementation Task should always return a Prompt (see here ) and it also makes sense to always to the prompt formatting stuff on the LLM side of things conceptually.
The current implementation leads to silent errors (prompt format is silently ignored without any warnings or errors) which is the worst choice. And additionally you cant format the prompts any other way than with the (limited) default templates. My change would make prompt_formatting_fn` consistent with the default prompt formats (taking precedence if both are provided).
(But it probably would make sense to improve/adapt the documentation in a few places as well :) )

Hi here @jphme thanks for the issue and the PR straight away! Our idea with this was mainly for the tasks that were not returning a Prompt so that one could define a custom Task and then override the prompt-processing function before it gets passed to the LLM.generate method. But fair, currently it has no point with the tasks that we provide as part of distilabel, so before merging we'll review the current situation of the prompt_formatting_fn arg and define whether it's needed or not. Thanks! 🤗

I figured that this is the reason but in the current implementation Task should always return a Prompt (see here ) and it also makes sense to always to the prompt formatting stuff on the LLM side of things conceptually.
The current implementation leads to silent errors (prompt format is silently ignored without any warnings or errors) which is imho the worst choice. And additionally you cant format the prompts any other way than with the (limited) default templates. My change would make prompt_formatting_fn consistent with the default prompt formats (taking precedence if both are provided).
(But it probably would make sense to improve/adapt the documentation in a few places as well, happy to help with that if you´ve decided on the preferred way forward :) )

@alvarobartt Any news on the review of prompt_formatting_fn?

Hi @jphme, yes, indeed we're currently going through a refactor and we're taking your feedback on this into consideration and the prompt_formatting_fn will now be provided to the LLM, and it will assume OpenAI as the input format and the the formatting will be applied and converted into whatever format is desired.
We saw some flaws in the current implementation, also thanks to the community issues as this one, and decided to proceed with a refactor that will clear a bit those things and the logic will be more straight forward but at the same time more scalable and flexible. We'll keep you posted, but thanks again for the feedback 🤗

@alvarobartt Any news on the review of prompt_formatting_fn?

Hi @jphme, yes, indeed we're currently going through a refactor and we're taking your feedback on this into consideration and the prompt_formatting_fn will now be provided to the LLM, and it will assume OpenAI as the input format and the the formatting will be applied and converted into whatever format is desired.
We saw some flaws in the current implementation, also thanks to the community issues as this one, and decided to proceed with a refactor that will clear a bit those things and the logic will be more straight forward but at the same time more scalable and flexible. We'll keep you posted, but thanks again for the feedback 🤗

Thanks for letting me know, surely makes sense. When trying to work around this, I encountered another bug (see #350 ) which prevents the use of OpenAI compatible endpoints without a model_list.
If you´re refactoring currently: Is the task parameter at LLM creation really necessary? I know it´s used for tokenization, but imho the API would be alot nicer, if Tasks and LLMs could be created independently of each other.

If you´re refactoring currently: Is the task parameter at LLM creation really necessary? I know it´s used for tokenization, but imho the API would be alot nicer, if Tasks and LLMs could be created independently of each other.

Yes, indeed that's the case now, the LLM and the Task are defined independently and then the Task uses the LLM to run only the inference, so that the formatting and such is within the Task itself.
Long-story-short we went from LLM has Task to Task uses LLM.


refactor(gui): Only enable full debug output on demand

This disables full wildcard debug output by default now,
leaving the possibility to manually enable debug output
via the DEBUG environment variable.
Change-Type: patch

Looks good to me. I'm curious though, what prompted the change?

A noisy neighbour called drivelist :)

Hmmm, so @jviotti spends lots of time adding debug code so that he can see what's going wrong, and then @jhermsmeier turns it all off again? ;-)
Are we really expecting end-users to be able to set the DEBUG environment variable when we ask them for help with tracking down what's causing them problems? Or have I grabbed the wrong end of the stick yet again?

Maybe a better approach is to blacklist the ones we're not interested in? Does debug support logging "levels"? If so, maybe we can use that to still see valuable information, but ignore unnecessary things?

Are we really expecting end-users to be able to set the DEBUG environment variable when we ask them for help with tracking down what's causing them problems? Or have I grabbed the wrong end of the stick yet again?

As far as I can see the only debug user so far is drivelist (and some stuff in pipage will be using it as well) – and that's generally really internal state debug output, not the kind we've got going for mountuils and such. So, I don't think we'll need users to enable that.

Maybe a better approach is to blacklist the ones we're not interested in? Does debug support logging "levels"? If so, maybe we can use that to still see valuable information, but ignore unnecessary things?

It doesn't have logging levels as such, but the DEBUG value is a selector, so you can do stuff like
# Debug everything expect drivelist
DEBUG="*,-drivelist"
# Debug only drivelist, and blockmap readstream
DEBUG="drivelist*,blockmap:readstream*"

Details here

Also, I didn't turn it all off – I just made it come on selectively ;P

As far as I can see the only debug user so far is drivelist

Oh, so is there some other kind of debugging used in e.g. mountutils? Should the different approaches be harmonised?
Apologies if I'm misunderstanding again, but is it worth adding any of this 'debug support' stuff to the Robot interface?

Also, I didn't turn it all off – I just made it come on selectively ;P

I was only pulling your leg, hence the smiley ;-)

Oh, so is there some other kind of debugging used in e.g. mountutils? Should the different approaches be harmonised?

Yeah, the mountutils debug output is coming from C++ land, hence the debug module wasn't used. And yup, I think having a standard approach to debug output would be sweet, preferably with debug (because being able to selectively enable only what's needed is splendid).

Maybe we should create a node-debug-cpp add-on :)

@jhermsmeier I've been taking a look at this, and I have an alternative proposal:

We can not log "OK" if the temporary script unlink phase was successful, and instead only log the error if an error happens (the OK acknowledge is not necessary and by removing it we cut 50% of the logs)

Instead of trying to unlink the file and not check its callback, maybe we can only proceed once the unlink function ended (successfully or not), to be able to correlate any unlink errors with a specific execution

Not disable global debugging, since the amount of logs will now probably be fine

What do you guys think?

From the example output Jonas posted above, I guess you'd also need to modify drivelist's write to not output anything if it succeeded?
BTW, does the +1s in the screenshot above mean that it took a whole second to write out the drivelist .sh file? That sounds awfully slow for such a small file? :-/  (or does it actually indicate something else?)

From the example output Jonas posted above, I guess you'd also need to modify drivelist's write to not output anything if it succeeded?

I guess knowing the final script path is good for debugging purposes.

BTW, does the +1s in the screenshot above mean that it took a whole second to write out the drivelist .sh file? That sounds awfully slow for such a small file? :-/ (or does it actually indicate something else?)

I think it represents the time between that message and the previous one, and I know for sure that drivelist is slow (which is also a reason why I want to go full native on it). I send a PR that makes things better: https://github.com/resin-io-modules/drivelist/pull/194, but still, running the actual script is the bottleneck

BTW, does the +1s in the screenshot above mean that it took a whole second to write out the drivelist .sh file?

Those times are times between the debug messages as @jviotti pointed out, meaning drivelist is called every second, and it takes ~750ms in total until it's done writing out, executing & deleting the script.

@jhermsmeier I've been taking a look at this, and I have an alternative proposal:

@jviotti I still don't think that we should have debug output enabled at all in a build, unless manually enabled. It has zero value 95% of the time, and cluttering up the console has a cost that we shouldn't ignore (if all of it would be written to a logfile that would be nuked on successful flash, that'd be a different story). Plus logging to the console is really slow compared to anything else, so this also has a speed impact.
I would counter-propose completely disabling it by default (as this PR does), but adding a global DEBUG() function users could call in the console should it become necessary. For example DEBUG() could toggle localStorage.debug = '*' and process.env['DEBUG'] = '*', and when called again, would disable it again.
What do you think?

Just to add my 2 cents: In the recently-released (and then pulled) v1.1.0 release it was really annoying having drivelist write spam the DevTools console every 2 seconds, and it makes the logs much harder to read. And of course it also results in the 'useful logs' quickly scrolling off the top of the screen :-/
So iMHO this part (at least) should be disabled by default ;-)
@jhermsmeier I'd be :-1: on a DEBUG() function that added as a 'toggle' - wouldn't it be better to have DEBUG('*') turn on debugging, and DEBUG('') (or DEBUG()) turn off debugging? Of course those functions in turn could be wrapped by ENABLE_DEBUG() and DISABLE_DEBUG() functions.

@lurch @jviotti I've added a DEBUG() function now, that works like this:
// Enable output for all subsystems:
DEBUG('*')
// Enable output for a specific subsystem:
DEBUG('etcher:drive-scanner*')
// Disable all debug output by passing any false-y value:
DEBUG(); DEBUG(false); DEBUG(''); DEBUG(null); DEBUG(undefined); DEBUG(0)

@jviotti is this still good to merge?

Some docs to go along with this (i.e. how to enable debugging messages when required) would be useful ;-)

I'll approve, but I agree with @lurch that some docs would be very nice.

Ah, never mind, I had already approved :P


Switched to noUiSlider and Found Problem in auth.js

I switched to noUiSlider instead of using jQuery UI, which was pretty simple.
However, I also changed the console.logs  and the invocation to callback in Authentication_Login.logUserIn() to be done inside of the .doesUserExist() callback because that's when the user is actually logged in/created. However, when this is done, it is easy to see that the user is never actually logged in because the .doesUserExist() callback is never called because, for some reason, the callback to usersRef.once("value", ...) is never called! I tried running the following script and it did not log anything:
var fbRef = new Firebase("https://contest-judging-sys.firebaseio.com");
var usersRef = fbRef.child("users");
usersRef.once("value", function(snapshot) { console.log(snapshot.val()); });

I have no idea why this isn't working, but it clearly isn't.

The reason that script didn't log anything, is because the value event is only fired once data has been queried from Firebase (short answer: You have no Firebase Query).
Auth.js is going to be removed soon, and replaced with a much better authentication system that I've been working on.
Will review the noUiSlider stuff whenever I get a chance.

Ah...Thanks for the explanation on auth.js!

I undid the merge of this PR, because it broke everything judging related (see screenshot).
http://take.ms/gOlgG

That shouldn't have happened and it's not happening on my version of the app. I got rid of all calls to $(...).slider() after getting rid of jQuery UI. You must've made other changes somewhere and added that call in, so you have to convert it to noUiSlider.

Why didn't you merge my changes to entry.js in? That's what messed everything up:
https://github.com/Team-Delta-KA/KA-Contest-Judging-System/blob/master/Client/scripts/entry.js#L168

It looks like for some reason, you had to merge it manually and you didn't merge my noUiSlider in. It got deleted during the merge: https://github.com/Team-Delta-KA/KA-Contest-Judging-System/commit/ff911e356ed4d6faf35dae25a31f48b58aa763fc

Wait...You clearly merged my noUiSlider changes in. I don't see how that could've happened in entry.js because there is no $(...).slider(): https://github.com/Team-Delta-KA/KA-Contest-Judging-System/commit/3a8423219e067e99b1e3ca17cbd557b55ccba7b8


not show link to migration on repo reate page when it was disabled

As title.

Lint failed in tmpl file


⚠ Bofrost.de has degraded performance

In 0a639b8, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1098 ms

Resolved: Bofrost.de performance has improved in d5ab250.


Provide means to enforce ordering for xml schema processing

Provide a way to define an order in which extension schemas are looked up and processed by the Ehacache xml configuration parser. Such requirement will arise when an existing ehcache extension schema like the clustered extension is extended further. In that case the ehcache clustered extension should be loaded and processed before the new extension is processed.

Fixed as of 3.10.0 (e7e4d17484e1f2972addd3cab6243545ebf1e1ca)


Support for 2016 and deployment without multi tenet environment

Hi,
Can we have windows 2016 support in windows? also i have noticed that current ansible is designed for multi-tenet deployment. Can we have the palybook created for non-multi-tenant DSM deployments?

Hi @tsheth ,
Thanks for using our playbook.
This playbook was verified on various platforms, including Windows 2016.
However, we couldn't update the support list as Ansible Galaxy only offer 2012R2 for windows:
https://galaxy.ansible.com/api/v1/platforms/
About the multi-tenant part, really sorry for the confusing, our playbook is not limited to multi-tenant deployment only.
You can use it without "tenant_id", and your DSA will be activated to T0, which is used by the non-multi-tenant DSM deployments.
We'll create an internal case to update the document to avoid the confusing afterward.
Please let us know if you encounter any problem.
Best Regards,
Deep Security Open Source Support team

Hi @tsheth,
We've updated the README to address the support for non-multi-tenant DeepSecurity Manager.
Please let us know if you have other concerns.
Best Regards,
James


Blog app: add authorization rules

The project requireents
Install CanCanCan in the project.
Add a role column to the users table.
A user can delete a post if it is theirs or if they have an admin role (column role has value "admin"). Use CanCanCan for this authorization.
For that you need to implement the post deleting functionality. Add the "Delete" button to the view and make sure that only authorized users can see it.
A user can delete a comment if it is theirs or if they have an admin role (column role has value "admin"). Use CanCanCan for this authorization.
For that you need to implement the comment deleting functionality. Add the "Delete" button to the view and make sure that only authorized users can see it and
This PR adds authorization rules. Some of the notable changes include:
Added Roles Admin and user to the Users table on the schema
Added Delete button for posts for both admins and users
Added Delete button for comments for both admins and users

Hi @franclobo,
Thank you!


Secret key broken

Need to create API to add the secret key to auth requests. The current token retrieval process is broken.

Add a Digital signature to requests to avoid forgery response.
https://github.com/RubyCrypto/rbnacl/wiki/Digital-Signatures

Resolved


Bug: Inviting user resulting in 503 error

Issue Summary
Unable to send invitations to the user, the POST /api/v2/admin/invites returning a 503 response. No email is sent to the user, although refreshing the blog shows the invited user under the staff tab.
These are the Ghost logs that I received on sending the user invites.
To Reproduce

Click on staff
Click on invite people
Enter the email, choose the role and click on send invitation now

Technical details:

Ghost Version: 2.25.9
Node Version: 10.16.0
Browser/OS: Brave/ macOS Mojave
Database: MySQL v5.7.27

Hi @utkarsh-var :wave: Have you followed our guide to set up email configuration?
Just tried it locally and it worked just fine with default configuration running Admin API v2.

@gargol @utkarsh-var Tried locally and works fine for me too. @utkarsh-var If you have hosted your blog on GCP, may be this would help: https://ghost.org/faq/mail-config-error-google-cloud/

I'm not sure if it is related or not, but I am having an issue sending mail to users on a different domain.  Ie i can successfully invite users on myblogdomain.com, but it gives a 500 error and doesn't send the email when inviting<EMAIL_ADDRESS>

Looks like it's an issue with misconfiguration of email service. Feel free to raise a topic on our forum  to go through the details and we can move it here once it's a confirmed bug :wink:

The issue is solved after adding Mailgun for the email service. Thanks, @gargol @lunaticmonk


is a bug??

I have this routes
FlowRouter.route('/test', {
    action: function() {
        FlowLayout.render('mainLayout', {area: 'test'});
    }
});

FlowRouter.route('/', {
    action: function() {
        FlowRouter.go('/test');
    }
});

Locally is we go to URL: localhost:3000/
"test" template is not rendered and got a Exception:
Exception from Tracker recompute function:
debug.js:41Error: You can't use reactive data sources like Session inside the `.subscriptions` method!
    at utils.js:133
    at Tracker.Computation._compute (tracker.js:294)
    at Tracker.Computation._recompute (tracker.js:313)
    at Object.Tracker._runFlush (tracker.js:452)
    at onGlobalMessage (setimmediate.js:102)

I put on Meteorpad http://meteorpad.com/pad/puMYoFS3hFSddSop2/flowrouter-test and there it work
fine. Tested with meteorpad URL http://app-******.meteorpad.com/

It's a bug right now. If you need to do this, use a trigger or do this
inside a setTimeout, until we do a fix.
On 2015 ජූනි 6, සෙන at පෙ.ව. 2.39 bySabi<EMAIL_ADDRESS>wrote:

I have this routes
FlowRouter.route('/test', {
action: function() {
FlowLayout.render('mainLayout', {area: 'test'});
}
});
FlowRouter.route('/', {
action: function() {
FlowRouter.go('/test');
}
});
Locally is we go to URL: localhost:3000/
"test" template is not rendered and got a Exception:
Exception from Tracker recompute function:
debug.js:41Error: You can't use reactive data sources like Session inside the .subscriptions method!
at utils.js:133
at Tracker.Computation._compute (tracker.js:294)
at Tracker.Computation._recompute (tracker.js:313)
at Object.Tracker._runFlush (tracker.js:452)
at onGlobalMessage (setimmediate.js:102)
I put on Meteorpad
http://meteorpad.com/pad/puMYoFS3hFSddSop2/flowrouter-test and there it
work
fine. Test with this http://app-wfgn2elu.meteorpad.com
—
Reply to this email directly or view it on GitHub
https://github.com/meteorhacks/flow-router/issues/145.