Reimplement Download All

Even after fixing the permissions issue, Download All fails for the typical workload > 100 files (though, it has succeeded with as many as 75 files). This appears to be a timeout issue with AppSync. Even manually increasing the "function" lambda timeout in the console does not make a difference.
Also, the current function does not take into account additional processing to strip out the metadata. The current workaround is downloading all the files for the assignment id from the bucket using the AWS CLI and then running a script to process all the files locally (see gist). Stripping the metadata is currently needed to support smooth uploads to SoundCloud. During the bulk upload process, SoundCloud will attempt populate track titles using some of the metadata but not in a consistent way or in a way that matches our format.
The issue with adding this metadata processing to a lambda is that ffmetadata (and any other similar node package for that matter) is a wrapper for ffmpeg binaries that need to be installed on the system.
I've briefly looked into using exodus (see Building a Media Transcoder with Exodus, FFmpeg, and AWS Lambda
) or container support for lambdas. Another option might be using Elastic Transcode.
Stripping metadata is not something actually needed for V1, as the expectation is that users would stream the playlist from within the app and we can easily control how the track names appear. However, the ability to do some processing (such as file conversion) may be a nice-to-have.
The immediate goal of this ticket is to redesign and implement this feature in a smarter way that does more work behind the scenes and not all at once when the admin user clicks the Download All button. By breaking up this workload, it should also be easier to support audio file processing asynchronously.

temp workaround script that was used:
https://github.com/thephilgray/mew-app/tree/feature/%236-reimplement-download-all-wip/scripts/processSubmissions

this was reworked as a fargate task.


Having Issues installing this on latest ubuntu under WSL

root@TestVM:/# sudo cast install teamdfir/sift-saltstack
WARN[0000] using unauthenticated github client, could result in API rate limiting
INFO[0000] checking operating system support             component=distro owner=teamdfir repo=sift-saltstack
INFO[0000] operating system is supported                 component=distro owner=teamdfir repo=sift-saltstack
INFO[0000] rendering manifest                            component=distro owner=teamdfir repo=sift-saltstack
FATA[0000] template: pillar_template:1: function "default" not defined
I am running
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.1 LTS
Release:        22.04
Codename:       jammy

Interesting, thank you for the report. I will take a look.

Can't immediately reproduce it. What version of cast are you using?

@ekristen  Thanks for the tip. Changing cast version to latest one did solved this.


still planning to implement webgl version?

Hello, thanks for making such a great tool. I like the clustergrammer and now we're testing it. Btw, I read from your paper that you were planning to build webgl version to incorporate larger dataset. How is it going? we wish to load ~500 genomes by ~10/15k features.

Hi,
Hi, we're glad you like Clustergrammer. Yes, we're working on the WebGL version of Clustergrammer (Clustergrammer-GL and a new WebGL Jupyter widget Clustergrammer-Glidget), but it is still under heavy development. Feel free to have a look at the linked repos and we can keep you up to date as things start to finalize. Here's a link to a Jupyter notebook showing the old and new widgets: CCLE Clustergrammer Glidget Jupyter notebook. Also, here's a link to a visualization with 25,000 rows (visualizing the CyTOF data included in the paper.
In terms of matrix size, I've been able to visualize matrices with about 7-10 million matrix-cells (e.g. 1000 cell lines with 7,000-10,000 genes using a 2017 iMac), but this depends on your graphics card. Your matrix of 500 genomes (columns) and 10,000 features (rows) ends up being >5 million matrix cells. So, it could work but your Visualization-JSON file will be large.
Let us know if you have any questions or feature requests.
Best,
Nick

Hi @hyoo, I'm closing this issue since we are currently actively developing Clustergrammer-GL https://github.com/ismms-himc/clustergrammer-gl. Please let us know if you have any feedback.


EHRbase doesn't build Docker image on aarch64

Background
I am building a docker container for EHRbase from source on my MacBook with M1 (aarch64) processor. The new 0.23.0 version of EHRbase doesn't build as aarch64 natively, since Temurin/Adoptium 17 is not available in aarch64 (c.f. the JFrog repo for aarch64 vs. that for x86_64).
Enhancement
As long as Adoptium 17.x is not available for aarch64, a possible workaround would be using a different JDK, perhaps Azul Zulu.
Further information
I've created a variant of the Dockerfile for the current (2023-01-12) version of Azul Zulu. I've tested building with no build arg (i.e. TARGETPLATFORM = 'linux/arm64' successfully on my laptop, and have also successfully stood up EHRbase on a real Linux AMD64 machine. It doesn't work with targetplatform=linux/x64 or linux/amd64 on the MacBook, but I think that's okay. This is certainly a somewhat niche usecase, but developers must be happy too when running their local EHRbase during client development, and Rosetta is slooow. This wasn't an issue with older versions of EHRbase, I've been running 0.21.1 under arm64 for a while now. ;)

I just stumbled upon this issue. Any plans to add support for M1?

I’ve had a similar workaround as well.
Usually I build the jar on my machine and then copy it into a Java runtime base image which has arm64 support.
For some reason building the jar within an arm64 docker image made the server run slower for me (think we had this discussion in another issue #604)
Having native support for an arm64 image would be great for sure.


Create Story generator.md

📝 Description
This agent takes a Title of the story as a prompt and it will generate a story
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

@Harshil-08 Hey mate:
we just checked your prompt and seems you have some issues with your variable. It doesn't show up correctly because it's in red color. It will show in Green color if everything is all right.

@donqu1xotechina the issue have been fixed.

@Harshil-08 :
One more issue here, please change the branch from main to beta.

@Harshil-08 :
One more issue here, please change the branch from main to beta.

Done


Is there any support for adding html tags in tabTitle in ionic 3?

I need to add a heading and a span text in tabTitle in ionic tabs.
  <ion-tabs>
      <ion-tab tabTitle="
Heading

This is a span text"></ion-tab>
  </ion-tabs>

I have tried like above but it didn't work.But i want like above.

Hello! Thank you for opening an issue with us!
The Ionic Framework issues list is designated for the reporting of bugs with the Ionic Framework or for requesting new features. As this seems like more of a support question I am going to close this issue.
I urge that you ask this question on our forum or on our slack channel.
Thanks for using Ionic!


OpenBLAS + TBB

Hi,
I want to use OpenBLAS with application that uses TBB multi-threading for parallel computation, but it is too slow in all configuration (Single-threaded, Multi-threaded, OpenMP). As I understand it is memory bottleneck, coz OpenBLAS uses the same global memory in different TBB threads. Maybe do you have any ideas?

Can you provide some more details please - your hardware and operating system, which version of OpenBLAS you are trying to use and which BLAS/LAPACK functions (and perhaps matrix sizes) you are experiencing this slowness with ? It may also help to know which application it is (unless it is a closed source one of your own)

Care to mention operating system?

It is applicable for Windows and Linux (Ubuntu), CPU is i7 (8 logical cores). I am calling cblas_sgemm() for matrix multiplication. If I set the number of threads to 8 it is fast, but when I use TBB for parallel execution (for example, create a thread for 2 matrix rows) and single-threaded OpenBLAS it is too slow. I've reviewed code and found in memory.c OpenBLAS allocates memory array using the number of cores (threads) and uses lock/unlock for reading/writing to this memory area ... I suppose it is bottleneck, if each external thread try to use the same memory.

Also I've observed the same problem https://github.com/xianyi/OpenBLAS/issues/1283 in valgrind.

Can you post full CPU name that everybody can google?
What you get is core oversubscription, I have in works omp hierarchy support.
You are not right about shared memory. you dont have NUMA in your small CPU, unless data fits in 2.5MB outer cache attached to core (with 2 hyperthreads) - base RAM is same for all.
How big is sgemm input and output?

From my (still very limited) understanding of TBB, it is C++ only, while OpenBLAS itself is a mix of C (BLAS part, handful of re-implemented LAPACK functions) and FORTRAN (LAPACK imported from the netlib reference implementation). So my current impression is that adding proper TBB support to OpenBLAS would imply a major redesign/rewrite. If a major bottleneck is actually  memory.c, perhaps building for a very low number of threads (like make NUM_THREADS=2) would provide an acceptable tradeoff ?

A possible issue here is that the TBB parallelism is breaking down problems into very tiny blocks so that when OpenMP is called it is having to deal with small matrices.  What are the sizes of the matrices being passed into OpenMP sgemm() calls?

Sorry for late reply. The sizes of the matrices are small. I think your suggestion is right, because when I don't split matrix multiply using TBB, results are satisfactory.

This appears to be an example of the broader problem that occurs whenever there is parallel threading going on at multiple levels in the software without communication between the levels- If the processor appears to have 8 cores, then the top level will start 8 threads in parallel.  Later on, each thread will start 8 threads at its level, and all of a sudden you have 64 threads contending with each other.  The best solution is to use a single multithreading scheme (e.g. OpenMP) throughout the code and let higher level threads determine when lower-level threads should go single threaded.   Another possibility is to compile the lower-level code (e.g. OpenBLAS) to just always run single threaded.  A third option (often best when the lower level is BLAS and nearly all of the computational work is at the lower level) is to compile the upper-level code to run single-threaded.

Thanks for the detailed answer. In my case, I'm using  single threaded OpenBLAS version with TBB parallelism. I think the issue can be closed.

@sbudenkov Can you please tell me how did you achieve single threaded OpenBLAS with TBB parallelism?

@goplanid check makefile.rule - disable threading and enable locking.


merge from arlac77/template-arlac77-github,arlac77/template-esm-only

package.json

chore(deps): add ^7.3.4 remove ^7.3.3 (devDependencies.c8)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


general helper NPI provider does not provide function prototype data (and other issues)

the NPI modules in eBPF core and netext has several issues. This issue is used as a placeholder for tracking all these.

[x]  Global helper NPI provider in EC (for map functions) dont provide the prototype information. They are obtained from the net ebpf extension's provider modules.
[x] In interpreter case (load_byte_code) program type specific helper function addresses are not loaded. Only the global ones are loaded.
[x] net ext uses ebpf_provider_load wrapper functions defined in ebpf core. This complicates the code as provider modules pass data unnecessarily. e.g. hook provider maintains a local_registration->provider_data which is not required etc. These APIs are simply wrappers over NMR and adds no value to net_ext modules. IMO this code needs to be cleaned up.

This issue can now be closed.


Deployment with Surge or similar

Right now I just ran the program, did npm run build, and then surge -p build. Five seconds later I can show this to others:
https://highfalutin-sofa.surge.sh
I can also customize the URL to whatever I want (if available).
I feel like this is a great feature that could be added—one command, no config deployments. Imagine you're learning React and did something cool and want to share it but have no idea what Heroku, Digital Ocean, etc. are. This did it for you!
surge can be added as a dev dependency and a deploy script can be added that would be npm run build && surge -p build.
What do you think?

I +1ed, but please consider that it puts a dependency on a third party service. maybe gh-pages?

So good! We've thought about adding upload to gh pages as well. Need to figure out how to integrate it with the experience.

also possible - ngrok, if you don't want to 'host' elsewhere

@threepointone Very good point—Surge recently had issues with Digital Ocean taking their sites down, though the actual product and team are very good and helpful.
GitHub pages may work as well but I've actually never used them ha, I just want easy deployment :-)

I vote for GH pages.
But we need to fix the “root URL” issue for them:
https://github.com/facebookincubator/create-react-app/blob/cb7b9583b9d2722afee9dfdac084a64cd4cb177b/config/webpack.config.prod.js#L38-L40

the publicPath is not needed for production, if you change the '/'  for '', it will work on GitHub pages.
I don't understand quite well the black magic behind this, but is how my deployments are made.
You can check the output here:
Without publicPath specified
With publicPath
NOTE: I'm a beginner and don't know the implications of doing this

Woah, this is growing really fast, I'll just reference a PR for this https://github.com/facebookincubator/create-react-app/pull/94

the publicPath is not needed for production, if you change the '/' for '', it will work on GitHub pages.

Won’t this break single page apps with client side history API routing though?
Like I describe here: https://github.com/facebookincubator/create-react-app/issues/95#issuecomment-234639120

Yes, indeed, @gaearon!

Hey friends, I put some thought into this discussion and wrote a blog post about how to deploy this project to Netlify. I am sure the same steps can be made with surge and other things like Firebase as well.
Netlify can perform continuous deploys similar to GH pages pretty easily, app.netlify.com is React app hosted with netlify 🐶  + 🍔  = 😎
As far as the routing, we have a solution in our documentation for that here.
We do the following in a _redirects file for our project:
/* /index.html 200

@MrOutis
This works when not using React Router. As soon as React Router is introduced, things get strange!
As @gaearon mentioned in another issue, this has to do with browserHistory. I tried setting a basename for React Router, but still no luck.
The assets do get loaded, just nothing renders on the page.
So I guess for now this will not work as a static website with React Router.
I have been trying everything!

@selfup, have you tried the redirect method mentioned by @bdougie?
It's kind of dirty, and it makes me some noise how this monkey patch would be treated when the project is ejected.
There's also this repository explaining the "hack" https://github.com/rafrex/spa-github-pages
However, people are baking a pretty good solution right here: https://github.com/facebookincubator/create-react-app/pull/94, have you checked it?

@MrOutis
I will check the spa-github-pages link.
I have tried the publicPath route, but maybe I did something wrong! I'll keep at it 👍

@MrOutis
So I used the #94 solution, and my assets are loading just fine now on gh-pages!
For the bundled js it points to: <link href="http://selfup.me/react-storage/main.cd0d88d3b4d6fca8d4afa4a741b83ee9.css"
So it knows where to grab stuff.
However, nothing is rendering. No errors in the console. Could it be my Router setup?

Ok so if you remove browser history it works on gh-pages.
However, that is not the best approach.
How could this work with browser history?

GH pages won’t really work well with browser history (and IMO that’s fine).

@gaearon Agreed. Yea I mostly use gh-pages for fun. Deploying to a VPS will be fine once MVP is proven.

We added support for gh pages in #94 and #162.
Please see #162 for a description of how it works, and potential improvements for other services.

I tried the Github pages deployment and the command
git push origin :gh-pages

threw an error saying
error: unable to delete 'gh-pages': remote ref does not exist
error: failed to push some refs to 'git@github.com:name/repo'

Should we be including that command by default?

I also just noticed that it fails when using a custom subdomain with GH Pages, since then the URL is someurl.com/reponame/. With the absolute paths in the index.html that point to /main.hash.js those files don't exist since they're at someurl.com/reponame/main.hash.js

I was trying to figure out a universal set of commands for replacing a remote branch but couldn't quite manage them. Maybe there's something like "delete if exists"? Subtree push doesn't seem to have a "force" option. Alternatively we could display commands that delete everything except build, move its files in the root, and do regular force push.
I'm not sure I understand your point about custom domain. How is this different from regular GH Pages? If you put homepage in config, it should infer correct public path, no?

To be clear, GH Pages (regular or custom domain) would only work in master and only if you specify homepage. This is what #94 added.

Ah, that's my problem. 👍 didn't realize a fix for this was already in!

If you specify homepage, we now print instructions for GH Pages deployment. I agree it would be great to integrate this directly into the tool, and I think @ForbesLindesay was planning to look into this. In the meantime I’ll close this issue because it is not very actionable if you’re not planning to directly contribute to this in form of PRs.


App crashes when image returned via resume event, Android

Bug Report
Problem
Per "android quirks" section of readme, camera may return imageData via cordova "resume" event, when Android closes main app while camera is open.  I'm triggering this scenario manually using Android settings > System > Developer Options > "Don't keep activities".  This works OK in cordova-plugin-camera 4.1.0. (imageData is returned per docs, via "resume" event.)  But in cordova-plugin-camera 5.0.1, app crashes just after taking a picture, and before/as app restores.
What is expected to happen?
App should not crash when it restores, per scenario above.  (Works per documentation in version 4.1.0)
What does actually happen?
App crashes after taking a picture.
Information

version 4.1.0 works per docs
App doesn't appear to launch again at all.  (e.g. crash happens, and an alert("x") on the first line of JS never fires.)

Command or Code

Camera call I'm using:

	navigator.camera.getPicture(
		function (imageData) { alert("Camera callback, normal way."); },
		function (msg) { alert('Camera error: ' + msg); },  // Error CB
		{ quality:95, sourceType:Camera.PictureSourceType.CAMERA, destinationType:Camera.DestinationType.FILE_URI, correctOrientation:true, allowEdit:false, targetWidth:2000, targetHeight:2000 }
	);

I'm triggering this "manually" using [Android settings > System > Developer Options > "Don't keep activities"].  That setting is important to triggering this issue.  Unless there's another way to reliably trigger the "android quirks" scenario.

Environment, Platform, Device
Voltbuilder // Android // Sony Xperia ZX2 Compact
Version information
Building with Voltbuilder.  I didn't specify a cordova version, so I assume they're using something recent.
Checklist

[x] I searched for existing GitHub issues
[x] I updated all Cordova tooling to most recent version
[x] I included all the necessary information above

@blukis  are you able to test my fork? I did make a change to the Android plugin code. You can install the fork with next command
npx cordova plugin add https://github.com/PieterVanPoyer/cordova-plugin-camera/#bugfix/issue-665-save-instance-restore-bug

@blukis are you able to test this? Kind regards Pieter

@blukis are you able to test this? Kind regards Pieter

Unfortunately, as I only use VoltBuilder (where they only allow manually-approved plugins from npm), I'm unable to test.  Thank you for the fix!

Unfortunately, as I only use VoltBuilder (where they only allow manually-approved plugins from npm), I'm unable to test.  Thank you for the fix!

@blukis I think you are absolutely right, the described scenario resulted in an error and a plugin crash. I did make a change and a Pull Request to the Android plugin code. You can install the fork with next command. Are you able to test my fork?
npx cordova plugin add https://github.com/PieterVanPoyer/cordova-plugin-camera/#bugfix/issue-665-save-instance-restore-bug

First remove the old plugin offcourse.

Thanks, that works for me!!!

@blukis I think you are absolutely right, the described scenario resulted in an error and a plugin crash. I did make a change and a Pull Request to the Android plugin code. You can install the fork with next command. Are you able to test my fork?
npx cordova plugin add https://github.com/PieterVanPoyer/cordova-plugin-camera/#bugfix/issue-665-save-instance-restore-bug

First remove the old plugin offcourse.

Thanks, that works for me!!!

Hi @PieterVanPoyer ,
After using this plugin npx cordova plugin add https://github.com/PieterVanPoyer/cordova-plugin-camera/#bugfix/issue-665-save-instance-restore-bug android crash issue fixed, but my app is restarting again sometimes, when i click photo  what is the issue, my device is android 10.

Hi @PieterVanPoyer ,
After using this plugin npx cordova plugin add https://github.com/PieterVanPoyer/cordova-plugin-camera/#bugfix/issue-665-save-instance-restore-bug android crash issue fixed, but my app is restarting again sometimes, when i click photo  what is the issue, my device is android 10.

Hi @PieterVanPoyer ,
After using this plugin npx cordova plugin add https://github.com/PieterVanPoyer/cordova-plugin-camera/#bugfix/issue-665-save-instance-restore-bug android crash issue fixed, but my app is restarting again sometimes, when i click photo what is the issue, my device is android 10.

Restarting is perfectly normal in Android when the app goes in the background. For more information see: https://github.com/apache/cordova-plugin-camera/issues/665#issuecomment-757835110

Hi @PieterVanPoyer ,
After using this plugin npx cordova plugin add https://github.com/PieterVanPoyer/cordova-plugin-camera/#bugfix/issue-665-save-instance-restore-bug android crash issue fixed, but my app is restarting again sometimes, when i click photo what is the issue, my device is android 10.

Restarting is perfectly normal in Android when the app goes in the background. For more information see: https://github.com/apache/cordova-plugin-camera/issues/665#issuecomment-757835110

I've also updated the latest plugin 5.0.2, but the same problems occur on Android 10 (device - redmi not 7pro). The plugin 5.0.2. Forcefully close the application once we capture a photo. This happens on some specific devices. Please update the new version I'm waiting for the new version
Node -v ----14.16.0
ionic -v ---- 6.13.1
cordova -v----10.0.0
ionic cordova platform add<EMAIL_ADDRESS>npm i<EMAIL_ADDRESS>tested and both same crash.
I'm waiting for your response, thanks


removed <dirent.h> preventing compile on some archs

On some architectures <dirent.h> is not available (preventing the code from compiling); meanwhile, the code does not appear to use any of the documented functionality of the same:

 DESCRIPTION
The internal format of directories is unspecified.

The <dirent.h> header defines the following data type through typedef:

DIR
A type representing a directory stream.

It also defines the structure dirent which includes the following members:

ino_t  d_ino       file serial number
char   d_name[]    name of entry

The type ino_t is defined as described in <sys/types.h>.

The character array d_name is of unspecified size, but the number of bytes preceding the terminating null byte will not exceed {NAME_MAX}.

The following are declared as functions and may also be defined as macros. Function prototypes must be provided for use with an ISO C compiler.

int            closedir(DIR *);
DIR           *opendir(const char *);
struct dirent *readdir(DIR *);
int            readdir_r(DIR *, struct dirent *, struct dirent **);
void           rewinddir(DIR *);
void           seekdir(DIR *, long int);
long int       telldir(DIR *);

The code appears to build fine without it, at least on x86_64-apple-darwin18.6.0 and on riscv32-unknown-elf. It would be rather odd (and possibly depending on undocumented functionality?) if this removal thus prevented the code from compiling on any other architectures / in any other environments.

Looks like you're right, thanks for creating this PR. At some point emubd.c was probably using dirent.h, but I guess that's no longer the case.
Interesting to hear this was prevent compilation on other platforms. From your output does this mean it's possible to run the tests on Mac?
Interestingly, this may have been the reason for needing this fix https://github.com/ARMmbed/littlefs/commit/abd90cb84c818a663b584575b019258d01d0065e

Interesting to hear this was prevent compilation on other platforms. From your output does this mean it's possible to run the tests on Mac?

So <dirent.h> is a standardized POSIX header, so MacOS has it. I'm working with the riscv32-unknown-elf toolchain for gcc though, which does not support the same. I imagine when porting toolchains to many new platforms, headers that are more needed for standards-compliance and sophisticated programs (rather than basic low-level programs) tend to be towards the end of the to-do list.

From your output does this mean it's possible to run the tests on Mac?

It should be possible; if it can be run on Linux using only POSIX APIs, I don't see why you wouldn't be able to run them on Mac. That being said, I tried running them and almost all of them passed - test_corrupt.sh fails with printf () being angry about an invalid number (line 112). I'll look into it later and probably send in a PR to get the tests to pass on Mac, because I think the only issues that prevent the tests from completing are bash-isms in the shell script that might be version-challenged.

@geky is there anything else that you think needs changes prior to merging?

Nope! It looks good. Thanks for this!
Just waiting to coalesce a few PRs into a patch release.

I think the only issues that prevent the tests from completing are bash-isms in the shell script that might be version-challenged.

I was just curious, I don't have access to a Mac so I've never tried. Though the fact I'm running bash commands directly does mean there might be problems...

Nope! It looks good. Thanks for this!
Just waiting to coalesce a few PRs into a patch release.

Gotcha. Sounds good!

I think the only issues that prevent the tests from completing are bash-isms in the shell script that might be version-challenged.

I was just curious, I don't have access to a Mac so I've never tried. Though the fact I'm running bash commands directly does mean there might be problems...

I'd be interested in helping to get this figured out a few weeks out from now if you think it's worthwhile (my work backlog is a little full at the moment); I use a Mac as my main machine for development at work.


DefaultKafkaHeaderMapper$NonTrustedHeaderType encountered with a minimal SCS pipeline

Please see https://github.com/tjormola/scs-problems-demo code demonstrating this problem. As the issue description, I will copy and paste the README here.

This project demonstrates a problem I was having when using the Stream Cloud Stream Kafka Binder with a project written in Kotlin.
The following chain of SCS handlers will throw
org.springframework.messaging.MessageDeliveryException: failed to send Message to channel 'input'; nested exception is java.lang.IllegalArgumentException: Unknown type for contentType header value: class org.springframework.kafka.support.DefaultKafkaHeaderMapper$NonTrustedHeaderType

when bound together using the SCS Kafka Binder. When using SCS Rabbit Binder, it'll work just fine.
Source configuration defined in demo.source.HttpSourceApplication:
@EnableBinding(Source::class)
@Configuration
class HttpSourceConfiguration {
    @Bean
    @StreamEmitter
    @Output(Source.OUTPUT)
    fun publishIncomingJsonPayload(): Publisher<Message<String>> {
        return IntegrationFlows
                .from(WebFlux.inboundChannelAdapter("/")
                        .requestMapping { it
                                .methods(HttpMethod.POST)
                                .consumes(MediaType.APPLICATION_JSON_VALUE)
                        }
                        .requestPayloadType(ResolvableType.forClass(String::class.java))
                        .statusCodeFunction { HttpStatus.OK })
                .toReactivePublisher()
    }
}

Processor configuration defined in demo.processor.ReactiveProcessorApplication:
@EnableBinding(TestProcessor::class)
@Configuration
class ReactiveProcessorConfiguration {
    @StreamListener
    @Output(TEST_PROCESSOR_OUTPUT_CHANNEL_NAME)
    fun processJsonMessage(@Input(Processor.INPUT) input: Flux<String>): Flux<String> {
        return input
                .map { it }
    }
}

Run the tests demo.test.RabbitReactiveIntegrationTest and demo.test.KafkaReactiveIntegrationTest under the integration-tests subproject to trigger the effect. The Kafka test will use an embedded Kafka instance and the Rabbit test expects the default configuration, i.e. Rabbit running on localhost with default guest credentials.
I wonder if Kotlin native types stuff is interfering here somehow. https://docs.spring.io/spring-kafka/docs/2.1.4.RELEASE/api/org/springframework/kafka/support/DefaultKafkaHeaderMapper.html#addTrustedPackages-java.lang.String...- refers to java.util, java.lang as being trusted packages by default. I haven't tried implementing this in plain Java, maybe it would work.
But as Kotlin is now very much liked in Spring world and the same code works with the other official binder implementation, I think this should work with Kafka out of the box, too. Thus I consider this behaviour a bug.

Try adding org.springframework.http (where MediaType lives) to the default header mapper's trustedPackages.
You can provide the bean name of the customized mapper in the binder's headerMapperBeanName property.

We should probably consider adding this to the default list of trusted packages, either in the binder, or in spring-kafka itself.

Thanks a lot for the suggestion. I did this in https://github.com/tjormola/scs-problems-demo/commit/2adb27f755034545fef757b9a19ed8486194eb4f and the DefaultKafkaHeaderMapper$NonTrustedHeaderType exception is now sorted. Unfortunately, now we got a bunch of new exceptions related to header processing and JSON de-serialization in ObjectMapper. I don't fully understand what's happening here without further research, I might have time to debug this later, but here's the console log from the test run in case the problem cause is evident to you with a short glimpse..
https://gist.github.com/tjormola/2639373dbdca5d848f7261317a5da7d9

@tjormola I was able to reproduce it and I believe the issue is that we don't have deserializer for MediaType, only for MimeType. In the controlled environment passing org.springframework.http.MediaType triggers MimeTypeJsonDeserializer and your content type json string gets properly converted to MimeType.
While I didn't follow the discussion you and Gary had so I don't know the context of the conversation, but I am wondering if passing a org.springframework.http.MediaType is something you control.

Hi, I've managed to get this to work with the following configuration/code:

Add Bean allocation (with related object mapper and support for MediaType)

	@Bean
	DefaultKafkaHeaderMapper mapper(){
		ObjectMapper objectMapper = new ObjectMapper();
		SimpleModule module = new SimpleModule();
		module.addDeserializer(MediaType.class, new JackDeserialization());
		objectMapper.registerModule(module);
		DefaultKafkaHeaderMapper defaultKafkaHeaderMapper = new DefaultKafkaHeaderMapper(objectMapper);
		defaultKafkaHeaderMapper.addTrustedPackages("org.springframework.http");
		defaultKafkaHeaderMapper.addTrustedPackages("org.springframework.util.MimeType");
		defaultKafkaHeaderMapper.addTrustedPackages("org.springframework.http.MediaType");
		LOG.info("**** Added trusted packages **** ");
		return defaultKafkaHeaderMapper;
	}

Create custom deserialization

public class JackDeserialization extends StdDeserializer<MediaType> {
	public JackDeserialization() {
		this(null);
	}

	public JackDeserialization(Class<?> vc) {
		super(vc);
	}

	@Override
	public MediaType deserialize(JsonParser jp, DeserializationContext ctxt)
			throws IOException, JsonProcessingException {
		JsonNode node = jp.getCodec().readTree(jp);

		MediaType mediaType = new MediaType("application", "json");
		return mediaType;
	}
}

Don't forget to add the following properties (important for headerMapper)

spring.cloud.stream.kafka.binder.headerMapperBeanName=mapper

Closing this issue.


Add documentation for Email Activity API

Closes #669
Checklist

[X] I have made a material change to the repo (functionality, testing, spelling, grammar)
[X] I have read the [Contribution Guide] and my PR follows them.
[X] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[X] I have added necessary documentation about the functionality in the appropriate .md file
[ ] I have added in line documentation to the code I modified

Short description of what this PR does:

Updates the USAGE.md and USE_CASES.md for the client
Updates the use-cases with references to relevant documentation

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

Hello @reedsa,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


Update packages for new atom version

The packages pathwatcher 0.16.0 and runas 0.5.4 are not supported in Atom 124+ (because of the Chrome 36 and Node 0.11.13 upgrade).

Resolved with above commit.


[STORY][Prototype] As a passport holder, I want to lock my tokens on my stake so that I can stamp on my identity for a grants round

Acceptance Criteria:
 Scenario:  User wants to continue the staking process

GIVEN I am viewing the homepage of The Staker
AND I have token(s) staked successfully in The Staker app
WHEN I click the lock tokens button
THEN the tokens in The Staker app should be locked
AND I get visual confirmation that the tokens are locked
AND I see metadata around the locked tokens (how many tokens, how long they are locked for, etc.)
AND I see next steps prompting me towards Passport to claim the associated stamp

Acceptance Criteria completed with Demo: https://youtu.be/aQYW1hmuH1g

Refer Staking.sol stake function - https://github.com/moonshotcollective/id-staking/blob/dev/packages/hardhat/contracts/IDStaking.sol#L92


Disable on-by-default Firefox privacy protections when running crawls

We should disable whatever privacy protections are enabled by default in Firefox (third-party tracker cookies, fingerprinters, miners, TPL, etc.), to avoid needlessly limiting Privacy Badger's learning.

Related: https://github.com/mozilla/OpenWPM/issues/405


refactor(ingest): Remove auto_workunit

Am looking at the auto_ source helpers for the 0 usage stats aspect emission and saw this was unused. Removing if we can actually do that
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

I'm actually going to start using it in this PR https://github.com/datahub-project/datahub/pull/7912

Ah nice lol. I wish we could get rid of MCEs
Closing because we need this function


feat: remove enqueue_graph routes/methods

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[ ] Yes
[ ] No, because:

Have you updated all relevant documentation?

[ ] Yes
[ ] No

Description
feat: remove enqueue_graph routes/methods
This is totally extraneous - it's almost identical to enqueue_batch.
Related Tickets & Documents

Related Issue: split out from #4686, where it has no business being

QA Instructions, Screenshots, Recordings
This route was used for controlnet image processing and ad-hoc upscaling:

Upscale an image using the ad-hoc upscaling button, ensuring it upscales
Set a control image, ensuring it auto-processes

@maryhipp The corresponding route and session queue method can be removed.


[Feature request] Highlight outer border

I use 2 window v-splits most of the time so the inner window border highlights often don't provide good feedback for me (given a global statusline). Having the outer border highlighted also would solve this.

I use 2 window v-splits most of the time so the inner window border highlights often don't provide good feedback for me (given a global statusline). Having the outer border highlighted also would solve this.

In fact, I didn't find a better display method, any display method will block other components, such as winbar, statusline

I hope you can provide a better display

Maybe the following repo can help as inspiration: https://github.com/b0o/incline.nvim

Maybe the following repo can help as inspiration: https://github.com/b0o/incline.nvim
Also noice.nvim's classic bottom cmdline is able to overlay a floating window on top of the statusline: https://github.com/folke/noice.nvim

Floating window cannot be moved to the winbar and cmdline


Custom loss using intermediate tensorflow probability layer with Eager execution

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes, a colab here.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Locally, Ubuntu 20.04, otherwise current colab.
TensorFlow installed from (source or binary): See colab.
TensorFlow version (use command below): v2.3.0-0-gb36436b087
Python version: Locally, 3.8.2

Describe the current behavior
When using a custom loss based on tensorflow probability DistributionLambda, or a distribution without DistributionLambda, I cannot move into Eager execution. In the first case, the gradients are not propagated properly, so variables say they have no gradients when training, and, in the second case, the layers are symbolic tensors when creating the network, so they cannot be used in the Eager execution loss.
Describe the expected behavior
I understand the problem. What I would like is a way to have the intermediate values within the network to be used in the loss. A regularization loss is not enough, as I need the outputs of the whole network, together with the intermediate layer values in order to compute the loss.
Standalone code to reproduce the issue
A colab.
Other info / logs
Please see the attached colab. A useful piece of information is that I am using a class to create the model because this is part of a bigger project, and the network is much bigger, so while in the example I showed the problem in one of the last layers, this could happen for a layer that is in the middle of the network, and I would not like to add all intermediate layers as outputs of the model.

The only workaround I have found is to add the intermediate layers as outputs of the model and access them in the loss function, although then I have to post process outputs before giving them to further code (this is part of a bigger project). I think that this solution works, but it is definitely what I wanted.

@cserpell
For the error " No gradients provided for any variable" please refer to #27949 #41162 link
For the error " SymbolicException: Inputs to eager execution function cannot be Keras symbolic tensors" #34944 link link1, and let us know.

Reading many issues and questions like those mentioned, the only workaround was adding them as outputs. There is no bug, but it seems no easy way to do it without adding them as outputs.

@cserpell
Please verify and let us know if it helps resolve the issue faced.

As I mentioned, I added the intermediate layers as outputs to the model, and then read the outputs in the loss. Although not what I wanted, it works.

@cserpell
Thank you for your update, can we move this to closed status as issue is resolved.

@cserpell you had the same issue as mine, please take look at this gist Here, i think you'll find what you're looking for.

@cserpell
Please update as per above comment.

My model has probabilistic random layers, so I needed the actual output instead of re evaluations of the layers, so I concatenated the internal values as outputs of the model, and that helps me at evaluation time as well, as I can print the actual internal layers activations. Thanks anyway, I think that also helps.


statespace_limits with Tabular and IncrTabular issue

Hi, I've been testing rlpy already some days. I've encountered a problem when I wanted to put different statespace_limits in my domain. When the values in different dimensions are not the same, it just doesn't work. I tried with RBF and Fourier and I don't have that problem. I add the space limit I use:
 statespace_limits = np.array([[-10., 10.], [-5, 5]])  # not working
 # statespace_limits = np.array([[-10., 10.], [-10., 10.]]) # working

And I add the error trace so you can see it.
Traceback (most recent call last):
  File "test.py", line 103, in <module>
    visualize_performance=False)  # show performance runs?
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Experiments/Experiment.py", line 350, in run
    self.evaluate(total_steps, episode_number, visualize_performance)
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Experiments/Experiment.py", line 445, in evaluate
    total_steps, visualize=visualize > j)
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Experiments/Experiment.py", line 212, in performanceRun
    a = self.agent.policy.pi(s, eps_term, p_actions)
  File "/home/aronnax/dev/rlpy_test/tuto/eGreedyTut.py", line 26, in pi
    b_actions = self.representation.bestActions(s, terminal, p_actions)
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Representations/Representation.py", line 358, in bestActions
    Qs = self.Qs(s, terminal, phi_s)
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Representations/Representation.py", line 170, in Qs
    phi_s = self.phi(s, terminal)
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Representations/Representation.py", line 222, in phi
    return self.phi_nonTerminal(s)
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Representations/Tabular.py", line 31, in phi_nonTerminal
    hashVal = self.hashState(s)
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Representations/Representation.py", line 299, in hashState
    ds = self.binState(s)
  File "/usr/local/lib/python2.7/dist-packages/rlpy/Representations/Representation.py", line 335, in binState
    assert (np.all(s >= limits[:, 0]))
AssertionError

Thanks for reporting. I have trouble reproducing this issue. Can you provide a minimal working example?


Preview button greyed out for Lab 2.2: Create a unified customer profile

Module: Lab 2.2: Create a unified customer profile
Exercise 1 - Unify the data
Task: 04 - Precision
Step: 04
Description of issue
The Preview button is required for steps 4 and 7 but it is showing as greyed out.

The issue might be due to the Matching rules and Deduplications rules are still refreshing. If you wait for the refresh process to complete, the Preview button should become active.
Please see the screenshots, as I can see the preview button is active.

Thank you for raising this issue. We will update on this soon. Thank you for your patience!

Hi,
This issue is not reproducible . I have waited till all the rules are refreshed.
Preview is not greyed out for me.
Refer Screenshot below :

Closing issue as our tester determined that graying out of Preview button was due to some jobs still running. Preview button will appear when all jobs are complete.


Syntax highlight bug when method calls are split onto multiple lines

Describe the bug

Syntax highlighting seem to 'break' when there are method calls chained that are split onto multiple lines
Seems to work with both .fs files in a project & with .fsx files too

Screenshots:

Sample code (same as on the screenshots):
open System.Xml.Linq

XDocument.Load("some_path")
         .Element(XName.Get "Element1")
         .Element(XName.Get "Element2")
         .Elements(XName.Get "Elements") |> ignore

open System.Xml.Linq

let hm1 = XDocument.Load("some_path")
let hm2 = hm1.Element(XName.Get "Element1")
let hm3 = hm2.Element(XName.Get "Element2")
let hm4 = hm3.Elements(XName.Get "Elements")

hm4 |> ignore

Machine info

OS: Windows
VSCode version: 1.52.1
Ionide version: v5.2.0

Seems that this is already a reported issue in #1430, so closing this

Seems that this is already a reported issue in #1430, so closing this


[.NET CORE/ADAPTIVE CARD][Adaptive card returns undefined in Teams] [Adaptive card returns undefined]

Platform
What platform is your issue or question related to? (Delete other platforms).

.NET Core

Author or host
Sending adaptive  card to Teams
Details
I am trying to send an Adaptive Card to Microsoft Teams and it returns undefined in Teams. It is working fine in emulator. I am using c#. Below is the image from emulator and code used to send response to Teams.
Code:
public async Task EventCardMessage(Activity activity, JToken row)
{
    using (var connector = new ConnectorClient(new Uri(activity.ServiceUrl), _config["MicrosoftAppId"], _config["MicrosoftAppPassword"]))
    {
        //adaptive card
        Activity replyToConversation = activity.CreateReply();
        replyToConversation.Attachments = new List<Attachment>();

        AdaptiveCards.AdaptiveCard card = new AdaptiveCards.AdaptiveCard();
        Card adaptiveCard = new Card();
        adaptiveCard.Count = row["total_count"].ToString();
        // Add text to the card.
        card.Body.Add(new TextBlock()
        {
            Text = "There are " + adaptiveCard.Count + " Holidays this Year",
            Size = TextSize.Normal,
            Weight = TextWeight.Bolder
        });

        // Add Columns to card.
        var list = row["card"]["data"]["list"];
        foreach (var li in list)
        {
            adaptiveCard.Name = li["label3"].ToString();
            adaptiveCard.Date = li["label1"].ToString() + " " + li["label2"].ToString();
            adaptiveCard.Time = li["label5"].ToString();
            adaptiveCard.Address = li["label4"].ToString();

            var current = new ColumnSet();
            card.Body.Add(current);
            var column1 = new Column();
            current.Columns.Add(column1);
            var column2 = new Column();
            current.Columns.Add(column2);
            column1.Size = "40";
            AddTextBlock(column1, adaptiveCard.Date, TextSize.Large, TextColor.Accent, false);
            column2.Size = "60";
            AddTextBlock(column2, adaptiveCard.Name, TextSize.Normal, TextColor.Default, false);
            AddTextBlock(column2, adaptiveCard.Address, TextSize.Normal, TextColor.Default, false);
            AddTextBlock(column2, adaptiveCard.Time, TextSize.Normal, TextColor.Default, false);
        }

        // Create the attachment.
        Attachment attachment = new Attachment()
        {
            ContentType = AdaptiveCards.AdaptiveCard.ContentType,
            Content = card
        };
        replyToConversation.Attachments.Add(attachment);

        var reply = await connector.Conversations.SendToConversationAsync(replyToConversation);
    }
}

@banubhooti can you provide the JSON that is outputted to help us debug?
@siduppal are you able to take a stab at this?

@banubhooti ping!

Closing due to lack of information


Add regridding benchmark

Closes #1556

Do we have enough context here to add this to the benchmark post?  If you give me bullet points and an image (if one makes sense) I'm happy to write up words.

@aulemahal: Thanks for the input. I'll add a follow-up issue to look into some of the suggestions for increasing the complexity of this workload.

IIUC, these benchmarks are more geared towards dask ?

It is, but it's also aimed at reflecting real workloads. Would more complex methods also result in more complex computations or just in more complex weight generation?

We do have some code to make the weights generation in parallel but I would say it is still experimental and of limited scope.

Please let us know if we can help with anything from a Dask perspective.
@mrocklin:

Do we have enough context here to add this to the benchmark post? If you give me bullet points and an image (if one makes sense) I'm happy to write up words.

I'll whip something up. At first glance, this benchmark seems to do alright; it's mostly an embarrassingly parallel computation. Performance and # of tasks could probably look better but that's already a lot better than some of the other benchmarks.

Would more complex methods also result in more complex computations or just in more complex weight generation?

Mostly more complex weight generation, which is totally on the ESMF side, so partly in C/Fortran I think. Maybe two very different grids (curvilinear ones for example) and more complex methods would make for weights with more connected nodes, but I don't think this would affect the computation so much.

@slevang, thanks for the additional input! Would you be interested in contributing a benchmark implemented with xarray-regrid?

thx


SelectByAttributes - Unhandled Exception

This issue was imported from CodePlex
jirikadlec2 wrote 2011-03-03 at 01:46
Create a new FeatureSet (FeatureType.Point) and add it as a layer to the map.
In DemoMap main menu, go to Selection - Select By Attribute
An unhandled exception is thrown in SQLQueryControl.cs :
Unable to cast object of type 'System.Data.DataColumn' to type 'DotSpatial.Data.Field'.
This exception occurs for all types of shapefiles.
jirikadlec2 wrote 2011-03-05 at 07:54
This issue only occurs when the FeatureSet is in-memory or when the Shapefiles are opened with OgrVectorProvider.
olivier57 wrote 2011-03-05 at 11:44
I think that this is not the only problem with SelectByAttribute.
http://dotspatial.codeplex.com/discussions/247502

@mogikanin Do you have any idea what function is meant by

In DemoMap main menu, go to Selection - Select By Attribute ?

I tried the Query from the Attribute table editor as well as the Find button in the toolbar in the main window on a temporary layer (create with shape editor) without causing an error. If one of those was meant I'd say the error no longer exists.

@jany-tenaj yes I think they meant selection inside Attribute Table Editor or inside outdated window SelectByAttributes. Also I see that this SelectByAttributes not used anywhere and can be safely removed.

Ok, I've reused this SelectByAttributes window, now it's functionality available under Selection menu.


Loading image to wrong cells

Hi,
I possibly being quite stupid at this, but I am new to swift.  It loads images to wrong cells after scroll or refresh.
Would you provide some suggestion on verifying this? I have attached some of my code here.
Thanks in advance.
import UIKit
import OAuthSwift
import SwiftyJSON
import Kingfisher

class PbTCollectionViewController: UICollectionViewController {

    var json = JSON.null

    override func viewDidLoad() {
        super.viewDidLoad()
        loadPublicTimeline()
    }

    func loadPublicTimeline() {
        let URL = "http://api.fanfou.com/statuses/public_timeline.json"
        let urlParams = [
            "count": 20
        ]

        oauthswift.client.get(URL, parameters: urlParams,
                              success: {
                                data, response in
                                let jsonDict: AnyObject! = try? NSJSONSerialization.JSONObjectWithData(data, options: [])
                                self.json = JSON(jsonDict)
                                self.collectionView?.reloadData()
            }, failure: { error in
                print(error)
        })
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }

    override func numberOfSectionsInCollectionView(collectionView: UICollectionView) -> Int {
        return 1
    }

    override func collectionView(collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return self.json.count
    }

    override func collectionView(collectionView: UICollectionView, cellForItemAtIndexPath indexPath: NSIndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCellWithReuseIdentifier("PbTCell", forIndexPath: indexPath) as! PbTCollectionViewCell
        cell.lbContent.text = self.json[indexPath.row]["text"].stringValue
        cell.imgInPost.kf_setImageWithURL(NSURL(string: self.json[indexPath.row]["photo"]["imageurl"].stringValue)!)
        return cell
    }

}

How did you solve it?


Implement catalog functionality

Requirements.

Taking input configuration from user for each catalog.
Configuration should cover all the required details to connect to the catalog instance.
Securely store any confidential data mentioned in the configuration.

Solution Overview (Tech tasks)

Question:

Is there any DML support for catalog? e.g. CREATE CATALOG catalog, SHOW CATALOGS. Does user could create catalog dynamically?
Does the default catalog mapping to the current OpenSearch cluster?
What auth type is support for CATALOG? Basic Auth? Access-Key/Id?

Will include all of these into separate tasks.


GH-39488: [Ruby] Add support for ChunkedArray in Ractor

Rationale for this change
We can't use @cache ||= build_cache idiom in Ractor because Ractor requires that shared objects are immutable.
What changes are included in this PR?
Compute caches before making ChunkedArray immutable.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.

Closes: #39488

+1


Issues with environment variables and Angular2+

From @rafaelcmrj on August 21, 2018 20:32

Did you verify this is a real problem by searching the NativeScript Forum and the other open issues in this repo?
Yes
Tell us about the problem
I'm using Angular2+ and the environments file isn't loading the production file when using --release. I've tried --env.prod, --env prod, --env=prod and many other options. None of them worked.
Which platform(s) does your issue occur on?
Both
Please provide the following version numbers that your issue occurs with:

CLI: 4.1.2
Cross-platform modules: 4.1.1
Runtime(s): ios 4.1.0 / android 4.1.3

Please tell us how to recreate the issue in as much detail as possible.
Any app using Angular2+ and environment files.
Is there code involved? If so, please share the minimal amount of code needed to recreate the problem.
The environment files are located under /app/environments/enviroment.ts and /app/environments/enviroment.prod.ts
Copied from original issue: NativeScript/NativeScript#6206

@rafaelcmrj I believe you can achieve this by tweaking your project's webpack.config.js and environment.*.ts, if you are using @nativescript/schematics in your project.
I have written an article to outline steps required to utilize environment.*.ts in NativeScript Angular's project in mobile and/or web platforms: https://medium.com/@derekfong/nativescript-angular-setup-environment-ts-for-different-environments-631b4c5219b6


add acmeHttp01SolverImage

Pull Request Motivation
When using a custom registy the values.yaml will allow to specify a custom image (e.g. selfhosted registry).
But the acmesolver can not be specified itself.
When there is a registry without connection to internet (and thus to quay.io) the pull of the image will fail.
To be able to specify the image for the acmesolver pod I've extended the chart
Also see the problem described in discussion #4949.
If you prefer other naming in values than suggested acmeHttp01SolverImage feel free to suggest another key.
Kind
feature
Release Note
Adding support for specify custom acme HTTP01 solverimage in Helm chart.

When using a custom registy the values.yaml will allow to specify a custom image (e.g. selfhosted registry).
But the acmesolver can not be specified itself.

Thank you for your contribution however it is already possible to configure this and other flags via Helm chart values today using the extraArgs field. I think we don't want to  make it confusing by introducing two separate ways how to configure the same flag via values.yaml.

I understand your argument to use an extraArgs field to keep the values.yaml not that complex.
For me there is nevertheless an added value if the registry can be changed:
When using the Helm chart as subchart I want to depend on the Helm chart version also for the solverimage (only with changed registry - not tag).
By using a extraArgs I would have to specify the version of cert-manager twice (Chart dependency and subchart values configuration).
This could lead to the version only being updated in one place.
Maybe you have an other solution for this issue?

Is there any update yet?

This does not fix the issue, that there is still no imagePullSecret defined in the generated pod spec and thus the images still won't get pulled!
Am I right, that this is something that would have to be configured inside the issuer https://cert-manager.io/docs/configuration/acme/http01/#configuring-the-http01-ingress-solver ?

@irbekrm is there is a new awareness for this PR?

/assign @jakexks

duplicate of https://github.com/cert-manager/cert-manager/pull/5554
(this PR was first, but #5554 has a better/ more complete implementation)

duplicate of https://github.com/cert-manager/cert-manager/pull/5554
(this PR was first, but https://github.com/cert-manager/cert-manager/pull/5554 has a better/ more complete implementation)


v3 and tailwind v1.3 causes error

I get a strange require() is not a function error, when installing v3 alongside tailwindcss v1.3:
./src/css/main.css
Module build failed (from ./node_modules/mini-css-extract-plugin/dist/loader.js):
ModuleBuildError: Module build failed (from ./node_modules/postcss-loader/src/index.js):
TypeError: require(...) is not a function
    at Object.<anonymous> (/[path]/tailwind.config.js:310:37)
    at Module._compile (internal/modules/cjs/loader.js:1158:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1178:10)
    at Module.load (internal/modules/cjs/loader.js:1002:32)
    at Function.Module._load (internal/modules/cjs/loader.js:901:14)
    at Module.require (internal/modules/cjs/loader.js:1044:19)
    at require (internal/modules/cjs/helpers.js:77:18)
    at /[path]/node_modules/tailwindcss/lib/index.js:71:107
    at /[path]/node_modules/tailwindcss/lib/processTailwindFeatures.js:32:20
    at LazyResult.run (/[path]/node_modules/postcss/lib/lazy-result.js:295:14)
    at /[path]/node_modules/webpack/lib/NormalModule.js:316:20
    at /[path]/node_modules/loader-runner/lib/LoaderRunner.js:367:11
    at /[path]/node_modules/loader-runner/lib/LoaderRunner.js:233:18
    at context.callback (/[path]/node_modules/loader-runner/lib/LoaderRunner.js:111:13)
    at /[path]/node_modules/postcss-loader/src/index.js:208:9

Any idea why this happens? (v2.x works fine).
Thank you!

Hey @rowild I'm not able to reproduce this. Can you please share your config, or (if you have time) even better, give me access to a simple repo that reproduces the issue?

Same here with<EMAIL_ADDRESS>

@Javid-Izadfar I still cannot reproduce this, but if you give me access to a codebase where you have this error, I can take a look and hopefully fix.

this is my tailwind.config.js:
module.exports = {
  purge: [],
  theme: {},
  variants: {},
  plugins: [
    require('tailwindcss-gradients')(),
  ],
}

with<EMAIL_ADDRESS>coupled with<EMAIL_ADDRESS>I'm on windows 10 running<EMAIL_ADDRESS>sadly, I can't give you access to codebase right now.

@Javid-Izadfar Can you please try literally any other Tailwind plugin instead of this one and see if it works?

One of my errors seems to have been the plugin inclusion:
plugins: [
  // wrong
  require('tailwindcss-gradients')()
  // right
  require('tailwindcss-gradients')
]

(Has this by any chance been the case with a very early tailwindcss version?)
However, "gradients" still do not work, while your "tailwindcss-animations" work fine. I prepare a repo here:
https://github.com/rowild/tailwindcss-plugin-tests

Hey @rowild, thank you so much for creating that repo. Unfortunately I cannot reproduce the issue, both plugins work fine here (tried both npm run dev and npm run build):

I tried with the latest versions of node and npm (14.4.0 and 6.14.5 respectively) as well as versions 11.15.0 and 6.7.0 (using nvm). No issues at all. :/ I'm running macOS, not sure if that changes anything?

Look at that! – I set my whole node environment again (also on Mac, also nvm, with node 11, 12, 14), and look at that again: it works! No idea, what I have messed up! But as long as I know how to get it going again, everything is fine.
Closing this issue. Thank you for your feedback and help! Stay safe!

You too!


fix url to Dockerhub

The URL to the Dockerhub is not working, fix the URL.

I'm accepting it, but cross-compiler is not used in the latest version, so I will remove that line.


Can't open RAW file ?

Hi, this is a nice work. I want to ask some questions about the datasets that the raw file, e.g. static/raw/test_1.zip, can't open in Metavision Studio. I got some error information, as follows:

I really hope you can help me to solve the problem. Thanks!

Hi @qsdsatom, is this the only .raw file that you are facing an issue opening or are there others as well?

Thank you for your quick reply! I found the same error in test_1.zip, test_2. zip and train1. zip files.

Could you please try downloading the .zip files again? It’s possible that the folder got corrupted either during the download process or while it was being unzipped.

Thanks for your reply! I downloaded the .zip file again and unzipped it, but it still can't open in the Metavision studio. Perhaps this file does not need to be opened by the studio toolkit?

Are you able to stream the data using Metavision SDK instead of studio?

I would also like to know how to open a .raw file？


[$1000] Message with button gets cut off early for certain length of emails

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open ND on android
Click on FAB > split bill > enter an amount
Share with account emails which are not renamed (for easier reproduction use these emails for testing<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Split the bill
Click on the IOU card
Select one of the contacts from the side panel
Observe the Message with button

Expected Result:
Message with button should not be cut off early
Actual Result:
Message with button gets cut off early.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.47-1
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/3730e72e-d70f-4245-9c80-9c7e3f37d3b6

Expensify/Expensify Issue URL:
Issue reported by: @nathan-mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690444037073499
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013be117aa6991d578
Upwork Job ID:<PHONE_NUMBER>802149888
Last Price Increase: 2023-07-28

Proposal
Please re-state the problem that we are trying to solve in this issue.
Message with button gets cut off early for certain length of emails
What is the root cause of that problem?
ther is no style flex1 on title which is important for android when using ellipsis to take the full space available.
What changes do you think we should make in order to solve the problem?
add style styles.flex1 to titleTextStyle here.
https://github.com/Expensify/App/blob/975146fa30bc65a19df8e0cca525af32ce2ceda5/src/components/MenuItem.js#L81
What alternative solutions did you explore? (Optional)
we can pass style styles.flex1 in props.titleStyle if we need to support this case only

Confirmed on browserstack

Seems like a regression to me, couldn't find any. It's hard to reproduce, needs a proper root cause here why it occurs thanks!

Proposal
Please re-state the problem that we are trying to solve in this issue.
A message with a button gets cut off early for a certain length of emails
What is the root cause of that problem?
In React Native, the Text component does not automatically take the full available space within its parent container by default. This behavior is different from web-based HTML elements, where elements such as <div> automatically take the full width of their parent container.
The reason why the Text component doesn't take full space is because of its inherent behavior as a text-rendering component. The width of the Text component is determined by the size of its content, and it will wrap to the next line if the content exceeds the available width. The default width of a Text component is essentially based on the width required to display its content without overflowing.
If we want the Text component to take full space or expand to fill the available width of its parent container, we need to explicitly specify a width style for the Text component or use flexbox properties to control its dimensions.
What changes do you think we should make in order to solve the problem?
 1) using Flexbox `flex:1`
 2) Using Fixed Width:  `width:100%`

What alternative solutions did you explore? (Optional)

@Santhosh-Sellavel can you take a look at this root cause analysis today?

Left a comment here for volunteers to take over

I found if we add flex1 or flex:1 to MenuItem.js will create another issue.
https://github.com/Expensify/App/blob/975146fa30bc65a19df8e0cca525af32ce2ceda5/src/components/MenuItem.js#L81
Before adding flex1

After adding flex1, It will move the Check icon to the end of the row.

Hey @Ollyws - good to have you! Can you take a look at the root cause proposal here when you get a chance and let us know what you think?

Proposal
Updated

@Ollyws PR is now ready for review.

@Ollyws have you started a discussion around regression or created a regression test? Once you do, I'll process payment on this issue.

@Ollyws are you getting paid via NewDot or Upwork?

Paid via Upwork. Closing!


SASS source

Is there any plan to generate this from SASS? I am finding situations where, for instance, I wish to import a particular animation into a style for a pseudo-element because I can't apply a class to it. Having SASS partials would make these fringe cases possible.

Please see #277.


oe-gdb often does not display symbols

Illustration below, on a program started inside oe-gdb. When attaching (oe-gdb -p) or loading core files (oe-gdb -c), we never see any symbols. This makes debugging quite difficult.

$ objdump --debugging coco | wc -l
420375
$ objdump --debugging libsimplebank.signed.so 2>/dev/null | wc -l
5246662

$ /opt/openenclave/bin/oe-gdb ./coco
GNU gdb (Ubuntu 7.11.1-0ubuntu1~16.5) 7.11.1
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
<http://www.gnu.org/software/gdb/documentation/>.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Source directories searched: /opt/openenclave/lib/openenclave/debugger/gdb-sgx-plugin:$cdir:$cwd
No symbol table is loaded.  Use the "file" command.
No symbol table is loaded.  Use the "file" command.
No symbol table is loaded.  Use the "file" command.
Setting environment variable "LD_PRELOAD" to null value.
Reading symbols from ./coco...done.
(gdb) set args libsimplebank.signed.so raftelectiontimeout=100000 rafthost=<IP_ADDRESS> raftport=36735 tlshost=<IP_ADDRESS> tlsport=37750 log=0.log nodecert=0.pem enclavetype=simulate raftloglevel=4
(gdb) run
Starting program: /tmp/v-amcham_0/coco libsimplebank.signed.so raftelectiontimeout=100000 rafthost=<IP_ADDRESS> raftport=36735 tlshost=<IP_ADDRESS> tlsport=37750 log=0.log nodecert=0.pem enclavetype=simulate raftloglevel=4
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
[New Thread 0x7ffff5e9c700 (LWP 9893)]
[New Thread 0x7ffff569b700 (LWP 9894)]
[New Thread 0x7ffff4e9a700 (LWP 9895)]
[New Thread 0x7ffff4699700 (LWP 9896)]
[New Thread 0x7ffff3e98700 (LWP 9897)]
[i]Starting new node.
[~]Creating new node.
[i]tls server bound to <IP_ADDRESS>:37750
[i]raft server bound to <IP_ADDRESS>:36735
[~]Receiving new TLS connection.
[~]new TLS connection:<PHONE_NUMBER>26040
[~]tls in:<PHONE_NUMBER>26040, size 399

Thread 1 "coco" received signal SIGSEGV, Segmentation fault.
0x00007fffe0286c71 in ?? ()
(gdb) bt
#0  0x00007fffe0286c71 in ?? ()
#1  0x00000020e0510c30 in ?? ()
#2  0x00007fffe052cbd8 in ?? ()
#3  0x00007fffe027833b in ?? ()
#4  0x00007fffe0278475 in ?? ()
#5  0x00007fffe0282afb in ?? ()
#6  0x00007fffe0268ad3 in ?? ()
#7  0x00007fffe0220a49 in ?? ()
#8  0x00007fffe0222d96 in ?? ()
#9  0x00007fffe02269e8 in ?? ()
#10 0x00007fffe0217b17 in ?? ()
#11 0x00007fffe0217b83 in ?? ()
#12 0x00007fffe0098190 in ?? ()
#13 0xa9f86694cbd3ddd2 in ?? ()
#14 0xffffffffffffffff in ?? ()
#15 0x0000000000000018 in ?? ()
#16 0x00007fffe052cb88 in ?? ()
#17 0x00007fffe052ca50 in ?? ()
#18 0x00007fffe052cba0 in ?? ()
#19 0x00007fffe892b860 in ?? ()
#20 0x00007fffe027849c in ?? ()
#21 0x0000000000000018 in ?? ()
#22 0x0000000000000001 in ?? ()
#23 0x00007fffe052cb88 in ?? ()
#24 0x0000000000000018 in ?? ()
#25 0x00007fffe892b890 in ?? ()
#26 0x00007fffe0282afb in ?? ()
#27 0x0000000000000018 in ?? ()
#28 0x0000000000000000 in ?? ()

The log for this issue indicates that the issue could have been observed with simulation mode:
(gdb) set args libsimplebank.signed.so raftelectiontimeout=100000 rafthost=<IP_ADDRESS> raftport=36735 tlshost=<IP_ADDRESS> tlsport=37750 log=0.log nodecert=0.pem ** enclavetype=simulate** raftloglevel=4
Originally oe-gdb did not support simulation.
Support for simulation mode was added in #1099
In release mode, we are not able to reproduce this issue at all.
Closing this.
Feel free to create a new issue if you encounter this again.


Position history

It 's a simple question but i don't found the answer in the documentation, in the dashboard server or in github.
I would like to have the history positions of a device (for example the last 10 positions).  Is there a solution with the API or with the server dashboard?
Thanks for your help

Is it possible to use the sames query params with GET /api/v1/location/FAMILY/DEVICE ??

Which query parameters do you refer?

Hey, i also would like to have a history of my device, but i can't find this data neither in dashborad, neither via api. Do you have some tips?


fix(minimal-slp-wallet): Using v3 of minimal-slp-wallet

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add check for FreeBSD to specify location for doctest

doctest isn't found on FreeBSD without specifying the directory it lives in. I don't know if this is required in other environments, so the if statement can be expanded as necessary.
For this to work on FreeBSD, you will need to the doctest package (devel/doctest).
This addresses: https://github.com/berthubert/simplomon/issues/5

@nreilly are you considering doing a port for this? if not, I'll be happy to do so

@nreilly are you considering doing a port for this? if not, I'll be happy to do so

I was considering, but have never done one and happy to leave it to someone more experienced.

I wonder if we are "holding it wrong" here. The solution may work but it feels like we're solving the problem from the wrong end. https://github.com/mesonbuild/meson/issues/4468 also has some discussion.

From the issue mentioned above "Now this is not a blocker for me right now, because it's easily worked around by pkg install pkgconf" - might that help?

Tested on Freebsd 14, the following makes meson setup build/ work:
pkg install git meson pkgconf lua53
but fails later on for reasons I don't know:
root@freebsd:~/simplomon # meson compile -C build
INFO: autodetecting backend as ninja
INFO: calculating backend command to run: /usr/local/bin/ninja -C /root/simplomon/build
ninja: Entering directory `/root/simplomon/build'
[1/11] Compiling C++ object simplomon.p/netmon.cc.o
FAILED: simplomon.p/netmon.cc.o
c++ -Isimplomon.p -I. -I.. -I../subprojects/nlohmann_json-3.11.2/single_include -I../subprojects/fmt-10.1.1/include -Isubprojects/cpp-httplib-0.13.1 -I../subprojects/cpp-httplib-0.13.1 -Isubprojects/simplesockets -I../subprojects/simplesockets -I/usr/local/include/lua53 -I/usr/local/include -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -std=c++20 -O0 -g -pthread -DCPPHTTPLIB_OPENSSL_SUPPORT -DCPPHTTPLIB_ZLIB_SUPPORT -MD -MQ simplomon.p/netmon.cc.o -MF simplomon.p/netmon.cc.o.d -o simplomon.p/netmon.cc.o -c ../netmon.cc
In file included from ../netmon.cc:6:
In file included from ../simplomon.hh:6:
In file included from ../notifiers.hh:3:
../sol/sol.hpp:14541:32: warning: unknown warning group '-Wmaybe-uninitialized', ignored [-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wmaybe-uninitialized"
                               ^
../sol/sol.hpp:17294:32: warning: unknown warning group '-Wmaybe-uninitialized', ignored [-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wmaybe-uninitialized"
                               ^
[2/11] Compiling C++ object testrunner.p/notifiers.cc.o
In file included from ../notifiers.cc:1:
In file included from ../notifiers.hh:3:
../sol/sol.hpp:14541:32: warning: unknown warning group '-Wmaybe-uninitialized', ignored [-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wmaybe-uninitialized"
                               ^
../sol/sol.hpp:17294:32: warning: unknown warning group '-Wmaybe-uninitialized', ignored [-Wunknown-warning-option]
#pragma GCC diagnostic ignored "-Wmaybe-uninitialized"
                               ^
2 warnings generated.
ninja: build stopped: subcommand failed.

The new code in netmon.cc assumes linux netinet headers/structures and won't compile on FreeBSD without more significant porting than I'm able to do.


Why is the same value as 4 of the mapper appears to have some NES can be simulated and some can not be simulated?

Why is the same value as 4 of the mapper appears to have some NES can be simulated and some can not be simulated?

Mapper 4 includes both MMC3 and MMC6. After a quick scan of the code, I am unsure if jsnes currently supports the 1KB internal PRG RAM for MMC6 which could be the source of your problems.
(http://wiki.nesdev.com/w/index.php/MMC6)


Fix unix tests

Noticed https://github.com/http-rs/tide/pull/539 was failing, which seems caused by #531 not having been updated for master. This patch fixes that. Thanks!

Fixes the CI failure on master -- merging!


Expose response handler and update attributes' access modifiers

Make ResponseHandler attributes protected for extendability.
Add optional "clientId" param to generateCacheRecord.
Expose "CacheRecord" and "ResponseHandler" as part of msal-browser "internals".

Codecov Report

Merging #6284 (30d0842) into dev (81d34b4) will decrease coverage by 7.79%.
Report is 226 commits behind head on dev.
The diff coverage is 18.96%.

Flag
Coverage Δ

msal-angular
?

msal-browser
?

msal-common
82.92% <ø> (-1.63%)
:arrow_down:

msal-core
?

msal-node
79.29% <ø> (-4.10%)
:arrow_down:

msal-node-extensions
40.61% <18.96%> (-35.03%)
:arrow_down:

msal-react
94.24% <ø> (-0.45%)
:arrow_down:

node-token-validation
?

Files Changed
Coverage Δ

...ions/msal-node-extensions/src/utils/Environment.ts
20.51% <0.00%> (-22.08%)
:arrow_down:

...l-node-extensions/src/broker/NativeBrokerPlugin.ts
3.73% <3.73%> (ø)

...node-extensions/src/persistence/FilePersistence.ts
51.94% <20.00%> (-21.83%)
:arrow_down:

.../msal-node-extensions/src/error/NativeAuthError.ts
28.57% <28.57%> (ø)

...msal-node-extensions/src/lock/CrossPlatformLock.ts
70.73% <33.33%> (-10.35%)
:arrow_down:

...-extensions/src/persistence/KeychainPersistence.ts
66.66% <37.50%> (-21.57%)
:arrow_down:

...extensions/src/persistence/LibSecretPersistence.ts
72.22% <40.00%> (-16.02%)
:arrow_down:

...c/persistence/FilePersistenceWithDataProtection.ts
59.45% <47.05%> (-21.63%)
:arrow_down:

...node-extensions/src/persistence/BasePersistence.ts
64.28% <50.00%> (-2.39%)
:arrow_down:

...tensions/src/persistence/PersistenceCachePlugin.ts
71.05% <55.55%> (-4.63%)
:arrow_down:

... and 8 more

... and 217 files with indirect coverage changes


Update name & icon

Since our project's name changed, all references to GreenNav must be changed to nav-e. The icon and color scheme are also subject to change

I'd like to work on this.

We already got an icon, I'll link that :)

For references:

Which one would you like me to inculcate?

B2 looks best to me :) @bfmags even provided svgs

Working on it.. will update my PR

@maxlorenz Kindly check the PR


[MOB-21298] Make some required fields as optional for ICON template

Making some required fields as optional for Icon templates
Description

Five Icon template does not need title and body fields, but they as treated as required field in AEPPushTemplate (Base class) and later in the flow, AEPPushNotificationBuilder.construct method uses the field. So in case of icon template we have made this as empty field.
Related Issue
https://jira.corp.adobe.com/browse/MOB-21298

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[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 change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

The changes look good to me overall. Can we add a couple of tests verifying that a multi-icon template can be created if the data payload does not have title or body KVP?

Can we open this pr against https://github.com/adobe/aepsdk-ui-android/tree/dev-v3.0.1 instead as 3.0.0 was released?


security: upgrade to go 1.20.5

Python SDK failures are unrelated.  We are addressing those in a followup.


jquery ui js and css

Taso,
1.11.4 of  wp-includes/js/jquery/ui/widget.min.css is not forward compatible with jquery 1.12 in js_libs/jquery
action: plz change  jquery ui js and css libraries to version 1.11.4 in js_libs/jquery/

Implemented loading of WordPress's integrated jQuery UI library to avoid future conflicts


Jump to definition does not work with import qualified as

Jump to definition does not work with import qualified as, pressing M-. does not nothing.
For example, I have a function doSomething in the Lib module, and in the Main module, if I do
import Lib
and do
doSomething
Pressing M-. on doSomething will jump to the definition in the Lib module.
If I do
import qualified Lib
and do
Lib.doSomething
It also works. But if I do
import qualified Lib as L
and do
L.doSomething
Then jump to definition does not work anymore. But if I first do
import qualified Lib
and then change to
import qualified Lib as L
Then it still works.

Similar issue as mentioned in #312


Add possibility to profile Natalie's compiler using StackProf

This patch adds two new options to bin/natalie:
--transform-only - Only transform the given ruby code to cpp code without writing it to a file. This is useful for measuring the time needed by the compiler to transform ruby code to cpp.
-pc - Profiles the compiler using the stackprof gem. When speedscope is installed, the profile will be opened using speedscope!
Let me know what you think. We could definitely add more here. I'm thinking profiling the different passes for example.

This is awesome!
What do you think about adding these new options under the -d debug flag. Seems more tidy, yeah?

What do you think about adding these new options under the -d debug flag. Seems more tidy, yeah?

Hm... I feel like the transform-only option would be a perfect fit for the debug flag. However, I don't feel like the profiling would fit there? It would make it hard to add more customizability to the profiler (for example only profile Pass1). I initially thought about integrating it into the -p profiling flag so we would do something like bin/natalie -p compiler spec/core/numeric/step_spec.rb. What do you think about this?

I initially thought about integrating it into the -p profiling flag

Ahh, yes, that sounds even better. Let's do that!

Ok I changed the options. What do you think?

Great!


Create a model to save applicant solicitor details

Before creating a pull request make sure that:

[x] commit messages are meaningful and follow good commit message guidelines
[x] README and other documentation has been updated / added (if needed)
[x] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-1697
Change description

Capture Civil Service Claimant and Defendant information.

Set GA Applicant and Respondent details and their organization details

Related PR
https://github.com/hmcts/civil-service/pull/824
https://github.com/hmcts/civil-general-applications/pull/162
https://github.com/hmcts/civil-ccd-definition/pull/650
Does this PR introduce a breaking change? (check one with "x")
[x] Yes
[ ] No

functional test is failing for random reason so merging to sprint5integration


Add a sample of calling WMI from FFI

This example requires a good range of COM objects and calls to complete successfully:
https://docs.microsoft.com/en-us/windows/win32/wmisdk/example-creating-a-wmi-application

Experimenting with this in the wmi branch.


Segfault after mounting a partition 128 times.

You can mount a partition 127 times.
The 128th try will cause a SEGFAULT in lkl_disk_add .
More precisely in lkl_sys_virtio_mmio_device_add
with

dev->base being negative (it is calculated in register_iomem and overflows)
dev->irq = 129

This is problematic because node-lkl calls lkl_disk_add for each partition it mounts (even one on a disk which already has a partition mounted) and never calls lkl_disk_remove
I've added a test that causes this segfault in the segfault-on-128th-mount branch

This is partially fixed by #29.
But we still can't attach more than 128 disks at the same time.


(maint) Add run-as option to developer docs

This adds the 'run-as' option example to bolt-server's developer docs

Waiting for CLA signature by @lucywyman
@lucywyman - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/
Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

Also just notice that tmpdir is listed twice in https://github.com/puppetlabs/bolt/blob/master/developer-docs/bolt_server.md#winrm-target-object

I played around with adding "Only available for SSH / WinRM" targets to the transport specific options, but had a hard time finding a way to do it that wasn't clunky + way too many words for too little information. I thought about adding little '*' next to transport-specific options, but I think that'll just be confusing. Ultimately it seems like having the options under different headings is probably enough?


Point cloud modified when depth frame aligned

Required Info

Camera Model
D415

Operating System & Version
Ubuntu 18

Platform
PC

SDK Version
2.23

Language
C++

Issue Description
The point cloud calculated from the depth map, using the rs2::pointcloud class, is different when the depth frame is aligned to the color frame (using rs2::align). The points are shifted in x and y by a non constant value. z values remain the same.
Is this expected behavior? What is the cause of this difference?

@descamps2 , that's correct.
The rs2::pointcloud 's origin coordinate (x,y,z=0,0,0) is defined as the position of the left IR sensor, while the origin of the pointcloud produced from "Depth aligned to color" is the position of the RGB sensor.
Additionally , in the process of alignment the reprojected depth values are clamped to the RGB grid (aliasing), which adds up to the bias.

Ok, thanks for the clarification!


🇬 refactor: Update default Google Models and Parameters

Summary
Gemini models default to different parameter settings. Eg. the latest models have temperature from 0-2 with default of 1. Max output defaults to 8192, top_p defaults to 0.95. The default model and parameters are for chat-bison which is pretty much a deprecated model that no one uses. My suggestion is to change the default model along with the default parameters to be more in line with the current offerings. Most LibreChat users will use aistudio instead of Vertex anyway, where bison models would be irrelevant.
This PR changes the default model from chat-bison to gemini, as well as updates the default parameters. I also updated the .env.example with the latest / greatest model tags as of 05/18. This should result in a better default experience for most users. I also removed the the PALM / bison family of models from .env.example as they have been mostly superseded by Gemini.
The resulting code is more consistent with the other providers, and would be easier to maintain moving forward. Incoming PR to update documentation as well. https://github.com/LibreChat-AI/librechat.ai/pull/30
Change Type
Please delete any irrelevant options.

[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)
[ ] This change requires a documentation update
[ ] Translation update

Testing
Please describe your test process and include instructions so that we can reproduce your test. If there are any important variables for your testing configuration, list them here.
Test Configuration:
Checklist
Please delete any irrelevant options.

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] I have written tests demonstrating that my changes are effective or that my feature works
[x] Local unit tests pass with my changes
[x] Any changes dependent on mine have been merged and published in downstream modules.
[x] A pull request for updating the documentation has been submitted.

thanks!


Additional Tests

[ ] Copy ThriftTest into this project. This will be used to test various language primitives.
[ ] Copy StressTest into this project. This will be used for stress-testing Thrift in Zephyr.
[ ] Choose a subset of upstream C++ tests and implement them in ZTest within this project
GanttStart: 2022-06-20
GanttDue: 2022-06-30

In upstream Thrift, test/cpp/src/TestServer.cpp does the majority of the checks for the ThriftTest.thrift service (thereby exercising code paths for a large number of language features).

Upstream application tests actually decide which low-level transports or protocols to use at runtime.
https://github.com/apache/thrift/blob/master/test/cpp/src/TestServer.cpp#L621
However, we can use testcase.yaml to do this for us using C preprocesor defines.
https://github.com/zephyrproject-rtos/zephyr/blob/main/tests/net/socket/tcp/testcase.yaml


tiltfile: remove k8s resource assembly v1

We don't have to merge this now, just was in the code and thought that I might as well queue up the PR for when we decide to delete it.

@nicks should we merge this now or wait for right before v1?

actually, i think we should add a deprecation warning before we remove this. We still have it in some of our own demo repos, cf https://github.com/windmilleng/servantes/blob/master/Tiltfile

@nicks actually it is already deprecated https://github.com/windmilleng/tilt/blob/master/internal/tiltfile/k8s.go#L886:L886

@jazzdan Ya, that confused me too! But that's not the right warning. That warning shows up only if you're on version 1. It doesn't show up if you're on version 2. Since we're removing the function - even for version 2 users - we need to have a deprecation warning for that

another option would be to just remove the implementation, but make the warning something like "this function is now a no-op, you can remove it"

Going to close this. Too many merge conflicts and we want to deprecate it first anyways.


(vcredist2005) Import and update to newest version

Description

imported vcredist2005 from https://github.com/MarkRobertJohnson/ChocolateyPackages
cleaned up metadata and install script
updated to the latest version

Motivation and Context

The existing package on chocolatey.org started failing because Microsoft removed the (outdated) installers used by that package version.
How Has this Been Tested?

Manually, in a Windows 10 2004 VM.
Screenshot (if appropriate, usually isn't needed):
Types of changes

[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 change)
[x] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[x] My change requires a change to documentation (this usually means the notes in the description of a package).
[x] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

Original Location

Original Repository
[ ] Include the link to the opened PR that removes the package from the original location
[ ] The migration guidelines have been followed *

:white_check_mark: Package verification completed without issues. PR is now pending human review

Thank you for the review, all good points (and I learned something about AU behavior :-) )
Just out of curiosity, what is the reason for including the package id in tags?

Just out of curiosity, what is the reason for including the package id in tags?

TBH, I don't really remember if there is a reason or just common practice.
There could be something regarding searching, but I really don't know/remember.

:white_check_mark: Package verification completed without issues. PR is now pending human review

The ($env:chocolateyForceX86 -ne $true) condition should probably also be added to vcredist2008 and vcredist2010, right?

@jberezanski your changes have been merged, thanks for your contribution 👍


Cannot read property 'resolved' of undefined when removing

I am debugging some issues with LazyHydrate and I stumpled upon something I really can't explain.
I have a piece of code like so:
<LazyHydrate :trigger-hydration="hydrateProductList">

		<template v-for="(chunk, chunkIndex) in chunkedProducts">

 { pageBreakVisible(isVisible, entry, chunkIndex) } }"
				class="page-break" />

= (chunkedProductsWithoutSkeleton.length - chunksPerFetch)) }" class="columns">
				<template v-for="(product, index) in chunk">

 1)" class="skeleton-detector" />
					<ProductDisplay
						v-if="product.title || product.isSkeleton"
						:key="chunkIndex+index"
						:product="product"
						:hydrate="page > 1 || ((searchQuery) ? true : false)"
						@productClicked="saveLastClickedProduct(product, determinePageFromChunk(chunkIndex))"
						:showLabels="true"
						:trackingList="'ProductList ' + getListHeader" />
					<QouteDisplay v-if="product.qoute" :material="materials[0] || false" :key="chunkIndex+index" />
				</template>

		</template>

</LazyHydrate>

And I wish to remove the "LazyHydrate" components wrapping the whole section. The code works when the <LazyHydrate> component is present, however, when I remove them, I get the following error:
client.js?06a0:97 TypeError: Cannot read property 'resolved' of undefined
    at patchVnode (vue.runtime.esm.js?2b0e:6284)
    at updateChildren (vue.runtime.esm.js?2b0e:6193)
    at patchVnode (vue.runtime.esm.js?2b0e:6319)
    at updateChildren (vue.runtime.esm.js?2b0e:6193)
    at patchVnode (vue.runtime.esm.js?2b0e:6319)
    at updateChildren (vue.runtime.esm.js?2b0e:6193)
    at patchVnode (vue.runtime.esm.js?2b0e:6319)
    at updateChildren (vue.runtime.esm.js?2b0e:6193)
    at patchVnode (vue.runtime.esm.js?2b0e:6319)
    at VueComponent.patch [as __patch__] (vue.runtime.esm.js?2b0e:6482)

Any suggestions?

Without more code I think it's hard to debug. One generic fix I can think of is not lazy-loading the involved components.
Check for
components: {
   ProductDisplay: () => import(/**/),
   QouteDisplay: () => import(/**/),
}

and change it to
import ProductDisplay from 'src/components/whatever';
import QouteDisplay from 'src/components/whatever';

components: {
   ProductDisplay,
   QouteDisplay
}

Because the error could indicate that Vue is trying to render something that is not yet (lazy-)loaded. But the error is fairly generic. So this is just an vague guess.

Interesting @j0Shi82 I think you could be right. Let me check that.

That seemed to do the trick, @j0Shi82


postgres timestamp mapping error: SQL type TIMESTAMPTZ) is not compatible with SQL type TIMESTAMP

Description
I want use postgres timestamp without time zone type in sea-orm
Steps to Reproduce

(in postgres create statement)created_at timestamp without time zone default current_timestamp not null
sea-orm-cli generate entity -o src/tables_codegen
Orders::find().one(&db).await.unwrap().unwrap();

Expected Behavior
The generate code created_at field is
#[derive(Clone, Debug, PartialEq, DeriveEntityModel)]
#[sea_orm(table_name = "orders")]
pub struct Model {
    pub created_at: DateTime,
    ...
}

Actual Behavior
thread 'tokio-runtime-worker' panicked at 'called `Result::unwrap()` on an `Err` value: Query("error occurred while decoding column \"paid_at\": mismatched types; Rust type `core::option::Option<chrono::datetime::DateTime<chrono::offset::utc::Utc>>` (as SQL type `TIMESTAMPTZ`) is not compatible with SQL type `TIMESTAMP`")', src/handler/order.rs:17:56

The generate code created_at field is DateTimeUtc type not DateTime (without time zone)
#[derive(Clone, Debug, PartialEq, DeriveEntityModel)]
#[sea_orm(table_name = "orders")]
pub struct Model {
    pub created_at: DateTimeUtc,
    ...
}

Reproduces How Often
always
Versions
├── sea-orm v0.6.0
│   ├── sea-orm-macros v0.6.0 (proc-macro)
│   ├── sea-query v0.21.0
│   │   ├── sea-query-derive v0.2.0 (proc-macro)
│   ├── sea-strum v0.23.0
│   │   └── sea-strum_macros v0.23.0 (proc-macro)

psql --version
psql (PostgreSQL) 13.6

Linux ww 5.10.102-1-MANJARO #1 SMP PREEMPT Wed Feb 23 14:44:40 UTC 2022 x86_64 GNU/Linux

Additional Information
postgres orders table schema
psql (13.6, server 12.9 (Ubuntu 12.9-0ubuntu<IP_ADDRESS>))
SSL connection (protocol: TLSv1.3, cipher: TLS_AES_256_GCM_SHA384, bits: 256, compression: off)
You are now connected to database "w" as user "postgres".
w=# \d orders
                                Table "public.orders"
   Column   |            Type             | Collation | Nullable |      Default
------------+-----------------------------+-----------+----------+-------------------
 team_id    | bigint                      |           | not null |
 product_id | bigint                      |           | not null |
 user_id    | bigint                      |           | not null |
 service_id | bigint                      |           | not null |
 status     | order_status                |           | not null |
 type_      | order_type                  |           | not null |
 paid_at    | timestamp without time zone |           | not null |
 price      | integer                     |           | not null |
 id         | bigint                      |           | not null |
 created_at | timestamp without time zone |           | not null | CURRENT_TIMESTAMP

relative issue:
https://github.com/SeaQL/sea-orm/issues/586

Hey @pymongo, thanks for the report! It's more related to SeaORM codegen. Please track this issue on SeaQL/sea-orm#586. I'll close this now