DOM elements from home page appearing in Markdown docs page

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
When hosting a static Docusaurus site (so when no SSR occurs), navigating directly to a docs page that has been created using Markdown uses the DOM element from the home page's React Component, including any styling for that element. This does not occur when you navigate to the docs page via a link within the Docusaurus site.
e.g. Navigating via an internal link, the page looks as expected:

However, navigating directly to the docs page retains the DOM element from the main page - including the associated styling:

Reproducible demo
https://stackblitz.com/edit/github-dzhl4y?file=src%2Fpages%2Findex.tsx
Steps to reproduce

Add a Markdown docs page
Create React component for the Home Page, in Pages/index.tsx . It is the direct child element of the Layout element that seems to persist:

    <Layout>
      <div id="this-is-the-element-that-persists" >
        ...
      </div>
    </Layout>

In my example I have added styling (e.g. background-color) to that div to show clearly when it persists
3. Build using docusaurus build
4. Serve the ./build directory using a static webserver that does not do Server-Side rendering, e.g. serve (note: the bug does not occur when using docusaurus start or docusaurus serve)
5. Navigate directly to /docs/intro (the URL of the markdown page added in step 1.) via the browser, so the website is loaded afresh (note: the bug does not occur if you use internal links from the website to navigate to the docs page)
Expected behavior
The docs page should have no styling from the home page
Actual behavior
A DOM element from the home page is retained in the DOM on the docs page.
Screenshot of DOM on docs page (e.g. /docs/intro), note the div with the ID home-page-container:

That ID is created in the Home Page React Component in my example:

This causes all the associated styles of that DOM element to be applied, in my example that is a hotpink background colour:

Note: For comparison, if you navigate to the docs page via a link within the Docusaurs site, the DOM behaves correctly and looks like:

Your environment

Public source code:
Public site URL:
Docusaurus version used: 2.4.3
Environment name and version (e.g. Chrome 89, Node.js 16.4): Chrome 118, Node 16
Operating system and version (e.g. Ubuntu 20.04.2 LTS): Windows 10

Self-service

[ ] I'd be willing to fix this bug myself.

Serve the ./build directory using a static webserver that does not do Server-Side rendering, e.g. serve (note: the bug does not occur when using docusaurus start or docusaurus serve)

A static webserver that "does not do Server-Side rendering" does not mean anything to me. Docusaurus does not do SSR, it only does SSG (which technically is SSR at build time)
vercel/serve is what Docusaurus uses, but your Stackblitz is using express/serve-static, a totally different package. We can only support you using the official package through the Docusaurus integration, and if you choose to use something else, you are responsible to configure it appropriately to make it work for Docusaurus.
In any case, both do not do SSR and they juste serve static files found in the build folder.

So, what happens is:
"start": "npm run build && npm run serve-static",

On initial page load of /docs/intro, your web server will send the page at build/index.html instead of the page at build/docs/intro/index.html: that's a problem and a configuration issue in your web server that you need to fix if you absolutely want to use it with Docusaurus (or any static html deployment). This can be seen in stackblitz more easily if you use low 3g network throttling: it first shows your homepage, and then the weird intro doc.
The weird issues you see are related to React hydration mismatches between initial HTML, and what the SPA tries to hydrate. In Docusaurus v3 you would get a lot of errors in the console due to that problem, but in v2, React 17 doesn't emit any error.
So, this is not a bug, this is a bad usage/configuration of express/serve-static. I don't know this web server and can't tell you how to configure it appropriately, but if you remove the -s option the problem doesn't appear anymore.

-s, --single                        Rewrite all not-found requests to index.html

Note some npm packages are not as good as others. I'm not sure I would recommend this one based on first impression considering it's old and -s seems to be a legit option to use, but apparently doesn't work as expected 🤷♂
And reirecting to index.html is not even good: we want to serve build/404.html for these cases, with an appropriate code, which is a convention most static CDNs (Vercel, Netlify, GitHub Pages, Cloudflare Pages...) understand by default.

I maybe should have put this in the original bug report, but using express/serve-static was just coming up with a repro which can be done using the tools that are provided by https://new.docusaurus.io/. . In our actual environment we are hosting the built website with NGINX and that gives the same problem.
This still feels either like a bug or a possibility for a DX improvement - it's failing in a non-obvious way without any clear explanation to the developer around what the problem is. It should fail fast and fail loud.

In our actual environment we are hosting the built website with NGINX and that gives the same problem.

This is because your NGINX is not configured properly to serve a static deployment. We at Docusaurus are not expert at NGINX. I have never used it in my life and don't really plan to, considering using a CDN is always going to be a better option.
If your company impose you the usage of NGINX, then your company must have NGINX experts that can configure it properly to host a static deployment.
The base requirement is that your NGING is configured so that /docs/intro serves either /docs/intro.html or /docs/intro/index.html, like any decent static host does for you out of the box.
If you want to see how CDNs serve static files out of the box, here's a nice host-by-host comparison: https://github.com/slorber/trailing-slash-guide. You will have to try to mimic those using your NGINX config.

This still feels either like a bug or a possibility for a DX improvement

Docusaurus is only responsible for building a static deployment. Once this is done, you have the responsibility to host it properly. There's an infinite and ever-growing set of options to host a static deployment, each coming with its own configuration options.
We can't document all this exhaustively so if you don't use any option we recommend, you have to figure this yourself reading the documentation of your host.
You can easily google this and find a solution:

https://jvns.ca/blog/2022/01/24/hosting-my-static-sites-with-nginx/
https://tutorials.tinkink.net/en/nginx/nginx-static-file-server.html

it's failing in a non-obvious way without any clear explanation to the developer around what the problem is. It should fail fast and fail loud.

It's not possible for us to "fail fast" because... Docusaurus actually works, and succeeded to build the static ./build folder. It's not us that failed, it's you, when deploying the site.
Docusaurus v3 will be using React 18, which now lets us log React hydration errors to the console thanks to its new onRecoverableError callback.
I understand why you'd want this error to be "louder", but the truth is we can't really make it louder. React emits "recoverable" errors because most of the time these errors do not lead to significant problems (unlike in this case). Also such errors can be caused by third-party browser plugins altering the DOM, google Chrome autotranslate and other things like that. If we added a big alert('it doesn't work') then some users might be affected by it while in practice the problem does not lead to any significant problem.
So, I do agree about fail-fast, fail-loud, but in practice I don't have any technical solution to deploy this.
If you want to fail-fast, you'd have to do this on your own: deploy some bot that checks your website and tell you if there's any console error or UI bug.

Thanks for the responses.
We solved this by setting trailingSlash:true in docusaurus.config.js: https://docusaurus.io/docs/api/docusaurus-config#trailingSlash


Status Attestation Responses

Refactor the descriptions of the HTTP responses for easier reads.
Key changes:

all the possible HTTP status codes for a response are listed in the table so that readers can have a complete reference at a glimpse
avoid response body when no useful information is returned to the client (example: in 5xx errors)
error codes are listed in a dedicated table

Solves https://github.com/italia/eudi-wallet-it-docs/issues/219

Please note I tried to be more clear on the use of error codes:

Error codes are meant to provide additional information about the failure so that the User can be informed and take the appropriate action.

I wanted to enforce the concept that the error code MUST NOT revert the meaning of the http response (which is a failure).


fix: delete clusters including related subnet tags

https://issues.redhat.com/browse/KFLUXDP-7
tested via prow job: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/57763/rehearse-57763-periodic-ci-konflux-ci-e2e-tests-main-rosa-old-clusters-cleanup/1846116735363584000

Lets remove load balancers as well
https://github.com/konflux-ci/konflux-qe-definitions/blob/main/common/tasks/rosa/hosted-cp/rosa-hcp-deprovision/rosa-hcp-deprovision.yaml

Lets remove load balancers as well
https://github.com/konflux-ci/konflux-qe-definitions/blob/main/common/tasks/rosa/hosted-cp/rosa-hcp-deprovision/rosa-hcp-deprovision.yaml

agree, the code started to look really ugly, so I tried to refactor it and added that LB deletion functionality

tested via https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/57763/rehearse-57763-periodic-ci-konflux-ci-e2e-tests-main-rosa-old-clusters-cleanup/1846235122341253120


Move repos over to shared bash utils

Main scope of this is #5 for cbbr and pluto, but will have to close #6 (in a first iteration) to get there most likely. Workflow is currently working through data products one at a time and building locally, moving over to the new "global" bash utils and fixing errors as they pop up
Todo list in #5 is tracking current state of testing

Pluto built successfully!
https://github.com/NYCPlanning/data-engineering/actions/runs/5205041860
https://github.com/NYCPlanning/db-pluto/issues/447


Add shuriken variants

Task description
There is currently only 1 type of shuriken in the game, just called "Shuriken".
This weapon is crafted from iron bars, which implicitly makes it part of the iron tier of weapons, even though there are no shurikens made of other metal types.
The current shuriken should be renamed to "Iron shruiken" and new shurken items should be added for dungium, agonite and noctis metal tiers.
References/notes
New assets should be drawn using the dungeonz colour palette:

Adding new item guide: https://github.com/Arcanorum/dungeonz/blob/master/guides/items/ITEMS.md
Item configs: https://github.com/Arcanorum/dungeonz/blob/master/server/src/configs/Items.yml
Projectile configs: https://github.com/Arcanorum/dungeonz/blob/master/server/src/configs/Projectiles.yml
Crafting recipe configs: https://github.com/Arcanorum/dungeonz/blob/master/server/src/configs/CraftingRecipes.yml
Acceptance criteria

Rename the "Shuriken" item to "Iron shuriken".
Rename the "Shuriken" projectile to "IronShuriken".
Add a "DungiumShuriken" projectile type config.
Add a "AgoniteShuriken" projectile type config.
Add a "NoctisShuriken" projectile type config.
Add a dungium shuriken item config.
Add a agonite shuriken item config.
Add a noctis shuriken item config.
Add a dungium shuriken crafting recipe config.
Add a agonite shuriken crafting recipe config.
Add a noctis shuriken crafting recipe config.
Add dungium shuriken client assets for GUI icon, pickup, and projectile.
Add agonite shuriken client assets for GUI icon, pickup, and projectile.
Add noctis shuriken client assets for GUI icon, pickup, and projectile.

This would need a huge nerf on iron shurikens, cuz it does the same damage as a noctis arrow rn

Maybe not a huge one, as they have much shorter range.


Add Event as Second Parameter to onChange

any way for onChange to receive an event?
<CompactPicker
    onChange={(color, evt) => {
      evt.stopPropagation;
      //this does not work
      this.updateColor(color);
  }}
color={lessonColor}
/>

Any reason you are stopping the propagation? If you take that line out does it work?

evt is undefined, thus accessing stopPropagation causes an error.

You can do this now in<EMAIL_ADDRESS>

Using any callback with onChange in SketchPicker causes the circle to not update for me, and I don't know why. Any clues would be much appreciated.


user story 11 - pool data with the same date

Value Proposition
As a user I want to have all my data from the same date displayed on one card, so that I may have a quick overview of each day.
Description

Acception Criteria

[ ] show all data from same date on one card
[ ] newly added data with existing date shall be incorporated in existing card
[ ] graph displays all data with each date only once as label (x-axys)

Tasks

[ ] convert each attribute to an array in db model schema
[ ] when posting new data check if the same date already exists

[ ] yes: use PATCH http method
[ ] no: use POST http method

[ ] change card component to read attributes from array instead of single data sets
[ ] change graph to read attributes from array instead of single data sets

Hello @Roland-Hufnagel  I integrated the graph into the Wire Frame, so that the user story can be better understood :)

yes, Im using chart.js


rename strokeDashArray file

renames strokeDashArray.tsx to strokeDasharray.tsx to match include statements

yo Matt, got another annoying lil fix here. This time I followed your lead and just renamed the file to match what's in the codebase.  fwiw I'm running into this by running the code in a node:19.9 docker container


[docs] update cpp source line numbers

I'm just going to merge this. I highly doubt it breaks the C++ build. If this CI problem persists it'll need to be looked into.


Removed deprecated functionality

Issue #565
Removed deprecated functionality.
Changes

app.js

removed deprecated trigger endpoint, and updated function comments.

README.js

minor edits.

package.json
'Prettier' found a trailing  it didn't like.

Related Issues

Issue #563
Pull Request #564 - You may need to accept #564 prior to be able to merge this PR.

Hi @martinhalford I missed this PR. LGTM. I can't merge because of some conflicts (I think with your previous PR on the README). Could you please rebase or fix it?

Rebased / fixed and replaces by #568


Login, Forgot Password, Set New Password - UI pages done

Hi @rakeshar3796 please review and merge

ok @muralidharan92


DTC entries of note

It looks like DrugTargetCommons is full of high-throughput screening data. We should review these records and select for ones that are of note / likely value. It is hard to believe that this team has actually "curated" 14.8 million records by hand–there is certainly some bulk importing going on.
I will look through the resource and work with them to identify those records of high value / likely significance.

Addressed in #454 now.


feat: 'Companies' page layout 💼

Description ✏
Closes #536
Describe what this PR does.

Bullet points are helpful.
Screenshots are helpful, if applicable.

Type of Change 🐞

[x] Feature - A non-breaking change which adds functionality.
[ ] Fix - A non-breaking change which fixes an issue.
[ ] Refactor - A change that neither fixes a bug nor adds a feature.
[ ] Documentation - A change only to in-code or markdown documentation.
[ ] Tests - A change that adds missing unit/integration tests.
[ ] Chore - A change that is likely none of the above.

Checklist ✅

[x] I have done a self-review of my code.
[ ] I have manually tested my code (if applicable).
[ ] I have added/updated any relevant documentation (if applicable).

@Wisesofthemall I'm curious to hear reasoning for why we should have each of the parts of this page (employees, reviews, etc.) into separate pages? My initial thought was that since there isn't really too much data it's fine to implement this issue by just making a change in a single file where we just only render whatever section is selected (so even if all the data is loaded, we only show a section of it at a time for ease of use). Just curious to hear if you considered this, and which approach you think might be better here!

Given the example listed (Recap page), that page uses separate pages to render specific pieces of data, so I was just following that logic. Additionally, with the use of separate pages, I'm utilizing Server-Side Rendering (SSR) as opposed to Client-Side Rendering (CSR), and here are the benefits:
Improved Performance: SSR can reduce the initial load time, especially for users on slower networks, as the content is pre-rendered on the server.
SEO Benefits: Pages rendered on the server are better optimized for search engines, which can improve the visibility of the content.
Scalability: As data grows, separating sections into different pages can help keep each page lightweight and maintain performance.

But of course, I can refactor to Client Side Routing if its necessary


Invalidate cache after write operation

Fix #13186

Hi @apc999 Please take a look at this PR, thanks.

Codecov Report

Merging #13790 (c7b7c58) into master (dd48588) will decrease coverage by 18.89%.
The diff coverage is 0.00%.

:exclamation: Current head c7b7c58 differs from pull request most recent head 0c28581. Consider uploading reports for the commit 0c28581 to get more accurate results

@@              Coverage Diff              @@
##             master   #13790       +/-   ##
=============================================
- Coverage     36.45%   17.55%   -18.90%
+ Complexity     7387     2675     -4712
=============================================
  Files          1385     1385
  Lines         80481    80506       +25
  Branches       9801     9801
=============================================
- Hits          29340    14135    -15205
- Misses        48596    65118    +16522
+ Partials       2545     1253     -1292

Impacted Files
Coverage Δ

...c/main/java/alluxio/client/file/MetadataCache.java
0.00% <0.00%> (-85.72%)
:arrow_down:

...xio/client/file/MetadataCachingBaseFileSystem.java
0.00% <0.00%> (-84.51%)
:arrow_down:

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/alluxio/wire/AlluxioMasterInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/job/meta/JobIdGenerator.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/stress/BaseParameters.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/client/UnderStorageType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...main/java/alluxio/underfs/options/ListOptions.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 544 more

Continue to review full report at Codecov.

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

@apc999 Thanks for the review and suggestion, addressed your comments except the adding ut request, will add it soon

Can you revert the change in tests/src/test/java/alluxio/client/hadoop/FileSystemRenameIntegrationTest.java? Otherwise, LGTM

@apc999 Thanks for pointing out my bad, I revert the accidental changes, PTAL

alluxio-bot, merge this please


Update delete binding to use the isExplorerActive filter.

Updated the explorer delete binding to use the isExplorerActive filter, otherwise if you have a folder/file selected in it and hit delete (even whilst in insert mode!) you'll delete that folder.
I was getting random files being deleted whilst working and couldn't work out why. Luckily I have everything in git so it was no hassle, but could have been a lot worse. Since my keyboard has a function layer which sticks delete right on the home row, I use it a lot more than maybe other vim users who would need to move their hands to hit it.
What I'm less sure about is why I didn't hit this back when the delete was originally added, since it was added in Jan I think. Perhaps it was just because I was using the explorer less than I am now.

Definitely, I agree - I just started a pre-release for 0.3.4 which has this and #2154


Manifest for packer vms.

Add base manifest for packer VMs so that they have a base bootstrap.
Also update concat module because it was doing weird things at the previous version.

lgtm :+1:


[Bug] Synchronizing Database State Across Devices

I think I've discovered a limitation of HubPress if you use it on multiple devices. If you make changes on one device, and have a token on another device, the second device does not get a refreshed state of the updates.
I'm going to check now whether clearing Chrome Cache and Data will reset the database. I would think this rather drastic step would do this.
Update
This workaround does work for blogs that have become out of sync with reality.

@jaredmorgs it should be fixed with the last version, can you give it a try?

I'll rebase and give it a go tomorrow.
On Thu, Feb 26, 2015, 03:01 Anthonny Quérouil<EMAIL_ADDRESS>wrote:

@jaredmorgs https://github.com/jaredmorgs it should be fixed with the
last version, can you give it a try?
—
Reply to this email directly or view it on GitHub
https://github.com/HubPress/hubpress.io/issues/87#issuecomment-76002423.

@jaredmorgs is it ok for you?

Way overdue for testing this issue out @anthonny
Sorry for the delay: I have verified that database sync works flawlessly between devices now.


Add Hostname template variable

It is a really important variable, which is not documented. It is really useful if docker is used in AMI and images are cloned. Hostname is unique in every instance and can be safely used as stream name
Here is an implementation:
https://github.com/moby/moby/blob/8e610b2b55bfd1bfa9436ab110d311f5e8a74dcb/daemon/logger/loginfo.go#L83

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit 0f21912d05ed052c8b003d0396915bf2c926a657
https://deploy-preview-8558--docsdocker.netlify.com


pudl_setup does not update ~/.pudl.yml if called more than one time

Describe the bug
I made a workspace directory and called pudl_setup <workspace_dir_1>. Then I changed my mind about the location, made a new directory and called pudl_setup <workspace_dir_2>. This successfully created all the contents, but did not update ~/.pudl.yml to reflect the new (second) location. It still referenced <workspace_dir_1>.
Bug Severity
How badly is this bug affecting you?

Low: The bug isn't causing me problems, but something's still wrong here.

To Reproduce
cd <test_location>
mkdir ./test_dir_1
pudl_setup ./test_dir_1
cat ~/.pudl.yml
mkdir ./test_dir_2
pudl_setup ./test_dir_1
cat ~/.pudl.yml
Expected behavior
expected pudl_setup to adjust all config to point to the new location
Software Environment?

OS: Ubuntu 20.04
How did you install PUDL? clone; hash 2a0fe38f

whoooops! yes --clobber does the trick. Missed that somehow.


Create Teacher repositories

Create teacher abstract, mock and implementation of Teacher repository.

[ ] Abstract repository
[ ] Mock repository
[ ] Anorm repository


Allow users to specify the Azure RG to deploy into

Reference Issues or PRs

Allow users to specify (via the nebari-config.yaml) which Azure Resource Group they deploy into. This also tries to preserve backward-compatibility so if nothing is provided, the default (current) behavior will be the same.
What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Any other comments?

I would still like to add a resource group name validator.

The kubernetes test will fail until #1856 is merged.

Minor comments.  LGTM otherwise assuming it runs.  I'll approve.

@Adam-D-Lewis I tested a few permutations of this (alongside some of the changes in #1930) and things appear to be working well. I'd like to add a test for this in our integration tests.


Add conversion webhook support

Description of the change:
This PR adds support to scaffold conversion webhook in CSV,
when CRDs include conversion webhookClientConfig.
Follow-up from: #3762
Co-authored-by: Eric Stroczynski<EMAIL_ADDRESS>Motivation for the change:
Support conversion webhook scaffold in SDK
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

@varshaprasad96 this is still in-flight, correct?

Yes, this is still in my to-do list. Will pick this up and continue working on it.

/remove-lifecycle stale

key an eye on it, the kubebuilder book cronjob example always fails about the missing conversionReviewVersions when enable webhook.

@qinggniq those were added in https://github.com/kubernetes-sigs/kubebuilder/pull/2176 but not to the book tutorials. @varshaprasad96 would you mind doing that when you have a chance?

@varshaprasad96 you have to make generate with go 1.16
Aside: we may want to consider running generation targets within a golang container so env issues like this do not happen.


Support for custom trace

We want to implement tracing where we have full control to start and stop the tracing. For example we want to track the time needed from user tap Submit button in screen A until screen B fully loaded. We currently use
https://firebase.google.com/docs/perf-mon/custom-code-traces?platform=flutter#add-custom-code-traces but want to implement it in datadog.
We have tried current solutions from dd-sdk-flutter but found some restrictions:

addTiming seems to only suitable to track screen opened -> something loaded
track-user-action has restriction of 10s and will automatically stopped when user move to another screen
track-custom-resource need to be treated as http request, need url, method etc.

We also noticed that datadog android has capability of tracing, is it equivalent to firebase perf custom trace and can we implement it in flutter?
https://docs.datadoghq.com/tracing/trace_collection/dd_libraries/android/?tab=kotlin
Thanks

Hi @fahmisdk6, thanks for filing an issue.
We currently looking into ways to support this exact use case across all of the mobile SDKs, as well as eventual support for custom tracing in the Flutter SDK, but we're still working out the specifics.
For better tracking, please also make sure you reach out to your CSM and have them file a feature request. This helps us gauge interest and prioritize appropriately.
As a workaround, Note that some folks have used start / stop resource for this. While it does require a url / http request method, you can just supply constant fake values for them. Datadog should accept the resource anyway.
However, for your use case, it sounds like you want the resource to span between two views. Is that accurate?

Thank for your response, so I can conclude that the implementation is still in "gauging-interest" phase, right?

However, for your use case, it sounds like you want the resource to span between two views. Is that accurate?

Yes, the trace should be view agnostic as we only need to measure the duration between event A to event B.

Hi again @fuzzybinary, can you at least confirm if our use case can be catered by these two native implementations?
https://docs.datadoghq.com/tracing/trace_collection/dd_libraries/android/?tab=kotlin
https://docs.datadoghq.com/tracing/trace_collection/dd_libraries/ios/?tab=cocoapods
If yes, then my company consider to create unofficial plugin to implement these in flutter.

Hi @fahmisdk6,
We're a bit farther from gauging interest but since we are also still working out some particulars around implementation I've decided to use that tag for now (and also I want to make sure anyone who needs similar functionality reaches out to their CSM so we can get a clear picture of whether we're hitting everyone's needs).
Native tracing might work for your use case, yes. However, we actually stopped using native tracing in the Flutter SDK because of this bug which completely broke iOS tracing. It's possible this has been fixed on the Apple side, but I'm not sure.


Fix old or broken links on NDT test page.

Links to the NDT source were still pointing to code.google.com, which is defunct.
There was a link to "Download NDT client", but no such thing exists, and least not on the Internet2 site or Github, and the links simply pointed to the source, which was misleading.... so remove that link altogether.
The "more information" bullet was pointing redundantly to the source, and also pointing to a page at internet2.org which no longer existed.
The canonical M-Lab website address is prefixed with www, so all links should point to that domain.

This change is

@nkinkade LGTM!


Limit CI runs to 4 threads

During CI/CD runs of #307 the number of parallel tests was having an impact on overall execution time.
In order to improve performance, this updates the github actions to limit test threads to 4.
This was previously the effective limit, as no test file had more than 4 tests in it.
Fixes #310

Thanks for the update! Limiting the test runner to a single thread worked quite well on the first try: The integration tests were done in 8 minutes on Windows: https://github.com/rust-osdev/bootloader/actions/runs/3842801795/jobs/6544467654
I restarted the job for good measure, but unfortunately it hangs again on the second try: https://github.com/rust-osdev/bootloader/actions/runs/3842801795/jobs/6550043485 . So I it looks like there is really something going wrong sometimes which results in an endlessly running test.
I think the best path forward is to finish https://github.com/rust-osdev/bootloader/pull/314 first to see whether we run into some panic. After we hopefully found the issue, we can experiment with different thread counts to improve the CI's run time.


Add license and creator to schema

and additional fields that we still need to define on the Galaxy side, like

release: If workflow is maintained on github, could correspond to release tag. We already use this in the IWC to define which release to create
derived_from: useful if the workflow had been modified in Galaxy to identify what the source of the workflow is. Could be a TRS id if imported via TRS, URL if imported via URL ? Or both if available?

Done in https://github.com/galaxyproject/gxformat2/pull/83


Missing "0" before "one letter" months in "from dates"

Tool generates "from dates" with missing zero before "one letter" months, e.g. 19.2.2017 - 18.03.2017.

Thanks for reporting that issue. I'll take a look on that.

Fixed.


Add scvi hub — built on hugginface

Tests and docstrings TBD

Note to self: add latent mode to model props

Note to self: add tests and complete docstrings


Ignore DateArchive_UsesDateFromCurrentTimeSource on Travis with NetCore

@304NotModified The first PR required for the master to build again. (on Travis)

thx!


Bug: spam stops too early

The spammer prepares batches with funds, but when the first batch runs out, it’s unable to use the second batch and stops. For the following command it stops after ~3 mins
./evil-tools spammer --spammer.type guava --spammer.rate 25 --spammer.duration 10m
2024-02-01T12:16:36+01:00       INFO    EvilTools.Spammer       Errors encountered during spam:
no Fresh wallet is available: no fresh wallet is available: custom conflict batch could not be prepared: 1

2024-02-01T12:16:36+01:00       ERROR                           failed to post block: Post "http://localhost:8050/api/core/v3/blocks": context canceled

Please check all possible spam scenarios.


Caching not working (as well as it could?) when using docker...

I don't know if this problem is Docker-specific or not, but the subject says it all... for local development cycles we use Macs, but our CI/CD servers deploy via Linux.  In order for things to work properly for our devs to locally deploy I've got numerous deployments of this plugin, but we've noticed the ever-increasing time of the requirements step.  The docker log shows that it uses cache, it appears this is only for the caching of the downloaded modules from the web, before they are setup.
What I'd like is if the requirements.txt file has not changed (md5 sum or last modified time) to not go and attempt to download and re-setup the requirements, ideally not even launching docker, since docker will do the same thing over and over.  In this case, all the plugin should do is re-symlink/copy the contents in place from the requirements cache folder, then zip up the lambda package, and then clean up after itself (as it does now).
I don't know if this is a bug, or a feature request.  But when I hear that this library is "caching" requirements, I assume it's caching more than just the downloading of packages, it should be caching the "output" of the requirements step, not the "prefetch" of the requirements step.
I'd love some feedback from some other contributors if this is a solid approach.  I wouldn't mind doing this feature myself, but would like feedback before wasting time on it if it was supposed to work differently already.

#145 combined with using pipCmdExtraArgs might address the issues.
I don't currently use caching myself (deps aren't big enough to worry about it) but after the v4 relase with all these changes i'll probably try it out too.

@dschep Thanks for the feedback, I don't know if that will fix it exactly... I don't see that problem, since I can see the docker is using the cache... but it appears it's only for caching the downloaded modules, not in the properly initialized form.  And I've tried with pipCmdExtraArgs setting a custom cache dir... but maybe there's some other pip command I don't know about to do "more" caching?  I'll leave this post up and maybe do some research on pip and such... I was hoping for not dropping into docker at all if not necessary... Maybe I'll come up with something.

I don't think #145 will fix this. pip doesn't cache the results when using --cache-folder, only the downloaded packages.

Hmm. ok, I'll have to dig a bit deeper.

I haven't gotten around to trying it yet... but perhaps this is what we're looking for...?
https://github.com/paylogic/pip-accel

^ attempted to implement in the above pull request.  pip-accel seems like it would do the job, but it hasn't been kept up to date in the last 2 years and has some malfunctions on a few of the python modules I tried, so thus not a great solution.  Could use some input/feedback on that PR, and/or someone else take the idea and run with it since I'm not a great JS dev.


EditorFor generate input name empty for the collections

I retrieve an list of items with the action below
    public IActionResult Edit(int id)
    {
        ObservationRepository _obsRepo = new ObservationRepository(_db);
        List<Observation> allMsg = null;
        allMsg = _obsRepo.getObservations(id);
        ViewBag.ModeEdit = true;
        return PartialView("_Observations", allMsg);
    }

The model's view is defined as @model List<Mediagral.Core.Models.Observation>
there an example of input generated for a property with an attribut name that contain an indexed array without name.

@branciat am I correct the partial view named _Observations.cshtml contains something like the following?
@using System.Collections
@model List
...
@Html.EditorFor(m => m)

If so you may get better results using @model List<Observation> instead.  In any case please provide the view and detail the unexpected results.

@branciat Could you please share exactly what your cshtml view looks like and also the Mediagral.Core.Models.Observation type so that we can try to reproduce the issue? Thanks!

the entity
using System;
using System.Collections.Generic;
using System.ComponentModel.DataAnnotations;
using System.ComponentModel.DataAnnotations.Schema;
using Mediagral.User.Models;
namespace Mediagral.Core.Models
{
public partial class Observation
{
public int Id { get; set; }
    public string CreatedById { get; set; }

    public int AddressableId { get; set; }

    [Required, DataType(DataType.Text)]
    public string Content { get; set; }

    [Column(TypeName = "datetime2")]
    public DateTime Created { get; set; }

    public virtual MediagralUser CreatedBy { get; set; }
    public Addressable Addressable { get; set; }
}

}
and the partial view
@model List<Mediagral.Core.Models.Observation>

Observations

        @if (ViewBag.ModeEdit)
        {
            Html.BeginForm(FormMethod.Post, new { data_ajax = "true", data_ajax_mode = "replace", data_ajax_update = "#AjaxContent" });
        }

Contenu
Date
Créé par
Actions

                                @for (int i = 0; i < Model.Count; i++) {

                                        @if (ViewBag.ModeEdit)
                                        {

                                                @Html.AntiForgeryToken()
                                                @Html.EditorFor(m => m[i].Content, new { HtmlAttributes = new { @class = "form-control" } })

                                                @Html.HiddenFor(m => m[i].AddressableId)
                                                @Html.EditorFor(m => m[i].Created, new { HtmlAttributes = new { @class = "form-control" } })

                                                @Html.HiddenFor(m => m[i].Id)
                                                @Html.EditorFor(m => m[i].CreatedBy.UserName, new { HtmlAttributes = new { @class = "form-control" } })

                                        }
                                        else
                                        {
                                            @Html.DisplayFor(m => m[i].Content)
@Html.DisplayFor(m => m[i].Created)
@Html.DisplayFor(m => m[i].CreatedBy.UserName)
                                        }

 Supprimer

                                }

                                    @if (ViewBag.ModeEdit)
                                    {

 Valider

 Annuler

                                    }
                                    else
                                    {

 Editer

                                    }

        @if(ViewBag.ModeEdit)
        {
            Html.EndForm();
        }

The partial view
@model List<Mediagral.Core.Models.Observation>

            Observations

        @if (ViewBag.ModeEdit)
        {
            Html.BeginForm(FormMethod.Post, new { data_ajax = "true", data_ajax_mode = "replace", data_ajax_update = "#AjaxContent" });
        }

                                    Contenu
                                    Date
                                    Créé par
                                    Actions

                                @for (int i = 0; i < Model.Count; i++) {

                                        @if (ViewBag.ModeEdit)
                                        {

                                                @Html.AntiForgeryToken()
                                                @Html.EditorFor(m => m[i].Content, new { HtmlAttributes = new { @class = "form-control" } })

                                                @Html.HiddenFor(m => m[i].AddressableId)
                                                @Html.EditorFor(m => m[i].Created, new { HtmlAttributes = new { @class = "form-control" } })

                                                @Html.HiddenFor(m => m[i].Id)
                                                @Html.EditorFor(m => m[i].CreatedBy.UserName, new { HtmlAttributes = new { @class = "form-control" } })

                                        }
                                        else
                                        {
                                            @Html.DisplayFor(m => m[i].Content)
                                            @Html.DisplayFor(m => m[i].Created)
                                            @Html.DisplayFor(m => m[i].CreatedBy.UserName)
                                        }

                                                 Supprimer

                                }

                                    @if (ViewBag.ModeEdit)
                                    {
                                        <button type="submit" class="btn green" data-ajax="true" data-ajax-mode="replace" data-ajax-update="#AjaxContent">
                                             Valider
                                        </button>
                                        <button type="reset" class="btn yellow" data-ajax="true" data-ajax-mode="replace" data-ajax-update="#AjaxContent">
                                             Annuler
                                        </button>
                                    }
                                    else
                                    {

                                             Editer

                                    }

        @if(ViewBag.ModeEdit)
        {
            Html.EndForm();
        }

The entity
using System;
using System.Collections.Generic;
using System.ComponentModel.DataAnnotations;
using System.ComponentModel.DataAnnotations.Schema;
using Mediagral.User.Models;

namespace Mediagral.Core.Models
{
    public partial class Observation
    {
        public int Id { get; set; }

        public string CreatedById { get; set; }

        public int AddressableId { get; set; }

        [Required, DataType(DataType.Text)]
        public string Content { get; set; }

        [Column(TypeName = "datetime2")]
        public DateTime Created { get; set; }

        public virtual MediagralUser CreatedBy { get; set; }
        public Addressable Addressable { get; set; }
    }
}

@branciat which part of the generated HTML is not what you expect?  anything similarly unexpected in the HTML generated when !ViewBag.ModeEdit?

@dougbu Presumably the name attribute is empty - we should try to reproduce this. Could you please try it out? Thanks!

@dougbu
I'm at home, i live in france. I try the negation after my config is the same at work.

@dougbu
This don't change anything. I've still the attribute name empty
<input class="form-control text-box single-line" data-val="true" data-val-required="Le champ Content est requis." id="z0__Content" name="[0].Content" value="Le lundi 5 septembre 2011 13:34:55" type="text">

@branciat did you edit the HTML slightly?  everything looks as I'd expect except it should be a self-closing tag e.g. <input ... />.
and what in particular is empty?

Yes i forgot the tag  in example, no override editor template, i have the same problem with other partial view and other entity, always a collection as model.
I use visual studio 2015 ctp6 and asp mvc master beta 3.

the name attribute has value "[0].Content" in the sample you provided.  what is the problem?

Yes it's the problem for each view with a collection as model.

I add  "id" in the name of the html generated with the web development toolbox of firefox, no problem, the FromForm hydrate the entity.

I add "id" in the name of the html generated with the web development toolbox of firefox, no problem, the FromForm hydrate the entity.

Does the submission work correctly if you remove [FromForm]? What if you do [FromForm(Name = "")] (explicitly empty name).
To explain: traditional model binding does a pass where it tries first using the parameter name as the binding prefix - id[0].AddressableId, id[0].Content, etc.... If no result is found then it tries again with the the empty string as prefix - id[0].AddressableId, id[0].Content, etc.... When using an attribute like [FromForm(...)] it looks like it's requiring the parameter name as-prefix unless told otherwise.
/cc @harshgMSFT

good timing @rynowak I just reproduced this issue locally.  it occurs with or without [FromForm] on the parameter.  also tried MVC 5 (just the no attribute case) and see things working fine there.  so this is a regression from that version.
I observe the data is submitted correctly and has the expected keys in the HttpRequest.  however the List<Observation> parameter is consistently null.
[FromForm(Name = "")] is a rather counter-intuitive workaround but it works.

Is this related to https://github.com/aspnet/Mvc/issues/1865?

@harshgMSFT no, the collections are at different levels. also #1865 reproduces with MVC 5 and this bug does not.

I see, the cause of concern seems to be similar for both bugs, i think this is because in https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.ModelBinding/Binders/GenericModelBinder.cs#L29 we return a value even if there was no value bound. Which is why it does not fallback to empty prefix. We should fix this asap.

might be why the bug's assigned to someone (me in this case).  I'm debugging further...

I'm interested by the workaround [FromForm(Name = "Model")], but appear not implemented in the beta3. Only dev branch ?

@branciat is [FromForm] missing from beta3 or just the Name =?.
If the former, subclass FromFormAttribute and implement IModelNameProvider. If the latter, just ModelBinderAttribute - you won't be able to restrict model binding to form-data, but this will allow you to set a name.

@branciat we're working on a fix for this.  it should be in our dev feed later this week.
for Beta3, the workaround is to add [Bind(Prefix ="")] to your action parameter.

updating the subject because reported issue is about problems model binding collections.  HTML generation is fine (at least in this respect :smiling_imp:)

commit 94e326f

Perfect ! Thanks


Ledger comm/validate response

Codecov Report

Merging #558 (600d824) into main (73faf1e) will decrease coverage by 34.96%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #558       +/-   ##
===========================================
- Coverage   60.93%   25.97%   -34.97%
===========================================
  Files         175       24      -151
  Lines       17998     1105    -16893
  Branches     3966      185     -3781
===========================================
- Hits        10967      287    -10680
+ Misses       2847      700     -2147
+ Partials     4184      118     -4066

Flag
Coverage Δ

integration-aries-vcx
?

unittests-agency-client
25.97% <ø> (ø)

unittests-aries-vcx
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

aries_vcx/src/libindy/utils/anoncreds.rs

aries_vcx/src/libindy/utils/cache.rs

aries_vcx/src/libindy/utils/ledger.rs

...ion/prover/states/presentation_request_received.rs

...cx/src/protocols/issuance/holder/states/initial.rs

aries_vcx/src/messages/a2a/mod.rs

...n/prover/states/presentation_preparation_failed.rs

aries_vcx/src/handlers/mod.rs

...x/src/protocols/issuance/holder/states/finished.rs

...src/protocols/issuance/issuer/states/offer_sent.rs

... and 141 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Unable to inherit from multiple classes using six.add_metaclass

Originally reported by: Julien Danjou (Bitbucket: jdanjou, GitHub: Unknown)

The following code snippet does not work with six and should AFAIU.
#!python

import six

class MetaA(type): pass
class MetaB(type): pass
class MetaC(MetaA, MetaB): pass

class A(object):
    __metaclass__ = MetaA

class B(object):
    __metaclass__ = MetaB

class C(A, B):
    __metaclass__ = MetaC

# Six version
@six.add_metaclass(MetaA)
class A(object): pass

@six.add_metaclass(MetaB)
class B(object): pass

class MetaC(MetaA, MetaB): pass

# This fails
@six.add_metaclass(MetaC)
class C(A, B): pass

Bitbucket: https://bitbucket.org/gutworth/six/issue/127

I believe this issue as an instance of the caveat that benjamin highlighted here. I'm not aware of a workaround other than to use the alternate with_metaclass.

This issue is a more general description of the issue described in #118, so probably can be closed as a duplicate.

On second thought, #118 appears to be different, as the exception is not due to the class hierarchy and isn't a TypeError, as is exhibited here.


Add "java" to project description

https://www.google.com/search?q=java+gmp doesn't reach this page, and I realized the word "Java" isn't in the project description or front page, just "jna".

 Maybe @swankjesse can.


update the protobuf library to google.golang.org/protobuf/proto

The linter was complaining about deprecation of github.com/golang/protobuf/proto.

Codecov Report

Merging #2554 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2554   +/-   ##
=======================================
  Coverage   86.17%   86.17%
=======================================
  Files         122      122
  Lines        9550     9550
=======================================
  Hits         8229     8229
  Misses        983      983
  Partials      338      338

Continue to review full report at Codecov.

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

Merging without review, so that we can continue without the CI bugging us about it all the time.


Python 3 compatibility

/srv/venv/lib/python3.6/site-packages/lucid/scratch/web/svelte.py in build_svelte(html_fname)
     33     print(subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT))
     34   except subprocess.CalledProcessError as exception:
---> 35     print("svelte build failed!\n" + exception.output)
     36   return js_fname
     37

TypeError: must be str, not bytes

When using a notebooks/building-blocks/SemanticDictionary.ipynb which manually installs 0.0.5 (not the latest commit where that has been fixed)
File "/srv/venv/lib/python3.6/site-packages/lucid/scratch/web/svelte.py", line 31
    print cmd
            ^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print(t cmd)?

First problem is outstanding. The second was fixed by #33.


Upgrade org.sonarsource.scanner.maven:sonar-maven-plugin <IP_ADDRESS>4 -> <IP_ADDRESS>94

This PR contains the following updates:

Package
Type
Update
Change

org.sonarsource.scanner.maven:sonar-maven-plugin (source)
build
minor
<IP_ADDRESS>4 -> <IP_ADDRESS>94

Release Notes

SonarSource/sonar-scanner-maven (org.sonarsource.scanner.maven:sonar-maven-plugin)
v<IP_ADDRESS>94
Compare Source
Release notes - Sonar Scanner for Maven - 3.10
New Feature
MSONAR-193 Set property `sonar.java.enablePreview` when Java's enablePreview feature is set
Task
MSONAR-184 Properly escape multi-valued properties if some paths contain comma
MSONAR-187 Upgrade github-action_release to v4
MSONAR-188 Stop leaking Maven classes in the scanner engine classloader
MSONAR-189 Add ITs with java 17
MSONAR-192 Run all tests nightly
MSONAR-195 Reduce the scope of the maven-compat dependency
MSONAR-198 Add Maven 4 tests to the pipeline
MSONAR-199 Use master build for peachee jobs
Improvement
MSONAR-190 Default sources to "src/main" instead of "src/main/java"
MSONAR-197 Maven Scanner relies on old slf4j version

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade sonar-maven-plugin <IP_ADDRESS>4 -> <IP_ADDRESS>94

See:
- https://github.com/SonarSource/sonar-scanner-maven/releases/tag/<IP_ADDRESS>94
- https://github.com/SonarSource/sonar-scanner-maven/compare/<IP_ADDRESS>4...<IP_ADDRESS>94


Using 'center' as a directive name conflicts with other code

If I include the center attribute anywhere that this directive has been loaded, it causes this error:
Error: [$compile:ctreq] Controller 'leaflet', required by directive 'center', can't be found!

This occurs because I have another plugin that uses a center attribute as configuration. Would you consider namespacing this attribute as leaflet-center or something else non-conflicting?
Here is a demo link: http://jsbin.com/qurava/1/edit?html,js,console,output

Any progress or official dirtyhack on this? :)

hey Tombatossals. Any update on this issue? Would be nice to have it corrected. :)
thanks

Any update on this?
Thanks.

+1 lf-

There is no workaround that I'm aware of for this.

@tombatossals
Hi, this bug is blocking us in our current project (we need leaflet and google maps directives and they both use center). Are you working on this?
I thought I could fix that and I came up with few ideas how to make that:

As proposed before, we could add prefix to those directives (center, bounds, marker, paths). The drawback is we loose backward compability.
We could try not to force leaflet directives for marker directive (set warning and some return to prevent logic to perform. I am not sure if this will work, but that's the easiest way to do it.
We could move all logic from those directives to services, replace directives by main directive parameters. The leaflet directive syntax would be untouched, so we would still have backward compatibility. That's to hardest way to do it, but I think it is also the most elegant solution.

What do you think? Did you plan some major changes too so we could drop backward compatibility? (maybe in version 1.0? ;) ) Should I try resolve this by myself or you have been working on that?
My gmail and hangout login is aga.glabala if you are interested in longer discussion about this issue.
Agnieszka Świecznik

Any update on this? adding lf- sounds like a great idea! :)

Hey all we'll get to this; nagging won't help. remember this is free. Feel free to dig in and try to give back and fix it.

Moved to 0.9.0 milestone since this is a breaking change.

Consider adding lf-center already now and remove center later in 0.9?

Hmm.. ok we should really do all of them with lf- and leave the originals. But it honestly complicate the code now when just cutting to 0.9.0 will just be directive renames. Not sure about this; but I guess we could just do this for center.

#914 PR


Fix #4137 The UI should show the displayName instead of the name for entities.

Fixes #4137
Describe your changes :

I worked on issue #4137

Type of change :

[x] Improvement

Frontend Preview (Screenshots) :

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
@open-metadata/ui

@Sachin-chaurasiya Can you please take care of the code smell?

@Sachin-chaurasiya Can you please take care of the code smell?

@vivekratnavel , if it's not blocking then we can merge this PR, I will take care of the code smell in my next patch.


Enhancement to radio, checkboxes and conditionally revealing content

I have noticed that for all radios, checkboxes and conditionally revealing content they all assume no value has been chosen. In most cases this is good as it forces the user to make a choice.
What about the following scenario:
A user returns to a page with the ability to update an already populated form, which may or may not include radios, checkboxes and conditionally revealing content based on form selection.
I would have expected that if the input has "checked" or checked="checked" then all required styling would be applied, but this does not look to be the case.
Is this something to be added in future releases?

Following the launch of the GOV.UK Design System, GOV.UK Elements will now only get major bug fixes and security patches, so I'm going to close this.
Note that this has been implemented in GOV.UK Frontend – if a radio or checkbox is checked then the conditional content associated with it should be visible on page load.


Access to the sanic package on PyPI

I am trying to get access to the sanic package on PyPI.
Up until a few weeks ago @channelcat had maintained the package alone. He decided to move the GitHub repo to an organization and added some more maintainers. The PyPI repo has not been moved so no one has access and we have not heard from him any more.
We would like to get a release out.
Here is my account on PyPI: https://pypi.org/user/brewmaster/
In addition to the sanic repo, the community organization also maintains the sanic-openapi: https://pypi.org/project/sanic-openapi/
Can someone assist me in how we can get access to these two packages?

Hi @ahopkins! The easiest way for this to move forward would be for @channelcat to transfer ownership to your PyPI user so that you can then add additional maintainers and owners from the new organization.
If @channelcat is unavailable or unresponsive, I see the following facts:

The current homepage URL for the sanic package
http://github.com/channelcat/sanic, does forward to the repository at
https://github.com/huge-success/sanic which indicates an intentional migration by @channelcat.
Maintenance of the codebase appears to be occurring in the new repository, including activity from @ahopkins.

I think given the following we should be OK to transfer the project ownership:

A confirmation from another active maintainer of the new org (@ashleysommer, @yunstanford, @vltr, or @seemethere) that transferring to the brewmaster user on PyPI is 👍.

@dstufft @di do you have any other stipulations?

I've reached out to @channelcat for confirmation. If they don't respond in a timely fashion, I'm OK with transferring this project to you, as it seems that by transferring the Github repo, they intended for others to be able to maintain the project. Preferably we'd add some other owners of the project in addition to yourself.

@ewdurbin Sounds like we're on the same page 😂

You don't actually need access to the pypi account to do releases. Just tag a release on GitHub and it should automatically post it up to the pypi account.

Is there a concept of organizations on pypi? Would like to move ownership of this repository to an organization similar to what we have here on GitHub.

@seemethere not currently, that is being tracked in #201

@ewdurbin @di we are working very hard to maintain Sanic up and running, not only with code but creating a community centric project with more immersion from developers around the world.

+1 from me for transfer of the pypi access for the Sanic project to brewmaster.

@seemethere There is not, but there can be multiple "contributors" on pypi.
@di Thanks. I hope @channelcat does get back. This is the last communication I have had from him:
https://groups.google.com/d/msg/sanic-dev/WXbBxNo7Zek/UWisOI28BgAJ

@di Just curious ... how long is a "timely fashion"?

@ahopkins I've added you as Owner of sanic and sanic-openai. Please make sure to add at least one other maintainer as an owner so this is less likely to happen again.

Thank you. I plan on it.


[4.4 Dev 5] Empty Project High CPU usage.

Tested versions
Reproducible in: 4.4.dev5
System information
Godot v4.4.dev5 - Windows 10.0.26100 - Multi-window, 1 monitor - OpenGL 3 (Compatibility) - AMD Radeon(TM) Graphics (Advanced Micro Devices, Inc.; 32.0.12019.1028) - AMD Ryzen 7 9700X 8-Core Processor (16 threads)
Issue description
High CPU usage with an empty project. low_processor_usage_mode() does not make a difference. 1 of the threads on my CPU is at 100% usage. This applies to the editor and projects run from the editor.
Steps to reproduce
Launch the editor. Also create a Control node and run a project. The editor and the running project will each use 100% of a CPU thread.
Minimal reproduction project (MRP)
N/A

See #99593

I suspect thats right and this is a duplicate of #99593 which was fixed by https://github.com/godotengine/godot/pull/99688


fix: set smoke test result and do not hold pipeline till smoke is done

if smoketest is aborted it shows green in the pipeline
green pipeline: https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk19/job/jdk19-alpine-linux-x64-temurin/52/
its smoketest job: https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk19/job/jdk19-alpine-linux-x64-temurin_SmokeTests/17/
the reason is, we only do a catch to see if any error to call smoketest but not check its result.
also by default, jenkins does "wait:true" so if we have a smoketest timeout after 10 hours,AQA wont get run till smoketest is done (regardless result)

The comment indicates the intention was to only run AQA if SomeTest succeeds?
https://github.com/adoptium/ci-jenkins-pipelines/blob/6be5a31e9f712c4638efa29df5c5c1ce445e26cb/pipelines/build/common/openjdk_build_pipeline.groovy#L281

yeah, i think if we want to keep "only good smoke then start run AQA" I can remove the "wait: false", also set the pipeline result to smoketest result.
at the moment, it is not doing the correct thing, so the check for smoketest result is still needed.

Run smoke tests, which should block the running of downstream test jobs if there are failures.

That is ultimately the behaviour we want.  We had not yet enabled this behaviour, because there were still many smoke tests failing at the time of the original change and were asked not to block other testing due to them failing.
Reminding that the smoke tests from the temurin-build repo are checking "did we build the right thing", if they fail, it means we did not build the right thing and we should not waste resources on extra testing for it.

update the logic of this PR to: gate aqa-tests by the result of smoke if it is not successful.

I am not sure if all the smoke tests are green. The following test failed for Openj9 at Adopt, but passed for Semeru.
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-linux-aarch64-openj9_SmokeTests/190/

FYI @AdamBrousseau

Looks sane as an isolated change. lots of Whitespace changes again though

to enable "hide whitespace"?

run tests quick 11

reverted the part to check result of smoketest, only set timeout to 1hr.
so we can proceed this PR (if we are not ready to use smoketest result to gate pipeline)

May I know why we do not use the existing parameter TIME_LIMIT and introduce a new parameter SMOKETEST_TIMEOUT?

Basically I see two ways to add timeout

in pipeline set timeout over stage then call downstream smoketest job  ( as in this PR) which is similar to coding style to other stage https://github.com/adoptium/ci-jenkins-pipelines/blob/master/pipelines/build/common/openjdk_build_pipeline.groovy#L74 this introduces a new constant

in the smoketest job itself by re-using TIME_LIMIT, which is similar to https://github.com/adoptium/aqa-tests/pull/3928/files?diff=split&w=1 then in the pipeline add jobParams.put('TIME_LIMIT', "1")

Or, did I miss something as preferred way?

Somehow I imagined to use the known pre-existing variable TIME_LIMIT and add an extra put (jobParams.put("TIME_LIMIT", 1) in getSmokeTestJobParams()
     def getSmokeTestJobParams() {
        def jobParams = getCommonTestJobParams()
        jobParams.put('LEVELS', "extended")
        jobParams.put('GROUPS', "functional")
        jobParams.put('TEST_JOB_NAME', "${env.JOB_NAME}_SmokeTests")
        jobParams.put('BUILD_LIST','functional/buildAndPackage')
        def useAdoptShellScripts = Boolean.valueOf(buildConfig.USE_ADOPT_SHELL_SCRIPTS)
        def vendorTestRepos = ((String)ADOPT_DEFAULTS_JSON['repository']['build_url']).minus(".git")
        def vendorTestBranches = ADOPT_DEFAULTS_JSON['repository']['build_branch']
        def vendorTestDirs = ADOPT_DEFAULTS_JSON['repository']['test_dirs']
        if (!useAdoptShellScripts) {
            vendorTestRepos = ((String)DEFAULTS_JSON['repository']['build_url']).minus(".git")
            vendorTestBranches = DEFAULTS_JSON['repository']['build_branch']
            vendorTestDirs = DEFAULTS_JSON['repository']['test_dirs']
        }
        jobParams.put("VENDOR_TEST_REPOS", vendorTestRepos)
        jobParams.put("VENDOR_TEST_BRANCHES", vendorTestBranches)
        jobParams.put("VENDOR_TEST_DIRS", vendorTestDirs)
        jobParams.put("TIME_LIMIT", 1)
        return jobParams
    }

I also prefer not to introduce another new parameter. Especially, there is an existing parameter that serves the purpose. Adding an additional parameter to resolve a similar issue will cause extra maintenance and confusion.

Do you feel it is better to have https://github.com/adoptium/aqa-tests/pull/3928/files?diff=split&w=1 and https://github.com/adoptium/ci-jenkins-pipelines/pull/394 together as solution?

if we all agree on the solution in https://github.com/adoptium/ci-jenkins-pipelines/pull/394 then I can have that one open and close this one? @llxia @smlambert @sophia-guo

have this one closed and continue with https://github.com/adoptium/ci-jenkins-pipelines/pull/394


Update tutorial.md

some more comments:

Agenda 3.1:  -> Visualize (spelling)
line 72: "for example normal, Gaussian. (is Gaussian always normal?)
line 88: give link to explain Euclidean etc. Wikipedia
line 107: give link to explain Pearsons, Speamans etc. Wikipedia
Fig 4/5 has nor description on the axis
line 143: on what questions?
Hands on step 1: when importing the data from our data library, there are 7 different folders called "Clustering in ML" - no idea which one t take to find the data! Very confusing
line 185: give a short description (or picture) with explanation what petals, sepals etc are for all non-botanists
line 210: be careful to use the newest version of tools (if required). When searching "Numeric clustering" in the tool search bar by hand under Machine learning, it opens automatically an older version of the tool
line 349: DBScan could be explained earlier
try to use block letters ad paragraphs everywhere

For the tutorial: non of my numerical clusterings has run successfully with the parameters given in the tutorial. Error report have been send to the Galaxy team

ping @anuprulez @khanteymoori


UMD wrap

Can you add an UMD wrap for the dist files.

The fix is coming for this, along with #136


chore(main): release 0.1.1

:robot: I have created a release beep boop
0.1.1 (2024-11-23)
Bug Fixes

deps: update rust crate serde_json to v1.0.133 (c63c894)
deps: update rust crate url to v2.5.4 (#8) (7270656)
Don't overload -h (8396227)
Update files if they exist in OWUI (0059728)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/philipcristiano/owui-rag-sync/releases/tag/v0.1.1 :sunflower:


Call private googlesheets?

Any way to retrieve infirmation of a private google sheet?

Not at the moment, unfortunately. What’s the use-case you have in mind?

We need to pull some data from sheets but we can't  change this files to be public, due to the information is private. So we are using this library but only could retrieve information for public sheets.

That makes sense! I’ve built one-off solutions in the past that include that functionality, and am happy to discuss, but unfortunately this open source project currently doesn’t have that functionality. Sorry!


add formatter

Background.
There's an argument to be made about how the code looks.
This isn't a very productive argument, but it helps that the successor has a clean code!
Sometimes formatters are used to avoid this discussion and to formulate code that is in line with convention
What I did.

Make option to choose one of three formatters: black, yapf and autopep8.
Added make format to Makefile.
Added information about the formatter in README.md

Thank you very much for the valuable contribution @funwarioisii ! I think the formatted libraries need to be added in requrirements.txt.

Thank you for the accurate confirmation.
Corrected the part you pointed out.
Please check it out.

LGTM


Add SECURITY.md to the Kurator Repository

What would you like to be added:
A security policy.
Why is this needed:

Kurator Repository doesn't have a security policy as of yet.  For the health of the project, we need add a SECURITY.md

/assign @LiZhenCheng9527


Use aebytecode repository and remove apps/aebytecode

Are these two lines necessary now?
.dockerignore:32:apps/aebytecode/src/aeb_asm_scan.erl
.gitignore:41:apps/aebytecode/src/aeb_asm_scan.erl

Are these two lines necessary now?
.dockerignore:32:apps/aebytecode/src/aeb_asm_scan.erl
.gitignore:41:apps/aebytecode/src/aeb_asm_scan.erl

Yes, this is a generated file from aeb_asm_scan.xrl

Yes, this is a generated file from aeb_asm_scan.xrl

@ThomasArts https://github.com/aeternity/epoch/pull/1977


Download TF-target genes

Hi there!
Thanks for the great work!
I am interested in obtaining all TF-target gene links for human and mouse. Instead of downloading thousands of separate files using URL: https://chip-atlas.dbcls.jp/data/Genome/target/Protein.Distance.tsv, I was wondering if there is a way to download a single dataset? If possible, can I have the info for cell type as well?
I look forward to your reply. Thanks in advance for your time and help!
Best,
Yiqi

Hi Yiqi,
I can give you a tar.gz file containing all the TSV files of the TF target gene. Does this help?
Zhaonan

Hi Zhaonan,
Yes, that would be perfect! Thanks so much!
Best,
Yiqi

I have prepared gzip files of your request. You can download them from the following URLs:
https://chip-atlas.dbcls.jp/data/share/target_genes_hg38.tar.gz
https://chip-atlas.dbcls.jp/data/share/target_genes_mm10.tar.gz
Could you let me know after the download?

I am able to download the files. I really appreciate your great help!


Don't show "Disabling YouCompleteMe's completion for python files"

Hi maintainers,
I absolutely love this autocomplete package for vim, but use more languages than Python. That's why I've started using YouCompleteMe. I now use YCM alongside Kite. But even though YCM is disabled for the Python filetype, I still always get the message "Disabling YouCompleteMe's completion for python files". I think it's okay to show this message if Python isn't on the YCM blacklist, but why show it if Python is on the YCM blacklist?
Suggestions
I propose checking if python is on the blacklist of each other autocomplete engine. This would look like something in the line of:
-if exists('g:loaded_youcompleteme')
+if exists('g:loaded_youcompleteme') && !exists('g:ycm_filetype_blacklist.python')
     let g:ycm_filetype_blacklist.python = 1
     call kite#utils#warn("disabling YouCompleteMe's completions for python files")
 endif

Another option would be to just remove the message and don't let the user know YCM is being disabled.
Pro's

No more 'random' messages everytime a python buffer is opened

Cons

Someone needs to find out how the blacklist is stored for all other completion engines and add these in
There are multiple blacklists, especially with YCM: g:ycm_filetype_blacklist, g:ycm_filetype_specific_completion_to_disable which makes it harder to keep track of it.

Please let me know what you think about this issue. Glad to help if there's anything I can do.

Thanks for the suggestion – it sounds like a good idea.  We'll try to implement it soon.

Thanks again.


tcR and dplyr 1.0.3

In dplyr 1.0.3, about to be released, count() and tally() become generic, and this seems to create a problem with tcR:
# tcR

<details>

* Version: 2.3.2
* GitHub: https://github.com/immunomind/immunarch
* Source code: https://github.com/cran/tcR
* Date/Publication: 2020-06-09 11:10:02 UTC
* Number of recursive dependencies: 76

Run `cloud_details(, "tcR")` for more info

</details>

## Newly broken

*   checking S3 generic/method consistency ... WARNING
    ```
    ...
      function(x, ..., wt, sort, name)
    count.frames:
      function(.data, .frame, .head, .coding)

    count:
      function(x, ..., wt, sort, name)
    count.inframes:
      function(.data, .head, .coding)

    count:
      function(x, ..., wt, sort, name)
    count.outframes:
      function(.data, .head)

    See section ‘Generic functions and methods’ in the ‘Writing R
    Extensions’ manual.

    Found the following apparent S3 methods exported but not registered:
      count.frames count.inframes count.outframes
    See section ‘Registering S3 methods’ in the ‘Writing R Extensions’
    manual.
    ```

*   checking Rd \usage sections ... NOTE
    ```
    S3 methods shown with full name in documentation object 'get.inframes':
      ‘count.inframes’ ‘count.outframes’ ‘count.frames’

    The \usage entries for S3 methods should use the \method markup and not
    their full name.
    See chapter ‘Writing R documentation files’ in the ‘Writing R
    Extensions’ manual.
    ```

## In both

*   checking installed package size ... NOTE
    ```
      installed size is  7.6Mb
      sub-directories of 1Mb or more:
        data   1.4Mb
        doc    3.8Mb
        libs   1.7Mb
    ```

Hi @romainfrancois  thank you so much for the comment here, I truly appreciate it! We decided to archive tcR in favour of Immunarch.


Tenants stay in Terminating status after perf test resource cleanup

What happened:
The current tenant is [{{ } {apivu0-testtenant    /api/v1/tenants/apivu0-testtenant cc7d6bed-30d7-4c92-88c9-1eceaf157fb6<PHONE_NUMBER>368039408<PHONE_NUMBER>82946049 0 2020-08-07 12:24:51 +0000 UTC 2020-08-07 13:12:05 +0000 UTC  map[] map[] [] nil []  []} {[arktos] 0} {Terminating}} {{ } {ewkofo-testtenant    /api/v1/tenants/ewkofo-testtenant 16786a9a-e45c-46a2-9de1-4b95ebbb1b46<PHONE_NUMBER>473647024<PHONE_NUMBER>81897473 0 2020-08-07 12:12:31 +0000 UTC 2020-08-07 13:12:05 +0000 UTC  map[] map[] [] nil []  []} {[arktos] 0} {Terminating}} {{ } {iji6fe-testtenant    /api/v1/tenants/iji6fe-testtenant 708bfced-3bae-4c6d-badf-11b1319db1cc<PHONE_NUMBER>346924560<PHONE_NUMBER>81373185 0 2020-08-07 12:07:41 +0000 UTC 2020-08-07 13:12:05 +0000 UTC  map[] map[] [] nil []  []} {[arktos] 0} {Terminating}} {{ } {okylhk-testtenant    /api/v1/tenants/okylhk-testtenant a64f8fc5-cd95-4980-92a7-63ffc4199467<PHONE_NUMBER>855445695<PHONE_NUMBER>83470337 0 2020-08-07 12:22:06 +0000 UTC 2020-08-07 13:12:05 +0000 UTC  map[] map[] [] nil []  []} {[arktos] 0} {Terminating}} {{ } {rb9d8a-testtenant    /api/v1/tenants/rb9d8a-testtenant 5d3a0b98-d756-4ba5-8ab6-7b55838efa14<PHONE_NUMBER>59769529<PHONE_NUMBER>81897474 0 2020-08-07 12:18:30 +0000 UTC 2020-08-07 13:12:05 +0000 UTC  map[] map[] [] nil []  []} {[arktos] 0} {Terminating}} {{ } {system    /api/v1/tenants/system a3724c7e-5d02-4a08-9d43-f0c0ee3d06c4<PHONE_NUMBER>801997139<PHONE_NUMBER>65815297 0 2020-08-06 16:48:09 +0000 UTC   map[] map[] [] nil []  []} {[arktos] 0} {Active}} {{ } {y4wgmg-testtenant    /api/v1/tenants/y4wgmg-testtenant 04b86f5a-7e99-4053-b319-1c7ae29628a4<PHONE_NUMBER>748280980<PHONE_NUMBER>73369857 0 2020-08-07 12:08:46 +0000 UTC 2020-08-07 13:12:05 +0000 UTC  map[] map[] [] nil []  []} {[arktos] 0} {Terminating}}]I0807 13:50:15.438614    2771 simple_test_executor.go:74] Waiting for the chaos monkey subroutine to end...
What you expected to happen:
All the tenants are removed
How to reproduce it (as minimally and precisely as possible):
Fetched my latest branch https://github.com/jshaofuturewei/arktos/tree/create_tenant
./run-e2e.sh --nodes=100  --provider=kubemark --kubeconfig=$HOME/go/src/k8s.io/arktos/test/kubemark/resources/kubeconfig.kubemark --report-dir=$REPORTDIR --testconfig=testing/density/config.yaml
Anything else we need to know?:
Environment:

Arktos version (use kubectl version):
https://github.com/jshaofuturewei/arktos/tree/create_tenant

Cloud provider or hardware configuration:

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

Kernel (e.g. uname -a):

Install tools:

Network plugin and version (if this is a network-related bug):

Others:

It can be deleted now. However, it takes minutes to do it.
Please see attached.  A kubectl delete might take around 1 minutes, which is quicker.

Joe:
I found a bug in https://github.com/jshaofuturewei/arktos/commit/0d6d5b1a842f6a8930f4ff2e71cb88d22e97f5c5 and left a comment. Could you update the code?
Qian

Thanks and updated. However, it is not related to tenant deletion especially for new created tenants.

Joe:
I found a bug in jshaofuturewei@0d6d5b1 and left a comment. Could you update the code?
Qian

Per offline discussion, I will jump in if there is another repro. So far, I could not repro it in my local env.

close it as it is not actively reproing. Feel free to reopen if it is back.


Lookup helper won't work with block parameters

{{#each names as |name id|}}
  {{name}}

  {{lookup . 'name'}}

{{/each}}

works for {{name}} but not for {{lookup . 'name'}}
jsfiddle
I intend to use this with dinamic partials:
{{> (lookup . 'name') }}

I can't say if this is supposed to work, since the block parameters are not part of this model.
I think this needs to be specified (#1277, "We need a spec")

Do you have to use "lookup" to solve your problem. I think you could just use {{> (name)}} like in https://jsfiddle.net/vva7musq/9/


bug: component issue fixing required

Describe the bug

[ ] Need to add all the bugs in components.

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Stack Trace
The stack trace of the bug if any.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

this issue was related to adding validProps and fixing componenets, but we creating seprate issue for every componenets. So closing this issue (invalid)


[release-4.12] OCPBUGS-32234: fix: CVE-2023-45288

cherry-picked from 0971d1588a645ca80ec466ea42fa9e4a9e883508
updates net package to avoid being affected through gRPC server

/jira refresh


Create ndarray from BFarray ctypes struct

This allows a bifrost ndarray to be instantiated from a BFarray struct (as returned by ndarray.as_BFarray())
Minimal example:
        a = bifrost.ndarray(np.arange(100), dtype='i32')
        aa = a.as_BFarray()
        b = bifrost.ndarray(aa)
        np.testing.assert_equal(a, b)

(PS: Apologies, meant to push this to my telegraphic fork and then open PR request)


Button on downloads page shows dollars then downloads EU

From the CC Certificate Slack channel:  "I am trying to find By-NC and BY-NC-SA svg  with US dollars not EU symbol. When I went to downloads page-https://creativecommons.org/about/downloads the image shows dollars then downloads EU. "
===================
Finally, we're going to put someone's name here, so they get alerted to the new
issue and can assign it to the right person. @little-wow

This should be fixed!


[Collapsible] Add Collapsible components and hooks

Closes https://github.com/mui/base-ui/issues/84
Docs: https://deploy-preview-481--base-ui.netlify.app/base-ui/react-collapsible/
Extra demos:

Animated with framer-motion: https://deploy-preview-481--base-ui.netlify.app/experiments/collapsible-framer/
Crude accordion using 3 Collapsibles + CSS transitions: https://deploy-preview-481--base-ui.netlify.app/experiments/collapsible-accordion/
hidden="until-found": https://deploy-preview-481--base-ui.netlify.app/experiments/collapsible-hidden-until-found/

[x] I have followed (at least) the PR section of the contributing guide.

@colmtuite It's ready for you to have one last look before we merge it ~


drain fail

Hi
I am getting the following error when using the drain function.
The drain start and completes, so it is only the "ansible job" which in indicating something went wrong.  Unfortunately I am a terrible at python...
{
    "module_stdout": "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py:847: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings\r\n  InsecureRequestWarning)\r\nTraceback (most recent call last):\r\n  File \"/home/ansible/.ansible/tmp/ansible-tmp-1643867344.0378213-97863888650931/AnsiballZ_rancher_node.org\", line 102, in <module>\r\n    _ansiballz_main()\r\n  File \"/home/ansible/.ansible/tmp/ansible-tmp-1643867344.0378213-97863888650931/AnsiballZ_rancher_node.org\", line 94, in _ansiballz_main\r\n    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)\r\n  File \"/home/ansible/.ansible/tmp/ansible-tmp-1643867344.0378213-97863888650931/AnsiballZ_rancher_node.org\", line 40, in invoke_module\r\n    runpy.run_module(mod_name='ansible.modules.rancher_node', init_globals=None, run_name='__main__', alter_sys=False)\r\n  File \"/usr/lib64/python3.6/runpy.py\", line 208, in run_module\r\n    return _run_code(code, {}, init_globals, run_name, mod_spec)\r\n  File \"/usr/lib64/python3.6/runpy.py\", line 85, in _run_code\r\n    exec(code, run_globals)\r\n  File \"/tmp/ansible_rancher_node_payload_dzj4hizb/ansible_rancher_node_payload.zip/ansible/modules/rancher_node.py\", line 176, in <module>\r\n  File \"/tmp/ansible_rancher_node_payload_dzj4hizb/ansible_rancher_node_payload.zip/ansible/modules/rancher_node.py\", line 168, in main\r\n  File \"/tmp/ansible_rancher_node_payload_dzj4hizb/ansible_rancher_node_payload.zip/ansible/modules/rancher_node.py\", line 50, in rancher_node_drained\r\n  File \"/tmp/ansible_rancher_node_payload_dzj4hizb/ansible_rancher_node_payload.zip/ansible/modules/rancher_node.py\", line 19, in get_node\r\nKeyError: 'pagination'\r\n",
    "module_stderr": "Shared connection to NODE.DOMAIN.ORG closed.\r\n",
    "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error",
    "rc": 1,
    "_ansible_no_log": false,
    "changed": false
}

I'm wondering if the Rancher API has changed. It looks like the pagination result wasn't returned by the API.
@andreas-theswede Please provide the version of Rancher you're using as well as the code for the task where this failed.

We are running rancher 2.6.2, @seancallaway .

Interesting. I am going to see if the API has changed.


Conversion error

When I try to convert 0x5 in decimal using :Hex2Dec both in normal and visual mode I get the following error: Error converting base - unknown digit: X

Does it happen to you @rr-?


Remember the time chosen

Can you please make the app to remember the timeout setting so that it can be enabled with a single tap? And long tapping can allow us to change the timing.
Thanks!

Hey there, sorry for a late reply. But the app is EOL because I don't really have time to maintain it. Feel free to fork or contribute. Thanks!


export

Case

HELP QUESTION

Issue

Running the export command, many errors are reported.
Starting dev server...
events.js:182
throw er; // Unhandled 'error' event
^
Error: listen EADDRINUSE :::8080
at Object._errnoException (util.js:1021:11)
at _exceptionWithHostPort (util.js:1043:20)
at Server.setupListenHandle [as _listen2] (net.js:1344:14)
at listenInCluster (net.js:1385:12)
at Server.listen (net.js:1469:7)
Info

Operating System: OSX Sierra 10.12.6
Node-Version: 8.7.0

Reproduce

run npm run export

If the problem is not solved already:
Error: listen EADDRINUSE :::8080 means that port 8080 is already used by another application.

thanks,solved already


removeAllComponents will create weird behavior

Hi,
I may be wrong, but calling something like: Entity. removeAllComponents() will create issues with Object3DComponent.
Indeed, when calling Entity.removeObject3DComponent or Entity.remove
Fix

A simple fix would be to do the cleaning / unparenting directly in the reset() method of the component. This method is always called.
A more general / robust solution would be to clear a bit what is called and when. Right now, it seems a bit odd to reset the component via its pool via a call to dispose(). It seems that everything should be handled by a final call to just the managers (component / entity). Also, calling removeAllComponents should obviously at some point call the Entity.removeComponent, otherwise people extending the default entity will anyway end up with those problem.

I can make a PR for that on Ecsy side if you want to.
Bye!

I think for now we can just override Entity.removeAllComponents(), check if the entity has an Object3DComponent and call Entity.removeObject3DComponent().
I do think we need a concept of a ComponentManager which can handle this sort of cleanup in a more extensible way.


flash mount problem

hello
i am using W25Q128FV flash chiop and CYC8CKit-050 cypress kit
i tried to implement SPIFFS, but got SPIFFS_ERR_NOT_A_FS, with SPIFFS_USE_MAGIC enable
how can i solve this?
i am attaching my code here
TFT_SSD1963.cywrk.Archive04.zip

Have you formatted the Flash using spiffs_format before you tried to mout it?

Before being able to write to the File System, be sure it is formatted. With Spiffs, you have to unmount the File System, format it using spiffs_format and then remount it.
After this you should not encounter any problems reading and writing Files, at least if your Hardware abstraction works as expected.

Unmount,format and remount got -10025 error
On 20-Aug-2018 7:00 pm, "sensslen"<EMAIL_ADDRESS>wrote:

Before being able to write to the File System, be sure it is formatted.
With Spiffs, you have to unmount the File System, format it using
spiffs_format and then remount it.
After this you should not encounter any problems reading and writing
Files, at least if your Hardware abstraction works as expected.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pellepl/spiffs/issues/220#issuecomment-414317122, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUTque8Q50CAkUg7Nlsng6z1ZraBQwGsks5uSroSgaJpZM4WBPmU
.

Please verify, that your Hal driver works properly.
As an alternative, I recommend to simulate spiffs with a small desktop application with a file as backend...

Can you see any mistake in my HAL driver,because it ia working correct if i
read write and erase without spiffs
On 20-Aug-2018 9:55 pm, "sensslen"<EMAIL_ADDRESS>wrote:

Please verify, that your Hal driver works properly.
As an alternative, I recommend to simulate spiffs with a small desktop
application with a file as backend...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pellepl/spiffs/issues/220#issuecomment-414377839, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUTqubVtfeFlP3kxvbniRzMiNuYqMjQ6ks5uSuMRgaJpZM4WBPmU
.

hi
i solved the issue , it was in erase function of HAL driver
thanks for support


fix: link with rel="alternate" should be unique based on hrefLang

since it's kinda hard to add support for this within the current code base.. I refactor the implementation for deciding which key is considered the primary key a bit.. the updated code should also fix #279..
within the updated code, priority for which keys should be considered first is based on the primaryAttributes array given to getTagsFromPropsList (leftmost first).. this makes the ordering the attributes appear in user code irrelevant..

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

We'd be willing to send a small Amazon gift card in exchange for having this PR merged :-)

Any progress on this?


Add repolinter support and fix link in SECURITY.md

Signed-off-by: Arnaud J Le Hors<EMAIL_ADDRESS>Type of change

New feature

Description
Per the TSC, it was decided that all repositories must support repolinter and adhere to the common repository structure.
Additional details
See https://wiki.hyperledger.org/display/TSC/Projects+have+two+quarters+to+comply+with+common+repo+structure

Hit integration test flake https://jira.hyperledger.org/browse/FAB-18305

/azp run

@lehors Thanks! Did the repolint pass in your local env?
In the future, would there be a job that runs that automatically checks this in CI?

@lehors Thanks! Did the repolint pass in your local env?
In the future, would there be a job that runs that automatically checks this in CI?

Yes, it did:
$ repolinter .
Target directory: /Users/lehors/Projects/github.com/hyperledger/fabric
Axiom language failed to run with error: Unexpected end of JSON input
Lint:
✔ apache-license-file: Contains Apache License.*Version 2.0 (LICENSE)
✔ code-of-conduct-file: Contains https://wiki.hyperledger.org/community/hyperledger-project-code-of-conduct (CODE_OF_CONDUCT.md)
✔ security-file-matches: Contains https://wiki.hyperledger.org/display/SEC/Defect.Response (SECURITY.md)
✔ readme-file-exists: Found file (README.md)
✔ readme-references-license: Contains license (README.md)
✔ maintainers-file-exists: Found file (MAINTAINERS.md)
✔ contributing-file-exists: Found file (CONTRIBUTING.md)
✔ changelog-file-exists: Found file (CHANGELOG.md)
✔ test-directory-exists: Found file (core/testutil)
✔ integrates-with-ci: Found file (ci/azure-pipelines.yml)
✔ notice-file-exists: Found file (NOTICE)
⚠ source-license-headers-exist:
	⚠ docs/build/html/_static/doctools.js: The first 5 lines do not contain the pattern(s): Copyright, License
	⚠ docs/build/html/_static/documentation_options.js: The first 5 lines do not contain the pattern(s): Copyright, License
	⚠ docs/build/html/_static/jquery-3.2.1.js: The first 5 lines do not contain the pattern(s): Copyright, License
	⚠ docs/build/html/_static/searchtools.js: The first 5 lines do not contain the pattern(s): Copyright, License
	⚠ docs/build/html/_static/language_data.js: The first 5 lines do not contain the pattern(s): Copyright, License
	⚠ docs/build/html/_static/jquery-3.5.1.js: The first 5 lines do not contain the pattern(s): Copyright, License
	⚠ docs/build/html/_static/underscore.js: The first 5 lines do not contain the pattern(s): Copyright
	⚠ docs/build/html/_static/underscore-1.3.1.js: The first 5 lines do not contain the pattern(s): Copyright
	⚠ docs/build/html/_static/jquery.js: The first 5 lines do not contain the pattern(s): Copyright
	⚠ docs/build/html/_static/websupport.js: The first 5 lines do not contain the pattern(s): Copyright, License
	⚠ docs/build/html/_static/js/html5shiv-printshiv.min.js: The first 5 lines do not contain the pattern(s): Copyright
	⚠ docs/build/html/_static/js/html5shiv.min.js: The first 5 lines do not contain the pattern(s): Copyright
	⚠ docs/build/html/_static/js/modernizr.min.js: The first 5 lines do not contain the pattern(s): Copyright, License
	⚠ docs/build/html/_static/js/theme.js: The first 5 lines do not contain the pattern(s): Copyright, License
✔ binaries-not-present: Excluded file type doesn't exist (**/*.exe,**/*.dll,!node_modules/**)
ℹ package-metadata-exists: ignored due to unsatisfied condition(s): "language=javascript"
ℹ package-metadata-exists: ignored due to unsatisfied condition(s): "language=ruby"
ℹ package-metadata-exists: ignored due to unsatisfied condition(s): "language=java"
✔ license-detectable-by-licensee: Licensee identified the license for project: Apache-2.0
ℹ package-metadata-exists: ignored due to unsatisfied condition(s): "language=python"

lgtm complained with:
[2021-02-01 14:50:00] [build-err] Only found JavaScript or TypeScript files that were empty or contained syntax errors.

lgtm complained with:
[2021-02-01 14:50:00] [build-err] Only found JavaScript or TypeScript files that were empty or contained syntax errors.

I need to get Ry to disable LGTM until we can get around to configuring it properly, to day we just take it at its default config, which does nothing for us save for finding some syntactical issues when they are fixed.
@ryjones Can you disable LGTM for the Fabric repo, only an Org Admin can do this, not a repo admin?

lgtm complained with:
[2021-02-01 14:50:00] [build-err] Only found JavaScript or TypeScript files that were empty or contained syntax errors.

I need to get Ry to disable LGTM until we can get around to configuring it properly, to day we just take it at its default config, which does nothing for us save for finding some syntactical issues when they are fixed.
@ryjones Can you disable LGTM for the Fabric repo, only an Org Admin can do this, not a repo admin?

Yeah, I don't recall a discussion with the project maintainers to enable lgtm in the first place. If somebody has a compelling reason to enable them, we can have that discussion, but until there is some consensus on the topic, let's disable them.

@denyeart this was enabled as advisory for all GitHub repos years ago. It never should have been a requirement, unless maintainers chose to enforce it. I've disabled it for now.

A few comments and questions. Can we also format the JSON file properly, right now most of the lines are one lined which makes reading and updating difficult.

Fair questions. I'll admit I merely copied the file from https://github.com/hyperledger-labs/hyperledger-community-management-tools/blob/main/repo_structure/repolint.json
I think the original idea was to have the same config file used across Hyperledger but I have to say that I'm not convinced creating copies all over the place is a good idea. This is going to be a pain to maintain. At least, tailoring it for each repo would justify doing so rather than pointing to a global config file. From what I understand this used to be impossible but it is now.

What would be the issue with simply adding it to CI on PR's? If this is the expected way a repo is meant to be setup, then I would expect that we should make sure that a PR doesn't deviate from that by running it at PR time.

I think it's merely a question of trade-off. But there is nothing prevent us from doing so if we don't mind paying the price of that extra check on every PR.

Thanks

Thanks to you. Based on the various attempts by a few of us to implement the TSC decision so far I think this needs a bit more discussion. I'll put it on the TSC agenda.

The saga continues in https://github.com/hyperledger/fabric/pull/2366

I reduced this PR to simply fixing the broken link in SECURITY.md so this can be merged without waiting for us to figure out the proper way of adding repolinter support.

Signed-off-by: Arnaud J Le Hors<EMAIL_ADDRESS>Type of change

New feature

Description
Per the TSC, it was decided that all repositories must support repolinter and adhere to the common repository structure.
Additional details
See https://wiki.hyperledger.org/display/TSC/Projects+have+two+quarters+to+comply+with+common+repo+structure

The description no longer matches the change

@lehors The description no longer matches the change

Fixed. Thanks.


wit.ai backend

Hi. I've patched your module to be able to use the wit.ai engine as a backend instead of google. If you're interested in integrating this into your project the patch is here:
https://gist.github.com/maverickagm/56ee25f830ac4440cc70
To switch to the wit.ai backend you must specify stt_engine as 'wit' and provide a valid key:
r = sr.Recognizer()
r.stt_engine = 'wit'
r.key = 'my_wit_api_key_here'
In the recognize_wit() function, the show_all variable now toggles whether just the recognized text is returned to the entire json object (including entities)
Thanks.

This is amazing! I'd love to use this if you don't mind, but I'm not sure how to apply this patch to the SpeechRecognition module.
I'd really appreciate some help if you have the time! Thanks.

I totally missed this one earlier, great work! This will be integrated in the next major release.

Added as part of v3.0.0!

Awesome! The wit.ai backend works perfectly as of 3.1. However, I'm having a bit of trouble getting exactly what I need from the response. For example, if I need just the intent of what is being said. I would really love some help. Thanks!

Hi @TheAtomicNord,
Try sr.recognize_wit(audio, show_all=True) - the show_all option will return the raw response from Wit.ai, which will contain the intent info.

Thanks @Uberi, but I've got that part down. The problem is when I just want the intent. The show_all option returns far more data than I need. Again, thanks for any help you can provide!

Ah, you can drill down in the response object since it's a JSON dictionary. Here's the response format!
Basically, the library does not modify the response object in show_all mode. The intents should be in there.

@Uberi
Hey sorry to keep bothering you, but I really can't seem to figure this out. I'm a fairly novice programmer as you can probably tell. I need to get each part of the "Outcomes" list into a separate variable and python just isn't cooperating with me.
Thanks again for all the help!

Hi @TheAtomicNord,
No worries, you can do so as follows:
result = some_SOME_RECOGNIZER.recognize_wit(SOME_AUDIO_DATA, SOME_KEY, show_all=True)
outcomes = result["outcomes"]
first_outcome = outcomes[0] # you can also say first_outcome = result["outcomes"][0]` here

Note that there aren't necessarily any outcomes - if you just say something random, outcomes will be an empty list and accessing outcomes[0] will result in an error. Make sure to check how many outcomes there are before accessing them. Here's a complete example:
import sys

import speech_recognition as sr

WIT_AI_KEY = "INSERT WIT.AI API KEY HERE" # Wit.ai keys are 32-character uppercase alphanumeric strings

# obtain audio from the microphone
r = sr.Recognizer()
with sr.Microphone() as source:
    print("Say something!")
    audio = r.listen(source)

# recognize speech using Wit.ai
try:
    result = r.recognize_wit(audio, key=WIT_AI_KEY, show_all=True)
except sr.RequestError:
    print("Could not request results from Wit.ai service")
    sys.exit(1)

for outcome in result["outcomes"]:
    print("Intent:", outcome["intent"])

@Uberi
You are truly a beautiful human being. Thank you so much for making my life easier and supporting this awesome module! Can't wait for 4.0!