[Feature] Added Contact Form

Hey, @SaptarshiSarkar12 I have added the contact form to your website. Kindly verifies it.

Light Mode

Dark Mode

.deploy

Any Issues @SaptarshiSarkar12

Any Issues @SaptarshiSarkar12
I don't know why I can't deploy previews of PRs. Can you see the workflow file and tell me if there is any problem?

Any Issues @SaptarshiSarkar12

@KIET7UKE Where does the message comes, when the submit button is pressed?

@SaptarshiSarkar12 Yes, I forgot to mention that you have to just add your mail action tag
e.g: action = "mailto:yourmailid".

@SaptarshiSarkar12 Yes, I forgot to mention that you have to just add your mail action tag e.g: action = "mailto:yourmailid".

Please add that

@SaptarshiSarkar12 I have created slides of steps on how to add this check below:
https://scribehow.com/shared/Get_message_from_visitors__xI1aPzEsR3SWdCcXGz0UWA

@SaptarshiSarkar12 I have created slides of steps on how to add this check below: https://scribehow.com/shared/Get_message_from_visitors__xI1aPzEsR3SWdCcXGz0UWA

@KIET7UKE Will this form work if I add "action=mailto:mail_ID"?

@SaptarshiSarkar12 I have created slides of steps on how to add this check below: https://scribehow.com/shared/Get_message_from_visitors__xI1aPzEsR3SWdCcXGz0UWA

@KIET7UKE Will this form work if I add "action=mailto:mail_ID"?

Yes, It'll work but sometimes won't
The best case is to make an account at https://getform.io/
register here with your email address, and simply add the generated form endpoint in the action tag. easy.

@SaptarshiSarkar12 Did this all solved you problem?

@SaptarshiSarkar12 Did this all solved your problem?

No @KIET7UKE , getform.io allows upto 50 submissions. i researched and found that we can also use "php" file to send e-mail directly. Here is the article - https://blog.hubspot.com/marketing/html-form-email [method 2]

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then again make a pr?
Or should I raise another PR for this contact form connection

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

@KIET7UKE

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

That's I think will conflict in future

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

That's I think will conflict in future

@KIET7UKE What about this : https://www.youtube.com/watch?v=0X0kovjzLrw ?

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

That's I think will conflict in future

@KIET7UKE What about this : https://www.youtube.com/watch?v=0X0kovjzLrw ?

This is same as I did earlier with getforms.io
For now keep this things working, later on many contributors will improve this as well.

@SaptarshiSarkar12 Okay, so what are you saying, will I do that PHP connection also and then merge into this pr?
Or should I raise another PR for this contact form connection

Will php work for github pages?

That's I think will conflict in future

@KIET7UKE What about this : https://www.youtube.com/watch?v=0X0kovjzLrw ?

This is same as I did earlier with getforms.io
For now keep this things working, later on many contributors will improve this as well.

ok @KIET7UKE

@KIET7UKE Then, what to do with this PR?

@KIET7UKE Then, what to do with this PR?

Merge it for now, will raise another issue for contact form redirection.
And I will work on this.

@KIET7UKE Then, what to do with this PR?

@SaptarshiSarkar12 Merge it for now, will raise another issue for contact form redirection. And I will work on this.

ok.

@SaptarshiSarkar12 done

@SaptarshiSarkar12 done

@KIET7UKE Don't you think that a form with no action doesn't seem good? I think that I shouldn't introduce the form section as the action hasn't been configured yet.

 I think that I shouldn't introduce the form section as the action hasn't been configured yet.

Also, "mailto:" opens the e-mail client , so , I think this issue needs more help from others.

@SaptarshiSarkar12 Okay so give me some time, I'll make this form working and again commit it

ok @KIET7UKE .


Historical lie

Adam Turing DIDN'T crack the enigma code - he led the team of cryptologists. The person who cracked the enigma code was Marian Adam Rejewski.

What do you think of the following suggestion?

Alan Turing was one of the most important persons in computer science. He created many of the foundations in theoretical computing, such as the Turing machine and the Turing test, and made important breakthroughs in cryptology. Building on the work of the Polish Cipher Bureau, he led the team that was responsible for deciphering the Enigma code during the Second World War. His impact on the outcome of the war can hardly be overestimated.
He was heavily discriminated against because of his homosexuality. He died, probably by his own hand, in 1954.

sorry, didn't want to close this issue directly. Just put the text to discussion.


Improved sampling

This PR seems to have undone much of @palp's deduplication work from #57. I can try and untangle that if you like...

You are right. We will have to fix that soon. Sorry about that.

@jenuk No worries. I already did that in #76 😁


[BUG] Scroll Position Remains Fixed, Even When The Content Rerenders

Describe the bug
When using the side menu to navigate through the documentation, there is a problem where, if you are at the bottom of a page and you select a different section from the menu, the page does not automatically scroll back up to the top. Instead, you remain at the bottom of the page, which can be inconvenient and make it harder to navigate efficiently through the documentation. This behavior can be confusing and frustrating for users who are trying to quickly browse through the content.
To Reproduce
Steps to reproduce the behavior:

Go to https://gympass.github.io/yoga/components/getting-started
Scroll down to the very bottom
Select any of the links from the side navigation bar
Notice how the content re-renders, but the scrollbar remains fixed

Expected behavior
Instead of the scroll position remaining static, it would be convenient if the user does not have to scroll back up every time they open a new link.
Screenshots
https://user-images.githubusercontent.com/115006878/231274800-efd84dde-8144-44dd-b8f6-77e41e2700ca.mp4
Desktop (please complete the following information):

OS: Windows 10
Browser: Google Chrome, Opera, Firefox

@tabishnaqvi1311 You mean on clicking a link it should go to the top of the page right?


Downloading lucene based json request on a dashboard adds extra fields

Found on branch 6.0 commit: 3f73e4aaa28a3dc433d09d7078e6693a7a03858c
Create a dashboard against an index.
Use any query terms.
Add EventId and StreamId columns to the table.
Download the request json.
The json contains the following fields:
  1       fields : [ {
  2         name : StreamId,
  3         expression : ${StreamId},
  4         format : {
  5           type : NUMBER
  6         }
  7       }, {
  8         name : EventId,
  9         expression : ${EventId},
 10         format : {
 11           type : NUMBER
 12         }
 13       }, {
 14         name : StreamId,
 15         expression : ${StreamId}
 16       }, {
 17         name : EventId,
 18         expression : ${EventId}
 19       } ],

When stroom runs an index query it always silently adds StreamId and EventId fields to the query so it can use these for extraction.  We need to do:

rename these added fields to something more unique so we can then remove them from the request being downloaded.
only add the fields if a field of that name and expression does not already exist. This would reduce the volume of data passed if a user chooses EventId/StreamId.
a combination of the two above.

The code that adds the eventId/StreamId is ensureHiddenField() in TablePresenter.  The code that downloads the json is DownloadQueryActionHandler.
The Field class has a visible field on it and these two special fields set that to true so the simplest option is for the code that downloads the query to filter out these hidden fields, probably by passing some mapping options to the mapper.


Invalid Argument Error when training

When training I get this error:
tensorflow.python.framework.errors_impl.InvalidArgumentError: assertion failed: [oh y mi esposo tambi\303\251n]
I don't think this is related to your code but maybe you can help us. The part in italic (the transcription) changes everytime I try it. It is not a problem of Spanish characters because sometimes happens with sentences with only English characters. Any idea of what is going on?

maybe it is because the version of Tensorflow or some library? which one are you using?

Hi @jannaescur @vrenkens ,
I ran into the same issue when training on timit. How did you go about fixing the characters not defined in alphabet issue?
Thanks


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Now getting "undefined is not a function"

I'm using the API exactly like the example but something has been updated and it's not working anymore.

@thelinuxlich  could you give me your code or paste error message for me. thanks

there are some typo readme, already fixed it. @thelinuxlich please run again.
or you can run https://github.com/clonn/slack-node-sdk/tree/master/example , tell me result then.
thanks

worked!


SearchOrder as part of the API (no impl yet)

May I ask what we are going to use this for? We have only one search box. Do we want to let the user choose (via some drop-down menu) how to rank? I would assume we just need to tweak the way we combine the different metrics in order to get good results.

May I ask what we are going to use this for?

Immediately: I want to replace PackageVersion.detectedTypes (which is flutter-only) with pana-recognized platforms. E.g. /flutter/packages will query search with platforms=flutter (maybe turn of the rest, so that only flutter packages are shown), and it needs to order them in reverse chronological update time (SearchOrder.updated here).
Short term: Query the most popular packages (optional: group by platform), to be displayed on the site's index page.
After that: It kind of enables a search-box like the top-left side of https://www.npmjs.com/search?q=
Whether we want to do something like that is up for debate and UX is to be figured out, but at that point it is just a quick add-on.

I would assume we just need to tweak the way we combine the different metrics in order to get good results.

I believe that is correct, I don't imagine too much work needed to implement it. We will still need the API part for it though...


[DOCS]: Get the Lit Docs To Be On Par With The React Documentation

This issue was brought about after a careful observation by Corbin and is targetted to help the Lit devs community to be able to get a very useful and well detailed documentation that helps them in every way possible with their integrations of Tanstack forms in their projects and also to help with general core engineering information as to how the library works with Lit.
For reference:

Tanstack Forms Current React Docs
Tanstack Forms Current Lit Docs

Related to #746.

@crutchcorn I suppose you are mostly talking about the guides section? Or also the examples?
Which of these guides do you want to have available in the Lit docs?

@Christian24 both, ideally.  I'd love to have all of them except for React Native, SSR, and Debugging
FWIW I know that @tobySolutions has started working on the docs for Lit, you might wanna check with him to see if there's any overlap between y'all's potential collab

Maybe assign it to him then? @tobySolutions Let me know if I can give you a hand.

In addition Lit does have SSR, but I never tried it: https://lit.dev/docs/ssr/overview/

In addition Lit does have SSR, but I never tried it: https://lit.dev/docs/ssr/overview/

Thank you, Christian, yes, I started work on the docs guides for Lit and am currently working on things there. Of course, I'll make sure to hit you up if I need your help on things 😁.


profile and root url deeplinking (also fix a sentry error)

Why

tapping profile URLs should open in the app if app is installed. Also support opening root url in app.
Getting this error of accessing profile of undefined object. Fix types in swr and add a null check https://sentry.io/organizations/showtime-l3/issues/3476062340/?project=5860034&query=is%3Aunresolved

Test Plan

Tapping profile links should open in app if app is installed

these changes got merged in https://github.com/showtime-xyz/showtime-frontend/pull/1366. So closing this one 😬


feat: prepare chart scripts for the main branch

Allows a tag with a pre-release postfix of "%Y-%m-%d-%H-%M-%S"

bors merge


Conv2D with XLA jit_compile=True fails to run

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.11.0.dev20220914
Custom Code
No
OS Platform and Distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
The following code works well without jit_compile=True. However, if we enable XLA compilation by adding jit_compile=True, it will throw an error. Reproduced in CoLab notebook here.
Standalone code to reproduce the issue
import tensorflow as tf
from keras import layers

class MyModule(tf.Module):
    def __init__(self):
        super().__init__()
        self.conv = layers.Conv2D(2, 1, padding='valid', dtype=tf.float64, autocast=False)

    @tf.function(jit_compile=True) # without jit_compile=True works fine
    def __call__(self, i0):
        o0 = tf.floor(i0)
        o1 = self.conv(o0)
        o2 = tf.add(o1, o0)
        return o2

def simple():
    inp = {
        "i0": tf.constant(
            3.14, shape=[1,1,3,2], dtype=tf.float64
        ),
    }
    m = MyModule()

    out = m(**inp) # Error!

    print(out)
    print(out.shape)

if __name__ == "__main__":
    simple()

Relevant log output
2022-09-18 01:33:53.096156: I tensorflow/compiler/xla/service/service.cc:173] XLA service 0x55a9ace73180 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2022-09-18 01:33:53.096176: I tensorflow/compiler/xla/service/service.cc:181]   StreamExecutor device (0): NVIDIA GeForce RTX 3080 Ti, Compute Capability 8.6
2022-09-18 01:33:53.098645: I tensorflow/compiler/mlir/tensorflow/utils/dump_mlir_util.cc:268] disabling MLIR crash reproducer, set env var `MLIR_CRASH_REPRODUCER_DIRECTORY` to enable.
2022-09-18 01:33:53.537659: I tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc:428] Loaded cuDNN version 8100
2022-09-18 01:33:54.161249: I tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc:5341] Disabling cuDNN frontend for the following convolution:
  input: {count: 1 feature_map_count: 2 spatial: 1 3  value_min: 0.000000 value_max: 0.000000 layout: BatchDepthYX}
  filter: {output_feature_map_count: 2 input_feature_map_count: 2 layout: OutputInputYX shape: 1 1 }
  {zero_padding: 0 0  pad_alignment: default filter_strides: 1 1  dilation_rates: 1 1 }
  ... because it uses an identity activation.
2022-09-18 01:33:54.749772: I tensorflow/compiler/jit/xla_compilation_cache.cc:476] Compiled cluster using XLA!  This line is logged at most once for the lifetime of the process.
Traceback (most recent call last):
  File "/home/colin/code/test_proj/scripts/tflite2.py", line 41, in simple
    out = m(**inp)
  File "/home/colin/miniconda3/envs/py39/lib/python3.9/site-packages/tensorflow/python/util/traceback_utils.py", line 153, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/home/colin/miniconda3/envs/py39/lib/python3.9/site-packages/tensorflow/python/eager/execute.py", line 52, in quick_execute
    tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
tensorflow.python.framework.errors_impl.UnknownError: CUDNN_STATUS_NOT_SUPPORTED
in tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc(5151): 'status' [Op:__inference___call___46]

Hi @Co1lin !
It is replicating in 2.9, 2.10 and nightly (but not in 2.8).
Work around is to use auto clustering with xla. work around gist.
@gadagashwini !
Could you look at this issue.
Thank you!

Thanks for the bug report! Filed b/249449866 to track internally.

@bviyer @cheshire I can confirm that @bviyer's suggestions are correct. The error occurs on a machine using CUDNN 8202 and it can work smoothly on another machine using CUDNN8500+. But I think it is ideal to make XLA more robust putting a more reasonable message or disabling such optimization for TensorFlow with CUDNN <= 8205. Thanks!

Hi @Co1lin ,
This issue has been resolved in the latest versions. Please find the gist here for your reference.
Thank you!


Update readme.md

added github username and emoji and X
Objective
the assigment
Changes made
added github username and emoji and X
Who needs to review this Pull Request?
@mention some teammates who will check your Pull Request for any mistakes.
Teammates, leave comments in a review if you approve this change

@b-supriya-reddy


Set Up Database Connection for FinTrack

Is your feature request related to a problem? Please describe.
FinTrack requires a reliable and efficient way to store and manage user data, including personal profiles, expenses, budgets, financial goals, income sources, and asset values. Currently, the application does not have a database connection set up, which limits its ability to persist user data and provide comprehensive financial tracking and analysis.
Describe the solution you'd like
We need to set up a database connection for FinTrack. This involves selecting an appropriate SQL database (e.g., MySQL, PostgreSQL), configuring the database schema, setting up the connection in the application, and ensuring secure and optimized interactions between the application and the database. The solution should include:

Database Selection:

Choose the SQL database to be used.

Database Configuration:

Define tables, relationships, and constraints to support the application's functionalities.

Database Connection Setup:

Configure the application to securely connect to the database.

Environment Setup:

Set up database access for development and production environments.

Testing:

Verify successful database connections and CRUD operations.

Documentation:

Document the setup process and provide instructions for future configurations.

Describe alternatives you've considered

Local Storage: Considered using local storage for data persistence, but this approach would not scale well and would limit the application's ability to manage data efficiently and securely across different environments.

Addtional Context:
here are some useful youtube tutorials:
Java to MySQL tutorial: https://youtu.be/3OrEsC-QjUA?si=-BRQjP_MICiF-3Fs
Vaddian to Database access: https://vaadin.com/docs/v23/tutorial/database-access
Please review and provide feedback on the proposed database setup and connection configuration.
Thank you!

approved

Multiple Contributors - shyke0611 and sdes755


Can't start node after checking out latest commit from #1431.

Checked out master after the latest commit from #1431. Starting node with "run -txindex=1". Fails even after adding "-maxCacheBlocksCount=300" to command line options.

Looks like the error may have to with MaxCacheBlocksCount in StoreSettings.cs

This is odd it seems to default to 300
 this.MaxCacheBlocksCount = nodeSettings.ConfigReader.GetOrDefault("maxCacheBlocksCount", DefaultMaxCacheBlocksCount);

BlockStoreCache constructor is called before the StoreSettings.Load so we are using uninitialized value

I can confirm, the latest from master throws this exception.


New version: GodotEngine.GodotEngine.Mono version 4.2

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Current manifest version 4.1.2 has 3 parameters, which is greater than this PR's version number 4.2's 2 parameters.

Hi alexsussy

Current manifest version 4.1.2 has 3 parameters, which is greater than this PR's version number "4.2"'s 2 parameters.\n\nIs this intentional?

Hi @stephengillie ,
Forgive me, I'm still quite new to this! I'm unsure how to resolve the parameter issue, but as far as I can tell, the 4.1.2 manifest is near identical and installing 4.2 locally works as expected with no missing functionality. I did push some minor corrections.
Please correct me if I'm wrong, any advice is appreciated.

Hi @alexsussy,
Just checking if the version number is coming from the developer. We have a few situations where a version number will have parameters added or removed arbitrarily, and this can cause issues with determining which version is the most recent.
But if the developer has called this version 4.1.2 then no worries.

Hi @alexsussy

Current manifest version 4.1.2 has 3 parameters, which is greater than this PR's version number 4.2's 2 parameters.

Is this intentional?


Reorder HTML headings for Accessibility

[x] Fix some HTML headers (h1, h2, h3). There must be no skipped levels (h1 > h3 without h2 for example.)
[x] Remove duplicated meta tags (<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no"/> appeared twice)
[x] Replace some span by div (span cannot contain any block-level element (h1, div, etc...)
[x] Replace some div by span (label tag cannot contain any div)
[x] Correct a small english typo ("This is the your primary email address")
[x] correct notification red color (missed in a previous commit)

Functional tests:

[x] actuator-form-login
[x] actuator-form-login-ldap
[x] actuator-form-login-ldap-userauthz
[x] shibboleth-idp-service-selection

Hello @mmoayyed :)
Can you run the github ci on this pr, to be sure i forgot nothing please ? (Or can i myself ?)

Sure. We'll have to run it for you.

@apereocas-bot runci

Thank you :)

I've corrected these Puppeeters tests, that now pass successfully :

[x] single-logout-cas-frontchannel
[x] mfa-gauth-register
[x] message-bundles-custom
[x] saml2-idp-login-sp-logout-redirect

delegated-login-cas-idp-scim and scim-login-success seems to fail due to an issue not related to this PR (Error: connect ECONNREFUSED ::1:9666)
@mmoayyed, can you re-run the CI please ?

Sure. Could you resolve the conflicts please?

Conflicts solved. You can run the CI ;)

@apereocas-bot runci

Thank you!
It was my last PR for this year.
I wish you a very happy holiday season :)

Thank you very much for your help and contributions. Hope to see more of you in the new year. Happy holidays!


keras-nightly: cannot import name '_tf_keras' from 'keras.api'

Repro steps:
~: pip install --upgrade keras-nightly
Requirement already satisfied: keras-nightly in ./.local/lib/python3.12/lib/python3.12/site-packages (3.3.3.dev2024061803)
~: pip install --upgrade tensorflow
Requirement already satisfied: tensorflow in ./.local/lib/python3.12/lib/python3.12/site-packages (2.16.1)
~: python
Python 3.12.4 (main, Jun  6 2024, 18:26:44) [Clang 15.0.0 (clang-15<IP_ADDRESS>)] on darwin
>>> import keras
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "~/.local/lib/python3.12/lib/python3.12/site-packages/keras/__init__.py", line 23, in <module>
    from keras.api import _tf_keras
ImportError: cannot import name '_tf_keras' from 'keras.api' (~/.local/lib/python3.12/lib/python3.12/site-packages/keras/api/__init__.py)

#19864

I cannot reproduce; it's likely an environment conflict on your end. Try first uninstalling all keras / tensorflow packages:
pip uninstall keras keras-nightly tensorflow tf-nightly -y

~: pip uninstall keras keras-nightly tensorflow tf-nightly -y
WARNING: Skipping keras as it is not installed.
WARNING: Skipping keras-nightly as it is not installed.
WARNING: Skipping tensorflow as it is not installed.
WARNING: Skipping tf-nightly as it is not installed.
~: pip install keras-nightly tf-nightly
Collecting keras-nightly
  Downloading keras_nightly-3.3.3.dev2024061803-py3-none-any.whl.metadata (5.8 kB)
Collecting tf-nightly
  Downloading tf_nightly-2.18.0.dev20240617-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (4.2 kB)
~: python
Python 3.12.3 [GCC 13.2.0] on linux
>>> import keras
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "~/.local/lib/python3.12/lib/python3.12/site-packages/keras/__init__.py", line 23, in <module>
    from keras.api import _tf_keras
ImportError: cannot import name '_tf_keras' from 'keras.api' (/home/lutzroeder/.local/lib/python3.12/lib/python3.12/site-packages/keras/api/__init__.py)

I can repro now, I think I was hitting a cache previously. Will fix it, it's a simple issue.


feat(#557): Use direct xpath instead of global in XmlMethod implementation

Fix the bug related to global xpaths in XmlMethod implementation.
Closes: #557
History:

feat(#557): use direct xpath instead of global xpath

PR-Codex overview
The focus of this PR is to update the XPath expressions in XmlMethod.java to target elements relative to the current node.
Detailed summary

Updated XPath expressions to target elements relative to the current node instead of absolute paths.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 7min)

@rultor release, tag is 0.4.2, title is 0.4.2.

@rultor release, tag is 0.4.2, title is 0.4.2.

@volodya-lombrozo OK, I will release it now. Please check the progress here

@rultor release, tag is 0.4.2, title is 0.4.2.

@volodya-lombrozo Oops, I failed. You can see the full log here (spent 10min)

@rultor release, tag is 0.4.2, title is 0.4.2.

@rultor release, tag is 0.4.2, title is 0.4.2.

@volodya-lombrozo Oops, I failed. You can see the full log here (spent 9min)

@rultor release, tag is 0.4.2, title is 0.4.2.

@rultor release, tag is 0.4.2, title is 0.4.2.

@volodya-lombrozo OK, I will release it now. Please check the progress here

@rultor release, tag is 0.4.2, title is 0.4.2.

@rultor release, tag is 0.4.2, title is 0.4.2.

@volodya-lombrozo OK, I will release it now. Please check the progress here

@rultor release, tag is 0.4.2, title is 0.4.2.

@volodya-lombrozo Oops, I failed. You can see the full log here (spent 9min)

@rultor release, tag is 0.4.2, title is 0.4.2.

@rultor release, tag is 0.4.2, title is 0.4.2.

@volodya-lombrozo OK, I will release it now. Please check the progress here

@rultor release, tag is 0.4.2, title is 0.4.2.

@volodya-lombrozo Done! FYI, the full log is here (took me 10min)


PETSc fails when computing absorption probabilities

When computing absorption probabilities on the lung using g_fwd.compute_absorption_probabilities(use_petsc=True, solver='gmres', n_jobs=8), my kernel dies and I get the following error message in the terminal:

[0]PETSC ERROR: ------------------------------------------------------------------------
[0]PETSC ERROR: Caught signal number 13 Broken Pipe: Likely while reading or writing to a socket
[0]PETSC ERROR: Try option -start_in_debugger or -on_error_attach_debugger
[0]PETSC ERROR: or see https://www.mcs.anl.gov/petsc/documentation/faq.html#valgrind
[0]PETSC ERROR: or try http://valgrind.org on GNU/linux and Apple Mac OS X to find memory corruption errors
[0]PETSC ERROR: configure using --with-debugging=yes, recompile, link, and run
[0]PETSC ERROR: to get more information on the crash.
application called MPI_Abort(MPI_COMM_WORLD, 50162059) - process 0
[unset]: write_line error; fd=-1 buf=:cmd=abort exitcode=50162059
:

Versions:
cellrank==1.1.0+gb36eac8 scanpy==1.6.0 anndata==0.7.4 numpy==1.19.5 numba==0.52.0 scipy==1.5.3 pandas==1.1.3 scikit-learn==0.23.2 statsmodels==0.12.0 python-igraph==0.8.3 scvelo==0.2.2 pygam==0.8.0 matplotlib==3.2.2 seaborn==0.11.0

...

Update 1: Running just g_fwd.compute_absorption_probabilities(n_jobs=8) works fine.
Update 2: Using just a single core, i.e. g_fwd.compute_absorption_probabilities(use_petsc=True, solver='gmres', n_jobs=1) also works fine.

Which multiprocessing backed are you using (e.g. default is loky). Could try with backend='threading'.
Could you also try with e.g. n_jobs=2?

Also, do you have longer error log? The one above does not really help.

Updates:

2 jobs works
4 jobs works

6 jobs gives
---------------------------------------------------------------------------
TerminatedWorkerError                     Traceback (most recent call last)
<ipython-input-31-fb9aa4c52d9e> in <module>
----> 1 g_fwd.compute_absorption_probabilities(use_petsc=True, solver='gmres', n_jobs=6)

~/Projects/cellrank/cellrank/tl/estimators/_base_estimator.py in compute_absorption_probabilities(self, keys, check_irred, solver, use_petsc, time_to_absorption, n_jobs, backend, show_progress_bar, tol, preconditioner)
    479
    480         # solve the linear system of equations
--> 481         mat_x = _solve_lin_system(
    482             q,
    483             s,

~/Projects/cellrank/cellrank/tl/_linear_solver.py in _solve_lin_system(mat_a, mat_b, solver, use_petsc, preconditioner, n_jobs, backend, tol, use_eye, show_progress_bar)
    463
    464         # can't pass PETSc matrix - not pickleable
--> 465         mat_x, n_converged = parallelize(
    466             _solve_many_sparse_problems_petsc,
    467             mat_b,

~/Projects/cellrank/cellrank/ul/_parallelize.py in wrapper(*args, **kwargs)
     99             pbar, queue, thread = None, None, None
    100
--> 101         res = jl.Parallel(n_jobs=n_jobs, backend=backend)(
    102             jl.delayed(callback)(
    103                 *((i, cs) if use_ixs else (cs,)),

~/miniconda3/envs/cellrank_revision/lib/python3.8/site-packages/joblib/parallel.py in __call__(self, iterable)
   1052
   1053             with self._backend.retrieval_context():
-> 1054                 self.retrieve()
   1055             # Make sure that we get a last message telling us we are done
   1056             elapsed_time = time.time() - self._start_time

~/miniconda3/envs/cellrank_revision/lib/python3.8/site-packages/joblib/parallel.py in retrieve(self)
    931             try:
    932                 if getattr(self._backend, 'supports_timeout', False):
--> 933                     self._output.extend(job.get(timeout=self.timeout))
    934                 else:
    935                     self._output.extend(job.get())

~/miniconda3/envs/cellrank_revision/lib/python3.8/site-packages/joblib/_parallel_backends.py in wrap_future_result(future, timeout)
    540         AsyncResults.get from multiprocessing."""
    541         try:
--> 542             return future.result(timeout=timeout)
    543         except CfTimeoutError as e:
    544             raise TimeoutError from e

~/miniconda3/envs/cellrank_revision/lib/python3.8/concurrent/futures/_base.py in result(self, timeout)
    437                 raise CancelledError()
    438             elif self._state == FINISHED:
--> 439                 return self.__get_result()
    440             else:
    441                 raise TimeoutError()

~/miniconda3/envs/cellrank_revision/lib/python3.8/concurrent/futures/_base.py in __get_result(self)
    386     def __get_result(self):
    387         if self._exception:
--> 388             raise self._exception
    389         else:
    390             return self._result

TerminatedWorkerError: A worker process managed by the executor was unexpectedly terminated. This could be caused by a segmentation fault while calling the function or by an excessive memory usage causing the Operating System to kill the worker.

The exit codes of the workers are {SIGABRT(-6), SIGABRT(-6), SIGABRT(-6), SIGABRT(-6), SIGABRT(-6)}

Oh, after that error, also 2 jobs fails. Let me restart my kernel and try again.

Okay, after restarting the kernel, it does work with 6 jobs.

For 8 jobs, it still fails, longer error log below
libc++abi.dylib: terminating with uncaught exception of type std::runtime_error: Couldn't close file
libc++abi.dylib: terminating with uncaught exception of type std::runtime_error: Couldn't close file
[0]PETSC ERROR: ------------------------------------------------------------------------
[0]PETSC ERROR: Caught signal number 13 Broken Pipe: Likely while reading or writing to a socket
[0]PETSC ERROR: Try option -start_in_debugger or -on_error_attach_debugger
[0]PETSC ERROR: or see https://www.mcs.anl.gov/petsc/documentation/faq.html#valgrind
[0]PETSC ERROR: or try http://valgrind.org on GNU/linux and Apple Mac OS X to find memory corruption errors
[0]PETSC ERROR: configure using --with-debugging=yes, recompile, link, and run
[0]PETSC ERROR: to get more information on the crash.
application called MPI_Abort(MPI_COMM_WORLD, 50162059) - process 0
[unset]: write_line error; fd=-1 buf=:cmd=abort exitcode=50162059
:
system msg for write_line failure : Bad file descriptor
/Users/marius/miniconda3/envs/cellrank_revision/lib/python3.8/site-packages/joblib/externals/loky/backend/resource_tracker.py:318: UserWarning: resource_tracker: There appear to be 2 leaked file objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '
/Users/marius/miniconda3/envs/cellrank_revision/lib/python3.8/site-packages/joblib/externals/loky/backend/resource_tracker.py:318: UserWarning: resource_tracker: There appear to be 8 leaked semlock objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '
/Users/marius/miniconda3/envs/cellrank_revision/lib/python3.8/site-packages/joblib/externals/loky/backend/resource_tracker.py:318: UserWarning: resource_tracker: There appear to be 2 leaked folder objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '
/Users/marius/miniconda3/envs/cellrank_revision/lib/python3.8/site-packages/joblib/externals/loky/backend/resource_tracker.py:333: UserWarning: resource_tracker: /var/folders/mx/0hyv8t2s26jdj79f55kvc_b80000gn/T/joblib_memmapping_folder_5593_40c65181fc744a9ca57ec0230f7941dd_2713869d33424586bc1f571723ca1820: FileNotFoundError(2, 'No such file or directory')
  warnings.warn('resource_tracker: %s: %r' % (name, e))
[I 10:11:57.137 NotebookApp] KernelRestarter: restarting kernel (1/5), keep random ports
WARNING:root:kernel 04d152c9-f0a9-4298-8f39-1a8425757e3b restarted

Using backend='threading as in g_fwd.compute_absorption_probabilities(use_petsc=True, solver='gmres', n_jobs=8, backend='threading') works!

I'm closing this, I think this problem is specific to my machine.

Hello, I've been using CellRank recently, and have been running into this same error, so far only when running cr.tl.lineages() with the parameter backwards=True, as well as with cr.tl.initial_states(). I'm using the most recent version of CellRank (v1.1.0)

Hello, I've been using CellRank recently, and have been running into this same error, so far only when running cr.tl.lineages() with the parameter backwards=True, as well as with cr.tl.initial_states(). I'm using the most recent version of CellRank (v1.1.0)

Hi @BioFalcon, the most recent version is 1.2, could you update and try again? If that doesn't help, could you try passing backend='threading'? The above is a parallelisation issue, if nothing else helps, you can probably resolve this by turning off parallelisation, i.e. setting n_jobs=1.

Hi, I tried doing all the above and still getting the error. Somehow, there is output being inserted into the adata object, but I was just wondering if this might impact downstream analyses.

Hi @BioFalcon, I will reopen this issue for you and @michalk8 will look into this with you - however, that will take until next week as it's exam season at the moment, sorry about that!

@BioFalcon, can you please post the error you are getting when you run the function without parallelisation, i.e. cr.tl.lineages(n_jobs=1)?

I'm assuming that this issue has been solved.

Hi,

I get the same issue. tried setting all parameters as discussed here:

cr.tl.lineages(adata,backward=False,n_jobs=1, solver='gmres',backend='threading')

My jupyter kernel still crashes.

However, I receive the following warning:

Computing absorption probabilities
WARNING: There is only `1` terminal state, all cells will have probability `1` of going there

as a workaround, I  tried to use the workaround with kernels and estimators, to no avail:

from cellrank.tl.kernels import VelocityKernel

vk = VelocityKernel(adata)

vk.compute_transition_matrix()

from cellrank.tl.kernels import ConnectivityKernel

ck = ConnectivityKernel(adata).compute_transition_matrix()

combined_kernel = 0.8 * vk + 0.2 * ck

from cellrank.tl.estimators import GPCCA

g = GPCCA(combined_kernel)
print(g)

g.compute_schur(n_components=5)
g.plot_spectrum()

Always crashing at the step, where the shur decomposition is calculated.

This is the output of cellrank.logging.print_versions():

cellrank==1.5.1 scanpy==1.9.1 anndata==0.8.0 numpy==1.21.2
numba==0.56.3 scipy==1.9.3 pandas==1.5.1 pygpcca==1.0.4
scikit-learn==1.1.3 statsmodels==0.13.5 python-igraph==0.10.2
scvelo==0.2.4 pygam==0.8.0 matplotlib==3.6.2 seaborn==0.11.2

Thanks!

Nevermind, reinstalling environment seemed to help


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bridge documentation update

Bridge documentation update
Hey guys,
The Bridge got a little bit of a face-lift and when (hopefully this week) the branch will finally get merged into main, this document/page needs some update and we go to audit soon maybe it will be helpful to have up-to-date.
Here are my notes what shall be changed, so that hopefully it will be easier to incorporate.
1.
Instead of: The bridge is a set of smart contracts and a frontend web app that allow you to send testnet ETH and ERC-20 tokens between Sepolia and Taiko.

use: The bridge is a set of smart contracts and a frontend web app that allow you to send testnet ETH and ERC-20, ERC-1155 and ERC-721 tokens between Sepolia and Taiko.
2.

sendMessage diagram: shall be slightly modified, I’d use 2-distinguished diagram. Ether only vs. Others*

(others can be: ERC20, or ERC20+Ether, ERC721 or ERC721+Ether, ERC1155, ERC1155+Ether but more on them later.)

The first case (Ether only):

-  The user/UI interacts directly with the Bridge, (via sendMessage) and not with the ‘’TokenVault”.

 - The L2: EtherVault observation is correct (only on L2) but this is not a ‘decision object’ so I feel the SignalService ‘path/route of a decision’ is a bit confusing. This storage of the message happens independently. (I know we wanted to express this, but the diagram is a bit confusing.)

 The other cases (ERC20, etc..):

-  The user/UI interacts directly with the ERCXXXVault contracts, and this vault contract then calls the Bridge contract which will proceed to the SignalService.

- We have both ERCXXXVault contracts deployed on both of the chains, and this will hold custody.

3.
processMessage diagram: So the first point of contact here is indeed the Bridge, if Ether only, it will distribute the ether (the way it is described) but in case message.data encodes some function data then it is an ERCXXX transfer and if that is the case then the Bridge will call (via InvokMessageCall) the appropriate ERCXXVault.
Worth to mention also:

A “new” (or not really new but not implemented) feature is this:
You are able to bridge ether only - (this is not new) - but now you are also able to bridge ether (optionally) in the exact same transaction where you are bridging other tokens. ( ERC20+ether, ERC721+ether, etc.).

How to technically bridge ether:

1) setting the to to the recipient address and data to "", and value to the amount of ether,

2) setting the to to an app on the destination change, and value to the amount of ether, and inside the app, a function will further forward the ether to the recipient address encoded in data.

4.

Instead of: A contract for the ERC-20 must first be deployed on the destination chain (will be done automatically by the TokenVault if not already deployed)
… be done automatically by the ERCXXVault  if …
5.

Instead of: Call sendERC20 on the source chain TokenVault, this will transfer the amount by using the … to the TokenVault….
… Call sendToken on the source chain …. to the ERCXXXVault.
6.
Between point 2 and 3 in (Bridge from canonical chain to destination chain section) , there is a step, where the ERCXXXVault will call the Bridge contract which then will construct the message and then invoke the SignalService.
7.
Instead of: It will attempt to invoke the receiveERC20 method encoded in the message
invoke the receiveToken method
So every sendERC20 and receiveERC20 can be replaced with sendToken and receiveToken
Rest is fine, maybe we shall mention that other token + ether thing + that the ERC721 and ERC1155 works the same way as the ERC-20.
Let me know if you have any questions (here or DM).

Lets get this done by Sept 15th? @d1onys1us

For point 3. Is it possible to extend the concept by packing multiple transfers into a multicall?


test

test

✅ By submitting this issue, I have verified the following

[x] Checked to see if the issue has already been discussed before 🤔
[x] If proposing new content to be added, made sure enough details were provided 🔍

test


Missing solution for dictionary challenge

The instructor mentions at the end of "Dictionary Solution" lesson that it will be available on the repo, but I don't see it there. After contacting support via Intercom, I was directed to leave an issue on the repo. Thanks!
I would imagine it would be this file, but filled in https://github.com/mike-works/typescript-fundamentals/blob/362113346f26fde233919a04a9e8a87f81c8e319/challenges/dict/src/index.ts

here is the sol i have(map from video, reduce just mocked the map style)
export type Dict<T> = {
    [anyKeyWeWant: string]: T | undefined;
};

// Array.prototype.map, but for Dict
export function mapDict<T, S>(
    dict: Dict<T>,
    fn: (arg: T, idx: number) => S
): Dict<S> {
    const out: Dict<S> = {};
    Object.keys(dict).forEach((keyName, idx: number) => {
        const thisItem = dict[keyName];
        if (typeof thisItem !== 'undefined') {
            out[keyName] = fn(thisItem, idx);
        };
    });
    return out;
}

// Array.prototype.reduce, but for Dict
export function reduceDict<T, S>(
    dict: Dict<T>,
    fn: (cur: T, sum: S ) => S,
    initVal: S
) {
    let out: S = initVal;
    Object.keys(dict).forEach((keyName) => {
        const thisItem = dict[keyName];
        if (typeof thisItem !== 'undefined') {
            out = fn(thisItem, out);
        };
    });
    return out;
}


Does eosjs support multisig transaction?

Does eosjs support multisig transactions? Such as modify active and owner keys to multiple keys, proposal, approval and execution?

proposed transaction: https://github.com/EOSIO/eosjs/issues/284#issuecomment-412168935


Added Horizontal Obstacles

I think maybe it would be better to integrate the lane choosing logit in the method that is already demanded for that, rather than adding it after that method


Accentuation prints char without accentuation + char with accentuation

Describe the bug
When inputting a char with accentuation it prints the char concatenated with the char with accentuation, example below.
à -> aá
â -> aâ
Keyboard Layout: PT-PT
Environment

OS: Arch Linux
scrcpy version: 1.17
installation method: pamac
device model: Huawei Nova 5T
Android version: 10

I believe the issue is on the keyboard read and not on receive since copy-pasting accentuated char has no issue.

I need to correct the description, I can note that it prints the unaccented char and later it prints the char again accentuated meaning it is not printing them concatenated but sequencing them.

scrcpy --prefer-text

https://github.com/Genymobile/scrcpy#text-injection-preference

thank you


fix: Search tag when searching nothing

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #5166
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 10 usd

😍 Perfect, I’ve sent the payout
💵 $10 @ 34.86 USD/KSM ~ 0.287 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x588c7aca4eab5d48bca2859ccd1d4b20fad6da1ac29344a639cd820bc77edf9c
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Is it reasonable to register OptionalAsNoContentMessageBodyWriter at the same time as all the ExceptionMappers?

Right now, it appears that client apps ought to be calling ExceptionMappers.visitExceptionMappers, followed by registering OptionalAsNoContentMessageBodyWriter separately, making it easy to forget. Is it reasonable to include OptionalAsNoContentMessageBodyWriter in the registering visitor? (this probably needs a rename of ExceptionMappers)

we've changed registration to use a Jersey feature, which makes this all sane.


fix: workspace context menu is not permission driven

Description

Now even more workspace context menu options are permission driven.

Also now when the user doesn't have any Action Creation options in the omnibar, Create New option will be hidden.

Fixes #18643
Fixes #18030
Media

https://www.loom.com/share/d20e0b22eb524c389565a60ef41a8196
https://www.loom.com/share/27cb577e321449588d0f697f2764ea2f

Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual

Checklist:
Dev activity

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

/ok-to-test sha=d285669

/ok-to-test sha=c896580

/ok-to-test sha=bbecf3f

/ok-to-test sha=89d0271

@berzerkeer Just sharing a loom video here notifying about something I noticed that'll happen when this PR gets merged:
https://www.loom.com/share/d20bd7ab520b4d4a9fde44f5216b8e2a

@berzerkeer
Developer access before:

Developer access after:

@berzerkeer Just sharing a loom video here notifying about something I noticed that'll happen when this PR gets merged: https://www.loom.com/share/d20bd7ab520b4d4a9fde44f5216b8e2a

@trishaanand So before developer access and app viewer access users were only able to see Leave workspace option in the menu options. Now since I added a permission check to Members ,Share & Leave workspace options which is only visible if the user is a member of the workspace (inviteUsers:workspace) . These things are visible means , the server is sending the user with App viewer Developer who are also members of the workspace the said permission. Is this correct or should it be different ?

@berzerkeer Just sharing a loom video here notifying about something I noticed that'll happen when this PR gets merged: https://www.loom.com/share/d20bd7ab520b4d4a9fde44f5216b8e2a

@trishaanand So before developer access and app viewer access users were only able to see Leave workspace option in the menu options. Now since I added a permission check to Members ,Share & Leave workspace options which is only visible if the user is a member of the workspace (inviteUsers:workspace) . These things are visible means , the server is sending the user with App viewer Developer who are also members of the workspace the said permission. Is this correct or should it be different ?

No, we would have to revert to bring this back to the existing behaviour. I suggest the following : You must have manage:workspace to see Import (which is AS IS today, correct)? You must have manage:workspace + inviteUsers:workspace to see Members page You must have inviteUsers:workspace permission to leave the workspace
I do not understand why there is a Share option when we give the Share modal right above.

Actually, the other day on call we discussed that Import is basically a create new application. As of today the +New on workspace is behind manage:workspaceApplications hence I used the same permission for Import. Should this be manage:workspace then ? thanks.

/ok-to-test sha=e062764

/ok-to-test sha=b3ed56f


Docs/fix 2698

Description
"Just" a documentation update:

deleted obsolete --cluster references in the tutorial section
rephrased MPI section

References to SLURM Executor Plugin and snakemake-profiles in both changed paragraphs.
@johanneskoester today, I am not particularly good with words, but I hope its ok-ish.

PS this PR addresses issue #2698
PPS I just hoped to get my way with the semantic commits ...

At #2698 it has been commented that the API docs are missing, too. Not sure: Have these been auto-generated in the past @johanneskoester ?

Yes, they are auto-generated. They are also there, but need some polishing: https://snakemake-api.readthedocs.io/en/stable/
And a link from the main docs as well.


Cannot access class 'kotlinx.datetime.LocalDate'. Check your module classpath for missing or conflicting dependencies

Cannot access class 'kotlinx.datetime.LocalDate'. Check your module classpath for missing or conflicting dependencies error for below  code         Column {
KalendarEarthy(
modifier = Modifier.fillMaxSize()
)
}

The issue you're encountering isn't a problem. The function employs the Kotlin datetime library as opposed to the default one. To resolve the error, simply add the following dependency to your build.gradle file:
implementation("org.jetbrains.kotlinx:kotlinx-datetime:0.5.0")

Please let me know if this solution works for you. 😊


Evaluation,what is the format of eval_file?

I want to run eval.py. But I don't know what is the format of --eval_file.

Same as your train.txt file


Colors infinite loop

Due to (the intentional) broken colors.js version 1.4.1 sfcc-ci is currently not working.
The solution is to fix the dependency to version 1.4.0.
Long term I would try to switch to a community fork of colors as I except more trouble with this package.
I also noticed that colors is part of the dependencies and dev-dependencies, which can be removed.

+1, this is breaking our CI pipelines right now

Might be related but our pipeline is running for 20 min now only on sfcc-ci auth.

So this is the reason.
https://developers.slashdot.org/story/22/01/09/2336239/open-source-developer-intentionally-corrupts-his-own-widely-used-libraries
https://github.com/Marak/colors.js/issues/285

Temporarily fixed by patching the package.
https://www.npmjs.com/package/patch-package

Fix with release https://github.com/SalesforceCommerceCloud/sfcc-ci/releases/tag/v2.8.1


fix(cuboid_transformer):  Modified the argument order of '_generalize_padding' so that 'pad_h' comes before 'pad_w'.

Issue:
In Line 280 of the PatchMerging3D class, the order of the pad_h and pad_w arguments in _generalize_padding is incorrect.
AS-IS: x = _generalize_padding(x, pad_t, pad_w, pad_h, padding_type=self.padding_type)
TO-BE: x = _generalize_padding(x, pad_t, pad_h, pad_w, padding_type=self.padding_type)
Description of changes:
The order of arguments in '_generalize_padding' has been modified so that pad_h comes before pad_w.
--
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for fixing! Could you help delete unnecessary changes such as indentation and spacing?

It's done in PR #48. Thank you!


Disable cachin

Hello,
i would like to know how I can disable the caching on disk. I tried to do this:
// NSURLRequest *request = [[NSURLRequest alloc] initWithURL:_photoURL
//cachePolicy:NSURLRequestReturnCacheDataElseLoad
// timeoutInterval:0];
        NSURLRequest *request = [NSURLRequest requestWithURL:_photoURL];

But it has the same effect for me... its still caching.
Thank you in advance.
Greets

I have the same question. How to disable caching. I cant store images at all due to security reasons.


once accessory is removed it can't be re-added

HomeKit: Starting server

HomeKit: Using existing accessory ID: 18:EE:9A:93:4A:C4
HomeKit: Found admin pairing with EC33C480-1E6A-4203-9615-AEC0BA641370, disabling pair setup
HomeKit: Waiting for IP

How do you overcome this situation?

Sorry for the dumb question,  but if you happen to get stuck in the situation you need to erase.
make -C examples/led_strip erase_flash

I actually found this great small piece of code to generate MAC addresses.
I hope it's something you could implement.
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <time.h>

int main() {
  int i, tp;

  srand(time(NULL) + getpid());

  for (i = 0; i < 6; i++) {
    tp = rand() % 256;
    printf("%s%X%s", tp < 16 ? "0" : "", tp, i < 5 ? ":" : "\n");
  }
  return EXIT_SUCCESS;
}

source: https://gist.github.com/slim/2401338

@idumzaes when device knows it was removed from HomeKit (that means that device was online and accessible from Home.app when it was removed), then there is no problem of reconnecting it back, since when Home.app removes accessory, it connects to it and sends command to remove pairing. When last admin pairing is removed, accessory switches back to discovery and pairing mode.
If you removed accessory offline, then you need to factory reset your accessory. Various accessories have different ways to do that. E.g. with Sonoff Basic people usually implement reset on long button press. Some lightbulbs (e.g. LIFX) require you to turn it on and off for 1 second 5 times in a row. So certified hardware also has this reset problem if you do it offline. My suggestion: just don't unpair offline or be prepared to do factory reset/flash erase.

@maximkulkin @RavenSystem
That makes a lot of sense.
Thanks for the thorough explanation guys.

The same issue.
When I added the fourth step of the device, the phone was stuck, and the phone was not added successfully. But the chip starts reporting the same log and the phone cannot be added again.
s it possible to consider not storing the admin pair?

homekit_server_reset()

@RavenSystem , could you please give me some hint of how (or where) to use homekit_server_reset() function?
I've tryed to insert it into setup() at ESP32_homekit.ino, but it didn't work.
Thanks.


Roll Engine from 00fa56f1adc7 to 0576eefdb84c (1 revision)

https://github.com/flutter/engine/compare/00fa56f1adc7...0576eefdb84c
2021-06-08<EMAIL_ADDRESS>Roll Skia from 89d460f27bd4 to 70c21e34ca4f (5 revisions) (flutter/engine#26639)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Preview of linked post in blockquote extends past background

The preview of a linked post in a block-quote will extend past the right boundary of the yellow background.
Here's what I mean:

A minor issue, just FYI. This example is from here.

a slight CSS tweak will fix that:
    overflow-wrap: break-word;
    width: 630px;
    padding: 10px;

...should do.
Will push an update soon...

Awesome!  just close this and comment when you do.

done @cullub :)


Add time window counter sensor

Add time window counter sensor to have the number of lightning during the current time window only

Can you give me a feedback ?


ffado: 2.4.1 -> 2.4.2

Motivation for this change
Bumped version and did some fixes to binaries.

fixed broken ffado-diag (hardcoded kernel module paths)
fixed ffado-mixer (broken Qt5)
enabled ffado-test
added desktop file for ffado-mixer
formatted using nixpkgs-fmt

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] 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 nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)

[x] ffado-diag
[x] ffado-test
[x] ffado-mixer

[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@goibhniu could you please take a look?

A dependent package (tambura) fails to build both with this change and in the master:
Failing package review:

nixpkgs-review rev HEAD
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0
$ git worktree add /home/miminar/.cache/nixpkgs-review/rev-bb317ecbc684fddc12cc78d607272826ed29a9c7/nixpkgs 6c48c7b81f230e469becfe7f8447c434a955f08b
Preparing worktree (detached HEAD 6c48c7b81f2)
Updating files: 100% (21248/21248), done.
HEAD is now at 6c48c7b81f2 fd: 7.4.0 -> 7.5.0
$ nix-env -f /home/miminar/.cache/nixpkgs-review/rev-bb317ecbc684fddc12cc78d607272826ed29a9c7/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit bb317ecbc684fddc12cc78d607272826ed29a9c7
Auto-merging pkgs/top-level/all-packages.nix
Automatic merge went well; stopped before committing as requested
$ nix-env -f /home/miminar/.cache/nixpkgs-review/rev-bb317ecbc684fddc12cc78d607272826ed29a9c7/nixpkgs -qaP --xml --out-path --show-trace --meta
19 package updated:
aj-snapshot carla faust2jack faust2jaqt ffado (2.4.1 → 2.4.2) ffado (2.4.1 → 2.4.2) giada jack1 jack2 jackmix lsp-plugins ltc-tools luppp pulseeffects shotcut soundtracker Tambura tetraproc tuxguitar

$ nix build --no-link --keep-going --option build-use-sandbox relaxed -f /home/miminar/.cache/nixpkgs-review/rev-bb317ecbc684fddc12cc78d607272826ed29a9c7/build.nix
warning: ignoring the user-specified setting 'sandbox', because it is a restricted setting and you are not a trusted user
waiting for locks or build slots...
builder for '/nix/store/505csnqn7ic0z93hxbpvy9fh9w4yblmy-Tambura-1.0.drv' failed with exit code 1; last 10 log lines:
  building
  sh: which: not found
  sh: which: not found
  sh: which: not found
  /nix/store/4364p9986svv5fzski7dj1yfy3v07xni-faust2jaqt-2.20.2/bin/.faust2jaqt-wrapped: line 29: which: command not found
  /nix/store/4364p9986svv5fzski7dj1yfy3v07xni-faust2jaqt-2.20.2/bin/.faust2jaqt-wrapped: line 29: which: command not found
  sh: which: not found

  start parser
  ERROR : unable to open file oscillator.lib
cannot build derivation '/nix/store/ys7871z1z0h6i5bcfgnx63v34zqrihh9-env.drv': 1 dependencies couldn't be built
[4 built (1 failed)]
error: build of '/nix/store/ys7871z1z0h6i5bcfgnx63v34zqrihh9-env.drv' failed
1 package failed to build:
tambura

17 package built:
aj-snapshot carla faust2jack faust2jaqt ffado giada jack1 jack2 jackmix_jack1 lsp-plugins ltc-tools luppp pulseeffects shotcut soundtracker tetraproc tuxguitar

Failing build in master:

nix-build $(pwd) -A tambura
these derivations will be built:
  /nix/store/w4zc1a582ja8dmfqs94nvzglcpk7j4d5-Tambura-1.0.drv
these paths will be fetched (1.39 MiB download, 8.42 MiB unpacked):
  /nix/store/1s1gvnww2rzpzz3wds79nvgdpmd8xkc5-ffado-2.4.1
  /nix/store/91wxhs4rp6dacz76ya7rrkdh3rg8anjj-jack2-1.9.14
  /nix/store/dja1nhnr23f4r4324ifxs52dzx50lacr-faust2jaqt-2.20.2
copying path '/nix/store/1s1gvnww2rzpzz3wds79nvgdpmd8xkc5-ffado-2.4.1' from 'https://cache.nixos.org'...
copying path '/nix/store/91wxhs4rp6dacz76ya7rrkdh3rg8anjj-jack2-1.9.14' from 'https://cache.nixos.org'...
copying path '/nix/store/dja1nhnr23f4r4324ifxs52dzx50lacr-faust2jaqt-2.20.2' from 'https://cache.nixos.org'...
building '/nix/store/w4zc1a582ja8dmfqs94nvzglcpk7j4d5-Tambura-1.0.drv'...
unpacking sources
unpacking source archive /nix/store/a58fp3a7f3dpxiyhv1gcw2r1jp1lrk6f-source
source root is source
patching sources
configuring
no configure script, doing nothing
building
sh: which: not found
sh: which: not found
sh: which: not found
/nix/store/dja1nhnr23f4r4324ifxs52dzx50lacr-faust2jaqt-2.20.2/bin/.faust2jaqt-wrapped: line 29: which: command not found
/nix/store/dja1nhnr23f4r4324ifxs52dzx50lacr-faust2jaqt-2.20.2/bin/.faust2jaqt-wrapped: line 29: which: command not found
sh: which: not found

start parser
ERROR : unable to open file oscillator.lib
builder for '/nix/store/w4zc1a582ja8dmfqs94nvzglcpk7j4d5-Tambura-1.0.drv' failed with exit code 1
error: build of '/nix/store/w4zc1a582ja8dmfqs94nvzglcpk7j4d5-Tambura-1.0.drv' failed

That can be addressed in a follow-up I guess.
cc @magnetophon

@GrahamcOfBorg build ffado

@michojel That tambura error is unrelated.
It will be fixed when https://github.com/NixOS/nixpkgs/pull/82650 is merged.


jo_sprite_set_palette() Doesn't Work in JO_480p Mode

I'll stop annoying you now, just wanted to file another bug report.
For my next project I want to use palette swaps for my characters and use 480i mode. "demo - hardcoded image" sets a palette for an 8bit sprite. When compiled for standard resolution the demo works. When I added "JO_480p = 1" to the makefile the 8 bit image still displays, but it is no longer using the palette.
Is this an issue with Jo Engine or with SGL?

Neither. This mode is still in developement. Don't use it :)


Notification emails are sent "to:" all recipients in the clear.

Moderation notification emails are sent with every recipient's email visible in the "to:" line, which seems yucky. BCC-ing everyone makes more sense.

A better solution would be to send a distinct email to each recipient. This would allow personalisation (such as greetings, and possibly an 'unsubscribe from these notifications' link with an embedded login token), and more importantly, is going to be necessary if/when we implement per-user language settings.

Ah yes, that would be preferable.
I suppose there are some considerations about how mail servers might respond to Wagtail performing bulk emails, but they'd be just as concerned with a massive bcc, so perhaps it's no more problematic. Mailing would need to be queued, if it isn't already.

We previously had code to queue email using Celery when available, to account for this, but took it out because it was effectively pushing more setup demands on the developer without warning them: http://docs.wagtail.io/en/v1.0/releases/1.0.html#celery-no-longer-automatically-used-for-sending-notification-emails .
I think it makes sense for it to be the developer/administrator's responsibility for putting a suitably robust email infrastructure in place - something that Django already provides extensive configuration options for - rather than Wagtail trying to push one particular approach. Perhaps it would be a good idea to add a note about django-celery-email to the performance section of the docs, when we make this change.

So without celery, would that mean the mailing process would be blocking?

I think so, yes - although that's where my knowledge of mail servers gets a bit hazy. It may well be that in the typical setup (SMTP to localhost), it's only blocking for as long as it takes to push the data to the local SMTP server, and the actual sending-across-the-internet bit happens asynchronously.

Created #1973

Implemented in e18f877286308a8a172fcd99a4ea79d9f4aa3c14.


Avoid recreation of ClusterController when config changes

@bratseth please review.
@hmusum @hakonhall @baldersheim this is the alternative.

Yes, this was much simpler than I thought at first—obvious once I realised the original code was written like this 🤦


cli add escape to execute an external command

Something like this
MariaDB [(none)]> \! ps -ef | grep mysql
mysql    12290     1  0 Aug22 ?        00:00:00 /bin/sh /usr/bin/mysqld_safe --basedir=/usr
mysql    12671 12290  1 Aug22 ?        03:17:18 /usr/libexec/mysqld --basedir=/usr --datadir=/var/lib/mysql --plugin-dir=/usr/lib64/mysql/plugin --log-error=/var/log/mariadb/mariadb
root     21612 10086  0 22:15 pts/1    00:00:00 mysql -p
root     21625 21612  0 22:15 pts/1    00:00:00 sh -c  ps -ef | grep mysql
root     21627 21625  0 22:15 pts/1    00:00:00 grep mysql

Doesn't seem relevant to Influx, or I'm not clear on what the request actually is.

If you do not know what the request is, how can you conclude it is not relevant? What kind of engineering education does not teach some basic logics?
I do not know what to say, get someone that does understand. Don’t be lazy.


WG Comms sustainability week presentation

As part of sustainability week we will present about WG comms.
To Do:

[x] Create plan
[x] Create slides
[ ] Finalise slides

Just to have all the doc here too:

link to presentation
brainstorming/notes

We finalised the slides! Also we have a new great presentation and template for Comms (future WG Sustainability Advocacy!).
We can close this issue 😄


fix typo

Overview
Dumper argument should have been dumper

/do-e2e-tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1b3aa31). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #389   +/-   ##
=========================================
  Coverage          ?   73.36%
=========================================
  Files             ?       30
  Lines             ?     2718
  Branches          ?      461
=========================================
  Hits              ?     1994
  Misses            ?      614
  Partials          ?      110

Impacted Files
Coverage Δ

taskcat/_cfn/stack.py
88.33% <ø> (ø)

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 1b3aa31...92b7b4d. Read the comment docs.


Issue with adding the SPM package in a new project (Swift Tools Version 5.8)

I'm getting this error saying that the package is using Swift Tools version 5.8. Ive never seen this before with any SPM packages, so not sure if this is something I should fix on my end, but wanted to drop a message in case it's something that should be adjusted on the Defaults side.

Ah I'm on 14.1, that's probably what the issue, sorry about that!


Ionic 2.2.0 CORS issue.

After updating a current project to Ionic 2.2.0, ng2-ui-auth suddenly stop working properly; im always receiving a CORS errors on Android devicen from a login call. The app still work fine in ionic LiveReload (using the Allow-Control-Allow-Origin chrome plugin). ng2-ui-auth 6.0.0 was the first thing implemented in the projects months ago and worked fine until the update to 2.2.0.
I also tested the 6.1.0 release but same results. Nothing was changed in the code of the project itself, only libs in node_modules were updated in the process. The update itself went without errors or warning (missing dependencies, etc).
Does ng2-ui-auth have been tested on the latest Ionic platform? Is it a known issue?
Thank you for any input you may have.

Im using it with ionic 2.3 with no problem.. can you share some code and the full error you are getting?

Its a simple call to a server where the url is defined in a route variable and the credentials JSON object is passed as post params. Been working fine for the last 3 months.
this.jwtHttp.post(this.route, this.formatData(data)).map(res => <Token>res.json());
The full error is a straight forward basic CORS error:
 {"isTrusted:"true"}
By the way, to update to 2.3 do you only have to set "ionic-angular" to "2.3.0" in package.json and re-install via npm? I could not find a procedure for upgrading to 2.3.

I was able to make it work in a fresh new Ionic 2.3.0 projects. Ill come back and close this issue when i finished my test and fixed the problems. Its clear its has nothing to do with a compatibility between the plugin and Ionic 2.3.0

My problem is fixed. The plugin do work with ionic 2.2.0, ionic 2.3.0 and ionic 3 (beta).


OPRUN-3529: UPSTREAM: : manifests: add hostPath mount for /etc/containers

In order for operator-controller to support disconnected clusters in OCP, it needs access to the /etc/containers directory on the node. This PR adds a hostPath volume mount to give it that access. It also changes the security configuration of the namespace and pod to allow privileged access (which is required for hostPath volume mounts).

/retest

/hold
I want to do some manual verification of this

/retest

/retest

/test openshift-e2e-aws-techpreview

SCC permissions still seem off. Will take a deeper look today.

I stood up a clusterbot cluster for this PR and everything looked okay (operator-controller pods deployed successfully and I could see /etc/containers mounted in as expected). Still not sure why the e2e test would complain about lacking SCC permissions, so let's just try to retest.
/retest

I think I see what's happening here. cluster-olm-operator is deploying both the RBAC and the deployment. I logged into a CI cluster during the test and see the following.

It eventually becomes ready
$ k get deployment
NAME                                     READY   UP-TO-DATE   AVAILABLE   AGE
operator-controller-controller-manager   1/1     1            1           66m

However, it doesn't start out that way:
$ k get events -o json | jq -r '.items[] |select(.type=="Warning" and .reason=="FailedCreate").message'
Error creating: pods "operator-controller-controller-manager-5d54bcd89f-" is forbidden: unable to validate against any security context constraint: provider "privileged": Forbidden: not usable by user or serviceaccount

Why not, you ask? Because the deployment exists prior to the RBAC
$ k get deployment operator-controller-controller-manager -o jsonpath={.metadata.creationTimestamp}
2024-09-25T17:16:20Z

$ k get clusterroles operator-controller-manager-role -o jsonpath={.metadata.creationTimestamp}
2024-09-25T17:16:23Z

$ k get clusterrolebindings operator-controller-manager-rolebinding -o jsonpath={.metadata.creationTimestamp}
2024-09-25T17:16:28Z

The solution seems likely to involve cluster-olm-operator which is responsible for managing operator-controller's rollout.

Started a clusterbot test job that includes this PR and https://github.com/openshift/cluster-olm-operator/pull/69
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/release-openshift-origin-installer-launch-aws-modern/1839051061738868736

https://github.com/openshift/cluster-olm-operator/pull/69 merged so hopefully the e2e issues here are resolved.
/hold cancel
/retest

Test failed again in the same way. Might take some time for there to be a CI payload that includes the fix made in cluster-olm-operator.  I'll try again tomorrow.

Oh nevermind, I see it is in 4.18.0-0.ci-2024-09-26-002917
So let's try again!
/retest


[CI Problems] It seems that pytest-perf is failing for cygwin

I noticed that recent builds for documentation-only changes started failing for cygwin:

https://github.com/pypa/setuptools/runs/7039869057?check_suite_focus=true#step:4:512
https://github.com/pypa/setuptools/runs/7038552895?check_suite_focus=true#step:4:512

The relevant part of the error message seems to be:
>               raise CalledProcessError(retcode, process.args,
                                         output=stdout, stderr=stderr)
E               subprocess.CalledProcessError: Command '['git', 'remote', 'get-url', 'origin']' returned non-zero exit status 128.
/usr/lib/python3.9/subprocess.py:528: CalledProcessError
----------------------------- Captured stderr call -----------------------------
fatal: unsafe repository ('/cygdrive/d/a/setuptools/setuptools' is owned by someone else)
To add an exception for this directory, call:
	git config --global --add safe.directory /cygdrive/d/a/setuptools/setuptools

If required, we can add a git config command before running the tests, but I am not exactly sure about the path for the directory that should be used. Would git config --global --add safe.directory $PWD work? (That looks like a weird workaround).

@jaraco, @lazka, do you have any information that could help to deal with this error?

I think you can set safe.directory to * too

Thank you very much @lazka, I am trying something first in https://github.com/pypa/setuptools/pull/3410, before using a wildcard. Hopefully it will work 🤞

GITHUB_WORKSPACE will be a Windows style path, so I'm not sure if that will work


Provide an accessible option to enable the screen reader for the editor

Testing #2258
We currently provide a setting "editor._screenReaderNVDA" to enable screen reading of the editor contents. To enable early feedback we should also provide an accessible way to enable the screenreading.

Giving bug label to get it in verification


endpoints require a object key in readme.md

In documentation https://github.com/martinheidegger/dns-query#javascript-api says:
    /* Options (optional) */
    endpoints,
    retry: 3, // (optional) retries if a given endpoint fails; -1 = infinite retries; 0 = no retry
    timeout: 4000, // (optional, default=30000) timeout for single requests
    signal, // (optional) an AbortSignal to abort the request
  })

Replace by:
    /* Options (optional) */
    endpoints: endpoints,
    retry: 3, // (optional) retries if a given endpoint fails; -1 = infinite retries; 0 = no retry
    timeout: 4000, // (optional, default=30000) timeout for single requests
    signal, // (optional) an AbortSignal to abort the request
  })

endpoints need a object key.

Thank you for the response @WHK102, the example is using ES2015 shorthand notation for the endpoint documentation. It seemed simple enough for me. Do think the colon is necessary?

For most people today I think yes, in a few more years no, but today yes because it causes confusion because the name of the variable is the same as the name of the property and this causes confusion.
Maybe I'm wrong and you're right, but not all of us know javascript in such a strict way.


Add NIP-50 support and bug fixes

Enhancements

Completed feature request #37 [FR] Support for NIP-50 (search capability) for both content and tags

Support for NIP-50 search queries
Searches tags and content fields
Compatible with searching NIP-99 Classified Listings

Bug fixes

Fixed #39 [BUG] Escape characters included in tag query

Closing as this functionality was added in #42

Will open new FR to continue to add new search features


NIFI-13049: Unify nimbus-jose-jwt dependency overrides

Summary
NIFI-13049
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Thanks for the comment @exceptionfactory, the main purpose of the PR is to fix the CVE-2023-52428 vulnerability, There are two places where still lower versions are used of this library which are affected.

Thanks for the background note @mark-bathori. Resolving the vulnerability finding makes sense. Under the circumstances, it would be better to set the dependency overrides at the applicable level. Can you rework the pull request to scope it to the impacted modules?

Sure I'll rework it.

Merged in https://github.com/apache/nifi/commit/86ddf7758c33b813d895b7db4b593057ef8381da


Component Decoration

A PR targeting #17, @harry-sm suggestion.
Consider this as a POC and take special notice on the Class name change for component.
Use of this would be as follows
import { radi } from '../../radi/src/index';

const { r, mount, component, Component } = radi;

@Component
class MainComponent {
    constructor() {
        this.state = { greeting: 'World' };
    }

    view() {
        return
Hello {this.greeting}!
;
    }
}

mount(new MainComponent(), document.body);

Regards

Nice!


NER labels problem

I want to know the purpose of the "NIL" label. because the CoNLL03 datasets don't contain it

As described in the Section 4.3 of our paper, we address the NER task by enumerating all possible spans in each sentence as entity name candidates, and classifying them into the target entity types or non-entity type. The NIL class corresponds to the non-entity type.


forc-fmt panics with utf chars in comment

forc-fmt -p ./test/src/e2e_vm_tests/test_programs/should_fail/utf_chars

Related  #5006


Cassandra 19126

Thanks for sending a pull request! Here are some tips if you're new here:

Ensure you have added or run the appropriate tests for your PR.
Be sure to keep the PR description updated to reflect all changes.
Write your PR title to summarize what this PR proposes.
If possible, provide a concise example to reproduce the issue for a faster review.
Read our contributor guidelines
If you're making a documentation change, see our guide to documentation contribution

Commit messages should follow the following format:
<One sentence description, usually Jira title or CHANGES.txt summary>

<Optional lengthier description (context on patch)>

patch by <Authors>; reviewed by <Reviewers> for CASSANDRA-#####

Co-authored-by: Name1 <email1>
Co-authored-by: Name2 <email2>

The Cassandra Jira

https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1266/workflows/1fa282fe-4b1b-460a-b2ab-a98202e32372

https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1284/workflows/921dad0e-1ead-4eac-8739-dca875f256b3 (j11)
https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1284/workflows/c03f87e3-75ef-4c0c-92dc-740990e933cb (j17)
tests look good


Adopting this project into Apache Arrow

As described on https://issues.apache.org/jira/browse/ARROW-3050 I wish to fork this project and maintain the codebase as an optional component of Apache Arrow. The Python interface to it would thus ship as a part of the pyarrow library.
There is a natural affinity between the HS2 columnar protocol and the Arrow columnar format. This would provide better performance to users of pandas, and we could also ship first-class native Thrift clients to other languages, like R.
Thoughts?

I support the idea. Obviously there hasn't been much activity on this project recently, and its still missing some significant features, so if the Arrow community wants to take it on that's great.

As an extra data point, Apache Hive is working on adding native Arrow support to some of its protocols. It's conceivable that HS2 (if not already planned) will gain a native Arrow mode that circumvents the current Thrift serde protocol.
I have begun working to integrate this code into the Arrow code tree in https://github.com/wesm/arrow/tree/hs2client-fork. Since this is Cloudera Apache 2.0 code I don't think we need necessarily to go through the energy of doing a software grant or IP clearance so long as we attribute in LICENSE.txt


[WIP] Add installStrategy for addon and upgrade to v1beta1

Signed-off-by: Jian Qiu<EMAIL_ADDRESS>

/lgtm

/hold
I want to rethink a bit upgrade path


[JSDoc] Add missing documentation for adaptive-expressions/builtinFuntions files 1/10

Addresses # 2602
Description
This PR adds the missing documentation in adaptive-expressions/builtinFunctions/ files.
Some documentation is ported and adapted from botbuilder-dotnet
Specific Changes

Added JSDoc comments in all public methods.
Added @private attribute to private methods.
Fixes linter spacing warnings.

Testing
The following image shows the ESLint Report with no JSDoc errors.

Promoted


links for fullscreen blockquotes

steven and i talked about adding links to those big sweeping block quotes to relevant stuff. so like the ravenhill quote would also have some link to the about/vision page.

going to wait till we've got content for this


Implement support for Lua 5.1

@victorhora @zimmerle any movement here?

All good, merged at https://github.com/SpiderLabs/ModSecurity/commit/dee989844914f07e38367fb6b1ead906541c0849. Thanks @p0pr0ck5 :)


strictNullChecks: inconsistent handling of undefined variable declarations

TypeScript Version:  2.0.6
let foo: string = undefined; // Type 'undefined' is not assignable to type 'string'.
let bar: string;  // no error

I would expect an error for bar since 'undefined' is not included in the type.
I understand that this would break a lot of existing code, but isn't that the whole point of --strictNullChecks? Maybe we can have a --strictUndefinedChecks flag?

i guess the 'use before assigned' error covers this.


Cosign Files Not Present in Release

Following the Download & Install from Release artifacts docs: https://github.com/google/yamlfmt/blob/main/README.md#verifying-release-artifacts
There is a step to validate the release artifacts with cosign:
curl -sfLO https://github.com/google/yamlfmt/releases/download/vA.B.C/checksums.txt
curl -sfLO https://github.com/google/yamlfmt/releases/download/vA.B.C/checksums.txt.pem
curl -sfLO https://github.com/google/yamlfmt/releases/download/vA.B.C/checksums.txt.sig

These files are not present in the release. Only the checksums.txt file is present - https://github.com/google/yamlfmt/releases/tag/v0.13.0
It looks like this is the case for other recent releases as well.

Hi @thkawcha,
That cosign support was only added recently and will start working from the next release onward. I added a clarifying note to the README. Thanks for catching that.


Provide framework for component initialization

Includes:

Initializer service for running and re-submitting initialization tasks
until successful
Barrier methods to control access while initializing

Resolves #80

Merged afcecc1d67.


[stable/telegraf] Check for service.enabled

What this PR does / why we need it:
Check for service.enabled
Which issue this PR fixes

fixes #15262

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Title of the PR starts with chart name (e.g. [stable/chart])

Hi @raphaelquati ,
It seems like port of service will depend on the config.inputs values
Instead of adding an option to disable the service, keep it and please configure a similar logic for rabbitmq here:
https://github.com/helm/charts/blob/b91e719e6812a15bec9afcd9f0b6e7554cb564ba/stable/telegraf/templates/service.yaml#L12-L42
?
I think that should work.
Cheers!

I think you will need {{ trimPrefix ":" $value.url | int64 }} instead of {{ trimPrefix ":" $value.service_address | int64 }} like the ones that are there

I think the purpose of the service ports is intended to plugins make Telegraph listen external ports (like https://github.com/influxdata/telegraf/tree/master/plugins/inputs/socket_listener or https://github.com/influxdata/telegraf/tree/master/plugins/inputs/influxdb_listener). In the case of url in rabbitmq example, is the address of RabbitMQ server, and not make Telegraph listen that address.

Thanks for the explanation.

/ok-to-test
/lgtm

Needs a chart version bump according to CI test result @raphaelquati

/lgtm


Add origin tracking to files

All files now have an origin map, with a type string (device, user, job, or unknown) and an id field pointing to the corresponding document. If a file was stored before this change, the type returns unknown.
In addition, routes that return an object, or objects, with a files array, can now take an optional join=origin request parameter. This will add a join-origin key to each object, containing a context map of the various origin documents. Use this to prevent N calls to various other endpoints.
Example:
// on the file object
 "origin": {
      "type": "user",  // can be user | device | job | unknown
      "id"<EMAIL_ADDRESS>    }

Example of a join-origin:
{
    "files": [
        { "origin": { "type":"user", "id":"foo" }}
        // ...
    ],

    "join-origin": {
        "users":    {"foo": obj, "bar": obj },
        "devices":  {"foo": obj},
        "jobs":     {"foo": obj}
    }
}

Along the way, the job generator had to gain additional insight to throw job IDs onto upload URLs, the Placer gained an origin field for placing files, not-Placer uploads had origin hacked into them, and the base request init now knows a bit more about the requester.
Closes #163.

LGTM - I like the enum for origin type.

LGTM - just a couple of comments.

Comments addressed and rebased.
I was going to call it good for merge, but when testing a packfile upload:
  File "webapp2.py", line 570, in dispatch
    return method(*args, **kwargs)
  File "./api/handlers/listhandler.py", line 399, in packfile
    return upload.process_upload(self.request, upload.Strategy.packfile, origin=self.origin)
  File "./api/upload.py", line 80, in process_upload
    placer.check()
  File "./api/placer.py", line 163, in check
    validators.validate_data(self.metadata, 'packfile.json', 'POST')
  File "./api/validators.py", line 29, in validate_data
    validator = from_schema_path(schema_url)
  File "./api/validators.py", line 101, in from_schema_path
    schema, resolver = _resolve_schema(schema_url)
  File "./api/validators.py", line 67, in _resolve_schema
    base_uri, schema_name = re.match('(.*/)(.*)', schema_url).groups()
AttributeError: 'NoneType' object has no attribute 'groups'

Any thoughts on this @rentzso?
I only ran into it after rebasing, so it's a good thing I did :smile:
I'll be out for a decent chunk of today, so if you have ideas feel free to throw them on this branch.
Otherwise we can work on it later.

@kofalt I will check. It is related to the new validation endpoint for serving schemas.

New commit 6b5a42a has fixed the problem as far as I can tell. Merging.


[GEARPUMP-227] remove unneeded licenses

After replacing Jekyll with Mkdocs in https://github.com/apache/incubator-gearpump/pull/88, some licenses are no longer needed

R: @huafengw for review

Current coverage is 72.04% (diff: 100%)

Merging #101 into master will increase coverage by 0.36%

@@             master       #101   diff @@
==========================================
  Files           186        186
  Lines          5985       5985
  Methods        5456       5456
  Messages          0          0
  Branches        529        529
==========================================
+ Hits           4290       4312    +22
+ Misses         1695       1673    -22
  Partials          0          0

Powered by Codecov. Last update fb37ce8...68d83d6

+1


Change leftover references to scanner.cc to scanner.c

This should fix #40, hopefully.

Since I can't seem to request reviews here: @yannham @ErinvanderVeen


[AMBARI-24137] Fixed mpack display on service summary view.

What changes were proposed in this pull request?
The approach to locating the mpack information was wrong. Services must always be differentiated by service group from now on. Looking up information about a service by name alone is no longer a viable approach.
The problem was fixed by providing the mpack information along with the rest of the service information shown on the page, rather than trying to look it up by service name after the fact. I also added the service group name to a tooltip displayed in the left nav menu so that the user can tell which instance of a service they are clicking on.
How was this patch tested?
All unit tests passing.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2762/
Test FAILed.
Test FAILured.


Add canonical tag to posts copied to fast ruby

Hey @etagwerker @lubc
This PR is adding the canonical url only to the posts will copy to Fast Ruby Blog.
Here is an example:

Please, take a look. Thanks.

Hey @etagwerker I just updated the PR. I't ready to review. Thanks.


Add a license to the repository

I recently came across your project qiskit and I noticed that the repository github.com/kosycoder/qiskit currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Thank you for your information.
I added a license file.
If you notice anything else, feel free to contact me.
Thank you.

Thank you!


cannot install

Dalamud does not seem to be able to install the plugin. Only happens with this specific plugin too. Any idea how to fix please?

Not update for dawntrail


Fix JSON data error in Prepare system for update

The JSON data is incorrect invoking prepareForUpdate;
Add Apply update step in Update via REST section.
Added description of tar-ball files built by OpenBMC.

This change is

Thank you for your contribution, but this repository is managed via gerrit at https://gerrit.openbmc-project.xyz .  Your same github credentials will allow you to log into that server.  Can you please push the commit there instead of through a pull-request?


Support directories in the scaffold base

Hi,
Seems like this does not support folders inside of the scaffolding directory.
I get:
EISDIR: illegal operation on a directory, read

Due to https://github.com/chenasraf/simple-scaffold/blob/master/scaffold.ts#L89

Hi, sorry for the LONG delay - can you elaborate what you mean to help me reproduce?


add release instructions to contributing.md

What do you think about making this a separate document? This isn't really for contributors.

PTAL

lgtm


Consider removing requirement for .encrust.json if only running fatten sub-command

The encrust fatten command doesn't reference any of the data in the ~/.encrust.json file. This command is quite useful on it's own. While it's trivial to create an empty json file, not requiring it would make things easier for the end-user.

In case the emoji wasn't clear, I would absolutely accept a PR doing this


Add batching to Graphite inputs

This change integrates the new PointBatcher with the Graphite inputs. Most of the work involves passing batching configuration parameters to the Graphite inputs and having those same inputs use the batcher.
There was also one change made to the batcher itself. Previously client code had to pass in channels for sending and receiving points (and point-batches). Now the batcher creates those same channels itself, and returns them. That can be seen in this commit: https://github.com/influxdb/influxdb/commit/a9ba189a813a3f562d1bd645bd0ba644b8a47c50
All Graphite inputs come up with a batch size of 1 by default. This means the lowest performance, but the maximum safety with regards to data loss.

@corylanou

This looks good.  My only concern is that we don't drain a batch on shutdown, which could mean we lose data that was in flight.  We could always add that later as well if we don't want to support that now.

Thanks @corylanou -- I agree that is a good idea. Let me get on that immediately after merging this, OK?

+1 :ship:


Add CI status for Ubuntu 16.04

Since CI for Ubuntu 16.04 is up and running. I changed the readme to report the status.

LGTM


More a question than an issue.

Apologies if this isn't the right forum for this.
If I do %(my_func.parameters)s % meta it's fine, but the bare %(my_func.parameters) gives
...docrep_init_.py", line 105, in safe_modulo
return s % meta
ValueError: unsupported format character '?' (0xa) at index 139
I'm just not sure what's happening and if there's something I should change in my code.  For example escaping literal % characters.

hey @LyonsDo. thanks for bringing this up!
"%(my_func.parameters) " is invalid python formatting syntax, that's why you are getting this ValueError. Don't know how to fix this within safe_modulo. The only thing you can do is escaping it yourself with "%%(my_func.parameters) ". Or what would you suggest?

That seems to work fine.  Thanks.

Thanks @Chilipp.


Update to 7.10.3

https://mail.haskell.org/pipermail/ghc-devs/2015-December/010736.html

:+1:

Unfortunately I do not have time for this right now, happy to dish out maintainer rights to anyone who can take over. Otherwise, it might be a while until I find the time to dig in to this.

@etrepum, I'll try to do a build this weekend as the whole thing is sorely problematic on el capitan :-)


Fix error when nbdb and sbdb leader are located on different nodes by

adding --no-leader-only flag

/cc @flavio-fernandes

/label acknowledge-critical-fixes-only

/retitle SDN-4189: Fix error when nbdb and sbdb leader are located on different nodes

/tide refresh


Error: Cannot read properties of undefined (reading 'map')

Run<EMAIL_ADDRESS>  with:
    path: UnnamedClockProject-3D_top.png
    GITHUB_TOKEN: ***
    title: PCB
    uploadHost: https://litterbox.catbox.moe/
    annotationTag: [--]
    annotationLevel: notice
Error: Failed to upload images
Error: Cannot read properties of undefined (reading 'map')

It might just be something I've misconfigured...
https://github.com/KenwoodFox/UnnamedClockProject/blob/22053a8d6d00088100b44c9347cd59f84a6e46e9/.github/workflows/hardware_workflow.yml#L53-L69
But even if it is, i don't understand what map its talking about, the error should be more descriptive of what I did wrong? Thanks and lmk!

Hi! thanks for the bug report, can you try using https://litterbox.catbox.moe/resources/internals/api.php on the uploadHost instead (or remove it so it uses the default)?

Hi! thanks for the bug report, can you try using https://litterbox.catbox.moe/resources/internals/api.php on the uploadHost (or remove it so it uses the default)?

Thanks! That.. kinda worked? it didn't error out! but it also didn't comment the image like i was expecting, instead it just sort-of errored out like this:

Let me know what might be helpful to try next!

I'll update the default url #8 , that is weird that the image is not attached, is it a big image? Is there any sort of link?

Hmm, it still seems to have not included an image, it certainly didn't put it in the pull request comment and if i click on the annotation it just looks like this

The URL it gave me is this: https://camo.githubusercontent.com/fac3beb61110da650284910205f309067ef2d8b3933fdf81f725bbd6020f9534/4e6f2072657175657374207479706520676976656e2e
if i go there i see this

The image is pretty small, do you have maybe a test-repo where this is setup and working?

I'll setup a public one, is it possible to have the image

Should be fixed now
https://github.com/edunad/actions-image/actions/runs/6233546335/job/16918995083
Can you try v2.0.3

That looks like it did it, thanks so much!


DOC: confusing note about numpy.asarray not copying the coordinate values

The docs at https://shapely.readthedocs.io/en/latest/manual.html#numpy-and-python-arrays about the interaction with numpy say:

The numpy.asarray() function does not copy coordinate values – at the price of slower Numpy access to the coordinates of Shapely objects.

I find this a bit confusing or misleading.
First, it is still copying the original coordinate values  (it's not a view on the original GEOS coordinates), although it is true it does one copy less (it keeps the ctypes array as "base" of the numpy array, instead of taking a copy of the input, sincenumpy.array() has a copy=True default)
Second, I also don't really understand "at the price of slower Numpy access". What exactly becomes slower?

Ah, that sentence is ancient. I must have had misconceptions about asarray being a view.

Thanks!
I also updated the section on the array interface in the RFC: https://github.com/shapely/shapely-rfc/blob/df79554e19aa7182796eaa3fed22e0e7084607ef/rfc/0001-shapely-2.0-roadmap.md#the-array-interface-and-ctypes-attribute