auto-detect default branch on [github nycrc osslifecycle]

..with this one wierd trick :sparkles:
Shout out to @andersk for this excellent suggestion which allows us to solve this problem in a large number of cases with one very small code change and no impact on performance :smile:
Arguably this makes a very slight backwards-incompatible change in that if your repo has a branch which is called master but which isn't the default, right now https://img.shields.io/osslifecycle/user/repo is looking at your master (by name) branch but this will change it to look at your default branch (whatever that is called). I'm not particularly concerned about this. I think it unlikely many users are consciously relying on that behaviour and we can regard this as a fix which brings us into line with how it should have worked in the first place/what most users will expect anyway.
As a side note, we should probably move nycrc and osslifecycle under the github service family and set up redirects, but that's another issue.
I've only addressed

github
nycrc
osslifecycle

in this PR. The other services mentioned in https://github.com/badges/shields/issues/5215#issuecomment-646293697 need a different solution (WIP), although solving this for all the affected services under the github family is not to be sniffed at.
I think this should also give us a fix for #4114 ?
There will be some failing tests due to #5230 . Once #5232 is merged, I will rebase to get the tests passing.

Warnings

:warning:
This PR modified service code for github but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against 089299514f4781ea085d563eba1471eeaccc1471

This is a fantastically simple solution, love it!
I suppose another rather fringe case: a repo that has both master and HEAD branches, with master as the current default (which i think GitHub will technically let you do) :eyes:
IMO this is still the right approach, maybe we can just enumerate the handful of edge cases like this in the issue with workarounds should anyone run into these in "real life". Just the tradeoffs we're having to make to avoid doubling the badge response time and doubling our GH token quota usage that the alternatives would've required

I suppose another rather fringe case: a repo that has both master and HEAD branches, with master as the current default (which i think GitHub will technically let you do)

You can't have a branch which is explicitly called HEAD in git:
$ git init
Initialised empty Git repository in /home/chris/temp/asdf/.git/
$ git checkout -b HEAD
fatal: 'HEAD' is not a valid branch name.

You also can't make a branch called HEAD~1 or HEAD@{1} etc for the same reason.
..or do you mean something different?

GitHub (not git) will apparently let you do it from the web portal which I found surprising. To your point though, doing so would be spectacularly unhelpful given the client side implications and thus not real/valid badge use case.
I would still maintain however that as part of this change it would be a good idea to create (and pin) and issue that succinctly summarizes the change we're making, and any known edge cases and workarounds. That way if anyone does encounter an edge case they can easily find the workaround, and/or report any cases we haven't seen yet (as opposed to having to go through lengthy issues/PRs)

So you can :open_mouth: Every day's a school day :mortar_board:

True story: I nearly just tried doing this on a repo I actually care about, then I realised that I might be about to make it impossible to check the repo out so I switched and did it on a test repo :rofl:  I find it incredibly surprising that github lets you do that. I might have a quick play and see what nonsense ensues if I actually try and do something with this.
Side note: I'm also working on another (less nice) PR for bitbucket, gitlab and travis ci which I'm aiming to submit tonight or tomorrow.

I had a quick play with this. You're totally right that if you do create a branch called HEAD, github's UI/API assumes you mean HEAD (the branch) not HEAD (the ref) so if you make a branch called HEAD and its not the default then this would result in unexpected behaviour. If you try and check it out locally (at least with the version of git I have), you basically can't interact with that branch at all though so it is quite pointless. I guess you could have a branch called HEAD on a repo that you only ever interact with via the github UI, or (more likely) a branch called HEAD that you created, decided was a terrible idea but then didn't delete.
I think we both still think its the right approach in the grand scheme of things, but dammit that is an annoying edge case.

I think we both still think its the right approach in the grand scheme of things, but dammit that is an annoying edge case.

Absolutely, on both counts! IMHO this is much better than the alternative, and I'd rather have unexpected behavior for the hypothetical (and presumably nonexistent) HEAD branch case instead of impacting every single valid/standard badge use case.
Have you had a chance to do any additional testing with this? The code changes and approach LGTM, though I think it'd be worth manually validating the behavior with a sample repo and contrived data if no one's done so yet

As a side note, we should probably move nycrc and osslifecycle under the github service family and set up redirects, but that's another issue

Agreed. I keep thinking I've convinced myself one way or the other, only to then change my mind

Have you had a chance to do any additional testing with this?

Yeah good question. I've tried making some ad-hoc requests on some repos where the default branch is/isn't master. It does appear to work as intended, but its not a super well documented feature.
In terms of automated testing, its essentially a feature of an external service so I haven't added any additional tests - I'm just relying on the existing tests to verify the change doesn't break anything really.
If you've got any other thoughts on stuff I should look at, I'm all ears..

I've tried making some ad-hoc requests on some repos where the default branch is/isn't master. It does appear to work as intended, but its not a super well documented feature.

That's all I had in mind. Was going to do the same but figured I'd check first.

In terms of automated testing, its essentially a feature of an external service so I haven't added any additional tests - I'm just relying on the existing tests to verify the change doesn't break anything really.

If you've got any other thoughts on stuff I should look at, I'm all ears..

Nah I don't think we need anything else, sounds like it's working as expected :+1:


⚠ Samsung has degraded performance

In 8b6da07, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2358 ms

Resolved: Samsung performance has improved in eca0290.


How to extend the dictionary

Is it possible to add external english2kanji data?
Thanks a lot 🤗

Does english2kanji data means dictionary like Kana to kanji conversion?
alphabet2kana dictionary data is only for one-to-one conversions.  e.g., a is エー, 0 is ゼロ.
Conversion to Kanji, including Hiragana, Katakana, can not be uniquely identify.  e.g., a can convert  あ,  ア,  阿,  亜, etc.


Make util.empty public for conversions

Some downstream components want to use a StringSet in their internal
objects, but leaving util.empty private means that generated conversions
cannot create constructors generically for those objects.  This makes
Empty public until such a time as the generator supports private types.

Over to @lavalamp.

LGTM

Added godoc

LGTM

@thockin for ok-to-merge

LGTM


Plan term pdf

…ates

Fixing title and desc:
add plan term header to pdf and change agreement dates to plan term dates
#416


"Only update existing fields" doesn't work when key is empty

"Only update existing fields" option doesn't work if the property/key doesn't have a value.
With the option enabled, the following will not have the timestamp added after an edit:
modified:

But this will:
modified: test

Expected: When enabled, only update files with modified:(or custom) present, regardless if it has a value or not.
Thanks.

Fixed in 1.3.2

It works, thanks!


[release/2.6] integration tests: update certificates

Cherry-pick #2594

The decrease in coverage looks unrelated to the changes here, so this otherwise looks good.

The decreased coverage is related to a subset of tests being run on PRs, will no one rid this project of codecov?


[UX Issue]: Page index is not reverted to 1 after applying the filter

Hi all,
It would be nice if we could fix this UX issue.
The default paging number is not reverted to 1 after applying the filter.
 E.g: Alfreds Futterkiste [ALFKI]
Grid is showing empty record and the pager is showing Page "9/1"

The expected result is the Page index should be reverted back to 1 instead of 9
Thank you.

Related with issue: https://github.com/volkanceylan/Serenity/issues/2177

My work around:
protected quickFilterChange(e: JQueryEventObject): void {
super.quickFilterChange(e);
let currentPage = Number(this.element.find('input.slick-pg-current').val());
let btnFirst = this.element.find('div.slick-pg-first');
if (currentPage > 1 && btnFirst)
	btnFirst.click();

}
It would be nice if we can include this by default.

I agree that's confusing, this commit should fix that:
d1646e9de64625b06b63e04ed613f5ab346ee634
It also does for sorting, quick filtering, advanced filtering

You are fast, thank you Volkanceylan.


[Rust] [Experiment] Trigonometry kernels

This is on top of #9297
I was curious if (ab)using the compute::unary kernel would perform better on slightly complex functions.
I implemented the Haversine function, which calculates the distance between two geographic coordinates.
I then benchmarked an implementation that I tried to simplify and optimise with unary kernels, vs one that I'd have to write if I couldn't use the unary kernels for things like:

arithmetics with scalars
functions that would otherwise require generating intermediate arrays (e.g. sin(x) * cos(x) would be multiply(sin(x), cos(x)))

The function that uses unary kernels for the above, is slightly faster.
I ran this on an M1 CPU, with the below options
cargo bench --bench trigonometry_kernels
cargo bench --bench trigonometry_kernels --features simd
RUSTFLAGS="-C target-cpu=native" cargo bench --bench trigonometry_kernels
RUSTFLAGS="-C target-cpu=native" cargo bench --bench trigonometry_kernels --features simd

haversine_no_unary 512  time:   [14.074 us 14.140 us 14.216 us]

haversine_unary 512     time:   [11.191 us 11.308 us 11.436 us]
haversine_no_unary_nulls 512
                        time:   [15.902 us 15.985 us 16.083 us]

haversine_unary_nulls 512
                        time:   [12.486 us 12.552 us 12.625 us]

The biggest benefit is from setting the RUSTFLAGS, the non-null benches go 3-10% faster.

IMO looked good to me (before the rebase), however, a more through review would need a rebase xD

IMO looked good to me (before the rebase), however, a more through review would need a rebase xD

I've rebased.
Ideas for the PR before I work more on it:

Are the trig kernels fine (is the direction good)?
Should I remove Haversine? I was using it to evaluate the performance benefit of using unary

I've rebased.
Ideas for the PR before I work more on it:

Are the trig kernels fine (is the direction good)?
Should I remove Haversine? I was using it to evaluate the performance benefit of using unary

Codecov Report

Merging #9313 (ef36a3d) into master (53d392c) will increase coverage by 0.00%.
The diff coverage is 87.01%.

@@           Coverage Diff           @@
##           master    #9313   +/-   ##
=======================================
  Coverage   81.90%   81.91%
=======================================
  Files         216      217    +1
  Lines       52990    53066   +76
=======================================
+ Hits        43402    43468   +66
- Misses       9588     9598   +10

Impacted Files
Coverage Δ

rust/arrow/src/compute/kernels/trigonometry.rs
86.48% <86.48%> (ø)

rust/arrow/src/compute/kernels/arity.rs
92.85% <100.00%> (+1.19%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #9313 (ef36a3d) into master (53d392c) will increase coverage by 0.00%.
The diff coverage is 87.01%.

@@           Coverage Diff           @@
##           master    #9313   +/-   ##
=======================================
  Coverage   81.90%   81.91%
=======================================
  Files         216      217    +1
  Lines       52990    53066   +76
=======================================
+ Hits        43402    43468   +66
- Misses       9588     9598   +10

Impacted Files
Coverage Δ

rust/arrow/src/compute/kernels/trigonometry.rs
86.48% <86.48%> (ø)

rust/arrow/src/compute/kernels/arity.rs
92.85% <100.00%> (+1.19%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks @jorgecarleitao, I'm currently constrained with time during the week. I'll finish this up over the weekend

@nevi-me I am closing this PR for the time being to clean up the Rust/Arrow PR backlog.  Please let me know if this is a mistake


added windows 11

after reading this article, i confirmed windows 11 is supported

Learn Build status updates of commit fbdf374:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows/security/threat-protection/security-policy-settings/modify-firmware-environment-values.md
:white_check_mark:Succeeded

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


Registering global helpers

Is it possible to register helper functions in a global manner that the calls to render uses the helper when referenced.
I would like to i18n my templates and as far as I can see the helper function needs to be sent with each call to render.

What about in 2015? ;)

I'd go more with something of the likes of @bobthecow's example: https://github.com/bobthecow/mustache.php/commit/f77cf3729de0e46d6187da2353a4ab438657b6fe.
Someone could write it and publish it as a plugin.

Seems like it's for a PHP compiling process, isn't it possible to add a helper, or a global filter/function/other, without having to compile server-side in PHP ?

Sorry, I should have explained myself better. I meant that I prefer @bobthecow's syntax where text is surrounded by a lambda, which performs the translation.
It's very easy to hack that functionality into mustache.js. More or less along the lines of:
function i18nLambda (text, render) {
  text = dict[text] || text
  return render(text)
}

var Context = mustache.Context
Context.prototype._lookup = Context.prototype.lookup
Context.prototype.lookup = function i18nLookup (name) {
  return (name === 'i18n' ? i18nLambda : this._lookup(name)
}

You could then:
{{#i18n}}Hello.{{/i18n}} {{#i18n}}My name is {{ name }}.{{/i18n}}

Thanks @dasilvacontin this trick worked perfectly!

Published as an npm module: https://github.com/dasilvacontin/mustache-i18n


Extend "molecule" related playbooks

In order to allow molecule job reproducer, we must slightly extend the
existing CI playbook, to alow setting a custom hosts as well as
passing down environment files/parameters to ensure we're reproducing
the right job.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

/approve
/lgtm

/approve

/retest
/lgtm

recheck

@pablintino proposal:
current run failed one molecule, rhol_crc, due to a collection to fail to install:
fatal: unable to access 'https://github.com/containers/ansible-podman-collections/': Could not resolve host: github.com
ERROR! Failed to clone a Git repository from `[https://github.com/containers/ansible-podman-collections`.](https://github.com/containers/ansible-podman-collections%60.)

Can we consider this patch as green and override rdoproject.org/github-check ?

@pablintino proposal: current run failed one molecule, rhol_crc, due to a collection to fail to install:
fatal: unable to access 'https://github.com/containers/ansible-podman-collections/': Could not resolve host: github.com
ERROR! Failed to clone a Git repository from `[https://github.com/containers/ansible-podman-collections`.](https://github.com/containers/ansible-podman-collections%60.)

Can we consider this patch as green and override rdoproject.org/github-check ?

Yeah, I 'm more than fine with that.

/override rdoproject.org/github-check
/lgtm


【用户管理】补全黑名单管理相关API

close #250
微信文档真的坑，
openid_list写成了opened_list（手动测试过了）。

Codecov Report

Merging #254 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #254      +/-   ##
==========================================
- Coverage   76.14%   76.04%   -0.11%
==========================================
  Files          72       72
  Lines        3002     3010       +8
==========================================
+ Hits         2286     2289       +3
- Misses        716      721       +5

Impacted Files
Coverage Δ

wechatpy/enterprise/client/api/tag.py
34.88% <0%> (+0.59%)
:arrow_up:

Continue to review full report at Codecov.

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


Local Docker Setup commands not working after new changes to it

We tried setting up local docker setup at our end in multiple systems. We are facing the below issues:

The commands make setup-local and even new command stated by @danoswaltCL which is make setup doesn't seem to work to install the npm modules. When I checked through the Makefile in dev branch, there is no command for setup, instead we still have setup-local which we had before nix changes and that time it was working for me. Can @kawcl or @danoswaltCL verify at their ends and let us know. (For now we are running npm ci commands manually in all folder locations to install packages )

We see 2 errors (frontend and backend error) while running the make command to generate all the docker images and start them. Though, I don't know if they are affecting some functionality or not.

This is my system specific, I am not able to view logs on right clicking container and selecting View Logs with the error: docker command not found. Though it works for yashil. Any help on this may be a plus for me.

hey @ppratikcr7,
Issue 1

The Makefile command should be make setup-local in the dev branch.  When that command is used the Makefile will run ./docker-setup.sh -l.  This script uses NixOS so that the node & npm version is standardized for everyone. The setup for the NixOS script is in docker.nix, right now it will make sure everyone installs the node_packages using node v12. Whenever we migrate to node v16 we will need to update to nodejs-12_x in docker.nix to nodejs-16_x.  Long story short, the script uses NixOS so that means people will need to install that on their computer right now. I'm contemplating using docker again for this step to run nix in a container for this step so people wouldn't need to explicitly install it, but I need to work on that.

Issue 2

I'm not exactly sure on this one because I couldn't find any info to confirm what I think these errors are for.  However they haven't affected functionality for me or anyone else that I know has tried this step so far here so I think they are OK for now.

Issue 3

I'm not sure why this is occurring.  It seems it might be an issue with the vscode plugin. This is the plugin's github, though I haven't found anything with a quick search. Otherwise maybe try reinstalling the plugin?

I'm contemplating using docker again for this step to run nix in a container for this step so people wouldn't need to explicitly install it, but I need to work on that.

Interesting idea.  I'd rather require only nix (and use nix to get docker), but that might not be possible since docker requires a daemon to be running.

@kawcl Thanks for the clarifications, let us try as per your comments. We will get back in case we still face any issue.

on point 3, it is probably issue with it finding docker in PATH, which docker can show where it's installed, could echo $PATH to see if that PATH seems reachable.
Other thing to try would be to open up the docker desktop app, all same things are going to be found there, that's where I tend to look at the logs anyway, and some things I see googling tdocker command not found seem to suggest that after opening the docker app the first time, in background it fixed the PATH issue and docker commands worked as expected.

Yea I'm not sure which direction would be best..
Have nix maintain the dependencies and run the docker containers or have docker run and manage nix which manages the dependencies.  There is a docker image for nix but a quick search for how to use it properly hasn't proven that fruitful. I have, however, found a source for running docker containers with nix
I'd like to make it so you only need to install either-or on your machine instead of both.
I have to take a break from this though to focus on some other work right now, I'll try and revisit it soon.

@kawcl Thanks, installing Nix did work for me, I am able to run make setup-local now.
Let me know if this issue should be closed if we don't want anything else to be discussed here on this ticket, else it's fine if we continue to discuss further on Nix.

@danoswaltCL I will check with your resolution and let you know. Thanks :)

@kawcl We also observed a weird behaviour of docker script where it creates the containers with a different name. For e.g.: Instead of upgrade-postgres-1 the container name is upgrade_postgres_1. So, we have to change the environment variable in the local docker env file: TYPEORM_HOST=upgrade-postgres-1. Any idea about this inconsistency?

@ppratikcr7
Yes we noticed before and we should document it, this is likely the issue where in docker setting Use Docker Compose V2 needs to be toggled on:

@ppratikcr7 Dan would be correct on this, depending on which version of compose you are using it append names differently with "_" or "-"

Okay, thanks. Yeah, we should document all this thing in our Upgrade documentation.

@kawcl @danoswaltCL Should we close this issue?


DataBlock.summary(show_batch=True) as a shortcut to dls.show_batch()

Shorten the often-repeated 3-line debugging idiom of seeing the batch from:

tweak DataBlock(...... long list of options ...)
create DataLoaders
then call dls.show_batch(<some args>)

to

tweak DataBlock(....)
call DataBlock.summary(path, show_batch=True, **some_args)

Previously I had a quick discussion with @lgvaz on how to refactor the common 3-step dls.show_batch(...) to something simpler.
Then I saw DataBlock.summary(), which is the suggested debugging tool for DataBlock in fastbook, and already creates a transient DataLoaders object --- that would be a convenient spot to add more convenience to the common debugging task of tweaking DataBlock and  just want to see the effect on one batch`, wouldn't it?

p.s. I also added example documentation in the nb -- wonder if it might be useful to fastbook's 05_pet_breeds.ipynb  as well, just an idea.

Thanks, this sounds useful!
Note that it won't change the output in the notebook 05 of fastbook since that summary fails.

Now does the DataBlock.summary() create DataLoaders for us? If so then how do we use them? I used to do dls = DataBlock.dataloaders()

No. The just ends by printing a show_batch.

Note that it won't change the output in the notebook 05 of fastbook since that summary fails.

Ah yes, you're right, it won't change that case.  I wonder if it maybe useful to add an example  in the 05 notebook, to demo this shortcut summary(.... show_batch=True, ncols=..., nrows=...) .
@tumbleintoyourheart :

Now does the DataBlock.summary() create DataLoaders for us

summary() was designed as a debugging aid I believe, and not for constructing a DataLoaders object, not to mention the multiple ways to build a DataLoaders.
This change is to tag along the debugging tool of show_batch() with that of summary()'s.
If you don't care about the DataBlock() object nor the summary() debugging output, and only need to use the DataLoaders object, and consider appending the call to .dataloaders(...)  to DataBlock(.... long list of options ...):
# get handles to `DataLoaders`, then `Learner`, and disregard the `DataBlock` object
dls = DataBlock(blocks = (ImageBlock, CategoryBlock),
                 get_items=get_image_files,
                 splitter=RandomSplitter(seed=42),
                 get_y=using_attr(RegexLabeller(r'(.+)_\d+.jpg$'), 'name'),
                 item_tfms=Resize(460),
                 batch_tfms=aug_transforms(size=224, min_scale=0.75)).dataloaders(path/"images)
learn = cnn_learner(dls, resnet34, metrics=error_rate)

Or, if you only need the DataBlock object, and don't care about the DataLoaders object, you may consider calling DataBlock.dataloaders(...) only when you are ready to use the DataLoaders, such as when creating the Learner object:
# get handles to `DataLoaders`, then `Learner`, and disregard the `DataBlock` object
pets = DataBlock(blocks = (ImageBlock, CategoryBlock),
                 get_items=get_image_files,
                 splitter=RandomSplitter(seed=42),
                 get_y=using_attr(RegexLabeller(r'(.+)_\d+.jpg$'), 'name'),
                 item_tfms=Resize(460),
                 batch_tfms=aug_transforms(size=224, min_scale=0.75))
pets.summary(... do some debugging, maybe show_batch=True,.....)

learn = cnn_learner(pets.dataloaders(path=....), resnet34, metrics=error_rate)

I wonder if it maybe useful to add an example in the 05 notebook

No, that would make the notebook be out of phase with the book now. We can have examples in the docs and the tutorials.

Got it, thx!


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 54 ms

Resolved: ClamAV performance has improved in 71d9bb6.


IDIR / BCeID / Keycloak Integrations

Describe the task
Commencing IDIR login implementation with keycloak
Acceptance Criteria

[x] IDIR access provisioned by SSO team
[ ] BCeID documentation/onboarding completed after we made request to SSO team.
[ ] IDIR integrations working for api and frontend as appropriate
[x] Technical question: are we hosting own Keycloak instance for full RBAC? Probably?
[ ] BCeID integration
[ ] User can login with IDIR (or BCeID?)

Additional context

Do we need BCeID integration at this time? Discuss and create ticket as appropriate.
Or here

Made SSO dev request.

Client Secret and Key provisioned for DEV environment
Documentations required only for PROD
Ministry Keycloak added as an IDP in our project Keycloak

configure front end with kc_idp_hit values as below
IDIR: kc_idp_hint=idir
BCeID Basic: kc_idp_hint=bceid-basic
BCeID Business: kc_idp_hint=bceid-business
BCeID Basic or Business: kc_idp_hint=bceid-basic-and-business
GitHub: kc_idp_hint=github (Available in DEV and TEST)

POC task completed for IDIR login/logout
Working on BCeID login/logout

@nikhila-aot to update with info regarding secondary request to separate BCeID / IDIR login screens.

We now know how to integrate IDIR and BCeID for our login screens. There are 2 options :-

Raise a single SSO  request with the identity providers we require, in our case IDIR and Basic or Business BCeID, but this prompts the user with both IDIR and BCeID logins on redirection
In case we require separate user actions for IDIR and BCeID login, we have to raise individual SSO request, one for IDIR and one for Basic or Business BCeID


feat: update notifications-panel version to 0.0.9

REVIEW
@QilongTang

hi @Enzo707 Do we need a new release with this?


libcxx: Undefine GLIBC to avoid sim environment conflict with the host

Summary
Remove accidently in PR: https://github.com/apache/incubator-nuttx/pull/2243
Impact
without this patch, libcxx can't pass the compile with toolchain's libm.
Testing

__GLIBC__ is not a predefined macro. it's defined by system headers.
it canl be defined again by system headers included after __config.
IMO it's better to avoid checking __GLIBC__ in libcxx.

__GLIBC__ is not a predefined macro. it's defined by system headers.
it canl be defined again by system headers included after __config.
IMO it's better to avoid checking __GLIBC__ in libcxx.

Yes, but it isn't avoidable if we want to use the toolchain library(libm). match.h provided by system bring in GLIBC macro. Undefine __GLIBC__ could fix the most issue.

__GLIBC__

i closed this by mistake. sorry. i reopened it.

__GLIBC__ is not a predefined macro. it's defined by system headers.
it canl be defined again by system headers included after __config.
IMO it's better to avoid checking __GLIBC__ in libcxx.

Yes, but it isn't avoidable if we want to use the toolchain library(libm). math.h provided by system bring in __GLIBC__ macro. Undefine __GLIBC__ could fix the most issue so far.

which __GLIBC__ check in libcxx causes the issue with the toolchain libm?
i guess it's better to make it like !defined(__NuttX__) && defined(__GLIBC__).

__GLIBC__ is not a predefined macro. it's defined by system headers.
it canl be defined again by system headers included after __config.
IMO it's better to avoid checking __GLIBC__ in libcxx.

Yes, but it isn't avoidable if we want to use the toolchain library(libm). math.h provided by system bring in __GLIBC__ macro. Undefine __GLIBC__ could fix the most issue so far.

which __GLIBC__ check in libcxx causes the issue with the toolchain libm?

here: https://github.com/apache/incubator-nuttx/pull/2283/commits/85b64208a5e378819be83bade7873b6027bd1bf9

i guess it's better to make it like !defined(__NuttX__) && defined(__GLIBC__).

Ok, let's fix GLIBC reference instead.


Implementing styles for checkbox and radio button

The src/forms/index.html file seems like it could be deleted entirely, right?
Nice work on this. One question - how hard would it be to make it possible to support the following html contract?
<div class="omrs-checkbox">
  <label>
    <input type="checkbox" name="thing" />
    The label text
  </label>
</div>

^ this is nice because it doesn't require html id attributes. However, sometimes the id attribute is also nice, so it would be cool to support both. Might be hard to support that though. What do you think?

Not difficult i have implemented it as recommended

What I said is that it would be good to have it work both ways (both with input inside of label and with input as sibling of label). It might not be worth implementing that, though.

@donaldkibet , any updates on last comment from @joeldenning ?

@donaldkibet , any updates on last comment from @joeldenning ?

working on it today

I am no longer going to review this. @fatmali @brandones can you take over


Webhooks/events for integration with external services

We should be able to configure (on a per-course or per-assignment basis) actions that will be triggered when certain events happen, such as receiving a new backup. This could either send an HTTP POST request to a URL, or call one of our "extensions" that can be used for this purpose. This will allow cleaner separation of concerns, and allow staff to swap in their own custom autograder instead of using ours.
Potential use cases:

Running the autograder for each new submission
Autograding all submissions
Running moss on all submissions
Uploading to Canvas API on each new score
Notifying an online cheating detection system with each new backup
Updating hog contest rankings for each new submission

We should look at the GitHub webhooks docs for inspiration.

Here's an implementation:

There are a certain set of events that we support (New Backup, New Submission, New Score, New Group)
There'll be a webhook table that will allow us to create webhooks specific to a course or specific to an assignment.
The route responsible for creating the new object will also be responsible for making the webhook.

The better, but more annoying alternative is to have commit hooks on the SQLAlchemy models

The actual sending of the webhook will be a deferred task (with it's own queue).

It should be delayed because sending the webhooks should not ever delay the original task.

+1 on sending webhooks being a deferred task. We should also retry delivery, and show which delivery attempts have failed.
The body of the webhook request should contain the JSON representation of the new object.

Here's a possible schema:
class Webhook(Model):
    event_type  # enum of possible event types
    course_id
    assignment_id  # nullable. if null, applies to entire course.
    webhook_url  # URL to POST to
    secret_token  # security token to include with webhook request

I'm not sure how this would handle our own integrations (e.g. bCourses). We could just fill in an internal URL (e.g. the URL is https://okpy.org/...)

A course has requested a basic version of this (trigger a webhook upon submission).


No provider for SQLite

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x ] 3.x
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Ionic unable to start because No provider for SQLite.
Expected behavior:
SQLite available and working.
Steps to reproduce:
Create a new project, add SQLite support with ionic-native: https://ionicframework.com/docs/native/sqlite/
Other information:
Ionic Framework: 3.0.1
Ionic Native: ^3.5.0
Ionic App Scripts: 1.3.0
Angular Core: 4.0.0
Angular Compiler CLI: 4.0.0
Node: 7.8.0
OS Platform: OS X El Capitan
Navigator Platform: MacIntel
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36

This sounds like more of a support request than a bug report. I suspect failure to declare SQLite as a provider in the app module as documented at the end of here.

Hi there! This doesn't appear to be an issue with the framework it self, so I'm going to close it. Please post your question on the Ionic Forum instead.

Thanks a lot for help. It was really my fault, @rapropos hint helped.


Danganronpa V3: Killing Harmony Demo version [PCSB01130]

App summary

App name: Danganronpa V3: Killing Harmony Demo version
App serial: PCSB01130
App version: 1.0

Vita3K summary

Version: v0.1.6 2980-856c1628
Build number: 2980-856c1628
Commit hash: Vita3K/Vita3K@856c1628
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: User01
OS: Windows 10
CPU: AMD Ryzen 5 2600 Six-Core Processor
GPU: NVIDIA GeForce RTX 3060 Ti/PCIe/SSE2
RAM: 8 GB

Issues

Firstly going into the options menu and the new game crashes the game due to an unknown format on Vulkan
[15:35:10.656] |E| [renderer::vulkan::texture::translate_swizzle]: Unknown format 0x88000000
[15:35:10.657] |E| [renderer::vulkan::texture::translate_format]: Unknown format 0x88000000
on OpenGL, The Game goes ingame- with good performance but major graphical issues
Screenshots

Log
vita3k.log
Vulkan crash.log
Recommended labels

ingame-
Graphical bug
C?

as https://github.com/Vita3K/Vita3K/commit/b5aa1ec5dd373ed7f8f65b1b7593392d4b145477, the text issue is fixed, which i believe the game is now playable.
Screenshots

Log
vita3k.log
Recommended labels
Playable


182488560 claw animation

Based on the network tab, it looks like the assets don't get loaded until they're needed. That doesn't necessarily mean when they're visible--for example, if a claw animation is in the curriculum, the images will be loaded basically immediately. But if there is no dataflow node with the images, they will not be loaded, even if there is a dataflow tile.


We now have them


Dragging with right mouse button in output windows should work without focus

Currently you have to actively select the output window to drag the view.
This inconsistent to the graph window that can be dragged when hovered without focusing first.

fwiw im not experiencing this issue, i can pan the output view with right click without giving the window focus, and dragging it doesnt give it focus either

Very interesting! I looked into this, and the problem seems to be limited to the 3d-content:
https://user-images.githubusercontent.com/1732545/193640195-d3473042-fa54-48ff-8ec3-50b1d514845b.mp4


Updated README.md

Improved the Readability

Fixed the broken link to the research paper
Center aligned the tables(to improve readability)
Fixed grammatical error

@Aryanutkarsh it should be DINOv2 models produce without s. It was correct.

Fixed the broken link to the research paper

Note there is already a paper link in the link bar just above, so I would perhaps not call this broken and I am not sure this is exactly needed (but fine).

Center aligned the tables

That's also not exactly critical (but again fine). However this does not seem to work as intended, some of the contents of the table (e.g. "ViT-g/14") gets centered as well (using GitHub's markdown preview).

@patricklabatut sir thank you so much for being so understanding and guiding. I am new to open source,I want work with people like you rather making pr to first contribution repos.This thought me how to resolve the reviews
Thank you Sir 🙂

@Aryanutkarsh Remove the div elements that were introduced, I don't think they should not be needed anymore with the "margin: auto" styling. Test and if that's indeed fine, that should be good to merge.

Done sir 👍


Add Deployed Documentation Link

Here's the documentation deployed from the Wiki!
The platform should be a much more pleasant writing experience (a WYSIWIG Markdown Editor). You can also query sections of the reference docs, which autogenerates periodically, and it'll tell you if it's grown outdated.
You need to sign in though to make edits (takes less than 1 min) so that a random person can't make edits 😄.
JIC you're wondering, exports to Markdown & GH Wiki will soon be supported as well.

Given the website no longer seems to be working, we've decided not to use this service to host our documentation and instead maintain the github wiki.


BlockId removal in fc-rpc

Substrate refactored away BlockId usage in favor of Hash. This PR aims to align fc-rpc with that strategy whenever possible.

@sorpaas could you take a look this PR?


Improving the contributing.md

I tried to make the process as logical as possible. From easiest to hardest in the workflows.

Docker only tests
Locall dev with python only
IDE use
Compiling custom ops
Work interactively in docker
For googlers: run with bazel

What do you think about adding a ready to use  dev-container image?
See https://github.com/microsoft/vscode-dev-containers/issues/14

What do you think about adding a ready to use dev-container image?

If that can be tested in the CI and that it doesn't require too much maintenance, feel free to open a pull request to add it :)

We need to host two files here. Then the PR there Is trivial
I.e. Check TensorFlow-Examples
https://github.com/microsoft/vscode-dev-containers/tree/master/repository-containers/github.com/aymericdamien/TensorFlow-Examples/.devcontainer

Contributions welcome, you can open an issue if you wish that someone else open the pull request.

@gabrieldemarmiesse I am try to prepare it. Just to go quick forward are tensorflow/tensorflow:2.1.0-custom-op-ubuntu16 and gpu related ready to be a dev environment or it is for running test?
If are just for running test what other packages need to be installed?

You can take a look to the dockerfile we use https://github.com/tensorflow/addons/tree/master/tools/docker
We use python:3.5 to run the cpu tests because the image is much smaller than tensorflow/tensorflow:2.1.0-custom-op-ubuntu16. For the gpu though, you can see in the dockerfile that we need to use tensorflow/tensorflow:2.1.0-custom-op-gpu-ubuntu16

@gabrieldemarmiesse Yes I saw what I meant if you have the source code mounted in these images are you ready to develop inside the container or you need other kind of stuff installed like i.e. flake, clang-format and so on..?

Can we consider pre-commit.Dockerfile as the "real" dev container?

Mhh I also see 'sanity_check.Dockerfile` so I think that currently we don't have a complete developer environment but it is fragmented on stages.

I've added https://github.com/tensorflow/addons/issues/1305 ticket just to understand what Dockerfile receipt is required for a full in container developer experience for this repo.


Update release-a-new-version.md (Release issue template)

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Update the release-a-new-version.md (Release issue template)
Why
The current one is out of date.

Reference Server Preview is available here:https://anchor-ref-pr894.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr894.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr894.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr894.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr894.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr894.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr894.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr894.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr894.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr894.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr894.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr894.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr894.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr894.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr894.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr894.previews.kube001.services.stellar-ops.com/


[6.x] Add dropAllViews functionality to the SQL Server builder

Currently, trying to drop all views on a SQL Server database (e.g. with protected $dropViews = true; in your test case) throws a LogicException because the SQL Server builder does not override dropAllViews().
This PR adds that functionality, overriding dropAllViews() in Illuminate\Database\Schema\SqlServerBuilder.

Can someone else that uses SQL Server confirm this?

@taylorotwell I'm definitely all for having someone else confirm this, but wondering if there's anything I could do to make that easier/faster. A SQL fiddle showing the generated command running against SQL Server for example, or a sample repo with migrations that generate views, and a test that has $dropViews = true, maybe. Or is it just a matter of waiting for someone appropriate to see this at this point?

A SQL fiddle would be nice if that is possible.

Hopefully better late than never, here's a quick fiddle to demonstrate dropping an existing view with the generated command that's now been merged:
http://sqlfiddle.com/#!18/9a983/8
(With the final failure of course being expected, because the view has successfully been dropped.)


Await error when running documentation example

I'm trying to run the following example, shown in documentation:
const jsonArray=await `csv().fromString(myCSV);

I tried it like this:
const myCSV='col0,col1,col2\n1,2,3\n4,5,6';
const jsonArray=await csv().fromString(myCSV);
console.log(jsonArray);

But I'm getting:
SyntaxError: await is only valid in async function

Is this expected in the current version? I tried the solution given in issue https://github.com/Keyang/node-csvtojson/issues/272, but got the same problem.
I'm using version<EMAIL_ADDRESS>node v8.12.0.
Thanks!

As #272 pointed out, await can only be used in an async function. try this:
async function run(){
const myCSV='col0,col1,col2\n1,2,3\n4,5,6';
const jsonArray=await csv().fromString(myCSV);
console.log(jsonArray);
}
run();

Ok, now I got it!
Is there a way to return the json object synchronously along the code? It's an interesting behaviour, when you need to return the json object to an outside function.

Sorry, I think I solved my problem. I can return a Promise with then and wait for it.
Thanks for the attention!

hey @Coqueiro  can you share the how you have return the promise and use the json outside the function

@Coqueiro @Keyang I too would like to get the result back synchronously before proceeding further in my code. Is there a documented way of doing something like the following:
const jsonObj = csvSync().fromFile(csvFilePath)
const foo = jsonObj['some property']
module.exports = foo

Thanks @WORMSS using fromString is wise way if you want sync functionality. I was looking for same.
csv().fromString(fs.readFileSync(csvFilePath, 'utf-8')); // (synchronous)


Broken links on images

Hi @TSallamCode
Issue description
Images not showing up because the links to the images are "broken".
Resolution
Images references are case sensitive, and your code is referencing aboutme.jpg but the actual image in GitHub is aboutme.JPG.
Would recommend keeping all of your images, files, assets, everything as completely lowercase.
Screenshots
Browser inspector editor where image set to matching file name

Browser inspector network tab showing which files are throwing not found errors

turns out I had a '/' before the directory. Thanks for this! Really appreciate it!


Fix Issue https://github.com/LabVIEW-DCAF/Scan-Engine-Module/issues/48

Fix https://github.com/LabVIEW-DCAF/Scan-Engine-Module/issues/48
Edit Line:ignore error from disconnect (that occurs when previous tag
mapping empty)
Dialog:
success on OK and no error
change on success and different value for line
remove initialization call to value signaling for access type

Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
Scan Engine editor node.lvclass--Edit Line.vi.png:
Scan Engine editor node.lvclass--Line Configuration Dialog.vi.png:


overall issues for organisations is limited to num_repos from the user

Gogs version (or commit ref):  <IP_ADDRESS>3
Git version: 2.1.4
Operating system: Linux Debian
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[x] Yes (provide example URL) not possible due to user requirement
[ ] No
[ ] Not relevant

Log gist:

Description
The list of issues for an organization is incomplete when the user itself has less repositories than the organisation. In the query the LIMIT X is used where X is filled as the num_repos from the user table.
To reproduce:

Create organisation
Add 1 repository
Add 1 issue
Make sure the currently logged-in user has no repositories
Go to issues at the top
View the organisation
no issues are displayed (!!)
Create a repo for the user
Go to issues at the top
View the organisation
The issue is displayed (!!)

For now I workaround this issue by setting num_repos in the user table to 999 for each user. But this isn't a good way to solve this.

I am able to reproduce this.

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io.

Thanks @Unknwon it is fixed on https://try.gogs.io. My test user now has all the issues I expect to be there! :D
For our own installation we wait for the tagged releases before upgrade.
Thanks!

Nice work all for catching and fixing!


fix(ci): install rsync for release & docker env

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Introduced in #16435, rsync seems to be only pre-installed in macOS. Install them to fix build failures.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Oops. It appears that @xiangjinwu has drafted a fix: https://github.com/risingwavelabs/risingwave/commit/6bb359de3f05cf0595f6dcdb2305c2deeafe2e1a 😄


Add support for snappy http2 content decompression

Motivation:
Since netty already supports the decompression of snappy's http content encoding, it should also consider supporting snappy's http2 content decompression
Modification:
Add support for snappy http content decompression
Result:
Netty supports snappy http content decompression

@skyguard1 can you add some unit tests ?

@normanmaurer I think io.netty.handler.codec.http2.DataCompressionHttp2Test.snappyEncodingSingleMessage() has tested it. Do we need other test cases to test snappy http2 content decompression?

@skyguard1 thanks

It is my pleasure


Exporter exports different ores

Issue description:
I want to export tin ore and copper ore in the pulveriser but if I turn OFF the damage filter it exports all the ores: lead, gold, etc. If I turn ON the damage filter it exports only a type of tin, since I use IC2, mekanism, thermal foundation, I have "different ores" of the same type (for example: IC2 copper ore, mekanism copper ore, galacticraft copper ore, etc.)
I can't use the same exporter filter for each ore of the same type, it would be a mess to do...
What happens:
I can't export the ores of the same type
What you expected to happen:
Exporting the same type of ore with the same filter.
Steps to reproduce:
1.Take a chest
2.Put an exporter on the chest
3.Trying to export that ores.
...
Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>38
Refined Storage: 1.6.15

Does this issue occur on a server?
no

This is intended. Specify every ore you want to export.


settings: permit temporarily allowing a BLE connection

I find my phone (Android) changes its mac address most times I toggle bluetooth, whereas my desktop, laptop and raspberry pi are fine.
As mentioned on a previous PR, this is a potential issue - I'd like to keep my whitelist but be able to allow one-shot connections (e.g. if I'm doing a GPS update from my phone).
This PR adds a setting menu for bluetooth that temporarily disables the whitelist check on NRF connection, allowing me to connect temporarily, without having to remember to re-enable the whitelist afterwards. Happy to hear other's thoughts on this of course.

Seem to have some trouble testing this - I can't upload the JS to my watch:
Error: Unreachable point. logically broken. in widget_utils

Does it make sense to just add this to the Make Connectable menu option? It feels like that's the idea of that - to make the device connectable no matter what the settings are...
Removing all connect handlers is a bit of a pain as folks might be looking at the Bluetooth symbol in the top right to see if they have a connection?
You could do this I think (remove any handlers that have 'whitelist' in the function code):
var whitelist = ((NRF["#onconnect"] instanceof Array)?NRF["#onconnect"]:[NRF["#onconnect"]]).find(f=>f.toString().includes("whitelist"));
if (whitelist) NRF.removeListener("connect",whitelist);

But it's a bit hacky. Although I guess if it ever breaks it should just not remove the handler rather than causing problems.
I guess the alternative is modifying bootupdate.js to make the whitelist handler check before applying the whitelist.

Yes, the Make Connectable menu makes more sense actually - shall move.
And I see what you mean about the handlers, too - I'll have look at tweaking bootupdate.js and push an update.

I didn't want to add a global variable and picked NRF as the "host", does this seem reasonable?

Yes - looks good to me! Merging...


extra_validation function reference

I'm very new to elixir, phoenix and addict, so maybe it's a dumb question.
I created new app, added 'addict', configured it all properly - everything works well locally.
Now I want to deploy my app, so I'm running mix release. First time I did it, I faced this error:
Cannot add file sys.config to tar file - [{error,{83,erl_parse,["syntax error before: ",["Fun"]]}}]
===> Provider (tar) failed with: {error,
                                         {rlx_prv_archive,
                                          {tar_unknown_generation_error,
                                           "blog_phoenix","0.0.1"}}}

I checked sys.config file and found that it fails on this line of config.exs:
extra_validation: fn ({valid, errors}, user_params) -> {valid, errors} end,

It's being converted to this content in sys.config:
{extra_validation,#Fun<erl_eval.12.50752066>}
Then I found configs.md here and tried to replace this: extra_validation: fn ({valid, errors}, user_params) -> {valid, errors} end with this: extra_validation: BlogPhoenix.User.validation
Of course I added this to my User model:
def validate({valid, errors}, user_params), do: {valid, errors}

Now 'mix release' fails with undefined function BlogPhoenix.User.validate/0 (module BlogPhoenix.User is not available).
Do you have any idea how to fix it and why initial code didn't worked at all?

Hey @AvaelKross! No idea why it didn't work, will have to try to make a release and get back to you, thanks for reporting!
Regarding your new function, thanks for the heads up, that example is wrong. The way to capture a function, instead of invoking it, is via &your_func/n, where n is the number of arguments to the function. In your case, that would be
# config.exs
(...)
    extra_validation: &BlogPhoenix.User.validate/2
(...)

Check the official docs here for more information into this :)

@trenpixster It still does not work during mix release.

As a temporary solution I made a fork where I call configs.user_schema.addict_validate function explicitly. It's more like a workaround, but may be useful for someone else

Any update on this?

I'm a newbie but according to this guy the config has to be an MFA tuple(don't know what that is)
Relevant links:
https://github.com/bitwalker/distillery/issues/185#issuecomment-285924281
https://github.com/mrluc/deferred_config/blob/master/README.md

@rickyhan with latest versions of elixir/phoenix/this module, the following code works to me:
extra_validation: {AppName.ModelName, :addict_validate_method_name}

Hi it still didnot work for me for some reason, I dug into the source code and found the Helper.exec function
So I logged the module name and function name
The module name has Elixir appended in the front: Elixir.ModuleName.ModelName
Is it the expected behavior?

@AvaelKross's solution solved this issue for me.


glTFTranslator.py  not showing up in plugin manager

glTFTranslator.py is  not showing up in plugin manager.
I placed everything in right directory but still this is not working...
Windows10, Maya2017

Hi @jinshim!  Are you maybe using Maya LT?  That version won't work.
If that's not the case, then could you provide the full paths to where you copied the files to so we can verify that is correct? You can omit your username if you like.
Also, have you checked out? https://github.com/WonderMediaProductions/Maya2glTF It has more features and fortunately has a release build that should work for you.

Going close this.  Please let me know if it turns out this is not a Maya LT compatibility issue.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Codecov Report

Merging #572 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #572   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          28     28
=====================================
  Hits           28     28

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 8b1bc00...8e5d162. Read the comment docs.

Coverage remained the same at 97.561% when pulling 8e5d162537ef32171dac16ff59060eeedc610a00 on template-sync-1 into 8b1bc00e314d88474adc449163141ef6856f60cf on master.


Optimized sparse conv from open3d

This repo is using masked dense convolutions because it is optimized in torch. However, would this implementation speed things up or too complicated getting this work here?

Thanks for this! This framework looks easy to use but it doesn't support depthwise or grouped convolution, while many CNNs rely on them (resnext, convnext, mobilenet, etc.). We are still using masked conv for now since we found no framework currently has a sufficiently efficient implementation (i.e., faster than masked conv in torch) of depthwise/grouped convolution on GPUs.

How about torch sparse++?


Feature/multi filter testing

Closes: #135
Closes: #99

Codecov Report

Merging #137 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #137   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          376       398   +22
=========================================
+ Hits           376       398   +22

Impacted Files
Coverage Δ

django_property_filter/filters.py
100.00% <100.00%> (ø)

django_property_filter/filtersets.py
100.00% <100.00%> (ø)

django_property_filter/utils.py
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 c7af324...7aeadda. Read the comment docs.


Can’t build kext

Hello, thanks so much for this project. I’m unable to build the kext from xcode, would you have a ready to use kext please?
The error is:
‘apple80211/high_sierra/IO80211Controller.h’ file not found
In apple80211.h.

You should have selected Catalina target in Xcode, not High Sierra.
Now I've removed the obsolete targets and released the binaries.


WAC and server 2016 core

When I install on server 2016 core in domain server cant connect to herself "Error
The WinRM client sent a request to an HTTP server and got a response saying the requested HTTP URL was not available. This is usually returned by a HTTP server that does not support the WS-Management protocol." when I install WAC to GUI version all is correct

When you install from the command line, you must configure TrustedHosts manually. Sorry this wasn't included, I'll update the documentation.

#please-close


starlark: add StringDict.Keys method

...and remove three ad-hoc implementations.

LGTM.
Two observations from reading surrounding code:
We might want to migrate from bytes.Buffer to strings.Builder in a few places. I can send a PR for that later.
The handling of path in writeValue is subtle. I had to read carefully to see that it didn’t need to be reset on every iteration through the dict keys. I’d suggest adding some docs. Worth mentioning in those docs that the path argument passed in should be short-lived on pain of pinning memory. (Or better, nil out recently appended elements after each recursive call.)

(I can send a PR for the second comment as well. Just let me know for each whether you’ll handle or you’re waiting for me.)

Thanks. I keep thinking of strings.Builder as too new, but now that 1.12 is out, it's fair game. I would be grateful for that CL.

1.12 should come out in Feb. I'll file a reminder issue to do this once it has been released.

I agree the use of append in this way is a little subtle, but what would you add beyond

Given that path doesn't escape, there's not that much value to being able to re-use it across iterations. I'd consider having writeValue lazily initialize path as needed, and making most callers pass in nil. (And maybe add a wrapper that doesn't take a path parameter at all.) Would need to check that perf is ok for that.
If it was a doc-only change, I'd suggest something like:
// writeValue writes x to out.
// path is used to detect cycles.
// It contains the list of *List and *Dict values we're currently printing.
// (These are the only potentially cyclic structures.)
// Callers should generally pass a temporary slice of length zero but non-zero capacity.
// It is safe to re-use the same path slice for multiple calls.

Thanks. I've added the comment, removed the preallocations, and verified that it does not escape.


Editor breaks when parent has transform applied

Because, in all their infinite wisdom, browser vendors made position: fixed relative to the nearest transformed parent, not the viewport.
Only way I can think of fixing this (or at least mitigating its effect) is by making the editor a singleton that's injected into the page on edit, in a similar way to simpla-img v1. Then things would only break if body has a transform applied, which we can mitigate by ensuring editor width/height is never more than 100vw / 100vh.
This sounds like a pain to do properly, so I think it's a job for @bedeoverend to hook up 👍

FYI this effects simpla.io docs...

Shimmed in https://github.com/SimplaElements/simpla-markdown/commit/e42f9385968830c9b438fe96da11d2440f62273a
I think this is a perfectly fine solution, basically just means that if a parent of markdown is transformed, markdown editor can't be bigger than that parent. Which sounds pretty safe to me (ie: markdown editor would at least be size of markdown content).


handleCellSelected() not working sometimes after scroll to segment

Hi JT,
I'm using version 7.1.3. I'm really happy with this library and have a small issue that I hope you can help with.  I created some buttons and functions to scroll ahead or back to the next segment by either year or month.  This feature works perfectly when stepping in one direction for any number of clicks, but if you click the opposite direction, the calendar does scroll to the correct segment, it does select the correct date, but it does not add the selectedView colors.  The weird part is that if you keep stepping in the opposite direction it will eventually start adding the selected view color again.  I'm at a loss and hope you can take a look at the code.  I created a single view project for you to replicate the issue.
https://github.com/REAOrlando/JTAppleCalendarTest

checking this now.

Problem is in your handle code.
You have this:
guard let validCell = view as? CustomCell else { return }

the cell is nil at this point and so it exits your funciton.

root cause lies here
    @IBAction func minusMonthBtn(_ sender: UIButton) {
        let next = userCalendar.date(byAdding: .month, value: -1, to: calendarView.selectedDates[0])!
        calendarView.scrollToDate(next, animateScroll: false)
        calendarView.selectDates([next], triggerSelectionDelegate: true)
    }

    @IBAction func plusMonthBtn(_ sender: UIButton) {
        let next = userCalendar.date(byAdding: .month, value: 1, to: calendarView.selectedDates[0])!
        calendarView.selectDates([next], triggerSelectionDelegate: true)
        calendarView.scrollToDate(next, animateScroll: false)
    }

For example:
calendarView.scrollToDate(next, animateScroll: false)
The calendar may not yet finished scrolling to the date. It may be in the process of getting to the date and there for the date is not yet visible on the screen.
Therefore when you call this -> calendarView.selectDates([next], triggerSelectionDelegate: true)
It is possible that the selection finishes first (while the calendar is still scrolling to the date).
therefore cell will = nil -> therefore, your guard let validCell = view as? CustomCell else { return } will return because the cell is still nil.
You have two options for a fix

change your handle code
use a closure to make sure selection is done after scrolling is complete

calendarView.scrollToDate(date) {
    self.calendarView.selectDate.....   // This happens on completion
}

Perfect! The closure does the trick.  Thank you again.


pythonPackages.cryptography-vectors: normalise pname

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg eval

While at it we could also consider dropping this package from pkgs/top-level/python-packages.nix. IMO it doesn't really belong there and it should be enough to expose it via (e.g.) passthru.vectors.

Good idea!

@ofborg build python27Packages.cryptography python39Packages.cryptography python310Packages.cryptography


[Urgent] Deployment Failures of This Repository Since April.

@kailuowang
It appears that the deployment of this repository has been failing since around April this year.
I've contemplated and applied various strategies to resolve this issue, including updating and downgrading sbt-ci-release, incorporating decodekey.sh, and utilizing setup-java. Regrettably, none of these methods have been successful.
A plausible cause for these errors could be the expiration of the GPG-Key. Your attention to confirm this would be greatly appreciated.
Here are the associated error logs for your reference:
[info] gpg: no default secret key: No secret key
[info] gpg: signing failed: No secret key
[error] java.lang.RuntimeException: Failure running 'gpg --batch --yes --pinentry-mode loopback --passphrase *** --detach-sign --armor --use-agent --output /home/runner/work/play-swagger/play-swagger/core/target/scala-2.12/play-swagger_2.12-1.1.10.pom.asc /home/runner/work/play-swagger/play-swagger/core/target/scala-2.12/play-swagger_2.12-1.1.10.pom'.  Exit code: 2

Hello, @joprice .
I am @Javakky-pxv, the current maintainer of the play-swagger project.
I am considering transitioning this repository from com.iheart. I was wondering if you are still using this plugin?
Also, I am thinking of taking over as the owner in my domain.
Would you be interested in becoming the new owner of the repository, or do you have the motivation to participate in future development?

@kailuowang
The following has been migrated to the below repository.
https://github.com/play-swagger/play-swagger
The groupId for Maven Central has also been added.
addSbtPlugin("io.github.play-swagger" % "sbt-play-swagger" % "1.2.3")

Awesome news! Thanks so much for the hard work.
On Thu, Jul 27, 2023, 9:24 AM Kazuma Iemura @.***>
wrote:

@kailuowang https://github.com/kailuowang
The following has been migrated to the below repository.
https://github.com/play-swagger/play-swagger
The groupId for Maven Central has also been added.
addSbtPlugin("io.github.play-swagger" % "sbt-play-swagger" % "1.2.3")
—
Reply to this email directly, view it on GitHub
https://github.com/iheartradio/play-swagger/issues/597#issuecomment-1653048827,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAUKOJ4RCN3W5HSG7SCCZ3XSIJTNANCNFSM6AAAAAA2PQTIYA
.
You are receiving this because you were mentioned.Message ID:
@.***>


MPI macOS tests time out sometimes

We have observed CI failures of macOS with MPI due to timeout recently - not in every CI run but maybe something like every second run? Some failing jobs are

https://github.com/trixi-framework/Trixi.jl/actions/runs/5644250934/job/15287672140
https://github.com/trixi-framework/Trixi.jl/actions/runs/5664951100/job/15349017631
https://github.com/trixi-framework/Trixi.jl/actions/runs/5655829278/job/15325487560

A working run is

https://github.com/trixi-framework/Trixi.jl/actions/runs/5659469799/job/15333080417

This is quite painful since MPI coverage is only run with macOS at the moment. We need to investigate this and find a workaround.

[ ] Do other OS/systems work again with coverage?
[ ] Would an update to Julia v1.10 help?
[ ] Does it change anything if we restrict max heap usage further?
[ ] Any other ideas?


fix: removing for speedy testing

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


Add MPPT, Original CC/CV, Reduce ADC Jitter and fix LCD freeze on DPS5020

Hi,
Please find a pull request for the following changes:

Apply the LCD freeze workaround to the DPS5020 model.
Reduce jitter by using ADC oversampling

Oops, sorry - the pull request was only supposed to be for the first 2 commits.
The rest are entirely optional, but CC/CV makes the unit behave much more like the original :)

I can confirm its working, flashed on dps5020 v2.6, bugs are i cant switch output off and no documentation about mppt. it always shows on.

Hi @art103,
Does your re-implementation of the cc/cv mode work with control over serial?

Hi @art103,
Does your re-implementation of the cc/cv mode work with control over serial?

It was a long time ago and I haven't tried it, but looking back through the code changes, I think it should...

I can confirm its working, flashed on dps5020 v2.6. nO BUGS yet. I build from your master branch and i wonder how you would use mppt? Any documentation? Its a bad decision to start in mppt mode! Couldnt we set cc/cv mode as default instead? I created pull request to your fork!

Sorry - it's been a while!
I agree that for normal use cases, starting in MPPT is a bad idea. For my use case, the unit would power down when there was no solar input, and I wanted it to come up in MPPT mode.
I ended up modifying one of my units to have HW controlled PWM based on Input Voltage instead of Output Current. This allows much better (and faster) MPPT control targeting a particular solar panel voltage after doing a MPPT sweep.


vs code

when I am running the app then in debug console I am getting following message:
Connecting to VM Service at ws://<IP_ADDRESS>:57714/kOYGq91qB-o=/ws   what does it mean?

It's mean the Dart VM has been started with the flag "--observe" which allows you to access the Dart Observatory which gives additional data about how the Dart VM are running your code. You can e.g. see memory consumption, how much time Dart are using in each method, see the assembly code which your program has been compiled into and much much more.


Removed django warning: RemovedInDjango19Warning

site-packages/userena/utils.py:133: RemovedInDjango19Warning: django.db.models.get_model is deprecated.   profile_mod = get_model(*settings.AUTH_PROFILE_MODULE.rsplit('.', 1))

@nooperpudd this issue is fix #485, but you must wait to the new release

Fixed and released as part of #505


how to add tis module in angular 2

This is great module of image placeholder. but i want to add this in angular 2 project so please guide me how to add this in angular 2 project.

Hi, apologies for the late reply. Please take a look at the official guide - https://angular.io/guide/upgrade, you will be able to find migration details there. I really don't have time to work on an Angular 2 compatible version for this module. Maybe you can send in a pull request for that ? Please feel free!


Remove $  from README's commands

Hi,
I just cloned the repository by following the instructions. I am a bit annoyed by the $  character you added at the beginning of each code block. I suggest removing the $  prefix on all lines. This would make it easy to copy-paste, especially for multi-line commands.
EDIT: if the $  is here to express it must be run from a command line, I would suggest writing a small phrase at the beginning, like: "Open your terminal and run the following commands:"
And thanks for the repository, I need to learn about flask, this is a good start to look at your examples.

I'm okay to remove them, feel free to make a PR.


Use devfile structs from devfile/api

Is your task related to a problem? Please describe.
There are already struct defined that are representing devfile in https://github.com/devfile/api
and is currently the main source of truth for devfile schema.
Describe the solution you'd like
Library should be using the structs defined in kubernetes-api to represent the devfile instead of having defined new structs.
Describe alternatives you've considered
---NA--
Additional context
This whole repo defines DevWorkspace CRD, and devfile is essentially a subset of DevWorkspace.

I'm closing this issue as it's already done.


Error: Cannot read property 'length' of undefined

In my use case of ngTagsInput, If the bound ngModel is undefined is get the following error;
"Cannot read property 'length' of undefined"
at the following;
ngModelCtrl.$setValidity('maxTags', scope.tags.length <= options.maxTags)
maybe this should be changed to something like;
ngModelCtrl.$setValidity('maxTags', scope.tags.length || 0 <= options.maxTags)
and also the same for options.minTags validity.

Currently the tagsInput directive automatically initializes your model as an empty array if it's undefined, so that error shouldn't occur.
Could you provide a plunker showing your problem? You can use this template.

Hi Michael,
I thought I'd recreated our use case in the following plunker.However this doesn't recreate the issue in our solution.
I'm happy for you to close this off and If I can repro I'll post a plunker. Apologies for any inconvenience.


HDDS-9523. Ozone manager doesn't honor property 'hadoop.security.token.service.use_ip'

Hadoop use property "hadoop.security.token.service.use_ip" to control when the IP or the host name will be used as  service name in delegation token.
If hadoop.security.token.service.use_ip is true(default).  then IP address will be used to construct the Token service field, otherwise, then use host name.
Currently in Ozone, service name in delegation token always use host name.
What changes were proposed in this pull request?
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9523
How was this patch tested?
Tested by the enhanced existing unit test.

Could you please document that Ozone uses this Hadoop property? It would be very helpful in standalone Ozone installations.

Currently none of delegation used properties are documented. I created a track JIRA https://issues.apache.org/jira/browse/HDDS-9586 for this.

@jojochuang , could you help to take another look?

Thanks @ChenSammi for the patch, @jojochuang, @myskov for the review.

@adoroszlai ,thank you for review and merge the patch.


generalization of CSV dataset + shuffle feature

Rebased + added some features: https://github.com/lisa-lab/pylearn2/pull/1072

@denadai2 If you are still interested in getting these changes merged, can you rebase on top of the current master? I'll have a new look after that.

ahahan not bad after 4 months
Now I'm rather busy, I'll rebase ASAP

I was about to submit a pull request allowing for multiple targets. What's the status of this?

@evanhaldane @lamblin done :)

Thanks for the rebase!

thanks folks! :+1:

Ok I finished it. @lamblin @evanhaldane

Great, thanks!

thanks


Missing DPI infos in JXL format

Most formats have image DPI infos in the file header, no such thing in JXL?
Converting PNG (with DPI infos in the file header) to JXL and back (using cjxl.exe and djxl.exe) => DPI infos are lost.

Do you mean the pHYs physical image dimensions that PNG files can have?
We don't have such a thing in the JPEG XL codestream headers; we did not consider it render-impacting metadata since in practice, most viewers would not try to render an image at some specific physical dimension, but rather they would typically show it at dimensions dictated by CSS (in case of browsers), whatever the pixel density of the screen is (in case of most viewers if the image is smaller than the screen), or just scaled to fit the screen (in case the image is larger than the screen). The DPI info is maybe somewhat relevant for printing use cases but in most cases also there it has become a rather meaningless piece of info.
Anyway, if you want to keep it, you can do it by using Exif or XMP metadata, which does have that field. I'm sure there must be some way to use exiftool or exiv2 to get the DPI from the png pHYs chunk and add XMP data with the same information (or update the XMP to add that information, if there is already XMP data in the png).
I suppose in cjxl, when given png input, we could look for the pHYs png chunk and translate it to a corresponding Exif or XMP blob (or add it to a blob if there is already one). That's quite a messy thing to do though (it involves adding Exif or XMP manipulation functionality to cjxl, which currently we've managed to mostly avoid with the only exception being Exif orientation).

Yes, the easiest way is probably to use Exif metadata tags of a JXL file:
0x11A = XResolution
0x11B = YResolution

Btw, there is a minimal set of required tags to comply w/ Exif spec, you can't just add these two. For compressed images, these should also be present:
ResolutionUnit
YCbCrPositioning (I guess this was for JPEG only?)
ExifVersion
ComponentsConfiguration (also JPEG only specific?)
ColorSpace
PixelXDimension
PixelYDimension

See also #3533

Isn't this also required to consider JXL as a fitting (as in full featured) replacement for classic JPEG? Especially considering the growing number of input file formats I would suspect that users expect at least some support for such information for printing workflows.
See for example #3193, some might have the expectation that lossless encoding also includes metadata ;)


Removed unnecessary php version checks after php dependency bump to 7.0

Fixed todo from #197

Check all PHP_VERSION and version_compare checks for relevance

Great thanks


commenting out LA features for Pascal

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

Before: https://main--prisma-cloud-docs--hlxsites.hlx.page/
After: https://--prisma-cloud-docs--hlxsites.hlx.page/
Worker: https://prisma-cloud-docs-production.adobeaem.workers.dev/?branch=

done by Anagha through another PR


Cookie policy

Allow setting up cookie policy on websites

📝 Changed routes:

/policies/cookies
/settings

(view full size)
(view full size)

And 2 other routes:

/api/database/cookie-policy/settings
/api/database/website/[websiteId]

Commit 885e1174c3c8b9b58efde8d355aba10724a3bb0e (https://green-analytics-9rovwf8f8-unknown-studios.vercel.app).


[resource] Bedrock agent resources

Some Bedrock resources are in a dedicated API called "Bedrock Agent", this is currently not covered by the deletion of Bedrock resources I am working on
Documentation: https://docs.aws.amazon.com/bedrock/latest/userguide/agents-how.html
Update: PR created 😄

:tada: This issue has been resolved in version 3.32.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Make voting page accessible

Currently the voting page (https://webpack.js.org/vote/) is unusable with out a mouse. This makes is unusable for some people.
The arrows to vote for the items should be usable with only a keyboard.

@montogeek have you started working on this? I'd like to pick this up if you haven't started.

@anshumanv Feel free to do it :)

Sorry for the delay, PR up at - https://github.com/webpack-contrib/voting-app/pull/19
Have just included the vote keys in tab order.

This has been fixed and can be closed now, @EugeneHlushko @montogeek


Remaining water/brine in solar evaporation plant

Using Mekanism 8.1.7, the solar evaporation plant internal tanks do not completely empty out after the water/brine supply is turned off or exhausted. Most often, there is some tiny amount of fluid left in the tanks that is too small to convert to brine/lithium, so it just remains there. There is no option to dump it or remove it either.
I got out of this problem of water in the tank once by having exactly 0mB left and then it was really empty. However, now I encounter the same issue with the brine. There is 2mB of brine left. I can't drain it out, and I can't make more brine because water cannot get into the tank with the remaining brine there.

If I recall correctly, a Gauge Dropper lets you remove small amounts of liquids and gases from the machine GUI.  pick up and click the dropper on the tank you want to empty, then hold it and (shift?) right click it on the ground [voids fluid] or another tank to empty it.

Right, that's part of the reason why I developed the gauge dropper.

Thanks that worked great. Thanks!


aliyun cs DescribeClusterV2UserKubeconfig not working in 3.0.70

In version 3.0.70 the command aliyun cs DescribeClusterV2UserKubeconfig --ClusterId <id_of_existing_cluster> fails with the following error:
ERROR: product 'cs' need restful call

Use `aliyun cs {GET|PUT|POST|DELETE} <path> ...`

This call was working at least in 3.0.68 and I have seen no documentation that it will be deprecated.
Also such a deprecation should not happen in a patch version as to semantic versioning https://semver.org/

was fixed in a later version.


Migration banner

Closes #1325
Closes #1326

Coverage increased (+0.5%) to 38.348% when pulling f46b6d0922790ac82469a473e593e462a468ff14 on na3d:migration-banner into 5a7767b82f72e68dedca020a30e70b96d0d4668b on linode:develop.

This PR should be against develop-transitional

Coverage increased (+12.02%) to 50.12% when pulling f96960b2c22dd5c9661c9e7c7468cd97fd382752 on na3d:migration-banner into 4883bfe9865e4c817a0d3a380b2dedee9b500062 on linode:develop.


Sprites turning into letters

Describe the bug
The bug is that, when testing the game with the run button, sometimes all of the sprites on a scene will turn into letters, and their sprites will not render properly until the program restarts.
To Reproduce
I am not entirely certain of what causes this, but I have GB Studio running in the background when I'm working on sprites and backgrounds in other programs, and I can leave GB Studio idle for long periods of time. It's possible that this happens when GB Studio is left in the background for too long.
Expected behavior
I would naturally expect that the sprites would appear as intended.
Screenshots

(What the game appears as when the bug occurs)

(What the scene looks like)
Platform (please complete the following information):

OS: [e.g. macOS 10.14.1]
App version [e.g. 1.1.0]

Additional context
I am not breaching any limits within the scene for maximum assets, as seen in screenshot 2. Thanks!

Just to make sure, could you double check if that the App version correct? latest GB Studio version is 1.2.1.


Not a table warnings for block scalars

Output of helm version:
Client: &version.Version{SemVer:"v2.8.2", GitCommit:"a80231648a1473929271764b920a8e346f6de844", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.8.2", GitCommit:"a80231648a1473929271764b920a8e346f6de844", GitTreeState:"clean"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.2", GitCommit:"5fa2db2bd46ac79e5e00a4e6ed24191080aa463b", GitTreeState:"clean", BuildDate:"2018-01-18T10:09:24Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"8+", GitVersion:"v1.8.11+coreos.0", GitCommit:"01cc6989ee0202f3d4dc5f9df61ab80c0143315d", GitTreeState:"clean", BuildDate:"2018-04-06T16:38:17Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
AWS / kube-aws
Deploying stable/mysql I get these warnings:
2018/04/22 13:46:57 warning: destination for configurationFiles is a table. Ignoring non-table value <nil>
2018/04/22 13:46:57 warning: skipped value for configurationFiles: Not a table.
2018/04/22 13:46:57 warning: skipped value for configurationFiles: Not a table.

This is from the values file:
mysql:
  configurationFiles:
    mysql.cnf: |-
      [mysqld]

      # Increase max query size
      max_allowed_packet = 32M

      # Flush transactions log buffer to disk once per second, rather than immediately
      innodb_flush_log_at_trx_commit = 0

      # Store tables in main tablespace rather than separate files (MySQL <=5.5 behaviour)
      innodb_file_per_table = 0

Hi @whereisaaron
Can you determine which templates and values are giving the error?

And are you using --set from the Helm CLI?

Hi @jascott1 it occurs when providing a values file to the stable/mysql chart. But this is just an example, I've seen it often with my own charts, particularly when block-scalar values are used.
I suspect it occurs when a user supplied values file overwrites/updates the default values.yaml in the chart. And the user supplied a values file includes a block-scalar values for an empty map in the default values.yaml.
Everything works as expected. These warnings seem erroneous. But they are annoying me 😃
Hi @technosophos I do have some unrelated --set arguments, but the 'configurationFiles` map that triggers the error is provided purely from a values file:
mysql:
  configurationFiles:
    mysql.cnf: |-
      [mysqld]

      # Increase max query size
      max_allowed_packet = 32M

      # Flush transactions log buffer to disk once per second, rather than immediately
      innodb_flush_log_at_trx_commit = 0

      # Store tables in main tablespace rather than separate files (MySQL <=5.5 behaviour)
      innodb_file_per_table = 0

I think it should be fixed by this https://github.com/kubernetes/helm/pull/3323

Ok, great, well I can revisit this when that change is released.

default value must be empty object {}


No version flag

$ silverback --version
Usage: silverback [OPTIONS] COMMAND [ARGS]...
Try 'silverback --help' for help.

Error: No such option: --version

Taking this, the conversation on implementation details is over at the ApeWorkX Discord.

@CreedsCode this is the implementation we use in ape
https://github.com/ApeWorX/ape/blob/d43ea7785eaa3547d17b391d7f973d558f4122a0/src/ape/_cli.py#L178


Pick Block from FZ not compatible with AE2

Hello,
I just wanted to mention that the pick block key command that you add to MC isn't compatible with AE2. Ordinarily, I would simply use the regular MC Pick Block key, but yours actually allows me to pick ME cables from AE2 while they are placed in the world, and that is fantastic! Where the incompatibility comes into play is when you try to force AE2 to craft items that are craft-able and already stocked.
For example, in AE2, if I want to craft 10 of an item that I currently have 5 of, I simply middle-click (pick block) on the item and tell it how many I want. This doesn't work with the FZ Pick Block. Is that something you can fix or implement?

Unfortunately it's simply not possible to improve middle click in a sane way that is also compatible. I suggest binding FZ's to an alternative key; I used alt.

Okay that works. Just know that I posted another issue where if FZ Pick Block is set to NONE and you close and open MC again, it crashes. This on can be closed.


An error occurs when running the code

hello
I installed the yaml environment as the readme and tried to run robustness_test.py. on decoding it comes up with not being able to find . /assembly/temp_batch _output.fasta file error. So I'm looking for help from the author.

Hi, thanks for your attention to our work.
I notice that the MUSCLE version you installed is v3.8.1551. Could you reinstall MUSCLE v5.* and rerun the process?
Here is the link MUSCLE. Version 5.1 is recommended since other version is not be tested.
Thank you for reminding me to specify MUSCLE version number and it will be updated to README.


e2e: wait for seccomp profiles to be available

What type of PR is this?
/kind failing-test
What this PR does / why we need it:
We now wait up to 30 seconds to seccomp profiles to be available after
the workload has been removed.
Which issue(s) this PR fixes:

None
Does this PR have test?
Yes

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/lgtm


add more test

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@df61ab5). Click here to learn what that means.
The diff coverage is 66.66%.

@@           Coverage Diff            @@
##             master    #406   +/-   ##
========================================
  Coverage          ?   55.3%
========================================
  Files             ?     127
  Lines             ?    5817
  Branches          ?     867
========================================
  Hits              ?    3217
  Misses            ?    2303
  Partials          ?     297

Impacted Files
Coverage Δ

...rc/main/java/com/pingcap/tikv/codec/MyDecimal.java
68.02% <66.66%> (ø)

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 df61ab5...e36ad97. Read the comment docs.


Analyze bams under subdirectly

Hi Daniela,
Thank you for sharing MitoHPC for us. I believe this tool should move my project forward.
One thing that I wanted to ask you today is that how can I run with the input bam files that are located in the subdirectly?
There is no problem at all running with your test bam files and my bam files of interest if I put them under one directly.
However, as you may know, the file size of each bam is too big to copy or move, therefore, I tried to read each bam from the original subdirectly but did not work well so far.
I think the code I should modify is from line 119 to line 129 of init.sh, typically line 127.
Below is the original init.sh from 119 to 129 for your information.
PWD=pwd -P`
export HP_FDIR=$PWD/fastq/      # fastq input file directory ; .fq or .fq.gz file extension
export HP_ADIR=$PWD/bams/	# bams or crams input file directory
export HP_ODIR=$PWD/out/        # output dir
export HP_IN=$PWD/in.txt        # input file to be generated
if [ -d $HP_ADIR ] ; then
if [ ! -s $HP_IN ] ; then
find $HP_ADIR/  -name ".bam" -o -name ".cram" -readable | ls2in.pl -out $HP_ODIR | sort -V > $HP_IN
fi
fi`
I have modified above code by using -type, -exec or even . for my purpose but no luck (I am not so good at shell script).
Will you please give me suggestions or feedback regarding the issue I have encountered, please?
Best regards,
Ken
P.S.
Because the original script have `` in there (PWD = pwd -P), I could not insert the code here properly.
I am sorry for any inconvenience caused.

Hi, although it was not a perfect way, I figured out by myself. Thank you!

Hi Ken, I am glad it worked!  Other options would be to create symlinks to the input files and have them all under $HP_ADIR , or generate a custom $HP_IN input file.

Thank you Daniela, I do keep in mind and will give it a try later!
K


Improve scene playing and reloading

This PR improves the scene reloading behavior in the editor. When you play a scene, the play button you pressed turns into reload arrow. In case of play custom, it reloaded again the scene you picked, but for play current it always played the currently opened the scene, despite the button also changes. I also added a tooltip to better indicate this behavior and refactored code a bit to decrease duplication.

Fixes #31515

If you rebase, this can be reviewed. We've collectively agreed this is a useful workflow change.

Thanks!


Remove TrainedModel CR reconciliation

With the TrainedModel CRD being deprecated in favor of using the InferenceService CRD for both single model and multi model deployments, we can go ahead and remove support for TrainedModel reconciliation. Our current support is still using the old serving.kubeflow.org/v1alpha1  APIGroupVersion anyway.

/assign
I can help on this issue.


Make --quiet a level of verbosity

Currently, a Config has a quiet field and a verbosity field. This is somewhat silly, they should be a single verbosity field, with quiet being represented as Verbosity::Quiet.

If the quiet flag is specified and also there's > 0 verbose flags, what should be the user experience? Would the verbose flags, if at all present, overwrite the quiet flag?

I don't have a strong justification for it, but I think a quiet flag would override verbosity flags.


[cPP ENHANCEMENT] Required detail level for description of entropy collection

Provide the location of the issue
D.1, Design Description
What is the enhancement request for the cPP? Please describe.
NDcPPv2.2e Annex D.1 includes the following requirement: “The documentation [EAR] will describe the operation of the entropy source to include how entropy is produced, and how unprocessed (raw) data can be obtained from within the entropy source for testing purposes.” Please clarify the level of technical details documenting data collection methodology.
Note: See RFI 202119. https://ccusersforum.onlyoffice.com/Products/Files/DocEditor.aspx?fileid=7590282&action=view
Describe the solution you'd like
Suggested change:
The documentation must describe how unprocessed (raw) data was obtained for the analysis. This description shall be sufficiently detailed to explain at what point in the entropy source model the data was collected and what effects, if any, the process of data collection had on the overall entropy generation rate.
Additional context
This command and solution were submitted by the Canadian Centre for Cyber Security. [Reference CCCS-PP-45]

This is existing NDcPPv2.2e RFI that was mistakenly not propagated to NDcPPv3 due to timelines of its release and draft release.


Need to improve the function of the Tab key

What's the Problem?
In the case of Files, after I rename a file, when I press the tab key to rename the next one, the name edit area closes automatically, as if I just hit Enter. In Windows Explorer, pressing Tab (while renaming files) automatically opens the area to rename the next file, which makes the workflow very agile.
Solution/Idea
The app would be perfect if the tab key function works when navigating between folders and files. As I work with many files daily, this is an indispensable function for me. When (and if) they implement this function, I will revert to using Files. Until then, I will remain with the native explorer.
Alternatives
Implementing bulk file renaming capabilities, but I believe just applying the Tab key function between filename text boxes should do the trick.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
Versão: <IP_ADDRESS>
Windows Version
OS Version: 10.0.22000.527
Comments
No response

Merging with #6655

Lol -> #6675 thanks

Lol, Now I agree! Thank you @gave92 😃


Cycle Action: Automation- Dependant Zoom

This is a copy from my post on the Reaper-Forum (I hope that's ok) because I figured it would fit better here.
Hi everybody,
So I recently got into cycle actions.
But I quickly found a roadblock.
I wanted to make a cycle action that depending on if your active Automation lanes were visible.
It would either zoom to the selected Item or the selected item and its visible Automation - Lanes.
I tried this by using a cycle action and some SWS Actions like this (Picture).

And well it works fine except for one detail.
It totally screws up where it zooms in.
It mostly will show me the first Track, while my selected item is in the fourth or so.
Thank you very much in advance!

Ok I figured it out,
found the "Prevent UI refresh"- Tick-box in the Cycle Actions Editor.
Yeah that fixed it.
So If someone has a similar issue with UI refreshing after cycle actions, this is it.
If someone knows how I can make this post better to find for people with similar issues I'd be glad to know, cause I'm kinda a new user.
Cheers!