Probot double-posting comments.

Bug Report
Current Behavior
The bot for some reason posts 2 identical comments instead of 1. I'm sure I've made a mistake somewhere, but I've tried everything from reverting to plain code to trying other requests such as deleting the issue.
/**
 * This is the main entrypoint to your Probot app
 * @param {import('probot').Application} app
 */
const id_regex = /\w{8}-\w{4}-\w{4}-\w{4}-\w{12}/gm;
const token_regex = /s[.][a-zA-Z0-9]{24}/gm;

module.exports = (app) => {
  app.on("issues.opened", async (context) => {
    reply_text = "";
    const id_of_issue = context["payload"]["issue"]["node_id"];
    const number_of_issue = context["payload"]["issue"]["number"];
    const body_of_issue = context["payload"]["issue"]["body"];
    const title_of_issue = context["payload"]["issue"]["title"];
    const user_of_issue = context["payload"]["sender"]["login"];
    if (body_of_issue.search(id_regex) != -1) {
      reply_text =
        reply_text + "\n" + construct_reply("ROLE-ID/SECRET-ID", "BODY");
    }
    if (title_of_issue.search(id_regex) != -1) {
      reply_text =
        reply_text + "\n" + construct_reply("ROLE-ID/SECRET-ID", "TITLE");
    }
    if (body_of_issue.search(token_regex) != -1) {
      reply_text = reply_text + "\n" + construct_reply("TOKEN", "BODY");
    }
    if (title_of_issue.search(token_regex) != -1) {
      reply_text = reply_text + "\n" + construct_reply("TOKEN", "TITLE");
    }
    var issue_body = body_of_issue;
    var issue_title = title_of_issue;
    issue_body = issue_body.replace(id_regex, " [REDACTED] ");
    issue_body = issue_body.replace(token_regex, " [REDACTED] ");
    issue_title = issue_title.replace(id_regex, " [REDACTED] ");
    issue_title = issue_title.replace(token_regex, " [REDACTED] ");
    issueComment = context.issue({
      body:
        "@" +
        user_of_issue +
        "\n" +
        reply_text +
        "\nYour post has been deleted but please re-submit as follows, without sensitive data: \n Title: >" +
        issue_title +
        "\n Body: >" +
        issue_body +
        "\n\n Thanks,\n Vault Team",
    });
    context.github.issues.createComment(issueComment);
    if (reply_text != "") {
      context.github.issues.createComment(issueComment);
      //delete_issue(context, id_of_issue)
      //create_issue(context,title,body)
    }
  });

  function create_issue(context, title, body) {
    return 1;
  }

  function delete_issue(context, id) {
    const mutate = `mutation deleteIssue($input: DeleteIssueInput!) {
                 deleteIssue(input: $input){
                     clientMutationId
                 }
               }
           `;

    const a = context.github.graphql(mutate, {
      input: {
        issueId: id,
      },
    });
  }

  function construct_reply(what, type) {
    return (
      "I've detected that you've used a " +
      what +
      " in your post's " +
      type +
      ". Keep in mind that role-id, secret-id and tokens are CONFIDENTIAL and must be treated as such."
    );
  }
};

SCREENSHOTS: (of terminal + git-issue)
https://imgur.com/a/aj8JOLD
I'm using npm start instead of probot run. Not sure if that's what's causing it.
It acts the same way for all requests, by the way including deletes and new issues.

Try removing the context.github.issues.createComment(issueComment); in line 48
- context.github.issues.createComment(issueComment);
  if (reply_text != "") {
    context.github.issues.createComment(issueComment);

This was just there to replicate the issue. I tried both ways.
One thing that I noticed may have fixed it, is adding the
app.log("Yay, the app was loaded") line at the top... Any idea why? Coincidence or does the app actually look for that?
I'll do some more testing in a little bit to see if that were the issue and let you know either way.

@gr2m I even tried creating a new probot app on my laptop and the issue still persisted. Tried with various versions of probot and nodejs, still. Any thoughts?

Hmmm ... is it possible that you have the app running twice somehow?

That was my initial thought as well so I checked and that wasn't the case. Then I created a new probot app on my laptop and still the same problem (without importing any source code, just using the default). I'll try on a different network as well just in case it's my network, and post an update here.

Different network fixed the issue... Weird. Anyway, thanks for the help @gr2m

Wow, very weird indeed. Thanks for letting us know, glad you figured it out!

Different network fixed the issue... Weird. Anyway, thanks for the help @gr2m

what did you end up using for network?


Bug on move Path

Problem when trying to move the Path Objects by clicking on its name.

I checked this issue.
I will fix it today.

I fixed this problem.
Please test it again.

Now its works. Thank you.


HRA Organ information is not generated when a user is not logged in (SenNet)

This can be reproduced by visiting any organ page on the SenNet Data Sharing Portal (e.g. https://data.sennetconsortium.org/organs/heart) while not logged in or in incognito mode. The default token that gets set is 55488bfad6fa6946e15209040938668e which is not valid for SenNet and prevents any public block information from being gathered. The default behavior when no token is passed should be to not set it at all.

The other token is an HRA-API session token, so ignore that. But I do see you are passing 'undefined' to the sennet ds-graph endpoint:

Which causes an error. I believe that is the root of the issue. if you just replace undefined with the empty string and produce this url https://apps.humanatlas.io/api/ds-graph/sennet?token= it should work again.


CTRL_CLOSE_EVENT

Here's how to catch CTRL_CLOSE_EVENT in Windows (end task or close console window). There was some effort to translate this to SIGTERM but requires blocking which isn't compatible with the signal.Notify model.
The right way to stop a service would be sc stop or net stop and I believe these stop signals are sent during reboot/shutdown, I'll have to double check. If they are this is probably just "interesting" and adding this would help in the case of someone killing the process instead of stopping the service gracefully.

This issue may be resolved by Go 1.14:

On Windows, the CTRL_CLOSE_EVENT, CTRL_LOGOFF_EVENT, and CTRL_SHUTDOWN_EVENT events now generate a syscall.SIGTERM signal, similar to how Control-C and Control-Break generate a syscall.SIGINT signal.

https://golang.org/doc/go1.14#os/signal


Add scope to not  tag

Versions:

VueJs: 2.5.2
Vee-Validate: 2.0.0-rc.19

Description:
How can I use data-vv-scope on not  tag?
Steps To Reproduce:

You can either add it on your inputs directly or on form tags only.


Potential minor enhancements to the IOB curve calculation to increase flexibility

The current IOB curve calculation has hardcoded coefficients derived from a basic intuitive concept of the insulin activity rising from zero to a peak amount at a peak time, and then decreasing back to zero between the peak time and the DIA.  As implemented, the curve can be "stretched" or "squeezed" by changing the DIA, but the relationship between the peak time and the DIA is not configurable.  Since it's possible to calculate the area under the curve analytically, it's possible to expose the peak time to the user, and therefore have more control over the curve shape.  This could be useful as faster insulins become available.  In addition, the current curve seems to have (this may be wrong) some slightly unnatural behavior around the peak time, and the analytical implementation might help to smooth it out. The enclosed spreadsheet lays out all these points.
OpenAPS_IOB_curve_analysis.xlsx

Hi, just to add some information:
@warstar2187 also added a 4 point curve (0, 2 setup points, dia). In AndroidAPS you can already choose between them:
Original:

Prolonged (the new one):

Fiasp users report this curve to work better for them.

Thanks for that info - I was playing with something similar also in order to try to match published fiasp curves as it does seem like it's hard to match the fiasp curve with a "three point" curve.  Was waiting for feedback and more data to post it - glad to see it's already done!

I think we're going to take the https://github.com/openaps/oref0/issues/544 approach to this.  Any reason to do the three-point-curve approach as well?

No reason I can think of.


[stable/rabbitmq] Persistence not working on GKE and AKS

Is this a request for help?: Yes
Is this a BUG REPORT or FEATURE REQUEST? (choose one): Bug report
Version of Helm and Kubernetes: Helm 2.11.0, K8S 1.10.11-gke.1
Which chart: stable/rabbitmq
What happened:
I installed RabbitMQ with persistence enabled. After a node failure due to some far too heavy load being deployed, RabbitMQ automatically came back up, but all users were gone, all exchanges were gone and all queues were gone. I had to manually recreate all the users and exchanges.
What you expected to happen:
Recovery with all persistent data intact.

Anything else we need to know:
Values.yaml used;
rabbitmq:
  username: XXXXXX
  plugins: |-
      [rabbitmq_management,rabbitmq_peer_discovery_k8s].

  clustering:
    address_type: ip
    k8s_domain: cluster.local

  configuration: |-
      ## Clustering
      cluster_formation.peer_discovery_backend  = rabbit_peer_discovery_k8s
      cluster_formation.k8s.host = kubernetes.default.svc.cluster.local
      cluster_formation.node_cleanup.interval = 10
      cluster_formation.node_cleanup.only_log_warning = false
      cluster_partition_handling = autoheal
      ## queue master locator
      queue_master_locator=min-masters
      ## enable guest user
      loopback_users.guest = false

persistence:
  enabled: true
  accessMode: ReadWriteOnce
  size: 8Gi

resources:
  requests:
    memory: 256Mi
    cpu: 100m

replicas: 2

nodeSelector:
  beta.kubernetes.io/arch: amd64

ingress:
  enabled: false

metrics:
  enabled: true

Thanks for the information. We will update this ticket when we have more details or when we release a fix for the issue.

Still happening

The sadness remains insurmountable :(

Setting rabbitmq.clustering.address_type to hostname fixed the issue for me.

@KevinVecchione that is interesting. If I try that, does that mean I still lose the original data that is on there now? I assume so?
And can anyone explain WHY this works?

@gboor Yes, you'll lose your current data. Rabbit creates directories named after itself in the mnesia database. Hostnames are stable between pod restarts so it's able to find it's old data.

I've created this PR https://github.com/helm/charts/pull/12677 which should solve the persistence issues of the RabbitMQ chart.
However, as some of you have pointed out, you will lose your current data in the upgrade process.
I am wondering if someone knows a workaround that could be documented in the readme.


MaxHeight and scroll

Hello,
i'm trying to use ng-autosize, when i put maxHeight at 300px and i write text more than maximum, I don't see a scroll. overflow is set to hidden even if max is exceed.
thank's

I am using [maxHeight]="100" and don't see a scroll bar either.. any help appreciated.


Augment this page based on community discussion "Private Link using Azure Backbone (same region) - network hops?"

[Enter feedback here]
Hi - as discussed in this community Q/A
https://learn.microsoft.com/en-us/answers/questions/1121318/private-link-using-azure-backbone-same-region-netw.html
0 Votes"
0
msrini-MSFT answered • 3 hours ago
Hi,
If this source and the destination VNET is in the same region, traffic flows via Azure Backbone will not get routed via another region.
Can you raise a github issue on that page, where you can request for the doc update ?
Regards,
Karthik Srinivas

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

ID: fbfc7ccc-7fd0-9bb7-7d39-e9cfa09736ce
Version Independent ID: c0e7f101-5707-63ff-8193-c74a3e1d9a75
Content: What is Azure Private Link?
Content Source: articles/private-link/private-link-overview.md
Service: private-link
GitHub Login: @asudbring
Microsoft Alias: allensu

@sfc-gh-tbourboulas
Thanks for your feedback! We will investigate and update as appropriate.

@sfc-gh-tbourboulas Thanks for the feedback! We have created an item in our backlog and will update the article if appropriate.
#please-close


Pilot: adaptive debounce

Currently we have fixed 100ms debounce window. We do have one level of backpressure, though: we will only compute PC one at a time. So its really min(100ms, last push context init).
However, this still can lead to overload scenario. For example, if I can init PC in 100ms, and have constant event every 101ms. We will debounce and generate PC every 101ms. Those PC will not be pushed, as they will queue up. If the start of the queue is still processing old versions, we will burn PC initializations that are never pushed
For example, we may end up with queue:
[proxy1@v1, proxy2@v1, ..., proxy20@v1, proxy21@v10]
Secondly, if we do not get backed up, we will be pushing to each proxy every 100ms. This is probably a bit much. Some sort of dynamicism is probably helpful, to do some backoff. This could be adaptive in some way - based on the size of XDS, CPU load, etc.
Note to anyone interested: This is probably not a good first issue; I am confident we can do better here but I don't know how. This will involve a lot of work to come up with a mechanism and measure it at scale. To avoid wasted effort please discuss here before implementing anything.

We have a env PILOT_DEBOUNCE_AFTER can be tunned. If you meant to dynamically tune it, which should be out of scope of istio. It can be very complex and user can have their own solution outside

I mean tuning at runtime - not restarting istiod to change a static env
var. it doesn't make sense to push 10x per second but you also don't want
to have a high debounce or updates are slow. Fixed interval is not optimal
On Mon, May 29, 2023, 2:21 AM Zhonghu Xu @.***> wrote:

We have a env PILOT_DEBOUNCE_AFTER can be tunned. If you meant to
dynamically tune it, which should be out of scope of istio. It can be very
complex and user can have their own solution outside
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/45142#issuecomment-1566830008, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXNGRUTUTT3SPIFUDTDXIRTB5ANCNFSM6AAAAAAYO7WTMA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fixed predictions

now there is no need Y to run predictions

Changes Unknown when pulling 665d212ceadc0d5c9d5334f38f3644a13f50b349 on dev into ** on master**.


Tag version 0.7

Addresses #2925.
TODO:

Write a summary of all changes since version 0.6.
Build and upload binary wheels for CPU code
Build and upload binary wheels for GPU code (this may take a while)

Codecov Report

Merging #2975 into master will increase coverage by 0.41%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master   #2975      +/-   ##
===========================================
+ Coverage     43.19%   43.6%   +0.41%
  Complexity      207     207
===========================================
  Files           155     155
  Lines         11614   12188     +574
  Branches       1161     450     -711
===========================================
+ Hits           5017    5315     +298
- Misses         6267    6687     +420
+ Partials        330     186     -144

Impacted Files
Coverage Δ
Complexity Δ

src/tree/param.h
59.45% <0%> (-4.05%)
0% <0%> (ø)

include/xgboost/objective.h
36.36% <0%> (-3.64%)
0% <0%> (ø)

src/common/math.h
60.71% <0%> (-2.25%)
0% <0%> (ø)

include/xgboost/gbm.h
12.5% <0%> (-1.79%)
0% <0%> (ø)

src/gbm/gbtree.cc
19% <0%> (-1.13%)
0% <0%> (ø)

include/xgboost/tree_model.h
31.98% <0%> (-0.85%)
0% <0%> (ø)

tests/cpp/metric/test_elementwise_metric.cc
100% <0%> (ø)
0% <0%> (ø)
:arrow_down:

tests/cpp/predictor/test_cpu_predictor.cc
100% <0%> (ø)
0% <0%> (ø)
:arrow_down:

src/tree/updater_basemaker-inl.h
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

tests/cpp/common/test_compressed_iterator.cc
100% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 60 more

Continue to review full report at Codecov.

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

The release note should be in the tag v0.7

I'm going through commits to document major changes since the last release.


Importing of groups showing 0 members.

Using a distribution group shows up in autocomplete for manual import, acts like it is trying to import, but ends up with 0 members.

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

ID: fb8d5c18-3f84-f47a-7320-930cbd4ae5a0
Version Independent ID: aa593032-c7c0-5814-5b18-d360295a260e
Content: Import user groups from connected apps in Cloud App Security
Content Source: CloudAppSecurityDocs/user-groups.md
Service: cloud-app-security
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

@gimmic Thanks for reaching out to us. It can take up to 2 hours for results to show up. Please let me know if you're still having problems. thanks

I can verify the members are now showing up as expected. The UI should not reflect "import completed successfully" and display 0 members. It should reflect the import is not yet finished.


print can handle 0 samples

fixes the last issue of #129.
now
block = TensorBlock(
            values=np.zeros((0, 2)),
            samples=Labels([], np.empty((0, 2), dtype=np.int32)),
            components=[],
            properties=Labels(["properties"], np.array([[5], [3]], dtype=np.int32)),
        )

gives
TensorBlock
    samples (0): []
    components (): []
    properties (2): ['properties']
    gradients: no

:books: Documentation preview :books:: https://equistore--166.org.readthedocs.build/en/166/

I cleaned the code of the print function thanks to @PicoCentauri

I always like it when we fix bugs and the code becomes simpler. Can you take a look at the tests?

super wierd in my pc works, but the CI is doing pytest while my local files for this branch are still all in unittest. That's what it is complaining about.

i can try to rebase

@PicoCentauri, you are amazing


Furnaces are weird AF

So I made a program to manipulate items into and out of furnaces and it makes furnaces into those bugged always on furnaces when I run it, it only happens when I use my program though it doesn't happen when I use the Lua command line to do the exact same thing.
pastebin for the code: t5uGQ3AK
if you want to use it it requires a bit of manual set up ( I haven't gotten around to making an installer yet)
output: where smelted items go
itemInput: where smeltable items go into the system
coalInput: where coal is put into the system
itmBlk: how many items your item can smelt (you can just leave it alone)
you need all of the inputs and the output connected via a wired network and some furnaces connected to the same wired network along with the computer (should work with basic computers)
then just run it and put coal and cobblestone or something in and it should break the furnaces and error: the block has changed or something.
Edit: I'm using Plethora for item manipulation I realized after I posted this that I put it on the wrong thing

Plethora, not CCTweaks. I'll copy it across.


Manual update of common-files.

Please provide a description of this PR:
Supersedes https://github.com/istio/istio/pull/42804.
Needed because https://github.com/istio/istio/pull/42791 updated common-files and that gets reverted by this change. Fixed the lint errors merged in https://github.com/istio/istio/pull/42791 in this PR.

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

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
2764053b8cd9aa6b04e0595510661979f1089a05
link
true
/test integ-distroless

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


Add documents, engine, tag helpers to snapshot

The project snapshot now maintains a RazorProjectEngine as well as set
of Tag Helpers that are known for that snapshot.
Pivoted some more services to be snapshot-centric.
Also added the ability to track .cshtml documents to the project system.
For now most components just ignore document changes.

/cc @NTaylorMullen - please take a look, these are the changes we discussed yesterday that start tracking documents through the PSM.
Additionally we no longer use a 'work queue' - tag helpers now hang off of the snapshot and can be computed on demand. This responsibility is moved to the document tracker.


storage: enforce sorted traversal during Walk

This ensures sorted order of elements encountered during walk. This open up a number of optimizations for the catalog API, in addition to other enumeration methods.
The error propagation for Walk has also been fixed.
Attempts to address issues in #1190.

LGTM.

LGTM


Add an iter_mut method to Fields

Iterating over the fields is very useful, but I currently have a need to mutate the fields. I could implement this behavior locally in my project, but I think others might find this useful.

Thanks, done. Sorry I missed that one.


System.InvalidOperationException - won't run.

System.InvalidOperationException
  HResult=0x80131509
  Message=Cannot load Counter Name data because an invalid index '' was read from the registry.
  Source=System.Diagnostics.PerformanceCounter
  StackTrace:
   at System.Diagnostics.PerformanceCounterLib.GetStringTable(Boolean isHelp)
   at System.Diagnostics.PerformanceCounterLib.get_NameTable()
   at System.Diagnostics.PerformanceCounterLib.get_CategoryTable()
   at System.Diagnostics.PerformanceCounterLib.CounterExists(String category, String counter, Boolean& categoryExists)
   at System.Diagnostics.PerformanceCounterLib.CounterExists(String machine, String category, String counter)
   at System.Diagnostics.PerformanceCounter.InitializeImpl()
   at System.Diagnostics.PerformanceCounter..ctor(String categoryName, String counterName, String instanceName, Boolean readOnly)
   at System.Diagnostics.PerformanceCounter..ctor(String categoryName, String counterName, String instanceName)
   at RunCat.RunCatApplicationContext..ctor() in D:\Coding\C#\RunCat_for_windows-1.8\RunCat\Program.cs:line 60
   at RunCat.Program.Main() in D:\Coding\C#\RunCat_for_windows-1.8\RunCat\Program.cs:line 34

is this Cpu dependant ?

Not an issue with the code ... apparently win 10 can get out of whack with it's performance counters:

following the steps in this link fixed it up


Exception while debuggin in utils.js

Hello,
I am using BlueBird 3.5.0 and node version 6.10.2. I am facing the following exception in bluebird\js\release\util.js when I am attached with my debugger:
(Exception has occured: Object)
http://i.imgur.com/lk3NJWm.png
I am writing a REST client and upon request timeout I throw an exception to a parent promise based function. I am not entirely sure if this exception is triggered because I am doing something wrong in my code or because it's a bug in bluebird.
It's a rather complex code I have to paste to show you what I am doing. I tried to simplify it as much as possible:
agenda.define('upsertLeaderboards', (job, done) => {
    const promiseArray = []
    promiseArray.push(apiHelper.getData1(true))
    promiseArray.push(apiHelper.getData2(true))

    Promise.all(promiseArray).then(() => {
      done()
    }).catch(() => {
      logger.error('Error during upsert task')
    })
  })

function getData2(forceRefresh = false) {
  const promiseArray = []
  return getWikiData()
    .then((data) => {
      for (let i = 0; i < data.length; i++) {
        promiseArray.push(getDetails(data[i].name, forceRefresh))
      }

      return Promise.all(promiseArray)
    })
}

I'm having the exact same issue. It occurred out of nowhere.  As in, I had been debugging 20 times in a row and then the 21st time it has this error and doesn't go away. Not sure what to do here.

@lumberbeard Are you using VSCode + Typescript by any chance?


Sentence-Similiarity

To pick most redundant, but also distinct sentences, somehow I need to compare every sentence to every already chosen sentence.
Proposed Solution
There should be atleast one function which gets the distance of two paths/sentences.
Then there should be a function which somewhat weights the metric-score with the distance to already chosen sentences.
Possible Problems:
Maybe it's hard to make a nice, functional solution for it.
Related Issues:
This is a subtask for #11
Additional Context:
There are many ways to compare sentence-similiarity. One Example Article

Jaccard Similarity would be an easy one, but seems to be rather weak.
It is:
Size(Intersection of words) / Size(Union of Words)

Cosine Similiarity
https://medium.com/@sumn2u/cosine-similarity-between-two-sentences-8f6630b0ebb7
would be much better.


No package should be builtin

Currently the four packages base, ghc-prim, integer-simple and array are considered to be builtin. We do not analyze their source. We are only interested in the symbols they export. It would be better if we did not have any builtin packages.

array is no longer builtin


"Running cells with {0} requires ipykernel" message has wrong interpreter displayName

Testing microsoft/vscode#127378
This doesn't happen on a native notebook, but if I switch to this kernel:

I get an error saying it needs ipykernel. And the error is pointing to the wrong kernel. This doesn't happen with a notebook.

Looks like the displayName might be incomplete for the busted interpreter https://github.com/microsoft/vscode-jupyter/blob/28739ba1128061328d1582d898a97c08abc9704e/src/client/datascience/jupyter/interpreter/jupyterInterpreterSubCommandExecutionService.ts#L94 https://github.com/microsoft/vscode-jupyter/blob/afddee11c14a4bfdb0bb0c338a1e40efcdd4dd5b/src/client/datascience/jupyter/interpreter/jupyterInterpreterDependencyService.ts#L139

Should be fixed now, closing for verification


Inject connection to factory

Is it possible to inject the AmqpConnection in a factory of a custom provider?
I tried this but without success:
import { Connection as AmqpConnection } from 'amqplib'
import { ConfigService } from 'nestjs-config'

export const queueServiceProvider = {
  provide: 'QueueService',
  useFactory: (amqp: AmqpConnection, configService: ConfigService) => {
    return new QueueService(amqp, configService.get('queue'))
  },
  inject: [AmqpConnection, ConfigService]
}

Yes it's possible. The problem with the nestjs-amqp package is when having multiple connections you need unique tokens so I made a constant which has a int on the end like AMQP_CONNECTION_PROVIDER_0 or if one connection is present (or using the forRootAsync) I used 'default'. If you see here I made a function that does all this so you could inject the provider using the token instead like so
import {ConfigService} from 'nestjs-config';
import {createConnectionToken} from 'nestjs-amqp/utils';

export const queueServiceProvider = {
    provider: 'QueueService',
    useFactory: (amqp, configService: ConfigService) => new QueueService(amqp, configService.get('queue),
    inject: [createConnectionToken('default'), ConfigService],
}

If you would like to make a PR to add this to the read me feel free! :)


Feature request: Resource effective mode support in the Progressive Rollout

By resource effective mode, it means the process of the progressive rollout needs to take resource utilization as top 1 priority, even if it can sacrifice the availability. During the rollout in this mode, never increase the utilization of the resources(unless there is something we cannot control any more, like traffic-driven workloads).

We defined the two rollout modes for the new revision: 1. availability mode. This mode takes service availability as the top priority. It will scale up the pods for the new revision first and then scale down the pods for the old revision. 2. resourceutil mode. This mode takes the usage of the resource as priority. It makes sure that no more resource is consumed than what has been used now. It will scale down the pods for the old revision first and then scale up the pods for the new revision.


Migration to tailwind CSS

This PR focuses on converting the existing CSS styles of the Home and Movie Details pages to utilize Tailwind CSS.
In addition, dark mode support is implemented for both pages, enhancing the user experience and accessibility.
Files Modified:
Home.js
MovieDetails.js
Navbar.js
tailwind.config.js

@AayushiJapsare15 You removed a link Verify from navbar which was previously implemented by other contributor.


[Feature Request] Snippet - Add an easier way to inject queue items as part of the snippet config

As per the comment in #1332 -
I have been considering providing an easier way to inject queue items as part of the snippet config, but for now the above would be the approach.
Rather than requiring everyone to directly reference the aiSdk.queue array to add callback methods, provide a callback that is executed once the Sdk is successfully loaded and initialized.
Usage of the aiSdk.queue array is problematic as during initialization the value array is actually deleted (become undefined) and if the sdk has already been loaded and initialized by something else injected on to a hosting page then the reference appInsights instance may not actually be the active sdk instance.

CDN Deployment rolled back due to #1452

CDN Deployment rolled back due to #1452


⚠ Fortnite has degraded performance

In 5f75914, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1514 ms

Resolved: Fortnite performance has improved in 3c799b2.


reset InterruptHandle

I need to interrupt the execution of user-provided wasm code if it cannot complete in a limited time, say 1 second.
The documentation of Interrupt says:

Interrupt interrupts currently executing WebAssembly code, if it's currently running, or interrupts wasm the next time it starts running.

So my question is: how to ensure it only interrupts the current execution but not the next?
For example:
go func() {
    time.Sleep(time.Second)
    interruptHandle.Interrupt()
} ()

// execute wasm code

As two goroutines are involved, there are race conditions, and it is possible for Interrupt to be called after the wasm code finishes current execution, which will result in the next execution be interrupted at the start.
There may be some tricky workarounds for this requirement, like execute a dummy wasm function to reset the Interrupthandle before execute the desired code, but all of them look ugly.
Is there a better way to reset the InterruptHandle?

I've transferred this issue to the wasmtime repository itself since the feature will need to be implemented here first.

This API has since been removed from Wasmtime so I'm going to close this


Bicep param decompile vscode action

VS Code action to decompile .json into.bicpparam

Microsoft Reviewers: Open in CodeFlow

I couldn't find any examples of e2e tests on vscode side that test a command with some input menus. Should I try to write one for decompileParams command?

I couldn't find any examples of e2e tests on vscode side that test a command with some input menus. Should I try to write one for decompileParams command?

There's a decompile.test.ts test under src/vscode-bicep/test/e2e that you could use as an example. It's not exercising menu navigation but just dispatching a vscode command by name. It should be a pretty small addition.

I couldn't find any examples of e2e tests on vscode side that test a command with some input menus. Should I try to write one for decompileParams command?

There's a decompile.test.ts test under src/vscode-bicep/test/e2e that you could use as an example. It's not exercising menu navigation but just dispatching a vscode command by name. It should be a pretty small addition.

Still having a hard time figuring out how to interact with navigation menu in the test (that prompts for a bicep file). I can't really seem find any test for commands that deal with any menus.
The test passes but I have to pick the bicep file or select none during the test 😅


[US] (i18n) - Database Tables to Suport i18n

As a platform developer, I want tables and a view to support i18n, so that internationalization is supported on the applications.

MoSCoW
Risk
Story Points

Must
Low
3

Acceptance Criteria

Successful creation of the meta tables and view (languages, keys, values, and translations)

Risks

No apparent Risk

Pre-requisites

[x] Meet with the stakeholder to clarify the requirements
[x] Complete the requirements of this US

Tasks

[x] Research the proper way to utilize i18n
[x] Adding 3 tables within the meta schema languages, keys, and values
[x] Adding a view for translation having id, language(ref), key(ref), and value

Preview

Demo
to be uploaded soon

Looks good! Signing off!


Remove patchesStrategicMerge that is deprecated

We use patchesStrategicMerge in our kustomization yamls under the config dir,
however, since version 5.0.0 this was deprecated and we need to migrate it.
See the following warning message:
# Warning: 'patchesStrategicMerge' is deprecated. Please use 'patches' instead. Run 'kustomize edit fix' to update your Kustomization automatically.

To be accurate, we don't need to migrate until it's removed and we use a kubectl version that doesn't support it anymore.
@liornoy wanna take this?

yes, with pleasure.
and you're right. I confused deprecated with removed.

see my comment here: https://github.com/metallb/frr-k8s/pull/2#discussion_r1233331925


The release version does not contain some fixes

Describe the bug
The version you are currently downloading when installing from the npm does not contain some fixes. In particular, I was looking for this (found in the release logs of v2.4.0:
local link type added #247 (gopherine)

Others, too, may be missing.
To Reproduce
Steps to reproduce the behavior:

Install the package via npm
Try using the graph and set custom types for the input links.

Expected behavior
The links with the custom type should look accordingly
Screenshots

These two-way links should have their custom type set to CURVED_SMOOTH, but they still show up as STRAIGHT
Environment:
react-d4-version: 2.4.0
Additional context
I believe that the npm run dist script was not launched. I have cloned the repo locally, ran the script, and the lib/components/graph/graph.builder.js has a different output than what comes when installing via npm. In particular, there is the buildLinkProps function which is missing this line:
const type = link.type || config.link.type;

(and then the following usages of type)

Hi @NoMercy235, maybe you're running against a cached version in your node_modules. I believe the current version was published to npm. Also if you take a look into the live sandbox you'll be able to use custom link.type.

Hey Daniel! I have just recently started using react-d3-graph, so I'm pretty sure there are no cache issues involved and that the installed version is indeed 2.4.0. I have also noticed that the setting does work in the sandbox, which was why I had to dig deeper into node_modules and see what was happening. This is how I noticed that the src/components/graph/graph.builder.js file looked different.

As stated in the #312 ticket, the update to v2.4.1 brought all the changes. Thanks!


Playermarker plugin

PlayerMarker is a plugin that gives you the option to mark players. You also can specify a Sharing Channel in the configuration where your marks will be posted to. If another player uses the same Sharing Channel each others markers will be shared.
It uses a Redis Database to communicate between clients.
Each packet contains the raw player name and a list of marked players formed into a json using Gson.

What is this used for? It looks like it would be used for pvp scouting.

What is this used for? It looks like it would be used for pvp scouting.

It should give groups of players a better visualization of crowded regions and better localization in them.
Also every marked player is being unmarked after disabling the plugin or changing a channel.

This would fall under multiple guidelines.

This would fall under multiple guidelines.

Which guideline violations are we talking about?

Scouting, marking for attacking, marking opposing clan?

Scouting, marking for attacking, marking opposing clan?

Its like your friend telling you that you should look out for player "xyz".
So you have to manually check every player you encouter if they are "xyz".
This plugin just uses the information you are being provided with, and also can access, in a better way and wont give you any further informations. I dont see where this would be an unfair advantage against other players without the plugin or how this should violate anything at all? Maybe you could give me an more clear example on how this would be an clearly unfair advantage.

Its like your friend telling you that you should look out for player "xyz". So you have to manually check every player you encouter if they are "xyz".

Player Indicators would be perfect for this

In earnest, there is a difference between verbally calling out an opposing player and offering a coordinated system that marks a specific player's location on screen. The former, you have to verbally communicate where they are on screen accurately enough for everyone to comprehend what you said, either with looks or location. The latter, said player is visually marked on screen and you are brainlessly told where to click--it takes all of the processing out of it.
As such, I still don't see a use for this outside of PVP that is within 3rd party client guidelines.

In earnest, there is a difference between verbally calling out an opposing player and offering a coordinated system that marks a specific player's location on screen. The former, you have to verbally communicate where they are on screen accurately enough for everyone to comprehend what you said, either with looks or location. The latter, said player is visually marked on screen and you are brainlessly told where to click--it takes all of the processing out of it.
As such, I still don't see a use for this outside of PVP that is within 3rd party client guidelines.

In my opinion its not a unfair advantage (looking at other plugins) but since its purpose is not usable enough i can understand that. Its violating the guidelines as you say so i cant do anything about that i will close this pull request then.


Make "max buffer size" configurable

The 16MB limit is hardcoded at the moment.

Closed by 3d78946e80fc1ee1280fb06a33e0c4e56e28fa3d


chore: add explicit toolkit dep in this package

Signed-off-by: blam<EMAIL_ADDRESS>Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

https://github.com/graphql/graphiql/issues/2405 for reference!


pkg/fuzzer: don't triage saturated calls

Currently we throw away saturated calls only after triage/minimization.
Triage/minimization is unnecessary for saturated calls,
we already know we will throw them away later.
Don't send saturated calls for triage/minimization.

@a-nogikh PTAL


[Release] sdk/resourcemanager/oracledatabase/armoracledatabase/0.1.0

https://github.com/Azure/sdk-release-request/issues/5192

/azp run prepare-pipelines

/azp run go - armoracledatabase - ci

@ArcturusZhang @lirenhe @tadelesh Hi there, can you clarify what exactly it is that I need to do to get this PR fixed and merged?

@ArcturusZhang @lirenhe @tadelesh Hi there, can you clarify what exactly it is that I need to do to get this PR fixed and merged? Seems test pipeline is failing, are there instructions on how to run this pipeline locally?

Those two pipeline failures were expected.

Please hold the release until the TypeSpec was merged.


Error while trying to execute ./runMaster.sh

I get the following terminal output after executing ./runMaster.sh
bench_dir='', bf_sz='1024', max_docs='128', malicious='true'
src/master/master.go:20:5: package common is not in GOROOT (/opt/homebrew/Cellar/go/1.20.5/libexec/src/common)
Can anyone please help me resolve this/ tell me what am I doing wrong?

This project was developed using an older version of Go where you had to manually set the GOPATH environment variable. If you're using a newer version of Go, you might need a go.mod file.

Hi. Any chance you can share the version of fo with which this was created?


Updated all dependencies

Generated by running:
$ yarn run clean
$ yarn run update-deps
$ yarn run ci

Also fixed the yarn run update-deps to ignore the @conductor/shared package

Summary by CodeRabbit

New Features

Updated various dependencies across multiple packages to enhance compatibility and performance.

Bug Fixes

Incremented versions of critical packages such as firebase and firebase-admin to address potential issues.

Chores

Routine maintenance updates to development dependencies, ensuring the latest features and fixes are integrated.

@coderabbitai pause


A simple workflow to pull the repo

This is a test of logging of the id-token permission on GitHub Runners

With default settings, a checkout succeeds and permissions are listed as:
GITHUB_TOKEN Permissions
  Contents: read
  Metadata: read
  Packages: read

With write permissions in workflows, a checkout succeeds and permissions are reported as
GITHUB_TOKEN Permissions
  Actions: write
  Attestations: write
  Checks: write
  Contents: write
  Deployments: write
  Discussions: write
  Issues: write
  Metadata: read
  Packages: write
  Pages: write
  PullRequests: write
  RepositoryProjects: write
  SecurityEvents: write
  Statuses: write

With contents and id-token permissions set, a checkout succeeds and permissions are reported as
GITHUB_TOKEN Permissions
  Contents: read
  Metadata: read

Addendum: neither the Attestations or Discussions token permissions logged in the workflow runs appear to be documented


Feature/rename tests to enable pytest

This PR enables pytest to run without issues as described in this comment.

Coverage remained the same at 98.071% when pulling a3cae039acac3e2ac10c852c87a826d51262a059 on hakkeroid:feature/rename-tests-to-enable-pytest into 9770e069ffce2f9ff449dcaf2db48a743fc3310f on staffanm:master.

Coverage remained the same at 98.071% when pulling 1ca75829c913db0be92651512f129a7e180f2ec2 on hakkeroid:feature/rename-tests-to-enable-pytest into 9770e069ffce2f9ff449dcaf2db48a743fc3310f on staffanm:master.

Looks good, now merged!

Great, thank you.


Use "--parallel" in Docker build

Proposed Changes

Speed up building multiple Docker images by using --parallel flag.

Additional comment
It was introduced a while ago in version 1.24.0.

Ok... So it's not a good idea. It actually slow our CI tests down quite a bit:

E2E Tests - 33% slower (16.5 minutes vs. 12.5 minutes),
Backend linters - 100% slower (9 minutes vs. 4.5 minutes).

Reason: lack of caching that is very helpful in backend images, which share common layers.


Bump to 2.1.0rc1

Bump to 2.1.0rc1

Note: tbump automatically incremented the jinja2 version since it was also at 2.0.0. The second commit fixes this.


Investigate switching from distutils to setuptools

Setuptools has many nice features like "setup.py develop" and will allow you to upload a wheel to pypi along with the source tarball.
Care may need to be taken to handle pep-440 versions, since the current version numbers will be normalized by setuptools.

Using setuptools might require additional restrictions on the version numbers. I think that's why I was using distutils in the first place. So I want to become very lucid about our version number requirements before jumping into this. So, what are our version numbering requirements?

Our main distribution portal is PyPi. It appears that PyPi is likely to permit versions that adhere to pep-440. But I'd like to run some tests to be certain.
I like to use the bdist_msi installer on Windows, because it integrates well with the Windows package/program install/uninstall system. The MSI installers require, I believe, versions with up to three dot-separated series of digits, e.g. "1.4.109". In particular, they don't allow a "pre" suffix, like I have usually used between releases. This behavior sometimes prevents me from accidentally completing a release without updating the version number. I suspect that using a suffix like "rc0" would have the same effect. Again, I should probably test this conjecture.
I have been trying to incorporate the full three-component mainline OpenVR version number into the pyopenvr version number; while appending our own version number into two more digits tacked onto the third component: https://github.com/cmbruns/pyopenvr/wiki/Release-Checklist . From ngoldbaum's pull request at https://github.com/cmbruns/pyopenvr/pull/18, it sounds like leading zeros (like in "1.0.0202") might not remain after setuptools has its way. Question: is there any chance that something might think that "1.0.0202" is a newer version number than "1.0.203"? If not, it's probably OK to migrate to a version format like the latter.

Plus, of course, we would need to test our whole release process using the setuptools version of setup.py before committing to the new way. At least up to the testpypi stage.

For now if you want the same experience as setup.py develop, just do pip install -e . - pip is actually importing setuptools (which monkeypatches distribute) so this has the same effect (including the version munging.
In response to your question - PEP 440 version strings can be compared with packaging.version.parse (see e.g. http://stackoverflow.com/questions/11887762/compare-version-strings)- packaging is available on pypi (pip vendors it and uses it internally).

I installed setuptools long ago, so presumably I'm using the monkeypatched distutils in my actual release process. I'm unsure whether it would work as I expect in a vanilla unpatched python install.

Setuptools monkeypatches distutils at import time, not at install time. Just installing setuptools doesn't change anything about your python installation.

Fixed in issue #24


Deploy 2 load balancers from the logstash chart

Chart version:
7.7.0
Kubernetes version:
1.8
Kubernetes provider: E.g. GKE (Google Kubernetes Engine)
EKS
Helm Version:
2.17
I'm using a nlb load balancer for TCP connections, but I would also need another load balancer for the UDP because it is not currently possible to mix TCP and UDP. Is it possible to deploy 2 load balancer from the logstash chart ? I was unsucessful adding another one after the first one, only the last one is deployed.

Hi @carl-chenet,
Thanks for opening this issue.
Unfortunately we currently don't have time to work on it so please be patient.

still valid

still valid

Hi @carl-chenet,
Looking at https://github.com/kubernetes-sigs/aws-load-balancer-controller/issues/1608 and https://kubernetes.io/docs/concepts/services-networking/service/#load-balancers-with-mixed-protocol-types, it seems you can now use TCP and UDP in the same LoadBalancer type Service instance starting K8S 1.20.
Where you able to test it or do you still have a need for a 2nd Service for UDP?

@jmlrt @carl-chenet I managed to test it, I was able to create a service (on k8s 1.20) using TCP and UDP protocol on the same service with the following manifest:
apiVersion: v1
kind: Service
metadata:
  labels:
    app: my-app
  name: my-app-np-service
spec:
  type: ClusterIP
  ports:
  - name: tcp-port
    port: 443
    targetPort: 443
    protocol: TCP
  - name: udp-port
    port: 443
    targetPort: 443
    protocol: UDP
  selector:
    app: my-app


Error when sending funds without the decimal point

What I wanted
I wanted to send some SBD to an account via Busy.
(I'm making the issue here because the URL was steemjs.com when the problem happened.)
What happened
I've got an error message.

How to reproduce
Send integer SBD to an account.
(As a non-native speaker I don't know if it's the correct term for a number without decimal point. What is important here that try to send e.g. 1 SBD, not 1.000 SBD. You wouldn't have problem the latter, only with the former one.)

This issue has been solved.


RxBleRadio operation off main thread

Summary
I would like to have a way to schedule bluetooth operation off the main thread.
Ultimately, it would be nice if the library was doing this on it's own so that everybody can benefit from it without further interventions. But I understand this might mean keeping a list of devices that needs to execute on the main thread.
We have such fix in the wild right now (not in mass production but tested on lots of various devices), and never heard about problems with it. In our code, if phone is samsung and at API level 18, we do stuff on main thread, on all other cases, it's being done on a dedicated single thread scheduler

@dariuszseweryn Copying your answer from a different thread (see here):

I am a bit afraid that there may be more devices that are more stable while using the main thread than we know of. In this situation we would need to keep a list of devices that need a specific solution - which could be a burden.

I agree with that statement and I don't want to put to much burden on you guys about maintaining this list, mainly also because it means that we get aware of such problems when people report them.
From our current tests on multiple different devices (nexus 4, nexus 5, LG g5, motorola, a Chinese phone, Samsung s3 mini, HTC tablet, another tablet I don't recall the model), we never had any issue running off the main thread yet. Note that we never tested on API 18 however, our lowest target was a Samsung S3 mini running API level 19.
An in-between could be to keep main thread only for API level 18 (or even more restricted).
But I understand it might difficult for you guys to maintain. As a first thing, you would guys allow a way for us, users of the library, to change the default scheduler?
Not sure how it would look like, but if you agree, I would be happy to check some ways about doing this.

Hello,
I am thinking of advantages that may come from this change. I do not think that it is good to give this flexibility to the users. The purpose (and responsibility) of this library is to allow BLE interactions in a way that is most stable - with this change we could expect people trying to use a non main thread for all devices (because of the better performance) even with Samsung S3 API 18 and related bugs.
I would imagine that it may be a good idea to restrict BLE operations to be run on the main thread only to Samsung and API 18. I wonder if on >API 18 devices this change would impact with a lower number of successful connections - this scenario would be hard to spot without A/B testing on a large number of devices.
Could you give information what is the performance gain of changing the operation execution thread? Maybe with a custom operation support ( #137 ) this change is not that relevant?

@dariuszseweryn When I first implemented the flashing, I did it in a debug screen that is doing little to no UI at all. Then, when I ported the code to the actual real UI screen, the performance was dramatically being impacted by I would say a 1.5x to 2x factor (going from 70s to around 130s depending on the run and device).
It was to such extend that we suggested to beta testers to actually close the phone screen when flashing to improve speed (no UI was giving the BLE operations almost full control of main thread).
But, indeed, a custom operation could resolve both of my issue. First, I would be able to schedule the operation off the main thread (doing it at my own risk) and also access lower level components for speed improvements.
I will try to come up with some numbers once I integrated the changes we did last week into the project. At this point, I will test the impact of running the operation on & off the main thread (and only this change) and come up with some numbers for you.
Like I say, the impact it not really important when we do minor operations (read & write a few characteristics). It was more painful when transferring lots of data and UI displaying some intensive graphics (progress, chart, etc.)

This may become relevant to an app I'm developing to support a BLE peripheral data acquisition device. It sends blocks of data (up to 4KB) as fast as possible. The data needs to be graphed on the Android screen as it arrives, so there may be little or no benefit to receiving it on a different thread. I had been thinking to receive on an io thread, perhaps analyze and format the data on a compute thread, and hand off to the UI thread just to draw the graph. Any advice or comments appreciated—and apologies in advance if I am misunderstanding the issue.

@RobLewis You say that the data needs to be displayed on the screen as it arrives - so I assume that the central (phone) is receiving the data via notification mechanism - in this situation there is no real benefit of this issue as it only touches operations that are performed from the central side (explicit characteristic read, characteristic write, setup notifications, etc.). Getting the notifications is a totally different story as it is executed on a different thread all along.
@maoueh Have you been able to check if this issue is still valid for you by any chance?

@dariuszseweryn On my side, since the addition of custom operations, I can do it on my own if needed. So for me, it's not relevant anymore.

@maoueh Perhaps you have checked the difference between Write Long Op vs Custom Op? This info could be informative for people interested in #41
Anyway - closing this issue for now. If anyone would need this - feel free to ping me here.


web hosting

hi, I can't run the code on my own server, may you explain how i can upload and set up it on ftp server please? thanks

Just to let you know, I am no longer actively maintaining this project, so some parts of it (especially the PHP version used and probably all dependencies) are outdated and may not work on modern PHP hosters. I used to deploy this project with a simple script - maybe that's helpful for you.
Apart from that, I can't offer more support. Feel free to fork the project and adapt it to your needs if that's an option.
Thanks for your understanding, and good luck with your setup!

thanks!


Unable to validate transaction

Unable to send BCC. Message: "Unabled to validate transaction when try to send Bitcoin Cash". It is impossible to send BCC in Ledger. I tried to use a lot of computer, remove and install chrome app, but not solved.

Verify that you're using the Bitcoin Cash app on the device.
On Aug 3, 2017 9:05 PM, "andrehortabh"<EMAIL_ADDRESS>wrote:

Unable to send BCC. Message: "Unabled to validate transaction when try to
send Bitcoin Cash". It is impossible to send BCC in Ledger. I tried to use
a lot of computer, remove and install chrome app, but not solved.
[image: image]
https://user-images.githubusercontent.com/11886676/28938778-6ffcfdb0-7865-11e7-9686-48419e315357.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/LedgerHQ/ledger-wallet-chrome/issues/41, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ADAF0DS-wnTpypzq_zquBiKpaxny08ptks5sUhnvgaJpZM4Os3IW
.

Yes, I choosed Bitcoin Cash - Main

I'm talking about the application on the device. You need to load the
"Bitcoin Cash" app from Ledger Manager
On Aug 3, 2017 9:15 PM, "andrehortabh"<EMAIL_ADDRESS>wrote:

Yes, I choosed Bitcoin Cash - Main
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LedgerHQ/ledger-wallet-chrome/issues/41#issuecomment-320063768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADAF0K1ekEQRT1ZlB6VjLA_fCwmc8QWzks5sUhw_gaJpZM4Os3IW
.

Understood. At Ledger Manager there is no Bitcoin Cash application. Have only: Bitcoin, Dash, Dogecoin, Ethereum, Fido U2F, Litecoin, Stratis e ZCash.

Update to firmware 1.3.1 first. Also read our Medium post
On Aug 3, 2017 9:25 PM, "andrehortabh"<EMAIL_ADDRESS>wrote:

Understood. At Ledger Manager there is no Bitcoin Cash application. Have
only: Bitcoin, Dash, Dogecoin, Ethereum, Fido U2F, Litecoin, Stratis e
ZCash.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LedgerHQ/ledger-wallet-chrome/issues/41#issuecomment-320066226,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADAF0H2oDUVu62zTFays6eQika8LyOQUks5sUh6ggaJpZM4Os3IW
.

I will lost my Bicoin Cash updating firmware?

No. Read our Medium post before proceeding
On Aug 3, 2017 9:34 PM, "andrehortabh"<EMAIL_ADDRESS>wrote:
I will lost my Bicoin Cash updating firmware?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LedgerHQ/ledger-wallet-chrome/issues/41#issuecomment-320068283,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADAF0COwJY4B17Cw9WbG87XXq8_w1zthks5sUiC6gaJpZM4Os3IW
.

Thank you! Sorry about that (put a message: you need to update your firmware to use bitcoin cash)


CIP68-Draft review

    The current CIP68 is in DRAFT

https://cips.cardano.org/cips/cip68/
There are several issues with the draft:

1 token per UTxO - extremely wasteful for large series
NFTs sent to an unspendable address, will cause the minAda to be lost forever
the format still assumes generic metadata, which won't help smartcontracts
the standard does not specify if the metadata should be inlined or not
since the reference carries the metadata and the tokens can be minted separately
the actual expected format is decided afterwards, when looking up. Which is annoying
for an indexer

Proposed additions:

extend metadata to include multiple tokens as a list or assetclass as key
require inlined metadata
(ensures that it needs to be provided, but adds extra min ada fee)
NFT standard should not require locked ADA (metadata can be provided in redeemer/last spent)
Potentially make a fix set of fields (required and optional) with potential generic addition,
so that the metadata can be easily used from smart contracts

Originally posted by @szist in https://github.com/zoeldevapps/cardano-assets-server/issues/9#issuecomment-1295746813

Discussion moved here: https://github.com/cardano-foundation/CIPs/pull/377


Client Allocation Request for: Modern Electric Solutions

Client Allocation Questions
Core Information

Name: Modern Electric Solutions
Website / Social Media: https://modern-electric-solutions.business.site/
https://nextdoor.com/pages/modern-electric-solutions-la-palma-ca/
https://www.instagram.com/modern_electric_solutions/?hl=en
Region: North America
DataCap Requested: 1 TB
Addresses to be Notarized: f01947458
Notary Requested: f1mdk7s2vntzm6hu35yuo6vjubtrpfnb2awhgvrri

Sorry wrong info


Remove use of "so called" in the docs

While working through the quickstart sections, I've seen the phrase "so called" several times.
For example:

We use the so called authorization code flow with PKCE to connect to the OpenID Connect provider. See here for more information on protocol flows.

A common definition is "falsely or improperly so named" (example "deceived by a so-called friend").  So it's a little jarring to read.
The extra words aren't needed, so just removing entirely is sufficient.
There are 6 instances that need replaced:
https://github.com/IdentityServer/IdentityServer4/search?q="so+called"&unscoped_q="so+called"

Thanks. Did you see the "edit" link on the docs pages?

@ngbrown Are you sending a pull request?


Markers moving when zooming (Android 28)

The markers move from their location as the map is zoomed in or out. This happens with both a custom and the default marker.
This is the same issue as described here: https://stackoverflow.com/questions/54811451/osmdroid-default-marker-moving-when-zooming-out-on-android-api-28#=
The suggested solution of using custom markers doesn't work but turning off hardware acceleration (adding android:hardwareAccelerated="false" in the manifest) seems to fix it. Is there another fix for this that doesn't require disabling acceleration?

Assuming the issue is related to the marker...
On the stackoverflow post the (default) marker looks tiny.
Maybe this issue is a side-effect of a change made recently by Google API and that caused other osmdroid issues: a .png file in the drawable folder is no longer automatically displayed bigger in xxx configuration, but keeps its (small) size.
Which version of osmdroid are you using? Can you test with a drawable in the right size and the right folder?
In the stackoverflow post there's a Polygon displayed: do you also display a Polygon? If so, are you sure that the Polygon is correct and the marker is not, or could that be the other way around?

Thank you for following up. I'm using the latest release of osmdroid. Using the default untouched marker I get the same behavior as with my custom one. Drawables are in the right size and folder. I'm not drawing polygons, just markers. The marker indeed is drawn tiny but it is also not sticking to its position in the map but gliding away as I zoom in and out. The issues get completely resolved (both size and location) by disabling GPU acceleration, which I'm guessing points to a different cause. My problem is that for my app i require GPU acceleration, otherwise other functionalities of the app suffer of very low frame rate.

Perhaps it's a side effect of using the deprecated Resources.getDrawable(int) method. Can you try using your own drawable and a not deprecated version of getDrawable?

Using the non-deprecated ContextCompat.getDrawable(context, R.drawable.marker) didn't fix it. Again only disabling hardware acceleration fixes it but at least I found it can be done at the view level with setLayerType(View.LAYER_TYPE_SOFTWARE, null) so only the map view is suffering from low frame rate and not the whole app. On the other hand, the issue is still a problem since it makes map browsing not smooth.

@hectorC Could you try to extend the Marker class, and to override its drawAt method that way:
@Override
protected void drawAt(final Canvas pCanvas, final int pX, final int pY, final float pOrientation) {
	final int markerWidth = mIcon.getIntrinsicWidth();
	final int markerHeight = mIcon.getIntrinsicHeight();
	Log.e("drawAt", "w,h=" + markerWidth + "," + markerHeight); // added
	final int offsetX = pX - Math.round(markerWidth * mAnchorU);
	final int offsetY = pY - Math.round(markerHeight * mAnchorV);
	mRect.set(offsetX, offsetY, offsetX + markerWidth, offsetY + markerHeight);
	RectL.getBounds(mRect, pX, pY, pOrientation, mOrientedMarkerRect);
	mDisplayed = Rect.intersects(mOrientedMarkerRect, pCanvas.getClipBounds());
	if (!mDisplayed) {
		return;
	}
	if (mAlpha == 0) {
		return;
	}
	if (pOrientation != 0) {
		pCanvas.save();
		pCanvas.rotate(pOrientation, pX, pY);
	}
	mIcon.setAlpha((int)(mAlpha*255));
	mIcon.setBounds(mRect);
	mIcon.draw(pCanvas);
	if (pOrientation != 0) {
		pCanvas.restore();
	}
}

I'd like you to test with this new MyMarker class with hardware acceleration on/off: I'd be curious to see if you have the same icon width and height (cf. Log.e), and if the display is different.

I also think it may be related to how we calculate the anchor point of an image. The anchor point is the actual lat/lon that the mark represents. We then calculate an image offset so that the center of the image is drawn at the specific lat/lon. If that calculation is off then it would appear as if the markers drift while zooming. I haven't seen this issue in a long time though.

@hectorC Any luck with the tests of the Marker variation I suggested?

@hectorC Any luck with the tests of the Marker variation I suggested?

Sorry for the delay. I got to test it today and both with and without acceleration the width and height is reported as the same (w,h=261,252)

@hectorC And I assume that even with this version you still have the size/position bug with the acceleration on, am I right?
Could you test again with that version?
@Override
protected void drawAt(final Canvas pCanvas, final int pX, final int pY, final float pOrientation) {
	final int markerWidth = mIcon.getIntrinsicWidth();
	final int markerHeight = mIcon.getIntrinsicHeight();
	Log.e("drawAt", "w,h=" + markerWidth + "," + markerHeight);
	Log.e("drawAt", "x,y,orientation=" + pX + "," + pY + "," + pOrientation);
	Log.e("drawAt", "rect=" + mRect);
	final int offsetX = pX - Math.round(markerWidth * mAnchorU);
	final int offsetY = pY - Math.round(markerHeight * mAnchorV);
	mRect.set(offsetX, offsetY, offsetX + markerWidth, offsetY + markerHeight);
	RectL.getBounds(mRect, pX, pY, pOrientation, mOrientedMarkerRect);
	mDisplayed = Rect.intersects(mOrientedMarkerRect, pCanvas.getClipBounds());
	if (!mDisplayed) {
		return;
	}
	if (mAlpha == 0) {
		return;
	}
	if (pOrientation != 0) {
		pCanvas.save();
		pCanvas.rotate(pOrientation, pX, pY);
	}
	final Paint paint = new Paint();
	paint.setStyle(Paint.Style.FILL);
	paint.setColor(Color.RED);
	pCanvas.drawRect(mRect, paint);
	if (pOrientation != 0) {
		pCanvas.restore();
	}
}

The idea is to replace your icon with a red square: is the size of the red square the same with acceleration on/off?

I'm having this issue as well. I'm testing on Android 10 on my Pixel 2xl. Hardware Acceleration enabled because performance is pretty bad with it off. The marker also appears unusually small, I had to scale it up to get it the right size. I can get it to not move around by tuning the anchor to something like .06 and .11, but then the popup menu on click appears like a cm to the right of the marker.

@anthonyebiner Couldn't it be because you didn't populate all the *dpi folders with your custom icon?

I am having the same exact issue: changing the marker icon to a custom png image causes it to drift away when zooming.
Also the initial positioning of the marker is not accurate: I long press a location but the marker appears at another location (quite near).
This does not happen with the default icon or with another osmdroid-provided icon, like osm_ic_follow_me_on.
Using mapView.setLayerType(View.LAYER_TYPE_SOFTWARE, null) solves it for me as well.

@mminelli Turning off the hardware acceleration as suggested initially by @hectorC can be performed either with mapView.setLayerType(View.LAYER_TYPE_SOFTWARE, null) or android:hardwareAccelerated="false".
That being said, what happens if you copy the osmdroid-provided icons in your project folders and use them? If that looks OK, maybe you will find the differences between "our" icons and "yours", regarding the size or the anchor position.

@monsieurtanuki Thanks for the suggestion: I took an icon (the follow_me_on one) from osmdroid's mdpi folder and I copied it with a new filename to my folders drawable and drawable-v24. I also removed the code for disabling HW acceleration.
The problem is still there: on zooming, the icon drifts. As mentioned in the discussion above, could it be because osmdroid has several versions of the icon for several screen densities?

@mminelli I don't remember exactly the purpose of drawable-v24. What I can say is that with earlier versions of android you were supposed to populate 5 versions of the same icon in 5 different folders: drawable-hpi, mdpi, xhdpi, xxhdpi, xxxhdpi, as you can see in osmdroid for most icons. And I never trusted the bare drawable folder regarding icons.
My suggestion: copy the 5 versions of the icon in the 5 folders and ignore the drawable and drawable-v24 folders.

@monsieurtanuki OK, I think I was confused because back in the days, the drawable-XXX folders for the different densities would be created automatically when creating a project. But this seems not to be case anymore and these folders have to be manually created if needed.
Instead, Android Studio currently creates the following folders:
drawable      layout  mipmap-anydpi-v26  mipmap-mdpi   mipmap-xxhdpi   navigation
drawable-v24  menu    mipmap-hdpi	 mipmap-xhdpi  mipmap-xxxhdpi  values

and the mipmap-XXX folders are supposed to contain the application icon, not the drawables.
I will try with creating the 5 folders and I will report back. Thanks!

@monsieurtanuki Problem confirmed as solved: creating the folders
drawable-xhdpi
drawable-xxhdpi
drawable-xxxhdpi
drawable-mdpi
drawable-ldpi
drawable-hdpi

and populating them with my icon (which is also placed in drawable) makes it stay correctly in place even when I zoom in/out. The screen size of the icon is constant.

@mminelli Cool! Does it work regardless of the hardware accelerated parameters?
@anthonyebiner What about you?

@monsieurtanuki Yes, I removed the code that disabled HW acceleration and everything is fine.

@Mr-Ramzan Quickly said: markers have an anchor, with a default value that maybe does not fit your needs.
For instance, if your marker icon is a pin, the anchor should probably be at the end of the pin. If your marker icon is a target, the anchor should probably be at the center of your target.


Why nested templates are bad under Aurelia?

The readme example says that nested templates are bad under Aurelia. Why so? I'm using nested templates for containerless conditions and cycles. For example:
            <template if.bind="!isLoading">
                <template if.bind="isLoadingSucceeded">
                    <template if.bind="models.length > 0">
                        <template repeat.for="model of models">
                            <entity-row model.bind="model"></entity-row>
                        </template>
                    </template>
                    <template if.bind="models.length === 0">

No records

                    </template>
                </template>
                <template if.bind="isLoadingFailed">

Loading failed

                </template>
            </template>
            <template if.bind="isLoading">

<spinner></spinner>

            </template>

Can you suggest a better solution? I'd rather not to use nested divs, it could break my design. Also note that it is not possible to use if.bind and repeat.for on the same element. https://github.com/aurelia/binding/issues/250

note that it is not possible to use if.bind and repeat.for on the same element

I've listed this idea in aurelia tools linter issue (couple of months before the first commit in this project ;))

hmm I actually don't know if that is valid or not; if it actually works then I'll remove the rule. I was going off the aurelia cheat-sheet which specifically says that:
<template>

    <template repeat.for="customer of customers">

        ${customer.fullName}

    </template>

</template>

is "Illegal Table Code".  perhaps @EisenbergEffect can clarify?
It only occurred to me that this has misled me completely. If nested templates work like that I will patch the rule out, or at least make its use optional (config flag to disable it)
For example in latest blog, it talks about using slots for content projection; I think there was only one use-case left for nested templates where you need to use the "replace-part" attribute.

I believe it is illegal only in tables, and it would be great if it worked See https://github.com/aurelia/binding/issues/112

ah ok... I will rewrite it so its only illegal under containers then.

Great, thanks!

Should be able to upgrade to 0.5 and it will stop complaining.


API for determining if keypress would cause completion

I would like to be able to handle the Enter key, but only if the user did not press it to accept a suggested completion. There does not appear to be an API for detecting this.
The closest I could come was,
editor.onKeyDown((e) => {
    if (e.keyCode === monaco.KeyCode.Enter) {
        // If the editor is suggesting a completion of the user's input, we
        // should not intercept the key event.
        //
        // See vscode/src/vs/editor/contrib/suggest/suggestWidget.ts
        // See https://stackoverflow.com/q/62395001
        const suggestWidgetData =
            editor._contentWidgets["editor.widget.suggestWidget"];
        const isSuggestWidgetOpen =
            (suggestWidgetData !== undefined) &&
            (suggestWidgetData.widget.state === 3 /* Open */);
        if (isSuggestWidgetOpen) { return; }

        // ...
        e.stopPropagation();
        e.preventDefault();
    }
});

But it requires using some implementation details of the editor.
I'm not sure if it would be appropriate to also check whether the key is in commitCharacters of a potential completion entry.

@rgov You can use context keys and negate the completion accepting keybinding:
The completion accepting keybinding is (you can find this by running vscode and using F1 > Preferences: Open Default Keyboard Shortcuts (JSON):
{ "key": "enter",
  "command": "acceptSelectedSuggestion",
  "when": "acceptSuggestionOnEnter && suggestWidgetVisible && suggestionMakesTextEdit && textInputFocus" }, // let's call this X

You want that your keybinding executes when the focus is in the editor (and not in the find widget for example), so you want editorTextFocus && !X. textInputFocus is an alias for editorTextFocus that is there even for simple editor widgets, so you would end up with the when condition: editorTextFocus && (!acceptSuggestionOnEnter || !suggestWidgetVisible || !suggestionMakesTextEdit). That needs to be distributed because the context key language does not support parens, arriving at editorTextFocus && !acceptSuggestionOnEnter || editorTextFocus && !suggestWidgetVisible || editorTextFocus && !suggestionMakesTextEdit
e.g. at https://microsoft.github.io/monaco-editor/playground.html#interacting-with-the-editor-adding-a-command-to-an-editor-instance
var jsCode = [
	'"use strict";',
	'function Person(age) {',
	'	if (age) {',
	'		this.age = age;',
	'	}',
	'}',
	'Person.prototype.getAge = function () {',
	'	return this.age;',
	'};'
].join('\n');

var editor = monaco.editor.create(document.getElementById("container"), {
	value: jsCode,
	language: "javascript"
});

editor.addCommand(monaco.KeyCode.Enter, function() {
    alert('my command is executing!');
}, 'editorTextFocus && !acceptSuggestionOnEnter || editorTextFocus && !suggestWidgetVisible || editorTextFocus && !suggestionMakesTextEdit')


Fix flaky spec due to trainee#withdraw_date

Context
This flaky spec occurs (or at least, one of the ways to consistently reproduce it is) when withdraw_date is before course_start_date, as that fails this validation
Changes proposed in this pull request
Ensure that the withdraw_date is always after course_start_date
Guidance to review

Change this line to be a date before course_start_date, eg:

withdraw_date { Faker::Date.between(from: potential_course_start_date - 2.years, to: potential_course_start_date) }

run bundle exec rspec spec/forms/withdrawal_form_spec.rb to reproduce a consistent failure.

 Flakey withdrawal form feature test


chore: Upgrade to IBC 2.0

Fixes #1485.
Fixes #1486.

Description

Motivation and Context

[ ] I have raised an issue to propose this change (required)
[ ] My issue has received approval from the maintainers or lead with the design/approved label

How Has This Been Tested?

Types of changes

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

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I've read the CONTRIBUTION guide
[ ] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thanks @arijitAD - I don't see the tag on https://github.com/ovrclk/cosmos-sdk - am I just missing it?

Thanks @arijitAD - I don't see the tag on https://github.com/ovrclk/cosmos-sdk - am I just missing it?

Added tag: v0.44.5-akash-1.


What is the difference between Buffer() and ViewBuffer()?

I'm trying to understand the difference between Buffer() and ViewBuffer(). My understanding is that ViewBuffer() should only return what is actually viewable on the screen. However, this is not the behavior seen when running the _examples/buf.go demo. It also does not work with the _examples/demo.go when pressing KeyCtrlW, which saves the entire text of Tom Sawyer. Am I missing something obvious? Thanks!

From what I understood, the difference is mostly useful when Wrap is true. In that case, ViewBuffer returns the buffer with newlines as they are shown on screen, while Buffer has newlines only when there are actually in the text.
If you need to calculate the position of the Cursor/Origin, you should use ViewBuffer. If you need to get the contents of a view to handle the text, use Buffer.

Exactly. Long story short, what you see in the screen is a slice of ViewBuffer().


Feature request for Model::populateModels method

Craft 2 has a very useful helper method, populateModels, for mass-populating models based on an array of attribute arrays. One of several use cases is to create models from an array of active records, for example:
$myRecords = MyRecord::find()->all();
$myModels = MyModel::populateModels($myRecords);

Without this one must loop through each record:
$myRecords = MyRecord::find()->all();
$myModels= [];
foreach ($myRecords as $myRecord) {
    $myModels[] = new MyModel($myRecord->getAttributes());
}

If this is a valid feature request then please consider it as one :)
Yii 2.0.0

Why would you populate one model with data from another model?

You might want to populate and return a model from an active record to limit access to the database record.

Sounds too abstract. Would be easier to get real world example.

Ok, say I have a database table that stores contacts. I want to send a contact model to a template or a controller, however I don't want to grant it access to change anything in the database table. In this case I fetch the contact using an active record (ContactRecord), but I want to populate and send a model (ContactModel) rather than the record. Make sense?

Yes. Well, usually the models you need in view are representing the view thus likely to have fields different from what's in AR.

So you'll need to configure what fields of AR model to map to fields of your DTO/view model.

Means either:
$myRecords = MyRecord::find()->all();
$myModels= [];
foreach ($myRecords as $myRecord) {
    $myModel = new MyModel();
    $z = someprocessing($myModel->c);
    $myModel->setAttributes(['x' => $myRecord->a, 'y' => $myRecord->b, 'z' => $z])
    $myModels[] = $myModel;
}

or
$myRecords = MyRecord::find()->all();
$result = SomeClass::populateModels($myRecords, MyModel::class, function($model) {
  $z = someprocessing($myModel->c);
  return ['x' => $myRecord->a, 'y' => $myRecord->b,  'z' => $z];
});

Looks quite similar to me...

Note to whoever will merge it: don't squash (multiple contributors are involved).

In my case, a model representing an active record will generally contain a subset of the record's fields, so this works perfectly:
$myModel->setAttributes($myRecord->getAttributes());
Meaning that in a single call I can get my models populated with the record's fields (albeit with the same names):
$myModels = MyModel::populateModels($myRecords);
But I see your point about mapping fields and perhaps this use case is specific to my needs. Thanks for your input and for taking this under consideration.

Overall, thinking about it, I prefer not to include it in the framework: simple form looks good but it too specific. Complex form is actually looks better with foreach.
For your case I suggest you to extend your ARs from a base class and add it there or add a helper that does it.

Yes, I wlll do that. Thanks again for the discussion!!


[bug] Menubar Submenu Enabled state doesn't update automatically

Describe the bug
When you change the enabled state of the submenu items in the menubar (using code), the visible enabled/disabled state doesn't update.
The visible enabled state of the submenu items only updates when:

the submenu is 'physicly' updated:

by mouse hover
by keyboard 'hover' (by pressing Alt and then use the arrow keys)

the window loses it's focus:

By pressing Alt-Tab
By clicking into another window (make sure you can still see the tauri window!)

Reproduction
The bare minimum that you need is in this repo: https://github.com/Miniontoby/tauri-menubar-bug
The steps are in the README over there, but I will put the actual reproduction steps (if you try with or without the given code):

Have an app with menubar with submenu items and have a way to toggle the menubar submenu items their enabled state.
Toggle the Enabled state of the items.
You will see the other submenu's don't update their visible enabled state.
Now use one of these methods:

Hover over one of the submenu's.
Press Alt and then use the arrow keys to navigate to the other submenu's. (They might all already update when pressing Alt, but not everytime)
Hold Alt and press Tab (you can hold Shift and press Tab to 'stay' on the window)

If you have a second screen, just click on any window on your second screen.
If you don't have a second screen, then place any window next to the tauri window and do step 3 and then click into the other window.

That should have shown the bug

Expected behavior
I was expecting the menubar submenu items to just visibly update their Enabled state when changing it with code.
Full tauri info output
PC Windows 11:
[✔] Environment
    - OS: Windows 10.0.22631 x86_64 (X64)
    ✔ WebView2: 130.0.2849.56
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.82.0 (f6e511eec 2024-10-15)
    ✔ cargo: 1.82.0 (8f40fc59f 2024-08-21)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.11.0
    - npm: 10.2.4

[-] Packages
    - tauri 🦀: 2.0.6
    - tauri-build 🦀: 2.0.2
    - wry 🦀: 0.46.3
    - tao 🦀: 0.30.5
    - @tauri-apps/api : 2.0.3
    - @tauri-apps/cli : 2.0.5

[-] Plugins

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - bundler: Vite

Laptop windows 10
[✔] Environment
    - OS: Windows 10.0.19045 x86_64 (X64)
    ✔ WebView2: 130.0.2849.56
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.77.2 (25ef9e3d8 2024-04-09)
    ✔ cargo: 1.77.2 (e52e36006 2024-03-26)
    ✔ rustup: 1.27.0 (bbb9276d2 2024-03-08)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 18.19.0
    - pnpm: 7.25.0
    - npm: 10.9.0

[-] Packages
    - tauri 🦀: 2.0.6
    - tauri-build 🦀: 2.0.2
    - wry 🦀: 0.46.3
    - tao 🦀: 0.30.5
    - tauri-cli 🦀: 2.0.0-alpha.2
    - @tauri-apps/api : 2.0.3
    - @tauri-apps/cli : 2.0.5

[-] Plugins

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - bundler: Vite

Stack trace
No response
Additional context
I also got an pre-built .exe in the releases tab on the repo so you don't need to build the project
And here is a screen recording showing the behaviour:
https://github.com/user-attachments/assets/93bf7b5a-bfb3-42cb-857e-8d5b656281ec

run cargo update -p muda inside src-tauri, and the bug should be fixed

Yeah that fixes it! Thanks!
I really don't get why no-one has found this before...


adding example responses for common issues

support request
code bug report

This change is

@heckj 👋 Would you please un-check and re-check the box to allow edits from maintainers? Thanks in advance. 🙇

done!


fr(google-maps): Tile Overlays/Layer support

Feature Description
Add support for Tile overlay/layer like the Android Google Maps SDK or the ImageMapType class of the Javascript SDK to add custom tyles
Use Case
To add custom tile overlays/layer to the map

Has there been any progress on this? I'm looking at the available classes and it seems like there is no supported manner to add imageOverlay layers to the overlayMapTypes property of a map


Class new line between docstrings / vars / methods

Input:
class ClassWithInit:
    def __init__(self):
        pass

Current output:
class ClassWithInit:
+
    def __init__(self):
        pass

Desired output:
class ClassWithInit:
    def __init__(self):
        pass

Input:
class ClassWithDocstring:
    """docstring"""
    var = 10
    def __init__(self):
        pass

Current output:
class ClassWithDocstring:
    """docstring"""
    var = 10

    def __init__(self):
        pass

Desired output:
class ClassWithDocstring:
    """docstring"""

    var = 10

    def __init__(self):
        pass

Pull Request Test Coverage Report for Build 369

7 of 7 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 95.488%

Totals

Change from base Build 366:
0.01%

Covered Lines:
2053

Relevant Lines:
2150

💛  - Coveralls

Pull Request Test Coverage Report for Build 369

7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 95.488%

Totals

Change from base Build 366:
0.01%

Covered Lines:
2053

Relevant Lines:
2150

💛  - Coveralls

@ambv, anything else I should take care of here? PEP 257 is now enforced and class def and first method are not separated by a new line any more.

Sorry for the too-late comment. But I think this is wrong, and the old behaviour was correct.

Desired output:
class ClassWithInit:
   def __init__(self):
       pass

For one, having a blank line is more consistent with the behaviour you get if you have a class docstring. And second, it's consistent with the PEP8 rule the methods should be surrounded by a blank line.

PEP 8 also says this:

Method definitions inside a class are surrounded by a single blank line.

It doesn't go into much detail, but it doesn't mention any exception like the class definition. Therefore I think it should be treated as there should always be blank line before and after the method.

I believe this should be consistent regardless of what the first line after class declaration is, be it a method, variable or docstring. And you wouldn't put an empty line between class and docstring, as per PEP 257.
If you check the wording in PEP 257 it actually clarifies that the one empty line rule is meant to be applied between 2 methods only:

... generally speaking, the class's methods are separated from each other by a single blank line
(src: https://www.python.org/dev/peps/pep-0257/)

Various PEPs use this style in examples, including PEP 8, PEP 484 and PEP 525.

@renstrom, if you decide to deconstruct particular wording in PEP 8 and PEP 257, you might be technically right but I don't think that would be right in the spirit of those documents, they aren't written in a particularly strict style (pun not intended). The authors of those PEPs write code that looks like what Luka implemented in this PR. Previous behavior was very unpopular and a big criticism of Black for "putting unnecessary empty lines" between the first method and the class signature.


[CI] Allow empty git commit when deploying docs to gh-pages

I believe with mkdocs
v1.6.0 we may have to allow empty commits, because the compressed sitemap shouldn't change anymore on every rebuild.

Uhm, this is still installing mkdocs 1.5.3: https://github.com/ukri-excalibur/excalibur-tests/actions/runs/8783703326/job/24100487531?pr=309#step:4:86

Mkdocs-material now supports mkdocs 1.6.0, and this is finally working: https://github.com/ukri-excalibur/excalibur-tests/commit/f0f3c1638eb7567ad02cca546a5ddac8d0d0afd5 is an empty commit, no change to the compressed sitemap! :partying_face:


Global use

@pafnuty @lazyphpdev не забудь посмотреть этот тиккет :ticket:
Good morning, how could it be done to use the module globally? For example, adding the insert from header and that it can be used in the whole site without being limited only to the default.tpl, thanks!

Hi @BerkerSP!
To apply another template for display userinfo, you shoul use additional param &template=templatename
You can add {include file="engine/modules/userbox.php?&userName=login&template=yourtemplate"} to any plase

Thank you very much for your prompt response!
I've been testing the template tag and it works perfectly but I wonder in addition to this, I've tried to make the {custom} tag work also with the Userbox module to be able to customize the image quality as well as the amount of news that are displayed but I failed, is there any method for that tag to be covered too? Thank you very much!

@BerkerSP Could give the code wich you use please?

Sure, like example i've this in main.tpl to call slider using owl-carousel
{include file="engine/modules/userbox.php?&userName=this&template=app/modules/sections/car-2"}
and i want the carousel can recognize the module userbox to use public xfvalue userfield but i'm calling menues too and with that the code to request news of site

 					Best news

 					{custom template="/app/modules/content/slider/test" category='19,20,21,22,23' from='0' limit="12" cache="no"}

Home

FAQ

Help

But this don't recognize with that method and the link of images appear like:
https://images1-focus-opensocial.googleusercontent.com/gadgets/proxy?url=[xfvalue_poster]?q=[xfnotgiven_cdx_img-q]100[/xfnotgiven_cdx_img-q][xfgiven_cdx_img-q]10[/xfgiven_cdx_img-q]&fm=jpg&container=focus&refresh=31536000


why are two cats spawning at once

see #catching, theres a Rickroll cat and a sus cat that spawned in the same second

insufficient information


Update license with up-to-date year

Only made 1 change:

Update LICENSE.md to include the latest year in the form of a date range.

No other changes were made.

Thanks for the pull. Unfortunately changing licenses isn't a one-project edit and should be made across all our projects. I'd like to avoid updating all licenses every year and just leave it as is. If you're looking for ways to contribute, see here


Drop serie tags while dropping another serie

Steps to reproduce:

I had a serie (Serie1) on our database defined by one tag (tag1)
I injected new datas with same tag as Serie1 and a new tag (tag2).
I dropped the serie defined by tag1 and tag2 using DROP SERIES WHERE tag1='valuetag1' and tag2='valuetag2' to drop the points injected at step 2.
I queried our database using SELECT * FROM m WHERE tag1='valuetag1' expecting to get the Serie1 as it was at step 1 but the result was a serie where tag1 has 'disapeared' (blank field in the web admin interface and None field throught the python API).

I don't understand what happened there, I supposed the tag is 'still there' because I can still make query on it but I can't see its value.

Can you distill your repro steps into actual insert and selects that we can run to reproduce the issue?
Also, what version are you using?

I'm using InfluxDB 1.2.0. I tried to build another database and to reproduce the case that I explained previously but no bugs appear : when I use DROP SERIES with WHERE on tags, the series which match the WHERE clause are deleted but the tags of the other series don't "disappeared" as it occured in my original database.
Moreover, I just noticed that all tags of all series have been "cleaned" of my database (except the tags of the series which have been builded when I injected new datas to my original series adding a new tag to these datas to be able to find them easily) : I can't see them when I query with web interface or python api, but I can still query on these tag to find datas of a serie.

Looks like all tags are back after a reboot of our database.


The new version of Tokens

According to https://github.com/mintlayer/core/pull/67 the format of the token was changed.
Old functions for creating and sending a token were removed due to we cannot sign transactions on the node side, all transactions must be created and signed on the wallet side. In this version, we only verify is tx correct or not.
Now we have an additional data field that is optional. This field using for:

Token transferring
Token issuance
Token burn (not tested yet).
NFT mint (tested weakly).

Additional unit and functional tests will make later in another PR.
Also, we have not finished yet discussed fixing the verification function, but decided that return here after the test-net: https://github.com/mintlayer/core/issues/81
Unfortunately, we have a bunch of cycles there but it works correctly.
 Expect any comments from me to be very slow :)

Good job. We're getting much closer to what tokens should look like. I think generally the PR is OK. Just a few more tests and the other few comments I mentioned and we're probably good to merge.

Agreed with Sam, although in all honesty I've not been through in as much detail as I'd like and I haven't been able to run it for obvious reasons. I think we're close.

It seems that I have done @muursh @TheQuantumPhysicist