Windows Store install fails when default app save location not set to C:

When default new app save location was set to D:, WT would download and attempt to install but then fail with a non-specific error (sorry no screenshot but should be repeatable).  Changing the default app save location to C: allowed it to install.
There are a pair of Event Logs describing the event:
Fault bucket<PHONE_NUMBER>692753289, type 5
Event Name: AppxDeploymentFailureBlue
Response: Not available
Cab Id: 0

Problem signature:
P1: 80073D13
P2: 5
P3: 0
P4: NONE
P5: 10.0.18362.175
P6: 765
P7:
P8:
P9:
P10:

Attached files:
[large number of log paths redacted]

These files may be available here:
\\?\C:\ProgramData\Microsoft\Windows\WER\ReportArchive\NonCritical_80073D13_e3ddba5cc7432f8eac6ff587b7cea67b50cc29f4_00000000_614f2c6e-6497-4c50-998f-0bf772615a9e

Analysis symbol:
Rechecking for solution: 0
Report Id: 614f2c6e-6497-4c50-998f-0bf772615a9e
Report Status: 268435456
Hashed bucket: b966d53c8a39f29fd0f6b67c7e88a789
Cab Guid: 0

and
Fault bucket<PHONE_NUMBER>761225254, type 5
Event Name: StoreAgentDownloadFailure1
Response: Not available
Cab Id: 0

Problem signature:
P1: Acquisition;PTIClient
P2: 80073d13
P3: 18362
P4: 175
P5: Windows.Desktop
P6: 1
P7:
P8:
P9:
P10:

Attached files:
\\?\C:\ProgramData\Microsoft\Windows\WER\Temp\WER195D.tmp.WERInternalMetadata.xml

These files may be available here:
\\?\C:\ProgramData\Microsoft\Windows\WER\ReportArchive\NonCritical_Acquisition;PTIC_2431d4c27ca3d41ba382d17ec8f30eadeab812e_00000000_fd9920f6-19ed-4ead-b665-b667864e43cc

Analysis symbol:
Rechecking for solution: 0
Report Id: fd9920f6-19ed-4ead-b665-b667864e43cc
Report Status:<PHONE_NUMBER>
Hashed bucket: 8ef9d1b328f66c23c3fa9f9e36328826
Cab Guid: 0

Wow! I'm going to file an internal bug to the store folks, as this should absolutely not be the case.

Still a problem with same event log error (except error code is 80073CF3) on Windows 2004 19041.208


ciao-image: Add unit testing for uploadImage function

Add the test case for uploading an Image. Basically, adds the new
HTTP PUT request to the list of image API tests.
This fixes #575
Signed-off-by: Munoz, Obed N<EMAIL_ADDRESS>

Coverage increased (+0.09%) to 66.852% when pulling 7ba3b18b1b39dd70da07db256605ea024c2e496e on obedmr:unit-testing into b938a427b384da39ae9d23da803f902451f42d47 on 01org:master.

@kaccardi , when you have a chance, could you take a look on this? as far I could see, this was the only missing piece to test this uploadImage function.


Update ProcessSignoutCallbackAsync

Fix spelling of signout in debug log from EndSessionEndpoint.ProcessSignoutCallbackAsync.

Hi @dls314, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

thanks!


[MSYS-661] Fix chef_client_scheduled_task idempotence

Description
Updated cookbook for making chef_client_scheduled_task resource idempotent.
Issues Resolved
Fixes https://github.com/chef-cookbooks/chef-client/issues/490

Would the scheduled task resource update if someone wanted to change the functionality of the scheduled task? For example, would a new task be created if one wanted to change the interval or splay?

Per current chef-client cookbook chef_client_scheduled_task resource will not update the interval or splay.
I have made it idempotent for only if scheduled task is created already.

@btm @tas50 , Please review.

Won't this prevent the task from ever updating if the options or other properties change?

@tas50 , can you please confirm the reason of adding start_time in case of minutes frequency: https://github.com/chef-cookbooks/chef-client/blob/master/resources/scheduled_task.rb#L45? Does removing it has any bad impact?
We checked the commit history of the file, but the reason isn't very clear from the commit messages.

We have identified a bug in windows_task resource for frequency :monthly that it is not idempotent. Also found that for frequency :once , frequency_modifier in windows_task option should not be provided. We will be fixing it soon on another PR so closing this now.


Seems to be an older version

Hi, I found your comments on the parent-project (gradusnikov) very helpful and was hoping that your fork would incorporate these ideas.
I built the Plugin as described but the preferences dialog is all about OpenAI.
Instead of "Ollama API Base Address" (like in the screenshot) I get "OpenAI API Base Address" and "OpenAI API Key"
My local ollama installation seems to be working correctly since it responds on http://<IP_ADDRESS>:11434
I guess the latest sources (of the version shown in the screenshots on your github project page) are not on github since I didn't find any hint to "ollama" neither in PreferencePage.java nor in the rest of the plugin-code.

Yeah, I removed all the stuff that uses the Ollama API as it was too buggy and since then Ollama added an Open AI compatable API it isn't really needed anymore.
You can still use it will Ollama if you add /v1 to the URL, but it won't show the list of models like is did before:
http://localhost:11434/v1/

or possibly:
http://localhost:11434/v1

I do plan to add the ability to get a list of models again, but all the OpenAI compatable servers have a slightly different way of listing the model names.
Once I've done that I will redo all the screenshots for Github as there are quite a lot of improvements I've added since them:

Spell checking in the user input box.
More right click options for the AI page.
Reloads the AI pages on open/close.

and a lot of other little tweaks.

Thanks alot for the quick answer.
Got it running with ollama and llama3.2:latest and I am amazed by the features you added.

Thanks alot for the quick answer. Got it running with ollama and llama3.2:latest and I am amazed by the features you added.

No problem and glad you got it going!
I also removed a lot of features from Wojciech's version:

The LLM "function-calling" stuff as it didn't really seem to work well with local LLMs and in general didn't seem all that useful.
Anything that was Java-specific like the code Wojciech had written to parse the AST of the Java source files (I wanted it to work with C++ via Eclipse CDT, Python via PyDev, etc).
I had to remove all the @inject Dependency Injection stuff as it was a huge source of really weird NullPointerException bugs... Sadly I think these were probably 99% of the reason people struggled to get Wojciech's version working and why he might have stopped working on it :/

The only things left on my ToDo list of things to add are:

Add back the model selection stuff (possibly on the main view instead of in the settings pages so much easier to switch models).
Add code to export the chat as a text file in Markdown.
See if I can add either multiple views and/or a branching chat (eg: undo, redo, left child, right child tree traversing like a lot of LLM frontends use).
Try to get LaTeX working using KaTeX or MathJax (I've tried this a couple of times without managing to figure it out sadly).

I've added back the ability to get the model list but only tested it on openrouter and openai, so not sure if it works with ollama or llama.cpp's server...
Beware the settings page is very buggy unless you quit and reopen it and Apply and Restore Defaults don't work properly a lot of the time (I'll try and get to the bottom of this later).

I'd hold off as the current method isn't really usable at all:
https://github.com/jukofyork/aiassistant/discussions/4


ChannelInboundMessageHandlerAdapter cannot be resolved to a type

I am just trying to apply the web socket example, this error appears:
ChannelInboundMessageHandlerAdapter cannot be resolved to a type
i am new to netty i successfully add netty to class-path, i use eclipse.

@manee1982 where you have the example from ? This class does not exists since 4.0.0.Final was released for sure


React to errorhandler and errorpage extensions changes

@danroth27

Hi @kichalla, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

:shipit:


Use the new delete dialog in more case

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] ~Server: I added multiple integration tests (Spring) related to the features~
[x] ~Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)~
[x] ~Server: I implemented the changes with a good performance and prevented too many database calls~
[x] ~Server: I documented the Java code using JavaDoc style.~
[x] Client: I added multiple integration tests (Jest) related to the features
[x] ~Client: I added authorities to all new routes and check the course groups for displaying navigation elements (links, buttons)~
[x] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I added multiple screenshots/screencasts of my UI changes
[x] Client: I translated all the newly inserted strings into German and English

Motivation and Context

We want to have less code duplication and reuse delete dialog everywhere, as described in this issue #833. It will also resolve issue #763.
Description

To encapsulate the knowledge about opening delete dialog in this PR I created a special directive that takes care of it. However, there is still duplication of button styles, but for that we will need to avoid using button-group.
I extended the functionality of delete dialog to also show two additional checkboxes, the values of the checkboxes are returned in the event and can be handled by the component calling the dialogue.
New dialog now used in deletion of:

File Upload Exercise
Quiz Exercise
User deletion (in User managment)
Motification deletion (in Notification managment)
Lecture (added type name to confirm to avoid accidental removal of lectures)
Participation (additional checkboxes text is red, the same as for programming exercise deletion)
Programming Exercise

Steps for Testing

Log in to Artemis
Navigate to Course Administration
...

Screenshots

Deleting programming participation

Deleting lecture

Deleting quiz exercise

Deleting file upload exercise

Deleting user

Deleting notification

Note: here we had notification id, but maybe makes more sense to use title.
Deleting Programming Exercise

Good, that we have consistent dialogs here now, thank you!
There are some translation issues and I would generalize the checkboxes in the delete dialog more.
Also, I don't really like the flow of opening the dialog itself, but as I said in my comment, this is open for discussion

Thank you for checking @ungaralex . I applied your suggestions for the translation strings. Also generalized the checkboxes and made the return value more clear, so what you pass is what you receive in the delete event.
The flow of openning the dialog would be of course better with the component, but I think considering the changes needed for this it is better for now to have directive.  The good thing with the directive approach is that it can be easily switched to the component, if needed (only small changes in the template required).

The button layout in the course admin page with all exercises is not correct any more:

Delete and Reset buttons for programming exercise and quiz exercise do not align correctly. Please fix this.
It would also be cool, if we could reset and delete consistently (reset top and delete bottom) for both exercise types

The button layout in the course admin page with all exercises is not correct any more:

Delete and Reset buttons for programming exercise and quiz exercise do not align correctly. Please fix this.
It would also be cool, if we could reset and delete consistently (reset top and delete bottom) for both exercise types

You are right, I missed the bottom margin. Now it looks properly and also I switched the position of Reset and Delete for Quiz Exercise


More verbose Authentication/permission errors

Hey,
I have two suggestions.

When authenticating I overlooked that I was using the wrong clientId. The error the module gave was generic. Microsoft itself probably returned a far more detailed error but this wasn't returned. Would it be possible to just pass through the error Microsoft gives? In powershell I'd implement this as throw "Authentication error: $($_.Exception.Message)".

When trying to retrieve data I missed that the application did not have permission to the specific table. Instead of returning an error I just got empty data. I only got an error when I tried to look up a specific record.

P.S. love the module so far, its feeling far more versatile than Microsoft.Xrm.Data.PowerShell which is stuck on powershell 5.

Thank you @rvdwegen, these are good suggestions. I will certainly look into it.

I will soon push the new change.  How did you do that?

I don't even remember haha, I just grabbed a random web based image edit tool

I don't even remember haha, I just grabbed a random web based image edit tool

I thought it's a PowerShell / Terminal thing, because it looks like it's been replaced with ASCII characters 😄


Copy to clipboard initial implementation

Initial implementation of #7.

Can we have the migration to Ace-Editor in another PR?

I will close this PR for now since I realized copying to clipboard will interfere with the editor changes. I will raise a PR for ace editor first then implment the copy to clipboard on top of it.


New version: Piriform.CCleaner version 6.13

- [X] Have you signed the Contributor License Agreement?

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.4 schema?

Resolves #111578

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/111601&drop=dogfoodAlpha

/azp run

Manual Validation failed with:

Faulting application name: CCleaner64.exe, version: <IP_ADDRESS>17, time stamp: 0x64809d27
Faulting module name: ntdll.dll, version: 10.0.19041.1288, time stamp: 0xa280d1d6 Exception code: 0xc0000005 Fault offset: 0x00000000000a1fe7
Faulting process id: 0xdec
Faulting application start time: 0x01d9b1423420702e
Faulting application path: C:\Program Files\CCleaner\CCleaner64.exe
Faulting module path: C:\Windows\SYSTEM32\ntdll.dll Report Id: 9fb3d1f3-4735-416c-8701-bf2409d7eb36
Faulting package full name:
Faulting package-relative application ID:


[FLINK-5825][UI]make the cite path relative to show it correctly

In yarn mode, the web frontend url is accessed from yarn in format like "http://spark-91-206:8088/proxy/application_1487122678902_0015/", and the running job page's url is "http://spark-91-206:8088/proxy/application_1487122678902_0015/#/jobs/9440a129ea5899c16e7c1a7e8f2897b3".
One .png file called "horizontal.png", which is very small can not be loaded in that mode, because in "index.styl" it is cited as absolute path.
We should make the path relative.

[x] Tests & Build
Before the fix:

After the fix:

Thanks for the pull request. Let me try this out and merge it if everything works as expected.


Change sorting, And fine-tuning some weapons

🤔 What type of PR is this? (check all applicable)

[ ] 🍕 Addition
[ ] ⌨ Productivity
[x] 🐛 Bug Fix
[x] 🔥 Optimization
[ ] ⚙ Configuration
[ ] 🌟 Quality Of Life
[x] ✨ Enhancement
[ ] 📝 Documentation

📝 Description
Change sorting, And fine-tuning some weapons, Fix turret.

🧐 The Rationale

🎯 Key Objectives

🚦 Testing
√

⏮ Backwards Compatibility

📚 Related Issues & Documents

🖼 Screenshots/Recordings

📖 Added to documentation?

[ ] 📜 README.md
[ ] 📓 Javadoc
[ ] 🍕 Comments
[x] 🙅 no documentation needed

😄 [optional] What gif best describes this PR or how it makes you feel?

To readd the Hecate it will need to be remastered with the new system it currently lacks:
Customization (No different parts)
Animations for the new system
And the model is outdated and below the standard for the mod.
Again, you should ask designers if they are up to making a Hecate.


Treat all warnings as errors

Nabla should be compiled with /WX (for MSVC, for GCC/Clang -Werror) flag enabled for compiler and linker which treats all warnings as errors, then all of them should be eliminated and sources adjusted. Doing so may save us in future, recursive warning generation may lead to unresponsive builds taking more then half an hour, also we decrease a chance for undefined behaviours at runtime.

this should be toggle-able, introduce NBL_TREAT_WARNINGS_AS_ERRORS and enabled by default but let user to override it as CMake option

You'll run into problems with that, because whenever you include a 3rdparty header you get all of its warnings.
So it's not as easy as just enabling WX because you don't want to be bogged down into forking every dependency and "fixing" their warnings.
P.s. there was a story one about an intern or a junior who fixed uninitialised variables in openssl and broke it cause they were actually a source of entropy for the rand gen

I would limit fixing them to our headers & sources only

who fixed uninitialised variables in openssl and broke it cause they were actually a source of entropy for the rand gen

yeah I get you, though I don't think that would be a case if this happened in our codebase, I rather stick to initialize variables not to some random values which may lead to bugs, but of course it's not a case as well to fix all 3rdparty headers

You'll run into problems with that, because whenever you include a 3rdparty header you get all of its warnings.

well another problem this could introduce is an implicit vendor/toolset dependency, but since we are targeting latest toolsets and cpp standard I would not care much

by the way, there are solutions how to disable/ignore warnings in external headers coming from 3rdparty and still treat all warnings as errors with our codebase
#pragma warning(push, 0)
// Some include(s) with unfixable warnings
#pragma warning(pop)

also https://devblogs.microsoft.com/cppblog/broken-warnings-theory/

by the way, there are solutions how to disable/ignore warnings in external headers coming from 3rdparty and still treat all warnings as errors with our codebase
#pragma warning(push, 0)
// Some include(s) with unfixable warnings
#pragma warning(pop)

also https://devblogs.microsoft.com/cppblog/broken-warnings-theory/

with that you don't have this problem you mentioned

because whenever you include a 3rdparty header you get all of its warnings.


Grey background color/text highlight in high contrast mode

Link to PDF file:
https://mozilla.github.io/pdf.js/web/viewer.html
Configuration:

Web browser and its version: Firefox Developer Edition 88.0b9
Operating system and its version: Windows 10 19042.870
PDF.js version: Latest version
Is a browser extension: No

Steps to reproduce the problem:

Switch to high contrast mode.
Open a PDF file.

What is the expected behavior

Text and background colors are as defined in the file:

What went wrong?

Text has a a highlight/grey background in high contrast mode:

Possible solution:

Add the following code after line 32 in pdf_viewer.css

.pdfViewer > .page > .textLayer {
    forced-color-adjust: none;
}

forced-color-adjust, however, is not supported in Firefox.

forced-color-adjust, however, is not supported in Firefox.

For reference, that's being tracked in https://bugzilla.mozilla.org/show_bug.cgi?id=1591210

I've reopened the issue since it's still reproducible in Firefox.

I've reopened the issue since it's still reproducible in Firefox.

https://bugzilla.mozilla.org/show_bug.cgi?id=1591210 has been fixed, so hopefully this should work now.

WFM, when testing various high-contrast themes in Windows 11 and Firefox Nightly 115.0a1.


Port ranges for entrypoints in static configuration

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Hi, I have a problem similar to #1677, even if you have added template support for file provider(dynamic configuration),but it isn't for static configuration,as you mentioned in docs:

Go Templating only works with dedicated dynamic configuration files. Templating does not work in the Traefik main static configuration file.

So, you dosen't solve the problem in #1677(want to use template in entryPoints key,but entryPoints key must be in static configuration)  actually in my opinion, do you have any advices for this problem?
Thank you in advance !

Any updates on this topic? 🤞

@traefiker don't auto-close

I'm also having this issue. Basically, I want a custom HTTPS port in my redirect so that I can run multiple different compose stacks on the same machine bound to localhost. This works fine with the docker provider and labels but I am having an issue where sometimes Windows Docker is not starting the containers correctly so that traefik is not discovering the other services, so I figured a file provider will fix that.

Also, another way to happily solve this would be if cmd line args could layer over the top of the static file... if thats possible?

Any update on this issue?

Seems to be of interest at least for 5 years and now it also hit me.

I currently have to accept a range of 50-100 ports to be forwarded to the target containers.
So I have to write those manually in the config file for now. Would be nice to have a range configuration available

Same problem.

Same here

Hey there, any idea on a timeline for this issue? Thank you :)

Hi guys,
Any updates on this? 😄

Any movement on this?

Seems like a really useful feature, but for very few people
https://github.com/traefik/traefik/issues/1677

Hello @xzycn
We are interested in this issue, and we would love some help from the community.
If you or another community member would like to contribute on this, let us know, and we will work with you to make sure you have all the information needed.
Let us know here before you start. We prefer to work with our community members to ensure that we are aligned and can move quickly with the review and merge process.

please reconsider this

@xucian I'm not sure what should be reconsidered as we asked for some help from the community on this.
If anyone can contribute, we would love to help!


[ZEPPELIN-5834] Remove last references to Hadoop 2.6 as JDK11 is not supported.

What is this PR for?
This PR updates Hadoop 2.6 dependencies to 2.7.
What type of PR is it?
Improvement
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5834

How should this be tested?

CI

Questions:

Does the licenses files need to update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

If no further comments are received till Thursday, I will merge the change into the master to proceed quickly with JDK 11 support.


[Sage-885] Description - Design Spec Updates

Description

Updates the Description to new design specs and props.
Figma
Screenshots

Testing in sage-lib

Rails

View Rails component
Check that component renders as expected and new props function as intended.

React

View Storybook
Check that new prop functions as expected

Testing in kajabi-products

(MEDIUM) Updates the Description to new design specs and props. Some props have been deprecated. This requires a patch to Kajabi Products to address.
Related

Closes #885

Hi Ashley! Looking great so far. I added a couple of notes and also these can be added to the components_helper.rb file.
react_component_slug: "sage-description--default",
figma_embed: "https://www.figma.com/embed?embed_host=share&url=https%3A%2F%2Fwww.figma.com%2Ffile%2F9Km09NjlZHYWsMP7EGT8tI%2FSage-3-for-Admin%3Fnode-id%3D19501%253A25347",

@monicawheeler @philschanely Thanks for the feedback! I'll get these changes in soon.

Updates:

Removes link prop
Prop documentation
action prop
Grid styling when no action is present
CSS custom prop logic for title and action

@philschanely Appreciate you taking this over in my absence. These changes look very good!


[Fix](inverted index) fix missed array inverted index null bitmap #38907

cherry pick from #38907

run buildall


Save unreplicated optimizer in lm1b example.

Whaaat.  I thought I'd fixed this... I guess I did so in the full translation wmt example and somehow just forgot to fix it on lm1b.   Thanks so much for catching this, no desire to save stupidly redundant checkpoints!


Support passing stepsize directly to utility adaptor

It turns out that Turing doesn't use the utility consturctors (i.e. passing integrators to adaptors) and we need the interface of passing floats or vectors of floats back.

Thanks for merging this. I think now we can

make a new release for AHMC
wait the CI for https://github.com/TuringLang/Turing.jl/pull/1218 to pass
make a new release for Turing

Sounds good to me!

Cool I will take care of it.


lists with trailing comma give an unhelpful warning

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed.

Specifially, check out the wishlist, open RFCs,
or feature requests.

Reduced the issue to a self-contained, reproducible test case.

Description
def foo: list nat := [ 1,2, ] results in the somewhat unhelpful warning declaration 'foo' uses sorry. (Notice the trailing comma in 1,2,.)
I noticed that recently Lean has started allowing a trailing comma when creating structures. Can we allow the same flexibility here?
Versions
Lean (version 3.2.1, commit 3392aa90b5ec, DEBUG)

@gebner The issue is here:
https://github.com/leanprover/lean/blob/master/src/frontends/lean/parser.cpp#L1383
Is this a typo, or did you have a reason for not reporting an error on the empty case?

@leodemoura This is a terrible hack, and I'm not happy with it either.  To recap, the problem this line tries to solve is the following:

We expose a pexpr parser to the VM.  This parser is supposed to fail (without reporting an error) if there is no expression.  If there is no expression, we hit this line.
On the other hand, if we parse an expression from C++, we never backtrack and always want an expression.  In this case, we want an error though.

I see multiple solutions:

Report the errors, but store them in a list first.  On backtracking in the VM pexpr parser, we can revert the error messages.
Remember the last backtracking point.  The VM pexpr parser stores the current position before parsing an expression.  Then we know whether to backtrack without an error (say via an exception), or report an error and continue.

I had hoped that nobody would notice until we refactored the parser in #1674.  I'll try option 2 now.


Incorrect message when opening clearnet links from browser (Linux & OSX)

Steps to reproduce:

Launch SAFE browser and navigate to site with clearnet links.
Eg safe://safenetworkprimer
Click on clearner link

Expected result:
Cleanet link will open in default web browser, message in SAFE browser relates to that clearnet URL.
Actual result:
Clearnet link opens in default web browser, message in SAFE browser references the SAFE net URL:
_"Detected HTTP/S protocol.
Redirecting safe://safenetworkprimer to be opened by your default Web browser."_
This is seen in Linux:

And Mac:

Testing the handling of clearner links in the most up to date browser (built from master today):

We can see that the redirect message now does indeed reference the link that has been clicked on - ie is the desired behaviour.

Moved to done.


Fix for alt-tabbing out of input controls

Tab is no longer considered to be captured exclusively because of global ALT-Tab hotkey:
https://discord.com/channels/531175899588984842/534492173362528287/882301824349122620

Looks good!  Fixes, adequately, #420.


UI: Update AddAccountOrUserToProject to show a drop down for account & user

Description
This PR fixes https://github.com/apache/cloudstack/issues/7216

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

@vishesh92 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9072 (QA-JID-339)


Thunn/add convert for frequency to midi

Added convert functions (https://github.com/interactiveaudiolab/penn/issues/13):
- frequency_to_midi(frequency) -> midi notes
- midi_to_frequency(midi) -> frequency
Implemented in torch based on librosa numpy implementations.
Added tests to confirm outputs match librosa.
Added github workflows to run pytest. (these could optionally be removed form the pr)

Thanks for the quality PRs! I have merged and released both as part of version 0.0.14.


Zuul simple routing whether to supplement the fallback feature

Hi @spencergibb @ryanjbaxter
I know that zuul integration eureka default support hystrix function. When the route fails, we can get the reason of the failure by FallbackProvider , at the same time allow users to customize the return information
But zuul is routed through SimpleHostRoutingFilter  when we do not integrate eureka. FallbackProvider  is not provided in this case.
Finally, I would like to say：
whether or not zuul integrates with eureka, should we all provide FallbackProvider  feature?
If you think I made sense, I'm willing to submit PR. look forward to your reply, thanks again : )

No. Because the integration is actually with ribbon which can be configured without eureka using static URLs

@spencergibb
Thank you for your reply, I think I still do not quite understand.
When we route static URLs, I know that we are using SimpleHostRoutingFilter  instead of RibbonRoutingFilter. (Because the conditions to determine whether RibbonRoutingFilter  is executed ctx.get(SERVICE_ID_KEY) != null ) Processing failed in SimpleHostRoutingFilter  is returned as a ZuulRuntimeException . I did not see any logic related to FallbackProvider  in SimpleHostRoutingFilter.
Finally, thank you again for your reply to my question during the holidays

You would go thru ribbon routing filter using static configuration server list rather than eureka server list

@spencergibb
Thank you very much, I think I got it. Like you said when using the ribbon is the default integrated FallbackProvider . But I would like to ask another question is why we do not provide FallbackProvider  features when using SimpleHostRoutingFilter ?

Because the fallback mechanism is part of Hystrix which is not part of simple host routing filter

@spencergibb
I try to understand what you mean:
Zuul is default integrated hystrix and provides the fallback mechanism when using ribbons. When we do a simple host routing we do not need to provide a fallback mechanism (get the error directly back to the user will be better)
I understand this right? :)


[ENH] Legacy test refactor - move test_data_processing, mtype handling in test_classifier_output

This PR makes changes to legacy tests:

moves test_data_processing into datatypes since it tests only functions in there (not in datasets)
changes the logic in test_classifier_output that obtains number of instances in X_new - to make this compatible with arbitrary mtype, uses the check_is_scitype function

(approved as part of #2508)


Footer Option?

Html5 Tables have 3 components.
Header, Body, and Footer.
The code revolves around header and body, but completely ignores the footer. It's completely overlooked, and ignored, when modifying columns with js. Not complaining... Just pointing out that, it's lacking a key area. I want to try to help...

+1

This is al ready solved in the dev-branch :) Any help testing this branch is appreciated! ✌
Cheers,
Nadan Gergeo

On 6 apr. 2016, at 15:13, Konstantin Pluschnikow<EMAIL_ADDRESS>wrote:
+1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub


lora resume error

command:
python -m mlx_lm.lora --model ./Phi-3-mini-128k-instruct --adapter-path adapters-pluc100 --resume-adapter-file adapters-pluc100 --data ./data-pluc100  --train --iters 200 --lora-layers 8
worked in previous version (0.14) but now it gives with 0.14.3
File "/Users/pro/mambaforge/envs/torch/lib/python3.11/site-packages/mlx_lm/lora.py", line 173, in train_model
print(f"Loading pretrained adapters from {resume_adapter_file}")
^^^^^^^^^^^^^^^^^^^
NameError: name 'resume_adapter_file' is not defined

me too

This is fixed on main. We will do a release soon with the update.


1127 Product Homepage Category List

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #1127
Test URLs:

Before: https://main--danaher-ls-aem--hlxsites.hlx.page/
After: https://1127-product-homepage-categories-list--danaher-ls-aem--hlxsites.hlx.page/us/en/products
After: https://1127-product-homepage-categories-list--danaher-ls-aem--hlxsites.hlx.page/us/en/products/brands/sciex

@rgravitvl  and @sharanagoudapatil  - I made changes to the text and headers, plus removed grey paragraph.  Let me know if you are all good with the changes.


Support for version 5.3

any Chance u can Upgrade it to 5.3?
Thanks in Advance!
i really love your Plugin.

I will tomorrow thaanks Bro! As always Good Job ^=^
seiko_dev @.***> schrieb am Do., 21. Sep. 2023, 15:16:

OK, try it out.
https://github.com/seiko-dev/UI_Blooman/releases/tag/v0.1.4
—
Reply to this email directly, view it on GitHub
https://github.com/seiko-dev/UI_Blooman/issues/66#issuecomment-1729565445,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX7ECJ7CHSZLEQWFGRM42LX3Q4ZTANCNFSM6AAAAAA46HAW2Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

as always good job works good!
Am Fr., 22. Sept. 2023 um 01:19 Uhr schrieb Kuro Sekai <
@.***>:

I will tomorrow thaanks Bro! As always Good Job ^=^
seiko_dev @.***> schrieb am Do., 21. Sep. 2023, 15:16:

OK, try it out.
https://github.com/seiko-dev/UI_Blooman/releases/tag/v0.1.4
—
Reply to this email directly, view it on GitHub
https://github.com/seiko-dev/UI_Blooman/issues/66#issuecomment-1729565445,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX7ECJ7CHSZLEQWFGRM42LX3Q4ZTANCNFSM6AAAAAA46HAW2Q
.
You are receiving this because you authored the thread.Message ID:
@.***>


Submitting Quickstart For Summit HOL

@shawnnamdar, per @iamontheinet:

Everything under "Sign Up for a Sigma Trial:" in Overview step should someone just go under Setup step. (Basically all the code, script, etc, required should be moved.) See this as a reference.

The last step should be renamed to "Conclusion and Resources" and should look like this

@sfc-gh-ccardillo  and @iamontheinet I have made the requested updates. Please let me know if anything else needs to be modified.


[FLINK-35808] Let ConsumerConfig.(KEY|VALUE)_DESERIALIZER_CLASS_CONFIG be overridable by user in KafkaSourceBuilder

What is the purpose of the change
Let ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG and ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG be overridable by user in KafkaSourceBuilder, in order to enable the Large Message use-case discussed in this mailing list discussion.
This allows users to easily implement the claim check large message pattern without bringing any concerns into the Flink codebase otherwise, by specifying a value.deserializer that handles it, but otherwise passes through the bytes.
Note: overriding value.serializer is already supported on the Producer side.
Other Reading:
https://www.kai-waehner.de/blog/2020/08/07/apache-kafka-handling-large-messages-and-files-for-image-video-audio-processing/
https://www.conduktor.io/kafka/how-to-send-large-messages-in-apache-kafka/#Option-1:-using-an-external-store-(GB-size-messages)-0
Brief change log

Updates key and value deserializers to be overridable by users in KafkaSourceBuilder

Verifying this change

[x] Test that both key and value  deserializers can be overridden
[x] Tests to ensure that the user-supplied deserializer(s) returns bytes (byte[])

Does this pull request potentially affect one of the following parts:

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

Documentation

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

Thanks for your review @fapaul ! I did another pass on trying to incorporate your comments, and also updated some of the documentation that mentions the deserializer configuration.

Thanks for the review. Parameterized the tests and ready for another look 🙏

@klam-shop the CI is failing due to not correclt formatted code https://github.com/apache/flink-connector-kafka/actions/runs/9957480138/job/27552347466?pr=108#step:15:127 can you run mvn spotless:apply ?

When I ran mvn spotless:apply locally, no diffs were made..
$ mvn spotless:apply

[INFO] Scanning for projects...
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.flink:flink-connector-kafka-python:pom:3.3-SNAPSHOT
[WARNING] 'build.plugins.plugin.(groupId:artifactId)' must be unique but found duplicate declaration of plugin org.apache.maven.plugins:maven-dependency-plugin @ line 147, column 21
[WARNING]
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
[WARNING]
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] Flink : Connectors : Kafka : Parent                                [pom]
[INFO] Flink : Connectors : Kafka                                         [jar]
[INFO] Flink : Connectors : SQL : Kafka                                   [jar]
[INFO] Flink : Connectors : Kafka : E2E Tests                             [pom]
[INFO] Flink : E2E Tests : Streaming Kafka base                           [jar]
[INFO] Flink : E2E Tests : Streaming Kafka                                [jar]
[INFO] Flink : E2E Tests : Common Kafka                                   [jar]
[INFO] Flink : Connectors : SQL : Kafka : Python                          [pom]
[INFO]
[INFO] -----------< org.apache.flink:flink-connector-kafka-parent >------------
[INFO] Building Flink : Connectors : Kafka : Parent 3.3-SNAPSHOT          [1/8]
[INFO]   from pom.xml
[INFO] --------------------------------[ pom ]---------------------------------
[INFO]
[INFO] --- spotless:2.27.1:apply (default-cli) @ flink-connector-kafka-parent ---
[INFO] Spotless apply skipped
[INFO]
[INFO] ---------------< org.apache.flink:flink-connector-kafka >---------------
[INFO] Building Flink : Connectors : Kafka 3.3-SNAPSHOT                   [2/8]
[INFO]   from flink-connector-kafka/pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- spotless:2.27.1:apply (default-cli) @ flink-connector-kafka ---
[INFO] Spotless apply skipped
[INFO]
[INFO] -------------< org.apache.flink:flink-sql-connector-kafka >-------------
[INFO] Building Flink : Connectors : SQL : Kafka 3.3-SNAPSHOT             [3/8]
[INFO]   from flink-sql-connector-kafka/pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- spotless:2.27.1:apply (default-cli) @ flink-sql-connector-kafka ---
[INFO] Spotless apply skipped
[INFO]
[INFO] ----------< org.apache.flink:flink-connector-kafka-e2e-tests >----------
[INFO] Building Flink : Connectors : Kafka : E2E Tests 3.3-SNAPSHOT       [4/8]
[INFO]   from flink-connector-kafka-e2e-tests/pom.xml
[INFO] --------------------------------[ pom ]---------------------------------
[INFO]
[INFO] --- spotless:2.27.1:apply (default-cli) @ flink-connector-kafka-e2e-tests ---
[INFO] Spotless apply skipped
[INFO]
[INFO] ----------< org.apache.flink:flink-streaming-kafka-test-base >----------
[INFO] Building Flink : E2E Tests : Streaming Kafka base 3.3-SNAPSHOT     [5/8]
[INFO]   from flink-connector-kafka-e2e-tests/flink-streaming-kafka-test-base/pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- spotless:2.27.1:apply (default-cli) @ flink-streaming-kafka-test-base ---
[INFO] Spotless apply skipped
[INFO]
[INFO] ------------< org.apache.flink:flink-streaming-kafka-test >-------------
[INFO] Building Flink : E2E Tests : Streaming Kafka 3.3-SNAPSHOT          [6/8]
[INFO]   from flink-connector-kafka-e2e-tests/flink-streaming-kafka-test/pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- spotless:2.27.1:apply (default-cli) @ flink-streaming-kafka-test ---
[INFO] Spotless apply skipped
[INFO]
[INFO] --------< org.apache.flink:flink-end-to-end-tests-common-kafka >--------
[INFO] Building Flink : E2E Tests : Common Kafka 3.3-SNAPSHOT             [7/8]
[INFO]   from flink-connector-kafka-e2e-tests/flink-end-to-end-tests-common-kafka/pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- spotless:2.27.1:apply (default-cli) @ flink-end-to-end-tests-common-kafka ---
[INFO] Spotless apply skipped
[INFO]
[INFO] -----------< org.apache.flink:flink-connector-kafka-python >------------
[INFO] Building Flink : Connectors : SQL : Kafka : Python 3.3-SNAPSHOT    [8/8]
[INFO]   from flink-python/pom.xml
[INFO] --------------------------------[ pom ]---------------------------------
[INFO]
[INFO] --- spotless:2.27.1:apply (default-cli) @ flink-connector-kafka-python ---
[INFO] Spotless apply skipped
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for Flink : Connectors : Kafka : Parent 3.3-SNAPSHOT:
[INFO]
[INFO] Flink : Connectors : Kafka : Parent ................ SUCCESS [  0.060 s]
[INFO] Flink : Connectors : Kafka ......................... SUCCESS [  0.002 s]
[INFO] Flink : Connectors : SQL : Kafka ................... SUCCESS [  0.001 s]
[INFO] Flink : Connectors : Kafka : E2E Tests ............. SUCCESS [  0.001 s]
[INFO] Flink : E2E Tests : Streaming Kafka base ........... SUCCESS [  0.001 s]
[INFO] Flink : E2E Tests : Streaming Kafka ................ SUCCESS [  0.005 s]
[INFO] Flink : E2E Tests : Common Kafka ................... SUCCESS [  0.001 s]
[INFO] Flink : Connectors : SQL : Kafka : Python .......... SUCCESS [  0.001 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  0.188 s
[INFO] Finished at: 2024-07-17T09:18:42-04:00
[INFO] ------------------------------------------------------------------------

I tried to apply the fixes locally.

I followed the Code Formatting instructions in https://nightlies.apache.org/flink/flink-docs-release-1.19/docs/flinkdev/ide_setup/#code-formatting and used the IDE to reformat the file.

I think I found the culprit. I checked out your branch, and initially, mvn spotless:apply didn't produce any changes, although the check style failed. After switching to Java 11 locally and running mvn spotless:apply, the code formatted correctly.
I also verified that running mvn org.apache.maven.plugins:maven-checkstyle-plugin:3.1.2:check passes now locally.

I think I found the culprit. I checked out your branch, and initially, mvn spotless:apply didn't produce any changes, although the check style failed. After switching to Java 11 locally and running mvn spotless:apply, the code formatted correctly.

I also verified that running mvn org.apache.maven.plugins:maven-checkstyle-plugin:3.1.2:check passes now locally.

Changed to Java 11 locally and ran
mvn spotless:apply
mvn org.apache.maven.plugins:maven-checkstyle-plugin:3.1.2:check


analytic adjoints

Just a start for now. I think I also may need to change the API for the rate constants, since based on the benchmarks in testing_ad.jl on this branch, things seem a LOT faster if it's all done via compute_k rather than making the model objects callable. @DhairyaLGandhi, does that make sense? If so I'll go ahead and make that change throughout. We can keep the callable syntax for demonstration purposes but just have it call out to compute_k rather than the other way around as it is presently.

Codecov Report

Merging #28 (80dd1a0) into diy_integrals (3077427) will decrease coverage by 0.49%.
The diff coverage is 0.00%.

@@                Coverage Diff                @@
##           diy_integrals      #28      +/-   ##
=================================================
- Coverage          43.07%   42.58%   -0.50%
=================================================
  Files                  7        7
  Lines                260      263       +3
=================================================
  Hits                 112      112
- Misses               148      151       +3

Impacted Files
Coverage Δ

src/kinetic_models.jl
72.41% <0.00%> (-2.59%)
:arrow_down:

Continue to review full report at Codecov.

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


LoongArch64: Rename targets

The existing targets naming convention for LoongArch64 is not appropriate(lack of microarchitecture-specific thoughts).
Now I use LA464, LA264 to replace the original targets.

@yinhiyou @martin-frbg please help me review those patches.


show button: send beacon

First time users either

don't know that they need to send the beacon (and other requirements (be in team Gridcoin, ...) to get rewards for their BOINC work) (A) or
don't like the terminal-way after knowing what to do.

In any case, it would hasten their reward process (considering superblocks are taking long time) when they see a message that shows them what to do (or tells 'em: "click the beacon button")
related:

startup wizard: configure before launching client

(A) at least the linux users, I remember someone saying in Windows client there may be a hint somewhere

#562

This is partially addressed by #1646. When @cyrossignol puts up the wizard, which is coming with the mandatory changes for Fern, this will be closed out.


Added empty stat div

And I also added any image files from .gitignore while testing, so that you don't have to remove them and commit it; take a look for yourself
.gitignore
# Ignore IDE folders
.vscode
.idea

# and some other stuff as well
venv
yolov4-pytorch.pth
+ static/*.png
+ static/*.jpg
+ static/*.jpeg

I can't believe I've put some time into this, maybe that's it.


Update README instructions and Docker image

This PR updates the site's README to push people a little more in the direction of using the Docker image versus their local Hugo. It also updates the Docker image itself to not use the "extended" Hugo (for the sake of consistency, as we're not yet using Hugo extended on the Netlify side...the Docker image hasn't presented issues yet so let's not allow it to 😄).

/lgtm
/approve


⚠ Secret Website 3 has degraded performance

In 7e67946, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2830 ms

Resolved: Secret Website 3 performance has improved in 738c77f.


Status of the registers if HBlank DMA is started during HBlank

HBlank DMA should not be started (write to FF55) during a HBlank
period (STAT mode 0).
If the transfer's destination address overflows, the transfer stops
prematurely.
The status of the registers if this happens still needs to be investigated.

Er, context, please?

Er, context, please?

First warning box here: https://gbdev.io/pandocs/CGB_Registers.html#lcd-vram-dma-transfers
Also, see https://github.com/gbdev/pandocs/pull/365. I've added a link to mention this issue so it doesn't remain a "todo" in the text as it's now


Support for ng-bootstrap dropdown

The team at ng-bootstrap recommends not to use bootstrap.js, jquery and popper. AFAIK, this library intends to completely replace bootstrap javascript with angular typescript. Therefore, the dropdown is different from bootstrap (https://ng-bootstrap.github.io/#/components/dropdown/examples).
Is it possible to support ng-bootstrap? I have can't create a dropdown-multiselect which drops up (that's basically the only thing that doesn't work for me so I don't think there should be much of differences)

@softsimon Any chance you can have a look at this?


Add option to go back from admin to main screen

closing as merged with https://github.com/roots-id/cardano-watcher-web/pull/8


Fix deadlock on shutdown in kubernetes backend

Please describe the change you are making
Fix a bug that when shutting down with active connections running containerssh deadlocks until it's force-killed.
Are you the owner of the code you are sending in, or do you have permission of the owner?
Sent with permission of the owner
The code will be published under the MIT-0 license. Have you read and understood this license?
Yes

The test failures don't appear to be because of those changes master fails the same way

@tsipinakis thank you for your contribution. Please feel free to add yourself to the public contributors list here: https://github.com/ContainerSSH/containerssh.github.io/blob/main/contributors.yaml


Fsharp functions

sin x, cos x.... etc
derivative x expr...
conj a b...
tests...
🦆

Codecov Report

Merging #304 (85d7de2) into master (8d6f6e0) will increase coverage by 0.52%.
The diff coverage is 95.42%.

@@            Coverage Diff             @@
##           master     #304      +/-   ##
==========================================
+ Coverage   76.01%   76.54%   +0.52%
==========================================
  Files         107      107
  Lines        8806     9265     +459
  Branches     1485     1485
==========================================
+ Hits         6694     7092     +398
- Misses       1625     1686      +61
  Partials      487      487

Impacted Files
Coverage Δ

AngouriMath/Core/Antlr/AngouriMathParser.cs
77.80% <87.34%> (-0.77%)
:arrow_down:

AngouriMath/Core/Antlr/AngouriMathLexer.cs
99.19% <100.00%> (+0.24%)
:arrow_up:

AngouriMath/Convenience/MathS.cs
70.61% <0.00%> (+5.67%)
:arrow_up:

Continue to review full report at Codecov.

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


Why isn't this array access out of bound?

I'm trying to understand some part of the solver and am stuck here.
See
https://github.com/tromp/equihash/blob/39a91772abc503ace4b5ae5e1af1fde482eb077b/equi_miner.h#L908
digit4() and digit7() both access slot1->bytes[4], but the union slot1, which is a htunit, holds at bytes a   uchar[sizeof(tree_t)] presentation. tree_t is either u16 or u32, so sizeof(tree_t) is either 2 or 4. How can the access to bytes[4] not be out of the array's bound?

Slots are in fact larger than a single htunit:
typedef htunit slot0[HASHWORDS0+1];
typedef htunit slot1[HASHWORDS1+1];
So the bytes[] in one htunit can extend into subsequent htunits.

Doh, didn't see it was a htunit[] array, sorry!


[19.03] firefox 65.0.2 -> 66, 60.5.2esr -> 60.6.0esr, firefox-bin: 65.0.2 -> 66.0

Motivation for this change
Backport of #56689
cc @mweinelt
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

built, and run. Thanks for backporting this.


Bump version of pkgdown

Summary
Build R documents with new version of pkgdown.
Motivation
1.5 version introduces repo$url config option. This option will allow to drop our ugly postprocessing JS workaround for incorrect source files links as our R package is located in a subdirectory:
https://github.com/microsoft/LightGBM/blob/master/R-package/pkgdown/extra.js
References
https://github.com/microsoft/LightGBM/blob/2c18a0f3ed732df77d4304ed1d2b46a345d661c8/docs/conf.py#L238

You can now control the links to source files (in reference pages and
articles) and issues and users (in the NEWS) with new repo$url config
option (https://github.com/r-lib/pkgdown/issues/1238). This makes it easier to use pkgdown with GitHub enterprise, packages in subdirectories, and other source hosts (like bitbucket).

https://github.com/r-lib/pkgdown/releases/tag/v1.5.0

Closed in favor of being in #2302. We decided to keep all feature requests in one place.
Welcome to contribute this feature! Please re-open this issue (or post a comment if you are not a topic starter) if you are actively working on implementing this feature.

re-opening this since I think it will be fixed by open PR #3072
Leaving some notes for myself. You can see the details of the relevant pkgdown changes in R/build.r section of https://github.com/r-lib/pkgdown/pull/1253/files, namely:
#' @section YAML config - repo:
#' pkgdown automatically generates links to the source repository in a few
#' places
#'
#' * Articles and documentation topics are linked back to the
#'   underlying source file.
#'
#' * The NEWS automatically links issue numbers and user names.
#'
#' * The homepage provides a link to  "Browse source code"
#'
#' pkgdown automatically figures out the necessary URLs if you link to a GitHub
#' or GitLab repo in your `BugReports` or `URL` field. Otherwise, you can
#' supply your own in the `repo` component:
#'
#' ```yaml
#' repo:
#'   url:
#'     home: https://github.com/r-lib/pkgdown/
#'     source: https://github.com/r-lib/pkgdown/blob/master/
#'     issue: https://github.com/r-lib/pkgdown/issues/
#'     user: https://github.com/
#' ```
#'
#' * `home`: path to package home on source code repository.
#' * `source:`: path to source of individual file in master branch.
#' * `issue`: path to individual issue.
#' * `user`: path to user.
#'
#' The varying components (e.g. path, issue number, user name) are pasted on
#' the end of these URLs so they should have trailing `/`s.

So to check that a fix  to this worked, after removing  R-package/pkgdown/extra.js, check that:

GitHub button in the top right corner of the site should point to https://github.com/microsoft/LightGBM/tree/master/R-package
Source: links  on each function's page should point to the right file in GitHub, like this:


removed setValue from switcher for oneOf, setValue was messing things up...

removed setValue from switcher for oneOf, setValue was messing things up by adding fields from the current editor to the one being switched to. This may need a better solution later, maybe should be mapping fields if matches can be found, but throwing out ones that don't match

I should have updated; setting '          no_additional_properties: true, ' is sufficient to remove the additional properties and to make sure that default numbers are set.
Sorry to delay.  I was working on a patch that would allow the title of the schema used to validate against to be set in the  function and never got it to work well.

No worries, I ran into this issue as well, so I needed to fix it anyway. But it sounds like setting no_additional_properties: true is a better option, so thanks for that. Although I kind of think that should be the default behavior, but oh well.

@jdorn could you please catch up and merge all those good pull requests? Your main branch is running bahind

I can see usecases for both behaviors - how it is now and how the pull request does it.  Maybe this should be an option - global and per-editor.  So something like this can be done:
{
  "type": "object",
  "oneOf": [],
  "options": {
    "keep_values": true
  }
}

Yes. I thought that I didn't need this until I realized that I wanted the new "default" value to overwrite the old one in some circumstances. This solution is important.

I just added a new option keep_oneof_values to toggle this behavior. It defaults to true to keep the current behavior and not break backwards compatibility.
Set globally:
JSONEditor.defaults.options.keep_oneof_values = false;

Set per JSONEditor instance:
var editor = new JSONEditor(el, {
  schema: {...},
  keep_oneof_values: false
});

Set for a single oneOf instance:
{
  "options": {
    "keep_oneof_values": false
  },
  "oneOf": []
}


Update scoring code to ignore test targets

Given the update in the rules to exclude test targets from scoring we should update scoring.py.
Specifically get_index_that_reaches_target should ignore the test target when it gets the index.

To be more specific, these lines need to be updated: https://github.com/mlcommons/algorithmic-efficiency/blob/main/scoring/scoring.py#L130-L139


[C++] OTF API refuses to parse new message versions

Trying to parse the upcoming (version 6) CME MDP 3.0 release using the OTF API and old .sbeir data says:
Error no IR found for message with templateId=27 and version=6 [E117] at offset 8

This is useless.  It should possibly warn, but it should parse the backwards-compatible part of the message.  After all, that's the whole point of backwards compatibility.

It's worse.  With new IR, the OTF API refuses to parse old versions.
How exactly is this code intended to be used?

Looking further, IrCollection::message takes a version parameter.  I think it shouldn't take a version parameter.
I don't know without further inspection whether the message parser can handle an IR template that has a different version (and thus a different acting block length) from the message being parsed.

Trying to generate an SBE IR file that contains two versions seems impossible with the tools provided.  For example, if I pass both XML templates into the compiler, I end up with an IR file that is only compatible with whichever template I pass second on the command line.

The CME controls these schemas. We do not. We also do not currently test against them. That may or may not change in the future.
RC3 is in the works and not yet complete. In addition, as the OTF has been redone to be in line with the Java OTF, the way this works has been redone. Also, as mentioned a while ago IrCollection has gone away. So, management of IR from the OTF perspective is now totally up to the user of the OTF decoder.
With all of the above, I will set this to a question until such time as we close as invalid or make additional changes.

I haven't looked at the new OTF stuff, and I won't be able to until next
year.  Will the new code be able to decode a message generated by a version
of schema that differs from the version of the schema that was used to
generate the .sbeir file?  If so, then this should be okay.  If not, then
it's still a bug.

If it has followed the extension rules for the spec and the app is using the IrDecoder and IR correctly with the OTF decoder, then it should, yes. But that is a lot of ifs and restrictions.

As the C++ and Java OTF decoders are the same now, changing the title.

Closing for now as it has been rewritten. Please reopen a new issue with a specific example if it does not work.


Event equivalent of callback

Given this code:
request('http://google.com', function( err, res ){
  if (err)
    return console.error(err);

  console.log(res);
});

What is the equivalent events, to get the same result? Say I wanted to pipe a file:
var r = request('http://google.com')
  .on('error', console.error)
  .on('response', console.log);

fs.createReadStream('some-file').pipe(r);

It doesn't seem like response is actually the same in the last example, and from the documentation I wouldn't guess either.

You can still have a callback when uploading a file:
var r = request('http://google.com', function( err, res ){
  if (err)
    return console.error(err);
  console.log(res);
})
fs.createReadStream('some-file').pipe(r)

If you want to stream the response then you should listen for the finish event of the write stream:
var f = fs.createWriteStream()
f.on('finish', function () {})
r.pipe(f)

Still seems like something is different.. For instance, the accept: application/json header does not get appended with json: true.


Retrieve rstudio-server pkg direct from rstudio.com

There are no public repos that contain the rstudio-server package. This PR duplicates the pattern used for the shiny server to construct a download URL and pull the pkg file from rstudio.com.

+1 for getting this merged in


Added pest architecture test

As discussed in https://github.com/tempestphp/tempest-framework/issues/116, I've added an example of how architecture tests could be implemented with Pest.
I've purposely left it failing to showcase that it's working. Lowercase, Uppercase and DoesNotEndWith rules are not final and readonly like others are.

I think I worked on those rules :D, that's why I would like to have architecture testing, sometimes I just forget about this things.
But I think the benefits of adding this kind of tests is clear for everyone, the questions if what tool to use, should we go with Pest, which changes a lot in terms o syntax and style or just add phpunit plugins.
Personally I like Pest syntax, nowadays I'm working more in javascript than php, and like the it('some test case') syntax.
But more important than syntax would be to pick a tool which cand be used to enforce most or ideally all the architectural conventions.
@Plytas thanks for looking in to this.

The point of this PR is to show how architecture tests would look like with Pest. It also demonstrates that it doesn't affect other tests that could still be written with phpunit if the team prefers that.
I believe that @brendt will work on an alternative using phpunit before they make a decision.
I briefly looked into those alternatives and while they do achieve the same thing, I believe that Pest's ease of use and developer experience is unmatched.

@Plytas - Do you mind running the drift plugin on this PR? I think if we were to switch to Pest we'd want to make the tests consistent, regardless of Pest's compatibility with PHPunit.

@aidan-casey drift plugin kinda fails in this repo. Mainly because tests include helper classes in the same file. They would have to be extracted manually.

@Plytas we should do that anyway since we are PSR-4 autoloading tests. I'll create a separate issue for that and we can rebase this when that's done.

@aidan-casey I've opened a separate PR https://github.com/tempestphp/tempest-framework/pull/144 to showcase Pest syntax.

FYI, I wanna look into PHPAT as an alternative. I do have some other things on my todo list though, so please allow me a couple of days.

Closing in favor of #151 as discussed on #116


Feature Request

I use BLE with my trackers (several). Instead of the tracker identifying with a number, it would be useful if identified with the callsign+SSID.

as you can change the callsign during use, but bluetooth is only initialized at boot up once , this was the reason for using this numbers (which are MAC address numbers)

added into new release with webflasher config ;)


Handle 503 when not from server overload

Currently cannot download all files for this url: https://www.amazon.com/Art-Gathering-How-Meet-Matters/dp/1594634920
The problem is that some of the pages return a 503 (I assume because amazon returned that robot check) so the bot just gets stuck with them.
It would be good to add a flag about how to handle 503 or something of the sort. Either skip 503s, download  anyway, or only retry X times. Something like that could help with this.
Here's an example of the page that will always be 503: https://web.archive.org/web/20190506092829/https://www.amazon.com/Art-Gathering-How-Meet-Matters/dp/1594634920
Thanks for making this awesome project :)

Thank you for raising this issue! Fix (using a maximum number of retries — default == 3 but configurable) is pushed and now available in v0.3.6.


unison-ucm: 1.0.M1d-alpha -> 1.0.M1e-alpha

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

cc maintainer @virusdave


fix: Select the right section when categories and tags have the same name in Viz Gallery

SUMMARY
This PR fixes an issue that was causing the Viz Gallery to select the category-related viz types when a tag with the same category name was selected.
Fixes: #16012
BEFORE
https://user-images.githubusercontent.com/60598000/127860476-b066f95e-f617-48f4-9aba-ef4ae9a6f349.mp4
AFTER
https://user-images.githubusercontent.com/60598000/127860005-c3aed9a0-6c07-4603-b5fc-671d556dd2f4.mp4
TESTING INSTRUCTIONS

Open the Viz Gallery, look for Treemap and see 'Distribution' in the Treemap tags
Click #Distribution on the left selection menu and look for Treemap
Click 'Distribution' under category on the left selection menu and look for Treemap

ADDITIONAL INFORMATION

[x] Has associated issue: #16012
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@rosemarie-chiu @jinghua-qa

thanks for the fix! when things slow down, I will go through all the Tags, Descriptions and Categories again to make sure they make sense, and are optimized for search. sorry for the trouble 🙏

/testenv up


Ruby Upgrade v2.4.1 (#826, #1420)

cleaning unused gems and upgrade to Ruby v2.4.1
fix node modules issues when running the upgrade
update travis.yml to Ruby v2.4.1
Fixes #826 and #1420 .
Changes proposed in this PR:

@bhavi appears to work fine on dmponline-dev. Is there any reason to use 2.4.1 and not 2.4.4 which is the latest stable 2.4 ruby?


Simplify creation of dates.txt

Attached some script to build custom paintings by noting down the days per week that should be marked.
Regenerate dates.txt by running ./dates.sh 2019-07-14 < dow.txt > dates.txt.
You could even do something like:
./dates.sh 'today -42 weeks' < dow.txt > dates.txt
Which would roughly align so the latest commit is at the end of this week.
I'd need to check if a LANG=C LC_ALL=C is necessary for date arithmetic to work properly. In case of doubt it doesn't hurt.

This is good. 'today -47 weeks' centers it nicely, and I suppose '-42 weeks' lets one set it and forget it for a couple months as it scrolls left.
Is there any reason not to just integrate dates.sh into the paint script?  Otherwise, we'll need to regenerate the dates.txt periodically so it works out of the box.
Something is making mine fade: mine looked crisp after I pushed, and then when I check back a few minutes later, it finds an extra 100 or 180 commits to that repo in the "E", and the earlier letters turn a lighter green.  I think it's unrelated to this PR, but I didn't notice until I ran from this branch.  I'm curious if this is happening to anyone else.

No particular reason, just didn't want to mix those scripts together. If you like I can also build a mixed-together version; shouldn't be too hard.

since there is a 4 color palette available, I think keeping the last column in dates.txt is a good idea.
I'm working on rasterizing a given font in an .xcf with a 4 color indexed-palette, with the goal of producing 'anti-aliased' text.
I definitely agree that there should be a time-agnostic intermediate representation, and then a just-in-time conversion to a dates file.

It's actually even five colors. That means: Even more fun … ;-)
AHHH, that explains the last column of the original dates.txt. I'll add it in and redo that commit so that the generated dates.txt includes the pixel color column again.

Seems color levels are: ceil(max/4*level), for me with 5 contributions max this gives:
0 -> 0
1 -> 1
2 -> 2,3
3 -> 4
4 -> 5

Please confirm. Based on that calculations for needed number of commits can be made.

@BenBE I wasn't counting the 0/grey color since those dates don't appear in the list at all (assuming maxCommits is set appropriately).
https://github.com/LindseyB/github-contribution-graph-art-generator asserts a minimum value for maxCommits at 4, and then has the following logic:
ar commitUnit = Math.ceil(maxCommits / 4);

  var color_map = [
    {color: "#c6e48b", commits: commitUnit},
    {color: "#7bc96f", commits: commitUnit * 2},
    {color: "#239a3b", commits: commitUnit * 3},
    {color: "#196127", commits: commitUnit * 4}
  ]

However, to force unlisted dates (which have at most maxCommits, preexisting) to be 0, the number of commits for a color 1-4 could be set to color*(maxCommits+1). this also requires that 4 appears somewhere in the image or the colors will be wrong.
This also has the property that successive runs (once we use your fix for #2 ) will render properly, as the max commits will keep scaling up. Obviously this is not ideal, and either deleting and recreating the branch or force pushing the parent of the graphiti commits is a 'more correct' approach.

In the last commit I added while reworking the comments from earlier) I made the paint.sh script use the last column directly use the base unit for scaling. While you could randomly select any commit count between the lower and upper bounds for any color to cause that one, it is basically to noisy and unreliable to ensure the picture is as crisp as can be. With slight modifications to the command I posted in #2 you could even compensate for pre-existing commits per day, but that's nothing you want to implement in bash, except if you're looking for pain …

@BenBE @rossabaker hey yall totally bash-clueless here on osx 10.14.6
when i try to run ./dates.sh (with any arguments), including the examples in the PR description, i get:

i tried LANG=C via https://blog.frd.mn/puzzling-with-os-xs-date/  and nothing seems to be working, any help? didn't wanna open an issue because i suspect it's just me being clueless
thanks!!

if you add set -x as the second line of the script, then it will print
the commands it is running. then we could see exactly what is failing.
it's possible that instead of LANG=C we need a locale that uses the ISO
date format being used?
On Mon, Aug 17, 2020, 01:38 lindsaylevine<EMAIL_ADDRESS>wrote:

@BenBE https://github.com/BenBE @rossabaker
https://github.com/rossabaker hey yall totally bash-clueless here on
osx 10.14.6
when i try to run ./dates.sh (with any arguments), including the examples
in the PR description, i get:
[image: Screen Shot 2020-08-17 at 4 36 22 AM]
https://user-images.githubusercontent.com/57350178/90375636-3aabf500-e043-11ea-8369-a71ce726fd9a.png
i tried LANG=C via https://blog.frd.mn/puzzling-with-os-xs-date/ and
nothing seems to be working, any help? didn't wanna open an issue because i
suspect it's just me being clueless
thanks!!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rossabaker/abolish-ice-graphiti/pull/8#issuecomment-674744474,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAHXYXZ5FRN35IFE67ZMFTSBDT75ANCNFSM4N5UVKJQ
.

set -x was always in there 🤔 (is in dates.sh in master)

that's set -e which makes it stop in the first error set -x will echo
commands as they run
On Mon, Aug 17, 2020, 09:13 lindsaylevine<EMAIL_ADDRESS>wrote:

set -x was always in there 🤔 (is in dates.sh in master)
[image: Screen Shot 2020-08-17 at 12 12 34 PM]
https://user-images.githubusercontent.com/57350178/90418316-fa6c6700-e082-11ea-9972-16bf06bca57e.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rossabaker/abolish-ice-graphiti/pull/8#issuecomment-674973060,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAHXYTM6GTLDXQ6B45CGSLSBFJLTANCNFSM4N5UVKJQ
.

oof my bad!! here ya go

Thanks, it looks like the OSX date command has different syntax.
in the first case I think date --date 2020-08-17 +%u  should become date 2020-08-17 +%u
in the second case date --date '2020-08-17 - 4 days' +%Y-%m-%d should become date -v-4d 2020-08-17+%Y-%m-%d ?
@lindsaylevine can you run the follwoing three commands and show us the output?
date --date 2020-08-17; echo $?
date 2020-08-17 +%u
date -v-4d 2020-08-17+%Y-%m-%d

@lindsaylevine wait, I think -v might try to change your system time :roll_eyes: make the last command date -j -v-4d 2020-08-17+%Y-%m-%d instead, sorry.

https://www.shell-tips.com/linux/how-to-format-date-and-time-in-linux-macos-and-bash/#format-a-date-on-unix-macos-using-the-bsd-date-command-line

sorry for delay!

@lindsaylevine the first command 'worked', its exit code, in $?, indicates an error; so we will be able to detect the BSD version of the date command and make adjustments. for the rest it looks like we need to specify an input format as well as the output format.
can you try:
date -f %u 2020-08-17 +%u
date -j -v-4d -f %u  2020-08-17 +%Y-%m-%d

!!!!!!!!
assuming this is now what we want pls lmk how to update dates.sh !
(thanks so much btw)

this is good news! (tho i should have used -f %Y-%m-%d for the first command, same as the second)
I would like to write a small wrapper that can be source'd by both this script and my raster.sh to provide platform-agnostic date functions.  I'll open a separate issue for that i think.
it should be quick work, but if you'd like i can show you tomorrow what needs to be changed locally to get you unstuck.

saw the issue, thanks so much :)


Feature/put error handler in context

Checklist

[x] the [contributor license agreement][] is signed
[x] commit message follows [commit guidelines][]
[ ] tests are included
[x] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
Here is a big PR which is all about error handler.  There are 36 files involved but most changes are minor.
You will see two context:  AppRenderErrorContext and SearchPageRenderErrorContext.  The first one provides an error handler which targets the whole App, which means all components (e.g. SearchPage, LegacyContent and even IndexPage ) can access it. The second one is specific to the New Search UI and its child component.
Firstly, I refactored the New Search UI and put all changes in one commit as these changes are minor.
In the next step when I started refactoring the whole App, I realised there would be massive changes so I decided to make one commit for one component. During implementation, I also refactored some components as they seemed hard to read even though they were originally written by myself, such as SearchFilterSettingsPage and 'SettingsPage'. In addition, you will see two approaches to use context because we still have some class components.
In order to get tests pass without extra changes, I also refactored a couple variables (e.g. oeqTheme) to functions
Lastly, I dropped useError. And I have not used error boundary yet. It seems easy to implement but let's discuss a bit more in person.
A side refactoring for AppConfig is also included in this PR. I replaced all direct use of AppConfig.baseUrl with calling function getBaseUrl and made AppConfig private.
Cloud provide page:

Contribution Wizard:

Settings page:

Search filter settings page

Search page

Errors handled in Selection Session.


Support .ssh/config for specifying keys if ssh-agent fails

Update (2018-10-30)
There's a workaround below for those interested:
[net]
git-fetch-with-cli = true

Original description
When cloning an SSH repository the only currently supported method of authenticating is picking up a  key through ssh-agent. This can fail, however, for example if it's just not running! Cargo should support parsing .ssh/config and/or otherwise having a reasonable fallback in trying to find public/private keys on the filesystem. Currently libssh2 does not support this, so an external library will be required.
As to the rationale for this issue, apparently when using CircleCI with a deploy key it will add this to ~/.gitconfig:
[url "git@github.com:"]
  insteadOf = https://github.com/

Which means that clones of the index will be rewritten to<EMAIL_ADDRESS>(SSH) instead of HTTPS. The ssh-agent apparently also isn't running, so it relies on ~/.ssh/config to point SSH at the right keys, which Cargo isn't itself looking at.

Is there a workaround for this for CircleCI? Running into this:
$ cargo build --release --verbose
    Updating registry `https://github.com/rust-lang/crates.io-index`
error: failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  [23/-1] username does not match previous request

@travisofthenorth that specific bug should be fixed on nightly I believe, although it may still not work on nightly when all put together

Thanks @alexcrichton. The nightly build does fix that issue. cargo build --verbose still seems to hang. I suppose that's what you were alluding to?

Hm, that's odd that it hangs! It should either always make progress in one way or another or give an error message. Maybe there's just a slow download or slow network performance?

Network performance seems average on CircleCI. Downloading and installing rust takes ~15s. How long would you expect a build to take, and what output? This is all I saw after a few minutes:
$ cargo build --verbose
    Updating registry `https://github.com/rust-lang/crates.io-index`

Hmmm, am I doing something wrong here? I'm so puzzled, I'm getting no additional output:
$ RUST_BACKTRACE=1 cargo build --release --verbose
    Updating registry `https://github.com/rust-lang/crates.io-index`

Oh I meant moreso attaching a debugger while it's running (RUST_BACKTRACE is only used on panics).
Another option may be to use RUST_LOG=cargo and see if that prints something useful

Are there docs on doing this? I'm pretty new to Rust (I'm actually working in a Ruby app and a Rust lib is getting compiled in a gem).
FWIW:
$ RUST_LOG=cargo cargo build --release --verbose
DEBUG:cargo::build: executing; cmd=cargo-build; args=["cargo", "build", "--release", "--verbose"]
DEBUG:cargo::ops::cargo_compile: compile; manifest-path=/home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7/Cargo.toml
TRACE:cargo::ops::cargo_read_manifest: read_package; path=/home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7/Cargo.toml; source-id=file:///home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7
DEBUG:cargo::ops::cargo_compile: loaded package; package=faster_path v0.0.1 (file:///home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7)
DEBUG:cargo::core::registry: load/missing  file:///home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7
TRACE:cargo::core::source: loading SourceId; file:///home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7
TRACE:cargo::ops::cargo_read_manifest: read_package; path=/home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7/Cargo.toml; source-id=file:///home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7
TRACE:cargo::core::resolver: resolve; summary=faster_path v0.0.1 (file:///home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7)
TRACE:cargo::core::resolver: activating faster_path v0.0.1 (file:///home/ubuntu/myapp/vendor/bundle/ruby/2.3.0/gems/faster_path-0.1.7)
DEBUG:cargo::core::registry: load/missing  registry https://github.com/rust-lang/crates.io-index
TRACE:cargo::core::source: loading SourceId; registry https://github.com/rust-lang/crates.io-index
    Updating registry `https://github.com/rust-lang/crates.io-index`

Probably not, but that looks like it's just a slow clone. So far it doesn't look like it's necessarily a Cargo problem so much as ensuring that a filesystem/network connection is looking good. If you can ssh into the builder (I think circleci allows that?) and attach a debugger to the Cargo process you can find out for sure, but it looks like libgit2's just chugging along cloning the index.

attach a debugger to the Cargo process

^ that's what I meant when I was asking "are there docs on doing this". These snippets are actually from my ssh session on a Circle box trying to install manually. I let it sit for 30m-1hr and it never finished, so I'm skeptical that it's just git.

Ah yeah, to do that you should just need gdb -p <pid-of-cargo> which you can learn through something like ps aux | grep cargo. You could also just do gdb --args cargo build I believe. Once you're in gdb and it's paused when Cargo is hanging you should just need bt all I think

@alexcrichton I just ran rm .gitignore and it worked fine. So, I suppose it's a similar issue still present in the nightly?

Oh this may be libssh2 going into the weeds which is being redirected via ~/.gitconfig and replaceWith (or some key named like that...)

It seems like insteadOf is the only key defined on CircleCI, at least on the box I was using.

Seems like this is actually an issue with Circle. I worked around it with the following:
checkout:
  post:
    - git config --global --unset url.ssh://git@github.com:.insteadof

But the configuration is actually broken. The value assigned to this variable isn't correct which makes anything that gets transformed by it into a broken url, eg. ssh://git@github.com:/rust-lang/cargo

Is there a reason this is stalled? The current behaviour (ssh-agent being required) is very frustrating.

Agreed with @wrl. I'm having the same exact issue too. I don't use ssh-agent.

It turns out to be pretty nontrivial to emulate ssh's behavior. We'd need to write a parser for ~/.ssh/config for example.
I think a more promising route is to support fetches by shelling out to the git binary if it's available. We'd use git2 for everything else and still support git2-based fetches if git isn't present, but it seems better to just use git rather than trying to emulate its exact authentication behavior.

I think a more promising route is to support fetches by shelling out to the git binary if it's available.

Oh, yes, please do so! :+1:
I am trying to get Cargo to work with a private git repository in a Single-Sign-On environment, but got lost in the limitations of libgit2. Here {ssh,https}://mygitserver.domain.tld/myproject works locally without authentication on the git layer, but happens on the transport layer - configured in OpenSSH configuration or HTTP libraries like libcurl understanding how to perform (negotiate) authentication. (In my case I use Kerberos authentication.) This means that users have a single canonical URL and giving that to git clone "just works" from all machines. I'd like to have Cargo use this system configuration as well and by shelling out to local git binary this should "just work".

As an update for those following this issue, Cargo now supports global .cargo/config configuration that looks like:
[net]
git-fetch-with-cli = true

which will instruct Cargo to fetch git repositories with the git CLI rather than with libgit2. Using this will read .ssh/config because that's what git does.

@alexcrichton I fetch failed by the following error when using the git client
fatal: Refusing to fetch into current branch refs/heads/master of non-bare repository
error: failed to load source for a dependency on `cargo`

Caused by:
  Unable to update https://github.com/rust-lang/cargo

Caused by:
  failed to clone into: /Users/wangsijie/.cargo/git/db/cargo-e7ff1db891893a9e

Caused by:
  process didn't exit successfully: `git fetch --tags --quiet 'https://github.com/rust-lang/cargo' 'refs/heads/*:refs/heads/*'` (exit code: 128)

I just create a new crate dependent on cargo
[dependencies]
cargo = { git = "https://github.com/rust-lang/cargo" }

My rust version is
rustc 1.31.0-nightly (1cf82fd9c 2018-10-30)

Is there something wrong with my git config?

@xinghun92 hm there's probably not much wrong with your git config! The error message there:
fatal: Refusing to fetch into current branch refs/heads/master of non-bare repository

is one I'm not so sure of! I'm pretty unfamiliar with the advanced aspects of the git CLI, but it looks like Cargo isn't executing quite the right git command here. According to this question it sounds like we need to pass the --update-head-ok option, I'll send a PR for that!

@alexcrichton thank you! That works.

I tried the workaround by adding a ~/.cargo/config file with the code given above, but now I'm getting an authentication error, specifically:
error: failed to load source for a dependency on `csv`

Caused by:
  Unable to update registry `https://github.com/rust-lang/crates.io-index`

Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  process didn't exit successfully: `git fetch --tags --force --update-head-ok 'https://github.com/rust-lang/crates.io-index' 'refs/heads/master:refs/remotes/origin/master'` (exit code: 128)
--- stderr
fatal: unable to access 'https://github.com/rust-lang/crates.io-index/': SSL certificate problem: Invalid certificate chain

I'm on a MacBook running El Capitan and I just updated everything with rustup update. I can see that --update-head-ok is being used in the git call. How do I make sure I have a valid certificate chain?

@alexcrichton just to add to your rationale, that .gitconfig setup
[url "git@github.com:"]
  insteadOf = https://github.com/

is frequent among Go programmers that have to work with private repositories. The Go dependency system tries to clone repos using HTTPS by default, but if you use SSH to authenticate with your private repos you must use that trick on .gitconfig. So it's not only for the CircleCI case. I guess there must be other cases too.

is frequent among Go programmers that have to work with private repositories

It's unfortunate that such things require global configuration changes rather than being localized to the affected repositories.

As an update for those following this issue, Cargo now supports global .cargo/config configuration that looks like:
[net]
git-fetch-with-cli = true

Is there any way to do this on the command line?
cargo install --git-fetch-with-cli --git ssh://...

You should be able to set it via an environment variable:
$ env CARGO_NET_GIT_FETCH_WITH_CLI=true cargo install --git ssh://...

@sfackler Works like a charm, thanks!

To followup on this issue, when I updated my .cargo/config file, I started getting the following error:
Updating crates.io index
error: failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  could not execute process `git fetch --tags --force --update-head-ok 'https://github.com/rust-lang/crates.io-index' 'refs/heads/master:refs/remotes/origin/master'` (never executed)

Caused by:
  No such file or directory (os error 2)

Note that I am trying to run this inside of a docker container. Anyone have ideas on what the issue maybe here?

You need to install git in the docker container.

I'm on macOS 10.15.2 (beta), with cargo 0.26.0 (41480f5cc 2018-02-26) and the workaround doesn't work for me.  I've tried both a ~/.cargo/config, and the env var (CARGO_NET_GIT_FETCH_WITH_CLI=true) approach.  Am I missing a step?

The explanation on this ticket doesn't seem quite right: I do use ssh-agent (and my .ssh/config file is completely empty) and yet I still hit this error with cargo.
The workaround using git CLI does fix the problem.

@Diggsey Are you on Windows and using the built in OpenSSH ssh-agent? If so, libssh2 doesn't support that ssh-agent, though there is work being done to remedy that.

@yodaldevoid yes

I am experiencing this just trying to go through the getting started guide and run cargo build after adding the ferris-says dep: https://www.rust-lang.org/learn/get-started
Why does it even need an SSH key at all to download this?

Why does it even need an SSH key at all to download this?

This should only be an issue if specifying dependencies via git with an ssh URL so something seems wrong in your case anyway.

Please also support custom SSH agents. I use my password manager to handle my SSH keys and as such my SSH config looks like this:
Host *
	IdentityAgent "~/Library/Group Containers/2BUA8C4S2C.com.1password/t/agent.sock"

Host github.com
    HostName github.com
    User git
    IdentityFile ~/.ssh/id_github.pub
    IdentitiesOnly yes

What's more, I use https to ssh rewriting to support private repos in go projects, and my global git config is:
[url "git@github.com:"]
	insteadOf = https://github.com/

This setup absolutely doesn't work with cargo.

export CARGO_NET_GIT_FETCH_WITH_CLI=true

Using ssh access without the ssh-agent running seems like a fairly unusual setup to be using intentionally; I ran into this issue because I just forgot to start up the agent on OS X. Perhaps the hint message pointing users to net_git_fetch_with_cli could also suggest starting up ssh-agent as an alternative?


Start position is out off full hd screen

My screen is not Full HD and I cannot see the mini video with my webcam.

Fixed!


feat: docker images for iroh

Description

Breaking Changes

Notes & open questions

Change checklist

[x] Self-review.
[x] Documentation updates if relevant.
[ ] Tests if relevant.
[ ] All breaking changes documented.

@Arqu once we merge this, can we automate it, to publish this to dockerhub on release?

@Arqu we should have one minimal test in CI running the docker image and calling iroh node status or sth, to make sure the basic interactions always work

@Arqu we should have one minimal test in CI running the docker image and calling iroh node status or sth, to make sure the basic interactions always work

We got a node stats test now too.


lint rule to prevent imports in cypress

Closes: RHOAIENG-7329
Description

Made three eslint rules:

All type imports need to have the type keyword in the import statement
Imports must not import from patternfly
Flag outside imports that are not needed.

How Has This Been Tested?

Add a restricted import to the cypress directory. It should be flagged.
Test Impact

Eslint rule. No test necessary
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

There is also a typescript error defined in registeredModelArchive.cy.ts where the overload wasn't defined in the past --- just added a fix to it in the pr as well.

works as expected
/lgtm
/approve


chore(doc): add breadcrumb to e-commerce demo

Summary
Add a breadcrumb to the e-commerce example (following #2451).
Result
Now there is a breadcrumb widget displayed on the example, making it easier to see the facet hierarchy.

Nice @marielaures Thanks for taking into account the design elements :)


YAML empty objects get sripped away

I've experienced that empty objects in the yaml gets stripped away. An example for a request body in openapi with an epmty object gets stripped away.
E.g.
example:
  default:
    key1: "value1"
    key: {}

becomes
example:
  default:
    key1: "value1"

Is this related to the parsing library used in openapi-format? Do you have any suggestions how to preserve the empty object?

So, I found the location responsible for removing empty objects: openapi-format.js#L443. When commenting out, it works as intended.
What are the reasons to remove empty objects? Can we just delete lines 440 to 444 in openapi-format.js?

In my use case there is an example for a request body which contains a key with an empty map.

That would be awesome

Any news on this issue?

hi @a-waider
There is a fix ready #73
If the empty object is in a default example, it gets skipped from being deleted.
Would that solve you case?

hi @a-waider
I improved the logic to delete empty objects, like you suggested and released a new version 1.13.1 of openapi-format.
Let me know, if it behaves as you expect.

Thanks for you effort. The new version solves this issue :)


Tab components should show visible headings for all sections on mobile screen sizes

This issue is from May 2023 external accessibility audit report.
WCAG Reference:
2.4.10 Section Headings (Level AAA)
Understanding Section Headings | How to Meet Section Headings
Issue ID:
DAC_Section_Headings_01
URLs:
https://govuk-frontend-pr-3503.herokuapp.com/full-page-examples/start-page
Screen shots

Although when contained within tab panels, when viewing on a desktop, the sections of content were separable from one another and other content on the page. The lack of section headings in the alternative view did not enable users of screen reading assistive technologies to easily understand the purpose of the skip links.
This can also be confusing for other users when viewing the page with magnifying options as the skip links indicate the sections of content they will direct the user to, however, there are no headings to relate those links to the specific sections of content when viewing in this way.
Current code ref(s)

You’ll need a debit or credit card to use this service.

It’s £9.50 cheaper to apply for a passport online than by post.

It should take around 6 weeks to get your first UK adult passport, but it can take longer.

All other application types (for example, renewing a passport or getting a child passport) should take 3 weeks. It can take longer if more information is needed or your application hasn’t been filled out correctly.

You should use a different service if you need your passport urgently.

Screen-reader comments

“With both VoiceOver and TalkBack, I found that the tabs instead of switching the content to a different view were in page links and were intended to move my focus to the relevant point of the page.

With VoiceOver when I selected the in-page links instead of moving to the indicated point on the page I was moved to the top of the page. This was disorientating as it was not initially clear where I had landed. I also became frustrated as it was time consuming to move back to the in-page links and then manually past the item to read the information. This functionality did not occur with TalkBack as focus moved to the intended point.

I then found that both sections were present on the page at once. This was confusing as no headings were available to advise me of where one section ended and the next began. It was therefore not clear whether I was viewing the ‘More info’ section or the ‘Other ways to apply’.

Ensuring that I can easily understand the difference between the sections and can move directly to them for example by placing a heading at the beginning of each section will prevent any confusion”

Low vision user comments

“While viewing this page with ‘windows high contrast BLACK’ and with ‘Chrome zoom x200%’ I found the ‘More information’ and ‘Other ways to apply’ tab links did not relate to any content on the page, and I found it difficult to see the different sections of content. Adding text to introduce the sections would help me to determine which information I am reading.”

Solution
Consider adding headings to each tab panel to provide additional heading structure for users of screen reading assistive technologies, also providing them with an additional method of navigation.
These headings should also be provided when viewing in the alternative magnified and mobile views.

This issue has been assessed to not be a high-severity accessibility concern. It only meets 1 of our 3 criteria for a high-severity accessibility concern.

There is reasonable evidence that it makes it difficult or impossible for some people to complete certain tasks. ✅
Important content (headings) is disappearing in two configurations:

The mobile layout for tabs.
The non-JavaScript layout for tabs.

It does not appear to affect the accessibility of essential services or critical infrastructure. It's an issue in guidance clarity and not in code. ❌
There is no blocker to individual service teams efficiently resolving it on their own. ❌

As such, this will not be flagged as a high-priority issue.
In the coming weeks we will investigate whether it fails WCAG level AAA criterion: 2.4.10 Section Headings.


Drag and drop in tree control

To parent or unparent an element in the tree view the logical thing would be drag and drop.

It was decided to remove this functionality in the new editor.