Prerequisite Steps:
1. Make sure your provider has community provider attributes:
  Example:
  $ provider-services query provider get akash1<REDACTED> -o text
  ...
  attributes:
  ...
  - key: host
  value: akash
  - key: tier
  value: community

Ref documentation:.
2. Make sure your provider *.ingress resolves to your provider IP (ideally worker node IP)
host ingress.aresakash.tech (ip <IP_ADDRESS>)

Example:
$ host anything.ingress.akash.pro
anything.ingress.akash.pro is an alias for nodes.akash.pro.
nodes.akash.pro has address <IP_ADDRESS>

More info on what DNS records should look like are available in the Akash Documentation here.

3. Please make sure your Akash provider doesn't block any Akash specific ports.

If you are using a firewall, please follow this doc:
https://docs.akash.network/providers/build-a-cloud-provider/akash-cloud-provider-build-with-helm-charts/step-9-firewall-rule-review
If you are behind NAT, then you need to make sure these ports are open between your provider worker nodes and the internet.

Audit Steps:
1. Title the issue: " [Provider Audit]: Provider Address" (e.g. "[Provider Audit]: provider.europlots.com")
2. Wait for response via comments. If no issues during provider Audit, process will be complete, provider should start bidding on leases, and Audit ticket will be closed.
3. If there are issues during the provider Audit, debug those issues, and Audit will be complete.
4. Audit Issue will be closed by core team member.
Leave contact information (optional)

Name (Vladyslav)
Discord handle or Telegram handle (Dis Jolandemo#7347
Contact email address<EMAIL_ADDRESS>

Hi @doggyspacer  :wave:
It looks like your provider isn't seeing my deployment order requests - dseq 15209169.
Please check if your RPC node is syncing with the tip of the chain.
Can you please pass me the output for these commands:
kubectl -n akash-services get pods

kubectl -n akash-services get pods -o custom-columns='NAME:.metadata.name,IMAGE:.spec.containers[*].image'

kubectl -n akash-services exec -ti statefulset/akash-node-1 -- akash status

kubectl -n akash-services get statefulset/akash-provider -o yaml |grep -A1 AKASH_NODE

kubectl -n akash-services logs statefulset/akash-provider --tail=10000000 | grep -E '15209169'

Please send me the provider logs too, this is done in two steps:

Create archive with your provider logs

kubectl -n akash-services logs statefulset/akash-provider --tail=10000000 | gzip -c > provider.log.gz

Upload that archive to transfer.sh using this CLI command:

curl -T provider.log.gz https://transfer.sh

The curl -T ... command will give you the output - the link where I can download your provider logs from. Please share this link with me here.
Current attributes:
$ provider-services query provider get akash1ejh0cjczhywcaw32dplngugvpgg63takurp3qj -o text
attributes:
- key: organization
  value: arescc
- key: arch
  value: amd64
- key: host
  value: akash
- key: tier
  value: community
host_uri: https://provider.aresakash.tech:8443
info:
  email: ""
  website: ""
owner: akash1ejh0cjczhywcaw32dplngugvpgg63takurp3qj

kubectl -n akash-services get pods
NAME                                      READY   STATUS    RESTARTS      AGE
akash-hostname-operator-6795445db-h8nb5   1/1     Running   7 (16h ago)   7d20h
akash-provider-0                          1/1     Running   0             70m
kubectl -n akash-services get pods -o custom-columns='NAME:.metadata.name,IMAGE:.spec.containers[*].image'
NAME                                      IMAGE
akash-hostname-operator-6795445db-h8nb5   ghcr.io/akash-network/provider:0.4.8
akash-provider-0                          ghcr.io/akash-network/provider:0.4.8
kubectl -n akash-services exec -ti statefulset/akash-node-1 -- akash status
Error from server (NotFound): statefulsets.apps "akash-node-1" not found
kubectl -n akash-services get statefulset/akash-provider -o yaml |grep -A1 AKASH_NODE
- name: AKASH_NODE
value: http://<IP_ADDRESS>:26557
    - name: AKASH_NODE
      value: http://<IP_ADDRESS>:26557

kubectl -n akash-services logs statefulset/akash-provider --tail=10000000 | grep -E '15209169'
Defaulted container "provider" out of: provider, init (init)
https://transfer.sh/dkn9wbTou5/provider.log.gz
maybe it just coincided that at that moment +- there were several rentals at once

Got the bid! dseq 15297464
Perhaps it just takes slightly longer time than usual for your provider to bid..
Might monitor your RPC node and I recommend you to deploy your own RPC node eventually.
I've tested & signed your provider.
tx hash 06A0A7AB790AED7EE6632CDA7B3296886DAAE6FC794DCC48867F74C8A56C7782


OTEL_EXPORTER_OTLP_TIMEOUT contains ms

Version report
Jenkins and plugins versions report:
Latest release 0.19 with the latest jenkins LTS 2.289.2

What Operating System are you using (both controller, and any agents involved in the problem)?

Docker
Reproduction steps
Start your jenkins instance with the latest plugin version:
pipeline {
    agent any
    stages {
        stage('Checkout') {
            steps {
                git(branch: 'feature/add-maven-traces', credentialsId: '2a9602aa-ab9f-4e52-baf3-b71ca88469c7-UserAndToken', url: 'https://github.com/v1v/apm-agent-java.git')
                sh(label: 'mvn extension', script: '.ci/otel-maven.sh')
            }
        }
        stage('Clean') {
            steps {
                sh(label: 'mvn clean', script: "./mvnw -Dmaven.ext.class.path=.mvn/opentelemetry-maven-extension.jar clean")
            }
        }
    }
}

Results
Expected result:
No failures
Actual result:

OTEL_EXPORTER_OTLP_TIMEOUT=30000ms

As far as I see:

The maximum waiting time, in milliseconds, allowed to send each OTLP trace and metric batch. Default is 10000.

in https://github.com/open-telemetry/opentelemetry-java/blob/main/sdk-extensions/autoconfigure/README.md


Toolbar tooltips lock current selection

Godot version
4.1.beta2
System information
Windows 11, Forward+, RTX 3080
Issue description
If a tooltip of one of the items in the main toolbar features tooltips, those tooltips appearing for a selection will lock that selection in & hovering over other options won't have any effect. Canceling out of the selection through other means (clicking a spacer) will return the selection list to standard behavior
Steps to reproduce
https://github.com/godotengine/godot/assets/17489292/14718381-def2-4183-aa1c-995a53447185
Minimal reproduction project
N/A

Can you please check, if this is a duplicate of #73926?

Potentially? I don't know if progress was made since, but in my case the only area affected was the toolbar. Other areas, like the inspector, handle tooltips without issue
https://github.com/godotengine/godot/assets/17489292/6f75627a-0257-4c74-972f-bb8917a2cf1c

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/73926.


[WIP]Add shell and logs actions for container in pod detail page

https://github.com/rancher/rancher/issues/12656

@loganhz can you revisit this and the comments?

Close this PR first as it has backend dependency. The type is still  /v3/project/schemas/container in rancher/rancher:master


Feature/frontend

Adds SvelteKit app with minimal UI, basic auth with e-mail/password.

was able to register and authenticate, I failed the password puzzle (could probably usually a hint somewhere on requirements). But more of a future thing.
lgtm


Add major tag v2

What do you think about adding a tag v2 which points to the latest v2 release?
This tag could then also be used in the examples in the README, so that it does not have to be updated with every release.
There is a major tag for v1.

There's already a v2 branch.

What do you think about editing to examples to use pnpm/action-setup@v2?
/cc @zkochan

We don't frequently release new versions, so I don't really see the point.

I came here to ask about this very thing -- I was confused as to why this GitHub Action suggests specifying an exact (X.Y.Z) version (currently<EMAIL_ADDRESS>with one exception*), when most GitHub Actions seem to suggest specifying only the major version, e.g. actions/checkout@v3.
Would it be difficult to start tagging releases with v2 in addition to e.g. v2.2.4? (Hopefully I'm correct in assuming that would resolve this issue)

*: While I was poking around, noticed one of the three was out of sync: https://github.com/pnpm/action-setup/pull/64

Just a stupid question: I'm using "pnpm/action-setup@v2" and that means I'm not using th latest one? :O

@pitgrap You are using the latest one.

@KSXGitHub, I think v2 is currently even with v2.2.3, so slight behind v2.2.4:

https://github.com/pnpm/action-setup/compare/v2...v2.2.4
https://github.com/pnpm/action-setup/compare/v2...v2.2.3

(but maybe I'm missing something)

@mrienstra I see. It seems @zkochan forgot to update v2 when he published v2.2.4. Anyway, v2 is up-to-date now.

There are two parts to this issue:

v2 should be kept current: it sounds like this is the intention and generally the case, so it seems like that part of this issue has been resolved for now.
Changing README to show pnpm/action-setup@v2 (drop minor & patch version numbers): where did we land on this? Personally, I think it would reduce confusion. But if the official word is "not changing at this time", then I think this issue can be closed.

@mrienstra I don't object your second point.

Optimistically opened a PR. :)

Thank you, guys!


thanks and quick feature requests

this is just awesome. been looking for a tui for github for ages. now it's finally here, thanks a lot for your work.
i'm loving browsing github in this.
just a small feature request:
would it be possible for the cycle keybind (ctrl+n/p) to only cycle issue titles and message titles and not the top filter (maybe make a new shortcut just for the filter bar) or the four top-left windows (assignees/lables/milestones/projects)? just because it would be faster to cycle through them...
edit: never mind this request. been using it it more and it seems to work great as is. it's simple and it works.
also, maybe have an option for having favorite repos for quick access.
and an option to also show the "owner" column before the "repo" column.
again, thanks for your work.

just a small feature request:
would it be possible for the cycle keybind (ctrl+n/p) to only cycle issue titles and message titles and not the top filter (maybe make a new shortcut just for the filter bar) or the four top-left windows (assignees/lables/milestones/projects)? just because it would be faster to cycle through them...

I'm considering how focusing  panels quickly.
Please wait a little :)

also, maybe have an option for having favorite repos for quick access.

Sounds good.
I'm considering feature of bookmark to quick access your favorite repository.

and an option to also show the "owner" column before the "repo" column.

What kind of do you have?

I'm considering feature of bookmark to quick access your favorite repo
Cool, thanks. I'll be looking forward to that.

What kind of do you have?
I don't understand your question (language barrier maybe?).
What i mean is, instead of, for example, having:

Repo       Number State Author    Title
github-tui 1      OPEN  shanekira Title name here
would have this instead:
Repo                 Number State Author    Title
shanekira/github-tui 1      OPEN  shanekira Title name here

What kind of do you have?
I don't understand your question (language barrier maybe?).

Sorry, My English is bad haha

What i mean is, instead of, for example, having:
Repo       Number State Author    Title
github-tui 1      OPEN  shanekira Title name here
would have this instead:
Repo                 Number State Author    Title
shanekira/github-tui 1      OPEN  shanekira Title name here

Oh, I got :)
It looks like good, I'll implement it.

this is just awesome. been looking for a tui for github for ages. now it's finally here, thanks a lot for your work.
i'm loving browsing github in this.
just a small feature request:
would it be possible for the cycle keybind (ctrl+n/p) to only cycle issue titles and message titles and not the top filter (maybe make a new shortcut just for the filter bar) or the four top-left windows (assignees/lables/milestones/projects)? just because it would be faster to cycle through them...
edit: never mind this request. been using it it more and it seems to work great as is. it's simple and it works.
also, maybe have an option for having favorite repos for quick access.
and an option to also show the "owner" column before the "repo" column.
again, thanks for your work.

this is just awesome. been looking for a tui for github for ages. now it's finally here, thanks a lot for your work.
i'm loving browsing github in this.
just a small feature request:
would it be possible for the cycle keybind (ctrl+n/p) to only cycle issue titles and message titles and not the top filter (maybe make a new shortcut just for the filter bar) or the four top-left windows (assignees/lables/milestones/projects)? just because it would be faster to cycle through them...
edit: never mind this request. been using it it more and it seems to work great as is. it's simple and it works.
also, maybe have an option for having favorite repos for quick access.
and an option to also show the "owner" column before the "repo" column.
again, thanks for your work.

this is just awesome. been looking for a tui for github for ages. now it's finally here, thanks a lot for your work.
i'm loving browsing github in this.
just a small feature request:
would it be possible for the cycle keybind (ctrl+n/p) to only cycle issue titles and message titles and not the top filter (maybe make a new shortcut just for the filter bar) or the four top-left windows (assignees/lables/milestones/projects)? just because it would be faster to cycle through them...
edit: never mind this request. been using it it more and it seems to work great as is. it's simple and it works.
also, maybe have an option for having favorite repos for quick access.
and an option to also show the "owner" column before the "repo" column.
again, thanks for your work.

I implemented in https://github.com/skanehira/github-tui/pull/13

very fast, thanks!

What kind of do you have?
I don't understand your question (language barrier maybe?).

Sorry, My English is bad haha

What i mean is, instead of, for example, having:
Repo       Number State Author    Title
github-tui 1      OPEN  shanekira Title name here
would have this instead:
Repo                 Number State Author    Title
shanekira/github-tui 1      OPEN  shanekira Title name here

Oh, I got :)
It looks like good, I'll implement it.


fixgroup by expr may be bound twice in bind agg slot

Proposed changes
pick from master https://github.com/apache/doris/pull/28771

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

run buildall


Fixed bug parsing netmask cisco acl

[x] Redmine Issue: https://redmine.pfsense.org/issues/11569
[x] Ready for review

test case
<?php

function assertSame($expected, $actual, $message)
{
    if ($expected === $actual) {
	echo "$message\t ok\n";
	return;
    }

    echo "$message:\nFailed asserting that \n - - - \n'$expected'\n is identical to\n - - - \n'$actual'.\n";
    exit(1);
}

//test case #10469
$rules = parse_cisco_acl([
    'ciscoavpair' => [
	'ip:outacl#1=permit udp host <IP_ADDRESS> host <IP_ADDRESS> range 3110 5000',
	'ip:outacl#2=permit tcp any host <IP_ADDRESS> gt 333',
	'ip:inacl#3=permit udp host <IP_ADDRESS> host <IP_ADDRESS> lt 566',
	'ip:inacl#4=permit udp host <IP_ADDRESS> host <IP_ADDRESS> neq 899',
	'ip:inacl#6=permit icmp host <IP_ADDRESS> host <IP_ADDRESS>',
	'ipv6:outacl#1=permit udp host 2001:DB8::4444 host 2001:DB8::7777 range 3110 5000',
	'ipv6:outacl#2=permit tcp any host 2001:DB8::7777 gt 333',
	'ipv6:inacl#3=permit udp host 2001:DB8::3333 host 2001:DB8::7777 lt 566',
	'ipv6:inacl#4=permit udp host 2001:DB8::3333 host 2001:DB8::7777 neq 899',
	'ipv6:inacl#5=permit tcp host 2001:DB8::2222 host 2001:DB8::5555 eq 999',
	'ipv6:inacl#6=deny udp 2001:DB8:1234::/64 2001:DB8:6789::/48',
	'ipv6:inacl#7=deny icmp 2001:DB8:1234::/64 2001:DB8:9999::/56'
    ]
], 'ovpns1');

assertSame(<<<EXPECTED
pass in quick on ovpns1 inet proto udp from <IP_ADDRESS> to <IP_ADDRESS> port < 566 no state
pass in quick on ovpns1 inet proto udp from <IP_ADDRESS> to <IP_ADDRESS> port != 899 no state
pass in quick on ovpns1 inet proto icmp from <IP_ADDRESS> to <IP_ADDRESS> no state
pass out quick on ovpns1 inet proto udp from <IP_ADDRESS> to <IP_ADDRESS> port 3109 >< 5001 no state
pass out quick on ovpns1 inet proto tcp from any to <IP_ADDRESS> port > 333 no state
pass in quick on ovpns1 inet6 proto udp from 2001:DB8::3333 to 2001:DB8::7777 port < 566 no state
pass in quick on ovpns1 inet6 proto udp from 2001:DB8::3333 to 2001:DB8::7777 port != 899 no state
pass in quick on ovpns1 inet6 proto tcp from 2001:DB8::2222 to 2001:DB8::5555 port = 999 no state
block in quick on ovpns1 inet6 proto udp from 2001:DB8:1234::/64 to 2001:DB8:6789::/48 no state
block in quick on ovpns1 inet6 proto ipv6-icmp from 2001:DB8:1234::/64 to 2001:DB8:9999::/56 no state
pass out quick on ovpns1 inet6 proto udp from 2001:DB8::4444 to 2001:DB8::7777 port 3109 >< 5001 no state
pass out quick on ovpns1 inet6 proto tcp from any to 2001:DB8::7777 port > 333 no state

EXPECTED
, $rules, 'test case #9206');

//broken test
$rules = parse_cisco_acl([
    'ciscoavpair' => [
	'ip:inacl#1=permit tcp <IP_ADDRESS> <IP_ADDRESS> any',
	'ip:inacl#2=permit tcp <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> eq 80'
    ]
], 'ovpns1');

assertSame(
    <<<EXPECTED
pass in quick on ovpns1 inet proto tcp from <IP_ADDRESS>/32 to any
pass in quick on ovpns1 inet proto tcp from <IP_ADDRESS>/32 to <IP_ADDRESS>/32 port = 80

EXPECTED,
    $rules,
    'broken test v2.5.0'
);

@jim-p Removed all changes that are not related to the error


relative path import does not work when module is "system"

Here is an example:
https://github.com/coolcodeexamples/timely-wealth-game
Change tsconfing.json to module: "system"
and then run tsc.
The error is:
error TS2307: Cannot find module './App.ts'.
Please ignore the "Module ''domready'' has no default export" when compiling in .commonjs or in system with 1.7.5. That's related to systemjs (I'm using JSPM to run so it is working correctly in both count).

Seems like resolving relative path only works for commonjs. system, amd, umd, es6/es2015 all report the same error.

@unional You can set "moduleResolution": "node" while still use "module": "system"

Excellent!  Is there any doc related to moduleResolution? I only see it in the compileOptions but it doesn't mention what it does (and how it works).

All doc of the compiler options can be found on this wiki page. Basically it decides how to resolve modules


fix:issue515 I want to be able to go to GitHub and Twitter and etc.. …

…of the speaker in SpeakerDetailFragment .
Issue

close #515

Overview (Required)

If click speaker image, show SNS link button on bottomSheet.

Links

Screenshot
after

Almost LGTM 👍
If you can, please cherry-pick this?
https://github.com/DroidKaigi/conference-app-2018/commit/519bc4ac613a6ce581058dbd52207f840ce8a9c0

Thanks 👍 LGTM

Thank you for very useful feature 🥇

Thank you for refactoring!! I cherry-picked it!! 😄


Update MatrixMapiComparison-report-github.md

Fixed Markdown table

Hello, @mateuszlewko, and thank you for your interest.
Although, these tables are generated automatically by the script and therefore use BenchmarkDotNet markdown format. If you want to fix the formatting for some reason, I recommend you to either

fix the issue within BenchmarkDotNet, or
fix the script to add | to every table line

Also, is there a concrete reason this is bothering you? Some external Markdown renderer doesn't render this properly? Please clarify.

Hi,
GitHub doesn't render it properly and I wanted to see the benchmarks, so I edited the file for myself but then I felt that it's worth doing PR (so others may see the results of benchmark without leaving GitHub). That's all.

Sorry, now I can see the issue; the tables are really rendered incorrectly. I'll fix the script in the meantime, and I accept your PR now. Thanks again!


Tighten native node modules cleanup rules

@Tyriar Please merge if the node-pty change makes sense (omitting the deps folder when shipping, for both VS Code and the server). I built a Linux build and everything worked fine when testing it, so it LGTM: https://az764295.vo.msecnd.net/insider/bfcbc20faecc18d514bec2036b8cca48bcf27372/code-insider-1559121823.tar.gz


Try miniforge

Mambaforge is being sunsetted, so we have to switch to miniforge3 for the github workflows. Not sure if this will fix the problems in build.yml as well.
https://conda-forge.org/news/2024/07/29/sunsetting-mambaforge/

Nice! Looks like tests passed on the last one?


Memory can provide fine-grained control

google translation, we non-English speaking countries  :)
My team fell in love with NEDB!
We tried to use these databases：
sqlite
EJDB
Firebird
H2
HTML5 indexedDB
Leveldb
LowDB
PouchDB
StoreDB
and NeDB！
We feel NEDB best for us.
it's very flexible and very fast.
We hope you will stick with it
We really need him.
But there are disadvantages that consume too much memory. We use this database on the client, our software memory footprint is 40M, but our ultimate stress test data, our software footprint is 110.
I hope you can provide a method for controlling a memory so that we adjust memory usage, because not all of the data takes a long time to load in memory.
Thank you for your work!

As of v1.4, NeDB automatically selects the best method to persist data on the browser (IndexDB then WebSQL then localStorage), so the limit is removed. WARNING: it's not back compatible with v1.3, but that should not be a major issue as user apps can resync entirely.


Fixes unholy package.json clobbering RN version.

The unholy template copies it's package.json overtop of the one installed by React Native so we end up losing the user's choice.
This fixes that.

good catch.


Date time col name

pm_annual_caaqs didn't seem to like values for dt that aren't date_time. This PR is meant to fix that and add a test. Here is the bug reproduced:
library(rcaaqs)
library(dplyr)
#>
#> Attaching package: 'dplyr'
#> The following objects are masked from 'package:stats':
#>
#>     filter, lag
#> The following objects are masked from 'package:base':
#>
#>     intersect, setdiff, setequal, union

pm_annual_caaqs(pm25_sample_data, by = c("ems_id", "site"))
#> Calculating PM 2.5 daily average
#> Calculating PM 2.5 annual average
#> Calculating PM 2.5 annual CAAQS metric
#>
#> CAAQS results for PM2.5 annual
#>   * Includes CAAQS ambient results only
#> --------------------------------------
#> 4 elements:
#>       * daily_avg ( 10901 rows x 8 columns )
#>       * yearly_avg ( 30 rows x 14 columns )
#>       * three_yr_rolling ( 30 rows x 21 columns )
#>       * caaqs ( 30 rows x 12 columns )
#> Access them with the get_ functions (see ?get_caaqs)

pm25_sample_data %>%
  rename(DATE_TIME = date_time) %>%
  pm_annual_caaqs(dt = "DATE_TIME", by = c("ems_id", "site"))
#> Calculating PM 2.5 daily average
#> Error: arrange() failed at implicit mutate() step.
#> x Could not create a temporary column for `..3`.
#> i `..3` is `date_time`.

Good catch, thanks @boshek! Travis is broken I think, and I haven't set up GH Actions - assuming it passes locally for you I'll take my chances and merge it now if you think it's okay?

Good catch, thanks @boshek! Travis is broken I think, and I haven't set up GH Actions - assuming it passes locally for you I'll take my chances and merge it now if you think it's okay?

@ateucher - fixed a couple other minor things. Check still complaining about a couple minor issues but they aren't show stoppers.

@ateucher - fixed a couple other minor things. Check still complaining about a couple minor issues but they aren't show stoppers.


Add ability to specify the output file name on the command line.

This is required, in a maven multi-module setup, where relative path could point
to different files, when started as a parent or a sub-module build.
With this change, in the pom.xml you can specify --outputFile ${basedir}/validation.json

Netcetera » valdr-bean-validation #102 SUCCESS
This pull request looks good
(what's this?)

Coverage increased (+0.15%) to 90.82% when pulling 341aca73180ac431408eaf6376dff56dbe380b26 on gzsombor:output-file-name into f944d20a489c696ff3263f1ffc7404da6428890e on netceteragroup:master.

I'm going to change this so that passing -outputFile to the CLI overrules any outputFile which may have been set in valdr-bean-validation.json.

Good idea, thanks !


only allow adding whitelisted validator into the validator pool

To ensure the security of the delegated assets, when add_validator or add_validators, check whether the selected validator is in the whitelist.

Resolved in #92


[decompile.exe] Removing extra spaces and empty tags from the response.

The following is a part of the real response from the decompile.exe module to Ghidra:
Each extra space and empty tags are colored.
The original text has been reformatted for human readability.
The reason I used the image here instead of a code snippet is the inability to highlight text in any way in it.

After the proposed patch the response look like this:
(Also after reformatting)

There is an issue in the after image just before the end of the first funcproto. It says <syntaxcolor="keyword">void</syntax> the space between syntax an color is missing.

@caheckman Superseded and fixed with 6be9943?


Map Field

I've been using this map field ( https://github.com/RocKhalil/rails_admin_map_field ) for some time, and I think it's a good option to include it in the main rails admin repo.
Do you think it's worth working on migrating it to the main repo or keep using it as an external gem ?

I have a some concerns around maintaining non-Rails-native functionality in rails_admin (which has been done for PaperTrail, Pundit, Carrierwave, etc.), mainly since it increases the maintenance burden of keeping up with any / all changes to those libraries in one place with one release process. It feels helpful (and easier now that we have the railsadminteam Github org) to have rails_admin keep / maintain an interface for these types of extensions, and split those out to separate gems (something I've been meaning to propose that over in the Roadmap discussion) - which would mean keeping that project as a separate gem, thoughts?

Yes, makes total sense. Just like react-native-community, we can start porting useful packages into railsadminteam that can enhance rails_admin and its functionalities.


Implement tags (partly)

This implements 3 out of the 4 properties listed in #394.

Sorry for the delay.
Thanks!

Thanks & I didn't see any delays. :-)


scripts: extract_dts_inculdes: Fix reg prop size/addr cell calculation

When we determine the address and size cells to determine how to parse a
reg property, we need to stop at the parent of the node, not at the node
itself.  If we have #address-cells or #size-cells in a node its meant
for the children of that node and not the node itself.
Fixes #4568
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6282   +/-   ##
=======================================
  Coverage   53.15%   53.15%
=======================================
  Files         412      412
  Lines       40139    40139
  Branches     7732     7732
=======================================
  Hits        21337    21337
  Misses      15667    15667
  Partials     3135     3135

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 1e159de...4832ff3. Read the comment docs.


Baseline 3D-LUNeXt

Baseline upgrade of the 3D-LUNeXt model.

@edyoshikun I'm trying to keep this the same as we are using for the mantis figures. I've tested that /hpc/projects/comp.micro/mantis/2023_08_18_A549_DRAQ5/3-registration/prediction/predict.yml still predicts the same results.
I will handle renaming and other refactoring after this merges.


Fix: dataframe_subquery example Optimizer rule common_sub_expression_eliminate failed

Which issue does this PR close?

Closes #8015.
Rationale for this change

make cargo run --example dataframe_subquery run
What changes are included in this PR?

Use out_ref_col instead of col.
Are these changes tested?

Are there any user-facing changes?

No.

Thank you @smallzhongfeng
I wonder if there is some way to add these examples to the CI / ensure they are passing (so we don't accidentally break them again without knowing it) 🤔

In fact, I am currently doing this, but the method is relatively simple, which is to traverse all the rs files in the example directory through the shell, and then execute them in sequence. I don’t know if you need it. 🤔

But currently, you can only run test cases based on local or memory, and tests using external storage may need to be skipped. : (

the test like flight_server also need be skip, those test are start a server.


Episode 020: Understand Namespace Policy

Check the following items when you plan to start an episode.
Assign people to each role by filling @Handle in the following parentheses.

[x] Coordinator: (Yu Su)

[x] Designer: (Cunjie Gan)

[x] Host: (Yong Zhang)

[x] Technical writer: (Yu Liu)

Before an episode

[ ] [Host] Pick a topic from the episode idea backlog and create a epispde issue.

[ ] [Coordinator] Create a folder for the episode in the episode folder, name it in numeric order, and create a README with necessary information.

[ ] [Coordinator] Fill in the necessary information on Bilibili, see here.

[ ] [Designer] Create an image for the episode, send it to cn_growth channel for review, and upload it to image folder.

[ ] [Coordinator] Publish an episode announcement with the following information to the following platforms.

Information:

Topic：Understand Namespace Policy

Host：Yong Zhang

Date：2020/08/09

Time：11:00 AM

Duration：1h

Live streaming link: https://live.bilibili.com/21468418

Shishuo meeting link：https://www.slidestalk.com/w/251

Platform:

Pulsar Slack #china channel

Pulsar Slack #tgip-cn channel

WeChat (on Thursday, Friday, and Sunday)

[ ] [Coordinator] Send a reminder to WeChat groups 1 hour before live streaming.

[ ] [Host] Set up live stream on Bilibili with OBS.

Note: if this is the first time for you to live stream, ask Yu to give you the username and password of StreamNative Bilibili account.

Tip: for how to live stream on Bilibili with OBS, see here.

[ ] [Host] Get everything tested well:

Network

Live streaming device, such as camera, audio, and so on

During an episode

[ ] [Host] Record the episode with OBS.

[ ] [Host] Make notes with HackMD.

[ ] [Host] Interact with audience by using Bilibili Live Chat feature.

[ ] [Coordinator] Interact with audience and answer questions, keep tracking of data

After an episode

[ ] [Coordinator] Upload the recorded video to StreamNative Bilibili and Tencent.

Set the image as the cover image for this episode if possible.

Add this video to TGIP-CN playlist.

Add other additional information, such as title, description, and so on.

[ ] [Coordinator] Ask inputs from the host and fill in the README.md.
Here is a template.
Here is an example.

[ ] [Coordinator] Upload the slide to slidestalk.

[ ] [Coordinator] Add the episode to the playlist.md.

[ ] [Coordinator] Write a tech blog about the episode.

[ ] [Technical writer] Improve Pulsar/SN doc based on the episode (if needed).

[ ] [Coordinator] Promote the episode with the following information to the following platforms.

Information:

Topic

Host

Date

Time

Duration

README link

Study note link

Recorded video link

Tech blog link

Slide link

Platform:

Pulsar Slack #china channel

Pulsar Slack #tgip-cn channel

WeChat

[ ] [Coordinator] Make a summary, add suggetsions for the episode here, and share it to cn_growth channel on Monday.

@SylviaBABY some info in the issue template need to be updated, could you please update the TGIP template?


Tracing view v1

This only supports apollo server v2 tracing.
I want to update for apollo server 3 but I can do it in a separate PR.
Also, I'm not sure if there is a 3rd party package for setting up a flame chart. I couldn't find one but I didn't search too hard.
Let me know if there should be any other changes. I only have some basic tests.
Placement of the text depends on the start time and length of the resolver/request.
Apollo v2 tracing is always (should always) be turned off in production. These are screenshots from a secret project, hence the censorship but they are real requests using Apollo v2 tracing.

"No tracing found" images.

Really nice work @christo8989. This feature looks awesome. I left a couple of comments.

@warrenday I think it only follows apollo server v2 tracing so I still need to update it for apollo server v3.

Is it okay that I changed the build node version to 14.x?

@christo8989 are you happy for this to be merged?
Also, could you provide a link to a website where I can see tracing working? I tried a couple of sites that use apollo v2 and couldn't see anything in the tracing tab.
I can only see this working using the mock data at the moment.

I'm new to GitHub workflows so I need to change the check from build 14.4.x to build 14.x.
Also, Apollo tracing is never turned on in production. I can send a screenshot of some work tracing but I'll need to blur out the info.
I'm curious what people use for tracing because I can update it for other systems. Maybe people will submit issues / maybe I should add a better message about it.

@warrenday
I think there are some minor versioning issues. I tried going back to 14.4.x but it was still complaining. I'm not exactly sure if I did something wrong or if this is an existing issue but I'll leave it as is for now.
I added screenshots to the description and there was a bug fix in the mix (related to the new hook).
I'm ready to merge if you are pleased.

Merged, thank you for the feature. I'll release to the store this week.


Miscellaneous Spotbugs issues

Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
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.

@Kuanysh-kst thanks for the PR. Can you link the issue this PR is for in the PR description under the title Issues Resolved and provide a description about this PR under Description. This will help the maintainers to understand the context of the PR. Thanks again.

@Kuanysh-kst thanks for the PR. Can you link the issue this PR is for in the PR description under the title Issues Resolved and provide a description about this PR under Description. This will help the maintainers to understand the context of the PR. Thanks again.

Those links would help.
FYI this is a smaller resubmit of #580 based on my request here and the issue being addressed is #552.


Arguments have not been passed to cargo test

Version of VSCode: 1.13.1
Version of the extension: 0.4.2
OS: Mac OS X 10.10.5
Description: Arguments have not been passed to cargo test
    "rust.customTestConfigurations": [
        {
            "title": "nocapture",
            "args": [
                "-- --nocapture"
            ]
        }
    ],

$ rustup run stable-x86_64-apple-darwin cargo test "-- --nocapture"
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
     Running target/debug/deps/etl-c4bbe75dac381507
thread 'main' panicked at '"Unrecognized option: \' --nocapture\'."', src/libtest/lib.rs:260
note: Run with `RUST_BACKTRACE=1` for a backtrace.
error: test failed, to rerun pass '--lib'

    "rust.customTestConfigurations": [
        {
            "title": "all",
            "args": [
                "--all"
            ]
        }
    ],

Although it works in this case.

@banyan,
Of course it doesn't work.
"--" and "--no-capture" are two different arguments therefore you should write it like:
    "rust.customTestConfigurations": [
        {
            "title": "nocapture",
            "args": [
                "--",
                "--nocapture"
            ]
        }
    ],

I'm closing the issue because there is no any issue.
If you have any questions, you can ask me here.

Ah I see. Thanks. I thought it was a regression since it worked until 0.4.0.

@banyan,
I don't believe you) I didn't change it.


Segfault after several function calls

I recently found a segfault that occurs after invoking the same function several times (approx 30 thousand times). Code to reproduce the issue is as below:
import numba

def foo(x):

    @numba.njit(cache=True)  # The issue occurs even without cache=True, yet it's way slower
    def bar(b):
        return b+1

    return bar(x)

for i in range(200):
    print(i)  # Just to show the progress
    for j in range(3000):
        foo(j)

The program terminates approximately i=XX (for some value of j) with a segfault. Sometimes I get instead the error LLVM ERROR: Unable to allocate section memory!.
I've noticed the segfault does not always occur at the same value of i and j but changes slightly, e.g. when using a different virtual environment.
Tested on Numba 0.60, Python 3.11, Debian 12.
The issue does not appear if bar is defined outside foo in the global scope. I don't know if @numba.njit is not supposed to be used for locally-scoped functions, yet in such case it would be preferable to get any error/warning message preventing its usage.

Thanks for the report. I suspect the issue is due to the following effects:

Defining a @jit decorated function in a closure results in Numba considering the function as a "new" function every time (as it has the scope of the closure). In the code above, this creates 200 * 3000 new functions.
Numba has a long standing issue to do with not being able to successfully free memory associated with compiled functions from the LLVM MCJIT compiler it uses to undertake the compilation.

With 1. and 2. combined, I suspect the system is running out of memory and something either signals the process to say "out of memory" or LLVM asks for more memory and it is denied, which would probably result in a SIGABRT/that LLVM ERROR.
Hopefully this explains what you are observing. The long term fix is for Numba to migrate to using LLVM's OrcJIT, where resources are more easily tracked and released, this however is a large undertaking.

Duplicate of #5976 and likely similar issues?


java.lang.RuntimeException in net.lingala.zip4j.util.Zip4jUtil.getCompressionMethod::Zip4jUtil.java:121 zip4j 2.9.0

This vulnerability is of java.lang.RuntimeException, and can be triggered in latest version zip4j (2.9.0).
It is caused by an unchecked exception or wrapping a general unchecked exception around another exception that extends RuntimeException and can be used for attackers to launch DoS (Denial of Service) attack for any java program that uses this library (since the user of zip4j doesn't know they need to catch this kind of exception) ( CWE-248: Uncaught exception).
Likely, the root cause of this crash is in net.lingala.zip4j.util.Zip4jUtil.getCompressionMethod::Zip4jUtil.java:121.
https://github.com/srikanth-lingala/zip4j/blob/ce1cff61c019d61f5db603cba908118a3a0e3b93/src/main/java/net/lingala/zip4j/util/Zip4jUtil.java#L121  The variable "localFileHeader" is not NULL however its component "getAesExtraDataRecord" is NULL and it results in RuntimeException.
See more detail from the following crash stack.
Crash stack:
The crash thread's stack is as follows:
net.lingala.zip4j.util.Zip4jUtil.getCompressionMethod::Zip4jUtil.java:121
net.lingala.zip4j.io.inputstream.ZipInputStream.getCompressedSize::ZipInputStream.java:318
net.lingala.zip4j.io.inputstream.ZipInputStream.initializeEntryInputStream::ZipInputStream.java:222
net.lingala.zip4j.io.inputstream.ZipInputStream.getNextEntry::ZipInputStream.java:113
net.lingala.zip4j.io.inputstream.ZipInputStream.getNextEntry::ZipInputStream.java:83
com.test.Entry.main::Entry.java:37

Steps to reproduce:

Build the following java code with the corresponding zip4j library (version 2.9.0).

## Download zip4j_env_reproduce.tar.gz from https://drive.google.com/file/d/1MekCBIghKxIW4j-TLjZkm8ovvLb_grm5/view?usp=sharing
tar -xf zip4j_env_reproduce.tar.gz
cd zip4j_env_reproduce
bash build.sh

Run the built program to see the crash by feeding one of the poc file contained in the pocs.tar.gz, e.g. :
(poc file can be downloaded from https://drive.google.com/file/d/1-tI49fljs6XYhaphgJbO7M_inEQwZd2W/view?usp=sharing)

java -jar target/Entry-1.0-SNAPSHOT-jar-with-dependencies.jar pocs/crash-2e99f99d7396ae948c4d180fc306de98070ab7af

Any further discussion for this vulnerability including fix is welcomed!

Hey Srikant, If I just have this dependency in my pom:

net.lingala.zip4j
zip4j
2.11.2
   /dependency>
I get this error:
symbol:   class Zip4jConstants
[ERROR]   location: package net.lingala.zip4j.util
then I was adding following additional dependency and it worked:

net.lingala.zip4j
zip4j
1.3.2
compile

But why we need to add this, they are same dependency, and the 1.3.2 version has vulnerabilities too, which gives Blackduck issue to my project. Can you please comment and help me on this?


Cleanup packaging for resource_retriever

While we are here, also switch over to package.xml version 2.

Thanks, merged.


Review and modify retry policy

Story details: https://app.clubhouse.io/cartoteam/story/124556

I'm having this issue while testing locally:
central_1             | rake aborted!
central_1             | NameError: uninitialized constant Google::Cloud::PubSub::RetryPolicy
central_1             | /var/lib/gems/2.5.0/bundler/gems/cartodb-common-1bca84990a40/lib/carto/common/message_broker.rb:20:in `<class:MessageBroker>'
central_1             | /var/lib/gems/2.5.0/bundler/gems/cartodb-common-1bca84990a40/lib/carto/common/message_broker.rb:17:in `<module:Common>'
central_1             | /var/lib/gems/2.5.0/bundler/gems/cartodb-common-1bca84990a40/lib/carto/common/message_broker.rb:5:in `<module:Carto>'

I think there may be some issue with either autoloader or versions of google pubsub ruby client library. Investigating it...

Now I get this other error:
central_1             | rake aborted!
central_1             | LoadError: cannot load such file -- google/cloud/pubsub/retry_policy
central_1             | /var/lib/gems/2.5.0/bundler/gems/cartodb-common-a87ccdcdb88e/lib/carto/common/message_broker.rb:2:in `require'
central_1             | /var/lib/gems/2.5.0/bundler/gems/cartodb-common-a87ccdcdb88e/lib/carto/common/message_broker.rb:2:in `<top (required)>'

Essentially the RetryPolicy object does not exist in the 1.2.x series: https://googleapis.dev/ruby/google-cloud-pubsub/v1.2.2/Google/Cloud/PubSub.html
I suggest moving to 1.10.0 and if everything goes right, we can consider moving to the 2.x series.
More info here: https://github.com/googleapis/google-cloud-ruby/blob/master/google-cloud-pubsub/CHANGELOG.md


[aggregation] add JsonProperty to constructor arguments

Interesting, don't we need a @JsonCreator as well, or does Jackson 2.7.x handle this?

Yeah, it works without @JsonCreator.
In any case, I'm going to add it for consistency.


Update tls-1.0-and-1.1-deprecation-for-office-365.md

Updated the document as TLS 1.0 and 1.1 disablement is complete
per issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/10190

@JohanFreelancer9 Please copy review

@WorkShay Please merge this PR. Thanks

Learn Build status updates of commit 675de42:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/tls-1.0-and-1.1-deprecation-for-office-365.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit d931ad4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/tls-1.0-and-1.1-deprecation-for-office-365.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


DynamicSurroundings-1.16.4-<IP_ADDRESS> listed as version <IP_ADDRESS> in Forge Mods menu.

Install Dynamic Surroundings v<IP_ADDRESS>.
Launch the instance.
Click Mods on the main menu.
Scroll until you find Dynamic Surroundings. See that it's labeled as v.<IP_ADDRESS>.


Function to report progress of column metadata extraction

Creating a function to be used for the progress bar showing how far Sprout is in extracting the csv column metadata.
This function, therefore, relies on the function created in #17 that actually scans and extracts the metadata.

Closed from #134


How to schedule a Daily task and background task has no parameters

I am using django-background-tasks in my project and it is running perfectly from last two years but now I have some new requirement to run a background task which I have created but facing some issues to configure and execute through django-background-task.
@background(schedule=Task.DAILY)
def BatchProcessedReset():
currdateobj = date.today()
#todaydate = currdateobj.strftime("%Y-%m-%d")
try:
orgobj = Organisation.objects.filter(status=True)
for org in orgobj:
if org.batch_reset_date == currdateobj:
org.batch_processed = 0
org.batch_reset_date = datetime.now()+timedelta(days=365)
org.save()
except:
pass
return
this is the background task without any parameters and required to run daily once only. Second this task does not need to call from any django project it is isloated task that needs to run onec a day.
I tried to enter this task through Django admin when it execute it return below errors.
(msaaspro) Asifs-iMac4K:src asifkhan$ python manage.py process_tasks
Rescheduling BatchProcessedResting
Traceback (most recent call last):
File "/Users/asifkhan/Development/msaaspro/lib/python3.6/site-packages/background_task/tasks.py", line 32, in bg_runner
args, kwargs = task.params()
File "/Users/asifkhan/Development/msaaspro/lib/python3.6/site-packages/background_task/models.py", line 214, in params
args, kwargs = json.loads(self.task_params)
ValueError: not enough values to unpack (expected 2, got 0)
Marking task BatchProcessedResting as failed
seems that django-background-tasks can not configure without parameters.
anyone know, please advice how to handle this.

Is there a way to set background task manually instead setting up from one of project apps or urls to create a background task.??

I am closing this because I found out the way to add any background task in Task model manually for a background task with out calling from any project app or urls.py to set the background task to run. It is simple and we can set the frequency to daily or weekly what we want. thanks.

I am closing this because I found out the way to add any background task in Task model manually for a background task with out calling from any project app or urls.py to set the background task to run. It is simple and we can set the frequency to daily or weekly what we want. thanks.

I'm looking for such way. Could you please post here your way of doing it?


Add query builder "orderBySub" and "orderBySubDesc" methods

NOTE: Don't merge this yet. 🚨
Another approach was suggested to me, which I'd like to explore first.

This PR adds two new methods to the query builder to allow ordering by sub queries. This includes a new orderBySub($query, $direction) method and a shortcut orderBySubDesc($query) method, similar to orderByDesc($column).
Previously this was only possible using orderByRaw(), but that's dangerous, because the direction (asc/desc) cannot be used as a binding, meaning developers were responsible for verifying the direction themselves.
These two methods intentionally use orderBy() instead of orderByRaw() to take advantage of the direction check that exists in that method.
Here are a couple examples of how this can be useful:
// Order users by their last login date
$users = User::orderBySub(function ($query) {
    $query->select('created_at')
        ->from('logins')
        ->whereColumn('user_id', 'users.id')
        ->latest()
        ->limit(1);
})->get();

// Order courses by the start date of the first lesson
$courses = Course::orderBySub(
    Lesson::select('starts_at')
        ->whereColumn('course_id', 'courses.id')
        ->oldest('starts_at')
        ->limit(1)
)->get();

Me going through this PR.

Closing this PR in favour of #29563, which I believe is a better approach to adding this feature.


CMake suggestion

I had issues installing hoomd-dlext on midway3.
It finds my local hoomd installation (home directory) just fine.
But the find_package(hoomd) causes issues, since it find the system wide installation hoomd to query that.
And that has requirements, we cannot fulfill thrust 1.5.0.
Removing the line from the CMakeList.txt resolved the problem, but I am not sure if I breaking other things now.

It finds the wrong hoomd installation.
HOOMD_ROOT="$(python -c 'import site; print(site.getsitepackages()[0])')/hoomd/"
12:31 umbrella_integration@midway3-login2 171$ echo $HOOMD_ROOT
/software/python-anaconda-2020.11-el8-x86_64/lib/python3.8/site-packages/hoomd/


Wrong parsing of url

I am trying to parse this url:
https://github.com/tterranigma/Stouts.openvpn.git

This is what I am doing:
import giturlparse
name = 'https://github.com/tterranigma/Stouts.openvpn'
print(giturlparse.parse(name))

I get:
Parsed(pathname='//github.com/tterranigma/Stouts.openvpn.git', protocols=['https'], protocol='ssh', href='https://github.com/tterranigma/Stouts.openvpn.git', resource='https', user=None, port=None, name='Stouts.openvpn', owner='/github.com/tterranigma')

The owner field is wrong.
I tried with version 1.1 and it works correctly.

Sorry for the bug @tterranigma.  Pushed a fix to pypi. \m/


Is this MIT OR Apache-2.0 dual-licensed?

Cargo.toml doesn't have a license field.
cc @philipc, @main--, @jrmuizel and @roblabla (all authors excluding myself)

Ah, I always sort of expected this project was MIT/Apache-2.0, since other crates in the gimli org (like gimli and addr2line) are.

So did I. I searched for it just in case, as I wanted to copy code to another project.
I agree to allow my contributions to the unwind-rs project to be distributed under the terms of the "MIT OR Apache-2.0" license.
@roblabla Is my assumption based on your response that you too allow your contributions to be distributed under this license correct?

I agree to allow my contributions to the unwind-rs project to be distributed under the terms of the "MIT OR Apache-2.0" license.

Yes, of course!
I agree to allow my contributions to the unwind-rs project to be distributed under the terms of the "MIT OR Apache-2.0" license.

I agree to allow my contributions to the unwind-rs project to be distributed under the terms of the "MIT OR Apache-2.0" license.

Ping @main--

Sorry, totally missed this!
I agree to allow my contributions to the unwind-rs project to be distributed under the terms of the "MIT OR Apache-2.0" license.

Ok, thanks! I will open a PR to add the licenses.


Simplify layer creation with string literals

Manifest an array of string literals for all the default layers available to the Globe.  The individual layer names can be used by the developer to add layers to the Globe without having to traverse the WorldWind library to discover them on his or her own.

Globe now accepts string identifiers or numbers when adding layers to the globe.

Added the defaultLayers array defining valid string identifiers for layers. This array can be used in the UI.
addLayer now accept a string matching an entry in the defaultLayers array. Partial strings are OK. The first matching entry is used if a partial string is provided. The strings are case-sensitive.
addLayer now accept a number corresponding to an index in the 'defaultLayers' array.


Always use CLI command to load cRPD license

The expected license file location was changed in Junos 22 and later. To keep the code backwards and forwards compatible we use the CLI command to add a license file from a stable location.
Closes #1350

thanks @mzagozen
this lgtm
I presume you tried it with the crpd 23+? I think in general containerlab strives to support the latest releases, to go in a lockstep with the current NOS offerings; with that in mind I think what is important is that newer version runs smoothly, and older ones might be less prioritized in terms of the support

Right, I only have 23.4R1.9 available for testing. I remember using the CLI command way back anyway :)
Like I mentioned in #2013, the SSH config massaging code is now unnecessary, But I guess since it does not interfere with the current latest it may as well remain in place for older versions.


Comment counter

Worked on the comment  counter functionality

Nice job so far @NewtonOmbese  I would suggest you fix the eslint


setup.py: add "bump" command

This command will increment the last number in __version__ automatically and commit all changes to Git with a standard commit message.
(In the vast majority of cases, we simply want to bump the final version number part, so this optimizes for that scenario to make that easier. If we want to go to "2.0.0" or "3.0.0", that will have to be done by hand, or else add more user_options to this command to make it more flexible.)

This is going to make releases easier (read: faster)

This and pr #153 look great. But I am very worried about a full release with code changes that doesn't pause before moving forward and no confirmation.
It would be ideal to:

display the actual changes to be made (e.g. "previous version: 0.0.1, new version: 0.0.2")
prompt the user for acceptance of the release

I would like some tests as well on top of this, but that may be a bit harder to do right

I've added some code to print the "old version: new version:" information to the console, and it now prompts so the user can at break out of python setup.py bump if necessary.

thank you for the changes, almost good to go

Thanks! I've rebased this branch, hope it looks good now

thank you so much for the changes


Door Lock State Update

I have a door lock Yale YRD226L TSDB and the state of the door lock did not update if I lock or unlock the door manualy or if the door been locked automaticaly
for I get the correct status of the door lock I need to access the zigbee2mqtt addon at home assistant, select the device and click at the refrest arrow buttom

running version 1.22.0-2

First check if the lock communicates the unlock/lock state after it has been changed manually. Check the logs set them to warn level info or debug (not sure which one) and open the lock. IF you dont receive a message chances are that the lock does not communicate its state. this means there is no way for z2m to know.

At the log, the lock did not send any communication when I lock or unlock
the lock.
But If I unlock the lock manually and go to the lock device expose at
zigbee2mqtt and select the refresh buttom
[image: image.png]
The following communications are displayed at the Log Session and the state
to the lock been updated
Debug Received MQTT message on 'zigbee2mqtt/Fechadura/get' with data
'{"state":""}'
Debug Publishing get 'get' 'state' to 'Fechadura'
Debug Received Zigbee message from 'Fechadura', type 'readResponse',
cluster 'closuresDoorLock', data '{"lockState":2}' from endpoint 1 with
groupID 0
Info MQTT publish: topic 'zigbee2mqtt/Fechadura', payload
'{"battery":100,"linkquality":105,"lock_state":"unlocked","state":"UNLOCK"}'
I would like to know if there is a way for I make zigbee2mqtt execute this
refresh automatically every 5 minutes for example
Regards
Igor
Em ter., 23 de nov. de 2021 às 08:38, sygys @.***>
escreveu:

First check if the lock communicates the unlock/lock state after it has
been changed manually. Check the logs set them to warn level info or debug
(not sure which one) and open the lock. IF you dont receive a message
chances are that the lock does not communicate its state. this means there
is no way for z2m to know.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zigbee2mqtt/hassio-zigbee2mqtt/issues/234#issuecomment-976429848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ4UKVEXAQABUTMZO3ODLQLUNN4LDANCNFSM5IAUM43Q
.
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.

--
Igor Fernandes


CFP: cilium connectivity test to support dropping capabilities

Cilium Feature Proposal
Thanks for taking time to make a feature proposal for Cilium! If you have usage questions, please try the slack channel and see the FAQ first.
Is your proposed feature related to a problem?
It is currently not possible to run cilium connectivity test when admissionControl PodSecurity enforces anything above privileged. This is the case for Talos by default, see https://www.talos.dev/v1.6/kubernetes-guides/configuration/pod-security/
You will get errors like these for the DaemonSets (and no Pods will be created):
pods "host-netns-9bz74" is forbidden: violates PodSecurity "baseline:latest": non-default capabilities (container "host-netns" must not include "NET_RAW" in securityContext.capabilities.add), host namespaces (hostNetwork=true)

or

(container "echo-external-node" must set securityContext.capabilities.drop=["ALL"]; container "echo-external-node" must not include "NET_RAW" in securityContext.capabilities.add)

Describe the feature you'd like
Command line arguments like we have them for the helm chart (securityContext.capabilities.cleanCiliumState and securityContext.capabilities.ciliumAgent) when running cilium connectivity test
Alternative is to disable enforcement for the namespace:
kubectl label namespace cilium-test pod-security.kubernetes.io/enforce=privileged

This is a great suggestion! We should totally thin down the set of privileges required by the connectivity test pods.

FYI, I'm moving this issue over to the https://github.com/cilium/cilium-cli/ repository.


0.5.0 release candidate testing

Due to the increasing amount of users and diverse environments testing is a lot work if we are to avoid regressions. For this reason, I'm considering introducing release candidates. I want this to be an opportunity for everyone to help sort out any issues before a release goes out.
I do not expect people to have time but if you do it would be much appreciated. All I'm asking is that you check out the 0.5.0 branch https://github.com/Wramberg/TerminalView/tree/0.5.0, use it for a little while and report back any issues you may encounter.
@rwols @randy3k I hope you have time to help do some general macOS testing and specific testing of the terminal_view_send_string and terminal_view_exec commands (if you haven't already ;-)).
@bfelder If you have time to help as well that would be nice.
Like I said just use TerminalView like you normally would - hopefully it won't be too much of a bother that way :-)
If anyone else sees this and want to help out let me know and I will mention you next time I have a release candidate.

Great! I have a look at it tomorrow. As mentioned I run Linux Mint 18.2 which is essentially a recent release of Ubuntu in its core.

Great job! terminal_view_send_string is working correctly.

Thanks guys. Should probably mention I plan to release 0.5.0 towards the end of next week. Should hopefully be enough time to test it a bit.

@Wramberg Works like a charm! :+1:
I tested usual command like echo 3, python my_python_file.py same output as before.
I have also written a companion plugin to TerminalView which automatically open a virtual environment if such a path is present with a project settings file. This also works as before.
I now tested with htop. A system monitor providing you with live updates on cpu load, network traffic etc ... Importantly this output is colored. And indeed TerminalView gave me a beautiful color terminal!
(Admittedly I did not test the previous version of TerminalView with htop. So, I can not say whether this is new.)
Let me know if you need any other test specifically.

There's nothing about terminal_view_send_string in the readme, shouldn't there a little section devoted to it?

Even pudb works fine in TerminalView. This is the first time a python debugger runs find in Sublime.
Hence, @Wramberg single-handedly transformed a common editor into full-blown python IDE. :)
Still the color scheme of my system's shell is not transferred to TerminalView. The standard text is all white.
See closed issue https://github.com/Wramberg/TerminalView/issues/50

@bfelder https://github.com/Wramberg/TerminalView/pull/53 should liven up the colors 😄

@bfelder agree, I have been waiting a working terminal for so.... long.

@rwols to the rescue. I like to have candle light dinner with Marianna!
@randy3k a terminal as a fundamentally enabling tool opens up so many great opportunities to integrate useful features into sublime. I am particularly fond of the idea to integrate a minimal python debugger with
TerminalView. Something simple, just wrapping ipdb or bpdb, but using both the editor view to indicate current line and terminal view for accepting commands and display of variable values.

There's nothing about terminal_view_send_string in the readme, shouldn't there a little section devoted to it?

Yea I should probably include a bit about it. Think Ill do that and then release it as soon as I find time.

@Wramberg Color might be of low priority right now.
Still, let me tell you that my normal bash indeed features color.
The most important aspect is to color the prompt (unchangeable) differently from user input (changeable) for easier telling them apart.
Switching to another shell is no option for me, regarding how deeply integrated bash is with my Linux distro.

Still, let me tell you that my normal bash indeed features color.

Probably because your terminal sets TERM to xterm or something.

Switching to another shell is no option for me

In that case you can look into changing your prompt manually. https://www.cyberciti.biz/faq/bash-shell-change-the-color-of-my-shell-prompt-under-linux-or-unix/

@Wramberg Thanks for the link a thousand times. The command export PS1="\e[0;31m[\u@\h \W]\$ \e[m " not only colors the prompt but also shortens the path, making the prompt much tidier.
I use that as an arg to the TerminalView command until xterm will get integrated into TerminalView.
(Of course, I am also going to checkout @rwols color packages. Though you suggest they may not work.)

No problem :-) I think @rwols PR will work just fine. But it only changes text that is already colored to another color - it won't add new colors anywhere (I think this is intended).

Yeah, that PR attempts to assign colors of the current scheme to the 8 basic terminal colors black, white, red, blue, green, cyan, magenta and yellow. It doesn't extend to 256 or even 16 colors.

0.5.0 is now live. Thanks for helping out everyone. Seems like 0.6 is already going to include a lot of new exciting things :-)


regression-testadd ctas join test

Proposed changes
Issue Number: close #xxx

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

run buildall


Am I do it wrong?

I set the session life time to one minute to test it out, my app is SPA with vuejs so instead of using laravel caffeine middleware I use following code to send drip:
setInterval(function () {
     Vue.http.get('/api/drip');
 }, 10000);

I define a new drip route like so:
$router->get('api/drip', '\GeneaLabs\LaravelCaffeine\Http\Controllers\LaravelCaffeineController@drip');

I see that in network tabs it sends the drip every 10 seconds, I leave the page open for a while after filled the form and after few minutes hit the submit button and I still get token mismatch exception. Am I missing something?

is your route using the "web" middleware group, so it knows about the session?

Yes, in Laravel 5.2 the web middleware is automatically applied to all routes.

Are you sending the CSRF token with the Vue request?
Answer found on this post:
Put this in the <head> element:
{{-- This is the token Laravel requires for non-GET requests --}}
<meta id="_token" value="{{ csrf_token() }}">

If you are using the Vue http lib, use this code to include the token:
Vue.http.headers.common['X-CSRF-TOKEN'] = document.querySelector('#token').getAttribute('value');

If you are using jQuery, use this code:
<script>
        $.ajaxSetup({
            headers: {
                'X-CSRF-TOKEN': $('meta[name="csrf-token"]').attr('content')
            }
        });
</script>

This will automatically send the token with each request.

Hi @reefki, unfortunately this package isn't really built for SPAs. If you want to keep an SPA from timing out, it's best to increase the lifetime of the session. The only benefit this might have is that your session will never time out, as long as the browser is open.
Also, no need to create a route another route. Use the route provided in the package. In the configuration you have the option of overriding the default route for Caffeine, but that route should not exist in your app, the package will use that to create its own route for you.
@ivanvermeyen thanks for helping @reefki out! :)

Just as a side-note:I am considering implementing functionality that will work better with SPAs going forward, as I am adopting them more often as well, but haven't given it too much though yet.

@ivanvermeyen Thanks for your reply. Yes, I include the token via interceptor:
Vue.http.interceptors.push(function (request, next) {
    request.headers['X-CSRF-TOKEN'] = document.querySelector('meta[name="csrf_token"]').getAttribute('content');

    next(function (response) {
        return response;
    });
});

The form is working if the csrf_token still valid. As I said in the beginning the problems occurs if the token has expired.
@mikebronner I think it should be work either SPA or non SPA as long the app sending drip, right?
Thanks for the tips about Caffeine route, but since a SPA is using wildcard route and I set the url to non hashbang !# I still have to define a new route because using Caffeine route makes the AJAX request redirected to index page.

@reefki the package should be creating the route on the PHP server side of things. You don't need to create a route in your SPA, since that's only for client functionality. Or am I understanding the scenario incorrectly? If you don't create a route, what error do you get?

@mikebronner No, I didn't create drip route on the frontend, see on initial post I defined a new route on routes.php.
I've tried to use the Caffeine route http://app.dev/genealabs/laravel-caffeine/drip but instead of returning 204 status code the server redirect the request to the wildcard route and return 200 status code. See image below, the response contains code from index.blade.php.

Here is my routes.php:
$router->any('{any}', function () {
    return view('index');
})->where('any', '.*');

@reefki, thanks prividing the details of the router and the response. The problem is that you have an "any" route defined that will respond to any request. That's not something I would ever recommend doing, it makes a whole bunch of assumptions. I would only create the route that is valid for your SPA, and return 404 for anything else that is not a valid route.
What's happening here is that your route is hijacking all incoming requests, and not letting any other routes take a stab at them. Once you get away from a wildcard route, the drip will work again.

@mikebronner The "any" wildcard is something that really important in my app, I wouldn't mind to define my 50+ routes both on routes.php and routes.js, a SPA only requires one page to be rendered once it's loaded which is index.blade.php in my case. If I follow your advice:

I would only create the route that is valid for your SPA, and return 404 for anything else that is not a valid route.

It would only makes my routes.php full of redundant:
$router->get('/', function () {
    return view('index');
});

$router->get('/abc', function () {
    return view('index');
});

$router->get('/xyz', function () {
    return view('index');
});

// the rest here (about 47 routes)

Remember, as I said earlier my app is using HTML5 History API for the router instead of !#so if I define one route / only in routes.php then when visitor access my other pages that I've defined on my routes.js for example http://app.dev/pages/contact they will redirected to 404 page because that route isn't defined on routes.php. I think catch all route using any wildcard is the best option for me.

Thanks for the clarification on your use-case. Unfortunately, I would recommend you not use Laravel Caffeine at this time, as it provides no real benefit to you. Right now, the way it works with SPAs is the exact same effect as setting sessions to a large expiration. I think that is the better solution for you at this point.
Sorry, I know this wasn't the answer you are looking for, but Laravel Caffeine wasn't designed for SPAs, as those rely heavily on their JS framework for routing.
I will close this for now, and will continue to evaluate how Laravel Caffeine could be implemented for SPAs.
Best of luck!


[Feature] <ClienOnly/>

          > > Can I be assigned this issue?

As mentioned above, I wanted to work on it, is there anything I can contribute to?

I assigned you. Thanks for your contributing!

Could you make component with current implementation internally without exposing it as public api please?

I would really appreciate it if you could explain your intentions more clearly.

Make <ClientOnly/> in src/components/ClientOnly.tsx to avoid tsup build entry.
Use <ClientOnly/> in src/Suspense.tsx. but Don't expose <ClientOnly/> in index.ts yet. In my opinion, We should expose <ClientOnly/> as public api gradually. so I think this should be next step.

Originally posted by @manudeli in https://github.com/toss/suspensive/issues/528#issuecomment-2185584827

@Collection50 Could I assign you for this issue?
Before assign you, I want to get approval from other maintainers.
@gwansikk @kangju2000 @bigsaigon333 @SEOKKAMONI
Check this new interface to expose
If we get approval, Let's make pull requests to do below.

Expose <ClientOnly/>,
Make docs for https://suspensive.org

@manudeli Sure!

Before assign you, I want to get approval from other maintainers.

I agree, I think it is ready to be released.

I agree. Not only does the interface itself seem useful, but it also serves as a great example of using useSyncExternalStore, providing developers with good ideas. Excellent work!

If we expose this component, I'm concerned that the meaning of the fallback prop might become ambiguous.
I'm worried that users might not immediately understand that this fallback is the UI shown on the server side. What do you think about this?

I agree with exposing this component!
But I think it would be better to either use a more explicit prop name or hide it altogether.

I'm worried that users might not immediately understand that this fallback is the UI shown on the server side. What do you think about this?

Cool feedback! Thanks! @kangju2000
So I found dictionary to check what naming "fallback" means
https://www.dictionary.com/browse/fallback
an act or instance of falling back.
something or someone to turn or return to, especially for help or as an alternative:
example: His teaching experience would be a fallback if the business failed.
For now, We are supporting fallback prop for Suspense,  ErrorBoundary,  Delay already.
In my opinion, naming "fallback" was used for alternative ui for children's objective
Consistency: Good for me below, in my opinion
<Suspense fallback={"alternative for not success of suspending"}>
  {/* Guarantee success of suspending */}
</Suspense>

<ErrorBoundary fallback={"alternative for not no error"}>
  {/* Guarantee no error */}
</ErrorBoundary>

<Delay fallback={"alternative for not delayed"}>
  {/* Guarantee delayed */}
</Delay>

So, I thought "fallback" is good prop naming for ClientOnly (alternative for not client only environment). How do you think of this logic?
<ClientOnly fallback={"alternative for not client only environment"}>
  {/* Guarantee client only environment */}
</ClientOnly>

I thought that we can use naming "fallback" for InView component too.
Not perfect. but I thought that we can use naming "fallback" like this consistently
<InView fallback={"alternative for not client only environment"}>
  {/* Guarantee in viewport or any element */}
</InView>

By this consistency, I thought fallback could be good common naming for these kind of cases
Inconsistency: Bad for me below, in my opinion
<Suspense suspending={"alternative for not success of suspending"}>
  {/* Guarantee success of suspending */}
</Suspense>

<ErrorBoundary withError={"alternative for not no error"}>
  {/* Guarantee no error */}
</ErrorBoundary>

<Delay delaying={"alternative for not delayed"}>
  {/* Guarantee delayed */}
</Delay>

<ClientOnly server={"alternative for not client only environment"}>
  {/* Guarantee client only environment */}
</ClientOnly>

<InView notInView={"alternative for not client only environment"}>
  {/* Guarantee in viewport or any element */}
</InView>

@manudeli
Thank you for your response!
I agree that maintaining consistency with other components is good. Additionally, it would be helpful for users if the prop explanations were included in the documentation.

it would be helpful for users if the prop explanations were included in the documentation.
Cool 👍

@bigsaigon333 re-alert you🙇

Inexplicit but Consistency: Good for me below, in my opinion

Consistency 🥇🥇🥇

@Collection50 Could you do this please?

@manudeli Sure!! 👍

@Collection50 Could you share your plan to do this?

@manudeli

Add index file to expose.
Write docs for ClientOnly

I'll make it until 3pm!

@manudeli

Add index file to expose.
Write docs for ClientOnly

I'll make it until 3pm!

Thanks! We follow convention that public api should be in src/[publicApi].tsx (because of tsup setting's entry field)
So move src/components/ClientOnly.tsx -> src/ClientOnly.tsx please.


New iOS App Store rule requires to report inappropriate ads

Hi,
This is from Apple's email that was sent on June 5, 2023:
Added to 2.5.18: “Apps that contain ads must also include the ability for
users to report any inappropriate or age-inappropriate ads.”

How can we add a "report" button to native ads(and other ad types)?
Thanks!

Thanks @huycozy .  I will follow those threads.


jsk_3rdparty: 2.1.21-3 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository jsk_3rdparty to 2.1.21-3:

upstream repository: https://github.com/jsk-ros-pkg/jsk_3rdparty.git
release repository: https://github.com/tork-a/jsk_3rdparty-release.git
distro file: melodic/distribution.yaml
bloom version: 0.9.3
previous version for package: 2.1.21-2

assimp_devel

No changes

bayesian_belief_networks
* add missing packages, closes https://github.com/ros/rosdistro/pull/26216 (#211 <https://github.com/jsk-ros-pkg/jsk_3rdparty/issues/211>)
* Contributors: Kei Okada

collada_urdf_jsk_patch
* add missing packages, closes https://github.com/ros/rosdistro/pull/26216 (#211 <https://github.com/jsk-ros-pkg/jsk_3rdparty/issues/211>)
* Contributors: Kei Okada

dialogflow_task_executive

No changes

downward

No changes

ff

No changes

ffha

No changes

gdrive_ros

No changes

jsk_3rdparty

No changes

julius

No changes

julius_ros

No changes

laser_filters_jsk_patch

No changes

libcmt

No changes

libsiftfast

No changes

lpg_planner

No changes

mini_maxwell

No changes

nlopt

No changes

opt_camera

No changes

pgm_learner

No changes

respeaker_ros

No changes

ros_speech_recognition
* add missing packages, closes https://github.com/ros/rosdistro/pull/26216 (#211 <https://github.com/jsk-ros-pkg/jsk_3rdparty/issues/211>)
* Contributors: Kei Okada

rospatlite

No changes

rosping

No changes

rostwitter

No changes

sesame_ros

No changes

slic

No changes

voice_text

No changes

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/preparing-for-melodic-sync-2020-08-19/15988/4


XPilot

@scribblemaniac looks like the links we have for XPilot are down, can you find alternate links ?

Working on my vacation :sweat:
I did a bit of searching, but found no alternative links. I did find some contact information so I could do that if need be, or we could just go straight to archiving it. I'm pretty sure the app still works though. It doesn't hurt to at least try this:
@stealthboy Could you perhaps shed some light on what happened to 7b5labs.com and if an alternative place to access the source code for XPilot will be made available?

@scribblemaniac oops sorry have a good vacation


Update Crocoddyl with latest changes in example-robot-data + use ANYmal for examples - [merged]

In GitLab by @cmastalli on Nov 7, 2019, 16:54
Merges topic/update-robot-data -> devel

In GitLab by @cmastalli on Nov 7, 2019, 16:54
enabled an automatic merge when the pipeline for ee88d2df0b46bbd4665d335d5e43038431308a29 succeeds

In GitLab by @cmastalli on Nov 7, 2019, 17:28
merged

In GitLab by @cmastalli on Nov 7, 2019, 17:28
mentioned in commit df4f47cc5e696b56acd01f0404fb532cfc363794


Use native Plugin Settings instead of custom menu

Logseq allows for custom Plugin Settings in the Settings Menu:

focus mode shouldn't introduce an own settings menu, as it adds clutter to the UI and leads to an inconsistent UX.

Thanks for bringing this to my attention.

@MrEbbinghaus which version are you using? I am not seeing the plugin settings in the latest nightly build of Logseq.

Yes, the settings are only displayed under certain circumstances.
https://github.com/logseq/logseq/blob/master/src/main/frontend/components/settings.cljs#L654


Vertical scrolling not working on some demos on Android

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Here’s a video to show you the issue more clear:
https://github.com/bitfoundation/bitplatform/assets/43121952/82f0479e-efe3-4e63-8a55-ed7756fc8bb3
Expected Behavior
No response
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
No response
Anything else?
No response

Thanks for contacting us. We're investigating this issue.
We'll let you know if it's possible to work on this issue.

The issue is accepted and planned. Resolving it will start ASAP.

Resolving this issue has started. It will be announced when this issue is resolved.

This issue has been resolved and it will be available in the next pre-release version. It will be announced when the pre-release becomes ready.

The pre-release has been published. Please give it a try and give feedback.


⚠ Bofrost.si has degraded performance

In 995fd89, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 3852 ms

Resolved: Bofrost.si performance has improved in da28a87 after 7 minutes.


Introduce internal types for sql blobs

This is step 1 of the following
1. Introduce these types
2. Add mappers which will convert to and from thrift types
3. Add mappers which will convert to and from proto types
4. Update the encoder interface methods to accept these internal types - the encoder will then use the mapper written in step 2 to convert to thrift blob before encoding.
5. Update the decoder interface methods to return these internal types - the decoder will then use the mapper written in step 2 to convert from thrift blob to internal before returning.
6. Implement encoder and decoder for proto just like we did for thrift

Coverage decreased (-0.05%) to 58.957% when pulling 0625c67cdba1a7b15a8716c4d1fb9bf31214cf9e on store_proto into 0d10cd3e92ba0f1e92ff6dfa5b9aa329b8cc1d77 on master.


git-credential: disable prompt when not attached to a tty

Fixes #234

Codecov Report
Patch coverage: 45.45% and project coverage change: -1.17 :warning:

Comparison is base (885a03d) 81.34% compared to head (03568af) 80.18%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #235      +/-   ##
==========================================
- Coverage   81.34%   80.18%   -1.17%
==========================================
  Files          27       27
  Lines        4005     4012       +7
  Branches      706      708       +2
==========================================
- Hits         3258     3217      -41
- Misses        624      660      +36
- Partials      123      135      +12

Impacted Files
Coverage Δ

src/scmrepo/git/credentials.py
58.98% <45.45%> (-1.57%)
:arrow_down:

... and 6 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fixed C++ import of timing functions

Fixes #45337

in general change looks good, need to fix whitespace issues reported by CI.

Please rebase, remove the merge commit, and squash the whitespace fixes into the original commit.


GCP DNS Connector change to the Function runtime version (3 to 4) introduced a bug that requires the host.json file to be updated with a higher minimum version for the extension bundles.

There's technically not a correlation between extension bundle versions and the Functions runtime version. However, starting with version 4.x the Functions runtime enforces a minimum version for extension bundles.
If you receive a warning about your extension bundle version not meeting a minimum required version, update your existing extension bundle reference in the host.json as follows: JSON
{
"version": "2.0",
"extensionBundle": {
"id": "Microsoft.Azure.Functions.ExtensionBundle",
"version": "[2.*, 3.0.0)"
}
}
Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

@rushriva : Please review and provide your feedback.Thanks!

@tjsawyer : Please resolve the conflicts.

We will check with Prathibha from FTE team and let you know the update.

@tjsawyer : we didn't seen any files changed

@tjsawyer: Could you please update the .zip package as well then only we can test.Thanks!

@tjsawyer: Could you please update the .zip package as well then only we can test. Thanks!

@tjsawyer: Could you please update the .zip package as well then only we can test. Thanks!


Added guidelines for WorldBrain's Webmemex

I was wondering if a 'Language/Framework' column can be added to the table..

@arpitgogia just saw the comment about Langauge/Framework. Nice idea!
I think we should definitely add the language column - that's usually a bias/factor of selection that people have. Not so sure about framework since not all projects may use a framework. But it would be nice to have others weigh in here too.
cc @tapasweni-pathak @vaibhavsingh97 @amarlearning @jarifibrahim @anubhakushwaha

Thanks again, @arpitgogia!
I created #11 for adding a language/framework column. Please comment there.


[WIP] Add BSON to CodableRouting benchmark

This PR uses OpenKitten's BSON encoder/decoder to add BSON content-type support to the CodableRouting benchmark.
Two lua scripts for wrk are provided, specifying a POST content-type and accept type of JSON and BSON, respectively.  These can be used to POST the structure "value": "Hello World" to the /postHello route, which then echoes the same data back again with the requested content type.
This exercises Kitura's Content-Type and Accept type determination for Codable routing (https://github.com/IBM-Swift/Kitura/pull/1309) in order to select the appropriate encoder and decoder.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Add highlight for WinSeparator

Starting in 0.10, Nvim will be deprecating VertSplit in favor of WinSeparator for window borders. By default WinSeparator will link to Normal, which doesn't look great.

Added dedicated highlight for WinSeparator 4721ec1


Support multiple hosts

Should now be supported with https://github.com/chiangf/Flask-Elasticsearch/pull/10


Review IAM Crossplane resource

What: Follow instructions in README under package/aws
Why: Consistency among Crossplane packages

https://github.com/dfds/crossplane-configuration-dfds/pull/19


Add AWS link and a few extra notes to the release process

Current dependencies on/for this PR:

main

PR #63   👈

This comment was auto-generated by Graphite.

LGTM


Changed UI

changed the ui of the onetime intro page into a nice design and also changed the splashscreen of the app.

Fixes Issue
#51

Changes proposed
Succesfully changed the splash screen UI of the app.

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

Could you please provide me with the demo video of the application?

@AryanShivva,
Please rename the images based on their usage:

Kindly rename this drawable image appropriately.

Please note that this image contains a Canva watermark, which is not acceptable. Ensure that the image is free from any watermarks before using it.

Furthermore, once you have successfully resolved these issues, please send the pull request (PR) in a single commit by squashing them.

@AryanShivva
Any Update?

@AryanShivva Could you please provide me with the demo video of the application?

@AryanShivva please upload this.

https://github.com/Taaveez/Taaveez-android/assets/124916476/33ff6aa0-2d74-4bbf-b4e4-750936565315

https://github.com/Taaveez/Taaveez-android/assets/124916476/33ff6aa0-2d74-4bbf-b4e4-750936565315

Why isn't the splash screen shown the first time?

According to the code already written the splash screen will not appear for the first time when we are opening the app.One time Activity screen will directly appear for the first time.

@AryanShivva
Please rename the file "new_background.xml" and remove any mention of your name. Additionally, to avoid copyright issues, please ensure that any images used are free of restrictions or consider using a paid image.

Could you please provide a different image or use a paid version of the image? The current image you shared still has visible watermark signs. Thank you.

According to the code already written the splash screen will not appear for the first time when we are opening the app.One time Activity screen will directly appear for the first time.

Hi @AryanShivva this is kinda wrong UI, because Splash Screen should be shown each and every time.
Please update the code to show Splash Screen every time.

no response


EVM module: implement account destruction

Is your feature request related to a problem? Please describe.
Our Evm module does not support account destruction.
Describe the solution you'd like
Determine what is the desired behaviour (should we delete the account in the state) and implement it.

Account destruction (selfdestruct opcode) is a very big problem in Ethereum and generally with PMT, it will be removed from Ethereum in next hardfork.
The main problem is that big contract accounts can be removed causing very big updates to PMT that are very hard to correctly price in gas.


[Feature Request] Contour tools

What feature or change would you like to see made?
Will there be segmentation tools with contour instead of labelmap?
I don't quite understand what the difference is between contour and labelmap? Except that the contour uses Geometry and the labelmap uses Volume.
Why should we prioritize this feature?
Can you make tools with contour too?

There is a big push for this right now. You can try this example, and lot more coming very soon
https://www.cornerstonejs.org/live-examples/planarfreehandcontoursegmentationtool


pip install fails due to case-sensitivity for Cython vs cython

Starting from a quasi empty conda env, I try to install tofu with pip.
I get an error about Cython, from the setup.py.
Cython is not known, because cython is listed in the dependencies.
Note: installing with pip into a conda env works only if we use the pip that installed by conda into that same env (check before running pip install):
(base) [ DV226270 spica ~ ] conda create -n empty python=3.6 pip ipython
(base) [ DV226270 spica ~ ] conda activate empty
(empty) [ DV226270 spica ~ ] which pip
~/.conda/envs/empty/bin/pip
(empty) [ DV226270 spica ~ ] pip install tofu
Collecting tofu
  Downloading https://files.pythonhosted.org/packages/5a/06/cbb85978151b0854bd2d2f022cfad1ff15a77db88593aeb236cf671bcd89/tofu-1.4.1.tar.gz (19.6MB)
     |\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588\u2588| 19.6MB 12.6MB/s
    ERROR: Command errored out with exit status 1:
     command: /Home/DV226270/.conda/envs/empty/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-aptzshyh/tofu/setup.py'"'"'; __file__='"'"'/tmp/pip-install-aptzshyh/tofu/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-install-aptzshyh/tofu/pip-egg-info
         cwd: /tmp/pip-install-aptzshyh/tofu/
    Complete output (5 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-aptzshyh/tofu/setup.py", line 14, in <module>
        import Cython as cth
    ModuleNotFoundError: No module named 'Cython'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

In ipython, importing cython and Cython does not make a difference (except the file from which they are loaded, Cython is the init.py, cython is cython.py).
=> Replace all Cython occurences by cython

Doesn't pass tests... Cython is not a package
=> maybe the other way around ? Cython in requirements and setup instead of cython ?

Doesn't pass tests... Cython is not a package

"Cython" is not a package or "cython" ?

Ok this is normal... Related to issue #250 !! you are installing the old tofu version !!

OK I tried the other way around, it works for conda, but I need to create a pip sdist and try to install it localy to be sure. Will do it tomorrow morning

you can try to install the pre release version pip install --pre tofu

https://github.com/coleifer/unqlite-python/issues/30
I finally understood what's going on: setup.py needs python (and numpy) thus the installation always fail.
https://github.com/coleifer/unqlite-python/issues/30

Thanks,
I tried introducing cython and numpy in the setup_requires, but it still fails with the same error.
I think it is because it needs to import cython when first importing the setup (and before reaching the setup_requires line).
It would need first to import the setup without cython, read the setup_require line, install cython and then run the setup() function.
A workaround is proposed here:
https://stackoverflow.com/questions/37471313/setup-requires-with-cython

Why dont you leave Cython/cython as it was, and just add the setup_requires (i did this in a branch)
it's impossible it will fail...

I added the setup_requires,still fails... can you take over this branch from here ? I have a meeting in 15 min.
I have edited the setup.py to define all necessary workarounds, as described in the link, plus added a few comments (questions)
After pushing:

create a sdist: python setup.py sdist
=> will create a sdist in tofu/dist/tofu-version...
create an empty environment with just python and pip: conda create -n empty python=3.6 pip
activate that environment
try installing tofu with pip from the local sdist directly. It should work and handle all dependencies

Doesn't work yet...

I can confirm, after local a test, that hord-coding the numpy.get_include() path solves the problem.
The only remaining problem is the import numpy that happens before the call to setup()

This is a good description of our problem:
https://stackoverflow.com/questions/54117786/add-numpy-get-include-argument-to-setuptools-without-preinstalled-numpy

Opened a corresponding issue on setuptools:
https://github.com/pypa/setuptools/issues/1910

That's probably the way to go, what do you think (before calling the setup() function):
dist.Distribution().fetch_build_eggs(['Cython>=0.15.1', 'numpy>=1.10'])

from:
https://luminousmen.com/post/resolve-cython-and-numpy-dependencies
https://github.com/pypa/pip/issues/5761

The setuptools guys answered the opened issue and rediected towards:
https://www.python.org/dev/peps/pep-0518/
https://www.python.org/dev/peps/pep-0517/

That's probably the way to go, what do you think (before calling the setup() function):
dist.Distribution().fetch_build_eggs(['Cython>=0.15.1', 'numpy>=1.10'])

from:
https://luminousmen.com/post/resolve-cython-and-numpy-dependencies
pypa/pip#5761

I tried this, and it didn't work on my case..

Ok, so only PEPs are ledt as a solution, crossing fingers...


Register v0.1.0

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/93491
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" d3779bcea1d66e181cc8f7c54f8e6d35b4a0e42a
git push origin v0.1.0

@JuliaRegistrator  register()

Registration pull request updated: JuliaRegistries/General/93491
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 0c467fcadd3e85797082b555540ce452ebb04df4
git push origin v0.1.0

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/97529
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.1 -m "<description of version>" 6cde5ee549b301e6decbed1d3f700aa348898e97
git push origin v0.1.1

Also, note the warning: This looks like a new registration that registers version 0.1.1.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.


XBMC/UNLEASH X ALWAYS SHOWING BUSY FOR DVD DRIVE

Is this normal running latest titan bios?

This is going to be a duplicate of your other issue (#7), some DVD drives act weird with large HDD's in the system. It'll be investigated (and hopefully resolved) under #7.
But as I said in the other issue was that I was able to reproduce this behavior under M8+ without ever patching it to use Titan. It's possible the DVD drive is starting to fall out of spec (failing) but will require more investigation.