Add user resources (dependency injection) and finalizers

Adds finalizers and user defined resources. Finalizers will run at the
end of the Context where they were specified and in between retries.

Codecov Report
Attention: Patch coverage is 87.09677% with 4 lines in your changes missing coverage. Please review.

Project coverage is 81.51%. Comparing base (3110fa2) to head (7e12f6e).

Files
Patch %
Lines

lib/resonate.ts
85.18%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #135      +/-   ##
==========================================
- Coverage   81.54%   81.51%   -0.03%
==========================================
  Files          17       17
  Lines        1138     1158      +20
  Branches      315      344      +29
==========================================
+ Hits          928      944      +16
+ Misses        210      207       -3
- Partials        0        7       +7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Replace Card's Buttons with Links

Which USWDS component(s) does this relate to?
Card
What USWDS Version was this component introduced by?
3.5.0 (missed in #2682)
Describe the bug
Replace Buttons with Links using theusa-button class, and be sure spacebar activation works, per https://github.com/uswds/uswds/pull/4385
Code snippet
Before:
<Card>
  ...
  <CardFooter>
    <Button type="button" secondary>
      Stick out
    </Button>
  </CardFooter>
</Card>

After:
<Card>
  ...
  <CardFooter>
    <Link href={"#"} className="usa-button">
      Stick out
    </Link>
  </CardFooter>
</Card>

Further detail
https://trussworks.github.io/react-uswds/?path=/docs/components-typography-link--default#styled-as-button

May I take up this issue?

Absolutely @devesh-2002, thanks!

I just replaced the Buttons with Links (on Card) , but to me it seems like the spacebar activation is not working. Am I going wrong anywhere?
https://drive.google.com/file/d/1tDZbH295XVppYluXaijjJ2_kMZsuWUoY/view?usp=sharing

@devesh-2002 can you please make a PR and I'll take a look at it? You'll need to implement this function, as space doesn't activate links by default.


Fixed image upload fail

If hostname == wx2.qq.com
The file host should be "file2.wx.qq.com"

Hi @nothingisover,
Thanks for catching the bug and send this PR!
However, it looks that @mukaiu has a better PR #415  which will be more reliable, so I'll merge #415 instead of this one.
Thank you again for your contribution, have a good day!


Refactor specification to incorporate gittuf design

I'm updating the spec document to incorporate the design. It includes the changes discussed in #13 and #15 and should guide the development of the implementation as we move away from the current prototype.

I've moved the SHA-256 bits to a standalone document. I see this going forward as:

primary specification that handles the TUF access control policies and corresponding workflows
other accompanying documents that describe extensions to this specification for hash algorithm agility, attestations, anything else we want to add

I believe the specification document is ready for reviews / discussions. We should tackle the SHA-256 bits separately.


Namespace overlap for populations.parameters

Issue:
Variables of the same name in different mechanism files cannot be modified/varied unless they are connections. E.g. variable named v_shift in multiple mechanisms cannot be varied. It would be nice to be able to specify the mechanism name with the parameter name in case of namespace conflict.
Solution:
ImportModel should be modified to enable namespace specification in populations.parameters.

furthermore, two connection mechanisms in the same connection also have this namespace issue.

61be49b
hacked a namespacing method.  Use the internal namespacing.
Example:
So if you have E cells with mechanisms Kleak.mech and Leak.mech, but both having gLeak parameter, you can access them individually using:
E_Kleak_gLeak
E_Leak_gLeak
If you just write gLeak, it will modify both mechanisms, which can also be a useful behavior at times.


Wrong example on "Persistent and Orphaned Disks"

this is not "jobs".  I think that the correct is "instance_groups". right?
https://github.com/cloudfoundry/docs-bosh/blob/master/persistent-disks.html.md.erb#L43

You're right, these job references should be updated to instance group.
Historically, "instance groups" were called "jobs"... there are some old references like this which need to be fixed. Thanks for pointing it out.

@dpb587-pivotal
oh! I didn't know the historical background.
thank you.


Calculating Supply Venus Apy

Hi,
What is the formula for calculating the supply and borrow Venus APY for each market (vToken)?
The formula which I found in the docs does only calculate the  supply Apy and borrow Apy of the vToken but not incl. the XVS APY.
Thank you!

const underlyingDecimals = vTokenToGetVenusApy.underlying.decimal;
const vTokenDecimals = 8; // always 8
const comptroller;
const vToken;
const apxBlockSpeedInSeconds = 3;

  let venusSpeed = comptroller.venusSpeeds(vToken);
  let xvsPrice = need to get xvsPrice in oracle;
  let totalBorrows = vToken.totalBorrowsCurrent();
  let totalSupply = vToken.totalSupply();
  let exchangeRate = vToken.exchangeRateCurrent();

  // Total supply needs to be converted from vTokens
  const mantissa = 18 + parseInt(underlyingDecimals) - vTokenDecimals;
  exchangeRate = +exchangeRate.toString() / Math.pow(10, mantissa);

  venusSpeed = venusSpeed / 1e18; // XVS has 18 decimal places
  venusPrice = venusPrice / 1e18;  // price feed is USD price with 18 decimal places
  totalBorrows = +totalBorrows.toString() / (Math.pow(10, underlyingDecimals));
  totalSupply = (+totalSupply.toString() * exchangeRate) / (Math.pow(10, underlyingDecimals));

  const venusPerDay = venusSpeed * parseInt((60 * 60 * 24) / apxBlockSpeedInSeconds);

  const venusBorrowApy = 100 * (Math.pow((1 + (venusPrice * venusPerDay / totalBorrows)), 365) - 1);
  const venusSupplyApy = 100 * (Math.pow((1 + (venusPrice * venusPerDay / totalSupply)), 365) - 1);

  console.log('Venus Borrow APY %:', venusBorrowApy);
  console.log(Venus Supply APY %:', venusSupplyApy);

Please refere this code

This is very helpfull. Thenk you so much!

Hi again,
I tried the provided code but the results are wrong.
For example, with the above code and provinded information I get the following results for vBNB:
--- original values---
underlyingDecimals: 18
vTokenDecimals: 8
venusSpeed:<PHONE_NUMBER>4177580
venusPrice:<PHONE_NUMBER>000000000
totalBorrows:<PHONE_NUMBER>06861362380971
totalSupply:<PHONE_NUMBER>860885
exchangeRate:<PHONE_NUMBER>31430868798509334
------calculation----
totalSupplyTimesExchangeRate: 0.00017980968664005922
venusPerDay: 2002.1680677363145
venusBorrowApy: 3275.793821046289
venusSupplyApy: Infinity
Expected result (based on https://api.venus.io/api/governance/venus) should be:
"venusBorrowApy": "13.40595320365917853",
"venusSupplyApy": "5.648654352585174061"
I must be doing something wrong, I just dont know what.
I would really appreciate if you could have a look at it.

This was very helpful, one small comment is that the actual formula is
underlyingPrice = underlyingPrice / 1e18;  // price feed of the underlyingToken in USD with 18 decimal places

const venusBorrowApy = 100 * (Math.pow((1 + (venusPrice * venusPerDay / (totalBorrows * underlyingPrice)), 365) - 1);


Question: How to let istanbul coverage ignore folders?

Since the slack link is not working I am asking my question here.
I am trying to force istanbul coverage reporter to ignore some folders.
For example, I would like to ignore ./src/app/tests folder.
What I tried so far is to add the path in config/webpack.test.js:
//**
 * Instruments JS files with Istanbul for subsequent code coverage reporting.
 * Instrument only testing sources.
 *
 * See: https://github.com/deepsweet/istanbul-instrumenter-loader
 */
{
  enforce: 'post',
  test: /\.(js|ts)$/,
  loader: 'istanbul-instrumenter-loader',
  include: helpers.root('src'),
  exclude: [
    '/tests/',
    /\.(e2e|spec)\.ts$/,
    /node_modules/
  ]
}

I thought it would be as simple as adding '/tests/' to exclude section, however it was not.
I also tried '/tests/**/*', '/src/app/tests/' and some other variations.
Any suggestions?

Hi,
I use nyc for the istanbul coverage, it let you do this into your package.json:
"nyc": {
    "exclude": [
      "folder1/file.js",
      "folder1/subfolder",
      "folder2"
    ]
  }

Have you tried without any / ?

Hi,
yes I tried without any / but it did not work.
I have had a look at nyc but I do not exactly understand what the benefit should be over using the already integrated istanbul-instrumenter-loader ;)

You run your tests from npm or yarn, I don't see why you use a loader for istanbul, nyc and istanbul works fine in cli ...
What I see is that without the webpack loader and launching from the cli, the result is the same, but it works in that case :smile:

@SherlockStd I have had a second look at nyc yet. I understand now what the benefits are. nyc provides a wide range of easily applicable settings. I will use nyc from now on and recommend others to do so if problems with istanbul configurations occur.
Thanks @SherlockStd. :)

@andorfermichael always a please :smile:


@conda-forge-admin, please add bot automerge

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/edrixs-feedstock#3.


Kernel fails to start

OS: Win11x64
Python: 3.11
Language: SWI-ProLog
Hey there! I have tried using your kernel to no avail so far. I have followed you guide of installing the kernel (well, all two lines of it) and expected it to work, or at least load. This is however not the case: Whenever I try to start up the kernel, I get the message: FileNotFoundError: [WinError 2] The system cannot find the file specified. in , line 1538, in _execute_child     hp, ht, pid, tid = _winapi.CreateProcess(executable, args,. Have I missed any important steps? SWI-Prolog is installed.

I got it - the OTHER powershell has to be installed as well, which you can do by winget install --id Microsoft.PowerShell --source winget


Unable to use tflite java on macos

Click to expand!
Issue Type
Feature Request
Source
source
Tensorflow Version
2.9.1
Custom Code
No
OS Platform and Distribution
MacOS
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Currently TFLite on Maven only distributes .so files for JNI which are not usable on macos (as it's not a mach-o file). I am trying to run some tests on a library I'm working on outside of the android emulator, but can't run the tests due to the lack of .dylib files. Is it possible to add a configuration for mac builds or possibly add a .jar artifact to maven with .dylib files for jni?

Standalone code to reproduce the issue
Built it locally and tried to run tests with JUnit.

Relevant log output
No response

Any update?

Hi there! I'm following up on this issue! We're looking for .dylib files for Android development unit testing

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The TFLite team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the TFLite space.
Thanks.


feat: include PkgPath information in image cve list and list export in zui

What type of PR is this?
feature
Which issue does this PR fix:
Partially addresses https://github.com/project-zot/zot/issues/2175
What does this PR do / Why do we need it:
This PR displays the Package Path information for the package list for a given CVE in the vulnerabilities list.
Since there is more data being displayed, this PR also brings in a change to display this information in the form of a card that is vertically arranged.
Testing done on this change:
Screenshots:

CSV export:
id,severity,title,description,reference,packageName,packagePath,packageInstalledVersion,packageFixedVersion
CVE-2016-1000027,CRITICAL,spring: HttpInvokerServiceExporter readRemoteInvocation method untrusted java deserialization,"Pivotal Spring Framework through 5.3.16 suffers from a potential remote code execution (RCE) issue if used for Java deserialization of untrusted data. Depending on how the library is implemented within a product, this issue may or not occur, and authentication may be required. NOTE: the vendor's position is that untrusted data is not an intended use case. The product's behavior will not be changed because some users rely on deserialization of trusted data.",https://avd.aquasec.com/nvd/cve-2016-1000027,org.springframework:spring-web,usr/local/artifacts/spring-web-5.3.31.jar,5.3.31,6.0.0

XLSX export:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Ideally, this should not break upgrades or downgrades as the older graphQL query should continue working just fine as well as the updated query.
No, updating a running cluster has not been tested.
Does this change require updates to the CNI daemonset config files to work?:
N/A
Does this PR introduce any user-facing change?:
Yes
The package list for a given CVE is now displayed in the form of vertically arranged cards.
Additionally, the package path for a given CVE is also displayed.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @andaaron , @rchincha , the PR is ready for review. Looking forward to your feedback and suggestions. Thank you!

Thank you @vrajashkr for the PR!
In addition to the code comments.
Design-wise I'm not necessarily the biggest fan of the extra nested card. I feel like maybe a section for each Package with the Package name as a heading would be cleaner imo.
Also a point that we should consider regarding mobile view is if this additional information is something we consider useful in that scenario. We should just hide out package information that would not be useful for a mobile viewer
Up to @rchincha on that point.

@raulkele , thanks a lot for the feedback! I'll address the review comments.

I feel like maybe a section for each Package with the Package name as a heading would be cleaner imo

This sounds like a good idea! I'll give it a try to see how it looks.

I made an attempt at trying out the sections as suggested. Here's what it looks like:

@raulkele, what do you think about this design?

IMHO the earlier implementation in the description looks better (but I'm not a front-end developer so I don't know what to say about how the code is written behind the scenes).

Personally, I'm torn between the two. I can't seem to settle on one 😅.
One key item here (at least in my opinion) appears to be the need to handle a variation in the length of the package name and the package path.
In both approaches, what stands out to me is that there's a lot of horizontal space that's not being used. For instance, the python vulnerability has no paths and the library name and versions are short so we could ideally have 3 cards in the same line on wide screens and they can automatically scale to 2 on a line or 1 per line on smaller mobile screens. However, when there's a long path like in the spring web one, it needs to take up the full width on its own.
The counter is that while this may lead to better use of horizontal space, it becomes somewhat inconsistent as some are multi per line and others are single per line depending on the package path length, so I'm not sure if it's such a good idea either.
Any thoughts @raulkele @andaaron ?

Design-wise I think I prefer the second implementation. I don't think it is an issue on larger screens that the row is empty if the path is not specified, we have examples where we do that in the ui. I'll take a look at the code again as well.

Thanks for the inputs @raulkele ! I've updated this PR to address the review comments so it's green. Any further feedback is welcome!
For the section approach, I'll raise a new PR.

Design-wise I think I prefer the second implementation. I don't think it is an issue on larger screens that the row is empty if the path is not specified, we have examples where we do that in the ui.

Maybe hide those rows entirely? Do not show both "Package Path" and the value?

Maybe hide those rows entirely? Do not show both "Package Path" and the value?

One thought about this that I had was related to consistency. For "Fixed Version", ZUI shows "Not Specified".
Displaying "Package Path" as "Not Specified" would help to keep it consistent so the user knows what to expect in this case.

We decided to use https://github.com/project-zot/zui/pull/428 instead.


Running some tasks only once

Some periodic task should run only once at a time. For example tindex_cached_layers
Here there is some good hint to get this done:
https://stackoverflow.com/questions/4095940/running-unique-tasks-with-celery

@capooti  Is the purpose to index cached layers one by one? I mean, the goal is to avoid search engine saturation, right?
If so, what about bulk indexing?

Yes, I am bulk indexing. In production that task is scheduled every 2 minutes. When checking services with many layers cache will be full of layers to index and it can happen that in 2 minutes that task is not completed, and there will be more ones running at the same time which can cause problems to solr


Re-enable Tests on Travis

This commit will re-enable tests to run on Travis.
Changes:

Broke down the build steps to stop Travis from timing out
Added testing back!
Enabled openjdk8 testing on Linux (we do not use the container infrastructure any more, this will slow down initial builds because dependency caching is not enabled)

This does not address #591.

@aforward / @TimLethbridge: do we think openjdk8 testing is more important than faster builds?

@TimLethbridge Build now passes, fixed the jdk_switcher issue again.

Great. I merge the pull request
I notice that not all the tests are running on travis.
template.test is running but the following ant cll

is not being done. This is part of the full build and requires running php, java and ruby, so from what I read it couldn’t be tested on
Travis OSX, but could be tested on Linux. What do you think? We will need to keep CruiseControl until this is being run.
Also I notice that Travis is still running tests on old branches that were long ago merged. See
https://travis-ci.org/umple/umple https://travis-ci.org/umple/umple
Yet Travis seems to have a different idea what is an active branch as compared to Github
https://github.com/umple/umple/branches/active https://github.com/umple/umple/branches/active
Any idea what is going on.
Thanks for all your help.
Tim

On Dec 4, 2015, at 15:27, Kevin Brightwell<EMAIL_ADDRESS>wrote:
@TimLethbridge https://github.com/TimLethbridge Build now passes, fixed the jdk_switcher issue again.
—
Reply to this email directly or view it on GitHub https://github.com/umple/umple/pull/665#issuecomment-162072977.

Timothy C. Lethbridge, PhD, P.Eng., I.S.P., CSDP
Professor of Software Engineering and Computer Science
Faculté de genie / Faculty of Engineering
Université d'Ottawa / University of Ottawa
Tel:<PHONE_NUMBER>x6685   Fax:<PHONE_NUMBER>    Mobile:<PHONE_NUMBER>
http://www.site.uottawa.ca/~tcl

I missed a testing call! I'll add it, and will attribute it to #616.

In regards to old branches, this was because I removed some of the old merged branches and accidentally removed some I didn't mean to. So when they were re-added, I guess Travis picked up that they changed?
Once a branch is merged, it should be removed, but I didn't want to remove anything active in the past month.


Notification Popup Cuts Off Content

Issue Type: Bug

Click on the bell icon at the bottom right
If no notification is present, the popup cuts off the content

VS Code version: Code 1.36.1 (2213894ea0415ee8c85c5eea0d0ff81ecc191529, 2019-07-08T22:56:38.504Z)
OS version: Darwin x64 18.6.0

@JSRobertOuimet does that reproduce with every notification?

@bpasero I don't think it matters. Once all notifications are dismissed, the resulting popup (with the text “NO NEW NOTIFICATIONS”) is not tall enough to accomodate its content.

I cannot reproduce that. Anything special about your computer?

I can't think of anything unfortunately.

Can you try to run out of sources to see if it reproduces reliably? https://github.com/microsoft/vscode/wiki/How-to-Contribute


FI-2622: Update meta tags for link unfurling

Summary
Adds custom meta tag handling for URL unfurling. There is a follow-on to investigate the usage of React Helmet to replace dynamic title generation.
Testing Guidance
Install the Google Chrome extension META SEO Inspector (https://chromewebstore.google.com/detail/meta-seo-inspector/ibkclpciafdglkjkcibmohobjkcfkaef) and check the meta tags for title and description. Once merged and pushed to prod, this should also be tested against https://www.zelolab.com/free-tools/preview-as/ for other social media unfurls.

The problem with adding more fallback in the erb is that it won't be properly overwritten by React meta tags if they have the same identifier, I can investigate further here. Ideally yes, the backend would be handing everything over but I also don't have a good grasp of what would need to happen to make that feasible.


use ChatGPT to write hugging face connectors

This is a fun gimmick, but you can actually just use ChatGPT to write connectors for the local Hugging Face Flask server. The prompt below should be fairly self-explanatory, but I've also updated the README with instructions on using create_hugging_face_connector.py.
The prompt to ChatGPT:
Given the description below, please write a Python function that follows the following format:
- The function should be named `import_and_return_<model_name>`.
- The function should return a tuple of three items: a model, a tokenizer, and a function that takes the model, a string, and the tokenizer as arguments and returns the output of the model on the string.
- If there is no need for, for instance, a tokenizer, then the function should return `None` for that item in the tuple. Same with the model.
- The function should be able to be called as `model, tokenizer, model_fn = import_and_return_<model_name>()`.
- If no examples are given in the description, then simply return "No examples given on the page.".

Below is an example.

def import_and_return_tk_instruct_3b_def():
    # Example import function for Huggingface model. Returns model, tokenizer,
    and code for running the model.
    tokenizer = AutoTokenizer.from_pretrained(ALLENAI_INSTRUCT3B_MODEL)
    model = AutoModelForSeq2SeqLM.from_pretrained(ALLENAI_INSTRUCT3B_MODEL)

    def model_fn(model, string, tokenizer):
        input_ids = tokenizer.encode(string, return_tensors="pt")
        output = model.generate(input_ids, max_length=10)
        output = tokenizer.decode(output[0], skip_special_tokens=True)
        return output

    return model, tokenizer, model_fn

================== DESCRIPTION ==================
{description}
=================================================
Your response:

Will check this out soon - thank you!


Reexport window.navigator() in leptos_dom

Hi!
Thanks for this awesome project!
I am building my website and want to be able read out the users language string which is usually returned from the navigator object within the window object in JS world.
The "Navigator" feature is not used in leptos_dom and now I'm wondering which approach would be the best way to archive this:

import web_sys with "Navigator" feature myself?
Fork leptos_dom and add this as optional feature for leptos_dom?

Any help or hint is highly appreciated.
Thanks in advance!

You can just add your own web-sys dependency in Cargo.toml
web-sys = { version = "0.3", features = ["Navigator"] }
and this will be merged with what leptos_dom already uses.


OVN DNC: ignore all transit switches when cleaning up stale Nodes OVN switch

📑 Description
When default network controller (DNC) restarts and a sync is invoked, it may not exclude transit switches that support different L3 networks when attempting to cleanup stale Node switches.
Therefore, connectivity between Nodes will be impacted for Pods attached to L3 networks because its transit switch maybe removed during DNC sync.

Fixes # https://issues.redhat.com/browse/OCPBUGS-44487
Additional Information for reviewers

✅ Checks

[ ] My code requires changes to the documentation
[ ] if so, I have updated the documentation as required
[x] My code requires tests
[x] if so, I have added and/or updated the tests as required
[ ] All the tests have passed in the CI

How to verify it

Looks like my e2e test restarting the ovnkube node pod is highlighting a different bug:
2024-12-09T09:32:58.707275499Z stderr F F1209 09:32:58.707225   41989 ovnkube.go:137] failed to run ovnkube: failed to start node network controller: failed to start NAD controller: initial sync failed: failed to sync network network-segmentation-9533.gryffindor: [node-network-controller-manager network manager]: failed to start network network-segmentation-9533.gryffindor: failed to add network to node gateway for network network-segmentation-9533.gryffindor at node ovn-worker2: could not add VRF mp53-udn-vrf routes for network network-segmentation-9533.gryffindor, err: failed to add route {Ifindex: 198 Dst: <IP_ADDRESS>/32 Src: <nil> Gw: <IP_ADDRESS> Flags: [] Table: 1198 Realm: 0} for VRF device mp53-udn-vrf, err: route manager: failed to add route ({Ifindex: 198 Dst: <IP_ADDRESS>/32 Src: <nil> Gw: <IP_ADDRESS> Flags: [] Table: 1198 Realm: 0}): failed to apply route ({Ifindex: 198 Dst: <IP_ADDRESS>/32 Src: <nil> Gw: <IP_ADDRESS> Flags: [] Table: 1198 Realm: 0}): failed to add route (gw: <IP_ADDRESS>, subnet <IP_ADDRESS>/32, mtu 0, src IP <nil>): network is unreachable

/hold
Investigating:
2024-12-09T09:32:58.707275499Z stderr F F1209 09:32:58.707225   41989 ovnkube.go:137] failed to run ovnkube: failed to start node network controller: failed to start NAD controller: initial sync failed: failed to sync network network-segmentation-9533.gryffindor: [node-network-controller-manager network manager]: failed to start network network-segmentation-9533.gryffindor: failed to add network to node gateway for network network-segmentation-9533.gryffindor at node ovn-worker2: could not add VRF mp53-udn-vrf routes for network network-segmentation-9533.gryffindor, err: failed to add route {Ifindex: 198 Dst: <IP_ADDRESS>/32 Src: <nil> Gw: <IP_ADDRESS> Flags: [] Table: 1198 Realm: 0} for VRF device mp53-udn-vrf, err: route manager: failed to add route ({Ifindex: 198 Dst: <IP_ADDRESS>/32 Src: <nil> Gw: <IP_ADDRESS> Flags: [] Table: 1198 Realm: 0}): failed to apply route ({Ifindex: 198 Dst: <IP_ADDRESS>/32 Src: <nil> Gw: <IP_ADDRESS> Flags: [] Table: 1198 Realm: 0}): failed to add route (gw: <IP_ADDRESS>, subnet <IP_ADDRESS>/32, mtu 0, src IP <nil>): network is unreachable

/unhold

/assign @tssurya

Thank you.
I wonder what's the purpose of the L2 test though ...
AFAIU, you're only preventing the transit switch from being deleted for the default network. And that can't be L2.

Correct. i included L2 to increase test coverage.

Rebase only - no changes upon rebase.


Empty scope is not handled

I'm using https://github.com/heroku/docker-registry-client to interact with my registry.
When getting /v2/ the challenge returned does not contain a scope  (seems legit).
But the heroku lib specify scope even if it's empty : GET /auth?scope=&service=Docker+registry .
This is not well handled on docker-auth side : Bad request: invalid scope: "" .
According the spec https://docs.docker.com/registry/spec/auth/token/#/requesting-a-token , the scope might be empty.

should've been fixed, thank you!

thx to you @rojer When the latest image should be updated and available?

just pushed, in :latest now

Thx. I confirm, it's fixed.


Coveralls client no longer supports Python 2.6

The coveralls client code uses '{}'.format() style formatting, without numbered placeholders, which is not allowed in Python 2.6. This style is used in multiple places in the client code.
Submitting coverage to coveralls.io...
Traceback (most recent call last):
  File "/home/travis/virtualenv/python2.6.9/bin/coveralls", line 11, in <module>
    sys.exit(main())
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/coveralls/cli.py", line 77, in main
    result = coverallz.wear()
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/coveralls/api.py", line 168, in wear
    json_string = self.create_report()
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/coveralls/api.py", line 182, in create_report
    data = self.create_data()
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/coveralls/api.py", line 237, in create_data
    self._data.update(git_info())
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/coveralls/git.py", line 68, in git_info
    'id': gitlog('%H'),
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/coveralls/git.py", line 29, in gitlog
    '--pretty=format:{}'.format(fmt))
ValueError: zero length field name in format

Neither do we.


Would be great if we can run utm bundle from anywhere.

Hello today I was testing bare qemu and utm. Suprisingly qemu was able to boot some machines off external drives.

Hello and thanks for your suggestion. Work is ongoing to add this feature.
This issue is a duplicate of #132


The best way to acheive this effect

Hi there. This SLKTextViewController is great and I wanna use it in my current project. But I need to achive this effect:
When I hit the left button, the SLKTextView gone and the record button will show. When I hit the left button again, the record button hide and the textView show. Is there a specific interface here to solve this kind of problem?

What about the record button? Add a subview on the tool bar?


Better name for azure-spring-boot-metrics-starter starter

The correct name should be azure-metrics-spring-boot-starter. Naming convention is defined at here.

Duplicate of #462 due to network issue.


Create Example App that shows PKCE

We want to show that Cognito can handle a workflow with a public client using PKCE.
AC:

JS application that manages the login through Cognito and requires the code challenge (PKCE)
JS application securely stores the tokens (secure Cookies)


Prevent the creation of useless col-*-push/pull-12 classes

Fixes https://github.com/twbs/bootstrap/issues/16463

/CC @mdo

I've no idea if folks are actually using those classes, so I feel better leaving them in. Plus, technically you could make the argument we'd be nuking a class, and that's not backward compatible.

Thanks though!

Ok... at least will this be considered for Bootstrap v4?

Possibly. It's not currently addressed in v4 as I'm iffy on how much of a problem this is for folks.

It is not a problem but it is unnecessary/redundant code so why have it in the first place. Just makes no sense

Well it's unnecessary and redundant to you and I, but I've honestly no idea if folks are actively using or expecting these classes to be there. Moreover, we tell folks we have a 12 column grid, but when there are only 11 column grid classes available, that's where questions come up.
I'm iffy is all I'm saying. It does very little harm in the grand scheme of things. I'd be fine to revisit this in v4 though when the alpha comes out.

... and having those extra classes made me question them also (https://github.com/twbs/bootstrap/issues/16463) lol :) Anyways, I just tried to contribute. Have a nice day.

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <Re: [bootstrap] Prevent the creation of useless col-*-push/pull-12 classes (#16480)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.


Avoid importing object

As per the coding guideline.

Jenkins, test this please.

Jenkins CI test (for commit eda97c83ab1bf43ba542e700ded7eac87aa87fbd, target branch master) failed with status FAILURE.

LGTM! Test failures are not related to this PR.


Allow testing a "shallow" render

Currently it is challenging to test a component without rendering it deeply.
One thing that makes this challenging is the lack of something like rspec's stub_template method
It's also impossible to mock render on the controller.view_context as this is a new instance on every call.
I've found this workaround, but it's rather unwieldy:
  let(:instance) { MyComponent.new(my_params: 'val') }

  before do
    allow(MyComponent).to receive(:new).and_return(instance)
    allow(instance).to receive(:render)
  end

  it "calls render" do
    render_inline(MyComponent, my_params: 'val')
    expect(instance).to have_received(:render).with('catalog/facet_group', param1: 'foo')
  end

Why would you want to mock the rendering if the actual thing you want to test is the output HTML?
So maybe your component view has some conditions, then you would want to check that (given the conditions are met) the view does reflect those by rendering the expected HTML.
Copying some ideas from react, they tend to use "snapshots" with Jest which are not that useful in my opinion but they do the work on letting us know wether the changes we introduced in the view layer are breaking some specs/tests for the view. So maybe we don't need to write an expectation for the whole html, but just for the bits we are interested the most.
What do you think about testing it like this:
require 'rails_helper'

RSpec.describe SomeComponent, type: :component do
  describe 'html' do
    subject do
      render_inline(SomeComponent, some_title: 'Some value').css('.top-level-selector-here').to_html
    end

    it 'renders the html correctly' do
      expect(subject).to include 'Some value'
    end
  end
end

@jcoyne thanks for raising this.
Do you have any thoughts on what a preferred testing API might look like for this functionality?
Perhaps we could pair again if you'd prefer.

Would something like a registry of components and templates that we know not to pass to the parent render work?   Something like this?
stub_render SubComponent, with: 'Phony result'
stub_render 'template', with: 'Phony template render'

Be a place we could start?

Is this something worth starting on?

Definitely!
My memory is fuzzy, but I think I remember that we looked at this when we paired last, and that you had some ideas on how we might implement this in a way that matched an existing Rails convention.
Do you remember that?

@joelhawksley @jcoyne Maybe we can pass a parameter to the render_inline method itself?
render_inline(MyComponent.new(my_params: 'val'), shallow: true)

@radiantshaw perhaps? Might you write a draft PR with an example API and test cases?

Just stumbled upon this. Testing small components that render large ones is tricky because of this.
Without shallow render you'd have to do something like this right now:
my_collection.each_with_index do |item, index|
  # this helper also checks for correct arguments
  allow_next_instances_of(
    OriginalChildComponent,
    index,
    some_prop: item
  ) do |instance|
    allow(instance).to receive(:render_in).and_return('original-child-component')
  end
end

render_inline(described_class.new)

expect(page).to have_text('original-child-component', count: collection.count)

What ViewComponents could offer could look like this:
shallow_render_inline(described_class.new) # every ViewComponent is stubbed as a custom element now, except for the tested one

expect(page).to have_selector('original-child-component-stub', count: collection.count)
my_collection.each_with_index do |item, index|
  expect(shallow_renders[OriginalChildComponent][index]).to eq([item])
end

Slots might be tricky to solve here, they could be either rendered within a <div data-slot="slot-name"> or using declarative ShadowDOM.

@CyberAP interesting! Would you be up for starting a proof-of-concept PR? Doesn't have to be pretty ❤


gallery overlay close button is clicking behind & loader missing

Bug Report or Feature Request
- [X] bug report
- [X] feature request

OS and Version?

macOS Sierra

Versions
Angular CLI: 1.5.0
Node: 8.7.0
OS: darwin x64
Angular: 5.0.3
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
angular/cdk: 5.0.0-rc.1
angular/cli: 1.5.0
angular/flex-layout: 2.0.0-beta.10-4905443
angular/material: 5.0.0-rc0
angular-devkit/build-optimizer: 0.0.33
angular-devkit/core: 0.0.21
angular-devkit/schematics: 0.0.37
ngtools/json-schema: 1.1.0
ngtools/webpack: 1.8.0
schematics/angular: 0.1.7
typescript: 2.4.2
webpack: 3.8.1
Repro steps

Close button  issue:
just click on the close button from the gallery overlay and if you have some link, button with an action, even the gallery itself behind the close button it triggers automatically, the only fix that i could find without changing the source code was in my .css changed the position absolute atribute for position fixed, then the problem stops.
Loader missing issue:
the loader is missing, i dont know which kind of file is.. img,svg,gif so i cant fix the console error saying that "loader", in this case oval is not found.

Please tell me where i can find the loader, Thanks!

Hi @deokey, After beta.8 the loader icon should take the full path of the icon, the doc is not updated yet
For the loader, try
loader: {
    icon: "{PATH_TO_ICON}"
},

You can also use an inline data to avoid the icon request. the document will be updated as soon as I fix the build setup problem


Support slashing queries

Ensure all metadata is in the contracts.
Provide adaptor for the gRPC clients.
Ensure we have all info needed for CosmWasm query request support as well
Builds on https://github.com/confio/tgrade-contracts/issues/280 Please comment there for any other info needed by the sdk. Especially for slashing/jailing.

@alpe can you see if this is still relevant

Closing the issue because it is already implemented.


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update functions-bindings-sendgrid.md

Fix from property to match the SendGrid API for nodejs function according to https://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/index.html

@simonaco,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@simonaco : Thanks for your contribution to the Azure documentation! The author, @rachelappel, has been notified to review your proposed change.

#sign-off


Can't update static costmap layer, no map received. Connecting to the GO2 Robot Via webrtc

I followed the guide, on a Unitree Go2 EDU Plus, after running the commands:

source install/setup.bash
export ROBOT_IP="my_robot_ip"
export CONN_TYPE="webrtc"
ros2 launch go2_robot_sdk robot.launch.py

I get the error "Can't update static costmap layer, no map received."
Also, on rviz I get the error on RobotModel (as shown in the attached image)
The model in rviz does not move and I do not receive any points with the lidar.
The only thing I can get is the camera streaming.

For more details this is the output: https://pastebin.com/2NQufuhw

Hello, can you please give me more details? Are you using the robot's wifi IP address? How did you get it? have you ping it in the terminal?

Yes, we are using the robot ip address by exporting the ROBOT_IP variable as shown in the readme.
We got the address from the mobile application, which is <IP_ADDRESS>.
We can also see another address on the network which is <IP_ADDRESS>.
We can ping both addresses, but can only connect through ssh to <IP_ADDRESS>.
We think that the connection works because we are able to see the streaming of the camera when using <IP_ADDRESS>

Hi,
I have the same problem... Are there any solutions?


Show image index for infer_many

This mostly helpful when debugging shallow copy errors because it's easy to see when the first 16 images are wrong and everything else is right, for example. Hopefully with #603 those issues will go away and never come back, but this seems like it could still be useful.
Here's a screenshot of this feature used in conjunction with #608 (adds numbering to the bottom table):

Thanks, this looks good to me.
Question: I was wondering whether we should show the index of images from the list in the file instead/too? The indices will differ if there are images we failed to load. Also, shall we show the paths to those images (with an error message instead of a set of predictions)?

I was wondering whether we should show the index of images from the list in the file instead/too? The indices will differ if there are images we failed to load. Also, shall we show the paths to those images (with an error message instead of a set of predictions)?

That's a good idea. Sounds like that would require passing even more data around from the ModelJob to the InferenceJob and then back again - right?

That's a good idea. Sounds like that would require passing even more data around from the ModelJob to the InferenceJob and then back again - right?

I don't think that would be the case because the inference job already passes the list of path IDs it was able to successfully load through the inputs['ids'] list, like so:
https://github.com/NVIDIA/DIGITS/blob/master/digits/model/images/classification/views.py#L419


Add BASIC Token

Token Address: 0xf25c91c87e0b1fd9b4064af0f427157aab0193a7
Token Name: BASIC Token
Token Decimals: 18
Token Symbol: BASIC
Etherscan: https://etherscan.io/token/0xf25c91c87e0b1fd9b4064af0f427157aab0193a7
Coingecko: https://www.coingecko.com/en/coins/basic/
Coinmarketcap: https://coinmarketcap.com/currencies/basic/
Project website: http://www.thebasic.io
Twitter: https://twitter.com/thebasicfinance
Facebook: https://www.facebook.com/basicfinanceofficial/
Telegram: https://t.me/thebasic
Github: https://github.com/bcdc-defi
Medium: https://medium.com/thebasic
Linkedin: https://www.linkedin.com/company/thebasicio/

Thank you!


Update package.json for fork (scoped/prefixed package name, updated project URLs)

Description of the Change

Change the package name to be the scoped/prefixed @atom-ide-community/atom-package-manager.
(If this is the wrong prefix, please let me know.)
Alternate Designs

We could use some other scope/prefix, rather than @atom-ide-community.
Benefits

Allows us to publish to the npmjs.com package registry, clearly distinguishes our source tree from upstream's.
Possible Drawbacks

None, unless I'm misunderstanding and we can publish to the scoped name without changing package.json... In which case this might be unnecessary, but I'd still like to do this to distinguish our source tree from upstream's.
Verification Process

Compare with how @rails/webpacker does this in their package.json:
https://github.com/rails/webpacker/blob/4fbb7bcadcd190b2c21cfc3c6bd894328e648e65/package.json#L2
Applicable Issues

N/A

@aminya I made some PRs at this fork.
Please take a look if you have the time, or if you prefer I can handle merging. I can basically take care of/be the main maintainer of this repo myself if you like.

If you end up merging any of these, please merge this PR FIRST. I want our history of changes to always have a package.json that reflects the fact that we've forked from upstream.
Friendly reminder: If we are going to use a different prefix than @atom-ide-community, then less discuss that before merging this, of course. I'm okay with that prefix, personally, but if there's consensus that's even better.

Hi, thanks for looking at/reviewing/updating this.
If you can, I'd like to not merge master into our PRs. It makes the commit history more complicated, and I try to make neat PRs, so while it doesn't mean a lot, it kinda makes me sad to see the extra noise.
ANYHOW, we are making good progress.

Oh, and thanks for activating CI!! :heart:


HTMLCanvasElement toBlob

We're using this issue to track the pending content merge.

@jeffposnick - can we close this.


fix loop forever

fix compile error and loop forever when use ? in main fn

Thanks! Good catch.

Interestingly, i get a compile error now:
error[E0308]: mismatched types
   --> src/lib.rs:149:31
    |
149 |         match Mode::from_bits(umask as u32) {
    |                               ^^^^^^^^^^^^ expected `u16`, found `u32`
    |
help: you can convert a `u32` to a `u16` and panic if the converted value doesn't fit
    |
149 |         match Mode::from_bits((umask as u32).try_into().unwrap()) {
    |                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

After looking the code in nix::libc, the type is actually specified as mode_t.
Can we go for  match Mode::from_bits((umask as mode_t) then?
For me, it compiles with Rust (1.51.0).

Ok, i've tested the build with GitHub actions and Ubuntu, seems that mode_t is also working on Linux. Hence i would push this change to master and release the crate.

@cossonleo  daemonizr-0.1.3 is released now.


[BUG] Voice input doesn't stay on

Describe the bug
Absolutely love the voice recognition feature! But in a previous version, I would be able to speak, collect my thoughts, then keep speaking.   I think this bug was introduced when the 'Voice continuation' feature was added which makes sense because when that mode is enabled, it should automatically determine when to stop 'listening' and send the message. But I'd wager many users (including myself) rarely use 'voice continuation'
Where is it happening?

All platforms

To Reproduce
Steps to reproduce the behavior:
Use the 'voice input' with voice continuation disabled, say a few things, stop, then see it stop recording.
Expected behavior
See above

I am on Mac. Your proposal makes sense, but if I'm being honest I don't think I would use it because I would likely just switch voice continuation off. I can think of times when I'd take much longer than 5s to collect my thoughts when speaking to it. (maybe i'm just slow 🤠)
I believe with voice continuation off, it should never stop transcribing unless the user hits the microphone button to explicitly tell it to stop listening. With voice continuation on, your proposal makes perfect sense. To me, I am mainly concerned about the voice continuation off case.

Thanks, I'll prototype this (maybe with a 30s max delay or an "off" option).
The complexity is handling 2 different logics (mobile and desktop behave differently) in all 3 modes (mic is also used for the "Call" function, where I need a timeout signal).
Good inputs.

Added a configurable timeout which works well both for people who want to have quick automatic chats, as well as people that can want to collect thoughts, and something in between.

What do you think?

Love it!!
Just one minor bug is that you need to refresh the page for the change to take effect.

Fixed it, thanks for reporting @ramicaza!


Changed Color themes according to Default or Darcula

Changed Color themes according to Default or Darcula
Still I am trying to figure out how to highlight function and variable, however, someday we need to redesign BNF rules. ;)

Is there support for custom function and variable highlighting?
I only see these types highlighted:

I just discovered this package and am very excited to switch from RStudio.

Not yet, but at least variable highlighting should be easy to do by patching com.r4intellij.highlighting.RSyntaxHighlighterColors
I think, there's no easy way to distinguish custom from library functions on the highlighter level.

Thanks for the response!
By patching, I assume that you mean adding a TextAttributesKey here? Is there a good location to look for all possible R_*  values?

I think, there's no easy way to distinguish custom from library functions on the highlighter level.

I don't see any function highlighting. Am I missing an option?
I can see that there may be some use cases to highlight custom and library functions, but for my purposes any function highlighting would be great.
Thanks!

The colors are bound to the lexer in com.r4intellij.highlighting.RSyntaxHighlighter
Just coloring function calls might be tricky because of the way call expressions are defined in r.bnf
call_expression ::= expression argument_list

expression can be a almost anything here (see the grammar spec)
Custom function calls would  be lexed to something like primitive_expression argument_list but just when actually resolving reference we know the type (variable of custom function definition) of each primitive_expression.
So in order to highlight function calls we would somehow need to use resolver information for highlighting. I'm not sure if this is possible with the platform API ( @ktisha ?).

Thanks for your response.
I may try to patch in order to get variable highlighting.


Tests fail on PHP 7.1

Running tests on PHP 7.1 throws two warnings and nearly all tests fail with "InvalidArgumentException: Value must be provided."
$ phpunit --version
PHPUnit 5.7.5 by Sebastian Bergmann and contributors.

$ php -v
PHP 7.1.0 (cli) (built: Dec  2 2016 03:30:24) ( NTS )
Copyright (c) 1997-2016 The PHP Group
Zend Engine v3.1.0-dev, Copyright (c) 1998-2016 Zend Technologies
    with Xdebug v2.5.0, Copyright (c) 2002-2016, by Derick Rethans

Full error dump: http://laravel.io/bin/E3VK2

Please discard


Refactoring work-in-progress

Thanks for this Shayne, I enjoyed pairing on it! 😄

Closing this pull request as it's been merged into this one.


[Passive] COMET TEARS

Shot speed down + Homing. Tears have a large trail that will burn enemies on contact

 COMET TEARS: Shot speed down + Homing. Tears have a large trail that will burn enemies on contact


I can't run this code

There a lot of problems when I run the code. Can you provide information about your experimental environment? Thanks!

Hi,
Could you clarify the exact issues that you are facing?
Thanks!

Hi,
The previous issue may be caused by the environment version, which has now been resolved. Thanks for your reply!
In STEP/classifier_stgcn_real_only/main.py, there is a small issue on line 75. There is no function 'split_data' in 'loader.py'. Its function seems to be merged into  function 'load_data'.
Anyway thanks for your code!
In addition, I did not find affective features in dataset. Will affective features be made public?

Hi, thanks a lot for identifying the issues! We'll address them and push the updated code!
We are also in the process of preparing a proper documentation. Hopefully, things will become much clearer once that is up.
We have python codes for creating the affective features corresponding to the joint information, we'll push that soon as well. Meanwhile, I have uploaded the affective features for the real 2,177 data points in the Google drive with the rest of the dataset. The files are called 'affectiveFeatures.h5' (our 342) and 'affectiveFeatures_ELMD.h5' (1,835 from ELMD). Please let us know if you have any issues using them.

Closing this since there has been no further questions


Add the possibility of source function to Apify.openRequestList

There are use cases where the sources need to be loaded in various ways, from APIs, spreadsheets, databases. It would be useful to allow the user to provide a function in place of sources. This function would only be run once, because later, the key value stored sources will be used. Users then wouldn't have to include their own logic to prevent running of this function on restart.

I guess this is long time implemented, right?
https://sdk.apify.com/docs/typedefs/request-list-options#sourcesfunction
https://crawlee.dev/api/core/interface/RequestListOptions#sourcesFunction


requirejs sample loads 164 individual .css files

the dojo build sample produced what i'd expect, but i counted 164 unconcatenated stylesheets loading when i launched the built requirejs sample.
STR:

clone repo
cd into ./bower/requirejs
npm install
npm build (all green so far)
run /dist/index.html and inspect network traffic.

can anyone else confirm the behavior?

It looks like r.js needs separate tasks to build the css. For now you can add this to the requirejs: section of the Gruntfile.
      esricss: {
        options: {
          cwd: './',
          cssIn: './src/esri/css/esri.css',
          out: './dist/esri/css/esri.css',
          optimizeCss: 'standard'
        }
      },
      dijitcss: {
        options: {
          cwd: './',
          cssIn: './src/dijit/themes/nihilo/nihilo.css',
          out: './dist/dijit/themes/nihilo/nihilo.css',
          optimizeCss: 'standard'
        }
      },

And update the build task.
grunt.registerTask('build', ['clean', 'requirejs:support', 'requirejs:single', 'requirejs:esricss', 'requirejs:dijitcss', 'copy']);
I'll test a little further before making the changes, but that should work.
Thanks!

thanks for the help rene.  i'll test and PR this addition (along with a few other little things).


Binary using pkger not working outside of source folder

Hi, I ran pkger in a go modules enabled project, built the binary and ran it from the source folder and it worked.
When I run this binary from /tmp it fails.
log:
~/p/play-pkger (master|✚2…) $ cat cmd/play-pkger/play-pkger.go
package main

import (
	"fmt"

	"github.com/markbates/pkger"
)

func main() {
	err := run()
	if err != nil {
		panic(err)
	}
}

func run() error {
	f, err := pkger.Open("/LICENSE")
	if err != nil {
		return err
	}
	defer f.Close()

	info, err := f.Stat()
	if err != nil {
		return err
	}

	fmt.Println("Name: ", info.Name())
	fmt.Println("Size: ", info.Size())
	fmt.Println("Mode: ", info.Mode())
	fmt.Println("ModTime: ", info.ModTime())
	return nil
}
~/p/play-pkger (master|✚2…) $ pkger
~/p/play-pkger (master|✚2…) $ tree
.
├── cmd
│   └── play-pkger
│       └── play-pkger.go
├── go.mod
├── go.sum
├── LICENSE
└── pkged.go

2 directories, 5 files
~/p/p/c/play-pkger (master|✚2…) $ go build -o /tmp/play-pkger
~/p/p/c/play-pkger (master|✚2…) $ /tmp/play-pkger
Name:  LICENSE
Size:  11353
Mode:  -rw-r--r--
ModTime:  2019-11-20 10:46:18.276189286 +0100 CET
~/p/p/c/play-pkger (master|✚2…) $ cd /tmp/
/tmp $ /tmp/play-pkger
2019/11/20 10:51:45 exit status 1: "go list -json -m": go list -m: not using modules

panic: open /LICENSE: no such file or directory

goroutine 1 [running]:
main.main()
	/home/play/pr/play-pkger/cmd/play-pkger/play-pkger.go:12 +0x44
/tmp [2] $

It seems that this binary also would not work on a machine which doesn't have go installed.

I think pkged.go is not built in your binary.
Adding _ "path/to/your/module" in your imports should fix the problem.

Nope the import doesn't help. There is a pkged.go file in the root folder next to go.mod (see tree output in the log)

Use pkger -o cmd/play-pkger to put the pkged.go file in the package you need it.

@ContiGuy  what if you compile it on the current directory and then move binary to /tmp ?
Also, what is your version of go ?


non-intuitive pipeline trigger behaviour

"Pipeline completion triggers use the Default branch for manual and scheduled builds setting to determine which branch's version of a YAML pipeline's branch filters to evaluate when determining whether to run a pipeline as the result of another pipeline completing. By default this setting points to the default branch of the repository."
... this is really annoying and non-intuitive behaviour for pipelines in the same repo.  Is there any way this can be changed to evaluate pipeline yaml from the current branch rather than the default branch?  (especially given that it's the pipeline yaml in the current branch that will actually be run)

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

ID: 86285f72-9e28-da97-59bb-c29eb60f627d
Version Independent ID: 18d5a591-a7d3-c261-6bff-8808ae433f54
Content: Configure pipeline triggers - Azure Pipelines
Content Source: docs/pipelines/process/pipeline-triggers.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @steved0x
Microsoft Alias: sdanie

@benjaminclewis Thanks for your question. You should checkout the dev team board. See if there's any discussion and - if not - enter feature request.


Resolves bundle issue.

Now supports test dependencies.
Now supports multiple hops between dependencies in a bundle.

Resolves https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/issues/425.

Build failed


Blank canvas when getting element by ID

Is it possible to use getElementByID to only print a specific element to canvas?
Using the latest version and it will only work with document.body
	$("#myButton").click(function () {
		html2canvas(document.getElementById('myTable'), {
			allowTaint: true,
			imageTimeout: 15000,
			logging: true,
			useCORS: true
		}).then(function (canvas) {
			document.body.appendChild(canvas);
			const b64 = canvas.toDataURL("image/png").replace(/.*,/, '');
			console.log("data:image/png;base64, " + b64)
		});
	});

Maybe your element is scrolled up or down. Try by adding scrollX: -window.scrollX and scrollY: -window.scrollY
	$("#myButton").click(function () {
		html2canvas(document.getElementById('myTable'), {
			allowTaint: true,
			imageTimeout: 15000,
			logging: true,
			useCORS: true,
                        scrollX: -window.scrollX,
                        scrollY: -window.scrollY
		}).then(function (canvas) {
			document.body.appendChild(canvas);
			const b64 = canvas.toDataURL("image/png").replace(/.*,/, '');
			console.log("data:image/png;base64, " + b64)
		});
	});

Perfect, that's solved the issue thanks.

Please write these two properties to the main examples acording to often scrolling while screenshoting. See comment above  https://github.com/niklasvh/html2canvas/issues/2368#issuecomment-701920277
scrollX: -window.scrollX,
scrollY: -window.scrollY


optional nightly SIMD implementation

Hey,
Would it make sense to add a simd flag and give an alternative implementation that uses std::simd?
I would like to work on this if it makes sense

closing this since it doesn't make sense after looking into it


Not running default start script

Is it just me, or do I have to manually include the start script in order for npm-run-all to pick it up?
"scripts": {
  "watch": "...",
  "dev": "npm-run-all --parallel watch start"
}

Issuing:
$ npm run dev

Did not perform default node server.js command. No errors, just not running my server.
However when I manually add start script:
"scripts": {
  "start": "node server.js",
  "watch": "...",
  "dev": "npm-run-all --parallel watch start"
}

all is well. this is on Windows.

Thank you for this issue.
Hmm, I'm not sure whether or not npm has the default script of start command.
npm said "missing script: start".
$ npm run start
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Users\\t-nagashima.AD\\nodist\\v-x64\\nodev4.2.3\\node.exe" "C:\\Users\\t-nagashima.AD\\nodist\\bin\\node_modules\\npm\\bin\\npm-cli.js" "run" "start"
npm ERR! node v4.2.3
npm ERR! npm  v3.5.0

npm ERR! missing script: start
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\t-nagashima.AD\Documents\GitHub\eslint\npm-debug.log

$ npm start
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Users\\t-nagashima.AD\\nodist\\v-x64\\nodev4.2.3\\node.exe" "C:\\Users\\t-nagashima.AD\\nodist\\bin\\node_modules\\npm\\bin\\npm-cli.js" "start"
npm ERR! node v4.2.3
npm ERR! npm  v3.5.0

npm ERR! missing script: start
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\t-nagashima.AD\Documents\GitHub\eslint\npm-debug.log

On the other hand, npm has the default script of restart.
$ npm run restart
$ npm restart
$

I'm referring to this:
https://docs.npmjs.com/cli/start
It says if there is no "start" defined in "scripts" that it runs node server.js

Wait I think I'm beginning to understand. That's not a default script as much as the default behavior of a different command npm start. I was under the wrong impression that npm start was simply shorthand for npm run start, but it's more like npm start will execute npm run start if there is astart script

Whoops, bumped the close issue button.
To finish my thought: npm start  just runs node server.js for you if there is no startscript. Then my only remaining question is: does npm-run-all report any error if you specify a script which does not exist? I did not see any error when executing npm-run-all start without a "start" script defined.

Yeah, npm-run-all should report the error, but not. I will fix it.


Select Materialize doesnt work using ng-repeat angular.

its my source:

its my result:

After inspect Element:

if i look the select option is work but why UL not repeat like select.
Thanks.

up

@bylinggha, Materializecss when initializes the material_select the value of noTelp is empty, you have to $('select').material_select(); after $scope.noTelp is updated with the new values


COLLAPSE_MIN_LINES

is there a way to modify these numbers?
const COLLAPSE_MIN_LINES = 25; // Set this to the maximum number of lines you want to display before collapsing
const COLLAPSE_MIN_LENGTH = 3000; // Set this to the maximum number of characters you want to display before collapsing

also, can someone confirm that newer version has removed the download icon button?

Currently you cannot modify these values. I confirm we removed the download button because we added a copy button below messages.

got it. thanks. currently, is there a feature that we can download or share the chat?

Sharing a chat is coming!

Is there a plan to add an export feature in the future?

Opened to contributions!


Add Line{Circle|Sky}Region

One step towards fixing https://github.com/astropy/regions/issues/45

@keflavich , others : Currently the line __init__ takes start and end. For the parser it would be easier if this was actually a len 2 list of SkyCoords (like a Polygone with 2 vertices). Opinions?


Text on header hero section shouldn't change with theme

Here is the problem:

So since the background image doesn't switch with the theme, but the text in the header section does with change in theme, when one selects the theme as light, the text is rendered darker. Since the background is still the same, the text is not readable.
I see few possible solutions:

The text color shouldn't change when changing themes only for the header section.
There is a css tint on the background image, say for example when it is light theme the tint is white and the text changes to dark. When dark theme is selected the tint is dark and the text changes to white.
There is a an ability to tie a background image to each theme.
I can just delete the text and put a background that has the text as part of the background image.

Personal choice is point 1 above.
What do you think?

I think option1 is better.
I thought that I did implement it but it was not
I'll add the option titleColor in the header index file.
---
...
- type: img
    ...
    titleColor: green
    ...
...
---

I uploaded!

Although it fixed this issue, but now the whole blog is not responsive. I opened other issue #80 where there was a white bar at the right side and the header was not responsive. After your change, now the whole blog is not responsive :thinking:

Ok, I'll find out what's going on

This is also fixed.
Thank you :tada:


Problem with eigenanatomy decomposition and cortical thickness data

Dear ANTs experts,
I'm working with some developmental data (children between 6 and 14 years) and wanted to apply eigenanatomy decomposition to reduce the dimensionality of the data for further analysis of the relationship between eigenanatomy components and cognitive test scores. The analysis of FA images worked really well, but I'm having some difficulties with cortical thickness data.
For pre-processing of T1-weighted volumes, I applied rigid co-registration to a T1-template and removed all images with excessive motion artefact. Then, I ran the images through the antsCorticalThickness (ANTs v1.9.2) pipeline with priors from the OASIS dataset - following your publication [1]:
antsCorticalThickness.sh
-a T1_file.nii.gz
-e T_template0.nii.gz
-m T_template0ProbabilityMask.nii.gz
-f T_template0
-p priors%d.nii.gz
-o T1_file-
The resulting cortical thickness masks looked reasonable to me.

But I cannot find the expected effect of decreased cortical thickness with age with a t-test comparison across all voxels.

Then, I used the 'CorticalThicknessNormalizedToTemplate' images produced by antsCorticalThickness for eigenanatomy decomposition in ANTsR v0.3.2. Here, I used the setting described in [2], i.e. 32 components, 1/32 sparsity, 20 iterations, 1 smoothness, 1000 voxels per cluster:
sparseDecom(inmatrix = mat,
inmask = mask,
nvecs = 32,
sparseness = 1/32,
its = 20,
smooth = 1,
cthresh = 1000,
ell1 = 0.5,
verbose = TRUE)
However, the decomposition could only explain 83% of the variance (FA-based decomposition from the same sample explained 94%). Further, the resulting decomposition is dissimilar to published results [2] and makes little anatomical sense (see attached ‘CorticalThicknessEanatDecomp’).

I would be extremely grateful for any advice on what might be going wrong here.

[1] Tustison, N. J., Cook, P. A., Klein, A., Song, G., Das, S. R., Duda, J. T., et al. (2014). Large-scale evaluation of ANTs and FreeSurfer cortical thickness measurements. NeuroImage, 99, 166–179. http://doi.org/10.1016/j.neuroimage.2014.05.044
[2] Cook, P. A., McMillan, C. T., Avants, B. B., Peelle, J. E., Gee, J. C., & Grossman, M. (2014). Relating brain anatomy and cognitive ability using a multivariate multimodal framework. NeuroImage, 99, 477–486. http://doi.org/10.1016/j.neuroimage.2014.05.008

What type of model are you using to detect developmental effects?  something like this might be good:
mdl = lm(  mat ~  age * gender + I(age^2) * gender + ICV + ... )
bmdl = bigLMStats( mdl, 0.001 )
tvals = bmdl$beta.t["age",]

or something along those lines ( note use of ICV ) .... regardless of covariates, i am not too surprised that the multiple comparisons correction leads to relatively few voxels surviving the significance threshold.  you might visualize uncorrected values just to see if the pattern of effects is reasonable.
re: sparseDecom (or maybe eanatDef ), you probably should pass scale(mat) into the function if you have not done so already.  you may also try scale( mat, scale=FALSE ) ... the function produces different results depending on if/how you scale the input data, similar to SVD vs PCA.   note also that the paper: http://www.ncbi.nlm.nih.gov/pubmed/24852460 provides a method for "customizing" ROI sets based on your prior notions and this method is available in eanatDef ... generally speaking, one would not expect a largely data-driven method to recapitulate the traditional conception of how neuroanatomy should be divided ... eanat takes into account (for better or worse) variability in the matrix that may included other effects such as signal reliability, scale, etc.
overall, your current results look like they are in the ballpark of what one would expect.

Thank you very much for your help! Unfortunately, I'm still not getting much of an age effect in this sample (linear or quadratic), even in the uncorrected image. Linear effect shown:

Model:
gmat = imageListToMatrix(ilist, mask)
mdl = lm(gmat ~  Age_in_months*Gender + I(Age_in_months^2)*Gender + ICV, data=regressors)
mdl = bigLMStats(mdl)
gmstats = mdl$beta.pval["Age_in_months",]
gqvimg = makeImage(mask, 1-gmstats)
antsImageWrite(gqvimg, 'uncorrected_age_effect.nii.gz')

However, going back to literature, it could be that the age range is just too narow to detect an age effect with this sample size.
Regarding the scaling in the eigenanatomy script, the analysis is still running, but I was a bit surprised that the explained variance is only around 28% when running sparseDecom with the scaled data. Is that possibly indicating that something is going awry?
Thank you again for your help

the model looks reasonable ... did you smooth your data at all before performing the GLM?   something like
list = imageFileNamesToImageList( ... )
for ( i in 1:length( list ) ) {
  smoothImage( ... )    # 6mm FWHM
  resampleImage( ... ) # to 2mm
  }

could "help".  this just changes the spatial scale you are studying and may reduce some of the random effects of anatomical variability.
you may also find that this helps your eanat analysis, at least in terms of speed.
our estimated variance explained indeed goes down when you scale the data in this way, due to the positivity constraint.    it's not something to worry about ... you can compute the "real" variance explained after the fact with standard linear regression.  what's used inside the function is a quick proxy.

also ref: http://www.nature.com/articles/sdata20153/figures/3


Tweak BaseMaterial3D heightmap property hints

Allow finer adjustments of the heightmap scale.
Allow increasing the heightmap level detail (at the cost of performance).

See https://github.com/godotengine/godot-proposals/issues/1843.

Thanks!


Enhance model test CI status

Signed-off-by: chensuyue<EMAIL_ADDRESS>Type of Change
validation
Description
Update model test CI status. When model test failed with perf gap, do not upload artifacts, so pipeline will works with 'Rerun failed jobs'
Expected Behavior & Potential Risk
CI performs as expect.
How has this PR been tested?
CI
Dependency Change?
No

/Azurepipeline run Model-Test

/Azurepipeline run Model-Test

/Azurepipeline run Model-Test


Multilabel Classification

Is it possible to do multilabel classification with creme-ml. Can't find anything in the docs

Hey there @foodiehack. Yes, you can do multi-label classification with creme. Take a look at ClassifierChain.

@MaxHalford So I am initially training a model with 5 labels. On a weekly basis, 1k data appears and it is possible that new label comes too. Is there a way to retrain the model with that new label too without training data from scratch?

Is there a way to retrain the model with that new label too without training data from scratch?

Yep, that's the intention. At least, every algorithm in creme is designed from the ground up to support new features, classes, and outputs to be added on-the-fly. However, this isn't something that we have tested a 100% because of lack of resources.
In short: yes, it should work. If it doesn't, then please tell us and we'll get it fixed.

@MaxHalford Thanks for the prompt reply. I will try it out. As I can see in the example Classfier Chain is used with linear model LogisticRegression.
Can I somehow use LinearSVC or SGD(loss='modified_huber'') along with Classifier Chain or any other classification algo.

I assume that you're referring to models from scikit-learn's linear_model module? If so I'm not sure what you're asking. You can use any creme classifier as an input to the classifier chain.


My account

my account and channel please back to me

Please check and back to me my account telegram (my channel)(😭😭😭)


misc: update golangci-lint and use cached action

Disable depguard as it's unconfigured and ignore the noop logger for unused parameters.

Do you want to merge the other PR as-is, @hanssto, and then we can rebase and merge this one? It looks like it'd be nice to have the full history for these smaller changes.

I'm torn between "I shouldn't spend more time on this" and "I should rework the other PR so that it's more sane in the event of us ever merging this back upstream". I will let tomorrow-me decide as I have to pick up the spawn.

My suggestion is, @hanssto:

Merge the base branch of this PR as a squash as all the changes are the same in nature.
Merge this via rebase and merge to the main branch.

Au voila.

@thor I've cleaned up #34 now, squashing the commits manually and giving them a more unified explanation. Lemme know if it's readable.

@hanssto Looks like a high-quality commit message to me. 🙌 Let's rebase and merge this into #34, then rebase and merge it to main?


Add "Start PowerShell" menu option for Connections objects

When navigating objects in the Connections menu, and you right-click on an object, please add "Start PowerShell" option for this menu.  This functionality already exists for Object Explorer in SSMS.  I want you to start by copying/re-producing that functionality.
When you click the "Start PowerShell" option ADS should either:

A) Start a new PowerShell script (.ps1 file) in ADS and include the Provider-location on the top line of that new script;
B) Or, it should chnge the PowerShell terminal location ( cd / Set-Location) to the Provider-location of the object.

I would prefer option "A" but I suppose you could offer both options via a menu-flyout (I think that's what y'all call it?).
Another idea might be to do option "A" if they have the PowerShell extension installed.  If they don't have the PowerShell extension installed, do option "B".
No matter which variation of the above options you do, this feature will have vastly more adoption in ADS than it ever did in SSMS.

This request is similar to #4665 but different because the starting point for this is ADS, not SSMS.

Adding a little more clarity to "B)" in the original description:
In the 'launching a terminal session and setting the scenario' it should launch a [regular, plain] PowerShell Terminal if you don't have the PowerShell extension installed.
If you do have the PowerShell extension installed, it should set the location of the "PowerShell Integrated Terminal" instead.
Obviously, the client machine must have PowerShell Core or Windows PowerShell installed as well as the SqlServer module in order to make this work.  A simple error message informing the user to install PowerShell Core or the SqlServer module & a valid URL is all that's needed.

This would be very useful. I use PowerShell every day and it would be very nice to have this feature at my fingertips.


Update sys-dm-db-resource-stats-azure-sql-database.md

Fix typo (memory instead of CPU)

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

File
Status
Preview URL
Details

docs/relational-databases/system-dynamic-management-views/sys-dm-db-resource-stats-azure-sql-database.md
:white_check_mark:Succeeded

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

@daschall Thanks for the edit, Daniel!

#sign-off


SBT 0.13.9 resolves to 0.13.8

Newly installed SBT 0.13.9 reports its version as 0.13.8 unless version is specified explicitly in build.properties file. There is only one SBT installation on the system.
The synopsis of the problem is given here.

Probably a mistake during packaging. Are you also using the Windows installer?

Yes, I am. I tried using the zipped file and overwrite the installation dir - that bumped the reported version to 0.13.9 but introduced this error as I tried to build the project in IDEA:
[info] Applying State transformations org.jetbrains.sbt.ReadProject from C:/Users/homeDirectory/.IntelliJIdea14/config/plugins/Scala/launcher/sbt-structure-1.0.jar
java.lang.ClassNotFoundException: org.jetbrains.sbt.ReadProject$
	at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
	at java.lang.Class.forName0(Native Method)
	at java.lang.Class.forName(Class.java:348)
	at sbt.ModuleUtilities$.getObject(ModuleUtilities.scala:14)
	at sbt.BasicCommands$$anonfun$call$2$$anonfun$6.apply(BasicCommands.scala:130)
	at sbt.BasicCommands$$anonfun$call$2$$anonfun$6.apply(BasicCommands.scala:130)
	at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:244)
	at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:244)
	at scala.collection.immutable.List.foreach(List.scala:318)
	at scala.collection.TraversableLike$class.map(TraversableLike.scala:244)
	at scala.collection.AbstractTraversable.map(Traversable.scala:105)
	at sbt.BasicCommands$$anonfun$call$2.apply(BasicCommands.scala:130)
	at sbt.BasicCommands$$anonfun$call$2.apply(BasicCommands.scala:125)
	at sbt.Command$$anonfun$applyEffect$1$$anonfun$apply$2.apply(Command.scala:59)
	at sbt.Command$$anonfun$applyEffect$1$$anonfun$apply$2.apply(Command.scala:59)
	at sbt.Command$$anonfun$applyEffect$2$$anonfun$apply$3.apply(Command.scala:61)
	at sbt.Command$$anonfun$applyEffect$2$$anonfun$apply$3.apply(Command.scala:61)
	at sbt.Command$.process(Command.scala:93)
	at sbt.MainLoop$$anonfun$1$$anonfun$apply$1.apply(MainLoop.scala:98)
	at sbt.MainLoop$$anonfun$1$$anonfun$apply$1.apply(MainLoop.scala:98)
	at sbt.State$$anon$1.process(State.scala:184)
	at sbt.MainLoop$$anonfun$1.apply(MainLoop.scala:98)
	at sbt.MainLoop$$anonfun$1.apply(MainLoop.scala:98)
	at sbt.ErrorHandling$.wideConvert(ErrorHandling.scala:17)
	at sbt.MainLoop$.next(MainLoop.scala:98)
	at sbt.MainLoop$.run(MainLoop.scala:91)
	at sbt.MainLoop$$anonfun$runWithNewLog$1.apply(MainLoop.scala:70)
	at sbt.MainLoop$$anonfun$runWithNewLog$1.apply(MainLoop.scala:65)
	at sbt.Using.apply(Using.scala:24)
	at sbt.MainLoop$.runWithNewLog(MainLoop.scala:65)
	at sbt.MainLoop$.runAndClearLast(MainLoop.scala:48)
	at sbt.MainLoop$.runLoggedLoop(MainLoop.scala:32)
	at sbt.MainLoop$.runLogged(MainLoop.scala:24)
	at sbt.StandardMain$.runManaged(Main.scala:53)
	at sbt.xMain.run(Main.scala:28)
	at xsbt.boot.Launch$$anonfun$run$1.apply(Launch.scala:109)
	at xsbt.boot.Launch$.withContextLoader(Launch.scala:128)
	at xsbt.boot.Launch$.run(Launch.scala:109)
	at xsbt.boot.Launch$$anonfun$apply$1.apply(Launch.scala:35)
	at xsbt.boot.Launch$.launch(Launch.scala:117)
	at xsbt.boot.Launch$.apply(Launch.scala:18)
	at xsbt.boot.Boot$.runImpl(Boot.scala:41)
	at xsbt.boot.Boot$.main(Boot.scala:17)
	at xsbt.boot.Boot.main(Boot.scala)

Wasn't sure how to proceed to so I reinstalled the MSI-packaged distro back to the installation dir, cleaned the .ivy and .sbt directories and then did a Invalidate And Restart in IDEA. That helped to build the project back but also left the SBT version on 0.13.8. Only after build.properties intervention did that report the expected version (0.13.9).

Just a note:  We're probably going to be forcing build.properties to have the sbt version in the future, and you shoudl ALWAYS have it specified now.
I'm a bit suprised, but this could be related to the intellij sbt plugin using the 0.13.8 launcher.

Fair enough - I too think having explicit version set is a good thing. Still, I think a utility shouldn't mis-report its version at any time.
FWIW, my IDEA global SBT settings have Launcher point at the sbt-launch.jar at the install directory, not the bundled version; the SBT still reported the version as 0.13.8.

This issue makes me so frustrated. I hope this problem is fixed soon.

installing via All Platforms > SBT-0.13.9.ZIP worked for me and actually installed version 0.13.9.

I think the root cause is that I didn't modify the wixProductUpgradeId in sbt/sbt-launcher-package#89.
There's now a new installer available from http://www.scala-sbt.org/download.html page.


Docker DNS server - Server misbehaving

Describe the bug
Gatus reports random failed health checks due to errors with Docker's internal DNS server: server misbehaving.
The service in question is accessible and OK. When running dig from another container pointed at this dns, I get a valid domain name resolution.
dig @<IP_ADDRESS> subdomain.domain.tld

; <<>> DiG 9.18.24 <<>> @<IP_ADDRESS> subdomain.domain.tld
; (1 server found)
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 14720
;; flags: qr rd ra; QUERY: 1, ANSWER: 2, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 1432
;; QUESTION SECTION:
;subdomain.domain.tld.		IN	A

;; ANSWER SECTION:
subdomain.domain.tld.	3761	IN	CNAME	cname.tld.
cname.tld.	52	IN	A	X.X.X.X

;; Query time: 1 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>) (UDP)
;; WHEN: Fri Mar 15 16:30:43 CET 2024
;; MSG SIZE  rcvd: 102

Note: The domain names and IPs have been redacted.
What do you see?
The UI reports an error as followed:
Get "https://subdomain.domain.tld/health-check" dial tcp: lookup subdomain.domain.tld on <IP_ADDRESS>:53: server misbehaving

Note: the domain name and path have been redacted.
I tried to setup dig inside the Gatus container itself but the docker image is based on scratch.
What do you expect to see?
I expect to see only errors related to the service, not Docker's internal domain name resolution.
List the steps that must be taken to reproduce this issue
The bug is not reproducible as it is random. However, here is the basic configuration.

Setup an HTTP/API check set to run every 60s
Run the application with 120s delay
At random times, the health check will failed due to a "server misbehaving" error with Docker's DNS

Version
5.7
Additional information
How can we troubleshoot this issue i.e. a problem with Docker or Gatus itself?

@tnesztler what happened? Did you manage to fix it? If so, how did you fix it?
I'm just asking in case somebody comes across the same problem in the future :)

@TwiN I reworked my entire DNS setup (I was using a DynDNS service). With the numerous calls (every minute), it was causing a lot of issues (CNAME -> CNAME (DynDNS) -> A). My public IP is changing once every few months so I manually change an A record that I own and that solved it.
Now Gatus is working as it should.


Podboat : When the title of an episode contains /, it should not add a folder

Newsboat version (copy the output of newsboat -v or the first line of git show):
newsboat 2.20.1 - https://newsboat.org/
Copyright (C) 2006-2015 Andreas Krennmair
Copyright (C) 2015-2020 Alexander Batischev
Copyright (C) 2006-2017 Newsbeuter contributors
Copyright (C) 2017-2020 Newsboat contributors

Newsboat is free software licensed under the MIT License. (Type `newsboat -vv' to see the full text.)
It bundles:
- JSON for Modern C++ library, licensed under the MIT License: https://github.com/nlohmann/json
- optional-lite library, licensed under the Boost Software License: https://github.com/martinmoene/optional-lite
- expected-lite library, licensed under the Boost Software License: https://github.com/martinmoene/expected-lite

newsboat 2.20.1
System: Linux 4.15.0-109-generic (x86_64)
Compiler: g++ 7.5.0
ncurses: ncurses 6.1.20180127 (compiled with 6.1)
libcurl: libcurl/7.58.0 GnuTLS/3.5.18 zlib/1.2.11 libidn2/2.3.0 libpsl/0.19.1 (+libidn2/2.0.4) nghttp2/1.30.0 librtmp/2.3 (compiled with 7.58.0)
SQLite: 3.22.0 (compiled with 3.22.0)
libxml2: compiled with 2.9.10

Steps to reproduce the issue:

For the following RSS podcast : https://rss.acast.com/les-couilles-sur-la-table

One of the title of the podcasts is : Nourrir son homme : le bon steak et le joli morceau (2/2)

When enqueuing that episode to podboat, then downloading it, it will create a folder called Nourrir son homme : le bon steak et le joli morceau (2 and create the mp3 file inside that folder.

I think that this / should be escaped to avoid creating folders.

Thanks for the report, @edi9999, but this is a duplicate of #836. Please subscribe there to get a notification when it's fixed.


Register metrics defined at Resource level

Backport of #920
If a Resource class is annotated with either:

@Timed,
@Metered
@ExceptionMetered

all method defined in such class will 'inherit'
the corresponding annotation.

@jplock, @patrox: Please review.

@arteam, @jplock Sorry, I was away for few days, enjoying some short vacations and St. Patrick's day.
Thanks for pulling it in !


SelectProductNameForDirectory returns empty string when user is not in /etc/passwd

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
While running Jenkins jobs inside the mcr.microsoft.com/powershell docker image, I encountered issues when trying use Install-Module of PowerShellGet (version 2.2.5):
PS /> $ErrorActionPreference = "Stop"
PS /> Install-Module -Name PSScriptAnalyzer -Force -Verbose
Install-Module: The 'Install-Module' command was found in the module 'PowerShellGet', but the module could not be loaded. For more information, run 'Import-Module PowerShellGet'.
PS /> Import-Module PowershellGet -Verbose
VERBOSE: Loading module from path '/opt/microsoft/powershell/7/Modules/PowerShellGet/PowerShellGet.psd1'.
VERBOSE: Loading 'FormatsToProcess' from path '/opt/microsoft/powershell/7/Modules/PowerShellGet/PSGet.Format.ps1xml'.
VERBOSE: Loading module from path '/opt/microsoft/powershell/7/Modules/PowerShellGet/PSModule.psm1'.
Split-Path: Cannot bind argument to parameter 'Path' because it is an empty string.

I tracked this down to PowerShellGet's usage of SelectProductNameForDirectory.
I would expect the XDG path to still be emitted, even if the current user is not listed in /etc/passwd, since this is not uncommon in container environments.
Expected behavior
docker run --rm -u 1000 mcr.microsoft.com/powershell pwsh -Command 'Write-Host ("USER_MODULES path is <" + [System.Management.Automation.Platform]::SelectProductNameForDirectory("USER_MODULES") + ">")'
USER_MODULES path is </.local/share/powershell/Modules>

Actual behavior
% docker run --rm -u 1000 mcr.microsoft.com/powershell pwsh -Command 'Write-Host ("USER_MODULES path is <" + [System.Management.Automation.Platform]::SelectProductNameForDirectory("USER_MODULES") + ">")'
USER_MODULES path is <>

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.3.4
PSEdition                      Core
GitCommitId                    7.3.4
OS                             Linux 6.1.29-hardened1-1-hardened #1 SMP PREEMPT_DYNAMIC Wed, 17 May 2023 19:21:55 +0000
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

@237dmitry
% docker run --rm -u 1000 mcr.microsoft.com/powershell pwsh -Command '$Env:PSModulePath'
/usr/local/share/powershell/Modules:/opt/microsoft/powershell/7/Modules

Since this might become relevant later, here's the entire environment:
% docker run --rm -u 1000 mcr.microsoft.com/powershell pwsh -Command 'gci env:'

Name                           Value
----                           -----
DOTNET_SYSTEM_GLOBALIZATION_I… false
HOME                           /
HOSTNAME                       62f4f0f5d547
LANG                           en_US.UTF-8
LC_ALL                         en_US.UTF-8
PATH                           /opt/microsoft/powershell/7:/usr/local/sbin:/us…
POWERSHELL_DISTRIBUTION_CHANN… PSDocker-Ubuntu-22.04
PS_INSTALL_FOLDER              /opt/microsoft/powershell/7
PSModuleAnalysisCachePath      /var/cache/microsoft/powershell/PSModuleAnalysi…
PSModulePath                   /usr/local/share/powershell/Modules:/opt/micros…

What will return id? There are not users paths in $env:PSModulePath.
docker run --rm -u 1000 mcr.microsoft.com/powershell pwsh -Command 'id'

If id is root (0) then create /root/.local/share/powershell/Modules directory. Or if user exists users same directory. I think Install-module does not know where install modules.
Try:

create user in docker
or run with(out) -u 0
or use Install-Module -Scope AllUsers

@237dmitry My issue is not with Install-Module not working. I can work around that. It's merely the way I discovered the behavior of SelectProductNameForDirectory("USER_MODULES"), where it returns an empty string when it is unable to lookup the name for the current UID.
I consider this to be a bug. I don't think it should return an empty string.
One may argue if it should throw an error instead, though I would say that it shouldn't need to depend on that lookup at all, since $Env:HOME is defined.

Yes, this is still an issue.
% docker run --rm -u 1000 --pull always mcr.microsoft.com/powershell pwsh -Command 'Write-Host ("USER_MODULES path is <" + [System.Management.Automation.Platform]::SelectProductNameForDirectory("USER_MODULES") + ">"); $PSVersionTable'
latest: Pulling from powershell
Digest: sha256:31ae004d24a210506986e138c90306ae40456f26289551c132cca4f03bb6b4d6
Status: Image is up to date for mcr.microsoft.com/powershell:latest
USER_MODULES path is <>

Name                           Value
----                           -----
PSVersion                      7.4.0
PSEdition                      Core
GitCommitId                    7.4.0
OS                             Ubuntu 22.04.3 LTS
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0


safe opcode type creation

Part of #170. Make u8 -> All safe using the newtype approach; make All -> Ordinary safe by extending the existing macro to build a match-based partial conversion function. What do you think?

Oh, rust 1.14.

Hi, it's possible to make your PR work with 1.14.0 (travis). I have not thought about it in more depth, so I don't know if it is actually a good fix for #170, but the 1.14.0 compiler issues shouldn't be a problem. Feel free to reopen your PR :)

@sgeisler Thanks!

I didn't notice this PR. I think it's great.

@apoelstra Since the branch associated with this PR is from @kazcw's repository I don't think so. But I could open a new one with the fix applied.

Closing in favor of #198


protect from forgery so that verify authenticity is defined

When running with Ruby 3.2.1 and Rails <IP_ADDRESS> I get an ArgumentError (Before process_action callback :verify_authenticity_token has not been defined). When I add protect_from_forgery to  subscriber_controller.rb it pulls in the definitions from request_forgery_protection.rb from the ActionController::RequestForgeryProtection module


InvalidParameterCombination: Cannot find version 8.0.mysql_aurora.3.02.2 for aurora

Hello,
We are using this module(version 7.2.1) to create RDS MySQL aurora cluster and we recently upgraded to engine version to "8.0.mysql_aurora.3.02.2" manually. The highest version this module supports is 5.7.mysql_aurora Is there a solution to accommodate 8.0.
Any recommendations you can provide on this?
Thank You!

The highest version this module supports is 5.7.mysql_aurora
This module does not restrict which versions are available - where are you seeing this?

: InvalidParameterCombination: Cannot find version 8.0.mysql_aurora.3.02.2 for aurora
This is the error we see in the code build when trying to upgrade the version from MySQL 5.6 to MySQL 8.0

This is not related to the module

Error: creating RDS Cluster: InvalidParameterCombination: Cannot find version 8.0.mysql_aurora.3.02.2 for aurora
│   status code: 400, request id: 2c000dc4-49ce-4b4e-806a-956ed4a4630b
│
│   with module.aws_rds_cluster.this[0],
│   on .terraform/modules/rds_cluster/main.tf line 47, in resource "aws_rds_cluster" "this":
│   47: resource "aws_rds_cluster" "this" {

This issue is resolved. The engine name for Aurora MySQL 2.x is aurora-mysql; the engine name for Aurora MySQL 1.x continues to be aurora.
so in my case


UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd6 in position 0:

I used another dataset for training，for Chinese to English. After model trained，when I run sample_ensemble.py to translate a chinese text, something wrong happen.
2019-05-17 09:35:00.359418: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1103] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 1409 MB memory) -> physical GPU (device: 0, name: GeForce 940MX, pci bus id: 0000:02:00.0, compute capability: 5.0)
[17/05/2019 09:35:00] <<< Loading optimized model... >>>
[17/05/2019 09:35:03] <<< Optimized model loaded. >>>
[17/05/2019 09:35:03] <<< Model loaded in 9.3835 seconds. >>>
[17/05/2019 09:35:03] <<< Loading Dataset instance from datasets\Dataset_ZhEnTrans_zhen.pkl ... >>>
[17/05/2019 09:35:03] <<< Dataset instance loaded >>>
[17/05/2019 09:35:03] Removed "val" set output with id "target_text.
Traceback (most recent call last):
File "sample_ensemble.py", line 62, in
sample_ensemble(args, params)
File "C:\Users\think\Desktop\nmt-keras-master\nmt_keras\apply_model.py", line 41, in sample_ensemble
dataset = update_dataset_from_file(dataset, args.text, params, splits=args.splits, remove_outputs=True)
File "C:\Users\think\Desktop\nmt-keras-master\data_engine\prepare_data.py", line 79, in update_dataset_from_file
overwrite_split=True)
File "c:\users\think\src\keras-wrapper\keras_wrapper\dataset.py", line 1042, in setInput
bpe_codes=bpe_codes, separator=separator, use_unk_class=use_unk_class)
File "c:\users\think\src\keras-wrapper\keras_wrapper\dataset.py", line 1693, in preprocessTextFeatures
for line in list_:
File "C:\Users\think\AppData\Local\Programs\Python\Python36\lib\codecs.py", line 711, in next
return next(self.reader)
File "C:\Users\think\AppData\Local\Programs\Python\Python36\lib\codecs.py", line 642, in next
line = self.readline()
File "C:\Users\think\AppData\Local\Programs\Python\Python36\lib\codecs.py", line 555, in readline
data = self.read(readsize, firstline=True)
File "C:\Users\think\AppData\Local\Programs\Python\Python36\lib\codecs.py", line 501, in read
newchars, decodedbytes = self.decode(data, self.errors)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd6 in position 0: invalid continuation byte

Hi @RookieCXL ,
it seems that the new file you're trying to translate is on a codification different than utf-8. I suggest you to convert your file to utf-8 for avoiding these encoding issues.
Feel free to reopen this issue if after converting your file the error persists.


Use Checksum Address for tokenAddress param

This change allows to pass non checksummed address as hardhat tasks parameters.

Hey @andrejrakic,
The tokenAddress, receiver, feeTokenAddress, and router addresses are now checksummed.
I don't have time to update the other tasks but the same issue can happen.
Thanks for this tools!


fs: reuse buffer for sync read and write in unix

Fixes: https://github.com/libuv/libuv/issues/4038
uv_fs_read and uv_fs_write can reuse caller's buffers in sync mode. In async mode, if the actual number of buffers is lesser than or equal to the size of bufsml then use that, otherwise allocate memory dynamically based on nbufs. Also, the dynamically allocated memory is freed once the operation is completed.`

cc @bnoordhuis

https://github.com/libuv/libuv/pull/4070 seems more compact code than this, but I don't have a strong argument towards either as they appear to be functionally pretty similar. This one just added a helper function that made things more verbose.

I ended up doing a more extensive refactoring (and it turned out the optimization wasn't sound for uv_fs_write) but thanks anyway for the pull request.


[SPARK-50599][SQL] Create the DataEncoder trait that allows for Avro and UnsafeRow encoding

What changes were proposed in this pull request?

Currently, we use the internal byte representation to store state for stateful streaming operators in the StateStore. This PR introduces Avro serialization and deserialization capabilities in the RocksDBStateEncoder so that we can instead use Avro encoding to store state. This is currently enabled for the TransformWithState operator via SQLConf to support all functionality supported by TWS
Why are the changes needed?

UnsafeRow is an inherently unstable format that makes no guarantees of being backwards-compatible. Therefore, if the format changes between Spark releases, this could cause StateStore corruptions. Avro is more stable, and inherently enables schema evolution.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Amended and added to unit tests
Was this patch authored or co-authored using generative AI tooling?

No

The comment link does not show the comment, which I assume the comment is outdated via new commit.
I'm merging on behalf of @brkyvz . Thanks! Merging to master.


Bug : unknown field 'UniformBucketLevelAccess' in struct literal of type storage.BucketIamConfiguration

Operating System version: Ubuntu/MacOS
Firebase SDK version: v.3.10.0
Firebase Product: auth

Steps to reproduce:
Get the latest update and you will find that cloud.google.com/go have been updated.
Error:
root/vendor/cloud.google.com/go/storage
vendor/cloud.google.com/go/storage/bucket.go:558:4: unknown field 'UniformBucketLevelAccess' in struct literal of type storage.BucketIamConfiguration
vendor/cloud.google.com/go/storage/bucket.go:727:4: unknown field 'UniformBucketLevelAccess' in struct literal of type storage.BucketIamConfiguration
vendor/cloud.google.com/go/storage/bucket.go:735:4: unknown field 'UniformBucketLevelAccess' in struct literal of type storage.BucketIamConfiguration
vendor/cloud.google.com/go/storage/bucket.go:1077:18: b.UniformBucketLevelAccess undefined (type *storage.BucketIamConfiguration has no field or method UniformBucketLevelAccess)
vendor/cloud.google.com/go/storage/bucket.go:1080:39: b.UniformBucketLevelAccess undefined (type *storage.BucketIamConfiguration has no field or method UniformBucketLevelAccess)
The issue occurs after the recent update from cloud.google.com/go when it moved from 0.47.0 to 0.48.0.

cloud.google.com/go packages are developed at https://github.com/googleapis/google-cloud-go. Please report there.


Add an option to 'Pick-fruit' on fruit trees before clearing the patch

[x] Add a new state for the fruit tree to trigger the pick fruit step
[x] Add a new option of "Pick fruit before clearing"

Think that is all thats required, can take a look at this at the weekend.

I have this working in my feature/pick-fruit as of a few weeks ago.

I have this working in my feature/pick-fruit as of a few weeks ago.


Explorer Practice: enable button should be another button like Explorer and Practice

Explorer Practice enable button is on the Explorer window, which makes sense, but is hard to discover. it should be placed next to the Explorer and Practice buttons (if there is space).

closing as part of https://github.com/Siderite/lichessTools/issues/220