Ovest - Cannot read property 'startsWith' of undefined

Hi,
I am getting the following error when I try to grab a picture.
topic: "Ovest"
errorDescription: "Cannot read property 'startsWith' of undefined"
payload: true
_msgid: "5c3dff57.46454"
Thanks, Jens

My mistake...


Best way to destroy and recreate a tilemap?

Hello and thank you for this awesome crate!
I'm encountering an issue with the 0.11 version while attempting to make a save system:
I've an exclusive system that removes all the entities (including tiles and tilemap) and then spawns a new tilemap with fresh storage and tiles. It all runs in a specific state. It works fine but however I randomly get the following messages in the log:
Failed to 'insert or spawn' bundle of type bevy_ecs_tilemap::render::extract::ExtractedTilemapTextureBundle into the following invalid entities: [374v2]

Am I doing something wrong? Is this a bug?

I'm going to close this because after version 0.15 so much has changed in the related code that it seems unlikely that this precise bug could still be present. Please feel free to open a new issue.
We also just added a new spawn_and_despawn_tilemap example.
Sorry for the very long delay in responding.


Multiple featherlight.js galleries on page

@marcandre Hello,
Filter doesn't work for me, or I'm missing the point. Please help, my code is below.

Thanks

SO question answered. Closing this.
I guess I should start offering paid support for this type of questions.

Hi!
My name is Marc-André and I'd like your help. I'm not the original author but I liked the project so much I started contributing more and more; now I'm the (only) maintainer.
How you can help
I'd like you to help me in keeping the issues of this project organized. In particular, I'd very much like questions to be asked on Stack Overflow(reasons just down below) and I think you could help.
To serve that goal, I wrote that questions should be asked on Stack Overflow in the READ ME file. Still, some people asked questions here.
I requested the same in the Contributing Guidelines. Even though GitHub has a paragraph saying "Please review the guidelines for contributing to this repository." that shows up whenever someone creates an issue, some people still asked questions here.
Finally, GitHub allowed us to write a template issue, in which I wrote

This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use
http://stackoverflow.com/questions/ask .
Provide working code, starting from http://jsfiddle.net/JNsu6/15/.
We monitor the tag featherlight.js.

Still, some people like you delete this text and ask a question here. So my question is: why did you feel it was the right thing to do? What could I possibly have done so that this basic wish was respected? I must admit that I feel disrespected whenever someone completely ignores my requests. I understand that most developpers don't feel the urge to help by digging in, inspecting the code and contributing back with great bug reports or pull requests, but it saddens me when people show little care and disregard maintainer wishes.
It would help me a lot if you helped me find a solution so that questions are asked on Stack Overflow.
Why StackOverflow?
I feel that StackOverflow is much better organized for questions and answers than GitHub. Not too surprising, that's exactly what it was designed for!
It has a much better interface to find similar questions, so there's a better chance a question and its answers will help someone else.
I monitor the tag for this project, so I normally will read the question. Sometimes, someone else has already answered the question, which is awesome for whoever wrote the question and for me! That never happens with GitHub issues, btw.
Otherwise I can answer it, especially if it shows what the requester has tried, and has a working example. If something is not clear, there's the comment section, that insure that the discussion doesn't have to be re-read by others that only want the answer.
If the question is badly asked, for example if it has no working example, I can also ignore it and downvote it. Maybe someone will answer it, either way, I don't have an opened issue being stale in GitHub forever.
Indeed, I try as best as I can to keep the list of open GitHub issues down to a minimum. Things there should mostly be things that need to be fixed one way or another. Having a bunch of half baked questions in there would not help at all.
I'm hoping you will reply with good hints on how to achieve this goal. Thanks.

Thanks for answering my question but my problem is not solved. Maybe it is not possible yet and it is a feature request!?


zpages HandlerFuncs are named Handler

Both RpczHandler and TracezHandler are HandlerFuncs but they are named as handlers. It is confusing for the user.
/cc @ramonza

I think we should deprecated them. Having to consider using one or the other independently complicates the HTML. For example, it precludes creating links between them.

Closed via 6e008f8971f53840fe7b528ab3d733a38e54a4bb.


Using README as title page?

Is there a way to use the readme as the initial page? Maybe I'm just missing it in the docs...
this is what I have, which executes jsdoc well, but doesn't have the title page (like your docs :))
    return Docma.create()
      .build({
        src: [
          './index.js',
          './README.md'
        ],
        dest: './docs'
      });

Sure. You just need to set app entrance.
{
    src: [
      './index.js',
      './README.md'
    ],
    dest: './docs',
    app: {
         entrance: 'content:readme'
    }
}

See build confoguration docs and this repo's own docma.config.json for an example.
Closing this. Pls re-open if you need to.

cheers! thanks for the quick response


Search should take precedence

Issue Type: Feature Request
With the recent update, the search function is not a separate tab but part of the explorer.
However, in explorer, the folder structure pane and the open editors pane is usually expanded. In this case, search results will be covered up as the search pane will not expand and take over most of the space.
The search pane does not offer scrolling in this case either. I am basically being forced to close the folder structure pane in order to do searches.
I think I'd prefer either of the solutions:

Search pane should be able to expand to compress the folder structure pane, I am fine with needing to manually close search pane in order to use the folder structure pane.
Expand which ever pane that I click into.

Apologies if there are duplicated issues on this already.
VS Code version: Code 1.52.1 (ea3859d4ba2f3e577a159bc91e3074c5d85c0523, 2020-12-16T16:30:02.420Z)
OS version: Darwin x64 19.6.0

Can you please provide screenshot.
Also are you sure you haven't dragged Search to Explorer pane by accident? You should be able to drag it to panel or  action bar to make it separate.

(clean 1.52.1 installation)

As @IllusionMH said it sounds like you've possibly dragged the search view out of its default location. You can right click on the header and select "Reset Location" to move it back. (or drag it back to the side bar)

Whoah. I did not know that could be dragged around.
In that case, I am pretty sure the culprit would be my cat, threading on my touchpad.
Thanks @IllusionMH @JacksonKearl for the help!


"Fauxbar Lite: Add as search engine..." quietly fails

The "Fauxbar Lite: Add as search engine..." context menu entry offered in search fields like the one on Wikipedia fails to work in Google Chrome 107.0.5304.107. It fails quietly, without creating a new search engine.
This was reported on Fauxbar Lite reviews as early as 2019.

Yeah, Fauxbar (and Fauxbar Lite) hasn't received any updates since 2019 due to:

Lack of motivation
Chronic health issues on my end
WebSQL being deprecated
and lately, Chrome Manifest V3 being required coming up, which will render Fauxbar completely dead.

I've dabbled with refactoring Fauxbar to just use JavaScript arrays and objects, without relying on a database, but the whole project is so intertwined and built around WebSQL, that it is probably most feasible for Fauxbar to be rebuilt from scratch.
As such, yep it's broken in places, and will probably remain broken, and the whole extension will stop working soon in 2023.
Don't want to close this GitHub (freeze on ice), but I'd recommend not getting attached to anything.
Time will tell if I get so frustrated at Fauxbar not working at all in the coming months that it motivates me to do something about it again, but I do not know yet.

Thank you Chris
Considering the importance of this bug and its persistence, can someone with the necessary permissions "pin" this ticket?


main sync from beta/1.1.9

Description:
Apply beta/1.1.9 fixes to main.
chore:
chore(v2i_interface) : send udp log output disable https://github.com/eve-autonomy/v2i_interface/pull/47
Tests performed
this PR is applying beta/1.1.9 fixes to main.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

1.1.8のとき、同様に自分のマージcommitが残るため,mainから削除するように対応を実施


Fix typo

<p/> -> </p>, https://GitHub.com, https://Github.com -> https://github.com

Verified editorial corrections.


Fails to render events (border style) if Array prototype have been extended

"_renderDataSourceDay" should check if events.hasOwnProperty(i), like this:
...
for(var i in events)
{
if (events.hasOwnProperty(i)){
if(boxShadow != '') {
boxShadow += ",";
}
boxShadow += 'inset 0 -' + (parseInt(i) + 1) * weight + 'px 0 0 ' + events[i].color;
}
}
...
In my case, another library had extended Array.prototype with a remove method, and it ended up with the calendar not displaying all Events.

Hi,
Sorry for the late answer. The problem should now be fixed, I replaced all the "for" loops in the code (see commit 709c3bea2d79c8eeb7a178f744d112f059eaeb7f)

Thanks for fixing, and I'm sorry for being late in closing this issue (not familiar with Github, so I wasn't aware I needed to close it).


V2.3.1 Headset not found - firewall disabled

I'm using ALVR v2.3.1 - both server and client app on Oculus Go.
Getting headset not found message from server.

Now seems to be able to detect and connect for a few seconds and then disconnect.


cmd/link/internal/ld/elf: does not write dynamically imported syms correctly

Even on x86, Go's internal linker does not write .dynsym / .symtab entries as expected.
In particular, functions imported from libraries such as libc.so (e.g. via loadcgo) when written to .dynsym / .symtab are listed as OBJT (STT_OBJECT) instead of as FUNC (STT_FUNC).
Unknown what effect this is having at this time.

Note that this expectation is not unique to Solaris; a "readelf -a" of various executables on a Linux system shows the same expectation.

I remember that this is intentional, although I do not remember the reason.

The explanation I found (but can't find again at this moment), was roughly that since Go considers C symbols that are imported to be unique to each package, they don't want them "exported" in a way that affects linking.
However, that would only make sense when Buildmode != BuildmodeExe.  I'm just leaving this open so that I can investigate at a later point or if I discover it to be causing other problems.


[Source] thebeatles.com/my-life

https://www.thebeatles.com/my-life
https://www.thebeatles.com/songs
I didn't find a song that is not available in current sources but,
Official sources should be preferred because it is correct
Why make source for 1 band ? thier lyrics are good.
Most popular old rock band.

I don't think it's not worth the initial (and ongoing) development effort to build an extra source just for a single artist, especially not when lyrics for those artists are already widely available on other sources. True, they might be considered the canonical source, but many lyrics websites (such as Genius) allow/encourage community contributions.


Make resolution cache use sets instead of arrays for easier checking

@typescript-bot perf test this faster

After talking to @DanielRosenwasser and confirming that his perf concerns were infact noise merging this in


dockerfile plugin output docker/image-name, not docker/image_name

unfortunately there is no :embarrassed: emoji on github

@gimaker @davidxia @negz

👍


pr/notifier: Switch to bazel and update

The newer tool has 3 modes - normal, dry_run, and report
report dumps a report, whereas dry_run shows what would be posted to slack
This PR enables the ci in pr runs using --dry_run so you can see what would be posted

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

you can see output here https://github.com/envoyproxy/envoy/actions/runs/6470490290/job/17566815179?pr=30054#step:3:88

@jmarantz i spotted a couple of mistakes, will update now ...

updated, hopefully should be good to land

arg, still not quite right - debugging

seems to work correctly now


Fix browsertest M1 builds and optimize node modules

Description

Removes the explicit use of tini  (replaces it with the docker --init flag)  in order to fix the m1 qemu image error:
qemu-x86_64: Could not open ‘/lib64/ld-linux-x86-64.so.2’: No such file or directory
Moves the node install to the image build and save it to an anonymous volume to prevent overwriting/contamination with local file system.
Deletes package-lock, since this uses yarn to install packages

Release notes:
n/a
Checklist

[x] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
Fixes browsertest for m1 macs

Codecov Report

Merging #3187 (0819e34) into main (f1688af) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3187   +/-   ##
=========================================
  Coverage     78.48%   78.48%
  Complexity     2219     2219
=========================================
  Files           280      280
  Lines          8914     8914
  Branches        599      599
=========================================
  Hits           6996     6996
  Misses         1696     1696
  Partials        222      222

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Part 1: Data scraping

Part 1: Data scraping
In this phase, you will focus on collecting data from the web in order to compute the following metrics:
Daily death rates
Cumulative death rates
Data should be both "raw" (absolute numbers) and normalized by population (relative to 1M of people). Collected data should be separated by country, and stored on the disk in the JSON format to facilitate searching of information. You can aggregate data from multiple online sources, but you should keep track of the provenance from each data point.
For this part, you are expected to commit both your scrapping code, and the JSON file on github for at least two consecutive days. Your code must implement a module named ScrapeWebsite that contains a function scrape_country. scrape_country should have at least two input parameters: the name of a country, and a website for retrieving covid stats for said country. scrape_country should return data associated with a given country.

[ ] What countries/how many? (need 3 to 10) --> USA, France, Brazil
[ ] Which websites (need at least 3 sources)? Average of websites, or output for three different sources for comparison purposes (whichever is easiest) (Elliott)

https://coronavirus.jhu.edu/map.html
https://www.worldometers.info/coronavirus/
https://www.nytimes.com/interactive/2021/us/covid-cases.html

[ ] Need code to convert data to JSON (daily death rates, cumulative death rates) (Laura find code)
[ ] Claire find how JSON needs to look for input.


updated syllabus to make sense

@decause I spoke with AJ and these seem to align with the schedule that you are thinking of. With GDC and the previous year's schedule this seems to pack a lot into a small time so let me know if you have any comments.

:100:


Update from Go 1.7 to 1.8

Increase build speed by as much as 15%
More info on Go 1.8: https://blog.golang.org/go1.8

PG1 1363

PG approved. @bjustin-ibm, please review/merge. :)

Confirmed blue pg.


Idempotent param

cc @kyleknap @JordonPhillips @jamesls

Current coverage is 97.62% (diff: 100%)

Merging #1081 into develop will increase coverage by <.01%

@@            develop      #1081   diff @@
==========================================
  Files            44         44
  Lines          6936       6948    +12
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6771       6783    +12
  Misses          165        165
  Partials          0          0

Powered by Codecov. Last update 5b0bcf3...721ab59

I think all that's really missing now is documentation for idempotent parameters.

I changed the wording to match elsewhere in the docs.

Moved everything to shape.py and params.py I found that the metadata dict is populated with the idempotency property, so it was easier than I thought it would be. Still probably needs to be made more general.


Migrating old sites to Gem-based themes

Instructions
1. Determine any special functionality of this theme
If your theme has any customizations to the docs/src/_layouts, docs/src/_includes you need to get those files into docs/_layouts/whatever.html and docs/_includes/whatever.html (basically just drop the individual files down one level in the tree)
Example:
' Copy this:
docs/src/_includes/custom_thing.html
' To here:
docs/_includes/custom_thing.html

NB: Mailchimp sidebar
The mailchimp code has been moved to an extension file called docs/_includes/sidebar.html. Please put only the mailchimp stuff there.
2. Create docs/Gemfile
# frozen_string_literal: true

source "https://rubygems.org"
gem "jekyll-remote-theme"

3. Alter docs/.gitignore
Gemfile.lock

4. Alterdocs/_config.yml
Add lines:
exclude:
  - src
  - LICENSE
  - README.md
  - _config.yml
  - Gemfile*   # <----- just this at the end of the excludes

# NOTE: Add these 3 lines near the bottom of the file
# ----------------------------------------------------
remote_theme: Riverscapes/riverscapes-jekyll-theme
plugins:
  - jekyll-remote-theme

Remove Lines:
# Where our layouts and templates are stored
layouts_dir:  src/_layouts
includes_dir: src/_includes

5. Remove these files:

docs/assets/css/app.css
docs/assets/css/app.css.map

6. Remove these folders:

docs/assets/fonts/
docs/assets/js/
docs/src/
docs/assets/images/favicons/

7.


nbgv prepare-release "Uncommited changes in directory"

I am attempting to evaluate use of the Nerdbank.GitVersioning package and tools and I am getting a message "Uncommited changes in directory" when trying to run nbgv prepare-release.
git status shows no changes.

Hmmm... What version of nbgv are you using?

3.0.50; just installed it this morning using dotnet tool install -g nbgv

Strange. It works for me. But libgit2sharp (which we use underneath) did recently introduce a temporary file issue where it makes the working copy dirty. But IIRC I upgraded to apply their fix and I haven't seen the problem since.
I wonder if you could use a managed debugger to launch nbgv so you can break/pause the process when the exception is thrown, then run git status on the directory to see if anything is dirty.

I stepped through and it looks like something went wrong with line endings - there were two csproj files showing changes in repository.RetrieveStatus().Modified. I opened them, added a space, saved, removed the space and saved again and then committed. This did NOT create a commit, but now repository.RetrieveStatus().IsDirty is false and nbgv prepare-release is working.


code corrections for updated Rust toolchain

After updating Rust toolchain with "rustup update", building this repo results in compilation errors. This PR updates the source code to correct compilation.

This appears to be due to a breaking change in the rust typechecker rust-lang/rust#79904. I don't think we should take this change until we're set up to migrate our infrastructure to rust >= 1.49.
For now, you should downgrade to a version less than 1.49 (our infra uses 1.45.2) with e.g. rustup toolchain install 1.45.2 && rustup default 1.45.2).


feat: delete line vim command

I'm a little less sure this will work than https://github.com/lapce/lapce/pull/148.
Attempting to close https://github.com/lapce/lapce/issues/104

Not even close to working properly. Will have a bit more of a play at another time


Make a PR Template

Description
We need a PR template in github.
It should have:

closes # to encourage folks to add the issue
a checklist: Did you add specs? Is the build green? etc
Description header for the changes, and why we want them.

I've never made a PR template before and would like to figure out how to make one! 👀
Would like to pick this up if possible 🙌

Thanks! This is super helpful!


Fix documentation for  Data.Text.Read hexadecimal

Embarrassingly trivial PR here... docs were saying decimal digit instead of hexadecimal digit.

Thanks!


Remove anyhow dependency

The mainline requirement for anyhow isn’t necessary, but I’m going to leave it in for the build.rs script. This will be in the 1.0.1 release.


have re-enter jira source setting for epic key setup when creating new source on config-ui

Describe the bug
When creating new Jira Source, no epic key setting, that is ok

But when I hit Save Connection button, it brings me to listing page

So I have click on newly created source entry in order to complete the whole setup

Expected behavior
Can we stay on detail page instead of going back to listing page?

@klesh We need to save a connection first before allowing  Type mappings to be defined. The Type mappings are only available live from the API. If no valid connection exists, the type mapping selectors will have no data to drive them. I think what you are expecting is user is routed to The Connection Details/Edit Screen instead of the Manage Connections page.

@yumengwang03  you may want to take a look at this problem


Update web34ever.toml for testnet-15

Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

Merged PRs:
https://github.com/anoma/namada-testnets/pull/1271
https://github.com/anoma/namada-testnets/pull/695


pub.dev badges for Dart and Flutter with logo - take over badges.bar

I'm a longtime fan of this project and service.
I've been maintaining project and service called badges.bar. It includes the Dart icon on the left, and the pub points/popularity/likes, just like shields provides today. I'm planning to eventually sunset this service so I wonder if shields would be able to render a similar looking badge to what badges.bar currently does. And would like to take over the domain and redirect to it.

Thanks for reaching out and for your support of this project!
As you noted, we already provide badges with the same data points, so my understanding of your ask is that we change our existing badges to look more like badges.bar. In short, no I don't think that's something we'd be willing to do.
Our badges are based on our own specification and approach to defaults. As such any changes to our pub.dev badges would start diverging from the way we do things. We support the ability for our users to inject logos into our badges, but it's not something we do by default with these types of badges.
Accordingly, I really don't think it would make sense for us to take on any of the domain/networking aspects of badges.bar either. You could consider incorporating the redirect yourself (including with the ?logo=... param if you'd like to maintain those) but I don't think there's anything we can/will do from our end.
Will leave this open for a bit though to allow others from the ops and/or core team to weigh in

I think broadly we wouldn't be looking to take on the domain or service itself.
That said (and expanding on Caleb's comment), if you are looking to sunset badges.bar and direct users to use our dart/pub badges instead, my suggestion would be to replace it with some redirects. So you retain control of the badges.bar domain but replace what is running there now with a very lightweight service that implements the following logic:

if it gets a request to https://badges.bar/sentry/likes you issue a 301 redirect to https://img.shields.io/pub/likes/sentry?logo=dart
if it gets a request to https://badges.bar/sentry/popularity you issue a 301 redirect to https://img.shields.io/pub/popularity/sentry?logo=dart
if it gets a request to https://badges.bar/sentry/pub points you issue a 301 redirect to https://img.shields.io/pub/points/sentry?logo=dart

You can probably find hosting that allows you to operate that at low or zero cost and "set it and forget it". This would allow you to stop maintaing the service but preserve continuity of service for your existing users.
You could also use the existing customisations (such as they are) to make our badges look a bit more like yours if you wanted. For example

Instead of redirecting https://badges.bar/sentry/pub points to https://img.shields.io/pub/points/sentry?logo=dart you could redirect to https://img.shields.io/pub/points/sentry?logo=dart&label=pub points to retain the same label
Instead of using the SimpleIcons logo, you could redirect to a URL that uses a custom one. e.g: https://img.shields.io/pub/likes/sentry?logo=data:image/svg+xml;base64,PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHdpZHRoPSI2NCIgaGVpZ2h0PSI2NCI+PHBhdGggZD0iTTE3LjU4MiAxNy40MjRsLTQuMTM4LTQuMTQuMDE2IDI5LjkwMy4wNSAxLjM5NmMuMDIuNjYuMTQ1IDEuNC4zNDUgMi4xN2wzMi43NzUgMTEuNTYgOC4yLTMuNjMuMDA3LS4wMTJMMTcuNTggMTcuNDI0eiIgZmlsbD0iIzAwYzRiMyIvPjxwYXRoIGQ9Ik0xMy44NTYgNDYuNzUzaC4wMDNjLS4wMDMtLjAxMi0uMDA4LS4wMjYtLjAxNC0uMDQuMDA3LjAxNS4wMDcuMDMuMDEuMDR6bTQwLjk2NiA3LjkzbC04LjIgMy42My0zMi43Ny0xMS41NmMuNjI1IDIuNDA0IDIuMDEyIDUuMTA2IDMuNTAyIDYuNThsMTAuNjkgMTAuNjM3IDIzLjc4OC4wMyAyLjk4LTkuMzE3eiIgZmlsbD0iIzIyZDNjNSIvPjxnIGZpbGw9IiMwMDc1YzkiPjxwYXRoIGQ9Ik0xMy41NTYgMTMuMjg1TC44MTMgMzIuNTNjLTEuMDU4IDEuMTMtLjUzIDMuNDYyIDEuMTczIDUuMThsNy4zNTYgNy40MTYgNC42MjQgMS42M2MtLjItLjc2OC0uMzI2LTEuNS0uMzQ1LTIuMTdsLS4wNS0xLjM5Ni0uMDE1LTI5LjkwM3oiLz48cGF0aCBkPSJNNDYuOSAxMy42N2MtLjc3LS4xOTUtMS41MDgtLjMxOC0yLjE3My0uMzM3bC0xLjQ3OC0uMDU0LTI5LjgwNS4wMDcgNDEuMzkyIDQxLjM4NiAzLjYzNi04LjItMTEuNTctMzIuOHoiLz48L2c+PHBhdGggZD0iTTQ2Ljg2MiAxMy42NjNjLjAxMy4wMDUuMDI3LjAwOC4wMzguMDF2LS4wMDRjLS4wMTMtLjAwMy0uMDI1LS4wMDMtLjAzOC0uMDA4em02LjYxNSAzLjUxOGMtMS41MDUtMS41MTUtNC4xNy0yLjktNi41NzctMy41MDhMNTguNDcgNDYuNDdsLTMuNjQgOC4yIDguODgyLTIuODM4LjAyLTI0LjM1Mi0xMC4yNTYtMTAuM3oiIGZpbGw9IiMwMGE4ZTEiLz48cGF0aCBkPSJNNDUuMjc4IDkuMDc1bC03LjQtNy4zNkMzNi4xNTIuMDE4IDMzLjgyLS41MTIgMzIuNy41NDRsLTE5LjI0NCAxMi43NCAyOS44MDUtLjAwNyAxLjQ3OC4wNTRjLjY2NS4wMiAxLjQwNC4xNDIgMi4xNzMuMzM3TDQ1LjI4IDkuMDczem0tMzEuODMzIDQuMiIgZmlsbD0iIzAwYzRiMyIvPjwvc3ZnPg==
etc

Additionally, although we probably wouldn't directly take changes to the pub badges just to make them look more like the ones at badges.bar, if you have suggestions for improvements to the pub badges which meet our usual guidelines, feel free to contribute.

if it gets a request to badges.bar/sentry/likes you issue a 301 redirect to img.shields.io/pub/likes/sentry?logo=dart
if it gets a request to badges.bar/sentry/popularity you issue a 301 redirect to img.shields.io/pub/popularity/sentry?logo=dart
if it gets a request to badges.bar/sentry/pub%20points you issue a 301 redirect to img.shields.io/pub/points/sentry?logo=dart

These look good, and would serve as a replacement.
The main reason I'm reaching out is because I don't have the time to maintain the project and also because I don't want to continue to pay for the domain. If you folks don't want to take over the domain, I think I'll just set a date to sunset the service and suggest that folks adopt shields instead.
Thanks for your reply.

Any final solution?

You can use shields.io:
Example:

https://github.com/getsentry/sentry-dart/pull/889


feat(bin): added run time option to config file path and name

Hey, It would be really good to have an option for custom path to configuration
especially if you hate dozen of files in your project root folder (like me)
or have to run different configuration for different needs
Thanks in advance
Regards, Dima

@johnpapa please review

i like the idea. @cgmartin is this how you would like to do it? i prefer yargs myself.

This looks good to me. I like the fact that it matches names with the browsersync --config option.
Yargs is what I'm most comfortable with, but minimist is fine. There are several popular args modules out there that all score pretty high. What we're doing here is pretty simple/straightforward in terms of arg options. Easy to replace if need be.
@johnpapa good to merge? Thanks for the PR @valorkin !

i havent had a chance to test it ... have you Chris? If so, Im cool with it as long as it ups the version number by a minor.

Tested it, works fine. Merging and bumping minor version.

Since it changes the API (not breaking), Shouldn't this be a minor bump to 2.1.0 ? (not a patch)

i bumped it to 2.1.0 and published

yes, minor, I fubar. Thanks for fix

no worries. thanks for testing


VPN can't connect on Kubuntu 24.04

When I click connect a pop up only appears with a duration of 0 days as in the photo

the result of the command ./vpn.sh log
07/06/2024 16:22:05.231 INFO [Log log] [Messaging]      Server fingerprint: LONG BONE EAT WENT LAND BUSY HOP HE FREE LACY MILT DEAF
07/06/2024 16:22:05.231 INFO [Log log] [Messaging]      Automatic proxy replacement: true
07/06/2024 16:22:05.231 INFO [Log log] [Messaging] Sending INIT_DATA_EX message:
07/06/2024 16:22:05.231 INFO [Log log] [Messaging]      Allow only packets from sws: false
07/06/2024 16:22:05.239 INFO [Log log] [CShell] Initialized successfully
07/06/2024 16:22:05.557 INFO [CShellHTTPHandler proceedHandleRequest] Method name: get_is_connected
07/06/2024 16:22:05.561 INFO [CShellHTTPHandler proceedHandleRequest] Method name: get_finished
07/06/2024 16:22:45.273 INFO [Log log] [Messaging] Received DISCONNECTED message, Error 32: Cannot establish connection to SSL Network Extender gateway. Try to reconnect.
07/06/2024 16:22:45.274 INFO [CpComponent run] Received 'Disconnect' message from SNX:
ID: 32 MSG:Cannot establish connection to SSL Network Extender gateway. Try to reconnect.
07/06/2024 16:22:45.274 INFO [TunnelChecker stop] stop(true)
07/06/2024 16:22:45.274 WARNING [TunnelChecker disconnectTunnel] Can't disconnect tunnel, client director is not defined.
07/06/2024 16:22:45.274 WARNING [TunnelChecker stop] Can't stop disconnect checker, processed handle is not defined.
07/06/2024 16:22:45.612 INFO [CShellHTTPHandler proceedHandleRequest] Method name: Uninitialize
07/06/2024 16:22:45.612 INFO [TunnelChecker stop] stop(true)
07/06/2024 16:22:45.612 WARNING [TunnelChecker disconnectTunnel] Can't disconnect tunnel, client director is not defined.
07/06/2024 16:22:45.612 WARNING [TunnelChecker stop] Can't stop disconnect checker, processed handle is not defined.
07/06/2024 16:22:45.617 INFO [CShellHTTPHandler proceedHandleRequest] Method name: stop
07/06/2024 16:22:45.617 INFO [TunnelChecker stop] stop(true)
07/06/2024 16:22:45.617 WARNING [TunnelChecker disconnectTunnel] Can't disconnect tunnel, client director is not defined.
07/06/2024 16:22:45.617 WARNING [TunnelChecker stop] Can't stop disconnect checker, processed handle is not defined.

This issue is related to the Checkpoint server side. Please open a ticket with your local network services team and provide them with the error details.
This may be connected to the information found in Check Point Secure Knowledge https://support.checkpoint.com/results/sk/sk106339
I apologize for not being able to assist further.


[GN] Move dist logic into each module

BUG=https://crosswalk-project.org/jira/browse/XWALK-6369

@DonnaWuDongxia PTAL

Look Good To Me! Thanks!


Resize handles on all sides of the item

Allow resizing from extra handles within an item. Bottom-right-corner-only is not enough.

i need this ,, also

Me 2 :)

Me 2


feat(sdk): OpenMetrics support for System Metrics

Fixes WB-NNNN
Fixes #4903
Description
This PR implements support of OpenMetrics endpoint consumption for our System Metrics.
This should work in the general case, but is specifically targeted at the NVIDIA DCGM Exporter.

Notes:

For now we will only support the Counter and Gauge Prometheus metric types.
Added telemetry to track usage.
New setting for OpenMetrics endpoints to be monitored (Map string -> string -- tag -> uri)
The metrics are posted in the following format: system/openmetrics.<tag>.<metric name>.<label index> (to ease name-spacing on the FE). OM labels are hashed and associated with an index to adapt it to the way we store and display the system metrics. For example, the following OM data

# HELP DCGM_FI_DEV_MEM_COPY_UTIL Memory utilization (in %).
# TYPE DCGM_FI_DEV_MEM_COPY_UTIL gauge
DCGM_FI_DEV_MEM_COPY_UTIL{{gpu="0",UUID="GPU-c601d117-58ff-cd30-ae20-529ab192ba51",device="nvidia0",modelName="Tesla T4",Hostname="gke-gke-dcgm-default-pool-eb7746d2-6vkd",container="",namespace="",pod=""}} 10.4
DCGM_FI_DEV_MEM_COPY_UTIL{{gpu="1",UUID="GPU-a7c8aa83-d112-b585-8456-5fc2f3e6d18e",device="nvidia1",modelName="Tesla T4",Hostname="gke-gke-dcgm-default-pool-eb7746d2-6vkd",container="dcgm-loadtest",namespace="default",pod="dcgm-loadtest"}} 99.1

will be converted to {"system/openmetrics.DCGM.DCGM_FI_DEV_MEM_COPY_UTIL.0": 10.4, "system/openmetrics.DCGM.DCGM_FI_DEV_MEM_COPY_UTIL.1": 99.1}. We could store the label hash -> labels mapping (todo for a future pr.)

TODO: in a future PR, need to figure out how to do job (e.g. cluster job scheduled with SLURM or something) name-spacing

Testing

[x] Unit test
[x] Fake flask-based local prometheus endpoint
[x] Multi-node multi-GPU GKE cluster with DCGM/DCGM Exporter deployed there

Checklist

[x] Include reference to internal ticket "Fixes WB-NNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[x] Ensure PR title compliance with the conventional commits standards

@raubitsj I hardened the network paths, as requested, and made this feature an opt-in one.


does not work with react-native-web 0.19

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
moti does not work with react-native-web 0.19.x, it yields TypeError: Cannot convert undefined or null to object
upstream issue: https://github.com/software-mansion/react-native-reanimated/issues/4335
there is a closed issue, but since it is closed, i think its better to reopen a new one (https://github.com/nandorojo/moti/issues/103)
i can confirm it works properly with react-native-web 0.18.0.
Expected Behavior
should work with the latest react-native-web
Steps To Reproduce
No response
Versions
- Moti:0.24.2
- Reanimated: 3.10
- React Native: 0.71.6

Screenshots
No response
Reproduction
todo

looks like this fixes it https://github.com/software-mansion/react-native-reanimated/pull/4352/files

looks like this fixes it https://github.com/software-mansion/react-native-reanimated/pull/4352/files

could not test it though, i currently downgraded react-native-web

You could patch reanimated if you want, but yeah that also works for now

I think reanimated 3.3 fixes this.


Revision history

re: "Preface" and "Appendix G Revision History"
This concept is obsolete with the proposed workflow. (And the description in the preface is incompatible with the removal of highlighting.)
The wording in the preface needs updating but the appendix should probably stay.

...

Moved to https://github.com/cf-metadata/convert/issues/1


Add native Romanian (RO) translation

@capital-technologies Thanks so much for taking the time to do this 🙏
Looks good to me but will wait for @sindresorhus or @kevva to give it the once over before merging as I haven't had much to do with localisation.

@sindresorhus , @kevva , @lukechilds any updates on this?

@capital-technologies yeah, we use a provider called CrowdIn to manage translations. It currently only supports one-way sync (CrowndIn => GitHub) not two-way support (GitHub => CrowdIn) like in this PR.
@sindresorhus thinks he can resync these changes over to CrowdIn but if we just merge this right now we're not quite sure what will happen and it might break CrowdIn. We don't want to risk that right before the Beta launch so we'll release without it but will look into this after.
We'll keep you posted.

Ok, the sync trick worked.

@capital-technologies There's a couple of remaining string to translate. Would you be able to do them too? https://crowdin.com/translate/hyperdex/34/en-ro (Use the CrowdIn UI, not GitHub)

@sindresorhus I filled the missing gaps, everything's ok now.

Perfect. Thank you :)


Train on custom dataset

Dear author,
I am trying to train your model on my custom dataset i am using following command
!python src/train.py with deformable tracking   resume=r50_deformable_detr-checkpoint.pth  output_dir=models/custom_dataset_train_deformable  mot_path=data/custom_dataset train_split=train val_split=val epochs=20
i get following error message
frame_id = self.coco.imgs[idx]['frame_id']
KeyError: 'frame_id'

I think you should convert your own dataset to COCO-style first, so you can get the key of 'frame_id'

Hi @atousaz,
Check your dataset. It might have the key 'image_id', instead of 'frame_id'. Some tools can do that. Such as CocoAnnotation.

There is a difference between image_id and frame_id. Please check the generate_coco_from_mot.py to understand each field. The latter does not exist in the original COCO annotations and is related to the position of the image in as a frame in a video sequence.


Add blog embed to homepage

This is a bit broken on mobile, but I think the way to fix this would be in the minetest/blog repo

@Calinou  please may you take a look at this?


Add support for Elasticache Redis

Add terraform to enable support of Elasticache Redis.
If this is done after memcached, the code should be reused. If it's done before it should take into account that we also want to support memcached.

Related to #168

Related to #168

After investigation, I prefer use this, which supports both Memcache and Redis.

The aws_elasticache_cluster resource will make a cluster for memcached but it only supports a single instance group for redis. For Redis you should use a replication group. I would suggest using the cloudposse modules instead of just the provider.

Decision:
combine both as a single module with selection of redis or memcache:

https://registry.terraform.io/modules/cloudposse/elasticache-memcached/aws/latest
https://registry.terraform.io/modules/cloudposse/elasticache-redis/aws/latest

Support both Redis and Memcached, ready for go backend to use.
PRs:

Module: https://github.com/commitdev/terraform-aws-zero/pull/32
Infrastructure: https://github.com/commitdev/zero-aws-eks-stack/pull/177
Backend(go): https://github.com/commitdev/zero-deployable-backend/pull/55


add important option for use with tools like cleanslate

I thought that this could potentially be useful for other people, we are using this with widgets that are loaded on other peoples site and are using https://github.com/premasagar/cleanslate to ensure our styles aren't overwritten.

Interesting, give me a ping when this is ready for me too look at :)

@wnr I've made the necessary changes for the object strategy as well as scroll and added some tests, we're actually using this in production already so I think it's good to go.

Thank you @sanpoChew. Indeed, the CI failing seems to be something else. I'll look into it. Actually, when I think about it, would it make sense to make everything always important? I can't come up with a case where one would like these properties to get overridden by another style rule. So I'm thinking about removing the option, and always enable this feature. What do you think?

@wnr makes sense to me, can't think of a good reason why not 👍


SafeSettings doesn't allow rename of the repo reverts it back

I Understand this is a safety net links back to https://github.com/github/safe-settings/issues/556


Signs

Implementing signs

Thanks a lot for the PR! Great work!
I'll merge this into a dedicated branch as discussed on Discord to make sure the last small things are fixed up. It'll be merged before the next release!


[elasticsearch] show 8.5.0's changelog until they fix it

use 8.5.0's link until they publish 8.5.1's changelog

ref : #1206


Check efficacy of a posteriori training

We have to confirm that the a posteriori training is effectively going in the right direction. The loss does not decrease enough to confirm this and it is very unstable suggesting some problems.
A basic test could be

[ ] confirm that a posteriori training converges to something better than no closure

then later we have to confirm that

[ ] given a specific CNN architecture and data structure, CoupleNODE produces results comparable to NeuralClosure

[ ] implement a callback function that plots some 'validation loss' measured on a constant dataset (not used for training)


Deploying Grakn to mainline repos

Goal
Grakn Core users on Linux distributions should be able to install Grakn with their package manager, i.e.: apt-get install grakn-core-server (Deb-based distros) yum install grakn-core-server (RPM-based distros)
Approaches
There are two approaches: self-hosting and submitting a project to mainline repos.
Self-hosting
Advantages

Full control over the repo, which means:
— there's nobody to reject our packages
— we push updates on our schedule
Unified deployment workflow with test repos ⇒ less code to write ⇒ fewer bugs

Disadvantages

User has to add our repo (one extra step)
We have to maintain our self-hosted repo

Submitting to mainline repo
Advantages

User doesn't have to add extra repos

Disadvantages

Communities might reject our packages due to a variety of reasons (i.e. DEBIAN Reject-FAQ) — and we'll have to work with them one-by-one
I'm not sure how deployment would work, whereas in self-hosted variant, it's a simple curl command

For now, we seem to have settled with using Sonatype Nexus as our repo (deployed at https://repo.grakn.ai) for both Debian [Ubuntu] and Fedora [CentOS] deployment. If need ever arises, let's created another issue.


Update Developer Workflow

This PR makes a few changes to make it easier to develop the project cleanly. They are listed below:

Updates requirements to only include the bare minimum to run. Moves things that are not strictly required into dev dependencies.
(This means deployments no longer have to install using the dev flag.) (dotenv is not disabled on deployments.)
Adds and enforces eslint on a project-wide level for ts and tsx files. Adds a workflow to enforce linting on the repository.
Fixes pre-existing linting issues.
Adds github hooks to test and lint code before committing to make it easier for developers.
Updates notion docs with new steps to setup, test, and lint your project.

Affected notion docs:

Installation Frontend

Thanks!

Thanks!


Dataverse Connector: Error in Bound Action "AddMemberList" for Table "Marketing List"

Describe the Bug with repro steps
When following the steps below, get an error that the parameter is not valid. However, when manually manipulating the JSON to rename the parameter "EntityId" instead of "body/EntityId" the action is successful. This needs to be updated in the connector action so that the call can work correctly.
Steps

Create a Logic App
Add an action for Dataverse > Bound Action.
Table = Marketing List, Action = AddMemberToList
Row Id = Marketing List Id
body/EntityId = Record to Add

Error
"Error identified in Payload provided by the user for Entity :'', For more information on this error please follow this help link https://go.microsoft.com/fwlink/?linkid=2195293  ---->  InnerException : Microsoft.OData.ODataException: The parameter 'body/EntityId' in the request payload is not a valid parameter for the operation 'AddMemberList'.\r\n   at Microsoft.OData.ODataParameterReaderCore.GetParameterTypeReference(String parameterName)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightParameterDeserializer.<>c__DisplayClass3_0.b__0(PropertyParsingResult propertyParsingResult, String parameterName)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightDeserializer.ProcessProperty(PropertyAndAnnotationCollector propertyAndAnnotationCollector, Func2 readPropertyAnnotationValue, Action2 handleProperty)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightParameterDeserializer.ReadNextParameter(PropertyAndAnnotationCollector propertyAndAnnotationCollector)\r\n   at Microsoft.OData.ODataParameterReaderCore.ReadImplementation()\r\n   at Microsoft.OData.ODataParameterReaderCore.InterceptException[T](Func`1 action)\r\n   at Microsoft.Crm.Extensibility.ODataV4.CrmODataActionPayloadDeserializer.Read(ODataMessageReader messageReader, Type type, ODataDeserializerContext readContext)\r\n   at System.Web.OData.Formatter.ODataMediaTypeFormatter.ReadFromStream(Type type, Stream readStream, HttpContent content, IFormatterLogger formatterLogger)."
What type of Logic App Is this happening in?
Standard (Portal)
Are you using Preview Designer or GA Designer
Preview
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Edge
Additional context
No response

I’m able to save the workflow just fine and it fails on the run. It runs fine when I manually update the parameter to EntityId instead of body/EntityId.  Sent from my iPhoneOn May 23, 2023, at 7:55 PM, preetriti1 @.***> wrote:
@berellax - It is not clear on what is not working here, Are you able to save the workflow with your changes and workflow run fails?
Or the workflow save itself is failing?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@berellax - Can you please share network logs in your scenario so we know from where body/EntityId if coming from? We will need that info to see why those parameters were returned from connector.
You can share the logs to me on my email<EMAIL_ADDRESS>


fun e

What type of pull request is this?

[ ] Non-breaking code change
[ ] Breaking code change
[ ] Documentation change/addition
[ ] Tests change

Description

Changes

Checklist

[ ] I've formatted my code with Black
[ ] I've ensured my code works on Python 3.10.x
[ ] I've tested my code

/ok-to-test

/ok-to-test


composer lock

please add a composer lock file to deploy it on heroku

Okay thanks!

welcome buddy ! waiting for composer lock so that i can use this repo on heroku. Without composer lock i am unable to deploy it on heroku showing error.

@nk932714
Sorry I've forgotten it
Will do it this evening

@nk932714
Done :)

thanks a lot buddy :)


[Feature] Support ReduceLrUpdaterHook

We still need to assert the lr get from lr_updater.get_lr(fake_runner, 1) is expected. They are missing currently.


collectstatic

Hello,
Could you tell me why I get such an error while I run:
python manage.py collectstatic
Traceback (most recent call last):
File "manage.py", line 21, in
main()
File "manage.py", line 17, in main
execute_from_command_line(sys.argv)
File "/home/myproject/env/lib/python3.8/site-packages/django/core/management/init.py", line 401, in execute_from_command_line
utility.execute()
File "/home/myproject/env/lib/python3.8/site-packages/django/core/management/init.py", line 395, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/home/myproject/env/lib/python3.8/site-packages/django/core/management/base.py", line 328, in run_from_argv
self.execute(*args, **cmd_options)
File "/home/myproject/env/lib/python3.8/site-packages/django/core/management/base.py", line 369, in execute
output = self.handle(*args, **options)
File "/home/myproject/env/lib/python3.8/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py", line 187, in handle
collected = self.collect()
File "/home/myproject/env/lib/python3.8/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py", line 113, in collect
handler(path, prefixed_path, storage)
File "/home/myproject/env/lib/python3.8/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py", line 348, in copy_file
self.storage.save(prefixed_path, source_file)
File "/home/myproject/env/lib/python3.8/site-packages/django/core/files/storage.py", line 52, in save
return self._save(name, content)
File "/home/myproject/env/lib/python3.8/site-packages/django/core/files/storage.py", line 248, in _save
os.makedirs(directory, exist_ok=True)
File "/usr/local/lib/python3.8/os.py", line 213, in makedirs
makedirs(head, exist_ok=exist_ok)
File "/usr/local/lib/python3.8/os.py", line 223, in makedirs
mkdir(name, mode)
PermissionError: [Errno 13] Permission denied: '/home/myproject/static/site'
I have root permissions on the volumes

I have resolved by running root

Before creating the images create the static and media folders that are missing in this repo:
mkdir -p src/myproject/media
mkdir -p src/myproject/static

Another option is to ssh as root and change the permissions:
docker exec -u 0 -it django_docker_gunicorn_1 bash
chown -R myproject static
chown -R myproject media

From now on you can continue using just myproject user privileges


Indexing with alignment and broadcasting

I think we can bring all of NumPy's advanced indexing to xarray in a very consistent way, with only very minor breaks in backwards compatibility.
For boolean indexing:

da[key] where key is a boolean labelled array (with any number of dimensions) is made equivalent to da.where(key.reindex_like(ds), drop=True). This matches the existing behavior if key is a 1D boolean array. For multi-dimensional arrays, even though the result is now multi-dimensional, this coupled with automatic skipping of NaNs means that da[key].mean() gives the same result as in NumPy.
da[key] = value where key is a boolean labelled array can be made equivalent to da = da.where(*align(key.reindex_like(da), value.reindex_like(da))) (that is, the three argument form of where).
da[key_0, ..., key_n] where all of key_i are boolean arrays gets handled in the usual way. It is an IndexingError to supply multiple labelled keys if any of them are not already aligned with as the corresponding index coordinates (and share the same dimension name). If they want alignment, we suggest users simply write da[key_0 & ... & key_n].

For vectorized indexing (by integer or index value):

da[key_0, ..., key_n] where all of key_i are integer labelled arrays with any number of dimensions gets handled like NumPy, except instead of broadcasting numpy-style we do broadcasting xarray-style:

If any of key_i are unlabelled, 1D arrays (e.g., numpy arrays), we convert them into an xarray.Variable along the respective dimension. 0D arrays remain scalars. This ensures that the result of broadcasting them (in the next step) will be consistent with our current "outer indexing" behavior. Unlabelled higher dimensional arrays triggers an IndexingError.
We ensure all keys have the same dimensions/coordinates by mapping it to da[*broadcast(key_0, ..., key_n)] (note that broadcast now includes automatic alignment).
The result's dimensions and coordinates are copied from the broadcast keys.
The result's values are taken by mapping each set of integer locations specified by the broadcast version of key_i to the integer position on the corresponding ith axis on da.

Labeled indexing like ds.loc[key_0, ...., key_n] works exactly as above, except instead of doing integer lookup, we lookup label values in the corresponding index instead.
Indexing with .isel and .sel/.reindex works like the two previous cases, except we lookup axes by dimension name instead of axis position.
I haven't fully thought through the implications for assignment (da[key] = value or da.loc[key] = value), but I think it works in a straightforwardly similar fashion.

All of these methods should also work for indexing on Dataset by looping over Dataset variables in the usual way.
This framework neatly subsumes most of the major limitations with xarray's existing indexing:

Boolean indexing on multi-dimensional arrays works in an intuitive way, for both selection and assignment.
No more need for specialized methods (sel_points/isel_points) for pointwise indexing. If you want to select along the diagonal of an array, you simply need to supply indexers that use a new dimension. Instead of arr.sel_points(lat=stations.lat, lon=stations.lon, dim='station'), you would simply write arr.sel(lat=stations.lat, lon=stations.lon) -- the station dimension is taken automatically from the indexer.
Other use cases for NumPy's advanced indexing that currently are impossible in xarray also automatically work. For example, nearest neighbor interpolation to a completely different grid is now as simple as ds.reindex(lon=grid.lon, lat=grid.lat, method='nearest', tolerance=0.5) or ds.reindex_like(grid, method='nearest', tolerance=0.5).

Questions to consider:

How does this interact with @benbovy's enhancements for MultiIndex indexing? (#802 and #947)
How do we handle mixed slice and array indexing? In NumPy, this is a major source of confusion, because slicing is done before broadcasting and the order of slices in the result is handled separately from broadcast indices. I think we may be able to resolve this by mapping slices in this case to 1D arrays along their respective axes, and using our normal broadcasting rules.
Should we deprecate non-boolean indexing with [] and .loc[] and non-labelled arrays when some but not all dimensions are provided? Instead, we would require explicitly indexing like [key, ...] (yes, writing ...), which indicates "all trailing axes" like NumPy. This behavior has been suggested for new indexers in NumPy because it precludes a class of bugs where the array has an unexpected number of dimensions. On the other hand, it's not so necessary for us when we have explicit indexing by dimension name with .sel.

xref these comments from @MaximilianR and myself
Note: I would certainly welcome help making this happen from a contributor other than myself, though you should probably wait until I finish #964, first, which lays important groundwork.

What is the current status of this proposed major changes?
Is there any starting basis for them?
I have recently some free time.

I haven't had the time to start working on this -- help would be very gratefully appreciated!
In rough order, I would suggest:

Write a design doc for any new API. We basically have this here, but there may be a few aspects that still need to be thought through, so I would your critical review.
Write the test suite for the new functionality. We could even submit this, with the tests marked as xfail.
Implement the core functionality and get the new tests to pass. This is the fun part :).
Get the existing test suite to pass. This will likely be the most painful part, because there will be a number of completely unrelated test failures. I can definitely help when we get to this stage.

@shoyer Thanks for the suggestions.
I don't think I fully understand how boolean indexing works.
Could you show me some example use-cases?
I started from the vectorizing indexing by updating Variable.__getitem__ method,
in my folked repo (test).

@fujiisoup My first two bullets for boolean indexing are actually new functionality, so we wouldn't need that for a first pass here. It actually would probably be better to save it for a second PR.
My third bullet on boolean indexing is basically just saying that da[key_1, ..., key_n] and ds.sel(x=key) should be handled in a consistent way with the new indexing behavior when given a boolean array. I don't think this will require any special adjustments -- it should fall out pretty immediately once you get the rest working.

I opened a pull request with your branch in #1473 so I can comment on your changes. I hope that's OK!

Vectorized indexing was implemented by https://github.com/pydata/xarray/pull/1473
I've opened a new issue #1887 for multi-dimensional boolean indexing.


Could not find gem 'itamae-plugin-resource-iptables (>= 0) ruby'

Could you push this gem into rubygems.org?

Actually I'm not using this gem now!
If you're using this gem, I'll transfer this repository to you and feel free to push it to rubygems.org .
Are you interested in it?

Thank you for your kindness!
Yes, I'm interested in that.
I'd like to use this cool gem!

I just requested repository transfer to you.

Thank you so much! :heart:

Closed via https://rubygems.org/gems/itamae-plugin-resource-iptables


Making DocumentSnapshot.data() nullable

This PR makes DocumentSnapshot.data() nullable and introduces a non-nullable QueryDocumentSnapshot type.

Comments addressed. I left the Firestore.xcodeproj/project.pbxproj in this PR as this change shows up in all my diffs.
Is this an update we need to make? Otherwise, I will remove it.

Comments LGTM, please squash and merge.


Dailymotion support

Any chance you could look at adding dailymotion support?

A pull request would be great!

Added #10


docs: update vite.mdx to fix npm command reference

Fix npm command reference.
Checklist

[x] Tests updated (or not required).
[x] Documentation updated (or not required).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks!


Remove Python 3.8 workarounds

Support for Python 3.8 ends after Amaranth 0.5 (https://github.com/amaranth-lang/amaranth/issues/1504).
Workarounds for enum errors can be removed:
https://github.com/amaranth-lang/amaranth-soc/blob/9d6bd2c54b4ca28ea4c96cae38edb4c9c3bfdf51/amaranth_soc/csr/bus.py#L56-L62

Closed by https://github.com/amaranth-lang/amaranth-soc/pull/102.


Turtle pretty-print: Blank node in object position of a single Statement

Where possible when pretty-printing Turtle, if a blank node is used as the object of a single Statement, and the subject of one or more other statements, the subject-use Statements should be nested under the single object Statement, to remove the use of a labelled blank node and replace it with an anonymous blank node notation.

This is a follow-up to the initial pretty-printing implementation added in issue #207


explain the goals of Jakarta Data and motivate/justify/explain the selected design

Currently, section 1.1 of the specification explains its goals as follows:

Jakarta Data works in a tight integration between Java and a persistence layer, where it has the following specification goals:

Be a persistence agnostic API. Therefore, through abstractions, it will connect different types of databases and storage sources.
Be a pluggable and extensible API. Even when the API won’t support a particular behavior of a storage engine, it might provide an extensible API to make it possible.

This is completely inadequate:

it does not mention what problem Jakarta Data trying to solve, nor motivate me to care,
it provides me no yardstick to decide whether Jakarta Data even fulfills its own goals,
it doesn't explain what is meant by "agnostic" here (from the rest of the spec, it's clear that "agnostic" certainly doesn't mean "portable"), and
it fails to consider other options already available in the Jakarta platform or the Jakarta ecosystem: why entity-specific repositories? Why not an EntityManager-style API? Why not Active Record like, e.g. Panache? Why not a more free-form solution like in Hibernate 6 where the user gets to decide the organization of code?

Later, in the preamble for chapter 2 the notion of a "repository" is introduced and sort-of defined, with some handwaving that completely fails to capture the essential differences between an entity-specific repository, Active Record, or an EntityManager. Let's go through this line-by-line.

This pattern focuses on the closest proximity of entities and hides where the data comes from.

I have no idea what this sentence means. What on earth is the "closest proximity of entities"? The data comes from the business. Is it the goal of Jakarta Data repositories to "hide" the business?

The Repository pattern is a well-documented way of working with a data source.

It's not good enough that it's documented somewhere else. Jakarta Data is, in its current form, a specification for repositories, and so it needs to document what a repository is internally, not by reference to a book that I have no interest in reading and would have to pay for.
Next we get a quote from Martin Fowler:

A repository performs the tasks of an intermediary between the domain model layers and data mapping, acting in a similar way to a set of domain objects in memory. Client objects declaratively build queries and send them to the repositories for answers. Conceptually, a repository encapsulates a set of objects stored in the database and operations that can be performed on them, providing a way that is closer to the persistence layer. Repositories also support the purpose of separating, clearly and in one direction, the dependency between the work domain and the data allocation or mapping.

Unfortunately, this quote is pretty useless to us:

Does a DAO-style repository really "act in a similar way to a set of domain objects in memory"? Hard to say, since I'm not even sure what that means, if it means anything at all.
The remaining part of the quote sounds like it's describing an EntityManager.

The next line:

It also becomes very famous in Domain-Driven Design: Tackling Complexity in the Heart of Software by Eric Evans.

Seems to be no more than a handwave to authority, and doesn't belong in the spec. [Yes, OK, somebody wrote a book once, awesome, I'm happy for them.]
The first two pages of the specification should be rewritten.

Hi @otaviojava, so, what you've written here does read a lot better than before.
I'm still not satisfied, but it will do for now. We can come back later and iterate it once the rest of the specification is much better-defined.
P.S. I really wish you would remove references to Domain Driven Design. There's simply nothing about Jakarta Data which is specific to DDD, and it's just not relevant to the specification, in my opinion.

@gavinking
Thank you for the guidance.
I will reopen to remove the DDD reference, then.

Thank you! 🙏


DaprInvocationHttpClient should correctly populate headers in InvokeMethodResponse

Expected Behavior
DaprInvocationHttpClient should correctly populate headers in InvokeMethodResponse
Actual Behavior
DaprInvocationHttpClient does not correctly populate headers in InvokeMethodResponse
Steps to Reproduce the Problem
Environment:
Self-Hosted
Mac OS 12.3.1
CLI version: 1.7.0
Runtime version: 1.7.0
dapr python sdk: 1.6
Set up a dapr sidecar and HTTP service with a valid endpoint.
Then use the python api to invoke. Headers will not be populated.
with DaprClient() as d:
     print(d.invoke_method('service',method','').headers)

I suspect the issue is in this code

@ChrisFajardo-TRI I just released Python SDK 1.6.1 which cherry-picked the fix into the existing 1.6 release.
Please note:
The headers field is of type MetadataDict which really is a Dict[List[Union[str, bytes]].
What this means is that when you access a header like Content-Type you will find that the value is a list. The first value will be the value you are looking for.
So...
res = DaprResponse()
res.headers['Content-Type'][0]  # could be 'application/json' etc

Thanks @berndverst
It works!


Grid-like pattern

Hi Shaoyan, really appreciate your work here. When trying out the model, all of my predicted images exhibit a grid-like pattern matching the inference sliding window. I'm curious if you also saw this during your own training.

I've tested using 20 training images for 500 epochs, as well as 144 training images for 180 epochs, and the pattern appeared for both.

Here are three options for improving the grids:

In the Jupyter notebook, under the "Build the testing function" section, modify the code as follows:
from monai.inferers import SlidingWindowInferer
img_num = 12
overlap = 0.5
inferer = SlidingWindowInferer(img_size, img_num, overlap=overlap, mode='constant')

Change the last line to:
inferer = SlidingWindowInferer(img_size, img_num, overlap=0.75, mode='gaussian')

The network might not be fully optimized. Try training for more epochs. In my experience, these patterns diminish significantly with longer training.

The grids appear only in the x-y plane, not in the z-plane, for the MRI-CT task. You can increase the patch size in the x and y dimensions and reduce it in the z dimension.

Thanks, really appreciate the help. I am curious about the reasoning behind the patch-based approach, does using smaller patches/windows provide benefits besides faster training?

Actually patch-based approach is not desiable, it make your performance worse, slower training and generation. The best way is to fit the whole volume (e.g., 256x256x128) for training.
However, our GPU does not have enough memory to accpet the whole volume, so patch-based approach is necessary to let our GPU can accept the data (64x64x4) then start to train. Depends on your GPU, make the patches as large as possible while maintain some reasonable batch size. For example, if your GPU is very large to accpet 16x128x128x4 volume, you can consider to increase the patch size to 256x256x4 but the batch can reduce to 4. Still good. Then you can start to train.

Hi, Christopher
May I ask what your dataset is? Is it a public dataset? I couldn't get good results with my dataset, and I am wondering about your results on your data.

I am using the SynthRAD2023 dataset, my most recent results are not as good as those reported in Shaoyan's paper, but they are reasonable (~82 MAE).

One more thing, don't forget to average five generation samples for each patient, which should be explained in my paper. So, for each patient, you should generate five samples then average them as the final result. They can achieve better quantative result but less realistic visual appearence.

Hi Shaoyan, thanks for the advice. My training loss seems to have flattened after many epochs. I am wondering how your learning rate and weight decay values were chosen, were they selected empirically or did you test different values using a validation set?

I just try different values to make the quantative result to be the best for my tasks. I think you will have different optimal learning rate for your tasks. But to be honest, the improvement should be tiny, at least should not have statistical difference even you use not optimal learning rate.


Move from_unixtime, now, current_date, current_time functions to datafusion-functions

Which issue does this PR close?

Closes #9466
Rationale for this change

Functions are migrated to the new crate and use the new patterns as described in https://github.com/apache/arrow-datafusion/issues/9286 & https://github.com/apache/arrow-datafusion/pull/9216
What changes are included in this PR?
Code, tests moved to new location, references updated.
Are these changes tested?
Yes
Are there any user-facing changes?
No

🚀  - thank you @Omega359


Add test to verify short PHP tags

Let's add a test to verify that all the PHP files (except the ones under /concrete/vendor) don't have PHP short open tags (<?).

Nice idea. These shorts tags were a real nightmare when I installed one of the latest versions of Concrete5 with PHP7 where it is disabled by default. Of course I didn't know what was preventing Apache from parsing php files...

lol this is certainly a nice way for us to get our test count up ;)


Unable to retrieve secrets from azure keyvault

In Azure VM , when executing our python application exe , which uses
ClientSecretCredential(tenant_id: str, client_id: str, client_secret: str, kwargs: Any) function to Authenticate and retrieve secret's value from Azure Key Vault and the VM  doesn't have Client Certificate and Azure CLI.
We are getting the following error where credentials are not successfully authenticated and we get following response:

The below one is the code snippet we have used :

For more context: a similar application, written in C#, is able to successfully fetch a token in the same environment. For the Python application, the secret request successfully reaches Key Vault and we do get a response (the 401 challenge shown in the log screenshot). However, token requests are hanging and timing out repeatedly until there's eventually a connection error from a lack of a response. Since it's specifically the token request that's failing, this seems like an azure-identity-specific configuration issue.

To separate the question, could you try code:
cred = ClientSecretCredential('tenantid', 'clientid', 'client_pass')

token = cred.get_token("https://vault.azure.net/.default")

print(token)

And see if identity works?

Confirmed offline that this environment isn't using any proxy configuration. We unfortunately can't access logs for the C# application that successfully makes token requests.

Did they have a chance to try the code above? @mccoyp

@xiangyan99 not yet, but I bumped the suggestion in our chat. Some other folks who are digging into the networking think that the issue could be a firewall -- which would make sense based on what we're seeing, but would also raise the question of how the C# application doesn't have this issue

@mccoyp  when we tried with DefaultAzureCredential() function we were able to successfully authenticate and get the AKV secret value successfully

@Praveen288g Thanks for the update.
To make sure I understand correctly, you mean it failed when you used ClientSecretCredential but succeeded when you use DefaultAzureCredential?
This makes me think maybe other types of credentials worked? e.g. MSI?

Yep you are correct @xiangyan99

@xiangyan99 yep , i need confirmation whether AzureDefaultCredential() is best method to use in SFTP server/Azure VM , want to make sure whether it could cause any problem in future

It should not cause problems if you use AzureDefaultCredential() in a production environment (means it is stable and will not change).
If you want to have a more deterministic behavior, you can use a specific type of credential rather than AzureDefaultCredential which is a chain of credentials.


Fix bug where reminder date can be set to a date that has already passed

The current implementation of Reminder allows a date to be set in the past, which should not be the case. Let's fix the bug to close AY2122S2-CS2103T-T17-3#141

Codecov Report

Merging #185 (a99156a) into master (55bdfbf) will decrease coverage by 0.05%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master     #185      +/-   ##
============================================
- Coverage     79.77%   79.71%   -0.06%
  Complexity      748      748
============================================
  Files           111      111
  Lines          2210     2214       +4
  Branches        234      235       +1
============================================
+ Hits           1763     1765       +2
- Misses          379      380       +1
- Partials         68       69       +1

Impacted Files
Coverage Δ

...in/java/seedu/address/model/reminder/Reminder.java
87.50% <ø> (ø)

...ava/seedu/address/storage/JsonAdaptedReminder.java
61.53% <ø> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
85.89% <50.00%> (-2.11%)
:arrow_down:

...in/java/seedu/address/model/date/ReminderDate.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Introduce VariantTag and remove VariantUnwrap

Closes #1927 and closes #1928. Instead of doing PRs where I would remove VariantUnwrap and then introduce VariantTag in the same places again, I just renamed VariantUnwrap into VariantTag, fixed the type signature and the rewriting rule. It touches a number of files, but most of the changes are mechanical. I had to give up on rewriting tags with a special sort TAG, as we now have to returns tags as strings.

[x] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[x] Entries added to ./unreleased/ for any new functionality

Codecov Report

Merging #1936 (6797f51) into unstable (a021c65) will decrease coverage by 0.00%.
The diff coverage is 94.44%.

:exclamation: Current head 6797f51 differs from pull request most recent head 2432381. Consider uploading reports for the commit 2432381 to get more accurate results

@@             Coverage Diff              @@
##           unstable    #1936      +/-   ##
============================================
- Coverage     77.08%   77.07%   -0.01%
============================================
  Files           415      415
  Lines         12677    12668       -9
  Branches        603      569      -34
============================================
- Hits           9772     9764       -8
+ Misses         2905     2904       -1

Impacted Files
Coverage Δ

.../forsyte/apalache/tla/bmcmt/SymbStateDecoder.scala
78.07% <ø> (-1.25%)
:arrow_down:

...yte/apalache/tla/bmcmt/SymbStateRewriterImpl.scala
93.02% <ø> (ø)

...ache/tla/bmcmt/rules/aux/DefaultValueFactory.scala
38.77% <0.00%> (ø)

...syte/apalache/tla/bmcmt/rules/VariantOpsRule.scala
94.11% <100.00%> (+2.22%)
:arrow_up:

...ache/tla/bmcmt/rules/aux/RecordAndVariantOps.scala
91.20% <100.00%> (-0.10%)
:arrow_down:

.../apalache/tla/bmcmt/rules/aux/ValueGenerator.scala
98.38% <100.00%> (ø)

.../at/forsyte/apalache/tla/imp/StandardLibrary.scala
100.00% <100.00%> (ø)

...forsyte/apalache/tla/typecheck/etc/ToEtcExpr.scala
94.95% <100.00%> (+0.20%)
:arrow_up:

...in/scala/at/forsyte/apalache/tla/lir/Builder.scala
99.12% <100.00%> (ø)

...at/forsyte/apalache/tla/lir/oper/VariantOper.scala
93.75% <100.00%> (ø)

... and 4 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 a021c65...2432381. Read the comment docs.

I knew you would say that. However, the PR would blow up :)

I knew you would say that. However, the PR would blow up :)

How? You add 64 LOC, so there's at most 64 calls to the builder (and practically probably < 10)


--nodeAuditSkipDevDependencies causes "Could not perform Node Audit analysis. Invalid payload submitted to Node Audit API"

Describe the bug
Using --nodeAuditSkipDevDependencies on package-lock.json causes "Could not perform Node Audit analysis. Invalid payload submitted to Node Audit API"
Version of dependency-check used
dependency-check 8.0.1
Log file
logfile
To Reproduce
Steps to reproduce the behavior:
1.
dependency-check.sh -s package.json -s package-lock.json --nodeAuditSkipDevDependencies
Expected behavior
--nodeAuditSkipDevDependencies should exclude dev dependencies from audit instead of producing an error
Additional context
dependency-check works as expected when --nodeAuditSkipDevDependencies is removed

You have packages that are both in devDependences and dependencies. Determine the best spot for those package(s) and try the scan again.
This is because of how ODC determines if the package is a dev dependency or not when building the node audit payload. If a package is in both, then the package-lock.json will set "dev": true for that dependency. ODC then will add the package to the "requires" part of the audit payload but not include it in "dependencies" section if skipDevDependencies (--nodeAuditSkipDevDependencies) is set to true.
I guess if it's possible, ODC should work, but whether this is a ODC bug or not, this will have to be the workaround for now. I know npm used to warn you if you had a package in both, so maybe it's acceptable now.

For anyone that is going to fix this or for me later:
package.json:
{
  "name": "skipDevDependencies-test",
  "devDependencies": {
    "dotenv": "10.0.0",
  },
  "dependencies": {
    "dotenv": "10.0.0",
    "shell-quote": "1.7.2",
   }
}

Sample node audit payload:
{
    "name": "skipDevDependencies-test",
    "requires": {
        "dotenv": "10.0.0",
        "shell-quote": "1.7.2"
    },
    "dependencies": {
        "shell-quote": {
            "version": "1.7.2"
        }
    }
}

{
    "statusCode": 400,
    "error": "Bad Request",
    "message": "Invalid package tree, run  npm install  to rebuild your package-lock.json"
}

@Driste in your case that was the root and fixing up your package json worked around it, but in the case of @A-Fitz-Nelnet I don't see a duplicate dependency between dev and normal in the package json of https://github.com/jeremylong/DependencyCheck/issues/5359#issuecomment-1482851451.
So the issue appears to be there in more cases (most likely candidate: as soon as dependencies appear in both the transitive tree of devdependencies and regular ones)

@aikebah I found that @ngrx/store-devtools is repeated. I’ll test it shortly.


Not playing back the entire audio file

This only seems to happen with lines recorded in Akorin, and not to other audio that's been loaded in. Needs more investigation

This bit might be suspect
public void Read ()
        {
            if (File.Exists(FullName) && !recorded)
            {
                byte[] rawBytes = File.ReadAllBytes(FullName);
                data = new ArraySegment<byte>(rawBytes, 46, rawBytes.Length - 46).ToList();
                stream = Bass.CreateStream(rawBytes, 0, rawBytes.Length, BassFlags.Mono);
            }
            else if (recorded)
            {
                byte[] reRead;
                var format = new WaveFormat(44100, 16, 1);
                using (MemoryStream ms = new MemoryStream())
                using (WaveFileWriter wfw = new WaveFileWriter(ms, format))
                {
                    wfw.Write(Data, Data.Length);
                    reRead = ms.GetBuffer();
                }
                stream = Bass.CreateStream(reRead, 0, reRead.Length, BassFlags.Mono);
            }
        }

Maybe prefer to read from file over rereading from data when something was just recorded. Rereading seems to cut the audio off halfway- could be the Mono flag?

Just needed to write for twice the length of Data with the WaveFileWriter lol


[Enhancement] Filter message list by read status

I'm back (and will soon follow-up on the issues you updated lately), already bothering you with new feature requests!
It would be nice to have the possibility to show only [Unread] emails in the message list for faster sorting when late with email reading. I don't know if this should be a standalone feature or if it should be just one of the possibilities of the email search feature (read/unread could be a tag to be searched for) when it is implemented, but this is a big milestone.

Right, this functionality would make sense to have. Let me think about the UI, but I'd think it would fit in a some filter/sort functionality, separate from search.

Hi, I finally got a chance to look at this now. In the above commit I've implemented basic support for message list filtering. ! will show only unread messages, @ will show only messages with attachments. The idea behind these default key mappings is that unread status is indicated in column 1 (shift 1 on US keyboard is !), etc.
The message filter list is not dynamically updated. If a new message arrives and one has filtering on, one needs to toggle the filter off/on again, for the filtered message list to refresh. So in a sense the filter behaviour is similar to message search, where the user gets a snapshot of the matching messages at that point in time. The reason for this is mainly to minimize the complexity of the code.
Let me know if you encounter any issues with the filtering. Thanks!

I wouldn't have expected the list to refresh dynamically when filtering so that sounds perfect to me, thank you for delivering this and making nmail better and better so fast.
On 2021-02-27 08:27 Kristofer Berggren<EMAIL_ADDRESS>wrote:

Hi, I finally got a chance to look at this now. In the above commit I've implemented basic support for message list filtering. ! will show only
unread messages, @ will show only messages with attachments. The idea behind these default key mappings is that unread status is indicated in
column 1 (shift 1 on US keyboard is !), etc.
The message filter list is not dynamically updated. If a new message arrives and one has filtering on, one needs to toggle the filter off/on
again, for the filtered message list to refresh. So in a sense the filter behaviour is similar to message search, where the user gets a snapshot
of the matching messages at that point in time. The reason for this is mainly to minimize the complexity of the code.
Let me know if you encounter any issues with the filtering. Thanks!
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/d99kris/nmail/issues/34#issuecomment-787029534

Ok, that's great to hear!


Review and fix all NSI Endpoints

Would be nice to get all of this in an automated way. To be checked

https://github.com/sdn-sense/autogole-monitoring/commit/660cb9f2168546a906015a63f9a726fac4e557af implemented here. There are few caveats to take care:

Need launch pod on igrok-01/2 and automatically query all APIs (NSI endpoints not accessible for outside usually).
Once config prepared, do kubectl cp siterm-agent-k8s-igrok-02.calit2.optiputer.net-5b9d496bcb-6rmfr:/root/autogole-monitoring/scripts/prometheus.yml prometheus.yml --kubeconfig $KUBE_CONF -n $KUBE_NAMESPACE and copy it to local machine;
Publish it to prometheus configmap and restart prometheus pod.

Auto config generation ready: https://github.com/sdn-sense/autogole-monitoring/commit/ca2e010f18678a8ce0f617a7c65ccc82445aa715


feat: stale flags (FE)

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes

Allows defining whether a tag indicates a permanent feature
Prevents deletion / adds icon for system generated tags
Adds tooltip for STALE tags
Standardises rendering tag content
Resets create/edit tag modal on close (before it would stay on the create tag state)
Fixes a 1px tooltip arrow gap and incorrect background

https://github.com/Flagsmith/flagsmith/assets/8608314/7a476e54-1413-4da2-9f1d-83bb818643c0
How did you test this code?

Please describe.

We should update this tooltip as well based on the addition of the is_permanent attribute:

I think you already did here https://github.com/Flagsmith/flagsmith/pull/3606/files#diff-87ffbe66e88677f5a26bc18b4f651b84a9a9264f03a350a11caa74f6df5077e2L23

We should update this tooltip as well based on the addition of the is_permanent attribute:

I think you already did here https://github.com/Flagsmith/flagsmith/pull/3606/files#diff-87ffbe66e88677f5a26bc18b4f651b84a9a9264f03a350a11caa74f6df5077e2L23

I updated the logic, but I'm talking about the actual content of the Tooltip itself (this line of code). It's still referencing the old behaviour.

Added

@matthewelwell Just checked and project settings is also persisting the newly added stale days project config

@kyle-ssg one thing I've noticed here is that the FE seems to let you edit the project setting regardless of the plan that the organisation has. The API restricts this to enterprise subscriptions only - can we update this?

@kyle-ssg we might want to consider some extra documentation around this somewhere (not exactly sure where yet), because this feature will only work with feature versioning enabled. Perhaps an extra line on the tooltips for the setting in projects settings and the one on the tag itself?

Hmmm I guess this will be a little tricky since versioning is enabled per environment.
I'll note it in the tooltip and also start managing this UI with the versioning feature flag

@kyle-ssg looks like this has broken:

@matthewelwell sorted


Add default parsers to JWT middleware

Description:
Adds the applications default parsers to the JWT middleware.
When a user was authenticating with a service for the first time, the
middleware was throwing an execption because it couldn't parse the
request. This was because the Request was not given any default parsers
which caused the request to fail on any/all "Content-Type"s.
Example stack trace from before this PR:
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/edx_rest_framework_extensions/auth/jwt/middleware.py", line 257, in _get_user_from_jwt
    if jwt_authentication_class:
  File "/usr/local/lib/python3.6/dist-packages/edx_rest_framework_extensions/auth/jwt/authentication.py", line 80, in authenticate
    self.enforce_csrf(request)
  File "/usr/local/lib/python3.6/dist-packages/edx_rest_framework_extensions/auth/jwt/authentication.py", line 167, in enforce_csrf
    reason = check.process_view(request, None, (), {})
  File "/usr/local/lib/python3.6/dist-packages/django/middleware/csrf.py", line 295, in process_view
    request_csrf_token = request.POST.get('csrfmiddlewaretoken', '')
  File "/usr/local/lib/python3.6/dist-packages/rest_framework/request.py", line 424, in POST
    self._load_data_and_files()
  File "/usr/local/lib/python3.6/dist-packages/rest_framework/request.py", line 272, in _load_data_and_files
    self._data, self._files = self._parse()
  File "/usr/local/lib/python3.6/dist-packages/rest_framework/request.py", line 344, in _parse
    raise exceptions.UnsupportedMediaType(media_type)
rest_framework.exceptions.UnsupportedMediaType: Unsupported media type "application/json" in request.

JIRA:
MICROBA-498
Additional Details

Dependencies:: N/A
Merge deadline: N/A
Testing instructions:

Create a new account in LMS
Make a request to a service that has ENABLE_SET_REQUEST_USER_FOR_JWT_COOKIE set to True (currently ecommerce or demographics).
See an UnsupportedMediaType in the logs because there are no matching parsers for the Content-Type passed in (often "application/json")
Create another new account in LMS
Upgrade the other service to use this branch
Again make a request to the service and notice the exception is gone.

Author concerns: N/A

Reviewers:

[x] tag reviewer

Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[x] Version bump if needed
[x] Changelog record added
[x] Documentation updated (not only docstrings)
[x] Commits are squashed

Post merge:

[ ] Create a tag
[ ] Check new version is pushed to PyPi after tag-triggered build is
finished.
[ ] Delete working branch (if not needed anymore)

Added a test version that fails if the parsers option isn't there. Merging and releasing.

Thanks so much for this fix tuchfarber!


Create Unit Test

As a ...  Developer
I need ...  A unit test for the Create request
So that ...  I can assure that my Create request functions properly and that I have covered as many possible flaws in my program as possible
Assumptions:

... I have a fully functioning Create request

Acceptance Criteria:
Given ...A working Create test
When ...
*...I have validation for the use of invalid data properties of the recommendation
*...I have validation for when ids that already exist (avoid creating existing objects)
*...I have validation for when an empty Create Request is made
*...I have an appropriate response for when an invalid Create Request is made
*...I have successfully generated a new recommendation upon a successful Create Request
*...I have 90%+ code coverage
Then ... My  unit test will be ready for deployment.

Unit tests are not stories.


Error during install on node 6 due to outdated optipng-bin and jpegtran-bin dependencies

express-cdn is not compatible with node 6, giving the following error with optipng-bin and jpegtran-bin.
npm ERR! Darwin 15.5.0
npm ERR! argv "/Users/charlesdegroote/.nvm/versions/node/v6.2.0/bin/node" "/Users/charlesdegroote/.nvm/versions/node/v6.2.0/bin/npm" "install"
npm ERR! node v6.2.0
npm ERR! npm  v3.8.9
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>postinstall: `node index.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node index.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the optipng-bin package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node index.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs optipng-bin
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls optipng-bin
npm ERR! There is likely additional logging output above.

I tried to update the two dependencies but express-cdn is not compatible with them.

duplicate as #101

fixed in 0.2.9


annotation overlays are a tad dark

Unclear if this is a blending issue or an alpha channel issue (or both)

This was a bug caused by a space existing after the channel name. Channel names and tokens now have whitespace stripped off when the VizProject is created.


Allow changing scrollEnabled to true from initially false

We have a use case where a carousel is the second page of a pagingEnabled={true} scrollview. I know that nesting is frowned upon, but we've gotten it to mostly work. We set the carousel's scrollEnabled to true, and the outer's to false, when the outer scrollview hits the second page.
However, we noticed the first card required scrolling all the way to the next page before we could easily swipe between cards:

If we allow changing scrollEnabled to true in componentWillReceiveProps, we can accept a scrollEnabled change before any user interaction:

This might also address some issues referencing the scroll "bouncing back" to the first card.

@tomauty Thanks for the PR and for making it visual!
I was going to ask you to keep the 4-spaces indentation before merging it, but I don't want a commit for 2 missing spaces ;-)

Thanks! And yeah, we use tabs at work. Thought I got em all but guess I missed some :^)


Fix race with replace in pokemon-showdown launcher

Trying to go to bed but I keep thinking about this - i think it's wrong, its too risky. Instead we should make it so we always build except in the simulate-battle case, at which point we rely on the heuristic. No reason to cause other users headaches.

Alright, I think this now has the correct tradeoff between speed and safety.

My concerns do apply to all of the command-line functionality besides starting the server, not just simulate-battle.


Mutate out return value;

Is your feature request related to a problem? Please describe.
Stryker does not have return value substitution mutator
Describe the solution you'd like
a mutator which changes
return <expr>

to
<expr>
return <constant of type of expr>

Describe alternatives you've considered
A generic mutator replacing <expr> with const dummyConst = <expr>;<constant with type of expr>).
Additional context
Demonstration
import { CalledWithMock, mockFn } from "jest-mock-extended";

function foo(bar: () => boolean): boolean {
  return bar()
}

function fooMutated(bar: () => boolean): boolean {
  bar()
  return false;
}

describe("No mutation for return value substitution", () => {

  let myBar: CalledWithMock<any, any> & Function;

  beforeEach(() => {
    myBar = mockFn()
    myBar.calledWith().mockReturnValue(true);
  });

  test("we check for calling the function ", () => {
    foo(myBar)
    expect(myBar.mock.calls.length).toBe(1)

  });

  test("we check for calling the function, on the mutated function ", () => {
    fooMutated(myBar)
    expect(myBar.mock.calls.length).toBe(1)
  });

  test("we check the return value", () => {
    expect(foo(myBar)).toBe(true);
  });

  test("the mutant is killed here", () => {
    expect(fooMutated(myBar)).toBe(true);
  });

});

Hi @magwas 🙋♂

a mutator which changes
return <expr>

to
<expr>
return <constant of type of expr>

This is almost impossible to do. That is, it is difficult to know the type of expression. This is easier to do:
- return <expr>
+ <expr>
+ return;

A few thoughts:

It might result in equivalent mutants when the <expr> results in undefined.
It might result in hard-to-kill mutants, when the <expr> results in a nullish value
The mutant might not add much value if <expr> was already mutated.

In your example, what is bar's body? Isn't the return value already mutated there?

"it is difficult to know the type of expression"
My point exactly. Some mutators are only possible if the types are known. I would like to discuss how to achieve it with stryker.
This specific mutator is to make sure that the return value is checked. Forgetting to test for it is a common mistake.
The function under test was:
  run = (changeUserAction: ActionArgument<typeof changeUser>, storeConfigAction: ActionArgument<typeof storeConfig>): Observable<Heroes> => {
    const user = changeUserAction.payload;
    if (user == null)
      return of();
    const baseURL = storeConfigAction.payload.baseUrl
    return this.httpClient.get<Heroes>(baseURL.concat(UrlMapConstants.GET_HEROES_URL))
  };


Rescan certains posts again after a short period

See Charcoal-SE/metasmoke#565 for more info.
Rescans a post instead of handling scan results directly if it meets certain criteria:

is a question
is on WordPress Development
caught for "bad keyword in body"
has an aggregate reason weight of no more than 100 for its initial version
during spam hour (extended to 2 AM to 2 PM UTC - 2 hours before and after typical spam hour)

Closing stubs