Properly enable ansible-lint

This PR groups all of the needed pieces to enable proper ansible-lint.
Until now, it wasn't checking anything. This pull-request ensures we now
run the check properly in the pre-commit run.
Since it didn't run, we have to correct most of the existing code...
This PR is therefore grouping "some" patches in order to make things
easier to revert in case of need.
Here's the list:

Correct ansible-lint configuration and run
Correct "ci" directory content for ansible-lint
Update ci_framework/tests directory for ansible-lint
Correct artifacts role for ansible-lint
Correct ci_framework/hooks directory for ansible-lint
Correct ci_framework/playbooks directory for ansible-lint
Correct build_containers role for ansible-lint
Correct build_openstack_packages for ansible-lint
Correct ci_setup role for ansible-lint
Correct cifmw_block_device role for ansible-lint
Correct cifmw_ceph_client role for ansible-lint
Correct cifmw_ceph_spec role for ansible-lint
Correct cifmw_cephadm role for ansible-lint
Correct cifmw_create_admin role for ansible-lint
Correct copy_container role for ansible-lint
Correct dlrn_report role for ansible-lint
Correct edpm_build_images role for ansible-lint
Correct edpm_deploy role for ansible-lint
Correct edpm_deploy_baremetal role for ansible-lint
Correct edpm_kustomize role for ansible-lint
Correct edpm_prepare role for ansible-lint
Correct hive role for ansible-lint
Correct install_ca role for ansible-lint
Correct install_yamls role for ansible-lint
Correct libvirt_manager role for ansible-lint
Correct local_env_vm role for ansible-lint
Correct openshift_login role for ansible-lint
Correct openshift_provisioner_node role for ansible-lint
Correct openshift_setup role for ansible-lint
Correct operator_build role for ansible-lint
Correct operator_deploy role for ansible-lint
Correct os_must_gather role for ansible-lint
Correct os_net_setup role for ansible-lint
Correct pkg_build role for ansible-lint
Correct polarion role for ansible-lint
Correct registry_deploy role for ansible-lint
Correct repo_setup role for ansible-lint
Correct reproducer role for ansible-lint
Correct rhol_crc role for ansible-lint
Correct run_hook role for ansible-lint
Correct set_openstack_containers role for ansible-lint
Correct tempest role for ansible-lint
Correct test_deps role for ansible-lint
Correct .github directory for ansible-lint
Correct couple of config file for ansible-lint
Correct YAML format
Enable "production" policy for ansible-lint

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/test list

/test pre-commit

/test pre-commit

/test pre-commit

/test pre-commit

recheck

/test pre-commit

/test pre-commit

recheck

Thank you @pablintino @cjeanner for the great work.

/approve

/unhold


Operation Add does return 404.

While trying to use the Add-operation to create a comment I always receive a 404.
My call looks like this:
https://dev.azure.com/{myorg}/{my project as string/guid}/_apis/wit/workItems/{Id of the workitem i.e. '182'}/comments?api-version=5.1-preview.1
The body (application/json):
{
"text":"SomeComment"
}
Is there anything I am missing?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 65b095b5-b487-1e94-bd37-1cec2845cc88
Version Independent ID: 90068655-f5dd-f792-be73-f0ed6b31ad7e
Content: Comments (Azure DevOps Work Item Tracking Comments)
Content Source: docs-ref-autogen/5.1/witcomments/Comments.yml
Product: devops
GitHub Login: @wnjenkin
Microsoft Alias: whjenkin

@wnjenkin is there any news on this?
The example in docs references version 5.1-preview.3, but it still returns 404

Hi @herrmeisenmann , at the time when you posted the issue (Feb 21) it was possible that the feature was not yet enabled on your scale unit. Also, please note (as @alexdesignworks mentioned) you need to use "5.1-preview.3" version. If you still see 404, could you please create a ticket on https://developercommunity.visualstudio.com/ and attach full request/response so we can investigate further.


Adding indent/outdent hotkeys for multiple lines

Greetings
I want to ask about adding one text-editing feature which I'm missing a lot when switching to DBeaver from VSCode
There are these optional key preferences in VSCode:

It can add/remove tabulation to/from the beginning of multiple lines at once - that's pretty handy for formatting nested SQL queries

I can perform it by selecting several lines and pressing Tab or Shift+Tab this way
https://github.com/dbeaver/dbeaver/assets/28875055/e05623ba-5a2e-4847-91a2-a86af18425f2
Am I correctly understood what you mean? Does it satisfy your needs?

Also it may be a duplicate of #7280.

However, it's been a while since no update on this issue. If the issue is still actual please let me know and provide additional information regarding the requested behavior.


Refresh/Repaint Layer

First thank you! (skell 2 baseline sample)
I have some issue when a do a ajax post and update an element in
,
the element doesn't refresh, only on F5.
When debug i can see, the element has sucessfully updated in code, but in screen doesn't.
I try several approaches like (selector is parent div of and button is inside skel-layers-fixed):
$("#mydiv").hide()
$('#mydiv').get(0).offsetHeight;
$("#mydiv").show(0);
$("#mydiv").height($("#selector").height() - 1);
$("#mydiv").height($("#selector").height() + 1);
$("#mydiv #mybutton").fadeTo(100, 0.1).fadeTo(200, 1.0);

$("#mydiv #mybutton a").css('background-color', '').css('background-color', 'red');
var z = $("#mydiv #mybutton a");
var y = $("#mydiv #mybutton a").parent();
$("#mydiv #mybutton a").remove();
y.append(z);

None of this approaches worked.
PS.: when i resize page to a small width, to "mobile version", work but when resize again the element still incorrect.
(Sorry for my English, i speak Portuguese)

I don know cause of repaint/refresh doesn't work on header, but now i replace only text on html node and work. Thank any way!


HDDS-6064. ozone debug ldb should print proper JSON with --with-keys=true

What changes were proposed in this pull request?
Use --with-keys=true to print the proper JSON.
What is the link to the Apache JIRA
https://issues.apache.org/jira/projects/HDDS/issues/HDDS-6064
How was this patch tested?
docker-compose test

@errose28 Please take a look.

Hi @adoroszlai thanks for picking this back up. I think this patch needs some more work. Changing -> to : is still not valid json. Piping the output to jq gives parse error: Expected string key before ':' at line 1, column 8. I think we should wrap the whole withKeys output block in a dictionary {} with commas between entries.
Some additional fixes are still outstanding as well:

Modify TestOmDBCli to send all stdout to a different stream, and read this into gson to test that it is valid json.
Send all DBScanner scanner output that isn't json (like the 'added definition ...' output) to stderr
Flag change mentioned here

This may be a problem with the new compatibility guarantees we have discussed, but IMO debug commands are intended for developers to use on the fly, not for end users to put in scripts, so they should allow looser compatibility requirements.

@errose28 Can you please take a look at the updated patch?

With the container V3 support in ldb tool and significant test refactoring I deem this not worth it to do a merge rebase. I will open a new PR with some diffs here later.


Linting Errors

There are a few linting errors that occur when using Awesome Lint. These are "very minor" linting errors but I would like to eventually solve these issues besides just removing the icons.

Emoji's cannot be included in the table of contents because this is inconsistent with the section headers. I was unable to add emoji's because it broke the markdown link. I've decided to remove emoji's altogether and figure out a different method to include images for each section.
The YouTube icons are causing an "Invalid list item link" error in the linter when placed in the link name, like this: - [YouTube Video ![YouTube](https://github.com/bytecauldron/awesome-gamemaker/raw/main/icons/youtube.png)](https://www.youtube.com/watch?v=linkGoesHere) - Channel Name.
Placement of the 💸 emoji (indicating it's a paid asset) is also causing lint errors. I'll need to find a new spot to place this so the linter doesn't get angry.

Ok, all linting errors are now fixed.
I'll close this issue once I add a way to show YouTube links. I could just tag the YouTube icon at the end of the description. That appears to work without causing the linting errors.

YouTube icons were added back in. The linter won't complain if I put the icons at the end of each YouTube link. It's not as readable as I would like but for now, I think it works just fine. Eventually I'll have a better legend system to distinguish each link, whether it's an open source library or another type of resource.


Consistently use anyhow in ra_project_model

ra_project_model crate does a lot of interraction with other processes (cargo and rustc). It is bound to fail in interesting ways, so we need to make sure that the error reporting is top-notch.
We should switch it to the anyhow crate and use .context method proacticaly to make sure that all errors are reported with context.

cc https://github.com/rust-analyzer/rust-analyzer/issues/3032

The crate in question is here:
https://github.com/rust-analyzer/rust-analyzer/blob/29f5e7eebf606c1929d5a77ad66624cd4f3fcf49/crates/ra_project_model/src/lib.rs

I'm willing to take a stab at this.

Did #3119 close this @matklad  ?


docs: explain the roadmap

Closes #176

@comius what's next after "ready to pull" label? Is the process documented anywhere I can read?

@comius what's next after "ready to pull" label? Is the process documented anywhere I can read?

It's waiting internal review from protobuf team.

How about if we don't ask them for review? They've had five days now, and I was planning to send more PRs for this repo once I confirmed that the process is working.


ControlNetInpaintPipeline: Errors for loading Controlnet Diffusers folder

Thanks for your work, After I converted the Controlnet.pth to diffusers, an error still occurred while loading. Did my conversion fail?How can I solve this problem？The error log is as follows：
The config attributes {'controlnet_hint_channels': 3} were passed to UNet2DConditionModel, but are not expected and will be ignored. Please verify your config.json configuration file.
Traceback (most recent call last):
File "/home/sd/Harddisk/zxp/ControlNet-for-Diffusers/Test.py", line 8, in
pipe_control = StableDiffusionControlNetInpaintPipeline.from_pretrained("./control_sd15_seg",torch_dtype=torch.float16).to('cuda')
File "/home/sd/miniconda3/envs/Controlnet/lib/python3.9/site-packages/diffusers/pipelines/pipeline_utils.py", line 865, in from_pretrained
loaded_sub_model = load_method(os.path.join(cached_folder, name), **loading_kwargs)
File "/home/sd/miniconda3/envs/Controlnet/lib/python3.9/site-packages/diffusers/models/modeling_utils.py", line 567, in from_pretrained
raise ValueError(
ValueError: Cannot load <class 'diffusers.models.unet_2d_condition.UNet2DConditionModel'> from ./control_sd15_seg/controlnet because the following keys are missing:
up_blocks.1.resnets.2.conv_shortcut.weight, ... ....
up_blocks.2.resnets.2.norm2.bias, up_blocks.3.resnets.0.norm2.weight, up_blocks.2.resnets.0.conv1.bias, up_blocks.1.resnets.1.norm1.bias.
Please make sure to pass low_cpu_mem_usage=False and device_map=None if you want to randomely initialize those weights or else make sure your checkpoint file is correct.

Can you provide your full script? Which controlnet model do you convert? @little-misfit

python /home/sd/Harddisk/zxp/diffusers/scripts/convert_controlnet_to_diffusers.py --checkpoint_path control_sd15_seg.pth  --dump_path control_SD15_seg --device cpu
thanks

Can you provide your full script? Which controlnet model do you convert? @little-misfit

I'm trying to convert a “seg_controlnet", which is fllow your //ControlNet + Inpainting//

What is your diffusers version? Please install diffusers using the provided commit id instead of the official repo. I know that diffusers has officially supported this op, but there may have some slight difference. Once you successfully convert it, then you can load it with the recent diffusers version. @little-misfit


PHP >= 7.0

composer.json:
require": {
+        "php": ">=7.0"

fixed by: https://github.com/italia/spid-php-lib/commit/72c9d1a8b01c142ebb3d23a9a5b50ecf7663f5e5


Add custom game icon / title

Now with cleaner code and less(shorter) ASM™

I can change everything tomorrow

I now changed everything except the one wiht the FMLDeobfuscatingRemapper

Is there anything I still need to change? :thinking:

Now removed the bluescreen classes because of licensing issues with it

Have you resolved the issue Jamies brought up earlier here

Due to mojangs wishes/rules of not changing the title or the icon of the game and how users would perceive a bluescreen we won't be merging this PR (As was decided before a lot of the older staff retired also but now backed by myself) Sorry about the long wait and wasted time.


Add bsky link to socials

We also need to tweak DNS settings on ratatui.rs to get the @ratatui.rs handle. I don't remember if I have access to regery though...

@joshka can you help me setting that up maybe? We just need to add some DNS records


Cannot change src directory

Hi there!
I've got an error no such file or directory, stat '/var/www/html/src'. It's true - I have not this directory, but my source files are located in another directory (named resources), so I want to change path to it via config.
I guess, it hard-coded here, so maybe add configuration option like srcDir (will be src by default) or anything like this?

@czernika
First of all, thank you very much for your issues.
PR Welcome,
The source code is here .
If you don't have PR, I will update it in Oct.
Thx.

upgrade v0.0.4 @czernika


Unbalanced Samples

推荐使用英语模板 General question，以便你的问题帮助更多人。
首先确认以下内容

我已经查询了相关的 issue，但没有找到需要的帮助。
我已经阅读了相关文档，但仍不知道如何解决。

描述你遇到的问题
[填写这里]
相关信息

pip list | grep "mmcv\|mmcls\|^torch" 命令的输出
[填写这里]
如果你修改了，或者使用了新的配置文件，请在这里写明

[填写这里]

如果你是在训练过程中遇到的问题，请填写完整的训练日志和报错信息
[填写这里]
如果你对 mmcls 文件夹下的代码做了其他相关的修改，请在这里写明
[填写这里]
In my dataset, there is a very large number of categories. Therefore, I divided the data of this category into several folders. I need to change the 'train.txt' to use different folders during training in different epoch. What should I do to reload the data during training by epoch?

In my dataset, there is a very large number of categories. Therefore, I divided the data of this category into several folders. I need to change the 'train.txt' to use different folders during training in different epoch. What should I do to reload the data during training by epoch?

In your description, the image_list in the dataset has been changed during the training, the data_load should be rebuilt.
Can you explain why and how you divided train.txt？

Yes,the dataloader need to rebuilt. Because the dataset has more than ten categories, the number of one category is dozens of times that of other categories. I hope the cnn could use data of different categories to in every batch. However, I don't have enough gpu to use many samples in one batch. At the same, I hope the cnn could be trained by as many samples as possible in the category with the largest number.
Therefore, I choose a compromise by dividing the data of category with the largest number to several folders. I hope the cnn could be trained by the different data with the largest number of category and same data in other categories during training in different epoch. At present, I make different training files with writing different image path with the largest number of category and same data in other categories.

In my dataset, there is a very large number of categories. Therefore, I divided the data of this category into several folders. I need to change the 'train.txt' to use different folders during training in different epoch. What should I do to reload the data during training by epoch?

In your description, the image_list in the dataset has been changed during the training, the data_load should be rebuilt. Can you explain why and how you divided train.txt？

Yes,the dataloader need to rebuilt. Because the dataset has more than ten categories, the number of one category is dozens of times that of other categories. I hope the cnn could be trained by the data of different categories to in every batch. However, I don't have enough gpu to use many samples in one batch. At the same, I hope the cnn could be trained by as many samples as possible in the category with the largest number.
Therefore, I choose a compromise by dividing the data of category with the largest number to several folders. I hope the cnn could be trained by the different data with the largest number of category and the same data in other categories during training in different epoch. At present, I make different training files with writing different data path with the largest number of category and the same data path in other categories.

You can try ClassBalancedDataset.
If you really want to do what you describe, you may need to implement a hook, refer to the doc

This issue will be closed as it is inactive, feel free to re-open it if necessary.


[npm] Package.json dependencies get filled in with old versions

Issue Type: Bug
When filling in a dependency in a package.json file, the version used can be very, very out of date. Hovers still use the latest version though.
The fix for #75274 (commit 9bc4699) replaced the old API used for this with one from npms.io, but that service apparently has indexing issues.
Steps to reproduce:

In a package.json file, add a dependency for '@typescript-eslint/eslint-plugin' and press tab.

Expected result:
"@typescript-eslint/eslint-plugin": "2.3.0"
Actual result:
"@typescript-eslint/eslint-plugin": "1.13.0" (published on: July 21)
Notes:
GET: https://registry.npmjs.com/-/v1/search?text=@typescript-eslint%2Feslint-plugin&size=1
GET: https://registry.npmjs.com/-/package/@typescript-eslint%2Feslint-plugin/dist-tags
This seems to be pretty common unfortunately.

package
actual
expected

@types/node
10.0.3
12.7.5

eslint
6.1.0
6.4.0

typescript
3.4.3
3.6.3

VS Code version: Code - Insiders 1.39.0-insider (8b6e0efc0a98639085fabeb5226a516785424796, 2019-09-19T03:15:21.720Z)
OS version: Windows_NT x64 10.0.18362

Yes, the intellisense results/hover should have the most recent versions since they make a separate request to the NPM registry via npm view --json <package>. It's the one that gets autocompleted when you start typing a fresh new dependency that seems to use npms.io data.
So basically: <enter> " @ typescript <tab> and let it fill in the rest of the name + version string.
Here is my result when visiting that url:

[
    {
        "package": {
            "name": "typescript",
            "scope": "unscoped",
            "version": "3.4.3",
            "description": "TypeScript is a language for application scale JavaScript development",
            "keywords": [
                "TypeScript",
                "Microsoft",
                "compiler",
                "language",
                "javascript"
            ],
            "date": "2019-04-09T23:21:12.530Z",
            "links": {
                "npm": "https://www.npmjs.com/package/typescript",
                "homepage": "https://www.typescriptlang.org/",
                "repository": "https://github.com/Microsoft/TypeScript",
                "bugs": "https://github.com/Microsoft/TypeScript/issues"
            },
            "author": {
                "name": "Microsoft Corp."
            },
            "publisher": {
                "username": "typescript",
                "email"<EMAIL_ADDRESS>            },
            "maintainers": [
                {
                    "username": "typescript",
                    "email"<EMAIL_ADDRESS>                }
            ]
        },
        "score": {
            "final": 0.44991672429970486,
            "detail": {
                "quality": 0.29698267740127254,
                "popularity": 0.6975864411790185,
                "maintenance": 0.3333333333333333
            }
        },
        "searchScore": 1010.8733,
        "highlight": "typescript"
    },
    {
        "package": {
            "name": "@typescript-eslint/typescript-estree",
            "scope": "typescript-eslint",
            "version": "1.13.0",
            "description": "A parser that converts TypeScript source code into an ESTree compatible form",
            "keywords": [
                "ast",
                "estree",
                "ecmascript",
                "javascript",
                "typescript",
                "parser",
                "syntax"
            ],
            "date": "2019-07-21T17:03:27.502Z",
            "links": {
                "npm": "https://www.npmjs.com/package/%40typescript-eslint%2Ftypescript-estree",
                "homepage": "https://github.com/typescript-eslint/typescript-eslint#readme",
                "repository": "https://github.com/typescript-eslint/typescript-eslint",
                "bugs": "https://github.com/typescript-eslint/typescript-eslint/issues"
            },
            "publisher": {
                "username": "jameshenry",
                "email"<EMAIL_ADDRESS>            },
            "maintainers": [
                {
                    "username": "jameshenry",
                    "email"<EMAIL_ADDRESS>                }
            ]
        },
        "score": {
            "final": 0.7935211582302197,
            "detail": {
                "quality": 0.835530206802869,
                "popularity": 0.5512435346582717,
                "maintenance": 0.9997910258827543
            }
        },
        "searchScore": 18.459545,
        "highlight": "@typescript-eslint/typescript-estree"
    }
]

...and when visiting the site:

...and after a bit of digging: https://github.com/npms-io/npms-analyzer/issues/244#issuecomment-528980226

Wow, this morning I no longer get the versions I got yesterday. Now I get 3.4.3 again.

Sorry, my bad. For code completion on the version (after :), we use also use npm view
For code completion of the module name, which also inserts a version, we use npms-io.
I suggest we wait for npms-io to fix their issue. If they can't, we have to look out for a better rest service. The challenge is to find one targeted for code complete.

Closing as 'upstream'. I will reopen if this isn't fixed by npms-io in the near future.


Fixed link in documentation (...) instead of ...()

 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.

Thanks @passionkind!


Handle FedEx tracking response failure code 9045

I'm not sure of the source of this failure. I accidentally left a 6 second interval polling task open in a terminal for most of the day to the FedEx test endpoint, so maybe FedEx cut me off. It's also possible their test service is not functioning correctly at the moment. At any rate, it's better to handle the code because the error message and class which were returned to me weren't helpful.
If we don't handle this error, we get:
ActiveShipping::Error: Tracking response does not contain status code

If we do handle this error, we get:
ActiveShipping::ResponseContentError: Sorry, we are unable to process your tracking request.  Please retry later, or contact Customer Service at 1.800.Go.FedEx(R)<PHONE_NUMBER>.

I dumped the response xml from a pry session to use as a test fixture and red/greened the fix. Here is the red output before it went green.
2) Failure:
FedExTest#test_response_failure_code_9045 [/Users/adacosta/projects/forks/active_shipping/test/unit/carriers/fedex_test.rb:334]:
[ActiveShipping::ResponseContentError] exception expected, not
Class: <ActiveShipping::Error>
Message: <"Tracking response does not contain status code">
---Backtrace---
/Users/adacosta/projects/forks/active_shipping/lib/active_shipping/carriers/fedex.rb:618:in `parse_tracking_response'
/Users/adacosta/projects/forks/active_shipping/lib/active_shipping/carriers/fedex.rb:161:in `find_tracking_info'
/Users/adacosta/projects/forks/active_shipping/test/unit/carriers/fedex_test.rb:335:in `block in test_response_failure_code_9045'
---------------

Thanks.
Test fails are the UPS remotes.


more boolean handling

On the back of #160, this patch fixes an issue where setting an option as boolean sets the value to false unless otherwise specified in minimist. I've factored the defaults from opts.js into defaults.json so they can be shared from opts.js and the bin script and also the aliases. Before the handleOptionsMethod was set to false even when --cors was turned on because of the boolean setting.

Released as v1.1.2.
Thanks! This refactor has been a long time comin'.


Rename HighlyNullDataCheck to HighlyNullColumnsDataCheck

With the addition of HighlyNullRowsDataCheck in #2222 we should rename HighlyNullDataCheck for clarity and issue a deprecation warning as well.

Let's wait to move on this until we resolve the conversation in #2222

We can close this because we took a different direction in #2222.


JS-Threads: AddrBook for ThreadStore

Closing this in favor of more specific issues when the time comes.


AutoTune - On shutdown provide ability to saving profiling without garbage collection

Hi Rob,
I used to do some sneakyness to write profiling data on shutdown without doing garbage collection (I do GC for unittests since they are often short-run, but GC is too expensive when I shut down my mighty web app that's been running for a month). This doesn't work any more:
SpiEbeanServer spi = (SpiEbeanServer) ....getEbeanServer();
DefaultAutoFetchManager dam = (DefaultAutoFetchManager) spi.getAutoFetchManager();
dam.serialize();

Can we have a way to collect and save on shutdown but skip garbage collection?

Currently on shutdown() if the profiling is on (and skipCollectionOnShutdown is false) then the profiling is saved but additionally System.gc() is called via collectProfiling().
There are 2 options that we can do here:

on shutdown() have a separate setting/flag to control if GC is executed
expose the method to write the profiling xml file

I presently have my app set up to call a method to write the file, so I'm good with that, but either way would be fine. I suppose my app would be a little cleaner if there was a separate flag for GC control.
Thanks.

Yes, I want to do both.
Currently I'm just thinking the existing method names and skipCollectionOnShutdown flag are not that clear (rely on javadoc a bit too much imo).

Changes:
Rename skipCollectionOnShutdown to  skipGarbageCollectionOnShutdown in AutoTuneConfig
## skip GC on shutdown
ebean.autoTune.skipGarbageCollectionOnShutdown=true

Add skipProfileReportingOnShutdown to AutoTuneConfig
## skip reporting profiling on shutdown
ebean.autoTune.skipProfileReportingOnShutdown=true

// Add reportProfiling() to Autotune API

Ebean.getDefaultServer().getAutoTune().reportProfiling();

Note that the shutdown behavior is now:
  @Override
  public void shutdown() {
    if (profiling) {
      if (!skipGarbageCollectionOnShutdown && !skipProfileReportingOnShutdown) {
        // trigger GC to update profiling information on recently executed queries
        collectProfiling(-1);
      }
      if (!skipProfileReportingOnShutdown) {
        saveProfilingOnShutdown(false);
      }
    }
  }

Note that triggering GC only makes sense to execute if reporting is performed.

Looks good, thanks!


scrollstart or scrollend event is not firing - need implementation

SDK 4.1.0 has scrollstart and scrolled event for listView but when I added this module, it's not firing.

Hey @mpociot,
I can confirm this, those events should be added to the module or they can be connected with ListView api automatic?

@mpociot I'm implementing the scrollstart and scrolled event for listView, any change to you review it?

@mpociot @viezel  please review and test,  probably need some adjusts.  (seems to work ok here)
<sdk-version>4.1.1.v20150816214206</sdk-version>

https://github.com/nuno/TiCollectionView/commit/55d36fddfbc9b316b88bf6a1b1920e6190509586

If I was you guys I'd go that extra mile and implement also the scroll event itself. This is something I really don't understand why it's missing on the ListView API, apart from forcing users to use markers…

Hey @yuchi I need this events for  markers exactly.
Try follow this algorithm:

App was loaded, with collectionView items via normal "xhr request",  "page?1" each page have 24 items . setItems

if we scroll-up we load "loadmore" data. Is our pagination via "xhr request", each page "page?1...2...3 " have 24 items.  appendItems

if I scroll down near to the top and items are between 0 to 4  firstVisibleItemIndex we load "loadprevious"  data. Is our pagination via "xhr request", each page  "page?3...2...1 " have 24 items.  insertItemsAt

We want use this module, we did the work first with Listviews since the collectionView module have not yet support this events.
If there is a better way to do this without those events, special the "scrollend"-event, I do like to hear. :)

Definitely there is a leak!  Im trying to see if I introduced this leak.
I probably  did not release/clean the event somewhere.

Hello everyone I found the module have a leak already in 1.4 version.
@yuchi about the scroll event itself is actually a good idea.
If the author @mpociot  have like 10 minutes to his time, to check my simple and humble implementation and tell me if Im going to the right direction, I appreciate.

Hey guys, I tried to implement it in #66, try it out! #69 should also be closed as a dupe of this one (cc @nuno).


[Keyboard Maestro - List Macros] ...

Extension
https://www.raycast.com/eluce2/list-keyboard-maestro-macros
Description
really appreciate this great extension. so helpful ... it will be great if keyboard maestro macros show up as "commands" in the main Raycast app like Apple Shortcuts so that we can trigger them directly from the main raycast entry point after entering cmd-space. This way we will not have to click into the extension to search and execute macros. Thanks.
Who will benefit from this feature?
all keyboard maestro users
Anything else?
No response

This is not possible with the Raycast API, as each command has to be hard-coded, but I may be able to add support for arguments so that you can invoke a search for your macros a bit more quickly

Thanks Eric! Search arguments will be awesome. Best wishes!
On Tue, Jun 6, 2023 at 8:07 AM Eric Luce @.***> wrote:

This is not possible with the Raycast API, as each command has to be
hard-coded, but I may be able to add support for arguments so that you can
invoke a search for your macros a bit more quickly
—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/issues/6952#issuecomment-1578944977,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIQFXBXVISKJ7LI7MI7YILXJ5BTZANCNFSM6AAAAAAY3QMWHM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Thanks,

Jasdeep

@raycastbot close this issue


GitHub Actions should also check devnet docker builds correctly

It is being done today. Closing the issue.


Switchover staging tests to use Ubuntu bots instead of Debian

Switch tasks running on staging bots to use ubuntu bots instead of debian.

Setup new properties section in ci.yaml for Ubuntu.
Start moving staging tests to use the new ubuntu section/bots for runs.

Add more ubuntu capacity to staging and prod pools, including highmem machines: go/cirgs/4697373

Engine switched over: https://github.com/flutter/engine/pull/33498

Framework builders switched over: https://github.com/flutter/flutter/pull/104294

All hosts moved to Ubuntu.


Error page should render actual error response text

Currently it does not:

It would be nice to at least have something in the console for debugging

same for me. debugging this now. i have correctly updated uid, and all twilio details on my local.json per readme...

Ah! I think I know why -- I'm hitting this on /post, except my patch returns an error response from that path instead of redirecting to an error page. heh

just updating. i figured my problem as well  (which was unrelated to @k88hudson issue) , in case you want to close this out.
Looks like mine was a problem in updating the uid correctly and hence errored in profile.getByUID(op, function (err, user) { ..}


Formatting error in the 'Media graph concept' page

There is a formatting error in the first line of the 'RTSP source' section that is causing a link to be displayed incorrectly.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9ebfa099-3ed5-ac9a-e8c2-c7d2c5d1cb65
Version Independent ID: fe310eb0-b10a-99b5-cad0-c92c13d532b2
Content: Media graph concept - Azure - Live Video Analytics on IoT Edge
Content Source: articles/media-services/live-video-analytics-edge/media-graph-concept.md
Service: media-services
Sub-service: lva-edge
GitHub Login: @Juliako
Microsoft Alias: juliako

@conor-joplin Thanks for the feedback! We are taking a look into this and will get back to you soon.

@conor-joplin Thanks again the feedback. This has been resolved now. We will now close this out but if you feel you need more information please just let us know.


RunTimeException not caught for second call to resolve() in handleMethodCallExpression()

In the current master version of CPG, catching UnsolvedSymbolException has been replaced with catching a RunTimeException because the newer JavaParser can now throw it and we want to catch it gracefully.
I think the fix implemented in commit https://github.com/Fraunhofer-AISEC/cpg/commit/fe7c17740246c6a5003140b8315331b94ab6cd69 also has to be applied to the catch statement here https://github.com/Fraunhofer-AISEC/cpg/blob/master/src/main/java/de/fraunhofer/aisec/cpg/frontends/java/ExpressionHandler.java#L697
This resolved the problem for me when running CPG on a large set of Java projects.

Ah, good point, I fear we need to go through all resolve() calls and apply it there. Meh, I still don't know why they would bury a runtime exception deep down in the resolve instead of properly throwing an UnsolvedSymbolException

Ok I am wondering why, we are calling resolve() in this case anyway, because the result is not used anyway.


[gateio] [gateio-streaming] Add gate.io api v4, gate.io streaming support

Added gate.io api v4
Extended gate.io streaming

ping @timmolter @walec51

@douggie @timmolter any chance to review and merge?
I'm using this code for sophisticated high frequency trading 24/7 on gate.io for several months now. Any bugs discovered were fixed right away and merged in the branch, so it is tested and working.
Would be good to share it with the community


feat: teach LayoutBatchStream to filter by indices

I have some thoughts on how to make the stream.rs logic easier to follow; however, this, I think, is the fewest changes to achieve indices filtering.
This change evaluates the filter on any row in a chunk containing at least one kept index. I have some thoughts on how to avoid evaluating the filter at a sub-chunk level but they require more substantial changes.

All comments addressed.

@danking with this pr could you switch https://github.com/spiraldb/vortex/blob/develop/bench-vortex/benches/random_access.rs to use it? This would be a real benchmark now instead of wishful thinking

Yes

Following up with bench change.


HIVE-27152 : Revert "HIVE-21388: Constant UDF is not pushed to JDBCStorage Handler…

JIRA : https://issues.apache.org/jira/browse/HIVE-27152
current_date_timestamp.q - This change was committed in HIVE-21388 without validation.

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

merged. Thanks @amanraj2520


Don't configure the sugar reporter for xdist slaves

This fixes #64 .

Thanks, I'll take a look later.

@Frozenball We probably need have the build install different plugins to ensure nothing breaks.

Now I'm getting a different error:
INTERNALERROR>     terminal_reporter.tests_count += len(items)
INTERNALERROR> AttributeError: 'NoneType' object has no attribute 'tests_count'
scheduling tests via LoadScheduling
INTERNALERROR> Traceback (most recent call last):
INTERNALERROR>   File "/home/omer/.virtualenvs/zBackend/lib/python2.7/site-packages/_pytest/main.py", line 84, in wrap_session
INTERNALERROR>     doit(config, session)
INTERNALERROR>   File "/home/omer/.virtualenvs/zBackend/lib/python2.7/site-packages/_pytest/main.py", line 122, in _main
INTERNALERROR>     config.hook.pytest_runtestloop(session=session)
INTERNALERROR>   File "/home/omer/.virtualenvs/zBackend/lib/python2.7/site-packages/_pytest/core.py", line 521, in __call__
INTERNALERROR>     return self._docall(self.methods, kwargs)
INTERNALERROR>   File "/home/omer/.virtualenvs/zBackend/lib/python2.7/site-packages/_pytest/core.py", line 528, in _docall
INTERNALERROR>     firstresult=self.firstresult).execute()
INTERNALERROR>   File "/home/omer/.virtualenvs/zBackend/lib/python2.7/site-packages/_pytest/core.py", line 394, in execute
INTERNALERROR>     res = method(*args)
INTERNALERROR>   File "/home/omer/.virtualenvs/zBackend/lib/python2.7/site-packages/xdist/dsession.py", line 514, in pytest_runtestloop
INTERNALERROR>     self.loop_once()
INTERNALERROR>   File "/home/omer/.virtualenvs/zBackend/lib/python2.7/site-packages/xdist/dsession.py", line 532, in loop_once
INTERNALERROR>     call(**kwargs)
INTERNALERROR>   File "/home/omer/.virtualenvs/zBackend/lib/python2.7/site-packages/xdist/dsession.py", line 570, in slave_slavefinished
INTERNALERROR>     assert not crashitem, (crashitem, node)
INTERNALERROR> AssertionError: ('devices/malware/tests/unit/test_features_serializer.py::describe_features_serializer::describe_serialization::it_should_not_return_an_object_id_attribute_in_the_serialization_result', <SlaveController gw3>)

@Frozenball Any idea?

@Frozenball Ok so now pytest-sugar doesn't crash when xdist is used but it still doesn't add the new terminal reporter.

:+1:

@codingjoe Any idea how to make the progress bar work?
I can't seem to get the number of tests that are going to be run with xdist installed.

I'd want this.

@aadis help in getting the progress bar to work will be much appreciated.

@Frozenball can you help me resolve this?

the progress bard needs help from the xdist side, get at me on freenode

@RonnyPfannschmidt Got it, will do.

Was wondering if there was any update on this feature - would love to have it

I just need to commit the changes I made and this is ready for merge.

I have no idea why the CI isn't running. Hold on.

I rewrote the build process to test with supported and unsupported versions of xdist and added 3.5 to the matrix among other things.
The tests are now run in CI using tox like they should.

Seems like pytest 2.6 doesn't support python 3.5. I removed that combination from the build matrix.

Everything seems to be in order.
@Frozenball Please merge.

Any chance to get this merged?

Hola. Apologies for the inactivity. I'll try to look at this tonight. I think xdist support would be great. :)

Thanks. It's done, working and passing tests.

My apologies for being so inactive with this project. I can publish a new version on PyPi after @or code reviews

Cheers, @thedrow. I tried it and it worked very well. Also well done on the extensive travis reconfiguration.
I merged it, @Frozenball, but you better look over the travis stuff sometime.

Thanks guys. Please release this as soon as possible.

Awesome !
Thanks :)
On Fri, Mar 18, 2016 at 1:46 PM, Omer Katz<EMAIL_ADDRESS>wrote:

Thanks guys. Please release this as soon as possible.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/Frozenball/pytest-sugar/pull/65#issuecomment-198247837

0.6.0 released


Use 'shelf_cors' package instead of custom CORS middleware.

Replaces the custom CORS middleware. Please consider for inclusion.

Added missing comment back, sorry.

lgtm - thanks!


Prevent Analytics Triggering

I have been trying out the code and it works great! I am currently using it in paired with a web crawler for sites who volunteer the crawl. I'd like to be able to render a web page without freaking out the users analytics, but I am struggling with ways to do that.
I found a Stackoverflow question similar to the issue, but I am not seeing how I can implement it within this container. Here is the link: http://stackoverflow.com/questions/22274112/is-there-a-way-to-disable-google-analytics-tracking-in-phantomjs
Has anyone discovered a way to do this? Or is it just not possible right now?

Hello @Failcookie
Following the example that you provided on StackOverflow you can easily add the following code to the template file https://github.com/microweber/screen/blob/master/templates/screen-capture.php#L16
page.onResourceRequested = function(requestData, request) {
    if ((/http:\/\/.+?\.css/gi).test(requestData['url']) || requestData.headers['Content-Type'] == 'text/css') {
        console.log('The url of the request is matching. Aborting: ' + requestData['url']);
        request.abort();
    }
};

Source: https://github.com/ariya/phantomjs/blob/master/examples/loadurlwithoutcss.js#L12
If you manage to tackle this issue feel free to submit a PR.


provide a decent main

mainly because the cookiecutter implementation kinda does nothing and is killing our test-coverage :(

[x] should address the suggestion made in #1 to have ability to inject a urn:base
[ ] should fall back on the «service» api provided by SyncFsTriples (to be hooked up on sembench)
[ ] have good cli tests for coverage

moving this to later, think we can live without atm

for the tests - best to follow the pattern that travharv started using
https://github.com/vliz-be-opsci/py-trav-harv/blob/d0193255947d10fcc8407996d4dd0f5f665da4d8/travharv/__main__.py#L238
https://github.com/vliz-be-opsci/py-trav-harv/blob/d0193255947d10fcc8407996d4dd0f5f665da4d8/travharv/__main__.py#L221
https://github.com/vliz-be-opsci/py-trav-harv/blob/d0193255947d10fcc8407996d4dd0f5f665da4d8/tests/test_main.py#L28

fix included in PR #13

closed by merging #13


Make text search the default search type and change its name from "CRE name" to "Topic text"

Fixed, see pull request: https://github.com/OWASP/common-requirement-enumeration/pull/91


Alerts absent from Children's Health Defense pages; other possible issues

Symptom A
Whilst https://mediabiasfactcheck.com/childrens-health-defense/ warns of quackery and Conspiracy-Pseudoscience:

the yellow CP icon is absent from pages such as https://childrenshealthdefense.org/news/plague-of-corruption-book-the-talented-dr-judy-mikovits/

Please see https://github.com/RatNeurons/fact-layer/issues/2

Symptom B
Whilst https://mediabiasfactcheck.com/daily-beast/ reports MOSTLY FACTUAL:

the extension reports Factual reporting: HIGH at https://www.thedailybeast.com/stella-immanuel-trumps-new-covid-doctor-believes-in-alien-dna-demon-sperm-and-hydroxychloroquine

Lazy guess: a fix for Symptom A will also fix Symptom B …

My icon extension has been superseded by this one.


COR-414 fix authnz-bouncer path prefix

The authnz-bouncer is ran as a ext_authz plugin on envoy. It's a catch-all requests, so we need to listen to all requests.
The current path prefix was only listening to /apis/authorization.nrc.no/v1...

/cc internetti
/cc neb42

/lgtm


Add missing Gitbook markdown parsing feature

Questions
Answers

Description?
This PR adds support for parsing Gitbook's specific markdown syntax

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fix https://github.com/PrestaShop/PrestaShop/issues/29287 and related to https://github.com/PrestaShop/PrestaShop/issues/29216

How to test?
See below

Possible impacts?

How to test

In the file admin-dev/themes/default/js/help.js, line 28 change https://help.prestashop-project.org with https://integration-help.prestashop-project.org:

- const psDocsDomain = 'https://help.prestashop-project.org';
+ const psDocsDomain = 'https://integration-help.prestashop-project.org';

In the file classes/controller/AdminController.php, line 1669 change https://help.prestashop-project.org with https://integration-help.prestashop-project.org:

- $this->context->smarty->assign('help_link', 'https://help.prestashop-project.org/' . Language::getIsoById($this->context->employee->id_lang) . '/doc/'
+ $this->context->smarty->assign('help_link', 'https://integration-help.prestashop-project.org/' . Language::getIsoById($this->context->employee->id_lang) . '/doc/'

In the file src/PrestaShopBundle/Resources/config/services/core/help.yml, line 9 change https://help.prestashop-project.org with https://integration-help.prestashop-project.org:

- - "https://help.prestashop-project.org/"
+ - "https://integration-help.prestashop-project.org/"

Go on the BO and check that the contextual help is well displayed both in legacy and migrated controllers

This, https://github.com/PrestaShop/PrestaShop/issues/29287, could be related.

@atomiix What is the next step? QA test?

@matks I guess!

@atomiix When you add "Waiting for QA", don't forget to modify the github issue https://github.com/PrestaShop/PrestaShop/issues/29216 status to "To be tested" too 😄

Hello :)
This PR can be tested by a dev : make sure you check most of the BO contextual help and it will be OK.
Thanks a lot for your help!

@khouloudbelguith everything should be fixed except the issue 2 that is not related to this PR but to the core!

Thanks @khouloudbelguith! It should be good for the legacy page errors.
For the migrated page, I think this is also happening on 1.7.8.x, it shouldn't be related to this PR

Thank you @khouloudbelguith and @atomiix


Feature Request: Dice Roll for Increment

Fantastic module. I've been using it track crafting for my players, I'm trying to implement a method for crafting where there's a DC check, followed by a dice roll (which varies on the difficulty of the item) for the item's progress. Could it be possible to add an additional optional field (of String type), that if used would be a dice roll (ex d100) / integer used to increment progress for all progression types?

Hey, thanks for getting in touch! This is a little bit of an odd request, and it's one I'll have to think about whether or not I want to implement.
I say it's odd not because it's an uncommon use case, but because I'm not entirely certain the slight increase in ease of use will outweigh the amount of hacking I have to do behind the scenes to get such a field to work. One of the things I try to do is make sure that every field in the configuration menu is absolutely, transparently clear as far as what it does and what impacts it will have. Something like this, where it's functionality could be changed depending on the type of activity, is inherently unclear.
I'm not sure there's a good way to solve that problem at the moment, so I'll have to think on it. This might get rolled into my thoughts about providing alternative ways of defining how much an activity gets increased by. So you could do things like, "if skill check > dc, increase by 1d10" rather than the current "if skill check > dc, increase by 1". Something like that is a ways down the line, though, if it comes at all.
If you need a fix in the meantime, you can accomplish this sort of thing using the Macro activity type fairly easily. I can whip something up to show you how to go about it, if you want. 😄

Here's how you'd set it up!

And here's the macro:
/*
  This is a sample macro for use with Crash's Tracking & Training.
  It rolls a skill check for a given actor against a DC.
  If the check is successful, it rolls another die and increases
  the value of the activity's progress by that amount.
*/

// ======= CONFIG ========

let actorName = `Test Doggo`; //The name of the actor who owns the activity to update.
let itemName = `Crafting Example`; // The name of the activity to update.
let checkDc = 15; // The DC of the check
let skillToRoll = `med`; // The abbreviation of the skill to roll.
let increaseBy = `1d10`; // The die roll to increase the progress by if the check succeeds.

// ==========================

// Get our actor and item
let item = CrashTNT.getActivity(actorName, itemName);
let actor = game.actors.getName(actorName);

// Make necessary rolls
let skillRoll = await actor.rollSkill(skillToRoll);
let total = skillRoll.total;

// If total is high enough, roll and increase progress
if(total >= checkDc){
    let increaseRoll = await new Roll(increaseBy).toMessage();
    let increase = increaseRoll._roll._total;
    let newProgress = item.progress + increase;
    CrashTNT.updateActivityProgress(actorName, itemName, newProgress);
}

Thanks @crash1115, I'll give it a try. If you'd like, I could try a PR to let you comment and test out the concept but I totally respect your design and architecture decisions. Thank you again for the macro!

Closing because the functionality is attainable using macro activities, and I've decided the work/payoff isn't worthwhile.


[Question] Persisting folds states and cursors positions

How could I persist my fold states and cursor position per file?
just to be clear on how I'm using this plugin, I'm doing nothing fancy or out of the ordinary, I'm creating and loading sessions and I have the "automatically save a session when you exit Neovim" bit from the README, nothing more.
As for the "per file" in my question, consider the following scenario:
Steps:
1 - :e aaa.lua
2 - move my cursor to line 20 and open a folded function there.
3 - :e bbb.lua
4 - move my cursor to line 10 and open a folded function there.
5 - save session require 'resession'.save('test')
6 - exit and reopen nvim
7 - load session require 'resession'.load('test')
Observations:

when opening bbb.lua cursor position is persisted but not the fold state at line 10
when opening aaa.lua neither the cursor position nor the fold state at line 20 are persisted

I hope this has made my question clearer. cheers!

This is a very clear request and it's a feature that I would love to support! At the moment, I don't think there are...really any decent APIs for the fold state in Neovim. I've had to interact with it a bit for my other plugin aerial.nvim, and I had to do a bunch of hacks to make it work anything close to correctly. For something like saving and restoring fold state...I think this is going to require some changes in Neovim core. There's already an open issue for it, we just have to wait for some brave soul to put up a PR https://github.com/neovim/neovim/issues/19226

That's good to hear! yes the fold API would be a very nice interface.
However I do remember :mksession being able to store folds along with their states, I don't remember how exactly tho (I could look into it if you'd like).
That was all about persisting folds, what do you think about persisting cursors positions?

:mksession encodes the folds with raw normal! zc commands. It would be a gross hack but I could technically do the same thing, except for the fact that I don't think there's an API to get the fold status.
I'm a bit confused about the question of persisting cursor positions, as that's something we should already be saving/restoring. Do you have a repro where that doesn't happen?

It does! I've moved the discussion of cursor position to a new issue, so we can separate it out. That's something I think I can address in the short-term, whereas the folding issue is probably not. We can leave this issue open for folds, and I'll come back to it once an API is available. Unfortunately, if there's no way to get the current state of the folds, there's no way to save/restore them :/


Error on startup with pre release 1.1.7

Following the install instruction I get the following error.
❯  conda activate cnapy-1.1.7
❯ cnapy
Traceback (most recent call last):
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/bin/cnapy", line 6, in <module>
    from cnapy.__main__ import main_cnapy
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/cnapy/__main__.py", line 31, in <module>
    from cnapy.application import Application
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/cnapy/application.py", line 42, in <module>
    from cnapy.gui_elements.main_window import MainWindow
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/cnapy/gui_elements/main_window.py", line 10, in <module>
    from cnapy.core import model_optimization_with_exceptions
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/cnapy/core.py", line 15, in <module>
    import efmtool_link.efmtool4cobra as efmtool4cobra
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/efmtool_link/efmtool4cobra.py", line 1, in <module>
    import efmtool_link.efmtool_intern as efmtool_intern
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/efmtool_link/efmtool_intern.py", line 9, in <module>
    jpype.startJVM()
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/jpype/_core.py", line 184, in startJVM
    jvmpath = getDefaultJVMPath()
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/jpype/_jvmfinder.py", line 74, in getDefaultJVMPath
    return finder.get_jvm_path()
  File "/home/sthiele/miniconda3/envs/cnapy-1.1.7/lib/python3.8/site-packages/jpype/_jvmfinder.py", line 212, in get_jvm_path
    raise JVMNotFoundException("No JVM shared library file ({0}) "
jpype._jvmfinder.JVMNotFoundException: No JVM shared library file (libjvm.so) found. Try setting up the JAVA_HOME environment variable properly.

Maybe it has something to do with openjdk which is now on version  openjdk=8.0.332=h166bdaf_0 for me?

What is the jpype1 version of your environment?

jpype1=1.4.1=py38h43d8883_1

Can you try an older version of jpype1? I still have got 1.2.1. Strangely though, the PR checks did not show this issue.

I downgraded to jpype1=1.2.1=py38h1fd1430_0 and have the same issue.

Strangely though, the PR checks did not show this issue.

Might be because I'm not on an ubuntu machine.

Can you try to replace the line 678 in reactions_list.py with
self.grp_test_model.add_reactions([reaction])

add_reaction is deprecated but should actually still work.

Oh yeah, some other things I noted.

The scenario name is in an unfortunate position I think I can click on it but nothing happens.
Idea: Maybe replace the icon for scenario open with the name of the scenario
The other toolbar buttons (save/as scenario) if they are disabled should be greyed out
How do I know that the scenario has been changed/unsaved? I think there should be some indication if the current scenario is not the same as the one that is on the label.


Loading text in the Options

There is a "Loading…" text in the options card. (It made me feel like I have to wait for something to appear or happen until I realized it’s just a text.)
Tested in: ovirt-web-ui-<IP_ADDRESS>.20180814git2ec0a1e-1.fc28.noarch.rpm, Mozilla Firefox for Fedora

And nothing happened?
it's not supposed to be a text,  there should be the key loaded async. Or made empty, I guess, if none is set

@michalskrivanek Nothing happened, it's just a text.

regression of #505


respect return code of external test command

https://github.com/leptos-rs/cargo-leptos/issues/90
This pr raises an error when the external test command returns anything other than 0.
That allows for e.g. a ci pipeline to fail when there have been tests failing.
I am happily awaiting a review !

Maybe we should have a test case?

I like your approach. Did run and fail on my ci :)

It looks like it passes on the ci here.

Hehe I meant something different I guess. I used the new code to run a playwright test that fails and that did make my (hello-world) pipeline fail, which it did not do before the change.

Ahhhh now I understand 🤣
Ok, looks good. Thanks for the contribution


Allow dry runs with custom regexes.

Previously, if custom regexes were specified and we were in "dry-run" (-y --do-it) mode, the code would exist
with a warning about checking the regexes carefully. But it is precisely the  "dry run" mode that is helpful in
checking those regexes!
This change retains the warning but allows the dry run to continue - after all it is not the thing the warning is
warning about. It is the "wet run" that is potentially dangerous, and the dry run is what helps mitigate that risk.

Previously, if custom regexes were specified and we were in "dry-run" (-y --do-it) mode, the code would exist

-y --do-it is the reverse of dry-run - --do-it actually does it :smile:

Without --do-it the tool will run in the dry-run mode and not actually delete anything: https://github.com/arcivanov/pypi-cleanup/blob/master/src/main/python/pypi_cleanup/init.py#L179

Previously, if custom regexes were specified and we were in "dry-run" (-y --do-it) mode, the code would exist

-y --do-it is the reverse of dry-run - --do-it actually does it 😄

Ah yes, that was a typo.

Gotcha, so that might just need better documentation. Right now it appears that -y confirms --do-it, but you're saying that -y confirms -r?

Correct. Without -r, -y has no effect and only --do-it takes you out of the dry-run mode.


Create workflow sdjones

.github

.github

package main
import (
"fmt"
"github.com/bet365/jingo"
)
// sample struct we'll encode
type MyPayload struct {
Name string json:"name"
Age int     json:"age"
ID int      // anything we don't annotate doesn't get emitted.
}
// Create an encoder, letting it know which type of struct we're going to be encoding.
// You only do this once per type!
var enc = jingo.NewStructEncoder(MyPayload{})
func main() {
// now lets encode something
p := MyPayload{
Name: "Mr Payload",
Age: 33,
}
// pull a buffer from the pool and pass it along with the struct to Marshal
buf := jingo.NewBufferFromPool()
enc.Marshal(&p, buf)

fmt.Println(buf.String()) // {"name":"Mr Payload","age":33}

// return the buffer to the pool now we're done
buf.ReturnToPool()

}
github.com/bet365/jingo


ucl_myriad profile

I'm having some problems using the ucl_myriad profile.
My error message is: Unknown configuration profile: 'ucl_myriad'

What was your full command? This works for nf-core pipelines, but for Eco-flow ones, you need to specify the config using -c

nextflow run main.nf -bg -resume -profile singularity --input input.csv -c myriad.config
Where myriad.config was downloaded from nf-core configs


Wallet

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: shivam-plutonic:x: AyurtYou have signed the CLA already but the status is still pending? Let us recheck it.


Can it be released on time on December 1, 2021?

I can see on zenhub that the time set for version 0.2.1 is December 1. Can it be released on time? I hope to use oidc in my project.

@littlefisher666 Yes, we're planning on releasing 0.2.1 on Dec 1.


[Event Request] table 83 "Item Journal Line" - OnAfterSetUnitOfMeasureCode

Can you add event OnAfterSetUnitOfMeasureCode in table 83 to be able to override "Unit of Measure Code" before it is validated?

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. We will update this issue with information about availability.


Add ScopeClaim support

ScopeClaim support

@skoruba oh sorry. ScopeClaim is removed in IDSRV 4. Thanks

I can found it in IS4 EF Core entity model. Can you please send more centext about it?
Thanks!


React Native compatibility hint

Hi!
I've just experimented with replacing Algolia with Meilisearch in my react-native application. I had a bit of a problem for a while which turned out to be that I needed a polyfill for URL on RN. I added https://www.npmjs.com/package/react-native-url-polyfill to my project which resolved the issue.
Leaving here more for the benefit of future react native developers than anything, however, it might be a good win for some documentation since that seems to be all that's needed to get Meilisearch in more react native apps as far as I can tell :)

For react native Algolia has this but I don’t know if you need a different library
https://www.algolia.com/doc/guides/building-search-ui/going-further/native/react/

Still instant search, but with the react native components library (part of instant search)

Did you use meilisearch-react or plain instant-meilisearch or Algolia instant search  ? Please guide me on the library that was used , if you could share some code will be great .

@emeagenciadigital for react native, I used @meilisearch/instant-meilisearch and react-instantsearch-native. react-instantsearch-native is maintained by Algolia but you can easily use it with meilisearch as well.
You can initialize the the client using  @meilisearch/instant-meilisearch and the hooks from react-instantsearch-native which is documented thoroughly here: https://www.algolia.com/doc/guides/building-search-ui/going-further/native/react-hooks/
Code to initialize the client:
import { instantMeiliSearch } from "@meilisearch/instant-meilisearch";

const searchClient = instantMeiliSearch(
  <YOUR_URL>
  <YOUR_API_KEY
);

Also asking for react native. I tried to do
import React from "react";
import "instantsearch.css/themes/algolia-min.css";

import {
  Hits,
  SortBy,
  SearchBox,
  Pagination,
  Highlight,
  ClearRefinements,
  RefinementList,
  Configure,
  Snippet
} from "react-instantsearch-dom";
import { instantMeiliSearch } from '@meilisearch/instant-meilisearch';
import { InstantSearch } from 'react-instantsearch-hooks';

const searchClient = instantMeiliSearch(
  "http://xxx:7700/",
  "xxxxx"
);

const Discover = () => (

    <InstantSearch indexName="test" searchClient={searchClient}>

        <SearchBox />
        <Hits hitComponent={Hit} />

    </InstantSearch>

);

const Hit = ({ hit }) => {

      <Highlight attribute="Entity" hit={hit} />

};

but this didn't work. createConnector.js:186 Uncaught TypeError: this.props.contextValue.store.getState is not a function
any sample codes please?

Hey instead of using react-instantsearch-hooks can you try with this library:react-instantsearch-native ?
See this doc

not critical and stale, closing


[Bug] New input widget is automatically bound to table data that was created via CRUD

Description
New Input widget added to the canvas is automatically binding to table data that was generate using CRUD
Steps to reproduce the behaviour:

Generate a page using CRUD operation
Drag and drop input widget on to the canvas and observe the input widget automatically binding itself to the table data but not appear in the outgoing entities of table data

Important Details

Version: Cloud
OS: Win10
Browser: Chrome
Environment: Release

@ramsaptami not high because it doesn't block a flow. It is odd that the relationship is there though.
@pranavkanade can you please check this?

Marked it high cause I'm not able to decouple the binding and this issue is now on prod. Might be an issue for users generating a page using our CRUD feature now.

https://user-images.githubusercontent.com/13262095/134308153-703905ae-1c41-490f-a77f-979342cd82ed.mov
@ramsaptami I checked it with our users sample datasource, did not observe the problem. Is it only related to g-sheets?

@pranavkanade Looks to be the case only with GSheets source

@ramsaptami @Nikhil-Nandagopal This is much bigger problem than what it appears on the surface. It can arise any time if name of widget 1 Input1 is substring of widget 2 colInput2 then we'll see this bug.


fix(egen): fixes the path in the colab enterprise link

The notebook `xai_image_classification_feature_attributions.ipynb`'s Colab enterprise link missed "notebooks/" in its path. This PR adds "notebooks/" to the path in the link.

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

/gcbrun


Questions about the Competition

Hi @yunhuiguo .
Thank you for hosting this competition, it's really interesting and challenging!
After carefully reading the rules on your website and in your paper, I still have some questions, please correct me if I misunderstood.

Is it allowed to use the paper submitted to your workshop as the paper required in this competition?
Is it allowed to modify those hyper-parameters for training? (e.g.  epoch, n_query)
Intuitively, training a module by the normal way will have a better performance than by the meta-train way in current setting. (baseline will see more images)

In one epoch:
baseline will see: 16 (batch size) * 2400 (dataloader length) = 38400 images
protonet will see: (5+16)(n_support + n_query) * 5 (n_way) * 100 (dataloader length) = 10500 images

During testing (in finetune.py), it seems only concat the trained backbone with a learnable classifier.
Other mechanisms in the module will not affect, right? (e.g. calculate the prototype in PN, relation module in RN).
Or we can modify the code, let other mechanisms also work during testing?

Looking forward to your reply, thank you so much.

Hi @JiaFong

Yes, it is fine.  But you need to submit your paper both to the challenge track and the workshop track.
It is fine to modify the hyper-parameters such as epoch, learning rate related to the training process but not n_query. And the detailed hyper-parameters need to be specified in the paper.
Not very sure about your question. You are free to modify the classification algorithms.

Hi, thank you for your reply.
Sorry about the question 3, I didn't noticed that we can modify the classification algorithm.
Is there any other restriction we need to followed during fine-tune stage?
(e.g. in the current codes, the linear classifier will be re-created when a new task coming. is it allowed to use the same classifier during whole testing? )
Thank you.

Hi @JiaFong , it is not allowed to use the same classifier during whole testing. Each episode during testing should be viewed as a different task. Generally, you are free to modify the classification algorithm, but the evaluation settings (number of query examples, number of support examples, etc.) should stay the same.

Hi @yunhuiguo .
So the backbone、classifier algorithm need to be reset at the beginning of each test episode.
No information can be recorded from the previous task to the new one, right?

@JiaFong That's right. Each trial (or episode) should be an independent task.

Sure, thank you for your explanation.


Fix Year type

Error when i try to build the project:
$ make build
[jam] Build
# github.com/budkin/jam/music
music/lib.go:87: cannot use tracks[i].Year (type uint32) as type int in field value
music/lib.go:128: cannot use tracks[i].Year (type uint32) as type int in field value
make: *** [Makefile:61: build] Error 2
`̀``

Signed-off-by: Nicolas Lamirault<EMAIL_ADDRESS>

Thanks


termbox status bar

closes #487

Coverage decreased (-0.35%) to 62.36% when pulling 57e664642bc34d74a20e54175dc6d9c7d76ec8d6 on zoli:termbox_statusbar into 38504a64643d00166ce21bdd6a72cd879871eb65 on limetext:master.

Coverage decreased (-0.41%) to 62.31% when pulling 236a70accfdb946ef4c57b9349a901350d7d86d7 on zoli:termbox_statusbar into 38504a64643d00166ce21bdd6a72cd879871eb65 on limetext:master.

LGTM, excellent!


This repo is 70MB! Should consider deleting and remaking this repo?

The 70MB is because the vendor folder used to be checked in in this repo.

No need.  Just prune it.
http://stackoverflow.com/questions/9683279/make-the-current-commit-the-only-initial-commit-in-a-git-repository

Pruned.


LongPress crashing iOS in KeyboardExtendedPressable

Hey there, me again. It seems like long pressing KeyboardExtendedPressable is causing iOS to crash. Doesn't happen on Android. Overwriting onLongPress seems to mitigate this issue, but something seems to be wrong there.

Upon further investigation, it seems like iOS is crashing at the following line:
[RNCEKVExternalKeyboardView contextMenuInteraction:configurationForMenuAtLocation:] at node_modules/react-native-external-keyboard/ios/RNCEKVExternalKeyboardView/RNCEKVExternalKeyboardView.mm:197
- (UIContextMenuConfiguration *)contextMenuInteraction:(UIContextMenuInteraction *)interaction configurationForMenuAtLocation:(CGPoint)location  API_AVAILABLE(ios(13.0)){
    self.onContextMenuPress(@{});
    return nil;
}

there, it shows some bad access error while calling self.onContextMenuPress:
Thread 1: EXC_BAD_ACCESS (code=1, address=0x10)

Hello @proohit ,
Sorry for deleting previous message, I just use work account instead of personal.
I will merge and release version a little bit later, probably in some hours.
Thank you for help and PR.
P.S. Sorry for lots of bugs and troubles, but I just don't have an environment for testing. I will try to test it carefully in future.

No worries, thanks for your work in this package, it helps us a lot in displaying keyboard focus :)

Has been merged and fixed with 0.3.6 version.
Thank you for help and contribution.


Update to Node 20

Updates runner to v20 to address https://github.com/SpeedCurve-Metrics/speedcurve-test-action/issues/76
Needed to update @vercel/ncc (a dev dependency) to the latest version to make the install happy on node20. Other changes are the config to specify v20 and the output of running npm run all.

Awesome, thank you! I'll merge this and run some tests on the main branch, and then I'll make a new release.


fix failing changelog action

Not sure why this is failing.
Initial guess is that version  in .version.yaml is not incremented in previous changelog action

By default the version number is supposed to be incremented
Optional skip-version-file: Do not update the version file. Default 'false'.

Still fails, it claims due to branch protection https://github.com/lamalab-org/chem-bench/actions/runs/9675336278/job/26692649416

The action apparently (https://github.com/search?q=repo:TriPSs/conventional-changelog-action+protection&type=issues) has no good options to fix this other than turning off branch protections

I now disabled branch protections. Let's see what happens

we have this branch open  from long back. I wonder if this would have solved the issue

but we would then manually need to do PRs?

- name: Approve Changelog PR
        if: steps.changelog.outputs.skipped == 'false'
        run: |
          gh pr review --approve ${{ env.PR_BRANCH }}
        env:
          GH_TOKEN: ${{ secrets.GH_OWNER_TOKEN }}

      - name: Merge Changelog PR
        if: steps.changelog.outputs.skipped == 'false'
        run: |
          gh pr merge --squash --auto --delete-branch ${{ env.PR_BRANCH }}
        env:
          GH_TOKEN: ${{ secrets.GH_OWNER_TOKEN }}

there is approval and merge as well, but  probably need a GH_OWNER_TOKEN with elevated rights

ok, I can try to add this token

actually it is already there :D

and the push it tried was to main /usr/bin/git push origin main --follow-tags

now still has some token issues that seem related to https://github.com/actions/checkout/issues/417

One option would be to have changelog generation in pull request. and then we anyways merge the PR manually.

Ok - this seems like a good option! Then we could also still edit the changelogs

duplicated in https://github.com/lamalab-org/chem-bench/issues/309


Added SQL authn option

Supports postgres and mysql. Example YAML block is provided below.
sql_auth:
  connection:
    # only postgres and mysql drivers are currently supported
    driver: "postgres"
    # data_source_name is driver specific
    data_source_name: "postgres://postgres:password@localhost:5435/postgres?&sslmode=disable"
  table: "users"
  user_column: "username"
  password_column: "password"
  labels_column: "labels"

This change is

If I'm not mistaken, there are a few vendor dependencies not specified/missing:

github.com/go-sql-driver/mysql
github.com/lib/pq
github.com/lib/pq/oid

Hello!
I really need this feature, how can I help getting this merged?

Thanks for this PR :)
Closing in favour of #280

Thanks for this PR :)
Closing in favour of #280


5.4 kernel is randomly not able to detect the PHY on a BBB industrial

The exact device where this was reported is:
https://www.cnx-software.com/2016/01/28/beaglebone-black-industrial-4g-is-red-and-offers-an-extended-temperature-range/
On 4.14 (previous OS):
May 20 07:32:39 localhost kernel: davinci_mdio 4a101000.mdio: davinci mdio revision 1.6, bus freq 1000000
May 20 07:32:39 localhost kernel: davinci_mdio 4a101000.mdio: detected phy mask fffffffb
May 20 07:32:39 localhost kernel: MDIO: davinci_mdio: dt: updated phy_id[2] from phy_mask[fffffffb]
May 20 07:32:39 localhost kernel: libphy: 4a101000.mdio: probed
May 20 07:32:39 localhost kernel: davinci_mdio 4a101000.mdio: phy[2]: device 4a101000.mdio:02, driver SMSC LAN8710/LAN8720

On 5.4 (new OS):
May 20 07:47:29 localhost kernel: libphy: Fixed MDIO Bus: probed
May 20 07:47:29 localhost kernel: davinci_mdio 4a101000.mdio: davinci mdio revision 1.6, bus freq 1000000
May 20 07:47:29 localhost kernel: mdio_bus 4a101000.mdio: MDIO device at address 0 is missing

We are still gathering details to help pinpoint the versions, but this would probably need a reboot loop test with ethernet check.

Tried to catch the problem on our hardware we have here and we could not reproduce the problem in over a few thousands reboots. So if the customer is able to ship us one of the boards that exhibit the problem then we can better implement the workaround.


Propagate signals to child processes

WHY
WHAT
REF

Codecov Report

Merging #27 into master will decrease coverage by 0.34%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master     #27      +/-   ##
=========================================
- Coverage   22.45%   22.1%   -0.35%
=========================================
  Files          34      34
  Lines        1020    1036      +16
=========================================
  Hits          229     229
- Misses        775     791      +16
  Partials       16      16

Impacted Files
Coverage Δ

pkg/grapicmd/internal/module/command/command.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[BUG] Null value issue if return type is discriminated union

Hi again @jodydonetti,
I'm experiencing a problem with the GetOrSetAsync method and can't figure out what's causing it. I personally don't like throwing exception in my code. That's why I prefer ErrorOr package to be able to use discriminated union. But the problem is that Fusion Cache returns null if I use ErrorOr type. That appears to be a serialization issue, however there is no serialization error log in Fusion Cache. Is there something I'm missing?
Steps to reproduce:

Run the application and call both endpoints ( Memory + Distributed)
Stop the application ( Distributed)
Run the application again and call both endpoints.

The first endpoint loads data from distributed to memory and then returns it successfully. The second one loads data from distributed to memory successfully but returns null.
You may find the related code here to reproduce.
Versions
I've encountered this issue on:

FusionCache version : 0.19.0
.NET version: .NET 7
OS version : Windows 11

Additional context
There is a catch here. The one with the ErrorOr type keeps working until I stop the application for the first time.

Hi @akoken and thanks for using FusionCache!
This is not a FusionCache bug, but a problem with the ErrorOr library and json serialization, or at least with the json System.Text.Json serializer.
TO be more precise, the ErrorOr<int> values seem to be serializable but not deserializable.
See this sample:
using ErrorOr;
using System.Text.Json;

ErrorOr<int> e1 = 42;
ErrorOr<int> e2 = Error.NotFound();

Console.WriteLine($"e1: {e1.Value}/{e1.IsError}");
Console.WriteLine($"e2: {e2.Value}/{e2.IsError}");

var j1 = JsonSerializer.Serialize(e1);
var j2 = JsonSerializer.Serialize(e2);

Console.WriteLine("------");

Console.WriteLine($"j1: {j1}");
Console.WriteLine($"j2: {j2}");

e1 = JsonSerializer.Deserialize<ErrorOr<int>>(j1);
e2 = JsonSerializer.Deserialize<ErrorOr<int>>(j2);

Console.WriteLine("------");

Console.WriteLine($"e1: {e1.Value}/{e1.IsError}");
Console.WriteLine($"e2: {e2.Value}/{e2.IsError}");

The output of this is:
e1: 42/False
e2: 0/True
------
j1: {"IsError":false,"Errors":[{"Code":"ErrorOr.NoErrors","Description":"Error list cannot be retrieved from a successful ErrorOr.","Type":1,"NumericType":1}],"ErrorsOrEmptyList":[],"Value":42,"FirstError":{"Code":"ErrorOr.NoFirstError","Description":"First error cannot be retrieved from a successful ErrorOr.","Type":1,"NumericType":1}}
j2: {"IsError":true,"Errors":[{"Code":"General.NotFound","Description":"A \u0027Not Found\u0027 error has occurred.","Type":4,"NumericType":4}],"ErrorsOrEmptyList":[{"Code":"General.NotFound","Description":"A \u0027Not Found\u0027 error has occurred.","Type":4,"NumericType":4}],"Value":0,"FirstError":{"Code":"General.NotFound","Description":"A \u0027Not Found\u0027 error has occurred.","Type":4,"NumericType":4}}
------
e1: 0/False
e2: 0/False

As you can see, the serialized json seems to be there, but when deserializing it doesn't "come back" correctly.
I assume you can solve this by using a custom converter.
Regarding the fact that FusionCache is not throwing serialization errors: that is because, as you can see in the sample above, the System.Text.Json serializer is not throwing an error, but simply creating an ErrorOr instance with default values (0 and false), so FusionCache cannot possibly know that something went "wrong".
Hope this helps!

Thanks for the quick and detailed reply @jodydonetti.

As you can see, the serialized json seems to be there, but when deserializing it doesn't "come back" correctly.

To be honest, I did not expect  that :)

I assume you can solve this by using a custom converter.

Yes, that could work!

Regarding the fact that FusionCache is not throwing serialization errors: that is because, as you can see in the sample above, the System.Text.Json serializer is not throwing an error, but simply creating an ErrorOr instance with default values (0 and false), so FusionCache cannot possibly know that something went "wrong".

I'm not sure, but record struct can cause it. I'll dig deeper into ErrorOr source code.
Thank you for taking the time!


Fix String#index! documentation

The current documentation re-uses the one from String#index which is evidently wrong. So this commit adapts the documentation for String#index!.

Probably should update the docs for String#rindex! too while we're at it?

Even if they don't call out nil explicitly, the examples are still just wrong given they're not even using the bang version of the method. Can save it for another PR, but I still think it's something that needs to be dealt with.

Let's do that in a separate PR.

@Blacksmoke16 @straight-shoota I'm currently going through the string.cr file (went through the first 1000 lines today)  to fix and update the API documentation as well as to make it more coherent. Will make another pull request with these changes once I'm through.


Removed failing test file

When calling pytest an error would appear due to the file 'test_future_perfect_subjunctive', therefore for the time being it is better to delete such file.

Thankyou very much!


Make RustVec methods public

We can't use these methods from outside of the framework unless they are public.

Thanks!


Move "doclint=all" from Github workflow to pom.xml.

Use maven default goals "clean verify apache-rat:check clirr:check checkstyle:check spotbugs:check javadoc:javadoc" for Github workflows.

Coverage remained the same at 97.981% when pulling fbcc4d674391a94336c551c0a5567cab5de683a6 on PascalSchumacher:github_workflow into 9ba989259f4e3f65dccf9053c8a7ff2e6791a869 on apache:master.


bestObjectiveValue and difference calcuation

Currently the difference gets calculated as
const double difference = bestObjectiveValue_ - objectiveValue;
which, since bestObjectiveValue is infinite at start, will result in difference also being infinite.
So checking for if (difference < 0) { will never find a better bestParameter. Since we presumably want to minimize the objectivevalue the arguments for difference should be switched. This will still return -infinity for the first iteration though, and might be a problem for some algorithms?

This was fixed within the functional branch by b3e779a2f8ea784f55ef77b9dd78f4aa37682787


added transitions and effects to the buttons

#17  done you can check

@GauravKesh bro when i am running the project its showing errors in layout.tsx , and I didn't changed or deleted your logic I only added the tailwind CSS

@devxMani Okay give me some i'll look into the issue

@GauravKesh okk suree


Add dynamic column sizing to wsk activation list command

This addresses https://github.com/apache/incubator-openwhisk-client-go/issues/119 and adds dynamic sizing behavior to the Kind and Status columns, as well as skips the column heading if there are no activation records to display.
With this change, the behavior will be:
wsk activation list --skip 20 --limit 5

Datetime            Activation ID                    Kind     Start Duration   Status  Entity
2019-03-14 14:49:11 847b942103bb4b43bb942103bb7b4376 nodejs:6 warm  2ms        success guest/helloworld:0.0.1
2019-03-14 14:47:48 d5c208b0336c40328208b0336ca03222 nodejs:6 cold  33ms       success guest/helloworld:0.0.1
2019-03-14 09:04:49 d8b4fb863117413ab4fb863117513af5 nodejs:6 cold  31ms       success guest/greeting:0.0.1
2019-03-12 19:28:25 daa12f0d612f4426a12f0d612f8426c4 nodejs:6 cold  129ms      success guest/greeting:0.0.1
2019-03-12 18:20:32 daa5acdcc48947e0a5acdcc489f7e04e nodejs:6 cold  116ms      success guest/greeting:0.0.1

And, then for next pagination we can see how the columns change dynamically (see the Status and Entity columns):
wsk activation list --skip 25 --limit 5

Datetime            Activation ID                    Kind     Start Duration   Status            Entity
2019-02-28 10:08:27 3d578d89fd0f4106978d89fd0f110650 nodejs:6 cold  741ms      success           guest/kafkaFeed:0.0.2
2019-02-27 15:17:05 05b8edb3f5aa40cbb8edb3f5aaf0cbc9 nodejs:6 warm  126ms      success           guest/kafkaFeed:0.0.2
2019-02-27 15:16:16 b18da1f06f4143718da1f06f41937150 nodejs:6 warm  129ms      success           guest/kafkaFeed:0.0.2
2019-02-27 15:14:28 5d8d23b5082d4dba8d23b5082d4dba8a nodejs:6 warm  99ms       application error guest/kafkaFeed:0.0.2
2019-02-27 15:10:34 3d7e4912dc224273be4912dc227273a9 nodejs:6 cold  966ms      success           guest/kafkaFeed:0.0.2

This PR is dependent on https://github.com/apache/incubator-openwhisk-client-go/pull/120, and when that has been merged into client-go, the build.gradle file needs to be updated to the commit hash of that PR.

[x] I have signed a CLA

@larandersson I see this error in Travis
:buildDarwin386# github.com/apache/incubator-openwhisk-cli/commands
commands/activation.go:57:28: undefined: whisk.ActivationFilteredRow
commands/activation.go:107:53: undefined: whisk.ActivationFilteredRow
commands/activation.go:435:19: undefined: whisk.StatusCodes
commands/activation.go:439:71: undefined: whisk.StatusCodes
commands/activation.go:440:17: undefined: whisk.StatusCodes
commands/util.go:215:9: undefined: whisk.ActivationFilteredRow
commands/util.go:216:12: cannot range over collection (type interface {})
 FAILED

@rabbah Did you observe the dependency to https://github.com/apache/incubator-openwhisk-client-go/pull/120? It's not expected to work until that PR has been merged.

Thanks. Just merged it.

@rabbah Updated build.gradle. Now it should work.

@rabbah Updated vendor.json as well.


Docs: link Scour as an immutable helper

Big disclaimer: I'm the author of scour ;)

Wow, you're killing it with these PRs, thanks a ton!


Not able to receive messages from SQS queue unless the message subject matches with model class GetType().Name

Our application wants to receive SES event notifications. These notifications are sent from SES to SNS and then to SQS. We are trying to read messages from the SQS queue.
SES notification message looks like this :
{
"Type" : "Notification",
"MessageId" : "6d3fa15a-f085-52b3-aa59-fb59bd88dce7",
"TopicArn" : "arn:aws:sns:eu-west-1:test:test",
"Subject" : "Amazon SES Email Event Notification",
"Message" : "{"eventType":"Open","mail":.........}
Here the message subject is "Amazon SES Email Event Notification" which is something AWS provides and is not customizable.
We have created the model class with name "AmazonSESEmailEventNotification" .. since the subject and class name are not matching, handler is not picking up the queue messages. But if i manually publish a message with subject same as classname from the queue , handler is able to pick it up and is working fine.
I appreciate any help to solve this issue

Also i see there is a ConstantSubjectProvider which inherits from IMessageSubjectProvider. so is there a way to set it to this one rather than NonGenericMessageSubjectProvider

Found a workaround finally
var busWithLogger = CreateMeABus.WithLogging(_loggerFactory);
busWithLogger.MessageSubjectProvider = new ConstantSubjectProvider("Amazon SES Email Event Notification");
public class ConstantSubjectProvider : IMessageSubjectProvider
{
private readonly string _subject;
        public ConstantSubjectProvider(string subject)
        {
            _subject = subject;
        }

        public string GetSubjectForType(Type messageType)
        {
            return _subject;
        }
    }

Please add description to this property "MessageSubjectProvider" in the documentation

Thanks for the report - we'll incorporate this into our documentation efforts for when we release version 7.


if you re creating cmd terminal why using same looks ?

Description of the new feature/enhancement
look i was here to see if am right place cause i am tired of stupid windows terminal if i start with normal privilage i can use theme background image it looks cool but with administrator privilage it dumb old crap looking terminal
Proposed technical implementation details (optional)
if you guy re creating terminal why in the world you make same looks why even you wasting time to create which is already exist from decades if you wanna re create it go for some stuff that should looks real change good looking and nice UI sci fi type or at least 1 simple theme thats it

I think what you're running into is /dup #10276. When you run a CLI app directly as admin, the platform can't hand it off to the Terminal. However, you can always just start wt directly as admin.


Implement net_version RPC method

net_version method is not implemented and some services like Metamask use it
Ref: https://ethereum.org/en/developers/docs/apis/json-rpc/#net_version

to fix your trouble check this solution click
maybe this will solve your problem.


Can't undo moving an entire path

If you accidentally drag a path around, it doesn't update the command stack and so can't be undone. This is especially annoying when attempting to realign the points from moving it previously.

I removed moving entire paths, so this is no longer relevant.


create fake tasks to fetch update the database with data from rest api with rust celery

https://github.com/rusty-celery/rusty-celery
or use normal celery

rusty-celery doesn't seem to have documentation uptodate. also support for Redis is limited. falling to python celery


Catch all handler?

I'm looking to create a catch all handler, similar to a default behavior in a switch statement, is there a way to do this? Apologies if this is obvious or I'm going about this wrong!
Thanks!

I think you're going about this in an unconventional way.

We return the previous state in the default case. It's important to return the previous state for any unknown action.

https://github.com/reactjs/redux/blob/master/docs/basics/Reducers.md

@yangmillstheory thanks for the reply, just to clarify... how would you replicate something like this with redux-actions, or this is not a valid use case?
  switch (action.type) {
  case 'CREATE_EXERCISE':
    return {
      ...state,
      [action.id]: {
        ...action.exercise
      }
    };
  case 'UPDATE_EXERCISE':
    return {
      ...state,
      [action.id]: {
        ...state[action.id],
        ...action.exercise
      }
    };
  default:
    if (action.entities && action.entities.exercises) {
      return merge({}, state, action.entities.exercises);
    }
    return state;
  }
}```

Thanks!


Fee Percentage Wrong

Regarding TriArb
TriArbTrader
The fee calculation is incorrect it appears.
line 573 ----   fee_percentage = 0.1*3          #divided by 100
pseudo -> fee_percentage  = 10%*3 = 30%
Shouldn't the fee percentage be 0.001 * 3 not 0.1 * 3 ?

Hello - I believe when fee percentage is used, it is either divided by 100 or it compares to a value with a similar order of magnitude.

on line 691, you can see that the calculated fees are multiplied times 100, converting them into percentage. So fee percentage is in percentage,
line 573 ----- fee_percentage = 0.1%*3

Also the fee percentage will be different if you pay with Binance coin as well.


Non-iid

Hi, when I took the experiment in noniid setting, it shows that the path "/dataset_noniid/MNIST" can't find.
I would like to know where is the package or how can I get it.
Wish you all the best~

Hi, thanks for pointing it out.
The non-iid MNIST dataset divides the hand-written digits into 10 categories. It can be downloaded in the following link:
https://drive.google.com/file/d/17D99-OE-LWVL_xd4xkBiLoj0QTJg2U_X/view?usp=sharing
I also update it in the Readme page.
Best wishes


error and uncertainty with GTDB_metadata_filter.R

Sorry I came across another issue I am not sure how to resolve...
When running the R script for GTDB_metadata_filter.R, it threw this error:
Reading in file: https://data.gtdb.ecogenomic.org/releases/release202/202.0/ar122_metadata_r202.tar.gz Error in fread(url, sep = "\t", check.names = TRUE) :   embedded nul in string: 'ar122_metadata_r202.tsv\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\00000664\00002241\00001753\000024736501\014037146135\0015426\0 0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0ustar  \0uqpchaum\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0dataadmin\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0accession' Execution halted
Also, the R.utils package needs to be installed separately in the conda struo2 env, maybe that is expected?
And is there documentation about this particular R filtering script parameters? what are the defaults and what areas should be changed for custom filtering? I'm sorry, like I said I've been having trouble following and understanding the current readme documentation. Thanks a lot for your help.

It's due to a weird character in that line. Maybe changing the comment character for fread would help. I'll have to check

Actually, it's because the file is a tarball. For now, it can be fixed by first downloading the file and uncompressing it. I'll update the code to handle tarballs.
I've added r-r.utils to the conda env yaml.

OK, the R script should now be able to handle tarballs. Reopen if you still have problems.


docs(readme): Refined some wording and inconsistencies to improve clarity and readability.

This pull request aims to enhances the README by refining certain phrases and improving overall readability. Minor wording adjustments were made to ensure clearer communication and easier understanding for new users. These changes aim to make the documentation more accessible and straightforward.

@ghevia01 Thank you so much for your PR! Totally open for more suggestion especially to improve the documentation.

@ixartz You're welcome! Glad I could contribute to the project. I'll definitely continue to review and see where else I can provide support or suggestions. Looking forward to contributing more in the future.


Compile Error after v0.1.31 updated

Describe the bug
Hi Pynecone Team,
I met some errors after v0.1.31 updated, how to solve this? Thanks
- info Using wasm build of next-swc
- warn Attempted to load @next/swc-win32-x64-gnu, but it was not installed
- warn Attempted to load @next/swc-win32-x64-msvc, but an error occurred<EMAIL_ADDRESS>is not a valid Win32 application.
<EMAIL_ADDRESS>- warn Found lockfile missing swc dependencies, patching...
- event compiled client and server successfully in 1749 ms (1132 modules)
- wait compiling...
- event compiled client and server successfully in 181 ms (1132 modules)
- error Failed to patch lockfile, please try uninstalling and reinstalling next in this workspace
TypeError: fetch failed
    at Object.fetch (node:internal/deps/undici/undici:11522:11)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at runNextTicks (node:internal/process/task_queues:64:3)
    at listOnTimeout (node:internal/timers:540:9)
    at process.processTimers (node:internal/timers:514:7)
    at async fetchPkgInfo (C:\Users\15800\Desktop\PeneconeDev\PreEnlight\.web\node_modules\next\dist\lib\patch-incorrect-lockfile.js:65:17)
    at async Promise.all (index 0)
    at async patchIncorrectLockfile (C:\Users\15800\Desktop\PeneconeDev\PreEnlight\.web\node_modules\next\dist\lib\patch-incorrect-lockfile.js:161:26) {
  cause: Error [ERR_SOCKET_CONNECTION_TIMEOUT]: Socket connection timeout
      at new NodeError (node:internal/errors:399:5)
      at internalConnectMultiple (node:net:1099:20)
      at Timeout.internalConnectMultipleTimeout (node:net:1638:3)
      at listOnTimeout (node:internal/timers:575:11)
      at process.processTimers (node:internal/timers:514:7) {
    code: 'ERR_SOCKET_CONNECTION_TIMEOUT'
  }
}
- wait compiling / (client and server)...
- event compiled client and server successfully in 1766 ms (1855 modules)

** Specifics (please complete the following information):**

Python Version: 3.10
Pynecone Version: 0.1.31
OS: win11
Browser (Optional): edge

this looks like a network error. Can you confirm if you still face this?


Add additional info(Metadata and Metrics) for codepush deployment history

Issue: Old code-push cli has install metrics with deployment history command. Appcenter-cli doesn't have metrics in deployment history command.
Solution: Added Metrics for output of deployment history command and made output view as in code-push cli.
Before:

After:

Fix implemented by @yuri-kulikov.

Yeah, we are working on tests right now but it’s just a separate user story

No problem - ping me when you're happy with the tests and I'll take a fresh look.

Hey @owenniblock, thanks for reply!
Is it possible if we’ll add tests in a separate PR and will merge this one if you don’t have any objections?
Otherwise we can add tests in this PR, just worried a little bit if PR size wouldn’t be too large and thus harder to review.
Anyway we’re flexible with whatever discussion you prefer, just let us know what works best for you (await merging this one and push changes to tests directly here or do it in a separate PR)?

@yuri-kulikov - I'd rather have the tests in this PR if that's OK? If I merge this and then the tests highlight some issues we hadn't of thought of - it will make the history a little messy. Having the tests should help the review (since it will show that areas of code behave as we'd expect).
Also if I merge to master we risk releasing before the tests are ready. :)

@owenniblock, thanks, let’s stick with this approach, I’ll ping you when we push changes for tests.

Test data extracted to separate file.


Added Strava authentication

as per https://developers.strava.com/docs/authentication/ it is yes:

approval_prompt string, in query
force or auto, use force to always show the authorization prompt even if the user has already authorized the current application, default is ‘auto’.

I just wanted to test it, it can be removed and would default to the auto setting

Yes. You should remove it.


Volume is too small

Attempting to format the volume results in Volume is too small: https://github.com/rafalh/rust-fatfs/blob/50b8dec36ab664b02d7532422a164e47144eecac/src/boot_sector.rs#L596
Trying to grow the stable memory any further results in an error.
Perhaps there is a bug somewhere which is giving the impression that there isn't enough available space.


Miken/test all hooks

closes https://github.com/airbnb/MvRx/issues/476
Hooks to allow calling a function during the launcher activity test all functionality.  Specific use case is to allow Dropbox to run screenshots every fragment swap

@elihart wasn't sure how to test, let me know if there are integration tests somewhere. I can also add a hook to the sample to at least get api coverage

should be ready to go, I recreated from master again as the rebase gave me trouble


Remove and replace IMTEL logo from project

The old logo can not be used as it contains the NTNU logo.

Was fixed in pull request #489