Skip or suppress rule in TemaplateAnalyzer?

Hi,
The docs for the .gdnconfig show only a handful of variables for the TemplateAnalyzer: https://github.com/microsoft/security-devops-action/wiki#templateanalyzer-options
Of which, there doesn't appear to be a way to suppress or skip certain rules.
The docs for TemplateAnalyzer indicate you can skip objects: https://github.com/Azure/template-analyzer/blob/main/docs/customizing-evaluation-outputs.md#template-analyzer-rule-object
Is this possible in Defender for DevOps currently, and it's more of a documentation gap or is this not yet a feature we can leverage?
Thanks

Thank you for reaching out! Currently the version of TemplateAnalyzer in Defender for DevOps does not support this feature, but I will follow up with my team about this and see if I can get an ETA for when it will be available.

Thank you for reaching out! Currently the version of TemplateAnalyzer in Defender for DevOps does not support this feature, but I will follow up with my team about this and see if I can get an ETA for when it will be available.

Thank you!

We are planning an update on TemplateAnalyzer in the next couple of weeks and now we will be including the -c parameter in that update, which should enable you to use a configuration file and in turn grant access to this feature. A rough ETA for release would be about 2-3 weeks.

We are planning an update on TemplateAnalyzer in the next couple of weeks and now we will be including the -c parameter in that update, which should enable you to use a configuration file and in turn grant access to this feature. A rough ETA for release would be about 2-3 weeks.

That's great, thank you for the speedy update, appreciated. If you require any feedback or testing I'd be happy help if need be 😃

We are planning an update on TemplateAnalyzer in the next couple of weeks and now we will be including the -c parameter in that update, which should enable you to use a configuration file and in turn grant access to this feature. A rough ETA for release would be about 2-3 weeks.

Hey, did this feature get release yet? We've got a requirment for this too, as we've had to create a custom role definition within our ESLZ, and the scan is currently failing because  'Using custom roles is treated as an exception and requires a rigorous review and threat modeling..'. I can't see any way I can supress this and accept the risk? I was hoping to be able to include the Id (TA-000020) in the config file, as documented on the Terrascan repo.

We have similar problems: for example, we have some storage accounts where we cannot set IP firewall rules, so it is important to us, to be able to suppress some of the rules.

We've actually stopped using this entirely now and fallen back to PSRule as we get full control of the scanning, rule exclusions, etc.

We are planning an update on TemplateAnalyzer in the next couple of weeks and now we will be including the -c parameter in that update, which should enable you to use a configuration file and in turn grant access to this feature. A rough ETA for release would be about 2-3 weeks.

Hey, any update on those 2-3 weeks?

At long last, I believe this has finally been deployed. I am closing this issue accordingly, please re-open or create a new issue if you continue to experience problems with the -c parameter.


Bump commons-io from 2.7 to 2.8.0

Codecov Report

Merging #79 (c3991a0) into master (1506675) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #79      +/-   ##
============================================
+ Coverage     61.33%   61.37%   +0.04%
- Complexity      193      194       +1
============================================
  Files            11       11
  Lines           913      914       +1
  Branches        139      140       +1
============================================
+ Hits            560      561       +1
  Misses          303      303
  Partials         50       50

Impacted Files
Coverage Δ
Complexity Δ

...bs/ci/sauceconnect/AbstractSauceTunnelManager.java
58.69% <0.00%> (+0.18%)
27.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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

Codecov Report

Merging #79 (c3991a0) into master (1506675) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #79      +/-   ##
============================================
+ Coverage     61.33%   61.37%   +0.04%
- Complexity      193      194       +1
============================================
  Files            11       11
  Lines           913      914       +1
  Branches        139      140       +1
============================================
+ Hits            560      561       +1
  Misses          303      303
  Partials         50       50

Impacted Files
Coverage Δ
Complexity Δ

...bs/ci/sauceconnect/AbstractSauceTunnelManager.java
58.69% <0.00%> (+0.18%)
27.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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


WM ext browser support - Edge

Context
WM extension compatibility with the Microsoft Edge browser.
Todos
TBC

Bibi to do: test on Edge and then should be ready to close

To test: How it works in split-window mode (https://github.com/interledger/web-monetization-extension/pull/366#discussion_r1673836423)

In split view, the extension is only taking into the consideration only the first pane from the split view when updating icon. If we are on a non-monetized website in the first pane and on a web monetized one in the second one, the icon will have the non-monetized website identifier.

Arc's split view should be checked as well.

Blocked as we wait for the MS support team

https://github.com/microsoft/MicrosoftEdge-Extensions/discussions/102 🤷🏽♂🤦🏽

Found the sender port in Runtime.Port message handler returns correct tab 🎉
port.onMessage.addListener((message, sendingPort) => {
   console.log(sendingPort.sender?.tab?.id);
});

So, we can make a short connection on content script inject and store tabId for use in future messages?

Everything works once we get right tabId, except Edge doesn't update the icon (even with correct tabId!)


x/pkgsite: floating header obscures printout

If you go to a multipage doc like https://pkg.go.dev/github.com/google/go-cmp/cmp and print from Chrome, you can see in the PDF preview that the floating header is printed at the top of every page. Unfortunately, on all but the first page, the floating header is printed on top of actual text that would be useful to be able to read.
If I edit the page with Chrome to make the UnixFixedHeader div "display: none" then the printout is fine. So maybe doing that in print-specific CSS would be enough.

Yes, and we should also hide DocNavMobile, I'm working on it.

Thanks!


Remove taskchampion-lib

This was originally intended to be a general-purpose C API for TaskChampion, but it's really not great. It's also quite difficult to use as a crate, rather than including the code within the emebedding project, as Taskwarrior has done. With this change, TaskChampion is just a Rust library, used by Taskwarrior but usable by anything.
The integration tests mostly tested the C API, and those are removed. However, two provided integratoin tests for the taskchampion crate, and those are moved to taskchampion/tests.
This was prompted by the necessity of duplicating changes to lib both here and in the copy of the same code in https://github.com/GothenburgBitFactory/taskwarrior.

(I jumped the gun a little and marked the crate as deprecated. Sorry!)

The CI codegen failure would seem to be indicative.


Repo Maintenance

Repo Maintenance
User Story
As a user, I want the website page to be accessible, enticing, and easy to read.
Acceptance Criteria

Have a clean repository that meets quality coding standards (file structure, naming conventions, best practices for class/id naming conventions, indentation, quality comments, and so on).
Have a quality README (including a unique name, description, technologies used, screenshot, and link to the deployed application).
Be deployed to GitHub Pages.

This has been completed


(#1013) Add undo color support to PlayerLayer

Close #1013

@The-Renaissance @NilsIrl I have no idea what is going on in this PR. The GitHub interface is extremely confusing. If you guys have any specific suggestions feel free to submit a separate PR.


SSL certificate

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
Tell us what should happen
Actual behavior
Tell us what happens instead
Steps to reproduce

Environment
K-9 Mail version:
Android version:
Account type (IMAP, POP3, WebDAV/Exchange):

From the 29th march, K-9 Mail Version: 5.403 on Android 7 has thrown a  SSL certificate error.
My wife, k9 on her phone (same is & k9) also has it. We have same host.
I haven't touched any settings
I can still get my email on my desktop from the host and I can't see anything in their status log about SSL certificates.

Please use the mailing list for support requests.


Print layer output while training?

Is there a way to print the output of a layer during training? I am trying to debug a attention model in which the loss is not at all changing during training. I have tried changing learning rates, checking data sanity, but to no success. I want to see whether the layer outputs actually make any sense during training (eg. is my Lambda layer slicing correctly etc.).
I know I can print the weights by using,
Wr=m.nodes['Wr'].get_weights()
emb=m.nodes['emb'].get_weights()

but I want the output. When I do m.nodes['emb'].get_output() I get a theano variable instead.
Please make sure that the boxes below are checked before you submit your issue. Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps
[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps
[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

This is covered in the Keras FAQ. You should define a Theano function with
the same input as the network input, and m.nodes['emb'].get_output() as
output.
On 29 February 2016 at 15:30, Shyam Upadhyay<EMAIL_ADDRESS>wrote:

Is there a way to print the output of a layer during training? I am trying
to debug a attention model in which the loss is not at all changing during
training. I have tried changing learning rates, checking data sanity, but
to no success. I want to see whether the layer outputs actually make any
sense during training.
I know I can print the weights by using,
Wr=m.nodes['Wr'].get_weights()
emb=m.nodes['emb'].get_weights()
but I want the output. When I do m.nodes['emb'].get_output() I get a
theano variable instead.
Please make sure that the boxes below are checked before you submit your
issue. Thank you!

Check that you are up-to-date with the master branch of Keras. You can
update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps
If running on Theano, check that you are up-to-date with the master
branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps
Provide a link to a GitHub Gist of a Python script that can reproduce
your issue (or just copy the script here if it is short).
—
Reply to this email directly or view it on GitHub
https://github.com/fchollet/keras/issues/1858.

@shyamupa  how do you solve it.

the loss is not at all changing during training. I have tried changing learning rates, checking data sanity, but to no success

I meet the same question and don't know how to debug it ? Thanks.

I don't remember exactly but this usually happens when the network is somehow degenerated. Like one layer outputs always 1 and so derivatives go to zero.

Hi, @fchollet
I am kind of new to keras, so forgive me if I am sounding too rookie.
I want to train a CNN with 4 LSTMs in an end2end process. The goal is that after the CNN output I altered into 4 different orientations and then feed it to each of the LSTM.
Is this possible to do when defining the model?
I tried something like this:

And after getting the output data transformation occurs and the LSTMs kick in

@apinto92 did you find the answer?

@monsieor  I was able to use a function in Theano that performs the partition of the image in subpatches. I developed my own Partition layer to be able to feed small sequences of the whole image, and incorporated in the model.


typo

Pull Request Test Coverage Report for Build 5387

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 65.444%

Totals

Change from base Build 5385:
0.0%

Covered Lines:
7371

Relevant Lines:
11263

💛  - Coveralls


WIP: DNS-over-HTTP support for Client

This is a quick attempt at adding DNS-over-HTTP support to Client, inspired by https://github.com/miekg/dns/pull/647#issue-175710683.
This method uses http.Client and stubs the behaviour of the net.Conn present in Conn, it should be otherwise fully functional. It's entirely possible to build this same system from a raw *tls.Conn using the x/net/http2 package, but it's not clear to me how much benefit this would have.
Note: When using DNS-over-HTTPS, errors won't be surfaced from Dial, instead they'll show up in Write.

It would also be possible to simply push this upwards a layer and only support DNS-over-HTTPS in the various Exchange methods.

Codecov Report

Merging #649 into master will increase coverage by 0.03%.
The diff coverage is 46.29%.

@@            Coverage Diff             @@
##           master     #649      +/-   ##
==========================================
+ Coverage   57.91%   57.94%   +0.03%
==========================================
  Files          37       37
  Lines        9991    10045      +54
==========================================
+ Hits         5786     5821      +35
- Misses       3155     3172      +17
- Partials     1050     1052       +2

Impacted Files
Coverage Δ

client.go
58.17% <46.29%> (-1.68%)
:arrow_down:

msg.go
78.6% <0%> (+1.29%)
:arrow_up:

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 5b169d1...67ba6d6. Read the comment docs.

"I have some misgivings on supporting a transport in only some methods."

That's definitely understandable, although it would be the cleanest place to support this. I'd suggest any v2 rewrite (as you mentioned later) should probably only support Exchange style methods and drop the lower-level Read and Write methods.

"A quite like the hackiness of the approach you take :) But doing the actual Dial inside the WriteMsg method is icky."

It works :man_shrugging:. I can try a different implementation that came to mind, that would move the Dial into Dial. I don't think it would be cleaner than this implementation though.

"Not sure if this will fit this library ever."

I think DOH support would be worthwhile, but it obviously depends on how cleanly it can be integrated—baring a rewrite.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] WIP: DNS-over-HTTP ..." ]

"I have some misgivings on supporting a transport in only some methods."

That's definitely understandable, although it would be the cleanest place to support this. I'd suggest any v2 rewrite (as you mentioned later) should probably only support Exchange style methods and drop the lower-level Read and Write methods.

Back in the day Exchange was the only thing, then stuff hapenend and PRs got
merged. If you have connection pooling in a http.Transport like thing, Exchange
will be enough and you don't need to expose the lower level bits.

"Not sure if this will fit this library ever."

I think DOH support would be worthwhile, but it obviously depends on how cleanly it can be integrated—baring a rewrite.

True. I would be nice if this worked; even though the implementation is not the
cleanest.
For any rewrite I would like to retain the API as much as possible. Also worth
looking what's happening in x/dns (the new upstream "official" dns package) and
integrate build on that - that said I don't have a whole lot of time to start
doing that.


Enable multiwallet support for Bitcoin Core when activating the wallet

Specter calls uses multiple additional bitcoin core wallets.
Joinmarket does not have a wallet specified currently and uses the default wallet.dat.
The problem is when Specter is running (with active wallets) the wallet.dat is not specified, but there are multiple entries in:
bitcoin-cli listwallets
To specify a new wallet need to use multiwallet in Bitcoin Core: https://bitcoincore.org/en/releases/0.15.0/
Needs editing the bitcoin.conf an include at least one wallet= entry and restart.
Solution:
will need to add  wallet=wallet.dat in the bitcoin.conf when the wallet feature is activated or the latest when joinmarket is installed.
Need to specify the rpc_wallet_file = wallet.dat in the joinmarket.cfg as well.
Related issues:
https://github.com/openoms/joininbox/issues/3
https://github.com/rootzoll/raspiblitz/issues/1162

@openoms trying to get a RC2 ready this week ... should I wait for a PR on this?

Will have a go at it on Wednesday, should be solved for v1.6.
Should be ok to test the many new features in the RC2 knowing that this is pending.
JoinMarket has updated too (should not affect the install process):
https://github.com/JoinMarket-Org/joinmarket-clientserver/blob/master/docs/release-notes/release-notes-<IP_ADDRESS>.md

Tested that wallet.dat is always present and JM uses it by default.
Old JM configs will continue to work after recovery and no rescan is required.

OK fix is merged :)
@openoms maybe last point: bonus.cryptoadvance-specter.sh is activating the btc wallet still by manually chaning the bitcoin conf .. should we change this also to use the network.wallet.sh script?

@openoms maybe last point: bonus.cryptoadvance-specter.sh is activating the btc wallet still by manually changing the bitcoin conf .. should we change this also to use the network.wallet.sh script?

can do for consistency, but Specter is not affected. It calls it's own wallets via RPC and does not rely on any specific file in the bitcoin directory.

can do for consistency, but Specter is not affected.

ah ok, then we can leave it as it is

can do for consistency, but Specter is not affected.

ah ok, then we can leave it as it is

committed this directly https://github.com/rootzoll/raspiblitz/commit/5cdeb2ffa49856e253e6226f7e03958027399627
TheOutcome is the same, but spares a bitcoind restart if the wallet is already active.


comment:null leads to errors with style-dictionary

Hi,
thanks for this great plugin!
One thing I discovered in the latest version:
Missing comments are exported as comment: null, which leads to errors when using style-dictionary (I am using 2.10.3) to transform the styles. Changing these to comment: ""or removing them altogether leads to a working build.

Hey @MoritzStefaner,
thank you very much for raising this issue. I am working on it and will let you know once it is fixed.

Ah, interesting. Here's my config:
{
  "source": ["./in/**/*.json"],
  "platforms": {
    "scss": {
      "transformGroup": "scss",
      "buildPath": "out/",
      "files": [
        {
          "destination": "theme.scss",
          "format": "scss/variables"
        }
      ]
    },
    "css": {
      "transformGroup": "css",
      "buildPath": "out/",
      "files": [
        {
          "destination": "theme.css",
          "format": "css/variables"
        }
      ]
    },
    "js": {
      "transformGroup": "js",
      "buildPath": "out/",
      "files": [
        {
          "destination": "theme.js",
          "format": "javascript/es6"
        }
      ]
    },
    "json": {
      "transformGroup": "js",
      "buildPath": "out/",
      "files": [
        {
          "destination": "theme.json",
          "format": "json/nested"
        }
      ]
    },
    "scss-kit": {
      "transformGroup": "scss",
      "buildPath": "../www-svelte-kit/static/styles/",
      "files": [
        {
          "destination": "theme.scss",
          "format": "scss/variables"
        }
      ]
    },
    "css-kit": {
      "transformGroup": "css",
      "buildPath": "../www-svelte-kit/static/styles/",
      "files": [
        {
          "destination": "theme.css",
          "format": "css/variables"
        }
      ]
    },
    "js-kit": {
      "transformGroup": "js",
      "buildPath": "../www-svelte-kit/static/styles/",
      "files": [
        {
          "destination": "theme.js",
          "format": "javascript/es6"
        }
      ]
    },
    "json-kit": {
      "transformGroup": "js",
      "buildPath": "../www-svelte-kit/static/styles/",
      "files": [
        {
          "destination": "theme.json",
          "format": "json/nested"
        }
      ]
    }
  }
}

Nice, I will try this. I created a PR that should fix it by removing empty comments. I will try to test and merge it tomorrow.

Hey, I just published version 2.1.3 which changes the behavior to avoid any non-string comments

Sweet, thanks! Can confirm my issues are resolved.


[FLINK-29047][k8s] Shade fabric8 kubernetes with org.apache.flink.shaded prefix in flink-kubernetes

Shade the fabric8 kubernetes deps with org.apache.flink.shaded prefix in flink-kubernetes
What is the purpose of the change
For supporting stepDecorators SPI(pluginable decorators), we propose to package the implementation class and associated dependencies into a plugin jar.
So we need to load the said dependencies from parent class loader, as the most part / all of plugin decorators depend on the fabric8 kubernetes dependency, such as replies on the kubernetes models/client from fabric8.
So we need to shade all the said classes in flink-kubernetes and flink-dist.
Brief change log

Shade the io.fabric.kubernetes with org.apache.flink.shaded prefix.
Exclude the original left dependencies / classes / SPI files in flink-dist.

Verifying this change
Build the all flink packages, and verify the flink-kubernetes jar and flink-dist jar had re-leverage the affected dependecies, classes and files.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): ( no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@wangyang0918 , PTAL.

I will be great if we also shade the io.fabric8:zjsonpatch.

Could we simply shade everything under io.fabric8. @wangyang0918 ?

@gyfora Yes. We could shade all the dependencies under io.fabric8.

Ok, I will shade the said package io.fabric8:zjsonpatch with org.apache.flink.shaded prefix

Sorry for late...

@wangyang0918  Thanks


Logger addEventListener

Not sure if you think this is in the scope of this module, but one thing that would be very useful for me would be some way to hook-in a listener, so that I can react to logs (I tend to want to log and also send telemetry in similar locations). I wouldn't mind impl myself, but I wonder if this is something that you think would be useful?
Something like one of these:
const options = { blahblahblah }
const log = new Logger({ level: Logger.level.debug, options, tags })

log.addEventListener(({ level, content }) => {
  // do whatever here
})

const options = { blahblahblah }

options.onLog = ({ level, content }) => {
  // do whatever here
}

const log = new Logger({ level: Logger.level.debug, options, tags })

This looks interesting actually, I could see it very useful to collect telemetry and diagnostic usage indeed
Since I haven't yet encountered the use-case for this yet, would you mind sharing a bit more details about how you'd think it'd work ? Mainly the following points would need a bit more clarification:
API Interface
Implementing EventTarget seems good since it keeps consistency with existing conventions, but I wonder if removeEventListener would ever be called, and it'd probably feel weird to expose dispatchEvent.
Also addEventListener usually take the event name in first argument (which could be the loglevel, like addEventListener("warn", () => {})) and while it could seems a good idea to create events by channels, maybe it'd actually be less ergnomic if users want to trigger their listener for "warn and higher" for example. So I don't know wether we should to this or just offer a generic event listener whatever the channel is triggered.
In this case, maybe just a listeners (either a map/set/array) property could be more fitting, and I guess we could also add an options to populate it directly in the constructor too.
Listeners and log level
Would listeners always react independantly from log level ?
Consider the following:
const log = new Logger({level:Logger.levels.error})
log.warn("foo") // Would this be still dispatched to listeners despite not logging ?

If we use the latter pattern mentioned in the previous section, then we could trigger it everytime and it's the listener that would check the loglevel themself. Not sure what makes the more sense here
Listeners arguments
What kind of arguments would you expect to be usable within the listener ?
I saw you already listed level and content, but if needed it could also be possible to refactor a bit the lib to offer the caller, timestamps, date, etc.
However I guess they'll still be subject to enabled options because I think caller option probably slow down performances (since it needs to examine stack trace). Maybe it could be mitigated with by passing caller as a lazy function though.
Interoperability with Logger.with()
Currently Logger.with creates a "sub-clone" of a logger which inherits all options from parent logger.
Would listeners also be inherited ?
And if yes, should they be local to said logger (meaning that they don't bubble up) ? It could also be handled with the event.stopPropagation() API, but maybe it makes less sense if EventTarget is not emitted. Or maybe it makes more sense to not propagate in any case...

Oh wait yikes haha. Thanks for putting some brain cycles into this! tbh now I think I'm having second thoughts after reading:

Would listeners always react independantly from log level ?

It's a great question to ask, but makes me feel like my feature request is actually kind of improper... reason being that logs are code that have zero impact on a user; the kind of code that is least important. While I think this kind of mechanism could be very convenient, I can't help but think that tying logic to be triggered by the logger itself feels wrong in a sense. Like semantically it reeeeeally should probably be a wrapper or higher-order function that calls both the logger and the telemetry.

Yeah I agree that it'd be weird for logging to introduce side effects
I think like you said it'd make more sense to just extend the Logger to perform additional actions in case such a design pattern wants to be introduced by a user. Additionally I guess that if it's to send analytics to a remote server, there's a high change that you use fetch which would mix async/sync behaviours which could be a hell to handle with resources sanitization, etc.
However maybe something like this would be a better pattern:
class Analytics {
  #collected
  collect(data:Record<PropertyKey, unknown>) {
    this.#collected.push(data) // Could also add timestamp, caller data, etc.
  }
  async [Symbol.asyncDispose]() {
    await fetch(...) // Send collected data, could be a file or a remote server I guess
  }
}

await using a8 = new Analytics()
try {
  throw new Error()
}
catch (error) {
 a8.collect(error)
}
// Normally the async disposal will do the "cleanup" here

Yaya agree.  Could be interesting to have another lib that wraps this one, but I'll close this issue for now since it seems out of scope.


Interstitial and rewarded not presented for Flutter Android in emulator.

System/Configuration
Flutter version 3.13.2
Dart version 3.1.0
Android SDK version 33.0.2
Android Studio (version 2022.2)
google_mobile_ads: ^3.1.0
CocoaPods version 1.12.1
(Xcode 14.3)
Interstitial and rewarded are loaded but not presented in Flutter Android. Flutter iOS is working fine.

Hi @AnyMindG
I'm still seeing Interstitial and Rewarded ads on Android emulator with plugin sample code:

Interstitial
Rewarded

Please also try running this sample on your side. If the ads still can't be shown, please provide the emulator info (OS version, model). Thanks!

Closing this issue as I also cannot replicate - if this issue still persists, please provide more information, per @huycozy


Redshift: add migration script for user-agent-utils enrichment fields

closing, the enrichment is to be removed


Authentication using mount_gradio_app

Is there a way to authenticate using mount_gradio_app, as I want a certain endpoint to require a login

i am also interested in this feature

I was able to add this by updating to a newer version that supports auth.
https://github.com/zylon-ai/private-gpt/compare/main...niciliketo:private-gpt:secure_ui
I have not yet been able to write suitable tests and therefore have not submitted a PR

I was able to add this by updating to a newer version that supports auth. main...niciliketo:private-gpt:secure_ui
I have not yet been able to write suitable tests and therefore have not submitted a PR

Is this working properly for anybody ? Because I too wanted to authenticate my gpt.


Extract unit tests into its own workflow

Summary

Running e2e tests and image build again when merging a PR to master does not make much sense, since the tests ran in the PR before. Thus skip them in master branch to not waste resources and time
We still run unit tests in the master branch solely for the purpose of publishing CodeClimate test report. And the docker images are also built anyway since we push master tags.
Adds more badges

Checklist

[x] Keep pull requests small so they can be easily reviewed.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking,
as they show up in the changelog
[x] Update the documentation.

Running e2e tests and image build again when merging a PR to master does not make much sense, since the tests ran in the PR before. Thus skip them in master branch to not waste resources and time

I disagree. When multiple PRs are merged, and they were not rebased onto master before merging them, they could interfere with each other.

Running e2e tests and image build again when merging a PR to master does not make much sense, since the tests ran in the PR before. Thus skip them in master branch to not waste resources and time

I disagree. When multiple PRs are merged, and they were not rebased onto master before merging them, they could interfere with each other.

Running e2e tests and image build again when merging a PR to master does not make much sense, since the tests ran in the PR before. Thus skip them in master branch to not waste resources and time

I disagree. When multiple PRs are merged, and they were not rebased onto master before merging them, they could interfere with each other.

Right. We could set up a branch protection rule with "Require branch to be up to date". We must then select which checks absolutely must pass (I'd suggest the lint only) before merging, and so we can only merge branches that are up to date. This makes a rebase a requirement before merging, but I believe GitHub offers a one-click-rebase-branch button if the PR is outdated.
Would that be ok?

Running e2e tests and image build again when merging a PR to master does not make much sense, since the tests ran in the PR before. Thus skip them in master branch to not waste resources and time

I disagree. When multiple PRs are merged, and they were not rebased onto master before merging them, they could interfere with each other.

Right. We could set up a branch protection rule with "Require branch to be up to date". We must then select which checks absolutely must pass (I'd suggest the lint only) before merging, and so we can only merge branches that are up to date. This makes a rebase a requirement before merging, but I believe GitHub offers a one-click-rebase-branch button if the PR is outdated.
Would that be ok?


Reproducing ML-1M results from the paper.

In the paper you report 0.2946 recall@10 for ML-1M, however when I run duorec.sh command, I'm getting result around 0.25-026. Could you please share the run configuration which I can use to reproduce paper results?

I got the result when I set the dropout ratio as 0.1, you can have a try.


Parsing message 3 from longer slice

When implementing the EDHOC option for CoAP, the user of this library needs to do some CBOR parsing of the message3 structure on their own, just to hand the whole thing over to Lakers, possibly pulling in an own CBOR library.
I don't have a precise plan yet for how to best do this; possibly, a parse_message_3_from_stream() convenience method would do like parse_message_3, but return the consumed length as part of the Ok. (Technically there are some error states in which it could still indicate a parsed length; practically, this is irrelevant because what comes after the EDHOC option can only be used through a successful EDHOC operation anyway).

I am also not sure on the best way. We could either have a parse_message_3_from_stream() or something like edhoc_with_oscore::parse_comb_payload that directly returns the split msg3 and oscore items.


fix(data/list/basic): typo in pariwise_iff

[x] reviewed and applied the coding style: coding, naming
[x] make sure definitions and lemmas are put in the right files
[x] make sure definitions and lemmas are not redundant

Ups. Thanks for the correction.


New version: PhysicalParticles v1.2.3

Registering package: PhysicalParticles
Repository: https://github.com/JuliaAstroSim/PhysicalParticles.jl
Created by: @islent
Version: v1.2.3
Commit: 67a8693ec6f67147607b6bb9525c2aae968d7ba5
Git reference: HEAD

Close PR. Switch back to v1.2.2


Add FAQ JSON File

This PR adds the faq.json to facilitate the update on frequently asked questions displayed in the help dialog.

Thanks!
@Rensvandeschoot and the team can do new PRs to update this file with the information of preference!


Ionic - undefined is not an object

I'm trying to use this plugin with a new Ionic app, when pulling in the plugin and calling the plugin in my home controller as a test, hooking up a button to fire one of the functions on this plugin. Code & error message below.
I'm testing this via the Ionic Serve command with the Ionic DevApp.
Part of my home component
import { Component } from '@angular/core';
import { NavController } from 'ionic-angular';
import { Printer, PrintOptions } from '@ionic-native/printer';

declare var cordova; // DECLARING Cordova

@Component({
  selector: 'page-home',
  templateUrl: 'home.html'
})
export class HomePage {
   constructor(public navCtrl: NavController, private printer: Printer) {...}
...

Test Function
brotherTest() {
    cordova.plugins.brotherPrinter.findNetworkPrinters((data)=>{
        console.log("Success");
        console.log(data)
    },(err)=>{
        console.log("Error");
        console.log(err)
    });
  }

Error Message

undefined is not an object (evaluating 'cordova.plugins.brotherPrinter.findNetworkPrinters')

dunno what to tell you... what specifically is undefined? you didn't say...
make sure the plugin is actually installed, regenerate platform if need be. what platform btw?
sorry I can't be more help, you didn't give much to go on. shouldn't you use $ionicPlatform.ready and $window.cordova?

Hello, did you ever figure this out? Feel free to re-open if you still have issues.


Offline mode

Description
Add a new option called Offline that makes the module work with no network and use only the default value.
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking changes (change that is not backward-compatible and/or changes current functionality)

Closes issue(s)
Resolve #131
Checklist

[x] I have tested this code
[x] I have added unit test to cover this code
[x] I have updated the documentation (README.md and /docs)
[x] I have followed the contributing guide

Coverage increased (+0.05%) to 96.941% when pulling 56e7c8ea4f61f801cee293ffdf0134afacb61c3a on feat/131-offline into ca3840b6806246dc967d362c4f8383216f96ac8b on main.


Improve test coverage for Probes module

It's the least-effort highest-reward place for testing right now:

Coverage from the probe module is now up into the mid-80's in the branch I'm working on.  I've added some tests that open GUI visualizations---just need to make sure Travis doesn't break & merge, then this issue can be closed.

Coverage from the probe module is now up into the mid-80's in the branch I'm working on.  I've added some tests that open GUI visualizations---just need to make sure Travis doesn't break & merge, then this issue can be closed.


Commitment Tx page

Make the first pre-filled page to show the user the format for the rest of the chapter

https://www.figma.com/design/LqjK3Tpvd9KJ4buFArCJBQ/Saving-Satoshi?node-id=8869-20083&node-type=frame&t=mel0DdAeZpePkAxg-0

Closed by #1121


copyright perms testing

Fix copyright.yaml workflow
Remove other actions
Adding permissions and always running copyright on branches

Motivation

Future Work

Closing this was a test to get the copyright action running


[AutoRelease] t2-apimanagement-2022-11-11-20916(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3403
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=966

issue link:https://github.com/Azure/sdk-release-request/issues/3403


Purchasing Pro from settings menu causes settings view to appear broken

Hi!
Not sure if I’ll be able to reliably duplicate this bug as it involves purchasing the pro upgrade, but I noticed today after installing the App Store version (congrats btw!) and opting to purchase pro, when I went back to the main settings page, it appeared “broken” so to speak.
Here’s what I saw
Version 1.0.1 build 38, running on the latest iOS 12 Beta.

I believe this is fixed in v1.0.3. If you see this again please re-open this issue!


Props Table Enchancements for Custom propType Declarations

Feature Request
PropsTable enchancements for custom propType declarations
Description
In many cases custom propType shapes are declared for reuse or read-ability. Sometimes this custom propTypes are either co-located in the component file or in a seperate file for import. Currently only the main propTypes declared for a given component are read and the custom shape doesn't have a clean method of adding to documentation.
Either a method of including the propType reference or file directly into documentation or having the doc gen follow then import path to populate the table would be perfect.
@ Mentions
@mjhenkes

Investigation:
There was a similar issue logged in react-docgen issue 33. For which even a PR is out react-docgen PR 352.
Should we wait for it to get merged?


Video Tutorial - Template Engine

Summary
What: Video tutorial (acts of visual documentation) for how to use Drash's template engine (rendering, creating, extending etc), how it works, how to use them etc - acting as the documentation but in video form. Though it's open to whoever takes on this issue how they'd like to do the video and what content it will have.
Could be useful to include in the description of the video, times of topics, eg:
0:00 - introduction
0:23 - what is the template engine
5:10 - creating a view
Why: Supports visual learners, and thus creates another platform for people to understand Drash's template engine and how to use it - further enforcing "documentation driven"
Information

OBS can be used as the software to screenshare and video recording

Includes the drashland introduction? (speak to @crookse)

Includes the drash logo with the greenish background as the thumbnail

Video can be added to a "Drash" playlist (can also be created if it doesn't exist)

Make it chill :)

closing - moving this to the private task area


Unable to use Scroll API when dumping index

elasticdump version: 4.7.0
elasticsearch version: 5.6
node version: 8.15.1
Command:
NODE_TLS_REJECT_UNAUTHORIZED=0 elasticdump --tlsAuth=true --input="https://<host>/<index>" --input-key=key.pm --input-cert=cert.pem --input-ca ca.pem --output=./2019.02.04.json --type=data --limit=200 --size=1000 --sourceOnly=true --debug=true

Output:
Mon, 15 Apr 2019 17:33:38 GMT | starting dump
Mon, 15 Apr 2019 17:33:39 GMT | got 10 objects from source elasticsearch (offset: 0)
Mon, 15 Apr 2019 17:33:39 GMT | sent 10 objects to destination file, wrote 10
Mon, 15 Apr 2019 17:33:39 GMT [debug] | scrollRequest: {"uri":"https://<host>/_search/scroll","method":"POST","body":"{\"scroll\":\"10m\",\"scroll_id\":\"DnF1ZXJ5VGhlbkZldGNoBQAAAAAAApDlFnktUmdyeTZMUnJhUFl6TnJ2bEdvM1EAAAAAAAMA-BZHRklwajdxS1FXZU1kbDB3OUVhcldnAAAAAAACh3IWX1JTRjhTcDdTZ0NfeDB3NmJhcGpnZwAAAAAAAwUgFm1fMFplRG5CUnFXaVhzTFFQSFZyNXcAAAAAAAKo3hYyYmpZTlR4V1NDYTBYWVlaOXlOb3lB\"}"}
Mon, 15 Apr 2019 17:33:39 GMT [debug] | body: "404 page not found\n"
Mon, 15 Apr 2019 17:33:39 GMT | Error Emitted => 404 page not found

Mon, 15 Apr 2019 17:33:39 GMT | Total Writes: 10
Mon, 15 Apr 2019 17:33:39 GMT | dump ended with error (get phase) => Error: 404 page not found

I tested this in Kibana devtool with a search request asking for it to scroll. Got back a _scroll_id that I then send on a second request formatted similar to how it is shown in the debug message and got a response back properly. Not sure why this tool is giving a 404 on the request.
Kibana devtool:
POST <index>/_search?scroll=10m
{
    "size": 1
}

POST /_search/scroll
{
    "scroll" : "10m",
    "scroll_id" : "DnF1ZXJ5VGhlbkZldGNoBQAAAAAAAptfFnFoUm9fU1VHUmV5dktzcHJiWVJaQ2cAAAAAAAZ_1xZtWVdCd3JSRlR2cUhjUFJuZ2Q5REF3AAAAAAABiIcWR2xycEpYZ0xTb3kyRkVNS2tLUVg3QQAAAAAAAgkcFjU5VzgyeXRoUndpcEZzSHB3R1RCdkEAAAAAAAKWWhY1SnRkLVRMd1NIaVBGYkZjOWhtYjl3"
}

Multiple calls to the second request results in a new document in the "hits" section of the response and a _scroll_id in the response.

This might be an issue on our side. Looks like the endpoint I was given for the ES cluster is actually a proxy and the /scroll API may not have been exposed. Following up with the Ops on the ES cluster and will close this issue if that is the case.


Add minimum burning period to OSLP [partly RTM-100-025]

User story
As a Net Manager,
I want a device to burn for a minimum number of minutes,
so I prevent significant decrease of the light's life

@marcelvanhal what is the priority of this user story?


Factory.Create no longer runs Configure on the Endpoint

Firstly thank you for this excellent library, we started using it about a year ago and love it's simplicity and power.
We're currently updating from 5.2.1 to the latest and we noticed that Configure is no longer called on the Endpoint when using Factory.Create.
I've looked at the source code and it seems to have been introduced from the following commit:
dependency resolving support for endpoint Configure() method - 77d656ed56310307913a71e04371253b3f2098dc

the crux of it is that ImplementsConfigure and EndpointAttributes are no longer set. I was going to PR my change which gets it workin, seen below on lines 32 and 33, but I imagine that's now how you want to implement it.

Test added:
public class FactoryCreateTests
{
    [Fact]
    public async Task ConfigureIsExecuted()
    {
        var ep = Factory.Create<ConfigureEndpoint>();

        ep.Definition.Routes.Should().ContainSingle("configure/test");
    }
}

public class ConfigureEndpoint : Endpoint<Request, Response>
{

    public override void Configure()
    {
        Verbs(Http.GET);
        Routes("configure/test");
    }

    public override Task HandleAsync(Request r, CancellationToken ct)
    {
        return SendErrorsAsync();
    }
}

wow, thanks so much for the quick turn around. We're testing it now.

you're most welcome. it was fast because you found the exact cause :-)
a patch release v5.21.2 has been pushed to nuget.
should be live in a few minutes.

It worked perfectly by the way, thanks again!


feat(initializable): add initializable

#107

@ItsNickBarry IDK why renamed folders are counted in the last commit. Renamed is done.

The code from this PR has been included in #199 and modified in such a way that it does not work as originally intended.  Published in v0.0.55.  Thanks @wiasliaw for the contribution.


Collection CYGNSS_L2_V3.1 failing to add to concise

https://github.com/podaac/concise/actions/runs/5659486246

@davidcolemanjpl please test HITIDE OPS and UAT

@davidcolemanjpl also the concise (for merging granules) version is shown in the nc file history_json.  look for concise

HiTIDE-UI (OPS): v4.16.0
HiTIDE Profile - v4.9.8
L2SS-Py - v2.5.0
CONCISE v0.6.1
HiTIDE-UI (UAT): v4.16.0-RC.6
HiTIDE Profile - v4.9.8-rc.1
L2SS-Py - v2.6.0rc8
CONCISE uat (v0.7.0a19)
CONCISE functionality passed in HITIDE OPS and UAT
Test Complete
Note:
Side Observation: CYGNSS_L2_V3.1 currently has no associated Thumbnail Images in OPS  / UAT


Add support for Thread Performance Checker scheme diagnostics option

Context 🕵♀
In Xcode 14 there's a new "Thread Performance" runtime checker that is enabled by default. This can be configured within the scheme's Run action (LaunchAction).

What 🌱
The new option appears in the LaunchAction only if disabled and omitted if enabled (similar to main thread checker).
    <LaunchAction
       buildConfiguration = "Debug"
       selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB"
       selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
       disableMainThreadChecker = "YES"
+      disablePerformanceAntipatternChecker = "YES"
       launchStyle = "0"
       useCustomWorkingDirectory = "NO"
       ignoresPersistentStateOnLaunch = "NO"
       debugDocumentVersioning = "YES"
       debugServiceExtension = "internal"
       allowLocationSimulation = "YES">
       <BuildableProductRunnable
          runnableDebuggingMode = "0">
          <BuildableReference
             BuildableIdentifier = "primary"
             BlueprintIdentifier = "FB2F4AA5286220B400DF0A62"
             BuildableName = "TemplateApp.app"
             BlueprintName = "TemplateApp"
             ReferencedContainer = "container:TemplateApp.xcodeproj">
          </BuildableReference>
       </BuildableProductRunnable>
    </LaunchAction>

Proposal 🎉

Introduce a disablePerformanceAntipatternChecker property on XCScheme.LaunchAction
It should be placed immediately after disableMainThreadChecker in the generated scheme

Notes

This option currently appears under the Run action (LaunchAction) only and not under the Test action.

Hi @kwridan!
I can tackle this one :)

That's great to hear @leszko11! Let us know if you need any help 🙂

Thanks @leszko11 !
Looks like @Bogdan-Belogurov has already submitted https://github.com/tuist/XcodeProj/pull/694


[Question] Purpose of no Cuda in examples

Hi,
I'm new here and just noticed there's a C++ example file, cpp/examples/basic_no_cuda.cpp, which uses std::vector instead of cudaMalloc. I’m curious about the benefit of doing so. Does it allocate the buffer in regular RAM instead of GPU memory? If so, what is the purpose of using Kvikio in this senario? And if not, what is the difference between CUDA and non-CUDA when using Kvikio?

Yes, the no-cuda example allocates in RAM (host memory). In fact, the example does not use the GPU at all (we should make that more clear).
The benefit of KvikIO in no-cuda mode is twofold. First it unifies the IO interface so that the same API can be used no matter the setup. Second, KvikIO will often outperform regular POSIX because of threading - try setting KVIKIO_NTHREADS to something greater than 1 (the default).

Thank you for the response, that's very clear.


audit: allowlist some formulae homepage cert issues

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

relates to https://github.com/Homebrew/homebrew-core/pull/60591

Do these all work in e.g. Safari?

yeah, it works on safari:

allowlist these formula would prevent this PR (https://github.com/Homebrew/homebrew-core/pull/60720), the installation from source still failed for dashing and etc.

actually this is the url,

we might need to bypass the check for rubydocs as well.

libpqxx also suffered the issue as well (https://github.com/Homebrew/homebrew-core/pull/61385)


Allow setting the fragmentation of packets option when sending a ping

Similar to how you can do:
ping -M do -s 1472 google.com

We should be able to tell pinger to not fragment packets if too large (due to MTU size).
The option seems to be available as https://pkg.go.dev/golang.org/x/net/ipv4#HeaderFlags (I think).

+1 This is useful for MTU tests

I need this as well. This can be set with a socket option. Additionally setting DF is not useful in and of itself if we cannot also set the size of the ICMP payload which as far as I can tell is currently not supported in this library. I have proposed a PR that adds some functions that are needed to support setting socket options (SO_MARK in this case) here: https://github.com/prometheus-community/pro-bing/pull/36
If this is accepted and merged I can re-use some of the code to do something like this which I believe will allow setting the DF bit:
syscall.SetsockoptInt(int(fd), syscall.IPPROTO_IP, syscall.IP_MTU_DISCOVER, syscall.IP_PMTUDISC_PROBE)

I'm happy to take a pass at this one.


Grammar fixes

#sign-off

@alvinmorales1 Can you take a look at this PR? Not sure the process to get it merged in this repo.

Ping @alvinmorales1.

#please-merge

@thecloudtaylor Could you merge this PR?


Default handler for XmlSlurper

Codecov Report

Merging #879 into 2.0.x will decrease coverage by 0.03%.
The diff coverage is 46.15%.

@@             Coverage Diff              @@
##              2.0.x     #879      +/-   ##
============================================
- Coverage     56.17%   56.13%   -0.04%
- Complexity     2188     2190       +2
============================================
  Files           229      229
  Lines         10395    10403       +8
  Branches       1447     1451       +4
============================================
+ Hits           5839     5840       +1
- Misses         3753     3756       +3
- Partials        803      807       +4

Impacted Files
Coverage Δ
Complexity Δ

...k/cloud/contract/verifier/util/ContentUtils.groovy
58.36% <37.5%> (-1.02%)
59 <1> (+1)

...ct/verifier/wiremock/WireMockToDslConverter.groovy
35.57% <60%> (+1.57%)
19 <1> (+1)
:arrow_up:

...loud/contract/stubrunner/TemporaryFileStorage.java
48.38% <0%> (-6.46%)
5% <0%> (-1%)

.../contract/stubrunner/ScmStubDownloaderBuilder.java
79.41% <0%> (-0.99%)
10% <0%> (ø)

...loud/contract/stubrunner/AvailablePortScanner.java
91.17% <0%> (+2.94%)
9% <0%> (ø)
:arrow_down:

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 e8f774f...d246fb7. Read the comment docs.

Codecov Report

Merging #879 into 2.0.x will decrease coverage by 0.03%.
The diff coverage is 46.15%.

@@             Coverage Diff              @@
##              2.0.x     #879      +/-   ##
============================================
- Coverage     56.17%   56.13%   -0.04%
- Complexity     2188     2190       +2
============================================
  Files           229      229
  Lines         10395    10403       +8
  Branches       1447     1451       +4
============================================
+ Hits           5839     5840       +1
- Misses         3753     3756       +3
- Partials        803      807       +4

Impacted Files
Coverage Δ
Complexity Δ

...k/cloud/contract/verifier/util/ContentUtils.groovy
58.36% <37.5%> (-1.02%)
59 <1> (+1)

...ct/verifier/wiremock/WireMockToDslConverter.groovy
35.57% <60%> (+1.57%)
19 <1> (+1)
:arrow_up:

...loud/contract/stubrunner/TemporaryFileStorage.java
48.38% <0%> (-6.46%)
5% <0%> (-1%)

.../contract/stubrunner/ScmStubDownloaderBuilder.java
79.41% <0%> (-0.99%)
10% <0%> (ø)

...loud/contract/stubrunner/AvailablePortScanner.java
91.17% <0%> (+2.94%)
9% <0%> (ø)
:arrow_down:

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 e8f774f...d246fb7. Read the comment docs.

@OlgaMaciaszek fixed comments but still can't have a successful build
[INFO] Spring Cloud Contract Docker ....................... FAILURE [01:47 min]
Last time Code Coverage was failing with a file I didn't even touch.

Have you rebased against 2.0.x ?

@marcingrzejszczak I will rebase and try again.


Feature/rxjs6 lint

Add linter for rxjs 6 to avoid common pitfalls

Probably waiting until after the big PRs are merged first

Added even more lint rules


Added moogladder test

Variables shouldn't be declared in the for loop. Declare them outside the for loop, then zero them inside the for loop.


rtorrent: broken after libressl bump

With the last libressl bump, rtorrent search for libssl.so.47 & libcrypto.so.45 and there is a libssl.so.48 & libcrypto.so.46
It can be solved with a simple ln -sf ... almost for me.
@jedavies-dev

https://github.com/kisslinux/community/issues/656#issuecomment-629761446

You just need to rebuild rtorrent. Looks like another package not bumped for the release (otherwise this would have automatically happened). Closing this as it isn't a bug per se.
@jedavies-dev Simply send a PR bumping the second field of the version file. No rush on this.

I can't rebuild rtorrent: log

It looks like you need to rebuild libtorrent and then rtorrent. @jedavies-dev

Thanks


Replace fcl-subscribe with fcl.events

https://docs.onflow.org/fcl/reference/api/#event-polling-utility

Does not

Support a fromBlockHeight range
Allow subscribing to multiple events


[BUG] Tweet button under Gita shlok generator does not paste correct text on Twitter

Description
Describe the bug:
The tweet button in the Gita Shlok generator does not paste the shlok onto the tweet when redirected to Twitter.
Expected behavior:
The expected behavior is that when redirected to twitter and the tweet shows up, it should already have the shlok copied onto the tweet.
Actual behavior:
Current behavior is that instead of copying the shlok, it copies "" on to the tweet
Reproduction Steps
Steps to reproduce the behavior:

Go to Gita Shlok Generator
Click on Tweet button
See the tweet text after being redirected to Twitter

Expected result:
At step 3, I expected to see the shlok ready to be tweeted
Actual result:
It just showed "" on the tweet
Screenshots

Additional Information
Device (please complete the following information):

OS: Linux
Browser Chrome
Version 123.0.6312.122 (Official Build) (64-bit)

Additional context:
Kindly assign this issue under my name with the relevant gssoc and level labels :)

Go for it!

thanks! on it


Update to fix documentation issue 2011

Closes https://github.com/cloudflare/terraform-provider-cloudflare/issues/2011

once you add that file back, run make docs locally and you should be good to go.


Use named records in data constructors for some syntax classes

With @W95Psp's enhancements to the records syntax, it has become easier to use named records in the types of the data constructors. This PR migrates term and sigelt' in FStar.Syntax.Syntax to use named records (https://github.com/FStarLang/FStar/pull/2928/files#diff-aa5ca827a66c86f3d7c8b7e782db082621512d6e1bc76bef33a3a8692c395b50).
It is an internal, syntactic change, no changes in the client code.

Very nice :)


Add a target to apply style

Add a target making it easier to apply style, just type make style and you code is styled

@tqchen


🌱 Merge branch 'production-site' into merge-main

@mdunbavan there is some diff between main and production-site. Should we remove these extra files from production-site or do these need to be merged into main as well? Want to make sure that both these branches stay in sync.

This looks like this may no be longer required, but feel free to reopen, if needed.


Automatically determine CLI version in sidecar and controller

Registry automatically determines the version indicated by the CLI using ./registry/utils/version.go. Controller and sidecar should do something similar, as the CLI version is continually out of sync with the actual version of the code.

It's also a good opportunity to extract the ./registry/utils/version.go code into a ./pkg/version package.


Strengthen root node selection deprecation

The original goal of this deprecation is to prevent the root node from being selected as a variant. This way, for a given resolution, a variant can either be intended for consumption as a variant, or act as a root node, but not both.
Previously, we detected this scenario by checking if there were any incoming edges to the root node by the time the graph was finished resolving.
We update this to trigger a deprecation if an incoming edge is ever added to the root node in any circumstance.
This is stronger than the original deprecation, since this also handles cases where edges temporarilly target the root node but are then later retargeted to another node for some reason.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


too many input argument

Hi Teun,
It seems that there is something wrong here. In this file, the function outlier_removal_class is called with three input arguments. However, the function seems to need only two and the first argument has been removed previously although in function description it is still there.
https://github.com/imphys/smlm_classification2d/blob/b5cbe60c9915d0b93c8c6c5456722bc31da9efd8/matlab_functions/reconstructPerClassFunction.m#L66
Could you please have a look at that?
Bests,
hamidreza

Hi Hamid,
Thank you for catching the bug! I have resolved the issue.
Please let me know whether it works for you now.
Best,
Teun


Error when trying to debug in VS Code with Python 2.6

Traceback (most recent call last):
File "c:\Users\jhill.BJTEST.vscode\extensions\ms-python.python-2019.6.22090\pythonFiles\ptvsd_launcher.py", line 21, in
import ptvsd
File "c:\Users\jhill.BJTEST.vscode\extensions\ms-python.python-2019.6.22090\pythonFiles\lib\python\ptvsd_init_.py", line 13, in
from .vendored import force_pydevd
File "c:\Users\jhill.BJTEST.vscode\extensions\ms-python.python-2019.6.22090\pythonFiles\lib\python\ptvsd_vendored_init.py", line 2, in
from importlib import import_module
ImportError: No module named importlib

Closing this issue as we do not (Unfortunately) support Python 2.6


Some maps are failing in high resolution displays

Context
When entering this map in a high resolution display (i.e: retina macbook) both the data and the label layer disappears. However, in other maps it works great (created after the high resolution changes).
Current Result

Expected result
Data and label layers must not disappear.
.carto file
AirBNB listings madrid (on 2018-03-21 at 18.36.09).carto.zip

This bug is already fixed for new basemaps, so, as a workaround, just change your basemap to a different one and back to the original and that should fix the issue. We are working in fixing this automatically for all affected users.

Should be fixed now


Fix standalone installation

When installing the package without installing the whole robingas repository, I noticed some missing dependencies.
In this pull request I fixed that and updated the docs. Basically a simple dependency was missing on the package.xml and the installation instructions did not tell that cloud_proc was required. Also, despite being obvious, the documentation didn't list PyTorch and torchvision as dependencies.
Worked in my case. Tested on a Docker container to be sure, and it looks like now it's working. Anyway I suggest someone to test on a clean setup if you wish to merge, just to be sure.

Thank you! Indeed, these dependencies are necessary.


Updating bundletool to version 1.0.0

@adborbas ready to merge?

Now that the CI has passed, yes. Thanks @lpusok !


MD5 returned by SQS does not match the calculation on the original request

I'm trying to use a java application with fake_sqs but am getting this:
Caused by: com.amazonaws.AmazonClientException: MD5 returned by SQS does not match the calculation on the original request. (MD5 calculated by the message attributes: "c3d29a3de5c6497a057496fd58a56e89", MD5 checksum returned: "null")
    at com.amazonaws.services.sqs.MessageMD5ChecksumHandler.sendMessageOperationMd5Check(MessageMD5ChecksumHandler.java:120)
    at com.amazonaws.services.sqs.MessageMD5ChecksumHandler.afterResponse(MessageMD5ChecksumHandler.java:80)
    at com.amazonaws.handlers.RequestHandler2Adaptor.afterResponse(RequestHandler2Adaptor.java:49)
    at com.amazonaws.http.AmazonHttpClient.afterResponse(AmazonHttpClient.java:416)
    at com.amazonaws.http.AmazonHttpClient.execute(AmazonHttpClient.java:322)
    at com.amazonaws.services.sqs.AmazonSQSClient.invoke(AmazonSQSClient.java:2419)
    at com.amazonaws.services.sqs.AmazonSQSClient.sendMessage(AmazonSQSClient.java:1015)
    at org.apache.camel.component.aws.sqs.SqsProducer.process(SqsProducer.java:58)

I see that one of the ruby examples uses "apiVersion: '2012-11-05'" - perhaps this is required for all clients?

Any movement on this?

I'm seeing the same issue, any updates?

Also have this problem.  fake_sqs doesn't return this at all:
 <SendMessageResponse>
    <SendMessageResult>
        <MD5OfMessageBody>740682d831a40200b5147a683bc1a7d7</MD5OfMessageBody>
        <MessageId>d7a3913a-8c91-4776-b061-dff1c4087a78</MessageId>
    </SendMessageResult>
    <ResponseMetadata>
        <RequestId>dea697d5-fbb5-4c3e-9692-ed27fa1a3d46</RequestId>
    </ResponseMetadata>
</SendMessageResponse>

And when getting the item, the attributes seem to be missing

Has this project has been abandoned?

@algorist No, patches are welcome if this is still an issue.

When is this going to be fixed? This is a critical bug. I cannot use fake-sqs to send events in my development whatsoever. The issue is open from November 2015 but still no changes. People have to look for alternatives.

Simply when someone decides to take time to fix it. I'm using fake_sqs without hitting this bug. A first good step would be to post code/versions here to re-produce the issue.

I'm in Python land, but I guess this might happen with newer versions of boto3 as well. Send message should send a correct MD5 back, but it obviously doesn't in this case:
https://github.com/iain/fake_sqs/blob/master/spec/acceptance/message_actions_spec.rb#L24
Again, if you want me to help you, I would appreciate a SDK version and a code snippet (even if just one line). This way I don't have to spend time, guessing versions, when I also have to setup a Java environment.

Does not complain latest boto3.
Fake:
{
    u 'MD5OfMessageBody': '444bcb3a3fcf8389296c49467f27e1d6', 'ResponseMetadata': {
        'RetryAttempts': 0,
        'HTTPStatusCode': 200,
        'RequestId': '465c6a9a-3b14-4007-a628-fce678fb4280',
        'HTTPHeaders': {
            'content-length': '306',
            'x-xss-protection': '1; mode=block',
            'x-content-type-options': 'nosniff',
            'server': 'WEBrick/1.3.1 (Ruby/2.3.7/2018-03-28)',
            'connection': 'Keep-Alive',
            'date': 'Mon, 10 Sep 2018 15:48:33 GMT',
            'x-frame-options': 'SAMEORIGIN',
            'content-type': 'text/html;charset=utf-8'
        }
    }, u 'MessageId': '30adc959-9e40-4b62-bfa0-bde8ab131501'
}

Real:
{
    u 'MD5OfMessageBody': '444bcb3a3fcf8389296c49467f27e1d6', 'ResponseMetadata': {
        'RetryAttempts': 0,
        'HTTPStatusCode': 200,
        'RequestId': '<obfuscated>',
        'HTTPHeaders': {
            'x-amzn-requestid': '<obfuscated>',
            'content-length': '378',
            'server': 'Server',
            'connection': 'keep-alive',
            'date': 'Mon, 10 Sep 2018 15:50:27 GMT',
            'content-type': 'text/xml'
        }
    }, u 'MessageId': '8f30bb07-cb92-4930-b6e0-ee97bf268ab0'
}

I am getting the same issue with aws-java-sdk-sqs 1.11.362 and fake_sqs 0.3.1

FYI:  I don't use this library, but I have been struggling with a similar error when using a java client.  (I found this by googling my error message).  It seems that the java client does two checksums.  One for the body and one for the attributes.  As I have seen this problem in multiple "SQS Testing Implementations", i'm guessing the java client is doing more work than others?  Anyway, calculating the MD5 on the attributes as well as the body seems to be the trick to making the java APIs happy.


ARROW-3527: [R] remove unused variables

I'm so excited about the Arrow R package! Started working with it tonight to see where I may be able to help.
While doing this, I noticed a few "unused variable" compiler warnings.
DataType.cpp:118:7: warning: unused variable 'n' [-Wunused-variable]
int n = x.size();

RecordBatch.cpp:132:7: warning: unused variable 'nc' [-Wunused-variable]
int nc = tbl.size();

Please consider this PR to remove the offending lines.

@romainfrancois in the C++ libraries we fail Travis CI for things like this. Could we do the same for R?

🤔 Not sure how to do it, but I've added -Wall locally so at least I'll see them. The PR looks good for the time being.


Running a new thread in Fiber.schedule block raises FiberError

Describe the bug
Snippet below fails with FiberError.
Fiber.set_scheduler AsyncScheduler::Scheduler.new
Fiber.schedule do
    Thread.new{}.join
end

To Reproduce
Open a console by bin/console, and then run below.
[1] pry(main)> Fiber.set_scheduler AsyncScheduler::Scheduler.new
=> #<AsyncScheduler::Scheduler:0x0000000102f239d0 @blockings=#<Set: {}>, @input_waitings={}, @output_waitings={}, @waitings={}>
[2] pry(main)> Fiber.schedule do
[2] pry(main)*   Thread.new{}.join
[2] pry(main)* end
#<Thread:0x00000001064c72a8 (pry):3 run> terminated with exception (report_on_exception is true):
/Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb:49:in `resume': attempt to resume a resumed fiber (double resume) (FiberError)
        from /Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb:49:in `unblock'
FiberError: attempt to resume a resumed fiber (double resume)
from /Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb:49:in `resume'

Expected behavior
It does not raise an error.

I have found the reason.
debug-issues#31
TL; DR
FiberScheduler registered in the parent thread is somehow inherited to the child thread.
parent thread (with scheduler) ---> create a fIber instance ----> fiber.resume ---> <blocked> ---> resume again
                                                                                \----> child thread (with scheduler)  ---> #join

bundle exec ruby test.rb                                         ✔ │ 06:07:27
#<Thread:0x00000001033f9630 test.rb:10 run> terminated with exception (report_on_exception is true):
/Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb:60:in `resume': attempt to resume a resumed fiber (double resume) (FiberError)
        from /Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb:60:in `unblock'

#########################
{:parent_thread_id=>1400, :child_thread_id=>1460}

#########################
{:parent_scheduler_id=>1420, :child_scheduler_id=>8}

#########################
resumed fiber (id=1440) at /Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb32 inside of Scheduler (id=1420) in Thread (id=1400)
1460
resumed fiber (id=1440) at /Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb59 inside of Scheduler (id=1420) in Thread (id=1460)

I have found the reason.
TL; DR
FiberScheduler registered in the parent thread is somehow inherited to the child thread.
Reproduction with debugging logs

Checkout to 12bdeaa2461cdde3fcf5283ea10d4c785cf0ef29
Run $ bundle exec ruby test.rb

This stdouts as blow.
#<Thread:0x000000010f3d3de8 test.rb:10 run> terminated with exception (report_on_exception is true):
/Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb:59:in `resume': attempt to resume a resumed fiber (double resume) (FiberError)
        from /Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb:59:in `unblock'

### thread_ids ####
{:parent_thread_id=>1400, :child_thread_id=>1480}

### scheduler_ids ####
{:parent_scheduler_id=>1420, :child_scheduler_id=>8}  ### memo: 8 is an object_id of nil

### All points where tasks are resumed ####
From now, resume a fiber (id=1440) at /Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb#32 inside of Scheduler (id=1420) in Thread (id=1400). Currently in the fiber (id=1460). Callers are: ["test.rb:9:in `schedule'", "test.rb:9:in `<main>'"]
From now, resume a fiber (id=1440) at /Users/daiki-kudo/repos/async_scheduler/lib/async_scheduler/scheduler.rb#58 inside of Scheduler (id=1420) in Thread (id=1480). Currently in the fiber (id=1500). Callers are: []

My understandings

A parent thread (id=1400) starts. This thread holds a FiberScheduler (id=1420).
In the parent thread, Fiber instance (id=1440) is created, and executed until it is blocked somewhere around Thread.new{}.join.
In the parent thread, all the lines after Fiber.schedule are executed. Obviously there is no line in this case.
In the parent thread, the parent thread's FiberScheduler#close is called. This time the pending Fiber task is obviously the one (id=1440), so the parent thread executes this task again in the event loop. Note that the task has not been removed yet from @blockings in the scheduler.
In the task (id=1440), it calls Thread#join method. This forces the child thread to be finished, so the execution of the child thread begins. The main Fiber in this child thread is probably id=1500, and this Fiber is under the one (id=1440) 🟥.
The child thread is executed until the end. This time there's no line.
The child thread seems to share the FiberScheduler (id=1420) with the parent thread and at this moment, the child thread's FiberScheduler#close seems to be called. This executes the task (id=1440) which is still held as @blockings. As written in 🟥, it is calling the task itself, so it throws FiberError (double resume).

How to fix this issue
Shortly saying, the current version (e54d9aabf1176a9433a7a94534b057250ceb771b) of AsyncScheduler is not thread safe.

For the short term solution, patch the scheduler to raise an error when it is used in multithread programming.
For the long term solution, make the scheduler to be thread safe.


GNOME updates 2023-05-01 (staging)

Description of changes
(I guess we can ignore some docs build change for tracker since we use release tarballs :thinking:)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build tracker

@ofborg build tracker
@ofborg build tracker-miners

https://gitlab.gnome.org/GNOME/tracker/-/issues/402. I think I saw this in https://github.com/NixOS/nixpkgs/pull/228922 and will probably not wait for a fix :thinking:

* bobby285271 stops thinking.
(For the record I built gnome-music on 8a32fa2e4f625d9f114fe7d768223e184286f81a using ofborg, so mass breakage shouldn't happen)
@ofborg build tracker
@ofborg build pkgsi686Linux.tracker


Issue 380 - Empty string as a valid value for empty list

As stated in #380 by @michellemay (colleague of mine), it is currently impossible to override a config value with an empty list using standard command line arguments (e.g. -Dfoo=).
This pull request attempts to solve the problem by allowing empty strings as valid values for lists and by converting them to empty lists.
This means foo="" would become valid for all getList methods and all would return an empty list of appropriate type.
It would also be possible to override a config value with an empty list using command line arguments like so: -Dfoo= .

Good!  Thanks Philippe

Thanks for working on this!
I think a helpful test to add would be one that does concatenation or merging between an empty string and a list. Like
foo = ""
bar = ${foo} [ 1, 2, 3 ]

or perhaps
foo = ""
bar += 3

I'm not sure it's easy to make this work, but whatever it does we should ensure the test verify what it does.

I'm not sure whether/how we handle that for the pretend-a-numeric-indexed-object-is-a-list case, either. But if we do handle it for that, we could do the same here.

Thank you for your comments!
I managed to make pretty much everything you said work (concatenation with empty strings for lists and indexed objects).
For indexed objects, I had to make it so that foo="" is also a valid for empty non-indexed objects, which I thought made sense anyway. That way, all objects can also be overridden with command line parameters using the same syntax as for lists.
There is a few problems I ran into which make me think it may not be realistic, after all, to implement such feature and to have consistent behavior across all existing features.
The way I got it to work normally is, when a config value is parsed, as soon as there is one object or list inside the value, all unquoted whitespace is removed so that it's not joined to empty strings, as it would be, normally, for string values.
That way, there is no ambiguity in ConfigConcatenation#join between :
foo=""
bar=${foo} [1, 2] (with whitespace)

and
foo=" "
bar=${foo}[1, 2] (no whitespace)

Here is one problem :
foo=""
bar=""
x=${foo} ${bar} (with unquoted whitespace)

Since there is no object or list, I have no way of knowing that I should ignore unquoted whitespace.
Hence, x gets a values of x=" " (single space).
Then if I call getList on that, I obviously get a WrongType exception, and I doubt we would want to accept quoted whitespace as valid list/object values.
There is also some unexpected behavior which may come up such as:
foo=""
bar=[${foo}, [1], [1]]

You may want to have bar=[ [], [1], [1] ] but we can't really know that.
So, of course you get bar=[ [""], [1], [1] ].
So, I'm no sure we can make this work seamlessly anymore.

An idea could possibly be to have reserved string symbols for empty arrays/objects. Something like foo="[]" and bar="{}" and to parse that right upfront as List or Object, but of course these are also valid string values.

Thanks for digging in - sometimes the complexities aren't obvious up front!
Idea I just had that could solve this and some similar issues; what about a thing like this:
java -Dconfig.literal="foo.bar=[]\nblah={}"

Where we take the value of config.literal, parse it as a config file (it can be a whole bunch of stuff in there), and insert it in the config stack at the same time we insert system properties (probably properties should override the stuff from config.literal, but presumably that case wouldn't arise often).
There might be a better name than config.literal ... config.insert, config.extra, I don't know.
This would not be analogous to -Dconfig.file because it wouldn't replace application.conf, it would be in ConfigFactory.defaultOverrides() instead.
Using this is a little bit annoying because the quoting rules are double-layered; you have shell quoting followed by JSON quoting. But at least it would make it possible to insert any and all config values on the command line, and even do things like -Dconfig.literal='foo.bar += 42' so you could append values to a list for example.

maybe config.overrides as the name...


Partial representations from meshes or point clouds

Hello everyone,
I am trying to implement a shape completion encoder-decoder with Kaolin. Since the premise of shape completion is to have as input an incomplete point cloud, mesh or voxel grid, I wonder if there if I could adapt any of the existing kaolin.transforms to obtain this or I need to implement it from the ground up.
The idea is to be able to load examples from ModelNet or ShapeNet as partial representations in order to use them as input to the encoder.

Closing an old issue, as it's no longer relevant to the library.


Responsible AI Service is way to sensitive

Responsible AI Service is way to sensitive.
I was working with an Arduino and making some code for it. I asked copilot this question:
Can you write an explanation for the following connections on my D1 mini please:

TX, RX, 5 SCL, 4 SDA, 0, 2, GND, VBUS, RST, AO, 16, SCK 14, MISO 12, MCSI 13, SS 15, 3V3

Which may seem totaly innocent at first glance, but Responsible AI Services was smart enough to see that I had "SS" in the request meaning that I was up to no good and probably a nazi. Responsible AI Services did the right thing and blocked my horrible and dangerous request.
I eventually figured that the "SS"  might be the problem so I asked this instead:
Can you write an explanation for the following connections on my D1 mini please:

TX, RX, 5 SCL, 4 SDA, 0, 2, GND, VBUS, RST, AO, 16, SCK 14, MISO 12, MCSI 13, 15, 3V3

And this was accepted. I hope that whoever is responsible for Responsible AI Services can adjust their regex pattern so that you don't get flagged for merely including "SS"  in a question.
Thanks for reading

That "responsible AI service" is just some stakeholders at github trying to exist. It is a very good argument to try other, better tuned, ai copilot.

@Kireobat i'm having trouble reproducing this issue. when i use the same prompt, i get a response back from copilot chat in vs code. are you still having this problem with the same prompt?
thank you

@thispaul I just started a new chat and copy and pasted that prompt

I still have the issue when including the "SS"

Thanks @Kireobat we are troubleshooting this with the RAI team

I can confirm that my prompt "Please put the elapsed time in hh:mm:ss format" was also flagged.

I have the same issue but for another prompt. However, I haven't found the reason why. Especially as I entered a prompt in German, where I haven't anything special.
Therefore, is there the possibility to get the reason, why the information was filtered, to troubleshoot the process myself?

flagging double "s" is ridiculous. Mine got flagged for "killTask" which is a function in a library.

+1 - trying to have it evaulate any JSON is a nightmare given the random characters that might hurt its feelings... makes it unusable.

Thanks for creating this issue! We figured it's covering the same as another one we already have: https://github.com/microsoft/vscode-copilot-release/issues/2625. Please refer to that one for updates and discussions. Feel free to open a new issue if you think this is a different problem.


linux: remove hack for old kernels

Description of changes
The oldest kernel supported in Nixpkgs is now 4.19 LTS, so we don't need this any more.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Building 6.5.7-hardened1

Had a conflict, there was a related change on staging that hasn't made it to staging/next: https://github.com/NixOS/nixpkgs/pull/245968

Yeah, that's why this PR targets staging.

Ah right, missed that and was using git am. Already passed the config phase and finishing up compilation.


Missing ada

It says my real balance is **** next to my ada on my nami wallet. not sure what it means but how do i get the ada back into my wallet?

there was a random transaction of 554 ada. the transaction id was 1bd607deb60f3f82adebb5b38b37f3a2c69f91dbde5227776a15b807420b70ab

What do you mean with random transaction? Did you not initiate that one?

On Thu, 27 Jan 2022 at 6:31 pm, alessandrokonrad @.***>
wrote:

What do you mean with random transaction? Did you not initiate that one?
—
Reply to this email directly, view it on GitHub
https://github.com/Berry-Pool/nami-wallet/issues/314#issuecomment-1022923353,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXPRF22ZD6GCD7NPZPIBT4LUYDYGVANCNFSM5M4YNIIQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I never sent 554 Ada to anything. At the time I was minting a cnft. In the
transaction it says I sent the 60ada which was for the mint, but there was
an extra 494ada sent. U can see it through the transaction id.

The transaction:
https://cardanoscan.io/transaction/1bd607deb60f3f82adebb5b38b37f3a2c69f91dbde5227776a15b807420b70ab
If you click on the incoming and the additional outgoing address, you see that they belong to the same stake key, to the same wallet. It's just change coming back. That happens in almost all Cardano transactions.
This cannot be a transaction initiated from Nami, because Nami only sees and uses the first address of your wallet. That's why Nami now shows a wrong balance, but their should be an “i” next to the balance that opens a popup informing you about the real balance.
Unfortunately, most other wallet apps than Nami will move ADA away from this first address and so are hard to use in parallel.
ccvault.io has a “Single address mode” in the settings that is specifically for behaving nicely in cooperation with Nami. To see the funds again, you will have to move them to the first address, the one given to you by Nami as receive address from the other wallet app you are using.

so does that mean its going to come back into the wallet eventually. it is linked to my ccvualt wallet i think thatss where the issue wouldve come from.

No, it is in your wallet. ccvault should show the right balance.
It is just not, where Nami sees it, on the first address. And it will not go back to that one automatically eventually.
What I would do:

In ccvault, go to settings. Enable “Single address mode (for Nami / CNTools)” and “Enable "Send All"”.
In Nami, copy the receive address.
In ccvault, go to send, paste the receive address (Be sure, it is the Nami receive address!), select “Send All” and confirm the sending. You are sending everything to yourself, basically, but to the address that Nami can see.
Go to settings and disable “Enable "Send All"” again. It's dangerous. There's a reason it's disabled by default.
In Nami, the whole funds should reappear, when the transaction has gone through.

If you wanted to have collateral set (your transaction looks like it used ADA set aside for collateral):

In Nami, set the collateral again.
In ccvault, go to settings. Enable “Collateral”. It should find and select the collateral set aside by Nami, automatically. And it will not use/move away that collateral again.

ok will do, why is send all dangerous? is there a chance it wont work

The developers of ccvault say: “Sending all Ada and tokens is considered dangerous as you might accidentally send away your whole wallet content. Only enable this feature temporally and when you need it.”
So, it's a precaution to accidentally hit that button and send everything you have to the wrong address.

okay thanks will do it now

it worked thanks alot

hello i know this thread has been awhile but i have been meeting the same issues and cannot be solved,
can someone help me please


Extra spaces between ?? operator in *.js code

Description
The results of beautification are not what I expect.
Input Before Beautification
This is what the code looked like before:
this.behaviours.rejectGroupInvites = options.RejectGroupInvites ?? false;

Expected Output
The beautified code should remains untouched and should have looked like this:
this.behaviours.rejectGroupInvites = options.RejectGroupInvites ?? false;

Actual Output
The beautified code actually looked like this:
this.behaviours.rejectGroupInvites = options.RejectGroupInvites ? ? false;

Causing the following error while executing the code on Node v14:
/home/nolddor/gems-steam-bot/classes/Bot.js:37
        this.behaviours.rejectGroupInvites = options.RejectGroupInvites ? ? false;
                                                                          ^

SyntaxError: Unexpected token '?'

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: N/A
Checklist
I have:

[Y] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[Y] Reloaded (or restarted) Atom to ensure it is not a caching issue
[Y] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[Y] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[N] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

I'm guessing this is the same issue as the one I came to report, which is that when I try to say something like
myvar ||= "default";
it becomes
myvar || = "default;
which is a JS syntax error.

A similar problem arises with EJS code that are inside a script tag:
<script type="text/javascript">
   initStatisticsTable( <%- JSON.stringify(MODULE_SCORE) %> );
</script>

becomes
<script type="text/javascript">
  initStatisticsTable( < % -JSON.stringify(MODULE_SCORE) % > );
</script>

Added spaces around the percent sign and removed a space between the introductory tag and the first word. This breaks the code.

If you still need this and are looking for something to fill the spot that Atom left, I am using Pulsar the successor to Atom...   This package is working except for one deprecated error listed in #2605
If you don't need this anymore, can you please close this out so it's not just sitting there? Thanks

Rest in peace...


Keyboard display issues

On my device I often find the keyboard behaviour irritating:

While typing I realize that I am in the wrong language. I switch and the keyboard disappears although I am not done typing. (reproducible)
On the other hand sometimes the keyboard unnecessarily pops up over dictionary list hiding the dictionaries in the lower part (not reproducible)

On the first one: On the other hand, for people who finish typing and then switch the language the current behaviour is more convenient.
What do you think about making it dependent on whether an exact match for the word is found?
It won't be 100% (for example short words likely would have a match that isn't the word meant, so maybe there should be a minimum length). I could imagine it to either be quite convenient or really annoying because of the inconsistency...
If you have any additional information on the second one, like in which circumstances it happens most, that would be nice to know.

Hmm, I disagree on the first one. I usually don't type out the whole word, only as much as necessary to make the desired word show up. That's why I switch the language midway. For that reason your fix wouldn't help until I get used to typing more.
Also making the keyboard disappear is less necessary (I can still see the result) and quicker (slow phone) than making it reappear.
I'll keep an eye on the 2nd one.

That's why I switch the language midway. For that reason your fix wouldn't help until I get used to typing more.

Sorry? My proposal would NOT hide the keyboard UNLESS whatever you typed matches a word exactly.
So if you typed "tes" it would keep the keyboard, if it was "test" it would hide it.
That said you're probably right it might be better to just keep it.

Sorry, I didn't read your proposal carefully.
But still, I want to look up "testicles" and suddenly, while typing, the keyboard disappears... which makes me tap something unintentionally. (OK, not a problem here, since tapping an entry doesn't do anything, but this is my favourite. In the old QuickDict. the dictionary list often jumped around just when I tapped a flag, which made me tap the wrong flag. Good riddance this was fixed.)

fixed in 30277f5ad1cd198460db6df61698d37bc761f1e5

To keep things less confusing, I suggest you open a new ticket if you have any further info/hints on 2).
One ticket per issue works better for me :)