[HUDI-4260] Change KEYGEN_CLASS_NAME without default value

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Thanks, i would suggest to keep it in order to be null-safe, or we should check all the code paths that use the option for null checking. For example, such validation now throws NPE:
conf.getString(FlinkOptions.KEYGEN_CLASS_NAME).equals(TimestampBasedAvroKeyGenerator.class.getName()

Thanks, i would suggest to keep it as it is in order to be null-safe, or we should check all the code paths that use the option for null checking. For example, such validation now throws NPE: conf.getString(FlinkOptions.KEYGEN_CLASS_NAME).equals(TimestampBasedAvroKeyGenerator.class.getName()

Done. Thanks for review


Service-side read without allowance

This compensates for the abstraction mismatch described in #2916.

Green.


[DSA]: Binary Search Techniques and Its Complexity Analysis

Description
Discuss about Binary Search Techniques and Its Complexity Analysis.
Domain
DSA
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign

@TechnoPhasePRO Are you working on this issue??


auto-merge functionality

There should be an optional setting at the hub level for whether to auto-merge PRs upon successful validation.

I suspect that we would want to implement this as part of something like hubValidations::validate_pr rather than as part of an action?

Some useful resources here: https://github.com/pascalgn/automerge-action
The above action uses label to decide what to automerge. We could make the validation workflow create an automerge label on the PR. If validation succeeds, it could then trigger the automerge workflow.
Depending on how exactly we implement this, it could be set via hub config or it could be a whole separate GitHub Action that needs to be explicitly added to a hub that is either linked to the validations workflow or just monitors for the automerge label which the validations workflow creates.
It would be good to spell out when exactly this will be activated. for example, if a PR contains changes to files not validated, should auto-merge still be performed if validations pass?

Useful resources supplied by @sbidari related to a sandbox hub where the functionality has been implemented

https://github.com/CDCgov/covidhub-sandbox
https://github.com/CDCgov/covidhub-sandbox/blob/main/.github/workflows/merge-validated-pr.yaml
https://github.com/CDCgov/covidhub-sandbox/blob/main/.github/workflows/validate-submissions.yaml

See also related issue https://github.com/hubverse-org/hubValidations/issues/9 about having auto-merge functionality apply to authorised submittersonly which is also implemented globally in the above example. Ideally we'd want each model_id directory to only be editable by the team members specified in the a model metadata property.


Hotspots misplaced in fullscreen mode in Safari on OS X

When switching to fullscreen mode in Safari (9.0.3) on OS X hotspots are misplaced and aspect ratio is skewed. Non fullscreen mode works fine.
Example: http://www.panoheld.de/portfolio-item/ferienwohnung-studio44/

I think this is fixed. Can you please test the fix?

It is! Working fine now.


Missing of raw data

Hi, thank you for your work! However, I encountered one problem when trying to reproduce your work: In your code of 'downloading the raw images', it required a file containing the url info of images used in the dataset, but this file is nowhere to be found in your GitHub or huggingface repositories. Do you mind uploading that file as well?

Due to license issues, I won't be able to upload the raw data directly. You can find the raw data here: https://cseweb.ucsd.edu/~jmcauley/datasets.html#amazon_reviews. It has a files says item metadata. I have also uploaded the file to get the necessary raw data from the files. Let me know if you have any other questions. Thanks!

Due to license issues, I won't be able to upload the raw data directly. You can find the raw data here: https://cseweb.ucsd.edu/~jmcauley/datasets.html#amazon_reviews. It has a files says item metadata. I have also uploaded the script to get the necessary raw data from the files. Let me know if you have any other questions. Thanks!

Hi, I did not find the scripts of 'getting necessary raw data' in this repository. Do you mind uploading them when you are available?

https://huggingface.co/datasets/mm-graph-org/mm-graph/tree/main/raw-data


Fix offset calculation.

ASN1_GENERALIZEDTIME and ASN1_UTCTIME may be specified using offsets.
To convert the offset time back to GMT, the offsets are supposed to be
subtracted, not added. e.g. 1759-0500 == 2359+0100 == 2259Z.

Checklist

[x] tests are added or updated

Description of change

Full unit tests are in PR #2632

Interesting... X509_cmp_time also adjusts the offset in the wrong direction.

Let me look.

Would it be possible to make a test for this?

I can look at a test; more complete tests are available in PR #2632, but that PR is on Hold.

At least for certificate validity, the time zones are in violation of RFC5280
and certificates should probably rejected if they use that.
Why do we support that at all?

@bernd-edlinger: I agree, but I can't answer that. If OpenSSL were used as a general ASN.1 library, the support would be required. That still doesn't fall under the scope of this change.

@bernd-edlinger

Interesting... X509_cmp_time also adjusts the offset in the wrong direction.

Actually, that one seems correct (based on unit tests).

So, the X509_cmp_time() returns -1 on equality? That seems odd, and throws off the test; so it now returns a sane value.

So, the X509_cmp_time() returns -1 on equality? That seems odd, and throws off the test; so it now returns a sane value.

I commented a moment ago that you change would throw this change to 1.2.0 because of semantics change.  I looked again and had a change of heart, as the semantics change is minimal and really does correct buggy behavior.  It helped to look at other code, which all expects X509_cmp_time to return 0 on equality, so...

X509_cmp_time returns 0 if there is a syntax error in the time value, that must stay IMHO.

@levitte I see code like this:
    i = X509_cmp_time(X509_CRL_get0_lastUpdate(crl), ptime);
    if (i == 0) {
        if (!notify)
            return 0;
        if (!verify_cb_crl(ctx, X509_V_ERR_ERROR_IN_CRL_LAST_UPDATE_FIELD))
            return 0;
    }

    if (i > 0) {
        if (!notify)
            return 0;
        if (!verify_cb_crl(ctx, X509_V_ERR_CRL_NOT_YET_VALID))
            return 0;
    }

However, when I look at OCSP_check_validity
I would think that there is a bug, because,
if thisupd contains 4 or more fractional digits,
it will likely be accepted by  ASN1_GENERALIZEDTIME_check
but rejected by X509_cmp_time, which returns 0 on error,
and that means that validity never ends.

The X509_cmp_time() function is undocumented. If we were to change the behavior, we'd need to document it. I've seen at least one comparison function that returns [-1, 0, 1], and anything else (specifically -2) is error. And We might want to consider that for this function in 1.2.0.
The problem I was trying to point out is that I can't write a unit test for equality for X509_cmp_time(). I also really didn't look at usage or history (it'd be interesting to know why this returns -1 for equality). I suppose I could add additional tests that have values 1 second more; which would be close enough.

Yes, as long as there is no documentation it is difficult to write a unit test.
Note, that #2668 also contains unit test for the new behaviour of X509_cmp_time,
but still no documentation.

I've restored the X509_cmp_time() behavior, and now explicitly check for +1 second to return 1. If the function is ever changed, these tests will fail.

So, if/when #2668 is committed, this will need to be updated.

Ping? This has been updated. Removed X509_cmp_time() updates/testing since @ekasper fixed that.

Ping?

Rebased onto latest master.

Ping? Rebased to latest master.

This is marked for master, 1.1.0 and 1.0.2 - but this won't apply to all branches. Is just the fix itself being backported? If so it might be better to split this into two commits.
The travis failures appear unrelated to this change.

The Travis build is unrelated, but seems to shadow this change.  time_offset_test.c isn't compiled in the failing builds.
Rebasing on a fresher master should clear the un-related errors.

When I rebase, it sounds as though you want this as two commits?  One with the fix, the other with the unit tests?

or make a separate PR with just the fix for the other branch(es)

Just made a 1.1.0 PR #3335

@levitte reconfirm?

Good to go, along with PR #3335?

I'll merge them both

Merged. 20ee2bf138323c6688b6e8d71d695cf2bd53f857


M1 Pro Wine segmentation fault

Describe the bug
We are working on an Electron application that we also want to build for Windows, but we don't have any Windows machines.
For this reason we want to build it via Docker using Wine, which should be supported by Electron.
However, we are unable to build the Dockerfile on M1 based machines due to a segmentation fault while running winecfg.
The Dockerfile is correct, as it builds properly on an older Intel based Macbook.
To Reproduce

Place the Dockerfile in a directory (See additional context section)
Enter that directory with your terminal
Build the Docker image with docker build .

Expected behavior
The Docker image builds successfully on ARM based CPU's
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.6.3
Commit: 10d8fa4cb507142066b51b2febbd367720a5732a (v1.6.3)
System info:
macOS: 14.5 (23F79)
CPU: arm64, 10 cores
CPU model: Apple M1 Pro
Model: MacBookPro18,1
Memory: 16 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-07-02T07-38-23.210460Z.zip

Screenshots and additional context (optional)
Dockerfile contents
FROM electronuserland/builder:wine-mono

RUN apt update
RUN apt install -y nodejs

# The segmentation fault occurs on this line
RUN rm -rf /root/.wine && winecfg
ENV WINEDLLOVERRIDES=""

Terminal output of docker build .

Unfortunately I don't think there are going to be any improvements for emulating 32-bit x86 programs on Apple Silicon. You may have better luck with wine64.

Sadly the same thing also happens when we use Wine64.
It also isn't caused by winecfg, as the segmentation fault seems to happen while it's creating the wine prefix.
Winecfg variant

Explorer variant

The updated Dockerfile
FROM electronuserland/builder:wine-mono

RUN apt update
RUN apt install -y nodejs

ENV WINEARCH="win64"
ENV WINEPREFIX="/root/.wine64"

RUN rm -rf /root/.wine

# The segmentation fault occurs on this line
RUN wine64 winecfg

ENV WINEDLLOVERRIDES=""

@kdrag0n Unfortunately after messing around a bit more we still haven't been able to get it to work, even the 64 bit version.


Intentional Architecture in Business

Intentional Architecture in Business
Purpose: Just as in IT, intentional business architecture would involve a set of purposeful statements, models, and decisions aimed at achieving a future state.
Dynamic Capabilities: Intentional architecture can align with the "sensing" capability by setting up frameworks to actively scan the market and internal environment for opportunities and threats.
Strategy Alignment: Intentional architecture can be tightly aligned with business strategy, ensuring that all architectural elements are designed to support strategic objectives.

Intentional Architecture
Purpose: Intentional architecture is a purposeful set of statements, models, and decisions that represent some future architectural state. It focuses on the essence of the system without being unduly influenced by the functionality it provides (Pages 64, 63).
Strategy Alignment: Lessons learned from the strategy field apply to intentional architecture too. The success of intentional architecture depends on the reliability of commitments (Pages 61).
Where to Include: /Chapter04-Frameworks-and-Models


Clarify list-item-spacing message

It seems there is no way to satisfy both rules for lists with long lines of am I mistaken?

list-item-spacing has to do with the fact that if a list-item spans multiple lines, it and the following list-item should be joined by an empty line.

Oh, now I get it. I thought it was complaining that I had an item that wasn't starting with a list indicator (* or -).
What do you think about changing the rules output to something like Missing new line after multiline list item.?

edit: Missing new line after list item.
Just figured that all list items should be loose.

Agreed, thanks for the suggestion.
I’ll combine this with some more stuff for the next major release, as it’s breaking.
P.S. Are you interested in PRing?

@wooorm I'll put it on my todo list. Can't promise anything, but I'll try!


Fix/fix issues

There is a transitiv dependency on frozenlist, which updated their package for python 3.11, but didn't add the package to pypi wheels. Thus, building on python >= 3.11 fails the whole docker building process (issue). Switching to python 3.10. Secondly, there are two more vulnerabilities due to cosign, which cannot be fixed on our side. Ignore listing them until cosign publishes a new version.
Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide
[x] Added tests (if necessary)
[x] Extended README/Documentation (if necessary)
[x] Adjusted versions of image and Helm chart in values.yaml and Chart.yaml (if necessary)

Codecov Report
Base: 96.81% // Head: 96.81% // No change to project coverage :thumbsup:

Coverage data is based on head (a7b100a) compared to base (da020e7).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #812   +/-   ##
========================================
  Coverage    96.81%   96.81%
========================================
  Files           22       22
  Lines         1226     1226
========================================
  Hits          1187     1187
  Misses          39       39

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


MAKE OPPORTUNITIES PAGE RESPONSIVE AND ENHANCE THE DESIGN

The Project page is not responsive

Can I have this issue ?

I want to work on this issue

@Hemang417 Assigning this issue to you. Happy Coding !


⚠ Bofrost.at has degraded performance

In cb3409b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1429 ms

Resolved: Bofrost.at performance has improved in 6c13539.


chore(main): release 1.7.1

:robot: I have created a release beep boop
1.7.1 (2024-10-18)
Miscellaneous Chores

release 1.7.1a0 (d44b6eb)
release 1.7.1a1 (6582a70)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/pythoninthegrass/gopappy/releases/tag/v1.7.1 :sunflower:


"isDismissEnabled" not available with @objc

Can this be prefixed? Otherwise this works fine in a test project.

Added; will be included in next release

Thanks!


Class 'Illuminate\Foundation\Application' not found

I am trying to integrate this package into my lum project but encounter this error "Class 'Illuminate\Foundation\Application' not found".

lumen?

  &&


NoMethodError after column name change

I renamed a column in my Attachments table from "url" to "url_or_code" via migration. Also changed the datatype string :limit => 255 to :limit => 1024 - this is MySQL, so the database fields are varchar.
My dashboards.rb contains:
ActiveAdmin.register Attachment

Everything had been fine, but after rake db:migrate, now I get:
NoMethodError in Admin/attachments#index
undefined method `url_or_code_contains' for #<MetaSearch::Searches::Attachment:0x000001059a7b70>
Extracted source (around line #1):
1: render renderer_for(:index)

Eliminating the register call for Attachment resolves the problem.
What am I missing - does ActiveAdmin cache column names someplace?

You might want to restart your server. :)

Nope, restarting the server has no effect. I've also upgraded to 0.4.2, and no joy (that caused its own problems).
I think the column rename is a red herring. Since this is project is not yet launched, I have the flexibility to rework older migrations - so I dropped the database, changed the create migration to use the right column name, and dropped the name change migration. Recreated the database, applied the new migrations, re-seeded the database ... and still have this problem.
Any ideas of where to look next?

Not sure why this was closed: the issue persists, despite updating to master, recreating the database from scratch (without column renames), and restarting the server.


Bail early in cache find command for WordPress < 4.9

Also fixes the command's description which was previously copied from another command.
Fixes #19.

I guess this has to be force-merged or combined with #23.

I force-merged #23.
Can you merge latest master to see if the tests now all pass?

@swissspidy Looks like PHP gets the namespace wrong in the broken test. You should explicitly state global namespace for the function:
$x = \_wp_oembed_get_object( ... )

@schlessera Doesn't look like it: https://travis-ci.org/wp-cli/embed-command/jobs/320294449
I guess that function wasn't readily available in older versions of WordPress and the necessary file first needs to be included.

Here's the relevant changeset: https://core.trac.wordpress.org/changeset/38361

From that changeset, something like this should help:
if ( ! function_exists( '_wp_oembed_get_object' ) {
   require_once( ABSPATH . WPINC . '/class-oembed.php' );
}

Note you can test different WP versions locally with eg WP_VERSION=3.7.11 vendor/bin/behat.

Taking the liberty here to make the tests compat with older WP versions and to add a polyfill for WP_oEmbed::get_provider().


Poll valid PredictionParams

Different models have different prediction parameters (e.g., some look for an image, others look for text input).
It would be useful if the API had an endpoint we could probe to get the input a particular model expects, without running inference.
We could then use this to create the PredictionParams for a given model.

Hi @Wheest. That information is available in the schema of the model version you're running.
>>> import replicate
>>> v = replicate.models.get("stability-ai/stable-diffusion-3").latest_version
>>> list(v.openapi_schema["components"]["schemas"]["Input"]["properties"].keys())
['seed', 'image', 'prompt', 'num_outputs', 'aspect_ratio', 'output_format', 'guidance_scale', 'output_quality', 'negative_prompt', 'prompt_strength', 'disable_safety_checker']


in login page adding sign up with google option

In login portal adding option of login with google
please assign this to me under gssoc ext and hacktoberfest 2024

it's a python based package, we didn't provide user interface yet


syncer tool

with having executors i think we can implement basic syncing tool to copy code over remote machines now

JS-NG> xmonader = j.clients.sshkey.new("xmonader")
JS-NG> xmonader.private_key_path = "/root/.ssh/id_rsa"
JS-NG> localclient = j.clients.sshclient.get("xmonader")
JS-NG> localclient.host = "localhost"
JS-NG> localclient.sshkey = "xmonader"
JS-NG> localclient.sshclient.run("hostname")
Bishoy
(0, 'Bishoy\n', '')


Merged main branch into dev-1.25

Merge main branch into dev-1.25 to keep in sync, part of the Docs activities in the release process.
/cc @kcmartin @cathchu @didicodes @sethmccombs @reylejano

Redeployed netlify preview which was a success
https://deploy-preview-36049--kubernetes-io-vnext-staging.netlify.app/


Implement Dataset from text

Implement Dataset.from_text.
Analogue to #1943, #1946.

I am wondering why only one test of "keep_in_memory=True" fails, when there are many other tests that test the same and it happens only in pyarrow_1...


audreybot

Adding my bot Botflux (audrey) to list of bots.
This is my first GitHub pull request, let me know if I did anything suboptimally!

Welcome to our newest bot!


can somene explain how to use chatterbot voice

can somene explain how to use chatterbot voice

Hi @edwinjose900,
Would an example be helpful?
https://github.com/gunthercox/chatterbot-voice/blob/master/examples/example.py

The chatterbot-voice module is designed to provide text to speech and speech synthesis capabilities for ChatterBot. It does this by wrapping the espeak and [SpeechRecognition](https://github.com/Uberi/speech_recognition) libraries.


UB-1405: unmount flow should not fail if discovery fails

In this PR , the following idempotent issue with Unmount flow is fixed:
if a discovery of a volume fails, no need to fail the Unmount command, just need to skip the UnmountDeviceFlow.
more info:
if volume's mpath NOT discovered (because the device does not exist at all in multipath -ll) then we should skip UnmountDeviceFlow instead of failing (reminder - the UnmountDeviceFlow is the flow that does umount the device and cleanup of the mpath device) .
Why its idempotent? - because there is a chance that unmount flow will fail  at the end of UnmountDeviceFlow after it unmounts and cleans the mpath device. So to be idempotent the unmount needs to handle unmount request while the mpath device was already cleaned.

This change is

Coverage increased (+0.5%) to 57.133% when pulling 3e45a8a5d1e198b24de24884a094b7a6d11dedfa on fix/UB-1405_unmount_should_not_fail_if_mpath_isnot_discovered into 0246249a321902ddc1901f732d683fe5819b37ec on dev.

please add the internal ticket number that this PR refer to.


fix(region): update huawei cloud stack update credential

What this PR does / why we need it:
*update huawei cloud stack update credential fix
Does this PR need to be backport to the previous release branch?:
/area region
/cc @zexi @ioito

/lgtm


fix: react node typings inaccuracy

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] commit message follows commit guidelines

Description of change

React.FC (which is used by all the shapes) got children props removed since React v18. Adding children in common Shape props shouldn't break it for past React versions but would add compatibility with the latest.
(non-ts) Examples have Rect as a container with no style property. Rect does have non-optional (i.e. required) style which means it should've been used with style containing at least an empty object like <Rect style={{}}>...</Rect>. Since all RectStyle properties are optional there is no reason to keep style non-optional. Same thing with Text.
(non-ts) Examples have draggable="true" which brings a bit of confusion since the .ts type for draggable is boolean.
justifyContent typo.

Codesandbox (see type hint errors)

Thanks for your PR!


Added the ideas generated from our last meeting.

Resolves #3

Should be fixed now.

Looks good!

Could you approve the PR?

Yup! Just gotta figure out how to fix merge conflicts now that the file's been renamed in master.

Oh just fixed it!


refactor(sdk): Remove unused interface, update method comments

Codecov Report
Base: 91.02% // Head: 91.07% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (8f710e6) compared to base (12427c6).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
+ Coverage   91.02%   91.07%   +0.05%
==========================================
  Files          50       51       +1
  Lines        2017     2029      +12
==========================================
+ Hits         1836     1848      +12
  Misses        127      127
  Partials       54       54

Impacted Files
Coverage Δ

cmd/wallet-sdk-gomobile/api/activitylogger.go
100.00% <ø> (ø)

cmd/wallet-sdk-gomobile/openid4ci/interaction.go
73.07% <ø> (ø)

pkg/openid4ci/openid4ci.go
87.03% <ø> (ø)

cmd/wallet-sdk-gomobile/api/credentials.go
93.75% <100.00%> (+3.75%)
:arrow_up:

cmd/wallet-sdk-gomobile/api/diddocresolution.go
74.07% <100.00%> (ø)

cmd/wallet-sdk-gomobile/api/stringarray.go
100.00% <100.00%> (ø)

cmd/wallet-sdk-gomobile/did/creator.go
83.33% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Project Submission] - Community Garden

Dear community,
I followed the instructions provided to add my project to the IPFS Community Showcase.
Looking forward to your thoughts 🙂

Thank you for your submission! I've made 1 minor edit to remove some unused fields, and I will merge now :).


global exception handler

It would be usefull to be able to define an exception handler, that would be always called when an exception occures, regardless of the origin.
It shouldn't be possible for the application to crash without first calling this handler. Currently the handler that is defined via the \Zend\Log\Logger::registerExceptionHandler does not work properly.
For example I have in module/Application/Module.php:
class Module
{
    public function onBootstrap(MvcEvent $e)
    {
    ...
    // logger
    $logger = new \Zend\Log\Logger();
    $writer = new \Zend\Log\Writer\Syslog();
    $logger->addWriter($writer, 0);
    \Zend\Log\Logger::registerErrorHandler($logger);
    \Zend\Log\Logger::registerExceptionHandler($logger);
    ...

but this registered handler is not used for all occuring exceptions. For example, if an exception is thrown in a controller, than the exception is not logged to syslog.

Not everybody has a use case for this, and there are known issues with what you're attempting.
As you note, exceptions thrown in controllers... are actually caught. This is done so we can present 500, 404, and other HTTP error statuses. You can always add listeners to the dispatch.error event if you also want to log that information. (This is true of the render event as well.)
For exceptions thrown elsewhere — bootstrap, sending the response — you will need to register your own exception handler. But that's a detail for your own application, not for the framework.


Transfer Learning- Changing learning rate after loading the model

Description
Is it possible to change the learning rate when a model is loaded? I was not able to find any related documentation. For example, if I train a transformer-based model on thousands of data and want to fine-tune it (retrain) for a smaller, new series, using the same training configuration as the source model (e.g., learning rate) does not make sense.
Link
No response

Hey!
So, you can do this like so:
nf = NeuralForecast(models=models, freq='M')
nf.fit(df=Y_df)
nf.save()
loaded_model = nf.load()

Then, we can fit on another dataset with
loaded_model.models[0].learning_rate = 1e-4
loaded_model.fit(new_df)

I hope this helps!


Add blst_core_verify_pk_in_g1 binding

Hey blst is missing some go bindings but this one is quite significant for golang-managed threads
Ts there a chance to get it merged ? :)

@dot-asm  If I'm correct - you're talking about the usecase when you calculate bigger payload and would like to distribute this over more threads.
What this particular PR and binding is for - is to process massive amount of smaller and unrelated payloads.
In this case we very much like to have this process being ran on one and only one thread. Because other hundreds of payloads are being processed on different system threads on the same time.

you're talking about the usecase when you calculate bigger payload

No, even single-input verification benefits from multi-threading. Significantly.

we very much like

Well, it's not really about what any particular party likes, but about what's best overall;-) In the context one can reformulate it as follows. Would using a pair of threads for single-input verification be detrimental in the case you describe? The amount of computational work is the same no matter how you slice it. Yes, using a pair of threads for single-input verifications initiated from multiple threads would result in more context switches and a slightly larger memory footprint. But the computational part totally dominates the execution time, and the additional overhead is negligible in comparison. While on the other hand the benefit for another use case would be significant, ~40%.

ok so let me try to understand the changes you were talking about. That's the function right?
https://github.com/supranational/blst/blob/master/bindings/go/blst.go#L555-L562
Imagine I need to verify 10k simultanous payloads, in "the same time".
I can run 100 goroutines that would simultanously process 100 payloads.
Each payload is exactly one msg (a slice) how come the loop of separate goroutines helps here?
The benefit in that case would need to come from asynchronous calculations of blst_aggregated_in_g2 before the main gets returned?

Each payload is exactly one msg (a slice) how come the loop of separate goroutines helps here?

For a single input there won't be a loop of goroutines. There will be one additional goroutine called. And if there is a CPU available, the subroutine as a whole will finish significantly sooner. And if there are no free CPUs, it will be a little bit slower because of synchronization overhead.

The benefit in that case would need to come from asynchronous calculations of blst_aggregated_in_g2 before the main gets returned?

Well, what's "main" here? But either way, yes, the gain is that blst_aggregated_in_g2 gets a chance to execute in parallel with blst_pairing_commit. The two are the second most expensive operations. (Just in case for reference, the most expensive operation is the final exponentiation.)

the benefit for another use case would be significant, ~40%.

Oops, this is an overstatement. I've simply calculated the ratio between (2*miller_loop + final_exp) and (miller_loop + final_exp), but one thread has more things to do, namely hashing the message. So the gain would be less than 40%, but still significant.

So, you still assume that you have CPU available :)
If I had to process more msg that my CPU can handle in bursts, so I'd need to queue them up, there is no gain for me in adding additional scheduler overhead, introduce additional context switching, doing additional allocations, and time to transfer data though CGO. Neither we need to run go sched that is not really helping in my scenario.
As we're talking about different language bindings, it would be great to leave the flow mechanics to the user of this lib.
To be honest I really didn't thought it would be a problem to just enable this particular one, as there are no changes to your code.
If you really dislike enabling this whole thing, maybe then we can create it slightly differently:
export blst_aggregated_in_g{1,2) so people can decide to run it on different thread
Create and export c function like PAIRING_Aggregate_PK_in_G1 just without https://github.com/supranational/blst/blob/master/src/aggregate.c/#L325-L332  (what as I understand is blst_aggregated_in_g , or is that a commit?) and then, the third one that has both PAIRING_Commit and PAIRING_FinalVerify. Preferably not sharing the underlying c context, but run as separate beings.
This way people can really independently decide how to run, and what's better in their case.
As what you see in my PR  - if you see entire operation - performs better in my particular case, with few times less allocations.

So, you still assume that you have CPU available :)

Again, the assertion is that if there is one available, it would be a significant win, and if there is none, then it's virtually no loss. Even in the scenario you describe. If you can refute [the second part of] the assertion, do it with data. [And probably with better rationale for why would you start 100 goroutines. If rationale is "because it's cheap," then why 200 isn't?]
As for why not this-n-that. It's not like we're not open for suggestions, but expectation is that it's more concrete. As in "this is a use case, and this is a bottleneck." The objective is an overall experience, not answering speculative questions.


Fixes to how output items are displayed

Codecov Report

Merging #4945 (af18c69) into main (883e228) will decrease coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff           @@
##            main   #4945    +/-   ##
======================================
- Coverage     76%     75%    -1%
======================================
  Files        403     404     +1
  Lines      26650   26687    +37
  Branches    3859    3861     +2
======================================
- Hits       20351   20215   -136
- Misses      4690    4884   +194
+ Partials    1609    1588    -21

Impacted Files
Coverage Δ

src/client/datascience/notebook/helpers/helpers.ts
76% <ø> (ø)

...lient/datascience/jupyter/kernels/cellExecution.ts
78% <100%> (+<1%)
:arrow_up:

...t/datascience/notebook/helpers/executionHelpers.ts
67% <100%> (ø)

...ience/variablesView/variableViewMessageListener.ts
22% <0%> (-78%)
:arrow_down:

...ent/common/application/webviewViews/webviewView.ts
14% <0%> (-72%)
:arrow_down:

...c/client/datascience/variablesView/variableView.ts
18% <0%> (-58%)
:arrow_down:

src/client/datascience/webviews/webviewViewHost.ts
19% <0%> (-53%)
:arrow_down:

src/client/datascience/webviewExtensibility.ts
32% <0%> (-40%)
:arrow_down:

...t/datascience/jupyter/jupyterInvalidKernelError.ts
66% <0%> (-34%)
:arrow_down:

...on/application/webviewViews/webviewViewProvider.ts
80% <0%> (-20%)
:arrow_down:

... and 35 more


Release a new major version: 3.0.0

Coverage remained the same at 98.182% when pulling 0b409aa601d30593df9051b0079f9ef07c025448 on biniambekele:patch-1 into fd0849b712883d46a113de3fb8aefbb4ef3a4236 on Points:master.


ChickenEgg: identity_id is missing from Core 'http' access log

The Logger middleware is registered before the JWT middleware. That means the Logger does not have the context where the Token is stored and can not log identity_id.
Moving JWT middleware before Logger would cause requests with failing Token validation to not be logged.
The only idea I can think of is to split JWT in two, one Context creator and one validator. Then have Logger in between. Requires rewrite of the built in JWT middleware, and possible miss match of the assumption that the token in Context is actually valid.

Yes, we might have to split JWT into two so writing our own middleware. I'll ping goa developers in their slack channel

Closing it! It was addressed by #1555.


Add tomcat metrics crawler plugin

This PR supports to gather tomcat server metrics.
The detail information is described at issue #196
This plugin gathers server status metric of tomcat as follows.
application	"tomcat_jvm"	{"free":"76526408","total":"120061952","max":"1390936064"}
application	"tomcat_memory"	{"name":"PS Eden Space","type":"Heap memory",...,"used":"30755648"}
application	"tomcat_memory"	{"name":"PS Old Gen","type":"Heap memory",...,"used":"9136520"}

Codecov Report

Merging #244 into master will increase coverage by 0.24%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #244      +/-   ##
==========================================
+ Coverage    90.1%   90.34%   +0.24%
==========================================
  Files          94       98       +4
  Lines        3407     3492      +85
==========================================
+ Hits         3070     3155      +85
  Misses        337      337

Impacted Files
Coverage Δ

...wler/plugins/applications/apache/apache_crawler.py
100% <ø> (ø)
:white_check_mark:

...plugins/applications/tomcat/tomcat_host_crawler.py
100% <100%> (ø)

...wler/plugins/applications/tomcat/tomcat_crawler.py
100% <100%> (ø)

...ns/applications/tomcat/tomcat_container_crawler.py
100% <100%> (ø)

crawler/plugins/applications/tomcat/feature.py
100% <100%> (ø)

Continue to review full report at Codecov.

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

Looks good, thank you!


Cookies need to be set as "secure" on https websites

If the site is being served on the HTTPS scheme, the Secure flag needs to be set on the WebEngineSession cookie, otherwise browsers will start to treat it as a cross-domain cookie.

All sessions will use the secure cookie by default now, and luckily all major browsers consider the localhost domain as a secure endpoint - http://localhost on any port will be considered secure.


nixos/odoo: apply suggestions from review

Motivation for this change
ref https://github.com/NixOS/nixpkgs/pull/144172 cc @aanderse
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

:heart:


Add faculty of computer and data science, Alexandria university

alexu.txt

@minaKiroloos The request does not have any file. I have to close it without merging.


smart-plug-v2: add id for power button

Setting an ID for the power button allows its behavior to be extended or modified by configuration that uses this package, for example:
packages:
  Athom_Technology.Smart_Plug_V2: github://athom-tech/athom-configs/athom-smart-plug-v2.yaml

binary_sensor:
  - id: !extend power_button
    # Additional settings here

If your wanting to get this added in, I would suggest that you also add this into the:
v1 Plug: https://github.com/athom-tech/athom-configs/blob/main/athom-smart-plug.yaml
v3 Plug: https://github.com/athom-tech/esp32-configs/blob/main/athom-smart-plug.yaml
to ensure consistency of the functionality / code between ALL versions of the plug. Previously this was lacking, but in the last few months was just fixed up.

That's reasonable. Added the same change to v1 here, and v3 in https://github.com/athom-tech/esp32-configs/pull/27.


With prawn 2.2.0. Error uninitialized constant ActionView::CompiledTemplates::ThinReports

https://github.com/thinreports/thinreports-generator/issues/84#issuecomment-304413617
It is said that.

related: https://github.com/takeshinoda/thinreports-rails/pull/13


[Rails] Artworks

Visual Spec:

Also be sure to take all edge cases into account that are outlined in the design spec https://github.com/artsy/eigen/issues/1441.

This still needs to switch between 4 and 3 columns on iPad.

Closing - https://github.com/artsy/emission/pull/292


(PUP-9389) Add staging_location parameter to File

Description of Changes
When Puppet creates a new file it creates it in a temporary location. This location is currently the location the file will exist in plus a unique name, eg /etc/foo123fsd for /etc/foo. In cases where a user wants to first render a file to a temporary staging location, perform some validation commands and then move the file to the final location.
This avoids issues where, for example, an application reads all the files in a directory when performing a task (sudo) or an application reloads configs automatically from a directory.
Open Questions
It is unclear to me if I need to do anything different in the case where Puppet is running on Windows and we want to render first to a temporary staging location.
(edit) Oh nice, the tests run on Windows and looks to work.
Jira
https://tickets.puppetlabs.com/browse/PUP-9389

Can one of the admins verify this patch?

Waiting for CLA signature by @bkochendorfer
@bkochendorfer - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/
Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

@bkochendorfer , we fixed this flakey test on master, if you rebase it should pass.  Thanks!

One test seems to have had a segfault. Is there a way to kick the build again? Or is it related to something I changed?
bundle : C:/projects/puppet/lib/puppet/util/windows/principal.rb:115: [BUG] Segmentation fault
At line:14 char:1
+ bundle exec rake parallel:spec[2]
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (C:/projects/pup...mentation fault:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

ruby 2.5.3p105 (2018-10-18 revision 65156) [x64-mingw32]

CLA signed by all contributors.

The segfault on Windows is nothing related to any of your changes. It's just something that happens occasionally on AppVeyor. Not entirely sure why.

@bkochendorfer Looks good to me. Do you want a chance to squash those last two cleanup commits into the one that creates the test before I merge things?

Thank you @jhelwig, squished!

@bkochendorfer Thanks for all your patience, and work on this!


Project Dependencies and dotNet Integration?

Just to throw this idea out there; would it be possible to have support for integration between node.js, edge.js and dotNet libraries?
Use case would be to have a node.js project that is dependent on a .Net project. In my case, I want the dotNet side for CPU heavy tasks and feeding the results back into the node.js server. Edge.js lets me so this, but more native support for doing this in visual studio (plug debugging) would be great.

Intriguing suggestion, but I don't think EdgeJS is popular enough that we would consider adding explicit support for it anytime soon.

Fair enough; in the end I actually abandoned node.js in favor of Owin (Nowin implementation) purely because I need direct access to nuget libraries that do heavily lifting.


ndt7/s/r/file.go: no colons in file names

Closes #64

This change is

Pull Request Test Coverage Report for Build 462

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.34%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ndt7/server/results/file.go
0
1
0.0%

Totals

Change from base Build 458:
0.0%

Covered Lines:
778

Relevant Lines:
1248

💛  - Coveralls


[JA CVVC Phonemizer] Multiple bug fixes + added function

Fixed "double VC" bug.
Add "dz" + "z" substitute.
Revert を back to plainVowels (the large majority of CVVC banks treat this as a vowel).
Voice color fix (a proper fallback system for non-vowels and non-initials was not defined, which resulted in the error).
VV and crossfade vowel tests are now switched, meaning that VVs get priority.

TODO for future PR: Create を fallback? (i.e., if there's no VV note, fallback to VC note)

Looks like this may be good to go IMO


Error adding AddAutoMapper in Startup.cs

I get error trying to add AutoMapper to my Startup.cs file in a dotNET Core project (a simple tutorial):

installed package from VS Code terminal:
dotnet add package AutoMapper
(this added:

to my .csproj file).
Added line inside ConfigureServices method:
services.AddAutoMapper(AppDomain.CurrentDomain.GetAssemblies());
added "using" namespace in my Startup.cs file:
using AutoMapper;

The error I get is:
'IServiceCollection' does not contain a definition for 'AddAutoMapper' and no accessible extension method 'AddAutoMapper' accepting a first argument of type 'IServiceCollection' could be found (are you missing a using directive or an assembly reference?)
dotnet --version
= 3.1.402
What am I missing, please?

Ok, it seems package "AutoMapper.Extensions.Microsoft.DependencyInjection" is required as well and I didn't realize it.
Running next command fixes the issue:
dotnet add package AutoMapper.Extensions.Microsoft.DependencyInjection

@all-contributors please add @miki-nis for bug


deps(ci): remove actions-rs

Replaces all the actions-rs actions with standard rustup/cargo commands now that actions-rs is no longer maintained.

Re-minimal toolchain with the missing components: I wonder if it works because the toolchain is being cached from previous runs?

Re-minimal toolchain with the missing components: I wonder if it works because the toolchain is being cached from previous runs?

This is from some testing I did:
Run rustup toolchain install stable --profile minimal
info: syncing channel updates for 'stable-x86_64-unknown-linux-gnu'
info: latest update on 2023-02-09, rust version 1.67.1 (d5a82bbd2 2023-02-07)
info: downloading component 'clippy'
info: downloading component 'rustfmt'
info: downloading component 'cargo'
info: downloading component 'rust-std'
info: downloading component 'rustc'
info: removing previous version of component 'clippy'
info: removing previous version of component 'rustfmt'
info: removing previous version of component 'cargo'
info: removing previous version of component 'rust-std'
info: removing previous version of component 'rustc'
info: installing component 'clippy'
info: installing component 'rustfmt'
info: installing component 'cargo'
info: installing component 'rust-std'
info: installing component 'rustc'

It's definitely cached, but the minimal profile also seems to download rustfmt and clippy, not quite sure what to make of that.

Yeah, only took a quick look at the rustup repo but couldn't find anything conclusive. This is probably fine for now but if it starts breaking we'll know why.

On second thought, let's maybe specify the component flag: https://rust-lang.github.io/rustup/concepts/components.html


luarocks: explicitly set 'configurePlatforms = [ ];'

Without the change build with config.configurePlatformsByDefault = true
fails as:
Error: Unknown flag: --build=x86_64-unknown-linux-gnu

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

I dont know about this option I suspect it's about cross-platform compilation (EDIT: I see the issue you linked).
Would it be interesting to modify upstream to accept the flag instead (doesn't seem like it would) ? having to add configurePlatforms = [ ]; to this many packages kinda feels off.

It's a good question.
Generally presence of ./configure implies autoconf-compatible script: options like --prefix= and --sysconfdir= (present in luarocks) and --build= / --host= / --target= (missing in luarocks). I would say it's relatively rare not to support these options, but it does happen (ffmpeg or perl are good examples). Thus sprinkling configurePlatforms = [ ]; around does not seem to be widespread (so far).
Adding --build= / --host= / --target= should be fine. My only mild worry is that these options might be complete no-ops: luarocks seems to only install .lua and .shell files (but also runs a lua interpreter).

There has to be a rock that compiles a native library I guess? We can see how this looks like in the wild, maybe

some rockspecs build C libraries via make or cmake e.g. https://luarocks.org/modules/xpol/rapidjson/0.7.1-1 so if luarocks could forward these flags, it may be useful ?
Note that for the cmake libraries, it's best to build them via nix rather than via luarocks and then tell the lua library to use the nix library (it is recommended by the luarocks author too)

But yeah, would be nice to see if stuff like rapidjson can be cross-compiled with Nixpkgs


Save State Problem

Can you elaborate how to save state using Event.kt and EventObserver.kt ?
How to passing id to all views?

@furkanaskin fyi

Hi @guizot ,
You can read this blog : Medium
Also you can check this discussion : EventObserver.kt

@guizot , also you shouldn't give id to your all views. For example if you give id to your scrollView, it will save/restore own view state but if you not give an id to scrollView, it will lost their state on replace. With this situation you must give id to your scrollviews, recyclerviews, root views etc.


py pkg decorator/ipython # attribute 'decorator."5.0.7"' at undefined position - attribute '5.0.7' missing, at /nix/store/c7504ppqk1lfa57rnddwh9qj8slmxri6-nix-pypi-fetcher/default.nix:23:17

latest pypiDataRev leads to failing of decorator/ipython:
pypiDataRev = "01e45c3f9b64a41653bb5d0d0d718be4e7fde753";  pypiDataSha256 = "1xgvzvhixprggnr8zz0vc66i37pbs5p9rqgbb96z43x9fqbi1pqx"; # 2021-04-16T20:18:14Z
error: while evaluating the attribute 'buildInputs' of the derivation 'shell' at /home/ae/ae_share_VM/z_DS_setup/toolbox/nixMachNixEnv/testingMachNix/Nix_buildEnv__JL_.nix:262:13:
while evaluating the attribute 'passAsFile' of the derivation 'bEnvJL' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/build-support/trivial-builders.nix:7:7:
while evaluating the attribute 'passAsFile' of the derivation 'python3-3.8.5-env' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/build-support/trivial-builders.nix:7:7:
while evaluating 'requiredPythonModules' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/top-level/python-packages.nix:64:27, called from /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/development/interpreters/python/wrapper.nix:15:13:
while evaluating 'requiredPythonModules' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/top-level/python-packages.nix:64:27, called from /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/top-level/python-packages.nix:82:33:
while evaluating anonymous function at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/lib/lists.nix:643:24, called from /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/top-level/python-packages.nix:66:6:
while evaluating the attribute 'out.outPath' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/lib/customisation.nix:156:13:
while evaluating the attribute 'nativeBuildInputs' of the derivation 'python3.8-decorator-5.0.7' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/development/interpreters/python/mk-python-derivation.nix:108:5:
while evaluating 'chooseDevOutputs' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/lib/attrsets.nix:475:22, called from undefined position:
while evaluating 'optionals' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/lib/lists.nix:270:5, called from /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/development/interpreters/python/mk-python-derivation.nix:120:10:
while evaluating 'hasSuffix' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/lib/strings.nix:212:5, called from /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/development/interpreters/python/mk-python-derivation.nix:120:25:
while evaluating the attribute 'src.name' at /nix/store/xnanigwr38r59b679nxcrp8sjy9g8ij9-nixos-20.09.2344.a3a3dda3bac/nixos/pkgs/build-support/fetchurl/default.nix:122:3:
while evaluating the attribute 'decorator."5.0.7"' at undefined position:
attribute '5.0.7' missing, at /nix/store/c7504ppqk1lfa57rnddwh9qj8slmxri6-nix-pypi-fetcher/default.nix:23:17

Hey, thanks for that. I'm currently reworking large parts of the crawler pipelines. The goal is to to simplify some parts, get rid of the sql DB, add support for python 3.9/3.10, and have all the crawlers running as github actions, so people can easily check the logs and fork/modify the pipelines.
After this is done, errors like this should hopefully disappear. Let's check again after this is done

Some of the old crawlers was stuck and out of sync. It should be fixed with the newest pypi-deps-db.

works again, thanks.


Support provisioning with replication enabled through configuration file.

Allow for initialisation using MONGO_INITDB_ROOT_USERNAME, MONGO_INITDB_ROOT_PASSWORD and replication enabled.
Currently, when trying to run a mongo instance with INITDB env vars and replication enabled through a config file it throws the following error. It works fine when run with --replSet.
{"t":{"$date":"2021-02-18T18:02:00.753+00:00"},"s":"I",  "c":"NETWORK",  "id":51800,   "ctx":"conn2","msg":"client metadata","attr":{"remote":"<IP_ADDRESS>:52968","client":"conn2","doc":{"application":{"name":"MongoDB Shell"},"driver":{"name":"MongoDB Internal Client","version":"4.4.4"},"os":{"type":"Linux","name":"Ubuntu","architecture":"x86_64","version":"18.04"}}}} uncaught exception: Error: couldn't add user: not master :

Nice, thanks for the PR. Should it be limited to just replication.replSetName to mirror the --replSet removal? (https://docs.mongodb.com/manual/reference/configuration-options/#replication-options)
Does it need to be limited to only removing when --replSet is removed?
https://github.com/docker-library/mongo/blob/56e1d6804eac98556737d1e3e1090ae23231c341/docker-entrypoint.sh#L271-L273

@yosifkit I believe that removing entire .replication node is on par with the current behaviour where the entirety of .net is also removed. None of these remaining options affect behaviour of mongod when replication is disabled.

Thanks for confirming!

Doh, sorry for letting this slip through the cracks!!  Thank you for the contribution :bow:


gnome3.nautilus: add thumbnailers

Motivation for this change
Nautilus, resp. gnome-desktop, scans thumbnailers directories under the paths in XDG_DATA_DIRS. gdk-pixbuf was not, for some reason, listed in the variable, therefore Nautilus did not generate image thumbnails.
I also add librsvg to the variable so that SVG files can be rendered. It does not work at the moment, though, because of incorrect path to the renderer.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

cc @lethalman @teh

Works for me! Thank you!


Add Google's Model card toolkit support to collect the model metrics and build the model documentation automatically

Is your feature request related to a problem? Please describe.
Using this Toolkit will help users of zenml align with best practices in the industry for model reporting. This will also help with increased transparency in the workflow and being more accountable towards the stakeholders in the project and the users.
Describe the solution you'd like
A suggested pseudo code from @hamzamaiot  could look like this:
# pseudo-code
model_card = mct.scaffold_assets()
# fill in ur model card details
pipeline.add_model_card(model_card)

Describe alternatives you've considered
Alternatively one could use Dalex as a model agnostic tool to build the model card self. Here is an example in a jupyter notebook. Otherwise, can you view it here

Update Jan 15: This is slightly blocked by API design discussions regarding custom Steps. Will keep open and align with a PR soon

Update Jan 15: This is slightly blocked by API design discussions regarding custom Steps. Will keep open and align with a PR soon

Update 29 Jan: API design is complete with 0.2.0 but the model-card-toolkit conflicts with some core dependencies of ZenML, including ml-metadata, tfx etc. Looking at the repo, it looks like it is still WIP and the first GitHub release was made a few days ago. Hopefully it will have looser dependencies as things move forward, then we can look at this again

Update 29 Jan: API design is complete with 0.2.0 but the model-card-toolkit conflicts with some core dependencies of ZenML, including ml-metadata, tfx etc. Looking at the repo, it looks like it is still WIP and the first GitHub release was made a few days ago. Hopefully it will have looser dependencies as things move forward, then we can look at this again


Rename the project to Shopify node

Either from a "SEO" inside opensource community and for other reasons "obvious" to someone, I think it'd be wise to rename the project to be sth like Shopify node or alike.
@Alua-Kinzhebayeva @lpinca @carlosvillu @davidgf thoughts?

It's hard to find a good available name on npm:

https://www.npmjs.com/search?q=shopify-node
https://www.npmjs.com/search?q=node-shopify

unless we use a scoped name.

If we go for a scoped name I would just use "shopify".

We definitely want to get rid of "nodify" "node" and keep it cleaner with just: Shopify.
Either scopes or I can try to approach: https://www.npmjs.com/package/shopify to see if they'd be ok to release their name

:+1:

Here are some available names that might be good:

shoppify
shopify-api-node
shopify.node
shopify-nodejs

done!
https://github.com/microapps/shopify-api-node


Can't require Bankai/http

This a 🐛 bug report

Expected Behavior
This should work (as the readme say)
var bankaiHttp = require('bankai/http')

Current Behavior
Error: Cannot find module 'bankai/http'
    at Function.Module._resolveFilename (module.js:557:15)
    at Function.Module._load (module.js:484:25)
    at Module.require (module.js:606:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/home/yerko/.nvm/versions/node/v9.7.1/lib/node_modules/mtalk/bin.js:11:14)
    at Module._compile (module.js:662:30)
    at Object.Module._extensions..js (module.js:673:10)
    at Module.load (module.js:575:32)
    at tryModuleLoad (module.js:515:12)
    at Function.Module._load (module.js:507:3)

Possible Solution
Add a files section to package.json.
I think that adding index.js and bin.js would be enough

Wuuh. It's in the package though<EMAIL_ADDRESS>

Last time I checked was still an issue. Let me explain my context.
I have a package here (https://github.com/YerkoPalma/mtalk) where I have bankai as a dependency and where I require bankai/http, and it works fine. But when I install that package (npm i -g mtalk) I get the error posted. Migth be my error, just don't know why (I guess that if you install the package you'll see the same error)

hmm no that works fine for me :confounded:


Loosen rack requirement to allow testing with rack master branch

This loosens the rack version requirement to allow testing with the rack master branch as currently being required by rails 5. It also adds to travis as an allowable failure, and fixes/ignores some rubocop warnings

+1, the current strict requirement on rack ~> 1.2 means that not only oauth2 but also omniauth are incompatible with rails 5.

Why hasn't this been merged?


[MINOR] Increase MLContext test coverage

Create MLContext tests to test previously untested methods.
Update MLContext and MLContextConversionUtil to avoid NPEs.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2032/

@gweidner Thanks for reviewing!


Is there a FastAPI equivalent to next() and router.use() that is in Express.js?

Currently migrating a personal project that was written in Express.js over to FastAPI. So far, after googling and scanning through FastAPI documentation, I haven't seen the equivalent of a next() or router.use() that I've utilized in Express.js (see more here). Is there something perhaps in the documentation that I missed? Or a possible work around for this particular issue? Any help/guidance is greatly appreciated!

It seems you are looking for https://fastapi.tiangolo.com/tutorial/bigger-applications/

I am currently using APIRouter() in my code now. I didn't really see my answer on that part of the website, thus my question (I am new to FastAPI as a whole). Would it be the equivalent of just adding another app.include() with a file going to another router with no path operations?
This is my main.py file
#!/usr/bin/python

import uvicorn
from fastapi import FastAPI
from app.router import router

app = FastAPI()
app.include_router(router)

if __name__=='__main__':
  uvicorn.run(app, host='localhost')

my router.py file is below
#!/usr/bin/python

from fastapi import APIRouter
from app.controllers.first_controller import FirstController
from app.controllers.second_controller import SecondController
from starlette.requests import Request
from starlette.responses import JSONResponse

router = APIRouter()
@router.post("/sendData")
async def fc_func(request: Request):
  dataToSend = await FirstController().check_for_info(request)
  return JSONResponse({"info":"{}".format(dataToSend)})

@router.post("/tryit"):
async def sc_func(request: Request):
  responseText = await SecondController().tryIt_func(request)
  return JSONResponse(responseText)

In my use case, am I adding just another async def func() to my code in router.py with no decorator or do I need to create another app.include_router(new_router_file.py) which goes to different routes that have no decorators? Hope this gives a better idea of what I'm trying to do.


Fix and add Tests For character length and concat

#240 #241

 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.think seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

hi @jczuchnowski
Thanks for the nice zio-sql intro talk.
I was just trying to do my first PR here. Sorry I am very new to scala, so please bear with me on this one.
The current PR is only doing this basic test query for char length val query = select(CharLength("first_name")) from customers
but looking at your talk I realised this should be possible val query = select(CharLength(Concat(fName ++ lName))) from customers.
 type mismatch;
[error]  found   : zio.sql.postgresql.PostgresModuleTest.Selection[zio.sql.postgresql.PostgresModuleTest.Features.Union[zio.sql.postgresql.PostgresModuleTest.Features.Source,zio.sql.postgresql.PostgresModuleTest.Features.Source],zio.sql.postgresql.PostgresModuleTest.Customers.customers.TableType,zio.sql.postgresql.PostgresModuleTest.SelectionSet.Cons[zio.sql.postgresql.PostgresModuleTest.Customers.customers.TableType,String,zio.sql.postgresql.PostgresModuleTest.SelectionSet.Cons[zio.sql.postgresql.PostgresModuleTest.Customers.customers.TableType,String,zio.sql.postgresql.PostgresModuleTest.SelectionSet.Empty]]]
[error]     (which expands to)  zio.sql.postgresql.PostgresModuleTest.Selection[zio.sql.postgresql.PostgresModuleTest.Features.Union[zio.sql.postgresql.PostgresModuleTest.Features.Source,zio.sql.postgresql.PostgresModuleTest.Features.Source],zio.sql.postgresql.PostgresModuleTest.Customers.customers.TableType,zio.sql.postgresql.PostgresModuleTest.SelectionSet.Cons[zio.sql.postgresql.PostgresModuleTest.Customers.customers.TableType,String,zio.sql.postgresql.PostgresModuleTest.SelectionSet.Cons[zio.sql.postgresql.PostgresModuleTest.Customers.customers.TableType,String,zio.sql.postgresql.PostgresModuleTest.SelectionSet.Empty.type]]]
[error]  required: zio.sql.postgresql.PostgresModuleTest.Expr[?,?,(String, String)]
[error]       val query = select(CharLength(Concat(fName ++ lName))) from customers

I would like to add test for a query like select(Concat(fName ++ lName)) ++ (CharLength(Concat(fName ++ lName))) from customers. Just wanted to know if its currently possible. thank you.

Hi @vvidlearn
Not sure if it helps, but I was working on something similar (concat_ws) on https://github.com/zio/zio-sql/pull/267 Spoiler alert it's still VERY dirty :) but maybe it will help you
From the stack I guess, that you are constructing a Selection (++) but using it as an Expr. IMO use a comma instead of ++
Having said that, we  both might have issue with extending our solution... We are unlucky to have varargs and atm natively zio-sql supports up to 4 args. I've tried adding varargs yesterday, got stuck in "type hell"

Hi @pjazdzewski1990 thanks for the reply and for the generous comment :)
Watching the talk yesterday I heard about dependent types and  that zio-sql uses it for compile time safely. Hence I was trying to writing a test that I think would combine SelectionSet with FunctionDef but ye the rather lengthy types takes some getting used to. Good luck with your PR too :)

@vvidlearn would you mind moving the tests to FunctionDefSpec.scala? We're keeping these function tests in there.

@vvidlearn to answer your question - no, the syntax of select(CharLength(Concat(fName ++ lName))) won't work at this point. I'll create the issue for that.

sure will do, thanks

@jczuchnowski moved tests to FunctionDefSpec as suggested.


Add synonym to  "main" sequence specific TF term

can we add the synonym
broad "transcription factor"  to GO:0000982
It is difficult to locate key terms because "transcription factor" has 500+ hits!
https://github.com/pombase/canto/issues/1083#
cheers
Val

Done.


Unable to connect to XMPP on Debian 64 bit installation

The XMPP Port is open but I cannot connect
lsof -i -P gets
node      29414        root   22u  IPv6 2966443      0t0  TCP *:5222 (LISTEN)
telnet localhost 5222
Trying <IP_ADDRESS>...
telnet: Unable to connect to remote host: Connection refused
Let´s chat shows no errors
My settings look like this
env: development # development / production
http:
enable: false
port: 5000
https:
enable: true
port: 5001
key: xxx
cert: xxx
files:
enable: true
provider: local
local:
dir: uploads
xmpp:
enable: true
debug:
handled: true
unhandled: true
port: 5222
host: xxx.de
tls:
enable: true
key: xxx
cert: xxx
database:
uri: mongodb://localhost/letschat
secrets:
cookie: xxx
local:
dir: uploads
s3:
accessKeyId:
secretAccessKey:
bucket:
region:
auth:
providers: [local]
local:
enableRegistration: true
salt: xxx# Required when upgrading from version < 0.3

I am just realizing that the port is opened only for IPv6. It is not opened for IPv4 by lets chat or any other program (just checked that)

This is what I get on Mac OS X, looks similar to what you get:
node      84996 sibartlett   29u  IPv6 0xa2a7dbb01734667d      0t0  TCP *:5222 (LISTEN)

Telnet worked for me though.

The SSL Port looks like this
node      29414        root   10u  IPv4 2966442      0t0  TCP *:5001 (LISTEN)

My local instance reports HTTP/HTTPS as IPv4, and XMPP as IPv6.

So does mine, but for me XMPP really only works with IPv6. Connecting to 5001 only works with IPv4.

I realize my comments haven't been very helpful so far.
@geoffwhittington @TuxOtaku do you guys happen to have a debian install you can test the XMPP functionality on?

@sibartlett I can set one up in the lab for you tomorrow if you like. :smile:

I've done all my testing on Ubuntu, I just assumed Debian was identical but I'm wrong often. I'll see what i can do!

I have setup a barebone fresh Debian 64 VM and just installed the necessary stuff (you are missing a probably too obvious prerequisite in the installation instructions, namely "git" itself :-) ). With just http and XMPP enabled lets chat will open Port 5000 with v4 only and Port 5222 v6 only.

I played a little more with my test installation. Althought port 5222 is only listed as IPv6 you can connect with IPv4. A telnet localhost 5222 gives a trying ::1 and then a Connected to localhost, so it defaults to IPv6. A telnet <IP_ADDRESS> 5222 will still connect. This is different from the behaviour of lets chat on my server althought the listing of the ports with lsof is the same.

It looks like you can connect to XMPP on 5222, is that right?

Yes on my own server with IPv6 on the test installation IPv4 and IPv6

Ok great - good to hear!

I am having the same issue.. XMPP only work with ipv6.. Posted on lets-chat google group..

Proposed patch: file xmpp/node/node_modules/lets-chat/app/xmpp/index.js" replace this segment:
function xmppStart(core) {
var options = {
port: settings.xmpp.port,
domain: settings.xmpp.domain
};
with this:
function xmppStart(core) {
var options = {
port: settings.xmpp.port,
host: settings.xmpp.host,
domain: settings.xmpp.domain
};
And add xmpp.host entry to "xmpp/node/node_modules/lets-chat/settings.yml" with a default value of "<IP_ADDRESS>".


dirPaginate: Watch end of clearing pagination

Hi Michael, great job on the dirPaginate! Thanks!  =)
I just have a little question: Is there anything I can watch so I know when the items finished loading?
My problem is that I have to momentarily remove pagination, export the table, and then re-paginate. But it is exporting the filtered table and removing the pagination after my method ends. Do you know if I can put a watch on something to control that?
Thank you very much!

Hi, I'm having trouble understanding what exactly you are trying to do. Could you show me a bit of code relevant to the issue?

Sorry...
The table is loaded like this:

 ...

And I'm trying to do the following on my script:
// Function called by a button to export the html table to Excel
$scope.exportGridToExcel = function () {
    // I save the current "maxItemsPerPage"
    oldMaxItemsPerPage = $scope.maxItemsPerPage;
    // And set it to null, so the table won't be paginated.
    $scope.maxItemsPerPage = null;

    // ** THE PROBLEM HAPPENS HERE **
    // It starts exporting my grid before it is visible on screen. The rows are not loaded yet.
    $("#gridIndex").table2excel({ exclude: ".no-export", name: "Report", filename: "Report" });

    // Put pagination back.
    $scope.maxItemsPerPage = oldMaxItemsPerPage;
 };

Please tell me if you need more pieces of the code.
Thanks!

Hi, Michael.
I was able to accomplish what I wanted following finishingmove's answer on StackOverflow here:
http://stackoverflow.com/a/15208347/4519548
I created a directive to listen to the end of the ngRepeat:
var module = angular.module('testApp', [])
    .directive('onFinishRender', function ($timeout) {
    return {
        restrict: 'A',
        link: function (scope, element, attr) {
            if (scope.$last === true) {
                $timeout(function () {
                    scope.$emit(attr.onFinishRender);
                });
            }
        }
    };
});

And on my controller I catched that with $on:
$scope.$on('ngRepeatFinished', function(ngRepeatFinishedEvent) {
    //you also get the actual event object
    //do stuff, execute functions -- whatever...
    if ($scope.isExportingGridIndex) {
        $scope.isExportingGridIndex = false;

        $("#gridIndex").table2excel({ exclude: ".no-export", name: "Report", filename: "Report" });

        $scope.maxItemsPerPage = $scope.oldMaxItemsPerPage;
    }
});

With html that looks like this:

...

It worked the way I wanted. Only exporting when angular's finished loading all the items...
But thank you for the quick response!  =)

hi ricfwolff, can u kindly let me know how does ur ng-click have because im stuck in same situation, what i want to do is when i click the export button, i want to remove pagination then export and then repaginate. i created same controller listener and directive but cant seem to trigger it on pressing the button, im new to angular

Hi @xbipin! Sorry for the delay.
My angular controller have the following functions:
Note: I used a component called ALaSql (http://alasql.org/) to do the magic on exporting angular data to excel.
    $scope.loadAllItens = function () {
        var takeNumber = 99999;
        $scope.loadList(takeNumber);
    };

    $scope.exportGridToExcel = function () {
        $scope.loadAllItens();
        $scope.isExportingGridIndex = true;
    };

    $scope.$on('ngRepeatFinished', function () {
        if ($scope.isExportingGridIndex) {
            $scope.isExportingGridIndex = false;
            $scope.exportData();
        }
    });

    $scope.exportData = function () {
        var reportName = 'Report ';

        var queryColumns = 'Column A, ColumnB, ColumnC';

        alasql('SELECT ' + queryColumns + ' INTO XLSX("' + reportName + '.xlsx", {header:true}) FROM ?', [$scope.myData]);
    };

And my HTML has the following on ng-click:
Export To Excel

Hope that helps you!
Rick

well the problem i was having was i tried using table2excel with dir-paginate and i had a few filters in it for search etc and if the pages exceeded to more than 1 and then u remove pagination then on n-finish-render event the table would export properly but if the table was just one page length then removing pagination wouldnt generate the event to export the table


Chat is not working

I have loggedin using two different user into two different mobile. I am sending message from one user to another. Another user is not able to get the message. Why this is not working?

The user who is sending message is able to see his message with icon not delivered and the recipient is not able to receive the message.

Have you installed https://github.com/chat21/chat21-cloud-functions as described on README?

Hi @rizwan-dev What's your problem?

Hi,
I am having the same problem. I have set up two devices with chat 21, they are not able to read each other's messages. Is there any advice you can give?
I have logged in using two different users in two different emulators. I send message from one user to another. Another user is not able to get the message. Please test your sdk and confirm that it is actually working.

Does anyone have a forked version of this SDK that is working? This would be a lot of wasted effort to get this far and for it not to work.


Clarify if this project includes example client app

Hello,
from my understanding this project includes only API reader which needs to be compiled to dll and after this it be used inside some client application. Am I right?
If so, I think it would be a good idea to clarify this in Readme, or put a link do other project with example client (if any exists).

I don't have an example project yet, this project started as I had an idea of a project in mind but decided to delay it for a bit and started this project instead.
And yes, this typically will be used for some sort of client, though you will be able to use on a server it doesn't make any difference. It's just the most common use will probably be on a client.


Storageclasses.storage.k8s.io is forbidden: User Cannot create resource "storageclasses" in api group storage.k8s.io at the cluster scope

Rancher OS version 1.5.4
Ran through the steps as outlined https://github.com/longhorn/longhorn/blob/master/docs/csi-config.md
Installing via catalog from Rancher Management interface. Anything I should specifically provide for this?
"annotations": {
"lifecycle.cattle.io/create.helm-controller_c-gpqwj": "true"
},
"answers": {
"csi.attacherImage": "",
"csi.attacherReplicaCount": "",
"csi.driverRegistrarImage": "",
"csi.kubeletRootDir": "",
"csi.provisionerImage": "",
"csi.provisionerReplicaCount": "",
"defaultSettings.backupTarget": "",
"defaultSettings.backupTargetCredentialSecret": "",
"defaultSettings.backupstorePollInterval": "300",
"defaultSettings.createDefaultDiskLabeledNodes": "false",
"defaultSettings.defaultDataPath": "/var/lib/rancher/longhorn/",
"defaultSettings.defaultLonghornStaticStorageClass": "longhorn-static",
"defaultSettings.defaultReplicaCount": "3",
"defaultSettings.guaranteedEngineCPU": "0",
"defaultSettings.replicaSoftAntiAffinity": "true",
"defaultSettings.storageMinimalAvailablePercentage": "5",
"defaultSettings.storageOverProvisioningPercentage": "500",
"defaultSettings.taintToleration": "",
"defaultSettings.upgradeChecker": "true",
"ingress.enabled": "false",
"persistence.defaultClass": "true",
"persistence.defaultClassReplicaCount": "3",
"service.ui.type": "Rancher-Proxy"
},
"appRevisionId": "apprevision-2j6vf",
"baseType": "app",
"conditions": [ 4 items
{
"lastUpdateTime": "2019-12-12T04:00:17Z",
"status": "True",
"type": "Migrated"
},
{
"status": "Unknown",
"type": "UserTriggeredAction"
},
{
"lastUpdateTime": "2019-12-12T04:00:20Z",
"message": "failed to install app longhornstorage-p-qs6z8. Error: release longhornstorage-p-qs6z8 failed: storageclasses.storage.k8s.io is forbidden: User "u-dp4hnbvpzk" cannot create resource "storageclasses" in API group "storage.k8s.io" at the cluster scope\n",
"reason": "Error",
"status": "False",
"type": "Installed"
},
{
"lastUpdateTime": "2019-12-12T04:00:20Z",
"status": "True",
"type": "ForceUpgrade"
}
],
"created": "2019-12-12T04:00:17Z",
"createdTS":<PHONE_NUMBER>000,
"creatorId": "u-dp4hnbvpzk",
"externalId": "catalog://?catalog=library&template=longhorn&version=0.7.0",
"id": "p-qs6z8:longhornstorage-p-qs6z8",
"labels": {
"cattle.io/creator": "norman",
"mcapp": "longhornstorage"
},
"links": {
"revision": "…/v3/project/c-gpqwj:p-qs6z8/apps/p-qs6z8:longhornstorage-p-qs6z8/revision",
"self": "…/v3/project/c-gpqwj:p-qs6z8/apps/p-qs6z8:longhornstorage-p-qs6z8",
"update": "…/v3/project/c-gpqwj:p-qs6z8/apps/p-qs6z8:longhornstorage-p-qs6z8"
},
"multiClusterAppId": "longhornstorage",
"name": "longhornstorage-p-qs6z8",
"namespaceId": null,
"notes": "---\n# Source: longhorn/templates/NOTES.txt\n1. Get the application URL by running these commands:\n  kubectl get po -n $release_namespace\n\n",
"projectId": "c-gpqwj:p-qs6z8",
"prune": false,
"state": "installing",
"targetNamespace": "longhornstorage-p-qs6z8",
"timeout": 300,
"transitioning": "error",
"transitioningMessage": "failed to install app longhornstorage-p-qs6z8. Error: release longhornstorage-p-qs6z8 failed: storageclasses.storage.k8s.io is forbidden: User "u-dp4hnbvpzk" cannot create resource "storageclasses" in API group "storage.k8s.io" at the cluster scope\n",
"type": "app",
"uuid": "e84d05eb-1c93-11ea-9438-0242ac1a0002",
"wait": false

The error you provide means you haven't launched this cluster role correctly. Could you check the RABC config for Longhorn?

I suppose I dont understand. Rancher was configured using a docker command, a cluster was then built with the UI. From there I manually configured each worker as defined in the docks, then followed the ui in rancher to deploy longhorn. How would I check the cluster role?

I installed version 0.6.1 instead of 0.7 and it worked. Maybe an issue with 0.7?

Tried to build another new cluster, using the UI same problems when installing with 0.6.1

Previously, I missed some important info failed to install app longhornstorage-p-qs6z8....User "u-dp4hnbvpzk" cannot create resource. This means that the Rancher user/account you used doesn't have the permission to deploy StorageClass for app Longhorn.

For the upgrade from v0.6.1 to v.07.0, you can refer to this doc

@shuo-wu This is an out of the box install of rancher, with a cluster deployed by rancher and I am logged in as admin. According to the doc https://github.com/longhorn/longhorn seems like all i should do is press install from the app catalog? Is there something I an run to test this?
I have built 4 different clusters, deployed rancher management twice (one with nginx in front, one by itself) each time except on the installer fails.
Latest message: Failed to install app longhorn-p-h8ktm. Error: release longhorn-p-h8ktm failed: storageclasses.storage.k8s.io is forbidden: User "u-tpwqefki6x" cannot create resource "storageclasses" in API group "storage.k8s.io" at the cluster scope

If i deploy using

The StorageClass indeed requires a Kubernetes cluster level permission. Longhorn is a system-level software, provides the storage to whole cluster, and one cluster can only have one installation. So we suggest installing Longhorn as a cluster-admin.

Got ya, maybe it should default to that role? As it is in rancher it has project role default, and no mention doc I saw . Could have missed it but man frustrating for something that was a checkbox.

@majerus1223 You're using multiple clusters app deployment? That's the only place I saw with selection of cluster role and project role. Not sure if Rancher allows any application to change the default role there, you might want to check with Rancher.

Just a single cluster, maybe add it to the deployment notes or something for rancher? Not sure what I am going to check with Rancher on really.

@majerus1223 Can you describe the reproduce steps, e.g. when did you click on the Rancher UI step by step?

Yep for sure..
Built a cluster with Rancher.
Created a project named "Longhornstorage"
Navigated back to main cluster page
Clicked "Apps" on the top bar
Pressed "Launch" in the upper right hand corner
Selected "longhorn"
In the long horn configuration options
Typed in name "longhornstorage"
Selected the target "Longhornstorage"
Did not modify "available roles" default is "project"
Scrolled to the bottom and pressed "launch"

@majerus1223 I think you're using the multicluster app.
On step Navigated back to main cluster page, if you see a list of clusters instead of what's inside the cluster, then you're at global page (instead of cluster's main page). After clicking apps, if you see Multi-Cluster Apps on the top left corner of the page, it means you're using multicluster app feature. Single cluster app doesn't need to select target.
To access the cluster's main page, hover over Global, and click the name of the cluster.

Just redeployed by clicking
Cluster Name
Selected 'Default' project
Clicked 'Apps'
Deployed longhorn with defaults. Waiting to see what that does. I honestly didnt know there was a difference between apps when selecting from the local cluster and global. If there is a difference does that mean longhorn isnt a multi cluster app?

Looks like that deployment failed.

@yasker thanks for clearing this up, I had the same issue.
Deployment were successful btw.

@majerus1223 For the multicluster app, you need to manually select the role. And you still need to keep longhorn-system as the namespace.
Any error message from the failed deployment?

Failed to create new replica set "longhorn-driver-deployer-584b7bf585": replicasets.apps "longhorn-driver-deployer-584b7bf585" is forbidden: unable to create new content in namespace longhorn-p-85sc2 because it is being terminated; Deployment does not have minimum availability.

ReplicaSet "longhorn-driver-deployer-584b7bf585" has timed out progressing.; Deployment does not have minimum availability.

There is definitely an issue with 0.7 on a clean install. 0.6.2 installs just fine, but upon upgrade the warning in OP is triggered, and then the the driver deployer image has a segfault. Or if you try to go straight to 0.7 you just hit the segfault. This is on stock Ubuntu 18.04.

@qeternity I think your issue is different. Which segment fault you're hitting?


Improve support for testing XPath functions defined in XSLT variables or XQuery variables

In either XSLT or XQuery, you can define a variable whose value is an XPath function and then call that function using the variable.
It would be nice if XSpec could test an XPath function stored in variable. You can do this in some circumstances, but it's not that intuitive. This issue is to see if we can make the support more intuitive and agree on what that design would look like.
@xspec/xspec : Any thoughts from the XSpec community?
Example
XSLT
<?xml version="1.0" encoding="UTF-8"?>
<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
    xmlns:xs="http://www.w3.org/2001/XMLSchema"
    xmlns:myf = "x-urn:example:xpath-fcn"
    version="3.0">

    <xsl:variable as="function(*)" name="myf:my-xpath-fcn" select="
            function ($x as xs:string) as xs:string {
                'input was ' || $x
            }"/>

</xsl:stylesheet>

XQuery
module namespace myf = "x-urn:example:xpath-fcn";

declare variable $myf:my-xpath-fcn as function(*) :=
    function ($x as xs:string) as xs:string {
        'input was ' || $x
    }
;

XSpec Design Ideas
<?xml version="1.0" encoding="UTF-8"?>
<x:description xmlns:x="http://www.jenitennison.com/xslt/xspec"
    xmlns:myf = "x-urn:example:xpath-fcn"
    stylesheet="../xpath-fcn.xsl"
    query="x-urn:example:xpath-fcn"
    query-at="../xpath-fcn.xq"
    run-as="import">

    <x:scenario label="Design 1: Test an XPath function using XSpec variable">
        <!-- Does not work with run-as="external", but that limitation aligns with other
             limitations of external transformations -->
        <!-- Requires irrelevant x:call or x:context to avoid compiler error -->
        <x:variable name="fcn-call-result" select="$myf:my-xpath-fcn('xyz')"/>
        <x:call function="true"/>
        <x:expect label="XSpec variable value stores expected result of XPath function call"
            test="$fcn-call-result"
            select="'input was xyz'"/>
    </x:scenario>

    <x:scenario label="Design 2: Test an XPath function using x:call" pending="compiler error">
        <!-- x:call/@function value is not schema-valid due to $ -->
        <!-- If function variable name is in a namespace, compiler returns
            src\compiler\base\util\compiler-eqname-utils.xsl:30:14: Fatal Error! Invalid namespace prefix '$myf'
            -->
        <x:call function="$myf:my-xpath-fcn">
            <x:param select="'xyz'"/>
        </x:call>
        <x:expect label="$x:result stores expect result of XPath function call"
            select="'input was xyz'"/>
    </x:scenario>

</x:description>

Possible Improvements
Design 1
In the XSpec compiler, remove the requirement to provide x:call or x:context in each scenario, or consider x:variable an acceptable substitute.
As a side benefit, that would make it easier to test other things besides functions stored in variables, such as

XSLT accumulators (example)
XSLT keys
XML content that you generated using a language other than XSLT or XQuery, where you'd like to make assertions about patterns in the XML. Maybe this is an edge case or out of scope, but I did come across a real-life situation once where it seemed like a useful thing to do.

Design 2
Update XSpec schema so x:call/@function can hold a reference to a variable whose value is a function.
Investigate the compiler-eqname-utils.xsl error and adjust whatever is treating the $ as part of the namespace prefix. I haven't looked into this yet, but I can do so if we want to support this design.

Maybe a new attribute for x:call[@function] is the easiest to implement, has virtually no impact on the other areas and requires only a few tests.
Suppose it's x:call[@function]/@type
        <x:call function="myf:my-xpath-fcn" type="variable">
            <x:param select="'xyz'"/>
        </x:call>

and then x:function-call-text() in src/compiler/base/compile/compile-scenario.xsl can prepend $ when @type is variable.

Ease of implementation sounds good, especially since this would be kind of a niche feature. Your idea makes sense from the perspective that myf:my-xpath-function is the name and the prepended $ is just the syntax for referring to the variable.
It would be nice if type="variable" had a counterpart representing the default behavior, to help distinguish the two choices from each other. However, the ideas I have so far all seem unclear or not really correct. In the spirit of brainstorming, here's what I thought of. Does any pair look reasonable?

type="function-declaration" (default) vs. type="variable"
declaration-type="function" (default) vs. declaration-type="variable"
type="module-function" (default) vs. type="function-item" (The XSLT 3.0 spec uses terms "stylesheet function" and "function item." I tried to broaden "stylesheet function" to make it less XSLT-centric but I'm afraid the meaning is not exactly the same.)
call-as="function" (default) vs. call-as="variable"
language="native" (default) vs. language="xpath"
language="host" (default) vs. language="xpath"

A data point: I showed your proposed syntax to someone who has some applicable instances, and they thought it was a nice improvement over the syntax they're currently using.

More possibilities to add to the list:

type="module-function" (default) vs. type="variable"
call-as="module-function" (default) vs. call-as="variable"


Rebind hand animation control to HolisticTracker

This change adds back the left and right hand's landmark outputs to their corresponding controllers. #44 reverted this binding as a part to rollback the previous change.
Scene is changed to link the game object of wrists to the controller.

@superkudo This change adds back another thing the rollback commit #44 removed. It may also slow things down because I have added Unity code (like setting transform.rotation) for hand wrist rotation.

LGTM


Enable update management for a VM using python sdk

Requesting for python sdk module to enable update management for a VM using a existing log analytics and automation account
I couldn't find any particular documentation related to this specific operation .Please let me know if we have options to enable update management using python sdk

Adding @msvijayn

Almost, there is complete support for the required steps from Azure Automation & Update Management as well as Log Analytics. But it seems dependent service like OperationsManagement  solutions, can be used only available in ARM and not in Python SDK.
I am listing (summarising) below the usual steps for using Update Management via ARM template/API and its equivalent in Python SDK.
1. Creating Log Analytics Workspace

ARM: https://docs.microsoft.com/azure/templates/microsoft.operationalinsights/2015-11-01-preview/workspaces

Python:  https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/loganalytics/azure-mgmt-loganalytics/azure/mgmt/loganalytics/operations/workspaces_operations.py

2.  Creating Automation Account

ARM: https://docs.microsoft.com/azure/templates/microsoft.automation/2015-10-31/automationaccounts
Python: https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/automation/azure-mgmt-automation/azure/mgmt/automation/operations/automation_account_operations.py

3. Linking Automation account & Log Analytics workspace

ARM: https://docs.microsoft.com/azure/templates/microsoft.operationalinsights/2015-11-01-preview/workspaces/linkedservices
Python: https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/loganalytics/azure-mgmt-loganalytics/azure/mgmt/loganalytics/operations/linked_services_operations.py

4. Setup OMS/LA Solution like Update Management

ARM: https://docs.microsoft.com/azure/templates/microsoft.operationsmanagement/2015-11-01-preview/solutions
Python: Not Available (to my knowledge)

5. Onboard machine to a solution using VM extensions

ARM: https://docs.microsoft.com/azure/templates/microsoft.compute/2019-07-01/virtualmachines/extensions
Python: https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/compute/azure-mgmt-compute/azure/mgmt/compute/v2019_07_01/operations/_virtual_machine_extensions_operations.py

Thanks for the details


ecr-credential-provider Binary

Environment: All

AWS Region: us-west-2
Instance Type(s):
EKS Platform version: eks.3
Kubernetes version: 1.25.9
AMI Version: Custom
Kernel: amd64

I would like to know how I can fetch the ecr-credential-provider binary from s3 the way it is done for the official EKS AMIs. Is there an official public source for someone who is building their custom AMI?
Issue: I tried to build the ecr-credential-provider and placed it in the required location, however the plugin fails to fetch credentials. With the following error: failed to authorize: failed to fetch oauth token: unexpected status: 403 Forbidden" image="602401143452.dkr.ecr.us-west-2.amazonaws.com/amazon-k8s-cni-init:v1.11.4"
I believe that the plugin I am building using the ecr-credential-provider files from cloud-provider-aws is not built correctly and would like to use the binary provided by AWS.

We publish this binary in the amazon-eks S3 bucket as well, for example:
aws s3 cp s3://amazon-eks/1.27.1/2023-04-19/bin/linux/amd64/ecr-credential-provider .

To resolve the 403 make sure your instance profile/temp creds/etc. can hit ecr.GetLoginPassword. You can verify that with the AWS CLI.

Great, thank you so much!

@cartermckinnon is there is list of Binaries and their s3 paths I can refer to for older versions?

We publish it for every k8s version + build date since 1.20, e.g.:
aws s3 ls --recursive s3://amazon-eks/1. | grep "ecr-credential-provider$"

Thank you once again!


ETA for next release?

Hi, thanks for the great job with this operator.
We have a big issue about this: https://github.com/presslabs/mysql-operator/issues/543
For what we can see, the fix should be available in the next release, as described in the "unreleased" section of the changelog.
is there an ETA for it?
Thanks

Hi, somebody can explain us how we can use the fixed code in helm without wait for the next release?

bump?

@Leen15 you can make your own build of the operator using the Makefile.
Obviously, you'll need to adjust certain parameters e.g. the docker registry.

@Leen15 you can make your own build of the operator using the Makefile.
Obviously, you'll need to adjust certain parameters e.g. the docker registry.

it has been >110 commits to master since last tag, would be great to have a tag.

I use it like this:
    helm install mysql-operator presslabs/mysql-operator --version=0.5.0-rc.2 \     --set antiAffinity="hard" \     --set ApplyMySQLPromotionAfterMasterFailover=true \     --set orchestrator.image="quay.io/presslabs/mysql-operator-orchestrator:v0.5.0-rc.2" \     --set image="quay.io/presslabs/mysql-operator:v0.5.0-rc.2"

@hmilkovi I'm able to start the helm with your command, but I'm not sure the mysql version is up to date on a new cluster created.
In the specific, the #543 is still present.
The image used by the operator for start mysql is something like:
percona@sha256:68dad5e2efeb6893e2d7d116a1eae144f2c641c17d00e7869397395590c91651
Maybe is still the old one?


feat(client/properties-provider): multi instance option to none tasks

Closes #128

Fixed merge conflicts.


Use proper steps in Dockerfile build

Fixes #1064
Changes:

Remove make, g++, and python installation.
Change yarn to npm.
Add npm run build step.

@shreyasgune1 does the Dockerfile work for you with this update?

@bonustrack I have tested it myself first and it failed (it had to), it works on new Dockerfile

@bonustrack
I tested it on my office machine. Will rest tommorow.

Tested docker build with new dockerfile. This time it refuses to even build. The warnings are:
PS C:\Users\Mr. Shreyas\documents\codes\busy> docker build busy
Sending build context to Docker daemon  53.89MB
Step 1/8 : FROM node:alpine
Get https://registry-1.docker.io/v2/: dial tcp: lookup registry-1.docker.io on [::1]:53: read udp [::1]:37375->[::1]:53:
 read: connection refused

However, I get the same error on same step in using older dockerfile also.
It appears something else is wrong.....

You should be running docker build . as you are already in busy folder.

Ok I'll try!
On Mon, Nov 13, 2017 at 1:58 PM, Wiktor Tkaczyński<EMAIL_ADDRESS>
wrote:

You should be running docker build . as you are already in busy folder.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/busyorg/busy/pull/1065#issuecomment-343846360, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUKJfBAp5rtlIojXZENsCE7263lxJom5ks5s1_2hgaJpZM4QaPvA
.

--
Regards,
Shreyas Gune
Assistant Director
National Mission for Clean Ganga

@shreyasgune1 any progress?

Can't seem to get the app running. Tried many things. The problem seems to
be that docker-machine is not running. To run it, hyper V needs to be
enabled. To enable that, a lot of memory is required which I don't have.
Please test the issue at your end. Thank You.
On Fri, Nov 17, 2017 at 2:14 AM, Wiktor Tkaczyński<EMAIL_ADDRESS>
wrote:

@shreyasgune1 https://github.com/shreyasgune1 any progress?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/busyorg/busy/pull/1065#issuecomment-345056602, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUKJfP0immqF4tmuB15lbJr4HBNaknUZks5s3J6UgaJpZM4QaPvA
.

--
Regards,
Shreyas Gune
Assistant Director
National Mission for Clean Ganga

As I mentioned I tested it on my machine and works properly.
I think we can merge that.
cc @bonustrack


React Native Clipboard is Deprecated.

Clipboard has been extracted from react-native core and will be removed in a future release. It can now be installed and imported from '@react-native-community/clipboard' instead of 'react-native'. See https://github.com/react-native-clipboard/clipboard

https://github.com/GeekyAnts/NativeBase/pull/4148