RuntimeError: Atomics.wait cannot be called in this context

I'm trying to use wasm-bindgen-rayon 1.0.3 in my project, I'm using Chrome 103.0.5060.114 on windows.
I've this code to init the wasm part:
import init, { initThreadPool } from "../physics/pkg";

async function main() {
    await init();
    await initThreadPool(navigator.hardwareConcurrency);
    await import("./index");
}

main();

However I got this issue:
bootstrap.ts:10 Uncaught (in promise) RuntimeError: Atomics.wait cannot be called in this context
    at core::core_arch::wasm32::atomic::memory_atomic_wait32::h5440a6c4a0a06846 (80b835a348a02018643f.wasm:0x3ff326)
    at std::sys::wasm::mutex::Mutex::lock::ha4a05c48513bbf2a (80b835a348a02018643f.wasm:0x2de528)
    at std::sys_common::mutex::MovableMutex::raw_lock::he81e94f0080a928c (80b835a348a02018643f.wasm:0x3fdc57)
    at std::sync::mutex::Mutex<T>::lock::he4f53ecd85d7d977 (80b835a348a02018643f.wasm:0x3ef0cb)
    at spmc::channel::Sender<T>::send::hd7b538f9d01fbf7a (80b835a348a02018643f.wasm:0x17b836)
    at wasm_bindgen_rayon::wbg_rayon_PoolBuilder::build::{{closure}}::hd621a7d1ef754496 (80b835a348a02018643f.wasm:0x29fa94)
    at <rayon_core::registry::CustomSpawn<F> as rayon_core::registry::ThreadSpawn>::spawn::h176a0862da85c4cb (80b835a348a02018643f.wasm:0x241cad)
    at rayon_core::registry::Registry::new::h90276e299016d06f (80b835a348a02018643f.wasm:0xdbf98)
    at rayon_core::registry::init_global_registry::{{closure}}::h848379b093be6bd0 (80b835a348a02018643f.wasm:0x2abb05)
    at rayon_core::registry::set_global_registry::{{closure}}::h73f6010cde18cd75 (80b835a348a02018643f.wasm:0x24aa0b)

Should I call initThreadPool from an webworker or something?

Yes, this is described in the Caveats section.

So  I should call initThreadPool from a worker thread? Would you please kindly upgrade your test project? From what I can tell, https://github.com/GoogleChromeLabs/wasm-bindgen-rayon/blob/c3a12c4843b35d5c528fbed178aad1e9142c8072/test/index.js#L18 is calling it from main thread and It does not work on my machine.

I've noticed https://github.com/GoogleChromeLabs/wasm-bindgen-rayon/issues/18#issuecomment-960949808 and I'll check out the demo instead of the test.

So I should call initThreadPool from a worker thread? Would you please kindly upgrade your test project? From what I can tell,
https://github.com/GoogleChromeLabs/wasm-bindgen-rayon/blob/c3a12c4843b35d5c528fbed178aad1e9142c8072/test/index.js#L18
is calling it from main thread and It does not work on my machine.

Yeah for tests it didn't matter for some reason, but demo was intended to be an actual end-to-end demo of how you'd realistically use the library.


Fix LoadGen Python binding missing symbol error

Fix LoadGen Python binding missing symbol error

@guschmue Could you merge this? This is a bug I introduced... 😢 Thanks!


digitalocean module should include forge

If, as we suspect, we do need to keep forge around to generate SSH keys, then we should bundle/browserify it with the freedomjs module.

Hmm, not feasible. Seems like node-forge is just not amenable to this:
https://github.com/digitalbazaar/forge/issues/126
Will open a separate issue to move us off of forge.


ONN doesn't seem to converge

I'm trying the simple Online Neural Network (ONN). I'm try to do multiple partial fit with your test data in your README, but it's not converging.
For example when I run 100 times the partial_fit function with [0.1,0.2] -> 0 and [0.8,0.5] -> 1 I always have zero (0) as the predictions, so the training accuracy is 50% in this toy example for some reason.
Am I doing something wrong here?
Thanks!

Hmm, any news? Sorry for the delay, I was very busy these days.
I will have a look at this. :D

@skaftanis The example in the README is just a fake example of how you need to input your data into the function. It will not converge with just these data. You can test with generated data from scikit-learn for test purposes. Have a look at the jupyter notebook example in the code, there will be everything you need to know about how to use the algorithm.
ONN_Usage_Example
Thanks!

Closing this issue because no answers were given.


Fix errors in SBOM download commands

 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.


adjustment

User description
aaa

Type
enhancement, documentation

Description

Simplified the PR URL retrieval method in the GitHub provider to use the pr.html_url attribute directly.
Enhanced the PR description generation process to include a detailed walkthrough of changes and a list of file changes. This includes support for inline file summaries and dynamic URLs in final update messages.
Refactored several methods in pr_description.py to support the new features and fixed a formatting issue by adding a missing newline at the end of the file.

Changes walkthrough
Relevant filesEnhancement

github_provider.pySimplify PR URL Retrieval Method

pr_agent/git_providers/github_provider.py
Updated get_pr_url method to use pr.html_url directly instead of constructing the URL manually.

+1/-1

pr_description.pyEnhance PR Description Generation and Formatting

pr_agent/tools/pr_description.py
Added support for including a changes walkthrough and file changes in PR descriptions.  Enhanced PR description generation to support inline file summaries and final update messages with dynamic URLs.  Refactored methods to return additional information for PR descriptions, including changes walkthrough and file changes list.  Fixed missing newline at the end of the file.

+26/-20

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

Preparing PR description...

PR Description updated to latest commit (https://github.com/Codium-ai/pr-agent/commit/1275cf0123753f06e7aa203da11843f3a6820fbf)

[ ] Copy walkthrough table to "Files Changed" Tab <!-- /describe --copy_walkthrough_to_diffview_tab -->

/describe

Preparing review...

Preparing review...

PR Review

      PR feedback                       ⏱ Estimated effort to review [1-5]
3, because the PR includes both enhancements and refactoring across multiple files, which requires a detailed review of the logic changes, potential side effects, and adherence to best practices.

 🧪 Relevant tests
No

 🔍 Possible issues

The refactoring in github_provider.py might affect other methods that rely on the previous get_pr_url implementation. It's crucial to ensure all dependent functionalities are tested with the new direct URL retrieval method.

In pr_description.py, the addition of new features and refactoring could introduce bugs if not thoroughly tested, especially the changes in PR description generation and formatting.
 🔒 Security concerns
No

 Code feedback:
relevant filepr_agent/git_providers/github_provider.py
suggestion

Consider validating the pr.html_url attribute before returning it from get_pr_url to ensure it's not None or malformed. This can prevent potential runtime errors or issues with downstream consumers of this method. [important]

relevant linereturn self.pr.html_url
relevant filepr_agent/tools/pr_description.py
suggestion

Ensure that changes_walkthrough and pr_file_changes are initialized properly before use to avoid UnboundLocalError in scenarios where they might not be set due to conditional paths. [important]

relevant linepr_title, pr_body, changes_walkthrough, pr_file_changes = self._prepare_pr_answer_with_markers()
relevant filepr_agent/tools/pr_description.py
suggestion

It's recommended to handle exceptions for the new logic introduced in process_pr_files_prediction method to ensure that any errors in processing PR files do not halt the execution or lead to incomplete PR descriptions. [medium]

relevant linewalkthrough_gfm, pr_file_changes = self.process_pr_files_prediction(walkthrough_gfm,
relevant filepr_agent/tools/pr_description.py
suggestion

Consider adding a final newline at the end of the file to adhere to POSIX standards and avoid potential issues with tools that expect it. [medium]

relevant linereturn ''.join(parts)

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.
When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:
/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:
[pr_reviewer]
some_config1=...
some_config2=...

  Utilizing extra instructions
The review tool can be configured with extra instructions, which can be used to guide the model to a feedback tailored to the needs of your project.
Be specific, clear, and concise in the instructions. With extra instructions, you are the prompter. Specify the relevant sub-tool, and the relevant aspects of the PR that you want to emphasize.
Examples for extra instructions:
[pr_reviewer] # /review #
extra_instructions="""
In the 'possible issues' section, emphasize the following:
- Does the code logic cover relevant edge cases?
- Is the code logic clear and easy to understand?
- Is the code logic efficient?
...
"""

Use triple quotes to write multi-line instructions. Use bullet points to make the instructions more readable.

  How to enable\disable automation

When you first install PR-Agent app, the default mode for the review tool is:

pr_commands = ["/review", ...]

meaning the review tool will run automatically on every PR, with the default configuration.
Edit this field to enable/disable the tool, or to change the used configurations

  Auto-labels
The review tool can auto-generate two specific types of labels for a PR:

a possible security issue label, that detects possible security issues (enable_review_labels_security flag)
a Review effort [1-5]: x label, where x is the estimated effort to review the PR (enable_review_labels_effort flag)

  Extra sub-tools
The review tool provides a collection of possible feedbacks about a PR.
It is recommended to review the possible options, and choose the ones relevant for your use case.
Some of the feature that are disabled by default are quite useful, and should be considered for enabling. For example:
require_score_review, require_soc2_ticket, and more.

  Auto-approve PRs
By invoking:
/review auto_approve

The tool will automatically approve the PR, and add a comment with the approval.
To ensure safety, the auto-approval feature is disabled by default. To enable auto-approval, you need to actively set in a pre-defined configuration file the following:
[pr_reviewer]
enable_auto_approval = true

(this specific flag cannot be set with a command line argument, only in the configuration file, committed to the repository)
You can also enable auto-approval only if the PR meets certain requirements, such as that the estimated_review_effort is equal or below a certain threshold, by adjusting the flag:
[pr_reviewer]
maximal_review_effort = 5

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the review usage page for a comprehensive guide on using this tool.

Preparing review...

PR Review

      PR feedback                       ⏱ Estimated effort to review [1-5]
3, because the PR involves both enhancements and refactoring across multiple files with changes in logic and formatting. Understanding the context and ensuring the changes align with the project's architecture requires a moderate level of effort.

 🧪 Relevant tests
No

 🔍 Possible issues

In github_provider.py, directly using self.pr.html_url assumes self.pr is always initialized and has the html_url attribute. This could lead to AttributeError if self.pr is None or not properly initialized.

The changes in pr_description.py introduce new variables and logic that could potentially alter the behavior of PR description generation. Without tests, it's hard to ensure these changes don't introduce regressions or unexpected behavior.
 🔒 Security concerns
No

 Code feedback:
relevant filepr_agent/git_providers/github_provider.py
suggestion

Consider adding a check to ensure self.pr is not None before accessing self.pr.html_url. This can prevent potential runtime errors if self.pr is not properly initialized. [important]

relevant linereturn self.pr.html_url
relevant filepr_agent/tools/pr_description.py
suggestion

Ensure that changes_walkthrough and pr_file_changes are initialized and handled correctly in all code paths to prevent UnboundLocalError. Specifically, verify their initialization and usage in both _prepare_pr_answer_with_markers and _prepare_pr_answer methods. [important]

relevant linepr_title, pr_body, changes_walkthrough, pr_file_changes = self._prepare_pr_answer_with_markers()
relevant filepr_agent/tools/pr_description.py
suggestion

Consider handling exceptions more gracefully in process_pr_files_prediction. Instead of just logging and passing, it might be beneficial to ensure that the function returns a consistent type (e.g., an empty string or list) to avoid potential issues with the callers. [medium]

relevant linepass
relevant filepr_agent/tools/pr_description.py
suggestion

To improve readability and maintainability, consider refactoring the large blocks of conditional logic in _prepare_pr_answer_with_markers and _prepare_pr_answer into smaller, more focused methods. This can make the code easier to understand and test. [medium]

relevant linedef _prepare_pr_answer_with_markers(self) -> Tuple[str, str, str, List[dict]]:

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.
When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:
/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:
[pr_reviewer]
some_config1=...
some_config2=...

  Utilizing extra instructions
The review tool can be configured with extra instructions, which can be used to guide the model to a feedback tailored to the needs of your project.
Be specific, clear, and concise in the instructions. With extra instructions, you are the prompter. Specify the relevant sub-tool, and the relevant aspects of the PR that you want to emphasize.
Examples for extra instructions:
[pr_reviewer] # /review #
extra_instructions="""
In the 'possible issues' section, emphasize the following:
- Does the code logic cover relevant edge cases?
- Is the code logic clear and easy to understand?
- Is the code logic efficient?
...
"""

Use triple quotes to write multi-line instructions. Use bullet points to make the instructions more readable.

  How to enable\disable automation

When you first install PR-Agent app, the default mode for the review tool is:

pr_commands = ["/review", ...]

meaning the review tool will run automatically on every PR, with the default configuration.
Edit this field to enable/disable the tool, or to change the used configurations

  Auto-labels
The review tool can auto-generate two specific types of labels for a PR:

a possible security issue label, that detects possible security issues (enable_review_labels_security flag)
a Review effort [1-5]: x label, where x is the estimated effort to review the PR (enable_review_labels_effort flag)

  Extra sub-tools
The review tool provides a collection of possible feedbacks about a PR.
It is recommended to review the possible options, and choose the ones relevant for your use case.
Some of the feature that are disabled by default are quite useful, and should be considered for enabling. For example:
require_score_review, require_soc2_ticket, and more.

  Auto-approve PRs
By invoking:
/review auto_approve

The tool will automatically approve the PR, and add a comment with the approval.
To ensure safety, the auto-approval feature is disabled by default. To enable auto-approval, you need to actively set in a pre-defined configuration file the following:
[pr_reviewer]
enable_auto_approval = true

(this specific flag cannot be set with a command line argument, only in the configuration file, committed to the repository)
You can also enable auto-approval only if the PR meets certain requirements, such as that the estimated_review_effort is equal or below a certain threshold, by adjusting the flag:
[pr_reviewer]
maximal_review_effort = 5

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the review usage page for a comprehensive guide on using this tool.

Preparing review...

Preparing review...

Persistent review updated to latest commit https://github.com/Codium-ai/pr-agent/commit/1275cf0123753f06e7aa203da11843f3a6820fbf

Preparing review...

Persistent review updated to latest commit https://github.com/Codium-ai/pr-agent/commit/1275cf0123753f06e7aa203da11843f3a6820fbf

Preparing review...

Persistent review updated to latest commit https://github.com/Codium-ai/pr-agent/commit/1275cf0123753f06e7aa203da11843f3a6820fbf

Preparing review...

Persistent review updated to latest commit https://github.com/Codium-ai/pr-agent/commit/1275cf0123753f06e7aa203da11843f3a6820fbf

Preparing review...

Persistent review updated to latest commit https://github.com/Codium-ai/pr-agent/commit/1275cf0123753f06e7aa203da11843f3a6820fbf

Preparing review...

Persistent review updated to latest commit https://github.com/Codium-ai/pr-agent/commit/1275cf0123753f06e7aa203da11843f3a6820fbf

Persistent review updated to latest commit https://github.com/Codium-ai/pr-agent/commit/1275cf0123753f06e7aa203da11843f3a6820fbf

Generating PR code suggestions
Work in progress ...

Generating PR code suggestions
Work in progress ...

Generating PR code suggestions
Work in progress ...

Generating PR code suggestions
Work in progress ...

Generating PR code suggestions
Work in progress ...

Generating PR code suggestions
Work in progress ...

Generating PR code suggestions
Work in progress ...

/test github_provider.py

/analyze

/test run


Mekanism pipes invisible when Connected Textures Mod is installed.

Issue description:
Hi, on 1.10 Connected Textures Mod breaks Mekanism pipes rendering.
Specifically the mekanism pipes are all invisible.
Relevantly CTM is a required library of chisel which sees wide usage.
Steps to reproduce:

Place down a basic universal cable
Observe that it is invisible

Version (make sure you are on the latest version before reporting):
My minecraft version is 1.10.2
Forge: <IP_ADDRESS>7
CTM-MC1.10.2-<IP_ADDRESS>
Mekanism: <IP_ADDRESS>1
MC Multipart: 1.4.0
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
https://gist.github.com/Dilnu/7d73ced44946c60f691176fa09ebc689

Can confirm this: http://i.imgur.com/KiIsalS.png

As a work around reverting to version 10.22 of Chisel and removing CTM fixes the issue. I don't think there are any new textures added in that interval.

Linking this to Chisel-Team/Chisel#459
There's a test version of the Connected Texture Mod in that thread which should fix the issues you're describing.


[BUG] GDI font rendering. Sometimes letters disappear from the page after time. (may be leaks)

Describe the bug
Sometimes letters disappear from the page after time,
while browsing other pages and switch back, and so on.
I need only to restart the browser, to restore normal rendering.
To Reproduce
Don't know because this happens only sometimes. (may be after some hours, while browser open).
for me it was happened on Github, and on other sites.
i use --process-per-site switch, if it mater.
Screenshots
1:

2: (here I have selected empty area on other page, to check what is inside)

Config:

OS: WinXP x86
https://github.com/win32ss/supermium/releases/tag/v122-hf
chrome.dll: latest available: (17.03.2024) https://github.com/win32ss/supermium/releases/download/v122-dbg/122_new_pdb_32.7z
progwrp: latest (26.03.2024) : https://github.com/win32ss/supermium/files/14752565/progwrp_32.zip

UPD: now it is happened again:

What I see on page:

2. What I see if I selected all page with "Select all text" with mouse:

3. What I see if I restarted the browser (all ok again):

UPD: now another picture (some letters are gone):

Same text selected:

Text after restarting the browser:

I haven't been able to reproduce this issue, but I noticed that most renderer processes have only 4 GDI handles open, but some have anywhere from 50 to 300. Another user reported this issue occurring on Stack Overflow as well. I'll have to see if GitHub or Stack Overflow result in the renderer opening more GDI handles.

its happened for me now again. But GDI handles are not so much for this tab: 45-75 per tab with issue. 3 such tabs per process. Not so much.. May be something else in code ?

UPD: its real mistic.... :) now its restores back to normal -- without restarting the browser, Just switching to another tabs.. browsing there some time., and switching back )).
"The bugs lives here.." (c) my

I confirm that this bug exists: I've seen it myself too.
But I have to say, on old Chrome 49 I saw this strange behaviour too (but much less often).

now its restores back to normal -- without restarting the browser, Just switching to another tabs..

Switching page scale also restores page rendering to normal, but when you back to initial scale (nor necessarely 100% scale), some words or letters disappear again.

 I realized later that the problem was related to alignment of the wchar_t Unicode string that was not correct because of a custom memory allocator. Fixing the alignment solved the problem. I have no idea if this might be related but It was hard to find for me.

Chromium is indeed "lazy" when it comes to drawing text. It uses ExtTextOutW to draw individual glyphs in GDI and it does not draw the text again if it has already drawn it in the past in the context of a renderer process, even if refreshing or switching tabs. I could try forcing complete redraws of text on XP and below.

Yes, if you want to enforce more drawing as a workaround, then provide a switch to turn that off. As it didn't happen to me yet. It would be a pity to suffer from any overhead for non-affected people.

this case stable reproduced on my machine:  (on Github its not reproduced while)
I have used --process-per-site
this 3 pages from same site:
https://web.archive.org/web/20091224143500/http://dev.opera.com/articles/view/css3-transitions-and-2d-transforms/
https://web.archive.org/web/20090207114919/http://my.opera.com/core/blog/2009/02/04/vega
https://web.archive.org/web/20090206133936/http://my.opera.com/core/blog/2009/02/04/carakan
Are in one renderer process.:

Yes, if I change the Zoom to other value, it renders ok, because of recreating the glyphs (I think).
If I open other pages from same site, not changing zoom : the drawing corrupt (some letters are empty space).
I have not changed anything, Just browsing another pages some time. I'm going back (not right away). And it restores back to normal.
So whats happened with already drawed glyphs ?
Also I have found this:
https://learn.microsoft.com/en-us/windows/win32/api/wingdi/nf-wingdi-exttextouta
"For ExtTextOutW, the glyph indexes are saved to a metafile. However, to display the correct characters the metafile must be played back using the same font. For ExtTextOutA, the glyph indexes are not saved."
"ETO_IGNORELANGUAGE
| Reserved for system use. If an application sets this flag, it loses international scripting support and in some cases it may display no text at all."
Or may be somethisn another, which influences to this.
May be GDI font drawing was restored not so correct ?

UPD: now the above post text here on Github was affected ))). Now it touches only italic text.
the Italic text appears empty, even if I select it. :

Moreover: If I translate it to another language, It also are empty here on italic places :| :|

GDI font rendering is available in Supermium on all versions of Windows. However, no one has reported this issue on any version after Server 2003 (there are a few known Supermium users on 10/11 specifically for GDI).
But with the various instances of degradation of GDI since 2016, I will try to replace any skia GDI code with 2016-era code if possible. This would allow me to remove the various offset workarounds in ui/gfx/render_text.cc.

Hello, Who and why closed this issue and where it was fixed ?

The problem still here and appears time to time also on youtube, when adding videos to playlists. Making it unusable. As results lists are empty. Or semi-empty.
Can it be marked as "Critical bug" and fixed ?

The problem still here and appears time to time also on youtube, when adding videos to playlists. Making it unusable. As result lists are empty (without letters on its positions). Or semi-empty.
Can it be marked as "Critical bug" and fixed ?

After issue #656 was found to occur only with GDI, it was determined that it would be best to implement DirectWrite wrapped in GDI in an attempt to rectify all remaining issues with GDI rendering using the remnants of the GDI renderer. At this point it is sufficient to load most of the UI text.
This particular bug remains difficult to encounter on my end, having only happened twice since Supermium started running on XP. But both instances occurred on Wikipedia so it will be used for testing on my end.

This problem is not specific to particular websites. It has occurred many times to me on totally random websites. I haven't been able to reproduce it but it seems more likely to happen after one have browsed for some very long time.

I think it depends not on the text size, but on count of different text styles in CSS, which interferes one to others. And somewhere the cache corrupts... But where? .....Yes, its a good thing to find 100 % reproducible way to catch this situation.
And may be it depend on Metafile as I mentioned in above post. It requeres more investigation how it works.
Or may be some font attributes are handled incorrectly while caching.

You may be right, as I know nothing about CSS styles. The website I used for testing is
https://opentomb.github.io/TRosettaStone3/trosettastone.html

I have no any problems with that test page while. All rendered right.
I dont know, how it works, If it caches all fonts/glyphs from all sites in one common cache, then cache corruption will depend on other sites you opened and browsing with the session.
I have detected this behaviour, (about it restores to normal), in one of earlier posts. When I switched to another tabs with different sites and come back then.

I've found the cause
I saved that page on the local computer (as html only) and THAT causes the problem. I tried reading the page from original source and it was OK. Perhaps has something to do with CSS styles as you said. since the CSS files etc are not present on my computer.

The CSS seems embedded here into the HTML (I have checked now), So when you saved it, its saved with CSS automatically.
I also done this (saving on local computer and opening from there), and it rendered ok for me. May be it depends on system locale, dont know.. Its not reproduced on my side. All rendered ok here.

Have a bad news.
Using now v126-r1. With DirectWrite for XP.
But problem comes back again (youtube):
when I trying to open playlist to add video. Whole lines disappeared again as was in previous Supermium releases:

So DirectWrite was not helped here.
Its not reproduced alway of cource, (as usual). Its randomly. Its another floating bug.
Can it be marked as "Floating bug" ?


ARTEMIS-1318 Uses unbuffered MAPPED Journal if no data-sync is required

In order to reduce memory footprint, CPU usage and reduce latencies it avoid to use the TimedBuffer on the MAPPED journal if no datasync is required.

@clebertsuconic I'm just concerned about the callbacks behaviour...do you have any advices?

@franz1981 what about replacing the TimedBuffer from another delegating buffer that would call the callbacks?
Now, you have to make sure that callbacks from replication would still be used, as in this mode you would need it to work.

@franz1981 I feel like you should do more tests before merging this Franz.

@clebertsuconic
Yes I need to run the whole test suite, add some others tests to see if it is working as expected plus provides the NIO version too.
What I'm not understanding is what is happening here: https://github.com/franz1981/activemq-artemis/blob/31d5758885214ff65aebfdeb6024f7b275397bff/artemis-journal/src/main/java/org/apache/activemq/artemis/core/journal/impl/JournalImpl.java#L2646-L2646
Seems the only visible difference when the TimedBuffer is not enable (on NIO too)...


CRM error with Add User to Record Team

Hi When I try to add to the Team Template in the Add User to Record Team I get the following error
Microsoft Dynamics 365
To use this saved view, you must remove criteria and columns that refer to deleted or non-searchable items. If you contact support, please provide the technical details.
here is the log file error Unhandled Exception: System.ServiceModel.FaultException`1[[Microsoft.Xrm.Sdk.OrganizationServiceFault, Microsoft.Xrm.Sdk, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35]]: Server was unable to process request.Detail:

eac77754-1c8b-4331-89d3-8498424cd35c
-2140991216

Server was unable to process request.
2018-07-13T15:16:41.5594803Z
false

Thanks
Lee

Hello Lee,
This issue can be resolved using approach similar to following - https://crmtipoftheday.com/747/if-your-view-is-broken/
Thanks,
Andrew


Support any types that has UnmarshalJSON/MarshalJSON method

I wonder if you could support non-primitive types  such as time.Time just like the standard json.Unmarshal json.Marshal does.   I only need it when such types implemented UnmarshalJSON/MarshalJSON methods.

Sure! This is already supported. Any type implementing custom JSON unmarshaler will be respected as you expect.
Example
Let us know if it works for you, or you need anything else 🙏

@knoguchi making sure you've seen this one before I'm closing the issue

Thank you. I have one example that I can't figure out.  I will post it shortly.

@knoguchi any news here?

Closing due to no response


ODRIVE V3.6 (NRND) integration ROS2

Hi, Does this package works with the V3.6 model?

It's not tested with ODrive v3.6. The CAN protocols are somewhat similar, so it might partially work, perhaps with small modifications, but no guarantees.


manifest.json

there is a description, overview, and config in the manifest. since those are going to be repeated in the readme, putting them here seem duplicative (wait, is that a word). the short description would probably be ok
what are the sizes of the images?
@irabinovitch

@technovangelist just saw this the image sizes are listed here: https://github.com/DataDog/IntegrationTemplate
120x60.png
128x128.png
200x128.png

Closing, see https://github.com/DataDog/integrations-core/pull/398


Tidied up cli

Put options in their own file
Better mechanism for -q and -v flags
Added options.common decorator which adds -v, -q, --config, and short -h flag (by default click only supports long version --help)

Do you think functools.wrap() help in passing of attributes?

Definitely! Thanks for pointing that out


Add keepAliveOperation

Memo:

https://devpixiv.hatenablog.com/entry/2016/11/30/103730
https://github.com/tootsuite/mastodon/blob/master/app/javascript/mastodon/actions/streaming.js

It's now testing in Delir.


wss:// support

Does this plugin support WebSocket Secure? If so, how would one enable it? I see there's a protocols property on the configuration object, but I'm not sure what to add there to add wss (if thats even where it would be added)
Thanks

It does not.
The underlying libraries (PocketSocket and TooTallNate) do support SSL/TLS though.
So implementing the secure feature is feasible, but I think there is a problem with the certificates:
one would need a self-signed certificate as a certificate delivered by a trusted authority could not be used. I believe.
I had already tried to add the secure feature but without success.
So any help would be appreciated.
By curiosity, could you provide details of the context in which you would need wss://? thanks
The protocols option is about Sec-WebSocket-Protocol

Thanks for the reply. I am thinking of developing a kiosk app for tablets that would accept payment information on one tablet and send it to another, and figure it might be a good idea to do it over wss. Worst case scenario I can look into other encryption options.

Supporting SSL-secured Websocket server connections on a device would require you to distribute your private key along with the public certificate, which in most case wouldn't be a good idea...

There are some use cases, where you may have a need to access a local process, and due to the policy on browsers to avoid "mixed-content" ( see google about it ),  it is a good idea to have TLS support, even if you don;t exchange sensitive data, just to obey browsers rules  about rules  about mixed-content.
Adding a key to your device is no different that adding to a box acting as a server.  In this case here, your device is the server.

Any update? I'm running server with IONIC 4, and want to connect also from ionic app. So ionic uses chrome webview under the hood. Chrome does not allow to connect "ws://" and throw error: connection refused. So it is needed that this support wss.

I ran into a use case where I'd like to communicate from a website to my app which runs the websocket server. This would only work if the website didn't use https, because otherwise it would be mixed content. The issue here is that I need to use getUserMedia on the website, which is only allowed in secure contexts.
Is there any chance you are still evaluating this, @becvert?

I ran into a use case where I'd like to communicate from a website to my app which runs the websocket server. This would only work if the website didn't use https, because otherwise it would be mixed content. The issue here is that I need to use getUserMedia on the website, which is only allowed in secure contexts.
Is there any chance you are still evaluating this, @becvert?


problems debug project code running in the designer

[Enter feedback here]
When I try to follow the code under "To debug project code running in the designer" the second visual studio do not stop in my code instead it auto detach when I enable project code in the first instance of visual studio
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fa80a87f-b85b-2282-f95f-d697f2e5b4c5
Version Independent ID: c869f446-fe53-01f0-a170-30a3c43ff041
Content: Debug or disable project code in XAML Designer - Visual Studio (Windows)
Content Source: docs/xaml-tools/debugging-or-disabling-project-code-in-xaml-designer.md
Product: visual-studio-windows
Technology: vs-xaml-tools
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@jhv2017kam -- thank you for your feedback.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@jhv2017kam -- this repo is solely for documentation issues, and this is product-related.  Please add it here:

Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


computeInstanceMetrics - panic: runtime error: invalid memory address or nil pointer dereference

I am running 0.2.3 app version with aws role that has admin access
When I run this app locally against Account A (with only 1 db instance) it worked
However when I run this app locally against Account B (this one has multiple instances) I get the below error
{"time":"2023-10-12T16:13:00.840914676Z","level":"INFO","msg":"starting the HTTP server component"}
{"time":"2023-10-12T16:13:06.138535387Z","level":"DEBUG","msg":"received query"}
{"time":"2023-10-12T16:13:06.13861547Z","level":"INFO","msg":"get RDS metrics"}
{"time":"2023-10-12T16:13:06.138727887Z","level":"DEBUG","msg":"fetch quotas"}
{"time":"2023-10-12T16:13:06.13939422Z","level":"DEBUG","msg":"fetch usage metrics"}
{"time":"2023-10-12T16:13:06.311188054Z","level":"DEBUG","msg":"usage metrics fetched","metrics":{"AllocatedStorage":192824,"DBInstances":349,"ManualSnapshots":238,"ReservedDBInstances":0}}
{"time":"2023-10-12T16:13:06.589292012Z","level":"DEBUG","msg":"quota metrics fetched","metrics":{"DBinstances":1200,"TotalStorage":214748364800000,"ManualDBInstanceSnapshots":400}}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x14b1df8]

goroutine 52 [running]:
github.com/qonto/prometheus-rds-exporter/internal/app/rds.(*RDSFetcher).computeInstanceMetrics(_, {0x0, 0x0, 0x0, {0x0, 0x0}, {0x0, 0x0}, {0x400043c028, 0x7}, ...}, ...)
	/home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/rds/rds.go:227 +0x308
github.com/qonto/prometheus-rds-exporter/internal/app/rds.(*RDSFetcher).GetInstancesMetrics(0x40004f7790)
	/home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/rds/rds.go:163 +0x24c
github.com/qonto/prometheus-rds-exporter/internal/app/exporter.(*rdsCollector).fetchMetrics(0x4000181680)
	/home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/exporter/exporter.go:275 +0x208
github.com/qonto/prometheus-rds-exporter/internal/app/exporter.(*rdsCollector).Collect(0x4000181680, 0x4000667f58?)
	/home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/exporter/exporter.go:379 +0x144
github.com/prometheus/client_golang/prometheus.(*Registry).Gather.func1()
	/home/runner/go/pkg/mod/github.com/prometheus/client_golang@v1.16.0/prometheus/registry.go:455 +0xd8
created by github.com/prometheus/client_golang/prometheus.(*Registry).Gather in goroutine 37
	/home/runner/go/pkg/mod/github.com/prometheus/client_golang@v1.16.0/prometheus/registry.go:466 +0x4c8

I suspect the problem is related to the nil information returned somewhere

Sorry about that issue. It sounds like it fails on the return value for the Performance insight, I tried a fix on add-pi-information.
Is there any chance that you can try the add-pi-information branch:
git clone git@github.com:qonto/prometheus-rds-exporter.git
cd prometheus-rds-exporter
git checkout add-pi-information
go run . --debug

Also, can you share which engines (e.g. PostgreSQL, MySQL, ...) are used in this AWS account (we may have some specificities)?
Thanks,

That worked for me. Thank you so much!
Do you need me to provide anymore information?

That worked for me. Thank you so much!!
do you need me to provide anymore information?

No, we are fine.
Thanks for taking the time to test the pre-release.
I created the pull request to fix it at https://github.com/qonto/prometheus-rds-exporter/pull/12. Once approved, we'll release a new minor version.

I just released the 0.2.5 that contains this fix.
Thanks for reporting the issue.


Design Update: Navbar

This story is to update design for the navbar to match the new designs
ACs

[ ] It should work in dark/light mode
[ ] It should match the design
[ ] It should be hidden on mobile and toggled open/close

Question: There is a minimised version on desktop. How will this work? Does it auto-minimise after X many seconds from last hover? Does it get toggled on hovering?

Answer: Condensed desktop gets minimised version above tablet breakpoint.


DevOps: remove unused imports + variables during precommit

Describe the task
Integrate a tool to automatically remove unused imports and variables from Python and JavaScript/TypeScript files into the pre-commit hooks. The suggested tools are autoflake for Python and eslint-plugin-unused-imports for JavaScript/TypeScript, but if there are better alternatives, feel free to use those instead.
Acceptance Criteria

[ ] Install a tool for automatically removing unused imports and variables from Python files (e.g., autoflake).
[ ] Install a tool for automatically removing unused imports from JavaScript/TypeScript files (e.g., eslint-plugin-unused-imports).
[ ] Update the pre-commit hook to include the chosen tools and ensure they run automatically before each commit.
[ ] Validate that the tools are effectively removing unused imports and variables in the respective files.

If you're OK with a different python linting tool, ruff supports fixing unused imports: https://docs.astral.sh/ruff/rules/unused-import/

@SKalt Agreed, I was actually leaning toward switching to ruff, which I've started using for other projects recently. I'll get a ticket up.

obsolete because of #841, which switches to ruff


slowness of sillm.chat on M2 Air with 16GB Ram

Any invocation of python -m sillm.chat model seems much slower on my machine than in the reference video--more than a minute to get to the prompt, and maybe 1-2 TPM in the response.
I have tried sillm.chat with two different models downloaded from HF via the download.py scripts in the SiLLM-examples repo: [Mistral-7B-Instruct-v0.2] and [Qwen1.5-7B-Chat]; a LLAMA3 model that I downloaded directly from HF exhibited the same behavior.
Machine specs: MacBook Air M2 16GB memory on Sonoma 14.4.1, running Python 3.12.3 in a conda environment.

This might be an mlx-lm issue, as mlx_lm.generate is also very slow.

I can run sillm.chat with Llama3 8b at 10.79 tok/sec on a MacBook Air M1 2020 w/ 16GB RAM. I'm using Python 3.11.

I suspect the inference to be memory-constrained in this case, if you're trying to run the full 7B and 8B models. Without quantization, Llama-3 8B will take 15,316 MB of memory on my Mac Studio before even starting the chat. This means your 16 GB MacBook Air starts swapping memory with the disk and the speed drops significantly.
Try quantizing the model (argument -q4 or -q8) when running sillm.chat. On my MacBook Air M2 16GB (sounds like the same config) I'm getting 9.20 tok/sec with Llama-3-8b-instruct quantized to 8-bit with under 8 GB of memory used.
FYI the reference video with the MacBook Air is using the Gemma-2B-it model, which is a small/fast model and larger models (7B and 8B) will run slower.

Seems to it, quantized to 4 bit had reasonable performance.

Duh, I can't believe I forgot to mention my Llama3 8b was 4-bit quantized...


Optimize Profile Display Code for Improved Performance and Error Handling

I was going through the profile.js  file and noticed many ways that we can optimize this code. I would like suggestions to refactor and improve the following code  -:
Improvement to be made

Enhance error handling and ensure the connection is released properly.
Replace MySQL's callback pattern with async/await for cleaner code.
Ensure token validation is handled securely and efficiently.
Simplify and organize the structure of the response object.

@Harshdev098  you can assign this task to me and add hacktoberfest and gssoc labels along with it


Will IdentityServer4 2.3 support .NET Core 2.2?

Is your feature request related to a problem? Please describe.
Using X509Certificate2 on non windows platforms have a (.NET Core) bug when decrypting data.
Short explanation:
A bug deep in .NET Core prevents decrypting on non windows platforms (e.g. Docker) due to the missing certificate store. This happens on decrypting data .NET Core always looks up the certificate store and fails if it is not present.
See that data protection issue and the referenced issues for a more detailed explanation.
In an Identity Server 4 scenario X509Certificate2 can be used for the SigningCredentials (and optionaly for data protection e.g. on protected resources/API side).
I found out that with IdentityServer 2.3 preview 1 you moved all repositories to .NET Core 2.1. The bug described above will be/is fixed in .NET Core 2.2 (as far as I understood the linked issues).
Maybe it is only me but I haven't got SigningCredentials working on docker nor the data protection for my IdentityServer4 nor the protected resources. Anyone else having issues?
If at least 2.3 (more preferably 2.1 or 2.2) would support .NET Core 2.2 IdentityServer4 could be run in docker????
Describe the solution you'd like
If IdentityServer 2.3 would support .NET Core 2.2 SigningCredentials as X509Certifcate2 could be used on docker too.
Describe alternatives you've considered
X509certifcate not working on non windows platforms (or maybe it is just me messing up)
So far for testing I use the developer signing credentials.
I haven't figured out a  (working) 3rd option on docker so far.

We have many customers running on linux containers using certs for their signing credentials. I don't know of anything that has changed in .NET Core or ASP.NET Core that's already released with what's planned for 2.2 that should cause those working solutions to break. I did not yet read the link you provided above. Are you saying something is changing in 2.2 whereby those existing solution will no longer work?

Are you saying something is changing in 2.2 whereby those existing solution will no longer work?
No. If there are already people got it working than its just me. And if it works now it will still work in 2.2.
When I experienced problems my research brought me to the link above and I thought that would be the solution. But with the given information I will habe to recheck.
Well, from my side it could be closed...
Thanks


How to override Policy of specific Model in Laravel

Version information

Laravel: v8
Voyager: v1.4.2
PHP: 7.3
Database: mysql

Description of problem

How I can override the policy of specific model using voyager BasePolicy ?

Same as https://github.com/the-control-group/voyager/issues/5278


Nuxt ssr element-ui has a problem

Nuxt ssr element-ui has a problem
Problem demo
@yyx990803 Can help look at how to solve it

Please do not use issues to ask questions.
Nuxt issues should posted to Nuxtjs' repo.


staticmap version not easy to find from docker container

Can you add the staticmap version to either the HTTP header or the 404 root page?
Apache supports this server string in the HTTP header.
Server: Apache/2.4.6 (CentOS) OpenSSL/1.0.2k-fips mod_fcgid/2.3.9 mod_wsgi/4.7.1 Python/3.6 PHP/5.4.16 mod_perl/2.0.11 Perl/v5.16.3

Your web server could do something similar like this.
res.Header().Set("Server", fmt.Sprintf("staticmap/%s", version))

Which should return something like this
Server: staticmap/0.6.3

Current 0.6.3 behavior
$ curl -i http://<IP_ADDRESS>:3000/
HTTP/1.1 404 Not Found
Content-Type: text/plain; charset=utf-8
X-Content-Type-Options: nosniff
Date: Sat, 30 Nov 2024 04:35:39 GMT
Content-Length: 19

404 page not found

Exposing the software version using the Server header is commonly seen as a bad idea in the security community and general recommendation is not to expose it:
https://cheatsheetseries.owasp.org/cheatsheets/HTTP_Headers_Cheat_Sheet.html#server


Basic auth and HTTPS

Using basic auth with an HTTPS endpoint cause the following error :
tfx login --auth-type basic --service-url https://server/tfs --username user --password pwd
Connection failed. Check your internet connection & collection URL.
Message: unable to get local issuer certificate

It seems to be the same problem I had with Git : http://blogs.msdn.com/b/phkelley/archive/2014/01/20/adding-a-corporate-or-self-signed-certificate-authority-to-git-exe-s-store.aspx
So how I can specify my certificate file ?
(Trace doesn't help, CLI version is 0.3.3)

Same problem.  Alternatively, I would accept an option to ignore certificate validation failures.

Thanks. I will look into this, but it won't be until after New Years.
On Wed, Dec 23, 2015 at 3:02 PM, markjgardner<EMAIL_ADDRESS>wrote:

Same problem. Alternatively, I would accept an option to ignore
certificate validation failures.
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/tfs-cli/issues/63#issuecomment-166980851.

I would like to add that it will be nice to have the ability to use personal access token in on premise TFS server. this issue is blocking my team selecting TFS as our cross platform automation system.

Any update?  This still appears to be a problem.  And asking users to send their enterprise domain credentials across the network unencrypted when using on premises TFS seems like a rather large security hole that Microsoft has not addressed and the problem has been known for over 6 months.

@grawcho - note that this tfs-cli issue and "cross platform automation system" are a bit orthogonal.  The new vsts-agent supports ntlm across platforms so you should be unblocked on the automation system side: https://github.com/Microsoft/vsts-agent


Pre cleanup for PR

test docs command fix => use check_call and fstrings
test https://github.com/ioggstream/bandit-report-artifacts/pull/6

Hello @sarnold! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file setup.py:

Line 187:80: E501 line too long (80 > 79 characters)


feat(schema) nicer error message for typedef errors

A typo in a typedef could lead to very cryptic error messages. This adds a metatable which catches accesses to unknown fields, producing an immediate error that is more evident for plugin authors. Since the metatable only hits on invalid accesses which are always plugin development bugs, this metatable adds no runtime overhead.

Nice!


BUG: layer function with Series parameters is incorrect if indexes are not the same

[ ] I have checked that this issue has not already been reported.

[ ] I have confirmed this bug exists on the latest version of staircase.

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Code Sample, a copy-pastable example
import staircase as sc
import pandas as pd

temp = pd.Series(pd.date_range("2020", periods=6, freq="D"))
sc.Stairs().layer(
    start = temp.iloc[0::2].copy(),
    end = temp.iloc[1::2].copy(),
).step_values

Problem description
The above results in a series
0.0    4
1.0    3
2.0    4
3.0    3
4.0    4
5.0    3
Name: value, dtype: int32

Expected Output
0.0    1
1.0    0
2.0    1
3.0    0
4.0    1
5.0    0
Name: value, dtype: int32

Dependency Versions``
Please run the following code:
import staircase
import pandas
import numpy

for pkg in (staircase, pandas, numpy):
    print(pkg.__name__, pkg.__version__)

staircase 2.0.1
pandas 1.3.1
numpy 1.21.1

take


add waitgroup in ReadFromEgress function for AgentFinalStreamHandler

The ReadFromEgress function is inside a goroutine already
And in some case, It's need to wait until complete from the caller
So just return a waitGroup pointer that user can choose if need to use wg.Wait() or not.

PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[x] Describes the source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

update:
found that the old implementation in for xxx := range handler.egress will never end...
And will cause goroutine leak
fixed this problem using HandleAgentFinish method
go func() {
		defer close(handler.egress)
		for data := range handler.egress {
			callback(ctx, data)
}()


Updating dom-bind customStyle does not work

Consider fresh Polymer starter kit project.
app.customStyle['--paper-toolbar-background'] = 'red';
app.updateStyles();

But if the app is setup as polymer element, it works.
Works: http://plnkr.co/edit/hIPUqR654bpMBWmpkBUV?p=preview
Does not work: http://plnkr.co/edit/DWa0FCl9Oqe9wwHiBau2?p=preview

This behavior is by design but needs explanation. updateStyle updates custom property styling in an element and its local DOM (Shady or Shadow DOM). Polymer template based elements like dom-bind produce dom but do not create local dom. Instead they emit dom into the scope in which they exist. In this case, that's the document. It's that scope on which updateStyles should be called. For the document scope, this is Polymer.updateStyles.
There also hasn't been a way to add customStyle to the document scope until a recent commit to master that will be included in the next release, which should be available shortly. After the next release, you will be able to do the following:
app.changeTheme = function () {
  Polymer.StyleDefaults.customStyle['--paper-toolbar-background'] = 'red';
  Polymer.updateStyles();
 };


Error: The non-abstract class 'FlutterAvifWire' is missing implementations for these members

I get the following error when running flutter install:
../../.pub-cache/hosted/pub.dartlang.org/flutter_avif_platform_interface-1.0.0/lib/bridge_generated.dart:289:7: Error: The non-abstract class 'FlutterAvifWire' is missing implementations for these members:
 - FlutterRustBridgeWireBase.drop_dart_object
 - FlutterRustBridgeWireBase.get_dart_object
 - FlutterRustBridgeWireBase.new_dart_opaque
Try to either
 - provide an implementation,
 - inherit an implementation from a superclass or mixin,
 - mark the class as abstract, or
 - provide a 'noSuchMethod' implementation.

class FlutterAvifWire implements FlutterRustBridgeWireBase {
      ^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dartlang.org/flutter_rust_bridge-1.54.0/lib/src/ffi/stub.dart:33:8: Context: 'FlutterRustBridgeWireBase.drop_dart_object' is defined here.
  void drop_dart_object(int ptr) {
       ^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dartlang.org/flutter_rust_bridge-1.54.0/lib/src/ffi/stub.dart:28:10: Context: 'FlutterRustBridgeWireBase.get_dart_object' is defined here.
  Object get_dart_object(int ptr) {
         ^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dartlang.org/flutter_rust_bridge-1.54.0/lib/src/ffi/stub.dart:38:7: Context: 'FlutterRustBridgeWireBase.new_dart_opaque' is defined here.
  int new_dart_opaque(Object obj) {
      ^^^^^^^^^^^^^^^

It looks like this is caused by newer versions of flutter_rust_bridge.
Adding flutter_rust_bridge: 1.51.1 to pubspec.yaml fixes the issue.

I published version 1.0.1 limiting the flutter_rust_bridge version. Thanks.


fix(pkg/git/git.go): write the pre-receive hook script directly to the file descriptor

Summary of Changes
Instead of creating the pre-receive hook script and writing the results to an intermediate *bytes.Buffer, this change writes the script's results directly to an *os.File.
Issue(s) that this PR Closes
Fixes #321
Associated End To End Test PR(s)
None. This path only changes an implementation detail.
Associated Documentation PR(s)
None. This path only changes an implementation detail.
Associated Design Document(s)
None. This path only changes an implementation detail.
Testing Instructions

Create a Deis Cluster
Register an app with deis create
Add your SSH key with deis keys:add
git push deis master from your Buildpack or Dockerfile app (for example, example-go)

The application should be successfully deployed.
Pull Request Hygiene TODOs
Please make sure the below checklist is complete.

[x] Your pull request is concise and to the point (make another PR for refactoring nearby code)
[x] Your commits follow the commit style guidelines

is this punt-able to v2.2 or does this need to go in for this release? I'm assuming it's punt-able since it's an enhancement but just wanna make sure. :)

This is puntable to 2.2. Moving

LGTM, but needs to pass CI and get a rebase before merging :)

@bacongobbler thanks for the bump. I had forgot about this trying to get a successful CI run, and then it went out of date. I may end up closing as this is far from a required item for 2.2. Stay tuned.

[Current coverage][cc-pull] is 41.71%

Merging [#358][cc-pull] into [master][cc-base-branch] will increase coverage by 0.13%

@@             master       #358   diff @@
==========================================
  Files            25         25
  Lines          1140       1141     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            474        476     +2
+ Misses          638        637     -1
  Partials         28         28

Powered by Codecov. Last updated by [80d8cf9...1f3939c][cc-compare]
[cc-base-branch]: https://codecov.io/gh/deis/builder/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/deis/builder/compare/80d8cf922b555f263251cd7963f626dabcfc8588...1f3939ce6300d6954b2f09c4cab454daaf6bec01?src=pr
[cc-pull]: https://codecov.io/gh/deis/builder/pull/358?src=pr


Command line arguments for wrapped scripts are not made available

When trying to send arguments to a, in haibu-carapace, wrapped script the command line arguments specifically meant for the wrapped script where not available in the process.argv of the wrapped script.
Included is a test script showing the deficiency and a fix for the bug.

Cherry-picked in 5d78926fa77653782488fbc4358ec6855e9f006e. Published in<EMAIL_ADDRESS>

Thanks!!! :-)


Reaching invalid state by dispatching continuosly

I am trying to run re-frame on clojure. I am running into a invalid route when I dispatch continuosly.
The error msg I get is:
Exception in thread "pool-1-thread-1" clojure.lang.ExceptionInfo: re-frame: router state transition not found. :scheduled :finish-run {:fsm-state :scheduled, :trigger :finish-run}

I have created a repo to reproduce this:
https://github.com/shakdwipeea/reframe-state-transition-bug

It looks like there may be a race condition here in the Clojure (note, not ClojureScript) code. I could reproduce this as well. After looking at the code for a few minutes, my best guess is that an event is dispatched while the executor is running and the FSM is in the :running state. The executor tries to run the :finish-run trigger, but underneath it, the FSM was moved back to the :scheduled state.

Fixed in #471

That feeling when I've been stumbling upon a weird error for some hours, finally I get convinced that nothing obvious in my code is responsible for it, I find this issue which has exactly the same symptoms and at the end of it I can read « Fixed » ❤

@scgilardi and @danielcompton thanks a lot for your work in https://github.com/Day8/re-frame/pull/471. However, might I suggest you to release it? ^^ looks like it's currently in master, but latest release comes from February.


[Bug] After displaying the 'Cut-Copy-Paste' menu from response text can't get rid of it

Overview

Insomnia Version: 5.0.20
Operating System:  Windows 10 Pro
Summary:  Minor bug described in the following steps and video (video is marked 'unlisted')

Video
https://youtu.be/wwYGGJMQrwA
How To Reproduce

Create new request
Perform a request which returns a response with some text
select some response text
right-click > copy
keep left-clicking and right-clicking everywhere in the window but can not get rid of cut-copy-paste menu, it keeps popping up

Great report @kasradzenika!!
Just an FYI: Unable to replicate on MacOS El Capitan (10.11.6)

I just tried this on Mac, Windows 10, and Elementary OS (Linux) and couldn't reproduce. I have heard this from at least one other person via email, though.

Same here on Win7 x64, Insomnia 5.0.20

@marcioadr88 not sure which comment you are responding to. Can you clarify whether or not you can or can not reproduce?

I'm sorry, I can reproduce the bug mentioned by @kasradzenika.

I am getting same issue in windows 10 PRO, its mostly happens after copy or paste through context menu.

Does someone who can reproduce this want to try fixing it? electron-context-menu is the module used to create/manage these menus and is instantiated in window-utils.js.
I have a few ideas for what might be wrong:

It's being instantiated at the wrong point in time
It's somehow being instantiated more than once

Same issue on OSX, after hitting esc key a huge number of times it finally releases the Cut, Copy & Past menu

I was just able to reproduce this by closing (not quitting) and reopening the window.

Is this closed out in a pending release? obviously still occurring here under #253

@TakesTheBiscuit woops, I didn't realize you stated v5.1.0 in your other submission. Going to re-open this for now.
Can you reliably reproduce this? I was able to fix at least one cause of this bug but there could be others.

I am able to reproduce in the same way as the youtube clip in the OP by @kasradzenika
It seems to mainly focus around the timeline part of the application if that helps

@TakesTheBiscuit unfortunately, I can't reproduce it by following the same steps as in the video.
I only have my Mac laptop's trackpad to test in on, though. Maybe it's related to using a mouse? Do you have a trackpad to try it on?
Are you able to reproduce it immediately after a fresh launch of the app?

To confirm the 100% reproducible as my above post is a bit scatty:
Trackpad AND apple mouse same behaviour:

Double or triple or quad click to highlight a word in 'source' pane. (CLIKCLICKCLICK)
Right click on highlighted word
Left click Copy
Left click anywhere on insomnia or another desktop (anywhere) and context menu appears (even outside the app)

Quit insomnia
Re-open
Issue goes away.
Run a POST request
Issues not coming back.
I have quit and reopened insomnia several times today -- quitting seems to be a 'temp' fix.

I could reproduce the bug on MacOS like this:

trigger context menu
move cursor outside the app

I do believe this issue is related to moving outside the boundaries of the window and / or context menu target
http://prntscr.com/fbzvb8

I think I am able to reproduce it. For me, it only starts happening after switching requests.

copy/paste
switch to different request
copy/paste

I'll see if I can fix it now. Thanks for the help @ValeriaVG and @TakesTheBiscuit

Fyi i also get a really odd little visual glitch
Right click context
Click Copy
Word that was highlighted appears in bright white text ghosting over the top of the original for about 10ms (just long enough to see it) just off to the right of the mouse pointer.
Might sound really stupid (as i haven't dug into any of the source yet) but why can't the OS use native copy and paste dialogues??

@TakesTheBiscuit I have noticed that too, but it only seems to occur when the duplicate context menu bug is happening.
I'm pretty sure it can't use native OS dialogs because it is not a native (Appkit) text label. Insomnia uses Electron, which means Chromium is rendering the text itself instead of delegating that to the native OS UI components.
By default, Electron does not include any context menus, so you need to implement it yourself. I chose to use the popular electron-context-menu library for this.
After some inspection, it seems like the problem might be caused when React re-uses an element that had previously been removed from the DOM.

I hate disagreeing, but the 'ghost' bug appears for me even when the context menu is behaving itself.
This bug isn't urgent anyway, smashing escape is fine for now.
Thanks for looking

No worries. Just trying to get to the bottom of this. Is this what you were describing? It seems to stay visible until the context menu disappears.

OMG finally, I think I got this 🐛! Check out the PR above for the fix 😄


Create Dev Container to have an stable contributing environment

Is your feature request related to a problem? Please describe.
New contributors need to install bunch of libraries and maybe they develop on windows and it may be a little hard to work around.
Describe the solution you'd like
So a universal development environment like docker development container is a good idea to start developing fast.

Lapce is a graphical application so running it in container is not supported, it also puts maintainence burden on us to keep devcontainer up-to-date.

I think the idea of this issue was more to support something like VSCode DevContainer. This is a really nice things as it allows to declaratively configure your development environment and share it easily. No need to install LSP servers, linters, compilers, and whatever on your machine. It installed/started when you start your IDE.


Exceed 64k bytes.

When run domino license analysis utility, "There was an Error in the (CheckLicenseTrackingSettings) function. The details of the Error are below: Error: Notes error: adding entry will cause text list to exceed 64K Entry not added. #: 4000  line #: 109" error occur.

This is a duplicate of https://github.com/HCL-TECH-SOFTWARE/domino-license-analysis-utility-DLAU/issues/32


Validation of HTTPs url

I think it's time to disable URL validation on the plugin and allow passing in http:// URLs instead of https:// URLs for the server url field.

This was done in 705a260f4b155c1ea8e7ed85ef73fcfa0a56e889


Revert "[5.7][ConstraintSystem] Extend availability check to cover unavailable ext…"

Reverts apple/swift#58487

@swift-ci please test

Original change exposed an issue related to availability checking so I'm going to revert this until that issue gets addressed.


Publish 6/11/2020 10:30 AM PT

Docs Build status updates of commit e3a1530:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/addsadministration/New-ADFineGrainedPasswordPolicy.md
:white_check_mark:Succeeded

docset/windows/addsadministration/Set-ADFineGrainedPasswordPolicy.md
:white_check_mark:Succeeded

docset/windows/dism/Add-AppxProvisionedPackage.md
:white_check_mark:Succeeded

docset/windows/smbshare/Get-SmbShare.md
:white_check_mark:Succeeded

docset/winserver2012-ps/activedirectory/New-ADFineGrainedPasswordPolicy.md
:white_check_mark:Succeeded

docset/winserver2012-ps/activedirectory/Set-ADFineGrainedPasswordPolicy.md
:white_check_mark:Succeeded

docset/winserver2012-ps/smbshare/Get-SmbShare.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/addsadministration/New-ADFineGrainedPasswordPolicy.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/addsadministration/Set-ADFineGrainedPasswordPolicy.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/smbshare/Get-SmbShare.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[Bug]: webpackChunkName magic comments sometimes not respected

Version
System:
    OS: macOS 14.6.1
    CPU: (11) arm64 Apple M3 Pro
    Memory: 4.08 GB / 36.00 GB
    Shell: 3.7.1 - /opt/homebrew/bin/fish
  Browsers:
    Chrome: 129.0.6668.60
    Safari: 17.6
  npmPackages:
    @rsbuild/core: 1.0.7 => 1.0.7
    @rsbuild/plugin-react: 1.0.2 => 1.0.2
    @rsbuild/plugin-svgr: 1.0.2 => 1.0.2
    @rsbuild/plugin-type-check: 1.0.1 => 1.0.1

Details
Hello again! I hate opening issues without reproducible examples but I'm a bit lost here.
I believe something changed between 1.0.1-rc.3 and 1.0.5 and now (when using the default split by experience) the webpackChunkName is not respected anymore for us in some cases.
This is breaking the rsbuild plugin shared here: https://mmazzarolo.com/blog/2024-08-13-async-chunk-preloading-on-load/.
It just seem that in some cases async chunks are generated correctly but don't use the name as set in webpackChunkName anymore: the name used in webpackChunkName don't show up in compilation.namedChunkGroups nor in the stats file.
I can't pinpoint why, though, but I can reproduce the issue consistently switching between 1.0.1-rc.3 and 1.0.5 (in our big codebase). I can also still reproduce it on 1.07.
This week I was also pinged by a couple of folks wrt to the rsbuild plugin article posted above telling me that it doesn't work. I thought it was some issues on their end, but now that I had a chance to update our app, I guess it might be related to this issue.
Does this ring any bell? Any recent change to the way the magic comments or code splitting is happening? Thanks!
(If it can help, in our rsbuild we're not setting any cutom performance settings)
Reproduce link
No link :(
Reproduce Steps
I can't create a small reproducible example—sorry.
I can reproduce it in my big SPA where, in a file with a ton of lazy loaded pages (some even prefetched), only one of them is actually being named correctly. I don't know why only that specific one. In 1.0.1-rc.3 they're all built and named correctly.
const Page1 = lazy(() => import(/* webpackPrefetch: true, webpackChunkName: "Page1" */ `./pages/Page1`));
const Page2 = lazy(() => import(/* webpackPrefetch: true, webpackChunkName: "Page2" */ `./pages/Page2`));
const Page3 = lazy(() => import(/* webpackPrefetch: true, webpackChunkName: "Page3" */ "./pages/Page3"));
const Page4 = lazy(() => import(/* webpackPrefetch: true, webpackChunkName: "Page4" */ `./pages/Page4`));
const Page5 = lazy(() => import(/* webpackPrefetch: true, webpackChunkName: "Page5" */ `./pages/Page5`));
const Page6 = lazy(() => import(/* webpackPrefetch: true, webpackChunkName: "Page6" */ `./pages/Page6`));
const Page7 = lazy(() => import(/* webpackPrefetch: true, webpackChunkName: "Page7" */ `./pages/Page7`));
const Page8 = lazy(() => import(/* webpackPrefetch: true, webpackChunkName: "Page8" */ `./pages/Page8`));

const Page9 = lazy(() => import(/* webpackChunkName: "Page9" */ `./pages/Page9`));
const Page10 = lazy(() => import(/* webpackChunkName: "Page10" */ `./pages/Page10`));
const Page11 = lazy(() => import(/* webpackChunkName: "Page11" */ `./pages/Page11`));
const Page12 = lazy(() => import(/* webpackChunkName: "Page12" */ `./pages/Page12`));
const Page13 = lazy(() => import(/* webpackChunkName: "Page13" */ `./pages/Page13`));
const Page14 = lazy(() => import(/* webpackChunkName: "Page14" */ `./pages/Page14`));
const Page15 = lazy(() => import(/* webpackChunkName: "Page15" */ `./pages/Page15`));
const Page16 = lazy(() => import(/* webpackChunkName: "Page16" */ `./pages/Page16`));
const Page17 = lazy(() => import(/* webpackChunkName: "Page17" */ `./pages/Page17`));
const Page18 = lazy(() => import(/* webpackChunkName: "Page18" */ `./pages/Page18`));
const Page19 = lazy(() => import(/* webpackChunkName: "Page19" */ `./pages/Page19`));
const Page20 = lazy(() => import(/* webpackChunkName: "Page20" */ `./pages/Page20`));
const Page21 = lazy(() => import(/* webpackChunkName: "Page21" */ `./pages/Page21`));
const Page22 = lazy(() => import(/* webpackChunkName: "Page22" */ `./pages/Page22`));
const Page23 = lazy(() => import(/* webpackChunkName: "Page23" */ `./pages/Page23`));
const Page24 = lazy(() => import(/* webpackChunkName: "Page24" */ `./pages/Page24`));
const Page25 = lazy(() => import(/* webpackChunkName: "Page25" */ `./pages/Page25`));
const Page26 = lazy(() => import(/* webpackChunkName: "Page26" */ `./pages/Page26`));
const Page27 = lazy(() => import(/* webpackChunkName: "Page27" */ `./pages/Page27`));
const Page28 = lazy(() => import(/* webpackChunkName: "Page28" */ `./pages/Page28`));
const Page29 = lazy(() => import(/* webpackChunkName: "Page29" */ `./pages/Page29`));
const Page30 = lazy(() => import(/* webpackChunkName: "Page30" */ `./pages/Page30`));
const Page31 = lazy(() => import(/* webpackChunkName: "Page31" */ `./pages/Page31`));
const Page32 = lazy(() => import(/* webpackChunkName: "Page32" */ `./pages/Page32`));
const Page33 = lazy(() => import(/* webpackChunkName: "Page33" */ `./pages/Page33`));
const Page34 = lazy(() => import(/* webpackChunkName: "Page34" */ `./pages/Page34`));
const Page35 = lazy(() => import(/* webpackChunkName: "Page35" */ `./pages/Page35`));

If I had to guess, maybe something changed in the way the default performance strategy work and now things are not being "named" unless >= a certaing threshold?

Should related to https://github.com/web-infra-dev/rsbuild/pull/3510
@Timeless0911 cc

@chenjiahan , @Timeless0911 , yup, can confirm that manually setting preserveAllComments: true in the config fixes it!
Curious as to why if it's set to false only some of the webpackChunkName are ignored, I would have expected that all of them will be ignore 🤷
Thanks for the quick response (as always!). This unblock us, but I assume you might want to deal with it in some ways on the rsbuild side too to avoid this to happen by default, right? Up to you to keep this issue open/close.

Sure, what should I do specifically?
As soon as I paste my code (or even just a simple single import with webpackChunkName) and enable one of mangle, minify, or compress, webpack magic comments are stripped away. Not sure if that's what you expect (I can't see an option for preserveAllComments in the playground)

Ok, nevermind, I saw that the option is in the JSON editor side. If preserveAllComments is enabled, magic comments are not stripped (regardless of mangle, etc.). If it's set to fale or not set, they get stripped away.

Can you share me a palyground of which magic comments are stripped away？

OK! Found the issue. It's the double quotes vs "`" in the import statement. That's also why only a single one is imported (see the third one).

I tried creating a minimal reproducible project, but... I can't.  If you don't have any idea on what could be happening here, that's OK, I guess we can close it?

Yes, we've tried that as well, but we still can't reproduce the issue.
We can close this issue first, and if someone else encounters a similar problem and can provide a way to reproduce it, we will reopen it.


OAK-9595 XPath queries don't support bind variables

Found while testing the documentation for keyset pagination at https://jackrabbit.apache.org/oak/docs/query/query-engine.html#Keyset_Pagination

Travis and CI test failures are unrelated to the changes (container problem, blob problems,...)


about

i want to add the app.py ( in huggingface ) into the colab to realize the Gradio app online ,before the update i can get the result ,but now it can't show the result i don't know why!

Im not familiar with gradio @AK391 do you know why?

@yeqingmei123 @mchong6 just tried the app in spaces https://huggingface.co/spaces/akhaliq/JoJoGAN and it is working for me,

are you trying to run the app in colab,
that can be done by doing
import gradio as gr gr.Interface.load("spaces/akhaliq/JoJoGAN).launch()

@yeqingmei123 @mchong6 also might be interested, we are having a event for the new Gradio blocks API https://huggingface.co/Gradio-Blocks, would be great to update jojogan with blocks or to submit another demo: https://huggingface.co/Gradio-Blocks


Fixed flaky BWC test.

Description
Fixes flaky BWC test. Fixes #4445
The test SecurityBackwardsCompatibilityIT.testDataIngestionAndSearchBackwardsCompatibility had a 1:100 chance to send empty bulk request bodies which would result in 400 Bad Request errors.

Category: Test fix
Why these changes are required? The CI was flaky
What is the old behavior before changes and new behavior after changes? no behavioral change.

Issues Resolved

#4445

Testing
Check List

[ ] New functionality includes testing - this is a test fix
[ ] New functionality has been documented - no new functionality
[x] Commits are signed per the DCO using --signoff

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

BTW, what's up with these windows int tests? These seem to fail quite often:

Tests with failures:
 - org.opensearch.security.DefaultConfigurationSingleNodeClusterTests.securityRolesUpgrade
 - org.opensearch.security.DefaultConfigurationSingleNodeClusterTests.shouldLoadDefaultConfiguration
 - org.opensearch.security.DefaultConfigurationSingleNodeClusterUseClusterStateTests.securityRolesUpgrade
 - org.opensearch.security.DefaultConfigurationSingleNodeClusterUseClusterStateTests.shouldLoadDefaultConfiguration
 - org.opensearch.security.api.SslCertsRestApiIntegrationTest.classMethod
 - org.opensearch.security.SecurityConfigurationTests.testParallelTenantPutRequests
 - org.opensearch.security.api.ConfigRestApiIntegrationTest.classMethod
 - org.opensearch.security.api.DefaultApiAvailabilityIntegrationTest.securityConfigIsNotAvailableByDefault

@nibix There was a BindException on the test run. I restarted the CI Check for Windows integ tests JDK 17
This is a known issue that still needs to be addressed: https://github.com/opensearch-project/security/issues/4259
org.opensearch.transport.BindTransportException: Failed to bind to <IP_ADDRESS>:47300
	at org.opensearch.transport.TcpTransport.bindToPort(TcpTransport.java:487) ~[opensearch-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
	at org.opensearch.transport.TcpTransport.bindServer(TcpTransport.java:451) ~[opensearch-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
	at org.opensearch.transport.netty4.Netty4Transport.doStart(Netty4Transport.java:166) ~[transport-netty4-client-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
	at org.opensearch.common.lifecycle.AbstractLifecycleComponent.start(AbstractLifecycleComponent.java:77) ~[opensearch-common-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
	at org.opensearch.transport.TransportService.doStart(TransportService.java:306) ~[opensearch-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
	at org.opensearch.common.lifecycle.AbstractLifecycleComponent.start(AbstractLifecycleComponent.java:77) ~[opensearch-common-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
	at org.opensearch.node.Node.start(Node.java:1523) ~[opensearch-3.0.0-SNAPSHOT.jar:?]
	at org.opensearch.test.framework.cluster.LocalOpenSearchCluster$Node$1.run(LocalOpenSearchCluster.java:428) [integrationTest/:?]
	at java.base/java.lang.Thread.run(Thread.java:840) [?:?]
Caused by: java.net.BindException: Address already in use: bind


PRO-3232: Apply updatable to false on CreationDate columns

  - change applied to both submitdata and invitedata tables

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Closing this PR as it is already merged from demo-> master


Expose email field in identities API

As an API user, I'd like to be able to create an actor identity with a given user email, and have this identity match during the actor's first and subsequent sign ins. I'd like this field to show when GETing the identity as well.

[ ] Expose a persisted email field on the identities endpoint for POST, PATCH, and GET requests
[ ] Don't match incoming userinfo email against provider_identifier on first sign in
[ ] Try to match sub -> provider_identifier on sign ins, and if that fails, match userinfo[email] -> email. In the former case, update the email field with the userinfo[email] value. In the latter case, update the provider_idenfier with the sub value.
[ ] Update Add/Edit/Show UI elements to display new email field

@bmanifold Tried to capture the details of this one to iron out any edge cases. LMK if I missed anything!


undefined symbol: _ZN5model6Engine10ParseSpecsEN6config18CompoundConfigNodeE

Hello, I've tried to install gamma with timeloop, and the installation seems to be successful. But when I run the example scripts by bash run_gamma_timeloop.sh and bash run_gamma_timeloop_multiObjective.sh, there's an ImportError caused by undefined symbol.
The complete log is as follows.
root@83fe54e1ae36:~/gamma-timeloop# bash run_gamma_timeloop.sh  Traceback (most recent call last):   File "/root/gamma-timeloop/src/main.py", line 1, in <module>     from gamma_timeloop_env import GammaTimeloopEnv   File "/root/gamma-timeloop/src/gamma_timeloop_env.py", line 7, in <module>     from timeloop_env import TimeloopEnv   File "/root/gamma-timeloop/src/timeloop_env.py", line 9, in <module>     from pytimeloop.app import Model   File "/root/pytimeloop/pytimeloop/__init__.py", line 1, in <module>     from bindings import * ImportError: /root/pytimeloop/bindings.cpython-39-x86_64-linux-gnu.so: undefined symbol: _ZN5model6Engine10ParseSpecsEN6config18CompoundConfigNodeE

Hi, I have the same issue, and it is related to pytimeloop, take a look here:
https://github.com/Accelergy-Project/timeloop-python/issues/29
For now I managed to circumvent it by using the Docker container provided with Timeloop (see here), however you need to manually build barvinok and install it inside the container, but that's easy by following the same commands as the Makefile for Timeloop's infrastructure: https://github.com/Accelergy-Project/accelergy-timeloop-infrastructure/blob/1f6bf7cf0071d43e87d5351be3ff960f03af523d/Makefile#L159
I know it's been 4 months, but maybe I could still help.
Let's trust the pytimeloop guys to fix this issue sooner or later.


Facing issue with multipath stream

I tried to run the example script. it compiled successfully and while running it doesn't execute the Streamcall back. I only have these logs can you please help me with this?

You should use discussions first.
You need to read the doc of the library for authentication.
https://github.com/mobizt/Firebase-ESP8266#authentication


Added step to range slider

Hi,
I am using react-native-material-kit in my project and I needed a step property for the range slider.
Here is my solution:
Default step = 1,
Step must be a divisor of max, in order to work, otherwise throw an error

@andramarin Thanks a lot for the great PR!


[v2.9] Update GKE with new BuildUpstreamClusterState functions

Issue: https://github.com/rancher/gke-operator/issues/92
Issue:

Problem

Solution

Testing

Engineering Testing
Manual Testing

Automated Testing

Test types added/modified:

Unit
Integration (Go Framework)
Integration (v2prov Framework)
Validation (Go Framework)
Other - Explain: EXPLAIN
None
REMOVE NOT APPLICABLE BULLET POINTS ABOVE

If "None" - Reason: EXPLAIN THE REASON

If "None" - GH Issue/PR: LINK TO GH ISSUE/PR TO ADD TESTS

Summary: TODO
QA Testing Considerations

Regressions Considerations

TODO
Existing / newly added automated tests that provide evidence there are no regressions:

TODO

@kinarashah are we good with bumping github.com/rancher/rancher/pkg/apis v0.0.0-20240126142034-676c3eb3dfa5 ?


Idea: Images are not shown in /community page

Improvement Idea:
Fix images that are not shown in /community page.
https://www.typescriptlang.org/community

Why:
Because images are not shown in /community page

Alternative ideas:

When you view the page source code of https://www.typescriptlang.org/community ,
you can see that
<img src="[object Module]" class="icon img-square" alt="logo of Hamburg TypeScript"/>

whose src is not proper.

Ah, perfect, thanks


(Bug) Fixing Instrument gif not loading in About Section

🐛 Bug Report
In the about section, the gif for the instrument is not loaded/showing.
I will fix that bug.
Add relevant screenshot or video (if any)

@nilisha-jais Please Assign me under LGMSOC21.

Assigned @ArshWalker


Hi, I am unable to access the documentation

NOTE: Issues that are not bugs or feature requests will be closed. Please ask usage questions on StackOverflow.

TensorFlow installed from (source or binary)?:
TensorFlow version:
Bazel version (if compiling from source):
CUDA/cuDNN version:
GPU Model and Memory:
Exact command to reproduce:

Describe the problem clearly
Source Code / Logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full-traceback. Large logs and files should be attached. Try to reproducible test-case code the bare-minimum necessary to generate the problem

https://www.tensorflow.org/get_started/get_started

Thanks! Sorry, I mean the documentation of TensorFlow Debugger (tfdbg) https://github.com/tensorflow/tensorflow/tree/master/tensorflow/python/debug/examples

I can not see too, is there any document of tensorflow which is whriten by java or scala?

@ayat-khairy : Thanks for pointing out that broken link. In the mean time you can find documentation for tfdbg in https://www.tensorflow.org/programmers_guide/debugger and https://www.tensorflow.org/programmers_guide/tfdbg-tflearn
@caisq : Do you want to update the link in the README?
@geektcp : I'm not sure I understand your question. If you have a separate bug report about documentation, please file an issue with more details. Or if this comment answers your question, great.
Thanks!

A fix has been submitted internally and will be pushed to GtiHub son.


API - break out ownership and person in the db

*Issue:*https://github.com/bcgov/entity/issues/23291 , https://github.com/bcgov/entity/issues/22833
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-204-xsbcv8e4.web.app

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-204-xsbcv8e4.web.app


Fix SetColStyle

PR Details

SetColStyle was returning with no action when no columns were added
to the worksheet before.
Description

This commit uses the same approach as the other col.go functions and
adds the columns to the worksheet if there are none.
Related Issue

#467
Motivation and Context

Fixes 467
Now if styles are added before any column it's present, they are not ignored.
How Has This Been Tested

Unittests.
The test was failing before the fix, and it's passing now
Types of changes

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

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #694 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #694   +/-   ##
=======================================
  Coverage   96.67%   96.67%
=======================================
  Files          30       30
  Lines        8938     8941    +3
=======================================
+ Hits         8641     8644    +3
  Misses        183      183
  Partials      114      114

Impacted Files
Coverage Δ

col.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

@remoremorali Thanks for your PR. I have tested setting column style on the master branch code, and it works well. So I think maybe needn't fix that. After reading related issues, I still don't know the problem clearly. Could you provide the complete code and attachment to reproduce the issue?

I had a problem with applying a date format which worked with SetCellStyle, hence I looked at the code and I thought that I had spotted the issue, and indeed from my tests the style was not there; but it did not influence the styling and it works, it turns out, except when a date value is set before applying the formatting; which is a different bug that this PR won't fix.
You can discard this PR, thank you. I'll check if a bug is already there for the date issue and open one otherwise.


[ALLUXIO-2297] Mark member variable mDirectory final in class Inode

https://alluxio.atlassian.net/browse/ALLUXIO-2297
In Inode.java (alluxio.master.file.meta.Inode),
change
protected boolean mDirectory;
to
protected final boolean mDirectory;
as this member variable is not changed after init

Can one of the admins verify this patch?

alluxio-bot, check this please

alluxio-bot, check this please

alluxio-bot, check this please

alluxio-bot, check this please


🍒 Cherry pick PR #19127 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/19127 to staging 🍒

🚀 Cherry-picked to staging by https://github.com/OSBotify in version: 1.3.15-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.15-12 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Can't craft up mekanism piles

Hey, I know you said I could craft them up with a hammer and 4 poor or 2 normal. But they're not showing up in NEI. Nor I can craft them the way you described (I assume it's shapeless).
Since I disabled few of the mods, I'm getting this sort of message spam every time I boot up(related to every mod I removed):
[23/8/2015 22:08:28 PM] [Client thread/INFO]: [CHAT] ERROR: 1Crafting.zs:52: Could not resolve
Nothing about mekanism though, is there a chance that I somehow buggered up my scripts?
And there's this, but I'm not sure if it's relevant:
[23/8/2015 22:08:20 PM] [Server thread/WARN] [FML]: [HelpFixer] Command mk incorrectly overrides compareTo: mekanism.common.CommandMekanism
[23/8/2015 22:08:20 PM] [Server thread/WARN] [FML]: [HelpFixer] Command mk incorrectly overrides compareTo: mekanism.common.CommandMekanism
[23/8/2015 22:08:20 PM] [Server thread/WARN] [FML]: [HelpFixer] Command minetweaker incorrectly overrides compareTo: minetweaker.mc1710.server.MCServer$MCCommand
[23/8/2015 22:08:20 PM] [Server thread/WARN] [FML]: [HelpFixer] Command minetweaker incorrectly overrides compareTo: minetweaker.mc1710.server.MCServer$MCCommand
[23/8/2015 22:08:20 PM] [Server thread/WARN] [FML]: [HelpFixer] Command mk incorrectly overrides compareTo: mekanism.common.CommandMekanism
Best regards.

Yes as I've said in my description I've heavily modified most recipes. This is reason nothing is working as expected. By removing non-optional mods you've broken the scripts. So most of my scripts haven't even loaded.
I have no issue with anyone modifying the pack anyway they wish. However I can't offer support on how to fix. I've segregated mods scripts as much as possible, but integration is key to the flow of the pack. So the pack works as expected and the problems you are experiencing are a result of your changes.

Please do not log issues about pack unless they also occur with full basic pack loaded.


fixed a bug in SDK not parsing GetMyeBaySellingRequestType response well.. after getting the response GalleryURL of PictureDetails was PictureURL instead

Hey David!
I fixed a bug in SDK not parsing GetMyeBaySellingRequestType response well.. after getting the response GalleryURL of PictureDetails was PictureURL instead.
This problem resulted in me not able to get thumb images of products in my ActiveList.
Please mke sure though that I'm not affecting anything else as I'm not with full knowledge on the code of the SDK.
This is related to issue #113 I opened

According to the eBay documentation the GalleryURL field is not a member of PictureDetailsType. I don't see why you need to add it. Or why the change is required for GetMyeBaySellingRequestType.

Just seen GalleryURL been returned in the API response for the sandbox. Will try production.

Thanks for the fast response David.. If my fix is not good I'll be really happy if you can fix the problem of not getting thumb image in the response for My eBay Selling request

GalleryURL was dropped back in version 997. It looks like version 1019 has just been released. It's possible that either GalleryURL has been put back, or eBay have accidentally added it.

Yes I can also see that the documentation shows PictureURL.. but the response as you can see in my opened issue has only GalleryURL.
Thanks again.

I'm going to have to close this. You can't remove PictureURL as that field is been returned in a lot of calls. For example GetItem.
In addition the file is generated from the WSDLs as stated. These changes are going to be lost the next time I re-create the SDK from the WSDLs.
I'll open a new issue about adding GalleryURL as I'm seeing it been returned in other calls.


GEOMESA-1860 Track label process

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

What documentation should we add to this?

I'll add to the docs - looks like we're missing some other ones there too.

Ayup, I just making sure to ask for docs upfront now since we can do that.


Move to cucumber-ltd organisation

Please @mattwynne then we don't have to bug you to add people like @romaingweb

It's in cucumber now which is fine.


Barrier API

What if we have to check auth token before particular Remote Operation (Query or Mutation) and renew if it is required?
API example:
const authBarrier = createBarrier({
  active: {
    source: combine({ token: $token, now: $now }),
    fn: ({ params }, { token, now }) => parseToken(token.exp) > now,
  },
  perform: tokenRenewMutation,
})

postpone(query, barrier)

Ref — https://redux-toolkit.js.org/rtk-query/usage/customizing-queries#automatic-re-authorization-by-extending-fetchbasequery

Done and will be released in v0.11


feature request: converters with poise context

Currently, we can only use serenity::Context in ArgumentConvert, PopArgument and SlashArgument, because all these traits are imported from serenity.
But what if we want to make a string-to-command converter? We are unable to use the ctx.framework() method in any way to get the command. The only solution would be to export the commands variable from other file, which is not a good solution.
Now the trait looks like this:
#[async_trait]
pub trait ArgumentConvert: Sized {
    type Err;

    async fn convert(
        ctx: &Context /* <- serenity::client::context::Context, NOT poise::Context<'_, Data, Error> */,
        guild_id: Option<GuildId>,
        channel_id: Option<ChannelId>,
        s: &str,
    ) -> Result<Self, Self::Err>;
}

How about creating a custom implementation, with Poise context support? It would be very helpful.

I attempted something similar before, I think it could be made to work.
For motivation purposes and curiosity - what would you need a string-to-command converter for?

Yeah, that's the only use I can think of either.
You can see my attempt from two years ago here in this todo card and this commit


faster_rcnn_nas_coco can't set --num_clones

I've previously tried two models: ssd_mobilenet_v1_coco_2017_11_17 and faster_rcnn_inception_resnet_v2_atrous_coco_2018_01_28 which will both run like this:
run_train()
{
export CUDA_VISIBLE_DEVICES=0,1,2
python3 /home//training/train.py --logtostderr --pipeline_config_path=/home//training/faster_rcnn_nas_coco.config --train_dir=/home//training/models/train --num_clones=3 --ps_tasks=1
unset CUDA_VISIBLE_DEVICES
}
I have 4 GPU's so I've been setting the first three to train and the last one to eval. However, for some reason I'm unable to do the same for the model faster_rcnn_nas_coco_2018_01_28. I get the error:

WARNING:tensorflow:num_readers has been reduced to 1 to match input file shards.
Traceback (most recent call last):
File "/home/awelch/training/train.py", line 184, in
tf.app.run()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/platform/app.py", line 126, in run
_sys.exit(main(argv))
File "/home/awelch/training/train.py", line 180, in main
graph_hook_fn=graph_rewriter_fn)
File "/tensorflow/models/research/object_detection/trainer.py", line 285, in train
clones = model_deploy.create_clones(deploy_config, model_fn, [input_queue])
File "/tensorflow/models/research/slim/deployment/model_deploy.py", line 193, in create_clones
outputs = model_fn(*args, **kwargs)
File "/tensorflow/models/research/object_detection/trainer.py", line 177, in _create_losses
train_config.use_multiclass_scores)
ValueError: not enough values to unpack (expected 7, got 0)

Could anyone please explain why this is or how I can fix it?

Hi ,
Struck with the same issue .....
update_ops = tf.get_collection(tf.GraphKeys.UPDATE_OPS, first_clone_scope)
training stops at this line
Did u find any solution ??
working fine for other models