Renamed "host" field to "hostname"

ceph orch host ls --format=json has renamed "host" field to "hostname", causing the following error in ceph-salt:
admin:~ # ceph-salt deploy node3.octopus.com
Checking if ceph-salt formula is available...
Syncing minions with the master...
Checking if there is an existing deployment...
Traceback (most recent call last):
  File "/usr/bin/ceph-salt", line 11, in <module>
    load_entry_point('ceph-salt==15.1.0+1583164799.g42f8dbf', 'console_scripts', 'ceph-salt')()
  File "/usr/lib/python3.6/site-packages/ceph_salt/__init__.py", line 55, in ceph_salt_main
    cli(prog_name='ceph-salt')
  File "/usr/lib/python3.6/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3.6/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.6/site-packages/ceph_salt/__init__.py", line 98, in deploy
    retcode = executor.run()
  File "/usr/lib/python3.6/site-packages/ceph_salt/deploy.py", line 1272, in run
    retcode = self.check_deploy_prerequesites(self.minion_id)
  File "/usr/lib/python3.6/site-packages/ceph_salt/deploy.py", line 1264, in check_deploy_prerequesites
    if minion_short_name == host['host']:
KeyError: 'host'

Signed-off-by: Ricardo Marques<EMAIL_ADDRESS>

@ricardoasmarques As of which version has this change taken place? It must be fairly new:
admin:~ # ceph orch host ls --format=json

[{"addr": "node3", "hostname": "node3", "labels": [], "status": ""}, {"addr": "node2", "hostname": "node2", "labels": [], "status": ""}, {"addr": "node1", "hostname": "node1", "labels": [], "status": ""}, {"addr": "admin", "hostname": "admin", "labels": [], "status": ""}]
admin:~ # ceph versions
{
    "mon": {
        "ceph version 15.1.0-1521-gcdf35413a0 (cdf35413a036bd1aa59a8c718bb177839c45cab1) octopus (rc)": 3
    },
    "mgr": {
        "ceph version 15.1.0-1521-gcdf35413a0 (cdf35413a036bd1aa59a8c718bb177839c45cab1) octopus (rc)": 1
    },
    "osd": {
        "ceph version 15.1.0-1521-gcdf35413a0 (cdf35413a036bd1aa59a8c718bb177839c45cab1) octopus (rc)": 6
    },
    "mds": {},
    "overall": {
        "ceph version 15.1.0-1521-gcdf35413a0 (cdf35413a036bd1aa59a8c718bb177839c45cab1) octopus (rc)": 10
    }
}

@ricardoasmarques As of which version has this change taken place? It must be fairly new:
...

@smithfarm Probably since https://github.com/ceph/ceph/pull/33474

Got it - it's the JSON field in the output that was renamed, not the ceph orch host ls command itself:
admin:~ # ceph orch host ls --format=json

[{"addr": "node3", "hostname": "node3", "labels": [], "status" . . .

Got it - it's the JSON field in the output that was renamed, not the ceph orch host ls command itself:

Yes

Now
admin:~ # ceph orch host ls --format=json

[{"addr": "node3", "hostname": "node3", "labels": [], "status" . . .

Before
admin:~ # ceph orch host ls --format=json

[{"addr": "node3", "host": "node3", "labels": [], "status" . . .


Create Small Business Support data visualizations

Create the data visualizations which will be used by the Small Business Support dataset.
The layout of Atlas v2's detail pages will be dynamic rather than hardcoded. That means two separate API responses define what is rendered on the detail page, the raw data of the entry and the layout description which describes how to present that data to the user.
There is an example of each in the frontend/mocked_api_responses directory: data for Adelaide and a layout description for Small Business Support. There will eventually be an API endpoint on the backend which returns JSON responses like those. These examples represent what would be rendered when a user visits /explore/small-business-support/401011001.
There's also some documentation about the format of the layout description file, for reference.
There's a Figma mockup of what detail_layout_example_small_business_support.json should look like as well. The goal of this issue isn't to completely finish the detail page, just to replicate what's at that Figma link using the provided data.
Feel free to move and/or rename the mocked_api_responses folder to wherever you think makes most sense.

Resolved.


Change python code to gpio zero?

gpiozero might provide cleaner code in the long run.  Note the nice pin numbering explaination.
gpiozero pinout

I was able to start the mcu poweroff timer but I wan unable to read the button presses. Here's my current code:

Code
#!/usr/bin/env python
#
# pd-powerkey.py - monitor GPIO to detect power key press from Power MCU (PCU)
#

import time,os,sys
sys.path.append('/storage/.kodi/addons/virtual.rpi-tools/lib') # Import gpiozero in LibreELEC
from gpiozero import Button,LED

print("pidesktop: power button service initializing")

pwrBtn = Button(13) # PCU to Pi - detect power key pressed

pcu = LED(6) # Pi to PCU - start/stop shutdown timer
pcu.off() # tell PCU we are alive
pcu.on() # cause blink by starting shutdown timer
time.sleep(0.5)
pcu.off() # clear timer we really are alive

# callback function
def powerkey_pressed(channels):
    print("pidesktop: power button press detected, initiating shutdown")
    os.system("sync")
    os.system("shutdown -h now")
    sys.exit()

# wait for power key press
pwrBtn.when_pressed = powerkey_pressed
print("pidesktop: power button monitor enabled")

# idle - TODO: use wait
while True:
    time.sleep(10)

The button reports high both pressed and released so I don't know where to go from here.

I made it work. I'll publish it soon.


The search bar in speakers' profiles seems to be useless

When someone taps on that search bar and write any word or letter, nothing happens. I don't know if it should do a specific action or it is useless. If it should do an action, it has to be fixed, and if it is useless it should be removed in order to have a better UI

Searching inside the speaker detail page is not required, so you can go
ahead and remove this if you want. :)
On Wednesday, December 28, 2016, Alejandro Heredia<EMAIL_ADDRESS>wrote:

When someone taps on that search bar and write any word or letter, nothing
happens. I don't know if it should do a specific action or it is useless.
If it should do an action, it has to be fixed, and if it is useless it
should be removed in order to have a better UI
[image: screenshot_20161227-151128]
https://cloud.githubusercontent.com/assets/15079370/21523860/c06fc5c0-ccdf-11e6-83f2-5616f66e546c.png
[image: screenshot_20161227-151138]
https://cloud.githubusercontent.com/assets/15079370/21523861/c070c0f6-ccdf-11e6-9363-e3df3516b2e6.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/911, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AGQtlsW0B0_bLZpuLsLeQ_GrZ3I-BZvEks5rMnHEgaJpZM4LW8_g
.

--
Best,
Harshit Dwivedi

@Heredroky are you working on this issue?

@the-dagger shouldn't we keep the search button in case where we have many sessions by the same speaker

I don't think that the session for a single speaker will ever exceed 5, so
it makes no sense for having a search button for the speaker detail page.
On Friday, December 30, 2016, utkarshnath<EMAIL_ADDRESS>wrote:

@the-dagger https://github.com/the-dagger shouldn't we keep the search
button in case where we have many sessions by the same speaker
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/911#issuecomment-269734526,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGQtluoeCvDv0EdYfFrwLWHLr4glKcJ9ks5rNJjJgaJpZM4LW8_g
.

--
Best,
Harshit Dwivedi


Update rollup to 4.3 and use new --forceExit flag

My feature got merged and released so the patch is no longer needed !

Woohoo, congrats Raph!


Create e2e tests

Goal: Have e2e tests to ensure that the oria-operator is working as expected when installed on a cluster and ScopeTemplates and ScopeInstances are created.
RukPak for reference:

e2e tests: https://github.com/operator-framework/rukpak/tree/main/test/e2e
Makefile: https://github.com/operator-framework/rukpak/blob/main/Makefile

Acceptance Criteria:

e2e tests that ensure the oria-operator:

Installs correctly onto a cluster
Reconciles ScopeTemplate and ScopeInstance CRs appropriately
Uninstalls correctly from a cluster

Makefile target to:

Create an ephemeral KinD cluster
Run the e2e tests
Delete the KinD cluster

/assign
/unassign @laxmikantbpandhare


ENH: Allow to fill orders at limit price or configurable price field

Generalize SlippageModel by adding properties market_field and
limits_enhance and subclasses that extend it and configure the
properties.
Thus the slippage model can be configured whether to execute at open
or close price and whether to trigger limit and stop orders early at
limit, stop price (if they fall within [low..high] range) or at the
configured market_price.
Default settings are backward compatible with respect to execution
price (at least in the tests). To get the new behavior set the
slippage model to a subclass.
E.g.:
def initialize(context):
context.set_slippage(ConservativeCloseSlippage())
Once configured, the slippage model either always executes limit
orders at limit price or always at market_price. It is not designed
to use limit price and market_price for different orders in the same
simulation.
A related PR #1376 contains some discussion about limit prices.

Coverage decreased (-0.003%) to 86.127% when pulling 9487a9cfd7ebe17979bc8a6653048724d31f559c on volconst:limit-orders into 7fe1a56730bb36c3c12eeeffe5c69285f927d60a on quantopian:master.

I am closing this PR, because ConservativeCloseSlippage has a bug with stop orders and the open price slippage models does not need enhance_limits, but should switch dynamically depending on marketability. I may create another PR if I implement new ideas.


tps is very low, while trying to benchmark the performance

We've tried to store some table into EOS via smart contract, and using eosjs as API interface. all thing work. but when we do the performance test, the eosjs consume lots of cpu, and prompt following message:

Seems like it took time to sign the transactions, if lots of requests flow into eosjs. Could anyone can help? Thanks.

Sure, I have an idea .. Changing the signature algorithm still needs a little more research.
ref: https://github.com/EOSIO/eosjs-ecc/issues/20#issuecomment-397506197

@liliyongleo what exactly are you trying to performance test?

@liliyongleo I would recommend creating a bunch of transactions using broadcast false then submitting them all at once.  You'll find the thruput without signing as part of the test.
If you're just wondering about how many actions you can handle then you could put a bunch of actions into a single transaction.  You'll only sign it once so that won't be a factor BUT the actions will execute in sequence in the contract so you can see how it does.

Thanks, chris. We've tried put a bunch of actions into a single transaction. It helps. But the problem is: it requires same private key for all these actions. In our senario, the requests can be from different account.

@chris-allnutt I'm wondering if Aarin can tell us why nodeos differs from the rfc6979 signing algorithm only by starting its nonce starts with 1 (instead of zero).  That actually skips a step in the signature discovery process.  So it would be good to know why nodeoes starts its nonce with 1.
Assuming this comment is correct, this finds that nodeos starts the nonce ("k") with 0: https://github.com/EOSIO/eosjs-ecc/issues/20#issuecomment-397506197
This will give me some safe options to use so I can performance test..


TST: add native POWER8 to CI

add native POWER8 testing to
Travis CI matrix with ppc64le
os entry

The new job ran successfully in about 7 minutes on my fork--let me know if more POWER-specific build flags might be desired? Also, probably a good idea to check the log for the new entry.
I was hesitating to add more stuff to the already-busy Travis matrix, but I see @xianyi is now working to get Azure pipelines up and running, so that should provide a medium for shifting burden off Travis if needed.

@xianyi Yes! Azure pipelines are visible /  running in this PR now!


Great module!

Thanks man for making this ps module available. Saved a ton of time to the point that it has made it viable to automate okta for my employer. Well done!

Hey, thanks! Glad it worked for you.


Wavelet type

What type of wavelets should I use?  '--wavelet_type', default='db2', help='haar, db2, bior1.5, bior2.4, coif1, dmey'

If I'm using an X-ray picture, what type of wave transform do you recommend? Hope to get your answer, thank you! @Yanfeng-Zhou

If I'm using an X-ray picture, what type of wave transform do you recommend? Hope to get your answer, thank you! @Yanfeng-Zhou

The differences between different wavelet bases are not very obvious. If you are not sure, you can use haar or db2

Ok thank you very much, you're using db2 in xnet, right?

Ok thank you very much, you're using db2 in xnet, right?

I compare the performance of various wavelet bases in the ablation studies of the paper. From the experimental results, db2 is better, so I used db2.

Thank you for your answer, I'll try db2 and haar.


Interpolate function typing  😅

After my last PR a bug was introduced where the responsive helpers minMedia and maxMedia are not typed properly.
Currently they are typed as receiving only an array of template strings, hence doing this is fine:
maxMedia`font-size: 14px;`

as it translates roughly to:
maxMedia(['font-size: 14px;'])

however if we pass a dynamic value:
maxMedia`font-size: ${mySize};`

it translates to:
maxMedia(['font-size: ', ';'], mySize)

and hence  Typescript will complain that maxMedia expects one argument but two were supplied.
( you can learn how tagged templates work here)
I decided to type the rest arguments as any for simplicity. We also need to reference the arguments to prevent linting complains, however styled-components will grab the values magically for you ✨
Suggestions for better typing very welcome 😛

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Helper routine to space-pad columns for outputting

There are a few places in the codebase that could use a shared routine for space-padding columns, Ex: code for env list.

For those tuning in at home, basically, there is a set of routines where we pad stuff into columns. For example, env list does something like:
NAME        NAMESPACE URI
us-east/dev default   ...

The basic idea is to pull this out into its own routine and use that any place we have a padded column list.

Is that using text/tabwriter?

ksonnet/ksonnet-specific feature. No longer relevant.


feat: Upgrade to v2 API

Description
Upgrades env vars to v2 API

⚠ Note! We'll likely want to hold off on merging this until the actual cutover for the API v1 --> v2 happens!
What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Related to: https://github.com/open-sauced/ai/issues/302
Related to: https://github.com/open-sauced/api/issues/478
Mobile & Desktop Screenshots/Recordings
Steps to QA
Went through the code and there's nothing that was touched with the API generation endpoints - could use help here to further validate if needed.
Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

cc @bdougie - this has merged since we cut the beta API to v2. Does this require additional steps to release to the chrome / firefox plugin stores?


WebGL build not working by default

Hello there,
While exploring the glTFast and glTFDemo, I want to develop WebGL build for the same. I am able to build the WebGL output, but for some reason, the Main scene doesn't load. The chrome/firefox console doesn't even spew any errors.
While building for WebGL, I do see a warning that "Script attached to SceneSelection is missing".
Are there any build instructions or documents or wiki? Please do the needful.
FYI, I was able to generate a clean build for Windows.

Hi,
Thanks for your interest and for reporting.
WebGL build is a bit different, since I embed it in a special project via iframe (see glTFastWebDemo) which has HTML5 menu (a crappy one, but still) and thus the IMGUI is not shown.
Let me check what I do different.

Thanks @atteneder for the quick reply.
I read the code and realized that Unity's GUI was #defined only for Editor and not WebGL. Disabling that at least I was able to build the WebGL build with the demo's GUI to navigate around the different scenes.

cool!


Create QIR controller input schema

Creates the input schema.

@achocron, can you please target this PR to the feature/azure-quantum-simulator branch?

@achocron since the PR in the qsharp-runtime repo meant to fix the build issues is already merged, I think we should revert the merge from main in this PR to be able to merge it to the feature branch. Afterwards we can merge from main to the feature branch as a separate PR.


onChange is fired even while cursoring around in the textarea

Cursoring around in a textarea should not be firing an onChange.

Afaik, It's inherit lower level library behavior. But i believe there is isDirty method somewhere, which can be useful in your callback

It seems pretty grossly inefficient.  I can't rely on just a dirty flag.  A user could type a character then delete that character.  Now it's flagged as dirty, but the content has not changed.  This means instead of a dirty flag, I have to string comparison the entire content on every onChange even though they may be just cursoring around in the text.  If the text is extremely long, this could be very CPU intensive.
What is this "lower level library" you're referring to?  It's just using a textarea.  Seems like it is perhaps firing on keyup which is incorrect.  It should just fire on input propertychange which I don't believe includes just cursoring around in the textarea.

What is this "lower level library" you're referring to?

The keyup and change, they are both inherited from jQuery.

It's just using a textarea. Seems like it is perhaps firing on keyup which is incorrect. It should just fire on input propertychange which I don't believe includes just cursoring around in the textarea.

This ought to be very simple. But unfortunately it isn't. onkeyup maybe a terrible event for input detection, you got the point. But Cut/Paste via contextmenu unfortunately is also input, and so is backspace.
I haven't test the performance comparison of using keyup against input or propertychange, but I'm sure we're looking at other issue here.


[4.15] OCPBUGS-33697: add a controller that reconciles SCCs' volumes

Manual backport of https://github.com/openshift/cluster-kube-apiserver-operator/pull/1675 that's had conflicts in vendor/modules.txt
/assign @deads2k

/jira refresh

/jira refresh

/label cherry-pick-approved

/label backport-risk-assessed

/lgtm

/cherry-pick release-4.14


Dos plotting

Adding all the final bits of functionality discussed in #10; namely:

DOS plotting alongside the unfolded band structures. This also sets the colours of ($lm$-decomposed) orbitals in the sumo DOS plot to match that of the projected band structure (if using the plot-projections option).
Add --atoms option to just specify which atoms to project onto the band structure, without having to specify atomic indices.
Some refactoring and consolidation of plotting options (i.e. changing --vscale to --colour-norm to make more intuitive, and using this instead of --intensity for plot-projections (same effect)).
Show CLI help messages with -h option
Major docs updates showing these features.
Tests for all these features
Update docstrings and some general code cleanup

Attaching a PDF of the updated NaBiS2 example which illustrates most of these features (some of the sphinx note/tip boxes don't render on my local, but do on the website version):
Disordered NaBiS2 with atomic_orbital projections & DOS plotting — Easyunfold.pdf

Also this means the easyunfold 1.0.0 GH project is now completed. Ready to have the first major release @zhubonan?

Thanks for this! Looks fantastic. I will take a look and have it merged soon.

Ok will get to implementing these once I have time!
Just one thing about the --intensity point. This is what I suggested before in #10;

(Seán): To me, the current setup of vscale is counter-intuitive (see https://smtg-ucl.github.io/easyunfold/cli/). It's a "scaling factor for the colour mapping", but decreasing it increases the colourmap intensity, whereas intuitively as a user I would've expected the opposite behaviour. Obvs a very minor thing, but I think it would be better to reverse this behaviour?

And your thoughts at the time were:

(Bonan): Well I think the current behaviour is fine as it is the colour range scale, so using a small scale should increase the intensity. We can rename it as colour_scale to make it more informative.

If you're now happy to refactor this (now colour-norm to intensity – I agree!) then I'll go ahead with this! 😄

Thanks, I see, actually what I have in mind is to have both --vscale and --intensity options available for now.
The --vscale behaves as before (value scale), so setting it to be less than 1 will make the colour more intensity. And the new option --intensity also affects the colour but in the opposite way, larger value increase the intensity of the colours. This is probably more intuitive. Compared to renaming --vscale to --colour-norm or --intensity with new behaviour, this approach ensures the backward compatibility.
We can issue deprecation warnings for --vscale if need in the future. What do you think?

Yeah I just think the average user is not so familiar with vscale. We can improve this with a better help message explaining it, but as you said (and I was saying before), intensity is definitely a lot more intuitive...
We could keep both options as you say, but at this point is backwards-compatibility really important? Like you, myself and Adair have been the main users so far, and maybe some others from the group, but hasn't been massively advertised / grown the user/dependency base yet, so I'd suggest that this isn't really important right now?
There's already a lot of options for the CLI plotting, so I would think best to not have any redundant ones like this, and also two options doing the exact opposite thing could also just confuse people? (The intensity and vscale options confused me at first when I first came back to look at the code a few weeks ago)

Right, I think maitaining backward compatible is important. It is probably one of the things prevant people from using a code under active development (and complain about). Since we have opted in sematic versioning so users would expect backward compatible behaviour. I agree that the help message should definitely be updated to more informative as you said (which this PR already improved 🚀). Hopefully we are  gaining more users soon (after MC16 😉 ).
Regarding the number of options - yes I think they are a bit too many at the moment. It should be possible to use a file to provide them instead with, which works by  internally modified the default values. I will look into that after the PR is merged.
See also:https://click.palletsprojects.com/en/7.x/commands/#overriding-defaults

Hi @zhubonan! Sorry I had a few deadlines this week so just getting to this now. 😬
Ok fair enough, though we could also just bump to a major version update seeing as there's some major upgrades that have been added, but your call! 😃
For the CLI options, I think it's nice to have a good bit of customisation control as we have now, but also trying to maximise the control while minimising the number of options (and having a good choice of defaults so the user shouldn't need to use a lot of the options in most cases), which was why I was against the redundant options. For something like this, I would have thought that any users relying on the old CLI (which I don't think there is any) would be fixed to that version, or just have installed that version on their local, to ensure compatibility. And if they have it installed locally and are just updating, it seems like something one would figure out quite quickly after getting e.g. colour_norm option not found, and realising intensity had been added instead.
What you say about a config.yml option I think would be a nice addition (though as a separate CLI option would be best for user-friendliness I think, rather than replacing the CLI options). We have this in ShakeNBreak, e.g.:

https://github.com/SMTG-UCL/ShakeNBreak/blob/main/SnB_input_files/example_generate_config.yaml

Worth adding as a TODO for after this PR & JOSS?

Will try get to your comments on the PR and wrap this up ASAP!

@zhubonan all requested changes & updates have now been implemented 😃
I've also merged in #26 as requested, so I'll close that PR now. Let's get this merged and spread the word! 🚀


add parquet-schema

Problem
To write batch files we will need Parquet schema for every DSNP Announcement type.
link to Pivotal Tracker #177940101
TODO: review the schema and make sure they are up to date with the message fields.
Solution
Add the schema files
with @pairperson1

looking great!

amazing!


Missing link

Problem description
Searching for docker info about size of images on Google led me to the broken link below
Problem location
https://docs.docker.com/v17.09/engine/userguide/storagedriver/imagesandcontainers/

Hello @gmisagm, thank you for reporting the broken link issue. This issue has been fixed.
You can find more information about Docker images in the following topics:

https://docs.docker.com/develop/develop-images/dockerfile_best-practices/
https://docs.docker.com/develop/develop-images/baseimages/
https://docs.docker.com/develop/develop-images/image_management/
https://docs.docker.com/docker-hub/official_images/

We recommend that you refer to our latest documentation - https://docs.docker.com/ to learn about Docker.
Hope this helps!


main.BandMemberController is missing

Implement, comment and test for main.BandMemberController

the method getBandsBandMember: get all the bands in which an artist was once a member?
if correct, should return a List ?

solved bug in 3f8d0b9


Fix default loader used when Meltano CLI runs

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/86
Originally created by @iroussos on 2018-09-11 13:00:26

When Meltano CLI is run without a provided loader_name, the default loader is set to 'Postgres' (with a capital P) instead of postgres.
This is a bug that was not caught when we reverted the Capitalization of Extractors and Loaders due to issues faced with the DBT transformations.

View 2 previous comments from the original issue on GitLab


[SPARK-9728][SQL]Support CalendarIntervalType in HiveQL

JIRA: https://issues.apache.org/jira/browse/SPARK-9728

Merged build triggered.

Merged build started.

Test build #40175 has started for   PR 8034 at commit fce7795.

We should also add unit tests for CalendarInterval itself.

It would also be good to actually fill out the description instead of just linking the JIRA, since this becomes the commit message.

Test build #40175 has finished for   PR 8034 at commit fce7795.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #40215 has started for   PR 8034 at commit 7fe9a5e.

Test build #40215 timed out     for PR 8034 at commit 7fe9a5e     after a configured wait of 175m.

Merged build finished. Test FAILed.

Test build #1413 has started for   PR 8034 at commit 7fe9a5e.

Test build #1412 has started for   PR 8034 at commit 7fe9a5e.

Test build #1413 has finished for   PR 8034 at commit 7fe9a5e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Failure due to org.apache.spark.network.sasl.SaslIntegrationSuite.testNoSaslClient

Test build #1412 has finished for   PR 8034 at commit 7fe9a5e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Going to merge this. Thanks!


[Remote debugging] Consider to change transport name into "PowerShell Remote Debugging"

I think specifying the tools part is important since it is our extension, aka, PowerShell Tools, which is providing that transport. I think if we had just PowerShell that could imply something officially related to PowerShell.

Closing and not changing name.


feat: flutter/dart license checker

Description
Add flutter/dart licence checker
Related Issue(s)
Closes #485
Description of Changes

Add license checker for catalyst-voices
Add Apache-2.0 license to all catalyst-voices related packages

Related Pull Requests
https://github.com/input-output-hk/catalyst-ci/pull/248
Please confirm the following checks

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

Closed due to merge conflicts with main.  Can re-open after they are resolved.


feat: update plugins/docker to latest version.

update docker plugin to the latest version to support multi-stage build.
ref: https://github.com/go-gitea/website/pull/43

LGTM

LGTM


0.5.x-events sync from 0.5.x

Problem
0.5.x-events branch is way behind 0.5.x.
Modification
Synced 0.5.x-events to 0.5.x
Result
Branches in sync and I can resume working on it now!

Since this is just a sync with no code changes, I have merged without review.


DTVF v3.1.0

This new version of the framework brings in minor features, bug fixes, and documentation updates.  A full list is detailed below.
Bug Fixes:

Scroll bar sizing in the Layers tree
Default visibility of layers in Cesium visualisations
Toggling visibility for KML layers in Cesium visualisations
Allow specification of positions for 3D tile data sources
Fixed hover effects for features containing properties with null values

Features:

Imagery layer with labelled satellite data
Ability to define default expansion states in the layer tree
Ability to define layer order globally (i.e. irrespective of grouping)
Can now use PAGE_UP and PAGE_DOWN buttons to zoom in Cesium visualisations
Can now select the feature name to fly to it in Cesium visualisations

Misc:

Updated GitHub wiki content
Updated READMEs
Refined camera options

Please remove the file \web\digital-twin-vis-framework\library\utils\python.log. It's 81 MB...

Please update the DTVF version number in web/digital-twin-vis-framework/README.md.

In web/digital-twin-vis-framework/README.md, you seem to have changed "CesiumJS can handle 2D data" to "CesiumJS can handle 2D raster data" and you have added "at the time of writing we have not implemented functionality to support 2D vector data". I find this highly misleading. The Cesium version handles vector data just fine (WMS layers of points, lines, and polygons). Do you mean vector layers are not currently served as vector to the front-end, but instead as image/png? If so, I would also add that that applies to both the Cesium and Mapbox versions. Either way, please clarify.

You have added
MapHandler.MAP_USER = "";
to the Cesium version of index.html, but not the Mapbox version. Is this intentional?

Please remove the file \web\digital-twin-vis-framework\library\utils\python.log. It's 81 MB...

Removed, and added an ignore file to prevent future *.log files being committed.

Please update the DTVF version number in web/digital-twin-vis-framework/README.md.

Done, nice catch.

In web/digital-twin-vis-framework/README.md, you seem to have changed "CesiumJS can handle 2D data" to "CesiumJS can handle 2D raster data" and you have added "at the time of writing we have not implemented functionality to support 2D vector data". I find this highly misleading. The Cesium version handles vector data just fine (WMS layers of points, lines, and polygons). Do you mean vector layers are not currently served as vector to the front-end, but instead as image/png? If so, I would also add that that applies to both the Cesium and Mapbox versions. Either way, please clarify.

I have updated the README, it reads clearer to me now but if you'd like further changes then please post on this PR.

I assume commenting out the (previously uncommented) search bits in web/digital-twin-vis-framework/library/src/ts/manager.ts is intentional?

Yes, this is a feature that's still in progress. It will be continued in v3.2.0 (rather than holding up this PR).

You have added MapHandler.MAP_USER = ""; to the Cesium version of index.html, but not the Mapbox version. Is this intentional?

Funnily enough, this is less critical in the Mapbox visualisation than the Cesium one. For consistency I have added it to the template index.html for the Mapbox visualisation too.


Add programs related to people perception topic

I added programs which publish topics related to people perception.
http://doc.aldebaran.com/2-4/naoqi/peopleperception/index.html#naoqi-peopleperception
I used ALMemory Keys below.
This requires https://github.com/ros-naoqi/naoqi_bridge_msgs/pull/35 for msgs.
I didn't add ALMemory Keys from ALWavingDetection and ALSittingPeopleDetection because in my case, there seemed to be no data change.
I tested it with Pepper + NAOqi <IP_ADDRESS> + ROS indigo.
reference:
PeoplePerception/Person/<ID>/EngagementZone
http://doc.aldebaran.com/2-4/naoqi/peopleperception/alengagementzones-api.html#alengagementzones-api
PeoplePerception/Person/<ID>/AgeProperties
PeoplePerception/Person/<ID>/ExpressionProperties
PeoplePerception/Person/<ID>/GenderProperties
PeoplePerception/Person/<ID>/SmileProperties
PeoplePerception/Person/<ID>/FacialPartsProperties

http://doc.aldebaran.com/2-4/naoqi/peopleperception/alfacecharacteristics-api.html#alfacecharacteristics-api
PeoplePerception/Person/<ID>/EyeOpeningDegree
PeoplePerception/Person/<ID>/GazeDirection
PeoplePerception/Person/<ID>/HeadAngles
PeoplePerception/Person/<ID>/IsLookingAtRobot
PeoplePerception/Person/<ID>/LookingAtRobotScore

http://doc.aldebaran.com/2-4/naoqi/peopleperception/algazeanalysis-api.html#algazeanalysis-api
PeoplePerception/Person/<ID>/AnglesYawPitch
PeoplePerception/Person/<ID>/Distance
PeoplePerception/Person/<ID>/IsFaceDetected
PeoplePerception/Person/<ID>/IsVisible
PeoplePerception/Person/<ID>/NotSeenSince
PeoplePerception/Person/<ID>/PositionInRobotFrame
PeoplePerception/Person/<ID>/PositionInTorsoFrame
PeoplePerception/Person/<ID>/PositionInWorldFrame
PeoplePerception/Person/<ID>/PresentSince
PeoplePerception/Person/<ID>/RealHeight
PeoplePerception/Person/<ID>/ShirtColor
PeoplePerception/Person/<ID>/ShirtColorHSV

http://doc.aldebaran.com/2-4/naoqi/peopleperception/alpeopleperception-api.html#alpeopleperception-api

This is a new feature, which may cause unexpected trouble when it is merged.
I close this and improve this in my local.
I'm sorry to leave this for a long time.


Add tito configs

This also fixes a typo in a spec file
Signed-off-by: Vadim Rutkovsky<EMAIL_ADDRESS>

Note, that this PR won't bind us to tito, it will allow us to build SRPMs for COPR easier


Increment operator "--" does not work

Challenge Waypoint: Decrement a Number with Javascript has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
var myVar = 11;

// Only change code below this line
myVar = myVar--;

it should be myVar-- not myVar = myVar--

oops, my bad

close #5738


Fix Intel compiler compilation issues on windows.

Changes from PR #1616 caused compilation errors on Windows and Intel Compiler. This changes fix compilation issues.

extend compiler flags to -QxCORE-AVX2
looks like Intel Compiler should use intrin.h
this type __m128i_u is unknown

I suppose we no longer need this after #1638 ?

I suppose we no longer need this after #1638 ?
Yes we do not need


Solution not found for broken download

While downloading serenity it keeps saying No solution for broken download.

While this is annoying, this is somewhat normal. Network errors might lead to such errors.
Below steps, in order, are known to help immensely with such:

Wait until Wabbajack finishes with error state, saying broken download, and restart install process. Wabbajack will re-try failed files (and re-use everything what was properly downloaded);
Use network workaround in WJ options, if broken file resides on WJ CDN / LL / MEGA;
For Nexus downloads: change download server in your Nexus settings (only for premium users);
Download files manually (possible for anything apart of WJ CDN files).

Also I would advise you to join Wabbajack and/or Emporio's Emporium - dedicated to Serenity - discord servers.

Closing the issue, as this is not an essential issue with the mod list. Use discord servers mentioned for troubleshooting files download / Wabbajack install process.


Change composer part installation

To make it works out of the box

The paragraph above the code must be changed as well to reflect the code block change or it will talk about something that isn't there any more.

I've changed it in https://github.com/burzum/cakephp-imagine-plugin/commit/2e9b5b39e5fa95e3b3a679ad1224ab35c46553b4


Constant functions trigger events, however they should not

Tested in testrpc and parity, where constant functions do not trigger events.

should not happen anymore


Saleor 1855/implement new cards for return statuses

I want to merge this change because...

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Translated strings are extracted.
[ ] Number of API calls is optimized.
[ ] The changes are tested.
[ ] Data-test are added for new elements.
[ ] Type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Test environment config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: SALEOR-1855 Implement Returned and Returned & Refunded Cards


[REQUEST] Box search of range search for

Is your feature request related to a problem? Please describe.
Since openVDB uses a tree structure, is there any way I can efficiently perform a box search to get all values in a BBox ?

I just added this functionality to NanoVDB. I will be part of OpenVDB version 10 which will be released in a few weeks.
I could add the same functionality to the OpenVDB tree is for some reason you prefer not use NanoVDB

I just added this functionality to NanoVDB. It will be part of OpenVDB version 10 which should release in a few weeks.
I could add the same functionality to the OpenVDB tree if for some reason you prefer not use NanoVDB. However, it will be much slower to perform this type of computation in OpenVDB since, unlike NanoVDB, it does not encode statistics like min/max per tree node.

Thank you so much. Looking forward to seeing that.


Docs: Fix broken link in the quickstart and fix error in code example, capitalize S in TypeScript

The quickstart here deploys to the testnet, and AFAIK transactions will revert since they are not signed on testnet. @mohammadfawaz, do I have this right?

The quickstart here deploys to the testnet, and AFAIK transactions will revert since they are not signed on testnet. @mohammadfawaz, do I have this right?

You're right we should not be adding --unsigned here! I was actually specifically talking about the changes to the Rust SDK test. I.e. replacing
    let (instance, _id) = get_contract_instance().await;
    // Now you have an instance of your contract you can use to test each function
    let result = instance.increment().call().await.unwrap();
    assert!(result.value > 0)

with
    // Increment the counter
    let _result = instance.increment().call().await.unwrap();

    // Get the current value of the counter
    let result = instance.counter().call().await.unwrap();
    assert!(result.value > 0);

oh perf. Done, thank you @mohammadfawaz !


addEventListener and removeEventListener doesn't work with EventListenerOptions

Please see details about EventListenerOptions specification here:
https://github.com/WICG/EventListenerOptions/blob/gh-pages/explainer.md
https://www.chromestatus.com/feature/5745543795965952
The issue is that third parameter of addEventListener and removeEventListener methods is treated as boolean useCapturing parameter. If EventListenerOptions object is used instead then event listener will not be removed with removeEventListener call. This happens because findExistingRegisteredTask task uses reference equality (===) to determine equality of useCapturing parameter. This way it's possible to remove event handler only by passing exactly same object of EventListenerOptions to removeEventListener call.
This behavior seems very counter-intuitive to me. Please add support of EventListenerOptions parameter.

@vevnin , thank you for posting the issue, I will check it.


Misreporting html errors with radio buttons and checkboxes if the text is capitalized with css.

This is something that I just noticed. Its pretty common to set a radio button inside a label like so:
<label>
  <input type="radio">
  Some radio
</label>

In these cases Wallaby shouldn't complain about the label missing a for but it does. The reason why is kinda interesting.
If the label has some css on it like so text-transform: capitalize then in the browser the text will appear as "Some Radio". If you then try to click the radio using something like: click(radio_button("Some Radio")) you'll get the "label is missing a for" error. The reason you get this error is because our xpath query is case sensitive and the xpath looks at whats actually in the DOM. So because the label doesn't match anything we fall into the error flow. We then try to be helpful and validate the HTML. We do that using a query like: css("label", text: "Some Radio"). When the css query is executed it correctly finds the label! This is because when we check text with css we use the text webdriver endpoint which returns rendered text instead of the text thats actually in the DOM. Because our validate html check is finding the label, and because the label doesn't have a for attribute Wallaby is misreporting this as an HTML bug.
There are a couple of ways to fix this. One is to not use the same validate_html logic for checkboxes and radio buttons. This is similar to what we do for buttons. Another would be to normalize all of the casing across our api. This is somewhat tricky because it would mean doing a translation in the xpath queries. We would have to do it in xpath since we can't change what the webdriver returns from text.
The other option is something I've been talking about for a long while but its translating ALL queries into xpath under the hood. In this case it would actually have solved this issue since we'd always be using the same querying operations. That seems like a more involved fix though.
Somewhat related it would be nice to have a specific error for when you search for something that has text and you accidentally get casing wrong or misspell the text. I'm not sure if there is a good way to do that or not but I'd love to find a solution that could accommodate some helpful hints like that.

I suspect I ran into a similar problem with this code: find(session, button("SIGN OUT")). The button has type="button", yet I get this error:
** (Wallaby.QueryError) The text 'SIGN OUT' matched a button but the button has an invalid 'type' attribute.

     You can fix this by including `type="[submit|reset|button|image]"` on the appropriate button.


Fix small typo in CONTRIBUTING.md

In a section about rustfmt there was small typo where it saidrustup-preview instead of rustfmt-preview.

@fitzgen No worries. 😄 Please pardon the tiny PR.


FIX IT

IT'S NOT WORKING !!11!

I WILL PUT 0 STARS IN PLAY MARKET !!11!

Please, calm down!
Our team will look on your request as soon as possible.
Thanks for choosing us


New buffering APIs on HttpResponse and updates to implementation

Description
Adds two new APIs to HttpResponse, bufferAsync which asynchronously buffers the response and isBuffered which indicates whether the response has been buffered into memory and replayable.
Updates were made to implementations:

The synchronous JDK HttpClient response now performs more checks for the byte[] existing, removing some usages of the InputStream given the response has been buffered into memory.
Default buffering implementations now use getBodyAsBinaryData and uses the corresponding toReplayable and toReplayableAsync to buffer the contents of the response.
BufferedHttpResponse no longer uses Flux<ByteBuffer> logic and relies on the BinaryData passed to it to be replayable.

BinaryData now implements Closable to allow closing the backing content. In many cases this is a no-op.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

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


Add scripts for PR metrics from github API

These scripts allow to track a number of metrics about PRs over time, using the github API.
So far they've only be used by me on my machine, but they should be a shared team resource, so here's a PR. The current versions are not intended to be final, they're a starting point that we can evolve in future PRs as needed.
(Some previous review history at https://github.com/ARMmbed/mbedtls-test-old/pull/153 (private).)

Labeling "size:S" based on review time (I would guess a size:M based on coding time, but this is relatively easy to review since checking what a library function does is easier than looking for a library function that does a given thing).


Thunderstorm command does not start thunderstorm

Affected Product(s)
SpongeVanilla
Version
1.16.5-8.0.0-RC1077-universal
Operating System
Windows
Java Version
17.0.1
Plugins/Mods
none

Describe the bug
On a normal singleplayer world, the commands /weather rain and /weather thunder cause rainstorms, but the thunder version also makes it a thunderstorm. The sky visibly becomes darker and lightning strikes occasionally.
In a SpongeVanilla server, /weather thunder produces the same visual result as /weather rain and never produces thunder. I waited for roughly 20 minutes and still no thunder/lightning occurred. It's unclear whether thunderstorms happen naturally without the command, but /weather thunder does not seem to produce the correct vanilla behavior.
Link to logs
No response

It was never actually changing into thunderstorm mode. Enjoy the weather


Little problem using ngx-contextmenu with Safari

Using nix-contextmenu with Safari the "original" context menu is disabled but beside the ngx-contextmenu another native context menu appears (Search with Google, Add to iTunes). How could I prevent this native one?

I just tested the demo with Safari 10.1.2 and it works.  Did you call event.preventDefault() on the event?


With thousands of open orders in the UI grinds to a halt.

There are performance problems with medium-sized amounts of order events coming into the UI. Currently with 4388 orders the UI grinds to a halt.
Investigate the potential areas of improvement and see what can be done!
Ideas
Batching Order Events
Currently OrderEvents for updated orders are sent one at a time. Instead, we should send this as one event (with an array payload).
Then, in log-handlers.ts in the UI we need to handle the OrderEvents with-array case in mind and dispatch for each log received.
Middleware Processing
It's possible that the middleware processing is heavyweight in a way that is affecting the ui.
Events being handled in same event loop as we receive them from the mesh
Perhaps we need to schedule the event handlers in a nextTick?

This issue is pending some optimization work on the 0x side about how they handle holding onto the DOM thread during syncing.
For now @justinbarry is also moving the mesh to a web worker, ref #7759

This is currently fixed by running in a web worker + 0x is doing performance optimizations which may let us get rid of the WW.


USHIFT-3315: Increase delete openshift-ingress pod timeout

/lgtm
/test microshift-metal-tests

/cherry-pick release-4.16


NN not working after loading from json

What is wrong?

I train network and do net.run(element) and it return good number (for example 0.57).
I save network to JSON let json=net.toJSON() and load it net.fromJSON(json) and make net.run(element) and it return 1.00 in output with same NN json data.
Where does it happen?

When I trying save and load NN.
Windows 10 node version 11.3.0
How do we replicate the issue?

try load and save from examples (I tried it and it's works)

How important is this (1-5)?

4
Expected behavior (i.e. solution)

Save and load NN, and  net.run(element) with succesful output.
Other Comments
My listing code for run, save, load
          // NN initialized: let net = new brain.NeuralNetwork();

            //run with training NN
            let output = net.run([0.7, 0.1, 0.1, 0.2]);
            console.log(parseFloat(output).toFixed(2)); //return 0.57

            //save and load NN
            const jsonFileContent = JSON.stringify(net.toJSON());
            const json_from_save = JSON.parse(jsonFileContent);

            const net_from_save = new brain.NeuralNetwork();
            net_from_save.fromJSON(json_from_save);
            let output_from_save = net_from_save.run([0.7, 0.1, 0.1, 0.2]);
            console.log(parseFloat(output_from_save).toFixed(2)); //return 1.00

After loading from json NN activation set to "sigmoid", but in json activation is "leaky-relu"

I have a fix for this, and will post it probably tomorrow sometime.  Internet it a bit spotty at the moment.


Adding a role to a user when receiving a message

I'm trying to add a role to a user once they send my bot a specific command. I'm currently not having any errors, but it is never calling my reply saying it has added the role. I also could not find a clean way to get access to the guild member, so the way I've done it is a bit disgusting.
$rawMemberArray = json_decode(json_encode($message->__get('member')), true);
$member = [
    'user' => $rawMemberArray,
];

$guildMember = new \CharlotteDunois\Yasmin\Models\GuildMember($this->client, $message->__get('guild'), $member);
$guildMember->addRole('583686377992880167')->done(
    function () use ($message) {
        $message->reply('I have successfully synced your character and updated your roles.');
    },
    function($error) {
        throw new \Exception($error);
    }
);

Any ideas?

My idea is that you've did everything wrong there. If the constructor is not documented, then the constructor is internal only (it means it's not safe for use). The $member array does not even have the correct format.
What is wrong with the GuildMember from $message->member? It returns the GuildMember or null (= uncached, should never or rarely be the case).
$member = $message->member;

$member->addRole('583686377992880167')->done(
    function () use ($message) {
        $message->reply('I have successfully synced your character and updated your roles.');
    },
    function($error) {
        throw new \Exception($error);
    }
);

Hmm, sorry - I see now you have an @internal docblock on the GuildMember constructor.
I did try your version and got nothing. I've got the role id by sending @role in discord and copying the id, and it does hit that addRole code, but just doesn't do anything after that.
It doesn't log anything or proceed into the callbacks, nor does it add the role to the user. It also doesn't throw any errors. Any ideas?

Rightclicking the mention and clicking on copy id does NOT copy the role id. It copies the message id (blame discord).
However it should always give you an error, regardless of the ID. Does the process exit? Does anything happen at all?
You can listen on the debug event and print the message to stdout to see what is going on. Additionally there is an error event, also attach a listener there and print it to stderr/stdout.
This will give you more insight into what is happening inside the library.

Yeah sorry, I actually ran \@member to get the role id.
It seems my throw exception was not working (or being caught within the enclosing try/catch. The error is actually missing permissions, which is interesting. The bot role has Admin Server, Manage Roles, and all admin permissions, and the bot actually has that role.

Followed this: https://github.com/y21/discordcaptcha/wiki/DiscordAPIError-50013-Missing-Permissions and got it fixed. Turns out role permission matters more then I thought.
Thanks mate for your help!


Pull

Advantage 360 Pro PR template
What's changed:
Why has this change been implemented:
What (if any) actions must a user take after this change:

Please dont open PRs into main for personal keymap changes


Merged files to utils

Merged js files in utils directory.
This PR will resolve the issue #7 .

Umm, I think you just deleted the files(?). There is no utils dir.

Umm, I think you just deleted the files(?). There is no utils dir.

Umm, I think you just deleted the files(?). There is no utils dir.

Oops, Sorry! Can you please check now.

Umm, I think you just deleted the files(?). There is no utils dir.

Oops, Sorry! Can you please check now.

You changed the location of files, but didn't update the locations in the other files that use them.
I'd like you to follow the instructions in the Contributing Guide and set up the bot, that will help you recognise the errors that this will throw.

You changed the location of files, but didn't update the locations in the other files that use them.
I'd like you to follow the instructions in the Contributing Guide and set up the bot, that will help you recognise the errors that this will throw.


[stable/external-dns]Fix typo in values-production

Signed-off-by: Carlos Rodriguez Hernandez<EMAIL_ADDRESS>What this PR does / why we need it:
Fix a typo in the values-production:
$ helm template . -f values-production.yaml --name foobar
Error: failed to parse values-production.yaml: error converting YAML to JSON: yaml: line 288: did not find expected key

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[ ] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test
/lgtm


[suggestion] reaggrange list in programming-languages by stars

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[ ] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

[x] My suggested edits are not about curating a new topic or collection, or at least not a single one

Thanks For Contributing @MaxGraey. But I think there must be a reason to sort them upon stars. Because people get fake stars pretty easily.
Here are some resources I read before this comment and I think might be useful for you to understand:

A Stack Exchange Question
A Article By Traefik Labs
A Dev.to Article

Plus, I don't think that sorting will make any difference in the programming languages

I agree that this is not the best metric. Perhaps it is worth considering some weighted average of the number of stars, the number of forks and the number of questions on stackoveflow, for example. But for this you have to write a whole script. Another option is just leave as is.

@MaxGraey If You Are Working All The Metrics Then It Is Perfect.

@MaxGraey If You Are Working All The Metrics

Not yet, but I could try write such script and share it and retrieved results. Just give me some time)

👋🏼  @leereilly - I'd love to know your thoughts on this :)

👋 I like the idea of sorting by stars, but I think keeping the list sorted as star counts change will be a difficult and time consuming task.

I'm wonking on script which take into account stars, forks and probably other metrics like count of stackoveflow questions (but not sure about this)

Alright, here:
https://gist.github.com/MaxGraey/e80843bf15e9eacc4f2881126b590648
node.js script which requests explore/collections/programming_languages/index.md extract repos and sort them by simple weighted geometric mean score computed from number of stars, subscribers and forks. Afterwards generate new index.md page with properly sorted names.
Unfortunately I reached rate limit for GItHub's API and can't retrieve results yet 🤦

We don't currently have sorting capabilities on these collections, so we're sticking to sorting by added date. If this changes we'll update the docs here.


Only run codecov when CI environment variable is set

npm run test makes codecov run. That should probably only happen when it's in a CI environment. This PR changes it to be a noop when CI environment variable is missing.

[Current coverage][cc-pull] is 95.90%

Merging [#238][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #238   diff @@
==========================================
  Files            30         30
  Lines           952        952
  Methods          85         85
  Messages          0          0
  Branches        206        206
==========================================
  Hits            913        913
  Misses           39         39
  Partials          0          0

Powered by Codecov. Last updated by [6876cb2...c0dc8bc][cc-compare]
[cc-base-branch]: https://codecov.io/gh/bvaughn/react-virtualized/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/bvaughn/react-virtualized/compare/6876cb27a89b626ccb5c4227b7d3739550880657...c0dc8bc0536a08f3432abc51a4620d64a558b7bf
[cc-pull]: https://codecov.io/gh/bvaughn/react-virtualized/pull/238?src=pr


ValueSet validation does not work properly when valueSet contains not-present system or http://loinc.org system

We are having a ValueSet https://simplifier.net/testprojectt/fiphr-vs-testvalueset, that has 3 systems defined. Two of these we have our own CodeSystems resources and third one uses loinc.org as a system.
CodeSystem1:
system "https://simplifier.net/testprojectt/fiphr-cs-code1and2"  (https://simplifier.net/testprojectt/fiphr-cs-code1and2) has codes Code1 and Code2
CodeSystem2:
system  "http://phr.kanta.fi/CodeSystem/fiphr-cs-notpresent" (https://simplifier.net/testprojectt/fiphr-cs-notpresent) does not have any codes present
When I POST a resource (https://simplifier.net/testprojectt/observation-example) with a system = "http://phr.kanta.fi/CodeSystem/fiphr-cs-code1and2" & with code ="Code", there is no validation error given.
Validation error is given only when system is http://loinc.org and code is not from the valueset,
Resource: https://simplifier.net/testprojectt/observation-example-duplicate-2
Response:
{
"severity": "error",
"code": "processing",
"diagnostics": "None of the codes provided are in the value set http://phr.kanta.fi/ValueSet/fiphr-vs-testvalueset (http://phr.kanta.fi/ValueSet/fiphr-vs-testvalueset, and a code from this value set is required) (codes = http://loinc.org#Code3)",
"location": ["Observation.category"]
},

We are using hapi 3.6.0


Sidebar folders' anchor tag has no href tag

Every folder configured has no href set on its anchor tag in the sidebar. Chrome Lighthouse warns about this issue in the following way:

Just realized that this is the wrong repo. Will reopen on the right one.


Build infrastructre on Windows to support compilation using the /MDd

Visual Studio on windows normally configures debug builds to use the /MDd flag.  This links to the multi-threaded debug libraries.   The debug libraries are not compatible with the release build multi-threaded libraries which are linked in with the flag /MD.
It is very useful to have a way to compile the ortools.lib with /MDd.  Debug (/MDd) and Release (/MD) ortools.lib builds can then be added to Visual Studio in the default way.
A top level flag is added to Makefile.local to support compilation with /MDd.

Hmm.  I signed the CLA a few days ago, and this is my second pull request.
Is there something else I need to do so that it recognizes me?

Ok.  I figured out why the CLA didn't find me automatically.  My globally configured .gitconfig e-mail was different from my google account.


v1.0.0-rc.0 fails to fetch https://registry.npmjs.org/bl/-/bl-4.0.3.tgz

With v1.0.0-rc.0 while pulling bl/4.0.3 I get the following error:
ERROR: /home/XXX/repos/XXX/projects/support/icon-converter/BUILD.bazel:3:22: //projects/support/icon-converter:.aspect_rules_js/node_modules/bl/4.0.3/pkg depends on @npm__bl__4.0.3//:source_directory in repository @npm__bl__4.0.3 which failed to fetch. no such package<EMAIL_ADDRESS>at offset 773, object has duplicate key: "license"
ERROR: Analysis of target '//projects/support/icon-converter:icon-converter' failed; build aborted:
INFO: Elapsed time: 0.252s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (1 packages loaded, 0 targets configured)
FAILED: Build did NOT complete successfully (1 packages loaded, 0 targets configured)

Inspecting the package.json from https://registry.npmjs.org/bl/-/bl-4.0.3.tgz shows the following with is of course broken.
{
  "name": "bl",
  "version": "4.0.3",
  "description": "Buffer List: collect buffers and access with a standard readable Buffer interface, streamable too!",
  "license": "MIT",
  "main": "bl.js",
...
  "license": "MIT",
...
}

This malformed package.json was fixed with https://github.com/rvagg/bl/blob/v4.0.4/package.json
yarn2 handles this fine.

This is the same issue seen here, https://github.com/aspect-build/rules_js/issues/148 where the underlying problem is json.decode is too strict and fails on duplicate keys. @alexeagle filed an issue with Bazel here: https://github.com/bazelbuild/bazel/issues/15605.

Thank you :)


Wistar roller blinds - not fully functional

Hi @make-all,
I've tried using today the Wistar roller blinds profile but it's not fully functional. I've encountered 3 apparent issues:
Issue 1. The Reversed switch (dpID 103) being enabled actually means normal operation, not reversed operation. I couldn't find an option to revert it in the device profile.

Issue 2. Only one command is available: Close or Open, depending on the status of the Reversed switch.

Issue 3. The current open/close position of the motor is not reflected in HA. Unless I set a position, it never changes.

P.S. The response of the motor is so much better now than it was months ago. Congratulations. 👍🏼

I think the problem of open being disabled when the blind is closed may also be linked to the reversal of the reverse setting.
Position can only report what the device is reporting.  If the device is only using the position as input, then it will not change unless explicitly set.

I just made a change that flips the meaning of reversed c3d78509bc3c.  After this, it will be clearer what problems are separate issues that still remain.

Thanks @make-all. https://github.com/make-all/tuya-local/commit/c3d78509bc3cf77099ea521a25877d0ea75ee8b9 fixed the reverse switch.
But issues 2 and 3 remain.

Hi @make-all,
Here are de device diagnostics.
tuya_local-bf945b4460e06d653429729f892ca577-Living Room Shutter-ce09bf37ebf974b74724daba715db421.zip

One potential issue is that I named an attribute "state", which might interfere with Home Assistant's expectation of what the main state for a cover device is.  I don't really know what the extent of the effect of this would be, but everything else looks normal.
Another possibility - have you checked that the position sensor is working? (eg via Tuya cloud).  The current position is being reported as 0 (fully closed), which if it is not correct will prevent the close function from working, since Home Assistant seems to disable the open/close controls when it is at the respective end..

Yes, The position is reported correctly in the Tuya app, but also in the localtuya integration which I'm currently using for the shutters.

Position in the diagnostics is 0 (fully open).  This is what is being read from the device.
Can you take the diagnostics when it is stopped somewhere in the middle, as per the screenshot for Issue 2.  Also if you can send diagnostics when reproducing Issue 3.

I am not sure that Issue 3 is a bug, it may be just the way that Home Assistant works.  Is current position separately listed under Attributes?

Position in the diagnostics is 0 (fully open). This is what is being read from the device.

Position 0 is fully closed, not fully open. DPID 112 should reflect the current position but as far as I can see it remains 0.
DPID 102, which is "set position" does change even when only using the arrow to move the motor.
tuya_local-bf945b4460e06d653429729f892ca577-Living Room Shutter-ce09bf37ebf974b74724daba715db421.json (1).txt

It seems the position sensor in this device is not functioning, and that is why you are not seeing the position reflected.  It seemed to be working in #228, so is there a possibility that it has broken since then?

Please try removing the device, and re-adding it with the new profile I just added (wistar_roller_blind_nopos.yaml)
This ignores the current_position dp, and just uses position.

The issue seems to be caused by how often Tuya Local checks the value of dpID 112.
Tuya Local shows dpID 112 as 100 (fuly open) instead of 79, but localTuya and iot.tuya.com read it properly as 79.
Tuya Local showed dpID 112 as 79 only after reloading the integration. I've waited about 5 minutes before reloading the integration and the value hadn't been updated.

Tuya-local doesn't check individual dps separately. It gets full updates from the device on each poll.

I don't know if localtuya or iot.tuya check the dpIDs individually, I was just mentioning my findings. No disrespect whatsoever to your work.
I forgot to mention in my previous post that I had tried the profile wistar_roller_blind_nopos but it still wasn't displaying the corect position.
However I've upgraded now to v2023.1.0 and tried again with the profile wistar_roller_blind_nopos and the position of the motor gets updated properly. But it does take a long time between updates ~ 15 seconds. Is that configurable?
Thanks a lot for your work.

The update interval seems to depend on the platform, I'm not sure where in Home Assistant this is determined and whether it is configurable at all.  I am however working on moving to persistent connections instead of polling, which is how localtuya manages to reflect state changes so quickly.  But there is a big learning curve here with how Python and Home Assistant manage threads and asynchronous I/O.  Localtuya forked their own version of the pytuya library after the upstream developer abandoned it, and have converted their version to use asyncio, but tinytuya, which is the maintained fork I chose to use still does not support asyncio, so it isn't so easy to fit into Home Assistant's async architecture.

Thanks a lot for the explanation. It's all much clearer now. 👍

@make-all I've started using the WiFi motors with version 2023.1.2 and it's working really well with the persistent connections you added in 2023.1.1.
Congratulations on the new implementation. 👍


:bug: Schema for groups is not working

Looks like the schema for the user thinks groups is a string, while checking the code it seems to be more a list of strings and that matches with the yip library and how it parses the groups.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

whats with tumbleweed nowwwww

All credit to nan0man on slack for finding it out!

Fixes: https://github.com/kairos-io/kairos/issues/1083

Dont think it needs to wait for the full CI


feat!: New AlternatePattern argument for SequenceEffect

This PR fixes math in MoveToEffect so that negative progress did not reflect the direction of the delta offset vec.
Corrected progress getter overload in SequenceEffect to correctly map to the progress when _index < 0.
Added onStart() in recede() initial condition so that effects like MoveToEffect can recalculate their initial data.
Removed unnecessary math in DurationEffectController to avoid potential negative zeros.
Introduced a new type AlternatePattern which allows users to include or exclude the last effect in the sequence when the pattern reverses. NOTE: When playing forward, the last effect is List.last, but in reverse it is List.first.
Default value is now AlternatePattern.repeatLast.
Made the TiledGame example project more aesthetically interesting.

The camera now pans to all 4 corners WRT the bounds of the loaded map and uses the new AlternatePattern feature along with all the new fixes.

[x] I have followed the Contributor Guide when preparing my PR.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[x] I have updated/added relevant examples in examples or docs.

TO DISCUSS

I have skipped the test checkbox until I had a review and we could discuss these changes.
We may need to consider an onEnd() for Effects. Or provide onStart() with an initial progress value to calculate from.

Breaking Change?

[ ] Yes, this PR is a breaking change.
[x] No, this PR is not a breaking change.

Related Issues
I looked but didn't immediately see anyone reporting these issues. I discovered them myself.

There's still the topic to discuss: I'm in favor of providing onStart() with an initial progress value to calculate the starting conditions from since MoveEffect can be affected by bounds checks.

In the case that dProgress becomes negative it is supposed to go backwards. Say that the previous progress is 0.9 and it it supposed to move backwards to 0.8, then dProgress should be -0.1, not 0.1.
Try running only the the MoveToEffect with alternate: true.

Please try running the tiled_example/lib/main.dart example with from my PR and compare it with the same example in upstream after adding the ViewportAwareBounds to the camera in the upstream version. From our conversation, this issue seems to be missed and it's crucial to understanding the problem that is being addressed with the addition to onStart() in the recede() function and using a non-negative duration value.

I'll give it a proper try when I'm back home again next week!

@TheMaverickProgrammer I haven't had the time to test it properly yet unfortunately, but there is a test failing, could you have a look at that first?
 test/effects/sequence_effect_test.dart 271:9

@TheMaverickProgrammer I haven't had the time to test it properly yet unfortunately (forgot that it was on my table 😅), but there is a test failing, could you have a look at that first?
 test/effects/sequence_effect_test.dart 271:9

The test failing is related to the issue at hand. I have thought a lot about it and can propose a quick solution but it's not ideal: There can be a MoveTo effect and a MoveToWithCorrection effect. The latter can be used for components that may not arrive at their destination and need to be adjusted when the effect begins. This new effect would update its deltas every time onStart() is invoked (see PR for the additional callsite for onStart()) and the former effect would simply cache its delta the first time onStart() is ran. This way the test can pass and then this MoveToWithCorrection component can be used on components like the Camera when ViewportAwareBounds behavior is used.

There can be a MoveTo effect and a MoveToWithCorrection effect.

Hmm, I'm not too fond of that idea, it feels like a less clean API.
How come that it currently passes the test? I didn't think MoveToEffect had any correction built in to it, since it wasn't designed to stack with multiple effects of the same type (like MoveByEffect is)?

There can be a MoveTo effect and a MoveToWithCorrection effect.

Hmm, I'm not too fond of that idea, it feels like a less clean API. How come that it currently passes the test? I didn't think MoveToEffect had any correction built in to it, since it wasn't designed to stack with multiple effects of the same type (like MoveByEffect is)?

Because in the PR I said I added a new callsite to onStart() for effects in order to allow MoveTo effect to recalculate its starting position whenever it was unable to ensure its component arrived to its destination. This edge case happens in the camera with viewport aware bounds behavior but can potentially happen for other effects/behaviors that dictate a component's position.


[fix] add type hints for yolo_neck

Motivation
超级视客营mmdet基础任务，选择链接中任一part代码添加type hints，为part-9的yolo_neck.py添加type hints
Modification
为yolo_neck.py中的DetectionBlock和YOLOV3Neck相关函数添加type hints
Checklist

[x] 1. Pre-commit or other linting tools are used to fix the potential lint issues.
[x] 2. The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[x] 3. If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
[x] 4. The documentation has been modified accordingly, like docstring or example tutorials.

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


[SUPPORT] Metadata table thows hbase exceptions

Description
We're running on a cloudera cdp stack and want to upgrade to hudi 0.11.1 and take advantage of the metadata table feature. We tried to run a simple hudi write with generated data an got the attached stacktrace.
We have used this hudi package: org.apache.hudi:hudi-spark3.1-bundle_2.12:0.11.1.
The exception indicates that maybe something is not compatibe with the hbase version which hudi is compiled against. Unfortunately Cloudera provides hbase in verison 2.2.3. A far as we understood, hbase is only used if index type is set to hbase, so we're not sure why hudi need the hbase class here.
If we set hoodie.metadata.enable to false it's working, but we want to take advantage of this feature.
We tried 2 things to get rid of this exception.

Set Index type to BLOOM ->  no effect
Especially add the hbase server and client jar to the spark shell in the version hudi is compiled against -> no effect

Environment Description

Hudi version : 0.11.1

Spark version : 3.1.1

Hive version : 3.1.3

Hadoop version : 3.1.1

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

Running on Docker? (yes/no) : no -> yarn on cloudera cdp 7.1.7

Additional context
. Example write:
df.write.format("hudi")
  .option(HIVE_CREATE_MANAGED_TABLE.key(), false)
  .option(HIVE_DATABASE.key(), "db_demo")
  .option(HIVE_SYNC_ENABLED.key(), true)
  .option(HIVE_SYNC_MODE.key(), "HMS")
  .option(HIVE_TABLE.key(), "ht_hudi_11_1_metadata")
  .option("hoodie.table.name", "ht_hudi_11_1_metadata")
  .option(KEYGENERATOR_CLASS_NAME.key(), "org.apache.hudi.keygen.NonpartitionedKeyGenerator")
  .option(OPERATION.key(), "upsert")
  .option(PRECOMBINE_FIELD.key(), "sequence")
  .option(RECORDKEY_FIELD.key(), "id")
  .option(TABLE_NAME.key(), "ht_hudi_11_1_metadata")
  .option("hoodie.index.type","BLOOM")
  .option("hoodie.metadata.enable", true)
  .mode("append")
  .save("hdfs:///.../hudi_11_1_metadata")

Stacktrace
Caused by: java.lang.ExceptionInInitializerError
        at org.apache.hudi.org.apache.hadoop.hbase.io.hfile.HFileContextBuilder.<init>(HFileContextBuilder.java:54)
        at org.apache.hudi.common.table.log.block.HoodieHFileDataBlock.serializeRecords(HoodieHFileDataBlock.java:105)
        at org.apache.hudi.common.table.log.block.HoodieDataBlock.getContentBytes(HoodieDataBlock.java:131)
        at org.apache.hudi.common.table.log.HoodieLogFormatWriter.appendBlocks(HoodieLogFormatWriter.java:158)
        at org.apache.hudi.io.HoodieAppendHandle.appendDataAndDeleteBlocks(HoodieAppendHandle.java:404)
        at org.apache.hudi.io.HoodieAppendHandle.doAppend(HoodieAppendHandle.java:382)
        at org.apache.hudi.table.action.deltacommit.BaseSparkDeltaCommitActionExecutor.handleUpdate(BaseSparkDeltaCommitActionExecutor.java:84)
        at org.apache.hudi.table.action.commit.BaseSparkCommitActionExecutor.handleUpsertPartition(BaseSparkCommitActionExecutor.java:322)
        ... 28 more
Caused by: java.lang.RuntimeException: hbase-default.xml file seems to be for an older version of HBase (<IP_ADDRESS>.1.7.0-551), this version is 2.4.9
        at org.apache.hudi.org.apache.hadoop.hbase.HBaseConfiguration.checkDefaultsVersion(HBaseConfiguration.java:74)
        at org.apache.hudi.org.apache.hadoop.hbase.HBaseConfiguration.addHbaseResources(HBaseConfiguration.java:84)
        at org.apache.hudi.org.apache.hadoop.hbase.HBaseConfiguration.create(HBaseConfiguration.java:98)
        at org.apache.hudi.org.apache.hadoop.hbase.io.crypto.Context.<init>(Context.java:44)
        at org.apache.hudi.org.apache.hadoop.hbase.io.crypto.Encryption$Context.<init>(Encryption.java:110)
        at org.apache.hudi.org.apache.hadoop.hbase.io.crypto.Encryption$Context.<clinit>(Encryption.java:107)
        ... 36 more
........
22/08/12 08:19:20 ERROR scheduler.TaskSetManager: Task 0 in stage 6.0 failed 4 times; aborting job
org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 6.0 failed 4 times, most recent failure: Lost task 0.3 in stage 6.0 (TID 9) (hdl-w05.charite.de executor 1): org.apache.hudi.exception.HoodieUpsertException: Error upserting bucketType UPDATE for partition :0
        at org.apache.hudi.table.action.commit.BaseSparkCommitActionExecutor.handleUpsertPartition(BaseSparkCommitActionExecutor.java:329)
        at org.apache.hudi.table.action.commit.BaseSparkCommitActionExecutor.lambda$mapPartitionsAsRDD$a3ab3c4$1(BaseSparkCommitActionExecutor.java:244)
        at org.apache.spark.api.java.JavaRDDLike.$anonfun$mapPartitionsWithIndex$1(JavaRDDLike.scala:102)
        at org.apache.spark.api.java.JavaRDDLike.$anonfun$mapPartitionsWithIndex$1$adapted(JavaRDDLike.scala:102)
        at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsWithIndex$2(RDD.scala:915)
        at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsWithIndex$2$adapted(RDD.scala:915)
        at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:52)
        at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:373)
        at org.apache.spark.rdd.RDD.iterator(RDD.scala:337)
        at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:52)
        at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:373)
        at org.apache.spark.rdd.RDD.$anonfun$getOrCompute$1(RDD.scala:386)
        at org.apache.spark.storage.BlockManager.$anonfun$doPutIterator$1(BlockManager.scala:1440)
        at org.apache.spark.storage.BlockManager.org$apache$spark$storage$BlockManager$$doPut(BlockManager.scala:1350)
        at org.apache.spark.storage.BlockManager.doPutIterator(BlockManager.scala:1414)
        at org.apache.spark.storage.BlockManager.getOrElseUpdate(BlockManager.scala:1237)
        at org.apache.spark.rdd.RDD.getOrCompute(RDD.scala:384)
        at org.apache.spark.rdd.RDD.iterator(RDD.scala:335)
        at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:52)
        at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:373)
        at org.apache.spark.rdd.RDD.iterator(RDD.scala:337)
        at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:90)
        at org.apache.spark.scheduler.Task.run(Task.scala:131)
        at org.apache.spark.executor.Executor$TaskRunner.$anonfun$run$3(Executor.scala:497)
        at org.apache.spark.util.Utils$.tryWithSafeFinally(Utils.scala:1439)
        at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:500)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.NoClassDefFoundError: Could not initialize class org.apache.hudi.org.apache.hadoop.hbase.io.crypto.Encryption$Context
        at org.apache.hudi.org.apache.hadoop.hbase.io.hfile.HFileContextBuilder.<init>(HFileContextBuilder.java:54)
        at org.apache.hudi.common.table.log.block.HoodieHFileDataBlock.serializeRecords(HoodieHFileDataBlock.java:105)
        at org.apache.hudi.common.table.log.block.HoodieDataBlock.getContentBytes(HoodieDataBlock.java:131)
        at org.apache.hudi.common.table.log.HoodieLogFormatWriter.appendBlocks(HoodieLogFormatWriter.java:158)
        at org.apache.hudi.io.HoodieAppendHandle.appendDataAndDeleteBlocks(HoodieAppendHandle.java:404)
        at org.apache.hudi.io.HoodieAppendHandle.doAppend(HoodieAppendHandle.java:382)
        at org.apache.hudi.table.action.deltacommit.BaseSparkDeltaCommitActionExecutor.handleUpdate(BaseSparkDeltaCommitActionExecutor.java:84)
        at org.apache.hudi.table.action.commit.BaseSparkCommitActionExecutor.handleUpsertPartition(BaseSparkCommitActionExecutor.java:322)

the version of hbase api is 2.4.9

Thanks for your response, I already saw this in the pom file. Thats why we started our spark-shell with the hbase jars in this version, but unfortunately with no success. I guess they are the last one on the classpath.
I was wondering why the hbase lib is mandatory for the metadate table. Where is the metadata stored when we're setting this option? .option("hoodie.index.type","BLOOM")

changed hbase.defaults.for.version   ture  in hudi-common/src/main/resources/hbase-site.xml

changed hbase.defaults.for.version true in hudi-common/src/main/resources/hbase-site.xml

I have unpacked the jar, changed the parameter and packed it again. But without success. Still the same failure.

Thanks for your response, I already saw this in the pom file. Thats why we started our spark-shell with the hbase jars in this version, but unfortunately with no success. I guess they are the last one on the classpath.
I was wondering why the hbase lib is mandatory for the metadate table. Where is the metadata stored when we're setting this option? .option("hoodie.index.type","BLOOM")

Same question... I didn't find the option either

@rbtrtr so the issue is in the first exception in your stack-trace:
Caused by: java.lang.ExceptionInInitializerError
        at org.apache.hudi.org.apache.hadoop.hbase.io.hfile.HFileContextBuilder.<init>(HFileContextBuilder.java:54)
        at org.apache.hudi.common.table.log.block.HoodieHFileDataBlock.serializeRecords(HoodieHFileDataBlock.java:105)
        at org.apache.hudi.common.table.log.block.HoodieDataBlock.getContentBytes(HoodieDataBlock.java:131)
        at org.apache.hudi.common.table.log.HoodieLogFormatWriter.appendBlocks(HoodieLogFormatWriter.java:158)
        at org.apache.hudi.io.HoodieAppendHandle.appendDataAndDeleteBlocks(HoodieAppendHandle.java:404)
        at org.apache.hudi.io.HoodieAppendHandle.doAppend(HoodieAppendHandle.java:382)
        at org.apache.hudi.table.action.deltacommit.BaseSparkDeltaCommitActionExecutor.handleUpdate(BaseSparkDeltaCommitActionExecutor.java:84)
        at org.apache.hudi.table.action.commit.BaseSparkCommitActionExecutor.handleUpsertPartition(BaseSparkCommitActionExecutor.java:322)
        ... 28 more
Caused by: java.lang.RuntimeException: hbase-default.xml file seems to be for an older version of HBase (<IP_ADDRESS>.1.7.0-551), this version is 2.4.9
        at org.apache.hudi.org.apache.hadoop.hbase.HBaseConfiguration.checkDefaultsVersion(HBaseConfiguration.java:74)
        at org.apache.hudi.org.apache.hadoop.hbase.HBaseConfiguration.addHbaseResources(HBaseConfiguration.java:84)
        at org.apache.hudi.org.apache.hadoop.hbase.HBaseConfiguration.create(HBaseConfiguration.java:98)
        at org.apache.hudi.org.apache.hadoop.hbase.io.crypto.Context.<init>(Context.java:44)
        at org.apache.hudi.org.apache.hadoop.hbase.io.crypto.Encryption$Context.<init>(Encryption.java:110)
        at org.apache.hudi.org.apache.hadoop.hbase.io.crypto.Encryption$Context.<clinit>(Encryption.java:107)
        ... 36 more

Your hbase-default.xml is for older version of HBase than the Hudi is using (2.4.9). Would it be possible for you to upgrade it?

@alexeykudinkin no we‘re not able to upgrade.

@rbtrtr can you try this remediation:
https://stackoverflow.com/questions/16497480/hbase-error-hbase-default-xml-file-seems-to-be-for-and-old-version-of-hbase-nu

@rbtrtr can you try this remediation?
https://stackoverflow.com/questions/16497480/hbase-error-hbase-default-xml-file-seems-to-be-for-and-old-version-of-hbase-nu

@alexeykudinkin You mean adding the hbase xml and set the skip version parameter? I already tried this in some comments above. Or did you think of a different approach?

The metadata table in Hudi uses HFile as the base file format.  This issue should be fixed by #6114 (setting hbase.defaults.for.version.skip to true) which is included in 0.12.0.  @rbtrtr could you double-check?  And there is no need to start spark-shell with the HBase jars.

The metadata table in Hudi uses HFile as the base file format. This issue should be fixed by https://github.com/apache/hudi/pull/6114 (setting hbase.defaults.for.version.skip to true) which is included in 0.12.0. @rbtrtr could you double-check? And there is no need to start spark-shell with the HBase jars.

To clarify what @yihua was saying: this should have already been addressed in 0.12. Can you try 0.12 and see if that works for you?

I tried 0.12.0, but with the same stacktrace like before. In the jar file itself are two hbase xml files

hbase-default.xml with hbase.defaults.for.version.skip = false
hbase-site.xml with hbase.defaults.for.version.skip = true

Might this cause the error?

This is a miss. Filed HUDI-4836

@alexeykudinkin should it work if we unpack the jar. Change the parameter and pack it again? Just as a workaround until it is fixed?

Yeah, you can try to remove "hudi-default.xml"

@rbtrtr To provide more info, "hudi-default.xml" is packaged from the HBase library (shaded and packaged in the Hudi bundle) and it is required to load the default HBase config values.  To avoid the config conflicts with other HBase libs in the classpath, Hudi repo introduces "hbase-site.xml" to override the values ("hbase-site.xml" is loaded after "hudi-default.xml").
Check this for more details.  https://github.com/apache/hudi/pull/5004 .

Addresses HBase 2.x conflict with HBase 1.x in Hive 2: In Hudi, we newly include hudi-common/src/main/resources/hbase-site.xml containing HBase default configs from the hbase-common 2.4.9 we use, to override the default configs loaded from hbase-default.xml from an older HBase version on the classpath and to ensure correct default configs for Hudi HBase usage. In Hive, the Hive server loads all lib jars including HBase jars with its corresponding hbase-default.xml into class path (e.g., HBase 1.1.1), and that can cause conflict with the hbase-default.xml in Hudi bundles (HBase 2.4.9). The exception is thrown as follows: Caused by: java.lang.RuntimeException: hbase-default.xml file seems to be for an older version of HBase (1.1.1), this version is 2.4.9. Relevant logic causing such exception can be found in HBaseConfiguration::addHbaseResources(). To get around this issue, since HBase loads hbase-site.xml after hbase-default.xml, we provide hbase-site.xml from the bundle so that HBaseConfiguration can pick it up and ensure the right defaults.

The main problem I think is that there should not be any other HBase libs in the class path since Hudi bundles already introduce the HBase classes.  The reason you see below is that the HBase class shaded in Hudi (org.apache.hudi.org.apache.hadoop.hbase.HBaseConfiguration) reads the hbase-default.xml from cdh HBase, which causes.  This unfortunately happens because hbase-default.xml is going to be under the root as well, without isolation.  @rbtrtr Could you share how you start the spark-shell?  Wondering if you have to pass in cdh HBase libs to Spark?
Caused by: java.lang.RuntimeException: hbase-default.xml file seems to be for an older version of HBase (<IP_ADDRESS>.1.7.0-551), this version is 2.4.9
        at org.apache.hudi.org.apache.hadoop.hbase.HBaseConfiguration.checkDefaultsVersion(HBaseConfiguration.java:74)

Thanks all, finally the issue was that cloudera provides the configuration for the parameter hbase.defaults.for.version.skip. After overwriting the parameter within cloudera manager it works.

@rbtrtr glad that you've sorted out!

@rbtrtr Glad to hear that.  To clarify, do you know if the cloudera manager writes the hbase.defaults.for.version.skip config to a separate hbase-site.xml file?  That could be the reason why it causes Hudi to throw the exception from HBase.

@yihua yes this parameter is placed in separate hbase-site.xml which is used by spark.

@yihua yes this parameter is placed in separate hbase-site.xml which is used by spark.

Thanks for the confirmation!  I'll also list this as a workaround in our FAQ.

@rbtrtr Thks!!!


Have a question about ng-alain

First, the framework is very good.
Second I have a question that Development documentation on running this command ng g module trade -- the routing that automatically help us create trade.module.ts and trade-routing.module.ts two ts files, and in terms of the forms folder below the routes, why, found that only forms.module.ts file and without forms-routing.module.ts file.  because the two files together? I am not very clear about that,thank you.
(文档写运行ng g module trade --routing会有两个文件，在demo里，发现只有xxxmodule.ts没有xxx-routing.module.ts文件)

thx.
just historical, i will base on the angular cli standars.


Improve PHPUnit assertions

Overview

Improve PHPUnit assertions.
Using the assertNull to assert expected is null.
Using the assertTrue to assert expected is true.
Using the assertInstanceOf to assert expected instance is same as result.
Using the assertCount to assert expected count is same as result.

@peter279k Second of all, thank you very much for contribution!
I absolutely agree with assertNull(), assertCount() and assertTrue(). I don't know why I didn't put it there myself ;D assertInstanceOf() in GuardedInvokerTest.php is also better, thank you.
But method directInstanceOf() I made specifically to check a "direct" instanceof, that is whether the class is exactly the same. So If you could but restore the original directInstanceOf(), I'd merge the PR right away :)
Thank you very much!


Feature: Add support for local songs

There may be times when users have their songs stored locally. In these cases, they may not need to use youtube-dl to retrieve the songs again. It would be a good idea to add support for locally stored songs.
Ideally, this feature should allow for both the specific path name of the song and a directory name for retrieving a set of random songs. Perhaps an intelligent fetch could automatically detect artist names as well?

I have a feeling that the schema is becoming overly complicated with features like these, and there needs to be some simplification for the 'music-*' element.

I have dreamed of this feature, but didn't know how to implement. :)

Okay, the fact that you've dreamed of this as well has got me excited! 😃
There's a messy way and a clean way to do this. I'm familiar with the messy way, so I'll start there and work towards creating something maintainable.

Sweet! (Context: I have like 11.8k music tracks on my external, and I would like to make radio shows for myself and friends with total control over the exact song chosen.)

After I see your "messy way", maybe I'll get a clue and figure out how to make things smoother. :-)


[Mod Info] 81 Tiles - Intercity buses don't work outside 25 tile area

Mod:
81 Tiles by BP, and all variants

Quistar is working on a complete new 81 Tiles implementation and is aware of the issue so hopefully that will solve the problem at some point in the future.

Stability:

Minor issues: it functions, but with some smaller issues (please elaborate below).

Cims won't board intercity buses at stops outside the central 25 tile area.
Other Information

Additional info: anything else you want to tell.

See comments on this workshop asset for more details: https://steamcommunity.com/sharedfiles/filedetails/?id=2049510825
Related TMPE guide: https://github.com/CitiesSkylinesMods/TMPE/wiki/Intercity-buses-not-working

This will be updated in the next catalog.

Quistar's version (in Extended Managers Library beta version) apparently fixes that issue (all the other older 81 Tiles mods are still affected).


Added Helm chart for RBD plugin

For my k8s cluster, I wrote a helm chart for the RBD deployment.  Here's that helm chart, I didn't know where to put it in the tree but it should be easily moved.

@sbezverk @rootfs

@kfox1111 thanks for the contribution， would you mind drafting a follow up doc PR? Just to show how install, load, and clean up the chart with some screenshot.

The chart looks good, some nits. I'll be mostly offline these days (I believe same for @sbezverk), please @gman0 for merge. Thanks!

@plnordquist If you could add documentation for this under docs/ please, otherwise this LGTM, thanks!

@plnordquist please add a followup document PR. thanks!


Fix CI style check errors

stylish-haskell in GitHub actions does not appear to use the config file in this repo.
This attempts to fix it by using v<IP_ADDRESS>.

@rvl the problem is a "recent" bump in stylish-haskell few weeks ago which isn't compatible with the configuration we use. I tried upgrading once (one day after the new release) but there were bugs only present in this new release with regards to the imports alignment. I haven't checked whether that bug had been fixed and resorted to slightly "fork" and modify the stylish-haskell script to accept a version number.
See it here:
https://github.com/input-output-hk/adrestia/blob/master/.haskell/stylish-haskell.sh
and used here:
https://github.com/input-output-hk/cardano-addresses/blob/master/.github/workflows/style.yml

Aha - that's it, thanks. Seems to work now.


Add gRPC documentation

Addresses: https://github.com/grpc/grpc-dotnet/issues/124
First draft of the gRPC documentation. I also started adding a tutorial and I'm considering adding a few more advanced scenarios such as sharing states between requests and duplex streaming calls.
Note that I did not add documentation for GrpcBindingOptions even though it's public since I'm not sure if any users will need to use it.
Review link: https://review.docs.microsoft.com/en-us/aspnet/core/grpc/index?view=aspnetcore-3.0&branch=pr-en-us-11163&tabs=visual-studio

:up::date:

Content is reviewed. @Rick-Anderson can you take a look at the UE?