release: 0.9.0-alpha.4

Prepare 0.9.0-alpha.4 release PRs.
Test integration on many examples before merging and pushing images:

[x] Kubernetes (reana-dev run-example -c DEMO -b kubernetes --submit-only)

Test HTCondor compute backends again:

[ ] HTCondor (reana-dev run-example -c DEMO -b htcondorcern --submit-only)

Test three launch examples:

[x] /launch

Shared components:

[x] pytest-reana (release: 0.9.0a4)
[x] reana-db (release: 0.9.0a5)
[x] reana-commons (release: 0.9.0a6)

Cluster components:

[x] reana-job-controller  @audrium https://github.com/reanahub/reana-job-controller/pull/362
[x] reana-message-broker (release: 0.9.0a1)
[x] reana-server (release: 0.9.0a4) https://github.com/reanahub/reana-server/pull/481
[x] reana-ui (release: 0.9.0-alpha.4) https://github.com/reanahub/reana-ui/pull/258
[x] reana-workflow-controller @audrium https://github.com/reanahub/reana-workflow-controller/pull/444
[x] reana-workflow-engine-cwl (release: 0.9.0a3)
[x] reana-workflow-engine-serial (release: 0.9.0a3)
[x] reana-workflow-engine-snakemake (release: 0.9.0a3)
[x] reana-workflow-engine-yadage (release: 0.9.0a3)

Client components:

[x] reana-client (release: 0.9.0a4)

Helm chart:

[x] reana https://github.com/reanahub/reana/pull/631
[x] test researcher-oriented helm installation method

Closing, everything was merged and released


write a docstring for this code.

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
how to choose language when i use write a docstring for this code.
Solution
No response

i want use chinese to write docsting.

@linnnff There are a couple of ways to do this:
If you always want the model to respond in Chinese, you can set "systemMessage": "Always respond in Chinese" in config.json.
If you want to customize only the docstring prompt, you can change it like this:
"experimental": {
    "contextMenuPrompts": {
      "docstring": "Write a docstring for this code in Chinese. Do not change anything about the code itself."
    }
  },

in config.json


Add Bunnymark and Sublime syntax

New entries:

Bunnymark,
Sublime syntax

Sections:

Added the Metadevelopment tools section with tools for writing assemblers and emulators (as opposed to writing Uxn code).

Dropped the descriptions and project names in the Uxntal language support section.

You're amazing Karol, y'know


Enum Field Issue

Hello
It doesn't recognise enum fields!
Unknown database type enum requested, Doctrine\DBAL\Platforms\MySQL57Platform may not support it

I believe it's a problem with Doctrine's DBAL library which this package depends on. An easy work-around is to add the following code to your AppServiceProvider's register method:
if ($this->app->environment() == 'local') {
    DB::getDoctrineSchemaManager()
        ->getDatabasePlatform()
        ->registerDoctrineTypeMapping('enum', 'string');
}


Proposal: Adding Debouncing Effect to SignUp page

Is your feature request related to a problem? Please describe.
When users type quickly on the registration form, input validations or API calls may trigger too frequently, leading to excessive server requests or validations. This behavior can create unnecessary load, slow down the page, and negatively impact the user experience, especially if multiple validations are happening simultaneously as the user fills in fields.
Describe the solution you'd like
Implement a debouncing function to limit the frequency of validation or API calls on the registration form. For example, when a user types in the username field, debouncing can ensure that validation only occurs after the user pauses typing, reducing the number of requests sent. The debouncing can be set to trigger after a certain delay (e.g., 300ms), improving the form's responsiveness and performance.
Describe alternatives you've considered
An alternative could be to use a throttling mechanism instead of debouncing. However, throttling controls the number of times a function is executed over time, which might not be as effective here, as we want to wait until the user pauses typing before validating. Debouncing better suits this use case for performance and user experience.
Additional context
This feature could be implemented on various input fields, such as username or email, where real-time validation is critical but should be controlled. Debouncing will prevent the system from overwhelming the server with validation calls and help users complete the registration smoothly. This update will also improve the form’s performance and responsiveness.

So are you hoping to get assigned to this issue, or you just raised it as an issue? thank you.

Hey @amin-xiv
Assign it to me, I want to work on this issue.

@AbhijitMotekar99 Hi, I am not a maintainer lol, I was just asking if you are just reporting it for someone to fix it or you would like to work on it, because I also wanted to contribute, but since you want no problem.

No, I want to work on this

@AbhijitMotekar99 go ahead, good luck with it!


[Needs investigation] Social media icon links not working

Social media icon links not working
The social media icons are not clickable i.e. They don't open the links that are configured.
This was supposed to be fixed by #433 but that doesn't seem to be the case.

@paramsiddharth , you are talking about the links in the footer? I ran the main branch with our most recent changes and they all appear to be working (except that one is a Sogorea Te' link that I forgot to change back).
https://user-images.githubusercontent.com/54147446/211158351-0d47d930-8f12-4157-8969-570a3c437019.mp4
Let me know if this is not the right section to investigate.

@paramsiddharth , you are talking about the links in the footer? I ran the main branch with our most recent changes and they all appear to be working (except that one is a Sogorea Te' link that I forgot to change back).
2023-01-07.09-27-16.mp4
Let me know if this is not the right section to investigate.

@DietBepis1 Strange... I'm confident I checked out the branch, ran the application, and tried clicking the icons and they didn't work. Let me check once more.

@paramsiddharth, at the risk of stating something obvious, did you pull the changes into main? 😜

@paramsiddharth, at the risk of stating something obvious, did you pull the changes into main? 😜

@DietBepis1 Noooo... I'm sure I didn't. I checked out your PR branch using GitHub CLI, which is what I always do when testing PRs on GitHub.

Hi guys, it works here too.

Closing as mistaken.


Opening Second Ace editor restricts height and width

Hi There,
I am using jarvus-ace with ExtJS6, while it works good when only a single/first instance is created. But any subsequent instance has weird CSS style rule imposed. Here is snippet of Styles being applied
element.style {
    margin-top: 0px;
    width: 59795px;
    height: 54px;
    margin-left: -52762px;
}

on element with class .ace_content and applied dynamically. This only happens when second/subsequent instance is created.
Here is Ace config being used
Ext.define('Testt.AceConfiguration', {
  override: 'Jarvus.ace.util.AbstractConfiguration',
  config: {
    options: {
      theme: 'ace/theme/monokai',
      mode: 'ace/mode/javascript',
      fontSize: 18
    }
  }
});

Here is how it is being used
   {
      xtype: 'jarvus-ace-editor',
      reference: 'my_editor',
      flex: 1,
      minHeight: 100,
      labelAlign: 'top',
      fieldLabel: 'Body',
      allowBlank: false,
      subscribe: {
        'Editor': [
            'blur',
            'change'
        ],
        'EditorSession': [
            'change'
        ],
        'Document': [
            'change'
        ]
      }
    }

And there is no JS. error seen in console

Is this project maintained actively?

@bagwanpankaj yep, but it may be a bit before I can dig into this particular issue. Pull requests welcome!

Hi There,
I am also facing same issue

@bagwanpankaj  @idealogyin I just picked up the project using this again, and am not seeing the described issue in my own use. Anyone have a patch or a reproducible test case?


Windows Auto dark/light theme

Description of the new feature / enhancement
There should be a feature where windows can automatically switch themes between dark and light depending on time of day or dusk / dawn cycle. While this sort of program does exist I think it will be a very good addition to PowerToys.
Scenario when this would be used?
This is supported natively in many other operating systems and would be a great improvement to windows.
Supporting information
No response

/dup #1331


Fixed issue 1520 - the collect_metrics options for the app is now taken into account for all actions

Fix Issue #1520 - The collect_metrics option for the app is not taken into account in all actions.
Describe the bug
According to the docs, collect_metrics app option allows to turn off sending telemetry data.
However looking at file: (https://github.com/mem0ai/mem0/blob/main/embedchain/embedchain/embedchain.py) this is correct only for add and delete methods.
Description
For each action/method, checked if the collect_metrics was checked before collecting metrics, where it was not the checked it has now been added.
Fixes # 1520
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please advise how this should be tested.
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] New and existing unit tests pass locally with my changes
[X] I have checked my code and corrected any misspellings

Maintainer Checklist

[X] closes #1520
[X] Made sure Checks passed

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

Hi There appears to be a breaking change in NLTK:  https://github.com/nltk/nltk/issues/3293
I don't believe there was any error in my fix to 1520, but the build of embedchain seems to have been impacted by the change in NLTK.
There is a suggested fix in the response to the NLTK issues discussion:
"so one should replace nltk.download('punkt') with nltk.download('punkt_tab')?"
But I don't know where this is being called in the build_embedchain build process?
Happy to help if someone can point me in the right direction or has this already been spotted and fixed?

Hey @ArthurHoward1 Can you please check the failing tests?

Hi @Dev-Khant,
Sorry I am new to this but keen to help, a little guidance would be appreciated, if possible.
I see the same error on the multiple builds against differing version of Python, I don’t see how it is anything to do with my code changes, the build is failing this test:
tests/loaders/test_xml.py:53:
It seems to be failing on some dependency deep in:
•	LangChain / document_loader
•	NLTK
related to the tokeniser - 'tokenizers/punkt_tab/english/', I don’t believe my code changes would have tiggered the need to use this.
I believe changes in these dependent packages have cause the failure not my code changes.
This NLTK issue seems to describe the exact error seen: https://github.com/nltk/nltk/issues/3293
Can you trigger a dependency refresh and rebuild/test my pull request/changes again?
Thanks, Arthur

Hey @ArthurHoward1 You are right, tests are failing because of another issue and not related to your changes. I taken note of it and will fix the tests.
Your changes looks good to me, it can be merged. Thanks for the contribution!


Tokenize three periods as ellipsis (not just unicode character …)

Thank you.
Note that you can also do \dots<tab> in the CLI version.


remove .sh suffix from manager script name

as discussed here
https://github.com/ChristopherHX/github-actions-act-runner/pull/21#discussion_r671754664
it is desirable to remove the .sh sufix from the github-act-runner.sh script.
The problem here is that there is a github-act-runner directory along with that in the /usr/bin directory. So, there can be no file with the same name in the same dir.
So, then we also need to rename the /usr/bin/github-act-runner/ directory to something else. What would be the good name for it? Maybe /usr/bin/github-act-runner-bin/?

I would move /usr/bin/github-act-runner/ to /usr/share/github-act-runner/bin.
Then move github-act-runner.sh also  to /usr/share/github-act-runner/bin/.
Only add a symlink from /usr/bin/github-act-runner to ../share/github-act-runner/bin/github-act-runner.sh.

Well, in my understanding it violates FHS: https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s11.html
/usr/share is for architecture independent data, while the runner binary is not a data and is not architecture independent.
I thought about moving everything to /usr/lib/github-act-runner/ but then it also is not clean, since /usr/lib is intended for re-usable stuff by other apps.
Maybe /opt/github-act-runner/?

Or, actually /usr/lib might be ok.
According to https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s06.html:

On some systems, it may also include internal binaries that are not intended to be executed directly by users or shell scripts.

I forget llvm uses lib not share on ubuntu 20.04
lrwxrwxrwx 1 root root 24 Mar 20  2020 /usr/bin/clang -> ../lib/llvm-10/bin/clang

Then /usr/lib/github-act-runner/ is the solution? I can make the change.

Yes for me this is a good solution.

I'll make a PR then


Fix typo in init.py and add version information

The package currently has a typo in webdataset/__init__.py. Also, it is not possible to query the package version using webdataset.__version__.
The PR adds a proposal to retrieve the version when loading the package (via pkg_resources). An alternative proposal would be to specify the version in a webdataset.version module, and import this in setup.py. This would slightly change the workflow for package maintainers when bumping versions, hence I opted for the way via pkg_resources (but happy to adapt to your preferences).
To test:
$ pip install .
$ python -c 'import webdataset; print(webdataset.__version__)'
0.2.12

Small update: In case anything goes wrong (I do not know if pkg_resources will work across all supported python versions), add exception handling and throw a warning in case package information cannot be inferred at runtime.

version information is updated by "invoke newversion". That was broken, but I have fixed it now.


Add note about which service to request when running locally with Docker & Webpack or Gulp

Description

Add a section about using Webpack or Gulp with Docker locally in order to make it clear which service should be addressed in order to see proper styles.
Checklist:

[x] I've updated the documentation or confirm that my change doesn't require any updates

Rationale

It was unclear to me (per https://github.com/cookiecutter/cookiecutter-django/issues/4129) that I needed to request the node service and not the django service when using the Webpack frontend_pipeline option. One should use the django service when using any other option, but using the node service when using the Webpack or Gulp option.

Hi guys, this happened to me and because I am not using docker but I am using Webpack locally, I suggest you also add this text to the following https://cookiecutter-django.readthedocs.io/en/latest/developing-locally.html#setting-up-development-environment

Just sent one: https://github.com/cookiecutter/cookiecutter-django/pull/4413


[backport stable-1.11] shimv2: Use BUILDTAGS when building shimv2

If not used, SELinux support will never be enabled.
It's important to mention that this issue has been already been fixed on
kata-containers/kata-containers, unintentionally, as part of
https://github.com/kata-containers/kata-containers/commit/e90c5d45b3ceef
Fixes: #2776
Signed-off-by: Fabiano Fidêncio<EMAIL_ADDRESS>(cherry picked from commit c51baf8df595bb19b84366860d07bd89841320c5)

/test-ubuntu

/test-clh


Community Report for longguikeji/arkid

Signed-off-by: Vinod K. Ahuja<EMAIL_ADDRESS>

@vinodkahuja your DCO failed.


fix(docs): Map reference error with new API key

Currently react-google-maps documentation is broken due APImapProjectError. It was due to api key expiration/ qouta limitatons. I replaced previous API key with new developer key with all required API restrictions. Documentation is working now. Here are added API restrictions:
Places API |  Geolocation API |  Directions API |  Maps JavaScript API |  Distance Matrix API |  Geocoding API

I would consider taking this opportunity to refactor the api key into a constant

@sakhisheikh Are you volunteering to pay for the entire 🌐 world's access to @tomchentw's documentation? 😁
I don't think this PR will be accepted just because doing so would mean you'll end up footing $$$$.$$ in the next 30 days 😸 💸 💸 💸 💸💸 💸
What would be a better fix would be to create an input that people could paste their API keys into, and view this documentation at their leisure and on their own 💵.

@milieu First of all this is developer key with limited daily quota and we're using API key at client side. For docs level, I think this is sufficient to use non-billing API key as everyone has it's own quota. If it's quota gets exhausted on excessive usage  for one person, It won't effect other users.
I really appreciate your idea to make an input field in docs for every user but most of the new users don't have knowledge regarding API access. Maybe this would turn down many new users for the sake of creating new API key. Maintainers should seriously take this issue in consideration as it is a huge blockage for users to leverage docs.

An API key is required to use any Google Map API now since their policy change, so new users would have to figure how to get a key anyway.
This change won't work once your $300 freebie is up

I think the solution should be removing the key parameter completely. That way the example will work, and the only difference is that they will have a "For development purposes only" watermark


[CI] Cloud build fails with "repository 'tmp' does not exist" on copy_git_directory

Already have image (with digest): gcr.io/cloud-builders/git
fatal: repository 'tmp' does not exist

Can't reproduce the bug

I'll close for now, will reopen if it recurs.


[fix][client] Release payload when contents are corrupted

Motivation
Release payload when contents are corrupted
Verifying this change
It seems hard to write unit tests,  anyone have an idea? :)
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/Shoothzj/pulsar/pull/1

/pulsarbot run-failure-checks

Codecov Report

Merging #18650 (95471d3) into master (cd85a67) will decrease coverage by 10.42%.
The diff coverage is 11.11%.

@@              Coverage Diff              @@
##             master   #18650       +/-   ##
=============================================
- Coverage     47.39%   36.97%   -10.43%
+ Complexity    10479     1961     -8518
=============================================
  Files           698      209      -489
  Lines         68070    14422    -53648
  Branches       7279     1573     -5706
=============================================
- Hits          32264     5333    -26931
+ Misses        32228     8505    -23723
+ Partials       3578      584     -2994

Flag
Coverage Δ

unittests
36.97% <11.11%> (-10.43%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...va/org/apache/pulsar/client/impl/ConsumerImpl.java
15.10% <0.00%> (-0.03%)
:arrow_down:

...pache/pulsar/client/impl/TableViewBuilderImpl.java
0.00% <0.00%> (ø)

...a/org/apache/pulsar/client/impl/TableViewImpl.java
0.00% <0.00%> (ø)

...pulsar/client/impl/TableViewConfigurationData.java
33.33% <50.00%> (+33.33%)
:arrow_up:

...apache/pulsar/client/impl/AutoClusterFailover.java
70.00% <0.00%> (-6.12%)
:arrow_down:

...he/pulsar/client/impl/PartitionedProducerImpl.java
30.34% <0.00%> (-5.13%)
:arrow_down:

.../pulsar/client/impl/ControlledClusterFailover.java
58.87% <0.00%> (-3.74%)
:arrow_down:

.../pulsar/client/impl/ProducerStatsRecorderImpl.java
84.04% <0.00%> (-0.62%)
:arrow_down:

...va/org/apache/pulsar/client/impl/ProducerImpl.java
16.43% <0.00%> (-0.59%)
:arrow_down:

...pache/pulsar/broker/admin/v2/PersistentTopics.java

... and 490 more


Windows 10 defender argue cppan.exe is a threat ?!

Hi,
I download latest version (https://cppan.org/client/cppan-master-Windows-client.zip) of client in Windows, but defender shows this:
....
Alert level:Severe
Category:Trojan
Details:This program is dangerous and executes commands from an attacker.
....
I double checked the md5 hash is the same as official provided.
What's happen, is this right or wrong?

Yes, I also saw this. VirusTotal show zero treats.
I'm not sure how to fix this at the moment. Maybe contact MS.
Could you check file properties (detail tab), please?
Must be something likie this

See https://www.virustotal.com/#/file-analysis/NWUyZTlkN2QzNmQ3YjI3YWIxZDJjZWU3NTA0NGU3ZTA6MTUyMDg1NTA4Nw==

Can you try to update defender database?
On the latest updates nothing is detected.

Yes, after defender updated, detects no threads.
Thanks !!


fix: running same task twice

While I was figuring out #21, I ended up adding in logic to prevent circularity. However, this also prevents the same task from running twice. But, ironically, having #21 done meant that it is easy to define a task with an argument and then have other tasks call it with different arguments.
[scripts]
build = "touch build/$1"
clean = "rm -rf build"
all = ["clean", "-mkdir build", "build foo", "build bar", "echo 'Done'"]

Currently: runs clean, mkdir build, build foo, echo 'Done' (skips build bar)
Expected: all of the steps run

Here's my analysis:
build => |touch
clean => |rm
all => clean => |mkdir => build => build => |echo

So is the problem when a composite task is called multiple times?

Only composite tasks can call other tasks. So the concern is that in the call chain you somehow end up calling yourself again.
Here's the simplest loop:
loop = ["loop"]

If a composite task cannot be called twice, we prevent this loop.

Here's a more complicated loop:
[scripts]
a = ["b"]
b = ["a"]
# a=> b => a => b...

Ok, so far so good.

What about a more real-world use case:
[scripts]
clean = ["rm -rf build", "rm build-artifacts"]
build = "touch build/$1"
run = ["clean", "build foo", "mv foo new-location", "clean"]

In this case, clean is a composite task, but we're calling it twice at the same level-- it's not cycling back to run.


Add --all or --export-all option to CLI export

To get all version of a photo, you currently need to use --export-edit, --export-bursts, --export-live.  How about a --export-all to get all versions of a photo.  Thanks @jystervinou for suggesting this.

I've changed the default behavior as of 0.28.2 to export all photos and added --skip-edit, --skip-bursts etc to negate this if desired.


Artifact filename change in Release 1.28.0 doesn't match install documentation

Not sure if its intentional, but the generated php binary/package has changed to be just phpbrew dropping the .phar from the file name, the install instructions still use the url  with the .phar included.

The new released version is going back to the .phar extension. Thanks for your reporting :).


Added ability to save plots to a different directory

No longer strips directories from the savefile name. Downside is that a savefile name which is a path will now cause a crash if the specified directory does not exist.

Thanks! I'll add the warning when pointing to a non-existent folder.


EP16-59_Wallboard_Scaffolding_Work

Added following pages:

Status
Roadmap
FAQ
Change Log
Getting Started

There are some duplicates I noticed...so we need to add this to our discussion later in the week.
[kp]

I learned we can create a sub-menu for our Components by using the following markup at the very top of the files that go within that section:
---
name: Alerts
menu: Components
---

Additionally, we can sort using the menu command in the doczrc.js file according to the documentation here: https://github.com/pedronauck/docz/releases/tag/v0.12.4

[ ] Move each UI component into a new directory /Components
[ ] Add menu identifiers into each file within components to break it out into a submenu
[ ] Update doczrc.js to set order

Should we merge the Introduction and Getting Started sections?


Standardize on SIMDEM_LOCATION

@rgardler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Capella iQ: enable SSO for sign-in to Capella

Is your feature request related to a problem? Please describe.
Capella users with SSO enabled/enforced for their org cannot sign-in to Capella to use iQ
Describe the solution you'd like
Provide the option to sign-in with SSO, prompting the user for the realm and integrating with the backend SSO provider.

We are waiting for Capella to update their auth mechanism.

Any update @deniswsrosa ?


[tcgc] add void and never handling for parameter and return types

resolve: https://github.com/Azure/typespec-azure/issues/795

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-client-generator-core

Show changes
@azure-tools/typespec-client-generator-core - fix ✏

add void and never handling for parameter and return types

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/797/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/797/


[Bug] API normalized_mutual_info_score may violate Symmetry.

Description

The output of  api normalized_mutual_info_score may violate Symmetry, although the official documentation claims that
"This metric is furthermore symmetric: switching label_true with label_pred will return the same score value. "https://scikit-learn.org/stable/modules/generated/sklearn.metrics.normalized_mutual_info_score.html
Reproduce

Run the code as follows:

from sklearn.preprocessing import LabelEncoder
from sklearn.metrics.cluster import normalized_mutual_info_score
a = "guest"
b = "user"
c = "admin"
src = [a,c,b,b,c,a,c,a,c,c]
tar = [a,b,a,a,b,c,b,b,c,a]
test_nums = 100
for i in range(test_nums):
le = LabelEncoder()
src_list = list(set(src))
tar_list = list(set(tar))
fit_list = tar_list + src_list
le.fit(fit_list)
src_col = le.transform(src)
tar_col = le.transform(tar)
test1 = normalized_mutual_info_score(src_col, tar_col,average_method='geometric')
test2 = normalized_mutual_info_score(tar_col, src_col,average_method='geometric')
print(f"iter:{i}:  test1:{test1}  test2:{test2}")
print(src_col,tar_col)
print(tar_col,src_col)
assert test2==test1

I tried different choices for average_method, but the error was still there.

Expected behavior
Keep the Symmetry property of normalized mutual information. We may need to rewrite code by ourselves.
Context

Operating System and version: Mac OS 12.5.1
sdgx==0.1.4.dev0
scikit-learn==1.3.2

I debuged this problem and found that the difference is the out of sum at https://github.com/scikit-learn/scikit-learn/blob/3f89022fa04d293152f1d32fbc2a5bdaaf2df364/sklearn/metrics/cluster/_supervised.py#L900
I think the reason is the float design of Python, so the output of normalized_mutual_info_score can be accurate within a certain precision.


LOG debug mode. See debug log errors without the visual debug

Something I've been using myself to see debug logs in console without the full visual debug (like showing all hidden elements). Though I'd open a PR so if you think it's useful you can merge it.
debug=log in url to use.

This is clever :)


@Required() does not validate @BodyParams

Informations

Type
Version

Bug
2.3.8

Description
When @Required() is added before @BodyParam(), it is expected that an exception would be thrown if no body params were posted to the endpoint or if the body params did not match the expected type.
E.g.
@Post('/')
async broadcast(@Required() @BodyParams() webhook: Webhook): Promise<any> {
    ,,,
}

and the models is
export class WebHook {
url: string;
name: string;
}

if I posted up {}, I'd expected the Required decorator to throw an exception.
if i posted up { a: '', b: ''} I'd expect the Required decorator to throw an exception because the properties do not match what is on the WebHook class.
if i posted up no body, I'd expect the Required decorator to throw an exception
if i posted up { url: 'http://google.co.uk', name: 'test'}, I'd expect the Required decorator to not throw an exception
if the body params has nested objects these should also be checked that they have been set.

At the moment Required does not validate the BodyParams at all.
Todos

[ ] Update Required decorator to validate the BodyParams

Hi @Ahagan,
It's a part of these stories #85 and #35 .
Actually Required work fine only with simple data like this:
@Required @BodyParam('name') name: string;

With a model, we need to implement a model validators. It's planed :)
See you

Ok, thanks for letting me know. Do you know when the work will be started on this?

October, I hope ^^.

Hey @Romakita,
Is it possible to add the class validation library now for existing apps? Then once it’s implemented then just move back?
I would love to help out on this but just don’t seem to have time. :-(
Just a thought!

@Romalita thanks, and keep up all the great work! 😊

@Ahagan the story for the validation feature is ready. You can follow this issue #85


Rapidly creating new toasts causes strange rendering behavior

Reproduction: https://codesandbox.io/s/wonderful-brook-9wrw4c?file=/src/App.tsx
Loom of what I see: https://www.loom.com/share/45ce40b6c6294585b503ef1d00725bca
The repro is obviously a worst-case, but we observe the behavior in file-uploading code that concurrently uploads 5 files at a time from a batch uploader and streams back processing results.  It creates toasts far less rapidly, but still suffers from this issue.
Otherwise, cool library!

This has been fixed in v0.2.0, https://codesandbox.io/s/epic-goldwasser-z7qujx?file=/src/index.tsx.


Autosave/auto add to session

It could be really cool if simply browsing could be saved for exemple i have a group "shopping" and open a new tab with middle-click the suddenly my boss come.
Then i change session to "work" session, fake working a bit and juste come back to "shopping" later where i can find the tab i opened before
I don't know if it's already possible, i just took the extension for my first time and i already love it ;)

In fact, we really support that
try this feature, before that, you can save open tabs from left nav bar

Sorry if i didn't explained well my suggestion, what i mean is, when i open new tab while working it doesn't autosave in my session like this :

Here we can see to the left i have PHP documentation open, if i swtch between two session "pause" then "dev" back, it will not restore my php documentation

This is one of the features I'm looking for in a tab management extension and would be a game-changer.
Prior to tab groups, window naming and also prior to tab management extensions, my workflow consisted of having a 4-digit amount of tabs open. Tab grouping was huge. But being able to have sessions automatically sync so that I never have to rely on having them open or going through the hassle of editing existing sessions and cross-checking the changes would be one additional major step towards an efficient and organised poweruser workflow.
When I research something I easily open and change 200 tabs in a short amount of time. It's not madness in my opinion because adding and managing bookmarks in the browser with the native manager is so much more time-consuming and impossible to organise. Especially when the topics are and categories can amount to a 3-digit number.
Moving the active session out of the browser helps a lot, because admittedly, I would strongly prefer not having a massive amount of tabs in a large number of windows open at once.
If sessions aren't automatically synced and tracked you end up in situations where you forget to manually save, lose track of tabs and groups as you move between groups, windows and close or open new tabs. Sessions how I use them consist of multiple windows as well, so this is a bit of a challenge. I also think sessions should name the window to the corresponding name (and hopefully in the future support multiple windows with a session).
To put into context how this would be useful, my workflow would (with the addition of this feature) look like this:

Projects, tasks, research sessions etc. are managed by LeanTime or Nextcloud project planners.

When working on a particular project I open a given project on LeanTime/Nextcloud along with the corresponding tab session for that project in OneTab.Group. Closing the session would save the progress so that it can easily be continued later.

Example of projects or sessions (having all of this synced helps a lot):

Something as simple as uying a new computer peripheral
Or something broad like a multi-step automotive repair project with multiple layers of research, buying, planning, referencing etc.
System configuration research (apps, system configuration, post-install script etc.)
Electronics projects
Or just a session of YouTube videos to watch or articles to read. I keep a long list of videos in groups by categories and I go through them when I need/want something to watch.


Cookbook should probably be CC0 licensed

Since the content of the cookbook is intended to be copied freely, it would be better licensed CC0, with basically no restrictions.
Edit:

We will track here the consent of all contributors (the list will be periodically updated):
Please drop a comment here if you consent or disagree.  Link to CC0 license. Once we gather consent following will be merged #210

[x] @alexcrichton
[x] @AlexEne
[x] @alisha17
[x] @AndyGauge
[x] @badboy
[x] @bjnyfv
[x] @bobbo
[x] @bostelk
[x] @brson
[x] @budziq
[x] @cetra3
[x] @charlesetc
[x] @CJStadler
[x] @crypto-universe
[x] @cschneid
[x] @derekdreery
[x] @dhardy
[x] @dhharris
[x] @FaultyRAM
[x] @glennsl
[x] @gsquire
[x] @hegza
[x] @j-haj
[x] @jaemk
[x] @jbolila
[x] @jeanm
[x] @jmcomets
[x] @jonashagstedt
[x] @Lapz
[x] @lilianmoraru
[x] @lucasem
[x] @ludwigpacifici
[x] @luisbg
[x] @luser
[x] @matklad
[x] @Michael-F-Bryan
[x] @mikeastock
[x] @mykalu
[x] @nocduro
[x] @Ophirr33
[x] @pepyakin
[x] @Phaiax
[x] @Piripant
[x] @portal-chan
[x] @rap2hpoutre
[x] @rofrol
[x] @sb89
[x] @sotocodes
[x] @tyoc213
[x] @V1shvesh
[x] @Wilfred
[x] @yandexx

It is not possible to change existing license without asking all contributors if they agree to that. We can also assume that they didn't care in the first place..

We are planning to ask all contributors (at least above typo fix ones) for their consent. At the moment we are blocked with this mostly due to lack of time.

Link to CC0

Is this the right place to give consent? If so, my consent is given.

I agree

I agree.

I agree!

I consent

darn api pagination I'll add the other half of contributors later today 😞

Fine with me!

Sounds great to me 😄

I consent

This is great, I am happy to consent!

I give permission for this license change.  Additionally I give unlimited consent to future licensing changes as the maintainers see fit.

I agree

No problem from me!

Fine with me

You have my consent.

I agree.

No problem on my side!

Fine with me.

I agree 👍

Sounds great!

I agree.

I got no problems!

sounds good to me

Sounds good 👍

👍

Sure, no problem :)

Fine by me.

I agree.
👍

Sure

👍

👍

I agree.

I consent

Sure

I consent. (I love CC0!)

👍 I agree

I agree!

I agree

I agree

I consent

I agree.

I consent.

I consent.

I consent!

Oh my.... I normally dont read some mails I just was curiost, didnt remember.
I consent!

I consent.

I give my consent.

👍
I consent

My change #353, and any other changes I make can be (re)licensed as CC0

I agree :smile:
(On vacations, sorry for the delay)

I consent

I consent

I consent 🎆

I consent

I consent

I consent

I consent

I consent

I agree

So, it is only @trentspi and @Yamakaky left in 2018 to agree on CC0 licensing for the cookbook.
What about new users from Jul 1, 2017?

only @trentspi and @Yamakaky left

Yep, sadly I was unable to reach them sofar.

What about new users from Jul 1, 2017?

@techtonik this list is continuously updated with new contributors.

I agree. Sorry for late response

I consent.

Thanks all! Ping @Yamakaky


Update README.md

Changed discrepancy where map2 had b set to 20 but a comment that b was equal to 50.

Thanks. I must have has a dyslexic moment.

If you entered the numbers on a numeric keypad the 5 and 2 are next to each
other...
On Fri, Feb 13, 2015 at 11:29 PM, Lee Byron<EMAIL_ADDRESS>wrote:

Thanks. I must have has a dyslexic moment.
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/immutable-js/pull/328#issuecomment-74362741.

--
[image: Christos Jonathan Seth Hayward] http://jonathanscorner.com/
Jonathan S. Hayward, a full stack web developer with Python/Django and
AngularJS/jQuery.
Articles on Technology http://jonathanscorner.com/technology/ • Book: Django
JavaScript Integration
http://www.packtpub.com/django-javascript-integration-ajax-and-jquery/book
• Email<EMAIL_ADDRESS>• Github http://github.com/JonathanHayward •
LinkedIn http://www.linkedin.com/in/jonathanhayward • Main Website
http://jonathanscorner.com/ • Professional Site http://jsh.name/
I invite you to visit my professional site! http://jsh.name/


Suggestion: colorize imported names in Visual Studio

I suggest to colorize imported names with a different color than the default, to differentiate them from normal variables.
E.g.
import something from "module";
var z = new something();   // <-- "something" should be colorized

:+1:

see #2890


Fix boolean config values if read from ini file

Use the pyramid asbool and aslistfunctions when reading config values. I haven't done any testing on this, so hopefully @deubs can chime in and confirm that issue is fixed.
Fixes #182

Coverage increased (+0.001%) to 99.578% when pulling 37432fac63ba2ed4d8c075b017d6166539f43f01 on sjaensch:fix-boolean-values-ini-config into 42bd17d72233681bd910c7dde904a1f5469ea03c on striglia:master.

Perhaps a touch of testing via http://stackoverflow.com/questions/32518604/gettings-settings-and-config-from-ini-file-for-pyramid-functional-testing and also manually setting the config values in a Settings object?
I'm a little paranoid of altering this before we have any test coverage. I suppose we'd already catch some of the Settings usages of these parameters because we use that in our current tests...

Yeah part of it is covered by our existing tests (including acceptance tests), but I'll work on creating a test that uses an INI file.

Coverage increased (+0.004%) to 99.58% when pulling 57f46b8fa5da03e68768466bf2f8a528c7012814 on sjaensch:fix-boolean-values-ini-config into 42bd17d72233681bd910c7dde904a1f5469ea03c on striglia:master.

Coverage increased (+0.004%) to 99.58% when pulling d305f5f4cb2d7431021959b6c57e8b6a4e2834fa on sjaensch:fix-boolean-values-ini-config into 42bd17d72233681bd910c7dde904a1f5469ea03c on striglia:master.

@striglia do you think this branch is good to go?

Coverage increased (+0.001%) to 99.583% when pulling 187d97f7ed5f1e4cd034b13fda09b686ee1f564e on sjaensch:fix-boolean-values-ini-config into 4de2a3a9fc6e16d279e67f902ff9b18a492d13b6 on striglia:master.


make enabled collectors configurable (based on code from node_exporter)

Ping @carlpett

This also includes other minor changes to make the code better match the node_exporter command

Except the small nitpick, looks really good!

Thanks, I constified the "os" string. Landing it after build passed


Shutdown due to idle timeout

Using emqx version 3.0 with the MQTT-SN plugin i get the following error messages:
2019-04-24 07:47:21.452 [error] MQTT-SN(<IP_ADDRESS>:40289): shutdown due to idle_timeout
2019-04-24 07:47:21.452 [error] MQTT-SN(<IP_ADDRESS>:40289): shutdown due to idle_timeout
2019-04-24 07:47:21.453 [error] MQTT-SN(<IP_ADDRESS>:38892): shutdown due to idle_timeout
2019-04-24 07:47:36.453 [error] MQTT-SN(<IP_ADDRESS>:40289): shutdown due to idle_timeout
2019-04-24 07:47:36.454 [error] MQTT-SN(<IP_ADDRESS>:40289): shutdown due to idle_timeout
2019-04-24 07:47:36.454 [error] MQTT-SN(<IP_ADDRESS>:38892): shutdown due to idle_timeout

Any idea what might be the problem?

Hi,
I have this issue too. I was attempted a rather large series of messages sent and received.
At first, I set the keepalive timeout in the CONNECT packet to 120 seconds. After these 120 seconds have passed, the server disconnects my device automatically, which is not expected as I've been communicating with the server every few seconds. I attempted to also send PINREQs every 10 seconds (but I am not responding to PINGREQ sent by the server), this did not fix the issue either.
However, increasing the timeout in the CONNECT packet to 10 minutes works correctly.

@pritstift  Sorry for late response, when the mqtt-sn gateway is in idle state which means that mqtt-sn gateway has not received connect packet of mqtt protocol. The default idle timeout value of this state has been hard coded as 10s. So if you do nothing for a long time which longer than 10s in idle state of emqx-sn gateway. It will log shutdown due to idle_timeout error.

Hi, @ServerTech , is that the device you refered is mqtt-sn device rather mqtt device?

@Gilbert-Wong Thanks for the info. What is the reason that an idle timeout is required? Logging shutdown due to idle_timeout every 10 secs is not very helpful if no device does connect using mqtt-sn for some time.
It may be useful to either configure a higher timeout or to turn off logging of this error.  What would be the implications of a higher timeout?

@Gilbert-Wong answering for @ServerTech . Yes, It is an MQTT-SN device.

I was presuming that talking to the gateway in any shape or form resets the keepalive timer. Originally set to 60 seconds in the CONNECT packet, I publish and receive subscription messages every 10-15 seconds or so, and the gateway disconnects the device (and does not respond to subsequent PUBLISH messages) after the 60 seconds have passed.
As mentioned in the previous message, I also attempted to send frequent PINGREQs (with successful PINGRESPs from the server). The server still disconnects me once the 60 seconds have passed.
However, I was able to fix this by setting the keepalive timeout in the CONNECT packet to an unreasonably high value of 10 minutes (which means that I'll be able to keep the connection active for 10 minutes).
Am I doing something wrong and misunderstanding the protocol, or is the keepalive bugged?

I believe we are speaking of a slightly different issue. Mine is with the gateway disconnecting the device after the timeout expires (however the timer should have been reset). I will open a new issue. Apologies!

Never mind.
On May 7, 2019, 5:08 PM +0800, Shreyas Vinod<EMAIL_ADDRESS>wrote:
I believe we are speaking of a slightly different issue. Mine is with the gateway disconnecting the device after the timeout expires (however the timer should have been reset). I will open a new issue. Apologies!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/emqx/emqx-sn/issues/110#issuecomment-489999672, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE4H5U7ZA64SPFKKCEJ2BZLPUFBIXANCNFSM4HIBINGQ.

@Gilbert-Wong Are there any updates on this yet?

It would be fixed in emqx-3.2-rc.1

What is the timeline on that rc?
Gilbert<EMAIL_ADDRESS>schrieb am Mi. 19. Juni 2019 um 08:04:

It would be fixed in emqx-3.2-rc.1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/emqx/emqx-sn/issues/110?email_source=notifications&email_token=ABBYYMWPMFBUWPT3THPSNILP3HD7JA5CNFSM4HIBING2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYAZFNQ#issuecomment-503419574,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBYYMSZFMNHN4A6M6HVXMTP3HD7JANCNFSM4HIBINGQ
.

--
Embever GmbH, Carl-Miller-Straße 6, 39112 Magdeburg
<EMAIL_ADDRESS><EMAIL_ADDRESS>embever.com http://embever.com
Registergericht: Amtsgericht Stendal, HRB: 25007,
Ust.-ID: DE315088960,
Geschäftsführer: Sascha Thiergart und Prof. Dr. Andreas Welling
Bitte
beachten Sie: Im Zuge der E-Mail-Korrespondenz werden persönliche Daten
gespeichert und verarbeitet. Näheres entnehmen Sie bitte unserer
Datenschutzerklärung https://embever.com/privacy-policy/.

Hi, @pritstift, emqx-3.2-rc.1 has been released now. Welcome to try with it.


Reorganize the documentation files in ACA-Py to reduce the adjustments needed when deploy the aca-py.org docs site

The Aries Cloud Agent Python documentation site at https://aca-py.org is currently assembled by copying the various markdown files from parts of the ACA-Py repo into the “right place” in the source repository (https://github.com/hyperledger/aries-acapy-docs) for the documentation site.
The script that generates the mkdocs site can be used as the model for where the documents should go.

Is this ready to be closed?

Yup — alll done, including updates to the https://aca-py.org site, which has been significantly simplified.


Flutter web can't run in debug mode on Chrome nor Brave

I just switched to Brave because Chrome would only run in release mode but I need to debug and it just has the loading stripe at the top of the browser viewport going left to right and never loading.
flutter doctor -v
[✓] Flutter (Channel stable, 2.5.3, on Ubuntu 20.04.3 LTS 5.11.0-38-generic, locale
    en_CA.UTF-8)
    • Flutter version 2.5.3 at /home/nobu/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (4 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /home/nobu/Android/Sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at:
      /home/nobu/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7621141/jr
      e/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/brave-browser-stable

[✓] Android Studio (version 2020.3)
    • Android Studio at
      /home/nobu/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7621141
    • Flutter plugin version 62.0.1
    • Dart plugin version 203.8452
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.62.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.28.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Brave Browser <IP_ADDRESS>

• No issues found!

command used:
flutter run -d chrome
Launching lib/main.dart on Chrome in debug mode...
Waiting for connection from debug service on Chrome...             29.9s

Actually even in release mode the first time never works but the second time does (in Brave).
Switching back to Chrome Executable and flutter doctor -v
[✓] Flutter (Channel stable, 2.5.3, on Ubuntu 20.04.3 LTS 5.11.0-38-generic, locale
    en_CA.UTF-8)
    • Flutter version 2.5.3 at /home/nobu/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (4 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /home/nobu/Android/Sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at:
      /home/nobu/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7621141/jr
      e/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/google-chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at
      /home/nobu/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7621141
    • Flutter plugin version 62.0.1
    • Dart plugin version 203.8452
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.62.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.28.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 95.0.4638.69

• No issues found!

command flutter run -d chrome results in same loading animation at top of browser viewport and never loading.
Now try flutter run -d chrome --release results in pretty much the same as Brave, seems to work the second time.
What is the workaround?  I read somewhere even if I downgrade Chrome, it's going to upgrade itself so is there a way to downgrade to Chrome 93 (might have heard this one works) and then lock in that version?

Actually an app with no flutter_login package runs fine.  So I filed an issue with them: https://github.com/NearHuscarl/flutter_login/issues/271

Hi @nyck33
Given your last message I feel safe to close this issue, if you disagree please write in the comments and I will reopen it.
Thank you


Add govuk-csp-forwarder lambda function

This commit creates a new lambda function and deploys the govuk-csp-forwarder to it.

Don't worry, it'll all be in the dev docs :)


Add unique_hhook

This is for SetWindowsHookEx and UnhookWindowsHookEx.

@sylveon Please also update documentation.

@oldnewthing Thanks for the reminder, done.


Feature request: Async configuration via webpack.config.js

Feature request: Async configuration via webpack.config.js
Webpack version:
1.12.2
Please tell us about your environment:
Linux (OS independent problem)
Current behavior:
Configuration in webpack.config.js must define an object as module.exports.
Expected/desired behavior:
Configuration in webpack.config.js should also support async resolving of module.exports (e.g. by assigning a promise, future and/or callback to module.exports).
What is the motivation / use case for changing the behavior?
I stepped upon the same problem as described in defineplugin-from-async.
In my case I would like to include some git information (branch, last commit, current value of git-describe) as a constant into the bundled javascript code. With help of nodegit, it is relatively simple to retrieve this - but nodegit is an asynchronous node library.
While there is a work-around (by running webpack through the node API, see link above), my feeling is that allowing an async configuration webpack would simplify more advanced use-cases.

Yeah. With webpack 2 you get
exports default function(options) {
    return {
        // ...
        devtool: options.dev ? "cheap-module-eval-source-map" : "hidden-source-map"
    };
}

Expanding to
exports default function(options, cb) {
    cb(null, {
        // ...
        devtool: options.dev ? "cheap-module-eval-source-map" : "hidden-source-map"
    });
}

would seem like a good move to me.

You can return a Promise with webpack@2

It also works with Webpack 1. It doesn't work yet with Webpack 1's development server, however. See https://github.com/webpack/webpack-dev-server/pull/698 for that.

For reference, for anyone stumbling here, returning a promise like what @sokra said looks something like this:
const configPromise = new Promise(function(resolve, reject) {

    // ... do async stuff ...

    // then at some point call `resolve()` with the config object:
    resolve({
        entry: { /* ... */ },
        resolve: { /* ... */ },
        // etc
    })
})

module.exports = configPromise


Changelog v1.17.0-rc1

please code comment so we can apply them

To Add:

[ ] #20021
[x] #20024
[ ] #20027

https://github.com/go-gitea/gitea/issues?q=milestone%3A1.17.0+is%3Aclosed+label%3Akind%2Fbreaking

https://github.com/go-gitea/gitea/issues?q=milestone%3A1.17.0+is%3Aclosed+label%3Akind%2Fbreaking has a little bit more...

strange ... wrong priorizing within https://github.com/go-gitea/gitea/blob/main/.changelog.yml ?

https://github.com/go-gitea/gitea/issues?q=milestone%3A1.17.0+is%3Aclosed+label%3Akind%2Fbreaking has a little bit more...

strange ... wrong priorizing within https://github.com/go-gitea/gitea/blob/main/.changelog.yml ?

Indeed. Make AppDataPath absolute against the AppWorkPath if it is not (#19815) for example is under bug.

https://gitea.com/gitea/changelog/issues/61
next changelog generation will be better - cc @delvh

ok I'm now fine with this as is - so it will get my LGTM - now we just more of them

https://github.com/go-gitea/gitea/pull/20030

All PRs backported and skip-change should be removed from changelog.

All PRs backported and skip-change should be removed from changelog.

If not added manualy they should never have been in here...

Shouldn't this be mentioned in the changelog? https://github.com/go-gitea/gitea/pull/18971
Although these error messages are quite self-explanatory, people may wonder why they appear suddenly after upgrading to 1.17.

🚀

Shouldn't this be mentioned in the changelog? #18971

Probably should be mentioned in the blog post for 1.17.0

It's only an rc atm ...

I think we dont have to frontport until we do the big release


Task not run when used with Vite server

Intro
When used with Vite start-server-and-test fails to run the specified subtask
This is a continuation of the prematurely closed bahmutov/start-server-and-test/issues/312 but additionally provides a minimal repro:
https://github.com/oliverturner/start-server-vite-minimal-repro
Bug report
Version:

"start-server-and-test": "1.13.1",
"vite": "2.4.4"

Platform:
System:

OS: macOS 11.5.1
CPU: (4) x64 Intel(R) Core(TM) i7-7660U CPU @ 2.50GHz
Memory: 203.41 MB / 16.00 GB
Shell: 3.3.1 - /usr/local/bin/fish

Binaries:

Node: 12.22.3 - ~/.volta/tools/image/node/12.22.3/bin/node
npm: 7.20.2 - ~/.volta/tools/image/npm/7.20.2/bin/npm

Expected behavior
The specified task is run after Vite has spun up
Actual behavior
Vite starts but the task is not run
CLI output:
><EMAIL_ADDRESS>test
> start-test dev 3000 subtask

1: starting server using command "npm run dev"
and when url "[ 'http://localhost:3000' ]" is responding with HTTP status code 200
running tests using command "npm run subtask"

><EMAIL_ADDRESS>dev
> vite --clearScreen false

  vite v2.4.4 dev server running at:

  > Local: http://localhost:3000/
  > Network: use `--host` to expose

  ready in 149ms.

Subsequently...
Error: Timed out waiting for: http://localhost:3000
    at /Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/wait-on/lib/wait-on.js:132:31
    at doInnerSub (/Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/rxjs/dist/cjs/internal/operators/mergeInternals.js:21:26)
    at outerNext (/Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/rxjs/dist/cjs/internal/operators/mergeInternals.js:16:70)
    at OperatorSubscriber._this._next (/Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/rxjs/dist/cjs/internal/operators/OperatorSubscriber.js:28:21)
    at OperatorSubscriber.Subscriber.next (/Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/rxjs/dist/cjs/internal/Subscriber.js:72:18)
    at AsyncAction.work (/Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/rxjs/dist/cjs/internal/observable/timer.js:28:28)
    at AsyncAction._execute (/Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/rxjs/dist/cjs/internal/scheduler/AsyncAction.js:76:18)
    at AsyncAction.execute (/Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/rxjs/dist/cjs/internal/scheduler/AsyncAction.js:64:26)
    at AsyncScheduler.flush (/Users/oliver.turner/Dev/xpmt/vite/start-server-vite-minimal-repro/node_modules/rxjs/dist/cjs/internal/scheduler/AsyncScheduler.js:39:33)

Have you read https://github.com/bahmutov/start-server-and-test#note-for-webpack-dev-server-users ?

Thank you for the pointer. Having read it, here's the output of running Vite and wait-on in parallel:
Terminal tab 1:
start-server-vite-minimal-repro on  main via  v12.22.3
➜ npm run dev

><EMAIL_ADDRESS>dev
> vite --clearScreen false

  vite v2.4.4 dev server running at:

  > Local: http://localhost:3000/
  > Network: use `--host` to expose

  ready in 334ms.

Terminal tab 2:
start-server-vite-minimal-repro on  main via  v12.22.3
➜ npx wait-on -lv http://localhost:3000/index.html
waiting for 1 resources: http://localhost:3000/index.html
making HTTP(S) head request to  url:http://localhost:3000/index.html ...
  HTTP(S) result for http://localhost:3000/index.html: {
  status: 200,
  statusText: 'OK',
  headers: {
    'access-control-allow-origin': '*',
    'content-type': 'text/html',
    'cache-control': 'no-cache',
    etag: 'W/"197-lq2+FOV5/NFodCUBMeCoaeYaQtc"',
    date: 'Tue, 03 Aug 2021 08:17:47 GMT',
    connection: 'close'
  },
  data: ''
}
wait-on(2881) complete

It appears that the issue is unrelated to the http verb used (http-get: also succeeds)


#160927001 Consume Authentication API with Fetch API on Templates

What does this PR do?
Provide implementation for authentication endpoints, both admin and users, login and sign up,
Description of Task to be completed?
Make templates consume sign up and login endpoints with fetch API
How should this be manually tested?

Clone the app folder, change directory to the root of the folder, run npm install, start your local server then open your the templates pages locally and try to signup or login signup and login pages

Any background context you want to provide?
This implementation does not have validation for now
What are the relevant pivotal tracker stories?
#160927001

Pull Request Test Coverage Report for Build 162

6 of 6 (100.0%)  changed or added relevant lines in 3 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.5%) to 80.422%

Files with Coverage Reduction
New Missed Lines
%

api/v2/middlewares/orderValidator.js
3
65.85%

Totals

Change from base Build 154:
-0.5%

Covered Lines:
440

Relevant Lines:
513

💛  - Coveralls


CI/CD Performance Enhancement

Adds .eslintrc.js to each project
Speeds up internal ci/cd pipeline by 275% (22m to ~8m)

@fredbonin - Antoine figured out how to bring down the time it takes eslint in the ci/cd pipeline down to about 6-7min - 20+minutes is insanely frustrating. He ended up adding the .eslintrc to each project, for some reason having only 1 was making the eslint go hunting for references for a long time. Adding them to each project dropped each from 20-30 sec to 5-6 seconds each. Is this a good approach? How do we ensure each project has the same eslint settings? Do we add a script to auto-add and update these as projects are added?

This is ready for review. Main modification is parallelizing lint and typecheck.

45 minutes to 13minutes is very impressive!

Note for future reference, as the solution moves to sdk v3 and starts importing specific sdk libraries for each project, it will improve compilation time for the "Typecheck" step which recursively run pnpx tsc --noEmit. The compilation time gets better since tsc will spend less time on i/o,parse,bind and check.
Comparison below of a random project compile time (note Total time).
SDK Currently in-use v2
Files:                         445
Lines of Library:             8102
Lines of Definitions:       553661
Lines of TypeScript:           259
...
Nodes of Library:            22493
Nodes of Definitions:      1619617
Nodes of TypeScript:           928
...
Identifiers:                601234
Symbols:                    377991
Types:                      131593
Instantiations:              12546
Memory used:               597267K
Assignability cache size:     5199
Identity cache size:             5
Subtype cache size:             13
Strict subtype cache size:       0
I/O Read time:               0.38s
Parse time:                  2.85s
ResolveModule time:          0.36s
ResolveTypeReference time:   0.01s
Program time:                3.68s
Bind time:                   0.98s
Check time:                  2.90s
printTime time:              0.00s
Emit time:                   0.00s
Total time:                  7.56s

real    0m8.077s
user    0m11.336s
sys     0m0.757s

SDK v3
Files:                        426
Lines of Library:            8102
Lines of Definitions:       47212
Lines of TypeScript:          258
...
Nodes of Library:           22493
Nodes of Definitions:      151928
Nodes of TypeScript:          916
...
Identifiers:                59080
Symbols:                    53415
Types:                      17724
Instantiations:             11909
Memory used:               95829K
Assignability cache size:    3391
Identity cache size:            5
Subtype cache size:            13
Strict subtype cache size:      0
I/O Read time:              0.07s
Parse time:                 0.59s
ResolveModule time:         0.37s
ResolveTypeReference time:  0.01s
Program time:               1.09s
Bind time:                  0.27s
Check time:                 1.14s
printTime time:             0.00s
Emit time:                  0.00s
Total time:                 2.50s

real    0m2.895s
user    0m4.756s
sys     0m0.273s

Please don't merge this PR yet, I found an alternative that might reduce the time even more (still need to investigate more though).
It also does not require new .eslintrc.js for each project.

Hi,
Here is the PR that reduce the time even further : https://github.com/aws-samples/aws-secure-environment-accelerator/pull/852. I suggest that we close this one and merge the other one instead.

Closed in favor of #852


Examples for the serde and serde-json usage

Could you please add some examples for the usage of the serde and serde-json features?

Done. You'll find the examples in the examples folder.


[Event Request] report 699 "Calculate Plan - Req. Wksh." -  OnAfterItemOnPostDataItem

Could you please add the following parameters to the event OnAfterItemOnPostDataItem?

CurrTemplateName: Code[10];
CurrWorksheetName: Code[10];
FromDate: Date
ToDate: Date

dataitem(Item; Item)
{
    DataItemTableView = SORTING("Low-Level Code") WHERE(Type = CONST(Inventory), Blocked = CONST(false));
    RequestFilterFields = "No.", "Search Description", "Location Filter";

    ...

    trigger OnPostDataItem()
    begin
        //+Change
        //-OnAfterItemOnPostDataItem(Item);
        OnAfterItemOnPostDataItem(ItemOnAfterItemOnPostDataItem, CurrTemplateName, CurrWorksheetName, FromDate, ToDate);
        //-Change
    end;
}

[IntegrationEvent(false, false)]
//-local procedure OnAfterItemOnPostDataItem(var Item: Record Item) //OLD
local procedure OnAfterItemOnPostDataItem(var Item: Record Item;  CurrTemplateName: Code[10]; CurrWorksheetName: Code[10]; FromDate: Date; ToDate: Date) //NEW
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


sbi utils missing

Hi,
I'm not sure where this issue should go, because it's certainly a problem on my computers end, but it's also somewhat unique to sbi.  I have it installed and working on my computer but for a variety of reasons I need it on another conda enviroment.  However when I run for example: from sbi.inference.base import infer, I get this error:
Traceback (most recent call last):
File "sbitest_full_dp.py", line 14, in
from sbi.inference.base import infer
File "/home/exx/.conda/envs/sbip36/lib/python3.6/site-packages/sbi/inference/init.py", line 14, in
from sbi.inference.abc.mcabc import MCABC
File "/home/exx/.conda/envs/sbip36/lib/python3.6/site-packages/sbi/inference/abc/mcabc.py", line 8, in
from sbi.inference.abc.abc_base import ABCBASE
File "/home/exx/.conda/envs/sbip36/lib/python3.6/site-packages/sbi/inference/abc/abc_base.py", line 11, in
from sbi.simulators.simutils import simulate_in_batches
File "/home/exx/.conda/envs/sbip36/lib/python3.6/site-packages/sbi/simulators/init.py", line 1, in
from sbi.simulators.linear_gaussian import diagonal_linear_gaussian, linear_gaussian
File "/home/exx/.conda/envs/sbip36/lib/python3.6/site-packages/sbi/simulators/linear_gaussian.py", line 11, in
from sbi.utils import within_support
File "/home/exx/.conda/envs/sbip36/lib/python3.6/site-packages/sbi/utils/init.py", line 8, in
from sbi.utils.get_nn_models import classifier_nn, likelihood_nn, posterior_nn
File "/home/exx/.conda/envs/sbip36/lib/python3.6/site-packages/sbi/utils/get_nn_models.py", line 15, in
from sbi.neural_nets.mdn import build_mdn
File "/home/exx/.conda/envs/sbip36/lib/python3.6/site-packages/sbi/neural_nets/mdn.py", line 9, in
import sbi.utils as utils
AttributeError: module 'sbi' has no attribute 'utils'
I think its due to mutliple copies of code named sbi in the enviroment but I really only have the sbi module in this enviroment although I have sbi modules outside of this enviroment.  Obviously you can see sbi.utils exist as it gets called earlier in the Traceback

From here maybe the problem is python 3.6: https://github.com/bayesiains/nflows/issues/7 .  However running CentOS I think I have a problem with this: libstdc++.so.6: version `GLIBCXX_3.4.21'  error when I run in python command line and downgrading to python 3.6 fixed this.  For example see this: https://github.com/tensorflow/tensorflow/issues/26826

Hi @CameronFen, and thanks for reporting this! Were you able to solve the issue?

Yes downgrading the env python to 3.6 fixed this.  I think this error has something to do with libstdc version for python 3.7+ is not compatible with nflows.  I had this same problem with PyTorch geometric so it’s not unique to your stack.


Persistit cleanup

When gremlin is shut down without shutting down the graph, this error appears and the shell hangs:
[CLEANUP_MANAGER] WARNING Transaction abandoned Transaction_100000003 depth=1 status=<ts=17,896 tc=ABORTED mvv=0> owner=main

Recommend closing >> Persistit related issues.


no popup?

Hi,
IntelliJ IDEA 2021.2.2-1 (Community Edition)
It's work,But no popup anything?I can use shortcut,but no popup?

Have you installed https://plugins.jetbrains.com/plugin/15976-which-key/ ?
Do you have source ~/.intellimacs/which-key.vim in your .ideavimrc?

Yes,But I don't know how to give you more information.

Sorry, but for me it's working both in CLion and IntellJ IDEA 2021.2.2 on linux.

I don't know how to reproduce your issue.
Make sure you have pulled the latest version of this repo.


Modify integration and unit test for sl-alert for better test coverage

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

tests/integration/components/sl-alert-test.js, line 41 [r1] (raw file):
The value used here should not be one of the theme(s) (values) supported by the component.  The purpose of this test is to see whether or not the theme property is used in a certain way, in this case concatenated with some other text.  By using "warning" two things have now occurred: 1) why just "warning"?  why not all of the other values too?  If the answer is that you're not specifically testing "warning" then it leads to 2) by using a value to test with that this component likely has no support for in its logic this test will ensure that during a future refactor logic isn't introduced to hard code the "warning" value or something similar.  By using the value of "testValue" we get the most test coverage for our effort.

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 1 unresolved discussion.

tests/integration/components/sl-alert-test.js, line 41 [r1] (raw file):
After conversation with @juwara0 my comment can be disregarded.

Comments from the review on Reviewable.io


Fix #1205 - move rgb() to CSS var definition.

Changes in this Pull Request:

Moved rgb() function to variable definition.
In the Backend the following values must be set if light navigation bar is desired:
--oscm-navbar-color: rgba(var(--oscm-white), 0.8);
--oscm-navbar-links-color: rgb(var(--oscm-primary));

Mandatory checks:

[X] Branch is up to date (latest changes were fetched from the upstream branch before creating this PR)
[X] Changes were tested manually
[ ] Java code changes are unit tested
[ ] Webtests are successful

To be checked by the reviewer:

[ ] Clean Java code and unit tested changes
[ ] UI customizablity is preserved: No hard-coded styling and URL references in JSF views

To be checked by the pull request owner:

[ ] .MASTER/job/BES_MASTER_IT/
[ ] .MASTER/job/OSCM_WS_Tests_INTERNAL/
[ ] .MASTER/job/OSCM_WS_Tests_OIDC/
[ ] .MASTER/job/OSCM_Integration_Tests_INTERNAL/
[ ] .MASTER/job/OSCM_Integration_Tests_OIDC/

Cross repository and core changes:

[ ] Interdependent changes have been communicated to the team (if applicable)
[ ] This pull request includes high risk modifications or core changes (e.g API, datamodel, authentication...). Mandatory reviewers: @goebell or @kowalczyka.

OSCM Build References:

<CI node number>: <tag>, example: ci10:goebell
-est6

Screenshot (if applicable):

add a screenshot illustrating your change

This change is

Close PR as in the meanwhile the color space has been modified from rgb to hsl, therefore above changes are not necessar. The --oscm-navbar-color variables are given as hsl values.


Support auto-discovery for client definition defaults

If the sensu-client could derive more client configuration default values from the running system (e.g. use the hostname as the default client name), it could greatly simply monitoring in highly ephemeral environments (e.g. container-based environments).
Minimum requirements:

[ ] Auto discover/set the client name
[ ] Auto discover/set the client address

Ideally, the minimum configuration required for a Sensu client should be a transport connection definition (which may be provided via configuration or environment variables).

Somewhat related: #1276

Implemented auto-discovery of client name and address in sensu/sensu-settings#37, shipped in sensu-settings 6.0.0.
https://github.com/sensu/sensu/pull/1379 adds dependency on sensu-settings 6.0.0, so this feature will ship with 0.26.


Open Utau does not install on Linux

Describe the bug
I recently uninstalled Ubuntu Studio 21.04 to install version 21.10 and when I went to install the app in App.image format, it didn't open. I tried to do it through the terminal and this warning appeared:
Unhandled exception. System.DllNotFoundException: Unable to load shared library 'libdl' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: liblibdl: cannot open shared object file: No such file or directory
at OpenUtau.LibraryLoader.dlopen(String fileName, Int32 flags)
at OpenUtau.LibraryLoader..ctor(String path, String library) in /media/sf_Code/OpenUtau/OpenUtau.Core/Util/LibraryLoader.cs:line 28
at OpenUtau.Audio.AudioOutput..ctor() in /media/sf_Code/OpenUtau/OpenUtau.Core/Audio/AudioOutput.cs:line 36
at OpenUtau.App.Program.InitAudio() in /media/sf_Code/OpenUtau/OpenUtau/Program.cs:line 67
at OpenUtau.App.Program.Main(String[] args) in /media/sf_Code/OpenUtau/OpenUtau/Program.cs:line 28
Aborted (core dumped)
Environment:

OS: Linux (Ubuntu Studio)
Version [21.10]

see https://github.com/stakira/OpenUtau/issues/124#issuecomment-1034489541


add passive scalars

This adds passive scalar support to Quokka.
The number of passive scalars is set at compile time with HydroSystem_Traits<>::nscalars. This updates the component indicies used in both HydroSystem and RadSystem, as well as the total number of variables in RadhydroSimulation. A new function is added to set the names of the passive scalars as used in the plotfiles.

/azp run

/azp run

/azp run

/azp run


Events With Multiple Local Handlers Can Be Interfered By Each Other

Today, an event with multiple local handlers registered can interfere with each other as all handlers are given the same shared copy of events.
For instance,
# anywhere in code

dispatch(Events.USER_CREATED, {"key_in_payload": 123})

# in handlers.py

@local_handler.register(event_name=Events.USER_CREATED)
async def handle_user_created(event: Event):
    _, payload = event
    payload.pop("key_in_payload")

@local_handler.register(event_name=Events.USER_CREATED)
async def handle_user_created_2(event: Event)
    _, payload = event
    payload["key_in_payload"]  # KeyError

Proposal
A copy of the payload should be passed to the handlers.

The proposal will solve this issue on a small level, but when the payload size is big and there are multiple consumers who are consuming the dispatched event at the same time this will become a big issue it will block all the memory. Especially for long-running tasks or coroutines.
A quick fix that comes into my mind is writing a wrapper inside dispatch to make the payload a frozen dict.
multiple handlers could accept it, frozen dict won't change. if anyone wants to change any property of that dict they could create a separate copy of that dict or get items from that dict.

@danielhasan1 you're right. I haven't tried to address this issue because the payload can technically be of any type, not just dict. Either approach risks breaking the user's code if the payload contains values (custom objects, etc) that cannot be cloned or frozen.
I'm also not aware of any "standard" approach to making a dict immutable as PEP 416 was rejected and PEP 603 is still under drafting stage.
I think it's best we keep this unfixed for now.

I think it's not about this library, it's about our design. events should be immutable. as a developer you should try design to have immutable events. for example you can try pydantic frozen classes or frozen dataclass or something that guarantee immutable events.
one suggestion is to change library to only accept immutable events data :D


refactor(fee): use valid resource bounds in blockifier

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#511  👈
#510
#509
#508
#507
#506
#505
#504
#503
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nimrod-starkware and the rest of your teammates on  Graphite


Migrate to GH Actions

https://github.com/metanorma/metanorma-build-scripts/issues/23

@CAMOBAP795 failing on macOS and Windows.

Two problems here:

Some incompatibility issues for Ruby 2.6 on Windows, I noticed if https://github.com/riboseinc/asciidoctor-bibliography.git has been removed from Gemfile, it works IDK why, so I propose to remove it from Gemfile because we don't use it

Issue on windows

C:\Users\IEUser\asciidoctor-rfc>ruby bin/asciidoctor-rfc2 some.adoc
spec/examples/hoffmanv2.xml.adocTraceback (most recent call last):
        8: from bin/asciidoctor-rfc2:4:in `<main>'
        7: from C:/tools/ruby25/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:39:in `require'
        6: from C:/tools/ruby25/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:135:in `rescue in require'
        5: from C:/tools/ruby25/lib/ruby/2.5.0/rubygems/core_ext/kernel_require.rb:135:in `require'
        4: from C:/tools/ruby25/lib/ruby/gems/2.5.0/gems/metanorma-ietf-1.0.6/lib/asciidoctor/rfc/v2/converter.rb:3:in `<top (required)>'
        3: from C:/tools/ruby25/lib/ruby/gems/2.5.0/gems/metanorma-ietf-1.0.6/lib/asciidoctor/rfc/v2/converter.rb:3:in `require_relative'
        2: from C:/tools/ruby25/lib/ruby/gems/2.5.0/gems/metanorma-ietf-1.0.6/lib/asciidoctor/rfc/common/base.rb:10:in `<top (required)>'
        1: from C:/tools/ruby25/lib/ruby/gems/2.5.0/gems/metanorma-ietf-1.0.6/lib/asciidoctor/rfc/common/base.rb:11:in `<module:Asciidoctor>'C:/tools/ruby25/lib/ruby/gems/2.5.0/gems/asciidoctor-1.5.8/lib/asciidoctor.rb:1616:in `const_missing': uninitialized constant Asciidoctor::Rfc (NameError)

Separate issue reported for failed tests on windows https://github.com/metanorma/asciidoctor-rfc/issues/108

Thanks @CAMOBAP795 !


Python class/function textobjects should include decoraters

This will make textsubjects consistent with nvim-treesitter's @class.outer and @function.outer,
and much more useful when working with whole function/class.

Yea sure, merged C++. I don't remember being notified of the PR, must've gotten buried in my inbox.


[SPARK-23377][ML] Fixes Bucketizer with multiple columns persistence bug

What changes were proposed in this pull request?
Since 2.3, Bucketizer supports multiple input/output columns. We will check if exclusive params are set during transformation. E.g., if inputCols and outputCol are both set, an error will be thrown.
However, when we write Bucketizer, looks like the default params and user-supplied params are merged during writing. All saved params are loaded back and set to created model instance. So the default outputCol param in HasOutputCol trait will be set in paramMap and become an user-supplied param. That makes the check of exclusive params failed.
This patch changes DefaultParamsWriter and only save user-supplied params.
How was this patch tested?
Modified test.

Test build #87283 has started for PR 20566 at commit 7785cac.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/769/
Test PASSed.

Test build #87283 has finished for PR 20566 at commit 7785cac.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87283/
Test FAILed.

Merged build finished. Test FAILed.

Test build #87285 has started for PR 20566 at commit 3b5e7c6.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/771/
Test PASSed.

Merged build finished. Test PASSed.

@viirya that's a good question. Honestly my idea is that if the user doesn't set a value, he/she doesn't care about it, so it is good to use the new version default IMHO. But it is also true that changing a default may cause unexpected behavior in user code.
So, it LGTM, but I'd like to hear others' opinion on this too.

Yeah, IMHO, when the user loads a model from old version into new version to run, I think it is reasonable to run it with current default value because the param is not explicitly set and should use "default" value of current system.
Thanks for your comment. Let's wait for others' option.

Test build #87285 has finished for PR 20566 at commit 3b5e7c6.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87285/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/775/
Test PASSed.

Test build #87289 has started for PR 20566 at commit 6228006.

Test build #87289 has finished for PR 20566 at commit 6228006.

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

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87289/
Test FAILed.

I believe this will break persistence for LogisticRegression. I believe the issue is that the threshold param on LogisticRegressionModel doesn't get a default directly, but only gets it during the call to fit on LogisticRegression. This is currently fine because the Model can only be created by fitting or by being read from disk and in both case some value gets set for threshold. With this change that's no longer the case. Here's a test to confirm, https://github.com/apache/spark/commit/5db2108224accdf848b41ef0d8d1c312b49f49c6.
I believe LinearRegression may have a similar issue.
Our current tests don't seem to cover this kind of thing so I think we should improve test coverage if we want to make this kind of change.

Not only threshold, the default params of NaiveBayes, LogisticRegression (maybe more, I'm looking up now) are all set in the estimator, not in their model. The models are received the default values at the end of fit.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/785/
Test PASSed.

Test build #87299 has started for PR 20566 at commit daceafe.

Test build #87299 has finished for PR 20566 at commit daceafe.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87299/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/787/
Test PASSed.

Merged build finished. Test PASSed.

Test build #87301 has started for PR 20566 at commit c1fb657.

Test build #87301 has finished for PR 20566 at commit c1fb657.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87301/
Test FAILed.

retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/788/
Test PASSed.

Test build #87302 has started for PR 20566 at commit c1fb657.

Test build #87302 has finished for PR 20566 at commit c1fb657.

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

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87302/
Test PASSed.

Merged build finished. Test PASSed.

cc @MLnick @jkbradley

Thanks for the patch @viirya
As always, I'll request that we put design decisions & long discussions in JIRA so that they are easier to uncover.  It can also be good to get quick feedback about design before implementation.  I'll comment in JIRA.

@jkbradley Thanks! I will post the problem and proposed design on the JIRA.

I'd close this and favor the quick fix #20594 based on the discussion in JIRA. Will re-open it if it is needed later.


Bulk Recommendations Uploads: Catch Exceptions When Parsing a CSV Row

Context
Sentry Error
This exception looks like it's caused by bad data, but we currently raise any exceptions all the way up to the controller and do nothing to handle it. We have a ValidateCsvRow class that validates the row on initialisation. Any errors are added to a messages array, and the valid? method just checks if there are any messages in that array. But if an exception is raised, we raise this error all the way up to the controller and the user will just see the standard 500 error page.
Changes proposed in this pull request
The Sentry error points to the transliterate method being the main culprit. So I'm now rescuing any errors in that method and adding an appropriate error message with the text that caused the exception. That will then be handled just like any other error with the CSV:
def transliterate(string)
  return unless string

  I18n.transliterate(string.dup.force_encoding(ENCODING), replacement: "")&.downcase
rescue ArgumentError
  @messages << error_message(:transliteration, string:)
end

I'm also catching any other exceptions in the validate! method and adding a more generic error message if it's one we haven't thought of. That will at least be a more graceful failure for the user.
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Bulk Recommendations Uploads Sentry Error


Profiles for websites!

Is making profiles for certain Websites/URL's an idea for Aurora?
Example: Youtube= Red Spacebar, Arrowkeys, Numbers etc.

Will look into. I will put it on the back stove for now, as it would require memory reading or I would prefer to have a plugin for Chrome/Firefox to show website name and other information to be displayed on the keyboard.
Good suggestion though. But I will need help with the plugin/extension creation.

Thank you very much for the prompt reply. I understand and wish you the best of luck with extending  this amazing project!

I will be looking into this for version 0.5.1

Hi,
May I suggest that in your brower plugin you add a javascript api for websites so websites that are aware of your plugin can add their custom behavior to lighting ? For example, javascript games hosted on websites can add lighting feedback...
Thanks :)

Hey! I'm javascript developer, so I can build that browser plugin (at least for Chrome/FIrefox/Opera, also Safari and MS Edge shouldn't be a problem as far as I know). That can be a browser extension and built-in server inside Aurora which should realize communication between them. So, if you want - I can build that javascript part.

Also, I can make an SDK for website developers which can be loaded with browser plugin.

@ozio That would be great! Do you want to add me on steam or something so that we can discuss it more?

Is someone still working on this or has it kind of stalled out?

@AlpacaFur After some discussions with @simon-wh I started working on it, but yes, it's kind of stalled. The most important thing which is not defined yet is the interface and transport between Aurora and browser extension. When it will be defined - I can implement everything without much difficulty.


Standalone: Resource files obtained by importlib.resources.files(...).iterdir() cannot be sorted

Problem description:
Another PyPI package, Pyphen, assumes that objects returned from importlib.resources.files(...).iterdir() can be compared using the < operator, since it calls sorted() on the .iterdir() result, but the importlib.resources.abc.Traversable Protocol does not guarantee that objects returend by .iterdir() can be compared using the < operator. See also https://github.com/Kozea/Pyphen/issues/54 for more info. Perhaps of an unfortunate misunderstanding the issue on Pyphen is already closed, while I still think that one should not rely on functionality that appears to be working but is not backed by documentation describing that it should.
Nonetheless the Nuitka developer/maintainer asked me to create an issue on Nuitka, since he wants to stay as compatible as possible with plain Python functionality.
So here we go:
Versions
> python -m nuitka --version
1.7.10
Commercial: None
Python: 3.10.4 (tags/v3.10.4:9d38120, Mar 23 2022, 23:13:41) [MSC v.1929 64 bit (AMD64)]
Flavor: CPython Official
Executable: D:\workspace\projects\nuitka-problem\.venv310\Scripts\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.37.32822\bin\Hostx64\x64\cl.exe (cl 14.3).

Installation info
> where python
D:\dev\Python310\python.exe
....

Virtual environment
Created with
python -m venv .venv310

Activated with
> .venv310\Scripts\activate
> where python
D:\workspace\projects\nuitka-problem\.venv310\Scripts\python.exe
D:\dev\Python310\python.exe
...

Installed packages
> python -m pip install -U pip
...
> python -m pip install Nuitka
...
> python -m pip freeze
Nuitka==1.7.10
ordered-set==4.1.0
zstandard==0.21.0

Source files
See attached .zip file → implib_sorted_resources_problem.zip
> dir /B/S D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem
D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\main.py
D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\some_package
D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\some_package\handle_resources.py
D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\some_package\some_resources
D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\some_package\__init__.py
D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\some_package\some_resources\file_A.txt
D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\some_package\some_resources\file_B.txt

Output using plain Python
> cd implib_sorted_resources_problem
> python -m main
WindowsPath('D:/workspace/projects/nuitka-problem/implib_sorted_resources_problem/some_package/some_resources/file_A.txt')
WindowsPath('D:/workspace/projects/nuitka-problem/implib_sorted_resources_problem/some_package/some_resources/file_B.txt')

Nuitka build command
> python -m nuitka --standalone --include-package-data=some_package main.py
Nuitka-Options:INFO: Used command line options: --standalone --include-package-data=some_package main.py
Nuitka:INFO: Starting Python compilation with Nuitka '1.7.10' on Python '3.10' commercial grade 'not installed'.
Nuitka:INFO: Completed Python level compilation and optimization.
Nuitka:INFO: Generating source code for C backend compiler.
Nuitka:INFO: Running data composer tool for optimal constant value handling.
Nuitka:INFO: Running C compilation via Scons.
Nuitka-Scons:INFO: Backend C compiler: cl (cl 14.3).
Nuitka-Scons:INFO: Backend linking program with 9 files (no progress information available for this stage).
Nuitka-Scons:INFO: Compiled 9 C files using clcache with 8 cache hits and 1 cache misses.
Nuitka-Options:INFO: Included data file 'some_package\some_resources\file_A.txt' due to package 'some_package' package data.
Nuitka-Options:INFO: Included data file 'some_package\some_resources\file_B.txt' due to package 'some_package' package data.
Nuitka:INFO: Keeping build directory 'main.build'.
Nuitka:INFO: Successfully created 'main.dist\main.exe'.

Actual output using Nuitka-built version
> main.dist\main.exe
Traceback (most recent call last):
  File "D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\main.dist\main.py", line 9, in <module>
  File "D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\main.dist\main.py", line 5, in main
  File "D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\main.dist\some_package\handle_resources.py", line 7, in doit
TypeError: '<' not supported between instances of 'nuitka_resource_reader_files' and 'nuitka_resource_reader_files'

Additional info:
> dir /B D:\workspace\projects\nuitka-problem\implib_sorted_resources_problem\main.dist\some_package\some_resources
file_A.txt
file_B.txt

I see, comparison is also something I have not seen before, but sorting is of course a thing bound to be desired.

Thanks, that was super easy to use. The only way it could have been better would be to provide project options with  # nuitka-project: as described in the user manual, but this super easy to reproduce, thanks a lot.
Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback if this is working is very welcome, just please do not share plans of doing it, but rather confirmations or denials of it working.

It works now. Thank you!

It works now. Thank you!

Explanation: Richard works in the same project as me. He verified this problem with the factory version.

No worries, I do not actually manage to track who said what anyway. If you have more like these, swing them my way. Nuitka is supposed to be compatible, but for all I know, there is no ultimate API description that I can test against, so this is continously catching up to what Python does and thus people expect to happen. Very happy with these reports.

This was released with 1.8 and should definitely be in 1.9, this was forgotten to be updated.


Add a new run loop function to coalesce requests until timer ends

We are needing a new run loop function that works similar to debounce, but functions a little differently.
How it would work is as follows:

A call is made to the coalesce function and the method is run immediately. A timer is also started for that method signature.
Each additional call cancels the current timer and creates a new one (like debounce). A counter is also incremented for each time the method signature is called.
When the timer is complete, if the counter is greater than 0 (meaning the method was called more than once during the timeout period), the method is called a second time.

Our use case is, we have a web socket that pushes data updates to the front end on any backend change. We want the first data update to fire immediately, but then coalesce all the remaining data updates until the timer expires. We find that we are updating the data several times on the backend (with various backend processes), but want to limit the number of requests on the front end until we know the changes are complete.
Debounce and throttle don't meet this requirement. I looked at extending backburner locally, but that would require rewriting and implementing many of the internal functionality to add this new method. It would be much easier to add it in backburner directly.
I'm adding an issue here, to see if I created a pull request, if this is something that you could see being used else where and would be pulled into backburner to make available for ember?

This sounds like debounce with immediate. See:

https://github.com/ebryn/backburner.js/blob/30d4f505e935f7b588317f94a489dbdf44ec3a0b/tests/debounce_test.js#L84
https://github.com/ebryn/backburner.js/blob/30d4f505e935f7b588317f94a489dbdf44ec3a0b/lib/backburner.js#L298

Passing true for immediate runs the method right away, then debounces the 2nd to the nth calls.

It is close, but the difference is, if you set immediate to true, it only runs the method once and then debounces until the timer is cleared. I want two calls to the method, once immediately, and once when the timer clears if the method was called more than the initial immediate time.
Using debounce with immediate, if the method was called 5 times before the timer cleared, the method would be called immediately and then not again until the timer cleared.
Using the proposed coalesce, if the method was called 5 times before the timer cleared, the method would be called immediately, and then once again when the timer cleared. Coalescing all the calls after the first into one final call at the end.

@aaronbhansen you get this coalescing behavior using requestAnimationFrame instead of setTimeout: see https://github.com/ebryn/backburner.js/pull/163


not reaching the interface for python

Issue Type: Feature Request
not able to reach the sourse file after download in case of python extention instaling without interface verifing
Extension version: 2020.9.114305
VS Code version: Code 1.50.0 (93c2f0fbf16c5a4b10e4d5f89737d9c2c25488a3, 2020-10-07T06:10:52.432Z)
OS version: Windows_NT x64 10.0.17763

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please edit your opening comment by providing a filled-in copy of our issue template. Once you are done please leave a comment to that effect.


Install script is not always run by pip?

Seen on @jpmjanelle's machine.

We should confirm that this is still an issue with the new build script on @jpmjanelle's machine.

should i try to reinstall gpkit with the pip instructions?

There's a couple other changes that'll need to be made before reinstalling,
so I can do it next group meeting.

Status of this?

Could not reproduce, can probably be closed.
We added a "blank settings file" warning to import during Jenkins setup, so if this occurs again we'll get a useful error message.


[Feature] Display deployed verision

When leveraging the gui, it would be nice to display in the gui some sort of reference to the deployed version number.
Bonus functionality would be to quickly lookup the current version vs the latest github version and show some sort of indicator of whether the deployment is the latest version available.

This was implemented earlier today in 48d607fb966b33876f8f7f31cf0a7d607e40dca6. It's now included in v1.0.6.

Great! I have been providing feedback based on your hosted demo.  I'll deploy the latest version tonight to test all your new updates.


Missing role attributes in Core 5.6.1

Describe the bug
ARIA roles and attributes are missing in Clarity Core v5.6.1, failing our React Testing Library tests that use await screen.findByRole('button').
How to reproduce
Steps to reproduce the behavior:

I could not make this work with Stackblitz or Code Sandbox, so had to go the repro repo route: https://github.com/astorije/clarity-missing-role
Clone, install dependencies (yarn install)
Run yarn test on the main branch and see the error in the console output
Switch to the core-5.6.0 branch, install dependencies again (yarn install), and run yarn test to see the same tests pass

Here is the error output on v5.6.1:
 FAIL  src/App.test.tsx
  App
    ✕ foo? (1042 ms)

  ● App › foo?

    Unable to find role="button"

    Ignored nodes: comments,


Provide sample environment variables file

I think it would be a good idea if you provided a sample environment variables file. Like .env.example @ochukochris

Yes, true I just added it thank you for pointing that out

Yes, true I just added it thank you for pointing that out

Alrighty!!