Start logging successful URLs in jobs

The app should log the URLs downloaded and failed against a job.
I've noticed some sites occasionally throw timeouts (quite regularly).
To help ensure consistency we should start logging
We should start logging
urls_successful = 200
urls_failed = 40x error/timeouts/50x error
urls_skipped = remote feeds that are ignored (if applicable), see #17
urls_pending = urls yet to be processed
Current response looks like;
  "jobs": [
    {
      "id": "6e536c66-2546-492a-aa86-70211ef2072d",
      "feed_id": "f3b2b84d-7a6e-5998-963c-60289c310d28",
      "state": "running",
      "run_datetime": "2024-07-30T14:53:34.575593Z",
      "earliest_item_requested": "2020-01-01T00:00:00Z",
      "latest_item_requested": "2024-07-30T14:53:34.575178Z",
      :
      "info": ""
    }
  ]

  "jobs": [
    {
      "id": "6e536c66-2546-492a-aa86-70211ef2072d",
      "feed_id": "f3b2b84d-7a6e-5998-963c-60289c310d28",
      "state": "running",
      "run_datetime": "2024-07-30T14:53:34.575593Z",
      "earliest_item_requested": "2020-01-01T00:00:00Z",
      "latest_item_requested": "2024-07-30T14:53:34.575178Z",
      "urls": [
           "successful": [
                 "https://google.com",
                 "https://xyz.com"
            ],
            "failed": [
                  "https://123.com"
            ],
            "skipped": [],
            "pending": [],
      ],
      "info": ""
    }
  ]

@fqrious blocked by #20

@fqrious response for skipped URLs seems hardcoded:
curl -X 'GET'
'http://<IP_ADDRESS>:8000/api/v1/jobs/'
-H 'accept: application/json'
{
  "page_size": "50",
  "page_number": 1,
  "page_results_count": 1,
  "total_results_count": 1,
  "jobs": [
    {
      "id": "465a8166-c2c1-4c61-9afc-8f1a1f753260",
      "count_of_items": 20,
      "feed_id": "f3b2b84d-7a6e-5998-963c-60289c310d28",
      "urls": {
        "retrieved": [
          "https://grahamcluley.com/smashing-security-podcast-377/",
          "https://grahamcluley.com/smashing-security-podcast-371/"
        ],
        "skipped": [
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link",
          "https://example.com/blog-link"
        ],
        "failed": [],
        "retrieving": []
      },
      "state": "success",
      "run_datetime": "2024-07-31T16:51:19.647871Z",
      "earliest_item_requested": "2020-01-01T00:00:00Z",
      "latest_item_requested": "2024-07-31T16:51:19.647382Z",
      "info": ""
    }
  ]
}

Should show the actual url of skipped page

I can't, we don't have the url because these jobs ran before the update and the link for those jobs are not saved... it'll only show on new jobs.

@fqrious ah, ok. that's fine then

@fqrious it's working well, except we need to account for slight variations on the domain.
To demo, for https://www.grahamcluley.com/feed/
See how https://grahamcluley.com is retrieved but https://www.grahamcluley.com is not.
We should consider a domain i
e.g. https://www.grahamcluley.com/feed/ entered as a value should consider all urls that match the pattern
*grahamcluley.com
so posts from
http://grahamcluley.com
https://grahamcluley.com
http://www.grahamcluley.com
http://sub.grahamcluley.com
would all match.
Example of current output...
{
  "id": "3ecdce7a-94c9-4c9e-b46e-7e307443e334",
  "count_of_items": 561,
  "feed_id": "846d036f-1472-5708-bb75-f6e3b95e350f",
  "urls": {
    "retrieved": [
      "https://grahamcluley.com/ebrd-hacker-twitter/",
      "https://grahamcluley.com/feed-sponsor-av-comparatives/",
      "https://grahamcluley.com/porn-wielding-zoom-bombers-disrupt-twitter-hack-court-hearing/",
      "https://grahamcluley.com/google-porn-titles-train-station-search-results/",
      "https://grahamcluley.com/smashing-security-podcast-190-twitter-hack-arrests-email-bad-behaviour-and-fawkes-vs-facial-recognition/",
      "https://grahamcluley.com/garmin-staggers-back-online-after-ransomware-attack/",
      "https://grahamcluley.com/twitter-phone-spear-phishing/",
      "https://grahamcluley.com/a-scam-letter-warn-your-vulnerable-loved-ones-to-be-on-their-guard/",
      "https://grahamcluley.com/garmin-ransomware-attack/",
      "https://grahamcluley.com/smashing-security-podcast-189/",
      "https://grahamcluley.com/free-iphone-apple-bug-hunters/",
      "https://grahamcluley.com/smashing-security-podcast-188/",
      "https://grahamcluley.com/feed-sponsor-recorded-future-4/",
      "https://grahamcluley.com/uk-government-russia-report/",
      "https://grahamcluley.com/mitre-the-creepy-company-checking-your-fingerprints-on-facebook-for-the-us-government/"
    ],
    "skipped": [
      "https://www.grahamcluley.com/graham-cluley-on-totally-unprepared-politics-podcast/",
      "https://www.grahamcluley.com/amazon-ring-staff-spied-videos/",
      "https://www.grahamcluley.com/currys-pc-world-dixons-data-breach/",
      "https://www.bitdefender.com/box/blog/iot-news/cryptojacked-routers-reduce-78-se-asia-following-operation-goldfish-alpha#new_tab",
      "https://www.grahamcluley.com/stop-everything-update-firefox-now/",
      "https://www.tripwire.com/state-of-security/featured/man-jailed-using-webcam-rat-women-bedrooms/#new_tab",
      "https://www.grahamcluley.com/smashing-security-160-snafus-ms-word-amazon-ring-and-tiktok/",
      "https://www.grahamcluley.com/city-of-las-vegas-wakes-up-to-a-cyber-attack/",
      "https://www.grahamcluley.com/travelex-ransomware/",
      "https://www.grahamcluley.com/feed-sponsor-av-comparatives-1/",
      "https://www.grahamcluley.com/ransomware-shuts-company/",
      "https://www.grahamcluley.com/travelex-still-offline-after-discovering-malware-on-new-years-eve-and-other-banks-currency-services-are-also-affected/",
      "https://www.grahamcluley.com/smashing-security-159-rap-robbery-and-iot-holiday-hell/",
      "https://www.grahamcluley.com/feed-sponsor-recorded-future/",
      "https://www.tripwire.com/state-of-security/security-data-protection/waco-water-bill-attack-click2gov-breaches/#new_tab",
      "https://www.grahamcluley.com/smashing-security-158-the-man-behind-the-missing-cryptoqueen/",
      "https://hotforsecurity.bitdefender.com/blog/web-h


create button below the fold...

The Create button is below the fols. It would be good to have this above the fold so that it's obvious to the user

.

This is no longer an issue, the Create Button is now under the calendar


EPUB Dictionaries and Glossaries

Hello,
this is just a question regarding the e-Publishing service. Is there a way for us to submit a glossary, along with the .zip that the call requires, so that the service generates an EPUB Dictionary as per the standard?
I'm not sure whether this should be discussed at all within the scope of the project, I'm just asking out of curiosity.
Thank you!

Technically, it surely is possible. However, this month it wont fit in my schedule anymore. If still wanted, as part of the project, I can have a look at it in January.


[3.0] Contributor experience improvements

Adds a basic for-contributors getting started README for building the repo, detailing what you need to get started. Closes #2403 which also does the same.
Adds a "best effort" attempt to resolve native binaries when used as a ProjectReference.
Adds the ability to disable building for Android/iOS by adding properties to Directory.Build.props which are then used when setting TargetFrameworks

The PR check is triggering because I've modified the native project to not include an Android build if disabled. This does not warrant a rebuild.

@curin Ready for review.

@joskuijpers hopefully this helps? PTAL

Good changes! Playground now runs without SDL DLL issues, and the contributor document was very useful too.


Directly use conda environment in Actions workflows

Tox slows things down, and adds an additional layer that makes it harder to understand what is going on. For CI especially we would be better served by having the conda environment installed on the runner, and directly running pytest, et al. To keep it fast we will want the conda environment cached.
This blog post goes into a way to do that, but I don't particularity like it, and it also uses actions that are not available within the organisation. I rather think we are better using a very similar approach to pull-request-checks.yaml.txt, with some tweaks from the miniconda action documentation.
The key things are that the workflow is installed by default to ${{ env.CONDA }}/envs, the environment lock file should be the hash key, and rather than messing about with login shells the conda environment can be directly added to PATH with:
echo "${{ env.CONDA }}/envs/my-conda-env/bin" >> $GITHUB_PATH

After this work we should see about parallelising the testing of the different python versions.

The main advantage of Tox is environment isolation, but in CI we are running on a clean system each time anyway.

Composite actions can't be used as the action repository is required to be public.


GH-36541: [Python][CI] Fixup nopandas build after merge of GH-33321

See https://github.com/apache/arrow/pull/35656/files#r1257540254
https://github.com/apache/arrow/pull/36542 fixed the no-pandas build by actually not having pandas installed, but some PRs merged at the same time introduced new failures for this build.

Closes: #36541

@github-actions crossbow submit python


Add negative test case for duplicate virtual services

For https://github.com/istio/istio/issues/30321. Just asserting the
current, broken, behavior
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
6d0cd65756313e87721bcb6aac2adf6b47c5c109
link
/test integ-multicluster-k8s-tests_istio

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

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
6d0cd65756313e87721bcb6aac2adf6b47c5c109
link
/test integ-multicluster-k8s-tests_istio

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

/retest

/retest

/retest

/retest


add an example yaml for access auth

https://osc.github.io/ood-documentation-test/access-oidc/
This adds a simple example for OIDC access users.
┆Issue is synchronized with this Asana task by Unito

I'm getting a build error on the branch with the following output:
sphinx-build -b html -d build/doctrees  -W source build/html
Running Sphinx v1.8.0

I can tell by the version of sphinx that's not right. We're on 2.x.  I (and the CI workflows) use the container to build. It looks like you've pip installed sphinx or a virtual environment?
here's the container & command
https://github.com/OSC/ood-documentation/blob/27e31bf1eb4a73cff78781c06c1a4b2d14822da4/.github/workflows/publish.yml#L18
here's a requirements txt if you want to install a virtual env.
https://github.com/OSC/ood-documentation-build/blob/master/requirements.txt
That said - you bring up this issue: building the docs isn't super intuitive or at least when there's N options they all need to be in sync.

I see the issue. I'm using the rake docker:build command which uses:
namespace :docker do

  desc "Build docs using docker"
  task :build do
    exec 'docker run --rm -i -t -v "${PWD}:/doc" -u "$(id -u):$(id -g)" ohiosupercomputer/docker-sphinx make html'
  end
end

And so it's that ohiosupercomputer/docker-sphinx image which doesn't work correctly and getting run each time I issue rake docker:build.
I've never had this error before which is what seems odd. Should I open an issue?
Using the ohiosupercomputer/ood-doc-build:v2.0.0 image builds as expected.

Also, I opened issue #725 to address the building discussion from earlier.

Should I open an issue?

My feeling is nearly always yes. I feel like a ticket is cheap to make and very useful long term as we're likely to forget any given oddity along the way.


ng build --prod output size bigger than ng build

1. Created a new app with cli.
2. With no other changes ran ng build which creates following bundles

chunk    {0} polyfills.bundle.js, polyfills.bundle.js.map (polyfills) 157 kB {4} [initial] [rendered]
chunk    {1} main.bundle.js, main.bundle.js.map (main) 4.03 kB {3} [initial] [rendered]
chunk    {2} styles.bundle.js, styles.bundle.js.map (styles) 9.77 kB {4} [initial] [rendered]
chunk    {3} vendor.bundle.js, vendor.bundle.js.map (vendor) 2.39 MB [initial] [rendered]
chunk    {4} inline.bundle.js, inline.bundle.js.map (inline) 0 bytes [entry] [rendered]

3. Then ran ng build --prod which by default enables aot, tree shaking, rollup etc creates bundles that are bigger than ng build (esp main.js bundle)

chunk    {0} polyfills.f52c146b4f7d1751829e.bundle.js (polyfills) 157 kB {4} [initial] [rendered]
chunk    {1} main.e088c8ce83e51568eb21.bundle.js (main) 26.8 kB {3} [initial] [rendered]
chunk    {2} styles.d41d8cd98f00b204e980.bundle.css (styles) 69 bytes {4} [initial] [rendered]
chunk    {3} vendor.4fcbac8c60aff59a60f5.bundle.js (vendor) 1.16 MB [initial] [rendered]
chunk    {4} inline.d72284a6a83444350a39.bundle.js (inline) 0 bytes [entry] [rendered]

I would expect ng build --prod output to be smaller than ng build. Is this behavior expected?

Here's a good explanation:
https://github.com/angular/angular-cli/issues/4896#issuecomment-281984124

Dupe of #4896 like @ctrl-brk said (cheers!). I'd like to add that with Angular v4.0 and up the main bundle might be bigger, but the vendor bundle is much smaller.


Build Windows native executable by Quarkus Maven plugin

Summary
We experience file access race on Windows during native build. So far it only happens when build is done directly by our FW via io.quarkus.bootstrap.app.QuarkusBootstrap. Related code is too complex, debugging would take me weeks.
From now on, we build native executable on Windows always via Quarkus Maven plugin. Whether custom build is required or not. This way we are building executable in a same manner as users will, which makes this approach better in my eyes and we might want to use it in other scenarios in the future when this solution has baked. I've tested it locally with Linux as well.
One palpable difference is that we need to properly propagate those of config properties that are build-time properties (as build is run in a new process without having system properties propagated), hence the quarkus.native.container-build change.
On Windows there can be an issue with long paths in case we have many nested project modules, but I think we can handle it with enabling long paths in case we experience it (which I don't expect because I shortened resulting path and QE TS Windows native runs just couple of tests).
Tested in #950.
Please check the relevant options

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[ ] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[ ] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[x] Example scenarios has been updated / added
[x] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

@rsvoboda I expect #950 to pass because first time CI was green except for one module where I had problems with long paths which I already solved. Please have a look so that we can see if this helped to mitigate daily build issue.

Made tiny change (one word) for future (to make re-building native executable in same application folder work in case of restart).

Commits are not squashed


Browser Console Violation

Describe the feature in detail (code, mocks, or screenshots encouraged)
It would be nice te address Violation that is showing in browser console:
"Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952"

What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
No response

I encountered this problem and here is the temporary solution I found: copy this into

;(function () {
const originalAddEventListener = EventTarget.prototype.addEventListener
EventTarget.prototype.addEventListener = function (
  type,
  listener,
  options,
) {
  if (type === 'touchstart') {
    if (typeof options !== 'object' || options === null) {
      options = { capture: !!options, passive: true }
    } else {
      options.passive = true
    }
  }
  originalAddEventListener.call(this, type, listener, options)
}

})()


Zoom behavior: clip scale to scaleExtent

Currently, it is possible to manually set a scale value outside a zoom behavior's scaleExtent.
var zoom = d3.behavior.zoom();
zoom.scaleExtent();
 > [0, Infinity]
zoom.scale(-1);
zoom.scale();
> -1
zoom.scaleExtent();
> [0, Infinity]

This seems broken to me. If you disagree, fine, close the PR. There's already a clipping function scaleTo so we just call that.

Merged into #2591 for 3.5.7.

I am out of the office until 11/09/2015.
Hello I will return to the office on Monday, 9 November but feel free to
email me or call my cellphone if this matter is urgent. Othewise, for
Safety Analytics issues you can reach Mr. Doug Packard at<PHONE_NUMBER>.
Regards,
Rudy Quiles
Note: This is an automated response to your message  "Re: [d3] Zoom
behavior: clip scale to scaleExtent (#2128)" sent on 11/2/2015 12:25:20 PM.
This is the only notification you will receive while this person is away.
**
This email and any attachments may contain information that is confidential and/or privileged for the sole use of the intended recipient.  Any use, review, disclosure, copying, distribution or reliance by others, and any forwarding of this email or its contents, without the express permission of the sender is strictly prohibited by law.  If you are not the intended recipient, please contact the sender immediately, delete the e-mail and destroy all copies.
**

I am out of the office until 11/09/2015.
Hello I will return to the office on Monday, 9 November but feel free to
email me or call my cellphone if this matter is urgent. Othewise, for
Safety Analytics issues you can reach Mr. Doug Packard at<PHONE_NUMBER>.
Regards,
Rudy Quiles
Note: This is an automated response to your message  "Re: [d3] Zoom
behavior: clip scale to scaleExtent (#2128)" sent on 11/2/2015 12:25:21 PM.
This is the only notification you will receive while this person is away.
**
This email and any attachments may contain information that is confidential and/or privileged for the sole use of the intended recipient.  Any use, review, disclosure, copying, distribution or reliance by others, and any forwarding of this email or its contents, without the express permission of the sender is strictly prohibited by law.  If you are not the intended recipient, please contact the sender immediately, delete the e-mail and destroy all copies.
**


C24 indoor cameras not working correctly

I just added some C24 indoor cameras to my setup and when I refresh the integration in home assistant they show up as entities fine, but none of the toggle buttons associated with the cameras do anything.
For example trying to toggle the Enable switch turns of for a split second before turning back on. I also cannot change the armed status of these cameras no matter what I do.
I have a battery powered eufy doorbell connected to my hub 2 which works fine with both these use cases.

I've got exactly the same Problem.

Doorbell is working. (T8200)
Indoor Cam is not.  (T8410 - P24 - Indoor Cam Pan&Tilt)

In Eufy Addon Log the Indoor Cam moreover shows the following Error:
ERROR P2PClientProtocol.lookup(): station: T8410P31220916BB - All address lookup tentatives failed.

I replaced the C24 indoor cams with C22 outdoors and they are working fine FYI.

Well I don’t want to replace my indoor cam by another product. So this won’t help me.


Self invite button does not work half the time

I have replaced Vrchat menu usage with VRCX completelly to Self invite myself to instances i want to join
Used to be super easy, now however the quick action self invite button does not appear to work all the time, causing to go trough more menuing or in worst cases have to use Vrchats own wolrd or friends list to join on friends.
When the button does not work it does not give the self invite text hovering over it.
Request this to be fixed soon

self invite was broken by a recent change, update to VRCX nightly and try again.

That seemed to help ty.


Harbor clair not starting

@granth7 saw that when deploying a Loopie cluster, Harbor clair was stuck. The error was related to not being able to connect to the database.
We might want to wait for the cluster-operator to really be done configuring everything before installing this stuff.

This is halfway done. The services and have been created, but it not implemented in setup yet. Moving back to backlog.

Harbor credentials are currently hard coded and should be created by the setup job

DONE


bash --html

When attempting to run the above Bash - az webapp up --location westeurope --name <app_name> --html , I am receiving the following error: az: error: unrecognized arguments: --html
Please advise and thank you,  Jeff Carter

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

ID: a825f031-250e-034d-e552-4b0d976d88b1
Version Independent ID: 626d7451-0b8e-051e-e512-0bf90be319a0
Content: QuickStart: Create a static HTML web app - Azure App Service
Content Source: articles/app-service/app-service-web-get-started-html.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

Hi @jeffreywcarter, Thanks for the feedback! We are currently investigating and will update you shortly.


Governance fails at producing cohesive, strategic decision making and resource allocation

Dash's governance system is a double edged sword.
It allows us to come to consensus on controversial issues, protects us from centralized take over and corruption, and allows us to avoid contentious hard forks.
But on the other hand, it's our largest liability. It forces a group (which by definition are not qualified) to make the business decisions. The result is egregious spending, fragmented decision making, and a failure to act strategically. This puts Dash at a huge disadvantage when compared to projects that have more focused decision making structures.
Dash's governance is fairly primitive at this point. It has hardly been changed since it was originally developed. Many new ideas and improvements have surfaced in the realm of decentralized governance, and Dash is due for some large improvements.
Here are links to discussions and resources related to the subject:
https://www.reddit.com/r/dashpay/comments/l986xh/daos_tokenbased_governance_is_a_fantasy_extremely/
https://www.reddit.com/r/dashpay/comments/kkybkm/the_next_generation_of_dash_governance_and_issues/
https://app.dashnexus.org/proposals/dash-town-hall/overview
https://www.dash.org/forum/threads/pre-proposal-solving-the-free-money-treasury-problem.51107/

Cardano built upon Dash's governance system, and improved it by lowering the barrier to entry. Anyone who holds ADA can stake and vote with it.
https://cardano.org/governance/
Polkadot took it even further, everyone with DOT can stake and vote, except that Polkadot introduced a council to represent the DOT holders.
https://wiki.polkadot.network/docs/en/learn-governance

An introduction to decentralized governance on Cardano
https://forum.cardano.org/t/an-introduction-to-decentralized-governance-on-cardano/44590
Governance and Cardano
https://www.youtube.com/watch?v=7_zIzjWZHgg&ab_channel=CharlesHoskinson

The topic came up again here (long thread) and here.
The new idea is to have a proposal for creating a Dash experts group that communicates the strategic priorities, its opinion on proposals to the community/MNOs so they can make an informed decision. This is effectively a lightweight version of townhall that doesn't require any changes to the existing governance system or protocol.

On Governance (The Nick Szabo Video)
https://www.youtube.com/watch?v=w2bhIQdzeI4

The Dash Advisory Board proposal attempts to help with this issue.

VeChain implements a governance system around a community-elected Steering Committee

https://docs.vechain.org/thor/learn/#governance


Naming a field as "source" will cause the error E0599

Code as follows:
use thiserror::Error;

#[derive(Error, Debug)]
enum CustomError {
    #[error("Unable to convert value from {source} to {destination}")]
    UnableConvert { source: String, destination: String },
}

fn main() {
    println!("Hello, world!");
}

Running cargo check will cause the error E0599:
error[E0599]: the method `as_dyn_error` exists for reference `&String`, but its trait bounds were not satisfied
   --> src/main.rs:6:21
    |
6   |     UnableConvert { source: String, destination: String },
    |                     ^^^^^^ method cannot be called on `&String` due to unsatisfied trait bounds
    |
   ::: /path to/rustlib/src/rust/library/alloc/src/string.rs:279:1
    |
279 | pub struct String {
    | -----------------
    | |
    | doesn't satisfy `String: AsDynError`
    | doesn't satisfy `String: std::error::Error`
    |
    = note: the following trait bounds were not satisfied:
            `String: std::error::Error`
            which is required by `String: AsDynError`
            `&String: std::error::Error`
            which is required by `&String: AsDynError`
            `str: Sized`
            which is required by `str: AsDynError`
            `str: std::error::Error`
            which is required by `str: AsDynError`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0599`.
error: could not compile `thiserror-test`

To learn more, run the command again with --verbose.

But if I rename the source field to src, E0599 disappears this time. It looks so weird. Am I doing something wrong? Or something missing?
TFTHAOT
Jason

My bad.

The Error trait's source() method is implemented to return whichever field
has a #[source] attribute or is named source, if any. This is for
identifying the underlying lower level error that caused your error.
The #[from] attribute always implies that the same field is #[source], so
you don't ever need to specify both attributes.
Any error type that implements std::error::Error or dereferences to dyn std::error::Error will work as a source.
#[derive(Error, Debug)]
pub struct MyError {
    msg: String,
    #[source]  // optional if field name is `source`
    source: anyhow::Error,
}


fix issue #5727: Edge document with user provided key is inserted as …

…many times as the number of shards, violating the primary index

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/266/


Linkage tests are passing when they should fail

clazy passed CI in Homebrew/homebrew-core#77005, but then failed the linkage check in this CI run (Homebrew/homebrew-core#77434).
It was then given a revision bump to rebuild it, and I thought that shook off the linkage (as CI passed after the revision bump), but the new bottle still shows broken linkage:
❯ brew linkage clazy
System libraries:
  /usr/lib/libSystem.B.dylib
  /usr/lib/libncurses.5.4.dylib
  /usr/lib/libxml2.2.dylib
  /usr/lib/libz.1.dylib
Variable-referenced libraries:
  @rpath/ClazyPlugin.dylib
Broken dependencies:
<EMAIL_ADDRESS>(llvm@11)
Dependencies with no linkage:
  qt

This seems to be the step that causes it: https://github.com/Homebrew/homebrew-core/runs/2597888223#step:7:768

@carlocab Can you reproduce this locally? This feels like it may be a brew linkage bug rather than test-bot no?

brew linkage --test clazy fails for locally. I think the reason why it doesn't fail on CI is that test-bot does not uninstall llvm@11 at any point before running brew linkage --test.
At least, that was my theory when I said

This seems to be the step that causes it: https://github.com/Homebrew/homebrew-core/runs/2597888223#step:7:768

I should've explained that more clearly; sorry. brew linkage --test clazy does not fail for me when I have llvm@11 installed.
My view was that test-bot should be uninstalling llvm@11 here, but I suppose one could also argue that brew linkage --test shouldn't succeed when you have undeclared dependencies with linkage installed.

https://github.com/Homebrew/homebrew-test-bot/blob/6e16d91a51060bba094a8ec967312665e5addda5/lib/tests/formulae.rb#L472-L475 is probably the relevant code here. It's balance between correctness and timeliness. It maybe warrants being tweaked so it keeps the bottle around and reinstalls from that when needed.

@carlocab Is this still a problem?

It hasn't come up since, but I don't think it's been fixed. We can probably close this out since it's not a huge problem -- and we should get at least a warning for this with brew linkage --test --strict.

Thanks @carlocab!


Errors with arrow function declarations in class component after upgrading eslint to > 4.0

Is this a bug report?
Yes
Can you also reproduce the problem with npm 4.x?
produced with npm 4 and 5
Environment

node -v: 8.1.4
npm -v: 5.3.0
yarn --version (if you use Yarn): 0.24.6
npm ls react-scripts (if you haven’t ejected<EMAIL_ADDRESS>
Then, specify:

Operating system: MacOS 10.12.5
Browser and version (if relevant): Chrome

Steps to Reproduce
(Write your steps here:)

Create app and install eslint@3
Write a class based component like

import React, { Component } from 'react';
import logo from './logo.svg';
import './App.css';

class App extends Component {
  state = {
    msg: 'eslint please work for me'
  }

  changeMsg = () => {
    this.setState({
      msg: 'nooooo'
    })
  }

  render() {
    return (

Welcome to React

          <button onClick={() => this.changeMsg()} > Update </button>

          {this.state.msg}

    );
  }
}

export default App;

works fine.
add eslint@latest and restart.
breaks at definitions of state and changeMsg

Expected Behavior
allow those declarations
Actual Behavior

Reproducible Demo
react-eslint.zip
works as is but not after adding eslint@latest.

I don’t understand this step:

add eslint@latest and restart.

It is not supported to explicitly add eslint. You’ll get a different version which may not be compatible with the one we use.
If you want to manually manage ESLint versions, you would need to eject.
If you just want to run it separately from the development server, there’s an open PR that will allow it, but it’s not merged yet.
Hope it helps!

@gaearon sorry for commenting on a closed thread, but, there is a way to use arrow functions on class components without eslint warnings currently? I mean, I'm trying to figure out something to autobind this without declaring it on class constructor...


fix CPDB filenames

related to https://github.com/NYCPlanning/ae-cp-map/issues/44
follow-up to https://github.com/NYCPlanning/data-engineering/pull/1095
I mixed up which files needed zero-padding.
Will return to refactor this code for simplicity/readability (e.g. repetitive str(geography_number).zfill(2)), just wanted this change to be as obvious/atomic as possible first.
since these csvs are needed for version 24prelim, repeat builds on this branch here

making a couple of changes requested by AE


Debugging fails in python 2.7 when using print

Issue Type: Bug
Running python 2.7 (32bit), randomly triggers exception on 'print' .
Code to reproduce:
from __future__ import print_function
import os

for i in range(400):
    print(i)

Exception details:
  File "c:\Users\user\.vscode\extensions\ms-python.python-2018.10.1\pythonFiles\experimental\ptvsd\ptvsd\_vendored\pydevd\pydevd.py", line 1925, in main
    debugger.connect(host, port)
  File "c:\Users\user\.vscode\extensions\ms-python.python-2018.10.1\pythonFiles\experimental\ptvsd\ptvsd\_vendored\pydevd\pydevd.py", line 1283, in run
    return self._exec(is_module, entry_point_fn, module_name, file, globals, locals)
  File "c:\Users\user\.vscode\extensions\ms-python.python-2018.10.1\pythonFiles\experimental\ptvsd\ptvsd\_vendored\pydevd\pydevd.py", line 1290, in _exec
    pydev_imports.execfile(file, globals, locals)  # execute the script
  File "c:\Github\test\test_print.py", line 5, in <module>
    print(i)
  File "c:\Users\user\.vscode\extensions\ms-python.python-2018.10.1\pythonFiles\experimental\ptvsd\ptvsd\_vendored\pydevd\_pydevd_bundle\pydevd_io.py", line 30, in write
    r.write(s)
IOError: [Errno 0] Error

Extension version: 2018.10.1
VS Code version: Code 1.29.0 (5f24c93878bd4bc645a4a17c620e2487b11005f9, 2018-11-12T07:47:15.448Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5820K CPU @ 3.30GHz (12 x 3298)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.90GB (8.43GB free)

Process Argv

Screen Reader
no

VM
67%

What's the path to your python interpreter in python.pythonPath in your settings.json file?

What's the path to your python interpreter in python.pythonPath in your settings.json file?

"python.pythonPath": "c:\Python27\python.exe"
This worked perfectly till I updated VS Code this morning.

Do you have PTVSD installed in your python environment? If you do, please uninstall it.

Do you have PTVSD installed in your python environment? If you do, please uninstall it.

PTVSD is not installed. Checked with pip uninstall.
Cannot uninstall requirement ptvsd, not installed

Are you still able to replicate this issue?
What's your folder structure like?

I have made no changes, so I expect it to be the same. Yes, I can still reproduce the issue, same exception, thrown at random, at different values of 'i' each time.
Which folder structure do you need? Python27 or VS code or something else?

Which folder structure do you need? Python27 or VS code or something else?

Your project

I started a new project with Blank folder (new folder in VS code), added test.py with same two lines of code, same result.

Again, please do not use the name test.py, please use something like sample.py

Again, please do not use the name test.py, please use something like sample.py

Renaming has no effect, same result.
I also have python 3.7, when I change VScode settings to use python3, it works perfectly there.

Does it work without the print statement?

Yes, the exception is specifically on the 'print'. I replaced print with 'pass' or even writing to a file, all seems to work.  All my other code seems to work fine too.
On another machine, with a similar setup, I cannot reproduce the problem, however occasionally I get this in the console after the program ends on both machines.
Exception in thread ptvsd.stopping (most likely raised during interpreter shutdown):

This IOError exception only occurs when running in the Integrated terminal. Using the external terminal option, this is not seen.
However the shutdown exception Exception in thread ptvsd.stopping occurs in both places.

Closed on debugger side last year.


initial commit

Hi Karan, Can you please add me as a reviewer. i am unable to provide any comments.

Use Server side rendering to fetch data for homepage, PLP Pages.
Capture eslint, Stylelint and lighthouse report
Capture jest coverage


Contributor Name: rcegan
Application/Executable: Azure Connected Machine Agent - gc_worker.exe
WTF Behavior Description: Spawns a process that runs encoded Powershell strings. Triggers when the agent downloads new policies from Azure.
Link to Documentation of Behavior: N/A - I can't find any resources online that mention this behaviour.
Please provide any images for additional evidence.

Added in 6fb27f5c72e15c6aa696b46eb00db9c55db9553d


Hoqto update a did:sov

Hi there,
I want to update a did:sov and I´m not sure, whether it´s not working or I´m going the wrong way. Are there any Howto´s for this scenario.
Im getting an Error 500, which points to an internal error.
My course of action:

I copied the complete Scret, I got by creating the did into the SECRET-field.
I copied the complete DID-Document as shown by the resolver in the DID-Document-field and edited the changes.

I really appreciate this new developments of DID.
Thanks for your help!
Andreas

Hello @andreas-c-hofmann , in general your approach is correct with regard to the API and protocol, i.e. yes the idea is that you could pass the secret and the new DID document to update the DID. But unfortunately in the open-source did:sov registrar driver this isn't implemented yet (see https://github.com/decentralized-identity/uni-registrar-driver-did-sov/).
There's also a (non-open-source) service at https://godiddy.com/ where create/update/deactivate are possible for did:sov, did:indy, and others. This builds on top of the open-source DIF components.
Note that there is also an alternative architecture for a DID registrar service, where the secrets are not passed back and forth, but instead always stay on this client side, this is called "client-managed secret mode": https://identity.foundation/did-registration/#client-managed-secret-mode
Hope this helps..

Hi,thanks for your reply and explanations.Are there any DID-methods, that could be administered in complete by the open source version?Regarding godiddy, I only had a first glimpse at it. I am not sure, whether I could administer DIDs created by another registrar.Client-managed DIDs sounds quite good and is a consequent understanding of decentralized administration - the core of DIDs. I'll look it up as well. Unfortunately I do not have any developer's competence, If that were a requirement.Again, many thanksAndreas --Dr. Andreas C. Hofmannwww.andreashofmann.euMobil: +49<PHONE_NUMBER>6
-------- Ursprüngliche Nachricht --------Von: Markus Sabadello @.> Datum: 23.01.23  10:05  (GMT+01:00) An: decentralized-identity/universal-registrar @.> Cc: "Andreas C. Hofmann" @.>, Mention @.> Betreff: Re: [decentralized-identity/universal-registrar] Hoqto update a did:sov (Issue #68)
Hello @andreas-c-hofmann , in general your approach is correct with regard to the API and protocol, i.e. yes the idea is that you could pass the secret and the new DID document to update the DID. But unfortunately in the open-source did:sov registrar driver this isn't implemented yet (see https://github.com/decentralized-identity/uni-registrar-driver-did-sov/).
There's also a (non-open-source) service at https://godiddy.com/ where create/update/deactivate are possible for did:sov, did:indy, and others. This builds on top of the open-source DIF components.
Note that there is also an alternative architecture for a DID registrar service, where the secrets are not passed back and forth, but instead always stay on this client side, this is called "client-managed secret mode": https://identity.foundation/did-registration/#client-managed-secret-mode
Hope this helps..
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


PToJSVal be a parent class of IsJSVal?

This way we can have a consistent way to convert to jsval with just pToJSVal, instead of sometime using jsval sometimes using pToJSVal.
Also currently we can't use pToJSVal to convert Maybe JavaScript.Object to a nullable JSVal, because JavaScript.Object is a IsJSVal but not a pToJSVal.

Duplicate of #64

Oops wrong issue

Would this solve https://github.com/ghcjs/jsaddle-dom/issues/15 ?


Any plans to put this repo on PyPi?

It would be great if it was possible to install via:
pip install wagtailmodeladmin

I agree, that would be benefitial, and something I'd like to happen. I just haven't done it before, so wasn't sure what the process was. I'm hoping i'll have time to look into it and sort.
It might be worth mentioning that this package is currently under review for inclusion into wagtail itself (in some form or other) in version 1.2 (https://github.com/torchbox/wagtail/issues/1657). So, you might not need to have it as a dependency at all in the near future.

Hi!
Good news about inclusion in wagtail core.
It's really simple to add to pypi, since you've already got your setup.py.  I think it's just a couple of commands.  See this post: http://peterdowns.com/posts/first-time-with-pypi.html
Scroll down to near the bottom: python setup.py register -r pypitest .  It's been a while since I did it but think it's just that commands and the ones after it.
Hope helps!

:+1: Please, PyPi!

Thanks for the link @seddonym. Work is a little manic at the moment, but I'll add this to my to-do list for when things calm down.

Hi Andy,
I hope you don't mind, but I have now pushed this to Pypi as I needed it for a deployment.
I can transfer ownership of the repo over to you if you let me know your Pypi account.
In the meantime, wagtailmodeladmin can now be installed via pip install wagtailmodeladmin.
Incidentally, all I needed to do was:
git clone git@github.com:ababic/wagtailmodeladmin.git
python setup.py register
python setup.py sdist upload

Then if you make any changes, you just bump the version number up and then run:
python setup.py sdist upload

Let me know if you need any help, and I hope you don't mind me pushing it forward.

Hi @seddonym No problem at all. I was always for the idea, and even laid some initial groundwork, but have just been too distracted with other things lately.
I don't have a Pypi account, but will set one up soon and get back to you regarding the transfer. I don't even mind it being in your account... just might be a little inconvenient when it comes to getting new changes out.

No problem - just let me know and I'll make you the owner.  Thanks again!
Perhaps we can close this ticket?

Hi @seddonym. Could you transfer it over to ababic when you get chance? Thanks.

Done.  Let me know if any questions.


Controller not respecting top and height constraints

Thanks for the library.  I'm using it in two area of my app.  The first is working perfectly, the second is not respecting the top/height constraints.  In my view controller I have added a view and set the left, right, top, and height constraints (not bottom).  Height constraint is set to 400.  Top is set to master view top + 100 (master view top is safe area).
I reference the view and it's constraints with:
@IBOutlet var bottomView: UIView!
@IBOutlet var topConstraint: NSLayoutConstraint!
@IBOutlet var heightConstraint: NSLayoutConstraint!

Then I init the bottomController with:
bottomController = CTBottomSlideController(topConstraint: topConstraint, heightConstraint: heightConstraint, parent: view, bottomView: bottomView, tabController: nil, navController: nil, visibleHeight: 250)
but the view still opens all the way to top of screen.  Any help is appreciated!

Hi, I'm not with the computer right now so can't give you detailed answer quickly. But there are 2 ways of doing what you are trying to achieve:

Check setExpandedTopMargin in the readme.
If above does not work for you check out iphone x safe area support section and following that give the wrapper view whatever constraints you want.

If neither works for you let me know and I'll give you more detailed answer a bit later

Any updates?
I'll be closing the issue by tomorrow


nanos Aware models need to be interfaces for Java interop

EnableAware, LastModifiedAware, etc, need to be interfaces for Java as the Java classes already extend AbstractFObject for example.
Hitherto I've dealt with this by creating a second interface js file, for example: EnableAwareInterface.js (foam.INTERFACE).   I'm only using Java for GAE Datastore so no Java logic depends on the model properties.  In addition, this also requires the build distinguish between java and non-java files.
So for example: files.js
{name: "com/serviceecho/locale/LocaleResource", flags: ['js']},
{name: "com/serviceecho/locale/LocaleResourceInterface", flags: ['java']},
This requires passing flag 'js' to build.js, for non-java builds.

pull-request https://github.com/foam-framework/foam2/pull/350 resolves this issue.


WAL-2455: Remove id from userparams

Description
Removing id from User Params and making the passkeyName optional. If not provided, infer from the JWT token (projectUserId).
Test plan
Create a user with a passkey.

closing in favor of: https://github.com/Crossmint/crossmint-sdk/pull/587


chore: disallow optional chaining

Disallow the use of optional chaining (a?.b, a?.()), except when running on Node.js.

But after minification it is not that verbose: https://esbuild.github.io/try/#dAAwLjIxLjQALS10YXJnZXQ9ZXMyMDE2IC0tbWluaWZ5AGE/LmI

See https://github.com/vuejs/core/pull/4882#issuecomment-957064008
Additionally, in the case of a?.b?.c, it will be larger.


Tabs links not work

Hi, I have tabs with custom links. (https://codepen.io/anon/pen/dKBdGg)
When I click on test 8 (in the option menu) it does not work directly. The last one to work is test 6.
Only the first 6 links work.

Version used: 1.0.0-rc.2
Browser Name and version: Chrome V 67.0.3396.99
Operating System and version (desktop or mobile): Windows 10
Link to your project (if appropriate): https://codepen.io/anon/pen/dKBdGg

I'm not really sure what you are trying to do? Tabs doesn't support switching of tabs by hash links. You can try using the select method instead.


Custom form component and column construction commands

All good! Quick thought, I might have missed some $hidden props on some of the alias commands.

I'll check...

Yeah you did, fixed.


cataloger 'apkdb-cataloger' failed to parse entries

What happened: When running syft on the jenkins:2.60.3-alpine image, it failed to parse everything and created an error which looks like:
[0001]  INFO cataloging image
[0002]  INFO identified distro: alpine 3.5.2
[0002]  INFO building the catalog
[0003] ERROR cataloger 'apkdb-cataloger' failed to parse entries (reference={id:1895 Path:/lib/apk/db/installed}): failed to parse APK DB file: bufio.Scanner: token too long
NAME                                    VERSION                TYPE
                                                               java-archive
Java Runtime Environment                1.8.0_121              java-archive
access-modifier-annotation              1.12                   java-archive
acegi-security                          1.0.7                  java-archive
[...]

What you expected to happen: It to parse correctly, although this is an egregiously old version of jenkins using a very old version of alpine.
How to reproduce it (as minimally and precisely as possible): Using origin/main, make bootstrap && make build, then running $ syft jenkins:2.60.3-alpine -v
Anything else we need to know?: This is an older version of jenkins, but it was the last one which was published on docker hub as an official image, so it's possible someone might accidentally pull it and attempt to use it.
Environment:

syft version (use syft version -v):

Application:   syft
Version:       v0.1.0-SNAPSHOT-1fc4629
BuildDate:     2020-10-09T23:49:13Z
GitCommit:     1fc46291a6e885cee57ddb2f00ec6c74c51a63a3
GitTreeState:  dirty
Platform:      linux/amd64
GoVersion:     go1.13.8
Compiler:      gc

OS (e.g: cat /etc/os-release or similar):

$ cat /etc/os-release
NAME="Ubuntu"
VERSION="20.04.1 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.1 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

I have seen the same thing, but wanted to add that I took a brief look around and I think the issue may be that older versions of the alpine images (and any other things layered on top) may be outputting differently and producing tokens larger than what can be held in the default internal token buffer size setup by bufio.  The default token size appears to be 64KB, which does seem pretty large, and the error message seems to indicate the token buffer is not large enough.  I might try playing with it a bit more incase more information would be more helpful.

@pdevine-cb the fix that @jhujasonw put in should make it in the next release (v0.3.0). I noticed something odd with the output you posted as well:
NAME                                    VERSION                TYPE
                                                               java-archive.  (<--- this line)
Java Runtime Environment                1.8.0_121              java-archive

...it seems like the name and version is missing for one of the artifacts discovered. I created a new issue #220 to capture this.

@wagoodman Yeah, I saw that too which is why I included it, but wasn't sure if it was related to the original issue.


version each release

An autoincrementing id (or something) accessible from global object, so we can more easily debug problems. Also should be made to be reported to sentry..
Not sure how to automate it so it's persisted to release commit, preferably without us having to do anything (or commit anything). Is it doable?

Are you looking for something like https://docs.sentry.io/workflow/releases/?platform=browser#create-release?
Also can't we use the last commit hash as the build version as that will always be unique?

Thanks for the link. I meant something exposed from our app itself so that you can check the version even if nothing was thrown and logged to sentry, but we can start with that, for sure.

+1 on using the latest commit hash

@dwelle @petehunt  Sentry is already reporting the SHA where the issue was first introduced. :)

Is the result from git describe --tags --always sufficient?

Without tags it will be the start of the SHA, i.e 0ed6a96
after a version is tagged, we will see this version, i.e v0.1.1
commits after this will be in the following format v0.1.1-3-g0ed6a96 - so you have tag-commits since taggSHA

I am not familiar with the stack on this project, but playing around i see that the following in package.json
diff --git a/package.json b/package.json
index ebf6352..3a4ea3c 100644
--- a/package.json
+++ b/package.json
@@ -74,7 +74,7 @@
   "scripts": {
     "build": "npm run build:app && npm run build:zip",
     "build-node": "node ./scripts/build-node.js",
-    "build:app": "REACT_APP_GIT_SHA=$NOW_GITHUB_COMMIT_SHA react-scripts build",
+    "build:app": "REACT_APP_GIT_SHA=$NOW_GITHUB_COMMIT_SHA REACT_APP_VERSION=$(git describe --tags --always) react-scripts build",
     "build:zip": "node ./scripts/build-version.js",
     "eject": "react-scripts eject",
     "fix": "npm run fix:other && npm run fix:code",

Will let this version string be accessible in the project (as process.env.REACT_APP_VERSION), so one could expose it wherever it is needed. For instance:
diff --git a/src/components/LayerUI.tsx b/src/components/LayerUI.tsx
index 75d8304..b254991 100644
--- a/src/components/LayerUI.tsx
+++ b/src/components/LayerUI.tsx
@@ -219,6 +219,7 @@ const LayerUI = ({
                   />
                 </Island>
                 {renderEncryptedIcon()}
+
v:{process.env.REACT_APP_VERSION}

             </Stack.Col>

This might solve #1751
regards - and thank you for the work that is put into this project :love_you_gesture:

@dwelle is it fair to close this since Sentry includes the commit hash?

We still need to expose it on window so that we have some way of debugging spicier issues.

Alright that should be a small addition.

If we want to use a commit hash, I can't think of anything other than what you're doing with the i18n percentages post-push action.

Hmm sorry I lost you. The commit hash is already there as a variable. All we need is to expose it to the global window object.


recursion_limit intended use is unclear

Description
This issue may not actually be a bug but i've found the recursion_limit option on the parser to not work the way I expect. My expectation is that this feature is guarding against stack overflows and should correlate to the number of stack frames deep the parser is while parsing but this doesn't appear to be the case.
For example, the following query fails when the recursion limit is set to 5:
    {
        field
        field2
        field3
        field4
        field5
        field6
    }

    let parser = Parser::new(QUERY)
        .recursion_limit(5)
        .parse()

When digging into the code, it appears that sibling fields happen sequentially (allocating a field(p) stack, returning, and then allocating another one for the next field) meaning that I would expect the maximum recursion depth to be ~2 for this query but instead I get Iter([ERROR@75:75 "parser limit(5) reached" ]).
I admit this could be my misunderstanding this feature but I'd appreciate some clarity on how this feature should be used.
Steps to reproduce
You can see the full example here.
Expected result
Sibling fields should reset the recursion depth after returning for a single selection set.
Actual result
Sibling fields do not reset the recursion depth and selection sets with many fields (possibly an arbitrarily large number given the ability to use aliases) hit the recursion limit.
Environment

Operating system and version: macos
Shell (bash/zsh/powershell): fish
apollo-rs crate: parser
Crate version: HEAD as of 2022/04/04

It seems like this is fixed by the pending PR: https://github.com/apollographql/apollo-rs/pull/462.

You're correct about what the recursion limit is for. Currently field parsing is implemented with recursion:
https://github.com/apollographql/apollo-rs/blob/8d071eb94def9452e9ed39f5b8381d7908b63484/crates/apollo-parser/src/parser/grammar/field.rs#L45-L50
So it's intended that it increases on sibling fields.
I do want to get rid of that recursion though, ideally the field() parser would only parse one field at a time and multiple fields would be handled by the caller.

I see the code you linked in selection.rs now--maybe it would already work if we just got rid of the recursion in field(), 😆


It's not clear if the render? method should be public or private

This came up in a discussion when reviewing a few PR's, it's not clear if render? should be a public or a private method. This means we end up with some components defining it publicly, and other methods defining it privately with no real consistency.
I think this has a few side-effects

There's no clear best practice
External libraries, and view component itself can't rely on the method being publicly accessible
People end up having debates about what the visibility should be

One idea was to replace the render? method with a DSL method that allows the library to control visibility. Inspired by Rails' before_action, we could potentially do something like:
skip_render_if :method_or_predicate
render_if :method_or_predicate

This takes us one step away from view component being PORO based, but does solve some of the above problems.

Looking at the source code of ViewComponent::Base I see that #render? is public, so IMO we should keep it public in our components too.


Duplicate menu items don't get unmounted, corrupted sidebar

🐛 Bug Report
Duplicate menu items don't get unmounted
Have you read the Contributing Guidelines on issues?
yes
To Reproduce
Put two items with the same name in a sidebar

When clicking between two docs pages that use sidebars, one of the items does not get unmounted
https://www.youtube.com/watch?v=OZsxQXXqJgs
Expected behavior
fail the build, or unmount both items on transition
Actual Behavior
build silently passes, weird mysterious behavior when browsing the site later on
Your Environment

Docusaurus version used: 2.0
Environment name and version: Chrome 84
Operating system and version (desktop or mobile): mac

Reproducible Demo
https://github.com/rx-store/rx-store/tree/5028da4e162d91329b8d90de451a3036e9951496/apps/rx-store-website

Oh, this is weird 🤪
It happens only in prod or also in dev too?
I've made a PR here that should fix it for alpha 62. Please report me if alpha 64 does not fix this problem.
https://github.com/facebook/docusaurus/pull/3418


Fix two type of issues raised by mypy - implicit optional issues and arg-type mismatch

Title. Here is the mypy run failing with 24 errors - https://github.com/pytorch/data/actions/runs/8428471909/job/23081016202.
With these two changes, the mypy errors are now down to 8 [https://github.com/pytorch/data/actions/runs/8428898766/job/23082258705?pr=1234]
Changes

Add optional where it was implicit earlier
Change ignore type to arg-type

@ejguan Yes I am trying to fix all the errors. Got it down from 24 to 8. When I run the local command that you provided above, I am getting 11 errors. I will try and get to the remaining 8 in a separate PR as they warrant a bit more debugging

LGTM but noob q: where can we see that the errors decreased?


SELF_SIGNUP setting doesn't work for LDAP authentication provider

Affected module
impact the UI
Describe the bug
The authenticate page shows the option "Create Account" when using ldap provider.
For the setting AUTHENTICATION_PROVIDER , when using basic value, this works fine. But if it's ldap, it's doesn't work as expected
To Reproduce

change the AUTHENTICATION_PROVIDER config to ldap
change the AUTHENTICATION_ENABLE_SELF_SIGNUP to false
open the main page (like image attachment)

Expected behavior
When setting the AUTHENTICATION_ENABLE_SELF_SIGNUP to false, it's expect that the user can't sign in creating new user because the account must exist in LDAP provider.
Version:

OS: AlmaLinux 9.2 using docker image openmetadata/server:1.2.4
Python version:
OpenMetadata version: 1.2.4
OpenMetadata Ingestion package version: not used

Thanks same issue!


Shuffle test

Reference Issues/PRs
Fixes #152
What does this implement/fix? Explain your changes.
This changes the test for gaussian mixture shuffling by greatly increasing the number of samples generated before the shuffle. Previously, there were only 10 samples being generated for each dataset, and then they were both shuffled, before comparing to make sure the labels were not in the same order. However, the labels are binary, so by non-zero bad luck, both datasets could be rearranged in an order such that the labels looked the same.
Any other comments?

you can just set a seed to guarantee performance every time, even with low samples. I think that's preferable, as technically this can still fail (rarely)


Fix wrong description of the entry-server.tsx docs

I just noticed this small incorrectness while reading through the docs. But anyways, awesome project 🔥

Thank you


Review example preview images

Remove text from the example preview images.
This makes integration clearer in the example home page: the text of the page doesn't mess with the text of the images (no duplication, no mixed information, the preview focussed on the diagram rendering, the text on the example summary).
Previously, there were also inconsistencies between previews: some had text, others hadn't.
This change should also allow to put the title on top of the example cards. See https://github.com/process-analytics/bpmn-visualization-examples/pull/80#issuecomment-726014270
Live environment of examples
https://cdn.statically.io/gh/process-analytics/bpmn-visualization-examples/feat/review_example_preview_images/examples/index.html
Implementation notes
To generate the preview, some examples have been temporary modified (add extra margin, remove some texts, ...).
I have make the changes available in a dedicated commit which is then reverted in case of (in the future) we would like to redo the screenshots or know how they have been done.

Old Home

New Home


Builds not running?

https://travis-ci.org/simon-weber/Unofficial-Google-Music-API/builds
I checked a few of the other builds and it didn't seem that they were, either.

I ended-up with self-hosted travis-cron, which is much simpler in setup than alternative written and Ruby.


authentication problem layer 7 load balance

hello,
I have installed rancher v2.2-rc2 with my F5 big IP as load balancer layer 7 in my cluster of kubernetes, I also have deployed the longhorn v0.4 and chose the option loadbalance layer 7 to install it, the problem is that I access the URL published without asking me for authentication.
if I change the Rancher-proxy option I access Rancher's URL but I have to be authenticated in UI rancher.
Is it possible to add authentication and use LB layer 7 as being F5?

@fedeostrit
Currently, we only support authenticate through Rancher UI (using rancher-proxy option) if you want authentication in Longhorn UI.
If you want to expose Longhorn UI directly to the load balancer, then we would need to add authentication to the Longhorn itself to support that. Or you can configure your LB pointed to the Longhorn endpoint behind the Rancher-proxy for now.


To extract comments out as separate templates

Instead of leaving them in the code.

done from ea9729b3a74937153b00e9d6a364ab14273d8b07 onwards


Distorted Text

I am using RichTextFx library for code editor in my parallel computing framework. Sometimes i get distorted text in the whole text area. Is this problem bcz of JDK8u20 or from any bug in RichTextFX? Also when i press ctrl+z, Code written in codeArea gets clear. Is there any thing to prevent it from happening and adding a proper undo function? Also i will be very thankful to you if you suggest any tips to make a proper code editor for the framework.
Thanks :-)

Closing this issue as a duplicate of #88.

@TomasMikula I just want to confirm for you that the artifacts on the two words "License headers" in his screencap is also similar to what I saw with the HTML WebView JFX control. Especially when horizontally scrolling it would rip the bottom half of some sentences off and distort them, which directly contrasts with the other bugs that are more present with vertical scrolling. The bold letters look like they are missing Antialiasing.


[Patterns] VM debugging support

Items to address

[x] https://github.com/dart-lang/sdk/issues/51512

@alexmarkov can we call this closed as the only item addressed is merged?

We might need to add more tests for stepping through patterns code in the debugger.
@derekxu16 Could you take a look?


Add presets for beach volleyball pitch

Please could you add presets for beachvolleyball  pit tagged as leisure=pitch + sport=beachvolleyball + surface=sand.
It's currently recorded more than 10K, see taginfo pages:
Key:sport & sport=beachvolleyball
Thanks
http://en.wikipedia.org/wiki/en:Beach_volleyball

Thank you :-)


require has no method 'load'

I tried to run locally the following test using the command line testling test.js.
// Load testling dependency
var test = require('testling');

// Load js files
require.load('./service.js');
require.load('./js/jquery.js');

test('service.createClient()', function (t) {
    var client = Service.createClient();
    t.ok(client);
});

and I got:
TypeError: Object function require(path) {
    return Module._load(path, self);
  } has no method 'load'
    at Object.<anonymous> (/home/lbdremy/workspace/nodejs/service/test.js:5:9)
    at Module._compile (module.js:402:26)
    at Object..js (module.js:408:10)

require.load doesn't exist. I used the example available at http://testling.com/docs/#require.load and I get the same error.
I installed testling from npm, the version published on npm is it up to date?
Could you help me, thanks.

require.load shouldn't work; that's not a thing that exists (perhaps it existed when those docs were written, but hasn't in many years).


Remove cruft from OG network handling (since Client.forNetwork exists)

turns out we mostly can't remove this without rewriting, its just marginally more useless now.


fix: Provide a fallback to GET request when HEAD request fails

Fixes https://github.com/shaka-project/shaka-player/issues/5959

Now, the HEAD request (don't know why there're 2 of them) is followed be a GET request.
But in my specific case, the response status is 302 Found, meaning that the resource has temporarily moved.
And it looks like the redirection is not followed:

If I manually enter the URL https://ntg-zeop-content.netgemplatform.net/api/channels?id=coeurocean&deviceId=1d46bbfc-ffa4-55e0-cc80-b4c9a8c92478209062a1&ip=<IP_ADDRESS>%3A4352 in my browser, the redirection is followed and the .m3u8 manifest is correctly downloaded.
What do you think? Is it really a redirection issue?

@avelad Do you have any idea about my issue? Is it a problem on my side or in Shaka Player?

OK, my bad. We forgot to set Access-Control-Allow-Origin on our server.
It's now done but the GET request is not mage any longer...
I still see the two 404 errors for the HEAD request, though.

I found another edge case.
The HEAD is not authorized, the server returns a 404 status code, ending up as the following error:

The error code is 1001, whereas shaka.util.Error.Code.HTTP_ERROR is 1002.
Hence, the GET request is not done.
I think the test should be:
    if (error && (error.code == shaka.util.Error.Code.HTTP_ERROR || error.code == shaka.util.Error.Code.BAD_HTTP_STATUS)) {
      request.method = 'GET';
      const response = await netEngine.request(type, request).promise;
      mimeType = response.headers['content-type'];
    }

What do you think, @avelad?


Appstudio update test-component-pac-wvvp

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-wvvp-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
22 seconds
clone-repository

✅ Succeeded
37 seconds
build-container

✅ Succeeded
21 seconds
deprecated-base-image-check

✅ Succeeded
20 seconds
inspect-image

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
50 seconds
clamav-scan

✅ Succeeded
17 seconds
sbom-json-check

✅ Succeeded
18 seconds
optional-label-check

✅ Succeeded
18 seconds
label-check

✅ Succeeded
7 seconds
show-summary

✅ Succeeded
6 seconds
show-sbom

Pipelines as Code CI/test-component-pac-wvvp-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
13 seconds
clone-repository

✅ Succeeded
39 seconds
build-container

✅ Succeeded
15 seconds
inspect-image

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
32 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
17 seconds
label-check

✅ Succeeded
17 seconds
optional-label-check

✅ Succeeded
9 seconds
show-sbom

✅ Succeeded
9 seconds
show-summary


User Error -> StackOverflow AssertHit! (999+ times)

I managed to get a very nasty error by accidentally putting a MonoInstaller in the list of PrefabInstallers (attached to SceneContext).
This is a user error, in that I just wasn't paying enough attention when I added the installer, but it did result in Unity freezing for quite a while and then printing Zenject Exception: Assert hit! 999+ times to the console.
Aside from locking Unity up for a minute or two the error message is less than obvious (especially for a beginner).
Steps to reproduce:

Add SceneContext
Create MonoInstaller
Add MonoInstaller as component of SceneContext gameobject
Add MonoInstaller to PrefabInstaller list
Run project

Without ever having touched the prefab installer before, I'm guessing this causes it to instatiate itself in an infinite loop.
I'm running Unity 2019.1.14f1 (though I doubt that matters).

I wholeheartedly recommend that you switch to this repo:
https://github.com/svermeulen/Extenject
It is maintained by the original Zenject author and unlike ModestTree who only know how to sue, but not provide support for "their" software, he actually still maintains Zenject. Just post your question there and you will most likely get an answer.

@nexxogen wow! Thanks for the tip


chore(Analysis): delete "simp can prove this" porting notes

This PR checks all porting notes of the form "simp can prove this" and if this is indeed the case (and the proof appears to make sense), removes the associated porting note.

bors merge


[Error] PPO on a single environment

Sorry, this might be a trivial question
I am trying the recurrent PPO examples with my own single environment. In any case I will get this size error.
File "ppo_lstm", line 66, in compute
rnn_input = states.view(-1, self.sequence_length, states.shape[-1])  # (N, L, Hin): N=batch_size, L=sequence_length
RuntimeError: shape '[-1, 128, 23]' is invalid for input of size 736

23 is the observation size * 32 mini-batch size = 736
I registered the environment to vectorize it, setting the num_workers to 1, but I am not 100% sure if it worked. Making gym.vector.make() at least didn't give an error:
gym.envs.registration.register(id="PendulumNoVel-v1", entry_point="Custom_Working_Single_File_Env_With_Step_And_Reset:CustomEnv")
env = gym.vector.make("PendulumNoVel-v1", num_envs=1, asynchronous=False)
env = wrap_env(env)

I also tried without vectorizing, but it also didn`t work.
SAC is working fine of course without vectorizing.
Basic information

skrl version: 1.0.0-rc1
Python version: 3.8
OS: Win11
Torch + gym

Hi @user-1701
Can you please share the agent configuration and the rollout buffer (memory) statements?


Grid size

I would like an option to change grid size.
On a 5''+ screen I would use 5 items per row.

The value is hard coded to 4 or 5, depending on your screen size/resolution.
See:
https://github.com/seizonsenryaku/HayaiLauncher/blob/master/app/src/main/res/values/column_count.xml
And:
https://github.com/seizonsenryaku/HayaiLauncher/blob/master/app/src/main/res/values-sw600dp/column_count.xml
If you can compile you can try changing those. I agree it'd be better if this was configurable though.

I am confused. As it is now, shouldn't 5 columns be displayed for resolutions wider than 600 px? Instead I am getting 4 columns on a full HD display.

I confirm that I can switch to 5 columns by changing the hard coded value in values/column_count.xml, but my device does not pick automatically the value in values-sw600dp/column_count.xml, even if the screen is 1980x1020.

This is the only feature I miss in HayaiLauncher. It would be nice to adjust icon grid in case of low vision.


Update copyright to GLIDE-for-redis

Issue #, if available:
*Description of changes: Update copyright to GLIDE-for-redis
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

should be GLIDE-for-Redis


System instability ,crashing , slow to respond and freezes Windows 11 OS (build:22621.2506 ,Windows Feature Experience Pack 1000.22677.1000.0)

Dev Home version
0.601.297.0
Windows build number
10.0.22621.2506
Other software
OS Build Version: 10.0.22621.2506.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.23
Steps to reproduce the bug
Instabilidade no sistema ,travamento , demora para responder e congela o sistema operacional Windows 11 (build:22621.2506 ,Windows Feature Experience Pack 1000.22677.1000.0)
Expected result
Instabilidade no sistema ,travamento , demora para responder e congela o sistema operacional Windows 11 (build:22621.2506 ,Windows Feature Experience Pack 1000.22677.1000.0)
Actual result
Instabilidade no sistema ,travamento , demora para responder e congela o sistema operacional Windows 11 (build:22621.2506 ,Windows Feature Experience Pack 1000.22677.1000.0)
Included System Information
Physical Memory: 31.91GB (26.3GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.601.297.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_0.600.283.0_x64__8wekyb3d8bbwe

Bing Translate:
Steps to reproduce the bug
System instability ,crashing , slow to respond and freezes Windows 11 OS (build:22621.2506 ,Windows Feature Experience Pack 1000.22677.1000.0)
Expected result
System instability ,crashing , slow to respond and freezes Windows 11 OS (build:22621.2506 ,Windows Feature Experience Pack 1000.22677.1000.0)
Actual result
System instability, crashing, slow to respond and freezes the operating system

Hi @FormigaElectra. It sounds like you're having issues with the OS, rather than Dev Home. Please file feedback in Feedback Hub (keyboard shortcut: win+F). That will ensure that your problem gets to the right people to fix it.
If I'm misunderstanding and you do have a problem with Dev Home, please give us more information so we can understand what is wrong.

Não sei se o problema e o sistema operacional ou o próprio software .


baseline repo groups

Description
Baseline repo maintainers and codeowners
Issues Resolved
Fix #31
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

You need to invite Max Ksyunz, Yury Fridlyand and add them as maintainers before merging this PR. CODEOWNERS file is not green now.

Added per the ticket, merging now


Removed address_by_endpoint from the endpoint registry

The smart contract function address_by_endpoint allowed a malicious node
to set its address for an endpoint which it did not control. The
functionality was not used and buggy, this removes it.

Yes, thank you @err508 for pointing it out.
@hackaugusto , please rebase.


lisk-migrator is UNABLE to save node Forging Status information to the disk

Expected behavior
lisk-migrator can save node Forging Status information to the disk
Actual behavior
Unable to save the node Forging Status information to the disk, please find it below instead:
[
  {
    "address": "da0fe279669a8395d58e47a7fcda5bfbdb37f82b",
    "forging": false
  },
  {
    "address": "dea5007ff22309f3687da052b5bd8b7ae29e1ab6",
    "forging": false
  },
  {
    "address": "62dbc020b55bbb629b11f6cdb585c4f97028666e",
    "forging": false
  }
]

Steps to reproduce
I used a custom height to test the tool.
CMD: lisk-migrator --snapshot-height 20436848 --output ~/.lisk/lisk-core/config/testnet --auto-migrate-config --auto-start-lisk-core-v4
Which version(s) does this affect? (Environment, OS, etc...)
OS: Ubuntu 20
// node -v
v18.18.0
// npm -v
9.8.1
// lisk-core -v
lisk-core/3.1.0-rc.1.337536d linux-x64 node-v18.16.1
FULL TERMINAL LOG
USERNAME@HOSTNAME:~$ lisk-migrator --snapshot-height 20436848 --output ~/.lisk/lisk-core/config/testnet --auto-migrate-config --auto-start-lisk-core-v4
Verifying if backup height from node config matches snapshot height... Snapshot height matches backup height
Verifying snapshot height to be multiples of round length i.e 103... Snapshot height is valid
Verifying Lisk Core version... 3.1.0-rc.1.337536d detected
Compiling Lisk Core configuration... done
Loading Lisk Core configuration... done
Waiting for snapshot height to be finalized: [============------------------] 40% | Remaining: 153/259 | Height: 20436695/20436848 | ETA: 25m 30s
Unable to save the node Forging Status information to the disk, please find it below instead:
[
  {
    "address": "da0fe279669a8395d58e47a7fcda5bfbdb37f82b",
    "forging": false
  },
  {
    "address": "dea5007ff22309f3687da052b5bd8b7ae29e1ab6",
    "forging": false
  },
  {
    "address": "62dbc020b55bbb629b11f6cdb585c4f97028666e",
    "forging": false
  }
]
Waiting for snapshot height to be finalized: [==============================] 100% | Remaining: -2/259 | Height: 20436850/20436848 | ETA: 0
Creating database instance... done
Creating genesis assets... done
Exporting genesis block to the path /home/USERNAME/.lisk/lisk-core/config/testnet/15f0dacc1060e91818224a94286b13aa04279c640bd5d6f193182031d133df7c... done
Creating backup for old config... done
Setting config property 'system.version' to: 0.1.0.... done
Setting config property 'system.logLevel' to: info.... done
Setting config property 'network.port' to: 7001.... done
Calculating and updating config property 'genesis.minimumCertifyHeight'.... done
Validating migrated user configuration... done
Exporting user configuration to the path: /home/USERNAME/.lisk/lisk-core/config/testnet/15f0dacc1060e91818224a94286b13aa04279c640bd5d6f193182031d133df7c... done
Installing lisk-core v4... !
Error: Command failed: npm i -g<EMAIL_ADDRESS>npm ERR! code EACCES
npm ERR! syscall mkdir
npm ERR! path /usr/lib/node_modules/lisk-core
npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, mkdir '/usr/lib/node_modules/lisk-core'
npm ERR!  [Error: EACCES: permission denied, mkdir '/usr/lib/node_modules/lisk-core'] {
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'mkdir',
npm ERR!   path: '/usr/lib/node_modules/lisk-core'
npm ERR! }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.
npm ERR! A complete log of this run can be found in: /home/USERNAME/.npm/_logs/2023-09-25T00_31_46_670Z-debug-0.log
    at ChildProcess.exithandler (node:child_process:419:12)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1091:16)
    at ChildProcess._handle.onexit (node:internal/child_process:302:5)

Re-thinking about this bug report, it might also be because I started the node with the delegates config but they never forged any block (forger-info DB is empty) on this lisk-migration rc1 test server. If this is the reason, my bad.


Mac: Sync stuck at 0%

DeFi app version: 2.1.15

New install
Sync stuck at 0%
Tried to set Reindex after saving to True, but when I click it, it reconnects to peer, but when I navigate to settings again, the option is not selected.
App log attached.

Preflight Checklist

[x] I have searched DeFiChain’s Github Issue Page and none matches my issue
[x] I have read through How To Submit Issues for DeFi App

Issue Details

Current DeFi App Version:

Previous DeFi App Version:

Operating System (OS):

Screenshot(s) / screen recording(s) of issue

Actual Behavior

Expected Behavior

App log / binary log / app config

app.log

confirmed on linux and windows a well

looks some nodes currently have issues. You can add a synced one by console command. You can use my, if you like:

addnode "<IP_ADDRESS>:8555" add

Hi @inyo 	, regarding the issue Mac: sync stops at 99% workaround, if your problem still persist we can further discuss on the problem there. We would like to keep same issues in one thread. Thank you.
Hi @Darkwinde appreciate the help!


Dockerize LectureCut #1

Closes #1

Adds Dockerfile for LectureCut
Adds workflow for docker build and push to Github Container Registry
Updates README with Docker instructions

Let me know if you want any tweaks. Thanks!

Other than my few nitpicks this looks really good. If possible try to follow the style guide for future commit messages. But there is no need to change the already existing ones.

Thanks for the comments. I think I addressed all of them except the commit style requests;I don't have gitomji's unfortunately.


What is payload to be passed for string to be inferenced?

I am confused on how to pass inference request for a string object in MLServer. I am passing currently my inference requests as the following:
inference_request = {
    "id": "translation_request",
    "inputs": [
        {
            "id": "request-1",
            "name": "text",
            "datatype": "BYTES",
            "data":"Today",
            "shape": [5]
        }
    ],
}

This is the inference code I have used named in a file called inference.py:
import requests

# Inference_request above was defined in this line

endpoint = "http://localhost:8080/v2/models/enmltranslation/infer"
response = requests.post(endpoint, json=inference_request)
print(response)

My ML server model is as follows:
from transformers import AutoTokenizer
from transformers import AutoModelForSeq2SeqLM
from transformers import pipeline
from mlserver import MLModel
from mlserver import types
from mlserver.types import InferenceRequest, InferenceResponse

class Enmltrans(MLModel):
    async def load(self, model_name="Helsinki-NLP/opus-mt-en-ml"):
        self.tokenizer = AutoTokenizer.from_pretrained(model_name)
        self.model = AutoModelForSeq2SeqLM.from_pretrained(model_name)
        self.translator = pipeline(
            "translation", model=self.model, tokenizer=self.tokenizer
        )

        self.ready = True
        return self.ready

    async def predict(self, payload):
        try:
            for inp in payload.inputs:
                sample_text = inp.data
                output = self.translator(sample_text)
                translation = output[0]["translated_text"]

                assert inp.id == "request-1"
                return types.InferenceResponse(
                id=payload.id,
                model_name=self.name,
                model_version=self.version,
                outputs=[
                    types.ResponseOutput(
                        name="translated_text", datatype="BYTES", data=translation
                    )
                ],
                )
        except Exception as e:
            return {"errors": e.args[0]}

When running the command mlserver start . and running inference.py. I am getting the below error which is quite confusing to me:
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/uvicorn/protocols/http/h11_impl.py", line 366, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/uvicorn/middleware/proxy_headers.py", line 75, in __call__
    return await self.app(scope, receive, send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/fastapi/applications.py", line 261, in __call__
    await super().__call__(scope, receive, send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette/applications.py", line 112, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette/middleware/errors.py", line 181, in __call__
    raise exc
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette_exporter/middleware.py", line 135, in __call__
    await self.app(scope, receive, wrapped_send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
    raise e
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette/routing.py", line 656, in __call__
    await route.handle(scope, receive, send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette/routing.py", line 259, in handle
    await self.app(scope, receive, send)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/starlette/routing.py", line 61, in app
    response = await func(request)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/mlserver/rest/app.py", line 27, in custom_route_handler
    return await original_route_handler(request)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/fastapi/routing.py", line 227, in app
    raw_response = await run_endpoint_function(
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/fastapi/routing.py", line 160, in run_endpoint_function
    return await dependant.call(**values)
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/mlserver/rest/endpoints.py", line 59, in infer
    inference_response = await self._data_plane.infer(
  File "/home/kurianbenoy/MLServer/microservice-ai-en_ml_translation_mlserver/env/lib/python3.8/site-packages/mlserver/handlers/dataplane.py", line 63, in infer
    prediction.id = payload.id
AttributeError: 'dict' object has no attribute 'id'

Hey @kurianbenoy-sentient ,
Thanks for sharing all that extra detail. That definitely helps when answering these types of issues.
My guess is that the problem comes from the except block in your custom code, where you return a dictionary. Instead, the predict() method must always return an InferenceResponse object (since that's what's expected upstream). For cases where you want to raise an error, you can raise an exception from the MLServerError class hierarchy. This exception will get handled accordingly, returning the right HTTP error code, as well as the right message, back to the client.
For example, in your case this could look like:
from transformers import AutoTokenizer
from transformers import AutoModelForSeq2SeqLM
from transformers import pipeline
from mlserver import MLModel
from mlserver import types
from mlserver.types import InferenceRequest, InferenceResponse
from mlserver.errors import MLServerError

class Enmltrans(MLModel):
    async def load(self, model_name="Helsinki-NLP/opus-mt-en-ml"):
        self.tokenizer = AutoTokenizer.from_pretrained(model_name)
        self.model = AutoModelForSeq2SeqLM.from_pretrained(model_name)
        self.translator = pipeline(
            "translation", model=self.model, tokenizer=self.tokenizer
        )

        self.ready = True
        return self.ready

    async def predict(self, payload):
        try:
            for inp in payload.inputs:
                sample_text = inp.data
                output = self.translator(sample_text)
                translation = output[0]["translated_text"]

                assert inp.id == "request-1"
                return types.InferenceResponse(
                id=payload.id,
                model_name=self.name,
                model_version=self.version,
                outputs=[
                    types.ResponseOutput(
                        name="translated_text", datatype="BYTES", data=translation
                    )
                ],
                )
        except Exception as e:
			raise MLServerError(msg=e.args[0])

Could you try this out @kurianbenoy-sentient ?

@adriangonz I tried with code you shared with following inference script:
"""Inference model"""
import requests

inference_request = {
    "id": "request-1",
    "inputs": [
        {
            "id": "request-1",
            "name": "text",
            "datatype": "BYTES",
            "data": "Today is a great day",
            "shape": [0],
        }
    ],
}

endpoint = "http://localhost:8080/v2/models/enmltranslation/infer"
response = requests.post(endpoint, json=inference_request)
print(response.json())

I am getting a 500 internal server error with following logs as shown in link

@kurianbenoy-sentient is there any chance that you're setting the status_code kwarg of MLServerError as None? The error from that stacktrace seems to suggest that. Alternatively, you can play around with the MLServerError class, and even extend it to create your own.
The key point is that errors should be raised as exceptions, and that the predict method should always return an InferenceResponse object.

I was facing the same issue when I tried this tutorial also. Then I realized the issue was with my dependencies and on a fresh install of mlserver it downgraded starlette from 0.19 to 0.17.1. This resulted in fixing my code after following the steps @adriangonz mentioned.
The final code can be found here:
"""Translation model - Inference class - seldon MLServer"""
import json

from transformers import AutoTokenizer
from transformers import AutoModelForSeq2SeqLM
from transformers import pipeline
from mlserver import MLModel
from mlserver import types
from mlserver.codecs import StringCodec
from mlserver.types import InferenceRequest, InferenceResponse
from mlserver.errors import MLServerError

class Enmltrans(MLModel):
    async def load(self, model_name="Helsinki-NLP/opus-mt-en-ml"):
        print("Initialize model")
        self.tokenizer = AutoTokenizer.from_pretrained(model_name)
        self.model = AutoModelForSeq2SeqLM.from_pretrained(model_name)
        self.translator = pipeline(
            "translation", model=self.model, tokenizer=self.tokenizer
        )

        self.ready = True
        print("model is ready")
        return self.ready

    async def predict(self, payload):
        try:
            inputs = {}
            for inp in payload.inputs:
                inputs[inp.name] = json.loads(
                    "".join(self.decode(inp, default_codec=StringCodec))
                )
            sample_text = inputs["echo"]["text"]
            output = self.translator(sample_text)
            translation = output[0]["translation_text"]

            return types.InferenceResponse(
                id=payload.id,
                model_name=self.name,
                model_version=self.version,
                outputs=[
                    types.ResponseOutput(
                        name="translated_text",
                        datatype="BYTES",
                        data=[translation],
                        shape=[len(translation)],
                        parameters=types.Parameters(content_type="str"),
                    )
                ],
            )
        except Exception as e:
            raise MLServerError(msg=e.args[0])

That's really interesting @kurianbenoy-sentient! Thanks for sharing what solved it for you.


chore: bdd-sbhg-1585674035 to 0.0.1

chore: Promote bdd-sbhg-1585674035 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


error in codception when i use deffrent enviroment

when i try to use @$scenario->env('xxx'); in my cept file the output will be No tests executed! i did the setup of env in acceptance yml
env:
env2:
modules:
config:
WebDriver:
url: 'http://bb.dev/'
browser: 'firefox'
env3:
modules:
config:
WebDriver:
url: 'http://cc.dev/'
browser: 'firefox'

Any one can advice

sounds more like a codeception issue than webception, you can report here:
https://github.com/Codeception/Codeception/issues
and try something like: http://stackoverflow.com/questions/27634821/codeception-run-acceptance-tests-in-multiple-environments


Sliced tensor rendering different depending on system (system-gl vs mesa ?)

Investigating regression test issue with plots/label.py on Windows. The tests in question involve sliced tensors.
I see strange shadows/black sections in the render plots.
I created a tensor plot of grad_tensor from noise.silo.
Set stride to 1 in the Tensor plot atts.
Add slice operator.
Zoomed in on a cluster of the glyphs.
Here's what I see on Windows and my rhel6 desktop:

Here's the same plot from Pascal:

Not sure this is really a bug, but wanted to document it.

Investigate the poly data being generated by the tensor glyphing. It looks like it's 3d, but the slice should generated 2d.

Do any of our operators that manipulate geometry (slice, 3-slice, clip, transform) operate on the geometry of glyphs? Glyphs feel like they live down-stream of the operators in the rendering aspect of things and so don't really integrate with the operators as we might assume. For example,  a slice of a 3D vector field yields a 2D slice with 3D vectors on it, doesn't it?
Well, I just tried slicing the disp vector variable from globe.silo. Indeed, I do get a 2D slice (projected to 2D) and the vectors too are projected to 2D. However, they all kinda look like they are maybe the same length as if they were in 3D and so I am not entirely sure the vectors are being projected to 2D in this case correctly. And, when I UNcheck the slice operator's Project to 2D option.  I think I should see those vectors in their original 3D glory but just for the points that are on the slice.


ExoCTK v 1.2.5

Major updates to contam tool and limb darkening tool. Drops support for python 3.6 and 3.7.

Going to test this on test server before merging with master.


python310Packages.mlflow build failure on x86_64-linux as of 9c3e077e

Issue description
Build of python310Packages.mlflow failed on x86_64-linux as of 9c3e077edde86ef672af00d2670d4ced7877a7ec. This is currently breaking python3Packages.wandb.
error: builder for '/nix/store/1rk11sk31xyc63wxss5mc9iy81jxljyl-python3.10-mlflow-2.3.1.drv' failed with exit code 1;
       last 10 log lines:
       > Requirement already satisfied: markdown<4,>=3.3 in /nix/store/acy37gfx7yri4b9snz6rcgc70v0cmahp-python3.10-markdown-3.4.3/lib/python3.10/site-packages (from mlflow==2.3.1) (3.4.3)
       > Requirement already satisfied: gitpython<4,>=2.1.0 in /nix/store/8509vn20p4hndpxv0ycy78sdl4i26vrk-python3.10-gitpython-3.1.31/lib/python3.10/site-packages (from mlflow==2.3.1) (3.1.31)
       > Requirement already satisfied: packaging<24 in /nix/store/2043nb400zw6mxdpzm4g0cab6ac1gb7s-python3.10-packaging-23.0/lib/python3.10/site-packages (from mlflow==2.3.1) (23.0)
       > Requirement already satisfied: protobuf<5,>=3.12.0 in /nix/store/98n776b902zpp65hrh3dxiydrdpn9950-python3.10-protobuf-4.21.12/lib/python3.10/site-packages (from mlflow==2.3.1) (4.21.12)
       > Requirement already satisfied: pyyaml<7,>=5.1 in /nix/store/4pmjlni52z4kw660sjyg3z7zkyyr9dgw-python3.10-pyyaml-6.0/lib/python3.10/site-packages (from mlflow==2.3.1) (6.0)
       > Requirement already satisfied: scipy<2 in /nix/store/ahc2xnwnk7dh9zrq2ll3296kkc9p6k7v-python3.10-scipy-1.10.1/lib/python3.10/site-packages (from mlflow==2.3.1) (1.10.1)
       > Requirement already satisfied: entrypoints<1 in /nix/store/n0di0w49np6fwszww1jmimr8j1wafzaf-python3.10-entrypoints-0.4/lib/python3.10/site-packages (from mlflow==2.3.1) (0.4)
       > ERROR: Could not find a version that satisfies the requirement pyarrow<12,>=4.0.0 (from mlflow) (from versions: none)
       > ERROR: No matching distribution found for pyarrow<12,>=4.0.0
       >
       For full logs, run 'nix log /nix/store/1rk11sk31xyc63wxss5mc9iy81jxljyl-python3.10-mlflow-2.3.1.drv'.
error: 1 dependencies of derivation '/nix/store/f2y655n9r923jnp1cbfz6svabq0qk6bm-python3.10-wandb-0.15.0.drv' failed to build

full build log
python3Packages.wandb maintainers cc: @samuela
python310Packages.mlflow maintainers cc: @tbenst
Other cc: n/a
Steps to reproduce

Checkout nixpkgs at commit 9c3e077edde86ef672af00d2670d4ced7877a7ec
Run nix-build -A python310Packages.mlflow

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.0-1036-azure, Ubuntu, 22.04.2 LTS (Jammy Jellyfish), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.15.0`
 - nixpkgs: `/nix/store/x84vq9qnxgz5y6j4ccf470g5mr5dc643-source`

Contents of ~/.config/nixpkgs/config.nix:
{ allowUnfree = true; cudaSupport = true; cudaCapabilities = ["8.0"]; }

Misc.
This issue was automatically generated by nixpkgs-upkeep.

CI workflow that created this issue.
Internal tags: -4125767957708070056<PHONE_NUMBER>474297635

https://github.com/NixOS/nixpkgs/pull/229782


feat: add Host struct and convert from String and Url

Implements #15

Yes, I was also unsure about the feature flag. If there's no objections towards no feature flag I'd remove it.

thank you, I agree, no need for the feature flag

Ok, I removed the feature flag!

ping @dignifiedquire can we merge this?

Thank you, released as 0.3.3