some doubt about Asynchronous Compensation Pixel Alignment?

Hi, thanks for sharing the great paper. I just wonder that does ACPA is not a normal operation? as for multimodal detection, when prepare training data, we will consider the asynchronous problem, it a very common operation that will do in many codebase like mmdetection3d 、centerpoint。or it's not a common operation in Panoptic Segmentation? And I don't see ablation study about ACPA, only ACPA with other module.

Thanks for your attention. Previous object detection or semantic segmentation methods such as PointPainting, PointAugmenting and IMF mostly directly align each lidar frame with the images at the closest timestamp, while our ACPA aims at transforming the 3D coordinate into a new position at the time when images are captured. Detailedly, we decompose such transformation into the combination of  two sub transformations. Furthermore, different from detection settings, panoptic segmentation needs more low-level discrminative features in multi-modal fusion to better recognize point-level instance information.
For ablation study, it can be observed that our ACPA improves PQ performance contrasting the first two lines of Table 3. Please note that SC represents simple concantenation and it is not a another proposed module, since LiDAR-only baseline do not need point-pixel feature concatenation. Moreover, we provide qualitative comparisons of ACPA in Figure 7 of the Appendix。

Thanks for your attention. Previous object detection or semantic segmentation methods such as PointPainting, PointAugmenting and IMF mostly directly align each lidar frame with the images at the closest timestamp, while our ACPA aims at transforming the 3D coordinate into a new position at the time when images are captured. Detailedly, we decompose such transformation into the combination of two sub transformations. Furthermore, different from detection settings, panoptic segmentation needs more low-level discrminative features in multi-modal fusion to better recognize point-level instance information.
For ablation study, it can be observed that our ACPA improves PQ performance contrasting the first two lines of Table 3. Please note that SC represents simple concantenation and it is not a another proposed module, since LiDAR-only baseline do not need point-pixel feature concatenation. Moreover, we provide qualitative comparisons of ACPA in Figure 7 of the Appendix。

I know how you do ACPA，as I’m familiar with multi-modal 3d object detection，when find point-image corresponding relation，In nuScenes，we will first convert point cloud to global coordinate and then transform into the ego vehicle frame for the timestamp of the image，and I thought the operation is a default operation for multi modal method。
I think you can give a result that only concatenation is used to say that ACPA is useful，rather than ACPA with concatenation。

Well, it may not be a default operation as a common concept in my understanding, as you can see that direct projection without step-2 and step-3 of ACPA in Figure 7 usually cause some misalignment in the edges as previous approaches do. It is not so good but can be still used. When adding asynchronous compensation, the misalgnment is refined. Also, more accurate projection is important for the following fusion module.
Moreover, thanks for you second advice and we will consider more experiments settings to dig deeper. Quantitatively, our ACPA has showned performance improvement and prove the usefulness. And my initial thought is that you have to firstly decide which pixels in which camera to be aligned, then you can fuse/concatenate them together, and thus concatenation is naturally accompanied with ACPA. We will pay attention to more experiments design.


Build process does not exit non-zero when it fails to load template

I'm submitting a ...

[x] bug report

Current behavior:
gulp build on Mac succeeds but fails on Linux build system (due to case sensitivity difference in the filesystem).  Gulp build process returns a successful "zero" exit code.
Expected / new behavior:
When the template fails to be read, the gulp build command exits with with a zero exit code (successful build).  However, Jenkins handles errors based on process exit code.  So this will be a misreported success and should have failed, instead.
Minimal reproduction of the problem with instructions:
Let's pretend a developer on a Mac names their template file path/to/somedirective.html.  The following configuration will succeed because Mac has a case insensitive filesystem.
angular.module('someModule')
    .directive('someDirective', function (someReportFactory, $timeout, someEventFactory,
                                        notificationService, someConstants, $mdDialog, $q, $window, someConstants, pollingService) {
        return {
            scope: {
                basePath: '=',
                reportOptions: '=',
                reportInvalidRules: '=',
                reportFieldValidation: '=',
                route: '='
            },
            restrict: 'E',
            templateUrl: 'path/to/someDirective.html',

            controller: function( $scope ) {
....

However, on Linux (like a Jenkins build system running builds on Linux) the filesystem is case sensitive so path/to/someDirective.html is not the same as path/to/somedirective.html.

gulp 3.9.1
gulp-concat 2.6.0
gulp-cssnano 2.1.2
gulp-sass 2.0.4
gulp-util 3.0.7
gulp-angular-embed-templates 2.2.0
gulp-json-angular-translate 1.0.0

Browser: N/A
Anything else:

See also:

https://github.com/angular/angular.js/issues/16429
https://github.com/gulpjs/gulp/issues/2117

Example stack trace on Linux:
gulp-angular-embed-template [Warning] Can't read template file: "path/to/someDirective.html". Error details: Error: ENOENT: no such file or directory, open 'path/to/someDirective.html'

@frederikprijck helpfully did some sleuthing and mentioned<EMAIL_ADDRESS>is probably the issue.


fix: Update to only print logs on failure. Also describe pods, sometimes p…

…ods are pending and it causes failures but I can't see why they're pending.

:tada: This PR is included in version 1.0.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update index.html

Added my name

@Netlob  thnx for contributing , try this year aswell with updated repo and make sure to follow me and star this repo so that you don't miss updates regarding the new changes and issues during Hacktoberfest 2020.


Availability via conda-forge?

Hi, @walkerke! Are you considering providing a release via conda-forge? This would increase your (awesome) package's visibility.

ADD A RECIPE

cc @knaaptime, cc @jvtcl

Hi @jGaboardi!  Yes, I need to do this.  I'll try to tackle this month!


Feature: check for pull-requests permission in Token-Permission

pull-request allows changing PRs so it's dangerous

I'm actually worried this will trigger too many false positives. I think pull-request permission is also needed to update labels on a pull requests https://docs.github.com/en/rest/reference/permissions-required-for-github-apps#permission-on-pull-requests
Basically one can merge or update PR codes with this permission, but it's also the same permission needed to do more mundane things like label PRs, update description, etc.

cc @josepalafox


more specs

Here are five more specs that don't pass yet but I think should:
#1
it 'keeps numbers at the end of mentions and hashtags' do
  text = "#le1101 #artistQ21 @someone12 @someoneelse1 and @somebody1980"
  pt = PragmaticTokenizer::Tokenizer.new(text, clean: true)
  expect(pt.tokenize).to eq(
    ["#le1101", "#artistq21", "@someone12", "@someoneelse1", "and", "@somebody1980"]
  )
end

#2
# the below text.bytes returns:
# => [226, 157, 132, 239, 184, 143, 226, 157, 132, 239, 184, 143, 226, 157, 132, 239, 184, 143]
# so each of the three snowflake consists of 6 bytes, but is tokenized into two tokens with 3 bytes each
it 'handles snowflakes' do
  text = "❄❄❄"
  pt = PragmaticTokenizer::Tokenizer.new(text)
  expect(pt.tokenize).to eq(
    ["❄", "❄", "❄"]
  )
end

#3
it 'deals with missing whitespaces' do
  text = "this is sentence one!this is sentence two.@someone"
  pt = PragmaticTokenizer::Tokenizer.new(text)
  expect(pt.tokenize).to eq(
    ["this", "is", "sentence", "one", "!", "this", "is", "sentence", "two", ".", "@someone"]
  )
end

#4
it 'cleans a prefixed weird hyphen' do
  text = [104, 105, 103, 104, 32, 173, 116, 101, 109, 112, 101, 114, 97, 116, 117, 114, 101, 32, 97, 110, 100, 32, 173, 119, 105, 110, 100].pack("U*")
  pt = PragmaticTokenizer::Tokenizer.new(text, clean: true)
  expect(pt.tokenize).to eq(
    ["high", "temperature", "and", "wind"]
  )
end

#5
it 'separates tokens' do
  text = "football≠soccer"
  pt = PragmaticTokenizer::Tokenizer.new(text)
  expect(pt.tokenize).to eq(
    ["football", "≠", "soccer"]
  )
end

Number 3 above contradicts a previous spec:
it 'knows what is not an email address' do
  text = "the great cook.@someone something else@whoever"
  pt = PragmaticTokenizer::Tokenizer.new(text,
    remove_emails: true
  )
  expect(pt.tokenize).to eq(["the", "great", "cook.@someone", "something", "else@whoever"])
end

# => ["the", "great", "cook", ".", "@someone", "something", "else@whoever"]

I will update this previous spec as it seems that the behavior you want is in line with number 3 above.

These are all now passing (https://github.com/diasks2/pragmatic_tokenizer/commit/28d216dcd4d61b9f8f8f8ad8601d2e4fee09721a) although we still need a more elegant way to handle emoji but that is handled in issue https://github.com/diasks2/pragmatic_tokenizer/issues/20


The idea plugin is not very reasonable when formatting the api file.

Describe the bug
The idea plugin is not very reasonable when formatting the api file.
Screenshots

Environments (please complete the following information):

OS: Macos m1
IDEA:
IntelliJ IDEA 2023.1.1 (Ultimate Edition)
Build #IU-231.8770.65, built on April 27, 2023
Subscription is active until June 23, 2023.
Runtime version: 17.0.6+10-b829.9 aarch64

More description
Although there is no problem with this, I personally don't like it and may have the same OCD as me.

It will only appear when the idea plugin is formatted, there will be no problem using command line formatting.


[Tests] Nested formatting issues

Here are some Tests demonstrating razor formatting issues.

This PR contains two change sets.

The first has all the Tests that fail.
The second fixes one of these test.

I tried to fix the other two as well but wasn't able to. Also I'm not sure if my approach in fixing the first one is aligned to how things should be done.
Please check if my assumption about supposed formatting is actually correct.
If you can give me some guidance I'm willing to fix the other two tests as well.

Thanks for the feedback I will investigate. And submit another PR for tests only.

Removed the fix change set from this PR.
Also created issues for each test.
#8227
#8228
#8229


webpack velocity issue when importing materializecss

I'm having this issue: Uncaught TypeError: k.velocity is not a function at c

but only when I import via js, if I do insert the materializecss+jquery via CDN it works, like this:
<script type="text/javascript" src="https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js"></script>
<script type="text/javascript" src="https://cdnjs.cloudflare.com/ajax/libs/materialize/0.98.0/js/materialize.min.js"></script>

on my index.js I'm importing materializecss this way:
import 'materialize-css/bin/materialize';

I'm using webpack to bake everything together.

Yeah, I'm having the same issue.
I've searched and found some issues here, talking about adding an alias to Webpack ( #2690, #2681 ) but somehow I cant get this to work.

Closed due inactivity. Feel free to reopen it, if it's still necessary.


build: update dependencies

lit ssr update includes a fix for defer-hydration
https://github.com/lit/lit/issues/3939

lit ssr update includes a fix for defer-hydration
lit/lit#3939

material-web is ssr compatible?

material-web 1.0 has a best-effort but not yet official support for SSR. They currently all work in SSR but require more extensive testing.
Though, this means that it is SSRable with the lit-ssr library. Those frameworks need to know to call the lit-ssr library. For NextJS there is an official Lit plugin:
https://www.npmjs.com/package/@lit-labs/nextjs
For Nuxt we have worked with some community members to enable this integration:
https://github.com/prashantpalikhe/nuxt-ssr-lit
Astro already supports Lit SSR out of the box
There is an official plugin for Eleventy + Lit SSR:
https://www.npmjs.com/package/@lit-labs/eleventy-plugin-lit
We currently do not know anything about Svelte and SSR integrations, but if you are interested in contributing, I recommend joining the Lit discord and expressing interest in the #get-involved channel

I'm in the process of building a svelte wrapper, so I'll definitely look in to integrating this.

@Aworldc Highly recommend chiming in on the #get-involved channel on the lit discord if you don't know where or how to get started integrating it. That channel is there to help contributors. team members there would be super happy to point you in the right direction and answer questions about requirements to get lit-ssr running / integrated etc.


bump: Go version to 1.20

Additional steps taken: go mod tidy & go mod vendor

[#185619304]

superceded by https://github.com/cloudfoundry/credhub-acceptance-tests/pull/12


E2E: go workflow_dispatch main SLSA3 config-noldflags

Repo: https://github.com/ianlewis/example-package/tree/main
Run: https://github.com/ianlewis/example-package/actions/runs/2492244882
Workflow name: go workflow_dispatch main SLSA3 config-noldflags
Workflow file: https://github.com/ianlewis/example-package/tree/main/.github/workflows/e2e.go.workflow_dispatch.main.config-noldflags.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Tue Jun 14 01:39:55 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/2492556075
Workflow name: go workflow_dispatch main SLSA3 config-noldflags
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.workflow_dispatch.main.config-noldflags.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Tue Jun 14 03:24:08 UTC 2022
Tests are passing now. Closing this issue.


Closes #95 and #96

Fixes Issue
Closes #95 and #96
open source library and privacy policy hindi translate
margin support attribute added to support older version than api 17

Changes proposed

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers
UI changes has made in this PR

issue #72 has been fixed
close #72

@Deepshikhayadav this issue was assigned to someone else, so you cant work on it

@Deepshikhayadav
Provide me the screen recording for these changes.


Add certbot-related targets to Makefile

Add three new certbot-related targets to Makefile. These has helped me to automate the process of obtaining and renew certificates for any https-server defined in servers.json.
validate-certs
Iterates the servers in servers.json, returns a non-zero exit status if any of the servers (with https: true) does not have fullchain.pem and privkey.pem files in the expected locations.
generate-certs
Iterates the servers in servers.json, attempting to generate a certificate with certbot for any server with https: true.
renew-certs
Attempts to renew any certificates previously obtained with certbot.

Vad säger du @Dunklas, är det inte dags att dra in den här?

Haha oj. Men fan, då är det klart att den ska in! Ska se över mergekonflikten bara! :)


Release v3.5.0

New features

[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2179
[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2177
[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2204
[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2169
[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2190
[x] Improved armv6 support

[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2217

[x] Github CI Workflow improvements
[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2233
[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2218
Documentation Improvements

[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2216
[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2225

Other Bugfixes

[x] https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2235

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-14.0%) to 38.384%

Totals

Change from base Build<PHONE_NUMBER>:
-14.0%

Covered Lines:
418

Relevant Lines:
1089

💛  - Coveralls

Whats missing:
The need to update the jukebox.yaml for the coverart should be documented here.
Or we implement the automatic update / default value initialization before (which i would prefer :) ).

Event device is not merged yet

Event device is not merged yet

I know - it's considered a draft and just created to represent the next release early on (updated the description)

I think we have a problem on the update path with this release.
If currently described "just pull" isn't enough. And a rebuild of the webapp is also not possible if node is not installed in main.
Maybe we find a quickfix, so a rerun of the installation is possible, without massive changes?
Ideas:

Move to current Project Folder to "*-backup-$datetime"
Let the user rerun the installation
copy 'shared'-folder from backup to the current installation
give note that manually changes need to be transfered manually

A proper way can then be implemented with v3.6.0

I think we have a problem on the update path with this release. The currently described "just pull" method isn't enough. And a rebuild of the webapp is also not possible if node is not installed in main.
Maybe we find a quickfix, so a rerun of the installation is possible, without massive changes? Ideas:
* Move to current Project Folder to "*-backup-$datetime"

* Let the user rerun the installation

* copy 'shared'-folder from backup to the current installation

* give note that manually changes need to be transfered manually

A proper way can then be implemented with v3.6.0

Or we just state that updates are currently broken and a fresh installation has to be done.

Test errors for Web App download can be ignored. This is caused due to the version bump, as no bundle is build for a release version on develop.
Well be fixed after the release and the bump to the next develop version.


[Feature request] flag for nvm use that says "install it if you don't have it"

First off, thanks for this absolutely excellent tool! Much appreciated.
In nvm v0.39.1 (the latest at the time of writing), when I do nvm use for a version I don't have yet, I get a prompt to run nvm install:
$ nvm use

Found '/Users/tim/example/.nvmrc' with version <14.17.6>
N/A: version "14.17.6 -> N/A" is not yet installed.

You need to run "nvm install 14.17.6" to install it before using it.

It would be awesome if I could do something like:
$ nvm use --install-if-not-present
Found '/Users/tim/example/.nvmrc' with version <14.17.6>
N/A: version "14.17.6 -> N/A" is not yet installed.

Downloading and installing node v14.17.6...
...etc

(I don't know that --install-if-not-present is the best name for this flag, I just wanted to make the intent clear in this request)
nvm install does seem to also imply nvm use (although I haven't seen this documented). However, it takes several seconds even when you already have the version installed, whereas nvm use is immediate when I already have the version installed. If the intended solution is to use nvm install for this use case, then I'm happy to open an issue about the slow timing of nvm install instead.

This is the behavior nvm install already has, yes, and is the intended usage.
#2827 may improve perf here, but if not, please do file a new issue.


Fix RX/TX pins

Due typo che si annullavano a vicenda

This is actually not a typo but the intended behaviour. The RX of board is connected to the TX of the modem and vice-versa. It took me a while to figure it out at the beginning so I guess it's normal that it generates some confusion.. (See #8).


Changed styles

Apologies for the confusion, it seems I mistakenly targeted the wrong branch for this merge.


Service Discovery Design

I'm really interested in integrating service discovery, by default for our apps. This design doesn't include App Mesh integration, but I think that'd be additive.
What is service discovery?
Service discovery allows two services to communicate using an internal (within the same VPC) DNS.
An example would be, if you have two apps, backend and frontend, the frontend app might have a public loadbancer, but the backend might be isolated in a private subnet.
The frontend would be able to communicate with the backend service using a VPC-local DNS like backend.local. The backend would also be able to contact the frontend through a VPC-local DNS in the same way. This has the benefit of the data not transiting the internet and the inter-service communicate doesn't rely on the LB url.
What services to use?

AWS CloudMap
ECS's ServiceDiscovery (same thing, but handles the registering of tasks into CloudMap)

What do we have to do?
It's actually really easy.

In the environment, create a ServiceDiscovery Namespace (this is the DNS suffix). I'm thinking .local.

  ServiceDiscoveryNamespace:
    Type: AWS::ServiceDiscovery::PrivateDnsNamespace
    Properties:
        Name: local
        Vpc: !Ref VPC

Export the Namespace ID

In the app stack, create a discovery service (this is the actual DNS for this app):

  DiscoveryService:
    Type: AWS::ServiceDiscovery::Service
    Properties:
      Description: Discovery Service for the ECS CLI V2 services
      DnsConfig:
        RoutingPolicy: MULTIVALUE
        DnsRecords:
          - TTL: 60
            Type: A
          - TTL: 60
            Type: SRV
      HealthCheckCustomConfig:
        FailureThreshold: 1
      Name:  !Ref AppName
      NamespaceId:
               Fn::ImportValue:
               !Sub '${ProjectName}-${EnvName}-NamespaceID'

Update the service to use ServiceRegistries

      ServiceRegistries:
        - RegistryArn: !GetAtt DiscoveryService.Arn
          Port: !Ref ContainerPort

This will set up everything, but one tricky thing is our security groups don't permit inter-app communication right now (since each app gets its own security group and each security group only allows traffic from the ALB and itself. So, what we could do is move the security group up to the env, and have each app use the same security group. This is definitely the trickiest part.

Example
Calling a backend app from the frontend app. The backend is running on port 8080:
resp, err := http.Get("http://backend.local:8080/")

Questions

Is it possible to drop the suffix completly, so we can have a more docker-compose like experience (where you'd just make a request to http://backend:8080)?
Is it possible to not bind to a particular port in service discovery (in the case that a service may expose multiple ports)?
What role will/should AppMesh play in this?

I am dumping a few dumb questions, I need to do more research to understand service discovery:

In the environment, create a ServiceDiscovery Namespace (this is the DNS suffix). I'm thinking .local.

Did you fix it to a value like "local" on purpose (instead of say the env name) so that users testing their applications locally will get the same behavior once deployed?
I wonder if we should pass this value as an environment variable to the containers in the task definition?

In the app stack, create a discovery service (this is the actual DNS for this app):

Looking at Nathan's example there is only an SRV record with a much more aggressive TTL.  But that's not for Fargate tasks.  The tutorial instead only has an A record. Neither of them have a RoutingPolicy.
Can you explain to me how these are used 😵? Why can't we just use SRV since it returns both IP Addresses and ports per task?

It is recommended to use container-level health checks managed by Amazon ECS for your service discovery service.

https://docs.aws.amazon.com/AmazonECS/latest/developerguide/service-discovery.html
Does this mean we need to set a healthcheck value for each container in the task?

Ok, I think I have a better understanding of the concepts now :D I didn't know anything about DNS, so I hope this helps others too.

Service discovery is neat, and allows the application code to make requests like this:

resp, err := http.Get("http://backend.local:8080/")

We need to resolve backend.local to a real IP address. Route53 helps us do this by returning to us up to 8 ip addresses (because of MULTIVALUErouting policy) for the service, each record with a TTL set (60 seconds in David's post).
One of these ip addresses gets cached locally in the task for the duration of the TTL. Once the TTL expires, another request to Route53 is made.
If the downstream task goes down until the TTL expires, our application code will keep receiving errors. So it becomes important to balance the TTL value with cost.

It's important for Route53 to store only ip addresses that are healthy. This is achieved by setting a HealthCheck config to the important containers in our task definition. Whenever a container becomes unhealthy the ECS scheduler reports that back to CloudMap. CloudMap then removes the entry from Route53 after 30 seconds * FailureThreshold:
HealthCheckCustomConfig:
     FailureThreshold: 1

I'm still not sure what's the point of SRV records but it appears that they're used if we were to choose a weighted routing policy instead.
Note that the application still needs to specify the port that's exposed from the container it's making a request to. Service discovery only helps you resolve IP addresses from a logical name.


Kafka should not need Ansible for configuration

Ideally we need to be able to remove Ansible configuration that are used to setup Kafka

On the initial startup, Kafka should register the “health” topic.

Summarizing offline discussion with @DanLavine and @rabbah on this issue.
We are running kafka with autocreation of topics enabled.  But that doesn't allow us to set the various configuration parameters for the topic that are set in the ansible scripts.  It is believed setting these appropriately is important for performance.  One possible path forward is to upgrade kafka to 0.11.0.x.  The 0.11.0 release of kafka introduced the org.apache.kafka.clients.admin.AdminClient API which would allow us to create the topics with the needed configuration parameters programmatically from the controller.  This would eliminate the need for external scripting.

I submitted PR https://github.com/apache/incubator-openwhisk/pull/2948 to progress this.  If it is accepted, then we should be able to resolve issue with small additional effort.

#2948 was merged.  PR https://github.com/apache/incubator-openwhisk/pull/2991 covers the health topic.  After it is merged, I believe we can use a vanilla kafka container (won't need custom image).

#89 updates this project to take advantage of apache/incubator-openwhisk#2948.

resolved by #130; can be closed


Fix get in touch link

The markdown within the div was not being translated. The primary reason why is the markdown works only when it is context free. Thus, this must be converted to HTML to work.

Thanks.  Yeah, I'm so used to pandoc's markdown rules, this one keeps tripping me up.  (e.g. use.Rmd uses .Rmd to get around this very issue, not sure if there's a better way.)

Interesting. So apparently you can get blackfriday to parse markdown inside a merely by altering the arrangement of whitespace.  That is rather unexpected.  see https://github.com/rocker-org/website/commit/be7a9351587563331f216968ab6f364a76d13615 which lets this render markdown in the div.


Inquiry About Some Details

Thank you for your nice work! It's highly inspiring. I have some questions regarding certain details and would greatly appreciate your guidance.
In the "Model Structure" section, it is mentioned that "The imitative U-Net is initialized with a stable diffusion-1.5 [33] inpainting model. It takes a tensor with 13 channels as the input." My understanding is that this involves using the pre-trained model provided by diffusers and concatenating the results of multiple VAE (frozen) modalities. Is this correct? How exactly is the U-Net with 13 channels input initialized in this context?

Yes, the standard SD inpainting owns 9 channels, we changed it to 13 channels( with an additional 4 depth channels). In this way, the first conv_in layer is randomly initialized, and other weights are initialized with the standard SD inpainting.

Yes, the standard SD inpainting owns 9 channels, we changed it to 13 channels( with an additional 4 depth channels). In this way, the first conv_in layer is randomly initialized, and other weights are initialized with the standard SD inpainting.

Thank you for your detailed response. By the way, will the training code for the model be released like the Anydoor?


termsrv.dll 10.0.15063.994 not supported

Correctly listening, but RDPConf says that the attached dll is not supported.
Thanks for your job, waiting for an update to support the new dll.
termsrv.zip

I have the same problem with the same DLL. We will wait for an update

Sorry but i don't have the 32 bit version of the dll, I checked on SysWoW64 without success.

x86 architecture DLL is required to resolve this issue.

termsrv-x86.zip
Attached termserv.dll from Windows 10 Pro (x86) version.

@WBowley this is wrong DLL, build number should be 10.0.15063.994.

There is just an x64-version of termsrv.dll 10.0.15063.994.
The offsets are:
[10.0.15063.994]
; no x86 version
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=8CB01
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=15EA4
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=FAE5
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=234DC
SLInitFunc.x64=New_CSLQuery_Initialize

[10.0.15063.994-SLInit]
; no x86 version
bInitialized.x64      =E9468
bRemoteConnAllowed.x64=E946C
bMultimonAllowed.x64  =E9470
ulMaxDebugSessions.x64=E9474
bFUSEnabled.x64       =E9478
bServerSku.x64        =E9484
lMaxUserSessions.x64  =E9488
bAppServerAllowed.x64 =E948C


textarea_ stops changing along with model after edited manually

How to reproduce
Take this code:
{-# LANGUAGE OverloadedStrings #-}
{-# LANGUAGE RecordWildCards #-}
module Main where

import Miso
import Miso.String

data Model
   = Model
     { _i :: !Int
     , _txt :: !MisoString
     }
     deriving (Eq)

main :: IO ()
main = startApp App { initialAction = NoOp, ..}
  where
    model  = Model 0 "0"
    update = updateModel
    view   = viewModel
    events = defaultEvents
    subs   = []

updateModel :: Action -> Model -> Effect Action Model
updateModel AddOne m = noEff m { _i = _i m + 1, _txt = ms $ show $ _i m + 1 }
updateModel NoOp m = noEff m

data Action
  = AddOne
  | NoOp
  deriving (Show, Eq)

viewModel :: Model -> View Action
viewModel x = div_ [] [
   button_ [ onClick AddOne ] [ text "+" ]
 , textarea_ [] [ text $ ms (show $ _i x) ]
 ]

I removed the - button because I hate being negative about it.

Load the app
Click the + button a couple of times to see that it works at first
Edit the text area
Press the + button again

Expected output
Nothing is done on onInput. I expect to be able to type whatever into the textarea until I press +, which should replace the contents with the next number.
Actual result
The text area gets angry at me. It won't update anymore. When I click +, the old text remains in the textarea.
Notes

Adding an onInput action and linking it to the _txt of the model doesn't make a difference.

This turns out to be intended behaviour. I could even reproduce it in Elm.
The cause here is that a nested text won't work full stop. Elm has a value Attribute which can be used instead. The value in Elm, however, differs from the value in miso. Miso adds a value=... attribute, whereas Elm sets the value property of the textarea's Javascript object. Miso's method doesn't seem to display the value in the textarea.

For clarity, this should be the correct viewModel function:
viewModel :: Model -> View Action
viewModel x = div_ [] [
   button_ [ onClick AddOne ] [ text "+" ]
 , textarea_ [ value_ (ms (show $ _i x) )] [ ]
 ]

This will work when value sets the object's property instead of the attribute.


Implement usage for Angular

PR Type
What kind of change does this PR introduce?
[ x ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] angular.io application / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?
The Issue with the current Version is that i am unable to import the Libary. It runs into an Typescript issue "TS2306 file is not a module" due to the lack of an export member for the namespace. With the change i introduced this issue is fixed.
What is the new behavior?
We finally can import the Lib as follows
import * as terminal from 'jquery.terminal';
 declare var jQuery: any;

 jQuery(<SELECTOR>).terminal(function(command) {
 });

Does this PR introduce a breaking change?
[ ] Yes
[ x ] No

Other information

You've break typescript check, I have a typescript file that test the API against typescript definition.
Sorry, but this make no sens you just removed all the types from the file.
to use the types from the file you don't need to import anything because the value is added to jQuery namespace. Just use:
import "jquery";
import "jquery.terminal";

optionally you can import jQuery into $ but you don't need named export in terminal because it don't have it's own namespace.

You can see typescript file that is working here https://github.com/jcubic/jquery.terminal/blob/master/test.ts

Oh shit... i see, i see.
My bad :/


An unresolved conflict was merged into master in the Prerequisites section

In this page, obviously, there's an unresolved conflict in Prerequisites section
Please make sure you meet the following requirements: <<<<<<< HEAD

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

ID: 51faeed1-77c1-c8c6-c097-6806a3bbdf85
Version Independent ID: 84597429-d2e1-71ed-ee5d-fd2d66dcbff7
Content: Windows and Linux Containers on Windows 10
Content Source: virtualization/windowscontainers/quick-start/quick-start-windows-10-linux.md
Service: windows-containers
Product: windows-containers
GitHub Login: @taylorb-microsoft

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


WebDeploy (Package) from Visual Studio 2015 doesn't generate correct website name, setparameters.xml and deploy.cmd files

I'm trying to publish my ASP.NET Core app (on top of .NET framework) into a .zip file.
Test.pubxml
<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <PropertyGroup>
    <WebPublishMethod>Package</WebPublishMethod>
    <LastUsedBuildConfiguration>Release</LastUsedBuildConfiguration>
    <LastUsedPlatform>Any CPU</LastUsedPlatform>
    <SiteUrlToLaunchAfterPublish />
    <LaunchSiteAfterPublish>True</LaunchSiteAfterPublish>
    <ExcludeApp_Data>False</ExcludeApp_Data>
    <PublishFramework>net452</PublishFramework>
    <UsePowerShell>True</UsePowerShell>
    <DesktopBuildPackageLocation>Test.zip</DesktopBuildPackageLocation>
    <PackageAsSingleFile>true</PackageAsSingleFile>
    <DeployIisAppPath>TestSite/TestApp</DeployIisAppPath>
  </PropertyGroup>
</Project>

When I use this profile to publish into a .zip file package I get the one directory and two files:
Content/website
archive.xml
systeminfo.xml
From what I read about webdeploy so far there are two unexpected things:
Where is *.deploy.cmd file?
Why my website is called 'website' instead of 'TestSite/TestApp'?
The repro is to create ASP.NET Web Api Net Core (.NET Framework) app from template and use this profile to publish.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


How is porosity calculated

For a given packing size, for example, [nx,ny,nz], the generated particles may occupy in a region outside of the box defined by nx, ny and nz. I am just wondering how the final porosity is calculated? are you using the volume defined by the packing size defined in  generation.conf? or the actual domain size occupied by the generated particles.
Thanks for the great tool!
Best,

Hi,
sorry for a long time to reply.
The code assumes periodic boundary conditions: https://en.wikipedia.org/wiki/Periodic_boundary_conditions
So a particle can cross the boundary of the box, but "the same" particle (a periodically shifted copy of a particle) is crossing the box from another side. So the particle and its periodic copies occupy in one box exactly the same volume as one particle that is completely inside the box (4/3 pi R^3)
So you can calculate the density as \sum_i 4/3 pi R_i^3 / (nx * ny * nz) (where sum happens over all particles). Porosity is then 1 - density.
Hope this helps!
Best Regards,
Vasili

Hi,
sorry for a long time to reply.
The code assumes periodic boundary conditions: https://en.wikipedia.org/wiki/Periodic_boundary_conditions
So a particle can cross the boundary of the box, but "the same" particle (a periodically shifted copy of a particle) is crossing the box from another side. So the particle and its periodic copies occupy in one box exactly the same volume as one particle that is completely inside the box (4/3 pi R^3)
So you can calculate the density as \sum_i 4/3 pi R_i^3 / (nx * ny * nz) (where sum happens over all particles). Porosity is then 1 - density.
Hope this helps!
Best Regards,
Vasili


[process-agent][core-agent] Fix leaky HTTP connections

What does this PR do?
process-agent is currently leaking HTTP connections whenever the agent/status endpoint is reached or when we run the ./process-agent status command.
In order to build the status output, process-agent connects to its expVar server using a httpClient. This client is currently being recreated whenever getExpVars is called but according to the golang HTTP Client documentation:

The Client's Transport typically has internal state (cached TCP connections), so Clients should be reused instead of created as needed. Clients are safe for concurrent use by multiple goroutines.

This PR creates a global httpClient that can be reused across multiple calls to the /agent/status endpoint.
The core-agent is also recreating a new HTTP Client when

./agent status is triggered in order to fetch process-agent status
a new flare is built with ./agent flare to fetch process-agent runtime configs

The PR also updates the DoGet method from the api package in order take a ShouldCloseConnection argument. This parameter controls whether or not the underlying connection should be kept open after the request response is completely read.
Motivation
Fix HTTP connections leak in process-agent and core-agent.
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes
With the process-agent and the core-agent running

Reach to process-agent /agent/status endpoint using curl localhost:6162/agent/status. Verify that new HTTP connections are created only once even if curl is executed multiple times. You can use lsoft -i -P -n | grep "6162\6062" to verify it.

Collect the process-agent status with ./process-agent status multiple times and make sure that HTTP connections are not recreated

Collect the core-agent status with ./agent status multiple times and make sure that HTTP connections are not recreated

Collect the core-agent flare with ./agent flare multiple times and make sure that HTTP connections are not recreated

Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[x] The appropriate team/.. label has been applied, if known.
[x] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[x] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[x] If applicable, the need-change/operator and need-change/helm labels have been applied.
[x] If applicable, the config template has been updated.

Thanks for the follow up @olivielpeau. Since most teams have already approved the PR, I'll keep it as it is and I'll create a card on our side to improve the API server on process-agent to set these timeouts.
We still need to be able to control these settings on the client side because process-agent uses another HTTP Client to connect to the ExpVar server. We're currently using the default HTTP server to handle ExpVar requests so we don't have control over these settings.
https://github.com/DataDog/datadog-agent/blob/f4cda31151f05001e25e681a9cbc97c1fa2cbc58/cmd/process-agent/main_common.go#L344-L352


jb install failed - etcd-mixin: no such file or directory

What happened?
When attempting to install kube-prometheus, install fails due to issue with jsonnet-bundle
Did you expect to see some different?
kube-prometheus and dependencies installed.

I did notice this recent change in etcd repo - https://github.com/etcd-io/etcd/commit/2afa5ff45ff4cc7f7a77218f0d7e52b377733dc0
Wonder if its related ?

Sry I wanted to open a blank issue. This by no means is a bug. Not sure how to change it.

Looks like the new location for the etcd-mixin directory is https://github.com/etcd-io/website/tree/master/content/docs/current/etcd-mixin

seems like a bug to me?
could pin the commit sha from which kube-prometheus pulls the etcd-mixin - they didn't rewrite history so it would still be there in a previous commit. might be a good idea to pin all versions of dependencies to commits?

oh hmm ... seems the version is pinned but jb expects the path to exist in master as well as in the pinned revision? or something else is depending on the version in master that's not there? either way this is preventing not just 0.7.0 but all releases of kube-thanos to fail install.

Issue fixed in etcd repo. Things seem to work now.


Title in browser window includes the *

Issue in ./help/troubleshooting/miscellaneous/file-cannot-be-deleated-no-file-or-directory.md
The page title includes the asterisk that's included in the content:
The file can't be deleted. Warning! unlink: No such file or directory error from the Admin | Adobe Commerce

https://git.corp.adobe.com/AdobeDocs/commerce-knowledge-base.en/pull/500


Swadge Hero

Summary
It's a game, somewhere between DDR and Guitar Hero. It should have at least 10 playable songs with three difficulties each. Notes are mapped to buttons and the touchpad can be a whammy-like input.
Technical Spec
https://github.com/AEFeinstein/midi-ch can be used to auto-generate chart data from MIDIs
Info on the chart format exported can be found here: https://github.com/TheNathannator/GuitarGame_ChartFormats/blob/main/doc/FileFormats/.chart/Core Infrastructure.md
The exported chart is converted by the asset preprocessor into something more binary and swadge-friendly.
UI

Normal menu for picking a song, difficulty, etc.

Maybe color-shift it?

Not sure what to do for high score display yet

Mockups
Nah
How to Test
Play it! It should feel right, timings are important here.\
TODO
Gameplay

[x] Fail meter (hit is +1, miss is -2)
[x] Combo Counter
[x] Scoring

[x] Save best score when done

[ ] Touchpad to pitch bend. Only allow during holds?
[ ] Shake to star power? How to accumulate?
[x] Pause Button
[x] Investigate letter scoring system

I got a full combo on chowa grove but still scored a B+

[x] Investigate score saving (incorrect)

Also I scored 3654pts but my high score says 1218?

[x] Fix consistently late input lag

Graphics

[x] Replace circle notes with icons
[x] Add VFX for note hits & misses
[x] LED FX

UI

[x] End of song screen

[x] Show score
[x] Show progress bar
[x] Draw line between 'hit' and 'miss' notes on score screen

[x] High Score display (build into menu)
[x] Settings

[x] Fail off
[x] Note scroll speed
[x] Calibration mode?

Calibration isn't necessary with timestamped notes

Update Ocean City 1989 title


Make package compatible with Laravel 5

I created this in a 2.0 branch to automatically get a 2.0.x version on Packagist but I don't seem to be able to make a pull request to anything other than existing branches on your repo.
You can tag it as a new version in master or create branches for each version, as you wish.
I didn't update the README.md since it all depends on what version you tag it as.
I updated the folder structure to be PSR-4 compliant since Laravel 5 moved to that.

I'm a little low on time lately.  I've marked you as a collaborator.  Are you able to do what you need now?

Thanks, yeah I believe so! Only thing would be updating the package on packagist, don't know if you ever got round to setting up the Service/Webhook in GitHub?

Yup, i set i up a while ago.  I suppose we will see if it worked :).

Nice thanks. All worked out!


Fixed hard dependency on Symfony 3.0

There was a hard dependency on version 3.0 of the validator component.

That was a typo. Thanks for spotting it!


Completely updated de_de.lang

I think I got everything translated.
If something is wrong, just notify me.
Legondas

This also opens up the good old can of worms again of whether or not to translate "taint", since in this version it's sometimes translated and sometimes not.

Thanks for your review. I think I need a lot more practice.

This also opens up the good old can of worms again of whether or not to translate "taint", since in this version it's sometimes translated and sometimes not.

I was unsure. I think in the previous translation it was sometimes changed.

Exactly, and let's not touch that for now.


Enter key not working when software is running

Great little tool!
I have one small issue: As soon as I start the server, my enter key stops working. I can see that in the console, the output is scrolling, as if the enter key is always pressing down.

I should add that when started via systemctl service, this does not happen.

Hello, this looks like some race condition issue where rkvm grabs the keyboard and doesn't register the enter key release event and thus doesn't propagate it further. I think this could be mitigated by adding a short sleep before input devices are grabbed.
I don't have access to my computer right now, could you please check if it works by adding a time::sleep_for(Duration::from_millis(500)).await before the /dev/input directory is traversed and devices grabbed in input/src/event_manager.rs?

Hey, yep, that worked. I could no longer reproduce this.

Fixed in commit e7984e1.


Fix typo in Geocoding docs

Spotted a typo, thought I'd fix it :)

@jfversluis : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Use builtin types instead of deprecated numpy aliases

See https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
Fixes #1334

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Brandon Carpenter seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @hashstat this looks good! You need to sign the CLA, or if you can't get that to work you can post something like "I agree to the CLA".

@SteveDiamond I attempted to sign the CLA twice without success. :disappointed:
I agree to the CLA.


Yo drew life is down 😭

Looks like your socket.io service is down?

Can confirm

Looks like it is back now 👻


Fix docker based localhost setup documentation.

PTAL @pwittrock @johndmulhausen

Very nice.  Looks good to me.  Many thanks!

Haha, you were a bit faster than me :)
Looks good to me after the nit.

Thanks @pwittrock for the tech review, and thanks to @vishh for the PR. :)


Fix error tab index out of size print, issue 4997

Fixes the console printing set_current_tab: Index p_current out of size (get_tab_count()). when opening a project.
closes #4997

Wouldn't it be better to just remove the case?

I assumed it might be there for another reason...but I guess I could, wouldn't make much difference.


Using tests as filters is deprecated

TASK [Run docker fluentd] ***************************************************************************************************************************
[DEPRECATION WARNING]: Using tests as filters is deprecated. Instead of using `result|changed` use `result is changed`. This feature will be removed
 in version 2.9. Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
ok: [pilot-management]

Closing. The error is in a separate playbook and independent of this role


Add dependency information to from-scratch docs

Description
Updates documentation of needed repositories for from-scratch dependencies.
Issues Resolved
In support of #611
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #755 (75ed61c) into main (29a63a8) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##               main     #755   +/-   ##
=========================================
  Coverage     43.44%   43.44%
  Complexity      313      313
=========================================
  Files            69       69
  Lines          1982     1982
  Branches        138      138
=========================================
  Hits            861      861
  Misses         1105     1105
  Partials         16       16


DOC: update the example for PPoly.solve

Fixed the problem that currently the example for PPoly.solve actually uses PPoly.roots.

Thanks, merged.


boot2docker ssh docker run --volume failed

boot2docker 1.4.1/Windows 7.
I want to start the docker container in host (Windows) cmd shell
boo2docker ssh docker run -d --volume=/home/docker:/java larrycai/jenkins-slave-ubuntu

Unfortunately it reports
sh: syntax error: unexpected "("
error in run: exit status 2

WIthout --volume=/home/docker:/java it works fine, did I need to escape something to be passed into docker env ?
I noticed it append some environment to the path by using docker ssh -vvv ...
 debug1: Sending command: docker run -d --volume=C:\\Program Files (x86)\\Git\\home\\docker;C:\\Program Files (x86)\\Git\\java larrycai/jenkins-slave

Looks it is a bug ?

I noticed it pass the host home directory to docker env by using ssh -vvv
debug1: Sending command: docker run -d --volume=C:\Program Files (x86)\Git\home\docker;C:\Program Files (x86)\Git\java larrycai/jenkins-slave
Interesting to see it use boot2docker's host environment (Windows) instead of docker host environment (VM system).
Any possible to solve this ?

Try boo2docker ssh "docker run -d --volume=/home/docker:/java larrycai/jenkins-slave-ubuntu"
^ use "" to encapsulate the command that you are willing to run in ssh

Sorry, this repository is long-since deprecated in favor of Docker Toolbox (whose usage is now also discouraged in favor of Docker for Windows and Docker for Mac).


Report the rotation vectors from RunPCA.

Closes #17.

Codecov Report

Merging #18 (8631ae1) into master (1c5c9da) will decrease coverage by 0.01%.
The diff coverage is 96.49%.

:exclamation: Current head 8631ae1 differs from pull request most recent head aa5f8ba. Consider uploading reports for the commit aa5f8ba to get more accurate results

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
- Coverage   97.81%   97.79%   -0.02%
==========================================
  Files          64       64
  Lines        4071     4081      +10
  Branches       66       67       +1
==========================================
+ Hits         3982     3991       +9
  Misses         50       50
- Partials       39       40       +1

Impacted Files
Coverage Δ

...lude/scran/dimensionality_reduction/BlockedPCA.hpp
98.83% <87.50%> (-0.02%)
:arrow_down:

include/scran/dimensionality_reduction/RunPCA.hpp
93.24% <87.50%> (-1.50%)
:arrow_down:

...e/scran/dimensionality_reduction/MultiBatchPCA.hpp
99.47% <100.00%> (+<0.01%)
:arrow_up:

...clude/scran/dimensionality_reduction/pca_utils.hpp
97.05% <100.00%> (ø)

tests/src/dimensionality_reduction/RunPCA.cpp
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[CAMEL-13529] Update to make sure the camel-jdbc work with the spring tx manager

https://issues.apache.org/jira/browse/CAMEL-13529

Thanks the karaf features.xml file needs to be updated to have the spring-jdbc dependency added too. Also it would be good to add a little code comment or comment in the pom.xml that spring-jdbc is there to ensure transaction support works with Spring and JEE

I'll merge once we update the Karaf feature. LGTM so far.

@zhfeng are you working on this so we can get this fixed/merged

@davsclaus I add a comment in the pom.xml as your suggestion. And feel free to merge it.

@oscerd I add the fix for features.xml

thx

Merged on master. Can you please create a PR for camel-2.x? Thanks


[6.x] Make Blueprint support Grammar's macro

According to #31296, this PR makes Blueprint support Grammar's macro when building raw SQL statements from Fluent commands using toSql() method.

NB This PR was reverted.

@taylorotwell What was the problem? Should I change anything and send another PR?


Cannot access 'domain/api/user' when routes conf has already been set

environment : homestead 5.0.1
nginx conf
as document told
dictionary

/
┕---web
   ┕---index
        ┕---controller
            ┕---api
                ┕---UserController extend Controller  <-- Mix\web\Controller
                     ┕---public function actionIndex()

main_web

'rules'   => [
       'api/:controller/:action' => 'api/:controller/:action',
],

way to start mixPHP

sudo mix-httpd service/start -d -u

question
I cannot visit domain/api/user however domain/api/user/index can be visited

master 分支，路由歧义问题已经适配，该问题已经解决。


Graphics bug

First of all
Did you try disabling bob74_ipl to see if the issue is still there? Yes
Did you use the latest version of bob74_ipl ? Yes
Did you use an up to date server artifact ? Yes
Describe the bug
After some time, and when you travel around the map the graphics get lost and you can just see no textures on objects, fall through map
To Reproduce
It happens everywhere no matter where you are
Expected behavior
Fix the holes of map and add some gta Online interiors
Screenshots
none
Versions

6117
2.1.1

This is a well known FiveM Bug caused by Vehicles which stops the game from loading any textures from the location where you originally tried to load the correupted Vehicle.
Nothing from bob74_ipl is causing this!


Fix etherlime coverage

The proposed fix change the level we are looking for solc in node_modules when we are running etherlime coverage.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #123   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          17     17
  Lines         326    326
=====================================
  Hits          326    326

Continue to review full report at Codecov.

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


refresh the look of Bookinfo sample app

Refreshing the look of Bookinfo sample application. The functionality of the sample app is the same, I replaced Bootstrap with Tailwind, refreshed the design a bit, added the Izzy and Istio logo and links to istio.io in a couple of places.
Screenshots of the new look. If people agree on this change, I'll go through the rest of the docs to update any outdated screenshots (there are probably some).

Not sure if this change would require release notes (or not). Potentially.

Release notes: yes please. "Refreshed look and feel of Bookinfo".
My gut reaction to hearing about logos: other projects use this (largely for testing against us though I think!) and maybe they wouldn't if it was too overtly Istio branded?
(I think what i see in the screenshots is fine.)

End of an era
LGTM generally but some of our tests/docs grep against the html (to detect which version of reviews generally I think) so we will want to keep compat there?

@craigbox Istio logo is only on the sign-in dialog and since this is Istio's sample app, it should be fine (I can also easily remove the logo from there).
@howardjohn good point! I'll check the docs and make sure the HTML is similar so those instructions still work fine.

There was a change to the requirements.txt and dockerfile in the same exact folder like 10 min ago .. :|

@peterj: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
c74cb0f333891f62d9c460d1ec7954cb1686f383
link
true
/test integ-distroless

integ-security-multicluster_istio
c74cb0f333891f62d9c460d1ec7954cb1686f383
link
true
/test integ-security-multicluster

integ-telemetry-mc_istio
c74cb0f333891f62d9c460d1ec7954cb1686f383
link
true
/test integ-telemetry-mc

integ-telemetry-istiodremote_istio
c74cb0f333891f62d9c460d1ec7954cb1686f383
link
true
/test integ-telemetry-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

I think I messed up something here, because it's seems like it's reverting the changed that came in from https://github.com/istio/istio/commit/9a9671d05670b78d08248576510c599482490922

should be good now 🤞🏼

This looks cool 🙂

@peterj: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
6017a068b82a315b36c013c4d989159c1ba3fd99
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest

I'm not sure of the pipeline to getting new containers built, and the impact of doing so; I don't think they are in any way versioned to Istio releases.  When they are built, they'll apply to examples in all released versions of Istio.  Which is good, but means we will want to update screenshots in at least master and release-1.22.

I'm not sure of the pipeline to getting new containers built, and the impact of doing so; I don't think they are in any way versioned to Istio releases. When they are built, they'll apply to examples in all released versions of Istio. Which is good, but means we will want to update screenshots in at least master and release-1.22.

I think @howardjohn and @frankbu are two of the people that can run a build of the new images and then update the yamls to call out the new images (instructions in the README). Then we can update the docs in master and if we want to cherry-pick the yaml changes to 1.22, we can update there as well.

@howardjohn let me know once you've built the images and I can go and update the screenshots in the docs after that.

Having some issues with my machine, hopefully will be soon


[6.3.x] Upgrade Errai to 3.2.2-SNAPSHOT

+1 to merge.

I am wondering if we really should do this for 6.3.x? There are currently no plans for Errai 3.2.2.Final and moving to 3.2.2-SNAPSHOT would require a 3.2.2.Final release in November. Is there anything you need from Errai?

I am completely fine with upgrading to 3.2.1.Final. We definitely can not depend on 3.2.1-SNAPSHOT.

Yeah, I think that's better in this case as we're not expecting any needs for Errai for the 6.3. CR/Final releases.

OK. I will create new PR with upgrade to 3.2.1.Final. (I can't amend this one as the branch was created in blessed repository and it does not allow force pushes.)


One off slaves not getting deleted.

Jobs set for one off openstack slaves sit in offline purgatory indefinitely.
Poking around source it looks like:
openstack-cloud-plugin/openstack-cloud/src/main/java/jenkins/plugins/openstack/compute/JCloudsOneOffSlave.java
does not set 'isPendingDelete' to true.
Submitted pull request as possible fix:
https://github.com/jenkinsci/openstack-cloud-plugin/pull/19

Closing as #19 was replaced by #36.


Unrequired length check in autocomplete

Overview
I noticed that we were performing a similar check twice in the same file when reviewing the large upcoming typescript change. I tried to update the code to remove the redundancy.
Documentation

[ ] Updated Typescript types and/or component PropTypes
[ ] Added / modified component docs
[ ] Added / modified Storybook stories

If it helps get me workflow approval I did run my test locally and they passed.


update ZH_TW translate

Update some untranslated words or sentences
Modify where the translated sentences are not smooth
Modify a word or a wrong part of a sentence
Modify the mixture of traditional Chinese and simplified Chinese in a word or sentence

Refer to https://github.com/BetterDiscord/BetterDiscord/blob/main/CONTRIBUTING.md#translations

OMG
I wrote for three hours...

the json file you made can be imported through that tool


chore: Rename UserDefinedSQLPlanner to ExprPlanner

Which issue does this PR close?

Closes https://github.com/apache/datafusion/issues/11304
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@alamb @samuelcolvin fyi

merging up to resolve a conflict...


[Feature] Always Show Accounts Option & Hide Account Pictures

Requesting the option to be able to remove the dropdown area to select accounts, and simply have them listed condensed above the "unified mailbox" option.
Also requesting the ability to hide the circular picture icons next to the mailboxes.

I don't want to add these suggestions. You might be interested in #5570, though.

that would be fine


allow namespaces

the original exec comes with a handy namespace. it allows for the addition of trusted extra code
     # Dictionary to store definitions from the script
    script_namespace = {'myFunction': myFunction}

    # Execute the script within the module's namespace
    exec(script_content, script_namespace)

it would be handy to allow for extra functions to be imported within the execution context

Hmmmm. I'm not certain as to how implement this. I believe the args passed to the WASM execution environment are converted to string representations. I'll take a look at this further. Thanks for opening this!


Handle signals forwarded by child processes?

https://github.com/sashahart/vex/commit/7677c1e21889386486c7d425a1a392914cffa3c5

I ended up implementing this pretty much as per what is in vex, but with a finally to restore the default signal handler. And I also added a preexec_fn to restore the signal handler in the child process (which doesn't happen by default).
Will put together a PR if anyone wants it?


v1.0.0

Major changes

#144

API change, monitor and unmonitor are now async def functions

Bug Fixes

#149
#152
#161

Backward compatable as the interface did not effectively change.

#164

Some internal methods were made async. As these methods are internal, they are not covered by SEMVER.

#170

Features

#156
#165

Misc

#150  - Typo
#171  - Update supported versions documentation
#168  - Updates to the test library

As per SEMVER, #144 Changes the public API in a backward-incompatable way. I think this incurs the SEMVER major release rule.


Display relative path names to buffers

#18

LGTM, Ill make sure in the morning and test it out. Thanks for the PR!

This code should probably be around https://github.com/leath-dub/snipe.nvim/blob/main/lua/snipe/init.lua#L305 instead as annotate_with_tags is agnostic to the producer.

I moved it to get_buffer_name, is that what you had in mind?

This is good, thanks for the contribution !


Could not resolve all files for configuration ':react-native-svg:debugCompileClasspath'.

Could not resolve all files for configuration ':react-native-svg:debugCompileClasspath'.

hii @kundannetsetDemo I want to work on this issue please assign it to me

@last-hop

hii @kundannetsetDemo  I was busy with some other stuff , I will surely look into it today .

Same error here

This is what my repositories looks like:
repositories { google () mavenCentral()

@marvelous-ikechi what did you changed then , I tried by adding maven { url 'https://maven.google.com' } this , but app is just running normal  and  "dependencies": {
"react": "18.2.0",
"react-native": "0.73.1",
"react-native-reanimated": "3.6.1",
"react-native-svg": "^14.1.0"
},

@marvelous-ikechi @kundannetsetDemo  i have added a maven { url 'https://maven.google.com' } and before that i cleand the project and re install node modules  and its running in my system i am not getting any build error here ,  also what version of SVG you guys used ..?

Adding that fixed it now. Just checked again and it works. I had abandoned it @last-hop

alllRight  !! Hope that works  @kundannetsetDemo please Let me know


Multi-task classification of sequences containing strong motifs for multiple RBPs

Hello,
Thanks for putting together this resource and organizing these methods in a standardized way, it's been very helpful as someone new to machine learning classification.
I was wondering if I could get your advice on which RBP-RNA binding, classification method might be able to provide multiple positive classifications/ high scores for different binding proteins on the same sequence input. I'm currently trying to classify cassette exons as potentially under the regulation of different RBPs, by identifying binding motifs in their flanking introns. Using DeepRiPe and training on the Encode CLiP-seq datasets, this has worked well to some degree for a handful of RBPs I've looked at - identifying known regulatory relationships between binding and exon differential splicing. In some cases however, exon flanking regions contain multiple strong motifs  for different RBPs - a common case being the antagonistic relationship between QKI and PTBP1. With DeepRipe, regions known to bind both, with both motifs visible in the sequence, end up with high classifications scores for only a single RBP. The outputs of DeepRipe appear to pick only one of the RBP labels over the others in all cases, I'm not sure if this is something intrinsic to how these multi-task classifiers are set up, or perhaps due to imbalances in the training data, but ideally both clearly visible motifs would be identified and receive scores passing some threshold by which both could be properly identified.
Any advice, suggestions on which methods may work better, or pointers to how the multi-task code could be modified, would be greatly appreciated. Happy to discuss by email, off GitHub as well, if this more general question doesn't fit the criteria of an issue.
Thanks again,
Derek

Hi @dbogdano - thanks for your interest and apologies for the late reply! For multi-label classification, like in your case, I would indeed recommend using DeepRiPe or Multi-resBind, both have been performing well on our benchmark. Unfortunately I don't have an immediate explanation for your observations, but it sounds like something that should be further investigated. Have you checked whether this is due to erroneous predictions of the model, or whether the eCLIP data shows a similar picture? As a first step one could intersect the crosslink sites (see the peaks.crosslink.bed files in the  eCLIP directory) of the proteins you suspect to bind in the exon flanking regions and check whether this is correctly recovered by eCLIP.
As for the DeepRiPe model code, you can check in the author's repository. I believe the model described in their paper is referring to the code here.
Hope this helps and please don't hesitate to follow up if you encounter other issues!

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

Yes, the models are naturally biased towards the experimental condition of the training data and predicted binding probabilities are to be loosely interpreted as "prob. of binding if the given sequence were to be present in the experimental cell type". I'm glad you've found this repo useful!

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $1.500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1.500k in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @


scripts/gen_gpio_defaults.py is not working in caravel-redesign-2

The script gen_gpio_defaults.py outputs are identical mag files of the original gpio_defaults_block.
I am working on the branch caravel-redesign-2. The original gpio_defaults_block layout was updated by me.

There were several updates that addressed this problem, but the most recent one is pull request #441 in the main branch which corrects one last error and verifies that all the resulting GPIO blocks are correct (with a script in the scripts/ directory).


Incomplete collisions in Arcade when pushing objects

Phaser Version: v3.16.0 Beta 2
Physics: Arcade
Browser: Chromium

There is a collision bug when 1 object pushes another - at some moment objects start to go through each other, like there is no collision at all.
In the example code below (for labs.phaser.io) I have 2 ground platforms and all stars falling on it, and a player. Plus collisions between everything, even between stars themselves. Player initially is in the middle of the screen and stars.
If I move to the right - stars go through the right platform like there is no collision - it's a 1'st issue.
If I move to the left - I'm able to join only 2 stars - all other acts like physicsless and player/stars just go through them. So you get max 2 stars in a row - it's a 2'nd issue.
What can I do to make collisions always work as expected? Can you please fix that?
Example Test Code
var config = {
    type: Phaser.AUTO,
    width: 800,
    height: 600,
    physics: {
        default: 'arcade',
        arcade: {
            gravity: { y: 300 },
            debug: false
        }
    },
    scene: {
        preload: preload,
        create: create,
        update: update
    }
};

var player;
var stars;
var platforms;
var cursors;

var game = new Phaser.Game(config);

function preload ()
{
    this.load.image('sky', 'src/games/firstgame/assets/sky.png');
    this.load.image('ground', 'src/games/firstgame/assets/platform.png');
    this.load.image('star', 'src/games/firstgame/assets/star.png');
    this.load.spritesheet('dude', 'src/games/firstgame/assets/dude.png', { frameWidth: 32, frameHeight: 48 });
}

function create ()
{
    this.add.image(400, 300, 'sky');

    platforms = this.physics.add.staticGroup();

    platforms.create(400, 568, 'ground').setScale(2).refreshBody();
    platforms.create(900, 520, 'ground');

    player = this.physics.add.sprite(300, 450, 'dude');
    player.setCollideWorldBounds(true);

    this.anims.create({
        key: 'left',
        frames: this.anims.generateFrameNumbers('dude', { start: 0, end: 3 }),
        frameRate: 10,
        repeat: -1
    });

    this.anims.create({
        key: 'turn',
        frames: [ { key: 'dude', frame: 4 } ],
        frameRate: 20
    });

    this.anims.create({
        key: 'right',
        frames: this.anims.generateFrameNumbers('dude', { start: 5, end: 8 }),
        frameRate: 10,
        repeat: -1
    });

    cursors = this.input.keyboard.createCursorKeys();

    stars = this.physics.add.group({
        key: 'star',
        repeat: 11,
        setXY: { x: 12, y: 0, stepX: 70 }
    });

    this.physics.add.collider(player, platforms);
    this.physics.add.collider(stars, platforms);

    this.physics.add.collider(player, stars, collideElem, null, this);
    this.physics.add.collider(stars, stars, collideElem, null, this);

}

function collideElem(obj1, obj2) {
    obj2.body.setVelocityX(0);
    obj2.body.setVelocityX(0);
}

function update ()
{
    if (cursors.left.isDown)
    {
        player.setVelocityX(-160);
        player.anims.play('left', true);
    }
    else if (cursors.right.isDown)
    {
        player.setVelocityX(160);
        player.anims.play('right', true);
    }
    else
    {
        player.setVelocityX(0);
        player.anims.play('turn');
    }

    if (cursors.up.isDown && player.body.touching.down)
    {
        player.setVelocityY(-330);
    }
}

If that's technically and theoretically not possible in Arcade - please add a physics switch to disable energy exchange, so one object can't push the other objects (just stops instead).

The immovable switch is for this.

I think it still wouldn't work as you need and you should switch to using Matter.

Also: https://photonstorm.github.io/phaser3-docs/Phaser.Physics.Arcade.Body.html#customSeparateX__anchor

I'm Ok with replacing some Phaser's functions in my local environment, fortunately JavaScript easily allows it. I see that SeparateX does a part of job, but if I try to remove/modify lines which change velocity/position - I loose collisions or both objects are still moving. Probably something more needs to be changed. Could you guide me what do I need to edit in Phaser code to disable pushing? Just to stop an object at collision seems to be not a difficult task

One more note, that I'm sure is not worth of additional ticket. In SeparateX you have:
//  Can't separate two immovable bodies, or a body with its own custom separation logic
if (overlapOnly || overlap === 0 || ...
{
    //  return true if there was some overlap, otherwise false
    return (overlap !== 0) || (body1.embedded && body2.embedded);
}

.. is there a need to compare overlap !== 0 if it will always be zero?!

Thank you for submitting this feature request. We have implemented this and the feature has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.
Please see the new Body.pushable property and corresponding documentation.


Unable to run ockam relay create while following Quick start guide

Observed behavior
running ockam relay create return error OCK500
 Failed to create forwarder (origin: Api, kind: Invalid, source location: /project/implementations/rust/ockam/ockam_core/src/api.rs:144:44)
  help: Please report this issue, with a copy of your logs, to https://github.com/build-trust/ockam/issues

  Version 0.92.0, hash: 42124b03e6e979489d5be66862736fe7229ec147

stdout.log
stderr.log
Steps to reproduce
Follow Quick start guide
Expected behavior
relay is successfully initialized
Ockam Version
0.92.0

@adiSuper94 this is super helpful. Thank you.
Did the ockam enroll succeed for you?
If possible, could you share the output for the ockam enroll command. If you've lost the output, you can run:
ockam reset --yes
ockam

Did the ockam enroll succeed for you?

I think so. I entered the code in the browser, logged in using github and got  green tick ✅ with a success message.
In the terminal I waited for 1/2 a minute hoping to get a some ack. But i didn't so I assumed the command had successfully executed. And I CTRL-C exited from it.

I CTRL-C exited from it.
That would explain why its not working .. it can sometimes take a bit to provision your project, which then offers the relay service.

We should definitely improve the output and error messages to convey that better.
If you have some more time, please ockam reset and give the whole flow another try.

@adiSuper94 also when you get a chance please accept our CLA and add yourself to the Ockam Contributors list. We would love to acknowledge your contribution and feedback in our next release.

[ ] Accept the Ockam Contributor License Agreement by adding your Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a PR, this will make the commit as verified.

Even after waiting for a over a minute, ockam enroll did not return sucessfully.

It is, could you capture a screenshot of you screen

An one final try, if you're up for it:
ockam reset --yes
OCKAM_LOG=info ockam enroll

Capture the output and share with us please.
This will give us some more info on what might be happening.

@adiSuper94 could you also share screenshots of your terminal from after you entered things in the browser

This screen shot is taken after I entered the one-time code.
I ran ockam enroll again to see if generates any extra output. It did not write any log to the console for 7 mins.
Could you give me a link to an older version of ockam may 0.8x/0.7x. Cuz I remember it working on my machine earlier.

Hmm, interesting so nothing happened after you logged-in in your browser.
A message like this never came?
     │ Authenticated

     │ Email<EMAIL_ADDRESS>verified

We've had breaking protocol layer changes since older versions so they won't work with current version of Ockam Orchestrator.

Nope.

I also tried pasting the activate url in incognito tab, so that I can login manually.
I tried logging in through github, and by creating an account. Both did not work.

Interesting, super helpful.
That at least gives us a place to diagnose https://github.com/build-trust/ockam/blob/develop/implementations/rust/ockam/ockam_command/src/enroll/oidc_service.rs#L192

Ok. It worked.
I use ubuntu + i3 as my window manager. I did not work here.
But when I use ubuntu with the default window manager, it works.

At least it worked somehow, phew!

I use ubuntu + i3 as my window manager. I did not work here.

In that setting, could you try opening https://account.ockam.io/activate manually and entering the code and then coming back and pressing enter in the terminal.
I'm guessing the code to open in the browser might not be working
https://github.com/build-trust/ockam/blob/develop/implementations/rust/ockam/ockam_command/src/enroll/oidc_service.rs#L148

And now that I once logged in using plain ubuntu.
It is working in ubuntu+i3 as well!!
Why is this happening 😅?!

In that setting, could you try opening https://account.ockam.io/activate manually and entering the code and then coming back and pressing enter in the terminal.

I had tried doing that earlier. The ASCII progress load that comes on the terminal when you stuff is loading did not come.

Intriguing, you suspect something went wrong in the spinner code?
https://github.com/build-trust/ockam/blob/develop/implementations/rust/ockam/ockam_command/src/terminal/mod.rs#L429
If you're up for it, that would be a good thing to try and debug with a build of the current develop branch.

That sounds good.

@mrinalwadhwa you are right.

I'm guessing the code to open in the browser might not be working
https://github.com/build-trust/ockam/blob/develop/implementations/rust/ockam/ockam_command/src/enroll/oidc_service.rs#L148

This is exactly the place it is getting stuck.
open::that seems to be waiting for the spawned process to exit before returning on my machine(only on firefox).
open::that_detached seems to work perfectly for me, this create a fork and doesn't wait for the process to exit.
comments in src code of open::that
/// A [`std::io::Error`] is returned on failure. Because different operating systems
/// handle errors differently it is recommend to not match on a certain error.
///
/// # Beware
///
/// Sometimes, depending on the platform and system configuration, launchers *can* block.
/// If you want to be sure they don't, use [`that_in_background()`] or [`that_detached`] instead.
pub fn that(path: impl AsRef<OsStr>) -> io::Result<()> {

That's a great find 👏 👏
that_detached does seem like the better option for us here. @adrianbenavides @etorreborre what do you think?

That's pretty interesting! I'll check how this affects the desktop app. Maybe that was the reason of the problem we currently have about not being able to open the enroll url twice.


UIImageView+AFNetworking failure block

When loading a critical image resource, it's not possible to try again with the same URL since the af_imageRequestOperation is not nil.
Consider this code:
- (void)trySetImage {
  NSString *url = @"https://a248.e.akamai.net/assets.github.com/img/adfc0bef364ef82cd0a2bf395ed02fe56619d376/687474703a2f2f656e67696e656572696e672e676f77616c6c612e636f6d2f41464e6574776f726b696e672f61666e6574776f726b696e672d6c6f676f2e706e67";

  NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:[NSURL URLWithString:url]
                                                         cachePolicy:NSURLRequestUseProtocolCachePolicy
                                                     timeoutInterval:30.0];
  [request setHTTPShouldHandleCookies:NO];
  [request setHTTPShouldUsePipelining:YES];
  [self.imageView setImageWithURLRequest:request
                        placeholderImage:nil
                                 success:^(NSURLRequest *_request, NSHTTPURLResponse *response, UIImage *image) {
                                 } failure:^(NSURLRequest *_request, NSHTTPURLResponse *response, NSError *error) {
                                   [self trySetImage];
                                 }];
}

trySetImage will only run once in case the image resource fails to load.

In order to get around this, I temporarily introduced this method in UIImageView+AFNetworking.
- (void)forceSetImageWithURLRequest:(NSURLRequest *)urlRequest
              placeholderImage:(UIImage *)placeholderImage
                       success:(void (^)(NSURLRequest *request, NSHTTPURLResponse *response, UIImage *image))success
                       failure:(void (^)(NSURLRequest *request, NSHTTPURLResponse *response, NSError *error))failure {
  self.af_imageRequestOperation = nil;
  [self setImageWithURLRequest:urlRequest placeholderImage:placeholderImage success:success failure:failure];
}

I just pushed changes in f8fd95e26a4f6cc961f63f2c00daa0bed5bfa81d that should address the concerns about the UIImageView category (obeys cacheing policy, gets rid of image request operation after completion).
Please let me know how they work for you.

Thanks Mattt, works great so far!

We really appreciate it!


In Tekton examples, bump Triggers to v0.16.1 with template updates

generateName appends a unique string at the end and restricts the name to
63 characters.
On Tue, Nov 23, 2021 at 10:07 AM bitsofinfo @.***>
wrote:

@.**** commented on this pull request.
In examples/tekton/pipelines/test/triggers.yaml
https://github.com/bitsofinfo/cicdstatemgr/pull/13#discussion_r755217838
:

resourcetemplates:
- apiVersion: tekton.dev/v1beta1
kind: PipelineRun
metadata:

   name: test-$(tt.params.cicdContextName)-$(tt.params.appName)-$(uid) # uid here is the tekton unique RUN id

   generateName: test-$(tt.params.cicdContextName)-$(tt.params.appName)-

Doesn't this restrict only one run at a time? this isn't unique anymore
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bitsofinfo/cicdstatemgr/pull/13#pullrequestreview-813792168,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIYXGYSC23I7JXMECPWWETUNOU2XANCNFSM5ISQFKQQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Add support for tox 3.7.0

With the new tox version out, venv._getresolvedeps() was completely removed in favor of venv. get_resolved_dependencies.
As of now, the following error will raise when trying a tox run
py36 recreate: /app/.tox/py36
Traceback (most recent call last):
  File "/usr/local/bin/tox", line 11, in <module>
    sys.exit(cmdline())
  File "/usr/local/lib/python3.6/dist-packages/tox/session.py", line 47, in cmdline
    main(args)
  File "/usr/local/lib/python3.6/dist-packages/tox/session.py", line 54, in main
    retcode = build_session(config).runcommand()
  File "/usr/local/lib/python3.6/dist-packages/tox/session.py", line 467, in runcommand
    return self.subcommand_test()
  File "/usr/local/lib/python3.6/dist-packages/tox/session.py", line 590, in subcommand_test
    self.run_sequential()
  File "/usr/local/lib/python3.6/dist-packages/tox/session.py", line 597, in run_sequential
    if self.setupenv(venv):
  File "/usr/local/lib/python3.6/dist-packages/tox/session.py", line 509, in setupenv
    status = venv.update(action=action)
  File "/usr/local/lib/python3.6/dist-packages/tox/venv.py", line 222, in update
    self.hook.tox_testenv_install_deps(action=action, venv=self)
  File "/usr/local/lib/python3.6/dist-packages/pluggy/hooks.py", line 284, in __call__
    return self._hookexec(self, self.get_hookimpls(), kwargs)
  File "/usr/local/lib/python3.6/dist-packages/pluggy/manager.py", line 68, in _hookexec
    return self._inner_hookexec(hook, methods, kwargs)
  File "/usr/local/lib/python3.6/dist-packages/pluggy/manager.py", line 62, in <lambda>
    firstresult=hook.spec.opts.get("firstresult") if hook.spec else False,
  File "/usr/local/lib/python3.6/dist-packages/pluggy/callers.py", line 208, in _multicall
    return outcome.get_result()
  File "/usr/local/lib/python3.6/dist-packages/pluggy/callers.py", line 80, in get_result
    raise ex[1].with_traceback(ex[2])
  File "/usr/local/lib/python3.6/dist-packages/pluggy/callers.py", line 187, in _multicall
    res = hook_impl.function(*args)
  File "/usr/local/lib/python3.6/dist-packages/tox_pipenv/plugin.py", line 80, in tox_testenv_install_deps
    deps = venv._getresolvedeps()
AttributeError: 'VirtualEnv' object has no attribute '_getresolvedeps'

[root@localhost bin]# tox --version
['--version']
3.13.2 imported from /usr/lib/python2.7/site-packages/tox/init.pyc
registered plugins:
tox-pipenv-1.9.0 at /usr/lib/python2.7/site-packages/tox_pipenv/plugin.pyc
tox-venv-0.4.0 at /usr/lib/python2.7/site-packages/tox_venv/hooks.pyc
with 1.9.0
the same issue


Ref docs for Storage blobs - sample code for setting content type for upload

Package Name: @azure/storage-blob
Package Version: 12.2.1
Operating system: Win10-internal
[ ] nodejs

version:

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[X ] SDK API docs on https://docs.microsoft.com

Describe the bug
The page describing the options should have an example - content-type for uploaded files is all over SO - seems like a good one to include - that someone knows how to set the most common properties. Same options could probably be added to several option objects.
For example: BlockBlobUploadStreamOptions
The samples would include code like this SO answer, specifically this line:
const blobOptions = { blobHTTPHeaders: { blobContentType: 'text/plain' } };

Thanks for the suggestions! Contribution is also welcome!

@XiaoningLiu Is that just a PR to this repo?

Yes, the API reference like https://docs.microsoft.com/en-us/javascript/api/@azure/storage-blob/blockblobclient?view=azure-node-latest#query_string__BlockBlobQueryOptions_ are generated from source code. Add an sample into source code JSDoc is enough.

Fixed with #12938


driver.get not navigating to url

chrome opens successfully but it doesn't navigate to the url.  repeating a few times it eventually navigates.
botasaurus_driver = 4.0.66
chrome = 130.0.6723.70 (official build)

to temporarily solve.  i make the attempt and check the driver.current_url with url i want.  if they are not equal i close the browser, sleep for a bit and attempt again.

shame issues!


Responsive animation settings

I am using vanilla window.matchMedia to change the animation settings by breakpoint taking advantage of $tabs.responsiveTabs('option', 'animation', 'fade');
This works really well by the way !!
However if I have 3 breakpoints and I switch between fade, slide and default when default is set it is changing my original duration to 0
Can I suggest rather than overwriting this globally ie _this.options.duration = 0 this is just overwritten at the function level ie disabled: 0?
I can submit a PR if it would help?
What I have in mind would be a non breaking change

Feel free to submit a PR

I'm closing the issue since there has been no activity for a while. Feel free to reopen if you feel this is still unresolved.


Required space seems to be too large on zopen upgrade

When issuing zopen upgrade curl, I noticed that the message showing the newly required disk space seems to be too large.  I find it hard to believe that upgrading curl would require 14MB of additional disk space over the prior version.  Perhaps the code isn't subtracting the existing disk space?
angio ~> zopen upgrade curl
- Querying remote repo for latest package information
Installing package: curl
- Replacing curl version '<IP_ADDRESS>30912_215945' with '<IP_ADDRESS>31011_155722'
- Downloading curl-<IP_ADDRESS>31011_155722.zos.pax.Z file from remote to zopen package cache...
- Downloaded curl-<IP_ADDRESS>31011_155722.zos.pax.Z file from remote to zopen package cache.
- Processing curl-<IP_ADDRESS>31011_155722.zos.pax.Z...
After this operation, 14.53 MB of additional disk space will be used.

I expect you need both installed for a brief time before deleting the old one sont you? So you would want to include the space of the current one too? But - that’s not my code. @DevonianTeuchter ?

The calculation should be (space of new - space of current), and the sign of the result will decide on the adjective in the message (used vs. saved)

So total usage to do install is not something we should communicate (to ensure file system doesn’t croak?)

I don't think we can communicate that value properly, as by the time we're going to measure it, the new pax has already been expanded, and it's using disk space somewhere.  All that we can provide at this point is how much more/less space will be consumed after the install completes.

I believe the upgrade process is a bit different than other packages managers in that it installs a new version of the tool and doesn't replace the current version. @DevonianTeuchter Is this correct? Do we have an option to automatically remove old releases?
We also probably need to consider size of the pax file as since we are downloading it to the file system. Both the expanded size and pax file size are in the metadata:
https://github.com/ZOSOpenTools/meta/blob/main/docs/api/zopen_releases.json#L13-L14

Then perhaps we should think about this more.  Upgrading curl with a new version is fine, but I see that the existing insecure version is still left in place.  Perhaps we should remove old versions by default?

Perhaps a prompt (or option) that asks the user if they want to keep older versions?

Then perhaps we should think about this more. Upgrading curl with a new version is fine, but I see that the existing insecure version is still left in place. Perhaps we should remove old versions by default?

And perhaps we can add some smarts so that it doesn't remove personal builds via zopen build or some way to toggle it.

or a parameter such as -kill :)

Currently, the upgrade process for a package will:

query the cache for details about the release
if an update is available, then it will download the pax.Z to the local cache
reports the new size.
unpaxes and moves to the install location
switches around symlinks
It does not remove the previous version by default - that was the initial design of the package manager, to allow for [up/down]grading at will without a full reinstall, just using zopen-alt to switch symlinks - so that space will still be used. There would also be space taken up by the pax in the cache [if we're getting into that level of detail!].  The zopen clean command is there to remove older versions and reclaim space.
We could add a parameter on the upgrade (defaulting to [keep-old/remove-old]?) and a global configurable property [ in the etc/zopen heirarchy with other global properties so easily tweakable]. When upgrading with the "remove-old" parameter set, we can use make sure to use the correct terminology/adjectives.
Personal builds will not appear in the release-metadata so we can validate whether something is in that metadata before we uninstall [and possibly ask before we upgrade as a side-project since we will always upgrade!]

i personally like zopen clean being separate. One tool for one task Unix model.

So - currently this is 'working as designed', so closing this. @AnthonyGiorgio if you want some sort of 'install and clean' then please open a separate issue, in which case ideally we would want to indicate MAXIMUM disk usage and AFTER INSTALL disk usage.


Operation System typo

Is it "Operation System" or "Operating System", wanted to confirm before making changes to the readme file

Hi Iniubong, @codiini
Thank you for your message and for reminding me of this typo mistake.
In the fact, it's a new OS. But it will be a Linux Distribution.
It's a matter of whether Linux distribution is an operating system or not. (Oops)
It depends on how we develop and how far we go in the future.
I wonder how you found this organization? :)
Because it has not been made public yet and it is not even ready.
Although we published various scripts for compiling the kernel and ....
Regards,

i'm happy to help in a small way, not sure i could write a Linux distribution OS or contribute code to it except we're talking about the website, I'm a Front end Dev.
It just showed up on my feed that you created a new repo and i decided to check it out.
I think it's really cool what your'e building.
would DonaOS have any significant difference from other Linux Disros?

Unlike a binary software distribution, the source code is compiled locally according to the user's preferences.

Even the smallest contribution counts we would love to have you as a part of the community


console.log in renderer process only prints the first argument in stdout

Expected behavior, to be able to call something like:
console.log(arg1, arg2, arg3) and see the output in stdout separated by commas.
Actual behavior is that only the first argument is output.

yup was observing this for quite sometime wasnt sure of the cause though, as we just print the message obtained from the webcontents here

this issue is a duplicate of https://github.com/atom/atom-shell/issues/621

I did search for open issues but I guess the previous was closed. I think that at least one of these issues should be left open - perhaps tagged as blocked upstream?

We now disabled Chromium's logging completely and nothing from renderer will be printed anymore by default.