fix(field): field borders now are correctly colored even on Safari

Hi there! This is a small patch to fix an annoying bug that occurs with Fields on Safari, both on macOS and iOS.
As also described in issue #5668, Fields such as <md-outlined-text-field> and <md-outlined-select>, when focused, show an underlying dark line, even if the primary theme color is set to something else. This patch addresses said issue.
Here you can take a look at what is going on with the latest version of this library (v2.2.0 at this moment).
If you look carefully at the focused input in the first screenshot, you'll see that dark line just beneath the label, whereas all other parts of that input are brown-ish.
The second image shows the resulting page after having applied this patch. Here you can see how the dark line is no longer visible and all parts of the input border have the same color.

Before patch
After being patched

Even though these screenshots were captured on a iOS 18 Simulator, I've seen this issue both on real iOS devices, and previous versions of iOS Simulators. The same issue also occurs on Safari on macOS.
Moreover, every recent version of @material/web seems to be affected by this bug.
@asyncLiz could you take a look at this PR?
PS: Thanks for the awesome project!

Fixes: #5668

Sorry for the long delay on this, and thank you for putting together a PR!
I did a bit of debugging on this, and I see the issue in an iOS 17.2 simulator. However, when I try inspecting and adding "border-color: inherit" to ".outline-notch", I see that the color is fixed. When I unfocus and re-focus the text field, it's broken again until I toggle "border-color: inherit" on and off.
Without any CSS changes on our catalog, I also see that the border color is incorrect after focusing, but "resets" and fixes itself when hovering.
https://github.com/user-attachments/assets/f50afbbc-d7b3-4652-8f72-1a22c1c342fa
This is an indicator to me that there could be an underlying webkit style caching bug with pseudo elements, or an issue with how the border: inherit shorthand is applied when inheriting non-shorthand properties like border-color.
I'm hesitant to accept this PR until we know a bit more about why it's not working. We should confirm whether or not it's a webkit bug and file an issue if it is.
Do you mind investigating some with a standalone reproduction of this issue outside of our component? More than likely it's something to do with multiple levels of inheritance, pseudo elements, and/or border vs border-color.


How to remove all lines on an element

How can I remove all lines on an element? (v3)

Can you provide a better example of the problem you're facing and what you've tried so far?

for example I have used the Length tool and draw some lines on the enabled element. Now I want to remove all those lines at once.

cornerstoneTools.removeToolForElement(element, 'Length')
cornerstone.updateImage(element)

I think the toolState would still exist in that instance, but the tool that renders it would be gone.
You could also clear that tool's state for that image with removeToolState

It is
cornerstoneTools.clearToolState(element, toolName);

@all-contributers, please add @jlopes90 for answering questions

@all-contributors, please add @jlopes90 for answering questions

If you want to know more about how the CornerstoneJS tool states are stored and how to use the cornerstoneTools.clearToolState() function, you can follow the link to the StackOverflow answer you provided:
https://stackoverflow.com/a/73204850/2371987


Is it possible to disable all Rust safety?

Hi I wanted to ask you:
https://github.com/ixy-languages/ixy-languages/blob/master/Rust-vs-C-performance.md
in regards to this page, is it possible to disable all safety checks within Rust's compiler?

Hi @Raj2032, as mentioned on the page some compiler checks can be disabled (e.g., checks for integer overflows), some checks like the bounds checks on register access are part of ixy.rs and others like array bounds checks at runtime are part of Rust. The question is what checks you want to discard?
For compile options take a look here: https://doc.rust-lang.org/rustc/codegen-options/index.html

@ackxolotl Hi mate thanks for your response :)

some compiler checks can be disabled (e.g., checks for integer overflows),

The question is what checks you want to discard?

Is it possible to discard all safety checks?

I don't think there is any way to disable bounds checks on array access besides not using arrays in Rust. But why would you even want to do this? Most bounds checks do not induce any performance penalties.

I don't think there is any way to disable bounds checks on array access besides not using arrays in Rust. But why would you even want to do this? Most bounds checks do not induce any performance penalties.

If this doesn't induce any performance penalties, then that means this is done during compile time rather than run time?

Yes, some bounds checks can be performed at compile time and are eliminated. Others cannot. Take a look here: https://stackoverflow.com/questions/47542438/does-rusts-array-bounds-checking-affect-performance/47543246#47543246

Thanks mate :)


Issue

Sir, Can we delete SQL files if we are using MongoDB as database.
Any 1 is enough right?

Yes, any one is fine, if you are using MongoDB, you don't need SQL files.


UIF-302: Upgrade guava for 5.4.x and older.

We did it for 5.5.x and later: https://github.com/confluentinc/rest-utils/pull/215/files, this PR wants to fix back to 5.1.x when Guava first appears.

@confluentinc It looks like @xjin-Confluent just signed our Contributor License Agreement. :+1:
Always at your service,
clabot


Yet another CORS issue despite AWS config 😔

Have you read the Tutorials?
yep
Have you read the FAQ and checked for duplicate open issues?
yep
What version of Shaka Player are you using?
3.0.5
Please ask your question
Any tips on getting the player to work on localhost?  I tried pulling all the AWS access-control levers I could, and even attempted localhost on SSL.  Wondering if there's a setting within Shaka to help me cross the finish line?
The player appears to work as expected in prod.
I usually get 4 each of the following errors:
Access to fetch at 'https://mydomain.tv/Pro_Res_1.26.19.mpd' from origin 'http://localhost:8089' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
shaka-player.ui.js:formatted:15046 GET https://mydomain.tv/Pro_Res_1.26.19.mpd net::ERR_FAILED
Headers:

Thanks in advance for the help 🙌

Just re-reading my own question and brainstorming here -- thanks for coming along for the ride:
I notice that Access-Control-Allow-Origin is absent from the Response Headers, despite me setting them in AWS.  I wonder if the issue is with Cloudfront not sending them in the response?  Is that the smoking gun: to see Access-Control-Allow-Origin: localhost or similar in the Response Headers?

The browser will follow the directives of the response headers it receives.  So I would check into Cloudfront on that.  If you want access to be allowed from anywhere, set Access-Control-Allow-Origin to *.

Revisiting the same link without doing anything, I'm now seeing the proper Access-Control-Allow-Origin headers sporadically.  This seems to be an issue with Cloudfront cache -- maybe my invalidation didn't apply to response headers 🤷♂
Thanks @joeyparrish for the assistance.

Glad we could help!


Maya Glyphs: add prep script

Description
I added a new app in https://github.com/tidbyt/community/pull/2179
It relies on a large dict of static data, prepared from the original source. This is the script I used to generate that dict, in case the app ever needs to be updated. I've split it out just because having a python file in the app dir blocked automatic submission.
Copilot

copilot:all

Ah - today I learned. The problem wasn't having a random python script included. It was that originally I had it in a subdirectory. Now it's at the top level and all is hunky dory.

:shipit:


promote-assembly: improvements, docs, and more tests

Use doozerlib to parse assembly definitions. This allows promote job
to handle assembly inheritance.
Rename waives to promotion_permits.
Add more unit tests.

/lgtm


Recompile on request

The initial goal of this change was to automatically recompile our Elixir modules on a page reload, like Phoenix. Unfortunately, recompiling the modules is not enough, and we have to "touch" the templates that use those modules to see the changes. With this insight, I decided to change things up and remake the whole compilation mechanism. Instead of recompiling templates on file change, we'll recompile them when there's a browser request. This change doesn't affect the way we write a site in any way, and it turns out that this solution is much simpler than what we had, and we no longer have to keep track of the dependencies between files and all of that.
Back to the original plan, I copied some code from Phoenix that recompiles the modules on request and put it in a plug. I also added a registry to keep track of which input files generate which output files to recompile them when they are asked for.
There's also a new :run_type called :compile_metada used to inform the preprocessor that we don't really want to compile the file, and we only care about its metadata. Each preprocessor handles this type differently, but it does save a lot of time. When we start the dev server, we compile the metadata for every file to know which files to compile when the browser asks.
Along the way, there were other more minor issues that I found and fixed:

the base_url should be a relative path that's only relevant when the site is not deployed in the root of a domain.
the on_compile hook lives in the root module now. This makes it easier for us to change the implementation without breaking the API.
when a preprocessor module doesn't exist or its run function isn't defined, it throws a proper error.

a bit too much for me to be able to follow by just looking at the code, but on the surface, didn't see anything wrong with it, other than the two comments

@naps62 which comments?

the ones I had forgot to submit xD

I still have to write some tests for this, but what do you think @frm ?

Just these final comments/questions and should be good to go. Huge update, major props 👏

@frm updated 👍


Setting AppElement as aria-hidden="true" when modal is opened fails accessibility rules

Summary:
When the modal is opened, the site fails serious-level accessibility checks from axe's developer tooling. One could programmatically disable focusable elements on simple examples, but on a more complex site this is unfeasible.
Steps to reproduce:

Add Axe's Accessibility Checker tool as an extension for Chrome
open a react-modal and run a check against site
Accessibility check fails for this rule

Expected behavior:
React-modal is promised to be Accessible, serious-level failures would preclude this designation.
Link to example of issue:
Provided example linked from the Github documentation fails check for 'aria-hidden elements do not contain focusable elements' rule: https://codepen.io/claydiffrient/pen/ENegGJ

This rule seems like it's meant to keep someone from accidentally doing this - not doing it on purpose. react-modal traps focus, so under normal circumstances you can't focus something inside the AppElement when the modal is open (essentially just like the screen reader would experience it), which I think is what they're worried about. I'm wondering if this is a false positive since it can't detect what you can focus and what you can't.

I think @crswll is correct about the false positive. The behavior described is correct.
As long as react-modal don't let the focus escape the modal, this should be fine.

Thanks all, makes sense. I'll close this.


[Feature Request] Not all "Blocked services" show in app

Hey mate,
App is great, I've however noticed that within the Web console you can block Roblox. However within the android app, under block services Roblox isn't present. Assume others are missing too.
I'm using this to make it easier to allow/disallow access for the kids while on the move. Would be great if the app could have all the services that are available in web console show in the menu.
Thank.

Added on v1.11.0.

Appreciate it, confirmed I can see it fixed in latest version!
Thanks!


Enable packer logs during OVA build

Enabling logging for packer when performing OVA builds.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: abhay-krishna
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [abhay-krishna]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: abhay-krishna, vignesh-goutham
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [abhay-krishna,vignesh-goutham]

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


bitnami Update failed in Catalogs

Rancher Server Setup

Rancher version:v2.5.8
Installation option (Docker install/Helm Chart):EKS

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version:
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:

Describe the bug

bitnami
Update failed: fatal: not a git repository (or any parent up to mount point /var/lib) Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set). : exit status 128

**To Reproduce**

Result
Expected Result

Screenshots

Additional context

It was good before, but suddenly it's like this today

Happening for me as well on 2.5.12

Applications using bitnami cannot be upgraded and deployed

new error

Bitnami has been making several changes to their index.yaml recently.
Try changing the URL to https://raw.githubusercontent.com/bitnami/charts/index/bitnami
Then manually invoke refresh if needed.

Bitnami has been making several changes to their index.yaml recently.
Try changing the URL to https://raw.githubusercontent.com/bitnami/charts/index/bitnami
Then manually invoke refresh if needed.

@abhi1693 Thank you very much! Changing to this address solved it for me. Catalog in Rancher is now working again :-)

Is this the permanent solution?
I checked with Rancher 2.6 and it works fine with the following URL
https://charts.bitnami.com/bitnami
Is there a different method used in 2.5 than in 2.6?

I started seeing this problem last week as well with Rancher v2.4 - v2.5. I was leaving comments here: https://github.com/bitnami/charts/issues/8433

@lucathecreator Not a permanent solution at all. But something is better than nothing, right?

I have the same problem, trying to upgrade a bitnami mariadb chart. The url: https://charts.bitnami.com/bitnami wasn't working and giving me the error above. Then tried to change the url to the suggested: https://raw.githubusercontent.com/bitnami/charts/index/bitnami but I didn't get it to work. I tried the url: https://github.com/bitnami/charts and the repo did refresh, I also was able to upgrade via the legacy app page.
But then I run into the error and it doesn't install:
Wait helm template failed. Error: An error occurred while checking for chart dependencies. You may need to run helm dependency build` to fetch missing dependencies: found in Chart.yaml, but missing in charts/ directory: common : exit status 1

Not sure how to proceed. I do see in the chart.yaml the dependency is:
dependencies:
  - name: common
    repository: https://charts.bitnami.com/bitnami
    tags:
      - bitnami-common
    version: 1.x.x

So then iam back to the need to get the url: https://charts.bitnami.com/bitnami working I suppose?

I have the same problem, trying to upgrade a bitnami mariadb chart. The url: https://charts.bitnami.com/bitnami wasn't working and giving me the error:
Update failed: fatal: not a git repository (or any parent up to mount point /var/lib) Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set). : exit status 128

Then tried to change the url to the suggested: https://raw.githubusercontent.com/bitnami/charts/index/bitnami but I didn't get it to work. I tried the url: https://github.com/bitnami/charts and the repo did refresh, I also was able to upgrade via the legacy app page.
But then I run into the error and it doesn't install:
Wait helm template failed. Error: An error occurred while checking for chart dependencies. You may need to run helm dependency build` to fetch missing dependencies: found in Chart.yaml, but missing in charts/ directory: common : exit status 1

Not sure how to proceed. I do see in the chart.yaml the dependency is:
dependencies:
  - name: common
    repository: https://charts.bitnami.com/bitnami
    tags:
      - bitnami-common
    version: 1.x.x

So then iam back to the need to get the url: https://charts.bitnami.com/bitnami working I suppose?

exact same issue here. did you find a way to solve this?

For me, I kept the Helm charts I had deployed in the cluster. Then I deleted the Bitnami catalog, and re-added it, being sure to set it to Helm3 and using the URL https://raw.githubusercontent.com/bitnami/charts/index/bitnami
After that, my charts worked again and I could deploy new ones or upgrade existing ones.

Thnx i will try this. You didnt have the common dependency errors that are using the fixed  https://charts.bitnami.com/bitnami url? Or maybe you have used a chart that doesnt have this dependency?

@rjbaat All charts in bitnami uses common as a dependency.


cleverhans devs: developer tools collection that may be of use - WIP

I am working on stephenbhope/Revised-CommAI-env and as part of the process unified end user & developer tool box highlighting outstanding examples of opensource apps. Are there any wish list items from contributors here for either finding the right tool or to automate a tedious bit? I want to make the tool box a stop when developers are looking for something different to their current tools, or empty spots in their set.
I'd be happy to assist with setting up the webhooks to link this repo to any of github's testing, linting apps if it's wanted.

Thank you Stephen, this thread would be more appropriate for discussion within the developer ML at<EMAIL_ADDRESS>I am closing this for now, as this is not related to a bug/issue about cleverhans


iOS Implementation

[X] connect
[ ] connect with options
[ ] disconnect
[ ] add event listener without acknowledgement
[ ] emit event without acknowledgement
[ ] add event listener with acknowledgement
[ ] emit event with acknowledgement
[ ] remove all event listeners
[ ] remove particular event listeners

refer to #3


Autosuggestions don't seem to work with tab completions

Autosuggestions are awesome! Unfortunately, they don't seem to work with tab completions. For example, if I type out git com, it autosuggests git compact-log, one of my custom commands. However, if I hit tab, it expands out into git commit, which isn't what I'd expect.

This is default behavior.
Alternately you can use Ctrl-SPACE to complete auto suggestions.
I have following in my .zshrc file.
bindkey '^ ' autosuggest-accept


feat: Makes display of item value at cursor position in layer panel o…

…ptional

Thanks for this change --- overall it looks great, just a few comments.

Thanks!


Update the menu link from footer and header

The issue applied: https://github.com/CURIOSSorg/curioss.org/issues/39
Footer:

Header:

Thanks @direnakkoc!


1.4.1 version's graph with promQL issues?

why not use "http://nmg01-inf-dorishb00.nmg01.baidu.com:8090/graph" can not show anything?


how this be made to work to import files from current project using relative paths?

let's say I have a module in my project at: ./MyModule.ts
I would like to have it auto imported from anywhere within my project using a relative path, for example:
from ./OtherModule.ts as:
import * as MyModule from './MyModule'

and  from ./nested/OtherModule.ts as:
import * as MyModule from '../MyModule'

Currently  If I configure the plugin with:
        "namespaces": {
          "MyModule": {
            "importPath": "./MyModule"
          }
        }

the imports always use the literal './MyModule' regardless of where I import from

I think you can resolve it easier and don't wait the feature
just add alias to paths: "~/*": "src"
        "namespaces": {
          "MyModule": {
            "importPath": "~/lib/my-module"
          }
        }

I didn't check it, but maybe it will works and helps you :)


fix: Support --to option

fixes #80

Those travis failures on nightly and beta appear to be because the regex crate doesn't compile on those channels.

Pulled this commit into #87


Extend and refine JSDoc coverage

This issue tracks overflow from #263

[ ] Use class names instead of {Object} in @return statements where appropriate
[ ] Avoid non-descriptive or one-letter variable names (with the conventional exception of for loop iterators)

Three key undocumented areas (list ported from the earlier PR):

[ ] model/signals
[ ] model/rules
[ ] model/primitives/marks/manipulators

this raises

[ ] Identify (as a group) the JSDoc convention that will be used to document extensions of Marks

All of the above have been documented since this issue was opened. Thanks all!


Do you plan to support Windows?

I would like to support Windows. However, at this moment, I have no concrete plan for it.
#42


send_file: allow filename for inline content-disposition

send_file should support attachment_filename even for content-disposition inline.
Including a filename with inline files makes perfect sense e.g. when serving images users may want to "rightclick -> save as".
To avoid breaking API changes I would recommend keeping attachment_filename for it, but sending a content-disposition:inline header in case as_attachment is false. Alternatively we could deprecate attachment_filename in favor of a new client_filename, where the old kwarg would not add an inline disposition while the new one would.
This depends on #1850 since it moves send_file from Flask to werkzeug.

Since this is essentially a fresh start, we can just rename it client_filename with the behavior described. Flask's send_file will still exist as it has to get various Flask-specific data, so it can do the deprecation there.

download_name seems like a clearer description of what the parameter does.


fix: active language callout bug

Description
Root cause: There is a focuszone inside the component TeachingBubble. By default we should press key esc to close the call out. Press tab cannot be used to jump out of the focuszone.
For #5352, when only one language exists, the dropdown is disabled, which causes it cannot enter the focuszone because there is nothing to focus.
fix: use esc to close the callout. And enable the dropdown even if there is only one language exist.
Task Item
Closes #5597
Screenshots

Coverage remained the same at 55.034% when pulling 07bd4923e6cc61ef068c71610a0e1668988775f8 on callout into 837a7dd4bece13105032e47ffcfd98629175b24d on main.

Coverage remained the same at 55.034% when pulling 07bd4923e6cc61ef068c71610a0e1668988775f8 on callout into 837a7dd4bece13105032e47ffcfd98629175b24d on main.


[admin] remove HTML tags

Docs Build status updates of commit 4359d89:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/how-to-add-custom-ui-to-a-spreadsheet-document.md
:white_check_mark:Succeeded

docs/how-to-add-tables-to-word-processing-documents.md
:white_check_mark:Succeeded

docs/working-with-pivottables.md
:white_check_mark:Succeeded

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


cast between complex return None

I am writing a function, which uses the cast between "Complex" and other "Num"s such as "u64, f64, Complex". The cast between "Complex" and primitive will be correct. But how can I easily cast between "Complex" and "Complex"? This always return a "None" and leads panic.
    let a: Complex64;
    let b: Complex64 = Complex64 { re: (1.), im: (2.) };
    a = cast::<Complex64, Complex64>(b).unwrap();
    println!("{}", a);

Hmm, NumCast won't work because it goes through ToPrimitive, so you get None when there's an imaginary part.
I think we would ideally implement something like TryFrom<Complex<U>> for Complex<T> where T: TryFrom<U>, but that will probably conflict with the reflexive case, T = U.  Or we could just add a direct conversion method on Complex itself.


Where can I get the minified q.js?

You can get the minified file by running the grunt default task in the project folder. See the Gruntfile.js for more information.

Since someone is confused.. An easy way to use a minified version of Q is to use a CDN link, for example
https://cdnjs.cloudflare.com/ajax/libs/q.js/2.0.3/q.min.js
or download it from this location

@tyler-kearney That minified version seems to only work in NodeJS, and when loaded in a generic browser throws the error Uncaught ReferenceError: require is not defined


Change Roles to return Map of Roles instead of List

Move RolesDefinitions to testFixtures. This is a preparation to add more stuff to have cleaner tests, as Roles.roles() will return a Map<String, Role> instead of List<Role>.

Change Roles to return Map of Roles instead of List. List<Roles> was making tests easier, but a Map is more efficient and it's necessary for upcoming resolution of parent roles.

Relates: #12109

ok, I see, it makes sense to only do that in the impl, and leave the iface as is. thx!
Closing


"File not found"

Hi!
When I tried to upload an image file I got an exception "File Not Found".
FlowRequest request = await _requestReader.ReadGetAsync(context.HttpRequest);

        if (!IsValidRequest(context, request))
        {
            return context.HttpRequest.CreateErrorResponse(
                HttpStatusCode.BadRequest,
                "Invalid flow GET request");
        }

        string filePath = GetChunkFilePath(context, request);

        if (!await _fileSystem.ExistsAsync(filePath))
        {
            return context.HttpRequest.CreateErrorResponse(
                HttpStatusCode.NotFound,
                "File not found");
        }

        return context.HttpRequest.CreateResponse(HttpStatusCode.OK);
    }

The application is looking for a folder which is not exist.
"Images/chunks/84931-a1641090906bef9bjpg/00000001_84931-a1641090906bef9bjpg.chunk"
How can I resolve this problem?
Thanks advanced!
uracs

At what point to you get this error? The GET request is designed to return 404 when the file does not exists. This is how Flow.JS knows which parts of the file are already uploaded.

I get error here:
if (!await _fileSystem.ExistsAsync(filePath))
Is it a normal behavior? Sorry, maybe I misunderstood you.
I can see the image link in the list on the application page, but when I click on the link the browser cannot show the image because it cannot find. It is also a 404 error.
I guess I did not catch something. I try to use this example to upload an image using angularjs.
Where can I see the downloaded image?

The sample is by default configured to upload files to App_Data folder.
FilesController.cs
public FilesController()
{
  _fileSystem = new FileSystem
  {
        GetFilePathFunc = filePath => string.Format(
                    "{0}/{1}",
                    HostingEnvironment.MapPath("~/App_Data").Replace("\\", "/"),
                    filePath)
  };

  _flow = new Flow(_fileSystem);
}

Urls for the sample are configured set so that uploads both GET, POST actions of Flow.JS use the /uploads/{folderName} routes and downloading should hit the GetFile method of the FilesController by url /{*filePath}.
Have you done any changes to the sample?

@uracs did you solve your issue?

I ran your sample - after getting the same error as @uracs with my implementation and I am getting that error in both the sample and my implementation.
It appears that the HttpRequest.Method is being set to GET for both the Get & Post methods which is causing issues.
When I comment out
if (context.HttpRequest.Method == HttpMethod.Get)
{
     return await HandleGetRequest(context);
}

the drag and drop/single file work, but none of the rest do.


When scripting, can we have a boolean which answers 'y' to add newly created files when true

Aider v0.24.1
No git repo found, create one to track GPT's changes (recommended)? y
Added .aider* to .gitignore
Git repository created in the current working directory.
Model: gpt-4-0613 using diff edit format
Git repo: .git with 0 files
Repo-map: using 1024 tokens
Use /help to see in-chat commands, run with --help to see cmd line args
I am working with scripting.  I run coder.run('Using the example of ...'), and it decides the filenames based on the example.  But by default we see:
Allow creation of new file models.py? no
I understand when we create the coder, the example we're given reads:
fnames = ["foo.py"]

# Create a coder object
coder = Coder.create(client=client, fnames=fnames)

Suggesting by adding files to the coder we're giving permission to make changes.  But I don't want to foretell what the newly created files will be in my example as it's better if it makes up it's own names based on the use case and the examples
Could we have
coder.run('Using the example of ...', True)
as a way around this, so we can control the answer to the question of whether allowed to add?
Many thanks

aider -h shows --yes ... have you tried that?

I have a workaround in place already actually, just thought it would be a good functionality for scripters.
If you have a local checkout in ./aider-env, you can go to:
./aider-env/lib/python3.10/site-packages/aider/coders/base_coder.py
And remove this code:
           if not self.io.confirm_ask(f"Allow creation of new file {path}?"):
                self.io.tool_error(f"Skipping edits to {path}")
                return

Now it won't ask.  Not the most elegant, but ok for my local auto / scripting version of aider for now - may well cause gremlins I haven't come across yet this way.

There's a FAQ entry that might contain helpful information:
https://aider.chat/docs/faq.html#can-i-script-aider
I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


need more explanation...

Sorry i'm a newbie..
please explain these steps further..
This contains the 'decryption' algorithm, we want to 'extract' this and convert it to Python
The mostly likely things to change is the static ints in 'method_14' and/or the body in 'method_15'
The static array of (3) ints are renamed 'magic_bytes' and put at the top of 'decryption.py'
Make the Python class 'Decryption' in 'decryption.py' match the AS3 class inside of class_3

HI @ophott what bit are you stuck on?

oh got it solved,,


Dev UI: Stringify Jobs before rendering them

@jan-wilhelm does this fix your issue?

Coverage remained the same at 69.154% when pulling 5cb75e97e62d3a0d779eae015a231a63500b87cb on dev-ui-stringify-jobs-before-rendering into 49bf834a908ee1e025b4504bccd494fbeedea6e0 on main.


Use getApplicationIconBadgeNumber and setApplicationIconBadgeNumber

Expected Behaviour
I want to use the 2 functions but i didn't found how to implement them into my application.
I used $ionicCloudProvider.init() like in the ionic documentation to initialize the push plugin and not "var push = PushNotification.init()", but in the example the variable push is used so I dont know how to create this variable or an equivalent with the first method init().
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 6.0.1
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Asus Zenfone 2
Cordova CLI version and cordova platform version
cordova 7.0.1
android 6.2.3
ios 4.4.0
Plugin version
1.9.4
Sample Code that illustrates the problem
.config(function($ionicCloudProvider) {   $ionicCloudProvider.init({ ....   }); })
.run(function(...){     push.getApplicationIconBadgeNumber(function(n) {         console.log('success', n);     }, function() {        console.log('error');     }); }

That is certainly a Ionic issue then, you'll need to ask the guys there to expose the two methods you want to access through their thin layer API.
Either that, or you  need to read the docs they provide completely.


Expo: Spinner crashes after a device rotation change

Environment
Expo CLI 3.11.7 environment info:
System:
OS: macOS 10.15.3
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 8.11.4 - /usr/local/bin/node
npm: 6.9.0 - ~/.npm-packages/bin/npm
IDEs:
Xcode: 11.3.1/11C504 - /usr/bin/xcodebuild
npmPackages:
expo: ^36.0.0 => 36.0.2
react: 16.9.0 => 16.9.0
react-native: https://github.com/expo/react-native/archive/sdk-36.0.1.tar.gz => 0.61.4
react-navigation: ^4.0.10 => 4.2.2
npmGlobalPackages:
expo-cli: 3.11.7
Platforms
Only tested on iOS.
Versions

Android: n/a
iOS: 13
react-native-modal-datetime-picker: v8.5.0 +
react-native: https://github.com/expo/react-native/archive/sdk-36.0.1.tar.gz
react: 16.9.0

Description

Hi,
I've come across an issue where the picker either errors (on Expo) or shows a blank screen on opening of the picker after the screen orientation has changed.
Everything works fine initially, but when I rotate the device to change its orientation, it then causes problems. In Expo I get the following error:

When replicating on an iPad, the picker just doesn't appear after opening when the orientation has changed.
I've found that if I rollback to v8.4.0 of this picker, then it works as expected.
Reproducible Demo

To reproduce:

You can copy and paste the example code in the "Usage" section of the readme, into a project
Run it on the simulator with Expo or on a device (I used an iPad)
Click on the button to open the picker - it works fine as expected
Close the picker
Rotate the device to change it's orientation
Click on the button again to open the picker, in Expo it will show an error similar to the above. On a device the picker will no longer be visible.

Should be fixed by https://github.com/mmazzarolo/react-native-modal-datetime-picker/pull/403 once merged 👍

:tada: This issue has been resolved in version 8.5.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Chat isn't working as of now

I can login to development server with latest android app but can't enter or create a room. @nechaido @belochub @lidaamber

@lundibundi thanks, we need this to show at presentation next week


Error 'cannot be named' when using an exported union-type declaration

TypeScript Version: 1.8.7
file1:
export interface I1 {
  foo: string;
}

export interface I2 {
  bar: number;
}

export type I = I1 | I2;

file2:
import {I} from './file1';

export let createI = (): I => {
  return undefined;
};

Compile these files with declarations using
tsc --init
tsc -d

Expected behavior:
Javascript and .d.ts files are generated
Actual behavior:
An error is thrown: error TS4023: Exported variable 'createI' has or is using name 'I1' from external module "file1" but cannot be named.

Well, this is subtle. but it is by design.
The emitter tries to write a name for the type. type aliases are not really types, they are just aliases. so there is nothing called type I there is only I1 | I2.
Now the type for createI is inferred, though it is specified explicitly, it is specified in an assignment, so the actual type of createI as the compiler sees it is () => I1 | I2 (try to hover over it in your editor and you should see the same.); now it tries to write that, but I1 is not visible in this scope, and writing it would result in an invalid .d.ts. thus the error is generated.
You can specify the type explicitly, as such: let createI: ()=> I = () => { return undefined; } and that should be an "explicit" type for createI, and the emitter will honor it and write it the way it is. the other option is to export I as an interface instead of a type alias.

Hooray!
This finally got fixed in https://github.com/Microsoft/TypeScript/tree/v2.9.1


(Release 2.11) Update version to 2.11

updates version to 2.11
refines dependencies for test

Close/Reopen to rerun build checks -- 2.11 of egeria is now published.


share tools require http protocol

Fix for share tools, twitter requires http protocol
test links:  http://test-link.test.theonion.com/series/onion-film-standard'
http://test-link.test.theonion.com/video/scientists-continue-developing-alternative-energy--51011
@spra85 @collin

Coverage decreased (-0.2%) to 85.934% when pulling e48d0e2c54b7ee3a41014d43ca291fbc14ef434f on share-tools-fix into d87162c5bc901225952f0d9e4fa3cbcf33c3265c on master.

👍

Coverage decreased (-0.2%) to 85.972% when pulling 423fe9a196f8eb749729d05e4be91db39ad7d031 on share-tools-fix into d87162c5bc901225952f0d9e4fa3cbcf33c3265c on master.

👍
As mentioned, prefer "https" over "http" if possible

Whoops nevermind my HTTPS comment


Google Analytics events for attribution button click

Sends events to Google Analytics when the user clicks the attribution button
Fixes #187

screenshot of attribution events on GA

Decided on standup that this will be merged as-is and we'll create a separate issue to research how to use Google Analytics to figure out the number of distinct users that are clicking the attribution buttons.

In this separate issue, let's make sure to involve @sarahpearson and have that sign-off before implementation


[TW#12283] Support vTaskSuspend() and vTaskResume()

vTaskSuspend and vTaskResume are needed for our project, and in general any multi-task project. It seems that this functions needs to make multicore-compatible.
It would be nice for us to have these functions.

Implemented in 5728c8bd2.


Compile error on odroid ubuntu

[ 34%] Linking CXX shared library librealsense_viewer.so
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp: In member function ‘std::map<rs_stream, std::shared_ptr<rs::core::file_types::frame_sample> > rs::playback::disk_
read_base::find_nearest_frames(uint32_t, rs_stream)’:
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp:479:76: error: conversion to ‘std::vector<std::shared_ptr<rs::core::file_types::sample> >::size_type {aka unsigned
 int}’ from ‘std::map<rs_stream, long long unsigned int>::mapped_type {aka long long unsigned int}’ may alter its value [-Werror=conversion]
             auto prev = capture_time > m_samples_desc[prev_index[it->first]]->info.capture_time ?
                                                                            ^
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp:480:69: error: conversion to ‘std::vector<std::shared_ptr<rs::core::file_types::sample> >::size_type {aka unsigned
 int}’ from ‘std::map<rs_stream, long long unsigned int>::mapped_type {aka long long unsigned int}’ may alter its value [-Werror=conversion]
                 (capture_time - m_samples_desc[prev_index[it->first]]->info.capture_time ):
                                                                     ^
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp:481:55: error: conversion to ‘std::vector<std::shared_ptr<rs::core::file_types::sample> >::size_type {aka unsigned
 int}’ from ‘std::map<rs_stream, long long unsigned int>::mapped_type {aka long long unsigned int}’ may alter its value [-Werror=conversion]
                 ( m_samples_desc[prev_index[it->first]]->info.capture_time - capture_time );
                                                       ^
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp:482:76: error: conversion to ‘std::vector<std::shared_ptr<rs::core::file_types::sample> >::size_type {aka unsigned
 int}’ from ‘std::map<rs_stream, long long unsigned int>::mapped_type {aka long long unsigned int}’ may alter its value [-Werror=conversion]
             auto next = capture_time > m_samples_desc[next_index[it->first]]->info.capture_time ?
                                                                            ^
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp:483:68: error: conversion to ‘std::vector<std::shared_ptr<rs::core::file_types::sample> >::size_type {aka unsigned
 int}’ from ‘std::map<rs_stream, long long unsigned int>::mapped_type {aka long long unsigned int}’ may alter its value [-Werror=conversion]
                 capture_time - m_samples_desc[next_index[it->first]]->info.capture_time :
                                                                    ^
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp:484:53: error: conversion to ‘std::vector<std::shared_ptr<rs::core::file_types::sample> >::size_type {aka unsigned
 int}’ from ‘std::map<rs_stream, long long unsigned int>::mapped_type {aka long long unsigned int}’ may alter its value [-Werror=conversion]
                 m_samples_desc[next_index[it->first]]->info.capture_time - capture_time;
                                                     ^
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp:485:72: error: conversion to ‘std::vector<std::shared_ptr<rs::core::file_types::sample> >::size_type {aka unsigned
 int}’ from ‘std::map<rs_stream, long long unsigned int>::mapped_type {aka long long unsigned int}’ may alter its value [-Werror=conversion]
             sample = prev > next ? m_samples_desc[next_index[it->first]] : m_samples_desc[prev_index[it->first]];
                                                                        ^
/srv/maverick/var/build/realsense-sdk/sdk/src/cameras/playback/disk_read_base.cpp:485:112: error: conversion to ‘std::vector<std::shared_ptr<rs::core::file_types::sample> >::size_type {aka unsigne
d int}’ from ‘std::map<rs_stream, long long unsigned int>::mapped_type {aka long long unsigned int}’ may alter its value [-Werror=conversion]
             sample = prev > next ? m_samples_desc[next_index[it->first]] : m_samples_desc[prev_index[it->first]];
                                                                                ^

Hi @fnoop ,
The compiler is telling you that prev_index[it->first] is a long long unsigned int (uint64_t) while the vector's size_type is an unsigned int, and it issues a conversion warning which is treated as error due to the -Werror flag.
This issue was not detected by us since probably you are using a different compiler.
So first, which compiler are you using?
Second, you can work around this issue in numerous ways:

Remove the -Werror flag from the COMPILE_DEFINITIONS in cmake_includes/config_linux.cmake. This will add a warnings to your compile output but will enable you to build the SDK.
Add a flag to the compiler to ignore conversion warning. Add  -Wno-conversion to the COMPILE_DEFINITIONS in cmake_includes/config_linux.cmake). This will ignore all other conversion warning as well.
3.Change the source code by adding static_cast<decltype(m_samples_desc)::size_type>(VALUE) every time the code tries to access m_samples_desc using the subscript operator (m_samples_desc[VALUE]). This will tell the compiler that you know that the cast is safe from uint64_t to unsigned int.
This is probably the best way but I looked at the source code just now after writing all the above :) so hopefully you made it this far:
change the type of the prev_index and next_index variables from std::map<rs_stream, uint64_t> to std::map<rs_stream, decltype(m_samples_desc)::size_type> (or std::map<rs_stream, unsigned int> which is less robust but will work in your case). Since I don't know which compiler you are using that is using unsigned int for m_samples_desc::size_type , I'm not sure this won't cause any more warnings later on, so if it does please follow one of the above suggestions

Ziv

@zivsha Thanks for the quick response :)  This is using Ubuntu 16.04 on an Odroid.  Compiler is:
$ gcc -v
Using built-in specs.

Doesn't this need fixed in master?

@fnoop, Can you run the system_info script and post the results here?

Sorry, I got fed up with the odroid and sold it :(


sweetjs.org expired, on client hold

$ whois sweetjs.org
Domain Name:SWEETJS.ORG
Domain ID: D165783960-LROR
Creation Date: 2012-06-10T06:16:22Z
Updated Date: 2015-06-10T04:49:14Z
Registry Expiry Date: 2015-06-10T06:16:22Z
Sponsoring Registrar:Gandi SAS (R42-LROR)
Sponsoring Registrar IANA ID: 81
WHOIS Server:
Referral URL:
Domain Status: clientHold -- http://www.icann.org/epp#clientHold
Domain Status: clientTransferProhibited -- http://www.icann.org/epp#clientTransferProhibited
Registrant ID:DH2266-GANDI
Registrant Name:David Herman
...

Looks like @dherman just renewed it.


[CARBONDATA-1199] support dynamically enabling unsafe column page

Currently unsafe column page configuration is a static config, it should support dynamic configuration

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/509/

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/511/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2606/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2608/

retest this please

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2615/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/520/Failed Tests: 11carbondata-pr-spark-1.6/org.apache.carbondata:carbondata-spark-common-test: 11org.apache.carbondata.spark.testsuite.dataload.TestBatchSortDataLoad.test batch sort load by with out passing option but through carbon propertiesorg.apache.carbondata.spark.testsuite.dataload.TestBatchSortDataLoad.test batch sort load by with out passing option but through carbon properties with default sizeorg.apache.carbondata.spark.testsuite.partition.TestDataLoadingForPartitionTable.data loading for partition table: range partitionorg.apache.carbondata.spark.testsuite.partition.TestDataLoadingForPartitionTable.data loading for partition table: list partitionorg.apache.carbondata.spark.testsuite.partition.TestDataLoadingForPartitionTable.single pass data loading for partition table: range partitionorg.apache.carbondata.spark.testsuite.partition.TestDataLoadingForPartitionTable.single pass data loading for partition table: list partitionorg.apache.carbondata.spark.testsuite.partition.TestDataLoadingForPartitionTable.Insert into for partition table: range partitionorg.apache.carbondata.spark.testsuite.partition.TestDataLoadingForPartitionTable.Insert into partition table: list partitionorg.apache.carbondata.spark.testsuite.partition.TestDataLoadingForPartitionTable.badrecords on partition columnorg.apache.carbondata.spark.testsuite.partition.TestQueryForPartitionTable.detail query on partition table: range partitionorg.apache.carbondata.spark.testsuite.partition.TestQueryForPartitionTable.detail query on partition table: list partition

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2642/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/552/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3222/

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/626/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/44/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/55/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1230/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/599/


INCIDEN-660

Why
see https://govukverify.atlassian.net/browse/INCIDEN-660
Similar to https://github.com/govuk-one-login/ipv-spot/pull/951
What changed
Initialisation of SSMClient
Why did it change
To resolve ERROR InvalidSignatureException: Signature expired: 20240129T194125Z is now earlier than 20240129T194717Z (20240129T195217Z - 5 min.)
Based on SPOT resolution to a similar error
Issue tracking

Inciden-660

Cannot use the proposed solution i.e. top-level await as it is still experimental in terms of library support


Add cypress test for login

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Edit UI

Remove all system tests in order to change person card and person list panel to patient card and patient list panel.

Pull Request Test Coverage Report for Build 946

40 of 62 (64.52%)  changed or added relevant lines in 7 files are covered.
174 unchanged lines in 14 files lost coverage.
Overall coverage decreased (-3.6%) to 67.779%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/clinicio/model/ClinicIo.java
0
2
0.0%

src/main/java/seedu/clinicio/model/ModelManager.java
0
3
0.0%

src/main/java/seedu/clinicio/ui/MainWindow.java
0
8
0.0%

src/main/java/seedu/clinicio/model/util/SampleDataUtil.java
0
9
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/clinicio/logic/LogicManager.java
1
93.75%

src/main/java/seedu/clinicio/storage/JsonUserPrefsStorage.java
1
87.5%

src/main/java/seedu/clinicio/ui/HelpWindow.java
2
84.62%

src/main/java/seedu/clinicio/commons/core/Config.java
2
85.71%

src/main/java/seedu/clinicio/storage/StorageManager.java
2
87.5%

src/main/java/seedu/clinicio/model/UserPrefs.java
3
75.0%

src/main/java/seedu/clinicio/commons/core/LogsCenter.java
3
82.05%

src/main/java/seedu/clinicio/ui/ResultDisplay.java
4
80.95%

src/main/java/seedu/clinicio/ui/analytics/AnalyticsDisplay.java
5
0.0%

src/main/java/seedu/clinicio/ui/AppointmentListPanel.java
20
0.0%

Totals

Change from base Build 939:
-3.6%

Covered Lines:
2783

Relevant Lines:
4106

💛  - Coveralls


Avoid head-of-line blocking when handling large volumes

Separated across a few PRs.
The cleanup/GC part is still TBD - running with short timers to stress it to make sure it is robust and then will back of those 1 and 10 second timers.
This introduces a new workerpool package which provides the same API as the worker package, but runs separate workers in parallel based on the load. The count argument is the max number of workers it will run concurrently.
Note that should we exceed that number we explicitly log.Fatal. We could run this with an unbounded upper limit instead of 20, but that will consume more memory and potentially cause thrashing if we are doing that many volumes and/or CAS operations in parallel.
The changes to cmd/baseosmgr and cmd/volumemgr are quite small since the API (once we have the TrySubmit API) are basically identical.

I have never loved Try* as a function name, but that is just preference.

Any better suggestion? I know things like mutex_enter vs. mutex_tryenter.
Feels a bit more explicit to have a separate function as opposed to a "block bool" argument since the done return value only makes sense for the try variant.

do we need a done, err for TrySubmit? I think it might be more consistent with the language if we just return an error, and have a custom error worker.FullQueue. We can test for that. It also simplifies the if / else if statements after each TrySubmit

But semantically it isn't an error. That it what "Try" means.
Having done, err := foo() seems more clear to me that having to look at errors to say what the state is. Very different than to look an an error to tell who originated it (TLS vs. kernel TCP for instance); both of those are failures.
This is analogous to the language starting with "item := mymap[key]" and it just panics and then realizing that "item, ok := mymap[key]" is much more useful. Sure, could just error/panic and tell from the error/panic that the key was missing but why.

can we extract the default capacity of parallel workers to a public const in the package and just use it in NewPool?

Don't understand your suggestion.
Which default? The 20 they two callers use?
golang unlike python do not have default values for omitted parameters.

how does the submit actually work? It looks like we create a slice of worker.Worker of capacity 10 (or whatever length) but length 0, so how does it find any to submit?

First TrySubmit creates a new worker as long as it is below the limit.

Any better suggestion? I know things like mutex_enter vs. mutex_tryenter.
Feels a bit more explicit to have a separate function as opposed to a "block bool" argument since the done return value only makes sense for the try variant.

I wish I did :-)

Don't understand your suggestion.
Which default? The 20 they two callers use?
golang unlike python do not have default values for omitted parameters.

Yes. The default limit of 10 here limit is hard-coded into the function. It would be better if it were:
const (
    DefaultPoolSize = 10
)
// later
func NewWorkerPool(log worker.Logger, ctx interface{}, maxWorkers int, handlers map[string]worker.Handler) *WorkerPool {
 	workers := make([]myworker, 0, DefaultPoolSize)
	length := maxWorkers
 	if length == 0 {
 		length = DefaultPoolSize
 	}

First TrySubmit creates a new worker as long as it is below the limit.

I missed that, which was my question. And when a worker is done, it goes back to the pool?

Yes. The default limit of 10 here limit is hard-coded into the function. It would be better if it were:

That isn't a limit but just the initial capacity of the slice. As we append it will grow past that. I can get rid of that capacity - the efficency it adds would be in the noise.

I missed that, which was my question. And when a worker is done, it goes back to the pool?

They are always in the pool, but get skipped if they are busy i.e., if they are not blocked on reading on their request channel.
The GC removes workers from the pool if they haven't been used for a while (and I'll fix the "while" to have a more sane value but need the tests to use a short value).

This looks OK, but one statement from @deitch  caught my eye:

and I have no clue what most of the yetus errors mean

I don't think we can have that -- I think we need to have a pretty clear policy on when we merge (otherwise it is going to be very difficult to stay on the same page). Now, granted we have a brand new CI/CD with brand new checks in place -- so we have to agree on what is and what isn't part of the criteria. But we have to do it explicitly and pretty fast.
WDYT @eriknordmark @deitch -- what's the best forum to hammer out that agreement?

Unfortunately, I cannot find the output again. It was built in here before but now that it is merged, I cannot find it. Let's find it, post it to an issue, and then discuss it in that issue.

It is actually right there if click on top-level "Files Changed" tab in this very PR and keep scrolling.

Oh yeah, I see it now. I thought it would be in actions.


Set default values to a secure value - Recovery Services Vaults

All default values should comply with a security baseline, e.g. NIST 800
The build-in policies of Azure can be used as a reference.
The task would be to scan over each of the following policies and make sure, that the module is per default complying to them.
The following policies are the NIST 800 ones:
\built-in-policies\policyDefinitions\Backup\AzBackupRSVault_CMKEnabled_Audit.json
\built-in-policies\policyDefinitions\Backup\VirtualMachines_EnableAzureBackup_Audit.jso

Enabling encryption at vault creation using customer managed keys is in limited public preview and requires allow-listing of subscriptions. Agreed with Alex and Michael that we should not implement this feature for the moment.


Fix warnings & failing tests

Warnings and tests was introduced in #1854 from what I can see

@swift-ci please test


feat(track-line): add track line styles

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Chinese input sogou input window flutters horizontally

Issue Type: Bug
When sogou input window touches the bottom of the screen, it will display above the text, and then it starts fluttering horizontally.
It looks really horrible and is unendurable.
But I tested in an older version of vscode 1.59 it behaves correctly.
I guess this problem is because of some changes in recent updates.
VS Code version: Code 1.64.0 (5554b12acf27056905806867f251c859323ff7e9, 2022-02-03T04:22:20.678Z)
OS version: Windows_NT x64 10.0.19041
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8300H CPU @ 2.30GHz (8 x 2304)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
19.79GB (8.77GB free)

Process Argv

Screen Reader
no

VM
50%

Extensions: none

/duplicate of #142155


Oracle Ruling Image Posts

Like this, thanks to Mindwarped

Introduced with https://github.com/mdiehr/mtgnewsbot/pull/73


Kestrel API to retrieve raw http request bytes

I'm trying to port an application that uses the HTTP.SYS APIs to retrieve the Raw Bytes of the HTTP request -- starting from the first request line till the \r\n\r\n.
The reason it is being done is that the WebListener classes when presenting the data through HttpRequest caused the data to be converted to a .NET String and for certain classes of request this caused problems in this application.
I'd like to be able to have an API that can retrieve the raw bytes of the HTTP request. This will allow me to have backwards compatibility.
Kestrel does have the ability to get to the connection data using Connection Abstractions, which is how Kestrel abstracts the HTTPS code from the core library. I'm wondering if there is a less intrusive way of doing this that I've missed through reading the code.
If not I suppose my needs can be met with whatever exists, but thought I'd ask and see if this has been considered before or if others have asked for such a feature,

It's not possible today without bigger changes or reaching down to the lower connection level and reusing some of the internal kestrel APIs. Kestrel eagerly parsers headers today when they are encountered and we don't keep those bytes around in any other form but the string. The PlatformBenchmarks  project shows how to write a low level server with Kestrel APIs today.

The reason it is being done is that the WebListener classes when presenting the data through HttpRequest caused the data to be converted to a .NET String and for certain classes of request this caused problems in this application.

Is it problematic that they will be converted anyways?

Is it problematic that they will be converted anyways?

No, the majority of the codebase deals with the managed object and its representation, but certain parts rely on the bytes as-is.
I wonder if it's something people will want for this and/or other reasons. For example when a crash occurs today, we're able to find the raw bytes and replay them exactly as we encountered them and I've seen a few issues here that suggest enabling the Connection Logging to see what bytes really came over the wire.
If you think it may be reasonable to supply some code as part of Kestrel or some package then we can keep the issue open, otherwise I know what I'll have to do.


displayed:false on visible elements

Hi! While testing the Selenide API I found a case using the default Firefox Driver (using Firefox 39.0 on OS X Yosemite) where a plainly visible element on the page fails to be interacted with as Selenide believes it isn't displayed.
! com.codeborne.selenide.ex.ElementShould: Element should be visible {By.name: session[username_or_email]}
! Element: '<input autocomplete="on" class="js-username-field email-input js-initial-focus" name="session[username_or_email]" type="text" displayed:false></input>'

gets thrown when the following code is run:
open("https://twitter.com/login/");
$(By.name("session[username_or_email]")).val("name");
$(By.name("session[password]")).setValue("password");
$(".submit").click();

@zolrath The problem is that there is 3 such fields on the page.
Try to add the following line to your test:
$$(By.name("session[username_or_email]")).shouldHave(size(1));

Then you will get the folloing error:
ListSizeMismatch : expected: 1, actual: 3, collection: By.name: session[username_or_email]
Elements: [
	<input autocomplete="on" class="js-username-field email-input js-initial-focus" name="session[username_or_email]" type="text" displayed:false></input>,
	<input autocomplete="on" class="js-username-field email-input js-initial-focus" name="session[username_or_email]" placeholder="Phone, email or username" type="text"></input>,
	<input autocomplete="on" class="text-input email-input" id="signin-email" name="session[username_or_email]" placeholder="Phone, email or username" type="text" displayed:false></input>
]

There is another feature request that addresses the same problem: https://github.com/codeborne/selenide/issues/129
Thank you for reporting a very easy example (Twitter). I will use it for implementing issue https://github.com/codeborne/selenide/issues/129.

Thanks for the quick response!
$$(By.name("session[username_or_email]")).filter(visible).get(0).setValue("name"); was my workaround in this case.

I am facing the same error and my page definetly does not have more than one field with the same id.
Any idea what else could be causing this?
I am trying to select a check box by id and it says it is not visible...

Hi!
Can you code sample and html?
On Jan 19, 2016 4:29 AM, "Fernando Alves"<EMAIL_ADDRESS>wrote:

I am facing the same error and my page definetly does not have more than
one field with the same id.
Any idea what else could be causing this?
I am trying to select a check box by id and it says it is not visible...
—
Reply to this email directly or view it on GitHub
https://github.com/codeborne/selenide/issues/201#issuecomment-172699262.

@asolntsev I have the same issue currently and it is also, the same as from last commented, not connected with more than one element with the same id.
Initially I had PageObject with defined element using @FindBy:
@FindBy(id = "_portlet_currencyWorldwideCheckbox")
private SelenideElement allowAnyCurrencyCheckbox;

And was getting error when trying to allowAnyCurrencyCheckbox.setSelected(true):
Element should be visible {By.id: _portlet_currencyWorldwideCheckbox}
Element: '<input class="aui-field-input aui-field-input-choice icheck-input icheck[svz0a]" id="_portlet_currencyWorldwideCheckbox" name="_portlet_currencyWorldwideCheckbox" onclick="Liferay.Util.updateCheckboxValue(this); currenciesjsp_changeWorldwide(this);" type="checkbox" value="*" selected:true displayed:false></input>'

Was trying the same with $(By.id("_portlet_currencyWorldwideCheckbox")).click() or $(By.id("_portlet_currencyWorldwideCheckbox")).setSelected(true) - outcome "not changed"
I experimented a bit, and seems it happens on stage of wrapping WebElement to SelenideElement, you should know better, and when it marks it with displayed:false. We have a bit messy pages there, they are jsp's with JavaScript and jQuerry, but you actually could interact with that element, if you use WebElement of Webdriver, so as example this would work:
$(By.id("_portlet_currencyWorldwideCheckbox")).toWebElement().click()
So the problem, when I call SelenideElement to perform some action with it, but seems it is already rely on displayed:false, right?
Here more html snippet of the element and parents:

          <input id="_portlet_currencyWorldwide" name="_portlet_currencyWorldwide" type="hidden" value="*" />
          <input checked class="aui-field-input aui-field-input-choice"  id="_portlet_currencyWorldwideCheckbox" name="_portlet_currencyWorldwideCheckbox"  onClick="Liferay.Util.updateCheckboxValue(this); currenciesjsp_changeWorldwide(this);"  type="checkbox" value="*"   />

         The provider allows any currency.

Let me know if you need more info from me...

@stupchiy sorry for the long reply.
I tried your example, and it worked for me. I don't think there could be any difference between WebElement and SelenideElement.
Probably it's just a timing issue.
Look, the error message clearly says "Element should be visible" - this means that at that moment the element was invisible.

@asolntsev no worries about delay, thanks for reply in any case.
Well, I am sure it's not about timing, since it is not trying to perform that action before all elements loaded (tried with different delay's), my suspicion was that displayed:false not always properly set.
And also as I mentioned, when you try to perform $(By.id("_portlet_currencyWorldwideCheckbox")).click() or $(By.id("_portlet_currencyWorldwideCheckbox")).setSelected(true), when element is definitely loaded, it is not perform those action's.
Anyway thanks again, that took time to look into it.

@asolntsev @stupchiy
I just came across the same issue. And I think this could be a Selenium WebDriver issue, rather then a Selenide one.
I check a certain group of elements in my page for visibility, then some changes are made to the page and these elements should still be visible. And they are, but Selenide reports they are not, inserting displayed:false into the element. Basically, these are the same elements as in the first case, but for some reason they become invisible to Selenide.
Element should visible {.additional-navigation .help-block a}
Element: ''

And in my case it's certainly not the timing issue - I used debug mode to make sure the elements are properly loaded. I used css selectors and selector byText to locate the elements, the latest version of Selenide.
I tried to rewrite my check like this
assertThat(WebDriverRunner.getWebDriver().findElement(By.cssSelector(".blue-link")).isDisplayed()).isTrue();
and I am still getting an error, since isDisplayed() returns false, when elements are clearly visible.

I had this issues also and used javascript as a workaround to click on the checkbox. Would be nice if there will be a solution

I have the same issue in my code - I find element by Id, click on it and have 'Element should be visible' error message. Also use javascript click as a workaround.
Any chance that this ticket will be reopened?

@FrameBassman Sure, I have reopened the issue. I need to investigate your code with html.

@FrameBassman Please also provider your Java code that fails.

@asolntsev Java code is here https://gist.github.com/FrameBassman/cca80bb6c52ea34eb1dc70d82930f09f

@FrameBassman Thank you! I will research it in details,
but at a first glance, Selenium thinks that this element is not visible. Do you know why? Probably it has some magic CSS properties? Is it really visible to users?

@FrameBassman , I faced the same issue. Try XPath locator. It works for me :)

I had the same issue for a Oracle Jet checkbox and i tried the workaround by @stupchiy
$(Locator).toWebElement().click() it works now !

//selenide code
public Page closeWindow(){
        if ($("button.close").is(Condition.visible)){ // .isDisplayed() or .isEnabled() - the same result
            $("button.close").click();
        }
        return this;
    }

Element should be visible {button.close}
Element: ''
When button is visible - works fine. But when it's hidden - nope.

Is there any progress on this? I also have the error and always using xpath is annoying.

@thereiskeks @SEMrushTasks @sadsridh @lminiailo I investigated this problem once more and found that this element has CSS property opacity: 0. For Selenium it means that the element is not visible. But Selenide method $.click() only allows to click visible elements.
We probably should change $.click() implementation so that it also allowed clicking invisible elements (like this one with opacity: 0).
@BorisOsipov @rosolko @symonk @vinogradoff  What do you think?

@gedankennebel We are discussing a better method name in PR https://github.com/selenide/selenide/pull/851

Thanks @asolntsev this addresses a lot of failures due to ElementNotVisible errors.

hi,
I upgraded to selenide 5.2.0 and still see this issue with elements having zero opacity.
From the changelog for Selenide version 5.1.0 which says this is fixed..
#201 can click elements with zero opacity -- thanks to @vinogradoff for PR #874

@sadsridh Yes, it should be fixed.
Can you provide html code and exact error message with full stack trace?

Element: '<button aria-label="Edit the list" class="oj-button-jqui oj-button oj-component oj-enabled oj-default oj-button-full-chrome oj-button-text-only oj-component-initnode" data-bind="
    attr: {
      id: id + '-editButton',
    'aria-label': $root.resource('ns.common:resources.ariaLabelEditListEditButton')
    },
    ojComponent: {
      component: 'ojButton',
      label: $data.isViewButton() ? 'View' : $root.resource('ns.common:resources.editButton')
    },
    click: onEdit,
    enable: enabled" id="exclude-editButton" type="submit" displayed:false></button>'

Screenshot: file:/D:/.../uitests/admin/build/reports/tests/1552326455484.0.png
Page source: file:/D:/.../uitests/admin/build/reports/tests/1552326455484.0.html
Timeout: 10 s.
	at com.codeborne.selenide.impl.WebElementSource.checkCondition(WebElementSource.java:72)
	at com.codeborne.selenide.commands.Should.should(Should.java:35)
	at com.codeborne.selenide.commands.Should.execute(Should.java:29)
	at com.codeborne.selenide.commands.Should.execute(Should.java:12)
	at com.codeborne.selenide.commands.Commands.execute(Commands.java:144)
	at com.codeborne.selenide.impl.SelenideElementProxy.dispatchAndRetry(SelenideElementProxy.java:99)
	at com.codeborne.selenide.impl.SelenideElementProxy.invoke(SelenideElementProxy.java:65)
	at controls.admin.modules.AdminModule.scrollIntoView(AdminModule.groovy:244)
	at .controls.admin.modules.IXEditorModule.openModalForAdd(IXEditorModule.groovy:233)
	at controls.admin.modules.IXEditorModule.addItemsToPromotion(IXEditorModule.groovy:37)
	at controls.admin.modules.PromotionModule.setPromotionProperties(PromotionModule.groovy:386)
	at controls.admin.modules.PromotionModule.editPromotion(PromotionModule.groovy:319)
	at uitest.spec.promotions.ItemDiscountPromotionsSpec.Item discount amount off can created, edited, deleted(ItemDiscountPromotionsSpec.groovy:118)```

@asolntsev ,
After further debugging the actual issue is because a collapsible that displays a list of buttons after its clicked does not seem to acknowledge the click() and so it never expands to display the '-editButton' in the above element and so it is flagged with displayed:false which is correct !
However, after watching the test during debug the below element

does receive the click() but it does nothing. Manually this works fine in a the application.

Even i'm facing the same issue with 5.2.2 version.
Test Only
This is my element. Was able to get through this error with executeJavaScript but hoping we will have a solution for this.

Looks like I'm facing the same issue since I updated from 5.1.0 to Selenide 5.2.0 as some test scripts fail when being run headless with Chrome. The following error message is displayed:
Element should have text '3.0.86' {.label-title-block-protocol-version td[0]}
Element: '	'

Strange is that the capture screenshot doesn't show the complete browser window, as it seems to be 'truncated'. Only a part of the browser window is captured. The part that's missing has the element (.label-title-block-protocol-version td[0]).
Workaround for me:
Run the same test non-headless i.e.  Configuration.headless = false; or
revert to Selenide 5.1.0 (with the same ChromeDriver version) works fine in headless mode.
Selenide Version: 5.2.0, Chrome Version: 74.0.3729.131, Browser Driver Version: 74.0.3729.6
Selenium Version: 3.141.59, OS Version: Windows 10 Version 1803 (17134.765)

@metalmolly it seems chromedriver\chrome issue, not selenide, isn't it?

@metalmolly it seems chromedriver\chrome issue, not selenide, isn't it?

@BorisOsipov but for me that doesn't explain why headless works using the same driver version with 5.1.0?

I see the issue with Selenide 5.2.2 with the below config :
Selenide Version: 5.2.2, Chrome Version: 74.0.3729.131, Browser Driver Version: 2.46
Selenium Version: 3.141.59, OS Version: Windows 10
@asolntsev  This bug fix is one of the main reason we are upgrading to the latest version...

Even i'm facing the same issue with 5.2.2 version.

@g-one Can you specify what issue exactly are you facing? I tried to click an element with HTML that you provided - and it worked. Also in headless mode.
Probably you could create a new github issue with html and java code (so I could reproduce the problem)?

@sadsridh Sorry, I didn't understand which issue do you have in this setup?
Why are you using chromedriver 2.46? I believe Chrome 74 requires chromedriver 74 (not 2.46).
If your problem is still valid, please create a separate github issue with html and java code (so I could reproduce the problem).

Hi, I'm facing the same problem I think.
This is a snippet of the html code

Januari 2020

Trace :
Element should be visible {By.xpath: //div[contains(@class,'x-link-next-image')]}
Element: ''
I try to click on the right arrow in the calendar.


Test nodes interaction

Check availability from android client
Check availability from go backend client (websocket)

Add endpoint to controllers and .env

Done


Clean up UG documentation for GetTotalExpenditure and the budget managment commands

They're merged into the website already, they're at the top, along with all the other constraints


feat: add remote_repository_name field for query

Category

[x] New feature
[ ] Bug fix
[ ] Improvement
[ ] Refactor
[ ] etc

Description
add remote_repository_name message field for list query
Known issue

Thank you! : )


Excluded folders being indexed

My VScode config specifies some files and folders to ignore:
files.exclude": {
        "**/.git": true,
        "**/.svn": true,
        "**/.hg": true,
        "**/.DS_Store": true,
        "**/docs": true,
        "**/upgrade": true,
        "**/uploads": true,
        "**/node_modules": true,
        "**/.vscode": true
    }

But Intellisense is scanning some of these as can be seen in the output window:
Parsing file 1/2352: file:///private/var/www/sites/mysite/docs/temp/inc/admin.php
Parsing file 2/2352: file:///private/var/www/sites/mysite/docs/temp/inc/comments.php
Parsing file 3/2352: file:///private/var/www/sites/mysite/docs/temp/inc/core.php
Parsing file 4/2352: file:///private/var/www/sites/mysite/docs/temp/inc/courses.php

In this case the docs folder contains a lot of PHP files that are not part of my project. Is there a way to specify that some folders should not be scanned, or for Intellisense to not scan excluded folders?

This issue was moved to felixfbecker/php-language-server#246


[Bug] Homebrew version mismatch

This seems weird to me:

When you opt-in into a newer UX for tctl by running tctl config set version next, tctl -v will start printing 1.17.0-alpha.2 as the version
see https://github.com/temporalio/tctl/blob/e82d00ceeb9f7f14d86bf7158600fd2df1314d2e/cmd/tctl/main.go#L40 which switches the active version based on the value from config

In the main branch the version was also updated to tctl version 2.0.0-beta.1
This is so people can opt-in and start playing with tctl v2


Run file button in toolbar resembles tab slider tool #UI

Issue Type: Bug

Open many files
Look for one file in the row of tabs.  Maybe it's hidden to the right of the toolbar.
Click green arrow button that looks like a tab slider button, seen on other applications, that slides tabs to the left and brings hidden tabs into view.
In actuality the tool launches a script that you didn't want to launch.
Spend time cleaning up the results of the inadvertent script launch.

I've now done this twice.
Suggest that you alter or move the "run file in terminal" button so that it's less likely to be mistaken for a "tab slider" button.
VS Code version: Code 1.55.2 (3c4e3df9e89829dce27b7b5c24508306b151f30d, 2021-04-13T09:36:32.643Z)
OS version: Darwin x64 20.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5287U CPU @ 2.90GHz (4 x 2900)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 3

Memory (System)
8.00GB (0.09GB free)

Process Argv
--crash-reporter-id 02e12d1d-edfb-4535-b6e4-145f321323f0

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

go
gol
0.24.2

vscode-docker
ms-
1.12.1

python
ms-
2021.4.765268190

jupyter
ms-
2021.5.745244803

remote-containers
ms-
0.166.1

text-power-tools
qcz
1.27.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
pythonvspyt602cf:30294773
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvsnew554cf:30291488
pythontb:30283811
vspre833:30267464
pythonptprofiler:30281270
vshan820:30294714
pythondataviewer:30285071
vscus158cf:30286554

\extPython

vscode-python reports that the placement of the button is controlled by a vscode API.
So, you told me to report the issue to vscode-python, which I did, and they told me that it's a vscode issue.

The appearance is controlled by the Python extension.

Sorry, misunderstanding on my part; I misread what @dmcbride64 was commenting on about the arrow, thinking it was overflow behaviour, not the fact that it's an arrow itself.


change length descriptor according to specification

According to Matroska specification (http://www.matroska.org/technical/specs/index.html) "Any ID where all x's are composed entirely of 1's is a Reserved ID". Given a length l the valid range of values is 0 to  2^(7 * l)-2. Previous implementation caused corruption of blocks with data length 127, 16383, ..., 2^(7 * l)-1.
I noticed this because streams got eventually corrupted. The browser stopped decoding with MEDIA_ERR_DECODE. chrome://media-internals/ only stated that decoding the data has failed. Recording the stream into a file and processing it with ffmpeg gave
[matroska,webm @ 0x718560] Invalid length 0xffffffffffffff > 0x10000000 for syntax element 5
with a few frames dropped. Tracking it down which frames were dropped it appeared that there was a SimpleBlock with data length of 127 bytes. The remaining SimpleBlock's of that Cluster were dropped and processing continued on the next healthy cluster. After fixing the bug and re-encoding the same (corrput) webm file all frames were processed fine.

Okay, seems reasonable. Thanks.

published to npm as 2.1.0


Restructure tests

Split up kernel.js as it is getting too big.

Some course reorganisation has been done on the syscalls branch. It should be enough for now.

Let's close an issue only once it's merged into master. In this case, once we merge the syscalls branch

I would offer to make a PR for tracking this issue along with #43 and #57

No, I don't think they're connected.


[TextAnalytics] [Conversations] Update method naming in client.tsp

Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

Looks like the Model Validation step is reporting some issues in the examples due to the changes in this PR. Please resolve before we can merge.
https://github.com/Azure/azure-rest-api-specs/pull/29622/checks?check_run_id=27636351801


chore: fix workflow inputs print

Which problem does the PR fix?

The new inputs print is breaking this workflow https://github.com/camunda/camunda/actions/runs/9656735771/job/26637842094 . The workflow uses vault-secret-mapping input which is a multiline input including pipes, so it does not properly work with toJSON method.
What's in this PR?

Checklist
Please make sure to follow our Contributing Guide.

Before opening the PR:

[ ] In the repo's root dir, run make go.update-golden-only.
[ ] There is no other open pull request for the same update/change.
[ ] Tests for charts are added (if needed).
[ ] In-repo documentation are updated (if needed).

After opening the PR:

[ ] Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
[ ] Did all checks/tests pass in the PR?

@houssain-barouni thanks for the fix :raised_hands:


drivers: dac: add a SAM0 DAC driver

The Atmel SAM0 DAC is a single channel 10 bit DAC with selectable reference.
Tested using samples/driver/dac on the Arduino Zero.  Enabled for the SAMD20/D21/R21 only as the D5x/E5x are slightly different.
Uses the enum value to map between a DeviceTree string enum and the native value.  This should be reworked when #21273 lands.

@sslupky could you review please?

You mentioned this PR is valid also for SAM R21. Maybe I didn't find the right datasheet, but the SAM R21G and R21E don't seem to have a DAC.

Interesting, the asf files say that it has a DAC but the datasheet says no.  Changed.

I think you need add your board at platform_whitelist on samples/drivers/dac/sample.yaml to run on CI

Done

You mentioned this PR is valid also for SAM R21. Maybe I didn't find the right datasheet, but the SAM R21G and R21E don't seem to have a DAC.

Interesting, the asf files say that it has a DAC but the datasheet says no. Changed.

Just to mention, R21 would supposed to be a SiP D21 + RF233 (at SERCOM 4).

Just to mention, R21 would supposed to be a SiP D21 + RF233 (at SERCOM 4).

Yeah, it's strange.  According to the datasheet the DAC pin PA02 isn't present.  I wouldn't be surprised if the peripheral is there but it's not routed to a pin.


Vue: Fix Enabled Reactivity

What is the issue?
We pass a reactive (ref/computed) object to the enabled field of the query object in useSomething(). The generated enabled field incorrectly evaluates to true even when the query param(s) are undefined leading to queries such as updateId/undefined.
What is happening?
const getSomethingQuery = useGetSomething(computed(() => something));

...

// generated code
export const getSomethingQueryOptions = (..) => {
    ...
    // 'something' is of type MaybeRef<string> (for example)
    return { queryKey, ..., enabled: !!something }
}

What is expected?
export const getSomethingQueryOptions = (..) => {
    ...
    return { queryKey, ..., enabled: computed(() => !!unref(something) }
}

Hey guys, teammate here :)
So we're using vue-query and passing a ref parameter. ref is never undefined, it's always a proxy object, so the !! check doesn't work.
The proposed solution works for us, and we had a bit of discussion previously in here: https://github.com/gkweb/orval/pull/2#discussion_r1193278199

@Maxim-Mazurok @BenGane #931 fix the same issue right? Or I missing something?

Replied in https://github.com/anymaniax/orval/pull/944#issuecomment-1738380725 :)

hey, i'm currently evaluating orval for use in my (existing) vue/query app. i already have some composables written by hand, so i'm wondering what issues i might hit before this gets merged.
am i right that it fails if you pass an object of refs:
const myParam = ref("1")
const query = useSomeQuery({ myParam })

but it will be fine if you pass a ref object:
const myParam = ref("1")
const params = computed(() => ({
  myParam: unref(myParam)
}))
const query = useSomeQuery(params)

of course the latter is verbose and cumbersome, but just checking i understand the issue correctly. thanks!

hey, i'm currently evaluating orval for use in my (existing) vue/query app. i already have some composables written by hand, so i'm wondering what issues i might hit before this gets merged.
am i right that it fails if you pass an object of refs:
const myParam = ref("1")
const query = useSomeQuery({ myParam })

but it will be fine if you pass a ref object:
const myParam = ref("1")
const params = computed(() => ({
  myParam: unref(myParam)
}))
const query = useSomeQuery(params)

of course the latter is verbose and cumbersome, but just checking i understand the issue correctly. thanks!

I think that in both cases query will always be enabled in the current master, which is wrong. This PR will fix that.


Could not find xxx.opnv

The message I got is:
Write out database with 1 new entries
Data Base Updated

Client xxxx added, certs available at ~/xxxx.ovpn

But if I tried to find it,I got
cannot access /home/xxx/xxxx.ovpn: No such file or directory

Thank you very much

Check /root/xxxx.ovpn.


Update dependencies.

Coverage remained the same at 100.0% when pulling b81a35480bd3d45f73bb9fecdcc65b31814d1e73 on update-dependencies into 4c0af962bb070ff6dbb4bcef34677b025094cded on master.


⚠ ClamAV has degraded performance

In c719320, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 221 ms

Resolved: ClamAV performance has improved in 799e9e9.


Enable docker tests in pre-submit check

Some tests require docker and they are not enabled in pre-submit check. We need to add make test-docker for next branch in pre-submit check.

cc @droot


Add EdaTool doc comments and enable serde attributes on protobuf generated code

Codecov Report

Merging #48 (8f047ff) into master (bc0e469) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
- Coverage   62.07%   62.06%   -0.02%
==========================================
  Files          41       41
  Lines        3254     3242      -12
==========================================
- Hits         2020     2012       -8
+ Misses       1234     1230       -4

Impacted Files
Coverage Δ

edatool/src/plugins/ngspice_sim/tests/mod.rs
100.00% <100.00%> (+6.55%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Alphabetize reference entries and move guide content

Normally, one would expect the API reference to be alphabetically sorted by  function name within each object.  Currently, the order is unclear, probably some kind of "conceptual" order, which makes sense when it is the only doc.  But since we've started to separate out API and "guide" material, I suggest we do the following:

Move content from Settings and Routing sections into the appropriate sections under Guide.  The Middleware section is a leftover and hopefully we can just remove it nnow.

Alphabetize entries in the API reference.

Consider separating Method and Property references explicitly. Currently only the presence of a () indicates whether its a function or property.

Created a branch, reorg, to make the changes and keep track of the progress. Those who are interested, please compile the site from time to time, and provide feedback.
Point 3 (demarcating methods and properties) is addressed in the initial commit. Please note, the order is yet to updated in the main doc.

This is now merged into gh-pages branch and published.   Easier to find stuff now that methods and props are separated and aphabetized.


Updates for .NET Framework 4.8.1

[ ] Update .NET Framework versions and dependencies
[ ] Update What's new in .NET Framework
[ ] Update How to: Determine which .NET Framework versions are installed
[x] Update breaking changes for 4.8.1 target - see #30541

Related to #29985.

I'm especially interested in "breaking changes" page since .NET 4.8, similar to this one:
https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/runtime/4.7.2-4.8

I'm especially interested in "breaking changes" page since .NET 4.8, similar to this one: https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/runtime/4.7.2-4.8

@martinsuchan There are no breaking changes since 4.8. I am updating the breaking changes docs today to reflect this.

Is there documentation coming about the Arm64 support in .NET Framework 4.8.1? My understanding is that it requires some kind of opt-in from the application author, and that some features are not supported under the Arm64 version of .NET Framework. @richlander mentioned that there will be articles explaining how to opt-in to Arm64 support for .NET Framework apps -- are they coming soon? Is it required to target .NET Framework 4.8.1 for Arm64 support, or is there a way to opportunistically enable this support for apps targeting lower versions?

Is there documentation coming about the Arm64 support in .NET Framework 4.8.1? My understanding is that it requires some kind of opt-in from the application author, and that some features are not supported under the Arm64 version of .NET Framework. @richlander mentioned that there will be articles explaining how to opt-in to Arm64 support for .NET Framework apps -- are they coming soon? Is it required to target .NET Framework 4.8.1 for Arm64 support, or is there a way to opportunistically enable this support for apps targeting lower versions?

There is some documentation published already.  This issue is to track the additional updates which are needed.  Documentation that is already published is:
Blog post: https://devblogs.microsoft.com/dotnet/announcing-dotnet-framework-481/
Downloads page: Download .NET Framework | Free official downloads (microsoft.com)
Release notes: https://github.com/Microsoft/dotnet/tree/master/releases/net481/README.md
Blog post for Visual Studio 17.3: https://devblogs.microsoft.com/visualstudio/visual-studio-2022-17-3-is-now-available/

I've staged changes for these and more areas needing updates in in this PR: https://github.com/dotnet/docs/pull/30619

Update .NET Framework versions and dependencies
Update What's new in .NET Framework
Update How to: Determine which .NET Framework versions are installed


Fix Collective check for PS Tools 2017

Updated AU20009 to check the Data Archive connection type object, and
choose whether to use the old method for checking collective membership
or the new method made necessary by the new ClientChannel type used in
PI Powershell Tools 2017.
Fixes #255

I chose to check the object type on the Data Archive connection rather than directly checking the OSIsoft.Powershell version. If there are additional changes that we discover in this 2017 version or later versions, we may want to consider a global check of the version that can be used to decide code paths in audit checks that are affected.


Local History Not Updating

Type: Bug
This is what I did and the error I saw:
1.) SSH into remote server and have Autosave on
2.) Create and Save a Jupyter Notebook File
3.) Work on it over a few days, still SSH-ing into the same remote server and still with Autosave on
3.) Accidentally delete said file despite not clicking on that file to be deleted but instead clicking on other files to be deleted (I don't know how it was deleted)
4.) Go to Local History- where it should've had the most recent version. Instead I saw that it only had the version from when I made updates the very first day, so therefore all of my work besides what I did that first day was completely gone
VS Code version: Code 1.79.2 (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:57:04.379Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Remote OS version: Linux x64 5.14.21-150400.24.63-default
Extensions (13)

Extension
Author (truncated)
Version

jupyter-keymap
ms-
1.1.2

remote-containers
ms-
0.299.0

remote-ssh
ms-
0.102.0

remote-ssh-edit
ms-
0.86.0

remote-wsl
ms-
0.79.5

remote-explorer
ms-
0.4.0

python
ms-
2023.12.0

vscode-pylance
ms-
2023.7.20

jupyter
ms-
2023.5.1101742258

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

(1 theme extensions excluded)

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.80.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

@bpasero any idea?

Please diagnose the root cause of the issue by running the command F1 > Help: Troubleshoot Issue and following the instructions. Once you have done that, please update the issue with the results.
Happy Coding!

Curious about the result from https://github.com/microsoft/vscode/issues/187800#issuecomment-1645002304
Sorry to hear you lost data, that should not happen ideally. What is special about connecting to a remote such as SSH is that we store the local history in the file system of the remote and not locally where you connect from.
I just did an experiment to see if I could reproduce with a slightly different setup (using a GitHub Codespace) and was able to see all my entries even after I had deleted the parent folder.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Update docs for Release 4 #no-public-changes

In this PR, we add some documents for Release 4. We are about to ship release 4 with nice, clean and more understandable animation APIs 💪

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          1 Warning

:warning:
Consider adding / updating the demo ap.

  Generated by :no_entry_sign: Danger

Hey guys, @tbaranes @SD10 I have updated the docs. Please have a second look when you have time, thanks.

@tbaranes Have added Other breaking changes in IBAnimatable 4.0 Migration Guide.md. Can we merge this PR now? I think we can start the release process after that.

Ready to go, good job on this @JakeLin 👍


Point MAD at /data/reaggregate endpoint, since that's how we have it set up in prod.

I don't think that's the common setup for other folks using the stack. Perhaps, it would be better to provide a separate config file (or modify the base file) with a command line switch between them?


Build a stable build to ensure stable build is green @bpasero

not sure what happened here... please ignore...