Set tomcat_group in service_sysv_init.rb, for issue 275

Description
Sets TOMCAT_GROUP in /etc/init.d/tomcat_[instance] for SysV init scripts
Issues Resolved
275
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

This is in the category of obvioius fix: Bug fixes that change default return values or error codes stored in constants
Specifically it simply duplicates the default value code from tomcat_user to tomcat_group.
Please can this be merged and a new version pushed?

works for me. Thanks

Released in 2.3.4

Appreciate the prompt response.


(maint) Relax fqdn_rand regex

Previously, we required fqdn_rand(100000) to return a 5 digit number,
but it can return any number greater than or equal to 0 and less than
100000.  So relax the regex to require between 1 and 5 digits.
Also the regsubst function was defining a regex in a single quoted
string which is written to a manifest as:
regsubst("Cersei","Cer(\w)ei","Daenery\1")

But \w in a double quoted string is an invalid escape sequence,
causing puppet to output a warning, but somehow successfully apply
the regex:
Warning: Unrecognised escape sequence '\w'

This commit escapes the backslash so that the manifest contains:
regsubst("Cersei","Cer(\\w)ei","Daenery\\1")

This eliminates the warning, while continuing to verify that the
regsubst function returns Daenerys.

@er0ck Here's the PR for that issue. It should also resolve the issues in today's 3.x acceptance run https://jenkins.puppetlabs.com/view/legacy_Puppet/view/3.x/job/legacy_puppet_packaging-callback_3.x/9/

(facepalm) 1..5 of course.


winutil does not work at all

Describe the bug
When I start the downloaded winutil.exe with administrator rights absolut nothing appears on my screen and winutil does not work on my Win11 (24H2) PC system
Steps to reproduce

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

Expected behavior

Screenshots

Additional context

Issue validation

[ ] I checked for duplicate issues.
[ ] I checked for already existing discussions.
[ ] I checked for an already existing pull request addressing the issue.

Is the .ps1 working?
Run irm "https://christitus.com/win" | iex as admin in powershell (not cmd)


Mysql multi-table deletes is not supported

The following query is not supported -
DELETE t1, t2 FROM t1 INNER JOIN t2 INNER JOIN t3
WHERE t1.id=t2.id AND t2.id=t3.id;
got - ParserError("Expected FROM, found: t1")
is this query are going to be supported near future ?
thanks.

Hi @AviRaboah as far as I know, there are no active plans to implement this feature. However, we welcome contributions and would gladly collaborate with you if you'd like to take a stab at implementing it.

Hi @AviRaboah, yes that's intended — the workflow to use is a pull request which allows the maintainers of this repo to review your code before it's merged in.
You usually do this by forking the repo, pushing a branch, and then submitting a pull request. Let us know if you have any questions.

That’s exactly what I did bot for some reason got status code 403
On Tue, 18 Apr 2023 at 19:50 Ankur Goyal @.***> wrote:

Hi @AviRaboah https://github.com/AviRaboah, yes that's intended — the
workflow to use is a pull request
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests
which allows the maintainers of this repo to review your code before it's
merged in.
You usually do this by forking the repo, pushing a branch, and then
submitting a pull request. Let us know if you have any questions.
—
Reply to this email directly, view it on GitHub
https://github.com/sqlparser-rs/sqlparser-rs/issues/854#issuecomment-1513494516,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV4SR6FKGIK7SN6UUZFQVKTXB3A63ANCNFSM6AAAAAAXCJRNNE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Same key for both choices for the function SetUserShellFolderLocation -Custom

Select a folder for the "Music" folder
[S] Select  [S] Skip  [?] Help (default is "S"):
leaving it empty and pressing enter works in skipping the function, typing S causes it open the file dialog. different keys should be chosen

Agree. I'll fix it in 5.10.7. thanks


What's the license?

There's no mention of license or usage permissions in the README or in the class file.

MIT. Just added to the repo.


[review] Remove top QuantEcon org header and replace with components

In a drive towards generating as good a reading resource as possible we need to look at:

removing the top level quantecon organisation header bar
Replace necessary components into theme header bar or add a prominent link to the org site

@mmcky I reckon this is solved now?


Reduce "magic" and mandatory JS

MDL does a lot of work for the user, sometimes too much so. We should opt for boilerplate over doing to much in the background:

MDL shouldn't add or remove elements to the DOM when upgrading components, particularly if the new elements are created around the user-specified ones (see e.g. mdl-layout__container). This complicates debugging and customisation, by making the DOM different from the markup the user provided.
All markup needed for the component to work should be provided by the user. If something is not present, the component should continue working if possible, or throw an explicit error if not.
Interaction between components or conceptually separate elements within a component should be explicitly declared by the user (e.g. using a data-mdl-for attribute). This allows for users to override click handlers, for example, while still retaining style and positioning.
Adding or removing CSS classes to existing elements from JS is ok, and should be the main interaction between CSS and JS.

+1 for this. Currently this library is proving very problematic playing nicely with our web components and frameworks based on diffing, since:

they all have attachedCallback / detachedCallback behavior that the MDL triggers when we don't want it to / didn't choose to detach/attach our element;
not being able to upgrade elements unless they are attached (see issue here means that elements can't be upgraded in advance of returning them from a view, and an actual DOM diff (using something like the morphdom library) will then strip the elements of its upgrade since the new DOM element is not being represented as a MDL element; and
similarly using any declarative views via ES6 template strings (one way model -> state -> view ala choo library) is impossible unless putting <my-material-component onload=${(e) => componentHandler.upgradeElement(e.target) } /> onto every element in the view function.

Hope that helps, thanks!

+1 This is causing all sorts of issues when integrating to other frameworks - can we at least add a flag to turn off the need to add to another element?

Thanks, everyone! Changing this behaviour in MDL without breaking backwards compatibility might be hard; it would involve going through every component and adding the flag that @jsguy mentioned, as well as removing any assumptions from the component JS that the expected DOM structure will definitely be there, by adding checks to everything.
Unfortunately, we don't have the resources to take on this challenge in MDL, as we've focused our efforts on MDC-Web (previously known as MDL v2) going forward. This problem isn't present in MDC-Web, as it doesn't do DOM manipulation of the type that's causing problems here.
If anyone would still like to tackle this task in MDL itself, I'm happy to review PRs, merge it in, and release a new version, as long as it doesn't break backwards compatibility.

@sgomes Thanks for the update.

@sgomes I'd like to discuss what you mentioned via email.

Unfortunately, we don't have the resources to take on this challenge in MDL, as we've focused our efforts on MDC-Web (previously known as MDL v2) going forward. This problem isn't present in MDC-Web, as it doesn't do DOM manipulation of the type that's causing problems here.

If anyone would still like to tackle this task in MDL itself, I'm happy to review PRs, merge it in, and release a new version, as long as it doesn't break backwards compatibility.

Thanks a lot!

@sgomes I'd like to discuss what you mentioned via email.

I assume you're interested in taking up the challenge, then? :) Thank you! You can reach out to me on   google.com


Package docker image as platform specific executable

Kind of related to #172 , it would likely make things easier for people if we could package the software directly as executables (e.g. on Windows, as an exe file) --- I'm not sure exactly how to do this, and perhaps it makes more sense if/when we have a GUI front end.   But there is some discussion on Stack Overflow here, and I thought i'd create this issue as a placeholder for the idea/intent.  I think this would be a nice to have, once all core functionality (e.g. #170 ), and possibly a GUI front end to facilitate configuration/running, is implemented.

While having our software as an executable would be interesting, it seems Docker isn't the approach to achieve this.  For now, while we are using Docker, I'll close this issue as it will require heavy re-thinking of the whole architecture which isn't going to happen any time soon.


[Editor] Add ability to 'Show source file in Explorer' for an asset

Description

Add new right-click Asset context menu to 'Show source file in Explorer'
If the file no longer exists, then try to browse to the folder it belonged to.
If the folder no longer exists, then show an error message.
Menu is always disabled for Assets that don't have MainSource set.
Motivation and Context

I want to browse to the actual file that the asset is pointing to because I want to do something with it.
Types of changes

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

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

LGTM


feat(rbac): Audit log rbac backend

What does this pull request do
Implement audit logging for RBAC backend
Referenced issue:
https://issues.redhat.com/browse/RHIDP-2004

@PatAKnight I created alternative pr with your proposals: https://github.com/janus-idp/backstage-plugins/pull/1726

To fix the failing check, change your title from feat(rbac): Audit log rbac backend to feat(rbac): audit log rbac backend. Also, you have your commit set as a fix instead of a feat.

Closed on favor: https://github.com/janus-idp/backstage-plugins/pull/1726

schultzp2020, take a look, please https://github.com/janus-idp/backstage-plugins/pull/1726


add support to enflame DTO

Track tasks to add support to enflame DTU
Copied from #5138

[ ] deploy script: pre checks / installation items for each machine model
[ ] exporter: additional exporters for machine models besides nvidia gpu / cpu machine.
[ ] rest-server: When generating pod spec, rest-server should be aware of environmental variables used by other device in addition to NVIDIA_VISIBLE_DEVICES and PAI_AMD_VISIBLE_DEVICES.
[ ] scheduler: health report for computing device. This is not mandatory since node-level health check is provided by k8s already.

Hi, for specific work items, please refer to the following:
In Jan 2021 Release

[ ] add dtu device plugin in device plugin service

https://github.com/microsoft/pai/blob/86c73e32f8f2c8b2b651bd71a2855ff116c27c83/src/device-plugin/deploy/start.sh.template#L47-L49
https://github.com/microsoft/pai/blob/86c73e32f8f2c8b2b651bd71a2855ff116c27c83/src/device-plugin/deploy/delete.sh.template#L42-L43

[ ] add dtu device check in installation script

Please write a ansible role enflame.com_dtu and place it in https://github.com/microsoft/pai/tree/86c73e32f8f2c8b2b651bd71a2855ff116c27c83/contrib/kubespray/roles/requirement/computing-devices
Please reference it in https://github.com/microsoft/pai/blob/86c73e32f8f2c8b2b651bd71a2855ff116c27c83/contrib/kubespray/environment-check.yml#L55-L70
You can see the check for nvidia gpu for reference in https://github.com/microsoft/pai/tree/86c73e32f8f2c8b2b651bd71a2855ff116c27c83/contrib/kubespray/roles/requirement/computing-devices/nvidia.com_gpu

After Jan 2021 release, we expect openpai can support installation, scheduling and job submission with enflame.com/dtu.
In Future Releases

[ ] Metrics reporting: add exporter for dtu
[ ] webportal


Use babel in the build process.

This uses babel as a webpack loader to preprocess source files.  It uses babel-preset-env to decide what browsers to target, using browsers: ['defaults'] as the criterion.  Perhaps we want to explicitly target a set of browsers.
This resolves #418.

This is probably a good point to introduce a general set of polyfills to continue to support the oldest browsers we want to handle.  Specifically, babel-preset-env by itself will ensure language features, but not standard library features (such as Promise support in IE 11).  To handle that, we can use babel-polyfill and karma-phantomjs-shim.  The babel-preset-env options can then be used to remove undesired browsers so the built geo.js package doesn't become needless large.

There is something amiss with how coverage is being generated.  I'm looking in to it.

We use babel-plugin-istanbul to get coverage working with babel.

Using babel-plugin-istanbul or switching from karma-coverage to karma-coverage-istanbul-reporter both sort of work but have issues.  Specifically, the emitted coverage seems to reference columns that only exist in the transpile code.

It turns out my issue with coverage was caused by using Babel 7 and, switching back to Babel 6, on where babel-polyfills was required.  I've switched to Babel 6 and only added the polyfills to the non-lean build (and also require them in testing).

@zachmullen Can you review this again, please.


Support explicit max fetch time #52.

Disclaimer - no unit test added...

http-fetcher has been updated to support a fetch duration timeout. You will need to install the latest version locally in order to make use of this support.
The fetcher builder now sets up the fetch duration timeout in the configure method.


Github does not open while the filter is active.

I don't know the source of the problem. :/
I would be happy if anyone with an idea wrote an answer.

Could you please read some details from your AdGuard's Query Log?
(hover question mark icon in Response column)
Are you using any other lists?

Could you please read some details from your AdGuard's Query Log?
(hover question mark icon in Response column)
Are you using any other lists?

It's not just the Github page.
It doesn't open other pages either.
I just use this filter.

It's not just the Github page.
It doesn't open other pages either.
I just use this filter.

Okay, I can't replicate that issue on my end.
I suggest that you:

Check your AdGuard settings for misconfiguration
Check that you've added the allowlist too (https://github.com/hl2guide/Filterlist-for-AdGuard#allowlist-)
Try restarting your PC, if needed

Okay, I can't replicate that issue on my end.
I suggest that you:

Check your AdGuard settings for misconfiguration
Check that you've added the allowlist too (https://github.com/hl2guide/Filterlist-for-AdGuard#allowlist-)
Try restarting your PC, if needed


Skip song if it has been made unavailable from youtube

Pausing and showing an error disrupts the experience, and causes the user to take unecessary actions to continue. If a song has been removed or made unavailable from YouTube, there's not much the user can do about that.
Suggestion: When coming to a song that is unavailable, skip to the next song. If user needs to be informed, give information inline in the playlist regarding the availability of the song.

Especially useful when playing a game, or doing something afk while listening to the music. As it is now, the user has to manually skip to next song and start playback.

Closing, duplicate of #380


[Optimism/KEVM] ByteArray concat simplification breaks back-end

In the context of the Optimism engagement, the following simplification
rule
  { BA1:ByteArray #Equals BA2:ByteArray } =>
    { concatHead(BA1):ByteArray #Equals concatHead(BA2):ByteArray } #And
    { concatTail(BA1):ByteArray #Equals concatTail(BA2):ByteArray }
  requires
    #sizeByteArray(concatHead(BA1)) =/=K #sizeByteArray(BA1)
    orBool #sizeByteArray(concatHead(BA2)) =/=K #sizeByteArray(BA2)
  [simplification]

breaks the backend with the message
    kore-exec: [5500634] Error (ErrorException):
    Could not make sort
        SortBool{}
    match sort
        SortBytes{}
    This is a program bug!

The two functions that it uses, concatHead and concatTail, operate on KEVM ByteArrays and are defined as follows:
                         // Leftmost operand in a concat
    syntax ByteArray ::= concatHead(ByteArray) [function, total]
                         // Rest of concat
                       | concatTail(ByteArray) [function, total]

    rule concatHead(BA1 ++ _) => concatHead(BA1) [simplification(40)]
    rule concatHead(BA)       => BA              [simplification    ]

    rule concatTail(BA1 ++ BA2) => concatTail(BA1) ++ BA2 [simplification(40)]
    rule concatTail(_)          => .ByteArray             [simplification    ]

The problem should be reproducible by checking out the foundry-tests-calldata-petar branch of the Optimism audit repo, uncommenting the above simplification in the lemma file and then executing:
cd optimism_foundry
forge b
./run-prove-lemmas.sh

@PetarMax please also generate a bug report and attach here.

Yes, here it is.
kore-exec.tar.gz


Added validation for repo schema version.

cc @jsancio

Looks great! Merging

why push this into a non-master branch?

@mgummelt master has been replaced by version-1.x as the default branch.  The intention behind this is that anything in the version-1.x branch is compatible with the version 1 schemas.
When we move to version 2 schemas in the future those will be targeted at different branches. This is to allow the cli to validate the repo's version so that it can ensure that it will be able to perform the actions it knows about.

yea, my mistake for not reading the eng email.  thanks


Add config option to silence "Supermaven ... is running"

Hey,
I would appreciate it if there was an option to silence the Supermaven Pro is running message. It's just noisy and doesn't really provide any value for me.
Cheers!

Fixed by https://github.com/supermaven-inc/supermaven-nvim/pull/21

@super-jacob As far as I can tell #21 only fixes https://github.com/supermaven-inc/supermaven-nvim/issues/3.
This issue is asking for a config option to completely silence the "Supermaven Pro is running" messages. I try to keep my setup as minimal as possible, and the message just adds noise while providing little value.

I think that the best way to approach this would be like other plugins does this kind of things.
Create a logger that if the config passes debug = true, logs out to the neovim cmdline, otherwise logs out to a file on the plugin folder like ./logs/supermaven-nvim.log.
Then we use a function like this for example:
-- logger file
function Logger:log(level, debug, msg)
  if not debug then
  -- write to file
  return
  end
  -- write to file
  -- vim.notify("[supermaven-nvim] " .. level .. ": msg, vim.log.levels[level], { title = "Supermaven" })
end

function Logger:log_fmt(level, debug, msg, ...)
  if not debug then
  -- write to file
  return
  end
  -- write to file
  -- vim.notify("[supermaven-nvim] " .. level .. ": msg, vim.log.levels[level], { title = "Supermaven" })
end

-- print("Supermaven " .. tier .. " is running")
Logger.log_fmt("INFO", debug, "Supermaven %s is running", tier)

PLUGINS EXAMPLES

none-ls.nvim

I don't know anymore from the top of my head 😥

@sQVe I added the option and the information on how to use, making use of a logging system.


New Theme for YunoHost

YunoHost is a tool for setting up self-hosted services on a server or Raspberry Pi, but it doesn't have a Dracula theme yet.
This is a theme for the YunoHost user portal, using the Dracula colors.  It does not (yet) apply to the administrator console or any applications installed within YunoHost.
This theme is available in the following GitHub repository:
https://github.com/jkoby/dracula-yunohost

Hey, very cool! ✨
I just sent you an invitation to join the organization.
I also cloned your repository for Dracula's organization, so you can go ahead and delete or archive yours:

Your repository.
New repository.

Thanks for building this! 🎉


Property 'encrypt' is missing in type 'TreeNode'.

With rug-cli 0.21.3, this is what I'm getting now:
16:24 $ rug test
Resolving dependencies for atomist-rugs:travis-editors:0.9.0 ← local completed
Invoking TypeScript Compiler on ts script sources
Processing script sources ⠙ OpenJDK 64-Bit Server VM warning: You have loaded library /home/sylvain/libj2v8_linux_x86_64.so which might have disabled stack guard. The VM will try to fix the stack guard now.
It's highly recommended that you fix the library with 'execstack -c <libfile>', or link it with '-z noexecstack'.
Processing script sources failed

.atomist/editors/EnableTravisForRugArchiveTS.ts(120,17): error TS2322: Type 'TreeNode' is not assignable to type 'Travis'.
            let t: Travis = eng.scalar(project, pe);
----------------^
  Property 'encrypt' is missing in type 'TreeNode'.

This is all working now.


Selection checkboxes for multiple/single in datalist are disabled

When you set the selection mode to multiple/single, when you render the form, the checkboxes are disabled
Steps to reproduce the behavior:

Go to 'https://function-adminportal-test.shesha.dev/dynamic/Boxfusion.SheshaFunctionalTests.Common/news-details?id=b7f0594e-9a6b-478d-bf35-e71962070507'
Scroll down to the data list within panel4
Click the checkbox of any of the records
Actual Results: Unable to select the records
Expected behavior
The checkbox should be enabled to allow selection

Hi @AlexStepantsov, the single selection mode does show the tick inside the checked record, however when you use the multiple selection mode it does not show ticks inside the checked records only on 'select All' box. See below screenshot


Integration Test - Release - 2021/12/31 02:34:19

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#please-close

GitHub user PRMergerTest2 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#unassign PRMergerTest1

#unassign NonAllowedUser


refactor(consensus): Context is passed as a param instead of being held as a field by SHC

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2239
#2238  👈
#2237
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @matan-starkware and the rest of your teammates on  Graphite


dbt-core version pinning

Can dbt-trino be unpinned from the specific dbt-core patch version?
Per dbt's docs, this seems in line with the way they are managing dbt-core versioning:
https://docs.getdbt.com/docs/core-versions#upgrading-to-new-patch-versions
In my particular use case I want to take advantage of dbt-core's 1.0.4 patch.

Resolved by https://github.com/starburstdata/dbt-trino/commit/6822010551b647d92e514ea60d21b3023af888ed


Added extension to UIButton to  support remote images

I am using this library in some personals projects and I noticed about there is not support to set a remote image as a background or image, I ended with this solution for iOS that can help with the #13 . I hope it fits the requirements and guidelines of AlamofireImage.
Regards.

Okay this looks like an awesome start @jorjuela33. I'll spend some time here in the next few days digging through this in detail, but it looks really solid at first glance. The one thing that jumps out at me right now is that the test suite is failing on watchOS. I would guess that there needs to be some checks here for handling the different OSs properly. My guess here is that you probably only want these files included in the iOS framework and test target. Not sure about tvOS...so we should look into that as well.

@cnoon I didn't check for the watchOS, I just tested for iOS. But If I can help with something else please let me know. I will be glad.
Regards.

Hi @jorjuela33...thank you so much for all your hard work here. I squashed your two commits into a single one in 5722c336. Then I refactored a few things in 010750da (diff looks much worse than it really was). Then I added accept header tests in 3000c6ff and cancellation tests in 09a4f705. That last commit was pretty important since the previous implementation suffered from the same cancellation bug as the UIImageView extension did. Thankfully it's all fixed up and looking really solid.
Thanks again for putting this together. I'll hopefully get everything released tomorrow. 🍻

I would release tonight, but I need to put together a new section in the README for the UIButton extension.

Nice! thanks so much to consider this code and I hope to continue contributing
Cheers


Disable Confirmation dialog on delete

Hi @hirenkukadiya @Zicrael,
is there a way to disable the confirmation alert dialog box on delete
a config to enable/disable confirmation alert dialog on delete item

Hi, any solution?


st2-register-content job should clean up packs/actions/etc that no longer exist

Right now, the functionality of the HA-implementation makes it so that custom packs are installed via Docker image. This works fine and is in line with expected results.
What doesn't work as expected is the removal of those packs and actions. Removing a pack/action/rule/etc from the docker container leaves it showing in the list even though it will not function when called. This is because no cleanup step is performed to remove those packs.
I would propose that an expected behavior would be that any functionality not defined in values.yaml or else by the docker container would be reasonable to assume to be overridden. This is especially confirmed by the warning message displayed when connecting to the st2client pod.

This logic is not stackstorm-ha Helm chart responsibility, but how StackStorm pack management works.
Please submit a report/feature request in https://github.com/stackstorm/st2 instead.
Thanks!

After a quick search there looks like a lot of issues in st2 reported already:

https://github.com/StackStorm/st2/issues/3143
https://github.com/StackStorm/st2/issues/3016
https://github.com/StackStorm/st2/issues/3143
https://github.com/StackStorm/st2/issues/924
https://github.com/StackStorm/st2/issues/3313

Closing in favor of those.

Hi @armab I agree with what you mentioned here but even if we modify existing action pack stackstorm-ha isn't reflecting it. I had to delete the instance and create it again in order to reflect changes. That's not the case with st2.


Override workspace root with INSTA_WORKSPACE_ROOT

Fixes #180

I tested this via patch section in Cargo.toml, it seems to work.
I was unsure where I'd add a test for this. I suppose it has to be added to docs somewhere too?

Thanks. I merged this manually since the code changed a bit.

Thank you!


Added podspec

Added a podspec that passes lint. I think it's better if you push it to the trunk. If you think it is a god idea to be on cocoapods. :)

Hi @laptobbe, thanks for the pull request!
Podspecs don‘t need to be added to the repository, and since I don’t use CocoaPods I won’t be able to maintain it. I’d suggest that instead you push it to the specs repo per their documentation.
I’m going to close this out now, but thank you again for the effort!

If there's no objection from y'all I'm happy to manage this. We need Box to be in CocoaPods in order to have it as a dependency for Argo, so I have an interest in making sure it's available.

:ok_hand:

Done. All tagged versions of Box are now available through CocoaPods. Thanks for putting the podspec together, @laptobbe!


Make whole post preview clickable

Right now only clicking on the article headline or the preview picture opens the article. I'd suggest making the whole div of the article preview clickable just to make opening an article easier.

Will be fixed in new release/update

done


Update functions-bindings-service-bus-trigger.md

Added autocomplete setting to the function.json

@craigshoemaker, can you take a look at this?

@craigshoemaker can you take a look at this PR?

Approved with the current state.

Approved with the current state.

#sign-off

@apranaseth: I'm sorry - only the author of this article, @craigshoemaker, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Uploaded Internal APIM and ASE Scenario

OPS Build status updates of commit 8d716b4:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/example-scenario/apps/internal-apim-ase-scenario.md
:warning:Warning
View
Details

docs/example-scenario/apps/media/architecture-internal-apim-ase-scenario.jpg
:white_check_mark:Succeeded
View

docs/example-scenario/apps/internal-apim-ase-scenario.md

[Warning] Author ssarwa cannot be recognized in GitHub.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 89b4cdd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/example-scenario/apps/internal-apim-ase-scenario.md
:white_check_mark:Succeeded
View

docs/example-scenario/apps/media/architecture-internal-apim-ase-scenario.jpg
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 37b3270:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/example-scenario/apps/internal-apim-ase-scenario.md
:white_check_mark:Succeeded
View

docs/example-scenario/apps/media/architecture-internal-apim-ase-scenario.jpg
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

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

File
Status
Preview URL
Details

docs/example-scenario/apps/internal-apim-ase-scenario.md
:white_check_mark:Succeeded
View

docs/example-scenario/apps/media/architecture-internal-apim-ase-scenario.jpg
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 1e826bd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/example-scenario/apps/internal-apim-ase-scenario.md
:white_check_mark:Succeeded
View

docs/example-scenario/apps/media/architecture-internal-apim-ase-scenario.jpg
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

I think it's a great first start. I'd like to see a lot more detail around the implementation, how data flows through the scenario, and what each component is used for.
Any deployment specific information that exists in a GitHub repository should be removed from the scenario and just described in the repo.

I think it's a great first start. I'd like to see a lot more detail around the implementation, how data flows through the scenario, and what each component is used for.
Any deployment specific information that exists in a GitHub repository should be removed from the scenario and just described in the repo.

Updated the document with comments and uploaded a newer architecture diagram with sequencing

Docs Build status updates of commit 1e826bd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/example-scenario/apps/internal-apim-ase-scenario.md
:white_check_mark:Succeeded
View

docs/example-scenario/apps/media/architecture-internal-apim-ase-scenario.jpg
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions


Not showing any images (Not Working)

Following all of the guidelines given, and result is blank page. Its to hard to found the mistake. Because non of error given.
Please add a example.
Here the Code:
    <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.3/jquery.min.js"></script>
<script src="vegas/vegas.js"></script>
<script>
$(function() {
	$elem = $('.wrap');
	$elem.vegas({
		preload:true,
    	 	transitionDuration:4000,
		delay:10000,
		slides: [
			{ src: 'img/1.jpg' },
			{ src: 'img/2.jpg' },
			{ src: 'img/3.jpg' },
			{ src: 'img/4.jpg' },
			{ src: 'img/5.jpg' }
		]
	});
}
</script>

Anybody here to mark the mistake.

Having issues too..
I used the code from the github Readme
  $('body').vegas slides: [
    { src: '/images/1.jpg' }
    { src: '/images/2.jpg' }
    { src: '/images/3.jpg' }
  ]

Nothing's showing up

Hello Guys,
It works well for me. Vegas website uses the latest version. This not a Vegas bug.
Check your code or read the jQuery documentation. Vegas works like every other jQuery plugins.
@webtuts4u
Your code seens right.
There is some examples in the documentation and a demo on http://vegas.jaysalvat.com
@slavajacobson
This is not the code from the Readme.

Thanks @jaysalvat
Its to hard to get the right markup for js. I found my solution. If its possible for you, please add the small basic demo in the repository.
Thanks again.
:smile:


Remove transition

is there a way to remove the transition? i'd like to display 36 pictures of a car. with the drag of the pictures i could display them as a 360° image, but as long as there is any transition between the pictures, it's not gonna work. any idea?

Ok so what you mean is that you don't want the pictures to snap into place once you end the drag, is that correct?

My idea was zero animation between each picture. fortunately i got another plugin to work from mediaman  that solved my problem. this is no longer an issue for me. Thanks for your reply.

Ok got you. Turning the animation off would be very easy then. You could use this CSS:
doe-gallery::ng-deep doe-viewer ul {
  transition: none;
}

i wasn't aware of this easy solution, thank you very much!


Improve docker CI: push vX.Y tag (without patch) to DockerHub (#2507)

Following https://github.com/meilisearch/meilisearch/pull/2507
Fixes #2497

I tested to push the following git tags on this branch

v0.1992.345 -> the docker tags v0.1992.345 and v0.1992 has been pushed
v0.2022.1 -> the docker tags v0.2022.1 and v0.2022 has been pushed
v0.2023.0rc1 -> only the docker tag v0.2023.0rc1 has been pushed
v0.2029.0-testcloud.0 -> only the docker tag v0.2029.0-testcloud.0 has been pushed

Everything worked as expected ✅

bors merge

bors merge


feat: Simplified Google Sheets queries

Description
This PR migrates Google Sheets into the UQI format. Moreover, it introduces multiple functions to make the interface for UQI types cleaner.
Fixes #4559
Server side:
Fixes #11186
Fixes #14312
Fixes #14349
Fixes #14359
Fixes #14445
Fixes #14452
Fixes #14453
Fixes #14897
Client side:
Fixes #7003
Fixes #12223
Fixes #12332
Fixes #12349
Fixes #13538
Fixes #14030
Fixes #14244
Fixes #14276
Fixes #14278
Fixes #14279
Fixes #14280
Fixes #14281
Fixes #14314
Fixes #14476
Fixes #14479
Fixes #14480
Fixes #14560
Fixes #14600
Fixes #14669
Fixes #14765
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?

JUnit tests
Cypress tests
Jest tests
Manual QA on DP

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=e2fed56

/ok-to-test sha=aade330

/ok-to-test sha=aade330

/ok-to-test sha=1dcbf7c

/ok-to-test sha=c4a5a4a

/ok-to-test sha=76baa70

/ok-to-test sha=37961b6


Use "Connect to mainnet" text for Mint NFT button

Currently just does not display a button when another network is selected, which confuses users. The Stake/Unstake button is a good reference.

Fixed by #180.


Fix possible NPE.

Result:
Fixes #171

Codecov Report

Merging #173 into 5.3 will increase coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##                5.3     #173      +/-   ##
============================================
+ Coverage     63.54%   63.55%   +0.01%
+ Complexity      582      581       -1
============================================
  Files           263      263
  Lines         11250    11248       -2
  Branches       1876     1875       -1
============================================
  Hits           7149     7149
- Misses         3127     3128       +1
+ Partials        974      971       -3

Impacted Files
Coverage Δ
Complexity Δ

...va/com/alipay/sofa/rpc/client/AbstractCluster.java
64.42% <0%> (-0.24%)
0 <0> (ø)

...m/alipay/sofa/rpc/module/FaultToleranceModule.java
66.66% <0%> (-26.67%)
4% <0%> (-1%)

...ay/sofa/rpc/client/AllConnectConnectionHolder.java
52.51% <0%> (-0.28%)
0% <0%> (ø)

...com/alipay/sofa/rpc/context/RpcRuntimeContext.java
80.24% <0%> (+1.23%)
0% <0%> (ø)
:arrow_down:

...lipay/sofa/rpc/codec/antpb/ProtobufSerializer.java
50.72% <0%> (+2.89%)
9% <0%> (ø)
:arrow_down:

...pay/sofa/rpc/transport/ClientTransportFactory.java
65.15% <0%> (+6.06%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


ODE Areas will fail to display if the frame number is reset to 0

Since each Area can be owned by multiple triggers, all calling the same Area to display meta, the Area has logic to ensure that the meta is only added once to each frame for each source.  The below code in OdeArea::AddMeta() is using a < evaluation which clearly will be an issue if the frame number is reset to 0, i.e. when a camera stream is lost and regained or when a file source is restarted.
        // If the last frame number for the reported source is less than the current frame
        if (m_frameNumPerSource[pFrameMeta->source_id] < pFrameMeta->frame_num)
        {
            // Update the frame number so we only add the rectangle once
            m_frameNumPerSource[pFrameMeta->source_id] = pFrameMeta->frame_num;

            m_pDisplayType->AddMeta(pDisplayMeta, pFrameMeta);
        }

Merged into v0.20.alpha


docs(overview): add json logging example

Description
Adds an example of json logging to the docs
Motivation and Context
Json logging is a modern form of logging, which is used and can be consumed by many different tools, such as ELK and other logging solutions
https://github.com/oauth2-proxy/oauth2-proxy/issues/90
How Has This Been Tested?
Deployed to my own server with this example configuration

@max-wittig This is awesome and I hope you PR it back to oauth2-proxy to be merged.

Oh I set the wrong target.


Support for JWT with no "kid" in the Header

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
I am working with an old service that currently does not sign JWTs with a "kid" in the header. Cerbos probably correctly rejects this b/c it can't find the key in the keySet to validate the signature.
I'm wondering if you'd be open to allowing Cerbos to handle this special case by just using the single key in the single keySet it was configure with instead of relying on the kid header.
Hacky for sure, but would make my life easier in the interim :)
What would the ideal solution look like to you?
If JWT has no kid, and I only have one keyset with only one key, use that. Otherwise keep throwing the same error.
Anything else?
No response

Is it possible for your application (or a proxy) to extract the claims from the JWT sent by this legacy application and re-encode it in the correct format?
I can see your dilemma and would love to help here but there are few issues to consider here that would make it tricky to implement in Cerbos.

We delegate JWT validation to a third-party library and, from a quick glance, I don't think it supports this mode. We'll have to create a special case for this scenario and then maintain that for a long time (Hyrum's law applies) -- which is not quite sustainable.
Not quite sure whether this would weaken security or create an attack vector that we are not aware of. As a public project used by many other users, that's a big risk to take.

Totally understand your reasoning, makes sense. Thanks for your response :)


Support providing k8s native resource through --objects flag in kanctl

Change Overview
Support providing k8s native resource through --objects flag in kanctl command, while creating
actionset.
For ex
kanctl create actionset \
--action backup \
--namespace kanister \
--blueprint etcd-blueprint \
--objects /v1/pods/kube-system/etcd-minikube \
--profile kube-system/s3-profile-xhgl9

Per current implementation we are not able to do this. The validation fails.
Edit:
we have changed the format and now the core group's resources should be provided using the format
--objects v1/resource/namespace/name

for example
--object v1/pods/kube-system/etcd-minikube

And we dont support the format /v1/resource/namespace/name for the resources of core api group.
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[x] :bug: Bugfix
[ ] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Issues

#XXX

Test Plan
Provided on CR, pod and deployment through --objects flag while creating the actionset.

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

minor: does it work without leading / in object resource path
i.e --objects v1/pods/kube-system/etcd-minikube
`

Hey @PrasadG193
No it doesnt, it doesnt. We are eventually mapping gvr with this value, having this format will work in the cases when we have to actually provide group as well.

LG. I'll let @tdmanv to decide whether we want to support format like this --objects v1/pods/kube-system/etcd-minikube

@pavannd1 I thought the same but I didn't see any test file, I can add.

LGTM. Any test plan?

I am adding the test.

Thanks @pavannd1


rdmo exports xml elemts multiple

When exporting a rdmo domain via xml export individual database entries are being written multiple of what was a single database entry.
I expect this issues to be resolved, when the xml export has only a single xml element per database entry.
: RDMO v0.12
: python3
: local install sqlite3

A strong hint might be, that each descendant of the main attribute 'project' is exported the exact amount of times that corresponds to its position in the descendency , i.e. the main attribute is exported once, children of the main attribute are exported twice, children of the first children are exported thrice and so on.

Thanks for reporting, we will look into it soon.

Fixed in 01346bab82b34c945e2ea9704cdd7f90db4849d7.


regression test some glib builds against master branch

relates to https://github.com/Homebrew/homebrew-core/pull/131789

relates to https://github.com/Homebrew/homebrew-core/pull/131914
only walkmod failed, https://github.com/chenrui333/github-action-test/actions/runs/5072254550/jobs/9109704952?pr=148


T4 Text-Templates with IntelliSense

please work on IntelliSense.

Sure, I'll get right on it! :smile:
IntelliSense for T4 templates is a very large area, it would help if you could elaborate on what would be the most important in your usage of T4. Allow me to provide some background.
When I think of IntelliSense for T4 templates, I tend to separate the following two areas:

IntelliSense for the T4 syntax
IntelliSense for code blocks.

IntelliSense for the T4 syntax is implemented in the T4Toolbox itself, which includes a template scanner used for syntax colorization and a template parser used for quick info tooltips, outlining, error underlining in editor, error reporting in error list window and completion lists. While syntax colorization and error reporting for template display work OK, I think it's fair to say that support for template editing needs a lot of improvement. I suspect the template parser is too "strict" which prevents creation of completion lists when you are in the process of typing a line. For instance, when you stop typing <#@ , I'd expect to see a completion list names of the directives, however this list appears only when you have text caret in the middle of <#@ #>.
IntelliSense for the code blocks was simply never implemented. In Visual Studio 2013, Roslyn was not available and re-implementing C# and VB IntelliSense was not feasible. In Visual Studio 2015, Roslyn is in the box and all options are open. One of the options is projection buffers which are used by the Visual Studio HTML editor for Razor and WebForms code blocks. I think Web Essentials for Visual Studio 2015 takes advantage of this capability to implement IntelliSense for code snippets in markdown, which could be used as an example for code blocks in T4 templates.

One thought which crossed my mind could an all-or-nothing approach. As in, a T4 file is treated as purely T4, unless another known directive is supplied:
<#@ template language="C#" #>
<#@ output extension="cs" #>

This is to say, some T4 files may have a couple of headers and consume methods from imports, operating mostly over a C# template.
Other T4 files may be pure T4 files, all enclosed completely in <# #>.
Which is a more common scenario? If a file is mostly C#, would it not be more beneficial simply to enhance the standard C# syntax & highlighting buffer, and offer IntelliSense for <# #> blocks inside of the C# buffer? (Let's say, similar to classic VB/ASP inside of HTML, but named .asp)
This would be opposed to gearing for T4, and then trying to implement C# (or VB) syntax interpretation as an afterthought.
That was just one thought which crossed my mind, and I'll be looking into. (Confirm that Web Essentials does use IProjectionBuffer as a place to look at examples of implementation.)

Interesting, I didn't think about this before... So if we could extend the C# content type instead of defining a distinct text template content type we could let the C# editor extension take care of handling its own IntelliSense features while adding only the T4-specific ones. That would be awesome if it works.
I wonder if the C# editor supports projection buffers. I think that Web Essentials uses the built-in htmlx content type as a base for the markdown content type because the HTML editor has support for projection buffers and provides extensibility API allowing Web Extension to hook in the markdown IntelliSense components.

I don't know about the latter (if the C# editor supports projection buffers), and, I haven't checked, but... one thing which crossed my mind is F#/ (Whereas, I believe T4Toolbox is primarily geared for VB/C#, but T4 might also be supporting F#???)
I've never tested T4 + F#, but, the idea is very simple: such that T4 syntax highlighting becomes a natural extension of whatever the declared (supported or mapped) output language. (Which could naturally extend to .json, .js, .html, .whatever), or otherwise default to pure T4.txt.
at least that's how I think about things
/ End Thought

That would be awesome! If we detect the type of text being generated from the extension in the output directive and the language in the template directive, we could use projection buffers to provide built-in syntax colorization and IntelliSense in both text and code blocks of the T4 Templates.
I wonder if @madskristensen could help us understand how this was accomplished for code snippets in the markdown editor of Web Essentials for Visual Studio 2015.

How to access completion list? I cannot access it using Ctrl+Space.


[SUPPORT] Deltastreamer ignoring hoodie.datasource.write.precombine.field

I'm running Deltastreamer in AWS EMR Serverless and it seems that Deltastreamer is ignoring hoodie.datasource.write.precombine.field in my config file.
To Reproduce
Setup a source bucket of parquet files (Mine are from AWS DMS): s3://my-lakehouse/dms-output-raw/schema_1/table_1/
Add a properties file (s3://my-lakehouse/deltastreamer-config/deltastreamer.properties):
hoodie.schema.on.read.enable = true
hoodie.datasource.write.recordkey.field = origin_schema,id
hoodie.datasource.write.precombine.field = updated_at
hoodie.datasource.write.partitionpath.field = origin_schema
hoodie.datasource.write.keygenerator.class = org.apache.hudi.keygen.ComplexKeyGenerator
hoodie.deltastreamer.source.dfs.root=s3://my-lakehouse/dms-output-raw/schema_1/table_1/

Submit a job to EMR Serverless 6.10.0

Script Location: s3://my-lakehouse/deltastreamer-jar/hudi-utilities-slim-bundle_2.12-0.13.0.jar
Main Class: org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer
Script arguments

["--table-type","COPY_ON_WRITE","--target-base-path","s3://my-lakehouse/deltastreamer-out","--target-table","table1","--source-class","org.apache.hudi.utilities.sources.ParquetDFSSource","--props","s3://my-lakehouse/deltastreamer-config/deltastreamer.properties"]

Properties key 1: spark.serializer = org.apache.spark.serializer.KryoSerializer
Properties key 2: spark.jars = s3://ec-lakehouse-qa/deltastreamer-jar/hudi-spark3-bundle_2.12-0.13.0.jar

Everything else is default. I tried 0.12 and 0.13, but same result.
Expected behavior
Rows from source are upserted to destination.
Deltastreamer fails and when I look at /my-lakehouse/deltastreamer-out/mytable1/.hoodie/hoodie.properties I can see that hoodie.table.precombine.field=ts. That should be hoodie.table.precombine.field=updated_at because it's what's in the properties file, right?
Environment Description

Hudi version : 0.13

Spark version : 3.3.1

Hive version : Not sure for EMR Serverless 6.10.0

Hadoop version : Not sure for EMR Serverless 6.10.0

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : EMR Serverless 6.10.0

Stacktrace
Add the stacktrace of the error.
Job failed, please check complete logs in configured logging destination. ExitCode: 1. Last few exceptions: Caused by: org.apache.hudi.exception.HoodieException: ts(Part -ts) field not found in record. Acceptable fields were :[xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, created_at, updated_at, origin_schema, origin_table] Exception in thread "main" org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 2.0 failed 4 times, most recent failure: Lost task 0.3 in stage 2.0 (TID 5) ([2600:1f13:4df:c101:e7b0:42bd:e2f6:f474] executor 1): org.apache.hudi.exception.HoodieException: ts(Part -ts) field not found in record. Acceptable fields were :[xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, created_at, updated_at, origin_schema, origin_table] Caused by: org.apache.hudi.exception.HoodieException: ts(Part -ts) field not found in record. Acceptable fields were :[xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, created_at, updated_at, origin_schema, origin_table] org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 2.0 failed 4 times, most recent failure: Lost task 0.3 in stage 2.0 (TID 5) ([2600:1f13:4df:c101:e7b0:42bd:e2f6:f474] executor 1): org.apache.hudi.exception.HoodieException: ts(Part -ts) field not found in record. Acceptable fields were :[xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, xxx, created_at, updated_at, origin_schema, origin_table] 23/03/17 20:13:39 ERROR HoodieDeltaStreamer: Got error running delta sync once. Shutting down...

I tried this on emr-6.9.0 and got the same result.
The other settings from my properties file are being propagated over to the file in .hoodie like hoodie.table.partition.fields=origin_schema and hoodie.table.recordkey.fields=origin_schema,id.
It's just the hoodie.datasource.write.precombine.field = updated_at setting in my properties file ends up as hoodie.table.precombine.field=ts in the table properties file.

Also, no data is written to the table.

Also, also, the target table doesn't exist before the run.

That worked, thank you!


Wndows: issue with escaping auto-completed filenames after :!{cmd}

Create a file with the name file with spaces and a '.txt
Run gvim --clean
Type :read !type f and press Tab to auto-complete the filename
The command line becomes :read !type file\ with\ spaces\ and\ a\ \'.txt
Press Enter. "shell returned 1" message is displayed and an error emitted by the "type" command is put into the buffer.

Apparently this happens because "type" expects "file with spaces and a '.txt", not file\ with\ spaces\ and\ a\ \'.txt as the filename parameter (as perhaps any Windows command-line tool would). I tried searching through the docs for a solution, but didn't find anything, so maybe this should be considered a bug?
Vim: https://github.com/vim/vim-win32-installer/releases/download/v8.1.2168/gvim_8.1.2168_x86_signed.zip,
Windows 10 Home 1803.

I don't think this is a bug. Most vim commands expect an escaped filename, so that is what <Tab> does complete. Trying to make it complete depending on how the command would expect it is probably not worth the effort (it could still be wrong, because the user edits the command in front of it later)

Unfortunately with the current situation using <Tab> completion after :!{cmd} (at least on Windows sysyems, I'm not sure about other OSes) is not very helpful in case the filename has lots of spaces and other special chars, because for the command to work properly the user is required to manually remove all the backslashes and doublequote the filename..

If the filename has spaces, it is correct to escape it. Think of using :e and use tab to find the correct file. And changing tab to behave differently would be a backwards incompatible change.
What you can do instead is, use the command line window (e.g. press <c-f> in command line mode, see :h c_CTRL-F, edit the command there and use Ctrl-X Ctrl-F to complete the filepath there (which does not escape the path) and press enter to execute the command).
Perhaps it would be a good idea to make the file path completion of Ctrl-X Ctrl-F available in command line mode. Not sure what would be a good key for that, we are running out of ctrl keys for that. Looks like we still have c_CTRL-O, c_CTRL-S (that one is troublesome in terminals), c_CTRL-X and c_CTRL-Z (which is also troublesome). Perhaps using c_CTRL-X and then use whatever completion you would use in insert mode (e.g. CTRL-X CTRL-F would expand file paths literally)?

So you cannot make <Tab> work correctly after !?

So you cannot make  work correctly after !?

It depends on what you define as correct. It is highly dependent on whatever external command you run and how it expects its paths. It could expect spaces to be escaped or not, I don't think there is a general solution to this problem.

Perhaps using c_CTRL-X and then use whatever completion you would use in insert mode (e.g. CTRL-X CTRL-F would expand file paths literally)?

Sounds great to me, but of course I don't have a say in this.
At any rate, thank you very much for the command line window solution.


Extension is "leaking" kubectl processes after vscode closes

I'm using this extension as it's a dependency for Bridge to Kubernetes. It my current environment, whenever I close vscode, a kubeclt process is left running pointing to the workspace folder. From what I could gather, the command it's running is "proxy".
The expected behavior is to close any child processes when vscode is closed.

Environment:
Version:          Code - Insiders 1.63.0-insider (a7104f13ff61bcd9f4c827d8fe3c0d27bb479fd8, 2021-11-08T05:14:49.440Z)
OS Version:       Windows_NT x64 10.0.19044
Extension version: v1.3.3

Hi @bruno-brant , thanks for opening this. Can you please describe what you do exactly to replicate it? I was able to verify the problem you are mentioning (leaking kubectl processes) with the port-forward action but not by using Bridge to Kubernetes. It would be helpful to replicate your use case as well. Thank you!

Hey @lstocchi, thanks for the follow-up! I'm not exactly sure if the culprit is Bridge or if it's the Kubernetes extension itself, but  it seems to happen more frequently when I'm running Bridge. I'll try to reproduce it more exactly and update the issue.

This is still happening.  I have both the Kubernetes and the Bridge to Kubernetes extensions installed.  The "bridge" extension is disabled and I have no other K8s-related extensions installed.  I can open up any project with the Kubernetes extension install and a kubectl process will spawn.  When I close VsCode, the process stays every time.  The type of project doesn't seem to matter at all.  I discovered this because that uncontained kubectl process keeps my project directories lock and I can't delete old project etc from windows explorer.

That sounds frustrating - that directory locking thing is a colossal pain!  Could you try using Process Explorer (or similar tool) to find out the full command line - specifically the subcommand/arguments - of the leaked kubectl instance please?  That will help us to narrow down where the leaked process is coming from.  Thanks!

@itowlson Maybe we should merge https://github.com/vscode-kubernetes-tools/vscode-kubernetes-tools/pull/1001 even if it doesn't kill all kubectl processes. I'll try to see again why the observable doesn't work but that should be a good start.

@itowlson The complete subcommand/arguments on Windows is:
....\bin\kubectl.exe" proxy --port=0

And yes, not being able to delete the folders because of this is sometimes puzzling and definitely frustrating when it happens.
I have written the following Doskey macros (aliases) to kill all kubectl.exe processes (use kkc alias carefully as it may kill legitimate kubectl.exe processes):
kkc=taskkill /F /IM kubectl.exe
lkc=tasklist /v /nh /fi "IMAGENAME eq kubectl.exe"

This extension may help until this issue is fixed:
https://marketplace.visualstudio.com/items?itemName=sandipchitale.vscode-kubernetes-kubectl-ripper

@lstocchi looking over #1001, one would expect it to work. I proceeded to add "SIGKILL" signals to all the calls to .kill(), but alas I was still having issues. I eventually realized that kubectl proxy --port=0 is spawning a new process~/.kuberlr/windows-amd64/kubectl1.62.0.exe proxy--port=0. The first process does get killed as expected, but the second lingers. https://github.com/flavio/kuberlr seems to be a kubectl version manager, which is used by Rancher Desktop (which is what I use). Looking in Process Explorer, the second process doesn't seem to be attached to a tree at all, so it's not like killing the first process could even kill the child process.
I'm not sure if this is even within scope of this extension to handle (i.e. if there is some workaround), but I'm leaning on it's not. It is fairly frustrating, but I believe @sandipchitale's extension may solve my usecase

Actually, changing proxy.ts:26 to const dispose = () => { treeKill(proxyingProcess.childProcess.pid, 'SIGKILL'); }; using the https://www.npmjs.com/package/tree-kill does kill both processes. It seems to just be a wrapper around taskkill for windows to kill the whole tree: https://github.com/pkrumins/node-tree-kill/blob/master/index.js#L28-L49
@lstocchi would you be opposed to a PR that introduces this dependency (or separately, copies the contents of that module into this project to use) and replace the childProcess.kill() calls with treeKill(childProcess.pid)?

@mikeseese if it works i'm all in with this solution.
Thank you very much for your help!!


Provide the option to create (or update) template-registry.ts

I would like to see... 🙋♀🙋♂
In 0.4.1, the codemod adds the template registry to a component's backing class, because the implementation is simple.
In addons, it's easier for their consumers if the codemod can create (or update) the template-registry.ts file instead. Here is an example from ember-container-query:
/* src/template-registry.ts */
import type ContainerQueryComponent from './components/container-query.ts';
import type AspectRatioHelper from './helpers/aspect-ratio.ts';
import type HeightHelper from './helpers/height.ts';
import type WidthHelper from './helpers/width.ts';
import type ContainerQueryModifier from './modifiers/container-query.ts';

export default interface EmberContainerQueryRegistry {
  ContainerQuery: typeof ContainerQueryComponent;
  'aspect-ratio': typeof AspectRatioHelper;
  'container-query': typeof ContainerQueryModifier;
  height: typeof HeightHelper;
  width: typeof WidthHelper;
}

At the moment, end-developers of this codemod have to cut-paste the lines to update their template-registry.ts.
Why and how 💬
To know where template-registry.ts file should be created, we may want to (1) replace the --src option with a --type option, similarly to in ember-codemod-pod-to-octane or (2) assume that the file is always 1 folder-level up (e.g. from addon/components to addon/template-registry.ts, and from src/components to src/template-registry.ts).
At the moment, it's unclear how we will deal with calculating relative paths. There are 2 cases to consider: flat and nested component structure. It's also unclear whether we will need @codemod-utils/blueprints or @codemod-utils/ast-javascript (or maybe even plain node:fs) to generate the template-registry.ts file.

Will close this issue, as I didn't find time to work on it.


Feature proposal overfetch

Hi,
Congrats on the v2 release!!
I was thinking about one feature that might make sense in some situations.
Lets imagine that in one specific endpoint you don't need to fetch all the properties of one object.
What do think on having one additional property on the query model that allows you to define the properties of the model that you want to get or remove from the the response (more or less the behaviour of the 'select' on odata)?
Maybe using linq projections if some 'custom configuration' is defined (the same way you have on the custom mapping).
What do you think?

Hi,
Thank you for your comment.
This sounds like an interesting feature for a query library, but I think it involves a fair amount of development time, it might be the next thing I can work on, but due to time constraints I don't see this coming too soon.
There are some options you can try to achieve someting like this currently:

Manual projections: If you need a new field in the query you can create a class that inherit from QueryModel so you can add fields there and based on that field value perform manual projections in the query result.

Custom filters: As custom filters conditions are added at the end of other conditions you can add projection code there:

cfg.Property(c => c.Priority).CustomFilter((source, values, op) =>
{
    return source.Select(c => new Card { Id = c.Id, ColumnId = c.ColumnId, Priority = c.Priority });
});


EmitResult.emittedFiles list returned from ts.program.emit() contains relative and full path names of emitted files

TypeScript Version:  2.0.9
Expected behavior:
EmitResult.emittedFiles should use sourceFile.path rather than sourceFile.fileName as the fileName can be relative or canonical.
File names resolved from references within root files are always canonical. Root file names are always relative.
Each string/name in the emitFiles list should always contain the absolute path of where the output was written. The problem line ( as I see it is ):
emitOutputFilePathWithoutExtension = ts.removeFileExtension(sourceFile.fileName);

from
function getOwnEmitOutputFilePath(sourceFile, host, extension) {
        var compilerOptions = host.getCompilerOptions();
        var emitOutputFilePathWithoutExtension;
        if (compilerOptions.outDir) {
            emitOutputFilePathWithoutExtension = ts.removeFileExtension(getSourceFilePathInNewDir(sourceFile, host, compilerOptions.outDir));
        }
        else {
            emitOutputFilePathWithoutExtension = ts.removeFileExtension(sourceFile.fileName);
        }
        return emitOutputFilePathWithoutExtension + extension;
    }

The compilerOption listEmittedFiles is "internal" as it most likely was meant for use with tsc ( ??? I don't know the history here ). However, this option controls whether the emitter generates the list of files output when calling program.emit(). I really don't care if listEmittedFiles is marked as internal, but having it be internal and having an effect on a non-internal property in EmitResult is questionable to me. Just my 2 cents worth.

I am not sure i see the issue here. If this is about using the API and getting some mixed path results, then my recommendation here is if you are using the API to always provide absolute file names.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.

@mhegazy I think emittedFiles on ts.EmitResult should be made internal. The only way to populate that is to provide the internal listEmittedFiles on ts.CompilerOptions (see this line).
So either both should be internal or both should be public.


ImprovementOptimized some datetime function's return value

Proposed changes
Problem summary
make function year month day hour minute second use smaller usable return type.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[x] Is this PR support rollback (If NO, please explain WHY)

Further comments

run buildall

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 32.96 seconds
stream load tsv:          424 seconds loaded<PHONE_NUMBER>9 Bytes, about 168 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          72 seconds loaded<PHONE_NUMBER> Bytes, about 14 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230406063641_clickbench_pr_125335.html

run buildall

run buildall

run clickbench

run p0

run clickbench

run buildall

run buildall

run p0 run clickbench

run clickbench

run buildall

run buildall

run buildall

run buildall

run p1

run buildall

run p0

run buildall

run p0


Policy count should be 1

Getting the following error when trying to execute my initial deploy using "sls deploy -s dev -r us-east-1"
Error --------------------------------------------------
 Policy count should be 1! Please report this error to
 the alias plugin owner.

 For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

Stack Trace --------------------------------------------
Error: Policy count should be 1! Please report this error to the alias plugin owner.
at AwsAlias.aliasHandleLambdaRole (C:\Apps\Workspaces\Development\Services\development\Services\apitest\nodejs\node_modules\serverless-aws-alias\lib\aliasRestructureStack.js:396:28)
at runCallback (timers.js:649:20)
at tryOnImmediate (timers.js:622:5)
at processImmediate [as _immediateCallback] (timers.js:594:5)
From previous event:
at PluginManager.run (C:\Users\xxxxx\AppData\Roaming\npm\node_modules\serverless\lib\classes\PluginManager.js:156:22)
at Serverless.run (C:\Users\xxxxx\AppData\Roaming\npm\node_modules\serverless\lib\Serverless.js:95:31)
at serverless.init.then (C:\Users\xxxxx\AppData\Roaming\npm\node_modules\serverless\bin\serverless:23:50)
Get Support --------------------------------------------
Docs:          docs.serverless.com
Bugs:          github.com/serverless/serverless/issues
 Please report this error. We think it might be a bug.

Your Environment Information -----------------------------
OS:                 win32
Node Version:       6.9.4

Which Serverless version and which alias plugin version are you using.
I had a bug in the plugin where upgrading from Serverless 1.6 to 1.8/1.9 led to an error. Should be fixed in the 0.3.4 plugin version. Please check that, and in case it does not work, can you share some information about your service YAML file? Especially if you customized something in regards to the IAM role/policy configurations.

Frank,
We’re using Serverless 1.9 with serverless-aws-alias version 0.3.4-alpha1.  Below is a complete copy of our test yaml.

serverless.yml <<<<<<<<<<<

service: apitest
frameworkVersion: ">=1.5.0"
custom:
defaultStage: dev
defaultRegion: us-east-1
versionPath: v1
profiles:
dev: default
prod: default
package:
individually: true
exclude:
provider:
name: aws
profile: ${self:custom.profiles.${self:provider.stage}}
runtime: nodejs4.3
stage: ${opt:stage, self:custom.defaultStage}
region: ${opt:region, self:custom.defaultRegion}
timeout: 30
memorySize: 128
apiKeys:
role: arn:aws:iam::xxxxxx:role/LambdaExecute
deploymentBucket: serverless.${self:provider.region}.serverless-xxxxxx.com
stackTags:
Environment: ${self:provider.stage}
Application: Services

functions:
apitest:
name: ${self:service}-${self:provider.stage}
description: Sample function for apitest
handler: handler.handler
vpc: ${file(./serverless-${self:provider.stage}.yml):vpc}
environment:
dbname: ${self:provider.stage}DB
package:
include:
exclude:
events:
- http:
method: get
path:  ${self:custom.versionPath}/
cors: true
private: true
authorizer:
identitySource: method.request.header.Authorization
resultTtlInSeconds: 0
arn: arn:aws:lambda:us-east-1:xxxxxx:function:security-userguidauth-${self:provider.stage}
resources:
plugins:
- serverless-mocha-plugin
- serverless-aws-alias
From: Frank Schmid<EMAIL_ADDRESS>Reply-To: HyperBrain/serverless-aws-alias<EMAIL_ADDRESS>Date: Monday, March 27, 2017 at 1:28 AM
To: HyperBrain/serverless-aws-alias<EMAIL_ADDRESS>Cc: SantourA<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [HyperBrain/serverless-aws-alias] Policy count should be 1 (#21)
Which Serverless version and which alias plugin version are you using.
I had a bug in the plugin where upgrading from Serverless 1.6 to 1.8/1.9 led to an error. Should be fixed in the 0.3.4 plugin version. Please check that, and in case it does not work, can you share some information about your service YAML file? Especially if you customized something in regards to the IAM role/policy configurations.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for the YAML. I will investigate the issue with the given file. Hopefully I can reproduce it and find the bug soon.

Update: I have the assumption that it has something to do with the custom authorizer. I think I missed that while implementing the stack transforms.

That’s great news.  Would you have a patch version soon?  Looking forward to it, thank you.
From: Frank Schmid<EMAIL_ADDRESS>Reply-To: HyperBrain/serverless-aws-alias<EMAIL_ADDRESS>Date: Monday, March 27, 2017 at 8:52 AM
To: HyperBrain/serverless-aws-alias<EMAIL_ADDRESS>Cc: SantourA<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [HyperBrain/serverless-aws-alias] Policy count should be 1 (#21)
Update: I have the assumption that it has something to do with the custom authorizer. I think I missed that while implementing the stack transforms.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes. Will start working on it right now 😄

I had to restructure the APIG handling first as I found a semantic flaw in there. This is fixed now and I will continue on the authorizer stuff now.

I just tried plain Serverless with this declaration:
functions:
  testfct1:
    description: 'Echo function echoes alias'
    handler: handlers/testfct1/handler.handle
    events:
      - http:
          method: GET
          path: /func1
          authorizer:
            identitySource: method.request.header.Authorization
            resultTtlInSeconds: 0
            arn: arn:aws:lambda:us-east-1:XXXXXXX:profile-access-token-auth:dev

The authorizer resource is not created in the serverless CF file.
@SantourA Did you try custom authorizers with plain Serverless? Did it create a CF resource?

Debugged the Serverless code. It seems that the name property in the authorizer is mandatory - otherwise it won't merge the resource properly.

Ran into an issue in Serverless: https://github.com/serverless/serverless/issues/3413

Hi, has this been resolved?  I am not using authorizers and I am still getting the policy error.

The fix committed to master - was an additional error in the IAM policy handling.
I'll try to release a version soon.

Will you have a new build soon?

Sorry for the delay. I worked hard to get Serverless have the new event model, so unfortunately I did not spend much time on the plugin. Now I will switch back to this project and see that I can release the fixes in the upcoming week.

@SantourA Did you check if master works and resolves the issue for you? Be careful, as the APIG changes I did might need you to remove the alias stacks before deploying with the new version. Additionally I'd propose that you use it with a test project first, in case you use the new SLS 1.12.x version.

@SantourA Just released 0.4.0.


[Bug] Cannot Access MaxScale GUI Interface in Multi-Instance StatefulSet

Documentation

[ ] I acknowledge that I have read the relevant documentation.

Describe the bug
I am facing an issue while deploying MaxScale in a multi-instance setup. When I have multiple instances running, I am unable to access the MaxScale GUI interface. However, I can successfully access the GUI interface when deploying a single instance or when only one instance is running.
Upon investigating the issue, I found that when I try to access the GUI interface with multiple instances running, I receive a "401 Unauthorized" error. To gather more information, I inspected the browser console using the F12 developer tools.

Expected behaviour

Environment details:

Kubernetes version: [1.17.2]
mariadb-operator version: [0.0.27]
Install method: [helm]
Install flavor: [custom]

Additional context

Hey there @luohaha3123 ! Thanks for reporting
This should be solved by https://github.com/mariadb-operator/mariadb-operator/issues/563. We have plans to support it shortly.
Thanks!

@mmontes11 Thank you for your response. Based on #563 , I have tried modifying the ServiceAffinity to ClientIP mode and found that it allows for normal access. However, I'm unsure if this approach would impact the load balancing capability of the Service.


Wrap thrown system exceptions to provide useful information

Version: Swashbuckle.AspNetCore Version="5.4.1"
The bug(for me It's a bug) I'm talking about is that when GenSwagger is called and an exception is thrown while the core library is trying to do some reflection you get exceptions like:
 System.Reflection.AmbiguousMatchException: Ambiguous match found. with zero information which controller or operation caused this exception.
The problem is that It's quite hard to actually break when the exception is thrown and getting any information why It happens. So I would propose we wrap any exception which happens during GetSwagger to actually contain information in which controller and which operation the exception happened. I could also create a pull request implementing this but I'm not sure what kind of information the wrapped exception should contain.

Seems reasonable. Adding the related ApiDescription, which identifies the controller and action , among other things, would be a good start. With this in mind, I think the method SwaggerGenerator.GenerateOperation would be the appropriate place to put the try/catch/wrap/rethrow logic. Happy to accept a PR for this.

See 61b2fb28928ca211c9655fc212f086ee788f00c9

See 61b2fb28928ca211c9655fc212f086ee788f00c9


ctest3 on CentOS CI

ctest3 is running on centos now
http://math-ci.rocm.amd.com/blue/organizations/jenkins/compute-rocm-dkms-no-npi-hipclang%2Fprecheckin%2FrocPRIM/detail/PR-160/1/pipeline/328


Validation error due to missing mandatory elements that exist in the composition

I successfully created an EHR and committed some of the compositions that are included as test resources based on the default OPTs shipped with the code base, but after uploading my own OPT, and trying to commit a valid composition there are some unexpected validation errors due to missing mandatory elements. I've attached the composition.xml and OPT in my Gist. The returned errors are:
Internal Server Error: org.ehrbase.validation.constraints.wrappers.ValidationException:
Validation error at /content[openEHR-EHR-SECTION.problem_list.v1]/items[openEHR-EHR-ACTION.procedure-HAOC.v1]/description[at0001]/items[at0060], Mandatory element missing, expected:[1..1]
Validation error at /content[openEHR-EHR-SECTION.problem_list.v1]/items[openEHR-EHR-ACTION.procedure-HAOC.v1]/description[at0001]/items[at0.74], Mandatory element missing, expected:[1..1]
Validation error at /content[openEHR-EHR-SECTION.problem_list.v1]/items[openEHR-EHR-ACTION.procedure-HAOC.v1]/description[at0001]/items[at0.70], Mandatory element missing, expected:[1..1]
Validation error at /content[openEHR-EHR-SECTION.problem_list.v1]/items[openEHR-EHR-ACTION.procedure-HAOC.v1]/description[at0001]/items[at0002.1], Mandatory element missing, expected:[1..1]
Validation error at /content[openEHR-EHR-SECTION.problem_list.v1]/items[openEHR-EHR-EVALUATION.problem_diagnosis-ms_br.v1]/data[at0001]/items[at0002.1], Mandatory element missing, expected:[1..1]

But by inspecting the composition we can see that the elements are there. I tried increasing the log verbosity but didn't get much help from the console. I could trace that the composition with the data points above was processed, but I suspect the problem is that the OPT defines multiple /content[openEHR-EHR-SECTION.problem_list.v1] siblings (with the same archetype_node_id), but the template refine their contents differently by renaming and including/restricting different children in their slots, so EHRbase might be validating a content block against the wrong (possibly first) definition.

I also contacted @wolandscat about the proper way to discern different siblings with the same archetype_node_id. In the OPT from this scenario, the siblings are optional so relying on index/order might not be reasonable. Ocean's Template Designer (and I think I've also seen EtherCIS do something similar in the past) use the name attribute as a predicate to refine the path expression, but this may create issues due to relying on localized text, breaking stuff for instances of archetypes with multiple translations/different languages. Maybe archetyped term definitions could be used? What do you think?

Hi Ricardo, could you please post your composition xml and OPT? The provided links seem broken. Many thanks

Hi Christian, sorry about that, for some reason the Gist links changed. I've edited them in the first post.
(https://gist.github.com/ricardofago/7446ea8f8611add03faa14afe35603d1)

Seems the issue is that in the composition, there are multiple
 <content archetype_node_id="openEHR-EHR-SECTION.problem_list.v1" xsi:type="SECTION">

Causing the identification of node f.e. with path:
/content[openEHR-EHR-SECTION.problem_list.v1]/items[openEHR-EHR-ACTION.procedure-HAOC.v1]/description[at0001]/items[at0060]

to fail unless it is located at the top of the tree (in sequence). Thus a workaround is to avoid the above duplication by mean of template refactoring if it is possible until this issue is fixed (it is a low level issue AFAIK). Tx

@ricardofago I have pushed a fix related to the validation issue. However, there is another issue with sections sharing the same archetype node id, but with a different name. At the moment, the name is not properly propagated into the serialized json stored in the DB. As a result, the composition is not fully persisted in this particular scenario (NB. this issue only happens with SECTION AFAIK). One workaround is to use different section node ids or refactoring the template so structures are assigned to unique sections.

See https://github.com/ehrbase/ehrbase/tree/feature/24_validation_error_for_existing_mandatory_elements for details

@chevalleyc @ricardofago My understanding is that the issue has been successfully resolved and I can close the issue. Is this correct?

To be honest I got caught up in some other stuff and phased out so I haven't tested this recently, but from the latest/referenced activity I'm assuming the problem with generic SECTION instances with the same archetype_node_id but different names still remain... As arguable as it could be from a semantic perspective, I think it would still be a fairly common scenario in templates due to grouping/structuring blocks for data entry.
I'm not sure the issue is entirely resolved. @chevalleyc can elaborate better, or it might be more reasonable to close this and raise a more specific/tailored issue.

With the current code base (0.10.0), the above example composition example is now validated and stored properly including the SECTIONs with different names. I agree with @ricardofago , should this issue occurs please open a new CR.
NB. the validation now check DvCodedText value against localized terminology (here 'pt') please make sure that the value is actually localized as well (f.e. evento instead of event) etc.


meta_value questions

Hi,
thank You for Your quick responses so far, I hope You can bear with me.
I have two questions regarding the following piece of code (see https://github.com/microsoft/Cream/blob/main/lib/models/PrioritizedBoard.py#L27)
        elif self.cfg.SUPERNET.PICK_METHOD == 'meta':
            meta_value, cand_idx, teacher_cand = -1000000000, -1, None
            for now_idx, item in enumerate(self.prioritized_board):
                inputx = item[4]
                output = F.softmax(model(inputx, random_cand), dim=1)
                weight = model.module.forward_meta(output - item[5])
                if weight > meta_value:
                    meta_value = weight
                    cand_idx = now_idx
                    teacher_cand = self.prioritized_board[cand_idx][3]
            assert teacher_cand is not None
            meta_value = torch.sigmoid(-weight)
    ...
    return meta_value, teacher_cand

why use torch.sigmoid(-weight), and not torch.sigmoid(-meta_value) ?
weight will refer to the last entry in the board, and might not belong to the selected teacher_cand

2) why use torch.sigmoid(-weight), and not torch.sigmoid(weight) ?
since weight describes the matching degree (higher is better, the teacher candidate is also selected this way), torch.sigmoid(-weight) will go towards 0.
subsequently, a high matching value has lower L_kd impact on the loss function (https://github.com/microsoft/Cream/blob/main/lib/core/train.py#L64):
loss = (meta_value * kd_loss + (2 - meta_value) * valid_loss) / 2
Am I missing something?
Best,
Kevin

Hi,
Thanks for your interest in our project! It should be torch.sigmoid(meta_value). This code is extracted from a large codebase, this is caused by carelessness. We would fix it in a few days.
Best,
Hao.

Hi,
Thanks for your great job!
the way to simulate sgd updating.
  # simulate sgd updating
  def simulate_sgd_update(self, w, g, optimizer):
      return g * optimizer.param_groups[-1]['lr'] + w

According to Eq(6) in the paper, It should be return -g * optimizer.param_groups[-1]['lr'] + w ?

Hi,
Thanks for your interest in our project!
It seems that when we were re-organizing the codebase, we made some mistakes. We will fix it ASAP, and thanks for pointing it out!
Best,
Hao.


Search all apps during heroku publish

Adds the -A option to include apps from all organizations when searching app names for publish.

It looks like the check passed, not sure why it's showing as running in GH.


How to run PercepPan

Dear!
I am wondering how to run your code.

How to test? What arguments should be provided?
What is the training data format?

Thanks u in advance.

Thank you for your interest.
The code is based on BasicSR but not be cleaned,
so it contains some unused segment. Just pick what you need.

For test, please try test.py or test_ikonos.py, and specify arguments in file 'options/test/test_ESRGAN.json'
Raw data format is tiff, but it is transformed into .lmdb format for speeding. Please have a look at scripts/create_lmdb.py and scripts/extract_subimgs_single.py for detail.


input text loses formatting (whitespace) on WebUI

if you copy and paste text, for example code, it will lose the formatting.
for example, if you copy & paste the code below:
//GetAccountApps TODO list apps attached to account
func GetAccountApps(w http.ResponseWriter, r *http.Request) {
	w.Header().Set("Content-Type", "application/json; charset=UTF-8")
	w.WriteHeader(http.StatusOK)
}

it will look like this on the output:
//GetAccountApps TODO list apps attached to account
func GetAccountApps(w http.ResponseWriter, r *http.Request) {
w.Header().Set("Content-Type", "application/json; charset=UTF-8")
w.WriteHeader(http.StatusOK)
}
But if you edit the message, the formatting is still retained.
This looks like output formatting, where the whitespace is removed.
Okay. To be fair. The formatting is also lost here on github, unless the text is enclosed in a code tag.
this only happens on web, on the mobile app, the formatting is retained.

Thank you for spotting this! I needed to fix the css on displayed message.
I update the release artifact with the fix:
https://github.com/balzack/databag/releases/download/v1.1.1021/webapp.tar.gz
The release asset is repackaged from the latest build artifact:
https://github.com/balzack/databag/actions/runs/11447757544

Thank you ! It works !

It works!

thanks for your help :)


[Duplicate of #5783] - Introduction to Ensemble Methods in Machine Learning

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

`This closes #5431

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Hello @srishilesh  this is an update of #5783

Hello @srishilesh  this is an update of #5783

hello @srishilesh kindly re-open the PR so that i can push the commits

Hi @srishilesh kindly re-open the PR so that I can push the commits

Hi @EssyH ,
Thanks for making the changes.
It would be great if you could share the URL to your forked branch. I would have to take a look at it to see how your article is different from what we have here.
This is just to ensure to avoid duplicates of content, and improve the quality of the PRs.
Thanks.

Hi, @hector @section-engineering-education/content-moderator-team what do you think about this update https://www.section.io/engineering-education/ensemble-learning/


RemoveReviewers API: omitempty on required reviewers field causes spurious 422s

Summary
Calling RemoveReviewers when you want to only remove teams elicits a 422, due to how omitempty prevents the golang client from serializing the empty reviewers slice. This field is required on the server, so the golang client needed to send reviewers: [].
Workaround
Work around this by exploiting delete idempotency =) Just pass it a bogus username alongside real team slugs.

I'm confused by this report.
I'm looking at this endpoint:
https://github.com/google/go-github/blob/792bd15b871a8a6aea57cca226ef7a9e85e2955a/github/pulls_reviewers.go#L73-L86
and am not seeing the slice you are talking about.
I do, however, see these slices:
https://github.com/google/go-github/blob/792bd15b871a8a6aea57cca226ef7a9e85e2955a/github/pulls_reviewers.go#L13-L24
which are used with the RequestReviewers method.
Can you please clarify your request?
And while I'm here, would you like to submit a PR to resolve the issue?

EDIT: Ah!  I see it now.
OK, gotcha.
Would you like to create a PR to fix this issue, @nagl-resourcely ?

Heh, that probably would have been faster than typing up the issue. Sure - I'll send one after the workday ends.


Postal - Nodejs client - send error: Error: getaddrinfo ENOTFOUND https://postal.yourdomain.com

Hello developers,
I set my postal server, and everything is working just fine
now I am trying to send emails from my node js app
I am using node js client and following instructions found here:
https://github.com/krystal/postal-node
now when I send an email I am getting an error like this:
Error: getaddrinfo ENOTFOUND https://postal.yourdomain.com
of course, yourdomain.com should be replaced with my read domain name.
your help is much appreciated.
Thank you

I solved it.
in clients/nodejs documentation found here:
https://github.com/krystal/postal-node
it says I should add host name like this:
var client = new Postal.Client('https://postal.yourdomain.com', 'your-api-key');
I tried that, but it produced the error.
so I write the host name with out https:// . so the line now looks like:
var client = new Postal.Client('postal.yourdomain.com', 'your-api-key');
now it is working as it should.
I hope this helps you too.
Thanks

@ramioooz thanks for letting us know, can you make an issue on that repo in case that is something that helps future users of that library


Command line -c C does not include a way to quit

$ ev -c "24 8 * . quit"
192.0
ev> quit
$

This should quit without having to enter quit as an additional command.

The problem is probably here:
#   Run any command line tokens
if args.c:
    self.ev(args.c)

I should capture the return value and exit if it is EOF.

That was the correct fix, however, I also need to correct the unit test that didn't catch it (Test-Driven Development ® :smile:)