What is the kfserving_endpoint for a remote KFServing installation? Using http://$INGRESS_HOST:$INGRESS_PORT/deploy-model returns 404

/kind question
I have a standalone/remote KFServing installation running. When I put the ingress host (public IP) and the ingress port as the kfserving_endpoint  $INGRESS_HOST:$INGRESS_PORT, the pipeline calls http://$INGRESS_HOST:$INGRESS_PORT/deploy-model but that returns a 404 not found error.
I even tried it manually from the command line and got the same result.
What should I call to reach the deploy-model api?
export INGRESS_HOST=$(kubectl -n istio-system get service istio-ingressgateway -o jsonpath='{.status.loadBalancer.ingress[0].ip}')
export INGRESS_PORT=$(kubectl -n istio-system get service istio-ingressgateway -o jsonpath='{.spec.ports[?(@.name=="http2")].port}')
curl http://$INGRESS_HOST:$INGRESS_PORT/deploy-model -v

@PatrickGhosn you will need to pass the host header if you are using public ip not dns

https://github.com/kubeflow/kfserving#curl-the-inferenceservice

@PatrickGhosn you will need to pass the host header if you are using public ip not dns

I'm trying to call the API to create the inference service, not curling the service itself. It's in the code here:
https://github.com/kubeflow/pipelines/blob/1.4.0/components/kubeflow/kfserving/src/kfservingdeployer.py
Line 367.

For those interested in a follow up. Apparently the sample in kfdeployer needs to be run as a seperate pod, and use flask for the  the deploy-model API that communicates with the kf-serving deployment.


C-CDA output patient record

What do I need to do to create the patient record in C-CDA format? I saw the instruction on https://github.com/synthetichealth/synthea "Synthea will output patient records in C-CDA (requires running instance of Mongo DB) and FHIR STU3 formats in ./output." However, I am not sure how MongoDB instance is created or runs.
Thank You so much

The install process for MongoDB is different depending on whether you're on Windows, Mac, or Linux.  More information on the install process is available here: https://docs.mongodb.com/manual/installation/#mongodb-community-edition
That said, I would also encourage you to take a look at the synthea_java project, which can output patients in CCDA format without the dependency on MongoDB. It's not feature-complete to the level of the ruby version, but we expect it to be there within the next month or so.
https://github.com/synthetichealth/synthea_java
In synthea_java, you can enable CCDA export by editing the properties file at src/main/resources/synthea.properties and setting exporter.ccda.export = true .  You may also want to change generate.default_population to a larger number, since the default is 1.
Then you run synthea_java with the command ./gradlew run
For another alternative, depending on your requirements, you may find enough sample patients at   https://syntheticmass.mitre.org/ . We offer 1,000 sample patients in CCDA format at https://syntheticmass.mitre.org/downloads/2017_11_06/synthea_sample_data_ccda_nov2017.zip , or you can browse the individual patients in the dashboard and download them in CCDA format.

@dehall Thank you so much, this is very helpful and the synthea_java seems to be working fine for now. Also, thumbs up for your quick response.


Add 'comments to resolve' sticker

What
We'd like to see, if the story has some comments to resolve. The simplest solution, implemented here is adding a label to the story on Pivotal, which then will convert into sticker on our board.
How to review

Fire up the application
Go to pivotal and add comments-to-resolve label, to any of the active stories
Open the application
???
Profit - Not a materialistic profit.

Who can review
Probably @mtekel :D

works...


Scraping issues

Describe the bug
website displays wrong time
To Reproduce
Steps to reproduce the behavior:

Go to 'vaxx.nz'
look at some of the walk-in locations
compare the time

Screenshots

i'll fix phone number as well

I fixed that and made a new issue for telephone numbers


Fixed README typo

Coverage remained the same at 97.368% when pulling 75ab04dcdd0c83a46c195eb3d97cff9fd1abfa15 on joshkmartinez:patch-1 into 2a209bd4f11a84ff4543fbde930fed60dcbc0f68 on xgfe:master.


Add comments widget to 21ideas

Adding the zapthreads widget.
When running hugo -D server the "Log in" button does not appear - might have to do with Alby not working on localhost. It requires hugo -D server -b https://new.21ideas.org --appendPort=false but in that case livereload won't work. Not a big deal I think once integrated it can be tested on the actual https://new.21ideas.org
Draft PR until I fix an issue with publishing.

Thanks a lot for your time, @fr4nzap ! I'm wondering how logins will turn out here. On Habla we've got a login button on the website itself. I'm guessing some kind of Nostr login element will be needed. Or is it already integrated into zapthreads?

@bitcoin21ideas dude I completely missed your message.
Yes zapthreads has an integrated login. Only if an npub is provided externally it will auto-login.
A few notes about the Hugo integration

Unlike Habla, which uses naddr as anchors, I am assuming that the reference here is the actual URL, correct? We can do that in nostr with an r tag. It's a bit inconvenient when testing because URLs change (localhost:1313, new.21ideas.org, www.21ideas.org) and comments are only associated with one URL. My suggestion for tests is tricking Hugo to serve the "good" domain so all comments go there

I used hugo -D server -b https://www.21ideas.org --appendPort=false so that all comments end up under the www domain
Reminder that you can pass disable="publish:true" to disable publishing for testing purposes (same for likes/zaps if you wanna disable, they are just counts for now)

It appears your search component listens for the key S to focus, which means it not possible to use that letter when writing a comment. Probably best to disable that
I had to remove the outline and made some minor styling improvements, feel free to change https://github.com/fr4nzap/zapthreads#css

Lmk how it goes!

Other things coming soon:

keeping the login state when browsing different pages
multi language support

@mell4ce can you have a look into this please, especially the issue with the "s" and search.

Do you also post these articles as long form notes? It occured to me that an option might be to then include the naddr as the article front-matter , and pass that to the zap-threads component. Would prevent potential issues with URL variations

I commented out the line that enables the S search hotkey. It's a workaround which will have to be reapplied if you update the theme


Investigate why mobile nodes are publishing poorly encoded data to IPNS

On starting a mobile node, found the following in the IPFS logs on startup:
{"id":1,"level":5,"message":"error when determining the last published IPNS record for \u003cpeer.ID Qm*n1HYLG\u003e: datastore: key not found","module":"namesys","time":"2019-06-05T14:07:53
{"id":2,"level":5,"message":"Storing pubkey at: /pk/\u0012 \ufffd....\ufffd\ufffd","module":"namesys","time":"2019-06-05T14:07:53.759999297Z"}
{"id":3,"level":5,"message":"Storing ipns entry at: /ipns/\u0012 \ufffd....\ufffd\ufffd","module":"namesys","time":"2019-06-05T14:07:53.76001487Z"}

The keys for these values should look more like /pk/Qm.... Need to investigate this further.

sigh Totally a red herring. I assumed they should look like /pk/Qm... but every instance I find in the logs has them encoded in this way.... Won't investigate further.


Store and path to zip files isn't supported

Hi, great project !
Very interested by trying kivkio on large zarr datasets but my input are zarr.zip files.
I read in open_cupy_array doc that "Store and path to zip files isn't supported".
Is it a feature that is planned to appear in the future or is it conceptually impossible/useless ?
Thank you !

It is something we would like to look at but we there is no timeline :/
Since nvCOMP support Gzip, I don't think it is too much work.


More verbose output format with logcat in CI

To get more information via logcat, we could pass -v threadtime to logcat.
There would also be -v long, but based on current understanding, that wouldn't bring useful new information and would generate longer outputs.
Output formats documented at http://developer.android.com/tools/debugging/debugging-log.html#outputFormat.

I would rather prefer you / @yaronyg to pick the format instead of finding out the right one. FYI; threadtime doesn't have package name. Please make sure the format is playing nicely with the filtering efforts.

@obastemur: If I am not mistaken, none of the formats have package name. Please correct me if I am mistaken.
Your suggestion is good - I'll check with @yaronyg first what we should do with this one.

FYI, applying filter breaks the format. We do apply a filter to grab the app specific logs.

-v threadtime did not work on all devices so it was decided that it is not taken into use.


Update Bot Embed

Description
the embed for the roles command is displaying the wrong example to use.
the error example is:
 ^iam javascript

the real example should be
^iam add javascript

Thanks for opening this!

Thanks for opening this!

thanks for giving me the opportunity too!!

thanks for giving me the opportunity too!!

Is there someone working on this or can I create a PR?

Is there someone working on this or can I create a PR?

Hi @hernandoagf 😃, I believe no one is working on it since it has no assignee.
I'll assign you this issue and you can create a PR if you want 👍

Hi @hernandoagf 😃, I believe no one is working on it since it has no assignee.
I'll assign you this issue and you can create a PR if you want 👍

Hi @BOLT04 ! Ok, thank you, I will 👍

Hi @BOLT04 ! Ok, thank you, I will 👍

Thank you all 👍

Thank you all 👍


Breadcrumbs should display JSON representation of objects instead of "[object Object]"

Sentry Breadcrumbs display object.toString() representation for console.log('message', {value: 1}), which is quite useless. The console arguments should be sent to Sentry in JSON (JSON.stringify()) so as the values are displayed instead of [object Object].

Hey,
I will look into this.

Would be great if it's similar to console improvement made in Chrome since 60 (or 61), which show collapsed snippets of elements inside the Object before expanding the object in console logs.

Related to https://github.com/getsentry/raven-js/issues/623

So, I guess we will not add this by default right now.
You could run JSON.stringify(object) yourself before adding a breadcrumb.

can we use beforeBreadcrumb on react-native-sentry?

Is there a best practice for capturing objects in logs? Aside from refactoring the console logs in my app to JSON.stringify the contents? Thanks

Something like this:
  beforeBreadcrumb: breadcrumb => {
    if (breadcrumb.category === 'console') {
      breadcrumb.message = JSON.stringify(breadcrumb.data.arguments)
    }
    return breadcrumb
  }


Doesn't install on Node.js 16 (macOS only)

Describe the bug
On 7682aab aka v0.2.0, npm install on Node.js 16+ (the "Current" major version; will become "Active" LTS on 2021-10-26) fails during installation of the transitive dependency dd-trace:
  CXX(target) Release/obj.target/metrics/packages/dd-trace/src/native/metrics/Collector.o
In file included from ../packages/dd-trace/src/native/metrics/Collector.cpp:1:
In file included from ../packages/dd-trace/src/native/metrics/Collector.hpp:6:
In file included from ../packages/dd-trace/src/native/metrics/Object.hpp:3:
In file included from ../../nan/nan.h:56:
In file included from /Users/ssw/Library/Caches/node-gyp/16.0.0/include/node/node.h:63:
In file included from /Users/ssw/Library/Caches/node-gyp/16.0.0/include/node/v8.h:30:
/Users/ssw/Library/Caches/node-gyp/16.0.0/include/node/v8-internal.h:452:38: error: no template named 'remove_cv_t' in namespace 'std'; did you mean 'remove_cv'?
            !std::is_same<Data, std::remove_cv_t<T>>::value>::Perform(data);
                                ~~~~~^~~~~~~~~~~
                                     remove_cv
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/c++/v1/type_traits:776:50: note: 'remove_cv' declared here
template <class _Tp> struct _LIBCPP_TEMPLATE_VIS remove_cv
                                                 ^
1 error generated.
make: *** [Release/obj.target/metrics/packages/dd-trace/src/native/metrics/Collector.o] Error 1

To Reproduce
Steps to reproduce the behavior:

npm install using Node.js 16.x on any platform

Expected behavior
Installs without error
Additional context
I confirmed that the package can successfully be installed after applying
diff --git a/package.json b/package.json
index 53323be..7d567f1 100644
--- a/package.json
+++ b/package.json
@@ -71,6 +71,6 @@
     "@opentelemetry/core": "^0.16.0",
     "@opentelemetry/resources": "^0.16.0",
     "@opentelemetry/tracing": "^0.16.0",
-    "dd-trace": "^0.21.0"
+    "dd-trace": "^0.35.0"
   }
 }

but under that patch, 35 of the 45 non-skiped tests fail:
  35 passing (242ms)
  2 pending
  10 failing

For more info on the dd-trace bug, see https://github.com/DataDog/dd-trace-js/issues/1388.

Workaround: Rather than sending trace data from my Node.js process directly to Datadog, I'm probably gonna export it with @opentelemetry/exporter-collector-grpc to an OpenTelemetry Collector configured to run the Datadog exporter.

@spencerwilson 👋  hope all is well. apologies for the state of affairs of this repo, it is indeed a bit out of date with latest node and also just in general with latest opentelemetry-js. For context, we're working on adding OTLP Support directly to the datadog-agent, which would preclude the need for these helper libs in every language. In the meantime the suggested setup is to use the opentelemetry-collector (contrib flavor) and the datadogexporter.


About kubectl from rancher v2.2.2

resources:
rancher version：V2.2.2
docker version：18.06
Linux：ubuntu16.04
Problem Description:
Use the RKE cluster installed by helm, and add the subgroups of other hosts:

When I use kubectl to access the cluster, there is an error message. I try to use kubectl on both the web page and the host. The results are the same when I use kubectl。

I'm not sure what's the cause of the problem. Please help to find out. Thank you！

@superseb

What RKE version did you use to setup the HA cluster for Rancher? What is the server: value pointing to? Your own load balancer, if so what one, if not, is it a DNS name to one of the nodes? Did this work before? Since when doesn't it work or has it not worked at all?

@superseb
Thanks for helping
RKE version: RKE version v0.2.6
I have multiple clusters, only the kubectl of this cluster fails. I execute the command on the page of rancher:

The server that this cluster configuration file points to:：

DNS resolution of the server is normal. It was used normally before. I didn't have any operation to fail suddenly.

Only on one cluster? All the kubeconfigs point to the same hostname for Rancher I assume? Can you share whats different between a working and non working cluster? And did it ever work or never for this cluster?

Let me know if this needs help.


Remove useless if condition in EditorPropertyArrayObject as they always evaluate to false

I discovered two if condition that would always be evaluated to false  so I made this PR that just remove them I think it is a bad merge (most likely mine) that they are still here as I thought I had already removed them.

Hold on. This may be used for the "Add Metadata" section on the Inspector. Or maybe not?

I'd say this should be investigated properly and see where this was made non-functional, instead of removing it

This was removed when Node stoped being stored as NodePath inside metadata and actually stored as Nodes in their actual variables. They should have been removed at the time as the condition above was modified so that it checks only for string starting with indices and not also for the one starting with metadata. See #76389.

Tested this and it doesn't seem to break anything (the code was added in #73256), but I'm not even sure when that method is called. I added a print_line() to check the assigned property name and it doesn't print anything.

Yeah you are right checked a bit more and I don't see where it would be used it is so easy that places that could use it are just doing it themselves. But maybe it is called by some EditorProperty as the Object is used as base object for the generated EditorProperty


fix assets / dependencies

To get the Dropdown work I had to add some js libs because their old links were not working (take a look at the assets/layout/footer.php)
ToDo:

[x] check which dependencies are really needed
[x] delete not needed dependencies
[x] remove the meta tags at the beginning (each file!) which are still there from the original template like:

<meta name="generator" content="Jekyll v4.1.1">

Done by @Laurammf


Counter is no longer working?

Counter is no longer working? i have had no problems until the past couple of days when the view count wont exceed 1 ?

Could you provide some more information?  Laravel version, package version, stack trace etc.
Did this happen when updating the package?

hey ! yeh Laravel 5.5 , package version ^1.0 and i believe when running composer update is when the issue occured

Are you using $model->addPageView(); somewhere in the controller to increment the page views?
Have you checked the database table if the views are getting stored? It's probably the 'page-views' table.

i have $tutorial->addPageViews(); in controller
but looks like its no longer inserting IP address in DB

When running composer show -i you can see all installed versions. Which one are you seeing at this package? v1.0.3 or v1.0.4

This is also wrong: $tutorial->addPageViews();. It should be $tutorial->addPageView().

v1.0.3

"but looks like its no longer inserting IP address in DB"

This issue got fixed in v1.0.4 (#39)

ok cool

Try to update the package to v1.0.4 and check again! Thanks

All working thanks

Nice! Glad I could help!


Support abstraction over asynchronous computations, revisited

This a sketch how we might support abstraction of asynchronous code without compromising the "always" commit semantics of await, unlike previous proposals (https://github.com/dfinity/motoko/issues/1482).
The basic idea is to introduce a synchronous lazy T type  constructor that subtypes async T by providing an additional, synchronous force operation for forcing the computation delayed by lazy e.
Types:
Formation:
T ok
---------------
lazy T ok

(no restriction to T shared).

}

This sketch ignores the issues of scoped awaits, but the idea would be that lazy is indexed by an implicit scope parameter and is only well-formed (an introducible) at the current index in an async context (much as we deal with async types now).
Remark: Why is lazy T  a subtype of async T and not the other way around? Because lazy T supports await but also offers a additional observation (force).
Questions:

If we support subtyping, it seems sensible for lazy T to be stateful with force executing the computation exactly once. Another design might be to drop the subtyping, rename lazy to delay (or even sync) and repeat the computation each time the value is forced. This would make objects of that type values, that can be composed without incurring effects. But can we think of any motivating examples?

It would be nice to make the subtyping unconditional, for any T, by relaxing the restriction (T shared) on async T and imposing in on the codomains of shared functions instead. That would allow more useful composition and abstraction over async types and their contents and removes an artificial restriction that is probably not required by the implementation. This relaxation has been mooted before.

Sugar:
private func flip() : lazy Bool  {
  let blob = await Random.rand();
  blob[0] & 1 == 1
};

Hm, can you illustrate what introducing a second variant of async type achieves? Wouldn't a simpler version of this proposal consist of just adding a possibly_await operator on async itself? What does splitting that off into a new subtype accomplish, and what drawbacks does it avoid?

Isn't it already too late by the time the code reaches possibly_await or await? The spawning of a separate (self-)call that we want to avoid in code abstracting over await already happens during the evaluation of await …, so a new intro form is also needed, right? (Maybe it's ok to have the extra intro and elim forms, but still only one type?)

Did you mean:

Isn't it already too late by the time the code reaches possibly_await or await? The spawning of a separate (self-)call that we want to avoid in code abstracting over await already happens during the evaluation of ~await …~ async ..., so a new intro form is also needed, right? (Maybe it's ok to have the extra intro and elim forms, but still only one type?)

Yes, thanks. Edited.

Just for comparison, https://github.com/dfinity/motoko/issues/1482 propose adding an eager do async e block, that would synchronously executes until the first await that suspends and could be used instead of async e to avoid the overhead of scheduling.
So the proposal here is slightly different. Whether its better, I'm not sure. Maybe its nice to know that the effects will only happen at force and await, rather that depending on how the async is introduced.

Yes, we already discussed the option of having a may-commit operator of some form. Doesn't the current proposal still have the same problem, then? My view was that commits and await syntax ought to maintain a 1:1 relation, reflecting that async is a monad and await its bind operator. This proposal avoids the issue of not performing a commit where await is used, but it still allows a commit to happen in places (of a function) where await is not used. Effectively, the force operator circumvents the monad's bind operator. So from my perspective, it still obscures atomicity rules and worsens the concurrency footgun.
Weren't there plans to address the performance issue on the platform level by executing self calls directly?

First of all, isn't it a bit of stretch claiming that async is a monad? A monad, in my book, describes an effectful computation as an inert value that can be composed sans side-effect until eliminated. Async is manifestly not that.
The commit points are only important when they affect the state of other concurrent operations. I think we should provide users with the ability to write code that can still be performant and may or may not commit, when they know what they are doing and the commit makes no difference.
The optimization of scheduling self calls directly is bound to run into limits and, in my mind, compromises the fairness of the scheduler. I'm still reluctant to rely on that.

Yes, it's more second-class in Motoko, but I'd think it still forms a monad in the semantics. At least that's a common model for async, and one that provides nice reasoning principles.

The commit points are only important when they affect the state of other concurrent operations.

If a rollback is unintentionally partial, then you might be left with incoherent state forever, independent of whether this is observed by concurrent or sequential operations.

The optimization of scheduling self calls directly is bound to run into limits and, in my mind, compromises the fairness of the scheduler. I'm still reluctant to rely on that.

Perhaps. But between this possibility and the general advice to cleanly separate code into async (communication) and sync (internal) layers (like with the use of the IO monad in Haskell), do we have evidence of this being enough of a remaining problem in practice? After all, this addition would be a significant complication to the language that is likely to be more error-prone, increase dev confusion and reduce code clarity. So, I'd want to be certain that we are not just putting it in as a sort of premature optimisation to the design.

The separation in Haskell would not work at all if you couldn't abstract over patterns that involve IO (or is that your point? I'm a bit flight lagged)


Long Press stop working after add or delete some cell only ios9

Hey guys i'm using this awesome framework, congrats by the way... It was perfect working but its losing longpress after add or delete some cell, any ideas?
A Video about ->
https://dl.dropboxusercontent.com/u/30253301/bug.mov
or
https://vimeo.com/144121726
Tks!

Thanks @rafaj10 ! :smile:
Please see the contribution guidelines and update your issue.

I am also encountering this issue. It happens randomly as far as I noticed.

About my case only happens with xcode 7 and ios 9...
Xcode 7.1 its ok...

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


LoRA support in Diffusers version 0.10

Hi, I'm using a local environment for OneFlow along with diffusers (OneFlow-fork. Locally cloned). I need LoRA support to infer my Stable Diffusion pipeline. I can't upgrade Diffusers version because of OneFlow dependencies. So I'm trying to in-corporate the dependencies of diffusers 0.12 into my existing 0.10 version so I can use LoRA support. Is there a solution for this? I'm stuck in the error "ImportError: cannot import name 'DEPRECATED_REVISION_ARGS' from diffusers.utils (init.py)", where as both the deprecation_utils are same, of Huggingface diffusers latest and oneflow-fork diffusers.
Looking for a solution for this.

LoRA is supported now.
Please update to the latest version: https://github.com/Oneflow-Inc/onediff?tab=readme-ov-file#install-from-source
This issue is too old to follow and will be closed. Feel free to reopen it and continue.


Multi Query null check

Needs the same null series check as single query method.
I know it looks like you LITERALLY just implemented this, but needed to use immediately anyways.

Hi, thanks for the PR.
I however in the mean time changed the method to this (not pushed yet):
public async Task<IList<IList<Serie>>> QueryAsync(string dbName, string[] queries)
{
    var response = await this.RequestClient.Query(dbName, queries.ToSemicolonSpaceSeparatedString());
    var queryResult = response.ReadAs<QueryResponse>();
    var seriesList = new List<IList<Serie>>();

    Validate.NotNull(queryResult, "queryResult");
    Validate.NotNull(queryResult.Results, "queryResult.Results");

    // Apparently a 200 OK can return an error in the results
    // https://github.com/influxdb/influxdb/pull/1813
    foreach (var result in queryResult.Results)
    {
        if (result.Error != null)
        {
            throw new InfluxDbApiException(HttpStatusCode.BadRequest, result.Error);
        }

        seriesList.Add(result.Series.ToList());
    }

    return seriesList;
}

Which actually makes sense because if you executed multiple queries at once, you would actually expect to get grouped results (grouped serie lists) instead of a flat serie list.
I might however add an additional overload that would return a flat result (basically method that you're currently using, which you fixed). Just not sure what to name this method.
What do you think?

I suppose that would be good if the caller of QueryAsync explicitly needed to have the number of series match the number of queries. You would essentially be adding a null item to the list, which was what was throwing an exception before AddRange(null).
Seems like a mad idea to just add the null item to the list, as this would require the caller to always check first if any items are null in the list before even pulling data from the series.
In my opinion the caller would probably prefer a list of non-null objects, and they can use the Serie.Name to identify which of their queries produced results and if something is not found in the list, that means it didn't.

The thing is - I would like to be consistent to what InfluxDB returns in this case, and if you execute multiple queries at once - you will get multiple results where each of them can contain multiple series inside.
I would of course do the null check and return an empty serie list if the result didn't return any series.
That would then be the default way of multiQuery, and the other method would return a flattened list.

Ok, so I kept the same signature for QueryAsync for multiple queries and added an additional method called MultiQueryAsync which does the thing I described. Thanks for the contribution. :)

Ahh I was tired and wasn't fully understanding what you were trying to get to. Totally makes sense now, as the normal single query was flattening the results and since there should be only 1 result that totally makes sense, and was semi-useful to the caller (as the "result" contains no additional data currently, just series). You may want to remove the method flattening the list in that case, as it's not strictly necessary and the caller can easily flatten the model themselves. Also may be semi-confusing to someone not understanding that the ONLY difference between those two (currently) is that they are flattening the list of series.
So yeah IEnumerable<IEnumerable> may look funny but from a design perspective it is mirroring influxdb, so I agree. It may prove useful in the future if the "results" gain additional properties to identify themselves other than just the series information.


Race condition in cms4 and possibly in cms3

Description
While running the tests, only one of the tests failed in what seems to be a race condition (since we run the tests in parallel).
## Steps to reproduce
See the following.
https://travis-ci.org/github/divio/django-cms/jobs/668617109?utm_medium=notification&utm_source=github_status
Expected behaviour
Test should not fail with duplicate error.
Actual behaviour
Tests fail with duplicate error.
Additional information
None.

@jonathan-s we welcome and PR that helps to improve the stability of the tests.

Hello @jonathan-s,
Thank you very much for your contribution to the django CMS project. The django CMS project has recently joined forces under the banner of the newly founded django CMS Association to revive the development of django CMS. For this reason we have analyzed all pull requests and issues in detail and decided which ones fit the product vision we want to go with django CMS.
Due to the age of this issue we are unable to safely accept the contribution. If you feel this issue is still required please inform a member of our technical committee<EMAIL_ADDRESS>Nevertheless we thank you for your participation and would like to emphasize that we do not want to discourage you in your contribution to the django CMS project. On the contrary, we encourage everyone who is interested in the success of django CMS to join us!
If you want to learn more about the work of the technical committee, our product vision and how you can become a member of the dCA, you can find all information here:

The Technical Committee of the django CMS Association
Technical Roadmap and Product Vision for django CMS
How to become a member of the django CMS Association

We are looking forward to every companion who would like to shape the future of django CMS together with us!
If you have any questions, feel free to reach out to Nicolai, our Community Manager<EMAIL_ADDRESS>Sign up for the Association now!

Hoping to get you onboard @jonathan-s, we can reopen this if you feel this is still relevant.


To improve speed, changed crate from std::collections::HashMap to has…

Closes #3
What Changed

changed create from collections::HashMap to hashbrown::HashMap
fixed clippy warn

Evidence

main

PS > cargo bench
...
Gnuplot not found, using plotters backendread 90 records         time:   [2.2208 ms 2.2860 ms 2.3611 ms]
Found 8 outliers among 100 measurements (8.00%)
  2 (2.00%) high mild  6 (6.00%) high severe

Benchmarking read 90 records json: Collecting 100 samples in estimated 5.2486 s (1900 iterat                                                                                read 90 records json    time:   [2.6829 ms 2.7139 ms 2.7514 ms]
Found 10 outliers among 100 measurements (10.00%)  5 (5.00%) high mild  5 (5.00%) high severe

this PR

PS > cargo bench
...
Gnuplot not found, using plotters backend
read 90 records         time:   [1.9808 ms 1.9854 ms 1.9903 ms]
                        change: [-2.1321% -1.5279% -0.9885%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 8 outliers among 100 measurements (8.00%)
  7 (7.00%) high mild
  1 (1.00%) high severe

read 90 records json    time:   [2.5772 ms 2.5846 ms 2.5938 ms]
                        change: [+1.6065% +2.2200% +2.7533%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 6 outliers among 100 measurements (6.00%)
  2 (2.00%) high mild
  4 (4.00%) high severe

ご確認ありがとうございます。それではconflictを解消してマージしておきます

@hitenkoku お疲れ様です！時間がある時にconflictの解消とマージをお願いできますか？

Conflict解消が完了しました。マージいたします


V0.6 Support for Symfony 7

Thanks a lot for opening this PR. I was able to use your fork as a composer vcs repository and update a symfony application to version 7 successfully thanks to you

Hello, i use plesk, with no problem, but i planed to upgrade symfony to 7.0. With your PR install is ok, but now, no more connection my messagehandler ar no more call, and client have
 Error: socket hang up
It a difference from plesk ?

From plesk : (work with sf 6.4)
$wsServer = new WsServer(new WebSocketHandler());
$app = new HttpServer($wsServer);
$server = IoServer::factory($app, 8080);
$wsServer->enableKeepAlive($server->loop, 60);
$server->run();

And i try with doc provided :
$app = new \Ratchet\App('<IP_ADDRESS>', 8080, '<IP_ADDRESS>');
$app->route('/', new WebSocketHandler(), ['*']);
$app->run();

but my handler is never call ( with breakpoint) with this 2methods

WsServer.php
126:  $conn->send(Message::toString($response));
replace
17:
use GuzzleHttp\Psr7 as gPsr;
by
use GuzzleHttp\Psr7\Message;

Hi, Please I need This version for Sympony 7.0

why is this one not merged yet ?


Release 0.16.0

I plan to do a release in the next day or so. The main updates are improved single machine support (#514), and more documentation.

Version 0.16.0 is now available at https://pypi.org/project/cubed/ and https://anaconda.org/conda-forge/cubed


Upgrade to govukuk-frontend 5.3.1

👀 Purpose

To upgrade to the latest version of govuk-frontend-jinja (3.0.0) and govuk-frontend (5.3.1)

♻ What's changed

🖼 Images updated and removed St Edwards Crown images
🪮  Stylesheets updated to 5.1.0 and removed internet explorer 8 ones which are no longer supported
✂ JavaScript updated to 5.1.0 and ie8 removed (no longer supported)
🆙 Version of govuk-frontend-jinja to 3.0.2
🏗 To improve ease of future updates removed necessity to edit stylesheets. static_folder (repo folder) and static_url_path (stylesheet expected component path) now set in app/app.py.
♻ base.html refactored to replace refs to 4.3.0 with refs to 5.1.0 and links to stylesheets moved to block head only
🧪 Test updated to match change in subsequent page ref to Office 365

📝 Notes

The new version of govuk-frontend phase banner has changed to improve contrast for accessibility.

The gitleak error is picking up the stylesheet option to show/hide a password - not any actually shown password
The pylint error is a bug I think we previously discussed; it reckons some imports are in the wrong order - but they are clearly already in the order required.


[stable/metabase] Add yachtcaptain23 to as a reviewer and approver

Signed-off-by: Albert Wang<EMAIL_ADDRESS>Is this a new chart
No
What this PR does / why we need it:
Increase the number of maintainers and reviewers. I'm an employee at a tech startup that uses Metabase deployed on Kubernetes.
Special notes for your reviewer:
@pmint93 verbally approved me in https://github.com/helm/charts/pull/22320 as a maintainer.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @pmint93

Every changes to a chart need version bump. Please bump the chart version

@maorfr would you be able to assist? Thank you

@pmint93 would you re-review?

/ok-to-test

This PR looks good to me and can be merged. But reviewer also need to be added to this repository as read-only user, only repository maintainers can do so.
Maybe @davidkarlsen can help ?

@davidkarlsen Would you be able to assist?


Added resources, nodeSelector, tolerations, affinity and topoplogySpr…

Description
Added resources, nodeSelector, tolerations, affinity and topologySpreadConstaints sections to agent deployment template (similar to main pod deployment). The corresponding values are added under agent section in Values.yaml.
Related Issues
#4255
Testing
Manually tested with helm template to verify the output of the baked manifest.

I have added the sign-off. I am not entirely clear on the release procedures for the chart (changelog and version).

Manually add a bullet - Feature: item under the ## Next Release heading in charts/emissary-ingress/CHANGELOG.md.

It'd be good if this added a test under charts/emissary-ingress/ci.in/, but I honestly don't know how to do that, so it'd be unreasonable for me to ask that of you :)


Picture saving to local disk on local server install

My config/application.yml with regards to the google cloud config looks like:
# Google Cloud Storage API Bucket for image data. Deleting this will save to disk
##GCS_BUCKET: "GOOGLE_CLOUD_STORAGE_BUCKET_NAME_FOR_IMAGE_FILES"
# Google Cloud Storage ID for image data. Deleting this will save images to disk.
##GCS_ID: "GOOGLE_CLOUD_STORAGE: 'interop' id"
##GCS_KEY: "GOOGLE_CLOUD_STORAGE: 'interop' key"

If I try to take a picture it says:

Where do I need to adapt the config?

@Klimbim1 Please try this:

Run rails jobs:work. (This was recently added to the setup- I've updated the docs to reflect that)
Wait a few seconds.
Check the photo widget (http://localhost:3000/app/farmware)

Do any images come up on that page?
If you do see an image: you can safely ignore that log message and we will change the message in a future release. This is just a verbiage issue. You can find the real image URL by right clicking the image and selecting copy image address.
If you do not see any images: There may be another issue at play. Please send me more error logs as well as the contents of the Javascript console (CTRL + SHIFT + J) and I can take a deeper look.

Hi @RickCarlino,
it does not work; If I execute rails jobs:work this does something, here is a log file sequence which
comes over and over:

Delayed::Backend::ActiveRecord::Job Load (1.0ms)  UPDATE "delayed_jobs" SET locked_at = '2017-06-05 17:07:34.876563', locked_by = 'host:ubuntu pid:7511' WHERE id IN (SELECT  "delayed_jobs"."id" FROM "delayed_jobs" WHERE ((run_at <= '2017-06-05 17:07:34.875197' AND (locked_at IS NULL OR locked_at < '2017-06-05 13:07:34.875296') OR locked_by = 'host:ubuntu pid:7511') AND failed_at IS NULL) ORDER BY priority ASC, run_at ASC LIMIT 1 FOR UPDATE) RETURNING *
DEPRECATION WARNING: cleanup! is deprecated and will be removed from Rails 5.1 (use Rails.application.reloader.reload! instead of cleanup + prepare) (called from require at bin/rails:4)
DEPRECATION WARNING: prepare! is deprecated and will be removed from Rails 5.1 (use Rails.application.reloader.prepare! instead) (called from require at bin/rails:4)
Delayed::Backend::ActiveRecord::Job Load (1.5ms)  UPDATE "delayed_jobs" SET locked_at = '2017-06-05 17:07:39.996718', locked_by = 'host:ubuntu pid:7511' WHERE id IN (SELECT  "delayed_jobs"."id" FROM "delayed_jobs" WHERE ((run_at <= '2017-06-05 17:07:39.995229' AND (locked_at IS NULL OR locked_at < '2017-06-05 13:07:39.995330') OR locked_by = 'host:ubuntu pid:7511') AND failed_at IS NULL) ORDER BY priority ASC, run_at ASC LIMIT 1 FOR UPDATE) RETURNING *
DEPRECATION WARNING: cleanup! is deprecated and will be removed from Rails 5.1 (use Rails.application.reloader.reload! instead of cleanup + prepare) (called from require at bin/rails:4)
DEPRECATION WARNING: prepare! is deprecated and will be removed from Rails 5.1 (use Rails.application.reloader.prepare! instead) (called from require at bin/rails:4)
Delayed::Backend::ActiveRecord::Job Load (1.0ms)

Then this thingi appeared, I tried to go to the URL but it only says sorry :)

2017-06-05T10:10:59-0700: [Worker(host:ubuntu pid:7511)] Job ActiveJob::QueueAdapters::DelayedJobAdapter::JobWrapper (id=1) RUNNING

User Load (1.5ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1 LIMIT $2  [["id", 108], ["LIMIT", 1]]
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46] Performing ActionMailer::DeliveryJob from DelayedJob(mailers) with arguments: "UserMailer", "welcome_email", "deliver_now", #<GlobalID:0x0000000694a588 @uri=#<URI::GID gid://farm-bot/User/108>>
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46]   Rendering layouts/mailer.html.erb
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46]   Rendering user_mailer/welcome_email.html.erb within layouts/mailer
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46]   Rendered user_mailer/welcome_email.html.erb within layouts/mailer (0.9ms)
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46]   Rendered layouts/mailer.html.erb (2.8ms)
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46]   Rendering user_mailer/welcome_email.text.erb
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46]   Rendered user_mailer/welcome_email.text.erb (0.8ms)
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46] UserMailer#welcome_email: processed outbound mail in 24.9ms
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46] Sent mail to<EMAIL_ADDRESS>(8.5ms)
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46] Date: Mon, 05 Jun 2017 10:10:59 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Subject: Welcome to The FarmBot Web App!
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_593590a3cd3f6_1d57ccb0dc3162e";
charset=UTF-8
Content-Transfer-Encoding: 7bit
----==_mimepart_593590a3cd3f6_1d57ccb0dc3162e
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Welcome to FarmBot, myuser
You please verify your email by visiting http://<IP_ADDRESS>:3000/verify.html?token=ERROR
Thanks for joining and have a great day!
----==_mimepart_593590a3cd3f6_1d57ccb0dc3162e
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit

Welcome to FarmBot, myuser

  Please verify your email by clicking here
  here.

Thanks for joining and have a great day!

----==_mimepart_593590a3cd3f6_1d57ccb0dc3162e--
[ActiveJob] [ActionMailer::DeliveryJob] [f5d7909b-3775-46da-8c67-3fb863483f46] Performed ActionMailer::DeliveryJob from DelayedJob(mailers) in 44.34ms
[Worker(host:ubuntu pid:7511)] Job ActiveJob::QueueAdapters::DelayedJobAdapter::JobWrapper (id=1) FAILED (4 prior attempts) with Errno::ECONNREFUSED: Connection refused - connect(2) for "localhost" port 1025
2017-06-05T10:10:59-0700: [Worker(host:ubuntu pid:7511)] Job ActiveJob::QueueAdapters::DelayedJobAdapter::JobWrapper (id=1) FAILED (4 prior attempts) with Errno::ECONNREFUSED: Connection refused - connect(2) for "localhost" port 1025
(1.0ms)  BEGIN
SQL (1.3ms)  UPDATE "delayed_jobs" SET "attempts" = $1, "run_at" = $2, "locked_at" = $3, "locked_by" = $4, "updated_at" = $5 WHERE "delayed_jobs"."id" = $6  [["attempts", 5], ["run_at", 2017-06-05 17:21:29 UTC], ["locked_at", nil], ["locked_by", nil], ["updated_at", 2017-06-05 17:10:59 UTC], ["id", 1]]
(1.1ms)  COMMIT

If I visit this URL: http://<IP_ADDRESS>:3000/verify.html?token=ERROR
it says in the error java console log:

:3000/api/users/verify/ERROR Failed to load resource: the server responded with a status of 404 (Not Found)
verification.ts:66 Uncaught (in promise) Error: USER VERIFICATION FAILED!
at eval (verification.ts:66)
at step (verification.ts:32)
at Object.eval [as throw] (verification.ts:13)
at rejected (verification.ts:5)
at

Cheers!

@Klimbim1 May I see your application.yml? Self hosted setups are very different than the setup used on the main server. I suspect you are hitting an edge case that we do not test against.
Please remove these keys (if any) before sharing: GCS_KEY, RSA_KEY, ACME_SECRET, DATABASE_URL, DEVISE_SECRET, GCS_KEY, ROLLBAR_ACCESS_TOKEN, SENDGRID_PASSWORD.

@RickCarlino for sure, here you are:
` ░██████░░███████░░█████░░░██████░░
░██░░░░░░░░██░░░░██░░░██░░██░░░██░  READ THIS BEFORE USING IT.
░██████░░░░██░░░░██░░░██░░██████░░  SEE NOTES BELOW:
░░░░░██░░░░██░░░░██░░░██░░██░░░░░░
░██████░░░░██░░░░░████░░░░██░░░░░░
You will hit issues if any of these are set to the wrong value.
Please read each line of this file before starting the server.

=============================================================================

Self hosting users can safely delete this (a new key will be created).
This key is used to exchange secrets between bots and MQTT servers (important
if you don't use SSL)
SERVER WONT WORK IF YOU FORGET TO DELETE THIS EXAMPLE TEXT BELOW.
ADD A REAL RSA_KEY OR DELETE THIS LINE!!
##RSA_KEY: "Change this! Keys look like -----BEGIN RSA ........."
If you use Let's Encrypt for SSL,
you must set this when renewing SSL.
Otherwise, not required and CAN BE REMOVED.
##ACME_SECRET: "-----"
If your server is on a domain (eg: my-own-farmbot.com), put it here.
<IP_ADDRESS> is a good value because it exposes all IPs on your machine.
DONT USE localhost. USE A REAL IP!
API_HOST: "<IP_ADDRESS>"
3000 for local development. 443 is using SSL. You will need sudo to use PORT
80 on most systems.
API_PORT: "3000"
This can be set to anything.
Most users can just delete it.
This is used for people writing modifications to the software, mostly.
##DOCS: "Set this to any value if you want to generate API docs after running tests"
Most users can delete this.
Used by people who pay for managed database hosting.
DATABASE_URL: XXXX
MUST REPLACE.
Generate a secret by typing rake secret into the console.
DEVISE_SECRET: XXXX
Most personal server users can delete this.
##FORCE_SSL: "Remove this if not using HTTPS://"
Firmware update server. Use default if you don't have a special use case.
Off grid servers may have issues connecting to our update URL.
FW_UPDATE_SERVER: "https://api.github.com/repos/FarmBot/farmbot-arduino-firmware/releases/latest"
OS_UPDATE_SERVER: "https://api.github.com/repos/farmbot/farmbot_os/releases/latest"
Google Cloud Storage API Bucket for image data.
Deleting this will save to disk.
Most self hosting users will want to delete this.
##GCS_BUCKET: "GOOGLE_CLOUD_STORAGE_BUCKET_NAME_FOR_IMAGE_FILES"
Google Cloud Storage ID for image data.
Deleting this will save images to disk.
Most self hosting users will want to delete this.
##GCS_ID: "GOOGLE_CLOUD_STORAGE: 'interop' id"
Most self hosting users will want to delete this.
##GCS_KEY: "GOOGLE_CLOUD_STORAGE: 'interop' key"
Most self hosting users will want to delete this.
##HEROKU_SLUG_COMMIT: "This is set by Heroku, used by Frontend to show current version."
Where is your MQTT server running?
Use a REAL IP ADDRESS if you are controlling real bots.
<IP_ADDRESS> is only OK for software testing. Change this!
MQTT_HOST: "<IP_ADDRESS>"
Delete this line if you are not an employee of FarmBot, Inc.
##NPM_ADDON: "Used by FarmBot, Inc. to load proprietary extras, like Rollbar."
Same as above. Can be deleted unless you are a Rollbar.IO customer.
##ROLLBAR_ACCESS_TOKEN: "____"
FarmBot, Inc. uses SendGrid to send emails.
Delete these if you don't use send grid.
##SENDGRID_PASSWORD: "Used by FarmBot, Inc"
##SENDGRID_USERNAME: "Used by FarmBot, Inc"
For email delivery. Who is your email host?
SMTP_HOST: "smtp.sendgrid.net"
Optional with default of 587
SMTP_PORT: 587
Set this if you don't want to deal with email verification of new users.
(self hosted users)
NO_EMAILS: TRUE`

Please try the following:

Pull down the latest version: git pull origin master
Update dependencies: bundle install
Migrate your database: rails db:migrate
Install the latest frontend: ./install_frontend.sh
Start the server

and try taking a photo again. Do you see anything on the farmware page? Do you see any errors?
I found a few issues that might affect folks who do not have an email server, but I am not sure it was related to anything here.
Also, could you try running rspec spec? You should see 0 failures after running the tests.

Hi @RickCarlino,
@MrChristofferson recommended to me to fork all three projects, mqtt, frontend and web-api in order to be up to date and see the changes on a lets say weekly basis when I pull the latest versions.
Therefore I need to adapt the frontend starting thing. Could you elaborate how to correctly start the frontend?
However if I go into Farmbot-Web-API and type rspec spec it says the following:
myuser:~/Farmbot-Web-API$ rspec spec
WARNING: Skipping key "FW_UPDATE_SERVER". Already set in ENV.
WARNING: Skipping key "OS_UPDATE_SERVER". Already set in ENV.
WARNING: Skipping key "MQTT_HOST". Already set in ENV.
WARNING: Use strings for Figaro configuration. 587 was converted to "587".
WARNING: Use strings for Figaro configuration. true was converted to "true".
Randomized with seed 25127
F
Failures:

Api::ToolsController#create creates a new tool
Failure/Error: let(:user) { FactoryGirl.create(:user) }
NoMethodError:
undefined method `password=' for #User:0x0000000737c358
./spec/controllers/api/tools/create_spec.rb:6:in `block (3 levels) in <top (required)>'
./spec/controllers/api/tools/create_spec.rb:10:in `block (3 levels) in <top (required)>'

Finished in 0.08596 seconds (files took 8.45 seconds to load)
1 example, 1 failure
Failed examples:
rspec ./spec/controllers/api/tools/create_spec.rb:9 # Api::ToolsController#create creates a new tool
Randomized with seed 25127
Coverage report generated for RSpec to /home/klimbim/Farmbot-Web-API/coverage. 1174 / 1833 LOC (64.05%) covered.
I looked at the install_frontend.sh and I am basically stuck with
myuser:~/farmbot-web-frontend$ npm run build
it says at the end:
ERROR in /home/myuser/farmbot-web-frontend/src/images/__tests__/image_flipper_test.tsx (5,25): error TS7016: Could not find a declaration file for module 'deep-freeze'. '/home/myuser/farmbot-web-frontend/node_modules/deep-freeze/index.js' implicitly has an 'any' type. Child extract-text-webpack-plugin:                                    Asset     Size  Chunks             Chunk Names     3f0fc622e73ad38dc3e0bbb86eb73182.eot  77.5 kB          [emitted]       95a702087e984248205235ac8cfe1c43.eot  79.4 kB          [emitted]          [0] ./~/css-loader/lib/css-base.js 1.51 kB {0} [built]        [1]<EMAIL_ADDRESS>178 kB {0} [built]        [2] ./~/css-loader!./~/sass-loader!./src/css/_index.scss 43.9 kB {0} [built]        [3]<EMAIL_ADDRESS>82 bytes {0} [built]        [4]<EMAIL_ADDRESS>82 bytes {0} [built]        [5]<EMAIL_ADDRESS>103 kB {0} [built]        [6]<EMAIL_ADDRESS>103 kB {0} [built]        [7]<EMAIL_ADDRESS>106 kB {0} [built]        [8]<EMAIL_ADDRESS>106 kB {0} [built] npm ERR! code ELIFECYCLE npm ERR! errno 2 npm ERR<EMAIL_ADDRESS>build:node_modules/webpack/bin/webpack.js --config tools/webpack.config.prd.js --display-error-details`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/myuser/.npm/_logs/2017-06-07T20_09_10_499Z-debug.log`
I tried to install the deep-freeze module but that does not work either... :/ any ideas?

@Klimbim1 I saw that error yesterday and thought I patched it already :thinking: .
What do you see when you run git log -1?

/Farmbot-Web-API$ git status
On branch master
Your branch is up-to-date with 'origin/master'.
Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	deleted:    config/application.example.yml
	deleted:    config/database.example.yml
	modified:   db/schema.rb
	modified:   package.json

Untracked files:
  (use "git add <file>..."  As such, it will not have the fix. Your options would be to:

Pull down the latest version from farmbot/master, re-install dependencies and re-flash your device with the "edge" version of FarmBot OS (requires building from source).
Wait until version 4.0.0 (and a pre-compiled FarmBot OS image) come out, probably a week or so away.

Hi @RickCarlino;
thanks very much for the support. How can I see which OS version works together with which Web-API version?
Cheers
Klim

@Klimbim1

Please see FarmBot OS build instructions to learn how to compile FarmBot OS from source. Raise issues there if you require further assistance with building.
With regards to maintaining version compatibility: Your best bet is to simply use the latest version of each of the packages (Latest FBOS, latest MQTT, latest FE, etc). I would not recommend running older versions if possible. If you still wish to do this, you may use git reset to reset all repos to a particular date, but this is recommended only if you have a good understanding of Git.

We will begin tagging releases again with the upcoming 4.0.0 release.


Tests failing on ppc64le architecture for v1.6.0

Hi,
In my attempt to port this package to ppc64le architecture, I have come across a few failures. Attaching the logs I get after running the tests:
PHPUnit 8.5.26 #StandWithUkraine

Runtime:       PHP 7.2.24
Configuration: /MinkSelenium2Driver/phpunit.xml.dist

Testing Driver test suite
..F....SSSSS...................................................  63 / 177 ( 35%)
.......SSSSE....S...............................S.............. 126 / 177 ( 71%)
..................F............................E..F             177 / 177 (100%)

Time: 3.23 minutes, Memory: 10.00 MB

There were 2 errors:

1) Behat\Mink\Tests\Driver\Basic\IFrameTest::testIFrame
WebDriver\Exception\UnknownError: data did not match any variant of untagged enum FrameId at line 3 column 1
Build info: version: '3.141.59', revision: 'e82be7d358', time: '2018-11-14T08:25:53'
System info: host: '5dfaf1b6ce14', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'ppc64le', os.version: '4.18.0-348.el8.ppc64le', java.version: '<IP_ADDRESS>'
Driver info: driver.version: unknown

/MinkSelenium2Driver/vendor/instaclick/php-webdriver/lib/WebDriver/Exception.php:156
/MinkSelenium2Driver/vendor/instaclick/php-webdriver/lib/WebDriver/AbstractWebDriver.php:171
/MinkSelenium2Driver/vendor/instaclick/php-webdriver/lib/WebDriver/Session.php:290
/MinkSelenium2Driver/src/Selenium2Driver.php:426
/MinkSelenium2Driver/vendor/behat/mink/src/Session.php:321
/MinkSelenium2Driver/vendor/mink/driver-testsuite/tests/Basic/IFrameTest.php:17
phpvfscomposer:///MinkSelenium2Driver/vendor/phpunit/phpunit/phpunit:97

2) Behat\Mink\Tests\Driver\Js\WindowTest::testWindow
WebDriver\Exception\UnknownError: missing field `handle` at line 3 column 1
Build info: version: '3.141.59', revision: 'e82be7d358', time: '2018-11-14T08:25:53'
System info: host: '5dfaf1b6ce14', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'ppc64le', os.version: '4.18.0-348.el8.ppc64le', java.version: '<IP_ADDRESS>'
Driver info: driver.version: unknown

/MinkSelenium2Driver/vendor/instaclick/php-webdriver/lib/WebDriver/Exception.php:156
/MinkSelenium2Driver/vendor/instaclick/php-webdriver/lib/WebDriver/AbstractWebDriver.php:171
/MinkSelenium2Driver/vendor/instaclick/php-webdriver/lib/WebDriver/Session.php:273
/MinkSelenium2Driver/src/Selenium2Driver.php:418
/MinkSelenium2Driver/vendor/behat/mink/src/Session.php:311
/MinkSelenium2Driver/vendor/mink/driver-testsuite/tests/Js/WindowTest.php:17
phpvfscomposer:///MinkSelenium2Driver/vendor/phpunit/phpunit/phpunit:97

--

There were 3 failures:

1) Behat\Mink\Tests\Driver\Custom\TimeoutTest::testInvalidTimeoutSettingThrowsException
Failed asserting that exception of type "\Behat\Mink\Exception\DriverException" is thrown.

phpvfscomposer:///MinkSelenium2Driver/vendor/phpunit/phpunit/phpunit:97

2) Behat\Mink\Tests\Driver\Js\EventsTest::testRightClick
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'right clicked'
+'single clicked'

/MinkSelenium2Driver/vendor/mink/driver-testsuite/tests/Js/EventsTest.php:45
phpvfscomposer:///MinkSelenium2Driver/vendor/phpunit/phpunit/phpunit:97

3) Behat\Mink\Tests\Driver\Js\WindowTest::testWindowMaximize
Failed asserting that 720 is equal to 100 or is less than 100.

/MinkSelenium2Driver/vendor/mink/driver-testsuite/tests/Js/WindowTest.php:95
phpvfscomposer:///MinkSelenium2Driver/vendor/phpunit/phpunit/phpunit:97

--

Here's the steps I followed to reach to this error.
Some additional info:
OS : Red Hat Enterprise Linux 8.5
Arch : ppc64le
Selenium : 3.141.59
Gecko Driver : 0.30.0
Firefox : 91
I know that the tests here are using Firefox 47, Selenium 2.53.1 and Gecko Driver 0.10.0 by going through the github actions page. However Firefox 47 isn't available on ppc64le, which is why I decided to go with a newer version of Selenium. Tried this on an x86_64 VM and they work fine. Not sure if I'm missing anything here, would like some help.

Hey @stof, do you have any pointers on this?


Which coordinates does pairix return?

For example, let's say the .pairs output is:
READID	chrX	16145203	chrX	16213822	+	-	UU
Is 16145203 the left-most or the right-most coordinate on this read? Likewise for 16213822? Does this change based on the strand?

I think it's the 5'end of the read, but the pairs format itself does not require it to be.

Ok, so for the "chrX 16213822" then the actual read would extend from <16213822 to 16213822, since it is negative stranded, so the 5'end would have the highest number in terms of genomic coordinates? Would it be possible to get confirmation on this? I was hoping to make an analysis based on these ideas and so these questions are very useful.

It depends on how the file was generated. e.g.
https://github.com/4dn-dcic/pairix/blob/master/util/bam2pairs/README.md#usage
this utility has two options (5'end is default but could also use
leftmost). Where did you get tgese pairs files?
On Sun, Aug 8, 2021, 7:04 PM gabedolpten @.***> wrote:

Ok, so for the "chrX 16213822" then the actual read would extend from
<16213822 to 16213822, since it is negative stranded, so the 5'end would
have the highest number in terms of genomic coordinates? Would it be
possible to get confirmation on this? I was hoping to make an analysis
based on these ideas and so these questions are very useful.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/4dn-dcic/pairix/issues/70#issuecomment-894877848, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHLO3DTY5E64RNJXFIVX7LT34LPNANCNFSM5BWUFYWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Currently no way of defining comments

The compiler should have a way of ignoring lines of code / text

Related to: https://github.com/philippwiesner/vega/issues/8#issue-631508739


Add adaptive histogram equalization algorithm

Description
Adaptive histogram equalization(AHE) is used in cases when histogram equalization cannot properly contrast enhance due to the presence of both very bright and very dark regions.
AHE brings out the local information in an image by dividing the image into tiles and then individually performs HE on each of them. Later to remove tiling artefacts  a bi-linear interpolation is performed using mappings from the adjacent tiles. Finally to suppress noise, use contrast clipping to limit the #pixels of a particular value.
This PR adds code, tests & an example to show how to use the GIL AHE algorithm.
Dependency
This PR requires support of GIL histogram class and the global HE algorithm currently running in PR #499 & PR #514 respectively . Reviewing and merging can only be done post completion of former PRs.

References

Graphics Gems 4 - Paul S. Heckbert Pg. 474
Adaptive Histogram Equalization and its Variations
Discussion thread on boost-gil mailing list

Tasklist

[x] Add test case(s)
[ ] Ensure all CI builds pass
[ ] Review and approve

@codejaeger I've just git merge-updated your gsoc/ahe branch from the latest develop

@codejaeger I've just git merge-updated your gsoc/ahe branch from the latest develop


os initial-advertise-peer-urls

Bug reporting
A good bug report has some very specific qualities, so please read over our short document on
reporting bugs before you submit your bug report.
To ask a question, go ahead and ignore this.

I do not really understand this issue. Please make it clear it before re-open.


How do I know that QR is scanned on web interface?

Even after scanned QR code successfully and getting "Client is ready" message on the console I am trying to understand how can I notify or create an event for successful scanning of the QR. Please help me.

Hello @manjeetAGS
If you have any possibility, can you share here?


For component infers types wrong in Typescript

Hey 👋 Just found this library and decided to give it a try.
I just wanted to use it for a simple(ish) use case in my app so i followed the "List of messages example" in react section.
I created observable like this
const {modules} = useObservable({
  modules: loaderData.course.modules,
});

The type for modules looks like this
JsonifyObject<{
    id: string;
    status: "PUBLISHED" | "DRAFT";
    title: string;
    position: number;
    sections: {
        id: string;
        status: "PUBLISHED" | "DRAFT";
        title: string;
        position: number;
    }[];
}>[]

Then i render it like
 <For each={modules} optimized>
  {(courseModule) => {
    const index = getObservableIndex(courseModule);
    console.log(index, courseModule.id.get());

    return

;
  }}
</For>

However i think the type for courseModule is not inferred correctly because it says that property id does not exist. I wanted to call courseModule.id.get()
Property 'id' does not exist on type 'ObservablePrimitiveBaseFns<JsonifyObject<{ id: string; status: "PUBLISHED" | "DRAFT"; title: string; position: number; sections: { id: string; status: "PUBLISHED" | "DRAFT"; title: string; position: number; }[]; }> | undefined>'.

Am i missing something?

May be related #266 ?

Hmm, I’m not sure what it would be, but the types are much improved in the next version, so if it’s fix there then I think we’re all good - it’s coming to v3 alpha soon.


Rename ReconstructedContainer.tel_ids -> telescopes, fixes 1953

Isn't this less obvious than using tel_ids? There are multiple ways to refer to a telescope:  tel_ids, tel_indices, and tel_names (maybe more).  So just saying  "telescopes" it is not clear which one is expected.  tel_ids makes it clear this is a telescope ID, and not a telescope name, or telescope index.

@kosack the reasoning here is that e.g. when using the table loader, it is actually a boolean mask, not a list of tel_ids. see #1953

An alternative would be to just directly fill the mask and call it tel_mask?

ah, yes, I forgot that there is a translation to a mask when we write it, and then the True values are actually telescope_indices, not tel_ids... so ok, let's keep it as telescopes, and the representation of these should be up to the user.  I still like having it a list of tel_ids in the event, since that is somewhat friendlier to read


How to chanage app font and editor font

the default font is ugly,i want to chanage it,how?

and how to chanage language?

Sorry, I didn't see this issue before. Nowadays it isn't possible. But shouldn't be too complicated to make it configurable. Although I'm fine with the current font so I won't spend time on this. But I could help you out guiding you on making those changes.


Column options menu partially hidden

ID <164>
Reporter <@StrongestNumber9>

As a side note I think it could benefit from some divisors between option groups

This will be fixed when the pivot table has been replaced with datatables


Add support for OpenHarmony

This PR adds support for OpenHarmony targets:

aarch64-linux-ohos
arm-linux-ohos

Compiler team MCP: https://github.com/rust-lang/compiler-team/issues/568
OpenHarmony uses a fork of musl with minor modifications, so most of the code is shared with other musl targets. Additionally, although OpenHarmony uses musl 1.2, it is still ABI-compatible with musl 1.1 so the existing bindings should continue to work until https://github.com/rust-lang/libc/pull/3068 is merged.
A PR to add the targets to rustc will follow after this is merged.

Out of curiosity, does OpenHarmony provide a new polling system, or does it just use Epoll like Linux does?

These OpenHarmony targets run on top of the Linux kernel and therefore support the same standard mechanisms that Linux provides. However OpenHarmony also supports running on embedded devices & microcontrollers with a custom kernel (LiteOS), but these targets are not currently supported in this PR.

Thanks!
@bors r+

:pushpin: Commit 3f5128e77a04257a661335e9582a7f1b1e56d118 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 3f5128e77a04257a661335e9582a7f1b1e56d118 with merge da4f8f82e367c556f889992c66ec89e84c1cbf52...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing da4f8f82e367c556f889992c66ec89e84c1cbf52 to master...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing da4f8f82e367c556f889992c66ec89e84c1cbf52 to master...

:eyes: Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.


Alignment passthrough

This should add the missing functionality to train a student with alignment without having to use hacks.

Add alignment support to the Placeholder modifier, and
implement "retokenization" using a sentencepiece vocab so we

use word alignments (as opposed to spm token alignments) in our modifiers
but can pass the alignment info to marian in a useful manner

Right now I'm implementing this as an extension of the Placeholder filter for compatibility reasons. But it can be used as its own stand-alone filter, Retokenize.
The tokenizer parameter of that thing is a bit of a lie, as it always gives you the detokenized output. But it will use the tokenizer parameter to tokenize that detokenized output so it can compute the correct alignment between the new tokens and their indices.

(I'd really, really like to replace the hidden end-to-end tests in test_placeholders.py with proper single test case unit tests that can be reasoned about and aren't opaque and hidden in external files. We already have test_endtoend.py for those.)

One thing that I forgot to tell you is that i tested decoding with pre-spm'd text and it works, because we have one-to-one token match. This means that we should be able to apply spm modifications here and not inside marian during training and then we can solve the problem of the sentences that are getting marked as tainted due to modifications.

Oh cool! How would you pass this to Marian? Do you just give it the spm token ids as numbers?
I initially had much bigger plans for this change, but moved them over to #29 and tried to keep this change backwards compatible as much as possible. Having an option to actually produce spm token ids and specify things like spm sampling parameters would fit easily in that structure.

One thing that I forgot to tell you is that i tested decoding with pre-spm'd text and it works, because we have one-to-one token match. This means that we should be able to apply spm modifications here and not inside marian during training and then we can solve the problem of the sentences that are getting marked as tainted due to modifications.

Oh cool! How would you pass this to Marian? Do you just give it the spm token ids as numbers?
I initially had much bigger plans for this change, but moved them over to #29 and tried to keep this change backwards compatible as much as possible. Having an option to actually produce spm token ids and specify things like spm sampling parameters would fit easily in that structure.

I was overeager here, hmm. It works for simple sentences, but not for complex ones because spm_encode | spm_encode != spm_encode. My bad. Still, maybe can bring it up to the next marian meeting if we can get such a simple option in.

Sorry this pull request exploded a bit once I tried to fix the end-to-end tests and discovered some bugs thanks to them.
I think it is mergeable now, assuming we still train marian with full text (re https://github.com/marian-nmt/marian-dev/pull/1003) and we do the "oh this can be better" data type rewrite later (re #29).
Comments are welcome. Also on the DatasetReader reads more iterations than it is strictly supposed to according to until epoch N when batch_size is >> len(dataset), and whether we should fix that properly now or open an issue for it.

Also, after the rework of the tests, we get 110 sentences, as opposed to 200. It's still not an exact match of the number of lines in a batch, but closer. Should we just write this in the notes and call it a day?

I opened #30 for this.

How difficult would it be to drop invalid sentences with a warning? Last time i looked it got a bit hairy. Maybe just return "" would just drop the sentence altogether?

Just implemented that. With backtraces for debugging and all :)


Added brand accent to variables 70%

When I changed the @brand-primary the accent colours didn't change to match, also the colours in the nav bar didn't change.

Only bit I disagree with here is the addition of the 'Accent Pink`: it might be better added defined in the Sales Tenancy's theme stylesheet. Else somebody will see that colour and start using it, despite it not being in the style guide!
But otherwise thank you for sorting out those variables!


Nested inline panels are ignored by page copy

Issue Summary
Nested inline panels were introduced in Wagtail 2.8. However, there are no tests for if page.copy() copies them and it appears to be broken on Wagtail 2.11.
Not sure if this was broken by the recent refactoring that's been done on the copy method.
Steps to Reproduce

Create an instance of testapp.EventPage with a speaker and award
Call .copy() on that page
The resulting page will have the speaker but not the award

Technical details

Python version: 3.8
Django version: 3.0
Wagtail version: 2.11 Alpha
Browser version: Firefox 71

FWIW, nested inline panels were downgraded to 'experimental' in 7a92ca5ad916d7e37de11a24080f5b0ca06dbd10 as there are still some outstanding issues - in particular, ID assignment and JS initialisation are still erratic because the <script> blocks containing the HTML fragments get nested, resulting in IDs with multiple levels of __prefix__ placeholders and no way for the handling code to know which ones refer to the outer or inner fragment.


Adapt for EndpointContext.

Adjust observe test to set destination context.
Intended to provide first experience on using an endpoint context.
(DON'T MERGE NOW :-) )
Signed-off-by: Achim Kraus<EMAIL_ADDRESS>

I extend the PR to demonstrate how notifies could be processed with address change using a  new matcher.
The commits are split to make it easier to follow up the required changes.
Currently the DTLS session is only resumed, when the update reaches the timeout (about 80s).

I take a look at this. I think the main idea is here. I did not review it in details as I don't really know if this just a POC or more ?
Do you consider EndpointContext as finished ?
I plan to create a new branch cf_integration and start to integrate last californium changes.
This could allow us to validate last californium changes and help us to prepare a californium 2.0.0-M6 release ?

@sbernard31
My idea was to show  my "intention of using the endpoint context with leshan".
But though I'm didn't have a closer look now for months, it's possible, that this doesn't fit at all to your plans.

Do you consider EndpointContext as finished ?

Yes, at least, I think the feedback should now be from the consuming projects. Depending on that, there may be some changes (as usual :-) ).

I plan to create a new branch cf_integration and start to integrate last californium changes.
This could allow us to validate last californium changes and help us to prepare a californium 2.0.0-M6 release ?

Go for it! Or do you mean, releasing californium 2.0.0-M6 in order to make such a leshan cf_integration branch? We could do this, if it helps :-).

My plan :

Create a cf_integration branch in Leshan
Try to use the new EndpointContext feature.
Make changes in cf if needed.
Then release the Californium 2.0.0-M6.

Sounds great. I'm looking forward for the feedback.

I create a PR #440 which integrates your work.
That's sound really good.
I think we should consider to release a 2.0.0-M6 version for californium.
Do you have some PR/issue you want to fix before ?
Personally, I would like to reuse "test code" from californium so need your opinion about https://github.com/eclipse/californium/issues/471


fix: Specify Icon onPress prop is inherited from TouchableOpacity

Please verify the following:

[x] yarn ci:test jest tests pass with new tests, if relevant
[x] README.md has been updated with your changes, if relevant

See discussion here for more info.

Nice, that's better for sure.


magic-tech vs klayout-drc consistency, re: 5LM + 9k: If 9k refers to "6th metal" metaltop (gds:53), why doesn't magic-tech support it?

Actual Behavior
magic-tech & klayout-drc appear almost consistent in: Treating gf180mcuC as: "5LM", where upper-most metal is metal5 (gds:81/0). And there is no: via5, metaltop (gds:82/0, 53/0).
That is, klayout-drc has a primary METAL_LEVEL conditional that assigns meta-layers like top_via,top_metal, and only for "6LM" are top_via,top_metal assigned to via5,metaltop; for "5LM" they get via4,metal5.
(Although it unconditionally connects metal5 thru via5 to metaltop; which I believe is a bug: #8).
The klayout-drc has thickness-specific (6k, 9k, 30k) rule sections.
But 6k & 9k sections operate on metaltop (gds:53/0) (not meta-layer top_metal, i.e. metal5 for 5LM), which does not exist in gf180mcuC.
Then why mention "+ 0.9um thick top metal" in gf180mcuC PDK .config/nodeinfo.json description, if C does not use the layer?
Expected Behavior
If for "5LM + 9k", the "9k" describes thickness of metal5 (gds:81/0) and there are no via5,metaltop: klayout-drc 9k section needs fix, and must not connect using via5,metaltop (#8) and more rules on these layers need to be disabled.
If for "5LM + 9k", the "9k" means via5,metaltop (0.9um thick, gds:82/0, 53/0) ARE used for a "6th" metal (in addition to met1-5): magic-tech needs fix.
Specifications
github.com//efabless/globalfoundries-pdk-libs-gf180mcu_fd_pr.git : rules/klayout/drc/{gf180mcu.drc,run_drc.py}
8cbf289 2022-12-01T00:12:24-08:00 (HEAD -> main, origin/main, origin/HEAD) Update run_drc_parallel.py
volare-pdk (2022.11.25): gf180mcu/versions/35c7265f51749ad8d9fdbb575af22c7c8fab974e/gf180mcuC/libs.tech/magic/gf180mcuC.tech

@jeffdi and @mkkassem Could you please send us the correct stack used for the variant C?

5LM 9KA.  26 layers.  See attached.
-- Jeff
On Sun, Dec 4, 2022 at 7:30 AM Amro Tork @.***> wrote:

@jeffdi https://github.com/jeffdi and @mkkassem
https://github.com/mkkassem Could you please send us the correct stack
used for the variant C?
—
Reply to this email directly, view it on GitHub
https://github.com/efabless/globalfoundries-pdk-libs-gf180mcu_fd_pr/issues/11#issuecomment-1336440789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKAZZ5M46AFMPDU3ZDCVUPDWLS2JHANCNFSM6AAAAAASR3XGF4
.
You are receiving this because you were mentioned.Message ID:
<efabless/globalfoundries-pdk-libs-gf180mcu_fd_pr/issues/11/1336440789@
github.com>

-- Jeff
On Sun, Dec 4, 2022 at 10:42 AM Amro Tork @.***> wrote:

@jeffdi https://github.com/jeffdi Where the attachment?
—
Reply to this email directly, view it on GitHub
https://github.com/efabless/globalfoundries-pdk-libs-gf180mcu_fd_pr/issues/11#issuecomment-1336488214,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKAZZ5OA47HVR6G5MYIN5T3WLTQ2XANCNFSM6AAAAAASR3XGF4
.
You are receiving this because you were mentioned.Message ID:
<efabless/globalfoundries-pdk-libs-gf180mcu_fd_pr/issues/11/1336488214@
github.com>

@jeffdi Github doesn't take attachments.

Check you email.
On Sun, Dec 4, 2022, 10:58 AM Amro Tork @.***> wrote:

@jeffdi https://github.com/jeffdi Github doesn't take attachments.
—
Reply to this email directly, view it on GitHub
https://github.com/efabless/globalfoundries-pdk-libs-gf180mcu_fd_pr/issues/11#issuecomment-1336491353,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKAZZ5NRL243Y6ZQNCKXGDLWLTSVLANCNFSM6AAAAAASR3XGF4
.
You are receiving this because you were mentioned.Message ID:
<efabless/globalfoundries-pdk-libs-gf180mcu_fd_pr/issues/11/1336491353@
github.com>

@sto :  The 9K refers to the top level metal, which is not GDS layer 53.  I know this is confusing.  Please see the GF DRC document section 2.8, as they have different nomenclature (but the same names!) for "INTERNAL REFERENCE ONLY" and "DESIGN ACTIVITIES".  My conclusion (after a number of readings) is that the first one is used by the foundry internally, but all users and design software should use the 2nd.  In the second, "MetalTop" (GDS layer 53) refers only to metal 6 in the 6-metal stack.  For "DESIGN ACTIVITIES", the top metal layer is "Metal5" with a note that it "must comply with MetalTop rule[s]".  They couldn' t have made it less clear if they had dipped the whole thing in mud.

@RTimothyEdwards I disagree with your comment above. I believe @sto6 is right. This is extremely weird for me. Usually, each metal has a unique identifier.
But here is why @sto6 is right, please refer to the document that Jeff has shared about the tape-out mask layer numbers of the BEOL:

92
95
80
85
88
91
93
94
96
97
98
99

and those map to the following layers in the stack according to the table found: Mask Layer Numbering  :

FuseTop
Pad
Metal1
Via1
Metal2
Via2
Metal3
Via3
Metal4
Via4
MetalTop
PIB

Meaning that we should have all the routing on beyond Via4 on 53/0 layer.
Thanks @sto6 for catching this. My team are still new to this and didn't get it.
I have updated the rule deck to make sure that the correct metal stack selected based on the switches.

No, that is completely wrong.  You can determine that simply by looking at the foundry GDS for the I/O cells from the 5LM set.  Metal5 is the top layer of metal, and can be found in all the GDS files as layer 81.  There is no layer 53 in these files.

@RTimothyEdwards Could you review the Tape-out agreement that efabless has with Globalfoundries? @jeffdi Could you please send @RTimothyEdwards that document? It provides the mask number as 98 which stands for gds number 53 as I mentioned above.
BTW @RTimothyEdwards, I had the same understanding at the beginning.

@atorkmabrains :  A mask number is not a GDS layer number.  Yes, the process will use mask layer 98.  No, mask layer 98 is not equivalent to design layer GDS number 53.

"I have updated the rule deck to make sure that the correct metal stack selected based on the switches."
If you modified rule decks based on faulty assumptions which were not approved by me, then revert them immediately.

@FaragElsayed2 Please take a look

@RTimothyEdwards We have cleaned all of that. I'll close the issue.


Do not create multiple virtualbox interfaces

It seems there are multiple virtualbox interfaces created with the same IP, I'm not sure about the implications on that but we should avoid doing it as it seems that we also have some concurrent issues with the creation of the virtualbox dhcpserver. This seems to be related with the parallel creation of the VMs.
# ip a s
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
2: enp1s0f0: <BROADCAST,MULTICAST,SLAVE,UP,LOWER_UP> mtu 1500 qdisc mq master bond0 state UP group default qlen 1000
    link/ether 0c:c4:7a:e1:40:96 brd ff:ff:ff:ff:ff:ff
3: enp1s0f1: <BROADCAST,MULTICAST,SLAVE,UP,LOWER_UP> mtu 1500 qdisc mq master bond0 state UP group default qlen 1000
    link/ether 0c:c4:7a:e1:40:96 brd ff:ff:ff:ff:ff:ff
4: bond0: <BROADCAST,MULTICAST,MASTER,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default qlen 1000
    link/ether 0c:c4:7a:e1:40:96 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/31 brd <IP_ADDRESS> scope global bond0
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/31 brd <IP_ADDRESS> scope global bond0:0
       valid_lft forever preferred_lft forever
    inet6 2604:1380:1:5300::19/127 scope global
       valid_lft forever preferred_lft forever
    inet6 fe80::ec4:7aff:fee1:4096/64 scope link
       valid_lft forever preferred_lft forever
5: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
    link/ether 02:42:60:59:69:e3 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/16 brd <IP_ADDRESS> scope global docker0
       valid_lft forever preferred_lft forever
6: vboxnet0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP group default qlen 1000
    link/ether 0a:00:27:00:00:00 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global vboxnet0
       valid_lft forever preferred_lft forever
    inet6 fe80::800:27ff:fe00:0/64 scope link
       valid_lft forever preferred_lft forever
7: vboxnet1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP group default qlen 1000
    link/ether 0a:00:27:00:00:01 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global vboxnet1
       valid_lft forever preferred_lft forever
    inet6 fe80::800:27ff:fe00:1/64 scope link
       valid_lft forever preferred_lft forever
root@jenkins-node-10:/home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s/1.11-gopath/src/github.com/cilium/cilium/test# ip r s
default via <IP_ADDRESS> dev bond0 onlink
<IP_ADDRESS>/8 via <IP_ADDRESS> dev bond0
<IP_ADDRESS>/31 dev bond0 proto kernel scope link src <IP_ADDRESS>
<IP_ADDRESS>/31 dev bond0 proto kernel scope link src <IP_ADDRESS>
<IP_ADDRESS>/16 dev docker0 proto kernel scope link src <IP_ADDRESS> linkdown
<IP_ADDRESS>/24 dev vboxnet0 proto kernel scope link src <IP_ADDRESS>
<IP_ADDRESS>/24 dev vboxnet1 proto kernel scope link src <IP_ADDRESS>

16:27:41  There was an error while executing `VBoxManage`, a CLI used by Vagrant
16:27:41  for controlling VirtualBox. The command and stderr is shown below.
16:27:41
16:27:41  Command: ["dhcpserver", "add", "--ifname", "vboxnet0", "--ip", "<IP_ADDRESS>", "--netmask", "<IP_ADDRESS>", "--lowerip", "<IP_ADDRESS>", "--upperip", "<IP_ADDRESS>", "--enable"]
16:27:41
16:27:41  Stderr: VBoxManage: error: DHCP server already exists

One of the runs https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/2146/execution/node/61/log/

@aanm do you think that there is still a CI-improvement to be done here?


Judges#1

Designed to be responsive for all devices.
Other Contributors:- @Aryan-401 @Avirat201189221 @Ikshan-Tango @sanchirang @aditya-talwar21 @Ave-Eren
Solves #18
Please Review @aarushi353

Seems fine to me

Thanks for your contribution! Happy coding! :-)


Work around the status_bar issues on simulators running 16.1+.

Since 16.1, xcrun simctl status_bar commands have no effect. Thanks to @saagarjha's info here we have a workaround by setting the environment variable SIMCTL_CHILD_SIMULATOR_RUNTIME_VERSION=16.0 when booting the device: https://federated.saagarjha.com/notice/AUwNsSsOOCWFc8qCvY

Thanks, works great. @twostraws, please pull.

Thank you! This will take a bit of thinking on my end for reasons that will become known in about 30 minutes, but I'll merge it because this is a great change – thank you! 🙌


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in 9d27b6d after 5 minutes.


How to query "Jó" book ?

how to query the book of "Jó" ?
verses/acf/jo/1 brings "João" book and verses/acf/jó/1 is not found.
thanks

@humbertocruz we have a bug. I will adjust this and update in production.

@humbertocruz try verses/acf/job/1.
I will update the documentation with this information, thank you

Thanks


unix: fix epoll_event padding on riscv64

Guys, can this be backported to release-branch.go1.13 ?

You use x/sys/unix from a release branch?

Actually K8s pin some dependencies by branch so when they move to Go 1.13, they move to the release-1.13 branch of x/sys.
Ref. https://github.com/kubernetes/kubernetes/pull/82349#discussion_r323738349

@gopherbot, please backport.
/cc @tklauser

Actually this is already on the Go 1.13 release branch:
https://go.googlesource.com/sys/+log/refs/heads/release-branch.go1.13
(3rd one down)

Oh, my bad. I don't know how I missed it looking thru the commits. Sorry Brad.


Snakegame jagpreet153

PR Description 📜
Added Snake Game of catching fruits
Fixes #3063

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have specified the respective issue number for which I have requested the new game.

Thank you @Jagpreet153 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@Jagpreet153 resolve the conflicts !

Thank you  @Jagpreet153 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


include .idea settings in git

for easier setup after cloning

see what to share here:
https://intellij-support.jetbrains.com/hc/en-us/articles/206544839

Shouldn't be necessary. Importing as maven project would set up dependencies etc. rightly.


Scatter plots fail with shap_values.display_data = Pandas dataframe

First of all we would like to thank the team of developers for their efforts. It's a great library.
This new version of SHAP 0.37 that will generate an object to store shap_values works with list type data, NumPy and Pandas but there are times when these types are changed to generate errors. This is an example and I would like you to review the code to be compatible with these types.
Versión SHAP 0.37 (clone github 6 dic). The stable 0.37 has some other issues.
import xgboost
import shap

X,y = shap.datasets.adult()
model = xgboost.XGBClassifier().fit(X, y)

explainer = shap.Explainer(model)
shap_values = explainer(X[:1000])

X_display,y = shap.datasets.adult(display=True)
shap_values.display_data = X_display.values   # NumPy

shap.plots.scatter(shap_values[:, "Age"], color=shap_values[:,"Workclass"])

This code work well but if we use a Pandas dataframe variable in display_data
shap_values.display_data = X_display   # Pandas dataframe

shap.plots.scatter(shap_values[:, "Age"], color=shap_values[:,"Workclass"])

/usr/local/lib/python3.6/dist-packages/pandas/core/generic.py in __getattr__(self, name)
   5137             if self._info_axis._can_hold_identifiers_and_holds_name(name):
   5138                 return self[name]
-> 5139             return object.__getattribute__(self, name)
   5140
   5141     def __setattr__(self, name: str, value) -> None:

AttributeError: 'Series' object has no attribute 'reshape'

Problem still present with shap 0.42 and pandas 2.0.2.
I think the previous post is misleading: the error is caused by:
[c:\ProgramData\Miniconda3\envs\adex310\lib\site-packages\shap\plots\_scatter.py](file:///C:/ProgramData/Miniconda3/envs/adex310/lib/site-packages/shap/plots/_scatter.py) in ?(shap_values, color, hist, axis_color, cmap, dot_size, x_jitter, alpha, title, xmin, xmax, ymin, ymax, overlay, ax, ylabel, show)
    121     # this unpacks the explanation object for the code that was written earlier
    122     feature_names = [shap_values.feature_names]
    123     ind = 0
    124     shap_values_arr = shap_values.values.reshape(-1, 1)
--> 125     features = shap_values.data.reshape(-1, 1)
    126     if shap_values.display_data is None:
    127         display_features = features
    128     else:

[c:\ProgramData\Miniconda3\envs\adex310\lib\site-packages\pandas\core\generic.py](file:///C:/ProgramData/Miniconda3/envs/adex310/lib/site-packages/pandas/core/generic.py) in ?(self, name)
   5985             and name not in self._accessors
   5986             and self._info_axis._can_hold_identifiers_and_holds_name(name)
   5987         ):
   5988             return self[name]
-> 5989         return object.__getattribute__(self, name)

AttributeError: 'Series' object has no attribute 'reshape'

.display_data is irrelevant IMHO: the problem is the DataFrame shap_values.data missing .to_numpy() before  .reshape(-1,1).
I'll try to fix it

To add extra info: the problem raises with the KernelExplainer

Indeed the patch,
shap_values_mod = copy(shap_values)
shap_values_mod.data = shap_values_mod.data.to_numpy()
shap.plots.scatter(shap_values_mod)

works well!


zsh stuck after double tab before paths

Describe the bug
zsh stuck after double tab before paths
To Reproduce
Steps to reproduce the behavior:
type source and then double tab, then zsh stuck and cannot type anything.
Expected behavior

Screenshots

Desktop

OS + distribution: Ubuntu 18.04
Zsh version: 5.4.2
Plugin version: latest

Additional context

Hi, can you reproduce this problem by running zsh -df and sourcing zsh-autosuggestions manually? If not, can you try commenting out your ~/.zshrc and re-enabling one section at a time to figure out what might be causing this trouble?

I had the same problem,when i use "kubectl get"，this problem will be reproduce
running “zsh -df” not reproduce this problem
when I commenting out the command "source <(kubectl completion zsh)" in ~/.zshrc ，it's back to normal。


Provide for Scaffolding of Custom Service Bean

[x] create, if it doesn't exist, an api.xsp XPage
[x] create, if it doesn't exist, an xe:restService
[x] create and hook up an endpoint, for an xe:customServiceBean
[x] scaffold the (class that extends) CustomServiceBean w/ basic helpers and block out main methods
[x] fix app name/title (on imported and)
[x] ACL?
[x] write unit tests

Now that I've started using cheerio, this should become considerably easier. 🎉

Moving CLI args to a separate issue for separate feature and release. This will need to be addressed across all sub-generators.

While I'm at it, I should also probably figure out why my generator/app/index.js starting showing as missing three lines of coverage. This disagrees with both codecov and coveralls.

I have the test/app.js suite figured out; apparently on local the default options were being thought of as missed by istanbul, even though it was executing. See #53.

boob 🎆


Fix the offset setting in GoogleStorage#get

Fixes #7076.
Description
Upgraded the GCP HTTP client libraries to bring in the fix for https://github.com/googleapis/google-http-java-client/pull/447
Also added tests using MockHttpTransport to verify that skipping is happening as expected. There was another bug in the
org.apache.druid.storage.google.GoogleStorage#get(java.lang.String, java.lang.String, long).  offset was being set on the media downloader which is not actually used when executeMediaAsInputStream is invoked. So setting the offset was a no-op.

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

will fix the license checks

Thanks! This has been annoying me for 1.5 years but I never quite managed to fix it :)


Clearer tutorial instructions on readme

For me, the instructions were unclear on how to run the example

if you pip install -e . the original is right


[ Documentation request ] Fix alignment of keploy logo in nav bar

Brief description
On visiting Documentation website, you can see that at the navigation bar, keploy logo is not aligned with rest of the nav bar items. So, fix the alignment such that all the items appears in a line.

Your recommended content
You can refer to this file.

@SanskritiHarmukh hello sir i can help with it