Obstacle_Map calibration mynteye2.yml question

Prerequisites
Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.

[v ] I am running the latest version of GAAS
[ v] I checked the Gitbook documentation and found no answer
[ v] I checked to make sure that this issue has not already been filed
[ ] I Googled/Binged the error messages generated by Ubuntu, Gazebo, and PX4
[ ] I have posted questions regarding Ubuntu, Gazebo, and PX4 in corresponding forums.

Issue Template
Context
Please provide any relevant information about your setup.

Tutorial No.:[Tutorial 4]
Mavros: [ 0.18.4]
ROS: [Kinetic]
Gazebo: [7]
Ubuntu: [16.04]
System Platform: [jetson tx2]

Hello
I'm using mynteyes-s camera.
This is the screen that me put into action
1.sh run.sh
2./bin/match_pointcloud_and_slam_pose2

you can see very small  dense point cloud appeared,
Is this the problem caused by the obstacle_map/calibration/mynteye2.yml file not matching my camera?
Then, tell me how to convert my Yaml file to mynteye2.yml

This is my mynteye.yaml file

This is obstacle_map/calibration/mynteye2.yml
How to change?

in config.yaml, are you using rectified image or raw image? change left and right camera topics that ends with "image_rect"  if you are not using the right topics;
that tiny point cloud could be the result of a poor parameter selection for:
UseRadiusOutlierRemoval
which you can find in the bottom of config.yaml.
Try tuning Radius and MinNeighbor, they could give you a better result.

after the first two step, you should start getting meaningful point clouds, for better performance, follow step 3.

as far as I know, mynteye cameras provide a method to get instrinstic parameters including K, D, update correspondingly in camera config file.

Let us know if you can get a meaningful point cloud.

I'm testing GAAS software on real-drone.
I connected the pixhawk and the companion computer.
It activates the ygz-slam and checks the rviz to the vision position.
But as you can see in the picture, it creates a map that is different from the image.
What's the problem? I want to make a map exactly like the video GAAS uploaded on June 2.
Please help me

The problem was the slam.
Close.


feat: Added Sentinel Search

Added Sentinel Search Algorithm.

Hmm Agreed. Done with the required changes.


Two Intro Questions

I did not see a Discussion section, so asking here in Issues: (1) Has anyone published a Google Colab notebook? and (2) how was the animation created of the vase?
For (1), I can whip one up assuming that the CUDA matches what Colab gives. Otherwise, this needs tweaking
For (2), it looks like you are varying the seed and/or the text prompt and then perhaps creating keyframes which are then interpolated? It is a cool effect. The other would be to explore 3D printing of things like "the cactus shoe"

Hi @metaphorz,

We hope to publish one soon, I'll let you know when it happens.
We morphed different styles created by applying various text prompts on the same content mesh. See Fig. 9 in the paper.


[Documentation] Improve README

Many things are still missing from README.

The README and documentation have already been greatly improved, so I will close this issue for now.


Add Drupal configuration settings

On Drupal 8+ projects, add a settings.ddev.redis.php and include it from settings.php.

@rfay, the only place I have used that is in the tests. There, we cd into that directory at the beginning of the test. Would you still say we should use environment variables?

Fixes #2

Agree it's fine in tests.


Stream logs to other destinations than stdout

As part of a system I'm working on we're dumping (and restoring) some fairly large databases (+50GB). Obviously that takes some time and we would like to keep the users aware of the progress.
The problem I'm encountering is that the output destination is hard coded to stdout as seen here.
Please let me know if there's a better way of solving the above issue. But I think this would be best solved by adding a io.Writer to the ExecOptions struct. It should obviously default to os.Stdout but it could be used to stream the output to multiple destinations ranging from a simple file to a database while the dump is taking place.
Would you guys be interested in a PR implementing the above feature?

@tonsV2 Yes I am interested for your PR. No problem to add this feature to the library 👍

@clementlecorre Great! I'll send you a PR in a not to distant future

released here : https://github.com/habx/pg-commands/releases/tag/v0.6.0


pkg request: greenwithenvy: nvidia overclock/undervolt utility

Project description

https://gitlab.com/leinardi/gwe
"System utility designed to provide information, control the fans and overclock your NVIDIA card"

Metadata

homepage URL: https://gitlab.com/leinardi/gwe
source URL: https://gitlab.com/leinardi/gwe
license: gpl3
platforms: linux

Oops, of course if I search author name I find it:
pkgs/tools/misc/gwe/default.nix


How to get model predictions on all classes after applying Transfer Learning in TF2.0?

I have been following transfer learning with TFHub to implement transfer learning in my model for text classification. In the example, I do not understand how to get probabilities for all the classes (1000 classes before + 5 classes after implementing transfer learning). At the end of the tutorial, they only show predictions on the 5 classes on which the model was re-trained, but I want to have probabilities also on the 1000 classes. I can not find any documentation or explanation on how to achieve this. If anyone has any ideas or some useful links, please feel free to share. THANK YOU!
This was my main question.
About model, where I want to implement this idea of transfer learning.
I have a text dataset of 8000 classes, since the data size is too big so it can not be loaded into the RAM of my machine. So I want to train on small-small sizes of data i.e. 4000 classes and 4000 classes. I trained my model freshly on 4000 classes first, the learned model gives me good results. Now I want to retrain the saved model on the rest 4000 classes using Transfer Learning (I am also open to any other way), but after doing a lot of google searches seems like it is impossible to retrain a model on new classes, at least in text classification. If you have read this part and have some suggestions or ideas please share them, I will be very thankful.

@rishabhsahrawat ,
You need to Save the Model without the Final Layer of the Text Classification using tf.saved_model.save
Then load the previously Saved [Head Less Model]  (https://www.tensorflow.org/tutorials/images/hub_with_keras#download_the_headless_model) using this Method, ingest the next set of data corresponding to 4000 labels.
After that, you need to attach the Final Layer of Text Classification.

@rmothukuru thank you very much for your answer and explanation. After following your suggested steps, attaching the final layer like Final Layer of Text Classification I will still only get predictions among the next 4000 classes not 8000 classes because the last layer had 4000 output neurons. My question is still the same, I want to get the predictions from all the 8000 classes after using transfer learning.
Please let me know if I am missing something in your point.
-Rishabh Sahrawat

Hi Rishabh, interesting question.
First of all, your dataset doesn't have to fit in RAM. Would your model be too big to train using 8000 classes? You can use tf.data.Dataset which will iterate over data from disk.
(https://www.tensorflow.org/beta/tutorials/load_data/text#split_the_dataset_into_text_and_train_batches, https://www.tensorflow.org/beta/guide/keras/training_and_evaluation#training_evaluation_from_tfdata_datasets)
If model size is still a problem, you could in theory do this kind of training, but I am really not sure how stable it would be, i.e. training on 4000-head could significantly worsen performance on the other 4000-head.
You could technically do this by always keeping reference to the following:
feature_extractor_layer,
head1 = layers.Dense(4000, activation='softmax'),
head2 = layers.Dense(4000, activation='softmax').
Between trainings you would recompile:
model = tf.keras.Sequential([
  feature_extractor_layer,
  head1
])
model.compile(...)

Of course this won't fit into your memory, so you can use tf.saved_model.save(feature_extractor_layer) and tf.saved_model.save(head1) as described in @rmothukuru's answer.

Hallo @vbardiovskyg , thank you so much for your helpful reply. Since I am new to TF, until now I had been using train/test split function from Scikit-Learn, which made my machine run out of memory if I try to use all 8000 classes. So, after looking for solutions I found I can convert my data file into TFRecords and then use the whole dataset because it speeds up the data processing. Although, to be honest I have no idea if your suggested links (TF train/test split) and the TFRecord have the same data processing speeds.
Your second suggestion,

You could technically do this by always keeping reference to the following:
feature_extractor_layer,
head1 = layers.Dense(4000, activation='softmax'),
head2 = layers.Dense(4000, activation='softmax').

Do you mean I need to add these two head1, head2 output layers in the second model(retraining model)?. Will they be connected in parallel to each other?
If not, then I think the final layer of the model will still have 4000 classes which means it will give probabilities between those last 4000 classes.
-Rishabh Sahrawat

The mentioned scikit-learn function works over numpy objects in memory, so I highly recommend learning more on the tf.data.Dataset side and reading from disk. Then your problem might disappear completely.
You are right that the final model would have 4000 classes. This was just a theoretical example of how that could work technically. I would recommend getting a proper set-up instead (as mentioned above), so you can use full model with 8000 classes.

Thank you, I think this makes more sense. I will try this now then.

@rishabhsahrawat ,
Can you please confirm if you have tried @vbardiovskyg's solution and if the issue is resolved, so that we can close this issue.

@rmothukuru , not yet but I am working on it. It should be done soon. I will update here.

@rmothukuru Hi, as suggested by @vbardiovskyg to load data using tf.data.Dataset, I implmented this idea and it worked well without filling up the memory. Now I have two questions regarding it-

Before every epoch it fills up the buffer, which is a bit time taking event. Is it a normal behaviour or can I somehow disable it after the first time it loads? I also opened an issue.
How is the way different from tf.Record? Is it more better in terms of speed or something?

Thank you!
Rishabh Sahrawat

Closing this issue as it has been answered. Thanks!

Although I still have unanswered questions from my last comment but OK.

Hi Rishabh,
sorry about this. The questions you posted are not about TensorFlow Hub, but about tf.data, so we are not in a good position to answer them.
The one you posted on TF seems reasonable.

Hallo,
thank you for your clarification. I will try to ask them there.


[Windows] Support K8S v1.16

We should support K8S v1.16 on Windows cluster

lgtm

Linked kontainer driver metadata PR got merged and vendored in Rancher, moving it to test

verified with latest master-head - commit id: 2bd183e61.
Steps:

Bring up a windows cluster. AWS - custom cluster
Cluster spec:

1 etcd/control plane - Linux 18.04 LTS
1 worker node - Linux 18.04 LTS
2 Worker nodes - WIndows - Windows_Server-1903-English-Core-Base-2019.08.16

The cluster comes up successfully.
Deploy some workloads.
Deploy a node port workload.
Deploy an ingress
They work as expected.


Border in box-model

Border affects element sizes. Should it be in box-model? At least having option to choice will be nice, like in abandoned rational-order package.

I thought about that before but these changes would separate outline which is not included in CSS Box Model. In my opinion, it makes sense to group them together because they're similar. Should we include outline in box-model group as well, what do you think?
Moving border and outline to box-model group also makes it their current appearance group really small. Most of the time they would consist only opacity and background properties.
Although it's named as box-model, I'm actually thinking them as general sizing properties. Border also effects the element size like you mentioned but most of the time we do not use it that way. So, maybe naming is at fault? We can simply rename "box-model" group to "sizing"?
It would really appreciated if you have any thoughts about this @wimhendrikx.

I don't think outline should be moved too since it doesn't affect elements sizes. So border and outline aren't so similar from this point of view.

All else seems to make sense to me, just border stood out as in the wrong place, so I'd vote for this one too.

Do you guys think border should be between margin and padding or after or before margin/padding pair?
Between margin and padding:
.box {
  isolation: isolate;

  width: 200px;
  height: 100px;
  margin: 20px;
  border: 2px solid #3498db;
  padding: 15px;

  border-radius: 10px;
  box-shadow: 3px 3px 5px rgb(0 0 0 / 10%);
}

After margin and padding:
.box {
  isolation: isolate;

  width: 200px;
  height: 100px;
  margin: 20px;
  padding: 15px;
  border: 2px solid #3498db;

  border-radius: 10px;
  box-shadow: 3px 3px 5px rgb(0 0 0 / 10%);
}

I would vote for "after margin"


test(connlib): send DNS queries to non-resources

Currently, tunnel_test only sends DNS queries to a client's configured DNS resources. However, connlib receives all DNS requests made on a system and forwards them to the originally set upstream resolvers in case they are for non-resources.
To capture the code paths for forwarding these DNS queries, we introduce a global_dns_records strategies that pre-fills the ReferenceState with DNS records that are not DNS resources. Thus, when sampling for a domain to query, we might pick one that is not a DNS resource.
The expectation here is that this query still resolves (we assert that we don't have any unanswered DNS queries). In addition, we introduce a Transition to send an ICMP packet to such a resolved address. In a real system, these wouldn't get routed to connlib but if they were, we still want to assert that they don't get routed.
There is a special-case where the chosen DNS server is actually a CIDR resource. In that case, the DNS packet gets lost and we use it to trigger initiate a connection to the corresponding gateway.

@conectado The whole, DNS server is a resource thing is making this more involved than I originally thought! It basically means, I need to stub a small DNS server now on the gateway, otherwise the test fails because it is sitting there with an unanswered DNS query!
On the upside, it is exciting that we are actually hitting all these codepaths!

@conectado I extracted https://github.com/firezone/firezone/pull/5192 but otherwise this is ready for review.


All Question Goes Here.

Leave your Questions in the Comments.

When we'll start our first phase of development??a


KOGITO-3344 - Failover scenario for scaling Kafka to 0 using BDD tests

Many thanks for submitting your Pull Request :heart: :cake:!
KOGITO-3344

There is currently no other way to scale Kafka to 0 other than killing a pod. Kafka CR doesn't accept values less than 1 for number of instances.
Even though it's just a deletion of a specific pod, I added a separate method for scaling Kafka to 0 directly to the framework, so we have this functionality right there in case the framework is extracted to a separate repo in the future.

Please make sure your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

/jenkins test

/jenkins test

CI bot doesn't start the BDD tests, possibly due to repository rename

/jenkins test

@MarianMacik You need to rename your fork to kogito-operator for BDD tests to work :)

/jenkins test


Outdated documentation on floating figures in Typst vs. LaTeX?

The documentation still says
https://github.com/typst/typst/blob/87c0a5171af2e374cd6b0a0bd6d8d9531aaf796c/docs/guides/guide-for-latex-users.md?plain=1#L573-L578
but 78f96f844bc54c5385c1efda0b5faad3c49fa79b has been merged and released in 0.7.0. Doesn't that change the situation outlined in the aforementioned bullet point?

Yeah, it's a bit outdated. We'll update the LaTeX guide soon.

For future reference: This was fixed in f90701e132679f8372e8a32171dd013646c7f276.


Rethink UI placement for nav and user actions

Introduced the new discussion tag to denote that this isn't a request so much as something I wanted to talk about.
I think there are some things in the current UI that could be moved around to make things more user-friendly. For example, user stuff should probably be taken out of the nav menu so that it can be presented in more places. As it stands, to switch accounts or check your inbox, you have to go all the way back to the root SingleSubredditVC and open the nav menu from there. Users should be able to access that from any instance of SingleSubredditVC.
If we do that and remove the settings button from the nav menu, then the nav menu is no longer really a multi-purpose menu so much as a quick switcher for subs. And in that case I think we could make that trigger by clicking either a magnifying glass button or the subreddit name itself.
Here's a rough mockup of what I'm talking about.

We could then move the subreddit "more" button up to the top bar, making the whole thing subreddit-centric.
The only issue is where to keep the settings and account buttons. We could keep them in the bottom bar, or put them in a radial menu like so (another bad mockup incoming):

I think all this would go a huge amount toward improving the UX. Take or leave any amount of this -- just wanted to get a discussion going.

I think what would be useful is allowing the user to set any of the actions they want in the app. So they can set the fab button, either side button, and top buttons to their liking. It would also allow iPads to show more buttons than just one on the iPhone


"snap top left third" is snapping to the entire top left quarter

I have no problem using all of the snap positions, except for the "top left third" position. All of the other third positions work fine including the other corners, but whenever I try to do the top left third it ends up taking the entire top left quarter of my screen. I have no trouble getting Talon to transcribe top left third accurately and the recordings show that it's recording top left third, but every single time it ends up moving my window to be the entire top left quarter.
Nothing is showing up in the logs when I use this command either, it works except for the size difference. I'll dig into him more to more and see if I can finder reason this might be happening and if it requires a PR.

That's strange but I can think of one reason it might happen. There are two ways for talon to parse that phrase:

"snap top left third", the command we want
"snap top left" (snaps to top-left half) "third" (do that three times)

Try running sim("snap top left third") in the reply. I think there's also some debug command that does the same thing, it might be talon debug last - so, say snap top left third and then wait for it to recognize, then say talon debug last and look at the log.

Oh, actually, I can think of another reason. The "snap top left third" command was only added recently (it had been left out by accident), so make sure you have an up to date knausj. Otherwise "snap top left" "third" will be the ONLY way talon could interpret that command.

Oh wow I didn't even think about it being added recently because I only installed this earlier this week and the top right third worked, with the newest version it works great. Thanks!


Explore: why is opensha UCERF3 function called

Explore why UCERF3InversionConfiguration.getSmoothStartingSolution is called when running nzshm-opensha and if it can be avoided.
This function needed to be altered to work with the MFDs that the NZ NSHM uses, but cannot be altered in the upstream repo. Kevin thought that maybe we don't need to be calling the function at all. It is unknown (forgotten) if it is limited to a type of inversion (Puyseger, Hik, Crustal) or happens for all of them. It is unknown (forgotten) which inversion in particular required the modification to the function.
Can provide specific inversion parameters, if necessary.

@chrisdicaprio it's currently only called for subduction inversion water level.
See also the same investigation at https://github.com/GNS-Science/nzshm-opensha/issues/241

Action to be taken: https://github.com/GNS-Science/nzshm-opensha/issues/294


Broken links, formatting, grammar.

Lots of changes to links (esp. broken links), formatting to make reading easier from a tech point of view, and little grammar fixes.

Hello @Nerdwood,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @Nerdwood,Thanks again for the PR!                                         You have earned 6 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid


ElasticsearchFormatter: Force record values to be strings

Monolog version 2
ElasticsearchFormatter should force all records to be strings, since ElasticSearch will fail on field types mixing in an index.
It would also make sense to not allow deep arrays, basically flatten arrays and make them scalar.

I don't think we can really do that without breaking BC.. It just means that currently you have to manage your context data carefully yourself.. The data provided by Monolog itself is consistently typed AFAIK. IMO this fact makes ElasticSearch generally speaking a poor choice for log storage but I can't choose what people use..


Automated cherry pick of #9594: chore(util): update centos build version to 1.1-3

Cherry pick of #9594 on release/3.6.
#9594: chore(util): update centos build version to 1.1-3

Codecov Report

Merging #9595 (d980604) into release/3.6 (b9d30c0) will decrease coverage by 1.89%.
The diff coverage is 3.40%.

@@               Coverage Diff               @@
##           release/3.6   #9595       +/-   ##
===============================================
- Coverage         5.86%   3.96%    -1.90%
===============================================
  Files              828     170      -658
  Lines           129133   25501   -103632
===============================================
- Hits              7577    1012     -6565
+ Misses          120876   24375    -96501
+ Partials           680     114      -566

Flag
Coverage Δ

aFlag
3.96% <3.40%> (-1.90%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/cloudcommon/db/apply_manager_method.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/archs.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/db_dispatcher.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/models.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/scoperesource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/sharablebase.go
6.01% <0.00%> (-0.06%)
:arrow_down:

pkg/cloudcommon/db/virtualjointbase.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/virtualresource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/policy/policy.go
1.20% <0.00%> (-0.01%)
:arrow_down:

pkg/cloudprovider/instance.go
0.00% <ø> (ø)

... and 610 more

Continue to review full report at Codecov.

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


Unable to use github extension authoring with identity.

Describe the bug
When logged in, github extension doesn't work
To Reproduce
Steps to reproduce the behavior:

Sign in to arcade.makecode.com
New project and click on the github icon next to save icon
Sign in to github
Create a new repo
Github icon doesn't change and clicking on it again asks to create the repo again.

Expected behavior
Github icon should change so that github management UI can be launched.
Screenshots

Not specific to identity. I can repro when signed out.

Cannot repro on beta (sign in or otherwise).

Ok feel free to resolve as fixed in /beta. I have extension talk tomorrow. Bad timing I guess :-)

Sigh.
Appears to be fixed in /beta.


Building for one provider results in --resources flag is unkown error

I wanted to build terraformer for only one provider but I encoutered the same error as in #348, that the --resources flag is unkown. However, if I build for all providers it works.
See my steps below to reproduce:
$ git clone https://github.com/GoogleCloudPlatform/terraformer.git \
        && cd terraformer/ \
        && go mod download

$ go run build/main.go {aws}
$ go build -v

# Init terraform with aws as requried provider

$ ./terraformer-{aws} import aws --resources=vpc,subnet --filter=vpc=myvpcid --regions=eu-west-1
2022/01/03 10:46:35 unknown flag: --resources

$ ./terraformer import aws --resources=vpc,subnet --filter=vpc=myvpcid --regions=eu-west-1
2022/01/03 10:48:39 aws importing region eu-west-1

@thacoon -
Try building and use terraformer binary for AWS like this.
1.go run build/main.go aws
2. ./terraformer-aws import aws --resources=vpc,subnet --filter=vpc=myvpcid --regions=eu-west-1
3.

@sunil29feb thank you, your solution works.


404 at /compose-file/

I found a broken link : https://docs.docker.com/compose-file/

@rituparna1997, thanks for sharing the issue. Closing as stale unless more details are provided.


Separate Tap Parser

Use a separate parser for parsing tap output. This should clean up the code a lot more.

@MikkoH Yes I thought exactly the same, it would reduce A LOT the codebase, go for https://github.com/substack/tap-parser


NoSuchMethodError of TomcatAccessEvent

Hi man,
I use verison 1.6 in my Spring Boot project (Spring Boot version 1.3.3.RELEASE) and got an error:

java.lang.NoSuchMethodError: net.rakugakibox.springbootext.logback.access.tomcat.TomcatAccessEvent.setThreadName(Ljava/lang/String;)V
    at net.rakugakibox.springbootext.logback.access.tomcat.LogbackAccessValve.log(LogbackAccessValve.java:125)
    at org.apache.catalina.core.AccessLogAdapter.log(AccessLogAdapter.java:51)
    at org.apache.catalina.core.ContainerBase.logAccess(ContainerBase.java:1042)
    at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:571)
    at org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1095)
    at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:672)
    at org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1500)
    at org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.run(NioEndpoint.java:1456)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
    at java.lang.Thread.run(Thread.java:745)

Please help me verify the issue.
Thanks.

In your project, what version of Logback-access?
In the case of Spring Boot 1.3.3.RELEASE, it will be used Logback-access 1.1.5 by default.
But, "spring-boot-ext-logback-access" requires Logback-access 1.1.6 or higher.
Please use Spring Boot 1.3.4.RELEASE (Logback-access 1.1.7 by default) or higher.
Or, please use Logback-access 1.1.6 or higher.
For example on Maven-based projects (pom.xml):
<parent>
    <groupId>org.springframework.boot</groupId>
    <artifactId>spring-boot-starter-parent</artifactId>
    <version>1.3.4.RELEASE</version>
</parent>

or
<properties>
    <logback.version>1.1.6</logback.version>
</properties>

Thank for your reply.
The issue was caused by Spring Boot version 1.3.3.RELEASE with default logback-access 1.1.5 as you mention.
The document about "Spring Boot 1.3.0+" together with "Logback-access 1.1.6+" sometimes makes confusion.
I decide to upgrade Spring Boot version and It works well.
Thank for your support.

You're welcome.
I added a description about the version of Logback-access :)
https://github.com/akihyro/spring-boot-ext-logback-access/commit/79180d8ad0ba598f87fe31a99ee664a950f73fb9

Thank for your support.
I'm using Gradle.
So where can I specific logback.version globally?

Sorry for the late reply.
I have never used Gradle, so I can not help it properly.
Can the following articles help you?

79.1 Customize dependency versions - Spring Boot Reference Guide
To override dependency versions in Gradle, you can specify a version as shown below:
ext['slf4j.version'] = '1.7.5'

For additional information, please refer to the Gradle Dependency Management Plugin documentation.

@akihyro Actually I've already configured it out. Thank you.


Create mymail.txt for Texas State Technical College

Added Texas State Technical College

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

sorry, its fixed
On Fri, Aug 23, 2024 at 8:22 PM Philip Torchinsky @.***>
wrote:

@adoyal https://github.com/adoyal I have to close this request without
merging, because you requested to add a non-existent domain to this
repository: Host mymail.edu not found: 3(NXDOMAIN) Please ensure that you
are familiar with this repository rules located at the bottom of this page
https://github.com/JetBrains/swot and submit a new request with the
correct information. Thank you for your understanding!
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/22843#issuecomment-2307591923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKWCI57R3G2FQCYSZ5J62XLZS54WTAVCNFSM6AAAAABM6AYVJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBXGU4TCOJSGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Properly configure conformance tests to run source status conformance test against KafkaSource

Problem
Eventing added the ability to test sources status conformance in knative/eventing#3605 . We need to do the following to run this test against Kafka Source

Add a new test case that will use the helper from eventing.
Add a KafkaSource initialization function to and properly configure the conformance tests ComponentsTestRunner

Exit Criteria
A measurable (binary) test that would indicate that the problem has been resolved.
Time Estimate (optional):
0.5 d
Extra Context
Eventing issue knative/eventing#3117

/area sources
/area test-and-release
/assign

/area kafka


[Storage] Fix #27202 az storage entity insert: Fix case when using sas token with only add permission

Related command

Description

Fix case when using sas token with only add permission
Fix #27202
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage


Bug

I encountered this bug: https://sufix.cz/res/files/quests-bug.png
It appears when someone is online. Could you fix this?

Please try the 1.9.4 test build once it's out:
http://ci.ac3-servers.eu/job/Quests/100/

Ok, fixed. Thank you


storage.ManagementPoliciesClient.CreateOrUpdate Failed with RequestIsNotValidJson

Bug Report

import path of package in question, e.g. .../services/storage/mgmt/2019-04-01/storage
SDK version 31.0.0

Specify the exact commit if possible; one way to get this is the REVISION
column output by dep status "github.com/Azure/azure-sdk-for-go.

output of go version: go version go1.12.6 linux/amd64

Attempting to create a ManagementPolicyRule results in an error:
Error creating Azure Storage Management Policy : storage.ManagementPoliciesClient#CreateOrUpdate: Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="RequestIsNotValidJson" Message="The request cannot be interpreted as valid Json: {}."

Repro code here

It looks like this is an issue with the swagger for storage: https://github.com/Azure/azure-rest-api-specs/issues/6549

Found another issue  that also needs addressing for Management Policies

Bump - the swagger has been updated (see here) - what needs to happen to create a new release of the SDK from the updated swagger?
Thanks!

Hi @stuartleeks , this issue should be resolved in version 32.0.0

Kindly reactivate if you find the issue is not resolved in v32.0.0

Apologies - I was juggling this in tandem with some other work. I appreciate your help getting this fixed and release - the Terraform provider work that this unblocked has now been released :-) Thanks!


Actually change state in tests

The existing tests of Update and Remove behavior do not actually store the updated/removed states, so as a result, it's as of each Update/Remove were being applied on the clean group as created.  Instead, they need to be applied to the group on top of each other.
Fixing this bug revealed several ways in which the State logic was incorrect.  Most notably, the transcript hash computations were wrong, using digest.Sum() incorrectly.

LGTM .. ah! that digest error !!


vim: 8.2.4975 -> 9.0.0001

Description of changes
Upgrade to Vim 9.  Despite the major version bump, it should be backward compatible.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

LGTM
$ ./result/bin/vim --version
VIM - Vi IMproved 9.0 (2022 Jun 28, compiled Jan 01 1980 00:00:00)
macOS version - arm64
Included patches: 1
Compiled by root
Huge version without GUI.


CAP: minor correction

@flavour Please merge...

@flavour Please review and merge

I'm sorry - but this does not follow the relevant standards for REST nor CRUD, which makes it massively inconsistent and risky.

"Approve" is a method to change the db status, and should therefore be POST not GET.
Post-action redirection in S3Methods uses self.next, not redirect. You should never redirect from an S3Method directly
Error handling should go through r.error to be safe for non-interactive clients
The target resource can not be concluded from r.tablename (which is the master table) - all S3Methods must use self.resource and self.record_id to work so that they work on both components and link tables
You should never re-instantiate the target resource for manipulation, that is incorrect. Instead, if you need to operate on unapproved records, you must include them in the first place (REST)

I'm afraid this implementation is completely unacceptable (because it is incorrect).

Note that approval does actually not work from components (obviously can't approve a component without approving the master).
That is why "review" is only available for master requests - and thus it may be sufficient to throw an error for component requests, and otherwise stick with self.resource.
Re-instantiation is actually not necessary, because if you see the record in a list-view (and hence have that "Approve" action button), then you already have "unapproved" in CRUD. If you don't have that, then the user has no review permission anyway, and hence should not be able to use approve in the first place.
GET is not correct in any case, though. As the very minimum it must be caught behind a confirmation dialog - but actually it must be RESTful a POST request, a simple URL GET should be able to approve a record.
But what is dramatically wrong with that method is that you do not check whether the user is actually permitted to approve the record. You seem to take that for granted.

With this implementation, anyone who wants to publish an alert just needs to open it and change the URL method into /approve. They do not even need update permission, nor any complicated request construction - read-access and a browser are good enough.

The other weird question is why this is using the "approve" workflow at all.
In an earlier, similar case, we had implemented a "publish" workflow, which is why there is a PUBLISH permission. This simply used a "public" flag in the target record, and imposed a filter for anonymous users that hid all records with public=False.
This would be a lot easier and safer to implement, and all those weird modifications to the approval concept wouldn't be necessary, and interference with security could be avoided.

This publish-workflow was for CMS posts (newsfeed). Very simple resource-specific method, that was exposed for users with auth.has_permission("publish") == True (this permission does exist and can be managed).
If all you need is to hide alerts from anonymous users, then I think all this is effort and complications are way beyond reasonable.

...not everything needs to be implemented at the framework level ;)

Really - this is very confusing. Why do you "publish" by approving?
If approving an alert and publishing it is one and the same workflow step - then why make it complicated and use record approval?
I would have though that these are two independent steps - approval means "authorize it", and publish means "send it out", in which case indeed you need to have both an approval-concept and a publishing workflow.
But now it looks as if that is one and the same thing - so "approving" a record means to make it public at the same time. And if that is the case, then a simple publish-workflow would have been just as good - and all those inconsistent and insecure framework modifications wouldn't have been necessary.
If though "approving" and "publishing" are two independent workflows, as originally argued, then it seems wrong to make the "Publish" button call the "approve" method.

In my memory it was "you can only publish an alert when it has been approved" - without though making publishing mandatory at the moment the alert has been authorized. I would understand if that requirement has changed - but then it seems odd to employ the complexity of record approval for that purpose, and make major framework modifications just for that one case.
===
Generally I do see a problem with all this:
The normal meaning of record approval (in the sense of the concept that was implemented) is "to allow the record to enter the database" - which is why it has a reject-component. This meaning has been massively undermined by the recent changes, and records are now allowed to enter regular database workflows without first being approved.
But in the alert case it seems that it actually has the meaning "to make a record available for a wider audience" - while the record gets accepted into the database upfront and without further procedure. That is though "publishing" - and not "approval".
Both concepts can co-exist, which is why we do have separate permissions for them (APPROVE/REJECT vs. PUBLISH), and typically would implement separate functionality too.
But if you don't need both, then you should actually stick to the right concept instead of bending the opposite one to fit your needs while breaking it's original meaning - and without need, I must say.

I think this has all been terribly mixed up due to context-specific terminology being mapped to the wrong framework concept - and unfortunately it is causing the loss of that very valuable, and much needed security concept to prevent unwanted records from entering regular workflows.

To make that clearer:
The actual idea behind record approval is (as needed in NYCP and other templates):
General Public => Data Entry => Approval => Application Workflows
You need seems to be (SAMBRO):
Application Workflows => Approval => Publish => General Public
So, it's turning a framework concept completely upside down - so much that it actually gets lost.
And I do actually disagree that both concepts can be lumped together - because in fact they can co-exist and be separate workflows:
General Public => Data Entry => Approval => Application Workflows => Approval => Publish => General Public
...and should therefore be kept separate.

...and since public data entry (e.g. crowdsourcing) is still an aspect in Sahana, we need to retain the workflow entry barrier option as we had it, even if that is not required for SAMBRO.
In all previous cases where we needed controlled publishing (workflow exit barrier) we did not use the approval-workflow, but a "public" flag of some kind. Michael Howden has correctly recognized this as controlled=>uncontrolled transition (=output chain), and that it is different from the record approval concept (which is for the input chain).
However, SAMBRO project terminology seems to suggest that the application environment is uncontrolled while the general public is the controlled area - but even if that is the case from a business logic perspective, it does still not make "publishing alerts" an input chain, because as Fran explained: application workflows come before that.
Unlike what I first expected, SAMBRO doesn't seem to impose any input chain controls other than trusting the users - every workflow input (e.g. a new alert) is automatically trusted and allowed to enter application workflows if the user has permission to create it.
I did expect that alerts published by other sources and automatically imported into SAMBRO would require approval before being processed further in SAMBRO (=input control), and could be rejected if they were not trustworthy.
But instead, SAMBRO aims to impose output chain controls by requiring separate authorization for workflow items to become available beyond the application workflows (e.g. alerts to the general public, not from other sources).
So, I misinterpreted the need - and hence accepted certain modifications to the approval concept even though they appeared incorrect. But now I see that it is actually a completely different kind of thing that is needed :/

@flavour Please merge


IA-2360: Validate Feature Flags when project is updated

Some Feature Flags won't work without the 'REQUIRE_AUTHENTICATION' one added alongside.
Therefore, we mark those as requiring authentication and we validate on project creation/update.
Related JIRA tickets: IA-2360
Self-proofreading checklist

[X] Did I use eslint and black formatters
[X] Is my code clear enough and well documented
[X] My migrations file are included
[X] Are there enough tests

Changes
A new field was added on the model feature_flag called requires_authentication.
This allows us to check when the project is created/updated that the REQUIRE_AUTHENTICATION feature flag is also present or it raises a 400.
How to test

Go into the project edition screen.
Try to create or update a project
In the feature flag screen, add "Planning" only
Save and you should see an error
Add "REQUIRE_AUTHENTICATION" (might have to be added in the admin first, I don't think there is a migration for it)
Save and it should work

@beygorghor No, we don't, the error message returned by the backend is enough and well displayed in the UI

But is it translated?🤓

@mathvdh Yep, that's correct.
If you are trying to enable a feature flag with the boolean requires_authentication to true, the other feature flag REQUIRES_AUTHENTICATION must be enabled simultaneously. Otherwise, it will raise an exception.

@bmonjoie it seems that when I merge this branch the python tests on main fail ... it's because we have migration conflicts with the previous branch merged .. Can you please merge main and fix the migration conflict ? Or let me know if you want me to do it .. ?

@bmonjoie I fixed it .. No worries


Preview Works as intended, black screen when compiling for iOS/Universal Mac App

I managed to get outlines working in the editor, but as soon as I compile for iOS/Mac Universal App, I just get a black screen. Any idea what to tweak to make it work?

OK I found the issue. For anyone else having this problem, you need to set the 3 shaders from the package to always include.
Go to unity editor: Edit->Project Settings-> Graphics
Under Build-in Shader Settings, set to Always Include
Then Under Always Included Shaders increase the Size by 3.
Add the 3 shaders from the package:

Hidden/UnlitColor
Hidden/ViewSpaceNormals
Hidden/Outlines


Appengine & ClassPathScanner : throw AccessControlException since 4.1.0

What version of Flyway are you using?
4.2.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)

Flyway Java API
Maven dependencies
Appengine (latest SDK, 1.9.51) Standard Environment (JAVA 7) .

What database are you using (type & version)?
MySql 5.6.23
What operating system are you using?
Mac OSX El Capitan
What did you do?
Only append in the Development environment (as appengine libs are in a different classpath on Google Appengine server)
Basic configuration of flyway:

appengine Java helloworld project
resources/db/migration/V1_init.sql
fresh & clean database (only one table)
Add java Flyway migration in a simple servlet:

  Flyway flyway = new Flyway();
  flyway.setDataSource(url,user,password);
  flyway.migrate();

Result : Throw an exception :

org.flywaydb.core.api.FlywayException: Unable to scan for SQL migrations in location: classpath:db/migration

I've hunted the bug down to the location where it occurs :
In fact, when the ClassPathScanner try to findResourceNames, it hits the appengine sdk jar (in my case 'appengine-agentruntime.jar') that triggers a :
java.security.AccessControlException: access denied ("java.io.FilePermission" "/Users/CtrlX/google-cloud-sdk/platform/google_appengine/google/appengine/tools/java/lib/impl/agent/appengine-agentruntime.jar" "read")
Basically, after some research, appengine disallow that type of action (opening appengine jar at runtime) and so it raise that FilePermission exception.
This is due to this commit that change the logic of method findResourceNames by adding:
    ...
    try {
       jarFile = new JarFile(url.toURI().getSchemeSpecificPart());
    } catch (URISyntaxException ex) {
       // Fallback for URLs that are not valid URIs (should hardly ever happen).
        jarFile = new JarFile(url.getPath().substring("file:".length()));
    }
    ...

It doesn't catch the AccessControlException, and the exception bubble up to Scanner.scanForResources, eventually transformed into a FlywayException.
For those how have the same issue :
The bug appear in the release 4.1.0, so as a work around, you can use a previous version of flyway (like 4.0.3)
What did you expect to see?
Jar that trigger FilePermission exceptions should be ignored or just warned.
What did you see instead?
FlywayException encapsulating an AccessControlException

Thanks for the detailed investigation!


Functionality of allFonts argument in UI.SelectFont

Hey there,
Recently I used the SelectFont dialog within a Robofont script.
I imported the one from mojo.UI, which I assume is identical to what can be found at fontParts.UI. In the documentation I saw that there is also an allFonts argument. With the code below, I tried to figure out what it actually does, but I could not observe any difference between None, True, or False.
from mojo.UI import SelectFont

font = SelectFont(title='Check Fonts None', allFonts=None)
print(font)

font = SelectFont(title='Check Fonts True', allFonts=True)
print(font)

font = SelectFont(title='Check Fonts False', allFonts=False)
print(font)

Can anybody clarify what the intended use of SelectFont is?
Is it just a leftover for legacy reasons?
Thanks in advance!

see https://github.com/robofab-developers/fontParts/blob/master/Lib/fontParts/ui.py#L135
it either None or a list of font objects, this can be handy to show a different set of font...

Ah, great. Thanks for the help.
So SelectFont is definitely not the right choice when I want to have a UI with a list of fonts to choose from then. 😅


/view 404

failed to open <uuid>&e:
looks like if you pass the e qsarg with no = it ends up in the filename
need to confirm yet, just a guess saw it in the logs

can't repro


Bump scipy to 1.14

Is your feature request related to a problem? Please describe.
Greetings there,
Hope all are well. I would like to ask if you can bump the scipy version to 0.14 (latest version). I am trying to add support for your package to my package, however, given existing support for pytket and it needing scipy==0.14 I can't make a venv with your package included.
Describe the solution you'd like
If possible, please bump the version in your requirements.txt to latest.

Hi, sure thing. The version restriction was just due to problems with other dependencies. I will check if that is fixed now and then upgrade.

Done
Should also reach pypi, once the release runner is through.
Let us know if you have further issues :-)


JSON CRDT fast view decoder for binary codec

Closes https://github.com/streamich/json-joy/issues/156

:tada: This PR is included in version 7.12.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


upgrade mui to v5  && dark theme

[x] upgrade mui to v5
[x] dark theme

@uvd can you separate two pr one is reformat one is migrate to mui？


REM-24447 docs: improve readme wording

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@92952de). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   87.15%
=========================================
  Files             ?       29
  Lines             ?     1059
  Branches          ?        0
=========================================
  Hits              ?      923
  Misses            ?      136
  Partials          ?        0

Continue to review full report at Codecov.

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


Make “view site” admin link work on sapir

Codecov Report

Merging #633 (0ed109d) into master (331b266) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #633   +/-   ##
=======================================
  Coverage   89.32%   89.33%
=======================================
  Files          51       51
  Lines        2089     2090    +1
  Branches      323      323
=======================================
+ Hits         1866     1867    +1
  Misses        156      156
  Partials       67       67

Impacted Files
Coverage Δ

CreeDictionary/CreeDictionary/urls.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #633 (0ed109d) into master (331b266) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #633   +/-   ##
=======================================
  Coverage   89.32%   89.33%
=======================================
  Files          51       51
  Lines        2089     2090    +1
  Branches      323      323
=======================================
+ Hits         1866     1867    +1
  Misses        156      156
  Partials       67       67

Impacted Files
Coverage Δ

CreeDictionary/CreeDictionary/urls.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Confirmed working on site!

Confirmed working on site!


Applying angularjs ng-repeat to owl-carousel

I'm running into an issue where whenever the ng-repeat directive is applied to carousel the items are stacked vertically instead of being layout horizontally.
If I leave out ng-repeat and use static items then it works as it should.
Is there a directive I can write and apply to owl-carousel to maintain the layout?
Also what is about ng-repeat that is causing the carousel to break?
Is angular somehow stripping the owl-carousel classes applied to the carousel?

I too am interested in this. I actually have it working in one spot but cannot get a second directive call working on the same page.
I read somewhere that Owl2 was going to work with Angular. Do we need to do anything special?

This actually looks like an angular-filter issue.

Have you tried the angular-imagesLoaded ? It worked for my implementation of owl/angular.
$scope.imgLoadedEvents = {

	always: function(instance) {
	   // Do stuff always
	},
	done: function(instance) {
		// Do stuff if images load

		// Owl intialize
		$(document).ready(function(){
		  $(".owl-carousel").owlCarousel();
		});
	},
	fail: function(instance) {
	   // Do stuff if images fail to load
	}

};

Closed due to stale issue.


Create Index with Coordinates

NEST/Elasticsearch.Net version:5.5
Elasticsearch version:
Description of the problem including expected versus actual behavior:
Steps to reproduce:

I want to Create index with model like below:
public class company
{
public long id { get; set; }
public string name { get; set; }
public string address { get; set; }
public long areaCode { get; set; }
public string city { get; set; }
public string state { get; set; }
public string latestActivityTimestamp { get; set; }
public location location { get; set; }
}
public class location
{
[JsonProperty(PropertyName = "lat")]
public double latitude { get; set; }
[JsonProperty(PropertyName = "lon")]
public double longitude { get; set; }
}

I want to Create index it and mapping the location as GeoPoint ,I find some example like :
internal class Location
{
[JsonProperty(PropertyName = "lat")]
public double Latitude { get; set; }
[JsonProperty(PropertyName = "lon")]
public double Longitude { get; set; }

}
and then mapping it with:
.MapFromAttributes()
.Properties(p =>
p.GeoPoint(s =>
s.Name(n => n.Location).IndexGeoHash().IndexLatLon().GeoHashPrecision(12)
)
)
The Problem is I can not find MapFromAttributes(), IndexGeoHash() ,I don't know how to deal with my problem.

Hi @yanwei2508 ,
GitHub issues are reserved for bug reports and feature requests.
The best place to ask a general question is at the Elastic Discourse forums at https://discuss.elastic.co/c/elasticsearch. Feel free to link to your question there by editing this issue.
Thanks!


Update README.md

Just add one " so your code will be correctly rendered.

Hi, I caught the typo in a recent commit so it should now be fixed in develop.  Master will be updated with a new release momentarily.  Thanks for pointing out the issue.


PHP template

PHP template request
Expected Behaviour
Be able to deploy and run PHP functions
Current Behaviour
Missing a template for PHP applications
Possible Solution
Add PHP template

Hi there and thank you for your interest in OpenFaaS.
The community have provided third party support for PHP. Please see the faas-cli readme file for more details.
Thanks,
Alex

https://github.com/itscaro/openfaas-template-php/


Any plans to port this to Asp.net 5 CoreCLR?

A feature much appreciated!

Hi! I actually started doing that, but I've had no time for finishing it.


test: fix base image e2e test for v1.2.0-rc.1

Description
What this PR does / why we need it:

base image e2e script missing the cosign setup for ratify since cosign is already one built-in plugin within ratify for now.
licence verifier is still not in base image, thus its related test should be under plugin test bats

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Helm Chart Change (any edit/addition/update that is necessary for changes merged to the main branch)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] Does the affected code have corresponding tests?
[ ] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ] Do all new files have appropriate license header?

Post Merge Requirements

[ ] MAINTAINERS: manually trigger the "Publish Package" workflow after merging any PR that indicates Helm Chart Change

lgtm once comments resolved, thanks for fixing the test!

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

Project coverage is 68.15%. Comparing base (1bd347c) to head (e880f52).
Report is 47 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1501      +/-   ##
==========================================
+ Coverage   66.76%   68.15%   +1.38%
==========================================
  Files         116      119       +3
  Lines        6030     6139     +109
==========================================
+ Hits         4026     4184     +158
+ Misses       1620     1561      -59
- Partials      384      394      +10

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


OIDC integration supports additional claims and groups

This has a few OIDC changes that kind of go together:

Remove the existing call to the userinfo endpoint after authenticating. According to the specification, when the openid scope is included the response must include an ID token. In testing with Auth0, Google and Dex, this is true. This way we can reuse the same logic for ID token parsing everywhere.
Add the ability to request arbitrary additional scopes. This allows users to add the groups scope to the request, if their provider supports it.
Sync the groups that are returned, if applicable. I tested this by wiring up Dex to Github and syncing our org's team membership. This should also work for syncing group membership from LDAP, SAML, and Google (via Dex).
Add the option to disable checking email_verified. Even though it's in the spec some providers apparently don't support it.

Outside the scope of this PR, but just out of curiosity: given the experience you've developed with Dex, do you think it would be possible to deploy it in minikube in CI and emulate an authentication flow (for automated testing)?

I can spike this out next week - dex supports configuring a user with a fixed password, so it may be possible to cURL our way through the auth code flow.


docs: Add documentation for data preprocessor release

Description of the change

Related issue number

How to verify the PR

Was the PR tested

[ ] I have added >=1 unit test(s) for every new method I have added.
[ ] I have ensured all unit tests pass

Do we support txt format and packing for pre-training?

txt format is supported but not tested so not called out


BanditParameter implementation

Issue #, if available:
NA
Description of changes:
Parameter that represents a subset of the full Cartesian product of its subparameters.
Needs tests and an associated acquisition optimizer change (optimizing on the latent full Cartesian product of subparameters and rounding to nearest element of the valid subset is no bueno) before merge, but the Parameter implementation is ready for review.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Ok @apaleyes, I think I've addressed your comments.
Furthermore, using the RandomSearch acquisition optimizer assuages my concerns about using the Gradient acquisition optimizer with highly restricted design spaces, and there're unit tests for the BanditParameter now. Cursory in-house benchmarking and integration testing has supplied some anecdotal stress- and performance-test data - no catastrophic failures 👍

Codecov Report

Merging #301 into master will decrease coverage by 0.52%.
The diff coverage is 64.19%.

@@            Coverage Diff             @@
##           master     #301      +/-   ##
==========================================
- Coverage   89.78%   89.26%   -0.53%
==========================================
  Files         119      120       +1
  Lines        3839     3932      +93
  Branches      429      454      +25
==========================================
+ Hits         3447     3510      +63
- Misses        311      329      +18
- Partials       81       93      +12

Impacted Files
Coverage Δ

emukit/core/bandit_parameter.py
63.75% <63.75%> (ø)

emukit/core/__init__.py
100.00% <100.00%> (ø)

emukit/core/loop/outer_loop.py
90.56% <0.00%> (-1.75%)
:arrow_down:

emukit/core/loop/user_function_result.py
100.00% <0.00%> (ø)

emukit/core/loop/user_function.py
98.73% <0.00%> (+0.03%)
:arrow_up:

emukit/samplers/mcmc_sampler.py
86.36% <0.00%> (+3.03%)
:arrow_up:

Continue to review full report at Codecov.

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

Hey @MashaNaslidnyk @apaleyes - I've addressed the PR comments (thank you!) and have rebased onto master. Please give this another 👁 at earliest convenience.

@MashaNaslidnyk thanks for taking care of this! Just a small thing. Going forward please consider using "Squash and merge" for mergin pull requests. The main point is to keep the commit history clean, cause while working on PRs authors are free to do whatever they wish, and it isn't necessarily worth preserving.


Add Python Linter Github Action

We should add pylint so the python code is linted on push to ensure consistency and best practices

#8


Unable to create ClickHouse service integration

What happened?
Attempting to create an aiven_service_integration resource for instance using the following snippet (taken from #952):

resource "aiven_service_integration" "clickhouse_kafka_source" {
  project                  = aiven_project.clickhouse_kafka_source.project
  integration_type         = "clickhouse_kafka"
  source_service_name      = aiven_kafka.kafka.service_name
  destination_service_name = aiven_clickhouse.clickhouse.service_name
}

leads to the following error:
│ Error: cannot copy api response into terraform schema: Invalid address to set: []string{"clickhouse_kafka_user_config"}
│
│   with aiven_service_integration.clickhouse_kafka_source,
│   on services.tf line 22, in resource "aiven_service_integration" "clickhouse_kafka_source":
│   22: resource "aiven_service_integration" "clickhouse_kafka_source" {

The integration request is executed and the integration created (visible from the console). This issue seems related to #888, which was fixed by #892.
What did you expect to happen?
As a user, I would expect to the be to provide a clickhouse_user_config nested schema and access the user config's values after the service integration had been created.
What else do we need to know?
Run with latest released provider version.

Thanks for the very detailed report. The fix is already in review.


querySelectorAll('head link[href]') inconsistency between Chrome and AngleSharp

The following HTML shows an inconsistency between Chrome and AngleSharp's document.querySelectorAll for the following website's HTML and document query:
Website HTML
https://www.k-rauta.ee/
Chrome
document.querySelectorAll('head link[href]')
    NodeList(7) [link, link, link, link, link, link, link]

AngleSharp
var doc = await parser.ParseAsync(html);
var links = doc.QuerySelectorAll("head link[href]");
// links.Length is only 3

Analyzing the payload I don't think it has anything to do with QuerySelectorAll. I think this spots a problem with the parser (to be more specific I guess the tokenizer makes a mistake here somewhere).
I would need to debug this further to get the full picture, but I am almost sure QuerySelectorAll is not the one to blame here.

@FlorianRappl Just a FYI, I am using var parser = new HtmlParser();

Actually, it turns out AngleSharp is right.
The crucial difference here is that a noscript element was wrongly used in the given source. If you activate scripting (thus ignoring the content of the noscript effectively) the result is 7.
I guess you are not affiliated with the owner of the website. Otherwise, it would be good to report such bugs (e.g., in the given case the page would be pretty much unusable if noscript is applied).
To get around your issue I recommend:
var parser = new HtmlParser(new HtmlParserOptions { IsScripting = true });

Thanks.


Crash with nested StreamBuilders in CustomScrollView

Steps to Reproduce

Run flutter create bug_example.
Update main.dart with the code below.
Run application.

Code
class MyHomePage extends StatefulWidget {
  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final StreamController<int> rootStreamController = StreamController();
  final StreamController<int> secondaryStreamController = StreamController();

  Stream<int> populateStream() async* {
    int nextValue = 1;
    while (true) {
      await Future.delayed(Duration(seconds: 1));
      yield nextValue;
      nextValue++;
      nextValue = nextValue % 3;
    }
  }

  @override
  void initState() {
    super.initState();

    rootStreamController.addStream(populateStream());
    secondaryStreamController.addStream(populateStream());
  }

  @override
  void dispose() {
    rootStreamController.close();
    secondaryStreamController.close();

    super.dispose();
  }

  Widget _buildSliver(BuildContext context) {
    return StreamBuilder<int>(
      initialData: 0,
      stream: rootStreamController.stream,
      builder: (BuildContext context, AsyncSnapshot<int> snapshot) {
        if (snapshot.data == null || snapshot.data == 0) {
          return SliverToBoxAdapter(
            child: Container(
              child: Text("State 0")
            )
          );
        }
        else if (snapshot.data == 1) {
          return SliverToBoxAdapter(
              child: Container(
                  child: Text("State 1")
              )
          );
        }
        else {  // State 2
          return StreamBuilder<int>(
            initialData: 0,
            stream: secondaryStreamController.stream,
            builder: (BuildContext context, AsyncSnapshot<int> snapshot) {
              return SliverList(
                delegate: SliverChildBuilderDelegate(
                  (BuildContext context, int index) {
                    return Container(
                      child: Text("child $index")
                    );
                  },
                  childCount: snapshot.data
                ),
              );
            },
          );
        }
      },
    );
  }

  @override
  Widget build(BuildContext context) {
    return CustomScrollView(
      slivers: <Widget>[_buildSliver(context)],
    );
  }
}

Expected result: Expected that once state 2 is reached, a list of 0, 1, or 2 numbers is shown.
Actual results: Shows 'State 0', 'State 1', then crashes (presumably when building State 2)

Logs

[ +229 ms] <- compile package:bug_example/main.dart
[+2348 ms] I/flutter ( 9901): ══╡ EXCEPTION CAUGHT BY SCHEDULER LIBRARY ╞═════════════════════════════════════════════════════════
[   +1 ms] I/flutter ( 9901): The following assertion was thrown during a scheduler callback:
[        ] I/flutter ( 9901): 'package:flutter/src/rendering/viewport.dart': Failed assertion: line 1491 pos 12: 'center.parent ==
[   +1 ms] I/flutter ( 9901): this': is not true.
[        ] I/flutter ( 9901):
[        ] I/flutter ( 9901): Either the assertion indicates an error in the framework itself, or we should provide substantially
[        ] I/flutter ( 9901): more information in this error message to help you determine and fix the underlying cause.
[        ] I/flutter ( 9901): In either case, please report this assertion by filing a bug on GitHub:
[        ] I/flutter ( 9901):   https://github.com/flutter/flutter/issues/new?template=BUG.md
[        ] I/flutter ( 9901):
[        ] I/flutter ( 9901): When the exception was thrown, this was the stack:
[        ] I/flutter ( 9901): #2      RenderViewport.indexOfFirstChild (package:flutter/src/rendering/viewport.dart:1491:12)
[   +1 ms] I/flutter ( 9901): #3      RenderViewportBase.debugDescribeChildren (package:flutter/src/rendering/viewport.dart:794:17)
[        ] I/flutter ( 9901): #4      DiagnosticableTreeNode.getChildren (package:flutter/src/foundation/diagnostics.dart:2946:20)
[        ] I/flutter ( 9901): #5      TextTreeRenderer.render (package:flutter/src/foundation/diagnostics.dart:1168:43)
[        ] I/flutter ( 9901): #6      TextTreeRenderer.render (package:flutter/src/foundation/diagnostics.dart:1294:39)
[        ] I/flutter ( 9901): #7      FlutterError.dumpErrorToConsole (package:flutter/src/foundation/assertions.dart:652:11)
[   +1 ms] I/flutter ( 9901): #8      FlutterError.reportError (package:flutter/src/foundation/assertions.dart:741:14)
[   +1 ms] I/flutter ( 9901): #9      RenderObject._debugReportException (package:flutter/src/rendering/object.dart:1270:18)
[   +4 ms] I/flutter ( 9901): #10     RenderObject._layoutWithoutResize (package:flutter/src/rendering/object.dart:1587:7)
[        ] I/flutter ( 9901): #11     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:844:18)
[        ] I/flutter ( 9901): #12     RendererBinding.drawFrame (package:flutter/src/rendering/binding.dart:344:19)
[        ] I/flutter ( 9901): #13     WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:774:13)
[   +1 ms] I/flutter ( 9901): #14     RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:283:5)
[   +1 ms] I/flutter ( 9901): #15     SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1102:15)
[   +1 ms] I/flutter ( 9901): #16     SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1041:9)
[        ] I/flutter ( 9901): #17     SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:957:5)
[   +1 ms] I/flutter ( 9901): #21     _invoke (dart:ui/hooks.dart:259:10)
[        ] I/flutter ( 9901): #22     _drawFrame (dart:ui/hooks.dart:217:3)
[        ] I/flutter ( 9901): (elided 5 frames from class _AssertionError and package dart:async)
[        ] I/flutter ( 9901): ════════════════════════════════════════════════════════════════════════════════════════════════════
[   +9 ms] I/flutter ( 9901): Another exception was thrown: Updated layout information required for RenderViewport#07888 NEEDS-LAYOUT to calculate semantics.
[ +813 ms] I/flutter ( 9901): Another exception was thrown: 'package:flutter/src/rendering/object.dart': Failed assertion: line 1247 pos 12: '_debugCanPerformMutations': is not true.
[  +18 ms] I/flutter ( 9901): Another exception was thrown: A RenderViewport expected a child of type RenderSliver but received a child of type RenderErrorBox.
[ +133 ms] I/flutter ( 9901): Another exception was thrown: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 2430 pos 16: '!_dirtyElements[index]._active ||
_dirtyElements[index]._debugIsInScope(context)': is not true.
[ +824 ms] I/flutter ( 9901): Another exception was thrown: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 3227 pos 12: 'child._parent == this': is not true.
[   +4 ms] I/flutter ( 9901): Another exception was thrown: A RenderViewport expected a child of type RenderSliver but received a child of type RenderErrorBox.
[+1186 ms] I/flutter ( 9901): Another exception was thrown: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 2430 pos 16: '!_dirtyElements[index]._active ||
_dirtyElements[index]._debugIsInScope(context)': is not true.
[ +767 ms] Updating files
[  +64 ms] I/flutter ( 9901): Another exception was thrown: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 3227 pos 12: 'child._parent == this': is not true.
[   +6 ms] I/flutter ( 9901): Another exception was thrown: A RenderViewport expected a child of type RenderSliver but received a child of type RenderErrorBox.
[ +103 ms] DevFS: Sync finished
[   +2 ms] Syncing files to device Pixel 3a XL... (completed in 6,560ms, longer than expected)
[   +1 ms] Synced 0.9MB.
[   +2 ms] Sending to VM service: _flutter.listViews({})
[   +6 ms] Result: {type: FlutterViewList, views: [{type: FlutterView, id: _flutterView/0x73e26f7620, isolate: {type: @Isolate, fixedId: true, id: isolates/945811341266935, name: main.dart$main-945811341266935, number:
945811341266935}}]}
[   +1 ms] <- accept
[   +1 ms] Connected to _flutterView/0x73e26f7620.
[   +2 ms] �  To hot reload changes while running, press "r". To hot restart (and rebuild state), press "R".
[   +1 ms] An Observatory debugger and profiler on Pixel 3a XL is available at: http://<IP_ADDRESS>:52504/MeqvtkagceQ=/
[   +1 ms] For a more detailed help message, press "h". To detach, press "d"; to quit, press "q".
[ +880 ms] I/flutter ( 9901): Another exception was thrown: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 3227 pos 12: 'child._parent == this': is not true.
[   +5 ms] I/flutter ( 9901): Another exception was thrown: A RenderViewport expected a child of type RenderSliver but received a child of type RenderErrorBox.
[+1160 ms] I/flutter ( 9901): Another exception was thrown: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 2430 pos 16: '!_dirtyElements[index]._active ||
_dirtyElements[index]._debugIsInScope(context)': is not true.
[ +833 ms] I/flutter ( 9901): Another exception was thrown: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 3227 pos 12: 'child._parent == this': is not true.
[   +4 ms] I/flutter ( 9901): Another exception was thrown: A RenderViewport expected a child of type RenderSliver but received a child of type RenderErrorBox.
[ +868 ms] Application finished.

Analyzing bug_example...
No issues found! (ran in 8.5s)

[√] Flutter (Channel stable, v1.12.13+hotfix.7, on Microsoft Windows [Version 10.0.18362.592], locale en-NZ)
    • Flutter version 1.12.13+hotfix.7 at C:\sdk\flutter
    • Framework revision 9f5ff2306b (9 days ago), 2020-01-26 22:38:26 -0800
    • Engine revision a67792536c
    • Dart version 2.7.0

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at C:\Users\username\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: C:\Users\username\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\191.6010548\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Android Studio (version 3.4)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 38.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[√] Android Studio (version 3.5)
    • Android Studio at C:\Users\username\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\191.6010548
    • Flutter plugin version 41.0.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[!] IntelliJ IDEA Community Edition (version 2019.1)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2019.1
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[√] IntelliJ IDEA Ultimate Edition (version 2019.3)
    • IntelliJ at C:\Users\username\AppData\Local\JetBrains\Toolbox\apps\IDEA-U\ch-0\193.6015.39
    • Flutter plugin version 42.1.4
    • Dart plugin version 193.5731

[√] VS Code (version 1.41.1)
    • VS Code at C:\Users\username\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.8.0

[√] Connected device (1 available)
    • Pixel 3a XL • 92HAX012JY • android-arm64 • Android 10 (API 29)

! Doctor found issues in 1 category.

Hi @tfinlay
It seems like it can be solved by using SliverToBoxAdapter #25861 Comment #30253 Comment
I'm closing the current one as duplicate of #30253.
Thank you


Decoding example theme throws CastError exception

Using VSCode with "Uncaught Exceptions" turned on, when I try to decode all the example themes in this repo with ThemeDecoder.decodeThemeData(json), it throws the following exception:
_CastError (Null check operator used on a null value)

I think the exception is actually coming from the underlying package json_scheme2, but I'm not sure which part of the code is failing. Would you be able to help look into this?
PS I think the themes are still being decoded properly if we ignore the exception.
A related issue was raised in https://github.com/zeshuaro/flutter_theme/issues/157

I cannot reproduce this issue via the example in this repo.  I see an issue logged with flutter_theme, and the problem may be there.  If you can provide more info on how you are able to reproduce via just the code in this repo, I'd be happy to take another look.
As a FYI, any error coming from json_schema2 is really a warning rather than an error.  All the entities supported by this repo also come with a Schema file that in debug and profile mode it validates against.  If you're a Null check... error from json_schema2 means that you have a schema defined but it cannot be properly evaluated for some reason.  You can disable the schema validations by setting SchemaHelper.enabled = false; at the start of your application, or you can try to diagnose what is wrong with the schema in question.
I'll leave this open for a few days in case you would like to provide more info about how to repro the problem via the example app in this repo.

Looks like the issue has been fixed in the latest version 3.2.0. But it's definitely reproducible with the example app in this repo on 3.1.2+1.
With "Uncaught Exceptions" turned on in VSCode:

If you run the example app in this repo and then click on any of the themes, the debugger will pause with the error _CastError (Null check operator used on a null value) being raised in json_schema2.

Anyway thanks for the details on the schema validation. Since this issue has somehow been resolved in the latest version, I'll just close this issue and simply use the latest version 🙂


Undefined reference to 'main'

Hello!
I have been jumping into the world of OSDev and stumbled across your amazing write ups.
I am on step two creating the make file and I keep getting the error:
(.text.boot+0x34): undefined reference to 'main'
make: *** [Makefile:15: kernel18.img] Error 1
I am trying to figure out if the issue is in my linker.ld file or the Makefile itself. It seems to properly make the boot.o and kernel.o files but fails when trying to link them into one file.
I have checked multiple times my linker.ld file matches yours as well as the Makefile (minus the file path to the cross compiler).
Any suggestions of what I can try? Thanks for your help and for taking the time to make these tutorials!

Hi,
Great to hear from you! This error is what I'd expect to see if you didn't have a main() function defined in kernel.c.
Can you post the contents of your kernel.c file? If you're strictly going through the tutorial then this should read as follows for part2:
void main() {
   while (1);
}

It's very unexciting, it literally just spinning in an infinite loop doing nothing! I promise it gets more interesting as you go along though!
All the best,
Adam

Thank you so much for responding! After attempting to compile it again (with no changes to the code), it worked with no issue and booted on my Pi! I am not sure what was giving me the error for so long.  I am looking forward to continuing through the rest of your write up.

That's great news! Enjoy the journey ☺


Search box doesn't work

Hi @geojaz - I think we're in pretty good shape for this week, but I did notice that the search box in the top right of the screen does not request updated data.
Could you take a look?

@rosskukulinski I'm on it!

🙏


Bug: STOP_ASYNC_VALIDATION erases previous asyncErrors if currently blurred field passes validation

Are you submitting a bug report or a feature request?
Bug report
What is the current behavior?

Build a form with two inputs with asyncValidation
Invalid one of the inputs
Enter something in the second input that is valid (causes no errors)
STOP_ASYNC_VALIDATION does this: result = deleteIn(result, 'asyncErrors') https://github.com/erikras/redux-form/blob/master/src/createReducer.js#L341

What is the expected behavior?
asyncErrors still contains the error from the first input so the form can continue showing this field is invalid.
What's your environment?
Redux Form 6.6.3
Other information
I'm opening this as a bug report to start a discussion to at least confirm this behaviour is a bug. If so I'm happy to help work on a PR to fix it once I understand it properly. Is asyncErrors supposed to be state shared between fields like this?

Any updates on this?

Funny you should ask, I just made a PR. Looking forward to feedback.

@davedx you can find workaround for this here: https://github.com/erikras/redux-form/issues/980#issuecomment-278360610
Just merge previous errors and current error (or delete the key, if no errors for current input)

Fix published in v7.0.0.

I made a PR that was merged
Sent from my iPhone

On 16 Jun 2017, at 15:11, Nikita Shulipa<EMAIL_ADDRESS>wrote:
Any updates on this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Add random.binomial and random.multinomial

Add JAX counterparts of numpy.random.binomial and numpy.random.multinomial to jax.random package. See https://github.com/google/jax/issues/480#issuecomment-1272587620 for context. A current workaround is using the JAX substrate of TensorFlow Probability:
from jax import random, numpy as jnp
from tensorflow_probability.substrates import jax as tfp

def binomial(key, n, p, shape=()):
    return tfp.distributions.Binomial(n, probs=p).sample(
        seed=key,
        sample_shape=shape,
    )

def multinomial(key, n, p, shape=()):
    return tfp.distributions.Multinomial(n, probs=p).sample(
        seed=key,
        sample_shape=shape,
    )

key = random.PRNGKey(0)

key, subkey = random.split(key)
print(binomial(subkey, 9, .8, [2, 5]))

key, subkey = random.split(key)
print(multinomial(subkey, 9, jnp.array([.7, .2, .1]), [4]))

Output:
[[7. 8. 8. 7. 5.]
 [5. 8. 9. 8. 5.]]
[[7. 1. 1.]
 [4. 3. 2.]
 [5. 3. 1.]
 [4. 4. 1.]]

@sharadmv, is it possible to port tfp's implementation to jax.random?

Any plans to add random.multinomial now that random.binomial has been merged?
The distribution could be implemented either as sequence of negative binomials or by repeated categorical draws.
See also the Wikipedia entry on multinomial variate generation).

Thanks for reaching out – I don't know of anyone working on this currently.

A workaround if you want pure JAX is to take the log of your probabilities vector (that sums to 1):
jax.random.categorical(key, jnp.log(p))

(Based on the "contract" of jax.random.categorical:

logits: Unnormalized log probabilities of the categorical distribution(s) to sample from,
so that softmax(logits, axis) gives the corresponding probabilities.

)
That can't be the most efficient way to sample though...

Fyi: You would need to sum many categorical samples to get a multinomial
sample.
On Sat, Jan 27, 2024, 7:23 PM Andrey Portnoy @.***>
wrote:

A workaround if you want pure JAX is to take the log of your probabilities
vector (that sums to 1):
jax.random.categorical(key, jnp.log(p))
(Based on the "contract" of jax.random.categorical:
logits: Unnormalized log probabilities of the categorical distribution(s)
to sample from,
so that softmax(logits, axis) gives the corresponding probabilities.
)
That can't be the most efficient way to sample though...
—
Reply to this email directly, view it on GitHub
https://github.com/google/jax/issues/13327#issuecomment-1913383992, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJFSI62724M6LFSTMPHHGLYQWLABAVCNFSM6AAAAAASFQR3O6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGM4DGOJZGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I would use a multinomial feature if added! The categorical workaround is a nonstarter if I want a lot of samples.

👍

I would benefit from multinomial as well. I'm currently using the numpy one for efficient bootstrap sampling and not having it here makes it harder to make the switch to jax.

I've created a PR for this: https://github.com/jax-ml/jax/pull/25688.


Graph traits filtered

set types removed from grap traits + less duplication of using between public and impl of graph type

/intelci: run

/intelci: run

/intelci: run


Add two API endpoints

I moved the global variables within the Service.
The service is passed as a pointer to the Handler.

These kinds of datas are not properties of the Service, so should not be fields of the Service structure.
See #18 on how to have a interface, and then use any implementatino of the storage.

Replaced by https://github.com/teal-finance/rainbow/pull/25


Performance Fixes

add TileGroupHeader* as argument to ReadRecord to avoid lookup with GetTileGroup()
rewrite access to RW set in TransactionContext::Record* functions to avoid duplicated lookups

Coverage increased (+0.006%) to 77.405% when pulling a86e6b97497dad64c76fd02ff18f8366881cbead on tcm-marcel:fix/tilegroupheader into 0c6480658ba01928e9a74e0d2b4c0c9b5841b58b on cmu-db:master.

Callgrind call graph at PerformVectorizedRead before (left) and after the fix (right) when running oltpbench ycsb RecordRead. Only small changes, but it seems to be a little better. I don't know yet, why some of the invocation numbers are different.

Be aware that the colors are different between left and right when reading.


feat: add multiple option to attachment decorator

Proposed changes
Sometimes we need to attach multiple files to a model, e.g. a job application may contain CV, cover letter, portfolio, copy of degree certificates, etc.
This PR add an multiple option to the decorator. Like this:
class User extends BaseModel {
  @attachment({ multiple: true })
  public photos: AttachmentContract[]
}

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[ ] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

I am not able to run all the unit tests. Some tests failed with no details. The other tests don't even run...

Hi @guoyunhe.
Thanks for your PR. Did you discuss with Virk if he would like this feature adopted before beginning the PR?

@ndianabasi No.

This is SO damn useful, I hope you get permission + passing tests to get this added.

I think multiple attachments should be represented as multiple rows in the database. For example: I will rather have user_photos table and then give user the ability to delete or update each photo as a database row vs storing all photos in a single array.
So, for now I will skip this PR to keep the codebase simple :)


⚠ Bofrost.be has degraded performance

In 2dd97f0, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1000 ms

Resolved: Bofrost.be performance has improved in 3e06963 after 28 minutes.


Key value add field is still a bit buggy

The key value add field is definitely not complete. You can't delete headers, and some of states are not working properly.

Its not in a bad state now


chore: adjust content security policy

closes  https://github.com/Stand-With-Crypto/swc-internal/issues/274
fixes
What changed? Why?

PlanetScale deploy request

Notes to reviewers

How has it been tested?

[X] Locally
[x] Vercel Preview Branch
[ ] Unit test
[ ] Functional test

assuming https://cnv.event.prod.bidr.io/log/cnv is from our tag manager as well?

yeah, something called beeswax being injected through our GTM


build: bump vm-memory from 0.14.0 to 0.15.0

Summary of the PR
Please summarize here why the changes in this PR are needed.
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[ ] All commits in this PR have Signed-Off-By trailers (with
git commit -s), and the commit message has max 60 characters for the
summary and max 75 characters for each description line.
[ ] All added/changed functionality has a corresponding unit/integration
test.
[ ] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[ ] Any newly added unsafe code is properly documented.

@rbradford @likebreath


Non existing blueprint should return 404

Non existing blueprint should return 404, now an empty blueprint is returned

fixed by #382


ValueError: Input 0 of layer sequential is incompatible with the layer

demography = DeepFace.analyze(img_path)
ValueError: Input 0 of layer sequential is incompatible with the layer: : expected min_ndim=4, found ndim=3. Full shape received: (None, 48, 48)

duplicated - https://github.com/serengil/deepface/issues/674


Roll Engine from eb84fd266695 to 8ea5cb75e940 (3 revisions)

https://github.com/flutter/engine/compare/eb84fd266695...8ea5cb75e940
2022-03-27<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from LWC6VyqVC... to s177R8DTP... (flutter/engine#32280)
2022-03-27<EMAIL_ADDRESS>Roll Skia from 2ac7682b5303 to fae7be2a9c5b (1 revision) (flutter/engine#32279)
2022-03-27<EMAIL_ADDRESS>Roll Dart SDK from 4517dfc3774d to 04cbb6986163 (4 revisions) (flutter/engine#32278)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from LWC6VyqVCYHg to s177R8DTP3tA
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.