does run-sequence can make deps only run one same as gulp?

does run-sequence can make deps only run one same as gulp?
[02:51:46] Starting 'transactions:history:default'...
[02:51:46] Starting 'transactions:client'...
[02:51:46] Finished 'transactions:client' after 332 μs
[02:51:46] Starting 'transactions:history:sells'...
[02:51:51] Finished 'transactions:history:sells' after 5.04 s
[02:51:51] Starting 'transactions:client'...
[02:51:51] Finished 'transactions:client' after 15 μs
[02:51:51] Starting 'transactions:history:buys'...

transactions:client is dep for transactions:history:buys & transactions:history:sells
but it shouldn't run > 1 time
any options for this?

The way run-sequence works, it cannot do this nor can it support it. run-sequence works by asking gulp to run each item in the sequential list, then listening for the completion event, then starting the next. As far as gulp is concerned, each item in the list is run independently.
You'll need to remove the traditional dependency to support your use case. You could move the :history:buys and :sells tasks into "private" tasks, which are don't have any dependencies, then create new tasks with dependencies and use run-sequence to start the "private" tasks with the dependencies. That's obviously getting a bit convoluted, though.
gulp.task('$transactions:history:sells', () => {
    // existing task here
});
gulp.task('transactions:history:sells', ['transactions:client'], (cb) => {
    runSequence('$transactions:history:sells', cb);
});

gulp.task('full-sequence', (cb) => {
  runSequence('transactions:client',
        '$transactions:history:sells',
        '$transactions:history:buys', cb);
});

nvm i make a hack do this now gulp-run-seq-unique

Cool, works for me!


请适配AGP8

Please provide the following informations.

System version

Java version

Gradle version

Android Gradle plugin version

Booster version

Stack traces

please see this  PR


Spike: File based routing

Experimenting with file based routing.

I would suggest we close this while the hardest parts of the app are still not migrated to routes


Add github action to validate travis file

 Here is an overview of what got changed by this pull request:

Coverage increased per file
===========================
+ src/Config/Config.php  2
+ src/TransactionService.php  2
+ src/Mapper/RequestMapper.php  4
+ src/Entity/Browser.php  2
+ src/Entity/Payload/NvpPayloadData.php  8
+ src/Response/ResponseFactory.php  9
+ src/Entity/Redirect.php  8
+ src/Entity/Basket.php  2
+ src/BackendService.php  6
+ src/Entity/Device.php  4
+ src/Helper/XmlBuilder.php  10
+ src/Transaction/EpsTransaction.php  10
+ src/Entity/AccountHolder.php  1
+ src/Transaction/CustomFieldTransaction.php  15
+ src/Entity/Mappable.php  1

Coverage decreased per file
===========================
- src/Entity/CustomField.php  -2
- src/Helper/Enum.php  -2
- src/Entity/RiskInfo.php  -2
- src/Entity/TransactionDetails.php  -1
- src/Entity/CustomFieldCollection.php  -1
- src/Transaction/Transaction.php  -1
- src/Response/XmlResponseData.php  -1

See the complete overview on Codacy


Fix for external links

Fixes results for external links like the first one in https://www.gov.uk/search/all?keywords=uttlesford&order=relevance
I've not found any evidence of internal links that don't start with a '/' so I'm not sure why we're doing this check.
It's used in URI.join(Plek.current.website_root, path) and was added in https://github.com/alphagov/finder-frontend/commit/a408d1436707b960c86c2f26d3ca5d78615eb9ea
It's used in absolute_url_for in application_helper.rb, but that code treats the path the same whether it has a leading slash or not:
irb(main):002:0> URI.join("https://www.gov.uk", "bernard")
=> #<URI::HTTPS https://www.gov.uk/bernard>
irb(main):003:0> URI.join("https://www.gov.uk", "/bernard")
=> #<URI::HTTPS https://www.gov.uk/bernard>

The test fixture seems to indicate that it might have been specialist content from back in the day that was missing the leading '/', but this looks to have been fixed.

I wonder if even need a path method now?

Yeah, I wanted to make this quite small in case we needed to roll back


Implement/Update CRUD operations over a Post entity

Including repositories, services, controllers, UT/IT
Include api versioning strategy
Include automatic API documentation generation

Will be completed when #3 and #4 are completed


[Proposal] Make rocksdb an optional (disabled by default) dependency for snarkos-errors

💥 Proposal
I'm using snarkos-algorithms crate which depends on snarkos-errors. It takes forever to compile because it includes librocksdb as a dependency. This is a very heavy dependency to pull in which isn't getting used. It should be optional.

looks like it is already an optional feature! Not sure how I missed that

It looks like https://docs.rs/crate/snarkos-algorithms/1.1.4/source/Cargo.toml.orig might be the culprit for me in this situation, it's enabling the default feature of snarkos-errors. Perhaps not all of the default features need to be enabled?

It would be nice if it could be disabled by default, and not so aggressively enabled by other snarkos crates. Currently I have to disable a bunch of default features in other crates, and then I'm assuming go fishing for enabling features like "snarkos-algorithms/pedersen-parallel" to enable better performance.

Somehow librocksdb has snuck its way into my dependencies again via snarkOS libraries, it's really difficult to figure out which features might be causing it and from which crate. This is a real pain point in Rust for me.

This is no longer applicable, as snarkVM (which currently houses what used to be snarkos-algorithms) no longer depends on rocksdb; snarkos-consensus no longer depends on rocksdb by default either.


Created method, which check state of icon.

Method returns true if icon is enabled, otherwise false. Sometimes we need state for restoring the state of activity/fragment, show/hide other views if state is enabled/disabled.

Accepted. I will rename method to isEnabled, and include it in next release

:) 👍


Generate multiple passwords within one secret containing multiple key-value pairs as template

Is your feature request related to a problem? Please describe.
The problem is that I would like to generate a secret whose template contains multiple key-value pairs via the Password generator. The problem has been highlighted in discussion-3820. At the moment you can only generate one password per secret which is not very flexible.
Describe the solution you'd like
As describe in the referenced page one idea might be to simply allow the user to provide, as parameter, the generated output key name (e.g. refKeyName or overrideKeyName), instead of defaulting it to password (although password might still be the default if no value is provided).
Here an example:
---
apiVersion: generators.external-secrets.io/v1alpha1
kind: Password
metadata:
  name: random-string-0-9-a-z
spec:
  length: 64
  digits: 5
  symbols: 0
  symbolCharacters: ""
  noUpper: false
  allowRepeat: true
---
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: auth-secrets
spec:
  refreshInterval: "30m"
  target:
    name: auth-secrets
    template:
      type: Opaque
      data:
        key1: "{{ .mykey1 | toString }}"
        key2: "{{ .mykey2 | toString }}"
  dataFrom:
  - sourceRef:
       generatorRef:
         apiVersion: generators.external-secrets.io/v1alpha1
         kind: Password
         name: random-string-0-9-a-z
         refKeyName: mykey1 # --> here the key name override to be referenced within the template section
  - sourceRef:
       generatorRef:
         apiVersion: generators.external-secrets.io/v1alpha1
         kind: Password
         name: random-string-0-9-a-z
         refKeyName: mykey2   # --> here the key name override to be referenced within the template section

It would be nice to have this as example in the docs. Took me a while to figure out how to use generatorRef with rewrite.

Hi
This is in the examples
https://github.com/external-secrets/external-secrets/blob/main/design/005-secret-generator-group.md?plain=1
However the examples have an error (at least in 0.9.18) where the "-" in the rewrite is not accepted.


packfile: improve Index memory representation to be more compact

Instead of using a map for offset indexing, use a sorted slice.
Binary searching is fast, and a slice is much more compact.
This has a negligible hit on speed, but has a significant impact on
memory usage, especially for larger repos.
benchmark                         old ns/op     new ns/op     delta
BenchmarkIndexConstruction-12     15506506      14056098      -9.35%

benchmark                         old allocs     new allocs     delta
BenchmarkIndexConstruction-12     60764          60385          -0.62%

benchmark                         old bytes     new bytes     delta
BenchmarkIndexConstruction-12     4318145       3913169       -9.38%

Signed-off-by: David Symonds<EMAIL_ADDRESS>

@erizocosmico


Issue3523 mbp international settings v2

Fixes #3523
Replaces:
https://github.com/DoSomething/dosomething/pull/3640

Assigns Mandrill template name based on the $origin of the request:

user_register
user_password
campaign_signup
campaign_group_signup
campaign_reportback

and the dosomething_settings_get_affiliate_country_code().

All templates have been created and named in MailChimp based on this naming pattern. The MailChimp templates have been exported to Mandrill.

Looks way cleaner, awesome job @DeeZone! All looks good, just two questions:
1 - How does the name change in Mandrill work with deploying this?  Did you just take copies of the current templates as rename them with the country codes? Emails are currently working right? :rabbit:
2 - Just double checking - have you tested for local US and international?

@aaronschachter Yes, copies made of the existing Mandrill templates with entries for each affiliate as well as "us" - 9 countries x 5 templates = 45 new templates created. I'll remove the current templates once the new deployment starts to use the new / duplicated from the current templates.
I worked with @sergii-tkachenko to create tests of both us and the related affiliate sites on my local. The deployment process is:

bin/drush-intl vset --yes message_broker_producer_rabbitmq_password K*&%$@!**
Necessary to set password through drush script to keep password out of version control
Set environment variables for RabbitMQ configuration : @sergii-tkachenko has suggested this can be done on the production server:

DS_MB_RABBITMQ_HOST="<IP_ADDRESS>"
DS_MB_RABBITMQ_PORT="5672"
DS_MB_RABBITMQ_USERNAME="dosomething"
DS_MB_RABBITMQ_VHOST="dosomething"

enable message_broker_producer and dosomething_mbp modules as part of the affiliate site deployment: perhaps this can be done by updating /html/profiles/dosomething/dosomething.info?

@DeeZone That's an good idea, but you'd need to test it out first:

get a clean affiliate db
add them to the .info
run a ds build for international

:+1:


install not working

hello i followed the readme but i cannot get the success message to pop up after entering cf ~/.cheat into my alfred box.
am i missing a step?
this is what i did:

download the workflow
double click workflow to import into alfred
open the alfred search and type cf ~/.cheat

im on a mac m1 / os monterey version 12.3.1

Hey @dvnschmchr can you try again? Monterey 12.3 isn't support that that time and now we have a new release. That should work. Let me know!

Close as fixed.


Release Pipeline failed

For details see: https://github.com/terraform-ibm-modules/terraform-ibm-cos/actions/runs/8580148041

/home/runner/work/_temp/6084a9d2-0810-4d30-8bf8-e658cdda8b00.sh: line 1:  1971 Segmentation fault      (core dumped) npm install semantic-release @semantic-release/changelog @semantic-release/exec @semantic-release/git @semantic-release/github


Update log_shipping_to_elasticsearch_part1.md

Edited text under "Performing a GeoIP Lookup" to match screenshot

Thanks for catching that, @dougsfo!


How to mock tl.getEndpointDataParameter and getEndpointAuthorizationParameter?

Hi guys,
How can I mock in a test a call to tl.getEndpointDataParameter or getEndpointAuthorizationParameter?
Thanks,
Daniel

Try setting the env variables:
process.env[`ENDPOINT_AUTH_PARAMETER_${endpointId}_${authKey.toUpperCase()}`] = authVal;
process.env[`ENDPOINT_DATA_${endpointId}_${dataKey.toUpperCase()}`] = dataVal;

@fszlin Thanks for your help here. I'm also trying to mock the responses to these calls. I've tried using the environment variable names you suggested but I'm still getting an Endpoint auth data not present error. Do you have any examples of how you've used this in the past?
In my case I'm trying to test an Azure RM service connection, but I don't know what @damadei is trying to test.

@fszlin is correct.  that does mock it including the response.  When you call the tl.getEndpointXXX methods, it reads those environment variables.  So you're injecting data by setting the envvars.
What happens in the non mock real world case is the user selects the endpoint instance.  When the server sends the job to the agent, it includes the end point data.  The task worker process sets those environment variables (same as inputs), then invokes your script out of proc.  So the lib is think wrappers over envvars and stdout.

@bryanmacfarlane thanks for your help. I've now managed to get this working with the TaskMockRunner. Here's the sample code I've used in case anyone else has the same issue in future:
var endpointId = 'armService';
tmr.setInput('armService', 'armService');
process.env[`ENDPOINT_AUTH_PARAMETER_${endpointId}_SERVICEPRINCIPALID`] = "00000000-0000-0000-0000-000000000000";
process.env[`ENDPOINT_AUTH_PARAMETER_${endpointId}_SERVICEPRINCIPALKEY`] = "dummy-key-value";
process.env[`ENDPOINT_AUTH_PARAMETER_${endpointId}_TENANTID`] = "00000000-0000-0000-0000-000000000000";

I'm going to close here since the issue seems to be resolved. Feel free to reopen if appropriate.

Since this is the only place I have found details on how to mock out the service endpoints I'm adding what looks like the new way for this to work.
I am unsure on how it handles the casing for the passed parameters in the json object so those might need to be adjusted for your use cases.
var endpointId = 'armService';
tmr.setInput(endpointId, endpointId);
process.env[`ENDPOINT_AUTH_${endpointId}] = { "parameters":  {
    "serviceprincipalid": "00000000-0000-0000-0000-000000000000",
    "serviceprincipalkey": "dummy-key-value",
    "tenantid": "00000000-0000-0000-0000-000000000000",
    }
}

Hi, is there some official documentation on this? I need to mock a generic service and I always get an Endpoint not present: serviceEndpoint error.
Here's my code so far:
const endpointId = "serviceEndpoint"
taskRunner.setInput("serviceEndpoint", endpointId)
process.env[`ENDPOINT_AUTH_PARAMETER_${endpointId}_URL`] = "hello"
process.env[`ENDPOINT_AUTH_PARAMETER_${endpointId}_USERNAME`] = "hello"
process.env[`ENDPOINT_AUTH_PARAMETER_${endpointId}_PASSWORD`] = "world"

I also tried
const endpointId = "serviceEndpoint"
taskRunner.setInput("serviceEndpoint", endpointId)
// @ts-ignore
process.env[`ENDPOINT_AUTH_PARAMETER_${endpointId}`] = {
  "parameters": {
    username: "hello",
    password: "world"
  }
}

For those who stumble upon the same situation... I just have inspected the code and found my answer there: https://github.com/microsoft/azure-pipelines-task-lib/blob/master/node/task.ts
Be sure to check which functions you call and then just search for those functions in that task.ts file. You will find the actual environment variable names, for each case.

While this is old and closed it seems to be the primary search result for this question.
Many of the results above work but they are hooked into the underlying implementation and are therefore brittle.
The "Correct (TM)" way I have discovered is to use tmr.registerMockExport.

function endpointLookup(endpointId, param) {
	const endpoints = {
		serviceEndpoint: {
			'AzureKeyVaultDnsSuffix': 'example.com',
			auth: {
				"parameters":  {
					"serviceprincipalid": "00000000-0000-0000-0000-000000000000",
					"serviceprincipalkey": "dummy-key-value",
					"tenantid": "00000000-0000-0000-0000-000000000000",
				}
			}
		}
	};
	return endpoints[endpointId] ? endpoints[endpointId][param] : null;
}

tmr.registerMockExport('getEndpointDataParameter', endpointLookup);

tmr.registerMockExport('getEndpointAuthorization', function(endpoint) {
	return endpointLookup(endpoint, 'auth');
});

tmr.setInput('endpointId', 'serviceEndpoint');


[CircleGraph] Remove no-effect return statement

This commit removes no-effect return statement.
ONE-vscode-DCO-1.0-Signed-off-by: Seok NamKoong<EMAIL_ADDRESS>

Sorry but our analyzer does not find every warnings at once.... -_-;


Automate Zipping for Deployment

https://peteris.rocks/blog/creating-release-zip-archive-with-msbuild/

I'm declining this because I'll almost never change the manifest.json.


fix the "can't convert pgf.preamble to stri" bug when i import sciai

Signed-off-by: Zdmai<EMAIL_ADDRESS>I build sciai from src, but there is a bug when I import sciai
bug is "ValueError: Key pgf.preamble: Could not convert ['\usepackage[utf8x]{inputenc}', '\usepackage[T1]{fontenc}'] to str" in the sciai/utils/plot_utils.py file
MacOS's container
OS: Debian GNU/Linux 12 (bookworm) aarch64
Kernel: 6.6.26-linuxkit
aarch64
python 3.8.19

 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.


The name of the containers in service that has datavolumes linked to other service has missing numbers in their name field.

Server Version - Build from master on May 19
Steps to reproduce the problem:
Create environment
Create service1 with scale 2.
Create service2 with scale 2 and mount volumes from service1.
Activate environment.
The name of the containers from service2 had 2 instances but the name of the instances ended with "_2" and "_3" instead of "_1" and "_2"
select id,name from instance order by id;
|  7 | test554724_test5306_1   |
|  8 | test554724_test23719_2  |
|  9 | test554724_test5306_2   |
| 10 | test554724_test23719_3  |
|

Tested with server version - V0.21.0
I am not able to reproduce the issue where the container names have missing numbers.


Move to Bootstrap 4

@danroth27 i'm doing it already. If all works i'll do a PR

Done https://github.com/aspnet/Blazor/pull/619


game crashed when pressing note

Describe your problem here.
I was testing player sprites for a mod I was making and

this is probably a json error because uh idk I used the player sprite and the entire sheet broke because the sprite creator has one sheet that could be the problem
TactieP but zipped so i can send it.zip
m
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No.

ignore the build stuff lol its not lua i was just editing character files

Wait what version are you in

Oh nevermind, considering you are in (possibly) the latest version, you might wanna try fixing your character animations a bit, since I only found out it pinpoints to the boyfriend idle check thing.

well thats weird because i edited the other json in the same way and it worked


ContentTree - Event handling for Ctrl+Click and Shift+ArrowKeys

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the Modus style guidelines

Proposal
Currently the ContentTreeItem component handles the ItemClick event. We'd also like similar event handling for cases involving Ctrl+Click and Shift+ArrowKeys or a single event handler for when the selection is changed.
Motivation and context
In Creator we use the Content Tree to reflect the structure of the 3D object hiearachy in the in the viewer. Whenever we select an item in the ContentTree the same 3D object needs to be (de)selected. Single selection can be handled though the ItemClick event, but this does not handle cases where Ctrl+Click and Shift+ArrowKeys is used to select an item.

What is the issue regarding ?
@trimble-oss/modus-web-components
Priority
Medium
What product/project are you using Modus Components for ?
Creator
What is your team/division name ?
Nexus
Are you willing to contribute ?
Maybe
Are you using Modus Web Components in production ?
Not yet.

Maha to check with Robert for contribution

hey @robert-nagy, wanted to check with you if are interested in contributing?

Scope update: We have decided to introduce a new event called itemSelectionChange in the TreeView component to capture all selection changes, whether made by keyboard or mouse.


Consider stricter validation of Envoyfilter

Not sure if this is possible - but I applied config like
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: custom-protocol
spec:
  configPatches:
  - applyTo: CLUSTER
    match:
      context: SIDECAR_OUTBOUND
    patch:
      operation: ADD
      value:
        name: redis-test
        type: STRICT_DNS
        lb_policy: MAGLEV
        connect_timeout: '0.5s'
        load_assignment:
          cluster_name: redis-test
          endpoints:
          - lb_endpoints:
            - endpoint:
              address:
                socket_address:
                  address: redis-test
                  port_value: 6379

spot the error? address is indented wrong. We get no indication, it is silently ignored.
In theory we could possibly do strict marshaling here. In practice this might be complicated and even harmful (new field, for example, not in go-control-plane - but would we end up dropping it anyways? I think maybe yes)
cc @mandarjog @rshriram @esnible

I have been thinking about this recently, the best proposal I have is to have an istioctl envoyfilter validate, that validates against a specific version(s) of istio-proxy.  We should be able to validate again istio-1.6 and 1.7 for example, and see if everything works out. This requires the binary to understand multiple versions of proto. Either it will load proto descriptors and process them or find another way. The outcome should be to fail these types of validations.

Related: https://github.com/istio/istio/issues/24743
@mandarjog Our plan was always that istioctl version X validates for the version of Envoy and Istiod version X, possibly with support for +/- 1 version.
The CRD itself has several Objects and may be likely to be unmarshalled by istioctl even if it was intended for another version.
The Analyze framework has access to EnvoyFilter workloads selector, and the pods metadata, but I don't know if the injector currently adds anything to the annotations, nothing like istio.io/injector-version=1.7.1, that could be used so that istioctl analyze knows what version of Envoy to expect on the running pod.
@howardjohn I'd like to set the milestone for 1.8 and, at a minimum, validate and analyze against behavior that used to be possible in Istio, but is no longer possible, as well as unknown Object types.  Would you support that?  I recall at one point you felt it should be up to Envoy to reject, rather than trying to duplicate that in istioctl.

@esnible  proto.UnMarshal followed by proto.Validate() only works for a specific version. This part is trivial to implement.
However we really need to validate against multiple versions, not just the latest version.
If v+2 added a new enum value that is lacking from v+1, we don't want to pass validation because istioctl is built against go-controlplane version v+2.

@esnible one thing to be careful of is the EnvoyFilter needs to be valid by both Istiod AND Envoy.  The Istiod "valid" is a bit weaker, it just needs to unmarshal, but this is important for things like new/removed APIs - if we don't import them in the control plane, they cannot be used.
FWIW my specific issue was just about strict unmarshaling objects - I think you are discussing deeper validation of the actual object itself right? My concern with that is there may be 2 EnvoyFilters merged together, that form a valid one - its very tricky.
Its also tricky that we have a shared validation logic for multiple revisions... envoyfilter is fun


Cannot install a them through slack-themes

Hey team, seems it has stopped working for quite a while. I could see installation success message but nothing changed after restarting the slack.
Current OS:  macOS Catalina 10.15.3
Current slack ver.: 4.3.3

This question will become recurrent probably, so here's what to do:

type npm uninstall -g @mallowigi/slack-themes
type npm install -g mtslack
type sudo mtslack and check that you have latest version
reapply

Tell me if it solves it


SQL language in status bar

SQL Language status bar is present when editing Kusto query

Doesn't meet the bar for September since this isn't a regression.

Moving to dashboard, since the screenshot doesn't show anything notebook related.

Moving to dashboard, since the screenshot doesn't show anything notebook related.


Add g_idle_add()

I don't get what you want to do. If the function g_idle_add isn't binded, why uncommenting it ?

I was expecting it to be used soon. If you want the whole thing at once, that's allright too...

I'd prefer, otherwise it doesn't make sense. If didn't comment all unused functions, the only we'd have to know they're not binded would be with rust compiler warnings. Not very wonderful...
I'll let this PR open. When you have binded the function, please notify me.
Thanks in advance ! =D

I don't see anything wrong with having unused definitions in the sys crates (as long as they're correct).
Here, Rust is capable of expressing the SourceFunc type so substituting a gpointer for it doesn't look good.

@gkoz: It's mostly to avoid warnings and make easier the future bindings. Otherwise, we could just uncomment all the non-binded functions and say it's fine. Or maybe I didn't understand the point here ?

There hardly can be such thing as an unused published member of a library, so there won't be any warnings.
Uncommenting everything wholesale would be problematic because someone has to review the definitions so it makes sense to avoid that effort while there's no need for it.

Hum... Okay. But then why not providing directly the binding for this function ? That's where I'm lost.

I'll try to add the binding.
@gkoz Regarding GSourceFunc, I was basically copying what can be seen in g_timeout_add() - GSourceFunc is commented and a version with gpointer is used instead. Is there a reason for this?

@vojtechkral: It was to avoid boring stuff haha (super explanation !). It was a (successfull) test I made, I wanted to do the same with a closure but for now there is an ICE in rust for it and I didn't try to avoid it (I'm working on cairo's documentation and rustdoc). For me, it's totally fine, it allows to avoid annoying stuff but I'm not sure that's the best solution... I'll dig to find more information.

This is why we shouldn't be committing hacks. People try to stay consistent with the hacks, leading to more hacks.

Well, that's not really a hack.

Ok, I added a binding.
Using only a fn() for now, because I have no idea how to manage the lifetime of a closure. Glib doesn't seem to provide any means for this like it does with signals.
Here's code I used to test the thing: https://gist.github.com/vojtechkral/ea7112bff8eaff0d0ff2

"to test the thing". It sounds very scary that way haha. Thanks a lot for your work ! I wait for travis and appveyor's confirmation and I merge.

Wait wait I don't think this is ready to merge yet, there's a commented-out function header with closure which I'm not sure what to do about yet.
A version with only a fn() would probably be ok though, at least for now...

If you manage to make work a closure by passing it as a pointer, it could be nice. You can try if you feel like it. ;-)

You want g_idle_add_full actually, not g_idle_add. It's similar to g_signal_connect_data. Only, there we box a closure twice and give Box<Box<Fn()>> as user_data, but here I'm leaning toward Box<RefCell<Box<FnMut>>>: the added RefCell will make sure we don't violate the mutability guarantees.

Right, thanks. I'll have a look at how the signal closures are done...

@gkoz In the gtk signals.rs, I don't understand how Box<Box<Fn>> can be cast as a pointer to function for C... Isn't it a double pointer?

Note also how the event handlers return the Inhibit newtype on the Rust side to make the meaning of the return boolean explicit. This case calls for a similar type, the glib docs suggest Continue, it could also be Repeat.

@vojtechkral

In the gtk signals.rs, I don't understand how Box<Box> can be cast as a pointer to function for C... Isn't it a double pointer?

The closures are passed as user_data, any cdecl functions needed -- the callbacks (that in turn call the closures) and the destructor are plain functions.

Okay, I think I get it, will try to implement...

This is the code so far. Problem is if I try to use the FnMut() version instead, I get
error: cannot borrow immutable `Box` content as mutable

in the trampoline function.
PS. I'll add a proper return type later.

That's weird but can be sidestepped by using the DerefMut trait explicitly: http://is.gd/lOw3HJ
Don't forget the 'static bound BTW. I've been guilty of this too. The inner box needs to be Box<FnMut() + 'static>.

It works now, yay :-)
In the future, timeout_func.rs and maybe others would probably benefit from some of the machinery in Idle. Maybe merge? Not sure how to structure this. Also documentation...

We might take a moment to think how this and the timeout functions will for together, the implications for the module name so we don't have to rename it ina week.

Sure, by all means.

One way to approach this could be to name the module source and the function idle_add, move the timeout::add function into it as timeout_add, and any other g_source_* bindings could be added in the future.

@gkoz: Isn't it possible to just move every non method function in a global namespace (like source if you want) ? It could be nice to give an easy access to "global" gtk functions.

@GuillaumeGomez This is mostly about the internal organization.
I don't mind reexports in the topmost namespace. I've come to believe that the flat namespace is a necessary evil after all, because that's what other language bindings do. The module might not even be public.

@gkoz: My bad, I thought you wanted to create a file and a namespace for each type of global gtk function. It seemed a bit too much for me. Splitting these functions inside modules is good, we'll just reexport them as one.
I don't think the flat namespace is "evil". However, if you have a better solution in mind, I'd be glad to know it ! ;-)

My current preference is that the types and functions a user might need would be reexported at the top. (This will likely mean that the constants will have long names with just G_ or GTK_ stripped.) The "internals" stuff used by the bindings (utilities like glib::translate) would not be reexported.

Yes, that's what I meant. I think we don't understand each other on this. Nervermind haha.

Naming the module source makes sense, but only if you know GSourceFunc, which I don't think users do, mostly. However, all these functions are part of The Main Event Loop , so maybe naming the module mainloop would make sense? As in mainloop::add_idle(|| {  ... }) etc.
Just my two cents

They call it MainContext mostly though. And it's a bit of a rabbit hole from there. But since we're going to reexport the functions at crate root, they users won't have to hunt in the source module for them.

I can convert the idle module + I suppose the timeout module to the flat model, but I reckon the scope of this PR shouldn't be broader than that right?

Yeah, the idle_add and a similar reimplementation of timeout_add seem sufficient for now.

Ok, done. Test code also updated. Hope everything's ok.

Ok, I've added ignore. Making the test compile would make them much longer and I don't even know if it can be done without gtk...

Good job, thanks!
ping @GuillaumeGomez

@gkoz: Pong ! Thanks for taking care of this PR too !
@vojtechkral: Thanks for your work, I merge !

@gkoz Out of curiosity, what was the reason for the 'static bound again?

@vojtechkral We pass the ownership of the closure to the library, its lifetime is outside our control, it could outlive everything. There are ways like this to make some lifetime guarantees, but they're not terribly ergonomic either.

@gkoz Okay, thanks!


Write the blame symbol when pickling BlamedUntyped

T.untypeds coming from methods defined in payload RBIs had no untyped blame symbol, because it wasn't serialized after we computed the initial global state.
Motivation

Better tracking of T.untyped.
Test plan

test-pr-on-pay-server + manual test:
# typed: true
class TestUntyped
  extend T::Sig

  sig { returns(T.untyped) }
  def test
    2
  end

  def local_call
    a = test
  end

  def unsafe
    T.unsafe(1)
  end

  def method_call
    Method.new.call
  end
end

[{"path":"https://github.com/sorbet/sorbet/tree/master/rbi/core/method.rbi","package":"<none>","owner":"Method","name":"call","count":1},
{"path":"untyped_track.rb","package":"<none>","owner":"TestUntyped","name":"test","count":1},
{"path":"https://github.com/sorbet/sorbet/tree/master/rbi/sorbet/t.rbi","package":"<none>","owner":"T.class_of(T)","name":"unsafe","count":1}]

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_OAwufhxIid0lbX
→ https://go/builds/bui_OAwuZDkQ3zytzG
→ https://go/builds/bui_OAwuUBEwiZhBtB

Can we write a test/cli test to verify the result programmatically? (test/cli/track-untyped looks promising, though the output isn't quite what you presented in the PR.)

@froydnj Unfortunately none of the typed blame stuff is tested, because it's only valid when we're compiling with --config=untyped-blame, which we don't run our tests with.
We'd have to build a new test harness for like CLI tests but they're only ever built with a specific build configuration.
I think we might want to do that? But it's bigger than this PR, and at the very least, this untyped blame mode is already only ever run manually by members of our team, so the blast radius on bugs like this is low at the moment.


Bring recent changes to large_image into the girder5 branch

Note commit ee99616 converts some recent logging changes to use the girder 5 style logging


Notification is blank on Android

I'm not sure what I'm doing wrong since others appear to be using this library for Android but my push messages are appearing "blank".  I receive the push and I can see the push in the log with the message but the device just shows a blank push notification with the app icon and no text.
I switched to react-native-push-notification for Android and now my push notifications are appearing with text so something is going on with the wix push library.  I haven't had time to dig into it and I'm not an Android expert so I'm not sure where to look.  Hoping someone else can come to my rescue so I can use a single library for push.

For me, this was because in the payload I was sending the notification in an object with key "notification". Changing it to data fixes it:
"data": { "title": "Hello", "body": "World!" }

I am also having this issue. I do not have a way to specify the payload format when sending from the firebase console, has anyone got a workaround aside from switching libraries?

Ok I'm back to this plugin because the RNPN plugin conflicts with the wix RNN library.  So I am trying to solve this blank push issue.  I'm actually seeing the opposite from what @nkov describes.  My push payload is currently using "data" and we switched it to use "notification".  Now with "notification" we can see that the push is not blank, we see the title and body.
However now we are getting double push notifications in the app for each single notification that we send.  After a little research I found that this is because gcm and firebase are both being included as dependencies in my app and are causing the duplicates as referenced here... https://stackoverflow.com/questions/38167125/firebase-send-push-notification-twice
Altering my build.gradle to exlude firebase appears to have solved my issue with the double notification and sending both data and notification payloads seems to have solved the blank issue.
compile (('com.google.android.gms:play-services:9.2.1')){         exclude group: "com.google.firebase"     }

This is deal breaker 👎  I'm also experiencing this.

@ricardofbarros @swellingt0n We had to figure out how to fix that as well since we could not modify the payload easily of a 3rd party.  I ended up forking the repo and making a minor change to GcmMessageHandlerService.java.
You can see the change here:
https://github.com/wix/react-native-notifications/compare/master...jbolter:master
Notice that I'm hard-coding the "app name" as the title of the push notification.

@jbolter Ultimately doing what @nkov did worked for me. This also solved a problem with the payload coming in undefined.

Problem is actual


Reboot warning after updates

Hello,
Does this warning depend on the Authentic theme or Webmin itself?
Warning! Recent package updates (such as a new kernel version) require a reboot to be fully applied
The server has already been rebooted, but not from Webmin.
When we upgrade the kernel, we usually do it from the command line, and reboot from the command line as well. But it seems it's not taken into account for that warning.

Hm. Well it has nothing to do with the theme. Webmin does it. I would think that this message goes away after you do manual update and then refresh (green button) the system info or just wait for 5 minutes.
Tell me, by the way. How do you reload kernel from cmd?

It's a DigitalOcean server. Dist upgrade from command line, choose new kernel in DO interface, reboot server from command line.
Thanks for the information.

So did it work? Did the message go away?

Howdy -- if you're using Ubuntu/Debian, and the "/var/run/reboot-required" file exists, Webmin/Virtualmin will notify you that a reboot is required.
That particular file is added by apt during the package update process, Webmin/Virtualmin just checks to see if it exists.
However, some things can be cached on the System Information screen.  It's possible that the Status Collection process hasn't run yet to update the page ("refresh" as Ilia suggested should help with that).

No it didn't work. I gave up and restarted the server from Webmin. :wink:

I see. Next time you could start from restarting Webmin first. Might have worked. ;)


tiny change to make appveyor back to work

This tiny change can get appveyor going again, as shown here:
https://ci.appveyor.com/project/xied75/sharpziplib/build/1.0.2
Ref: http://help.appveyor.com/discussions/problems/5145-build-failing
I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the SharpZipLib open source product (the "Contribution"). My Contribution is licensed under the MIT License.

Thank you!
-Neil

@McNeight Did you disable Appveyor? It didn't run after merge.

@xied75 I didn't disable AppVeyor, but I've been mainly using it from my account because it was easier to set up and use. I'll try to get this working either today or tomorrow.
-Neil


Add git-pull-request-create and alternate git-[pull/repo]-* icons

I'm afraid the bookmark and the tiny arrows do not have sufficient padding (a minimum of 2px spacing would be required by our guidelines).

These latest icons should probably use some sort of name along the lines of inbox-{...}, but they also have some guideline violations I'm afraid:

I'm afraid the bookmark and the tiny arrows do not have sufficient padding (a minimum of 2px spacing would be required by our guidelines).

Tiny arrows, which do you mean? One of them was misaligned by mistake, fixed now…

Tiny arrows, which do you mean? One of them was misaligned by mistake, fixed now…

E.g. the spacing marked with yellow here:

Other alternative would be:

Or, maybe:

Or, maybe:

Or, maybe:

@karsa-mistmere Fixed the repo icons…

So next time try to split up your icons with multiple PRs grouped visually. This makes reviewing icon PRs a lot easier.

Yeah, I intended to… Just bundled all the git related ones together for now, I’ll break them down more now there is a better consensus…
I’ll move the repo icons to a separate PR, as I am happy with the versions I’ve tweaked offline… and I’ll leave this one open for the git command icons, as the arrows on those are trickier to solve.

I'm not sure about the naming. The icons can be used for other things as git-repos as well so maybe we can give them more generic names.

Something like book-marked?

And the git-stash icons should be named inbox-in and inbox-out. You may better add them to your other PR https://github.com/lucide-icons/lucide/pull/998, I think they belong there.

Agreed…

Still not sure about breaking the consistency with other git accounts. But the arrows are better now.
I keep this one open, I want to know what others think of this.

Still not sure about breaking the consistency with other git accounts.

I don’t think they should be inconsistent either, but no reason why they have to be… For reference:

Top row are additions to the existing set, with the same proportions. Row 2 is the existing set. Row 3 are adapted to keep consistency with row 4, which are the arrow variations.
Thoughts? @ericfennis @karsa-mistmere @jguddas

can you provide these in a Figma file first so that I can play around with them a bit? :)

@karsa-mistmere https://figma.com/file/LjycnaUj6vyISowz39qNfX/lucide%2Fgit-icons?node-id=0%3A1&t=uGJ176Kfok9Y3qfg-1

See latest update… @karsa-mistmere @ericfennis @jguddas

For me, I prefer to keep the old designs for git-compare and git-pull-request and add your new designs as alternative icons. So under a new name or add this as as second variant.

Ok, so I reverted scaling them all up, and instead implemented the following summary of changes…

Added git-commit-vertical (and subsequently renamed git-commit[-horizontal])
Added git-pull-request-create
Refined git-fork to better match network
Refined git-pull-request-closed to better match other *-x icons (as most xes are 5x5 px, not 6x6)
Added git-[pull-request-[create/compare]-arrow/s variants, only making them slightly wider for better consistency than just scaling them up on both axes.
Added git-graph icon as outlined above

@ericfennis See latest https://github.com/lucide-icons/lucide/pull/995#issuecomment-1509285280.
cc: @karsa-mistmere @jguddas


⚠ SolarIn has degraded performance

In 9a34613, SolarIn ($SOLARIN_URL) experienced degraded performance:

HTTP code: 200
Response time: 244 ms

Resolved: SolarIn performance has improved in 66eb520 after 10 minutes.


Trojan in .zip?

Have been having trouble getting RDP Wrapper working correctly.  Downloaded .zip on one machine and ran it unsuccessfully.  Indicators are green, but no go.  I am trying to test on a second machine and get a virus warning when downloading.  Scanners shows the following:

Is this w32.riskware.rdp a valid component or is it a trojan?

I'm getting a malware report too.

Is it AV being overly cautious (similar to reporting perfectly safe password revealers to be malware) or have the files been compromised?

I too would like to know if rdpwrap is safe.

Duplicate of https://github.com/stascorp/rdpwrap/issues/277


Simplify Dockerfiles.

Reduce maintenance by pulling the whole package.

Minor change not worth reviewing/waiting for. Self merging.

Awesome! I was going to follow up with this clean-up as well. Took me a good head-scratching hour figuring out that those were no-ops :)


documentation for pre_edge_details

Documentation for the pre_edge function is missing pre_edge_details.
Happily, autobk_details is documented, as is fit_details.
xftf_details and xftr_details seem also to be missing from the doc.

@bruceravel ah, yeah sorry for that.  Will do.


added BWV-Aachen

Teszt

@marcelweitzel Please provide us with:

the school official website URL
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@marcelweitzel You might miss my previous comment. Please react.

Hi, i have notified my school for the proof and am waiting for their Response. I can provide the first 2 requests already, if you want

My Teacher has supplied me with this Screenshot. It contains his View of the DENIC-WhoIs request. Denic is the registry of the .de Top-Level-Domain.

the schools official homepage is this: https://bwv-aachen.de/home.html
here is a list of the it-courses the school offers: https://bwv-aachen.de/bildungsangebote/berufsschulen/it-berufe.html

@marcelweitzel Pull request merged. Thank you!


[Feature] Support higher refresh rates on Quest 2

https://github.com/leetvr/hotham/pull/197 adds the ability to specify required layers, but openxr hasn't released since adding support for XR_FB_display_refresh_rate yet.

This is now possible!


How to solve the problem of not having enough resource Daemonset? daemonset pod is pending since insufficient cpu

Description
eks 1.24  karpenter: v0.30.0
  Warning  FailedScheduling  112s (x3 over 5m33s)  default-scheduler  0/x nodes are available: 1 Insufficient cpu. preemption: 0/38 nodes are available: 38 No preemption victims found for incoming pod.

Either you have to set good calculative amount of limits on your provisioner or not set it

I don't think there's enough information here for us to know exactly what issue you are hitting. Can you share some Karpenter logs as well as the resources and configuration (Provisioner/AWSNodeTemplates) so that we can understand the issue a bit better?

We solved this issue by giving the daemonset-pods a higher priorityClass. Thus, other workloads are evicted and resheduled to other nodes in order to have enough resources for the daemonset-pod

@rbonn Is this issue resolved then? Can we close it?

What I described is a general strategy to guaranty that daemonsets are deployed on each node. We had similar issues and solved it this way.
But I'm not sure, if I understood the problem from @txjjjjj correctly.


The type or namespace name 'Auth' does not exist in the namespace 'Xamarin' when building iOS Release

Xamarin.Auth 1.3.1-alpha-2 works nicely in Xamarin Studio for Mac with an iOS Debug config. However, when using a Release config, I get the error below:

With Xamarin 1.3.0 everything works fine.

Same happens when trying to build an Android Release actually. Debug config is fine.

This is also happening for Debug after I do a "Clean All". Only way to remedy it is to remove 1.3.1, install Xamarin.Auth 1.3.0, build, and update to 1.3.1 again.

Can you test if 1.3.1-alpha-03 fixes this problem.
Maybe XS caches the dll somewhere. We are investigating this one.

Thanks. Fixed after cleaning and rebuilding.


Backend: Setup a response handler for the API

The response handler is a helper function and for that, we can put this under the helper's directory.
The benefit of this is for front-end developers working on this project, for which fetching appropriate data from the endpoint would be easier and well structured.
The Response Handler should be of this structure:
{
    success (Boolean): If the data is successfully retrieved then it should be "true", otherwise it should be "false".
    code (Integer): Appropriate status code of the API call like 501 for bad request, 200 for OK, etc.
    message (String): Message describing the action performed like "Post Created Successfully", "Failed to Create the Post", etc.
    data (Object): Return the data in this field if retrieved successfully, otherwise it should be "null"
}

Can I work on this?

Can you assign it to me under GSSOC'21? @purnima143

okay

@purnima143 could you assign this to me and add the relevant labels too?


Add UI in Polaris

Apache Polaris can provide a UI (in addition of the swagger one) providing:

swagger/REST API documentation
browse namespaces/tables/views (entities), with option to manage entities (create, remove, ...)
display TMS policies
display external catalogs (federation)
browse internal catalogs (with bridge between internal catalogs)
later, we can imagine to add new views in the UI (like OpenLineage, RBAC management, ...)

I will provide a quick mockup and proposal document to illustrate this.

We need a UI/JS guru :)

Hopefully we can have contributions here 😄


fix: load manifest from .terraform if exists

@aliscott I briefly tested this before running into the the module loader bugs that we discussed. Might need another test locally to make sure though.

@hugorut sorry I took over your PR here. Can you review the latest commits please?

@aliscott yup just running some tests locally to make sure of functionality

@aliscott LGTM and works 🙆♀ on my machine. I'm merging


[bug] semtechudp/backend read mutex lock nest

Is this a bug or a feature request?
is a bug, i test lorasever project ,found  my gateway-bridge will block on handlePackage where try to get the read lock. and i think the block is cause by read lock nest
1.function handlePushData is called by handlePacket which will get the read lock,
2.fucntion handleStats is called by handlePushData and will get the read lock too
here, if somewhere get the write lock betwen 1 and 2, will cause deadlock
What did you expect?
may you merge the code which i think it fixed
What happened?
What version are your using?
How can your issue be reproduced?
Could you share your log output?

pr


Feature request: Open in a new tab

When going through questions and answering, multi-tabbing is a superpower.
Request: Make use of proper anchor tags over buttons in squeak and make support work easier.

Related: #162


Test 6.9: assumes reply has content

  6.9. WINDOW_UPDATE
    × Sends a WINDOW_UPDATE frame
      - The endpoint is expected to send the DATA frame based on the window size.
        Expected: DATA frame
          Actual: Error: The length of DATA frame is invalid.
    ✓ Sends a WINDOW_UPDATE frame with an flow control window increment of 0
    ✓ Sends a WINDOW_UPDATE frame with an flow control window increment of 0 on a stream
    ✓ Sends a WINDOW_UPDATE frame with a length other than a multiple of 4 octets

The endpoint I tested had a 0 length document.

Thanks for the report. Please use the document of 10 or more bytes of size.
I'm going to change the code to skip this test if the document size is less than 10 bytes.

Fixed in c4e383421fb7fd265f5c5e3392341e301f86ff67.

Thanks.


Fix bug in setting baggage.

This is wrong.

Coverage decreased (-5.9%) to 88.693% when pulling 65381c97ec3c3c3c9b4a5d13b89ce9a44c952d29 on isaachier:master into 4acbc70bb6f672562c77ea272bae3ee7cd88691d on uber:master.


tests: device: Optimize prj.conf to improve code coverage

The testcase cover the Z_SYSCALL_HANDLER(device_get_binding, name),
but the lcov tool didn't include. And add CONFIG_NEWLIB_LIBC=y to
make lcov tool include the function.
Signed-off-by: Peng Su<EMAIL_ADDRESS>

I dig it more. But I have not found why enabling "CONFIG_NEWLIB_LIBC" can make " Z_SYSCALL_HANDLER(device_get_binding, name)" included in code coverage report. So the change in this code view is a workaround. Without the workaround,  Z_SYSCALL_HANDLER(device_get_binding, name) it reported as "not be covered", although it has been called by the code in "tests/kernel/device".
The test cases in "tests/kernel/device" are enabled only for platform "native_posix" "native_posix_64" "qemu_x86". So it is safe to enable "CONFIG_NEWLIB_LIBC" on those platforms.
We need to get this workaround merged today to make function code coverage of kernel 100% for 2.0 release. After it, we will continue to look into it to find the root cause. Maybe, it is a gcov or lcov issue. We just need more time.
@nashif  and all reviewer, are you OK to approve this PR?

Close this PR as the issue will be fixed with #18825.


Captcha

Recently we've been getting some automated-looking submissions. Maybe it's time to add a captcha?

Hi yall! Stumbled on a smart solution that can kinda solve the bot submission problem: put an input field that is not visible or reachable by human users and use that as a flag for form-filling-bots detection. If the field is filled on submission, you'll know it was a bot and can disable the submission button.
That can be achieved simply by placing it at the beginning of the form with inline css like position: absolute; left: -5000px;.
What do you think? Have someone seen anything like that? Is it safe to use? If this is not unsafe

Works for me, but I think this may be a problem for padding on mobile

@elioqoshi one alternative to that would be hcaptcha

hCaptcha is available, but it's still not an open source alternative for Google's ReCaptcha.
I was able to find https://visualcaptcha.net/ (deprecated), but it seems usable, and is Open Source.

Before a captcha is implemented - is the "automated job submission" issue still an issue? I'm not sure it currently is.
I would agree with getting one ready, so we can turn it on or off, but I would disagree with implementing one forever.

I think this issue can be updated to a sort of 'spike' to still investigate captcha for when we might need it or create our own.
I'll update the issue copy!


Incompatible with Requiem

https://www.curseforge.com/minecraft/mc-mods/requiem

https://github.com/Ladysnake/Requiem#mod-interactions


Hard coded compiler flags prevent debugging

Describe the bug
The hard coded compiler flag -s prevents the creation of debug symbols regardless of the build type.
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-literal-suffix -s") This is found in line 53 in the top-level CMakeLists.txt.
Expected behavior
The -s shall be removed from the flags. For the  -Wno-literal-suffix a check needs to be performed if this is needed and if there are ways to configure this through CMake. Cause this might result in different behaviours when using a different cpp compiler like CLang.

I just checked building with Clang 6. which does not recognize both of the CXX flags.


Improve QueryBuilder API for SELECT DISTINCT (JAVA-475).

Looks good.
This applies to 2.0 as well, I should have set the fix version on the jira ticket. Don't bother creating another pull request, we'll cherry-pick it when we merge.


⚠ Bofrost.nl has degraded performance

In ddfa831, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 2489 ms

Resolved: Bofrost.nl performance has improved in 97c9bcf.


Auto updating replaces last node text with default node text

This plugin updates/saves its view very often and this causes the last added child node to lose its text very often.
The text gets replaced by the default child node text.
The steps to reproduce this bug are simple, just work normally for 20 minutes on a Mindmap and you'll encounter this bug constantly on the way.
It doesn't matter if I confirm a new child note with tab or if I click outside the child node. The bug happens no matter how I confirm.
Also, when this bug happens, the heading gets centered as well if my Mindmap is oriented towards the left

I'm afraid this is the problem of saving data at the bottom ,  I call a API method to save data
  this.requestSave()

can you  provide more detail about this?

https://github.com/MarkMindCkm/obsidian-enhancing-mindmap/blob/main/src/MindMapView.ts#:~:text=}-,mindMapChange,-() {

https://github.com/MarkMindCkm/obsidian-enhancing-mindmap/blob/8bf4200caaaaf3b64beba608fe265f8eac846a33/src/MindMapView.ts#L68


Create baseline model

This issue was fixed as part of #21 and with the guidance of our advisor (on the main branch).


Menu seems empty / transparent

Hi,
I'm at a complete loss and I could really use some help. There's something I must be doing wrong but I don't understand what.
The issue is that while I do have a right menu, and the slide animation works correctly, it shows what appears to be a completely transparent UIViewController. I have checked and my menu VC loads correctly, and when I do a modal segue for instance, everything is displayed correctly. My app works like this: there is an initial tab bar VC. One of the VC's tab leads to a SlideNavigationController (set in IB), which leads to a view controller. This view controller displays a menu on the right, instantiated from the storyboard.
This is how I configure things.
- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{

  // Side Menu
  UIStoryboard *storyboard = [UIStoryboard storyboardWithName:@"Main_iPhone" bundle:nil];

  ReviserSideMenuViewController* sideMenu = [storyboard instantiateViewControllerWithIdentifier:@"ReviserSideMenuViewController"];
  [SlideNavigationController sharedInstance].rightMenu = sideMenu;

  [SlideNavigationController sharedInstance].menuRevealAnimationDuration = .18;
  [SlideNavigationController sharedInstance].menuRevealAnimator = [[SlideNavigationContorllerAnimatorSlideAndFade alloc] initWithMaximumFadeAlpha:.9 fadeColor:[UIColor blackColor] andSlideMovement:100];

/////
}

@interface ReviserViewController : UIViewController <SlideNavigationControllerDelegate>
//
@implementation ReviserViewController

- (BOOL)slideNavigationControllerShouldDisplayLeftMenu
{
  return NO;
}

- (BOOL)slideNavigationControllerShouldDisplayRightMenu
{
  return YES;
}

I would really appreciate some assistance if you have an idea why things aren't working for me. Thank you.
Screenshot below.

I have edited my code after following the issues related to "black screens" (basically my issue). So now, I put the code into the ViewController that should have a menu.
- (void) setupSideMenu
{
    // Side Menu
    UIStoryboard *storyboard = [UIStoryboard storyboardWithName:@"Main_iPhone" bundle:nil];
    ReviserSideMenuViewController* sideMenu = [storyboard instantiateViewControllerWithIdentifier:@"ReviserSideMenuViewController"];
    [SlideNavigationController sharedInstance].rightMenu = sideMenu;
}

This still does not work. Should I set it at a different moment?


'version' keyword in GDScript for conditional compilation at build time

Edit: The original idea was to do the optimisations when compiling GDScript to bytecode but I realised that it makes more sense to instead do conditional AST generation to allow for optimisation even when interpreted.
I would like to propose adding version as a keyword in GDScript to allow conditional code execution and compilation based on the OS and custom features that are available.
At first glance
version (“feature”):
appears to just be syntactic sugar for
if OS.has_feature (“feature”):
and when interpreted this would be the case.
The true advantage of the version keyword would be when GDScript is compiled to gdc bytecode.
The version keyword could be detected at compile time and used to conditionally compile blocks of code depending on the feature set of the chosen export preset/template.
This would result in faster runtime speeds, smaller bytecodes and potentially faster compile times as code that is never used by your exported game will be skipped.
It also (in my opinion) results in better looking code that is arguably easier to maintain.
Here are some psudocode examples of use cases along with their current, run time counterparts:
Debug statements:
Run time
func my_function (value):
    if OS.is_debug_build():
        print(“Some debug text: Arg=”, str(value))

    #Function body here

    if OS.is_debug_build():
        print(“Result: “, result)

    return result

Compile time
func my_function (value):
    version(“debug”):
        print(“Some debug text: Arg=”, str(value))

    #Function body here

    version(“debug”):
        print(“Result: “, result)

    return result

Different code for different platforms:
Run time
#Platform independent code here

if OS.has_feature(“android”):
    #Android code here

if OS.has_feature(“ios”):
    #iOS code here

if OS.has_feature(“web”):
    #Web code here

Compile time
#Platform independent code here

version(“android”):
    #Android code here

version(“ios”):
    #iOS code here

version(“web”):
    #Web code here

Custom features & else:
Run time
if OS.has_feature(“my_custom_feature”):
    #Do something
else:
    #Do something else

Compile time
version(“my_custom_feature”):
    #Do something
else:
    #Do something else

Tool scripts:
Run time
func _draw ():
    if Engine.editor_hint:
        #Draw lines and stuff

Compile time
func _draw ():
    version(“editor”):
        #Draw lines and stuff

or

version(“editor”): # _draw function not included at all so it doesn't even get called
    func _draw ():
        #Draw lines and stuff

At compile time when a version block is found the compiler can check if the specified feature is available on the export platform (OS.has_feature) and either include the code in the block or ignore it meaning no runtime checks are required.
e.g when exporting for android
#Platform independent code here

version(“android”):
    #Android code here

version(“ios”):
    #iOS code here

version(“web”):
    #Web code here

compiles as
#Platform independent code here

#Android code here

completely omitting the iOS and web code from the compiled bytecode.
This proposal is based on the version keyword featured in D https://dlang.org/spec/version.html

Was also proposed/discussed in the now-closed #6340.

I didn't spot that one when searching. Not much point keeping this open then so closing.

Reopening. Note the other issue is closed; there can still be merit in discussing this one.

So... that's basically a specialized #ifdef. Conditional script parsing is a recurring theme, notably in https://github.com/godotengine/godot/issues/26177 as well

Perhaps something like D's static if would be more appropriate then. It would allow of more complex conditions such as static if not OS.has_feature(“android”) or static if OS.has_feature(“OSX”) or OS.has_feature(“X11”) although there is no reason that we couldn’t have something like this:
version(“android”) and not version(“x86”):
    #somecode

vs
static if OS.has_version(“android”) and not OS.has_version(“x86”)

but static if does allow for stuff like this:
static if false:
    #Doesn’t matter what’s here; It will not be compiled

I think that version is a better solution than static if because it can specifically focus on the availability features rather than arbitrary conditions that would require compile time code execution, is less verbose, is less likely to cause confusion (‘what’s the difference between if and static if?’ asked every new user for all of time) and should not result in issues caused by values not being known at compile time as the only valid argument type is a string constant.
As for checking the engine version that could be added to OS.has_feature i.e. OS.has_feature(“3.1”) or OS.has_feature(“>=3.0”) or something like that and assuming that version is just a compile time check of OS.has_feature it could be used like so:
version(“>=3.1”) and version(“<=3.2.1”):
    #Code only for Godot Versions 3.1 to 3.2.1

In comparison to #ifdef and #ifndef an important difference between the C preprocessor and D’s version and static if is that the C preprocessor has to run over all of the code first before compiling happens while D does it all in place during the compilation step AFAIK. I’m proposing to do this the D way, in a single pass as part of the compilation step with no preprocessor required. If the condition isn't met then just skip the block entirely.
Now that I think about it there would need to be warnings for when code outside of a version block references code inside the block as that code may not be included in the build and the version block likely wouldn’t have its own scope.
version (“android”):
    var x = 10

print(x) # Error. Variable x undefined on non-android platforms.

After some thinking I realised that this could work by skipping unused branches when building the AST. This would make version more efficient than if even when interpreted and not just when the code is compiled to bytecode as if the feature specified in the version statement isn’t supported on that platform then that entire branch of the AST is never generated.

I would prefer a shorter syntax, for example:
$$debug:
    print("debug")

$$mobile:
    print("mobile")

@dalexeev In GDScript, we tend to favor easy-to-read constructs. Code is read way more often than it is written :slightly_smiling_face:

@Calinou There is already a short syntax for get_node(). The dollar sign is often used in programming languages to denote a variable. Feature tags are also kind of variables: the variables that are used when exporting a build. Why not use two dollar signs?
Also easiness is a relative concept:
#!ifdef debug
func _ready():
    # ...
#!endif

In any case, the functionality is primary, not the name.

At the very least, it would be nice to give up quotes and brackets. And it is advisable to choose a shorter keyword.
version debug:
    print("debug")

only debug:
    print("debug")

at debug:
    print("debug")

Nim uses the when keyword as a compile-time if statement, we could borrow that as to be more generic.

I think that we should focus on clarity rather than verbosity. Personally, I think that version provides the clearest explanation of what is happening in the code (only works too). If a symbol were to be used then it should be shorthand for a keyword and not similar to an existing symbol. I think that $$ is too similar to $ and would make code confusing for people who are unfamiliar with it. I don't think @ is used anywhere, that could be used as a shorthand if there is demand for it.
version Windows:
    Windows Stuff

version X11:
    Linux and BSD Stuff

version OSX:
    Mac Stuff

or
@32:
    const MAX_INT =<PHONE_NUMBER>

@64:
    const MAX_INT =<PHONE_NUMBER>854775807

I don't think @ is used anywhere

Actually, it's used to create NodePaths :slightly_smiling_face:

I don't think @ is used anywhere

Actually, it's used to create NodePaths slightly_smiling_face

You learn something new everyday.
How about ?
?debug:
    print("debug")
?release:
    print("release")

@Calinou I agree. I was just humouring some of the other suggestions as an alternative shorthand.

$ and @ are precedents for such shorthands in GDScript. That's why I suggested it. But

Maybe I was wrong, but getting rid of the extra brackets and quotes is also right.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Regulation of 1-phosphatidylinositol 4-kinase activity

Hi,
I'd like to request the addition of positive/negative
regulation terms in the process ontology for '1-
phosphatidylinositol 4-kinase activity' (GO:0004430).
PubMed Reference: 14676841
Thanks,
Emily
(P60033 +IDA)
Reported by: edimmer
Original Ticket: "geneontology/ontology-requests/1553":https://sourceforge.net/p/geneontology/ontology-requests/1553

thanks www.movied.org


Investigate design time support for tag helpers

What code is being generated at design time.
How do we work with tags that are not balanced/closed.

Nothing to do here, this is tracked a other issues.


File format conversion

Hello, I have a text version. RVM file, how do I convert that to a file that feeds rvmparser

Currently rvmparser only reads binary rvm files and writes rev files. So you cannot convert from text to binary using rvmparser.
Issues #63 and #64 address this, but nothing has been implemented yet.

Thank you for your answer. Can I directly convert the text file to binary


How to Change Ocr Detecting language other than english ?

Ocr only supports Latin language for now.
You don't need to change any option, it should just work for any Latin languages.

Ok Thanks @liuyl


Add deletion option to delete the old index

As mentioned #23, adding "deletion" parameter to deletion the old index after reindexing.

Thanks!


Link to main gleam project in readme

The gleam link in README.md linked to the gleam.vim repo (which is odd!), I think it was intended to link to the main gleam repo.

Oops, thank you.


[FEATURE] Cannot Delete Only Admin Account On System

So i tried and tested it if you delete your only admin account you cannot log back in obviously so however since I'm hosting with you guys I cant access command line so now I'm locked out until I can get back in
How to replicate
Delete all users including you own and boom you are stuffed

i guess you can just open a ticket and they might do it for you 😄

I did but if it because a popular thing they do there gonna want to avoid idiots doing it cough cough like me

Good idea to prevent main admin account deletion

agreed on this its a +1
but to fix this issue IF IT HAPPENS for time being
ssh back into the server with paymenter installed cd /var/www/paymenter
re run the command
php artisan p:user:create
create the new admin account and re-login back in with the newly created account via artisan


Proposal for adding rotate algorithm similar to std::rotate()

Ticket: https://bugs.swift.org/browse/SR-125
Discussion thread: https://lists.swift.org/pipermail/swift-evolution/Week-of-Mon-20151214/002213.html

The proposal text looks good. Can you squash into one commit and then I'll merge?

I have received a lot of feedback on Swift Evolution channel. I will incorporate the feedback and will squash into one commit by tomorrow morning.

Sounds great, thanks!

I have updated and squashed my PR into single commit.

I'm closing this pull request since I'm still struggling with the implementation due to lack of knowledge of Swift stdlib. I will reopen this pull request in a month time once I will get a better understanding of the stdlib.

@bolshedvorsky If you have concrete questions, I'd be happy to help on swift-dev!


read docs folders

i could be misreading the regex, but it appears that
https://github.com/github/linguist/commit/3837fed59e03f50047bc27efe2ff213baffdddcb
broke reading the docs folder for GitHub Pages

it is because linguist thinks "documentation" should not be included in language stats - here is fix:
$ cat .gitattributes
docs/* linguist-documentation=false

broke reading the docs folder for GitHub Pages

What do you mean by that? Linguist shouldn't impact GitHub Pages...

@pchaigno i am using "master branch /docs folder" for github pages - because of that, i have to put my javascript files in that folder - because of that - linguist thinks that my javascript is "documents" and doesnt count it towards the language stats
the fix is as i put above, but maybe this whole situation should be reviewed

Oh, okay. I thought it was breaking your GitHub page. It works as expected then.
We expect most users don't want their documentation to be counted in the language statistics (otherwise, Java repositories could end up tagged as HTML for example).


"Verwalten": Change colour?

https://pr792.tobira.opencast.org/!v/JIY8Mv2B44J: Is there a reason "Verwalten" is bluish while "Download" and "Teilen" are black?

The "Verwalten" button acts as a link while the other buttons each open a popup. We had some feedback that users were confused about the fact that both use the same style, even though they do different things.
So the different color (which is used on other links as well) is a way of how one could distinguish the two different button types. But if that is also/even more confusing, then it might be worth exploring different options.


Return types

I would like to add PHPDocs to add PHPDocs to all methods to add more return types. Would you be willing to merge this? If so, do you think that this would require additional tests?

Hi! I'd be interested in seeing a small sample of what changes you're thinking. I don't know if it would require additional tests

note: maybe use TypeDeclaration
https://github.com/rectorphp/rector/blob/main/docs/rector_rules_overview.md#typedeclaration

coding standards configuration added in https://github.com/php-curl-class/php-curl-class/blob/master/tests/.php-cs-fixer.php.
#794 #795 #796


test: enable testing for upcoming challenges

I ran them locally and they completed fairly quickly and all passed, so I think we can start including them in CI.

Do we not want to do this before the curriculum build step, as the .env file is overwritten?

Yep, thanks!

Also, is echo-ing the variable into .env the recommended way to handle env logic?

Generally no, but using .env works quite well since we have so many environment variables.  Echoing is just a cute trick to replace the variable, since the last entry takes precedence.  Bit of a hack, but I think it's fine here.

Unrelated:
This is nice to see:
16930 passing (7m)
666 pending

But, the number of pending has gone up from its typical ~512. So, it appears to be some percentage of the total tests run which go to pending.
Although, most of the literature I have read suggests the pending is just due to incorrectly formatted tests:
test("some test without a callback"); // results in a pending test

Interesting. I'd always assumed it was challenges without tests, but perhaps not. I'll keep this in mind.

While poking in code interview prep section I've noticed adding solution to the challenge reduces that number by one. So it seems to at least include such challenges.

I just realised! What happens when the final part of a project is reached, where there is no next part with solution seed?

In that case you need to provide a solution. I'm guessing we didn't document this 🤦

@raisedadead how about this?  I've put it on the same level as the localize job.

As we already have the tests running for the "production" replica, would it not be nice/useful to skip re-running these same tests with SHOW_UPCOMMING?
Sorry, for the trickled picking 🙈

Sorry, for the trickled picking 🙈

Never be shy of asking questions. It's better to ask than not.

would it not be nice/useful to skip re-running these same tests with SHOW_UPCOMMING?

Yes - it would save us a few minutes. However, we want to test them alongside the other tests.
That way, you are testing them alongside any additional tooling you make have (ex. challenge helpers, parsers, etc.), which may have custom code for one or the other types of the challenges (current or upcoming).
The idea with coding tests is they should be dumb. The idea with coding logic is they should be DRY and smart.
Duplication is good with tests and suites because you are not thinking about how you want to spread the net. You get the widest net possible and catch them all.


Added type annotations #376

Brief Description

Added type annotations to improve code clarity and enable static type checking.
Requirements / Checklist

[x] I have read the PR Guidelines.
[x] I have read the Documentation.
[x] I have updated the documentation accordingly(if required).

Related Issue

https://github.com/abhiTronix/vidgear/issues/376
Context

This change addresses issue #376 by adding type annotations.
Types of changes

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

Screenshots (if available):

@abhisheksuran You can proceed with next file, maybe pigear.py?


Crawler ideas [discussion]

(talking with @danohuiginn)

Crawlers should give feedback on last run, next run etc.
[opt] Crawlers should be scheduled automatically
Crawlers need to be fully incremental
Crawlers are configured via their crawler class (no in-DB json schmu)

Ok, got a good bit of work done on this today. Here's the scraper screen:

Looks like all of this is implemented :)


Kubernetes 1.8 reference docs

Release 1.8 API reference docs

This change is

Thanks @pwittrock. Do you want a tech review? Or should I go ahead and merge?

@steveperry-53 Would you mind manually browsing around and making sure everything looks good.  I a quick once-over and the links and formatting, but would be good to have a second look.
@mengqiy Would you mind looking over the 1.8 ref docs for kubectl and the apis on netlify and make sure there is nothing glaring?

@pwittrock Did these docs get included?
https://github.com/kubernetes/kubernetes/pull/47934

@pwittrock I'd like to see this included, if not now, soon after the 1.8 release.
https://github.com/kubernetes/kubernetes/pull/50532

@pwittrock lgtm with one concern: I want to be sure the cloud controller manager docs are included.

@steveperry-53 Sounds like a good opportunity for @chenopis to get familiar with running the docs updates scripts

@steveperry-53 The cloud controller manager docs are unrelated to this.

@steveperry-53 LGTM too, so let's merge this PR and handle the cloud controller manager in a separate one.

@pwittrock Are you ready for us to merge this? Or do you want to wait for a review from @mengqiy?

@steveperry-53 Let me TAL really quick.

LGTM
@steveperry-53 kubernetes/kubernetes#50532 is included in this PR.
kubernetes/kubernetes#47934 is not, since it is managed by another script. I will send a PR to update that part shortly.

@pwittrock @mengqiy Are the v1beta1 and v1beta2 sections in the correct place?
I see this at the top:
http://kubernetes-io-vnext-staging.netlify.com/docs/api-reference/v1.8/#deployment-v1beta1-apps
And I see this lower down under OLD API VERSIONS
http://kubernetes-io-vnext-staging.netlify.com/docs/api-reference/v1.8/#deployment-v1beta2-apps

They should be move up.


Include tslib as a Shared Dependency

Description
Currently, piral-core (and dependents such as piral) use tslib. As such it should be a no-brainer to also expose tslib as a shared dependency. This can significantly reduce bundle size.
Background
tslib contains all runtime helpers of TypeScript. These helpers are auto-generated by default, but can be referenced implicitly using the "importHelpers": true setting in the compilerOptions of the tsconfig.
Naturally, the tsconfig of a Piral instance / pilets should contain this flag and use the then shared dependency to reduce output sizes.
Discussion
Maybe other shared runtime libs (e.g., for async code the regenerator package comes to my mind) should be shared similarly. Right now, however, it is unclear how its exposed. Potentially, this should therefore be left to the specific Piral instance to decide.

Landed in develop.


manager: fix reloading stores

Small one to fix store reloading when an entity does not yet exist.

There are yep - they shouldn't be returned in the query though.


update k8s to 1.30 on all platforms

Thank you for submitting a pull request to the WrongSecrets Party!
What kind of changes does this PR include?

[ ] Fixes or refactors
[x] Platform support: updated to k8s 1.30
[ ] A new feature
[ ] Additional documentation
[ ] Something else

Takes care of 1 task of https://github.com/OWASP/wrongsecrets-ctf-party/issues/664
Checklist:

[x] All the contributions made are solely the work of me and my co-authors
[ ] I tested the changes in this PR (if applicable)
[ ] I added tests to ensure my change works (if applicable)
[ ] The PR passes pre-commit hooks and automated tests
[x] updated and tested AWS
[ ] updated and tested azure
[x] updated and tested gcp
[x] updated and tested minikube

aws testing started

Fixed allf or AWS:

k8s challenges work
AWS challenges are now highlighted
logging shows what is used in AWS

Started Azure AKS testing

Azure works :)


Prepare for cordova-android 9.0.0 (androidx) and new cocoapod versions (static libraries)

Description

Changes mainly include:
this mainly included:

switching from the old android support libraries to androidx
switching to static (pod) libraries for ios, since manual signing with the new cocoapods version otherwise would no longer work (dynamic libraries are not used a lot anymore and it worked without)
bumping the required cordova (engine) versions where needed

Related Issue

https://github.com/havesource/cordova-plugin-push/issues/63
Motivation and Context

Since we are updating the general cordova ecosystem due to using newer crashlytics versions (including cordova-android 9 and androidx), there were a few things that had to be done in order to make this plugin work again. And i thought it would be nice to just create a PR to the upstream repo for the time when the upgrade will happen in this repo as well.
How Has This Been Tested?

The changes were tested by first preparing/building (debug) cordova apps (cordova 9.0.0; cordova-android 9.0.0; cordova-ios 6.1.1), then testing the js API via a local server and (web) debugger tools. For IOS, a Google-Services.plist file was used. the pure APNS approach was not tested.
Screenshots (if appropriate):
Types of changes

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

Checklist:

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

Check failure doesn't seem to be related to PR at all

ci gh action might be resolved in the v2.1.2 version of setup-node btw: https://github.com/actions/setup-node/releases/tag/v2.1.2

Hi, would it be possible to resolve this issue please ?
The  use of androidx import is pretty important nowadays and it would be much easier to not have to modify the import manually.

Hi, would it be possible to resolve this issue please ?
The  use of androidx import is pretty important nowadays and it would be much easier to not have to modify the import manually.

Hi, is there any plan to merge it and release a new version in the foreseeable future? I'd be extremely helpful for my project which natively uses AndroidX.

This PR focuses on way too many tasks. Its scope should be narrowed down to one. In this case, maybe focus only on AndroidX.
The PR itself needs to

Rebased with master branch to fix the fix conflicts
Remove all iOS-related changes
Remove the changes done in PushPlugin.m.
Relabel the PR title and commit. E.g. breaking(android): use AndroidX library

Also, to use Android X right now, have you tried the following?

Install Cordova-Android 9.x
Install this plugins latest version 2.0.0
Install plugin cordova-plugin-androidx-adapter
Add the preference variable AndroidXEnabled with the value of true in config.xml<preference name="AndroidXEnabled" value="true" />

Dear ED220Webfactory, thank you for your great work. It helped me updated my push plugin.

This PR has become stale. Please check the latest release as it should support the latest Cordova Android.
The Android code was also refactored to Kotlin.
The iOS Firebase library is has been updated to 8.1.1 in the main branch.


Added Code

this won't work, create an issue -> get it assigned -> then open a pull request.
Don't forget to make a folder with you project's name the next time you contribute


Update tooltips for the Equation Button and Style Picker

Fixes #1160.
Description of the changes:

Updates the equation button automation name and tooltip to use different resource items
Add tooltips to the colors in the style picker
Fixed the issue where the white color has the wrong automation name

How changes were validated:

manually with narrator and validated the tooltips

/azp run


Bump promises dependency

Was able to manually verify CocoaPods. Was also able to verify SPM after pointing the manifest's GUL's dependency to the latest revision.


Automated cherry pick of #9506

Cherry pick of #9506 on release-6.3.
/cc  @hmhealey
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: b2fee541f5f86559f54758f8c199f5843905e706


isActive not working as expected with v-slot API

Reproduction
https://github.com/jakedohm/router-link-slot-bug-reproduction
Expectation
When I use the v-slot API on a router-link, I'm able to use the isActive property to set classes on the link, like so:
<router-link
  v-slot="{ isActive }"
  :class="[isActive ? 'bg-red' : 'bg-blue']"
  active-class="underline"
  to="/"
  >Home</router-link
>

What's really happening
The isActive property doesn't appear to be set on initial render, causing the bg-blue class to always be set, even if that link is active. Vue throws this warning:

@jakedohm when you use v-slot, the data is available on the children of the component, you need to create the a tag and add the different attributes yourself


Fund debit

Can't able to credit/debit float amount fund in any segment to clear the book.Only Integer digits are allowed.
Please check and update it again.

Dear User,
Thank you for showing interest in Breeze API.
Set funds() SDK does not allow to pass float values.
You need pass integer amount only.
[E29CF836]
Thanks & Regards,
Akash Sapkal,
Team Breeze API,
ICICI Securities

From: Jye2347 @.>
Sent: 30 July 2024 13:51
To: Idirect-Tech/Breeze-Python-SDK @.>
Cc: Subscribed @.***>
Subject: [Idirect-Tech/Breeze-Python-SDK] Fund debit (Issue #120)
CAUTION: This email originated from outside the organization! Do not click links, open attachments or reply, unless you recognize the sender's email address! Use 'Report suspicious Email' button in the toolbar to report Phishing mails. Beware!
Can't able to credit/debit float amount fund in any segment to clear the book.Only Integer digits are allowed.
Please check and update it again.
—
Reply to this email directly, view it on GitHubhttps://github.com/Idirect-Tech/Breeze-Python-SDK/issues/120, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3TQZ43HSLABBHJSTJOMZALZO5EH5AVCNFSM6AAAAABLV2XQJSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQZTOMRRG4ZTINY.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
Print this mail only if absolutely necessary. Save Paper. Save Trees." "The information contained in this e-mail and any attachments to this message are intended for the exclusive use of the intended recipient and may contain proprietary, confidential or legally privileged information. If you are not the intended recipient, please note that you are not authorised to disseminate, distribute or copy this e-mail or any parts of it or act upon/rely on the contents of this e-mail in any manner. Please notify the sender immediately by e-mail and destroy all copies of this e-mail and any attachments. Please also note that ICICI Bank or its subsidiaries and associated companies, (collectively "ICICI Group"), are unable to exercise control or ensure the integrity of/over the contents of the information contained in e-mail transmissions and that any views expressed in this e-mail are not endorsed by/binding on the ICICI Group unless the sender does so expressly with due authority of ICICI Group. Before opening any attachments please check them for viruses and defects and please note that ICICI Group accepts no liability or responsibility for any damage caused by any virus that may be transmitted by this email. Any unauthorized use of this email or any attachment(s) may be treated as infringement of right(s) including right of privacy under the applicable laws of India including Information and Technology Act, 2000. Thank you for your cooperation. Regulatory Disclosures are available on https://www.icicisecurities.com (for institutional business) and www.icicidirect.com (for retail business).


Failed to get machines: The network connection was lost.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Create a linux machine (e.g: Alpine)
Access by Warp termnial: orb -m alpine -u root
Close Warp
See error Failed to get machines: The network connection was lost. on Machine tabs

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Logs
If applicable, attach log files by opening the OrbStack menu and clicking “Show Logs”, or email them for privacy.
console.log
vmgr.log
Info report

OrbStack info:
  Version: 0.5.2 (50200)
  Commit: 1070de4aca19fab0c2c0b57d7c0bacca7812ca23 (v0.5.2)

System info:
  macOS: 13.4 (22F5027f)
  CPU: amd64, 12 cores
  CPU model: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz

Additional context
Add any other context about the problem here.

Fixed for the next version. Thanks for the reliable reproducer!


Blue button to proceed to next step of workflow also at the end of the page?

Describe the improvement
After completing the metadata in the Description tab, it might make sense to be able to click the blue button to proceed to the next step at the bottom of the page instead of the top of the page.

Is no longer applicable to current implementation, and new implementation will change the behaviour completely.


Add snippets for renderable elements

A useful batch of snippets could be Drupal's Form & Render Elements
Something in the lines of the following:
"Render Array: Textfield": {
	"prefix": "RA Textfield",
	"body": [
		"[",
		"  '#type' => 'textfield',",
		"  '#title' => ${1:''}",
		"  '#default_value' => ${2:''}",
		"  '#size' => ${3:60},",
		"  '#maxlength' => ${4:128},",
		"  '#pattern' => ${5:''},",
		"  '#required' => ${6:FALSE}",
		"]"
	],
"description": "Provides a one-line text field form element.",
"scope":"php, theme, module"
}

Based simply on the sheer number of possible keys in renderable elements I'm not sure we want to make the decision which key/value pairs to provide snippets for on each element. Open to thoughts from community members


fix(aws_lambda): fixing path parameter name in API Gateway trigger

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Required body parameter not mandatory in some scenarios

For some operations in KeyVault (such as SetSecret and CreateKey), the body parameter object defined in the spec is required, but the corresponding complex object parameter in PowerShell is optional.

@fearthecowboy this has been fixed 😁


App tries to show "page views since 54 years ago"

Describe the bug
Screenshot of error in the app and in the Javascript console:

Reproduces almost immediately:

Clone the Vitesse Nuxt repo.
Run pnpm install.
Run pnpm run generate.
As instructed, run npx serve .output/public
Then open the app in the browser

Reproduction
The steps in the description show the error out-of-the-box, no reproduction repo is needed.
System Info
System:
    OS: macOS 14.1.1
    CPU: (10) arm64 Apple M2 Pro
    Memory: 958.78 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.0/bin/npm
    pnpm: 8.10.5 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 119.0.6045.199
    Safari: 17.1

### Used Package Manager

pnpm

### Validations

- [X] Follow our [Code of Conduct](https://github.com/antfu/.github/blob/main/CODE_OF_CONDUCT.md)
- [X] Read the [Contributing Guide](https://github.com/antfu/contribute).
- [X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
- [X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
- [X] The provided reproduction is a [minimal reproducible](https://stackoverflow.com/help/minimal-reproducible-example) of the bug.

It also reproduced with Node 20:
  System:
    OS: macOS 14.1.1
    CPU: (10) arm64 Apple M2 Pro
    Memory: 656.53 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
    npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm
    pnpm: 8.10.5 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 119.0.6045.199
    Safari: 17.1

It's just a flash of uninitialised content. in components/PageView.vue add a v-if="data?.startAt" to the div to hide it when it's not defined or is "0" - 1st Jan 1970 or currently 54 years ago


Building PowerShell Core on WSL generates error

FWIW I do seem to get a valid pwsh built and I can run it.  So that is good.
Steps to reproduce

Make sure you're running WSL Xenial 16.04.
Install PowerShell Core for Ubuntu 16.04 in WSL
Start PowerShell Core in WSL
Cd to your Windows clone dir e.g. cd /mnt/c/GitHub/PowerShell
ipmo ./build.psm1
Start-PSBootstrap
Start-PSBuild

BTW this is a darn convenient way to build/test for both Windows and Linux.  No need to push your changes to origin so you can pull them to Linux to test.
Expected behavior
pwsh builds - no errors.
Actual behavior
Get an error:
PowerShell output: /mnt/c/Users/Keith/GitHub/rkeithhill/PowerShell/src/powershell-unix/bin/Linux/netcoreapp2.0/linux-x64/publish/pwsh
Copy-Item : Cannot find path 'C:/Program
Files/dotnet/sdk/NuGetFallbackFolder/microsoft.netcore.app/2.0.0/ref/netcoreapp2.0/Microsoft.CSharp.dll' because it
does not exist.
At /mnt/c/Users/Keith/GitHub/rkeithhill/PowerShell/build.psm1:528 char:9
+         Copy-Item -Path $refAssemblies -Destination $refDestFolder -F ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:/Program File...soft.CSharp.dll:String) [Copy-Item], ItemNotFoundExce
   ption
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.CopyItemCommand

Sure enough, the powershell.inc file includes a whole bunch of paths that start with C:\.
Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.0.0-beta.8
PSEdition                      Core
GitCommitId                    v6.0.0-beta.8
OS                             Linux 4.4.0-43-Microsoft #1-Microsoft Wed Dec 31 14:42:53 PST 2014
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

BTW the root of the problem is that TypeGen dir - $PSScriptRoot/src/TypeCatalogGen - is not platform specific.  It probably should be $PSScriptRoot/src/TypeCatalogGen/$Runtime.  Easiest work-around for now is Start-PSBuild -Clean.

@rkeithhill : Another issue is also that you call Start-PsBuild without the -Runtime parameter because if this paramer is not passed, then $Runtime will just stay an empty string since dotnet internally takes then care of producing the right thing for the machine that is building.


Fixing version id to be 1.1.7

Updating the build.sbt so that when you do 'sbt publish', the right version path is created.

It took me a while to figure out, but what was wrong was the README using that version number. I changed the README to use the current 0.0.2.


How to run with features MFCC

I can't runing with MFCC features
My config:
dataset_conf:
batch_size: 32
num_class: 10
num_workers: 8
min_duration: 0.5
chunk_duration: 3
do_vad: False
sample_rate: 16000
use_dB_normalization: True
target_dB: -20
train_list: 'dataset/train_list.txt'
test_list: 'dataset/test_list.txt'
label_list_path: 'dataset/label_list.txt'
preprocess_conf:
feature_method: 'MFCC'
feature_conf:
sample_rate: 16000
n_mfcc: 40
melkwargs:
n_fft: 1024
hop_length: 320
win_length: 1024
f_min: 50.0
f_max: 14000.0
n_mels: 64
optimizer_conf:
learning_rate: 0.001
weight_decay: 1e-6
model_conf:
embd_dim: 192
channels: 256
train_conf:
max_epoch: 30
log_interval: 10
use_model: 'ecapa_tdnn'
RuntimeError: Given groups=1, weight of size [256, 64, 5], expected input[32, 40, 151] to have 64 channels, but got 40 channels instead

https://github.com/yeyupiaoling/AudioClassification-Pytorch/blob/77b7708a72f6e68a653c53960cdb7980d04d99e9/macls/data_utils/featurizer.py#L58
to
return self._feature_conf.n_mfcc

https://github.com/yeyupiaoling/AudioClassification-Pytorch/blob/77b7708a72f6e68a653c53960cdb7980d04d99e9/macls/data_utils/featurizer.py#L58
to
return self._feature_conf.n_mfcc

Thank you very mach ！


Safari performance extremely slow with nested flex-direction: column; boxes

Performance suffers greatly with multiple layers of column flex-boxes.  After 10 or so layers, each progressive layer causes 2x the delay from the previous layer.
Sample: http://codepen.io/anon/pen/YyOzMK?editors=001

Can you report this to webkit.org? And if you do, please post the link back here.

One of my colleagues reported it recently:
https://bugs.webkit.org/show_bug.cgi?id=150445

Nice, thanks!


Add signal core driver

All in the title I guess ^^ .

That's interesting, mypy passed on my laptop:
python -m mypy --ignore-missing-import --no-warn-unused-ignores -p qcodes_contrib_drivers
Success: no issues found in 156 source files

That's interesting, mypy passed on my laptop:

likely because you have Windows? and the CI is using linux. mypy understands platform differences so we can fix it, let me find out how ....

Well, I tried my best (can't make local test first)

Codecov Report

Merging #114 (6438011) into master (af6ec17) will decrease coverage by 0.27%.
The diff coverage is 0.00%.

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

ok, I am out of ideas...
Does someone has a hint?

Well, great thank for all that work!