[FRONTEND][TFLITE] Add support for TFLite's regular NMS operator

This PR adds support of regular NMS operator.
Open questions:

How to properly test added functionality?
Other NMS implementations, e.g., fast NMS, use a TF frozen graph from TF official website to convert a model to TFLite and keep NMS operations only. In order to create a similar test, we need to find an archive on TF official website that contains a frozen graph of a model compiled with --use-regular-nms=True flag. We haven't found it yet, so any help is appreciated.
Regular NMS requires two sort operations:

Sorting the scores after selecting scores above nms_score_threshold. This PR implements this with a simple bubble sort in order to prove the algorithm's semantics. We tried to replace it with tvm.contrib.sort.argsort. It works well when testing the regular NMS with run_tvm_graph as fast NMS test does or building and running the regular NMS with llvm target. At the same time, it fails to build (error is provided below) when target=ethos-u,cmsis-nn,c. It seems that __tvm_module_ctx variable is only being initialized when cpp runtime is chosen.
The error:
error: '__tvm_module_ctx' undeclared (first use in this function) 203 | if (TVMBackendGetFuncFromEnv(__tvm_module_ctx, "tvm.contrib.sort.argsort", &tvm_contrib_sort_argsort_packed) != 0) {
Sorting the scores of previous and current NMS steps. There are two alternatives here:

implement some sorting algorithm as part of hybrid script (to replace current bubble sort)
save the result of each NMS step and use argsort after the hybrid script part. This approach has a drawback as it requires significant amount of memory to store the results of each NMS step.

@leandron @Aleksei-grovety

cc @ekalda

Thanks for the patch @ilyag-grovety, looks all very neat and good to me. Hopefully the CI will go green as well once the model URL will be properly enabled.
Regarding to testing, from my research, I didn't find a way to create that operator through current TF or Keras APIs either, so I think using the model consisting of this operator is the best approach here.
I think using simple sorting algorithm is fine for now. I don't know what it would take to enable argsort for cpp runtime, but any effort of that kind should be a separate patch anyway. If bubble sort becomes a massive performance bottleneck in the future, then it is possible to further enhance this operator with enabling argsort or implementing a different sorting algorithm, but for now it should be fine.

Hi @mehrdadh! Could you please upload [the model] (https://github.com/Grovety/ModelZoo/raw/52fb82156ae8c8e3f62c7d7caf6867b25261dda4/models/object_detection/ssd_mobilenet_v1/tflite_int8/tflite_graph_with_regular_nms.pb)to S3? It's necessary for testing NMS.

@tvm-bot rerun


Raise db scope events to orbitdb instance scope event emitter

Adds a custom event emitter class to raise any events emitted by the db to the instance scope.
This enables listening to all db event types before the specific store has been opened / created.
Backwards compatibility is maintained by emitting db scope events as per before.
Requires orbitdb/orbit-db/pull/713

I like this idea. What do you think about moving the event relay from here to the createStore function in orbit-db proper, and making it a configuration option there?

@aphelionz e.g. https://github.com/orbitdb/orbit-db/pull/713/files#diff-f1c4721cce5f33ff9a226802788fe12aR444https://github.com/orbitdb/orbit-db/pull/713/files#diff-f1c4721cce5f33ff9a226802788fe12aR444 ?

Or do you mean the relay class itself, to make the class a configurable option?

Not quite sure yet and probably need to sleep on it... we had discussed a while back about utilizng more event passing in general, not just outward but internally between all the different components of the system.

Closing this based on conversation in Gitter, will be moving this functionality to orbit-db


Build the project manually

1. # export JAVA_TOOL_OPTIONS="-Dhttps.protocols=TLSv1.2" <-- to fix the HTTPS issues
2. rmiscout# ./gradlew shadowJar <-- to be checked the following errors
Picked up JAVA_TOOL_OPTIONS: -Dhttps.protocols=TLSv1.2
FAILURE: Build failed with an exception.

Where:
Build file '/root/rmiscout/build.gradle' line: 4

What went wrong:
Plugin [id: 'com.github.johnrengelman.shadow', version: '2.0.4'] was not found in any of the following sources:

Gradle Core Plugins (plugin is not in 'org.gradle' namespace)
Plugin Repositories (could not resolve plugin artifact 'com.github.johnrengelman.shadow:com.github.johnrengelman.shadow.gradle.plugin:2.0.4')
Searched in the following repositories:
Gradle Central Plugin Repository

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 5.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/4.10.3/userguide/command_line_interface.html#sec:command_line_warnings
BUILD FAILED in 0s

Hi alt3kx,
Hmmm... I'm not able to reproduce this one (I tried a variety of JDK versions). It appears the JDK configuration you are using is not enabling TLSv1.2 by default or has been modified. As best I understand, it's been the default for a while (https://blogs.oracle.com/java-platform-group/jdk-8-will-use-tls-12-as-default). See: https://github.com/gradle/gradle/issues/5940. It looks like beta/EA JDK builds set odd defaults.
Alternatively, here is a basic working Docker image:
FROM openjdk:8
COPY . /rmiscout
WORKDIR /rmiscout
RUN ./gradlew shadowJar
ENTRYPOINT ["./rmiscout.sh"]
CMD ["-h"]

Let me know if you think it's a bug with rmiscout gradle configuration or a JDK configuration issue.
Thank you!

Closing this issue as it appears to be a local JDK configuration issue. Thanks for the report!


Where to add the AppEUI when adding application on UI?

Just now, I setup the loraserver thru docker compose, When I Add an application, I can not find the place where to add the AppEUI on the UI, without this, the device can not join successfully now.
If the APPEUI  is nessasary ,where to add it? If it is not,what is the problem?Thanks.

https://forum.loraserver.io/t/where-do-i-set-the-appeui/381
This help me. In the latest version, you don’t need to set this anymore.


Weird numeric format

Hello,
Thanks for this tool and its documentation.
I just start using it, but I noticed a weird behavior: the numbers have two commas . here is an image example

While this may be a visual clue, it is a bit weird. I would prefer the units to collapse (to KB, or to MB) when possible.
Thanks a regards

That's just how Americans write numbers :)
I'd be open to a PR that either added localized formats or used kb, mb, etc.
On Sat, Jun 24, 2017 at 5:16 PM Daniel Rodríguez Rivero <
<EMAIL_ADDRESS>wrote:

Hello,
Thanks for this tool and its documentation.
I just start using it, but I noticed a weird behavior: the numbers have
two commas . here is an image example
[image: image]
https://user-images.githubusercontent.com/2270425/27511992-11d6483c-5933-11e7-80bf-e17f2dc51730.png
While this may be a visual clue, it is a bit weird. I would prefer the
units to collapse (to KB, or to MB) when possible.
Thanks a regards
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/danvk/travis-weigh-in/issues/13, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAF__dSRwd23Rqp7hahc0rQevHYWJwMYks5sHXyqgaJpZM4OEcSY
.


Feature - split py settings into seperate file that can be ignored by git

Addresses #20 by separating settings/secretes from polling script into file that can be gitignored.

Nice idea !
Maybe use an INI/YAML format for python settings file and using the open function:
        config = os.path.join(os.path.dirname(__file__), "settings.yaml")
        with open(config, "r") as f:
            settings = yaml.load(f)

What do you think ?
Thanks for your help, really appreciate !

Yep, good point. Thinking more, perhaps it would be best to point both the frontend and backend to the a single canonical settings file. After all, many settings (for DB and NEST API) are common for each. In this regard I certainly prefer yaml (as my background is PHP and frameworks like Drupal and Symfony), but in the interest of time it may be easiest to stick to ini. From what I can see both php and python have native parsing for ini, and I think additional libs would be needed for yaml. Anyway, I'm pretty new to python so I may not be seeing all the variables involved there.

I just pushed a proof-of-concept for canonical settings (in ini) format. This may still be a little rough, but it's something specific for additional discussion.


[PE-D] Suggestions to UG

Maybe can expand more on this? In terms of the user, they may not be proficient with computers and thus may not be sure how to check for the java version

Labels: severity.Low type.DocumentationBug
original: DannyDakota/ped#6

Noted, I'll try to see if I can find a SO answer and link it to this part, or failing that, come up with an additional section under it with more instructions.


update workflows that use set-output

Workflows (e.g.,https://github.com/nmfs-stock-synthesis/workflows/blob/85f79236758b28469b756acd5d3c92e9615b5c25/.github/workflows/update-ss3-version.yml#LL32C11-L33C69) in this repo use set-output, which will be deprecated by github in mid 2023. For now, a warning is generated on the github action runs that have not been updated.
All that needs to be done is that the set-output lines need to be changed slightly. For example,
echo "::set-output name=link_to_release::${link_to_release}"

should be changed to
echo "link_to_release=$link_to_release" >> $GITHUB_OUTPUT

(EDIT: curly braces around link_to_release are optional, so I removed them!)
An example of the change in another repo: https://github.com/noaa-fisheries-integrated-toolbox/toolbox_web_templating/commit/076a2ae87369593955a57a7480833eb74071fe31
See https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/ for more details.
@e-gugliotti-NOAA is this something you could work on?

Yes, I can work on this. I've changed all the occurrences I see but I do
have a couple questions that I can ask you about tomorrow.
Elizabeth F. Gugliotti (she/her)
Research Associate
ECS Federal contracted to
NOAA/NMFS/National Stock Assessment Program
Mobile:<PHONE_NUMBER>
On Tue, Dec 27, 2022 at 3:31 PM Kathryn Doering @.***>
wrote:

Workflows (e.g.,
https://github.com/nmfs-stock-synthesis/workflows/blob/85f79236758b28469b756acd5d3c92e9615b5c25/.github/workflows/update-ss3-version.yml#LL32C11-L33C69)
in this repo use set-output, which will be deprecated by github in mid
2023. For now, a warning is generated on the github action runs that have
not been updated.
All that needs to be done is that the set-output lines need to be changed
slightly. For example,
echo "::set-output name=link_to_release::${link_to_release}"
should be changed to
echo "link_to_release=${link_to_release}" >> $GITHUB_OUTPUT
An example of the change in another repo:
@.***
https://github.com/noaa-fisheries-integrated-toolbox/toolbox_web_templating/commit/076a2ae87369593955a57a7480833eb74071fe31
See
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
for more details.
@e-gugliotti-NOAA https://github.com/e-gugliotti-NOAA is this something
you could work on?
—
Reply to this email directly, view it on GitHub
https://github.com/nmfs-stock-synthesis/workflows/issues/62, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A4GU6UOALJRTY65ZDS7XNYLWPNGYHANCNFSM6AAAAAATKWKJJE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks, and sounds good!

@k-doering-NOAA can you relink that nmfs toolbox example? That might help me with the current issue because it's with an output call later on, not the initial place that we thought might be causing the error due to spacing.

I just gave you read permissions to the toolbox repo, I forgot it is set to private! Here is the toolbox example again.
My guess is windows is not parsing the  new output correctly, because it is working just fine on linux and mac. Seems like a github actions bug. (more info on using output parameters). I had an issue with one of the actions in ghactions4r that was similar.

I posted this discussion, hopefully so we can get some help: https://github.com/orgs/community/discussions/42790

Okay great! Yes, hopefully someone in the community will have an answer for
us!
Elizabeth F. Gugliotti (she/her)
Research Associate
ECS Federal contracted to
NOAA/NMFS/National Stock Assessment Program
Mobile:<PHONE_NUMBER>
On Wed, Dec 28, 2022 at 1:48 PM Kathryn Doering @.***>
wrote:

I posted this discussion, hopefully so we can get some help:
https://github.com/orgs/community/discussions/42790
—
Reply to this email directly, view it on GitHub
https://github.com/nmfs-stock-synthesis/workflows/issues/62#issuecomment-1366851777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4GU6UNBFQN2MTMARXXRRHDWPSDPVANCNFSM6AAAAAATKWKJJE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix typo

Proposed changes
I corrected a typo because it was a typo: UCP -> UDP.

Deploy preview ready!
Built with commit e06f390ef00c13b35c0e942bedaa9ce3f1c4a54f
https://deploy-preview-853--docker-docs.netlify.com

I looked at the changed page on the preview and it looks fine to me:
https://deploy-preview-853--docker-docs.netlify.com/engine/swarm/ingress/
What's next?

Thanks @aimichal :)


SCM shows stale diff contents after a pull

Reported by @sbatten

Open the same repo in insiders.vscode.dev and github.dev
Make a change to a file in insiders.vscode.dev
Make a non-conflicting change to the same file in github.dev
Commit from github.dev
Pull from insiders.vscode.dev (no merge conflicts should be flagged at this stage)
Open the diff from the SCM view
:bug: the diff reflects the combination of changes from the github.dev commit and the unpushed local commits. Clicking the refresh button in the SCM view is sufficient to fix the stale diff contents

Likely the LHS of the diff command still points to the SHA before pulling, and we need to recompute the SCM resource states and commands when the repository is synced to ensure that the diff editor reflects the correct contents.

I followed the steps above, did Commit & Push, pulled in VS Code Insider's Desktop on the same branch/same repo, and do not see the changes that I pushed in insiders.vscode.dev. But when I commit in Desktop and try to push, I'm told there are changes to pull 🤔 . Maybe a timing issue?

I tried the sequence again and it worked the second time around

I never saw a stale diff, so am considering this verified


translate.lua:3: attempt to index field 'ExtendedCmdLine' (a nil value)

I updated the package with git pull, when I use translate.lua got errors as below:
My env: Ubuntu 14.04 with Lua 5.2

torch/install/bin/lua: translate.lua:3: attempt to index field 'ExtendedCmdLine' (a nil value)
stack traceback:
translate.lua:3: in main chunk
[C]: in function 'dofile'
...nNMT/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:150: in main chunk
[C]: in ?

Can you check if the solution given in #117 works for you?

it's ok now, many thanks!
luarocks remove opennmt
luarocks make rocks/opennmt-scm-1.rockspec
ref:
https://github.com/OpenNMT/OpenNMT/issues/117#issuecomment-278649463


Env variables placeholder and orders

I was assessing the suitability of tye in a project and I have few questions that I don't know if they are possible.
1. Can we use placeholders in environment variable
consider following scheme, I want to pass the port of app2 into app1 as an environment variable.
name: tye-them-all
services:
- name: app1
  project: src/app1/app1.csproj
  env:
    - name: APP2_BASE_URL
      value: "http://localhost:{app2_port}/"
- name: app2
  project: src/app2/app2.csproj
  bindings:
    - autoAssignPort: true

2. Run initialisation scripts for projects and containers
We may need to run some initialisation scripts that run after a container/project is running.
name: tye-them-all
services:
- name: app1
  project: src/app1/app1.csproj
  # script should have access to ${postgres_port} env variable
  init_script: scripts/init-db.sh

- name: postgres
  image: postgres
  bindings:
  - autoAssignPort: true

3. Define orders
The previous suggestion will work better if we can define the order or dependencies between services.
name: tye-them-all
services:
- name: app1
  project: src/app1/app1.csproj
  # script should have access to ${postgres_port} env variable
  init_script: scripts/init-db.sh
  requires: ["postgres"]

- name: postgres
  image: postgres
  bindings:
  - autoAssignPort: true

Thank you for your help. :+1:

Can we use placeholders in environment variable

Right now, I don't believe we support environment variable replacements. We do for the connection strings, shown here.
However, I'm curious why you'd need to allow using the port as an environment variable. In your example, when you are in app1, you should have access to app2's port via configuration.
I'd check Microsoft.Tye.Extensions.Configuration, that should do what you need here.

Run initialisation scripts for projects and containers

I think this makes sense to add.

Define orders

We were discussing adding this primarily due to database startup, which you highlighted. cc @glennc

Thanks for your reply.

However, I'm curious why you'd need to allow using the port as an environment variable. In your example, when you are in app1, you should have access to app2's port via configuration.

It would be nicer to integrate without having to change the code to look for these environment variable (instead of say appsettings.json or however it's doing right now). I may not want to deploy using tye yet or modify the code (as well as all kubernetes config, secrets, Integration tests etc).
Of course these are easy when I start a new project but for existing project, the ability to include these ports with an existing variable will make the adaptation faster. I can try sending PR if it's something you'd consider but not sure if it's going to happen anytime soon.

Of course with an initialisation script and the ability to define order, I can map these environment variables using a bash script myself.

See https://github.com/dotnet/tye/issues/377 for 1.


Selecting a server or setting a custom endpoint

OpenAPI 3 allows for multiple Server Objects to be defined on a single document; however, it is not currently possible to select which server is wanted with the APIStar client, nor it is to set a custom base URL, except with setting the URL on the schema document itself and updating every Link URL.
I guess adding a base_url=... keyword argument to apistar.Client isn't hard, but how should one select an existing server from the schema, or list them? Currently, the URL from the first server is used: https://github.com/encode/apistar/blob/master/apistar/schemas/openapi.py#L357

Yup - not really sure just yet.
Thanks for digging into all this so much!
I've a bit pushed for time right now so might not be reviewing apistar tickets for the next few days, but I'll get back onto it in due course.

I just ran into this too.
According to the spec for the paths object, the path should be appended to the server object's url

The field name MUST begin with a slash. The path is appended (no relative URL resolution) to the expanded URL from the Server Object's url field in order to construct the full URL.

but OpenAPI.get_link uses urljoin to set the Link's url, which strips any path (like /api/v1) off the url that comes from the server object.
https://github.com/encode/apistar/blob/master/apistar/schemas/openapi.py#L465

+1
I tried to avoid this by stripping leading / in paths as follows, but this caused typesystem.base.ValidationError.
doc['paths'] = {
    path.lstrip('/'): path_item_obj
    for path, path_item_obj in doc['paths'].items()
}
apistar.Client(doc)

This is because properties of Paths object are required leading /.
https://github.com/encode/apistar/blob/2edeb694/apistar/schemas/openapi.py#L102

The required leading slash is a requirement from OpenAPI itself (see the Paths Object specs).
Here is the workaround we used:
from urllib.parse import urlsplit

class MyClient(apistar.Client):

    def __init__(self, *args, base_url=None, **kwargs):
        # Let APIStar do its parsing
        super().__init__(*args, **kwargs)

        # Strip scheme, hostname and absolute path from all link URLs
        for link_info in self.document.walk_links():
            original_url = urlsplit(link_info.link.url)
            new_url = ('', '', *original_url[2:])
            link_info.link.url = urlunsplit(new_url).lstrip('/')

        if base_url:
            # Ensure the base URL ends with a slash to prevent issues:
            # urljoin('http://a/b', 'c') → http://a/c
            # urljoin('http://a/b/', 'c') → http://a/b/c
            if not base_url.endswith('/'):
                base_url += '/'

            self.document.base_url = base_url

This goes through every parsed link in the Document instance to remove the scheme, hostname and leading slash, resulting in every API endpoint having a relative URL (a/b/c instead of http://myserver.com/a/b/c), then updates Document.url to a custom base URL when specified. This then allows Client.get_url to still use urljoin and do any of its checks, but with any base URL, including custom base paths.


History feed download is not downloading properly

Continuing the new viewing history feature in #1821 -
When running the following command:
$ youtube-dl ':ythistory' -u USER -p PWD --write-pages

The result is:
[youtube:history] Logging in
[youtube:history] History: Downloading webpage
[youtube:history] Saving request to History_https_-_www.youtube.com_feed_history.dump
[youtube:history] history feed: Downloading page 0
[youtube:history] Saving request to history_feed_http_-_www.youtube.com_feed_ajaxaction_load_personal_feed=1_feed_name=history_paging=0.dump
[youtube:history] history feed: Downloading page 1
[youtube:history] Saving request to history_feed_http_-_www.youtube.com_feed_ajaxaction_load_personal_feed=1_feed_name=history_paging=1384803931993001.dump
[youtube:history] history feed: Downloading page 2
[youtube:history] Saving request to history_feed_http_-_www.youtube.com_feed_ajaxaction_load_personal_feed=1_feed_name=history_paging=2769607863986002.dump
[youtube:history] history feed: Downloading page 3
[youtube:history] Saving request to history_feed_http_-_www.youtube.com_feed_ajaxaction_load_personal_feed=1_feed_name=history_paging=4154411795979003.dump
[youtube:history] history feed: Downloading page 4

with the following files created:
-rw-r--r--   1 user  staff  355768 Nov 27 22:58 History_https_-_www.youtube.com_feed_history.dump
-rw-r--r--   1 user  staff  353934 Nov 27 22:58 history_feed_http_-_www.youtube.com_feed_ajaxaction_load_personal_feed=1_feed_name=history_paging=0.dump
-rw-r--r--   1 user  staff  384896 Nov 27 22:58 history_feed_http_-_www.youtube.com_feed_ajaxaction_load_personal_feed=1_feed_name=history_paging=1384803931993001.dump
-rw-r--r--   1 user  staff  353934 Nov 27 22:58 history_feed_http_-_www.youtube.com_feed_ajaxaction_load_personal_feed=1_feed_name=history_paging=2769607863986002.dump
-rw-r--r--   1 user  staff  353934 Nov 27 22:58 history_feed_http_-_www.youtube.com_feed_ajaxaction_load_personal_feed=1_feed_name=history_paging=4154411795979003.dump

As you can see (and can be confirmed with a hash check), all files beside the second dump are exactly the same. I let this run for a long time and it got to a few thousand pages, so this is likely a bug.

How can I use the script from jaimeMF to get the TITLE and the URL. When I change print(entry['url']) to print(entry['title']) it tells me invalid argument or something like that.


Inconsistent indentation for a begin ... end block

Hello, I'm getting an error with the following code
begin :associations
  belongs_to :account
  has_many :things
end

Offenses:

test.rb:2:3: C: Inconsistent indentation detected.
  belongs_to :account
  ^^^^^^^^^^^^^^^^^^^
test.rb:3:3: C: Inconsistent indentation detected.
  has_many :things
  ^^^^^^^^^^^^^^^^

1 file inspected, 2 offenses detected

It seems to not detect the begin/end block

begin :associations
  belongs_to :account
  has_many :things
end

There's some symbol after begin, which seems pretty odd to me.

It is somewhat unusual, we use it for code folding as being/end blocks are recognised by most editors, and the symbol is just used for naming the block, it kind of doesn't do anything.

Btw, your example unveiled a bug in the Void check and in the latest RuboCop version you'll also start getting a warning about a symbol literal used in void context.
I understand your reasoning, but as this use-case is highly unidiomatic it doesn't make sense to add support for it in RuboCop.

:)
Well I'm happy to close this issue as I know it's not idiomatic


N-D structure to hold the datasets to be concatenated

Unsure of what the best structure to hold the datasets to be concatenated is.
I tried a numpy.ndarray but that did not play well with xarray, as discussed at pydata/xarray#2159.
Might have to resort to using a list of lists?
Whatever structure I use should be something that is easy for a user to create if necessary.

This stackoverflow answer seems to say that numpy arrays of dtype='object' will behave badly when trying to store anything which has a __len__ attribute.
Unfortunately the method proposed on the xarray issue above seems way more complicated than necessary for my purposes.
One (hacky) solution might be to use numpy to store an object whose only purpose is to contain a single xarray.Dataset but without exposing a __len__ or __getitem__ attribute, something like
class box:
    def __init__(self, thing):
        self.thing = thing

    def item:
        return self.thing

On the other hand if the only numpy functionality I'm using is np.apply_along_axis and np.ndenumerate then it might be easier to just use a list of lists, and recursively call a function (like this one) which concatenates the objects residing in the deepest lists.

I went for the "hacky" solution, but simplified by using a dictionary as the container class instead of defining a class just for this purpose, i.e.
split_data = [{'key': ds1}, {'key': ds2}]
ds = _concat_nd(split_data, dims=['dim1'])

This works, and the code can be seen at commit 303bfc868da8bb7ee742f1c2b981e107a44f975c.
Unless someone from the xarray community can suggest a much easier way of doing this then I will close this issue.


Investigate Geographic accuracy of Open Drone Map 2.1.0 (fallback to testing 2.0.0 if needed)

Task to do
A major version number change to the release of ODM needs to be checked out to see if geographical issues with and without GCP are resolved
Reason
Desire to provide automated ODM processing as part of workflows
Result
A determination whether v 2.1.0 or 2.0.0 provides a good solution.

Next step is to make sure the image's GCPs are accurate

made slight adjustments to GCP locations in GCP files since these are best-guess locations. This provided no real improvement on orthomosaic positioning in relation to plots shapefile

Also, added full set of images that contain GCP points to gcp file. This has produced the best orthomosaic image yet. There are still discrepancies between the plots on the image and the shapefile that would be resolved by having accurate GCP coordinates.

Conclusion: OpenDroneMap 2.1.0 produces viable orthomosaic images that, with the inclusion of CGP, are geographically correct. For the CIMMYT data it was necessary to specify the use_fixed_camera_params: True option through a parameters override file.


Tenant tier retrieval API call throws CORS error

Describe the issue:
$subject

How to reproduce:

Sign into console
Observe the network requests being sent.

Expected behavior:

This API should not be called in IS.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

Let's try to stop the /tier call by checking the existence of the of the API endpoint URL without having to use a config to stop the call

Verified the fix in the latest build.
https://github.com/wso2/product-is/assets/41533942/e800e577-bcf0-41d7-952d-2333974b9a5a


[Question] Is there a way to parse annotations from libraries?

I have a central library, that provides a common route for all my services. Is it possible to parse the annotations in that library, so that it can be tested per service via swagger ui?

Yes, it is possible to parse the annotations in your central library and generate Swagger documentation for each service.

I have a central library, that provides a common route for all my services. Is it possible to parse the annotations in that library, so that it can be tested per service via swagger ui?

Yes, it is possible to parse the annotations in your central library and generate Swagger documentation for each service.

@klaveriuzent would you mind sharing more insight on how to do this? I had no success yet - is this documented somewhere and I've overlooked it?

@klaveriuzent would you mind sharing more insight on how to do this? I had no success yet - is this documented somewhere and I've overlooked it?

While I may not be able to provide specific implementation details, here's a general outline of the four steps you need to follow:

Identify the annotations used in your central library: Review the annotations utilized in your central library, which define the API routes and parameters. Pay attention to annotations specific to your chosen framework or library.

Implement an annotation parser: Develop a parser capable of extracting relevant information from the annotations in your central library. Use reflection or framework-specific utilities to analyze the annotations and retrieve the necessary details such as routes, HTTP methods, and additional parameters.

Generate Swagger documentation: Once you have parsed the annotations and obtained the required information, leverage it to generate Swagger documentation. There are various libraries and tools available in different programming languages that can assist in programmatically generating Swagger or OpenAPI documentation.

Integrate with Swagger UI: Swagger UI is a popular tool for visualizing and testing APIs using Swagger or OpenAPI documentation. Ensure the generated Swagger documentation is integrated with Swagger UI in each service. This integration allows the API endpoints and their documentation to be accessible through the Swagger UI interface.

In general, having prior experience with MVC can make the process of parsing annotations and integrating with Swagger UI more intuitive. By leveraging the existing MVC structure and conventions, you can align the annotation parsing and Swagger documentation generation with the organization of your controllers. This familiarity with MVC principles can facilitate a smoother implementation of the overall process.
Then here what u need https://swagger.io/docs/

I have same issue for this. example:

I create a common library and define some model struct in it.

I create server app and use these model , and try use these model in app's code,

go build  <== successful
swag init <== fail with ParseComment error in file xxxx\http.go :cannot find type definition: model.CreateUserRequest

this is my demo code, If need, I can create a demo project.
@klaveriuzent
can swag support this ? thank you very much and waiting for your response.

import "github.com/xxx/common/model"

// @summary	Create User
// @tags	User
// @Accept	json
// @Produce	json
// @Param	info	body	model.CreateUserRequest	true	"request"               <== swag init fail for this.
// @Router	/api/v1/user [post]
func (server *HttpServer) createUser(ctx *gin.Context) {
	createRequest := model.CreateUserRequest{}
	err := ctx.Bind(&createRequest)     <== build successful
        ...
}

After more check, --pd  can Parse go files inside dependency folder


Avoid unecessary dependency on pytest-aiohttp module

If you look at the pytest-aiohttp module you will see it contains no code, it is just a wrapper silently doing the single line of this PR ;-)

But why? :) With this dependency we abstract away from implementation that might change in future. Also it is recommended way for doing testing by aiohttp author :)

Anyway thanks for your efforts

Well that's one less package to create and clearly it's doing nothing but this. If you consider one day having this package in Debian or Ubuntu official repository there's absolutely NO WAY to depends on a package like pytest-aiohttp, it will never be accepted as it contains absolutely nothing.
If so the package will be patched the way I did :p

Do I understand you correct - you package each dependency as a debian package, right?

Yup

Sorry, but this should be closed. It is very limited case and backporting internals of another package that may change  in future is not a good idea as for me.


[Bug]: Inpainting models aren't working because filename is renamed without -inpainting suffix

What happened?
Inpainting models aren't working right because the filename is changed when downloaded (or uploaded), which removes the -inpainting suffix at the end of the filename. The filename has to end with -inpainting.ckpt in order to work as an inpainting model in Auto1111. There is a PR for Auto1111 that changes this behavior that will recognize inpainting models without the suffix, but I don't think it has been implemented yet. Does Civitai rename the files when they are uploaded or downloaded?
Steps to reproduce the problem

Download an inpainting model
The model won't load in Auto1111 and throws errors
The model filename has to be renamed with -inpainting at the end, before the .ckpt extension, to get it to work

What should have happened?
The filename has to keep the -inpainting suffix at the end in order for Auto1111 to recognize the model as an inpainting model.
What platforms do you use to access the site?
Windows
What browsers do you use to access the site?
Microsoft Edge
Additional information, context and logs
No response

Thanks for the heads up on this. I'll address this now.

@Lushness I think I fixed this. Do you know a model that was causing issues so I can confirm?

@JustMaier it was this NSFW inpainting model. I think you fixed it, as it shows -inpainting at the end of the filename now when downloaded. Thanks!


gitignore not working

Hey all, I've been trying to get a gitignore to work with this package and I cant seem to get it to work. The gitignore is in the root of the repository.
I'm doing the following. Am I missing something?
	pushOptions := &git.PushOptions{
		RemoteName: "origin",
		Progress:   os.Stdout,
	}

	if client.Auth != nil {
		pushOptions.Auth = client.Auth
	}

	path := fmt.Sprintf("./_tmp/%s", client.Name)
	r, err := git.PlainOpen(path)

	if err != nil {
		return err
	}

	w, err := r.Worktree()

	if err != nil {
		return err
	}

	_, err = w.Add(".")
	if err != nil {
		return err
	}

	status, _ := w.Status()

	if len(status) == 0 {
	 	return nil
	/}

	_, err = w.Commit("downloads latest changes", &git.CommitOptions{
		All: false,
		Author: &object.Signature{
			Name:  "John Doe",
			Email<EMAIL_ADDRESS>			When:  time.Now(),
		},
	})
	if err != nil {
		return err
	}

	err = r.Push(pushOptions)
	if err != nil {
		return err
	}

	return nil

I've debugged the worktree status file, and diffStagingWithWorktree seems to be ignoring files properly when I run the w.Add method. But when I run commit, it commits all files that should have been ignored.
Any help would be greatly appreciated! Thanks in advance!

AFAIK gitignore is only used for worktree.Status(). You'll have to do the filtering yourself. As an example the code that does the filtering for status is here:
https://github.com/src-d/go-git/blob/25e9f61108d7097d6614872b4d65e15f4cb581fa/worktree_status.go#L143
It would be nice to have this functionality baked in Add and AddGlob. Related to #1201

I'm facing the same problem, using this lib to make changes on multiple git repo and it sometimes using the gitignore file in the root and sometimes not and pushing 500+ extra files, which is really annoying.

Hi @jfontan , any good workaround for this?


DestinationsNavigator.clearBackStack is not working

Hello, I didn't find any similar issue so probably this issue is unknown. clearBackStack does not have any effect.
Classic Navigation works next way:
Screen 1 -> Screen 2 -> Screen 3 -> (cleanStack = true) -> Screen 1
This would clean Screen 2 & 3 from the back stack. However, with compose-destinations this has no effect. Could you check it please?
My code:
fun DestinationsNavigator.navigateTo(
    destination: Direction,
    cleanStack: Boolean = false,
    popCurrent: Boolean = false
) {
    if (popCurrent) {
        navigateBack()
    }
    Timber.d("Navigating to ${destination.javaClass.simpleName}, cleanStack=$cleanStack")
    if (cleanStack) {
        clearBackStack(destination.route)
    }
    navigate(destination)
}

fun DestinationsNavigator.navigateBack() {
    Timber.d("popBackStack()")
    popBackStack()
}

I also tried
fun DestinationsNavigator.navigateTo(
    destination: Direction,
    cleanStack: Boolean = false,
    popCurrent: Boolean = false
) {
    if (popCurrent) {
        navigateBack()
    }
    Timber.d("Navigating to ${destination.javaClass.simpleName}, cleanStack=$cleanStack")
    navigate(destination, builder = {
        if (cleanStack) {
            clearBackStack(destination.route)
        }
    })
}

The same effect.


When selecting a preset for your project from BO section, the changes are not showing on the site

[ ] When selecting a preset for your project from BO section, the changes are not showing on the site

This needs adjustment on the css files of the preset

[ ] Write automate test to cover the point

Acceptance testing

[ ] When selecting a preset for your project from BO section, the changes are not showing on the site

This needs adjustment on the css files of the preset

[ ] Write automate test to cover the point

@akramsarkis Can you put more details regarding what sections are not updating and what are the css styles that you are expecting to see for them ? In order to give a correct estimation

@akramsarkis Here is the updated preset ENV variable that should be set on rancher dev ENV for Sections BO
NUXT_ENV_SECTIONS_CSS_PRESET=[{\"name\": \"Preset 1\", \"url\": \"https://s3.amazonaws.com/eweevtestbucketprivate/sections%2Fpreset+1b17fc59aaa98479793f4daf4d3cab9e1.css\"},{\"name\": \"Preset 2\", \"url\": \"https://s3.amazonaws.com/eweevtestbucketprivate/sections%2Fpreset+2b5b71a93498b46d3b1a37d718c512249.css\"},{\"name\": \"Preset 3\", \"url\": \"https://s3.amazonaws.com/eweevtestbucketprivate/sections%2F33dde5e53089e74d758bd0c03c37c08b13.css\"}]

@akramsarkis Here is the updated preset ENV variable that should be set on rancher dev ENV for Sections BO
NUXT_ENV_SECTIONS_CSS_PRESET=[{\"name\": \"Preset 1\", \"url\": \"https://s3.amazonaws.com/eweevtestbucketprivate/sections%2Fpreset+1b17fc59aaa98479793f4daf4d3cab9e1.css\"},{\"name\": \"Preset 2\", \"url\": \"https://s3.amazonaws.com/eweevtestbucketprivate/sections%2Fpreset+2b5b71a93498b46d3b1a37d718c512249.css\"},{\"name\": \"Preset 3\", \"url\": \"https://s3.amazonaws.com/eweevtestbucketprivate/sections%2F33dde5e53089e74d758bd0c03c37c08b13.css\"}]

@mahfouzhseineweev do I need to upload new files on my medias library ?  If you need to provide to me the updated files


#info database survives across backtracks

The database used by the #info interactive command isn't saved and reset when backtracking; this causes F* to report outdated information when proving #info for a position in a retraced section of the buffer.
Test case:
>>> #push 1 0
>>> module Info
>>> let a : int = 1
>>> #end #done-ok #done-nok

#done-ok

>>> #info a <input> 2 5
(defined at <input>(2,4-2,5)) Info.a: int
#done-ok

>>> #pop
>>> #info  <input> 2 5
(defined at <input>(2,4-2,5)) Info.a: int
#done-ok

is this perhaps a feature rather than a bug? ; )
i.e., F* will show you the last known type information at that point. Better than showing you nothing at all.

Dubious: with this feature, if you write something different at the same location, you'll get the type of the old thing instead of the type of the new one (the type of the new one comes from the id that's also included in the "info" query)

Also, the info is never reset, so it just accumulates (with multiple overlapping and often contradictory info spans on the same point).

FYI, I found an fixed a bug in the #info database. The fix is in nik_tactics, which should land in master with guido_tactics
https://github.com/FStarLang/FStar/commit/a7d005ab8fe39831e3922aee578c5054e336fd6d#diff-eaba7bdeef6a7c1a39c0a4bfcd42b984L125

Thanks for the pointer.  I think I can make a fix for this bug, which is distinct from the one you fixed.


Unable to reconnect a bluetooth

What version of Go are you using (go version)?

$ go version
1.12.5

Does this issue reproduce with the latest release?
no testing on latest 1.12.6, but done on both 1.7.4 & 1.12.5
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="arm"
GOBIN=""
GOCACHE="/home/pi/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="arm"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/pi/go"
GOPROXY=""
GORACE=""
GOROOT="/home/pi/go"
GOTMPDIR=""
GOTOOLDIR="/home/pi/go/pkg/tool/linux_arm"
GCCGO="gccgo"
GOARM="6"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -marm -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build293999460=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Implemented a Bluetooth server side, codes pasted as below
What did you expect to see?
Connection resumed  after retrying,
without restarting the whole program
What did you see instead?
The existing connection keep seeing bad FD error, until I restart the whole process:
Waiting... fd=-1  sa=2019/06/15 18:05:57 file descriptor in bad state
Waiting... fd=-1  sa=2019/06/15 18:06:02 file descriptor in bad state
....
Code: ----------------------------------------------------------------------------------
package main
import (
"fmt"
"log"
"bytes"
"strings"
"syscall"
"time"
"golang.org/x/sys/unix"
)
func init_socket() int {
fd, err := unix.Socket(syscall.AF_BLUETOOTH, syscall.SOCK_STREAM, unix.BTPROTO_RFCOMM)
if err != nil {
fmt.Println("Init BT socket err")
log.Println(err)
}
fmt.Println("Init BT socket sucessfully")
addr := &unix.SockaddrRFCOMM{
Channel: 1,
Addr:    [6]uint8{0,0,0,0,0,0},
}
_ = unix.Bind(fd, addr)
fmt.Println("Bind BT socket sucessfully")
_ = unix.Listen(fd,1)
fmt.Println("Listen BT socket sucessfully")
nfd, sa, err := unix.Accept(fd)
if err != nil {
fmt.Printf("Waiting... fd=%v  sa=%v", nfd, sa)
log.Println(err)
time.Sleep(5*time.Second)
return 0
}else{
//print client mac addr
//fmt.Printf("Conn addr=%v fd=%d\n", sa.(unix.SockaddrRFCOMM).Addr, nfd)
return nfd
}
}
func main() {
buf := make([]byte, 30)
bt_init:
nfd := init_socket()
if nfd == 0 {
goto bt_init
}
for {
_, err:= unix.Read(nfd, buf)
if err != nil {
//log.Println(err)
fmt.Println("error when reading bt, this error occurred when connection lost")
time.Sleep(2time.Second)
goto bt_init
}
fmt.Println("Connected")
n := bytes.Index(buf, []byte{0})
s0 := string(buf[:n])
fmt.Printf("Received: %v chars: %s\n", len(s), s )
}
}

The fd is -1 due to some error. Please change the unix.Socket call and also get the error and check that. It should become more clear then what is going on. It's better to avoid  ignoring errors with _, check them in stead.

This doesn't appear to be a Go issue, so closing. Please reach out for help at one of the links listed here: https://github.com/golang/go/wiki/Questions.

@beoran
Thanks for taking time to look at my code, the err is nil from unix.Socket statement,
I changed the code section,
error when reading bt
Init BT socket sucessfully
Bind BT socket sucessfully
Listen BT socket sucessfully
Waiting... fd=-1  sa=2019/06/15 22:51:37 file descriptor in bad state

Well, if you think this is not a Go issue, that's fine
Thanks for closing


deviceNotReady: Http status: 400. Unexpected registration response.

Did you follow all the instructions as specified in the Twilio Quickstart repositories?
I followed the Twilio Quickstart, and this was working not long ago.
What version of React Native are you running?
"react-native": "0.60.5"
What version of react-native-twilio-programmable-voice are you running?
"react-native-twilio-programmable-voice": "^3.21.1"
What device are you using? (e.g iOS10 simulator, Android 7 device)?
iPhone 6s+
Is your app running in foreground, background or not running?
Foreground
Is there any relevant message in the log?
{err: "Http status: 400. Unexpected registration response."}
If using iOS, which pod version are you using?
1.7.4
Step to reproduce
Follow the Quickstart tutorial and run the application
Advanced:
Have you tried adding break point using AndroidStudio or XCode and analyse the logs?
can share a project with issue?
Unfortunately no.
Did you try to reinstall the pods completely?
Yes
Other Notes:
I believe it is related to the Push Notifications credential, my credential is not expired and I only started experiencing this issue recently.  I've double checked all of my SID's and auth tokens and my webhook urls and following the tutorial over again.  Maybe this is related to iOS 13 update.
I'm using the iOS call to initWithTokenUrl and configureCallKit.   calling unregister didn't change anything and trying initWithToken had the same error.
Any help on this is highly appreciated

Learned that this is due to the iOS 13 update which means moving the TwilioVoice pod from 2.0.x to 2.1.  I'm going to try following their migration guide and use patch package for now


[Docs] Add the function to the module of resource that you can re-upload the file on the page of the file management.

https://github.com/apache/dolphinscheduler/pull/8359

Hi @calvinjiang please just use the English version picture here. ZH and EN doc could just use the English one

waiting CI

Thanks


Remove @inquirer/checkbox

Description

Remove invalid import { Choice } from "@inquirer/checkbox"
Create a new Choice type

Result
Choice type
type Choice<Value> = {
  name?: string;
  value: Value;
  disabled?: boolean | string;
  checked?: boolean;
  type?: never;
};

I was wondering about this, too. In my commits, I basically did the same thing, but didn't commit it since it seemed out of scope. Curious whether this is also throwing an error in your build?

I was wondering about this, too. In my commits, I basically did the same thing, but didn't commit it since it seemed out of scope for the work I was doing. Curious whether this is also throwing an error in your build @nicoalbanese?

Hey! I haven't ran into any build issues with this but I'm happy to merge this as it removes a dependency


Button paging not show

I already get 13 data from API, but the paging button not show. how to show the paging button??? and works the pagination from API

not working "'setPaging' does not exist on type "


CTE generation follow ups

a few extra things we should do once #166 is closed

deprecate %sqlrender and replace it with %sqlcmd snippets {name}
we need to update internal and external (duckdb, mindsdb, hydra) tutorials
%sqlcmd snippets should display a table with name and snippet
remove warnings so using --with raises an exception (in a separate PR so we give more time for users to update)

Update Product analytics tutorial

also update this test: https://github.com/ploomber/jupysql/pull/544

For this issue, do I update the tutorials (integrations and product analytic tutorial) by replacing %sqlrender with %sqlcmd snippets {name} and removing --with to avoid exceptions in a separate PR?
@edublancas

For this issue, do I update the tutorials (integrations and product analytic tutorial) by replacing %sqlrender with %sqlcmd snippets {name} and removing --with to avoid exceptions in a separate PR?

yes.

And could you provide clarification on what you meant by internal and external tutorials? What is the difference between the two?

internal tutorials are the ones we have in our documentation. external ones are in other websites. let's focus on internal ones for now

Acceptance Criteria:

Replace %sqlrender with %sqlcmd snippets {name} from the internal tutorials and ensure it displays a table in the first PR
Remove --with from the internal tutorials in the second PR

The internal tutorials are referring to tutorials under User Guide, Integrations, How-To, and Tutorials tabs

one more thing: show a FutureWarning when calling %sqlrender and tell people that they should use %sqlcmd snippets instead (also link to the documentation).
this will help people transition smoothly before we remove %sqlrender from the codebase

I understood what I should do for 2) from the AC and that I should add code that raises FutureWarning when a user calls %sqlrender. What I don't understand yet is the 1) from the AC.
This is what we get when we call %sqlrender,

and from what I observed, %sqlcmd snippets only takes 3 arguments that are all related to deleting a snippet. So do I modify %sqlcmd snippets so that it's able to take name of a snippet as an argument so that it can display a table with name and the current output of %sqlrender? And then do I change the tutorials reflecting the change I make?

@bbeat2782: you're right, there's no way to replicate %sqlrender with %sqlcmd snippets. let's pause this and work on adding such feature.
opened https://github.com/ploomber/jupysql/issues/647

@edublancas
I found that there is an open issue with inferring CTE depencencies (#684)
So for remove warnings so using --with raises an exception (in a separate PR so we give more time for users to update) from the first comment, do I wait until #684 is closed or can I work on it now?

thanks for bringing this up, most likely, we'll end up bringing back --with. so ignore that and work on the other points from this issue

@edublancas
I believe this can be closed now.

deprecate %sqlrender and replace it with %sqlcmd snippets {name}

#681

%sqlcmd snippets should display a table with name and snippet

#716

right, closing!
to close this automatically, you can put closes #NUMBER in your PRs. this way the issues are automatically closed when we merge the PR


Dashboard - Sole voter card - for user which is Sole voter and he/she delegated to a DRep and Retired as Drep - in card is non zero Voting power

Occurs as it is stated in title, when user is:

Sole Voter
He/she is already delegated his/her Voting power to a DRep
And retired a DRep
All above happened couple days earlier so all data should be up to date in db-synct.

In this case user sees that as Sole voter his/hers Voting power is > 0, should be there 0??

Tracked #962


Add KopernicusTech

This mod conflicts with KittopiaTech and Kopernicus rather than providing them due to this providing modified versions with the aim to better integrate the two.
Looking around the planet making threads it seems that some need this, some need Kopernicus+KittopiaTech and some work with either...
I followed the example from Kopernicus to filter the System.cfg since it seems that one is changed by mods to add stuff.
According to the thread "This is probably the last release of KopernicusTech, since Kopernicus is being actively developed again now. Sooner or later, it will have it's own PQS loader, and doesn't need KittopiaTech anymore, at least in most cases."

I'm going to throw out a wild guess here that this depends on https://github.com/KSP-CKAN/CKAN-meta/pull/342 to work since the identifier I'm trying to conflict with doesn't exist yet and that's what Jenkins doesn't like.

This should probably have it's own KopernicusTech-Config since Kopernicus-Config would be incompatible with this thing.


sdp3x: improve robustness, try to reconfigure on transfer error

reconfigure on transfer error (allows recovery from sensor reconnect)
add State to avoid usleep
add -k option to keep running and retrying even if no sensor present (we have a system that might boot w/o sensor attached).

add -k option to keep running and retrying even if no sensor present (we have a system that might boot w/o sensor attached).

This is another one we should think about generalizing. It might even make sense to do automatically for things like mags once calibrated.


TERRAFORM CRASH

Hi! Should i send this crash report to terraform? Maybe it will be interesting for you.
Terraform v0.12.5

provider.proxmox (unversioned)
https://files.promo.ru/s/cdxHoT8YWfjL8N7

main.tf that cause terraform crash
resource "proxmox_lxc" "gw" {   #vmid = "Alferov/lxc/1002"   vmid = "1002"   start = true   cores = 1   features {       nesting = true   }   description = "debian"   hostname = "Alferov-gw"   mountpoint {     #id = 0     mp = "/vault"     #size = "8"     volume = "local-lvm:8"   }   network {       #id = 0       name = "eth0"       bridge = "vmbr1"       ip = "<IP_ADDRESS>/16"       gw = "<IP_ADDRESS>"       #ip6 = "dhcp"   }   onboot = true   ostemplate = "local:vztmpl/debian-10.0-standard_10.0-1_amd64.tar.gz"   password = "111111"   pool = "default"   rootfs = "local:16"   storage = "local"   target_node = "localhost"   unprivileged = false

In the crash.log I notied that the Proxmox API responds the CPU limit as string, not as integer. "cpulimit":"0". This repo defines cpulimit as integer. My guess is though, that this is a Proxmox bug. The zero should be a JSON integer, not a string.

In the crash.log I noticed that the Proxmox API responds the CPU limit as string, not as integer. "cpulimit":"0". This repo defines cpulimit as integer. My guess is though, that this is a Proxmox bug. The zero should be a JSON integer, not a string.

can you advice please how to solve this? i should send issue to terraform?

No to Proxmox, their API should respond cpulimit as integer, rather than string.

but could it be that the cpulimit value is passed as a string instead of an integer value?
2020-07-10T17:57:20.100+0300 [DEBUG] plugin.terraform-provider-proxmox: Accept-Encoding: gzip 2020-07-10T17:57:20.100+0300 [DEBUG] plugin.terraform-provider-proxmox:  2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: 2020/07/10 17:57:20 <<<<<<<<<< RESULT: 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: HTTP/1.1 200 OK 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: Content-Length: 470 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: Cache-Control: max-age=0 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: Connection: Keep-Alive 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: Content-Type: application/json;charset=UTF-8 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: Date: Fri, 10 Jul 2020 14:57:20 GMT 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: Expires: Fri, 10 Jul 2020 14:57:20 GMT 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: Pragma: no-cache 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: Server: pve-api-daemon/3.0 2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox:  2020-07-10T17:57:20.110+0300 [DEBUG] plugin.terraform-provider-proxmox: {"data":{"mp1":"local-lvm:vm-1008-disk-1,mp=/vault ,size=8G","rootfs":"local-lvm:vm-1008-disk-0,size=8G","memory":256,"console":1,"arch":"amd64","features":"nesting=1","cmod e":"tty","digest":"6ccc3ad3f8a4af669b2db043701d8881039f7e7c","swap":512,"net1":"name=eth0,bridge=vmbr1,hwaddr=D6:5D:FF:3A: 3A:5A,ip=dhcp,ip6=dhcp,type=veth","cores":1,"cpulimit":"0","cpuunits":1024,"protection":0,"onboot":1,"description":"debian \n","ostype":"debian","hostname":"fisherman","tty":2}} 2020-07-10T17:57:20.113+0300 [DEBUG] plugin.terraform-provider-proxmox: panic: interface conversion: interface {} is strin g, not int

found this in proxmox-api-go
cores := 1         if _, isSet := lxcConfig["cores"]; isSet {                 cores = int(lxcConfig["cores"].(float64))         }         cpulimit := 0         if _, isSet := lxcConfig["cpulimit"]; isSet {                 cpulimit, _ = strconv.Atoi(lxcConfig["cpulimit"].(string))         }
maybe cpulimit should be int?

seems that i understand
if len(featureSetList) > 0 {
// only apply the first feature set,
// because proxmox api only allows one feature set
config.Features = featureSetList[0].(map[string]interface{})
}
and if i disabling mountpoint, creation process goes well

No to Proxmox, their API should respond cpulimit as integer, rather than string.

that is what he said

that is a bug, report to proxmox

https://bugzilla.proxmox.com/


merge package template from Kronos-Tools/npm-package-template

Updated standard to latest version

Codecov Report

Merging #100 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #100   +/-   ##
=======================================
  Coverage   83.33%   83.33%
=======================================
  Files           1        1
  Lines          54       54
=======================================
  Hits           45       45
  Misses          9        9

Continue to review full report at Codecov.

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


UIShellSideNav bug

Hi!
when Shell Side Nav have two itens with same name both show as active:

And any plans to make a UIShell compoments based too?

@mfeitoza Thank you for raising this issue.
For your second point, I think it would be good to make the UIShell component-based for consistency with the rest of the library. However, this may be a post-1.0 release

Thank you @metonym for you reply!
I will be a active user/contributor because I started to use in an client web app (is not in production)! So I will work in an component based Side Nav this week.


Inheritance Flow:

Same old screens are required.

Verified this issue on keeper app v 1.1.12(303)


fix: provision issue for QnA, split QnA template

Description
Issue: https://github.com/Azure/azure-sdk-for-js/issues/10186
QnA template doesn't support 'condition' variable which could control whether to deploy qna or not, so split the template file to two parts: qna resource and other resources
If createQnAResouce is true, deploy qna resource based on the independent qna template and other resources with the original template
if createQnAResource is false, deploy the other resources based on the original template

Task Item
Closes #4114
If we add api-version, the arm template would pop up another error message indicates the issue https://github.com/Azure/azure-sdk-for-js/issues/10186

Screenshots

Coverage remained the same at 55.313% when pulling 70e5610e6361e761a318a10249610603b624fd8e on qika/qnaSplit into 9ba5e8412eb81852fdd140b692ad46528f9c7b1b on main.


Please help!

I have successfully clone this repo, but after run this command:
docker-compose exec ico yarn
I got error : ico_1 container not found.
Please is there any instruction/help.

Hello! It seems like you clash the same problem as described here: https://github.com/secret-tech/backend-ico-dashboard/issues/184

What i need to do, i am new with docker-compose i don't know what to do.
Can you instruct me what i need to do.
My client is asking again and again, Today i need to final it.
Thanks


Refactor ControllerLeaderManager

This PR refactors ControllerLeaderManager by using dependency injection instead of singleton.
The purposes of doing this are:

it's clear to know where the object comes from.
the object is still initialized only once and is passed to all the callers.
controller code can easily be tested, e.g more than 1 pinot controller can be instantiated in one test.
singleton pattern is never explicitly freed (implicitly freed on program termination).

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5344efb). Click here to learn what that means.
The diff coverage is 89.28%.

@@            Coverage Diff            @@
##             master    #4054   +/-   ##
=========================================
  Coverage          ?   45.75%
=========================================
  Files             ?     1037
  Lines             ?    51426
  Branches          ?     7192
=========================================
  Hits              ?    23529
  Misses            ?    25879
  Partials          ?     2018

Impacted Files
Coverage Δ

.../periodictask/ControllerPeriodicTaskScheduler.java
100% <100%> (ø)

.../pinot/controller/api/upload/SegmentValidator.java
66.03% <100%> (ø)

.../pinot/controller/ControllerLeadershipManager.java
76.47% <100%> (ø)

.../helix/core/realtime/SegmentCompletionManager.java
31.59% <100%> (ø)

...lix/core/realtime/PinotRealtimeSegmentManager.java
73% <100%> (ø)

...i/resources/PinotSegmentUploadRestletResource.java
67.32% <100%> (ø)

...not/controller/validation/StorageQuotaChecker.java
26.74% <66.66%> (ø)

.../core/realtime/PinotLLCRealtimeSegmentManager.java
53.11% <80%> (ø)

...org/apache/pinot/controller/ControllerStarter.java
63.22% <85.71%> (ø)

Continue to review full report at Codecov.

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


Memory leak on a Kafka Observation due to the metric "spring.kafka.listener.active"

In what version(s) of Spring for Apache Kafka are you seeing this issue?
3.3.0 and 3.3.1
Describe the bug
We have an application using Spring Boot v3.3.6 and Spring Kafka v3.3.0. And we have seen that the metric called spring.kafka.listener.active leads to an increasing number of activeTask instances of type DefaultLongTaskTimer from Micrometer, which are never garbage collected.
We can see in the screenshot the memory and CPU usage of the process, which shows a classic memory leak trend :

The issue doesn’t appear on another application we have that uses spring Kafka v3.2.2
The symptoms are similar to the one in this issue: https://github.com/spring-projects/spring-security/issues/14030, where the stop method is not call on the observation.
We can see by debugging the code in method doInvokeRecordListener in the class KafkaMessageListenerContainer, that the finally bloc containing the observation.stop is not called because the listener is an instance of RecordMessaginMessageListenerAdapter.
When we disable this property, system resource consumption seems to return to normal : spring.cloud.stream.kafka.binder.enableObservation
Further compounding the issue, prometheus scrapes regularly this metric, which uses even more CPU and leads to timeouts or broken pipes on the scraping endpoint (/actuator/Prometheus)
See the stack trace associated to the scrapping workload : threaddump-1734718231951.zip
To Reproduce
We have been able to create a minimal sample project to reproduce the issue.
This is a simple Kafka producer / consumer using the latest Spring Boot v3.4.1 and Spring Kafka v3.3.1 versions.
We changed the rate of the producer to 1 millisecond (so around 1000 messages per second) to speed up the phenomenon.
We see millions of instance counts (and growing) for the active task after about 1h of running the test :

Sample
sample

I see where is the problem.
The KafkaMessageListenerContainer has the logic like:
				if (!(this.listener instanceof RecordMessagingMessageListenerAdapter<K, V>)) {
					observation.stop();
				}

where we assume that invoke() of the super class of that RecordMessagingMessageListenerAdapter is invoked.
And that one has a logic like:
Observation currentObservation = getCurrentObservation();
...
		finally {
			if (listenerError != null || result == null) {
				currentObservation.stop();
			}
		}

However, turns out, Spring Cloud Stream Kafka Binder uses KafkaMessageDrivenChannelAdapter from Spring Integration and that one has its own private class IntegrationRecordMessageListener extends RecordMessagingMessageListenerAdapter<K, V> {, which does not call the mentioned invoke().
So, yeah, confirmed as a bug.
Not sure yet how to fix.
Thank you for so simple reproducible sample!


feat: 🚀 add filterable Search Form Item

(These words are translated.)
First of all, thank you for your PR submission. For the requirement of adding filterable to el select and el tree select, you can actually put the function in 'column. search. props', and the contents will be automatically bound to the element (because we use v-bind).

like this

This is perfect! I am loving the updated dynamic proTable by the way!


Add Chinese(Traditional) Language Support

I made some translation files base from the Traditional Chinese's version of 3D Pinball Resources that preinstalled in Chinese Version of Windows XP(And something translate by myself).
Maybe can merge into the future's version.
(Sorry I'm a newer of GitHub... so my repo have some redundant works)

I try to decompline some apks and found that the folder name of Traditional Chinese maybe is "values-zh-rTW"

I correct the folder names and little change the Translation(miss a little Translate)

Thank you very much!

🙂


Autocomple comboBox for "opcodes"

Autocomple comboBox for "opcodes"
Hi Matt...
An image worth thousand words :)

Here is an example:
http://fxapps.blogspot.com/2016/03/simplest-javafx-combobox-autocomplete.html

Thanks for your hard work!

Done!


release-21.1: util/log,server/debug: fixes to logging and the logspy api endpoint

https://github.com/cockroachdb/cockroach/pull/66328 --- Release note (api change): The /debug/logspy API does not any more enable maximum logging verbosity automatically. To change the verbosity, use the new /debug/vmodule endpoint or pass the &vmodule= query parameter to the /debug/logspy endpoint.  For example, suppose you wish to run a 20s logspy session:  - Before:      curl %27https://.../debug/logspy?duration=20s&...%27    - Now:      curl %27https://.../debug/logspy?duration=20s&vmodule=...%27      OR      curl %27https://.../debug/vmodule?duration=22s&vmodule=...%27   curl %27https://.../debug/logspy?duration=20s%27    As for the regular vmodule command-line flag, the maximum verbosity across all the source code can be selected with the pattern *=4.  Note: at most one in-flight HTTP API request is allowed to modify the vmodule parameter. This maintain the invariant that the configuration restored at the end of each request is the same as when the request started.  Release note (api change): The new /debug/vmodule API makes it possible for an operator to configure the logging verbosity in a similar way as the SQL built-in function crdb_internal.set_vmodule(), or to query the current configuration as in crdb_internal.get_vmodule(). Additionally, any configuration change performed via this API can be automatically reverted after a configurable delay. The API forms are:  - /debug/vmodule - retrieve the current configuration. - /debug/vmodule?set=[vmodule config]&duration=[duration] - change   the configuration to [vmodule config] . The previous configuration   at the time the /debug/vmodule request started is restored after   [duration]. This duration, if not specified, defaults to twice the   default duration of a logspy request (currently, the logspy   default duration is 5s, so the vmodule default duration is 10s).   If the duration is zero or negative, the previous configuration   is never restored.

Created during a test of an issue-creation script. Issue is not needed. Closing.


DFT iteration takes 5s but profile only ~100ms

The structure optimization in nanoDFT.py takes 5s wallclock each iteration (the call  jitted_nanoDFT(*tensors) ), but the popvision profile is ~100ms.
Goal. Figure out what takes the 4.9s and remove it.
Things I'd try:

put a print inside _nanoDFT to see if it traces multiple times.
move jax.jit(partial(_nanoDFT  ...)` outside and pass the jitted function as input to nanoDFT
run with XLA_VISIBLE_DEVICES=1 python ... (hunch time is spent loading executables to many chips)

The problem seems to be related to the use of partial (docs here).
Caching the jitted nanoDFT function works faster (ugly code to prove it):
cached_jitted_nanoDFT = None
def nanoDFT(mol, opts):
    global cached_jitted_nanoDFT

    # Init DFT tensors on CPU using PySCF.
    tensors, n_electrons_half, E_nuc, N, L_inv, grid_weights, grid_coords = init_dft_tensors_cpu(mol, opts)

    # Run DFT algorithm (can be hardware accelerated).
    if cached_jitted_nanoDFT is None:
        cached_jitted_nanoDFT = jax.jit(partial(_nanoDFT, opts=opts, mol=mol), backend=opts.backend)
    jitted_nanoDFT = cached_jitted_nanoDFT
    vals = jitted_nanoDFT(*tensors)
    logged_matrices, H_core, logged_energies = [np.asarray(a).astype(np.float64) for a in vals] # Ensure CPU

    # It's cheap to compute energy/hlgap on CPU in float64 from the logged values/matrices.
    logged_E_xc = logged_energies[:, 4].copy()
    density_matrices, Js, Ks, H = [logged_matrices[:, i] for i in range(4)]
    energies, hlgaps = np.zeros((opts.its, 6)), np.zeros(opts.its)
    for i in range(opts.its):
        energies[i] = energy(density_matrices[i], H_core, Js[i], Ks[i], logged_E_xc[i], E_nuc, np)
        hlgaps[i]   = hlgap(L_inv, H[i], n_electrons_half, np)
    energies, logged_energies, hlgaps = [a * HARTREE_TO_EV for a in [energies, logged_energies, hlgaps]]
    mo_energy, mo_coeff = np.linalg.eigh(L_inv @ H[-1] @ L_inv.T)
    mo_coeff = L_inv.T @ mo_coeff
    return energies, (logged_energies, hlgaps, mo_energy, mo_coeff, grid_coords, grid_weights)

Tried caching mol instead, but it doesn't seem to make a difference - haven't tried caching all arguments though.

Hypothesis: problem is nanoDFT(mol, opt) needs access to mol while Jax traces nanoDFT. Jax allows this with static_argnums, by relies on hashing to check for recompilation. Because mol/opt doesn't support hash, which we fixed by using partial (which in turn causes problem).
One potential fix may be to add hash to mol/opt which only changes when we need to recompile (i.e. when N changes). @balancap  any thoughts?

Neat! Let's revise and make it into a PR :)
Q1. What happens if we remove all the jax tree stuff? (I might just be misunderstanding, but don't see why it's needed in this case)
Q2. Could we be a bit cheeky and do mol.__hash__ = self.nao_nr; mol.__eq__ = lambda self, other: ...?

Perhaps the wrong issue to attach this to, but seeing as you're in there anyway - I have just been looking at what's going on to see what/how this could demonstrate that IPUs are faster, and can see the following (all this FYI so pls ignore if you know this already!):
On both IPU and CPU, the time for jax.jit is instantaneous... i.e. at least one go-round of jitted_nanoDFT itself is required to get anything to compile/happen.
Then on IPU we see that:
The very first time (i.e. when a compilation is needed), it takes 149 seconds.
When loading from the cache, it takes 7.5 seconds
But when subsequently used in the same program, it takes 16ms
CPU comparisons are that the first time it takes 3.5 seconds, subsequent pass 933ms.
    # Run DFT algorithm (can be hardware accelerated).
    start = time.perf_counter_ns()
    jitted_nanoDFT = jax.jit(partial(_nanoDFT, opts=opts, mol=mol), backend=opts.backend)
    jit_complete = time.perf_counter_ns()
    vals = jitted_nanoDFT(*tensors)
    computation_complete = time.perf_counter_ns()
    vals = jitted_nanoDFT(*tensors)
    computation_complete2 = time.perf_counter_ns()

    logged_matrices, H_core, logged_energies = [np.asarray(a).astype(np.float64) for a in vals] # Ensure CPU

    ## No need to print this is it is instantaneous
    ## print(f"JIT duration was {(jit_complete - start) // 1000000}ms.")
    print(f"1st pass duration was {(computation_complete - jit_complete) // 1000000}ms.")
    print(f"2nd pass duration was {(computation_complete2 - computation_complete) // 1000000}ms.")

To "boot it up" , it needs to be run once... so it isn't clear how we could (or should?) use this sample to demonstrate that an IPU is faster than a CPU, as we'd need to be doing more than one iteration in the same program to show that.

Thanks for taking the time to dive into the code Anthony! :)
My take: We should change default to an interesting use-case that has 10^3 iterations (e.g. like this video, each frame is one iteration, so do a movie with 10^3 frames).
Fun note: With 16ms we do do ~60 FPS on IPU compared to ~1 FPS on CPU. We could make a "side-by-side" comparison of frame-rate using different CPU/IPU backends.

@AlexanderMath

Q1. What happens if we remove all the jax tree stuff? (I might just be misunderstanding, but don't see why it's needed in this case)

In fact, nothing happens, it works without those methods and without registering it. It was a misunderstanding on my side, I thought registering it as a PyTree was necessary (docs link) to be handled correctly by jit, but it is not.
Thanks, this simplifies the code!

Nice! Rule of thumb in JAX: registering custom class with the PyTree mechanism should really be exceptional. 99% of the time, we should be fine be passing tuple, namedtuple, dataclass (using chex lib), dict, ...

@AlexanderMath

Q2. Could we be a bit cheeky and do mol.hash = self.nao_nr; mol.eq = lambda self, other: ...?

Doing mol.__hash__ = ... mutates it and makes it unhashable if I understand correctly, but we could use setattr instead to change the original class without mutating it, for example:
setattr(pyscf.gto.mole.Mole, "__hash__", lambda self: hash(self.nao_nr()))
setattr(pyscf.gto.mole.Mole, "__eq__", lambda self, other: self.nao_nr() == other.nao_nr())

If you are happy with this strategy, it would be helpful to know:

whether we should keep these globally in the file, or maybe add them in build_mol to make it a bit clearer what we're doing?
whether it's fine to use just nao_nr or whether we want a list of attributes to count which makes from a domain expert's point of view?

Other options considered:

refactoring mol out of the code, but this seems a bit drastic as it's not just a few occurrences
use the derived class, but it is probably too complicated for what we need

Thanks!


Does increasing max_decoder_steps affect on quality?

I've question.
We increased max_decoder_steps to 10^5, and as long as mels are generated from data during the training, and the data points are always smaller than max_decoder_steps, if it affect on the performance.
P.S. We are fine-tuning on pre-trained model.
See the tensorboard logs:
Attention on 500 steps:

Attention on 10500 steps (I'm not sure what the hell is happening here):

Mel predicted on 10500 steps:

max_decoder_steps has no effect during training, only inference.
https://github.com/NVIDIA/tacotron2/blob/c76ac3b211fee3e958af316ce37ec4438c6d5626/model.py#L418

@rafaelvalle  Thank you!
And, could you explain the attention shown in 2nd image?

Hard to say without looking at your loss plots.

Closing due to inactivity.


[bug] mutate provides not intuitive error messages

With version v0.2.1:
>>> from palmerpenguins import load_penguins
>>> pen = tidyframe(load_penguins())
>>> pen
# A tidy dataframe: 344 X 8
   species     island  bill_length_mm  bill_depth_mm  flipper_length_mm  body_mass_g  ...
  <object>   <object>       <float64>      <float64>          <float64>    <float64>  ...
0   Adelie  Torgersen            39.1           18.7              181.0       3750.0  ...
1   Adelie  Torgersen            39.5           17.4              186.0       3800.0  ...
2   Adelie  Torgersen            40.3           18.0              195.0       3250.0  ...
3   Adelie  Torgersen             NaN            NaN                NaN          NaN  ...
4   Adelie  Torgersen            36.7           19.3              193.0       3450.0  ...
5   Adelie  Torgersen            39.3           20.6              190.0       3650.0  ...
6   Adelie  Torgersen            38.9           17.8              181.0       3625.0  ...
7   Adelie  Torgersen            39.2           19.6              195.0       4675.0  ...
8   Adelie  Torgersen            34.1           18.1              193.0       3475.0  ...
9   Adelie  Torgersen            42.0           20.2              190.0       4250.0
#... with 334 more rows, and 2 more columns: sex <object>, year <int64>

These three are valid and equivalent:

pen.mutate({'ldm': ("np.maximum", ['bill_depth_mm', 'bill_length_mm'])})
pen.mutate({'ldm': (lambda x, y: np.maximum(x, y), ['bill_depth_mm', 'bill_length_mm'])})
pen.mutate({'ldm': lambda x: np.maximum(x['bill_depth_mm'], x['bill_length_mm'])})

Now,
>>> pen.mutate({'ldm': (np.maximum, ['bill_depth_mm', 'bill_length_mm'])})
TypeError: unsupported callable

is not a valid call as the functionnp.maximum does not take arguments x and y. But this should provide a clearer error message.

For the grouped case, we get a different error, again not intuitive:

>>> pen.mutate({'ldm': (np.maximum, ['bill_depth_mm', 'bill_length_mm'])}, by = 'species')
AssertionError: arg 'column_names' should contain valid column namesThese column(s) do not exist: ['HuPVeoasPkPuWIkSMQhM']

Solution (draft): Check for the args when number of args >= 2 and see if they are named appropriately. If not, throw a meaningful error right away. @grahitr what do you think?

@talegari On debugging, I realized that the real cause of the error is _is_kwargable check(in turn a call to inspect.getfullargspec) in mutate. All the numpy functions and python builtin written directly in C throw the same exception with inspect.getfullargspec. https://stackoverflow.com/questions/27769462/using-inspect-getargfullspec-to-find-out-about-functions-not-working
np.maximum works on the two arrays of equal sizes, actual names of two arrays in function signature is not source of the error.


Add actions cache example.

Signed-off-by: Han Verstraete (OpenFaaS Ltd<EMAIL_ADDRESS>
Description

Add an example on how to use the GitHub Actions cache with actuated.
Motivation and Context

Show users it is possible to use the Actions cache with Actuated runners.
How Has This Been Tested?

Verified the docs locally. Workflows where tested when writing the associated blog post.
Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s

Included an additional example for manual configuration and added a link to the configuration examples for different languages and frameworks.


Add School zgb-Buchen

this is the webpage https://www.zgb-buchen.de/

@PaulJScript I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host ppl.on.ca not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Upgrade to Pydantic V2

Feature request
Langchain is currently using Pydantic V1, and I believe it would be good to upgrade this to V2
Motivation
V2 is far more performant than V1 because of the python-rust binding
Your contribution
I can create a PR here.

Langchain use v1, while Fastapi use v2. they can't work together. 😔
Hopefully v2 will be supported soon.

I'm actually working on that upgrade now. Hopefully I can settle it in the next week. You'll still have to wait until the next release : /

Please reference the discussion here: https://github.com/langchain-ai/langchain/discussions/9337
LangChain is currently cross-compatible with both versions of pydantic. When using pydantic v2, langchain relies on the v1 namespace of pydantic v2.
We cannot yet upgrade to v2 since that'll break LangChain to any users that are dependent on v1. v1 is more popular currently than v2 and is used by many of our users. (https://www.pepy.tech/projects/pydantic?versions=2.&versions=1.)
FWIW:
FastAPI includes v1/v2 comaptibility on its roadmap: https://github.com/tiangolo/fastapi/issues/10370, https://github.com/tiangolo/fastapi/issues/10360
LangChain should be usable with fastapi and pydantic v2 -- as long as LangChain schema isn't used directly in the fast api request/responses (can clone the schema with pydantic v2 use that for fast api validation, and then internally convert to schema with pydantic v1 when working with LangChain).
With that said, any work you can do to bring the code base to a state that will allow us to replace v1 models with v2 models will be appreciated.
There's some automation that's available for this: https://github.com/pydantic/bump-pydantic, but it doesn't catch all cases and in a code base like LangChain there'll be a lot of manual work likely (we've got hundreds of pydantic classes).

Can we use a variable to control which version is used?

@mokeyish I'm not sure at which level you mean
Users can control at the level fo the dependencies and install either pydantic v1 or pydantic v2. Everything will work as long as they don't only use v1 namespaced objects with langchain primitives; otherwise they can use pydantic v2 freely in their applications.
If you mean to use a variable at the application level to configure LangChain to use the proper v2 namepace, that would be great, but I'm not sure how feasible, if you can come up with a solution let us know. LangChain is CI already runs with both pydantic v1 and v2 for unit tests so you can test out to see how well your solution works.
As best as I can tell it's a bit of an all or nothing game with the pydantic migration -- I don't know that there's a safe way to upgrade some objects, but not others. :(

@eyurtsev , I bump into this issue due to need of my project to use pydantic v2.

@nikokaoja please follow the guidelines here: https://github.com/langchain-ai/langchain/blob/master/docs/docs/guides/pydantic_compatibility.md

many thanks @eyurtsev ! it worked.

@nikokaoja please follow the guidelines here: https://github.com/langchain-ai/langchain/blob/master/docs/docs/guides/pydantic_compatibility.md

please this page isn't showing up


Update README

Add in depth overview of project, as well as deployment instructions, an architecture diagram, and any other potentially useful information about the application.

README updated.


Make research/dashboard highlights somehow visible to non-logged in visitors

@mrueschman, I'm guessing this will be changed to make blog visible to users visiting the landing page?
Thanks!

Yeah, that was the thought.


extract docs for zat create and zat update

cc @zendesk/vegemite @chucknado extracted from #128 since it might take a little longer to merge.

👍

@zendesk/documentation would somebody be able to give this a review and :+1:?

@chucknado can you please review this?

👍

I'm on PTO. Go ahead and merge and I’ll take a look when I get back.
On Monday, September 26, 2016, Roman Sandler<EMAIL_ADDRESS>wrote:

@chucknado https://github.com/chucknado can you please review this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendesk/zendesk_apps_tools/pull/144#issuecomment-249495148,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC0YvWYy-j6WjSxUIQ12eN3R1LO-dYz2ks5qt2_BgaJpZM4KBF8n
.

--
Charles Nadeau
Platform Content Manager
Zendesk http://www.zendesk.com/
Cell:<PHONE_NUMBER>


docs(interfaces): document important interfaces and classes

Write JSDoc documentation for Observer, Subscriber, Subscription, Notification, Scheduler, Action,
etc.

I rebased and also moved interface documentations to files MiscJSDoc.ts which are not compiled to any release bundle, but appear in the ESDoc site.
Can haz merge now?

LGTM.

Rebased. :white_check_mark:

Merged with 8da7a8f, thanks @staltz :)


Collect & log thread dump of the timed-out task/process

Followup to #1096 / #6643 which first implements a timing out hanging or otherwise errant long running tasks.
Desired Behavior
When a task is timed out it would be desirable to collect stacktraces/thread dumps from the timed-out task and any workers or sub-processes it spawned.
Ideally, there would be a best-effort basis to collect the relevant stack traces, not just the one from the thread the hanging task was orchestrated from. A hanging JavaExec, worker API, Test task worker should have the relevant thread state captured.
Context
Preventing the very frustrating  failure mode of an indefinite hang in CI and on local developer machines is highly desirable. But if the timeout mechanism itself complicates or impedes root-cause analysis it comes at a steep cost.
In practice, most of the hangs we've experienced have been related to test execution. A minority of hangs have been from other programs called via JavaExec. I recall no hangs in Gradle itself or any Gradle plugins we have used.
Your Environment
(mostly) Java multi-project build. Several forms of code generation. Light but increasing usage of composite build. Several different test frameworks, mocking frameworks, and a homegrown bytecode transformation or two.

@oehme Can we postpone this one to 5.2?

We actually did this for 6.8: https://github.com/gradle/gradle/issues/14960
It doesn't trace into forked processes, only what's going on in the build VM.

We actually did this for 6.8: https://github.com/gradle/gradle/issues/14960
It doesn't trace into forked processes, only what's going on in the build VM.

@ldaley that is certainly an improvement, but I don't think it is capturing a thread dump as suggested in this issue, which I think is a very useful indicator of why a task might be stuck. I suggest you consider reopening this issue.

@ldaley that is certainly an improvement, but I don't think it is capturing a thread dump as suggested in this issue, which I think is a very useful indicator of why a task might be stuck. I suggest you consider reopening this issue.

It reports a stack trace of the thread that is effectively preventing the task from stopping. I'm not convinced that showing the state of all threads would be an improvement and it's unlikely that we will add that unless someone makes a good case for it.

It reports a stack trace of the thread that is effectively preventing the task from stopping. I'm not convinced that showing the state of all threads would be an improvement and it's unlikely that we will add that unless someone makes a good case for it.

ah, I see... I missed that it prints the stacktrace.  Thanks for doing this.

ah, I see... I missed that it prints the stacktrace.  Thanks for doing this.


Link to an internal site instead of github.

Describe the bug
if you try to open the link in the driver description, you will end up on a non-existent page.
To Reproduce
Steps to reproduce the behavior:

Go to info tab
Click on icon with description
Click on Changelog
See error

Expected behavior
Opening a link from github, not from my site.
Screenshots & Logfiles

Versions:

Adapter version: 1.7.0 (the error was present on previous versions as well.)
JS-Controller version: 3.0.18 (the error was present on previous versions as well.)
Node version: v12.16.2 (the error was present on previous versions as well.)
Operating system: linux

This adapter has been deprecated.
So most certainly this this issue will not be asolved anymore


End-to-End test support.

This PR adds end-to-end testing support in pytest. The most important files are main.py, manual.py, and test_e2e.py. In the test we check the whole process: loading the NN archive, and setting up the neural network nodes, camera nodes, and parsers. Each test is ran as a child process with a timeout set to 30sec. If the model is not supported on the platform or the input size/type is not supported we skip the model. We also skip if the slug is not available in the ZOO etc.
Features:

Test the pipeline on specific model slug(s)
Test the pipeline on specific local NN archive(s)
Test the pipeline on all public ZOO models
Test the pipeline on all public ZOO models with specific parser (test only one parser)

One can run test locally with main.py (pytest) or manual.py. See the README for more info.
Downloading models first time takes some time (~4GB). Because of this I made a hardcoded list of all subset of available slugs so we can test the first GH run. I also made a parser-slug mapping dictionary so we dont need to download all models and read their parsers from nn archives every time. This must be optimized - I am talking with hubai dev team about it. Best would be if it parser info could be included in API response.
Github workflow si fired manually. There is an additional parameter we can set. Default: -all - means all slugs are tested. We can define any available parameter (e.g. --parser or --slug). This way we can quickly test open PRs changes.
Simple README is also added, where user can read about testing instructions.

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

Please upload report for BASE (main@c13e6b5). Learn more about missing BASE report.

:white_check_mark: All tests successful. No failed tests found.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #122   +/-   ##
=======================================
  Coverage        ?   33.46%
=======================================
  Files           ?       68
  Lines           ?     3732
  Branches        ?        0
=======================================
  Hits            ?     1249
  Misses          ?     2483
  Partials        ?        0

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


Feature/digital 27: add content

Added content for all new drupal content types:
Basic page
topics
source
services
news
guide
communities
authors
I did not create content for these content types:
Short Post
Test
Also, created two events that have their dates very far in the future:
node/29
node/30
Created landing pages for all links from the main menu on digital.gov:
BLOGS (news)
EVENTS
RESOURCES
COMMUNITIES
GUIDES
TOOLS (services)
JOB BOARD (?)

For WYSIWYG, let's add 2 paragraphs of lorem ipsum to get a better feel of the space.

Is this to test that markdown is not interpreted?

I wonder if we should put HTML in all our fields that allow it just so we can ensure it's possible.

You're 'AUTHORS-all fields' piece of content looks like a combination of 'Real Content' & 'All fields' content that needs to be created. The 'all fields' one should just be random text / text that includes the field name / lorem ipsum. The 'Real Content', should be an author copied directly from Digital.gov into whatever fields correspond to the Drupal version.

'BASIC-PAGE-all fields' looks like it's the 404 page, but that's not a basic page, it's a 404 type page (which reminds me I need to make a ticket to put that template in place).
Basic pages are in content/about. Copy 2 of those for the 'real' content and use lorem ipsum for the 'all fields' and 'required fields' one.

Just in general, for the rest of these: For each content type there should be 4-5 entries. For communities this might be:
Community - All required fields
Community - All fields
The above 2 only have lorem ipsum / field names in their field values.
Communicators
Plain Language
User Experience
The above 3 have the exact same corresponding fields to Digital.gov, it would be as though we migrated these with the scripts. If we were to finish the site, they would look exactly the same. Except if they have a bunch of references that maybe we don't have time to create right now.


move monkey-patched play method into plugin constructor

to prevent screwing up other players on the page.
fixes #437

Coverage increased (+0.2%) to 75.136% when pulling 911ea5b6ad56ac66358e3ecd06a7261313c1afc6 on monkey-patch-437 into ae80f4a0b568330617c3ea44f9ad9b04060e2a07 on master.