Browser version giving error 404

I couldn't get this to work in the browser, I wrote a small test program that just runs a query and ran it with node and then in the browser.
This is probably just me using it wrong, if so I apologize
Thanks in advance

Turns out that this was actually a same origin policy problem, apologies for the pointless issue


Windows CMD plugin recording system paths in wrong order

The existing rezplugins/shell/cmd.py has a classmethod CMD.get_syspaths which is currently scrambling the order of the system paths returned from an inspection of the Windows registry.
The cause is this line:
cls.syspaths = set([x for x in paths if x])

The cast to a set eliminates duplicate items, but since a set is inherently unordered it also loses the original sort order, which can be very relevant in a shell search. Additionally this only happens in the Windows plugin, all others return in their original order and without duplication removal.
Working on a fix that will still do a quick removal of duplicates but will preserve order.

Fixed in #341.

This issue cropped up for me this week, as the cmd and powershell plug-ins in the master branch are still using a set(), and scrambling the path on Windows. Any chance we can get this commit wrapped into master?
https://github.com/Pixomondo/rez/commit/ae10c1580f66ae11d9d11ef1392b2a9b8b33e9ac

I'll take a look at this soon.
Thx
A
On Thu, Dec 12, 2019 at 11:22 PM bsilvaLS<EMAIL_ADDRESS>wrote:

This issue cropped up for me this week, as the cmd and powershell plug-ins
in the master branch are still using a set(), and scrambling the path on
Windows. Any chance we can get this commit wrapped into master?
Pixomondo@ae10c15
https://github.com/Pixomondo/rez/commit/ae10c1580f66ae11d9d11ef1392b2a9b8b33e9ac
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/344?email_source=notifications&email_token=AAMOUSQ2O2AUYG2YMIITJDTQYIUH7A5CNFSM4CNWIWE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGWPYTQ#issuecomment-564984910,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOUSX44BQKAU3BYDUVLMLQYIUH7ANCNFSM4CNWIWEQ
.

This is fixed now AFAIK.


Add Support for GE Water Filters

Add initial support for GE Water Filters.
Right now added sensors for mode of the filter, filter life, and addition to flow rate & day usage.
HA Integration:

Great job adding this support and thanks for the contribution!!  Just a few minor changes requested and we can get this merged into the code.

I make these changes - I also noticed ( when loading on my main instance with HACS) that some of the imports are wrong. I'll push that up too.

Any updates on the changes?  You mentioned adding some additional bugfixes so didn't want to just push this over and fix up the couple of comments.


Uninitialized constant ActiveRecord when running in production

Trying to use puffer with MongoDB without any ActiveRecord models, thus I don't load ActiveRecord at all.
I've created custom User model, overrided current_puffer_user and has_puffer_access? methods.
In development environment everything works great. But when I try to run the app in the production env, I'm getting this:
vendor/ruby/1.9.1/bundler/gems/puffer-b3016bfffcef/app/models/puffer/puffer_user.rb:1:in `<top (required)>': uninitialized constant ActiveRecord (NameError)
	from vendor/ruby/1.9.1/bundler/gems/puffer-b3016bfffcef/app/models/puffer_user.rb:1:in `<top (required)>'
	from vendor/ruby/1.9.1/gems/activesupport-3.1.1/lib/active_support/inflector/methods.rb:124:in `block in constantize'
	from vendor/ruby/1.9.1/gems/activesupport-3.1.1/lib/active_support/inflector/methods.rb:123:in `each'
	from vendor/ruby/1.9.1/gems/activesupport-3.1.1/lib/active_support/inflector/methods.rb:123:in `constantize'
	from vendor/ruby/1.9.1/gems/activesupport-3.1.1/lib/active_support/core_ext/string/inflections.rb:43:in `constantize'
	from vendor/ruby/1.9.1/gems/actionpack-3.1.1/lib/action_controller/metal/params_wrapper.rb:148:in `_default_wrap_model'
	from vendor/ruby/1.9.1/gems/actionpack-3.1.1/lib/action_controller/metal/params_wrapper.rb:167:in `_set_wrapper_defaults'
	from vendor/ruby/1.9.1/gems/actionpack-3.1.1/lib/action_controller/metal/params_wrapper.rb:128:in `inherited'
	from vendor/ruby/1.9.1/gems/actionpack-3.1.1/lib/abstract_controller/railties/routes_helpers.rb:7:in `block (2 levels) in with'
	from vendor/ruby/1.9.1/gems/actionpack-3.1.1/lib/action_controller/railties/paths.rb:7:in `block (2 levels) in with'
	from vendor/ruby/1.9.1/bundler/gems/puffer-b3016bfffcef/lib/puffer/controller/dsl.rb:29:in `inherited'
	from vendor/ruby/1.9.1/bundler/gems/puffer-b3016bfffcef/app/controllers/admin/puffer_users_controller.rb:1:in `<top (required)>'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/engine.rb:417:in `block (2 levels) in eager_load!'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/engine.rb:416:in `each'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/engine.rb:416:in `block in eager_load!'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/engine.rb:414:in `each'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/engine.rb:414:in `eager_load!'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/application/railties.rb:8:in `each'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/application/railties.rb:8:in `all'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/engine.rb:412:in `eager_load!'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/application/finisher.rb:51:in `block in <module:Finisher>'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/initializable.rb:30:in `instance_exec'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/initializable.rb:30:in `run'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/initializable.rb:55:in `block in run_initializers'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/initializable.rb:54:in `each'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/initializable.rb:54:in `run_initializers'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/application.rb:96:in `initialize!'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/railtie/configurable.rb:30:in `method_missing'
	from config/environment.rb:5:in `<top (required)>'
	from config.ru:4:in `block in <main>'
	from vendor/ruby/1.9.1/gems/rack-1.3.3/lib/rack/builder.rb:51:in `instance_eval'
	from vendor/ruby/1.9.1/gems/rack-1.3.3/lib/rack/builder.rb:51:in `initialize'
	from config.ru:1:in `new'
	from config.ru:1:in `<main>'
	from vendor/ruby/1.9.1/gems/rack-1.3.3/lib/rack/builder.rb:40:in `eval'
	from vendor/ruby/1.9.1/gems/rack-1.3.3/lib/rack/builder.rb:40:in `parse_file'
	from vendor/ruby/1.9.1/gems/rack-1.3.3/lib/rack/server.rb:200:in `app'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/commands/server.rb:46:in `app'
	from vendor/ruby/1.9.1/gems/rack-1.3.3/lib/rack/server.rb:301:in `wrapped_app'
	from vendor/ruby/1.9.1/gems/rack-1.3.3/lib/rack/server.rb:252:in `start'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/commands/server.rb:70:in `start'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/commands.rb:54:in `block in <top (required)>'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/commands.rb:49:in `tap'
	from vendor/ruby/1.9.1/gems/railties-3.1.1/lib/rails/commands.rb:49:in `<top (required)>'
	from script/rails:6:in `require'
	from script/rails:6:in `<main>'

Gotcha.

Fixed, it seems ) Check cutting edge please.

Now works, thanks)

aeee


feat: Enhance and improve basic features to support casdoor-springboot-starter development

ref: casdoor-spring-boot-starter README
casdoor-springboot-starter depends on this sdk, so it must wait for the completion of this sdk

@Kininaru

@tangyang9464 I think @Kininaru maybe doesn't have time for this moment. Can you do it instead?


Add hasPrevPage and hasNextPage booleans

This PR adds hasPrevPage and hasNextPage booleans to SnapCarouselResult to indicate if there's a previous or next page to scroll to.
@richardscarrott, I have updated the README.md and Storybook components to use these new properties. You may also want to update the CodeSandbox example after this change has been released.
Closes #28.

This is available in a pre-release https://github.com/richardscarrott/react-snap-carousel/releases/tag/v0.5.0-next.1 -- npm add<EMAIL_ADDRESS>I'll run it in production for a little bit and then make it available as v0.5.0 proper.

Now published as 0.5.0 https://github.com/richardscarrott/react-snap-carousel/releases/tag/v0.5.0

Thanks @richardscarrott! 🎉
Unrelated, have you considered adding support for autoplay?
It’d be neat if react-snap-carousel could be used to create something similar to Apple’s scroll-based carousels with autoplay:
https://github.com/user-attachments/assets/34d1c0cf-c78a-4a2b-a453-3f4e0f6a8675
https://www.apple.com/iphone-16-pro/

@jasongerbes I think that UI could be built relatively easily on top of what react-snap-carousel already offers -- imo it's prob best react-snap-carousel doesn't get involved with the timer logic as, for example in that instance, each item's video could be a different length so an out-the-box autoplay function likely wouldn't suffice -- there is a very basic example of autoplay here -- https://github.com/richardscarrott/react-snap-carousel/issues/17
re: playing the video when an item becomes active, that could be done using activePageIndex (similar to showing the text here) or alternatively I've found
useInView from framer-motion to be quite effective as it factors in all scroll parents.


[release] v6.1.2

After running pnpm dedupe building the docs results in an error: nextExternals is not a function. After running pnpm dedupe, next is pointed to ^14.2.14 instead of ^14.2.13, but I haven't found the root problem just yet. Apparently no other users with this problem just yet.

This change in next is the culprit: https://github.com/vercel/next.js/pull/70382/files
I'll fix it in another PR and come back to this one.

@oliviertassinari comments addressed in https://github.com/mui/material-ui/pull/43968


Automatic PR from RestAPI-PR1857

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1857

Codecov Report

Merging #1542 into master will decrease coverage by <.01%.
The diff coverage is 53.77%.

@@            Coverage Diff             @@
##           master    #1542      +/-   ##
==========================================
- Coverage   54.23%   54.23%   -0.01%
==========================================
  Files        3585     3601      +16
  Lines       88233    88526     +293
==========================================
+ Hits        47852    48009     +157
- Misses      40381    40517     +136

Impacted Files
Coverage Δ

...s/job/data_lake_analytics_job_management_client.py
96.77% <ø> (+2.83%)
:arrow_up:

...models/compute_policy_account_create_parameters.py
100% <ø> (ø)
:arrow_up:

...mt/datalake/analytics/job/models/job_statistics.py
62.5% <ø> (ø)
:arrow_up:

...gmt/datalake/analytics/job/models/job_data_path.py
62.5% <ø> (ø)
:arrow_up:

...e/analytics/job/models/job_pipeline_information.py
100% <ø> (ø)
:arrow_up:

...odels/data_lake_store_account_update_parameters.py
100% <ø> (ø)
:arrow_up:

.../analytics/job/operations/recurrence_operations.py
89.65% <ø> (ø)
:arrow_up:

...talake/analytics/job/models/base_job_parameters.py
100% <ø> (ø)
:arrow_up:

...t/datalake/store/models/data_lake_store_account.py
100% <ø> (ø)
:arrow_up:

...mgmt/datalake/analytics/job/models/job_resource.py
100% <ø> (ø)
:arrow_up:

... and 75 more

Continue to review full report at Codecov.

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

@lmazuel  Thanks.  Which file defines which Swagger specs to generate the code from?  I think it's using 2016-11-01 version of job.json when it should be using 2017-09-01-preview.

@ro-joowan ADL configuration is defined here:
https://github.com/Azure/azure-sdk-for-python/blob/master/swagger_to_sdk_config.json#L707-L745


Is there any way to set the border in MaterialSkin Window.

Is there any property that can be used to show border on the screen.

Another issue is even if you have an item FILLING the form, the theme causes so much "empty" space especially like at the bottom. Currently have a tab control filling the form and the amount of space at the bottom makes everything look like trash.
I'm worried this is a long dead project its time for users to take it up I guess.

I like that someone @IgnaceMaes  take his time and implement this project. I use it on an 8-9 project that all based on WinForms. The software was shipped but it does take a lot of work that I don't think is required to produce that thing.
I moved from this dead project to https://github.com/material-components/material-components-web
which is supported by Google, Now use IE11 for web browser control and I can use my same HTML skill on web and desktop.
Why waste my time on this project when it's not supported (no offense) and I can get a better result from Google supported material supported component. At least they will give me a notice or better thing to move on, (instead of provide no-support).
I am not blaming here the developer but it was never going well. I use it on 8-9 project and I have wasted a lot of time.
The theme limitation (you can't set the font for 2 different labels or 2 different colors), I write my own code to sort out this issue)
Late on I find that If I us my html skill I can better same skill for everything (web and mobile).
If anyone looking for a sollution or alternative, I can suggest you IE11 based web browser control in winforms is good to go with this repos
https://github.com/material-components/material-components-web
If you can afford or force your user to use 1803 (latest window update) you can use WebView which Microsoft released last month at build.


Exclude Escape Characters from Property Names

What are the schemas that are affected by the issue
all
What are examples of products that are impacted by the issue
all products that need to escape certain characters in property names.

@trieloff do you have some examples of cases where this is an issue? Is it only from other preexisting specifications that we want to incorporate?

@trieloff - is this still a valid concern. If so please provide more details, if not please close.


Provide an easy way to ensure that new warnings/errors are not introduced to a project

Angular wants a way to ensure that they have not introduced additional warnings to pub build- would be great if there was a default way to do this.
I'll add a utility which takes a file of expected warnings and does an error if it differs, but seems like something which would be generally useful (like a file for warning suppressions and a 'fail on warning' flag).

This issue has been moved to dart-lang/pub#946.


Added multibit-hd 0.4.1

There already was #15009.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

How does this relate to the current Cask?

multibit Cask is MultiBit Classic which is legacy version.
multibit-hd is modern version with better security.
You can read about differences here.
I created new cask because update from MultiBit Classic to HD is not trivial.
Also we can move multibit to multibit-classic in cask-versions as it's legacy version. But I think that we should keep multibit-hd name because it's not desired behaviour if cask users suddenly will get MultiBit Classic updated to HD.

Thank you for the explanation @mxl.


npmjs version needs a bump

The npm version is outdated still uses the old way to get the React component like
import React from 'react-native';

instead of this
import React from 'react';

There is v0.6.0 now

Nice thanks!


Fix error on non-redirecting wildcard route, fixes #41

See https://github.com/Greentube/localize-router/issues/41#issuecomment-291875798

Fixed in 7.0.0


dial code instead of iso2 code?

How can I show dialcode instead of iso2 code in the extra input field?

The plugin doesn't have a list of dial codes, it uses country codes. If you're interested in dialing codes, please take a look at the International Telephone Input plug-in, upon which this one is based ((https://github.com/Bluefieldscom/intl-tel-input.git). It will take care all of your international telephone handling needs.

Thank you


Release 0.2.3

Modified save_model in model.py to work as intended in Vertex KFP Pipelines using set environment variable for model save directory, created separate requirements files for both Python 3.6 and 3.7.

Forgot -S in commit


New Header CSS

Should match the library site/orangelight/etc.

I'll interpret this as follows, please correct if I'm wrong:

Update the logo, add site title next to it ala catalog site.
remove banner image area (currently an image of "nassau hall" script) where site title is now.
Otherwise update heights, etc, of orange / grey header bar area
leave search bar as-is

this appears to be the relevant orangelight PR https://github.com/pulibrary/orangelight/pull/954/files

header and footer are pulling from https://github.com/pulibrary/pul_assets; awaiting word from @axamei and co. for next steps.

@hackmastera @axamei and I discussed this, and we think it would be best to wait for us to finish the header and footer patterns in lux and then to add them that way. We have started these, but the Order Manager components were considered higher priority. Let us know if the headers/footers should take priority instead.

@sdellis I don't think this is higher priority. I guess we'll need webpacker to pull in via lux. I'll make a ticket for that.

After more discussion re: server-side rendering, it sounds like we should go ahead and update pul_assets for now. It will be some time before lux is ready to take care of this for us.

okay, holding for lux again!


Copy Paste (Clipboard) Keyboard Shortcut Issue

I have noticed that using the keyboard shortcuts (cmd/ctrl-c and cmd/ctrl-v) to copy or paste text from a input field does not work.
I have however seen that using the right click context menu works as expected.
Note I am on MacOS 11.
To Reproduce

Download and Install official Tauri example app from https://tauri.studio/en/docs/getting-started/setup-macos
Run App
Go to Communication tab
Enter text in the text box
Notice that you cannot select all copy or paste using keyboard shortcuts.

Screenshots

Platform and Versions (please complete the following information):

OS: Mac OS 11
Node: v12.18.3
NPM: 6.14.6
Yarn: 0.1.0
Rustc: 1.46.0 (04488afe3 2020-08-24)

Hmm I thought https://github.com/webview/webview/pull/405 would fix this. @abeMedia do you know something about this?

I have the same issue, also ctrl z does not undo text input like it does in the browser.

@lucasfernog that PR only allows clipboard access from JS. You still need to write the JS event to capture it.

Thanks for the leads. Don't know how I didn't find anything related to this when searching.
For those wondering the solution is to simply listen for a copy or paste event and simply call document.execCommand('copy'); or document.execCommand('paste');
Rudimentary vue example below_
<template>
  <Editor @keydown.meta.86="pasteEvent()" @keydown.meta.67="copyEvent()"  ... />
</template>

<script lang="ts">
// ...
  copyEvent() {
    document.execCommand('copy');
  }

  pasteEvent() {
    document.execCommand('paste');
  }
// ...
</script>

@matthewmullin01 Thank works thank you. But i'm still getting a system beep every time I press cmd-c or cmd-v, on Mac. Any ideas on how to prevent the system beep?

Just for reference, everything works as intended on Linux.

Just for reference, everything works as intended on Linux.

This has been fixed in the dev branch of Tauri and will be available in the next release.

I'm still seeing the contextmenu show up on paste while using macOS and<EMAIL_ADDRESS>Not sure if this version was the "next release"

I'm also seeing the same behaviour... :(
This is on 1.0.0-beta.2

I have still the problme and also beeps. yes, it beeps.

I'm running into the same issue.

For shurtcuts like this to work on macos you need to add a window menu. See #2397 for more or this template for inspiration.
The beeping sound is also known, see #2626. Unfortunetly we haven't found a solution so far.

@FabianLars thank you!

Managed to make it work with the A menu as you suggested but with some differences because the template used not sable API.
let about_menu = Submenu::new("App", Menu::new()
  .add_native_item(MenuItem::Hide)
  .add_native_item(MenuItem::HideOthers)
  .add_native_item(MenuItem::ShowAll)
  .add_native_item(MenuItem::Separator)
  .add_native_item(MenuItem::Quit));

let edit_menu = Submenu::new("Edit", Menu::new()
  .add_native_item(MenuItem::Undo)
  .add_native_item(MenuItem::Redo)
  .add_native_item(MenuItem::Separator)
  .add_native_item(MenuItem::Cut)
  .add_native_item(MenuItem::Copy)
  .add_native_item(MenuItem::Paste)
  .add_native_item(MenuItem::SelectAll));

let view_menu = Submenu::new("View", Menu::new()
  .add_native_item(MenuItem::EnterFullScreen));

let window_menu = Submenu::new("Window", Menu::new()
  .add_native_item(MenuItem::Minimize)
  .add_native_item(MenuItem::Zoom));

let help_menu = Submenu::new("Help", Menu::new()
  .add_item(CustomMenuItem::new("Learn More", "Learn More")));

let menu = Menu::new()
  .add_submenu(about_menu)
  .add_submenu(edit_menu)
  .add_submenu(view_menu)
  .add_submenu(window_menu)
  .add_submenu(help_menu);

app
.menu(menu)
.on_menu_event(|event| {
  let event_name = event.menu_item_id();
  match event_name {
    "Learn More" => {
      shell::open(
        "https://www.my-app.com".to_string(),
        None,
      )
      .unwrap();
    }
    _ => {}
  }
})

Let it be known that there are some not specified menu items show up (probably the OS adds them???). Also, this should be added to the official documentation at some point :)


Reuse test data on https://github.com/webmachinelearning/test-data.

Thanks @Honry It's fixed by #54, close it.


Truncate internal eval.

Fixes #907
Implements a new util function for uploading strings to paste service.
Truncates internal eval responses in a similar fashion to snekbox's eval and updates snekbox to use the new function.

That is an issue on my end but I cannot resolve it. I won't be able to test this PR myself.


iexfinance.utils.exceptions.IEXVersionError: The requested endpoint is only available using IEX Cloud.

Summary (include Python version)
I´m getting an error with the latest version of iexfinance 0.4.0 when using IEX Cloud API endpoints like Stocks, with sandbox as well as default token from IEX CLoud.
Date/time of issue
2019-03-09
Expected behavior
According to this reply it seems functions like get_balance_sheet() should be working with version 0.4.0
Actual behavior
Throwing error instead
iexfinance.utils.exceptions.IEXVersionError: The requested endpoint is only available using IEX Cloud.
Is the implementation of it missing in 0.4.0?

I'm getting this too

What API version are you using? To use get_balance_sheet the IEX_API_VERSION environment variable must  be set to iexcloud-beta. If it is not, IEXVersionError is raised, as this method is not available with the legacy API

Yep that was the issue. os.environ["IEX_API_VERSION"] = "iexcloud-beta" fixed it for me


Use travis-cargo for documentation updates

Fixes #223

Woops - probably shouldn't have been a minor version bump for the toml update. Sorry! >_<


Add Feedly to track CLs tweets

It would be great to keep track of what CLs are tweeting via Feedly.

https://docker-community-leaders.github.io/alphasite/feedly/

https://docker-community-leaders.github.io/alphasite/feedly/

Thanks. I just created a new Docker Community Leaders Twitter account https://twitter.com/dockercommunity and then created a list. Modified a new feedly.

Thanks. I just created a new Docker Community Leaders Twitter account https://twitter.com/dockercommunity and then created a list. Modified a new feedly.

lets close this !

lets close this !


Changed all instances of fetch_mnist to mnist

not sure if this is what you were looking for from the discord, but updated examples and such to mnist()

You didn't test this at all


fix(TDI-46475): Deprecate the list of components in 7.3

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-46475
What is the new behavior?
https://jira.talendforge.org/browse/TDI-46475
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull request test result: failure :x:


Add custom validators to rg-text-control

https://rockabox.tpondemand.com/entity/11309

is there any tests for these?

@nwhite89  test added, some small css fix to be reviewed by @ohskylab as well.

thanks @jess0407 Angular side alright @ohskylab to sign off think there should be a squash as well :+1:

@nwhite89 Yes, if the CSS fix is OK it should be squashed. I'll pick up review this morning.

@jess0407 @ohskylab Squashy?

I still need to review. It's in my queue.

CSS fix dropped; pick it up properly when making *-text-control pseudoclasses play properly with Angular.
No angular changes since approval, so I'm merging.


dependency-resolver: produce stable output

Without this, the order of the config entries changes every time the
dependency resolver is executed. As a result the order of the variables in
the .config file changes as well. This is rather annoying if the .config
file is stored elsewhere (e.g. in another git repository).
Enforce a defined order by sorting the items. This way the files only
change when with actual dependency changes.
Signed-off-by: Michael Olbrich<EMAIL_ADDRESS>

+1
@dorileo could you take a look?

applied, thanks...


Some attributes (schema.org) not working

I do not get attributes like itemscope, itemtype... for schema.org structured data.
I'm trying to add them to a div, like the schema.org examples show (but I've tried other elements too).
Other elements and attributes do work (e.g. playing for audio, or the picture element).
I'm using Brackets 1.3 on Mac (current at time of writing).
Thanks!

Hi there. Thanks for reporting this - I can reproduce the issue here (Brackets 1.3 on Windows 8.1). I'll look into this and see if I can get it fixed


Is it possible to Limit the GPU to 80%

import torch
import time
Checking if CUDA is available
if http://torch.cuda.is_available():
# Selecting the GPU
torch.cuda.set_device(0)  # Assuming GPU 0
while True:  # Infinite loop
    # Getting the current memory allocation and reserved memory
    allocated = torch.cuda.memory_allocated()
    reserved = torch.cuda.memory_reserved()

    print(f'Memory Allocated: {allocated / (1024 ** 2):.2f} MB')
    print(f'Memory Reserved: {reserved / (1024 ** 2):.2f} MB')

    time.sleep(5)  # Sleep for 5 seconds before checking again

else:
print('CUDA is not available.')

import os
import tensorflow as tf
Set the CUDA_VISIBLE_DEVICES environment variable
os.environ['CUDA_VISIBLE_DEVICES'] = '0'  # This sets the process to use only this # GPU
Set up GPU options
gpus = tf.config.experimental.list_physical_devices('GPU')
if gpus:
try:
tf.config.experimental.set_virtual_device_configuration(
gpus[0],
[tf.config.experimental.VirtualDeviceConfiguration(memory_limit=102480.8)])  # 80% of 8GB
except RuntimeError as e:
print(e)

This is really outside the scope of what I want this set of custom nodes to do. It should be possible to write code to do that, but I guess two concerns to keep in mind are

why?
doing it in conjunction with ComfyUI's VRAM optimizations, like making them trigger early.


Add gc monitor to warn users about excessive gc

It is not uncommon in large projects for the jvm to silently be running
frequent full gcs in the background. This can slow progress to a crawl.
Usually the fix is to bump the -Xmx parameter but if the users do not
realize that their tasks are slow because of gc thrashing, they may not
think to do that. This PR adds a monitor that hooks into the jvm's event
notification system to keep track of how much time is spent in GC. If
the ratio of the amount of time in gc to the total elapsed time exceeds
some threshold, we emit a warning.
I was motivated to do this because publishLocal can take forever in the
zinc project because a 1G heap isn't big enough.

Here it is in action in the zinc project:


feat: add applicant cookie banner

Description :sparkles:
Add cookie banner boiler plate. Needs actual logic for toggling scripts on/off based on user input.

Codecov Report

Merging #2160 (da1bbf5) into develop (7b67120) will decrease coverage by 12.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##           develop    #2160       +/-   ##
============================================
- Coverage    29.98%   17.93%   -12.05%
============================================
  Files          816      122      -694
  Lines        16786     2921    -13865
  Branches      3650      732     -2918
============================================
- Hits          5033      524     -4509
+ Misses       11137     2285     -8852
+ Partials       616      112      -504

Files Changed
Coverage Δ

...benefit/applicant/src/components/layout/Layout.tsx
0.00% <0.00%> (ø)

... and 694 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

API branch is deployed to platta: https://helsinkilisa-pr2160.api.dev.hel.ninja/healthz :rocket::rocket::rocket:


Use sys-uname instead of Platform gem

This PR replaces the Platform gem with the sys-uname gem. This mainly consists of replacing "Platform" with "Sys::Platform", and updating require statements, throughout the code base. I did also have to change :win32 to :windows for Platform::OS checks where used.
Using sys-uname has a few advantages over the Platform gem:

It's current and maintained.
It's an actual interface to the uname function (on Unixy systems), so it's not static information, unlike various Ruby internal bits that Platform uses.
It works properly with JRuby.
It doesn't conflict with other gems with the same name (Platform vs platform).

@matthewd I see you touched this last in 9e606dea9bc6da6ace0d570aab17bec60c88e458, any ideas?

Chatted with @matthewd in irc, his prior change to that test "fixed" an issue where the connection following that test would cause other tests to hang completely.  It did not resolve the sporadic test failure seen here.  We've restarted the failing travis job.

It doesn't conflict with other gems with the same name (Platform vs platform).

:laughing:

It works properly with JRuby.

@djberg96 is sys-uname pure ruby?

@jrafanie Yep. It uses FFI (Unix) or Win32OLE (Windows) under the hood.

It's current and maintained.
@djberg96 you forgot to mention that YOU wrote most of sys-uname so you know it

I like how it's a nearly drop in replacement for the Platform gem.

Looks good to me.

@jrafanie Haha, yep, thought you knew.
It made sense within the context of the sys-uname lib to add a Platform class, too, so that was convenient, and other people might like it better, too.


[FLINK-34343][rpc] Use actor path when rejecting early messages

Prevent an issue where an NPE can occur when an RpcEndpoint is under construction, the rpcServer isnt set yet, but the underlying infrastructure is already set up (aka, the RpcServer already exists) and receives a message.
Here it was possible to access the address of the RpcEndpoint, which would fail if the rpcServer field is still null.
Fix by not relying on the endpoint address but the actor path, which by and large is similar in terms of information (lacks host/port, but we still know which actor was the target).
No test because we cant reasonably test this race condition.

I guess that's worth creating 1.18 and 1.17 backports.

Will do.


Noise in MS2 data

Presently, no noise is added to MS2 peaks.
Need to:

understand how their mz is generated and then
allow the system to either

inherit the m/z deviation from their parent chromatogram or
add (normal) noise, with a sd set from a set of known ROIs

Check that intensity noise is added to the MS2

The skeleton codes to add MS2 noise are already there, but we forgot to actually implement it. See https://github.com/sdrogers/vimms/blob/master/vimms/DataGenerator.py#L579-L585.

Thanks @joewandy -- to do comparisons between DIA and DDA I think we need to implement both (maybe; but no harm in doing it).
Can we add normal noise to both?

Also see related discussion https://github.com/sdrogers/vimms/issues/13

To be fixed in issue #2

Added the following commits:

https://github.com/sdrogers/vimms/commit/d0efae50fa0340a5bc308b14e39216afcd2d0311
https://github.com/sdrogers/vimms/commit/164efd5d20b7cdb0018922f395e7f6a0e6caab1c.

Two noise classes have been added: NoPeakNoise and GaussianPeakNoise.
We can pass an instance of this noise class when initialising the Mass Spec:
        peak_noise = GaussianPeakNoise(0.1, 1000)
        mass_spec = IndependentMassSpectrometer(ionisation_mode, fragscan_dataset_peaks, fragscan_ps, peak_noise=peak_noise)

This class is then used to apply noise when generating the m/z and intensity values of a peak: https://github.com/sdrogers/vimms/blob/master/vimms/MassSpec.py#L592-L598
        noisy_mz_peaks = []
        for i in range(len(mz_peaks)):
            original_mz, original_intensity = mz_peaks[i]
            noisy_mz = self.peak_noise.get_mz(original_mz, query_rt, original_intensity, ms_level)
            noisy_intensity = self.peak_noise.get_intensity(original_mz, query_rt, original_intensity, ms_level)
            noisy_mz_peaks.append((noisy_mz, noisy_intensity))
        return noisy_mz_peaks

I think some student is supposed to improve the noise process? It can be done by improving the classes above, or making a new one.

The noise class is further improved in issue #69.


Injecting to another application requires INJECT_EVENTS permission

Hello! Help please! How avoid java.lang.SecurityException: Injecting to another application requires INJECT_EVENTS permission when use only wrappers from scrcpy. How get permissions INJECT_EVENTS ?
Thanks!

You can't from an Android app. That's one of the reasons scrcpy is run from shell with shell permissions.

@ameaninglessname What if you enable "Security settings"? https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#mouse-and-keyboard-do-not-work


Update automation-troubleshooting-automation-errors.md

Fixed some typos.

Hi @alexandair, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@SnehaGunda Sneha, can you take a look at Aleksandar's contributions to this article and sign-off if they are correct? Thanks,

@alexandair thanks for your contribution. We'll merge the changes.

#sign-off


Dead code attributes with -write are not always idempotent

Running reanalyze https://github.com/rescript-association/reanalyze/commit/389dd682223201b42c33e5c444e5c79e0805adad on Goblint https://github.com/goblint/analyzer/commit/a544002114324a207df72f9e79eb656c3ebe7b5b revealed the following.
Going through the following steps:

dune build @check @all to get cmt files.

reanalyze.exe -write -dce-cmt _build/default/ prints:
Analysis reported 512 issues (Warning Dead Module:20, Warning Dead Type:31, Warning Dead Value:370, Warning Dead Value With Side Effects:13, Warning Redundant Optional Argument:32, Warning Unused Argument:46)

dune build @check @all to get cmt files with written attributes.

reanalyze.exe -write -dce-cmt _build/default/ prints:
Analysis reported 164 issues (Warning Dead Module:8, Warning Dead Type:1, Warning Dead Value:30, Warning Dead Value With Side Effects:47, Warning Redundant Optional Argument:32, Warning Unused Argument:46)

Even though the first analysis adds dead code attributes, the second analysis just keeps on adding more. For example:

At the code from #155, a duplicate attribute is added:include (Batteries : module type of Batteries with module Format := Batteries.Format [@dead "Ana.result.Error"]  [@dead "Ana.result.Error"] )

At most other places, some definitions inside an already dead-attributed definition are marked additionally. For example, here the attribute is added to oc, despite it being contained in print_to_file, which has the attribute from the first run:let print_to_file (fileName: string) (fileAST: file) =
  let oc = Stdlib.open_out fileName [@@dead "+oc"]  in
  dumpFile defaultCilPrinter oc fileName fileAST [@@dead "+print_to_file"]

Everything inside dead-attributed definitions should be assumed dead by default, without getting additional warnings.

If the annotations added are incorrect, there's no way this is going to be idempotent.
So this seems another instance of that other issue with adding annotations.
If both the annotations are added correctly, and the result is not idempotent, it would be interesting to see a small repro example.
Otherwise, this can be closed in favour of the other existing issue.

The case of #155 is the odd one out here. I think all the other cases were such that if an unused function was correctly annotated with dead by reanalyze, then on the second run it finds some variable inside that function as being dead. Which of course it also is, but it would be very inconvenient to add dead annotations to everything inside a dead-annotated function.

There's some logic that if a function is reported dead, then the values inside it are not reported.
Technically, if the function is annotated @dead then it is not reported, so the logic does not apply.
So this is indeed a violation of idempotency.
I thought this was done for a reason, but not sure might be bad memory.
As an aside, in interactive use, you would annotate the function as @live, which does not give this problem.

I think I now remember that idempotency is tested by running a ppx which deletes the entire function when annotated dead, so that's the sense in which it is idempotent. The values inside won't be reported as they disappear.
So in this sense, the process makes sense if one proceeds by either:

marking the function @live, or
deleting the function

I think this process is reasonable, so I'd be inclined to consider this as a non-issue.

So I would say that the issue is about documenting what the intended process is, rather than the current behaviour.

From the readme:

The main difference between @dead and @live is the transitive behaviour: @dead values don't keep alive values they use, while @live values do.

If the intended process only involves using @live or deleting code (so no annotation is necessary at all), then the @dead annotation shouldn't be used at all (other than the output of -write I suppose). But the README also clearly describes usage for @dead:

@dead suppresses reporting on the value/type, but can also be used to force the analysis to consider a value as dead. Typically used to acknowledge cases of dead code you are not planning to address right now, but can be searched easily later.

According to that, there is legitimate reason for keeping @dead annotations around. Blindly changing them to @live just to avoid the non-idempotency issue creates further semantic confusion as the use for @live is described for FFI, etc reasons.
I understand the difference of the two annotations as to their transitive behavior, but there's a subtlety in the phrase "values they use". For example, in the snippet from above:
let print_to_file (fileName: string) (fileAST: file) =
  let oc = Stdlib.open_out fileName [@@dead "+oc"]  in
  dumpFile defaultCilPrinter oc fileName fileAST [@@dead "+print_to_file"]

It's about a dead function print_to_file containing something dead (oc). But the inner definition is dead by virtue of the outer one, so the annotation on it is excessive. This is why I think the dead code analysis should consider the situation of containment specially/differently.
Of course the situation would be different if oc was outside print_to_file and in that case I agree with the @dead vs @live distinction regarding transitivity of usage.
Or if in such dead containment case it is really expected to use the @dead annotation on both (or even more if there are more inner definitions), then the idempotent/consistent behavior would be for -write to add annotations to both on the first run when neither has any annotation.

It's pretty tempting to remove the entire concept of @dead annotation and just suggest to comment-out the code.
Except, one loses the fact that annotated code still compilers and does not code rot while refactoring.
Plus, the ppx has been used as a form of automatic dead code elimination when vendoring libraries or other code.

for -write to add annotations to both on the first run when neither has any annotation.
That would be pretty noisy.

Looks like proper idempotency is the way to go. It's only a minor code complication where dead-annotated declarations are not filtered out early on, but survive until it's time to report things. And they won't be reported but be used to silence inner reports.

This should take care of it: https://github.com/rescript-association/reanalyze/pull/160
There's not really a test for it, as tests run under the ppx, but seems to work.


bau: add apprule tests for console appender

Fixes #1
Also use less hard-coded filenames in the file appender test
Pair: WillPa

:+1: make things open and make them better :)


error

/root/.local/share/virtualenvs/OpenDevin-vZQ1SGmD/lib/python3.11/site-packages/pydantic/_internal/fields.py:151: UserWarning: Field "model_id" has conflict with protected namespace "model".
You may be able to resolve this warning by setting model_config['protected_namespaces'] = ().
warnings.warn(

I have the same error and I don't know if it's linked to this error that comes later during the first prompt:
OBSERVATION:
Error condensing thoughts: No healthy deployment available, passed model=anthropic.claude-3-sonnet-20240229-v1:0
Exited before finishing

I get the same

This is an issue with pydantic and not OpenDevin
https://github.com/pydantic/pydantic/discussions/7276

this is just a warning, nothing to worry about.
Closing in favor of https://github.com/OpenDevin/OpenDevin/issues/797


Add gometalinter to CI

There are a few warnings from gometalinter on the runtime. Would be nice to address them and start using gometalinter on this project.

Damien is doing something else


ICredentialProvider  question

Hello team,
Can you provide ICredentialProvider  sample? How assign    AppId and Password in runtime depending from where from/to request come in?
Thanks

+1  Can someone give an example here? I'm having the same issues.

https://docs.botframework.com/en-us/technical-faq/#how-can-i-support-multiple-bots-at-the-same-bot-service-endpoint points to a comment with some sample code.

Above url is not working . Can you please provide the another one.

https://docs.microsoft.com/en-us/bot-framework/troubleshoot-general-problems#how-can-i-support-multiple-bots-at-the-same-bot-service-endpoint
Where there is a reference to: https://github.com/Microsoft/BotBuilder/issues/2258#issuecomment-280506334 (with an example implementation of ICredentialProvider).


Slice Sampling Improvements

Current coverage is 82.13%

Merging #70 into master will not affect coverage as of 9b3e015

@@            master     #70   diff @@
======================================
  Files           19      19
  Stmts          907     907
  Branches       129     129
  Methods          0       0
======================================
  Hit            745     745
  Partial         32      32
  Missed         130     130

Review entire Coverage Diff as of 9b3e015

Powered by Codecov. Updated on successful CI builds.


Add remix framework as example project using changesets

What is the Change?

Add remix framework as example project using changesets

Motivation For Change

Showcase another popular web framework using changeset package ✨

References

Example usage of changesets package at remix/package.json
Remix website: https://remix.run/

I'm going through these PRs and appending each new ones to last. There's a case in the past where SvelteKit was injected after Astro. Maybe we can revisit sorting it by type, but I'll edit this for now to merge all these PRs first


update privacy grade algorithm

Task/Issue URL: https://app.asana.com/0/392891325557410/709741655065320
Tech Design URL:
CC:
Description:

Implement new Grade calculation algorithm
Remove major networks data and replace with prevalence data
Implement TOSDR scoring algorithm
Use entity list for entity lookups (instead of Disconnect)
Update the UI to support new B+ and C+ grades
Show trackers count rather than networks on dashboard (as per iOS/extensions)

Note: Will do the site report generation as a separate PR (probably).
Steps to test this PR:

Perform a migration and perform thorough test
Perform a fresh install and perform a thorough test

Hacker News (news.ycombinator.com) is a good source of links that should generate the broad range of grades (A, B+, B, C+, C, D)

Internal references:
Software Engineering Expectations
Technical Design Template

Epic review @CDRussell thanks!  Please re-review when you get chance.


Loading models into Realm from a relational REST server

I am trying to reconcile Realm's NOSQL style database with my API server that uses relational modeling.
I set up my Realm models like this:
class Author: Object {
    dynamic var id = 0
    dynamic var profile: Profile?

    override static func primaryKey() -> String? {
        return "id"
    }
}

class Profile: Object {
    dynamic var id = 0
    dynamic var author: Author?

    override static func primaryKey() -> String? {
        return "id"
    }
}

Let's say that my server provides the following JSON responses when queried:
/api/authors/1/
{
  "id": 1,
  "profile": 2
}

/api/profiles/2/
{
  "id": 2
}

And then in my view controller I try to fetch and load my data:
fetchAsync(URL: "/api/authors/1").then { data in
  try! realm.write {
    realm.create(Author.self, value: data, update: true)
  }
  return fetchAsync(URL: "/api/profiles/\(data["profile"])/")
}.then { data in
  // save profile object to Realm
}

But obviously this isn't going to work because my server is returning foreign keys instead of embedding the Profile model inside of Author. Can Realm accommodate my relational data using some pattern that I'm not aware of?

Hi @jaydev, that's a great question! There's nothing currently built into Realm that would allow converting a primary/foreign key mapping into a proper relationship.
How about doing something like this?
var authorID = 0
fetchAsync(URL: "/api/authors/1").then { data in
  try! realm.write {
    let author = realm.create(Author.self, value: data, update: true)
    authorID = author.id
  }
  return fetchAsync(URL: "/api/profiles/\(data["profile"])/")
}.then { data in
  try! realm.write {
    let profile = realm.create(Profile.self, value: data, update: true)
    let author = realm.objectForPrimaryKey(Author.self, key: authorID)
    author.profile = profile
    profile.author = author
  }
}

I'd also recommend that you use inverse relationships to avoid doing the double-bookkeeping you're currently doing (setting both author.profile and profile.author).

Yes, that is not a bad way to go about it, though it requires more bookkeeping that I'd like. But it appears that linkingObjects would not work in this case because the relationship between Profile and Author is one-to-one.

Never mind, just returning the first element of linkingObjects would fix that :smile:

Never mind, just returning the first element of linkingObjects would fix that

Exactly!

Yes, that is not a bad way to go about it, though it requires more bookkeeping that I'd like.

Here's a different way to accomplish the same thing, with a bit less bookeeping, but the author only being created when all the info for author & profile are available:
var authorDict = NSDictionary()
fetchAsync(URL: "/api/authors/1").then { data in
  authorDict = data
  return fetchAsync(URL: "/api/profiles/\(data["profile"])/")
}.then { data in
  try! realm.write {
    authorDict["profile"] = data
    realm.create(Author.self, value: authorDict, update: true)
  }
}


Use typographical apostrophe

Proper typography prefers a  curly/typographic apostrophe ’ rather than a vertical/typewriter apostrophe, ' (see https://en.wikipedia.org/wiki/Apostrophe#Typographic_form). AFAICT this convention applies to all locales.
I suggest replacing all occurrences of typewriter apostrophe in translations with a typographical apostrophe.
Instead of editing all occurrences of Queen’s Birthday, I moved this to a separate translations file (and added a few translations based on Wikipedia).

Thanks! I'll guess I'll start to make some proper contributor documentation: there are quite a number of 'requirements' when creating providers or updates.


Add internal forward-mode rules for ranges

This is part 1 one solving https://github.com/EnzymeAD/Enzyme.jl/issues/274. It does the forward mode rules as those are simpler. A separate PR will do the WIP reverse mode rules as that seems to be a bit more complex.

How's that?

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.

Project coverage is 71.11%. Comparing base (1e15769) to head (b0c3cd1).
Report is 3 commits behind head on main.

Files
Patch %
Lines

src/internal_rules.jl
0.00%
12 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1655      +/-   ##
==========================================
- Coverage   74.03%   71.11%   -2.93%
==========================================
  Files          31       31
  Lines       13016    12832     -184
==========================================
- Hits         9637     9125     -512
- Misses       3379     3707     +328

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

How's this? Is there anything like this batchsize utility?

You should be able to just call width(..). Also per above you should probably do something like
ntuple(Val(width(RT))) do i
Base.@_inline_meta
Thing
End
There are several examples of this in this file


Parameters to scopes

Hi !
I want to define a scopes in my mapper but I need to pass some arguments. How can I do that?
Thanks !
PS : Your project is awesome !!

That is not possible. Scopes are shortcuts to frequently used simple queries.
You should probably add a new method to your mapper instead.

Actually, it should work. The only thing to note is that the $query object will always be the first parameter: https://github.com/vlucas/spot2/blob/master/lib/Query.php#L793

You can just define extra parameters on your scope as normal:
<?php
namespace SpotTest\Mapper;
use Spot\Mapper;
use Spot\Query;
class Event extends Mapper
{
    /**
     * Custom scopes applied to Spot\Query
     *
     * @return array
     */
    public function scopes()
    {
        return [
            'page' => function (Query $query, $page = 1, $items = 20) {
                return $query->limit($items)->offset($page > 0 ? $page * $items : 0);
            }
        ];
    }
}

And use it like this:
$mapper->where(['type' => 'post'])->page(1);


[Contribution Wanted] Logo

The current logo is just a placeholder. Before going really "public" (i.e., declaring this thing - API - stable and doing more blog posts etc.) we want to come up with something really good.
Does anyone have some good idea? Is someone interested in "sponsoring" / "contributing" a cool logo design? Any kind of concepts or sketches would be much appreciated.

Hi @FlorianRappl what about this logo? give me your feedback.

Hi @mirzazulfan that looks promising. Great work! Could you also make one in green?
What are your thoughts @lschoett @iotcloudarchitect ?

What do you think? If you like it, i'll send all files. Regard!

Hi @mirzazulfan, good work, thanks. Do the individual parts of the logo carry a specific meaning?

Sure. I combine the initials p with "O" in the middle as a portal. Thanks.

Available in the preview.


Remove Some Static IDs

A couple of changes to ids.nix, against RFC 52, have slipped through the cracks.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ElvishJerricco should we maybe also add a couple of people to CODEOWNERS for this file? it's still possible to miss the comment on the top if your editor just drops you somewhere in the file (because you used a search function or such) and right now no-one would be notified if someone allocates new IDs.
I don't mind volunteering to get pinged for changes to this file, now that I'm aware :)

@r-vdp Done :) Look good to you?


Fixed Issue #3776

Related Issue
Fixes #3776
Description
This PR fixes the crash reported by #3776. The crash happens while unwrapping <p> tags from the text. There was missing </p> tags. Fix was made to shared MarkDownParser code. The parser code returns immediately when it sees that a given token is marked as Head. This only happens to emphasis and strong emphasis tokens. If a string is a single emphasis (*) and strong emphasis (**), a token can be both head and tail, so returning from translation upon seeing the tag is marked head is a programming error.
How Verified
Added unit tests to cover those cases.
Inspected all codes to make sure that only emphasis and strong emphasis tokens exhibit this behavior.
Microsoft Reviewers: Open in CodeFlow

Hey @nesalang, can you update the title of the PR to be a bit more descriptive/related to what it is that you are addressing rather than just "fixes issue# <>" ? Thanks!


Observation search by species guess broken

Selecting 'species names' under order does add the order_by=species_guess param e.g.
http://www.inaturalist.org/observations?taxon_id=47851&order_by=species_guess&order=desc
vs http://www.inaturalist.org/observations?taxon_id=47851&order_by=observations.id&order=desc
but does not change the order, also the search UI shows order = 'date added' after this search is made rather than the selected 'species name' value that the user chose

mentioned on the google group here https://groups.google.com/forum/#!topic/inaturalist/VyTRAIeFLD4

I believe this is fixed with 57c87f72138213dca63ec62b7139685166df398c and 1afcbff57929b94e338df8dc22e8dcfa5dde3024

looks fixed to me


Audio 1.0!

OST, plays on game start and loops
FX, plays on shooter fire
Removed Engine.log

Commented out Engine.log*


Use debian's iso-codes

Hi. Maybe use debian's iso-codes or this python port? Yes, pycountry is large dependency - about 10 MB, but it contains countries/langs/currencies and their translations. It can be optional dependency.

Originally posted by @dimastbk in https://github.com/pydantic/pydantic-extra-types/issues/14#issuecomment-1471319694

Thanks so much @dimastbk, that's a great idea.
iso-codes looks very useful.
I think we should consider using that, and including the bits of if we need in this repo.
I'm sure if we:

only select the bits we need
are efficient about how we use them, that we can give the size down to something reasonable.


Add the new header for GoCD service API

In the latest GoCD release, a new HTTP header is added in its material notification API: https://api.go.cd/current/#notify-git-materials. As a result, the current GoCD github-services is broken. Adding http.headers['confirm'] = true will fix this. (I have tested it in my GitHub Enterprise instance)
(This patch also converts line-endings to Unix-style)

@zmingxie Can you separate the line ending changes into another commit? It's really hard to follow this otherwise. Thanks!

That said, this is something we consider more than a bug fix as it's a change GoCD has made rather than a bug in our code: https://github.com/github/github-services/blob/master/.github/CONTRIBUTING.md#updating-an-existing-service so I'm not sure if we'll merge this.

@MikeMcQuaid I just back out the line-endings conversion, so you can see this is just a simple one line change. (My editor converted those line-endings automatically earlier, sorry about that!)
I understand these github services are no longer actively maintained, but I've confirmed this is backward compatible with earlier versions of GoCD. On the other hand, thanks to GitHub Supports help, I've patched this in my GitHub Enterprise server. Just want to share this patch in case other people face the same issue after upgrading their GoCD server..

Thanks @zmingxie!


can_fork_chain_at() should ignore blocks below the finalized tip

Motivation
We can't fork the chain at a finalized block height below the finalized tip height, but the sent_non_finalized_blocks data structure contains the heights of some finalized blocks close to the final checkpoint.
We might need to store the queued finalized block list separately, rather than re-using an existing data structure that other code depends on.
(This is an existing bug, but the code in the linked audit PR added more finalized blocks to the data structure.)
Originally posted by @teor2345 in https://github.com/ZcashFoundation/zebra/pull/6335#discussion_r1144234829

Whilst it seems like we want to fix it, I don't think we should classify this as an audit fix...
The reason why we were working on #6335 to close #862 was not because #862 was identified as a potential issue from the audit. Instead, #862 was flagged as an issue that was mentioned in a TODO comment but was closed, potentially making the TODO comment redundant...
We should finish off #6335 and #862 since we have already started work on them, however I would like to prioritise actual audit issues first and fix these other items before a stable release.

I just checked the code that validates and writes non-finalized blocks, it will correctly return NotReadyToBeCommitted error without panicking, even if it sends a block based on these incorrect block hashes. So this is mainly a performance and maintainability issue.
The code that writes finalized blocks ignores the incorrect block hashes, it just uses a single most recently sent hash.

We've actually seen this issue in production, so this is a medium priority.

Hey team! Please add your planning poker estimate with Zenhub @arya2 @conradoplg @dconnolly @oxarbitrage @teor2345 @upbqdn

I just remembered that we reviewed this code, and checked that even if it was called incorrectly, the write block task would return a similar error.
So @Pili I don't think it's a blocker for the stable release at all, and I think it's actually a low priority.


Add <os/lock_private.h> and OSUnfairRecursiveLock

Use os_unfair_recursive_lock on supported Darwin-OS by default.
The downstream user can disable the behavior by setting "OPENCOMBINE_OSLOCK_PRIVATE=0" in their environment.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5113764) 96.84% compared to head (f04ad34) 96.31%.

:exclamation: Current head f04ad34 differs from pull request most recent head 837ee40. Consider uploading reports for the commit 837ee40 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
- Coverage   96.84%   96.31%   -0.54%
==========================================
  Files         106      106
  Lines        8079     8025      -54
==========================================
- Hits         7824     7729      -95
- Misses        255      296      +41

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ROX-17546: Remove support for Central PVC from Helm Chart

Description
This is the first part of deprecate the central support for "stackrox-db" PVC.
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing

[ ] inspected CI results

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

change me!

/test ocp-4-12-qa-e2e-tests

/test ocp-4-12-ui-e2e-tests
/test ui-component-tests
/test ocp-4-12-nongroovy-e2e-tests
/test gke-nongroovy-e2e-tests

/test ocp-4-16-nongroovy-e2e-tests
/test ocp-4-16-ui-e2e-tests


VSTS Copying a Task from a PBI to another

Is there any way to copy Tasks from a PBI to another PBI in VSTS? I haven't found any resources in the VSTS documentation and from searching on google.
I know how to copy a PBI but if that PBI has some tasks, they will not be copied.
I don't want to manually create exact same tasks for different PBIs.

Not sure this is related to the built-in tasks offered by VSTS, seems its more of a general VSTS question.
As far as I know, you can move tasks between PBIs, but since everything is possible in PowerShell I'm sure you can add a VSTS PowerShell script task and in it, use TFS API to read a task and create a new one with same information just with different linked PBI.
If that is indeed how you wanted it to work, via a VSTS task

closing since not an issue with a task in this repo.
Sorry, can you clarify "PBI"?

PBI is Product Backlog Item, its a Work Item in VSTS/TFS to help manage the development work based on Scrum template, that's why at the beginning of my answer I wrote that I didnt think the question related to this repo.


added link to README.md

Summary of your changes
added Wscube Tech under Youtube Channels section and rearranged in alphabetical order.
Description

WsCube Tech: Free full-length tutorials and courses to Learn and develop your career skills.

Checklist

[x] My change follows the Contributing Guidelines
[x] I have added only one new link to the list.
[x] I have checked that the link that I added does NOT exist in the project already.
[x] I have sorted the link alphabetically under the related section.

@sdmg15 please review


Layout.pie: NaN start/end angle when all values are zero

I found a problem when using d3.layout.pie + d3.svg.arc to draw a simple piechart from collection.
When all data values are 0, pie layout returns arc objects with NaN start and end angles (but startAngle of first object is 0). When at least one value is defined, all is fine, and 0 arcs have same start/end angle as expected.
Actually I can check for NaN values by custom svg.arc accessors, but hope you can solve by returning objects with both start and end angle at 0.
You can find an example and a fiddle here:
var pie = d3.layout.pie();
var data = [0, 0];
// pie(data) returns:
[{ startAngle: 0, endAngle: NaN, data: 0, value:0, padAngle: 0 },
{ startAngle: NaN, endAngle: NaN, data: 0, value:0, padAngle: 0 }]

https://jsfiddle.net/et7c4hxt/
Moreover, maybe this request could be the fix:
Layout.pie: Division by zero when sum(values) is 0
https://github.com/mbostock/d3/pull/2420

+1 to the suggested fix. In the meantime I am fixing NaN values in startAngle and endAngle with custom accessors:
var arc = d3.svg.arc()
    .startAngle( function ( d ) { return isNaN( d.startAngle ) ? 0 : d.startAngle; })
    .endAngle( function ( d ) { return isNaN( d.endAngle ) ? 0 : d.endAngle; });

:+1: Also seeing this. Performing a NaN check does seem like a decent way of fixing this.

Covered by #2420.


Optimize DrawTexturePro and DrawRectanglePro transformations

Add check so rotation is only applied if rotation != 0.0f.
Replace matrix usage by calculating the vertex data directly.

@ChrisDill Excuse me for not merging yet, I'd like to do some tests first.

@ChrisDill Thanks for the improvement! I did some tests and the gain is considerable! :)


The method of creating kernels

I noticed that the function for creating kernel ('anisotropic_gaussian_kernel_matlab') is different from the standard gaussian distribution (e.g. the method that used in IKC, _https://github.com/yuanjunchai/IKC/blob/2a846cf1194cd9bace08973d55ecd8fd3179fe48/codes/utils/util.py#L244_). I am wondering why a different way is used here. Actually, a test dataset created by IKC with same sigma range seems to have poor performance on MANet, and vice versa.

In addition, I found a code in the 'util.py line 354 shifted_l = l - scale + 1'. I am curious why a shifting process is needed here to make the maximum point away from the center.

Thanks for your questions. We basically follow the implementations of Kernel-GAN, NeurIPS2019, Sec. 5.2 and USRNet, CVPR2020, Sec. 3.1, in which the blur kernel is shifted and the upper-left pixels are kept in downsampling to avoid subpixel misalignments. The generation of blur kernel is identical to Matlab, as note in https://github.com/JingyunLiang/MANet/blob/34f90ba8888f4a1dd2a1127b97c2ec3706f06598/codes/utils/util.py#L234

Feel free to open it if you have more questions.


[dep] build-essential.

debian, opensuse, Ubuntu can be found via https://pkgs.org/download/build-essential

For some Linux distros that look commonly registered, such as gentoo, fedora, this pkg isn't found. Also confirmed individually:

gentoo https://packages.gentoo.org/packages/search?q=build-essential
fedora https://apps.fedoraproject.org/packages/s/build-essential

Fedora looks like development-tools : https://unix.stackexchange.com/questions/1338/what-is-the-fedora-equivalent-of-the-debian-build-essential-package
Maybe @allenh1 or @awesomebytes could suggest a gentoo equivalent?

So, in the case of gentoo, that will be there for sure. we could, I suppose, introduce a virtual package that would pull in those deps (if it is needed for some package), but the build essential is most equivalent to the stage3 you start with.

@130s what is the reason for adding this department rule?

@allenh1 I have a corporate internal pkg that builds Debian and it depends on this pkg.

If those tools are expected to already be installed on gentoo, we can just give it an empty list to say that nothing needs to be installed gentoo: []

It just came to my mind that on a Gentoo install based on binary packages it isn't necessarily true that build-essentials stuff will be there (in fact, I just came up with not having CMake in such a system).
Maybe gentoo could depend on:
gentoo: [sys-devel/gcc, sys-libs/glibc, sys-devel/make, sys-devel/binutils, app-arch/bzip2, app-arch/xz-utils, dev-lang/perl, dev-libs/gmp, dev-libs/mpfr, dev-libs/mpc, sys-devel/gcc-config, sys-libs/ncurses, sys-libs/readline, app-shells/bash, app-arch/xz-utils, sys-apps/sed, sys-apps/baselayout-prefix, dev-libs/libffi, sys-devel/m4, sys-devel/flex, sys-apps/diffutils, sys-devel/bison, sys-devel/patch, sys-devel/binutils-config, sys-apps/coreutils, sys-apps/findutils, app-arch/gzip, app-arch/tar, sys-apps/grep, sys-apps/gawk, sys-apps/file]

I came up with this list by checking what does build_essentials in Ubuntu provide + what is bootstrapped on Gentoo on the first stage of bootstrapping (https://gitweb.gentoo.org/repo/proj/prefix.git/plain/scripts/bootstrap-prefix.sh). Maybe there is something missing, or something extra.
There may be a better way, some meta-package that already brings all the necessary, but I don't know it and haven't found it.

Thanks for the feedback. I'm not sure if I understand all feedback, so I appreciate if each of you could review @allenh1 @awesomebytes @tfoote

That looks good as far as gentoo tag is concerned (in my opinion), but I defer to @tfoote's opinion.

Archlinux has the base-devel package group which is roughly equivalent to build-essential, although it includes the packaging utilities for Arch rather than dpkg-dev.
However, I'm not sure that we necessarily want to try and incorporate non-debian-derived distros into this key by assuming that a package which depends on build-essential would function adequately on another distro with its packaging tools.
The Debian Policy manual also explicitly recommends against adding build dependencies on build-essential unless restricting the dependency to a specific version. Depending on build-essential without a version restriction is a lintian error.

I'm fine with leaving out the non-deb people, since I'm not aware of any packages getting blocked by this one.
We'll make a key & a virtual package for gentoo if it becomes an issue.

Archlinux has the base-devel package group which is roughly equivalent to build-essential, although it includes the packaging utilities for Arch rather than dpkg-dev.
However, I'm not sure that we necessarily want to try and incorporate non-debian-derived distros into this key by assuming that a package which depends on build-essential would function adequately on another distro with its packaging tools.

I'm not sure either. As a user, I would expect exactly the same software (despite version difference between distro) if the rosdep key is the same. But I'm not sure, as @tfoote suggested a package for Fedora that is not exactly the same software https://github.com/ros/rosdistro/pull/17751#issuecomment-386731757, that might be ok in ROS land?
@nuclearsandwich

The Debian Policy manual also explicitly recommends against adding build dependencies on build-essential unless restricting the dependency to a specific version. Depending on build-essential without a version restriction is a lintian error.

Thanks for the pointer. Are you suggesting to not add a rosdep key for build-essential, or is it just a warning when one uses this key? Personally I still like to add this key as usage should be up to users (developers).

Are you suggesting to not add a rosdep key for build-essential

Thank you for asking and giving me the opportunity to voice my concerns. I think it has a high potential to cause confusion and cross-platform headaches if incorporated in the general rosdep database. I would like to share two alternatives which I think keep this rosdep database optimized for approachable cross-platform use:

Identifying what members of build-essential are required by the internal package and depending on them directly. Contributing those rosdep keys upstream.
Including build-essential in an additional list of rosdep keys used internally by your organization.

I recognize that both require extra effort for you and your team but I think that it can save effort across the community preventing dependence on a virtual package with a lot of cross-platform nuance where I cannot see a generically suitable resolution.

Personally I still like to add this key as usage should be up to users (developers).

I hear you. Flexibility is paramount to successful community and it is not my intention to exercise anything like a power of veto, even if I had such a thing. I also don't want to discourage contributing rosdep definitions upstream. However, I think this case in particular, and platform-specific virtual packages / package groups in general, are worth the additional deliberation we've had to consider whether their inclusion has a net benefit. As I understand it, rosdep's architecture enables you to augment and modify the rosdep database by including additional sources on your machines so we are not preventing your usage of the key but rather preventing its use by default.

Since build-essential is a debianism that both doesn't have equivalents on other platforms and even by debian standards is generally not recommended as a dependency due to it's virtual nature. I think it does make sense to not put this in as a rosdep rule and recommend that the more specific dependencies that are actually needed. (It has some significantly different dependencies, from multiple compilers to runtime libraries and build systems. for which several of them already have rules available) There is a level of flexibility we want to leave to the users to choose. But if we cannot keep a consistent resolution across all platforms that will devalue the rosdep database as it won't reliably be able to resolve across platforms. We also have to limit choices that cause conflicts for better usability.

I see. I just did not think of such a complication this PR would inject. Case closed.

I would like to share two alternatives which I think keep this rosdep database optimized for approachable cross-platform use:

Noted. Thanks for the suggestion.


Matrix Shield for ESP32 and horizontal lines not displaying

Hi everybody.
I just bought a RGB Matrix Shield for ESP32 dev module with two diferent wirings: Smartmatrix and P3RGB64X32 (Neocat)

Out of the Box is incompatible with this library, so i follow the wiring in order to find out the real pinout of both ports.
The result:
`//SMARTMATRIX
#define R1_PIN 2
#define G1_PIN 15
#define B1_PIN 4
#define R2_PIN 16
#define G2_PIN 27
#define B2_PIN 17
#define A_PIN 5
#define B_PIN 18
#define C_PIN 19
#define D_PIN 21
#define E_PIN -1 // required for 1/32 scan panels, like 64x64. Any available pin would do, i.e. IO32
#define LAT_PIN 26
#define OE_PIN 25
#define CLK_PIN 22
//P3RGB64X32
//#define R1_PIN 25
//#define G1_PIN 26
//#define B1_PIN 27
//#define R2_PIN 21
//#define G2_PIN 22
//#define B2_PIN 23
//#define A_PIN 12
//#define B_PIN 16
//#define C_PIN 17
//#define D_PIN 18
//#define E_PIN -1 // required for 1/32 scan panels, like 64x64. Any available pin would do, i.e. IO32
//#define LAT_PIN 32
//#define OE_PIN 33
//#define CLK_PIN 15`
Well, then i modify the examples here in order to use custom GPIOs.
I´ve tried both ways, from the skecht with define i2S pins: HUB75_I2S_CFG::i2s_pins _pins={R1_PIN, G1_PIN, B1_PIN, R2_PIN, G2_PIN, B2_PIN, A_PIN, B_PIN, C_PIN, D_PIN, E_PIN, LAT_PIN, OE_PIN, CLK_PIN};
and modifying the "ESP32-HUB75-MatrixPanel-I2S-DMA.h" file.
Both ways result in displaying with the same horizontal separation/missing  lines:

Any ideas of how to deal with this?
Thanks in advance

Something is wrong with the B_PIN

Hi @GrimorJander, any update on this issue? I'm facing kinda similar issue.

Anyone can assist me on this issue? Thank you

@secangkirkopipanas
I really didn't find a solution.
I have simply stopped using the board with this library.
The adapter works fine with other libraries, and when I modify the pinout in the code to make it compatible with this one, the problem appears.
Likewise, the library works fine with the default pinout, so I made my own adapter.
I only have 64x32 panels, so it´s more standar and i dont have to modify "ESP32-HUB75-MatrixPanel-I2S-DMA.h" file .

Looks to me like the 'A' line is missing....

That looks like the VirtualMatrixPanel test pattern?

Try not using the virtualmatrix class, you don't need it for just a horizontal chain.

Looks to me like the 'A' line is missing....

Agreed
And also looks like the use of the VirtualMatrixPanel class which is not required on a simple horizontal chain.

Hi all, thank you for your response. My bad, I realized that I was using WROVER for this. Not an issue at all once I changed to WROOM.

This is just my simple testing on 64x64 chained panels. I will extend it to 6 panels with 2 rows and 3 columns.

My bad, I realized that I was using WROVER for this. Not an issue at all once I changed to WROOM.

Is the problem solved now?
By the way - why there are 4 HUB connectors on the shield - after all, neither library supports parallel output to 2 channels, as far as I know

You have to choose just one from 2 diferent pinouts (P3RGB64X32 or SMARTMATRIX) and two variants (3.3V or  5V with the level shifting buffers.
https://www.tindie.com/products/lightwell/rgb-matrix-panel-shield-for-esp32/
Seens good product (the only board available when i bought), and my only complain is can`t make it work with this library.


Feature/error settings

Added two error settings. When I turned on tslint for our project it kicked out something like 16000 errors and it was impossible to deal with. This adds a reporting cap and a way to change the default delimiter used in the error reporting.
I added tests and ran all of the existing ones and didn't see anything out of line.

Addressed feedback; sorry about that.

Thank you for the PR!


STYLE: Make PrintSelf implementation style consistent

Make PrintSelfimplementation style consistent following the ITK SWG coding style guideline and the available ITK macros:

Use the os << indent << "{ivarName}: " << m_ivarName << std::endl recipe: do not print the leading m_ of the ivar; use a colon after printing its name; leave a whitespace between the colon and the printed value of the ivar.
Always print the superclass first.
Use itkPrintSelfObjectMacro to print objects that can be null.
Remove statements printing the hard-coded class name and its own pointer, e.g. os << indent << "ArrowSpatialObject(" << this << ")" << std::endl; PrintSelf does this without requiring the removed line.

Apply the same principle to the << operator overloads.
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

Looks great! There are still a few compile errors.

I know. Still WIP/draft. I have other changes locally, but unsure whether I should split it across at least 2 PRs, as it is becoming difficult to deal with.
I also grant that some changes belong to ENH, while others are more STYLE, but it is not evident to me that they should be split/how to split them without creating dozens of commits/PRs then even more difficult to deal with locally.

Having it as a single commit is fine. I know that splitting into themed commits requires extra effort.

Amazing work getting this more consistent!

Use itkPrintSelfObjectMacro to print objects that can be null.
It may be clearer to say that this should be used for all pointer like objects, as pointers could be null.

One other issue that some time occurs with the print self methods is when "pixel types" are char. When they are a char then they are printed as an ASCII char and not a numeric value as expected. This is a case where the NumbericTraits::PrintType must be used.  This is clearly separate work.

Amazing work getting this more consistent!

Thanks Brad.

Use itkPrintSelfObjectMacro to print objects that can be null.
It may be clearer to say that this should be used for all pointer like objects, as pointers could be null.

Will rework the commit message as time permits and as I revise all the changes contained.

One other issue that some time occurs with the print self methods is when "pixel types" are char. When they are a char then they are printed as an ASCII char and not a numeric value as expected. This is a case where the NumbericTraits::PrintType must be used. This is clearly separate work.

Yes, had that in mind, but did not take the chance to make the changes. Will try to do that for the ivars that I'll be streaming, but will not be checking the existing ones as a general rule.
Note that this process is not automated, and I do not intend it to be comprehensive across the entire code base, but just a first step: I have mainly looked at some given patterns (like containing equal signs, etc.). Also, it will take me some time until I can get a working version. For now, it is still WIP.

I assume this will be ready for review after a rebase?

I assume this will be ready for review after a rebase?

Thanks for the follow-up @dzenanz. Yes, I need to rebase. I also have some other local changes that I need to add, and I will also do this:

One other issue that some time occurs with the print self methods is when "pixel types" are char. When they are a char then they are printed as an ASCII char and not a numeric value as expected. This is a case where the NumbericTraits::PrintType must be used. This is clearly separate work.

And adapt the commit message.
Probably will be ready during the weekend. Note that it will still not be comprehensive as this is not automated.

M:\Dashboard\ITK\Modules\Numerics\FEM\include\itkImageToRectilinearFEMObjectFilter.hxx(302,3): error C3861: 'itkPrintSelfObject': identifier not found [M:\Dashboard\ITK-build\Modules\Registration\FEM\test\ITKFEMRegistrationTestDriver.vcxproj]
  itkPrintSelfObject(Material);
  itkPrintSelfObjectMacro(Element);

Does itkPrintSelfObject exist? S. this line

Thanks @jhlegarreta.
BTW, there are many other errors from this PR in Modules/Registration/FEM, s.
https://open.cdash.org/viewBuildError.php?buildid=8462352

BTW, there are many other errors from this PR in Modules/Registration/FEM, s.
https://open.cdash.org/viewBuildError.php?buildid=8462352

Oh, my bad. I did not have the FEM modules turned on and blindly relied on my local build/CI results, and did not spot those in the yesterday dashboard result. Thanks for the heads-up @issakomi.

Again, thanks for the heads-up @issakomi. You're right, it should be itkPrintSelfObjectMacro. I'll push another patch set.

Oh, my bad. I did not have the FEM modules turned on and blindly relied on my local build/CI results, and did not spot those in the yesterday dashboard result. Thanks for the heads-up @issakomi.

PR #3918.


MIRI review changes

Summary of changes

For MIRI data: adjusted pedestal to be pedestal = group2 - pedestal.  Kept the original pedestal for marking saturated pixels
added 2 plotting routines for noisy pixels
added a routine: read_slope_integration - to replace read_slope_file to handle reading in multiple integrations when there are different number of integration in different exposures. Vstack did not work for multiple integration data when the number of integrations varied between exposures
Pulled out saturated data based on DQ flag of group 1 from the low_pedestal flagging

good suggestion on the outdir for the plots. I added that.


Adding OpenApi Schema Properties on Models

It's a very initial implementation, in order to work in the same way it should also be extended into the ReflectionClass package to support it from this site better.

Please try to reduce (remove) all unrelated codestyle changes, hard to digest exactly what you're adding

I removed the codestyle changes.

As I'm not using OpenApi, it's hard to digest how this works.
Before having to write a test, can you share some examples how it works, what's the model look like before/after?

Here is the official OpenAPI 3 Schema Specification
http://spec.openapis.org/oas/v3.0.3#reference-object
Bellow I have a model example:
/**
 * App\User
 *
 * @OA\Schema ()
 * @mixin \Eloquent
 * @property int $id
 * @property string $name
 * @property string $email
 * @property string $password
 * @property string|null $remember_token
 * @property string $role
 * @OA\Property(property="id",type="int",description="")
 * @OA\Property(property="name",type="string",description="")
 * @OA\Property(property="email",type="string",description="")
 * @OA\Property(property="password",type="string",description="")
 * @OA\Property(property="role",type="string",description="")
 * @OA\Property(property="remember_token",type="string|null",description="")
 */

As you can see, I am adding the @OA\Schema () to the object and then I am adding the object properties as @OA\Propery. With this the composer package darkaonline/l5-swagger can detect that and define it as a OpenApi Schema with the following properties. The result specification can use this generated schema into controllers and other annotations. It also generates the complete OpenAPI specification in json and/or yml format (so you can view it in Swagger Editor).

My gut feeling: this doesn't belong into this ide-helper ; rather it could be a plugin (*) for ide-helper, because that's really a outside-laravel-framework use-case and depends on a third party package.
(*) there is no dedicated "plugin" support, rather I'm meaning some kind of hack/integration/workaround
Just my 2c, I don't feel this "3rd party dependency special case" belongs into the library.

But I would be all for it allowing other pacakges to easily hook into ide-helper to provide their additional logic

Yeah, your perspective looks fair enough, but this plugin is already doing a great job and re-inventing the wheel in order to do an additional thing looks not smart. Having a hooking/attaching an other plugin into that and allow other plugin vendors to extend it's functionalities sounds a wonderful idea.
Is there a way for Feature suggestion in order to start a discussion on that part?

Probably a good to ping @barryvdh what he things :-)

I understand that it kinda makes sense if you're doing much of the same logic already. But not sure if it's in scope for this project. And OpenAPI doesn't always map directly to a model, does it?

And OpenAPI doesn't always map directly to a model, does it?

Models are converted to OpenAPI Schemas and then they can used as $ref into the rest of the API-Spec (as request body, response, etc.)

I firmly believe this shouldn't be in this package as it's very narrow focused.
Rather it can live in your code base or a package based on this, providing an adjusted command which hooks into the places where you need the modifications.
But as already mentioned, I would be all-in ensuring this is technically possible.


How to view image clipart?

I tried to view diagrams from /mxgraph-master/javascript/examples/grapheditor/www/viewer.html.
My xml:
<mxGraphModel dx="1361" dy="839" grid="1" gridSize="10" guides="1" tooltips="1" connect="1" arrows="1" fold="1" page="1" pageScale="1" pageWidth="850" pageHeight="1100" background="#ffffff">
  <root>
    <mxCell id="0"/>
    <mxCell id="1" parent="0"/>
    <mxCell id="4" style="edgeStyle=orthogonalEdgeStyle;rounded=0;html=1;jettySize=auto;orthogonalLoop=1;" edge="1" parent="1" source="2" target="3">
      <mxGeometry relative="1" as="geometry"/>
    </mxCell>
    <mxCell id="2" value="" style="image;html=1;labelBackgroundColor=#ffffff;image=stencils/clipart/Earth_globe_128x128.png" vertex="1" parent="1">
      <mxGeometry x="200" y="250" width="80" height="80" as="geometry"/>
    </mxCell>
    <mxCell id="5" style="edgeStyle=orthogonalEdgeStyle;rounded=0;html=1;jettySize=auto;orthogonalLoop=1;" edge="1" parent="1" source="3">
      <mxGeometry relative="1" as="geometry">
        <mxPoint x="690" y="300" as="targetPoint"/>
      </mxGeometry>
    </mxCell>
    <mxCell id="3" value="" style="image;html=1;labelBackgroundColor=#ffffff;image=stencils/clipart/Virus_128x128.png" vertex="1" parent="1">
      <mxGeometry x="460" y="260" width="80" height="80" as="geometry"/>
    </mxCell>
    <mxCell id="6" value="" style="rounded=0;whiteSpace=wrap;html=1;" vertex="1" parent="1">
      <mxGeometry x="700" y="270" width="120" height="60" as="geometry"/>
    </mxCell>
  </root>
</mxGraphModel>

My xml had image cliparts. When I click show button, The image cliparts was changed into square.
Help me, please! Thanks!

I tried debug with viewer.html and index.html. I found a litle reason:

viewer.html: in func mxShape.prototype.paint (mxShape.js), in line 891, this.paintVertexShape(c, x, y, w, h)  is function of mxShape.js.
Index.html: in line 891, this.paintVertexShape(c, x, y, w, h)  is function of mxImageShape.js.
Because when I used viewer.html that not showed image, it changed to square.

Hi @alderg, Can you help me, please?

For programming questions on mxGraph, please post on Stack Overflow, using the mxGraph tag, http://stackoverflow.com/questions/tagged/mxgraph.

@davidjgraph thank you, I also posted in stackoverflow :)

I posted in stackoverflow. But nobody answer. I have a idea: I use editor to view diagram. I want to set div.geBackgroundPage (child of div.getDiagramContainer) full size, not top, not left, not right, not bottom => div.getDiagramContainer doesn't have scrollbar. What do I must do?

I am having the same problem now @tanobi92 with clipart images not loading. Did you ever solve the problem?

@JacoBezuidenhout , I used /mxgraph-master/javascript/examples/grapheditor/www/index.html to load xml in stead of /mxgraph-master/javascript/examples/grapheditor/www/viewer.html.


events pages could be easier to navigate

From https://okfn.org/get-involved/ -> "find an event near you" -> https://okfn.org/get-involved/events/ which has a nice listing of international events but nothing near me. This phrasing should link direct to : http://www.meetup.com/OpenKnowledgeFoundation/  (I think).
Also it was hard to find the meetup list at all, i read the main body of https://okfn.org/get-involved/events/ and assumed there was no local events, but then i found a tiny link in the sidebar!  We should have the same content linked from the explanatory text on a page as in the sidebar, here.

WONTFIX


[core-http] remove cookie support

Resolves https://github.com/Azure/azure-sdk-for-js/issues/24654

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-core-http